SunilSamra
2eb6223aad
Merge remote-tracking branch 'origin/feature/901-change-functionality-acl-rules' into feature/901-change-functionality-acl-rules
2023-07-17 19:42:29 +01:00
SunilSamra
bacb42833f
#901 - ran black pre-commit over observations.py to fix it
2023-07-17 19:42:05 +01:00
Sunil Samra
a2d99080cd
Apply suggestions from code review
2023-07-17 18:36:13 +00:00
Chris McCarthy
678f953ced
#1631 - Added the DEFCON 703 header to all possible files
2023-07-17 19:28:43 +01:00
Christopher McCarthy
3186fcb489
Merged PR 121: #1629 - Added rllib test
...
## Summary
Quick test that uses RLLIB in a session
## Test process
The learning session completes then we check that the number of rows in both the average reward per episode and all transactions csv files.
## Checklist
- [X] This PR is linked to a **work item**
- [X] I have performed **self-review** of the code
- [X] I have written **tests** for any new functionality added with this PR
- [ ] I have updated the **documentation** if this PR changes or adds functionality
- [X] I have run **pre-commit** checks for code style
#1629 - Added rllib test
Related work items: #1629
2023-07-17 17:28:51 +00:00
Marek Wolan
4032f3a2a8
Change typehints after mypy analysis
2023-07-17 16:22:07 +01:00
SunilSamra
00d01157da
#901 - Changed num_eval_steps back to 1 in ppo_seeded_training_config.yaml
2023-07-17 15:54:15 +01:00
Sunil Samra
d67df9234d
Apply suggestions from code review
2023-07-17 14:21:37 +00:00
SunilSamra
ded5a6f352
#901 - Fixed bug in implicit rule - comparing it to string ALLOW or DENY in access_control_list.py
2023-07-17 13:58:06 +01:00
SunilSamra
8008fab523
#901 - Removed flatten from training configs
...
- Added flatten operation in observations.py when there are multiple obs components
- Updated config.rst docs
2023-07-17 13:44:16 +01:00
SunilSamra
cb4089a0ba
#901
...
- Removed bool apply_implicit_rule
- Set default implicit_rule to EXPLICIT DENY
- Added position to ACLs in laydown configs
- Removed apply_implicit_rule from training configs
2023-07-17 13:00:58 +01:00
Chris McCarthy
ee0c797c3c
#1629 - Added assertion in the test that checks the length of the all transactions file too.
...
- Added supporting function on the TempPrimaiteSession class that reads the all transactions csv file.
- Some renaming of the functions.
2023-07-17 12:14:47 +01:00
Czar Echavez
6cb475ecdc
#1595 : set default tc and ldc to None for AgentABC and PrimaiteSession + adding a comment for cli load flag
2023-07-17 11:54:54 +01:00
Marek Wolan
ef8f6de646
Add typehint for agent config class
2023-07-17 11:21:29 +01:00
SunilSamra
3e7f6cc98d
#901
...
- Added check in access_control_list.py which sets implicit permission to NA if boolean is False
- Changed the defaults in training_config.py so that each scenario has an EXPLICIT ALLOW rule as default implicit rule
- Updated the test_seeding_and_deterministic_session.py because of change no2 adds an extra rule to that scenario
2023-07-17 10:27:56 +01:00
SunilSamra
ab45c7e3f9
#901 - added to config.rst and added new ACL main config options
2023-07-17 10:08:12 +01:00
Marek Wolan
98ac228f90
Fix types according to mypy
2023-07-14 16:38:55 +01:00
Czar Echavez
232908ac00
#1595 : minor fix to cli command
2023-07-14 16:04:34 +01:00
SunilSamra
a2461d29b4
#901 - amended comment in observations.py
2023-07-14 16:04:13 +01:00
Czar Echavez
c783965001
#1595 : added loading sessions to run command + test + documentation for how to use loading sessions
2023-07-14 15:51:38 +01:00
SunilSamra
b4f8514206
#901 - amended comment in training_config_main.yaml
2023-07-14 15:49:18 +01:00
SunilSamra
fc1a575fd0
#901 -
...
- Added comments in access_control_list.py
- Changed obs_shape to max_number_acl_rules from max_number_acl_rules + 1 as index starts from 1
- Commented episode and step print line from test_single_action_space.py
2023-07-14 15:27:37 +01:00
SunilSamra
6b8cf73207
901 - Added another test and tidied up comments in test_observation_space.py and tidied up comments in observations.py
2023-07-14 14:51:26 +01:00
Marek Wolan
e522e56ff1
Add typehints
2023-07-14 14:43:47 +01:00
Czar.Echavez
fbdb8aab28
#1595 :
...
- Added ability to load sessions via PrimaiteSession
- PrimaiteSession loading test
- Added a NotImplemented RLlib loading for now
- Added the ability to load sessions for hardcoded agents
- Moved Session metadata parsing to utils
2023-07-14 14:14:03 +01:00
Chris McCarthy
8c0ca8cfbc
#901 - Dropped temp_primaite_sessiion_2 from conftest.py.
...
- Re-added the hard-coded mean rewards per episode values from a rpe-trained agent to the deterministic test in test_seeding_and_deterministic_session.py
- Partially tidies up some tests in test_observation_space.py; Still some work to be done on this at a later date.
2023-07-14 14:13:11 +01:00
SunilSamra
4a0d688ae6
901 - fixed test_observation_space.py, added test fixture for test_seeding_and_deterministic_session.py and increased default max number of acls
2023-07-14 12:29:50 +01:00
Marek Wolan
c57ed6edcd
Added type hints
2023-07-14 12:01:38 +01:00
Czar Echavez
dce0d10383
#1595 : fix poorly merged tests + files
2023-07-14 11:21:59 +01:00
Czar Echavez
a92ef3f4ad
#1595 : test to make sure that the loaded agent trains + remove unnecessary files + fixing agent save output name
2023-07-14 10:56:28 +01:00
Czar Echavez
bc7c32697f
Merge branch 'dev' into feature/1595-finalise-and-test-the-loading-of-trained-agents
2023-07-14 08:39:52 +01:00
Marek Wolan
a923d818d3
Add More Typehint
2023-07-13 18:08:44 +01:00
SunilSamra
0bcaf0696d
901 - removed print statements and merged with dev
2023-07-13 17:14:59 +01:00
SunilSamra
79d77c85c4
Merge remote-tracking branch 'origin/dev' into feature/901-change-functionality-acl-rules
2023-07-13 16:48:02 +01:00
Czar.Echavez
c3023d4d7c
#1595 missed hardcoded_abc file in commit
2023-07-13 16:24:30 +01:00
Czar.Echavez
e2d5f0bcff
#1595 :
...
- SB3 Agent loading
- rename agent.py -> agent_abc.py
- rename hardcoded.py -> hardcoded_abc.py
- Tests
- Added in test asset that is used to load the SB3 Agent
2023-07-13 16:24:03 +01:00
Marek Wolan
4e4166d4d4
Continue Adding Typehints
2023-07-13 12:25:54 +01:00
SunilSamra
771061a218
901 - fixed test_single_action_space.py test
2023-07-13 11:45:23 +01:00
SunilSamra
ad4198da13
901 - changed acl current obs from list to numpy.array, changed default ACL list in training_config.py to FALSE, and tried to make test_seeding_and_deterministic_session.py test without fixed reward results
2023-07-13 11:04:11 +01:00
Marek Wolan
d2bac4307a
Type hint ACLs
2023-07-12 16:58:12 +01:00
SunilSamra
e192d24800
Merge remote-tracking branch 'origin/dev' into feature/901-change-functionality-acl-rules
...
# Conflicts:
# src/primaite/acl/access_control_list.py
2023-07-12 10:45:03 +01:00
SunilSamra
ae6c90a670
901 - fixed how acls are added into list with new logic - agent cannot overwrite another acl in the list
2023-07-12 09:47:16 +01:00
Czar.Echavez
e51391b0cf
#1595 : Moved hardcoded agent into its own file
2023-07-11 15:03:02 +01:00
SunilSamra
6547789d5d
901 - changed implicit_acl_rule from str to enum name
2023-07-11 12:36:22 +01:00
SunilSamra
f14963b6fb
Merge remote-tracking branch 'origin/dev' into feature/1566-configure_episode-steps-learn-eval
...
# Conflicts:
# src/primaite/config/training_config.py
2023-07-11 11:39:21 +01:00
SunilSamra
728ff01076
1566 - fixed the test_training_config.py test file by removing num_steps from init
2023-07-10 13:24:34 +01:00
SunilSamra
9738d2b889
1566 - added correct num_train_episodes etc values to configs, fixed test_reward.py
2023-07-10 11:25:26 +01:00
Marek Wolan
56f369cfe0
Merge remote-tracking branch 'origin/dev' into bugfix/1587-hardcoded-agent
2023-07-10 09:15:25 +01:00
Marek Wolan
07a0581fce
Merge remote-tracking branch 'origin/dev' into feature/1572-fix-docs-formatting
2023-07-09 18:13:57 +01:00
Marek Wolan
aac0e2cc0f
Merge remote-tracking branch 'origin/dev' into bugfix/1587-hardcoded-agent
2023-07-09 18:07:30 +01:00