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.

Sign up for GitHub

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

Jump to bottom

Changes after v 1 3 0 #44

Merged
merged 42 commits into from
Nov 1, 2023
Merged

Changes after v 1 3 0 #44

merged 42 commits into from
Nov 1, 2023

Conversation

duncanpo
Copy link
Member

@duncanpo duncanpo commented Nov 1, 2023

No description provided.

duncanpo and others added 30 commits July 28, 2023 16:10
@duncanpo
Initial submit of metrics
89df507
@duncanpo
Update build.yml to use latest version of opentelemetry-cpp
9533d73
@duncanpo
Update build.yml to use latest version of opentelemetry-cpp
c1cc7d1
@ricktu-mw @duncanpo
doubleCounter
cc208df 
1. meterProvider->cgetMeter
2. meter->createDoubleCounter
3. counter->add
@ricktu-mw @duncanpo
fix: comment and headers
b89079b 
incorporate feedback in 8/11 code review.
@duncanpo
Initial submit of metrics
d37755e
@duncanpo
Update build.yml to use latest version of opentelemetry-cpp
62d036a
@duncanpo
Update build.yml to use latest version of opentelemetry-cpp
aa3c642
@ricktu-mw @duncanpo
doubleCounter
0302808 
1. meterProvider->cgetMeter
2. meter->createDoubleCounter
3. counter->add
@ricktu-mw @duncanpo
fix: comment and headers
e90e6a4 
incorporate feedback in 8/11 code review.
@duncanpo
Update metrics code to use utils function from common folder
9e729e3
@duncanpo
Merge branch 'metrics' into metrics_rebase
55d2542
@duncanpo
Merge pull request #8 from mathworks/metrics_rebase
1fff38c 
Metrics rebase
@duncanpo
Merge from branch changes_after_v_1_2_0
587c3f6
@duncanpo
fix a fallout from previous merge
23ec1df
@duncanpo
Merge pull request #9 from mathworks/simplify_build_process
aba14a6 
Simplify build process
@duncanpo
Merge branch 'changes_after_v_1_2_0'. Fix issue with repeated patching.`
1c29977
@duncanpo
Merge pull request #12 from mathworks/fix_patch_issue
2bc00e2 
Fix patch issue
@ricktu-mw
feat: UpDownCounter, Histogram, tmetrics
7783141 
1. Class: UpDownCounter, Histogram
2. Test: tmetrics
@ricktu-mw
fix: reverse patch cmd in cmake
e0bd4e7
@ricktu-mw
fix: merge updates in cmake
72ac9a3 
patch cmd fix
@ricktu-mw
fix: attribute pairs input indexing
ccfe101
@ricktu-mw
fix: attributes and bucket tests
c44d0e5 
incorporate feedback from PR#13 for tmetrics
@duncanpo
Merge pull request #13 from mathworks/sync_instruments
5588c3e 
feat: UpDownCounter, Histogram, tmetrics
@duncanpo
Merge branch 'changes_after_v_1_2_0' into merge_1_3_0_changes
078e346
@duncanpo
Merge branch 'metrics' into merge_1_3_0_changes
f47cccf
@duncanpo
Merge pull request #15 from mathworks/merge_1_3_0_changes
a28e450 
Merge 1 3 0 changes
@duncanpo
download collector automatically for tests
c573ada
@duncanpo
Change tests to automatically download collector if not installed
27953ad
@duncanpo
update GitHub workflow
15cfc4c
@duncanpo duncanpo merged commit bb15a99 into main Nov 1, 2023
1 check passed
@duncanpo
Copy link
Member Author

duncanpo commented Nov 1, 2023

Fixed #20, #36

@duncanpo duncanpo deleted the changes_after_v_1_3_0 branch November 1, 2023 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@duncanpo @ricktu-mw