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

SHOT-4404: Performance #74

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

SHOT-4404: Performance #74

wants to merge 5 commits into from

Commits on Nov 21, 2024

  1. No need to return a value, exception raised on failure.

    @staceyoue
    staceyoue committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    4fce7b1 View commit details
    Browse the repository at this point in the history

  2. Modify create_wrapper method to allow any data type and the onus is o… 

    …n the subclass to determine what data type is allowed. This allows list values to be processed.
    @staceyoue
    staceyoue committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    3035015 View commit details
    Browse the repository at this point in the history

  3. Remove unused function _create. Update validate function doc.

    @staceyoue
    staceyoue committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    f2f044a View commit details
    Browse the repository at this point in the history

  4. Add AliasApiRequestListWrapper class for lists of api requests.

    @staceyoue
    staceyoue committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    9a18585 View commit details
    Browse the repository at this point in the history

  5. Handle multipe api requests in single Qt event 

    * Execute multiple Alias API requests in a single Qt event
* Batch requests are contained in an AliasApiRequestListWrapper, which
  ensures that all Alias API requests are invoked in one Qt event
* This improves performance greatly because it means that Qt UI does not
  refresh after every Alias API call, instead all API calls are executed
  and Qt refreshes once after they are done
    @staceyoue
    staceyoue committed Nov 21, 2024
    Configuration menu
    Copy the full SHA
    91e8521 View commit details
    Browse the repository at this point in the history