-
Notifications
You must be signed in to change notification settings - Fork 67
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
feat: Added a new GCBM status endpoint #180
base: master
Are you sure you want to change the base?
feat: Added a new GCBM status endpoint #180
Conversation
046c8d5
to
bf8899b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tests continue to fail. Can you validate once again?
The test case is failing for some other endpoint of GCBM simulation. Earlier I added some test cases for this endpoint they were not working as expected so I had to remove them. I am working on adding some new test cases for this endpoint. |
@SanjaySinghRajpoot @HarshCasper - what is the status of this PR? |
The test cases are failing for this some other endpoint in this PR. In this PR we are trying to get the status of GCBM simulation, using |
Pull Request Template
Description
A new endpoint is created called
/gcbm/status
. This endpoint will return the status of the running GCBM simulation and the logs generated insimulation.txt
. This endpoint will be called by the FLINT-UI to get the latest status of the GCBM simulation. In this manner, the user will be able to check if everything is working correctly or if there is any error. If the simulation is not running a400
status will be returned to the user.Fixes #179
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
It has been tested on a local host. The above image giving a error message as no simulation is running.