{"action":"closed","issue":{"url":"https://api.github.com/repos/tensorflow/tensorboard/issues/329","repository_url":"https://api.github.com/repos/tensorflow/tensorboard","labels_url":"https://api.github.com/repos/tensorflow/tensorboard/issues/329/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorboard/issues/329/comments","events_url":"https://api.github.com/repos/tensorflow/tensorboard/issues/329/events","html_url":"https://github.com/tensorflow/tensorboard/issues/329","id":248308513,"number":329,"title":"Tensorboard ignores misspelled flags ","user":{"login":"potiuk","id":595491,"avatar_url":"https://avatars3.githubusercontent.com/u/595491?v=4","gravatar_id":"","url":"https://api.github.com/users/potiuk","html_url":"https://github.com/potiuk","followers_url":"https://api.github.com/users/potiuk/followers","following_url":"https://api.github.com/users/potiuk/following{/other_user}","gists_url":"https://api.github.com/users/potiuk/gists{/gist_id}","starred_url":"https://api.github.com/users/potiuk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/potiuk/subscriptions","organizations_url":"https://api.github.com/users/potiuk/orgs","repos_url":"https://api.github.com/users/potiuk/repos","events_url":"https://api.github.com/users/potiuk/events{/privacy}","received_events_url":"https://api.github.com/users/potiuk/received_events","type":"User","site_admin":false},"labels":[{"id":628257156,"url":"https://api.github.com/repos/tensorflow/tensorboard/labels/core:backend","name":"core:backend","color":"1d76db","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2017-08-07T05:05:40Z","updated_at":"2017-11-10T23:31:53Z","closed_at":"2017-11-10T23:31:53Z","author_association":"NONE","body":"Tensorboard silently ignores misspelled flags. Whatever flag you pass to tensorboard it swallows it without even blinking. This might seem like a small problem, but it causes some real-life problems.\r\n\r\nA real-life example - this caused significant cost increase in our case for our tensorboard installation in GCP. We run it in kubernetes cluster and we used the https://github.com/GoogleCloudPlatform/appengine-tensorboard project as a base for our installation. Unfortunately some time recently the name of --reload-frequency parameter has changed to --reload-interval (which is BTW. much better name for what it was) but the appengine-tensorboard project have still the old parameter in configuration (I created pull request https://github.com/GoogleCloudPlatform/appengine-tensorboard/pull/1 to fix it).\r\n\r\nDue to silent swallowing of misspelled parameters, we have not even noticed that the parameter name has changed since the appengine-tensorboard project was released (actually there was even weird inconsistency in the appengine-tensorboard project that in one place RELOAD_INTERVAL variable was mentioned but not used). \r\n\r\nAll-in-all - it would have been much better if tensorboard - following good practice of about few hundred thousands of other projects ;) - would fail if you specify wrong parameter."}}
{"actor":{"display_login":"nfelt"}}