Skip to content

Conversation

@alarshi
Copy link
Contributor

@alarshi alarshi commented May 9, 2023

I am pushing this branch which we we have been using for our global mantle flow models. This PR is only to get the relevant commits from this branch merged into current main.

@gassmoeller : The commits using the rounded corners and interpolation between two very closely spaced faults are : 3e88817
d18d1c4
e33e739

@MFraters
Copy link
Member

Hey @alarshi,

Thanks that you are willing to contribute this work back to the world builder! I can take a closer look at the code when I get back from traveling. Could you rebase in the meantime and squash your commits down a bit? Preferably in logical blocks. Thanks!

@tjhei
Copy link
Contributor

tjhei commented Jun 23, 2023

It would be nice if you could rebase to the latest GWB main version as it is quite difficult to configure this current version with the current ASPECT master.

@alarshi
Copy link
Contributor Author

alarshi commented Jun 24, 2023

@tjhei : Yes, I agree that this version is not compatible with the main ASPECT and I have been working around this with the old CMakeLists. However, I was not able to rebase this version on the main WB either because a lot of the core code has changed/restructured on which the commits of this branch were based of. I discussed with @MFraters on leaving this branch here for reference and getting the functionality in WB as separate PRs. I still need to figure out the differences between the main and this branch results for our models. This is one of my goals for this hackathon.

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.

5 participants