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

issue-250-ridge-transforms #298

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Hojat-Shirmard
Copy link

Search for "hjt" in the file; you'll see I've added lines of code in four different places.

Search for "hjt" in the file; you'll see I've added lines of code in four different places.
@michaelchin michaelchin marked this pull request as draft October 24, 2024 03:19
@michaelchin michaelchin changed the title First commit issue-250-ridge-transforms Oct 24, 2024
@michaelchin
Copy link
Contributor

@Hojat-Shirmard

moved the two functions to the end of the file and fixed the indentation.
added"ridge_transforms" as property(return self.ridges + self.transforms), similar to this https://github.com/GPlates/gplately/blob/bea66aaff046fa7d08e33c94c41b63695f8ecbe9/gplately/plot.py#L383-L387
@Hojat-Shirmard
Copy link
Author

Hi @michaelchin
I made a second commit considering your requested changes. To test the code, should I add plot.plot.ridge_transforms to the code you shared and run it in the Jupyter notebook?

@michaelchin
Copy link
Contributor

Hi @michaelchin I made a second commit considering your requested changes. To test the code, should I add plot.plot.ridge_transforms to the code you shared and run it in the Jupyter notebook?

yes, add plot_ridge_transform() funciton call and comment out other plotting code to see the result more clearly.

And you can see your code changes here https://github.com/GPlates/gplately/pull/298/files. there are still errors in the code changes.

@Hojat-Shirmard
Copy link
Author

How can I know where the errors are?

@michaelchin
Copy link
Contributor

How can I know where the errors are?

Have you tried to click this link https://github.com/GPlates/gplately/pull/298/files?

if you did, see the screenshot below for example.

You need to make sure the changes are intended. And only make changes when it is necessary.

Screenshot 2024-11-02 at 3 56 22 PM

@Hojat-Shirmard
Copy link
Author

I reckon I need to create a new environment in Anaconda and install my modified branch to check if it’s working properly, right?
I've also passed some courses on YouTube and learned a bit about Docker, so if there are any instructions or steps to follow with Docker, I’d be happy to try that as well.

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