Removed the interface
option to
taxsim_calculate_taxes()
. Now, ‘wasm’ is the only interface
option. Users are not able to send the data to the TAXSIM server via ssh
or http. This feature was removed because we have seen unexpected
changes to the TAXSIM server’s output, which could silently introduce
errors.
Patch
- Convert errors to messages when TAXSIM cannot be connected to, as
per CRAN policy ‘Packages which use Internet resources should fail
gracefully’.
- No longer run tests on CRAN that rely on calling the API, as per the
guidance in HTTP
Testing in R.
Patch
- TAXSIM changed its output for ssh and http and returns additional
columns. Incorporate these changes, but still only return the columns
that were present before the change. Only return the original columns so
that ssh and http output aligns with wasm output.
Patch
- TAXSIM added back the
tficar
column to http (see 0.5.1
for more information). Added the tficar
column back to
tests.
Patch
- TAXSIM changed how it returns http results from its server. It now
adds a trailing comma at the end of each line. This causes the import
functions to think there is an additional column. To solve, update http
import functions to remove trailing commas.
- http results from TAXSIM no longer include the column
tficar
. This caused tests to fail. Updated tests that match
http and ssh results to account for this difference.
Major Changes
- Add
interface
parameter to
taxsim_calculate_taxes()
, which allows users to select the
method of interfacing with TAXSIM.
- Add ‘wasm’ option to
interface
parameter. The
functionality incorporates Aman Gupta Karmani’s JS / WebAssembly tooling
into the package. Therefore, tax calculations can be conducted locally
without send and retrieving data from the TAXSIM servers. (@thomascwells, PR
#11)
- Add ‘http’ option to
interface
parameter. This option
uses curl to send and retrieve data from TAXSIM via https. (@thomascwells, PR #9)
Minor Changes
- For ssh, create known_hosts file in temporary directory instead of
.ssh.known_hosts. This is needed so that the package does not write to
files outside the tempdir, in violation of CRAN requirements.
- For ssh, only use port 22.
Minor Changes
Updated the input column names so that all input columns in TAXSIM
are represented. Changed spelling on one input column name to match
TAXSIM (changed ‘ui’ to ‘sui’). (#7, @thomascwells)
Major Changes
- Updated to TAXSIM 35. Data is now sent to the TAXSIM 35 server
instead of the TAXSIM 32 server.
- Changed column names of input columns so that they now match TAXSIM
35’s input column names.