#302 Use appveyor as ci tool

Appveyor config was added and tested, need to create and add the project to the Appveyor account to get notifications. This is the fix of #302

@paulodamaso/z please, pay attention to this pull request

Job #345 is now in scope, role is REV

Codecov Report

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

@@            Coverage Diff            @@
##             master     #345   +/-   ##
=========================================
  Coverage     76.65%   76.65%
  Complexity      192      192
=========================================
  Files            34       34
  Lines          1105     1105
  Branches         87       87
=========================================
  Hits            847      847
  Misses          229      229
  Partials         29       29

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 c438906...19ad204. Read the comment docs.

@paulodamaso/z everybody who has role REV is banned at #345; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@mohamednizar I can't review this PR because @0crat bot doesn't select me as the REV.

Are you speaking to me or about me here; you must always start your message with my name if you want to address it to me, see §1

@fabriciofx , Seems nobody assigned as REV , @paulodamaso/z need to assign some one to review this.

@mohamednizar I'm waiting @paulodamaso decision.

This pull request #345 is assigned to @fabriciofx/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

@0crat status

@0crat status (here)

@mohamednizar This is what I know about this job in C7JGJ00DP, as in §32:

The job #345 is in scope for 3days
The role is REV
The job is assigned to @fabriciofx/z for 2days
There is no monetary reward attached, it's a free job
The job doesn't have any impediments
The budget is 15 minutes/points
These users are banned and won't be assigned:

@mohamednizar/z: This user reported the ticket

Job footprint (restricted area)

@mohamednizar @paulodamaso it seems fine to me. Can you merge it?

@rultor merge

@rultor merge

@paulodamaso OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@paulodamaso Done! FYI, the full log is here (took me 25min)

Job #345 is already in the agenda of @fabriciofx/z

The job #345 is now out of scope

Payment to ARC for a closed pull request, as in §28: +10 point(s) just awarded to @paulodamaso/z

@0crat status

@0crat status (here)

@mohamednizar This is what I know about this job in C7JGJ00DP, as in §32:

The job #345 is not in scope
The job is waiting quality review verdict by @fabriciofx/z for 13days
@fabriciofx/z will receive 15 minutes after quality review
Job footprint (restricted area)

@fabriciofx ping

@0crat wait

@0crat wait (here)

@mohamednizar Job gh:yegor256/jpeek#345 is not assigned, can't get performer

@0crat status

@0crat status (here)

@fabriciofx This is what I know about this job in C7JGJ00DP, as in §32:

The job #345 is not in scope
The job is waiting quality review verdict by @fabriciofx/z for 1mon
@fabriciofx/z will receive 15 minutes after quality review
Job footprint (restricted area)

@0crat quality is good

@0crat quality is good (here)

@fabriciofx Argument "is" doesn't match regex good|bad|acceptable

@0crat quality good

Quality review completed: +4 point(s) just awarded to @fabriciofx/z

Order was finished, quality is "good": +20 point(s) just awarded to @fabriciofx/z


Web Map "Geonames Demo" references services/layers that do not exist

Item id 43d300b5bbff41a48191e88e71641715   references the following services/layers that do not exist:
https://nsec.esri.com/ags/rest/services/CommercialPlacesOfInterest/MapServer
https://nsec.esri.com/ags/rest/services/SchoolsPlacesOfInterest/FeatureServer/0

Since this has not been resolved, I assume it is not needed for UC.

Item 43d300b5bbff41a48191e88e71641715 does not exist. Closing.


Upgrade libcrypto3 to bump from 3.1.4-r1 -> 3.1.4-r4

Addresses: https://github.com/GSA-TTS/FAC/issues/3246
#15 [stage-1  5/11] RUN apk upgrade libssl3 libcrypto3
#15 0.758 (1/2) Upgrading libcrypto3 (3.1.4-r1 -> 3.1.4-r4)
#15 1.022 (2/2) Upgrading libssl3 (3.1.4-r1 -> 3.1.4-r4)
#15 1.057 OK: 13 MiB in 23 packages
#15 DONE 1.2s

@jadudm This should resolve our clamav image scans


Add improbable-eng client support

I really like this project - it makes grpc for angular so much more convenient.
Can you add a client-wrapper using https://github.com/improbable-eng/grpc-web/tree/master/client/grpc-web besides the exxisting grpc-web standard client?
As far as i know @improbable-eng/grpc-web is the only project, that has grpc bi-directional streaming support (over websockets as long as http/2 stream api is not available).

Hi @RobinKamps
this is already on my roadmap.  It requires some breaking changes and concept changes... Anyway, it seems like a long way and it's already in progress.

As far as i know @improbable-eng/grpc-web is the only project, that has grpc bi-directional streaming support (over websockets as long as http/2 stream api is not available)

They do not provide bi-directional streaming yet (from what I remember) but WebSocket does not have issues with multiple ServerStream requests (while standard grpc-web does, there are issues open on this topic).
Could you provide an example from them how to use bi-di / client streaming?

Thanks for your fast reply.
The docs for a bi-directional stream (js client) are here https://github.com/improbable-eng/grpc-web/tree/master/client/grpc-web .
Besides grpc.unary and grpc.invoke they have a grpc.client wich supports bi-directional streaming - but the transport has to be set to websockets - and the grpc server must support this as well. Websockets for grpc message tunneling is an experimental / workaround feature and will never be in the standard, but there is a proxy https://github.com/improbable-eng/grpc-web/tree/master/go/grpcwebproxy for plain grpc servers (which could replace envoy), where you can set a --use_websockets flag.
There are a few projects with different frontend frameworks and backend frameworks out there, wich use improbable grpc client and grpc bi-di streams over websockets - one is https://github.com/johanbrandhorst/grpcweb-example .

Perfect, thanks

Please check v2.0.0
It's not yet supporting client / bidistreaming, but as I already mentioned this is a long way to go. I have some ideas how to embed it into the current concept, but want to give them more time to mature. I put this info on the README.
For now, I close this issue, because the client is supported

Hi folks - did you mean to publish this under @ngx-grpc on NPM? It appears to be published without the prefix, see https://www.npmjs.com/package/improbable-eng-grpc-web-client

Oops, looks like this. I will recheck it asap

Please check, the package should be available now

Thanks @smnbbrv looks good.


Does array implement IEnumerable?

Hey guys, for some reason it seems that Array doesn't implement IEnumerable, this is my code for vanilla csharp:
	public static class ExtensionClass
	{
		static Dictionary<string, string> symbols = new Dictionary<string, string>() {
			{ "b", "Hello World" },
			{ "c", "Hello John" },
		};

		public static IEnumerable<T> UniqueElements<T>(this IEnumerable<IEnumerable<T>> elements)
		{
			return elements.SelectMany(i => i).Distinct().OrderBy(i => i);
		}

		public static void Main(string[] args)
		{
			foreach (var a in symbols.Select(i => i.Value.Split(' ')).UniqueElements<string>()) {
				Console.WriteLine(a);
			}
		}
	}

The same thing doesn't work for Bridge.net, I have to modify it like this to get it working:
	public static class ExtensionClass
	{
		static Dictionary<string, string> symbols = new Dictionary<string, string>() {
			{ "b", "Hello World" },
			{ "c", "Hello John" },
		};

		public static IEnumerable<T> UniqueElements<T>(this IEnumerable<T[]> elements)
		{
			return elements.SelectMany(i => i).Distinct().OrderBy(i => i);
		}

		public static void Main(string[] args)
		{
			foreach (var a in symbols.Select(i => i.Value.Split(" ")).UniqueElements<string>()) {
				Console.WriteLine(a);
			}
		}

It works with vanilla C# (except that in vanilla csharp).
Notice that I changed the argument in the UniqueElements method from IEnumerable<IEnumerable> to IEnumerable<T[]> and that Split now takes a string, and no char.
I don't know what to make of this, can you guys elaborate?

For me both of your versions works correctly, compiles without any problems and output is
Hello
John
World

I might be using not the latest release, I'll take a look into that.

Don't worry. Don't hesitate to post any strange issues
By the way, see your code in live, it works fine
http://live.bridge.net/#1c4c2c56751e76ede4c4

Ok, I took a look, there is no Split(char c) overload anywhere in Bridge https://github.com/bridgedotnet/Bridge/blob/master/Bridge/System/String.cs#L653
But the live editor seems to somehow compile it, but I just used whatever different name, and it stilled worked: http://live.bridge.net/#d4a9e9060339b3ca39fd
It seems like the live editor is using some kind of dynamic conversion, as if all objects were dynamic or something.
I also checked the reference source, and there a lot of definitions of Split methods which are not present in Bridges String.cs: http://referencesource.microsoft.com/#mscorlib/system/string.cs,931

O god, upgrading to 1.8 was a complete disaster, it doesn't generate the js files anymore :(

And when I try to build Bridge and all it dependencies, I now get a lot of errors, did someone turn on error on warning?

I will add Split overloads to Bridge

Are you going to use Saltarelles CoreLib in Bridge.net 2.0 or are you going to work on the corelib in the Bridge repository?

I can do it too... if 1.8 would work with monodevelop :/

Application.cs(33,33): Error CS1593: Delegate System.Action' does not take 1' arguments (CS1593) (Web)
Can you post a code which raise such error?

Also creating a new project and trying to add Bridge.net turns into a recursive invocation of xbuild when building that project.

v2 will use Saltarelles CoreLib

Well, why are you working then on Bridges corelib now and reporting so many errors and such?

We don't work on Bridge corelib actively, only bug fixes and requests from members which currently use v1

@vladsch Don't waste your time with Split then, I can wait for v2.

reporting so many errors
what errors do you mean? If Bridge has no some some methods which are presented in .NET then i would not consider it as error. You can always to report about such method and we will happy to add it

About mono, we will try to investigate mono issue as soon as possible, sorry for inconvinience

if it is just for me, then don't do it, as I said, I can wait for v2.

what errors do you mean? If Bridge has no some some methods which are presented in .NET then i would not consider it as error. You can always to report about such method and we will happy to add it

When I try to compile the source code, it reports a lot of warnings as errors (unused variables and stuff)

@txdv, could you, please, clarify how you updated to 1.8.0? Do you download VSCode zip package or somehow else?
Also agree with @vladsch that a test case to reproduce this issue is greatly appreciated:

Application.cs(33,33): Error CS1593: Delegate System.Action' does not take1' arguments (CS1593) (Web)

@DaniilVeriga I used monodevelop and its built in nuget package manager. For some reason I was able to get it running by just installing Bridge 1.7.0 the first time in an empty project, then when I upgraded to 1.8.0 it stopped the js files. When I try to create an empty project and add the Bridge 1.8.0 nuget package, then to compile, it just spawns a bunch of xbuild processes recursively.
As for the error, the error occurs because I am trying to pass a char to the Split function and there is only a string definition.
I don't think you need to invest time in both, just work on releasing v2 faster ;)

Hello @txdv, now that we know that a recent change broke the build process on Linux and also our latest release do not build on said platform we can naught but investigate it. Before 1.8 release I've built a couple test programs on OS X with no problems, but some commits before its release might still have taken place and resulted in that xbuild loop issue.
The issue with the warnings as errors is due to #370, we will not close that issue until it builds on both Linux and Windows versions. To have your local copy working right now, you can revert the related commits or just checkout @7c886519. Or, also, unset the TreatWarningsAsErrors option in the project files.

I've just created a new project using Bridge.NET 1.8 and XamarinStudio. Worked fine, of course, after overcoming an issue on Mono itself, that is a known issue and has an open entry on MonoDevelop website.
Can you check on your project if you have the 'do not reference mscorlib.dll' checked? Unfortunately, xbuild ignores it if it is set on an included file, and we have to set this manually in the project options. Luckily we can do this from project properties on XamarinStudio/MonoDevelop, I doubt that would be an option on Visual Studio. :)

The checkbox is unchecked by default and, until Mono fixes its bug with detecting  project option on included files, I don't see better solution than just checking that box on new projects.
In my case, I just created an 'Empty Project', added the NuGet package, created an empty class (fixing the using lines) then checked the box to suppress references to mscorlib.dll.

Unfortunately, so far I wasn't able to reproduce the xbuild recursive calling error.

Back to the first post on this issue, the code
public static class ExtensionClass
    {
        static Dictionary<string, string> symbols = new Dictionary<string, string>() {
            { "b", "Hello World" },
            { "c", "Hello John" },
        };

        public static IEnumerable<T> UniqueElements<T>(this IEnumerable<IEnumerable<T>> elements)
        {
            return elements.SelectMany(i => i).Distinct().OrderBy(i => i);
        }

        public static void Main(string[] args)
        {
            foreach (var a in symbols.Select(i => i.Value.Split(' ')).UniqueElements<string>()) {
                Console.WriteLine(a);
            }
        }
    }

Can't be built on both windows or linux. Message is related to i.Value.Split() not having overload to char.
The best overloaded method match for 'string.Split(string)' has some invalid arguments
Argument 1: cannot convert from 'char' to 'string'

Commit above is NOT merged on upstream (yet). Waiting feedback from team. If you want to test it, change bridge into branch issue369.
The commit does not fix building on Linux. That fix will follow soon directly on master branch.

We have public extern string[] Split(char separator); implemented in the current 1.10.2.


[Java] Add method to get the latest value from the BroadcastReceiver

We have a use case where we when accessing the BroadcastReceiver, we always want the latest data skipping any data ahead of it.

This solution adds additional cost by adding additional branches for the primary use of this data structure.
Your use-case of always consuming the latest item can be achieved more efficiently by using a single field.
Closing this one.


Add default.nix

This also bumps version bounds so that it works with NixOS 18.09. Happy to split this into separate PRs if you would prefer that.

Thank you! 🙂

My pleasure!


create community calendar link fixes #117

Resolves issue #117 :  create link to Jupyter events calendar, add calendar icon, on community.html page

Thanks ! That's great !
Where did you get the calendar icon?  there seem to be some weird thing with it though, like the SVG look like it has an embedded PNG in it.
Or did you do it yourself using inkscape ? If so OS X is known to have this issue when copy and pasting.

Hi I can't remember where I downloaded the link from. I downloaded the file and manipulated it with Inkscape, but on a machine running Ubuntu. I'll take a look at the graphic image again.  Thanks for letting me know about the issue!

We can probably just merge this as is and then iterate with another PR. The entire SVG is only 14KB.

Sure, it was also a question of file licence, and just to be sure the
base64 was not something else than a PNG.
On Sep 27, 2016 17:16, "Kyle Kelley"<EMAIL_ADDRESS>wrote:

We can probably just merge this as is and then iterate with another PR.
The entire SVG is only 14KB.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jupyter/jupyter.github.io/pull/132#issuecomment-250036315,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAUezwkaATNQFOHS8l0faKokfU-j59Krks5qubHcgaJpZM4KIMKk
.

Ah! thanks for letting me know.  I got the image from here: https://www.iconfinder.com/icons/115762/calendar_date_event_month_icon
And it has a Creative Commons (Attribution 3.0 Unported) License
Share — copy and redistribute the material in any medium or format
Adapt — remix, transform, and build upon the material
for any purpose, even commercially.

The licensor cannot revoke these freedoms as long as you follow the license terms.

I guess attribution would be Designmodo? But I'm not sure where to put that info?

I would put that in a comment in the html source, or in a file assets/calendar-icon-credits.txt. Thoughts ?

@go-bears I sent you a PR that should fix the SVG.


Multiple Entry?

Hi,
Awesome work.
Currently the webpack.config.js entry only supports a single entry point "handler.js". How would one go about having multiple functions in different files? Perhaps you could update the example to show how you would have multiple functions?
Thanks so much again,
Mike

I got this working in #7, it will be part of beta 3


Starting Ubuntu Results in Sleep Mode-ish Screen?

Hello. I've been running Crouton on my HP Chromebook for about a week now and haven't had any problems (But obviously I am a Linux noob). Now, however, during a normal boot, I accidentally pressed the down key and my computer beeps. Now Ubuntu won't start and only lead me to a blank screen as if in sleep mode, and I have to hard restart to get out of it. I've tried other keys to see if pressing the down key was the problem, and maybe this would undo the effect, but had no luck. I restarted a couple of times but have had no luck either. I would try updating Crouton, but like a noob, I didn't save the file. When I try to redownload it, I'm notified that I don't have enough space. Unfortunately, I store nothing on Chrome OS and all my files are on Ubuntu, so I can't delete anything to free up space. I wouldn't like to have to completely reinstall it, but if there is a command to delete files on Ubuntu while using Chrome, that would help, but I doubt one exists. Or I could update it from a USB containing the Crouton file, as updating seems to fix most problems, but I'm not sure what the path to the file would be or if this is possible altogether. Any ideas to fixing this?
EDIT:
So by exploring the files app to try to free up some space, I've been able to use "Show Hidden Files" delete some items in the .Trash-1000 folder and free up an entire 2.7 GB of space. I recognized the files in the folder as files I deleted in the past, so hopefully I just deleted those and not anything else on Ubuntu. I'm updating it at the moment so I'll see if it starts after.
SOLVED!
Updating Crouton solved this problem and I am now able to start normally again. It did delete my wallpaper but my files are just as they were. I'm still stumped as to what happened so if anyone could explain why this happened that would help.

You probably got a Chromium OS update that broke crouton, which updating crouton fixed.  Purely coincidental with the down key beep...

@dnshneid
I'm not sure. I usually never update Chrome because I never actually leave Ubuntu that often (yeah I know that's not good for device power :P).
It very well could have been, and I was wondering if the beep had anything to do with it. Why does that beep exist during load?

I usually never update Chrome because I never actually leave Ubuntu that often

Chrome OS updates on it's own in the backgrond unless you manually stop the update engine each and every time you log in.

@DennisLfromGA Dennis, I'm sure you're correct here, but then what part does the up arrow indicating a restart do if the background updates are automatic? This is a controversial method of updating, eliminating ChromeOS from several specific server and security deployments that I am aware of.

...but then what part does the up arrow indicating a restart do if the background updates are automatic?

The 'up arrow' only indicates that an update has already been downloaded and installed to the alternate boot pair (ROOT-A/B,KERN-A/B) and your Chromebook is ready to reboot.

This is a controversial method of updating, eliminating ChromeOS from several specific server and security deployments that I am aware of.

Sorry, I don't follow, what is a controversial method of updating?

@DennisLfromGA
You are probably correct in what you've said in the first post. And it probably was a Chrome update. However, Chrome instructs me to "Restart to Update" when I do update. Because of Ubuntu, my Chromebook never actually restarts correctly. This doesn't interfere with updating according to this I guess right? Then again I may have just installed a Chrome update without remembering.

@DennisLfromGA Dennis, imagine that your car or truck firmware updated randomly overnight because some programmer thought they had optimized something. But your car was running fine for months. You're planning a long trip through the desert and there was a corner case where the programmer (and QA) didn't account for, say temp. or humidity or something, and you got stuck in the desert with no easy way to revert to the working firmware. Not fun.

Chrome instructs me to "Restart to Update" when I do update.

Yep, I think that wording is a little misleading but I suppose if you never restarted the Chromebook, you wouldn't get the update so I guess in that sense it's correct.

Because of Ubuntu, my Chromebook never actually restarts correctly. This doesn't interfere with updating according to this I guess right?

I usually shutdown/logoff any running chroots and exit any active crosh shell sessions before I hit the restart up arrow, that way I know I've left things in a 'clean' state. Leaving a chroot running when you hit the restart up arrow should not interfere with updating Chrome OS but it may leave your chroot in a bad way, and possibly Chrome OS too. Chrome OS isn't aware of any running chroots so it doesn't send any 'shutdown' signal to them - I don't believe.

Then again I may have just installed a Chrome update without remembering.

That happens to me too, I've shutdown my Chromebook without noticing the 'up arrow' and when I start up again I find I'm running a new version of Chrome OS.

@tedm,

Sorry, I don't follow, what is a controversial method of updating?

 But your car was running fine for months...

Okay, I see what you were alluding to earlier now.
Google's update method of Chrome OS has been around since it's inception and, I for one, welcome it and I think a lot of other Chromebook users/owners do too. I suppose it's controversial to some but I think they're in the minority.
We're getting a little off track here so I think we should stop. This philosophical discussion about Google's update scheme could be carried on ad nauseam I believe...

@DennisLfromGA
We should probably stop but I just have one more question. When I try to choose shut-down or restart from the Ubuntu menu, my computer never does anything but continue running. I used to be able to choose log-off and get back into Chrome. Now, however, logging off takes me to this same screen and I hard-restart to get back on. Is this not supposed to happen? Or is this bad for the chroot?

@etpof - Sorry, I wasn't meaning you should stop, you have a legitimate question/problem, I was referring to our rant about the Chrome OS update mechanasm.

When I try to choose shut-down or restart from the Ubuntu menu, my computer never does anything but continue running.

I don't think it's a good idea to choose shutdown or restart from the Ubuntu menu, logging off is sufficient to shut down the chroot - if it's working.

Now, however, logging off takes me to this same screen and I hard-restart to get back on.

If by the same screen you are referring to the 'Sleep Mode-ish Screen', then that's a real problem and we should try to find out why it's happening.
If you could show us the output of sudo edit-chroot -al, we can find out what release, platform, and xmethod you're using and get closer to a resolution.

Here's the copied text and screenshot:
name: trusty
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/trusty...
crouton: version 1-20150620222151~master:89796a99
release: utopic
architecture: amd64
xmethod: xorg
targets: xiwi,unity
host: version 7077.111.0 (Official Build) stable-channel falco
kernel: Linux localhost 3.8.11 #1 SMP Tue Jul 28 16:55:35 PDT 2015 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/trusty...

I am aware I am running Utopic, an unsupported version, and its a long story explaining why I upgraded. I would downgrade, but most recommend I don't. Hopefully I can still be helped? I'm running an HP Chromebook 11 if that is important and or helps. Thanks.

I don't think it's a good idea to choose shutdown or restart from the Ubuntu menu, logging off is sufficient to shut down the chroot - if it's working.

I'm aware of this, and I don't usually use these, however, when I first downloaded Ubuntu, I used to be able to shut-down and restart after granting access when it prompted me for my password. Now, however, it does nothing, if this at all is important.

If by the same screen you are referring to the 'Sleep Mode-ish Screen'...

Yes, I am. Should've been more specific. My bad. There may be a better term to use for this situation that I'm not sure of. The computer hums and continues to run, but there is no display on the screen.

@etpof,

host: version 7077.111.0 (Official Build) stable-channel falco

I'm running an HP Chromebook 11 if that is important and/or helps.

I think that's an HP Chromebook 14 (Falco) like one of mine.
I currently have three chroots installed with xiwi - mdwatson(precise), trusty, and wheezy and I haven't experienced anything like you're describing so maybe it's 'utopic' specific.

Do you have any screensavers running in your chroot?

Do you always run your chroot full-screen (-X xiwi-fullscreen) or have you tried running it in a tab (-X xiwi-tab) or window(-X xiwi) ?

In any of the three xiwi modes, I have always been able to get back to Chrome OS via the F5/switch window key. Is that not working for you?

host: version 7077.111.0 (Official Build) stable-channel

I'm running an HP Chromebook 11 if that is important and/or helps.

I think that's an HP Chromebook 14 (Falco) like one of mine.

Whoops! My bad. Forgot that was listed there and forgot my own Chromebook model as well! Anyway, I only have 1 chroot installed (as I only need 1).

Do you have any screensavers running in your chroot?

Do you always run your chroot full-screen (-X xiwi-fullscreen) or have you tried running it in a tab (-X xiwi-tab) or window(-X xiwi) ?

I never run screensaver a in my chroot, and I run it in fullscreen. I've tried the crouton extension to run it in a tab, (not xiwi-tab) but I never really liked it so I switched back. Is the extension similar to xiwi-tab?

In any of the three xiwi modes, I have always been able to get back to Chrome OS via the F5/switch window key. Is that not working for you?

This does not work for me, no. I have wondered why these shortcuts have been like this. Come to think of it however, when I was able to log-off, restart, and shut-down correctly from Ubuntu, this was with trusty. Maybe this is a utopic specific problem like you said? And maybe there is a safe way to downgrade back to trusty? Though I'm doubtful.


Do not fix on /usr/bin/perl, use plain perl instead.

Prefer invoking perl command from path, e.g. yocto sdk - called poky - would set PERL5LIB to own perl library path in case local perl is installed,  breaking /usr/bin/perl

Background for this was #1566, but if the only actual problem was with the yocto toolkit and File::Temp as used in c_check, I would prefer to fix only that rather than risk introducing new incompatibilities.

It is universally available in all POSIX systems, just Linux prepends it to many scripts ?in hope somebody does not complain about broken tools?

e.g
#!/usr/bin/env python3
has good chance to be fixed up by user, while
'#!/usr/bin/python3.6`
Makes them rely on well behaved root user.

Counter arguments are typically about security, e.g. https://pagure.io/packaging-committee/issue/700 and I am not sure if we could rely on "env" itself being universally available in /usr/bin. Again if it is just the File::Temp that is causing problems I would prefer to fix only that. Or does the yocto PERL5LIB setting cause other conflicts ?

It could be #!perl
I am sad issue report did not get feedback from OPs

Definitely not, that would only work if you had perl in the current directory. From your comment in #1566 I had assumed you had done a build with yocto, poky or whatever.

Yes, I installed a toolkit, once you install build tools like gcc and make, and re-run oe-init-build-env , /usr/bin/perl appears broken.
I dont know - sanitize script to permit web mode without parsing variables (perl -Tt)
Or re-execute with environment cleaned (env could help here too - we sort of need basic basic perl)


igure out what this is supposed to do n fix itx

                // todo: figure out what this is supposed to do n fix it

https://github.com/dgmjr/TRex/blob/ead96f51b1a27abb43d4e98a22a708b38610fcf5/Source/TRex.Metadata/Filters/TRexSchemaFilter.cs#L183

Dammit I'm sorry I don't know why these aren't being created on my own fork...


Add "sideEffects": false to package.json

I'm using this package as a dependency, and I want Webpack to be able to optimize it out automatically if it ends up being the case that I do not use it (or only use parts of it).
Skimming through the source code, I don't see any of the files running any code as the files are loaded, which should mean that the project qualifies as being side-effect free and can be optimized by Webpack.

Published in v2.11.1


Input class and my search class overlapping together, due to which input search bar appears first on clicking icon,

This issue has been assigned to you @DYNAMICMORTAL .  Make sure you mention the issue number in the description of your PR.

Hey @rohitmultani  assign this issue to me


Move import of mpi4py to inside MPIPool initialization

We have a downstream user with a broken MPI installation, which crashes on importing mpi4py. She also can't use emcee in serial mode, because the mpi_pool module tries importing mpi4py.MPI.
This change fixes that (certainly rare) issue by moving the mpi4py import to inside the MPIPool initialization.

I don't think this is that rare. Another example is often times if you're on the login node of a cluster, anything touching MPI will error out, so you can't test emcee unless you get a compute node. This PR fixes the issue for me in the couple of places I've run into it, it'd be great to have it merged.

Thanks! Sorry about the delay.


Cosmos - Nested PartitionKey Support

Description
Read the property value from the request data using jsonpath to support nested partition keys.
Format for configuring nested partition key is
"address.zip" - fields separated by dot - similar to a .NET Bulk executor library
Example:
Configuration:
apiVersion: dapr.io/v1alpha1
kind: Component
metadata:
  name: send-event
spec:
  type: bindings.azure.cosmosdb
  metadata:
  - name: url
    value: URL
  - name: masterKey    # Azure Storage Account Name
    value: MASTER_KEY
  - name: database     # Azure Storage Account Key
    value: DB
  - name: collection  # Azure Storage Container Name
    value: CONTAINER
  - name: partitionKey
    value: "address.zip"

JSON:
{
    "data": {
    	"id": "1",
        "name": "test",
        "address": {
             "zip": "35220"
        }
    }
}

Issue reference
PR will close: #288
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation

Thanks!


Corrected a typo to fix issue #22

Before this fix, puppet-unattended_upgrades would create the invalid syntax
# /etc/apt/apt.conf.d/50unattended-upgrades
Unattended-Upgrade::Allowed-Origins {
        "${distro_id} {$distro_codename}-security";
        "${distro_id} {$distro_codename}-updates";
};

This fix corrects the invalid {$distro_codename} to valid ${distro_codename}:
# /etc/apt/apt.conf.d/50unattended-upgrades
Unattended-Upgrade::Allowed-Origins {
        "${distro_id} ${distro_codename}-security";
        "${distro_id} ${distro_codename}-updates";
};

@leoarnold do you think you could add a spec test to check for this kind of thing?

@igalic I could, but @cpick already did, so I'd recommend merging PR #14 instead.

Yes, @leoarnold is right this is a dupe of #14
Both of our patches fix the problem in exactly the same way, but #14 includes a spec test.

closing in favour of #14
thanks everyone!


Error Installing

Please fix this issue.

This has been fixed in 3d42b6e64be12fbec484c8afe8ded3e28e12d7dc and will be available in v0.18.


[Feature Request] Filter Packages by Supported Platform

When searching for a NuGet packages it would be ideal to have a way of filtering on the supported platforms.
Rationale
It is extremely difficult to know what packages out there are compatible with your project or with your PCL that you have created. You don't get an indication of error until after you try to install the package. NuGet already knows what the package supports via the nuspec and should allow filtering down to a specific target.
Single Project
For instance I am in a Xamarin.Android project, I would love it if by default it showed me everything, but I could filter down to packages that include Xamarin.Android as their target. This could be either a PCL or just a Xamarin.Android library.
PCL Project
For portable class libraries it would then narrow down to packages that target PCL that fit the supported profile of the PCL I currently and searching for.
Solution Level
When dealing with the solution level NuGet installation it is a bit trickier as it would be opt-in to filter down the packages. Ideally there is a drop down of what projects you want to install into (all by default) that you could narrow down, and then NuGet Gallery would narrow down what is visible based on what projects are selected and what is compatible to install in all.

:+1:

@yishaigalatzer not sure if client or server

The issue is that it is not known based on the nuspec only, to know 100% you need the whole package and to run the install script (in a packages.config scenario). Furthermore we can't put the knowledge in the server directly because it means new projects don't just work and people using privates servers have to upgrade frequently as well as server authors having to keep up.
We have ideas in this area to approximate based on package folders, and the server will need to send this information down.
We can keep this issue around to track the server side work.
We still got to figure out the client story.
It gets even more interesting when new versions packages drop older frameworks. For example webapi 2 dropped support for net40, so in theory webapi will still be applicable for a net40 project but only with 1.x versions. Solving this problem makes things more complicated on two levels

You can't filter these packages out or mark them as not matching
Need to design a way to move this data around (perhaps by ranges of versions)

Could we at least list the platforms a package is made for, on the version you're looking at? e.g. "net40", "dotnet52". One of the frustrating pieces with using a package on multiple platforms is I have to download the package and unzip it to determine what it supports. I'd love to see something as simple as list of folders in /lib on nuget.org.
Related is the annoyance of the .zip download link not being a .zip, which makes the tooling another step. I'll search for issues on that in a moment and create if it doesn't exist.

We have a similar idea being discussed, it will require client changes as well.


Impossible to save on Mac

Hello,
Great work for we gamers :)
But I'm facing with an issue on my MAC (Mojave) when saving or importing a skin I'm getting this error:
Impossible to convert  ".skincomposer/temp" into alias type (-1700)
Any idea about this issue ?

I might have seen that before using the .app bundle version of the tool. Can you try downloading the skin-composer-mac.jar version and run with the instructions here: https://github.com/raeleus/skin-composer/wiki/Getting-Started-With-Mac#running-skin-composer-from-jar-file ? Thanks and sorry for the trouble. I might just drop the .app bundle version if that proves to be the problem.

I tested it, and the JAR version seems to work fine. Please use that going forward. Thank you.


Issue adding fastembed as dependency in poetry due to python requirement

Installing the current project: infinity_emb (0.0.2)
infinity_emb $poetry lock --no-update
Resolving dependencies... (0.1s)

The current project's Python requirement (>=3.10,<4.0) is not compatible with some of the required packages Python requirement:
  - fastembed requires Python >=3.8.0,<3.12, so it will not be satisfied for Python >=3.12,<4.0

[tool.poetry.dependencies]
python = ">=3.10,<4.0"
fastapi = "^0.103.2"
fastembed = {version = "^0.0.5", optional=true}

ONNX Runtime does not support 3.12 and breaks for that — hence the lock for less than 3.12. We'll support 3.12 when ONNX Runtime does.

Yes, that sounds right. ONNX Runtime does not support 3.12, so Poetry does flag it as broken. I believe that's the desired behaviour for most Poetry use cases.

onnx issue for the reference https://github.com/microsoft/onnxruntime/issues/18793

The ONNX issue that was blocking support for Python version 3.12 has been now resolved: https://github.com/microsoft/onnxruntime/pull/18814

With #111 merged this issue seems to be ready to be closed as completed.
Thank you for your work @NirantK !


Fix out of bounds array access in lexer

If the type char is signed, then in the function yylex_GetFloatMaskAndFloatLen(), *s can have a negative value and be converted to a negative int32_t which is then used as an array index. It should be converted to uint8_t instead to ensure that the value is in the bounds of the tFloatingFirstChar, tFloatingSecondChar, and tFloatingChars arrays.

https://travis-ci.org/rednex/rgbds/builds/544384725

I think that all the information you've written in the PR should be in the commit message as well.

I've added a commit to fix #367 because it's closely related and it required modifying a test I added in this PR.

I've fixed #326 as well now. I put it here because I realized that I would have to move the code to skip the UTF-8 BOM in order to properly detect a * on the first line.

After merging another PR there are conflicts on this one. Could you fix them?

I've fixed the conflicts.


[VS Code] Joining collaboration session: Request DebuggerHostService.getLaunchConfigurations failed unexpectedly without providing any details.

Error:
Joining collaboration session: Request DebuggerHostService.getLaunchConfigurations failed unexpectedly without providing any details.
Steps to Reproduce:
1.
2.

Version Data

extensionName
VSLS

extensionVersion
1.0.471

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.35.1

platformName
MacOS

platformVersion
18.6.0

Duplicate of #2411


Added missing dependencies, upgraded SimpleFOC version

Closes #33, adds Wire, SPI, and Preferences to the library dependencies in library.json.
Upgrades the SimpleFOC version to 2.3.2, which addresses ADC initialization bugs for the ESP32S3, which has blocked progress on current sensing support.

Looks like we also need to add Arduino JSON to the dependencies, driver does not compile without this either.

Dependencies appear to be installed correctly now.


Canvas gets tainted when drawn with an image using the custom scheme

Details:
We are migrating from UIWebview to wkWebview for iOS, we got in to problem where Canvas gets tainted when loaded with an image using the custom scheme
Problem
For iOS, we are using setURLSchemeHandler to load the resources from Library folder. Resources (images, videos, pdfs etc) are loaded fine after doing necessary additions to meta tag to allow the new custom scheme being used. How ever canvas is getting tainted when we draw the image using the custom scheme and throws security CORS error when tried to get the canvas data using toDataURL.
I see error message "SecurityError: The operation is insecure." on call to toDataURL in console.
I know wkwebview is having issues( or strict) with CORS but as per document (https://cordova.apache.org/docs/en/latest/guide/appdev/whitelist/) this issue should not occur even for Canvas with access origin='*' setting and custom scheme file loading, when its working fine for other elements.
Note: toDataURL works fine when Image element being drawn is set with base64 data instead of url with custom scheme
Environment:
iPad pro with iOS 12.1.3
cordova-plugin-wkwebview-engine : 1.1.4
Is this a bug in the cordova-plugin-wkwebview-engine?

We also tried with setting the crossorigin attribute to "anonymous" for the image being drawn on to canvas but this resulted in the below error even with access origin='*':
[Error] Origin null is not allowed by Access-Control-Allow-Origin.
[Error] Failed to load resource: Origin null is not allowed by Access-Control-Allow-Origin.

Same error, I have an image with this src: "file:///var/containers/Bundle/Application/27131FF1-0862-4240-851F-FBDFFDDC47D4/HelloCordova.app/www/49dfd8c7f5effbcd57afcb1c060d6322.png"
When I run this code:
const image = document.getElementById("image");
var canvas = document.getElementById("canvas");
var ctx = canvas.getContext("2d");
ctx.drawImage(image, 0, 0);

I get this error: SecurityError: The operation is insecure. on call to toDataURL.
Using UIWebview this code works.

This plugin doesn't use custom scheme handlers, so I don't think the issue is caused by it.


Small typo in module 5

Module 5 line 28 or so (directly under the ## Load packages section header) has one instance of mosaic typo'd as mosiac.

Will fix. Thanks!


pub error messages should include more context

We got a bug report with an error:
    Running pub update fails on web-components with the following message:
    Package 'js' is depended on from both sources 'hosted' and 'git'.
I suspect the underlying bug was already fixed. But, it would be nice if the error message had printed more context. For example, which pubspec.yaml referred to it as "git" and which was "hosted"? We need something like a "stack trace" of dependencies to help users track down this kind of bug.
https://github.com/dart-lang/dart-web-components/issues/86

This issue has been moved to dart-lang/pub#216.


What should I pay attention to training my own data?

Hi,
I want to train my own training. There are only 5 categories. In order to make the model better, what do I need to do or what should I pay attention to?ex,How should the config param be built?
Thank you for your reply.

it depends on your data. you may refer to the original paper and the tf implementation for some insights.


Java build problem: javac plugin? javac command line?

This happens when trying to build the Pantheon project (https://github.com/PegaSysEng/pantheon) with Infer.
The error message complains about the -XepExcludedPaths flag (used by the ErrorProne gradle plugin), but the shown command line doesn't work anyway because -sourcepath receives no files – though maybe that command line is only informative and not actually used internally?
Infer version v0.16.0-d37374dd8 (built from source)
macOS Mojave 10.14.5, java version "1.8.0_51"
Command run: infer run -- ./gradlew build -x test
Command output at https://gist.github.com/hmijail/12a3baa0e944a531b635c50c82cb36a2

Thanks for the report. When you built infer were you using Java8 as well? You might want to run ./build-infer.sh again to make sure. Currently infer is built differently for Java 8 and Java 11.
Note that the gradle integration is very fragile and just parses the verbose output of gradle looking for javac commands. It often parses these wrong since they lack escaping in the gradle output.

(After a long time, I managed to get back to this. My original description was wrong, but I'm writing here since it's the same problem; let me know whether to close this and open a new issue.)
Infer version v1.0.0-671d9fa29
javac 11.0.5
When Infer gathers the javac CLI flags into its javac_args.tmp.xxxxx files, it does so by putting each new argument into a new line. But this causes the invokation to fail in the case of -Xplugin flags, since those need to be on the same line, as shown in the ErrorProne plugin instructions.
So, for example, this works:
javac ... \
'-Xplugin:ErrorProne  -Xep:EqualsGetClass:OFF' \
myfile.java

But this fails with error: invalid flag: -Xep:EqualsGetClass:OFF
javac ... \
-Xplugin:ErrorProne \
-Xep:EqualsGetClass:OFF \
myfile.java

Closing because this same issue is better explained at  #1404.


Creating UISideMenuNavigationController via Storyboard if installed using Carthage not working

New Issue Checklist

I have read the guidelines for contributing and I understand:

[x] My issue is happening in the latest version of SideMenu.
[x] My issue was not solved in the README.
[x] My issue can not be answered on stackoverflow.com.
[x] My issue is not a request for new functionality that I am unwilling to build and contribute with a pull request.
[ ] My issue is reproducible in the demo project.

Xcode 9.1, macOS High Sierra 10.12.6, Carthage 0.26.2
Issue Description
The identity inspector doesn't recognize UISideMenuNavigationController, nor SideMenu as an applicable module if I'm using Carthage. Not reproducible with Cocoapods.

@michael-mckenna Carthage support was added awhile back by another contributor. Admittedly, I do not use Carthage. Do you have any idea what the solution is?

Hey thanks for the quick response. Sorry, I'm unfamiliar with the inner-workings of Carthage so I do not know.

Hey @soheilbm, it's been a while since this was added, but have you had any issues with SideMenu + Carthage?
Or anyone else, for that matter?
I'll likely opt for removing the Carthage support from the README rather than fixing it myself (no time), unless someone else is willing to figure out what's going on.

Hey @jonkykong I notice that the class and module don't have autofills in the interface builder when installing with Carthage, but typing them in works without issue. I can also subclass, though again the autofill doesn't recognize either as a valid UINavigationController subclass. Loosely related may be that I also don't see the IBInspectables correctly in the attributes inspector, but if I make a subclass and override the leftSide or menuWidth properties with getter/setters the delegate to super, then it works.

@jsnelling Is your project in Swift 3? SideMenu is Swift 4. I suspect that mixed Swift versions may be causing some of these problems.

@michael-mckenna does typing in the class names manually work for you?

@jonkykong I'm getting the same result as jsnelling in that autocomplete in the text field doesn't work and the IBInsectables aren't showing in the attributes inspector. Using Swift 4

@michael-mckenna so you are able to get it to work by entering them manually?
@jsnelling @michael-mckenna try cleaning your projects and deleting your derived data to see if that gets the properties to appear.
I would also suggest trying Cocoapods instead to see if that works (if it doesn't, then it sounds like something is up with the project rather than Carthage).
Unfortunately, a cursory search on the web shows a bunch of possible causes without a clear cause... since this is a rarely reported issue I'm not going to investigate a solution to this (sorry). Sticking to the rule of 80/20, I'm trying to only invest my time on problems that face more than 20% of people.

@jonkykong - I'm Swift 4.0 + iOS 11 on XCode 9.1. It repros with a brand new project. Create blank application for iPhone. Add sidemenu to Cartfile and update. Add to linked libraries. Drag a new Navigation Controller onto the storyboard and try typing class and module. Delete/rebuild has no effect.
Since it's still fully functional, I'm not worried :-) Mostly I was just trying to add more context to this task since - at least for me - the title seems inaccurate. Creating via storyboard is working, it doesn't doesn't autocomplete. Storyboard configuration via IBInspectable doesn't work but a shallow local subclass with overrides is a totally reasonable workaround.
Very nice project by the way! Thank you!

+1 to jsnelling comment on Nov 3
Happens to me too, Swift 4.0 + iOS 11 on XCode 9.1.

I also have this issue with Carthage... I cannot select the custom class as UISideMenuNavigationController. Even though I manually type the custom class, the properties do not appear in the Attributes inspector.
It's sad since that project really pleased me and the explanations were quite clear to set it up...

@damsc69 if you have the time, I'm looking to the community to troubleshoot the issue and produce a fix for everyone's benefit. Feel free to create a pull request with a solution, if you find it.

+1 to jsnelling's comment on Nov 3
Happens to me too, Swift 4.0 + iOS 11 on XCode 9.3
I see your looking for a community fix, I'll try and see what I can come up with.

I have the same issue with Cocoapods as well. Xcode 9.4 (9F1027a).

I am having the same issue with carthage but not with cocoapods. @jsnelling can you share the subclass for a workaround as of now.

@mastermind247 sure! So if you have a class like UISideMenuNavigationController that you'd normally be able to configure with interface builder and can't, I found (if I remember correctly) that creating a local class in my project MyUISideMenuNavigationController that extends from UISideMenuNavigationController but doesn't override anything, would behave correctly and allow me to select it as the type and configure in IB. Hope that helps.

I am able to configure everything in IB, but as others mentioned, @IBInspectable's don't appear under the nice UI in IB. However, you can still set them using IB KVO like so:

Admittedly this is only really needed for leftSide property which seems to be set to right by default which i found unusual compared to how most apps are on the market.
Nevertheless, everything works perfectly. Amazing work putting this together @jonkykong

Update: you should now be using SideMenuNavigationController instead of UISideMenuNavigationController.


APPSRE-10117 soak days

:warning: This is still WIP
Add soak_days for auto-promotions
Taking over https://github.com/app-sre/qontract-schemas/pull/519 to have my own branch during development

Seems pretty similar to https://github.com/app-sre/qontract-schemas/pull/519

Taking over https://github.com/app-sre/qontract-schemas/pull/519 to have my own branch during development

merged #519


Improve websocket timeouts and node connection open/close logging

Helps diagnose node connectivity issues like in #1948
The default 2s keepalive ping timeout on the agent websocket might also be a bit optimistic, and it leads to to grid instability if the agents start bouncing.

Bump default websocket ping timeouts to 5s, configurable via KONTENA_WEBSOCKET_TIMEOUT

Log error with timeout delay if timeout exceeded

Log warning with ping delay if threshold (timeout / 2) exceeded

Log debug with ping delay

Server

Log warnings when marking stale nodes as disconnected

Improve websocket timeout, close and node unplug handling and logging

Log different warnings when disconnecting a client, vs when a client disconnects

Agent

Fix agent LogWorker to also stop log streaming when the server closes the connection

TODO

Better server websocket specs

Still working on better websocket specs, but otherwise reviewable.

Having some trouble with my WIP server WebsocketBackend specs, needs more and better debug logging to figure out what's going wrong.


Update admin-mobile-app.md

Added information about changing language in the admin app

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/3575

@JohanFreelancer9


Fix issue with unloading double wrapped modules

Resolves #1485, but note that some additional solutions are mentioned in thet issue.
This checks that when unloading a PEFT model, if the ModulesToSaveWrapper contains a tuner module, it is correctly unloaded. The unloaded model should not have PEFT layers at the end.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


..

https://github.com/0xPolygon/cdk-sequence-sender/blob/develop/test%2Fconfig%2Flocal.node.config.toml

Hi, @Carlo0704 ! Please provide a clear description of the issue.


Add new tab bug

Awesome library!
I did find a small issue though, I'd like to be able to put the + button for new tab items on the left rather than the right, doing so causes a number of issues if you click the X button on an active tab item.
https://jsfiddle.net/arxqm737/1/

https://github.com/zfedoran/PhotonKit-Components/commit/39fb29ce26e12953a25663a7a27ca0d4e4d80200

Also fixed in the new version of tab group. https://jsfiddle.net/Loo3pyuo/


Additional content

I'm using gatsbyjs which by default brings Helmet installed. I would like to add some more content to my , in this case just a comment with a ref id. Would it be possible without any property of helmet tag for additional content? I'm missing something like a headAdditional property that takes a string and put it in the head exactly as it was written

@migueloop Check out https://www.npmjs.com/package/react-helmet-with-visor newly created fork of the react-helmet. And injects arbitrary html strings through the HelmetsOpenedVisor element.


feat: Add Hooks Extension Framework tooling

Description:
This PR adds a new module for the tools needed by the Hooks Extension Framework. This project is described in more depth in the OEP-50.
This tooling will allow us to extend the platform by adding triggers in specific places, those triggers will be able to execute a list of actions/filters previously configured.
For that to happen, we are introducing two functions called triggers_{operation} where operation can be action or filter. When called, these functions will execute a Pipeline with the list of functions specified by the configuration. This pipeline is defined in ADR Hooks tooling: pipeline
Now, this configuration as indicated in ADR: Configuration for the Hook Extension Framework is defined using Django settings and looks like this (extended format):
HOOKS_EXTENSION_CONFIG = {
      "1st_trigger_name": {
               "pipeline": [
                     "somelibrary.filters.1st_function",
                     "other_library.filters.2nd_function"
                     ],
               "async": False
     },
    "2nd_trigger_name": {
               "pipeline": [
                     "some_library.actions.1st_function",
                     "other_library.actions.2nd_function"
                     ],
               "async": True
      }
}

Pipeline is the list of functions that the pipeline runner will execute, and async indicates how the pipeline is going to run, asynchronous or synchronous. Please be aware that this PR does not add the asynchronous feature, only the synchronous, that feature is meant to be added in another PR to reduce complexity.
JIRA:
XXX-XXXX
Dependencies:
List dependencies on other outstanding PRs, issues, etc.
Merge deadline:
None
Installation instructions:

Run make lms-shell
Inside the shell run pip install -e git+https://github.com/eduNEXT/edx-django-utils.git@MJG/hooks_pipeline_tool#egg=edx_django_utils

Testing instructions:

Run make lms-shell
Inside the shell run pip install -e https://github.com/eduNEXT/openedx-basic-hooks.git, you can also use your own test library. Here are defined the functions that will be executed by a trigger.
Here's a quick and dirty way to create and use triggers on edx-platform. We defined three triggers and the configuration needed to use this framework. PENDING: add link to the PR against edx-platform that includes the first implementation of a trigger.
(Optional) If you're using our demo and sample openedx-hooks-library, you have two types of triggers on the platform ready to use, two trigger_action and one trigger_filter. What you'll have to do is execute the logic that the hook extended, for example:

Go to register
Enter data for registration and in the field Year of birth add 2020.
Then, an error will be displayed caused by the HookException raised by our filter function defined in https://github.com/eduNEXT/openedx-basic-hooks/blob/5f2cf73a06d34506a82f40d6e33be2b41019ef66/openedx_basic_hooks/filters/pre_register.py#L13

Reviewers:

[ ] tag reviewer

Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[ ] Version bumped
[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed

Post merge:

[ ] Create a tag
[ ] Check new version is pushed to PyPi after tag-triggered build is
finished.
[ ] Delete working branch (if not needed anymore)

Author concerns:

[ ] Is this the best place for this feature? This concern is raised when reading:

Note that some utilities may warrant their own repository. A judgement call needs to be made as to whether code properly belongs here or not. Please review with the Architecture Team if you have any questions.

[ ] Will there be a problem with adding celery? Adding it will cause other applications to crash?

Thanks for the pull request, @mariajgrimaldi! I've created OSPR-5688 to keep track of it in JIRA, where we prioritize reviews. Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
This is currently a draft pull request. When it is ready for our review and all tests are green, click "Ready for Review", or remove "WIP" from the title, as appropriate.

Hey! I made a PR showing how it'll look adding the async feature -if we decide that this repository is the best place for Hooks Extension Framework-:
https://github.com/edx/edx-django-utils/pull/111

Closing PR following recommendations made on this post: https://discuss.openedx.org/t/configuration-for-the-hooks-extension-framework/4527
From now on, this PR is going to be found here: https://github.com/eduNEXT/openedx-filters


[BUG] Training step breaks because of type assumptions on class label

Describe the bug
When running train, if the dataset object has previously encoded class labels, it will break.
This is because it tries to coerce this to a class and fails.
To Reproduce
Run train with any data
Expected behavior
Training to be conducted as normal
Suggested fix
Added a condition to skip type change if it is correct
Screenshots
NA

Fixed in 2.5.0


Remove a foreign definition

Fixes #31722
ping @rbuckton as the assignee

Fixed by Fixed by #35998.


docs: sidenav/subnav active page

Closes #1306
What I did

simplify navigation HTML, CSS, JS
de-sassify and de-bem-ify nav css
apply active style to pages with a subnav

Testing Instructions

don't forget mobile breakpoints

@bennypowers side nav for Patterns is not visible: https://deploy-preview-1307--red-hat-design-system.netlify.app/patterns/

Thanks @nikkimk PTAL
wrt tokens search: :shrug:


Conditional where clauses for PhalconAdapter

Hello friend, what if aliases and PhalconAdapter are used?
`
$builder = $this->modelsManager->createBuilder()
->from(['o' => Orders::class])
->columns([
    'o.id as id',
    'o.sticker as sticker',
    'o.status as status',
    'item.id as item_id',
    'item.name as item_name',
    'c.id as client_id',
    'c.phone as client_phone',
    'section.title as section_title',
    'cat.title as cat_title',
    'brand.title as brand_title',
])
->leftJoin(Items::class, 'item.barcode = o.barcode', 'item')
->leftJoin(Sections::class, 'section.id = item.section_id', 'section')
->leftJoin(Cats::class, 'cat.id = item.cat_id', 'cat')
->leftJoin(Brands::class, 'brand.id = item.brand_id', 'brand')
->leftJoin(Clients::class, 'c.id = o.client_id', 'c')
->getQuery()
->getSql();

$dt = new Datatables(new PhalconAdapter($this->di, 'db'));
$dt->query($builder['sql']);

$dt->filter('status', function ($status) {
    $this->log(123); //this code is not executed at all <<<<<<<<<<<<<<<<<<<<<<<<<<<
    return 'o.status = 1';
});

echo $dt->generate()->toJson();`

That's because $this refers to FilterHelper class. If I have find free time, I will change it to normal behavior.


Upgrade from .NET6.0 to .NET7.0

Updated .csproj version to net7.0
Upgraded Microsoft.AspNetCore.SpaProxy from 6.0.15 to 7.0.13

Closing because .NET 8.0 will be released very soon(Probably at .NET Conf)!
Source: https://devblogs.microsoft.com/dotnet/announcing-dotnet-8-rc2/?WT.mc_id=dotnet-35129-website


Add step in migration docs related to RXT file update

@msm1992 , are these changes applicable to 3.1.0? If yes, please send a similar PR to the 3.1.0 and master braches.


Option to strip incoming Received headers

To activate the desired option, add strip_received_headers: true to the smtp_server section of your postal.yml config file... then restart Postal.

@catphish Do these changes look ok to you?

Yes, this looks simple but effective.


:wastebasket: Removal of deprecated, unused functions.

Changes

:wastebasket: Removal of deprecated, unused functions.

/kind cleanup
Follow up on #294

/cc https://github.com/orgs/knative-sandbox/teams/eventing-writers
/cc https://github.com/orgs/knative-sandbox/teams/productivity-writers
/cc @tayarani

/cc @knative-sandbox/eventing-writers
/cc @knative-sandbox/productivity-writers

/cc @coryrc
/cc @upodroid
/cc @psschwei
/cc @chaodaiG
/cc @kvmware

The failing downstream tests are because of #294 and are covered by the following PRs:

knative-sandbox/eventing-kafka-broker#2413
knative-sandbox/eventing-natss#348
knative-sandbox/eventing-rabbitmq#867
knative-sandbox/kn-plugin-event#203
https://github.com/knative-sandbox/discovery/pull/280

Let's proceed. Merging it will not influence the downstream projects further.
/unhold


Minor branding issue

In Settings > Privacy and security > Safe Browsing, there are a number of claims that Safe Browsing (and other services) are provided by Geometry. I'm fairly sure this is a Google service. You should be clear and accurate about to whom the users' data is being sent (Geometry, Google or both?).
In any event, this part is definitely wrong: "...other Geometry services, like Gmail..." If Google catches you (accidentally) claiming to own Gmail, they won't be happy.
Just a heads up.

It's pretty obvious (even for Google) that this is a translation issue and that it comes from the auto-replacement of Google by Geometry (in order, exactly for this, to not create confusion).
Obviously Safe Browsing is Google service, and this SafeBrowsing screen is so deep in the UI that it was forgotten, but you know it :)
It's fixed.

As of 95.0.4638.75 it's back.
If you run the same Find/Replace for every release, it's time to change the habit.


KNOX-2397 knox failed to start with error "java.lang.NoSuchMethodError: org.eclipse.persistence.internal.oxm.mappings.Field.setNestedArray(Z)V"

(It is very important that you created an Apache Knox JIRA for this change and that the PR title/commit message includes the Apache Knox JIRA ID!)
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. For instance: running automated unit/integration tests, manual tests. Please write down your test steps as detailed as possible)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)
Please review Knox Contributing Process before opening a pull request.

[ERROR] testConcurrentGetToken(org.apache.knox.gateway.service.knoxtoken.TokenServiceResourceTest)  Time elapsed: 0.025 s  <<< FAILURE! org.junit.ComparisonFailure: The token expirations should be the same. expected:<159350284[8]000> but was:<159350284[9]000> 	at org.apache.knox.gateway.service.knoxtoken.TokenServiceResourceTest.testConcurrentGetToken(TokenServiceResourceTest.java:209
The failed test seems not be related this pr, and it can be successfully executed in my local knox code.


New state preps multimode

Description of the Change:
This fully implements the single and multi mode state preparations in the fock and tf backends as specified in #16 and adds various corresponding unit tests tests. The implementation is a little bit more general than would have been necessary, as it allows the target mode list to be out of order and then correctly reorders the modes.
TODO

The multimode functions are not yet included in the base.py
The API needs some thought. It might make sense to remove the single mode versions of the state preparation methods (wich already now simply call through to the multi mode variants) and rename the multimode variants
Benefits:
Allows the preparation of arbitrary single and multi mode states or arbitrary modes
The order of the specified modes is taken into account when interpreting the input
The state() methods that returns the state can now also return correctly reordered states of a list of modes that needs not be sorted.
Possible Drawbacks:
The newly implemented code does not always use the tensor(), insert(), and similar methods in the corresponding ops.py files, but rather calls numpy/tensor flow directly, as these were written in a way that was hard to generalize.
Related GitHub Issues:
#16

The last few commits implement the API changes we discussed earlier. This resolves the two open TODO points mentioned in the initial pull request.

If all tests pass, I will, as a final step, run local test with tensorflow==1.3 and then give green light for merging if all goes well.

Turns out that the way I am now using tf.tensordot() in /tfbackend/ops.py in line 744, in replace_modes, namely:
revised_modes = tf.tensordot(reduced_state, replacement, axes=0)
raises a
ValueError: 'axes' must be at least 1.
with tensorflow<=1.4. From 1.5 onwards all works.
Would a dependency on tensorflow>=1.5 be acceptable, or shall I look for a workaround? Alternatively one can probably add a fake dimension and then tensordot that away conditional on tensorflow<=1.4.

Hmm... It would definitely be preferable to find a workaround that doesn't require checking which version of Tensorflow is installed

As python packages have no numerical version numbers, comparing version is indeed a not a good idea (tough there are solutions: https://stackoverflow.com/questions/20180543/how-to-check-version-of-python-modules).
It turns out that one can indeed replace
            revised_modes = tf.tensordot(reduced_state, replacement, axes=0)
by
            revised_modes = tf.tensordot(tf.expand_dims(reduced_state, 0), tf.expand_dims(replacement, 0), axes=[[0],[0]])
and the latter works with tensorflow>=1.3.
I am sure this hack of adding extra dimensions it not exactly efficient, so I put a todo so that we can remember to remove it once tensorflow>=1.6 becomes a dependency.


fix: login commnad with --api-token #96

A lot of changes have been applied in this PR. Can you also take a look at, please? @guoard
And I'm cool with the idea of separating API calls to a new Liara class. We can extract this class to it's own package in the future and release it as @liara/client so everyone can interact with our REST API easily.

@vashian Shall you please rebase master branch and resolve conflict first?


Inconsistent temperature

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

semantic bug in chatreadretrieveread.py:

in line 134 temperature is set to 0 then in 231 if there is no override it is set to 0.7. this makes the messages after the first one inconsistent
Any log messages given by the failure

Expected/desired behavior

temperature should remain 0 if no override.

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

ChatReadRetrieveRead makes 2 calls to ChatGPT for two different purposes. The first call turns the user question into a search query. The second call actually answers the question. We use 0 for the first call as we don't desire creativity, but we use 0.7 for the second so that the answers are more creative. You may change 0.7 if you'd like less variability in the answers.


Prevent gcc 4.7 from thinking a static func shadows an arg

Fixes #11867

I recommend closing this pull request without review. I know that it is a minimal change, but if the only purpose is to support a compiler that we no longer support, let us not do this.


Implement the tile intrinsics

Here
There doesn't seem to have GCC intrinsics for those: rather, they are implemented as inline assembly.

@antoyo assign this to me

Done. Don't hesitate if you have any questions.


Migrating project to use groups in allowlist

This PR migrates the project to use groups of IPs in their allowlist.
By referring to groups to IP addresses, we can centralize the definition of groups of ip addresses.
If these lists require changing in the future, we can change the definition once and future deploys across all services will automatically include these new IPs.
1 allowlist(s) have been detected that can be migrated.
Allowlist: helm_deploy/create-and-vary-a-licence/values.yaml
New Groups
The effect of applying this PR is as follows:

The following groups will be applied: internal,prisons,private_prisons
The size of the allowlist defined in this file will change: 45 => 2 (43 removed)

Added IPs
The new Group membership will result in the following IPs being added to your allowlist by applying this PR:

sodexo1 (<IP_ADDRESS>/32)

sodexo2 (<IP_ADDRESS>/32)

sodexo3 (<IP_ADDRESS>/32)

sodexo-bronzefield (<IP_ADDRESS>/32)

Removed IPs
The following IPs have been identified as unnecessary and will be removed by applying this PR:

quantum (<IP_ADDRESS>/32)

quantum_alt (<IP_ADDRESS>/32)

quantum3 (<IP_ADDRESS>/28)

dxc_webproxy1 (<IP_ADDRESS>/32)

dxc_webproxy2 (<IP_ADDRESS>/32)

dxc_webproxy3 (<IP_ADDRESS>/32)

dxc_webprox23 (<IP_ADDRESS>/32)

health-kick (<IP_ADDRESS>/32)

Makes a big difference! LGTM

I think the other two can probably go too! - need to check kibana


Perception: Add CIPV

Please review this PR again. I addressed all changes requested except a unit test. I will add unit test in next PR.

@jinghaomiao, Fixed all requested items. Please review again.

Waiting for a review, I added unit test of cipv. Thank you for reviewing it.

LGTM. Could you fix the failed CI?

My computer's lint couldn't detect it but github detected it. I fixed it.

@techoe could you please fix the conflicts?


Refactor: Non-Interactive Setup Process

To prepare the setup over Webbrowser in future versions, its needed to re-arrange the setup flow in that way that all setup-data (names, password, seeds, ) are gathered up-front and then are written to a setup-file and the rest of the setup runs non-interactive.
Things to keep in mind:

Once the LND wallet was initilased to recover from SEED there should be no restart - otherwise the LND will forget the recovery window and not find any on-chain funds
This should also be combined with a TOR as default approach

Take care that also a re-factory fixes:
Make BTCPayServer install on update/recovery #1198

This should be one of the firsts todo for a v1.7.1 .. because its needed as a prepartion for the v1.8 WebUI setup ans also before a possible c-lightning integration to keep merge conflicts low.
It sould be started after project planning for v1.8 WebUI is done.

A more detailed description of the non-interactive setup process can be found here: https://github.com/rootzoll/raspiblitz/issues/43#issuecomment-828840191

Would like to add some relevant notes:

Fast update option - inactivate services before switching off for an SDcard update https://github.com/rootzoll/raspiblitz/issues/2250
Minimize / confirm restarts after installing services: https://github.com/rootzoll/raspiblitz/issues/2249

Also we can think ahead to be able to implement full disk encryption.
In that case (this is how Nodl works) only the boot partition is unencrypted and only does offer a minimal login page to ask for the encryption password.
Once that is given all other passwords could be used automatically (LND autounlock can be applied by default).
This would leave the user with only one password to manage and use after every restart ideally in a web interface (could be over Tor / LAN HTTPS/ LAN HTTP (fallback) or even utilise the touchscreen or a some 2FA (eg yubikey) optionally in the future).
All other passwords (for the LN wallet, bitcoinRPC, SSH login) can be randomized during the setup and shown securely over an encrypted tunnel like Tor / HTTPS / ssh.

The disk encryption could be part of the boot from ssd feature #1235 - but that I see pushed back to v1.9 ... v1.8 is already a big package, we cannot do all at once.
For this issue about the "Fast update option" I dont think we need a special feature. Am I correct that your your goal is to keep Lightning downtimes minimal to not decrease QualityOfService to network? If yes then I think with regard to this issue this can be done thru:
In the non-interactive setup/recovery process make sure that the Bitcoin & Lightning services are started as early as possible to minimize downtime.
Additional apps can install after that - this may slow down the update/recovery process but reducing downtime to the network is getting a bigger priority now that the network matures.

Yes, agree that building the extra services after LND is started and unlocked is the best solution.
Re. the disk encryption I wasn't thinking of implementing it just yet, but it is good to keep in mind how we could make it easier to be used in the future.

This is a first draft how non-interactive provision process could be part of the future v1.8 setup:

Source File: https://docs.google.com/drawings/d/1iYYXrZTFqH5xMVxSXP7BSmf38vIj6GhXtJoVGFCHrQE/edit?usp=sharing

I know the words does not fit inside the box, but focus on the arrangements.
BlitzStructure.odf

@nyxnor sure you can see migration/setup dialogs as one part (that seither done by WebUI or SSH) but the detection if its a fresh/exiting/mograting blitz happens before the user interaction. But there is one point that the previous model maybe missed ... thats the I have a HDD from another fullnodeOS and I dont want to migrate - I just want to setup a raspiblitz and maybe just reuse the blockchain, etc.
So maybe I was missing a general step in the process, where after the Gather All System Info the user has to be asked what is TODO with that in detail. That would also mean that Update would not start automatically as its now. So that there always will be an interaction. Will ponder on that. I think for v1.7.1 I will keep as it is - but that can be then rearranged with the step to v1.8
But adding info on the Tor bridges can really happen so late in the process? The Provision Process would already has internet requests (getting binaries, etc).

So maybe I was missing a general step in the process, where after the Gather All System Info the user has to be asked what is TODO with that in detail. That would also mean that Update would not start automatically as its now. So that there always will be an interaction.

About missing general step, yeah, I understand this as a draft also, I missed some things too but that would add a lot of text and blocks for now.
The updates I imagine just clicking update from given githubUsername and that is it, no other interaction.

But adding info on the Tor bridges can really happen so late in the process? The Provision Process would already has internet requests (getting binaries, etc).

It can be moved to the first acccess to an interface (ssh, or webui).

I updated the draft .. as you can see the tor briges should be able to set in the first System-Ready loop:

Source File: https://docs.google.com/drawings/d/1iYYXrZTFqH5xMVxSXP7BSmf38vIj6GhXtJoVGFCHrQE/edit?usp=sharing

I final setup reboot was still needed ... but basically the Setup process is now ready for the WebUI future.


Claims without fee amount (have address/currency), get fails

The Issue
During the protobuf conversion, some amounts were either too small or invalid, so they were dropped. We should mark them as free or the max LBC amount so they can't be purchased (not sure which is better) , and let the claim owner update eventually. These are probably few and far between...
C:\Users\SoloDolo\Desktop>lbrynet get nsfw#6f7c7042e4f4ee9628acd7e7e6e406f81a21080c
{
  "error": "(1, 'the transaction was rejected by network rules.\\n\\n65: dust\\n[01000000015c245c8abf0fad2ea5cec1ec1643a76d3bc281dbe24b82487f7e627c89b5d638010000006a47304402204a28677f25d2c0707e979085290ed6fb5f9f7d1863a3864697f2f02f3bba6aaf0220389040b5882d31d0eabf0ccb2a39ce77faebb8e19d0bca361c489c6729bff6b201210327743de2c00ec69c68b5c49b3130c8798105b8b37f7985c7ac83c87471339314ffffffff0200000000000000001976a914d741a6e27ba64efe70b8659117fe4d2bf5ebfd8c88aca9fa7000000000001976a9147bf2d3f2c0a087b6f13e5af1a99f9ecad211ae0e88ac00000000]')"

System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

This is now fixed, the claims can be downloaded.


new data integrity checks

[ ] naming of OTU directories should be of the format: <similarity-threshold>-otus
[ ] expected sequences files should be named expected-sequences.fasta, and no other fasta files should be present in those directories

I agree with item 1 but have some "devil's advocate" questions regarding item 2.
In some ways, the source directory could be useful as a sort of "junk drawer" for the mock community, and contributors could include other information that don't it elsewhere. For example, a list of Genbank accession #s for whole genome sequences (which might not be appropriate in the "expected taxonomy" directories that are specific for reference databases that provide taxonomy information). Of course, we have control over this so the files would never be "junk", just a collection of useful files that do not fit in the other directories (which are more regulated).
Naming conventions in source could also have some flexibility. For example, expected-sequences.fasta can be rather vague — instead, full-length-16S-expected-sequences.fasta or V4-domain-expected-sequences.fasta could be more informative.
What do you think?

I think that all makes sense, I'm good with it.

What should we do for shotgun metagenome datasets? I think I support keeping the <similarity-threshold>-otus requirement across the board for simplicity's sake, and such datasets could be labeled 100-otus. But would it be better to enforce this rule only for marker-gene datasets, and use different rules for metagenome datasets?

I think your suggestions would work well.

Thanks! I will make that rule standard then when I update the integrity checks.


Tiny updates in CONTRIBUTE

Coverage remained the same at 97.81% when pulling 281cc3480bf7e8b523273ca2707c159a35af5c30 on rossant:tiny-updates-contribute into aeb3078e9a8bac80119299384b8fffe0405fe9dc on kwikteam:master.

Ready to merge @nippoo
Also: in future instances, if you need to make small changes to a PR of mine (or the other way around), one simple solution might be for me to give you commit access to my fork.


Add a method to NavigationState for adding routes below

Based on https://stackoverflow.com/questions/46259751/how-to-push-multiple-routes-with-flutter-navigator
Steps to Reproduce
Import package:flutter/scheduler.dart and set timeDilation = 100;
Push two routes one right after the other:
    Navigator.push(
      context,
      new MaterialPageRoute(
        builder: (BuildContext context) {
          return new Scaffold();
        },
      ),
    );
    Navigator.push(
      context,
      new MaterialPageRoute(
        builder: (BuildContext context) {
          return new Scaffold(
            appBar: new AppBar(
              title: new Text('42'),
            ),
          );
        },
      ),

You'll see the white blank route transitions faster than the route with the app bar, on iOS.

Instead we should provide a way to insert routes underneath the current one, once the animation completes. (ModalRoute provides an animation property that we might be able to use to delay the insertion until the initial route's transition is complete.)

Should be relatively straight-forward, basically a mix of how we do replace and push today.

While attempting to fix this in #14490 I've discovered that there is actually something less trivial than I though in order to properly support this feature.

OverlayState does not currently support inserting layers at arbitrary indexes.
in order to fully deal with precise timing with animations we need callbacks or events that can be listened to, when push/pop animations complete (e.g. a didPush void callback argument in the push() method);
It appears that calling route.didReplace(null) is not enough to actually insert a route properly, I still see some visual glitches when popping a route with a previously inserted route using this method and I also hit some failed asserts in some specific setups;
In order to fully deal with precise timing with animations we need callbacks or events that can be listened to, when push/pop animations complete (e.g. a didPush void callback argument in the push() method);
In general I'm not really convinced about the current API of the Navigator. I think that a declarative approach may be better. In other words, be able to provide/modify the stack of the desired routes by directly placing them where desired (a bit like what is already being done when building Widgets tree), and have the Navigator automatically deal with that by inserting/replacing/removing/pushing/popping them in its overlay. I understand that this approach would cause some problems in the way results are currently returned during push/pop, but these could be tackled in other ways, and I think it would also be much more coherent to the way things are actually built with Flutter, which I like a lot.

Agree with @sroddy on 4).
Navigation is such an important part of application development and in my opinion, current mobile framework (flutter, cocoa and especially android sdk) failed to provide a simple but flexible API.
I would be delighted to see Flutter offer a small revolution here.

I had some discussion with @mehmetf and would like to add my thoughts on this issue.

I need to recreate the navigation stack on program start. That means I will have to push multiple screens to the stack at once.

If a page is about to be shown on screen (pushed or popped), I would like to be able to push/pop other pages. This improves separation of concerns, because the business logic of the page knows possibly the best if it has to be skipped or not.

Both points have a relation with the original issue which will become clear in my explanation here below.
Explanation point 1
Assume I will give all my routes a name, the same name as my pages, and I might have a widget tree like this:

S100 Main menu

S200 Order pick

S210 Select order

S220 Confirm pick location

S230 Pick nr of pieces

S240 Report shortage

So S240 is now the screen that the user sees.
Suppose the app is killed and restarted. I can navigate right away to my named route S240 but then my widget tree will look like this:

S240 Report shortage

The problem is that when I press back, it will close the program. The whole navigation stack/widget tree is not rebuild on application start.
Explanation point 2
Sometimes the best place to make decisions about navigation, is in the page that is about to be shown. For example, if 'S210 Select order' is about to be shown, the page may query the number of lines available and do some logic:

Number of rows == 0
there is no need to show the list as there are no items, show message 'No orders available' and then pop screen of stack

Number of rows == 1
there is no need to show the list as there is only 1 item available, push S220 on the stack

Number of rows > 1
do not change navigation stack anymore, let the user select an order from this page

Of course both options (point 1/ point 2) have to be possible without showing navigation glitches. We need a solution that is not based on 'precise timings'.  Therefore we need a point in time when it is absolutely sure that a navigation has to be performed from one screen to another screen.
Something similar is the .commit function in Android.
https://developer.android.com/reference/android/support/v4/app/FragmentTransaction#commit
Only that function is 'static', the route may not change along the way.
I suggest another implementation. I have a similar solution in C# (Xamarin), served my navigation needs for years. The interface for that implementation is shown here below.
/// Fired when the screen is on top of the stack. Can be used to skip screens
/// </summary>
/// <param name="stackChange">Describes how it came on top (PushedOnTop/ReturnedFromScreen)</param>
/// <param name="previousScreen">A reference to the previous screen</param>
Task ScreenOnTop(DalNavigation.StackChangeEnum stackChange, View previousScreen);

/// <summary>
/// Fired when the screen is actually about to be shown. No changes to the ScreenStack are allowed anymore
/// </summary>
Task ScreenIsShown();

/// <summary>
/// Fired when the screen is about to be popped. Prevent popping by returning false, otherwise return true
/// </summary>
/// <returns></returns>
Task<bool> ScreenIsPopping();

If someone is interested, I could post the actual implementation in C# also.

@jsroest Thanks for the explanation. Your first point is squarely related to this issue but the second point is not. Your second point is suggesting a different way of implementing navigation which requires discussion. I have a couple of suggestions that you can try out. I suggest editing the comment above and leaving only point 1 and opening a new issue for point 2. We can discuss my suggestions on the new issue.

I'm supporting this. It would also be nice if there were more ways to access and modify the history.

@jsoref I would be interested in seeing this. I'm looking for a way to access the Navigator details directly. Like what's on the stack and where? Is it currently transitioning? etc

@ThinkDigitalSoftware I think you meant @jsroest

Is there any progress going on on thins feature?

@Hixie This is kinda blocking our development right now. Any updates, or anything I can do to help?

This will be supported with Navigator 2.0 See https://github.com/flutter/flutter/issues/45938

It's possible right now with Navigator.pages

Just to clarify how this can be achieved with Navigator.pages here's the DartPad sample with usage of setState. It could be much more convenient with some kind of common class for handling the navigation, but for simplicity let's just pass callback to update list of pages. The key is to notify Navigator that the list of pages has changed, that's why list is recreated via List.unmodifiable() or List.of().

And the same with timeDilation = 20:

That's a great example @orestesgaolin.
I think this can be closed, since this works in the master channel. Please reply if you think this is still an issue.


/lib64/libm.so.6: version `GLIBC_2.29' not found

Tested versions
v4.3.rc.custom_build [3978628c6]
System information
Centos8
Issue description
I used Ubuntu 20.04.5 to export a dedicated server, and when running the exported game server on CentOS 8, the following error occurred:
-- Logs begin at Sat 2024-08-03 08:26:07 CST. --
Aug 06 11:58:04 game systemd[1]: Started Engine.
Aug 06 11:58:04 game engine.x86_64[2935612]: /data/wwwroot/engine/engine.x86_64: /lib64/libm.so.6: version `GLIBC_2.29' not found (required by /data/wwwroot/engine/engine.x86_64)
Aug 06 11:58:04 game systemd[1]: engine.service: Main process exited, code=exited, status=1/FAILURE
Aug 06 11:58:04 game systemd[1]: engine.service: Failed with result 'exit-code'.
Aug 06 11:58:04 game systemd[1]: engine.service: Service RestartSec=100ms expired, scheduling restart.
Aug 06 11:58:04 game systemd[1]: engine.service: Scheduled restart job, restart counter is at 5.
Aug 06 11:58:04 game systemd[1]: Stopped Engine.
Aug 06 11:58:04 game systemd[1]: engine.service: Start request repeated too quickly.
Aug 06 11:58:04 game systemd[1]: engine.service: Failed with result 'exit-code'.
Aug 06 11:58:04 game systemd[1]: Failed to start Engine.

# Ubuntu 20.04.5
ldd (Ubuntu GLIBC 2.31-0ubuntu9.16) 2.31
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Written by Roland McGrath and Ulrich Drepper.

# Centos8
ldd (GNU libc) 2.28
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Written by Roland McGrath and Ulrich Drepper.

Which version of the operating system should I use for compilation?
Steps to reproduce
I used Ubuntu 20.04.5 to export a dedicated server, and when running the exported game server on CentOS 8
Minimal reproduction project (MRP)
not

This is expected.
You should create binary on system with lowest glibc version as possible and you will be able to run app on systems with same or greater version of glibc
    Ubuntu:
        Ubuntu 18.04 LTS (Bionic Beaver): glibc 2.27
        Ubuntu 20.04 LTS (Focal Fossa): glibc 2.31
        Ubuntu 22.04 LTS (Jammy Jellyfish): glibc 2.35

    Debian:
        Debian 9 (Stretch): glibc 2.24
        Debian 10 (Buster): glibc 2.28
        Debian 11 (Bullseye): glibc 2.31
        Debian 12 (Bookworm): glibc 2.36

    CentOS:
        CentOS 7: glibc 2.17
        CentOS 8: glibc 2.28
        CentOS Stream 8: glibc 2.28
        CentOS Stream 9: glibc 2.34

    Fedora:
        Fedora 33: glibc 2.32
        Fedora 34: glibc 2.33
        Fedora 35: glibc 2.34
        Fedora 36: glibc 2.35

    RHEL (Red Hat Enterprise Linux):
        RHEL 7: glibc 2.17
        RHEL 8: glibc 2.28
        RHEL 9: glibc 2.34

This is expected. You should create binary on system with lowest glibc version as possible and you will be able to run app on systems with same or greater version of glibc
    Ubuntu:
        Ubuntu 18.04 LTS (Bionic Beaver): glibc 2.27
        Ubuntu 20.04 LTS (Focal Fossa): glibc 2.31
        Ubuntu 22.04 LTS (Jammy Jellyfish): glibc 2.35

    Debian:
        Debian 9 (Stretch): glibc 2.24
        Debian 10 (Buster): glibc 2.28
        Debian 11 (Bullseye): glibc 2.31
        Debian 12 (Bookworm): glibc 2.36

    CentOS:
        CentOS 7: glibc 2.17
        CentOS 8: glibc 2.28
        CentOS Stream 8: glibc 2.28
        CentOS Stream 9: glibc 2.34

    Fedora:
        Fedora 33: glibc 2.32
        Fedora 34: glibc 2.33
        Fedora 35: glibc 2.34
        Fedora 36: glibc 2.35

    RHEL (Red Hat Enterprise Linux):
        RHEL 7: glibc 2.17
        RHEL 8: glibc 2.28
        RHEL 9: glibc 2.34

This is very detailed; thank you for your help. Additionally, I would like to ask if there are any relevant documents for compiling on CentOS 8, as the official documentation seems not to cover the compilation methods for CentOS.

Closed since its resolved.


Optional generated column for insert/update (default constraint)

Use case: I have a field createdat which is auto generated. I'd expect it to maintain its value regardless of when I update it. Basically looking for a way to enforce a generated column to NOT update its value when an UPDATE command is issued.
Feature description:
Table schema
CREATE TABLE mytable(
    somefield STRING,
    createdat TIMESTAMP GENERATED ALWAYS AS CURRENT_TIMESTAMP(3)
)

INSERT INTO mytable (somefield) VALS ('some string')

'some string',<PHONE_NUMBER>000

UPDATE mytable SET somefield = 'another string' WHERE somefield = 'some string'

'another string',<PHONE_NUMBER>000

In the above example, it'd be great to persist the original value even if any of the other column values are updated.

Hi @bputt! You can just use
UPDATE mytable SET createdat = createdat, somefield = 'another string' WHERE somefield = 'some string';
This will keep the initially generated value of the generated column.

Closing but feel free to re-open if the answer didn't solve your problem.

I guess to support this use case in a better way we'd have to implement the default constraint (which we currently don't support)?

I agree, the default constraint would be the preferred option.

ref: apache phoenix for hbase
http://phoenix.apache.org/language/index.html#upsert_values

We added support for DEFAULT in 4.0 (https://github.com/crate/crate/pull/8794)


Update CBOR sample for a more clear twin story

Adds parsing functionality to full twin document, not just desired section
Adds logic to only parse full twin if 200 GET response
On a received PATCH desired property request, only send back a reported property if it was updated locally.
Update logging.

Adds parsing functionality to full twin document, not just desired section
Adds logic to only parse full twin if 200 GET response
Moves all receive_message calls to for-loop
Update logging.