Update Bad Pixel Monitor to use Django DB Models

[mfixstsci]

No description provided.

[pep8speaks]

Hello @mfixstsci, Thank you for updating !

• In the file jwql/website/apps/jwql/monitor_pages/monitor_bad_pixel_bokeh.py:

Line 285:62: W291 trailing whitespace
Line 289:25: E127 continuation line over-indented for visual indent

Comment last updated at 2024-11-12 19:14:12 UTC

[bhilbert4]

These changes look good to me. What about in monitor_bad_pixel_bokeh.py? That should be querying the database in order to produce the plots, right?

[mfixstsci]

These changes look good to me. What about in monitor_bad_pixel_bokeh.py? That should be querying the database in order to produce the plots, right?

@bhilbert4 you are right, I just looked at the file. I will go ahead and update it.

[mfixstsci]

@bhilbert4 okay, so the updates to monitor_bad_pixel_bokeh.py are in. I think should do it, let me know if I missed anything.

[mfixstsci]

With @york-stsci 's updates are we prepared to merge? @bhilbert4 would you like to review the changes with Brian's changes in?

[bhilbert4]

This looks good to me. I hope you used a tool like Black rather than changing all those single quotes to double quotes manually. :)

The only issue is that the migration file needs to be included.

[bhilbert4]

LGTM. Thanks for clearing up the empty defaults!

[york-stsci]

Looks good