bushwallyta271828 / logic_graph_scalable

This repository should support the logicgraph.dev website, taking over from logic_graph.
MIT License
0 stars 0 forks source link

Let's delete claims! #145

Closed bushwallyta271828 closed 6 months ago

bushwallyta271828 commented 6 months ago

TODO:

bushwallyta271828 commented 6 months ago

TODO:

bushwallyta271828 commented 6 months ago

(Update: conditional probability parsing is still being bizarre with 0, and Docker also started behaving bizarrely and I had to clear the cache.)

bushwallyta271828 commented 6 months ago

Oh wait a minute, I think I see the bug: 0 is being evaluated as falsy! I need to explicitly check if my objects equal null!

bushwallyta271828 commented 6 months ago

It works almost all the time, but it occasionally crashes. I'm getting a RangeError: Invalid array length at Array.push.

bushwallyta271828 commented 6 months ago

TODO:

bushwallyta271828 commented 6 months ago

Now it's crashing on (aaa implies aaa) implies aaa.

bushwallyta271828 commented 6 months ago

Okay, I think the parser is actually working! I can't get it to crash or give strange answers after some playing around with it. Let's finally move on!

That said, the syntax text has a few bugs.

bushwallyta271828 commented 6 months ago

Those bugs are now fixed. TODO:

bushwallyta271828 commented 6 months ago

TODO:

TODO for future branches:

bushwallyta271828 commented 6 months ago

TODO:

TODO for future branches:

bushwallyta271828 commented 6 months ago

TODO for future branches:

bushwallyta271828 commented 6 months ago

Looks good! Let's merge!

bushwallyta271828 commented 6 months ago

I'm actually going to drop "Go through parsing files and use "never" to get type checker to verify that cases are exhaustive." I need to move on from the parser. If it's giving me problems in the future I can always go back and clean it up.