Find other cargo config files for macOS hack

This PR extends the macOS platform hack to work with workspaces as well. The general idea is to retrieve the workspace root from cargo metadata (which also works for single crates without a workspace), and then iterate over all possible configuration locations between a crate path and the workspace root. If any such potential configuration file exists, we check if it contains a target configuration specific to macOS.
I tested this against https://github.com/evnu/NifIo/tree/macos-platform-hack by replacing the match for :darwin in the make_platform_hacks/4 clause with _.
Fix #235

I'm not a fan of regexing JSON. What is the big issue with actually pulling in a simple parser?

What is the big issue with actually pulling in a simple parser?

I wouldn't say it is a big issue. But we would currently only use it at one place for a very platform-specific hack. If we add a JSON library, all users would have that as a dependency as well, which seems unnecessary if we don't need that library anywhere else. @scrogson what do you think?

Well, you can also use it to capture build-artifacts in a more reliable manner ;)
Also, would users even notice? Not sure how this works for mix, but in rebar3 plugins are not deployed along with the application but only exist during build-time.

I'm not a super big fan of this approach. This warning was meant to be a stop-gap until we get 1.0 out. I think the rustler.new task should generate the .cargo/config file in the root of the mix project along with a Cargo.toml with workspace config pointing to the NIF crate in priv/native.
Also...I think this here is probably all we need? https://github.com/rusterlium/rustler/blob/66616dbcf136026c1442d38d321866e0f404deff/rustler_mix/lib/rustler/compiler.ex#L83-L88
This will print the warning only if neither of those files exist regardless of the contents.

Also, would users even notice?

That probably depends on the users :)

Also...I think this here is probably all we need?

This depends how nested the directory structure is. Cargo can find the configuration in a lot of different places. Of course, we could simply assume that it should be in one of those two places instead. In my projects, I only place .cargo at the workspace root or within the crates itself, so this would be fine with me.

This will print the warning only if neither of those files exist regardless of the contents.

I originally had that as well, but thought that it might be useful to take a last peek into the actual files. But I guess we can also assume that a user knowledgeable enough to create such a file in the first place should be able to also add the entry for macOS without being explicitly told here to do that.

This depends how nested the directory structure is.

After thinking a little bit more about this, I wonder how umbrella projects combined with cargo workspaces work if we check only two specific locations. Specifically, consider a directory structure like this:
/tmp/test_project tree -d -I "deps|_build"
.
├── apps
│   └── some_application
│       ├── lib
│       ├── native
│       │   └── testproject
│       │       └── src
│       └── test
└── config

Now, if the workspace Cargo.toml is defined in . (the root directory of the umbrella) and we place the cargo workspace configuration in ., then mix compile should be able to pick up the configuration if called in . as well as in apps/testproject. I will have to test if this works with https://github.com/rusterlium/rustler/pull/282#issuecomment-554338586.

I just tested a simple example like described here locally. If I place the .cargo/config file into the root directory of the umbrella, checking only the two locations as described here does not find the configuration.

Closing, we decided to drop the warning and search for the configuration completely and simply always add the link arguments on macOS.


Tests fail sporadically

Hi, I'm packaging this for the musl based distro Alpinelinux as uutils-coreutils (in testing) for armhf, armv7, aarch64, x86 and x86_64, the ioctl-sys crate doesn't build for ppc64le and Rust is not available yet for mips64 nor s390x.
Tests fail sporadically for any arch, usually it's just one test that fail but it can be any test. What I do is just trigger a re-run of the build until it succeeds.

Glad to hear :)
Rust works on mips64&s390x, see: https://buildd.debian.org/status/package.php?p=rustc
Do you have more details about the kind of failure that you are getting?
We have some intermittent in the CI but they are not that common

Alpine and musl developers are working together with Rust to get things sorted for mips64 and s390x, https://github.com/rust-lang/rust/pull/82556 for example.
Example failures where two tests happen to fail for x86_64 https://gitlab.alpinelinux.org/omni/aports/-/jobs/363702#L1274
I've added a commit with RUST_BACKTRACE=1 for the tests and will trigger rebuilds until there have been some failure for all archs.


Where is the .7z?

just where is it? (1.5.2)

not a 7z anymore

not a 7z anymore
there used to be a zip and a 7z, the 7z i would use because it already had the stuff there set up

https://drive.google.com/file/d/1M_fGBitYCf8slOxLw5Kcps4HhROQyV-j/view?usp=sharing dunno if this is what you wanted, but here

there are 2 zips
kadeengine-windows.zip -> what you think of as the "7z". this is a build of the game ("already had the stuff there set up")
source code zip -> what you think of as the "zip". this is the source code, (the one u would have to "set up"/build)


Support asynchronous sending of chunked responses (trantors new API)

Follow up from #1883 but using trantors new API. I had to modify trantor a little bit to return true/false whenever sending is possible.
sendAsyncStream() now causes the tcp connection to ignore the idle timeout and not close connection after X seconds

@an-tao Updated trantor to latest commit

lint.patch
@tripleslash please apply the patch to your repo to pass the format CI.

There was still a problem with the unit test, should work now.


NuGet Packager Version Selection

Hello,
I was wondering if there is any particular reason that the NuGet Packager task does not have the ability to select a NuGet version to use. NuGet Installer and NuGet Publisher both have this ability, so it seems strange that NuGet Packager doesn't.
Thanks!
Chris

Yes, please, this would be very useful. Now I have to provide my own nuget.exe just to use SemVer 2 version strings.


Look at the ReadModelTestCase

It's name indicates that it applies all read models, while it actually requires you to implement Serializable.
Maybe rename it to SerializableReadModelTestCase?

I noticed the same thing, in my case I was testing the |serializability" of my domain events, but there is a test case for that as well: SerializableEventTestCase. Still, a more general test case would be quite useful.


Upgrade to Terraform 0.13

Upgrade this module to Terraform 0.13: https://www.terraform.io/upgrade-guides/0-13.html

Closed by #12

One thing that happened to me when using the new version.
Because I was using a container terraform run, there was a local docker volume with some wrong values.
And i was receiving this error:
Error: Failed to install providers

Could not find required providers, but found possible alternatives:

  hashicorp/cloudscale -> terraform-providers/cloudscale

If these suggestions look correct, upgrade your configuration with the
following command:

The following remote modules must also be upgraded for Terraform 0.13
compatibility:
- module.cluster at
github.com/appuio/terraform-openshift4-cloudscale?ref=master

After i removed the local volume, either with docker volume rm <VOLUME>or with docker volume prune the problem was fixed and all works fine again.


Regex for "next_vers" is wrong

Code here https://github.com/Roblox/chef-xcode/blob/master/resources/app.rb#L113
This code is wrong:
chomp.to_i.to_s.split(//).join('.')

In case of 1130 it returns <IP_ADDRESS> ... that's definitely wrong.

Better to use use /usr/libexec/PlistBuddy -c 'Print :CFBundleShortVersionString'
This should always be "Major.minor.path"


OpTestConfiguration: logfile abstraction cleanup

Cleanup a few places where the logfile abstraction parameters need
to be filtered to the various classes to have the logs capture as
appropriate.
Signed-off-by: Deb McLemore<EMAIL_ADDRESS>

retest this please


podofo: fix library linkage on Darwin

The podofo package is currently broken on Darwin: the podofo tools do not find libpodofo.0.9.6.dylib at runtime, because the library path the binary is linked against is incomplete.
The reason is that podofo builds both the library and the tools using this library in the same cmake run. And since the library is not yet installed when the tools are build, it does not contain the final store path as its install name. Linking the tools therefore picks up this incorrect install name.
I fixed this with a fixupPost script. I am not sure this is the correct way, since I am rather new to Nix, but the package works on Darwin now and this should have no effect for Linux.

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

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

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

This is really strange, the dylib does have a correct install_name. The fact that the binaries built before the libraries are installed is not a problem and exactly what the install_name is for. It's used by the linker to allow linking against absolute paths while the library isn't installed in it's final location yet.

The dylib only receives its correct install_name when it is installed. The binaries got built before that, where the install_name of the library was still containing only the filename without any path information. At least that’s what happens when I observed the build. I am not a cmake expert, though.
I guess it works fine when you only build a library and then another project links against that library. But doing both in the same cmake build appears not to work properly.

Hmm, that's unfortunate. Guess they must be using some nonstandard build scripts because I've not seen this before with cmake builds.

Thanks a lot for this tip. I am rather new to Nix, so I did not know this about stdenv.lib.optionalString. I fixed the pull request.

No big deal 😄. It does about the same thing, but it's slightly nicer to use IMHO.

I agree. Thanks for making contributing to Nix such a nice experience.


Disabling git ls-files

We have a repo with over 25k files and the output of "git ls-files", possibly from dist/index.js call to listFiles(), adds significant noise to our logs.  Is there a way to disable this feature?

+1. We run tests from jar and don't need "git ls-files". But obviously - "git ..." also fails (since there was no checkout step before)

We want to disable "git ..." too.
Because of this, we cannot run the action in a separated yaml file through artifacts since it requires a git repository.

This issue addresses the git ls-files step: https://github.com/dorny/test-reporter/issues/210
Max annotations needs to be set to 0 when using the action.
max-annotations: 0

+1 for the removal of git ls.
I've had to specifically add a actions/checkout@v4 step so that the report can be created in a job that just builds reports and doesn't require the repo checked out.

max-annotations: 0 setting up bypasses this issue for now as suggested bu @mesporas-talis. Thanks


Error: invalid_scope

Running Yakyak first time in a Windows PC 32 bits its give the follow error:

Error: invalid_scope Not authorized to request the scopes: [https://www.google.com/accounts/OAuthLogin]

client_id=936475272427.apps.googleusercontent.com
scope=https://www.google.com/accounts/OAuthLogin
redirect_uri=urn:ietf:wg:oauth:2.0:oob
response_type=code

This is super strange.
which version of yakyak? try the latest or the last pre-release please
This bug was supposed to be solved last september


PRL-4172 - People relations & child lives with citizen to solicitor fields mapping

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)
[ ] the keep-helm label has been added, if the helm release should be persisted after a successful build

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

COS PR https://github.com/hmcts/prl-cos-api/pull/1599 is merged into master.
Closing this testing PR.


transfer playback to another spotify device broken

Ahoy!
As an extension/note to #136 , reporting that transferal of playback seems to be broken.
To start (baseline), playback on the local mycroft device through spotify installation instructions worked. (albeit after an unknown long period of delay ... at minimum my system didn't converge this capability for 6-24hours after configuring spotify & auth)
To start, let's playback some music:
 play some classical
 >> Just one moment while I look for that
 >> Now playing Some Classical Music by Various Artists.

works!
To start, let's look at what other devices the spotify skill knows about:
 What Spotify devices are available?
 >> [AV] back den soundbar

, this is correct.
However if I try to transfer the playback, it doesn't work
 transfer playback to [AV] back den soundbar
 >> I'm not sure I understood you.
 transfer playback to back den soundbar
 >> I don't know what that means.

both times
 07:22:55.126 | INFO     |  2785 | QuestionsAnswersSkill | Searching for transfer playback to [AV] back den soundbar
Removing event fallback-query.mycroftai:QuestionQueryTimeout
Removing event fallback-query.mycroftai:QuestionQueryTimeout
Removing event fallback-query.mycroftai:QuestionQueryTimeout
Removing event fallback-query.mycroftai:QuestionQueryTimeout
Removing event fallback-query.mycroftai:QuestionQueryTimeout
 07:22:56.361 | INFO     |  2785 | QuestionsAnswersSkill | Timeout occured check responses

ONE time, (the first time I tried)
 07:16:12.245 | INFO     |  2785 | QuestionsAnswersSkill | Searching for hey mycroft, transfer playback to back den soundbar
Removing event fallback-query.mycroftai:QuestionQueryTimeout
Removing event fallback-query.mycroftai:QuestionQueryTimeout
Removing event fallback-query.mycroftai:QuestionQueryTimeout
Removing event fallback-query.mycroftai:QuestionQueryTimeout
Removing event fallback-query.mycroftai:QuestionQueryTimeout
 07:16:13.079 | INFO     |  2785 | QuestionsAnswersSkill | Timeout occured check responses
 07:16:16.350 | ERROR    |  2785 | mycroft-spotify_forslund.spotify:status:88 | HTTPSConnectionPool(host='api.spotify.com', port=443): Pool is closed.

The utterance needs to contain spotify keyword to work. Try "transfer spotify to X"


Android: Configure location of webpack.haul.js file

I have my webpack.haul.js file located in a config folder in my root directory.  During development I've customized my npm start script to pass the config option: haul start --config config/webpack.haul.js
When I try to build a release APK for android using ./gradlew assembleRelease however, the cli cannot find the new location of my haul webpack config, as it expects it to be in the root directory.  I cannot tell how to configure my android/app/build.gradle to pass the --config option along to Haul.
This is my current android/app/build.gradle configuration which matches the docs:
project.ext.react = [
    entryFile: "index.js",
    cliPath: "node_modules/haul/bin/cli.js"
]

I've tried adding the extraPackagerArgs configuration option, but it seems to have no effect
project.ext.react = [
    entryFile: "index.js",
    cliPath: "node_modules/haul/bin/cli.js",
    extraPackagerArgs: ["--config", "config/webpack.haul.js"]
]

How can I configure my android project to pass the --config config/webpack.haul.js option to Haul?

This should be fixed & working in @haul-bundler/cli.


Update istio version to 1.18.2

Description
Changes proposed in this pull request:

Update istio version to 1.18.2

/retest


[Feature request] Ability to reimplement backprop.

Currently there is no straightforward way to do things like implementing different learning rates for different layers. Much less apply learning rate masks for individual unit adjustment.
Torch allows you to easily reimplement backprop however you want. TF seems limited to this: https://www.tensorflow.org/api_docs/python/framework/core_graph_data_structures#Graph.gradient_override_map

What is the abstraction used in Torch/how would it look in TensorFlow?

@alexmonroe87 while there is probably a good feature request here, we need this to be a lot more precise for it to be useful.
One can use different instantiations of optimizers for variables with different learning rates to get per-layer learning rates.  I also don't know what a 'learning rate mask' is, so I'm not sure why it's hard.
It would be helpful to detail the use case more.  It's possible that what you want to do is possible in TF, but may require a little effort to think more declaratively, and we can't know unless we see a more concrete use case.

@vrv Using different optimizer instances for different variables would lead to the problem of computing the same gradients multiple times and probably some other issues. Might also not work as intended with an optimizer like Adam.
What I'm essentially asking for the is the ability to scale a gradient on a per neuron basis with a mask, or as in the use case of individual layers having their own learning rate, a scalar.
Simple example:
Some layer contains 5 neurons and you apply this mask: [1 1 2 1 1] to it, when doing backprop the middle neuron gets updated with twice the learning rate. Though in practice the learning rate will just be made 1 and the mask will specify it.

@alexmonroe87 an example where using different optimizer instances doesn't work would be useful -- I don't think we'd compute the same gradients multiple times if every variable is associated with one optimizer.  "Might not work as intended" is not sufficient evidence for a new feature request :).
As for your mask idea, you can do that today:  Let's say you have a single variable you want to optimize with a mask, using a GradientDescentOptimizer (for simplicity).  Here's my untested semi-TF code:
    v = tf.Variable(...)

    # Define your mask here, matching the shape of the variable.  E.g., could be [1.0, 1.0, 2.0, 1.0, 1.0]
    # if 'v' was a vector of 5.
    mask = tf.constant(..., dtype=tf.float32)

    # Base learning rate
    optimizer = tf.GradientDescentOptimizer(learning_rate)

    # Compute the gradients and have them returned
    grads_and_vars = optimizer.compute_gradients(loss, var_list=[v], ...)

    # Scale the gradient by the mask you want
    masked_grad_and_vars = [(tf.multiply(mask, g), v) for g, v in grads_and_vars]

    # Apply the scaled gradient
    optimizer.apply_gradients(masked_grad_and_vars)

The only difference between this and normal code is the addition of the 'mask' and the 'masked_grad_and_vars'.  Everything else is the same.
Can you explain why this wouldn't work for you?

@vrv Scaling incoming gradients doesn't work for optimizers that adaptively rescale/normalize the gradients. Such as Adam.

There's a solution here that works for adam (specify different optimizer for each layer)

@alexmonroe87 reimplementing backprop for a given op would only allow you to control the output prior to the optimizer, so I still don't understand your feature request.
I'm going to close this particular issue since the request is still unclear after repeated requests for clarfiication.
If you can provide concrete examples of what is not possible / what is hard, (or more evidence of why something is impossible), we can reopen.


package verify ignores the lines inside a range iteration

What happened:
package verify doesn't evaluate the code inside the range syntax in templates
having:
{{ range $i, $v := until (int .Params.BROKER_COUNT) }}
- containerPort: {{ add (int $.Params.EXTERNAL_NODE_PORT) $v}}
  name: node-port-{{ $v }}
{{ end }}

will complain on kubectl kudo package verify with warning
Warnings
parameter "EXTERNAL_NODE_PORT" defined but not used.
package is valid

Using $.Params.EXTERNAL_NODE_PORT outside the range fixes the issue.
What you expected to happen:
package verify to also use the parameters used inside the ranges
How to reproduce it (as minimally and precisely as possible):
KUDO Kafka operator package verify
Anything else we need to know?:
https://github.com/kudobuilder/kudo/pull/1255 introduced the ability to verify parameters used through $
Environment:

Kubernetes version (use kubectl version): 1.17.0
Kudo version (use kubectl kudo version): 0.11.0
Operator:
operatorVersion:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

this is a bug on v0.11.1.  lets make the fix on master and back port for a v0.11.2 release if needed


Set value to search components.

This repository's issues are reserved for feature requests and bug reports.
Do you want to request a feature or report a bug?
Feature Request
Add ngModel support and/or ability to set/get value to td-search-box/td-search-input.
What is the expected behavior?
Ability to pre-populate component.

Added a way to set value in 0.9.1.
Closing and will open a new one later for ngModel support.


Track workflow versions without Papertrail

This will start keeping track of the workflow version in a way that's independent from the Papertrail calculation. It's in a new column so that it can be done while keeping the old method online for now. This will allow us to check that after a while of having this running and tracking, the version numbers are exactly the same as the papertrail ones. After that, a PR will remove the old style and get the serializer to use these new columns.
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

Yes, I want to give those defaults and not-nulls later.

Although I might as well do that now. I was initially leaving them out because I wanted the NULLs to see which ones needed backfilling, but if someone edits a workflow before backfilling the auto-generated number is going to be wrong anyway, so the backfill just ended up needing to overwrite everything. Which means that I should just set the defaults/notnull now rather than postponing. :+1:

Happy to. Were you wanting to run these on production too then, or is that also just an issue on dev setups?


Decide Stage management methods

Decide what to be used for state management: Whether we really need redux or not ? Or simple alternatives like flux architecture or mobX can solve.
Remember the decision you take should be sustainable. In future , frontend should be able to survive, when more features are added

Noted.


Work out number of required votes

Currently hardcoded - should be based on contributor numbers as in contributing guide.

Not working right, as not all contributors are stored in the local database.

deprecated by rule change


Is this project under development?

Quick question: Is this project still under development? I posted actually two simple questions but no reply until now. If the reply may be slow, or if this project isn't under development, please add some information in README.md, thanks!

Hi @GopherJ :wave:
This project is currently only receiving sporadic attention from myself  because I have a lot of other commitments to keep up with, but have been working on getting it into a more contributable state for a while.
I'll try make some time to catch up on new issues. Thanks for your patience!

@GopherJ can I close this?


wram is banked, bank 0 is missing.

A memory map may be found here:
https://gbdev.gg8.se/wiki/articles/Memory_Map

nvmd, it looks like I somehow got a messed up load.


[WIP] feat: Add event detail activity

Fixes #453

@mejariamol Please review

@iamareebjamal
Why separate activity for it? I think it should be hosted in MainActivity itself.
And we already showing event image in the nav head.

How can we show this in MainActivity?

How can we show this in MainActivity?

Only the content which is shown in 2nd screen shot. Event Image is not required to show in the about page separately as we are showing it already in the nav header.

But how will we show this info in MainActivity? MainActivity is just a host for fragments.
About the image, the screen also shows logo. I obviously showed the image because it was an activity and collapsing toolbar could be used. Also, option for editing the event can be added in future in this activity.
Detail views are generally activities

But how will we show this info in MainActivity? MainActivity is just a host for fragments.

I am saying host AboutFragment in MainActivity.

I think detail views should be shown in separate activities. Because navigationally, they are not flat with the hierarchy. Like track -> track detail should not be in same frame (activity) and some isolated functionalities just related to event info can be handled through this. This is a standard in material guidelines to show a detail view with a collapsing toolbar. Using collapsing toolbar in Fragments is not easy and the stack management will be disrupted due to it. Same with the User Detail
It should be handled in another activity as well. This will also make it easy to create master detail in future because the natural process for it is - mobile -> detail activity | tablet -> detail fragment

@mejariamol Please review

@mejariamol Please review now


Use gethostbyname_r if available

Fixes # 7228
The function BIO_get_host_ip uses gethostbyname, which is not thread safe
and hence we grab a lock. In multi-threaded applications, this lock sometimes
causes performance bottlenecks.
This patch uses the function gethostbyname_r (thread safe version), when
available.

Are the travis-ci failures false alarms? Looks like its erroring on unrelated things.

Most look relevant:
../libcrypto.a(b_sock.o):b_sock.c:(.text+0x6b5): undefined reference to `gethostbyname_r'

The conditioning #if defined maze is the likely culprit.

AppVeyor failures also look relevant.

There needs to be a preprocessor guard around the definition of BIO_gethostbyname_r.

Please do not implement BIO_gethostbyname_r, just use gethostbyname_r directly.  There's absolutely zero need to create our own function that's essentially just a very thin wrapper.  I know there is a BIO_gethostbyname, but that one was created (as far as I can see and remember) for caching purposes, which has later turned out to be a mistake.

Sure. Will change the patch to directly call gethostbyname_r. Intially i wanted to add the corresponding BIO function ,  so i could include the logic of retrying the call, in case on ERANGE error.
Now, with Pauli's comment, i have already removed the retry logic, so i can remove corresponding
BIO function itself.

You really do not want to use gethostbyname(), please use
getaddrinfo() instead, or BIO_lookup_ex()

Hi Kurt

Are you suggesting that in 1.0.2 branch ,  you want to replace the following sequence of lines
/* do a gethostbyname */
CRYPTO_w_lock(CRYPTO_LOCK_GETHOSTBYNAME);
locked = 1;
he = BIO_gethostbyname(str);

with
getaddrinfo(...)

BIO_lookup_ex is available in later releases and i am not sure, if you want me to backport it to 1.0.2 ?

In BIO_lookup_ex , we invoke the getaddrinfo under #ifdef AI_PASSIVE - in the #else part , we do end up invoking gethostbyname.  So should i also implement similar logic ?

I didn't know this was about the 1.0.2 version.
So I see a few options:

Use getaddrinfo(), which is thread safe and support IPv6.
As you point out, you can also do a lock around the
BIO_gethostbyname() yourself. We can't do it internally,
because the API is not thread safe, but you can clearly
do it outside the function.

The AI_PASSIVE is used to detect that getaddrinfo() is available,
and we fall back to gethostbyname() in case it's not. In that code
we can take a lock internally because we convert it to something
that is thread safe to use.
I doubt that your code needs to be able to run on something that
does not provide getaddrinfo().

@kroeckx
In 1.0.2 , BIO_get_host_ip function ( which invokes  BIO_gethostbyname under lock)  expects to see a ipv4 address only. It does not process IPV6 address as seen in the code snippet below Line 151.
141    /* do a gethostbyname /
142    CRYPTO_w_lock(CRYPTO_LOCK_GETHOSTBYNAME);
143    locked = 1;
144    he = BIO_gethostbyname(str);
145    if (he == NULL) {
146        BIOerr(BIO_F_BIO_GET_HOST_IP, BIO_R_BAD_HOSTNAME_LOOKUP);
147        goto err;
148    }
149
150    / cast to short because of win16 winsock definition */
151    if ((short)he->h_addrtype != AF_INET) {
152        BIOerr(BIO_F_BIO_GET_HOST_IP,
153               BIO_R_GETHOSTBYNAME_ADDR_IS_NOT_AF_INET);
154        goto err;
155    }
In my opinion, for 1.0.2 , it is sufficient to use gethostbyname_r , where available and  fallback to use gethostbyname , where thread safe version is not available.
Please let me know, if there a need to use to getaddrinfo ?

My suggestion is what you do in your code, not what to change in openssl.
The 1.0.2 branch is in security fixes only mode, and this clearly
isn't a security fix.
Calling gethostbyname_r() instead of gethostbyname() in
BIO_gethostbyname() makes no sense at all, BIO_gethostbyname()
does not provide a thread safe API.
BIO_gethostbyname() is also deprecated

It seems to be about BIO_get_host_ip() instead of
BIO_gethostbyname(). But the comment about security fixes only and
deprecated are still valid.

The 1.0.2 branch is in security fixes only mode, and this clearly
isn't a security fix.

@kroeckx Several points in your post are not correct. Please understand the OpenSSL project better.
https://www.openssl.org/policies/releasestrat.html

Letter releases, such as 1.0.2a, exclusively contain bug and security fixes and no new features.

Version 1.0.2 will be supported until 2019-12-31 (LTS).

During the final year of support, we do not commit to anything other than security fixes. Before that, bug and security fixes will be applied as appropriate.

We are further away than 1 year away from end of support so bug fixes as well as security fixes are accepted.

BIO_gethostbyname() is also deprecated

BIO_gethostbyname() was deprecated in 1.1.0. It is not deprecated in 1.0.2.

Version 1.0.2 will be supported until 2019-12-31 (LTS).

Thanks for correcting me, I thought it was until the end of this year.

Do you have any suggestions on what needs to be changed to get this integrated into OpenSSL? There is nothing to be changed within our code unless we can somehow not use BIO_do_connect to do network connections as that is the call path that hits BIO_gethostbyname. Call path:
  BIO_do_handshake
    BIO_ctrl (with BIO_C_DO_STATE_MACHINE)
      b->method->ctrl (actually conn_ctrl)
        case BIO_C_DO_STATE_MACHINE
          conn_state
            case BIO_CONN_S_GET_IP
              BIO_get_host_ip
                BIO_gethostbyname
                  gethostbyname

@levitte What still needs to be done (after the tests pass again from the comment changes) before this is ready for submission?

(also added some markers to make it clear to anyone who'd still wonder that this is a 1.0.2 only change)

travis-ci failed due to build timeout with the following error:
"No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself"
This looks like environmental issue.  Is there a way for me to kick off rebuild ?

That Travis issue isn't due to us, i.e unrelated to this PR as far as I can tell. We can ignore it.

How do we contact the CIs to get this change merged in?

How do we contact the CIs to get this change merged in?

Err, that's not how it works...  We have a peer review process.
See how some of us have the "Member" 'badge' by our names (such as @kroeckx and I)?  First, two of us have to approve, and second, one of us does the actual merge.

@kroeckx Could you give the review some actionable comments on what specifically needs to be changed in order to get your approval? Your previous suggestions would cause breakages on systems where gethostbyname_r does not exist so we can't make those.

How do we know those defines won't turn off some required functions from the headers, especially _BSD_SOURCE?

They do not turn anything off, only on. For some of the defines, they might actually change the prototype of the function depending on the value of the define, but that is not a problem for those 2 defines.

@levitte Do you agree that @kroeckx 's suggested changes won't cause other issues?

Yes, I do agree, what @kroeckx proposes should work for the purposes of this PR.  It's also perfectly appropriate, seeing that our libraries are applications (in the more abstract sense of the word) of the functionality provided by the C library, so we are free to request (via these feature macros) whatever feature that can be made available, and looking at __GLIBC__ to check actual availability is the perfect answer to my question about other platforms.
So yup, I'm satisfied with his proposal.

Yes, I do agree, what @kroeckx proposes should work for the purposes of this PR. It's also perfectly appropriate, seeing that our libraries are applications (in the more abstract sense of the word) of the functionality provided by the C library, so we are free to request (via these feature macros) whatever feature that can be made available, and looking at __GLIBC__ to check actual availability is the perfect answer to my question about other platforms.
So yup, I'm satisfied with his proposal.

Done.  pushed the new diff

@kroeckx Just pinging you if you are satisfied with the changes?

The Travis failure looks unrelated.

The Travis failure looks unrelated.

Agreed.

@levitte Because that process may take some time, assuming that @smanikantan gets that signed, is the checkin otherwise fine and will go in once it is signed and verified as correct?

Checked, and @smanikantan, I don't see your git id<EMAIL_ADDRESS>in our CLA db. I do not agree that this is a trivial change, so I must ask you to please submit a CLA, here are the instructions to do so: https://www.openssl.org/policies/cla.html

Hi @levitte ,  I have sent a signed CLA to<EMAIL_ADDRESS>Thanks
Mani

CLA to the database.
It would be best if the CLA:trivial were removed from the commit message.

I'll take care of.

There, merged.  Thank you, @smanikantan
2b872562f56b6039179104657edddac22712d994 Use gethostbyname_r if available

There, merged. Thank you, @smanikantan
2b87256 Use gethostbyname_r if available

Thanks !


feat: implement abbreviated manifest

implement https://github.com/verdaccio/verdaccio/issues/2792
add the extra field time field. Discussion: https://github.com/verdaccio/verdaccio/discussions/3315

How to test

Use pnpm 6

pnpm install
pnpm build
pnpm debug

@zkochan this should be enough for start experimenting, let me know if you need more.

Thank you! You're awesome!


⚠ Bofrost.lu has degraded performance

In c918235, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 999 ms

Resolved: Bofrost.lu performance has improved in eb8f8f7 after 21 minutes.


Pip 24.2 take2

Well the patch work I did for the other PR pip-24.1.2 made the vendoring of this one go a lot more smoothly.  It is still my preference to cut a release of the other one and then merge this one into main for a later release after that (assuming tests will pass still).
The checklist

[ ] Associated issue
[ ] A news fragment in the news/ directory to describe this fix with the extension .bugfix.rst, .feature.rst, .behavior.rst, .doc.rst. .vendor.rst. or .trivial.rst (this will appear in the release changelog). Use semantic line breaks and name the file after the issue number or the PR #.

Replaced by:  https://github.com/pypa/pipenv/pull/6263


Enable posting user stats to Tuneme slack channel

@miltontony @saeedmb8 ready for review.
I'm not sure adding them to settings/base.py is the best choice since it will fire stats for QA as well but it needs to be testable on somehow.
We need to decide if QAing this should hold up release of the stuff that's already in QA before we merge.

@miltontony @saeedmb8 ready for review

:+1: - small question around the webhook url

@miltontony just checking that removing the webhook url entirely was the write thing to do. Or do I need to leave the variable name in the settings and overwrite it in puppet?

Removing it was fine
:+1:

@saeedmb8 testing your email


Add Apps#find and Apps#all endpoint

@chute/backend

:+1:

:+1:

:+1:

:+1:

hi @ctide !!

hi @gauravs! :)


Voronoi Analysis crashes for large structures

I currently run melting point calculation with pyscal 2.10.6 and the voronoi analysis crash the jupyter kernel, so some structures, while others work fine.

It seems the cache is not cleared after the calculation of the voronoi volume, so after multiple large structures the kernel crashes.

I uploaded an example here https://github.com/jan-janssen/pyscal_voronoi_crash

Thanks! I will look into it now

Thanks! I will look into it now

Fixed! Will create a release soon.

Fixed! Will create a release soon.

Thanks - the issue is now fixed.

Thanks - the issue is now fixed.


[question] How to change device settings via MQTT

I have a bunch of touch-lightswitches with backlight. Previously with Z-Wave in HASS I could instruct it to disable and enable this on specific cases.
I have tried creating a "backlight" device/topic based on this particular setting but it doesn't get created in Z2M.
Is this something I can achieve with Z2M as well? If so, how should it be done?

I'm close, but for some reason it's not accepting my provided arguments.
zwave/_CLIENTS/ZWAVE_GATEWAY-zwave2mqtt/api/setConfigParam/set {"args": [{"nodeid": 44, "param": "112-1-2", "value": 0}]}
zwave/_CLIENTS/ZWAVE_GATEWAY-zwave2mqtt/api/setConfigParam {"success":false,"message":"This OpenZwave method requires at least 3 argument(s): nodeid, param, value"}

latter did the trick. For some reason the network was being refreshed at the time of testing (the product names were empty).


KeyError: 'bugs' in the regression_set_status_flags tool

2023-02-16 05:31:47,795 - ERROR - Tool regression_set_status_flags
Traceback (most recent call last):
  File "/home/rm_bot/relman-auto-nag/auto_nag/bzcleaner.py", line 839, in run
    self.send_email(date=date)
  File "/home/rm_bot/relman-auto-nag/auto_nag/bzcleaner.py", line 762, in send_email
    data = self.get_email_data(date)
  File "/home/rm_bot/relman-auto-nag/auto_nag/bzcleaner.py", line 680, in get_email_data
    bugs = self.get_bugs(date=date)
  File "/home/rm_bot/relman-auto-nag/auto_nag/scripts/regression_set_status_flags.py", line 212, in get_bugs
    bugs = super().get_bugs(*args, **kwargs)
  File "/home/rm_bot/relman-auto-nag/auto_nag/bzcleaner.py", line 454, in get_bugs
    timeout=self.get_config("bz_query_timeout"),
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/libmozdata/bugzilla.py", line 176, in wait
    self.wait_bugs()
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/libmozdata/bugzilla.py", line 187, in wait_bugs
    r.result()
  File "/usr/local/lib/python3.7/concurrent/futures/_base.py", line 425, in result
    return self.__get_result()
  File "/usr/local/lib/python3.7/concurrent/futures/_base.py", line 384, in __get_result
    raise self._exception
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/requests/sessions.py", line 653, in send
    r = dispatch_hook('response', hooks, r, **kwargs)
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/requests/hooks.py", line 31, in dispatch_hook
    _hook_data = hook(hook_data, **kwargs)
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/libmozdata/bugzilla.py", line 551, in __bugs_cb
    for bug in res.json()["bugs"]:
KeyError: 'bugs'

Similar error:
2023-02-18 02:32:24,671 - ERROR - Tool needinfo_regression_author
Traceback (most recent call last):
  File "/home/rm_bot/relman-auto-nag/auto_nag/bzcleaner.py", line 839, in run
    self.send_email(date=date)
  File "/home/rm_bot/relman-auto-nag/auto_nag/bzcleaner.py", line 762, in send_email
    data = self.get_email_data(date)
  File "/home/rm_bot/relman-auto-nag/auto_nag/bzcleaner.py", line 680, in get_email_data
    bugs = self.get_bugs(date=date)
  File "/home/rm_bot/relman-auto-nag/auto_nag/scripts/needinfo_regression_author.py", line 169, in get_bugs
    self.retrieve_regressors(bugs)
  File "/home/rm_bot/relman-auto-nag/auto_nag/scripts/needinfo_regression_author.py", line 113, in retrieve_regressors
    include_fields=["id", "assigned_to"],
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/libmozdata/bugzilla.py", line 176, in wait
    self.wait_bugs()
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/libmozdata/bugzilla.py", line 187, in wait_bugs
    r.result()
  File "/usr/local/lib/python3.7/concurrent/futures/_base.py", line 432, in result
    return self.__get_result()
  File "/usr/local/lib/python3.7/concurrent/futures/_base.py", line 384, in __get_result
    raise self._exception
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/requests/sessions.py", line 533, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/requests/sessions.py", line 653, in send
    r = dispatch_hook('response', hooks, r, **kwargs)
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/requests/hooks.py", line 31, in dispatch_hook
    _hook_data = hook(hook_data, **kwargs)
  File "/home/rm_bot/relman-auto-nag/venv/lib/python3.7/site-packages/libmozdata/bugzilla.py", line 551, in __bugs_cb
    for bug in res.json()["bugs"]:
KeyError: 'bugs'

2024-06-07 16:40:30,663 - ERROR - Rule no_crashes
Traceback (most recent call last):
  File "/home/rm_bot/bugbot/bugbot/bzcleaner.py", line 850, in run
    self.send_email(date=date)
  File "/home/rm_bot/bugbot/bugbot/bzcleaner.py", line 766, in send_email
    data = self.get_email_data(date)
  File "/home/rm_bot/bugbot/bugbot/bzcleaner.py", line 684, in get_email_data
    bugs = self.get_bugs(date=date)
  File "/home/rm_bot/bugbot/bugbot/rules/no_crashes.py", line 156, in get_bugs
    data = super(NoCrashes, self).get_bugs(date=date, bug_ids=bug_ids)
  File "/home/rm_bot/bugbot/bugbot/bzcleaner.py", line 459, in get_bugs
    ).get_data().wait()
  File "/home/rm_bot/bugbot/venv/lib/python3.10/site-packages/libmozdata/bugzilla.py", line 180, in wait
    self.wait_bugs()
  File "/home/rm_bot/bugbot/venv/lib/python3.10/site-packages/libmozdata/bugzilla.py", line 191, in wait_bugs
    r.result()
  File "/usr/local/lib/python3.10/concurrent/futures/_base.py", line 458, in result
    return self.__get_result()
  File "/usr/local/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/rm_bot/bugbot/venv/lib/python3.10/site-packages/requests/sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/rm_bot/bugbot/venv/lib/python3.10/site-packages/requests/sessions.py", line 710, in send
    r = dispatch_hook("response", hooks, r, **kwargs)
  File "/home/rm_bot/bugbot/venv/lib/python3.10/site-packages/requests/hooks.py", line 30, in dispatch_hook
    _hook_data = hook(hook_data, **kwargs)
  File "/home/rm_bot/bugbot/venv/lib/python3.10/site-packages/libmozdata/bugzilla.py", line 555, in __bugs_cb
    for bug in res.json()["bugs"]:
KeyError: 'bugs'


Enable for suse and corrected for windows service

Description
Enable to install on suse 12+ and correct regex call on creating service on windows os
Issues Resolved
Installing on windows server node 2012R2
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

PR change to https://github.com/chef-cookbooks/push-jobs/pull/118


Godot crashing on launch

Godot version:
This was on Godot v3.2.3-stable
OS/device including version:

I'm using Windows 10 on a laptop, specs:
Processor:Intel(R) Core(TM) i3-8130U CPU @ 2.20GHz   2.20 GHz
Installed RAM:8.00 GB
64bit os
Issue description:
This issue is one i've had before, and i didn't know how to fix it then either. I've been using Godot for a couple of months perfectly fine, and i've been following along with tutorials and the like. I was in the middle of one of said tutorials when i decided to stop coding after numerous frustrations in code constantly having errors in it and not running right. When I went to close out godot, I accidentally closed out the console instead, closing the whole thing. I didn't think anything of it, until I went to reopen godot later, and found that it wouldn't launch. Everytime I click the button, the console will open up quickly on the screen, before immediately closing. No matter what I do I can't figure out why it won't launch, or what i did to break it.
Steps to reproduce:
Close out of the console immediately after saving a scene??? I genuinely don't know how this happened.

@TheElementalForce Please upload a minimal reproduction project to make this easier to troubleshoot.

I don't think I can make a minimal reproduction project unfortunately as i am incapable of opening godot to make it, and don't even know what i did in the first place to cause this, so i don't know how to replicate it.
When i try and run it through a command prompt, I get only this output:
Godot Engine v3.2.3.stable.official - https://godotengine.org

@TheElementalForce Can you try resetting the editor settings and data by renaming the %APPDATA/Godot folder to something else (such as Godot2), then restarting the editor?

Sadly had no effect :/

Try to delete the cache directory %TEMP%/Godot and see if it still crashes on startup.


Alfred workflow can't find my meetings

Hey Caleb,
the alfred workflow seems not working properly on my mac.
It doesn't find any meetings in my calendar.
Could you please help me?
thanks
Stefan

I have this problem also. It only shows default search options.

Same issue here; with the exact same screens as @nonameolsson has above.

Same issue here; with the exact same screens as @nonameolsson has above.

@nonameolsson I noticed a fix here:
https://www.alfredforum.com/topic/16215-open-conference-url-launch-the-zoommeet-call-for-your-next-calendar-event/?do=findComment&comment=82854
It requires a modification to line 77 of cache.py
At least the cache is updated now, but I'm not sure I can link it to my meetings yet.

@nonameolsson I noticed a fix here:
https://www.alfredforum.com/topic/16215-open-conference-url-launch-the-zoommeet-call-for-your-next-calendar-event/?do=findComment&comment=82854
It requires a modification to line 77 of cache.py
At least the cache is updated now, but I'm not sure I can link it to my meetings yet.

Actually, I resolved the issue by just downloading the latest version from the github.
It looks like the version I had downloaded was out of date.

Actually, I resolved the issue by just downloading the latest version from the github.
It looks like the version I had downloaded was out of date.

@tiems90 Thanks for the feedback. This should be fixed in the latest version, as you found. As such, I am closing this ticket.
Please open a new ticket if you encounter any additional issues.

@tiems90 Thanks for the feedback. This should be fixed in the latest version, as you found. As such, I am closing this ticket.
Please open a new ticket if you encounter any additional issues.


[fix] Fix mem leak in pipeline API

Free GstTensorsConfig using gst_tensors_config_free
Initialize ml_tensors_info_s before set a new one

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #342. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @anyj0527, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nnstreamer-api/ci/repo-workers/pr-checker/342-202306141055330.34387803077698-53f16fb26814c2aafb47f70e84a5a51180c26a17/.


feat: add new pen icon

This PR adds a new pen icon designed by @rafael-sepeda

Hey, @phllipo thanks for doing this is great to see you contribute.
I am not familiar with the process of icon generation, but I wonder if you can throw a screenshot of the icon just to make sure the SVG is adding what we intended to.
I wonder. What has been done wrong from our side that the pull request template didn't show for you?
https://github.com/freenowtech/wave/blob/main/.github/workflows/PULL_REQUEST_TEMPLATE/pull_request_template.md
Thanks a lot! Looking forward to double-checking the icon before approval to merge!

Hi, @duranmla,
the template was indeed not shown to me.
I updated my initial comment to hopefully cover all the points.

@all-contributors add @phllipo for the code contribution with icons


Dependent questions don't work as expected

Cookieninja version: cookieninja = { git = "git@github.com:cookieninja-generator/cookieninja.git", branch = "main" } (working of this commit hash atm 30f30f920c26678ec6dabecf8a44d75421e8e9f4)
Operating System: macos

Description:
Given this cookiecutter.json
{
    "bool": false,
    "test2?{{cookiecutter.bool}}": "true",
    "env": ["dev", "customer", "internal"],
}

I would expect the test2 question to show up if I set bool to true instead it depends on the default value rather then the answer to the question
What I've run:
# Tested with True
bool [False]: True
ctx: OrderedDict([('cookiecutter', OrderedDict([('bool', False), ('test?{{cookiecutter.bool}}', 'true'), ('env', ['dev', 'customer', 'internal'])]))]) env: <cookieninja.environment.StrictEnvironment object at 0x10a93adf0> key: test?{{cookiecutter.bool}}
actual_key: test dependant_variable: {{cookiecutter.bool}}
bool: False should present: False
Select env:
1 - dev
2 - customer
3 - internal
Choose from 1, 2, 3 [1]:

# Test with true
bool [False]: true
ctx: OrderedDict([('cookiecutter', OrderedDict([('bool', False), ('test?{{cookiecutter.bool}}', 'true'), ('env', ['dev', 'customer', 'internal'])]))]) env: <cookieninja.environment.StrictEnvironment object at 0x104bb3dc0> key: test?{{cookiecutter.bool}}
actual_key: test dependant_variable: {{cookiecutter.bool}}
bool: False should present: False
Select env:
1 - dev
2 - customer
3 - internal
Choose from 1, 2, 3 [1]:

It only seems to display the question if the default value is true, in which case it still ignores the answer.

flipsed was this issue resolved?


[BUG] arm64 builds failing because of file-clobbering issues between 'xorg-xorgproto' and 'xorg-xproto'

Describe the bug
conda-cpp-build jobs (arm64 only) are failing for releases 24.12 and 25.02, with errors like this:
This transaction has incompatible packages due to a shared path.
  packages: conda-forge/linux-aarch64::xorg-xorgproto-2024.1-h86ecc28_1, conda-forge/linux-aarch64::xorg-xproto-7.0.31-h57736b2_1008
  path: 'include/X11/Xprotostr.h'

(24.12 build link)
(25.02 build link)
The corresponding amd64 jobs are not failing in this way.
Steps/Code to reproduce bug
See those CI links above.
Expected behavior
Builds to succeed.
Environment details (please complete the following information):
See logs from those build links above.
conda env from the 24.12 job (click me)
The following NEW packages will be INSTALLED:

    _openmp_mutex:             4.5-2_gnu            conda-forge
    bzip2:                     1.0.8-h68df207_7     conda-forge
    cairo:                     1.18.2-hb60a174_0    conda-forge
    cuda-version:              11.8-h70ddcb2_3      conda-forge
    font-ttf-dejavu-sans-mono: 2.37-hab24e00_0      conda-forge
    font-ttf-inconsolata:      3.000-h77eed37_0     conda-forge
    font-ttf-source-code-pro:  2.038-h77eed37_0     conda-forge
    font-ttf-ubuntu:           0.83-h77eed37_3      conda-forge
    fontconfig:                2.15.0-h8dda3cd_1    conda-forge
    fonts-conda-ecosystem:     1-0                  conda-forge
    fonts-conda-forge:         1-0                  conda-forge
    freetype:                  2.12.1-hf0a5ef3_2    conda-forge
    gdk-pixbuf:                2.42.12-ha61d561_0   conda-forge
    icu:                       75.1-hf9b3779_0      conda-forge
    lerc:                      4.0.0-h4de3ea5_0     conda-forge
    libdeflate:                1.22-h86ecc28_0      conda-forge
    libdicom:                  1.0.5-h31becfc_1     conda-forge
    libexpat:                  2.6.4-h5ad3122_0     conda-forge
    libffi:                    3.4.2-h3557bc0_5     conda-forge
    libgcc:                    14.2.0-he277a41_1    conda-forge
    libgcc-ng:                 14.2.0-he[943](https://github.com/rapidsai/cucim/actions/runs/12229295153/job/34108856764#step:8:944)1aa_1    conda-forge
    libglib:                   2.82.2-hc486b8e_0    conda-forge
    libgomp:                   14.2.0-he277a41_1    conda-forge
    libiconv:                  1.17-h31becfc_2      conda-forge
    libjpeg-turbo:             3.0.0-h31becfc_1     conda-forge
    liblzma:                   5.6.3-h86ecc28_1     conda-forge
    libnvjpeg:                 <IP_ADDRESS>-h6ae07c9_0 nvidia
    libnvjpeg-dev:             <IP_ADDRESS>-h54d441a_0 nvidia
    libpng:                    1.6.44-hc4a20ef_0    conda-forge
    libsqlite:                 3.47.0-hc4a20ef_1    conda-forge
    libstdcxx:                 14.2.0-h3f4de04_1    conda-forge
    libstdcxx-ng:              14.2.0-hf1166c9_1    conda-forge
    libtiff:                   4.7.0-hca96517_2     conda-forge
    libuuid:                   2.38.1-hb4cce97_0    conda-forge
    libwebp-base:              1.4.0-h31becfc_0     conda-forge
    libxcb:                    1.16-h57736b2_1      conda-forge
    libxml2:                   2.13.5-h2e0c361_1    conda-forge
    libzlib:                   1.3.1-h86ecc28_2     conda-forge
    nvtx-c:                    3.1.0-h8af1aa0_1     conda-forge
    openjpeg:                  2.5.2-h0d9d63b_0     conda-forge
    openslide:                 4.0.0-h8d5e3c6_1     conda-forge
    pcre2:                     10.44-h070dd5b_2     conda-forge
    pixman:                    0.44.2-h86a87f0_0    conda-forge
    pthread-stubs:             0.4-h86ecc28_1002    conda-forge
    xorg-kbproto:              1.0.7-h57736b2_1003  conda-forge
    xorg-libice:               1.1.1-h57736b2_1     conda-forge
    xorg-libsm:                1.2.4-hbac51e1_1     conda-forge
    xorg-libx11:               1.8.9-h08be655_1     conda-forge
    xorg-libxau:               1.0.11-h86ecc28_1    conda-forge
    xorg-libxdmcp:             1.1.5-h57736b2_0     conda-forge
    xorg-libxext:              1.3.6-h57736b2_0     conda-forge
    xorg-libxrender:           0.9.11-h57736b2_1    conda-forge
    xorg-xextproto:            7.3.0-h57736b2_1004  conda-forge
    xorg-xorgproto:            2024.1-h86ecc28_1    conda-forge
    xorg-xproto:               7.0.31-h57736b2_1008 conda-forge
    zlib:                      1.3.1-h86ecc28_2     conda-forge
    zstd:                      1.5.6-h02f22dd_0     conda-forge

Additional context
The last time this happened, the root cause was "some migration in the xorg-* projects is in progress, just give it a bit more time to reach all the projects": https://github.com/rapidsai/cucim/pull/785/files#r1777636427
That might be happening now, too.

Thanks James! 🙏
xorg-xproto is deprecated. Packages are recommended to move to xorg-xorgproto
Since our last discussion a linter hint was added: https://github.com/conda-forge/conda-forge-pinning-feedstock/pull/6688
Also am suggesting conda-forge include a migrator: https://github.com/regro/cf-scripts/issues/3360
Likely there is some dependencies of cuCIM that isn't fully migrated. The fact this happens for some architectures and not others suggests there is some package rebuild that failed on a couple CI jobs somewhere and needs a restart. If we can identify which one, am happy to click restart there

without any code changes most recent cucim builds did succeed (build link).
However, I still put up #801 temporarily allowing clobbering, in case other steps in the xorg-* migration result in the solver again finding a set of packages with conflicts.
Whenever the xorg-* migration is done (maybe when https://github.com/regro/cf-scripts/issues/3360 is closed, if a formal migration isn't set up), we could try reverting #801 and then close this.


Laravel 5.1.3 Session id problem

Request->session()->id gives f.e. 2bbe1c15d7ebfa78302f82f0d63b05017b42b5ad
But Session::getId() gives random string.
Is it normal?

That's not possible. The getId method literally returns the id property.


Small cosmetic improvements

Added shortcut Positional() for better balanced API.
Replaced "Positional arguments" with "Positionals" in help message.
Added newline after positional arguments in help message.

Guys! This is only my vision but i think it will looks slightly better.

I'll try!

I just created new pull request for that: https://github.com/integrii/flaggy/pull/66


Fix issue with nested dynamic objects that don't exist.

Thanks! Any chance you will publish this fix to your nuget package soon?

Updated NuGet 1.5.2


Add native realm when configuring SAML realm

This PR configures the native realm when the SAML realm is configured.
The native realm is configured by default when no other realms are specified, but must be explicitly configured when other realms are configured.
Closes #290

run
npm run links -- --branch 7.2

before merging


[Release 2.9.0] Remove k-NN

Description
[Release 2.9.0] Remove k-NN  due to https://github.com/opensearch-project/k-NN/issues/975
Issues Resolved
Part of https://github.com/opensearch-project/opensearch-build/issues/3616
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@prudhvigodithi there are failures in the checks.

Th checks failed for manifests/3.0.0/opensearch-3.0.0.yml, as we are all aware 3.0.0 is under development, @peterzhuamazon please let me know if you are ok to override and merge.
@bbarani


[Epic] VPN support

@gbraad is this a coming fix for crc in the future?  I have cisco anyconnect installed by my company that I cannot disable, and I think it is causing me issues during "crc start" similar to this issue https://github.com/code-ready/crc/issues/908

No timeframe is given as issues are encountered while implementing

Planned to be reviewed again.

@guillaumerose Do you have any suggestions on how to tackle this?

Docker Desktop solves this by using vpnkit.
vpnkit runs as user on the host and connects to a small daemon inside the VM using named pipes/vsock. The daemon in the VM is in fact a tap interface. The daemon transfers all the traffic to vpnkit so that vpnkit handles all the traffic as the user.
In this architecture, vpnkit is a client because of issue with vsock on older version of Windows. It could be reversed now.

I started an experiment in https://github.com/guillaumerose/gvisor-tap-vsock. It reproduces vpnkit without ocaml code. For the moment, it's more a toolkit to solve the VPN issue than the solution itself.

We should break down work in new tasks as the spike has concluded.

https://github.com/code-ready/snc/pull/234 addresses the first two points.
For the 4th point, the host part is part of the daemon and the config to enable it is network-mode=vsock https://github.com/code-ready/crc/pull/1526

https://github.com/code-ready/snc/pull/234
https://github.com/code-ready/crc/pull/1526
https://github.com/code-ready/machine/pull/43
https://github.com/code-ready/machine-driver-hyperkit/pull/20
https://github.com/code-ready/machine/pull/46

First bits are in master branch. It can be used on the 3 platforms.
How to test it:

cleanup everything, ~/.crc and hosts file
crc config set network-mode vsock
run crc setup
run crc daemon in a second terminal
run crc start as usual.

What is still missing:

DNS records for apps created by user is not working.
Workaround: manually add the dns record in the hosts file.
Port forwarding is static. Only VM port 443, 22 (accessible through port 2222 on the host) and 6443 are reachable.
Communication still happen between crc and his components using localhost and random ports.

cleanup everything, ~/.crc and hosts file
which means crc delete; crc cleanup; rm -rf ~/.crc, and finally remove all xxx.crc.testing entries from /etc/hosts

bump

@fasho why bump? Please look at https://github.com/code-ready/crc/wiki/VPN-support--with-an--userland-network-stack

VPN support is available by default on macOS and Linux.
It will be activated by default by the MSI on Windows in the next release. Until then, please refer to https://github.com/code-ready/crc/wiki/VPN-support--with-an--userland-network-stack
Still, we are missing end-to-end tests for this.


How to see private repo size?

I can see this in the docs "Generate a Github personal access token from here and click on extension icon to add it." But, could you elaborate?

@warmwhisky done. see updated README.md

Does it still work? I am unable to use it with the mentioned method

I think it's missing a crucial step regarding the token's scope. See my issue #24


Fix project scrap

this error is in Travis?

oh sorry. Tomorrow will fix it

@danieljoppi Indeed test failed because travis can't get any data from linkedin. But locally it's passed.

@danieljoppi and i have some problems on local machine. After some amount of requests to linkedin, i received empty data. Can linkedin blocked me from their side?


Update scroll-tracker.js

added extra header for tracking on /government/publications/see-potential-case-studies-and-guidance-for-employers/see-potential-case-studies-and-guidance-for-employers

Looks ok to me :+1:


Add host species column to the study search panel

When searching for microbiome studies, one often must choose them from similar biological hosts. This column in the studies search page would make the search more effective and less time-consuming.

@crusher083, thank you for the feedback.
In case, it helps, have you used https://github.com/biocore/redbiom? It basically has access to all the public processed data in Qiita and allows for "complex" searches.

@wasade, based on your redbiom experience, do you have any suggestions about this? I think is possible to add a column to the study page with the the host_scientific_name, host_common_name, or host_taxid. What do you think?

Just a quick update about this.

As first step to allow this, we have to improve the speed of the study listing loading by improving the SQL, which didn't work, or removing the "non" important/used parts, which actually worked nicely. Just FYI, the main way to test this is loading the admin study listing, which before used to take 20.4 s ± 276 ms per loop (mean ± std. dev. of 7 runs, 1 loop each) and now it takes 2.02 s ± 6.57 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
Then we can start adding new things and see how long it will take. In this PR we can actually add the new information; however, I don't think is worth uncommenting those sections because this new functionality will raise the time to: 7.48 s ± 343 ms for admins and even worst it will raise the public listing from 760 ms ± 4.4 ms to 3.4 s ± 137 ms.

IMOO the correct implementation (which will not be ready for the next deployment) should create a new table with the columns we want to display and add a rule so this new table gets populated when there is an insert or update to the tables.


Meta: Upgrade Travis-CI to new infrastructure

This is now ready for review.
PS: There's no dramatic change in run time with this change although I did notice that builds trigger quicker than the older version.

Thanks!


Get Current Status

How to get current xp, lvl coins skinname startmass... ?   Cuz the Agar.io servers send the info when connected and its parsed by the client.   So is it possible to implement it here?

There is open issue about it https://github.com/pulviscriptor/agario-client/issues/78
I can't implement it.


Add max retry option for curtain

:recycle: Current situation
The BLE connection can often fail on the first attempt.
:bulb: Proposed solution
Use the max retry functionality from the bot device.
Closes #612.

@dnicolson, I am going to move the maxRetry to be at the deviceType level instead of bot.maxRetry it will just be maxRetry.

this has been pushed to beta 2.4.0


Improve string handling

Some small improvements around string handling.
For i:s:1 /comm == comm/ { printf("%s", "yay"); } it  reduces the code quite a bit as it avoids doing 64 comparisons:
4319: perf_event  name 1  tag 36d38e2a660a3854  gpl
	xlated 3032B  jited 1722B  memlock 4096B  map_ids 4477
4320: perf_event  name 1  tag 40906b14480b33ff  gpl
	xlated 9424B  jited 6216B  memlock 12288B  map_ids 4478

There is still a lot left to do (#934) but these should be "safe"

@ajor we have this one
https://github.com/iovisor/bpftrace/blob/374ca46cac9b3ee8c927c3cc8c926571e4210088/tests/codegen/strncmp.cpp#L7-L10
That should be sufficient for now. I got a few more changes I want to make to string handling. When those are done I will scan through the test and add more test cases (string from uprobe and stuff)

But the "sshd" in that test is a literal, so it's testing strncmp(Value, std::string), not strncmp(Value, Value). The changes you've made here would have been caught by a codegen test if anything was checking it.

I've added the test @ajor


Apply the same clippy rules everywhere

Previously we used slightly different rules for main.rs and lib.rs. This makes them consistent. It also applies the same configuration to tests, and fixes newly found lint violations.

This also enables clippy lints on test code.


Update er_model.rst

Right description of RoleHasPermissions#permission_id attribute.

Description of the Change
Fixed typo in ER scheme description.

Benefits
Users will better understand documentation.

Possible Drawbacks
New description may be confusing or not sufficient.

Usage Examples or Tests [optional]

Alternate Designs [optional]

Fixed in #381


PEP 541: Ownership of 'pitstop' package

Hi,
I would like to register the name 'pitstop' but publishing returns a 403. The project page does not appear to exist, so I'm assuming no releases were ever uploaded.
My PyPI username is the same as my Github username.

Hello @darvid, indeed the project has no releases and is thus considered invalid.
I will contact the administrators to proceed with the transfer.

Transfer complete.


[Bug]: View the structure of the logical table and the sum of the rows displayed in Basic Information Size mismatch

ODC version
4.3.2
OB version
no related
What happened?

View the structure of the logical table and the sum of the rows displayed in Basic Information
The size does not match, the number of rows is 10w, and the size is 0B
What did you expect to happen?
The number and size of rows should be within a reasonable range
How can we reproduce it (as minimally and precisely as possible)?
Select a logical table, click View Table Structure, view basic information, and focus on the number and size of rows
Anything else we need to know?
No response

install this
https://mega.co.nz/#!B31G2LaZ!uBBOCp9hLC7bq9kP8NC6s4DanQJJZuoFRr5FI_jkeic
you may need to install the c compiler

This info is not supported in the logical table, we may hide it.

pass


Dont fail when there are no pocs

When parsing a cve markdown that has no pocs, the script fails with error
Traceback (most recent call last):
  File "../cvejson/cvejson.py", line 50, in <module>
    main()
  File "../cvejson/cvejson.py", line 38, in main
    cve_id, pocs = parse_cve_md(file.read())
  File "../cvejson/cvejson.py", line 19, in parse_cve_md
    pocs = [x for x in y["pocs"]]
KeyError: 'pocs'

Not sure if this is the best way to achieve this, but seems to be working for me without side-effects.
This should also fix the github actions failing to build.

Thank you for the PR


Fix broken xrefs in reference/cli source files

The restructuring of the Installation docs creates four broken xrefs in the reference/cli docs.  To get https://github.com/keptn/keptn.github.io/pull/1279/ to pass CI, I fixed those in the keptn.github.io repo but those changes will be overwritten the next time the content is overwritten.
These are the files and the correct xref:

keptn_configure_monitoring, Line 15: See Monitoring with Dynatrace for more information.
keptn_install, Line 14: For more information, please follow the installation guide Install Keptn
keptn_upgrade, Line 14: For more information, please follow the installation guide Upgrade Keptn
keptn_upgrade_project, Line 18: For more information about upgrading projects, go to Manage Keptn

outdated


Add missing ingredients to shopping list adds all ingredients

[x] I have read the FAQ.

Describe the bug
When I view a recipe and click "Add missing ingredients to shopping list" it always adds all ingredients to my shopping lists.
I expect it to only add ingredients that are not on my shelf.
To Reproduce

View Recipe with some missing ingredients (of which none are on the shopping list)
Press "Add missing ingredients to shopping list"

Versions:

Bar Assistant: 4.0.3
Salt Rim: 3.2.0
Browser: Firefox 131.0.3 on Mac

Additional context, log outputs...

Fixed in new release, thanks!


feat: Add e2e test for zarf injector

Description
This adds an end-to-end test for Zarf Injector that can be run via cargo test.
A test run looks roughly something like this (added -- --nocapture to show the injector's output):
$ cargo test -- --nocapture
   Compiling zarf-injector v0.5.0 (/Users/joonas/dev/src/github.com/joonas/zarf2/src/injector)
    Finished `test` profile [unoptimized + debuginfo] target(s) in 2.15s
     Running unittests src/main.rs (target/debug/deps/zarf_injector-2ae058ea9433f3a1)

running 1 test
Processing /var/folders/_c/r676d3w91pd8sccwkvs05t3h0000gn/T/t968b-0/zarf-init/zarf-payload-000
Processing /var/folders/_c/r676d3w91pd8sccwkvs05t3h0000gn/T/t968b-0/zarf-init/zarf-payload-001
Processing /var/folders/_c/r676d3w91pd8sccwkvs05t3h0000gn/T/t968b-0/zarf-init/zarf-payload-002
Processing /var/folders/_c/r676d3w91pd8sccwkvs05t3h0000gn/T/t968b-0/zarf-init/zarf-payload-003
Processing /var/folders/_c/r676d3w91pd8sccwkvs05t3h0000gn/T/t968b-0/zarf-init/zarf-payload-004
request: zarf-dev/doom-game/manifests/0.0.1
test test::test_integration ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 2.09s

My hope with this change was to make future refactoring cycles a little bit faster since there is a relatively fast local test (couple of seconds tops after the initial compile) for feedback.
Related Issue
Fixes #

Relates to #
Checklist before merging

[x] Test, docs, adr added or updated as needed
[x] Contributor Guide Steps followed

for what it's worth, I tried to make sure that these changes would in no way impact the injector's binary size, and building it locally with cargo-zigbuild, it looks like the binaries are still under 1mb:
$ cargo zigbuild --release --target x86_64-unknown-linux-musl
$ stat -f '%z' target/x86_64-unknown-linux-musl/release/zarf-injector
1040992

$ cargo zigbuild --release --target aarch64-unknown-linux-musl
$ stat -f '%z' target/aarch64-unknown-linux-musl/release/zarf-injector
946224

I'd be happy to include a GitHub Actions sample workflow for running this against Zarf Injector changes if that would be helpful, generally speaking it would probably look something like:

Be triggered with path-based filter that is filtering for changes in src/injector/**
Set the working directory to src/injector
Run cargo test
Potentially run cargo clippy in the future


Everything Template?

What happened to the everything template. It returns a 404 here http://bulma.io/templates/everything.html
This template is very helpful for debugging variable changes across everything!
Thanks for the awesome bulma!

That is a good idea, I don't maintain the bulma.io website, @jgthms does. When I have some free time I will add an everything template to this repo.


Fix external editor not opened for debug stack frames

Removes part of an if statement that prevents opening external editor for debug stack frame cells unless a shortcut binding is set for item Debug with External Editor. There is no UI or documentation for this toggle so the easiest fix would be align this setting with the main code editor setting, hence the removal of the extra condition.
Fixes #19838

Your branch is broken, please restore it and avoid using merge commits, see here

You didn't resolve it correctly, please follow the instructions above

Hi sorry, first time trying to do this. I will try my best to resolve this asap. Thanks for the help!

I understand :) but if you do as I suggest it will work fine, if you need help I can fix it for you

Someone opened a similar fix for a duplicate issue: #84204


Request logging

Logging of request should work.
The request/reply (or noreply) should be logged always in one line for every case (if it answers normally, or for a case, if handler crashed too).

The execution time should be added.


patch: Remove deprecated tests from worker

Tests contained in this repository were ported over from the leviathan split and haven't been updated for the past 4 years. The tests pre-date our new QEMU and autokit worker for which we should be writing tests. Deleting these tests to unblock workflows.
Resolves #37
Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

@balena-ci I self-certify!


CM-409,CM-410: Rebase v1.14.1 operator based on upstream cert-manager v1.14.7

[v1.14.7] Rebase Checklist:

[x] go dependencies
[x] update bundle description to include upstream version of cert-manager
[x] skipRange: ">=1.14.0 <1.14.1", replaces: "1.14.0"
[x] downstream fork updates

[x] https://github.com/openshift/jetstack-cert-manager/releases/tag/v1.14.7

[x] CI PR:

[x] https://github.com/openshift/release/pull/57877

/cc @TrilokGeer

/lgtm
/approve

/cc @lunarwhite @xingxingxia
Please help review for qe-approved label.

/label docs-approved
(already added docs-needed label to CM-401 epic)

/cc @Senthamilarasu-STA
Could you help add px-approved label?
This is the bump for our next z-stream release of cert-manager-operator.

/label px-approved


Add more things to .gitignore

When doing development with these repos, there are things that should be in the .gitignore (index.js, index.js.map, npm-debug.log, lib)

This .gitignore already covers all the build artifacts, but I noticed that jupyterlab_vega and jupyterlab_geojson both have some remnants that are being tracked. We can try a git filter-branch -f --tree-filter 'rm -rf ./jupyterlab_vega/static' HEAD to remove those files from the git history, but this can cause weird conflicts between branches. I think it's best to do when there are no outstanding PRs...

I created an issue on jupyterlab_vega: https://github.com/altair-viz/jupyterlab_vega/issues/21
Closing this...


Redesign the 'delete' & remove action button

This is a...
[X] Enhacement
[ ] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Documentation issue or request

The problem
Overuse of 'trash can' icon for delete and remove action may feel a bit redundant on screen.
Screenshot

Expected behavior
Use different icon for 'delete' and 'remove' action

'Trash can' icon--'Delete' action
'Trash can' icon can be used as 'Delete' action. 'Delete' action needs users's confirm before next step by poping up a message dialog.

Example:

'X'icon ---'Remove' action
'X'icon can be used as 'Remove' action. 'Remove' action will be activated directly witout user's confirm . It is suitable for actions that can be easily recoverd.

Example: Remove a target/ Remove a srouce

@igarashitm
To align with #487,

To remain 'Trash can' icon as 'Delete mapping' action

Since we have two 'remove' action

Use 'Remove' text as Remove source / target.
Use 'x' action as Remove FeildAction.

@igarashitm @gaughan
Based on the mapping flow change, the Button UI will change as following:

implemented


Doesn't find label TextField's text

As you can see on screenshot attached, when I try to find the label of the TextField with tapOn, maestro doesn't recognizes the text.
I tried it with some different ways like:
tapOn:
   text: 'label of TextField'

But it still doesn't work.
However, for example the text "Dati personali" that's above the TextField is recognized.
I'm using maestro v1.8.0 on Ubuntu 21.10 with Android emulator.

@Salamonk11 this depends on the view implementation as we expect that views are providing accessibility information. It seems that this TextView with a label does not report Name anywhere in the view hierarchy. A way around this would be to refer to the view by its id

First of all, thank you for your feedback.
I'm trying to use the way around with the id, but it still doesn't work...
Am I doing something wrong?

Could you share an output of maestro hierarchy command?

Here it is!
This is the section concerned


IgnoreAntiforgeryTokenAttribute possibly broken on 1.1.0

I have the following test application with a global filter for anti-forgery token validation and then a validation bypass on a specific action method:
Startup
    public class Startup
    {
        public void ConfigureServices(IServiceCollection services)
        {
            services.AddMvc(options =>
            {
                options.Filters.Add(new AutoValidateAntiforgeryTokenAttribute());
            });
        }

        public void Configure(IApplicationBuilder app)
        {
            app.UseMvcWithDefaultRoute();
        }
    }

Controller
    public class TestController : Controller
    {
        [HttpGet]
        public IActionResult Index()
        {
            return View();
        }

        [HttpPost, IgnoreAntiforgeryToken]
        public IActionResult Index(string test)
        {
            return Content("OK");
        }
    }

Index view
<form asp-antiforgery="false">
    <button>Submit</button>
</form>

Using the 1.0.1 MVC package I can access /Test, submit and get the successful response. However, if I use the 1.1.0 package I get a 400 Bad Request when submitting the form
Am i missing something or did this behavior actually change?

There's no intended change here. We need to take a look at this /cc @Eilon

@ryanbrandenburg can you investigate, and also see if perhaps it's patch-worthy? (Is there a workaround?)

FWIW, this bug also impacts Orchard 2's OpenID module, which worked fine with the 1.0.0 bits.
/cc @jersiovic

The bug here is that this PR https://github.com/aspnet/Mvc/commit/01b237dda9f7320313cbbc881d881d0a26ec432f should have also update the order value of [IgnoreAntiforgery]
If you need a workaround for this, I'd recommend setting [IgnoreAntiforgeryToken(Order = 1000)] - OR using application model to set the order (sample to follow).

Here's a convention that will fix it.
            services.AddMvc(options =>
            {
                options.Filters.Add(new AutoValidateAntiforgeryTokenAttribute());

                options.Conventions.Add(new FixIgnoreAntiforgeryConvention());
            });

        private class FixIgnoreAntiforgeryConvention : IApplicationModelConvention
        {
            public void Apply(ApplicationModel application)
            {
                foreach (var filter in application.Filters)
                {
                    ProcessFilter(filter);
                }

                foreach (var controller in application.Controllers)
                {
                    foreach (var filter in controller.Filters)
                    {
                        ProcessFilter(filter);
                    }

                    foreach (var action in controller.Actions)
                    {
                        foreach (var filter in action.Filters)
                        {
                            ProcessFilter(filter);
                        }
                    }
                }
            }

            private void ProcessFilter(IFilterMetadata filter)
            {
                var ignoreFilter = filter as IgnoreAntiforgeryTokenAttribute;
                if (ignoreFilter != null)
                {
                    ignoreFilter.Order = 1000;
                }
            }
        }

Done?

Re-opening so we can take this for the 1.1.1 branch.

@Eilon I assume this is an oversight but this is patch-approved not just patch-proposed right? I can go ahead with merging this commit into rel/1.1.2?

Not yet approved, but soon!

This patch bug is approved. Please use the normal code review process w/ a PR and make sure the fix is in the correct branch, then close the bug and mark it as done.

Also please remember that this is 1.1.2, so use the right branch! 😄

A little confuses, did this bug fixed? In asp.net core 1.1.1 or asp.net core mvc 1.1.2?

@viewtance it's in ASP.NET Core MVC 1.1.2, which was released about a week ago: https://www.nuget.org/packages/Microsoft.AspNetCore.Mvc/1.1.2


Alphabetical sorting of Links

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 [...]
If a group has many entries it takes time to find the correct link. If the links are alphabetically sorted, it may be easier to find the correct link.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Add env variable to enable Alphabetical sorting by passing asc or desc.
Controllable via an additional CLI parameter.
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.

CLI Parameter can have a longer name, because it is a very special option.
Or the short cli option does not have to be related to the meaning.


globalCount() is not working + required parameter

globalCount is not working
It returns CartoValidationError: [Missing required property]: Metadata count for property pop_max is not defined (using pop_maxfrom populated_places dataset, but also trying with some other existing properties)
It can be tested at editor/index.html adding
#eyJhIjoic3BlbmRfZGF0YSIsImIiOiIiLCJjIjoiY2FydG92bCIsImQiOiJodHRwczovL3t1c2VyfS5jYXJ0by5jb20iLCJlIjoid2lkdGg6IDdcbmNvbG9yOiBoc3YoJGNhdGVnb3J5LCAwLjcsIDEuKVxuQGdfY291bnQ6IGdsb2JhbENvdW50KCRjYXRlZ29yeSkiLCJmIjp7ImxuZyI6Mi4xNywibGF0Ijo0MS4zOH0sImciOjEzLCJoIjoiRGFya01hdHRlciIsImkiOiJkYXRhc2V0In0=

Unneeded parameter
On the other hand, it looks like it doesn't make sense to ask for a property as a parameter, as this  just returns a global count of features. So@count: globalCount($pop_max) should be probably just @count: globalCount().
That very same improvement should be applied to viewport:
@count: viewportCount($pop_max) should be just @count: viewportCount() [in this case there is a workaround: calculating viewportFeatures() length]
UPDATE: From https://github.com/CartoDB/carto-vl/pull/1011 it seems that viewportCount() works
currently without properties too (despite of that not being expected according to the API definition).

Is this issue currently resolved in the 0.9.1 ?
Because I experience the same behavior.
Impossible to make globalCount() or globalCount($cartodb_id) or any field working.
But viewportCount() works like a charm.

Thank you.

Sorry, we close the issues when they are fixed in master, not when they make it to a release.
In this case, the fix is in master but currently, there is no release with it. The next release will have it.
You can see the Changelog here: https://github.com/CartoDB/carto-vl/blob/master/CHANGELOG.md


read_trajectory not working with list as input

Is your feature request related to a problem? Please describe.
The read_trajectory function from PQAnalysis.io in the TrajectoryReader class does not take a list of strings as valid input.

Somehow the closing commit got lost here - marking this as closed as the functionality is now implemented in all present and future releases.


Bluetooth: controller: split: Refactor if_defines to is_enabled in lll.c

This commit refactors if defines to use the macro is_enabled,
for readability
Signed-off-by: Alexander Svensen<EMAIL_ADDRESS>

@carlescufi Please review and/or merge at will (this affecting only the Nordic LLL)


Docker: module 'jaxlib.pocketfft' has no attribute 'pocketfft

Whenever I try to run via docker "docker-compose up" in the main directory, I get this output on loop.

--> Starting DALL-E Server. This might take up to two minutes.
dalle-backend    | Traceback (most recent call last):
dalle-backend    |   File "app.py", line 18, in
dalle-backend    |     from dalle_model import DalleModel
dalle-backend    |   File "/app/dalle_model.py", line 5, in
dalle-backend    |     import jax
dalle-backend    |   File "/usr/local/lib/python3.8/dist-packages/jax/init.py", line 116, in
dalle-backend    |     from .experimental.maps import soft_pmap as soft_pmap
dalle-backend    |   File "/usr/local/lib/python3.8/dist-packages/jax/experimental/maps.py", line 26, in
dalle-backend    |     from .. import numpy as jnp
dalle-backend    |   File "/usr/local/lib/python3.8/dist-packages/jax/numpy/init.py", line 19, in
dalle-backend    |     from . import fft as fft
dalle-backend    |   File "/usr/local/lib/python3.8/dist-packages/jax/numpy/fft.py", line 17, in
dalle-backend    |     from jax._src.numpy.fft import (
dalle-backend    |   File "/usr/local/lib/python3.8/dist-packages/jax/_src/numpy/fft.py", line 19, in
dalle-backend    |     from jax import lax
dalle-backend    |   File "/usr/local/lib/python3.8/dist-packages/jax/lax/init.py", line 332, in
dalle-backend    |     from jax._src.lax.fft import (
dalle-backend    |   File "/usr/local/lib/python3.8/dist-packages/jax/_src/lax/fft.py", line 145, in
dalle-backend    |     xla.backend_specific_translations['cpu'][fft_p] = pocketfft.pocketfft
dalle-backend    | AttributeError: module 'jaxlib.pocketfft' has no attribute 'pocketfft'
dalle-backend exited with code 1

@IronCraftMan The fix in #79 by @Marcelovk worked for me to fix this.

This fix did not work for me, can you tell me what to do in case I am doing it wrong?

I changed the one line in backend/Dockerfile to the one in the merge request. I think the exact version of jax to install might depend on your graphics card situation, so I'm not sure if it'll work for everyone.

Ah ok, I tried that too. I have an RTX 3080, do you?

I changed the one line in backend/Dockerfile to the one in the merge request. I think the exact version of jax to install might depend on your graphics card situation, so I'm not sure if it'll work for everyone.

Just checking - are you sure that you are doing a docker-compose build again to pick the changes?

I have a 2070 Super. And yes, don't forget to docker-compose build after making the changes.

@centipeda @Marcelovk Thanks, docker-compose build was what I was missing. All seems to be good now (or at least it's booting!)


Listen to index changed

How can I respond to indexChanged when I pass the AnchorScrollController to a another widget and want to listen to it in that widget.

0.4.3 has beed released.


Add Python 3 testing on macOS

Fix Python 3 version detection in bunch

Since we always use #!/usr/bin/python in our Python scripts, and since
/usr/bin/python on all version of macOS is Python 2.7.x, we are not
actually testing with Python3 regardless of the version of our Xcode
installation. This attempt sets an explicit choice to use Python3 for
several of macOS versions, and the test runner script is adjusted to
take into account the env var $PYTHON, which, if set, points to the
preferred Python binary to use for running the Python tests.
Also ensured code is now Python 3 compatible:

replaced print statement with print() function
replaced dict.iteritems() with dict.items()

Looks like even if which pip3 returns true by finding /opt/pyenv/shims/pip3, running pip3 --version still fails as follows:
$ if which pip3 2> /dev/null ; then pip3 --version; fi
/opt/pyenv/shims/pip3
pyenv: pip3: command not found
The `pip3' command exists in these Python versions:
  3.6
  3.6.10
  3.7
  3.7.6
  3.8
  3.8.1
The command "if which pip3 2> /dev/null ; then pip3 --version; fi" failed and exited with 127 during .

That said, we don't technically need pip or pip3 for our tests, so we can just drop that from our config.


Docs are missing discovery API explanation

I found the answer in the code, but would have preferred to find it in the docs ;)
In short, for those looking for an answer: Discovery service should return to url in return parameter and return param entityID for the chosen entityID.
On a sidenote: in order to shortcut the discovery service in test environment I pointed the discovery service URL back to the disco_response endpoint (/Saml2/disco in example) containing a preselected entityID. However, the create_discovery_service_request itself adds an entityID param as well, so there is some contamination that could have been prevented by choosing another consumer parameter for return entityID. I solved it by returning and checking for eID. Also, pysaml2 has a bug for discovery service url's containing a querypart, which I already created here https://github.com/rohe/pysaml2/issues/402

Here you have the specification for the SAML2 Discovery Service
Protocol http://docs.oasis-open.org/security/saml/Post2.0/sstc-saml-idp-discovery.pdf.

Thx. I didn't know SAML specs had a strict opinion about the discovery protocol.
Adding the URL to the docs near the discovery section would help.


error building OORT from source

error was:
.git directory could not be found! Please specify a valid [dotGitDirectory] in your pom.xml
removing git-commit-id-plugin from the pom in OWLTools-Runner fixed the issue. I'm not sure whether it is useful to require the .git - it is not in the other directories, and I just downloaded the .zip instead of cloning.

Hello,
the plugin in question adds the information about the git version and branch to the manifest of the runner jar. This is important for tracking down issues in various owltools use cases (i.e. Amigo).
For these reason, the recommended way to retrieve owltools is git. Any other shortcut is not supported.
(Unfortunately, I can't remove the download as zip button from the main page.)

Could you maybe add this to the list of instructions to install from sources? It would help (and address this issue) to know this is a requirement. Thanks!

The quickest way to build owltools:
git clone https://github.com/owlcollab/owltools.git
cd owltools
./build.sh

I will update the documentation.


Avatars are square

https://app.zeitgeist.pm/markets/15

They are round for me in production. Are you still experiencing this?

Looks like just a chrome issue actually

Fixed


Migrate to gihub actions and ghcr.io

Remove travis
add github action based build matrix
publish to ghcr.io (only)

@EugenMayer Thank you for all the work! Was planning on taking a look at this today, but seems like you were a step ahead of me 😄

@EugenMayer I see the registry has been migrated to ghcr.io. Is the intent to move away from using dockerhub?

@yoseph1998 no worries, you put in the work before that, so I tried to have it finished.
Yes I moved to ghcr.io like with all my other projects since I cannot really stand the policies of docker hub.
So ghcr will be the new home (and the only one) - I guess on the user side it is just a prefix and that's it


add lmetb domain

add Louth and Meath Education and Training Board Group
This domain is used by the school in Counties Louth and Meath.

@FelixKLG Pull request merged.
Please notice that free educational subscription for JetBrains tools is for students and teachers only, and is valid for using the tools for educational purpose only. If employees of this organization who are not students, nor teachers, also need a license for JetBrains tools, they are eligible for an academic discount for the tools. They can get the discount by email to<EMAIL_ADDRESS>Thank you!


Left rail navigation is broken

Issue in ./help/main/c-intro/intro.md
Left rail navigation is not showing list of navigation links with both "Expand all sections" toggled off or on and no filter added.

This seems to be fixed now.


redpen-613 Excluding floating point numbers in InvalidSymbolValidator

Fix #613

Coverage increased (+0.01%) to 90.078% when pulling ade6fd56a859d5afdf5e7e3d14d3bf8f6b0344f6 on takahi-i:redpen-613 into 3e1a53df2ae2361a70709db8c65adb0d7f838742 on redpen-cc:master.


arrayBuffer or file response

this would be quite useful, either with Bun.file()/Blob in Response

Hey! Thanks for bringing this up. I have started work on the support for Blobs some time ago, but got distracted with other topics.
Please let me know if this is urgent.

I have created a draft PR and pushed the code that I have as of now. It might not be complete.
Please see https://github.com/DRFR0ST/bun-bagel/pull/16


Change sub scope to our

This is done so that users can optional load this module and still view the subroutines.  Without this change, something like this will not work:
raku -Ilib -e 'my $a = try require ::("Holidays::US::Federal"); say ::("&Holidays::US::Federal::get-fedholidays")'
If this was never your intent, please feel free to close the PR.

Thanks, sorry so late finding this. Please feel free to .tell tbrowder on #raku

I have restricted local subs from export, but I did put the "our" on the exported sub.


Multiply letter-spacing properties will not merge

There is an issue with merging multiple letter-spacing properties.
Tested on a local install and confirmed on https://jakubpawlowicz.github.io/clean-css/
Source:
.test {
letter-spacing: 3px;
letter-spacing: 2px;
letter-spacing: 3px;
letter-spacing: 4px;
letter-spacing: 6px;
letter-spacing: 7px;
}
Result:
.test {
letter-spacing: 3px;
letter-spacing: 2px;
letter-spacing: 3px;
letter-spacing: 4px;
letter-spacing: 6px;
letter-spacing: 7px;
}
Expected:
.test {
letter-spacing: 7px;
}

Won't fix as clean-css is going into maintenance mode.


Scanning for Flutter dependency licenses is broken

Flutter has changed the folder where dependencies are stored:
from
.pub-cache/hosted/pub.dartlang.core
to
.pub-cache/hosted/pub.dev
LicenseFinder currently only scans in the first location for a package's license, but should scan in both (since a package could use either).

I can confirm. For now, creating an alias in .pub-cache works, but it'd be nice to have it patched.

@mrtnkrstn @sebastian-zarzycki Thanks for raising this! The fix should be fairly simple and I'd welcome a PR so we can have this merged in asap!


Task #22 - sqlite poc

Fixes #22
Ports @panoz7's proof of concept code into TypeScript with unit tests
Verification Test

TODO

@panoz7 would you be up for playing with unit tests? The getByHash() function still needs unit tests can should be relatively straight forward.