search

SealUofI / SemPy

Simple Spectral Element Examples in Python
BSD 3-Clause "New" or "Revised" License
9 stars 1 forks source link

Format and clean code #5

Closed thilinarmtb closed 1 year ago

nchristensen commented 2 years ago

Is this from running autopep8?

thilinarmtb commented 2 years ago 
black -l 80 *.py */*.py */*/*.py`

followed by

isort *.py */*.py */*/*.py
thilinarmtb commented 2 years ago

There are a lot of flake8 failures. Probably need to be fixed before merging this.

thilinarmtb commented 2 years ago

What is with the no_install directory? Seems redundant.

nchristensen commented 2 years ago

Yeah we can probably resolve most of them by running autopep8 on all of the .py files like here.

nchristensen commented 2 years ago

We should probably set up CI at some point too.

thilinarmtb commented 2 years ago

@nchristensen : Remaining flake8 errors are mainly related to loopy kernels. Can you clean it up?