Added `get_adj_matrix()` #91

Merged

meliurwen merged 3 commits from 90-feature-method-to-get-the-adjacency-matrix into dev 2 years ago

meliurwen commented 2 years ago (Migrated from github.com)

Owner

Description

See the issue related, what is done in the commits and in the changelog below.

#90

Merge into Dev Checklist

Tests for the new code
Acceptance Testing

Changelog

Added a get_adj_matrix() function

(Optional) Extra info

None

AlessandroBregoli (Migrated from github.com) requested changes 2 years ago

AlessandroBregoli (Migrated from github.com) left a comment

Owner

get_adj_matrix can't be a method of NetworkProcess. It should be a specific method of CtbnNetwork

meliurwen commented 2 years ago (Migrated from github.com)

Owner

ok

AlessandroBregoli (Migrated from github.com) requested changes 2 years ago

AlessandroBregoli (Migrated from github.com) left a comment

Owner

I suggest to return the reference of the matrix instead of cloning it:

self.adj_matrix.as_ref()

AlessandroBregoli (Migrated from github.com) approved these changes 2 years ago

Reviewers

The pull request has been merged as 0eb427e5cf.

Sign in to join this conversation.

enhancement

No Assignees

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: unimib-thesis-mad/reCTBN#91