Merge pull request verilog-to-routing#1857 from ganeshgore/push_artifacts

[Github Actions] Push artifacts on regression test failure

Author: vaughnbetz

.github/workflows/test.yml

@@ -137,6 +137,17 @@ jobs:
         ./.github/scripts/build.sh
         ./run_reg_test.py ${{ matrix.suite }} -show_failures -j2
 
+    - name: Upload artifact
+      uses: actions/upload-artifact@v2
+      with:
+        name: failed_${{matrix.name}}_regression_log
+        path: |
+          vtr_flow/**/*.log
+          vtr_flow/**/*.out
+          vtr_flow/**/*.blif
+          vtr_flow/**/*.p
+          vtr_flow/**/*.net
+          vtr_flow/**/*.r
 
   Sanitized:
     runs-on: ubuntu-18.04
@@ -185,7 +196,7 @@ jobs:
         ./.github/scripts/build.sh
         ./run_reg_test.py odin_reg_basic -show_failures -j2
 
-  
+
   VQM2BLIF:
     name: 'VQM2BLIF Basic Tests'
     runs-on: ubuntu-18.04

vtr_flow/scripts/python_libs/vtr/log_parse.py

@@ -301,6 +301,9 @@ def load_script_param(script_param):
         script_param = script_param.replace(" ", "_")
     else:
         script_param = "common"
+    for spec_char in [":", "<", ">", "|", "*", "?"]:
+        # replaced to create valid URL path
+        script_param = script_param.replace(spec_char, "_")
     return script_param
 
 

vtr_flow/scripts/python_libs/vtr/task.py

@@ -495,6 +495,9 @@ def create_job(
     Create an individual job with the specified parameters
     """
     param_string = "common" + (("_" + param.replace(" ", "_")) if param else "")
+    for spec_char in [":", "<", ">", "|", "*", "?"]:
+        # replaced to create valid URL path
+        param_string = param_string.replace(spec_char, "_")
     if not param:
         param = "common"