Commit graph

1255 commits

Author SHA1 Message Date
Oriol Roqué Paniagua
84bdfe6083 Merged PR 4510: Fix sign
# Description

Fix sign...

# Checklist

- [X] The edited models and dependants run properly with production data.
- [X] The edited models are sufficiently documented.
- [X] The edited models contain PK tests, and I've ran and passed them.
- [ ] I have checked for DRY opportunities with other models and docs.
- [ ] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Fix sign

Related work items: #27609
2025-02-25 15:24:05 +00:00
Joaquin Ossa
c49239a7cf Final changes 2025-02-25 15:25:37 +01:00
Joaquin Ossa
daa93c056c commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
335fccd57e updated schema 2025-02-25 15:25:37 +01:00
Joaquin Ossa
68bddf2b7d Commit update 2025-02-25 15:25:37 +01:00
Joaquin Ossa
7f8e7489e4 finished schema 2025-02-25 15:25:37 +01:00
Joaquin Ossa
63a2eff6e2 finished schema 2025-02-25 15:25:37 +01:00
Joaquin Ossa
8b7dfa0839 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
f1a0b04388 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
56619982d2 commit wip on schema 2025-02-25 15:25:37 +01:00
Joaquin Ossa
76a247a55f commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
2f484054df commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
3f224d8fed commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
9fa62e0bf2 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
29755961e6 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
47d9b6f292 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
7551f6e515 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
c89dd02799 commit wip 2025-02-25 15:25:37 +01:00
Joaquin Ossa
271f017cf5 Merged PR 4482: Staging model of resolutions incidents
# Description

New staging model for Resolutions Incidents
It extracts all of the data from the incidents after the ingestion from CosmosDB.
This includes data on the host, the guest, the booking, the listing and all the resolution process until completed if completed.
Though it currently has a lot of problems like missing values, ids not existing in the back end or payments without currencies. Everything is more clarified here https://www.notion.so/truvi/Data-Quality-Report-Resolutions-Incidents-data-1a40446ff9c9807180eaccfeebb53045?pvs=4.

# Checklist

- [x] The edited models and dependants run properly with production data.
- [x] The edited models are sufficiently documented.
- [x] The edited models contain PK tests, and I've ran and passed them.
- [x] I have checked for DRY opportunities with other models and docs.
- [x] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Related work items: #27811
2025-02-25 14:15:27 +00:00
Joaquin Ossa
cace0f908d Merged PR 4507: Added timestamp to a/b test model
# Description

Added timestamp to a/b test model to start the measures from the adequate date and time

# Checklist

- [x] The edited models and dependants run properly with production data.
- [x] The edited models are sufficiently documented.
- [x] The edited models contain PK tests, and I've ran and passed them.
- [ ] I have checked for DRY opportunities with other models and docs.
- [ ] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Added timestamp to a/b test model

Related work items: #27721
2025-02-25 14:15:19 +00:00
Joaquin Ossa
1697bef090 timestamps 2025-02-25 15:14:54 +01:00
Oriol Roqué Paniagua
3c99c4f8bc Merged PR 4504: Onboarding MRR + Visualisation changes
# Description

Changes:
* Adds Onboarding MRR
* Refactors exclusion code for ongoing month / invoicing cycle
* Adds sign format on relative differences

# Checklist

- [X] The edited models and dependants run properly with production data.
- [X] The edited models are sufficiently documented.
- [X] The edited models contain PK tests, and I've ran and passed them.
- [X] I have checked for DRY opportunities with other models and docs.
- [X] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Related work items: #27609, #27805
2025-02-25 14:05:54 +00:00
Joaquin Ossa
03955aadd5 Finished schema 2025-02-25 14:54:59 +01:00
Joaquin Ossa
4a6158df28 Added timestamp to a/b test model 2025-02-25 14:48:37 +01:00
Oriol Roqué Paniagua
995027205c Merged PR 4502: Revert "Adds Onboarding MRR"
# Description

_Describe your motivation and changes here._

# Checklist

- [ ] The edited models and dependants run properly with production data.
- [ ] The edited models are sufficiently documented.
- [ ] The edited models contain PK tests, and I've ran and passed them.
- [ ] I have checked for DRY opportunities with other models and docs.
- [ ] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Revert "Adds Onboarding MRR"

Reverted commit `dab210c6`.
2025-02-25 11:44:24 +00:00
Oriol Roqué Paniagua
7c4b0e752b Merged PR 4501: Revert "Refactor display exclusion for clarity purposes"
# Description

_Describe your motivation and changes here._

# Checklist

- [ ] The edited models and dependants run properly with production data.
- [ ] The edited models are sufficiently documented.
- [ ] The edited models contain PK tests, and I've ran and passed them.
- [ ] I have checked for DRY opportunities with other models and docs.
- [ ] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Revert "Refactor display exclusion for clarity purposes"

Reverted commit `fdec616f`.
2025-02-25 11:43:46 +00:00
Oriol Roqué Paniagua
5eebb4f560 Merged PR 4500: Revert "Relative increments with sign"
# Description

_Describe your motivation and changes here._

# Checklist

- [ ] The edited models and dependants run properly with production data.
- [ ] The edited models are sufficiently documented.
- [ ] The edited models contain PK tests, and I've ran and passed them.
- [ ] I have checked for DRY opportunities with other models and docs.
- [ ] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Revert "Relative increments with sign"

Reverted commit `f5718ded`.
2025-02-25 11:43:05 +00:00
uri
f5718ded4d Relative increments with sign 2025-02-25 12:30:23 +01:00
Joaquin Ossa
f036e626d7 Commit removed unnecessary fields 2025-02-25 12:19:45 +01:00
uri
fdec616fe6 Refactor display exclusion for clarity purposes 2025-02-25 11:45:14 +01:00
uri
dab210c607 Adds Onboarding MRR 2025-02-25 11:33:37 +01:00
Oriol Roqué Paniagua
a6671ee4d0 Merged PR 4493: Adds Revenue Churn Rate in YTD/MTD Overview
# Description

Changes:
* Adds Revenue Churn Rate in YTD/MTD Overview. This has several implications, I finally understood how to properly compute a YTD.

The problem is that Revenue Churn Rate is a % of the Total "Churned" Revenue in a 12 m period vs. the Total Revenue in the same 12 m period. This is a bit tricky because it's not really additive, because of the Churn definition. Total Churned Revenue is the Revenue that the churned deals in a month generated on that past 12 months prior to churning.

So - in order to aggregate it properly, we need to do the sum of the Total Churned Revenue and retrieve the Total Revenue on these 12 months, and THEN compute the Churn rate.

This PR mainly retrieves the necessary input from the Churn models and then follows a similar computation as for the rest of YTD/MTD converted metrics.

I'll handle Onboarding MRR in a separated PR as this one is quite dense already.

# Checklist

- [X] The edited models and dependants run properly with production data.
- [X] The edited models are sufficiently documented.
- [X] The edited models contain PK tests, and I've ran and passed them.
- [X] I have checked for DRY opportunities with other models and docs.
- [X] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Related work items: #27609, #27805
2025-02-25 09:41:28 +00:00
Joaquin Ossa
78d42462a8 Merged PR 4489: Update athena models to remove id_accommodation
# Description

Removed `id_accommodation` from athena models
I remembered why I had add it before, it was to unite it with the edeposit model with the same number of fields but I fixed it already

# Checklist

- [x] The edited models and dependants run properly with production data.
- [x] The edited models are sufficiently documented.
- [x] The edited models contain PK tests, and I've ran and passed them.
- [ ] I have checked for DRY opportunities with other models and docs.
- [ ] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Update athena models to remove id_accommodation

Related work items: #27938
2025-02-25 08:32:45 +00:00
Oriol Roqué Paniagua
fdc8e2ae24 Merged PR 4491: Reduce metric name length to fit in PBI
# Description

Reduce metric name length to fit in PBI

# Checklist

- [X] The edited models and dependants run properly with production data.
- [X] The edited models are sufficiently documented.
- [X] The edited models contain PK tests, and I've ran and passed them.
- [ ] I have checked for DRY opportunities with other models and docs.
- [ ] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Reduce metric name length to fit in PBI

Related work items: #27609
2025-02-24 16:40:01 +00:00
Joaquin Ossa
62316778fb updated schema 2025-02-24 16:35:10 +01:00
Joaquin Ossa
1d514cec3a Addressed comments 2025-02-24 16:27:16 +01:00
Joaquin Ossa
d7d3422eb9 Update athena models to remove id_accommodation 2025-02-24 15:21:44 +01:00
Joaquin Ossa
34f8fbe499 commit wip 2025-02-24 15:06:55 +01:00
Oriol Roqué Paniagua
85c2c73da7 Merged PR 4483: Adding targets (to be revised). Allows multi-year last-date computation
# Description

Changes:
* Adds template for targets -> These need to be revised
* Adds comparison of values vs. targets
* Allows for multi-year last date computation (allowing multiple years to be selected, not just the last one)

# Checklist

- [X] The edited models and dependants run properly with production data.
- [X] The edited models are sufficiently documented.
- [X] The edited models contain PK tests, and I've ran and passed them.
- [X] I have checked for DRY opportunities with other models and docs.
- [X] I've picked the right materialization for the affected models.

# Other

- [ ] Check if a full-refresh is required after this PR is merged.

Related work items: #27609, #27805
2025-02-24 10:39:45 +00:00
Joaquin Ossa
62c2e44b86 Deduplicated records 2025-02-24 10:57:46 +01:00
Joaquin Ossa
2e19678182 Final tests update 2025-02-23 16:39:43 +01:00
Joaquin Ossa
f29b67addd Updated tests on S&P fees 2025-02-23 16:14:42 +01:00
Joaquin Ossa
fb4775bd07 Updated tests 2025-02-23 16:11:37 +01:00
Joaquin Ossa
fc85dd9e70 Fixed data_types and added tests 2025-02-23 16:05:11 +01:00
Joaquin Ossa
d6dcbcc67e commit wip 2025-02-23 15:04:59 +01:00
Joaquin Ossa
f3842c950b commit wip 2025-02-22 19:55:30 +01:00
Joaquin Ossa
9704426dd6 commit wip 2025-02-22 14:16:41 +01:00
Joaquin Ossa
05adc07674 commit wip 2025-02-21 17:44:00 +01:00
Joaquin Ossa
1b853a9503 commit wip (id_incident duplicates have to check) 2025-02-21 16:40:53 +01:00
Joaquin Ossa
725f2a54c3 Staging model for resolution incidents 2025-02-21 15:25:24 +01:00