Possibly can improve constraint set error

Open
#293 3 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
30/100
Issue type
Feature
Clarity
Needs clarification
Activity status
Stale
Tech stack
julia
Domain
backend

Research direction

Locate the code that formats the factorization constraint-set error and inspect how BitSetTuples.BoundedBitSetTuple is represented. Determine what a readable helper string should convey; done means the error gives users an understandable interpretation of the displayed constraint set.

Written by the indexing model from the issue text.

Description

documentation good first issue

What we get now is

Factorization constraint set at node UniSGP_AMF_29 is not a valid constraint set. Please check your model definition and constraint specification. (Constraint set: BitSetTuples.BoundedBitSetTuple(Bool[1 0 0 0 0; 0 1 1 1 0; 0 1 1 0 0; 0 1 0 1 0; 0 0 0 0 1]))

Maybe we can include a helper string to help users read this constraint set string. For example (acknowledging that I myself don't know how it is read, so just guessing) :

Factorization constraint set at node UniSGP_AMF_29 is not a valid constraint set. Please check your model definition and constraint specification. (Constraint set: BitSetTuples.BoundedBitSetTuple(Bool[1 0 0 0 0; 0 1 1 1 0; 0 1 1 0 0; 0 1 0 1 0; 0 0 0 0 1])).

Hint: The constraint set can be read as follows... There should not be a... (one in a certain position in each group?)... etc.

:)

Dominant language
Julia
Stars
46
Forks
8
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from ReactiveBayes/GraphPPL.jl

All issues in ReactiveBayes/GraphPPL.jl

Similar issues

More Julia issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.