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

Fix directory inside of SDSS Montage Chart to use user folder instead of static folder #13

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

KoT17
Copy link
Collaborator

@KoT17 KoT17 commented May 8, 2024

There may be a few references to inaccessible folders inside of the SciServer and the github. This is one that I saw was referenced but I still need to test on the SciServer to make sure this notebook works as expected.

@KoT17 KoT17 force-pushed the cleanup/remove-temp-workdir branch from 0d97224 to 4612ba3 Compare May 8, 2024 04:12
@KoT17 KoT17 force-pushed the cleanup/remove-temp-workdir branch from 4612ba3 to bf2255b Compare May 9, 2024 19:11
@KoT17 KoT17 self-assigned this May 14, 2024
@KoT17
Copy link
Collaborator Author

KoT17 commented May 14, 2024

Wasn't able to get the import step in this notebook to work so attempting again inside of the container. I'll probably build locally if my container doesn't import today.

@KoT17
Copy link
Collaborator Author

KoT17 commented May 21, 2024

Found the issue with my import step. It was due to running the notebook in the read only getting_started folder. I tested notebook and it gets past the initial set up. There needs to be a deeper investigation since I don't think the second half of the notebook is creating files/accessing sdss correctly.

Proposed changes inside of SciServer:
chrome_dkbRPXs1Jw

chrome_dxZ4s0NXIe

Error handling:
chrome_3lQrLHLMFH

@KoT17 KoT17 requested a review from raddick May 21, 2024 20:57
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 this pull request may close these issues.

None yet

1 participant