Science museum workflow

Fixes

Fixes #408  by @mathemancer
Description

Airflow DAG script to run the science museum script
Technical details

Faced few errors while testing not sure of its origin .
First occurrence of it was when the test passed and removed some comments  and then testing again failed .
/tmp/pytest-of-airflow/pytest-0/test_dag_loads_with_no_errors7
[2020-05-29 07:45:54,791] {dagbag.py:403} INFO - Filling up the DagBag from /tmp/pytest-of-airflow/pytest-0/test_dag_loads_with_no_errors7
[2020-05-29 07:45:54,792] {dagbag.py:229} INFO - File /usr/local/airflow/dags/science_museum_workflow.py assumed to contain no DAGs. Skipping.
{}

Mentioning this problem since not sure why this occurred suddenly.
Now the test is passing.
Tests

Screenshots

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the master branch of the repository.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I think (hope) that the issue was due to the file sync sometimes taking a moment to catch up between your local filesystem (where you were editing) and that of the docker container.  Sometimes that can cause issues.


"Angiotensin II" finds angiotensin as the first match

This is pretty much exactly what the phrase match is supposed to be for, so I have no idea why this isn't working:
https://name-resolution-sri-dev.apps.renci.org/lookup?string=Angiotensin II&offset=0&limit=10
Results:

CHEBI:48433 ("angiotensin")
PR:000036009 ("angiotensin II")

We now return CHEBI:2719 "Angiotensin II" as our top result. Closing.


XLSX.read is not a function Electron JS

Dears, I'm fairly new to this thing, so let me know what info I'm missing:
VSCode
Electron 11
Trying to read a local excel file in browser.
What I tried:

AND (not simultaneously)

What happened:
I wrote the html file, and opened it in chrome (87.0 64bit, latest as of 2021-jan-03) It worked fine, reading alright.
Then I npm-started the project in VSCode, and I got:
"Uncaught TypeError: XLSX.read is not a function
at FileReader.filereader.onload"
My Code Snippet:
`

Be sure that XSLX library loads before your main script.
...and i think you don't need both core and full. I think full includes core.

if nodeIntegration = true，window.module is not undefined，xlsx use CommonJs, so window.XLSX = {}
there is source code from xlsx
if(typeof exports !== 'undefined') make_xlsx_lib(exports);
else if(typeof module !== 'undefined' && module.exports) make_xlsx_lib(module.exports);
else if(typeof define === 'function' && define.amd) define(function() { if(!XLSX.version) make_xlsx_lib(XLSX); return XLSX; });
else make_xlsx_lib(XLSX);

maybe you can find an answer: https://juejin.cn/post/7006212507719172127/

@fujinxiang Does the demo project work for you?
The required arguments for electron 14 are:
	win = new electron.BrowserWindow({
		width: 800, height: 600,
		webPreferences: {
			worldSafeExecuteJavaScript: true, // required for Electron 12+
			contextIsolation: false, // required for Electron 12+
			nodeIntegration: true,
			enableRemoteModule: true
		}
	});

With that, const XLSX = require("xlsx") works from both the main thread and from the renderer thread

i have not try the demo, i use xlsx in script tag, not require, and the electron version of my application is 4.x.x.

Please try using the require form.  You can see how the demo does it by using a script tag to reference the JS file and performing a require from the script


Bump tailwindcss to v3

Description
Upgrades tailwindcss to v3 but results in increasing tailwind.css file from 1.7M to 3.1M
This increases the CSS transfer size from 175K (as of PR #2612) to 292K (as of now)
Followed the upgrade guide
Needs to be merged in before PR https://github.com/civiform/civiform/pull/2630
Upgrade effects
These upgrade effects were seen when ran with screenshot fail threshold of 0 instead of the usual 0.01
Fuller shadows

--

Some buttons are fatter

Release notes:
Upgrade tailwindcss from v2 to v3
Issue(s)
Partially addresses https://github.com/civiform/civiform/issues/1940

Might be a good idea to leave transform styles in, just in case those are influencing stacking issues

Codecov Report

Merging #3438 (4adcab2) into main (03e3275) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 4adcab2 differs from pull request most recent head 5342808. Consider uploading reports for the commit 5342808 to get more accurate results

@@            Coverage Diff            @@
##               main    #3438   +/-   ##
=========================================
  Coverage     80.52%   80.52%
  Complexity     2383     2383
=========================================
  Files           291      291
  Lines          9438     9438
  Branches        640      640
=========================================
  Hits           7600     7600
  Misses         1590     1590
  Partials        248      248

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@nbeloglazov the one failing screenshot test is a flake

Thanks Sami!