proposal: Go 2: negative slice/array indexes

Currently a negative index to a slice causes a panic.
This proposal doesn't want to change that behaviour.
...except at compiler level.
The compiler, when presented with a -x in the slice index pretends it is equivalent to len(s)-x.
It is essentially a macro (syntax sugar). It only applies when a human types it in code. It does not apply if you use an int variable of negative value.

v := m[-3]

// is equivalent to

v := m[len(m)-3]

This has been proposed in multiple different ways in the past. How does this proposal compare to the earlier issues? For example:
https://github.com/golang/go/issues/11245
https://github.com/golang/go/issues/16231
https://github.com/golang/go/issues/20176
I personally think that https://github.com/golang/go/issues/25594 was the best of the group, but it was ultimately closed by the author as it didn't seem like a clear win.

What are the constraints on the x in -x? Must it be an integer literal? What about (typed or untyped) constants?
What happens if x is an int with a negative value? (Consider https://play.golang.org/p/234hf3Youmr.)

You can't use a variable or constants.

Treating constants differently from literals seems very non-orthogonal.
Treating literals differently from variables seems surprising. Since the operation in question modifies the behavior of index expressions, I would expect it to be indicated by something that looks more like a variant of an index expression than an ordinary mathematical operator, especially for something that is merely syntactic sugar.

Here's an alternative to consider: if you want to reduce the stutter, consider the len([…]) expression to be elided.  So perhaps _ within an index expression could stand for len([…])?
x := someDescriptiveSliceName[_-3]

_ seems unnecessary to me.

This isn't purely at a compiler level because you can only know the value of an int at runtime. For instance:
func reversedIndex(slice []int, index int) int {
    return slice[-index] // we can't know if this is "-index" or "len(slice)-index" until runtime
}

Technicalities aside, I think that the necessity of having a special case for negative integers is a bit inconsistent, however I understand it's use because of backwards compatibility and out of necessity in order to flip the integer's sign before indexing normally into an array. But the necessary evil seems like an exception that makes the feature not feel... right.

huh? My proposal prohibits variables.

Sorry, I hadn't seen that. I thought I read the comments but didn't see that part.

The compiler, when presented with a -x in the slice index pretends it is equivalent to len(s)-x.
It is essentially a macro (syntax sugar). It only applies when a human types it in code. It does not apply if you use an int variable of negative value.

That part in the initial proposal seems to indicate that variables are allowed. Might want to reword that.

s[len-1]

I don't think that suggestion is a big enough advantage in the real world. The governing idea of my proposal is to reduce the cognitive processing involved when reading complex algorithms. Seeing a -1 "clicks" conceptually more seamlessly in my mind than seeing len-1.
Python is a language that regularly uses negative indices and it helps with readability (for me, immensely).

I don't know if that is a good idea but it would be a minor simplification and would avoid the non-orthogonality of treating negative constants specially.

I don't know much about the actual developer time to implement my proposal but I suspect it would be minimal to treat it as a special case.

@pjebs Actual developer time isn't a concern here.  The concern is lack of orthogonality in the language.  It's confusing to add a complex new concept: an expression that can be any non-negative value, or can be a negative constant, but cannot be a negative value that is not a constant.

@cespare I think that len with no argument would implicitly take as an argument the immediately surrounding index expression, without re-evaluating it.  It would be an error to use it outside of an index expression.  I suppose it should work with maps as well although it's hard to imagine people using it much.

@ianlancetaylor when you write s[len-1], isn't len part of a subtraction expression, not (immediately) inside an index expression?

Yes, by "immediately" I meant to describe what should happen with s1[s2[len - 1]].

The most common case here is probably len(s) - 1.  It's worth noting that if we had generics, we could have library functions like slices.Last(s) and slices.Pop(s) that would work with any slice type.  That might remove many of the cases where this is valuable.
I agree with @deanveloper that s[-1] does not click for me.

Here's an alternative to consider: if you want to reduce the stutter, consider the len([…]) expression to be elided.  So perhaps _ within an index expression could stand for len([…])?
x := someDescriptiveSliceName[_-3]

_ seems magic to me. Anything that makes me answer "it's magic" to a newcomer who asks me "what is this?" bothers me .
I prefer x[len-1] rather than x[_-1]

It seems that the current proposal is that len may appear in an index expression, without arguments or parentheses, to refer to the length of the innermost value (a slice, array, or map) being indexed.
The next question would be: how often does this case occur?  It should be possible to search some corpus of Go code for how often we see x[...len(x)...].  Anybody want to write a little program to find expressions of this sort?

@ianlancetaylor I wrote a quick regex hack in python to find single-line examples of this pattern. Found 799 examples in 379 files in the /src/ directory of this repo (master branch as of commit d2c039fb210429958bce6347a1e9d9a7b0ffd26d).
The command I ran was python path/to/lenslice.py $(find path/to/go/src -type f | grep "go$")
It only finds single-line examples of this and it can have some false-positives (although in a quick manual scan of the results I only found a couple of false positives, both as commented code). It may still help us get a general idea of how common (or not) this pattern is.
Here is the quick hack as a gist: https://gist.github.com/IanTayler/1db93fe558134c7b5f540ede13045725
And here are the results of running the command for find src -type f | grep "go$" files in this repo, as a pastebin: https://pastebin.com/8tK0bg2j

@IanTayler Thanks for doing that.
One thing I see right off the bat is that quite a few of the expressions have multiple len calls, such as the very first one which says lastRepeat[:len(lastRepeat)-len(after)].  With this proposal that would become lastRepeat[:len-len(after)].  Now I am much less comfortable with this idea.

@ianlancetaylor to me, that seems an argument in favor of @bcmills's _ proposal.
lastRepeat[:_-len(after)]

To me _ means "ignore the value", so I'm not sure I buy @bcmills 's suggestion that we should treat it as "elide the value."  I would be slightly happier with ..., which at least already stands in for the length when used as [...]int{}.

To me _ means "ignore the value", so I'm not sure I buy @bcmills 's suggestion that we should treat it as "elide the value." I would be slightly happier with ..., which at least already stands in for the length when used as [...]int{}.

Agreed with the first part that says _ means ignore in Go already. As I said before, _ is something that needs explanation to a newcomer.

Well, len still looks better to me than a symbol though I agree that ... would be more appropriate than _.
However, I still have doubts about whether it's worth doing anything here. In cases where a slice has a long name or its length is used more than once in a piece of code, I often cache the length in a simple variable and then use that instead. This is, in effect, what len (or a symbol) would be doing - albeit implicitly - but has the advantage that the variable can be used anywhere and not just in an index expression.
Also, as @ianlancetaylor intimated earlier, generic library functions would allow many of these cases to be expressed in a simple intuitive way. The example we're stuck on at the moment could be solved with  slices.Drop(lastRepeat, len(after).

That's a good point about '_' meaning “ignore” rather than “infer”. I agree that ... is a better fit.

If something like this goes in, I'm :+1: specifically on len() as in s[len()-1].
It's clearer than s[len-1] or s[len(...)-1] imo.
(It's been mentioned a couple times but always in posts with other proposals so just :+1:'ing those is ambiguous.)

If something like this goes in, I'm +1 specifically on len() as in s[len()-1].

Yes, I agree with that.
Although I prefer len to using a symbol, it seems incomplete (it is after all a function) and len() therefore looks better. len(...) is too long.

The examples shown in https://github.com/golang/go/issues/33359#issuecomment-549200884 show that using len with no arguments can be rather confusing in real code.  Using len() is not much better, as it still unclear at first glance what length is being taken.  Symbols like _ and ... are not particularly clear.  Nothing seems like a real improvement on the current code.
For these reasons this is a likely decline.  Leaving open for four weeks for final comments.
-- for @golang/proposal-review

The most common case here is probably len(s) - 1. It's worth noting that if we had generics, we could have library functions like slices.Last(s) and slices.Pop(s) that would work with any slice type. That might remove many of the cases where this is valuable.
I agree with @deanveloper that s[-1] does not click for me.

In my point of view. This slices.Last(s) and slices.Pop(s) is very clean and good understandable. It be could be even just Last(s)and Pop(s). Of course! The old behavior might still cause some confusion. Honestly... I didn’t even know that this is possible with slices.

No change in consensus.

I implemented it in v1.0.3 of https://github.com/rocketlaunchr/igo transpiler


Int128 Parsing support [PART 1]

This PR adds:

String#to_(u/i)128(?) methods
(U)Int128 literal parsing
(U)Int128 modulo/divide/multiply-overflow methods in compiler-rt (to allow this to even work)

With this you can do crazy things like:
$ bin/crystal eval 'puts 0xFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFFF_u128 - 1'
Using compiled compiler at .build/crystal
340282366920938463463374607431768211454

This is a subset of #11111 only including those changes not making the CI fail on crystal v1.1.1.
TODO:

[x] Find areas not supported on windows and put them behind flags

As far as I have read now, the int128 methods on MSVC require the use of SIMD (result expected to be placed in SSE register XMM0) - which crystal sadly does not explicitly support.
To not introduce breaking changes, the method deduce_integer_kind now uses the following priority:
Int32 > Int64 > UInt64 > Int128 > UInt128. I don't know if this should be changed.
Requires/Includes #11093
Related to #8373
Supersedes #10975
Closes #9516
Closes #7915
Related to #5545
Closes #11191

I found a hack to return the lower64 bytes of a 128 integer in the compiler-rt funs.
Floats are returned using the xmm0 register, which is exactly where llvm expects its 128-bit integers.
Crystal of course still needs SIMD support for this to fully work on windows

Finally, all checks passed!

Hmm.. I just found a breaking change..
Binary, octal and hexadecimal numbers containing underscores are not correctly parsed (throw an exception)
String.to_i does have a underscore option, but unline the crystal compiler, it does not support multiple sequential underscores.
Now I can either introduce a breaking change (which is probably not wanted), use String.filter('_') (which is probably slow) or write a lot of new code.. hmph..

I don't know why String#to_i fails on multiple sequential underscores. That actually seems like a rather arbitrary restriction. I certainly wouldn't have expected that, especially considering number literal syntax. I think they should really work the same way.
Changing the implementation of String#to_i to allow sequences of underscores is a technically a breaking change, but I think it's acceptable. Could even be seen as a bug fix if we consider the misalignment between String#to_i and number literals a bug.

I don't know why String#to_i fails on multiple sequential underscores. That actually seems like a rather arbitrary restriction. I certainly wouldn't have expected that, especially considering number literal syntax. I think they should really work the same way.
Changing the implementation of String#to_i to allow sequences of underscores is a technically a breaking change, but I think it's acceptable. Could even be seen as a bug fix if we consider the misalignment between String#to_i and number literals a bug.

The String#to_i implementation has a check built in just for this.
https://github.com/crystal-lang/crystal/blob/a067d0645d55ba7d42706d7957faa99fc530e0d4/src/string.cr#L595
It should be very easy to change this behaviour.

I don't know why String#to_i fails on multiple sequential underscores.

I'm having a hard time thinking up a use case where multiple _ would be wanted, so why allow it?
However, it fails in more cases:
puts "1_1".to_i

OTOH,
puts 1_1
puts 1__1

prints 11.
Ruby on the other hand returns 11
But Ruby also return 1 if you do "1__1".to_i
Perhaps because Ruby fails to parse 1__1.

However, it fails in more cases:
puts "1_1".to_i

puts "1_1".to_i(underscore: true) #=> 11

I'm having a hard time thinking up a use case where multiple _ would be wanted, so why allow it?

I concur, multiple _ doesn't make much sense to me either.

Btw there's another thing about current integer parsing which I think is a strange:
012 #=> Error: octal constants should be prefixed with 0o
0_12 #=> 12

Is this also a bug or should this stay like it is?

Or this bug in number parsing:
-0_u64 #=> Invalid UInt32: -0 (ArgumentError); you've found a bug in the Crystal compiler.
-0u64 #=> 0

I've extracted the underscore discussion to #11203 to stay focused on the PR here.
The examples in the previous two posts all look like legitimate bugs to me. If this PR happens to fix them, that sounds good to me.

Hi @BlobCodes, thanks for the hard work you put into this! If I may ask one more thing from you, would it be possible to break this PR even more, so we can look closely each bit? I'd say one PR for every bullet point in the description would be optimal. I can try to do it myself if you prefer. 🙏

Hi @BlobCodes, thanks for the hard work you put into this! If I may ask one more thing from you, would it be possible to break this PR even more, so we can look closely each bit? I'd say one PR for every bullet point in the description would be optimal. I can try to do it myself if you prefer. pray

Alright, I'll split it up.
These three steps however all depend on each other..
Can I already create all three PRs or is it better to create them one after another?

Let's do one after the other. That's cleaner.

1_.1 #=> Error: trailing '_' in number
1_e2 #=> Error: trailing '_' in number
-0u64 #=> Error: Invalid negative value -0 for UInt64
-0_u64 #=> Error: Invalid negative value -0 for UInt64
1__2 #=> Error: trailing '_' in number
0x_2 #=> Error: numeric literal without digits
0_12 #=> Error: octal constants should be prefixed with 0o

I'm not sure these changes are actually correct. IMO underscores should be allowed at any place in a number literal, including around decimal separator and literal base prefix. I don't think there is any harm in doing that, but it could allow more versatile use cases. I see no reason to put unnecessary restrictions if the intention is clear and unambiguous.
In Rust, all those literals are valid except for the unsigned -0. In contrast to Crystal, Rust allows leading zeros, hence 0_12 is accepted (because 012 is).

1_.1 #=> Error: trailing '_' in number
1_e2 #=> Error: trailing '_' in number
-0u64 #=> Error: Invalid negative value -0 for UInt64
-0_u64 #=> Error: Invalid negative value -0 for UInt64
1__2 #=> Error: trailing '_' in number
0x_2 #=> Error: numeric literal without digits
0_12 #=> Error: octal constants should be prefixed with 0o

I'm not sure these changes are actually correct. IMO underscores should be allowed at any place in a number literal, including around decimal separator and literal base prefix. I don't think there is any harm in doing that, but it could allow more versatile use cases. I see no reason to put unnecessary restrictions if the intention is clear and unambiguous.
In Rust, all those literals are valid except for the unsigned -0. In contrast to Crystal, Rust allows leading zeros, hence 0_12 is accepted (because 012 is).

Hmm.. In ruby, those are all throwing errors.
As far as I have seen from the comments above, I think "1__2" should definitely be disallowed.

(0_12 actually works in Ruby, it's interpreted as an octal literal (same as 012). Interestingly, 0o_12 is an error though.)
I don't see any convincing argument why the literals 1_.1, 1_e2, 1__2 and 0x_2 should become errors. If we were starting from scratch, it wouldn't be a different discussions. But they are currently valid and part of the language. I don't think there's enough benefit to justify the costs of changing that. Especially not as a incidental byproduct of a feature addition.
If somebody wants to propose such a change, they should start a dedicated discussion about that. But let's not mingle it with 128-bit support.

They certainly look like errors, are super-rarely used - if at all, are undocumented, so I don't get why shouldn't they be made errors - which they are - according to the non-existing specs and documentation - i.e. they were never designed to work that way.

But let's not mingle it with 128-bit support.
Hmm.. yeah, that's fair. But allowing int128 parsing requires a lexer refactor anyways, so I think there should be a discussion about this (because it's not that much added work).

IMO underscores should be allowed at any place in a number literal, including around decimal separator and literal base prefix

"including around decimal separator"
The number "1._1" would currently be recognized as calling the method _1 on the int32 1.
I think it should not be allowed to put an underscore before a decimal seperator too - that's just inconsistent

With String.to_i already raising when an integer has multiple underscores, I think this should be considered an error in the lexer too (Ary even called it a bug).
Searching on github for two sequential underscores returns zero number literals - nobody would actually do something like that

I don't see any convincing argument why the literals 1_.1, 1_e2, 1__2 and 0x_2 should become errors

I can kind of understand that you think 1_e2 or 0x_2 shouldn't become errors, but I think this is worthy of discussion

Anyways, the lexer refactor has been seperated into #11211 - let's continue the discussions there

Replaced by #11571


Warning in React 16

After updating to React 16, I get the following warning:
proxyConsole.js:54 Warning: React does not recognize the `timePickerDelay` prop on a DOM
element. If you intentionally want it to appear in the DOM as a custom attribute, spell
it as lowercase `timepickerdelay` instead. If you accidentally passed it from a parent
component, remove it from the DOM element.
    in input (created by n)
    in div (created by n)
    in n (created by t)
    in span (created by t)
    in t (at NewBooking.jsx:408)
    in div (created by StepContent)
    in div (created by ExpandTransitionChild)
    in div (created by ExpandTransitionChild)
    in ExpandTransitionChild
    in div
    in TransitionGroup (created by ExpandTransition)
    in ExpandTransition (created by ExpandTransition)
    in ExpandTransition (created by StepContent)
    in div (created by StepContent)
    in StepContent (at NewBooking.jsx:405)
    in div (created by Step)
    in Step (at NewBooking.jsx:403)
    in div (created by Stepper)
    in Stepper (at NewBooking.jsx:336)
    in div (at NewBooking.jsx:335)
    in VerticalLinearStepper (at NewBooking.jsx:229)
    in div (created by DialogInline)
    in div (created by Paper)
    in Paper (created by DialogInline)
    in div (created by TransitionItem)
    in TransitionItem
    in div
    in TransitionGroup (created by DialogInline)
    in div (created by DialogInline)
    in DialogInline

Fixed in 0.2.0


The AI eye can use the Arrivals shuttle

Description
If the AI eye is on the Arrival shuttle as it leaves it stays on the shuttle on it way back in contrast to other player character that get left behind if they are on the shuttle when it leaves.
Reproduction
As AI position your eye on the shuttle and wait for it to depart. You should then be on the shuttle as it is in FTL and a bit later you should arrive at the Arrival terminal.
Additional context
Im not 100% sure if that is an oversight or if its intentional but in case its the latter im making this report.

works on any shuttle going into FTL

Fixed by #32094


chore: add box-shadow to onboarding welcome popup

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description

Added box-shadow to onboarding welcome tour pop up .

Fixes #8208

if no issue exists, please create an issue and ask the maintainers about this first

Type of change

Please delete options that are not relevant.

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.

Test A

Added necessary box-shadow property through developer tools to preview and confirm the changes

To reproduce this simple open developer tools on the welcoming tour pop up window . Select the welcome tour pop element and inspect it's properties in styles tab . Double click the style properties and enter a new value for box-shadow as < box-shadow: 0px 12px 28px #df613c > . You can now see a faint box-shadow property applied to the welcome tour pop-up .

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Remove the space after 'feat' in PR title, this will solve the semantic test. feat: A PR title

Thanks @AR10X . It did the trick .

/ok-to-test sha=1c5d85f

@ramsaptami @ashish-patwal Can we keep the border? Shadow looks great.

/ok-to-test sha=d7b1197

Closing this PR as it is no longer relevant. Apologies for not prioritizing this PR earlier. 🙏🏽


frontend: Add node version and services selector into list

To keep the same output with kubectl, I made following changes:

Add selector for services list

Add node version to node list

I have set kubelet version as node version in table list. I left label parts because it will require UI design just as you said.


latest version 3.8.0 of liquibase-core is showing vulnerabilities

Facing below vulnerabilities in owasp dependency check.
[ERROR] One or more dependencies were identified with vulnerabilities:
[ERROR]
[ERROR] liquibase-core-3.8.0.jar: bootstrap.js: CVE-2018-14042, CVE-2019-8331, CVE-2018-14041, CVE-2018-14040
[ERROR] liquibase-core-3.8.0.jar: bootstrap.min.js: CVE-2018-14042, CVE-2019-8331, CVE-2018-14041, CVE-2018-14040
[ERROR] liquibase-core-3.8.0.jar: jquery-1.11.0.min.js: CVE-2015-9251, CVE-2019-11358
[ERROR]
[ERROR] See the dependency-check report for more details.

Is there any plan to handle these vulnerabilities in the latest release? Please suggest.

@prikshitsaini your question should be addressed to the liquibase team after investigating that these are actual vulnerabilities in their product


RFC: call preprocess on master node, too

Use case: initialization to enable simplifications of the merge
function, in case of a non-zero "neutral element."
Also clean up code duplication between run_for_dataset{,_async}
@uellue does this fix your issue?
Contributor Checklist:

[x] I have added myself to the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases

Awesome! Confirmed to work, merging. 👍


Upgrade Timezone file

Hello, im importing a datapump in the database 21.3.0, but I'm having an error with the timezone version:

Do you know how this can be solved?
Thanks

Hi @oscarodrigo,
Thanks a lot for using these images!
I have checked with my colleague who is responsible for this area and unfortunately there is no fix that can easily be done or provided.
The error that you are seeing happens because you are trying to import data from a database running with a Timezone version file 36 into a database (21c XE) with a Timezone version file 35, which is not allowed and hence this error is raised.
Usually the solution would be to apply a patch to update this Timezone version file to >= 36 but as there are no patches for XE, there is no way of doing that.
A remedy might be to use gvenzl/oracle-free instead which delivers the new Oracle Database 23c Free images.

Thanks!


Slideshow (automatic change of panoramic images)

Hi there,
Another suggestion: a slideshow option would be nice, with a fade-in-out effect between images. :-)
The setup could be similar to scenes, but here no user interaction would be needed...
Thanks for the hard work!!!! :-)
Kind Regards, Benke

would love that too!
Like a "autoRotateInactivityDelay" -> autoNextSceneInactiveDelay
Kind regards

Actually I need such a feature too. autoNextSceneInactiveDelay would be perfect.
Any chance this will work in the newar future?
Alternatively how can I switch to the next scene in the tour w/o hotspots using JS?

Scenes can be switch using the API's loadScene method, allow such functionality to be implemented using external code. I don't plan on integrating something like this.


Run dev server programmatically

Describe the feature
It currently doesn't seem to be possible to run the dev server programmatically since it is tightly coupled to the console command. Or did I miss something?
If so, imho this is a step back from Nuxt 2 where you could run everything programmatically.
Additional information

[ ] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

Yes, it is possible to run the dev server programmatically. You can check the implementation to see how to do it!
https://github.com/nuxt/nuxt/blob/a8f0a6d643f5d18cb12b1cdc8aa370e015522afc/packages/nuxi/src/commands/dev.ts#L37

@danielroe Yes I found that and I actually tried to implement this as an npm package. But it uses a lot of shared functions that are not exported, so I'd have to basically copy-paste a lot of the logic from nuxi. If things would be exported anywhere this would already help 🙂.

FWIW, the Nuxt commands are exported so you can invoke them programmatically if you don't need to customise things...

@danielroe Would it be somehow possible to pass a config through to this line?


chore(deps): bump github.com/Azure/azure-sdk-for-go/sdk/azidentity from 1.5.2 to 1.6.0

…

Fixes #
Description
How was this change tested?

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 97.766%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
36286

Relevant Lines:
37115

💛  - Coveralls


The widget cannot show anything

Dev Home version
<IP_ADDRESS>
Windows build number
10.0.22621.1778
Other software
OS Build Version: 10.0.22621.1778.arm64fre.ni_release.220506-1250
.NET Version: .NET 6.0.16
Steps to reproduce the bug
Add widgets in dashboard and windows 11 widget panel
Expected result
Nothing is showing up in the widgets.
Actual result
Blank widgets
Included System Information
Physical Memory: 15.47GB (6.52GB free)
Processor Architecture: ARMx64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_<IP_ADDRESS>_arm64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_<IP_ADDRESS>_arm64__8wekyb3d8bbwe

@clodderes Is the widget actually completely blank, or are you getting a "The widget could not be displayed" message, like in #722?


Added a playlist exporter

Hey,
I found your code and tweaked it a bit because I wanted to use it to get a single playlist.  I figured I'd see if you wanted to include it back into your project.
I didn't make any changes to your script but I did change some things about it in mine.  Specifically I changed the div you used to a text area and changed the css on it a bit.  The text area makes it easier to copy.
I also had to do some tweaking on the setInterval you were using, I'm not 100% sure but it seems like there might be a bug where it never terminates, I added a check to see if the show more button doesn't appear and call clear interval if so.

Merged, thanks!


add 'pin note' feature

i would like to be able to have quick access to some notes, maybe even if i haven't edited them recently but they are something i need to keep going back to look at because of their information.
with this feature, you could have the 'recently modified' section available unless you have any pinned notes, in which case that section is replaced with a 'pinned notes' section.
there are also others way to approach the UI aspect of this, for example making the text that currently says 'recently modified' with a minimalist toggle menu that allows you to choose which notes to display. this allows for further ui customization like 'recently viewed', 'recently created', etc.

Hi @pbogre. Thanks for the suggestion. Unfortunately, without a database, there wouldn't really be a clean way to store which notes were pinned. The only thing I can suggest is bookmarking the URL for the notes you need quick access to.

ah that makes sense. however i believe that through an improvement of the homepage ui (improvement for which has also been requested in other issues, like a grid view) could achieve something similar.
for instance, if we had an option to filter the notes seen in the homepage with something like tags, you could put a "pinned" tag on notes and by filtering only notes with that tag it would be the same thing as assigning a pinned value to the note. this would also add functionality to the tag feature which in my experience hasn't been very useful very often.
the suggestion about different types of notes to show on the homepage still stands however, and what i wrote in this comment uses that change to implement a way to pin notes based on tags with a filter

The idea of using tags is a good one. I hadn't considered that.

I'm pleased to say that this is now possible using the new Quick Access Notes feature added in v5.3.0.


add Maximizer connector

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[x] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hi @MaximizerIntegration Thank you so much again for your submission. As we ask all our partners to sign our CLA, may I ask you please to take a look at it and sign it so we can move forward with your connector.
Please let me know if you have any questions.
Thank you so much.

@MaximizerIntegration The apiProperties.json validator doesn't support the multi authentication. So, please ignore all the reported error messages by the APIProperties validator.
But, I can see that there 3-4 OAuth types added in your connection settings which is not common. Could you please let us know if you have tested all the connection types mentioned for your connector in your dev environment? If yes, could you please share a few screenshots of all the successful connections and flow runs of each oauth connection type.

@microsoft-github-policy-service agree [company="Maximizer Software Inc."]

@microsoft-github-policy-service agree company="Maximizer Software Inc."

Hello @MaximizerIntegration,
I hope you are doing well.
Congratulations, your pull request is approved and merged. Please submit the files to ISV Studio or update your submission using your latest commit ID, please allow us up to 1-2 weeks to review your submission, and our Engineers will notify you in the “Activity Control” area in ISV Studio. Please make sure that the icon meets all the requirements, that it’s size is 230x230, icon background is not white or transparent and it matches the iconBrandColor property in apiProperties.json file.
After making the submission in ISV Studio, please attach an intro.md file in the activity control section. This file is required and it's different from readme.md. You can find an example of intro.md file on this page at step 6.
Please Create an environment in the Preview region. You will use this environment later to test your connector after Microsoft is done with the functional verification for your connector.
We expect all tests to be completed within 1-2 weeks.
If your connector passes all tests, the deployment process begins, and it typically takes up to 3 to 4 weeks to deploy your connector to all of our regions.
Please let me know if you have any questions or run into any problems while creating or updating your submission.
Thank you very much for working with us.


Quill modules usage

HI
Do you have an example of how to use quill modules with react-quill.
I could not get quill-markdown-shortcuts to work. Here is the code:
import ReactQuill,  { Quill } from 'react-quill';
import { MarkdownShortcuts} from 'quill-markdown-shortcuts';
Quill.register('MarkdownShortcuts', MarkdownShortcuts);

...
render() {
...
        <ReactQuill value={'abcd'}
          modules={{
            MarkdownShortcuts: {}
          }}
        />
...
}
...

This is the Error that I get:
quill Cannot import modules/MarkdownShortcuts. Are you sure it was registered?
quill Cannot load MarkdownShortcuts module. Are you sure you registered it?
quill Cannot import modules/MarkdownShortcuts. Are you sure it was registered?
Uncaught TypeError: moduleClass is not a constructor
    at SnowTheme.addModule (quill.js:6052)
    at SnowTheme.addModule (quill.js:6696)

React-Quill version
1.3.0

I don't know about the particular module you're using, but I've had to Quill.import modules before I could use them.

I've encountered similar issue while using original quilljs (not react-quill) and managed to solve it. In my case, it is due to invalid modules added quill config.
Original Config
const quillOptions = {
  modules: {
    toolbar: `quill-toolbar`,
    variable: {}
  }
};

After removing variable from modules, it worked for me.
const quillOptions = {
  modules: {
    toolbar: `quill-toolbar`
  }
};

For anyone encountering this issue, you may want to check if you've added unregistered modules to your config.

I got the same error on quill on Angular (using ngx-quill). Like @pirsquare mentioned, mine was also a missing module issue, whose reference I had forgot to take out b/w projects.

::ng-deep .ngx-quill-editor
p > br
display: none
p:not(:last-of-type)
margin-bottom: 1rem

Any solution for it yet?

I'm getting this error after all the set-up done.

I had a similar issue when importing and registering a module, my solution was to:

Import the module from the dist folder : import BlotFormatter from 'quill-blot-formatter/dist/BlotFormatter'
When registering, register the module's default.: Quill.register('modules/blotFormatter', BlotFormatter.default);

Hope this helps.

I had a similar issue when importing and registering a module, my solution was to:

Import the module from the dist folder : import BlotFormatter from 'quill-blot-formatter/dist/BlotFormatter'
When registering, register the module's default.: Quill.register('modules/blotFormatter', BlotFormatter.default);

Hope this helps.

did you find any workaround?

error ReferenceError: document is not defined


Question about ITK addressList not mapped

Description
This is more a question and confirmation. The ITK assurance sample is tagged like this
 <!-- the address list is not needed, as the end point has been looked up and is in the SOAP header
                <itk:addresslist>
                    <itk:address uri="urn:nhs-uk:addressing:ods:EM396"/>
                </itk:addresslist> -->
                <!-- an audit identification of the sending organisation - use the send organisation's ODS site code -->

So it seems the address list is not mapped. I think the comment is based on the assumption, every organization will have its own adapter endpoint.
Med base plan to deploy just one adapter for all organizations that using NHS 111, as there is no value to deploy many adapters (deployment and configuration burden). In that case, the ITK addresslist will be essential to map to which organization the message needs to pass to. I believe this element should be mapped as they are in the original CDA document unless the element deems absolutely unnecessary in any circumtances.
Can you confirm if my assumption is correct, or there is something missing in my assumption?
Component
NHS111 adapter
Version
0.1.0
Step to Reproduce

Send a valid message
inspect that ITK AddressList is  not mapped

Test Data
Any data
Preconditions
None
Environment
local environment
Severity
Low,
Priority
Not urgent
Expected Behavior
ITK addressList should be mapped.

This will be implemented in the next release, itk:address uri will be mapped to destination in the header

Address lists from 111 always come in this format:

ODS code is not reliable- it is often perfectly fine for GP practice systems, however is sometimes not accurate for branch sites etc.  The second URI is the DOS Service ID.  It would be best to send both through in FHIR, but use the Service id as config receiving end to work out where to put this in the right place ie specific GP instance on a shared system etc.

The version 0.3.0 only outputs the first one, not sure about the second one as the input XML only have the first one only.

@Kusnaditjung  as discussed this can be closed


[ASCII-1434] Refactor api endpoints for components into registered providers

What does this PR do?
Previous work:

https://github.com/DataDog/datadog-agent/pull/24045
https://github.com/DataDog/datadog-agent/pull/24872

As part of the componetization effort:

make tagger, autoconfig, workloadmeta, collector, and logsAgent component dependencies of the api component instead of arguments for starting the server

Register routes with EndpointProviders from the respective components:

~/secrets and /secrets/refresh  are moved to the core/secret component~ since comp/core/secrets is a go module the api component needs to be one too similar to the status component, separating this into it's own PR
/workload-list is moved to the core/workloadmeta component
/tagger-list is moved to the core/tagger component
/config-check is moved to the core/autodiscovery/autoconfig component

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

there are e2e tests for most of these routes
/tagger-list is tested in https://github.com/DataDog/datadog-agent/tree/main/test/new-e2e/tests/containers
agent with jmx configuration will need to be tested as this modifies that API, TODO: write some instructions for that

/merge


providing pronunciation hints

It would be useful to be able to give yap a hint as to how to pronounce the name of a town when it reads meeting search results.
For example, if I search for meetings in Yakima, Washington, yap recognizes the town name, but when reading the meetings back, it pronounces it as 'UkEEma'.

I'm pretty sure this would also affect Geocoding responses.  For example if you enter a zip code or say a city or state, it reads back the location.

Yes, we would want incorrect pronunciations to be fixed for that as well, if possible.

This will be in the next yap 4.3.0 beta

yap 4.3.0 beta 8 was released.

Alas I'm not having any luck getting pronunciation hints to work. The example in the documentation is for Yakima:
static $pronunciations = [[
    "source"=>"Yakima",
    "target"=>"UkEEma"
]];

But UkEEma is what Twilio/yap currently says, and isn't correct.  It should be YAKima (/ˈjækɪmɑː/).  I tried various phonetic spellings (YAKima, Yaakima, etc), but none of them changed what was spoken.  Then I tried just "Yak" as the target, to see if it was getting picked up at all, and Twilio/yap is still saying UkEEma.
Should I be doing something different?

Is this still not working?

#943

Looks like there is a bug in the pronunciationReplacement function -- it just works on the first replacement source/target pair.  Need to go through all of the replacements.

Interesting, because str_replace() is supposed to replace all occurrences.  Will write another test for this and see what's up.

The "all occurrences" part is ok.  Here's the function:
    private function pronunciationReplacement($subject): string
    {
        foreach ($this->settings->get("pronunciations") as $item) {
            return str_replace($item['source'], $item['target'], $subject);
        }

        return $subject;
    }
}

So we iterate through the pronunciations, and return after processing the first item.  So you'd fix all the occurrences of "Sequim", but not "Yakima", for those two pronunciation hints.

gotcha, i understand now what you are saying!


KERALA STATE

Create State 'Kerala' and add some famous Tourist Spots of the State.
Please assign this issue to us!
Team taco

You can work on this issue. All the best !


React Native Support

Is there any reason this won't work with Firebase in React Native?

Good question. This library is based on Next.js middleware, so it requires to be a part of Next.js application.
If you need to authenticate with Firebase in React Native, you can use official solution, such as https://rnfirebase.io/


Translation error

Hello,
I have a question about the translation from cypher to gremlin. I have a simple query, like "MATCH (arch:Architecture {_id:'ARCH_ID'}) RETURN arch". But It throws error during the translation.
Following is the error message.
java.lang.ClassNotFoundException:org.apache.tinkerpop.gremlin.process.traversal.step.util.WithOptions
I think this is a very common cypher statement. Could anyone help to point out where I am wrong? Many thanks.

Hello @yuzhebj,
Related question: #270.
WithOptions were introduced in TinkerPop 3.4.0 and are not supported in:

Cloud databases like AWS Neptune and Cosmos DB.
Gremlin Servers that use TinkerPop < 3.4.0

For example, Janus Graph upgrade to TinkerPop 3.4.0 is not released yet

To address this:

For cloud databases use corresponding  flavors
For client-side translation use flavor that will not use WithOptions - gremlinServer33x
For server-side translation:

Install previous release of Cypher for Gremlin built for TinkerPop 3.3.4
Or use tinkerpop334 branch

Please let me know if you have any other questions, or close this issue.

@dwitry Thanks for your reply. I changed the translation library version to 0.9.13, and it works now.

Welcome.
Closing issue, as @yuzhebj confirmed that it works. Please open new issue if you have additional questions.


Login popup when starting the process

I'm in Ethereum since the early days and have see everything evolve. In my opion tBTC dapp UX lags in the first step because the button "beggin now" is unclickable and the "Connect to a wallet" button is completely in the more far away part of the screen. THIS MAKES THE USER UNRELATE BOTH concepts. Please see the screenshots and comparte with RenProject and Synthetix Mintr.
In my opinion there should be a prompt or more clear call to action to login with Metamask or whatever wallet.

Please see the following screenshots:

You don't need to file two issues on something, even if you feel strongly - it makes more work for the repo maintainers, and splinters discussion around potential improvements.
Closing this as a duplicate of #277.

splinters discussion around potential improvements.

As an example of this- I think the Mintr UX is a much better approach than Ren's full-page modal, because it displays value to the user before asking for personal information. Unfortunately, I didn't see this screen until finding your dup issue :shrug:


Make Id a simpler wrapper of std::reference_wrapper. Context will hold the memory.

Optimize the memory usage and save some copies.
Also pave the path to better support mutation in the handler.

Maybe no longer needed after recent refactoring.


Fixes for Bed Search OpenAPI spec

Camel case property names instead of snake case
Use service name enum
Use wrapper type as repsonse body rather than array of results

 Create a search endpoint for Match


(AFTER successful login)apiwrapper.py, metrics.py,pgoapi.exceptions.NotLoggedInException

[PokemonGoBot] [INFO] [login_successful] Login successful.
Traceback (most recent call last):
File "pokecli.py", line 502, in
main()
File "pokecli.py", line 131, in main
report_summary(bot)
File "pokecli.py", line 140, in report_summary
metrics.capture_stats()
pokemongo_bot/metrics.py", line 76, in capture_stats
response_dict = request.call()
pokemongo_bot/api_wrapper.py", line 90, in call
if not self.can_call():
pokemongo_bot/api_wrapper.py", line 58, in can_call
raise NotLoggedInException()
pgoapi.exceptions.NotLoggedInException
OS: Mac 10.11
Python Version: 2.7.12

bump, same problem

what's the cause of this problem?
I still got this problem from some of my machine (Linux and FreeBSD) with certain account (but the account is running fine on another machine —on another subnet.
Is it some kind of ban?


Bump dependencies

This bumps base64, rand, and md5, which are all very common dependencies, so that downstream crates do not have to build multiple versions of those crates. I did not bump hyper (which really needs bumping) because that requires a decent amount more work. The move from 0.11 to 0.12 is not straightforward.
In the process I also fixed a warning that appears on newer versions of the Rust compiler.

Thanks
The hyper-winauth crate was never more than a POC and intentionally is not at crates.io.
Also figuring out what broke the windows tests would be up first^^


FIX-#6273: fix DataFrame.min/max/mean/median/skew/kurt with axis=None

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #6273
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

@mvashishtha could you review it?

Leaving the last word for @mvashishtha.

@mvashishtha ready to merge

thank you @anmyachev !


set album input title maxlength to varchar(50) length

References #305 and #325
If need, set display length by css

What happens when you set an album title of 50 chars including specials chars like ä, ö, ü etc.? I guess they will take more space than the allowed 50 chars of the database field.

No, that works well.

hmm sorry, no it does not - after a reload. The size shown will be size minus the amount of umlaut chars.

I will update the database field with the next version (>3.0), so the title can contain more chars. Than we can adjust the limit in the Front-End.

PR has been out-closed by GitHub because I deleted the v3.0 branch. Will take care of it.

A length of 50chars is now allowed (see develop branch).


Add new warnings and constraints to phylo tree workflow

Description
This PR:

Replaces error message on number of samples and changes the lower limit.
Add new warning message on number of reads.

Type of change

New feature (non-breaking change which adds functionality)

Test

Verified that creation is blocked by too few (<4) or too many (>100) samples
Verified that a warning is displayed for samples with too few reads.

Maybe a dumb question but if the message says
"Would you like additional samples..."
Shouldn't there be no minimum? Maybe the message needs to change as well.

Makes sense... I would probably change to something more neutral like "Add additional samples..."


Expand Sidebar to Current Page

This PR adds logic to expand sidebar to whatever page is selected. This behaviour doesn't work by default as Mosaic uses breadcrumbs data to determine which page to expand, but we have disabled the Breadcrumbs plugin.
It is just a draft to try and diagnose a problem with nested pages collapsing in the Sidebar.

@lilyvc Could you add more content where this change has come from? Does this fix the active page state on the navigation menu?

Oh is there a problem with the active page state? This could help, but not sure of the issue. This PR is just because @feliciackh was adding a nested page in the sidebar and noticed it kept collapsing. We figured it is because of two things - it's reloading for some reason and when it reloads it doesn't know what should be expanded. This fixes the latter so I wanted to share so we could see the preview and if the former issue persists.

To provide more context, Dashboard pattern published recently uses accordion in sidebar.
Steps to reproduce the issue @lilyvc noted above:

click on Dashboards in the sidebar, accordion expands
click on Analytical Dashboard, accordion collapses when it should remain expanded on the active page


Web tell me 404 when i add page-about.hbs

Welcome to Ghost's GitHub repo! 👋🎉
Do you need help or have a question? Please come chat in Slack: https://ghost.org/slack 👫. Found a bug? Please fill out the sections below... thank you 👍
If your issue is with Ghost CLI, please report it on the CLI repo ➡ https://github.com/TryGhost/Ghost-CLI/issues/new.
Issue Summary
Ubuntu 16.04
Steps to Reproduce

At http://themes.ghost.org/docs/page-context.

For example, if you have an 'About' page with the url /about/, adding a template called page-about.hbs will cause that template to be used for the about page, instead of page.hbs, or post.hbs.

These templates exist in a hierarchy. Ghost looks for a template which matches the slug (page-:slug.hbs) first, then looks for page.hbs and finally uses post.hbs if neither is available.
3.I have page-about.hbs and page.hbs , and write the Example Code into it.
4.No matter myself theme or casper.All tell me 404.
Please tell me why.Thank you very mach!
Technical details:

Ghost Version:1.18.0
Node Version:6.12.0
Browser/OS:Chromium 62.0.3202.94 / Ubuntu 16.04 x64
Database:mysql 5.7.20

@wyx666518 You also need to go into the Ghost post editor and create a post with the slug "about", mark it as a static page, and then publish it, in order for the page to show up.
Going to close this as it's most likely not a bug in Ghost, and we handle support requests over in our slack channel. If it does turn out to be a bug in Ghost, we can reopen this :)


Consider provide more details on Back up and how it's performed

It can be useful to have additional details about Back Up for Cassandra MI
[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a28efb0b-65cc-5be9-e2a1-5e7fbd116916
Version Independent ID: 20e7bbca-879d-982e-beb9-07c668af2882
Content: Frequently asked questions about Azure Managed Instance for Apache Cassandra from the Azure portal
Content Source: articles/managed-instance-apache-cassandra/faq.md
Service: managed-instance-apache-cassandra
GitHub Login: @TheovanKraay
Microsoft Alias: thvankra

@DavideDelVecchio  - Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can update the document accordingly.

@DavideDelVecchio there is already documentation here: https://docs.microsoft.com/en-us/azure/managed-instance-apache-cassandra/management-operations#backup-and-restore. Is this sufficient? At minimum, we will update the FAQ To refer here.


Max_health mechanism issue

The output of /version on my server is: CraftBukkit version git-Spigot-2cf50f0-2b93d83 (MC: 1.12.2) (Implementing API version 1.12.2-R0.1-SNAPSHOT)
The output of /version denizen on my server is: Denizen version 1.0.2-SNAPSHOT (build 1651)
(If relevant) A link to a paste of a script demonstrating the issue: https://one.denizenscript.com/haste/50534
The issue is:
I don't know why only the mech max_health won't work here :
Entity_Test:
    type: entity
    debug: false
    entity_type: creeper
    has_ai: false
    max_health: 50
    gravity: false
    silent: true
    glowing: true
    custom_name: Test
    potion_effects: INVISIBILITY,0,20000000
    health: 20
    remaining_air: 20000000

It work when I do /ex spawn creeper[gravity=false;potion_effects=INVISIBILITY,0,20000000;silent=true;glowing=true;has_ai=false;custom_name=Test;max_health=50;health=50;remaining_air=20000000]

It work when using another mechanism : the health_data one


add IntelliJ IDEA plugin

like the atom plugin, this plugin can open Intellij IDEA project from shell.
just use:
idea . or idea /projects/test_project

Can you include a README file?

@robbyrussell
done!

what happened to this PR?

@Xerus2000 I don't know what happened to this PR however, you don't need this. You can simply create a idea cli script through the interface. IIRC it's in Tools > Create command line launcher


SettingsFile becomes PresetLibrary.

As I was writing the documentation (still in progress) it became clear that "settings" wasn't the best name.
I decided on a "library" of "presets" and changed the code to correspond.

+1


Property 'do' does not exist on type 'FirebaseListObservable<any[]>'

Hello guys,
I'm following the doc here but I got this error Property 'do' does not exist on type 'FirebaseListObservable<any[]>' when trying to run the "Retrieving the snapshot" code. My code is mostly the same with the one in tutorial.

var items = this.af.database.list('/organizations', { preserveSnapshot: true });
items
      .do(snapshots => {
            snapshots.forEach(snapshot => console.log(snapshot.key()));
      })
     .subscribe(snapshots => console.log(snapshots.length));

My angularfire2 version is 2.0.0-beta.0. NodeJS version is 4.4.4.
Thank you.

Hey @ngocnb-2nf. What do your imports look like? You may have to import the do operator from rxjs.

Hi @davideast, below is my import:

import { Injectable, Inject } from '@angular/core';
import { FirebaseRef, AngularFire } from 'angularfire2';

I read file utils/firebase_list_factory.ts and see the import below:

import 'rxjs/add/operator/mergeMap';
import 'rxjs/add/operator/map';

There's no operator do here. I tried to add it manually but got not luck :).

@ngocnb-2nf
Can you create a plnkr? This is usually a problem with the import statement.

@ngocnb-2nf
import 'rxjs/add/operator/do';
Adding this at the top solved the issue for me.

Thanks, @UzairNouman!

I import from rxjs, but still get the same error:
[ts] Property 'filter' does not exist on type 'FirebaseListObservable'.
[ts] Property 'map' does not exist on type 'FirebaseListObservable'.
[ts] Property 'do' does not exist on type 'FirebaseListObservable'.
Is there something different with the new version ("angularfire2": "^2.0.0-beta.7-pre")?  Any help appreciated.
http://stackoverflow.com/questions/41550132/property-filter-does-not-exist-on-type-firebaselistobservable

In my case I have
import 'rxjs/add/operator/do';
get the same error
here is my code
getFamilyType(): Observable<IFamilyType[]> {
    return this._http.get<IFamilyType[]>(this._familyTypeUrl)
    .do(data => console.log('All: ' + JSON.stringify(data)))
    .catch(this.handleError);
}

@TorajKhavari That might be because of the recent changes in RxJS, try the following:
import {tap} from "rxjs/operators"; // I believe tap replaces do now, gotta confirm
Also, now to use those operators, you need to use the pipe() for chaining like:
.pipe(tap(data => console.log('All: ' + JSON.stringify(data))))
This link should give you a good summary of the changes that appeared in RxJS 6:
https://auth0.com/blog/whats-new-in-rxjs-6/
Hope that helps.


MobileWeather app needs to be updated to use Managers.

Update mobile weather app to use all the latest features and managers in SDL.

Closed via #15


can't run demo correctly

Sorry to bother you. I get an error when I run this command：
sudo python main.py demo  --openpose /home/fjt/openpose/build --video resource/media/skateboarding.mp4
Starting OpenPose demo...
Configuring OpenPose...
Starting thread(s)...
Auto-detecting all available GPUs... Detected 1 GPU(s), using 1 of them starting at GPU 0.
OpenPose demo successfully finished. Total time: 32.509077 seconds.
*** Error in `/home/fjt/openpose/build/examples/openpose/openpose.bin': free(): invalid pointer: 0x0000000001afe130 ***
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x777e5)[0x7f5ad54a67e5]
......
......
(core dumped)
Traceback (most recent call last):
File "main.py", line 31, in
p.start()
File "/home/fjt/st-gcn/processor/demo.py", line 47, in start
video = utils.video.get_video_frames(self.arg.video)
File "/home/fjt/st-gcn/tools/utils/video.py", line 33, in get_video_frames
vread = skvideo.io.vread(video_path)
File "/usr/local/lib/python3.5/dist-packages/skvideo/io/io.py", line 144, in vread
reader = FFmpegReader(fname, inputdict=inputdict, outputdict=outputdict, verbosity=verbosity)
File "/usr/local/lib/python3.5/dist-packages/skvideo/io/ffmpeg.py", line 44, in init
super(FFmpegReader,self).init(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/skvideo/io/abstract.py", line 116, in init
"No way to determine width or height from video. Need -s in inputdict. Consult documentation on I/O.")
ValueError: No way to determine width or height from video. Need -s in inputdict. Consult documentation on I/O.
python 3.5
skvideo 1.1.11
openpose latest version
I really need your help.Looking forward to your reply.Thank you very much!

I have solved this problem.
I used source code to complie ffmpeg and solved it.

Thank you.


Registration

@JuliaRegistrator register()

Error while trying to register: Version 0.2.0 already exists

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/1451
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.1 -m "<description of version>" 30c546f561e347d218b50cbe726d2ebc97184841
git push origin v0.2.1


fix and add

Description
fix: Resolved an issue with incorrectly generated xposedMethodSnippet when the parameter type is generic.

add: Introduced xposedGenerateFieldSnippet.
Main.class.zip

@LanBaiCode looks good. Thank you!


up vm memory

composer runs out of memory during provisioning, this resolves the issue. fixes #103

I'm ok with this or even higher as the main sweet spots for developer laptops tends to range from 4-8G memory.


Update wasm-pack-action version to 0.4.0,

Fix wasm-pack version rather than using latest.
GitHub Action often uses previous versions.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 98.607%

Totals

Change from base Build<PHONE_NUMBER>:
-0.002%

Covered Lines:
45300

Relevant Lines:
45940

💛  - Coveralls


Panic if changing browser zoom in WASM with an overridden scale factor

Bevy version
Main and 0.13.
What you did
Given:

a WindowPlugin with a custom resolution and an overridden scale factor
a WASM app
the browser zoom is changed (CTRL + +/-)

the app crashes.
What went wrong
The problem is that the bevy_winit plugin reacts to the WindowEvent::ScaleFactorChanged, that's received from winit holding a borrow_mut on the ResizeScaleHandle, and immediately requesting a new inner size.
Happens also on the winit 0.30 PR, for the same reason.

This was fixed in https://github.com/rust-windowing/winit/pull/3727 which landed in Winit v0.30.2.

I confirm this is fixed!


[WIP] Add vep86 plugins

Starting work to add GeneSplicer and MaxEntScan plugins to VEP
work to be done:

[ ] review genesplicer recipe in Bioconda
[ ] add genesplicer training sets to data repo for download
[ ] add MaxEntScan recipe in Bioconda
[ ] check for compatibility with GRCh37/hg19/hg38

I've also added some comments in the code where I could use some help to further this effort.

I'm messing this up big time
sorry about that, I'll close this and create a new pull request based on the master branch


Implementing user definable behaviour for repeat, shuffle, and crossfade

I've split the previous commit, this only covers the changes to make the repeat, shuffle and crossfade reset behaviour definable in usersettings.json. If these changes pass the testing, I'll submit a pull request for the changes to TuneIn command coding and the addition of a "Favorites" command.

Hello,
Is there any chance you could try and merge this pull request?
Thanks

Hi, thanks for your patience. I’ve been pretty busy— I’ll try to look
tonight (pacific time).
On Tue, Mar 15, 2022 at 4:33 AM ashenshugarRET @.***>
wrote:

Hello,
Is there any chance you could try and merge this pull request?
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/ryanolf/node-sonos-nfc/pull/12#issuecomment-1067884971,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJSBSV2Z4QHHCLJYID2PI3VABYPJANCNFSM5MWECGAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks, I understand.

Hi @ashenshugarRET, just suggested a few tweaks in the code. The main thing is that for consistency with the URIs written to the cards for Apple, etc, we should use ":" as a delimiter. I'm happy for you to add "/" as an acceptable option as well if you'd like. I like being permissive.
The other thing is that the tests were not passing on my machine. The CI is still pending I think. Please run the test suite before you update the PR. Thanks!

You can run the test suite on your local repo with npm test. I just pulled the updates and tests are still failing. I think I can fix if you give me a minute.

Thanks, yours is the first repository I've used that has test etc, so a bit of a steep learning curve.
Thanks for your help resolving the problems.

Ok, we got it. Thanks for contributing.

Thanks, the node-sonos-http-api has just added the BBC Sounds service to stream BBC radio stations which I was hoping to add to this repository to as I think it will be very popular.
Would it be OK if I generate another pull request to implement this too?

Yeah, sure! Please add tests to the test suite for it and run the test
suite before you file the pull requests. You run the tests on your local
machine with npm test in the root folder of the project. You can read
more about running tests with npm in their docs.
Ryan
On Wed, Mar 16, 2022 at 11:58 AM ashenshugarRET @.***>
wrote:

Thanks, the node-sonos-http-api has just added the BBC Sounds service to
stream BBC radio stations which I was hoping to add to this repository to
as I think it will be very popular.
Would it be OK if I generate another pull request to implement this too?
—
Reply to this email directly, view it on GitHub
https://github.com/ryanolf/node-sonos-nfc/pull/12#issuecomment-1069485786,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJSBSXP6VKFXKOHNIYQT6LVAIVL3ANCNFSM5MWECGAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Explanation for chunking

Hey, not a cloud expert but wondering the rational for using the chunking you have. I see that the zarr files have rather large chunk sizes. For example, the model level variables have chunk size dask.array<chunksize=(48, 137, 410240). This works out to be about 10 gigs. My understanding was that a good chunk size for object storage is on the order of MBs. Wouldn't it make sense to have chunking (1, 1, 410240) for example?

You're right, the current chunking scheme is quite big. It would definitely be an improvement to split each chunk per level, as you have suggested. We can definitely prioritize this improvement for working with the CO version of the data (our current focus has been towards Phase 2). I'll leave this issue open to track the work.

Hello @loliverhennigh ,
I'm glad to inform you that I had created a pull request (#31) that addresses the current issue(https://github.com/google-research/arco-era5/issues/28 issue) which you opened. The changes in the pull request effectively solve the problem you reported.
Could you please review the pull request and, if everything looks good to you, can you marked as resolved this issue? As @alxmrs already merged this PR into main branch
Thank you for bringing this issue to our attention. If you have any further questions or concerns, feel free to let me know.


Implement Footer Component

Please do work for this task in a branch named issue-10.

Making a simple one, may add changes later


Add DESCRIBE to MySql dialect autocomplete

Reusing https://github.com/cloudera/hue/pull/1684

@agl29 Another one where we could see for adding "describe/describe_table.jison", in the mysql dialect

@agl29 Another one where we could see for adding "describe/describe_table.jison", in the mysql dialect


Fix reorder of nodes with multiple dependsOn nodes

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Fixes a bug in the reorder function for a testflow where a step defines multiple dependencies on other steps.

/hold

/hold


codeium: 1.30.2 -> 1.30.18

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for codeium is: Codeium language server
meta.homepage for codeium is: https://codeium.com/
meta.changelog for codeium is: https://codeium.com/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.30.16 with grep in /nix/store/nxp1rxyp66hbjfrjrqs6jf370j0bghdm-codeium-1.30.16
found 1.30.16 in filename of file in /nix/store/nxp1rxyp66hbjfrjrqs6jf370j0bghdm-codeium-1.30.16

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

codeium
emacsPackages.codeium

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/nxp1rxyp66hbjfrjrqs6jf370j0bghdm-codeium-1.30.16 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/4ee32ef303ffe162ef92e9b4445de359f17ffc0a#codeium

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

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

x86_64-linux

:white_check_mark: 2 packages built:

codeium
emacsPackages.codeium

Maintainer pings
cc @anpin for testing.

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

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

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for codeium is: Codeium language server
meta.homepage for codeium is: https://codeium.com/
meta.changelog for codeium is: https://codeium.com/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.30.18 with grep in /nix/store/wr70qdhi7al34jmyyx048la22kcwwyvn-codeium-1.30.18
found 1.30.18 in filename of file in /nix/store/wr70qdhi7al34jmyyx048la22kcwwyvn-codeium-1.30.18

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

codeium
emacsPackages.codeium

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/wr70qdhi7al34jmyyx048la22kcwwyvn-codeium-1.30.18 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/46858a2be45002b872ca8d04f95eb28afd77a805#codeium

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

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

x86_64-linux

:white_check_mark: 2 packages built:

codeium
emacsPackages.codeium

Maintainer pings
cc @anpin for testing.

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

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


chore: move doc for sanity plugin

WHY are these changes introduced?
Fixes [ENG-7195)]

WHAT is this pull request doing?
Move doc for sanity plugin
Testing

Checkout changed file
Checkout https://nacelle.com/docs/querying-data/nacelle-sanity-plugin-nacelle-input

Good call thank you @mdarrik, the link to the v1 documentation has been added!


Build Errors :Cannot access RuntimePermissions

FAQs
Hi ~
I use the version 4.1.0 with andoridx in my project .  I rebuild the project but error happened.
reminds me " Compilation failed; see the compiler error output for details." .
Then I run with the cmd  "gradlew compileDebugSource --stacktrace -info" in android stuidio terminal,
I see the error info " Error : Cannot access RuntimePermissions, cannot find permissions.dispatcher.RuntimePermissions class file", so I can't generate xxxxPermissionsDispatcher by this way.
But when I use the android studio plugin I get the xxxxPermissionsDispatcher and still cannot build, faced the same error.
 Help !
Thank you very much!
The build.gradle:
    implementation 'com.github.permissions-dispatcher.PermissionsDispatcher:permissionsdispatcher:4.1.0'
    annotationProcessor 'com.github.permissions-dispatcher.PermissionsDispatcher:permissionsdispatcher-processor:4.1.0'

The java content:
@RuntimePermissions
public abstract class PermissionCheckerDelegate extends BaseDelegate {

    @NeedsPermission(Manifest.permission.CAMERA)
    void startCamera(){
        LatteCamera.start(this);
    }

    @Override
    public void onRequestPermissionsResult(int requestCode, @NonNull String[] permissions, @NonNull int[] grantResults) {
        super.onRequestPermissionsResult(requestCode, permissions, grantResults);
        PermissionCheckerDelegatePermissionsDispatcher.onRequestPermissionsResult(this, requestCode, grantResults);
    }

}

tips: The BaseDelegate extends androidx.fragment.app.Fragment
sorry to bother.

well , I figure this out by  changing "implementation" to "api", the build errors disappeared.
api 'com.github.permissions-dispatcher.PermissionsDispatcher:permissionsdispatcher:4.1.0'

hope this can be helpful if someone has the same problem.


"self" does not exist

I am new to erlang/elixir. I am currently going through the examples. Executing the async http call:
HTTPotion.get "http://httpbin.org/get", [stream_to: self]
in my console (iex -S mix) gives me this warning:
variable "self" does not exist and is being expanded to "self()", please use parentheses to remove the ambiguity or change the variable name
I assume self has changed to self() in current versions?

Yeah, probably.


Domain DN not defined (opts.rootDn)

Hi,
I am migrating our application from LDAP to Windows Active Directory, i came across with passport-adauth, but after configuration below
  url:` 'ldap://example.com:1234',
  bindDn: 'CN=*****,CN=*****CN=*****,DC=*****,DC=****',
  bindCredentials: '*****',
  searchBase: 'dc=*****,dc=*****',
  searchFilter: 'samaccountname={{username}}'

I am getting error Message : Domain DN not defined (opts.rootDn) with a 500 Internal Server Error
Can you please help us, when do we get this error? what parameters are we missing?
Thanks,
Dhiraj

The domainDn required parameter is missing. I am assuming your domain DN is the same as the searchBase in your example, so try copying the searchBase parameter to another called domainDn.
  url:` 'ldap://example.com:1234',
  bindDn: 'CN=*****,CN=*****CN=*****,DC=*****,DC=****',
  bindCredentials: '*****',
  searchBase: 'dc=*****,dc=*****',
  domainDn: 'dc=*****,dc=*****',
  searchFilter: 'samaccountname={{username}}'

Hi @vsimonian, Thanks for you quick response
We tried using this and we got connection refused in the browser, what do you think we are missing?

Not sure why that would be happening. Could you provide more details?


nixos: add AppArmor PAM support

Enables attaching AppArmor profiles at the user/group level.
By itself, this doesn't do anything without the requisite AppArmor profiles. Also, it is optional
for now and so is mostly of interest to people who want to experiment with the feature.
I have verified that it is indeed safe to enable this, it only adds some minor printk spam. To
build an actual RBAC system you'd change session optional to required, but then it'd be possible
to lock yourself out of the machine ...

lgtm

I'm working on defining a set of sane defaults that people can build their own RBAC systems on.
It's all just regular AppArmor profiles after all, so you don't need much to get going, but it has to be safe to enable with 'session required' and that requires a bit of scaffolding. The general idea is that admin should be able to enable RBAC without causing disruption and then incrementally define additional roles per site requirements.
I'm thinking that if "^root" transitions into an unconfined shell you're pretty much safe to experiment with adding profiles for other roles without doing too much damage (you'll always be able to get in as root). This in itself would do much to minimize potential disruption (of course, with NixOS you can always boot into an older config, but that's a hassle).
I'm also thinking that it makes sense to have "^users" transition to a "login_user" profile which would basically allow anything a normal user would need to be productive (again to avoid disruption). Admin could constrain individual users by adding hats matching their login name (AppArmor is currently configured to match first login user name or else user's primary group, before falling back to "^DEFAULT").
The "default" profile (whatever "^DEFAULT" transitions into, if anything) could be a restricted version of login_user or could be left blank to block non-user logins. The most compatible option would perhaps be to allow "default" to read/write their own files and execute anything from the store subject to profile inheritance (/nix/store/*/bin/* Px, or whatever) but not do too much else.
Finally, we'd need to define profiles for the PAM-enabled executables that should trigger attachment at the user level (su, login, whatever). Without this, attachment won't occur at all, as far as I can tell (from experimenting on my own system).
Anyway, that's what I got thus far, there are probably lots of details to be hashed out. I'm not sure to what extent there is interest in having something like this in NixOS, but I think it'd be nice to at least have a somewhat sane path to a working RBAC system.

Pushed a new version which also checks for security.apparmor.enable.

I'm now thinking that it probably only ever makes sense to enable this for specific PAM services which deal with login/changing user/group. It doesn't matter currently, because of session optional, but please do not merge yet.

AppArmor is now enabled per service, as in security.pam.services.su.enableAppArmor = true, which does what you'd expect. Works for me. I believe it is ready to merge. It should make no difference for people who do not use it and should be safe to experiment with for those who care to do so. Also, the old code doesn't make sense, anyway.


[Feature]: Sign Helm Charts

Checklist

[X] I've searched for similar issues and couldn't find anything matching
[X] I've discussed this feature request in the K8sGPT Slack and got positive feedback

Is this feature request related to a problem?
No
Problem Description
Helm Provenance and Integrity
Helm has provenance tools which help chart users verify the integrity and origin of a package. Using industry-standard tools based on PKI, GnuPG, and well-respected package managers, Helm can generate and verify signature files.
Integrity is established by comparing a chart to a provenance record. Provenance records are stored in provenance files, which are stored alongside a packaged chart. For example, if a chart is named myapp-1.2.3.tgz, its provenance file will be myapp-1.2.3.tgz.prov.
Provenance files are generated at packaging time (helm package --sign ...), and can be checked by multiple commands, notably helm install --verify.
Right now there in ArtifactHub there it is showing as the helm chart is not signed. This could stop some people from adopting it.  https://artifacthub.io/packages/helm/k8sgpt/k8sgpt-operator
It is easy to do so I propose that we do it.  We can add this capability on to the helm-chart-releaser that we already use.
Solution Description

[ ] Create GPG Key and passpharse
[ ] Upload them to Github Secrets
[ ] Update GitHub action to sign the chart

Benefits
People will trust the charts more and enterprises with proper security processes and practices will be able to tick their compliance boxes and adopt it.
Potential Drawbacks
No Drawbacks
Additional Information
No response

We will need a maintainer to add the following environment variables for this task:
HELM_SIGNING_PRIVATE_KEY
HELM_SIGNING_PRIVATE_KEY_PASSPHRASE
They can make the key themselves or I can send it to them to upload as I do not have access, and then I can make the PR

I have no access to do it @thschue  you own the repo on artifact hub

it is actually the Github repo secrets. sorry I wasn't clear. Then we only need to add the annotations on the chart.yaml for the public key

Hey @AlexsJones ! I'd like to work on this issue.


Check HPA dataset processing

I think the design for processing images could change a bit.
Reason: The part for _process_image processes downloaded inputs in a selective-channel fashion (say compose the h5 files with 'nuclei' channel only - as the user requested for it). This might create an issue later to look for new channels. The only solution then is to remove the dataset and download it with the desired channel specification.
Solution: Store each channel into separate groups under raw: (say raw/microtubules, raw/protein, raw/nuclei and raw/er)
Do we think this is a good solution or should we opt for another way?
cc: @constantinpape

I think this is a good solution, but we then also need to combine the dataset to stack the selected channels on the fly.

Updated at https://github.com/constantinpape/torch-em/pull/351.


The question on _allocate_dict_space() in libenv.py

I have a question on _allocate_dict_space() in libenv.py
Is there any reason to allot arr.data[env_idx:](all data from env_idx to the last) to buffers instead of arr.data[env_idx:env_idx+1](data for env_idx)?
for space_idx, (name, space) in enumerate(dict_space.spaces.items()):
    actual_shape = (num_envs,) + space.shape
    arr = self._numpy_aligned(shape=actual_shape, dtype=space.dtype)
    result[name] = arr
    for env_idx in range(num_envs):
        buffers[space_idx * num_envs + env_idx] = self._ffi.from_buffer(
            arr.data[env_idx:]
        )

Good catch!  In this case though it looks like the length is discarded when its assigned into the buffers variable, which is a list of void* pointers.  Are you seeing something different?
I'll fix the code upstream just for clarity regardless.

Oh, I see.
You mean self._ffi.from_buffer(arr.data[env_idx:]) is equal to self._ffi.from_buffer(arr.data[env_idx:env_idx+1]) because buffers[space_idx * num_envs + env_idx] is the pointer to arr.data[env_idx:]?

Yes, if I knew of a way to do &arr.data[env_idx] that would be the most clear, but I don't see how to write that more directly.

Specifically, while self._ffi.from_buffer(arr.data[env_idx:]) != self._ffi.from_buffer(arr.data[env_idx:env_idx+1]), once you assign them into buffers, they should be equal since the void * pointers have no length and so that must be discarded.

Thanks for your answer.
I think the current code is okay.
I was confused because I am unfamiliar with the 'buffer' concept in Python.

memoryviews and buffers are both very confusing and poorly documented


first laptime delta's value (after resetting stopwatch) is buggy

This bug seems like it was caused by the resetWatch function not resetting the previousLap.value in StopwatchView.vue.
How to reproduce:

Go to the stopwatch page
Start stopwatch
make a few laps, then reset stopwatch
make a few laps again
The first lap delta will not match

Thank you for reporting this bug! It is already fixed in 0.1.0.


Feature/npm shrinkwrap

This project uses semver, please check the scope of this pr:

[x] #patch# - backwards-compatible bug fix
[ ] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

Changes Unknown when pulling 391821143d0e9ad0ad49930e00f599edb4a27611 on feature/npm-shrinkwrap into ** on master**.


Update governance-maintainter files per new documentation.

This updates the GOVERNANCE-maintainer file in correspondence with the documentation PR for the website.
It also adds some material to the template based on working with one project in implementing it.
It also adds a template for the MAINTAINERS.md file, which is required to support this governance structure, and is needed by all projects besides.

This must be merged at the same time as the documentation PR.

I agree with the feedback from @carolynvs and don't have anything to add for the template. I did make a few suggestions on the other PR. Thanks again for taking the time to work on this!


Throw error if non existing options.environment was specified

If options.environment was set to non existing key in configuration json, then plugin will silently fail and give back unexpected config value, that includes all existing environments. See https://github.com/ajwhite/gulp-ng-config/blob/develop/gulp-ng-config.js#L65
It would be better to throw an error, so that build could exit with non-zero code if incorrect environment was specified. Alternatively this can be made optional, so that it throws only if options.thrownOnBadEnviroment or something like that. is set.
What do you think about it?

Good call, I think this should fail :+1:


reset password

When you try resetting your password, the request to reset it does not go through. I haven't received any emails and the page goes back to the home screen.

this is related to enable SMTP service with CKAN, @tharanathkartoza i think we have an SMTP service with the docker container named ckan-mailer, i've tried to use without luck, will probably will test it again, any help here would be appreciated,

@gubuntu sending emails via CKAN is a matter of changing configuration, this is on devops team court for SMTP config, the following screenshot was taken when i was working with the xml files upload with an external SMTP provider

@ntobe-nyuswa @vutomim please request SMTP service details and credentials from OCIO so we can get the system to send emails for things like password resets, registration and notifications.

When trying to reset my password its seems the site now displays error 502. The request did not complete and I did not receive and email either.


Remove version: CrystalDewWorld.CrystalDiskInfo.KureiKeiEdition version 8.12.7

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://osdn.net/frs/redir.php?m=ipconnect&f=crystaldiskinfo%2F75914%2FCrystalDiskInfo8_12_7KureiKei.exe
Response Error: The underlying connection was closed: Could not establish trust relationship for the SSL/TLS secure channel.
(Automated message - build 923)


Have the connection signal be non-const

Also make NodeGraphicsObject::NodeGraphcisObject take in a const std::shared_ptr<>& to allow rvalues.

The first thing was resolved by #38


chore: fix urls

After integrating awesome_bot we got some urls that have some issues.
We need to take care of them.

@rolandjitsu, is this the reason why my PR are failing on travis? (https://github.com/AngularClass/awesome-angular2/pull/168)

@vlio20 yes, most likely.

Is there a way to know which links? I would be happy to help with removing the broken links

@vlio20 yes, see 172297971#L456.


fix: do not overwrite previous grants

fixes #109

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


Multiple linked DS in EulerOSI

Hello,
I already noticed it to V. Acary but I want to point out that in the EulerOSI the case where an interaction has multiple linked DS does not correctly compute the sub matrices of K (tested on NonLinearRelation).

Would it be possible to get an example of such system committed? It would
help get it fixed.
On Mon, Jun 28, 2021, 12:47 PM roccaa @.***> wrote:

Hello,
I already noticed it to V. Acary but I want to point out that in the
EulerOSI the case where an interaction has multiple linked DS does not
correctly compute the sub matrices of K (tested on NonLinearRelation).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/siconos/siconos/issues/408, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALM7CDBXWAMZQNNUFU2GSLTVBHLJANCNFSM47NWRG7Q
.

Sorry for the delay.
Here an example with the bug (do siconos run.cpp to compile it).
One of the interaction is linked to two DS.
example.zip
The system and its initial conditions are rubbish, but it is enough for showing the point.
I will produce a simpler one (more readable one) a bit later, I do not have a lot of time right now.


Doc Update: Fix Stacked DataLoader example typo

I noticed that personId is the parameter of the method, but id was being used in the method.
Closes n/a

Thanks for your help.


Issue when trying to upload a world folder/zip to convert.

I am trying to convert my Java world to bedrock and I am getting the following issue:
To Reproduce
Steps to reproduce the behavior:

Go to "https://chunker.app/#/".
Click on "Choose world folder" or "Upload archive".
Wait for the upload to continue.4.
Click the button to convert and wait.
See the error in web console (developer tools), which is either "WebSocket connection to 'wss://chunker.app/' failed: The operation couldn’t be completed. (kNWErrorDomainPOSIX error 54 - Connection reset by peer)" or "The request timed out"

Expected behavior
I expected the folder to convert itself.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: macOS M1 Silicon
Browser: Safari

Additional context
I don't see why I would get the issue as my folder of the world save is only ~350 MB.
Please comment if further details need to be provided/solution is presented.

Would you be able to try a different browser? / Try with a smaller world?

Closing as no response from original reporter.


Unable to mount NFSv4 folders in TCP mode over IPv6.

Describe the bug
I am unable to mount a folder via NFSv4 via the (default) TCP mode over IPv6.
To Reproduce
Steps to reproduce the behavior:

Run something like mount -t nfs4 <server>:<mountpoint> /mnt where the <server> is either an IPv6 literal or a dns name with an AAAA record.
Run ls /mnt and get error: ls: reading directory '/mnt': Input/output error
Check dmesg and notice error: NFS4: Couldn't follow remote path

Additional debugging steps:

Add the -o vers=4.1,tcp mount options to the mount command to get additional error messages: mount.nfs: Failed to resolve server <server>: No address associated with hostname (if using a dns host name) or Address family for hostname not supported if using an IPv6 literal address

Additional context
Strangely, enabling UDP support for NFSv4 on the NFS server and mounting using no extra mount options with an IPv6 address works just fine. Only mounting over TCP/IPv6 is borked.
Metadata

system: "x86_64-linux"
host os: Linux 5.9.16, NixOS, 20.09.2635.f5f66148b28 (Nightingale)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.10
channels(root): "nixos-20.09.2635.f5f66148b28, home-manager-20.09, sops-nix"
channels(tristan): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

It turns out I needed to use tcp6 instead of tcp for IPv6 :man_facepalming:.

It turns out I needed to use tcp6 instead of tcp for IPv6 :man_facepalming:.


Create MovieSource

I'm not sure if you realize but you've submitted an empty file

Princeton Taylor Ceo & Founder


select all items

it is possible to add key to select all items, <alt + a> ?

it is possible, you have to press : <ctrl - o>


Adding KRW(Korean Won) as the currency unit

Problem
Hello,
I'm always grateful for your app with good features.
I would like to request a new feature.

Please add ‘South Korean Won(₩, KRW)’ as the currency unit.

Solution
Add ‘Korean Won’ as the currency unit.

Additional details
No response

Hey @Citatap. Sure thing. This will be added today. :)

Added in 8.5.0 :)


CI for Node

Celestia Airdrop - Claim Your Free TIA Tokens Now!

Don't Miss Out on Your Share of 60 Million $TIA Tokens!
Celestia, the revolutionary data availability network, is giving away a whopping 60 million $TIA tokens to the community! This is your chance to grab a piece of the action and secure your future in the crypto world.
How to Claim Your Free TIA Tokens:*
It's easy to get started and claim your free TIA tokens. Follow these simple steps:

Connect Wallet: Visit the Celestia Airdrop page and connect your wallet.
Claim: Click the Claim button to receive your tokens.
Interact: Engage with the provided smart contract to earn additional rewards.

Why Wait? Start Earning Today!
With Celestia, the sky's the limit. Don't let this opportunity slip through your fingers. Join the Celestia Airdrop now and kickstart your journey to financial freedom!DehanKr, @Miljar, @olzhas23, @neilghosh, @dillondanielson, @ArturT, @cooldien, @kernel164, @devayanxr, @meijies, @hieutv-dng, @irnb, @SushovanHaldar, @philipcolejohnson, @niki-quinn, @everettjf, @orkhan103, @handlemyfinance, @Charlesonr, @truonghoangnguyen


WFLY-5079 Removing non existent mail server doesnt fail, when it should.

also move mail subsystem to use capabilites

If the capability and requirement stuff is unrelated to WFLY-5079, please submit it in a separate PR.


default transform_model friendlier

I find it cumbersome to set transform_model when using DDP.

Thank you!
Could you also please set this as the default for ExtensionsManager.__init__?

Thanks for a review.
I updated the code.

can you rebase please?

Got it. Done.

/test

/test


[6.5.x] GUVNOR-2503: Asset screen Validate-button should use incremental builder

See https://issues.jboss.org/browse/GUVNOR-2503
Mirror from: https://github.com/droolsjbpm/drools-wb/pull/265 (using JDK 7)

+1


Dependency tree for tools/upgrade.go

Using gvt (and presumably other tools that support automatic recursive dependency vendoring) leads to a big bunch of dependencies being added when vendoring go-sqlite3.
$ gvt fetch github.com/mattn/go-sqlite3
2016/12/08 16:00:03 Fetching: github.com/mattn/go-sqlite3
2016/12/08 16:00:10 · Fetching recursive dependency: golang.org/x/net/context
2016/12/08 16:00:14 · Fetching recursive dependency: github.com/PuerkitoBio/goquery
2016/12/08 16:00:21 ·· Fetching recursive dependency: golang.org/x/net/html
2016/12/08 16:00:21 ··· Fetching recursive dependency: golang.org/x/text/encoding/htmlindex
2016/12/08 16:00:40 ···· Fetching recursive dependency: golang.org/x/text/encoding/charmap
2016/12/08 16:00:40 ····· Fetching recursive dependency: golang.org/x/text/encoding
2016/12/08 16:00:40 ······ Fetching recursive dependency: golang.org/x/text/transform
2016/12/08 16:00:40 ······ Fetching recursive dependency: golang.org/x/text/runes
2016/12/08 16:00:40 ······ Fetching recursive dependency: golang.org/x/text/internal/utf8internal
2016/12/08 16:00:40 ······ Fetching recursive dependency: golang.org/x/text/language
2016/12/08 16:00:40 ······· Fetching recursive dependency: golang.org/x/text/internal/gen
2016/12/08 16:00:40 ········ Fetching recursive dependency: golang.org/x/text/unicode/cldr
2016/12/08 16:00:40 ······· Fetching recursive dependency: golang.org/x/text/internal/tag
2016/12/08 16:00:40 ·· Fetching recursive dependency: github.com/andybalholm/cascadia

It looks like this is because the tool/upgrade.go file imports github.com/PuerkitoBio/goquery, which is used to extract the download URLs from https://www.sqlite.org/download.html. In turn, this package has a big dependency tree.
As tool/upgrade.go isn't an core part of the package, would renaming the tool directory to _tool (so it follows the same naming convention as the _examples  directory) be reasonable? An underscore prefix would make it possible for vendoring tool users to (optionally) ignore the directory and keep a cleaner dependency tree, in the same way that any dependencies imported in the examples can be ignored.

Ooops, sorry.