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.

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

Judith thesis #441

Closed
wants to merge 98 commits into from
Closed

Judith thesis #441

wants to merge 98 commits into from

Conversation

felix-20
Copy link
Collaborator

@felix-20 felix-20 commented May 2, 2022

This PR will be split into multiple PRs for better understanding.

felix-20 added 10 commits April 11, 2022 18:25
commit 8652a67
Author: Judith <[email protected]>
Date:   Mon Apr 11 18:17:20 2022 +0200

    missed `handle_requests` on last commit

commit a354f89
Author: Judith <[email protected]>
Date:   Mon Apr 11 18:16:15 2022 +0200

    implement #356

commit fd54365
Author: Judith <[email protected]>
Date:   Mon Apr 11 17:56:35 2022 +0200

    docstrings

commit 9f43c9d
Author: Judith <[email protected]>
Date:   Mon Apr 11 15:07:47 2022 +0200

    fix docker test

commit b3e6e80
Merge: 6f64f9e 091eee6
Author: felix-20 <[email protected]>
Date:   Mon Apr 11 14:31:39 2022 +0200

    Merge branch 'development' into 379-multiple-experiments

commit 6f64f9e
Author: Judith <[email protected]>
Date:   Mon Apr 11 14:20:47 2022 +0200

    clean up docker port

commit 8c35e8a
Author: Judith <[email protected]>
Date:   Mon Apr 11 11:33:30 2022 +0200

    merge development

    commit 091eee6
    Author: Nikkel Mollenhauer <[email protected]>
    Date:   Mon Apr 11 10:43:00 2022 +0200

        Tests for `config_validation.py` (#404)

        * Don't get default data (shouldn't be necessary)

        * Unpack default data

        * Refactored utils functions to return dicts instead of strings

        * Adapted to new mock format

        * Adapted to new mock format

        * Some first tests

        * More tests

        * Fixed testcase-names

        * Moved file-endings to initial function call

        * Fixed tests

        * More asserts

        * More tests

        * More tests

        * More tests

        * `validate_sub_keys`-tests

    commit f8bc162
    Author: jannikgro <[email protected]>
    Date:   Fri Apr 8 13:20:42 2022 +0200

        [D] Stable baselines integration (#384)

        * refactored reinforcement learning agent to accept marketplace

        * adapted test_exampleprinter.py to marketplace initialization

        * add market option to accept continuos actions

        * fixed action space check

        * initial stable baselines integration

        * Agent init by env (#390)

        * introduced self.network in actorcritic_agent

        * added network_architecture in QLearningAgent

        * changed actorcritic_agent to network_architecture

        * set back training_scenario

        * am_configuration initialize rl-agent via marketplace

        * added final analyse to stable baselines training

        * added more stable baselines algorithms

        * added ppo algorithm

        * introduced stable_baselines_folder

        * renamed training to callback

        * satisfied linter

        * fixed loading problem

        * try to make tqdm run in stable_baselines

        * make tqdm running

        * reduced pmonitoring episodes in sb training

        * save model only if significantly better

        * fixed too long test time bug

        * moved back to 250 episodes testing

        * set timeout to 15 minutes

        * added first batch of fixes to @NikkelM feedback

        * added type annotations and asserts in stable_baselines_model

        * added sbtraining to training_scenario

        * applied comments in am_configuration

        * solved .dat problem and fixed crashing asserts

        * reintroduced _end_of_training

        * removed deprecated if

        * Moved '.dat' to function call instead of appending within function

        * Fixed assert

        * fixed model file ending bug

        * Add short explanation docstring

        Co-authored-by: Johann Schulze Tast <[email protected]>

        * fixed wrong docstring

        * Fixed tests

        Co-authored-by: NikkelM <[email protected]>
        Co-authored-by: Johann Schulze Tast <[email protected]>

commit 0183876
Author: Judith <[email protected]>
Date:   Mon Apr 11 11:31:15 2022 +0200

    name from `names` for container

commit ad70b7c
Author: Judith <[email protected]>
Date:   Sun Apr 10 16:32:47 2022 +0200

    fix #380

commit 8e1314c
Author: Judith <[email protected]>
Date:   Sun Apr 10 16:05:28 2022 +0200

    multiple experiments are supported on the webserver

commit 53a4000
Author: Judith <[email protected]>
Date:   Fri Apr 8 14:36:44 2022 +0200

    support for starting multiple container on docker side

commit 3c94f43
Author: Judith <[email protected]>
Date:   Fri Apr 8 11:46:07 2022 +0200

    ability to add `DockerInfo` for multiple container
@felix-20 felix-20 added research A feature offering value to Alex Team Kali We are connecting you to remote machines labels May 2, 2022
@felix-20 felix-20 self-assigned this May 2, 2022
@felix-20 felix-20 changed the title Introduce push notification from webserver Introduce push notification for webserver May 2, 2022
@felix-20 felix-20 changed the base branch from development to logging-for-api July 8, 2022 13:43
@felix-20 felix-20 changed the base branch from logging-for-api to development July 8, 2022 13:43
@felix-20 felix-20 changed the title Introduce push notification for webserver Judith thesis Jul 8, 2022
@felix-20
Copy link
Collaborator Author

Implemented in several other PRs

@felix-20 felix-20 closed this Jul 28, 2022
@felix-20 felix-20 deleted the 412-push-notification branch July 28, 2022 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
research A feature offering value to Alex Team Kali We are connecting you to remote machines Thesis - Judith
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants