Skip to content

Conversation

@bendichter
Copy link
Contributor

I think ultimately this should probably not all live in NeuroConv, but here is a proof-of-concept of a Python function that creates automated HED annotations

bendichter and others added 5 commits November 29, 2025 12:18
- Include HED NWB addendum in LLM system prompt
- Change definitions_allowed and allow_placeholders defaults to True
- Make api_key parameter optional with .env fallback
- Add verbose and definitions_allowed params to generate_hed_tags_for_trials
@codecov
Copy link

codecov bot commented Dec 3, 2025

Codecov Report

❌ Patch coverage is 0% with 243 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.61%. Comparing base (f69d301) to head (f40c32e).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/neuroconv/tools/hed.py 0.00% 243 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1614      +/-   ##
==========================================
- Coverage   87.41%   85.61%   -1.80%     
==========================================
  Files         156      158       +2     
  Lines       11148    11478     +330     
==========================================
+ Hits         9745     9827      +82     
- Misses       1403     1651     +248     
Flag Coverage Δ
unittests 85.61% <0.00%> (-1.80%) ⬇️

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

Files with missing lines Coverage Δ
src/neuroconv/tools/hed.py 0.00% <0.00%> (ø)

... and 6 files with indirect coverage changes

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

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