Error 57014 while querying Redshift Serverless

Hello !
I have a python function which aims to create a procedure stored in a SQL file (cf see code below).
At the first run, it fails as described below, but it works at the second run.
It's like the execute fails because the cluster is not really UP because it takes time but at the 2nd run it's UP and it works 🤔
Driver version
2023-03-21 10:17:23,505 DEBUG:{'user': 'XXXX', 'database': 'XXX, 'application_name': '__main__', 'replication': None, 'client_protocol_version': '2', 'driver_version': 'Redshift Python Driver 2.0.910', 'os_version': 'macOS-13.2.1-arm64-arm-64bit'}

Redshift version
PostgreSQL 8.0.2 on i686-pc-linux-gnu, compiled by GCC gcc (GCC) 3.4.2 20041017 (Red Hat 3.4.2-6.fc3), Redshift 1.0.48038

Python version
▶ poetry run python3 --version
Python 3.9.1

Problem description

Expected behaviour: To create the function described.
Actual behaviour: Waiting few seconds
Error message/stack trace:

redshift_connector.error.ProgrammingError: {'S': 'ERROR', 'C': '57014', 'M': 'Query cancelled.', 'F': '../src/pg/src/backend/tcop/node_fault_tolerance.c', 'L': '960', 'R': 'NFTSetupSessionStateOnRestore'}

Any other details that can be helpful:
My code is as follow

    LOGGER.info("Connecting To Redshift Serverless")
    with redshift_connector.connect(
        host=setup_host_url(),
        database='dw',
        user=os.environ["REDSHIFT_USER"],
        password=os.environ['REDSHIFT_PASSWORD'],
        is_serverless=True
    ) as conn:
        conn.autocommit = True
        with conn.cursor() as cursor:
            LOGGER.info("Creating function 'process_domain_fhir_reference'")

            with open(f"{PATH_MIGRATION_PROCEDURE}/process_domain_fhir_reference.sql", 'rb') as file:
                raw_sql_file = file.read()
                sql_query = sqlparse.split(raw_sql_file)[0].lstrip().replace("\n", " ")
                LOGGER.info(f"Executing query: \n {sql_query}")
                result = cursor.execute(sql_query)

Python Driver trace logs
2023-03-21 10:17:23,763 INFO:Creating function 'process_domain_fhir_reference'
2023-03-21 10:17:23,766 INFO:Executing query:
 /*  Implement RGDS exception when processing fhir api resources  */  CREATE OR REPLACE function process_domain_fhir_reference (VARCHAR, VARCHAR)   RETURNS VARCHAR STABLE AS $$     SELECT CASE WHEN $1 = 'rgds' AND SPLIT_PART($2, '/', 1) = 'Organization'         THEN 'lifen-' || $2         ELSE    CASE WHEN SPLIT_PART($2, '/', 1) IN ('Practitioner', 'PractitionerRole') THEN $2                 ELSE $1 || '-' || $2  END         END $$ LANGUAGE SQL;
Traceback (most recent call last):
  File "/Users/vincent.martinez/.pyenv/versions/3.9.1/envs/data-engineering/lib/python3.9/site-packages/redshift_connector/core.py", line 1631, in execute
    ps = cache["ps"][key]
KeyError: ("/*  Implement RGDS exception when processing fhir api resources  */  CREATE OR REPLACE function process_domain_fhir_reference (VARCHAR, VARCHAR)   RETURNS VARCHAR STABLE AS $$     SELECT CASE WHEN $1 = 'rgds' AND SPLIT_PART($2, '/', 1) = 'Organization'         THEN 'lifen-' || $2         ELSE    CASE WHEN SPLIT_PART($2, '/', 1) IN ('Practitioner', 'PractitionerRole') THEN $2                 ELSE $1 || '-' || $2  END         END $$ LANGUAGE SQL;", ())

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/Users/vincent.martinez/Software/data-engineering/.circleci/run_test_materialization.py", line 91, in <module>
    result = cursor.execute(sql_query)
  File "/Users/vincent.martinez/.pyenv/versions/3.9.1/envs/data-engineering/lib/python3.9/site-packages/redshift_connector/cursor.py", line 240, in execute
    self._c.execute(self, operation, args)
  File "/Users/vincent.martinez/.pyenv/versions/3.9.1/envs/data-engineering/lib/python3.9/site-packages/redshift_connector/core.py", line 1701, in execute
    self.handle_messages(cursor)
  File "/Users/vincent.martinez/.pyenv/versions/3.9.1/envs/data-engineering/lib/python3.9/site-packages/redshift_connector/core.py", line 1969, in handle_messages
    raise self.error
redshift_connector.error.ProgrammingError: {'S': 'ERROR', 'C': '57014', 'M': 'Query cancelled.', 'F': '../src/pg/src/backend/tcop/node_fault_tolerance.c', 'L': '960', 'R': 'NFTSetupSessionStateOnRestore'}

Any idea where it comes from ?
As you can see I already set is_serverless=True option.
Maybe a retry option would help here ?
Thanks !

Hi @Olaktal , thank you for reaching out. Based off the error message provided, it looks like the Redshift serverless instance was unavailable at the time you tried to execute the query. I do not believe the root issue (query cancelation) has to do with the client (redshift-connector) in this case.
While retry logic in the client could alleviate this issue, I'd recommend to reach out to Redshift customer support so they may investigate why Redshift server canceled the query execution, as it's likely you would see this behavior with other client tools such as JDBC and ODBC as well.


Improvement in Logging

Currently error logging in the code is very poor and it becomes tremendously difficult to find a bug in logic of the code. So, we need to improve it so that we can figure out quickly where the fault is.

Can u elaborate exactly what is the problem ?

It is a nice idea, how do you plan to store the log messages?

Can we get it listed into a log file or write it into a separate file or console??


【feature request】Can reopen a book after reload current page?

If achieved this feature, will make more easy to read and use.
Thanks.

That's not really something that should be a part of the library itself. If you're talking about the demo reader, I won't add something like that for now. Probably in the future I will release a more complete reader based on the library; currently the reader is only meant to be a simple example.


SIL: Give project_box a field index operand.

Allow project_box to get the address of any field in a multi-field box.

@swift-ci Please test and merge

@jckarter Are you going to document this is SIL.rst?

@eeckstein Yeah, I was going to update SIL.rst once I was further along with surfacing the syntax/serialization changes related to this.


Create a debug() alternative to info() that can be used in tests and turned on via a preprocessor definition

Replaced by https://github.com/AztecProtocol/aztec-packages/issues/1047


Minor spelling for PollingWatcher remarks

Hi @danielmarbach, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

Thanks!


WIP: Rewrite zone-printer app

This simplifies the sample app a great deal
since it no longer uses nginx (confuses users),
it has simpler deployment manifest
This new sampel app is a web server directly,
written in Go, it renders the flag of the gce
region in the response.
It is clean and understandable code as opposed
to ConfigMap/nginx.conf hack we used to have.

This will likely require a lot of README.md changes,
so please hold off until the other PRs regarding tutorial
updates are merged.
Signed-off-by: Ahmet Alp Balkan<EMAIL_ADDRESS>

I plan to make this image available on gcr.io/google-samples/zone-printer as soon as possible.
Preview screenshots:

This is from geopeeker, I have only 2 datacenters (EU and US) but normally you'd see a lot more flags.

Cool, ready to merge?

Yes. :shipit:


[BUG] Drag error: Cannot read properties of null (reading 'left')

Describe the bug
I have a document where drag reordering is not working for some reason (it always works in other docs at the same time). I'm not sure what is so unique about this doc, but when I open the dev console, I see this stacktrace:
Uncaught TypeError: Cannot read properties of null (reading 'left')
    at DragAndDropState.calculateLeftPadding (plugin:obsidian-outliner:1341:56)
    at new DragAndDropState (plugin:obsidian-outliner:1257:14)
    at DragAndDrop.startDragging (plugin:obsidian-outliner:1151:23)
    at DragAndDrop.handleMouseMove (plugin:obsidian-outliner:1068:22)

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: Desktop
Obsidian Version: 1.3.7
Plugin Version: 4.7.1

Additional context
Add any other context about the problem here.

Check my post in Issue 489


API for VWAP data ?

Is there api for VWAP of a quote?

Sorry for the delay.
May be I will take some time to look into it.
But personally, I don't recommend using the API to grab a lot of data, since is not official, if there is large amounts of traffic, it may be disabled by Webull.

@tedchou12 I agree. We can get a lot of public financial data from yahoo financial api.
https://stackoverflow.com/a/47505102/2251303

I would highly recommend using either IEX or Alpha Vantage, both have python libraries. Yahoo Finance tends to be a little slow on the take.

I would highly recommend using either IEX or Alpha Vantage, both have python libraries. Yahoo Finance tends to be a little slow on the take.


Updated note

removed reference to ODB

hi, I don't see that OneDrive has been removed from the note in the live site  - https://aka.ms/ExchangeMultiGeo


VSCode-Bard DOES NOT WORK... Cookie errors for plugins and API wrapper... still reporting cookie problems

I added the cookie as instructed.
Here's my prompt.
You
What is the os module in the Python Standard Library?
Bard ERROR MESSAGE:
"Failed to get bard params SNlM0e, pls retry or check your cookies. You can get your cookie by following the instructions provided in this link."
Google is blocking this VSCode extension, the same way it blocks Bard-API and other wrappers around the Bard Chat interface in a browser.
It looks like Google is forcing everyone to use their PALM API package.
https://github.com/dsdanielpark/Bard-API/issues/155
Too bad.  Great idea.  Not working though.
Big Tech, Cloud Monopoly Sledge Hammer comes down on developers again!!!

I have tried this after reseting cookie, it works:

you can reset your cookie and try again

What does mean 'reset cookie', How can I do it. Should I reset cookie in browser or in VS code. Can you provide instructions. Thank very much.

Today I try to solution 2 and it works.

Hi to all,
for accessing to Google Bard service more cookies are required in general. If following cookies are used it is not needed to have opened session to bard.google.com in any of web browser, but keep in mind that related __Secure cookies might be regenerated after re-login to Google services). No changes on the side of VSCode plugin is needed, plugin has sufficient support in codebase to proceed all related cookies, just need to be provided as string consist of separated cookies like following pattern: __Secure-<cookie_1_key>=<cookie_1_value>;__Secure-<cookie_2_key>=<cookie_2_value>;
Recommendation for extracting of cookies - Firefox users (feel free to align to your case/preference):
prerequisites: Linux OS, sqlite3, xclip, Firefox installed
Firefox ESR:
sqlite3 ~/.mozilla/firefox/*.default-esr/cookies.sqlite """SELECT name, value FROM moz_cookies WHERE host LIKE '%.google.com'   AND name LIKE '__Secure-1PSID%'""" | sed 's/$/;/g;s/|//g' | xclip -selection clipboard
Firefox Release:
sqlite3 ~/.mozilla/firefox/*.default-release/cookies.sqlite """SELECT name, value FROM moz_cookies WHERE host LIKE '%.google.com'   AND name LIKE '__Secure-1PSID%'""" | sed 's/$/;/g;s/|//g' | xclip -selection clipboard
Note for Firefox data gathering:
Based on many reasons extracting of SQLlite data from Firefox is possible only when given instance of Firefox is not running,
it means when Firefox is running there is lock on SQLlite database of given Firefox instance. Above mentioned SQLlite aproach for Firefox is reliable way for automating data gathering (just still need to take care carefully about data lifecycle changes dependend on service of given provider which is quite common, e. g. cookies might be regenerated after relogin to service, etc...). For Firefox advanced users using Firefox containers (or Firefox profiles) and eventually special configuration - they do not need to worry about some cases too much/very frequently.
Not for other browsers
There are more possibilities how to extract __Secure cookie data via other browsers, but might need more advanced  approaches to handle data gathering really reliable way from mid/long term perspective.
When needed just update plugin settings - provide up-to-data cookie data.
Suggestion for plugin owner/developer:
Would be really valuable if given cookie parameter could be stored in dedicated file to be able to automate updating of cookie data parameters.
Thank you.
Stefan

I was having a similar issue and found the solution to be rather simple I think.
I was using method 1 to no avail.  method 2 didn't work either....probably doing that one wrong.  anyway,  what worked for me was
using Chrome in windows 11: we go to the cookie in Application.
1 - from the bard.google.com/chat interface i went to Developer Tools
ctrl+shift+i
2 - go to Application (just like before)
3 - go to Storage > Cookies and right click on the bard.google.com cookie and select "clear".  this will wipe out the cookies for you here.  Then Refrersh the page.  ctrl+r

4 - now, you have a new set of cookies.  copy the new cookie info...
__Secure-1PSID=xxxxxxxxxxxxxxxxxxxxxxx-xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx.
I don't know if the "." at the end of the cookie is necessary but it worked for me just fine.  it was part of the cookie in the page.
I hope this helps anyone else.
I'm sure there is a pythonic way of doing this but I didn't care enough to try to do this.  i'm sure bard or gpt4 can write that out though.  ;)

The solution for me was to use ONLY THE COOKIE VALUE. So don't try to add the prefix.
This DOES NOT WORK, try to paste =>  __Secure-1PSID=12345abcde67890fghij12345abcde67890fghij
THIS WORKS!!!! paste only the cookie value => 12345abcde67890fghij12345abcde67890fghij

Hi to all,
cookies are working as expected, implementation of this plugin regarding handling of cookies is really sufficient
even from mid/long term perspective (more than reasonable good level of development experience of plugin author).
Problem is laying in more clear understanding of cookie usage, mostly because it is not doccumented anywhere so far
and might be eventually very confusing for many users (especially groups just following copy & past approaches).
Quick summary for better understanding about required data set to be provided for this plugin
(all of the following cookies are really required, it means extract all of them via preferred approach,
recommended structure for passing to plugin):
__Secure-1PSID=<data_part_01>;__Secure-1PSIDTS=<data_part_02>;__Secure-1PSIDCC=<data_part_03>;
Above described is highly recommended to be able to use of Google Bard service in general.
What is also important to keep in mind:

cookies might be re-generaded when you are relogin to Google services in different environment

it is really well implemented from Google side, they have considered a lot within implementation especially
from security perspective
when cookies are re-generated user has to re-export them in environmnet where Google Bard services will be used,
e.g. this plugin (reason is that some data in cookies are deeply thight to environment based on above mentioned)

for Firefox users let me recommend following: related secure cookies comment in this thread
just quick notes:

I am using Google Bard service with several other tools and above mentioned is based on outcomes i have recognized during interaction/working with such as tools
I have also shared possible automated approach how to extract cookie datasets (see note mentioned above, there is link to comment in this thread)

Summary
This plugin is implemented reasonable well/suffiently and problems with cookies are more-less consequences of misunderstanding/inpropper usage (frequently hard to understand related deeply, but just try to read this comment more carefully and issues could be gone)
Thank you for understanding and cooperation in advance,
especially many thanks to author of this plugin for his efforts spent on.
Have a nice day.

Solution 2 work fine. :) you need to copy and paste all cookies. The first solution is just only one.

The solution posted by V1x0r worked for me. Thanks!


Build MKL-DNN without tests

Set ExternalProject_Add MKL-DNN to only build the mkl.so without any tests.

TESTNOW


[Backport v2.9] [BUG] high memory usage for the aks.(*aksOperatorController).onClusterChange

Backport #45509

Since we have time, let's get this in for 2.9.3. We will not backport to 2.8.x or before

Validation Template
Root Cause
Recent updates to the client-go Kubernetes client changed the behaviour when caching network clients. This caused our use case to go from never caching clients to add a new cache entry for every client usage, which tied to this cache not having any cleanup mechanism or retention period causes the memory usage to grow indefinitely.
What was fixed, or what change have occurred
We adapted our Kubernetes client configuration to make the underlying caching possible while keeping the level of customization that our use case requires.
Areas or cases that should be tested
Creating a new managed cluster from a public cloud provider (AKS, GKE or EKS), then perform basic checks to ensure it works as expected: creation succeeds, it's connected and can be browsed from the Rancher UI.
The reproducer instructions in the original issue could also be used to verify the changes cause the desired effect (which was already verified on a customer side via provided hotfix image).
What areas could experience regressions
Connectivity with provisioned downstream clusters.

Verified on Rancher v2.9.3-alpha6, rke2 v1.28.8+rke2r1
Continuous memory usage increase no longer present for aks.(*aksOperatorController).onClusterChange from Azure AKS cluster deployments
Steps to reproduce:

Bring up HA Rancher v2.9.2
Access Azure, create 3 AKS clusters (using 1.28.0, 1.27 no longer available in defaults)

Home -> Kubernetes services -> Create+ -> Kubernetes cluster

Create Cloud Credentials in Rancher for Azure

To obtain credentials use:

az login

az ad sp create-for-rbac --role Contributor --scopes /subscriptions/{ID FROM LOGIN OUTPUT} --name {YOUR NAME}

id for Subscription ID, appId for Client ID and password for Client Secret

Import the AKS clusters into Rancher, wait for Active status

Home -> Cluster Management -> Import Existing -> Azure AKS

Set kubeconfig for local/upstream cluster and execute patch script for deployment emulation

while true
do
	for cluster in $(kubectl get clusters.management.cattle.io --no-headers -o custom-columns=name:.metadata.name | grep -v local)
	do
	  kubectl patch clusters.management.cattle.io ${cluster} --type='json' -p='[{"op": "replace", "path": "/status/serviceAccountTokenSecret", "value":""}]'
  done
done

Downstream AKS clusters can be observed toggling between Active and Unavailable status as outputs aggregate:
cluster.management.cattle.io/c-xxxxx patched
cluster.management.cattle.io/c-xxxxx patched
cluster.management.cattle.io/c-xxxxx patched
cluster.management.cattle.io/c-xxxxx patched

In a separate terminal, set kubeconfig for local/upstream cluster and execute heap monitor script

while true; do
	for pod in $(kubectl -n cattle-system get pods -l app=rancher --no-headers -o custom-columns=name:.metadata.name); do
		kubectl exec -n cattle-system ${pod} -c rancher -- curl -s http://localhost:6060/debug/pprof/heap -o heap
		kubectl cp -n cattle-system -c rancher ${pod}:heap ./heap

		go tool pprof -top --trim=false --show_from=generateAndSetServiceAccount ./heap | grep kubernetes.NewForConfig
	done
done

Flame Graph can be viewed using pprof locally, Note: To view aks.(*aksOperatorController).onClusterChange ensure heap data is not loaded from ShowFrom expression matched no samples curl.
Output indicates increasing memory usage over time while deployment emulation script is running, memory usage does not decrease when script is stopped and clusters return to holding Active status for several minutes
         0     0%  8.65%    10.51MB  5.51%  k8s.io/client-go/kubernetes.NewForConfig
ShowFrom expression matched no samples
ShowFrom expression matched no samples
         0     0%  8.65%    10.51MB  5.51%  k8s.io/client-go/kubernetes.NewForConfig
ShowFrom expression matched no samples
ShowFrom expression matched no samples
         0     0%  9.15%    10.51MB  5.82%  k8s.io/client-go/kubernetes.NewForConfig
ShowFrom expression matched no samples
ShowFrom expression matched no samples
         0     0%  9.15%    10.51MB  5.82%  k8s.io/client-go/kubernetes.NewForConfig
ShowFrom expression matched no samples
ShowFrom expression matched no samples
         0     0%  8.87%    11.01MB  5.74%  k8s.io/client-go/kubernetes.NewForConfig
ShowFrom expression matched no samples
ShowFrom expression matched no samples
         0     0%  8.87%    11.01MB  5.74%  k8s.io/client-go/kubernetes.NewForConfig
ShowFrom expression matched no samples
ShowFrom expression matched no samples
         0     0%  9.31%    11.01MB  6.02%  k8s.io/client-go/kubernetes.NewForConfig
ShowFrom expression matched no samples
ShowFrom expression matched no samples
         0     0%  9.31%    11.01MB  6.02%  k8s.io/client-go/kubernetes.NewForConfig
ShowFrom expression matched no samples
ShowFrom expression matched no samples
         0     0%  8.99%    11.01MB  5.81%  k8s.io/client-go/kubernetes.NewForConfig

A few minutes later with deployment emulation script still running (ShowFrom expression matched no samples removed):
         0     0% 15.34%    21.53MB 10.63%  k8s.io/client-go/kubernetes.NewForConfig
         0     0% 15.34%    21.53MB 10.63%  k8s.io/client-go/kubernetes.NewForConfig
         0     0% 13.96%    21.53MB  9.68%  k8s.io/client-go/kubernetes.NewForConfig
         0     0% 15.35%    21.53MB 10.64%  k8s.io/client-go/kubernetes.NewForConfig
         0     0% 15.35%    21.53MB 10.64%  k8s.io/client-go/kubernetes.NewForConfig

Several minutes later, after deployment emulation script stopped (ShowFrom expression matched no samples removed):
         0     0% 17.55%    25.03MB 11.86%  k8s.io/client-go/kubernetes.NewForConfig
         0     0% 17.36%    25.03MB 11.73%  k8s.io/client-go/kubernetes.NewForConfig
         0     0% 17.36%    25.03MB 11.73%  k8s.io/client-go/kubernetes.NewForConfig
         0     0% 17.36%    25.03MB 11.73%  k8s.io/client-go/kubernetes.NewForConfig
         0     0% 17.36%    25.03MB 11.73%  k8s.io/client-go/kubernetes.NewForConfig
         0     0% 17.36%    25.03MB 11.73%  k8s.io/client-go/kubernetes.NewForConfig

Steps for validation:

Bring up HA Rancher v2.9.3-alpha6
Access Azure, create 3 AKS clusters
Create Cloud Credentials in Rancher for Azure
Import the AKS clusters into Rancher, wait for Active status
Set kubeconfig for local/upstream cluster and execute patch script for deployment emulation
In a separate terminal, set kubeconfig for local/upstream cluster and execute heap monitor script
Verify only ShowFrom expression matched no samples outputs are present, aks.(*aksOperatorController).onClusterChange was not found in flame graphs
Verify flame graphs indicate no increasing memory usage on any aks components over time or while deployment emulation script is running. Memory usage is not impacted from any aks components when script is stopped or after clusters return to holding Active status for several minutes.


15 update documentation readability

Feel free to provide input on how the documentation could be better understood

Small nit: Can we use valid Sway on the L2 side of things?
So on the FuelMyToken contract, mapping(bytes32 => bigint) refundAmounts;
should become
refund_amounts: StorageMap<(b256, b256), U256>

Does this look good? @nfurfaro @simonr0204


Should delayed objects modify instances of classes?

from dask import delayed
from nose.tools import assert_true

class MyClass(dict):
    pass

@delayed(pure=True)
def foo_dict(arg):
    assert_true(isinstance(arg, dict))

@delayed(pure=True)
def foo_myclass(arg):
    assert_true(isinstance(arg, MyClass))

# works fine
res = foo_dict(MyClass())
res.compute()
# raises an error
res = foo_myclass(MyClass())
res.compute()

This returns an error because when passing through delayed, the instance goes from a MyClass instance to dict.
Is this intended? For my use cases, it doesn't bother me much, but I want to better understand the inner workings of the delayed object, and what assumptions I can make.
If I replaced delayed with a curry instance, this works (of course, now don't do the last compute step). I'm not sure if this helps narrow things down.
Thanks for the great library!

I suspect that this is due to an isinstance(obj, dict) check somewhere that should maybe be an type(obj) is dict check.

Do you mean in code above? I just tried replacing with type as you suggested, and still get the same result:
def test_delayed_passthrough1():
    class MyClass(dict):
        pass

    @delayed(pure=True)
    def foo(arg):
        assert_true(isinstance(arg, MyClass))

    res = foo(MyClass())
    res.compute()
def test_delayed_passthrough2():
    class MyClass(dict):
        pass

    @delayed(pure=True)
    def foo(arg):
        assert_true(type(arg) is MyClass)

    res = foo(MyClass())
    res.compute()

both functions raise AssertionErrors

No, I mean somewhere within Dask.  Dask uses dicts in some special ways.
Your class subclasses dict, which is somewhat atypical.  I suspect that
inheriting from dict is what is causing this problem.
On Tue, Mar 21, 2017 at 4:54 PM, Julien Lhermitte<EMAIL_ADDRESS>wrote:

Do you mean in code above? I just tried replacing with type as you
suggested, and still get the same result:
def test_delayed_passthrough1():
class MyClass(dict):
pass
@delayed(pure=True)
def foo(arg):
    assert_true(isinstance(arg, MyClass))

res = foo(MyClass())
res.compute()def test_delayed_passthrough2():
class MyClass(dict):
    pass

@delayed(pure=True)
def foo(arg):
    assert_true(type(arg) is MyClass)

res = foo(MyClass())
res.compute()

both functions raise AssertionErrors
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/2107#issuecomment-288215244, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszP6R9gcSMeUGtQ0lGoZSa2kx12x3ks5roDjtgaJpZM4MkYEX
.

Ah now I understand, isinstance(MyClass, dict) = True, thanks.
Found it. This does the trick (from latest master branch), but I think you already knew...
diff --git a/dask/delayed.py b/dask/delayed.py
index a9cbf63..b9a402d 100644
--- a/dask/delayed.py
+++ b/dask/delayed.py
@@ -86,7 +86,7 @@ def to_task_dask(expr):
             return (type(expr), args), dsk
         else:
             return args, dsk
-    if isinstance(expr, dict):
+    if type(expr) is dict:
         args, dsk = to_task_dask([[k, v] for k, v in expr.items()])
         return (dict, args), dsk
     return expr, {}

Should that be merged in along with the test I wrote above?

Yes, such a PR with a test would be great.  You might want to ping @jcrist who might be able to spot a corner case if one exists.

Looks fine to me. Can you submit a PR with a test? Something similar to https://github.com/dask/dask/blob/master/dask/tests/test_delayed.py#L28 (same thing, but for tuples) would be good.

sounds good, submitted, thanks!

Fixed by #2108. Closing.

The same issue seems to occur with distributed library (I fetched the latest version from master branch just in case). I am not sure where in the library is responsible.
from dask import delayed
from dask.delayed import Delayed
from distributed import Client
_pipeline_client = Client("<IP_ADDRESS>:8786")

def test_delayed_passthrough():
    ''' Test that a class that inherits dict isn't improperly interpreted and
        modified.
        This is from Issue https://github.com/dask/dask/issues/2107
    '''
    class MyClass(dict):
        pass

    @delayed(pure=True)
    def foo(arg):
        assert_true(isinstance(arg, MyClass))

    res = foo(MyClass())
    res.compute()

I have checked, and within the function foo, when running in a worker process, arg does have the type MyClass, but id(type(arg)) and id(MyClass) do not match: i.e., presumably deserialization has created two identical class definitions. If you pass the class, it works fine:
@delayed(pure=True)
    def foo(arg, cls):
        assert_true(isinstance(arg, cls))

res = foo(MyClass(), MyClass)

ah yes thanks, I should have seen that. I have noticed that issue in the past. A class defined in the same file code is run is not the same as a class imported (I don't understand the internals why, but it makes intuitive sense for a namespace safe system).
thanks again


Convert NEDJI projection to a binary operation with 'ed'

Add more tiers of operator precedence.
Make rounding operators bind looser than harmonic segments. Make min and max bind looser than addition.
Rename stdlib 'ed' to 'tet' to avoid the now-reserved word.
ref #86

NedjiProjection type is now unused. I should remove it but then I think this is good to go.


gadgets: Implement trace capabilities as image based one

Port trace capabilities to be an image-based gadget.

I think I missed this one. It seems to be very complete, I think the only missing part are the tests.


fix: Leaking username and password when using API service

I encountered this issue while working on a migration branch to support Pydantic v2. The problem is related to the leakage of confidential information in the API documentation. Specifically, the API documentation is revealing the username and password used by the Geolib calculation webservice. This is due to the metadata being included in the model, which, during runtime, exposes the username and password in plain text.

Thanks!


Fix list index out of rang error

drive_manager.py throws indexerror when no drives are available to look at. Need to do an exception
root@chianas03:~/plot_manager# ./drive_manager.py
send_new_plot_notification() Started
update_receive_plot() Started
Total Serverwide Plots: 0
Traceback (most recent call last):
  File "./drive_manager.py", line 747, in <module>
    main()
  File "./drive_manager.py", line 740, in main
    update_receive_plot()
  File "./drive_manager.py", line 500, in update_receive_plot
    if current_plotting_drive == get_plot_drive_to_use():
  File "./drive_manager.py", line 391, in get_plot_drive_to_use
    return (natsorted(available_drives)[0][0])
IndexError: list index out of range

Fixed in v0.8


Add graphql-tag

graphql-tag isn't added as a feature and this is an easy solution that would sidestep many of the issues brought up by https://github.com/angular/angular-cli/issues/1656

I signed it.

@tianhuil I think you might need to reply I signed it! exactly as is (i.e. with the ! ) for the google bot to pick up on it.

I signed it!

Thanks @thefliik.

Closing this as this is not something we want to support. ngx-build-plus is the appropriate way of handling non-standard webpack configurations now. We are in the process of opening the Architect API so these can be part of community provided packages.

Makes sense. I found https://github.com/meltedspark/angular-builders which is a repo attempting to catalog 3rd party builders.


Adjust to new wgpu-py

If we merge this, pygfx main requires the unreleased wgpu. Can also leave this PR open until we've made a release of new wgpu-py.

Let's hold off then until resolved in wgpu-py

i feel like a new version was just released right?
Typically I try to not use try/except statements. is there a version that code like this can depend on?

🥳


Battery information support

Provides objects and data for battery information (if available)

Hello @GermanBluefox ,
I'd like to bring this PR to your attention. It looks pretty straight forward and I am wondering if this PR is getting a chance to be reviewed and eventually merged into the main branch.
Please let me know if there is anything I can do to assist in the process?
Kind regards


Missing mathvariant attribute for <mi> tag in mathml

There's a compile error with the following in a view! macro in leptos 0.7.2 (but is fine without the attribute):
<mi mathvariant="normal">"Δ"</mi>

presumably due to there not being a "mathvariant" attribute here: https://github.com/leptos-rs/leptos/blob/main/tachys/src/mathml/mod.rs#L135.
Although this attribute is essentially useless at the moment and will be removed soon, it is generated by temml which is the easiest way to create mathml from LaTeX, and the error message doesn't lead you to the fix:
error: queries overflow the depth limit!
  |
  = help: consider increasing the recursion limit by adding a `#![recursion_limit = "256"]` attribute to your crate

Want to make a PR? It's a one-word change here to add mathvariant to the list of attributes supported on mi.

Sorry, I misled you due to my lack of familiarity with MathML — mathvariant is already supported as a global attribute.
The issue you are having is not that the method is not defined, but something else.
A trivial example (see below) compiles correctly, but feel free to provide an example that reproduces the issue and I'm happy to look at it.
#[component]
pub fn App() -> impl IntoView {
    view! {
        <mi mathvariant="normal">"Δ"</mi>
    }
}

Sorry yes, I was also overconfident about this being the issue after finding something like this a while ago.


How to configure clangd in macOS High Sierra?

I didn't know where to ask this question.
I'm using macOS High Sierra. I've installed clangd with the following commands:

brew install llvm
ln -s /usr/local/Cellar/llvm/5.0.0/include/c++ /usr/local/include

However it doesn't seem to work correctly. I've tested with the following C++ code:
#include <iostream>

int main() {
    std::cout << "hello\n";
}

It compiles fine but clangd shows the following error at the << operator:
invalid operands to binary expression ('std::__1::ostream' (aka 'int') and 'const char *')

Unfortunately I'm not an expert on clangd - @TalAmuyal might have some ideas here. It does seem like there is some configuration issue - there is a compile_commands.json file that you can use to specify some options: https://reviews.llvm.org/D37150 - might be worth checking that out. Given that Oni is showing the error, it seems like the Oni piece of the integration is working.
Might be worth checking out this mailing list too (or posting the question there): http://lists.llvm.org/mailman/listinfo/cfe-dev
Let us know if you find anything, it would be great to add it to the wiki: https://github.com/bryphe/oni/wiki/Language-Support

Sorry for the late reply.
I tried your snippet, it seems to work well for me.
I'm using a version of clangd I compiled from source though, so a newer version of clangd might help.

Thank, I needed to install Command Line Tools, Xcode wasn't enough.
Below is a basic howto of the process. I've only copied the header files to minimize conflicts because the latest macOS uses llvm 4 and having multiple versions installed is not advised. Hopefully clangd will be included in the next version of macOS.
How to install clangd on macOS

Install the Command Line Tools with the following command: xcode-select --install
Download or build llvm.
Copy bin/clangd to /usr/local/bin/
Copy lib/clang/«clang version»/include to /usr/local/lib/clang/«clang version»/
Copy include/c++/v1 to /usr/include/c++/

Thanks @kuglee for sharing the steps you used to resolve it!


[BUG]Firefox add-on reviewed failed and offer a build report

What happened?
I am using Plasmo and React developing a browser extendsion. When I submitted the extendsion to firefox, I got a info as below

Sources, specifically Build not reproducing: Reproducing the submitted release version based on the provided source code package and instructions failed. You can access the console output at https://prod.pastebin.prod.webservices.mozgcp.net/yj1sOucs . Environment used for building: Node 18.20.2, npm 10.5.0 on Ubuntu 20.04 Please test your build in a clean environment to make sure it is reproducible. If necessary, update the source code package and/or the instructions to reproduce.Please read through the instructions at https://extensionworkshop.com/documentation/publish/source-code-submission/ .

I have accessed the url(https://prod.pastebin.prod.webservices.mozgcp.net/yj1sOucs), it looks like a build error. This version(0.0.13) has publish on chrome and edge. So I think there is nothing problem in source code. Can anyone know how to fix this build error?
The package.json as below
{
  "name": "weread-web-extension",
  "displayName": "微信读书助手 - 前夕",
  "version": "0.0.13",
  "description": "自动阅读、自定义菜单、任意屏占比, 全方位打造沉浸式阅读体验",
  "author": "前夕",
  "scripts": {
    "start": "npm run start:chrome",
    "start:firefox": "plasmo dev --target=firefox-mv2",
    "start:chrome": "plasmo dev --target=chrome-mv3",
    "build:firefox": "npx plasmo build --target=firefox-mv2",
    "build:chrome": "npx plasmo build --target=chrome-mv3",
    "build:edge": "npx plasmo build --target=edge-mv3",
    "build:all": "npm run build:firefox && npm run build:chrome && npm run build:edge"
  },
  "dependencies": {
    "@plasmohq/messaging": "0.5.0",
    "@plasmohq/storage": "1.8.0",
    "daisyui": "4.9.0",
    "jquery": "3.7.1",
    "lodash": "^4.17.21",
    "lottie-web": "^5.12.2",
    "plasmo": "0.85.2",
    "react": "18.2.0",
    "react-daisyui": "5.0.0",
    "react-dom": "18.2.0",
    "zustand": "^4.5.2"
  },
  "devDependencies": {
    "@ianvs/prettier-plugin-sort-imports": "4.1.1",
    "@types/chrome": "0.0.258",
    "@types/jquery": "3.5.29",
    "@types/lodash": "^4.17.0",
    "@types/node": "20.11.5",
    "@types/react": "18.2.48",
    "@types/react-dom": "18.2.18",
    "autoprefixer": "10.4.19",
    "postcss": "8.4.38",
    "prettier": "3.2.4",
    "tailwindcss": "3.4.3",
    "typescript": "5.3.3"
  },
  "manifest": {
    "host_permissions": [
      "https://*/*"
    ]
  }
}

Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
Firefox
Relevant log output
No response
(OPTIONAL) Contribution

[ ] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

请问问题解决了吗？我也有一样的问题....

没解决, 摆烂不管他了.


ACME account private key account.pem is readable by all users on the Apache server

The account private key created is readable by all users:
$ ls -l /etc/apache2/md/accounts/ACME-.letsencrypt.org-0000/account.pem
-rw-r--r-- 1 root root 2654 Oct 19 09:22 /etc/apache2/md/accounts/ACME-.letsencrypt.org-0000/account.pem

Shouldn't read access be restricted? Otherwise any user (or process) on the server can read the account.pem private key.
The directory domains/ has permissions set to 600. Why not set accounts to the same thing?

It is be design, but it is also encrypted with a passphrase.
The reason for this is that the accounts key are needed to be accessed by the running httpd. That process uses, under most installations, another user id. The passphrase is stored in md_store.json and read at startup when the server still has privileges.
Hope that makes sense.


Don't localize the widget automatically

This is very inconvenient, for example I'm from Sri Lanka and here we use 3 languages. (English, Sinhala, Tamil)
By default the ReCaptcha renders in Sinhala and majority of Tamil people don't understand Sinhala and there is no easy option to switch the language either.
I know the developer has option to override the language but this is really getting frustrated. The captcha itself is a difficult thing to solve sometimes and this makes things even more complicated for international users.
Another example is: Android startup screen, language is automatically chosen.
Why don't we let the developer decide the localization rather than setting it automatically because it's a functional part of the UI/UX?
Thanks
Seems like other users also have the same problem: https://groups.google.com/forum/#!topic/recaptcha/KQC8oN0-2sA
https://groups.google.com/forum/#!topic/recaptcha/Z9-DNgt447Y
https://groups.google.com/forum/#!topic/recaptcha/9ec6tVmk59w

Thank you @vdharshan . It is really very inconvenient and can not move further for sign up process where reCaptcha is used.

I signed up for an account JUST so I could agree with this. I moved to Sri Lanka recently, I do not read Sinhala (though I'm learning) and google translate doesn't work on reCaptcha so when I get asked to identify something, I have no idea what it wants me to do. Please please PLEASE fix this, it's so frustrating to have to keep dealing with.

I don't understand the issue guys, you can provide the language you want the widget to have if you don't want it to autodetect.
<script src="//www.google.com/recaptcha/api.js?onload=onloadCallback&render=explicit&hl=LANGUAGE_HERE" async defer></script>

I don't own or have any power to change the website in question since I'm complaining about reCaptchas on other websites, like Tumblr.
I'm saying that as a USER, it's really annoying to go to various websites and have Captchas pop up in the local language because of auto-detection of the country. Making it match the browser language settings would be a lot more convenient.

Yep. Now you see my problem.
Also like the original poster said, there are three main languages in Sri Lanka. While everyone speaks English and can handle English instructions, the Tamils are unlikely to speak Sinhala so they'd be just as confused as I am.

Closing super old issues. Please re-raise if still relevant.


Training Time Problem

Hello
I would like to ask about a training time issue. I am trying to train this model with 3 RTX 3090s and the batch size is set to be 9. However, it shows that the training time is going to be around 370 hours which is much longer than what you posted.
I would really appreciate it if you could help with it.
Many thanks

Thanks, I have found dealt with it.

Thanks, I have found dealt with it.

How did you resolve it, I am facing the same issue


Request a model (with a detail-view) depending on its sequential number and a meeting's id

This can be accomplished maybe by doing these steps:

When in a meeting: Request all displayable models (displayable: with a detail-view) and their sequential numbers
List them in a dictionary like "{ [collection: string]: { [ <meeting_id/model.sequential_number> ]: model/id } }"
When navigating to a detail-view, we can easily get the id of a model by checking this dictionary
A further benefit of this approach would be, that if <meeting_id/model.sequential_number> returns undefined, the requested model does not exist.

It has to be waited for OpenSlides/openslides-backend#1130, since only motions have a sequential number at the moment.

These models have a detail view:

motion
motion_category
motion_workflow
motion_block
motion_statute_paragraph
motion_comment_section
topic
assignment
list_of_speakers
poll
projector


why isn't there an option to use nacon ps4 gamepad?

If the gamepad is PS4-compatible, there is a good chance it will work with the Driver Station software.
Adding explicit support for all available variants of gamepads is (1) not feasible because clone gamepads come and go as the wind blows (2) not particularly useful since only a select list of gamepads are allowed for competition.


escaped backticks in template literals aren't parsable by codegen

Describe the bug
Codegen doesn't know how to handle backtick escaping in template literals.
To Reproduce
Define your schema in a js or ts file, with the help of graphql-tag. And add a field description that uses backtickets to highlight some code in your document browser. (graphql playground for example).

My GraphQL schema:

import gql from 'graphql-tag';

export default gql`
  type Query {
    user(id: ID!): User!
  }

  type User {
    id: ID!
    "Choose a nice username, so users can \`@mention\` you."
    username: String!
    email: String!
  }
`;

Error thrown:
✖ GraphQL request
    Syntax Error: Invalid character escape sequence: \`.

    GraphQL request (7:42)
    6:   id: ID!
    7:   "Choose a nice username, so users can \`@mention\` you."
                                                ^
    8:   username: String!

Sandbox: https://codesandbox.io/s/codegen-backtick-escape-issue-qn1hb

My codegen.yml config file:

schema: schema.js
documents: operations.graphql
generates:
  types.ts:
    plugins:
      - typescript
      - typescript-operations

Expected behavior
Having User with a generated type like:
export type User = {
  __typename?: "User";
  id: Scalars["ID"];
  /** Choose a nice username, so users can `@mention` you. */
  username: Scalars["String"];
  email: Scalars["String"];
};

Environment:

OS: codesandbox
@graphql-codegen/add: 1.3.1
@graphql-codegen/cli: 1.3.1
@graphql-codegen/typescript: 1.3.1
@graphql-codegen/typescript-operations: 1.3.1
NodeJS: v10.16.0

Additional context
In the sanbox posted above, you can change the codegen.yml to use schema.graphql instead of schema.js to see it work. That's the desired outcome, only I need to be able to use .js / .ts files with graphql-tag.
Writing my descriptions like I do in the sandbox, does work, as Apollo parses it and graphql playground displays it. It's only codegen that doesn't compile it.

Hi @smeijer !
I checked and this error comes from the internal graphql validation functions.
Tested it as well in ASTExplorer, which uses just graphql for parsing, and the result is the same:

@dotansimha, do you use different parsers for graphql files as for js files than? Because in a graphql file, it is working. Its not working when using graphql-tag in a js file.
But, my application does compile. Apollo server does understand it. Graphql Playground, does understand it. So it's only codegen that doesn't handle the escaped backticks in js files.
If I just add that snippet to astexplorer, it does parse it:

In a normal graphql file, we should indeed not be escaping the backtick. The escaping is because it's written in a javascript file.
type User {
  id: ID!
   "Choose a nice username, so users can `@mention` you."
  username: String!
  email: String!
}

import gql from 'graphql-tag';

const typeDef = `
type User {
  id: ID!
   "Choose a nice username, so users can \`@mention\` you."
  username: String!
  email: String!
}
`;

I'm not sure if the fault is really in graphql.

I'm exploring the source, and it looks to me that you're loading the files with graphql-toolkit. However, I'm failing to make that one crash.

Tracked the issue down to graphql-tag-pluck, which is being used by grapql-toolkit to extract the typedefs from various files.
Created an issue there instead. https://github.com/DAB0mB/graphql-tag-pluck/issues/44

@smeijer When you parse a JS/TS file in ASTExplorer, you are basically using JS parser, and not GraphQL parser. The GraphQL inside the template is not being parsed, and just stored as string, and not as DocumentNode.
When you use GraphQL parser with a GraphQL string, it uses parse from graphql and do the actual parsing.
graphql-tag-pluck reads JavaScript/TypeScript code as string, convert it into code AST using babel, and then looks for a GraphQL string. Then, this GraphQL string is being parsed into DocumentNode by graphql.

Thanks for the response. I figured that out.
So somewhere between looks for a graphql string and parse grapql string into grapql, the string should be unescaped.

Ohh I see, we'll try to fix that in graphql-tag-pluck. Thanks


LuCI: Error: Network device is not present

Hi,
Version/Device
I'm using OpenWrt 19.07.5 r11257-5090152ae3 / LuCI openwrt-19.07 branch git-20.341.57626-51f55b5
on a Linksys WRT3200ACM.
Brief
After a power outage, the DHCP server in a wireless interface (routed AP) is not working.
History
The standard configuration was previously changed, being the wireless interface separated in a routed AP configuration, using LuCI.
The configuration was applied and saved. It worked correctly, the DHCP in the new created interface was operational.
After the 1st power down wireless clients were unable to get an address.
In the network menu, on the new interface -> Edit -> General Settings: Status: "Error: Network device is not present" was displayed.
Checking /etc/config/wireless as suggested in [1], the wifi-iface section lacks "option network".
Workaround

Following [2]
option network <new_wifi_name>

was added to to /etc/config/wireless -> wifi-iface section.

In /etc/config/network
option ifname was removed from the config interface <new_wifi_name> section.

Changes were applied [2]:
a. ifup <new_wifi_name>
b. /etc/init.d/firewall restart
c. /etc/init.d/dnsmasq restart

The clients were able to connect immediatly.
The notice disapeared from the interface status, displaying the correct radio.
Steps to reproduce

Separate the wifi from lan, using LuCI.
Create a new interface, configure DHCP, firewall, etc.
Apply and save changes.
Clients should be able to connect
Power down the router.
Power up the router.
Clients fail to get address.

References:
[1] https://forum.openwrt.org/t/error-network-device-is-not-present/55493
[2] https://openwrt.org/docs/guide-user/network/wifi/routedap
Thank you.

Already fixed in master with https://github.com/openwrt/luci/commit/7b5b7fbcd63af14658ad234cd39c076a823e4629 - backported to 19.07 branch in 6ba9740b61a3ca89355ae1b08f0456617e0ce3c0

Already fixed in master with https://github.com/openwrt/luci/commit/7b5b7fbcd63af14658ad234cd39c076a823e4629 - backported to 19.07 branch in 6ba9740b61a3ca89355ae1b08f0456617e0ce3c0


Add 4 new language wasms and update existing dependencies

@Gregoor thank you for collecting these wasms in one place. I was trying to bundle them myself, and depending on your package just makes more sense.

awesome, thanks!


Enabled component sub test in Makefile

What is the purpose of this change? What does it change?

Due to the typo the component sub test was completely missing form the Makefile
How to test changes?

make test-cmp-sub-e2e

/retest

[odo]  ✗  Unable to connect to OpenShift cluster, is it down?

/test integration

/lgtm


Does not work for 2D images

I am trying to pre process the data but unfortunately it does not work for 2D images like Umamba did , is there any way around?

Hi @SoberSalman ,
Here is the script to preprocess the 2D images.
https://github.com/bowang-lab/MedSAM/blob/main/utils/pre_grey_rgb.py
Could you please share the detailed error logs?


Update go-version to specify 1.19.0, and run the workflow on changes to the workflow

Codecov Report
Patch and project coverage have no change.

Comparison is base (dfcd7c1) 37.09% compared to head (6a575b7) 37.09%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #372   +/-   ##
=======================================
  Coverage   37.09%   37.09%
=======================================
  Files          45       45
  Lines        2723     2723
=======================================
  Hits         1010     1010
  Misses       1595     1595
  Partials      118      118

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


KOGITO-6422 GatewayFEELIT ConnectException: Connection refused

main branch fwd port of https://github.com/kiegroup/kogito-runtimes/pull/1790

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used on command line locally or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.

How to retest this PR or trigger a specific build:

Run all builds
Please add comment: Jenkins retest this

Run (or rerun) specific test(s)
Please add comment: Jenkins (re)run [kogito-runtimes|optaplanner|kogito-apps|kogito-examples|optaplanner-quickstarts|optaweb-employee-rostering|optaweb-vehicle-routing] tests

Quarkus LTS checks
Please add comment: Jenkins run LTS

Run (or rerun) LTS specific test(s)
Please add comment: Jenkins (re)run [kogito-runtimes|optaplanner|kogito-apps|kogito-examples|optaplanner-quickstarts|optaweb-employee-rostering|optaweb-vehicle-routing] LTS

Native checks
Please add comment: Jenkins run native

Run (or rerun) native specific test(s)
Please add comment: Jenkins (re)run [kogito-runtimes|optaplanner|kogito-apps|kogito-examples|optaplanner-quickstarts|optaweb-employee-rostering|optaweb-vehicle-routing] native

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

The (build) optaweb-employee-rostering check has failed. Please check the logs.

The (build) kogito-examples check has failed. Please check the logs.

✅ Kogito Runtimes Build / Maven Build (ubuntu-latest, 11) (pull_request) Successful in 47m
✅ (build) kogito-runtimes — Build finished. 2424 tests run, 60 skipped, 0 failed.
This is a JUnit test change. Build green. This is a fwd port to main, merging.


Requesting new build

Hello,
It has been more than 500 commits and 1 year since you didn't published a build.
I'm struggling to build from source, I'm 100% on Linux and .NET apps are quiet frustrating to build.
Thanks in advance,

https://github.com/ReClassNET/ReClass.NET/issues/47

When using msbuild and gcc with mono > 5 :
Extensions/XAttributeExtensions.cs(10,18): error CS8107: Feature 'default literal' is not available in C# 7.0. Please use language version 7.1 or greater. [/home/pierre/Documents/git/ReClass.NET/ReClass.NET/ReClass.NET.csproj]
Done Building Project "/home/pierre/Documents/git/ReClass.NET/ReClass.NET/ReClass.NET.csproj" (default targets) -- FAILED.

Build FAILED.

"/home/pierre/Documents/git/ReClass.NET/ReClass.NET/ReClass.NET.csproj" (default target) (1) ->
(CoreCompile target) ->
  Extensions/XAttributeExtensions.cs(10,18): error CS8107: Feature 'default literal' is not available in C# 7.0. Please use language version 7.1 or greater. [/home/pierre/Documents/git/ReClass.NET/ReClass.NET/ReClass.NET.csproj]

    0 Warning(s)
    1 Error(s)

Time Elapsed 00:00:01.52

When changing langVersion in csproj to 7.3 it still doesn't build.

What error do you get with C# 7.3?

After changing from <LangVersion>lastest</LangVersion> to  <LangVersion>7.3</LangVersion>.
I get the same error
Extensions/XAttributeExtensions.cs(10,18):
error CS8107: Feature 'default literal' is not available in C# 7.0. Please use language version 7.1 or greater.
[/home/pierre/Documents/git/ReClass.NET/ReClass.NET/ReClass.NET.csproj]

Done Building Project "/home/pierre/Documents/git/ReClass.NET/ReClass.NET/ReClass.NET.csproj" (default targets) --
 FAILED.

Btw I tested on two laptops and it happens every time. Thanks for your time.

<LangVersion>lastest</LangVersion> is the latest version available (7.3). As you see in the error message C# 7.0 is used. Maybe Mono doesn't support newer versions?

As an aside, it sounds like time for some continuous integration tooling to be added to the repo

@KN4CK3R Mono does support C# 7.3 since mid 2018 (mono 5.0.0):

Maybe it's still missing some features?
@stevemk14ebr I think you right, it could be done pretty easily with Travis CI as an app on github.

Disclaimer: I have no Mono knowlegde.
csc is the Mono compiler for .NET Core. But ReClass.NET isn't a .NET Core project (yet (I'm waiting for the .NET Core 3 release with WinForms support #73)). The makefile uses the msbuild compiler which may not support C# 7.3 (https://github.com/mono/mono/issues/6854). To be able to build again, you need to revert this commit efdf372. Maybe I will stick to C# 7.0 until the project is converted to .NET Core.

After reversing to efdf372. The error is even bigger.
I've no idea why it happens... Maybe not supported yet ?
UI/EnumComboBox.cs(11,60): error CS8107: Feature 'enum generic type constraints' is not available in C# 7.0. Please use language version 7.3 or greater. [/home/pierre/Documents/git/ReClass.NET-efdf3729f7c74256b9be0888515711a6f39da514/ReClass.NET/ReClass.NET.csproj]
Memory/RemoteProcess.cs(209,58): error CS8107: Feature 'unmanaged generic type constraints' is not available in C# 7.0. Please use language version 7.3 or greater. [/home/pierre/Documents/git/ReClass.NET-efdf3729f7c74256b9be0888515711a6f39da514/ReClass.NET/ReClass.NET.csproj]
Memory/RemoteProcess.cs(572,71): error CS8107: Feature 'unmanaged generic type constraints' is not available in C# 7.0. Please use language version 7.3 or greater. [/home/pierre/Documents/git/ReClass.NET-efdf3729f7c74256b9be0888515711a6f39da514/ReClass.NET/ReClass.NET.csproj]
Memory/MemoryBuffer.cs(160,51): error CS8107: Feature 'unmanaged generic type constraints' is not available in C# 7.0. Please use language version 7.3 or greater. [/home/pierre/Documents/git/ReClass.NET-efdf3729f7c74256b9be0888515711a6f39da514/ReClass.NET/ReClass.NET.csproj]
Memory/MemoryBuffer.cs(167,48): error CS8107: Feature 'unmanaged generic type constraints' is not available in C# 7.0. Please use language version 7.3 or greater. [/home/pierre/Documents/git/ReClass.NET-efdf3729f7c74256b9be0888515711a6f39da514/ReClass.NET/ReClass.NET.csproj]
UI/EnumDescriptionDisplay.cs(11,59): error CS8107: Feature 'enum generic type constraints' is not available in C# 7.0. Please use language version 7.3 or greater. [/home/pierre/Documents/git/ReClass.NET-efdf3729f7c74256b9be0888515711a6f39da514/ReClass.NET/ReClass.NET.csproj]
Done Building Project "/home/pierre/Documents/git/ReClass.NET-efdf3729f7c74256b9be0888515711a6f39da514/ReClass.NET/ReClass.NET.csproj" (default targets) -- FAILED.

Build FAILED.

MSBuild version
> msbuild -V
Microsoft (R) Build Engine version <IP_ADDRESS> for Mono
Copyright (C) Microsoft Corporation. All rights reserved.

I just tried the commit 779cb2e and it compiles.
Hoping .NET Core 3 release with WinForms soon

You shouldn't reverse to efdf372 but undo the changes that were made in this commit. After you have done this you can use the lastest source (minus this commit).

to compile reclass a few months ago i had to download stuff and remove something from some package manager or somethin

To be true, I've tried a lot.. and a lot to compile it.. it simply doesn't compile at any circumstances under mono in ubuntu 18.04.
I already have mono 5+, nuget (installed by apt, also .exe through mono nuget.exe) to install dependencies (like SQLite) and nothing, absolutely nothing makes it compile...
It would be awesome if a new release been published.
Note:
Also, the old release doesn't run under mono.

After fixing 3162e04 I was able to compile the project with Mono 5.20 without problems. Try it out!

I released ReClass.NET 1.2 for you.

Thanks so much, i'll try as soon i get home :)
Thanks

I could not stand to wait... haha

Seems working, I'll try to use as soon I get home.. o just had to see it working!


Fix lint error and add basic eslint config

Since grafana-toolkit does error on linting errors but there aren't any separate tools for linting in this repo yet, I thought it would be a good idea to aside from fixing the linting error introduced here also add a basic eslint config.
One that also covers exhaustive-deps, since the refactor I did encourages using function components with hooks and having that rule enabled makes it so much more pleasant to work with them.

@ryantxu While I appreciate that you approve of this PR, do you know what's up with CI?
I checked out the config but I simply don't know what those workspace persistance things are supposed to achieve and why they might trigger errors.


Bot orchestration - Unable to create master_bot_conf/conf on local macbook

Describe the bug
When running tests on docker compose build for dashboard on local macbook, getting the error below when starting master_configs
docker: Error response from daemon: Mounts denied:
The path /home/dashboard/hummingbot_files/templates/master_bot_conf/conf is not shared from the host and is not known to Docker.
You can configure shared paths from Docker -> Preferences... -> Resources -> File Sharing.

Steps to reproduce bug

Run hummingbot_with_dashboard compose file
Use dashboard development branch
Run dashboard on local MacOS
Start master config

Same step I get the following error
Error: No such container: hummingbot-master_bot_conf
docker: Error response from daemon: create ./hummingbot_files/templates/master_bot_conf/./certs: "./hummingbot_files/templates/master_bot_conf/./certs" includes invalid characters for a local volume name, only "[a-zA-Z0-9][a-zA-Z0-9_.-]" are allowed. If you intended to pass a host directory, use absolute path.
See 'docker run --help'.

any help for this issue, facing the same, followed the youtube playlist

have followed the same still issue is same

any help ?

ok will give it a try

Any updates, same issue here?

Try uninstalling then reinstalling Docker and make sure you are on the current version. It could also be a permissions issue on your end as well. I'm not able to replicate the issue on either an Intel or M2 Mac.

This worked! Just updated Docker and the error has gone! Thank you!


Outline View line wrap

It would be nice to see the whole line in outline view. Here's the screen where you can see that function signature isn't visible. Can we add setting for word wrapping here like we have for editor?

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_smiling_face: This feature request received a sufficient number of community upvotes and we moved it to our backlog. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Add default limit multiplier

Allow to set a default limit in the schema properties and allow to set a multiplier in table properties.

In tpch this is called scale factor (sf)


disable modern ui / themes

Disable Modern UI
Since Modern UI is not available for Zimbra OSE, please disable it.
zmprov mcf zimbraModernWebClientDisabled TRUE
zmprov mc default zimbraPrefClientType advanced

Open file /opt/zimbra/jetty/webapps/zimbra/public/login.jsp
vi /opt/zimbra/jetty/webapps/zimbra/public/login.jsp
Find clientModern (line 730) and give comment  (<%– and –%>) for that line
<%--   <option value="modern" <c:if test="${client eq 'modern'}">selected</c:if>> <fmt:message key="clientModern"/></option> --%>

The zmprov commands are for an existing installation and therefore cannot be done pre-build.  The same with the second one.
Also, editing the file pre-build, still doesn't address the issue that the zmprov commands will still be set to use modern.
You opened the issue in the wrong place - you should be opening this with Zimbra directly, since it is their repositories that are configure to use the Modern interface.  This has nothing to do with my scripts.
I will patch some things if it's possible, but when it requires using zmprov commands which are related to either config in LDAP or MariaDB database, then I will not be patching.  In this instance, like this situation, the issue should be opened with Zimbra, so that they can rectify it within their repositories.
With that in mind, I am closing this issue, as this is nothing related to my scripts or build process.


Ng update forces double quotemark

🐞 Bug report
Command (mark with an x)

- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [x] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Is this a regression?
Probably, I used ng update several times already and this never happened
Description
After running ng update to update from 7.3 to 8.0.0-rc.5 I get a big amount of tslint errors because it used double quotes (") on the modified imports. IE:
-import { MatDialogModule } from '@angular/material';
+import { MatDialogModule } from "@angular/material/dialog";

All the changed imports are from material, so I can't say if it's only an issue with material
🔬 Minimal Reproduction
run ng update --next in any angular project using material
🌍 Your Environment

Angular CLI: 7.3.9
Node: 10.13.0
OS: darwin x64
Angular: 8.0.0-rc.5
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... platform-server, router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.13.1
@angular-devkit/build-angular     0.13.1
@angular-devkit/build-optimizer   0.13.1
@angular-devkit/build-webpack     0.13.1
@angular-devkit/core              7.3.1
@angular-devkit/schematics        7.3.9
@angular/cdk                      8.0.0-rc.2
@angular/cli                      7.3.9
@angular/http                     7.2.15
@angular/material                 8.0.0-rc.2
@ngtools/webpack                  7.3.1
@schematics/angular               7.3.9
@schematics/update                0.13.9
rxjs                              6.5.2
typescript                        3.4.5
webpack                           4.29.0

This is somewhat related to the convo I had with @devversion last week over here https://github.com/angular/components/issues/16086

Yeah. Linking to the related Material issue as well: https://github.com/angular/components/issues/16035#issuecomment-493176528

As an aside, StringLiteral nodes do have an internal property (singleQuote) that can be used to force the use of a single quote when printing.

Something we also had to look into was the additional whitespace that is added to the ts.NamedImports. This is something that also deviates from the existing import that is rewritten. Ideally we'd really just run the TslintFix task for the generated/updated files to match the project rules.

I'm actually thinking a post-generate/add/update schematic  hook may be the ultimate solution with the hook passing a list of the modified/created files.  Similar to a lint-staged/husky setup for git.  This would allow the project to use whatever formatter desired (e.g., tslint, eslint, prettier, etc.),

The Angular Material part of this was fixed for 8.0.0 via https://github.com/angular/components/commit/a3856c7c9916d82733393f5b982c3c50b5c60a15.

I think we should update this issue to be more generic. i.e. talk about lint failures generally caused by arbitrary schematics.
The proposal should be that lint fix runs automatically after any generate schematic. That would also simplify the --lint-fix flag for @schematics/angular generate schematics.

It would be good to triage this issue as well (priority, etc) so that it gets some attention. While not super-severe, this seems to be a frequent problem.

Duplicate / same root cause of https://github.com/angular/angular-cli/issues/10777


Create result display page v1

Design
Same thing as in #27  page, use atomic compnents and only test stuff like routing and user changing in here.
Depends on #21 #25 #24 #48 #49

@0815Sailsman Kilian says this is implemented and should be closed


Batch updates of multiple Pods.

Try to batch updates of multiple Pods in Kubelet, to avoid multiple (expensive) calls to Docker.
This is slightly (by ~10-20%) reducing the latency of starting the last Pod when we're starting 50 equal pods on the same node.

Overall this looks good (although I'm guessing it's still a WIP).  The whole waiting seems a bit bad. I don't have a better solution though :) Do we know what kind of win we get from doing this versus making SyncPod() faster? From what we mentioned in #4119.

@dchen1107 taking this over from you, let me know if you would still like to review :)

Why do you think that this waiting is bad - I used a very similar pattern in my previous project and it worked pretty well. The missing this we may want to do here is to not try to accumulate a possibly infinite number of notifications, but instead set a limit for time (or number of those notifications that can be batched).
Regarding making SyncPods() faster - we definitely want to do it to. I will start working on it today. If you think that we should wait with this PR for that one, I'm fine with that.

I don't have a more technical reason than it being racy (although does improve performance in practice). I'm fine with the PR as is :)
LGTM and will merge, but if you have time tomorrow can you run your tests again and see the gain this gets us? I'm curious to track how we improve there,


Add option to focus bufferize output window

Fixes #11

Makes sense to me, thanks for contributing :+1:


feat: Enable ModelMetadata endpoint

MLServer and Triton both support the ModelMetadata endpoint, so we should surface it for end users.
Here, the ModelMetadata endpoint is enabled in the adapter logic for MLServer and Triton.
Ref: https://github.com/kserve/modelmesh-serving/issues/136

Pending on changes to the controller as well.

Tested with controller changes here: https://github.com/kserve/modelmesh-serving/pull/164

/lgtm


Can't download package from npm

Hello,
When doing the command npm install @maxgraph/core i have the following error :
npm ERR! Cannot read properties of null (reading 'matches')
2023-07-18T08_33_00_990Z-debug-0.log
It's not a brand new angular application but i just did a npm update before this, maybe the problem is from there.

Hi @Anthony-Wratpor
I am not aware of issues with the @maxgraph/core package. https://github.com/maxGraph/maxgraph-integration-examples uses the npm package without error (latest 0.3.0 and previous versions in the past). This is automatically tested in fresh environments using GitHub Actions.
According the provided logs, here is your environment

OS: windows
node v18.13.0
npm  v8.19.3

The error is generated by the npm cli. Did you try with a more recent version of the npm cli like v9?
You may also try a fresh npm clean-install
4640 verbose stack TypeError: Cannot read properties of null (reading 'matches')
4640 verbose stack     at Link.matches (C:\Program Files\nodejs\node_modules\npm\node_modules\@npmcli\arborist\lib\node.js:1114:41)

Glad to here you solve your issue.


JS -- Implement doc object

https://www.adobe.com/content/dam/acom/en/devnet/acrobat/pdfs/js_api_reference.pdf#page=335
it has all the properties/methods defined in the spec
unimplemented methods are there but with an empty body to avoid exception when calling an undefined method
implement zoom, zoomType, layout, pageNum, ...

More generally, do we actually need to run this._initializeJavaScript(pdfDocument); as early as is done currently?
Is it absolutely impossible for things to work correctly if we e.g. waited for onePageRendered, as we do for a number of other API-calls, to prevent them running very early and thus possibly delay initial rendering?

Good point, as far as I understand the specs, there is nothing which indicates that some actions must be ran before rendering (some of them must ran on "Page open" or "Document open" events).
Anyway I guess that the delay between end of rendering and the moment the user begins to interact with the document is a way enough to load the sandbox stuff.
@Snuffleupagus is it ok for you to move this._initializeJavaScript(pdfDocument); at the end of https://github.com/mozilla/pdf.js/blob/8b652d685b4e66ea95dc036bd7aa2a200766ceb5/web/app.js#L1330 ?

For example: If you're able to delay this._initializeJavaScript(pdfDocument); slightly you'd not need to re-fetch the "metadata" again and could instead use the already cached data.

is it ok for you to move this._initializeJavaScript(pdfDocument); at the end of
https://github.com/mozilla/pdf.js/blob/8b652d685b4e66ea95dc036bd7aa2a200766ceb5/web/app.js#L1330

That's most definitely the preferred solution here, assuming if works of course :-)

Looks like a good starting point. Now that we are interacting with the viewer I'd really like to see some tests when we have quickjs wired up.


MongoDB cannot login, always Authentication failed

MongoDB username and password are correct. I am using mlab.com.
If src in momyfile.json is my localhost, this code run without problem.
But, if src is another ip address (in this case i use mlab.com), I got message error

(node:4177) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): MongoError: Authentication failed.

Maybe in this code auth using MONGODB-CR, not SCRAM-SHA-1?
Regards

Hi @ardinusawan, momy just calls MongoClient. connect() method.
https://github.com/cognitom/momy/blob/v0.7.0/lib/tailer.js#L178
See also this document:
http://mongodb.github.io/node-mongodb-native/2.2/api/MongoClient.html#.connect
It seems that the library only supports GSSAPI or MONGODB-CR at this point.
https://github.com/mongodb/node-mongodb-native/blob/2.2/lib/url_parser.js#L307

@ardinusawan I've found the test for scram. It should work.
https://github.com/mongodb/node-mongodb-native/blob/V2.2.21/test/functional/scram_tests.js#L30

So the solution is
mongodb.connect(this.url2?authMechanism=SCRAM-SHA-1&authSource=admin&maxPoolSize=5).then(db => { ......
Right?

Looks close. You can write that in your momyfile.json. I haven't tried but It could be like this:
{
  "src": "mongodb://somewhere:27017/dbname?authMechanism=SCRAM-SHA-1"
}

Still cannot connect :(

@ardinusawan momy needs replica set cluster. Don't you use database in Single-node sandbox?
If you don't have the right to access oplog, it would fail.
http://docs.mlab.com/oplog/#connecting-to-the-oplog

Yes im using Single-node sandbox.
So, now im move to digitalocean droplet.
I try to run script in droplet, src as localhost.
But when I run

momy --config momyfile.json --import

There is another error message ( not error auth message again :D )

(node:4378) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: Cannot read property 'ts' of null

What wrong?

Hmm, it seems that momy couldn't get the information of ts (timestamp) from MySQL table. Could you check that mongo_to_mysql table was automatically created?
https://github.com/cognitom/momy#mysql
If not, your account might not have the privileges to create tables on MySQL.

Yeah mongo_to_mysql was automatically created :-/

Finally it work. Thx


Fix Create Branch

https://user-images.githubusercontent.com/5075484/219146400-9e5f9b2f-f194-4c59-ad20-e83be5536464.mp4

The timeout doesn't seem to get handled properly. I see this error in the console but the dialog state doesn't change:

(Ignore that foobar branch - it has status unknown)

The timeout doesn't seem to get handled properly. I see this error in the console but the dialog state doesn't change:

(Ignore that foobar branch - it has status unknown)

What should happen in this case?

@kldavis4  Updated so that this happens when it times out.

Looks good to me, @logan-anderson!


High Latency for push notifications using parse-server-sns-adapter module via parse-server

I am currently executing a call in the afterSave trigger for my "Messages" class that sends push notifications to select users. We have configured parse-server-sns-adapter (~0.0.7 in package.json) and have confirmed through the AWS SNS UI that the endpoints are properly added and the push notifications are sent and received fine.
Steps to reproduce
Here is the setup in our index.js (snippet):
var pushConfig =  { pushTypes : { android: {ARN : [REDACTED]},
                                  ios: {ARN: [REDACTED], production: false, bundleId: "com.something.dev"}
                                 },
                    accessKey: process.env.SNS_ACCESS_KEY,
                    secretKey: process.env.SNS_SECRET_ACCESS_KEY,
                    region: "us-east-1"
                  };

var SNSPushAdapter = require('parse-server-sns-adapter');
var snsPushAdapter = new SNSPushAdapter(pushConfig);
pushConfig['adapter'] = snsPushAdapter;

var api = new ParseServer({
  databaseURI: databaseUri,
  appId: process.env.APP_ID || 'myAppId',
  cloud: process.env.CLOUD_CODE_MAIN || __dirname + '/cloud/main.js',
  masterKey: process.env.MASTER_KEY || '', //Add your master key here. Keep it secret!
  serverURL: process.env.SERVER_URL || 'http://localhost:1337',  // Don't forget to change to https if needed
  webhookKey: process.env.WEBHOOK_KEY || 'hook',
  push: pushConfig,
  verbose: true
});

Here is the query we have for sending pushes:
var pushQuery = new Parse.Query(Parse.Installation);
    pushQuery.containedIn("activeUser", toUserPointers, { useMasterKey: true });
    Parse.Push.send({
      where: pushQuery,
      data: notificationData
    }, { useMasterKey: true }).then( function() {
          promise.resolve("Push notification successfully sent!");
       }, function(error) {
          promise.reject("Error while sending push notification: " + error.message);
       }
    );

As I mentioned above, this is triggered from the afterSave. When I create 10 simultaneous requests (saving object to Messages class) from just two devices (each sending a push notification to 9 other devices), it forces the latency to go up (like 10-15 seconds in some cases) for the full system. I can confirm it is not something fundamental with the afterSave itself getting triggered because I tried commenting out the promise that sends the push notifications and it resolved the latency problem completely.
What am I doing wrong with the pushQuery? Clearly, it's causing a strain on the system that makes it not production ready. I've already switched to SNS based on the feedback from @flovilmart and others when explaining some of the limitations of push on parse server.
Environment Setup

Server

parse-server version: 2.2.19
Operating System:     64bit Amazon Linux 2016.03 v2.1.3 running Node.js
Hardware:            t2.medium (AWS EC2)
Localhost or remote server? AWS Elastic Beanstalk

Database

MongoDB version: 3.2.7 (MMAPv1)
Hardware: Shared (this is a dev database)
Localhost or remote server? mLab

I've already switched to SNS based on the feedback from @flovilmart and others when explaining some of the limitations of push on parse server.

Switching over SNS won't change anything as the processing remains the same as long as you use Parse.Push.send.
On your pushQuery, are all the fields properly indexed?


Should the VFS be case-insensitive?

I saw a map where an image failed to load because the file's extension was written in upper case, and was surprised to learn that VFS paths are case-sensitive. I found this odd because everything else (like commands or any sort of config file) seems to be case-insensitive. So making VFS file opening case-insensitive seems more consistent, and might improve asset compatibility. @sweet235 Do you often see case-sensitivity problems when porting Tremulous maps?
Note: this should be fine for dpkdirs on case-sensitive filesystems because we always open VFS files by looking them up in the loaded file list first, rather than blindly passing the path to the OS's open function. We'd just need a 1-line change to PakPath::ReadFile to use the name from the file list rather than the requested name.

VFS is expected to be case-insensitive in DPK. If it's not, it's a bug to fix.
It would be cool to also have VFS being case-insensitive with DPKdir, but that's a feature request.

Fixed (for 0.55) by #1315.


New rule (Attachment): Calendar invite with suspicious link leading to an open redirect

/update-test-rules


Add test cases for FhirOperator.kt

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes https://github.com/google/android-fhir/issues/1075
Description
Clear and concise code change description.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: (Bug fix | Feature | Documentation | Testing | Code health | Builds | Releases | Other)
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

com.google.android.fhir.workflow.FhirOperatorTest > evaluateIndividualSubjectMeasure FAILED
    java.lang.OutOfMemoryError at FhirOperatorTest.kt:172

Also, enabled automerge

com.google.android.fhir.workflow.FhirOperatorTest > evaluateIndividualSubjectMeasure FAILED
    java.lang.OutOfMemoryError at FhirOperatorTest.kt:172

Also, enabled automerge

These test cases are passing locally and they were passing on CI as well but after merging recent changes from the master they are not passing on CI.


[NVIDIA] Move Transpose, VeriadicSplit layer tests to API 2.0

Details:

Move Transpose, VeriadicSplit layer tests to API 2.0

Tickets:

111380

@iefode Please take a look


Tree span distribution

Cool! Can you put some examples in here of what they look like with and without log-transform?

Here's an example without the log transform:

and one with it:

Nice :+1:

Needs a rebase now. The trick here will be to drop the first commit, because that conflicts with the PR we just merged. You drop a commit by deleting that line in the rebase "file" that you get to edit.

Now that you've fixed the conflicts it would be good to squash down to one commit. (The stdpopsim guide talks about this)


Add tests and docs to unwind-blank feature

This adds the missing tests and an example to the documentation for #285.
Also, it removes the trailing comma.
I like trailing commas ala air-bnb. However, we don't use them anywhere in the code base and I think that consistency is important.

Coverage remained the same at 85.35% when pulling 6f12970196688f8393301bcbea488f6dff29b3e0 on juanjoDiaz:cleanup_unwind_blank into d97a6683352054caaaa5ea9a6d0e23645562aedf on zemirco:master.


feat(workflow): added image validation

Will also add screenshots

The numbers here are incorrect but it will output this info to the user telling them exactly which file/folders failed the check:

All tests should also pass:

The validate images test currently fails because there is an image that doesn't comply:
https://github.com/newrelic/newrelic-observability-packs/blob/main/packs/nginx/icon.webp


Displaying Swiper components conditionally via another Swiper component completely breaks the sliders spacing and boundaries (w/ Ionic)

Check that this is really a bug

[X] I confirm

Reproduction link
https://github.com/amanape/is-bug
https://codesandbox.io/s/confident-platform-sfzn6
Bug description
I have Swiper sliders in my app header that also works as tabs that display additional sliders in the main body when clicked on. When the app first renders everything works as expected, it only starts to break down when I click on another slider in the header to display the other sliders in the body. The sliders in that are displayed in body are then all bundled together removing any space between properties added and the boundaries are removed as well allowing the sliders to move out of the screen as much as you want without the "bounce back" effect. For reference, this is what I mean by displaying the sliders conditionally:
{selectedSection === 0 && <FirstSection />}
{selectedSection === 1 && <SecondSection />}

where <FirstSection /> and <SecondSection /> contain the sliders I want to display.
I have tried various ways to build it how I want:

The way above which is the quickest and easiest
Use the useEffect() hook to update the section displayed depending on the selectedSection state because I thought it was an issue with the sections not re-rendering.
Create the sections as individual Ionic pages instead of components that I call and use Router to treat them as pages.

All these options I've tried cause the same exact issue as described, nothing worked.
It is also worth mentioning that in my original app, I am using the <IonTabBar> component where this bug exists in one of the tab buttons; when this issue occurs and everything breaks, instead of reloading the entire app, if I switch to another tab then back, the sliders seem to fix themselves, but then break again when I switch sections and back (that is why I thought it was an issue related with rendering).
I am also using a slightly older version of Swiper.js due to https://github.com/nolimits4web/swiper/issues/4871
Expected Behavior
The sliders to work as they always do when displaying them conditionally.
Actual Behavior
The sliders break when being displayed conditionally after the initial(?) render of the page.
Swiper version
6.8.4
Platform/Target and Browser Versions
iOS, Android
Validations

[x] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Edit:
After the swiper breaks, I noticed that adjusting the width of the screen un-breaks it.
Note:
I also added SwiperCore.use([IonicSwiper]); but it doesn't seem to affect this specific problem

Solved by setting the observer and observeParents parameters of the conditional Swipers to true. In this case, I changed
MySlides.tsx
const MySlides: React.FC<{ section?: number }> = ({ section }) => {
  const numOfSlides = [...Array(5).keys()];

  return (
    <Swiper slidesPerView="auto" spaceBetween={15} className="my-swiper-slides">
      {numOfSlides.map((number) => (
        <SwiperSlide key={Math.random()}>
          /* ... */
        </SwiperSlide>
      ))}
    </Swiper>
  );
};

to
MySlides.tsx
const MySlides: React.FC<{ section?: number }> = ({ section }) => {
  const numOfSlides = [...Array(5).keys()];

  return (
    <Swiper slidesPerView="auto" spaceBetween={15} className="my-swiper-slides" observer observeParents>
      {numOfSlides.map((number) => (
        <SwiperSlide key={Math.random()}>
          /* ... */
        </SwiperSlide>
      ))}
    </Swiper>
  );
};

And that fixed my problem


Doxygen command intellisense support

Fix/Feature/Other
Fix

[√] Link to issue. If there is no issue please describe it using at least the issue template
I am working on issue #30

Feature

[√] Description of what's added

It can trigger the auto complete after typing either @ or \ and will have a placeholder as the argument to the doxygen command. Some doxygen commands need to be paired with an ending command, and it will also be auto completed in the snippet.
Currently it looks like this:

However I don't know why vscode doesn't automatically add the leading * for block comment and format it like it does in javascript/typescript, like this:

In C++ it looks like this:

I might messed up some settings in the C++ formatter, but please let me know if it's the same behavior on your machine.

Update: Now it is almost there, see gif. I will make the documentation to proper markdown string

I haven't checked out how to use the automated testing, but please give me feedback.

All done, ready to be merged!

Sorry I missed you removing the WIP indicator. Great work, thank you for contributing!


Milestone report - Import Spinner and success toast

As an analyst
I want to see feedback that the import is working
So that I can be confident that the changes I made are being implemented
Description:
This ticket is to implement the Spinner animation when the Save and Import Data button is pressed and a success toast to appear when it completes.
Acceptance Criteria:

[ ] When import is occurring, show spinner (same as the one used for SOW)
[ ] When import is occurring & spinner is showing, save & cancel buttons are disabled
[ ] When import is complete, show success toast (similar to the one used for SOW)
[ ] Success toast slides out after 5 seconds
[ ] If import is not successful do not show spinner or success toast

Implementation Details

Hey team! Please add your planning poker estimate with Zenhub @AntBush @marcellmueller @rafasdc

I didn't get the success toast


Only 3 Departures delivered

I wanted to display the next 5 Trains but no matter how many connections i configure for the sensor,

there is always only data for the next three trains.

This is because there is no more result in the sensors attribut.
Thanks for your help !

Thanks for pointing out. Unfortunatly I wont be able to fix this, since different connections show different amounts of next connections. But now an warning will be displayed in the log and it is not possible to request more than 6 connections.


Implement an initial UI

This PR is very large and I've tried to arrange the commits so that they can be reviewed individually. IMPORTANT: the commit messages for some of these commits include important contextual information to aid the review process.
Let's start with the small fixup commits:

8306fec adds the hydra mock consent app back to the docker-compose configuration since it got lost somewhere.
490b201 ports a bugfix from django-boilerplate.
694b5da adds a convenience wrapper script to make calling tox easier.
8a2630d fixes a long-standing annoyance that you had to remember to create OAuth2 clients after a docker-compose up. We now do what we should have done all along and run the client creation script via docker-compose.

The following commits change the functionality of the Django app:

2978f55 adds some convenience class methods to the resource wrappers in the jwplatform module.
3d3fa73 adds support for serving static files via whitenoise. We do this in this PR because we make use of whitenoise in 2a942b9 to do some jiggery-pokery w.r.t. serving the frontend.
43eb35c adds a small API to the Django app which has support for listing and searching medai/collections and retrieving the current user's profile. This API is then used by the frontend.

And now the main meat:

b56ef5c is simply the output from create-react-app. It's broken out into a separate commit so that one can see the changes which have been made from the boilerplate app.
8475fdb adds support for this stub webapp for building a new bundle of JS and HTML on changes and wires in the styleguidist documentation tool. See the commit message for more details.
c25464a actually implements the frontend. Again, read the commit message.
2a942b9 wires in the frontend to the backend. This is a very complex commit and is probably going to be the hardest to review. I've attempted to make the commit message useful and have duplicated some of it in the README.

After that the remaining commits are fairly trivial:

940443f simply documents in the README the various HTTP endpoints that a developer might want to point their browser to. It doesn't include mailhog at the moment since I don't think we currently make use of it.
db12887 updates the package-lock.json file. It's split out into a separate commit to avoid clogging the various frontend commits by updates to this large, machine generated file.

Closes #58

Note to reviewers: the CircleCI failure is due to #64.

Indeed, this was the reason #64 was raised and #65 was merged in the first place :).

OK, failing for a different reason now. Will open an issue.

I've worked around the issue in 2469f3b but it's a bit ugly :(.


Add support for custom labels on local cluster

Is your feature request related to a problem? Please describe.
Rancher is deployed to the cluster local, and is set with the default provider.cattle.io label. Since we rely heavily on Fleet to provisioned all of our downstream clusters as well as our Rancher cluster (local). At the moment, I have to manually update Cluster resource to include any custom labels.

Describe the solution you'd like
It would be awesome if these custom labels can be set at the helm chart values deployment.
Describe alternatives you've considered
N/A
Additional context
Having custom labels on local clusters is very helpful as we're able to just have one Fleet GitOps Repository and rely on cluster label selectors when deploying custom configs to all downstream and local clusters.

If someone needs the same, I have managed to add labels using local-exec provisioner to configure them over the Rancher API:
resource "null_resource" "add_labels_to_local_cluster" {
provisioner "local-exec" {
command =  <<EOT
curl -k --insecure -u '${rancher2_token.auth.access_key}:${rancher2_token.auth.secret_key}'
-X PUT
-H 'Content-Type: application/json'
-d '{
"name": "${local.rancher_local}",
"labels": {
"rancher-name" : "${local.rancher_short_hostname}",
"owner": "xxx",
"provider.cattle.io" : "eks",
"accountID": "${local.account_id}"
}
}'
https://${var.rancher_hostname}/v3/clusters/${data.rancher2_cluster.local-cluster.id}
EOT

}
triggers = {
#Forces execution of API call on each "terraform apply", otherwise it's only ran the first time
always_run = timestamp()
}
depends_on = [data.rancher2_cluster.local-cluster]
}
data "rancher2_cluster" "local-cluster" {
provider = rancher2.admin
name = "local"
}
resource "rancher2_token" "auth" {
provider = rancher2.admin
ttl = 1200
depends_on = [resource.rancher2_bootstrap.admin]
}


wrong topology for convolutional layers

Currently, in convolutional layers, all neurons show up as connected to every neuron in the previous layer.

Fixed in commit b5dcf997ce9337cc99259cf3a2c14170ad7562ab
The fix is a bit hacky. In the convolutional layer method, called for instance in https://github.com/FluxML/NNlib.jl/blob/master/src/impl/conv_direct.jl, a certain cumulative behavior is allowed, in which the output is multiplied by yT(1), where yT is the eltype of the input. In ChainPlot, this turns out to be NeuronState, and it would be natural to set NeuronState(1) = hotneuron, but this propagates to every output layer. So the solution was to set NeuronState(1) = coldneuron.


Check if and how it is possible to switch Go module names to a domain <> the GitHub domain the module code resides on

What would you like to be added:
We have all Golang modules named like the github repository name they reside in, e.g. ocm.software/open-component-model/ocm.
Now we want to restructure the modules and during the same activity also change the name of the modules to a different domain than the code repository resides on. The new domain should be "ocm.software", so that the new Golang module would be named "ocm.software/open-component-model/ocm".
How can this be achieved and what needs to be done on the server serving requests for "ocm.software" to redirect requests from Go clients that want to use the module ocm.software/open-component-model/ocm to the correct location on the github repository ocm.software/open-component-model/ocm ?
Epic link: open-component-model/ocm-project#27
**Possible solution:
Set up a custom domain for your Go modules and configure the server to redirect requests to the correct location on the GitHub repository.
Here are the steps:

Change the module path in your Go files: Update the import paths in your Go files to use the new domain. For example, change import "github.com/ocm.software/open-component-model/ocm" to import "ocm.software/open-component-model/ocm".

Update go.mod file: Change the module directive in your go.mod file to reflect the new domain. For example, module github.com/ocm.software/open-component-model/ocm should be changed to module ocm.software/open-component-model/ocm.

Configure the server for ocm.software: You need to set up your server to respond with HTML meta tags that point to the new location of your Go module. When the Go command fetches a package by path, it does so through a series of HTTP requests. It first requests the import path as a URL with the go-get=1 query parameter. The server should respond with a meta tag in the HTML body of the response that specifies the source repository of the package.

Here is an example of how the meta tag should look:
    <meta name="go-import" content="ocm.software/open-component-model/ocm git https://github.com/ocm.software/open-component-model/ocm">

This meta tag tells the Go command that the package ocm.software/open-component-model/ocm is a git repository located at https://github.com/ocm.software/open-component-model/ocm.

Test the setup: You can test if everything is set up correctly by running go get ocm.software/open-component-model/ocm@version, where version is the version of your module. If everything is set up correctly, the Go command should be able to find and download your module.

For more information, refer to
Official Go documentation on [Custom Import Path(https://golang.org/doc/go1.4#canonicalimports) and Go Modules.
go.devgo.dev on Using Go Modules - The Go Programming Language
An introduction to the basic operations needed to get started with Go modules.

tested successfully with ocm.software/testpackage and
<html><head>
  <meta name="go-import"
        content="ocm.software/testpackage
                 git https://github.com/mandelsoft/testpackage">
  <meta name="go-source"
        content="ocm.test/testpackage
                 https://github.com/mandelsoft/testpackage
                 https://github.com/mandelsoft/testpackage/tree/master{/dir}
                 https://github.com/mandelsoft/testpackage/blob/master{/dir}/{file}#L{line}">
</head></html>


Classdscount

Count Classes without Docstrings
Description
A Feature PR that creates a function to fulfill the desire in the user story to count the number of classes in a given file without docstrings
Type of Change

[x] Feature
[ ] Bug fix
[ ] Documentation

Contributors

@rachaelharris

Reminder
All GitHub Actions should be in a passing state before any pull request is merged.

This is a good additions


Add support for returnObjects: true

What are you trying to accomplish?
Fixes #171
Add support for {returnObjects: true} parameter of t()
Checklist

[x] I have added a CHANGELOG entry for this change (or determined that it isn't needed)

/snapshot