Skip to content

Conversation

@AleksanderB-hub
Copy link
Contributor

@AleksanderB-hub AleksanderB-hub commented Dec 17, 2025

Description

  • I contributed a curriculum skill encoder model for skill extraction tasks, following the work: https://ceur-ws.org/Vol-4046/ (paper 5)
  • Followed the example commit from ConTeXTMatch example, tested on TECH and HOUSE against the paper reported scores.

Checklist

  • [yes] Added new tests for new functionality
  • [yes] Tested locally with example tasks with a caveat that I could not find example task that would match this problem (job posting sentence --> list of skills to select). I can add this to the test code if needed, just let me know!
  • [yes] Code follows project style guidelines
  • [yes] Documentation updated
  • [yes] No new warnings introduced

@Mattdl Mattdl changed the title feat(workrb): add curriculum encoder and benchmark tests feat(workrb): add curriculum encoder and benchmark tests [#19] Dec 17, 2025
@Mattdl Mattdl changed the title feat(workrb): add curriculum encoder and benchmark tests [#19] feat(workrb): add curriculum encoder and benchmark tests (#19) Dec 17, 2025
@Mattdl Mattdl self-requested a review December 17, 2025 16:13
@Mattdl Mattdl added the enhancement New feature or request label Dec 17, 2025
@Mattdl
Copy link
Collaborator

Mattdl commented Dec 17, 2025

Hey Aleksander, this is an awesome PR and contribution, great job!

Before I can merge it, you'll have to make sure that the tests are passing. If you will commit your changes locally and push them, they will now run remotely and you can check them in this repository. To do it faster, you can also run the tests you implemented locally on your computer, e.g. running pytest.

Currently in the tests there seems to be an error to find the model:

Repository Not Found for url: https://huggingface.co/Aleksandruz/skillmatch-mpnet-curriculum-retriever/resolve/main/adapter_config.json.
E               Please make sure you specified the correct `repo_id` and `repo_type`.
E               If you are trying to access a private or gated repo, make sure you are authenticated. For more details, see 

Let me know if you have any questions, happy to help (:

@AleksanderB-hub
Copy link
Contributor Author

I think it might have been because the model was private on HuggingFace. I made it public now, so it should work. Let me know whether the tests are passing now.

Copy link
Collaborator

@Mattdl Mattdl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-ran tests, tests passing! Great contribution, LGTM. Closes #19

@Mattdl Mattdl merged commit de6b99d into techwolf-ai:main Dec 18, 2025
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants