Skip to content

Conversation

@vahid-ahmadi
Copy link
Collaborator

@vahid-ahmadi vahid-ahmadi commented Aug 31, 2025

Fixes #1333

@vahid-ahmadi vahid-ahmadi self-assigned this Aug 31, 2025
Copy link
Collaborator

@MaxGhenis MaxGhenis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only rules in this repo

@vahid-ahmadi vahid-ahmadi requested a review from MaxGhenis August 31, 2025 21:26
Copy link
Collaborator

@MaxGhenis MaxGhenis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great to see this passing checks!

definition_period = YEAR
unit = GBP
documentation = "Total VAT charged on firm's sales (output VAT)"

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
defined_for = "firm_vat_registered"

quantity_type = FLOW


class annual_input_k(Variable):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make all numbers pounds not thousands

description = "Business firm variables from firm dataset."


class firm_id(Variable):
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make each variable its own file

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't think we need this or business_group - we're only targeting at this level not simulating at it

turnover = firm("annual_turnover_k", period) * 1000
threshold = parameters(period).gov.hmrc.vat.registration_threshold

return turnover > threshold
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

they can choose to register if under right? can be beneficial?

if so we may need another variable here that's from the calibrated dataset like registers_for_vat_if_not_required

output_vat = firm("firm_vat_on_sales", period)
input_vat = firm("firm_vat_on_purchases", period)

return output_vat - input_vat
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use adds/subtracts

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.

Add firm analysis files

3 participants