Skip to content

Conversation

bollwyvl
Copy link
Collaborator

While looking into the built installer locally, I'm also kicking the tires on testing the OpenCV example notebook.

@datakurre
Copy link
Contributor

Thanks. Anything that successfully imports cv2, would be enough. I have a proper internet connection for setting up virtual machine only in the Sunday evening. I’ll do final tests then.

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 11, 2020 via email

@datakurre
Copy link
Contributor

I can test OSX on Sunday even. The drafted release did work on OSX.

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 11, 2020 via email

@datakurre
Copy link
Contributor

Yes, the OpenCV example worked on OSX and has been working on Windows too (I just have not tested the last build).

Did you have time to try the draft Linux installer if it worked?

My Linux test on NixOS could have been false negative and I can only get working VM on Sunday evening.

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 11, 2020 via email

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 11, 2020

Vagrant.configure("2") do |config|
  config.vm.box = "ubuntu/bionic64"
  config.vm.provider "virtualbox" do |v|
    v.memory = 2048
    v.cpus = 2
  end
  config.vm.provision "shell", inline: "apt-get update"
  config.vm.provision "shell", inline: "apt-get install -y xfce4 virtualbox-guest-dkms virtualbox-guest-utils virtualbox-guest-x11"
  config.vm.provision "shell", inline: "sed -i 's/allowed_users=.*$/allowed_users=anybody/' /etc/X11/Xwrapper.config"
  config.vm.provision "file", source: "RobotLab-2020.01.0-Linux-x86_64.sh", destination: "RobotLab-2020.01.0-Linux-x86_64.sh"
  config.vm.provision "shell", inline: "bash RobotLab-2020.01.0-Linux-x86_64.sh -fbp ~/robotlab", privileged: false
end

@bollwyvl
Copy link
Collaborator Author

Can confirm cv2 does not import on linux box. It looks like the name of the package changed at some point to py-opencv.

@bollwyvl
Copy link
Collaborator Author

It also seems not particularly possible to install py-opencv against what's in the installer.

@datakurre
Copy link
Contributor

@bollwyvl Thanks for confirming the issue. I did not bother you for nothing. Weird it still works for Windows and OSX.

Btw, OSX test did flicker for me on Friday. Just triggered build again and it passed.

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 11, 2020 via email

@datakurre
Copy link
Contributor

Could you check these versions? I have added in notes that 2020.01.0 has version

opencv-3.4.1-py36h6fd60c2_1.tar.bz2

But 2019.12.0 has version

opencv-3.4.2-py36h6fd60c2_1.tar.bz2

That’s curious.

@datakurre
Copy link
Contributor

Anyway, maybe this could be fixed by just finding opencv version that includes Python bindings for all platforms. Linux missing cv2 may be just packaging error 🤔

@bollwyvl
Copy link
Collaborator Author

Yeah, I think adding matplotlib did some things to the final resolution.

@bollwyvl
Copy link
Collaborator Author

Ick: osx couldn't find the right FF during the build...

@bollwyvl
Copy link
Collaborator Author

Ok, all the OSX failures are now due to headlessfirefox... in the robotlab tests, we're having to do

Create WebDriver    Firefox    executable_path=${geckodriver}    firefox_binary=${firefox}

I'm thinking that patching the notebook on packaging will be the most robust solution, but it's going to take a sec.

The windows fail looks like simple flake.

}
SVG = SRC_DIR / "robotlab" / "src" / "robotlab" / "icons" / "starter.svg"

OPENCV_OSX = """
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not prod of this: any thoughts of ways to make it cleaner? Can we add something at the RFSL layer?

if line.startswith("Open Browser"):
lines[i] = OPENCV_OSX

opencv.write_text(json.dumps(nb, indent=2))
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeesh

@datakurre
Copy link
Contributor

I am ok with search and replacing headlessfirefox with firefox if that helps for now.

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 12, 2020 via email

@bollwyvl
Copy link
Collaborator Author

C'mon windows, no ❄️

@bollwyvl
Copy link
Collaborator Author

😆

@datakurre
Copy link
Contributor

No sleep? 🙃

I’ll try these out in 12 hours.

@datakurre
Copy link
Contributor

datakurre commented Jan 12, 2020

  • MacOS
  • Linux
  • Windows

@datakurre
Copy link
Contributor

What is the correct way to activate the env in Linux and MacOS.

source bin/activate

?

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 12, 2020 via email

@datakurre
Copy link
Contributor

That was quite a patch you had to do for OSX on OpenCV demo, and those patched notebooks ended up in the final distribution. It's ok for now, but need to think about those. I'd feel better to have simpler test notebooks separately for selenium and OpenCV in RobotLab instead of trying to execute everything crazy I dump into robotkernel examples :-)

@datakurre
Copy link
Contributor

I learned the way to test Linux-installer on top of NixOS. OpenCV works.

Once again you saved the day!

@datakurre
Copy link
Contributor

datakurre commented Jan 12, 2020

All green. Going to merge and relabel "More on robotkernel"-starter. And then move v2020.01.0 tag.

@datakurre datakurre merged commit cf644e8 into robots-from-jupyter:master Jan 12, 2020
@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 12, 2020 via email

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 12, 2020 via email

@datakurre
Copy link
Contributor

Needless to say I do appreciate a lot your effort and all the tests! There is your name and picture on the slides :-) https://github.com/robots-from-jupyter/robotkernel/blob/master/docs/robocon2020/index.ipynb This time I've prepared them with Notebook + RISE.

@datakurre
Copy link
Contributor

datakurre commented Jan 13, 2020

Tested and released. Thanks again! (Windows tests did flicker once.)

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 13, 2020 via email

@datakurre
Copy link
Contributor

datakurre commented Jan 13, 2020 via email

@bollwyvl
Copy link
Collaborator Author

bollwyvl commented Jan 13, 2020 via email

@datakurre
Copy link
Contributor

It is. Now I can have a good night sleep before (or try out what's broken in the latest RF 3.2 beta). And I did share the download links with the participants already in the morning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants