I'll still disagree here. I don't understand your desire to not allow
access to either side of the buffer, especially with the simple
implementation @scopatz and you suggested. I think the BatchReactor as
written is a perfectly acceptable implementation. @scopatz can arbitrate as
necessary.


On Mon, Dec 9, 2013 at 11:17 AM, Robert Carlsen <notifications@github.com>wrote:

> In src/Models/BatchReactor/batch_reactor.cc:
>
> >  }
> >
> > -//- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
> > -void BatchReactor::OffloadBatch() {
> > -  using cyclus::Context;
> > +//- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
> > +void BatchReactor::AddBatches_(cyclus::Material::Ptr mat) {
>
> Note that we no longer need the Pop(int back = true) for ResourceBuff. The
> idea is to just keep the spare, unbatched amount in a spare_ class member.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cyclus/cycamore/pull/162/files#r8203970>
> .
>



-- 
Matthew Gidden
Ph.D. Candidate, Nuclear Engineering
The University of Wisconsin -- Madison
Ph. 225.892.3192