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

Issue with "no directory" when running s21 #10

Closed
mar-palt opened this issue Sep 22, 2023 · 2 comments · Fixed by #12
Closed

Issue with "no directory" when running s21 #10

mar-palt opened this issue Sep 22, 2023 · 2 comments · Fixed by #12
Labels
bug Something isn't working

Comments

@mar-palt
Copy link

Hello! I'm just trying to run the pipeline with the example dateset for GJ 1214b, and I'm stuck on stage 21. I haven't had any issues when running the previous stages, but when I try to run s21, I get the error message: NotADirectoryError: [Errno 20] Not a directory: './run_2023-09-20_12-23-36_GJ1214_Hubble13021//extracted_lc/.DS_Store/lc_spec.txt'
I haven't changed anything in the src code, so I'm not sure why this issue is coming up and how to resolve it.
Thanks!

@sebastian-zieba
Copy link
Owner

Hi @mar-palt
Thanks for raising the issue! This is indeed a bug that I wasn't aware of which I think only occurs on MACs. I will work on this in the next few days!
The basic problem is that PACMAN does not expect any hidden files or directories in the meta.workdir + "/extracted_lc/" directory. The solution would be to either exclude any files starting with a "." (so hidden files) or only include files that start with a "2". I think I will implement the latter because the directories will always start with a "2" as they are in the typical '%Y-%m-%d_%H-%M-%S' format. And current years (%Y) start with a "2".

A new bugfix will be needed in the year 3000 then :)

I'll keep you updated.

Best,
Sebastian

@sebastian-zieba sebastian-zieba added the bug Something isn't working label Sep 25, 2023
gapp-c added a commit to gapp-c/PACMAN that referenced this issue Sep 27, 2023
@sebastian-zieba sebastian-zieba linked a pull request Sep 27, 2023 that will close this issue
sebastian-zieba added a commit that referenced this issue Sep 27, 2023
@sebastian-zieba
Copy link
Owner

Hi @mar-palt
This issue should be fixed now with the latest version of the code on master.

@MBSck MBSck mentioned this issue Nov 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants