Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

Conversation

@wyrnat
Copy link

@wyrnat wyrnat commented Dec 8, 2023

Not tested: Please review my issue #9 so I can test it on the latest version

Execution

what

  • add another filter property machine to the tags widget
  • first filter machine defines the combobox options of the other 3 properties (gain, region, objective)

why

  • different machines have different regions and objectives for their optimization routines
  • the current filter options are limited to slac only

how

  • adjust the plugin loading
  • adjust the class BadgerFilterBox to deal with another property which influences other properties
  • adjust routine_page where filters are defined for chosen routine
  • adjust home_page where the filters are applied to the list of routines

Supporting information

Badger Plugins

file_structure
The directory tags was added to Badger Plugins. As examples, LCLS and XFEL was added.

files
The init file conatins of a dictionary which contain the tags. The configs file only defines name and badger-opt as dependency

closes #10

- create action for change of machine tag (other tags choice change)
- adapt refresh_ui to also update machine tag
- outsource machine tag change to GUI Widget bc. home_page.py needs
  same functionality (debatable choice)
- connect signal of machine Combobox
- update filters when machine changes
- add filter to get_current_routines
@wenatuhs
Copy link
Contributor

@wyrnat Thanks so much for the contribution! It has been years since this PR (sorry for the inactivity in this repo! Recent activities have been happening in the new location) so I feel maybe we have to do some work on the code to adapt it to the new code base. I'll leave the PR as is and archive this old repo (to not cause further confusion). Talk to you in the new repo soon!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature: generic tags

2 participants