Skip to content

adding meeting notes from previous meeting and rng proprosal in an ma…#157

Open
bparbhu wants to merge 4 commits into
mainfrom
docs/update_meeting_notes_and_add_rng_proposal_markdown
Open

adding meeting notes from previous meeting and rng proprosal in an ma…#157
bparbhu wants to merge 4 commits into
mainfrom
docs/update_meeting_notes_and_add_rng_proposal_markdown

Conversation

@bparbhu

@bparbhu bparbhu commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

…rkdown format to share

@bparbhu
bparbhu requested review from Craigacp and andife July 7, 2026 03:34
@bparbhu bparbhu self-assigned this Jul 7, 2026

@Craigacp Craigacp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM aside from the things I commented on.


### Type Constraints

`T` in (`tensor(bfloat16)`, `tensor(double)`, `tensor(float)`, `tensor(float16)`):

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We need RandomUniform to range over all tensor types so we can make it generate bits.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Understood, I'll make the corrections. Apologies for the late reply to this.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Made the change please take a look


**T2** in (`tensor(bfloat16)`, `tensor(double)`, `tensor(float)`, `tensor(float16)`):

Constrain output types to float tensors.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Output types can be any tensor type.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Understood, I'll make the change.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Made the change please take a look

The discussion covered plans for integrating the Threefry implementation into the ONNX codebase and broader repository organization practices. Adam discussed determining the appropriate location for new implementation code, while both participants exchanged experiences managing work across personal and organizational GitHub repositories.

### Intellectual Property and Open Source
Adam and Brian discussed intellectual property considerations surrounding open-source contributions, including Oracle's internal policies and the advantages of contributing through organization-owned repositories. They also reflected on sustainability challenges facing open-source projects and the impact of cloud vendors on project funding and governance.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

s/Oracle's internal policies/Oracle's open source policies/

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah sorry about that I will omit this section.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's fine to stay in, we definitely discussed it and the way that Oracle contributes to open source projects is public (and similar to most large corporations). Just needs to be specific about what policy.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Made the change please take a look at let me know what you think.

@bparbhu
bparbhu requested a review from Craigacp July 24, 2026 03:45
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