file_dialog segfault

$ ./file_dialog

(file_dialog:82643): Gtk-WARNING **: 22:11:41.653: Theme parsing error: gtk-contained-dark.css:2871:228: Missing closing bracket for :not()

(file_dialog:82643): Gtk-WARNING **: 22:11:43.383: Invalid input string

(file_dialog:82643): Gtk-WARNING **: 22:11:43.383: Invalid input string

(file_dialog:82643): Gtk-WARNING **: 22:11:43.383: Invalid input string

(file_dialog:82643): Gtk-WARNING **: 22:11:43.383: Invalid input string
Traceback (most recent call last)
/tmp/owlkettle/examples/file_dialog.nim(59) file_dialog
/home/user/.nimble/pkgs/owlkettle-1.5.0/owlkettle.nim(72) brew
/home/user/.nimble/pkgs/owlkettle-1.5.0/owlkettle/widgets.nim(32) event_callback
/tmp/owlkettle/examples/file_dialog.nim(40) :anonymous
/home/user/.nimble/pkgs/owlkettle-1.5.0/owlkettle.nim(35) open
/home/user/.nimble/pkgs/owlkettle-1.5.0/owlkettle/widgetdef.nim(48) build
/home/user/.nimble/pkgs/owlkettle-1.5.0/owlkettle/widgets.nim(1369) build
SIGSEGV: Illegal storage access. (Attempt to read from nil?)
Segmentation fault (core dumped)

archlinux
Nim Compiler Version 1.6.6
to reproduce just click "Open" button

I am currently unable to reproduce this issue. Does it only occur when no item is selected? Which GTK version are you using?

Closing because of inactivity.

I finally was able to reproduce this when compiling with -d:danger. It should be fixed in https://github.com/can-lehmann/owlkettle/commit/d202473bb055201d34e7ae7245e83988f8410523.

Thank you for the detailed report. I think you still use the version without the fix, since it says "/home/user/.nimble/pkgs/owlkettle-2.0.0/owlkettle/widgetdef.nim (531, 6) Hint: 'gen_docs' is declared but not used [XDeclaredButNotUsed]" in the warning, but I renamed this function to "genDocs" since. Please try installing the latest version using nimble install owlkettle@#head.
I am also a bit confused why it does not render any text. The blue button should be labelled "Open".

@luntik2012 To get the version number of a specific library with nimble, you can do :
nimble list -i | grep owlkettle

Btw, with owlkettle 2.1.0, on Ubuntu 22.04.1,[DE: Gnome 42.4, WM: Mutter, WM Theme: Adwaita], this example file_dialog.nim works fine for me.
@luntik2012 Could you give us your WM & WM Theme ?
I'll try on my Arch desktop when at home.

I tried it with endeavor OS which is Arch based and it worked fine here.  Seems to either been resolved in owlkettle or through updates to arch repositories.

@beef331 Thanks for testing this. Since multiple people were now unable to reproduce this issue, I will close it again for the time being.

It also works on my Arch distribution. Sorry for the very late reply.

@dlesnoff No worries. I'm glad that it seems to be resolved.


Don't use lldb for macos because stdlib tests fail silently with it

it seems the signal passing of SIGINT from lldb doesn't behave the same on macos as it does on linux..

the issue can be seen in the macos CI builds in PRs for the past few months..
stdlib tests fail early and silently (because lldb doesn't pass through the non-zero program exit code) with the following:
Process 47813 resuming
Process 47813 stopped and restarted: thread 1 received signal: SIGINT
Process 47813 exited with status = 2 (0x00000002) Terminated due to signal 2


[REQ] Image zoom , panning.. like a PRO

Hi @Zeus64
I'm using Alcinoe+Skia for my VIP FMX app, and I need a function to display images on Android/iOS with modern, high-quality rendering (similar to what Apple or Flutter offers). I've been looking for examples, but I haven't found anything that achieves this level of quality.
Currently, I'm using a WebBrowser to enable smooth zoom functionality.
Do you have any tips or examples on how to use TALImage like a pro?
Thank you!
Mauro

What exactly you want to achieve ? I not understand what you mean by "I need a function to display images on Android/iOS with modern, high-quality rendering (similar to what Apple or Flutter offers)", can you show me some screenshot ?

New form, drop a image component, set it
Full screen, add any code for support zoom and panning,  compile for Android, Run...
At runtime che the zoom quality .. is terrible.
If you use a TWebbrowser for view image the zoom is good.

Ah, I see! In this approach, you can handle it manually. Start by creating a texture of the image using a function like ALLoadFromStreamToDrawable, specifying the maximum size you want to achieve. Then, in the FormPaint event, use ALDrawDrawable with the appropriate ADstRect to paint the image onto the form. When panning, simply adjust and increase the ADstRect. Since the texture is already large, you won't lose quality.


Are we missing  in singular element list?

Minifying:
asdf
qwer<wbr>zxcv
Result:
asdf
qwer<wbr></wbr>zxcv

released as 1.6.1


fix(ios): ignore "nothing to terminate" simctl error

Description
I was having problems running my detox tests in CI (but not locally?) that looked exactly like #3518 and it seemed easy enough to patch, so I tried it and it seems to work for me.
Unsure on formatting here but either CI will tell me or you all will :-) - I was hacking it locally via patch-package so don't have devtools for the repo set up + available at the moment sorry
@asafkorem you marked this as assigned to yourself, so perhaps you want to give it a look?
Fixes #3518 (I think - it does for me anyway)

For features/enhancements:

[ ] I have added/updated the relevant references in the documentation files.

For API changes:

[ ] I have made the necessary changes in the types index file.

Huge thanks @mikehardy!

Thanks for the link to the upstream issue on messaging, that explains why this only started showing up for me recently, but was not dependent on (for instance) trying Xcode 14 beta (which I'm also giving a look at from time to time)
Either way, happy to help, you all have my continued appreciation for maintaining Detox! Cheers

Version 19.9.2 was released with this patch 🙂
Thanks again Mike!


get-info task is dependent on module that can change

Currently running a module generates a journal file that records the state of the deployment run. Get info shows the successfully deployed contracts for a module. To do this we rebuild the state of the previous deployment run from the module + the journal (notable including validation). This works assuming the module does not change or get removed, but steps like validation require a working module and the reconciliation check requires that it matches.
The state of the deployment run is coupled with the original module, but we do not control the original module. This will break when the user manipulates or deletes the module - the question is whether we consider that a problem Ignition should live with.
Potential approaches to breaking the journal/module coupling:

build a reducer that works just against the journal without need for reference to the module or the graphs we create from it.
rework our internal phases so they can be optionally composed, effectively allowing us to run rebuild state without validation for instance. This doesn't solve the need to have the module + graphs built from it to rebuild the state. Two sub approaches to this might be:

include the execution graph in one of the journal commands so we can recreate state without the original module
export the execution graph along side the journal within a deployment package/artifact/whatever name makes sense, then refer to it in the start execution phase command in the journal

Part of #203.

No longer relevant.


Seperate solution from an exercise visually more

We discussed how solutions can be visually more distinguished from the exercise. Currently we use headings for archiving this. However, it would be great to have a technical solution. Here is our proposal:

Who was involved in that discussion? Especially, did you already talk to @elbotho? We had a very similar (not to say the exact same) design somewhere around two years ago. We could probably go back to that easily, but I think there were some considerations leading to the change.

@Knorrke We had this discussion during the meeting today with the academy. So we have backup by 20 people who where involved in the discussion (more or less actively). @elbotho gave a :+1: as I noticed seeing over...

Alright then :+1:

Sorry, the 👍 was for using the real serlo blue :)
We removed the blue bar when implementing the interactive exes.
Don't remember completely but I think it was a lot cleaner.
But since you are searching for a highlight…
What do you think about something like

Might look strange for some answers with images, but should be fine most of the time and would be easy to implement. Smth. like:
.collapse.in {
    display: block;
    background: #d9edf7;
}

Might also look strange for long answers... I'll ask on the channel for help...

Any news @kulla? As I've seen, your suggestion looks much like the new design on "Mathe für Nicht-Freaks". So should we go with that, to have a more uniform design?

I'll ask in the community...

@Knorrke Can you implement the blue bar only for text solutions (and not for other types of interactive answers)?

What do you mean by interactive answers?


[BUGFIX] [ODP-246], [Editor] Remove implicit transform of uploaded file

[DESC.]

Can one of the admins verify this patch?


add demo users

add demo users to the server in order to easily test and showcase the behavior (with an option to not add them on server startup)

Issue that arose:

when injecting the function, to add users, active games, and matches to the respective databases, into the UserService constructor, the result is as expected. (users, active games, and matches are persistent in the repositories and can be interacted with)
however, this breaks most (if not all) tests for the userService. I am not sure what causes all these tests to break, but I assume that the tests assume an empty database and they fail when other users etc are introduced before the tests run.
maybe we can make the tests ignore those users?

Update
Suggested solution from our TA: add a REST API endpoint without integrating it into the frontend. That way we can easily add these users for developing and external users (for the beta testing phase) can add these users as well, by typing in the required URL.
I will implement a guard (check) that checks if the users have been added to the server before (probably with inspiration from the singleton pattern, to only allow the function to be executed once, else nothing happens).


Find a good route with multiple POIs

I just did some research regarding the algorithms needed to find a good route that includes POIs chosen by the user.
To find a route that is not longer than 1.5 times the optimal round given a set of POIs two algorithms need to be implemented:

Kruskal's Algorithm to find a minimum Spanning tree
Christofides algorithm to solve the Traveling Salesman Problem with that minimum Spanning tree

To implement these two algorithms is quite a bit of work so i want to make sure the requirements to the route generation algorithm for routes including POIs are set and i understood them correctly.
In my understanding the user scenario we agreed to on wednesday would be something like the following:

The user chooses a set of unordered POIs and no expected route length
The algorithm returns the approximately shortest possible route with those POIs

In my opinion it could be difficult to include this in the user interface in a consistant manner and as a user I might ask myself: why can't I specify a route length when trying to include POIs in the route. For this reason I want to make the case for the algorithm that is implemented already, just to make sure we really want to change it.
what is implemented right now
Advantages:

User can specify a route length
POIs are chosen automatically by specifying a category

Disadvantage:

Order of POIs might be suboptimal, but that doesn't really matter since a route with the specified length is expected, not the shortest

A compromise between the two solutions would be that the user chooses the POIs and gives an expected route length, and a route with these parameters gets generated, using the nearest neighbor approach.
I'm looking forward to your feedback and apologize for the long issue. Implementing these algorithms just felt like its not really what we want and a bit too time consuming to just do it when we're not sure we want it.

I think the user should get the opportunity to choose a sequence of POIs and a minimum distance of the calculated route (I mean, he/she wants to hike right?). This would mean, the route could be longer anyways and it would look legit to the user that an increasing number of POIs would lead to an increasing length of his route.

The minimum route length parameter is a good idea imo. But for that the Traveling Salesmen Problem doesn't need to be solved either.. The only case in which it matters is when the user wants to hike the minimal required distance to reach the given POIs. And since we wouldn't solve that case optimally anyways I'm not sure whether the algorithms are really necessary or the nearest neighbor approach would be close enough already.

During our conversation last Wednesday, we decided on the following core functionalities for the time being:

determining a random route after specifying a fixed length
determine a route with point of interests you select before

The first functionality is already implemented like that.
The second point could be understand in two different ways:

user specifies what kind of poi's he is interested in (categories) and a route will be generated with those kind of pois
(e.g. interested in viewing platforms and hiking huts -> random route with viewing platform/s and/or hiking hut/s)

or

user specifies what kind of pois he is interested in and all those possible poi's in his range are now listed (with distance and sorted to the user) for him to select. After selecting an maximum amount of n poi's an random route will be generated with those routes, but an max length would not be required here

Personally I would prefer the second approch because it is much more interesing to see what kind of pois are nearby and to choose myself which of them should be included in my route.
But I see other people that are not so much interested in this point and just want to walk a nice route and see interesting stuff (from categories they specified) on their way.
Would it be possible to create all three behaviors?

random route
random route with poi categories
route with specified pois

What part would be the most time consuming?
UX-wise how to explain those three types of route search is hard indeed. Eventually we discuss this in the next meeting tuesday.

On my hand side I'd prefer a specified route length and if it happens to have certain POIs - nice to have but for me not mandatory. But I can also see that this kind of selection can be confusing to users. Therefore at least a "warning" like "route caluclation might take a minute or two" should be displayed and agreed to. I like Philips 3 cases of behavior and would UI sided blank out or disable non available options, always accompanied a a little explanary side note. Nothing fancy, but easy to understand.

Would it be possible to create all three behaviors?
1. random route

2. random route with poi categories

3. route with specified pois

What part would be the most time consuming?

I agree that we should implement these three scenarios. The time consuming part is to calculate an optimal route between the POIs, so we have to decide if thats worth it. We can implement all these three things without calculating an optimal route.

After selecting an maximum amount of n poi's an random route will be generated with those routes, but an max length would not be required here

What do you mean with max length? I think the behavior robin described, where a minimum route length is shown to the user and he can make it longer if he wants to, would be the easiest to include the consistently, since the length parameter is still there, just restricted by the number of POIs.
I would propose we implement these 3 scenarios with the nearest neighbor approach and see how that goes, if the results are too bad we can try the algorithms I described above.

What do you mean with max length?

I meant a length in itself. We discussed that, if the user wants to specify pois himself he must dispense with the specification of a route length.

think the behavior robin described, where a minimum route length is shown to the user and he can make it longer if he wants to, would be the easiest to include the consistently, since the length parameter is still there, just restricted by the number of POIs.

IMO we can go with the minimum route length on all three approaches and on scenario 3 the calculated routes start at an different minimum route length as stated from you, if you ment that by restricted.
I would also like to discuss the user handling of the routing process, because there are different specifications on scenario 1 and 2 in contrast to 3.


Problems importing SwinjectStoryboard

I'm facing with: No such module SwinjectStoryboard trying to use version 1.0.0 - Swift 3.

How did you install SwinjectStoryboard?

@igordeoliveirasa please feel free to reopen this issue with more information if you still face the problem😉


Made MessageDispatcher extendable.

and added convenience creator method for providing another ObjectMapper

I will solve this in another way, need to figure out how. Closing this.


Fixes Falsy Bouncer #47622

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #47622

My first PR. Let me know if you have any feedbacks. Thanks :D

I've committed the suggested changes.
The test description was already there before.

banger pr, thanks for contributing!

Thank you for guiding me through my first PR. :D

Awesome! I got to learn a lot from this PR, thank you!


[Idea] What about using MCP23017?

I was thinking, right now we have only three free gpios remaining, which could be used for Savegames or for SD Card, but not both at the same time/cartridge, so, what about using a MCP23017 (GPIO expansion module) in not-crucial timing I/O?
For example (disclaimer: I'm not yet familiar with the code), it's obvious that we couldn't use it with AD* pins, but what about Cold Reset? Or maybe for CS pins on SPI modules)
It's just an idea btw, so don't take it as a solid solution

Thanks for the suggestion. This is not something I am investigating. I have changed focus for this project to keeping it simple and reduce external components.


Remove the temporary fixed for Azure CLI 2.30.0

The temporary fixed has been removed because of this issue:
ERROR: AuthenticationException:
Message: AzureCliAuthentication requires azure-cli>=2.30.0 to be installed in the same python environment where azureml-sdk is installed.
InnerException None
ErrorResponse
{
"error": {
"code": "UserError",
"inner_error": {
"code": "Authentication"
},
"message": "AzureCliAuthentication requires azure-cli>=2.30.0 to be installed in the same python environment where azureml-sdk is installed."
}
}


[cli] Optimize Dockerfile

Current Dockerfile for CLI contains many unnecessary layers, is not well formatted, Docker seems not be in PATH (#8771), autobuild on Hub is not used etc.
Will be nice if some Docker experienced developer will rework it to follow all best practices. At least declare PR is welcome.

Thanks for your report!
With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.
If you think we should keep this issue open, please let me know.


🐛 Fix HTML report sorting order storage in jQuery unload event

🐛 Fix HTML report sorting order storage in jQuery unload event.
This is related to: https://github.com/nedbat/coveragepy/issues/986
I'm having a (probably) similar issue. In the generated HTML report, on the console, I'm seeing an error:
Uncaught TypeError: $(...).unload is not a function
    at HTMLDocument.coverage.index_ready (coverage_html.js:227)
    at mightThrow (jquery.min.js:2)
    at process (jquery.min.js:2)

Using $().unload was deprecated and removed in jQuery 3. The recommendation is to use $().on("unload", handler).
I see that it seems the included minified jQuery JS file says it has version 1.11: https://github.com/nedbat/coveragepy/blob/master/coverage/htmlfiles/jquery.min.js#L1
Nevertheless, when I generate an HTML report from scratch in a local project, the minified jQuery I get is indeed 3.x. I have no idea why.
Either way, the method with $().on("unload", handler) works with both versions.
For completeness, I'm on Chrome, on Ubuntu.

Thanks for the detailed report.  How did you install coverage?  I have a feeling the jQuery version difference is because you installed with an Ubuntu package rather than with pip install, is that right?

Not really, I installed through pytest-cov, with Fllit (equivalent to with pip), in a virtual environment.
Thanks for reviewing and merging!

This is now released as part of coverage 5.3.1.


Nullable types and EntityFramework.Filters

EntityFramework 6.
The query:
var item = context.Items.FirstOrDefault(x => x.NullableProperty == filter.NullablePropertyValue);

Without EntityFramework.Filters it translates to:
...
WHERE (([Extent1].[NullableProperty] = @p__linq__0) OR (([Extent1].[NullableProperty] IS NULL) AND (@p__linq__0 IS NULL)))

After adding filter interceptor (both in DbConfiguration class or in OnModelCreating) the query is translated to:
...
WHERE ([Extent1].[NullableProperty] = @p__linq__0)

DBContext.Configuration.UseDatabaseNullSemantics = false; does not fix anything.

If your filter.NullablePropertyValue is not null, then
WHERE ([Extent1].[NullableProperty] = @p__linq__0)

its right.
But If your filter.NullablePropertyValue so the query should be just
WHERE (([Extent1].[NullableProperty] IS NULL) AND (@p__linq__0 IS NULL))

EF 6  has new default behavior - it translates queries for nullable types using this syntax:
WHERE (([Extent1].[NullableProperty] = @p__linq__0) OR (([Extent1].[NullableProperty] IS NULL) AND (@p__linq__0 IS NULL)))

After adding EntityFramework.Filters interceptor to the context this behavior is changed and queries are translated using old syntax i.e. EntityFramework.Filters messes up EntityFramework's behavior and produces unpredicted results.
The behavior of EF is controlled by configuration key DBContext.Configuration.UseDatabaseNullSemantics, but when EntityFramework.Filters interceptor is added this configuration does not change anything.

I had the same issue in my own extended filterung framework. It's a pretty simple change of the new EF versions > 6.0. They introduced a new property context.Configuration.UseDatabaseNullSemantics. The same property was introduced in the class DbQueryCommandTree, which must be created when filtering the base tree.
To solve the issue you only have to change FilterInterceptor.cs:
....
interceptionContext.Result = new DbQueryCommandTree
(
queryCommand.MetadataWorkspace,
queryCommand.DataSpace, newQuery,
validate: false,
useDatabaseNullSemantics: interceptionContext.Result.UseDatabaseNullSemantics //otherwise we loose the setting
);
...
That's it :-)

I have fix localy by remplacing in FilterInterceptor.cs:
interceptionContext.Result = new DbQueryCommandTree(
queryCommand.MetadataWorkspace, queryCommand.DataSpace, newQuery);
by
interceptionContext.Result = new DbQueryCommandTree(
queryCommand.MetadataWorkspace, queryCommand.DataSpace, newQuery,true, context.Configuration.UseDatabaseNullSemantics);
If I have time, I fork and PR after work.


Chatbot Clear history API

Clear the history by passing ,uri, db, sessionid

Closing the issue already merged to DEV branch


Clear $stateParams

Hello,
Is there a way to clear $stateparams?
When routing between states some the $stateParams stays there and causing me a big mess in the app. I there a way to clear stateParams?
Thanks!

Hi, better to ask this sort of question on SOF
http://bit.ly/UIR-SOF

You can clear the values by setting the key to undefined when you transition.
For example, to transition to the current state but clear the time parameter you can do:
$state.go('.', {time: undefined} );

Thanks!


Enhancement Of Footer

Prerequisites

[X] I am an Open Source Contributor
[X] I checked to make sure that this issue has not already been filed

Description
Enhancement Of Footer
This issue is to make the footer of the website much more responsive and interactive.
The following features will be updated in this issue:

Interactive
Responsive
Social Media Links
Social Media Icons
Hover Effects

The changes before and after the improvements can be seen below in the Screenshots section.
#SWOC
Screenshots
Original Footer :

Footer after making Improvements:

Footer with Hover Effects :

Code of Conduct

[X] I follow Contributing Guidelines of this project.

I am a beginner and would love to put my learnings to make your project better. Hope I get this issue assigned and start contributing.

@CodeByRajeev Assigning, Proceed!!


fix: Push git refs atomically

Several times I've run into an issue where:

My main has diverged from upstream main.
I use cargo release to push both main and a tag.

In this situation, the tag push succeeds while the push to main fails. This isn't desired and in my case breaks automation, which expects the tag to be an ancestor of main.
Use an atomic push to avoid this issue. The man page says:
--[no-]atomic
Use an atomic transaction on the remote side if available. Either all
refs are updated, or on error, no refs are updated. If the server does
not support atomic pushes the push will fail.

I was a bit worried about the "push will fail" bit, but https://github.blog/2015-04-30-git-2-4-atomic-pushes-push-to-deploy-and-more/ says that atomic pushes were added in 2015. I can't imagine anyone running a 7 year old Git on a server at this point, especially since there have been multiple security vulns in Git since then.

Released in v0.22.3


Sampling step?

Thanks for your documentation and transparency here
Quick question, in sample_from_softmax(logits, disallow=None), you return:
tf.one_hot(tf.argmax(tf.nn.softmax(logits + gumbel_noise), -1, output_type=tf.int32), logits.shape[-1])
Wondering why tf.softmax is needed here if the result will be passed through tf.argmax anyways? Perhaps this is a holdover from another experiment?
Thanks!

Hey, did you find the answer to this?

Hi, I didn't, sorry


Make our tail walkable unless we eat food.

The pathy-snakes tail will move with our head so really its current cell is one we can walk to. We do however need to account for cases where we just ate food as this will cause our overall length to grow by one and make our tail not walkable.
https://github.com/es-na-battlesnake/snakes/blob/55eb57c5bf296e693192abd2b7e0256232c95048/snakes/go/pathy-snake/pathing.go#L8

Add in #125


Copying fifos hangs.

shelljs' cp will hang if trying to copy a fifo.
This PR skips copying more non-file-like entities to avoid this issue.

Hmm, intriguing.
cp -r also seems to act differently to a straight copy? (I was copying a directory, and it was what lead me to creating this PR)
eg:
mkdir a
mkfifo a/fifo
cp -r a b # no hang
ls b # fifo

So this PR isn't properly replicating cp, no... (Although, afaik, node doesn't support creating fifo's; except I guess if this shells out? :-s (Or uses some native bindings))
execve("/usr/bin/cp", ["cp", "-r", "a", "b"], 0x7ffd9a4e0168 /* 76 vars */) = 0
...
lstat("a/fifo", {st_mode=S_IFIFO|0644, st_size=0, ...}) = 0
mknod("b/fifo", S_IFIFO|0644)           = 0

When copying the file directly (eg cp fifo newFifo); cp does:
stat("newFifo", 0x7fffdfb9e570)         = -1 ENOENT (No such file or directory)
stat("fifo", {st_mode=S_IFIFO|0644, st_size=0, ...}) = 0
stat("newFifo", 0x7fffdfb9e300)         = -1 ENOENT (No such file or directory)
open("fifo", O_RDONLY)                  = 3
fstat(3, {st_mode=S_IFIFO|0644, st_size=0, ...}) = 0
open("newFifo", O_WRONLY|O_CREAT|O_EXCL, 0644) = 4
fstat(4, {st_mode=S_IFREG|0644, st_size=0, ...}) = 0
fadvise64(3, 0, 0, POSIX_FADV_SEQUENTIAL) = -1 ESPIPE (Illegal seek)
mmap(NULL, 139264, PROT_READ|PROT_WRITE, MAP_PRIVATE|MAP_ANONYMOUS, -1, 0) = 0x7f0cb3a6f000
read(3, "q\n", 131072)                  = 2
write(4, "q\n", 2)                      = 2

vs the following for shelljs:
open("fifo", O_RDONLY|O_CLOEXEC)        = 9
open("newFifo", O_WRONLY|O_CREAT|O_TRUNC|O_CLOEXEC, 0666) = 10
pread64(9, 0x26697d0, 65536, 0)         = -1 ESPIPE (Illegal seek)

(The point being read vs pread; wherein pread says "The file referenced by fd must be capable of seeking." 1 which is likely the issue here? :-s (This comes from uv by the look of it? 2)
(I'm just braindumping as I follow some of the rabbit hole :-) )

Okay, so I think I've fixed the scenario you've referenced :-)
(Setting that offset to -1 (was zero) means that libuv uses read rather than pread :-) )

@bruce-one is there documentation on setting the offset to -1? I'm not seeing any references to that in the node documentation or source code.

Ah, no, I just found it in https://github.com/libuv/libuv/blob/v1.x/src/unix/fs.c#L306 (and when looking through the libuv tests -1 is used for the offset quite a bit -- was trying to replicate it in straight uv and stumbled across the difference)

That switch has been there for a very long time; and it kinda makes sense (I think?) because it's the way to say "no need to worry about offset, so just read not pread":
https://github.com/libuv/libuv/commit/74999f8f9974970e4d71a0df521e178e3149f3df#diff-6a16903c26af4b4035eda9922a73ecc9R160 (via  a few steps in the middle)

(Sorry for the noise; trying to fix the tests on Windows. It looks like Appvoyer's environment (cygwin?) supports mkfifo, but fs.existsSync fails to recognise that fifo's exist, I'm assuming it's something to do with how cygwin(?) fakes fifo's, whilst node is trying to go direct?)

Codecov Report

Merging #727 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #727      +/-   ##
==========================================
+ Coverage   94.81%   94.83%   +0.02%
==========================================
  Files          33       33
  Lines        1254     1259       +5
==========================================
+ Hits         1189     1194       +5
  Misses         65       65

Impacted Files
Coverage Δ

src/cp.js
91.47% <100%> (+0.34%)
:arrow_up:

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 15558cf...5689dc0. Read the comment docs.

@bruce-one are we confident that isFIFO() always returns false for Windows, or is this something we need to investigate? I'm hesitant to add features that go untested on Windows--if there's any chance we hit those code paths in production, then we have unpredictable behavior. If we can guarantee that we don't hit that code for that platform, I feel a bit more comfortable.

From what I know/can ascertain (some but not extensive research), Windows can't put named pipes on a filesystem? (a la mkfifo x; ls # x Hence I don't know of a situation where isFIFO could return true on Windows.
Hence support for this in native windows isn't possible? (Cygwin emulates pipes, but unless node is built for Windows, I don't think node will report them - from my testing.)
I'm no Windows expert though... :-s

Another outstanding question with this is in the recursive type scenario: should we call mkfifo or something to create the fifo to better replicate cp? (afaict, node can't do that natively)
To illustrate:
mkdir a
mkfifo a/fifo
cp -r a b
ls b # fifo ; with shx this fifo wouldn't exist as it stands in this pr

Another outstanding question with this is in the recursive type scenario: should we call mkfifo or something to create the fifo to better replicate cp? (afaict, node can't do that natively)

Seems like the answer would be yes. Our goal here is compatibility with POSIX behavior.

Have added support for acting like -R has been specified (when recursive is true).
From some reading (eg https://unix.stackexchange.com/questions/18712/difference-between-cp-r-and-cp-r-copy-command) -r (by contrast to -R) is not well defined in this context, however based on the existing code and the GNU implementation I've just made -r act like -R. I don't know whether adding a --copy-contents flag becomes useful now? (It's not posix... From skimming the posix definition there's no equivalent of that GNU flag?)
Unfortunately creating character and block devices requires root permissions... I didn't know how to handle that. (Running ava with sudo and a grep for root (ish) after running it as non-root seemed like a tempting solution? But I wasn't sure what was preferred.)
So for now those two tests just don't run on CI :-s (And I haven't even manually tested them on OSX...)

@bruce-one thanks for following up on this

Unfortunately creating character and block devices requires root permissions...

I would like to steer clear of root-only features in ShellJS core.
After thinking a bit, your initial suggestion may have been right after all: skip these file types by default. We can indicate an error occurred via common.error() and copy all the other files.
Since this is a great PR though, I think this would be a good candidate for a plugin. We're adding a plugin API for v0.8, allowing users to extend ShellJS. We could create a plugin to add back support for fifo and other specialized file types, overriding the default behavior.
shell.cp('blockDeviceName', 'dest'); // default behavior
// cp: block device is not supported: 'blockDeviceName'

require('shelljs-plugin-cp-special-files');
shell.cp('blockDeviceName', 'dest');
// Now it succeeds! (assuming we're on Unix and have root permission)

ShellJS proper won't need to have any more platform-dependent code, and we can still provide support for the rare users who need it. This sounds like a win. What do you think?

I've done that cleanup based on the feedback from your review (thanks! :-) ).
I agree that making this a plugin does seem reasonable :-)
Hence, should this PR be changed to just log the error message and then independently create a plugin package?
Although, it might be nice to mention the plugin in the error/warning message? Hence maybe make the plugin, then change PR, then merge? Or are you not that keen on "centralising" plugins quite that much? (e.g. if it were called out explicitly (a la cp: block device is not supported: 'blockDeviceName' and at the end of the cp run Special files were encountered during this operation. Add the cp-special-files plugin if support for these is required.) then it might be nice to have it as shelljs/plugin-cp-special-files? Absolutely no issue if that's not how you want to handle it though :-)
One of the reasons I ask that is because a "shelljs/plugin-posix" could be useful (from past experience) because I know I personally use shelljs solely on posix-like OSs, and find it's consistency excellent (and still very useful), but being able to do "all the posix things" in one plugin could be nice?

One of the reasons I ask that is because a "shelljs/plugin-posix" could be useful (from past experience) because I know I personally use shelljs solely on posix-like OSs, and find it's consistency excellent (and still very useful), but being able to do "all the posix things" in one plugin could be nice?

Sounds like a good idea. Let's discuss the plugin over at #748. We'll leave this issue for only the cp-specific changes.

(btw, more than happy to just log and not copying when in non-recursive mode, was just explaining why I'd originally left that aspect alone because it feels like sound logic (hence why I'd left it :-p ), but I'm more than happy make the change :-) )


Fix double free in main.cpp's save

I should preface this by saying that I do not know if this is the right solution. This is probably hacky, as I do not know why a free was done there in the first place.
This is a follow-up to this comment I made on #138.
This should fix #138 and fix #140.

@utybo I am not sure, but I feel like you might just omit the release of ncnn::Mat or the image memory space. And I am afraid this may leads to memory leak issues.
Additionally, I remember ncnn had fixed the problem in the recently commits. Could you try update ncnn to the latest version and build again to see if the problem still exists? Because I don't have the problem you have and I have no way to see if this works.

@ArchieMeng I can still repro using the latest version of ncnn (tag 20210507). The output is the same as my initial comment

I am not sure, but I feel like you might just omit the release of ncnn::Mat or the image memory space. And I am afraid this may leads to memory leak issues.

To be honest, I don't really know what the loop does in the first place, but from what I understand, the freeing is unecessary as it is already done by getting out of the scope where the Task variable is declared (which is a value and not a pointer). The (implicit) destructor of Task automatically calls ncnn::Mat's destructor, which in turns calls ncnn::Mat::release which then does the problematic free for us. stbi_image_free is a macro which ends up leading to a free, so all branches of both if/elses (the one in code and the one using the preprocessor) end up doing the same thing, which is freeing v.inimage.data.
TL;DR: the code frees an internal pointer of ncnn::Mat which is already destroyed by Task's destructor, which calls ncnn::Mat's destructor, which takes care of that internal pointer.

fixed in https://github.com/nihui/waifu2x-ncnn-vulkan/commit/f332f3c256d3a7f9b7c18a23a05a4e402ea9fd18


[Sprint] Change pydantic to dataclass

Description

Change the pydantic that defined the API schema to a dataclass.

Checklists

[ ] I would create a corresponding branch for this issue from the designated (mostly develop) branch.
[x] This issue only contains a preceding agreement between project owners.

cc. @illian01 ❤🔥

Hello @Only-bottle. I’ve updated the auth client v2 to use dataclasses instead of pydantic as we decided for PyNetspresso.
Let me know if you have any feedback.

This issue has been resolved, so we're closing it.


Feature/bdd test support

CCR-BDD test support.
New user and roles are defines and ids assigned to html elements

RE the button not having an id, the solution is to get a reference to the form and call the submit method on it:
https://www.w3schools.com/code/tryit.asp?filename=FO93RE55234A
{code}

  First name:
  Last name:

Submit the form

Click on the submit button, and the input will be sent to a page on the server called "/action_page.php".

{code}


styling in the svelte REPL?

Hi, great component, very useful for me! question: how do I need to adapt the code to do the styling in the svelte REPL?
https://svelte.dev/repl/3434f0b23d534ba4bb962ec88a3ae1e5?version=3.48.0
Many thanks!

glad it helps,
a style section should suffice.
did you check the other demo link? it has  many examples which should just run with a copy paste into REPL

Thats what I did, the link above is the cut and pasted version with styled splitters, but the styling does not show up unfortunately...

ok. unfortunately not the best time for me to troubleshoot. please give me 48h ty

didn't try your version of the code, took one of the ones from the demo page, copy/paste
->https://svelte.dev/repl/95d67f36254c499ca9e5c69c32d7abbb?version=3.48.0
seems to work

ok, I had left a class="default-theme" in the split-and component....
thanks, all solved....

good.
Indirectly related, I found that a class prop on Pane could be super useful.
Change already pushed.
Sincerely

Hi, thank you for pointing issues with styling. I reworked it with a theme attribute and fixed the css styles.
it's now much easier to style and more consistent.  check the demos.
Sincerely


Make IV/buf in prov_cipher_ctx_st aligned

The benchmark and reason behind this alignment issue can be found in
https://gist.github.com/ZenithalHourlyRate/7b5175734f87acb73d0bbc53391d7140#file-2-openssl-long-md
In short, misaligned oiv/buf/iv in prov_cipher_ctx_st will lead to drastically performance drop when doing aes-ctr/cfb/ofb
This improves the performance of master when compiled to RISC-V, #18197 and #17640. Benchmark on other architecture is wanted but I do not have enough resources.
Generally, making a buffer aligned is wanted for most architecture. Though this leads to a larger memory footprint.
The issue is suggested by @cyyself and this solution is inspired by @dramforever

This pull request is ready to merge

Merged to master branch. Thank you for your contribution.


After upgrading Docker/Linux Database Assistant can't evaluate free memory using  12cR2 image

I recently build a new 12cR2 image from scratch and try to create a fresh database using:
[mochoa@localhost dockerfiles]$ docker run -e ORACLE_SID=TEST -e ORACLE_PDB=PDB1 --privileged --name test_env --hostname test_env -p 1521:1521 -p 5500:5500 -v /dev/shm --tmpfs /dev/shm:rw,nosuid,nodev,exec,size=2g  -v /home/data/tmp/test:/opt/oracle/oradata -v /home/mochoa/tmp/test:/opt/oracle/scripts/setup oracle/database:<IP_ADDRESS>-ee
Docker container start but Database Configuration Assistant can't find how much memory is free in my system, here the output:
ORACLE PASSWORD FOR SYS, SYSTEM AND PDBADMIN: ImBxMQaASOw=1

LSNRCTL for Linux: Version <IP_ADDRESS>.0 - Production on 11-JAN-2018 20:10:08

Copyright (c) 1991, 2016, Oracle.  All rights reserved.

Starting /opt/oracle/product/<IP_ADDRESS>/dbhome_1/bin/tnslsnr: please wait...

TNSLSNR for Linux: Version <IP_ADDRESS>.0 - Production
System parameter file is /opt/oracle/product/<IP_ADDRESS>/dbhome_1/network/admin/listener.ora
Log messages written to /opt/oracle/diag/tnslsnr/test_env/listener/alert/log.xml
Listening on: (DESCRIPTION=(ADDRESS=(PROTOCOL=ipc)(KEY=EXTPROC1)))
Listening on: (DESCRIPTION=(ADDRESS=(PROTOCOL=tcp)(HOST=<IP_ADDRESS>)(PORT=1521)))

Connecting to (DESCRIPTION=(ADDRESS=(PROTOCOL=IPC)(KEY=EXTPROC1)))
STATUS of the LISTENER
------------------------
Alias                     LISTENER
Version                   TNSLSNR for Linux: Version <IP_ADDRESS>.0 - Production
Start Date                11-JAN-2018 20:10:16
Uptime                    0 days 0 hr. 0 min. 8 sec
Trace Level               off
Security                  ON: Local OS Authentication
SNMP                      OFF
Listener Parameter File   /opt/oracle/product/<IP_ADDRESS>/dbhome_1/network/admin/listener.ora
Listener Log File         /opt/oracle/diag/tnslsnr/test_env/listener/alert/log.xml
Listening Endpoints Summary...
  (DESCRIPTION=(ADDRESS=(PROTOCOL=ipc)(KEY=EXTPROC1)))
  (DESCRIPTION=(ADDRESS=(PROTOCOL=tcp)(HOST=<IP_ADDRESS>)(PORT=1521)))
The listener supports no services
The command completed successfully
[FATAL] [DBT-08001] Unable to check for available memory.
[FATAL] [DBT-08002] Unable to check the value of kernel parameter {0}
cat: /opt/oracle/cfgtoollogs/dbca/TEST/TEST.log: No such file or directory
....

my oracle images are:
oracle/database                                       <IP_ADDRESS>-se2             7de3eeaf611b        5 hours ago         10.5GB
oracle/database                                       <IP_ADDRESS>-ee              4cdd6bc17b71        22 hours ago        10.6GB
oracle/database                                       <IP_ADDRESS>-ee              e194b38516ec        29 hours ago        13.3GB
oracle/database                                       <IP_ADDRESS>-xe              b012e54c0a0d        3 weeks ago         1.13GB

Oracle 12c R1 EE and SE start without any problem and creates a fresh database using same command line.
My Docker Linux environment is:
[mochoa@localhost dockerfiles]$ docker info
Containers: 2
 Running: 1
 Paused: 0
 Stopped: 1
Images: 717
Server Version: 17.12.0-ce
Storage Driver: btrfs
 Build Version: Btrfs v4.4
 Library Version: 101
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: btrfs local
 Network: bridge host macvlan null overlay
 Log: awslogs fluentd gcplogs gelf journald json-file logentries splunk syslog
Swarm: active
 NodeID: bonsq8mqqpr431xy0tmcujp8h
 Is Manager: true
 ClusterID: 2yg8s8m7g1tbow00nrhst4l3f
 Managers: 1
 Nodes: 1
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Number of Old Snapshots to Retain: 0
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
  Force Rotate: 0
 Autolock Managers: false
 Root Rotation In Progress: false
 Node Address: <IP_ADDRESS>
 Manager Addresses:
  <IP_ADDRESS>:2377
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 89623f28b87a6004d4b785663257362d1658a729
runc version: b2567b37d7b75eb4cf325b77297b140ea686ce8f
init version: 949e6fa
Security Options:
 apparmor
 seccomp
  Profile: default
Kernel Version: 4.4.0-109-generic
Operating System: Ubuntu 16.04.3 LTS
OSType: linux
Architecture: x86_64
CPUs: 4
Total Memory: 11.64GiB
Name: localhost
ID: YPLV:BNYY:3RJN:S7WD:L2G7:RQ4T:YJYJ:KCHT:UIWR:UXDP:Z4AR:OSQO
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Username: mochoa
Registry: https://index.docker.io/v1/
Labels:
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

any hint to solve this problem will be great, I am using 12cR2 for a long time on this environment, this is new and I can't find on the web what the message:
[FATAL] [DBT-08001] Unable to check for available memory.
[FATAL] [DBT-08002] Unable to check the value of kernel parameter {0}

means and how to fix that. Marcelo.

Note to myself: The scripts should validate the host name for invalid characters and prevent the creation of a container with such invalid host name.


Potential race condition with initialized handler

So @rjmholt and I have been debugging an issue with the PowerShell extension after updating to v3.19, and we believe have tracked it down to a race condition. But we don't know if this is an issue with OmniSharp, or maybe even with the LSP spec itself.
We see that the Initialize request handler, for LSP2, sets Initialized:
https://github.com/OmniSharp/csharp-language-server-protocol/blob/005c87f1c41da3f1b2b35f32c98e3d3132df99a3/src/Server/LanguageServer.cs#L316-L322
This made sense, because Initialize was a request and so the client can (and does) wait to hear from the server that Initialize was handled. When OmniSharp calls that initialized() method (for LSP2) then GetRequests() starts behaving "correctly" (handling all notifications and requests):
https://github.com/OmniSharp/csharp-language-server-protocol/blob/1b6788df2600083c28811913a221ccac7b1d72c9/src/Server/LspServerReceiver.cs#L20-L54
But then this was changed for LSP3, we have an Initialized (note the D) handler for the Initialized notification (which the client cannot wait on). This is problematic because the client then sends more notifications which race the server's handler for the Initialized notification, and this race ends up in the requests not being handled, as evidenced by the responses "Unexpected request" and "Unexpected notification" in GetRequests.
https://github.com/OmniSharp/csharp-language-server-protocol/blob/005c87f1c41da3f1b2b35f32c98e3d3132df99a3/src/Server/LanguageServer.cs#L327-L337
This correlates with the logs we have when we see a bug in the PowerShell extension:
2021-05-06 11:41:07.210 -07:00 [DBG] Finding descriptors for initialize
2021-05-06 11:41:07.382 -07:00 [DBG] Queueing "Serial":initialize:0 request for processing
2021-05-06 11:41:07.428 -07:00 [DBG] Starting: Processing request initialize 0
2021-05-06 11:41:07.444 -07:00 [DBG] Starting: Routing Request (0) initialize
2021-05-06 11:41:08.131 -07:00 [DBG] Finished: Routing Request (0) initialize in 687ms
2021-05-06 11:41:08.131 -07:00 [DBG] Finished: Processing request initialize 0 in 702ms
2021-05-06 11:41:08.211 -07:00 [DBG] Finding descriptors for initialized
2021-05-06 11:41:08.211 -07:00 [DBG] Queueing "Serial":initialized request for processing
2021-05-06 11:41:08.232 -07:00 [WRN] Unexpected notification workspace/didChangeConfiguration {"Method":"workspace/didChangeConfiguration","Params":[[[[[[[[[]],[[]]]],[[]],[[]],[[]],[[]],[[]],[[[[]],[[]]]],[[]],[[]],[[]],[[[[]]]],[[]],[[]],[[[[]],[[]]]],[[[[]],[[]]]],[[[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]]]],[[[[]],[[]],[[]],[[]],[[]]]],[[[[]]]],[[[[]],[[]],[[]],[[]],[[]]]],[[[[]],[[]],[[]]]],[[[[]],[[]]]]]],[[[[[[]]]],[[[[]],[[]],[[]],[[]],[[]]]],[[[[]],[[]]]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[[[]],[[]],[[]],[[]]]],[[]],[[]],[[]],[[]],[[]],[[[[]]]]]],[[[[[[]],[[]],[[]]]],[[]],[[]],[[]],[[]],[[]],[[[[]],[[]],[[null]]]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[]],[[[[]],[[]],[[]]]],[[]]]]]]],"TraceParent":null,"TraceState":null,"$type":"Notification"}
2021-05-06 11:41:08.233 -07:00 [WRN] Unexpected request powerShell/getVersion {"Id":1,"Method":"powerShell/getVersion","Params":null,"TraceParent":null,"TraceState":null,"$type":"Request"}
2021-05-06 11:41:08.234 -07:00 [WRN] Tried to send request or notification before initialization was completed and will be sent later {"Id":null,"Error":{"Code":-32002,"Data":null,"Message":"Server Not Initialized","$type":"ErrorMessage"},"Method":"powerShell/getVersion","$type":"ServerNotInitialized"}
2021-05-06 11:41:08.234 -07:00 [DBG] Starting: Processing notification initialized
2021-05-06 11:41:08.243 -07:00 [DBG] Starting: Routing Notification initialized
2021-05-06 11:41:08.366 -07:00 [DBG] Finished: Routing Notification initialized in 132ms
2021-05-06 11:41:08.366 -07:00 [DBG] Finished: Processing notification initialized in 141ms

We think this may be related to #556, @david-driscoll was there any update on your investigation into that issue? @majastrz does any of this make sense to you and sound related to your issue?

It could certainly explain my issue. I think I can add logging for the "Unexpected request" events in my repro. If I see that logged for legitimate LSP requests/notifications, it would confirm the hypothesis.

So the naming is wrong (more specifically it was never changed).  _initializeComplete == both the server and client are ready
What is happening is that in lsp v2, the server and client was assumed to be ready for everything immediately after the initialize request was finished.  With v3 the server must wait for initialized to be sent before responding to other requests.
Based on this language here.
As I'm typing this, it's entirely possible that I'm reading or understanding it wrong, and simply moving the _serverReceiver.Initialized(); to always happen at the end of initialize maybe the correct thing to do.

In addition there are two phases where you can add hooks.
Today there are three different hooks in the setup process (perhaps confusingly)

IOnLanguageServerInitialize and OnLanguageServerInitializeDelegate -- This allows you to manipulate client capabilities before they're read by the server
IOnLanguageServerInitialize and OnLanguageServerInitializedDelegate -- This allows you to manipulate the server capabilities before they're sent back to the client.
IOnLanguageServerStarted and OnLanguageServerStartedDelegate -- This is right after the initialized event is received.

If you have any logic in IOnLanguageServerInitialize or IOnLanguageServerInitialize that requires calls to the client, if you move it to IOnLanguageServerStarted things might start behaving better.

We initially were looking into our server -> client calls, but I think our issue is actually coming from a client call to the server.
Another example of a request we were seeing fail here was when we send a custom request at startup to query the PowerShell version being used, here. We were worried that this onReady() callback was the wrong place and that it was too early, so dove into the VSCode implementation to find out.
It turns out that onReady() is indeed only called after the initialize response is received and the initialized notification is sent, but the client can't wait for the notification to be received by the server (that doesn't make sense since it's a notification). So it can immediately also send another request or notification.
Essentially, we think there's some period of time in O# after the initialize response is sent but before the initialized notification is processed when the client is free to send messages to O# according to LSP, but O# will reject them. While O# is still processing the initialized notification, the client has already sent and O# has already received another notification and rejected it, even though O# has sent an initialize response permitting that behaviour from the client.

As I'm typing this, it's entirely possible that I'm reading or understanding it wrong, and simply moving the _serverReceiver.Initialized(); to always happen at the end of initialize maybe the correct thing to do.

I think this is the right move. The spec is a bit hard to understand, but first there's the Initialize request:

The initialize request is sent as the first request from the client to the server. If the server receives a request or notification before the initialize request it should error.

This is fine, our client (which just uses vscode-languageclient) doesn't send other requests at this point (we have some requests being sent by our server implementation, like to itself, and that confused us for a while, but they're being appropriately handled and aren't the cause of the bug).
Secondly, there's the Initialized notification:

The initialized notification is sent from the client to the server after the client received the result of the initialize request but before the client is sending any other request or notification to the server.

Ok, great, but this is annoying! Our client has sent initialized, a notification that it cannot wait on (in contrast to a request) and then is totally allowed to send any other request or notification, which it starts doing. But these subsequent requests and notifications are erroneously handled by LspServerReceiver.GetRequest() because of the race condition around _initialized, which is set in the server's handler for the Initialized request, and hence may still be false when the next notifications come in.
https://github.com/OmniSharp/csharp-language-server-protocol/blob/1b6788df2600083c28811913a221ccac7b1d72c9/src/Server/LspServerReceiver.cs#L22
In our case, we're depending on "Unexpected notification workspace/didChangeConfiguration" and "Unexpected request powerShell/getVersion" to actually be processed correctly, as they are not supposed to be unexpected since they were sent by the client after it sent its initialized notification.

v0.19.2-beta0002 worked perfectly with Bicep!

Yay!!! Awesome to hear @majastrz.


Tag a new release

https://github.com/mkitti/Napari.jl/commit/b836278698df9ef22a899a02de1934ba90ceb842 bumped the compatible OMETIFF versions, but a new version of Napari was never tagged so Napari v0.2.0 is holding OMETIFF.jl back.
ERROR: Unsatisfiable requirements detected for package OMETIFF [2d0ec36b]:
 OMETIFF [2d0ec36b] log:
 ├─possible versions are: 0.2.0-0.4.1 or uninstalled
 ├─restricted to versions 0.4 by an explicit requirement, leaving only versions 0.4.0-0.4.1
 └─restricted by compatibility requirements with Napari [ed46c112] to versions: 0.3.0-0.3.14 — no versions left
   └─Napari [ed46c112] log:
     ├─possible versions are: 0.1.0-0.2.0 or uninstalled
     └─restricted to versions 0.2.0 by an explicit requirement, leaving only versions 0.2.0

Do you mind registering the current main branch?

Done: https://github.com/mkitti/Napari.jl/releases/tag/v0.2.1

Pull request merged to the General registry: https://github.com/JuliaRegistries/General/pull/62748
Feel free to reopen if the issue is not solved.

thanks!


add shell.nix

I'm using NixOS full time at the minute. The changes to .gitignore are necessary. The shell.nix is not but I figured I'd put it up in case there are any other Nix-curious folks like me.

Theoretically, the Nix files and .DS_Store for Mac (as well as any other OS or editor specific config) should go to your global gitignore, this way you don't to ignore them per project: https://gist.github.com/subfuzion/db7f57fff2fb6998a16c :)


Connected Animation sample is wrong going (via back-button) from detail to gridview

Reduce window size so that the GridView is scrollable; scroll it to the bottom:

Click the lower-left image, which animates correctly to the details page.
Click the back button. The image animates to someplace below the fold -- where it's position would've been had the GridView not been scrolled. Once the animation completes, the image pops into its correct place again.

what build and SDK version are you using? I'm not able to repro it.

@shandilvarun : I just tried this with the latest bits, and it no longer repros. Feel free to close the bug. :-)

Ah, no. It works fine with XAML Connected Animation, but the bug repros when Custom Connected Animation is selected from the drop-down...

what build and SDK version are you on?

10.0.14393.0

Closing due to inactivity


Best Approach to Integration?

Hey there,
I created Pokelyzer, a database schema that supports doing advanced analytics on historical Pokemon Go data using GIS software (e.g. ArcGIS/QGIS) and BI tools like Tableau or PowerBI. You might have come across it in the Silph Road subreddit post: Finding Hotpots for Locally Rare Pokemon Using Tableau.
I recently built an endpoint for the PokemonGo-Map app to receive their webhook pushes: http://www.whackdata.com/2016/07/31/pokelyzer-webhook-integration-pokemongo-map/ I was wondering if you might consider adding something similar to your (if you haven't already - I didn't spot it)?
It looks like a really quick add on - maybe 15 lines of code total - and would be especially easy given yours is a fork of theirs.
If you do, keep me posted!

That would be great. Even more now that it's almost impossible to scan a decent area with a single worker.

I think this would be pretty cool too.

I agree. This could be very useful.

I don't see much point in creating a webhook mechanism - database is accessible from outside, you can poll for new changes (e.g. by storing last row ID and getting only new ones). Moreover, you'd also need to be prepared that X workers would simultaneously attack your app with hundreds of requests. Database (e.g. MySQL) is prepared for that, but is your app?
I think the best approach is to create a separate external process that polls for changes in database and does whatever you want with those - for example, pushes them to your app. I don't see any need why that process should be part of pokeminer. Pokeminer's main job is to put entries in the database.

Polling is an inefficient use of resources and doesn't scale well when you have more than one service requesting data - webooks were invented to solve that problem.
For example, one person that's using Pokelyzer right now is covering 100 distinct geographic areas, and each sends data as soon as it gets it into Pokelyzer, which ingests it immediately. The alternative - to have a service that polls 100 separate services every X seconds whether there is data or not - is not nearly as clean an approach.
The Pokelyzer webhook receiver runs server-side along with the applications that collect the data, so no ports are exposed to the public at all. If you chose to run them in separate sites, you could expose it to only the necessary IP addresses of your respective servers.
The reason why people build webhooks into their application is for precisely the thing you mentioned above: so that they can focus at what their good it. Pokeminer does a great job at gather data and storing it in a database for basic analysis. Pokelyzer focuses only on the latter part alone, allowing advanced BI and GIS tools to plug into it for doing a deeper dive for people with that skill set. Building a webhook interface allows you to make your product better, while also allowing it to integrate with tools like mine with no development effort on your behalf after the initial creation of a webhook.

If webhooks are out of the question could an external db option with extra fields be implemented?
Unfortunately pokeminer right now barely collect enough information on its on db. That's the reasons that I'm running 18 pokemaps with pokelyzer.

@Brideau I see the point of integrating with app without touching the code. I still don't think that webhooks are a must-have feature - I designed pokeminer so that data is easily accessible from outside via database. I will add it to my todo list, but it's a low priority. Nonetheless if you want to implement said 15 lines of code, feel free to open a pull request - if it fits into the rest of pokeminer I'll be happy to merge it.
@ferazambuja what information besides EncounterID is not stored?

Sorry, I was a bit dramatic :)  EncounterID and Time2Hidden.
Since pokeminer intent is to collect a large area and data, this two are crucial for later analysis.
Brideau how about a pokeminer database translator uploader meanwhile? With this two extra columns, you have all the info you need?

@modrzew Sounds good, I'll take a look into doing a pull request once I've cleaned up a few more things with my project.
@ferazambuja That could be useful as well, and would be a good stop-gap measure in the meantime. I'll have a look.

@ferazambuja Time2Hidden and expire_timestamp are the same information in 2 formats, aren't they? So the only thing really missing is EncounterID. But I've seen someone saying that the same Pokemon spawning at the same location at different times could have the same EncounterID, so I was reluctant on building database around it.

Brideau probably can expand a bit more on the time2hidden. PGO-mapscan-opt deals with them separate.
I never experienced the duplicated encounterID and the only way for us to know is if this data is saved.

"Time to hidden" is the number of milliseconds until the pokemon disappears, while expire timestamp is the present time plus "time to hidden".

Time to hidden decreases with time, expire timestamp is constant (there may be minor fluctuations ranging a few ms because of short delay between moment when you generate current timestamp and moment when you store value in the database, but I wrote workarounds for that). My question is: why would you want to store value that decreases over time in database? Especially when it contains exactly the same information as expire_timestamp, but in different format. IMO that's just asking for a headache related to duplicated data.

Yeah the approach we're taking now is to just update the past record with the newest one if a duplicate is detected, but in some cases it could be useful to have that data. If you're trying to find out if a rare Pokemon spawns for a long time, for example, you could hunt for the max value of time to hidden for a single spawn. But that's not exactly a safe default if the user isn't fully aware of how the data is stored or how to properly query it.

@modrzew I just created a pull request with this addition: https://github.com/modrzew/pokeminer/pull/162

Moving the discussion to #162, then.


Updated fix: remove_fill_values()

Added import of cf_xarray and detection of bounds for each
coordinate. And then sets _FillValue to None to avoid writing
bounds with _FillValue to netcdf.

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)

This PR fixes issue #xyz

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Documentation has been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated (with summary of main changes)
[ ] I have added my relevant user information to AUTHORS.md

What kind of change does this PR introduce?:

Does this PR introduce a breaking change?:

Other information:

Hi @ellesmith88, please check this PR that hopefully makes sure the coordinate bounds variables do not have _FillValue = NaN in output files (once fixed). Thanks

@agstephens I recently added this fix into clisops https://github.com/roocs/clisops/pull/204/files which fixes the fill values for the coordinate variables bounds in the output files. I think we could probably remove the fill value fix in daops completely, apart from the fact that it is currently used in the decadal fixes, as I implemented it as a fix before we decided to put it all in clisops.
If you want to add the bounds fix here as well then I think it needs to be something like:
try:
     bounds = ds.cf.get_bounds(coord_id)
     bounds.encoding["_FillValue"] = None
except KeyError:
     continue

as a KeyError is raised, rather than returning None, if the coordinate doesn't have any bounds. I tested with the file /badc/cmip6/data/CMIP6/CMIP/IPSL/IPSL-CM6A-LR/historical/r1i1p1f1/Amon/rlds/gr/v20180803/rlds_Amon_IPSL-CM6A-LR_historical_r1i1p1f1_gr_185001-201412.nc
Also, cf-xarray would need to be added to the requirements.
I can make these changes if you want to keep this in daops.

@agstephens I recently added this fix into clisops https://github.com/roocs/clisops/pull/204/files which fixes the fill values for the coordinate variables bounds in the output files. I think we could probably remove the fill value fix in daops completely, apart from the fact that it is currently used in the decadal fixes, as I implemented it as a fix before we decided to put it all in clisops.
If you want to add the bounds fix here as well then I think it needs to be something like:
try:
     bounds = ds.cf.get_bounds(coord_id)
     bounds.encoding["_FillValue"] = None
except KeyError:
     continue

as a KeyError is raised, rather than returning None, if the coordinate doesn't have any bounds. I tested with the file /badc/cmip6/data/CMIP6/CMIP/IPSL/IPSL-CM6A-LR/historical/r1i1p1f1/Amon/rlds/gr/v20180803/rlds_Amon_IPSL-CM6A-LR_historical_r1i1p1f1_gr_185001-201412.nc
Also, cf-xarray would need to be added to the requirements.
I can make these changes if you want to keep this in daops.

@ellesmith88 Your proposal sounds most sensible - that we just remove this as a dachar/daops fix because it is applied at the clisops level. Let's just fix it at source (in clisops) and keep everything else simpler.


steam-run: add derivation

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

More
This adds a new steam-run script, which gives one ability to run something with our assembled Steam Runtime in environment. This should come very handy for all kinds of games downloaded from various places (GOG, for example). Steam Runtime is the de facto expected Linux userspace for commercial games, so this should make running them much more easy.
Example:
> steam-run ./run-game.sh

This also makes hopefully small but incompatible change to userFHSEnv, renaming importMeta argument to meta (a more standard name). userFHSEnv is mostly an internal function used by buildFHSUserEnv, so this should not be a problem.
Merging this in few days unless there are any remarks.

Can you give an example to test?

I cannot give you a game itself ~_^ -- but if you have anything downloaded from GOG, Humble Bundle and others who distribute stand-alone Linux games, running steam-run ./start-the-game (where ./start-the-game depends on the actual game) should make it Just Work.

Ok, got it!


It doesn't seem to finish all tasks if the queue is generated in a for loop

I'm using the p-limit for limit the concurrence for file writing (about 6k files). The code below works fine:
`
var writeFilePLimit = pLimit(100);
var writeFileQueue = [];
Object.keys(allData).forEach(data => {
  writeFileQueue.push(writeFilePLimit(async () => {
    await fs.promises.writeFile(path, JSON.stringify(data));
  }));
});

`
However, if I do
`
var writeFilePLimit = pLimit(100);
var writeFileQueue = [];
for (var data in allData) {
  writeFileQueue.push(writeFilePLimit(async () => {
    await fs.promises.writeFile(path, JSON.stringify(data));
  }));
}

`
then it will only execute 102 files writes (out of 6k files in total). And the 102 files seems to be the first 100 files and the last two files.
I'm not sure whether there is some difference between for loop and Array.forEach() resulted in this. Just want to post this here to see whether I did something wrong.
Thanks!

It would be helpful if you would be able to create a minimal runnable example that reproduces the issue. I can't really do much with just some random code examples.
Or even better, submit a pull request with one or more failing tests.

actually u are relying in clousure property which may not work as expected. have u tried this
for (var data in allData) {
  writeFileQueue.push(writeFilePLimit((p,a) =>
fs.promises.writeFile(p, JSON.stringify(a)),
	path,data));
}


Mark as used if the private class is accessed in classOf

Minimal Reproduciable code
object `classof something` {
  private class intrinsically // shouldn't be asserted here
  def f = classOf[intrinsically].toString()
}

Expected behavior
\

upstream fix needed https://github.com/scalameta/scalameta/issues/1909

https://github.com/tanishiking/scalafix-unused/commit/d14642eaf25458a98c50a0569dba58e9c0329524


ERC-20 Upgradable Smart Contract

Description
Added ERC-20 Upgradable Smart Contract Tutorial and Project Files.
Fixes #140
Type of change
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this :

[x] Add a new project.

Domain of Contribution
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Project using Solidity and Hardhat.

Checklist ✓
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] l have performed a self-review of my own code or work.

[x] My changes generate no new warnings

[x] I'm HSOC'22 contributor.

Screenshots/Gif (Recommended)

@Envoy-VC  please  include  working video in the readme along  with the intro of the project ,

Video will be done as soon as possible, what kind of intro are we looking for?

Video will be done as soon as possible, what kind of intro are we looking for?

Intro about this project , Which probelm is being solved by this project and add all this at the top under the heading "About"
If already exsist make sure that is of 6-7 lines and also  under the "About" .
Hope u understand

@Vikash-8090-Yadav Hey Can you check now, I have done things as you said,

Excellent .Your code looks great and nice.You have written the clean code with comment s which makes user to better understand .
Thanks for contributing in my project
.Happy open source ☺


implementing P.840-9

Addressing Issue #81 P.840-9 update. Updates data set and specific attenuation through clouds at a slant path. Eq. 14 in P.840-9. Showing ~10^-6 error from validation sets.

There is some issue with including the gaseous attenuation component for plotting.  Data has a lot of nans and It only prints the right side of the plot.

import itur
f = 22.5 * itur.u.GHz    # Link frequency
D = 1 * itur.u.m         # Size of the receiver antenna
el = 60                  # Elevation angle constant of 60 degrees
p = 3                    # Percentage of time that attenuation values are exceeded.
Generate a regular grid latitude and longitude points with 1 degrees resolution
lat, lon = itur.utils.regular_lat_lon_grid(resolution_lat=5, resolution_lon=5, lon_start_0=False)
Comute the atmospheric attenuation
Att = itur.atmospheric_attenuation_slant_path(lat, lon, f, el, p, D)
itur.plotting.plot_in_map(Att.value, lat, lon,
cbar_text='Atmospheric attenuation [dB]')

The interpolator is having a hard time with the data orientation. I’m going to reshape the data and re-upload.


NICFI imagery panel is not working

NICFI imagery panel is not working, seems like frozen, not moving imagery, when trying to change to different date.

it was frozen by the dates (1011-05-30)  is not a valid date to request in Planet.
This should be solved by my latest changes as the dates default to 1970-01-01 when they are badly shaped.


Empty keys not allowed

The Base API doesn't allow you to put dictionaries with empty keys (""). Currently, the progress on this is the false traceback modified by #76 so it returns None instead of producing a KeyError.

Thanks, @hdadhich01 The #76 PR has been merged.


Average multiple sensors?

Related to #67 - Since we don't start off having even 2 minutes of data, I've been thinking about whether we might want to consider finding the 3 nearest sensors (ideally in different directions) and averaging them.

Not a bad idea! I'm hesitant to add things we can't link to as verification to avoid confusion, but maybe if we add some kinda settings and let folks toggle this on/off?
Also, we'd definitely want some limit on distance. In the Bay Area, due to our unfortunate history with wildfire smoke, the PurpleAir sensor network is rather dense. However, other places may not share that same density and it may not be as helpful to average together a sensor right next to someone and one that's very far away.


Cannot build 4swordshax

When trying to build, I get this:
sudo mount -o loop 4B513945hax.savedata 4swordshax mount: mount point 4swordshax does not exist Makefile:28: recipe for target 'all.dat' failed make: *** [all.dat] Error 32
It only outputs 4B513945hax.savedata, 4swordshax.elf, and all.dat.

You must create the 4swordshax directory it tried to use.

Thanks! It worked, but where do I get the DSi SD key? (probably console-specific)

It's included with a certain public tool but it could also be determined with 3DS ITCM data.

So it's not console-specific? In that case, I can just grab it from my 3DS ITCM. Where is it located?


How to do not show language index in url for default language

Hi @radek-altof, this plugin saved my day!
I'm trying to figure out how to set the plugin not to show the default language in the url, this option would be handy for SEO.
// routerOptions log
// this is my default language
{
    "_value": "/it/"
}

When i try to catch the value of currentRoute it obviously  '/' path and i can't do any redirect condition. Is there a way to remove language index from url?
Referenced issue #14 i figured out the path in the url is necessary, but if i make a normal a tag for default language, can i remove safely that index since the language has been set correctly?

I solved it for now!
here src/plugin/index.js
i replace with:
-- export { createLangRouter, i18n };
++ export { createLangRouter, i18n, setLanguage };

in my component i make condition to catch my default language:
<language-switcher v-slot="{ links }" class="language-switch" >
  <div v-for="link in links" :key="link.langIndex">
    <router-link :to="link.url" v-if="link.langIndex != 'it'">

        <img :src="`/images/flags/${link.langIndex}.webp`" alt="" width="20" />
    </router-link>
    <a v-else :href="link.url.replace(/\/it\//g,'/')" @click="setLanguage('it')">

      <img :src="`/images/flags/${link.langIndex}.webp`" alt="" width="20" />
    </a>
  </div>
</language-switcher>
...
import { setLanguage } from '../plugin';
...

This way it works, but i don't know how it works with history. In my case i don't use history.


[Release] sdk/resourcemanager/windowsiot/armwindowsiot/0.1.0

…on from spec commit: c09b26d4aaa40fb37380aae01b7019b75cff3e34

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/azp run prepare-pipelines

/azp run go - armwindowsiot - ci

/azp run go - armwindowsiot - ci

/check-enforcer override


xWindowsOptionalFeatureSet.schema.psm1: Fix Custom DSC Resource Kit PSSA rule violations

Details of the scenario you tried and the problem that is occurring
There's a number of script analyzer issues getting reported in xWindowsOptionalFeatureSet.schema.psm1. These should be fixed.
Verbose logs showing the problem
[00:05:21]     Context xWindowsOptionalFeatureSet.schema.psm1
[00:05:21]       [+] Should pass all error-level PS Script Analyzer rules 73ms
[00:05:21]       [+] Should pass all required PS Script Analyzer rules 121ms
[00:05:21]       [+] Should pass all flagged PS Script Analyzer rules 61ms
[00:05:21]       [+] Should pass any recently-added, error-level PS Script Analyzer rules 63ms
[00:05:22]       [+] Should not suppress any required PS Script Analyzer rules 119ms
[00:05:22] Custom DSC Resource Kit PSSA rule(s) did not pass.
[00:05:22] The following PSScriptAnalyzer rule 'DscResource.AnalyzerRules\Measure-ParameterBlockParameterAttribute' errors need to be fixed:
[00:05:22] xWindowsOptionalFeatureSet.schema.psm1 (Line 51): A [Parameter()] attribute must be the first attribute of each parameter and be on its own line. See https://github.com/PowerShell/DscResources/blob/master/StyleGuidelines.md#correct-format-for-parameter-block
[00:05:22] xWindowsOptionalFeatureSet.schema.psm1 (Line 54): A [Parameter()] attribute must be the first attribute of each parameter and be on its own line. See https://github.com/PowerShell/DscResources/blob/master/StyleGuidelines.md#correct-format-for-parameter-block
[00:05:22] xWindowsOptionalFeatureSet.schema.psm1 (Line 57): A [Parameter()] attribute must be the first attribute of each parameter and be on its own line. See https://github.com/PowerShell/DscResources/blob/master/StyleGuidelines.md#correct-format-for-parameter-block
[00:05:22] xWindowsOptionalFeatureSet.schema.psm1 (Line 61): A [Parameter()] attribute must be the first attribute of each parameter and be on its own line. See https://github.com/PowerShell/DscResources/blob/master/StyleGuidelines.md#correct-format-for-parameter-block
[00:05:22] For instructions on how to run PSScriptAnalyzer on your own machine, please go to https://github.com/powershell/PSScriptAnalyzer
[00:05:22] Common Tests - Custom Script Analyzer Rules will not fail unless you opt-in.
[00:05:22] To opt-in, create a '.MetaTestOptIn.json' at the root
[00:05:22] of the repo in the following format:
[00:05:22] [
[00:05:22]      "Common Tests - Custom Script Analyzer Rules"
[00:05:22] ]
[00:05:22]       [+] Should pass all custom DSC Resource Kit PSSA rules 656ms

Version of the DSC module that was used ('dev' if using current dev branch)
dev

I got this.


Update Introduction.md

Updated the docs to use classes and ReactDOM.
This allows the user to get the most updated approached to build React with react-router

looks good, thanks. the only change I'd like you to do is to follow this style:
import { render } from 'react-dom'

@taion yeah that's true, I use classes that's why it seemed correct for me. Difficult to convince @mjackson and @ryanflorence to classes.
But I think at least for now let's keep the React.createClass as we use mixins here and there.

Hipsters :p
Probably for this PR, switch everything back to React.createClass for consistency then.

:+1:

I think this change is going to come sooner or later, IMO. The ReactDOM part has already happened, and since everything is going towards ES6 and babel we maybe we can show both ES5 and ES6 examples but let people know that ES6 is what is going to be prevalent

@stanleycyang we might do that later, if you want to add the react 0.14 bits and pieces please add the changes @taion asked for. Thanks :smile:

Sooner or later, sure, but most likely not until there's a broadly accepted replacement for mixins - HoCs aren't sufficient yet for things that hook into lifecycle in some cases, unfortunately.

Updated it to account for import { render } from 'react-dom' per @knowbody

Thanks!

thanks @stanleycyang


Group rename - Save renamed group before syncing with storage

When using the newly introduced --update-parent-lists flag in the group rename command, it had the catastrophically undesired side effect of refreshing the new/renamed group from storage, discarding every permission that was added after the group creation.
The primary change is really in GroupRename, but to ensure that the bulk operation + syncing happens after the group is saved, I ended up adapting the StorageAdapter to return the messaging service request's future for continuation purposes.

LGTM, thanks!


Add build error textual reason to slack messaging

Slack messager informs textually build failure reason. Pylinted (pep8)

Created separate PR with separate commit actions. Closing this one


 should be stripped if there is none

🐛 Bug Report
When creating buttons for our application, we do something like:
<a title="Follow">
   <SvgIcon />
</a>

The above causes the output svg to contain a blank title (<title></title>), which in Chrome, stops the tooltip from showing up.
What if we completely stripped the title tag when it is undefined?
To Reproduce
Start a project with create-react-app, import an svg component and use it without a title attribute.

Hello @inssein, yes we can completely strip the tag if it causes some unexpected bug like yours. Could you submit a PR to strip it if not needed?

@inssein , @neoziro I have created a PR. Please have a look.


podman runtime specify youki create container Error

version
Linux version 3.10.0-957.el7.x86_64
gcc version 4.8.5 20150623 (Red Hat 4.8.5-36) (GCC) ) #1 SMP Thu Nov 8 23:39:32 UTC 2018
podman version 1.6.4
youki version 0.0.1
commit: 0.0.1-0-91b7632
..............................................................................................................
[root@localhost local]# podman --runtime /usr/local/youki run -d --name niginx -p 80:80 xx.xxxxx.com/nginx
Error: container 53ae3c4eefef6b261f9dcf9d361dbdab11e5df675841320dd8214cc0239a2dd7 does not exist.
ERRO[0000] Error removing container 53ae3c4eefef6b261f9dcf9d361dbdab11e5df675841320dd8214cc0239a2dd7 from runtime after creation failed
Error: [WARN crates/libcontainer/src/rootfs/mount.rs:253] 2022-01-04T22:50:46.267528492-05:00 Could not mount "net_cls,net_prio" cgroup subsystem
[WARN crates/libcontainer/src/rootfs/mount.rs:253] 2022-01-04T22:50:46.269389430-05:00 Could not mount "cpu,cpuacct" cgroup subsystem
[WARN crates/libcontainer/src/process/container_init_process.rs:140] 2022-01-04T22:50:46.284662733-05:00 masked path "/proc/latency_stats" not exist
[WARN crates/libcontainer/src/syscall/linux.rs:139] 2022-01-04T22:50:46.315288377-05:00 CAP_PERFMON is not supported.
[WARN crates/libcontainer/src/syscall/linux.rs:139] 2022-01-04T22:50:46.315326696-05:00 CAP_BPF is not supported.
Error: failed to create container
Caused by:
0: failed to wait for init ready
1: failed to wait for init ready
2: channel connection broken: OCI runtime error
..............................................................................................................
tks much.

@wyaow Can you post the output of youki info?

[root@localhost local]# youki info
Version           0.0.1
Kernel-Release    3.10.0-957.el7.x86_64
Kernel-Version    #1 SMP Thu Nov 8 23:39:32 UTC 2018
Architecture      x86_64
Operating System  CentOS Linux 7 (Core)
Cores             4
Total Memory      7821
Cgroup setup      legacy
Cgroup mounts
blkio           /sys/fs/cgroup/blkio
cpu             /sys/fs/cgroup/cpu,cpuacct
cpuacct         /sys/fs/cgroup/cpu,cpuacct
cpuset          /sys/fs/cgroup/cpuset
devices         /sys/fs/cgroup/devices
freezer         /sys/fs/cgroup/freezer
hugetlb         /sys/fs/cgroup/hugetlb
memory          /sys/fs/cgroup/memory
net_cls         /sys/fs/cgroup/net_cls,net_prio
net_prio        /sys/fs/cgroup/net_cls,net_prio
perf_event      /sys/fs/cgroup/perf_event
pids            /sys/fs/cgroup/pids
Namespaces        enabled
mount           enabled
uts             enabled
ipc             enabled
user            enabled
pid             enabled
network         enabled
cgroup          enabled

@wyaow I don't believe CentOS 7 satisfies the appropriate version for libseccomp either.
You can check with this simple script to see if you have the appropriate version of libseccomp: pkg-config --atleast-version=2.5 libseccomp && echo "good" || echo "bad"

Here is a good overview of packaging for libseccomp for various OSes: https://repology.org/project/libseccomp/versions

@wyaow Youki needs a linux kernel version of 5.4 or later 😭

@utam0k tks you anyway, I guess it would be better to note on README.
@tsturzl actually i tried it on an other Centos8 (Linux Kerner of 4.18 with libseccomp.so.2.5.1) os, it was same.


Merging autorest generated updates into 1.0 preview

Can one of the admins verify this patch?

Codecov Report

Merging #2120 into keyvault_1.0_preview will decrease coverage by 0.19%.
The diff coverage is 56%.

@@                   Coverage Diff                   @@
##           keyvault_1.0_preview    #2120     +/-   ##
=======================================================
- Coverage                 53.09%    52.9%   -0.2%
=======================================================
  Files                      4775     4798     +23
  Lines                    119851   120947   +1096
=======================================================
+ Hits                      63633    63982    +349
- Misses                    56218    56965    +747

Impacted Files
Coverage Δ

...e/keyvault/models/certificate_create_parameters.py
100% <ø> (ø)
:arrow_up:

...dels/pending_certificate_signing_request_result.py
71.42% <ø> (ø)
:arrow_up:

...ault/azure/keyvault/models/organization_details.py
100% <ø> (ø)
:arrow_up:

...keyvault/azure/keyvault/models/certificate_item.py
100% <ø> (ø)
:arrow_up:

...yvault/azure/keyvault/models/issuer_credentials.py
100% <ø> (ø)
:arrow_up:

...lt/azure/keyvault/models/certificate_attributes.py
100% <ø> (ø)
:arrow_up:

...t/models/certificate_operation_update_parameter.py
100% <ø> (ø)
:arrow_up:

...e-keyvault/azure/keyvault/models/key_properties.py
100% <ø> (ø)
:arrow_up:

...vault/azure/mgmt/keyvault/models/resource_paged.py
100% <ø> (ø)
:arrow_up:

.../keyvault/models/deleted_certificate_item_paged.py
100% <ø> (ø)
:arrow_up:

... and 148 more

Continue to review full report at Codecov.

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


sap.ui.unified.ColorPicker: dragging color selector leaves artifacts in Edge

OpenUI5 version: latest
Browser/version (+device/version): Edge 41.16299.1004.0
Any other tested browsers/devices(OK/FAIL): Chrome, Firefox OK
URL (minimal example if possible): https://sapui5.hana.ondemand.com/1.60.1/#/sample/sap.ui.unified.sample.ColorPicker/preview
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:

Bring up the sample in Edge browser
Use the browser zoom settings to set to 150%
Use the mouse to drag around the color selection cursor in the color square

What is the expected result? No artifact lines when dragging
What happens instead? When dragging, you'll see artifact lines in the color square. This only seems to happen at 150% zoom.

Any other information? (attach screenshot if possible)

Hi @cmastrandrea ,
Thank you for sharing this finding. I've created an internal incident:<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.
Regards,
Thodd

Hi,
The issue was fixed.
The fix will be available in version 1.71 of SAPUI5.
Kind Regards,
Nikolay


DP-1015 customizable search in cell provider

@katepol @timzam After discussion with Gleb we concluded that there should be a direct CellProvider test.
Also I hit an overloading problem: call getCells(List) gives unexpected result due to interpretation of list as a path. This commit also includes a possible renaming solutions for that problem, I propose to delete @Deprecated methods.

Looks good to me.

@timzam done!

@katepol done.


Add Github Action for testing on Windows

See https://github.com/Senzing/template-go/issues/90

Pull requests:

https://github.com/Senzing/observe/pull/42


about  feauture memory and GFLOPS.

For present 18:
Flops: 2 GFLOPs, feature memory=3GB

For mcn-mobilenet:
Flops: 579 MFLOPs, feature memory=5GB

And the question is why mobilenet get smaller flops, but with larger feature  memory. What's the relationship between FLOPS and feature memory?

MobileNet makes heavy use of group convolutions, which restrict the interactions between channels.  These significantly reduce flops, but not memory usage.

Thanks.


Specs metadata

This improves the test coverage of the specs package and introduces a meta object allowing metadata being attached without having to modify the structs.

Codecov Report

Merging #98 into master will increase coverage by 1.89%.
The diff coverage is 97.63%.

@@            Coverage Diff             @@
##           master      #98      +/-   ##
==========================================
+ Coverage   80.93%   82.82%   +1.89%
==========================================
  Files          54       59       +5
  Lines        3760     3919     +159
==========================================
+ Hits         3043     3246     +203
+ Misses        456      420      -36
+ Partials      261      253       -8

Impacted Files
Coverage Δ

pkg/dependencies/references.go
71.87% <75.00%> (-0.86%)
:arrow_down:

pkg/providers/hcl/specs.go
76.21% <78.57%> (-0.16%)
:arrow_down:

middleware.go
92.59% <100.00%> (+0.18%)
:arrow_up:

options.go
94.54% <100.00%> (ø)

pkg/checks/checks.go
100.00% <100.00%> (+3.33%)
:arrow_up:

pkg/dependencies/dependencies.go
85.29% <100.00%> (+3.24%)
:arrow_up:

pkg/functions/functions.go
92.90% <100.00%> (+13.16%)
:arrow_up:

pkg/functions/references.go
100.00% <100.00%> (ø)

pkg/lookup/lookup.go
91.60% <100.00%> (+8.50%)
:arrow_up:

pkg/providers/hcl/errors.go
85.71% <100.00%> (ø)

... and 22 more


Strange message in recommendation request

Hi,
I've tried to execute the demo script in order to integrate new evaluation logic and I've seen the following INFO msg on the logs:
INFO     [bg:reco-manager] 2015-07-09 12:20:34,121 (PollableSourceRunner-KafkaSource-r1) [INFO - eu.crowdrec.flume.plugins.interceptor.IdomaarRecommendationInterceptor.intercept(IdomaarRecommendationInterceptor.java:80)] Requesting recommendation for event [recommendation	44<PHONE_NUMBER>	{"reclen":5}	{"subject":"user:16"}\t757365720931360931343336343434343238373838097B7D097B227265636C656E223A357D095B7B226964223A2231393037363638222C22726174696E67223A22372E30222C2272616E6B223A2231227D5D]
it seams quite strange the string after the \t, do you know what is generating that?

This is what ZFrame.toString generates, see the toString method at https://github.com/zeromq/jeromq/blob/master/src/main/java/org/zeromq/ZFrame.java . I've now changed the .toString call to a better way of turning the ZFrame into string.


ami_block_device_mappings support

Is the plugin supports ami_block_device_mappings settings?
I want to create an instance with root device larger from the default.  What is the best practice for my use case?
Thanks,
Liora

Hi @lioramilbaum,
Currently plugin doesn't support ami_block_device_mappings settings.
For creating the root device larger than the default, probably the image will have to be created as per that and then using that image, create an instance.


React component naming should be consistent

[ ] Add Page suffix for every component with the title, e.g. SettingsPage etc.
[ ] Index is just a page. Rename App.react to Page.react?
[ ] Check whether filename is the same as className

@steida Does that seem legit to you?
[createRoutes.js]

import App from './app/Page.react';
import Auth from './auth/AuthPage.react';
import Firebase from './firebase/FirebasePage.react';
import Home from './home/HomePage.react';
import Intl from './intl/IntlPage.react';
import Me from './me/MePage.react';
import NotFound from './notfound/NotFoundPage.react';
import Profile from './me/ProfilePage.react';
import React from 'react';
import Settings from './me/SettingsPage.react';
import Todos from './todos/TodosPage.react';
import { IndexRoute, Route } from 'react-router';

export default function createRoutes(getState) {
  ...

  return (
    <Route component={App} path="/">
      <IndexRoute component={Home} />
      <Route component={Auth} path="login" />
      <Route component={Intl} path="intl" />
      <Route component={Firebase} path="firebase" />
      <Route component={Me} onEnter={requireAuth} path="me">
        <Route component={Profile} path="profile" />
        <Route component={Settings} path="settings" />
      </Route>
      <Route component={Todos} path="todos" />
      <Route component={NotFound} path="*" />
    </Route>
  );
}

... or do you want to import as:
import App from './app/Page.react';
import AuthPage from './auth/AuthPage.react';
import FirebasePage from './firebase/FirebasePage.react';
import HomePage from './home/HomePage.react';
...

Yes, exactly. No, import as Auth etc. is good enough.
"Check whether filename is the same as className" means whether class name is the same as containing file.

Ok, ok. Thanks for the feedback.
I will probably find some time to finish up the PR by the end of the week-end.

Take your time. I am just updating air-bnb eslint.


0365 cdn Invalid Referer

Hi @waldekmastykarz
Hope you are doing good!
I have configured cdn to my tenant and when i am trying to access the file from the url im getting below error:
Could you please help in this? Image attached for reference.

Many Thanks

This is by design as mentioned in the documentation at https://dev.office.com/blogs/office-365-public-cdn-developer-preview-release:

Notice. You can’t use a browser tab to test the URL returned by this API, due to the referer limitations in the CDN. You can set this URL to an asset hosted in SharePoint to validate the URL and that everything works properly

https://contoso.sharepoint.com/ _vti_bin/publiccdn.ashx/url?itemurl=https://contoso.sharepoint.com/siteassets/folder1/folder2/image.jpg
> Output: https://publiccdn.sharepointonline.com/contoso.sharepoint.com/1937006161eeae8fe.../folder2/image.jpg

@waldekmastykarz
Yes when i have referred the same file from the sharepoint page it is working fine. Also i have observed that few files are not fetched and few files are fetched properly.
Can you suggest that using reference from public cdn will improve the performance?
Thanks
Sid

That depends on which CDN you would use and how it is configured. Whether the performance of that CDN is better than the Office 365 public CDN is something you can only determine by verifying the performance from the same network as used by the users working with the final solutions.

@waldekmastykarz
I have used the public cdn and reffered the files frompublic cdn, what i felt is that everytime for a file to be referred, it is waiting for public cdn which is still worse.

Hi @sidjustsid, referer requirements are there since Office 365 CDN is targeted to be used in the context of SharePoint Online, not to host assets which are hosted from any other location. You can find the GA level documentation around the URLs and behavior from following blog post.

https://dev.office.com/blogs/general-availability-of-office-365-cdn

Since this is by design in the context of the Office 365 public CDN, will close this issue for now. If you do however have any additional questions, we can absolutely re-open the issue or continue the discussion here. Feedback and comments are more than welcome.

@VesaJuvonen
Thanks for the Information.


Ensure that packet headers are written in LE

https://github.com/mafintosh/libudx/blob/615860d0c0cf1f6102d07b36364ac1f91434d9b7/src/udx.c#L91-L100

@mafintosh If we go with network byte order, we can use the standard socket conversion functions: https://man7.org/linux/man-pages/man3/byteorder.3.html

Ah yes, but we should go LE always. We do that across stack so making it BE one place doesn't really help and 99% of systems are LE anyway

Makes perfect sense 👍


Add ObjectRemovalDetection and ObjectPlacementDetection

I have an IPC-T5442T-ZE running on firmware V2.840.15OG008.0.R (Build Date: 2022-02-18). I wanted to use the "Smart Object Detection" feature via this integration to detect when packages were left and removed. Initially I tried to do this by selecting the TakenAwayDetection and LeftDetection events but that didn't do anything. After some digging I realized its because my camera fires events with codes ObjectRemovalDetection and ObjectPlacementDetection here.
I'm not sure if this changed in newer firmware or different models do different things so I didn't modify the existing events. But it would be good if these two could get added to the list so folks like myself could use them.

Thanks!

Released in https://github.com/rroller/dahua/releases/tag/0.9.41


Theming 2

Resolves, or at least addresses (subject to further feedback and reviews) #2, #4, #5, and #6 :)

Thanks! Merged manually.


Sync existing code with amcl

Rework macros to be ready for curves with 192 and 256 bit security levels
Improve internal arithmetic of big, ecp, fp, fp2, fp4 and fp12

Add apis for:
BIG - Bitwise or of two bigs
ECP - Multiplication by curve cofactor and curve generator initialization
ECP2 - Curve generator initialization
FP12 - Conditional copy
FP4 - Scalar multiplication by and FP and conditional copy

Coverage decreased (-0.7%) to 88.911% when pulling bbf653e5e12bb484bd895f1a81c731427cdd0cb1 on sync-amcl into c731d153f57bf61599f21ccf6e6cb97cd32f883c on master.


Customize the sendAuthorizationConsent in OAuth2DeviceVerificationEndpointFilter .

Expected Behavior
I want to trigger different behaviors (redirect or return JSON) through different parameters, and I'm not sure if this is feasible
Current Behavior
Context

Thanks for getting in touch, but it feels like this is a question that would be better suited to Stack Overflow. We prefer to use GitHub issues only for bugs and enhancements. Feel free to update this issue with a link to the re-posted question (so that other people can find it).