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

merge from main. #18

Merged
merged 93 commits into from
Dec 8, 2023
Merged

merge from main. #18

merged 93 commits into from
Dec 8, 2023

Conversation

Anindyadeep
Copy link
Owner

No description provided.

Anindyadeep added 30 commits November 23, 2023 09:22
The reason of removing the model_map is the following:

- The schema of the model map is very much focussed for the summac model and other
keys of the schema is not been used by other models.

- Since, we do not have a general schema for all of the models, so it is just introducing
un-necessary complication in the structure. We can have a dedicated config where we can store
different configurations later. But for now, we can remove to keep things simple.
…side summac model.

Since, the SummaC model is been inspired from the implementation of https://github.com/tingofurro/summac
and that implemenattion only requires the specific strucutre of the model_map. So, keeping those function
and dict just there.
Since, the reason of removing of model_map is explained in previous commits, so
removed those dependecies inside hallucination_model too.
This new metric, is deriving from answer_relevancy_model classes.
@Anindyadeep Anindyadeep merged commit d3c1814 into Anindyadeep:main Dec 8, 2023
3 checks passed
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.

3 participants