Page MenuHomePhabricator
Differential D404

Simplex Experiment console
AcceptedPublic
Actions

Authored by galmonte on Mar 18 2021, 2:01 PM.

Details

Reviewers
jcmcdonald
ardunster
Summary

Overload insertion for SimplexResult, default constructor for Simplex, build experiment console

Test Plan

Builds, goldilocks tests unaffected

Diff Detail

Repository
rS SIMPLEXpress
Branch
simplexdemoer
Lint
Lint Passed

Event Timeline

ardunster created this revision.Mar 18 2021, 2:01 PM
Herald added projects: Programming [Dept], SIMPLEXpress [Project]. · View Herald TranscriptMar 18 2021, 2:01 PM
ardunster updated this revision to Diff 1876.Mar 18 2021, 2:02 PM
  • Change comments to default experimenter to off
jenkins added a comment.Mar 18 2021, 2:02 PM

Build is green

Building SIMPLEXpress on mpm-bionic-0003y7zr6hz9p as build 118.
Running tests.

See https://jenkins.mousepawmedia.net:8449/job/simplexpress_build/118/ for more details.

Restricted Application completed remote builds in Restricted Buildable.Mar 18 2021, 2:02 PM
jenkins added a comment.Mar 18 2021, 2:03 PM

Build is green

Building SIMPLEXpress on mpm-bionic-0003y8t99yrw0 as build 119.
Running tests.

See https://jenkins.mousepawmedia.net:8449/job/simplexpress_build/119/ for more details.

Restricted Application completed remote builds in Restricted Buildable.Mar 18 2021, 2:03 PM
ardunster requested review of this revision.Mar 18 2021, 2:03 PM
ardunster added a subscriber: jcmcdonald.Aug 22 2021, 9:34 AM

@jcmcdonald Also just noticed this also never got reviewed.

Herald added a project: Announce [Control]. · View Herald TranscriptAug 22 2021, 9:34 AM
jcmcdonald accepted this revision.Aug 24 2021, 4:39 PM
jcmcdonald edited reviewers, added: jcmcdonald; removed: wdede, memateo, mahussain.
jcmcdonald added a subscriber: rperez.

@ardunster Luckily, it looks land-worthy right now!

@rperez could you commandeer, rebase, and land this? Thanks.

This revision is now accepted and ready to land.Aug 24 2021, 4:40 PM
galmonte commandeered this revision.Aug 31 2022, 4:23 PM
galmonte added a reviewer: ardunster.

Revision Contents
Changeset List

PathSize
simplexpress-source/
include/
simplexpress/
10 lines
src/
25 lines
simplexpress-tester/
90 lines
16 lines

Diff 1876

View Options

simplexpress-source/include/simplexpress/simplex.hpp

Loading...
View Options

simplexpress-source/src/simplex.cpp

Loading...
View Options

simplexpress-tester/experimenter.hpp

Loading...
View Options

simplexpress-tester/main.cpp

Loading...
Copyright (C) 2019 MousePaw Media. All Rights Reserved. · Community Rules · Terms of Use · Privacy Policy · Terms of Development