Skip to content

Multi User환경을 위한 세션별 분리 #42

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
1 task
ehddnr301 opened this issue Apr 7, 2025 · 1 comment · May be fixed by #43
Open
1 task

Multi User환경을 위한 세션별 분리 #42

ehddnr301 opened this issue Apr 7, 2025 · 1 comment · May be fixed by #43
Assignees
@ehddnr301
Labels
enhancement New feature or request
Milestone
Mission: Version ...

Comments

@ehddnr301
Copy link
Collaborator

Why❓
현재 해당 부분처럼 구현되어 있어 여러 유저 사용시 같은 graph를 사용하게 되기때문에 꼬이게 될것으로 보입니다.
https://github.com/CausalInferenceLab/Lang2SQL/blob/07bf11fefa18c131609487c9e1343566b91d2b7e/interface/streamlit_app.py#L32C1-L42C6

How❓

  • session별로 분리되어 사용할 수 있게 session_state 를 사용

What❗️

  • session_state사용으로 각 세션별 분리된 graph를 사용할 수 있도록 작업
@ehddnr301 ehddnr301 added the enhancement New feature or request label Apr 7, 2025
@ehddnr301 ehddnr301 self-assigned this Apr 7, 2025
@ehddnr301 ehddnr301 linked a pull request Apr 7, 2025 that will close this issue
42 Multi User환경을 위한 세션별 분리 #43
Open
@ehddnr301 ehddnr301 linked a pull request Apr 7, 2025 that will close this issue
Open
@ParkGyeongTae
Copy link
Contributor

로그인을 안하는 환경을 원하는 요구사항이 있을 수 있으니 먼저 이 방식으로 대응하는것이 나쁘지 않다고 봅니당!
추후에 하나씩 업그레이드하면 좋을 것 같아요~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ehddnr301 ehddnr301

Labels
enhancement New feature or request
Projects
None yet
Milestone
Mission: Version 1.0.0 Complete ✅
2 participants
@ehddnr301 @ParkGyeongTae