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.

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

Add support for profiling Flexmeasures #721

Closed
Nischay-Pro opened this issue Jun 12, 2023 · 1 comment
Closed

Add support for profiling Flexmeasures #721

Nischay-Pro opened this issue Jun 12, 2023 · 1 comment

Comments

@Nischay-Pro
Copy link
Contributor

Currently, Flexmeasures has no support for profiling. This makes it difficult to identify and analyze performance bottlenecks in the code as the application matures.

Some proposed solutions are

  1. Integrate pyinstrument in Makefile:
    Integrate pyinstrument into the Makefile allowing the developer to call make with appropriate commands to start flexmeasures with profiler attached.
    A possible target in Makefile can be make run-profiler
    This calls the profiler which in turn starts flexmeasures. Can be run to profile the entire application.

  2. Integrate pyinstrument as a part of the pytest suite or CI:
    Integrate the profiler to benchmark and profile critical parts of the code and automatically generate profile reports which can be quickly viewed by the developer to check the impact on performance.

@nhoening
Copy link
Contributor

Closing this as #722 enable us to use it. Not sure if we ever want the pytest integration, as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants