-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AGVis Update 4.0 #72
Merged
AGVis Update 4.0 #72
Changes from 87 commits
Commits
Show all changes
100 commits
Select commit
Hold shift + click to select a range
5c4db2b
Legend.js
zmalkmus 32120d7
Window.js
zmalkmus c020c9e
Window.js
zmalkmus 23ef6f0
Updating Window.js comments
zmalkmus 4cb1664
Updating CanvasLayer.js comments
zmalkmus 4154216
Added CommunicationLayer.js header comment.
zmalkmus 1200969
Overhauling developer comments
zmalkmus ce66d9a
Updating ContMulti.js comments
zmalkmus 11c2127
Dev comments
zmalkmus 0bea9b3
Dev comments
zmalkmus 8966828
Dev comments
zmalkmus 6c77515
LayerControl.js Comments
zmalkmus 2e1427d
CanvasLayer comments
zmalkmus 75e0740
ContMulti.js comments
zmalkmus 2af4368
NDArray comments
zmalkmus 66bc7d7
Fixed header comment formatting
zmalkmus 5377345
Fixed header comment formatting
zmalkmus e5715f9
PlaybackControl.js Comments
zmalkmus 0edf3f4
Updating PlaybackControl.js comments
zmalkmus d65ec30
PlayMulti.js comments
zmalkmus dc2ed69
PlayMulti.js comments
zmalkmus 5f7abb8
SearcLayer.js Comments
zmalkmus c1dcc6d
Merge pull request #57 from CURENT/master
jinningwang fc9df7f
TimeBox.js Comments
zmalkmus efa19ee
TimeBox.js Comments
zmalkmus 3b2c84e
TopMulti.js comments
zmalkmus 63541e9
TopMulti.js comments
zmalkmus 48d1fe7
Finishing touches
zmalkmus 67c2b53
Finishing touches
zmalkmus 4011728
Merge pull request #58 from CURENT/js-file-commenting
jinningwang 62980e1
Added windows support for the agvis run command
zmalkmus 437eacb
Updated usage documentation for Windows
zmalkmus 32c0d4d
Setting up tests folder
zmalkmus 959a679
Runs server start command from the app.py directory
zmalkmus 8f69adc
Merge pull request #60 from CURENT/windows-support
jinningwang 5970050
basic test setup
zmalkmus 63147d1
updated requirements
zmalkmus 4322881
updating requirements
zmalkmus 914ef6c
Initial restructure of AGVis backend
zmalkmus 50b01df
AGVis backend restructure
zmalkmus 6cf58e1
Finished backend restructure. Added dev mode.
zmalkmus 3c27fb4
fixed typo
zmalkmus d619d2c
fixed typo in go.sh
zmalkmus dd71c0c
Updated agvis repository Docker uses
zmalkmus 78b5593
Fixed agvis pip install in Docker container
zmalkmus a496754
Added shell=False to subprocess
zmalkmus 7398147
Changed subprocess command for testing
zmalkmus 02328b3
Disabled running from directory. Currently breaks Docker.
zmalkmus 41e964b
reenabled popen for subprocess
zmalkmus 5a86b8a
reenabled popen for subprocess
zmalkmus ba53035
Fixed Docker issue. Added ability to build different branches using .…
zmalkmus 7301fd2
removed ! from go.sh
zmalkmus 51c2c92
Removed all syntax errors relating to an extra ! at the end
zmalkmus a090dde
Cleaned up object oriented implementation of flask app
zmalkmus 96c603c
Testing Web App
zmalkmus 08253ce
Writing tests
zmalkmus 20df131
tests
zmalkmus 465afdd
conftest
zmalkmus c368cc0
dime testing
zmalkmus d1e306e
Added header comments. Finished initial dime_client test for agvis
zmalkmus b34dc5e
Updated test name
zmalkmus c2e0ada
disabled docker test, looking into js testing
zmalkmus 2117e8a
adding tests for web
zmalkmus 6c1b7a4
client tests
zmalkmus 331cf5f
removed redundant test
zmalkmus 8742420
Added header comment and removed unused imports
zmalkmus a4dc9fc
Created docker test file
zmalkmus 16e65a7
debug ./go.sh file not being found
zmalkmus 5fde49a
debug ./go.sh file not being found
zmalkmus a378a79
debug ./go.sh file not being found
zmalkmus 3d2de10
Added checking out of code
zmalkmus 6b97b2c
updated go.sh script to include docker testing
zmalkmus 003114f
removed interactive tag
zmalkmus f951789
Disabling basic dime testing. Will keep the file in case it is needed…
zmalkmus ea4650c
added PR unit test protection for master
zmalkmus edc350d
Merge pull request #63 from CURENT/docker-agvis-bug
zmalkmus f717c5c
Resolving conflicts
zmalkmus 15d2e0f
Merge pull request #64 from CURENT/unittesting
zmalkmus d03a287
Custom Flask Configurations for AGVis
zmalkmus 505b125
Merge pull request #67 from CURENT/flask-config
zmalkmus b57aa97
Include pip upgrade in Dockerfile
zmalkmus 7eae9b3
Update citation
9d99a26
Update copyright
bd5488c
Merge pull request #68 from jinningwang/release
zmalkmus cf67459
Merge remote-tracking branch 'origin/master' into develop
zmalkmus 6f3b67a
Updated max zoom
zmalkmus ff170d2
Merge pull request #71 from CURENT/map-zoom
zmalkmus 973f011
Rename test.yml to pythonapp.yml
zmalkmus d53cecf
Update pythonapp.yml
zmalkmus 9dec485
Update dependency versions on pythonapp.yml
zmalkmus 38cead8
Added linting with flake8
zmalkmus f05c2f9
Remove flake 8 until its recommended fixes are corrected.
zmalkmus 98f69eb
Publish Python to PyPI in pythonapp.yml
zmalkmus 722b24d
Merge branch 'master' into develop
zmalkmus 9dc5d49
Fixed broken tests. Added agvis selftest functionality
zmalkmus 8d6ad9b
Fixing last unittest
zmalkmus f074dd2
Removing problematic test working locally but not on github
zmalkmus be0fd46
Merge pull request #73 from CURENT/enable-unittest-cli
zmalkmus 28b36a1
Cleaning up code for codacy
zmalkmus 4fb5bcc
Merge pull request #74 from CURENT/codacy-cleanup
zmalkmus File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
on: | ||
push: | ||
branches: [ "master", "develop", "unittesting" ] | ||
pull_request: | ||
branches: [ "master" ] | ||
|
||
jobs: | ||
test: | ||
name: Run test suite | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Checkout code | ||
uses: actions/checkout@v3 | ||
|
||
- name: Build the Docker image | ||
run: ./go.sh build | ||
|
||
- name: Run tests | ||
run: ./go.sh run_tests |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
@INPROCEEDINGS{10318583, | ||
author={Parsly, Nicholas and Wang, Jinning and West, Nick and Zhang, Qiwei and Cui, Hantao and Li, Fangxing}, | ||
booktitle={2023 North American Power Symposium (NAPS)}, | ||
title={DiME and AGVis: A Distributed Messaging Environment and Geographical Visualizer for Large-Scale Power System Simulation}, | ||
year={2023}, | ||
volume={}, | ||
number={}, | ||
pages={1-5}, | ||
keywords={Power system simulation;Data visualization;Distributed databases;Nonhomogeneous media;Real-time systems;Power systems;North America;Power grid;Open-source software;Large-scale system;High-concurrency Data;Geovisualization;Digital twin}, | ||
doi={10.1109/NAPS58826.2023.10318583}} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
# ================================================================================ | ||
# File Name: flask_configurations.py | ||
# Author: Zack Malkmus | ||
# Date: 1/18/2024 (created) | ||
# Description: Configuration file for FLASK APP | ||
# ================================================================================ | ||
|
||
class DefaultConfig(object): | ||
DEBUG = False | ||
TESTING = False | ||
CSRF_ENABLED = True | ||
|
||
class ProductionConfig(DefaultConfig): | ||
DEBUG = False | ||
|
||
class DevelopmentConfig(DefaultConfig): | ||
DEVELOPMENT = True | ||
DEBUG = True | ||
|
||
class TestingConfig(DefaultConfig): | ||
TESTING = True |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two things can be improved.
Let me know if you are interested in the package publishing process, otherwise I can take care of the second one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes I think it would be good to learn how package with github. I can rename the file as well.