Skip to content

Rework for New StreamingLearnerInvestigator - #29

Draft
BenCarter44 wants to merge 2 commits into
develfrom
feature/rework-StreamingLearnerInvestigator
Draft

Rework for New StreamingLearnerInvestigator#29
BenCarter44 wants to merge 2 commits into
develfrom
feature/rework-StreamingLearnerInvestigator

Conversation

@BenCarter44

@BenCarter44 BenCarter44 commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

This PR will fulfill issue #28 .

Here below is a draft spec of the new StreamingLearnerInvestigator.

A user simply subclasses it, and can add their own logic (and their own ROSE learner) to the callbacks. This StreamingLearnerInvestigator simply provides the batch support and plumbing.

Will need to be tested and cleaned up....

(Tests right now aren't expected to work, as this is a draft spec)

@BenCarter44 BenCarter44 changed the title Draft spec of new StreamingLearnerInvestigator Rework for New StreamingLearnerInvestigator Aug 27, 2026
@BenCarter44 BenCarter44 linked an issue Aug 27, 2026 that may be closed by this pull request
@andre-merzky

Copy link
Copy Markdown
Member

I know it is early, but quick question: why are we concerned about batch size on this level of abstraction? The Orbit layer is already batching requests and thus optimizing network latency impact. What is the purpose of additional higher level batching?

@BenCarter44

Copy link
Copy Markdown
Collaborator Author

Hello Andre,

I'm not as familiar with ORBIT's batching. Could you describe more?

I'm highly sure the batching here in StreamingLearnerInvestigator is different. The investigator still gets input one-by-one. The "batch" here is simply a group of investigator inputs sent to the active learner.

It's role is to simply answer: "instead of triggering an AL on every input, trigger it every X inputs, and group them up as you go".

Note that this batching does not affect inference, as inference runs on each input one-by-one to the investigator.

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.

Rework the StreamingLearnerInvestigator

2 participants