Skip to content
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.

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

call on module error in dj_config.py #658

Closed
andre-metzger opened this issue Oct 17, 2023 · 0 comments · Fixed by #662
Closed

call on module error in dj_config.py #658

andre-metzger opened this issue Oct 17, 2023 · 0 comments · Fixed by #662
Assignees

Comments

@andre-metzger
Copy link

Describe the bug
A clear and concise description of what the bug is.

Line 71 in dj_config.py has if os.path(filename).exists() which makes a call on the os.path module and will return in an error.
I changed my local file to be 'if os.path.exists(filename)' which then worked and is in accordance with the os.path documentation

Trying to set up a new dj_config file resulted in the error when following the guide set in notebook: 00_Setup.ipynb when trying to do python config/dj_config.py ametzger for me.

Should connect to dj and then ask for password which it did when I made the changes above

@edeno edeno mentioned this issue Oct 18, 2023
2 tasks
CBroz1 added a commit to CBroz1/spyglass that referenced this issue Oct 19, 2023
edeno pushed a commit that referenced this issue Oct 19, 2023
* Contrib doc edits

* #531

* #532: PR Template

* Fix numbering

* Typo

* #658 config. Start to phase out config dir

* Update changelog

* Add docstrings. Separate sql content from funcs
edeno pushed a commit that referenced this issue Nov 8, 2023
* Contrib doc edits

* #531

* #532: PR Template

* Fix numbering

* Typo

* #658 config. Start to phase out config dir

* Update changelog

* Add docstrings. Separate sql content from funcs

* #630 - Reduce varchar approach

* More varchar edits

* Fix typo. Note all Reduce Varchar changes

* Fix linting

* Blackify

* Update changelog, citation

* typo

* Remove unwanted import
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants