Skip to content

Radis:Integrate a Modern Lazy-Loading Alternative for Large-Scale Spectroscopic Database Processing (Zeyad Zahran)#17

Merged
nabobalis merged 2 commits into
OpenAstronomy:mainfrom
Zeyadzahran:zeyad-zahran
May 14, 2026
Merged

Radis:Integrate a Modern Lazy-Loading Alternative for Large-Scale Spectroscopic Database Processing (Zeyad Zahran)#17
nabobalis merged 2 commits into
OpenAstronomy:mainfrom
Zeyadzahran:zeyad-zahran

Conversation

@Zeyadzahran
Copy link
Copy Markdown
Contributor

@Zeyadzahran Zeyadzahran requested a review from a team as a code owner March 22, 2026 03:58
@github-actions github-actions Bot added the radis label Mar 22, 2026
@Zeyadzahran
Copy link
Copy Markdown
Contributor Author

I aleady upload the file as a PDF. Should I upload a markdown file

@nabobalis
Copy link
Copy Markdown
Member

I guess that is fine. It does make review difficult however.

@Zeyadzahran
Copy link
Copy Markdown
Contributor Author

I guess that is fine. It does make review difficult however.

Thanks for letting me know! I tried converting it to md, but unfortunately, the formatting broke and the images were lost.
​To make the review process easier for you without losing the current layout, would a Google Docs link with commenting access work?
​If an .md file is strictly preferred, I can definitely take the time to manually reformat it and upload it. Let me know what works best for you

@nabobalis
Copy link
Copy Markdown
Member

That is for the mentors to decide.

@prathamhole14
Copy link
Copy Markdown
Member

Pdf works as well.

Copy link
Copy Markdown

@minouHub minouHub left a comment

Choose a reason for hiding this comment

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

Zeyadzahran Interacted with the RADIS mentors and submitted several PR to the organization. Application approved 👍

@nabobalis nabobalis merged commit 428a01f into OpenAstronomy:main May 14, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants