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

[demo] Server: Add accuracy tracking #125

Merged
merged 30 commits into from
Jul 13, 2021
Merged

[demo] Server: Add accuracy tracking #125

merged 30 commits into from
Jul 13, 2021

Conversation

hkaur008
Copy link
Contributor

@hkaur008 hkaur008 commented May 3, 2021

@hkaur008
Copy link
Contributor Author

hkaur008 commented May 4, 2021

#92 table to track accuracy is created now I need to insert the timestamp , transaction hash , accuracy and model id .
i can create a separate function to insert values to the table and run that function when a particular API is called .

we are having following apis :-
// Health
// Get all models.
// Get model with specific ID.
// Insert a new model.
// DATA MANAGEMENT
// Insert a training sample.
// Get original training data.

when do I need to call function to check accuracy with which API ? or I just need to do function only ?

@juharris
Copy link
Contributor

juharris commented May 5, 2021

Would you add two new functions: 1) get the accuracy history for a model, 2) Add a new accuracy record for a model

@hkaur008
Copy link
Contributor Author

hkaur008 commented May 6, 2021

@juharris , I have created a function to 2.) add new accuracy record
Can you direct me how exactly I need to build get accuracy record function !

@hkaur008
Copy link
Contributor Author

hkaur008 commented May 6, 2021

Could you direct me about the getAccuracyRecord function? Would it be an SQLite command to select the accuracy history of a particular model_id or something else?

@juharris
Copy link
Contributor

juharris commented May 6, 2021

Yeah that's right. It would be good to be able to get the entire accuracy history of a model by providing a modelId OR an address.

@hkaur008
Copy link
Contributor Author

hkaur008 commented May 7, 2021

I had built the function and ordered it by timestamp and return the result as an object. Do let me know the further changes required. I will be happy to code

@hkaur008 hkaur008 requested a review from juharris May 13, 2021 17:37
@hkaur008 hkaur008 requested a review from juharris May 22, 2021 21:02
@hkaur008
Copy link
Contributor Author

hkaur008 commented May 22, 2021

Results so obtained
image
image

@juharris
Copy link
Contributor

Please let me resolve the issues.

@hkaur008 hkaur008 requested a review from juharris May 23, 2021 08:30
@hkaur008
Copy link
Contributor Author

Messages updated !

@hkaur008
Copy link
Contributor Author

Messages updated !

Please review my pull request @juharris Sir !

@juharris juharris added demo Relates to the code to demonstrate the framework enhancement New feature or request labels Jul 5, 2021
@hkaur008 hkaur008 requested a review from juharris July 13, 2021 19:57
@juharris juharris changed the title removed deprecated bodyparser [demo] Server: Add accuracy tracking Jul 13, 2021
@juharris juharris merged commit ece2ec5 into microsoft:main Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
demo Relates to the code to demonstrate the framework enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants