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.

Sign up for GitHub

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

Jump to bottom

Restructure repository #2

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

Restructure repository #2

wants to merge 19 commits into from

Conversation

gamatos
Copy link
Collaborator

@gamatos gamatos commented Jan 21, 2025

Restructure repository

  • Rename files to more descriptive names
  • Create pyproject.toml, deprecate requirements.txt
  • Update dependencies
  • Move source files into quixer folder
  • Update README.md
  • Use argparse for argument parsing
  • Add option to only run some models
  • Rename VAS to Transformer, QLINSVT to Quixer, in code
  • Add .gitignore

In addition:

  • Rename criterion to loss_function, add type
  • Add rest of types to arguments of batchify_s2s
  • Properly hint torch device types
  • Add missing type hint in evaluate
  • Remove unused parameters in train_epoch
  • Add types to evaluate_polynomial_state, apply_unitaries function signatures
  • Add the batch size as a Quixer class parameter
  • Rename variables in batchify_s2s for clarity
  • Rename bptt to window_size in setup_dataset
  • Clean up imports in run_comprehensive.py
  • Remove unused pad_token parameter in evaluate function

gamatos added 18 commits November 14, 2024 22:58
@gamatos
Add rest of types to arguments of batchify_s2s
7e16c84
@gamatos
Rename criterion to loss_function, add type
1d62ffd
@gamatos
Properly hint torch device types
1c9179e
@gamatos
Add missing type hint in evaluate
a13a25a
@gamatos
Remove unused parameters in train_epoch
ef6be59
@gamatos
Add types to get_batch_s2s
d95ef2e
@gamatos
Add types to evaluate_polynomial_state, apply_unitaries function …
98efa93 
…signatures
@gamatos
Add the batch size as a Quixer class parameter
76e5a95 
Required to avoid certain bugs
@gamatos
Rename model in code from QLINSVT to Quixer
a217cca
@gamatos
Rename variables in batchify_s2s for clarity
e6a1a20
@gamatos
Rename bptt to window_size in setup_dataset
da4934e
@gamatos
Restructure repository
d565f88 
Rename files to more descriptive names
Create `pyproject.toml`, deprecate `requirements.txt`
Move source files into `quixer` folder
Update `README.md` with new install instructions
@gamatos
Clean up imports in run_comprehensive.py
8eb38e9
@gamatos
Use argparse for argument parsing
c5a2184 
Add option to only run some models
Rename `VAS` to `Transformer` in code
Update README.md
@gamatos
Update dependencies
dd82fe1
@gamatos
Update bptt variable name in comment
f1391b9
@gamatos
Add .gitignore
521b05d
@gamatos
Remove unused pad_token parameter in evaluate function
46afdc5
@gamatos gamatos requested a review from nikhilkhatri January 21, 2025 14:13
@gamatos
Copy link
Collaborator Author

gamatos commented Jan 22, 2025

Added license file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nikhilkhatri nikhilkhatri Awaiting requested review from nikhilkhatri

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@gamatos