Skip to content

Added whitespace for POMDPs in DRN#227

Merged
sjunges merged 1 commit into
stormchecker:masterfrom
volkm:drn
May 28, 2025
Merged

Added whitespace for POMDPs in DRN#227
sjunges merged 1 commit into
stormchecker:masterfrom
volkm:drn

Conversation

@volkm

@volkm volkm commented May 27, 2025

Copy link
Copy Markdown
Contributor

Required after change in stormchecker/storm#717

@volkm volkm modified the milestones: 1.11, 1.10 May 28, 2025
@volkm

volkm commented May 28, 2025

Copy link
Copy Markdown
Contributor Author

Is needed for Storm version 1.10.0 but can be reverted if stormchecker/storm#722 is addressed.

@volkm volkm requested a review from sjunges May 28, 2025 08:29
@sjunges

sjunges commented May 28, 2025

Copy link
Copy Markdown
Contributor

LGTM, not sure why it is needed though.

@sjunges sjunges merged commit 6db779d into stormchecker:master May 28, 2025
10 checks passed
@volkm

volkm commented May 28, 2025

Copy link
Copy Markdown
Contributor Author

Thanks.
The latest change on the DRN parser always requires a whitespace after an observation, even if only a linebreak follows. I think we should lift this requirement in Storm if there is a linebreak, see stormchecker/storm#722.

@volkm volkm deleted the drn branch May 28, 2025 08:42
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