-
Notifications
You must be signed in to change notification settings - Fork 0
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
(draft) En analysis #35
base: main
Are you sure you want to change the base?
Conversation
@justaddcoffee is my understanding correct that this is only adding data files and not any of the code we fear might have some bugs? If so, feel free to merge :) (Sorry, by accident I closed and reopened the PR) |
Sorry but why is this a pull request into main? Shouln't short letter be an independent branch, maybe a fork? @justaddcoffee |
It is actually a branch, but yes, this PR should not be merged into main (and maybe should not be a PR at all). When it is polished we probably should put it in a separate fork or repo |
… for short letter
…d by Exomiser and o1
…omiser_o1_4o.ipynb for clarity
Update plot_exomiser_o1MINI_o1PREVIEW_4o.ipynb
No description provided.