egg|zzz|egg changed the topic of #principia to: READ THE FAQ: http://goo.gl/gMZF9H; The current version is Clifford. We currently target 1.2.2, and 1.3.1. <scott_manley> anyone that doubts the wisdom of retrograde bop needs to get the hell out | https://xkcd.com/323/
CommandoDiamond has joined #principia
Wetmelon has joined #principia
Moistmelon has quit [Ping timeout: 186 seconds]
LogBen has quit [Quit: WeeChat 1.4]
awang has quit [Ping timeout: 383 seconds]
awang has joined #principia
icefire has joined #principia
<lamont> heh now i’ve forgotten how to build Principia...
icefire has quit [Quit: Leaving]
<lamont> welp or it looks like it doesn’t compile on 10.12
<lamont> In file included from tools/generate_configuration.cpp:13:
<lamont> In file included from ./physics/solar_system.hpp:11:
<lamont> In file included from ./physics/continuous_trajectory.hpp:10:
<lamont> In file included from ./numerics/чебышёв_series.hpp:89:
<lamont> In file included from ./numerics/чебышёв_series_body.hpp:10:
<lamont> In file included from ./numerics/fixed_arrays.hpp:125:
<lamont> ./numerics/fixed_arrays_body.hpp:122:46: error: use 'template' keyword to treat 'Row' as a dependent template name
<lamont> typename FixedMatrix<Scalar, rows, columns>::Row<r>
<lamont> ^
<lamont> template
<awang> lamont: Fixed in master?
<awang> I think?
<awang> I submitted a PR, thought it was merged
<lamont> yeah the branch i’m trying to compile is behind master
<lamont> i’ll have to try to rebase
icefire has joined #principia
<lamont> ksp_plugin/interface_external.cpp:114:51: error: source file is not valid UTF-8
<lamont> if (manoeuvre_index >= flight_plan.number_of_man<9C>uvres()) {
<lamont> now that’s funny
<awang> lamont: That's a new one
<lamont> that’s in the code egg just wrote for me
icefire has quit [Read error: Connection reset by peer]
<UmbralRaptor> That's a reasonable error, especially from egg,
<lamont> yeah that was easy enough to figure out
<lamont> In file included from astronomy/solar_system_dynamics_test.cpp:28:
<lamont> In file included from ./testing_utilities/solar_system_factory.hpp:113:
<lamont> ./testing_utilities/solar_system_factory_body.hpp:21:1: error: 'static' can only be specified inside the class definition
<lamont> static void SolarSystemFactory::AdjustAccuracy(
<lamont> ^~~~~~~
<awang> More MSVC noncompliance fun?
Mike` has quit [Ping timeout: 207 seconds]
Mike` has joined #principia
Wetmelon has quit [Ping timeout: 186 seconds]
CommandoDiamond_ has joined #principia
CommandoDiamond has quit [Ping timeout: 186 seconds]
NolanSyKinsley has quit [Remote host closed the connection]
CommandoDiamond_ has quit [Read error: Connection reset by peer]
Wetmelon has joined #principia
Hypergolic_Skunk has joined #principia
egg|laptop|egg has joined #principia
icefire has joined #principia
Hypergolic_Skunk has quit [Quit: Connection closed for inactivity]
<GH> [Principia] pleroy closed pull request #1704: Use polynomials to implement the continuous trajectories (master...ContinuousTrajectory) https://git.io/vNFb6
NolanSyKinsley has joined #principia
<GH> [Principia] eggrobin opened pull request #1707: Report all failures (master...report-all-failures) https://git.io/vNxUQ
<egg|anbo|egg> lamont: oh right I should have encoded the file as UTF-8. I'll fix that.
<egg|anbo|egg> lamont: aaand I'll fix the stray static I guess
<GH> [Principia] eggrobin opened pull request #1708: Remove stray static (master...static-out-of-line) https://git.io/vNxYF
<egg|anbo|egg> lamont: the lamont-endpoints branch should have all your issues cleaned up now
<GH> [Principia] eggrobin commented on issue #1708: retest this please (a new kind of flakiness...) https://git.io/vNxGo
<egg|laptop|egg> lamont: have you managed to call the thing successfully?
<GH> [Principia] pleroy opened pull request #1709: Filter out the medians from the benchmarks (master...Median) https://git.io/vNxc9
<GH> [Principia] pleroy closed pull request #1708: Remove stray static (master...static-out-of-line) https://git.io/vNxYF
<GH> [Principia] pleroy pushed 2 new commits to master: https://git.io/vNxc7
<GH> Principia/master e5805ef Pascal Leroy: Merge pull request #1708 from eggrobin/static-out-of-line...
<GH> Principia/master e1e8d43 Robin Leroy: Fix a build error reported by lamont
<GH> [Principia] pleroy closed pull request #1707: Report all failures (master...report-all-failures) https://git.io/vNxUQ
<GH> [Principia] pleroy pushed 4 new commits to master: https://git.io/vNxc5
<GH> Principia/master fa06280 Robin Leroy: run all tests even if they die or fail
<GH> Principia/master 3b952f7 Robin Leroy: make a test crash and many tests fail
<GH> Principia/master 643cb5b Robin Leroy: Revert "make a test crash and many tests fail"...
<lamont> gotta wait for the coffee to settle this morning
<egg|laptop|egg> :D
stratochief_ has joined #principia
stratochief has quit [Ping timeout: 383 seconds]
stratochief_ is now known as stratochief
<lamont> In file included from tools/generate_configuration.cpp:13:
<lamont> In file included from ./physics/solar_system.hpp:12:
<lamont> ./physics/continuous_trajectory.hpp:201:21: error: declaration of 'Frame' shadows template parameter
<lamont> template<typename Frame>
<lamont> ^
<lamont> ./physics/continuous_trajectory.hpp:31:19: note: template parameter is declared here
<lamont> template<typename Frame>
<lamont> ^
<lamont> fatal error: too many errors emitted, stopping now [-ferror-limit=]
<lamont> 6 warnings and 2 errors generated.
<lamont> make: *** [obj/tools/generate_configuration.o] Error 1
<lamont> tried a rebase on origin/master and got the same
<awang> Yeah, it's erroring out in master
<awang> Not just egg's code
<awang> ./physics/continuous_trajectory_body.hpp:285:53: error: use 'template' keyword to treat
<awang> 'ReadFromMessage' as a dependent template name
<awang> There's that too
<awang> Polynomial<Displacement<Frame>, Instant>::ReadFromMessage<
<awang> ^
<awang> template
<awang> Might be a bit tricker to solve that the other error?
<awang> Since this is in a place where the function is being used, instead of just the declaration
<egg|laptop|egg> lamont: awang: argh.
<egg|laptop|egg> awang: can you look into those, it's awkward for me to build on linux from here
<egg|laptop|egg> I could build on mac I guess, but then I'd have to open my corp laptop and I might end up working which I want to avoid on weekends
<awang> egg|laptop|egg: Sure
<awang> egg|laptop|egg: What do you want me to do with the shadowed template parameter?
<awang> I just added a trailing underscore to see if the rest would build
<awang> But idk what you'd normally do
<awang> The other one might require a bit of back and forth, like the last dependent template name thing
<awang> Since I don't have VS 2015 installed
<awang> Hey, a third one
<awang> physics/continuous_trajectory_test.cpp:106:3: error: missing 'typename' prior to
<awang> dependent type name 'P::Coefficients'
<awang> P::Coefficients const coefficients = {Displacement<Frame>(),
<awang> ^~~~~~~~~~~~~~~
<awang> typename
<awang> Hmmm
<awang> physics/continuous_trajectory_test.cpp:120:10: error: use of undeclared identifier
<awang> 'degree_'; did you mean 'degree'?
<awang> return degree_;
<awang> ^~~~~~~
<awang> degree
<awang> Ah, last one is because degree_ is private
<egg|laptop|egg> awang: but the friend declaration should solve that?
<awang> egg|laptop|egg: Yeah, I see
<awang> idk what's going on
<awang> I had to change the template<typename Frame> part at least
<awang> Since template parameters can't shadow other ones
<awang> idk if that plays into it
<egg|laptop|egg> awang: try removing line 201 and saying friend class TestableContinuousTrajectory<Frame>
<egg|laptop|egg> awang: there should be no need for cross-frame friendship
<awang> Already tried that, Clang complained about trying to specialize a non-template class
<egg|laptop|egg> wah
<awang> ./physics/continuous_trajectory.hpp:201:16: error: explicit specialization of
<awang> non-template class 'TestableContinuousTrajectory'
<awang> friend class TestableContinuousTrajectory<Frame>;
<egg|laptop|egg> O_o
<awang> I'm trying a forward declaration first now
<egg|laptop|egg> awang: forward-declare
<egg|laptop|egg> yeah
<awang> Want me to place it right above the friend declaration or at the top of the file?
<egg|laptop|egg> uh, above the class definition
<egg|laptop|egg> not inside it, that wouldn't declare the right thing
<egg|laptop|egg> (so above class ContinuousTrajectory)
<egg|laptop|egg> I should install clang for visual studio on this laptop, it works pretty well now
<awang> Do you prefer above or below the using declarations?
<egg|laptop|egg> below
<awang> Er
<awang> Wrong channel >_>
<awang> But yeah
<awang> Forward declaration didn't work
<awang> I'm a bit stumped
<awang> Unless you want to make degree_ protected
<awang> Which I'm guessing should be avoided
<awang> Pushed my work
<awang> Can you check to make sure I didn't do anything dumb?
<awang> Most recent 4 commits of that branch
<GH> [Principia] pleroy pushed 2 new commits to master: https://git.io/vNxuX
<GH> Principia/master 47ee537 pleroy: Filter out the medians from the benchmarks.
<GH> Principia/master 58e59ba Pascal Leroy: Merge pull request #1709 from pleroy/Median...
<egg|anbo|egg> awang: use of undeclared identifier 'degree_age_'; unqualified lookup into dependent bases of class template 'TestableContinuousTrajectory' is a Microsoft extension
<egg|anbo|egg> all hail clang-cl
<egg|anbo|egg> awang: should be this->degree_age_ I think?
<awang> egg|anbo|egg: idk? I'm not familiar with that corner of C++
<awang> And my Clang didn't get that far
<GH> [Principia] pleroy commented on issue #1703: Decoded stack trace:... https://git.io/vNx2O
<egg|laptop|egg> awang: try this->degree_age_
<GH> [Principia] pleroy commented on issue #1703: Decoded stack trace:... https://git.io/vNx2O
<egg|laptop|egg> it will fix the thing
<egg|laptop|egg> (that's what clang-cl suggests, so it better fix it)
<awang> This is a different degree_
<awang> It's literally just degree_
<GH> [Principia] pleroy commented on issue #1677: #1611 was supposed to fix this "once and for all". Apparently it didn't. https://git.io/vNx2B
<egg|laptop|egg> awang: yeah, same, this->degree_
<egg|laptop|egg> awang: there are a bunch of these, it's all the same problem
<awang> Ah
<egg|laptop|egg> you refer to something in a class which is not you (a friend or a base), and which is a template, therefore you need this->
<awang> One of the times Clang's venerable error messages have failed me
<awang> I didn't know that
<egg|laptop|egg> because the compiler can't know what your template friends or bases might contain until it sees the specialization
<awang> Haven't done enough deriving from/friending of template classes, apparently
<egg|laptop|egg> (all specializations)
<egg|laptop|egg> hah
<awang> I see
<awang> Wonder if I should file a Clang bug, because the error message wasn't helpful
<GH> [Principia] eggrobin commented on issue #1677: It was supposed to fix it for the flight planner window, and as far as I know it did (once and for all!).... https://git.io/vNx26
<awang> 谢谢
C_Z has joined #principia
C_Z has quit [Client Quit]
<GH> [Principia] aw1621107 opened pull request #1710: Clang fixes (master...clang-fixes-1) https://git.io/vNxaI
<lamont> Okay pardon my french, but Hamilton can take his Hamiltonians and cram them up his ass…
<GH> [Principia] eggrobin closed pull request #1710: Clang fixes (master...clang-fixes-1) https://git.io/vNxaI
<egg|anbo|egg> lamont: what happened?