Skip to content

Comments

Bug fix: Correct a handful of typos/JS errors in decompositions#699

Merged
fdwr merged 2 commits intowebmachinelearning:mainfrom
inexorabletash:emu-fixes
Jun 5, 2024
Merged

Bug fix: Correct a handful of typos/JS errors in decompositions#699
fdwr merged 2 commits intowebmachinelearning:mainfrom
inexorabletash:emu-fixes

Conversation

@inexorabletash
Copy link
Contributor

@inexorabletash inexorabletash commented Jun 3, 2024

Typos of "builder", incorrect arrow function syntax.

No normative changes.


Preview | Diff

Typos of "builder", incorrect arrow function syntax.

No normative changes.
@inexorabletash
Copy link
Contributor Author

@zolkis and @huningxin - can you take a look?

Note that this just fixes some tiny and obvious errors. I have some follow up work that ensures that the decompositions don't reference undefined variables, but wanted to keep that separate.

Copy link
Collaborator

@zolkis zolkis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

Copy link
Collaborator

@fdwr fdwr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Contributor

@huningxin huningxin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@huningxin
Copy link
Contributor

@inexorabletash , could you please resolve the merge conflicts? So, I can merge it, thanks!

@inexorabletash
Copy link
Contributor Author

Conflicts resolved (hopefully) - going to bed now, I'll fix in the morning if you notice any glitches.

@inexorabletash
Copy link
Contributor Author

@huningxin this should be good to merge now

Copy link
Collaborator

@fdwr fdwr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 LGTM still.

@inexorabletash
Copy link
Contributor Author

Merge when you're ready!

@fdwr fdwr merged commit f133ff0 into webmachinelearning:main Jun 5, 2024
github-actions bot added a commit that referenced this pull request Jun 5, 2024
SHA: f133ff0
Reason: push, by fdwr

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@inexorabletash inexorabletash deleted the emu-fixes branch June 5, 2024 04:07
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.

4 participants