Skip to content

fix: toDot => to_dot#132

Merged
rmanhaeve merged 1 commit into
ML-KULeuven:masterfrom
werifu:fix_to_dot
Jul 16, 2025
Merged

fix: toDot => to_dot#132
rmanhaeve merged 1 commit into
ML-KULeuven:masterfrom
werifu:fix_to_dot

Conversation

@werifu

@werifu werifu commented Jul 16, 2025

Copy link
Copy Markdown
Contributor

When I ran python step-by-step.py tmp.dl --with-dot I found there were some call errors.

Traceback (most recent call last):
  File "path/problog/step-by-step.py", line 126, in <module>
    main(**vars(args))
  File "path/problog/step-by-step.py", line 76, in main
    print(gp.toDot(), file=f)
          ^^^^^^^^
AttributeError: 'LogicFormula' object has no attribute 'toDot'. Did you mean: 'to_dot'?

Besides, saveSDDToDot doesn't exist.

Traceback (most recent call last):
  File "path/problog/step-by-step.py", line 126, in <module>
    main(**vars(args))
  File "path/problog/step-by-step.py", line 93, in main
    nnf.saveSDDToDot(dotprefix + "sdd.dot")
    ^^^^^^^^^^^^^^^^
AttributeError: 'SDD' object has no attribute 'saveSDDToDot'

@rmanhaeve
rmanhaeve merged commit 59b1073 into ML-KULeuven:master Jul 16, 2025
3 checks passed
@rmanhaeve

Copy link
Copy Markdown
Contributor

Thank you for this fix!

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