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 imodelrescale, issue #142 #143

Closed
wants to merge 31 commits into from
Closed

Fix imodelrescale, issue #142 #143

wants to merge 31 commits into from

Conversation

ErikOsinga
Copy link
Contributor

Fixes #142

ErikOsinga and others added 30 commits July 17, 2024 14:09
… function for trimming headers, but its not working
@ErikOsinga
Copy link
Contributor Author

Argh, another branch issue, I thought this would only have the last commit 2709ffc ... my bad..

@Cameron-Van-Eck
Copy link
Collaborator

The fix looks good to me, but this PR has all the other commits/changes attached, so it's going to make a bit of a mess of things when I try to merge everything in. Is it possible to start over with a PR that only has the changes for this issue (I'm not sure if it can be changed without deleting this PR and making a new one)?

This is kind of on me for not keeping up on merging in PRs. I'll put it on my to-do list for sometime in the next few weeks to go through and merge in everything that's ready.

@ErikOsinga
Copy link
Contributor Author

See #144

@ErikOsinga ErikOsinga closed this Aug 29, 2024
@Cameron-Van-Eck Cameron-Van-Eck deleted the fixImodelrescale branch December 5, 2024 23:47
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.

Too many coeffs are written in rescale_I_model_3D.py
2 participants