Skip to content

Issue #1176 assign wells with point filters #1186

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

Conversation

JoerivanEngelen
Copy link
Contributor

@JoerivanEngelen JoerivanEngelen commented Sep 4, 2024

Fixes #1176

Description

  • Assign wells with point filters (filters with a zero length) to layers. Before, these would be removed in the compute_overlap method.
  • Add test case for wells with a point filter
  • Set validation so that screen_top can equal screen_bottom
  • Rename vague variable name "df" to "df_factor"

Checklist

  • Links to correct issue
  • Update changelog, if changes affect users
  • PR title starts with Issue #nr, e.g. Issue #737
  • Unit tests were added
  • If feature added: Added/extended example

Manangka and others added 2 commits September 5, 2024 14:10
Fixes # 1179

# Description
<!---
Thanks for opening a PR!

Please add your description here of changes made and how they are going
to
resolve the linked issue 
-->

# Checklist
<!---
Before requesting review, please go through this checklist:
-->

- [ ] Links to correct issue
- [ ] Update changelog, if changes affect users
- [ ] PR title starts with ``Issue #nr``, e.g. ``Issue #737``
- [ ] Unit tests were added
- [ ] **If feature added**: Added/extended example
@JoerivanEngelen JoerivanEngelen self-assigned this Sep 6, 2024
@JoerivanEngelen
Copy link
Contributor Author

JoerivanEngelen commented Sep 6, 2024

The request for type hints lead to some extra required type annotations to be implemented for MyPy. A no-regret, with the slight downside that this changeset now influences more places of the code.

JoerivanEngelen and others added 9 commits September 6, 2024 18:18
…1191)

Fixes #1190 

# Description
The TeamCity buildserver has some quirks making it sometimes hard to
reproduce failing tests/build steps locally.
In this PR this issue addressed by running the pipeline scripts in
docker containers.
These docker containers can be run locally as well, making it possible
for the developer to reproduce the failing build on their own machine.

# Checklist
<!---
Before requesting review, please go through this checklist:
-->

- [x] Links to correct issue
- [ ] Update changelog, if changes affect users
- [ ] PR title starts with ``Issue #nr``, e.g. ``Issue #737``
- [ ] Unit tests were added
- [ ] **If feature added**: Added/extended example
Copy link

sonarqubecloud bot commented Sep 9, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
10.8% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@JoerivanEngelen JoerivanEngelen merged commit fa2fcfc into imod5_converter_feature_branch Sep 9, 2024
5 of 6 checks passed
@JoerivanEngelen JoerivanEngelen deleted the issue_#1176_assign_wells_zero_filter_length branch September 9, 2024 13:37
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.

2 participants