Skip to content

Conversation

@magland
Copy link
Contributor

@magland magland commented Oct 24, 2025

I would like to update the URL for NWB Assistant to point to the new version. Feel free to try it out. The logic / prompts are the same... it's just the latest interface I am using shared across multiple chat applications.

@codecov
Copy link

codecov bot commented Oct 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.23%. Comparing base (801f92e) to head (c0b1995).
⚠️ Report is 1 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2140   +/-   ##
=======================================
  Coverage   95.23%   95.23%           
=======================================
  Files          29       29           
  Lines        2876     2876           
  Branches      740      740           
=======================================
  Hits         2739     2739           
  Misses         83       83           
  Partials       54       54           
Flag Coverage Δ
integration 74.44% <ø> (ø)
unit 84.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@stephprince
Copy link
Contributor

thanks for the update @magland !

One thing I noticed is that the new chat width does not expand to fill a wider screen display as the previous chat interface did, resulting in the "close assistant" tab dangling. Not sure where the best place to update that is?
Screenshot 2025-10-24 at 10 03 40 AM

@magland
Copy link
Contributor Author

magland commented Nov 11, 2025

@stephprince What you are showing in the screenshot is not an issue with the current PR. It seems some other changes were made elsewhere to the side panel... what you are showing in that screenshot is not consistent with what is seen in the deployed docs at

https://pynwb.readthedocs.io/en/stable/

Also, I have updated the url in this PR again to the latest location for this: https://nwb-assistant.neurosift.app

@magland
Copy link
Contributor Author

magland commented Nov 12, 2025

For now, I have redirected the old nwb assistant to the new one... so effectively it is giving https://nwb-assistant.neurosift.app

@stephprince
Copy link
Contributor

stephprince commented Nov 12, 2025

@magland, I see. I was looking at the readthedocs build from this PR but it looks like the tab issue I noticed was already merged to dev I believe as part of #2124. I will look into that as a separate issue.

@stephprince stephprince merged commit 460bcb5 into NeurodataWithoutBorders:dev Nov 12, 2025
40 of 41 checks passed
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.

2 participants