!VxymBsrMGuogkycwNa:matrix.org

poliastro-ci

22 Members
Testing, infrastructure, Continuous Integration problems, pytest regressions, Windows rants4 Servers

Load older messages


SenderMessageTime
6 May 2020
@ashvoid:matrix.orgAsh joined the room.18:53:15
11 May 2020
@jorgepiloto:matrix.orgjorgepilotoHi all, I just saw that the Azure Pipelines badge on the main page for the repo shows a failing message16:10:57
@jorgepiloto:matrix.orgjorgepilotoHowever, last merged changes some hours ago passel all the tests without problem 16:12:22
@astrojuanlu:matrix.orgastrojuanlu hi jorgepiloto ! thanks for the shout out, let me take a look 16:18:38
@astrojuanlu:matrix.orgastrojuanluoh, I see. numba 0.39.1 was released16:19:30
@astrojuanlu:matrix.orgastrojuanluhttps://github.com/poliastro/poliastro/pull/95816:21:19
@jorgepiloto:matrix.orgjorgepilotoCool! 🚀17:34:55
@jorgepiloto:matrix.orgjorgepilotoHmmm... This is weird, the issue is still present. Might the badge be pointing towards the last stable version?17:37:02
@astrojuanlu:matrix.orgastrojuanlunotice that I could not merge the PR18:02:08
@astrojuanlu:matrix.orgastrojuanluanother error appeared18:02:11
@jorgepiloto:matrix.orgjorgepilotoIs this again the same problem from some weeks ago? That one passing in Linux but failing in other OS?18:23:14
@astrojuanlu:matrix.orgastrojuanlu

not really:

astropy.utils.iers.iers.IERSRangeError: (some) times are outside of range covered by IERS table.

18:24:48
@astrojuanlu:matrix.orgastrojuanluit's weird because these tests were passing before. but I've heard bad things about the IERS service being down, and also there seems to be a new Astropy release... I still didn't have time to investigate the problem18:25:20
13 May 2020
@astrojuanlu:matrix.orgastrojuanlu * oh, I see. numba 0.49.1 was released07:58:39
@astrojuanlu:matrix.orgastrojuanlu jorgepiloto: the problem with the latest numba release was fixed, but we still get random failures on Windows. hopefully the next pipeline on master will pass 07:59:03
@jorgepiloto:matrix.orgjorgepilotoSo for the next merged pr the problem should be fixed by itself?08:52:28
@astrojuanlu:matrix.orgastrojuanluwell... https://github.com/poliastro/poliastro/issues/95909:07:44
17 May 2020
@ashvoid:matrix.orgAsh changed their display name from ashvoid to Ash.05:49:10
3 Jul 2020
@jorgepiloto:matrix.orgjorgepiloto astrojuanlu Did you find the reason behind those random failures? I just saw the pr 14:14:09
@astrojuanlu:matrix.orgastrojuanluthere's one test that is having floating point inaccuracies - I'm only addressing that one: https://github.com/poliastro/poliastro/pull/99014:20:05
@astrojuanlu:matrix.orgastrojuanluthe other one, the Tcl or Tk on Windows... no idea14:20:12
4 Jul 2020
@jorgepiloto:matrix.orgjorgepiloto

Cool! By the way, a new version of isort has been released today under the name Penny and the flag --recursive has been removed. As said in official changelog:

--recursive option has been removed. Directories passed in are now automatically sorted recursive.

https://timothycrosley.github.io/isort/CHANGELOG/

08:37:00
@jorgepiloto:matrix.orgjorgepiloto

I thought it would be as simple as removing the flag but noticed the following:

isort now formats all nested "as" imports in the "from" form. import x.y as a becomes from x import y as a.

08:44:16
@astrojuanlu:openastronomy.orgjuanluastrohahaha omg...08:44:35
@astrojuanlu:openastronomy.orgjuanluastrowell, our policy is to obey whatever black and isort say, so... 😅08:45:21
@jorgepiloto:matrix.orgjorgepilotoDo you want me to open a quick fix on this?08:45:45
@astrojuanlu:openastronomy.orgjuanluastrois your documentation PR failing because of that?08:46:19
@astrojuanlu:openastronomy.orgjuanluastroif so, you can push to the same branch08:46:34
@jorgepiloto:matrix.orgjorgepilotoYes, it is installing by default the last isort version08:46:59
@astrojuanlu:openastronomy.orgjuanluastroI guess all the PRs will start failing then, so let's fix it ASAP 08:53:26

Show newer messages


Back to Room ListRoom Version: 1