AllenInstitute / AllenInstitute/MIES

SF: Allow to use warnings

Open
#2,248 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement SweepFormula
Dominant language
IGOR Pro
Stars
30
Forks
11
Avg merge
1d 40m
Merged PRs (30d)
8

Description

In some cases we would like to warn the user only.
But this is currently not possible.

  • Add SFH_WARN() function
  • Revise GUI to also show warnings, then tackle also https://github.com/AllenInstitute/MIES/issues/1624
  • Rate-limit warnings
  • Store warnings and errors in results wave
  • Add convenience wrapper in ForeignFunctionInterface to execute SF code and return warnings and errors
  • Warn when an explicitly requested epoch for epochs does not exist

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing how SF code currently reports errors, then map the requested SFH_WARN() function through the GUI and results wave handling. Review the ForeignFunctionInterface entry point and the epochs lookup path; done means warnings are displayed, rate-limited, stored with errors, returned by the wrapper, and emitted for missing explicitly requested epochs.

Written by the indexing model from the issue text.

Assessment

Domain
tooling
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.