python312Packages.debugpy: 1.8.7 -> 1.8.8

https://github.com/microsoft/debugpy/releases/tag/v1.8.8
With this update, we also don't have to remove the pre-compiled binaries anymore because they were removed in https://github.com/microsoft/debugpy/pull/1723.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 354925 run on x86_64-linux 1

6 packages built:

home-assistant-component-tests.debugpy
python311Packages.debugpy
python311Packages.debugpy.dist
python312Packages.debugpy
python312Packages.debugpy.dist
vscode-extensions.ms-python.python


VHPIDIRECT: How to send/receive top level signals to DUT

First let me tell you my targeted intention with GHDL/co-sim: I want to do advanced verification of my large VHDL design in software with c++. So the idea is, I have the most top level synthesizeable design file: top.vhd, and there won't be any VHD testbench above it. That top.vhd will be written and read the data from its top pins by a software coroutine.
Alright so I went through a long route of building the gcc backend version of GHDL in Linux (Debian), ghdl is working.
My next thing is to run some starter example.
I went ahead and studied the shghdl example from this co-sim repo and decided that the proper elaboration route for me would be using gcc as entry point (Linking GHDL object files to C).
I do this:
ghdl -a tb.vhd
gcc main.c -o main -ldl
ghdl --bind tb
gcc test.c -shared -Wl,`ghdl --list-link tb` -Wl,--version-script=./test.ver -Wl,-u,ghdl_main -o tb.lib
./main

And I get the output:
Hello Something!
tb.vhd:7:5:@0ms:(report note): Hello shared/shghdl!
ghdl_main return: 0
tb.vhd:7:5:@0ms:(report note): Hello shared/shghdl!
ghdl_main return: 0
tb.vhd:7:5:@0ms:(report note): Hello shared/shghdl!
ghdl_main return: 0

Which is great, which means its all working. However, printing from top VHD file is cool, but now I need to start doing the real scenario which for me is: actually start driving data to the input pins of top.vhd and reading data from output pins of top.vhd (in this current simple example that file is tb.vhd, in my case I plan it to be non-testbench type synthesizeable vhdl file)
Can I have some hint on how to do it? would be nice to have similar to shghdl but more practical example.
So am I supposed to send/receive data through the ghdl_main function, or what is the proper mechanism?

anyway, it seems that for what I'm interested in, the VPI would be a way to go for this usage scenario.
feel free to close this issue if you agree.


tcp.client and tcp.client.persistent attempt TLS/SSL connection when disabled

When the Driver ID is set to ipcomm.tcp.client or ipcomm.tcp.client.persistent and TLS enable is not checked, the device still sends out a TLS Client Hello packet as if it is trying to connect via TLS. Device instance remains in the offline state

When the Driver ID is set to ipcomm.tcp.client or ipcomm.tcp.client.persistent and TLS enable is not checked, the device still sends out a TLS Client Hello packet as if it is trying to connect via TLS. Device instance remains in the offline state

There is a sslsocket.connect() test to see if TLS is required at the server but not enabled at the Driver configuration before the connection is "officially" established through the nio library.

Is it possible to disable that test as some devices that don't support TLS don't like receiving a bunch of "random" data?
What influences the offline / online state of a TCP client? The state indicator remains offline even when the server on theother end indicates it is connected to a client (The same client that sent the TLS data).

Is it possible to disable that test as some devices that don't support TLS don't like receiving a bunch of "random" data?
What influences the offline / online state of a TCP client? The state indicator remains offline even when the server on theother end indicates it is connected to a client (The same client that sent the TLS data).

I'll look at disabling the test but the underlying nio engine (I don't have access to SDK source code) did some really bad things when a TCP connection was attempted to a TLS required server - and vice versa.
The offline/online state depends on which variant you are using.  The driver handles that for you for the "persistent" variants but you are responsible for setting the parameter state from a script - check the github README or the at the Muse controller CLI (doc:list / doc:show ) for the Command: deviceonline documentation.

I noticed 2 things when the TLS Client Hello is sent:

The program on the device (I am running python) locks up completely
The device state remains offline (using the persistent version of the driver)
If I force the connection closed from the server, this driver will reopen it without sending the TLS client hello and normal functionality is restored, but it definitely seems like that TLS check is also doing bad things when the server doesn't support it

I noticed 2 things when the TLS Client Hello is sent:

The program on the device (I am running python) locks up completely
The device state remains offline (using the persistent version of the driver)
If I force the connection closed from the server, this driver will reopen it without sending the TLS client hello and normal functionality is restored, but it definitely seems like that TLS check is also doing bad things when the server doesn't support it

AMX-IPComm-1.0.3 has a change to the test where the TLS Client Hello should only be sent out when the TLS configuration property is TRUE.

Issue is resolved


Why isn't the certificate v2 issued?

Client version: <IP_ADDRESS>
Windows version: windows 2016 r2
Why isn't the certificate v2 issued?

I'll refer to #464 !


Could you build a qtum binary with musl?

Unfortunately I have an Alpine Linux (Postmarketos) running phone  that does not use/have glibc so I can't use the ARM 32bit binaries. Could you provide qtum ARM binaries compiled with musl too? Thanks.

We have no plans of this in the near future, but welcome contributors who would like to undertake this project. Our build system (gitian) makes using a different libc a huge ordeal.


Add Ballerina to Run with Arg doc

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

https://github.com/wso2-enterprise/choreo/issues/28713
Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Stoypath - Scion 2e: Drag & Drop functionality, Minor bug fixes, Update to K-scaffold

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
Bug Fixes

Completing a deed caused display issues for other deeds
Purviews were missing Innate Power field
Approach Names were incorrect

New Features

Drag and drop support for upcoming compendium products

Infrastructure changes

The sheet's infrastructure was updated to use the K-scaffold v0.3

Character Sheet Info Roll20 Internal Use only.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/38 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit 3f80c04:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-09-20-10-45-12 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-09-20-10-45-12

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Allow maximum job execution time to be configured

Previously Bacalhau had some hard-coded values for maximum job time. Having a maximum timeout makes sense on a public network where non-paying users should not be able to hog capacity. For private clusters, the behaviour should be configurable – users may wish to run jobs that span many days.
In this commit we provide a new config flag --max-timeout respected by bacalhau serve that allows these maximum job times to be set by a node operator. It also changes the default when the flag is unused to jobs having no maximum execution time.
The behaviour of the Bacalhau demo network is unchanged.
The timeout value has been changed from a floating point number of seconds to an integer number of seconds. There are no instances of anyone submitting jobs with fractional seconds on the public network, so this is a backwards compatible change.
Resolves #2561.

Happy to continue discussions above and open a further PR if we decide any more changes are needed 👍🏻

great to see this feature added - thank you @simonwo & team!


Add support for git dependencies.

NOTE: implementation was changed drastically, see comments below
Because this is highly experimental, this needs to be enabled in the config with allowGitDependencies = true.
The relevant commit for git dependencies: https://github.com/nmattia/naersk/commit/ea1555099f9f4a9f75309d11ec12e4b455a054c5
The other commits are preparation cleanup.
This works by first fetching the git dependencies with fetchGit, and then patching the Cargo.toml. A few notes:

the .cargo/config paths does not work, because cargo doesn't register it until after it has tried (and failed) to checkout the deps from the server
the replace mechanism of Cargo.tomls doesn't work, same reason as above
mimicking the $CARGO_HOME/git/{db,checkout} seems fairly simple (the format is {db,checkout}/<crate>-<siphash_of_url>/... but it turns out cargo (libgit2 really) expects a full checkout.

CC @basvandijk @Profpatsch @curiousleo

Found a much simpler, much more robust way of implementing this.

@basvandijk occurred to me that I could just copy what cargo vendor does; not sure why I didn't think about it the other day.

Can we upstream some of this so it reduces complexity for us in the future?

@profpatsch upstream where? Naersk or the git deps?

Upstream to cargo.


[WIP][BEAM-10670] Make SparkRunner opt-out for using an SDF powered Read transform.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Spark Runner Nexmark Tests

Run Spark ValidatesRunner

Run Spark StructuredStreaming ValidatesRunner

CC: @iemejia I got this passing all the tests but what is the state of streaming pipelines? I see that there is code for evaluating UnboundedSources but don't see any test coverage.

Run Spark Runner Nexmark Tests

cc: @ibzib @annaqin418

@iemejia I have updated the code and added a SparkProcessedKeyedElements using updateStateByKey to evaluate a splittable DoFn. I based the logic off of the SparkGroupAlsoByWindowViaWindowSet logic. I also added some SplittableDoFnTests and have been using org.apache.beam.runners.spark.translation.streaming.SplittableDoFnTest#testPairWithIndexBasicUnbounded as my base test to get it working. So far I am able to get output produced including having the splittable DoFn state saved and restored for the next round of execution. I do see most of output via the LoggingDoFn that I added to the test but the PAssert is failing because it doesn't see any of the output and it is also triggering too early as not all the output has been produced. Any suggestions as to what to take a look at?

@iemejia I figured out that the issue is that watermark holds aren't implemented for spark so the first batch completes which computes new watermarks so the watermark hold that was set by the splittable dofn implementation is ignored. This leads to timers being dropped and hence only some of the results being produced.
This is likely the cause for why the PAssert is dropping the elements that were produced as well but I haven't validated this yet.

The phenomenon of microbatches producing results early I noticed it too in the past when trying to enable the Read.Unbounded tests. I could not understand why, and I thought it was probably due to some glitch in Spark implementation or us screwing their scheduling but I struggled to debug the issue properly.

Since watermark holds don't seem to be implemented, does the GroupAlsoViaWindowSet hold back the watermark for elements that it currently has buffered?

Probably, at least that may explain some of the inconsistencies.

Can you explain how the GlobalWatermarkHolder works, can I register anything as a sourceId?

In all honesty I am not so familiar with watermark handling on the Spark runner. I took a look at the GlobalWatermarkHolder class and tried to figure out but it was not really evident.
My impression is that the sourceId is aligned somehow with Spark's assigned streamId, but I might be misinterpreting it.
https://github.com/apache/spark/blob/13664434387e338a5029e73a4388943f34e3fc07/streaming/src/main/scala/org/apache/spark/streaming/scheduler/InputInfoTracker.scala#L30
I wish I could help more but that part of the code is also not so well documented. I doubt that the original authors of the code still remember the details but maybe they remember at least the intentions of GlobalWatermarkHolder and its use, and maybe if there were any open issues. Just in case :crossed_fingers: maybe: @amitsela @aviemzur @staslev

The phenomenon of microbatches producing results early I noticed it too in the past when trying to enable the Read.Unbounded tests. I could not understand why, and I thought it was probably due to some glitch in Spark implementation or us screwing their scheduling but I struggled to debug the issue properly.

Since watermark holds don't seem to be implemented, does the GroupAlsoViaWindowSet hold back the watermark for elements that it currently has buffered?

Probably, at least that may explain some of the inconsistencies.

The Java based trigger implementation relies on this to produce correct results.

Can you explain how the GlobalWatermarkHolder works, can I register anything as a sourceId?

In all honesty I am not so familiar with watermark handling on the Spark runner. I took a look at the GlobalWatermarkHolder class and tried to figure out but it was not really evident.
My impression is that the sourceId is aligned somehow with Spark's assigned streamId, but I might be misinterpreting it.
https://github.com/apache/spark/blob/13664434387e338a5029e73a4388943f34e3fc07/streaming/src/main/scala/org/apache/spark/streaming/scheduler/InputInfoTracker.scala#L30
I wish I could help more but that part of the code is also not so well documented. I doubt that the original authors of the code still remember the details but maybe they remember at least the intentions of GlobalWatermarkHolder and its use, and maybe if there were any open issues. Just in case 🤞 maybe: @amitsela @aviemzur @staslev

That would be great if someone could give guidance here.

@iemejia Since streaming is effectively broken due to lack of support for watermark holds. What do you think about enabling SDF for Spark and it only working in batch?
I'll see what I can do about watermark holds but decoupling the two would be convenient.

@iemejia Since streaming is effectively broken due to lack of support for watermark holds. What do you think about enabling SDF for Spark and it only working in batch?

Can you be more explicit on what you mean on 'it only working in batch'? Isn't it the current case for Bounded SDF?
Or what you mean is to have now Unbounded Reads translated using SDF still with the same level of functionality than the old translation (no holds)?

@iemejia Since streaming is effectively broken due to lack of support for watermark holds. What do you think about enabling SDF for Spark and it only working in batch?

Can you be more explicit on what you mean on 'it only working in batch'? Isn't it the current case for Bounded SDF?
Or what you mean is to have now Unbounded Reads translated using SDF still with the same level of functionality than the old translation (no holds)?

BoundedSource converted to an SDF will work just as well as the current BoundedSource implementation since they don't rely on watermark holds. The current implementation of UnboundedSource and the new implementation using UnboundedSource as an SDF both set the watermark but nothing honors it since there is lack of support for watermark holds.

I see, so it is the full switch from Read.Bounded/Unbounded to SDF by default. Can you get this one green so we can test it and then merge it, I would like to see if there is some perf impact, and probably that we document how to get the previous Unbounded translation in case any existing users find any difference.
If I understood correctly you might intend to tackle watermark holds in the 'future'? Just for learning curiosity I assume this will be done in SparkProcessKeyedElements for Gbk/Stateful translation, might this need some extra changes? asking just because I am reading the translation of Portable Streaming runner and I see watermarks are taken into account from Impulse so I was wondering if something was missing here or if this is done in a different place maybe in core.

I see, so it is the full switch from Read.Bounded/Unbounded to SDF by default. Can you get this one green so we can test it and then merge it, I would like to see if there is some perf impact, and probably that we document how to get the previous Unbounded translation in case any existing users find any difference.
If I understood correctly you might intend to tackle watermark holds in the 'future'? Just for learning curiosity I assume this will be done in SparkProcessKeyedElements for Gbk/Stateful translation, might this need some extra changes? asking just because I am reading the translation of Portable Streaming runner and I see watermarks are taken into account from Impulse so I was wondering if something was missing here or if this is done in a different place maybe in core.

I'll be going on paternity soon so I'll try to see what I can get working with the GlobalWatermarkHolder implementation that exists. I think we should be able to use arbitrary ids in it it just might be really slow since the readers/writers should really care about their upstream watermarks (main and side input) so having a global broadcast seems less then desirable.

Excellent idea to break the PR into little ones, first one merged. waiting for the next!


Helm upgrade failed suddenly. Start showing it has not deployed release and hr reconcile failed

When doing reconciliation of helm release, sometimes it starts showing "Helm upgrade failed: project has no deployed releases." It's happening once in 2-3 days.
Q: Is helm controller restarted?
A: No
kubectl -n flux-system get pods helm-controller-6b456768d5-mcwcc
NAME                               READY   STATUS    RESTARTS   AGE
helm-controller-6b456768d5-mcwcc   1/1     Running   0          2d5h

project = fe-stack
Error:
flux get hr fe-stack -n qa-team
NAME    	READY	MESSAGE                                                 	REVISION	SUSPENDED
fe-stack	False	Helm upgrade failed: "fe-stack" has no deployed releases	        	False

kubectl describe helmreleases.helm.toolkit.fluxcd.io fe-stack -n qa-team
Last Helm logs:

preparing upgrade for fe-stack
resetting values to the chart's original version
performing update for fe-stack
creating upgraded release for fe-stack
    Reason:                        UpgradeFailed
    Status:                        False
    Type:                          Released
  Failures:                        1
  Helm Chart:                      qa-team/qa-team-fe-stack
  Last Attempted Revision:         v5.0.27
  Last Attempted Values Checksum:  6a7c1ebdbb475c995c093799e1f3824926229e71
  Last Handled Reconcile At:       2021-05-07T17:54:54.795085788+05:30
  Observed Generation:             22
  Upgrade Failures:                1
Events:
  Type    Reason  Age                    From             Message
  ----    ------  ----                   ----             -------
  Normal  error   19m (x19 over 120m)    helm-controller  reconciliation failed: Helm upgrade failed: "fe-stack" has no deployed releases
  Normal  info    9m53s (x30 over 29h)   helm-controller  Helm upgrade has started
  Normal  error   9m47s (x21 over 120m)  helm-controller  Helm upgrade failed: "fe-stack" has no deployed releases

Logs with debug level:
hr-controller.log

Hi @stefanprodan this is a production blocker for us to go live with fluxV2. I can also contribute if that helps.
Thank You

I suspect this to be related to / another version of #149, with a rich history of helm users themselves running into it:
https://github.com/helm/helm/issues/5595
https://github.com/helm/helm/issues/7160
Judging on the shared logs, there seems to be a correlation with the --wait behavior from Helm, that seems to corrupt the storage in some edge cases.

It's happening a lot of time during development. Is there are any workaround without deleting the helmrelease?

Its a manual activity but this helped the stuck release: https://github.com/helm/helm/issues/5595#issuecomment-717024123


[StimulusBundle/TwigComponent] Fix "use_yield" to "true" deprecation notice in CI

Q
A

Bug fix?
yes

New feature?
no

Issues

License
MIT

Only this one remains (in StimulusBundle), but I can't find a direct implementation related to the Environment in that test. Maybe also related to https://github.com/symfony/symfony/pull/53805 or thoughts anyone?
1x: Since twig/twig 3.9.0: Not setting "use_yield" to "true" is deprecated.
    1x in StimulusControllerLoaderFunctionalTest::testFullApplicationLoad from Symfony\UX\StimulusBundle\Tests\AssetMapper

Thanks @evertharmeling! We'll keep debugging the rest after this. For the TwigComponents deprecation, more work is needed - and I know that Simon has looked into it https://github.com/symfony/ux/issues/1390#issuecomment-1902764551

Thanks Evert!


Ollama model download fails on kubernetes

What is the issue?
Im using the latest helmchart of ollama and deployed it to kubernetes. When i enter the pods shell and use "ollama pull ", it starts the download at fast pace (around 250Mbps) but suddenly drops to a very low rate when reaching the last GB of the download. Most of the time it ends with an error, that i dont understand. The Kubernetes side is fine my monitoring and network speed tests verified everything. Do you have an idea? Do you think this is caused by ollama itself? Never had this issue locally

OS
Docker
GPU
Nvidia
CPU
Other
Ollama version
0.33.0

Hi @samyIO would it be possible to upgrade to 0.1.42 - this should be fixed. Let me know if that's not the case. Thanks for the issue!


fix: header method is undefined

Fix typo that led to undefined method error

It happens!

Shipped as 1.0.2


Release 3.2.2

This release is meant to deliver the updates for the ReactDOM update effort, and resolve the issue with CGO in the 3.2.1 release.

Released with https://github.com/mattermost/mattermost-marketplace/pull/317


[Feture Request] Force touch action

Description
Include a force touch action where the user can trigger a new entry inclusion
Motivation
The user would be able to open directly in the new entry view and perform the most common use of the app that is a daily input of data. This can also be used, on a late point, for actions from watch and/or extension

Hi @diogot can I work on this issue? I really like the app and force touch new entry inclusion would be nice to have.

Hi @DominikSecka, great!
I think that an action that opens the app with the "add" screen open would be a nice start.


Revert "[examples] Update create-react-app examples with styled-components to use package aliases"

Reverts mui-org/material-ui#27591
npm does not work with the current setup. Closes https://github.com/mui-org/material-ui/issues/27846
Alternativly we can specify in the project's readme that only yarn is supported at the moment.

Also need to revert the changes in the nextjs with styled-components example. (or again specify in the readme that only yarn is supported.
For nextjs I would vote for adding in the readme to use yarn as those changes also fixed the class name mismatch error.

How was it tested that resolutions works with npm?

I've never tested with npm. I assumed it worked based on https://github.com/mui-org/material-ui/pull/27088#issuecomment-885278495, but testing now the nextjs example it doesn't work as well. Maybe it was not tested from clean project and emotion was installed already.

For nextjs @eps1lon do you think it's ok if we note that it works only with yarn, as I remember that for some reason those changes fixed the class name mismatch error too.

For nextjs @eps1lon do you think it's ok if we note that it works only with yarn, as I remember that for some reason those changes fixed the class name mismatch error too.

This is not really viable. They are probably used equally often so not supporting either one is not really an option.


Ticker is active after leave page

@override
dispose() {
_iconAnimationCtrl.dispose(); // you need this
super.dispose();
}

Fixed in 85d7dd5c2e54d94998875d3e0cd6de766977fdaa. Thanks.


chore: update release_created to releases_created in README.md

I followed the instruction to create a release workflow and then I found a issue from the example in README.md.
In the example workflow, the outputs from the release-please-action to determine a release is created or not should be releases_created and not release_created.

Ran into this as well. The typo for the Output section was fixed in #626, but the examples were not updated.

Apologies for the confusion, there are 2 separate outputs: releases_created and release_created. The former (releases_created) will be true if any release was created (e.g. for a monorepo). The later release_created indicates if the "root" component had a release created or not.
The example use case is for a single component repo and so release_created should work (although releases_created should also work).


.rules function only adding rule to first element selected

jquery-validate 1.15.0
Chrome 52
$('[name_="FirstName"], [name_="LastName"], [name*="Message"]', form).rules('add', { required: true });
ought to make all elements with FirstName, LastName, or Message in the name required. Instead it just makes the first element that matches the selector required.
I was able to work around the bug by altering the code to:
$('[name_="FirstName"]', form).rules('add', { required: true });
$('[name_="LastName"]', form).rules('add', { required: true });
$('[name*="Message"]', form).rules('add', { required: true });
To test my theory, I also tried:
$('[name_="Name"]', form).rules('add', { required: true });
$('[name_="Message"]', form).rules('add', { required: true });
As I thought, that only selected one of the Name fields.

you should use .each as this is expected behavour.


[Query] How to show/hide Node Handle ports?

Is there a way to show/hide the Node Handle ports programmatically? Please see the illustration.

Use  component, and set custom styling, or use react_flow global css to override their style, then use opacity: 0 to hide them.


Batch clone

First of all thank you so much for this amazing program!
I am trying to run DALES dataset using the code from train_S3DIS.py, I was following steps from #72 but I reached an issue. while running the program, I am having trouble with function def forward(self, batch, config):
Specifically the line: x = batch.features.clone().detach()
The batch created from the code using DALES dataset creates a batch with these info:

where features is not formatted the same way in S3DIS dataset. Then I get an error that is originating from blocks.py line 361:
x = torch.cat((x, torch.zeros_like(x[:1, :])), 0)

I get this error:
x = torch.cat((x, torch.zeros_like(x[:1, :])), 0)
IndexError: too many indices for tensor of dimension 1

Now I am not sure why I am getting features to be a different tensor than when I run it on S3DIS dataset. If you can help me @HuguesTHOMAS I'd really appreciate it.

Can you make sure your feature matrix batch.features has 2 dimension (N, D) where N is the number of points and D the dimension of the features.
In your case D should be equal to 1, but the feature dimension should still be (N,1) and not (N,)


Pin versions of packages

We need to ping all unpinned versions of packages in setup.py & workspace environments.

@alexey-gruzdev is this about decreasing dependency solving time during installation? Pinning versions of packages makes it harder to integrate other frameworks with OpenFL. Package conflicts have already come up during previous integration efforts (MLCube, Frontier Development Lab - CRISP framework), so unless there is a fixed security vulnerability or API that is tied to a particular subpackage, we should try to stay as flexible as possible

Initially, we meant to solve an envoy environment degradation (a lot of experiment-specific packages with their dependencies installed one by one). I would close this with opening another issue.


PR Review Document

PR Review doc to document review process (bring PRs to a project, next step: assign reviewers, track its completion) and review guidelines (how to review a PR for quality, code style, etc.)
Comments
All team leads to contribute process/guidelines to this document

This Issue will be taken up in cQube v5.0

This is not considered in this current version .

Add in the Roadmap


GraphViz: add charset option to base64 image string.

Thanks for filing this PR!
However, the motivation is a bit unclear to me, so perhaps you can elaborate? When is this needed? When would one change the charset attribute?

Browser will decode the base64 string with the default browser charset, which often is iso-8859-1.
It may not be the case that the svg file is encoded with that encoding. This change allows you to specify the encoding for the browser.

Thanks, I understand :-) Can you provide a reproducible test case for this?

use Fhaculty\Graph\Graph;
use Graphp\GraphViz\GraphViz;

$graph = new Graph();
$graph->setAttribute('charset', 'UTF-8');

$modelvertex = $graph->createVertex("üuu");
$modelvertex->setAttribute('graphviz.color', 'blue');
$modeltypevertex = $graph->createVertex("äaa");
$edge = $modelvertex->createEdgeTo($modeltypevertex);
$edge->setAttribute('graphviz.color', 'grey');

$graphviz = new GraphViz();
$graphviz->setFormat("svg");
echo $graphviz->createImageHtml($graph);

This is an example which shows encoding problems with german umlauts.

Thanks, I'll play around with this a bit :+1:
At first sight, I'm tempted to promote (enforce) UTF-8 encoding here and remove the ctor argument, as I see little value in providing any other encoding.
I'll (try to) get back to this soon-ish.

@Ithilias Again thank you for filing this PR and for your patience!
I agree that this feature makes perfect sense and have just merged a slightly modified version with #27 (with proper attribution of course) :shipit: :tada:
I've added some documentation to emphasize that UTF-8 is the default. I've removed the constructor argument, one can now use the the charset attribute to change to a different encoding instead, but this is usually not recommended.
Again, thank you! :+1:


Possible spam ability usage when game nears conclusion

If there is a way to tell when the game has reached the last hour from the API (or by grabbing the time run so far) then maybe implement a spam of all abilities (minus nukes, raining gold, and all others automated to activate on certain levels).
Ones that would be useful are max elemental damage, cripple monster, decrease cooldowns (if this hasn't already been implemented to be used before nukes and such), throw money at screen, etc.
If suggestion will be considered seriously, then I will compile list of optimal abilities to spam.

Possible workaround at https://github.com/SteamDatabase/steamSummerMinigame/pull/117

Instead of using the Minigame API to get the reset hour/game time, just check if the current time is 30 minutes away from the reset time.

That's exactly what I did.
It checks the GMT time for (15) and then check the time left based on what the user has set.

This has been implemented.


how can i create a app without database connection ?

Is possible ?

Hi @luizamboni Thanks for trying out Kemalyst.
I have joined forces with the Amber team and we have moved most of the functionality of Kemalyst over to that project.  You will find it quite similar to Kemalyst but has improvements on the routing in that Amber supports pipes.
Regarding the app without database, we currently do not have that option for the kgen init app {name} but you can remove the shards for the db, kemalyst-model and config/database.yml manually.  There is no dependencies on that.  You can use crecto or other ORM if you wish.

i created with sqlite and not used this. Only to not need connect with a database.
I m consumming a rest api, not need persistency


[Dev Tools] Switch to Ava from tap, add Wallaby config

I've switched the test runner to ava, which has similar ergonomics to tap,
but runs everything in parallel.  It also supports integration with Wallaby for a
great test writing environment in the editor.
Test Plan:

[x] Unit tests pass!

All totally fair feedback! I haven't used tap in a very long time, so perhaps I made some more ignorant assumptions. Either way, test speed is not a problem for this repo :)
I use Wallaby for TDD development and it's honestly kinda life-changing with realtime feedback, code coverage, and time traveling debugging. Wallaby unfortunately doesn't support tap, but ava is supported and has a similar API.  The config file is required because ava isn't one of the test frameworks that isn't automatically detected and configured by Wallaby.
Totally okay with skipping this PR if you feel strongly about it!
Just figured I did the work to debug and test the bugfix with this, so I might as well put it up for review :)

I understand ava and it LGTM
What I don't agree with is the Wallaby - it is an opinionated tool. You can use it locally by adding it to the gitignore, but I don't want to push/advertise it through this repo

I understand ava and it LGTM
What I don't agree with is the Wallaby - it is an opinionated tool. You can use it locally by adding it to the gitignore, but I don't want to push/advertise it through this repo

Done!


White stripe after footer fix

issue #19755

Thank you for this fix, @Arhell 🎉
/lgtm

@Arhell / @onlydole - can you explain what these changes do and why these changes are the right fix?
(if you can provide this kind of information in the PR description, it will mean that reviewing takes less time - and the smaller the amount of time a review takes, the more chance that reviewers / approvers can have of finding a slice of time big enough).

/lgtm remove
/assign @sftim

/lgtm cancel

Oh sure
Scroll down, after the footer a white stripe.
https://kubernetes.io/
This PR solves this problem.
https://deploy-preview-19825--kubernetes-io-master-staging.netlify.com/

@Arhell, some more questions

do I need to use particular software to verify this fix?
what have you already done to test it?
how can we be sure this is the right way to fix the issue?

Hello! @sftim

Found this problem in chrome.
Checked on the desktop + mobile phone
I looked at what gave such an effect, the styles built into the browser, I reset them.

/assign @kbhawkey

@sftim and @kbhawkey - is there anything more you'd like to see in this PR from @Arhell, or would you prefer it be closed for now?

Really I'm (still) not sure how to be confident that this fix makes the site better / that it's the right way to fix the symptom. A more thorough explanation would help there I think.
(CSS is a bit like that).

Hi. I only see a very small white strip on the page, between the footer and the end of the browser window.
It is very difficult to see the problem when viewing the docs from firefox (strip is tiny).
A very narrow strip is evident on chrome. Here is a screen capture:

Perhaps the problem is more obvious on a mobile browser (I viewed on my desktop).
I think we could defer to fix this for now.

Apologies for the long back and forth, @Arhell! We’re going to close this PR for now and we may revisit later after we update to Docsy and implement some other UI/UX changes.
/close


Correctly handle scenarios with transformed stubs

Revises SnapshotStubMappingPostProcessor to run stub transformers first.  That way, if the transformer results in StubMapping requests being equal they are properly
deduplicated or turned into scenarios.
Cleans up process method to make clearer the three phases of processing.
Additionally:

Converts SnapshotStubMappingPostProcessorTest.TEST_STUB_MAPPINGS to instance variable to avoid sharing mutable state between test cases which caused the suite to be order dependent.

Fixes: #2137

Hey @tomakehurst gentle bump on this.  This issue has my team maintaining a kludgy internal fork of Wiremock, I'd love to get this into the main repo :).

Can we fix the build errors also? Then I'll take a look.

@tomakehurst AFAICT there are codestyle errors only.

@tomakehurst AFAICT there are codestyle errors only.

Merging did actually result in introducing a bug, I've resolved it.  All tests are passing now, and Spotless has been reapplied so this one should be good to go too.

Thanks @gsmith85 , and sorry for messing up the merge

@tomakehurst @oleg-nenashev are we a go here?  Anything else you need form me to get this merged?

@tomakehurst I believe I've addressed your comments, appreciate the guidance on this one.  Let me know if you need any other tweaks and I can be more responsive to help get this pushed over the line.

Thanks @gsmith85 !


API: Out-of-Session Issue Returned

It appears that the API:OOS issue has returned --yes, I'm up to date (jan 4 commit).
My account was banned (I knew this was a possibility) on top of this, so I'm going to assume that O'Reilly is starting to take action over this method.

Could you add some information?

Hi Guys,
Same over here, this is the output:
Traceback (most recent call last):   File "/usr/lib/python3.6/multiprocessing/process.py", line 258, in _bootstrap     self.run()   File "/usr/lib/python3.6/multiprocessing/process.py", line 93, in run     self._target(*self._args, **self._kwargs)   File "safaribooks.py", line 810, in _thread_download_images     for chunk in response.iter_content(1024): AttributeError: 'int' object has no attribute 'iter_content' [#] HTTPSConnectionPool(host='learning.oreilly.com', port=443): Max retries exceeded with url:

Anyone can give any help about this problem? I'm having this one too
`[-] Logging into Safari Books Online...
[*] Retrieving book info...
[#] API: Out-of-Session (You do not have permission to perform this action.).
[+] Please delete all the <BOOK NAME>/OEBPS/*.xhtml files and restart the program.
[!] Aborting...                   `

I think that we can say this project was nice but now RIP, not sure there is a workaround to this.

Just run once providing your credentials, this will generate the cookie.json file. Then run again without your credentials. It worked for me.

I have the very latest commit and I am getting this error again.
-] Logging into Safari Books Online...
[*] Retrieving book info...
[#] API: Out-of-Session (You do not have permission to perform this action.).
[+] Please delete all the <BOOK NAME>/OEBPS/*.xhtml files and restart the program.
[!] Aborting...

I have the very latest commit and I am getting this error again.
-] Logging into Safari Books Online...
[*] Retrieving book info...
[#] API: Out-of-Session (You do not have permission to perform this action.).
[+] Please delete all the <BOOK NAME>/OEBPS/*.xhtml files and restart the program.
[!] Aborting...
I tried to manually create the cookies.json file in the package where the script is, using the format provided here and used the code to fetch the values from my looged in browser for the values needed in the cookies file.
There was one that I couldnt find, corp_sessionid, except that I found everything replaced it with correct values and re-ran the script.
Failed with the same error as above.

i noticed that there's a groot_sessionid is placed. Anyone noticed this?

Same problem.
Has anyone solved this problem yet?

Same issue unfortunately.

Same issue here

same issue here

same issue


what if i'm a kid and i don't love the taste of cinnamon toast crunch

Oh..
Oh dear.
We shall have to update this repository immediately!


Open NSStreams and bind them to TCP socket

Starting implementation.

Hoping implementation will be done by tomorrow. And start testing.

PR should go out today.

Not yet ready for PR yet. hopefully tomorrow.

PR is out.

We need to update this code to adopt the new approach to handling thread safety.

The code and tests are finished but we are waiting to submit a PR until we get Alex's code accepted and then we will rebase this code onto Alex's code.

This is blocked on thaliproject/Thali_CordovaPlugin#880. Once Alex pushes the new version then Dersim will rebase on it and they will review each other's changes as a result of combining together.

Alex's story is merged.
I'm working on rebasing my changes.

Temporarily blocked on #881.


Use the reference to the dedicated page about service discovery in th…

…e site front page
.. because it covers both DNS and ZooKeeper

Will merge once build passes since it's trivial.


Updated UXUI-206 to exist inside of /holidayextras/ui-toolkit/tree/ma…

Updated UXUI-206 to exist inside of /holidayextras/ui-toolkit/tree/master/src/components
Was not able to get Unit Tests to work until told by Jack that Node v0.12.0 had issues with Jest.  Was able to downgrade to v0.10.21 and that got Jest working.  None of the v0.11's I tried worked, so make sure you do not upgrade Node if you are on v0.10.21 otherwise unit tests will no longer work ;)
With v0.10.21, all unit tests are passing.

The tests passed fine
Linting fails...
src/components/image/__tests__/image-test.js
  13:14  error  Strings must use singlequote  quotes
  14:14  error  Strings must use singlequote  quotes
src/components/image/index.js
  1:1  error  Newline required at end of file but not found  eol-last

Can you update the docs with an example, and maybe even update the tile to include the image and a child element?

@jodiedoubleday Pushed changes https://github.com/holidayextras/ui-toolkit/commit/5d3bb45e0e70fa6704971128d9353ed1386e0b1d
Also, question on the "can you update the docs with an example" ... are you talking about the README.md file in the component?  That already had the example, which is the same as the other components in format.  Perhaps you are referring to another set of docs?

Docs as is http://www.ui-toolkit.com
All these examples are held in docs/examples You will need to create and image section and add it to the right hand navigation etc.

you can also remove dev folder and readme.md files from the individual components now, they are no longer required as of - https://github.com/holidayextras/ui-toolkit/pull/13

Taking SE/DEV

Taking the IP/Dev review

Looks good, as Jodie says docs need updating.

This looks good to me, once Jodie's comments are addressed ill happily +1 it.

Latest commit has updated docs and removed dev / README files, per request.

This looks pretty good to me +1

:+1: Looks good. @jodiedoubleday might be worth adding more margin/padding between the article tags on the doc. They're a bit crammed together.

@dancork noted, ill be sorting that out in a separate PR

Look good :+1: (going to update tile in separate PR)


Stream response from the Langchain Assistant

WIP
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I tested this PR in my own project and it actually works as expected

Closing this PR since https://github.com/patterns-ai-core/langchainrb/pull/760 was merged.


Support for broadcast calendar

Hi Team,
Is there any support for broadcast calendar

Thanks for opening this request.
You can do something like
https://momentjs.com/docs/#/displaying/calendar-time/
This is a great question for Stack Overflow.

@phanivaranasi did you come up with a solution for this and care to share any examples? I am currently working on a project based on a broadcast calendar and would love any pointers.


Add wrapper script to gather cluster logs

Re-sending #2634 on the stg branch, copying over @jupierce 's comment with the background:
@joelsmith FYR. This relates to the work we have previously discussed to allow non-ops engineering team member to grab (non-application) logs from a select set of clusters. This script (modeled on the one you provided for verifying cluster operations initiated by Jenkins) will be the entrypoint for an ssh key in tower2's authorized_keys file.
Ultimately, an engineer should be able to down an ssh key and run:
ssh -i logkey.pem tower2 free-int > /tmp/free-int-logs.tgz

And receive a tarball with atomic-master journal entries.
The associated keys will be rotated weekly and made available to the engineering team in a private github repo which is already locked down to openshift developers and regularly culled.

Our most recent conversations about enabling engineering log access focused on a web application front end for log gathering, but I've backed off that idea until we have some spare cycles.
This script will be loaded onto tower using an RPM.

@codificat We've gotten through an initial review, but we've been asked to prompt for and log the developer's kerberos id (no validation necessary) in the verify script.
So the interaction would look something like:
sh -i logkey.pem opsmedic@tower2 free-int > /tmp/free-int-logs.tgz
Enter your kerberos id:  jupierce

The prompt would obviously need to be piped to stderr in order to not corrupt resultant tarball. Once you have this in place, can you schedule some time with @dbaker-rh to do a final review/test of this script from a security perspective?

I had more in mind putting kerb id on the command line:
ssh -i logkey.pem opsmedic@tower2 -u dbaker -c free-int
To recap the pertinent part of the conversation, having shared keys removes the ability to audit access.  Absent doing something high-cost like provisioning rotating keys for each developer with access, a procedural policy to self-identify goes a long way to closing that hole.

@dbaker-rh An argument based method works for me.
@codificat With multiple parameters, we will also need help text, -h, --help to describe those arguments out stderr.

@dbaker-rh @jupierce just pushed a new version that uses arguments. BTW, note that we need to separate ssh arguments from the command itself with --:
ssh -i logkey.pem opsmedic@tower2 -- -u dbaker -c free-int

Argument parsing is done with argparse which among other things provides the requested --help for free... albeit not via stderr. This is meant to be run more or less interactively, so if someone explicitly invokes with -h/--help I hope it's ok to get that help via stdout and a non-error exit status?
On the other hand though, one thing we lose in this version with argparse is logging of incorrect invocations, e.g. an invocation referencing an invalid cluster name or without a username will get an error, but this won't get logged via syslog.
Wanted to get your thoughts on this: If we need to handle the latter (i.e. log incorrect invocations) this version will have to be adjusted.

To clarify: we can log every invocation - it's just that argparse will handle incorrect arguments itself and won't explicitly syslog them. The question here is if we need an explicit log entry that explains incorrect arguments.
@dbaker-rh let me know if we can schedule a short meeting to review this (and the rest) thanks!

+1


OpenCV: 3.0.0: Grabbing+Writing: Very high CPU load.

Description:
I upgraded an application from opencv 2.4.9 to 3.0.0 on Windows. The application grabs Frames from 3 USB cameras via DirectShow. It filters them and later it writes an avi file to the HDD.
With OpenCV 2.4.9 it uses about 8 to 9% of CPU. With OpenCV 3.0.0 it uses 100% of CPU. my internal queues run over and the application stalls and finally crashes when the memory Limit of 2GB on 32 Bit get hit.
Even when i stop my graber and writer (delete the shared pointer and join all my workers) there are much more threads still hugging about 88% of my i7 CPU. I debugged the issue and saw that all my threads are confrmed to be stopped.
I have a lot of threads like "vcomp120.dll _vcomp::PersistentThread".
As a Workaround i downgraded again to 2.4.9 and all runs as expected.
I compiled 3.0.0 from git with MSVC2013. I would attach the configuration of CMAKE but it seems not possible.
Environment:

MSVC2013
OS Win7 x64
Application + OpenCV 3.0.0 32 bit
Driver DirectShow (700+x) x = camera number

Important configs (i suppose)

WITH_OPENMP
WITH_OPENCL

I have no Cuda device. Is OpenCL going crazy?
Best Regards
Georg

Here is the CMake Output
Looking for Mfapi.h
Looking for Mfapi.h - found
To enable PlantUML support, set PLANTUML_JAR environment variable or pass -DPLANTUML_JAR= option to cmake
Could NOT find PythonInterp (missing:  PYTHON_EXECUTABLE) (Required is at least version "2.7")
Could NOT find PythonInterp (missing:  PYTHON_EXECUTABLE) (Required is at least version "2.6")
Could NOT find PythonInterp (missing:  PYTHON_EXECUTABLE) (Required is at least version "3.4")
Could NOT find PythonInterp (missing:  PYTHON_EXECUTABLE) (Required is at least version "3.2")
Could NOT find JNI (missing:  JAVA_AWT_LIBRARY JAVA_JVM_LIBRARY JAVA_INCLUDE_PATH JAVA_INCLUDE_PATH2 JAVA_AWT_INCLUDE_PATH)
FEHLER:
Der angegebene Registrierungsschl?ssel bzw. Wert wurde nicht gefunden.
FEHLER:
Der angegebene Registrierungsschl?ssel bzw. Wert wurde nicht gefunden.
Could NOT find Matlab (missing:  MATLAB_MEX_SCRIPT MATLAB_INCLUDE_DIRS MATLAB_ROOT_DIR MATLAB_LIBRARIES MATLAB_LIBRARY_DIRS MATLAB_MEXEXT MATLAB_ARCH MATLAB_BIN)
VTK is not found. Please set -DVTK_DIR in CMake to VTK build directory, or to VTK install subdirectory with VTKConfig.cmake file
videoio: Removing WinRT API headers by default
General configuration for OpenCV 3.0.0 =====================================
Version control:               3.0.0
Platform:
Host:                        Windows 6.1 AMD64
CMake:                       3.2.3
CMake generator:             Visual Studio 12 2013
CMake build tool:            C:/Program Files (x86)/MSBuild/12.0/bin/MSBuild.exe
MSVC:                        1800
C/C++:
Built as dynamic libs?:       Try to set environment variable OMP_WAIT_POLICY to PASSIVE. Or disable openmp and enable openmp. See http://stackoverflow.com/questions/4738045/openmp-huge-performance-differences-between-visual-c-2008-and-2010  and  https://support.microsoft.com/en-us/kb/2689322


Draft: Added export to type Theme<T>.

Hi,
I would like to to use function render, but I can not typehint the theme argument in render.
That's why I added the export.
Best regards
Enough7

I do not know import { render } from 'resumed' does not work for me either.


It took a couple of updates before I could get it to run locally but I'm currently stuck- it looks like a problem with the SDK but not sure what.


Added namespace to TwitterAPIExchange

Hi, thanks for this. Unfortunately if I change the namespace now (it's currently in the global namespace, unfortunately) it'll break backwards compatibility massively for everyone. Perhaps in a future update. (Also, Laravel isn't 'MVC' ;-))


Wordcount

This patch includes a few things:

TV's patch to lower the buffer size for the FUSE library.  Along with #591, this fixes #581
Add another example for "word count", a classic big data application where you compute the number of occurrences for each word in a collection of documents.
Make the retry mechanism more robust.  Specifically, GCS sometimes return a 500+ error code on the creation of NewRangeReader which wasn't handled by the retry mechanism.

In the process of adding a README for wordcount.

Waiting on README ofc


notify-osd no longer builds

Issue description
Compilation of notify-osd fails because it can't find dbus-binding-tool.
Making all in src
make[2]: Entering directory '/tmp/nix-build-notify-osd-0.9.34.drv-0/notify-osd-0.9.34/src'
/nix/store/d20f169ryps7ds2qak0r5n1f4hhxr80h-bash-4.3-p42/bin/bash ../libtool --mode=execute /nix/store/hxsbjbjn7g1j1cf60n228yi9wnzrl4yk-dbus-glib-0.104/bin/dbus-binding-tool --prefix=stack --mode=glib-server --output=stack-glue.h notify-osd.xml
../libtool: line 10051: /nix/store/hxsbjbjn7g1j1cf60n228yi9wnzrl4yk-dbus-glib-0.104/bin/dbus-binding-tool: No such file or directory
Makefile:1109: recipe for target 'stack-glue.h' failed
make[2]: *** [stack-glue.h] Error 127
make[2]: Leaving directory '/tmp/nix-build-notify-osd-0.9.34.drv-0/notify-osd-0.9.34/src'
Makefile:330: recipe for target 'all-recursive' failed
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory '/tmp/nix-build-notify-osd-0.9.34.drv-0/notify-osd-0.9.34'
Makefile:261: recipe for target 'all' failed
make: *** [all] Error 2
builder for ‘/nix/store/i03p6rzqg8vjvkdkncawvzf8p6bllv8m-notify-osd-0.9.34.drv’ failed with exit code 2

Steps to reproduce
nix-env -i notify-osd
Technical details

System: (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...)
16.09pre82105.7465bcd (Flounder)

Nix version: (run nix-env --version)
nix-env (Nix) 1.11.2

Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion)
"16.09pre82105.7465bcd"

This is because notify-osd locates the binary via pkgconfig, which has:
prefix=/nix/store/hxsbjbjn7g1j1cf60n228yi9wnzrl4yk-dbus-glib-0.104
exec_prefix=${prefix}

I've seen this several times now; I wonder if we should globally pass --exec-prefix as well when passing --bindir.
cc @vcunat

I don't think exec_prefix should have anything to do with executables, and both of them are closest in $out. IIRC it's for systems where during installation you put files into some directory (prefix) but when actually using the files, they're located in a different one (exec_prefix).

So to be clear, the dbus_glib package has:
outputs = [ "dev" "out" "docdev" ];
outputBin = "dev";

... so the pkgconfig file would need to refer to dev for this to work.
The GNU autoconf manual gives this explanation:

Similar to ‘--prefix’, except that it sets the location of installed files which are architecture-dependent. The compiled ‘emacs’ binary is such a file. If this option is not given, the default ‘exec-prefix’ value inserted into generated files is set to the same value as the ‘prefix’.

which is not very helpful (to me) in describing what it should actually mean; i.e. are the apps that do things like
DBUS_GLIB_BIN="`$PKG_CONFIG --variable=exec_prefix dbus-glib-1`/bin"

mis-using the variable (that is what notify-osd uses).

I suppose we should patch this single package, unless there are (many) others using exec_prefix in this meaning.

Besides this, I've noticed 584d884bd7010fb6a990978dec03a4ffe22d2fba and c4ab7e2cd4f5ad39a2f1d590af2d2b72037fdfb2 in the repo so far.

I think the problem is that autotools uses many more variables (by default) than pkgconfig exposes in most packages (bindir, sbindir, libdir, includedir)
When I came across this issue, I was thinking about submitting upstream patches to fix dbus-glib-1.pc to include the other used variables (mainly bindir)
Then fix notify-osd (and others) to query bindir instead of using $exec_prefix/bin which is a breaking change so it will be hard to convince upstream.
Hopefully most packages can be configured with the variable DBUS_BINDING_TOOL which we could automatically set when dbus-glib is a build input.

@dezgeg, @groxxda, I understand there is possibly a bigger issue at play here and it should be solved in a proper way, but I'm wondering if you could tell me the one off changes required to notify-osd specifically working here. I'm not up to speed on all the changes from the closure-size merge and can't seem to wrap my head around what is necessary.

Heh, indeed too much talk and not enough action ;) I now pushed https://github.com/NixOS/nixpkgs/commit/367b2aa1e422f2ee8a0860df52e98a5d788cef6a since it's not (yet) clear that the other alternative(s) would be any better/more.

I confirm it builds on master.

:+1: thanks @dezgeg!


[fix][doc] Fix typos in doc for Validator class

Motivation
Fix typos in doc for Validator class
Modifications
belogs => belongs
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks


Allow extra logger configurations

By providing additional logger options, we should be able to better customize Lage's output. By using NPMLogger this customization is already possible, Lage should provide a parameter that accepts changes to NPMLogger disp, styles, levels.
Additionally, it would be helpful to be able to specify the log level to be used.

Resolved by #124


Nameplates is causing LUA errors

Description
Crashing with a ton of LUA errors  from nameplates
Version Number
The last update I see was on 4/22/19. Nothing I see has been fixed since that last patch
Steps to Reproduce

Step 1
Step 2
Step 3
Step 4

Expected Behavior

Actual Behavior

Only KkthnxUI Enabled

Additional Information

Error Log

The version you reported for was marked 8.1.5 , and worked flawlessly with 8.1.5. If you have to check "load out of date addons" to enable an addon, and then that addon doesn't work, you shouldn't be surprised.
Don't report a square peg not fitting in a round hole as a bug.


App compiling and working fine using PyInstaller but not with Nuitka

Hi There,
I am very new to Nuitka, I had a closed issue: https://github.com/Nuitka/Nuitka/issues/2534 after that I read docs and I was success in compiling the app.
./build.sh

PyInstaller Output
INFO:     Will watch for changes in these directories: ['/mnt/WorkSpace/Code/hello']
INFO:     Uvicorn running on http://<IP_ADDRESS>:5555 (Press CTRL+C to quit)
INFO:     Started reloader process [41671] using StatReload
INFO:     Started server process [41678]
INFO:     Waiting for application startup.
hello
INFO:     Application startup complete.

Nuitka Output
INFO:     Will watch for changes in these directories: ['/mnt/WorkSpace/Code/hello']
INFO:     Uvicorn running on http://<IP_ADDRESS>:5555 (Press CTRL+C to quit)
INFO:     Started reloader process [41671] using StatReload

App stuck at INFO:     Started reloader process [41671] using StatReload
I am adding a github repository url: https://github.com/krunaldodiya/hello
When i compile same app with PyInstaller, It works but wih Nuitka, Its is compiling successfully but application is not running, Can you take a look and help me out.
Sorry if it was too noob, any help would be appreciated.
Thanks

In the template, I have made it clear, that not filling out the issue template is not acceptable. I am closing this kind of report as invalid. The main reason is that I do not want to be left guessing. For you it's easy to erase the issue template and expect me to ask you questions only as necessary, but I cannot do that. I would much rather wait for you to create a new report, fill out the template, or wait for somebody with the necessary skill to do it.
In short: No issue template, no fun. Therefore I am likely just deleting the issue, as it serves as a bad example and detraction for searches.  And even worse, it might keep good reporters from reporting as is this "already known."


Misalignment of md-list-avatar icons inside an md-list-item

Bug, feature request, or proposal:
Feature request
What is the expected behavior?
An md-list-avatar in a single lined md-list-item should align horizontally with the md-list-item text.
What is the current behavior?
The md-list-avatar does not align, the icon appears 8px more to the top then the text of the list item.
What are the steps to reproduce?
Create an md-list with list items containing an md-list-avatar and md-line, the issue occurs also on md-nav-list as well as non dense lists.
<md-list dense>
    <md-list-item>
        <md-icon md-list-avatar>arrow_forward</md-icon>

Pepper

    </md-list-item>
    <md-list-item>
        <md-icon md-list-avatar>arrow_forward</md-icon>

Salt

    </md-list-item>
    <md-list-item>
        <md-icon md-list-avatar>arrow_forward</md-icon>

Paprika

    </md-list-item>
</md-list>

Providing a Plunker (or similar) is the best way to get the team to see your issue.
http://plnkr.co/edit/HsygqK66et7JhsNEDgq4?p=preview
What is the use-case or motivation for changing an existing behavior?
The layout doesn't look right without this change.
Which versions of Angular, Material, OS, browsers are affected?
Test on angular  2.4.2, material 2.0.0-beta.1, MacOs El Captain, Google Chrome 55.0.2883.95 (64-bit)
Is there anything else we should know?
The md-icon doesn't have a specific height for the usage in an md-list. Adding height:24px to the md-icon will fix it.

@tobi-or-not-tobi I'm not sure if it's a bug because md-icon is not recommended. However, setting a custom class on <md-icon md-list-avatar class="menu-icon">arrow_forward</md-icon> to override the default height i.e 40px to 24px manually is the real strategy.
I've been doing it too.

Thanks @codef0rmer that's indeed what I did to fix it. I didn't know it was not recommended, I actually think its a very common to use an md-icon in a list like you and I do. I'll rewrite the issue to change it from but to proposal.

To display an icon in list item use md-list-icon directive instead of md-list-avatar.

Thanks @ocarreterom I wasn't aware of that feature.


zkvm/blockchain: implement apply tx list procedure

Implements apply transaction list from blockchain spec

This PR adds 12 lines for a job of a one-liner iterator. Maybe we can just loop and apply where we need to, w/o intermediate generic function that does it?

Sure, that makes sense to me! Closing this PR :)


Questions about TreeExplainer runtime speed

I have tried running shap on a lightgbm model which has a training set size of 40k with 400 features and a test set of 90k. I have not yet been able to get output from shap because it seems prohibitively slow. I have let it run for hours and still it did not complete. I have made great efforts to reduce both training set size and feature size in order to get shap to run in reasonable amount of time. I'm wondering if

it is possible to provide estimated runtime remaining, as it might make waiting easier, so I know it's not going to take years to finish
There is a way to speedup computation. Is the "Independent" feature_dependence method any quicker or is it slower? I know "approximate" should be faster, but it doesn't seem much better than number of splits importance, as both are "inconsistent". Is shap runtime more dependent on number of features or sample size, and is it the training set or the test set which runtime is more dependent on?

Thanks,
Adam

Also meet the same question 👉https://github.com/slundberg/shap/issues/829

@aamster @manoshape I'll try to answer both questions right here for TreeExplainer.

TreeExplainer with feature_dependence="tree_path_dependent" does not depend on the training set size, only on the number and depth of the trees in the model. The strongest dependence is on the depth of the trees, where the runtime is quadratic in the actual depth of the trees.
When you use feature_dependence="independent" then TreeExplainer does not depend on the number of background samples you give it. If you give it 1000 background samples it will 100x slower than if you give it 1. I would never use more than 1000 background samples, and just subsample if I had more.
Of course both depend on the number of samples you are explaining. Explaining 1000 samples is about 100 times slower than explaining 10 samples. This gives you an easy way to estimate runtime for many samples, just try it on a few samples and extrapolate.

Are there specific cases where you find the runtime not reasonable given these options?

TreeExplainer with feature_dependence="tree_path_dependent" does not depend on the training set size, only on the number and depth of the trees in the model.

@slundberg I have run into issues with longer run times when using the TreeExplainer with models trained on larger training data sets, but with the same number of trees and the same depth for each tree. Below is an example.
import shap
from sklearn.ensemble import RandomForestRegressor
from sklearn.model_selection import train_test_split

X,y = shap.datasets.nhanesi()
X = X.dropna()
y = y[X.index]
X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.2, random_state=7)

m1 = RandomForestRegressor(n_estimators=20, random_state=0, max_depth=10)
m1.fit(X_train[:1000], y_train[:1000])

m2 = RandomForestRegressor(n_estimators=20, random_state=0, max_depth=10)
m2.fit(X_train[:100], y_train[:100])

# Confirm that there are no differences in tree depths
all([m1_est.get_depth() == m2_est.get_depth() for m1_est, m2_est in zip(m1.estimators_, m2.estimators_)])

## True

Timing reveals that the m1 model trained on the 10x larger training set takes ~4x the time to calculate the shap values with the default feature_dependence="tree_path_dependent" (using %%timeit in jupyterlab):
%%timeit
explainer = shap.TreeExplainer(m1)
shap_values = explainer.shap_values(X_test)

## 1.56 s ± 4.93 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

%%timeit
explainer = shap.TreeExplainer(m2)
shap_values = explainer.shap_values(X_test)

## 371 ms ± 1.57 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

I work with somewhat large training dataset so this time scaling becomes rather cumbersome to deal with. I am considering switching to the approximate calculation, but since you mentioned that the training data set size should not affect the calculation, I thought I would post here in case there is something that can be fixed with the TreeExplainer implementation so that timing would not increase with the size of the training data set.

@joelostblom the runtime with feature_perturbation="tree_path_dependent" (the new name for the feature_dependence option) only depends on the number of samples being explained and the size of the model. My guess is that while the max_depth of your models is 10, the random forest does not build complete depth-10 trees until it has a lot of training data. So what you are really seeing is the size of the model (specifically the size of the trees) growing as you have more data.

My guess is that while the max_depth of your models is 10, the random forest does not build complete depth-10 trees until it has a lot of training data.

@slundberg I checked that in my code above (also pasted below) and all the estimator have the same actual depth, not just max depth.
# Confirm that there are no differences in tree depths
all([m1_est.get_depth() == m2_est.get_depth() for m1_est, m2_est in zip(m1.estimators_, m2.estimators_)])

## True

So all trees with the small training data set reached depth 10, which is the current max_depth.
If I increase the allowed max depth, then the larger training data set creates deeper trees.

But do they fill out the entire tree to that depth? In other words do they have 2**10 nodes in each tree? If they do then something might be wrong, but I would expect that there would be more nodes in the trees from the model trained on more data.

Ah ok, you're right, the trees do have different number of leaves in them.
[m1_est.get_n_leaves() for m1_est in m1.estimators_]
## [189, 222, 182, 229, 185, 180, 260, 192, 180, 204, 208, 182, 213, 199, 207, 132, 196, 190, 191, 168]

[m2_est.get_n_leaves() for m2_est in m2.estimators_]
## [62, 52, 52, 56, 55, 58, 50, 63, 60, 45, 53, 47, 66, 44, 51, 51, 56, 63, 50, 59]

If I set max_leaf_nodes=40 for both m1 and m2, they take about as long to run with TreeExplainer.
So just to check that I got this down right: TreeExplainer depends on

The number of trees
Their depth
The number of leaves in each tree

The last two of those three will increase with more training data, so unless the model is limited to a set depth and number of leaves, more training data will lead to longer run time because of the more complex models that are constructed?

That's right! Nice analysis.

Just a quick (and maybe obvious) question:  should there be heavy calculation at initialization of the object or just on predicting SHAP values? I'm seeing pretty long runtimes at shap.TreeExplainer(randomForestRegressor(.)) before I get to .shap_values(X), is this expected, I haven't been able to look under the hood yet though so this might be a silly question.
Likewise, is there any possibility to add a timer?  I've been hurt too many times by Python, and I've come to love tqdm.  If something like that could be added, I think it might be much appreciated.

I verified that it was indeed shap_values(x) that was taking up the runtime, seems like the javascript might have suppressed the output of the notebook. Still would be great to get a timer, my model (500 trees trained on ~40k samples) took almost 17 hours on my comp, would be wonderful to know in advance if at all possible!

Just a quick (and maybe obvious) question: should there be heavy calculation at initialization of the object or just on predicting SHAP values? I'm seeing pretty long runtimes at shap.TreeExplainer(randomForestRegressor(.)) before I get to .shap_values(X), is this expected, I haven't been able to look under the hood yet though so this might be a silly question.
Likewise, is there any possibility to add a timer? I've been hurt too many times by Python, and I've come to love tqdm. If something like that could be added, I think it might be much appreciated.

Hello @M-Harrington , I'm experiencing the same thing when creating a TreeExplainer from a pyspark random forest model. It's a pretty huge model, so the inicialization of the explainer takes around 2.5h. Of course, then obtaining the shap_values takes a lot too, but I'm just wondering if there are any corners I might be able to cut :)


Update to the GA version of go-amqp

Updated readme message with GA announcement of azeventhubs.
Fix or Enhancement?


Changelog: add another EndpointType changelog to "Breaking" part

https://github.com/rust-embedded-community/usb-device/blob/7ebb10fe5f952955b416c5b05abc84d37bf82acf/CHANGELOG.md?plain=1#L22-L23
Since EndpointType is a public enum, and some crates may use the integer form of it. After this modification user will need to call .to_bm_attributes() to access numeric form. We might add a another changelog to tell user about this.
### Breaking
+++ * Acess numeric form of `EndpointType` variants now require a `.to_bm_attributes()` ([#60](https://github.com/rust-embedded-community/usb-device/pull/60)).

Thanks for noticing this.
Could you put this into a PR?

please review my PR


[DNM] use cluster name instead of id

I think changing all clusterName -> name is ok, @qiuyesuifeng

In struct Cluster's fields, using Name is ok, otherwise clusterName tend to unambiguous.
@siddontang

PTAL


link is not read by the rest scraper

The following link is not read by the rest scraper: https://services.arcgis.com/SzoH1oFM2apCSkx3/ArcGIS/rest/services/DEUR/FeatureServer/0
There was no error reported and the page won't stuck, just won't return anything on the page.

Hi Zac, that issue has been resolved here #16. The spatial reference was not where I expected it to be but I have accounted for this case and added safe guards to actions later to account for a spatial reference not found.


Rohd hcl parity

Description & Motivation
Note: Requires Find PR #37  to be merged in before this PR.
A modules for adding functionality of Parity.
Contains 2 modules for parity functionality:

ParityTransmitter Module - Appends parity bits to given data that can be used for transmission
ParityReceiver Module -

Takes the transmitted data with parity bit.
Splits the data into original data and parity bit.
Also checks for parity error (result) within original data

Related Issue(s)
Related to issue #39
Testing
Testing is handled within /test/parity_test.dart
Backwards-compatibility
Yes, this PR is backward compatible. This is an added functionality.
Prior modules are not unaffected or reused for completion of this functionality.
Documentation
Documentation is handled.

@mkorbel1 this PR is ready for a review. also note: this PR contains files from #37

@mkorbel1 @chaparalas @quekyj  can this pr be merged in :)

Sorry for the delay!

Sorry for the delay!

No worries :) Thanks...


Check static linking on Mac and Linux

Static linking of Clang and LLVM libraries is not easy, unfortunately. Clang uses zlib, ffi and other libraries. MacOSX SDK during compilation with these static libraries produces weird errors like redefinition of isnan, isinf or undefined type nullptr_t. Perhaps libclang-static-build has answers and a solution.

It's possible by using llvm as a submodule, implemented in separate branch.

It's possible by using llvm as a submodule, implemented in separate branch.


Add verify-mergeable

Depends on #439
A bunch of comments for verify-mergeable were left there, will cross reference.

@wlynch left comments about verify-mergeable in #439, tracking those comments here.
https://github.com/gittuf/gittuf/pull/439#discussion_r1649068232
It's a little weird that the base branch is a flag and the feature branch is an arg.
I wonder if we should change this to match git behavior?
e.g. git merge main means merge main into the current branch, so gittuf verify-mergable feature == verify that the feature branch can be merged into the current branch?
https://github.com/gittuf/gittuf/pull/439#discussion_r1649082476
Everytime I read this it throws me for a loop because we're failing open. I wonder if we should allow restricting this somewhere to require at least 1 verifier. 🤔
https://github.com/gittuf/gittuf/pull/439#discussion_r1649084802
Where is the gitObj signature being done? It's not immediately obvious from this code so this looks like we're checking the same thing twice.
https://github.com/gittuf/gittuf/pull/439#discussion_r1649085936
What is this tracking and why is it only checked the 2nd time?
https://github.com/gittuf/gittuf/pull/439#discussion_r1649088759
Also not clicking what this has to do with file rules. If this is for mergeability, e.g. "the commit has the number of required RSL entries needed to meet the threshold", then this should be renamed since it reads like "RSL entries are needed to meet the threshold"
https://github.com/gittuf/gittuf/pull/439#discussion_r1649099274
Why resolve the target ids via the RSL instead of via the refs directly?
Could the RSL potentially be behind the ref value if a new commit was pushed without updating the RSL?
https://github.com/gittuf/gittuf/pull/439#discussion_r1649100626
Taking a step back, there's a lot of verification logic here that feels like should be encapsulated elsewhere. Ideally I think this should look like:
What information do you need to be returned from verification in order to make policy decisions about mergability?

Taking a step back, there's a lot of verification logic here that feels like should be encapsulated elsewhere. Ideally I think this should look like:
What information do you need to be returned from verification in order to make policy decisions about mergability?

I think we may have addressed this in part with the previous PR, but fresh opinion would be great!


Fix: Blog Post "Like 👍"  functionality

Description
As mentioned in #116, the blog's "Like 👍" functionality needs improvement.
Tasks

[ ] When the user clicks the Like button, they should get a pop-up modal prompting them to register/login first.
[ ] If the user chooses to login/register, then allow them to mutate the Like count only once.

Currently, the usePostLikeBlog function (hooks/Queries/blog/usePostLikeBlog) currently uses axios only to make a POST request. Here's what you should change:

[ ] The usePostLikeBlog function should be moved to hooks/Mutations/blog/usePostLikeBlog since it's a mutation and not a query
[ ] Instead of using axios,  use a privateAxioswith@tanstack/react-query(reference:hooks/Mutations/shop/useMakeOrder`)

Hey. Would like to be assigned this.


lib/services_cli: Fix not-running service status always being "unknown"

This was buggy such that a not running service without a PID would default to showing as "unknown". This is because the service_get_status method would never fall through to the else condition of :stopped.  It was always being caught by service.unknown_status? because there was no PID and formula[:status] was blank as it wasn't being set first of all.

This fixes it such that formula[:status] starts off as :stopped, then we set it to :started if a PID exists, as we did before 44f7fc0aeb8f2e594caa5e1b6e3317d49ebbe8bd.

Given we're only checking if the service is started or otherwise errored, I renamed service_get_status to service_get_operational_status as it was clearer to me.

I removed the fallback test as there is no fallback - ":stopped" is a service's default value again now.

Fixes #368.

Before this change:
❯ brew services list
Name       Status  User   Plist
mysql      unknown
nginx      started issyl0 /Users/issyl0/Library/LaunchAgents/homebrew.mxcl.nginx.plist
postgresql unknown
redis      started issyl0 /Users/issyl0/Library/LaunchAgents/homebrew.mxcl.redis.plist
unbound    unknown

After this change:
❯ brew services list
Name       Status  User   Plist
mysql      stopped
nginx      started issyl0 /Users/issyl0/Library/LaunchAgents/homebrew.mxcl.nginx.plist
postgresql stopped
redis      started issyl0 /Users/issyl0/Library/LaunchAgents/homebrew.mxcl.redis.plist
unbound    stopped

Thanks @issyl0!


Fixes Norm.Contract breaking on modules containing functions without parentheses

Fixes #64 by adding:

A call-normalization step in Norm.Contract.def
Tests for modules that have functions without parentheses.

Thanks @Qqwy! This has been rebased and merged :tada:


Highlight current page in navigation

Right now, you can't look at the menu and tell what page you're currently on.

Hey, how are you?.
I help you on this case ✌.

This would be great. Let's just add a bottom border of green on the item in the menu.

I have a question.
The border bottom is only for the  tag element or  tag element?.

Probably the li.

This was difficult 😥


[extractor/instagram] Fix graphql, API, webpage extraction

Cloned from PR #4657 with additional changes:

Working graphql requests
The extractor will only try the mobile API if the user is logged in or has passed cookies. Mobile API only works for logged-in users
Extractor only attempts webpage extraction if API and/or graphql extraction fails. This will help to prevent the user from sending unnecessary requests and being rate-limited by Instagram

Fixes #4532
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[x] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

This fixes #4475 too, correct?


Add Barbados holidays

Proposed change

Adding Barbados holidays.
closes: #1153
Type of change

[x] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[ ] Existing code/documentation/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency update (version deprecation/upgrade)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts and has clean commit history
[x] The code style looks good: make pre-commit
[x] All tests pass locally: make test, make tox (we strongly encourage adding tests to your code)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

@KJhellico Could you please check again? Thank you :)

🇧🇧 LGTM

Super, I hope to contribute more going forward! :)


Incorrect the BitNav overridden CSS classes in the bit platform website

some classes of the BitNav component have been overridden in the Bit Platform website project. The names of these classes have been changed in the BitNav component. So current, any name starting with bit-nav- in the NavMenu.razor.scss file is incorrect.

it needs to be mentioned that this should be taken care of after the next prerelease. for now it's fine.


Fuel Codes | Autopopulate part 2 renewable content for new fuel code versions |1|

Story:
As an IDIR user, I want TFRS to autopopulate the part 2 renewable content decision (if applicable) for new versions of existing fuel codes so that I can easily and efficiently create a new field code version.
Scenario:
GIVEN that I am an IDIR user
AND I am creating a new version of an existing fuel code (e.g. BCLCF101.1)
AND the existing (previous version of the) fuel code is partially renewable, as indicated by the check box input field
AND I have inputted the fuel code into the "Low Carbon Fuel Code" input field on the New Fuel Code page
AND TFRS has autosuggested the next version of that fuel code
WHEN I click on the autosuggested next version of the fuel code
THEN the "Is Partially Renewable?" box is checked
AND the percentage of part 2 fuel that is renewable input field is autopopulated based on the data from the previous version of the fuel code
AND I am still able to change this data
Task:
This card extends the autopopulate functionality created in Card 1365 to the part 2 renewable content input fields created in Card 1333
https://trello.com/c/L3id3plo/1409-fuel-codes-autopopulate-part-2-renewable-content-for-new-fuel-code-versions-1

 Fuel Codes | Autopopulate part 2 renewable content for new fuel code versions |1|


Add additional benchmarks

Benchmarks used to verify https://github.com/rusterlium/rustler/pull/482 . At that time there was no benchmarks setup in rustler yet so I didn't include them in the PR.
As there is now, this might be of use for others or CI/CD integration.

Thanks! Can you fix clippy? Also, would be nice to see some example results, could you simply post the machine you used and the output of the benchmark here? :) I wonder what a good structure would be find "interesting" benchmarks if the set of benchmarks in the repository increases. Maybe we should have some logical structure w.r.t. what benchmarks we have.

As an addition, it would be nice if rustler also had a standard way for rustler projects to add benchrmarks/tests that would automatically be setup for PGO, following these steps: https://doc.rust-lang.org/rustc/profile-guided-optimization.html#a-complete-cargo-workflow
Something that could be configured in project Native.ex:
use Rustler,
    pgo_run: <Project>.Benchmark.run()

What about having a single Benchmark.run() that can read command line arguments and generate the benchee options? That way for local testing developers can run a single specific test without commenting code while at the same time there's a single place for CI/CD job to call to run all benchmarks as part of some regression job

W.r.t. regression in CI/CD, in my experience, that seldomly works well unless one puts in considerable effort. CI/CD is run on non-dedicated machines, so results can vary wildly. I wouldn't put too much work into that.
Enabling running all benchmarks with a single command sounds good. I guess for that we would need to make the benchmarks coherent, so that the runs "tell a story". Otherwise, someone running the benchmarks still needs to read the benchmark code first to understand what it is all about.

As an addition, it would be nice if rustler also had a standard way for rustler projects to add benchrmarks/tests that would automatically be setup for PGO,

Interesting suggestion. This goes out-of-scope of this PR, maybe create a separate ticket just for that? I have never played with PGO yet, so I don't know how complicated it is to use.

@cleaton seems I kind make the github actions run again. Can you rebase on master so that the tests run again?


Coding ...Greater than > last

Note: for support questions, please join our Discord server

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)

Expected result (what you hoped would happen)

Actual result (unexpected outcome)

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
How would I code an indicator to show greater than last....? Specifically obv

Simple code
..Obv greater than Obv 1 tick ago

Or is anyone available to create one indicator robot for bittrex? For a fee...Contact me directly<EMAIL_ADDRESS>

see #844 for a implementation of what you want :)


Add targeted contribution pre-linkage metric

I suggest adding a metric that does not just represent how much a set of selected columns represent a whole dataset (as done with the contribution score, but instead, represents how much the selected columns represent a specific set of variables (or how much they are correlated with them).
This means specifying a target set of variables. IMO, it is relevant to future use in practice where we may want to enrich a dataset A with data from a dataset B for a specific task (e.g. predict variable B_3 from variables A_1, A_2, B_1, B_2). In that case, the linkage variables should be well correlated with target variables B_3

Aim:
predict B3 from A1, AB2, B1,
predict salary from loan, age, university
salary: dataset B
loan:  dataset A
age:  datasets A and B
university: dataset A
Linkage variables:
AB2, AB3, AB4
age, nationality, residence, height, ... (both in A and B)
run target metrics at source B:
corr(AB2, AB3, AB4) -> B3
feature importance(AB2, AB3, AB4) -> B3
error(AB2, AB3, AB4) -> B3
is there a corr between {age, nationality, residence, height} and salary ?
extreme case: using matricule ?

I suggest moving this to Documentation as potential future investigation


[Bug] Saved query exports run out of DAG order if they depend on dimensions in other Semantic Models for filters / group-bys

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
metrics and saved_queries can depend on semantic_models directly (aggregating over their measures) or indirectly (grouping by / filtering on their dimensions). Today, dbt can detect and understand only the first type of dependency at parse time.
Previous discussion of this limitation:

https://github.com/dbt-labs/dbt-core/issues/7854#issuecomment-1622594683
https://github.com/dbt-labs/docs.getdbt.com/issues/3682

This has two negative consequences:

[Cosmetic] In dbt list and manifest.json, there are no links between metrics / saved_queries and the semantic_models they depend on indirectly (via dimensions), so we show an incomplete picture of the DAG
[Functional] When dbt Cloud customers are "exporting" saved queries, those need to run in DAG order relative to the models they depend on: [dbt] model -> semantic_model -> metric -> saved_query

Expected Behavior

dbt list -s +<my_saved_query> should include all semantic models that the saved query depends on
dbt build -s +<my_saved_query> should execute resources in appropriate order, with all requisite models upstream of the saved query (no-op / export)

In an ideal world, we'd capture these dependencies at parsing time, so that they would also be reflected in the saved query's depends_on.nodes. IMO it's sufficient to add these edges when we're constructing + linking the DAG, similar to how dbt build adds edges for tests (model -> test -> model).
Steps To Reproduce

Check out jaffle-sl-template
dbt list --select +saved_query:new_customer_orders --resource-type semantic_model — the saved query new_customer_orders depends only on semantic_model:jaffle_shop.orders, even though it needs to be grouped by customers.customer_name and filtered on customers.customer_type
dbt build runs new_customer_orders export (NO-OP in dbt-core) before customers

Relevant log output
$ dbt list -q -s +saved_query:new_customer_orders --resource-type semantic_model
semantic_model:jaffle_shop.orders

$ dbt build
...
11:20:09  29 of 37 NO-OP saved query new_customer_orders ................................. [NO-OP in 0.00s]
...
11:20:09  34 of 37 START sql table model dbt_jcohen.customers ............................ [RUN]
11:20:09  34 of 37 OK created sql table model dbt_jcohen.customers ....................... [SELECT 939 in 0.41s]

Environment
- Python: 3.10.11
- dbt: main

Which database adapter are you using with dbt?
No response
Additional Context
DSI PR:

https://github.com/dbt-labs/dbt-semantic-interfaces/pull/278

Spike of dbt-core implementation:

https://github.com/dbt-labs/dbt-core/compare/jerco/spike-sq-edges

"It works" (though the current implementation is very naïve):
$ dbt list -q -s +saved_query:new_customer_orders --resource-type semantic_model --profile garage-postgres
semantic_model:jaffle_shop.customers
semantic_model:jaffle_shop.locations
semantic_model:jaffle_shop.order_item
semantic_model:jaffle_shop.orders
semantic_model:jaffle_shop.stg_products
$ dbt build
...

I opened this issue in the dbt-core repository in the hopes that we could find a naïve way forward. After syncing with lots of folks (@plypaul @marcodamore @QMalcolm @tlento @ChenyuLInx ...):

There is no way to have a satisfying naïve implementation of this logic in dbt-semantic-interfaces; our best attempt still leaves tons of edge cases
Fully capturing those edge cases requires the full foundational logic of MetricFlow
Incorporating that logic is out of scope for dbt-core

Our path forward will be via direct integration between dbt + MetricFlow in dbt Cloud. That's already where we've documented support for the functionality (saved query exports / caching) that this bug is blocking.
In the meantime, we can choose between:

Leaving the functionality as-is: saved queries depend directly on semantic models for measures (used in metrics), but not for dimensions (used in group by / filter)
As naïve as it gets: every saved query depends on every semantic model

We're opting for (1), and to leave this as is.


Is there a bug with  riot+compiler.js ?

Below is my test code:
riot.tag('index_showTips','title:{opts.title}')
riot.mount('#main', 'index_showTips', {"title":"test"});
if I run with riot+compiler.js,the code won't be exec.
But if I run with riot.js,it runs well.
cause in my page,there is no script type="riot/tag"
I load my tag by requireJS.
I read the sourcecode and find bug is here:
riot.mount = function(a, b, c) {
var ret
riot.compile(function() { ret = mount(a, b, c) })
return ret
}
riot.compile(function() { ret = mount(a, b, c) })
The callback won't be exec.
in compile function:
// add to queue
if (promise) {
arg && promise.on('ready', arg)
// grab riot/tag elements + load & execute them
} else {
promise = riot.observable()
compileScripts(arg)
}
We need a judgement in  function compileScripts,if  script type="riot/tag" not exsit
just trigger "ready".
So ,I think maybe this is a bug?

I have tried with riot+compiler of the dev branch and it seems to work fine. http://plnkr.co/edit/1TvQpAYoqflkYaxebvJo?p=preview could you check it please?

Could you post a jsfiddle example please?


Add support for TLS

See https://github.com/mranney/node_redis/issues/246 for discussion.

+1 for Azure redis

+1

This pull request has been opened for over a year (and the underlying issue for 30 months). Is there something we can do to help you integrate this?

I've updated the code to fit with the new internal structure. It works empirically with redislabs+ssl but the tests don't work just yet. I'm new to stunnel, debugging through it now.

+1

+1

@paddybyers would you still be up for this PR? I'd like to this support for this in node_redis

Yeah but it's been a while so I'd like to check it out next week. Have you tried this branch out?

@paddybyers would you still be up for this PR?

I'm pretty sure it needs some work to rebase to latest master. I'm happy to look at it.

@danlash I did not have any chance to try this yet but it would be great if you could do that instead. The new test suite is probably also helping with this

@paddybyers that would be nice :)

:+1:

Ping :)

Ping :)

I'm working on it; hopefully I can push something tomorrow (Monday).

That would be great :)

Closing this in favour of https://github.com/NodeRedis/node_redis/issues/894.


[sitecore-jss-proxy] fix handling of HEAD requests when followRedirects is true

[ ] This PR follows the Contribution Guide
[ ] Changelog updated

Description / Motivation

Testing Details

[ ] Unit Test Added
[x] Manual Test/Other (Please elaborate)

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

@yavorsk Please, fill the details and update the CHANGELOG

Issue resolved with PR#1630 therefore closing this one.


chore: Update build script to use .NET 2.1

Description

Update build script to use .NET 2.1.

Checklist

[x] My code follows the existing style, code structure, and naming taxonomy
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have performed a self-review of my own code and included any verifying manual calculations
[x] I have added or updated unit tests that prove my fix is effective or that my feature works, and achieves sufficient code coverage.  New and existing unit tests pass locally and in the build (below) with my changes
[x] My changes generate no new warnings and running code analysis does not produce any issues
[x] I have added or run the performance tests that depict optimal execution times
[x] I have made corresponding changes to the documentation

force-merge needed. :(