search

Caleydo / confusionflow-ui

Visualize learning dynamics in multi-class image classifiers (e.g., convolutional neural networks) 💬 @gfrogat @thinkh
https://confusionflow.org
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

added new API proposal [DO NOT MERGE] #201

Closed gfrogat closed 6 years ago

gfrogat commented 6 years ago

!DO NOT MERGE!

We only use the review functionality for discussing the API for now.

gfrogat commented 6 years ago

I think we should separate the epoch data from the run meta data. Currently /run would return all runs with all epochs. So I suggest the following (missing) routes:

/run//epochdata -> returns all epoch data for the run

I would have simply returned the runs with empty epochdata arrays when we call /runs (also doesn't violate the schema definition), but it's probably better to have a separate call.

PUT /view with multiple run ids to create a new view -> return new view id

I didn't add a way to set the views yet because I'm still thinking about the automatic logging.

thinkh commented 6 years ago

I would have simply returned the runs with empty epochdata arrays when we call /runs (also doesn't violate the schema definition), but it's probably better to have a separate call.

An empty array for the epoch data is also fine for me.

I didn't add a way to set the views yet because I'm still thinking about the automatic logging.

Ok, take your time. 😄

thinkh commented 6 years ago

Discussion moved to https://github.com/gfrogat/confusionflow/issues/1.