Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Ported (exceptions.jl) from Stingray (Python) to Julia #14

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JawadAhmadCS
Copy link
Contributor

@JawadAhmadCS JawadAhmadCS commented Mar 12, 2025

Changes Made:

  • Added exceptions.jl in src/ directory.
  • Implemented Base.showerror for better error message formatting.
  • Updated test/runtests.jl to include exception handling test cases.
  • Ensured all tests pass successfully.

✅ Test Results:

image

image

This contribution is part of my efforts for Google Summer of Code (GSoC) 2025 to work on Spectral Timing in Julia. Contributing to Stingray.jl helps me gain codebase familiarity and deepen my understanding for meaningful GSoC contributions.

@JawadAhmadCS JawadAhmadCS changed the title Ported Exception Handling from Stingray (Python) to Julia Ported (exceptions.jl) from Stingray (Python) to Julia Mar 12, 2025
@JawadAhmadCS
Copy link
Contributor Author

Hey @matteobachetti , I have ported the exception handling from Stingray (Python) to Julia. I added Exceptions.jl, defined StingrayError, and implemented Base.showerror. All tests are passing successfully.

Looking forward to your review and feedback!

Thanks!

@JawadAhmadCS
Copy link
Contributor Author

Everything looks good on my end, and all tests are passing. Please review and let me know if there's anything that needs to be changed or improved before merging. Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant