multisn8 multisamplednight
  • wherever cute entities are
  • i need E

  • Joined on 2024-01-01
multisamplednight commented on pull request katzen-cafe/iowo#6 2024-01-19 10:47:44 +00:00
feat: Graph IR

I do think that there should only be at most one value because that might make some stuff a lot more elegant

  • What one value would you want for referring to both a specific instruction and…
multisamplednight commented on pull request katzen-cafe/iowo#6 2024-01-19 10:42:32 +00:00
feat: Graph IR

do you mean like "written down in code"? or am i misunderstanding?

No, you're correct. I'll update the docs to be more specific.

multisamplednight commented on pull request katzen-cafe/iowo#6 2024-01-19 10:41:53 +00:00
feat: Graph IR

Those would be still in the same GraphIr.

multisamplednight commented on pull request katzen-cafe/iowo#6 2024-01-19 10:41:29 +00:00
feat: Graph IR

Technically, yes. However, since the edges directly below refer to the instruction IDs directly, I believe listing the IDs of the instructions explicitly is worth it, as otherwise one would keep…

multisamplednight commented on pull request katzen-cafe/iowo#6 2024-01-19 10:38:07 +00:00
feat: Graph IR

I'll clarify the u16 part on the docs of SocketIdx.

multisamplednight commented on pull request katzen-cafe/iowo#6 2024-01-19 10:37:53 +00:00
feat: Graph IR

u16 is quite small for a huge graph

The SocketIdx only refers to the sockets on one instruction. So one instruction could hold at most 2^16 - 1 = 65535 inputs and outputs each at…

multisamplednight commented on pull request katzen-cafe/iowo#6 2024-01-19 10:33:43 +00:00
feat: Graph IR

To whom would the runtime "return" the metadata?

multisamplednight commented on pull request katzen-cafe/iowo#6 2024-01-19 10:33:10 +00:00
feat: Graph IR

The span is the ID. It allows the executor to refer back to the source in errors or warnings.

multisamplednight commented on pull request katzen-cafe/iowo#2 2024-01-19 02:06:20 +00:00
docs: some initial groundwork

Personally I believe it's unlikely we'll have any kind of teams, rather just a few maintainers and contributors. In both cases, with or without teams, answering the same questions every single…

multisamplednight commented on issue katzen-cafe/iowo#4 2024-01-19 02:01:52 +00:00
Keyword inputs and outputs

"ASAP" is definitely not necessary, lol. Do it anytime you want. And if you figure out that you don't, then we'll think of something else.

multisamplednight commented on pull request katzen-cafe/iowo#2 2024-01-18 21:48:43 +00:00
docs: some initial groundwork

@schrottkatze Thank you for the review!

multisamplednight commented on pull request katzen-cafe/iowo#2 2024-01-18 21:32:56 +00:00
docs: some initial groundwork

Hm. Thinking about it, Evaluation seems generic enough. I'll use that then.

multisamplednight commented on pull request katzen-cafe/iowo#2 2024-01-18 21:31:47 +00:00
docs: some initial groundwork

They are not necessary. In fact, the whole file is not necessary.

However, it serves a different purpose: It tries to lend a helping hand to whatever contributor might come along, regardless of…

multisamplednight commented on pull request katzen-cafe/iowo#2 2024-01-18 21:29:24 +00:00
docs: some initial groundwork

Those are all individual phases of execution.

multisamplednight commented on pull request katzen-cafe/iowo#2 2024-01-18 21:28:42 +00:00
docs: some initial groundwork

It does in some markdown viewers such as Obsidian or GitHub, but apparently not on Forgejo. I'll remove it then.

multisamplednight commented on issue katzen-cafe/iowo#4 2024-01-18 21:27:22 +00:00
Keyword inputs and outputs

From @schrottkatze in #2:

This is why I proposed using lists, dictionaries etc. in #4, to make this easier/cleaner

We'd need a type system before we could implement this then.

multisamplednight commented on pull request katzen-cafe/iowo#2 2024-01-18 21:25:39 +00:00
docs: some initial groundwork

Tbh then I'd rather completely remove the dots at the end, would that be fine, too?

multisamplednight commented on pull request katzen-cafe/iowo#2 2024-01-18 19:18:59 +00:00
docs: some initial groundwork

(ok did that)

multisamplednight commented on pull request katzen-cafe/iowo#2 2024-01-18 19:17:23 +00:00
docs: some initial groundwork

(fwiw please don't merge if you happen to approve — need to rebase onto main first)

multisamplednight commented on pull request katzen-cafe/iowo#6 2024-01-18 19:15:40 +00:00
feat: Graph IR

Fwiw for some reason the Conversation tab (this one) says that the broken commit is still contained in this PR. However, the Commits tab does not, which should be correct as I just force-pushed…