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

Add description for book of abstracts #41

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Conversation

rowanc1
Copy link
Member

@rowanc1 rowanc1 commented Jan 31, 2024

Feel free to move this file around, move the abstract into the readme, or change the formatting. Just hoping that the content will be expanded a bit!

This will power the abstract here:
https://agrogeo24.curve.space/articles/AG0030

@santisoler
Copy link
Member

Thanks for opening this PR @rowanc1! I'm not sure if I understood correctly what I can or can't do with the file. If I rename it, would the curve.space website build?

I would like to add an extended abstract to the README.md. Should we have it duplicated (one in README.md and one in index.md?

@rowanc1
Copy link
Member Author

rowanc1 commented Jan 31, 2024

If you would like to put it in the readme in this repo, that is fine! We will fork the repo and change the formatting a bit over here:
curvenote-submit#1

The published version will be built from that source and will require a few tweaks so I am waiting until the content is completed to do that.

@santisoler
Copy link
Member

I updated the abstract to the new extended version you can find in the README.md. I've also improved the list of prerequisites. Let me know what do you think. If you like it we can merge it (feel free to merge it if you have the privileges).

Copy link
Member

@lheagy lheagy left a comment

Choose a reason for hiding this comment

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

I added just a bit of intro and a statement about the concepts being transferrable to other methods. Please see what you think!

@rowanc1
Copy link
Member Author

rowanc1 commented Feb 8, 2024

Love the changes @lheagy -- thanks for adding them!!

@rowanc1
Copy link
Member Author

rowanc1 commented Feb 8, 2024

The image is duplicated here:
https://github.com/simpeg/agrogeo24/blob/main/images/dcr_inv.png

I can take this over the line @lheagy -- thanks for your help!

By the end of the tutorial, you'll be able to use SimPEG to invert your own
All of the simulation types that are available within SimPEG (gravity, magnetics, electromagnetics, etc.) are built with the same common framework, meaning many of the concepts illustrated in this tutorial are transferrable to other data types.

![dc-resistivity-inversion](images/dc-inversion-image.png)
Copy link
Member Author

Choose a reason for hiding this comment

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

It would be better to move this to the bottom of the file I think, I will clean it up!

@santisoler
Copy link
Member

Hi @rowanc1 and @lheagy! Is there anything I can do to see this PR merged?

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.

3 participants