-
Notifications
You must be signed in to change notification settings - Fork 66
Open
Conversation
New stuff includes: - Variable vector dirichlet boundary conditions, `CurvilinearDirichletCondition`, driven by uw.fn. - A system wide rotation matrix that can be used for stokes or rotating a vector mesh variable - Meshes with non cartesian geometries. New examples under, docs/examples/curvi_examples.
...ode/underworld2 into jgiordani/curvi-bits
jmansour
commented
Apr 22, 2021
Contributor
Lets schedule a chat for next week to walk over this.
julesghub
commented
Jul 6, 2021
Member
Author
See examples/curvi_examples for models in annulus geometry.
To be merged into master release once the user interface is agreed upon by development team. @julesghub + @jmansour.
* Fix issue 537 - vrms typo * Making sure the factor is an appropriate numpy type * Updates for docker image generation: (#550) * Switch deprecated `MAINTAINER` tag for new `LABEL` construct. * Set `/opt` to ugo+rwx to allow users to install new pip packages. * Remove `tini` which no longer appears to be required (and wasn't being used previously in any case). * Remove extras for notebook MPI usage, as this wasn't being used and is easy to reinstall. * Update UW image to point to newest base file image. * Update to CHANGES.md * Renaming guidelines.md -> development_guidelines.md * * Fixes h5 file save mode (underworldcode/underworld2/#533) * Minor documentation updates. * Minor redundant code removal. * Adding a pull request template! * Update `CHANGES.md` * Update to fix broken ReadTheDocs generation. * * Import scaling into uw. (#554) * Fix docstrings & doctests. * Rename pull request template as Github doesn't appear to register it with upper case naming. * Fix test in config which Py3.8 is complaining about. * * Added a check to ensure that the selected solver is available at runtime. (#553) The previous behaviour resulted in solves continuing unaware of solve difficulties, and failing elsewhere, or worse. * Added test for the solver check. * Solver check (#555) * * Added a check to ensure that the selected solver is available at runtime. The previous behaviour resulted in solves continuing unaware of solve difficulties, and failing elsewhere, or worse. * Added test for the solver check. * I've updated these models to run with "mumps" if available, and fallback to the default solver otherwise. * Using a relative path to reload swarm files * * For sequential IO operations, need to ensure that non-root procs open file with `append` mode instead of `write` mode (otherwise a new file is created by each proc, effectively deleting all data previously written). * Update mesh file mode to `write` mode. * Making test_long.sh executable * Adding a EBA example to the tests for now. Documentation will come next release when it's moved into examples. The 2 extra terms in the heat equation, viscous dissipation and adiabatic heating, should have equal volume integrals in the model, as per King et al. * Update CHANGES * github actions docker images * * Documentation, version numbers, and docker file updates. (#559) * File creation mode should be "w" here too. Interesting, "append" mode causes a test failure in the Docker container, but not baremetal Ubuntu. * Update docker_build_push.yml * Update docker_build_push.yml * Update docker_build_push.yml * Adding symlink to `LICENSE.md` for conda. Co-authored-by: John Mansour <mansourjohn@gmail.com>
rbeucher
commented
Nov 25, 2021
Contributor
@julesghub do we want to merge this?
rbeucher
force-pushed
the
development
branch
4 times, most recently
from
May 24, 2022 08:26
6ec398d to
2935e5f
Compare
rbeucher
commented
May 31, 2022
Contributor
What do we want to do with this. It's been a while and we have not discussed it.
julesghub
commented
May 31, 2022
Member
Author
I think we should aim to merge this code in. The interface isn't ideal but the functionality works and is part of Rohan and Thyagi's modeling.
Happy to release 2.13 first and then add this functionality in thereafter.
rbeucher
commented
May 31, 2022
Contributor
Yes, I think we release 2.13 and then we merged into dev.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The additions for curvilinear mesh and solving.
This PR includes code changes and examples.
Significant code changes in Python and C layer:
Examples:
They include simple regression testing.