Skip to content

Commit

Permalink
separate V- and Q-networks
Browse files Browse the repository at this point in the history
Summary:
* Our organization was a bit confusing. Some Q-networks (base classes) were in `sequential_decision_making`, while other Q-networks were in `common`. This moves them all into `sequential_decision_making`.
* This also means that files in `common` were importing from `sequential_decision_making` and vice-versa; preferably we minimize those.

Reviewed By: rodrigodesalvobraz

Differential Revision: D52988810

fbshipit-source-id: b7b398da77a824f379eaae5a65af0d71b8e6cd64
  • Loading branch information
Daniel Jiang authored and facebook-github-bot committed Feb 7, 2024
1 parent 791a5b3 commit 07c8e0c
Show file tree
Hide file tree
Showing 25 changed files with 696 additions and 676 deletions.
512 changes: 0 additions & 512 deletions pearl/neural_networks/common/value_networks.py

Large diffs are not rendered by default.

109 changes: 0 additions & 109 deletions pearl/neural_networks/sequential_decision_making/q_value_network.py

This file was deleted.

Loading

0 comments on commit 07c8e0c

Please sign in to comment.