Break the repo down into 4 main nodes #60

epignatelli · 2023-12-19T11:04:48Z

Helx
Agentx
Envx
Expx

…than example item

github-actions · 2023-12-19T11:05:08Z

************* Module helx
helx/__init__.py:1:0: C0114: Missing module docstring (missing-module-docstring)
************* Module helx._version
helx/_version.py:1:0: C0114: Missing module docstring (missing-module-docstring)
************* Module helx.config
helx/config.py:1:0: C0114: Missing module docstring (missing-module-docstring)
helx/config.py:27:0: C0115: Missing class docstring (missing-class-docstring)
helx/config.py:28:4: C0116: Missing function or method docstring (missing-function-docstring)
helx/config.py:27:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/config.py:62:0: C0115: Missing class docstring (missing-class-docstring)
helx/config.py:63:4: C0116: Missing function or method docstring (missing-function-docstring)
helx/config.py:62:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/config.py:102:0: C0116: Missing function or method docstring (missing-function-docstring)
helx/config.py:103:4: R1705: Unnecessary "elif" after "return", remove the leading "el" from "elif" (no-else-return)
helx/config.py:118:25: C0209: Formatting a regular string which could be an f-string (consider-using-f-string)
helx/config.py:102:0: R0911: Too many return statements (7/6) (too-many-return-statements)
helx/config.py:121:0: C0116: Missing function or method docstring (missing-function-docstring)
helx/config.py:130:0: C0116: Missing function or method docstring (missing-function-docstring)
helx/config.py:130:0: R1711: Useless return at end of function or method (useless-return)
helx/config.py:160:0: C0116: Missing function or method docstring (missing-function-docstring)
helx/config.py:161:48: W0621: Redefining name 'flags' from outer scope (line 21) (redefined-outer-name)
helx/config.py:171:29: C0209: Formatting a regular string which could be an f-string (consider-using-f-string)
helx/config.py:178:0: C0116: Missing function or method docstring (missing-function-docstring)
helx/config.py:180:4: W0101: Unreachable code (unreachable)
helx/config.py:180:4: C0415: Import outside toplevel (yaml) (import-outside-toplevel)
helx/config.py:182:9: W1514: Using open without explicitly specifying an encoding (unspecified-encoding)
************* Module helx.image
helx/image.py:1:0: C0114: Missing module docstring (missing-module-docstring)
helx/image.py:24:0: C0116: Missing function or method docstring (missing-function-docstring)
helx/image.py:29:7: C0209: Formatting a regular string which could be an f-string (consider-using-f-string)
helx/image.py:64:12: W1202: Use lazy % formatting in logging functions (logging-format-interpolation)
helx/image.py:64:12: C0209: Formatting a regular string which could be an f-string (consider-using-f-string)
************* Module helx.mdp
helx/mdp.py:36:0: C0301: Line too long (105/100) (line-too-long)
helx/mdp.py:1:0: C0114: Missing module docstring (missing-module-docstring)
helx/mdp.py:30:0: C0115: Missing class docstring (missing-class-docstring)
helx/mdp.py:30:0: R0903: Too few public methods (0/2) (too-few-public-methods)
************* Module helx.modules
helx/modules.py:1:0: C0114: Missing module docstring (missing-module-docstring)
helx/modules.py:25:0: C0115: Missing class docstring (missing-class-docstring)
helx/modules.py:25:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/modules.py:33:0: C0115: Missing class docstring (missing-class-docstring)
helx/modules.py:33:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/modules.py:43:0: C0115: Missing class docstring (missing-class-docstring)
helx/modules.py:43:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/modules.py:51:0: C0115: Missing class docstring (missing-class-docstring)
helx/modules.py:51:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/modules.py:59:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/modules.py:67:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/modules.py:75:0: C0115: Missing class docstring (missing-class-docstring)
helx/modules.py:75:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/modules.py:107:8: C0200: Consider using enumerate instead of iterating with range and len (consider-using-enumerate)
helx/modules.py:87:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/modules.py:148:4: C0116: Missing function or method docstring (missing-function-docstring)
helx/modules.py:154:11: C0209: Formatting a regular string which could be an f-string (consider-using-f-string)
helx/modules.py:160:8: W0201: Attribute 'modules' defined outside __init__ (attribute-defined-outside-init)
************* Module helx.spaces
helx/spaces.py:1:0: C0114: Missing module docstring (missing-module-docstring)
helx/spaces.py:25:0: W0404: Reimport 'Shape' (imported line 23) (reimported)
helx/spaces.py:34:0: C0115: Missing class docstring (missing-class-docstring)
helx/spaces.py:40:15: C0209: Formatting a regular string which could be an f-string (consider-using-f-string)
helx/spaces.py:44:4: C0116: Missing function or method docstring (missing-function-docstring)
helx/spaces.py:48:0: C0115: Missing class docstring (missing-class-docstring)
helx/spaces.py:48:0: R0903: Too few public methods (1/2) (too-few-public-methods)
helx/spaces.py:60:0: C0115: Missing class docstring (missing-class-docstring)
helx/spaces.py:60:0: R0903: Too few public methods (1/2) (too-few-public-methods)


Report
======
245 statements analysed.

Statistics by type
------------------

+---------+-------+-----------+-----------+------------+---------+
|type     |number |old number |difference |%documented |%badname |
+=========+=======+===========+===========+============+=========+
|module   |7      |NC         |NC         |0.00        |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|class    |15     |NC         |NC         |26.67       |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|method   |18     |NC         |NC         |77.78       |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|function |10     |NC         |NC         |40.00       |20.00    |
+---------+-------+-----------+-----------+------------+---------+



External dependencies
---------------------
::

    yaml (helx.config)



605 lines have been analyzed

Raw metrics
-----------

+----------+-------+------+---------+-----------+
|type      |number |%     |previous |difference |
+==========+=======+======+=========+===========+
|code      |318    |52.56 |NC       |NC         |
+----------+-------+------+---------+-----------+
|docstring |99     |16.36 |NC       |NC         |
+----------+-------+------+---------+-----------+
|comment   |91     |15.04 |NC       |NC         |
+----------+-------+------+---------+-----------+
|empty     |97     |16.03 |NC       |NC         |
+----------+-------+------+---------+-----------+



Duplication
-----------

+-------------------------+------+---------+-----------+
|                         |now   |previous |difference |
+=========================+======+=========+===========+
|nb duplicated lines      |0     |NC       |NC         |
+-------------------------+------+---------+-----------+
|percent duplicated lines |0.000 |NC       |NC         |
+-------------------------+------+---------+-----------+



Messages by category
--------------------

+-----------+-------+---------+-----------+
|type       |number |previous |difference |
+===========+=======+=========+===========+
|convention |37     |NC       |NC         |
+-----------+-------+---------+-----------+
|refactor   |16     |NC       |NC         |
+-----------+-------+---------+-----------+
|warning    |6      |NC       |NC         |
+-----------+-------+---------+-----------+
|error      |0      |NC       |NC         |
+-----------+-------+---------+-----------+



Messages
--------

+-------------------------------+------------+
|message id                     |occurrences |
+===============================+============+
|too-few-public-methods         |13          |
+-------------------------------+------------+
|missing-class-docstring        |11          |
+-------------------------------+------------+
|missing-function-docstring     |10          |
+-------------------------------+------------+
|missing-module-docstring       |7           |
+-------------------------------+------------+
|consider-using-f-string        |6           |
+-------------------------------+------------+
|useless-return                 |1           |
+-------------------------------+------------+
|unspecified-encoding           |1           |
+-------------------------------+------------+
|unreachable                    |1           |
+-------------------------------+------------+
|too-many-return-statements     |1           |
+-------------------------------+------------+
|reimported                     |1           |
+-------------------------------+------------+
|redefined-outer-name           |1           |
+-------------------------------+------------+
|no-else-return                 |1           |
+-------------------------------+------------+
|logging-format-interpolation   |1           |
+-------------------------------+------------+
|line-too-long                  |1           |
+-------------------------------+------------+
|import-outside-toplevel        |1           |
+-------------------------------+------------+
|consider-using-enumerate       |1           |
+-------------------------------+------------+
|attribute-defined-outside-init |1           |
+-------------------------------+------------+




-----------------------------------
Your code has been rated at 7.59/10

epignatelli and others added 30 commits July 22, 2023 10:18

refactor(envs): strip off all non essentials from env wrappers

cbc97d8

refactor: remove random, stax and interop modules

8a25261

refactor: everything

0abb43b

ops(cd): cd now inputs manual version

8de5c00

align install to new cd

a5a237b

setup: add jax_enums to requirements

fcd4705

remove logger

79e7afb

sync

9a32114

fix: importing procgen break OSX, import lazily

87028b5

fix: pytype breaks with Shape=NoReturn from jax.core.ShapedArray

d3313d2

refactor: ReplayBuffer now initialises using environment spec rather …

ced5570

…than example item

align dqn to the new changes

d7a9101

add CITATION.cff

de812cb

feat: add gymnax wrapper

ddda109

refactor: log is not part of train state

e112af6

add gymanx example

2e3dec5

format: black

209c63e

refactor: experiment.run now takes seed rather than key;

d2beaad

align SAC

e2c824e

align sacd

3888c4b

brax and navix

2e980b2

refactor: replace StepType with standard Array

1e72d67

working dqn suite

6e1f0d7

refactor: remove external downloads

8e4a9b6

losses are not embedded into the agents

d30fedc

add reporting;

83c0c0a

fix copyright header

98dc23a

add some missing copyright

0c38312

fix merge module and add observations to navix example

1cec821

feat: add navix wrapper

88857bc

epignatelli added 19 commits December 18, 2023 22:10

build: now requires lower python version (3.8)

96a4fdd

Update CI.yml

a8a72ce

test: remove unittests

b9aff16

chore(release): bump version

ebddf59

ci(test): remove tests

6d911b9

build(setup): add gymanx to req

bfc2742

build(setup): add brax to req

6da7f8d

build(setup): add navix to req

8ba3370

build(setup): requires py3.10

a41f6a4

Cull files belonging to exp, agentx and envx

1901979

fix: update requirements

7bf24a6

fix: update imports and install script

93ef97a

install: downgrade python requirements to 3.8

92bad7b

Very quick readme update

8e7c881

Update readme

759a083

Update readme

922b2e3

update readme

854892e

Fix CI

c7ba7eb

Exp to Expx

4e9fd8f

epignatelli self-assigned this Dec 19, 2023

Remove jax_enums from env

32a6eb3

epignatelli merged commit 9b4bcfc into main Dec 19, 2023
2 checks passed

epignatelli deleted the fork branch December 19, 2023 11:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Break the repo down into 4 main nodes #60

Break the repo down into 4 main nodes #60

epignatelli commented Dec 19, 2023

github-actions bot commented Dec 19, 2023

Break the repo down into 4 main nodes #60

Break the repo down into 4 main nodes #60

Conversation

epignatelli commented Dec 19, 2023

github-actions bot commented Dec 19, 2023