Skip to content

New prediction pipeline#28

Merged
sfluegel05 merged 3 commits intomainfrom
fix/new_pred_pipeline
Feb 24, 2026
Merged

New prediction pipeline#28
sfluegel05 merged 3 commits intomainfrom
fix/new_pred_pipeline

Conversation

@aditya0by0
Copy link
Member

@aditya0by0
Copy link
Member Author

@sfluegel05, I have committed package-lock.json file too,
https://stackoverflow.com/questions/44206782/do-i-commit-the-package-lock-json-file-created-by-npm-5

@sfluegel05
Copy link
Contributor

This looks good. I tried it myself and encountered some issues with the ELECTRA model (probably related to the torch version I am running). I also made the "type" for api/details optional since it is not used anywhere. Apart from that, I had no problems.

@sfluegel05 sfluegel05 merged commit 5aaa35b into main Feb 24, 2026
@sfluegel05 sfluegel05 deleted the fix/new_pred_pipeline branch February 24, 2026 11:21
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.

2 participants