Any way to close the dropdown menu form one of the callback functions?

I want the same behaviour the dropdown has when it is in single mode. A click on one option close the dropdown. Otherwise it would be good a 'close' button since for some un experienced users it might be confusing or not easy to understand that they need to click outside the dropdown to close it.
I tried to call toggleCheckboxes() from the callbacks without succeed, but I'm not expert in Angular.

Correct me if I'm wrong.. A click in one option close the dropdown in multi select mode?  Then, how will you select other options if it closes right after you click something?
I believe clicking outside (or the main button itself) to close a dropdown is a classic behaviour of these elements, including the native HTML dropdown. I believe non-experienced users won't have much problem.

Yes, that is the common behaviour, but my client asked me for that. And I also had the feeling that was not clear, I think it is because this is a very different dropdown, and opened and with many options, sometimes doesn't look like a dropdown.
A quick solution was to do jQuery('body').trigger('click') on the callbacks (element clicked, select all and select none).
If you want to select more than one option, you open the dropdown again and click another option. At least for my project, that looks like the best behaviour.

Hi @carlosminatti ,
I can't do that, as it will make the dropdown counter-intuitive. Imagine the rest of the users who are used to the common behavior.
If you want something like that, I suggest you to either modify the directive code yourself, or use a different directive. Slightly different, but this one should work just like your requirement: https://github.com/angular-ui/ui-select

i want to add a cross x mark on the selected options of my dropdown list. how can i do that in html/css and angular


How do restrict the app-auth for just a subset of site collections?

Is app-auth "all or nothing?"
I want to know how to specify an app-auth key but only for a certain subset of site collections.
So let's say you use the app-client-id, and app-client-secret... then you authenticate to your sharepoint online account using<EMAIL_ADDRESS>to get the refresh token.
Does my User's access<EMAIL_ADDRESS>dictate my permissions? Or does the app-auth key give me basically all the access in the world?

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

ID: 35a3017a-6d6c-237f-fc57-94247a8f7bf6
Version Independent ID: 93413445-18fa-9245-ea64-1c1d789dce1e
Content: Granting access using SharePoint App-Only
Content Source: docs/solution-guidance/security-apponly-azureacs.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

It doesn't work like that... it's tied to the TLD, so the tenant domain and thus, all site collections.

Thank you for your prompt response @andrewconnell ! super appreciated.

@andrewconnell so app-auth is not meant to be used for fine-grained access to a subset of the tenant or site collections, correct?
is the correct alternative a sharepoint service account authenticated through SAML?

@nddipiazza said:

app-auth is not meant to be used for fine-grained access to a subset of the tenant or site collections, if I read that correct.

Correct
@nddipiazza said:

is the correct alternative a sharepoint service account authenticated through SAML? are there additional options?

Not for app-only that I can think of


'url' is inaccessible due to 'internal' protection level

What?
The sample code in the reference cannot be compiled since url is inaccessible.
func onMultipartMessage(_ message: PCMultipartMessage) {
  for part in parts {
    switch part.payload {
      // other cases

      case .attachment(let payload):
        payload.url() { downloadURL, error in    <--- here
          // do something with the download url
        }
    }
  }
}

Environment

PusherChatkit version: 1.4.2
Xcode version: 10.1
Swift version: 4.2

@caldrian new version incoming in the next half an hour. Thank you for bringing this up.


add single source capability for defining C/C++ objects/functions inline to cython pyx files

I just added inline_module's and submitted a PR today here: https://github.com/cython/cython/pull/555
My intent was something like OpenCL's JIT/AOC compilation (PyCUDA/PyOpenCL also work this way) which is simple and high performance, and I saw cython had this capability and just needed a little bit of a wrapper to expose it better.
But after trying to put it into use for my own problems, where I would regularly want to use Eigen or other integration problems I realized I'd still have to write and save [many] code fragments to other files which cythonize-ing would read in through includes, when you need to work with something that doesn't have a cython cimport module.  This is pretty ugly and doesn't achieve the easier flow that I had wanted.
However if there was a way to embed C/C++ inside a pyx file (that is inline to strings), there would be no such issue.
edit: To clarify on where I had envisioned the workflow going, something closer but between the spectrum of asm block and  SYCL is desired, maybe I just need to give the verbatim docstream method a test drive, but at the time I opened this issue I was also spotting that there would be sections/fragments of python functions that I'd just want to "hop" out of python to c++ in, and in that sense it is a more powerful single source model than explicit function boundaries (of which the functions would be numerous, weakly purposed and have some boilerplate tax) - this is why I indicated preference for inline strings/asm like constructs and while the situation is better now with the verbatim docstring based implementations, there is still some gaps towards better usability/boiler plate reduction.

This would be a great feature that I have wanted many times. There are several situations where I need like 10 lines of C or C++ code to bridge something that Cython cannot quite do and thus have to add a separate file for. Would be wonderful to have single-source capability. Sad to see no response has been made here. One suggestion I have is:
cdef extern from _:
    """
    C or C++ code goes here in this string. Gets included as if it had been in a header file (instead of using _ above).
    """
    # Then the normal extern definitions go here.

Duplicate of #1915.
https://cython.readthedocs.io/en/latest/src/userguide/external_C_code.html#including-verbatim-c-code

One option that might work without too much extra syntax would be to overload eval and exec.
So something like:
something = eval("a ? 1 : 2;",  # just some example C code
                            c=True)

exec("""for (int x=0; x<10; ++x) {
    a += x;
}""", c=True)

Three obvious issues would be:

Return type of eval? It'd probably make sense for Cython-level type checking to just not happen for it.
Access to variables outside the statement - a would most likely not work because Cython typically mangles variable names. This would either require the user to set the cname of variables that they want to access, or for Cython to try to scan the string and swap a to the cname (which would probably be very difficult to do right, so undesirable).
When used at module scope I suspect users would expect to be able to create function definitions (which wouldn't work because the code would be inserted inside PyInit_modulename).

With those in mind I'm not really convinced this is viable.


Add Issue Templates

I've created some issue templates based on my comments #62. I've created a template for apiable, documentation, feature request, general issues and bug report. I've also created a label called Needs Review so it's easier to see which issues you haven't looked at yet when viewing the issues list.
I've marked this PR as a draft so you can go through and review the templates first before merging this PR into the repo. You can go to my repository in order to test each one and offer feedback.

I updated the stale wording again since I think it could be improved. Was thinking of something along the lines like:
Thank you for creating this issue. In order to keep issues relevant any issue hasn't had any activity in ninety days gets automatically marked as stale. If you think this issue is still relevant leave a comment otherwise it will get automatically closed in seven days.
I find it's better than what is there currently but would like to get your thoughts before I go and change the wording.
As mentioned in #99 I left in the action for automatically locking issues sixty days after it's been closed. I can remove that action if you'd like or maybe add a message saying that the issue is being locked and you can create a follow-up issue if needed?

Just looking at this now, and overall looks great! I really appreciate how you laid all these changes out, since it made it very easy to go through them.
My only ask is if you could remove the issue locking after 60 days for now. I totally get the sentiment of it, but I don't think it's necessary at this point, and might create more issues than it's worth. After that I'll merge away!

You do not mess around, and I can't say how amazing that is! Merged and online 🥇


tweak checkpoint directory to match README?

Doing
swift run -Xlinker -ltensorflow -c release MiniGo

like the README suggests in the parent directory is currently failing because the logic assumes it is in the MiniGo directory.  On the flip side doing a swift run inside the MiniGo directory will fail if this get merged.  Not sure what the correct/desired behavior is.  Can also write logic to check both locations but figured I would ask first.

Thanks for catching this!
I think there are two ways to fix this.

Assumes the MiniGoCheckpoint folder is under swift-models/MiniGo, i.e., sub-folder.
Assumes the MiniGoCheckpoint folder is under swift-models, i.e., the parent folder.

Both need to update the README file to make it super clear to users.
I personally prefer the second one as we recommend users to run the code in parent folder via swiftPM. but open to the first one.
Assuming we fix it by taking the second one. I think we could update the README as
To get the ckpt:
cd swift-models
mkdir -p MiniGoCheckpoint
gsutil cp 'gs://minigo-pub/v15-19x19/models/000939-heron.*' MiniGoCheckpoint/

To run
cd swift-models
swift run -Xlinker -ltensorflow -c release MiniGo
Then this will align with other models, like ResNet.
What do you think?

@xiejw I like that solution better, please see update!


Add properties to subjects  CR 10-11

Description of the change: (free-text)
Add these properties to subjects:

creator - The person, organisation or system that created or edited the subject.
modified - The date (and, optionally, the time) when the property was last modified. The initial value is the date (and, optionally, the time) of creation of the subject.
why - Why the metadata has been included.
how - The means by which the value was extracted from the content.
confidence
relevance
broader

Rationale for the change:
These properties are all available in NewsML-G2.
Reference to related IPTC discussion(s) (on Yahoo groups etc):
Formalised change request: (spec table etc.)
Add the following properties, all as strings, unless noted

"creator" The person, organisation or system that created or edited the subject.
"modified" (date time) The date (and, optionally, the time) when the property was last modified. The initial value is the date (and, optionally, the time) of creation of the subject.
"why" Why the metadata has been included.
"how" The means by which the value was extracted from the content.
"confidence"
"relevance"
"broader"

JAYSON:
All sound useful (all optional of course) and I am okay with this request

@hledwards as discussed, please take a look at this and update with new requests and/or examples if necessary!


Break out linting commands to a separate image

Our linting CI jobs are rather self contained and as such would benefit from being based on a separate image.
This will allow much faster feedback than we currently have (50% less time) and would also be "greener" as we'll be pulling smaller images.
We could also remove these tools from the base image (once unused) which should reduce it's size (albeit the reduction is unknown at this point and could prove rather negligible).
An added benefit is allowing these commands to be embedded in git commit hooks and run on each commit (or perhaps push) since the image, even if needs to be pulled, is very small.
A POC proves that an alpine based image results in ~80M size instead of the ~870M the current base image is:

Compared with execution with the current image:

While this might not seem a great saving (in absolute terms it's ~15s per job), in the long run the pros could outweigh the cons.
The work estimation is rather small as the POC already proves such a feat is quite obtainable and as such shouldn't take to long to implement, should we chose to do so.

Resolved by https://github.com/submariner-io/shipyard/pull/564


Motion Detection Start and Stop

Hello,
The same way we have the FIFO1 pipe to trigger Camera Start,
is it possible to automate the Motion Detection Start and Stop?
The reason is that I have already set up a cron that checks whether people are present at home, so when we leave house, I would like to auto-start Motion Detection,
and when we come back, auto-stop Motion Detection.
Thank you.

Nevermind!! I just have to use this:
echo 'md 1' > /var/www/FIFO


Virus Warning with DryIoc Version 5.4.3 on Some Antivirus Software

Description
We encountered an issue where our antivirus software flagged the DryIoc package version 5.4.3 as containing a virus (VirTool.MSIL.Obfuscator.f281). This warning appears when the package is installed via NuGet. However, using the DryIoc.dll directly from the same version does not trigger the antivirus warning.

Steps to Reproduce

Install DryIoc version 5.4.3 via NuGet.
Compile the project.
Observe the antivirus software warning about VirTool.MSIL.Obfuscator.f281.

Expected Behavior
No antivirus warnings should be triggered when using the DryIoc package.
Actual Behavior
Antivirus software flags DryIoc package version 5.4.3 as containing a virus.
Workaround
Using DryIoc.dll directly from the same version (5.4.3) does not trigger the antivirus warning.
Environment

DryIoc version: 5.4.3
.NET version: [Your .NET Version]
Antivirus software: [Your Antivirus Software]

Antivirus Software Details

QiAnXin TianQing (奇安信天擎):

Download link

Huorong Security (火绒安全):

Download link

These antivirus software packages can be used to verify and potentially resolve the issue.
Additional Information
This issue might affect other users, so it would be beneficial to investigate and resolve the root cause. We wanted to inform you about this and share our workaround. Please let us know if any further information or assistance is required.
Thank you for your work on DryIoc!

@raymond-liao Hi, interesting issue.
I am not sure what causing the issue. Probably something inside the package bundle.
Could you check if the same happens for the latest 6.0.0-preview-07 ?


can't load offline map with .tpk file

can't load offline map with .tpk file
my code like this

error log
flutter: Error: ArcgisError{code: 7, errorMessage: Not found., additionalMessage: Mobile map package is missing a .info file., innerErrorMessage: null, errorDomain: ErrorDomain.unknown}

@xyhuangjia
This error message implies that your ArcGIS application could not find the required .info file in your Mobile Map Package (MMPK).
A Mobile Map Package consolidates maps into a single file (.mmpk) that can be shared across ArcGIS products and used even when offline. The .info file is part of the MMPK and is necessary for the ArcGIS system to interpret the data.
Here are a few things you might check or try to fix this error:

Verify that the Mobile Map Package file is intact and complete. If you've downloaded it from a source or moved it, it's possible the file might have been corrupted or some parts of it may not have been transferred. If this is the case, you might want to try to download or move the file again.

Ensure that the MMPK file is accessible. Make sure that the path to the MMPK file is correct, and that the user running the ArcGIS software has the necessary permissions to access the file.

Recreate the Mobile Map Package. If you've created the MMPK yourself and it's missing the .info file, there might have been an issue during the creation process. You might need to recreate the package and make sure it completes without errors.

Update your ArcGIS software. If you're running an older version of the software, it might not be able to handle certain MMPK files correctly. Make sure you're using a version of the software that is compatible with the MMPK file you're trying to use.

If none of these suggestions solve the problem, it would be advisable to contact ESRI support or check in GIS-related forums like Geonet for a more specific solution.


Add multiple domain support in Access-Control-Allow-Origin

User can now provide a comma separated list to the CORS_ALLOW_ORIGIN environment variable (i.e. https://example.com,https://another.example.com). The backend will make the appropriate checks and include or exclude the header.

Stellar! Thank you very much @wphan for the contribution.


conmon: allow to specify additional arguments to the OCI runtime

we can specify multiple arguments that are passed down to the OCI
runtime, before the command name:
--runtime-arg --debug
will be used as:
RUNTIME --debug COMMAND ...
Needed by: https://github.com/containers/libpod/issues/2285
Signed-off-by: Giuseppe Scrivano<EMAIL_ADDRESS>

LGTM, very elegant!

/test e2e_fedora
/test e2e_rhel

LGTM, will be helpful for libpod work

/retest

/test e2e

/retest

We need to track the upstream commit that broke e2e. Most of them are failing with a similar issue.

/test e2e

/test e2e_rhel

Rebasing to master should fix the tests here.

/retest

/retest

I think rebase to master should work now after latest CI fixes.

/test kata-containers
/test e2e_fedora
/test e2e_rhel

/test kata-containers

/test kata-containers
/test e2e_fedora
/test e2e_rhel

/test kata-containers
/test e2e_fedora

/test kata-containers

/lgtm


How to support SDXL ControlNet and Inpainting

How to support SDXL ControlNet and Inpainting

@hitlmt Are you using diffusers?
Here are examples of:

controlnet:  https://github.com/siliconflow/onediff/blob/main/examples/text_to_image_controlnet.py
inpainting: https://github.com/siliconflow/onediff/blob/main/examples/reuse_pipeline_components.py

Too old to follow, and will be closed. Feel free to reopen it.


REFUGE2018 Glaucoma validation set label

Hello, do you mind giving the groundtruth of the REFUGE2018 Glaucoma validation set, the link to Baidu Brain is broken.

Sorry for the delay.
You may download the dataset from this link: https://drive.google.com/file/d/1B7ArHRBjt2Dx29a3A6X_lGhD0vDVr3sy/view.
Domain 4 is the validation dataset of REFUGE.

@EmmaW8 Zhu told me to register for the refuge2 competition and get data labels by the way. Thank you anyway...


Changed req.redirect to res.redirect in README.md

There seemed to be an err in the README.md since req.redirect is not a function. The correct way seems to be res.redirect. https://expressjs.com/en/api.html#res.redirect

Yep, you're right!


More Than one iOS device

Can I use more than one iOS device to connect? I've added on but when I try to add the second device it will not find home bridge.
Thanks.

Duplicate of #82 I believe - reopen if that's not the problem though!


Perl not documented

Code:
say $*PERL.perl

Result:
Perl.new(compiler => Compiler.new(id => "5936ABF0B3CFA2F3A55C158CFCDB944A79EE5D42.1459591474.45324", release => "", codename => "", name => "rakudo", auth => "The Perl Foundation", version => v2016.03, signature => Blob, desc => Str), name => "Perl 6", auth => "The Perl Foundation", version => v6.c, signature => Blob, desc => Str)

Note that $*PERL variable is documented and is searchable, but Perl class itself is not documented.

Not specced. I see that its existence is checked, but nothing else.

Seems to be a dup of https://github.com/perl6/doc/issues/503.


Update Model Catalogue docs

When trying to import a PyTorch model as explained in the docs (https://juliatrustworthyai.github.io/CounterfactualExplanations.jl/dev/tutorials/model_catalogue/#Importing-PyTorch-models). I get the following error:
ERROR: MethodError: no method matching pytorch_model_loader(::String, ::String, ::String, ::String)
According to Patrick, the code for importing has been moved to TaijaInteroperability.jl. This has to be updated in the docs.

Thanks @izagorac! In the meantime, you should be able to use the method as expected by first importing the package:
using CounterfactualExplanations
using TaijaInteroperability
...

I'm a bit surprised thought that the error message didn't already hint at this, since we've deprecated the method.
If it's not too much trouble, can you post a minimum working example here including information about the exact package version you're using?
Thanks!
@kmariuszk for info

Find my code and error message below. Im using CounterfactualExplanations v.0.1.31.
model_loaded = CounterfactualExplanations.Models.pytorch_model_loader(
        "$(pwd())/docs/src/tutorials/miscellaneous",
        "neural_network_class",
        "NeuralNetwork",
        "$(pwd())/docs/src/tutorials/miscellaneous/pretrained_model.pt"
    )

M = CounterfactualExplanations.Models.PyTorchModel(model_loaded, counterfactual_data.likelihood)

Can you try instead:
model_loaded = TaijaInteroperability.pytorch_model_loader(...)

It gives the exact same error. I'm using TaijaInteroperability v.0.1.0

Hmm alright thanks! @kmariuszk any idea what's going on here?

Hey @izagorac, did you also activate the PythonCall package in your environment?
 We did this to minimize the pain of users waiting for both PythonCall and RCall pre-compilation and to minimize the chance of errors caused by using both of these packages in the same environment.
Let me know if it solves the problem!

Also, @pat-alt regarding this bit:

I'm a bit surprised thought that the error message didn't already hint at this, since we've deprecated the method.

We didn't release the latest version of package and thus users can still use PyTorch functionality in the original way, i.e., using CounterfactualExplanations, PythonCall

Also, @pat-alt regarding this bit:

I'm a bit surprised thought that the error message didn't already hint at this, since we've deprecated the method.

We didn't release the latest version of package and thus users can still use PyTorch functionality in the original way, i.e., using CounterfactualExplanations, PythonCall

Ah yes, of course! Thanks for checking @kmariuszk

Thanks @kmariuszk and @pat-alt, it works when using PythonCall. Before I was using PyCall, does that mean that that package is not necessary anymore?

Thanks @kmariuszk and @pat-alt, it works when using PythonCall. Before I was using PyCall, does that mean that that package is not necessary anymore?

Yes, we decided not to use PyCall quite some time ago due to some bugs it was introducing. We're going to work on updating the documentation in the upcoming two weeks, so (hopefully) you won't experience such ambiguities anymore


Remove RequiresLeader option from persistent subscription

Changed: Remove RequiresLeader option from persisent subscription API.
Addresses issue #176

@YoEight I haven't run tests locally, so if there are any issues with the CI run I will adjust after. Thank you

@YoEight
Warning: go-version input was not specified. The action will try to use pre-installed version.

This shows for both the windows & mac checks. It seems the following var in Github is not set. Can you check when you have time?
    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-go@v5
        with:
          go-version: ${{ vars.GO_VERSION }}

Hey @seanppayne , you can't resolve that variable because you did a fork. I came up with patch to fix it and I suggest you to rebase your fork when that PR got merged. Sorry for delaying your review:
https://github.com/EventStore/EventStore-Client-Go/pull/182

@YoEight The tests passed. Please take a look when you have time.


gateway-api: requestHeaderModifier in HTTPRoute cannot set Host header

Bug description
Affected product area (please put an X in all that apply)
[X] Networking
Expected behavior

When requestHeaderModifier sets Host header and controller should add the header via different directive.
:authority also cannot be set with the same reason.

Steps to reproduce the bug
1. Try to set Host: foo.example.com in HTTPRoute
cat <<EOF | kubectl apply -f -
apiVersion: networking.x-k8s.io/v1alpha1
kind: HTTPRoute
metadata:
  name: host-header
spec:
  gateways:
    allow: FromList
    gatewayRefs:
    - name: test-gateway
      namespace: knative-serving
  hostnames:
  - test.example.com
  rules:
  - filters:
    - requestHeaderModifier:
        set:
          Host: foo.example.com
      type: RequestHeaderModifier
    forwardTo:
    - port: 80
      serviceName: ingress-conformance-host-rewrite-njaysumq
      weight: 1
    matches:
    - path:
        type: Prefix
        value: /
EOF

2. istio-ingressgateway rejects the HTTPRoute
$ kubectl  -n istio-system logs istio-ingressgateway-d5dfc5d8b-cl8bg -f
  ...
2021-04-01T06:25:15.732110Z	warning	envoy config	gRPC config for type.googleapis.com/envoy.config.route.v3.RouteConfiguration rejected: :-prefixed or host headers may not be modified

$ kubectl  -n istio-system  logs istiod-85f9ccb96d-ks8hd |grep -i header
2021-04-01T05:33:04.824343Z	warn	ads	ADS:RDS: ACK ERROR router~<IP_ADDRESS>~istio-ingressgateway-d5dfc5d8b-vms5m.istio-system~istio-system.svc.cluster.local-1 Internal::-prefixed or host headers may not be modified

Version
$ istioctl version
client version: 1.10-alpha.75974d56a327e9857f36d17151b2ef3146745b92
control plane version: 1.10-alpha.75974d56a327e9857f36d17151b2ef3146745b92
data plane version: 1.10-alpha.75974d56a327e9857f36d17151b2ef3146745b92 (2 proxies)

This is broken in VirtualService as well it seems...


⚠ Jerry Savelle Ministries International Australia has degraded performance

In d72191c, Jerry Savelle Ministries International Australia (https://jsmiaustralia.org) experienced degraded performance:

HTTP code: 200
Response time: 8263 ms

Resolved: Jerry Savelle Ministries International Australia performance has improved in 9dfe0cf after 6 minutes.


BUGFIX: Fix support for JWTs without 'kid' header

In IdentityToken::hasValidSignature() the method getMatchingKeyForJws() can be invoked with a $keyIdentifier
parameter of null. But the signature and implementation didn't support that case.

FYI: I created this PR against the 2 branch in case you want to release bugfix versions. It probably has to be upmerged to master for a new 3.x release

Still, if $keyIdentifier is not null but the array key 'kid' does not exist in $key, PHP might fail with an "invalid array offset" error, or?

@robertlemke doh, you're right of course.. Sorry for that, now it should be fine!?


JATOS stops sending data

I'm running a study built with JSPsych and using Jatos 3.9, I've had about 10-20% of participants reporting getting stuck on a 'Sending data' page after completing the experiment. However, I send data trial by trial using jatos.appendResultData. Looking at the recieved data, I notice that data stopped sending much earlier than the end but participants were still doing the study.
I've noticed a few issues raised with version 3.8 that were similar. Is there any way to circumvent the issue from my side, or do I need to wait for it to be fixed by the developers?
Thank you.

There is nothing to be fixed that I am aware of right now. But this is a recurring issue and can have many reasons. Some of them being:

result data being send to the JATOS server being to large and JATOS or an intermediate network component blocking them
some issue in the experiment's JS code that causes it to stop in the end of the experiment
unstable/slow/low bandwidth internet connection of the participants
the JATOS server being unstable (e.g. host's OS, JATOS or proxy)

Some questions that come to my mind:

Have you had a look at your participant's browser when this happened, especially any errors in the Dev Tools Console and the Network tab?
Do you know the size of your result data?
Have you checked your JATOS' application logs for any errors or exceptions?

Hello,
Thank you for the input. I don't believe the data size is too large, but it's possible that the data appending requests are too frequent, increasing the chance of facing an error. The trouble is that once such error is faced once, no other attempts at retrieving data is done (or at least not sucessful). Further, I don't believe it's an issue with my code, as it runs for most participants, and if it were, it would not be an error at the end. The loss of server connection and data transfer happens during the experiment, the message stuck saying sending data at the end of the experiment is simply a symptom, as I do not send data at that stage.
Of course, it is possible, even likely, that it's a combination of connection problems for the participants themselves or JATOS server. Nevertheless, the problem is less about these, as they are inevitable, but JATOS's inability to recover from them.
Regarding the questions:

Have you had a look at your participant's browser when this happened, especially any errors in the Dev Tools Console and the Network tab?
While I track the browser used, I'm not sure how to do that.

Do you know the size of your result data?
The total data packet could be as large as 2.5MB, but with recent iterations I've reduced it to 350KB. But I append the data on
each trial, meaning it should only be a few bites size. It may be more the frequency of data sending requests than the size.

Have you checked your JATOS' application logs for any errors or exceptions?
Nothing shows up, only the last attempt (unsure whether it's the sucessful last or the one that face the error) at appending
data, after which no logs are kept anymore.


Refactor benchmark jobs in ci.yml

Following this discussion on Discord.
These jobs in the ci.yml workflow are increasing infrastructure complexity and are a bottleneck on total presubmit time:

build_benchmark_tools
build_e2e_test_artifacts
test_benchmark_suites
compilation_benchmarks (opt-in)
execution_benchmarks (opt-in)
process_benchmark_results (opt-in)

When no benchmarks are enabled, the build_e2e_test_artifacts -> test_benchmark_suites jobs still run. The intent here was to verify that benchmark suite models can be successfully compiled and executed correctly even if they are not benchmarked. The reality is that the test_benchmark_suites workflows are only running one test: iree/tools/test/iree_run_module_correctness_test, and the build_e2e_test_artifacts workflow takes 9 minutes to build multiple platform variants of the same program. The coverage provided has redundancies and the overall setup has multiple excess minutes of overhead (not even counting time spent waiting for expensive self-hosted runners.
I propose we

remove the test_benchmark_suites matrix entirely
move build_benchmark_tools and build_e2e_test_artifacts to opt-in based on benchmarks being requested (using a preset?)
(as needed) add correctness integration test cases for benchmark programs to experimental/regression_suite rather than try to bolt something on to the existing benchmarking infrastructure like those "run module tests"
(as cleanup) eject the benchmark jobs from ci.yml and place them in benchmark.yml

they can still use presubmit artifacts via GCS storage or GitHub Actions Artifacts. Ideally they could also run on release artifacts
they can continue to use configure_ci.py and PR labels as they currently do

cc @pzread

FWIW, I looked at the logs for a sample run of build_e2e_test_artifacts to see if there were some easy optimizations to make and it doesn't look like it.

4m spent compiling various programs (no huge outlier program like other jobs have had before)
1m40s spent uploading outputs
1m fetching Python deps from pip
40 seconds downloading a Dockerfile
10s configuring CMake
< 5s downloading sources and importing them

just... lots of overhead and then a decent number of programs to compile

SG. I'll say from time to time we catch problems with build_e2e_test_artifacts, but we can switch and observe failure in postsubmit, then pick the important models into regression_suite

For the test_benchmark_suites, @hcindyl might know more about it

I'm seeing if I can pull some statistics for how long each job spends queued right now. (If that ends up taking longer than I'd like I may just proceed without data, but I'd like to know...). For queueing, we could also add more runners, or switch those jobs to use GitHub-hosted runners instead of our own.
I'd rather just eliminate the overhead and complexity instead of throwing more/different resources at it though. Queueing time / job orchestration is only part of the costs.

See https://github.com/openxla/iree/pull/16438#issuecomment-1949078215 for the use of test_benchmark_suites. It is placed in the CI to check LLVM codegen regression for a few realistic models. I have lost track on when they got turned off in https://github.com/openxla/iree/tree/main/tests/e2e, but historically the test gave us some warnings of the RISCV codegen breakage before it trickles down to the project that uses IREE, e.g., https://opensecura.googlesource.com/sw/vec_iree and https://opensecura.googlesource.com/ml-models-public

Slight adjustments to the plans from the initial issue description:

build_benchmark_tools, build_e2e_test_artifacts, and test_benchmark_suites are now opt-in and will only run when benchmarks are requested (including all postsubmit runs)
test_benchmark_suites has some useful coverage again, but those tests should be moved to a regular (non-benchmark) presubmit CI job - likely under experimental/regression_suite, but cross_compile_and_test in ci.yml could also be used in the short term
the benchmark jobs should still be moved out of ci.yml


pythonPackages.scikit-build: fix by bumping pytest-virtualenv, -shutil, -fixture-config, -server-fixtures

Motivation for this change
scikit-build appears to have broken when pytest was bumped to 4.x. However the only thing complaining about the presence of pytest4 was one of its dependencies, pytest-virtualenv. So what I did instead was bump that along with pytest-shutil, pytest-fixture-config and pytest-server-fixtures because they all actually come from the same "master package" https://github.com/manahl/pytest-plugins and all have a recent 1.7.0 release which makes them compatible with pytest4 and allows me to remove any pytest_3 requirement for them.
Finally I had to disable test_source_distribution in scikit-build to get its tests to pass. Previously I think it had actually been auto-skipping this test so I don't think I'm actually net disabling anything.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Included in https://github.com/NixOS/nixpkgs/pull/63169


option.log.child is not a function

hello there,
new FtpSrv by this
const ftpServer = new FtpSrv( {
  ...
  log: console
  })

what I got
xxxxxx\node_modules\ftp-srv\src\connection.js:18
    this.log = options.log.child({id: this.id, ip: this.ip});
                           ^

TypeError: options.log.child is not a function
    at new FtpConnection (xxxxxx\node_modules\ftp-srv\src\connection.js:18:28)
    at Server.serverConnectionHandler (xxxxxx\node_modules\ftp-srv\src\index.js:48:24)
    at Server.emit (node:events:390:28)
    at TCP.onconnection (node:net:1577:8)

it seems that I cannot change log instance
https://github.com/trentm/node-bunyan#logchild
why not make it custom by user self?

To disable the logs, you can use this.
const bunyan = require("bunyan");

const logger = bunyan.createLogger({
    name: "null",
    stream: fs.createWriteStream("/dev/null"),
});

const ftpServer = new FtpSrv({
  ...
  log: logger
})


[PROPOSAL] Define an integration certification process

Purpose:
As more and more third parties implement OpenLineage the need has emerged for a certification process.
There are two distinct cases for this:

Producers: They want to verify the metadata they are producing is valid according to the spec.
Consumer: They want to verify they understand metadata correctly and need example of all facets.

Both want to be able to keep up to date with the evolution of the spec and capture what they support or not. In particular, what facets they consume or produce.
Proposed implementation
Create tooling and a record of implementations of the spec.
Currently we have produced examples of all the core facets that can be used by consumers to validate their implementation. This is a bit short but provides a starting point.
Conversely, producers can use the json schemas to validate the syntax of the metadata they produce but not the semantics.
Input is requested to further this proposal
Discussion doc:
https://docs.google.com/document/d/1h_PI0HLX7ECVll068EmExZHF5xVYqVNGsNj7DCPiB5Y/edit

https://github.com/openlineage/compatibility-tests - current implementation.


[PT Run] [Settings plugin] New settings

Summary of the Pull Request
What is this about:
Adding new settings:

Screen saver
Connect wireless display pannel

How does someone test / validate:
Local build.
Quality Checklist

[x] Linked issue: #14729, #17159
[x] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[x] Installer: Added/updated and all pass
[x] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[x] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Tested on Windows 10 and it doesn't have all of them, but the PR LGTM!
You've tested on Windows 11, right? @htcfreek .
Thanks for the contribution.

No. On Windows 10.
What do you mean with "doesn't have all of them"? Can you please describe this a bit more.

Tested on Windows 10 and it doesn't have all of them, but the PR LGTM!
You've tested on Windows 11, right? @htcfreek .
Thanks for the contribution.

No. On Windows 10.
What do you mean with "doesn't have all of them"? Can you please describe this a bit more.

Nevermind, I was confusing the aliases in the resources files as actual different screens, but they're just aliases. PR is good.

@jaimecbernardo
Are we waiting for something or is there something that needs my attention?


Update to current HAPI format

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

25 of 26 (96.15%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.0%) to 85.621%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/hdx/scraper/hno/plan.py
25
26
96.15%

Totals

Change from base Build<PHONE_NUMBER>:
1.0%

Covered Lines:
262

Relevant Lines:
306

💛  - Coveralls


Highlight word under cursor

Is it possible to make this package listen position of cursor and automatically treat word under cursor like selected, so same words is highlighted without need to selection?

+1

+1 also.

+1
It would be sorta like: https://atom.io/packages/refactor

+1

Duplicate #96


Use definitions not $defs

Issue reported at https://github.com/gbif/ipt/issues/1955#issuecomment-1501819698

We adopted $defs based on this recommendation:

Although we can identify any subschema using JSON Pointers or named anchors, the $defs keyword gives us a standardized place to keep subschemas intended for reuse in the current schema document.

The issue with it not being recognized seems to be related to the datapackage-java implementation, as commented by @mike-podolskiy90 in https://github.com/gbif/ipt/issues/1955#issuecomment-1504784813:

Data package fails to generate and error is displayed:
Reference #$defs/version cannot be resolved

It happens under the hood in the frictionless datapackage-java. It can be a problem with the datapackage-java itself though. Anyway, I changed it as described above, and it works just fine: https://rs.gbif.org/sandbox/experimental/camtrap-dp/0.6/profile/camtrap-dp-profile.json

We would therefore prefer to keep the $defs convention and have it fixed in datapackage-java. @mike-podolskiy90 have you been in contact with the developers? If it cannot be fixed in datapackage-java, we could either change it in Camtrap DP or (manually) for the IPT schemas.

It's fixed in the profile schema (rs.gbif.org). I think that is enough


Arraywrapper4

Add more tests for const in Fortran wrappers

Functions which return const int *
Arguments of type const int **arg +intent(out)

Also, does numpy allow for pointers to const data?

Here is an attribute that sounds like it makes the data const: https://numpy.org/doc/stable/reference/c-api/array.html?highlight=writeable#c.NPY_ARRAY_WRITEABLE


Wrong start page in users info

In the home page at start before login we have :
Name: {{name}}

@gioppoluca The issue should be fixed. Could you test it again and close the issue?


"Code coverage" badge is broken

Always shows 100%

Hi, @leventov . The status icon of Teamcity is also broken. Maybe the administrator should follow
some steps in http://www.mikeobrien.net/blog/setting-up-teamcity-status-badges-with-shields-io to make it correct.

Currently, it shows Coverage|unknown

This is fixed by https://github.com/apache/druid/pull/8388


Add random delay when program output is cached

Instead of directly returning cache contents, add small 500ms-2s delay. This keeps user from suspecting weird stuff.

Is this done similar for the gotour?

I don't know how the gotour does it :-) The dlang-tour doesn't have restrictions imposed regarding random numbers or the timezone as the online code is just run within a container. That of course means that once cached those values are output again. This feature is just about adding a random delay before returning output to the user to make the output seem to be real-time.


Add color support for copy and toggle buttons

This adds color support for two more web component extensions: sphinx-copybutton and sphinx-togglebutton. It does this by adding another SCSS extensions file for each, and adds them to our docs.

sphinx-togglebutton seems (almost) redundant with sphinx-design dropdowns... I mean, the functionality is the same, but the styling is different because togglebuttons add that functionality to existing sphinx admonitions.  I'm not sure what else to say about that other than it feels non-DRY but I guess it makes sense for the theme to work smoothly with either one? 🤷🏻

I'm not sure what else to say about that other than it feels non-DRY but I guess it makes sense for the theme to work smoothly with either one

My feeling is that this theme should support a variety of extensions and sub-packages, as long as it does not require a lot of extra maintenance burden. In this case, it just means adding some special-cased CSS which shouldn't be too hard to maintain.
In the future if there is consolidation around a single package, we could re-work our CSS here to unify under a specific package, but at least for now there are multiple projects using each (for example, Jupyter Book / MyST-NB ship with sphinx-togglebutton, while Sphinx Design tends to be used more by end-users)


Different config for each Xcode Project

Hi, this is not bug! I have suggestion, would it be possible to chose specific config for project in Xcode, lets say, that I have git repo, where multiple programmers are working on project and the config is in this git repo as well. So it would be awesome if I could chose what config to use for specified project. For example chose by default config in root of project...

Anyway, great job for making this plugin!

Completely irelevant, my bad...

The plugin will use the first configuration file found in this order:

Configuration file found in the current directory of the source file.
Each parent directories of the source file.
Your Home folder and ~/.uncrustify/uncrustify.cfg

So lets say you have a project with subproject1 (team1) and subproject2 (team2), you can use a structure like that:
root project folder
	| subproject1
		| config_file -> team1

	| subproject2
		| config_file -> team2

There are some differences between Clang and Uncrustify but the whole logic is explained in the preferences window (see attached screenshot).

No worries.


Need a simple function for getting online count

Currently you have to filter the members to get the count, and it's overly complex -- even with the widget.json immediately shows the online count right near the parent, effortlessly. Since the point of a wrapper is to make things easier, this seems like an intuitive addition.
If anyone has free time and knows the code well, this would be an awesome helper function to add.

if you can't handle members.filter(m => m.presence.status === 'online') maybe you shouldn't be a computer programmer


Could davinci UI display in English?

I want to set all the information dispaly use English , but some UI information are Chinese , could I change to English? How to setup? thanks~

是否我可以在源码中更改界面的语言呢？入口在哪里呢？如下图中文字的地方

#1253

Related to #1743


Make sure that scopes are always correctly set

There seem to be a bug related to setting scopes for users. They may not correctly update all relevant tables. Test and fix.

Failed to replicate. I'm leaving this open for now though. Please provide a test case if you still encounter this issue.


Check that if A calls B, B is defined before A for WebGPU

Fixes #2067

I am pretty sure that all of the failing smoke tests are ones that fail without my PR.


patternProperties does not work

schema:
{
  "$schema": "http://json-schema.org/draft-06/schema#",
  "$id": "http://json-schema.org/draft-06/schema#",
  "type": "object",
  "patternProperties": {
    "^[A-Z]{2}+$": {
      "type": "array",
      "items": {
        "type": "string",
        "maxLength": 2,
        "minLength": 2
      }
    }
  }
}

Object:
{"sdfsdf":["aa","bb"]}

false positive:
The supplied JSON validates against the schema.

Tried for same object:
    "^[0-9]+": {```

Same output: `The supplied JSON validates against the schema.`

"patternProperties": {
    "sdfsdfsdf": {

Oh come on The supplied JSON validates against the schema.

https://github.com/justinrainbow/json-schema/pull/585

Your sample data is supposed to be valid, as it does not match any rule which restricts it. This behavior is correct.

Yeah, pattern is totally ignored. Great feature.

It isn't ignored. There is just no property which matches it.
The point of patternProperties is to apply a subschema to any property with a name that matches the pattern. Any property which does not match the pattern is irrelevant, and is not constrained by the subschema. Unless there is another constraint elsewhere that is violated by that property, it will be considered valid - the spec mandates that any value is valid by default.

Great explanation, thank you. 👍


fix: typo modal enregistrement

:tada: This PR is included in version 1.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Authentication on v3 without tls

Hi,
I'm wondering how to activate auth on a v3 cluster. When i do etcdctl auth enable after creating a root user, the cluster stops and doesn't start anymore.
What the exact procedure I need to do to make it works.
Thank you

I tried to change the default image of docker for 3.3.11 but same behavior.


perf: skip duplicate rules with hashing

📝 Description
continuing the work previously done in https://github.com/chakra-ui/panda/pull/1544
Closes #1883
CSS generation

hash each style objects into unique string to de-duplicate CSS rules and only generate them at most once (instead of previous approach of removing duplicates after all CSS has been generated using a postcss plugin)

Static CSS

config.staticCss refactoring + StaticCss engine
new RuleProcessor to unify the style rules creation

Misc

move core engines/context to @pandacss/core, remove some deadcode

Looks like all tests are passing. Is it possible to get this released?

@adaboese, I've got a couple of CI failures to fix.
Once we're all green, I'll merge it, and we'll cut a new release.

Looks like this was merged and released, but the Panda 0.24.1 still has issue described in https://github.com/chakra-ui/panda/discussions/1866

got it ! can you provide a minimal repro ? you can try running with PANDA_DEBUG=* pnpm panda debug to isolate the slow files

Created a new issue so it does not get lost https://github.com/chakra-ui/panda/issues/1966


Refactor NUnit v3 support

Changed the nunit version specific implementation into mixins
Added getCommandArgs to support gradle-opencover-plugin
Renamed parallel_forks to parallelForks to follow the same naming
Default working directory of NUnit v3 to build\nunit\

LGTM


Added files of Exercise 1

As of 6 March 2023

Week 1 Completed, with an additional program remaining.


Form checkboxes' label color is black(#000) as opposed to other input elements, for which it is grey(#444)

Describe the bug
Form checkboxes' label color is black(#000) as opposed to other input elements, for which it is grey(#444)
How to reproduce
Link to stackblitz demo:
https://stackblitz.com/edit/difference-between-checkbox-and-toggle?file=src/app/app.component.ts
Steps to reproduce the behavior:
See how the label of the checkbox looks darker than the label of the toggle.
Expected behavior
The checkbox's label uses the same font color as the other form input elements.
Versions
I reproduced this for v.2, but looking at the documentation examples the issue is valid for v.4 as well. I haven't checked v.3.
Device:
Tested on Windows and MAC in chrome browser.

I just noticed the same is true for radio buttons.


[WIP] Adjust default client timeouts

Codecov Report

Merging #130 into master will increase coverage by 0.31%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #130      +/-   ##
==========================================
+ Coverage   61.34%   61.65%   +0.31%
==========================================
  Files          31       29       -2
  Lines        1177     1038     -139
==========================================
- Hits          722      640      -82
+ Misses        455      398      -57

Impacted Files
Coverage Δ

hca/util/__init__.py
92.96% <100%> (+0.26%)
:arrow_up:

hca/upload/__init__.py
33.33% <0%> (-5.38%)
:arrow_down:

hca/dss/__init__.py
78.74% <0%> (-4.81%)
:arrow_down:

hca/upload/upload_area.py
33.33% <0%> (-2.04%)
:arrow_down:

hca/upload/cli/select_command.py
44% <0%> (-1.46%)
:arrow_down:

hca/upload/cli/__init__.py
94.73% <0%> (-0.51%)
:arrow_down:

hca/upload/cli/creds_command.py

hca/upload/credentials_manager.py

hca/upload/upload_config.py
34.09% <0%> (+0.75%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


Add kafka MQE schema

Codecov Report

Merging #1016 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1016   +/-   ##
=======================================
  Coverage   29.77%   29.77%
=======================================
  Files         174      174
  Lines        4634     4634
  Branches     1220     1220
=======================================
  Hits         1380     1380
  Misses       3254     3254

Continue to review full report at Codecov.

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

Coverage remained the same at 27.015% when pulling 5d932f54c90eb6a2bd960e0bf018a69784943df3 on alyiwang:master into 523b203dd62cb8b781446f940a2325182b1e6009 on linkedin:master.


8 bits only for X and Y in ILI9341_Draw_Char

The coordinate parameters X and Y in the function ILI9341_Draw_Char are only 8 bits but in horizontal mode the screen width is 320. The resut is the x position wraps at 255 and you cant draw text on the far RHS of the screen.
Fixed in my local copy and now working 100%.
Thanks for the library.

Had this issue. Lost 2 days on debugging.


Update Landing Cards

Todos:

[x] Update descriptions
[x] Fix old descriptions
[x] Add new Cards
[x] Minify Cards (2 lines max, otherwise create two cards?)

Landing cards and screenshots updated. For more information, see #41.


[SPARK-35323][BUILD] Remove unused libraries from LICENSE-binary

What changes were proposed in this pull request?
This PR removes unused libraries from LICENSE-binary file.
Why are the changes needed?
SPARK-33212 removes many Hadoop 3-only transitive libraries like dnsjava-2.1.7.jar. We can simplify Apache Spark LICENSE file by removing them.
Does this PR introduce any user-facing change?
Yes, but this is only LICENSE file change.
How was this patch tested?
Manual.

Test build #138182 has started for PR 32445 at commit 23f9941.

Also, cc @sunchao .

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42703/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42703/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/42703/

Thank you so much, @sunchao !

Test build #138182 has finished for PR 32445 at commit 23f9941.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/138182/

Thank you, @sunchao and @maropu .
Merged to master for Apache Spark 3.2.0.


Improve the docs around the TextSelectionHandleControls deprecations

Fixes https://github.com/flutter/flutter/issues/122421
In that issue, someone was confused that using both contextMenuBuilder and TextSelectionControls.buildHandle ignored contextMenuBuilder.  This is intentional due to deprecations.  The correct  way is to use TextSelectionHandleControls.buildHandle.
This PR clarifies this in the docs.
CC @cottonman132

@Piinks @gnprice I have moved to a two-step deprecation process as discussed above, using PRs https://github.com/flutter/flutter/pull/124262 and https://github.com/flutter/flutter/pull/124611.  I've updated the docs in this PR to reflect that.


Allow to specify a limit for  text length, truncate if above

Hi,
It would be nice to have a way to specify a limit for the text length in the title tag. If the title length is greater than the limit, it would truncate the part before the titleTemplate string.
Example
Let's say you have:
title: 'Your page example is brought to you by the example company',
titleTemplate: '%s | Example Website'

If you specify the limit to 50, which this text exceeds, the computed title would be:

Your page example is brought ... | Example Website

which is 50 characters long.
It would definitely be a nice-to-have feature for handling title lengths with Google limitations.
Cheers,
Jerome

Now possible with a function titleTemplate (v1.2.0), although it does look a little dirty:
const suffix = ' | Example Website';
const suffixLength = suffix.length;
const maxTitleLength = 50;
const maxChunkLength = maxTitleLength - suffixLength;

metaInfo: {
    titleTemplate: (chunk) => {
        const truncatedChunk = chunk.length > maxChunkLength ? `${chunk.substr(0, maxChunkLength - 3)}...` : chunk;
        return `${truncatedChunk}${suffix}`;
    },
},

Resulting title:

Your page example is brought ... | Example Website

Nice thanks!


it will be much more helpful if you could add nextcloud, redis and  onlyoffice

Will soon support Redis 7.

Already supports Redis

Will soon support nextcloud


p2.5-96x48-1515-24s-a1 display

I have a p2.5-96x48-1515-24s-a1 display with FM6124D chips . but i cannot get it to work.
I have
R1   | G1
B1   | GND
R2   | G2
B2   | GND
R3   | G3
B3   | GND
A     | B
C     | N
CLK | STB
OE   | GND
I have no idea how to define the R3 G3 And B3 ports an where the N is used for.
Can you help me with this issue.
Kind regards
Johan

You will need to use another library.
Looks like a panel that scans three lines in parallel at the same time which this library doesn't support.


test/control-plane: Add nil checks to shutdown logic

The agentHandle.tearDown function always assumes it is called after its hive and d fields are set. The call to this shutdown logic is deferred. If we encounter an error before setting these fields, the deferred logic would cause an nil pointer deference panic which masks the original error. See #22224.
This PR adds nil checks so tearDown can always be called.

Marking as ready to merge, Travis hits flake #21730. This change covers the control plane tests which are only ran in Travis so no need for jenkins tests on this one.


TFLu: Port TFL detection postprocess operator

@njeffrie Right, it is done.

@njeffrie I realized that the output dimension handling needed an update.

@mansnils can you please check below errors as well
`tensorflow/lite/micro/kernels/detection_postprocess.cc:18:10: error: module ///tensorflow/lite/micro/kernels:micro_ops does not depend on a module exporting '/flatbuffers/include/flatbuffers/flexbuffers.h'
build_cleaner ///tensorflow/lite/micro/kernels:micro_ops
#include "/flatbuffers/include/flatbuffers/flexbuffers.h"
^/tensorflow/lite/micro/kernels/detection_postprocess.cc:859:66: error: missing field 'profiling_string' initializer [-Werror,-Wmissing-field-initializers]
detection_postprocess::Prepare, detection_postprocess::Eval};`

@rthadur build errors and review comments resolved

@mansnils Can you please check @rthadur's comments and keep us posted ? Thanks!

@mansnils, Any update on this PR? Please. Thanks!

@rthadur Comment and conflicts resolved.

tensorflow/lite/micro/kernels/detection_postprocess.cc:288:20: error: unknown type name 'uint8'; did you mean 'uint'?
  float operator()(uint8 x)

third_party/tensorflow/lite/micro/kernels/detection_postprocess.cc:301:9: error: unknown type name 'uint8'; did you mean 'uint'

third_party/tensorflow/lite/micro/kernels/detection_postprocess.cc:302:21: error: unknown type name 'uint8'; did you mean 'uint'

third_party/tensorflow/lite/micro/kernels/detection_postprocess.cc:746:9: error: unknown type name 'uint8'; did you mean 'uint'

third_party/tensorflow/lite/micro/kernels/detection_postprocess.cc:746:45: error: unknown type name 'uint8'; did you mean 'uint'

@mansnils can you please check this errors ?

@rthadur errors resolved.

@mansnils can you please check below errors
/tensorflow/lite/micro/kernels/detection_postprocess_test.cc:19:10: error: module ///tensorflow/lite/micro/kernels:detection_postprocess_test_binary does not depend on a module exporting '/tensorflow/lite/micro/all_ops_resolver.h'
build_cleaner ///tensorflow/lite/micro/kernels:detection_postprocess_test_binary
#include "third_party/tensorflow/lite/micro/all_ops_resolver.h"
         ^
/tensorflow/lite/micro/kernels/detection_postprocess_test.cc:125:17: error: unused variable 'input_scale2'
    const float input_scale2 = ScaleFromMinMax<uint8_t>(input_min2, input_max2);
                ^
/tensorflow/lite/micro/kernels/detection_postprocess_test.cc:263:13: error: unused variable 'kInputElements2'
  const int kInputElements2 = tflite::testing::kInputShape2[1] *
            ^
/tensorflow/lite/micro/kernels/detection_postprocess_test.cc:266:13: error: unused variable 'kInputElements3'
  const int kInputElements3 =
            ^
/tensorflow/lite/micro/kernels/detection_postprocess_test.cc:319:13: error: unused variable 'kInputElements2'
  const int kInputElements2 = tflite::testing::kInputShape2[1] *
            ^
/tensorflow/lite/micro/kernels/detection_postprocess_test.cc:322:13: error: unused variable 'kInputElements3'
  const int kInputElements3 =
            ^
/tensorflow/lite/micro/kernels/detection_postprocess_test.cc:470:13: error: unused variable 'kInputElements3'
  const int kInputElements3 =
            ^
/tensorflow/lite/micro/kernels/detection_postprocess_test.cc:467:13: error: unused variable 'kInputElements2'
  const int kInputElements2 = tflite::testing::kInputShape2[1] *

@rthadur errors resolved

Apologies for the repeated back-and-forth Måns - we have an internal version of this CL now and there are a few more things we will need before we can land this:

We recently switched to TfLiteEvalTensor instead of TfLiteTensor during invoke stage.  Init and prepare can still use the TfLiteTensor, but they are de-allocated at invoke for significant memory savings. See https://github.com/tensorflow/tensorflow/commit/88461053262f02bbc15887daa172c02db7419780 for how we ported the cmsis-nn kernels over to this new kernel API.

It seems like there are some unused variables which are causing compiler errors. Can you try to compile with -Werror=unused-variable?

I left a few inline comments on the change as well.
Thanks and sorry again for the repeated back-and-forth.

@njeffrie Thanks for the comments. I added -Wunused-variable and -Werror but could not find any more unused variables. I also manually added an unused variable to provoke an error to see that the -W options was getting through.

@njeffrie Updated after comments.

@rthadur gentle ping for merge

@mansnils thanks for your patience , still in progress merging internally , will keep you updated ..!

@rthadur still merging internally?

yes, @njeffrie has been working on merging this internally , @njeffrie gentle ping for update ?


[STORM-2465] modify storm-redis's READEME.md and update storm-redis.md

https://issues.apache.org/jira/browse/STORM-2465
There is a lack of "$" in the module of storm-redis's READEME.md which named "How do I use it".And storm-redis.md is out of date,for example,there is a new class named "RedisFilterBolt" which didn't be displayed in storm-redis.md. So,storm-redis.md should be updated to be as same as storm-redis's README.md.

+1

Hi, @vesense  how are you?
I am sorry to bother you again.Would you help me to merge it?Thanks for your help these days.

Hello, @vesense
Could you help me to merge this PR?


Fix a few offsets in UI3DModule

Fix offset of NamePlateObjectInfoPointerArray and NamePlateObjectInfoCounter
Thanks to Banana Bread for finding them!

Those offsets changed because the object array was expanded again (to 599) in order to support 8 person cards.
So don't merge this until we can update the rest of the offsets too.
I'll try and do it tomorrow if no one else does.

Ahhhh, that makes sense! Well, I'm not able to research that at all, so I'll leave that open for now. Just close it if not needed anymore.

Might this be the same as for RaptureAtkModule and RaptureAtkModule.NamePlateInfo?
As this also seems to be broken somehow.

Yes that offset nearly guaranteed to have shifted too. I'm going to bed right now so I'll fix it tomorrow if no one else has by then.

Alright, thanks for the info! Have a good night and sleep well!


[Chat]Add ChatTheme for ImageGallery specific colors

What

ImageGallery requires some specific colors.
We add them into the new ChatTheme so Contosos can control them through themes.
Why

How Tested

Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

I noticed you were still using the inline-image conditional compile. I think we should use the image gallery one instead.


Re-work use of created_at, updated_at

Needs to go in statements table or get dropped.

Refs. https://github.com/alephdata/followthemoney/pull/527


possible crashing after getting a "DNS error: Temporary DNS error while resolving: www.toggl.com"

More info in case 126730

cannot reproduce at the moment


When I click on the cart icon, I should see summary of the cart

When a user clicks on the cart icon, and has some items in the cart, he should see up to 3 items in the cart, the total price, and a link to the /cart page.
When the cart is empty, it should just say that.

Hi, can you please assign me this issue?

@Badrri-Narayanan done!

@Badrri-Narayanan do you need any help with the task?

@bkielbasa , actually yes. The initial setup with docker-compose is not working. I tried using docker-compose command and it initially failed as there was no Dockerfile for frontend (I am using master branch). So, I added it and tried again, but when I started the react frontend, it gave me lots of typescript errors. So, I kept the other containers running and did a local setup of the frontend alone. But, I see no values in the homepage. So, I checked the network tab and I can see that the backend is not sending any records.

On further investigation, I found that the inital seeding/creation of sample data is not working.

Due to this, I am facing trouble recreating the issue. Can you please help?

Also, Kibana is also failing to start as the initial setup sets and username and password.

@Badrri-Narayanan sorry you had trouble with it.
I updated the master branch and added seeds service to docker-compose. You can run it with the following command:
docker-compose up seeds

It will populate the database with a few products. If you need more complex products please let me know. The project is in very early stage so we don't have any kind of back-office where we can manage the content.
I added the missing dockerfile to the front-end so if you run
docker-compose build
docker-compose up

in the project root folder, you should have everything up and running. You should see something like this

Please let me know if you have any problems with running everything. I'll be glad to help you out with it.


Fix sway/language to handle xkb_layout command

This change fixes a few issues:

Fix to handle users changing layouts via "absolute" commands like:
swaymsg input type:keyboard xkb_layout es
which previously evaluated {long}, {short}, {variant} as empty.
Only "relative" commands like:
swaymsg input type:keyboard xkb_switch_layout next
were handled.
Previously, layout information was only stored internally if it was
one of the available layouts on the device with the most such layouts.
Even though memory was allocated (and leaked, see below) for every possible
configurable layout, it wasn't stored internally in a way we could
look up later. Now, all are stored internally for later lookup, and
leaks plugged.

Simplify how layouts are stored internally, and how events are handled.
I admit I didn't understand the modifications to short_name to append
an integer and the lookup by short name logic, but I have removed it.
To simplify and reduce (but not eliminate) the incidence of the observed
layout flickering when a stream of events are coming in,
the following heuristic is applied:

initially, pick the "identifier" that has the most layouts, if any.
keep track of the event_count, i.e., how many times full_name
has changed for each identifier.
when handling events, if the new event_count is greater than that
of the previously chosen identifier, switch to the new identifier.

Fix the above memory leaks of all Layout instances, one for every
layout configured on the system.

Fix potential data race due to lack of lock in update().
It is called from a different thread than onCmd and onEvent.
Also take the lock in the constructor, to guarantee config occurs before IPC.

Handle errors in parsing the default xkb ruleset config.
The result is that we can't enumerate any layouts though.

Remove unused layout_ member in XKBContext.

A trivial demonstration is up at https://youtu.be/gDNsym6f7O4
Before this change, the xkb_layout "es" and "hu" changes would result in no layout information being displayed, instead of "Spanish" and "Hungarian" respectively.

My error to not check for existing sway/language PRs!

Re: #1192, my changes here match the existing format, wild or not, as the case may be. I can adapt this PR to rebase on it and fix new lines' formatting, no problem.
Re: #984, I actually have a branch containing such a change (not yet published), with the following differences from that PR:

added documentation (example config and manpage/scdoc)
format-icons mapping is keyed on {long} not the first two characters of {long}
mapping of icons is built in constructor -- in general, I see a lot of operator[] calls on mutable Json::Value instances, and that call can allocate, so I prefer to use const auto for such values instead of just auto, and create a simple map<string, string> in the constructor and just use find on it.


--dart-define doesn't override values passed with --dart-define-from-file

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Create an empty project and cd into it
flutter create example --platforms android --empty

Create a defines.json file in the project root:
echo '{"FIRST_KEY": "FIRST_VALUE"}' > defines.json

Modify lib/main.dart to display the dart defines provided to the program:
import 'package:flutter/material.dart';

void main() {
  runApp(const MainApp());
}

class MainApp extends StatelessWidget {
  const MainApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: Scaffold(
        body: Center(
          child: Column(
            mainAxisAlignment: MainAxisAlignment.center,
            children: [
              Text('Hello World!'),
              Text('FIRST_KEY=${const String.fromEnvironment('FIRST_KEY')}'),
              Text('SECOND_KEY=${const String.fromEnvironment('SECOND_KEY')}'),
            ],
          ),
        ),
      ),
    );
  }
}

Run the program, passing both --dart-define-from-file and --dart-define options.

Use case: I have many dart defines in my JSON config file, but I want to override one of these values, e.g. for debugging purposes.

flutter run \
  --dart-define-from-file defines.json \
  --dart-define 'FIRST_KEY=you just got overridden' \
  --dart-define 'SECOND_KEY=SECOND_VALUE'

Expected results

Actual results

Code sample
N/A / provided above
Screenshots or Video
N/A
Logs
N/A
Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.10.5, on macOS 13.4.1 22F82 darwin-arm64, locale en-PL)
    • Flutter version 3.10.5 on channel stable at /Users/bartek/fvm/versions/3.10.5
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (4 weeks ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/bartek/androidsdk
    • Platform android-33, build-tools 33.0.1
    • ANDROID_HOME = /Users/bartek/androidsdk
    • Java binary at: /Users/bartek/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.10121639/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.12.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Users/bartek/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/222.4459.24.2221.10121639/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1.4)
    • IntelliJ at /Users/bartek/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 74.0.4
    • Dart plugin version 231.9161.14

[✓] IntelliJ IDEA Ultimate Edition (version 2023.1.4)
    • IntelliJ at /Users/bartek/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/231.9225.16/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.80.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.68.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 13.4.1 22F82 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 114.0.5735.198

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Thanks for the report. Seeing the same behavior as reported.

 stable, master flutter doctor -v
[!] Flutter (Channel stable, 3.10.5, on macOS 12.2.1 21D62 darwin-x64, locale
    en-GB)
    • Flutter version 3.10.5 on channel stable at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 796c8ef792 (2 days ago), 2023-06-13 15:51:02 -0700
    • Engine revision 45f6e00911
    • Dart version 3.0.5
    • DevTools version 2.23.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.13.0-3.0.pre.91, on macOS 12.2.1 21D62
    darwin-x64, locale en-GB)
    • Flutter version 3.13.0-3.0.pre.91 on channel master at
      /Users/dhs/documents/fluttersdk/flutter
    ! Warning: `flutter` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/flutter, which is not inside
      your current Flutter SDK checkout at
      /Users/dhs/documents/fluttersdk/flutter. Consider adding
      /Users/dhs/documents/fluttersdk/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to
      /Users/dhs/Documents/Fluttersdk/flutter/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/dhs/documents/fluttersdk/flutter.
      Consider adding /Users/dhs/documents/fluttersdk/flutter/bin to the front
      of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5a866a031f (4 hours ago), 2023-07-16 17:36:30 -0700
    • Engine revision 683087731f
    • Dart version 3.1.0 (build 3.1.0-315.0.dev)
    • DevTools version 2.25.0
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/dhs/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13C100
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 65.1.4
    • Dart plugin version 213.7228

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (3 available)
    • Darshan's iphone (mobile) • 21150b119064aecc249dfcfe05e259197461ce23 • ios
      • iOS 15.3.1 19D52
    • macOS (desktop)           • macos                                    •
      darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)              • chrome                                   •
      web-javascript • Google Chrome 109.0.5414.119

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! Flutter recommends a minimum Xcode version of 13.
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] VS Code (version 1.62.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.21.0

[✓] Connected device (5 available)
    • SM G975F (mobile)       • RZ8M802WY0X • android-arm64   • Android 11 (API 30)
    • Darshan's iphone (mobile)  • 21150b119064aecc249dfcfe05e259197461ce23 •
      ios            • iOS 14.4.1 18D61
    • iPhone 12 Pro Max (mobile) • A5473606-0213-4FD8-BA16-553433949729     •
      ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • macOS (desktop)            • macos                                    •
      darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Chrome (web)               • chrome                                   •
      web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.


path problem, wrong images displayed

Details
Two of our guideline artlicles are displayed on readthedocs with wrong screenshots. On github everything is ok and the path is correct but when displayed on readthedocs the screenshots seem to be taken from wrong catalog and mixed. Please see the details below:

Read the Docs project URL: https://wiki.edi-n.com/ru/latest/services/EDIN_DOCflow/edin_docflow/instruktsia-biznes-administratora.html
https://wiki.edi-n.com/ru/latest/services/EDIN_DOCflow/edin_docflow/instruktsia-administratora-polzovatelei.html
Build URL (if applicable): https://readthedocs.org/projects/edi-n/builds/
Read the Docs username (if applicable): EDI-N

Expected Result
Article instruktsia-administratora-polzovatelei.rst should contain images from catalog pics_instruktsia-administratora-polzovatelei
Article instruktsia-biznes-administratora.rst should contain images from catalog pics_instruktsia-biznes-administratora

I can see the same images as the repo
https://github.com/EDI-N/Docs/blob/master/docs/source/services/EDIN_DOCflow/edin_docflow/instruktsia-administratora-polzovatelei.rst
https://github.com/EDI-N/Docs/blob/master/docs/source/services/EDIN_DOCflow/edin_docflow/instruktsia-biznes-administratora.rst

@stsewd thank you for dealing with the issue. I don't know how it happened but the screenshots are ok now.

Probably something with the browser cache. Closing.


Fix parameter handling for sentinel

Parameter handling for redis sentinel did not work. Overwriting sentinel options did not work due to incorrect use of array()+array() and the case of a ParametersInterface being passed was not handled at all.

Thanks for spotting, there was indeed a bug as Predis didn't override the values of password and database from user-supplied parameters, but your fix is still wrong because now it doesn't let users override the default timeout for sentinels (set to 100ms) by specifying a value for timeout. I pushed a commit with a different fix and some tests to avoid regressions.
As for not handling instances of Predis\Connection\ParametersInterface this is the intended behaviour here, we go on the assumption that when users are passing them to Predis\Client::__construct() they know what they are doing just like when passing fully-initialized connection instances.

@nrk What's about the ParametersInterface stuff from my commit. When using SncRedisBundle $parameters is no array, but an instance of ParametersInterface. Then the unset / timeout stuff is not done, which I think is an error.

I don't know how SncRedisBundle works internally but it shouldn't feed Predis\Client::__construct() directly with an instance (or an array of instances) of Predis\Connection\ParametersInterface, there's simply no need to do that unless for very specific and obscure reasons that I honestly can't think of. If it does then I think it should be updated to strip password and database when the replication client option is set to sentinel, and provide a default timeout for the sentinels.
Adding support for redis-sentinel in a minor release without causing any breaking change while still providing a decent way to configure the client proved to be quite a challenge for how the internals of Predis are designed right now, some things will be changed in v2.0 to improve upon that but right now I'm not so keen to change the current established convention through the whole library that any instance of Predis\Connection\ParametersInterface or Predis\Connection\ConnectionInterface passed to the client's constructor is left untouched.

Ok, thank you for the explanation (and this great library)! If I have time, I'll check what exactly SncRedisBundle is doing and why it is doing it, and will try to improve. I can live with using my own branch of Predis with the workaround for now.


New Extension: acrquery- Supporting the Azure Container Registry Metadata Search feature

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[x] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.


Log cat highlight

Just buf comment:

https://github.com/user-attachments/assets/0a0f4b01-9644-42ff-a1ea-e1408c327f3b


npm run dev不应该每次都要run bootstrap；npm run test error

Please makes sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's Contributing Guide.
[x] Make sure you are merging your commits to dev branch.
[x] Rebase before creating a PR to keep commit history clear.
[x] Add some descriptions and refer relative issues for you PR.

npm run test error for windows

团队的开发习惯就是 dev 之前自动运行一次 install

@QingWei-Li +1

npm run test error这个呢

@earlymeme 嗯 那个要修，麻烦保留 bootstrap 谢谢 : )

诶 为啥这么多内容。。。重新开一个 pr 吧 😂


Why does multithreading example use per-thread vertex and index buffers?

I noticed this when I saw validation errors on the example.  The mesh loader does not apply give the buffer memory the TransferSrc flag, so the copy command in setting up the per-thread buffers generates the errors.  However, I'm not clear on why each thread needs it's own copy of the static mesh data.  Replacing the copy code in prepareMultiThreadedRenderer with thread->mesh = meshes.ufo seems to work just fine and produce no validation errors.

This dates back to when the buffers weren't device local and may not be necessary anymore. I'll take a look at it this weekend.
As for the validation error from the missing usage flags, see #164.

ok, I was just trying to understand if I was breaking something. thanks.

For the record, I adapted this example. I use a single vertex and index buffer for all threads and on each thread I do:
vkBeginCommandBuffer(...);
vkCmdSetViewport(...);
vkCmdSetScissor(...);
vkCmdBindPipeline(...);
vkCmdBindVertexBuffers(...);
vkCmdBindIndexBuffer(...);
for_each (object) {
vkCmdPushConstants(...);
vkCmdDrawIndexed(...);
}
vkEndCommandBuffer(...);

Yes, I tested that too some weeks ago and it works fine. Haven't found the time to update this example yet though.


Can't bootstrap gke clusters (invalid name)

CONSOLE_CLUSTER=gke_project-id-123456_europe-west1-d_excitingdev
WORKER_CLUSTER=gke_project-id-123456_europe-west1-d_throwawaydev
kubemcsa bootstrap --target-context $CONSOLE_CLUSTER --source-context

created namespace "multicluster-service-account" in source cluster "gke_project-id-123456_europe-west1-d_throwawaydev"
created cluster role "service-account-import-controller-remote" in source cluster "gke_project-id-123456_europe-west1-d_throwawaydev"
kubemcsa: error: cannot bootstrap: ServiceAccount "gke_project-id-123456_europe-west1-d_excitingdev" is invalid: metadata.name: Invalid value: "gke_project-id-123456_europe-west1-d_excitingdev": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

By default, gke clusters include _ in the name. This makes them invalid as ObjectMeta.Names, so a ServiceAccount or other object can't just use the cluster name.

Thanks for filing this issue. This is new in v0.5.0, and I would have noticed if I had not also switched from GKE to k3d for my tests...
Here's my plan: if the cluster's name in the kubeconfig passes the regex check, it will be used as is. If not, kubemcsa bootstrap will fail and ask the user to supply another argument --target-name/--source-name, to be used as cluster name in the multicluster-service-account domain. Even if the kubeconfig cluster's name passes the regex check, those arguments will still be useful if the user wants a way to override.

that sounds like a much better solution than the road I was going down. I'm unclear on how this works under the hood. If server is the same, does the clusterName matter when connecting? i.e. can I work around this for now by renaming the cluster and context when I bootstrap?

You can safely rename clusters and contexts in a kubeconfig file. Make sure you refer to the new cluster name in the context. That would work around this issue. What matters when connecting is the server, certificates, etc.

The --target-name and --source-name options have been implemented in v0.5.1.

thanks, works great!


Run multiple instances from the same directory.

From my understanding to be able to have transfer from TheCenter to Scorched Earth you need to have two servers running from the same directory and the cluster flags set. Currently the instances have to be run from separate ark directories. Is there any way to run two instances from the same directory?

You could copy the configurations, and edit the .cfg files to use the same directory (and have them run on different ports)

Actually, see this: https://github.com/FezVrasta/ark-server-tools/issues/578 you can't run two instances from the same directory at the moment.

Closing the issue. Thankyou!


Confusion about key.length === value.length

So I have been writing a GraphQL API and loving it!
We have run into an N+1 issue, and after watching Lee's video,
we have decided to try out Dataloader.
Here there is an error about the key length and value length matching.
Say for instance we have an ArtistType. An ArtistType has a GraphQLList of ArticleType.
// ArtistType
export default new GraphQLObjectType({
	name: 'Artist',
	fields: () => ({
                ...
                articles: {
                       type: new GraphQLList(ArticleType),
                       resolve: (artist, args, ast, { rootValue }) =>  {
                               return ArtistArticlesLoader.load(artist.id)
                       }
                }
        })
})

My dataloader is something similar to this (note that I'm using Sequelize:
export const ArtistArticlesLoader = new DataLoader((ids) => new Promise((resolve, reject) => {
	sequelizeDB.query(
		`SELECT a2.*
                  FROM ARTIST a1
                  LEFT JOIN ARTICLES a2 ON a2.subjectId = a1.id
                  WHERE a1.id IN (:ids)`,
		{
                        type: Sequelize.QueryTypes.SELECT,
                        replacements: {
                        	ids: ids
                        },
                        raw: true
		}
	).then((results) => {
		let res = results.map(result => {
			return {
				...result
			}
		})
		resolve(res)
	)
}))

Let's say I throw a query like this:
query {
    artists {
        articles {
            title
        }
    }
}

Now since Artists' Articles are m:n, the keys and values lengths don't match and I'm getting that error.
Am I using this in the wrong way?
I've checked out examples like this, but don't know how the lengths match..

@Naoto-Ida yeah, you're not using it as intended. In your case, you're giving it a list of Artist IDs, and you're getting back a list of Article IDs for all of those artists mixed. You can either fetch articles by article ID, or do the batching/caching to fetch all articles for an artist per artist, in which case you would return not a list of articles, but a list of lists of articles, i.e. [[article1, article2, ...]]

@helfer Thanks for the clarification!
I also have other types that contain an articles field.
So now I've made these loaders:
export const ArtistArticleIdsLoader = new DataLoader((ids) => new Promise((resolve, reject) => {
    DOQUERYTOGETARTICLEIDS.then((results) => {
        let res = results.map(result => result.id)
        resolve(res)
    )
}))

export const EventArticleIdsLoader = new DataLoader((ids) => new Promise((resolve, reject) => {
    DOQUERYTOGETARTICLEIDS.then((results) => {
        let res = results.map(result => result.id)
        resolve(res)
    )
}))

export const ArticleLoader = new DataLoader((ids) => new Promise((resolve, reject) => {
    DOQUERYTOGETARTICLES.then((results) => {
        resolve(results)
    )
}))

Then in the ArtistType:
export default new GraphQLObjectType({
    name: 'Artist',
    fields: () => ({
                ...
                articles: {
                       type: new GraphQLList(ArticleType),
                       resolve: (artist, args, ast, { rootValue }) =>  {
                               return ArtistArticleIdsLoader.load(artist.id)
                              .then((articleIds) => {
                                      const promises = articleIds.map(id => ArticleLoader.load(id))
                                      return Promise.all(promises).then(results => results})
                              })
                       }
                }
        })
})

The first two loaders bunch up the ids of articles associated to a certain Artist or Event,
the last loader bunches up the results for those articles.


Add Connection Details for Opensearch

Description of your changes
Adds connection details for Opensearch:

Endpoint
Dashboard Endpoint
Kibana Endpoint

I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[x] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested
Doing a make uptest

@turkenf this is a continuation of this PR: https://github.com/crossplane-contrib/provider-upjet-aws/pull/1408

I have tested this using make uptest, but still found the connection object empty once the Opensearch cluster was up. Would this be because of late initialization issues? I am seeing the following error in my k8s cluster:
managed/opensearch.aws.upbound.io/v1beta1, kind=domain  failed to observe the resource: [{0 reading OpenSearch Domain (arn:aws:es:us-west-1:953144367104:domain/op-q15u7b3u): ThrottlingException: Rate exceeded

Are these documents relevant in helping solve this issue? https://github.com/crossplane/upjet/blob/main/docs/configuring-a-resource.md#late-initialization-configuration


Support Hiyobi

Download Images with Heliotrope Api, Get Information of Donginji,Manga and save it to File

The heliotrope api is a personal api.
I will make a request through Hiyobi api.

its yours lol

I have no intention to release my api.

Or, I would like to have a function to choose to use the user's Heliotrope API or HiyobiAPI.

Write a custom extractor: https://github.com/Saebasol/rabbit-ark/wiki/Script

Hiyobi was closed


Fix some spelling mistakes and a mypy error

The fallback code for not finding PIL re-assigns Image, which mypy does not like. Add an type: ignore for this.
Fixed some spelling mistakes in TODO.txt, README.rst and CHANGELOG.rst

FYI, I'm planning to get rid of travis integration.  Thanks for the cleanups.


New border type tool, resize bug-fix & functionality used to plug external comment popups

New border type tool: solid line, dashes, dots and dashes
Functionality that disables Mirador comment popup and tooltips and notifies external applications
Bug-fix: change implementation of resize function because latest merges assosiated with image coordinates doesn't work fine. It can be reproduced with the only image from http://oculus-dev.harvardx.harvard.edu/manifests/drs:48309543

@stanislavkostadinov  Sorry, I was renaming refactor to release2.1 and didn't realize that would close this PR.  Can you please redo?


Support updating custom claims in updateUser()

Closes #1876
Summary
Currently, the custome user claims can be updated with setCustomUserClaims(), and the user details can be updated with updateUser(). Both use same /accounts:update REST API under the hood.
This allows account and claims info to be updated in one hit using the updateUser() method.

@lahirumaramba any thoughts on this?

@lahirumaramba @prameshj I completed the test suite adding integration tests and unit tests. All relevant tests that were applied to setCustomUserClaims() are applied to the relevant claims functionality in updateUser(). Would appreciate a moment to review 🙏🏼 .


Update MS.AI reference

Third time is the charm as they say...

@karolz-ms,
Thanks for your contribution.  To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
The agreement will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot


Ability to remove background behind detected object

I consider object segmentation and background removal are out of scope for smartcrop.js.


User can not send photo if application is turkish

Hi,
I set the language to Turkish. I try to take and send a photo but it fails.
Steps to produce :
1 - Set the language as Turkish.
2 - Go to a room
3 - Select the option Video Kaydet veya fotoğraf çek
4 - take a photo
5 - on the preview screen press Tamam which means OK
6 - you will get an error instead of showing the Send As dialog.

If application is english or french, there wont be problem. I checked the image path and I can see the taken image in the path.

Sorry to hear that, thanks for reporting.

Can someone which file should I debug? I want to debug and fix the issue. This feature is very important for us.

Hi,
Let I'm having a quick look at this issue

Ok, I figured it out.
Taken photo is store in file "VectorImage_.jpg"
And the I becomes 'ı' (i without a point, specific to Turkish language) when toLowerCase() is called, which make the regexp fails to get the file extension...

Hi,
Thank you for your effort. Could you give me the file name and line so I can take a look at it and try to fix.

You've already fixed it. Thank you very much. I tested it and it works like a charm :+1:

Thanks for the test.
The fix will be available in the next release.
Benoit