Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Node store #4

Merged
merged 58 commits into from
Nov 6, 2024
Merged

Node store #4

merged 58 commits into from
Nov 6, 2024

Conversation

Jhsmit
Copy link
Owner

@Jhsmit Jhsmit commented Oct 29, 2024 •
edited
Loading

This PR refactors the web application part of dont FRET
Reactives are no longer subclassed but are replaced by ListStore and DictStore which are reactive compositions.

Data in the web application is now handled by a datamanager module which caches photon and burst futures. The frontend interacts with the datamanager throuh asynchronous solara.use_task

Other changes:

  • double click to change FRET node name
  • new filter window using altair histogram, click to create filter from selection

TODO:

  • fix tests
  • refactor BurstSettingsReactive
  • refactor SnackBarReactive
  • revert temporary loading test data

Jhsmit added 30 commits September 27, 2024 16:07
@Jhsmit
home page in more or less working state
f88b262
@Jhsmit
add duration property
ea7b88d
@Jhsmit
fix burstfigure plot settings
af10527
@Jhsmit
burst figure controller model
bad85c2
@Jhsmit
add select vue component
deedf4d
@Jhsmit
add tcspc graphs
dd2a63c
@Jhsmit
move some stuff around
e3da965
@Jhsmit
improve nested selections
4c169dc
@Jhsmit
add trace figure
f81f16f
@Jhsmit
add initial version of filter editor
da40e11
@Jhsmit
add liststore functionality
9c8b74b
@Jhsmit
filters as liststore
9f349b6
@Jhsmit
figurfromtask component
22026b4
@Jhsmit
comment on how to improve wrap_callback
e3d8457
@Jhsmit
add new filters dialog
f3d1465
@Jhsmit
update traces/burst view
e79188d
@Jhsmit
ruff
d5868ed
@Jhsmit
remove comment
574f311
@Jhsmit
cleanup
e453503
@Jhsmit
use reactive for edit settings
6a5e63a
@Jhsmit
generate selectors from object
9bfcf6c
@Jhsmit
to done!
9e9e010
@Jhsmit
file selectors
5ea407b
@Jhsmit
load style from script path
4ce8765
@Jhsmit
trace selection in global scope
f6a762d
@Jhsmit
fix altair not showing on initial open of the dialog
caeebc0
@Jhsmit
major cleanup
627098d
@Jhsmit
delete unused file
8a88ee4
@Jhsmit
use liststore
6abd60f
@Jhsmit
add missing requirements
797a4db
Jhsmit added 27 commits November 5, 2024 12:47
@Jhsmit
booleans to enable/disable pages
8aac582
@Jhsmit
burst figure as standalone component
c595ad5
@Jhsmit
revert defaults page active
71d48cc
@Jhsmit
refactor move new models to models.py
856cc76
@Jhsmit
figures should take the value rather than reactive
d6be859
@Jhsmit
add missing typevar
7526287
@Jhsmit
refactor to datamanager
0831ff1
@Jhsmit
set diable pages to false in testing
0107870
@Jhsmit
cleanup datamanager
337cf8d
@Jhsmit
remove comments
8a2dc50
@Jhsmit
clean up dataload template
528d51d
@Jhsmit
docstring typo
ec9ad07
@Jhsmit
fix order of returned results
e457242
@Jhsmit
update ui test
ad62640
@Jhsmit
ignore hook warning
4f7d53d
@Jhsmit
snackbar as composed reactive
fcd225d
@Jhsmit
reactives as stores
01ecce9
@Jhsmit
fix comma placement
638aa87
@Jhsmit
drop py39
7429b5f
@Jhsmit
make function to create burst nodes
d2b2934
@Jhsmit
fix tying on get node
c9c2a9d
@Jhsmit
update liststore, see use_reactive
95dfe37
@Jhsmit
simplify web models
b946998
@Jhsmit
fix web test
04bf4a4
@Jhsmit
fix task burst search test
ee36a94
@Jhsmit
python version in quotes
a6e8fd1
@Jhsmit
cleaning
c5a48ed
@Jhsmit Jhsmit marked this pull request as ready for review November 6, 2024 17:57
@Jhsmit Jhsmit merged commit f8ef773 into master Nov 6, 2024
3 of 7 checks passed
@Jhsmit Jhsmit mentioned this pull request Nov 7, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Jhsmit