{"action":"created","issue":{"url":"https://api.github.com/repos/lammps/lammps/issues/1130","repository_url":"https://api.github.com/repos/lammps/lammps","labels_url":"https://api.github.com/repos/lammps/lammps/issues/1130/labels{/name}","comments_url":"https://api.github.com/repos/lammps/lammps/issues/1130/comments","events_url":"https://api.github.com/repos/lammps/lammps/issues/1130/events","html_url":"https://github.com/lammps/lammps/issues/1130","id":362118216,"node_id":"MDU6SXNzdWUzNjIxMTgyMTY=","number":1130,"title":"possible bug with read_restart command","user":{"login":"ramisetti","id":9049630,"node_id":"MDQ6VXNlcjkwNDk2MzA=","avatar_url":"https://avatars1.githubusercontent.com/u/9049630?v=4","gravatar_id":"","url":"https://api.github.com/users/ramisetti","html_url":"https://github.com/ramisetti","followers_url":"https://api.github.com/users/ramisetti/followers","following_url":"https://api.github.com/users/ramisetti/following{/other_user}","gists_url":"https://api.github.com/users/ramisetti/gists{/gist_id}","starred_url":"https://api.github.com/users/ramisetti/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ramisetti/subscriptions","organizations_url":"https://api.github.com/users/ramisetti/orgs","repos_url":"https://api.github.com/users/ramisetti/repos","events_url":"https://api.github.com/users/ramisetti/events{/privacy}","received_events_url":"https://api.github.com/users/ramisetti/received_events","type":"User","site_admin":false},"labels":[{"id":51423231,"node_id":"MDU6TGFiZWw1MTQyMzIzMQ==","url":"https://api.github.com/repos/lammps/lammps/labels/invalid","name":"invalid","color":"e6e6e6","default":true}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-09-20T10:17:42Z","updated_at":"2018-09-20T10:32:56Z","closed_at":"2018-09-20T10:32:56Z","author_association":"NONE","body":"## Summary\r\n\r\n_read_restart_ command is restarting run differently than the run which produced the restart file written with _restart_ command followed by two filenames\r\n\r\n## Type of Issue\r\n\r\nBug Report (possible)\r\n\r\n## Detailed Description (Enhancement Suggestion)\r\n\r\nRestarting run is sometimes different when using _read_restart_ command (like **_read_restart_ restart.*.equi**) where **restart.*.equi** are 2 files produced from _restart_ command with two filenames (like **_restart_ restart.1.equi restart.2.equi**). I know that the code for read_restart command is looking for maximum timestep based on the file name and not from the file content and so in my case it is always picking up restart.2.equi file which in some cases may not be the last timestep.\r\n\r\n## LAMMPS Version (Bug Report)\r\n\r\n2 August 18\r\n\r\n## Expected Behavior (Bug Report)\r\n\r\nThe description for _read_restart_ manual says that \"If  a “*” appears in the filename, the directory is searched for all filenames that match the pattern where “*” is replaced with a timestep value. The file with the largest timestep value is read in.\" However, this description is true in only some cases when restart file name contains timesteps in the file name. \r\n\r\n## Actual Behavior (Bug Report)\r\n\r\nI think the description in the manual should also explicitly mention the largest timestep value is chosen based on the filenames and it does not work when restart files are written using _restart_ command with two filenames (e.g.: **_restart_ restart.1.equi restart.2.equi**)\r\n\r\n## Steps to Reproduce (Bug Report)\r\n\r\n1. run lammps with in.indent.txt (attached) which will create restart.1.equi and restart.2.equi\r\n2. make a copy of restart.1.equi and restart.2.equi\r\n3. run lammps with in_indent_restart.txt (attached)\r\n4. open the log.lammps file and check for the first timestep printed under Step. If the time step is 40000 the restart is as expected.\r\n5. swap the names of the copies of the restart.1.equi and restart.2.equi (from step 2)\r\n6. run lammps with in_indent_restart. Repeat step 4 to check the the first timestep written under Step. This time the timestep is 39000 which is the not the last step of previous run from step 1 (see attached a copy of my log file).\r\n\r\n## Further Information, Files, and Links\r\n\r\n[in.indent_restart.txt] (https://github.com/lammps/lammps/files/2400785/in.indent_restart.txt)\r\n[in.indent.txt] (https://github.com/lammps/lammps/files/2400786/in.indent.txt)\r\n[log.txt] (https://github.com/lammps/lammps/files/2400789/log.txt)\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/lammps/lammps/issues/comments/423132878","html_url":"https://github.com/lammps/lammps/issues/1130#issuecomment-423132878","issue_url":"https://api.github.com/repos/lammps/lammps/issues/1130","id":423132878,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMzEzMjg3OA==","user":{"login":"akohlmey","id":289251,"node_id":"MDQ6VXNlcjI4OTI1MQ==","avatar_url":"https://avatars3.githubusercontent.com/u/289251?v=4","gravatar_id":"","url":"https://api.github.com/users/akohlmey","html_url":"https://github.com/akohlmey","followers_url":"https://api.github.com/users/akohlmey/followers","following_url":"https://api.github.com/users/akohlmey/following{/other_user}","gists_url":"https://api.github.com/users/akohlmey/gists{/gist_id}","starred_url":"https://api.github.com/users/akohlmey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akohlmey/subscriptions","organizations_url":"https://api.github.com/users/akohlmey/orgs","repos_url":"https://api.github.com/users/akohlmey/repos","events_url":"https://api.github.com/users/akohlmey/events{/privacy}","received_events_url":"https://api.github.com/users/akohlmey/received_events","type":"User","site_admin":false},"created_at":"2018-09-20T10:32:56Z","updated_at":"2018-09-20T10:32:56Z","author_association":"MEMBER","body":"No, this is not a bug, but a misuse of a feature. The restart.1.equi and restart.2.equi files do not encode the time step into the file name (i would recommend against using numbers here to avoid this confusion, but call them restart.a.equi and restart.b.equi), so you cannot use a feature that depends on it. The read_restart option including a \"*\" is meant to be using in conjunction with the restart command using a filename including a \"*\" (which will be replaced with the current timestep number)."}}
{"actor":{"display_login":"akohlmey"}}