Skip to content
This repository has been archived by the owner on Jan 3, 2022. It is now read-only.

Issues with test project #46

Open
armandsLa opened this issue Oct 21, 2017 · 3 comments
Open

Issues with test project #46

armandsLa opened this issue Oct 21, 2017 · 3 comments

Comments

@armandsLa
Copy link

Wanted to post results of my two macs, but could get the test project to build because of this issue.

Is this really the best project to do benchmarking on as it requires so many dependencies which can lead to multiple issues? Also, the eidolon project is constantly updated, which can affect build times making results incomparable.

My suggestion would be to add a test project to this repo which doesn't use any 3rd party dependency, is easy to setup and is never changed. This would probably lead to more contributors.

@ashfurrow
Copy link
Owner

Sorry about the Eidolon issue, was going to fix it last week but got swamped with other work.

Eidolon is probably not the best project to use, but it is a large-scale codebase that's used in production, and approximates a real-world use case for a lot of devs. While external dependencies can cause issues, they also represent a common pattern for development. Additionally, eidolon is a "stable" project with very few planned changes. Here's the commit graph:

screen shot 2017-10-21 at 1 17 08 pm

I understand your reservations, but this wasn't a resource I ever planned on making scientific (there are other metrics for that, like Geekbench). It was just a way for developers to compare what relative build times look like across different machines.

I'll try to fix the Eidolon failures on Monday. Thanks for contributing!

@eaigner
Copy link

eaigner commented May 29, 2018

Was about to say... why can't we use a proper/plain Xcode project for this. Jumping through 10 hoops to install this crap, including asking for a Github user/password. No sir-io, thank you :)

Something like a stable tensorflow release branch would be more appropriate

  • No or very minimal changes/hotfixes
  • Huge codebase
  • Just run tensorflow/tensorflow/contrib/makefile/build_all_ios.sh from commandline

@ashfurrow
Copy link
Owner

@eaigner I used this project because this is the project I had two and a half years ago. It's a real-world, in-production app that exemplifies the kind of work that I do day-to-day. If you don't like it as a benchmarking app, you're free to go start your own Xcode performance-benchmarking project.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants