Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Latest MORSE changes #9

Open
wants to merge 119 commits into
base: master
Choose a base branch
from
Open

Conversation

hawesie
Copy link
Member

@hawesie hawesie commented Apr 10, 2015

@kunzel to check that I haven't screwed up please.

Arnaud Degroote and others added 30 commits July 23, 2014 09:58
This allow to modify at runtime the value of some properties, for
example to reflect changes in the capacity of the sensor.
On the basis of a private patch of Osama Salah.
Added a new ROS publisher for DepthCamera
which publishes images in 32FC1 format
fix no `_classpath` issue
TypeError as no `value` member, __str__() do the job.
New safer PollThread avoids conflicts on asyncore.socket_map.
Replace simulator_service.close by asyncore.close_all after thread stops.

Thanks to @BenICE for report and testing !
If we call close more than once, the thread is already dead, so just do
nothing in this case.
Fix morse-simulator#557
It fixes 'mix content' issue when using https to access Morse doc
since component_config and multinode_config are imported as python
script, using json.dump is not valid since it replace boolean
/{True,False}/{true,false}/.

fix morse-simulator#562
those steps works for groovy and fuerte as well
before:
    pose.alter(None, classpath='morse.modifiers.pose_noise.OrientationNoiseModifier')

now:
    pose.alter(classpath='morse.modifiers.pose_noise.OrientationNoiseModifier')
Default ControlType does not work for human, so force the use of
ControlType = Position
Use the current desktop resolution in fullscreen mode.
Following franco.carbognani <franco.carbognani@marssociety.it> mail on
morse-dev and Arnaud comment.
(on my laptop, this does not affect `morse run` behavior, but apparently
usefull when using blenderplayer)
Enable it for human.
For PR2, I don't have any idea how to call it at the morse.robot.pr2
speaks about Hand.Grasp.PR2, which does not exists ...
Should fix morse-simulator#565
Arnaud Degroote and others added 17 commits March 27, 2015 14:30
This commit reverts 2d11860 and fix it
in a "better" way, allowing to properly load the binary module in
Blender / Morse.

Tested on MacOSX 10.10 / blender 2.73a / python 3.4.2
Use sysconfig.get_config('LIBDIR') and sysconfig.get_paths['include'] as
hint for cmake to retrieve the 'correct' includedir / libdir for a
specific version of Python.

Remove all Frameworks handling for MacOSX, as it basically does not work
properly for non-system python (which is always the case for python3)
…der behaviour

Definitively not an adequate solution, but with these new values, test
pass reliability on various different blender version. It does not
really change test ideas
We don't store previous attitude in the right variable.
Reported and fixed by Corentin Chauffaut
@hawesie
Copy link
Member Author

hawesie commented Jun 8, 2015

@kunzel please can you check this, then close (if wrong) or merge (if appropriate). Alternatively please update this fork with the latest changes from the main branch (or ideally merge our changes back there).

@hawesie
Copy link
Member Author

hawesie commented Jun 8, 2015

(the reason is that they have now fixed all my OS X-specific bugs!)

@kunzel
Copy link
Member

kunzel commented Jun 9, 2015

Sure, I'll look into it it in the evening. Sorry for the wait.

On Mon, 8 Jun 2015 18:57 Nick Hawes notifications@github.com wrote:

(the reason is that they have now fixed all my OS X-specific bugs!)


Reply to this email directly or view it on GitHub
#9 (comment).

@hawesie
Copy link
Member Author

hawesie commented Jun 9, 2015 via email

@marc-hanheide
Copy link
Member

what's the status with this?

@marc-hanheide
Copy link
Member

@kunzel ??

@marc-hanheide
Copy link
Member

Hey, is this deprecated now, do we move onto 1.4?

@kunzel
Copy link
Member

kunzel commented Feb 25, 2016

Yes, I am happy that we use the official debian package. The difference is
the battery charging in simulation and the scene generation but nobody
(except me) uses these two features anyway.

On Tue, 23 Feb 2016 at 09:55 Marc Hanheide notifications@github.com wrote:

Hey, is this deprecated now, do we move onto 1.4?


Reply to this email directly or view it on GitHub
#9 (comment).

Dr Lars Kunze
Research Fellow in Intelligent Robotics
School of Computer Science
University of Birmingham
www.cs.bham.ac.uk/~kunzel

@marc-hanheide
Copy link
Member

But from what I have seen Morse 1.4 is not released for Ubuntu 14.04LTS, right?

@kunzel
Copy link
Member

kunzel commented Feb 25, 2016

Ah, ok. So you mean we should just update our fork?

On Thu, 25 Feb 2016 at 15:39 Marc Hanheide notifications@github.com wrote:

But from what I have seen Morse 1.4 is not released for Ubuntu 14.04LTS,
right?


Reply to this email directly or view it on GitHub
#9 (comment).

Dr Lars Kunze
Research Fellow in Intelligent Robotics
School of Computer Science
University of Birmingham
www.cs.bham.ac.uk/~kunzel

@hawesie
Copy link
Member Author

hawesie commented Feb 26, 2016

On 25 Feb 2016, at 14:53, Lars Kunze notifications@github.com wrote:

Yes, I am happy that we use the official debian package. The difference is
the battery charging in simulation and the scene generation but nobody
(except me) uses these two features anyway.

I use the battery charging as it’s very useful for testing routine and scheduled behaviours. I suspect Jay uses the scene generation too.

@marc-hanheide
Copy link
Member

OK, would it then just be a problem to update our own Morse?

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.

10 participants