Mees Fix

Results 49 comments of Mees Fix

Hey @BradleySappington and @bhilbert4 I know this has been sitting here a while. @BradleySappington do you know if these changes will conflict with your sorting PR?

@bourque and @cracraft I found out the yellow square issue. When the DB is empty and there is no data available, a 10x10 array of zeros is generated to take...

> Hello @york-stsci, Thank you for updating ! > > * In the file [`jwql/instrument_monitors/common_monitors/bad_pixel_monitor.py`](https://github.com/spacetelescope/jwql/blob/763cd1c5cbb56d57e30822dd5b15c56c712eda00/jwql/instrument_monitors/common_monitors/bad_pixel_monitor.py): > > > > [Line 805:1](https://github.com/spacetelescope/jwql/blob/763cd1c5cbb56d57e30822dd5b15c56c712eda00/jwql/instrument_monitors/common_monitors/bad_pixel_monitor.py#L805): [W293](https://duckduckgo.com/?q=pep8%20W293) blank line contains whitespace > > [Line 855:85](https://github.com/spacetelescope/jwql/blob/763cd1c5cbb56d57e30822dd5b15c56c712eda00/jwql/instrument_monitors/common_monitors/bad_pixel_monitor.py#L855):...

@penaguerrero I went ahead and checked/approved the changes, have you by chance made the changes to the web application to make the figures we generate here visible? Because as of...

Still reviewing, need to address pep8 issues.

Looks like I merged the TA monitor, accidentally force pushed, and then re-merged it again.

I also checked out the pep8 issues listed here and pulled your repo locally and it looked fine 🤷‍♂️

> As I was falling asleep last night, it occurred to me that we might want an option that will reset some subset of tables other than all tables for...

> @mfixstsci based on @york-stsci 's response in slack, I think this is ready to be merged. I'm not sure how deeply you want to review it. Bourque reviewed it...