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 TessTargetFile on windows #171

Merged
merged 2 commits into from
Sep 29, 2023

Conversation

mwcraig
Copy link
Contributor

@mwcraig mwcraig commented Sep 29, 2023

One of the comparison notebook fails on Windows because temporary files are handled somewhat differently there. This PR adds a test to (hopefully) reproduce the problem. Once I'm sure the test is checking what I want I'll push the fix.

@mwcraig mwcraig added the refactor Summer 2023 project to rewrite stellarphot label Sep 29, 2023
@mwcraig
Copy link
Contributor Author

mwcraig commented Sep 29, 2023

Self-merging since the fix clearly works....no changelog needed I think

@mwcraig mwcraig merged commit 32d6825 into feder-observatory:main Sep 29, 2023
@mwcraig mwcraig deleted the fix-windows-temp branch February 8, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Summer 2023 project to rewrite stellarphot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant