chrisknewman / tusas

Other
18 stars 7 forks source link

Move farzadi postproc from 'cummins' to 'farzadi' #119

Closed stvdwtt closed 2 years ago

stvdwtt commented 2 years ago

It looks like the changes from 29ece38a56e06237f5b3fce84350d183cacf7127 to add the postprocesssed fields for the regular farzadi test case (as opposed to farzadi_test) were added to the "cummins" section. This does not add the expected postprocessed output. Moving them to the next section for farzadi (starting on what is now line 1795) does.

chrisknewman commented 2 years ago

Sent with BlackBerry Work (www.blackberry.com)

From: Stephen DeWitt @.**@.>> Date: Monday, Feb 21, 2022, 2:16 PM To: chrisknewman/tusas @.**@.>> Cc: Subscribed @.**@.>> Subject: [EXTERNAL] [chrisknewman/tusas] Move farzadi postproc from 'cummins' to 'farzadi' (PR #119)

It looks like the changes from 29ece38https://github.com/chrisknewman/tusas/commit/29ece38a56e06237f5b3fce84350d183cacf7127 to add the postprocesssed fields for the regular farzadi test case (as opposed to farzadi_test) were added to the "cummins" section. This does not add the expected postprocessed output. Moving them to the next section for farzadi (starting on what is now line 1795) does.


You can view, comment on, or merge this pull request online at:

https://github.com/chrisknewman/tusas/pull/119

Commit Summary

File Changes

(1 filehttps://github.com/chrisknewman/tusas/pull/119/files)

Patch Links:

— Reply to this email directly, view it on GitHubhttps://github.com/chrisknewman/tusas/pull/119, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD23XRTBXR6ZTMNY6SWDPNDU4KTT7ANCNFSM5O7TTB2Q. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub. You are receiving this because you are subscribed to this thread.Message ID: @.***>