Skip to content

Fixes to work with default config.ini#5

Open
krishnasun82 wants to merge 1 commit intofaneshion:masterfrom
krishnasun82:headeradd
Open

Fixes to work with default config.ini#5
krishnasun82 wants to merge 1 commit intofaneshion:masterfrom
krishnasun82:headeradd

Conversation

@krishnasun82
Copy link
Copy Markdown

Changed the header to include
Changed config.ini to include new fields

Comment thread NN4IR.h
#include<algorithm>
#include<thread>
#include<omp.h>
#include<random>
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Gives compilation error with random_device on Ubuntu Server 18.xx

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@faneshion - please review this PR

Comment thread config.ini
# 0:robust04, 1:clueweb09 2:
TASK_TYPE = 0

# 0:siomoid; 1:tanh; 2:
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

New fields added to default config.ini

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@faneshion - please review this PR

@krishnasun82 krishnasun82 reopened this Jul 13, 2018
@krishnasun82 krishnasun82 changed the title Include random header to support random_device Fixes to work with default config.ini Jul 13, 2018
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.

1 participant