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.

Sign up for GitHub

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

Jump to bottom

Update from original #1

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

Update from original #1

wants to merge 61 commits into from

Conversation

ManelCeperuelo
Copy link

No description provided.

David Boddie and others added 30 commits December 14, 2015 13:48
Added the wait_for_commands and command_path options back to bdiana.
19b870d
@ManelCeperuelo
Merge ../diana-head
ccc63fc
@ManelCeperuelo
Updated REAME.md file
31fd0b2
@ManelCeperuelo
Updated REAME.md
aca90d1
@ManelCeperuelo
Added < and > for table legend when using line.intervals
c408b65
@ManelCeperuelo
Added < and > for table legend when using line.intervals
472e0e8
@ManelCeperuelo
Merge ../diana-head
5066fce
@ManelCeperuelo
Merge ../diana-head
05fd16a
@ManelCeperuelo
Merge ../diana-head
65c6105
@ManelCeperuelo
Merge ../diana-head
1538bc9
@ManelCeperuelo
Merge ../diana-head
e6ced23
@ManelCeperuelo
Merge ../diana-head
c15c1e7
@ManelCeperuelo
Merge ../diana-head
a747573
@ManelCeperuelo
Merge ../diana-head
35272e5
@ManelCeperuelo
Merge ../diana-head
5bcb11f
@ManelCeperuelo
Merge ../diana-head
b0f9663
@ManelCeperuelo
Merge ../diana-head
b7527bf
@ManelCeperuelo
Merge ../diana-head
13a5243
@ManelCeperuelo
Merge ../diana-head
66f1afd
@ManelCeperuelo
Merge ../diana-head
b109b7c
@ManelCeperuelo
Merge ../diana-head
dbd1e1e
@ManelCeperuelo
Merge ../diana-head
f8fc158
@ManelCeperuelo
Merge ../diana-head
78c2a11
@ManelCeperuelo
Merge ../diana-head
a668739
@ManelCeperuelo
Merge ../diana-head
097084e
@ManelCeperuelo
Merge ../diana-head
bd3aac5
@ManelCeperuelo
Input files are removed with c library
1aa3e13
@ManelCeperuelo
Added <iostream> header to compile with Ubuntu 16.04. The class could…
9e208fb 
… be no longer needed.
@ManelCeperuelo
Merge ../diana-head
bb0e233
@ManelCeperuelo
Merge ../diana-head
d9c3342
@dboddie
Copy link
Contributor

dboddie commented Oct 20, 2016

I don't think I have commit privileges to this repository any longer.

@lisbethb-met-no
Copy link
Collaborator

Hi,

I have the following comments:

  1. README.md
    The changes are not ok!

  2. diFieldPlot.cc
    Remove the old line, not just make it a comment.

  3. diPlotOptions
    Your change is not a part of "Try to add a whole interval (10,20,...50)",
    so keep it above this comment.

  4. bdiana_capi.cc
    Ok. We removed the feature because we thought nobody used it, but it is ok to put it back.

ManelCeperuelo and others added 17 commits November 23, 2016 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ManelCeperuelo @dboddie @lisbethb-met-no