Remove the type-driven ability for duplicates in a Linker

When Linker was first created it was attempted to be created with the
ability to instantiate any wasm modules, including those with duplicate
import strings of different types. In an effort to support this a
Linker supports defining the same names twice so long as they're
defined with differently-typed values.
This ended up causing wast testsuite failures module linking is enabled,
however, because the wrong error message is returned. While it would be
possible to fix this there's already the possibility for confusing error
messages today due to the Linker trying to take on this type-level
complexity. In a way this is yet-another type checker for wasm imports,
but sort of a bad one because it only supports things like
globals/functions, and otherwise you can only define one Memory, for
example, with a particular name.
This commit completely removes this feature from Linker to simplify
the implementation and make error messages more straightforward. This
means that any error message coming from a Linker is purely "this
thing wasn't defined" rather than a hybrid of "maybe the types didn't
match?". I think this also better aligns with the direction that we see
conventional wasm modules going which is that duplicate imports are not
ever present.

This looks reasonable to me. My understanding of the duplicate-imports discussion is that there are core-wasm use cases where linking based on the name identifiers alone isn't sufficient, however it also seems like core-wasm types and signatures also aren't sufficient for those use cases, because they're too low-level in general, so they'll need to do something else anyway.

Oh that reminds me of another reason to do this. Currently Linker is sort of a weird hybrid of trying to do name-based resolution but also doing some amount of positional-based resolution. Instead though I think it should lean one of the two directions and not try to bridge the gap which in the limit seems un-bridge-able.


Fixed error with unmatched_languages list

[X] Updated Unreleased in CHANGELOG.md to reflect the changes in code or data.

Partially fixed issue #432.
The program was not adding unmatched languages to the unmatched_languages list because the try/except statement to catch if a language was unmatched was continueing past the statement that added the language to the list.

Commit a05e1d9 should've been titled "Added casefold attribute in languages.json", not "Removed line", that was an accident.


Audit usize usage for 16-bit safety

Inspired by the looming feasibility of Rust on AVR, in the serialization code I've started to use checked pointer math to make sure we don't overflow usize. There are other places where I'm pretty sure we assume that usize is at least 32 bits, so we should find those and add appropriate checking.

In case it ever comes in handy, I experimented with deserializing the payload in batches: https://github.com/marshallpierce/hdrsample/commit/fc52e75748a62088a300c8d5d595ff7c868bb1fe
This avoids being limited by usize for the overall payload length (which can be up to 9x the size of the counts array if the counts were all large u64 values). I tried it hoping for a speed win due to better cache locality, but the increased complexity of the logic seems to have ended up making it a wash.
I don't imagine that people will typically be deserializing histograms on tiny systems, and I don't feel like it's worth merging in the extra complexity and possible tiny speed loss for an unlikely use case, at least not right now.


Update problem

When I updated Cats plugin it shows this error https://imgur.com/clB8l0y
I can't use it right now, What I have to do?

please try to uninstall Cats completely, restart Blender and then reinstall it


Make a 2D plot for each exposure rather than each grating

@RAShaw-sci this should fix the overwriting problem for 2D STIS plots.

@RAShaw-sci I think you already tested this, right? If so, can you approve the PR and I can merge it.


Strip off package name for telemetry policy

If a fully qualified module name is provided.

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

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

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

Regression introduced in https://github.com/Azure/azure-sdk-for-go/pull/21893


Cygwin Build Is Broken Again

$ make -j distclean
$ tools/configure.sh -c stm32f4discovery:nsh
$ make
...
CC:  readline.c
CC:  readline_common.c
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/system/readline'
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/builtin'
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file
make[3]: *** [/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:143: archive] Error 1
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/builtin'
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/examples/hello'
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file
make[3]: *** [/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:143: archive] Error 1
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/examples/hello'
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/nshlib'
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file
make[3]: *** [/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:143: archive] Error 1
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/nshlib'
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/platform'
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file
make[3]: *** [/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:143: archive] Error 1
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/platform'
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/system/nsh'
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file
make[3]: *** [/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:143: archive] Error 1
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/system/nsh'
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/system/readline'
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file
make[3]: *** [/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:143: archive] Error 1
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/system/readline'
make[2]: *** [Makefile:99: /cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/libapps.a] Error 2
make[2]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps'
make[1]: *** [Makefile:53: all] Error 2
make[1]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps'
make: *** [tools/LibTargets.mk:225: /cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/libapps.a] Error 2

This is a blocker; we cannot make any release with the Cygwin build broken.

With make V=1 I can see this is in the archive target:
...
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/system/readline'
for app in /cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/builtin /cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/examples/hello /cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/nshlib /cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/platform /cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/system/nsh /cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/system/readline; do \
        make -C "${app}" archive ; \
done
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/builtin'
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file
make[3]: *** [/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:143: archive] Error 1
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/builtin'
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/examples/hello'
...

Adding some debug output:
diff --git a/Application.mk b/Application.mk
index d1dba985..e3e250ac 100644
--- a/Application.mk
+++ b/Application.mk
@@ -140,6 +140,8 @@ $(CXXOBJS): %$(SUFFIX)$(OBJEXT): %$(CXXEXT)

 archive:
 ifeq ($(CONFIG_CYGWIN_WINTOOL),y)
+       $(warning BIN=$(BIN) -> "${shell cygpath -w $(BIN)}")
+       $(warning OBJS=$(OBJS))
        $(call ARCHIVE_ADD, "${shell cygpath -w $(BIN)}", $(OBJS))
 else
        $(call ARCHIVE_ADD, $(BIN), $(OBJS))

Does not shed any light on this for me:
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/builtin'
/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:143: BIN=/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/libapps.a -> "D:\Spuda\Documents\projects\nuttx\master\apps\libapps.a"
/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:144: OBJS=  builtin_list.cygdrive.d.Spuda.Documents.projects.nuttx.master.apps.builtin.o exec_builtin.cygdrive.d.Spuda.Documents.projects.nuttx.master.apps.builtin.o
/bin/sh: -c: line 0: unexpected EOF while looking for matching `"'
/bin/sh: -c: line 1: syntax error: unexpected end of file
make[3]: *** [/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:145: archive] Error 1
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/builtin'

I suspect it may be this line in ARCHIVE_ADD definition:
@echo "AR (add): $(notdir $1) $(2)"

Maybe the quotes in $1 are nested in the outer ones and bash expects these to be escaped

I'm guessing that quotes are handled different in cygwin.

I am thinking that I should see the AR command if was executed.  Hence, I am thinking the problem my be in the notdir in Config.mk:
define ARCHIVE_ADD
        @echo "AR (add): $(notdir $1) $(2)"
        $(Q) $(AR) $1 $(2)
endef

In that case $1 will be a windows path and notdir is possibly failing on that?
Yes, if I just remove the notdir:
diff --git a/tools/Config.mk b/tools/Config.mk
index c3eb0303b0..7bfc30efcd 100644
--- a/tools/Config.mk
+++ b/tools/Config.mk
@@ -317,7 +317,7 @@ endef
 #   CONFIG_WINDOWS_NATIVE - Defined for a Windows native build

 define ARCHIVE_ADD
-       @echo "AR (add): $(notdir $1) $(2)"
+       @echo "AR (add): $(1) $(2)"
        $(Q) $(AR) $1 $(2)
 endef

Then the build succeeds:
make[3]: Entering directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/builtin'
/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:143: BIN=/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/libapps.a -> "D:\Spuda\Documents\projects\nuttx\master\apps\libapps.a"
/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/Application.mk:144: OBJS=  builtin_list.cygdrive.d.Spuda.Documents.projects.nuttx.master.apps.builtin.o exec_builtin.cygdrive.d.Spuda.Documents.projects.nuttx.master.apps.builtin.o
AR (add):  D:SpudaDocumentsprojectsnuttxmasterappslibapps.a builtin_list.cygdrive.d.Spuda.Documents.projects.nuttx.master.apps.builtin.o exec_builtin.cygdrive.d.Spuda.Documents.projects.nuttx.master.apps.builtin.o
arm-none-eabi-ar rcs  "D:\Spuda\Documents\projects\nuttx\master\apps\libapps.a"    builtin_list.cygdrive.d.Spuda.Documents.projects.nuttx.master.apps.builtin.o exec_builtin.cygdrive.d.Spuda.Documents.projects.nuttx.master.apps.builtin.o
make[3]: Leaving directory '/cygdrive/d/Spuda/Documents/projects/nuttx/master/apps/builtin'

Although the displayed path is wrong.
@v01d The notdir was added by your commit 34b34e2d451 and this breaks the Cygwin build.  What would you propose?

I'm guessing that quotes are handled different in cygwin.

They shouldn't be different.  Same GNU make, same Bash.  The real difference is the Windows path.  I think that is what is making ARCHIVE_ADD puke.

So you mean notdir does not handle backslashes?

So you mean notdir does not handle backslashes?

That is my thought.  I am wondering why it is the full path to the tool?  I used to be only the tool name which is still arm-non-eabi-ar.  When why you add the full path, then you grow backslashes where not should be.

The notdir applies to the library path, so that it just reads AR (add): libapps.a. It is purely cosmetic. You could remove it and let it show the full path or maybe it needs quotes like $(notdir '$1') or $(notdir "$1"). I'm guessing the backslash is interpreted by bash as an escape.

The notdir applies to the library path, so that it just reads AR (add): libapps.a. It is purely cosmetic. You could remove it and let it show the full path or maybe it needs quotes like $(notdir '$1') or $(notdir "$1"). I'm guessing the backslash is interpreted by bash as an escape.

I don't think I an do that, the echo argument is already enclosed like:  @echo "AR (create): $(notdir $1) $(2)"

As a general rule, make functions will not accept Windows paths.  In this case, notdir breaks the build.  The last time it was broken, Xiang replaced the Bash dirname (which does handle windows paths) with the make dir command (which does not).

Ok, if you think notdir is the issue, we can just remove it.

See PR #1871

Ok, if you think notdir is the issue, we can just remove it.

yes, notdir with a Windows path is the issue.  I have a PR will submit in a moment which fixes the issue and retains the cosmetic output.

@v01d @btashton We really need to get some minimal verification of the Cygwin build into the PR checks.  I know that Brennan has done work on this and my understand is that there is hesitancy because the automated Cygwin build is very slow.  Is that true?

I was thinking the same, that we should try adding it to CI, maybe just running one config.

I was thinking the same, that we should try adding it to CI, maybe just running one config.

There is my PR. I'll clean it up and add an ARM target with it. Need to reread what you wrote about the toolchain. That's what held it up months ago, but you responded I just need to read.


[NNBD] No type promotion for a global variable

The following code fails in both analyzer and CFE with --enable-experiment=non-nullable option
class A {}
class C extends A {}

foo(C c) {}

A aGlobal = new C();

main() {
  A aLocal = new C();
  if (aLocal is C) {
    foo(aLocal);  // Ok
  }
  if (aGlobal is C) {
    foo(aGlobal);  // The argument type 'A' can't be assigned to the parameter type 'C'.
  }
}

Without --enable-experiment=non-nullable there's no error and the code works.
Dart VM version: 2.9.0-11.0.dev (dev) (Tue May 26 12:14:57 2020 +0200) on "windows_x64"

I believe you're seeing the following: Pre-null-safety code will not promote (because global variables are never subject to promotions—any non-trivial computation could change it, so it's almost never sound to assume that it hasn't changed), and null-safety code also won't promote it. But with pre-null-safety downcasts, there is no error. So this is working as expected.


error_code: remove default implementation

What problem does this PR solve?
default  to unknown means the compiler cannot tell us if error codes
are working. It is easy to accidentally end up with the unknown code.
It also requires all implemntations to use specialization.
What is changed and how it works?
Remove defaulting to unknown
use an error_unknown macro if there is no error code to set it to
unknown.
Related changes
None
Check List
Tests
Tested a case where the error code shows up after these changes. Previously wrappers were getting defaulted to unknown but are now properly dereferenced.
Side effects
Code may change from Unknown to a more specific code.
Release note
Fix error code propagation

/assign @5kbpers

/merge


Fork with new experimental functional that allow full source code modification in compiler pipeline

Hello All!
I create fork this project with new functional that allow full code modification in compiler pipeline  - add new piece of code, replace existing code to new one, remove existing code.
Added capability to ordered transform one piece code with many generators defined on it.
AOP Welcome :)
Sample OrderedGeneratorsSolution solution added in samples folder.
Example of usage:
    [TryCatchOrdered(ExecutionOrder = 1)]
    [DuplicateWithSuffixByNameOrdered("t", ExecutionOrder = 2)]
    [RemoveOrdered(ExecutionOrder = 3)]
    public partial class Foo
    {
        [DuplicateWithSuffixByNameOrdered("t", ExecutionOrder = 1)]
        public object `Prop1;

        [DuplicateWithSuffixByNameOrdered("k", ExecutionOrder = 1)]
        public int? Prop2;

        public void Test()
        {
            var aa = 1;
            var dd = 3;
            var bb = 0;
            var cc = aa / bb;
        }
    }

Each attribute has ExecutionOrder property for order transform current node
Also added capability to transform fields or  properties.
After transformation current piece of code will be replaced to:
    [TryCatchOrdered(ExecutionOrder = 1)]
    [DuplicateWithSuffixByNameOrdered("t", ExecutionOrder = 2)]
    [RemoveOrdered(ExecutionOrder = 3)]
    public partial class Foot
    {
        [DuplicateWithSuffixByNameOrdered("t", ExecutionOrder = 1)]
        public object Prop1;
        [DuplicateWithSuffixByNameOrdered("t", ExecutionOrder = 1)]
        public object Prop1t;
        [DuplicateWithSuffixByNameOrdered("k", ExecutionOrder = 1)]
        public int? Prop2;
        [DuplicateWithSuffixByNameOrdered("k", ExecutionOrder = 1)]
        public int? Prop2k;
        public void Test()
        {
            try
            {
                var aa = 1;
                var dd = 3;
                var bb = 0;
                var cc = aa / bb;
            }
            catch (Exception ex)
            {
            }
        }
    }

Transformations:

All methods wrapped to try catch block (in future generator may transformed for write all methods exceptions in log)
Created new copy of class Foo (after step 1) - Foot
Old class Foo will be removed

For create you own generation attribute - inherit this from  OrderedCodeGenerationAttribute.
For access full code transformation capability in generator - inherit this from IRichCodeGenerator, in GenerateRichAsync method assign RichGenerationResult.Members to list of ChangeMember, each member can do one of tree modification AddMember,   ReplaceMember, RemoveMember.
Not done yet:

speed optimization
partial regeneration when one part of partial entity (class, struct, interface) is changed.

Fork at:
GoldyD

My fork has same restriction as source project that can only generate new code.
In VS generation running in background, when base code class that has generator attributes will be changed and saved to disk.

WPF design time builds (happen in the background and also rewrite stuff, do they work properly?)
Not tested  - I think it will be worked (you can try)
intellisense code completion (in particular when editing code which is rewritten/replaced)
Worked
debugging code which was rewritten (also editing while debugging)
Partially worked - show debugger generated method body, but editing while debugging not supported

I don't think I'd ever go for this because the line numbers reported in exceptions, crashes, etc would not line up with source code. You're welcome to continue the effort of course, but please do so on your own forum.


Refactor migration controller

Remove an excessive token from allowed migration names regular expression.

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
none

Nope, it is not excessive.
\ in group should be escaped with \, but \ in ' (single quotes) string should be escaped with \ as well.
So we get escaped backslash (\\) escaped with backslash (\\\\)

@SilverFire thanks for the explanation!


chore(ChaosExperiment): Add Concepts docs for Chaos Experiment

Signed-off-by: Udit Gaurav<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Checklist:

[ ] Fixes #
[ ] Signed the commit for DCO to be passed
[ ] Labelled this PR & related issue with documentation tag

I am highly interested in improving the documentation part for Litmus Chaos. Si i  request maintainer to assign issue #387 to me !


Fix the Hyperband configuration formula

As highlighted in my blogpost [1], the formula used to computer the number of configurations for the successive halving step in hyperband deviates from the paper. In fact, there is an additional rounding operation on the s_max / (s+1) factor. Everything is explained in the blog post.
I also do not know why you are using an int() operator to round the value of n0 because it is originally a ceil operation. I suppose that this is why you need an additional max() in the computation of ns. The code is a bit intricated to me so I guess you can better tell than me. Do not hesitate to tell me if I can be of help or I am wrong.
[1] https://medium.com/data-from-the-trenches/a-slightly-better-budget-allocation-for-hyperband-bbd45af14481

Codecov Report

Merging #89 (0823035) into master (841db4b) will decrease coverage by 0.37%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #89      +/-   ##
==========================================
- Coverage   65.16%   64.78%   -0.38%
==========================================
  Files          28       28
  Lines        1846     1846
==========================================
- Hits         1203     1196       -7
- Misses        643      650       +7


Apostrophe doesn't register, so typing test gets stuck

Expected Behavior
If I type an apostrophe ' then it should register.
Actual Behavior
It doesn't register.
Steps to Reproduce the Problem
Inconvenient to reproduce since words in typing test are random, but with the project-gutenberg-2k list, the word haven't appears and once you reach the apostrophe, you cannot continue since ' isn't accepted by the app.

Addressed in #43 when fixing accessibility issues with typing area


Let CMake build position independent code by default

Configure CMake to compile static libraries with position independent code, as discussed in #451.
Closes #451.

@marbre why this is closed?

Because it landed in the repo. This was from a long time ago when we were using a weird flow for landing PRs: it marked things closed vs merged.

@stellaraccident Thanks. I missed that.


Strava - Get method for fetching GPX

will have to create the array of objects for Taddeo to interpret the distance


Make ctags executable configurable

Is your feature request related to a problem? Please describe.
When running ':Clap debug' just now, it complained about not being able to find ctags with proper JSON support.
Describe the solution you'd like
Variable with default value (e.g. vim.g.clap_ctags_exec) that can be explicitly set if required (e.g. 'uctags').
Describe alternatives you've considered
None, currently just patched source to use uctags instead of ctags for checking JSON support.
Additional context
Searched for configuration variable, but didn't find one. If I missed it, please let me know.

Why don't you just make an alias ctags for uctags? I mean I'm not against adding this support, but feel it not absolutely necessary as it seems not hard to work around it on your side.

Closing as this issue can be fixed by yourself without much effort, e.g., add alias ctags='uctags' to your bashrc. Feel free to reopen if this does not help.


First few images broken on Docker hub

https://hub.docker.com/r/dorowu/ubuntu-desktop-lxde-vnc/
the first few images on that page don't seem to load anymore

https://www.google.com/search?q=report+bug+how

@fcwu .... it's not a bug.... It's just that somehow the images on the docker hub page don't work anymore....................................
Screenshot: https://imgur.com/a/hTUb733

Thanks @thibaultmol I got the problem
Fixed in https://github.com/fcwu/docker-ubuntu-vnc-desktop/commit/4f187de35bf45ab1997b324039f8e29a0b3e8e57


Simplify static files and remove login manager

Simplify the restricted document handling introduced for #281 in #319 and remove the SipaLoginManager.
Please make sure though you did the following (tick off everything you
already did):

[x] Run the tests and see them pass
[x] Rebase your branch on top of develop
[x] Include tests for features you introduced / bugs you fixed

I'm just noticing that your changes rendered the tests unusable, because they just tested the custom login manager, and not the blueprint themselves. Unfortunately, I don't have time to look into this, so you will need to fix this before I can merge.

Since – judging by current observations – our user_loader exemption does not even work at the moment, we can

[ ] rebase this branch
[ ] implement a member_required decorator to account for ad779d13b57376ed1ab529f9160d30fe0a9a5efc
[ ] test the functionality

The decorator could look like this (analogous to the login_required implementation):
def member_required(func):
    @wraps(func)
    def decorated_view(*args, **kwargs):
        if request.method in EXEMPT_METHODS or current_app.config.get("LOGIN_DISABLED"):
            pass
        elif not current_user.is_authenticated \
                or current_user.is_authenticated and not current_user.is_member:
            return current_app.login_manager.unauthorized()
        return func(*args, **kwargs)

    return decorated_view


Fix iterator.next() being called twice in Publishers.Sequence

LGTM

  Generated by :no_entry_sign: Danger Swift against 97a9255c37349e6c640134bdfdf300e4799c6ec9


Allow option to specify image pull policy for component container image

/kind feature
Which functionality do you think we should add?
Odo should allow users to specify an image pull policy for a component's container image.
Why is this needed?
Using local images for components is particularly necessary when testing out stack image changes. Without being able to change the image pull policy, a local image cannot be used.

Related to: https://github.com/devfile/api/issues/89

/remove-lifecycle rotten
/area devfile
/lifecycle frozen
Since there's already an issue about this on devfile side, I'm setting lifecycle/frozen to this issue. It first needs to be addressed in devfile api and library, IMO.

recently released odo v3.6 added support for pod-overrides and container-overrides. This devfile features allows specifying imagePullPolicy.
Demo can be found in release notes https://odo.dev/blog/odo-v3.6.0/#extend-pod-and-container-specification-with-devfile-attributes-pod-overrides-and-container-overrides
adding the following lines to container component will change imagePullPolicy
    attributes:
      container-overrides:
        imagePullPolicy: IfNotPresent

/close


[QUESTION] Error when entering "Custom Learning"

WEB PART NOT LOADING: If you are getting the error: "Microsoft 365 learning pathways has a configuration issue. Ask your administrator for assistance". Please review the Diagnosing Pre-Check guidance before submitting your bug.

Submit your question
What is the context of your question

[] Installation
[] M365 learning pathways web part

What is your question
We provisioned a Learning Pathways site yesterday. When doing so, we received the error "Microsoft 365 learning pathways has a configuration issue. Ask your administrator for assistance". As such, we deleted the site, and re-provisioned it. This time around, we got the following error when attempting to enter the Custom Learning module:
"Error: The value for "_instanceId" must not be undefined
in t in t in o in t in t in t in t in t in section in div in div in t in Styledt in t in o in t in t in t"
Can someone please assist regarding what might be wrong?
Thanks.

Provide related screen-shots to help us understand your question

Can you validate how you provisioned it, was it using the lookbook service?  If so I would suggest that you delete the site collection again, but this time also delete the solution out of the tenant app catalog. Also, make sure you permanently delete the site collection in the admin center under "Deleted Sites".

Since we haven't heard back I'm going to assume that you've either corrected the issue or have moved on.  We'd be happy to help further if you are continuing to have an issue please open a new issue and reference this one. I'd also suggest you follow some of the guidance outlined here as that can help.

WEB PART NOT LOADING: If you are getting the error: "Microsoft 365 learning pathways has a configuration issue. Ask your administrator for assistance". Please review the Diagnosing Pre-Check guidance before submitting your bug.


[SPARK-21929][SQL] Support ALTER TABLE table_name ADD COLUMNS(..) for ORC data source

What changes were proposed in this pull request?
When SPARK-19261 implements ALTER TABLE ADD COLUMNS, ORC data source is omitted due to SPARK-14387, SPARK-16628, and SPARK-18355. Now, those issues are fixed and Spark 2.3 is using Spark schema to read ORC table instead of ORC file schema. This PR enables ALTER TABLE ADD COLUMNS for ORC data source.
How was this patch tested?
Pass the updated and added test cases.

Test build #82939 has started for PR 19545 at commit cc52547.

Test build #82939 has finished for PR 19545 at commit cc52547.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82939/
Test PASSed.

Hi, @gatorsmile . Could you review this PR?

LGTM

Thank you for review, @gatorsmile ! I updated the PR.

Test build #82942 has started for PR 19545 at commit dfc59fc.

Test build #82942 has finished for PR 19545 at commit dfc59fc.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82942/
Test FAILed.

Retest this please.

Test build #82949 has started for PR 19545 at commit dfc59fc.

Test build #82949 has finished for PR 19545 at commit dfc59fc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82949/
Test PASSed.

Merged build finished. Test PASSed.

It’s ready for review again, @gatorsmile. Thanks.

Thanks! Merged to master.

Thank you a lot, @gatorsmile !


Define multiple tags and remove suites in behat.yml

I want to define multiple tags in behat.yml which correspond to suite names. There are 4 suites inside the features folder and each of them have multiple .feature files.
Ex: admin, themes
Tags to be defined: @admin, @themes.
Previously the behat.yml file contained suites for admin and themes. Now I want to define 3 different profiles (server1, server2 and server3) corresponding to different testing environments and use tags instead of suites to run the feature files containing those tags. I have added the tags @admin and @themes in every feature file of themes and admin folder.
How should I implement this specific case in my config file? Any assistance is greatly appreciated

You can specify few tags:
tags: @admin&&@themes

Or specify not (~) tags:
tags: ~@not-automated&&~@skip

This is related to behat, so please close this issue prior to https://github.com/Behat/Behat/issues/1072


Proposal: Improve Generator Schematics documentation

Bug, feature request, or proposal:
Proposal
What is the expected behavior?
On this page of the docs, https://material.angular.io/guide/schematics#dashboard-schematic, it gives the following command: ng generate @angular/material:material-dashboard
I expect this to work.
What is the current behavior?
This command fails with the following error:
Schematic input does not validate against the Schema: {"dryRun":false,"inlineStyle":false,"inlineTemplate":false,"changeDetection":"Default","styleext":"css","spec":true,"flat":false,"skipImport":false,"export":false}

Data path "" should have required property 'name'.

This command fails because it omits the --name flag: ng generate @angular/material:material-dashboard --name=my-dashboard
What are the steps to reproduce?
Run ng generate @angular/material:material-dashboard.
What is the use-case or motivation for changing an existing behavior?
When I find a command in the official docs, I expect it to work on the first try. I had to do some research to figure out why it wasn't working.
Either the error message should be improved, or get the CLI to use a default name, or mention the fact that the command will fail without the --name flag.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 6.0.0
Material 6.0.0
Is there anything else we should know?

Someone just sent #11283 for this

Fixed in #11283


rootlocation does not correctly work

Using the rootlocation in a multi module build and having a sub module which contains rootlocation.
Calling mvn like this:
mvn -pl submodule package

Does not seemed to work correctly.

Did you mean this issue by that?
[INFO] Processing change of org.identifier:SubModule:1.0.139-SNAPSHOT -> 1.0.140-SNAPSHOT
[DEBUG] Applying change null:SubMlodule:1.0.139-SNAPSHOT -> 1.0.140-SNAPSHOT
[ERROR] Failed to execute goal org.codehaus.mojo:versions-maven-plugin:2.2:set (default-cli) on project SubModule: Execution default-cli of goal org.codehaus.mojo:versions-maven-plugin:2.2:set failed. NullPointerException -> [Help 1]

This is related to versions-maven-plugin. Apart from that you are referencing a very old version. This issue of the versions-maven-plugin has already been fixed with version 2.4...

This issue has been open for more than 3 years, the rootlocation goal is useless in multi-module builds (and in single-module builds nobody needs it). Is anything wrong with PR #48? Can someone please review it, then merge and/or manually fix the problem and release a new version? That would be wonderful.

i'll take a look. the ITs do not run on windows, but that should be fixed easily.

the PR #84 is not sufficient - it works only when the parent pom = aggregation root pom. if you have a parent that is a child pom separate from the aggregation root pom it returns the path of the parent pom which is not the intention.
i've created a new PR #92 which should work in all cases.
@kriegaex @tmcsantos can you verify that the PR works in your scenarios as well?

Oh, I forgot a detail: The build in your branch runs with version number 3.1.0-SNAPSHOT, but 3.1.0 is the current release already. Should it not rather be 3.1.1-SNAPSHOT then?

OMG, I just noticed why I had the wrong snapshot version: I accidentally built and tested the original PR, not the improved one. Yours links to the original and I just clicked the wrong link in order to get to the repository to clone from. Your PR also works for me.

Quick question: Yesterday you released the new version 3.2.0, but it still is unavailable on Maven Central. Usually my experience when releasing to Central is that it does not take that long to be published. Did you maybe forget to deploy the artifact?

it's under voting process which takes at least 72h - see mail thread
if no objection is raised it's published to maven central when the voting is completed.

Oh, of course.  This is OSS with a release process, not my private project which I can push to Maven Central anytime.


h2spec http2/6.5.3/1 -p 9000 -o 5 : it hangs

./h2spec --version
Version: 2.3.0 (cb03b47353d2691ede927b1d4a4ad9c417aeee60)
I have a very simple code: a socket accepting and printing the metadata. In ruby:
require "socket"

puts "Starting server on port 9000"
server = TCPServer.new(9000)

loop do
  sock = server.accept

  while !sock.closed? && !(sock.eof? rescue true) # rubocop:disable Style/RescueModifier
    puts sock.readpartial(1024).inspect
  end
end

When running the aforementioned spec, it hangs for 5 seconds, after which it opens a connection and immediately closes.
This doesn't seem correct, can anyone confirm it's a bug?

@summerwind I've narrowed this down to this snippet; the connection is somehow not properly established. If I comment this block, my test passes.
I've compared both created "conn"s, and the only different seems to be that the conn that passes the test has the max concurrent streams option set, and the other one hasn't:
# this one blocks on .Handshake()
conn: &{Conn:0xc0000b0270
Settings:map[]
Timeout:5s
Verbose:false
Closed:false
WindowUpdate:true
WindowSize:map[0:65535]
framer:0xc0001200c0
encoder:0xc0000b2600
encoderBuf:0xc00009b4d0
decoder:0xc000136480
debugFramer:<nil> debugFramerBuf:<nil>
server:false}

# this one is successfully initiated
conn: &{Conn:0xc00012a000
Settings:map[MAX_CONCURRENT_STREAMS:100]
Timeout:5s
Verbose:false
Closed:false
WindowUpdate:true
WindowSize:map[0:65535]
framer:0xc00012c000
encoder:0xc000126060
encoderBuf:0xc000128090
decoder:0xc000130000
debugFramer:<nil> debugFramerBuf:<nil>
server:false}

Do you have any idea on why this happens, or whether that check is really necessary?

Can you show me the output of h2spec http2/6.5.3/1 -p 9000 -o 5 -v ?

sorry for the delay: here it is:
./h2spec http2/6.5.3/1 -p 9000 -o 5 -v
Hypertext Transfer Protocol Version 2 (HTTP/2)
  6. Frame Definitions
    6.5. SETTINGS
      6.5.3. Settings Synchronization
             [send] SETTINGS Frame (length:6, flags:0x00, stream_id:0)
        × 1: Sends multiple values of SETTINGS_INITIAL_WINDOW_SIZE
        Error: Timeout

Thank you for the log.
This is not a bug. h2spec will wait a SETTINGS frame from your server to complete HTTP/2 handshake. So your server must send a SETTINGS frame.
See also: https://tools.ietf.org/html/rfc7540#section-3.5

Are you sure? The verbose log says that h2spec sent a settings frame, but my server never receives it, in fact, not even the magic handshake string arrives. Socket gets accepted and times out on read.

Yes, I can see that your server's code receives handshake string and SETTINGS frame as follows.
$ ruby server.rb
Starting server on port 9000
"PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n\x00\x00\x06\x04\x00\x00\x00\x00\x00\x00\x04\x00\x00\xFF\xFF"

Hmm, I don't know why the same message doesn't appear in your environment.

I'm using a macOS, what about you?
I've pulled wireshark, and I do see the packets going to the server, but the last one (that contains the settings) is marked as "101", not sure what that means... but I'm not getting the same as you.

I'm using macOS Catalina and tested with h2spec v2.3.0.

I've opened a separate branch to run h2spec unpatched, and I'm getting the same errors in the CI: https://gitlab.com/honeyryderchuck/http-2-next/-/jobs/365589322 .


Authorfeedback for interactive marquee

Add your specific features or fixes

Resolves: [MWPW-139795]
Test URLs:

Before: https://main--cc--suhjainadobe.hlx.page/drafts/suhjain/in-marquee
After: https://auhtorfeedback--cc--suhjainadobe.hlx.page/drafts/suhjain/in-marquee

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b2953fa) 100.00% compared to head (43a5e50) 25.26%.

Additional details and impacted files
@@             Coverage Diff              @@
##              main     #119       +/-   ##
============================================
- Coverage   100.00%   25.26%   -74.74%
============================================
  Files            1        1
  Lines           90       95        +5
============================================
- Hits            90       24       -66
- Misses           0       71       +71

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

PSI checks are failing as .page in cc is restricted to VPN. Please add .live urls in the description.

@suhjainadobe PSI checks are failing as .page in cc is restricted to VPN. Please add .live urls in the description. Also could you add relevant description to the ticket to add the context to this PR.

We do not have it on live pages. It will work only on .page


CPU stuck at 0% on 1.4.2 update version

Describe the issue
After updating to 1.4.2 everything is working again except the CPU usage, shows only 0%
Expected behavior
The CPU load percentage should change continuously
Screenshots

Software versions

Home Assistant version: < HA 2023.9.2>
openmediavault integration version: < v1.4.2 >
openmediavault software version: < 6.9.0-1 (Shaitan) >

Additional context
The system is installed on 8GB Raspberry pi 4

No related to v1.4.2
duplicate to #133


Random Data UI output refinements

When inserting random data in 1 table, this is the output:

Suggestions

Let's highlight the table name with backticks and a different color
Let's omit the last line if just 1 table, since the total number of records is the same as the first line. This will also remove the 1 tables error, and only show for 2 or more tables.

We did refinements regarding this.


Update Code - Proof_Of_Contribution.md

proof of contribution for whitepaper translation

Thanks. Please provide a link to which Whitepaper.

Ah I see the Hindi pull request now. Thanks.


Adds voice category prefix per channel

Set the preferred voice category name for voice channels created for games in a given channel using /set voice_category in that channel.

Codecov Report

Merging #679 (40988a7) into main (88df478) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 40988a7 differs from pull request most recent head 6d818de. Consider uploading reports for the commit 6d818de to get more accurate results

@@            Coverage Diff            @@
##              main      #679   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           67        68    +1
  Lines         2889      2922   +33
  Branches       308       309    +1
=========================================
+ Hits          2889      2922   +33

Impacted Files
Coverage Δ

src/spellbot/settings.py
100.00% <ø> (ø)

src/spellbot/cogs/admin_cog.py
100.00% <100.00%> (ø)

src/spellbot/interactions/admin_interaction.py
100.00% <100.00%> (ø)

src/spellbot/interactions/lfg_interaction.py
100.00% <100.00%> (ø)

src/spellbot/interactions/task_interaction.py
100.00% <100.00%> (ø)

...ns/6e982c9318a6_adds_voice_category_per_channel.py
100.00% <100.00%> (ø)

src/spellbot/models/channel.py
100.00% <100.00%> (ø)

src/spellbot/services/channels.py
100.00% <100.00%> (ø)

src/spellbot/services/guilds.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Extend logging for better correlation and retry visibility

Summary
Each of the core Event Hub client and primitive types are instrumented to perform operation-level logging using the EventHubsEventSource, and following the pattern of capturing Start, Stop, and Exception flow, as well as important context for the specific operation.
While the logging attempts to capture a rich set of information, it currently does not offer an operation identifier to help correlate entries for a single operation nor does it surface the current retry count, which would be helpful for diagnosing latency issues.
Scope of Work

Introduce an artificial Operation Identifier for each operation-level logging block that covers the Start, Error, and End events in that group.  Each should use the same operation identifier to allow an operation to be correlated end-to-end.

The Error events for an operation should surface the current retry count, starting with 0 so that it is visible to listeners when an operation experienced transient issues that were retried.

Events in both the Azure.Messaging.EventHubs and Azure.Messaging.EventHubs.Processor packages should be enhanced.

Success Criteria

The enhancements detailed in the scope have been implemented.

The tests necessary for its validation have been created or adjusted and pass reliably.

The existing test suite continues to pass and produce deterministic results.

Closing as a duplicate of #21440


Update maxBuffer size for runCommand

Update maxBuffer size for runCommand to 2MB
Fixes #873

Pull Request Test Coverage Report for Build 2242

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 68.012%

Totals

Change from base Build 2226:
0.0%

Covered Lines:
4076

Relevant Lines:
5679

💛  - Coveralls

this fixes #873


Unable to download YouTube shorts videos

Checklist

[ ] I'm reporting a bug unrelated to a specific site
[x] I've verified that I'm running yt-dlp version 2021.08.10
[x] I've checked that all provided URLs are alive and playable in a browser
[x] The provided URLs do not contain any DRM to the best of my knowledge
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

[debug] User config file: /data/data/com.termux/files/home/.config/yt-dlp/config

[debug] User config: ['-o', '%(upload_date)s - %(uploader)s -  %(title)s.%(ext)s', '--embed-thumbnail', '--add-metadata', '--default-search', 'ytsearch3']

[debug] Command-line config: ['-Fv', 'https://youtube.com/shorts/BGQWPY4IigY']

[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8

[debug] yt-dlp version 2021.08.10

[debug] Python version 3.9.6 (CPython 64bit) - Linux-4.4.153-perf+-aarch64-with-libc

[debug] exe versions: ffmpeg 4.4, ffprobe 4.4

[debug] Optional libraries: mutagen, pycryptodome, sqlite, websockets

[debug] Proxy map: {}

[debug] [generic] Extracting URL: https://youtube.com/shorts/BGQWPY4IigY

[debug] [youtube:tab] Extracting URL: https://consent.youtube.com/m?continue=https%3A%2F%2Fyoutube.com%2Fshorts%2FBGQWPY4IigY&gl=GB&m=0&pc=yt&uxe=23983171&hl=en&src=1

WARNING: [youtube:tab] A channel/user page was given. All the channel's videos will be downloaded. To download only the videos in the home page, add a "/featured" to the URL

[debug] [youtube:tab] Final URL: https://www.youtube.com/m/videos?continue=https%3A%2F%2Fyoutube.com%2Fshorts%2FBGQWPY4IigY&gl=GB&m=0&pc=yt&uxe=23983171&hl=en&src=1

Description

Unable to download YouTube shorts videos using yt-dlp. I'm using termux on Android if that's relevant.
youtube-dl works though.
Example YouTube shorts URL:
https://youtube.com/shorts/BGQWPY4IigY
Issue #375 ?
Thanks for reading

Normally this would work since the generic extractor would follow a redirect to the normal watch page. But that doesn't always happen by the looks - in this case, it has redirected to the consent screen.
So we probably do need to update _VALID_URL, as @pukkandan appears to have looked at in #375 but did not go through with it.

Thanks very much.


package io: read method can not return if no "EOF"

I found a bug in this package (I used version 1.6)
I use the following code:
io.Copy(local_conn, remote_conn)
And actually, it will call
for { 		nr, er := src.Read(buf) 		if nr > 0 { 			nw, ew := dst.Write(buf[0:nr]) 			if nw > 0 { 				written += int64(nw) 			} 			if ew != nil { 				err = ew 				break 			} 			if nr != nw { 				err = ErrShortWrite 				break 			} 		} 		if er == EOF { 			break 		} 		if er != nil { 			err = er 			break 		} 	}
if the data source doesn't include EOF, it will never return from "Read". How to handle it?

This is not a bug. This is the way io.Copy was designed and also the basic idea for IO in go.
The first sentence in the description of io.Copy explains this:
Copy copies from src to dst until either EOF is reached on src or an error occurs.

Most functions that do some kind of IO use io.EOF to signal the end of a stream of data. The functions and interfaces in the io package are built around this idea.
Seeing it from another angle, without the io.EOF many functions wouldn't know when to stop. The same holds for code that you write. Without an explicit mechanism like io.EOF you would need to know the exact amount bytes to read or use some other kind of mechanism (like delimiters).
If your source doesn't return an io.EOF, it is from the point of view of the io package, an infinite stream. If you don't want your code to "hang" forever, you can wrap your reader inside another reader, for example with a LimitReader or you could implement a type that wraps your io.Reader and adds timeouts to the Read() method.

You could also use io.CopyN, which uses io.LimitReader under the hood.


When Create Issue is executed, "ASCII code 0xdf" will be entered.

Issue Type: Bug
ctrl+s has conflict with normal file save command on windows.

change shortcut to ctrl+alt+ s
press create an issue button
edit newissue.md
ctrl+alt+ s
appear "Mysterious character"  on newissue.md: that has ascii code 0xdf

If click the UI button without using the shortcut key, it will work as expected.
I don't know if it's related,
GitHub.vscode-pull-request-github]: `remoteHub/pullRequest` is not a valid menu identifier
_logMessageInConsole @ workbench.desktop.main.js:3036

Extension version: 0.37.2022021515
VS Code version: Code 1.64.2 (f80445acd5a3dadef24aa209168452a3d97cc326, 2022-02-09T22:02:28.252Z)
OS version: Windows_NT x64 10.0.19044
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-4150 CPU @ 3.50GHz (4 x 3492)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.92GB (2.38GB free)

Process Argv
--crash-reporter-id 5beae045-4e82-45fa-a01a-426c34e588f3

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
pythonvspyl392:30443607
pythontb:30283811
pythonvspyt551cf:30345471
pythonptprofiler:30281270
vshan820:30294714
vstes263cf:30335440
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vsclayoutctrt:30451275
dsvsc008:30440022
pythonvsnew555:30450902
vsbas813:30436447
vscscmwlcmc:30438804
vscgsvid1:30447480
helix:30440343
dockersurvey3:30446832
pynewfile477:30450038

Thank you for your reply!

Even if autosave is off, I think it's better to can work with non-collision shortcut keys. Because not everyone likes auto-save. As a proof, by default vscode, autosave is disabled.
If you like this idea, I would be grateful if you could adopt it.
If necessary, let me know and I will create a new Issue.

It is too late to change the default keybinding for this, as many users will have gotten used to the current keybinding.
The ctrl+s keybinding to create the issue was also a user request, so while I agree that not every user has the same keybinding preferences, I will not be able to make both happy with the default even if I do change it.
I think this is a good example of why we let you configure the keybindings yourself if you want to :). If you have not modified keyboard shortcuts before, you can see how to do so here: https://code.visualstudio.com/docs/getstarted/keybindings?msclkid=d6df428da9f111eca4f91c5cfc10415f#_keyboard-shortcuts-editor

@alexr00 keybind is not working for me. on "autosave off". why😢?
This is so stressful while npm watch command are runnning. For example, on project using react or flutter and others.

https://github.com/microsoft/vscode-pull-request-github/issues/2556#issuecomment-1061906127
You can add your own keyboard shortcut for this if you want it though. The command is issue.createIssueFromFile.


Exploding gradients in training BERT from scratch

Hi, I am training (base) BERT with 4 GPU V100 (each 16GB) with this training configuration:
Data is only 200million words
--train_batch_size=8
--max_seq_length=512
--max_predictions_per_seq=75
--num_train_steps=7000000
--num_warmup_steps=70000
--learning_rate=1e-4
--vocab_size=70K
Tensorflow 1.15

Shouldn't this happen as the training already include gradient clipping? (grads, _) = tf.clip_by_global_norm(grads, clip_norm=1.0)
Or could it be because my vocab size is too big for my current data? I notice BERT vocab is only 30K.

My first observation is your batch size is very low, do remember that google bert code is not optimized to run on multiple GPUs, I highly recommend using this instead: https://github.com/NVIDIA/DeepLearningExamples/tree/master/PyTorch/LanguageModeling/BERT
this branch will help if you have scaling happening on a single machine with multiple GPUs

Hi @dhruvsakalley thanks for your reply.
You are right, its not for multi-gpu training. I 've checked my training, even with 1 GPU and batch_size 8, it works properly.
Thank you for your suggestion. Really help!


[SEDONA-311] Refactor Inferred*Expression base class for Sedona SQL

Did you read the Contributor Guide?

Yes, I have read Contributor Rules and Contributor Development Guide

Is this PR related to a JIRA ticket?

Yes, the URL of the associated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-311. The PR name follows the format [SEDONA-XXX] my subject.

What changes were proposed in this PR?
This PR refactored Inferred*Expression base classes to handle functions with any number of parameters <=16. Now there's only one InferredExpression base class, and we can use it to wrap functions with any number of parameters.
If there are multiple overloaded versions of the same function, we can specify which version to wrap using InferredExpression(inferrableFunction{N}(func)), where {N} is the number of parameters. Please refer to {{ST_Translate}} as an example.
How was this patch tested?
Running existing unit tests on this patch.
Did this PR include necessary documentation updates?

No, this PR does not affect any public API so no need to change the docs.

I have to address some conflicts with the newly merged ST_Dimension patch. I'll directly push them to the master branch.


Homematic time drifting away

The time on my homematic devices goes out of sync. Every day about 5 -10 min more in the future.
The docker time is correct.  After restarting the container the time is back in sync.
After some research, I figured out that the rfd process starts its own software clock an never sync it again with the system clock.
yahm, fhem and the orginal ccu  runs the $HM_HOME/bin/SetInterfaceClock <IP_ADDRESS>:2001 periodically to sync the rfd software clock with the system time.
Source:

https://forum.iobroker.net/viewtopic.php?t=7486
https://github.com/hobbyquaker/hmcon/issues/37
https://homematic-forum.de/forum/viewtopic.php?t=36875

On the way to work, i forked the project and make this change:
https://github.com/IceOnly/hassio-addons/commit/839eef200770517d8544ba37fa291cdabd59771d#diff-aecd1ec5d25063e39a74c2367c11a0ac
I will build the addon locally this evening and try it out.
If it work I will make a pull request.

Nice. I'm every time happy with contributions


Documentation on downloading in paperwhite/voyage/oasis browser

I didn't want to find a USB cable for my kindle, and figured I'd try downloading books in my Oasis/Paperwhite browser seen as "send to kindle" doesn't play nice with azw3 (thanks Amazon).
The download dialogue took a few seconds to appear, but the book downloaded directly to the device - and it shows at the top of my library 🥳
My suggestion is that:

we note this as method 2 under "Amazaon Kindle eInk Devices" in the FAQ.
we figure out if there's a way to guarantee the kindle will download the Kindle cover thumbnail instead of opening it in a new page.

I did a quick look and I guess it's Kindle not understanding mime-types/content disposition in the same way as a desktop browser. I found the optional text in php, but can't see what to check next.

The browsing experience is OK, and #126 addresses improving it.

Sure, we can add that to the Kindle how-to. Can you create a pull request?
I don't think it's going to be possible to get Kindle to correctly download the thumbnail, because it has to be placed into a special system folder for it to work. So, your Kindle would probably download it to some downloads folder, and where do you go from there? It'd be too complicated.

I will open a PR about it.
Okay, that makes sense about the thumbnail. Do you know if "send to kindle" would place it in an acceptable folder to be used as a thumbnail?
Send to Kindle supports the following document formats:
...
JPEG (.jpg), GIF (.gif), Bitmap (.bmp), and PNG (.png) images.

I highly doubt it, but you could try!


Daniel/remove delete devices UIA

On Kacper's request: Remove new user interactive auth (requesting password on operation) for deleting devices (in Matrix specifications).
Staging built on this branch and deleting devices works now w/o auth.

On Kacper's request: Remove new user interactive auth (requesting password on operation) for deleting devices (in Matrix specifications).

@danpe this is one from over a month ago that we forgot.
Was working on staging this whole time and never merged to master, nor pushed to production.


Transient orderer error is not being retried

Gateway service fails to retry with orderers when it receives a response status == SERVICE_UNAVAILABLE
WARN [gateway] Submit -> Error sending transaction to orderer txID=<TXN_ID> endpoint=<ORDERER> err="rpc error: code = Aborted desc = received unsuccessful response from orderer: SERVICE_UNAVAILABLE"

As per this logic, when the status is != Status_SUCCESS, it return rpc error as Aborted…
https://github.com/hyperledger/fabric/blob/main/internal/pkg/gateway/api.go#L467-L469
Whereas the calling function will retry with the next orderer only if the code is Unavailable…
https://github.com/hyperledger/fabric/blob/main/internal/pkg/gateway/api.go#L444-L446
So if the intention is to retry with the next orderer, do we need to handle this status Status_SERVICE_UNAVAILABLE in this condition…
Ref:
Discussion thread in fabric-dev slack channel: https://ibm-blockchain.slack.com/archives/CL3L0REMD/p1671447903462259

Adding slack conversations here for reference:

David Kelsey
If the service is unavailable then it means it managed to contact the orderer. So maybe the assumption is that if that orderer states the service is unavailable then all the orderers would respond with the same. Do you have a scenario where 1 orderer responds unavailable yet the others are available ?
Andrew Coleman
It looks like the orderer returns SERVICE_UNAVAILBLE if there is an error in the consensus logic. There might be a need for different logic in the gateway.
Vivekmani TSK
In our scenario, we’re running multiple test-cases concurrently and we’re seeing this failure intermittently while other cases are succeeding…
If the orderer failure is transient, then we expect the gateway to retry….
Andrew Coleman
can you see in the orderer logs why it is failing?
Vivekmani TSK
It failed with
Rejecting broadcast of normal message from <IP_ADDRESS>:39574 with SERVICE_UNAVAILABLE: rejected by Order: EOF

Orderer logs during that timestamp for ref:
[orderingservice2-5b9fdbd4bd-w8cbn orderer] 2022-12-16 09:07:35.570 UTC [orderer.common.broadcast] Handle -> WARN 628f7 Error reading from <IP_ADDRESS>:38816: rpc error: code = Canceled desc = context canceled

[orderingservice2-5b9fdbd4bd-w8cbn orderer] 2022-12-16 09:07:35.570 UTC [comm.grpc.server] 1 -> INFO 628f8 streaming call completed grpc.service=orderer.AtomicBroadcast grpc.method=Broadcast grpc.request_deadline=2022-12-16T09:08:05.565Z grpc.peer_address=<IP_ADDRESS>:38816 grpc.peer_subject="CN=itorg1peer3,OU=peer" error="rpc error: code = Canceled desc = context canceled" grpc.code=Canceled grpc.call_duration=2.574094ms

[orderingservice2-5b9fdbd4bd-w8cbn orderer] 2022-12-16 09:07:36.014 UTC [orderer.common.broadcast] ProcessMessage -> WARN 628f9 [channel: intg-shared-mixed] Rejecting broadcast of normal message from <IP_ADDRESS>:39574 with SERVICE_UNAVAILABLE: rejected by Order: EOF

[orderingservice2-5b9fdbd4bd-w8cbn orderer] 2022-12-16 09:07:36.014 UTC [comm.grpc.server] 1 -> INFO 628fa streaming call completed grpc.service=orderer.AtomicBroadcast grpc.method=Broadcast grpc.request_deadline=2022-12-16T09:08:06.012Z grpc.peer_address=<IP_ADDRESS>:39574 grpc.peer_subject="CN=itorg1peer3,OU=peer" grpc.code=OK grpc.call_duration=800.815µs

As Dave mentioned in the chat transcript, the logic I expect is that the transaction is "bad" and rejected by the orderer, the client should get a gRPC Aborted response. If the Gateway is unable to get a "good" or "bad" status from an orderer (typically because the orderer cannot be reached) then other orderers should be tried. The legacy SDKs use similar logic:
https://github.com/hyperledger/fabric-sdk-node/blob/2eee017965721a7514c14cbae5e12d5414bc1263/fabric-common/lib/DiscoveryHandler.js#L145-L151
If there are cases where the orderer can return a non-success status code for a message that is not confirmed "bad", we need to change the Gateway peer logic to try other orderers in that case rather than immediately return a gRPC Aborted response. An orderer status code >=500 (INTERNAL_SERVER_ERROR, NOT_IMPLEMENTED, SERVICE_UNAVAILABLE) seems to fit this category as it indicated the orderer (rather than the message) is the problem. Or perhaps:

200 (SUCCESS) means "good" so return gRPC OK
Status code >=400 and <500 (BAD_REQUEST, FORBIDDEN, NOT_FOUND) means "bad" so return gRPC Aborted
Anything else (including connection failure) is neither good or bad so should try other orderers
If we run out of orderers without getting either a "good" or "bad" status, return gRPC Unavailable

Possibly the Gateway could return a gRPC Internal (error) if no "good" or "bad" status is obtained and orderers are contactable to indicate an orderer problem and encourage clients not to simply retry, which they might reasonably do on receiving a gRPC Unavailable. On the other hand, you would hope that a system running multiple orderer nodes will have some functioning orderers so this is not an expected scenario.


Upgrade to helm v2.8.0

Upgrade helm v2.8.0 (including the packages from the k8s.io/helm path)

(Closed by mistake -- re-opening...)

Still incomplete due to a conflict -- will re-open once I have a fix. Thanks!


Powertoys.Awake High CPU Usage

Microsoft PowerToys version
0.41.3
Running as admin

[X] Yes

Area(s) with issue?
Awake
Steps to reproduce
This is a ton of CPU usage when I actually have the capability enabled.  I enabled this for 30 minutes to keep my machine awake and it was hovering in the 25% range.
✔ Expected Behavior
Not exactly sure, but the process doesn't consume anything when it is off.
❌ Actual Behavior

Other Software
No response

This is a known issue tracked by https://github.com/microsoft/PowerToys/issues/12052 and will be fixed on 0.43.


"Read and change all your data on all websites" on Chrome?

Hi, I went to install this for chrome and was confronted with the following:

I expected global read access, but global write seems unnecessary. What's going on there?

Hi Katie
unfortunately this permission is the only level of granularity that browsers allow. I need this because I inject JavaScript into every page you’re on to try to find rel=me links. You can see the source code here https://github.com/tvler/streetpass/blob/main/extension/src/content-script.ts
technically there would be nothing stopping me from “writing” arbitrary html to the web pages you’re on, but as you can see this script is just reading. If you’re not comfortable with these permissions, I have instructions for how to run the extension from a local build so that you can be sure that the code getting executed is safe and secure.

Also just to be clear, I don’t collect any data or analytics, and all data is stored locally with no way for me or anyone besides yourself to see

oh that makes sense! Thanks for taking the time to explain 🙌


maxListeners of 0 is unlimited

emitter.setMaxListeners(n)
By default EventEmitters will print a warning if more than 10 listeners are added for a particular event. This is a useful default which helps finding memory leaks. Obviously not all Emitters should be limited to 10. This function allows that to be increased. Set to zero for unlimited.

(emphasis mine)

Cause of failure in https://github.com/pouchdb/pouchdb/pull/4129 (due to https://github.com/request/request/pull/957/files)

fixed by #3


Add periodic license verification

Signed-off-by: Emruz Hossain<EMAIL_ADDRESS>

I think we should have a root package. Then a sub package kubernetes that will have a dependency on k8s and does the full check.


Added Key Vault access policies for App Service identities in az-isolated-service-single-region template

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My code follows the code style of this project.
[x] I ran lint checks locally prior to submission.
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

What is the current behavior?

The az-isolated-service-single-region template creates all app services in Azure with system assigned identities; however, the Key Vault does not have an access policy for those identities to allow them Get/List permissions to the resource objects.
Issue Number: #282
What is the new behavior?

[x] App services defined within the unauthn_deployment_targets will also have a key vault access policy created providing them with GET/LIST permissions
[x] App services defined within the authn_deployment_targets will also have a key vault access policy created providing them with GET/LIST permissions

Does this introduce a breaking change?

[NO]

While the behavior will be different, it should only add the permission to any existing deployments.
Any relevant logs, error output, etc?

N/A
Other information

N/A

Issue #289 created to track addition of integration tests needed to cover this addition to the template.


doc: en to zh-cn

pageId: sql-getting-started
English to Chinese

cc @momo-jun @DaveDuggins @D-2-Ed

@AdamTyn Hi, welcome to contribute your Chinese document translation in Crowdin. You can take this document as a reference.

This PR can be closed now.


Release kong-operator v0.5.0. Fix a link in v0.4.0 definition.

Updates Kong operator to v0.5.0.
Also, fixes a link to our GitHub in the previous release.
Updates to existing Operators

[x] Is your new CSV pointing to the previous version with the replaces property?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

/ok-to-test

/lgtm


Support for Resolving Indirect Function Pointers to Direct Calls in Decompiled View

In Ghidra, I'm working with functions that use indirect pointers (e.g., single or double indirect calls) to call other functions, often represented by BLX instructions in ARM assembly. I need a way to automatically replace these indirect function pointers in the decompiled view with the actual function names, showing direct calls (e.g., disable_interrupts() instead of (*(code *)PTR_disable_interrupts)). Is there a way to force Ghidra’s decompiler to fully dereference these function pointers and display the resolved function names in place of the indirect calls? Despite adding direct references and flow overrides, I haven't found a way for the decompiler to reflect these changes, and manual re-decompilation attempts haven’t resolved the issue.

Set their mutability to constant.

This works, thank you!


#2969: Ignore the timezone on a date only input.

The DatePicker just creates a datetime with 2022-06-04 00:00:00 (Europe/Amsterdam) and converts it to UTC which is 2022-06-03 22:00:00, and then converts it to a date only 2022-06-03. This doesnt make any sense when you want a date only so the timezone must be ignored.
#2969

Hey, I don't think this is the correct solution - people should just not be using the timezone() method with the Date Picker. I don't think it fixes #2969.


A permission doesn't function.

shopchest.buy
Though it's being added, can't be bought.

If you are using world guard you will have to execute the following command(s):
/rg flag  use-shop allow
you can also disable "World guard integration" in the config file

Or set the default value of use-shop to ALLOW in the config.


[FEATURE] Future milestones to have a dotted line

Description
We have milestones at the moment, but these are in the past, it would be great to have goals - these would need to be displayed slightly differently
Maybe a flag that is goal: true
Current data...
"milestones": [
    {
      "title": "Started Freelancing",
      "date": "May 2010",
      "icon": "dollar",
      "color": "grey",
      "description": "Started freelancing again",
      "url": "https://www.eddiejaoude.io/"
    }
]

We could add a flag...
"milestones": [
    {
      "isGoal": true,
      "title": "Started Freelancing",
      "date": "May 2010",
      "icon": "dollar",
      "color": "grey",
      "description": "Started freelancing again",
      "url": "https://www.eddiejaoude.io/"
    }
]

Currently milestones are displayed like this, but we could have a dotted line down to another group of cards and these could be slightly faded, to show they have not been achieved yet

Screenshots
No response
Additional information
No response

hi @eddiejaoude can you explain this a bit more

@ManishBisht777 I updated the issue description, I hope that helps

@eddiejaoude thanks for the detailed info
I would like to work on this feature
we can divide the task into parts:
1) ui for the goals (faded cards)
2) make it functional

I am starting with the UI

Both parts will need to come in the same PR though

sure

completed


Update trademarks in docs to match F5 branding style guide

Per updated style guide (June 2016): "Apply trademarks to first mentions of products and technologies in body copy (rather than headlines, subheads, captions, etc.)."
Ongoing -- update docs as they're edited.

@jputrino Is work completed for issue #138 ?  What is the DoD for this issue ?

@dflanigan - you can mark it as complete if you want to; this is an ongoing thing.


Fix screen context not correctly updated on screenView events

The screen context links events back to the specific screen view that generated the event.
At the moment it works correctly with the automatic screenView events only.
A screenView event handled outside the tracker is not able to update the internal tracker state causing the screen context not being updated correctly.

It's related to the Android issue: https://github.com/snowplow/snowplow-android-tracker/issues/350


cache-manager - adding disconnect to cache

Please check if the PR fulfills these requirements

[x] Followed the Contributing guidelines.
[x] Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Adds the ability to disconnect from the relevant stores inside the cache

@TimothyIp - thanks so much and will get this rolled out in the next couple weeks.

This is awesome! Thanks!


Verify speed up/cancel works

when canceling a transaction it stucks as pending for me

As in the 2nd transaction becomes pending and is stuck there, but the original transaction completed? But either way the transaction status for both should be picked up, right? Maybe related to #5888 ?


How many people use Fountain?  (apology within)

I APOLOGIZE IN ADVANCE for posting this here, since it really isn't what Github is about, but I've looked all over the place and can't find anywhere else to post this.  If someone can point me to a better place, I'd really appreciate it.  Meanwhile:
Is there any data out there indicating how many people are using Fountain?  I'm trying to make a case to my co-workers about using Fountain as the basis for some new application work, but without any sort of usage data, it's hard to make a convincing case.  Does anybody have anything relevant or useful?  Apologies again, and thanks!

Despite of "how many users" there is support from some popular screenwriting applications, there are even some wikipedia-articles about:

https://en.wikipedia.org/wiki/Fountain_(markup_language)#Implementations
https://en.wikipedia.org/wiki/Screenwriting_software#List_of_software (there are some with Fountain-support)

The counter itself isn't proving anything for real, but you can look at the star-counter of this project, some of them might use the Fountain-language, and some others don't use github to add a star ;)

I think the star on github only represents a tiny fraction of actual users. The fact that Amazon Storywriter supports Fountain is a clue at the size of your potential audience.
You might want to reach out to John August directly about this. I'd also recommend contacting the developers of the apps listed on the Fountain site, particularly Slugline and Logline, as both are excellent examples of apps that rely on Fountain as a native format instead of just an import/export option.

We don't have any concrete data on how prevalent Fountain is (we don't even know how we could reliably get that sort of data), but I will echo the sentiment already expressed that it's pretty popular. Aside from John, who created the format, there are numerous professional screenwriters working in Fountain now, and most screenwriting apps (and an increasing number of Markdown apps) support reading and writing Fountain files.
Best of luck!

Thanks for the comments!  I'll be digging into the pointers as plans here proceed.


Display --- for member counts of groups not managed by app

Right now, the groups not managed by the app display counts of 0 for the number of member accounts and member groups. This is confusing so it should probably show a blank instead of 0.

Closed by #152


Select field in Dialog, menu not stay with field when scrolling.

Problem description
Select field inside a dialog, with scroller.
the menu moves when scrolling
Steps to reproduce
just  create a select field in dialog.
Versions

Material-UI: 0.15.0
React: 15.0.2
Browser: chrome

Same issue: open select field moves when scrolling.
I guess it only appears on macOS because of smooth scrolling.
You can reproduce in here: http://www.material-ui.com/#/components/select-field

@riseremi, I can see that on debian, too.
OS: Linux alpha 4.6.0-0.bpo.1-amd64 #1 SMP Debian 4.6.4-1~bpo8+1 (2016-08-11) x86_64 GNU/Linux
Browser: Chrome Version 53.0.2785.116 (64-bit)

it is worse at 0.18.0.
the list is show under the dialog now.
version:
Material-UI: 0.18.0
React: 15.4.x
Browser: chrome


helix 23.03 issues

It seems that 23.03 was overlooked in the moderation approval queue since 22.12 was submitted simultaneously.
https://community.chocolatey.org/packages/helix/23.03
I'm not sure how the process works, but maybe you should poke them.

Closing since this seems to be resolved as of 23.05


Typo in claude2_extractor.py prompt

Hi! RefChecker looks really neat!
I noticed there is a typo in the claude2_extractor prompt:
Now geneate the KG for the provided input text:

(At line 75, from here https://github.com/amazon-science/RefChecker/blob/main/refchecker/extractor/claude2_extractor.py)
"geneate" should be "generate"

Also, from line 59 in the same file, "occured" should be "occurred":
("Announcement of Optimus", "occured at", "Artificial Intelligence (AI) Day event")

Both typos are part of the LLM prompt itself, not just typos, so they potentially impact the LLM behaviour.

Thank you for pointing this out, we have corrected them accordingly.


wants to change the product title font style

I want to add a better font style on the product title name.

@codervivek5  plaz assign me


white-space should be ignored in the spring_profiles setting

I had this line in my uaa.yml file:
spring_profiles: postgresql, ldap
This caused UAA to look for a provider called " ldap", with a leading space.
It's possible that the line StringUtils.commaDelimitedListToStringArray(profiles) should be
StringUtils.tokenizeToStringArray(profiles, ",", true, true)

https://github.com/cloudfoundry/uaa/pull/427

Forgot to point it at develop.

PR: https://github.com/cloudfoundry/uaa/pull/428


2.0b3 continuous scanning even when using with default settings

Hi
I recently did a yarn upgrade-interactive and for some unknown reason I got the 2.0 version proposed. Anyway, when running it via the following (what I consider to be the default component integration):
  <Scanner
            onResult={onQrCodeReceived}
            onError={error => console.log('scan error', error?.message)}
          />

I do get continuous scanning even though I no longer show a QR code to the camera (so I hear a loud beep every 1s or so).

Same here! Please teach me how to fix it?

I have similar issues: the Scanner continues scanning even when component got unmounted (routing away).
Seems that at https://github.com/yudielcurbelo/react-qr-scanner/blob/dev/src/components/Scanner.tsx#L43
an additional useEffect is required?
useEffect(() => {
 return stopScanning;
}, [])

something like that

Same here! Please teach me how to fix it?

I finally downgraded to V1... btw: the useMediaDevices hook in V1 could be better...it was not returning all I needed - so,  I created my own hook:
export const useCameraDevices = () => {
    const {
        data,
        isLoading,
        error
    } = useSWR("detectCameraDevices", async () => {

        if(isBrowser){
            const devices = await navigator.mediaDevices.enumerateDevices();
            return devices.filter(({kind}) => kind === "videoinput");
        }
        return []
    })

    return {
        devices: data ?? [],
        isLoading: isLoading,
        error
    }
}

Note that I'm on NextJS 14 with SSR and RSC...and further deps on swr lib

Version 1.2.10 !
We experience "continuous scanning" problems too that happen from time to time on an always on QR Scanner kiosk.
The value returned from the scanner when succesfull is an object akin to :

But when the bug happens we get a short string, each second, like :

The bug is hard to track and replicate, the only time we triggered without it happening randomly was just after a succesful scan

We can confirm that the bug can happen anytime, after seconds, minutes or hours.
The scanner becomes stuck as if it was scanning the same valid value on repeat and only a refresh of the webpage can reset the state. It's hard to understand how to trigger it, but it seems to be triggered by random movements in front of the camera.

Are there any ideas on how to fix this?

It could be tied to https://github.com/zxing-js/library which is used as dependency, which btw looks in maintenance mode only and has these issues open

https://github.com/zxing-js/library/issues/589
https://github.com/zxing-js/library/issues/575

I don't know if it "fixes" any issues behind the scenes but I've added a timeout, after i've collected a scan result, before I collapse (delete) the component.
It appears to have "fixed" this. (It no longer makes beeping noises, and appears to be off).
setTimeout(() => {
      setOpen(false)
    }, 1000)

We experienced similar issues. We solved it by enabling the scanner in useEffect:
function App() {
  const [enabled, setEnabled] = useState<boolean>(false);

  useEffect(() => {
    setEnabled(true);
  }, []);

  return (
    <LayOut>
      <Scanner
        onResult={(text) => {
          setEnabled(false);
          // DO ACTION
        }}
        onError={(error) => console.error(error)}
        enabled={enabled}
      />

    </LayOut>
  );
}

export default App;

This issue should be fixed on 2.0.0-beta.8. There are breaking changes for this beta release.


New PR to test SignCLA at 2017-12-04 14:19:21.000214352 -0800 PST m=+66.534954875

Body of PR

indiv-pass-staging: test succeeded


[Feature Request]: Trailing Cursor

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
add the customized mouse movement for the cursor. Like this⬇
https://github.com/HimanshuNarware/CareerZunction_Intern/assets/125206920/2227f393-e993-49d3-9141-bec987c59d48
Use Case
Adding a trailing effect for the cursor while moving through out the page
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@HimanshuNarware please assign this to me


JSONLDWriter::getSupportedSettings() throws exception

https://github.com/eclipse/rdf4j/blob/0483e977fb6fe945e29e6340620c379b182d279e/core/rio/jsonld/src/main/java/org/eclipse/rdf4j/rio/jsonld/JSONLDWriter.java#L184
This code adds supported settings to the list provided by the ancestor, but in the default implementation that list is an immutable empty list. Calling this method results in an exception:
java.lang.UnsupportedOperationException
at java.base/java.util.AbstractList.add(AbstractList.java:153)
at java.base/java.util.AbstractList.add(AbstractList.java:111)
at org.eclipse.rdf4j.rio.jsonld.JSONLDWriter.getSupportedSettings(JSONLDWriter.java:184)

There's actually a broader problem as well, which that several of our writers do not return a complete set of all supported settings.


Can't include this with a line of gradle?

I just want to try it out without having to haul in so much code by myself, would appreciate if I could do that if it was a one-liner to do so, just like all the android libs out there. Great work by the way, it looks very ..rigorous :)

Have in mind this is not a library, but a sample project. It's not adding any new functionality, but showing how animations work


[SUPPORT] Cannot run HoodieDeltaStreamer the second time when using async clustering

Tips before filing an issue

Have you gone through our FAQs?

Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
When running the following script I am able to ingest sims and cluster.
Environment Description

Hudi version : 0.10.1 (also saw this on 0.9.0)

Spark version : 3.0.3

Hadoop version : 3.2.0

Storage (HDFS/S3/GCS..) : AWS S3

Running on Docker? (yes/no) : No

Additional context
Add any other context about the problem here.
Stacktrace
Add the stacktrace of the error.
22/02/11 23:26:04 ERROR HoodieDeltaStreamer: Shutting down delta-sync due to exception java.lang.NullPointerException 	at org.apache.hudi.utilities.deltastreamer.DeltaSync.getClusteringInstantOpt(DeltaSync.java:864) 	at org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer$DeltaSyncService.lambda$startService$0(HoodieDeltaStreamer.java:648) 	at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1604) 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) 	at java.lang.Thread.run(Thread.java:748) 22/02/11 23:26:04 INFO HoodieDeltaStreamer: Delta Sync shutdown. Error ?true 22/02/11 23:26:04 INFO HoodieDeltaStreamer: DeltaSync shutdown. Closing write client. Error?true 22/02/11 23:26:04 ERROR HoodieAsyncService: Service shutdown with error java.util.concurrent.ExecutionException: org.apache.hudi.exception.HoodieException 	at java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:357) 	at java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1908) 	at org.apache.hudi.async.HoodieAsyncService.waitForShutdown(HoodieAsyncService.java:89) 	at org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer.lambda$sync$1(HoodieDeltaStreamer.java:182) 	at org.apache.hudi.common.util.Option.ifPresent(Option.java:96) 	at org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer.sync(HoodieDeltaStreamer.java:179) 	at org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer.main(HoodieDeltaStreamer.java:514) 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 	at java.lang.reflect.Method.invoke(Method.java:498) 	at org.apache.spark.deploy.JavaMainApplication.start(SparkApplication.scala:52) 	at org.apache.spark.deploy.SparkSubmit.org$apache$spark$deploy$SparkSubmit$$runMain(SparkSubmit.scala:928) 	at org.apache.spark.deploy.SparkSubmit.doRunMain$1(SparkSubmit.scala:180) 	at org.apache.spark.deploy.SparkSubmit.submit(SparkSubmit.scala:203) 	at org.apache.spark.deploy.SparkSubmit.doSubmit(SparkSubmit.scala:90) 	at org.apache.spark.deploy.SparkSubmit$$anon$2.doSubmit(SparkSubmit.scala:1007) 	at org.apache.spark.deploy.SparkSubmit$.main(SparkSubmit.scala:1016) 	at org.apache.spark.deploy.SparkSubmit.main(SparkSubmit.scala) Caused by: org.apache.hudi.exception.HoodieException 	at org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer$DeltaSyncService.lambda$startService$0(HoodieDeltaStreamer.java:666) 	at java.util.concurrent.CompletableFuture$AsyncSupply.run(CompletableFuture.java:1604) 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) 	at java.lang.Thread.run(Thread.java:748) Caused by: java.lang.NullPointerException 	at org.apache.hudi.utilities.deltastreamer.DeltaSync.getClusteringInstantOpt(DeltaSync.java:864) 	at org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer$DeltaSyncService.lambda$startService$0(HoodieDeltaStreamer.java:648) 	... 4 more

I could not reproduce locally w/ 0.10.1. Only suspicion I have is around schema providers. I also using parquetDFS, but have provided schema provider and could able to see async clustering running smoothly. I stopped and restarted jobs couple of times and is all good.

I could not reproduce. I also tried w/ ComplexKeyGen and empty partition path and no schema provider configs. yet could not reproduce. sorry. we might need reproducible steps w/ some dataset if feasible.
/bin/spark-submit --packages org.apache.spark:spark-avro_2.11:2.4.4,org --driver-memory 8g --executor-memory 8g --class org.apache.hudi.utilities.deltastreamer.HoodieDeltaStreamer path_to_/hudi-utilities-bundle_2.11-0.10.1.jar --props /tmp/parquet-dfs-cluster.props  --source-class org.apache.hudi.utilities.sources.ParquetDFSSource   --source-ordering-field created_at   --table-type COPY_ON_WRITE --target-base-path file:\/\/\/tmp/hudi-deltastreamer-gh1/   --target-table gh_hudi_tbl31  --op UPSERT --hoodie-conf hoodie.clustering.async.enabled=true --continuous --source-limit 4000000 --min-sync-interval-seconds 30

properties file contents
hoodie.datasource.write.recordkey.field=other,org.id
hoodie.datasource.write.partitionpath.field=
hoodie.datasource.write.precombine.field=created_at
hoodie.datasource.write.keygenerator.class=org.apache.hudi.keygen.ComplexKeyGenerator

hoodie.metadata.enable=false
hoodie.upsert.shuffle.parallelism=8
hoodie.insert.shuffle.parallelism=8
hoodie.delete.shuffle.parallelism=8
hoodie.bulkinsert.shuffle.parallelism=8

hoodie.deltastreamer.source.dfs.root=/dataset_path/

hoodie.clustering.plan.strategy.sort.columns=created_at
hoodie.clustering.plan.strategy.daybased.lookback.partitions=0
hoodie.clustering.async.max.commits=2

@codope @satishkotha : can you folks spot anything.

@satishkotha : Can you please follow up here when you get a chance.

@rohit-m-99 : Are you still looking for assistance here. of if you got it resolved, let us know.

We switched to clustering inline and stopped seeing problems. Was not able to get async clustering working unfortunately.

thanks! please reach us if you are looking for further assistance


feat: Update user data route

Create a separate route - /update for updating the existing user data

Will you please assign me this issue?


chore(website): parse markdown in docs

Sorry this took so long, for some reason I dreamed up the idea that the we couldn't just use markdown :sweat:

LGTM

Merged in at 11a9c12


udp suggestions

I've been looking at the udp code and I have some suggestions.
Use: {recbuf,1024*1024} on gen_udp:open
The UDP socket receive buffer can fill up surprisingly quickly and the default is dangerously low.
Use {active,100} or similar for receiving packets. It improves throughput.
Use inet_udp:send instead of gen_udp:send. If gen_udp:send is used, inet:getaddrs/3 will be called on every send. This causes a needless erlang:start_timer and erlang:cancel_timer call.

Thanks for the suggestions.
Given {active, once} mode, the larger buffer is essentially unnecessary since there will only be one packet in the buffer at any given time, correct? The typical DNS packet size is 512 bytes, unless EDNS specifies a larger size. Even in that case the typical packet size will still be less than the default 8k size used in Erlang.
So in the current setup the default buffer size is fine.
Now, let's consider if I switched to {active, 100}. In this case the buffer size would need to be larger. My understanding of {active, N}, based on http://www.erlang.org/doc/man/inet.html#setopts-2 , I would need to also listen for the {udp_passive, Socket} message and set {active, N} again. I'm not sure how much of a gain we'd get by letting the socket buffer, but I may try some testing.
Finally, regarding inet_udp:send versus gen_udp:send, are there any drawbacks?

About buffers you are very wrong. This is kernel space reserved for
incomming packets. Nothing to do with active setting.
Once it is full (because they were not processed fast enough) every arrived
udp packet is dropped. I had this problem on every project I used udp in
erlang. It was always filling up.
The difference in inet_udp is that IPs must be in tuple form. That timer is
so that it tries to resolve address in list form. But if it is tuple it
just sets timer, noop then cancel. Unfortunate design.
On Jan 21, 2016 23:31, "Anthony Eden"<EMAIL_ADDRESS>wrote:

Thanks for the suggestions.
Given {active, once} mode, the larger buffer is essentially unnecessary
since there will only be one packet in the buffer at any given time,
correct? The typical DNS packet size is 512 bytes, unless EDNS specifies a
larger size. Even in that case the typical packet size will still be less
than the default 8k size used in Erlang.
So in the current setup the default buffer size is fine.
Now, let's consider if I switched to {active, 100}. In this case the
buffer size would need to be larger. My understanding of {active, N},
based on http://www.erlang.org/doc/man/inet.html#setopts-2 , I would need
to also listen for the {udp_passive, Socket} message and set {active, N}
again. I'm not sure how much of a gain we'd get by letting the socket
buffer, but I may try some testing.
Finally, regarding inet_udp:send versus gen_udp:send, are there any
drawbacks?
—
Reply to this email directly or view it on GitHub
https://github.com/aetrion/erl-dns/issues/37#issuecomment-173736033.

Implemented two of the three suggested changes with #61


Create RMC compiler driver

Requested feature: We should be able to configure the compiler without having to modify its code.
Use case: Provide a robust and clean compilation module for RMC.
Link to relevant documentation (Rust reference, Nomicon, RFC):
Is this a breaking change? No
Success criteria: We replace rmc-rustc by a rust application call. I.e., this issue only covers the creation of the RMC compiler driver on the top of rustc. Follow up issues will track moving more logic out of our python scripts into an RMC application.

We still have rmc-rustc wrapper, but it's mostly needed to overcome a limitation of cargo rmc. I'll close this issue since it has captured the most important issue which is the dependency on a modified rustc and its bootstrap script.


Mention needing additional configuration for ts-jest

Summary
I added a link to ts-jest docs configuration. Currently, if you try to run parse .ts files without a config file, you are brought to this page on the docs. There is only a link to the github repo for ts-jest. This small change will help save time and googling for users. We won't have to worry about keeping this up to date since it's just a link to their documentation page.
Test plan
None, this is just a change to the docs.

Sure thing

OK, I think I got all the versions.


Enhance and Simplify hasRole-Directives

Simplify directives by combining switch and standard cases. Also, documentation for switch is missing.

add hasAnyRole
join hasRoleCase with hasRole
join hasNoRolesCase with hasNoRoles
rename hasOtherRoleCase to hasOtherRole
create/extend documentation for these directives

Draft: distingiush between switch and standard case by evaluating @Host in constructor ("Are you hasRoleSwitch?").

Milestone for 2.1?

The current plan is to have a stable 2.0 running with Angular 4.
Moving on, the focus is supporting Angular 5. I think we will start using the major version to follow Angular, so we would start with 5.0.0 for the next milestone which features Angular 5 compatibility. I expect working on this next week, as we currently have to wait for some peer dependencies to pick up ng 5.


Ensure that the LangaugeServiceHost is initialized when attempting to…

… fetch the contained langauge factory for a venus file.
Customer scenario
User opens a cshtml file while the project is still loading and this causes broken intellisense/language service for the project.
Bugs this fixes:
Fixes #1617
Workarounds, if any
Users need to close and re-open the solution.
Risk
Low risk.
This is a race condition where we are attempting to get the contained language factory before our language service host has been initialized. The fix is to ensure that we explicitly await langauge service host initialization at the beginning of GetContainedLanguageFactoryForFile.
Performance impact
Low
Is this a regression from a previous update?
Yes. Seems like this race condition has been exposed by some of our recent changes, though the race has existed in the code base since the initial implementation
Root cause analysis:
Noted above.
How was the bug found?
Automated tests.

Tagging @dotnet/project-system @abpiskunov @BillHiebert for review.

Rebased to master

Tagging @Pilchie @srivatsn to take to the Dev15 shiproom. Kevin, I will forward you the email thread with informal approval from Joc.

Approved in shiproom.


Script ran successfuly but still can't import IOKit

I know you provide the script without guarantees but maybe I'm doing something obvious wrong.
Ran the script and successfully and in my Makefile tried to add IOKit with name_FRAMEWORKS = IOKit or name_PRIVATE_FRAMEWORKS but without the result I expected; being able to import IOKit in the app.
Any tips are welcome!

To import into swift you need a bridging header that imports the IOKit stuff