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

Wq add anihfb #689

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Wq add anihfb #689

wants to merge 13 commits into from

Conversation

Manangka
Copy link
Collaborator

In GitLab by @jdelsman on Nov 17, 2022, 22:34

Add support for imod-wq ANI and HFB packages

  • write anifile with relative paths to runfile
  • write associated .arr files
  • write HFB file with relative path in runfile
  • tests
  • linting

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Nov 17, 2022, 22:34

requested review from @Huite

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Nov 17, 2022, 22:35

added 8 commits

Compare with previous version

@Manangka
Copy link
Collaborator Author

In GitLab by @JoerivanEngelen on Nov 29, 2022, 16:07

On the linting: I see the linting crashes on isort.

You can install isort in your conda environment with pip install isort, then run isort in your terminal.

Alternatively, you can also run tox -e format locally, which does the same (requires tox installed in your base environment)

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Nov 29, 2022, 16:57

added 6 commits

Compare with previous version

@Manangka
Copy link
Collaborator Author

In GitLab by @JoerivanEngelen on Nov 29, 2022, 17:45

added 2 commits

  • 03613a0 - 1 commit from branch master
  • ed4a922 - Merge branch 'master' into wq_add_anihfb

Compare with previous version

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Nov 30, 2022, 09:48

added 1 commit

Compare with previous version

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Nov 30, 2022, 09:57

marked this merge request as ready

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Nov 30, 2022, 10:04

marked this merge request as draft

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Dec 1, 2022, 09:16

@Huite , @JoerivanEngelen : with regards to testing, any ideas on that? Because of the imod-wq implementation, the runfile refers to a separate ani (or hfb) file. That is easily tested of course, but how to test the generated ani (of hfb) file?

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Dec 2, 2022, 13:33

added 1 commit

Compare with previous version

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Dec 2, 2022, 15:50

marked the checklist item tests as completed

@Manangka
Copy link
Collaborator Author

In GitLab by @jdelsman on Dec 2, 2022, 15:50

marked this merge request as ready

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

2 participants