New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 816641 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

Need VPython Wheel peakutils, pyserial and matplotlib

Project Member Reported by bsheedy@chromium.org, Feb 26 2018

Issue description

Link to PyPI: https://pypi.python.org/pypi/matplotlib
Wheel Name: matplotlib
Version: 2.2.0rc1
Platform(s) Required: Windows (32 or 64 bit), Linux (x86/64 and arm/arm64)

Link to PyPI: https://pypi.python.org/pypi/PeakUtils
Wheel Name: peakutils
Version: 1.0.3
Platform(s) Required: Windows (32 or 64 bit), Linux (x86/64 and arm/arm64)

We only need Linux x86/64 at the current moment - the other platforms and architectures are for possible future use.
 
Looks like PyPI has usable packages for all platforms! That makes this easy :)
Owner: iannucci@chromium.org
Status: Started (was: Untriaged)
(working on this)
So it looks like matplotlib is actually tricky, as it needs freetype and libpng too. Not impossible, but it means I have to get more familiar with dockerbuild than I have previously been.
Project Member

Comment 4 by bugdroid1@chromium.org, Feb 27 2018

Ok, so peakutils is working (depends on scipy and numpy though). Example .vpython entries:

wheel: <
  name: "infra/python/wheels/peakutils-py2_py3"
  version: "version:1.0.3"
>
wheel: <
  name: "infra/python/wheels/scipy/${vpython_platform}"
  version: "version:0.19.0"
>
wheel: <
  name: "infra/python/wheels/numpy/${vpython_platform}"
  version: "version:1.11.3"
>
Project Member

Comment 7 by bugdroid1@chromium.org, Feb 28 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1f5390a69825b20a1c636d61f4a21295ec231a01

commit 1f5390a69825b20a1c636d61f4a21295ec231a01
Author: bsheedy <bsheedy@chromium.org>
Date: Wed Feb 28 18:37:07 2018

Add VR vpython wheels

Adds entrys to .vpython for peakutils and scipy. These are used by the
VR latency test.

Bug:  816641 
Change-Id: Ia9a76166c6014cde0bec4fa648fdd270a83089c7
Reviewed-on: https://chromium-review.googlesource.com/940333
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539898}
[modify] https://crrev.com/1f5390a69825b20a1c636d61f4a21295ec231a01/.vpython

Summary: Need VPython Wheel peakutils, pyserial and matplotlib (was: Need VPython Wheel peakutils and matplotlib)
Looks like we also need PySerial 3.4
Project Member

Comment 9 by bugdroid1@chromium.org, Mar 1 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/1411b1144c32a9d7716b224083aa2ce162b14d43

commit 1411b1144c32a9d7716b224083aa2ce162b14d43
Author: Robert Iannucci <iannucci@chromium.org>
Date: Thu Mar 01 03:13:32 2018

[dockerbuild] Add PySerial 3.4

TBR=bsheedy@chromium.org, vadimsh@chromium.org

Bug:  816641 
Change-Id: I402fdc38202264f18274a868cd07ef6df86037c2
Reviewed-on: https://chromium-review.googlesource.com/942044
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Reviewed-by: Vadim Shtayura <vadimsh@chromium.org>
Commit-Queue: Robbie Iannucci <iannucci@chromium.org>

[modify] https://crrev.com/1411b1144c32a9d7716b224083aa2ce162b14d43/infra/tools/dockerbuild/wheel.py
[modify] https://crrev.com/1411b1144c32a9d7716b224083aa2ce162b14d43/infra/tools/dockerbuild/wheels.md

Cc: iannucci@chromium.org
Owner: ----
Status: Available (was: Started)
Heyo, do you still need matplotlib? Or can we close this one?
We can close this one - I went through and removed references to matplotlib since we don't actually plot anything while running the test on bots.
Status: Fixed (was: Available)

Sign in to add a comment