text:
Slider not showing decimal values

If "step" is in decimal then slider should show decimal values.

just add round parameter in your options, by instance round:2

It does not show decimals when set "from" and "to" properties. "round" property has no effect for me. Could it be related to a lower version of Angular? I use 1.2.14. Thanks.

can you send your options, I know I had many pb with decimal values...

This is the jsfiddle with your last version (it seems 2.2.7, but the release's name is 2.3.0). I didn't change the css, but I have had some problems with it too.
https://jsfiddle.net/g9e9n8xc/176/

yes indeed...from and to in decimal cause issue. I have to check but do not remember how it works right now. Quite busy I will check when I can, thanks for feedback.
https://github.com/darul75/angular-awesome-slider/blob/master/src/core/model/slider.factory.js
I would like take a look right now but can not until few days.

Ok, I will try to help you if I see something that could cause the issue. May be I can use a previous release at the moment. Nice work with this plugin. Thanks!

from and 2 were parse as integer values...

Nice! I reviewed your code but I didn't have time to check it properly. Many thanks for your time! Cheers.

you are welcome, do not forget putting stars in the sky ;)


text:
Does this library support React Native?

I'm trying to create Dash Wallet App with React Native but I'm getting following error
Error: Unable to resolve module fs from /Users/kryptomind/Desktop/Development/RN-Projects/DashWalletMobileApps-KM/node_modules/bls-signatures/blsjs.js: fs could not be found within the project or in these directories:
node_modules
The only thing seems to causing error is "fs" module which is not supported in react-native.
Expected Behavior
It's not working correctly while generating mnemonic.
Current Behavior
Getting following error
Error: Unable to resolve module fs from /Users/kryptomind/Desktop/Development/RN-Projects/DashWalletMobileApps-KM/node_modules/bls-signatures/blsjs.js: fs could not be found within the project or in these directories:
node_modules
Possible Solution

Steps to Reproduce (for bugs)

Create a react-native project ( npx react-native init AwesomeTSProject --template react-native-template-typescript )
Install rn-nodefy in react-native project and hack the Node so you can use node core module ( https://www.npmjs.com/package/rn-nodeify )
npm install dash
Generate Mnemonic using following code
import Dash from 'dash';
const client = new Dash.Client({
network: 'testnet',
wallet: {
mnemonic: null,
},
});
const mnemonic = client.wallet?.exportWallet();

Context
I'm not able to use dash sdk since It was the main ingredient to create dash wallet and add functionalities like send/receive and get transactions.
Your Environment

Version used: 3.21.2
Environment name and version (e.g. Chrome 39, node.js 5.4): Node : v14.18.1, React-Native: 0.66.1
Operating System and version (desktop, server, or mobile): Android/IOS

This comes from the WASM loader in the emscripten-generated WASM. The loader tries to understand whether the environment is nodejs or the browser, and tries to load WASM file through FS if run on nodejs or through fetch if run in the browser. I'm not familiar with options to load WASM in react-native, but I heard there indeed were some problems with it in react-native, so I don't know what would be the best way to solve this

Noted. Do i have use react-native specific WASM file in order to use this library?

fs module is nodejs core module .
see : https://nodejs.org/es/blog/release/v14.0.0/
that means this works for below 14 version of Nodejs

source: gharchive/issue

repo: dashevo/js-dash-sdk

license: MIT

license_type: permissive

license_source: github-api


text:
feat: min protocol version check for SML serialisation

Issue being fixed or feature implemented

This was reported/requested by @HashEngineering:

Older versions of our App won't sync due to  if (obj.nVersion == BASIC_BLS_VERSION) .  Older versions don't know what version a SML Entry is.  As such, they will never read the type field.  On the android client this causes an offset problem when reading the mnlistdiff and it will throw an exception that bans the peer that supplied it.  Soon enough, no peers will be left to connect to because they will all give the android client bad data.

What was done?

With this PR, SML will serialise the new v19 fields (nType, platformHTTPPort, platformNodeID) if the client's version is at least equal to 70227.
Note: Serialisation for hashing skips the above rule.
Also, functional test mininode protocol version is set to 70227.
How Has This Been Tested?

Breaking Changes

Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[x] I have assigned this pull request to a milestone

@UdjinM6 Don't know why linux64_tsan-test fails to start

@UdjinM6 Don't know why linux64_tsan-test fails to start

the failure was due to some infrastructure issues, restarted it

@UdjinM6 Moved the check above.

source: gharchive/pull-request

repo: dashpay/dash

license: mit

license_source: bigquery


text:
feat: mnlistdiff move nversion to first position

Issue being fixed or feature implemented
Version field should always be the first field of a message for better readibility.
What was done?

Introduced new protocol version MNLISTDIFF_VERSION_ORDER (70229).
nVersion serialisation order is changed for clients with protocol version greater than or equal to 70229.
For clients with protocol version >= 70225 and < 70229 the old order is used: can be deprecated in the future.
Increased functional test P2P mininode's protocol version to 70229.

How Has This Been Tested?
feature_llmq_rotation.py with new protocol version.
Breaking Changes
Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

light-NACK; Yes the version should be first but... do we really need to change it? I'd prefer to just avoid the complexity and added proto version unless mobile will have a hard time without this change (which looking at it, maybe they will/would, but I want that clarified)

authors: PastaPastaPasta ogabrielides


text:
fix: remove zero_length_array meta calculation in full_like

It is no longer necessary.

@agoose77 Just to make sure I didn't miss anything how does https://github.com/scikit-hep/awkward/pull/2333 affect this PR to remove an old workaround?

@lgray it shouldn't. That PR fixes the case that array-like arguments are evaluated with ==, rather than limiting it to scalars.

Thanks (for indulging my paranoia)! :-)

repo: dask-contrib/dask-awkward


text:
Add Row-Wise Mode Functionality (axis=1) and Improve Metadata Handling in _collection.py

Added functionality for row-wise mode calculation (axis=1) to support the Dask DataFrame API.
The new implementation dynamically handles row-wise mode and ensures consistent metadata handling across partitions.
Added validation for the axis parameter, with appropriate error handling for unsupported values.
Ensured compatibility with existing column-wise (axis=0) mode functionality, preserving the original behavior for that case.

Resolves Dask Expressions issue #1136 and Dask issue #11389

Unit tests for the added functionality have been submitted in the Dask repository’s PR due to the initial issue location and the entry point for the functionality being within dask/dataframe/core.py. Please note that this PR in Dask-expr must be approved before the Dask PR tests can successfully run, as the tests depend on the changes made here.

Apologies for the delay — I’ve had some commitments with my day job this past week. The current version has been simplified, with no explicit typecasting or nested functions. It has passed both my local bug script and the pending unit test in the Dask repo.
Note: I accidentally pushed my venv with GitHub Desktop earlier, but I’ve since re-pushed with the venv directory removed.

repo: dask/dask-expr


text:
Migrate Slurm to use rockylinux

It looks like the container that the Slurm container was previously based on hasn't been updated on Docker Hub for four years. However the GitHub repo is still active, you just have to clone and build the image yourself.
It felt easier to vendor in the Dockerfile and add our modifications to install Python. In dask-docker we used to do a remote build, but since docker buildx was made the default this got a lot harder, so I'm apprehensive to go down that road again. I also explored cloning the repo locally and building it before building the main container but using docker compose build to build images for pushing doesn't support using cached images that are on Docker Hub.

Builds are passing again so I'm going to go ahead and merge.

repo: dask/dask-jobqueue


text:
Optionally allow pyFFTW in place of numpy.fft

The pyFFTW provides a nice lightweight Python library that wraps FFTW. Given how well optimized FFTW is, this ends up being significantly faster. Thanks to its interfaces it is pretty easy to drop in a pyFFTW in place of NumPy's FFTPACK. Would be nice if there was an option (maybe a function?) to change up the FFT backend used in Dask.

Looking at https://github.com/dask/dask/blob/master/dask/array/fft.py can
you think of a good way to do this?  Might also want to expand to scipy?
On Thu, Mar 16, 2017 at 9:55 AM, jakirkham notifications@github.com wrote:

The pyFFTW provides a nice lightweight Python library that wraps FFTW.
Given how well optimized FFTW is, this ends up being significantly faster.
Thanks to its interfaces
http://pyfftw.github.io/pyFFTW/pyfftw/interfaces/interfaces.html it is
pretty easy to drop in a pyFFTW in place of NumPy's FFTPACK. Would be nice
if there was an option (maybe a function?) to change up the FFT backend
used in Dask.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/2092, or mute the thread
https://github.com/notifications/unsubscribe-auth/AASszD9_9z2-sFTIBia9EqIQNlGjSlgTks5rmT9lgaJpZM4MfVRT
.

This is not very clever, but I had thought about just reassigning what npfft points to after the fact.

Yeah, Dask probably can't do that responsibly.  You could in your own code thought.
Probably there is some way to change _fft_wrap to take a function name, like 'fft' and produce a function that takes an fft module as a keyword argument.  This keyword would default to np.fft but could be replaced by others.

A different idea might be to let people wrap their own FFTs. So we provide functions like *fft_wrap, which take an FFT function of that type and return a function that can perform an FFT of that form. So as an example fft_wrap(numpy.fft.fft) would return the same thing as dask.array.fft.fft.

Sure, that sounds like a great idea.  Presumably this works well now.  We might want to make the _fft_wrap function slightly more user-facing with a nice docstring with examples and such.

Ok, I'll give it a look.

Please see PR ( https://github.com/dask/dask/pull/2093 ).

Probably worth noting that as of pyFFTW 0.11.0+, they provide a Dask-like interface, which can be used as a drop-in substitute for the one we provide.
ref: https://pyfftw.readthedocs.io/en/latest/source/pyfftw/interfaces/interfaces.html#dask-fft

Awesome.  I'm glad to hear it.
On Mon, Nov 26, 2018 at 4:52 PM jakirkham notifications@github.com wrote:

Probably worth noting that as of pyFFTW 0.11.0+, they provide a Dask-like
interface, which can be used as a drop-in substitute for the one we provide.
ref:
https://pyfftw.readthedocs.io/en/latest/source/pyfftw/interfaces/interfaces.html#dask-fft
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/2092#issuecomment-441812636, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszAQis7hFat2jixJsmNAjV5vU6HH4ks5uzGKngaJpZM4MfVRT
.

authors: jakirkham mrocklin

repo: dask/dask


text:
Optimize scalar condition in where

Fixes https://github.com/dask/dask/issues/2407
If the condition provided to da.where is a simple bool or np.bool8 value, optimize out the call to where and simply return, which of the two arrays was selected.
As da.where and np.where also handle things like broadcasting and type promotion, these things need to be handled as well. However broadcasting and type promotion are also designed to be optimized out if they are no-ops. So in the ideal case one could merely get back one of the Dask Arrays they provided with no change.
Include a test that verifies da.where is optimized out for Boolean values.

Looks like this is now passing. Any more thoughts on this one?

Thanks @jakirkham !

Great, thanks!


text:
Support partition_on for pyarrow

[x] Tests added / passed
[x] Passes flake8 dask

This will require Arrow 0.8.0, leaving this for review until we have a release (hopefully in the next days).

This has merge conflicts. If all tests pass after resolving, then this looks good to me. Thanks @xhochy.

I've pushed a merge commit.  There are some changes here that I'm unsure of though.  @xhochy if you have a moment to take a look that might be best.

Looks fine 👍

authors: jcrist mrocklin xhochy


text:
Fixes broken links (#4058)

Some additional items addressed:

changed permissions to remove executable bit on dask/array/gufunc.py
altered sphinx configuration to not link check localhost and github.com (rate limiting) URLs
fixed all broken links identified in make linkcheck

[x] Tests added / passed
[x] Passes black dask / flake8 dask

Draft request currently, as part of 4058 will investigate whether any other alterations are required to support CI and what impact the removal of github checks have before finalising

This looks good to me.  I'd be happy to merge, but I'll hold off until you're ready.

Given the advice in the other issue I will look into other options, is it preferable to wait and add commits to this PR or would you prefer to merge and get the fixed links in sooner?
I should be able to look at this over the weekend

Hey @JimCircadian -- I know you were looking at pytest-check-links but is there anything else you wanted to get in to this PR?

Hi @gforsyth there were the broken link fixes that I found as a result of the pytest run I did (ref. the issue) so maybe worth adding these fixes to this PR? I should be able to do this Friday (deep in some other work at the moment, apologies!)

No worries @JimCircadian -- I just didn't want this to get lost in the shuffle.  We'll hold off on merging until you say otherwise.

I think this is largely complete but having trouble with the CI tests at the moment, I don't think that's related to my changes. There's more discussion in the related issue, so I consider it in a good place for the moment (the links are fixed!!!)

Hey @JimCircadian -- can you rebase or merge master to pick up the changes there?  That should remove the CI issues and we can get this in.

Or maybe I can re-run the tests?

@gforsyth Looks good now, thanks for fixing that issue!

Also @JimCircadian, I see this is your first PR to dask.  Welcome!  Thanks for fixing up the docs!

authors: JimCircadian gforsyth jsignell mrocklin


text:
CDAP-14474 Update GCP Speech transform

Update includes:

Rename the plugin from SpeechTranslator to SpeechToText as it is not a translator
Add description to configs
Fix the check done during the deploy time on the inputSchema to include the audio field and it to be bytes. This was not happening before.
The schema given by user is output schema and was being used as both input and output earlier
Updated to pick credential file from environment
Add Reference doc

Issue: https://issues.cask.co/browse/CDAP-14474

It will be good to emit FLL from this transform. I will add that in a separate PR after this one.

@albertshau Please have another look.

repo: data-integrations/google-cloud


text:
CARDS-1623: Add basic support for DICOM image storage

This PR implements CARDS-1623 by making the cards:DicomAnswer data type available which allows for the creation of Questionnaires containing DICOM based answers. The DICOM Answer is based on the File Answer with the extension of that when a DICOM file is uploaded, a preview is displayed in the Questionnaire and the DICOM file's metadata is made available in the Notes section for that DICOM question. The original DICOM file is also available for download through the completed Form.
To test:

Build this CARDS-1623 branch (mvn clean install).
Start CARDS in ClinDig mode (./start_cards.sh --project cards4clindig --dev).
Point your browser to http://localhost:8080 and login as admin:admin.
Create a new Questionnaire of type Imaging along with a new associated Subject.
Download the file MR-MONO2-16-knee.gz from https://barre.dev/medical/samples/#MR-MONO2-16-knee and extract it to a DICOM image with the command cat MR-MONO2-16-knee.gz | gzip -d > MR-MONO2-16-knee.dcm.
Upload the file MR-MONO2-16-knee.dcm as an answer to the DICOM question in CARDS.
Verify that a preview is displayed and that the associated metadata is correctly populated in that question's Notes field.
Try editing the Notes and re-visiting the Form. All should work as expected.
You may also wish to test with some of the other DICOM files available on https://barre.dev/medical/samples/

It does not work for the following image:

authors: IntegralProgrammer sashaandjic

repo: data-team-uhn/cards


text:
Fix SCIM Pagination default parameters in the Go SDK

Changes
This PR sets default parameters for the SCIM pagination APIs, specifically setting StartIndex to 1 (as specified in the SCIM RFC, startindex is 1-based index) and Count to 100 to ensure that we paginate requests to the REST API. This should help reduce load on the SCIM APIs by decreasing the size of response bodies for users of this SDK.
This change supersedes https://github.com/databricks/databricks-sdk-go/pull/693.
Tests

[ ] make test passing
[ ] make fmt applied
[ ] relevant integration tests applied

Codecov Report
Attention: 18 lines in your changes are missing coverage. Please review.

Comparison is base (516bf04) 16.86% compared to head (580f78a) 16.84%.

Files
Patch %
Lines

service/iam/api.go
0.00%
18 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #717      +/-   ##
==========================================
- Coverage   16.86%   16.84%   -0.03%
==========================================
  Files         103      103
  Lines       14089    14107      +18
==========================================
  Hits         2376     2376
- Misses      11507    11525      +18
  Partials      206      206

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

authors: codecov-commenter mgyucht

repo: databricks/databricks-sdk-go


text:
show_views macro fails on subsequent executions of the dbt run command

Describe the bug
This is a strange one.  I'm currently trying to set up a DBT workflow where each new feature branch of a DBT project would auto-provision a new catalog in Databricks Unity Catalog to isolate development changes until they are ready to merge into production.  To facilitate the workflow, I have set up the DBT project's profiles.yml file like so:
sample_project:
  target: dev
  outputs:
    dev:
      catalog: "sample_project_{{ env_var('DBT_CATALOG_SUFFIX') | replace('-', '_') | replace('/', '_') }}"
      host: "{{ env_var('DBT_HOST') }}"
      http_path: "{{ env_var('DBT_HTTP_PATH') }}"
      schema: sample_project
      token: "{{ env_var('DBT_ENV_SECRET_TOKEN') }}"
      type: databricks
    prod:
      catalog: production
      host: "{{ env_var('DBT_HOST') }}"
      http_path: "{{ env_var('DBT_HTTP_PATH') }}"
      schema: sample_project
      token: "{{ env_var('DBT_ENV_SECRET_TOKEN') }}"
      type: databricks

Where the catalog for the dev target is dynamically generated by appending an environment variable value to it, everything seems to work fine on the first execution of the dbt run command when I invoke it with the following flags:
dbt run  --profiles-dir . --target dev

However, on subsequent runs of the same command without any changes, I get the following error:
17:14:34  Encountered an error:
Runtime Error
  Runtime Error
    Runtime Error
      Fail to execute the command as the target schema `sample_project_feature_DA_2000.sample_project` is not in the current catalog. Please set the current catalog with 'USE CATALOG sample_project_feature_DA_2000' first.

After looking through the logs, I see this output:
[0m13:14:34.410049 [debug] [ThreadPool]: Databricks adapter: Error while running:
/* {"app": "dbt", "dbt_version": "1.6.0", "dbt_databricks_version": "1.6.1", "databricks_sql_connector_version": "2.7.0", "profile_name": "sample_project", "target_name": "dev", "connection_name": "list_sample_project_feature_DA_2000_sample_project"} */
show views in `sample_project_feature_DA_2000`.`sample_project`

It seems that the Databricks adapter is trying to run the SHOW VIEWS query in Databricks, and the docs seem to indicate that it doesn't support passing the catalog name in with the schema name.
I was able to solve the problem locally by modifying the source code for the macro from:
{% macro databricks__show_views(relation) %}
  {% call statement('show_views', fetch_result=True) -%}
    show views in {{ relation }}
  {% endcall %}

  {% do return(load_result('show_views').table) %}
{% endmacro %}

to this:
{% macro databricks__show_views(relation) %}
  {% set relation_parts = "{}".format(relation).split('.') %}
  {% call statement('use_catalog', fetch_result=True) -%}
    use catalog {{ relation_parts[0] }}
  {% endcall %}
  {% call statement('show_views', fetch_result=True) -%}
    show views in {{ relation_parts[1] }}
  {% endcall %}

  {% do return(load_result('show_views').table) %}
{% endmacro %}

I'm sure there is a more elegant solution to this problem.
Steps To Reproduce

Setup a DBT project with Databricks Unity Catalog
Configure profiles.yml to use a dynamic catalog in the target
Ensure the development catalog is created and the user you run the dbt process with has USE CATALOG, CREATE SCHEMA, and USE SCHEMA permissions in the catalog
execute dbt run and receive a successful result
execute dbt run again without any changes and receive a failure

Expected behavior
After the initial successful invocation of the dbt run command, any subsequent dbt run command invocations without changes should result in another successful result rather than a failure.
System information
The output of dbt --version:
Core:
  - installed: 1.6.0
  - latest:    1.6.0 - Up to date!

Plugins:
  - databricks: 1.6.1 - Up to date!
  - spark:      1.6.0 - Up to date!

The operating system you're using:
Apple M2 Pro macOS 13.4.1
The output of python --version:
Python 3.10.10
Additional context
My sample project has one model that materializes a table and one model that creates a view.  I added so much context about the dynamic catalog in the profiles.yml because when I ran the same steps with a static catalog in my profiles.yml file, this bug did not occur.

While this does look like a bug, I'm wondering why you are dynamically making catalogs rather than schemas?

Would you mind submitting a PR with your solution?  Given the limitations you described, it seems reasonable to me.

authors: benc-db zayscue

repo: databricks/dbt-databricks


text:
Implemented Series.argsort

This PR proposes Series.argsort
>>> kser = ks.Series([3, 3, 4, 1, 6, 2, 3, 7, 8, 7, 10])
>>> kser
0      3
1      3
2      4
3      1
4      6
5      2
6      3
7      7
8      8
9      7
10    10
dtype: int64

>>> kser.argsort().sort_index()
0      3
1      5
2      0
3      1
4      6
5      2
6      4
7      7
8      9
9      8
10    10
dtype: int64

The result in the description is different from pandas'.
>>> pd.Series([3, 3, 4, 1, 6, 2, 3, 7, 8, 7, 10]).argsort()
0      3
1      5
2      0
3      1
4      6
5      2
6      4
7      7
8      9
9      8
10    10
dtype: int64

The result in the description is different from pandas'.
>>> pd.Series([3, 3, 4, 1, 6, 2, 3, 7, 8, 7, 10]).argsort()
0      3
1      5
2      0
3      1
4      6
5      2
6      4
7      7
8      9
9      8
10    10
dtype: int64

Oh, sorry I should've updated PR description.

Thanks, @ueshin !

Anyway, I want to include @ueshin as an author this PR.
Maybe merged pull request in my repository is enough, or I have to do something ??

Nvm about https://github.com/databricks/koalas/pull/1793#issuecomment-698657964 , think I'll be able to address it !

Oops. why the co-author didn't applied ? Sorry let me figure out the way.

authors: itholic ueshin

repo: databricks/koalas


text:
validated vnsLDevCtx with context=""

Validation Type
[x] - Config
The empty context for vnsLDevCtx  are not supported and can cause unexpected forwarding issues.
What needs to be validated
if vnsLDevCtx.context=="", raise the alert before upgrading.
Why it needs to be validated
To prevent svg outage post upgrade
Additional context
Add any other context about the feature request here.

https://github.com/datacenter/ACI-Pre-Upgrade-Validation-Script/tree/welkin-issue-78

Would you mind sharing a little bit more details on this? For instance whether this is specific to upgrade, a bug, etc.

@welkin-he any more detail/bug you can provide for this specific check?

No further details shared, closing this one out

authors: monrog2 takishida welkin-he


text:
Add EndpointPayload ABC and Specific Payload Dataclasses for API Requests

Note:
~First PR in the stack: #202~ (merged)
~Second PR in the stack: #203~ (merged)
Third PR in the stack: #204 <--- Currently rebased up to this point
Fourth PR in the stack #205
Previous PR in the stack #206
Will manually rebase as PRs in the stack are merged to make it easier to review the changes introduced in this PR.

This PR adds 'payload' dataclasses for the API endpoints.
The EndpointPayload class is an Abstract Base Class, which defines the structure for all other payload classes. Payload classes are in charge of normalizing and validating the payload before it is used for a POST request. They also implement a .to_dict property for easy access to the correctly structured payload data as a python dictionary.
Details:

datacommons_client/endpoints/payloads.py: Added abstract base class EndpointPayload and specific payload classes (NodePayload, ObservationPayload, ResolvePayload, SparqlPayload) to structure, normalize, and validate payloads for different API requests.

Tests

datacommons_client/tests/endpoints/test_payloads.py: Introduced test cases for each payload class.

Thanks @jm-rivera ! To make things a little simpler with this stack of PRs, it may be easier to just address all comments in this PR (208). what do you think?

Thanks for all your reviews @dwnoble!  To recap a bit where things stand now:

#202 (merged)
#203 (merged)

I have also rebased all of the PRs throughout the stack, resolving conflicts and incorporating the right changes. That should hopefully help a bit with the review.

#204 new review requested. I've rebased it and then made the changes you requested before requesting a new review.
#205: Also rebased. Contains all the changes from 204, plus I've fixed the filename typo. Awaiting a new review.
#206: Also rebased, incorporating all changes from 204 and 205. It's been approved but will wait until we merge 204 and 205 to rebase again and merge.

And #208 is here, I've rebased incorporating all changes from 204, 205, 206 and resolving conflicts.
--
Happy to follow whatever you prefer in terms of how/where we resolve anything that is outstanding.

repo: datacommonsorg/api-python


text:
Size switches to custom from column X if map height adjusted

It should stick with the column selection

Fixed with https://github.com/datadesk/web-map-maker/commit/a83689e04fd0c5db620bd039e4757dc1577cfca7

repo: datadesk/web-map-maker


text:
Correction for raw affiliation ICAR, UMR 5191, Université Lumière Lyon 2

Correction needed for raw affiliation ICAR, UMR 5191, Université Lumière Lyon 2
raw_affiliation_name: ICAR, UMR 5191, Université Lumière Lyon 2
new_rors: 03rth4p18;04mn0nr69
previous_rors: 03rth4p18
works_examples: W2291355721
contact: 00f965982107aad966bebeac9c952f59:1e5046b14053ce43ca44a4fc57e7 @ ens-lyon.fr

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

repo: dataesr/openalex-affiliations


text:
Correction for raw affiliation Mohammed V University, Rabat 10090, Morocco,; Research Group in Biomedical Engineering and Pharmaceutical Sciences, ENSAM,; STI Laboratory, T-IDMS, Faculty of Sciences and Techniques, Moulay Ismail University of Meknes, Errachidia 52000, Morocco.; the High School of Technology ESTC, University of Hassan II, Casablanca 20153, Morocco.

Correction needed for raw affiliation Mohammed V University, Rabat 10090, Morocco,; Research Group in Biomedical Engineering and Pharmaceutical Sciences, ENSAM,; STI Laboratory, T-IDMS, Faculty of Sciences and Techniques, Moulay Ismail University of Meknes, Errachidia 52000, Morocco.; the High School of Technology ESTC, University of Hassan II, Casablanca 20153, Morocco.
raw_affiliation_name: Mohammed V University, Rabat 10090, Morocco,; Research Group in Biomedical Engineering and Pharmaceutical Sciences, ENSAM,; STI Laboratory, T-IDMS, Faculty of Sciences and Techniques, Moulay Ismail University of Meknes, Errachidia 52000, Morocco.; the High School of Technology ESTC, University of Hassan II, Casablanca 20153, Morocco.
new_rors: 04cnscd67;00r8w8f84;001q4kn48
previous_rors: 04cnscd67;018pp1107;00r8w8f84;001q4kn48
works_examples: W4362705033
contact: 92a4bfa85a04ad88d6c28728de40fd42:d94fa14e5edb86fecb84082957 @ ensam.eu

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.


text:
Correction for raw affiliation CAPHI - Centre Atlantique de Philosophie (Bât. 32B, Université de Rennes 1, Campus de Beaulieu, 35042 Rennes Cedex
Chemin de la Censive du Tertre - BP 81227 - 44312 Nantes Cedex 3 - France  - France); UR Médecine - Université de Rennes - Faculté de Médecine (2 avenue du Professeur Léon Bernard - CS 34317 - 35043 Rennes cedex - France)

Correction needed for raw affiliation CAPHI - Centre Atlantique de Philosophie (Bât. 32B, Université de Rennes 1, Campus de Beaulieu, 35042 Rennes Cedex
Chemin de la Censive du Tertre - BP 81227 - 44312 Nantes Cedex 3 - France  - France); UR Médecine - Université de Rennes - Faculté de Médecine (2 avenue du Professeur Léon Bernard - CS 34317 - 35043 Rennes cedex - France)
raw_affiliation_name: CAPHI - Centre Atlantique de Philosophie (Bât. 32B, Université de Rennes 1, Campus de Beaulieu, 35042 Rennes Cedex
Chemin de la Censive du Tertre - BP 81227 - 44312 Nantes Cedex 3 - France  - France); UR Médecine - Université de Rennes - Faculté de Médecine (2 avenue du Professeur Léon Bernard - CS 34317 - 35043 Rennes cedex - France)
new_rors: 015m7wh34;059ynac98
previous_rors: 015m7wh34
works_examples: W4384831296
contact: 102df354a2df9c164922038c390e9b80:01454bf730728f648b93da8fcfa41724 @ univ-rennes.fr

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.


text:
Correction for raw affiliation Laboratoire de Probabilités, Statistique et Modélisation (LPSM), Sorbonne Université and Université Paris Cité, CNRS UMR 8001, Paris, France

Correction needed for raw affiliation Laboratoire de Probabilités, Statistique et Modélisation (LPSM), Sorbonne Université and Université Paris Cité, CNRS UMR 8001, Paris, France
raw_affiliation_name: Laboratoire de Probabilités, Statistique et Modélisation (LPSM), Sorbonne Université and Université Paris Cité, CNRS UMR 8001, Paris, France
new_rors: 02en5vm52;05f82e368;02feahw73;02vnd0e65
previous_rors: 02en5vm52;05f82e368;02feahw73
works_examples: W4388814353
contact: 885b841f1efd744fdf1d9d9d355a08ae:158247062561603555b396d2c3 @ sorbonne-universite.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 King's University College at Western University, Canada

Correction needed for raw affiliation King's University College at Western University, Canada
raw_affiliation_name: King's University College at Western University, Canada
new_rors: ;02grkyz14
previous_rors: 048f5b183;02grkyz14
works_examples: W4388021888;W4383822577;W4323794426
contact: 12d00638807e5e27c3627087e46e322b:039fe3d903dc4b3a @ uwo.ca

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


text:
Feature/hacker localization nb no

Added Norwegian localization for Hacker (adjectives, nouns and verbs).

Codecov Report

Merging #755 (96aeeb3) into main (b6dcc89) will increase coverage by 0.05%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #755      +/-   ##
============================================
+ Coverage     92.80%   92.86%   +0.05%
  Complexity     2595     2595
============================================
  Files           281      281
  Lines          5127     5127
  Branches        530      530
============================================
+ Hits           4758     4761       +3
+ Misses          243      240       -3
  Partials        126      126

see 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Looks fine to me! Thanks for your contribution!

authors: Majid-J bodiam codecov-commenter

repo: datafaker-net/datafaker


text:
Track embeddable views pages

As an Admin, I want to track page views independently so I know where the traffic is coming from.
Acceptance criteria

[ ] I can find stats for embedded views pages in GA dashboard

Tasks

[ ] add tracking script in /view.html template

INVALID - it was already fixed during the main implementation

repo: datahq/frontend


text:
Switch to markdown-it from marked

Why?

Commonmark by default

fenced code blocks by default

Very extensible with plugins and this means support for stuff like:

toc - https://www.npmjs.com/package/markdown-it-table-of-contents
info sections

Other projects I've looked at seem to use it / consider it e.g. hackmd
Acceptance criteria

[x] we use markdown-it to convert markdown into html
[x] supports headers as anchors
[x] supports block containers (:::info)

Tasks

[x] use markdown-it instead of marked here - https://github.com/datahq/frontend/blob/master/lib/utils/index.js#L58
[x] user markdown-it-anchor for supporting headers as anchors
[x] use markdown-it-container for supporting block containers
[x] ensure all tests are passing

@anuveyatsu have we checked markdown-it can do everything we are using marked for see e.g. the docs route for an example of full usage.

@rufuspollock yes.
Headings can have an embedded anchor tag like on GitHub using plugins:

https://github.com/valeriangalliat/markdown-it-anchor
https://github.com/medfreeman/markdown-it-toc-and-anchor

markdown-it also discussing having it as built-in feature here - https://github.com/markdown-it/markdown-it/issues/28 - but probably not soon.
Re other options we're using with marked:

GFM and tables are enabled by default in markdown-it
smartLists - not sure what exactly does this option in marked - behaves similarly to me.
smartypants - there is something similar called typographer

FIXED. We do CSS stuff in a separate issue, e.g., for block containers. Atm, behavior is as the following:
:::info
some info message
:::

==>

some info message

authors: anuveyatsu rufuspollock


text:
Request: Updating the sqlglotrs dependency

Hi Datahub team,
Currently we are facing issues with adopting your library due to upstream dependencies. Currently your team is using sqlglotrs 0.2.8, which does not have a license in pypi: https://pypi.org/project/sqlglotrs/0.2.8/ and the licensing document was excluded from the library download. My team has requested that the sqlglotrs team update their download with the license agreement in version 0.2.10 and forward. I was wondering if you team could also update the dependency to pull from 0.2.10 or 0.2.11? My team is hoping to release relatively soon, but can not do so due to this. We are trying to do some legal reviews for our launch by September 14th and would love to continue using datahub.
Thanks,
Chris Chan

Hi @chriscc2 - could you contact @hsheth2 on DataHub Slack to carry this conversation further?

This PR https://github.com/datahub-project/datahub/pull/11331 will bump us to sqlglotrs 0.2.12

Thanks for the help @hsheth2. How can we track if the latest version with the dependency is available on pypi?

@dbjnbnrj @chriscc2 this is part of acryl-datahub v0.14.0.5

repo: datahub-project/datahub


text:
hillsborough_county_evictions_geocoded.csv formatting issue

Looks like there might be an issue with quoting vs delimiter issue  on a few rows?

I'm looking at this, but I'm no expert, so if someone else knows more, feel free to go for it.

It looks like the .csv is using doublequotes as an escape character for doublequotes inside fields. For instance, "11624 GROVEWOOD AVE ""B""" refers to the string '11624 GROVEWOOD AVE "B"'. That's confusing the read_csv() function because it reads anything in doublequotes as a string.
A regex to change the first doublequote to a more appropriate escape character, along with adding the escape character to the read_csv params, could work. I tried coming up with the regex myself but no luck so far.

I found and fixed the problems... PR coming..

Resolved by https://github.com/datakind/Mar21-housing-insecurity/pull/14

authors: DMW2151 LeeBreisacher dgillen snnehete


text:
Set up custom primevue theme

Setting up a custom theme for primevue instead of applying global css to primevue components

Set transition time to 0.05s (default: 0.2s)
Set the padding of datatable headers and cells to 0.5rem (default: offset between header and bodycell)
Hides the sorting icon until the header is hovered
Make the text in the cells appear on a single line (nowrap and overflow hidden)
Make the filter icon change color when a filter is active
Checkboxes color grey

pre-commit.ci autofix

repo: datalab-org/datalab


text:
Don't run "brew update" more than once

As requested here.

Codecov Report

Merging #59 (5ebb68d) into master (01dbb27) will increase coverage by 0.39%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##           master      #59      +/-   ##
==========================================
+ Coverage   64.56%   64.96%   +0.39%
==========================================
  Files           1        1
  Lines        1016     1019       +3
  Branches      192      193       +1
==========================================
+ Hits          656      662       +6
+ Misses        302      299       -3
  Partials       58       58

Impacted Files
Coverage Δ

src/datalad_installer.py
64.96% <25.00%> (+0.39%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 01dbb27...5ebb68d. Read the comment docs.

looks good, thank you!

authors: codecov-commenter jwodder yarikoptic

repo: datalad/datalad-installer


text:
RNS-800: Adds unparsing for Casting of [float value string] (eg: '45.34') to Integer in Bigquery

https://datametica.atlassian.net/browse/RNBS-800

3 test cases seems to be failing on the branch. Please fix them, refer attached snap for the same :

LGTM.

repo: datametica/calcite


text:
First action

please merge this time

committing

repo: datamike/hello-github-actions


text:
Make Batching Strategy configurable

use environment variable for batching of messages.

Fixed in commit 933d3a6. Added Env Variables BATCHING_ENABLED and BATCHING_MAX_PUBLISH_DELAY_MS, and updated README.

thank you @jdabello


text:
Check err on bundle.go

fixes https://github.com/datastax/cql-proxy/issues/109

It looks like there's already an outstanding patch for this: https://github.com/datastax/cql-proxy/pull/107

authors: kokizzu mpenick

repo: datastax/cql-proxy


text:
[HW]

Name: SUSHMA H S
Email: sushmahdkote@gmail.com
Linkedin Profile:
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:
sorry sir I couldn't complete my second homework

Thanks Sushma!

Sushma, were you able to get the app deployed to Netlify or did you run into trouble there?

Badge is not issued for me 🤔

Sushma, were you able to get the app deployed to Netlify or did you run into trouble there?

No I got

Hey SUSHMA H S, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/ISkq3cb6TvaKbiBZSdduSA

authors: HadesArchitect RyanWelford SonicDMG sushmahs2

repo: datastaxdevs/workshop-astra-tik-tok


text:
[Improvement] File with incorrect header / missing LICENSE information

What would you like to be improved?
I'm not 100% sure on this one, but this files seem to be a 3rd party file from https://github.com/tomsisso/spark-profiling-plugin/
/spark-connector/src/main/java/com/datastrato/gravitino/spark/connector/plugin/GravitinoSparkPlugin.java
The file is fairly trivial but if it was copied we shouldn't claim it is ours.
How should we improve?
If copies put back the original header and list the license and file in LICENSE and LICENSE.bin.

@FANNG1 can you please also take a look.

@justinmclean  it was not copied, the class is simple
public class GravitinoSparkPlugin implements SparkPlugin {

  @Override
  public DriverPlugin driverPlugin() {
    return new GravitinoDriverPlugin();
  }

  @Override
  public ExecutorPlugin executorPlugin() {
    return null;
  }
}

Do you think it is different enough to have the Datastrato header?

yes, I think we could add  the Datastrato header

@justinmclean  do you think we could close this issue?

Yes. Wha'st important is we've documented this decision and can point to it if needed in the future.

authors: FANNG1 jerryshao justinmclean

repo: datastrato/gravitino


text:
Improve Frontend User Experience

1. Form Validation
The current form setup only enforces minimum values without any client-side error handling. To improve the user experience, implement JavaScript validation that provides immediate feedback if the input is incorrect.
2. JavaScript Error Handling
The error handling in the fetch request currently only logs errors to the console. This could be improved by providing user-facing feedback if something goes wrong with the request. Add error messages that are displayed directly on the form.
3. Security Considerations
The code prevents the default action for the F12 key, but other shortcuts like Ctrl+Shift+I are still allowed. Expand the prevention logic to include other common shortcuts that open developer tools.
4. Form Data Handling
Currently, FormData is sent directly with the fetch request without any prior validation. It's important to sanitize and validate this data before sending it to the server to avoid potential security issues.
5. Interactive Form UX
The form fields and buttons could be more interactive by adding focus states and hover effects. Using Tailwind CSS, you can easily enhance the user experience by adding these states to the form elements.

Hi @datavorous
I am a participant of hacktoberfest. I would like to improve the interactive form UX and would work on this issue.
Kindly, assign this task on my name.
Thanks,
Shivam

@shivam123-dev I've assigned you with this issue. All the best :)

Hey @datavorous can I be assigned this issue for hackatober fest? I have experience in web development and python.

@hardikjumnani I'll assign you if @shivam123-dev does not respond within few (3) days from now. @shivam123-dev do you have any updates?

Hey @datavorous, can I be assigned the issue, it has been 5 days.

@hardikjumnani I've assigned you.

authors: datavorous hardikjumnani shivam123-dev

repo: datavorous/amine


text:
--metadata-update argument not recognized

Describe the bug
Using the --metadata-update argument leads to immediate crashing due to "unrecognized argument"
To Reproduce
Steps to reproduce the behavior:
ofscraper_windows_a42cfee.exe --metadata-update
Expected behavior
Per docs: "This will update files on system based on config, if a file does not exist on system it will be marked as not downloaded in data"
Screenshots/Logs
Traceback (most recent call last):
File "main.py", line 4, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\runner\load.py", line 6, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\runner\run.py", line 5, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\commands\picker\picker.py", line 1, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\commands\commands\check.py", line 13, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\api\archive.py", line 23, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\utils\constants.py", line 3, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\utils\config\custom.py", line 3, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\utils\config\file.py", line 7, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\utils\config\utils\context.py", line 5, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\prompts\prompts.py", line 14, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\prompts\prompt_groups\actions.py", line 4, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\prompts\promptConvert.py", line 10, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\utils\live\empty.py", line 4, in
File "", line 1176, in _find_and_load
File "", line 1147, in _find_and_load_unlocked
File "", line 690, in _load_unlocked
File "PyInstaller\loader\pyimod02_importers.py", line 419, in exec_module
File "ofscraper\utils\console.py", line 16, in
File "ofscraper\utils\args\accessors\output.py", line 4, in low_output
File "ofscraper\utils\args\accessors\read.py", line 23, in retriveArgs
File "ofscraper\utils\args\main.py", line 46, in parse_args
File "click\core.py", line 1157, in call
File "click\core.py", line 1077, in main
File "click\core.py", line 943, in make_context
File "cloup\constraints_support.py", line 183, in parse_args
File "click\core.py", line 1644, in parse_args
File "click\core.py", line 1405, in parse_args
File "click\parser.py", line 337, in parse_args
File "click\parser.py", line 364, in _process_args_for_options
File "click\parser.py", line 514, in _process_opts
File "click\parser.py", line 398, in _match_long_opt
click.exceptions.NoSuchOption: No such option: --metadata-update
[11452] Failed to execute script 'main' due to unhandled exception!
System Info

OS: Windows 10
Browser: Chrome
Binary version 2024.07.28_201807_dev/3.11_a42cfee_package

It is because it is a subcommand it does not have the same arguments as the main ofscraper command, and it has special arguments only for the ofscraper metadata comand
ofscaper metadata -h

to list possible options
ofscraper metadata --metadata-update

It is because it is a subcommand it does not have the same arguments as the main ofscraper command, and it has special arguments only for the ofscraper metadata comand
ofscaper metadata -h

to list possible options
ofscraper metadata --metadata-update

Alright, thank you. Please note that this page in the docs does not specify the corrext syntax for the command. It omits that it is a subcommand. It should get corrected when possible

authors: DADESUPER datawhores

repo: datawhores/OF-Scraper


text:
Naive fix to #106

Rather than having sys.stdout get evaluated in the function definition,
use None as a default value and replace None with stdout in the
function body. Not sure this is the right solution...

Looks good to me.

Thanks for the contribution!

authors: MatthewWest hoytak

repo: dato-code/SFrame


text:
persistState across windows

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[x ] Support request
[ ] Other... Please describe:

Basic repro here:
https://stackblitz.com/edit/akita-persiststate-windowed
Just looking for some guidance or best practices if this has been done before.
I'm attempting to access state across windows - ie from a window I've opened within the app using persistState (localStorage or sessionStorage). When the new window first opens, all good, state from the main window is read from storage.
Updating the state in one window obviously doesn't reflect in the other window though. Do you have any recommendations on how to sync state between windows?
One these hooks perhaps - deserialize and serialize, preStorageUpdate or preStoreUpdate?
Fantastic work on the project as well!

What about something like:
export class AppComponent {
  ngOnInit() {
    fromEvent<StorageEvent>(window, 'storage')
      .pipe(
        // listen to our storage key
        filter(event => event.key === 'AkitaStores'),
        tap(event => {
          const stores = JSON.parse(event.newValue);
          snapshotManager.setStoresSnapshot(stores);
        })
      ).subscribe();
  }
}

Actually, it will cause an infinite loop. Let me think about it.

Awesome, thanks for super quick response.
I thought there may have been some sort of existing hook or workaround, if not no worries I'll start having a more in depth look.
Using the storage events is a good idea, at the least it's a perfect place to start.

    fromEvent<StorageEvent>(window, 'storage')
      .pipe(
        // listen to our storage key
        filter(event => event.key === 'AkitaStores'),
      ).subscribe(event => {
        const stores = JSON.parse(event.newValue);
        snapshotManager.setStoresSnapshot(stores, { skipStorageUpdate: true });
     });

You're a legend! Thanks for the quick response again. I've just given it a quick spin but it is working perfectly.

You're welcome. I'm wondering what a real-world use case for this functionality is?

Yeah it's a bit of weird one. Basically it's for multiple monitor support. I'm prototyping replacing a desktop system that has a heavily utilised GIS display in separate form so users can interact with it independently of the main form on a different monitor. Plenty of common state between the forms...easy for desktop apps running in the same process :). The persons list in the example is completely contrived, in reality it would be a store of geographic objects.

Hi, @NetanelBasal, your solution looks like a workaround. Why don't you want to make this behaviour as default?
I see that AkitaStores was changed from one window and of course it is similar in another window because of localStorage, but the event did not happen. This is not right I'm guessing. All subscriptions must work well!

authors: NetanelBasal foreground-voice nickcam

repo: datorama/akita


text:
feat(schematics): implement 'flat' flag for feature

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/datorama/akita/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Feature schematic doesn't have flat flag.
Issue Number: #393
What is the new behavior?
Feature schematic has flat flag.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

@NetanelBasal the build is failing, but it's not related to the changes in this PR.


text:
Hyperlink DOIs to preferred resolver

Hello :-)
The DOI foundation recommends this new resolver. It may be a bit ironic that they would change the URL of their service, but it's now encrypted. Because the old dx subdomain is being redirected, there is no urgent need to do anything.
However, I'd hereby like to suggest to follow the new recommendation by applying this update to all static DOI links and the code that generates new DOI links.
Cheers!

@davewripley ping ;-)

repo: davewripley/website


text:
Issues in debugging in chrome devtools

Hi,
Not sure where to ask this ask this.
I start with this starter kit, and now using it with React-DnD.
Right now I am facing this debugging related issue with some DnD related code, but I have experienced this elsewhere as well earlier while using this kit.
Consider this code:
  drop(props, monitor, component) {
    const item = monitor.getItem();
    const delta = monitor.getDifferenceFromInitialOffset();
    var test = monitor.getClientOffset();
    debugger;
    console.log(delta);
    console.log(test);
    return { campaignId: props.data.campaign.id, sceneId: props.data.scene.id };
  }

Lets say I am debugging in chrome devtools on the debugger breakpoint, and within console if I write monitor.getDifferenceFromInitialOffset() or monitor.getClientOffset() then I get null.
Whereas, if I inspect variables delta or test, then I can see the actual value. This not being able to run code in console to try out things is really hurting my dev process.
I am clueless as to why this is happening. I feel this is some kind of side effect of Babel's transpilation or maybe something that webpack does.
Has anyone else ever experienced such kind of issues?
If anyone knows what issue I might be experiencing, please let me know. Thanks.

I experienced a similar issue when using React DnD but it wasn't with this starter kit.  Whenever I called monitor.getItem() inside a debugger it returned undefined but worked fine when actually running.  Can't remember what I did that fixed it, have you tried using firefox yet to see if it works fine in firefox vs. chrome just to confirm it might not be the starter kit?

Closing this until there is evidence that this is related to this project (ty for the followup @SpencerCDixon). Unfortunately I don't have the time to attempt to reproduce this when it's not clear that it's related to this project, but if you figure out that it is in some way specific to this project then I'd be more than happy to re-open and dig into it.

Update: I tried in firefox and do not face this issue. So the issue is most likely not related with this starter kit. Thanks.

authors: SpencerCDixon davezuko shobhitg

repo: davezuko/react-redux-starter-kit


text:
No such file or directory: './data/processed/IEMOCAP/MFCC12EDA.csv'

Hi david
I've run the following code but it don't create MFCC12EDA.csv file

Thanks you for your help

Hi,
It seems that the OPENSmile software is not properly installed in your environments, thus, the our_file was not generated.
Just before the cell (shown in the figure), there exists a part to set the path of the installed OPENSmile. Please check whether it is properly set.
Form the previous experience (previous issues reported in the repository), it is quite hard for me to give a guide for the software installation due to the differences in the system environments.
If the preprocessed corpus will help your investigation, please drop me an email with the IEMOCAP license. I'll provide you with the preprocessed corpus for the experiments!
(mail: mysmilesh@gmail.com)
Thank you,
David

Hii do any one have IEMOCAP database to download .. i have seen under a website but it gives large files including others .. so do any have IEMOCAP alone .. please do send me the link.

Hi,
You may ask the IEMOCAP team for the original dataset.
https://sail.usc.edu/iemocap/

thank you!!!

repo: david-yoon/multimodal-speech-emotion


text:
v5.0.0-b4: Inconsistency detected issue with showAllHeaders() method.

Hi @davideas
Thanks for great lib.
I am using FlexibleAdapter to display sticky headers in RecyclerView. It is working fine for the first time loading list data using adapter, however, I am getting below crash issue while pushing fresh list items in adapter for the second time onwards.
Below is my code.
Code in onCreateView() method:
mRecyclerView = (RecyclerView) view.findViewById(R.id.recycler_view);
        mRecyclerView.setLayoutManager(new SmoothScrollLinearLayoutManager(getActivity(), LinearLayoutManager.VERTICAL, false));

        //mRecyclerView.setHasFixedSize(true); //Size of RV will not change
        mRecyclerView.setItemAnimator(new DefaultItemAnimator() {
            @Override
            public boolean canReuseUpdatedViewHolder(RecyclerView.ViewHolder viewHolder) {
                //NOTE: This allows to receive Payload objects on notifyItemChanged called by the Adapter!!!
                return true;
            }
        });
        mRecyclerView.addItemDecoration(new DividerItemDecoration(getContext(), R.drawable.divider));

code in refreshData() method:(which will be called everytime when user want to refresh data)
ArrayList<AbstractFlexibleItem> items = new ArrayList<AbstractFlexibleItem>();
.....
......
add AbstractFlexibleItem objects(headers and simple items) to items.
....
.......
mAdapter = new FeedAdapter(items, FeedFragment.this);
mAdapter.setRemoveOrphanHeaders(false);
mRecyclerView.setAdapter(mAdapter);
mAdapter.showAllHeaders();
mAdapter.enableStickyHeaders(3);

Note: FeedAdapter extends FlexibleAdapter
I am not observing the crash issue while commenting showAllHeaders() method call, however, no headers are showing due to this.
Can  you please help me to get out of this issue?

E/AndroidRuntime: FATAL EXCEPTION: main
Process: com.example.code, PID: 23833
java.lang.IndexOutOfBoundsException: Inconsistency detected. Invalid item position 15(offset:21).state:21
  at android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:4405)
  at android.support.v7.widget.RecyclerView$Recycler.getViewForPosition(RecyclerView.java:4363)
  at android.support.v7.widget.LinearLayoutManager$LayoutState.next(LinearLayoutManager.java:1961)
  at android.support.v7.widget.LinearLayoutManager.layoutChunk(LinearLayoutManager.java:1370)
  at android.support.v7.widget.LinearLayoutManager.fill(LinearLayoutManager.java:1333)
  at android.support.v7.widget.LinearLayoutManager.onLayoutChildren(LinearLayoutManager.java:562)
  at android.support.v7.widget.RecyclerView.dispatchLayout(RecyclerView.java:2864)
  at android.support.v7.widget.RecyclerView.onLayout(RecyclerView.java:3071)
  at android.view.View.layout(View.java:15140)
  at android.view.ViewGroup.layout(ViewGroup.java:4866)
  at android.widget.FrameLayout.layoutChildren(FrameLayout.java:515)
  at android.widget.FrameLayout.onLayout(FrameLayout.java:450)
  at android.view.View.layout(View.java:15140)
  at android.view.ViewGroup.layout(ViewGroup.java:4866)
  at android.support.v4.view.ViewPager.onLayout(ViewPager.java:1627)
  at android.view.View.layout(View.java:15140)
  at android.view.ViewGroup.layout(ViewGroup.java:4866)
  at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1888)
  at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1742)
  at android.widget.LinearLayout.onLayout(LinearLayout.java:1651)
  at android.view.View.layout(View.java:15140)
  at android.view.ViewGroup.layout(ViewGroup.java:4866)
  at android.widget.FrameLayout.layoutChildren(FrameLayout.java:515)
  at android.widget.FrameLayout.onLayout(FrameLayout.java:450)
  at android.view.View.layout(View.java:15140)
  at android.view.ViewGroup.layout(ViewGroup.java:4866)
  at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1888)
  at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1742)
  at android.widget.LinearLayout.onLayout(LinearLayout.java:1651)
  at android.view.View.layout(View.java:15140)
  at android.view.ViewGroup.layout(ViewGroup.java:4866)
  at android.widget.FrameLayout.layoutChildren(FrameLayout.java:515)
  at android.widget.FrameLayout.onLayout(FrameLayout.java:450)
  at android.view.View.layout(View.java:15140)
  at android.view.ViewGroup.layout(ViewGroup.java:4866)
  at android.widget.LinearLayout.setChildFrame(LinearLayout.java:1888)
  at android.widget.LinearLayout.layoutVertical(LinearLayout.java:1742)
  at android.widget.LinearLayout.onLayout(LinearLayout.java:1651)
  at android.view.View.layout(View.java:15140)
  at android.view.ViewGroup.layout(ViewGroup.java:4866)
  at android.widget.FrameLayout.layoutChildren(FrameLayout.java:515)
  at android.widget.FrameLayout.onLayout(FrameLayout.java:450)
  at android.view.View.layout(View.java:15140)
  at android.view.ViewGroup.layout(ViewGroup.java:4866)
  at android.view.ViewRootImpl.performLayout(ViewRootImpl.java:2342)
  at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:2048)
  at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:1211)
  at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:6282)
  at android.view.Choreographer$CallbackRecord.run(Choreographer.java:788)
  at android.view.Choreographer.doCallbacks(Choreographer.java:591)
  at android.view.Choreographer.doFrame(Choreographer.java:560)
  at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:774)
  at android.os.Handler.handleCallback(Handler.java:808)
  at android.os.Handler.dispatchMessage(Handler.java:103)
  at android.os.Looper.loop(Looper.java:193)
  at android.app.ActivityThread.main(ActivityThread.java:5341)
  at java.lang.reflect.Method.invokeNative(Native Method)
  at java.lang.reflect.Method.invoke(Method.java:515)
  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:825)
  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:641)
E/NativeCrypto: ssl=0x60990e98 cert_verify_callback x509_store_ctx=0x61259940 arg=0x0
E/NativeCrypto: ssl=0x60990e98 cert_verify_callback calling verifyCertificateChain authMethod=ECDHE_RSA

Hi @noundla, in the refreshData() you shouldn't recreate the adapter and so reinitialize the RV.
Look my example app (dev branch) and you will discover that if you simply call updateDataSet(items) the list will be refreshed cleaner, also the headers will be automatically re-displayed if before they were:
public void updateDataSet(List<T> items) {
	mItems = items;
	notifyDataSetChanged();
	showAllHeaders();
}

Try it by calling that method and let me know.

Thanks :) It worked. I will check out dev branch for other things.

authors: davideas noundla

repo: davideas/FlexibleAdapter


text:
是否支持集群部署？

如题。

暂不支持集群部署

authors: davidfantasy xinjiayu

repo: davidfantasy/embedded-mqtt-broker


text:
Nested embla carousel - child carousel is causing parent carousel to scroll on drag

Hi,
This is a bit of a unique use case, but we need to nest a carousel within a carousel slide (we've essentially got a carousel of cards, and within each card is a carousel of images).
When we drag the slides for the inner carousel, then we don't want the event to bubble up to the parent carousel and therefore cause that to scroll too. However, it seems that attempting to drag the inner carousel also causes the parent to scroll.
I've created a minimal repro here: https://codesandbox.io/s/nested-embla-bvo96
Interestingly, it seems that events are successfully being prevented from propagation (clicking outside the inner carousel logs out "mouse down", but inside of it does), but the parent carousel still scrolls.
This could just be an implementation error on our side.
Great library, thanks!

Hi Lewis (@lmcq),
Thank you for your question. Embla Carousel has no official support for nesting carousels, but I’ll try an unofficial way to achieve what you want.
I’ll let you know when I’ve done so.
Kindly,
David

Great, thank you @davidcetinkaya!

Hello Lewis (@lmcq),
Just wanted to mention that the pressure on this lib is high right now (I'm the sole maintainer of this project), so I don't know when I will have the time to look into this. I thought it would be good to mention that so you can look for alternatives if you need this feature soon.
I hope you understand.

Ok, no problem - thanks for your response!

Hello Lewis (@lmcq),
I started looking into this yesterday and came to the conclusion that I don't have a clear picture of how this should work. For example, what should happen in this case:

If I drag the parent carousel and it's halfway to the next slide, transitioning
And I start dragging the nested carousel inside it

How should the parent carousel respond to this?
Kindly,
David

Hi Lewis (@lmcq),
So I finally got half an hour to create an example for you. Here's the CodeSandbox with nested Embla Carousels.
Please let me know if you're intending to check this out or not.
Best,
David

Hi David,
That works great! We'll definitely be looking at using this. Thanks for taking the time to creating this
Apologies for not getting back to you earlier
Thanks,
Lewis

Thank you for your response and supporting this project Lewis (@lmcq) 👍🏻. I’m closing this issue as resolved then. Let me know if you run into any issues with the implementation.
Enjoy,
David

See this gist for nested carousel using v8.

authors: davidcetinkaya davidjerleke lmcq

repo: davidjerleke/embla-carousel


text:
Convert project to typescript and implement test

Hello!
I'm thrilled to contribute to this project and enhance my Git skills while helping out. After reviewing the project, I had an idea to add TypeScript support. TypeScript brings type safety, which can catch bugs early in development and make the code more reliable. Additionally, I plan to add tests to cover various edge cases and ensure the project's robustness.
I would be more than happy to work on these features if they align with your project's goals. Please let me know your thoughts, and I'm here to assist you in any way I can. 😊

would love this

Sorry, for late response, I was bussy with my exams.
So, I am very excited to work on this, I ma starting the work from today and will open PR as soon as possible.
But, this is my first time collaborating on a project, so may need some guidance throughout the process, So, I expect support from your side.
Thank you :)

There's one thing i would like to ask.
While looking and testing the conversion functions like, HSVtoHSL or HexToHSB etc.. are returning array of floats instead of integers, For example:
console.log(HSVtoHSL([ 44, 99, 99 ])); // Should return [ 44, 98, 50 ] but returns [ 44, 98.02, 49.995 ],
So i was wondering if this is intended feature, or I should fix it to return array of integers instead of array of floats

I don't know if this behaviour is intended or not but i found that:
If you do: console.log(randomColor(hue: 'monochrome', count: 3)) you get range error my guess is beacuse hueRange for monochrome is set to null when populating the dictionary by the function loadColorBounds, correct me if I am wrong...
This can be easily fixed by setting hueRange for monochrome, I just don't know what range is for monochrome colors, according to me hue is same for monochrome colors... Again correct me if I am wrong
also if this is intended feature, we should implement check that, if hue is provided as monochrome, then we should not accept the count variable, I can add this check, just let me know which one isintended feature i will update according to that

I noticed that, the file published on npm is different then the file avaliabl in this repo, and due to this I was scratching me head for eternity, I was wndering why its not working in the cloned repo but works in test project where I installed the npm package ;_;
Please update github repo ASAP...

@blobitty Just asking... are you the owner of this repo or moderator or friend of the owner, like i found that, there are already type for this package and owner has not reviewed any issues/ PRs these days... So, is my effort worth, I will still continue the conversion even if its not worth it because I will learn a lot and I will still open the PR incase owner decides to accept it in future or whatever...

authors: aktselite blobitty

repo: davidmerfield/randomColor


text:
Feat: add include polyfill option

Changes

Add  the option for include as a polyfill option. This option only includes specified packages.
If no include option is given, keeps the current behavior, adding all packages.
If protocolImports is true, also adds node:[module], only to specified packages in include.
If both include and exclude options are given, the exclude option is ignored.
Also, adds a list of all available modules to README.md.

Fixes #24

@leticiavna Thank you so much for this PR! 🎉

authors: davidmyersdev leticiavna


text:
Grab flyout view bounds when rendering calculations are already done

Fixes #23

Perfect, thanks! Looks good. And this makes sense now that I see it. I was using Bounds previously, but the measuring was whack at that point. It was way too big or way to small depending on which Bounds I read.

authors: Depechie davidortinau

repo: davidortinau/Xappy


text:
Add bilateral clearing

Placeholder PR for incorporating bilateral clearing mechanism into the model.

Coverage decreased (-5.38%) to 80.0% when pulling 701c3f2b2780a447a9196cbabf93e6b99c5e9329 on add-bilateral-clearing into 35ec710c88966aead21fec163e5a3654bc738dd1 on master.

Coverage decreased (-4.43%) to 80.95% when pulling fd49254e8ba15e701ab41c6118767b5c06d265e3 on add-bilateral-clearing into 35ec710c88966aead21fec163e5a3654bc738dd1 on master.

Coverage decreased (-1.57%) to 83.81% when pulling 1bde450a216613c7888056e39ea2e11c28315b47 on add-bilateral-clearing into 35ec710c88966aead21fec163e5a3654bc738dd1 on master.

Coverage decreased (-1.56%) to 83.73% when pulling 491ccbe9f95d48029bab0a2390083abf67f451f3 on add-bilateral-clearing into 15677c245527034f5d3a862a36282b73c8060790 on master.

Coverage increased (+0.35%) to 85.65% when pulling 07412c0d70f1dca3e3d154c03b6532b69ee44506 on add-bilateral-clearing into 15677c245527034f5d3a862a36282b73c8060790 on master.

authors: coveralls davidrpugh

repo: davidrpugh/play-securities-exchange


text:
TypeError: Cannot read property "0" from undefined.

First, I would like to thank you for this tool.
Unfortunately, it is not working for me. I am cross-referencing my figures and I always get:
"TypeError: Cannot read property "0" from undefined."
I checked it out and I followed all of your instructions properly. I already use Paperpile in my manuscripts. Is that any known incompatibility?
Thank for your time.

There is no compatibility issue that I know of. Did you ever resolve this on your own?

Good morning.
I already moved to using Paperpile. I couldn't loose more time. I already
wrote 5 papers with that tool and that suits my needs.
Thank you for your late feedback, though.
Be safe during the pandemics.
Em ter., 28 de abr. de 2020 às 15:44, David Rowthorn <
notifications@github.com> escreveu:

There is no compatibility issue that I know of. Did you ever resolve this
on your own?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/davidrthorn/cross_reference/issues/34#issuecomment-620787443,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALRGOXFYBBVLOXJLCAAKQWLRO4PYRANCNFSM4HAJA33Q
.

--
Tito Aureliano
Geologist and Paleontologist
MSc student at the University of Campinas, Brazil
Research associate at the* Laboratório de Paleoecologia e Paleoicnologia, *
UFSCar*, São Carlos, Brazil
Research fellow at the Sociedade de História Natural*, SHN, Torres Vedras,
Portugal
CREA-SP: 5070236759

authors: PaleoTA davidrthorn


text:
ValueError when use compare.py

I tried both models '20170511-185253' and '20170511-185253' but got the same error
ValueError Cannot feed value of shape (1, 240, 240, 4) for Tensor 'pnet/input:0', which has shape '(?, ?, ?, 3)'

This is the iamge which caused the error

Its a png with RGBA (4 Channels) thus there is a mismatch, you can either convert all images to jpeg or sent channels=3 in the tf decode op

@AKSHAYUBHAT Thank you very much ! I changed images to jpg and it works well ! :)

authors: AKSHAYUBHAT Akemi-Homura

repo: davidsandberg/facenet


text:
Recognizing unknown faces?

I have read some issues here in facenet that to identify for unknown face is:
L2 distance > to your threshold (let us say 0.6)  = unknown face.
Is this the formula for L2 distance:
distance = np.linalg.norm(x, ord=None, axis=None, keepdims=False).
If the formula is wrong, can you direct me to the right one?
Or can I use compare.py as a reference?
Thanks guys. I hope you could help me

Hello @PHMark,
Do your solve Recognizing unknown faces issue . I'm also stuck on same issue

hello @menon92  Did you problem got solved , i am also in similar situation please help


text:
Issues Occured when add product into Ebay motors

Hi Please provide an example PHP code to upload product in Ebay Motors. Currently Product add working correctly in Ebay main site. I have Specified SiteID as 100 but Still not working. I could see the following errors.
**Warning: Invalid Payment Terms Payment Method is not supported on this site or the category you have selected. It has been removed from the listing.
Warning: PayPal settings removed. Your PayPal settings have been removed from the listing since you do not have a PayPal account or specified Deposit as none.
Warning: FullPaymentTime Defaulted Item was submitted with 7 days for DaysToFullPayment
Error: Duration Invalid. The duration "GTC" day(s) is not available for this listing type, or invalid for category "10156".
Error: Payment method is missing. You must choose at least one payment method.
Warning: Category 10156 requires custom item specifics. Category 10156 only accepts custom item specifics.
Error: Seller Provided Title Value is missing. Seller Provided Title Value is missing.
Warning: Return policy is not supported. Return policy is not supported for this category.**

hi, this doesn´t sound like a bug with the SDK, so you should consult the mailinglist, the official ebay forum or the ebay professional support. do your call with the api test tool before you do with the SDK, so you know what needs to be set.

authors: michabbb ssrgspdkt

repo: davidtsadler/ebay-sdk-php


text:
graph-tool patch no longer needed

The current version of graph-tool (2.33) no longer needs any patching, so maybe you should just tell users to upgrade.

You can also drop the "Cairo interface" issue from the README. Presumably, it was only mentioned because conda-forge's graph-tool package used to lack plotting support.  It is now supported in recent versions.

I updated the documentation, thanks for the input.

authors: count0 dawe stuarteberg

repo: dawe/schist


text:
DBInstance support host from Kubernetes secret/configmap reference

Example taken from here: https://github.com/db-operator/db-operator/blob/main/examples/instance-generic.yaml.
apiVersion: kinda.rocks/v1beta1
kind: DbInstance
metadata:
  name: "example-generic"
spec:
  adminSecretRef:
    Name: example-instance-admin-secret
    Namespace: << namespace of admin secret >>
  engine: postgres
  generic:
    host: << remote database server address >>
    port: 5432
---
apiVersion: v1
kind: Secret
metadata:
  name: example-instance-admin-secret
  namespace: << operator namespace >>
type: Opaque
data:
  user: cG9zdGdyZXM=
  password: c2VjdXJlcGFzc3dvcmQK

spec.generic.host is expected to be a string but our use case requires settings this from a Kubernetes Secret (of Configmap also fine). Can we copy the same logic for retrieving the admin credentials to the host field? This means the example becomes something like
apiVersion: kinda.rocks/v1beta1
kind: DbInstance
metadata:
 name: "example-generic"
spec:
 adminSecretRef:
   Name: example-instance-admin-secret
   Namespace: << namespace of admin secret >>
 engine: postgres
 generic:
   host:
     secretRef:
       name: example-instance-admin-secret
       namespace: <namespace>
   port: 5432
---
apiVersion: v1
kind: Secret
metadata:
 name: example-instance-admin-secret
 namespace: << operator namespace >>
type: Opaque
data:
 host: my-instance.rds.amazonaws.com
 user: cG9zdGdyZXM=
 password: c2VjdXJlcGFzc3dvcmQK

This is a bit tough. Because it's a breaking change, it will require a new api version.
Here is the WIP PR: https://github.com/db-operator/db-operator/pull/67
And the issue: https://github.com/db-operator/db-operator/issues/66
We are almost ready to be there, but there are some things that are not ready/designed yet, and some of them are big.
I would say, once we have these 3 ready/designed, we can have a new APi version.

https://github.com/db-operator/db-operator/pull/81 But I want to update everything, so we don't have logrus at all
https://github.com/db-operator/db-operator/issues/17 I need to go through all the postgres/mysel params and find out which of them can be configured via CRD, and update the db package
https://github.com/db-operator/db-operator/pull/87 It's ready, just waiting for a review from another dev

But as you see, two of them are not ready at all, and both of can be pretty complicated. So I can't promise that we have this feature soon.

What I can offer now. I can make the PR with a new API version buildable, and add this feature there. Then you'll be able to install this version using the dev image. But then we can't promise any kind of real support, because you'll be using a test version of the operator. It will take some time too, but not that long as real production implementation

Hi @allanger thanks for the quick response much appreciated! I haven't had time to go over the linked PRs in depth, but here is an idea that came to mind:
How about changing the type of host to either string or object using oneOf?  In code you can check if the value is a string which means the address was given or if it's an object (with the corresponding fields) with a reference to a secret. If we can get this to work no breaking change is required and it's more flexible.
From the CRD perspective this appears to be supported in Kubernetes:

for each field in an object and each item in an array which is specified within any of allOf, anyOf, oneOf or not, the schema also specifies the field/item outside of those logical junctors (compare example 1 and 2).
https://kubernetes.io/docs/tasks/extend-kubernetes/custom-resources/custom-resource-definitions/#specifying-a-structural-schema

@sjoukedv I haven't thought about it, thanks m. I will check it, and probably will be able to make it ready for release within a couple of weeks

Maybe I haven't spent enough time with that, but I don't think that oneOf is supported by controller-gen for crd generation.
But I can implement it this way without removing old fields, that it won't be a breaking change.
https://github.com/db-operator/db-operator/issues/96#issuecomment-1909811222

So, this is merged: https://github.com/db-operator/db-operator/pull/102
Will be released soon too

https://github.com/db-operator/db-operator/releases/tag/v2.4.0

authors: allanger sjoukedv

repo: db-operator/db-operator


text:
Indexing for query on data->>'created_at'

For the following query (PostgresSwitch4):
SELECT data
            FROM github.pulls
            WHERE (to_timestamp((data->>'created_at')::text, 'YYYY-MM-DD"T"HH24:MI:SS"Z"') >= NOW() - INTERVAL '3 months')
            LIMIT 10;

Why is it cast from text to timestamp and then back to text?
The problem is that this function is not immutable and cannot be indexed.
With the followingquery, an index on github.pulls ((data->>'created_at') asc) could be used
SELECT data
            FROM github.pulls
            WHERE data->>'created_at' >= to_char(NOW() - INTERVAL '3 months','YYYY-MM-DD"T"HH24:MI:SS"Z"')
            LIMIT 10;

I've spent very little time on SQL queries so far.
The main idea was to make 4 switches with different types of queries, including bad queries that run slow or even extremely slow, to find problems in monitoring or to show the impact of bad queries on performance.
Initially when I started the project, I wanted to compare MySQL, PG, MongoDB with the same data and queries. But then it turned into something more.
I plan to improve both Dataset and SQL switch queries in the future.
I'll tell you right away, Switch 1 and Switch 2 will be for good queries, Switch 3 and Switch 4 for bad, slow or no index.
I can also make some Switch to delete the index on power on and create it on power off.
There is a lot of room and flexibility for improvement.
You can add new tables and columns.
As a dataset I use GitHub data from API, right now only Repos and Pulls, but in the future I will add other data like Issues, Comments, Tags or something like that.
By the way, you can change the Dataset type to GitHub, put as GitHub_ORG your organization, for demo_app_dataset service. Then the app will import your organization's repository data into the dataset. But you also need to set GITHUB_API for using it

I'll try to add this enhancement in an upcoming release.
SELECT data
FROM github.pulls
WHERE data->>'created_at' >= to_char(NOW() - INTERVAL '3 months','YYYY-MM-DD"T"HH24:MI:SS"Z"')
LIMIT 10;
I would appreciate your other suggestions on how to improve SQL queries that generate load.

authors: FranckPachot dbazhenov

repo: dbazhenov/github-stat


text:
Some Postgres relationships don't show on table focussed ER diagrams

Version 4.0.x (first noticed in the last 3.8.x release)
For Postgres connections only the main schema ERD shows all relationships. When selecting a table, the ERD only shows relationships owned by the table (for MySQL the views work as expected showing both types):
Some Postgres examples:
For the main database ERD view, the files_file table has many outgoing relationships:

when viewed from files_file table focus, no connections are shown.

Another example where the main view shows 2 connections on the actions_email_attatchments table:

When viewed from the actions_email_attatchments table view  the connections are shown:

Finally a mixed example where both types of FKs are on the table:

When this table is viewed in focus only the owned FKs are shown:

For comparison, here is an example from MySQL working as expected in a similar context. The main ERD shows all connections:

And in the table focussed ERD the connections and associated tables still show up:

Can't reproduce on my test database.
Maybe problem is in column types or in specific fk definition.
Could you post sample DDL? Here or on serge@jkiss.org

Thanks you for looking at this.
I reduced the DDL down to 4 tables and on my install it still shows.
Postgres version: psql (PostgreSQL) 9.6.2
The DDL I used for the screenshot in this comment is:
DROP TABLE IF EXISTS dbeaver_test.import_dependant_imports;
DROP TABLE IF EXISTS dbeaver_test.import;
DROP TABLE IF EXISTS dbeaver_test.users_user;
DROP TABLE IF EXISTS dbeaver_test.file;

CREATE TABLE dbeaver_test.file (
	id int4 NOT NULL,
	CONSTRAINT file_pkey PRIMARY KEY (id)
);

CREATE TABLE dbeaver_test.users_user (
    id int4 NOT NULL,
	CONSTRAINT users_user_pkey PRIMARY KEY (id)
);

CREATE TABLE dbeaver_test.import (
	id int4 NOT NULL,
	created_by_id int4 NULL,
	import_file_id int4 NULL,
	report_file_id int4 NULL,
	CONSTRAINT import_pkey PRIMARY KEY (id),
	CONSTRAINT import_created_by_id_e72bfbc5_fk_users_user_id
	  FOREIGN KEY (created_by_id)
	    REFERENCES dbeaver_test.users_user(id),
	CONSTRAINT import_import_file_id_830b6c08_fk_file_id
	  FOREIGN KEY (import_file_id)
	    REFERENCES dbeaver_test.file(id)
);

CREATE TABLE dbeaver_test.import_dependant_imports (
	id int4 NOT NULL ,
	from_import_id int4 NOT NULL,
	to_import_id int4 NOT NULL,
    CONSTRAINT import_dependant_imports_pkey
      PRIMARY KEY (id),
    CONSTRAINT import_depen_to_import_id_a3a6136d_fk_import_id
      FOREIGN KEY (to_import_id)
        REFERENCES dbeaver_test.import(id)
);

Result:

FYI: I confirmed the same behaviour on DBeaver 3.7.8  and DBeaver 3.5.1 so it does not seem like a recent regression in DBeaver. Possibly this behaviour is brought about by a Postgres version 9.6.x upgrade.
(BTW: Going back to the older versions on my machine, I do realise now how much amazing effort has gone into the DBeaver project, and that I have been clicking "update" every time a new version is available without worrying about the new version breaking things. Your project is indeed a great example of KISS continuous improvement. DBeaver "moves fast" with out the "break things" part 👍. Thanks!)

Thanks for detailed report.
I did find the bug (it was in schema constraints/indexes caching function), it'll be fixed in the next version.

I have a similar (maybe worse?) issue with DBeaver 21.0.2.202104042040 and postgres-db, but not with an oracle-db.
In postgres, if I click a table from the tree, the ER-Diagram shows ONLY that table and NO foreign key relationships, even ones owned by the table.  But if I click the schema, all tables with fk relationships are shown.  E.g. see images below.
In oracle, if I click a table from the tree, the ER-Diagram shows that table in the center with all fk relationships shown in the diagram.

authors: markk-fmi rubenfilteris serge-rider

repo: dbeaver/dbeaver


text:
DDL copy produces extra / invisible characters

System information:

Operating system (distribution) and version
DBeaver version
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
When copying the DDL from a view, the text contains extra blocks of blank/invisible characters. When formatting this pasted code, it won't format correctly and takes up several more lines than necessary.
Here is pasted DDL from a view as seen in LibreOffice Writer with 'toggle formatting marks' turned on:

And here is the SQL as DBeaver is trying to format it, but is out of line because of the extra characters:

Steps to reproduce, if exist:

with an existing view / materialized view, copy the DDL using the standard method (right click > generate SQL > DDL)
past into a new DBeaver SQL editor window
use the code formatter

Hello @dpsspatial,
Your text editor displays these invisible characters. I can see only paragraph symbols out there. They shouldn't cause any formatting problems.
Can you please provide your formatting settings? They can be found in the preferences: Editors ⇒ SQL Editor ⇒ Formatting.

Hi @ShadelessFox,
Here's my SQL formatting settings, which otherwise work just fine - the extra spaces I'm reporting seem to conflict with these settings producing the result above:

Thanks!

The result is the same because your query was already formatted.
Don't be confused by this selection. It's typical behavior for all text editors. The paragraph (line break) symbol does not have any length, but visually it "breaks" the line. That's why you see this weird selection.

This kind of block of text doesn't show up in SQL that I've created from scratch or loaded from a saved file. It only shows up when I copy the DDL of a view/materialized view and paste it into an editor. Normal SQL formats just fine, with the settings I have in the formatter. The pasted DDL SQL is the issue.

Let's make sure that we are talking about the same thing.
Is that the block of text you have mentioned earlier?
https://user-images.githubusercontent.com/35821147/158586740-be750287-ebaa-4bf9-b47c-b3d08e35be63.mp4

Can you please send me a screenshot of the General ⇒ Workspace preference page and a file c containing the exported view definition if it doesn't contain any sensitive data? Please note that It must be a file. Pasting it here will not work. To achieve it, you can use Export SQL script action from the context menu.

In your animation, while you're selecting lines that are empty and likely a single ' '... hopefully the script I attached can demonstrate the issue. Keep in mind that if you aren't formatting with the 'insert line feed before comma' option checked, you won't notice the issue... my default formatting method is this way so I never have to look for a comma at the end of a line in a select, etc.
GitHub_Issue_Sample.sql.zip
And here's the Workspace window:

I wanted to ensure that line delimiters are the same as shown in preferences.
Let's go the other way. Are you sure that formatting any other snippet results in a different output? Something like this should do the trick:
select
    1,
    2,
    3,
    4;

If not, maybe the behavior was changed in the previous versions of DBeaver?

Hello @dpsspatial,
Sorry for the very late response.
To get rid of excess lines just disable the insert line feed before comma option.
This is expected behavior, so I'm not sure what the issue is. Please let me know if you have any questions left.

authors: ShadelessFox dpsspatial


text:
How to install and use this in arm based linux

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

none of the version working in arm based linux

There have been no updates on the issue for a long time.
If it is still actual for you - feel free to reopen it with additional info.


text:
ER diagram shows simple table even if table is built with multiple joins

Description
ER diagram shows simple table even if table is built with multiple joins.
I created a table which has multiple joins conditions and transformations.  but when I looked at ER diagram, it shows one flat table.  How can I get ER diagram to show all joins with conditions.
DBeaver Version
23.0.5
Operating System
Windows
Database and driver
Postgres 14.0
Steps to reproduce
I created a table which has multiple joins conditions and transformations.  but when I looked at ER diagram, it shows one flat table
Additional context
No response

Hello @jgrover21
Can you please provide footage showing your issue? And the object DDL. Thank you.

Does your query contain subqueries?

yes
Respectfully,
Jyoti Grover
From: Elizabeth @.>
Sent: Thursday, May 25, 2023 7:23 AM
To: dbeaver/dbeaver @.>
Cc: Grover, Jyoti @.>; Mention @.>
Subject:  Re: [dbeaver/dbeaver] ER diagram shows simple table even if table is built with multiple joins (Issue #20087)
Does your query contain subqueries?
—
Reply to this email directly, view it on GitHubhttps://github.com/dbeaver/dbeaver/issues/20087#issuecomment-1562733581, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A54CVIFJJ7ZTWKM4IBOEML3XH46K7ANCNFSM6AAAAAAYNZVDH4.
You are receiving this because you were mentioned.Message ID: @.***>

CONFIDENTIALITY NOTICE: This email message and any attachments are confidential and are meant to be read by the intended recipient only. If you are not the intended recipient, please notify sender immediately and destroy all copies of this message and any attachments without reading or disclosing their contents. Thank you

Yes, it has subqueries, it has joins, transformations etc.

Yes, it is nested query like in this one.
How can we get ER diagram for joins?
Respectfully,
Jyoti Grover
From: Elizabeth @.>
Sent: Thursday, May 25, 2023 10:29 AM
To: dbeaver/dbeaver @.>
Cc: Grover, Jyoti @.>; Mention @.>
Subject:  Re: [dbeaver/dbeaver] ER diagram shows simple table even if table is built with multiple joins (Issue #20087)
It sounds like a duplicate of #4790https://github.com/dbeaver/dbeaver/issues/4790
—
Reply to this email directly, view it on GitHubhttps://github.com/dbeaver/dbeaver/issues/20087#issuecomment-1563010875, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A54CVICPTFTC63YBJPUN4D3XH5UDTANCNFSM6AAAAAAYNZVDH4.
You are receiving this because you were mentioned.Message ID: @.***>

CONFIDENTIALITY NOTICE: This email message and any attachments are confidential and are meant to be read by the intended recipient only. If you are not the intended recipient, please notify sender immediately and destroy all copies of this message and any attachments without reading or disclosing their contents. Thank you

@jgrover21 If you're talking about Query Builder, it works for simple joins without nested tables.
I'm closing this issue as a duplicate of #4790
If you have any questions left, you can ask here. Also if you're using our paid product and have an appropriate license, consider creating your ticket on our website: https://dbeaver.com/submit-ticket/. You might get a quicker response there.

Oh..you mean create ER diagram first and get DDL.  I noticed that one.
I was looking for other way around.  Creating ER diagram from a query or DDL. Which I donot think is there.
That would be a useful feature.
Respectfully,
Jyoti Grover
From: Elizabeth @.>
Sent: Tuesday, May 30, 2023 1:01 PM
To: dbeaver/dbeaver @.>
Cc: Grover, Jyoti @.>; Mention @.>
Subject:  Re: [dbeaver/dbeaver] ER diagram shows simple table even if table is built with multiple joins (Issue #20087)
That's a brief example that I can provide.
https://github.com/dbeaver/dbeaver/assets/28875055/d4c0db4c-829e-4793-a597-c9e5bbfd0304https://github.com/dbeaver/dbeaver/assets/28875055/d4c0db4c-829e-4793-a597-c9e5bbfd0304
You may find documentation of how to use Visual Query Builder here https://dbeaver.com/docs/wiki/Visual-Query-Builder/https://dbeaver.com/docs/wiki/Visual-Query-Builder/
—
Reply to this email directly, view it on GitHubhttps://github.com/dbeaver/dbeaver/issues/20087#issuecomment-1568773123, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A54CVIDV6XK52KCKKMVB6RDXIYRUVANCNFSM6AAAAAAYNZVDH4.
You are receiving this because you were mentioned.Message ID: @.***>

CONFIDENTIALITY NOTICE: This email message and any attachments are confidential and are meant to be read by the intended recipient only. If you are not the intended recipient, please notify sender immediately and destroy all copies of this message and any attachments without reading or disclosing their contents. Thank you

I wrote the query first. Then I can idit it in gui with visual query builder or by modifying the query in the editor and switching to Visual Query Builder to see how the query I wrote looks like.