Why can't I combine circuits with distinct register objects but colliding register names?

Is there a reason that I cannot do the following, a very common workflow within the algos and applications space?
qc = my_function_producing_a_circuit(args)
qc.extend(my_function_producing_a_circuit(args))

The logic preventing this is here, barring me from combining two circuits with a matching register name. Almost by definition, a function which gives me a subcircuit for use in an algorithm will reuse the register names for each returned circuit. I'm forced to hack around this by creating dummy wrapper circuits (as is done in Aqua here), which is breakage prone and tightly coupled to the Terra API. A short list of examples where I'd like to use a circuit requested from a generating function more than once:

Constructing any variational circuit
Constructing any qkernel
Constructing any phase estimation circuit

Is there a specific error behavior in mind that drove this decision?
What is the expected enhancement?
Accelerate algos and applications work.

I think this is related but if I make two circuits
    qc1 = QuantumCircuit(2,2, name="bell")
    qc1.h(0)
    qc1.cx(0, 1)
    qc1.measure([0,1], [0,1])
    qc2 = QuantumCircuit(4, 4, name="superposition")
    qc2.h([0,1,2,3])
    qc2.measure([0,1,2,3], [0,1,2,3])

and try to run them together I get the following error
There was an error in the circuit!. Error = 'duplicate register q'

@dongreenberg To use subcircuits, use inst = subcircuit.to_instruction() to convert them to instructions and then stick them in the circuit with circuit.append(inst, [0, 1, 6]).
I don't know what your function my_function_producing_a_circuit() does, in the title it says "colliding register names" and in the description it says " matching register name". The logic you point to is checking register objects, not register names. The rationale is that it is not correct for circ1 to append to circ2 if they act on different qubits (registers). That's why to_instruction() was introduced -- it anonymizes circuits, meaning the register they act on becomes unimportant.
@jaygambetta i can't reproduce your error.

@ajavadia change the line https://github.com/Qiskit/qiskit-terra/blob/3a5780b0fe6838acc9b249e781d23d8559e0c3a2/examples/python/using_qiskit_terra_level_2.py#L54 to have (2,2) and it will error

@jaygambetta I was able to reproduce this now, and opened up #2558 with more details.
I think this can be closed now, @dongreenberg feel free to reopen if you are not convinced.


Fix mypy errors

TL;DR
Fixed all the mypy errors
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
^^^
Tracking Issue
Fixes https://github.com/flyteorg/flyte/issues/1030
Follow-up issue
NA

@eapolinario let's not include this in 1.2.x


Separate javadoc build, preinstall fontconfig for PlantUML

Perhaps we should skip checkstyle and stuff in matrix and do all that in the new job?

Replaced by PR #62.


python_version < "3.10"

e.g.
pypiDataRev="eec10b9df52f10d08cc5214a37d20949018e164d" ;  pypiDataSha256="03zgzszjbmdpyzia8c2x1jnyasiqy464rrjcvjznnzxlbk1gjrwg" ;  ## commit: master # 2022-04-16T08:19:14Z # DavHau/pypi-deps-db
Requirement already satisfied: pip>=6.0.0 in /nix/store/zfmcr0vrcrffhl82h0l3l1ndlkpdb3mp-python3.8-pip-21.1.3/lib/python3.8/site-packages (from pipdeptree==2.2.1) (21.1.3)
/nix/store/407v0jslqsrbg6swkxrlb1s1l5yiwcz8-python3-3.8.13/lib/python3.8/distutils/dist.py:274: UserWarning: Unknown distribution option: 'cffi_modules'
  warnings.warn(msg)
Requirement already satisfied: traitlets in /nix/store/rr80sqmc2sq21zy9bvhvv2jjz6z8mzxh-python3.8-traitlets-5.1.1/lib/python3.8/site-packages (from jupyter-core==4.9.2) (5.1.1)
Processing ./jupyterlab_widgets-1.1.0-py3-none-any.whl
ERROR: Could not find a version that satisfies the requirement zipp>=3.1.0; python_version < "3.10" (from importlib-resources) (from versions: none)
ERROR: No matching distribution found for zipp>=3.1.0; python_version < "3.10"

looks like python_version < "3.10" is not processed correctly in mach-nix

ipython / black is a package with the same issue


move xref from malignant glioma

create new class MONDO:0100342 malignant glioma and obsolete existing class
close #2932

closing, recreated these changes on a new branch


refactor: start on a functional domain design implementation for ontologies (DEV-227)

resolves DEV-227

Cool, I'm looking forward to working on this.
Is there a reason why everything is equally placed on the top level of the folder structure? I would create one folder for all the new code and inside it one folder for each compartment (project, schema, resource, list,...). But perhaps I don't overlook all the consequences.

for reasons of visibility. the modules are grouped by name. The webapi folder should at some point be renamed to something more fitting. take a look at something other larger scala repos:

akka: https://github.com/akka/akka
zio: https://github.com/zio/zio
zio-http: https://github.com/dream11/zio-http


[BUG] Test with document in database fails to recognize datatype

Expected behavior
Test should find document (eXist-db book p. 426),
Actual behavior
Reports that the resource is not a document node
Reproduction steps
Test is part of XQSuite tests for functions.xqm
Please provide the following

Java Version:
exist-db version:
pr-app version:
OS version:

Posted inquiry to eXist-db Slack on 2022-02-22, initial responses from Joe and Duncan, posted follow-up on 2022-02-23.

Joe confirms in the eXist-db Slack that it is not possible to test functions with non-atomic parameters (nodes) by passing the parameter values in with %test.args. He writes:

Re: your question, no, given the limitation of %test:args, you would need to perform the test in a separate function. I agree it would be nice to be able to supply tests and expected results in the function that does the important work. But, in fact, I find that setting up a companion function (or even a standalone module) just for testing actually allows you write better tests than if you mixed the function and its tests in a single function. For example, instead of returning just the items that your application needs, your test can examine and analyze the return values in more detail.
For an example, see the semver.xq repository - where the core functionality resides in a library module, and where the tests reside in a parallel folder and are organized around key aspects of the library. The tests get at many different aspects of the functions, and not simply whether they produce one expected result.

@gabikeane I'm not certain that the XQSuite reality, if I have now understood it correctly, is very different from the XSpec reality that we accept without question. In particular, in XSpec the tests also reside separately from the functions, but they differ from the XQSuite ones in that they supply parameters for testing the functions in a more flexible way (not restricted to atomic datatypes) than the sort of companion functions that Joe describes.

@gabikeane Separate test suite is working correctly, and able to test functions that require document nodes as input.
Close after review?

@gabikeane Test writes xml results to file system, where they can be transformed to html on the command line with a script. The file:serialize() function requires an absolute path; not sure how to work around that so that saving the results will work in different locations on different machines. If we can't fix it, we should trap the error.


Need to create special smoke tests to check that multiple rules will work together

Currently we don't have any smoke test framework that will use different rules and check them together.
Can be a duplicate of some issues - close if we have already created this one

Duplicate of #50 :)


Move build out of src folder

I think build in src folder is not a good practice. When searching in src folder, a lot of build results come out. So why not move it out?

Please file an issue for discussion. Many thanks for raising the problem!

/cc @chrisgarrity


Upgraded Rails to 4.1.12 and Ruby to 2.2.2. Upgraded gems by rebuilding Gemfile.lock file.

Upgraded gems by rebuilding Gemfile.lock file. See diff for specifics.
Also upgraded Rails to 4.1.12 and Ruby to 2.2.2.

Upgraded from rc1 to final 4.1.12 release. FYI.


vet: prune lockfile

When running cargo vet in #7900, it warned me that we should consider pruning some unused entries, etc. This is the result of running cargo vet prune.

Oh I think this has unfortunately undone the work of https://github.com/bytecodealliance/wasmtime/pull/7900 so we'll need to reland that for https://github.com/bytecodealliance/wasmtime/pull/7807

Should we revert this one?


No such file or directory

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.05.18. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.05.18

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones
[x ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ x] Question
[x] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.05.18
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
Hello.
I am a new Linux (Lubuntu) user, and I tryed youtube-dl for the 1st time to extract audio from this file:
LMFAO - Sexy and I Know It-wyx6JDQCslE.mkv
Everything run well during the download, so, after it finished I wrote:
ffmpeg -i wyx6JDQCslE sexy.mp3
and the result, no matter what I do, is always:
ffmpeg version 3.3.4-2 Copyright (c) 2000-2017 the FFmpeg developers
built with gcc 7 (Ubuntu 7.2.0-8ubuntu2)
configuration: --prefix=/usr --extra-version=2 --toolchain=hardened --libdir=/usr/lib/i386-linux-gnu --incdir=/usr/include/i386-linux-gnu --enable-gpl --disable-stripping --enable-avresample --enable-avisynth --enable-gnutls --enable-ladspa --enable-libass --enable-libbluray --enable-libbs2b --enable-libcaca --enable-libcdio --enable-libflite --enable-libfontconfig --enable-libfreetype --enable-libfribidi --enable-libgme --enable-libgsm --enable-libmp3lame --enable-libopenjpeg --enable-libopenmpt --enable-libopus --enable-libpulse --enable-librubberband --enable-libshine --enable-libsnappy --enable-libsoxr --enable-libspeex --enable-libssh --enable-libtheora --enable-libtwolame --enable-libvorbis --enable-libvpx --enable-libwavpack --enable-libwebp --enable-libx265 --enable-libxvid --enable-libzmq --enable-libzvbi --enable-omx --enable-openal --enable-opengl --enable-sdl2 --enable-libdc1394 --enable-libiec61883 --enable-chromaprint --enable-frei0r --enable-libopencv --enable-libx264 --enable-shared
libavutil      55. 58.100 / 55. 58.100
libavcodec     57. 89.100 / 57. 89.100
libavformat    57. 71.100 / 57. 71.100
libavdevice    57.  6.100 / 57.  6.100
libavfilter     6. 82.100 /  6. 82.100
libavresample   3.  5.  0 /  3.  5.  0
libswscale      4.  6.100 /  4.  6.100
libswresample   2.  7.100 /  2.  7.100
libpostproc    54.  5.100 / 54.  5.100
wyx6JDQCslE: No such file or directory
As you can see, I am not a expert using linux and console, so, I hope you can help...
Thanks
SirJams

Carefully read new issue template and provide all requested information.

PS- Downloaded the youtube file from:
https://www.youtube.com/watch?v=wyx6JDQCslE
Then, i wrote:
ffmpeg -i wyx6JDQCslE sexy.mp3
And the result is always:
ffmpeg version 3.3.4-2 Copyright (c) 2000-2017 the FFmpeg developers
built with gcc 7 (Ubuntu 7.2.0-8ubuntu2)
configuration: --prefix=/usr --extra-version=2 --toolchain=hardened --libdir=/usr/lib/i386-linux-gnu --incdir=/usr/include/i386-linux-gnu --enable-gpl --disable-stripping --enable-avresample --enable-avisynth --enable-gnutls --enable-ladspa --enable-libass --enable-libbluray --enable-libbs2b --enable-libcaca --enable-libcdio --enable-libflite --enable-libfontconfig --enable-libfreetype --enable-libfribidi --enable-libgme --enable-libgsm --enable-libmp3lame --enable-libopenjpeg --enable-libopenmpt --enable-libopus --enable-libpulse --enable-librubberband --enable-libshine --enable-libsnappy --enable-libsoxr --enable-libspeex --enable-libssh --enable-libtheora --enable-libtwolame --enable-libvorbis --enable-libvpx --enable-libwavpack --enable-libwebp --enable-libx265 --enable-libxvid --enable-libzmq --enable-libzvbi --enable-omx --enable-openal --enable-opengl --enable-sdl2 --enable-libdc1394 --enable-libiec61883 --enable-chromaprint --enable-frei0r --enable-libopencv --enable-libx264 --enable-shared
libavutil      55. 58.100 / 55. 58.100
libavcodec     57. 89.100 / 57. 89.100
libavformat    57. 71.100 / 57. 71.100
libavdevice    57.  6.100 / 57.  6.100
libavfilter     6. 82.100 /  6. 82.100
libavresample   3.  5.  0 /  3.  5.  0
libswscale      4.  6.100 /  4.  6.100
libswresample   2.  7.100 /  2.  7.100
libpostproc    54.  5.100 / 54.  5.100
wyx6JDQCslE: No such file or directory
Can you help?

Bother to read error message?


Initial Adoption of Continous Deployment using semantic-release

To ease the modification/contribution to publish pipeline there has been some discussion of adopting a continuous deployment system for AtomLinter packages. To accomplish this we are making use of semantic-release and @semantic-release/apm-config.
This pull request has been dynamically created using a script. While the result is not perfect, it does accompish most of the grunt work of adopting continous deployment. There is some reconciliation that needs to be done before this can be merged.
Among the things needed to be reconciled, an APM Token and GitHub Token will need to be added to this repo's continous integration (most likely Travis CI) configuration for automated deployments to work.
Again, this Pull Request has been created by a script made by @keplersj. Please mention him if something has gone wrong, and he'll be happy to help.
ref: AtomLinter/Meta#18
cc: @Arcanemagus

@AtomLinter/linter-swagger Any objections to the Conventional Commits requirement? There are alternative standards available if somebody has a strong objection to that standard, but we need to pick one to finish implementing this 😉.


Error event logging and alerts

Background
Currently our primary app does log some error events to mixpanel but from my understanding we haven't historically used them much for taking action such as knowing a bad release went out if error rate spikes.  While this would be a larger task to start getting better error reporting out of the main application, memoless presents an app with 1 critical flow that we might be able to experiment with some better error reporting and alerting so we aren't so reliant on users reporting bugs as the primary trigger to roll back a release.
Mixpanel also does offer custom alerts that could be monitored by operations.
Acceptance Criteria

add mix panel event triggers for event triggers or anticipated bad states to the new og app
determine how best to get decent information out of these events that help with debug-ability / traceability
create a mixpanel dashboard to track these errors
create an alert with the assistance of operations that can be monitored by them

The primary goal of this would be to move towards less reliance on users and being more proactive when a bad release goes out the door hopefully reducing the time it takes us to get a fix out (mttr type metric, which we don't actually track, yet)

review sentry vs bugsnag


Fix: Unnecessary Import of cv2

The opencv library was imported in the module but was not used.

bozo


ER: CSV/TSV/JSON output using console.sh

One of the reasons for the universal success of tools like awk,
SQLite, and R is that they support easy data-interoperability,
and I think that with little effort on the part of OrientDB developers,
OrientDB would gain a great deal of traction with a few enhancements.
In this ER, I'll focus just on the "output" side of things, i.e.
enhancements to support the use of console.sh (or console.bat) to produce output
in CSV, TSV, and JSON formats.
The first and most important step would be to support the functionality offered by sqlite3's commands:
.headers
.mode     # In particular, the csv, tabs, and list subcommands
.once     # Output to FILENAME for the next SQL command only
.output
.print
.separator

The second step would be to support JSON more comprehensively in the "console.sh" environment. To explain what I mean, consider the following typescript:
orientdb {db=demo}> select expand(out()) from #16:0

----+-----+------+----+--------------
#   |@RID |@CLASS|name|in_IsNeighbour
----+-----+------+----+--------------
0   |#16:1|Person|P2  |[#17:1]
1   |#16:2|Person|P3  |[#17:2]
----+-----+------+----+--------------

orientdb {db=demo}> select @this.toJSON() from (select expand(out()) from #16:0)

----+------+-----------------------------------------------------------------------------------------------------------------------------------
#   |@CLASS|this
----+------+-----------------------------------------------------------------------------------------------------------------------------------
0   |null  |{"@type":"d","@rid":"#16:1","@version":2,"@class":"Person","name":"P2","in_IsNeighbour":["#17:1"],"@fieldTypes":"in_IsNeighbour=g"}
1   |null  |{"@type":"d","@rid":"#16:2","@version":2,"@class":"Person","name":"P3","in_IsNeighbour":["#17:2"],"@fieldTypes":"in_IsNeighbour=g"}
----+------+-----------------------------------------------------------------------------------------------------------------------------------

orientdb {db=demo}> select @this["version"] from (select @this.toJSON() from (select expand(out()) from #16:0))

----+------
#   |@CLASS
----+------
0   |null
1   |null
----+------

(Using<EMAIL_ADDRESS>produces the same results.)
In short, it appears that .toJSON() returns a string, from which it is
not possible to extract information using the usual notational
devices.
Finally, I'd like to request that support for CSV and TSV output be provided with
the same level of attention to detail as exemplified by the @csv and @tsv filters in jq Version 1.5
These filters do a fantastic job of dealing with edge-cases (e.g. NUL).
See https://stedolan.github.io/jq/manual/

The cry for help (with notes of exasperation) at https://stackoverflow.com/questions/34279189/insert-50k-json-objects-into-database could be taken as an indicator of the urgency of this ER.  Or maybe I should say that by "easy data-interoperability" I did mean to imply that performant data-interoperability should also be easy to attain :-)

The SO thread, as I read it, is about performance and not necessarily about the missing ability of importing data formats.
Importing JSON, XML or CSV data is possible with ETL.
http://orientdb.com/docs/2.1/Extractor.html
On the exporting side, ODB only supports JSON output.
Scott

See also https://stackoverflow.com/questions/34446497/orientjs-how-to-get-standard-json-serialized-from-query

After some analysis, to allow a different output format we need to deeply redesign the console code.
About
select @this["version"] from (select @this.toJSON() from (select expand(out()) from #16:0))

this means embed a json parser inside the console itself to post-process the result set: non feasabile


Release v2.7.1

Closes #169
Closes #177
Closes #179
Closes #180
Closes #182

Pull Request Test Coverage Report for Build 789

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.865%

Totals

Change from base Build 774:
0.0%

Covered Lines:
103

Relevant Lines:
104

💛  - Coveralls


rubocops/cask: Clean up unnecessary requires

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

I suspect these were copy-pasted from other cops, like I did in https://github.com/Homebrew/brew/pull/14886#discussion_r1125569999.
The "forwardable" require is unnecesary if the cop doesn't extend Forwardable and use def_delegator.
The "uri" require is unnecessary if the cop doesn't call URI methods.

Review period will end on 2023-03-07 at 00:00:00 UTC.


[FEATURE]: Backup or Restore without assignments

Is your feature request related to a problem? Please describe.
It would be great to be able to exclude assignments in a back up or a restore/import, so that configurations can be imported into a tenant and assigned separately (e.g. manually)
Describe the solution you'd like
Add an option in IntuneCD-startbackup to exclude assignments in a backup or in IntuneCD-startupdate to not import assignments.
Describe alternatives you've considered
Manually editing the backup files, but that's not a lot of fun.

Hi Aaron,
Configurations should be able to import even though the group does not exist, but I do agree with you that it can be nice to skip exporting assignments all together sometimes.
I will look into adding a switch to skip exporting assignments when backing up configurations!

Thanks. The use case here is importing into unrelated tenants, particularly for consultants getting an Intune up and running from a baseline set of configs, in what might be an existing tenant with live endpoints.


Renamed plugin to @parodos/plugin-parodos

Motivation
Orion was a temporary name for the plugin. We want to use parodos everywhere.
Approach
Renamed everything to Parodos to make sure there is no Orion left.

@taras you need to run yarn prettier --write . from the root to fix the build


How to set language for non-packaged WinUI app?

I started creating an unpackaged WinUI 3 app that needs to support localization. Does anyone know what's the best way to set display language?
I have tried:
ApplicationLanguages.PrimaryLanguageOverride = cultureCode;
Not supported in unpackaged build
Windows.ApplicationModel.Resources.Core.ResourceContext.SetGlobalQualifierValue("Language", cultureCode);
This changes the language of resource context, and changes the strings from ResourceLoader. But it doesn't change the content in xaml from x:Uid attribute. Also, this call crashes when the app is running on Windows 11 with no exception.
Thread.CurrentThread.CurrentCulture = culture;
Thread.CurrentThread.CurrentUICulture = culture;
CultureInfo.DefaultThreadCurrentCulture = culture;
CultureInfo.DefaultThreadCurrentUICulture = culture;
Don't have any effect

https://github.com/microsoft/WindowsAppSDK/issues/1687


deps: upgrade aws/config and s3 packages

Hoping it will fix S3 credentials failure.
Related to #898
Suggested here: https://neondb.slack.com/archives/C039YKBRZB4/p1715340236979409

Helped, will do a v0.29.1


Plugin processes also binary files

It seems that (at least some) binary files get processed by the plugin, which is naturally not wanted: When enabled, the PNG images outcome invalid.
I'd suggest adding some kind of pattern support which files are to be processed or not: https://www.npmjs.com/package/multimatch

That's a good idea @juhamust.
As for me I met the same problem with some other plugins and used metalsmith-hideshow to solve that. I want to add pattern support there some day, but it's already useable.

now it is available in<EMAIL_ADDRESS>

Excellent! Thanks for the quick response time.


Issue with isValidated when used with Redux's @Connect

Hi I am using redux along with stepzilla, i need to save values at every step so I am using @connect with redux and isValidated() is not triggered , I tried to use @connect in Main component where we define steps and send required fields as props to each step but i am not able to get it done.
Please help me with this issue

@zeel would appreciate your help in solving this issue. Maybe create a new branch and an updated example in src/examples that uses redux and @connect... I can then help you find the solution...

This is related to #55

Duplicate of #55

@narendravenkata @zeel I has the same issue, and I saw that a lot of other people have this kind of issue as well.
I have seen several opened issues with the same topic without a proper solution. Is there some light at the end of the tunnel that we can hope for? Is there any hope that we can use redux and Stepzilla in the same project?
This is a great library and thanks for all the work you invested in.

Same issue with i18next package in next.js project.

Is this issue has a solution or not yet?


[Bug] Integer overflow

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
DBT transformation fails with an integer overflow most likely related to the cast to int where it could be dbt.type_bigint()?
https://github.com/fivetran/dbt_hubspot/blob/main/models/marketing/intermediate/int_hubspot__contact_merge_adjust.sql#L22
Relevant error log or model output
11:43:06
Database Error in model int_hubspot__contact_merge_adjust (models/marketing/intermediate/int_hubspot__contact_merge_adjust.sql)
11:43:06
Overflow (Integer valid range -2147483648 to<PHONE_NUMBER>)
11:43:06
compiled Code at target/run/hubspot/models/marketing/intermediate/int_hubspot__contact_merge_adjust.sql

Expected behavior
No error in sync
What database are you using dbt with?
redshift
dbt Version
1.5.1
Are you willing to open a PR to help address this issue?

[ ] Yes.
[ ] Yes, but I will need assistance and will schedule time during our office hours for guidance
[X] No.

hey there @stdiopt thanks for taking the time to open this issue!
i think you're certainly right that we'll need to cast as a bigint. i've made the change in a testing branch if you'd like to test it out:
# packages.yml -- replace hubspot reference with the below
packages:
  - git: https://github.com/fivetran/dbt_hubspot.git
    revision: bug/cast-vid_to_merge-bigint
    warn-unpinned: false

also i don't think this really changes things at all, but do you have the hubspot_contact_merge_audit_enabled variable set to true?
thanks!

Hey Jamie,

also i don't think this really changes things at all, but do you have the hubspot_contact_merge_audit_enabled variable set to true?

to be honest I'm not entirely sure since this is managed by fivetran and based on quickstart transformations, I just disabled those transformations now since I noticed that we aren't using it

Ah ok, the hubspot_contact_merge_audit_enabled  variable is false by default and I believe Quickstart wouldn't change it to true.
As for testing, since you're on Quickstart, we'll work to recreate the issue on our end, see if the cast as {{ dbt.type_bigint() }} does the trick, and prioritze the fix for an upcoming sprint 🤠

Hi @stdiopt , we have been able to recreate the error and find the fix on our end. We'll be including this fix in our upcoming sprint, so keep an eye out for a new release!

🙏 this is currently biting us -- any indication of when 0.17.1 might be released please?

Hi @EPNickBosma , thanks for the patience-- we've just merged the branch with fix, so expect the release within the next few hours!
cc @stdiopt


added support for cpp, cxx extensions and their header files (h, hpp, ...

sloc also supports these extensions: https://github.com/flosse/sloc/pull/50

Thanks.
On Wed, Mar 25, 2015 at 6:41 AM, Liamsi<EMAIL_ADDRESS>wrote:

sloc also supports these extensions: flosse/sloc#50
https://github.com/flosse/sloc/pull/50
You can view, comment on, or merge this pull request online at:
https://github.com/mark-hahn/line-count/pull/7
Commit Summary

added support for cpp, cxx extensions and their header files (h,
hpp, hxx)

File Changes

M lib/line-count.coffee
https://github.com/mark-hahn/line-count/pull/7/files#diff-0 (5)

Patch Links:

https://github.com/mark-hahn/line-count/pull/7.patch
https://github.com/mark-hahn/line-count/pull/7.diff

—
Reply to this email directly or view it on GitHub
https://github.com/mark-hahn/line-count/pull/7.

Released in 0.2.0

Thank you, too!
But this will only work properly for everyone if sloc builds a new releases which supports theses extensions, too packages.json in line-count gets updated to this (not yet released) version. Currently your addon will show "Warning: File extension 'cxx' is not supported". (Before that it would just ignore these files). I will create anoter pull request, when a new sloc version is released.

Ah, I just remembered I had to do a PR to sloc before.  What a pain.  I
wonder if I could patch sloc on the fly or replace it.
On Thu, Mar 26, 2015 at 2:49 AM, Liamsi<EMAIL_ADDRESS>wrote:

Thank you, too!
But this will only work properly for everyone if sloc builds a new
releases which supports theses extensions, too packages.json in line-count
gets updated to this (not yet released) version. Currently your addon will
show "Warning: File extension 'cxx' is not supported". (Before that it
would just ignore these files). I will create anoter pull request, when a
new sloc version is released.
—
Reply to this email directly or view it on GitHub
https://github.com/mark-hahn/line-count/pull/7#issuecomment-86427811.

You could, but as it does not really break anything (ok some warnings) I would just wait for the next release of sloc.


Add examples for sequence and traverse

It would be nice to have examples for all instances, but Array is a good start.

CI is failing because it's not pulling in the v0.14.0-rc3 purs binary. @milesfrain Could you rebase this on top of master?

@JordanMartinez Should be ready now

Thanks!


[Feature][DatePicker] test error

Describe the bug
Tests always fail due to date issues.
To Reproduce
Steps to reproduce the behavior:
npx jest DatePicker --coverage
Expected behavior
Mock date to specific day for test.
Screenshots
-----------------------|---------|----------|---------|---------|-------------------
File                   | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
-----------------------|---------|----------|---------|---------|-------------------
All files              |    91.3 |       75 |      75 |   91.46 |
 DatePicker            |   88.89 |    72.41 |      60 |    87.5 |
  DateInput.tsx        |   94.87 |    77.78 |   83.33 |   94.29 | 106,132
  PickerCalendar.tsx   |   90.91 |     62.5 |      60 |   90.48 | 49,87
  index.tsx            |      75 |    66.67 |      25 |   68.75 | 35-37,48-59
 DatePicker/Calendar   |    93.2 |    78.26 |   88.24 |   94.57 |
  Calendar.tsx         |   85.11 |       50 |   77.78 |   87.18 | 78,100-103
  CalendarDate.tsx     |     100 |    93.75 |     100 |     100 | 44
  CalendarMonth.tsx    |     100 |       75 |     100 |     100 | 65
  CalendarWeekDays.tsx |     100 |      100 |     100 |     100 |
-----------------------|---------|----------|---------|---------|-------------------
Snapshot Summary
 › 6 snapshots failed from 1 test suite. Inspect your code changes or re-run jest with `-u` to update them.

Desktop (please complete the following information):
X
Smartphone (please complete the following information):
X
Additional context
X

I think it will be solved by mocking as like
jest       .spyOn(Date, 'now')       .mockImplementation(() => new Date('2020-09-01).valueOf())

Because date value is used a lot in test cases, I set a standard date instead of using spyOn.

@oh3vci It is not solved yet. Snapshots change everyday.

https://github.com/facebook/jest/issues/2234

I think it's good to exclude snapshots from the Datepicker test.

@JeffGuKang How about we add today as a props for DatePicker? That would eliminate the need for mocking global.Date, and also allow us to use snapshot testing.

@JeffGuKang How about we add today as a props for DatePicker? That would eliminate the need for mocking global.Date, and also allow us to use snapshot testing.

But I couldn't control locale in dooboo-ui node environment for testing.
It seems to be dependent on the user's OS.
For example,
https://github.com/dooboolab/dooboo-ui/blob/29d45036764e2ab79566b13e6bb99a35d291d78b/main/DatePicker/Calendar/CalendarWeekDays.tsx#L37-L62
Weekdays would be 'Mon', 'Tue'... in en-EN. But in Korean OS, this appears to be '월', '화'...
And we cannot control locale manually as I said https://github.com/dooboolab/dooboo-ui/issues/337#issuecomment-692028016.

We can except validation from specific area of snapshots through matcher. It can be the one of the solutions.

@JeffGuKang I suggest we use react-intl package for internationalization instead of toLocaleString and add today props to DatePicker component. react-intl (from FormatJS) allows developers to wrap components with IntlProvider, so we can provide { locale: 'en' } for testing and provide user-specific locale for production.
Here's the relevant documentation : https://formatjs.io/docs/guides/testing#jest
Excerpt:
import React from 'react'
import renderer from 'react-test-renderer'
import {IntlProvider} from 'react-intl'

const createComponentWithIntl = (children, props = {locale: 'en'}) => {
  return renderer.create(<IntlProvider {...props}>{children}</IntlProvider>)
}

export default createComponentWithIntl

import React from 'react'
import createComponentWithIntl from '../../utils/createComponentWithIntl'
import AppMain from '../AppMain'

test('app main should be rendered', () => {
  const component = createComponentWithIntl(<AppMain />)

  let tree = component.toJSON()

  expect(tree).toMatchSnapshot()

  tree.props.onClick()

  tree = component.toJSON()

  expect(tree).toMatchSnapshot()
})

As adding locale prop to component, I implemented already this approach without react-intl. Can you check this? #347

@oh3vci It looks good to me. So I guess we just need to test DatePicker with locale and today props defined.
BTW do we have any timezone related issue with DatePicker snapshot testing?

I think timezone issue is already solved problem,
because I set en to the value of the locale prop that is the locale parameter of toLocaleString method in test code.
So the test snapshot will no longer be affected by the OS.

I think timezone issue is already solved problem,
because I set en to the value of the locale prop that is the locale parameter of toLocaleString method in test code.
So the test snapshot will no longer be affected by the OS.
Did you try locale props with 'kr-KR' or other timezones when you test?

@JeffGuKang I tried setting locale props to ko-KR in storybook. Korean characters shows up correctly on web, but it does not work on Android. The same thing happens for locale="es-ES".

@oh3vci Did you try locale props with 'kr-KR' or other timezones when you test? Did it work as like '월', '화'...?

Isn't it 'ko-KR', not 'kr-KR'?

@JeffGuKang I tried setting locale props to ko-KR in storybook. Korean characters show up correctly on web, but it does not work on Android. The same thing happens for locale="es-ES".

I guess it seems to be another bug of android or storybook. it works well in iOS.

@oh3vci How about testing environment? Testing is not work as expectation with locale control or mocking.

@JeffGuKang locale props works inside jest.
I tested this first,
<DatePicker selectedDate={standardDate} locale={'ko-KR'} />

then this,
<DatePicker selectedDate={standardDate} locale={'en-US'} />

Here's the snapshot diff:
- Snapshot  - 7
+ Received  + 7

@@ -247,11 +247,11 @@
                              "color": "#ff424c",
                            },
                          ]
                        }
                      >
-                       일
+                       S
                      </Text>
                    </View>
                    <View
                      style={
                        Array [
@@ -279,11 +279,11 @@
                              "color": "#565656",
                            },
                          ]
                        }
                      >
-                       월
+                       M
                      </Text>
                    </View>
                    <View
                      style={
                        Array [
@@ -311,11 +311,11 @@
                              "color": "#565656",
                            },
                          ]
                        }
                      >
-                       화
+                       T
                      </Text>
                    </View>
                    <View
                      style={
                        Array [
@@ -343,11 +343,11 @@
                              "color": "#565656",
                            },
                          ]
                        }
                      >
-                       수
+                       W
                      </Text>
                    </View>
                    <View
                      style={
                        Array [
@@ -375,11 +375,11 @@
                              "color": "#565656",
                            },
                          ]
                        }
                      >
-                       목
+                       T
                      </Text>
                    </View>
                    <View
                      style={
                        Array [
@@ -407,11 +407,11 @@
                              "color": "#565656",
                            },
                          ]
                        }
                      >
-                       금
+                       F
                      </Text>
                    </View>
                    <View
                      style={
                        Array [
@@ -439,11 +439,11 @@
                              "color": "#565656",
                            },
                          ]
                        }
                      >
-                       토
+                       S
                      </Text>
                    </View>
                  </View>
                  <ScrollView
                    data={

@0916dhkim Cool. We can use snapshot test again for DatePicker.

@oh3vci Did you try locale props with 'kr-KR' or other timezones when you test? Did it work as like '월', '화'...?

Isn't it 'ko-KR', not 'kr-KR'?

@JeffGuKang I tried setting locale props to ko-KR in storybook. Korean characters show up correctly on web, but it does not work on Android. The same thing happens for locale="es-ES".

I guess it seems to be another bug of android or storybook. it works well in iOS.

Maybe, this is related issue in android.

@oh3vci It is working well now with #351. 👍


#233 Hotfix

add vendor prefixes to the checkbox transform

🔥

Looks good in Safari, FF, and Chrome. Just minor detail:
left = PR, right = design toolkit now

The text for the language label sits right at the top of the box and is not centered with the text.

eagle eyes @EdwinGuzman  👀

:hawk-emoji:


[W5][W14-2] Chong Kay Heen

Would be better if your enchancement actually did something

something useful i mean


The phrase "q0" is highlighted as a keyword in the editor

Operating system or device:
Windows 10
Issue description (what happened, and what was expected):
Typing "q0" in your script gets it highlighted in yellow even though it's not a keyword.
Steps to reproduce:
Type "q0" into a script and bask in the yellow light.

Can confirm

@bojidar-bg and I had a look and it doesn't seem obvious to find the cause of this bug, so removing the junior job tag.

Here is a snippet from IRC, showing our finds (not from official godot engine channels):
<bojidar_bg> BTW, searching for "q0" ( https://github.com/godotengine/godot/issues/3843 ) in the repo didn't lead to any finds...
<Akien> bojidar_bg: Yeah I've tried to understand #3843 too but couldn't
<bojidar_bg> Akien: maybe some wierd hashing stuff?
<bojidar_bg> Akien: only place outside drivers is scene/resources/animation.cpp
<Akien> Well it probably wouldn't be "q0" directly, but more likely some parsing of a 'q' token and then '0'
<bojidar_bg> Akien: where were highlighted words defined? :D
<bojidar_bg> Akien: I grepped for 'q too, no luck
<Akien> bojidar_bg: Check tools/editor/plugins/script_editor_plugin.cpp
<Akien> The part after "set_syntax_coloring(true)"
<Akien> It then refers to functions in modules/gdscript, etc.
<Akien> I've looked at it but couldn't find the culprit hehe
<Akien> I guess with some prints of the actual keywords it could be narrowed down
<bojidar_bg> Akien: they come from gdscriptLanguage::get_reserved_words :P
<bojidar_bg> so, https://github.com/godotengine/godot/blob/master/modules/gdscript/gd_editor.cpp
<bojidar_bg> so, https://github.com/godotengine/godot/blob/6ea0863ed3e836127cfd38587a50679d40ca40bc/modules/gdscript/gd_script.cpp#L2678 :D
<Akien> Yeah I've checked that, but did not find any q :p
<bojidar_bg> Akien: https://github.com/godotengine/godot/blob/6ea0863ed3e836127cfd38587a50679d40ca40bc/modules/gdscript/gd_script.cpp#L2722
<Akien> s/q/clue/ ;
<bojidar_bg> maybe that one ? :D
<Akien> I wondered about that one too :p
<Akien> But that's still weird hehe
<bojidar_bg> Akien: who shall try removing it? :D
<Akien> Why would it make "q0" a keyword?
<bojidar_bg> Akien: no reason :P
<Akien> I have no clue what it does, but I'm not good with static const char* stuff :)
<bojidar_bg> Akien: maybe it would get the memory from another location
<bojidar_bg> so the q actually is some texture or something..
<Akien> You can try )
<Akien> :)
<bojidar_bg> Akien: also, if this makes some error... https://github.com/godotengine/godot/blob/master/modules/gdscript/gd_functions.cpp#L110
<bojidar_bg> but it has a ERR_FAIL_INDEX_V(p_func,FUNC_MAX,""); guard
<bojidar_bg> so, no
<bojidar_bg> Akien: nope, not from that zero
<bojidar_bg> nothing has changed
<bojidar_bg> Akien: actaully it seems like a condition to terminate the loop on 2728
<bojidar_bg> Akien: seems like no luck for today :P
<bojidar_bg> Akien: so, not a junoir job
<bojidar_bg> better if they don't try :P

The problem is in String::hash() in ustring.cpp.
It seems the hash generated for "q0" is the same as for the keyword "or" :)

here's an improved algorithm for the hashing function (less collisions and better performance)
`static const unsigned int FNV_PRIME = 16777619u;
static const unsigned int OFFSET_BASIS =<PHONE_NUMBER>u;
unsigned int hash2(char* str)
{
unsigned int hash = OFFSET_BASIS;
while ((*str))
{
hash ^= *str++;
hash *=FNV_PRIME;
}
return hash;

}`

PS - I tested and compared the performance of the two algs in a 10000000 for cycle. My suggested solution is faster, but only when the strings are relatively small. When I switched to longer strings (14chars), the performance was only slightly worse than the current one.

@nunodonato ~ BTW, your solution might (must) still have collisions. That's a general problem with hashing... Maybe a better solution would be to compare strings directly, even after the same-hash test passes :smile:
Still, if it is better, we might include your hash function too.

Feel free to make a PR with your hash function, if performance is about the
same there shouldn't be a problem.
That said, it may be a mistake to rely on hash functions for this feature
(or maybe longer/64 bit hashes should be used)
On Sun, Feb 28, 2016 at 6:43 AM, Bojidar Marinov<EMAIL_ADDRESS>wrote:

@nunodonato https://github.com/nunodonato ~ BTW, your solution might
(must) still have collisions. That's a general problem with hashing...
Maybe a better solution would be to compare strings directly, even after
the same-hash test passes [image: :smile:]
Still, if it is better, we might include your hash function too.
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3843#issuecomment-189832161.

I found another random false positive, "acR":

I guess we should both use hashes, and string comparison... or maybe the Dictionary class already does it?

I'm checking the code again, I can't find anywhere a hash alone is being used..

Something must be wrong and I am being unable to see it:
here the hash is computed
https://github.com/godotengine/godot/blob/master/scene/gui/text_edit.cpp#L836
and here the value is obtained from the hash and the string
https://github.com/godotengine/godot/blob/master/core/hash_map.h#L401
as you can see above, there is a string comparison happening too, so unless I'm missing something, it makes no sense that "q0" or "acR" are reported as keywords.
We should add some kind of tag "paranormal investigator job".

We should add some kind of tag "paranormal investigator job".

Yeah, because that was actually a very StrRange bug.


Install from latest stable build

Now mackerel-agent provides official OS X stable releases.

@motemen
Used head do ... end syntax to declare that the dependencies are for only HEAD.

LGTM, thank you! Maybe the ownership of this formula should be transferred to @mackerelio org.


Unable to to enable remote logging from onboarding screens

Description of problem
Remote logging could not be turned on when dev mode was enabled during onboarding. Tapping it did nothing. Turning on remote logging after onboarding was fine.
note: Enable proxy could also not be turn on. not sure if that was intentional though.
Expected behavior
We should be able to turn on remote logging from the onboarding screen
Steps to reproduce

Restart the app
Get to the "is this app for you screen"
Get into dev mode
attempt to turn on remote logging

Screenshots and/or log output
Environment

[ ] Occurs on Android
[X] Occurs on iOS

Build #: 2112
Android Device Model:
iOS Device Model:
Workaround
Severity

[ ] High
[x] Medium
[ ] Low

@jleach Can you confirm that remote logging was ever enabled (before it's unlocked) or is this a bug that needs to be reconciled. Please touch base with @knguyenBC as she was looking to have this to verify functional/UX issues with the application.

Further discussion as to why this functionality is not available before a user authenticates.


Repository dependencies

Add a package.json file in order to displays the full list of package dependencies of the project as Visual Studio NuGet package file format is not supported by Github. See https://help.github.com/articles/listing-the-packages-that-a-repository-depends-on for more informations.

As Github has recently be acquired by Microsoft, the dependency graph feature for Visual Studio projects could be implemented in a near future. Closing this for now.


Instruction update for new API requirement.

Enable the Scope: To use this scope, simply enable "vehicle_location" scope on your application from the Developer UI and include it in both /authorize and /token requests. Starting 1st January 2025, explicit user consent will be required for vehicle location access, so be sure to update your application accordingly and request additional consent from your users.

The vehicle_location scope must be enabled starting January 1st, 2025.
Note, does the EV2 app actually need to request the scopes energy_device_data and energy_cmds?  I think those are only required for the Powerwall nodeserver.

The energy device seem to be needed if user has both power wall and EV - At
least when I started developing this - I contacted Tesla but never head
back - it may have been fixed by now
On Wed, Dec 11, 2024 at 5:35 PM Robert Blake Von Haden <
@.***> wrote:

The vehicle_location scope must be enabled starting January 1st, 2025.
Note, does the EV2 app actually need to request the scopes
energy_device_data and energy_cmds? I think those are only required
for the Powerwall nodeserver.
—
Reply to this email directly, view it on GitHub
https://github.com/Panda88CO/udi-TeslaEV2/pull/13#issuecomment-2537552367,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APQ7ECQ4AIFW3XVPHZTUJQT2FDR5HAVCNFSM6AAAAABTOU6EV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZXGU2TEMZWG4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Oh, I think you are correct that they may be required because the EV2 and PW2 both share the same my.isy.io authentication.

i'll take a look later - I have a few items to finish on my yolink server,
but after that
Should be simple to fix
I am more worried about how to deal with the charging - working with Michel
at UDI (who it talking to Tesla) to find a solution - it may require more
changes :-(
On Wed, Dec 11, 2024 at 4:44 PM Robert Blake Von Haden <
@.***> wrote:

Enable the Scope: To use this scope, simply enable "vehicle_location"
scope on your application from the Developer UI and include it in both
/authorize and /token requests. Starting 1st January 2025, explicit user
consent will be required for vehicle location access, so be sure to update
your application accordingly and request additional consent from your users.
You can view, comment on, or merge this pull request online at:
https://github.com/Panda88CO/udi-TeslaEV2/pull/13
Commit Summary

d473838
https://github.com/Panda88CO/udi-TeslaEV2/pull/13/commits/d4738380fe0387e6e39a90ad4d88ac915fd81f3a
Instruction update for new API requirement.

File Changes
(2 files https://github.com/Panda88CO/udi-TeslaEV2/pull/13/files)

M POLYGLOT_CONFIG.md
https://github.com/Panda88CO/udi-TeslaEV2/pull/13/files#diff-caf18d2009bca5e3020408067150f3fbf4086ad2db70e0d24a8fa2a084ebd87f
(1)
M README.md
https://github.com/Panda88CO/udi-TeslaEV2/pull/13/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5
(1)

Patch Links:

https://github.com/Panda88CO/udi-TeslaEV2/pull/13.patch
https://github.com/Panda88CO/udi-TeslaEV2/pull/13.diff

—
Reply to this email directly, view it on GitHub
https://github.com/Panda88CO/udi-TeslaEV2/pull/13, or unsubscribe
https://github.com/notifications/unsubscribe-auth/APQ7ECXTO3UGUVP23ABU7JL2FDL7HAVCNFSM6AAAAABTOU6EV2VHI2DSMVQWIX3LMV43ASLTON2WKOZSG4ZTIMZXGQZTIOA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

If you need someone to test anything on the EV2 or PW2 nodeservers, just ping me.

Thanks
We need to get the install issue on the node store fixed - then I have something on the EV2 - I did update the PW2 but only a unit issue similar to your input on EV2

From: Robert Blake Von Haden @.>
Sent: Wednesday, December 11, 2024 6:00 PM
To: Panda88CO/udi-TeslaEV2 @.>
Cc: Panda88CO @.>; Comment @.>
Subject: Re: [Panda88CO/udi-TeslaEV2] Instruction update for new API requirement. (PR #13)
If you need someone to test anything on the EV2 or PW2 nodeservers, just ping me.
—
Reply to this email directly, view it on GitHubhttps://github.com/Panda88CO/udi-TeslaEV2/pull/13#issuecomment-2537583622, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APQ7ECS5RCVIGUTYIB2YTJT2FDU2RAVCNFSM6AAAAABTOU6EV2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMZXGU4DGNRSGI.
You are receiving this because you commented.Message ID: @.***>


Unable to run any of my original .ROM files

I am not able to run any of my .ROM files anymore. So its not just the physical carts now, it seems to be all my .ROM files. None of the ones on the SDCard load. I have checked the original Mattel versions and they all seem fine... just seems to be mine. Obvs they run fine on the OG Aquarius. Just tried a .CAQ version and it works perfectly. See screen shot for error when running rom file.

Roy, we'll need a few more details:

Take a picture of the main "Microsoft" screen, after the RETURN color cycle screen, so we can see the version of OS and pB you're running.
Show us the command you're using to launch the ROMs, as well as the path of the ROMs.
Try a LOAD of the ROM first to see if you get an error. If not, do a LIST (and a screen shot) so we can see the BASIC stub, if there is one.

Thanks!

Can I get a copy of a ROM image so I can test?

I saw that you replied with a request for a copy of the .ROM, but I can not see the comment now :( I assume you realised that it is on the SDCard? its the BC2-SE.ROM

I did find the ROM file on the SD card distro. The fix is in process.
In the meantime, please upgrade your Aquarius+ to v1.1 and see if the physical carts are working.

I did find the ROM file on the SD card distro. The fix is in process.
In the meantime, please upgrade your Aquarius+ to v1.1 and see if the physical carts are working.

Upgraded to V1.1 and pB v0.23n, still no joy with the physical cart. Just a grey blank screen when inserted and powered on.

Any news?


Neither the website nor the checkout process indicate the time of the event

What time should folks show up on the day of the event?

We're finalising the schedule at the moment, so when we publish that we'll update the site to make this clearer. Greater point though!

The agenda has now been added!


Optimal Transport from 1D Vector

This PR creates a version of ot_cost and ot_plan that receives usupport, vsupport and uprobs, uprobs. In case the vector of probabilities is not passed, it assumes that all entries have equal probability.
The PR has one test that is failing that I was not able to figure out where is coming from.
One thing that I tried doing was "NOT" sorting the vector, but returning the transport plan for the order given. I'm guessing this is the thing that is causing some issues, because when the transport plan is provided to the ot_cost function, it has to reorder it for to be properly used, and I'm guessing there might be some compilation issues.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

11 of 11 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 97.076%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
166

Relevant Lines:
171

💛  - Coveralls


Targetable

I have added functionality to target specific enemies using autoskill. To do so, simply include 't' and a number to indicate which enemy you want to target. 1, 2, or 3. The format for enemy targets based on screen are:
t1    t2    t3
So an autoskill such as a2t3ht1i would use 1st skill on 2nd servant, target the far right enemy, use 8th skill, target far left enemy, use 9th skill.
This has already been tested by a few of my friends including myself and has been found to work on both NA and JP servers. Anyone who wishes to give it more tests, feel free. And if you could double check my logic, that would be great.
Another note, the auto target is still active, and won't be interfered with using this, but I believe we should make it set to 0 by default in order to avoid confusion or possible mishaps with the autoskill. Essentially, the script will assume you are using autoskill to target first, then if you decide you don't wish to, you can toggle auto target on for simpler autoskill command lists.

You nailed this one!

Another note, the auto target is still active, and won't be interfered with using this, but I believe we should make it set to 0 by default in order to avoid confusion or possible mishaps with the autoskill.

I think it's better to leave it as is, because that's the behavior people are already used to, myself included.


REF_CODE_REFACTOR: comparisons.cc

Refactoring the reference code for the operators in comparisons.cc
BUG=refactoring existing code.

@advaitjain

Closing in favor of mutually agreed upon higher priority PRs. Please reopen if needed.


Added selectionsort.cpp

@kmalik31 : It would be great if you can add the description along with the issue number.
Feel free to use our contribution guidelines (https://github.com/fnplus/interview-techdev-guide/blob/master/CONTRIBUTING.md#how-to-contribute-an-implementation-code) for reference

is that good or more changes are needed? @jenistenxavier

https://github.com/fnplus/interview-techdev-guide/pull/224
is your PR number
Please assing the PR to an issue.

Check the following section to know more about contributing PR's linked to issues. https://github.com/fnplus/interview-techdev-guide/blob/master/CONTRIBUTING.md#how-to-contribute-an-implementation-code

Thanks for contributing!


PIM MFA Activation Graph API

Hello,
I am attempting to use the "selfactivate" rule within the Graph API for PIM. Unfortunately, when using this API with the roles that require MFA by default (User Administrator) the error message comes back that states "Elevation Requires Multi-Factor Authentication" but doesn't actually begin the process of conducting MFA. This is a huge blocker for our project, is this being worked on?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 791cb6b6-4733-1c1d-75c5-9318049163af
Version Independent ID: 4752b0bc-ab61-69e3-972f-80d6be2baa55
Content: Configure Azure AD directory role settings in PIM
Content Source: articles/active-directory/privileged-identity-management/pim-how-to-change-default-settings.md
Service: active-directory
GitHub Login: @rolyon
Microsoft Alias: rolyon

@KehindeOwens We have not heard from you in a while.We will now proceed to close this thread. If you have further questions, please tag me in the comments and I will gladly continue the conversation.


[Feature] Tap in user avatar to auto input the user name in the input box

This is an idea that I've found useful in Quasseldroid, a application used mainly for IRC.
Here's the video showing how it works.
NOTE: my screen recorder doesn't show the "tapping" process but I think it's enough to understand the overall idea.

This is really good idea!


Indicators from yaml files - cf-index-meta

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes #501

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)
[ ] bumpversion (major / minor / patch) has been called on this branch
[ ] Tags have been pushed (git push --tags)

What kind of change does this PR introduce?

Implements the generic index definitions from cf-index-meta into `xclim.indices.generic
New xclim.core.yaml submodules that handles creation of indicators from yaml-parsed dictionaries. But I plan to also add methods to create fake modules from yaml files (i.e. icclim, anuclim and friends)
Moved "xclim/locales" to "xclim/data" and addition of a `variables.yml" with definitions of "official" xclim variables.
Internal changes in the indicator creation to allow passing a parameters dict. This dict is used to override values parsed from the compute's docstring.

I copied and extended the  variables.yml file from cf-index-meta. The problem is that they do not define the dimensionality (units) and don't give descriptions of the variables, which we need to properly create the indicator and it's docstring.
Overall, the yaml file generated by cf-index-meta is relatively complete. But, for a complete xclim integration, I think the following features are missing:

There is a way to define a parameters passed to the generic function, but there is no explicit way for defining the default value of such a parameter, while letting the user decide. In addition to the default value, this is important for the docstring: right now if a parameter of the index function is mentionned in the yaml, it is injected, otherwise the user sees the name, default and description from the generic index function's docstring, so nothing informative.
Multi-output indicators (FWI, wind_speed_from_vectors, etc) are not supported. Maybe that's ok.
cf-index-meta defines which frequencies are allowed for a given indicator, and gives a default one. xclim doesn't have such a control. Easy to implement.
cf-index-meta doesn't define the following fields : abstract, title, keywords, comment, realm, missing. The last two are not so important, but missing the first two gives uninformative docstrings.

Does this PR introduce a breaking change?
I moved the locales json to a new folder, but it shouldn't impact users.

Last commits pushed something working! It can parse the yaml file that cf-index-meta outputs!
Most of the doc is in xclim.core.indicator. I moved the module creation function there.

Added a mechanism to create cfcheck functions "on-the-fly", needs parsing of the variables.yml file, but this might be down even more interactively downloading stuff from the cf conventions website. (Also there seems to be an issue with this right now...)
Added a mechanism to restrict allowed freq strings. Indicators now have a "allowed_periods" attibute, a list of base offsets to check against (A is the same as Y). The check is performed in the __call__.

TODO:

Find a way to elegantly define indicators in yaml from existing indicators. I currently is broken when you "inject" a parameter that appears in one of the metadata strings. (Ex: long name is "When X is over {thresh}" but you define thresh = 3 in the yaml). We would need a way to partially format metadata strings?
Find a way to reduce the number of indicator registry overwritings? It is possible to have similar (but maybe different) indicators in two submodules, with the same identifier. I think we could prepend the identifier by the submodule name in the registry for non-xclim indicators. (This way "ICCLIM.GD4" and "CF.GD4" would refer to different objects).
More testing and documenting.

Another large set of changes.

Fine tuning of the yaml importation, fix some tests.
Added cf.yml, anuclim.yml and icclim.yml to define these virtual submodules!

I think @tlogan2000 might be interested in looking into this. I went back to the documents of ECAD (icclim) and Anuclim to  be sure the metadata was correct, so there were some modifications made to the _anuclim.py file + a bit more metadata added into the yml files than was before in the build_module call.
Breaking change : xclim.icclim will not work anymore. Only xclim.indicators.icclim (same for cf and anuclim).

@tlogan2000 @Zeitsperre  I have partially rewritten the "mappings" notebook to introduce the new YAML way.  As now mappings are actually Indicators mapping, and not indices, the old way was transformed in something a bit more complex. I didn't have the courage, nor to I think it is useful, to convert the PCC mapping. However, it could be made into a YAML file and added permanently to xclim?
@huard Earlier in this PR I decided to introduce a mechanism so that indicator subclass that are NOT declared within xclim.indicators. would have their module name prepended in the registry. In order to prevent overwriting official indicators with eponymous copies of, say, icclim.  Now, that means that the indicators in live sessions (like a notebook) are prefixed by __main__, which ain't cute. Also, one can pass module to force a fake module. As you can see in the new mappings notebook, that looks a bit odd, passing module to each indicator creation before even creating that module...
Thoughts on this?

Hopefully, the only remaining test failures would be those related to the inconsistencies between xclim's and clix-meta's unit handling.

@huard  @tlogan2000  I will merge this at the end of my day. The only real change since your reviews is the addition of a atmosds pytest fixture and the submodules testing where we simply execute everything we can.


Updated FIR node and Butter node

Description

[x] I've included a concise description of what each node does

Styleguide

[x] My node adheres to the styleguide for Flojoy nodes

Docs

[x] I've submitted a PR for a documentation page for the new node(s) that contains usage examples (see docs.flojoy.io)

Testing

[x] This PR includes a unit test (example here and/or ideally a screenshot of the node's output on an example app.

Example for FIR node:

Example for BUTTER node:

Already merged with new API


Added javax.faces-2.3.0-m09.jar and maven install plugin

Added javax.faces-2.3.0-m09.jar and maven install plugin as suggested by Rohit

2.3.0-m09 can be added through pom as well, what is the need for adding jar file?
<!-- https://mvnrepository.com/artifact/org.glassfish/javax.faces -->
<dependency>
    <groupId>org.glassfish</groupId>
    <artifactId>javax.faces</artifactId>
    <version>2.3.0-m09</version>
</dependency>

Just tested. My bad. The latest snapshots are available on maven.
https://repo.maven.apache.org/maven2/org/glassfish/javax.faces/
Please continue to make the pom changes and don't use the jar.


RELEASE console release  v0.22.0

Console v0.22.0 has changes that impact the docs:

[x] Users now see a warning when setting a bucket to public access - PR #2473
[x] Login page has new styles - PR #2467
[x] Bucket now displays a human readable creation time - PR #2464](https://github.com/minio/console/pull/2464)
[ ] Updates to the Airgap registration page - PR# 2463

Bucket display affects a screenshot, but will update in a later pass.


Selenium

@b3vedi good job on your first PR, very well done!
We might not need the binary files (.exe) in this PR. I assume the code needs to be relevant to the Web crawling and/or Selenium/testing experiment in ITLAB i.e not sure whether we should be crawling our own website 😉
From the lab component:

Demonstrate the behaviour of Web Crawlers/ spiders (use XPATH, CSS PATH),extract
information and store it in the database.

From the corresponding theory component:

Introduction to a web crawler, the role of crawler in the internet, the concept of
page ranking

Either way, good progress!
/cc @Deadcoder11u2


Patch 215

Learn Build status updates of commit 37389b7:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/102623/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Service accessible via container IP but not via host IP even when port mapping is done

Description
I am trying to create a nerdctl local registry on my setup.
The registry which is running on port 5000 inside the container is mapped to port 5001 of my host.
I am able to access registry with container IP but not with host IP even when mapping is done correctly.
containers running:

container details, nerdctl inspect omnia-registry  :

registry details via container IP:

when trying to access registry via host IP/name:

port mapping:

ufw status:

What can be the issue here?
Steps to reproduce the issue

Created a service file for nerdctl-registry:
cat /etc/systemd/system/nerdctl-registry.service

[Unit]
Description=Nerdctl Registry

[Service]
Type=oneshot
RemainAfterExit=yes
WorkingDirectory=/opt/omnia/nerdctl-registry
ExecStart=/bin/bash nerdctl-registry start
ExecStop=/bin/bash nerdctl-registry stop

[Install]
WantedBy=default.target

cat /opt/omnia/nerdctl-registry/nerdctl-registry

#!/bin/bash

nerdctl_command=$1
if [ $nerdctl_command = "start" ]; then
    nerdctl_ps_result=$(nerdctl ps -a | grep "omnia-registry")
    if [ -z $nerdctl_ps_result ]; then
        nerdctl run -d -p 5001:5000 --restart=always --name omnia-registry \
          -v /opt/omnia/nerdctl-registry/images:/data \
          -e REGISTRY_STORAGE_FILESYSTEM_ROOTDIRECTORY=/data \
          -v /opt/omnia/nerdctl-registry/certs:/certs \
          -e REGISTRY_HTTP_TLS_CERTIFICATE=/certs/domain.crt \
          -e REGISTRY_HTTP_TLS_KEY=/certs/domain.key \
          registry:2
    else
        nerdctl start omnia-registry
    fi
elif [ $nerdctl_command = "stop" ]; then
    nerdctl rm -f omnia-registry
else
    echo "The parameter is wrong."
fi

Create certificates for registry

systemctl start nerdctl-registry

Describe the results you received and expected
Registry should be accessible at HostIP:5001
What version of nerdctl are you using?
root@monsooncp:~# nerdctl version
WARN[0000] unable to determine buildctl version: exec: "buildctl": executable file not found in $PATH
Client:
Version:       v1.5.0
OS/Arch:       linux/amd64
Git commit:    b33a58f288bc42351404a016e694190b897cd252
buildctl:
Version:
Server:
containerd:
Version:      1.6.16
GitCommit:    31aa4358a36870b21a992d3ad2bef29e1d693bec
runc:
Version:      1.1.4
GitCommit:    v1.1.4-0-g5fd4c4d
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
Others
Host information
root@monsooncp:~# nerdctl info
Client:
Namespace:     default
Debug Mode:    false
Server:
Server Version: 1.6.16
Storage Driver: overlayfs
Logging Driver: json-file
Cgroup Driver: cgroupfs
Cgroup Version: 1
Plugins:
Log: fluentd journald json-file syslog
Storage: aufs native overlayfs
Security Options:
apparmor
seccomp
Profile: default
Kernel Version: 5.4.0-176-generic
Operating System: Ubuntu 20.04.6 LTS
OSType: linux
Architecture: x86_64
CPUs: 48
Total Memory: 125.4GiB
Name: monsooncp.monsoon.test
ID: c2878b4c-63e9-4048-8f6d-a91534c6080d
WARNING: No swap limit support
WARNING: bridge-nf-call-iptables is disabled
WARNING: bridge-nf-call-ip6tables is disabled

what does the following net commands show on host.
 ip route show
 iptables -S
 netstat -tuln

And wondering does changing the ip to localhost works for you?

traffic should be translated by a dnat rule,
what gives : iptables -t nat -v -n  -L

Hi @sakshiarora13
nerdctl v1.5.0 is outdated and out of support now.
Would you be able to retry/confirm this with a recent version (eg: 2.0)?
Thanks.


ws-icon doesnt work for more than 9 workspaces

Describe the issue
I have a pretty normal setup with i3 and polybar with 10 workspaces in i3. I now tried to assign icons to workspaces but for workspace 10 it shows the same icon as for workspace 1. I already tried putting the number in quotation marks but that didnt solve the issue.
Expected behavior:
I should see the icon I assigned for workspace 10 and not the one for ws 1.
Actual behavior:
It shows the one for ws 1
Was it working before?
Never tried it before
To Reproduce
This is the configuration for the i3 module
[module/i3]
type = internal/i3
;format = <label-state> <label-mode>

ws-icon-0 = 1;
ws-icon-1 = 2;
ws-icon-2 = 3;
ws-icon-3 = 4;
ws-icon-4 = 5;
ws-icon-5 = 6;6
ws-icon-6 = 7;7
ws-icon-7 = 8;8
ws-icon-8 = 9;9
ws-icon-9 = "10;10"

index-sort = true
fuzzy-match = true
wrapping-scroll = false

label-active-font = 7
; Only show workspaces on the same output as the bar
;pin-workspaces = true

label-mode-foreground = #000
label-mode-background = ${solarized.yellow}

; focused = Active workspace on focused monitor
;label-focused = %name%
;label-focused-background = ${module/bspwm.label-focused-background}
;label-focused-underline = ${module/bspwm.label-focused-underline}
;label-focused-padding = ${module/bspwm.label-focused-padding}

label-focused = %icon%
label-focused-background = ${solarized.base02}
label-focused-underline = ${solarized.yellow}
;label-focused-padding = ${module/bspwm.label-focused-padding}
label-focused-padding = 1
label-unfocused-padding = 1
; unfocused = Inactive workspace on any monitor
label-unfocused = %icon%

;label-unfocused-padding = ${module/bspwm.label-occupied-padding}
; visible = Active workspace on unfocused monitor
label-visible = %icon%
label-visible-background = ${self.label-focused-background}
label-visible-underline = ${self.label-focused-underline}

List any other steps needed to reproduce the issue besides starting polybar with the config you posted above.
Polybar Log
N/A
Screenshots
This is what it shows. The icon on the far right should be a headphones icon instead of the code icon.

Environment:

WM: i3
Distro: I use arch btw
Output of polybar -vvv:

polybar 3.4.2

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: Release
Compiler: /usr/bin/c++
Compiler flags: -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -Wall -Wextra -Wpedantic -O3 -DNDEBUG
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

Additional context
I tried it with 2 monitors as well as with 1 doesn't make a difference

Duplicate of #2041
This is fixed in the upcoming release. For now, disable fuzzy-match


Set this.entryResponse to undefined after we have sent it

Fixes #

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing

Ah I see this now. Thanks!


FreeBSD/FreeBSD GSSAPI Fixes

Improve GSSAPI handling under FreeBSD
Use ifr->ifr_addr instead of ifr->ifr_ifru under FreeBSD

Note: configure.ac should be updated to #include <stdio.h> for any of the GSSAPI-related configure tests.

Either of you know the right incantation (or perhaps packages needed) on the cirrus-based freebsd image to get a test build with GSSAPI working?  I tried a quick configure --with-gssapi, but it failed to find what it needed and my knowledge of this world is severely lacking, so figured I'd ask.   Thanks!

You would want the port, with the GSSAPI option enabled: https://www.freshports.org/comms/conserver-com/

To install the port: cd /usr/ports/comms/conserver-com/ && make install clean

Ah...you’re earlier comment about using stdio.h did the trick. I shoulda paid a bit more attention.


Policy: ACLs

We can refer to the schema defined in this PR https://wwwin-github.cisco.com/netascode/nac-vxlan/pull/58/files

Uploading the draft merged schema and data yaml file having the route map, acl, prefix list data model.
updated_schema_datafile_draft.zip

@mtarking, we checked this in the sync up call and schema items are missing for this feature (policy: ACL).

@devegupt Correct. My draft is for route-maps, prefix-lists, communities, etc as they can be used in the former.


Use yarn.lock or package-json.lock to lock dependency versions

I noticed that there's no package-json.lock or yarn.lock file in the repo, this could be a potential problem in the future. I can submit a PR to fix this, but I'm unsure what the maintainers want to use.

@Jivanvl Thanks! We will add a package-json.lock later.


missing button for re-checking authentication status for middleware/hawkular providers

Same as #8912, it should be done for middleware providers as well
@miq-bot add_label providers/hawkular

@miq-bot assign @mtho11

Auth recheck enabled for middleware.


Promote IncludeCompiler into DI services

As a 3rd-party EFCore provider developer/maintainer, I need to be able to modify the behavior of the default IncludeCompiler. The current IncludeCompiler makes several assumptions about the underlying DB provider's capabilities especially with regards to GroupJoins, and results in non-performant runtime query execution.
It is possible to inherit from IncludeCompiler and override some functionality. However, doing so is currently very clunky since there's no service factory, no overridable factory method where it's used in EntityQueryModelVisitor, and IncludeCompiler's nested classes are all private, making it impossible to reuse or override them.
The net is result is that EntityQueryModelVisitor.OptimizeQuery must be completely reproduced, along with several other internal types (including IncludeLoadTree/Node/Base and every class that is directly instantiated in OptimizeQuery) for the sole purpose of modifying IncludeCompiler's functionality. This is bad design, since IncludeCompiler handles a public API feature (ie: IncludableQueries), but its functional use is effectively internal, making it very difficult for 3rd party providers to alter the behavior.
I'd like to propose at least one of the following changes:

Promote IncludeCompiler to a dependency-level service using proper dependency injection and a factory.

Promote IncludeCompiler.IncludeLoadTree and IncludeCompiler.IncludeLoadTreeNode to public interfaces.

Expose IncludeLoadTreeNodeBase.CompileReferenceInclude and IncludeLoadTreeNodeBase.CompileCollectionInclude as protected virtuals

These changes will make it much easier to override and modify the behavior of IncludeCompiler without having to re-implement several other EFCore types and services.
public interface IIncludeCompilerFactory
{
    IIncludeCompiler CreateIncludeCompiler(
        [NotNull] QueryCompilationContext queryCompilationContext,
        [NotNull] IQuerySourceTracingExpressionVisitorFactory
            querySourceTracingExpressionVisitorFactory);
}

public interface IIncludeLoadTreeNode
{
    IReadOnlyCollection<IncludeLoadTreeNode> Children { get; }
    Expression Compile();
}

public interface IIncludeLoadTree : IIncludeLoadTreeNode
{
}

public class IncludeCompiler
{
    /* etc. */

    protected virtual IIncludeLoadTree CreateIncludeNodeTree();
}

public class IncludeLoadTreeNode : IIncludeLoadTreeNode
{
    public virtual Expression Compile() { /* etc. */ }

    protected virtual Expression CompileReferenceInclude() { /* etc. */ }

    protected virtual Expression CompileCollectionInclude() { /* etc. */ }
}

Further technical details
EF Core version: ALL EFCore versions
Database Provider: Non-Specific (problem is in EFCore)

Tracking this for 3.0 as part of the the larger query refactoring that is tracked by #12795

Thanks for the timeline! This change will be a huge help for NoSQL providers.

This has been fixed. There is no separate include compiler. It has been integrated in nav rewrite.


[Windows] Issue while detecting if it's Novel

I just tried it out, did follow all the steps
but can't get it to work
Issue I am facing is on Line No 148-149
Basically it's saying IndexError

Try again plz

Thanks It's Working now.


TDSParser Error

Hi,
As lots of people have previously mentioned this bug, I will try and keep this short and concise.
I am getting the bug within a VSTO Excel plugin whereby an exception is thrown when attempting to open a SQL connection. It appears that the appropriate SNI libraries are not being copied properly in the publish folder, but are in the debug and release folders.
Authentication with Azure works fine for KeyVault, It is just AZ Sql that it fails with.
I have tried the solution listed in #446, https://learn.microsoft.com/en-us/sql/connect/ado-net/sqlclient-troubleshooting-guide?view=sql-server-ver15#unable-to-load-native-sni-server-name-indication-library. My attempts to fix the problem range from shadow copying the library, attempting to have a reference within the project to SNI, migrating from package.config to packagereference, forcing a local copy...the list goes on.
I will freely admit I am not an expert at MSBuild and could really do with some help, If I have missed something please let me know, I'm very very lost...
Code:
public DataSet RunReaderCommand(string command)
{
                DataSet ds;
                SqlConnectionStringBuilder = new SqlConnectionStringBuilder();
                SqlConnectionStringBuilder.DataSource = Az_SQLDB_svrName;
                SqlConnectionStringBuilder.InitialCatalog = Initial_DatabaseName;
                SqlConnectionStringBuilder.TrustServerCertificate = true;
                SqlConnectionStringBuilder.PersistSecurityInfo = false;
                SqlConnectionStringBuilder.Authentication = SqlAuthenticationMethod.ActiveDirectoryDefault;
                SqlConnection = new SqlConnection(SqlConnectionStringBuilder.ConnectionString);
                SqlConnection.Open();
                SqlCommand SqlCommand = new SqlCommand(command, SqlConnection);
                SqlCommand.CommandType = CommandType.Text;

                sqlDataAdapter.SelectCommand = SqlCommand;

                ds = new DataSet(Table.ToString());
                sqlDataAdapter.Fill(ds);
                SqlConnection.Close();
                return ds;
}

Exception message:
The type initializer for 'System.Data.SqlClient.TdsParser' threw an exception.:
Stack trace:

   at System.Data.SqlClient.TdsParser..ctor(Boolean MARS, Boolean fAsynchronous)
   at System.Data.SqlClient.SqlInternalConnectionTds.LoginNoFailover(ServerInfo serverInfo, Boolean redirectedUserInstance, SqlConnectionString connectionOptions, TimeoutTimer timeout)
   at System.Data.SqlClient.SqlInternalConnectionTds.OpenLoginEnlist(TimeoutTimer timeout, SqlConnectionString connectionOptions, Boolean redirectedUserInstance)
   at System.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, Object providerInfo, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling)
   at System.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.CreatePooledConnection(DbConnectionPool pool, DbConnection owningObject, DbConnectionOptions options, DbConnectionPoolKey poolKey, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
   at System.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
   at System.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions userOptions, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at System.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)
   at System.Data.SqlClient.SqlConnection.Open()

Further technical details
Microsoft.Data.SqlClient version: 5.2.0-preview1.23109.1 <--- I switched to the preview to check I couldn't force it to copy (It didn't)
Microsoft.Data.SqlClient.SNI version: 5.1.0
.NET Framework 4.8
SQL Server version: AZ Sql
Operating system: Azure VM Windows Pro 10/11
Additional context
This is my build output:
ResolveAssemblyReferences:
1>  Primary reference "Microsoft.Data.SqlClient.SNI".
1>C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(2352,5): warning MSB3245: Could not resolve this reference. Could not locate the assembly "Microsoft.Data.SqlClient.SNI". Check to make sure the assembly exists on disk. If this reference is required by your code, you may get compilation errors.
1>          For SearchPath "{CandidateAssemblyFiles}".
1>          Considered "C:\Users\Mharr\.nuget\packages\microsoft.sqlserver.types\160.1000.6\build\net462\..\..\runtimes\win-x86\native\SqlServerSpatial160.dll",
1>  			but its name "SqlServerSpatial160"
1>  			didn't match the expected name "Microsoft.Data.SqlClient.SNI".
1>          Considered "C:\Users\Mharr\.nuget\packages\microsoft.sqlserver.types\160.1000.6\build\net462\..\..\runtimes\win-x64\native\SqlServerSpatial160.dll",
1>  			but its name "SqlServerSpatial160"
1>  			didn't match the expected name "Microsoft.Data.SqlClient.SNI".
1>          Considered "C:\Users\Mharr\.nuget\packages\microsoft.web.webview2\1.0.1722.45\build\\..\runtimes\win-x86\native\WebView2Loader.dll",
1>  			but its name "WebView2Loader"
1>  			didn't match the expected name "Microsoft.Data.SqlClient.SNI".
1>          Considered "C:\Users\Mharr\.nuget\packages\microsoft.web.webview2\1.0.1722.45\build\\..\runtimes\win-x64\native\WebView2Loader.dll",
1>  			but its name "WebView2Loader"
1>  			didn't match the expected name "Microsoft.Data.SqlClient.SNI".
1>          Considered "C:\Users\Mharr\.nuget\packages\microsoft.web.webview2\1.0.1722.45\build\\..\runtimes\win-arm64\native\WebView2Loader.dll",
1>  			but its name "WebView2Loader"
1>  			didn't match the expected name "Microsoft.Data.SqlClient.SNI".
1>          For SearchPath "{TargetFrameworkDirectory}".
1>          Considered "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\Microsoft.Data.SqlClient.SNI.winmd", but it didn't exist.
1>          Considered "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\Microsoft.Data.SqlClient.SNI.dll", but it didn't exist.
1>          Considered "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\Microsoft.Data.SqlClient.SNI.exe", but it didn't exist.
1>          Considered "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\Facades\Microsoft.Data.SqlClient.SNI.winmd", but it didn't exist.
1>          Considered "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\Facades\Microsoft.Data.SqlClient.SNI.dll", but it didn't exist.
1>          Considered "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\Facades\Microsoft.Data.SqlClient.SNI.exe", but it didn't exist.
1>          For SearchPath "{AssemblyFoldersFromConfig:C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\AssemblyFolders.config,v4.8}".
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\Extensions\Microsoft\SqlDb\Microsoft.Data.SqlClient.SNI.winmd", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\Extensions\Microsoft\SqlDb\Microsoft.Data.SqlClient.SNI.dll", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\Extensions\Microsoft\SqlDb\Microsoft.Data.SqlClient.SNI.exe", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\PublicAssemblies\Microsoft.Data.SqlClient.SNI.winmd", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\PublicAssemblies\Microsoft.Data.SqlClient.SNI.dll", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\PublicAssemblies\Microsoft.Data.SqlClient.SNI.exe", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\ReferenceAssemblies\v4.5\Microsoft.Data.SqlClient.SNI.winmd", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\ReferenceAssemblies\v4.5\Microsoft.Data.SqlClient.SNI.dll", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\ReferenceAssemblies\v4.5\Microsoft.Data.SqlClient.SNI.exe", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\ReferenceAssemblies\v4.0\Microsoft.Data.SqlClient.SNI.winmd", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\ReferenceAssemblies\v4.0\Microsoft.Data.SqlClient.SNI.dll", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\ReferenceAssemblies\v4.0\Microsoft.Data.SqlClient.SNI.exe", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\ReferenceAssemblies\v2.0\Microsoft.Data.SqlClient.SNI.winmd", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\ReferenceAssemblies\v2.0\Microsoft.Data.SqlClient.SNI.dll", but it didn't exist.
1>          Considered "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\ReferenceAssemblies\v2.0\Microsoft.Data.SqlClient.SNI.exe", but it didn't exist.
1>          For SearchPath "{Registry:Software\Microsoft\.NETFramework,v4.8,AssemblyFoldersEx}".
1>          Considered AssemblyFoldersEx locations.
1>          For SearchPath "{AssemblyFolders}".
1>          Considered "C:\Program Files\IIS\Microsoft Web Deploy V3\Microsoft.Data.SqlClient.SNI.winmd", but it didn't exist.
1>          Considered "C:\Program Files\IIS\Microsoft Web Deploy V3\Microsoft.Data.SqlClient.SNI.dll", but it didn't exist.
1>          Considered "C:\Program Files\IIS\Microsoft Web Deploy V3\Microsoft.Data.SqlClient.SNI.exe", but it didn't exist.
1>          For SearchPath "{GAC}".
1>          Considered "Microsoft.Data.SqlClient.SNI", which was not found in the GAC.
1>          For SearchPath "{RawFileName}".
1>          Considered treating "Microsoft.Data.SqlClient.SNI" as a file name, but it didn't exist.
1>          For SearchPath "bin\Release\".
1>          Considered "bin\Release\Microsoft.Data.SqlClient.SNI.winmd", but it didn't exist.
1>          Considered "bin\Release\Microsoft.Data.SqlClient.SNI.dll", but it didn't exist.
1>          Considered "bin\Release\Microsoft.Data.SqlClient.SNI.exe", but it didn't exist.

@MattH678 thanks for opening the issue. We will investigate this.

it doesn't work on the platform ARM64 .NET Framework 4.8

5.1 Target platform support
.NET Framework 4.6.2+ (Windows x86, Windows x64)
.NET 6.0+ (Windows x86, Windows x64, Windows ARM64, Windows Azure Resource Manager, Linux, macOS)
.NET Standard 2.0+ (Windows x86, Windows x64, Windows ARM64, Windows ARM, Linux, macOS)


Set matrix transform for ImageLayer via Python

I have been unable to find a Python example which sets the coordinate space transform for any layer. Is there a way that the CoordinateSpaceTransform object can be use to set a the "layers.source.transform" field in the json state?
Currently I am creating a LocalVolumefrom a numpy array and ten adding a ImageLayer with the local volume as the source.
The motivation for this capability, is that I have performed affine registration between two volumes/images. I would like to just set these values to visualized the two images co-registered.

I just added an example here: https://github.com/google/neuroglancer/commit/e43ed3c56b9fe6ff7871f71e463f84b738803bf2

Great thank you.
Here is what I did with an ImageLayer:
 ng_transform = neuroglancer.CoordinateSpaceTransform(output_dimensions=dimensions,
                                                             matrix = [[1.0,0.0,0.0,translation[0]],
                                                                       [0.0,1.0,0.0,translation[1]],
                                                                       [0.0,0.0,1.0,0.0]])
layer = neuroglancer.ImageLayer(
            source=neuroglancer.LayerDataSource(volume, transform=ng_transform),
      )


Add ARIA roles to tabs

This builds on #405 to also add ARIA roles to tabs. Merge #405 first.

For reference, this is a comparison between this PR's branch, and that of #405: https://github.com/jasongrout/phosphor/compare/aria-menus...jasongrout:aria-tabs


refactor proposer to be in main folder

Proposer sits in a test folder, and it results very inconvenient to import it and use it outside tests as part of infrastructure code. We also expect to release scripts to help external parties deploy their own version of proposers. Therefore, it seems a better option to have a proposer folder in the main code structure and remove it from tests.
There is no change in functionality.

LGTM