Skip to content

Commit 4ce486c

Browse files
author
Lukas Stasytis
committed
default to rtlsim-based analytical model, reenable all tests in fifo sizing
1 parent 9e6c4d4 commit 4ce486c

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/finn/builder/build_dataflow_config.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,7 @@ class DataflowBuildConfig:
298298
#: to deduce the token access vectors empirically
299299
#: TREE_MODEL will use the tree mode of an operator if available, avoiding the generation
300300
#: of IP cores.
301-
tav_generation_strategy: Optional[TAVGenerationMethod] = TAVGenerationMethod.TREE_MODEL
301+
tav_generation_strategy: Optional[TAVGenerationMethod] = TAVGenerationMethod.RTLSIM
302302

303303
#: Which strategy will be used for token access vector generation for FIFO sizing.
304304
#: RTLSIM will result in performing RTLSIM for each node

tests/fpgadataflow/test_fifosizing.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -65,15 +65,15 @@ def fetch_test_model(topology, wbits=2, abits=2):
6565
"method",
6666
[
6767
"analytic_model_based",
68-
# "analytic_rtlsim",
69-
# "largefifo_rtlsim",
68+
"analytic_rtlsim",
69+
"largefifo_rtlsim",
7070
],
7171
)
7272
@pytest.mark.parametrize(
7373
"topology",
7474
[
7575
"tfc",
76-
# "cnv"
76+
"cnv"
7777
],
7878
)
7979
def test_fifosizing_linear(method, topology):

0 commit comments

Comments
 (0)