cc14b52584
* Long workload name layout fix (#269) * changed layout to fit experiment names - added span to show full name - shortened name to fit dropwdown - changed layout for added consistency * layout Fixes - refresh button is to the right - header is more consistent across different width screens * header layout update - center div makes turns into multiple lines if not all items fit * slight improvement for header/graph spacing * Fixed refresh button shape and function - moved find_causal_files to parser so that main and gui can access - resized refresh width to allow for same shape across different screens * all graphs now have the same width - graphs now have same width - chart headers start well below the header with filters --------- Co-authored-by: Jonathan R. Madsen <jrmadsen@users.noreply.github.com> * Causal GUI: Linting, synced y-range, remove unused imports/variables, and bug fixes (#274) * GUI: python linting workflow - runs flake8 on code in source/python/gui * GUI: flake8 settings in source/python/gui/setup.cfg - ignore E501 errors (line too long) - ignore W503 errors (line break before binary operator) * GUI: setup.py updates - remove unused imports * GUI: __main__.py updates - stddev is float value - remove unused imports - effectively propagate the --stddev argument * GUI: gui.py updates - remove unused imports - fix light mode - sync initial y range of all causal plots - fix error bars for causal data - set x-ticks to 5 - set y-ticks to 10 - only display top 99% of samples - separate global declarations and assignments to global values - remove unused variable assignments - fix mislabeled function_regex and exp_regex - change if X == False to if not X * GUI: header.py updates - remove unused imports - fix mislabeled function_regex and exp_regex * GUI: parser.py updates - add set_num_stddev function for manipulating global num_stddev value - remove unused variables - fix latency point object (duplicated __init__ function) - fix handling latency in JSON - fix formatting of validation format error message - replace if X == False with if not X - fix unused dataframe creation in add_latency - fix flake8 do not assign lambda for name_wo_ext (use def) * GUI: gui.py updates - replace misnamed "func_list" with "experiment_list" - replace misnamed "exp_list" with "progpt_list" * GUI: fix python workflow - quote python versions to avoid truncating 3.10 to 3.1 --------- Co-authored-by: JoseSantosAMD <87447437+JoseSantosAMD@users.noreply.github.com>
100 lines
2.7 KiB
YAML
100 lines
2.7 KiB
YAML
|
|
name: Formatting
|
|
run-name: formatting
|
|
|
|
on:
|
|
push:
|
|
branches: [ main, develop ]
|
|
pull_request:
|
|
branches: [ main, develop ]
|
|
|
|
concurrency:
|
|
group: ${{ github.workflow }}-${{ github.ref }}
|
|
cancel-in-progress: true
|
|
|
|
jobs:
|
|
python:
|
|
runs-on: ubuntu-20.04
|
|
strategy:
|
|
matrix:
|
|
python-version: [3.8]
|
|
|
|
steps:
|
|
- uses: actions/checkout@v3
|
|
- name: Set up Python ${{ matrix.python-version }}
|
|
uses: actions/setup-python@v4
|
|
with:
|
|
python-version: ${{ matrix.python-version }}
|
|
- name: Install dependencies
|
|
run: |
|
|
python -m pip install --upgrade pip
|
|
pip install black
|
|
if [ -f requirements.txt ]; then pip install -r requirements.txt; fi
|
|
- name: black format
|
|
run: |
|
|
black --diff --check .
|
|
|
|
cmake:
|
|
runs-on: ubuntu-20.04
|
|
|
|
steps:
|
|
- uses: actions/checkout@v3
|
|
- name: Install dependencies
|
|
run: |
|
|
sudo apt-get update
|
|
sudo apt-get install -y python3-pip
|
|
python3 -m pip install cmake-format
|
|
- name: cmake-format
|
|
run: |
|
|
set +e
|
|
cmake-format -i $(find . -type f | egrep 'CMakeLists.txt|\.cmake$')
|
|
if [ $(git diff | wc -l) -gt 0 ]; then
|
|
echo -e "\nError! CMake code not formatted. Run cmake-format...\n"
|
|
echo -e "\nFiles:\n"
|
|
git diff --name-only
|
|
echo -e "\nFull diff:\n"
|
|
git diff
|
|
exit 1
|
|
fi
|
|
|
|
source:
|
|
runs-on: ubuntu-22.04
|
|
|
|
steps:
|
|
- uses: actions/checkout@v3
|
|
- name: Install dependencies
|
|
run: |
|
|
DISTRIB_CODENAME=$(cat /etc/lsb-release | grep DISTRIB_CODENAME | awk -F '=' '{print $NF}')
|
|
sudo apt-get update
|
|
sudo apt-get install -y software-properties-common wget curl clang-format-11
|
|
- name: clang-format
|
|
run: |
|
|
set +e
|
|
FILES=$(find source examples tests -type f | egrep '\.(h|hpp|c|cpp)(|\.in)$')
|
|
FORMAT_OUT=$(clang-format-11 -output-replacements-xml ${FILES})
|
|
RET=$(echo ${FORMAT_OUT} | grep -c '<replacement ')
|
|
if [ "${RET}" -ne 0 ]; then
|
|
echo -e "\nError! Code not formatted. Detected ${RET} lines\n"
|
|
clang-format-11 -i ${FILES}
|
|
git diff
|
|
exit ${RET}
|
|
fi
|
|
|
|
includes:
|
|
runs-on: ubuntu-20.04
|
|
|
|
steps:
|
|
- uses: actions/checkout@v3
|
|
- name: check-includes
|
|
run: |
|
|
set +e
|
|
FILES=$(find source examples -type f | egrep '\.(hpp|cpp)(|\.in)$')
|
|
MATCHES=$(egrep 'include "timemory/|include <bits/' ${FILES})
|
|
if [ -n "${MATCHES}" ]; then
|
|
echo -e "\nError! Included timemory header with quotes or bits folder included\n"
|
|
echo -e "### MATCHES: ###"
|
|
echo -e "${MATCHES}"
|
|
echo -e "################"
|
|
exit 1
|
|
fi
|