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

Integrate bitemporal SQL reporting service (Tech Sprint 2024) #237

Draft
wants to merge 96 commits into
base: main
Choose a base branch
from
Draft

Integrate bitemporal SQL reporting service (Tech Sprint 2024) #237

wants to merge 96 commits into from

Conversation

refset
Copy link

@refset refset commented Nov 19, 2024

This Draft PR presents a set of changes delivered as part of JUXT's involvement with the Tech Sprint. The changes may be reviewed, discussed and considered for upstream usage as desired.

The changes are documented in this README.

However, based on recent discussions with key community members, it sounds likely that the ultimate approach to integrating the bulk of these changes (if at all) will depend on there first being some 'knowledge graph' strategy implemented within this repo.

Please advise if there are obvious changes we could make to aid with reviewing/assessing/discussing, thanks.

mpisanko added 30 commits August 20, 2024 13:02
@mpisanko
initial cut - generate project structure
be8bc6c
@mpisanko
fix problem w/ timestamp
297f6e4
@mpisanko
no need for explicit timestamp - we already have valid time and syste…
f4eaa98 
…m time
@mpisanko
use execute-batch! to use full SQL
e0e92cb
@mpisanko
use a volume to persist data
406de80
@mpisanko
move functionality to new namespace
a015396
@mpisanko
add webserver etc
207ace9
@mpisanko
adapt to run price service in docker compose and swap out reference s…
c4fa35b 
…ervice
@mpisanko
remove reference-data
fc4581a
@mpisanko
add dev friendly Dockerfile and Dockerfile.production
f7f4750
@mpisanko
intial README
170d592
@mpisanko
configure via ENV
7ac834c
@mpisanko
rename price service to reference service taking over reference data …
b02ebdc 
…(service)
@mpisanko
one more location with old name
1e09712
@mpisanko
add route for checking stock price - generates new price based on pre…
133ea56 
…vious price
@mpisanko
add websocket client to tap into trade order updates
d2aaf84
@mpisanko
WIP: query reference-service when creating trade to check current price
f832e62 
TODO: pass on the price to trade blotter and position and persist trades in XTDB
@mpisanko
add unitprice to trade and value to position
3bb58c7
@mpisanko
persist trades
ff27cb7
@mpisanko
enable publishing - so we can publish (on an interval - 2mins?) prices
ac13293 
consider if all prices or only those asked for
@mpisanko
start with buys - cannot sell what you don't have
dff69f7
@mpisanko
disallow selling stocks you don't have
df8d9ba
@mpisanko
negative amount doesn't make sense - just flip buy/sell
cf8edbc
@mpisanko
subscribe to /prices to periodically publish new prices of stocks
030329a
@mpisanko
rename tradeprice to stockprice
1fd726e
@mpisanko
fix order of arguments, log at a decent location, change topic name
f4ad9a3
@mpisanko
show market value
50b21d3
@mpisanko
no need to set 'initial market value' - it will come thru quickly
617d6b5
@mpisanko
add notifo about market price updates
3942e16
@mpisanko
generate all prices independently of subscriptions
a029820
mpisanko and others added 28 commits September 17, 2024 16:52
@mpisanko
no need to hide slider anymore
7d9c515
@mpisanko
only select last position for given valid_time in case of overlapping…
813a947 
… validity
@mpisanko
set highValue to 'All'
7532c9a
@mpisanko
add slider for prices which resets the other slider to latest point o…
7770e99 
…f valid time
@mpisanko
ocassionally create a closed position
1063915
@mpisanko
make the odds of closed position higher
e848a26
@mpisanko
add some doco
468e438
@mpisanko
remove duplicate functions
3a1123d
@mpisanko
redistribute README into more suiting places
54a7d69
@mpisanko
typo
f45c474
@mpisanko
make the top slider a single one to avoid confusion
544748f
@mpisanko
fixup labels
9699eb8
@mpisanko
reset to all time
535652a
@mpisanko
consistently use price update interval
f0577b1
@mpisanko
run price updates every 10 seconds
d883335
@mpisanko
show current state by default and call it 'Now'
9de01e3
@mpisanko
use styles to highlite background for gain/loss
8a0fefd
@mpisanko
Merge pull request #33 from xtdb/colsForPos
abd27c4 
use styles to highlite background for gain/loss
@mpisanko
change wording and decrease padding and column widths
7b2e9aa
@mpisanko
use another image for XT
3a1a1eb
@mpisanko
sliders can be used together
694b93f
@mpisanko
add prices as of string
6df7fad
@refset
styling tweaks
2c7abec
@mpisanko
show when ref srv is up and running
198ab19
@kauppilainen
Remove style width
ebdeb08
@kauppilainen
Fetch new data on slider release
f632673
@kauppilainen
Merge pull request #36 from xtdb/fel/slider-debounce
4b30666 
Fetch data on slider release
@kauppilainen
Reinstate 98% width on sliders
024dae6 
To prevent horizontal scrolling
@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

CLA Not Signed

@netlify Netlify
Copy link

netlify bot commented Nov 19, 2024 •
edited
Loading

✅ Deploy Preview for lucky-concha-f3599f ready!

Name Link
🔨 Latest commit 024dae6
🔍 Latest deploy log https://app.netlify.com/sites/lucky-concha-f3599f/deploys/673cf96c4620210008288953
😎 Deploy Preview https://deploy-preview-237--lucky-concha-f3599f.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

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.
3 participants
@refset @kauppilainen @mpisanko