Skip to content

Conversation

@TimTheHero
Copy link
Collaborator

  • Repair main.py
  • Repair PrepareDataset.py
  • Repair train.py
  • Retrain the model to fix a bug
  • Created settings.py
    -> place for managing all global variables
  • Create requirements.py
    -> put all requirements , needed to run the Project. Needs to be checked in the future for unnecessary imports.
  • Refactoring:
    -> Create docstrings for all Python Files in the Project
    -> Added exit() clause when program should stop if condition is invalid

Script for installing predefined list of packages, if not installed and prints version of each package
-fix edgecase len(repositories) == 0
User can use pip install -r requirements.txt to installation all needed packages
Version number needs to be changed according to running version requirements.
- create docstring for def create_output_folders, def download_repositories,
def create_encore_graphs, def create_matrix_structure, def parallell_processing and def prepare_dataset
- remove misplaced docstrings
- added docstring for class NodeTypes
-create docstrings for def one_hot_encoding and multi_hot_encoding
- change var from i to _ because i is not used once
- added docs for class EdgeTypes
Work in Progress:
Settings for all Python Files in the Project can be put here to access and change.
Fully integrate settings.py as settings for Project
Used PrepareDataset.py and train.py to train a new model
This model know should work in main.py without throwing the Attribute Error: 'GATConv object has no attribute 'res'
I thought it has something to do with torch.geometric but I am not sure about that.
I decided to push the mlruns for reviewing later.
-del unused variable
- make threshold changeable in settings.pc
@TimTheHero TimTheHero merged commit ff41be2 into main Nov 25, 2024
1 check failed
@TimTheHero TimTheHero deleted the refactoring branch November 25, 2024 09:25
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