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

segregate nlp and non-nlp (cv) namespace in evalem #25

Closed
NISH1001 opened this issue May 26, 2023 · 4 comments
Closed

segregate nlp and non-nlp (cv) namespace in evalem #25

NISH1001 opened this issue May 26, 2023 · 4 comments
Assignees

Comments

@NISH1001
Copy link
Collaborator

NISH1001 commented May 26, 2023

What

We need to segregate existing evalem namespace into different categories based on downstream tasks such as NLP, CV, etc.
Eg: evalem.nlp, evalem.cv...

How

For this to work, we need to have model wrappers, processors, et. at the right level of abstraction so that downstream namespaces could easily be differentiated using base/parent class types in the implementation.

@NISH1001 NISH1001 self-assigned this May 26, 2023
@NISH1001
Copy link
Collaborator Author

Related thread at:
#19

@rbavery
Copy link

rbavery commented Jun 12, 2023

addressed by #27

@thesujitroy
Copy link

adding NLP , restructuring the evalm - adding losses (DICE

@NISH1001
Copy link
Collaborator Author

NISH1001 commented Jun 20, 2023

The new restructuring with evalem.nlp and evalem.cv has passed the final tests at:
#27 (comment)

This PR has been merged into main branch now.

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

3 participants