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

Fixed initialisation of substitute models in ForwardMappingModel #85

Merged
merged 2 commits into from
Mar 25, 2016
Merged

Conversation

sigveka
Copy link
Contributor

@sigveka sigveka commented Mar 24, 2016

I think this fix ("elif" statement in line 188 in Strategy.py) is a temporary one.

The problem is a flaw in ModenaFireTask, run_task (line 980), where if 'point' in self will never be True for a ForwardMappingModel. Consequently, only the substitute models in for BackwardMappingModel s will be checked and instantiated.

@henrus henrus merged commit 65c6df4 into MoDeNa-EUProject:nextRelease Mar 25, 2016
@henrus
Copy link
Member

henrus commented Mar 25, 2016

Hi Sigve, Hi Christos,

merged.

@christos: Does this fix your problem?

Henrik

On 24/03/16 23:24, Sigve Karolius wrote:

I think this fix ("elif" statement in line 188 in Strategy.py) is a
temporary one.

The problem is a flaw in ModenaFireTask, |run_task| (line 980), where
|if 'point' in self| will never be |True| for a
/ForwardMappingModel/. Consequently, only the substitute models in for
_BackwardMappingModel_s will be checked and instantiated.


    You can view, comment on, or merge this pull request online at:

#85

    Commit Summary


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#85

Dr. Henrik Rusche
WIKKI Ltd.
Unit 459 Southbank House Tel: +44 (0)20 3176 7190
Black Prince Road E-mail: h.rusche@wikki.co.uk

London SE1 7SJ, United Kingdom

www.wikki.co.uk

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