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.

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

Google Apps Scipts function and python flask api. #2288

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

qnikil7
Copy link

@qnikil7 qnikil7 commented Aug 26, 2023

Description of the Change

Coolprop for google sheets.

I wanted to use coolprop in google sheets and wasn't able to find a way to call shared libraries in google sheet. So this is an alternate approach.

I have created a web app in python using flask which which is called through an api from google sheets using GET. I have hosted this currently in python anywhere. Link for google sheet.

Currently I have added only PropsSI and HAPropsSI functions.

Benefits

Use CoolProp in google sheets.

Possible Drawbacks

Sometimes the functions are slow to load.

Verification Process

  • There is a google sheet that demonstrates the functionality.
  • The google sheets have the https links to get the values.
  • test_api script to verify the values.

Applicable Issues

  • Probably calling a shared library using Apps Script (If possible) would be preferred.
  • If there is interest in this, probably hosting it somewhere and creating an addin in google workspace.
  • Also I selected python flask based on ease of use but may not be the most efficient.

@CLAassistant
Copy link

CLAassistant commented Aug 26, 2023

CLA assistant check
All committers have signed the CLA.

@ibell
Copy link
Contributor

ibell commented Aug 28, 2023

This is a very good idea and would be generally useful to have this service around and well-documented and supported. I found getting things working on PA to be not always so easy. I remember I had to re-write the app from scratch at least one time.

@ibell
Copy link
Contributor

ibell commented Sep 8, 2023

This is a nice idea! I wonder about accepting the PR though given that we cannot guarantee that your python anywhere instance will stay up. But it would indeed be handy to have a running API somewhere. Flask is a perfectly fine tool for this job.

@qnikil7 qnikil7 marked this pull request as draft September 9, 2023 17:51
@qnikil7
Copy link
Author

qnikil7 commented Sep 9, 2023

Thank you for the feedback.

I agree that the python anywhere instance would not be reliable. Moreover, would be good if its running somewhere where multiple people have access in case there are issues. I am not sure where to host this though.

On the google sheets side, would be good if its a google sheets add-in, maintained as a google cloud project, again multiple people having access to maintain.

I would like to improve it with better documentation and handling errors. Marking this PR as draft.

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.

None yet

3 participants