profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/dpgaspar/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Daniel Vaz Gaspar dpgaspar Lisbon, Portugal

dpgaspar/Flask-AppBuilder 3484

Simple and rapid application development framework, built on top of Flask. includes detailed security, auto CRUD generation for your models, google charts and much more. Demo (login with guest/welcome) - http://flaskappbuilder.pythonanywhere.com/

dpgaspar/Flask-AppBuilder-Skeleton 46

Skeleton application to use with Flask-AppBuilder (pip install flask-appbuilder)

dpgaspar/Flask-AppBuilder-Site 8

The Flask-AppBuilder Site

dpgaspar/Flask-AppBuilder-Skeleton-AddOn 6

Skeleton to Build AddOn's for Flask-AppBuilder

bkyryliuk/s3werkzeugcache 5

S3 implementation of the werkzeug cache

dpgaspar/fab_addon_audit 5

AddOn for Flask-AppBuilder. Will add audit to your ModelView's

dpgaspar/Mastering-Flask-Web-Development-Second-Edition 4

Mastering Flask Web Development, published by Packt

dpgaspar/incubator-superset 3

Apache Superset (incubating) is a modern, enterprise-ready business intelligence web application

dpgaspar/AutoAnt 2

AutoAnt is a generic processing automation package. It's ideal for file processing

dpgaspar/flask-babelPkg 2

i18n and l10n support for Flask based on Babel and pytz. Support for package development, merging translations

issue commentapache/superset

Support AWS OpenSearch / Elasticsearch

Hi @stockholmux, I'm a maintainer/creator of elasticsearch-dbapi. I'll look into including opensearch-py on elasticsearch-dbapi

nelson-lark

comment created time in 20 hours

PullRequestReviewEvent

Pull request review commentapache/superset

other: Provide option to add environment variables to only supersetNode

 spec:               value: {{ $value | quote }}             {{- end }}           {{- end }}+          {{- if .Values.supersetNode.env }}

is this if necessary?

dd-willgan

comment created time in 7 days

PullRequestReviewEvent
PullRequestReviewEvent

push eventdpgaspar/Flask-AppBuilder

Daniel Gaspar

commit sha 3c3c12c8a2d182ee15ba420e8ff2526a6d2559af

docs: fix blockdiag dependency on funcparserlib 2to3 deprecation on setuptools

view details

push time in 7 days

created tagdpgaspar/Flask-AppBuilder

tagdoc-test

Simple and rapid application development framework, built on top of Flask. includes detailed security, auto CRUD generation for your models, google charts and much more. Demo (login with guest/welcome) - http://flaskappbuilder.pythonanywhere.com/

created time in 7 days

PR opened dpgaspar/Flask-AppBuilder

docs: fix requirements funcparserlib

<!--- Thank you for contributing to Flask-Appbuilder. --> <!--- This repo uses a PR lint bot (https://github.com/apps/prlint), make sure to prefix your PR title with one of: --> <!--- build|chore|ci|docs|feat|fix|perf|refactor|style|test|other -->

Description

<!--- Describe the change below, including rationale and design decisions -->

ADDITIONAL INFORMATION

<!--- Check any relevant boxes with "x" --> <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->

  • [ ] Has associated issue:
  • [ ] Is CRUD MVC related.
  • [ ] Is Auth, RBAC security related.
  • [ ] Changes the security db schema.
  • [ ] Introduces new feature
  • [ ] Removes existing feature
+1 -0

0 comment

1 changed file

pr created time in 7 days

create barnchdpgaspar/Flask-AppBuilder

branch : fix/doc-requirements_2to3

created branch time in 7 days

delete branch dpgaspar/Flask-AppBuilder

delete branch : fix/docs-requirements

delete time in 7 days

push eventdpgaspar/Flask-AppBuilder

Daniel Vaz Gaspar

commit sha fdf57841cbb5fe52d8fbc5b7c60c03b073ce2930

docs: fix requirements (#1702)

view details

push time in 7 days

PR merged dpgaspar/Flask-AppBuilder

docs: fix requirements

Description

Fix read the docs requirements

ADDITIONAL INFORMATION

  • [ ] Has associated issue:
  • [ ] Is CRUD MVC related.
  • [ ] Is Auth, RBAC security related.
  • [ ] Changes the security db schema.
  • [ ] Introduces new feature
  • [ ] Removes existing feature
+1 -1

1 comment

1 changed file

dpgaspar

pr closed time in 7 days

PR opened dpgaspar/Flask-AppBuilder

docs: fix requirements

Description

Fix read the docs requirements

ADDITIONAL INFORMATION

  • [ ] Has associated issue:
  • [ ] Is CRUD MVC related.
  • [ ] Is Auth, RBAC security related.
  • [ ] Changes the security db schema.
  • [ ] Introduces new feature
  • [ ] Removes existing feature
+1 -1

0 comment

1 changed file

pr created time in 7 days

create barnchdpgaspar/Flask-AppBuilder

branch : fix/docs-requirements

created branch time in 7 days

push eventfedepad/Flask-AppBuilder

Federico Padua

commit sha 4b00aa4188fa5cbb3787785011418482383b1cc6

docs: fix issue 1700 (#1701) * Wrong property name. It was base_route but should have been route_base

view details

Daniel Vaz Gaspar

commit sha cf0b41a65142baa2fb0eb4af1d8a6c5c326ab8a5

Merge branch 'master' into ui/fix-datetimepicker-rendering

view details

push time in 7 days

push eventdpgaspar/Flask-AppBuilder

Federico Padua

commit sha 4b00aa4188fa5cbb3787785011418482383b1cc6

docs: fix issue 1700 (#1701) * Wrong property name. It was base_route but should have been route_base

view details

push time in 7 days

PR merged dpgaspar/Flask-AppBuilder

docs: fix issue 1700

<!--- Thank you for contributing to Flask-Appbuilder. --> <!--- This repo uses a PR lint bot (https://github.com/apps/prlint), make sure to prefix your PR title with one of: --> <!--- build|chore|ci|docs|feat|fix|perf|refactor|style|test|other -->

Description

<!--- Describe the change below, including rationale and design decisions -->

  • Wrong property name. It was base_route but should have been route_base
  • [x] Has associated issue: Fixes #1700
+2 -2

2 comments

1 changed file

fedepad

pr closed time in 7 days

issue closeddpgaspar/Flask-AppBuilder

Type error in the official rest_api example code

I am a new FAB user here, and found a minor but frustrating error in the official documentation.

  • https://flask-appbuilder.readthedocs.io/en/latest/rest_api.html
  • Flask-AppBuilder/docs/rest_api.rst

I was trying to follow through the above rest_api example, and copy the exact code below to my editor but it did not work. It turned out that it is just a simple type error. It should use [route_base] instead of [base_route], please correct this asap so as not to turn off new users.

`from flask_appbuilder.api import BaseApi, expose from . import appbuilder

class ExampleApi(BaseApi):

base_route = '/newapi/v2/nice'

@expose('/greeting')
def greeting(self):
    return self.response(200, message="Hello")

appbuilder.add_api(ExampleApi)`

closed time in 7 days

journeast
PullRequestReviewEvent

pull request commentdpgaspar/Flask-AppBuilder

docs: fix issue 1700

nice! thks

fedepad

comment created time in 7 days

PullRequestReviewEvent
PullRequestReviewEvent

Pull request review commentapache/superset

feat: add global max row limit

 def _try_json_readsha(filepath: str, length: int) -> Optional[str]: # Set this API key to enable Mapbox visualizations MAPBOX_API_KEY = os.environ.get("MAPBOX_API_KEY", "") -# Maximum number of rows returned from a database-# in async mode, no more than SQL_MAX_ROW will be returned and stored-# in the results backend. This also becomes the limit when exporting CSVs+# Maximum number of rows returned from a database, both SQL Lab, chart data+# requests and CSV exports

is it safe to say, on any query for a registered db connection?

villebro

comment created time in 8 days

Pull request review commentapache/superset

feat: add global max row limit

 from superset.utils.hashing import md5_sha_from_dict from superset.views.utils import get_time_range_endpoints +if TYPE_CHECKING:+    from superset.common.query_context import QueryContext

nit: # pragma: no cover

villebro

comment created time in 8 days

PullRequestReviewEvent
PullRequestReviewEvent

Pull request review commentapache/superset

feat: add support for generic series limit

 class QueryObject:  # pylint: disable=too-many-instance-attributes     annotation_layers: List[Dict[str, Any]]     applied_time_extras: Dict[str, str]     apply_fetch_values_predicate: bool-    granularity: Optional[str]+    columns: List[str]+    datasource: Optional[BaseDatasource]+    extras: Dict[str, Any]+    filter: List[QueryObjectFilterClause]     from_dttm: Optional[datetime]-    to_dttm: Optional[datetime]+    granularity: Optional[str]     inner_from_dttm: Optional[datetime]     inner_to_dttm: Optional[datetime]+    is_rowcount: bool     is_timeseries: bool-    time_shift: Optional[timedelta]-    groupby: List[str]-    metrics: Optional[List[Metric]]-    row_limit: int-    row_offset: int-    filter: List[QueryObjectFilterClause]-    timeseries_limit: int-    timeseries_limit_metric: Optional[Metric]     order_desc: bool-    extras: Dict[str, Any]-    columns: List[str]     orderby: List[OrderBy]-    post_processing: List[Dict[str, Any]]-    datasource: Optional[BaseDatasource]+    metrics: Optional[List[Metric]]     result_type: Optional[ChartDataResultType]-    is_rowcount: bool+    row_limit: int+    row_offset: int+    series_columns: List[str]+    series_limit: int+    series_limit_metric: Optional[Metric]     time_offsets: List[str]+    time_shift: Optional[timedelta]+    to_dttm: Optional[datetime]+    post_processing: List[Dict[str, Any]]      def __init__(  # pylint: disable=too-many-arguments,too-many-locals         self,-        datasource: Optional[DatasourceDict] = None,-        result_type: Optional[ChartDataResultType] = None,         annotation_layers: Optional[List[Dict[str, Any]]] = None,         applied_time_extras: Optional[Dict[str, str]] = None,         apply_fetch_values_predicate: bool = False,-        granularity: Optional[str] = None,-        metrics: Optional[List[Metric]] = None,-        groupby: Optional[List[str]] = None,+        columns: Optional[List[str]] = None,+        datasource: Optional[DatasourceDict] = None,+        extras: Optional[Dict[str, Any]] = None,         filters: Optional[List[QueryObjectFilterClause]] = None,-        time_range: Optional[str] = None,-        time_shift: Optional[str] = None,+        granularity: Optional[str] = None,+        is_rowcount: bool = False,         is_timeseries: Optional[bool] = None,-        timeseries_limit: int = 0,-        row_limit: Optional[int] = None,-        row_offset: Optional[int] = None,-        timeseries_limit_metric: Optional[Metric] = None,+        metrics: Optional[List[Metric]] = None,         order_desc: bool = True,-        extras: Optional[Dict[str, Any]] = None,-        columns: Optional[List[str]] = None,         orderby: Optional[List[OrderBy]] = None,         post_processing: Optional[List[Optional[Dict[str, Any]]]] = None,-        is_rowcount: bool = False,+        result_type: Optional[ChartDataResultType] = None,+        row_limit: Optional[int] = None,+        row_offset: Optional[int] = None,+        series_columns: Optional[List[str]] = None,+        series_limit: int = 0,+        series_limit_metric: Optional[Metric] = None,+        time_range: Optional[str] = None,+        time_shift: Optional[str] = None,         **kwargs: Any,     ):         columns = columns or []-        groupby = groupby or []         extras = extras or {}         annotation_layers = annotation_layers or []         self.time_offsets = kwargs.get("time_offsets", [])         self.inner_from_dttm = kwargs.get("inner_from_dttm")         self.inner_to_dttm = kwargs.get("inner_to_dttm")+        if series_columns:+            self.series_columns = series_columns+        elif is_timeseries and metrics:+            self.series_columns = columns+        else:+            self.series_columns = []

I have no strong opinions here, but, I do like to use simple ternaries. But current implementation seems more readable to me because of the nested ternary.

villebro

comment created time in 9 days

PR closed preset-io/elasticsearch-dbapi

update has_table to accept view or table

When trying to add a view as a data source in superset, it will error out because has_table only return true for "BASE TABLE" type. This was confirmed when looking at the other dialects that return true for views and tables.

+18 -1

1 comment

4 changed files

ccary64

pr closed time in 10 days

pull request commentpreset-io/elasticsearch-dbapi

update has_table to accept view or table

this was fixed on Superset: https://github.com/apache/superset/pull/16624 currently on master

ccary64

comment created time in 10 days

push eventdpgaspar/Flask-AppBuilder

Daniel Vaz Gaspar

commit sha 3d351c913d7e1e686e3ea939278c9afd5ac2d6b9

release: 3.3.3 (#1699)

view details

Daniel Vaz Gaspar

commit sha a5535078168d77473a2f508a53a3df416a629b52

Merge branch 'master' into chore/add-type-annotation

view details

push time in 10 days