Parameterized insert query not working for pyodbc with Cloudera Impala ODBC Driver

   insert_query = "insert into table_name(id,name,price) values(?,?,?)"
   cursor.execute(insert_query,1,"Apples","20 dollars")```

Error received:
The SQL contains 0 parameter markers but 3 parameters were supplied, 'HY000'

https://stackoverflow.com/questions/66166977/pyodbc-parameterized-insert-for-impala-odbc-driver

Python - 3.7.4, Pyodbc - 4.0.27, Anaconda - 1.7.2, OS - Windows 10

Please generate an ODBC trace log and upload it to this thread as an attachment.

Added as an edit .

There appears to be an issue with the ODBC driver that you are using. When pyodbc calls SQLNumParams it returns a parameter count of zero …
run             cb8-310c	ENTER SQLPrepareW
        HSTMT               0x000001844B72CBF0
        WCHAR *             0x0000018450949CB0 [      65] "INSERT INTO product_table (id, name) VALUES (?, ?);"
        SDWORD                    65

run             cb8-310c	EXIT  SQLPrepareW  with return code 0 (SQL_SUCCESS)
        HSTMT               0x000001844B72CBF0
        WCHAR *             0x0000018450949CB0 [      65] "INSERT INTO product_table (id, name) VALUES (?, ?);"
        SDWORD                    65

run             cb8-310c	ENTER SQLNumParams
        HSTMT               0x000001844B72CBF0
        SWORD *             0x00000068AEBDC358

run             cb8-310c	EXIT  SQLNumParams  with return code 0 (SQL_SUCCESS)
        HSTMT               0x000001844B72CBF0
        SWORD *             0x00000068AEBDC358 (0)

… whereas with ODBC Driver 17 for SQL Server (hitting a SQL Server database) SQLNumParams correctly returns the value 2:
gh_pyodbc_867   1da0-24cc	ENTER SQLPrepareW
        HSTMT               0x043803A8
        WCHAR *             0x02FEE378 [      50] "INSERT INTO product_table (id, name) VALUES (?, ?)"
        SDWORD                    50

gh_pyodbc_867   1da0-24cc	EXIT  SQLPrepareW  with return code 0 (SQL_SUCCESS)
        HSTMT               0x043803A8
        WCHAR *             0x02FEE378 [      50] "INSERT INTO product_table (id, name) VALUES (?, ?)"
        SDWORD                    50

gh_pyodbc_867   1da0-24cc	ENTER SQLNumParams
        HSTMT               0x043803A8
        SWORD *             0x010AF24C

gh_pyodbc_867   1da0-24cc	EXIT  SQLNumParams  with return code 0 (SQL_SUCCESS)
        HSTMT               0x043803A8
        SWORD *             0x010AF24C (2)

Please contact the vendor of your ODBC driver for assistance.

Oh okay. Seemed that might be the case..wanted to confirm . Thanks.


Purpose of "+ 1" in width/height JSON output of xyxy2xywh

Just as #450 described how COCO's width/height were subtracted by 1, I wanted to know what is the purpose of adding 1 to the width and height of the output JSON. Is this COCO-specific? It was noted in the 2018 COCO competition slides that "A single pixel shift can lead to 1 point drop." How did this manifest during production and how do these -1/+1 adjustments fix that?
Edit: Naive hypothesis is that the +1 simply "unsubtracts" the -1 applied to the COCO dataset. But how would this affect other datasets?

It depends on your annotation. Specifically, whether (x1, x2, y1, y2) contain the pixel of x2 and y2 or not.


[DOCFIX] Merge cn doc updates.

alluxio-bot, check this please

+1 LGTM

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10848/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10849/
Test PASSed.


Bugfix

Fixing condition and return value.

I don't think this is correct, since it removes the alignment requirement.

Thank you!


Transaction Gossip Network specification

What was wrong
Need a formal spec for transaction gossip network
How was it fixed
Converted the previous forum post to a local specification document.

I'm still trying to figure out something about how we do transaction gossip and at present, I'm inclined to leave the network defined roughly as-is.  I think that the approach currently written down will only be reliable in network conditions where there are enough nodes with a high enough radius.
The desired network topology is:

high level of connectivity between nodes with similar radius values.
high level of connectivity between nodes that are close to each other in the network.

The degenerative state that I think the current spec leaves us in is that full radius nodes are going to miss some transactions.  I'll try to write more down about this later, but the short of it is, that the current gossip mechanism doesn't provide strong enough guarantees that a transaction will escape all the small radius nodes and make it across the network to the high radius nodes that are not closeby.

@carver request for review is really not necessarily asking you to pick this apart at a low level, but more asking for your approval or feedback on merging this roughly as-is in terms of the level of completion.


GET /api/customization

The endpoint only returns the applicationId, but not all the object.

Done


feat: support default function for containerEnv on additionalWorkerGroups in ray-cluster helm chart

Why are these changes needed?
Make the deployment of ray-cluster helm chart with containerEnv not being specified easier using Terraform.

Related issue number
Closes #2569

Checks

[x] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

Couple of tests:
1- We don't specify the containerEnv variable, and we should have no error and nothing should be created since default is an empty list
Screenshots:

2- We test the old behavior where we set a containerEnv, and we should it being added in the manifest
Screenshots:


Home Assistant device class warning

Version of the moonraker integration

0.8.0
Describe your setup

Klipper, Moonraker, Mainsail, Pi 4.
Describe the bug

There's a warning regarding device classes shown in the Home Assistant log, suggesting it should be reported to the integration author.
Moonraker Logs
 Logger: homeassistant.components.sensor
Source: components/sensor/__init__.py:597
Integration: Sensor (documentation, issues)
First occurred: 10 April 2023 at 13:25:58 (7 occurrences)
Last logged: 10 April 2023 at 13:25:59

    Sensor sensor.ender_3_s1_print_projected_total_duration has device class duration, state class None and unit s thus indicating it has a numeric value; however, it has the non-numeric value: (<class 'str'>); Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
    Sensor sensor.ender_3_s1_print_time_left has device class duration, state class None and unit s thus indicating it has a numeric value; however, it has the non-numeric value: (<class 'str'>); Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
    Sensor sensor.ender_3_s1_print_duration has device class duration, state class None and unit s thus indicating it has a numeric value; however, it has the non-numeric value: (<class 'str'>); Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
    Sensor sensor.ender_3_s1_filament_used has device class None, state class None and unit m thus indicating it has a numeric value; however, it has the non-numeric value: (<class 'str'>); Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
    Sensor sensor.ender_3_s1_progress has device class None, state class None and unit % thus indicating it has a numeric value; however, it has the non-numeric value: (<class 'str'>); Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.

Thank you we will 👀


Updated to gestalt 5.0.1

Changed assets to use the new disposal hook.
GC'ed assets are now disposed once a frame.

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/270/
Hooray Jenkins reported success with all tests good!

Tested around some in different gameplay templates, doing what I could to mess with things. All looks well :-)


Fixed bug in Optimetrics parametric analysis (issue #2021)

Close #2021
bug n.1

Icepak has the key FastOptimetrics instead of CopyMesh within the ProdOptiSetupDataV2 dictionary.

Solution: After import of the base dictionary from OptimetricsTemplates, if app=icepak then the dictionary is corrected
bug n.2

The multiple "add" lines within the "NAME:Sweep Operations" section makes it incompatible with the dictionary structure.

Solution: use of a single add key in the dictionary with a list of lists of variations. Changes in LoadAEDTFile and DesignXPloration to account for this ad-hoc treatment
bug n.3

The dictionary related to the parametric setup remains empty when using the add_from_file function (in DesignXPloration, the function calls the ImportSetup native API in the Optimetrics module).
Solution: instead of using the native API, emulate it by defining the props dictionary within the function

@wilove997 @maxcapodi78 I fixed the IPython problem and I also fixed an small bug (it was missing the first point of the table). Please review it and merge it.


chore: release EntityFrameworkCore.Spanner v0.2.0

Changes in Google.Cloud.EntityFrameworkCore.Spanner version 0.2.0:

Commit 6664d8d test: add test to verify the use of DDL batches (#92)
Commit 68256c9 chore(deps): update dependency xunit to v2.4.1 (#97)
Commit e5d6018 chore: add renovate.json (#85)
Commit 85dd134 build: split build actions and skip integration tests on prod from forks (#93)
Commit 5b89350 chore: add history.md (#94)
Commit 7a35e22 chore: add autorelease scripts (#90)
Commit 1d01f5a feat: support JSON data type (#91)
Commit 4e510a7 monitoring: add custom client header for efcore (#89)
Commit 346115b chore: add CONTRIBUTING.md (#88)
Commit fa4aafe chore: add SECURITY.md (#87)
Commit 7644fe5 chore: add a Code of Conduct (#86)
Commit 4c43965 docs: add sample for using query hints (#84)
Commit c520460 chore: add missing license headers to files (#82)
Commit b846eb3 feat: propagate commit timestamp to current DbContext (#79)
Commit 730e4bf test: enable test assertion for emulator (#78)
Commit 122db74 docs: instructions on how to build and publish NuGet package locally. (#77)
Commit c0d13dc feat: validate entity model against database (#76)

Packages in this release:

Release Google.Cloud.EntityFrameworkCore.Spanner version 0.2.0

Created release for Google.Cloud.EntityFrameworkCore.Spanner-0.2.0


Interactive mode not working on windows?

when using interactive mode my model fails to generate any text. I am using windows 10. thanks so much for pyllamacpp!

Hi @JosephHampton,
Thank you for using pyllamacpp.
interactive mode is supported out of the box now, please take a look at interactive dialogue.


Add support for n-controlled gates

We add support for creating n-controlled gates using the decompositions described by Barenco et al. (https://arxiv.org/abs/quant-ph/9503016)
For this, we have added the following features:

Intermediate gate creation and memoization to create the appropriate circuits.
Recursively create n-controlled gates using qubit optimal and gate-depth optimal composition strategies.
Special cases for optimisation of gate-calls where appropriate.
Added example of n-controlled gate use case.
Unit tests for the standard and optimised special cases of the n-controlled (NCU) gate with U={Pauli-X, Pauli-Y, Pauli-Z}.

Hi there,
Just out of curiousity, does qHipster allow the direct simulation of an n-controlled unitary without decomposition? I.e. with the same runtime cost as a singly-controlled gate?

Hi @TysonRayJones to my knowledge this functionality does not exist in Intel-QS. Our proposed PR adds support for n-controlled gates using several decomposition strategies (using direct recursive, simple optimised known cases, or using auxiliary qubits to reduce depth).
Direct simulation without decomposition would be nice, but I am not aware of how this could be achieved to ensure adjustment of all coefficients in the appropriate manner. The above decomposition approach does achieve very good performance and utilisation of system resources. We have run it using between 8-14 control lines in recent experiments, and had good success using additional nodes in MPI with it.

Ah I see. I ask, because a direct n-controlled gate is actually quite straightforward if a 1-controlled gate already exists (and it's obviously faster and much more precise). Effecting an n-controlled gate involves modifying less amplitudes than the 1-controlled gate modifies.
The (serial) logic for the 1-controlled gate is:
for each amplitude:
    if control qubit is 1:
        update the amplitude according to the gate

The logic for the n-controlled gate is then simply:
for each amplitude:
    if all control qubits are 1:
        update the amplitude according to the gate

Even in a distributed application, evaluating the condition all control qubits are 1 is a simple bitwise test.
I implemented a distributed n-controlled gates in the Oxford simulator I work on (see here). While there's a bit of boilerplate there, the "main" part (that distinguishes it from the 1-controlled case) is here. Happy to explain the O(1) all control qubits are 1 test further if interested.
While supporting a decomposition to 1 and 2 qubit gates is certainly very useful to emulate precisely what a quantum computer (with those constraints) would effect (to e.g. capture noise accurately), it's unnecessary if one wishes to just effect the gate on the state. I'm happy to help out if you think this is worth adding to qHipster

Ah, I understand. That sounds sensible, though you are correct in saying that matching the device (1 and 2 qubit) allowed gate-calls was the plan for the above work. I think it may be best to do this in another issue/PR following the merging of this work, as this is partly the result of an external collaborative project from the following repo: https://github.com/ICHEC/QNLP
Though, I am also interested in your implementation of this from a pure simulation perspective, and would like to listen in if the Intel team are interested in pursuing this.

Hi Tyson, hi Lee,
Sorry for the late reply.
I think that including a direct n-qubit controlled gate is a great idea.
For how I see it, there are a few possibilities.

One possibility is a generalization of the 1-qubit controlled gate following the reasoning of Tyson and paying attention of which among the control qubits and target qubit are local/global. There are a couple of utility functions that could be helpful here, for example "check_bit()" defined inline in qureg.hpp.
Another approach is focusing on gates that are diagonal in the computational basis. In this case MPI communication is not needed and the gates are really simple to implement (look at the functions in "qaoa_features.cpp" since the implementation would be similar but with computing phases on the fly). In this approach, one could convert a CCX (2-qubit controlled X, non diagonal) to a CCZ (2-qubit controlled Z, diagonal) simply by applying Hadamard before and after the CCZ. I expect the overhead to be less than 3x (and most probably 2s) for reasonable implementations and all n-qubit controlled gates.

I agree with Lee that this should be a separate pull request.
Happy to discuss more about it.
Gian

Hi all, sorry for the long delay, but only recently had time to revisit this. I have not been able to replicate the errors that appear on the Github Action test. Running locally on my Mac natively, in Docker, and inside a localised Github action version all pass. Simiarly, the same things are observed on the HPC system. Any thoughts as for why this is happening would be greatly appreciate. I initially thought it may be some thread safety issues with the Google testing framework on the Action, but it seems that may not be the case either.


Elasticsearch adaptor stability

the current version is out of date with all of the breaking API changes that have been made between versions 1.X -> 2.X -> 5.X

Ideally, we'd not have to use a library since the protocol is http but given the complexities of elasticsearch APIs and the varying degree of changes between version it seems inevitable.
After reviewing an alternative library elastic,  it seems to be more actively maintained and the ability to support different versions of elasticsearch is fits well with the new interfaces coming in #220.

addressed with #233, will be cutting a versioned release soon


No exposure of a TextStyle property to change styling of annotation tools list

I cannot for the life of me see a way to pass a textstyle to the list of tools (Line, Text, Arrow etc) when the option is tapped from the toolbar. The examples on pub.dev show a tools list with formatted text but I cannot see how it's done?
This is what it looks like out of the box?

I've had to edit the SelectionItems class directly in the source, which is not what I want to be doing as it'll get overwritten and I dont want to have to make a local copy of the source.


[Doc] create a smart tutorial for JWS operations API

in the /docs folder

Done here
https://github.com/peppelinux/spid-cie-oidc-django/blob/dev/docs/JWT_SIGNATURE_ENCRYPTION_API.md


[BUG] render problem with lazy load component

if using ngif to handle component loading, like check if user login then render
button will case a render problem, since .Net have wrong properties and variables of that
component, and when Angular render that component then it will request props and vars
from .Net and here the problem.

Still not fixed, REOPEN.


user12986714: Watch (?-i:Sa7ndxKnUmA)(?#youtu.be)

user12986714 requests the watch of the watch_keyword (?-i:Sa7ndxKnUmA)(?#youtu.be). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:Sa7ndxKnUmA)(?#youtu.be) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


FileContentDsc fails in winget configure

Brief description of your issue
I am experimenting with the new winget configure system, and loving it so far. I wanted to start using it to edit some files, and selected the FileContentDsc module published by DSC Community.
The KeyValuePairFile resource does not work correctly through winget configure, though it works when invoked in Powershell directly.
I debated where to file this issue and eventually decided to at least start here because the resource works correctly when invoked directly from Powershell but fails when invoked from winget.
I've provided input files, command output and logs in a gist.
Steps to reproduce

Create the following file, replacing <username> with your username, or choose another writable path:

# save as filecontentdsc.yaml
properties:
  configurationVersion: '0.2.0'
  resources:
    - directives:
        allowPrerelease: true
      resource: FileContentDsc/KeyValuePairFile
      settings:
        Ensure: Present
        Name: foo
        Path: "C:\\Users\\<username>\\test.conf"
        Text: bar

Create the file "C:\Users\<username>\test.conf" with the content:

foo=baz

winget configure .\filecontentdsc.yaml --verbose-logs --accept-configuration-agreements

Expected behavior
I would expect the file to be updated to read:
foo=bar

Actual behavior
The configure command errors out with:
PS C:\Users\egerlach\src\dsc-test> winget configure .\filecontentdsc.yaml --verbose-logs --accept-configuration-agreements
Apply :: KeyValuePairFile
  C:\Users\egerlach\test.conf: Set foo to bar
  Module: FileContentDsc by DSC Community [Local]
    https://github.com/dsccommunity/FileContentDsc
    DSC resources for for manipulating the content of text files.
  Settings:
    Path: C:\Users\egerlach\test.conf
    Ensure: Present
    Name: foo
    Text: bar
You are responsible for understanding the configuration settings you are choosing to execute. Microsoft is not responsible for the configuration file you have authored or imported. This configuration may change settings in Windows, install software, change software settings (including security settings), and accept user agreements to third-party packages and services on your behalf.  By running this configuration file, you acknowledge that you understand and agree to these resources and settings. Any applications installed are licensed to you by their owners. Microsoft is not responsible for, nor does it grant any licenses to, third-party packages or services.
Apply :: KeyValuePairFile
  The configuration unit failed while attempting to test the current system state.
Cannot process argument transformation on parameter 'Encoding'. 'byte' is not a supported encoding name. For information on defining a custom encoding, see the documentation for the Encoding.RegisterProvider method. (Parameter 'name')
Cannot index into a null array.
<See the log file for additional details>
Some of the configuration was not applied successfully.

Full log
Environment
PS C:\Users\egerlach> winget --info
Windows Package Manager v1.6.2771
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.2428
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.21.2771.0

Winget Directories
-----------------------------------------------------------------------------------------------------------------------
Logs                               %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\Diag…
User Settings                      %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\sett…
Portable Links Directory (User)    %LOCALAPPDATA%\Microsoft\WinGet\Links
Portable Links Directory (Machine) C:\Program Files\WinGet\Links
Portable Package Root (User)       %LOCALAPPDATA%\Microsoft\WinGet\Packages
Portable Package Root              C:\Program Files\WinGet\Packages
Portable Package Root (x86)        C:\Program Files (x86)\WinGet\Packages
Installer Downloads                %USERPROFILE%\Downloads

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Admin Setting                             State
--------------------------------------------------
LocalManifestFiles                        Disabled
BypassCertificatePinningForMicrosoftStore Disabled
InstallerHashOverride                     Disabled
LocalArchiveMalwareScanOverride           Disabled

PS C:\Users\egerlach\src\workstation-setup> Get-Module

ModuleType Version    PreRelease Name                                ExportedCommands
---------- -------    ---------- ----                                ----------------
Manifest   2.0.0      preview00… FileContentDsc
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Management     {Add-Content, Clear-Content, Clear-Item, Clear-It…
Manifest   <IP_ADDRESS>               Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Ob…
Script     <IP_ADDRESS>               PackageManagement                   {Find-Package, Find-PackageProvider, Get-Package,…
Script     2.2.5                 PowerShellGet                       {Find-Command, Find-DscResource, Find-Module, Fin…
Script     2.0.7                 PSDesiredStateConfiguration         {Configuration, Get-DscResource, Invoke-DscResour…
Script     2.2.6                 PSReadLine                          {Get-PSReadLineKeyHandler, Get-PSReadLineOption, …

I did a couple of tests based on the error.
I tried to set Encoding: ASCII even though the Wiki states it's the default since the error was Encoding. byte is not a supported encoding name.
If the file doesn't exist, the configuration will succeed and create the file. I think this might be a bug in the DSC Resource.
How did you test with PowerShell directly?

PS C:\Users\egerlach\src\workstation-setup> cat C:\Users\egerlach\test.conf
foo=false
PS C:\Users\egerlach\src\workstation-setup> Invoke-DscResource -Name KeyValuePairFile -Method Set -ModuleName FileContentDsc -Property @{
>>     Path   = 'c:\Users\egerlach\test.conf'
>>     Name   = 'foo'
>>     Ensure = 'Present'
>>     Text   = 'bar'
>> }

RebootRequired
--------------
         False

PS C:\Users\egerlach\src\workstation-setup> cat C:\Users\egerlach\test.conf
foo=bar

It could absolutely be a bug in the resource. I'll admit I'm a n00b with DSC, though I know my way around PS okay. Based on what I was reading in the error logs it looked like it might be a problem with how this particular resource was invoked rather than with the resource itself. I'm not set up to debug through the winget-cli code into the resource invocation though, so I didn't know how I could proceed further.

Do you see the same thing in terms of calling Invoke DSC Resource failing to modify the key value pairs when the file exists with a different value?
It seems to work when there is no file (it creates one with the right key value pair), and it seems to return $true when the file has the correct pair. It only fails for me to change the value of the "value" in the pair.


Automate Garden Linux VM version bump

Description
This PR adds some logic to our CI in order to automate creation of a PR that bumps the Garden Linux VM used for testing when a new release for that distribution is done.
The bumper.py script will make a request to the GitHub API asking for the latest release of Garden Linux, then compare the version in its assets to the one configured in the repository and update the repo if the major or minor versions are bigger in the latest release. The Garden Linux version was moved to a separate file in order to prevent automation from changing anything other than that file.
The new workflow is set to run daily, this shouldn't be a big deal since Garden Linux releases at most every couple of weeks, the action used for creating the PR will not do anything if the latest version still matches the one in the repo (see https://github.com/marketplace/actions/create-pull-request for more details on how the action works).
Checklist

[ ] Investigated and inspected CI test results

Testing Performed

[x] A new PR should be created from this one, since the Garden Linux VM has not been updated on master yet.
[ ] Run with all-integration-tests to ensure the Garden Linux test still works correctly.

Just looked over the recent changes, and it still LGTM


Test / add segment & tier1 gateway testcases

Add nsxt-segment testcases
Add tier-1 Gateway testcases

@umeshkumhar : stopping tests as the backends.tf are not aligned on the resource type in the fixtures .
Also @renuka-kumari : please add your comments

Closing as it is a duplicate task.


Eureka Peer Server throwing java.lang.NullPointerException: null at com.netflix.eureka.registry.AbstractInstanceRegistry.renew

I have two Eureka server instances configured with peer awareness. I start the first server and wait for a while, seeing it looking for the second one:
2016-05-30 11:04:53.278  WARN 4936 --- [nfoReplicator-0] c.n.d.s.t.d.RetryableEurekaHttpClient    : Request execution failure
2016-05-30 11:04:53.278  WARN 4936 --- [nfoReplicator-0] com.netflix.discovery.DiscoveryClient    : DiscoveryClient_UNKNOWN/EXPRF026.suministrador.local:8761 - registration failed Cannot execute request on any known server

com.netflix.discovery.shared.transport.TransportException: Cannot execute request on any known server

Then I launch the second instance, and the first one registers it:
INFO 7720 --- [nfoReplicator-0] com.netflix.discovery.DiscoveryClient    : DiscoveryClient_UNKNOWN/clouddev:8761: registering service...
INFO 7720 --- [nfoReplicator-0] com.netflix.discovery.DiscoveryClient    : DiscoveryClient_UNKNOWN/clouddev:8761 - registration status: 204
INFO 7720 --- [nio-8761-exec-3] c.n.e.registry.AbstractInstanceRegistry  : Registered instance UNKNOWN/clouddev:8761 with status UP (replication=true)

Looking at the second server I see a NullPointerException in the log, although everything seems to work fine:
 INFO 5524 --- [nio-8762-exec-1] o.s.web.servlet.DispatcherServlet        : FrameworkServlet 'dispatcherServlet': initialization completed in 22 ms
DEBUG 5524 --- [nio-8762-exec-1] o.s.web.servlet.DispatcherServlet        : Servlet 'dispatcherServlet' configured successfully
 INFO 5524 --- [nio-8762-exec-1] c.n.e.registry.AbstractInstanceRegistry  : Registered instance UNKNOWN/clouddev:8762 with status UP (replication=true)
 INFO 5524 --- [nio-8762-exec-2] c.n.e.registry.AbstractInstanceRegistry  : Registered instance UNKNOWN/clouddev:8762 with status UP (replication=true)
 WARN 5524 --- [nio-8762-exec-3] c.n.e.registry.AbstractInstanceRegistry  : DS: Registry: lease doesn't exist, registering resource: UNKNOWN - clouddev:8761
 WARN 5524 --- [nio-8762-exec-3] c.n.eureka.resources.InstanceResource    : Not Found (Renew): UNKNOWN - clouddev:8761
 INFO 5524 --- [nio-8762-exec-5] c.n.e.registry.AbstractInstanceRegistry  : Registered instance UNKNOWN/clouddev:8761 with status UP (replication=false)
 INFO 5524 --- [nio-8762-exec-6] c.n.e.registry.AbstractInstanceRegistry  : Registered instance UNKNOWN/clouddev:8761 with status UP (replication=false)
 INFO 5524 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Disable delta property : false
 INFO 5524 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Single vip registry refresh property : null
 INFO 5524 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Force full registry fetch : false
 INFO 5524 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Application is null : false
 INFO 5524 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Registered Applications size is zero : true
 INFO 5524 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Application version is -1: false
 INFO 5524 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : Getting all instance registry info from the eureka server
 INFO 5524 --- [freshExecutor-0] com.netflix.discovery.DiscoveryClient    : The response status is 200
ERROR 5524 --- [nio-8762-exec-7] c.n.e.resources.PeerReplicationResource  : Heartbeat request processing failed for batch item UNKNOWN/clouddev:8762

java.lang.NullPointerException: null
	at com.netflix.eureka.registry.AbstractInstanceRegistry.renew(AbstractInstanceRegistry.java:379) ~[eureka-core-1.4.6.jar:1.4.6]
	at com.netflix.eureka.registry.PeerAwareInstanceRegistryImpl.renew(PeerAwareInstanceRegistryImpl.java:401) ~[eureka-core-1.4.6.jar:1.4.6]
	at org.springframework.cloud.netflix.eureka.server.InstanceRegistry.renew(InstanceRegistry.java:127) ~[spring-cloud-netflix-eureka-server-1.1.0.RELEASE.jar:1.1.0.RELEASE]
	at com.netflix.eureka.resources.InstanceResource.renewLease(InstanceResource.java:112) ~[eureka-core-1.4.6.jar:1.4.6]
	at com.netflix.eureka.resources.PeerReplicationResource.handleHeartbeat(PeerReplicationResource.java:148) ~[eureka-core-1.4.6.jar:1.4.6]
	at com.netflix.eureka.resources.PeerReplicationResource.dispatch(PeerReplicationResource.java:113) ~[eureka-core-1.4.6.jar:1.4.6]
	at com.netflix.eureka.resources.PeerReplicationResource.batchReplication(PeerReplicationResource.java:85) ~[eureka-core-1.4.6.jar:1.4.6]
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[na:1.8.0_74]
	at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) ~[na:1.8.0_74]
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) ~[na:1.8.0_74]
	at java.lang.reflect.Method.invoke(Unknown Source) ~[na:1.8.0_74]

I found a similar issue but I checked my dependencies and think are ok.
pom.xml:
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
	xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
	<modelVersion>4.0.0</modelVersion>

	<groupId>com.codependent.storyteller</groupId>
	<artifactId>archimedes</artifactId>
	<version>0.0.1-SNAPSHOT</version>
	<packaging>jar</packaging>

	<name>Archimedes</name>
	<description>Demo project for Spring Boot</description>

	<parent>
		<groupId>org.springframework.boot</groupId>
		<artifactId>spring-boot-starter-parent</artifactId>
		<version>1.3.5.RELEASE</version>
		<relativePath/> <!-- lookup parent from repository -->
	</parent>

	<properties>
		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
		<java.version>1.8</java.version>
	</properties>

	<dependencies>
		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-actuator</artifactId>
		</dependency>

		<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-starter-eureka-server</artifactId>
		</dependency>

		<dependency>
			<groupId>org.springframework.boot</groupId>
			<artifactId>spring-boot-starter-test</artifactId>
			<scope>test</scope>
		</dependency>
	</dependencies>

	<dependencyManagement>
		<dependencies>
			<dependency>
				<groupId>org.springframework.cloud</groupId>
				<artifactId>spring-cloud-dependencies</artifactId>
				<version>Brixton.RELEASE</version>
				<type>pom</type>
				<scope>import</scope>
			</dependency>
		</dependencies>
	</dependencyManagement>

	<build>
		<plugins>
			<plugin>
				<groupId>org.springframework.boot</groupId>
				<artifactId>spring-boot-maven-plugin</artifactId>
			</plugin>
		</plugins>
	</build>

	<repositories>
		<repository>
			<id>spring-snapshots</id>
			<name>Spring Snapshots</name>
			<url>https://repo.spring.io/snapshot</url>
			<snapshots>
				<enabled>true</enabled>
			</snapshots>
		</repository>
		<repository>
			<id>spring-milestones</id>
			<name>Spring Milestones</name>
			<url>https://repo.spring.io/milestone</url>
			<snapshots>
				<enabled>false</enabled>
			</snapshots>
		</repository>
	</repositories>

</project>

application.yml:
---
spring:
  profiles: archimedes1

server:
  port: ${PORT:8761}

eureka:
  instance:
    hostname: archimedes1
  client:
    serviceUrl:
      defaultZone: http://archimedes2:8762/eureka/

logging:
    level:
        org.springframework.web: 'DEBUG'
---

spring:
  profiles: archimedes2

server:
  port: ${PORT:8762}

eureka:
  instance:
    hostname: archimedes2
  client:
    serviceUrl:
      defaultZone: http://archimedes1:8761/eureka/

logging:
    level:
        org.springframework.web: 'DEBUG'

ArchimedesApplication:
package com.codependent.storyteller;

import org.springframework.boot.autoconfigure.SpringBootApplication;
import org.springframework.boot.builder.SpringApplicationBuilder;
import org.springframework.cloud.netflix.eureka.server.EnableEurekaServer;

@EnableEurekaServer
@SpringBootApplication
public class ArchimedesApplication {

    public static void main(String[] args) {
    	new SpringApplicationBuilder(ArchimedesApplication.class).web(true).run(args);
    }
}

Hi @codependent
Just wanted to let you know that I have the same issue with our 2 peer setup. It seems servers takes some times to be able to gossip properly. After a minute or so that both servers are online, the NullPointerException goes away. I didn't took the time to investigate this issue further.

@daniellavoie it doesn't seem important, since everything is working, it's just that it raises a false positive error in our monitoring.

@daniellavoie looks like a simple bug in eureka. A PR or issue there https://github.com/Netflix/eureka would be welcome.

T算是人口參與食用色素人喪生測試欸死欸欸月月欸願望次於王子錯誤我從未為維護😘😀😃😇😆🍝🍜🍞🍍🍆🍤🍝🍥⏩🔼
Daniel Lavoie<EMAIL_ADDRESS>於 2016年6月1日 星期三寫道：

Hi @codependent https://github.com/codependent
Just wanted to let you know that I have the same issue with our 2 peer
setup. It seems servers takes some times to be able to gossip properly.
After a minute or so that both servers are online, the NullPointerException
goes away. I didn't took the time to investigate this issue further.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/spring-cloud/spring-cloud-netflix/issues/1065#issuecomment-222872147,
or mute the thread
https://github.com/notifications/unsubscribe/AE3qYmP8m3OsiPPqD3mbO6C9eblmxUaJks5qHOajgaJpZM4IpoLY
.

--
Sent from my phone

I encountered the same error, while the replicas shows in the "unavailable-replicas".

Re-posting from the eureka repo:
Looking at the the issue on this thread and in the eureka codebase, I see that the NPE is due to an out of order creation of a metrics counter. This is a harmless NPE that should only happen at init time. We will take a look at a simple PR to address this regardless.
The details: this is caused due to the second server not completing initialization (specifically, the openForTraffic() method on the local registry that creates the metrics counter in question) before having the first server heartbeat to the second server, and trying to log the renew metrics on an as yet non-existent metrics counter.


Updated material based on feedback received

Updated the following:

Moved sections from the learning lesson  to the design lesson
Added a section in the assessment lesson
Updated the motivation section with a new exercise
Added a new lesson for the final exercise

I am not sure why the linting is failing there. Any idea @shiltemann or @hexylena?

@bebatut you can see the linting messages on the "files changed" field, it comments on the specific lines. Here it's the linter that bans all embedded YouTube videos due to our past problems with those ;) you can turn that linter off for your specific tutorial though, just add <!-- GTN:IGNORE:002 -->

I saw the linting messages. I was just confused that it fails with only warnings :)
Where should I add <!-- GTN:IGNORE:002 -->? In the tutorial.md file?

Anywhere is fine, bottom, top, first instance of it, your preference.
yeahh it is a warning, you're right, we should not have it fail on warnings huh.

Hi! I included a few changes in this PR https://github.com/BiodataAnalysisGroup/training-material/pull/6


Upgrade actions/checkout

GitHub Action: upgrade the actions/checkout action to v4.

Please just turn on dependabot instead to auto-upgrade these. urllib3 has an example for configuring it to do exactly that.

Maybe we should change it from monthly, to weekly? Looking around most go for daily (which is my pref)
https://github.com/PyCQA/bandit/blob/main/.github/dependabot.yml#L7

While this change is fine, as noted by @lukehinds and @sigmavirus24, a better fix is to update the schedule of the dependabot which is automatically checking for newer version of the actions.

Maybe we should change it from monthly, to weekly? Looking around most go for daily (which is my pref)
https://github.com/PyCQA/bandit/blob/main/.github/dependabot.yml#L7

FYI, I think this workflow isn't running because its in the wrong path. I opened issue https://github.com/PyCQA/bandit/issues/1055

Also if actions aren't run frequently enough, GitHub disables them so it may have disabled monthly dependabot

FYI, created PR #1057 to repair the dependabot file and also update the checks to weekly.

Closing in favor of #1057


Fix build MediaInfoShellExt

MediaInfoShellExt_.cpp(5): fatal error C1083: Cannot open include file: 'MediaInfoDLL/MediaInfoDLL_Static.h': No such file or directory

I don't see the reason it would fix something if the directory name is MediaInfoLib.
But it is good to not depend of the directory name, so patch is relevant


simple-horizontal-bar-chart issue with displaying values

Submit an Issue
Hi,
Expected behavior and actual behavior:
Expected:

Actual:

Environment (component version number, Browser (including version), operating system, hardware, etc):
Browser: Tried in Chrome(56.0.2924.87) and Safari(9.1.3) - issue persists in both browsers
OS: Mac
component version number: "version": "0.6.8"
Note: when I minimize the browser window and maximize the window back the values appear fine. The issue occurs only when the page loads.

I'm not seeing this issue in our demo page or in a CodePen. It looks like you're several versions behind (it's now at v0.8.5) - could you try running bower update and see if the issue still persists in the latest version?

Thanks randyaskin.
I tried updating the bower components by running bower update  but I am having  issues dealing component dependencies --
`Unable to find a suitable version for px-theme, please choose one by typing one of the numbers below:
1) px-theme#~1.1.2 which resolved to 1.1.4 and is required by px-popover#1.5.1, px-vis#0.5.13, px-vis-timeseries#0.3.6, px-vis-xy-chart#0.1.3
2) px-theme#^1.1.2 which resolved to 1.1.4 and is required by predix-seed
3) px-theme#^2.0.1 which resolved to 2.0.4 and is required by polymer-font-awesome#0.1.11, px-calendar-picker#0.5.18, px-card#0.6.10, px-card#0.5.14, px-chart#1.7.8, px-context-browser#1.6.11, px-data-table#1.14.22, px-datetime-common#0.6.17, px-datetime-field#0.5.13, px-datetime-range-panel#0.3.11, px-deck-selector#1.6.7, px-dropdown#2.0.8, px-overlay#0.3.10, px-percent-circle#0.1.12, px-polymer-font-awesome#0.1.11, px-polymer-font-awesome#1.1.5, px-rangepicker#1.2.15, px-sample-cards#0.6.6, px-simple-bar-chart#1.6.14, px-simple-horizontal-bar-chart#0.6.15, px-simple-line-chart#1.5.11, px-spinner#0.5.3, px-tooltip#0.9.19, px-tooltip#0.10.18, px-validation#0.3.8, px-view-header#0.1.10, px-widget-cards#2.1.11, px-widget-cards#2.3.1
4) px-theme#^2.0.2 which resolved to 2.0.4 and is required by px-simple-bar-chart#1.9.1
5) px-theme#^2.0.4 which resolved to 2.0.4 and is required by px-app-nav#1.10.9
Prefix the choice with ! to persist it to bower.json`
I selected based on the components I am using and when I built it the component disappeared. Can you let me know how to deal with these dependencies during updating the components or installing them?

Generally when this happens, you just need to rm -rf bower_components and re-run bower install

Any luck?

Nope. Tried it again today.

On second thought, I picked a bad time to ask. :)  Martin just started issuing major revisions to our components as part of the design refresh underway, which is going to cause temporary resolution issues all over the place until we can get all of the foundational elements cut over. For the time being, you can just choose the latest version (the largest numeric choice) when running bower update.

I think the problem this is via the seed, yes? Have you updated the seed? Even with the latest seed, you'll probably run into this problem because it is behind

Ok. When can we expect the latest revisions?
Thank you

The major revisions Randy is talking about are occurring this quarter. I dont know of an ETA when they will be done.
If you are using the seed, that is developed by another group so I can give no info on it or when it will be up to date with our components. You may want to open an issue on the seed if it's components are out of date.


[AutoRelease] t2-playwrighttesting-2024-01-23-75657(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4887
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=6427

issue link:https://github.com/Azure/sdk-release-request/issues/4887


What is the Roadmap of project dao-chain?

Hi, All,
I found that this project combines the most popular technology of the world, docker and blockchain.
Can you give us more details of the Roadmap of this Project?
Thanks a lot.

Good point!
First of all, We are expecting a good response from the first users, who use DaoChain as an immutable, decentralized checksum database for their docker images. Any bug reports, any document improvements are appreciated.
After that, I think we should to add some smart contracts to enable the developer to borrow money if they don't want to be a miner, and to sell their images if they like.
Any good idea? please feel free to talk about it.
Thanks!

Oh, Those are great minds. Cannot wait to see those.
Thanks @hainesc

What's more, as a member of first users who want to try dao-chain, I hope it can support personal accounts on DaoCloud as soon as possible. After all, not all users could afford to open an organization :(

Oh, that is a great suggestion. Thanks @clarkzjw
PTAL
/cc @hainesc @jjyr @Revolution1


Improve UI/UX for sharing magazine content.

#333 fixed #321 by showing the share panel, but this still feels a tad awkward. I think it should:

don’t show the Facebook/Twitter icons in the web view, instead show an Eigen ‘share’ button:

Eigen intercepts that link and passes the href to the share panel

This way the user only gets to see the Facebook or Twitter share options once and also only those that actually have enabled in their iOS system settings.
@orta @katarinabatina Do you agree?

we should make @craigspaeth do this instead in force IMO, us hijacking those buttons is likely to bite us later

@craigspaeth just made share that button ^

@orta @craigspaeth Where’s that at?

The share buttons on Force article pages are going to be under experiment soon. They will most likely not look like the generic share buttons on Eigen and abroad, so dunno how much we can help here. Feel free to follow up with Robert/Owen for more details.

Moving this to #433.


feat: Integrate Grafana Faro

Description
Adds the react integrations and Faro error boundary around the app.
Initializes the default tracing/web integrations for Faro to get us some internal metrics and tracing.
Related Tickets & Documents
N/a - related to ongoing observability and performance work
Mobile & Desktop Screenshots/Recordings
Here's a snapshot after running this locally and getting some of those metrics into our Grafana:

Steps to QA

Run app locally
Modify the environment variables to use the collector and set the env to "beta" to bypass check

Tier (staff will fill in)

[ ] Tier 1
[ ] Tier 2
[ ] Tier 3
[x] Tier 4

[optional] Are there any post-deployment tasks we need to perform?
Need to update Netlify env vars to use these
[optional] What gif best describes this PR or how it makes you feel?

After experimenting abit with this, we can definitely get this in and start getting some good latency data. But there are some missing integrations in the library with NextJS that i think make it abit half baked. We can draft this for now and I can watch to see when they land some Next integrations.

Pushing logs and errors through is working 👍🏼

I say let's get this in and we can keep an eye on their NextJS instrumentation coming through

I'm curious - is the entirety of your app client-side only, or are you doing server-side rendering of components where you're using faro?

is the entirety of your app client-side only, or are you doing server-side rendering of components where you're using faro?

We do have server side rendering. Unfortunately, Faro is not well supported for Next JS yet so we're uncertain the future of this integration.

Closing as this has grown stale and there is still no adoption in Grafana Faro of NextJS. Let's continue to watch https://github.com/grafana/faro-web-sdk/issues/496 and see if there's any movement there.


fixed issue that didn't allow a user to update a document when deleting elements in an array attribute

Use symmetricDifference instead of difference.

difference only returns values that are present in the edited attributes, but not in the originally fetched attribute vales. This always ends up returning an empty array.
I used symmetricDifference as it returns an intersection of difference from both sides and works perfectly for this use case, as the original attributes never change.

What does this PR do?
This PR closes https://github.com/appwrite/appwrite/issues/5476
Test Plan

In addition to the existing tests, I performed manual testing by adding and deleting elements in an array attribute.
I was going to change the way the difference function worked, but realized it was being used in other places, which broke things. I used the symmetricDifference function as it fits in perfectly for this use case.

Related PRs and Issues

https://github.com/appwrite/appwrite/issues/5476

Have you read the Contributing Guidelines on issues?
Yes ✅

Okay I just realized there's another bug.
Although elements can be deleted, existing items cannot be edited. The Update button stays disabled even after changing values. Should I create a new issue for that, or push a commit to fix it in this PR?

Okay so I figured out the issue that prevents updating of a document when editing the elements of an array attribute.
In the code it seems like $work stores the value of edited values. $work is populated by iterating over $doc, but they are referencing the same underlying value, no? Due to this, when $work is populated, $doc is updated as well, which results in both the arrays having no difference.
I can think of 2 solutions to this:

Use JSON.parse(JSON.stringify($doc)) to clone the $doc, and not reference copy.
Use a locally decalred variable instead of formValues[attribute.key][index] in <AttributeItem />, which can be passed into the <Attribute /> component. Obviously, the local variable will be updated when there's a change.

I have only tested with the first approach as it doesn't mess with the functionality outside of the required component.

Okay so I figured out the issue that prevents updating of a document when editing the elements of an array attribute.
In the code it seems like $work stores the value of edited values. $work is populated by iterating over $doc, but they are referencing the same underlying value, no? Due to this, when $work is populated, $doc is updated as well, which results in both the arrays having no difference.
I can think of 2 solutions to this:
1. Use `JSON.parse(JSON.stringify($doc))` to clone the $doc, and not reference copy.

2. Use a locally decalred variable instead of `formValues[attribute.key][index]` in `<AttributeItem />`, which can be passed into the `<Attribute />` component. Obviously, the local variable will be updated when there's a change.

I have only tested with the first approach as it doesn't mess with the functionality outside of the required component.

Hi @safwanyp! Thank you for the PR 😄
I'd say doing a deepClone with JSON.parse should be enough. I'd recommend creating an utility for it in helpers/object.ts

Hey @TGlide, thanks for the feedback and pointers!
I've just pushed a commit that creates a deepClone utility function, and uses it to create a copy of $doc into $work. Tests were successful on local copy. Appreciate any feedback!


Add RSpec/ExpectOutput cop

Fixes #277

@bquorning what do you think of this? I'm interested in looking into possible autocorrect but that can wait for a followup PR.

@bquorning ready for another pass


Update Gemma2 attention scale

Description
Current configuration uses incorrect attention scale. According to deepmind implementation the 2b and 9b versions use sqrt(d_head). This is the default scale in TL.
https://github.com/google-deepmind/gemma/blob/a0504162f99a1c238efb37b8197e711c0f3808fd/gemma/transformer.py#L152-L174
Fixes #693
Type of change
Please delete options that are not relevant.

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

Screenshots
Please attach before and after screenshots of the change if applicable.

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have not rewritten tests relating to key interfaces which would affect backward compatibility

Huh, you seem to be correct my bad. I'm not sure why my sanity checks didn't show an enormous divergence
Can you check if the attention patterns are notably closer to HuggingFace after this change?

HF implementation uses SPDA so I'd need to switch the attention implementation to access the pattern. I compared the output instead (hook_z) across layers

This question is very helpful for me and I am also being confused by this question! How do I adjust the Transformerlens' Gemma to ensure that the Transformerlens Gemma gives the same results as the HF? Which file should I add the above code to?

@mntss Thank you very much for this! @ccp123456 this will be put into a release relatively quickly, so you should be able to make use of this right away. The question of 100% accuracy is a little bit more complicated, and making sure Gemma models are 100% accurate is a pretty high priority task at the moment. If you interested in knowing more, DM me on slack. (If you are not on the slack channel, let me know and I can give you access.)


Embrace static_assert

Mark C++11 static_assert as required. Replace uses of emulation facilities with the real thing.

LGTM, thanks!


Updated subscription-manager instruction

Added 'subscription-manager repos --disable=*' before enabling required repos to prevent hundreds of repos being enabled by default.

This will speed up the following instructions if you attach to a subscription pool with a lot of products attached to it.


Add IssueHunt

Add IssueHunt that is an issue based bounty platform for OSS.
https://issuehunt.io

Thanks!


Multiple instance of L.Control.Layers not working

Instantiate more than one Control Layers on the same map causes an error in console.
"Uncaught TypeError: layer.onAdd is not a function"
About at row 2224:
_layerAdd: function (layer) {
		layer.onAdd(this);
		this.fire('layeradd', {layer: layer});
	},

Instantiate more than one Control Layers on the same map causes an error in console.

No, it doesn't. See https://playground-leaflet.rhcloud.com/coku/edit?html,output
If you think there is really a bug, provide a complete, minimal, reproducible test case.

yes sorry,  the error at console is caused from other


Extend delegations tests

~WIP since it is built on top of #1689~
Fixes #1641
Description of the changes being introduced by the pull request:
Extend test_delegation_graphs with tests for:

max number of delegations
invalid metadata
searching for target info

Please verify and check that the pull request fulfills the following
requirements:

[ ] The code follows the Code Style Guidelines
[ ] Tests have been added for the bug fix or new feature
[ ] Docs have been added for the bug fix or new feature

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (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.745%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
4036

Relevant Lines:
4114

💛  - Coveralls

Thanks for the suggestions, I find them very reasonable, I did several changes to include them.
There is one example not clear to me, I've mentioned it in the comment.


Unable to run nginx on GKE v1.23+

Trying to install nginx component on GKE version v1.23. Pod fails with the following log error
 type: 'Normal' reason: 'CREATE' ConfigMap kube-ingress/nginx-ingress-nginx-controller
E0608 06:53:29.957636       8 reflector.go:127] k8s.io/client-go@v0.19.3/tools/cache/reflector.go:156: Failed to watch *v1beta1.Ingress: failed to list *v1beta1.Ingress: the server could not find the requested resource

Suggesting to divorce to the helm chart distribution and change it in favour official from vendor. See nginx-ingress component

Same for GKE v1.22


Estebanreyl/container registry runtime sdk

Should finally address : https://github.com/Azure/azure-rest-api-specs/issues/6457
Readying a new release of the container registry runtime sdk along with brand new tests for all methods
History for Review (avoids showing most auto generated files):
https://github.com/Azure/azure-sdk-for-net/pull/7081/files/f957010c20a205aba4aae265a577799394ba9bae..97ff4d756099e72f3034938b720c4eef21b7f4b8?file-filters[]=.cs
Included:

An Example program (Would probably be best to break apart and instead use for documentation purposes?)
Functional Unit tests for 100% of the paths
A custom authentication file for our own oauth2

@erich-wang , @isra-fel Any chance I could get an update on what else, if anything, may be missing to get this SDK going?

@AlexGhiondea, could you take a look since it is data plane related PR?

I know its been some time and there's other work but is there any chance we can get this reviewed in the nearby future? V2 is getting really close and its been 2 weeks since the PR was ready.

Will close this in favor of releasing V2 since REST spec just went out,


fix double reconcilation

There is a double reconcilation for every component with useGlobal hook.
It looks like this:

Actual result in console:
App reconcilation
ModuleA reconcilation
App reconcilation
ModuleA reconcilation
Expected result in console:
App reconcilation
ModuleA reconcilation
Here is an example: https://codesandbox.io/s/great-mirzakhani-wntjq?file=/src/App.js:190-197
The issue was introduced by this commit 05f7b3f4314384f1cc. Those forces setState for empty react state.
But globalHook receives initalState which can be used as initial state for React.useState
The fix works correctly

Released as<EMAIL_ADDRESS>;)


initial commit of "Blockhead"

This is a simple udp or ethernet network block server

Looks fine to me..   Push the Button Frank...


ValueError: Input contains infinity or a value too large for dtype('float32').

Hi,
Thanks for your great work on scaden.
I've been recently trying to use scaden to recognize cell fractions.
With the newly released Scaden v1.1.2, I tried to run the 'scaden process' command but resulted in the same error showing
INFO     Scaling using log_min_max                                                                                functions.py:65
Traceback (most recent call last):
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/bin/scaden", line 8, in
sys.exit(main())
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/scaden/main.py", line 48, in main
cli()
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/click/core.py", line 1137, in call
return self.main(*args, **kwargs)
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/click/core.py", line 1062, in main
rv = self.invoke(ctx)
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/click/core.py", line 1668, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/click/core.py", line 763, in invoke
return __callback(*args, **kwargs)
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/scaden/main.py", line 155, in process
processing(
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/scaden/process.py", line 35, in processing
preprocess_h5ad_data(raw_input_path=training_data,
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/scaden/model/functions.py", line 67, in preprocess_h5ad_data
raw_input.X = sample_scaling(raw_input.X, scaling_option)
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/scaden/model/functions.py", line 42, in sample_scaling
x = mms.fit_transform(x.T).T
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/sklearn/base.py", line 699, in fit_transform
return self.fit(X, **fit_params).transform(X)
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/sklearn/preprocessing/_data.py", line 363, in fit
return self.partial_fit(X, y)
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/sklearn/preprocessing/_data.py", line 396, in partial_fit
X = self._validate_data(X, reset=first_pass,
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/sklearn/base.py", line 421, in _validate_data
X = check_array(X, **check_params)
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/sklearn/utils/validation.py", line 63, in inner_f
return f(*args, **kwargs)
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/sklearn/utils/validation.py", line 720, in check_array
_assert_all_finite(array,
File "/home/mhagiwara/Programs/anaconda3/envs/scaden/lib/python3.9/site-packages/sklearn/utils/validation.py", line 103, in _assert_all_finite
raise ValueError(
ValueError: Input contains infinity or a value too large for dtype('float32').

My command for 'scaden process' is:
scaden process tabula_MC38.h5ad dataset
I checked whether infinity data exist or not already.
We were successful when we changed only the genetic names.
2210010C04Rik (couldn't) ⇒ Foxp3 (OK!!)
May I have your suggestion on this issue? Thanks a lot!
Masaki

Hi @mHagiw ,
sorry for the late response. This sounds indeed weird, I can't tell you right away how this happened.
Did you figure it out by any chance, since you closed the issue?

Hi Kevin
I resolved my problem.
I found inf in simulated data(.h5ad), not in bulk data.
So, I could run finally.
Thanks!!
Masaki Hagiwara

Awesome, thanks for the explanation!


No such file or directory 'protein_go_train_triplet_filtered.txt'

The file 'protein_go_train_triplet.txt' exists in ProteinKG25 , but the code requires 'protein_go_train_triplet_filtered.txt'

protein_go_train_triplet.txt and protein_go_train_triplet_filtered.txt are the same file. And we have modified related code for inconsistent file names.


compare-hash behavior when MD5 hash is missing

Which version of the AzCopy was used?
v 18 0
Which platform are you using? (ex: Windows, Mac, Linux)
Windows
What command did you run?
azcopy sync $azUpload$SAS $FilePath --delete-destination=$deleteDestination --log-level INFO --from-to BlobLocal
azcopy sync $azUpload$SAS $FilePath --delete-destination=$deleteDestination --log-level INFO --compare-hash MD5 --from-to BlobLocal
What problem was encountered?
A large 5GB blob with no MD5 hash will get downloaded when using option --compare-hash MD5  even though the same file already exists in the destination.
Preferred would be, that azcopy sets the MD5 hash if it does not exist or would have the option to fall back on the modification date comparison as this would prevent incurring unnecessary costs.
The flag --missing-hash-policy=Generate mentioned in v17 release is not recognized both for Blob to Local and vice versa.
How can we reproduce the problem in the simplest way?
Upload > 5 GB blob with command
azcopy sync $FilePath $azUpload$SAS --delete-destination=$deleteDestination --log-level INFO --from-to LocalBlob
Run above two commands.
Have you found a mitigation/solution?
Ensure using put-md5 parameter when copying or syncing files to blob.

The flag --missing-hash-policy=Generate mentioned in v17 release is not recognized both for Blob to Local and vice versa.

This was a misnomer from how it was in development-- Initially, we had the option to not generate the hash. This is not the case in the official release. Simply having hash-based sync on will generate hashes on the local filesystem.

A large 5GB blob with no MD5 hash will get downloaded when using option --compare-hash MD5 even though the same file already exists in the destination.

No hash on the source or on the destination?
If there's no hash on a blob source, we won't calculate it and we shouldn't transfer it.
If there's no hash on the local destination, it should be getting hashed.

No hash on the source or on the destination?

Source blob has no hash

Source blob has no hash

Gotcha, I'll investigate. Those blobs shouldn't get transferred in the first place. Thanks for the quick response.

No hash on the source or on the destination?

Source blob has no hash

I can't seem to reproduce this. I get the appropriate message File was skipped because the source lacks an associated hash; please upload with --put-md5
Could you do me a favor and azcopy list <blob> --properties ContentMD5? I'm curious to see if you have a funky value in there or something.
Typically, when the service doesn't have a Content-MD5 for us, it returns nothing, and on the SDK side we get a nil back.


Cannot update flake inputs

Whenever I try to do nix flake update or nix-on-droid switch I get this weird error. I have my NixOS and nix-on-droid configs in the same repo and it stopped working at this commit
$ nix-on-droid switch --flake ~/.nix
Building activation package...
error:
       … while fetching the input 'git+file:///data/data/com.termux.nix/files/home/.nix'

       error: cannot link '/nix/store/.links/1isr4xv4akzdabjak4z4wlfcnzbn8kk924chbb3vdm6fy20yyvf2' to '/nix/store/1z7g3yq059v70sbrf2plmc7h96cfzdbb-source/.gitignore': Operation not permitted

https://git.nelim.org/matt1432/nixos-configs/src/commit/ff1002c97055d19fc13902145a854c77332d625d/common/nix-on-droid.nix#L20
Nix store optimise is sometimes causing problems because of the created hardlinks. You probably need to reset nix-on-droid completely to fix this..

@Gerschtli Thank you! That was indeed my issue. Is this mentionned anywhere?
I haven't seen that problem anywhere so it would be nice to have it documented before closing this issue.

Temporary fixed this without resetting by passing --option auto-optimise-store false when building and manually modifying the activate script (specifically the nix-env command during the installPackages phase) to also turn off auto-optimise-store.


Stabilize transactionApi

Stabilize the transactionApi preview feature #3104.

Closed with prisma/prisma-engines#1330

Not yet integrated in CLI & Client.

This isn't working for me when you run prisma generate without the feature flag.
// This is your Prisma schema file,
// learn more about it in the docs: https://pris.ly/d/prisma-schema

datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
}

model User {
  id    Int    @id @default(autoincrement())
  first String
  last  String
  posts Post[]
}

model Post {
  id     Int    @id @default(autoincrement())
  title  String
  body   String
  user   User   @relation(fields: [userId], references: [id])
  userId Int
}

npx @prisma/cli@dev generate

It shows up when I add the feature flag and re-generate.

What version are you testing with @matthewmueller?


ISPN-15315 docs refinement to upgraidng guide, HR property defaults

Documentation review of Upgrading content
https://issues.redhat.com/browse/ISPN-15315

@tristantarrant I was working on downstream upgrading doc and backported the updates.

Thanks @domiborges


Option to disable vendor chunks

In a recent update vite behaviour changed and vite is now creating a vendor chunk but in some case (for an app with only one entrypoint) we may want to get only one file.
Is it possible to add an option to disable it, or did I miss something ?
I looked at the code and this option is linked to other options :(
https://github.com/vitejs/vite/blob/main/packages/vite/src/node/build.ts#L364

You can disable it by setting build.rollupOptions.output.manualChunks to undefined.

You can disable it by setting build.rollupOptions.output.manualChunks to undefined.

You can disable it by setting build.rollupOptions.output.manualChunks to undefined. User options will overwrite the default.
not work, @Grafikart  did you find out ?

  build: {
      output: {
        manualChunks: undefined
      }
    }
  }

works a-okay for me!


ufuncs are missing the accumulate, at, outer, reduce, and reduceat methods

Title says it all. I have used the outer method several times of various ufuncs in numpy, I am sure the other ones are also useful in many cases.

I just found an old issue related to add.at at #1658 which is just one of the functions listed here. Thus, the following could are for a specific ufunc and if they could be adapted to any ufunc most of these would be completed:

cupyx.scatter_add is the same as add.at, cupyx.scatter_max is the same as maximum.at, cupyx.scatter_min is the same as minimum.at
cumsum is the same as add.accumulate and cumprod is the same as multiply.accumulate
sum is the same as add.reduce and prod is the same as multiply.reduce
outer is the same as multiply.outer

The only one missing is reduceat but even just getting 4 out of 5 by generalizing the above would be great.

Thanks for the feature request! I'm closing the issue as #2552 covers this topic, let's continue discussing there.


[BOOK] Typos and Grammar

Hi @SchrodingerZhu,
I've had a chance to read more of the book now, it looks like a good entry-level introduction to Clash. Thanks again for your hard work :smile:
Some typos / other language errors I've found in the book:
Position of the errors

Line 20 of chapter1.tex: "gosing" -> "going"
Line 68 of chapter2.tex: "operatior" -> "operator"
Line 153 of chapter2.tex: "hexdecimal" -> "hexadecimal"
Line  30 of chapter3.tex: The sentence "There is another fantastic thing you can also set a test bench for your circuits." isn't quite grammatically correct. I would recommend "Another fantastic thing you can do is create a test bench for your circuits."
Line 139 of chapter3.tex: "Nothong" -> "Nothing"
Line 3 of chapter4.tex: "What is CPU" -> "What is a CPU"
Line 82 of chapter4.tex: "Infomation" -> "Information"

@alex-mckenna thanks! due to some personal stuffs(school works), I will get back to this repo after May 26th. Sorry for the delay.

should have been fixed in the latest commit.


Pindexer remove unused appviews

Describe your changes
This removes:

The Dex AppView
The ShieldedPool AppView

The Dex AppView shouldn't be needed, given that we have an app view specific to the dex explorer now.
The shielded pool app view was just an example, which can be removed given the many other examples now.
Other AppViews which should be removed in a later PR after making sure they're not used:

the supply AppView

superceded by the insights app view, but is known to be used

the ibc AppView

I think there's ongoing development using this, potentially?

Testing
The other app views should continue to work.
Checklist before requesting a review

[x] I have added guiding text to explain how a reviewer should test these changes.

[x] If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

indexing only

SGTM

the supply AppView, superceded by the insights app view, but is known to be used

Where? If we know for sure, some proactive outreach might be warranted. Also happy to just leave it in place if you are.

https://github.com/radiantcommons/penumbra-token-info uses it ; should be a very simple change


Skip shoots with status.technicalID not set while deleting DWD secrets

How to categorize this PR?

/area cleanup
/kind bug
What this PR does / why we need it:
Skip shoots with status.technicalID not set while deleting DWD secrets to prevent gardenlet panic, if it restarts during while some workerless shoots are getting created and the field is not set.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
/cc @ary1992
Release note:

An issue causing gardenlet to panic while deleting `dependency-watchdog-access` secrets for workerless Shoots if the `status.technicalID` is not set for the Shoot is now fixed.

/cherry-pick release-v1.98

/cc @plkokanov @aaronfern
as reviewers of the original PR.

/approve


APIServer: fix first-letter case of AdditionalCORSAllowedOrigins

The missing quotes in json tag caused it to be ignored.

/lgtm
/approve

/approve

I used the bot account to button-merge this PR, asked by @mfojtik


Pass other input properties to each input

I wanted a way to pass additional input props to my inputs. For example, what if you want the returnKeyType to be different than the default.
additionalInputProps is an object that has keys that correspond to the names of each field. You can pass down any props as documented in the RN docs. Let me know if you need anything else for this PR.

I was picturing the additional props to looks something like this
addtionalInputProps = {
  name: {
    defaultValue: 'my name',
    maxLength: 40,
    placeholderTextColor: 'lightblue',
  },
  postalCode: {
    returnKeyType: 'go',
  },
};

Honestly a lot of the props available on the current package could be bundled into this object (ie labels, placeholders, and so forth). This would make this package more flexible with fewer updates to new RN features. I just wanted to be clear on what this PR was about.

Yep, it's pretty clear 👍
I'm just saying that CCInput and CreditCardInput both have the same additionalInputProps property, but those two are expecting different objects:

CreditCardInput expects:

{
  name: {
    defaultValue: 'my name',
    maxLength: 40,
    placeholderTextColor: 'lightblue',
  },
  postalCode: {
    returnKeyType: 'go',
  },
}

CCInput expects:

{
   returnKeyType: 'go',
}

lmk if you're OK with this & I will merge this later :)
-Sam

I see. Now I understand. I will make the changes when I get home to avoid confusion. Thank you for catching that.

👍
Just had a thought..
Would changing the propTypes as PropTypes.objectOf(TextInput.propTypes) and PropTypes.shape(TextInput.propTypes) respectively makes the props validation works better?
-Sam

I just need to update the readme. Sorry I didn't have the time tonight. I will try to do it by tomorrow.

All commits squashed and README has been updated.

Thanks! I'll test & merge them soon!

I fixed and typo this morning (with default props) and squashed again. Let me know if you need anything else from my end.

Thanks - Fixed.

Thanks again @sscaff1 🎉


updated webstorm (2016.1)

Thank you for the contribution. It needed some fixes, so they were made in 0fd287e6246aa3aad5987309f65b10a7d668fe9f. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


feat: update miden-base dependencies

This PR updates the miden-base dependencies to the current next branch tip.
#407 is currently blocked by the recently merged PR in base/miden-objects. This PR unblocks #407 by updating our dependencies to include it.
I opened this as a separate PR since there are some associated changes that need reviewing.
I'm also unsure of the cross-repo update procedure so maybe this is the wrong way of doing it.
Lastly, probably we don't want to track next so I've specified the current latest commit.

It seems that the no changelog label detection is not working. I would also recommend we add label triggers for that job so that it re-runs when a label is added/removed.


⚠ Spryr has degraded performance

In c4a116f, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 6005 ms

Resolved: Spryr performance has improved in 6bb8242 after 8 minutes.


[FLINK-31597][table] Cleanup usage of deprecated TableEnvironment#registerFunction

What is the purpose of the change
Removes the usage of the deprecated method TableEnvironment#registerFunction.
Brief change log

Replace the usage of TableEnvironment#registerFunction with TableEnvironment#createTemporarySystemFunction

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

ci failed.

Shouldn't it be replaced with createTemporarySystemFunction as mentioned in javadoc?
     * @deprecated Use {@link #createTemporarySystemFunction(String, UserDefinedFunction)} instead.
     *     Please note that the new method also uses the new type system and reflective extraction
     *     logic. It might be necessary to update the function implementation as well. See the
     *     documentation of {@link ScalarFunction} for more information on the new function design.

Also with new approaches to checks types for nullability. And there will be an issue with org.apache.flink.table.planner.runtime.utils.JavaUserDefinedScalarFunctions.NonDeterministicUdf since it has inputs as primitives in java (will be translated to NOT NULL) and used in scala tests for util.addTableSource[(Int, Int, String)]("MyTable", 'a, 'b, 'c) which are nullable.
I guess either test should be moved to java to test it with primitives or function definition should be changed

I tried to cleanup the registerFunction usage. But it seems it is a big effort because of the different behavior between registerFunction and createTemporarySystemFunction, e.g., the error message, the checks of nullability types, the plan digests, etc. I would like to close this PR first, and may separate it into sub-tasks later.


Compared with instant-NGP

Thanks for you great work!
I have a question about that does it improved rendering speed compared with origin CUDA code?

Hi @ElonKou. My implementation is still slower than the original repo instant-ngp. In the Lego scene, the original is 90+ fps while taichi-ngp is around 70. However, compared to the existing python implementation, taichi-ngp is still faster. Also, taichi allows you to run anywhere besides the Nvidia graph card. For example, you can run this code on a Mac through the Vulkan backend.
Note that there is still much room for improvement in taichi-ngp.


Adds Notification List on HomePage with sorting and filtering options.

Closes #30

Does this show all messages from all groups, or does it pre-filter based on the groups that the ODK user belongs to? For example, imagine there are groups: north_clinic, east_clinic. Imagine we have users north_admin (part of north_clinic group), east_admin(part of east_clinic group), and north_east_director (part of north_clinic and east_clinic groups).
If I'm logged in as north_clinic I would expect to only see messages for the north_clinic group (and all other groups messages would be inaccessible to me). If I'm logged in as east_clinic I would expect to only see messages for east_clinic group. If I'm logged in as north_east_director I would expect to see messages from both groups.
Does this PR add the option for noth_east_director to filter based on the multiple groups that he is a part of?

Yes initially it will show all the messages of groups which are subscribed by that user. And by clicking on filter, user can filter messages by group.

@linl33 @jbeorse I have implemented the search functionality according to the developer guide https://developer.android.com/guide/topics/search/search-dialog.
And I think it's not creating any extra instance of MainActivity, I have also checked with the android profiler.


Fix unsafe sound loading

Codecov Report

Merging #233 (f229221) into master (52c4b8d) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #233      +/-   ##
==========================================
- Coverage   11.29%   11.23%   -0.06%
==========================================
  Files         169      169
  Lines        2762     2776      +14
  Branches      914      921       +7
==========================================
  Hits          312      312
- Misses       2299     2313      +14
  Partials      151      151

Impacted Files
Coverage Δ

src/raylib/core/Sound.cpp
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Macro-based diagram syntax does not work in Documenter.jl for any diagram type except PlantUML

Is there an example of mermaid I could use? I only get plantuml to work.

Hi @albinahlback thanks for giving this a shot! I'm sorry for not getting back to you sooner, but my GitHub notifications tend to get out of hand at which point GitHub stops e-mailing me, so I only just noticed...
I've been meaning to get back to this package and adding some examples, but unfortunately, other priorities have gotten in the way. Happy to help out though! Can you elaborate a bit on what's not working for you?
(Parts of) PlantUML are the only diagrams that can be rendered to text at the moment (this is a restriction in the underlying libraries, not something this package can take care of). In all other cases, the textual description of the diagram is rendered instead. However, when the descriptions are created in a rich-media environment all supported diagram types should work "as-is". For instance, rendering the Mermaid example from Kroki's documentation in Pluto results in

Hope this helps, but do let me know if you need further clarification.

Yes, so I would like to use this in the documentation for a package I want to develop. In a markdown file I have then tried to write something like
```@setup diagram
using Kroki
mermaid_diagram = mermaid"""
graph LR
  A --> B
"""
```
```@example diagram
mermaid_diagram #hide
```

which does not work.

That sounds like the perfect use-case, and that should definitely work. But, I've verified it to currently indeed not work... From some quick samples I have tested, it looks like this actually affects all diagram types, except for PlantUML.
Given that the macro-syntax does work when used in the context of Pluto and that PlantUML also seems to work, there must be something at odds here in the context of Markdown and/or Documenter.jl for these diagram types. I'll have to look a bit deeper into that.
That being said, you can still get this to work by using the (slightly) long(er) form syntax like this:
```@setup LongformMermaidDiagram
using Kroki
mermaid_diagram = Kroki.Diagram(:mermaid, """
graph LR
  A --> B
""")
```
```@example LongformMermaidDiagram
mermaid_diagram #hide
```

I hope that's not too off-putting, but I certainly agree it's not ideal. I have actually been working on making the syntax
```mermaid
graph LR
  A --> B
```

work as well in the context of Documenter.jl, so there may be some light on the (distant) horizon. I'm not happy with the current implementation of that though, so I haven't shipped it yet.
I'll repurpose this issue to target the more general issue at hand, so I have a reminder to take a look at it.

Perfect, I will use this then. Thank you very much!
By the way, really good job and a big thank you for creating this package. I think more of the community should use this instead of rendering images.

Thank you for the kind words! I've been using these types of tooling more and more over the past couple of years and am missing them every time they're not readily available. Hence the existence of this package :smile: I think it's mostly a lack of knowledge of the existence of these types of tools, the value of visualization in the context of software documentation, and how easy those visualizations are to do with tooling like this. I'd like to try and change that a bit :smile:
Unfortunately, I haven't been able to use it in the context of my Julia work recently, but I'm looking forward to getting back into that. Feedback like this certainly helps!

I've tried to reproduce this issue again, but can no longer seem to do so. I've tried various Julia versions from v1.4 through to v1.7, but on all of them the string literal syntax works as expected. This can also be seen in the newly added examples section through #11.
This leaves the possibility that it was either an issue with the version of Documenter.jl (but I didn't update the version prior to running the tests) or that something was off with the Kroki service at the time. Either case is a bit difficult to verify, so I'm going to close this as the behavior works as intended again.
I'm sorry for taking this long to address this.


Projector Motions wrong election result

Projector shows at election results (Motions) "-1" or "-2" instead of "Majority" or "undocumented"

Thanks for reporting it! I'll fix it for 2.0.

Fixed with #2015.
@PaulchenP86 Please note also the important motion poll fix in #2017.
So, current master should works for you now. Let us know if not...


[Suggestion] Add chatGPT command

It would be cool to be able to use chatGPT directly in omni

It would be cool to be able to use chatGPT directly in omni

it's implemented in https://github.com/buttercannfly/AIpex,and we are willing to accept feature requests


First Azure AD Login creation note

A comment surrounding the idea of completeness of information based on the options available, and comes from someone new to Azure SQL Database config.
The comment is in regards to the following note on this page:
"Note
The first Azure AD login must be created by the Azure Active Directory admin. A SQL login cannot create Azure AD logins."
We use a group for the Azure Active Directory admin, not a specific user ID.  So my login as that Azure Active Directory admin is really to be done as my ID, as a member of that group that we have listed/specified in the Azure Active Directory admin field on the SQL server instance.  I wonder if it makes sense to mention that option more clearly in this note.
It is also interesting that when I connect to my Azure SQL Database instance (in Azure, listed as SQL server) via SSMS using my ID, as a member of the group I have listed in Azure Active Directory admin field, I must use the "Trust server certificate" option of Authentication "Azure Active Directory - Universal with MFA" in order to get connected.  Not sure this is correct or if I have any other option.

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

ID: 7da12444-113d-4e98-a337-08ece99c45eb
Version Independent ID: 6dc14aee-0616-8edf-d52e-37a6fa0f0dd6
Content: Create and utilize Azure Active Directory server logins - Azure SQL Database
Content Source: azure-sql/database/authentication-azure-ad-logins-tutorial.md
Service: sql-db-mi
Sub-service: security
GitHub Login: @GithubMirek
Microsoft Alias: mireks

@dbgitrdone
Thanks for your feedback! We will investigate and update as appropriate.

Hi @GithubMirek,
Could you please look into this document request. Thanks

Hi @dbgitrdone, thank you for your feedback on our documentation. I'll add in the note that an Azure AD admin can be a user or group. This is already listed in our overview, but it wouldn't hurt to clarify here also.
https://learn.microsoft.com/en-us/azure/azure-sql/database/authentication-aad-overview?view=azuresql#administrator-structure
Regarding your issue with connecting with TrustServerCertificate, there can be many different configurations which will determine connectivity, and would need troubleshooting to understand.

#assign:VanMSFT


Ruler in SQL Editor

Description
I need a horizontal at the top of the SQL Editor ruler to interpret to help quickly identify which token or keyword is in error.  The error notification does not put the cursor in the exact location of an error, so having a ruler will help find things in error faster.
DBeaver Version
22.3
Operating System
Windows 10 Pro
Database and driver
Postgresql
Steps to reproduce
This is a feature request.  It does not exist now.
Additional context
See original description.  This is a feature request.  It does not exist now.

You can observe all problems via Window->Show view->Other...->General->Problems.

It's been a long time since no update here. If suggested solution do not satisfy your needs and you have other ideas, please let me know.


🌱 Prepare for v2 beta1 release

Commit created from this PR will be used to generate the pre-release of scorecard-action. I've decided to name the tag - v2.0.0-beta.1.

(Let's also see if we can drain the remaining dependabot updates before cutting the release)

(Let's also see if we can drain the remaining dependabot updates before cutting the release)

Fair point. Looking into it.

(Let's also see if we can drain the remaining dependabot updates before cutting the release)

Fair point. Looking into it.

Getting errors running go mod tidy:
go mod tidy
go: finding module for package go.opentelemetry.io/otel/metric/registry
go: finding module for package go.opentelemetry.io/otel/semconv
github.com/ossf/scorecard-action/signing imports
        github.com/sigstore/cosign/cmd/cosign/cli/sign imports
        github.com/sigstore/cosign/cmd/cosign/cli/fulcio/fulcioverifier/ctl imports
        github.com/google/certificate-transparency-go imports
        go.etcd.io/etcd/v3 imports
        go.etcd.io/etcd/tests/v3/integration imports
        go.etcd.io/etcd/server/v3/embed imports
        go.opentelemetry.io/otel/semconv: module go.opentelemetry.io/otel@latest found (v1.8.0), but does not contain package go.opentelemetry.io/otel/semconv
github.com/ossf/scorecard-action/signing imports
        github.com/sigstore/cosign/cmd/cosign/cli/sign imports
        github.com/sigstore/cosign/cmd/cosign/cli/fulcio/fulcioverifier/ctl imports
        github.com/google/certificate-transparency-go imports
        go.etcd.io/etcd/v3 imports
        go.etcd.io/etcd/tests/v3/integration imports
        go.etcd.io/etcd/server/v3/embed imports
        go.opentelemetry.io/otel/exporters/otlp imports
        go.opentelemetry.io/otel/sdk/metric/controller/basic imports
        go.opentelemetry.io/otel/metric/registry: module go.opentelemetry.io/otel/metric@latest found (v0.31.0), but does not contain package go.opentelemetry.io/otel/metric/registry

Anyone have an idea?

Is this on your branch?

Is this on your branch?

No, this is on HEAD.

Is this on your branch?

No, this is on HEAD.

Looks like this is the problem https://github.com/open-telemetry/opentelemetry-go/issues/2938#issuecomment-1156255999

Is this on your branch?

No, this is on HEAD.

Looks like this is the problem open-telemetry/opentelemetry-go#2938 (comment)

Thanks, that's a great find! I'll try to see if I can use the comments there to resolve our dependency problem.

Is this on your branch?

No, this is on HEAD.

Looks like this is the problem open-telemetry/opentelemetry-go#2938 (comment)

Thanks, that's a great find! I'll try to see if I can use the comments there to resolve our dependency problem.

It is a replace directive as the project has broken the API, and the downstream hasn't changed. This is dependency hell.

(Let's also see if we can drain the remaining dependabot updates before cutting the release)

Fair point. Looking into it.

Most of them have been merged!

@justaugustus blocked on your approval. Dependabot alerts are now fixed.

Azeem, can you investigate the unit test failures?
I believe it's related to the signing logic.

@justaugustus blocked on your approval. Dependabot alerts are now fixed.

Azeem, can you investigate the unit test failures? I believe it's related to the signing logic.

They are intermittent failures since the issue is with the REST API the test talks to (its a mock request, so the failures are only happening on the test but prod works fine). The fix is going to be on the API side, no changes needed here. Will retry the test to get it to pass.

@justaugustus blocked on your approval. Dependabot alerts are now fixed.

Azeem, can you investigate the unit test failures? I believe it's related to the signing logic.

Fixed now with the retry :)

@justaugustus blocked on your approval. Dependabot alerts are now fixed.

Azeem, can you investigate the unit test failures? I believe it's related to the signing logic.

Fixed now with the retry :)

@justaugustus friendly ping.

@justaugustus friendly ping.

Thanks for the details on the failure.
Approved!


Added admonitions

Description

Added Admonition support as per the standard MD directive syntax :::note

Using remark-directive, rehype-components to convert directives into components

Images

Thanks @Fabrizz for doing this!