Skip to content
This repository has been archived by the owner on Jul 8, 2022. It is now read-only.

Add command line parameter to load a transfer function from file #78

Merged
merged 1 commit into from
Jul 20, 2015

Conversation

jafyvilla
Copy link
Contributor

No description provided.

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/162/
Test PASSed.

@@ -163,6 +164,10 @@ bool Config::init()
const ApplicationParameters& params = getApplicationParameters();
frameSettings->setFrameNumber( params.frames.x( ));

RenderSettingsPtr renderSettings = _impl->framedata.getRenderSettings();
const TransferFunction1Dc tf( _impl->framedata.getVRParameters()->transferFunction );
Copy link
Contributor

Choose a reason for hiding this comment

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

why copy?

@jafyvilla
Copy link
Contributor Author

Updated

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/164/
Test PASSed.

@jafyvilla jafyvilla force-pushed the tf_file branch 2 times, most recently from 36f039c to f10800a Compare July 15, 2015 15:56
@jafyvilla
Copy link
Contributor Author

Rebased and added transfer function tests. Please review again.

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/172/
Test FAILed.

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/171/
Test FAILed.

public:
static std::string header( const size_t numSamples )
{
return boost::lexical_cast< std::string >( numSamples ) + " uint8";
Copy link
Contributor

Choose a reason for hiding this comment

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

std::to_string

@tribal-tec
Copy link
Contributor

+1

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/173/
Test FAILed.

@jafyvilla
Copy link
Contributor Author

Retest this please

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/174/
Test FAILed.

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/178/
Test FAILed.

@jafyvilla
Copy link
Contributor Author

Retest this please

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/179/
Test FAILed.

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/180/
Test FAILed.

@bilgili
Copy link
Contributor

bilgili commented Jul 20, 2015

retest this please

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/183/
Test FAILed.

New TransferFunction1D constructor that takes a file path as
an argument
@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/185/
Test FAILed.

@bilgili
Copy link
Contributor

bilgili commented Jul 20, 2015

retest this please

1 similar comment
@bilgili
Copy link
Contributor

bilgili commented Jul 20, 2015

retest this please

@bbpbuildbot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://bbpcode.epfl.ch/ci//job/oss.Livre.github/186/
Test PASSed.

bilgili pushed a commit that referenced this pull request Jul 20, 2015
Add command line parameter to load a transfer function from file
@bilgili bilgili merged commit fd465ed into BlueBrain:master Jul 20, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants