Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP: Fix import for some Instruments profiles #216

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jlfwong
Copy link
Owner

@jlfwong jlfwong commented Mar 11, 2019

This is a WIP fix to #196. This solution is pretty unsatisfactory, and seems to have the wrong times for stack completely. I'm not sure why that is, and the change to get this to work seems like a huge hack that can't be how Instruments actually parses these files. I'm not sure what the issue is though.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 46.952% when pulling 0235594 on jlfwong/fix-instruments-10-import into 30ca629 on master.

Base automatically changed from master to main January 16, 2021 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants