Skip to content
This repository was archived by the owner on Nov 26, 2025. It is now read-only.

Package#12

Merged
micedre merged 85 commits intomainfrom
package
Dec 4, 2024
Merged

Package#12
micedre merged 85 commits intomainfrom
package

Conversation

@meilame-tayebjee
Copy link
Copy Markdown
Member

@meilame-tayebjee meilame-tayebjee commented Dec 4, 2024 •
edited
Loading

Added explainability pipeline
Restructured repo
Packaged torchFastText to distribute the model architecture with Poetry builder

To do next:

  • conditional dependency for lightning, captum ; remove dependency from numpy?
  • update and expand doc and notebook until explainability

@meilame-tayebjee
modified: MLproject
8d4a9df
@meilame-tayebjee
modified: src/utils.py
c7579e1
@meilame-tayebjee
modified: requirements.txt
d9da4db
@meilame-tayebjee
modified: .gitignore
0e243f0 
	modified:   src/model.py
	modified:   src/train.py
@meilame-tayebjee
modified: src/model.py
7d4ec75 
	modified:   src/tokenizer.py
@meilame-tayebjee
modified: src/model.py
fc38f8f 
	modified:   src/tokenizer.py
	modified:   src/utils.py
@meilame-tayebjee
modified: src/model.py
62d930c 
	modified:   src/tokenizer.py
@meilame-tayebjee
modified: .gitignore
281f435 
	new file:   src/experiments.ipynb
@meilame-tayebjee
modified: requirements.txt
57b6663 
	modified:   src/experiments.ipynb
@meilame-tayebjee
modified: src/model.py
b8b9123
@meilame-tayebjee
modified: src/experiments.ipynb
d4b5487 
	new file:   src/explainability_viz.py
@meilame-tayebjee
modified: src/experiments.ipynb
a3b3ae0 
	modified:   src/model.py
@meilame-tayebjee
modified: src/dataset.py
f186f08 
	modified:   src/experiments.ipynb
	modified:   src/model.py
	modified:   src/train.py
	modified:   src/utils.py
@meilame-tayebjee
renamed: src/dataset.py -> src/config/dataset.py
492996a 
	renamed:    src/preprocess.py -> src/config/preprocess.py
	modified:   src/experiments.ipynb
	renamed:    src/explainability_viz.py -> src/explainability/explainability_viz.py
	renamed:    src/utils.py -> src/explainability/utils.py
	renamed:    src/model.py -> src/models/model.py
	renamed:    src/tokenizer.py -> src/tokenizer/tokenizer.py
	new file:   src/tokenizer/utils.py
	modified:   src/train.py
@meilame-tayebjee
modified: src/experiments.ipynb
9039398 
	modified:   src/train.py
@meilame-tayebjee
modified: src/experiments.ipynb
fbf8a6c 
	modified:   src/explainability/explainability_viz.py
	modified:   src/explainability/utils.py
	modified:   src/models/model.py
	modified:   src/train.py
@meilame-tayebjee
modified: src/experiments.ipynb
5e972e5 
	modified:   src/explainability/explainability_viz.py
	modified:   src/models/model.py
@meilame-tayebjee
modified: src/experiments.ipynb
15d7c7f 
	modified:   src/explainability/utils.py
	modified:   src/models/model.py
	modified:   src/train.py
@meilame-tayebjee
modified: src/experiments.ipynb
54fe18c 
	modified:   src/explainability/utils.py
	modified:   src/models/model.py
@meilame-tayebjee
modified: .gitignore
6404177 
	modified:   src/experiments.ipynb
	modified:   src/explainability/explainability_viz.py
	modified:   src/explainability/utils.py
	modified:   src/models/model.py
@meilame-tayebjee
modified: src/experiments.ipynb
4aa5524 
	modified:   src/explainability/utils.py
	modified:   src/models/model.py
@meilame-tayebjee
modified: requirements.txt
e0672ee
@meilame-tayebjee
modified: src/experiments.ipynb
9453646 
	modified:   src/explainability/utils.py
	modified:   src/models/model.py
@meilame-tayebjee
modified: src/config/preprocess.py
b9b0649 
	modified:   src/experiments.ipynb
	modified:   src/explainability/explainability_viz.py
	modified:   src/explainability/utils.py
	modified:   src/models/model.py
@meilame-tayebjee
modified: src/experiments.ipynb
af5e7af 
	modified:   src/models/model.py
@meilame-tayebjee
modified: src/experiments.ipynb
8099d82 
	modified:   src/explainability/explainability_viz.py
	modified:   src/explainability/utils.py
	modified:   src/models/model.py
	modified:   src/tokenizer/tokenizer.py
@meilame-tayebjee
modified: src/experiments.ipynb
a9c1b43 
	modified:   src/explainability/utils.py
@meilame-tayebjee
modified: MLproject
80ed178 
	modified:   src/experiments.ipynb
	modified:   src/models/model.py
	modified:   src/train.py
@meilame-tayebjee
modified: requirements.txt
37b3a2b 
	modified:   src/experiments.ipynb
@meilame-tayebjee
modified: src/experiments.ipynb
b47f1fb 
	modified:   src/explainability/utils.py
meilame-tayebjee and others added 19 commits December 3, 2024 10:48
@meilame-tayebjee
Merge pull request #11 from micedre/refactor
f8b7959 
Refactor source
@meilame-tayebjee
Changed build method to integrate lightning upstream. Towards lightni…
0affbd0 
…ng "dependency on choice".
@meilame-tayebjee
Fixed bug after training and lightning checkpoint loading. Adapted no…
2e7f247 
…tebook. Changed config into preprocess as subpackage.
@meilame-tayebjee
Fixed imports. Added stratified split in preprocess. Updated README.
a3c81db
@meilame-tayebjee
Update README.
09222ea
@meilame-tayebjee
Update pyproject.toml
bf7d8ca
@meilame-tayebjee
Update pyproject.toml to include subpackage
4fe3870
@meilame-tayebjee
uniformizing names to torchFastText
088b718
@meilame-tayebjee
fixed subpackage imports
767d9e5
@meilame-tayebjee
REALLY fixed imports, reorganized slightly the code (utilities folder…
f30a112 
…) for lisibility
@meilame-tayebjee
fixed setup.py
a671614
@meilame-tayebjee
fixed setup.py
04eea62
@meilame-tayebjee
update version
1bf794d
@meilame-tayebjee
put tests into src
45a9981
@meilame-tayebjee
Moved from setuptools to poetry framework
ff7499d
@meilame-tayebjee
update to exclude pycache files
2112cd0
@meilame-tayebjee
Restructure folder to match poetry requirements (without src)
d26ad57
@meilame-tayebjee
Update preprocess.py to remove unnecessary APE-specific methods and a…
a0e3586 
…ll pandas lagging reference.
@meilame-tayebjee
Debugged preprocess/__init__.py and test on virtual environment
a6a7680
@meilame-tayebjee meilame-tayebjee self-assigned this Dec 4, 2024
@meilame-tayebjee meilame-tayebjee marked this pull request as draft December 4, 2024 14:31
@meilame-tayebjee meilame-tayebjee marked this pull request as ready for review December 4, 2024 14:32
@micedre micedre merged commit 2b243d1 into main Dec 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@jpramil jpramil Awaiting requested review from jpramil

@tomseimandi tomseimandi Awaiting requested review from tomseimandi

1 more reviewer

@micedre micedre micedre approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@micedre micedre

@jpramil jpramil

@meilame-tayebjee meilame-tayebjee

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@meilame-tayebjee @micedre @jpramil