View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002569 | OpenFOAM | Patch | public | 2017-06-03 23:56 | 2017-07-11 09:54 |
Reporter | aguerrehoracio | Assigned To | henry | ||
Priority | low | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | GNU/Linux | OS | Ubuntu | OS Version | 14.04 |
Product Version | dev | ||||
Fixed in Version | dev | ||||
Summary | 0002569: changeDictionary utility does not run for all the time range selected | ||||
Description | When running changeDictionary with -time option, the utility only works for the first time of the range provided. For example: changeDictionary -time 0: (we want to run for all time folders) works only for the time "0". | ||||
Additional Information | In file changeDictionary.C line 431 --> runTime.setTime(times[0], 0); only the first time of the timeList is used. In attached file I put this line inside a loop over "times" and the issue is fixed. | ||||
Tags | Pre-processing | ||||
|
changeDictionary.C (18,621 bytes) |
|
It doesn't make much sense to override the "instance" name with a command-line value in the time loop: forAll(times, timeI) { runTime.setTime(times[timeI], timeI); word instance ( args.optionLookupOrDefault("instance", runTime.timeName()) ); surely if the "instance" option is specified the time-loop should not be executed. |
|
Please test commit ed8538a85cedb9431c0bb6ed58138ffd4f1a87bc and let me know if there are any issues. |
|
From the lack of response from the reporter I assume this is now resolved. Please re-open if there are issues with new implementation. |
Date Modified | Username | Field | Change |
---|---|---|---|
2017-06-03 23:56 | aguerrehoracio | New Issue | |
2017-06-03 23:56 | aguerrehoracio | File Added: changeDictionary.C | |
2017-06-03 23:56 | aguerrehoracio | Tag Attached: Pre-processing | |
2017-06-20 23:09 | wyldckat | Category | Bug => Patch |
2017-06-26 17:52 | henry | Note Added: 0008269 | |
2017-06-26 19:14 | henry | Note Added: 0008270 | |
2017-07-11 09:54 | henry | Assigned To | => henry |
2017-07-11 09:54 | henry | Status | new => resolved |
2017-07-11 09:54 | henry | Resolution | open => fixed |
2017-07-11 09:54 | henry | Fixed in Version | => dev |
2017-07-11 09:54 | henry | Note Added: 0008381 |