fix(modal): fix copying from $scope

Fixes issue where $scope provided does not have properties present
on controller instance due to $new resulting in the property on the
prototype of the $scope copied from, which causes it to not be
enumerable

This fixes #5039

LGTM but I would change the titlte to "fix bindToController"


reset_cache() Decrease the Performance for Generation of Consecutive Inputs

When conducting generation for multiple consecutive inputs on a LoRA fine-tuned LLaMA, I noticed that using 'reset_cache' after each generation for one input will affect the performance of generation on the next input. However, if you load the model again after each generation, the performance stays good. But reloading consumes lots of time. Could you help provide some explanation why 'reset_cache' will decrease the performance of the generation on the next consecutive inputs?
Code: I modified the code of 'generate/lora.py' to enable consecutive generation on multiple inputs. Basically, just add a for loop and model.reset_cache()
  # support multiple inference
  outputs = []
  num_samples = len(input)
  for i in range(num_samples):
      sample = {"instruction": prompt[i], "input": input[i]}
      prompt = generate_prompt(sample)
      encoded = tokenizer.encode(prompt, bos=True, eos=False, device=model.device)

      t0 = time.perf_counter()
      output = generate(
          model,
          idx=encoded,
          max_new_tokens=max_new_tokens,
          temperature=temperature,
          top_k=top_k,
          eos_id=tokenizer.eos_id
      )
      t = time.perf_counter() - t0

      model.reset_cache()
      output = tokenizer.decode(output)
      output = output.split("### Response:")[1].strip()
      print(output)
      print(f"Time for inference: {t:.02f} sec total, {max_new_tokens / t:.02f} tokens/sec", file=sys.stderr)
      outputs.append(output)

model.reset_cache() every sample generation should be faster than recreating the model plus loading its state dict. If you are not seeing this, I would suggest that you profile the code to find out where the slowdown is coming from

Thanks for the quick reply. Yes, using model.reset_cache() for every sample generation avoids loading model every time and  is indeed much faster. However, what I noticed is that the quality of generated texts is only good for the first input but very bad for the second, third, ..., inputs. But if I reload the model after each generation, the quality of generated contents is great for each input.
Besides, my prompts has to be changed with different inputs, so each input has a different prompt. Is this changing prompt causing the generation quality issue?

That's strange. Can you share some steps for us to reproduce the issue?


build: fix testkit published version

Testkit didn't have its version updated on publish (and published 0.0.0 instead). Can publish a 1.0.0-M9 manually.

Published an 1.0.0-M9 for the testkit manually. Don't seem to be able to unpublish 0.0.0 — probably need the bot credentials to do this.


Code cleanup

Includes definitions to reuse code, removes unused definitions and redundant declarations.

/hold for #660

/unhold


ueberzugpp: 2.9.1 -> 2.9.2

Description of changes
https://github.com/jstkdng/ueberzugpp/compare/v2.9.1...v2.9.2
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 256194 run on aarch64-darwin 1

2 packages built:

ueberzugpp
ytfzf


Support for Android 10

Hi Team,
You have done great job of polygonal cropping.
I have implemented this library in my projects since 2017.
Now our client want app which will support Android 10 as well.
In the latest library update build.gradle file, it is upto Android 9 only .
So can you please confirm that will this same library support Android 10 or will there be any update?

It works, but only by adding  <uses-feature android:name="android.hardware.camera"/ to android manifest for newest versions of android

Library is upgraded to Android X with latest PR merge.


[BEAM-7205] Remove MongoDb withKeepAlive configuration

R: @aromanenko-dev

Run Java MongoDBIO Performance Test


Unexpected Composition Error due to Interface Being Interpreted as a Type

Hello
While integrating a new subgraph into our project, we encountered composition errors due to the subgraph schema's incompatibility with our two previously published subgraphs. This was an expected issue. However, an unexpected error arose, causing the only output received from schema:check to be "Unexpected composition error." All the specific errors due to supergraph composition were hidden.
When we ran the project with hive dev, we got a more descriptive error message: "Error: Expected interface, got object." The issue seems to occur when composing schemas where an entity is used as a Type in subgraph A and an Interface in subgraph B. If another type implements this interface, it is then presumed to be a type instead of an interface.
Here's a minimal example to reproduce the error:
cars.graphql
type Query {
  foo(id: ID!): String
}

interface Vehicle {
    id: ID!
}

type Car implements Vehicle {
  id: ID!
}

vehicles.graphql
type Vehicle {
    id: ID!
}

Command used:
hive dev \
  --service cars --url http://localhost:8080/graphql --schema cars.graphql \
  --service vehicles --url https://localhost:9090/graphql --schema vehicles.graphql

This results in the output: "Error: Expected interface, got object."
For reference, Apollo Rover outputs:
Caused by:
TYPE_KIND_MISMATCH: Type "Vehicle" has mismatched kind: it is defined as Interface Type in subgraph "cars" but Object Type in subgraph "vehicles"
INVALID_FIELD_SHARING: Non-shareable field "Car.id" is resolved from multiple subgraphs: it is resolved from subgraphs "cars" and "vehicles" (through @interfaceObject field "Vehicle.id") and defined as non-shareable in subgraph "cars"

I guess this is a bug?

Fixed on its way!

https://github.com/the-guild-org/federation/pull/74

@jenskm it should be in production in about 20 minutes


Add doc for manually editing resources in Cosmos for troubleshooting

Resolves #2901
What is being addressed
Adds doc in the troubleshooting section covering how to locate a resource Id for a TRE resource, then find the corresponding document in Cosmos and edit it, for troubleshooting purposes.

/test-force-approve


Avoid black thumbnails with fade-in videos

Hi @jdn06 ,
this could be useful indeed, but Travis is failing and after a quick look it is because the test video's duration is ~2 seconds and you use 2 seconds for the -ss parameter ...
I'm not sure what's the best option here:

extract duration from ffmpeg's output and check if it is > 2s
add a setting to customize the ss vaue.

I chose to add a parameter with a default value of 0. As everybody doesn't have the same kind of fade-in, it seems more useful than a check of the length of the video.

Thanks, but the tests still fails, could you check why ?

Great, thanks !


Rotation scans get an extra _1 on the end of the filename

It looks like Hyperion is adding the run_number on to the rotation scan twice i.e. see data in /dls/i03/data/2024/cm37235-2/TestInsulin/ins_9 where we get ins_9_8_1.nxs. Here the 8 is the runnumber appended by GDA and passed into Hyperion and the _1 is Hyperion adding another run number on. This is probably best fixed by just not including the run number when GDA gives the filename to Hyperion
Acceptance Criteria

Rotation scans do not have the additional _1

I believe this was being caused by the GDA line which gets the prefix for hyperion's rotation parameters:
prefix = extended_request.getDnaFilePrefix()[:-1]
In Hyperion's debug logs, the prefix from this line included the underscore + runnumber from the previous hyperion FGS. Then, the Hyperion line
@property
 def full_filename(self):
     return f"{self.prefix}_{self.run_number}"

Would add on another underscore run-number.

See solution at https://gerrit.diamond.ac.uk/c/gda/gda-mx/+/41720/1. To test, cherry pick this change onto i03-live and test on a beamline (doesn't require beam)

Deployed GDA change on i03_live and it appeared to fix issue


feat: update to node 16

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


Adding ipywidgets without an extra div

Hi!
When I use ipywidgets inside of ipyvuetify widget, they are wrapped in a

. This doesn't allow customizing layout with flexbox, because I can't change the class or style of this wrapper. Is it possible to display ipywidgets without an extra div? It seems that when I put ipywidgets widget inside another ipywidgets widget (VBox), there is no extra div.
Example:

import ipywidgets as w
import ipyvuetify as v

button = w.Button(description="ipywidgets button")

label = v.Html(tag="span", children=["Example label"])

v_button = v.Btn(children=["Vue Button"])

input = w.BoundedFloatText(
    value=10,
    layout=w.Layout(width="auto", min_width="var(--jp-widgets-inline-width)", flex="1 0 100%"),
)

example = v.Html(
    tag="div",
    style_="display: flex; flex-flow: row nowrap; align-items: center;",
    children=[label, v_button, input],
)

# solara run example.py:example

I found the code that creates that div in ipyvue repository. Maybe this issue should be moved there.
https://github.com/widgetti/ipyvue/blob/d307dbdceaa05f01fe22be57d157d02c9ee8fbdf/js/src/VueRenderer.js#L48-L50

I tried to create a comment node in render function, mount a JupyterPhosphorWidget as a sibling to this node and then remove the comment node: https://github.com/widgetti/ipyvue/compare/master...egormkn:ipyvue:master
It now renders correctly, however I'm not sure if such kind of manual DOM manipulation is safe in terms of Vue reactivity.
Also, in my tests in Jupyterlab I noticed a bug that is caused by that height: 100%: the wrapper div takes 100% of the output height.

yeah, the extra div is indeed a pain point, but we have not found a way around it yet.
I've tried your branch and it works for getting an embedded widget in. After I removed refNode.remove(); I could also replace an embedded widget. But changing the order of widgets of the same parent doesn't work.
Notebook example:
import ipywidgets
import ipyvue as vue

children = [ipywidgets.Button(description="hi1"), ipywidgets.Button(description="hi2")]

main = vue.Html(tag="div", children=children)
main

Then change the order:
main.children = children[::-1]

Notice the order of the children doesn't change in the DOM.
A workaround for the extra div is to have a div with the right properties in ipyvue and add the embeded widget to that.


Remove specialist browse category

Remove the detailed guide for tax advisers and agents.
This PR is based off of the work done in this: https://github.com/alphagov/whitehall/pull/1698 to do something similar.
https://www.pivotaltracker.com/n/projects/780127/stories/87918708 (pivotal)
After this is merged the following redirect should also be added to router-data (I think?): browse/tax/self-assessment/guidance-for-tax-advisers-and-agents,/government/organisations/hm-revenue-customs

Closing until #1969 is merged


THRIFT-3768: ensure TThreadedServer guarantees the lifetime of the client

TThreadedServer now guarantees the lifetime of the TConnectedClient.

I did not touch "go" or "py" implementations and that was the reason for the failure in #2467.2.  #2467.1 failed in three tests including go-cpp/json however given other go tests are failing it looks like the issue is likely in "go".

I ran the test that failed in the apache build in a loop and finally ended up in a deadlock situation, so I am looking at that.

I'm starting to believe that the TThreadServerPool implementation may have the same issue.  The cores I have been able to produce through repeated testing are showing a pthread running when nothing else is, and it usually cores because it is trying to lock a mutex that has been destroyed.  I'm looking into this.

@jfarrell Could you look into why the Jenkins build for apache thinks that a c_glib file needs to be merged manually and/or has an open pull request on the apache build system?

I think this change looks good for the most part, maybe we can get one of the maintainers to look at it?

Looking for an advocate to consider merging this.  It passed all three builds with the exception of something that happened on the apache jenkins where a cppcheck file was not created:

[Cppcheck] Starting the cppcheck analysis.
[Cppcheck] No cppcheck test report file(s) were found with the pattern 'cppcheck-result.xml' relative to '/home/jenkins/jenkins-slave/workspace/Thrift-precommit'.  Did you enter a pattern relative to the correct directory?  Did you generate the XML report(s) for Cppcheck?
[Cppcheck] Parsing throws exceptions. No cppcheck test report file(s) were found with the pattern 'cppcheck-result.xml' relative to '/home/jenkins/jenkins-slave/workspace/Thrift-precommit'.  Did you enter a pattern relative to the correct directory?  Did you generate the XML report(s) for Cppcheck?
Build step 'Publish Cppcheck results' changed build result to FAILURE

Looking for an advocate who would consider merging this fix despite the busted Apache Jenkins environment result.

LGTM. Thanks for fixing this issue.


feat(module:date-picker): support open and close method

Close #3352
PR Checklist
Please check if your PR fulfills the following requirements:

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

PR Type
What kind of change does this PR introduce?

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

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

我们应该对这种下拉框在什么时机弹出做规范：

受控模式，只能通过 input 的变更或者调用实例方法来控制下拉框的显示，不触发 visible change 等类似事件
非受控模式，用户交互能够控制下拉框的展示，且触发 visible change 及类似事件

cc @hsuanxyz @wenqi73 @vthinkxie API 需要确定一下


fix: [IDP]: Updated the title and sidebar labels for CCP

Purpose
Updated the title and sidebar labels for CCP
Merge requirements

[ ] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[x] Release notes/new features docs: Feature/version released to at least one prod environment

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://660fc162047d2d7271502b06--harness-developer.netlify.app


Allow batteries set to not visible to be shown if directly added under entities.

Batteries set to not visible don't show in the card, even when added under entities.
I would like for them to show if directly added under entities, it would also be nice to have a include filter to show all?

This will be possible to achieve in the next version via dedicated setting: #698

Hate being "that guy*, but can we get a new version tagged? ;D
Like, I'd help, if you need something tested or fixed, but I guess there's not much more we can support in this.


move to github

BTW, in case you wanted to see what I meant by testing it out, the pull request validation job prints out the new version-specific metadata and all the files it installs here:
https://ci.ksp-ckan.org/job/NetKAN/7140/console
Looks good to me, merging...


1.cmap_up.0:0:DECONVOLUTION:GPU:kernel weights has count 2097152 but 4194304 was expected

I ran into the following problem when  converting  densenet121_baseline_att_256x256_B_epoch_160.pth and also resnet18_baseline_att_224x224_A_epoch_249.pth into a trt model, which appears to be due to a mismatch between the model weight and the model definition. Does anyone know how to fix it?
>>> model_trt = torch2trt.torch2trt(model, [data], fp16_mode=True, max_workspace_size=1<<25) [08/01/2024-17:37:36] [TRT] [E] Error Code: 3: 1.cmap_up.0:0:DECONVOLUTION:GPU:kernel weights has count 2097152 but 4194304 was expected [08/01/2024-17:37:36] [TRT] [E] ITensor::getDimensions: Error Code 4: API Usage Error (1.cmap_up.0:0:DECONVOLUTION:GPU: count of 2097152 weights in kernel, but kernel dimensions (4,4) with 512 input channels, 512 output channels and 1 groups were specified. Expected Weights count is 512 * 4*4 * 512 / 1 = 4194304) [08/01/2024-17:37:36] [TRT] [E] Error Code: 3: 1.paf_up.0:0:DECONVOLUTION:GPU:kernel weights has count 2097152 but 4194304 was expected [08/01/2024-17:37:36] [TRT] [E] ITensor::getDimensions: Error Code 4: API Usage Error (1.paf_up.0:0:DECONVOLUTION:GPU: count of 2097152 weights in kernel, but kernel dimensions (4,4) with 512 input channels, 512 output channels and 1 groups were specified. Expected Weights count is 512 * 4*4 * 512 / 1 = 4194304) Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/root/miniconda3/envs/trtpose/lib/python3.8/site-packages/torch2trt-0.5.0-py3.8-linux-x86_64.egg/torch2trt/torch2trt.py", line 643, in torch2trt     outputs = module(*inputs)   File "/root/miniconda3/envs/trtpose/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1538, in _call_impl     result = forward_call(*args, **kwargs)   File "/root/miniconda3/envs/trtpose/lib/python3.8/site-packages/torch/nn/modules/container.py", line 217, in forward     input = module(input)   File "/root/miniconda3/envs/trtpose/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1538, in _call_impl     result = forward_call(*args, **kwargs)   File "/root/miniconda3/envs/trtpose/lib/python3.8/site-packages/trt_pose-0.0.1-py3.8-linux-x86_64.egg/trt_pose/models/common.py", line 76, in forward     return self.cmap_conv(xc * ac), self.paf_conv(xp * ap)   File "/root/miniconda3/envs/trtpose/lib/python3.8/site-packages/torch2trt-0.5.0-py3.8-linux-x86_64.egg/torch2trt/torch2trt.py", line 262, in wrapper     converter["converter"](ctx)   File "/root/miniconda3/envs/trtpose/lib/python3.8/site-packages/torch2trt-0.5.0-py3.8-linux-x86_64.egg/torch2trt/converters/native_converters.py", line 1496, in convert_mul     input_a_trt, input_b_trt = broadcast_trt_tensors(ctx.network, [input_a_trt, input_b_trt], len(output.shape))   File "/root/miniconda3/envs/trtpose/lib/python3.8/site-packages/torch2trt-0.5.0-py3.8-linux-x86_64.egg/torch2trt/torch2trt.py", line 146, in broadcast_trt_tensors     if len(t.shape) < broadcast_ndim: ValueError: __len__() should return >= 0

I had met the same issue on jestson nano，has you fixed it?

Hi. I meet the same problem when converting resnet18_baseline_att_224x224_A_epoch_249.pth to a trt model. The error is "[TRT] [E] Error Code: 3: 1.cmap_up.0:0:DECONVOLUTION:GPU: kernel weights has count 2097152 but 4194304 was expected". Have you fixed it?


SSO with Mixed Auth.

Dear Mohammad,
Kindly I want to ask the following questions:

If it is possible to use the mixed auth. along with SSO when login as windows and how?
I am using mixed auth on MVC, and I need to always log in as windows without the need to click the button, and this is applicable to open any controller/action and the system to get the windows identity, the same behavior when using Windows authentication

Please advise.
Thanks.

Hello Ahmad,
Yes, if you SSO server support any OWIN authentication provider, you should be able to use OWIN-MixedAuth in a similar manner.
As for the 2nd point, see this comment
Thanks!

Thanks a lot for the reply Mohammad, for the SSO, would you please advise for the steps needed to plugin the mix auth. to SSO if any. Thanks.

There is no specific steps, it depends on your SSO implementation and whether it supports OWIN providers or not!

Thanks a lot :)

Another question please, is the mixed auth. supports the claims identity and how can use it?

Yes it does, What do you mean by how to use it! Can you be more specific ?

I have an MVC controller, calling WebApi controller ( login action ), and I want to use mixed auth. would you please advise how I can login to WebApi when the user login as windows (i.e. how to send windows credentials to WepApi login action). Thanks.

You need to hit the authorize end point to get an access token, then use it along with your Api calls. See the sample project named SPA (The access token is retrieved here and the Authorize end point is defined here).

You mean I need to hit an Api controller to generate a token, if yes, based on what I want to generate it, actually I am using AspNetIdentity on the business logic layer, the BL is called by Api, and we are going to support forms and windows login, so, the login is done through Api, not MVC, our architecture was  to be able to remove the UI layer without effecting anything, so, there is no any dependency on the MVC.
Would you please advise. Thanks.

In both case you need to generate an access token, For forms authentication you need to use Password Login Flow on the token endpoint and for windows authentication you need to use External Login Flow  on the authorization endpoint.
See http://www.asp.net/web-api/overview/security/individual-accounts-in-web-api for more details

Thanks a lot Mohammad. appreciated. I will check and come back to you.

Dear Mohammad,
I have a case here, that is, I need when run a new IE session, and then click on windows button to display the windows login dialog not to login as the user previously logged in (saved by the cookie).
would you please advise.
Thanks.

See #4

Also this comment :)


added phony targets to make file

Added phony targets that improve make performance
added shell check before running errcheck
added a lint command to make

LGTM, feel free to merge when you think it's ready.


Some refactoring to improve code readability

🤖 AutoRest automatic publish job 🤖
success (version: 2.1.84)


initial commit for tensorflow-hub

conda recipe for hub

I'll follow up with another PR to enable package upload.


content/en/docs/concepts/storage/volumes.md: out-dated introduction

This is a Bug Report
Problem:
The "background" sections describes volumes as if they all were ephemeral:

A Kubernetes volume, on the other hand, has an explicit lifetime - the same as the Pod that encloses it.

It also starts with a comparison against Docker, which may or may not still make sense.
Proposed Solution:
Rewrite and link to the persistent-volumes.md and (once https://github.com/kubernetes/website/pull/22438 is merged) to ephemeral-volumes.md.
Page to Update:
https://kubernetes.io/docs/concepts/storage/volumes/

/sig storage
/priority backlog
I definitely agree about the Docker stuff - there's no need to focus on a particular runtime, nor on Docker as a local development tool.

/remove-lifecycle stale
/lifecycle frozen

/triage accepted

/triage accepted

/assign


[local_auth] provide the authentication in the app context not the phone context.

Hello, I am developing a very sensitive app, there is a behavior that we are trying to achieve

user installs the app for the first time.
user logs into their account for the first time using their credentials.
app checks if there are any biometric authentications registered that are related to this account.
since this is the first time for the user to log in on this device they are asked to register a local authentication mechanism.
next time when the user opens the app, they still have an active session so there is no need for them to log in again, they only need to locally authenticate to prove that they are the same person who started this session.

we only want the user who initially logged in to be able to open the app, this is how the package works for most cases except for when you have an active session and the user removes the previously registered biometric authentication mechanism, here is what I mean

user already has an active session and has registered a biometric authentication mechanism.
the user decides to remove the local authentication mechanisms but forgets to log out of the app (and sells the device or gives it to someone else)
The new owner of the device opens the app.
The previous owner's account is still logged in since his session has not expired.
the local_auth won't find any registered biometric authentication mechanism so it will ask the user to register a new one.
the new user registers a new authentication mechanism.
the new user now can go inside the application with the previous device user's account.

one way to solve this issue is to relate the biometric authentication data to an account, not a device (local_auth does not keep that data), when a user registers their biometric authentication information we store the information somewhere (be it the cloud or securely on the device) the next time the user tries to authenticate the newly provided data compared to the data they provided at the authentication. this way even if you change the device and forget to log out, the new owners won't be able to use the app with the previous owner's session.
I hope that is clear enough, if not please feel free to ask so I can try to clarify more.
if there is a better way to achieve this behavior please let me know.
Thank you.

local_auth is wrapping OS authentication functionality, which is at the device account level. If you have specialized needs beyond that functionality that involve validating the exact biometric details, you'd need a custom plugin as that's out of scope for local_auth.


Feature request: additional logging to debug connection pool deadlocks

I ran into a problem caused by code in my application which would request and hold on to two connections simultaneously on the same thread. When testing the system under heavy load with 50 threads sharing a pool of 10 connections it would deadlock. i.e. All of the application threads where blocked in getConnection(), and no threads were waiting on the database to complete work.
To debug this I added logging when a thread called getConnection() but already held an open connection It logs two stacktraces, one for the initial call, and one for the subsequent call.
It would be nice to have a feature like this added to Hikari.
Here is the quick hack I used to debug my application as an example of what I mean:
https://github.com/xxgreg/HikariCP/commit/bef009ddf775e1d8473a285d06335903234e4400

@xxgreg I do appreciate the effort required to diagnose an issue like this.  Having said that, providing these kinds of checks in the "hot path" of getConnection() would crush performance.
While in the case of your application opening and holding two connections in the same thread resulting in an issue, there are many frameworks, including Hibernate and Play/Slick that do so routinely as a matter of course.  Unfortunately, I don't think the diagnostic code is universally applicable.
Of course the nice thing about open source is that you were able to instrument HikariCP to suit your needs and diagnose the issue.

If an application is relying on being able to obtain multiple connections simultaneously from the same pool how can it protect itself against a deadlock? This sounds like a recipe for trouble to me - given sufficient contention it will always be possible to have all threads waiting on each other to release a connection in order to make progress.

I'm not sure how your application in particular got into that pickle, but the calculation of pool size in this scenario in order to avoid deadlock is fairly simple:
pool size = tn x (cm - 1) + 1
Where tn is the maximum number of threads, and cm is the maximum number of simultaneous connections required by a single thread.
For example, imagine three threads (tn=3), each of which requires four connections to perform some task (cm=4).  The pool size required to ensure that deadlock is never possible is:
pool size = 3 x (4 - 1) + 1 = 8
If you have a maximum of ten threads (tn=10), each of which requires two connections to perform some task (cm=4).  The pool size required to ensure that deadlock is never possible is:
pool size = 10 x (2 - 1) + 1 = 11

Thanks for the detailed comments.
The app had 50 jetty threads, 10 connections in the pool, connections were acquired and released multiple times per request, and two simultaneous connections were required for a thread to complete (this was introduced accidentally). So it was a deadlock waiting to happen. Glad that we caught it during load testing and not in production.
I wonder whether it is worth adding what you have written above to your very helpful article about connection pool sizing. Perhaps some users of Hibernate and Slick are unaware that their connection pool needs to be larger than their number of application threads to avoid deadlock.
Currently there is this:
"The prospect of "pool-locking" has been raised with respect to single actors that acquire many connections. This is largely an application-level issue. Yes, increasing the pool size can alleviate lockups in these scenarios, but we would urge you to examine first what can be done at the application level."

@xxgreg Good idea, I will add the above to the pooling page, as well as the FAQ.


[BI-625] Configure TLS (HTTPS) for sandbox/qa environments

The domain for bi-api-root was from localhost:8081 to localhost since biapi will not have a port exposed on teh host.
Fixed a bug in the docker-copmse.yml files so that the source code and tests are now copied into biweb when in production mode.

I'm a little confused how http://localhost gets routed to bi-api. But, I'm guessing you tried it out and it worked and biproxy is able to handle it, so I'm good with it.

Ports 80 and 443 for localhost are connected to the reverse proxy.  The rule set for the reverse proxy determines which requests are sent to a specific upstream server: in the present case that's either bi-web or bi-waf(which sends requests along to bi-api). Requests sent to /server-info and the /sso and /v1 namespaces are sent to bi-waf; all others are sent to bi-web.

I'm a little confused how http://localhost gets routed to bi-api. But, I'm guessing you tried it out and it worked and biproxy is able to handle it, so I'm good with it.

Ports 80 and 443 for localhost are connected to the reverse proxy.  The rule set for the reverse proxy determines which requests are sent to a specific upstream server: in the present case that's either bi-web or bi-waf(which sends requests along to bi-api). Requests sent to /server-info and the /sso and /v1 namespaces are sent to bi-waf; all others are sent to bi-web.


Update the package name and path in documents

Description

As the package name changes from whisk to org.apache.openwhisk, I updated the package name in documents.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[x] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[x] I updated the documentation where necessary.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@42831e7). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #4166   +/-   ##
=========================================
  Coverage          ?   81.22%
=========================================
  Files             ?      152
  Lines             ?     7291
  Branches          ?      470
=========================================
  Hits              ?     5922
  Misses            ?     1369
  Partials          ?        0

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 42831e7...4f016f8. Read the comment docs.


about the code of papers

Hi， Excuse，is  the code for "Iterative Deep Graph Learning for Graph Neural Networks: Better and Robust Node Embeddings " different form "Deep Iterative and Adaptive Learning for Graph Neural Networks"?
In the paper "Iterative Deep Graph Learning for Graph Neural Networks: Better and Robust Node Embeddings",you give the implementation: https://github.com/hugochan/IDGL. But, the page of github show  that ,it is for "Deep Iterative and Adaptive Learning for Graph Neural Networks".  I am confused about this.  thanks.

Hi， Excuse，is the code for "Iterative Deep Graph Learning for Graph Neural Networks: Better and Robust Node Embeddings " different form "Deep Iterative and Adaptive Learning for Graph Neural Networks"?
In the paper "Iterative Deep Graph Learning for Graph Neural Networks: Better and Robust Node Embeddings",you give the implementation: https://github.com/hugochan/IDGL. But, the page of github show that ,it is for "Deep Iterative and Adaptive Learning for Graph Neural Networks". I am confused about this. thanks.

Thank you for your interest to our work! The new IDGL paper extends the previous work by proposing a scalable version of IDGL, namely, IDGL-Anch. We will update this repo to include code of both models soon.

Hi， Excuse，is the code for "Iterative Deep Graph Learning for Graph Neural Networks: Better and Robust Node Embeddings " different form "Deep Iterative and Adaptive Learning for Graph Neural Networks"?
In the paper "Iterative Deep Graph Learning for Graph Neural Networks: Better and Robust Node Embeddings",you give the implementation: https://github.com/hugochan/IDGL. But, the page of github show that ,it is for "Deep Iterative and Adaptive Learning for Graph Neural Networks". I am confused about this. thanks.


ResourceNotFoundError: (404) Resource not Found

Your new release 3.3.0.b1 has some issue with the version. Can you please check what is the problem.
This is the error:
ResourceNotFoundError: (404) Resource not found
code;: 404
Message: Resource not Found

Hi @RAravindDS , can you provide some information about the package that you are using, python version, some sample code to reproduce the error etc so that we can better assist you ?


Wallet unable to calculate bitcoin fees, finds custom fees too high

User gets a notice that the wallet is unable to calculate fees. But it looks like it has an opinion about the fee "Fee is too high" anyway as the user tries to send the transaction with a custom fee.
I believe the error should say "Fee exceeds available balance". (because the wallet doesn't judge fees at all, I can set a fee rate of 20,000 sats/vB and it will not tell the user "it is too high" if the wallet has sufficient balance).
Although the user in this case said the the available balance showed sufficient I think it may have been caused by the API issues the user was experiencing in the first place. Likely Bitcoin API's were unreachable. The user doesn't see the error now but will create screenshots of the console if it pops up again.

This is labeled bug though seems more like a copy enhancement to Fee exceeds available balance for clarity?

It was a bit of an evolution, def an enhancement now.
I think it is a bug if the available balance showed sufficient yet this error is thrown but we can't confirm that based on the screenshot. Perhaps we can if we see this again. If there is no API available to determine the wallet balance I don't think it should advance to this step to avoid the confusion. Something to keep in mind when adding loading/error stats for that https://github.com/leather-wallet/extension/issues/3873

@fabric-8 worth incorporating this edge case into your send-related UX work in general?

A user confirmed this is popping up when fees can't be auto-computed, but the API does return fees:

{"fastestFee":55,"halfHourFee":48,"hourFee":42,"economyFee":42,"minimumFee":25}
GET https://mempool.space/api/v1/fees/recommended

@fabric-8 can review this issue to ensure the relevant case(s) are addressed by the new fees UX as part of approval UX? 🙏


Fixes #1666

This PR adds support for the iLabs OpenDec02 board by adding the ilabs_opendec02.h header file with specific settings for that particular board.

fixes #1666


Error with Elixir mix dialyzer when using the library

I am trying to use the library in a Elixir project and also use Dialyxir.
As indicated in the doc, I specified :systemd.ready() as:
# Elixir
defmodule MyProject.Application do
  use Application

  def start(_type, _opts) do
    children = [
      MyProject.Repo,
      MyProjectWeb.Endpoint,
      :systemd.ready() # <- IMPORTANT - this is a function call (it returns the proper child spec)
    ]

    Supervisor.start_link(children, strategy: :one_for_one)
  end
end

However, because of that, mix dialyzer fails with error Function start/2 has no local return:
$ mix dialyzer
Compiling 1 file (.ex)
Finding suitable PLTs
Checking PLT...
[:absinthe, :absinthe_plug, :absinthe_relay, :argon2_elixir, :asn1, :backoff, :bodyguard, :comeonin, :commanded, :commanded_ecto_projections, :commanded_eventstore_adapter, :compiler, :connection, :cowboy, :cowboy_telemetry, :cowlib, :crypto, :db_connection, :decimal, :ecto, :ecto_sql, :eex, :elixir, :elixir_uuid, :enough, :eventstore, :fsm, :gen_stage, :gettext, :inets, :jason, :kernel, :libgraph, :logger, :mime, :nimble_parsec, :phoenix, :phoenix_ecto, :phoenix_html, :phoenix_pubsub, :phoenix_view, :plug, :plug_cowboy, :plug_crypto, :postgrex, :public_key, :ranch, :runtime_tools, :ssl, :stdlib, ...]
PLT is up to date!
No :ignore_warnings opt specified in mix.exs and default does not exist.

Starting Dialyzer
[
  check_plt: false,
  init_plt: '/home/me/dev/myproject/_build/dev/dialyxir_erlang-24.2_elixir-1.13.3_deps-dev.plt',
  files: ['/home/me/dev/myproject/_build/dev/lib/myproject/ebin/Elixir.MyProject.Accounts.AuthorizationPolicy.beam',
   ...],
  warnings: [:unknown]
]
Total errors: 1, Skipped: 0, Unnecessary Skips: 0
done in 0m10.63s
lib/myproject/application.ex:6:no_return
Function start/2 has no local return.
________________________________________________________________________________
done (warnings were emitted)
Halting VM with exit status 2

Can you reproduce? Is it a problem with this library?
Information:

Elixir 1.13.3 (compiled with Erlang/OTP 24)
dialyxir 1.1.0
systemd 0.6.2

I get the same Dialyzer warrning - Function start/2 has no local return..
Łukasz could it be that Dialyzer never will get happy with that function call within start/2,
as it always expect to call child_spec/1 itself on :systemd to get the child spec?

Working on that.

Ah, the latest release is 0.6.2 from Dezember 2021. This seems not be released yet.
Can you please release this change as patched version 0.6.3?

@hauleth Ping! It would be lovely to get a new patch release to get this dialyzer fix.


Figure out how to make a React component a transition target.

Will probably involve creating a wrapper component that requires a name prop and gets its values from context.  It can pass those values down as props to its children.  (You could even have a simple one that wraps everything in a div and applies the transforms to itself.)
Stateless components support contexts now.  It would be sweet to do this without getting classes involved.
So, something like:

MaterialMotionProvider
MaterialMotionTarget
SimpleMaterialMotionTarget

Look into the explorations with Transitionable.  Will be interesting to see what the interface is between React's props and Material Motion's transactions.

Could be useful: https://github.com/mjackson/react-style/


Add user accounts

We should allow users to persist their progress and share it across devices by creating accounts.
I had good experiences with https://github.com/vvo/iron-session when creating user logic. But this was only for email login.
We could also think about adding more social logins and use https://next-auth.js.org/
The user data can then be stored in our mongodb. We need to find a smart way to keep both systems running next to each other (local storage & database)

first version is working


Feature 1380 tc_gen_dups

Running on Dan A's test data now produces the following 7 warning messages. These explain the discrepancy between 209 reported BEST track genesis events and 202 unique lat/lon's.
WARNING:
WARNING: GenesisInfoArray::add() -> Including multiple genesis events for the same storm:
WARNING: GenesisInfo: IsSet = true, IsBest = true, IsOper = false, IsAnly = false, StormId = "AL902018", Basin = "AL", Cyclone = "90", StormName = "INVEST", TechniqueNumber = -9999, Technique = "BEST", Initials = "(nul)", GenesisTime = "20180922_000000", InitTime = "NA", LeadTime = "000000", Lat = 35.00000, Lon = -45.00000, DLand = -9999.00000, NPoints = 7, MinValidTime = "20180922_000000", MaxValidTime = "20180923_120000", MinWarmCoreTime = "NA", MaxWarmCoreTime = "NA"
WARNING:
WARNING:
WARNING: GenesisInfoArray::add() -> Including multiple genesis events for the same storm:
WARNING: GenesisInfo: IsSet = true, IsBest = true, IsOper = false, IsAnly = false, StormId = "AL912018", Basin = "AL", Cyclone = "91", StormName = "INVEST", TechniqueNumber = -9999, Technique = "BEST", Initials = "(nul)", GenesisTime = "20181004_180000", InitTime = "NA", LeadTime = "000000", Lat = 14.50000, Lon = -83.20000, DLand = -9999.00000, NPoints = 9, MinValidTime = "20181004_180000", MaxValidTime = "20181006_180000", MinWarmCoreTime = "NA", MaxWarmCoreTime = "NA"
WARNING:
WARNING:
WARNING: GenesisInfoArray::add() -> Including multiple genesis events for the same storm:
WARNING: GenesisInfo: IsSet = true, IsBest = true, IsOper = false, IsAnly = false, StormId = "AL932018", Basin = "AL", Cyclone = "93", StormName = "INVEST", TechniqueNumber = -9999, Technique = "BEST", Initials = "(nul)", GenesisTime = "20181007_120000", InitTime = "NA", LeadTime = "000000", Lat = 7.50000, Lon = -21.60000, DLand = -9999.00000, NPoints = 9, MinValidTime = "20181007_120000", MaxValidTime = "20181009_120000", MinWarmCoreTime = "NA", MaxWarmCoreTime = "NA"
WARNING:
WARNING:
WARNING: GenesisInfoArray::add() -> Including multiple genesis events for the same storm:
WARNING: GenesisInfo: IsSet = true, IsBest = true, IsOper = false, IsAnly = false, StormId = "AL952018", Basin = "AL", Cyclone = "95", StormName = "INVEST", TechniqueNumber = -9999, Technique = "BEST", Initials = "(nul)", GenesisTime = "20181023_000000", InitTime = "NA", LeadTime = "000000", Lat = 19.00000, Lon = -46.30000, DLand = -9999.00000, NPoints = 17, MinValidTime = "20181023_000000", MaxValidTime = "20181027_000000", MinWarmCoreTime = "NA", MaxWarmCoreTime = "NA"
WARNING:
WARNING:
WARNING: GenesisInfoArray::add() -> Including multiple genesis events for the same storm:
WARNING: GenesisInfo: IsSet = true, IsBest = true, IsOper = false, IsAnly = false, StormId = "AL972018", Basin = "AL", Cyclone = "97", StormName = "INVEST", TechniqueNumber = -9999, Technique = "BEST", Initials = "(nul)", GenesisTime = "20180918_000000", InitTime = "NA", LeadTime = "000000", Lat = 7.20000, Lon = -37.70000, DLand = -9999.00000, NPoints = 17, MinValidTime = "20180918_000000", MaxValidTime = "20180922_000000", MinWarmCoreTime = "NA", MaxWarmCoreTime = "NA"
WARNING:
WARNING:
WARNING: GenesisInfoArray::add() -> Including multiple genesis events for the same storm:
WARNING: GenesisInfo: IsSet = true, IsBest = true, IsOper = false, IsAnly = false, StormId = "AL992018", Basin = "AL", Cyclone = "99", StormName = "INVEST", TechniqueNumber = -9999, Technique = "BEST", Initials = "(nul)", GenesisTime = "20180920_000000", InitTime = "NA", LeadTime = "000000", Lat = 9.60000, Lon = -7.30000, DLand = -9999.00000, NPoints = 11, MinValidTime = "20180920_000000", MaxValidTime = "20180922_120000", MinWarmCoreTime = "NA", MaxWarmCoreTime = "NA"
WARNING:
WARNING:
WARNING: GenesisInfoArray::add() -> Including multiple genesis events for the same storm:
WARNING: GenesisInfo: IsSet = true, IsBest = true, IsOper = false, IsAnly = false, StormId = "CP972018", Basin = "CP", Cyclone = "97", StormName = "WALAKA", TechniqueNumber = -9999, Technique = "BEST", Initials = "(nul)", GenesisTime = "20180926_180000", InitTime = "NA", LeadTime = "000000", Lat = 11.90000, Lon = -141.60000, DLand = -9999.00000, NPoints = 13, MinValidTime = "20180926_180000", MaxValidTime = "20180929_180000", MinWarmCoreTime = "NA", MaxWarmCoreTime = "NA"
WARNING:

So when tc_gen reports the total number of genesis events, should the numeric value printed in the log be total including duplicates, or total without duplicates? It still appears the total includes the duplicates, but a user can now grep for the number of occurrences of the new WARNING message to determine total number of unique events.

Dan,
Yes, the total does include the duplicates. I didn't actually change the
logic to ignore the duplicates. I just added a warning alerting the user to
the fact that these types of duplicates are present. Remember Dan H's
recommendation that setting the event category criteria would get rid of
the genesis events for INVEST's. This is just alerting the user that action
is required. And technically, these aren't "duplicates" because the storm
numbers do in fact differ. BUT they are two genesis events for the same
storm... which you probably don't want. And I don't want to automatically
throw anything away because then we'd need good logic to do so.
John
On Tue, Jun 23, 2020 at 11:50 AM Dan Adriaansen<EMAIL_ADDRESS>wrote:

So when tc_gen reports the total number of genesis events, should the
numeric value printed in the log be total including duplicates, or total
without duplicates? It still appears the total includes the duplicates, but
a user can now grep for the number of occurrences of the new WARNING
message to determine total number of unique events.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NCAR/MET/pull/1381#issuecomment-648318896, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFA4HKHA4E7XSBCEXAGGZSTRYDTMZANCNFSM4OCHKXNQ
.


add filter execution statistics to be analyzed by the clients

What type of PR is this?

/kind feature

Does this PR require a change in the driver versions?

What this PR does / why we need it:
Add stat fields to gen_event_filter_check to maintain filters exec statistics.
This will help clients to analyze the performance of the filters and make runtime decisions for the checks optimization/rearrangements.
the statistics may look this way:
    rule:{ name: Linux Kernel Module Injection Detected, src: syscall, types: [230, 231, 292, 293]}
            * NONE, 45941 0, LT, (6)
            [
                = NONE, 45941 45941, evt.type EQ (1)[ execve ]
                = AND, 45941 45941, evt.dir EQ (1)[ < ]
                = AND, 24288 0, evt.arg EQ (1)[ 0 ]
                = AND, 20877 20877, container.id NE (1)[ host ]
                = AND, 20877 20877, proc.name EQ (1)[ insmod ]
                = AND_NOT, 0 0, proc.args IN (1)[  ]
            ]

here we can see that if we move the check = AND, 20877 20877, proc.name EQ (1)[ insmod ] to the first place we will cut ~200K unnecessary extractions/comparisons.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: VadimZy
Once this PR has been reviewed and has the lgtm label, please assign molter73 for approval by writing /assign @molter73 in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

286911f add filter execution statistics to be analyzed by the clients
ffee997 add filter execution statistics to be analyzed by the clients. move interface to falco

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

286911f add filter execution statistics to be analyzed by the clients

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

286911f add filter execution statistics to be analyzed by the clients
9bb8367 add filter execution statistics to be analyzed by the clients; Signed-off-by: VadimZy<EMAIL_ADDRESS>

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

LGTM label has been added.  Git tree hash: dde8f7921b65c0384a417f879be48fe2a206df8f

/milestone 0.9.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, VadimZy
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr, VadimZy
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Bug fix for Undo/redo

Prevents element from moving during resize state of other elements, and adds additional lifecycle method to better support undo/redo.

LGTM!


rewrote illusion instead of hallucination

an illusion is the false interpretation of reality / whats actually happening.
an hallucination is the mind making up completely new things that are not observable in reality, the word doesnt fit for what is happening here

well I changed it to "feel"


Cannot install via HACS - getting 404 from GitHub

Getting this kind of messages:
Logger: custom_components.hacs
Source: custom_components/hacs/base.py:730
Integration: HACS (documentation, issues)
First occurred: 18:41:05 (1 occurrences)
Last logged: 18:41:05

Download failed - Got status code 404 when trying to download https://github.com/siku2/hass-komfovent-c5/releases/download/0.3.5/dingz.zip
Traceback (most recent call last):
  File "/config/custom_components/hacs/base.py", line 730, in async_download_file
    raise HacsException(
custom_components.hacs.exceptions.HacsException: Got status code 404 when trying to download https://github.com/siku2/hass-komfovent-c5/releases/download/0.3.5/dingz.zip

and indeed the file https://github.com/siku2/hass-komfovent-c5/releases/download/0.3.5/dingz.zip is missing on GitHub, but mentioned in hacs.json. I am using following setup:
2024.1.5
Supervisor
2023.12.1
Operating System
11.4
User Interface
20240104.0

What am I missing?

Woops, looks like I forgot to adjust the filename in the hacs.json file

custom_components.hacs.exceptions.HacsException: Got status code 404 when trying to download https://github.com/siku2/hass-komfovent-c5/releases/download/0.3.5/dingz.zip
Still getting the same error, but I see in the git repo the value in the file has been changed. Maybe Home Assistant is not refreshing the content or the missing file is mentioned elsewhere? How to clear appropriate HA cache?

Should be fixed now.


chore(deps): update common-dev-assets digest to 5c14bd5

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
a4ab5cf -> 5c14bd5

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This PR is included in version 8.2.0 :tada:
The release is available on:

GitHub release
v8.2.0

Your semantic-release bot :package::rocket:


unclutter: prevent service restarting too soon

Things done

[x] Tested
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Thank you


Simplify ZetaClient chains config

We currently have the following config for chains in ZetaClient.
type EVMConfig struct {
	observertypes.CoreParams
	Chain    common.Chain
	Endpoint string
}

This config could be simplified:

observertypes.CoreParams is not necessary to be stored since it can directly be queried on-chain
We don't need to store common.Chain object, we could directly store the chain ID since the full chain object can be retried from it

Closing for https://github.com/zeta-chain/node/issues/1608


FileObserver setup concurrent-safe

The FileStorage observer usually works great for me in concurrent environments (e.g. 50 nodes writing simultaneous).  This PR tries to fix this by the following changes:

Use os.makedirs with exists_ok=True flag (available for python >= 3.2)
Add queued runs the same way than started runs

Coverage increased (+0.005%) to 83.941% when pulling c6ed5b3ad231240e081a0ddb0e0fe4daccaeda5f on dekuenstle:patch-1 into 72633776bed9b5bddf93ae7d215188e61970973a on IDSIA:master.


Document options for GnuplotRB::Plot.new in source code docs

It would be great to have the documentation (viz. the options that can be  passed and all their possible values and effects) in the source code docs itself, so that one does not need to hunt through the notebooks for figuring out usage all the time.
Also, if it could be possible to configure plots through some kind of a DSL, that would be great. It would also improve extensibility since I can just pass a configuration object to a daru user and she can do whatever she wants with it.

I haven't added docs for options because they are covered in gnuplot doc(pages 105-190). I'll add some (most often used in my opinion) of them to the docs but I don't think it's good idea to rewrite here all gnuplot's doc.
Couldn't you also provide some examples of DSL you want to use with gnuplotrb?

Ok the most common can be documented. Also it would be great if there was a very visible link to GNU plot docs in the gnuplotRB docs somewhere.
I'm currently writing notebooks for daru using your gem, I'll post some DSL examples in a while.


☁ AML Dataset

Describe your changes
Add AML dataset support for data_dir, user_script and script_dir for evaluation and pass running.

In local system, olive will download aml dataset to local, then start olive optimization.
In aml system, olive can directly mount the data to evaluation/pass_run aml job with the way of Input(type=AssetTypes.URI_FOLDER, path="azureml://xxx/xxx"
Remove AzureCliCredential as we update the pipeline with SP credential.
Remove AzureCLI

Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Format your code by running pre-commit run --all-files
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

def get_non_local_resource(resource_path: ResourcePath, cache_dir: Union[str, Path] = ".olive-cache"):

the name is confusing. how about get_resource_local_path?

Refers to: olive/cache.py:110 in 3cb8a2b. [](commit_id = 3cb8a2b10584a3b7359eb4b9247d9d3f1f1f5fd1, deletion_comment = False)

download_resource

Updated.


Add initialFocus prop to ModalCurtain

I went for a selector-based API. Interested to get your thoughts on this. Would you prefer, this, or a ref?

@lavelle — I've revisted this since it's a requirement for moving away from ModalBase. I know this is originally your PR, but can you PTAL at the changes I've made?

LGTM! I think you will need to approve since it's my CR


Scons builder for cython extensions

Build cython files with scons instead of python setup scripts

Before merging, make sure that the new scons builder properly detects when dependencies have changed, e.g. make a change to common/params_pyx.pyx and make sure it rebuilds when running scons.


Tracker Mode for Tobii 4c

Hi,
I wounder, which option in "Tracker mode" selection is the best to use for my Tobii 4c.

I have a Tobii 4C and I prefer to use the one with eViacam. This way I don't need to wear a separate head tracker like the TrackIR. It also gives better accuracy than the head tracking in the 4C by itself.

I think I answered this question so I'm just going to close this issue. Feel free to reopen it if desired.


MAINT: DRY code for src subject and better src repr

Changes this:
larsoner@bunk:~/python/mne-python$ python -c "import mne; print(mne.read_source_spaces('/home/larsoner/mne_data/MNE-sample-data/subjects/sample/bem/sample-oct-6-src.fif'))"

<SourceSpaces: [<surface (lh), n_vertices=155407, n_used=4098, coordinate_frame=MRI (surface RAS)>, <surface (rh), n_vertices=156866, n_used=4098, coordinate_frame=MRI (surface RAS)>]>

Into this (moving coord frame to the end since it's shared/constant, adding subject):

<SourceSpaces: [<surface (lh), n_vertices=155407, n_used=4098>, <surface (rh), n_vertices=156866, n_used=4098>] MRI (surface RAS) coords, subject 'sample'>

And along the way DRYs our use of src[0]['subject_his_id'] by adding a convenience @property to SourceSpaces. Maybe it should be public (?), but for now I left as private.

thx @larsoner


Add this plugin to the mobile ribbon menu

I would like to access the Create new chat from template command from the mobile ribbon menu.

Ok, personally I don't need this anymore since I replaced the ribbon with the command palette. If anyone else would like the feature nonetheless, please comment or react to help prioritize the feature

@lukemt I use Commander (https://github.com/phibr0/obsidian-commander) for things like this!

@bramses Good idea though, didn't think of this


Release Google.Cloud.Channel.V1 version 1.6.0

Changes in this release:

Commit 2382311: docs: Small clarification

Created release for Google.Cloud.Channel.V1-1.6.0


Using Flink 1.15.0

Can this be closed now that https://github.com/apache/flink-statefun/commit/b4a85ef031e539e7c2580a796246e2896d533d8f was merged?


Added reference YAML files for RBAC configs for driver and shuffle service

What changes were proposed in this pull request?
Adds YAML files for reference RBAC configuration for the Spark driver and shuffle service. To complement https://github.com/apache-spark-on-k8s/userdocs/pull/14.
Ref: #500.
How was this patch tested?
Manual tests pending.
@foxish @kimoonkim

Config LGTM, feel free to merge on green.


fix popup-text.js - matches undefined

TypeError: Cannot read property 'groups' of undefined
fixes #758 #756

Thanks for the fix.  Sorry for not catching this myself!


Closure.js Support

I tried using browserify with a the libphonenumber package which uses closure.js. When I require it the browser throws an error:
Uncaught TypeError: fs.readFileSync is not a function

Is this a known limitation or is there a workaround for this?

Browserify with the brfs transform supports inlining fs.readFileSync. Unfortunately, closure.js's load_script uses readFileSync with dynamic paths, which brfs can not handle.
This is a known limitation.


Command line tab completion or suggestion

I would like to use spectre.console and spectr.console.cli but it seems to be missing command line completion capabilities. I have been using System.CommandLine but I see a lot in spectre that makes me want to try it out. But I do not want to lose the command-line completion capabilities that I get with System.CommandLine and dotnet-suggest.  Is command line completion on the roadmap for spectre.console.cli? I appreciate your time.

It's been in discussion with #267. There has been a few people taking a stab at it, but not a ton of progress. So it's still up for grabs if anyone wants to get after it.

Thanks! I had seen that issue, but it seems to have fizzled out.


TAB key puts cursor in a weird place when having more than 8 channels

How to reproduce:
Start FT2 clone, add 2 channels (making it 10) and keep pressing TAB. On the last channel, the view does not wrap around to channel 1 and the cursor is stuck in a weird place in between channels 9 and 10.
Something like this (where the square brackets should represent the cursor):
CH9         CH10
--- -- 0[00 -]-- -- 000

Note inputs then end up on channel 1.
This is with v1.28 on Win64.

Thanks for mentioning this bug, I have commited a fix. Will release v1.29 very soon (I think this is important enough of a bugfix to get a new version).

Release binaries uploaded. Please let me know if it's still not working correctly. :)


build: support 2024.2

Summary
This PR updates the build configuration to support version 2024.2 of JetBrains IDE
Testing

The plugin was successfully built and tested within IntelliJ IDEA running version 2024.2 (IU-242.20224.300)
All existing functionality remains intact and behaves as expected

Thanks for contributing


fix: explicitly pulls in the Python2.7 get-pip script

Description
Currently new nodes are not coming up as the bootstrapping script being fetched is meant for Python3 NOT Python2.7
Migrations required
NO
Verification
User data script execution failure extract:
+ python get-pip.py --user
Traceback (most recent call last):
  File "get-pip.py", line 24244, in <module>
    main()
  File "get-pip.py", line 199, in main
    bootstrap(tmpdir=tmpdir)
  File "get-pip.py", line 82, in bootstrap
    from pip._internal.cli.main import main as pip_entry_point
  File "/tmp/tmpnuTkF8/pip.zip/pip/_internal/cli/main.py", line 60
    sys.stderr.write(f"ERROR: {exc}")

Other mentions of the problem https://stackoverflow.com/questions/65896334/python-pip-broken-wiith-sys-stderr-writeferror-exc
Documentation
Please ensure you update the README in _docs/README.md. The README.md in the root can be updated by running the script ci/bin/autodocs.sh, requires terraform-docs version 0.8+

Oups. This has been solved by moving to Python3. A new version just hasn't been cut yet :)


PCI: Move to libuser. Implement 64-bit BARs.

Share the PCI implementation across multiple drivers by moving it to libuser. Should eventually move to its own service, but this is a good temporary measure.

news on this PR ? Is this ready for review ?

I'll probably have to re-do it entirely.

good call


Getting the right answer, still not passing

Challenge Caesars Cipher has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function rot13(str) { // LBH QVQ VG!
  var result = ' "';

  for(i = 0; i < str.length; i++){

    if(str.charCodeAt(i) == 32){result += ' ';}
    else if (str.charCodeAt(i) < 65 || str.charCodeAt(i) > 90){result += String.fromCharCode(str.charCodeAt(i));}
    else if (str.charCodeAt(i) < 77){result += String.fromCharCode(str.charCodeAt(i) + 13);}
    else{result += String.fromCharCode(str.charCodeAt(i) - 13);}

  }
 result += '" ';

  return result;
}

// Change the inputs below to test
rot13("SERR PBQR PNZC");

You're adding double quotes and a space around the output. The double quotes are simply there to signify the input and output are a string. They should not be in your output. In the future please visit the Help Chat to confirm what you're seeing is a bug and not an issue with your solution. Thanks and happy coding!

Yes I only did that because my original code without the double quotes would not pass, the problem still stands- I get the proper result but it will not pass me

The code you posted (minus the extra quotes) is still incorrect. The second test expects the result to be FREE PIZZA!" but your code outputs "FREE PI@@A!".
Hint: One of the numbers in your code is off by one.

Thanks I can get the code to output Free Pizza! by changing the 2nd else if to >= 77 so i get the right answer now but still won't pass

function rot13(str) { // LBH QVQ VG!
var result = ' ';
for(i = 0; i < str.length; i++){
if(str.charCodeAt(i) == 32){result += ' ';}
else if (str.charCodeAt(i) < 65 || str.charCodeAt(i) > 90){result += str.charAt(i);}
else if (str.charCodeAt(i) < 78){result += String.fromCharCode(str.charCodeAt(i) + 13);}
else{result += String.fromCharCode(str.charCodeAt(i) - 13);}

}
return result;
}
// Change the inputs below to test
rot13("GUR DHVPX OEBJA QBT WHZCRQ BIRE GUR YNML SBK.");

`function rot13(str) { // LBH QVQ VG!
var result = ' ';
for(i = 0; i < str.length; i++){
if(str.charCodeAt(i) == 32){result += ' ';}
else if (str.charCodeAt(i) < 65 || str.charCodeAt(i) > 90){result += str.charAt(i);}
else if (str.charCodeAt(i) < 78){result += String.fromCharCode(str.charCodeAt(i) + 13);}
else{result += String.fromCharCode(str.charCodeAt(i) - 13);}

}
return result;
}
// Change the inputs below to test
rot13("GUR DHVPX OEBJA QBT WHZCRQ BIRE GUR YNML SBK.");
`

Ok this code passes all tests manually, editor still won't pass it

Remove the space here: var result = ' ';

Yes already figured that out. Such a simple mistake . Thank you so much for putting up with this cranky old man


Request for Docker support in Momotok-Server.

Dear Shanwer:
I hope this message finds you well. I am writing to request the addition of Docker support to Momotok-Server. I believe that introducing Docker containers will greatly enhance the project's flexibility, portability, and ease of deployment for its users.
I am willing to assist in the implementation of Docker support by contributing code, documentation updates, or any other help deemed necessary. Please let me know how I can contribute to this effort.
Thank you for considering my request. Docker support would undoubtedly benefit the project and its community, and I look forward to your positive response.
Sincerely.

Dear shi9uma,
Thank you for your interest in adding Docker support to Momotok-Server. Your input is greatly appreciated, and I am happy to discuss this with you.
To begin, could you please provide more information about your desired Docker support? What specific architecture of Docker do you hope to integrate into Momotok-Server? Are there any particular use cases or scenarios you have in mind?
Additionally, could you share your technical expertise and familiarity with Docker? Do you have prior experience working with Docker containers and their integration into other systems? I apologize for my lacking experience of programming as I am a beginner of that. Besides, I am currently learning rust and operating system, so that time is precious for me to handle additional work either.
Please note that contributions to Momotok-Server are welcome, and your help in implementing Docker support would be greatly appreciated. However, given the complexity and nature of this work, we may need to establish a more formal collaboration plan and potentially seek out other community members who may be able to assist as well.
Thank you again for your interest in Momotok-Server. I look forward to your response and the opportunity to discuss this further.
Best regards,
Shanwer

Shanwer,
Thank you for your prompt response. I appreciate your willingness to discuss the addition of Docker support to Momotok-Server. Here are my answers to your questions:

Desired Docker Support:
I would like to integrate a multi-container Docker architecture for Momotalk-Server, separating the momotalk-talk application from a selectable SQL database. This architecture will provide better scalability for the project in the future. Additionally, it would be ideal to publish the application to Docker Hub so that others can easily use and promote Momotok-Server using technologies like Docker Compose.

Technical Expertise with Docker:
I have a good understanding of the principles behind Docker and I have previous experience packaging diverse repositories into Docker containers and migrating them to different hosts. I am confident in my ability to implement Docker support for Momotok-Server and ensure its proper functionality.

Additional Suggestion:
I would like to propose including the project's corresponding APK installation package within the Docker image. This would allow users to download the APK installation directly from a URL and promptly start using Momotok-Server's video streaming capabilities.

Please let me know how I can further assist with the implementation of Docker support for Momotok-Server. I am eagerly looking forward to contributing to the project.
Best regards,
shi9uma

Shanwer,
Thank you for your prompt response. I appreciate your willingness to discuss the addition of Docker support to Momotok-Server. Here are my answers to your questions:

Desired Docker Support:
I would like to integrate a multi-container Docker architecture for Momotalk-Server, separating the momotalk-talk application from a selectable SQL database. This architecture will provide better scalability for the project in the future. Additionally, it would be ideal to publish the application to Docker Hub so that others can easily use and promote Momotok-Server using technologies like docker-compose.

Technical Expertise with Docker:
I have a good understanding of the principles behind Docker and I have previous experience packaging diverse repositories into Docker containers and migrating them to different hosts. I am confident in my ability to implement Docker support for Momotok-Server and ensure its proper functionality.

Additional Suggestion:
I would like to propose including the project's corresponding APK installation package within the Docker image. This would allow users to download the APK installation directly from a URL and promptly start using Momotok-Server's video streaming capabilities.

Please let me know how I can further assist with the implementation of Docker support for Momotok-Server.I am more than willing to provide support for the project.
vanitas vanitatum et omnia vanitas,
shi9uma

Shi9uma sama,
Thank you for your continuous attention to the issue, but due to the shortage of time, forgive me for my short reply.
To implement the docker support, you can fork this repository and make commits to your own branch. Before making a pull request, ensure the server operates property and stably. And then you can publish it to Docker Hub. I will provide as much support as I cam provide to you.
domino gratias,
Shanwer


fix(focus-trap): apply aria-hidden to focus trap tab anchors

These anchors at the book ends of a FocusTrap'ed element have focus listeners that redirect focus
back to the element. However, some screen readers may access these focus traps without moving
programmatic focus, leaving the SR user wondering why an empty control lives on the page.  Android
TalkBack currently treats this as a stop with no announcement (because it has no content). Adding
the aria-hidden should prevent these from being accessed in SR contexts while preserving the core
functionality of redirecting focus when it's moved linearly (e.g., with tab).
cdk FocusTrap adds focus trap divs with tabindex=0 at the beginning and end of the trapped container to prevent focus from leaving when pressing tab or linearly navigating. When these elements are focused, a focus handler redirects the browser focus as to effectively cycle focus within an element. This works fine for e.g., OS X VoiceOver, which generally moves focus when navigating the page. However some screenreaders may routinely browse elements without moving focus. A notable example is TalkBack, which is one of Angular Material's supported environments https://github.com/angular/material2#browser-and-screen-reader-support. This is also true in some popular configurations of JAWS/NVDA.
Settings these to have aria-hidden=true will typically cause for SRs to ignore these when linearly navigating the page, if they're not already ignored.
I've tested a primitive version of this fix on TalkBack and it works well.
https://stackblitz.com/edit/angular-e9gqbr?file=app%2Fdialog-content-example.ts
https://www.youtube.com/watch?v=SN1chzTfvTs
I can imagine some rare edge cases where this could cause a regression in functionality. In cases where cdk FocusTrap is used but were relying on SRs to "see" the focus trap anchors and shift focus to them, so that the active focus handler would redirect focus, this may break things, and SRs may wrongly shift focus to a bad spot. However, doing it this way would have been extremely finicky in the first place. If you wanted SRs not to be able to escape the focus trap, you should have set aria-hidden on the rest of the page in the first place (MatDialog does this https://github.com/angular/material2/blob/d22f48c742c0bbdc971f79ea1261ec5c20d15a69/src/lib/dialog/dialog.ts#L345, and individual application code may wrap cdk Overlay to do this; it's a bit of a mystery to me why cdk Overlay or cdk FocusTrap doesn't do this itself, but that's a separate concern). Such a page would have been a pain for SRs in the first place since heading and landmark navigation, common SR navigation techniques, could easily have shifted focus to the underlying page anyway. These users should be encouraged to apply aria-hidden appropriately. That said, we have tested this on the following SRs on the latest versions and things appear to be no worse:
Android TalkBack - my fix should address the hidden SR stop
NVDA/JAWS with virtual cursor - similar fix, but virtual cursor configuration varies greatly among users; generally, the hidden tab stops aren't problematic
OS X VO - always ignored the empty tabindex=0 anchors anyway
Internal bug reference b/122041252.

This will cause a failure with Deque University Accessibility Checker Axe. More info can be found here:. I believe that this is a bandaid and not a solution.

Thanks for raising this Michael.
I can understand why the Axe checker recommends as it does but I believe the recommendation is imprecise or make an alternate suggestion to address the modal dialog use case. The spirit of the recommendation to my reading is to avoid accidentally obscuring semantically important focusable elements. However there are two major cases, both related to modal dialogs, where aria-hidden hides elements that fundamentally should be semantically hidden.

Hiding tab stop anchors (this PR). Tab traps are necessary in many modal and non-modal popup applications. In the modal case, it prevents focus from wrongly going to document behind the popup. In the non-modal case where reparenting (popup moved out of DOM order but semantically kept in order using aria-owns) is used, the tab focus traps redirect focus correctly when it leaves the popup. In both cases, NVDA, JAWS, and Talkback can all move the virtual cursor without moving focus, and these tab stops, which exist for no purpose other than to redirect programmatic focus, are wrongly exposed.
Hiding base page content. When a modal popup is opened, the base page should be hidden because in practice aria-modal="true" doesn't prevent the base page from being accessed in virtual cursor mode on any screen reader I've tested. In these cases, we must apply aria-hidden to the entire base page, which includes many interactive elements. Until SR vendors have wide support for aria-modal which I agree feels less bandaid-y, I think we have to use this approach.

If there's an alternative that satisfies these needs to display a usable modal dialog, please point me to it. https://www.w3.org/TR/wai-aria-practices/examples/dialog-modal/dialog.html actually fails on point 2 and doesn't even correctly hide the tab stops on point 1, causing overannouncing of semantically useless elements on some SRs. Our current approach to modal dialogs satisfies both points, has been reported as accessible by our testers, and only falls short of the letter of the Axe test.

Thanks, @stevenyxu for your feedback. I was merely stating an observation that I had come across. The only other option that I can think of -- still not standardized, although a polyfill is available -- is the ability to mark dome elements as inert. Thereby eliminating the need for tab stops altogether.
This would also handle the debate as to whether a keyboard trap on a dialog should exclude changing the focus from the viewport to the browser itself. I.e. allowing a user to navigate via keyboard (without the use of an SR) to the URL address bar without the need to dismiss the dialog.

Thanks Michael. We will monitor inert and how the proposal advances.
The polyfill in its current state has the same defect you reported since it relies on aria-hidden to prevent landmark/type navigation and uses tabindex="-1" to make focusable nodes untabbable, which doesn't stop them from being focusable and still violates the letter of the rule you cited.
Additionally, inert does not address the non-modal use of tabindex traps, which would require an additional new standard (comparable to aria-owns but working for standard tabbing) and polyfill.


add SCHISM submodule and build files

these changes were manually ported over from https://github.com/pvelissariou1/ADC-WW3-NWM-SCHISM-NEMS/, with commits amended to show authorship to @pvelissariou1 and @platipodium under the email addresses used in the commits

continuation of #43 , but this one is actually merging into master

still doing some testing so don't merge just yet

waiting on https://github.com/schism-dev/schism/pull/31

pushed commits to submodules, updated submodule pointer to correct commit head
testing build now

@platipodium I've merged feature/pahm into feature/schism, which included @pvelissariou1's updates to the build system but also the PaHM submodule
Let me know if there are any glaring problems, as I can always revert d2e0a24

looking good so far!

added a github workflow that validates submodule pointers


GradCam

I'd love to try and implement GradCam on this project, however the pretrained models seem to be packed with the Tensorflow save instead of Keras.  Do you have the checkpoints available for the evaluation or training code for finetuning/ custom implementations?

Do you necessarily need a Keras-style checkpoint for this? It should be possible to backprop through the TF SavedModel as well and get gradients wrt the input image. You may want to use the model.crop_model for this analysis, which is the single-person pose estimator at the core of the method, which takes a 256x256 cropped image. (See bottom of https://github.com/isarandi/metrabs/blob/master/docs/INFERENCE_GUIDE.md)


Cannot use variables where one of the keys has a dash in it.

See the following test (which fails currently) for an example. Not sure if it's work-as-design or not.
Btw, awesome project!
public function testArrayWithDashInKeyDoesParseCorrectly()
    {
        $this->assertTrue($this->evaluate(
            'foo-bar === [
                "foo", // This is foo
                "bar"  // And this is bar
            ]',
            ['foo-bar' => ['foo', 'bar']]
        ));
    }

If that's any help, I just tried the same test case but with an underscore and it does pass the test.
public function testArrayWithUnderscoreInKeyDoesParseCorrectly()
    {
        $this->assertTrue($this->evaluate(
            'foo_bar === [
                "foo", // This is foo
                "bar"  // And this is bar
            ]',
            ['foo_bar' => ['foo', 'bar']]
        ));
    }

Hi @Fuitad!
Thanks for your submission. This is actually by design. I tried to keep it as close to Javascript syntax as possible. Plus, if I plan on adding math equations in the future, the dash would be interpreted as minus (foo minus bar). I'm not sure if I'll ever implement this, but it crossed my mind.
However, if you're using the latest version (0.4.0 as of writing this), you can actually redefine tokens in user-land. This would allow you to do what you want.
Eg:
$rule->registerToken(Tokenizer::TOKEN_VARIABLE, '[a-zA-Z_][\w-]*');

I hope this helps!

@nicoSWD Is this no longer possible? Using 0.6.1 and there is no longer public access to the tokenizer, and all properties are private so I can't extend Rule and add a method to access from the container.

@cnanney hi! It's still possible, but the approach has changed. It's now preferred to use composition instead of inheritance.
Meaning, you'd have to create your own grammar (akin to this) and construct your own container using said grammar like this.
Rule.php is extendable so you should be able to override the constructor.


LDAP server entry 'duplicates' itself

Version: Gluu Identity Appliance 3.1.0.Final
Hi,
I'm trying to configure Gluu to use an LDAP source. I've used "Add source LDAP server" to add a new server entry on the "Configuration -> Manage Authentication" page, but it's lead to some weirdness.
Here's what my page has ended up looking like:

So the bottom entry is the one that Gluu ships with, but I haven't explicitly edited the title of it. Instead, any time I update my entry that I've added (the top one), it changes the title of the bottom one. In addition, there doesn't appear to be a way to remove my custom entry or the internal one.
What is the correct way to add an external LDAP authentication source? I've noticed there's also an area to configure LDAP servers in "Configuration -> Cache Refresh". Any hints would be appreciated.

Please use our support portal for any kind of assistance regarding Gluu server.


Animation finished event [Enhancement]

With my app the menu frequently appear over content (visually confusing) so my workaround is to fade the main content (opacity: 20%, say) while the menu is active via main-action().
This works great for the menu open, but ideally I'd like a way to un-fade the content after  the menu close animation is complete.  Something like an onAnimationFinished event, perhaps (though a full suite of onMenuOpenStart/ onMenuOpenFinished/ onMenuClosedStart/ onMenuClosedFinished events would be even cooler)
Thanks for a great widget.

Hi, that might actually be an interesting enhancement.
Anyway I think that with little trickery you can already achieve what you're after and have a smooth effect. If you conditionally apply a class like fadeOut to the whole element and attach it to the menu state you can easily control the opacity of the whole menu. Moreover you can decide that when closing the menu the fading is applied only after a certain delay.
For example:

Set the element to something like this:

<nav mfb-menu class="app-menu" menu-state="menuState"
     ng-class="{'fadeOut' : menuState === 'closed'}">

in your stylesheets:

.app-menu{
  opacity: 1;
  transition: opacity .9s; /* the menu fades IN at this speed */
}
.fadeOut{
  opacity: 0.3;
  transition: opacity .8s; /* the menu fades in OUT this speed */
  transition-delay: .5s; /* delay fading out the menu as much as we need*/
}

I made a quick Plunkr for this. Hope this helps.

After re-reading your issue I noticed I probably got your problem mixed up. You don't want to fade the menu, but the content of the page, is that right?
Anyway, with little modification with what outlined above you can probably tackle the situation the same way. Updated Plunkr here.
Let me know if this is what you're after.

Thanks for the prompt reply.

You don't want to fade the menu, but the content of the page, is that right?

Yes, absolutely.  (Though I'm open to suggestions)

Let me know if this is what you're after.

Too be honest, I'm not too sure - the animation delays are too long for me to discern when the transitions start/end. That plunkr might be doing the right thing in principle, but (for example) when the menu disappears there's an awkward delay before the content unfades.
The specific thing I think I want is
a) menu open: main body content fades immediately
b) menu close: main body content fades at the end of the menu animation
But these are just my specific requirement - I think something along the lines of the onMenuXXXXX events might handle many use-cases.
Thanks, and keep up the excellent work.


Fix file extension typo in CONTRIBUTING.md

Add CONTRIBUTING.MD instead of..

should be:

Add CONTRIBUTING.md instead of

File extensions are conventionally written in small.

@sonalkr132 I guess it is right https://github.com/NIT-dgp/chrome-search-extension/blob/master/CONTRIBUTING.md
is there any other problem you see

your commits in the PR should be of the form

should be:

Your commits in the PR should be of the form

..

where you make the change commit message

I couldn't understand what was intended here :point_up: .

you have independence to explain everything normally
there.

normally is extraneous.


Hugo server doesn't specify content-type and encoding of css files

When using font-awesome with sass, it generates some unicode characters in the resulting css file.  When hugo serves up the file with its server, it doesn't specify a charset.  I might be nice if for css files Hugo's server feature would specify Content-Type: text/css; charset=utf8 or something like that.

Sorry, I was confused, ignore this.


Bug: slideClick events not firing

we tried to add the link to any web page different by slide..
but is not working..
we code 👍
what is wrong ?
what additional file is required ?
we add the @slideClick="handleSlideClick"  in  <slider..  but  this not work

<!DOCTYPE html>

  <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  <title>Vue Carousel - Navigation example</title>
  <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  <meta name="robots" content="noindex, nofollow">
  <meta name="googlebot" content="noindex, nofollow">
  <meta name="viewport" content="width=device-width, initial-scale=1">

@hdkcreative Hi, I achieved that see below this code
<carousel   :perPage="1" :navigationEnabled="isArrowToggle" :rtl="false"
:scrollPerPage="true" :autoplayTimeout="5000" :autoplay="true" :mouse-drag="mouseDrag"
:loop="true"  @pageChange="onPageChange" >
To add @pagechange event on carousel and in JS side:
onPageChange: function (val) {
var currentIndex = val + 1; // Alerts counter with respect to total count, Ex. (1 of 4),(2 of 4)
}
val is value of currentIndex position number so you can use as page number
Let me know if any quries

Hey @hdkcreative, just test and it seems there is a bug with slideClick at the moment. Based on the issues we usually get, it's not a hugely used feature and we're missing tests for it so not 100% when it was introduced. I'll switch this to a bug fix request, and see if we can get it fixed
In the meantime, does @pratiksingh05 solution help you out?

@quinnlangille Yes! it'll work for me so definitely it'll work if he implements.

hi @pratiksingh05 pratiksingh05 , you can add your code a fiddle or codepen for example please ?

Hey @hdkcreative, just test and it seems there is a bug with slideClick at the moment. Based on the issues we usually get, it's not a hugely used feature and we're missing tests for it so not 100% when it was introduced. I'll switch this to a bug fix request, and see if we can get it fixed
In the meantime, does @pratiksingh05 solution help you out?

thank you @quinnlangille , we  can wait for this ! :)

I am unable to reproduce this error using the above code. The events are working as intended for me. I have tested in Chrome, Firefox and Edge on a Windows 10 machine.
Can someone provide a fiddle with an example?

(hi @ERPedersen , thank you for response, you can check the code in
)
wait your response please ;)

All right, works with latest vue if we rename the event slideclick instead of slideClick. Might be related to #310

Any update on this issue ? Tried today but couldn't make slideClick work.

Any update on this issue ? Tried today but couldn't make slideClick work.

No, it worked for me if you see above my example that will work for you as well
Just add within the carousel block of code :-

// Here you add your div which you want to show

Note:- Nevertheless It'd be default behavior of slide to show both the side for slide and just need to use a click event of the side arrow to make sure your page will change and get reflected in your screen
Need to do:- :scrollPerPage="true" and :loop="true" on carousel tag
Let me know if it's not work.

Thanks for your answer @pratiksingh05 , I managed to solve this with a local stored fork renaming slideClick to slideclick as suggested in #310.

Hey @sebastienbarbier @pratiksingh05, the difference is likely related to the projects vue version. I'm not 100% on this because their docs are fuzzy here, but I'm pretty sure at some point they deprecated camel and snake case event naming.
I can confirm that on my local slideClick does not work and slideclick does. There is a PR up to resolve this in the docs/event name to make the development experience around this better. Will ping here once it's released!

Does this problem solved? Because it seems I still have the problem even though I change slideClick  to slideclick  ? I try to use the https://codepen.io/hdkcreative/pen/wxrLop from above, and change the @slideClick to  @slideclick and nothing happen when I sliding through the carousel, or press the prev next button

Does this problem solved? Because it seems I still have the problem even though I change slideClick  to slideclick  ? I try to use the https://codepen.io/hdkcreative/pen/wxrLop from above, and change the @slideClick to @slideclick and nothing happen when I sliding through the carousel, or press the prev next button

You should use slideclick in the  tag.
Like this:
slide.screen(v-for="(img, index) in images" :key="index" @slideclick="openScreen(img)")

This seems to be totally resolved based on a: the number of answers here and , b: not hearing back from anyone. Will close now, thanks for your help guys!

Camel-case doesn't work for props. The Documentation needs to be updated.
use the following:
navigation-enabled  instead of navigationEnabled
navigation-next-label instead of navigationNextLabel
navigation-prev-label instead of navigationPrevLabel


Enhance SelectionOperatorUtils/Service to allow rows with type compatible data schema to merge.

Solve the problem of segments with not identical but type compatible data schema cannot merge.
All numbers are type compatible, but not type compatible with other types.
For selection order-by with limit 0, use SelectionOnly operator to improve performance.
Optimize the logic of selection only (data schema should be the same as selection columns).
Fix the bug of throwing exception when order by the same column multiple times.
Fix the behavior of swallow merge exceptions in server side.

Current coverage is 64.47% (diff: 75.59%)

Merging #902 into master will increase coverage by 0.10%

@@             master       #902   diff @@
==========================================
  Files           735        735
  Lines         34866      34901    +35
  Methods           0          0
  Messages          0          0
  Branches       5309       5319    +10
==========================================
+ Hits          22442      22502    +60
+ Misses        10854      10815    -39
- Partials       1570       1584    +14

Powered by Codecov. Last update 86f7e85...a607fe8


Not Proxying Connection

Heya,
Haven't had much luck getting this going, just sits and doesn't proxy the connection? Everything seems to be installed correct, certs in the correct store, and hosts pointing to the current IP of the server.
[2021-07-20T14:54:09.086] [INFO] default - [PROXY] Using pfx file ./gg-struggle-cert.pfx
[2021-07-20T14:54:09.087] [INFO] default - [DB] Loading entries C:\Users\shatt\AppData\Local\Temp/gg-struggle/gg-struggle.db
[2021-07-20T14:54:09.105] [INFO] default - [PROXY] Listening on 443
[2021-07-20T14:54:09.106] [INFO] default -

Upgraded to 2.0, exact same issue. No errors, running fine, just cant connect to servers in game. Updated hosts file to whatever the latest IP is of the servers.

For 2.0, there only needs to be the entry <IP_ADDRESS> ggst-game.guiltygear.com. The latest server ip addresses are dynamically resolved at runtime

So this is all that's in my hosts now:
<IP_ADDRESS>	ggst-game.guiltygear.com

Still getting No Connection and no output to the terminal, looks like its sitting waiting for connections but never gets any?

Do you have multiple network devices? Just a guess but maybe it's running on a specific network interface

I uhhhh, may have a few....
Disabled them all and sans actual NIC and its workin! THANK YOU!!!

Nice! closing because solution above


Expand / collapse nodes programatically

Please add more methods like: expand all, collapse all, collapse X, expand X...

These methods are really needed!!

Proposed methods, to be confirmed

expandNode(id)
expandAll(level)
collapseNode(id)
collapseAll(level)

level - would allow expanding and collapsing to a certain tree level

if "level" would be "-1" would it mean "all levels"?
Alternatively you could use "expandTo(level)" and "collapseTo(level)" and an additional "expandAll()" and "collapseAll()".

I had thought of having expandTo(level) as well but my intention (admittedly unclear in previous comment) was to make level optional.  If level was omitted then it would expand to all levels, or alternatively you could provide level = 99.
I'm also toying with the idea of adding the optional level argument to expand and collapse node methods; to provide more fine-grained control.

Please don't limit levels to 99 and make 99 a special number - you never know how people are using it. Either omit the parameter for "all" or use -1 as negative values are forbidden by semantics.
Could you please elaborate on the idea on adding a level argument to expand and collapse - I don't know what you're aiming towards...

I didn't say 99 would be a special number or a limit, just like I didn't suggest -1.  It was merely an example that you could expand to n levels, to effectively expand all levels.
Using level in expand and collapse nodes allows you to expand/collapse any given node of the tree by n levels.  Essentially expanding a single branch rather than the entire tree.  The default would be 1 level, so if omitted it would act as expected and only open the immediate node.

Okay, got your point. But having some possibility to really collapse all (either with a special method or with a constant) might be helpful - both for the user and for implementation since no level determination and/or recursion needs to be done.
I now get your idea on expand and collapse on any given node. So it is a "relative" expansion from the current node. I think this is a very good idea.


[Tasks] VMAS MPE (Vectorized)

This PR adds the MPE scenarios scenarios from VMAS.
So now users can choose either task=pettingzoo/simple_... or task=pettingzoo/vmas_...
cc #41 @shivarajabi @SashaBoguraev

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (75bc507) 89.51% compared to head (5f73e13) 46.52%.

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #42       +/-   ##
===========================================
- Coverage   89.51%   46.52%   -43.00%
===========================================
  Files          56       64        +8
  Lines        2576     2646       +70
===========================================
- Hits         2306     1231     -1075
- Misses        270     1415     +1145

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


[9.x] remove useless if statement from SupportReflectorTest

because of Laravel9 min PHP requirement is 8 and above , this if statement is useless
if (PHP_MAJOR_VERSION >= 8) {
    eval('
namespace Illuminate\Tests\Support;

Thanks!


.keys and .values returns Seq

The documentation for Map says that .keys,  .values, .kv, pairs return lists. They actually return Seq. Other methods, such as .antipairs, correctly notes that it returns a sequence. This extends into docs that rely on Map.

@lizmat maybe if you scroll through the thing you can notice other errors like this?