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

Improve statistics, add tests #46

Merged
merged 96 commits into from
Nov 16, 2024
Merged

Improve statistics, add tests #46

merged 96 commits into from
Nov 16, 2024

Conversation

ksopyla
Copy link
Collaborator

@ksopyla ksopyla commented Nov 16, 2024

Changes:

  • test refactoring - use base classes, move test data to fixutres,
  • add more test cases, focus on statistics contract
  • fix dates handling
  • refactor and performance improvements of statistics aggregation

ksopyla and others added 30 commits July 4, 2024 21:51
@ksopyla
Add bunch of posts with release notes
bd85fce
@ksopyla
Merge branch 'docs' of https://github.com/PromptSail/prompt_sail into…
2781457 
… docs
@sliwaszymon
better statistics for portfolio endoints (datapoints in time)
2cc8636
@ksopyla
Merge branch 'docs' into dev
abe88c1
@ksopyla
Improve the language and add more comments
1f4d68f
@ksopyla
fix checkboxes
904ddd1
@Bklzn
add / to proxy URL
a70e59b
@Bklzn
project details - statistics, change short names of providers
475d4fb
@sliwaszymon
portfolio details separated to details and usage_in_time
ce53b90
@sliwaszymon
Merge branch 'dev' of https://github.com/PromptSail/prompt_sail into dev
a16ffcc
@sliwaszymon
in portfolio usage_in_time total_cost is not cumulative
025e5af
@sliwaszymon
authorization for portfolio endpoint
f06d710
@Bklzn
add Portfolio subpage
dadda99
@Bklzn
Merge branch 'dev' of https://github.com/PromptSail/prompt_sail into dev
ebe4117
@Bklzn
Portfolio - add Tags costs chart
9afdc00
@sliwaszymon
"/" added into all example url's endings
d548237
@sliwaszymon
Merge branch 'dev' of https://github.com/PromptSail/prompt_sail into dev
daf0c3a
@WiolaGreen
fixing typos
f74daf3
@WiolaGreen
update link to organization's dashboard page
348329f
@sliwaszymon
refactor of saving transactions as two parts
1bb2d05
@WiolaGreen
adding a new section - demo
7c2a0d9
@WiolaGreen
adding links to demo page
f8b4268
@WiolaGreen
updated demo description and typo fixes
fd2c0de
@WiolaGreen
description update of first API call
6c44298
@WiolaGreen
typo fix
d2fc49c
@sliwaszymon
resizing photos only for transaction (details); full-size images in r…
1046987 
…awtransactions
@WiolaGreen
adding the transaction definition, title change
e5e3eb9
@sliwaszymon
single transaction and transaction list endpoints repaired
43beddc
@WiolaGreen
changing pprint function into print function
aa5b73a
@WiolaGreen
changing the default proxy_url (ps_api_base)
33dbb3f
@ksopyla
set the openapi version
f0d4790
@ksopyla
Refactor, add test to create transations, move the price list to conf…
97eccc7 
…ig module and add to config class
@ksopyla
Refactor the test count statistics. Move test_transaction files to fi…
605a5f2 
…xture folder, rename and add docstring to test.

Refactor the method read_transactions_from_csv
@ksopyla
fix imports, move truncate_float to testt_utils
3bcb580
@ksopyla
Add refactored test_count_statistics_refactored.py
61b9f07
@ksopyla
change to more meaningful names of request and response
0b4cba2
@ksopyla
add date format to docs
fe3247a
@ksopyla
change to more meaningful names the request, response params in store…
6f15c68 
…_transaction
@ksopyla
small refactor the read_transactions_from_csv
091656e
@ksopyla
add docstrings and change the arrnage section to use read_transaction…
3348d20 
…s_with_prices_from_csv with config
@ksopyla
refactor names in token and cost statistics, create new Test Case for…
003cad9 
… tokens_and_cost_statistics_refactored, for now leave originall file
@ksopyla
add test for transation cost calculation
ed8db1c
@ksopyla
remove the key
d1887c9
@ksopyla
remove the test key
738244a
@ksopyla
add generation speed tests
bad5f81
@ksopyla
Update Dockerfile
9f44020 
remove the price list
@ksopyla
refactor test_api dostring and names
ae5baaa
@ksopyla
refactor test file names, separate transaction tests from project, re…
44bce8e 
…move refactored files
@ksopyla
refactor transaction speed tests
f7824f6
@ksopyla
refactor the speed statistics, add missing test with error handling
24bd0da
@ksopyla
make dates for statistics speed requrired, remove the check_dates_for…
4c939a9 
…_statistics from get_transactions_speed_statistics_over_time, add more tests for checking error codes
@ksopyla
Merge branch 'transaction-cost-refactor' into dev
ca0b897
@ksopyla
Add tests for different time granulairyt and periods checking the dat…
9cc1cb0 
…a points behaviour
@ksopyla
wip: refactorization for speed statistics aggregation, refactore the …
f30c40b 
…method how aggregation was done, work on checking time zones,
@ksopyla
fix for storing transations via revers proxy
5007947
@ksopyla
add exception showing in the logs
d942aba
@ksopyla
work on more tests, some refactoring add test for porfolio
74d1697
@ksopyla ksopyla merged commit 11330b8 into main Nov 16, 2024
6 checks passed
This was referenced Nov 16, 2024
Closed
Closed
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.
4 participants
@ksopyla @sliwaszymon @Bklzn @WiolaGreen