Skip to content

Web changes for NWB conversion#344

Open
caroljiangg wants to merge 13 commits intomasterfrom
cj-nwb-conversion
Open

Web changes for NWB conversion#344
caroljiangg wants to merge 13 commits intomasterfrom
cj-nwb-conversion

Conversation

@caroljiangg
Copy link
Copy Markdown

No description provided.

@caroljiangg caroljiangg self-assigned this Oct 24, 2024
Copy link
Copy Markdown
Contributor

@deirdre-k deirdre-k left a comment

Choose a reason for hiding this comment

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

Looks great! Follow up with Mike to decide on what to do for the URL/domain stuff

Comment thread benchmarks/templates/benchmarks/central_profile.html
Comment thread benchmarks/templates/benchmarks/convert_nwb.html Outdated
Comment thread benchmarks/urls.py
Comment thread benchmarks/views/user.py Outdated
Comment thread static/benchmarks/example_json.json Outdated
Comment thread static/benchmarks/example_json.json Outdated
Copy link
Copy Markdown
Member

@mike-ferguson mike-ferguson left a comment

Choose a reason for hiding this comment

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

Looks great! A few small formatting changes, but great work overall.

Comment thread benchmarks/templates/benchmarks/convert_nwb.html Outdated
Comment thread benchmarks/forms.py Outdated
Comment thread benchmarks/templates/benchmarks/convert_nwb.html Outdated
Comment thread benchmarks/templates/benchmarks/convert_nwb.html Outdated
caroljiangg and others added 10 commits November 26, 2024 14:06
Co-authored-by: deirdre-k <95875723+deirdre-k@users.noreply.github.com>
Co-authored-by: Michael Ferguson  <mferg@mit.edu>
Co-authored-by: Michael Ferguson  <mferg@mit.edu>
Co-authored-by: Michael Ferguson  <mferg@mit.edu>
Co-authored-by: Michael Ferguson  <mferg@mit.edu>
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.

3 participants