MIT-LCP / waveform-annotation

Platform for annotating physiological waveform data.
11 stars 4 forks source link

Improves ann request / updates requirements #80

Closed Lucas-Mc closed 2 years ago

Lucas-Mc commented 2 years ago

This change should improve the feature which allows users to request specific events to view. This issue wasn't reproduced locally since I believe it was more client-sided since it depended heavily on browser type but these changes should address these (my theory was right about inconsistent loading times). Hopefully this solves the issue or at least makes it better!

rgmark commented 2 years ago

Thanks! Works great!Sent from my Galaxy -------- Original message --------From: Lucas McCullum @.> Date: 12/15/21 12:08 AM (GMT-05:00) To: MIT-LCP/waveform-annotation @.> Cc: Subscribed @.***> Subject: [MIT-LCP/waveform-annotation] Improves ann request / updates requirements (PR #80) This change should improve the feature which allows users to request specific events to view. This issue wasn't reproduced locally since I believe it was more client-sided since it depended heavily on browser type but these changes should address these (my theory was right about inconsistent loading times). Hopefully this solves the issue or at least makes it better!

You can view, comment on, or merge this pull request online at:   https://github.com/MIT-LCP/waveform-annotation/pull/80

Commit Summary

829dda3 Improves ann request / updates requirements

File Changes (2 files)

M
requirements.txt
(38)

M
waveform-django/waveforms/dash_apps/finished_apps/waveform_vis.py
(3)

Patch Links:

https://github.com/MIT-LCP/waveform-annotation/pull/80.patch https://github.com/MIT-LCP/waveform-annotation/pull/80.diff

—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.