Skip to content

Conversation

@niksirbi
Copy link
Member

@niksirbi niksirbi commented Dec 2, 2025

Description

What is this PR

  • Bug fix
  • Addition of a new feature
  • Other: documentation

Why is this PR needed?

Closes #83

What does this PR do?

Updates the project tree images to not include optional fields, like date and id.

References

I will open a similar PR on datashuttle as well.

How has this PR been tested?

Local docs build.

Is this a breaking change?

No.

Does this PR require an update to the documentation?

N/A

Checklist:

  • The code has been tested locally
  • Tests have been added to cover all new functionality
  • The documentation has been updated to reflect any changes
  • The code has been formatted with pre-commit

rallyplot
rallyplot approved these changes Dec 16, 2025
Copy link
Member

@JoeZiminski JoeZiminski left a comment

Choose a reason for hiding this comment

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

Awesome, cheers @niksirbi! (accidently submitted review from another account)

@JoeZiminski JoeZiminski merged commit a6eb893 into main Dec 16, 2025
3 checks passed
@adamltyson adamltyson deleted the ns-update-project-tree-img branch December 16, 2025 16:44
@niksirbi
Copy link
Member Author

Awesome, cheers @niksirbi! (accidently submitted review from another account)

Ououou, a secret alias! 🕵🏼

@JoeZiminski
Copy link
Member

Haha I had a grand plan once to make a quantitative finance software, but gave up and thought I'd at least release the plotting part

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.

Confusion over the example folder structure schematic

4 participants