-
Notifications
You must be signed in to change notification settings - Fork 139
Todays Sun, PR for branch: feature/WSA #3652
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
Conversation
…into feature/WSA
…te dynamic downloader
…load, putOnQ function is needed next.
…nce.cpp Co-authored-by: Emma Broman <emma.broman@liu.se>
Co-authored-by: Emma Broman <emma.broman@liu.se>
Co-authored-by: Emma Broman <emma.broman@liu.se>
Co-authored-by: Emma Broman <emma.broman@liu.se>
Co-authored-by: Emma Broman <emma.broman@liu.se>
Co-authored-by: Emma Broman <emma.broman@liu.se>
Co-authored-by: Emma Broman <emma.broman@liu.se>
Co-authored-by: Emma Broman <emma.broman@liu.se>
…into feature/WSA
Thanks all for all comments! Ylva asked about the GUI issue I've raised concerns about . I find that to not be an issue anymore, I think mainly since I redid the onChange function declarations and addProperty calls in the right order, making sure onChange was declared before adding property. The scale on the fieldliens: I need to scale them up with the size of the Sun. Is there a different place I should do that instead, or do I need to implement a property? Naming: We landed on Today's Sun. As the rest of the profile have good one word names, it was difficult to come up with one that would describe the general whole purpose with it. I don't think it's a good idea to add more words to the naming of the profile. Speaking of wordy. Mission panel info: Ylva has good points about simplifying the language. At the same time, I've written it in a way where I'm saying what content is there and how to use it. Ylva makes a great point about instead focusing on what the content is telling us " What are the fieldlines? What do they show? What do the colors mean? What do the colors on the sun mean? Why should we care about these things? How do they affect us? Etc. " To add this but at the same time keep it short is tricky and I'd appreciate help with it. Also on the topic of the mission panel. Using the mile stones. I've thought about this too and I've wanted to add periods where there were CMEs but I recently learned that WSA is not modeling the CMEs but only the ambient background. I also have been wanting to put the end time to be todays date every day, but I believe in LUA we've blocked the use of the system functions for security reasons, which unfortunately is where getting the time lives. If anyone have a work around on that I'd love to add it. talk about wordy... |
I think the scale of the fieldlines is fine, it's just the max value that seems very low compared the scale they actually have. So maybe change the max value of the property, if possible? If you want help to come up with some text for the mission panel, I'm happy to chip in! Chat gpt could also be a good tool for rephrasing things. Here are the videos that became 404: 2025-06-10.13-39-13.mp4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adressing the comments and for the detailed response on the comments @ElonOlsson ! This is starting to look good enough to merge to me, especially with the time limit.
In addition to addressing the duplicated code, some documentation should be added for the different renderables. However, this could also be done in a separate PR after this one is merged :)
I agree with Ylva about the wordiness of the mission description - and this is probably the most critical thing to fix before the release.
Doing as you suggested, and move the details on how to use the profile to the documentation page, would solve that problem. Then, the description in the mission could you be a shorter explanation that broadly answers the questions that Ylva so nicely formulated.
You asked for help in doing this, and I'm happy to help too. And as Ylva said, ChatGPT could probably help. But then you still need to provide the answers to the questions - since you're the expert 😉 Listing them again here for convenience
What are the fieldlines? What do they show? What do the colors mean? What do the colors on the sun mean? Why are there two different sizes of the sun? How are these things measured? Why should we care about these things? How do they affect us?
modules/fieldlinessequence/rendering/renderablefieldlinessequence.cpp
Outdated
Show resolved
Hide resolved
modules/fieldlinessequence/rendering/renderablefieldlinessequence.cpp
Outdated
Show resolved
Hide resolved
modules/fieldlinessequence/rendering/renderablefieldlinessequence.cpp
Outdated
Show resolved
Hide resolved
modules/fieldlinessequence/rendering/renderablefieldlinessequence.cpp
Outdated
Show resolved
Hide resolved
@ElonOlsson : To get the current time you can use |
data/assets/scene/solarsystem/heliosphere/2012/sun_earth_2012_fieldlines_pfss.asset
Show resolved
Hide resolved
data/assets/scene/solarsystem/heliosphere/bastille_day/fieldlines.asset
Outdated
Show resolved
Hide resolved
data/assets/scene/solarsystem/heliosphere/todayssun/fieldlines.asset
Outdated
Show resolved
Hide resolved
…nce.cpp Co-authored-by: Emma Broman <emma.broman@liu.se>
…nce.cpp Co-authored-by: Emma Broman <emma.broman@liu.se>
…nce.cpp Co-authored-by: Emma Broman <emma.broman@liu.se>
…into feature/WSA
This includes a lot, but the over all intention of this is to visualize the very latest simulation output from continuously running models to show the Sun as it is right now. Right now only one model WSA, in the future maybe ENLIL and maybe more.

This is a long time coming PR where I've implemented a Dynamic File Sequence Downloader, that for now, when given URL to CCMC server for a dataset where simulation outputs continuously get placed, this will keep track on the time in OpenSpace and only download relevant files accordingly.
This have been integrated with RenderableFieldlinesSequence which has been rewritten and RenderableTimeVaryingSphere which have gotten a lot of edits.
We are looking to get this as a new profile for the release, so with the PR also comes a profile file ready to be tested. You should be able to run the branch with the profile and see it right away.
I'm looking for eyes especially on :
naming of the profile,
naming for the folder for the assets for the profile,
the issue explained in the ui-rewrite slack channel,
the general explanation in the information panel (mission panel),
but also bugs and over all coding practices and probably more I'm forgetting right now.