Skip to content

Cleanup and additional debug check in parcel nearest#466

Merged
matt-frey merged 2 commits into3d-mpi-modelfrom
parcel-nearest-cleanup
May 3, 2023
Merged

Cleanup and additional debug check in parcel nearest#466
matt-frey merged 2 commits into3d-mpi-modelfrom
parcel-nearest-cleanup

Conversation

@matt-frey
Copy link
Member

@matt-frey matt-frey commented May 2, 2023

This PR

  • removes print statements used for debugging
  • adds an additional check in debug mode

@matt-frey matt-frey added enhancement / optimisation Improve algorithm or behaviour cleanup Cleanup the code 3D Issues and pull requests related to the 3D version MPI MPI related issues and pull requests labels May 2, 2023
@matt-frey matt-frey added this to the 0.13.0 milestone May 2, 2023
@matt-frey matt-frey self-assigned this May 2, 2023
@matt-frey matt-frey changed the title Cleanup and improvement of parcel nearest Cleanup and enhancement of parcel nearest May 2, 2023
@matt-frey matt-frey changed the base branch from main to 3d-mpi-model May 2, 2023 12:51
@matt-frey matt-frey changed the title Cleanup and enhancement of parcel nearest Cleanup and additional debug check in parcel nearest May 3, 2023
@matt-frey matt-frey marked this pull request as ready for review May 3, 2023 15:25
@matt-frey matt-frey requested a review from sjboeing May 3, 2023 15:25
Copy link
Collaborator

@sjboeing sjboeing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@matt-frey matt-frey merged commit f40c45f into 3d-mpi-model May 3, 2023
@matt-frey matt-frey deleted the parcel-nearest-cleanup branch May 3, 2023 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3D Issues and pull requests related to the 3D version cleanup Cleanup the code enhancement / optimisation Improve algorithm or behaviour MPI MPI related issues and pull requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants