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(idm): stress package loaders allocate duplicate iper/ionper variables #1732

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

mjreno
Copy link
Contributor

@mjreno mjreno commented Apr 16, 2024

Issue introduced in PR #1610.

  • This PR moved iper/ionper to dynamic loader base class
  • There are 2 contexts (loaders) for stress data- an input style specific block loader (e.g. list, array) and the generic dynamic loader that manages the load and allocates the input style loader. These were both allocating iper/ionper via the base class.
  • Fix is to move iper/ionper back into the higher level dynamic loader.

@mjreno mjreno changed the title fix(idm): stress package loaders allocated duplicate iper/ionper variables fix(idm): stress package loaders allocate duplicate iper/ionper variables Apr 16, 2024
@mjr-deltares
Copy link
Contributor

Hi @Manangka this is related to the problem you find in your work with #1701

@langevin-usgs langevin-usgs merged commit f7a2bd7 into MODFLOW-USGS:develop Apr 18, 2024
17 checks passed
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.

3 participants