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

BUG: Email attachment pdf does not calculate stats and charts are empty #225

Closed
vfedotovs opened this issue Dec 8, 2023 · 2 comments
Closed
Assignees
Labels
bug Something isn't working

Comments

@vfedotovs
Copy link
Owner

1 room apartment price analysis

Advertisement count: 0
Average price: 0 EUR
Min price: 0 EUR
Max price: 0 EUR
Price range: 0

2 room apartment price analysis

Advertisement count: 0
Average price: 0 EUR
Min price: 0 EUR
Max price: 0 EUR
Price range: 0

3 room apartment price analysis

Advertisement count: 0
Average price: 0 EUR
Min price: 0 EUR
Max price: 0 EUR
Price range: 0

4 room apartment price analysis

Advertisement count: 0
Average price: 0 EUR
Min price: 0 EUR
Max price: 0 EUR
Price range: 0

5 room apartment price analysis

Advertisement count: 44
Average price: 75715.57 EUR
Min price: 13000 EUR
Max price: 303240 EUR
Price range: 290240

@vfedotovs
Copy link
Owner Author

analytics.py file is creating file basic_price_stats.txt with invalid content
7: 4. basic_price_stats.txt later is used by pdf_cretor.py module to include in pdf file
44: write_lines(price_data, 'basic_price_stats.txt')

root@a5127c90ff7b:/# more analytics.log
2023-12-07 23:55:27,357 [MainThread ] [INFO ] : analytics_main: 31: --- Starting analitics module ---
2023-12-07 23:55:27,357 [MainThread ] [INFO ] : check_files: 70: Verifying if all required module exist
2023-12-07 23:55:27,358 [MainThread ] [INFO ] : categorize_data: 82: Categorizing all apartment data in categories based on room count
2023-12-07 23:55:27,364 [MainThread ] [INFO ] : create_multi_category_stats: 148: Creating stats_by_price report for each for apartment type category
2023-12-07 23:55:27,364 [MainThread ] [INFO ] : calculate_stats_by_price: 101: Calculating stats_by_price basd on column: Price_in_eur

2023-12-07 23:55:27,365 [MainThread ] [ERROR] : calculate_stats_by_price: 122: Column Price_in_eur has 0 prices <<< this is not correct

2023-12-07 23:55:27,365 [MainThread ] [INFO ] : calculate_stats_by_price: 101: Calculating stats_by_price basd on column: Price_in_eur
2023-12-07 23:55:27,366 [MainThread ] [ERROR] : calculate_stats_by_price: 122: Column Price_in_eur has 0 prices
<<< this is not correct

Need to add more verbose logging for dataframe content and make more robust checking for this function
calculate_stats_by_price: 122: Column Price_in_eur has 0 prices <<< this is not correct

@vfedotovs vfedotovs changed the title BUG : ### 1 room apartment price analysis ### Advertisement count: 0 Average price: 0 EUR Min price: 0 EUR Max price: 0 EUR Price range: 0 ### 2 room apartment price analysis ### Advertisement count: 0 Average price: 0 EUR Min price: 0 EUR Max price: 0 EUR Price range: 0 ### 3 room apartment price analysis ### Advertisement count: 0 Average price: 0 EUR Min price: 0 EUR Max price: 0 EUR Price range: 0 ### 4 room apartment price analysis ### Advertisement count: 0 Average price: 0 EUR Min price: 0 EUR Max price: 0 EUR Price range: 0 ### 5 room apartment price analysis ### Advertisement count: 44 Average price: 75715.57 EUR Min price: 13000 EUR Max price: 303240 EUR Price range: 290240 BUG: Email attachment pdf does not calculate stats and charts are empty Dec 8, 2023
vfedotovs added a commit that referenced this issue Dec 10, 2023
…nt-invalid

GH 225 basic price stats content invalid fix
@vfedotovs
Copy link
Owner Author

vfedotovs commented Dec 10, 2023

Resolved in 641fda2
·

@vfedotovs vfedotovs self-assigned this Dec 10, 2023
@vfedotovs vfedotovs added the bug Something isn't working label Dec 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant