Skip to content

Commit

Permalink
includes requested changes
Browse files Browse the repository at this point in the history
  • Loading branch information
hernandezc1 committed Sep 30, 2024
1 parent b5f715e commit 4f88d93
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 13 deletions.
2 changes: 1 addition & 1 deletion broker/consumer/lvk/vm_startup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ fi

#--- GCP resources used in this script
broker_bucket="${PROJECT_ID}-${survey}-broker_files"
PS_TOPIC_DEFAULT="${survey}-alerts_raw"
PS_TOPIC_DEFAULT="${survey}-alerts"
# use test resources, if requested
if [ "$testid" != "False" ]; then
broker_bucket="${broker_bucket}-${testid}"
Expand Down
16 changes: 6 additions & 10 deletions broker/setup_broker/lvk/setup_broker.sh
Original file line number Diff line number Diff line change
Expand Up @@ -55,9 +55,8 @@ if [ "${teardown}" != "True" ]; then
# create bigquery dataset and table
bq --location="${region}" mk --dataset "${bq_dataset}"

cd templates || exit
bq mk --table "${PROJECT_ID}:${bq_dataset}.${alerts_table}" "bq_${survey}_${alerts_table}_schema.json"
cd .. || exit
cd templates && bq mk --table "${PROJECT_ID}:${bq_dataset}.${alerts_table}" "bq_${survey}_${alerts_table}_schema.json" || exit 5
cd .. || exit 5

# create broker bucket and upload files
echo "Creating broker_bucket and uploading files..."
Expand Down Expand Up @@ -94,13 +93,10 @@ echo "Configuring VMs..."
#--- Deploy Cloud Functions
echo
echo "Configuring Cloud Functions..."
cd .. && cd .. || exit
cd cloud_functions && cd lvk || exit
cd .. && cd .. && cd cloud_functions && cd lvk || exit 5

#--- BigQuery storage cloud function
cd store_BigQuery || exit
./deploy.sh "$testid" "$teardown" "$survey"
cd store_BigQuery && ./deploy.sh "$testid" "$teardown" "$survey" || exit 5

#--- return to setup_broker directory
cd .. && cd .. || exit
cd .. && cd setup_broker || exit
#--- return to setup_broker/lvk directory
cd .. && cd .. && cd .. && cd setup_broker && cd lvk || exit 5
4 changes: 2 additions & 2 deletions broker/setup_broker/lvk/templates/bq_lvk_alerts_schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -89,13 +89,13 @@
"type": "TIMESTAMP"
},
{
"description": "estimation of the frequency of the signal's main component",
"description": "estimation of the frequency of the signal's main component [Hz]",
"mode": "NULLABLE",
"name": "central_frequency",
"type": "INTEGER"
},
{
"description": "Probability, under the assumption that the source is not noise, that at least one of the compact objects was a neutron star, that the system ejected a non-zero amount of neutron star matter, and that at least one of the compact objects has mass in the range 3-5 solar masses, respectively",
"description": "Probabilities, under the assumption that the source is not noise, regarding the physical characteristics of the event's progenitor and possible remnants.",
"fields": [
{
"description": "Probability that at least one of the compact objects was a neutron star",
Expand Down

0 comments on commit 4f88d93

Please sign in to comment.