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

Embeddings #213

Closed
wants to merge 20 commits into from
Closed

Embeddings #213

wants to merge 20 commits into from

Conversation

AnFreTh
Copy link
Collaborator

@AnFreTh AnFreTh commented Feb 12, 2025

This pull request includes significant updates to the testing workflow and various model classes to enhance functionality and maintainability. The most important changes involve adding a new GitHub Actions workflow for unit tests, removing the rotary embedding from the attention mechanism, and updating the embedding layer to handle additional embedding features.

Testing Workflow:

  • .github/workflows/pr-tests.yml: Added a new GitHub Actions workflow to run unit tests on pull requests targeting the develop and master branches. This workflow sets up Python, installs dependencies, runs tests using pytest, and enforces test success before allowing merges.

Model Class Updates:

Embedding Layer Enhancements:

Base Model Adjustments:

FTTransformer and Lightning Wrapper Updates:

@AnFreTh AnFreTh closed this Feb 12, 2025
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