Fix additional dataloader creation #220
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- adjusted initial Dockerfile - minor changes to convert_pgn_to_planes.ipynb and pgn_to_planes_converter.py
…yAra into dataset_creation
… have been used - added counter for games without positions for current phase
# Conflicts: # engine/src/rl/Dockerfile
- added analyse_game_phases.py to analyse game phase distribution and other information - minor changes
… phase - board->get_phase now expects the total amount of phases as an argument - phaseCountMap is now immediately filled
- added sample weighting to losses pytorch training files - load_pgn_dataset() now returns a dict - added file for easily generating normalized cutechess-cli commands
- added file to generate plots based on cutechess results
…ss through the trainer agent - you can now pass a phase to load_pgn_dataset to load a non default dataset
- added printing total positions in dataset when loading
…etting networkLoaded back to false
- adjusted cutechess run file to support 960 matches
- new and adjusted graphs in game_phase_detector.py (should be put into a separate file) - new plots in create_cutechess_plots.py
…and assuming a wrong input representation)
- moved game phase analysis code from game_phase_detector.py to own file (analyse_game_phase_definition.py) - minor changes in train_cnn.py - adjusted .gitignore
… assumes phases by movecount) - set default value for UCI_Chess960 back to false - minor fixes
- added colorblind palette
…against stockfish
…movecount" and corresponding searchsettings enum GamePhaseDefinition
…es back to initial state of fork
# Conflicts: # DeepCrazyhouse/configs/train_config.py # DeepCrazyhouse/src/preprocessing/pgn_converter_util.py # DeepCrazyhouse/src/preprocessing/pgn_to_planes_converter.py # DeepCrazyhouse/src/training/train_cnn.ipynb # DeepCrazyhouse/src/training/trainer_agent_pytorch.py # engine/src/agents/agent.cpp # engine/src/agents/agent.h # engine/src/agents/mctsagent.cpp # engine/src/agents/mctsagent.h # engine/src/agents/rawnetagent.cpp # engine/src/agents/rawnetagent.h # engine/src/environments/chess_related/board.cpp # engine/src/environments/chess_related/boardstate.h # engine/src/nn/neuralnetapi.cpp # engine/src/nn/neuralnetapiuser.cpp # engine/src/nn/neuralnetapiuser.h # engine/src/searchthread.cpp # engine/src/searchthread.h # engine/src/uci/crazyara.cpp
|
Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
In it's latest form, get_data_loader gets passed the full pgn_dataset_arrays_dict, but in the train_cnn notebook in the section where the additional data loaders are created it is still used in the old way of passing the contents of pgn_dataset_arrays_dict individually. This PR fixes this issue.