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

fix: position_in_room now creates array properly` #80

Merged
merged 2 commits into from
Jul 4, 2024

Conversation

epignatelli
Copy link
Owner

No description provided.

Copy link
Contributor

github-actions bot commented Jul 4, 2024

************* Module navix._version
navix/_version.py:1:0: C0114: Missing module docstring (missing-module-docstring)
************* Module navix.grid
navix/grid.py:483:0: C0301: Line too long (101/100) (line-too-long)
navix/grid.py:484:0: C0301: Line too long (123/100) (line-too-long)
navix/grid.py:494:0: C0301: Line too long (102/100) (line-too-long)
navix/grid.py:1:0: C0114: Missing module docstring (missing-module-docstring)
navix/grid.py:51:38: W0621: Redefining name 'coordinates' from outer scope (line 36) (redefined-outer-name)
navix/grid.py:452:0: W0102: Dangerous default value {} as argument (dangerous-default-value)
navix/grid.py:25:0: W0611: Unused List imported from typing (unused-import)
navix/grid.py:29:0: W0611: Unused jax.tree_util imported as jtu (unused-import)


Report
======
157 statements analysed.

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

+---------+-------+-----------+-----------+------------+---------+
|type     |number |old number |difference |%documented |%badname |
+=========+=======+===========+===========+============+=========+
|module   |2      |NC         |NC         |0.00        |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|class    |1      |NC         |NC         |100.00      |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|method   |4      |NC         |NC         |100.00      |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|function |22     |NC         |NC         |100.00      |0.00     |
+---------+-------+-----------+-----------+------------+---------+



609 lines have been analyzed

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

+----------+-------+------+---------+-----------+
|type      |number |%     |previous |difference |
+==========+=======+======+=========+===========+
|code      |213    |34.98 |NC       |NC         |
+----------+-------+------+---------+-----------+
|docstring |265    |43.51 |NC       |NC         |
+----------+-------+------+---------+-----------+
|comment   |49     |8.05  |NC       |NC         |
+----------+-------+------+---------+-----------+
|empty     |82     |13.46 |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 |5      |NC       |NC         |
+-----------+-------+---------+-----------+
|refactor   |0      |NC       |NC         |
+-----------+-------+---------+-----------+
|warning    |4      |NC       |NC         |
+-----------+-------+---------+-----------+
|error      |0      |NC       |NC         |
+-----------+-------+---------+-----------+



% errors / warnings by module
-----------------------------

+---------------+------+--------+---------+-----------+
|module         |error |warning |refactor |convention |
+===============+======+========+=========+===========+
|navix.grid     |0.00  |100.00  |0.00     |80.00      |
+---------------+------+--------+---------+-----------+
|navix._version |0.00  |0.00    |0.00     |20.00      |
+---------------+------+--------+---------+-----------+



Messages
--------

+-------------------------+------------+
|message id               |occurrences |
+=========================+============+
|line-too-long            |3           |
+-------------------------+------------+
|unused-import            |2           |
+-------------------------+------------+
|missing-module-docstring |2           |
+-------------------------+------------+
|redefined-outer-name     |1           |
+-------------------------+------------+
|dangerous-default-value  |1           |
+-------------------------+------------+




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

@epignatelli epignatelli merged commit 9792102 into main Jul 4, 2024
3 checks passed
@epignatelli epignatelli deleted the fix-keycorridor branch July 4, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant