Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

New Implementation #63

Merged
merged 31 commits into from
Dec 11, 2019
Merged

New Implementation #63

merged 31 commits into from
Dec 11, 2019

Conversation

sernst
Copy link
Owner

@sernst sernst commented Oct 23, 2019

This is the initial "working" implementation of a web-based UI.

@sernst sernst force-pushed the next branch 5 times, most recently from b714d7a to bc459cb Compare October 28, 2019 15:29
This is the initial "working" implementation of a web-based UI.
@codecov
Copy link

codecov bot commented Nov 2, 2019

Codecov Report

Merging #63 into master will increase coverage by 0.77%.
The diff coverage is 99%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
+ Coverage   98.19%   98.96%   +0.77%     
==========================================
  Files         226      287      +61     
  Lines        9880    12022    +2142     
==========================================
+ Hits         9702    11898    +2196     
+ Misses        178      124      -54
Impacted Files Coverage Δ
cauldron/render/encoding.py 87.5% <ø> (ø) ⬆️
cauldron/cli/commands/refresh.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_alias.py 100% <ø> (ø) ⬆️
cauldron/__init__.py 100% <ø> (ø) ⬆️
...ron/test/cli/interaction/test_interaction_query.py 100% <ø> (ø) ⬆️
cauldron/session/writing/components/definitions.py 88.23% <ø> (ø) ⬆️
cauldron/session/buffering.py 91.11% <ø> (ø) ⬆️
cauldron/runner/python_file.py 100% <ø> (ø) ⬆️
cauldron/cli/commands/reload.py 100% <ø> (ø) ⬆️
cauldron/session/definitions.py 100% <ø> (+1.16%) ⬆️
... and 227 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e6aad3...0694741. Read the comment docs.

Progress on UI-based synchronization capabilities.
Continued progress on UI-based synchronization capabilities.
Specifically added abort handling to remote connections.
More progress on UI-based synchronization capabilities.
@coveralls
Copy link

coveralls commented Nov 5, 2019

Coverage Status

Coverage increased (+0.04%) to 98.237% when pulling d610766 on next into 7e6aad3 on master.

Progress on UI-based synchronization capabilities.
@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #63 into master will decrease coverage by 3.35%.
The diff coverage is 72.04%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   98.19%   94.84%   -3.36%     
==========================================
  Files         226      255      +29     
  Lines        9880    10718     +838     
==========================================
+ Hits         9702    10165     +463     
- Misses        178      553     +375
Impacted Files Coverage Δ
cauldron/cli/commands/version.py 100% <ø> (ø) ⬆️
cauldron/test/test_test_support.py 100% <ø> (ø) ⬆️
cauldron/render/encoding.py 87.5% <ø> (ø) ⬆️
cauldron/test/test_init.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_open_opener.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_refresh.py 100% <ø> (ø) ⬆️
cauldron/cli/commands/clear.py 100% <ø> (ø) ⬆️
cauldron/test/environ/test_response.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_alias.py 100% <ø> (ø) ⬆️
cauldron/test/invoke/test_invoker.py 100% <ø> (ø) ⬆️
... and 182 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e6aad3...4efc20b. Read the comment docs.

6 similar comments
@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #63 into master will decrease coverage by 3.35%.
The diff coverage is 72.04%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   98.19%   94.84%   -3.36%     
==========================================
  Files         226      255      +29     
  Lines        9880    10718     +838     
==========================================
+ Hits         9702    10165     +463     
- Misses        178      553     +375
Impacted Files Coverage Δ
cauldron/cli/commands/version.py 100% <ø> (ø) ⬆️
cauldron/test/test_test_support.py 100% <ø> (ø) ⬆️
cauldron/render/encoding.py 87.5% <ø> (ø) ⬆️
cauldron/test/test_init.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_open_opener.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_refresh.py 100% <ø> (ø) ⬆️
cauldron/cli/commands/clear.py 100% <ø> (ø) ⬆️
cauldron/test/environ/test_response.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_alias.py 100% <ø> (ø) ⬆️
cauldron/test/invoke/test_invoker.py 100% <ø> (ø) ⬆️
... and 182 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e6aad3...4efc20b. Read the comment docs.

@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #63 into master will decrease coverage by 3.35%.
The diff coverage is 72.04%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   98.19%   94.84%   -3.36%     
==========================================
  Files         226      255      +29     
  Lines        9880    10718     +838     
==========================================
+ Hits         9702    10165     +463     
- Misses        178      553     +375
Impacted Files Coverage Δ
cauldron/cli/commands/version.py 100% <ø> (ø) ⬆️
cauldron/test/test_test_support.py 100% <ø> (ø) ⬆️
cauldron/render/encoding.py 87.5% <ø> (ø) ⬆️
cauldron/test/test_init.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_open_opener.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_refresh.py 100% <ø> (ø) ⬆️
cauldron/cli/commands/clear.py 100% <ø> (ø) ⬆️
cauldron/test/environ/test_response.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_alias.py 100% <ø> (ø) ⬆️
cauldron/test/invoke/test_invoker.py 100% <ø> (ø) ⬆️
... and 182 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e6aad3...4efc20b. Read the comment docs.

@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #63 into master will decrease coverage by 3.35%.
The diff coverage is 72.04%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   98.19%   94.84%   -3.36%     
==========================================
  Files         226      255      +29     
  Lines        9880    10718     +838     
==========================================
+ Hits         9702    10165     +463     
- Misses        178      553     +375
Impacted Files Coverage Δ
cauldron/cli/commands/version.py 100% <ø> (ø) ⬆️
cauldron/test/test_test_support.py 100% <ø> (ø) ⬆️
cauldron/render/encoding.py 87.5% <ø> (ø) ⬆️
cauldron/test/test_init.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_open_opener.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_refresh.py 100% <ø> (ø) ⬆️
cauldron/cli/commands/clear.py 100% <ø> (ø) ⬆️
cauldron/test/environ/test_response.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_alias.py 100% <ø> (ø) ⬆️
cauldron/test/invoke/test_invoker.py 100% <ø> (ø) ⬆️
... and 182 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e6aad3...4efc20b. Read the comment docs.

@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #63 into master will decrease coverage by 3.35%.
The diff coverage is 72.04%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   98.19%   94.84%   -3.36%     
==========================================
  Files         226      255      +29     
  Lines        9880    10718     +838     
==========================================
+ Hits         9702    10165     +463     
- Misses        178      553     +375
Impacted Files Coverage Δ
cauldron/cli/commands/version.py 100% <ø> (ø) ⬆️
cauldron/test/test_test_support.py 100% <ø> (ø) ⬆️
cauldron/render/encoding.py 87.5% <ø> (ø) ⬆️
cauldron/test/test_init.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_open_opener.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_refresh.py 100% <ø> (ø) ⬆️
cauldron/cli/commands/clear.py 100% <ø> (ø) ⬆️
cauldron/test/environ/test_response.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_alias.py 100% <ø> (ø) ⬆️
cauldron/test/invoke/test_invoker.py 100% <ø> (ø) ⬆️
... and 182 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e6aad3...4efc20b. Read the comment docs.

@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #63 into master will decrease coverage by 3.35%.
The diff coverage is 72.04%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   98.19%   94.84%   -3.36%     
==========================================
  Files         226      255      +29     
  Lines        9880    10718     +838     
==========================================
+ Hits         9702    10165     +463     
- Misses        178      553     +375
Impacted Files Coverage Δ
cauldron/cli/commands/version.py 100% <ø> (ø) ⬆️
cauldron/test/test_test_support.py 100% <ø> (ø) ⬆️
cauldron/render/encoding.py 87.5% <ø> (ø) ⬆️
cauldron/test/test_init.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_open_opener.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_refresh.py 100% <ø> (ø) ⬆️
cauldron/cli/commands/clear.py 100% <ø> (ø) ⬆️
cauldron/test/environ/test_response.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_alias.py 100% <ø> (ø) ⬆️
cauldron/test/invoke/test_invoker.py 100% <ø> (ø) ⬆️
... and 182 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e6aad3...4efc20b. Read the comment docs.

@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #63 into master will decrease coverage by 3.35%.
The diff coverage is 72.04%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #63      +/-   ##
==========================================
- Coverage   98.19%   94.84%   -3.36%     
==========================================
  Files         226      255      +29     
  Lines        9880    10718     +838     
==========================================
+ Hits         9702    10165     +463     
- Misses        178      553     +375
Impacted Files Coverage Δ
cauldron/cli/commands/version.py 100% <ø> (ø) ⬆️
cauldron/test/test_test_support.py 100% <ø> (ø) ⬆️
cauldron/render/encoding.py 87.5% <ø> (ø) ⬆️
cauldron/test/test_init.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_open_opener.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_refresh.py 100% <ø> (ø) ⬆️
cauldron/cli/commands/clear.py 100% <ø> (ø) ⬆️
cauldron/test/environ/test_response.py 100% <ø> (ø) ⬆️
cauldron/test/cli/commands/test_alias.py 100% <ø> (ø) ⬆️
cauldron/test/invoke/test_invoker.py 100% <ø> (ø) ⬆️
... and 182 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e6aad3...4efc20b. Read the comment docs.

sernst added 10 commits November 7, 2019 12:12
Improve project management when executing commands through the
new UI and continue to fill out the unit testing in the ui subpackage.
Adds support for opening and viewing `*.cauldron` files through the
same UX pathways as opening projects.
Improve testing in existing areas where refactoring caused coverage
gaps.
Improve the UX management and feedback with the kernel.
Repairs unit tests that were failing on linux systems where
`os.startfile` does not exist.
Improvements to UI performance and unit testing on the ui.statuses
module.
Allow for a fallback hash to be used for Python 3.5 compatibility.
Continue working through edge cases in the UI to make sure they are
properly handled.
Add filtering to step change management in the UI to prevent
out-of-order changes from being applied and only rendering
changes when they are significant (body has changed from its
previous state).

Also added unit testing for recently added CLI commands.
Expand display update window now that UX rendering is more
efficient to help prevent multi-threaded rendering race
conditions in the kernel.
Add unit tests to a number of the UI routes.
Update execution aborting to better support the new UI
flow. Also add additional unit tests to ui routes.
Improves step execution timing in both the front and
back ends. Additional unit testing within the backend
as well to fully cover the new ui subpackage and capture
other areas of recently added code.
@sernst sernst force-pushed the next branch 7 times, most recently from 04d3cad to 3daf795 Compare November 24, 2019 13:45
Add async command state to the UI status to more effectively
determine when to trigger queued steps. Also adds Gitlab
CI pipeline definition.
Prevent step cascade running on error. Also add additional
unit tests to recently added code.
Modify tests that no longer exercise legacy mock
functionality.
Improve the structure and action handling of the project menus. Also
expand unit testing.
Pygments has recently introduced Python2Lexer and made
PythonLexer the Python3Lexer. For maximum compatibility
of the test, the assertions will allow any of the lexers
to be used.
Adds containers for local UI containerized environments and
continued to expand unit testing.
Formatting and style cleanup.
Update readme and other documentation. Cleanup old files and code that
are no longer needed. Continue to expand unit testing.
Codacy hasn't been great for analyzing modern Python so it is being
removed.
Upgrade containers to run server and ui
Cauldron process as wsgi services behind
nginx with threaded static file support for
better performance. Also cleanup docker
build process to simplify and expand the
combinations of images available.
Adds a `cauldron uidocker` command for launcher the containerized version
of the UI from the command line. Also cleaned up some of the container
display outputs to avoid confusion on where port mappings are happening.
Add more display information to the broadcasting steps in
the CI pipeline to make issues easier to debug.
Add parser argument for auth directories in
the uidocker command.
@sernst sernst merged commit ed906f8 into master Dec 11, 2019
@sernst sernst deleted the next branch December 11, 2019 00:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants