Hi People, I Looked On This Line When Trains Try To Save Image.

Answered

Hi people,
I looked on this line when trains try to save image.
https://github.com/allegroai/trains/blob/772fd7f7508d0f5b31428084856b2e5485a013e2/trains/binding/frameworks/tensorflow_bind.py#L411
Why not use get instead? I got key error.
I fixed few lines using get function and checks in the _decode_image method and now it works.
Last time this file modified was in nov19 so it no so up to date

  				
Posted 
	4 years ago

					More
				  		
  Report
		
					CharmingShrimp37
				
					0
					 × 1

Votes Newest

Answers 7

Thanks CharmingShrimp37 !
Could you PR the fix ?
It will be just in time for the 0.16 release 🙂

  				
Posted 
	4 years ago

					More
				  		
  Report
		
					AgitatedDove14
				
					0
					 × 1

AgitatedDove14 I would like a little help in the PR. I checkout to a new branch, made the changes but I can't push it to the remote repo. What do I need to do now?

  				
Posted 
	4 years ago

					More
				  		
  Report
		
					CharmingShrimp37
				
					0
					 × 1

ok will try

  				
Posted 
	4 years ago

					More
				  		
  Report
		
					CharmingShrimp37
				
					0
					 × 1

yea no problem 🙂

  				
Posted 
	4 years ago

					More
				  		
  Report
		
					CharmingShrimp37
				
					0
					 × 1

Hi CharmingShrimp37
Go to Github to your newly forked repo, you should have a green button suggesting to take your branch and making it a PR. It is that simple 🙂

  				
Posted 
	4 years ago

					More
				  		
  Report
		
					AgitatedDove14
				
					0
					 × 1

Let me know if there is an issue 🙂

  				
Posted 
	4 years ago

					More
				  		
  Report
		
					AgitatedDove14
				
					0
					 × 1

done

  				
Posted 
	4 years ago

					More
				  		
  Report
		
					CharmingShrimp37
				
					0
					 × 1

Write your answer

1K Views

7 Answers

4 years ago

one year ago