TarekNasser[m] has quit [Quit: You have been kicked for being idle]
_slack_mlpack_22 has quit [Quit: You have been kicked for being idle]
_slack_mlpack_22 has joined #mlpack
<rcurtin[m]>
I think I may have mentioned this earlier---the place that hosts mlpack.org is shutting down, and so I needed to find a new colocation provider. I'll be moving the system on Friday, probably from something like 1200-1400 UTC (plus time to solve whatever problems I have :)). when it comes back up, it'll have a new IP, but everything else should be the same
<ShubhamAgrawal[m>
rcurtin: When we check earlier version of Gonum, we use list function. Now it only returns package name only not version unless we use module approach introduced in Go. Should we skip check for explicit version, and just pass 0.0.0 if its found otherwise error?
<rcurtin[m]>
I see---thank you for digging into that! (and all the other build fixes!)
<rcurtin[m]>
given that we don't have a specific requirement for the version of Gonum, I suppose it suffices to simply check that it is available
<rcurtin[m]>
so I think the solution you suggested is fine, but actually we could even drop the Gonum version variable altogether since it isn't used
<rcurtin[m]>
alternately, if you wanted, you could use the module approach to get the version number, to make it easier later if we need to enforce restrictions on the version of gonum or anything
<rcurtin[m]>
so personally I think either approach is fine; you make the call. and thank you again for looking into these issues---I want to, but simply haven't been able to find the time
<ShubhamAgrawal[m>
rcurtin[m]: I don't know much about golang for now. That's why
<ShubhamAgrawal[m>
I am thinking this as soft correction as from Go 1.18, it is ig mandated to use modules
<rcurtin[m]>
I also don't know too much about golang 😄
<rcurtin[m]>
I learned a good bit when we originally added them, but I don't use Go regularly
_slack_mlpack_22 has quit [Quit: You have been kicked for being idle]