Skip to content

Adding Features#4

Open
mohammadvazirpanah wants to merge 12 commits into
forsyde:masterfrom
mohammadvazirpanah:partial-change
Open

Adding Features#4
mohammadvazirpanah wants to merge 12 commits into
forsyde:masterfrom
mohammadvazirpanah:partial-change

Conversation

@mohammadvazirpanah
Copy link
Copy Markdown

Add zipU and add an example to test two scenarios in the untimed model

Comment thread examples/ut/two_scenario/controller.hpp Outdated
Comment thread examples/ut/two_scenario/controller.hpp Outdated
Comment thread examples/ut/two_scenario/top.hpp Outdated
UT::signal<int> from_source, from_constant, from_kernels;
UT::signal<unsigned int> zip_control;
UT::signal<std::tuple<std::vector<int>,std::vector<std::tuple<unsigned int, unsigned int, scenrio_func>>>> from_zip;
UT::signal<std::tuple<unsigned int, unsigned int, scenrio_func>> from_detector, from_detector2;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change all size and prod/cons rates to the size_t type.

Copy link
Copy Markdown
Collaborator

@shaniaki shaniaki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few comments and reviews are requested and the code seems not to be runnable.

Comment thread src/forsyde/ut_helpers.hpp
Comment thread src/forsyde/ut_process_constructors.hpp Outdated
Comment thread src/forsyde/ut_process_constructors.hpp Outdated
@mohammadvazirpanah mohammadvazirpanah changed the title Adding zipU Adding Features Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants