egg changed the topic of #principia to: READ THE FAQ: http://goo.gl/gMZF9H; The current version is Cesàro. We currently target both 1.2.2 and 1.3. <scott_manley> anyone that doubts the wisdom of retrograde bop needs to get the hell out | https://xkcd.com/323/
awang has quit [Read error: -0x1: UNKNOWN ERROR CODE (0001)]
awang has joined #principia
MrSavage has joined #principia
e_14159 has quit [Ping timeout: 183 seconds]
e_14159 has joined #principia
icefire has quit [Remote host closed the connection]
uj8efdjkfdshf has joined #principia
Rokker has joined #principia
MrSavage has quit [Remote host closed the connection]
lamont_ has joined #principia
lamont has quit [Ping timeout: 186 seconds]
lamont_ is now known as lamont
awang has quit [Killed (NickServ (GHOST command used by awang_))]
uj8efdjkfdshf has quit [Quit: Connection closed for inactivity]
uj8efdjkfdshf has joined #principia
awang has joined #principia
WetLemons has quit [Ping timeout: 186 seconds]
WetLemons has joined #principia
WetLemons has quit [Ping timeout: 198 seconds]
ferram4 has quit [Ping timeout: 186 seconds]
ferram4 has joined #principia
Asymptote has joined #principia
awang has quit [Ping timeout: 204 seconds]
Rokker has quit [Quit: Connection closed for inactivity]
icefire has joined #principia
NolanSyKinsley has quit [Remote host closed the connection]
icefire has quit [Quit: Leaving]
Rokker has joined #principia
Hypergolic_Skunk has joined #principia
MrSavage has joined #principia
<GH> [Principia] eggrobin deleted Cesàro-MacOS at 6ef3b02: https://git.io/vdUZg
<GH> [Principia] pleroy opened issue #1572: Use Product and friends instead of decltype https://git.io/vdUn5
<GH> [Principia] pleroy opened pull request #1573: Replace decltype by Product or Quotient (master...1572) https://git.io/vdUWX
WetLemons has joined #principia
Technicalfool_ has quit [Ping timeout: 198 seconds]
Technicalfool has joined #principia
awang has joined #principia
Jesin has joined #principia
uj8efdjkfdshf has quit [Quit: Connection closed for inactivity]
Jesin has quit [Quit: Leaving]
Jesin has joined #principia
<awang> Clang is warning about comparing principia::serialization::Frame_SolarSystemTag and principia::serialization::Frame_PluginTag
<awang> Does that make sense?
<awang> Also, is conversion to enum classes on the todo list?
<egg|zzz|egg> awang: hm, interesting warning, but it depends where this is, might be intended
<egg|zzz|egg> awang: and we use enum classes sparingly, they're a bit clumsy
<egg|zzz|egg> (like the rest of the #$%#%!#$ language arguably :-p)
<awang> Full stack trace: https://hastebin.com/hojekekuma.pas
<awang> At least it is pretty short for a C++ stack trace :P
<awang> What, no enum classes?
<awang> But
<awang> But
<egg|zzz|egg> probably semi-intended
<awang> type safety?
<egg|zzz|egg> yeah, but no iteration, no array subscripting, etc.
<egg|zzz|egg> also our enums typically live in classes so that they end up being used as foo::bar anyway
<egg|zzz|egg> Foo::Bar rather
<egg|zzz|egg> awang: btw, #1572 looks like the issue you were reporting, you can tag yourself so we remember to credit you for reporting it
<Qboid> [#1572] title: Use Product and friends instead of decltype | | https://github.com/mockingbirdnest/Principia/issues/1572
<egg|zzz|egg> (or you might even make a PR to fix it!)
<awang> True, iterating/subscripting with enum classes would be awkward at best
<egg|zzz|egg> ah nevermind phl made a pull request already
<awang> Dangit, I was just about to start working on that
<awang> Better than after I had already started working on it, I guess
<egg|zzz|egg> :D
<egg|zzz|egg> awang: I'm sure you can see other warnings though
<egg|zzz|egg> awang: maybe do a pass over the pessimizing moves/redundant moves
<egg|zzz|egg> those are bad
<awang> Yep, I see plenty
<awang> Although there actually weren't many for "make release"
<awang> And only 3 or 4 bad std::moves
<awang> Lot more warnings for plain "make"
<awang> Although those seem to be more in the testing code
<egg|zzz|egg> make Makemake
<egg|zzz|egg> UmbralRaptor: ^
<UmbralRaptor> <_<
<UmbralRaptor> Also, getting anything to build in c is a nightmare.
<egg|zzz|egg> good thing this isn't C
Hypergolic_Skunk has quit [Quit: Connection closed for inactivity]
<awang> Dangit, why does Google Mock not support override :(
<awang> Lot of warnings from that
<awang> What's with the #pragma onces appearing in .cpp files?
<UmbralRaptor> egg: well, anything sufficiently c-like. (C++, maybe C#. Definitely not Perl)
<egg|zzz|egg> awang: if there are pragma onces in cpp files, that's because someone was daft and typed that mechanically
<egg|zzz|egg> awang: yeah, gmock and override is annoyingly noisy
<awang> Are unused variables worth removing?
<awang> And when going through warnings, should I leave code in the deps/ directory alone?
<egg|zzz|egg> awang: nah, ignore unused variables for now, a lot of that is stuff for testing, documents what the expression is
<awang> Excuse to used [[maybe_unused]]?
<egg|zzz|egg> hmm, not sure whether that'll work with MSVC
<awang> What version of MSVC are you targeting?
<egg|zzz|egg> 2015, bug in 2017 prevents switching
<awang> :(
<awang> Bug still present in 15.3?
<egg|zzz|egg> awang: please make one pull request per type of warning btw, so that if one is spammy we can just discard that without affecting the rest
<awang> Since that was released relatively recently, I think?
<egg|zzz|egg> yup
<awang> Alright, will do
<awang> Awww
<awang> What's the bug?
<egg|zzz|egg> we reported it :-p
<egg|zzz|egg> awang: also, there are a lot of spammy warnings, so don't waste too much time on questionable stuff; std::move is a good one tho
<awang> Ouch, that's pretty bad
<awang> Hey, I have to procrastinate somehow :P
<egg|zzz|egg> :D
* egg|zzz|egg meanwhile flies an awacs in 630 km/h winds as part of the plan for the reunification of the ferram4 and egg sectors
<awang> Also, no -Weverything?
<awang> Slightly disappointed :P
<egg|zzz|egg> awang: so the makefile is there to make a linux build mostly, we develop on msvc
<awang> Oh right
<egg|zzz|egg> that's why all those clang warnings go unfixed too
<awang> I knew that
<egg|zzz|egg> we see them once a month and don't have time because we're releasing
<awang> Should I fix easy warnings stemming from code in deps/?
<awang> e.g. NULL vs nullptr should be a relatively easy sed fix
<awang> But those are from protobuf/glog code
<egg|zzz|egg> nope don't touch that
<awang> Alright
<egg|zzz|egg> there's a reason why that code is that way too, needs to be compatible
<awang> Compatible? I thought you're targeting a pretty modern toolchain
<egg|zzz|egg> awang: protobuf and glog are not us
<egg|zzz|egg> awang: and some people that use those might be using C++98
<awang> egg|zzz|egg: Right, but I thought you were using your own fork of those
<awang> What about NULL/etc in your own code?
Asymptote has quit [Read error: Connection reset by peer]
<awang> Also, do you know how badly link time optimization would break things?
<egg|zzz|egg> awang: there should be no NULL in our own code
<egg|zzz|egg> we have link time optimization on on windows I think? I don't know
<egg|zzz|egg> awang: we're using our own fork for tiny patches that we should upstream, we're *NOT* maintaining protobuf
<awang> Ah, ok
<egg|zzz|egg> that's ten more full-time jobs that we're willing to dedicate to principia :-p
<egg|zzz|egg> s/that/than
<Qboid> egg|zzz|egg meant to say: than's ten more full-time jobs that we're willing to dedicate to principia :-p
<GH> [Principia] pleroy closed pull request #1573: Replace decltype by Product or Quotient (master...1572) https://git.io/vdUWX
<GH> [Principia] pleroy closed issue #1572: Use Product and friends instead of decltype https://git.io/vdUn5
<egg|zzz|egg> awang: are you awang on github?
<awang> No
<awang> aw1621107
<awang> awang was taken, I think
<awang> Looks like there's a copy being made for a loop variable
<awang> ./integrators/ordinary_differential_equations_body.hpp:47:19: warning: loop variable 'p'
<awang> of type 'const principia::serialization::DoublePrecision' creates a copy from type
<awang> 'const principia::serialization::DoublePrecision' [-Wrange-loop-analysis]
<awang> for (auto const p : message.position()) {
<awang> Is that intentional?
<awang> Also, what about warnings about deprecated definition of implicit copy/move ctors/etc.?
<awang> Oh ho, this is new
<awang> clang apparently has thread safety warnings now
NolanSyKinsley has joined #principia
<egg|zzz|egg> awang: yeah, I know about the thread safety warning (we try to put annotations like GUARDED_BY), requires the libs to have corresponding annotations though.
<egg|zzz|egg> awang: uhm the auto const p doesn't look intentional
Jesin has quit [Quit: Leaving]
Rokker has quit [Quit: Connection closed for inactivity]
<egg|zzz|egg> UmbralRaptor: can you use the principia libraries for eggsoplanets :-p
<GH> [Principia] pleroy opened pull request #1574: Fix plotting of predictions (master...Predictions) https://git.io/vdUiD
<UmbralRaptor> Probably?
<egg|zzz|egg> UmbralRaptor: clearly you should do that (and 古在！)
<UmbralRaptor> Uh, still mostly getting beat up by the problems in Goldstein at the moment.
<UmbralRaptor> Also poking at eggsoplanet observations.
Jesin has joined #principia
<UmbralRaptor> (which sadly involve very little physics)
<egg|zzz|egg> UmbralRaptor: try poking bofh for eggsplanations of the more abstract aspects of the structures underlying this stuff (the exercises themselves mostly look like a lot of calculation, but knowing what goes on at a higher level can be useful)
<egg|zzz|egg> (or me but I should zzz)
<kmath> <mnxmnkmnd> Behavior gets distributed among the arms... ...somehow. Cool animals.
<UmbralRaptor> egg|zzz|egg: only sleeping because you stopped at an MSc?
<egg|zzz|egg> hah