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 ceo training data as classification input #324

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dfguerrerom
Copy link
Contributor

@dfguerrerom dfguerrerom commented Aug 6, 2024

This is a WIP. FTM this will be a draft pull request.

The idea of this PR is to add a new feature to SEPAL to retrieve training data for the classification recipe from CEO. Some of the points that we have discussed are:

  • Authentication has to be done using the SEPAL ui, users will provide username/password and it will be stored as cookie in user's client (?). SEPAL won't store these credentials.
  • After authentication, users will get the list of the projects that they have access (@jonnvega are we also listing public data?).
  • User select the project and then the data that want to use.
  • We retrieve the data to the client and do some checks and we plug-in to the interface for the csv input.

@jonnvega, this is just an idea, we can shape it more and create some specific tasks?

tasks:

  • : ...

@dfguerrerom
Copy link
Contributor Author

@jonnvega , do you have the endpoint of CEO to retrieve the projects, and to query the data from it? It would be nice to see an example of the response.

@dfguerrerom dfguerrerom changed the title add new buttons to use training data from CEO in classification add ceo training data as classification method Aug 7, 2024
@dfguerrerom dfguerrerom changed the title add ceo training data as classification method add ceo training data as classification input Aug 7, 2024
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

Successfully merging this pull request may close these issues.

1 participant