text:
Table of tests coverage

We need to document the test coverage.
I have pushed a page tests-coverage.md to master and enabled GitHub Pages so that it shows at https://data-apis.org/array-api-tests/tests-coverage.html (it also created a page for the README).  I started a basic table with statistical functions, based on https://github.com/data-apis/array-api-tests/pull/39. It's not yet completely filled out.
@honno Let me know what you think of the general styling and the categories. My thinking is each cell will have "Yes", "No", or "N/A", and if there is more to say about something we can add a footnote. The idea is based on tables you often see on Wikipedia (e.g., this one).
Feel free to edit it and to fill it out more. You can push directly to master if you want. I don't know if there is any other way to see a preview of the page without pushing to master and letting GitHub render it. You can also change the theme if you want (see https://github.com/data-apis/array-api-tests/settings/pages). I was also thinking of adding some Javascript that colors the cells green if they have "Yes" in them and red otherwise.
By the way, if you use emacs, the markdown-mode makes it really easy to edit the table. You can just press tab and enter to move between cells, and it automatically realigns them. I don't know if there are similar plugins for other editors.

IMO I think such a granular table isn't worth it. I'm going to spend time filling it out, and then 100% forget to update it in lockstep with test suite changes. We're already ~80% Yes with #39 anywho.
I think it's better to express the goals of the test suite, and just state that it's WIP for now, maybe giving broad allusions to what it lacks (testing some smaller function groups, usability things like nice error messages, etc.).

Well my understanding from the meeting is that we do want it. What do you think @rgommers @kgryte? If we're already close, then we should think about whether we are missing any categories. Otherwise, I think this will be useful to help us understand the remaining 20% that is missing.
Outlining other big TODOs here like better reporting is a good idea. Right now most of those ideas only live in the issue tracker, but that isn't very visible to people using the test suite, and I've often found myself re-explaining it.

@honno's point is about the level of granularity I think. This will be a huge table, perhaps it will be easier to parse if it had a single entry/row for "statistical functions" rather than one for each function. That would make sense to me. You'd implement support for a particular type of test (say promotion behavior) for a set of functions at once normally. So little info will be lost, and you may make the table ~5x shorter.

Well now I'm writing tests that that covers all bases at once, and all the existing tests cover all bases, so we can just say now "we need to implement set + utility + searching functions" in a sentence and not have any table or list at all.

Let's clarify what we want. My impression was we want a resource that could give Array API implementers a high-level overview of what this test suite does and doesn't. The current page does a great job doing that, so I'm thinking we just need to tweak it (remove table, readability changes, etc.) and we'll be done.

As discussed in our meeting, let's prioritise expressing the broad strokes for now, and sit with a "completed" test suite sit for a while before documenting the granular details. Good news that's mostly done, so I'll:

[ ] Review the page as-is and fix any minor errors
[ ] See about readability changes i.e. just restructuring the content a bit so skim-readers will still get a good impression
[ ] Go half-way with granularity for now, expressing what our tests generally do per function group

I think users of the test suite have gotten the jist of what the test suite is actually testing with the README as-is, so closing (feel free to re-open).

authors: asmeurer honno rgommers

repo: data-apis/array-api-tests


text:
cw_canaries stack fails due to unsupported CloudWatch synthetics version

Describe the bug
Refer to line 84 of deploy/stacks/cw_canaries.py here.
According to Runtime versions using Python and Selenium Webdriver, the runtime version syn-python-selenium-1.0 is no longer supported.
When the nested stack is created, the following error occurs:
Resource handler returned message: "Invalid request provided: Deprecated runtime version specified. (Service: Synthetics, Status Code: 400, Request ID: 2615cc97-59ab-42f1-b0df-4db59ce5ece6)" (RequestToken: 9b8c547f-26ec-45b2-423c-c19c883e5aba, HandlerErrorCode: InvalidRequest)

How to Reproduce
Include the configuration parameters to activate canaries in cdk.json:
    ...
    "enable_cw_canaries": true
    ...

Then, deploy dataall.
Expected behavior
The deployment should complete without error.
Your project
No response
Screenshots
No response
OS
Mac
Python version
3.12.2
AWS data.all version
2.6.0
Additional context
No response

Hi @jpdev42 - thank you for bringing this issue to our attention. We will look into the above and see if any blockers to upgrading the runtime version to the latest (or non-deprecated)
Will do some exploration on the above but please do let us know if you were able to find a workaround for the above issue as well

source: gharchive/issue

repo: data-dot-all/dataall

license_type: permissive

license_source: github-api


text:
Code fixes from develop to release/1.5

Code fixes from develop to release/1.5

Is the geo location commit intended to be here ?

Is the geo location commit intended to be here ?

it is just a documentation change in md file.

https://github.com/data-integrations/salesforce/pull/138 - this PR needs to be part of release 1.5

the release branch already contains the PR - https://github.com/data-integrations/salesforce/pull/138 that is required to be in v1.5. As long as we rebase and merge keeping these changes, we should be fine.

source: gharchive/pull-request

repo: data-integrations/salesforce

license_source: bigquery


text:
pass ctx properly in internal client.ExecuteStatement

Fixes https://github.com/databricks/databricks-sql-go/issues/242
Fixes https://github.com/databricks/databricks-sql-go/issues/231
This needs to actually pass ctx so custom authenticators can make use of the context.

@kravets-levko can you take a look at this PR, seems like it was some leftover code from a couple of years ago, im not entirely sure if the comment i removed in there still applies

repo: databricks/databricks-sql-go


text:
Fixes normalization behavior that erased non-URL funding identifiers when reading from DataCite XML. Adds normalization for Crossref Funder ID and ROR funderIdentifierTypes.

Purpose

When reading DataCite XML, bolognese would not return the funder identifier in a funding reference if the funder identifier did not originally appear as a URL.
closes: #142
Approach

When processing funding_references, a normalization step would attempt to render the funder_identifier as a URL:
https://github.com/datacite/bolognese/blob/dc8170d2f44271bc20a9c649c10130422007a4bf/lib/bolognese/readers/datacite_reader.rb#L161
If the schemeURI was present, normalize_id would be called on a concatenation of the schemeURI and funder identifier. Otherwise, normalize_id would be called on the funder identifier. If normalize_id could not render either as a URI, funder_identifier would become nil:
https://github.com/datacite/bolognese/blob/dc8170d2f44271bc20a9c649c10130422007a4bf/lib/bolognese/utils.rb#L614
This PR normalizes Crossref Funder IDs and RORs but performs no normalization steps otherwise, avoiding a funder_identifier nil value. It also adds the optional schemeUri attribute to the rendered funding reference, which was originally not included.
Open Questions and Pre-Merge TODOs

[] Does including schemeUri here have any downstream issues?
[] What changes in lupo are necessary to properly import and index fundingReference schemeUris?
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

[ ] New feature (non-breaking change which adds functionality)

[ ] Breaking change (fix or feature that would cause existing functionality to change)

Reviewer, please remember our guidelines:

Be humble in the language and feedback you give, ask don't tell.
Consider using positive language as opposed to neutral when offering feedback. This is to avoid the negative bias that can occur with neutral language appearing negative.
Offer suggestions on how to improve code e.g. simplification or expanding clarity.
Ensure you give reasons for the changes you are proposing.

This fix looks good :)
We might want to consider still calling the normalize_id function for types other than Crossref Funder and ROR, as it also cleans up URLs when present (removes UTM fragments etc, see https://github.com/postrank-labs/postrank-uri#example), which might be a useful thing to keep.
In that case, my proposition would be to restore the
if funder_identifier_type != "Other"
            funder_identifier = !funder_identifier.to_s.start_with?("https://","http://") && scheme_uri.present? ? normalize_id(scheme_uri + funder_identifier) : normalize_id(funder_identifier)

lines and make a modification to normalize_id here:
https://github.com/datacite/bolognese/blob/dc8170d2f44271bc20a9c649c10130422007a4bf/lib/bolognese/utils.rb#L623
to return id instead of nil, which essentially means that URLs still get cleaned up, but any non-URL value passed will just be returned as-is when it fails the if statement on that line.

Looking at normalize_id elsewhere, it seems like the function is used to normalize/validate URLs specifically:
https://github.com/datacite/bolognese/blob/dc8170d2f44271bc20a9c649c10130422007a4bf/lib/bolognese/readers/schema_org_reader.rb#L29
Also, some of the tests expect nil values:
https://github.com/datacite/bolognese/blob/dc8170d2f44271bc20a9c649c10130422007a4bf/spec/utils_spec.rb#L175
Maybe a separate function would be best? Although the naming of normalize_id certainly confuses things.

Using this logic flow:

Attempt to parse URL

If valid URL, check the scheme

If not http or https, return nil
If http or https, proceed to URL cleanup

If not valid URL, return as is

. . . it seems the following code gets flummoxed since it depends on input being nil if input is not a valid URL:
https://github.com/datacite/bolognese/blob/e16d372a8e72f26f9aa18d835464eb7d6124a9f8/lib/bolognese/metadata.rb#L20
This results in a few tests failing, like this:
https://github.com/datacite/bolognese/blob/e16d372a8e72f26f9aa18d835464eb7d6124a9f8/spec/readers/ris_reader_spec.rb#L16
Rather than refactoring metadata, I opted for a less elegant normalization step in datacite_reader. It doesn't accomplish

* If not http or https, return `nil`

. . . but it does normalize http and https URLs. @digitaldogsbody , do you think more work needs to be done here?

. @digitaldogsbody , do you think more work needs to be done here?

I think this is a good solution for now. My only outstanding question would be about no longer passing scheme_uri when it is present, as it did with the original behaviour:
if funder_identifier_type != "Other"
            funder_identifier = !funder_identifier.to_s.start_with?("https://","http://") && scheme_uri.present? ? normalize_id(scheme_uri + funder_identifier) : normalize_id(funder_identifier)

You might consider adding that logic back into the code in your else block here: https://github.com/datacite/bolognese/pull/143/files#diff-4cfebbeb62130d0a784d3dcc28af41aed547f5bd4da1ec9e4600419e0c4638b0R167, just in case it is depended on somewhere that isn't well tested.

authors: codycooperross digitaldogsbody

repo: datacite/bolognese

license: mit


text:
[sustainability] add more logging

Added more logging to debug flaky sustainability/Earth page (logging lines are quite verbose so these are temporary)
Fix a small bug

thanks for the quick review!

repo: datacommonsorg/website


text:
Dotted/disputed borders

Lebanon should have a solid border. Depending on the zoom level of the map, the border changes from solid to dotted. This happens with other countries as well that should have solid lines.

Basically can we switch to a data source (maybe Natural Earth) that doesn't include disputed borders? Or should we make all disputed borders solid black lines too?

hide settlements in Israel/West Bank (for now) since they come in as "regions" or state level

repo: datadesk/web-map-maker


text:
Correction for raw affiliation Department of Aerospace Engineering Ryerson University

Correction needed for raw affiliation Department of Aerospace Engineering Ryerson University
raw_affiliation_name: Department of Aerospace Engineering Ryerson University
new_rors: 05g13zd79
previous_rors:
works_examples: W4250566442;W4237159913
contact: e2e691803ca35d3549839afc1a1cc015:06709e42 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.

repo: dataesr/openalex-affiliations

license: MIT


text:
Correction for raw affiliation Laboratoire de Biologie et Modelisation de la Cellule, Ecole Normale Superieure de Lyon, CNRS, UMR 5239, Inserm, U293, Universite Claude Bernard Lyon , 46 allee d'Italie F-69364 Lyon, France

Correction needed for raw affiliation Laboratoire de Biologie et Modelisation de la Cellule, Ecole Normale Superieure de Lyon, CNRS, UMR 5239, Inserm, U293, Universite Claude Bernard Lyon , 46 allee d'Italie F-69364 Lyon, France
raw_affiliation_name: Laboratoire de Biologie et Modelisation de la Cellule, Ecole Normale Superieure de Lyon, CNRS, UMR 5239, Inserm, U293, Universite Claude Bernard Lyon , 46 allee d'Italie F-69364 Lyon, France
new_rors: 02vjkv261;029brtt94;02feahw73;04zmssz18;01bj4fd12
previous_rors: 02vjkv261;029brtt94;02feahw73;04zmssz18
works_examples: W3018273493
contact: 7aed393f7568e73b5897c9659ed0e940:0d908b34d80201c9cc433bb6b860 @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


text:
Correction for raw affiliation EDF R&D; Institut Photovoltaïque d'Île-de-France

Correction needed for raw affiliation EDF R&D; Institut Photovoltaïque d'Île-de-France
raw_affiliation_name: EDF R&D; Institut Photovoltaïque d'Île-de-France
new_rors: 03wb8xz10
previous_rors: 05jxfge78
works_examples: W4391010666;W4391461371;W4391461393
contact: 65dc56cdc4b5c28b4c3e95a170675758:dca95dcdd81b4c09b333ea3be03ba9 @ recherche.gouv.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


text:
fix(api): Supporting modificationCategory in oss

This is migration from this sass PR: https://github.com/acryldata/datahub-fork/pull/1495
Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

This change doesn't do much, unless all of the sass code for this file is added here. So closing the PR.

repo: datahub-project/datahub


text:
docs(bigquery): add change log and unit tests

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Created another PR without conflicts


text:
Worked on the feature to update group title

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

you're getting failing tests on CreateDomainResolverTest, CreateGlossaryNodeResolverTest, and CreateGlossaryTermResolverTest. Anywhere you instantiate CreateDomainResolver, CreateGlossaryNodeResolver, or CreateGlossaryTermResolver in those tests, you're going to need to pass in a new mock entity service.
Check out AddTagsResolverTest.java where we pass in EntityService mockService = Mockito.mock(EntityService.class); to AddTagsResolver resolver = new AddTagsResolver(mockService);


text:
Fix/PRD-1067 assert error

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Closing as this looks like a duplicate of https://github.com/datahub-project/datahub/pull/9888

authors: Kunal-kankriya asikowitz


text:
Optimize devcontainer environment

[x] Reduce image size (~3GB -> ~2GB).
[x] Improve build time through better caching
[x] Persist MySQL data. To purge, users can do this manually or simply recreate the Codespace.
[x] Add schema clean up function
[x] Set notebook kernel
[x] Cache docker images
[x] Update environment launch instructions

In light of the upcoming demo at Bonn University I'll merge as-is since it seems functional.

repo: datajoint/workflow-array-ephys


text:
Add independent expenditure scraper

Overview
See title.
Closes #27
Testing instructions
Run python -m scrapers.financial_disclosure.scrape_independent_expenditures

@antidipyramid I'm good for this to come in. You can address any data issues Marjorie reports, separately.

authors: antidipyramid hancush

repo: datamade/nmid-scrapers


text:
RepeatedLabelError when street name contains a number

Hi there, thanks for providing this module, it's super useful! I'm trying to use it to parse addresses that in a dataset that came from our county's Voter Registration office, but I'm having problems with some of the addresses. Specifically, some of the addresses have a street name that contains a number, because the street name is just 'STATE ROUTE NNN', for example:

>>> usaddress.tag('261  STATE ROUTE 590  ELMHURST TWP, PA  18444')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/luser/.pyenv/versions/2.7.11/lib/python2.7/site-packages/usaddress/__init__.py", line 178, in tag
    label)
usaddress.RepeatedLabelError:
ERROR: Unable to tag this string because more than one area of the string has the same label

ORIGINAL STRING:  261  STATE ROUTE 590  ELMHURST TWP, PA  18444
PARSED TOKENS:    [(u'261', 'AddressNumber'), (u'STATE', 'StreetNamePreType'), (u'ROUTE', 'StreetNamePreType'), (u'590', 'AddressNumber'), (u'ELMHURST', 'StreetName'), (u'TWP,', 'PlaceName'), (u'PA', 'StateName'), (u'18444', 'ZipCode')]
UNCERTAIN LABEL:  AddressNumber

When this error is raised, it's likely that either (1) the string is not a valid person/corporation name or (2) some tokens were labeled incorrectly

To report an error in labeling a valid name, open an issue at https://github.com/datamade/usaddress/issues/new - it'll help us continue to improve probablepeople!

For more information, see the documentation at https://usaddress.readthedocs.io/

Here's the full list of addresses that hit this error from the dataset: addresses.txt

Hey @luser,
Thanks for filing this issue! I can add your list of addresses to our next round of training data. If this pattern error is blocking your work, there are two steps you can take to work around it:

Preprocess your addresses to add commas  before the PlaceName in the case of State Route addresses, e.g.:

In [3]: usaddress.tag('261  STATE ROUTE 590,  ELMHURST TWP, PA  18444')
Out[3]:
(OrderedDict([('AddressNumber', '261'),
              ('StreetNamePreType', 'STATE ROUTE'),
              ('StreetName', '590'),
              ('PlaceName', 'ELMHURST TWP'),
              ('StateName', 'PA'),
              ('ZipCode', '18444')]),
 'Street Address')

Retrain the model locally with your new training data. Remember to make a PR so we can bring your changes in!

Either works well for us – let me know what you decide.
-jean

I resorted to the typical hack of "prefilter the data with a regex" which worked well enough to get me through what I needed to do. :)

Great to hear! Let us know if any more issues come up. I'll keep this issue open until your training data is added to a new release, at which point I'll close it.

authors: jeancochrane luser

repo: datamade/usaddress


text:
Chane cell limit for Table

Hello,
is it possible to change the cell limit of the Table object? I understand that DataTable does not have a cell constraint but the layout of the Table object fits my needs better. I am using datapane 0.15.4.

Hi @julian-belina what kind of cell limit do you need? Are you looking for a cell limit like "m rows x n columns"?

Yes exactly. When I try to display a large dataframe using the table object, I get an error message saying that the cell number is limited to 100 cells. I would like to increase this number even if it takes longer to create the report.

Got it. Let me discuss this with the team and get back to you.

A feature to (automatically) increase the row height so that the whole content of each cell could be seen without scrolling sidewards would be need. This would increase the readability of the table when inspecting the rows sequentially.

Hello @julian-belina!
We'll be upping the limit on dp.Table in an upcoming release - so we won't have to deal with the 100 cell limit for much longer. I'll leave this issue open with the release-pending tag for yourself and others to track until it's released.

A feature to (automatically) increase the row height so that the whole content of each cell could be seen without scrolling sidewards would be neat. This would increase the readability of the table when inspecting the rows sequentially.

The feature sounds interesting! Please post it over on our feature request board where you can track it and others can pitch in too 🚀

@julian-belina thanks for the feedback. As @shahinrostami said, please use our feature request board and make a suggestion. How many cells are you planning to show (200, 500, 1000, a lot more?)

I am using the table to display a debug log. Potentially there could be a lot of rows. In my test cases there are currently approximately 5000 cells. Having the filter features from the DataTable would great as well but the readability is the most important feature in the moment. I have created a post on the feature board.

Thanks for telling us about your debug log use-case. This is a great example. We will continue to look into this.

The cell limit has been removed @julian-belina , it's available in 0.15.5!

authors: julian-belina shahinrostami woobe

repo: datapane/datapane


text:
Make verification flow on the n0lleserver also add n0llegruppsroll to n0llan

Add n0llegruppsroll to Hodis and make the verification flow also add n0llegruppsroll to n0llan.
Note: This issue is low priority and only needs to be fixed before Mottagningen 2023.

Also intis

authors: lucubrator nylanderdev

repo: datasektionen/harmony


text:
Helm Chart : setting ambassador.id on helm installation breaks ambassador

Describe the bug
If the ambassador.id variable is set the chart installation works but ambassador doesn't.
To Reproduce

Installation:

 helm upgrade --install ambassador-dev --version 1.1.5 --namespace ambassador --wait --set ambassador.id=development stable/ambassador

Error:

[2019-02-28 09:59:39.502][000075][warning][config] [bazel-out/k8-dbg/bin/source/common/config/_virtual_includes/grpc_mux_subscription_lib/common/config/grpc_mux_subscription_impl.h:70] gRPC config for type.googleapis.com/envoy.api.v2.Listener rejected: Error adding/updating listener ambassador-listener-80: cannot bind '0.0.0.0:80': Permission denied

Changing the id to default fixes the issue:

helm upgrade --install ambassador-dev --version 1.1.5 --namespace ambassador --wait --set ambassador.id=default stable/ambassador

Versions (please complete the following information):

Ambassador: v0.50.3
Kubernetes environment : v1.11.6

Are you using the new helm chart https://hub.helm.sh/charts/stable/ambassador?

Yeah, version 1.1.5 from the stable repository

Nothing else changing? In the first screenshot it's trying to listen on port 80 which is a priveleged port, but the chart is by default running as non root.
In second screenshot it's using the charts default port of 8080.
And no other changed values from default? :)

I was trying to discard factors and I found out by trial and error that the bug happened when I set the ambassador.id value. if I leave it on default, that is "default", it works.
Sadly I need to have one ambassador instance per environment.

Same error i am also facing, i too running with custom ambassador id

when I add the ambassador id, the envoy proxy starts listening on port 80, which means, that means its not respecting the module configs service_port.

@ysaakpr @hugocarrasco
You have to set ambassador_id for the ambassador module
ambassador:
  id: my-awesome-ambassador

service:
  annotations:
    getambassador.io/config: |
      ---
      apiVersion: ambassador/v1
      kind: Module
      name: ambassador
      ambassador_id: my-awesome-ambassador
      config:
        service_port: 8080

Just got it working...I also found that in the ambassador config i was missing the ambassador id.
@Flydiverny thank you..made my day

authors: Flydiverny hugocarrasco richarddli ysaakpr

repo: datawire/ambassador


text:
Envoy v1.6.0

Please describe your use case / problem.
Is it possible to upgrade to V1.6.0 of the envoy, this release has som nice routing config based on query string.
Describe the solution you'd like
This sould then be possible :
annotations:
getambassador.io/config: |
---
apiVersion: ambassador/v0
kind: Mapping
name: mapping-getmap
prefix: /dataset
rewrite: /geoserver1/map
service: geoserver:8080
envoy_override:
query_parameters:
- name: service
value: GetMap
With kind regards,
Dennis Rutjes

Envoy 1.6.0 is now part of 0.34.2. Can you give it a try? (And would be happy to take a documentation pull request with an example of what you try to do!)

thanks!

Dennis,
It did not work for me. Did it work for you?
Ambassador Version: 0.39.0
Envoy Version: 1.7.0
getambassador.io/config: |       ---       apiVersion: ambassador/v0       kind:  Mapping       name:  pricing_prc-role-v351_v3_mapping       prefix: /pricing/prc-role/v3/       service: prc-role-v351.pricing:80       tls: false       envoy_override:         query_parameters:           - name: brand             value: brand1

authors: DennisRutjes jpfaria richarddli


text:
Updated TLS documentation

Updates to the TLS documentation for clarification.
Many old references have been removed and some have been kept. These changes can be reverted if we think we need to keep these old references around but I think they confuse the documentation.
TODO: Create document for serving HTTP and HTTPS on the same domain.

Regarding serving HTTP/HTTPS from same domain -- is that a separate document or can we cover that as an option in the TLS explanation docs?

authors: nbkrause richarddli


text:
Change objects_to_process tuple to use full filepath

My attempt at a fix for #494

I just realised this PR could perhaps use a renaming from filename to filepath in the process_all_objects function? Let me know if this fix seems sound and if you'd prefer the variable be renamed.

Yeah, I think I agree that filename needs to become filepath in Config.load_yaml and Config.process_all_objects... which leads us to the ugliness of whether to rename self.filename. Sadly, I think it's probably less ugly to do that now rather than waiting. 😦

I changed some more instances of filename but then I realized it's used in the source object which would require changing a lot of test json files.
I'll get a test environment setup tomorrow so I can run python tests and try it out in minikube!

@kflynn I wrote another possible fix that involves less drastic changes for the code base and the unit tests seem to work with it: https://github.com/datawire/ambassador/compare/master...lavoiedn:fix-process_all_objects-same-filename-redux?expand=1&w=1
If you'd rather I change the "filename" key/value in every instance of the source dict, I can do that too! I just don't want to break a portion of the code that uses "filename". I didn't seem to find any instance in the code that required source.filename to be unique across multiple sources, except maybe in the diagnostic html?
$ grep -r -e "{{ source.filename }}"
ambassador/templates/diag.html:
ambassador/templates/diag.html:

Let me know which solution you prefer! I think using filepath everywhere would be cleaner and more of a long-term fix, but I'm not sure of the implications.

Okay just confirmed that the second fix does not work with the diagnostics dashboard. We're out on a team building event this afternoon but I'll be able to invest more time into this tomorrow!

authors: kflynn lavoiedn


text:
manual assistant not working

python run.py  --agents-definition-folder definitions/boss-worker3/
gives an error:
File "[..]/OpenAI_Agent_Swarm/agents/manual_assistants/logger.py", line 21, in emit
'name': self.agent.id,
^^^^^^^^^^^^^
AttributeError: 'Agent' object has no attribute 'id'

fixed by recent commits.

authors: Quenos thehunmonkgroup


text:
mapillary.com support

Mapillary is a streetview-like.

Mapillary supported from version 0.4.3, released today. Thanks for your PR!

authors: david-r-edgar hugobqd

repo: david-r-edgar/MapSwitcher


text:
Use of LocalStorage is shared between all IPFS content

This application uses LocalStorage to store the "Last File" (filename) and "Last Document" (file content) keys.
Since markup.rocks is distributed over IPFS, and since every IPFS page is accessed from the same domain (the gateway), these keys are available to all other IPFS pages.
I'm pretty sure this isn't the only application that is vulnerable, and I don't yet know what the best fix is, but it's at least worth being aware of this.

E.g. check out https://ipfs.io/ipfs/QmYuNaKwYFjXipWufudu9Ru8NweJ322qwsPZWatWij1i2f after storing something using markup.rocks - it's a completely unrelated IPFS page.


text:
Add support for autogenerating stub files

It be super cool if we could extend the derive macros to be able to also store metadata information about the compiled module and automatically generate stub files. For example:
#[php_class(name = "Vodik\\Example")]
#[derive(Debug, Clone)]
struct Example(...);

#[php_impl]
impl Example {
    pub fn __construct(arg: u32) -> Self {
        // ...
    }

    /// Example documentation
    pub fn get_value(&self) -> u32 {
        // ...
    }

could result in automatically emitting a PHP file like this:
namespace Vodik\Example
{
    class Example
    {
        public function __construct(int $arg) {}

        /**
         * Example documentation
         */
        public function getValue(): int {}
    }
}

Another useful feature might be to automatically template out composer project as well? I don't know much about how native extensions are package, but at the very least ids stub could be packaged and installed via composer.

POC working here, keen to hear what you all think.
At the moment a function is generated in the extension with the signature fn ext_php_rs_describe_module() -> ext_php_rs_describe::Module {}. This function is only generated when debug_assertions are enabled (not cargo build --release). The returned module contains all the information that the macros crate has collected throughout the extension. Kinda similar to the PHP get_module function but with more detail (perhaps the get_module function can be modified to call this new describe function instead).
User can then call the CLI application (at the moment called ext-php-rs-cli but I'd like a different name) passing the path to the extension. CLI application loads the dylib, locates the describe symbol and calls the function. Stubs file is generated and outputted in the working directory at <ext-name>.stubs.php. This will only work if the describe function is present (i.e. won't work for non-ext-php-rs extensions).
Broken/not implemented:

Comments aren't detected so aren't written to the stub file. AFAIK this requires the PHP macro to be on top of the function?
Indentation is wrong.
Namespaces aren't accounted for (will generate class Vodik\Example\Example { /* ... */ }).
Property type and nullability is not described, and properties do not support visibility and static/non-static.

Ok, so I've consolidated some features down into a Cargo subcommand (will distribute as cargo-php, can call it as cargo php <...>:
$ cargo php
cargo-php 0.1.0

David Cole <david.cole1340@gmail.com>

Installs extensions and generates stub files for PHP extensions generated with `ext-php-rs`.

USAGE:
    cargo-php <SUBCOMMAND>

OPTIONS:
    -h, --help       Print help information
    -V, --version    Print version information

SUBCOMMANDS:
    help       Print this message or the help of the given subcommand(s)
    install    Installs the extension in the current PHP installation
    remove     Removes the extension in the current PHP installation
    stubs      Generates stub PHP files for the extension

Can install extensions into the PHP distribution (and remove them). Also generates stubs. Here's a basic extension:
use ext_php_rs::prelude::*;

/// Does some testing.
/// This is another doc comment.
#[php_class(name = "Example\\Test")]
struct Test {
    /// Test comment on `a` property.
    #[prop]
    a: i32,
    b: i64,
}

#[php_impl]
impl Test {
    /// does comment work
    const ASSOCIATED_CONST: i32 = 100;

    /// Constructor method. Test!
    pub fn __construct(a: i32, b: i64) -> Test {
        Self { a, b }
    }

    /// test_method does nothing.
    ///
    /// @param int $a input.
    ///
    /// @returns another int.
    pub fn test_method(&self, a: i32) -> i32 {
        a
    }

    /// Returns the two values in a string.
    pub fn print_values(&self) -> String {
        format!("a: {}, b: {}", self.a, self.b)
    }
}

/// Function comment here!
///
/// @param int $x Number
#[php_function]
pub fn mod_test(x: i32) -> i32 {
    x * 5
}

/// Some random constant.
#[php_const]
const X: i32 = 5;

#[php_module]
pub fn get_module(module: ModuleBuilder) -> ModuleBuilder {
    module
}

Associated stub file:
$ cargo php stubs --stdout
    Finished dev [unoptimized + debuginfo] target(s) in 0.03s
<?php

// Stubs for ext-php-rs-test

namespace {
    /**
     * Some random constant.
     */
    const X = null;

    /**
     * Function comment here!
     *
     * @param int $x Number
     */
    function mod_test(int $x): int {}
}

namespace Example {
    /**
     * Does some testing.
     * This is another doc comment.
     */
    class Test {
        /**
         * does comment work
         */
        const ASSOCIATED_CONST = null;

        /**
         * Test comment on `a` property.
         */
        public $a;

        /**
         * Constructor method. Test!
         */
        public function __construct(int $a, int $b) {}

        /**
         * test_method does nothing.
         *
         * @param int $a input.
         *
         * @returns another int.
         */
        public function testMethod(int $a): int {}

        /**
         * Returns the two values in a string.
         */
        public function printValues(): string {}
    }
}


text:
Files in the format "Name (YYYY-) 001.cbz" does not find the comic when searching

From xylamic on May 01, 2013 04:25:40
What steps will reproduce the problem? 1. Have a comic with file name, for example "Action Comics (2011-) 001.cbz"
2. Search for tag information What is the expected output? What do you see instead? No results found. What version of the product are you using? On what operating system? Latest, Win7 x64 Please provide any additional information below.
Original issue: http://code.google.com/p/comictagger/issues/detail?id=15

From bevi...@gmail.com on May 01, 2013 11:53:24
This isn't strictly a defect/bug, as this format doesn't follow the filename structure recommended here: https://code.google.com/p/comictagger/wiki/UserGuide#Filename_Parsing Do you come across this format "in the wild" much, or is this your personal preferred filename format?
At any rate, this should be easy enough to handle.  The parser just needs to strip out parenthetical phrases before looking for the the issue number.  (I actually thought it was doing that already.)
I'll try to get this in the next release.
Status: Accepted

From xylamic on May 01, 2013 11:58:01
I use this format quite a bit because it matches the format used by Comixology for identifying the 'volume,' such as "Wonder Woman (2011-)"

From bevi...@gmail.com on May 08, 2013 20:19:33
I've added handling of this to 1.1.9-beta.  The year will also be dumped into the volume number.

From bevi...@gmail.com on May 08, 2013 20:21:36
Status: Fixed

repo: davide-romanini/comictagger


text:
External Domain Clarification

Hi, sorry if my terminology is wrong, I am trying to iframe a website that is hosted on another server into a page on my server, and your code was given as a link that could do this?
Q. Does your code do this, or do I need access to both sites / files etc?
I cannot access the other websites files, it is a shutterfly website, so my code to call the page is this:

...and it needs to be embedded on our website in the appropriate area? Is this possible with your code? This has left me blank so far .
If it is possible to post a working example that would be great, as I have tried all examples and only get a height of approx 100px?
Many thanks in advance.

We have the same issue, however, we have access to the page, and can add script information to the HTML of the page. We do not have file access on the OtherDomain. Can this be done? Already tried  copying entire "iframeSizer.contentWindow.min.js" into the page between

Just add a script link to the iframe that loads the contentWindow script from your domain

authors: davidjbradshaw dazzlingbull xSeekTruthx

repo: davidjbradshaw/iframe-resizer


text:
window.parentIFrame no longer being set?

Hi David, I've been using (and loving!) iframe-resizer for about a year now. Suddenly I've noticed something really weird: it looks like window.parentIFrame isn't being set on the parent window at all – not even on your example page. Only noticed this because an if('parentIFrame' in window) type check has begun failing, without there being any code change on my side. This seems to be happening from anywhere I test, so I don't think it's a local issue either. Am I crazy?

Strange, just tried the example page in Chrome 53 and 54 without problem. Do you just see the issue in one browser?

I'm seeing this issue both across browsers and machines. So far I've tested locally on two different Macs using Chrome 54 and Firefox 49, and via BrowserStack using different versions of Chrome, IE and Firefox on both a Mac and a PC, but to no avail: every time I give the console window.parentIFrame, I get undefined. Meanwhile, window.iFrameResize initialises correctly.  I noticed the issue because a client told me a behaviour was no longer triggering, so they seem to be seeing the same issue.

I'm experiencing the same issue

My bad, please ignore my previous comment. I was calling window.parentIFrame before it was loaded. It works as expected when calling from inside readyCallback.

@davidjbradshaw I'm sorry to pester you with this, but as a sanity check, could you confirm that only window.iFrameResize initializes at https://utopia.de/nachhaltige-stellenangebote/, while window.parentIFrame doesn't?

I'm not @davidjbradshaw but I'm following this thread as I've had a similar issue.  I can confirm that window.parentIFrame returns as undefined in the console on the link you posted.

@shaynasymons thank you for checking that! Given that there could be any other kind of mistake on my side in the environment I linked to, do you see the same behaviour on David's example page?

@ggsp Yes, it returns as undefined there, as well.  It also returns as undefined in my project on every page unless I call it in the readyCallback function.

You need to test it in the iFrame, not on the parent page.

Also what I noticed is that you need to wait for the iframe to fully load and resized before window.parentIFrame is available. So, either you need to call it from inside readyCallback or for testing purposes you could call it after a timeout to make sure the iframe has loaded.

Had same problem with missing window.parentIFrame.
Found out, that I needed to set enablePublicMethods: true for iFrameResize in the parent window like this:.
iFrameResize({
log : true,
checkOrigin: false,
enablePublicMethods : true,
messageCallback : function(messageData){
if('Reload' === messageData.message){
location.href = location.href;
}
}
});

Since v3 there is no need to set that option.
You also don't need the messageCallback.

Why does "window.parentIFrame" stop becoming available after navigation within the iFrame itself? After a user clicks on a HREF to change page within the iFrame, "window.parentIFrame" disappears and becomes undefined?

You have to have the iframe page is file in every page you load into the iframe

You're right. Rookie mistake. The .js wasn't being loaded on requests properly!

authors: JohnBergAndersen davidjbradshaw ggsp harryfear jtlocsei shaynasymons


text:
D3 Merge - David Knight key feature 5

Added key feature 5 for part one of D3 - Analysis

First commit - added key feature 5
second commit - added bold and italicized classes, attributes


text:
Add parameters to error messages if provided

Closes #344

Looks great to me! Some minor spacing issues but I'll resolve that later.

authors: Psykar davidkpiano

repo: davidkpiano/react-redux-form


text:
Assigner action to spawn optional actor fails to compile on Typescript

Description
I have an action that spawns an actor and assigns it to the context of the machine. If the actor is a required field of the context that works great, but fails to be compiled by the Typescript compiler when the field is optional.
Expected Result
I expect that the following code compiles by the Typescript compiler. Otherwise, is there a different way to achieve the same outcome that compiles?
Actual Result
In the sandbox I get an error about a missing property. Interestingly enough, in my local project, the very same code fails because 'undefined' is not assignable to type 'SpawnedActorRef.
The error from the code sandbox:
Type 'AssignAction<{ ref: SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }; }, ToggleEvent>' is not assignable to type 'ActionObject<ToggleContext, ToggleEvent> | ActionFunction<ToggleContext, ToggleEvent>'.
  Type 'AssignAction<{ ref: SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }; }, ToggleEvent>' is not assignable to type 'ActionObject<ToggleContext, ToggleEvent>'.
    Types of property 'exec' are incompatible.
      Type 'ActionFunction<{ ref: SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }; }, ToggleEvent>' is not assignable to type 'ActionFunction<ToggleContext, ToggleEvent>'.
        Property 'count' is missing in type '{ ref: SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }; }' but required in type 'ToggleContext'.ts(2322)

The error in my local project:
error TS2322: Type 'AssignAction<{ ref: SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }; }, ToggleEvent>' is not assignable to type 'ActionObject<ToggleContext, ToggleEvent> | ActionFunction<ToggleContext, ToggleEvent>'.
  Type 'AssignAction<{ ref: SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }; }, ToggleEvent>' is not assignable to type 'ActionObject<ToggleContext, ToggleEvent>'.
    Types of property 'exec' are incompatible.
      Type 'ActionFunction<{ ref: SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }; }, ToggleEvent> | undefined' is not assignable to type 'ActionFunction<ToggleContext, ToggleEvent> | undefined'.
        Type 'ActionFunction<{ ref: SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }; }, ToggleEvent>' is not assignable to type 'ActionFunction<ToggleContext, ToggleEvent>'.
          Type 'ToggleContext' is not assignable to type '{ ref: SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }; }'.
            Types of property 'ref' are incompatible.
              Type '(SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }) | undefined' is not assignable to type 'SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }'.
                Type 'undefined' is not assignable to type 'SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>> & { ...; }'.
                  Type 'undefined' is not assignable to type 'SpawnedActorRef<OtherEvent, State<OtherContext, OtherEvent, any, { value: any; context: OtherContext; }>>'.

90       spawnActor: assign({

I copied the code from the sandbox into my local project without any changes.
Reproduction
https://codesandbox.io/s/boring-pine-yhnoe?file=/src/index.ts
Additional context
My local project uses the following stack:
xstate: 4.16.2
typescript: 4.2.3
In my local project I make use of @xstate/react as well but I tried the code from the sandbox without any React specific additions. Therefore I doubt that this has any influence on the outcome.
Thanks a lot for your help!

Try explicitly providing the context type as a generic type:
    actions: {
-     spawnActor: assign({
+     spawnActor: assign<ToggleContext>({
        ref: () => spawn(otherMachine, "other")
      })
    }

Great. This worked for me. Thanks a lot.

Awesome, I'm going to close this issue because this is also solved with the new createModel() work, and we're already tracking making the assign() types infer better (cc. @Andarist)

authors: critocrito davidkpiano

repo: davidkpiano/xstate


text:
test: use "should" instead of "ensure" as prefix

And inline function only used in 1 helper

#290  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite


text:
Issue when execute command "flutter pub pub run build_runner build" in flutter 1.0

Hi,
I have an issue when execute the build_runner command after I update to the flutter last version in stable channel.
The error displayed is the following
file:///C:/Users/jorjuela/AppData/Roaming/Pub/Cache/hosted/pub.dartlang.org/dart_style-1.2.2/lib/src/source_visitor.dart:1837:19: Error: Type 'SetLiteral' not found.
visitSetLiteral(SetLiteral node) {
^^^^^^^^^^
file:///C:/Users/jorjuela/AppData/Roaming/Pub/Cache/hosted/pub.dartlang.org/dart_style-1.2.2/lib/src/dart_formatter.dart:117:12: Error: The setter 'enableSetLiterals' isn't defined for the class 'analyzer.parser::Parser'.
Try correcting the name to the name of an existing setter, or defining a setter or field named 'enableSetLiterals'.
parser.enableSetLiterals = true;
^^^^^^^^^^^^^^^^^
file:///C:/Users/jorjuela/AppData/Roaming/Pub/Cache/hosted/pub.dartlang.org/dart_style-1.2.2/lib/src/source_visitor.dart:1837:19: Error: 'SetLiteral' isn't a type.
visitSetLiteral(SetLiteral node) {
^^^^^^^^^^
file:///C:/Users/jorjuela/AppData/Roaming/Pub/Cache/hosted/pub.dartlang.org/dart_style-1.2.2/lib/src/source_visitor.dart:1839:20: Error: The getter 'leftBracket' isn't defined for the class 'invalid-type'.
Try correcting the name to the name of an existing getter, or defining a getter or field named 'leftBracket'.
node, node.leftBracket, node.elements, node.rightBracket);
^^^^^^^^^^^
file:///C:/Users/jorjuela/AppData/Roaming/Pub/Cache/hosted/pub.dartlang.org/dart_style-1.2.2/lib/src/source_visitor.dart:1839:38: Error: The getter 'elements' isn't defined for the class 'invalid-type'.
Try correcting the name to the name of an existing getter, or defining a getter or field named 'elements'.
node, node.leftBracket, node.elements, node.rightBracket);
^^^^^^^^
file:///C:/Users/jorjuela/AppData/Roaming/Pub/Cache/hosted/pub.dartlang.org/dart_style-1.2.2/lib/src/source_visitor.dart:1839:53: Error: The getter 'rightBracket' isn't defined for the class 'invalid-type'.
Try correcting the name to the name of an existing getter, or defining a getter or field named 'rightBracket'.
node, node.leftBracket, node.elements, node.rightBracket);
^^^^^^^^^^^^
file:///C:/Users/jorjuela/AppData/Roaming/Pub/Cache/hosted/pub.dartlang.org/dart_style-1.2.2/lib/src/source_visitor.dart:1839:9: Error: The argument type 'invalid-type' can't be assigned to the parameter type '#lib1::TypedLiteral'.
Try changing the type of the parameter, or casting the argument to '#lib1::TypedLiteral'.
node, node.leftBracket, node.elements, node.rightBracket);
I saw that build, build_runner, and built_value_generator were updated, but I´m not sure if those libraries need to be updated on built_redux  to fix this issue.
Thank you in advance

This is nearly the same as #104 .
The analyzer has the wrong version. Is '>=0.33.0 <1.0.0' as overwrite for now.
There is the PR #105 which should resolve the issue. But @davidmarne is a bit quiet the last month.

Thank you,
I just fixed this by removing this:
dependency_overrides:
analyzer: ^0.33.0

authors: Findulas jkyon


text:
Character Set

QR Codes have 4 available character sets.  To make smaller it should be checked if there are:

Only numbers
Alphanumeric: 0–9, A–Z (upper-case only), space, $, %, *, +, -, ., /, :

before falling back to binary option.
https://en.wikipedia.org/wiki/QR_code#Storage

Yes!  Absolutely!
Other libraries do this.
Here's one:
https://github.com/nayuki/QR-Code-generator
demo: https://www.nayuki.io/page/qr-code-generator-library
code: https://www.nayuki.io/res/qr-code-generator-library/qrcodegen.js
Here's another:
https://github.com/kazuhikoarase/qrcode-generator
demo:https://kazuhikoarase.github.io/qrcode-generator/js/demo/
code: https://github.com/kazuhikoarase/qrcode-generator/blob/master/js/qrcode.js

authors: mctrivia zamicol

repo: davidshimjs/qrcodejs


text:
More Weather Information Needed

Thanks so much for the wonderful breakdown of each day. I really appreciate this.
I would like to see the following for each locations we use: humidity, air quality, pollen data, and most of all NWS weather alerts. Any chance of you including this information in future versions? Thanks

Hey @HonestAbe7 , I just posted a screenshot in issue #32 of the upcoming redesign that added a bunch of new data. I'd like your opinion on it, too :)
I plan to add air quality and weather alerts sometime later, too.

OK. Great! I commented over on that thread. 😁

I really appreciate your willingness to improve on this app. From my experience, most developers don't care what what the user wants. I have posted many suggestions on other apps that were never considered. So this is a plus that you are reaching out to see what the user wants. I can tell this could potentially be one of the best open source weather apps ever. Thanks and I appreciate all your efforts! 😁

Thank you for the kind words, it means a lot. And thank you for your feedback and suggestions. Without users like you, I would be poking around in the dark!

repo: davidtakac/prognoza


text:
FEATURE REQUEST: make it optional to interact with the mouse outside the focus-trap

We want to keep de keyboard tabs within the focus-trap but when a notification pops up the user should be able to interact with the notification with the mouse.
We don't have the need to have the clicks blocked outside the focus-trap. Would be nice if this is optional

Hi @PieterSwitten. This has already been discussed a number of times, and there have been a few abandoned PRs. Please search the issues and PRs to find the backstory, then open a new issue if you have a new idea or a PR if you're able to implement something. Thanks.

I have a dropdown component, which, when opened, will attach a "click away" listener to the document to close the dropdown when the user clicks outsides of the dropdown.  The FocusTrap seems to be blocking this click event from happening.  How can I implement this behavior without an option to disable the FocusTrap from interfering with the click events?

I just did some more digging and found the clickOutsideDeactivates option, that works, nevermind!
<FocusTrap
  active={isOpen}
  focusTrapOptions={{ clickOutsideDeactivates: true }}
>

repo: davidtheclark/focus-trap


text:
Zwifit Frontend and Zwifit-incline-tracker incline control not working

Describe the bug
Zwifit is unable to control treadmill incline until a random time (20+ minutes) after the treadmill is started.  I'm trying to work out how to switch my (Proform Pro 1000) into mode 2 (control mode) where the treadmill can be controlled by Zwifit.
When I say mode 2, I've seen it enter this mode via Zwifit debug code I've added when I was lucky enough to enter this mode. This allows the treadmill to be controlled, so I'm calling it control mode (a bad name for it I know).
This functionality is required so that the Zwifit Frontend and Zwifit-incliner can control the incline. The treadmill eventually switches to this mode, making a series of tones.  Does Zwifit instruct the treadmill to enter mode 2 (control mode), or does the treadmill tell Zwifit that it can now be controlled?
To Reproduce
Mode 0: Initial state when searching for the treadmill
Mode 1: Treadmill power turned off
Mode 8: Treadmill power turned on. Key not plugged it. Display off. ifit light on.
Mode 1: Treadmill key inserted after being in state 8
Mode 10: Treadmill started
Mode 2: Expect control mode to be entered here.
Mode 3: Treadmill stopped
Mode 8: Key removed
Mode 2 (control mode) when the treadmill can be controlled by Zwifit, seems to occur randomly after 20 minutes or so. So randomly that I'm having problems going into this mode as I type this up. Until this mode is in operation, changes in speed, Incline, etc made on the treadmill are seen by Zwifit.
Expected behavior
When Zwifit makes a connection with the treadmill, entering mode 10, it should then switch to allow control of the treadmill so that both monitoring and control are possible.
Screenshots
N/A
Desktop (please complete the following information):
Raspbian, Zwifit, Treadmill
Additional context
None

To clarify, during testing, the Raspberry Pi 3 is sat on the Proform Pro 1000 dashboard, directly under the treadmill display.
With Zwifit already running, as soon as the Treadmill is powered up, it connects that instant.
Also, I have entered this control mode at least 5 times and have been able to control the incline etc.

I've run two test and have some promising results.
Mode 2 has the ability to control the treadmill.
Entering mode 2 doesn't appears to be random as first thought. It takes precisely 5 minutes after pressing the start button (resulting in the belt moving). So 5 minutes after the belt starts moving.
Is this something hardcoded into Zwifit, or is this the treadmill getting the user to do a 5 minute warmup?
Test 1:

20:30:2 - Mode:  8 - Treadmill turned on - ifit light on
20:30:18 - Mode:  1 - Treadmill key inserted after being in state 8
21:6:15 - Mode:  10 - Treadmill started - Belt moving

5 minutes exactly

21:11:15 - Mode:  2 - Control mode - Belt moving
21:11:51 - Mode:  3 - Treadmill stopped - Belt stopped - Results shown
21:12:3 - Mode:  4 - Stop button pressed again - done shown on display

2 minutes exactly

21:14:3 - Mode:  1 - switched back to this automatically.

Test

2:21:20:48 - Mode:  8
21:21:23 - Mode:  1
21:34:27 - Mode:  10

5 minutes exactly

21:39:28 - Mode:  2

I hope this helps.

repo: dawsontoth/zwifit


text:
Exasol: incorrect data is displayed at the beginning when creating a new column

System information:

DBeaver version 7.3.0 EA

Connection specification:

Database name and version Exasol, DB2

Steps to reproduce, if exist:
Create a column Integer for a table
Expected: Column with correct type and length appears in the column list
Actual result: the column has length 50 at first and changes it on 36 only after saving

also DB2 problem

also DB2 problem

https://www.ibm.com/docs/en/db2-for-zos/11?topic=columns-string-data-types DB2 types which support modifiers after data type name.

Fixed for Exasol and DB2. Now length/precision of column are changing in case of column type (from text to numeric, from date to binary etc.) modification.

verified

authors: LonwoLonwo kseniiaguzeeva uslss

repo: dbeaver/dbeaver


text:
Dbeaver won't open anymore

Hello,
I was using Dbeaver for a couple weeks now without any problems. Yesterday I shutdown my laptop and this morning tried to launch Dbeaver as usual and it's not working anymore.
It says that an error has occurs and show the path to a log file.
What's in the log file is kinda mysterious (and it's always the same) :
!SESSION !SESSION !SESSION !SESSION !SESSION !SESSION !SESSION !SESSION !SESSION
That's it ! I tried to reboot my computer but still, not working....
System information:

W10
DBeaver version : 21.0.1
Java version : 1.8

Can anyone help me ? Stuck here...
Best regards.

Hi @iSpeeX ,
can you try to reset workspace?

authors: iSpeeX uslss


text:
DATE comparison not working correctly

System information:

MacOs 12.2.1
22.1.0
Additional extensions

Connection specification:

MySQL Ver 5.7.35 for Linux on x86_64 (MySQL Community Server (GPL))
MySQL
Do you use tunnels or proxies (SSH, SOCKS, etc)?  No

schedule_date is a DATE field but it does not filter correctly.  It thinks 2022-04-25 is 22-04-24

@PaulDokken unfortunately, cannot reproduce it.
Could you provide table DDL?

There has been no update for a long time.
If issue is still actual for you - feel free to reopen it

authors: PaulDokken uslss


text:
Firebird updatable views appear as tables

Description
Firebird 3 Views that are accompanied by triggers that allow Insert, update and delete (I use this for simplifiying processes that write to a fully normalized database) are displayed as tables and the wring ddl is shown.
DBeaver Version
22.1.3
Operating System
Windows 11 22H1
Database and driver
Firebird 3 latest version
jdbc:firebirdsql
Steps to reproduce
create a table with several fields
create a view that selects from that table
create triggers for that view that server Insert, Update and Delete
Open this database in DBeaver and display the updatable view
Additional context
No response

Hello @michaschumann
Please provide DDL examples.

It seems it was fixed in #10492
If the issue is still actual for you against the recent version - feel free to reopen the issue with additional info to reproduce it

authors: LonwoLonwo michaschumann uslss


text:
Issue with connection to the Postgres on the docker container from master host

Description
Suddently I can't connect to the PG from the DBeaver. Couple of days ago everything worked fine.
In the same time PgAdmin and SQL Manager for PostgreSQL connect with no problem.
DBeaver Version
Community Edition Version 24.0.3.202404211624
Operating System
Windows 11
Database and driver
PostgreSQL 13.x on the docker container
postgresql-42.7.3.jar

Steps to reproduce
No response
Additional context
No response

Have you updated DBeaver before it stoped working?

I think so. But I have tried to rollback to previous versions. They don't work either.

I've found out the workaround. I've set the IP of the Ethernet adapter vEthernet (WSL (Hyper-V firewall))

After that I've successfully connected. However, it's very strange.

Looks like a third-party issue depending on your virtual Hyper-V settings.

authors: E1izabeth iuaacvs serjiokov


text:
[PostgreSQL] Adding comments to constraints

DBeaver lacks the ability to add comments to table constraints (FK's, PK's and check constraints). It also doesn't detect and show existing comments of constraints. It would be great if DBeaver supported this feature. I think this should consist of:

When using Edit Constraint: new field for comment/description in the properties tab.
When using Create New Constraint: new field for comment/description.
Creation of comment DDL's like this: COMMENT ON CONSTRAINT constr_name ON public.table_name IS 'constraint comment';
Displaying constraint comments as mouse-over tool tips in the Navigator pane (just like for table columns).

Constraints and foreign key comments are now read from DB. Comment edit is also fully supported.
All constraint comments are also included in table DDL.

authors: lemon-juice serge-rider


text:
Using Space On Template

Hi, Templates working great but can't we use space or tab together. Using space bar would be awesome. And using special chars like s* would be great.
Thanks

Thanks for the feature request.
I'm closing it as a duplicate of #11553


text:
IO Error: Utility 'mysqldump.exe' not found in client home 'MySQL Connector/Net' DUPLICATE OF #2765

I've seen this closed at least twice without really addressing the issue.  Dbeaver not only does not include (or download) all the prerequisite files/paths that MUST be set for it's methods/functions to successfully complete, but does not even properly document them (I just searched your website as well), or even as a pure reactive after-thought even clarify to it's users how to mitigate the above TERMINATING error to be able to continue using the product.  Instead, luckily I found community members that had at least acknowledged this was an issue, albeit without any real resolution.
As was stated by community members in #2765, the only solution (?) is to go out and instead of a few binaries, do a full download and installation of MariaDB server because (as also stated above) if one would (wrongfully, in the case of dbeaver) assume that this app knows well enough that the needed binaries ALSO do live in the (much more appropriate for a client workstation) MySQL workbench - dbeaver will not properly even detect the required files are there.
This should be an open issue until addressed.  At the very least, dbeaver should either be 1. automatically downloading required files (which I was impressed in my case it DID do for the jdbc maria dll) so why not these other required files?.....or 2. Prompt the operator for a location where they might already be installed (before totally blowing up and terminating leaving the operator to have to open a different db management client), or at the absolute very least  3. Terminate and tell the operator exactly WHAT is missing in detail and most importantly WHERE TO COPY/INSTALL THEM to continue and also update the product website/kb/wiki to clearly state the product requirements specific to individual binary files needed for this product to operate on specific databases - since THAT is what this product's main purpose is, after all.

Agreed. This feature is very poorly documented. Unfortunately we have too many features and too few hands. Hopefully this situation will change in the near future (or we'll be buried under tons of feature requests).
Automatic native binaries download will be added in #3778. It has high priority so it has quite good chances.

authors: CollinChaffin serge-rider


text:
Create New Oracle Table / CSV Import

I have seen many questions and responses about importing data in a CSV into a database. The general protocol is to make a JDBC connection to CSV file, then export the CSV data into a database table with the Export Wizard. However, all the descriptions of this process assume that a table already exists in the database, but I'm always interested in creating a new table when I import a CSV. I'm unclear about whether the Export Wizard is capable of creating a table (as with eg SQL Developer), or whether I have to have a pre-existing table. If I have to have a pre-existing table, must it have the correct dimensions and data-type specifications, or can the Export Wizard create columns according to the structure of the CSV. If so, could somebody please outline the steps to import a CSV, assuming no table yet exists, describing in detail the process of creating a table that will permit a successful CSV import? If possible, consider assuming the CSV file is very wide, so that manually specifying column names and data types would be silly. The bottom line is that an Import Wizard like the one available in SQL Developer would be very nice in DBeaver, if it does not already exist. Presently, I use SQL Developer solely to import data (it works very well), and would like to fully transition to using DBeaver.

Perhaps this can be helpful: https://github.com/dbeaver/dbeaver/wiki/Data-transfer

Also, if you select database table in the export wizard from a csv source, choosing a nonexistent table should create it; however, it doesn't look it is possible in the import wizard.

I've discovered a process that works for my purposes.

Make new CSV connection.
Right-click desired CSV file and choose Export Data.
Choose database.
Choose desired Target Container (Database and Schema).
Select New (may need to deselect Target field) and name new table.
Select Columns to specify column names and change data types from VARCHAR(4000) default.
Click through to Finish.

This request is currently called "Create New Oracle Table / CSV Import", but I think it should be "Create New Table using CSV Import" since it applies to any database technology.
The solution from tfarkas86 is good. It's what I use as well. But it's has important limitations.

It's MUCH faster than CSV import. Roughly 10x faster for me. OK, this doesn't sound like a limitation. ;-)  ... but see the next point.
It does not have the option to treat empty strings as NULL values.

So I either get a lot of empty strings that I wish were NULL values... or I need to take 2 minutes instead of 12 seconds to load my table. Clearly this is not blocking. But it would be nice to get the best of everything.

I'm curious about why this got closed. It still seems like a useful feature.
@uslss, what was the motivation for closing?


text:
Improve Advanced Copy / result set

This is a feature request.
Add to Advanced Copy menu for Result set the items for :

Copy as HTML
Copy Aligned
Copy as SQL IN
Copy as SQL WHERE
Copy as SQL UPDATE
Copy as SQL INSERT

This resources help a lot on SquirrelSQL and is missing on Dbeaver.
Just like SquirrelSQL :

You already have most of them:

Copy as HTML: either Export Data... then HTML, or Open with -> web browser
Copy aligned: either Export Data... then TXT, or just copy-paste from text view
Copy as SQL IN: see screenshot below for the options I use. But this would be a nice shortcut, I suggest it to be put in the Generate SQL submenu
Copy as SQL WHERE/UPDATE/INSERT: see the Generate SQL submenu

Hi @p91paul ,
Yes, there are similar resources but isn't the same as SquirrelSQL.
The Dbeaver option is very very very less efficient.
Let put my point of view :

Dbeaver export option has a lot of steps to execute, make it painful for frequent use.
Dbeaver always reruns the SQL Statement, this is terrible, instead of copy from the result set which already cached. For slow queries this is painful.

About your solutions :

Export option is very annoying for who need these options frequently because is a lot of steps to execute it.
I see theses options I put as "shortcut" for more useful features.
Export data as HTML do not work like Squirrel.
At least for my objective where I use a lot HTML format to paste into Outlook emails and if you select copy to clipboard they (dbeaver) copy the HTML code, not as HTML content.
Copy Align from view text isn't the same because they copy with "|" chars and isn't that what I'm looking for, the idea is align text only with the values.
About the Generate SQL submenu, please, where is it?
I'm looking for it here and do not found it.
I tried on "quick access" box and neither there they appear. (dbeaver v6.0.1)

For generate sql, right click on the resultset after selecting the rows you want to export. It's one of the last options in the context menu!
For the others, I agree with you that a faster way to achieve those goals would be most welcome. My response was just to make you discover those alternatives in case you weren't aware of them.
I suggest the implementation to be based on the excel export, which features a button in the bottom bar which triggers a one-click export of the resultset (or of current rows/cols). That button should become a menu, and feature most data export commands (txt, csv, sql, excel, html, etc). Clicking on one of them should trigger export with default options of selected rows/cols (or entire resultset if nothing selected).
This is similar to how datagrip export works.
A more advanced version of that would allow to make the list of options in that menu configurable, allowing the user to save export configurations and reuse them with one click.

About the Generate SQL submenu, aren't you using some plugin?
This is my context menu for the result set :

A more advanced version of that would allow to make the list of options in that menu configurable, allowing the user to save export configurations and reuse them with one click.

That is a feature which I also think about :+1:

I suggest the implementation to be based on the excel export, ...

This one is a plugin which I do not test yet, I will test it now.

@ceinmart It is in data editor context menu, not SQL editor.
I think we can add "Copy As" function which will utilize data exporters but in simplified mode (export to clipboard using default options).
Mark as FR.

@ceinmart It is in data editor context menu, not SQL editor.

Hi @serge-rider,
Well, I think I was clicked there.... on my print screen I was right-cliked over the first row/data/field of column "SER" value 28.572 .
I also recheck if I understand right what do you means with "data editor" and looking the wiki appears I'm clicking correctly.
I also right-click everything else to check if some "Generate SQL" appears and found nothing.
My context menu do not show the "Generate SQL" like the this wiki

@ceinmart Generate SQL is only available if you are working with some particular table's data but not in custom SQL queries.

@serge-rider , thank you for this enlightenment!
I've found the "Generate SQL" option in the context menu at the "data" tab in the table properties.
Also found this #2484 issue to improve this.
I will add my comments there.
Thank you!

Reopened
Null pointer exception when trying to copy as xlsx

verified

authors: ceinmart p91paul serge-rider uslss


text:
MSSQL - DB connection is closed when canceling a query, and no results are displayed

System information:

Operating system (distribution) and version : Windows10 Pro 1903
DBeaver version : 6.3.5
Additional extensions

Connection specification:

Database name and version : Microsoft SQL Server 2016 (SP2-GDR) (KB4505220) - 13.0.5101.9 (X64)
Driver name : mssql-jdbc:RELEASE 7.4.1.jre8
Do you use tunnels or proxies (SSH, SOCKS, etc)?  NO

Describe the problem you're observing:
Connect to MSSQL and cancel while executing the query, the connection with database will be disconnected.
In version 6.1.5, the results obtained just before the cancellation were displayed and the connection to the DB was maintained.
Also, when connecting to DB2, this problem did not occur even with version 6.3.5.
Steps to reproduce, if exist:
See description above?
Include any warning/errors/backtraces from the logs

There is no error log in MSSQL. Exists for DB2.
The following is a runtime error.
org.jkiss.dbeaver.model.exec.DBCException: SQL Error: The connection is closed.
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.nextResults(JDBCStatementImpl.java:208)
at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.executeStatement(SQLQueryJob.java:536)
at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.lambda$0(SQLQueryJob.java:424)
at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:164)
at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.executeSingleQuery(SQLQueryJob.java:416)
at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.extractData(SQLQueryJob.java:774)
at org.jkiss.dbeaver.ui.editors.sql.SQLEditor$QueryResultsContainer.readData(SQLEditor.java:2914)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.lambda$0(ResultSetJobDataRead.java:110)
at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:164)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.run(ResultSetJobDataRead.java:108)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetViewer$17.run(ResultSetViewer.java:3421)
at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:103)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
Caused by: com.microsoft.sqlserver.jdbc.SQLServerException: The connection is closed.
at com.microsoft.sqlserver.jdbc.SQLServerException.makeFromDriverError(SQLServerException.java:234)
at com.microsoft.sqlserver.jdbc.SQLServerConnection.checkClosed(SQLServerConnection.java:1090)
at com.microsoft.sqlserver.jdbc.SQLServerStatement.checkClosed(SQLServerStatement.java:1058)
at com.microsoft.sqlserver.jdbc.SQLServerStatement.getMoreResults(SQLServerStatement.java:1339)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.nextResults(JDBCStatementImpl.java:206)
... 12 more

hi @eksla ,
unfortunately, cannot reproduce. could you post e.g. a video that could help to investigate an issue?

@eksla Also please post example of  query which cannot be canceled.

there is no update on issue almost a month.
if issue is still reproducible for you - feel free to reopen the ticket

https://user-images.githubusercontent.com/8275175/120313893-0099a000-c2b1-11eb-9c65-c4e3b6fabd8f.mp4
I'm facing the same problem when running certain queries  with MSSQL connection, not sure why. Here's a video showing the problem.
I have almost exactly the same settings from the OP but I'm running SQL Server 2008.

authors: eksla mateusviccari serge-rider uslss


text:
Incorrectly quoting values of a UUID array

System information:

Operating system (distribution) and version: Windows 10 Pro - 10.0.18363 Build 18363
DBeaver version: 7.0.5
Additional extensions: None

Connection specification:

Database name and version: PostgresSQL 12.2
Driver name: PostgreSQL
Do you use tunnels or proxies (SSH, SOCKS, etc)? Nope

Describe the problem you're observing:
When copying an array of uuids from a column, the uuids are quoted which then pasted into another column will fail.
Steps to reproduce, if exist:
Create a table with an array of uuids
CREATE TABLE test (
	id int NOT NULL,
	"uuidArray" _uuid NULL,
	CONSTRAINT test_pkey PRIMARY KEY (id)
);

INSERT INTO public.test (id, "uuidArray") VALUES (1, '{62bd91ce-10c8-4a21-b818-06f70cafd2c6, f2ce6ff8-4202-4eb1-9439-e887a39e87cc}')

Now when you view the table, you can see the values are quoted, when they shouldn't be.

If you then copy the value of the column you get:
{'62bd91ce-10c8-4a21-b818-06f70cafd2c6','f2ce6ff8-4202-4eb1-9439-e887a39e87cc'}

Instead of:
{62bd91ce-10c8-4a21-b818-06f70cafd2c6,f2ce6ff8-4202-4eb1-9439-e887a39e87cc}

I also tried advanced copy and it did nothing.

thanks for the bug report and sorry for the late response

No problem! Thank you for your response, look forward to seeing this fixed!

verified

authors: kierxn uslss


text:
NPE falls in "Database -> Transaction mode" in special cases

When draws a checkbox for smart commit mode
For objects like User, Procedure, Session manager etc.
npesmartcommit.txt

verified

authors: LonwoLonwo uslss


text:
#22168 Add OVER clause and some sggregate functions handling

ONE PR FOR TWO ISSUES #22336 #22168
Dear QA team, please check the cases from the issues.

verified


text:
Uncaught Error: ENOENT: no such file or directory, scandir '~/AtomBackups'

[Enter steps to reproduce:]

...
...

Atom: 1.15.0 x64
Electron: 1.3.13
OS: Unknown Windows version
Thrown From: config-import-export package 0.3.1
Stack Trace
Uncaught Error: ENOENT: no such file or directory, scandir 'C:\Users\FabioGomes\AtomBackups'
At fs.js:951

Error: ENOENT: no such file or directory, scandir 'C:\Users\FabioGomes\AtomBackups'
    at Error (native)
    at Object.fs.readdirSync (fs.js:951:18)
    at Object.fs.readdirSync (ELECTRON_ASAR.js:571:28)
    at ImportView.module.exports.ImportView.attach (/packages/config-import-export/lib/config-import-view.coffee:40:16)
    at /packages/config-import-export/lib/config-import-view.coffee:19:42)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/app.asar/src/command-registry.js:259:29)
    at CommandRegistry.handleCommandEvent (/app.asar/src/command-registry.js:3:59)
    at CommandRegistry.module.exports.CommandRegistry.dispatch (/app.asar/src/command-registry.js:160:19)
    at AtomEnvironment.module.exports.AtomEnvironment.dispatchApplicationMenuCommand (/app.asar/src/atom-environment.js:1162:28)
    at EventEmitter.outerCallback (/app.asar/src/application-delegate.js:312:25)
    at emitThree (events.js:116:13)
    at EventEmitter.emit (events.js:194:7)

Commands
  2x -6:50.9.0 window:toggle-menu-bar (atom-workspace.workspace.scrollbars-visible-always.theme-one-dark-syntax.theme-one-dark-ui)
     -6:33.6.0 window:toggle-dev-tools (atom-workspace.workspace.scrollbars-visible-always.theme-one-dark-syntax.theme-one-dark-ui)
  2x -6:08.8.0 application:add-project-folder (atom-pane.pane.active)
     -4:53 core:confirm (input.hidden-input)
     -3:42.3.0 core:backspace (input.hidden-input)
     -3:41.1.0 core:confirm (input.hidden-input)
  2x -2:55.8.0 core:backspace (input.hidden-input)
     -2:54.4.0 core:confirm (input.hidden-input)
     -2:26.2.0 core:delete (input.hidden-input)
     -0:13 config-import-export:import (input.hidden-input)

Non-Core Packages
config-import-export 0.3.1

If you look at the settings for this package in your editor, take a look at what the default path for your system, and make sure it is pointing to the directory that contains the JSON config file, or where you want it to be exported to.


text:
Can't add server

Hi! Can you explain please how I can add and connect to this server (Atlas Cluster from Mongo Explorer)?:
mongo "mongodb://cluster0-shard-00-00-jxeqq.mongodb.net:27017,cluster0-shard-00-01-jxeqq.mongodb.net:27017,cluster0-shard-00-02-jxeqq.mongodb.net:27017/aggregations?replicaSet=Cluster0-shard-0" --authenticationDatabase admin --ssl -u m121 -p aggregations --norc

In the server settings, set your cluster in the server url field as follow and tell me if it works or not cluster0-shard-00-00-jxeqq.mongodb.net:27017,cluster0-shard-00-01-jxeqq.mongodb.net:27017,cluster0-shard-00-02-jxeqq.mongodb.net:27017

Solved on stackoverflow
https://stackoverflow.com/questions/50949112/cant-add-serveratlas-cluster-in-mongo-explorer

authors: Andruno dboissier


text:
Add export command that exports schedule to ICS formatted file

If we allow export of the schedule to an ICS formatted file, we can put that file in ~/public_html/pytid.ics and then import with Google Calendar (or similar). We should let the command do this automatically if the ~/public_html folder exists. Otherwise print it to stdout or a named file.

This is done.

repo: dbosk/nytid


text:
use arcgis-rest-request for auth

work in progress, with help from this blog and encouragement from @tomwayson.
q's

[ ] polyfill fetch or bind to Ember's own implementation?
[ ] refactor more internal auth logic to DRY it out?
[ ] add support for locale (after https://github.com/Esri/arcgis-rest-js/pull/169 lands)

okay, thanks to the breadcrumbs in https://github.com/Esri/ember-cli-cedar/pull/90 i now understand how to get a reference to arcgis-rest via a UMD inside an ember app without using a shim.

[X] get a reference to the UMD
[ ] start porting the actual authentication logic.

Importing the arcgis-rest-js libs looks good. I'm curious why you had to add jQuery to the dummy app's index.html tho.

I'm curious why you had to add jQuery to the dummy app's index.html.

that was just to resolve a nag in the console regarding using bootstrap w/o it.

pushed up another commit to address @tomwayson's most excellent feedback.

polyfill fetch or bind to Ember's own implementation?

I think we let the consuming app handle that.

refactor more internal auth logic to DRY it out?

Let's get this into EAPS and revisit that. I think we can maybe have the session service take advantage of deprecating CPs to handle backwards compatibilities for the old API.

add support for locale (after Esri/arcgis-rest-js#169 lands)

Sure

authors: jgravois tomwayson

repo: dbouwman/torii-provider-arcgis


text:
Update Auth0_auth.py

As discussed here: https://github.com/dbrambilla13/dash-auth0-auth/issues/1

Any update on this? Would like to use it in a application i'm building. Thanks


text:
the trait embedded_hal_async::i2c::I2c is not implemented for Generic<embedded_hal_mock::eh1::i2c::Transaction>

Hello, thank you for this crate, it's awesome! I'm having an issue with mocking embedded_hal_async::i2c::I2c. Is there support for this?

I looked deeper into the source and see that it isn't [yet] so I added support for it.

repo: dbrgn/embedded-hal-mock


text:
NullReferenceException on [ShowNativeProperty]

If one of those property chains is null, it throws an error in editor (not playing the game).

Thank you very much mate, I will investigate.

Mate, I can't do anything to fix it. I can't detect a null object inside the property chain.
What you can do to prevent it is to just safeguard the chain.
Instead of FirstProp.SecondProp.ThirdProp you can do FirstProp?.SecondProp?.ThirdProp and that fixes it.

authors: dbrizov pippinmole

repo: dbrizov/NaughtyAttributes


text:
new issue report

Hi, i attach ASAN while compiling this repo, and feed some testcase to the binary by using following command:
 ./wavpack -bn -c -h -v  ./wav/diodes.wav -o ./
here is the sample file:
diodes.zip
and ASAN detect memory leak
 ==77452==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 4098 byte(s) in 1 object(s) allocated from:
    #0 0x4b0010 in malloc /home/bupt/桌面/tools/llvm-12.0.1/llvm/projects/compiler-rt/lib/asan/asan_malloc_linux.cpp:145
    #1 0x4f7769 in main /home/bupt/Desktop/WavPack/cli/wavpack.c:879:27
    #2 0x7f7cfc982c86 in __libc_start_main /build/glibc-CVJwZb/glibc-2.27/csu/../csu/libc-start.c:310

Direct leak of 8 byte(s) in 1 object(s) allocated from:
    #0 0x4b03a8 in realloc /home/bupt/桌面/tools/llvm-12.0.1/llvm/projects/compiler-rt/lib/asan/asan_malloc_linux.cpp:164
    #1 0x4f7d19 in main /home/bupt/Desktop/WavPack/cli/wavpack.c:885:23
    #2 0x7f7cfc982c86 in __libc_start_main /build/glibc-CVJwZb/glibc-2.27/csu/../csu/libc-start.c:310

Indirect leak of 26 byte(s) in 1 object(s) allocated from:
    #0 0x4b0010 in malloc /home/bupt/桌面/tools/llvm-12.0.1/llvm/projects/compiler-rt/lib/asan/asan_malloc_linux.cpp:145
    #1 0x4f7d42 in main /home/bupt/Desktop/WavPack/cli/wavpack.c:886:35
    #2 0x7f7cfc982c86 in __libc_start_main /build/glibc-CVJwZb/glibc-2.27/csu/../csu/libc-start.c:310

SUMMARY: AddressSanitizer: 4132 byte(s) leaked in 3 allocation(s).

is it some kind of new bug?

Hi, and thanks for reporting this!
This is not a new bug and It doesn't have anything to do with the file you are trying. The problem is the parameter -bn is incorrect because n is supposed to be a value (the number of bits or the bitrate):
 Options: -bn = enable hybrid compression, n = 2.0 to 23.9 bits/sample, or
                                           n = 24-9600 kbits/second (kbps)

For normal operations there should be no leaks in WavPack, however I have not made sure that everything allocated during start-up parameter parsing is correctly freed in the case of an error exit (which is what we have here). Some might consider this "bad practice" and I agree to some extent, however I also don't think it makes sense to add a lot of special case code whose only purpose is to free memory immediately before exiting in the case of an error. That code would still need to be maintained and is difficult to test fully and could eventually introduce crashing bugs on error (which are certainly worse than leaking memory before exit).
Keep in mind that all the resources, including memory, used by an exiting process are immediately returned to the OS, so really there is no actual leak (even if this happened in a non-error case) and some have argued that it's a waste of CPU resources to explicitly free memory before exit. I'm not sure I'd go that far, but it is a little like rearranging the deck chairs on the Titanic, if you know the reference.
If someone decided to rename the main() function of the command-line program and use it from some higher-level process then these leaks would be consequential, but the code is not intended for that use.

./wavpack -bn -c -h -v

thanks for your kind and detailed reply.

authors: Cvjark dbry

repo: dbry/WavPack


text:
[CT-695] [Feature] Disable custom tests from dbt_project.yml

Is this your first time opening an issue?

[X] I have read the expectations for open source contributors

Describe the Feature
Missing feature
In our project, we sometimes want to avoid running certain tests depending on the context (target.name) and/or the location of the test (e.g. in an installed package). To do that, we expected dbt_project.yml to help us conditionally disable tests (enbaled = false).
This feature seems to be available for models, but I could not disable tests in the same fashion.
I have unsuccessfully tried the following syntax:
tests:
    dbt_utils:
      +enabled: |
        {%- if  target.name == "prod" -%} true {%- else -%} false {%- endif -%}

This triggered the following error
There are 1 unused configuration paths:
- tests.dbt_utils

Oddly enough, the documentation seems to assert this feature is available, but I haven't been able to use this feature.
I documented my qui research in the following slack thread.
Why do we sometimes need to disable certain tests
In our CI tests, we build and test models that have been changed along with their dependencies everytime a published pull requests receives a new commit. Since we have a lot of models in our project, this might mean we need to rebuild hundreds of models everytime. To prevent expansive builds, we have decided to add a limit 100 to our model builds. This means we do not test our models with a full volumetry, but dbt's syntax and SQL are tested.
Yet some of the tests we want to run out of the CI (in production) require full volumetry, such as dbt_utils.recency. In the CI, we would like to disable tests that will fail without full volumetry.
Describe alternatives you've considered
I tried adding tags to tests when they were assigned to models. Indeed those tags can be excluded from CI workflow commands.
Example:
      tests:
          - dbt_utils.unique_combination_of_columns:
              combination_of_columns:
                - id
                - name
              config:
                tags: ['packages']

Can be excluded with the following command:
dbt test --exclude tag:packages

I also considered disabling dbt data testing from the CI workflow given that tests run tests that are mostly or totally incomplete. The result of those tests are therefore almost always irrelevant to the actual results we will get in production.
Who will this benefit?
I think disabling any node from the dbt_project can be useful for other applications. It seems natural to me that we should be able to manage any node's config from dbt_project.yml.
Are you interested in contributing this feature?
I am not familiar with dbt-core's source code at all, but I would like to review this feature's code for sure!
Anything else?
I tested this both on my professional project and in a dummy project to make sure this feature was not disabled by our config. I hope I am not mistaken!
More generally I feel like it is difficult for dbt users to explore nodes and figure out which resource is linked to which resources-path. Maybe there is a command that could let us output the resource-path of a node in the console?

Hey @LoicChesneau! Thanks for the clear write-up. I think the confusion here might be around the difference between disabling specific test resources, which are often instances of generic tests, versus the generic test definitions (macros) that are (re)used to define each of those resource. This is a subtle distinction, and I'll do my best to explain it below.
Specific instances of generic tests — actual test resources — are defined in specific yaml files. Those files are included in the test resources' fully qualified names, allowing them to be configured from dbt_project.yml. So if I have a dbt_utils.equality test defined in a file named models/subfolder/some_tests.yml:
# models/subfolder/some_tests.yml
version: 2
models:
  - name: some_model
    tests:
      - expression_is_true:
          expression: "1=1"

I can conditionally the tests defined in that file via:
tests:
  my_project:
    subfolder:
      +enabled: "{{ target.name == 'prod' | as_bool }}"

Generic test definitions are templates/functions, which can be reused to define specific tests. They can define default configuration for all tests within their functional definition. For example, I could redefine the equality test from dbt_utils to change its default configuration, while cleverly avoiding the need to copy-paste all its logic (very similar to the discussion in https://github.com/dbt-labs/dbt-core/issues/3841):
{% test expression_is_true(model, expression, column_name=None, condition='1=1') %}
    {{ config(enabled = (target.name == "prod")) }}
    {{ dbt_utils.default__test_expression_is_true(model, expression, column_name=None, condition='1=1') }}
{% endtest %}

Then, all specific instances of expression_is_true tests in my project will be disabled by default. This configuration will take precedence over configuration in dbt_project.yml, but it will be superseded by configuration on each specific test in its .yml file (https://docs.getdbt.com/reference/test-configs):
# models/subfolder/some_tests.yml
version: 2
models:
  - name: some_model
    tests:
      - expression_is_true:
          expression: "1=1"
          config:
            enabled: true

Does that help answer your question? I think the thing you want already exists—it may just be trickier to disentangle than it should be, and we could come up with clearer ways to expose or document the functionality.

Hi Jeremy,
Got it, that's exactly what I was looking for!
I redefined recency as a project test with the following syntax:
{% test recency(model, datepart, field, interval) %}
    {{ config(enabled = (target.name == "prod")) }}
    {{ dbt_utils.default__test_recency(model, field, datepart, interval) }}
{% endtest %}

And used the function in my tests just like this:
  - name: my_tested_model
    description: some description
    tests:
      - recency:
          datepart: day
          field: created_at
          interval: 2
          severity: warn

Under target prod, the test was run, under target dev, it was not.
Thank you! 👏

@LoicChesneau Excellent! I think it would be valuable to add this as an example in the docs. In the meantime, I'm going to close the issue as resolved.

repo: dbt-labs/dbt-core


text:
Enable an errored creation of external tables to not stop following ones

Describe the feature
A clear and concise description of what you want to happen.
Currently if an external table was unable to be created, it stops the following ones from being created. If there was a way to either handle the error or create them in parallel, this would help alleviate the issue.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Running it from dbt Cloud, we considered creating multiple steps in the job but since a failed step stops the following ones, this won't work.
Additional context
Is this feature database-specific? Which database(s) is/are relevant? Please include any other relevant context here.
I think this would be database agnostic. In this case, the issue was found when mounting from GCS to BigQuery.
Who will this benefit?
What kind of use case will this feature be useful for? Please be specific and provide examples, this will help us prioritize properly.
Anyone with multiple external tables being created at the same time.

+1 of importance to this feature, currently if you have multiple external sources and run through them, if 1 fails the rest wont be loaded, a skip error config would be really useful as it currently blocks loading of all other tables when an error occurs for 1

repo: dbt-labs/dbt-external-tables


text:
Move a page from Guides/Legacy to Docs

Description & motivation
Move the https://docs.getdbt.com/guides/legacy/managing-environments page out of the guides/legacy location and into the docs/ location.
Closes #2089
Checklist
If you added new pages (delete if not applicable):

[ ] The page has been added to website/sidebars.js
[x] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[x] The page has been removed from website/sidebars.js
[x] An entry has been added to _redirects
[ ] Run link testing to update the links that point to the deleted page

@runleonarun : could you review this, please?
question: should we add the page to the Docs left sidebar? or leave as is until the DevHub launch?
thanks so much!

I think moving it to the left nav is a good idea, @nghi-ly.

authors: nghi-ly runleonarun


text:
odig fails to compile with latest opam-lib in opam-repository

#=== ERROR while installing odig.0.0.1 ========================================#
# opam-version         1.2.2 (58ef3b8213100953848d362f7120a30356d7f77d)
# os                   linux
# command              ocaml pkg/pkg.ml build --pinned true --etc-dir /home/opam/.opam/4.03.0/etc/odig --lib-dir /home/opam/.opam/4.03.0/lib
# path                 /home/opam/.opam/4.03.0/build/odig.0.0.1
# compiler             4.03.0
# exit-code            1
# env-file             /home/opam/.opam/4.03.0/build/odig.0.0.1/odig-5-a2744c.env
# stdout-file          /home/opam/.opam/4.03.0/build/odig.0.0.1/odig-5-a2744c.out
# stderr-file          /home/opam/.opam/4.03.0/build/odig.0.0.1/odig-5-a2744c.err
### stdout ###
# [...]
# + ocamlfind ocamlopt -linkpkg -g -package 'cmdliner fmt.tty fmt.cli logs.fmt logs.cli' -package 'webbrowser webbrowser.cli' -package 'opam-lib compiler-libs.common' -package 'rresult asetmap fpath bos.setup mtime.os' src/odig_digest.cmx src/odig_trail.cmx src/odig_btrail.cmx src/odig_log.cmx src/odig_ocamlc.cmx src/odig_cobj.cmx src/odig_codec.cmx src/odig_etc.cmx src/odig_sexp.cmx src/odig_conf.cmx src/odig_html.cmx src/odig_opam.cmx src/odig_pkg.cmx src/odig_api_doc.cmx src/odig_dot.cmx src/odig_json.cmx src/odig_ocamldoc.cmx src/odig_ocamltop.cmx src/odig_odoc.cmx src/odig_top.cmx src/odig.cmx src/odig_cli.cmx src-bin/cli.cmx src-bin/cache.cmx src-bin/pkg_field.cmx src-bin/cobjs.cmx src-bin/pager.cmx src-bin/distrib_doc.cmx src-bin/doc.cmx src-bin/graf.cmx src-bin/guess_deps.cmx src-bin/help.cmx src-bin/listing.cmx src-bin/metadata.cmx src-bin/metadata_uri.cmx src-bin/ocamldoc.cmx src-bin/odoc.cmx src-bin/odig_bin.cmx -o src-bin/odig_bin.native
# findlib: [WARNING] Interface topdirs.cmi occurs in several directories: /home/opam/.opam/4.03.0/lib/ocaml/compiler-libs, /home/opam/.opam/4.03.0/lib/ocaml
# File "_none_", line 1:
# Error: No implementations provided for the following modules:
#          OpamTypesBase referenced from src/odig_opam.cmx
#          OpamPackage referenced from src/odig_opam.cmx
#          OpamFile referenced from src/odig_opam.cmx
#          OpamFilter referenced from src/odig_opam.cmx
#          OpamFormula referenced from src/odig_opam.cmx
# Command exited with code 2.
### stderr ###
# pkg.ml: [ERROR] cmd ['ocamlbuild' '-use-ocamlfind' '-classic-display' '-tag' 'debug'
# [...]
#      'src/odig_log.cmx' 'src/odig_json.cmx' 'src/odig_html.cmx'
#      'src/odig_etc.cmx' 'src/odig_dot.cmx' 'src/odig_digest.cmx'
#      'src/odig_conf.cmx' 'src/odig_codec.cmx' 'src/odig_cobj.cmx'
#      'src/odig_btrail.cmx' 'src/odig_api_doc.cmx' 'src/odig.cmx'
#      'src/odig.cmi' 'src/odig.mli' 'src/odig_cli.a' 'src/odig_cli.cmxs'
#      'src/odig_cli.cmxa' 'src/odig_cli.cma' 'src/odig_cli.cmx'
#      'src/odig_cli.cmi' 'src/odig_cli.mli' 'src/odig.bytetop'
#      'src-bin/odig_bin.native' 'etc/odig.conf' 'etc/odoc.css'
#      'etc/ocamldoc.css' 'toys/metagen.native']: exited with 10

see https://hub.docker.com/r/unikernel/mirage/builds/bpn57dzqfucgzgebo3rdgxk/

@avsm I have now migrated to use opam-format.

(in commit c21faa1fb85ae64a23197301a28)

authors: avsm dbuenzli

repo: dbuenzli/odig

license: isc