Armadillo unittest broken for older versions of Armadillo

Bug #912166 reported by Johannes Ring
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
DOLFIN
Fix Released
Medium
Johannes Ring

Bug Description

The Armadillo unittest does not work for Armadillo < 0.9.49:

http://fenicsproject.org:8090/builders/dolfin-lucid-amd64/builds/48/steps/dolfin%20check/logs/stdio

This is because 'randu' was named 'rand' before version 0.9.49. Should we require Armadillo >= 0.9.49 or can we modify the Armadillo unittest to also support older versions of Armadillo?

Revision history for this message
Garth Wells (garth-wells) wrote : Re: [Bug 912166] [NEW] Armadillo unittest broken for older versions of Armadillo

On 5 January 2012 09:10, Johannes Ring <email address hidden> wrote:
> Public bug reported:
>
> The Armadillo unittest does not work for Armadillo < 0.9.49:
>
> http://fenicsproject.org:8090/builders/dolfin-lucid-
> amd64/builds/48/steps/dolfin%20check/logs/stdio
>
> This is because 'randu' was named 'rand' before version 0.9.49. Should
> we require Armadillo >= 0.9.49 or can we modify the Armadillo unittest
> to also support older versions of Armadillo?
>

Yes, but it should keep the solve part. Feel free to change it.

Garth

> ** Affects: dolfin
>     Importance: Undecided
>         Status: Confirmed
>
> --
> You received this bug notification because you are a member of DOLFIN
> Core Team, which is subscribed to DOLFIN.
> https://bugs.launchpad.net/bugs/912166
>
> Title:
>  Armadillo unittest broken for older versions of Armadillo
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/dolfin/+bug/912166/+subscriptions

Revision history for this message
Johannes Ring (johannr) wrote :

I changed it to use 'randn' instead of 'randu' since 'randn' works for both old and new Armadillo.

Changed in dolfin:
status: Confirmed → Fix Committed
assignee: nobody → Johannes Ring (johannr)
importance: Undecided → Medium
Changed in dolfin:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.