etebase: fix runtime crash due to wrong pydantic..

...version. Upstream still uses pydantic 1, which it implicitly imports via fastapi. Overriding
that and its pydantic-using dependencies fixes it, until upstream updates.
also:

exposes the existing nixos integration test via passthru.nixosTests
adds myself as a maintainer
makes the package and pythonPackage used for the service configurable.

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@felschr Ideally, we could merge the package options soon, even if the rest of the changes require discussion. As it's currently broken and calendar sync can be quite critical, some users - like myself -  can't really upgrade since the last pyton-update.
With a package option one gains the capability to patch that single package without needing to vendor the whole module.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1405

@SuperSandro2000 I think i addressed all of your wishes in the recent commits and made it a proper python package while doing so.
Would be nice to have this merged soonish as this service has been broken on unstable for well over a month now.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1491


Smarter property type guess in ColorManager

🧰 Task

A property is understood as continuous if it contains 16+ unique labels: this is problematic for example when trying to map many keypoint names to their colors. I wondered whether there is room to improve the guess that is made: can this hardcoded value of 16 be removed? Or would it be preferable to guess categorical when variables are strings? https://github.com/napari/napari/blob/7f6d272f0d68ceded50519900b67d673363c3b6a/napari/layers/utils/color_manager_utils.py#L9-L31
I've patched it for now in our napari-deeplabcut plugin (https://github.com/DeepLabCut/napari-deeplabcut/pull/25), but I am curious about the original design decision. 😃

Hey @jeylau. Sorry about that. I can see how this would be frustrating.
I am open to changing the behavior. I believe the original decision to have such a threshold was because it is difficult to come up with high contrast categorical colormaps for a large numbers of categories and because integer valued columns could be categorical or continuous. Not catching the string values was an oversight. There is certainly room for improvement, but I'm not sure the guessing will ever work perfectly for all use cases.
It sounds like we should update the behavior such that:

if the column contains strings, it is categorical. I thought we already had this, so we should make sure this isn't being handled somewhere else.
floats are always continuous
I don't have a strong opinion if ints should always be considered continuous. It does seem to be more clear if integers are always treated as continuous, as there isn't a hidden threshold. If we choose to keep a threshold, we should document it better. I'm curious what others think.

I've patched it for now in our napari-deeplabcut plugin (https://github.com/DeepLabCut/napari-deeplabcut/pull/25)

In case you'd like to avoid monkey patching, you should be able to set the ColorManager.mode to cycle manually, which should correctly choose a categorical mapping. If you point us to the relevant code in your plugin, we may be able to provide some help.
P.S. I'm a deeplabcut fan! It is super cool to see the napari-deeplabcut plugin.


Off-Chain Trading Wireframe

Matt will be the owner here.

Personally I would consider closing this as "off chain" trading should look no different to the user than on chain trading


Cohort not working

In the current version of Google Analytics connector for the Power BI desktop doesn't support cohort dimensions and metrics.
When I select User Retention metrics and cohort dimension, it tells that ga:cohortuserretentionrate is not supported in this API version.
If this could be fixed, that would be great to show my user retention.
If any other alternative is available I'm interested in that too.
Thank You

Thanks for your comment, @vikram-relyfy. This looks like product feedback, rather than documentation feedback. You can submit product feedback or feature ideas by going to Power BI Ideas. You can also engage with the Power BI engineering team by being active in the Power BI Community.


.Net Coreify NSpec like all the cool kids

Discuss!

Definitely. Opening such an issue was on our list.
Possible steps:

Confirm current working state of NUnit on netstandard Net 4.6 / Net Core 1.0. So that we know we can port NSpecSpecs. This should be already a green flag.
Confirm current state of technologies currently used by NSpec (reflection, marshaling, ...) on netstandard Net 4.6 / Net Core 1.0. This could be done by looking at other libraries like Fixies. Or also by just trying next item and see if it builds.
Switch NSpec class library, NSpecSpecs, NSpec console runner to netstandard Net 4.6 build
Switch NSpec class library, NSpecSpecs, NSpec console runner to netstandard Core 1.0 build

Then, for test runners:

Understand API & protocol for working with dotnet test. There are some sparse docs now, plus work done by xUnit, NUnit.
If current VS test runner still has reasons to exist (e.g. for non-netstandard client code, don't know, or for older VS versions), split that in a core part, generic, and an adapter part specific to VS test runner
Create dotnet test runner, reusing previous core part

Just thrown there, not necessarily a plan. Let's see what actually makes sense.

The biggest thing would be the App Domain code (mostly half baked as is). We had that discussion here: https://github.com/mattflo/NSpec/issues/105

It looks like that path could work when code under test is in a class library, and anyway when you can reference it from a custom console app that acts as your custom test runner.
It completely leaves out former VS Test Adapter scenario, as well as current dotnet test scenario that seems to be "the way forward" (for console, not sure if that also replaces VS Test Adapter for future VS versions).

for console, not sure if that also replaces VS Test Adapter for future VS versions

I'd imagine future integrations would build off of dotnet test. That and VS Code is entirely electron/not vsix stuff.

BTW currently taking a stab at make it build & run on .NET Core. Current status:

NSpec class library updated to build
NSpecRunner console app not building yet.

Of course main issue there is the AppDomain / Marshaling technology. In test runners/adapters scenarios (that is, when you cannot put up a custom console app and reference project under test) one still needs some way of executing code across two assemblies: the runner assembly and the assembly under test.
At a first look this should be needed by other testing frameworks as well, e.g. NUnit, so I'd say taking a look at those should be helpful. Or maybe directly asking as well. What do you think?

E.g. see excerpt from this comment on NUnit:

The first step in this process was creating a PCL driver/agent that can load and execute tests without being tightly bound to a specific version of the NUnit Framework like NUnitLite and the current Xamarin runner. It is still in it's infancy, but the work in progress is in the nunit.portable.agent project.

That last project as move since, this is the current link

Some insights after a quick inspection:

Their dotnet test console runner (net core) depends on portable agent (net core). Don't know about other existing runners, like VS test runner, plain console runner, GUI runner

Portable agent does not depend on any NUnit framework library, but still is the driver used by other runners to operate with the framework

Portable agent dinamically loads at runtime the NUnit assembly (exists in several net flavours) that comes linked together with the assembly under test

Portable agent instantiates through reflection a specific class from NUnit assembly and can invoke its methods to perform a set of commands. Methods parameters and return values are all primitive types or Actions on primitive types. Complex data structures are "hidden" in text parameters as XML.

Such class with its methods is a façade to the operation allowed from the outside, and is basically the API to work with test framework. Actually, also the XML data structure is an implicit part of the API

The actual core work happens within the NUnit assembly where the façade class is defined. This approach should be what eliminates the need for .NET Remoting technologies (MarshalByRef and all that family), as code is already defined and runs in the same assembly (the one under test) as opposed to being defined in runner and having to execute in assembly under test.

It seems like a similar approach could be taken for NSpec as well, and it could also start with refactorings in the solution as is now on .NET platform, without switching yet to .NET Core.
@amirrajan waiting for your feedbacks!

Any thoughts of "downgrading" nspec specs to just be a console app that throws exceptions? Or something even more terrifying. Use NSpec to test NSpec?

We can test NSpec pretty much any way you want :smile_cat: , so I guess potentially even with a custom console app or self-testing it. But I don't see how this could help, sorry. The whole point is to be able to test newly created .Net Core projects with NSpec, beside NSpec itself I mean.
Right now in my team we already have a couple of project started on (ASP).Net Core, and we're pulling to test with NSpec because that's not available yet. If .Net Core becomes the (or a) way to go in .Net world, why leave NSpec behind?

The whole point is to be able to test newly created .Net Core projects with NSpec, beside NSpec itself I mean.

Sorry for the confusion. I was thinking that NSpec could be entirely built using .Net Core. If the only thing stoping the NSpec solution from moving to .Net core is NUnit, then our own NSpecSpecs test suite can be written in something compatible.

I was thinking that NSpec could be entirely built using .Net Core.
That's what I meant as well :+1: I think we should aim to build NSpec both for previous .Net plarform as well as .Net Core. To obtain this result, I assume the whole solution should be moved to the new VS 2015 / .Net Core / project.json tooling (even if the project.json should go away in the future): that allows to build a version for .Net Core, and if we add another target also for previous .Net. Otherwise, keep two solutions: one as-is targeting only previous .Net, plus a new one targeting .Net Core, but if we can get everything with one solution ...

If the only thing stoping the NSpec solution from moving to .Net core is NUnit
No, NUnit is actually already ready for .Net Core, so if at some point we have NSpec built for .Net Core, we can port also NSpecSpecs test suite to .Net Core, using the most recent version of NUnit

It looks like a conf-call would help cleaning a little bit, what do you think?

To obtain this result, I assume the whole solution should be moved to the new VS 2015 / .Net Core / project.json tooling

Seems like a good first step!

It looks like a conf-call would help cleaning a little bit, what do you think?

Introvert here :-P. G+ works if you need a faster response (ar@amirrajan.net). We can do Skype or something too if you insist. But I'm gonna expect like a free lunch or something if you ever come to Dallas.

Well, speaking about eating, nothing bad for Dallas, but I guess you better come over here in Italy ;) It is just for a faster response, I think G+ would do once we manage with our timezones

This is a temporary repository, to track how initial work and see how it goes. There should be an invitation in your mailbox too

Here's an update with current activities. BTW, this now works from command line:
dotnet path_to\NSpecRunner.dll path_to_spec_dll [classname]

Closing this issue, progress can be tracked in #143


python310Packages.chainer build failure on x86_64-linux as of 734b6c0a

Issue description
Build of python310Packages.chainer failed on x86_64-linux as of 734b6c0a1c7745df0078736ae83888977fc2c028. This is currently breaking python3Packages.einops.
error: builder for '/nix/store/6iaprldms6m7y6xw1m0fd280bqspb8my-python3.10-chainer-7.8.1.post1.drv' failed with exit code 1;
       last 10 log lines:
       > adding 'chainer-7.8.1.dist-info/RECORD'
       > removing build/bdist.linux-x86_64/wheel
       > Finished executing setuptoolsBuildPhase
       > installing
       > Executing pipInstallPhase
       > /build/source/dist /build/source
       > Processing ./chainer-7.8.1-py3-none-any.whl
       > ERROR: Could not find a version that satisfies the requirement six>=1.9.0 (from chainer) (from versions: none)
       > ERROR: No matching distribution found for six>=1.9.0
       >
       For full logs, run 'nix log /nix/store/6iaprldms6m7y6xw1m0fd280bqspb8my-python3.10-chainer-7.8.1.post1.drv'.
error: 1 dependencies of derivation '/nix/store/vy8r735qpn74v71k0jdpw0zzck1h14zz-python3.10-einops-0.4.1.drv' failed to build

full build log
python3Packages.einops maintainers cc: @yl3dy
python310Packages.chainer maintainers cc: @hyphon81
Other cc: n/a
Steps to reproduce

Checkout nixpkgs at commit 734b6c0a1c7745df0078736ae83888977fc2c028
Run nix-build -A python310Packages.chainer

Technical details
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.0-1022-azure, Ubuntu, 20.04.5 LTS (Focal Fossa), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.1`
 - nixpkgs: `/nix/store/bd6rjk9k0f06zvsispa5a0j0p5gh1bm0-source`

Misc.
This issue was automatically generated by nixpkgs-upkeep.

CI workflow that created this issue.
Internal tags:<PHONE_NUMBER>096795836 -4756057284282404257

not immediately clear to me why this is failing since our current version of six is 1.16.0:
https://github.com/NixOS/nixpkgs/blob/60974988b3e1e5d61341a5ef0357e7ae99fcf7f1/pkgs/development/python-modules/six/default.nix#L7-L9
and it hasn't been updated since 2021: https://github.com/NixOS/nixpkgs/commit/02d30961f489802ff24896c433abf9538731bb6d.

note to self: this is also breaking augmax

Was fixed with #199315


DM improvements

negotiator shrub which sets up mutual DM subscriptions and hub-and-spoke groupchats.
future milestone: users can accept or reject DM and groupchat joins.
stretch goal: pull in Circles to populate options for who to start DMs/groupchats with

From #11:
_"but we probably want to move to a different architecture:
aggregator
publisher
subscriber."_

also need to fix the ~zod bowl issue

I've split the logic between a publisher and a subscriber shrub. This feels cleaner on the backend side, but there's a lot of code re-use on the rendering side.
Next steps are to fix a synching bug with the subscriber dependencies and add UI elements for creating a new DM and adding people to a DM.
(Does "DM" imply only two people? These are multi-party now.)

top-level shrub is called chat
chat maintains intermediate state while negotiating the existence of DMs and groupchats

for example, if ~zod invites ~nec to a group chat and ~nec needs to accept, chat is responsible for knowing whether or not ~nec has accepted the invite

user can create a DM or a groupchat
DMs are uniquely categorized and named by @p, groups are given custom names
DM subscriptions are mutual

ex. ~nec/chat/~zod/theirs <> ~zod/chat/~nec/ours, ~zod/chat/~nec/theirs <> ~nec/chat/~zod/ours
groups hub-and-spoke using Liam's %chat and %chat-link examples, but renamed so that the negotiator can be called chat

DM and groupchat UIs should have iMessage-style chat bubbles (on the right-hand side if it's your message and left-hand side if it's somebody else's)

lib/messages.hoon

[ ] put .loader in correct spot
[ ] iMessage-style chat bubbles

con/messenger.hoon

[ ] display all dms and messages
[ ] for groupchats, have a dropdown:

[ ] that lists participants
[ ] if I'm the host, have an X next to name to remove participant
[ ] if I'm the host, have a box that lets me add a new participant

Testing

[ ] test extensively on multiple ships


(Long-term) Make minor extensions to SMIRFF format

In the issue discussing the SMIRFF format (https://github.com/open-forcefield-group/open-forcefield-data/issues/9 ) we laid out a number of longer-term additions we wanted to make to the format. This is to remind us to make these changes eventually:

We will make a modification of the file format to specify a prior for a parameter search (https://github.com/open-forcefield-group/open-forcefield-data/issues/9#issuecomment-219327074)
We will extend the format to allow macros such as "smallhals" in SMIRKS strings (https://github.com/open-forcefield-group/open-forcefield-data/issues/9#issuecomment-222592960 )
Files will include parameter id and parent_id which track their origin

Ultimately we can accommodate fractional bond order as well: https://github.com/open-forcefield-group/open-forcefield-data/issues/9#issuecomment-224364826

id and parent_id were added in a recent PR. The other changes will still be needed.


[font] System font collections don't get loaded correctly

E.g. the Cascadia Code font is a font collection of multiple fonts, but loading a specific weight always results in the first font in the collection, i.e. 200.

Wasn't actually the source of this problem: #27


Unexpected behavior when setting/not-setting format.json() explicitly

Version
"winston": "^3.0.0-rc1",
const splat = winston.format(function (info, opts) {
  if (info.splat) {
    info.message = util.format(info.message, ...info.splat);
  }
  delete info.splat;
  return info;
});

const consoleLogger = new winston.transports.Console({
  format: winston.format.combine(
    splat(),
    winston.format.json(), // this one
  ),
  level: 'debug',
});

If I don't explicitly push winston.format.json() in combine(), the info (log message) still contains property splat which I deleted in splat(). Also the splat() does not do any effect on the output, even though it is called everytime there is request to log.
Am I missing something form the document?
e.g.
// when not-setting format.json()
{
  level: 'info',
  message: 'some number %s',
  splat: [1],
}

// when setting format.json()
{
  level: 'info',
  message: 'some number 1',
}

I can confirm that using splat() is having no effect on our log output on rc1 in consequence the util.format style logs are not working.

Closing this as a duplicate of #1298. Now that we have a SPLAT symbol, we can use that to remove the spurrious splat property. Further, we can expose options that set info[MESSAGE] equal to the splat instead of expecting another format to output that for you.
In other words, having to add the json format is by design right now – all the splat format does is set info.message equal to the formatted message with the splat provided.


Add a Toggle Playback Button

Add a Button that will perform pause, resume actions based on the current IsPlaying boolean.
I believe that a Toggle Button Is a good addition that will allow the user to quickly pause and unpause the playing track
#28

Merged.
Will be included in next release.
Thanks for your contribution.


Issue with the Modification on Gmod !

2( error, your addons list, GMod language ) The wand is not working, it says Magic Wand is creating script errors I can't even right click to open menu or else i just can left click for a little " bzzt " I check the console and it's spamming some error message i can't copy it it's to fast , Addons: Magic Wand Rewrite, ULib , ULX , P2P Hero and Blood and Gore Overhaul IV , German
3( when this error happens ) when i first activated the Mod and equiped the wand

try changing gmod language to english

if that will work, try to download legacy version of the wand from GitHub and install it instead of workshop one or just reinstall workshop version

Changing the language ? What's the difference? it will be the same anyway just with english ui

just try it

I will be afk for some hours

I doubt that you have less than 8 addons. Citation needed

I just have those 4+1 Mods installed

Anyway, I'am looking for a Solution for this Problem. Not for an Argument.

what is the error doing
what is the console out
we cant go blind onto that one

Did you try to change your gmod language to English at least?

Ok, i've tested and yhea, it's the same problem since yesterday.
Remplace everything in the "dictionnary.lua" with this : https://github.com/Sachha/HPW_Rewrite/blob/7e5931a0326cd14908be98388aea3f43f29016f4/lua/hpwrewrite/dictionary.lua
I've sended a pull requested but they denied it, I don't know why

updated


CrossTalk for "Vietcong" and "Hidden & Dangerous 2"

Hi, we play Vietcong very often, although it is quite old.
The games IngameVoice is awesome, but in some missions it just breaks, as there are too many ambience sounds.
We just tried using Crosstalk in "Vietcong", but it doesnt recognize the game/player position.
We're going to test the same with "Hidden & Dangerous 2 - Sabre Squadron" soon.
"Ravenshield" didn't work either.
can you add compatibility for those games too?
It seems your method of catching the players position isn't universal.
my brother had the idea of just "listening" for DirectX threads and finding the current camera position and deriving the data from there - independent of which game or application is used.

Sorry for the delay, been at gamescom and rather busy there.
There is indeed no universal method for getting player positions from any game.
I'm implementing mumble link, a standard the game has to support for this to work.
http://wiki.mumble.info/wiki/Games
The list items marked in blue are supported by the manufacturer and CrossTalk, the other items have been reverse engineered and are supported by Mumble exclusively.
It's not in my cards time wise to add individual games by reverse engineering their positional data.
Sorry :(

Thanks for the reply.
We made the experience, that in some occassions the plugin doesnt get the player's position in HalfLife2DM, but we couldn't figure out what player actions cause that, except that switching out of the game to change crosstalk settings was involved, because we were trying to find proper values for the "rolloff" and "max distance".

Hey PvtPirate, good to see you here too. :)
You are best off asking for mumble plugin support at the mumble site. http://forums.mumble.info/viewforum.php?f=2
Imho this issue should be closed as wont-fix.
As a hint for others that might stumble upon this thread here is a list of games supported by mumble-link and therefore by this awesome TS-plugin: http://wiki.mumble.info/wiki/Games
For the ones eager enough to get things going themselves here is a starting point: http://wiki.mumble.info/wiki/Pluginguide
On a side note: Please open a separate ticket for unrelated issues. It helps keep things organized.

thanks zsawyer, we'll try building a mumbleplugin for vietcong. i hope it works to install that to teamspeak/crosstalk.
(Offtopic @zsawyer: As far as i've seen, your mumble-mod works good in MineCraft with Crosstalk.
but i couldnt convince my self of using the outdated MC version 1.8 longer than necessary for testing.)

Note that if you/someone does it like the minecraft mod does, so either as some sort of plugin for your game or as independent app that grabs the positional data and implements the mumble link protocol, that's the way to hit two targets (mumble + ts/crosstalk) with one.
I'll mark this invalid instead, makes me look less lazy! ;)
Have a nice weekend


Move from css-layout to yoga

Yoga now has an emscripten target and it would be good for us to move over to it instead of using css-layout. Yoga is now what React Native is using, and this will give us the highest level of consistency with those platforms.

Any idea of an estimate? We might need a label bigger than 1 day

@jongold 1 day seems reasonable. It shouldn't be that hard. the surface area for the css-layout stuff right now is pretty small and it should work somewhat similarly

I started looking into yoga (because I didn't play with it yet) here: https://github.com/airbnb/react-sketchapp/commit/a9a21fa9300ffa7bf3ddcd2855f7a25ba30d72fa
Unfortunately there is no documentation for the emscripten target and the document for css-layout seems to have disappeared.
One thing tho: I'm not sure there is a way to define margin-top only for example: https://github.com/facebook/yoga/blob/master/javascript/sources/Node.hh#L69


it seems look like no this package in maven repo~

@Fourth-fresh-man i haven't published to maven central yet. currently the package is published to GitHub registry but you'll need to include it as a repository first.
for publishing into maven central i'll find some time in next few weeks to do it

here is a link to how to configure GitHub's maven repository

Thanks! My problem have been resolved.

Now it is avaiable in https://search.maven.org/artifact/io.github.datafusion-contrib/datafusion-java


Test Bot is giving "technical error :( ]" error on demo

I tried to click on Try It --> Try live demo link click and open https://demo.tock.ai/. I enter the story same as on YouTube demo https://www.youtube.com/watch?v=UsKkpYL7Hto
but when click Test and Enter "greetings" it gives the error message "technical error :( ]"

Please help on it. why it's giving error.

There is an issue on this platfom. We work on it. Workaround: use the docker images at  https://github.com/theopenconversationkit/tock-docker

working with docker. thank you!


Bug: nil local cur

    {
        "oskarrrrrrr/symbols.nvim",
        lazy=false,
        config = function()
            local r = require("symbols.recipes")
            require("symbols").setup(r.DefaultFilters, r.AsciiSymbols, {
                -- custom settings here
                -- e.g. hide_cursor = false
            })
            vim.keymap.set("n", "<leader>o", "<cmd> Symbols<CR>")
        end,
    },

When I use any command to open Symbols it will throw this error and open the empty symbols buffer.

Thanks for letting me know, can you run this command: :lua vim.print(vim.opt.guicursor) and comment the output here? It seems that the guicursor has an unexpected value. Do you use any plugins that might be doing something unusual with the cursor maybe? I couldn't reproduce this issue.

I pushed a quick fix that should prevent the error from showing up: 73a5e033a74dd635974a35f1a32ec80bdd48c583
Now when the guicursor has an unexpected value we just ignore the cursor setting and avoid the crash.


The head of any existing erl_crash.dump now captured in riak-debug

Normally, an erl_crash.dump is not included in a riak-debug as the file can be extremely large.  This commit means the head (first 10 lines) of the file will be included in the debug rather than ignoring it completely.  This allows us to see the Slogan line at the top.

Don't forget riak-debug in the riak_ee repo!

Done.


can't be used with bluetooth api

BluetoothRemoteGATTCharacteristic has an event type of "characteristicvaluechanged" whereas this library only looks for event types that begin with "on".
This means typescript tells me Argument of type '() => void' is not assignable to parameter of type 'never'.ts(2345)
This would also apply to any EventTarget object that has an event type that doesn't begin with "on".

oh wait hang on I'm wrong, I don't know why I'm getting that type errpr


image_pipeline: 1.12.15-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository image_pipeline to 1.12.15-0:

upstream repository: https://github.com/ros-perception/image_pipeline.git
release repository: https://github.com/ros-gbp/image_pipeline-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.20
previous version for package: 1.12.14-0

camera_calibration
* better 16 handling in mkgray
  This re-uses #150 <https://github.com/ros-perception/image_pipeline/issues/150> and therefore closes #150 <https://github.com/ros-perception/image_pipeline/issues/150>
* fix OpenCV2 compatibility
* fix tests with OpenCV3
* [Calibrator]: add yaml file with calibration data in output
* Contributors: Vincent Rabaud, sambrose

depth_image_proc
* Add option for exact time sync for point_cloud_xyzrgb
* simplify OpenCV3 conversion
* Contributors: Kentaro Wada, Vincent Rabaud

image_pipeline

No changes

image_proc
* simplify OpenCV3 conversion
* Contributors: Vincent Rabaud

image_rotate

No changes

image_view
* simplify the OpenCV dependency
* [image_view] Configure do_dynamic_scaling param with dynamic_reconfigure
* [image_view] Scale 16UC1 depth image
* fix compilation
* Extract images which are synchronized with message_filters
* [image_view] Show full path when failed to save image
* [image_view] Enable to specify transport with arg
* [image_view] feedback: no need threading for callback
* [image_view/image_view] Make as a node
* Added sensor_msgs::Image conversion to cv::Mat from rqt_image_view in
  order to be able to create videos from kinect depth images (cv_bridge
  currently doesn't support 16UC1 image encoding).
  Code adapted from:
  https://github.com/ros-visualization/rqt_common_plugins/blob/groovy-devel/rqt_image_view/src/rqt_image_view/image_view.cpp
* simplify OpenCV3 conversion
* use the color conversion for display from cv_bridge
* Contributors: Carlos Costa, Kentaro Wada, Vincent Rabaud

stereo_image_proc
* simplify OpenCV3 conversion
* Contributors: Vincent Rabaud

Hold until new buildfarm rollout is complete early next week.

I know, I know, no worries.

I know that you know - just making sure every pending PR has the note that nobody clicks on merge :wink:


[BUG] install-gui.sh fails on Ubuntu 20.04 LTS

Describe the bug
After following the previous steps in https://github.com/Chia-Network/chia-blockchain/wiki/INSTALL#ubuntudebian, running install-gui.sh fails.
To Reproduce
Follow the steps in https://github.com/Chia-Network/chia-blockchain/wiki/INSTALL#ubuntudebian on a fully updated Ubuntu Linux 20.04 LTS system, and see it fail in the install-gui.sh step
Expected behavior
Succesfull installation of the GUI on Ubuntu 20.04 LTS.
Screenshots
Screenshots are not immediately applicable, but find attached the log file referred to in the final three lines of the output from running bash install-gui.sh below:
npm ERR! Additional logging details can be found in:
npm ERR!     /home/sune/src/mining/chia/chia-blockchain/chia-blockchain-gui/npm-debug.log
npm ERR! not ok code 0

Desktop (please complete the following information):

OS: Linux
OS Version/Flavor: Ubuntu Linux 20.04 LTS (fully updated)
CPU: Intel Xeon W3680

Additional context
See attached log file
npm-debug.log

It looks like you are using very old version of node.
error node -v v0.10.25
2117 error npm -v 1.3.24
Use node >= 10.14.2 and let me know.

Nicely spotted, @seeden, and thank you!
I am not entirely sure how that old package ended up on the machine, but after uninstalling it, and re-running the install steps, things worked as expected.
My apologies for the noise.


Feature controlupdates

@homu r+

:pushpin: Commit 3a0ac32 has been approved by Oksisane

:hourglass: Testing commit 3a0ac32 with merge edc2c51...

:sunny: Test successful - status

@Oksisane homu is down - test and merge manually using command line

@Oksisane nvm


Added built libs

Looking forward to getting embedded mode back in! Couple questions:

It looks like there's only Windows/Linux support here, what's the probability of getting macOS support as well?
Does it make sense to write a script that pulls the source and builds each shared library for each OS so that it's easier to keep them up to date?
Does it make sense to check these in with git LFS?


NetUtils: simplify logic in determineInterfaces

🦟 Bug fix
Removes an unused variable and simplifies complex logic in determineInterfaces method.
Summary

2f6056b: The preferred_ip array is initialized to {0} and no code writes to it; it is only read from, so just remove it.
61ce68f: Now there is an if/else if/else block that runs the same code in each branch, so just remove the logic and run that code unconditionally.

I believe this is functionally equivalent to the current code.
Checklist

[X] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[X] codecheck passed (See contributing)
[X] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-03-25-fortress-edifice-citadel/1343/1


Update UserRecord

Please advise on necessary changes and/or tests

Thanks @levimatheri. I would say, look at our Node.js or Java SDKs for some inspiration. We will probably need several different types. UserInfo, UserRecord and ExportedUserRecord. UserInfo could probably be an interface that UserRecord implements. ExportedUserRecord should extend UserRecord.


Button links with text state colors

Curious about community need / interest in this feature.  I often use btn-link buttons and want to be able to color the buttons per the text state colors provided by Bootstrap.  I have found it intuitive to try class="btn-link text-danger" for a delete link button, to no avail.
I now accomplish this by adding the below to my Bootstrap override LESS file.  I feel something like this should ship with Bootstrap.  What do the maintainers and the community think?
.btn-link {
    &.text-success {
        .text-emphasis-variant(@state-success-text);
    }
    &.text-info {
        .text-emphasis-variant(@state-info-text);
    }
    &.text-warning {
        .text-emphasis-variant(@state-warning-text);
    }
    &.text-danger {
        .text-emphasis-variant(@state-danger-text);
    }
    &.text-muted {
        .text-emphasis-variant(@text-muted);
    }
}

As mentioned in the docs, just use a wrapper span:
<button class="btn btn-link">Your text</button>

Seems a little markup heavy.  Though, I was ignorant of the docs on this.  Thanks for the super speedy review.

There are a bunch of places where the text color and background color utility classes don't work due to CSS specificity. Adding overrides for them all would really bloat the CSS, and would still fail for some users' custom CSS. The span trick is simple and works almost everywhere.

With span trick it's showing hover underline with text-primary, no matter what color text is. Also, using text-xxxxxx doesn't change color on hover. I wouldn't say this is a complete solution.

I noticed this as well.  The above override works well.

just had the same issue, updated example for bootstrap 4 / sass:
.btn-link {
  @each $name, $color in (success: $success, info: $info, warning: $warning, danger: $danger) {
    &.text-#{$name} {
      @include hover-focus {
        color: darken($color, 10%) !important;
      }
    }
  }
}


Microsite header: Design system website contextual example updates

Acceptance Criteria:

Add the following jump links below the properties section:

Link to feedback
Show version number

Add the following contextual examples:

Link to feedback
Show version number

Duplicate issue


Feature/178928604 enable access for private experiments

This feature allows users to see private experiments if they know the access key in the single cell.
Companion PR: atlas-web-core

We will need to evict the new privateExperimentAcession cache when performing CRUD operations. The change needs to in this file https://github.com/ebi-gene-expression-group/atlas-web-single-cell/blob/develop/app/src/main/java/uk/ac/ebi/atlas/experimentimport/ScxaExperimentCrud.java

Ok, I will fix this one.

We have this PR https://github.com/ebi-gene-expression-group/atlas-web-core/pull/70 to follow after rebase.


Test for release assets for generic generator

This is missing, let's add it.

Renamed a workflow to https://github.com/slsa-framework/example-package/blob/11cb5e03b88cb79f95662f49c9622da8cc11b5cc/.github/workflows/e2e.generic.tag.main.assets.slsa3.yml and updated the scripts.
Checking whether everything works before closing this issue.

This seems to be working fine.
https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.tag.main.assets.slsa3.yml


CommandAttribute did not work?

Environment

dotnet core sdk 3.0pre6
MicroBatchFramework: 1.2.0

Steps to reproduce

create console project(and set TargetFramework to netcoreapp2.1)
write following source to Program.cs
do dotnet run -- test -arg1 aaaa

using System;
using System.Threading.Tasks;
using MicroBatchFramework;

namespace microbatchframeworktest
{
    class MyBatch : BatchBase
    {
        [Command("test")]
        public void MyTest(string arg1)
        {
            Console.WriteLine("hello {0}", arg1);
        }
    }
    class Program
    {
        static async Task Main(string[] args)
        {
            await BatchHost.CreateDefaultBuilder().RunBatchEngineAsync(args);
        }
    }
}

Expected behavior
print "hello aaaa"
Actual behavior
print "Type or method does not found on this Program. args: test -arg1 aaaa" then exit.
Additional

dotnet run -- MyBatch.MyTest -arg1 aaaa seems to work
did dotnet run -- list, then output MyBatch.MyTest was output

Oh!
Thank you, I'll check soon.

I found some code where it might be the cause.
https://github.com/Cysharp/MicroBatchFramework/blob/f6a549c07d77b4a6f1ac8b0eda27cbd63ac863d5/src/MicroBatchFramework/BatchEngineHostBuilderExtensions.cs#L203-L207
and I fixed the code like following, it seems to be fine(still it has not been verified enough)
diff --git a/src/MicroBatchFramework/BatchEngineHostBuilderExtensions.cs b/src/MicroBatchFramework/BatchEngineHostBuilderExtensions.cs
index 7a7002c..e29c7db 100644
--- a/src/MicroBatchFramework/BatchEngineHostBuilderExtensions.cs
+++ b/src/MicroBatchFramework/BatchEngineHostBuilderExtensions.cs
@@ -200,43 +200,48 @@ namespace MicroBatchFramework

         static (Type, MethodInfo) GetTypeFromAssemblies(string arg0)
         {
-            var split = arg0.Split('.');
-            if (split.Length != 2)
-            {
-                return (null, null);
-            }
-            var typeName = split[0];
-            var methodName = split[1];
-
             var batchBaseTypes = GetBatchTypes();
             if (batchBaseTypes == null)
             {
                 return (null, null);
             }

+            var split = arg0.Split('.');
             Type foundType = null;
-            foreach (var item in batchBaseTypes)
+            MethodInfo foundMethod = null;
+            foreach (var baseType in batchBaseTypes)
             {
-                if (item.Name == typeName)
+                if (split.Length == 2)
                 {
-                    if (foundType != null)
+                    if (baseType.Name.Equals(split[0], StringComparison.OrdinalIgnoreCase))
                     {
-                        throw new InvalidOperationException("Duplicate BatchBase TypeName is not allowed, " + foundType.FullName + " and " + item.FullName);
+                        if (foundType != null)
+                        {
+                            throw new InvalidOperationException("Duplicate BatchBase TypeName is not allowed, " + foundType.FullName + " and " + baseType.FullName);
+                        }
+                        foundType = baseType;
+                        foundMethod = baseType.GetMethod(split[1]);
                     }
-                    foundType = item;
                 }
-            }
-
-            if (foundType != null)
-            {
-                var method = foundType.GetMethod(methodName);
-                if (method != null)
+                else
                 {
-                    return (foundType, method);
+                    foreach (var (method, cmdattr) in baseType.GetMethods().
+                        Select(m => (MethodInfo: m, Attr: m.GetCustomAttribute<CommandAttribute>())).Where(x => x.Attr != null))
+                    {
+                        if (cmdattr.CommandNames.Any(x => arg0.Equals(x, StringComparison.OrdinalIgnoreCase)))
+                        {
+                            foundType = baseType;
+                            foundMethod = method;
+                        }
+                    }
                 }
             }
-
+            if(foundType != null && foundMethod != null)
+            {
+                return (foundType, foundMethod);
+            }
             return (null, null);
+
         }
     }
 }
\ No newline at end of file

Thanks, I've released v1.3.0.


Fix:Missing space before hashtag in typedef declaration of parameterized class

Fixes #850

Thanks for your first contribution! The potential change for this issue will need to touch a lot less files though.
The intent of issue #850 is make the formatter generate the desired formatting with the space in front of the hash.
You have done a lot of work changing tests to also use that formatting in the input, but that is not what the issue is worried about. In fact, we should keep them as is, as we want the parser to accept any form of the input.
The main focus should be to change the formatter to output the space according to the issue. So you will have to change the implementation of the formatter and its tests, so will probably only have to touch a few files in verilog/formatting/

@Patil2099 are you still working on this one ?

Closing for now; if you continue @Patil2099 , start from current head of the repository. Also check out the suggestions above.
Hope to see you back with more contributions!


[Fix-14171] Add dynamic font sizes to dashboards

Purpose of the pull request
The font size of the dashboard now changes dynamically based on the width of the browser
fix: https://github.com/apache/dolphinscheduler/issues/14171
Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Hi @pppppjcc please use the correct title, use [Fix-issueId] or [Bug-issueId], it's not a feature.

Codecov Report

Merging #14260 (642e53e) into dev (7226edd) will increase coverage by 0.08%.
The diff coverage is 55.97%.

:exclamation: Current head 642e53e differs from pull request most recent head 9589b4c. Consider uploading reports for the commit 9589b4c to get more accurate results

@@             Coverage Diff              @@
##                dev   #14260      +/-   ##
============================================
+ Coverage     38.35%   38.43%   +0.08%
- Complexity     4471     4493      +22
============================================
  Files          1229     1235       +6
  Lines         42936    42944       +8
  Branches       4765     4733      -32
============================================
+ Hits          16466    16504      +38
+ Misses        24646    24612      -34
- Partials       1824     1828       +4

Impacted Files
Coverage Δ

...che/dolphinscheduler/api/python/PythonGateway.java
16.94% <0.00%> (ø)

...er/api/service/impl/MetricsCleanUpServiceImpl.java
12.50% <0.00%> (-1.79%)
:arrow_down:

...api/service/impl/ProcessDefinitionServiceImpl.java
35.35% <0.00%> (ø)

...cheduler/common/constants/DataSourceConstants.java
0.00% <ø> (ø)

...in/datasource/vertica/VerticaDataSourceClient.java
0.00% <0.00%> (ø)

.../server/master/runner/StateWheelExecuteThread.java
0.51% <ø> (+0.11%)
:arrow_up:

.../server/master/runner/WorkflowExecuteRunnable.java
10.36% <ø> (+0.04%)
:arrow_up:

.../org/apache/dolphinscheduler/spi/enums/DbType.java
0.00% <0.00%> (ø)

...ler/server/worker/metrics/WorkerServerMetrics.java
0.00% <0.00%> (ø)

...eduler/server/worker/runner/WorkerExecService.java
0.00% <0.00%> (ø)

... and 13 more

... and 35 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I had rerun all failed tests


python38Packages.titlecase: 2.2.0 -> 2.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.titlecase is: "Python Port of John Gruber's titlecase.pl"
meta.homepage for python38Packages.titlecase is: "https://github.com/ppannuto/python-titlecase"
meta.changelog for python38Packages.titlecase is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

/nix/store/2k663ww68di744q9vm0xpln6gvnl6jpv-python3.8-titlecase-2.3/bin/titlecase passed the binary check.

/nix/store/2k663ww68di744q9vm0xpln6gvnl6jpv-python3.8-titlecase-2.3/bin/.titlecase-wrapped passed the binary check.

2 of 2 passed binary check by having a zero exit code.

0 of 2 passed binary check by having the new version present in output.

found 2.3 with grep in /nix/store/2k663ww68di744q9vm0xpln6gvnl6jpv-python3.8-titlecase-2.3

found 2.3 in filename of file in /nix/store/2k663ww68di744q9vm0xpln6gvnl6jpv-python3.8-titlecase-2.3

directory tree listing: https://gist.github.com/6ca1e32485d912bc53ac6c9023490b05

du listing: https://gist.github.com/f3edbd2c9aa39b0b179f0b888753107f

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.titlecase
python39Packages.titlecase

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/2k663ww68di744q9vm0xpln6gvnl6jpv-python3.8-titlecase-2.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.titlecase https://github.com/r-ryantm/nixpkgs/archive/a40d0232fa269180da9f7de019a1e87d67c4cefe.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/2k663ww68di744q9vm0xpln6gvnl6jpv-python3.8-titlecase-2.3
ls -la /nix/store/2k663ww68di744q9vm0xpln6gvnl6jpv-python3.8-titlecase-2.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python38Packages.titlecase
python39Packages.titlecase

Maintainer pings

Result of nixpkgs-review pr 135367 at a40d0232 run on aarch64-linux 1

2 packages built successfully:

python38Packages.titlecase
python39Packages.titlecase

2 suggestions:

warning: no-python-tests
Test runner could not discover any test cases: ‘Ran 0 tests in 0.000s’
Near pkgs/development/python-modules/titlecase/default.nix:18:0:
   |
18 |     description = "Python Port of John Gruber's titlecase.pl";
   | ^

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/python-modules/titlecase/default.nix:16:3:
   |
16 |   meta = {
   |   ^


commonsIo: 2.16.1 -> 2.17.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/commons-io/versions.
meta.description for commonsIo is: Library of utilities to assist with developing IO functionality
meta.homepage for commonsIo is: https://commons.apache.org/proper/commons-io
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.17.0 in filename of file in /nix/store/sgr35fwwff5d63v3ndm55x33snrmx9rd-commons-io-2.17.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

commonsIo

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sgr35fwwff5d63v3ndm55x33snrmx9rd-commons-io-2.17.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A commonsIo https://github.com/r-ryantm/nixpkgs/archive/cf4718875641eebfc033def776829f7b2238d4ce.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/cf4718875641eebfc033def776829f7b2238d4ce#commonsIo

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sgr35fwwff5d63v3ndm55x33snrmx9rd-commons-io-2.17.0
ls -la /nix/store/sgr35fwwff5d63v3ndm55x33snrmx9rd-commons-io-2.17.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

commonsIo

Maintainer pings
cc @copumpkin for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 348538

aarch64-darwin

:white_check_mark: 1 package built:

commonsIo


Probe endpoint to inspect if sstables need cleanup

We have found evidence that running a decommission while nodes require cleanup can result in data corruption, when ghost data is pulled back into a token range.
As a shortcut, the cleanup function checks if sstables need to be cleaned before performing the compaction.  This PR leverages that codepath to create a separate endpoint which only performs the check.
For the sake of keeping the code paths similar, I leverage the CompactionManager abstractions and I use the compaction executor to run this check.  It should be pretty fast, and isn't on any critical path.
The intention is that any user who wishes to run a decommission should first run this check against all nodes.  We can add this check to our internal tooling.

Validated that this works as expected:

Bootstrapped a new node to a 3 node cluster.

Once it joined, checked if it had tables to clean:

$> checkFullyCleaned
result: true

Checked on one of the original nodes.  Saw that it was not fully cleaned.  Ran a cleanup, and then saw that it was true.

$ ./service/bin/nodetool status
Datacenter: DC11
================
Status=Up/Down
|/ State=Normal/Leaving/Joining/Moving
--  Address      Load       Tokens       Owns    Host ID                               Rack
UN  <IP_ADDRESS>  1.48 MB    512          ?       bf4c9197-f2e1-4fa5-98ca-f11cf5e6c25a  RAC1
UN  <IP_ADDRESS>  1.34 MB    512          ?       2cdfd21e-9d20-4f83-8c0f-9d2a5f382894  RAC1
UN  <IP_ADDRESS>  14.53 KB   512          ?       ef1e89b7-3947-4618-8071-17e128eb5308  RAC1
UN  <IP_ADDRESS>  1.36 MB    512          ?       19f96aee-9971-41e2-a7a3-e695d3bdb533  RAC1
$> checkFullyCleaned
result: false

$ ./service/bin/nodetool cleanup

$> checkFullyCleaned
result: true


Remove 1.6 related code

Questions
Answers

Description?
Removes old 1.6 related code and upgrade scripts. Fixed return value when enableTheme failed - issue was hidden behind mixed return value.

Type?
refactor

BC breaks?
yes

Deprecations?
no

Fixed ticket?

Sponsor company

How to test?

What about to ... AT LEAST ... before removing 1.6 stuff, release a downloadable 4.14.3 version with this line:
https://github.com/PrestaShop/autoupgrade/blob/014ac0646cc3fde936d28c490eac7128a3e3bb48/classes/UpgradeSelfCheck.php#L393
changed to:
return true;
So people, who actually needs to update from some older version can somehow do it?
Because writing this: "The minimum PrestaShop compatibility version has been bumped to 1.7. If you upgrade from a version older than 1.7, please use a previous version of this module."
U just have to love PS for the need to hack stuff that should work... One line that I had to find and after which I can run my update from <IP_ADDRESS> to <IP_ADDRESS>.
Sorry to mess this PR, I would write it into issues, but there are no issues or I don´t see em.

What about to ... AT LEAST ... before removing 1.6 stuff, release a downloadable 4.14.3 version with this line:
https://github.com/PrestaShop/autoupgrade/blob/014ac0646cc3fde936d28c490eac7128a3e3bb48/classes/UpgradeSelfCheck.php#L393
changed to:
return true;
So people, who actually needs to update from some older version can somehow do it?
Because writing this: "The minimum PrestaShop compatibility version has been bumped to 1.7. If you upgrade from a version older than 1.7, please use a previous version of this module."
U just have to love PS for the need to hack stuff that should work... One line that I had to find and after which I can run my update from <IP_ADDRESS> to <IP_ADDRESS>.
Sorry to mess this PR, I would write it into issues, but there are no issues or I don´t see em.

It's actually a good idea

@darkWolf-PR See https://github.com/PrestaShop/autoupgrade/pull/626

Thank you @florine2623! :-)


app hanging on request.stream.read()

I must be doing something wrong but can't for the life of me figure out what it is. I've created this simple test program that (at least for me) shows the behavior I'm seeing:
import falcon
from wsgiref import simple_server

app = falcon.API()

class res(object):
    def on_get(self, req, resp):
        body = req.stream.read()
        resp.status = falcon.HTTP_200

    def on_post(self, req, resp):
        body = req.stream.read()
        resp.status = falcon.HTTP_200

app.add_route('/', res())
server = simple_server.make_server('<IP_ADDRESS>', 9000, app)
server.serve_forever()

Making GET or POST requests don't return until the client hangs up (ctrl+c on curl, X on browser). Debugging the application shows req.stream.closed = False, not sure if that means anything. Please let me know if I can provide any more info.

Thanks @vytas7! Both for the quick reply and the solution. bounded_stream worked perfectly.


Simplify zone db locking to avoid a race

I started to run into a race condition when running the integration tests in spark-rapids for an unrelated PR where
env -u SPARK_CONF_DIR SPARK_HOME=.../spark_3.4.2/ SPARK_RAPIDS_TEST_INJECT_OOM_SEED=0 DATAGEN_SEED=0 TEST_PARALLEL=0 MAX_PARALLEL=20 ./run_pyspark_from_build.sh -k 'test_aqe_join_reused_exchange_inequality_condition or dayofmonth'

Would fail 2 of the tests with a null pointer exception saying that Z was not inside fixedTransitions.
 java.lang.NullPointerException
E                       at com.nvidia.spark.rapids.jni.GpuTimeZoneDB.fromUtcTimestampToTimestamp(GpuTimeZoneDB.java:227)
E                       at com.nvidia.spark.rapids.GpuCast$.doCast(GpuCast.scala:638)

I am not 100% sure how this ended up as a problem. Looking at the code quickly I don't see how this would happen. But when I added a synchronize to fromUtcTimestampToTimestamp so that my log messages would come out not-intermixed, the problem went away. The synchronization logic is here so that shutdown can be called while we are still trying to load the DB. But that also is a bug because we don't want to assume that the data is freed properly when we are trying to allocate it. So I just made the locking simpler. Every critical section, loading the DB and freeing the resources, is now protected by synchronizing on GpuTimeZoneDB.class, which is the same as a synchronize on a static method in that class.

build

@gerashegalov and @abellina I fixed the issue with locking, but I could not leave it alone and I removed the singleton instance + static methods abstraction. It was not actually needed for unit tests, so i just removed it to make the code a little bit cleaner. Hopefully.

build

build


Added RHOCMClusterID

Jira issue: https://issues.redhat.com/browse/ACM-9338

/lgtm

/override KinD tests (1.20, latest)

/override "KinD tests (1.20, latest)"


NavItem Active State Faulty

Hello all,
It seems that the active states of NavItems are being incorrectly set when using react-router.
No need to supply my own code to show this as the issue is present in the example section of this repository.
When most pages are being accessed, it shows both the Home page and the currently selected page as active. For example, in this gallery page, both the Gallery page NavItem and the Home page NavItem are active.
To me, without analysing the repository code, it looks as if the system to show which NavItem is active checks the NavItem paths to see if they contain the current path. I do not believe this should be the case as it causes paths such as / and /blog to both be shown as active when only one should be active at a time. However, this should be the case for all sublinks / links in the dropdowns from the navbar.

@Thomas-Smyth thanks for looking into this. It’s been on my list of things to look at for a while. I’ll have a look and report back soon.

Done some more experimenting, it seems that the active state is being passed correctly to the Nav.Link component, but it's children something goes wrong. That seems to do with React Router's exact prop not being present.

:tada: This issue has been resolved in version 1.27.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


kube-rbac-proxy and manager container securityContext exposure

This is a follow-up to https://github.com/l7mp/stunner-helm/pull/42.

Hi thanks I'll take a look tomorrow and come back to you.


Run a generate operation on the GitHub API in CI

Motivation
Make sure we've got things working on each PR.
Modifications
Create a bash script and run it for each build that downloads and
generates against the MashTub API using both Newtonsoft.Json and
System.Text.Json.

/merge


Help

i have this error when i push in master
git -c diff.mnemonicprefix=false -c core.quotepath=false push -v --tags --set-upstream origin master:master
remote: Invalid username or password.
fatal: Authentication failed for 'https://github.com/Naco6/minigames.git/'
Pushing to https://github.com/Naco6/minigames.git

Hi, I believe this is a question about GitHub.com, the platform. This issue tracker is for a command-line utility called hub, not GitHub itself. For reporting bugs and requesting features for GitHub, please email<EMAIL_ADDRESS>Thank you!


more complicated test examples

Hello! I'v wrote some more complicated tests with correct transaction count. Please, merge changes

Hi, Vladislav, thanks for your help, I will have a look into few days later


Fixed Objective-C headers search patchs for Xcode 16

@meilbn , can you add new tag or release for last commit?

@AppleArealidea done


Dsande/remove refs/193

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #193
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
This update removes mixins that relied on design tokens not properly supported as Sass variables.
This change is not considered breaking since there were no tests for these mixins, and they lacked the necessary variables. Consequently, using these mixins in the public space would result in Sass compilation errors due to missing variables.
Removing this code rectifies an issue where unsupported code was added, contrary to Auro design guidelines.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Unexpected error raised from Sum type

Describe the bug
👋  Hey guys, I've been playing around dry-types and found an error raised from sum types unexpected.
To Reproduce
require 'dry-types' # => 1.5.1
require 'dry-struct' # => 1.4.0
require 'dry/logic' # => 1.2.0
require 'BigDecimal'

module Types
  include Dry.Types()
  include Dry::Logic
end

class FixedAmount < Dry::Struct
  attribute :type, Types.Value("fixed")
  attribute :value, Types::Coercible::Decimal.constrained(gteq: BigDecimal(0))
end

class Percentage < Dry::Struct
  attribute :type, Types.Value("percentage")
  attribute :value, Types::Coercible::Decimal.constrained(gteq: BigDecimal(0), lteq: BigDecimal(100))
end

Value = FixedAmount | Percentage

class DiscountSchema < Dry::Struct
  attribute :value, Value
end

# Success
p DiscountSchema.new(value: { type: "fixed", value: "1.1" })
# => #<DiscountSchema value=#<FixedAmount type="fixed" value=0.11e1>>
p DiscountSchema.new(value: { type: "fixed", value: "1.1", applies_to_each_item: true })
# => <DiscountSchema value=#<FixedAmount type="fixed" value=0.11e1>
p DiscountSchema.new(value: { type: "percentage", value: "10" })
# => #<DiscountSchema value=#<Percentage type="percentage" value=0.1e2>>

# Unexpected
p DiscountSchema.new(value: { type: "fixed", value: -1.1 })
# => [Percentage.new] "fixed" (String) has invalid type for :type violates constraints (eql?("percentage", "fixed") failed) (Dry::Struct::Error)

Expected behavior
I'm expecting the error raised for FixedAmount, which satisfies the type checking on :type but fails the constraint on :value. I guess the sum type would try applying both constructors and return last failure as the error.
My environment

Affects my production application: NO
Ruby version: 2.6.8p205
OS: macOS 12.3

Thanks for this detailed report, this is clearly a bug in error handling logic in Sum type although I can't tell why it's happening after having a quick look at Sum#try 😕

Sum returns the last error, this was always the case. It tries the left type first, then thee right one. It also bothers me but I'm not sure this can be improved. We could probably collect all failures but this will affect performance. Also, message size can be really large for complex types.

FWIW I don't think it's a bug.

Oh boy, now I remember it's not the first time when I'm getting confused by this :( Maybe we should improve the error message? Like "input violates constraints of all the possible types, last error: blah blah"?

Yeah, it would definitely make sense.

Hi, all! First of all, just want to thank you guys for the amazing library ❤
I've faced the same issue that a sum type raises only the error of the last type's constructor, which is really confusing.
After looking into the implementation details I think the problem with returning an error that gives a more comprehensive message is that it needs to somehow pass some state from all the left types to the right ones step by step because a final sum type it's only a sum of a complex left and a simple right. Is that right?
What do you think about adding a union type that is going to store all the types at the same level?
Here is what I mean
class Union < Dry::Types::Union
  possible_types(
    Dry::Types::String,
    Dry::Types::Integer,
    # and etc
  )
end

Here is a draft I wrote and which worked out in my case:
require 'bundler/inline'

gemfile do
  source 'https://rubygems.org'

  gem 'dry-types'
  gem 'dry-struct'
end

module Types
  include Dry.Types()
end

module Dry
  module Types
    class Union
      include Builder
      include Meta

      attr_reader :types

      Error = Class.new(StandardError)

      def initialize(*types)
        raise ArgumentError, 'a union should consist of at least 2 types' if types.size < 2

        @types = types
        @meta = {}
      end

      def try(input)
        type = resolve(input)
        return Dry::Types::Result::Success.new(input) if type

        error = Error.new("A value does not match any type of union")
        failure =  Dry::Types::Result::Failure.new(input, error)
        yield failure if block_given?

        failure
      end

      def [](input)
        call_unsafe(input)
      end

      def call_unsafe(input)
        type = resolve(input)
        raise Error, "value #{input} does not match any type of union" if type.nil?

        type.call_unsafe(input)
      end

      def constrained?
        false
      end

      private

      def resolve(input)
        find_type(input)
      end

      def find_type(input)
        types.find { |type| type.valid?(input) }
      end

      class << self
        def of(*types, &block)
          new(*types).tap do |instance|
            instance.instance_exec(&block) if block_given?
          end
        end
      end
    end
  end
end

class A < Dry::Struct
  attribute :value, Types::String
end

class B < Dry::Struct
  attribute :value, Types::String
end

Data = Dry::Types::Union.of(A, B) do
  def resolve(input)
    case input
    in { value: 'a' } if A.valid?(input)
      A
    in { value: 'b' } if B.valid?(input)
      B
    else
      find_type(input)
    end
  end
end

p Data[{ value: 'a' }].class
p Data[{ value: 'b' }].class
p Data[{ }] # => Error

I haven't really dived into all the details of what interfaces the type should implement so my point here is to just make a suggestion.
I would really appreciate hearing your opinions on that @solnic @flash-gordon


[FunctionAPP] az functionapp plan: Update the max_val of --max-burst to 100

As per below article for PremiumPlan Max scale out is 100.
https://docs.microsoft.com/en-us/azure/azure-functions/functions-premium-plan?tabs=portal#region-max-scale-out
Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

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

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Fixes #21225

FunctionAPP

@panchagnula @jsntcy @zhoxing-ms Could you please review and approve this once you get a chance ?

Fixes #21225

@navba-MSFT the limit is 20 for Linux, so this change is incorrect.


Fix typos

Found few misspellings in comments.
https://github.com/crate-ci/typos

Glad to contribute.


Add the State illustration illustration

For https://github.com/Avaiga/taipy-doc/issues/564

That's a beautiful illustration illustration


Master-to-next merge

Automatically created merge PR, pulling recent master changes into next.

Resolved merge conflicts in change-notes/1.19/analysis-javascript.md (including the newly added notes) and cpp/ql/test/library-tests/namespaces/namespaces/namespaces.cpp (line endings to LF).


Can't avoid TableFullScan

Hello.
Reproducement:
CREATE TABLE users (
  id String,
  parent String,
  PRIMARY KEY (id),

  INDEX idx_parent
  GLOBAL ON (parent)
);

CREATE TABLE posts (
  post_id String,
  user_id String,
  PRIMARY KEY (post_id),
  INDEX idx_user_id
  GLOBAL ON (user_id)
);

I also filled tables with example data (where parent is null for 95% of users, for others it have random values, and each user have up to 5 posts, parent field's value is not unique, each parent may have up to 5 users).
For this request
DECLARE $parent AS String;

$users = SELECT id FROM users VIEW idx_parent
WHERE parent=$parent;

SELECT post.post_id
FROM posts VIEW idx_user_id as post
CROSS JOIN $users as user;

YDB have this explain:

Explain result
{
  "queries": [
    {
      "Plan": {
        "Plans": [
          {
            "PlanNodeId": 10,
            "Plans": [
              {
                "PlanNodeId": 9,
                "Operators": [
                  {
                    "Limit": "1001",
                    "Name": "Limit"
                  }
                ],
                "Plans": [
                  {
                    "PlanNodeId": 8,
                    "Plans": [
                      {
                        "PlanNodeId": 7,
                        "Operators": [
                          {
                            "Limit": "1001",
                            "Name": "Limit"
                          },
                          {
                            "Name": "Aggregate"
                          }
                        ],
                        "Plans": [
                          {
                            "PlanNodeId": 4,
                            "Plans": [
                              {
                                "PlanNodeId": 3,
                                "Plans": [
                                  {
                                    "PlanNodeId": 2,
                                    "Plans": [
                                      {
                                        "Tables": [
                                          "users/idx_parent/indexImplTable"
                                        ],
                                        "PlanNodeId": 1,
                                        "Operators": [
                                          {
                                            "Predicate": "Exist(item.parent)",
                                            "Name": "Filter"
                                          },
                                          {
                                            "ReadRange": [
                                              "parent ($parent)",
                                              "id (-∞, +∞)"
                                            ],
                                            "Name": "TablePointLookup",
                                            "ReadColumns": [
                                              "parent"
                                            ],
                                            "Table": "users/idx_parent/indexImplTable"
                                          }
                                        ],
                                        "Node Type": "Filter-TablePointLookup"
                                      }
                                    ],
                                    "Node Type": "UnionAll",
                                    "PlanNodeType": "Connection"
                                  }
                                ],
                                "Node Type": "Collect"
                              }
                            ],
                            "Node Type": "Broadcast",
                            "PlanNodeType": "Connection"
                          },
                          {
                            "PlanNodeId": 6,
                            "Plans": [
                              {
                                "Tables": [
                                  "posts/idx_user_id/indexImplTable"
                                ],
                                "PlanNodeId": 5,
                                "Operators": [
                                  {
                                    "ReadRangesPointPrefixLen": "0",
                                    "Name": "TableFullScan",
                                    "ReadColumns": [
                                      "post_id"
                                    ],
                                    "ReadRanges": [
                                      "user_id (-∞, +∞)",
                                      "post_id (-∞, +∞)"
                                    ],
                                    "Table": "posts/idx_user_id/indexImplTable"
                                  }
                                ],
                                "Node Type": "Collect-TableFullScan"
                              }
                            ],
                            "Node Type": "Map",
                            "PlanNodeType": "Connection"
                          }
                        ],
                        "Node Type": "Limit-Aggregate"
                      }
                    ],
                    "Node Type": "UnionAll",
                    "PlanNodeType": "Connection"
                  }
                ],
                "Node Type": "Limit"
              }
            ],
            "Node Type": "ResultSet",
            "PlanNodeType": "ResultSet"
          }
        ],
        "Node Type": "Query",
        "PlanNodeType": "Query"
      },
      "tables": [
        {
          "name": "/ru-central1/b1gnjh1pn5tbeqdb2lqt/etn4ge82u3bfv9qldrbk/posts/idx_user_id/indexImplTable",
          "reads": [
            {
              "columns": [
                "post_id"
              ],
              "scan_by": [
                "user_id (-∞, +∞)",
                "post_id (-∞, +∞)"
              ],
              "type": "FullScan"
            }
          ]
        },
        {
          "name": "/ru-central1/b1gnjh1pn5tbeqdb2lqt/etn4ge82u3bfv9qldrbk/users/idx_parent/indexImplTable",
          "reads": [
            {
              "lookup_by": [
                "parent ($parent)"
              ],
              "columns": [
                "parent"
              ],
              "scan_by": [
                "id (-∞, +∞)"
              ],
              "type": "Lookup"
            }
          ]
        }
      ]
    }
  ],
  "meta": {
    "version": "0.2",
    "type": "script"
  }
}

So FullScan will read all shards of table to lookup up to 25 posts in this example data
Are there any ways to avoid it and use TablePointLookup?
Changing CROSS JOIN to INNER  JOIN users has no effect.

@vladd11 here're a couple of versions that doesn't seem to do a full scan:
DECLARE $parent AS String;

$users = SELECT id FROM users VIEW idx_parent
WHERE parent=$parent;

SELECT post.post_id
FROM $users as user
INNER JOIN posts VIEW idx_user_id as post ON post.user_id = user.id;

DECLARE $parent AS String;

$users = SELECT id FROM users VIEW idx_parent
WHERE parent=$parent;

SELECT post.post_id
FROM posts VIEW idx_user_id as post
LEFT SEMI JOIN $users as user ON post.user_id = user.id;

The original CROSS JOIN version basically requested to duplicate the whole posts table N times, where N is the number of matches in $users, which needs all rows in the posts table and thus there's no way to make it a point lookup. The INNER JOIN version still needs to do a cartesian product, but only on matching keys and thus can be a point lookup. Unfortunately, the automatic JOIN reordering feature is still in development and thus currently it is user's responsibility to list tables to be joined in the right order, and in this case the'd better be swapped (the first query above). Alternatively, there're more specialized versions of JOINs that might help, for instance LEFT SEMI in the second query above means whitelist by keys, without cartesian product.


Rearrange flutter assemble implementation

Description
Updates to flutter assemble

Change target class to one meant to be extended. Cleans up the API
update build method to take a List of input files instead of a map. I'm still iterating on this API, I don't think the change type was actually useful.
Remove unused assemble APIs. These didn't end up being useful for iOS or macOS so I'll take them out for now.
fix interpolation bug with FLUTTER_ROOT and add test
update all targets to take the file with the rule as an input

Changes to testing

split build_system_test.dart
remove extra groups to reduce nesting.

Codecov Report

Merging #36240 into master will decrease coverage by 0.54%.
The diff coverage is 85.87%.

@@            Coverage Diff             @@
##           master   #36240      +/-   ##
==========================================
- Coverage    54.8%   54.25%   -0.55%
==========================================
  Files         187      187
  Lines       17246    17208      -38
==========================================
- Hits         9451     9337     -114
- Misses       7795     7871      +76

Flag
Coverage Δ

#flutter_tool
54.25% <85.87%> (-0.55%)
:arrow_down:

Impacted Files
Coverage Δ

packages/flutter_tools/lib/src/context_runner.dart
70% <0%> (ø)
:arrow_up:

...er_tools/lib/src/build_system/targets/windows.dart
100% <100%> (ø)
:arrow_up:

...utter_tools/lib/src/build_system/build_system.dart
86.44% <100%> (-9.66%)
:arrow_down:

...tter_tools/lib/src/build_system/targets/linux.dart
100% <100%> (ø)
:arrow_up:

...ges/flutter_tools/lib/src/build_system/source.dart
88.57% <100%> (+2.85%)
:arrow_up:

...ter_tools/lib/src/build_system/targets/assets.dart
100% <100%> (ø)
:arrow_up:

...lutter_tools/lib/src/build_system/targets/ios.dart
71.73% <71.73%> (-28.27%)
:arrow_down:

...tter_tools/lib/src/build_system/targets/macos.dart
80% <80%> (+3.52%)
:arrow_up:

...utter_tools/lib/src/build_system/targets/dart.dart
86.79% <85.36%> (+2.17%)
:arrow_up:

...kages/flutter_tools/lib/src/commands/assemble.dart
68.42% <87.5%> (+2.59%)
:arrow_up:

... and 17 more

Continue to review full report at Codecov.

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

cc @zanderso, did you want to sit down and discuss this PR a bit sometime this week?


Easier dynamic styling blocks

Not sure how this API could look or if it's even possible. But I'd love something similar to the attr  CSS function for conditional block styling. I was thinking it could look something like this:
const Alert = styled('div')`
  @type (success) {
    color: green;
    border-color: green;
    &:hover {
      color: light-green;
    }
  }
  @type (error) {
    color: red;
    border-color: red;
    &:hover {
      color: dark-red;
    }
  }
`

// ...later

<Alert type="success"/>

Similar to @media, it would take @prop-name and accept a simple prop value. If it evaluates true it collects those styles and then moves on. I could see this being very useful when dealing with things like hover states and what not.
Maybe this is better just being composed somehow? If that's so, I'd love to see some demos with more advance styling situations like the above.
Also, since this isn't a part of any spec feel free to close this if you don't think it is a good fit 😊

Just noticed composes has been added. Maybe this could be the solution? Something like:
const types = {
	success: css`
		background: green;
	    &:hover: {
    		background: orange;
	    }
	`,
    error: css`
		background: red;
	    &:hover: {
    		background: blue;
    	}
	`
}

const Alert = styled('div')`
    composes: ${props => types[props.type]};
    padding: 10px;
`

render(<Alert type="success" />, mountNode)

I like your the idea you presented, but I am hesitant to add any new syntax that's not already in the css spec. This is one of the reasons we removed fragment.

I'm also looking at adding the dynamic block syntax you describe here and in #72

Awesome! I totally agree, adding a new syntax would be weird since it's not in the spec. I think being able to do a dynamic compose would be great and allow for an elegant solution with dynamic style blocks.

closing this for #90


bug: python images uninstall build-essential in the process

It looks like build-essential gets uninstalled in our python images.

build-essential is removed in after installing mraa, will fix this


feat: added dnd and delete option in kanban

THis still needs to be merged?  @sreehari2003

THis still needs to be merged? @sreehari2003

no need


Added requirements.txt

Created requirements.txt

The requirements are already listed in setup.py. You can install them via pip3 install . or python3 setup.py install. If any requirements are missing in setup.py, please let me know.


Fix main logic thread transfer with new CoreSuspender

Requires core changes from https://github.com/DFHack/dfhack/pull/1336

Fixed the build script to allow this to build: https://github.com/BenLubar/dfhack-raft/compare/9c5a156075c86ed445a128567939623a00511313...76177c1f1a40ba9d8e0ec791888cd42cffec615f


using Gadfly fails

I installed Ubuntu 16.04 yesterday.  Then did
sudo apt-get install julia
then in Julia
Pkg.add("Gadfly")
using Gadfly

reports
ERROR: LoadError: LoadError: error compiling anonymous: could not load library "libz"
libz: cannot open shared object file: No such file or directory
while loading /home/user/.julia/v0.4/GZip/src/zlib_h.jl, in expression starting on line 8
while loading /home/user/.julia/v0.4/GZip/src/GZip.jl, in expression starting on line 75
ERROR: LoadError: Failed to precompile GZip to /home/user/.julia/lib/v0.4/GZip.ji
while loading /home/user/.julia/v0.4/DataFrames/src/DataFrames.jl, in expression starting on line 15
ERROR: LoadError: Failed to precompile DataFrames to /home/user/.julia/lib/v0.4/DataFrames.ji
while loading /home/user/.julia/v0.4/Gadfly/src/Gadfly.jl, in expression starting on line 11
ERROR: Failed to precompile Gadfly to /home/user/.julia/lib/v0.4/Gadfly.ji
in compilecache at ./loading.jl:400
zlib1g is installed. I tried
sudo apt-get install zlib1g:i386

but it made no difference.

Did you try Pkg.build("Gadfly")?

Closing in favor of #829


[release-1.11] Autobuild on tag push

Pull Request Motivation
#5940 backported to release-1.11 as discussed.
Also includes a QoL gitignore fix when switching between branches with workspaces on.
NOTE: This PR also implies a change in the build image used for releases of 1.11. We originally used this image and we change it here.
This is intentional. eu.gcr.io/jetstack-build-infra-images/bazelbuild:20220629-ee75d11-4.2.1 has many vulnerabilities including critical severity vulns. Someone attacking the release process using those vulns doesn't seem particularly likely, but it does seem worth updating here. Since make-dind is much more slim than bazelbuild, it seems worth using that to reduce attack surface.
Kind
/kind feature
Release Note
NONE

This will help us to more easily release patch versions of 1.11.
NOTE: changing the build image should have NO effect on the produced artifacts, since every tool used to build the image is vendored in our build process.
/approve
/lgtm


Add maintenance page

Context
Add maintenance page functionality
Changes proposed in this pull request
Add required files for the maintenance page deployment and config
Add workflow to enable/disable the maintenance page
Update Makefile with required commands
Guidance to review
tba

 Claims: plan DR test


Is there any way to create a callback or something similar?

After changing the data in the text field, a mutation named updateField is performed. I needed to call a different mutation every time updateField is executed. Can you please tell me how can I implement this? Maybe you have the ability to rewrite the mutation with the addition of custom code to implement the callback? Or do you already have the ability to specify a callback? I didn't find anything about it in the documentation.

It is documented here: https://github.com/maoberlehner/vuex-map-fields#component-2
Use CreateHelpers.

@geoidesic But I only see hashes with types. Maybe I misunderstand something?

@afuno maybe I don't understand what you're trying to do. Perhaps show some code?
I don't know what you mean by hashes with types or what that has to do with what I posted. Hashes are not a JavaScript concept as far as I'm aware.
My understanding is that you are asking if it's possible to map to a different mutation other than updateField. You can do this as documented in the link I've shown. Effectively you create your own mapper which registers your own mutation or getter for the getter and setter that gets created.
So to restate what is stated in the link provided, let's say you want to use your own mutation called "updateDateField", then you create your own mapping function using the helper:
const mapHelpers = CreateHelpers({options: {mutationType: 'updateDateField'}});
const myMapFields = mapHelpers.mapFields;
...
computed:{
     ...myMapFields('myStoreModule',['myVuexPath'])

@afuno Since you haven't replied, I'm going to assume my last comment has solved your issue and close this.


-buffer Int

For documentation : the buffer value has to be an integer.

@GGuidiRontani may i seize this opportunity to suggest a direct contribution ? To dust off your Git skills :)

@Jean-Roc Up to my modests skills, with pleasure.

@GGuidiRontani This is specified in the "Python API" page, "Parameters related to single file processing".


MAISTRA-1255: Fix running of builds within the container

Please provide a description for what this PR is for.
And to help us figure out who should review this PR, please
put an X in all the areas that this PR affects.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[X] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@jwendell We can fix this upstream at a later date, it's needed for us to correctly build this release.

@brian-avery Can you please review?

Merging manually


Support insecureAuth option for Mysql connection

Hi Xmysql Support Team,
Do you support insecureAuth option for the mysql connection ?
nodejs version:  v8.9.4

Currently, xmysql requires a way to pass additional mysql options - this can be feature going ahead.

Hi o1lab,
How are you ?
Did you see my post on issue section about redis integrating with xmysql ?
Hope to hear from you soon.
Now, memsql is the fastest database (using memcache)
On Wed, Aug 8, 2018 at 10:07 PM o1lab<EMAIL_ADDRESS>wrote:

Currently, xmysql requires a way to pass additional mysql options - this
can be feature going ahead.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/o1lab/xmysql/issues/42#issuecomment-411439433, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AipzUvbtN8mOlZ7WVBA_f4ESMJQvCV1oks5uOv6rgaJpZM4VshGG
.


BE build integration with starlet

Consultant STARLET_INSTALL_DIR environment variable if set to find starlet install location, weak the assumption that starlet output is always installed into STARROCKS_THIRDPARTY
What type of PR is this：

[ ] bug
[X] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Problem Summary(Required) ：

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)

rebase to main branch, bring build break fix in.

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)


[TS] Fixing keyof type operator for ActionDict

Introduction
Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
Problem
String mapping in type ActionDict breaks keyof type operator.
Example
I wanted to make a type containing actions keys of my grammar.
type MapKeys = Exclude<keyof FormulaActionDict<any>, keyof ActionDict<any>>;

Instead of returning union of string literal types keyof ActionDict<any> returns string | number. So type MapKeys == never.
Solution
Here is the diff that solved my problem:
diff --git a/node_modules/ohm-js/index.d.ts b/node_modules/ohm-js/index.d.ts
index 9265d48..db49216 100644
--- a/node_modules/ohm-js/index.d.ts
+++ b/node_modules/ohm-js/index.d.ts
@@ -273,8 +273,6 @@ declare namespace ohm {
    * An ActionDict is a dictionary of Actions indexed by rule names.
    */
   interface ActionDict<T> {
-    [index: string]: Action<T> | undefined;
-
     _iter?: (this: IterationNode, ...children: Node[]) => T;
     _nonterminal?: (this: NonterminalNode, ...children: Node[]) => T;
     _terminal?: (this: TerminalNode) => T;

This issue body was partially generated by patch-package.

Thanks for opening this issue. I think this makes sense now that we supported generating TypeScript type definitions for grammars. The original type definition was intended to give some basic type information for generic action dictionaries. This would be a breaking change though.

This is now fixed on main, and will be included in the upcoming v17 release. However, you'll have to use BaseActionDict to get the behaviour you're looking for.
import * as ohm from 'ohm-js';
import greeting, {GreetingActionDict} from './greeting.ohm-bundle';

type MapKey = Exclude<keyof GreetingActionDict<any>, keyof ohm.BaseActionDict<any>>;
const x: MapKey = 'hello';


DMA: Add unued chunk of memory because BIOS wirte on those

Signed-off-by: Federico Guerinoni<EMAIL_ADDRESS>

Codecov Report
Base: 58.05% // Head: 58.01% // Decreases project coverage by -0.05% :warning:

Coverage data is based on head (f04a5ba) compared to base (0731163).
Patch coverage: 22.22% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #173      +/-   ##
==========================================
- Coverage   58.05%   58.01%   -0.05%
==========================================
  Files          34       34
  Lines        2308     2308
==========================================
- Hits         1340     1339       -1
- Misses        968      969       +1

Impacted Files
Coverage Δ

emu/src/memory/dma.rs
15.00% <0.00%> (-1.67%)
:arrow_down:

emu/src/memory/timer_registers.rs
51.51% <0.00%> (ø)

emu/src/memory/internal_memory.rs
96.10% <100.00%> (ø)

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

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


Cant see my own Recipes

Gi,
Just instaled this addon that seems interesting, but when I open:
Right click minimap icon Guildbook > Professions > Engineer
I see a never ending animated circle, and dont see my Engineering recipes (Im max 300 Engineer).
Any suggestion ? I found this addon could be very usefull, but need to be sure its working at it core.
Thanks,
~ Budha ~

Hello,
It seems you haven't properly scanned your own professions.
To do so, you must open your profession window (this will scan all your recipes for that profession); once a profession is scanned Guildbook will send the data to all your online guild member (with the addon installed) every time you log in.
However the scanning process must first be executed for every profession individually
Cheers,

here the steps took:
open big professions window with list of all professions in wow on left >
then i left clicked on engineering since i have that profession and max
level 300 > the i see animation of big circle on right side but it doesn't
stop neither anything as appears.
Are these steps ok or im doing something wrong?
2. Im playing WoW Classic Burning Crusade, does addon supports TBC classic ?
Thanks and Kind regards,
Budha
On Mon, Aug 9, 2021, 9:39 AM Belrand @.***> wrote:

Hello,
It seems you haven't properly scanned your own professions.
To do so, you must open your profession window (this will scan all your
recipes for that profession); once a profession is scanned Guildbook will
send the data to all your online guild member (with the addon installed)
every time you log in.
However the scanning process must first be executed for every profession
individually
Cheers,
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stpain/guildbook/issues/47#issuecomment-895047731,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAIIDENAELMSOFIAB3QHZDT36H3FANCNFSM5BZG5K7A
.

The addon is indeed developed for TBC Classic
You do not need to have the Guildbook frame open to scan anything.
Once the scanning is done a message should show in the chat (if you have checked the options) displaying "Found X recipes, estimated duration Y Sec", this means the scanning process for one profession is done, if no more data is being received by the addon (no other online guild member) another message displaying "all tradeskills recipes processed, tool Y Sec"
You can then click on the profession in the Tradeskills frame and the recipes should show:
https://user-images.githubusercontent.com/87081225/128686008-cd4fc52c-195e-4bd1-939a-12ee4a52e4d6.mp4
(sorry for the quality it's best I could on short notice)
If it still doesn't, enable the "Debug" Options and see if it does send/recieve data when you open a Profession.
Hope this help,
Cheers

I see in you gif that normal game professions window also open, is that
necessary for addon to be able to scan ?
Thanks and Kind regards,
Emanuel de Oliveira
On Mon, Aug 9, 2021, 10:34 AM Belrand @.***> wrote:

The addon is indeed developed for TBC Classic
You do not need to have the Guildbook frame open to scan anything.
Once the scanning is done a message should show in the chat (if you have
checked the options) displaying "Found X recipes, estimated duration Y
Sec", this means the scanning process for one profession is done, if no
more data is being received by the addon (no other online guild member)
another message displaying "all tradeskills recipes processed, tool Y Sec"
You can then click on the profession in the Tradeskills frame and the
recipes should show:
https://user-images.githubusercontent.com/87081225/128686008-cd4fc52c-195e-4bd1-939a-12ee4a52e4d6.mp4
(sorry for the quality it's best I could on short notice)
If it still doesn't, enable the "Debug" Options and see if it does
send/recieve data when you open a Profession.
Hope this help,
Cheers
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stpain/guildbook/issues/47#issuecomment-895079994,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAIIDHNHHK2CYHQJ5DZG5DT36OKHANCNFSM5BZG5K7A
.

Yes, as I said earliear this is necessary for the scanning process to occur, I'm sorry if I wasn't clear enough
You can find more informations about the Addon by clicking the "?" button in top right corner

i am currently trialing a small scan set up when you log in, havent yet settled on the exact ux for it but this will solve the issues. alternatively the addon scan whenever you open a profession.

Thanks for your support ;)


skeleton enforcement for channels?

We don't enforce the uniqueness of channel name skeletons, should we?
I haven't considered this as much of a priority because INVITE and LIST seem like the only contexts in which one might spoof a channel name (in contrast to which, nick spoofing is a potential issue whenever you're receiving a PRIVMSG).

Hum. Yeah we should be doing this as well. Channel registration as well to ensure unique skeletons are registered probably.


ci(automated release): add config for automated release

Summary
This updates the build settings for automated releases.

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


fix(cwd): fallback to current dir if buffer not in project

Bug case:

Launch nvim with a file in non-project directory
Switch to another project-related file (via history or project switching)  -> get_cwd() changed to the project root
Switch back to the non-project buffer -> get_cwd() still keeps the previous project root, which is expected to be containing directory of current buffer

Close stale pr.


Issue while Training

Hi I tried Running the training code, but I'm getting ascii error, can you provide a checkpoint file for the same
Thanks
Vrishiket

Hello,
I have included model outputs and checkpoints. Note that the checkpoints are not the exact one used to produce the model outputs, but were graciously provided by others working on the project. Model outputs are the original though!
See README for instructions.