Tap without focused object is not responded

Hi,
I noticed that in the latest release of HoloToolkit-Unity, there is no interface for responding to tapping event, if there is no focused object. It seems that "OnInputUp", "OnInputDown", "OnInputClick" are sent to the focused object only. Is there access point to the "background" tapping event?
In the old version, the GestureManager can do something like this:
recognizer = new GestureRecognizer(); recognizer.TappedEvent += (source, tapCount, ray) => {     MyParseTapEvent(); }; recognizer.StartCapturingGestures();
I can still add this to the my c# script, but it will be a little bit redundant and weird in the Event-Interface design pattern.

You need to register as global listener the game object that contains the script. Add the SetGlobalListener to the object.

Nice! Thanks


Conditional type is not resolved when used in generics

Bug Report
🔎 Search Terms
conditional types, generics
🕗 Version & Regression Information

Seems to be broken in every version available in TS playground

⏯ Playground Link

Playground link with relevant code
💻 Code

interface PayloadMap {
    hello: string,
    none: never,
}

type Payload<T extends PayloadMap, K extends keyof T> = T[K] extends never ? [] : [payload: T[K]];

class Base<T extends PayloadMap = PayloadMap>  {
    emit<K extends keyof T>(type: K, ...payload: Payload<T, K>): Base<T> {
        return this;
    }
}

class Implementation<T extends PayloadMap = PayloadMap> extends Base<T> {
    constructor() {
        super();

        this.emit('hello', 'asdf'); // Argument of type '["asdf"]' is not assignable to parameter of type 'Payload<T, "hello">'.(2345)
        this.emit('none'); // Argument of type '[]' is not assignable to parameter of type 'Payload<T, "none">'.(2345)
    }
}

// all following `emit` calls are fine
let impl = {} as Implementation<{test: string} & PayloadMap>;
impl.emit('hello', 'asdf');
impl.emit('none');
impl.emit('test', 'asdf');

class GenericlessImplementation extends Base<{test: string} & PayloadMap> {
    constructor() {
        super();

        this.emit('hello', 'asdf');
        this.emit('none');
        this.emit('test', 'asdf');
    }
}

🙁 Actual behavior
This code example doesn't compile and I think it should as the generic is always constraint to PayloadMap.
🙂 Expected behavior
The example has compile errors because compiler is unable to find out that T is always compatible with PayloadMap.

Some context for the issue I'm dealing with - I'm trying to change typings to a base class we have in our library that is responsible for emitting events and acts as a base class for all other objects that need to emit events in the lib. We have an interface with eventName: PayloadType mapping - it basically defines all events that we emit and what payloads they have. This map is currently static but we want to make it generic so that users of the library can extend it and emit custom events. We need to support events with no payload - those are represneted by the never type. If the payload is never, emit function should be callable only with one argument, if the payload is any other type, the emit function has to be called with 2 arguments - that's what the Payload type does (and it works well - until we introduce generics).
I was looking at this bug report as well but it seems to be about something different, but I'm not 100% sure - https://github.com/microsoft/TypeScript/issues/33369.

Resolving of conditional types including unbound type arguments is deferred. This is a design limitation.
Your example would not be type-safe either way, as the type { hello: 'abc', none: never } extends PayloadMap, but the string "asdf" would not be assignable to the type "abc".

What do you mean by hello: 'abc'? There's only hello: string but maybe I'm missing something.

But there's no string literal in the playground example or in the original post, there's only this:
interface PayloadMap {  hello: string, none: never }
let impl = new Implementation<{test: string} & PayloadMap>();

The payload map is extended.
Anyway - the main problem is that the type system has a limitation and cannot resolve this? Can it be implemented differently? I just want to avoid having 2 overloads for the emit method as that would mean that events with payloads can be emitted without it (which is something we don't want).
This is what I want to avoid:
emit<K extends keyof T>(type: K): Base<T>;
emit<K extends keyof T>(type: K, payload?: T[K]): Base<T> {

Because this call should be invalid:
impl.emit('hello');

Extending PayloadMap doesn't mean the property hello will have the type string, because the extending type may provide a more specific type that extends string. I gave you an example why your code can't work safely. There's simply no way to know that the string "asdf" is valid.


FC007: accept 'suggests' and 'recommends'

Chef supports the suggests and recommends as keywords for soft dependencies. They are a NOOP and the semantics are a bit unclear but they're used for "most people don't need this" dependencies, e.g. depending on the windows cookbook.
Fixes acrmp/foodcritic#159.

You probably don't want to be using suggests or recommends at all since if we ever do implement them then if the semantics we implement do not agree with what you intended, your cookbook will be broken.

FC052 and FC053 mark those as unimplemented, https://github.com/acrmp/foodcritic/commit/aa554ce2b8b64a2ddb122981a30126431770612b


Move responsability to add required headers from clients to the transport client

this is not really part of the scope of this PR, but adding those parameters to the withRequiredHeaders makes it more obvious: it's tedious and error prone to have all the clients call this method to add the headers. I think we should move the responsibility to the transportClient to add the required headers in the send method. But I tihnk we can move forward with this PR and address this point in another PR, as well as the review of how authentication and tokens are used / named.

Originally posted by @xhanin in https://github.com/IZIVIA/ocpi-toolkit/pull/14#discussion_r1378445840

It concerns

.withRequiredHeader()
.authenticate()

As this requires an object holding a reference to the PartnerRepo and partnerUrl I can see two simple ways of making this work.
a) making TransportClient abstract, and implement send to add these things before calling a doSend that needs to be implemented by actual implementation. Migration effort would be small, and extensible, as implementation could still override existing methods
b) have a "middleware" that implements send method and wraps the actual transport client

The a) approach would be nice. It's important to be extansible on this part


Configurable Number of Lines in Healthcheck Log Output

While I was configuring Kamal for our deployments I was limited by number of lines that (by default) Kamal is printing out when running healthcheck. It is hardcoded to "50" and quite often I was only looking at the bottom of stack trace and could not figure out what has gone wrong.
Container is removed after failed deployment and I couldn't find a way to get these logs from anywhere.
So this PR is adding configurable number of lines for healthcheck.

Hi @rience, thank you for the PR!
There are some conflicts, if you could resolve them when you have time?
Also what do you think about changing the setting name to log_lines?

Hi @djmb,
That's rebased and name of setting has changed to log_lines. There is one test that is failing - I cannot see how this is connected to my test though.

@rience - The tests passed on a re-run. The integration tests are running with docker inside docker inside a GitHub action so we do get the odd random failure. Thanks for the PR!

Sure - one more thing @djmb - I've also added this short description to kamal-site: https://github.com/basecamp/kamal-site/pull/28


Use single-quoted string literals whenever possible

The additional features of double-quoted strings create more potential for mistakes, and might be slower for PHP to parse.
Hopefully there's a PHP-CS-Fixer rule for this.

There indeed is: https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/blob/master/doc/rules/string_notation/single_quote.rst

For the record, this has no impact on performance: https://www.npopov.com/2012/01/09/Disproving-the-Single-Quotes-Performance-Myth.html

For the record, this has no impact on performance: https://www.npopov.com/2012/01/09/Disproving-the-Single-Quotes-Performance-Myth.html

I thought so, since I could think of an implementation which would not impact performance.


[#1275640]: Fix broken Travis build

Uses @richardTowers Ruby pinning as that appears to be important
Actually fixes broken test too (see commit message for what was going wrong)
I've upgraded the awscli and changed the test as it was lacking loads of features but happy for that to be done separately if that is a contentious change

Closes #150
Closes #151

Fixes https://www.pivotaltracker.com/n/projects/1275640

I think you need to be more specific with the link


'xx' is not defined for arrow functions in react

I'm receiving errors that a function is not defined within a react component, on the line of the function, i.e.,
submit = () => {}

gives me 'submit' is not defined.  I know I can // eslint-disable-line but is there anyway to have standard recognize this? I'm using with babel-eslint.

Are you intentionally trying to set a global variable? In that case, you can add /* global submit */ to the top of the file. Otherwise, you probably want const, let or var before that line, e.g:
const submit = () => {}

That was a method of a class, this is in a react component, such as:
class Example extends React.Component {
  submit = () => {}
}

This issue is fixed by updating to standard v11. Cheers! ✨


Multiple controllers: Uncaught RangeError: Maximum call stack size exceeded

This is a (multiple allowed):

[x] bug

Swiper Version: 4.3.3 and lower

Platform/Target and Browser Versions: Chrome, Fx…

Live Link or JSFiddle/Codepen or website with issue: https://jsfiddle.net/bgo0rcft/19/

What you did
Just trying to inter-connect 3 or more swiper. Code example with different tests in the Fiddle (in comment on the JS section)
Expected Behavior
All swiper connected
Actual Behavior
Get the "Uncaught RangeError: Maximum call stack size exceeded" error in console when trying to swipe one slider.
I see this is an old issue, not really solved. Hope my code example could help. If you have a real solution for this or a patch/trick :)
Related to https://github.com/nolimits4web/Swiper/issues/1322

I have the same problem on my current project.
I needed to make 3 connected sliders (at least)
It seems to me that the problem of memory leakage is the cross-calculating transition and transform.
My friend suggested to me the option to implement the mechanics of the controller without using it.
That's how I solved my problem: https://codepen.io/7iomka/pen/xJddmR?editors=0010
This is a temporary solution that has a number of caveats:

This implementation is not suitable for cases where the width of the slides is different
This implementation requires additional style processing to ensure cross-browser compatibility

I would like to hear from the creator of the plugin any comments on this account, why controllers do not work crosswise? And can we somehow solve this problem in a different way?
Thanks in advance!


better types for events

The events can have much better types, and pretty much need them.

closing and merging with #5


no output

Hi,
I've installed miio, to find the token for a xiaomi plug
However when I execute miio, i have not output (comand lasts no more than 1 second to execute).
Im running this in a raspian jessie distro. Any feedback of what might be the problem?

This library uses UDP multicast for discovering devices. A good starting point would be to check that your network does not block these request and that your Pi is on the same network as the device you are trying to reach.


Fans

Description:
Fan component and insteon hub support (I still want to implement tests for insteon to make sure everything is golden before merge!)
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If code communicates with devices, web services, or a:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

I hope to get an MQTT Fan platform done before this gets merged but no promises.

@Teagan42 Does the Insteon platform use the existing Insteon Hub component? The file name should be insteon.py if it does. Otherwise, why does it not use the hub?

@robbiet480 Followed the light component - it uses the hub. The hub loads all the devices, the components just filter the devices as needed (fan, light, switch [todo], sensor [todo]). I'll rename

Can you add some tests for the component in test_init.py?
fan/__init__.py is on 52.08%

Robbie will be with MQTT platform. I can try to do some but the insteon fan
and isy (the two hubs I am familiar with) do not use the setup_platform
method. I can add it to coverage for now. I'll see what I can do though.
On Aug 25, 2016 12:23 AM, "Johann Kellerman"<EMAIL_ADDRESS>wrote:

Can you add some tests for the component in test_init.py?
fan/init.py is on 52.08%
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2964#issuecomment-242291859,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC2fZSB_HdEpG1e1XKFd4633s6glFfYSks5qjTTogaJpZM4JsMwJ
.

I'm a fan :dolphin:

This needs a demo platform and tests that use that platform.

Not every component has a demo, but I'd be happy to work on it later.
On Aug 25, 2016 7:03 AM, "Martin Hjelmare"<EMAIL_ADDRESS>wrote:

This needs a demo platform and tests that use that platform.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/2964#issuecomment-242377730,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC2fZeDF26zLX5qMVPjHCzu0d8arPwGZks5qjZKSgaJpZM4JsMwJ
.

coverage/coveralls Coverage decreased (-0.3%) to 93.631%

Please don't merge things that hurt the code coverage like this. That's why we require new components to always have a demo platform and have tests based on the component service methods (turn_on, turn_off, etc) to interact with the demo component. That way we can ensure everything works, we can actually build UI and it won't hurt our test coverage.


Enable ChunkWriter to append to an existing dataset

Allows a ChunkWriter to append to a dataset as opposed to always writing datasets from scratch. This feature is useful when the ChunkWriter is part of a 'resumable' task. The default setting is to begin writing from index 0 for backward compatibility.

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


Update design-best-practices.md

Contributing to Cortex XSOAR Content
Make sure to register your contribution by filling the contribution registration form
The Pull Request will be reviewed only after the contribution registration form is filled.
Status
In Progress
Ready
In Hold - (Reason for hold)
Related Issues
fixes: https://github.com/demisto/etc/issues/29503
Description
A few sentences describing the overall goals of the pull request's commits.
Screenshots
Paste here any images that will help the reviewer

@evisochek Done.


HADOOP-16765. Fix curator dependencies for gradle projects using hadoop-minicluster.

Gradle behaves differently than Maven when processing exclude rules through transitive
dependencies. This causes gradle projects (e.g. in Kudu) that are using Hadoop MiniCluster
to fail because different Guava version come to the classpath through Curator. In this
fix we make sure that ZooKeeper, SLF4J and Gradle gets always excluded from Curator Client
even if they would come through other Curator dependencies transitively.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
29m 58s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
17m 57s
trunk passed

+1 :green_heart:
compile
0m 20s
trunk passed

+1 :green_heart:
mvnsite
0m 22s
trunk passed

+1 :green_heart:
shadedclient
31m 13s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 20s
trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 14s
the patch passed

+1 :green_heart:
compile
0m 12s
the patch passed

+1 :green_heart:
javac
0m 12s
the patch passed

+1 :green_heart:
mvnsite
0m 15s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  2s
The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
13m  9s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
0m 16s
hadoop-project in the patch passed.

+1 :green_heart:
asflicense
0m 32s
The patch does not generate ASF License warnings.

78m 46s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1763/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1763

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml

uname
Linux 1926d931dd29 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / ab32762

Default Java
1.8.0_222

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1763/1/testReport/

Max. process+thread count
419 (vs. ulimit of 5500)

modules
C: hadoop-project U: hadoop-project

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1763/1/console

versions
git=2.7.4 maven=3.3.9

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Fix test failures caused by the deleted Vault server

Bug description
Tests under

security/pkg/nodeagent/caclient/providers/vault/
tests/integration/security/sds_vault_flow

fail because the cluster hosting the test Vault server on was deleted.
To fix these tests, a new test Vault server needs to be created, and the test cases need to be updated to use the IP address of the new test Vault server. In addition, Vault related user guides need to be updated to use the new IP address.
Expected behavior
These tests pass before the cluster hosting the test Vault server was deleted.
Steps to reproduce the bug
Run the above tests.

#13439


Conflict between Drag and Arms Reach

On my console it appears that this module is causing conflict with Arms Reach, apparently I can't use Arms Reach which is also a module that I love with this one but I can't use it.

How does the conflict manifest? What features do break when both modules are enabled?

It appeared only once and I could not print the problem on the console, even adding the module, I enter the settings to manage the module's resources, and there is no option to manage, even installed and activated, making it impossible to create that distance to the doors. Another thing is that my fighters continue to be able to open and close doors from anywhere on the map, with no distance limit.

It appeared only once and I could not print the problem on the console, even adding the module, I enter the settings to manage the module's resources, and there is no option to manage, even installed and activated, making it impossible to create that distance to the doors.

Sorry, I cannot follow there. Can you please try to rephrase that? Maybe adding a screenshot or video would also help with the explanation. What exactly would you expect to happen and what is erroneously happening?

Another thing is that my fighters continue to be able to open and close doors from anywhere on the map, with no distance limit.
Open the libwrapper settings and prioritize the Arms Reach module. That might fix it.

It should appear in the List of modules with normal priority

my foundry is in my portuguese language, but you should be able to understand it, i still haven't been able to solve the problem, i managed to take these prints, i hope it helps to understand me, my native language is not english so i may have a little difficulty trying to communicate with you, I apologize.

One of your screenshots indicates that the Drag Ruler module might not be installed correctly. Please try removing and re-installing Drag Ruler.

I just did, the same problem appeared on the console. I uninstalled and reinstalled, it's giving the same problem. Do you know what it could be?

Is Drag Ruler working correctly?

It has worked very well so far, only Arms Reach, which is a module that I also love, is having problems, is there a possibility that the problem is in Arms Reach itself and not in Drag Ruler? I'll try to message the developer of Arms Reach, maybe it's not very well adapted for V9.

If you disable Drag Ruler, does the Arms Reach module start working?

@sethmod  are you using arms reach in foundry 0.8.9? otherwise the version you indicated 2.0.11 is very old the current is 2.0.23 on foundryvtt 9.
Try to uninstall and reinstall arms reach either.


About restic backup success or failure

When I check the content of the backup log, which looks like as below, I can only identify how many files were backed up and how much space was occupied. Since there are neither error nor success messages, how can I make sure the backup completed successfully and without errors? Do no warnings and errors seen in this log mean "backup completed successfully and without errors"?
Files:           6 new,     0 changed,     6 unmodified
Dirs:            0 new,     3 changed,     0 unmodified
Added to the repo: 8.802 GiB
processed 12 files, 128.390 GiB in 52:25
snapshot 0863adf4 saved
If there are some files and part of them were backed up with errors while the others were backed up successfully without errors, can those backed up successfully be restored normally?

Your backup log doesn't contain any lines saying "error" thus all files were read successfully. See #2526 for an example of a log containing errors. The last line snapshot 0863adf4 saved indicates that restic successfully created a snapshot from which you can restore later on. Note that the exit status of restic currently does not reflect whether there have been any errors or not.
Even if there were some errors, restic would still backup every file it can read. That is you should be able to restore all files which were backed up without printing an error.

@aaa3060 This is a usage question (as opposed to a bug report or feature request), which belong in the forum (as mentioned in the issue template). @MichaelEischer cave you the answer, so I'm closing this now. Feel free to use the forum for usage discussions, it's actually pretty active!


Timezone setting of git repositories

There seem to be some inconsistency in handling timezones in pkg git repos compared to svn. For example, svn log on https://hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/Biobase/ returns
r129129 |<EMAIL_ADDRESS>| 2017-04-24 21:50:57 +0200 (Mon, 24 Apr 2017) | 1 line

bump x.y.z versions to odd y after creation of 3_5 branch

while git log on https://git.bioconductor.org/packages/Biobase gives
commit 867db36c18c4738157710dba2f9889459a0002da
Author: Herve Pages<EMAIL_ADDRESS>Date:   Mon Apr 24 19:50:57 2017 +0000

    bump x.y.z versions to odd y after creation of 3_5 branch

    git-svn-id: file:///home/git/hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/Biobase@129129 bc3139a8-67e5-0310-9ffc-ced21a209358

It would be good to configure the git server to use the same timezone setting as the builders in order to have consistent date format e.g. in build reports.

There is an option which does that as per git-svn but I don't think its worth it, because we use the git-svn interoperability to update the most recent release and master (trunk) branches.  The --convert-timezones apparently breaks that interoperability. I could give it a test go, but i'll let @mtmorgan decide if he thinks this is worth it.
Does it break anything in the build reports? Other than having the timezone difference?
--convert-timezones;;
	Store Git commit times in the local timezone instead of UTC.  This
	makes 'git-log' (even without --date=local) show the same times
	that `svn log` would in the local timezone.

This breaks interoperability with SVN, but may be cosmetically
desirable when converting a repository from SVN to Git.

Probably the builds need to do something like git log --date=local anyway, because otherwise the commits are presented in the time zone of the committer.
mtmorgan@CND5360DSX:/tmp/Rsamtools$ git log |head -n 3
commit d0079c84a0c267759d11f16e7ca9ebc84459c559
Author: Martin Morgan<EMAIL_ADDRESS>Date:   Thu Jun 15 01:54:30 2017 -0400
mtmorgan@CND5360DSX:/tmp/Rsamtools$ git log --date=local |head -n 3
commit d0079c84a0c267759d11f16e7ca9ebc84459c559
Author: Martin Morgan<EMAIL_ADDRESS>Date:   Thu Jun 15 01:54:30 2017

So far I think we should NOT convert time zones.

@aoles Let me know if this helps.

Thanks @mtmorgan! The local setting to git log seems helpful. Funny enough, apparently there is a bug in git log which prevents me from getting the desired format with the timezone offset field set correctly. Seems to be a relatively fresh thing, so not sure when a version containing a bugfix will be released and available on the builders, see<EMAIL_ADDRESS>> svn co https://hedgehog.fhcrc.org/bioconductor/trunk/madman/Rpacks/vsn
> cd vsn
> svn info |grep "Last Changed Date"
Last Changed Date: 2017-04-24 15:50:57 -0400 (Mon, 24 Apr 2017)

> git clone https://git.bioconductor.org/packages/vsn
> cd vsn
>git log --max-count=1 --date=format-local:"%Y-%m-%d %H:%M:%S %z (%a, %d %b %Y)" --format=format:"Last Changed Date: %ad%n"
Last Changed Date: 2017-04-24 15:50:57 +0000 (Mon, 24 Apr 2017)

Note the wrong timezone offset +0000 in the git output.


Cluster configuration upgrade of type StandAloneBaselineUpgradeState is already in progress and cannot be interrupted.

I'm trying to upgrade the configuration of my cluster. But i'm getting the following error:
Start-ServiceFabricClusterConfigurationUpgrade : System.Runtime.InteropServices.COMException (-2147017627)
FabricException: Cluster configuration upgrade of type StandAloneBaselineUpgradeState is already in progress and cannot be interrupted.

If I execute Get-ServiceFabricClusterConfigurationUpgradeStatus:
           UpgradeState ProgressStatus ConfigVersion Details
           ------------ -------------- ------------- -------
RollingForwardCompleted              0 1.0.0

What to do? Logs aren't telling me much...

I'm having the exact same problem. I want to run a cluster upgrade but got the message above when trying to start it. I ran Get-ServiceFabricClusterUpgrade and got the following:

I know what I need to do to fix this UnhealthyEvaluation (ironically, that's why I'm trying to run the configuration upgrade), but I can't get past this. Is there a way to get this to either rollback or complete even with the unhealthy evaluation?
This is a production cluster and we're stuck.

Can you guide me how to change service fabric app health state to green, I'm facing issues in local, sometimes its continuously refreshing for 3-4 hours, but green status is not coming, showing warnings or errors. ?


Support for permute, prior box and flatten in model_compiler

Adding support in caffe_to_nnir for reshape, flatten, permute , prior box
Adding support for flatten, permute and prior box in nnir and nnir_to_openvx
Fixes bug in concat in caffe_to_nnir and niir

This pull request introduces 3 alerts and fixes 1 when merging 8d3258ee2bc47ecbafb305ae42ffaa8a6cbbce50 into 84744efaa14276f6a46b3cd58dadf92bbc2d5655 - view on LGTM.com
new alerts:

2 for Variable defined multiple times
1 for Unused local variable

fixed alerts:

1 for Wrong number of arguments for format

Comment posted by LGTM.com

This pull request fixes 1 alert when merging 13be03170bfd1307380f4530857ad6d013445fcd into 7a71b9f794bbec2b38b40d5c344dec4895a12e32 - view on LGTM.com
fixed alerts:

1 for Wrong number of arguments for format

Comment posted by LGTM.com


chore(main): release app 0.5.0

:robot: I have created a release beep boop
0.5.0 (2024-01-08)
Features

update readme (#58) (52735bc)

Bug Fixes

app: remove console on production + fonts (bf8b611)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/FullStack-DevTool/FSDT/releases/tag/app-v0.5.0 :sunflower:


Python 3 release

Right now using 1.0.1 from conda I get an error under Python 3: missing parentheses in call to 'print' (util.py, line 307). It looks like this line is removed from master, but no release has been made since. I'm not sure if additional changes need to be made or not.

For sake of keeping this thread up to date: started in #16, pypi release made, just waiting on conda-forge package.

@RSully see https://github.com/conda-forge/cc-plugin-ncei-feedstock/pull/3
PS: Do you want me to add you as a maintainer there?

@ocefpaf thanks! They're showing up now. Is it possible to remove py3* packages for the previous version, since they don't work?
re: maintainer, I don't think that is necessary yet, I am just fine forking for the few things I submit :)

re: maintainer, I don't think that is necessary yet, I am just fine forking for the few things I submit :)

You would still need to do that :smile: but you could also merge simple updates. We have a policy that self-merges on updates is OK (and desirable) to avoid delays with new releases.

Is it possible to remove py3* packages for the previous version, since they don't work?

Done.

Hey just to confirm, we're all set with Python 3 now right? @ocefpaf

Hey just to confirm, we're all set with Python 3 now right? @ocefpaf

All can say is that the tests are passing :smile:
I am not a ncei plugin user and never tested this out in the wild. @RSully any comment?


Does not shows the real image

I'm trying to use your example for viewing the DICOM image. But when I'm trying to render the DICOM Image it does not shows the real image or it could be said as the image gets pixelated or corrupted. I'm using the ant design upload component to convert the image to base64. Is there any other way to resolve the problem? Or is it an conversion error in the upload component that I've been using.

Thanks in advance...

Hi,
So, it's definitely capable of showing real DICOM images, I've already done that on another project.
I will take a look at it and share my code here or update the repository.

today probably

Hey,
I've just updated the repository to use a real DICOM image and it's using this library https://github.com/rii-mango/Daikon for parsing.


Add gpu ci workflow

Add GPU workflow

暂时关掉了 megdnn_test 验证 megbrain_test, 修复版同步出来 rebase 后再放开 megdnn_test

LGTM

Please squash your commit and place code change in a separate commit for easier cherry-pick.

Please squash your commit and place code change in a separate commit for easier cherry-pick.
done


Security Groups for OnPrem with AAD Authentication

Hi,
just a quick question: BC22 introduced the new Security Groups in the System Application to replace the existing User Groups in BC.
In BC22.1 this functionality does not work for OnPrem installations using Azure AD Authentication, is there any plan to make this work in the future before User Groups get replaced completely?
Following Call Stack will be run when trying to get the existing Security Groups from Azure in this context:

Lookup of the NewAadSecurityGroupName field on the New Security Group Page
LookupAvailableGroups()
SecurityGroup.GetAvailableGroups(Rec);
InitializeAadGroups()
AzureADGraph.GetGroups()
AzureADGraphImpl.GetGroups()
if not CanQueryGraph() then

and then in the Initialize of CanQueryGraph() the first check is:
if not EnvironmentInformation.IsSaaS() then
            exit(false);

Which will always return false here, stopping execution of any code after this.

I just upgraded a customer from BC14 to 22.1 and they rely heavily on user groups. They also use AAD authentication, so this will need to be fixed before the user groups are removed. It was a bit of a dissapointment to them that they couldn't manage the user/permission group mapping in AAD since they now have to do that also in BC.

I will have @IhorHandziuk make an official statement.

@JesperSchulz @IhorHandziuk How about that statement? We need to know if we have to abandon using Security Groups.

We've had a lot of internal discussions to which extend we can set aside time to enable security groups for on-prem. Unfortunately it will be hard to find time to add support at this point in time. However, you are not the only ones look into this limitation. Amongst a few others, @miljance has been looking into this as well. I wonder if this could be completed as an open source project amongst interested parties?

@JesperSchulz
Currently we are analysing the impact on our OnPrem customer. A lot of them are in the queue for the update. I would say some of them are authenticating over AAD because of the multifactor authentication and all of those that did update did already hit the limitation. Currently we did not decide to invest in correcting this issue but if we do so I will be glad to contribute. For now, we are notifying the customers what is the limitation and that event though security groups sound cool, authenticating over AAD is not currently supported.

I have done further investigating in to the problem. This is what I have tried and what I have found out:

I created my own "New" action on the Security Groups page.
I copied all the code following the Call Stack provided in the original post by @ptrk-tr into new codeunits that allowed me to remove the EnvironmentInformation.IsSaaS() check.
The copied AzureADGraphImpl.GetGroups() now works and returns a complete Dictionary of Security Group Ids and Names.
This allows the lookup page to be populated.

This leads me to believe that if the if not EnvironmentInformation.IsSaaS() check is changed to something like if not (EnvironmentInformation.IsSaaS() or IsOnPrem()) the Security Groups lookup would work for both Cloud and OnPrem.
Another alternative that would be very simple for you to implement, would be to give us an IntegrationEvent at the start of the IsSaas() procedure, that lets developers override the return value.

That means that with this approach windows groups are enabled with Azure AD Authentication. I have asked @IhorHandziuk and @JesperSchulz and they would approve a pull request for this issue under following circumstances:

Make AAD security groups work OnPrem when the user is AAD authenticated
Add some extra logic to make sure we don't mix and match Windows groups and AAD groups in the same environment (maybe just adding some dialog that tells the admin that they need to delete Windows groups first if they attempt to add AAD security groups  the environment with Windows groups)

@joelsigurd That means that approach you have provided is not the desired end-state. At least that is how it sounds.

Make AAD security groups work OnPrem when the user is AAD authenticated

My first idea was using the IsAccessControlServiceAuthentication procedure in Codeunit 9801 or the underlying DotNet version, but looking into the CustomSettings.config 'AccessControlService' could also be used for Windows Azure Access Control Service as authentication type. Now the authentication documentation doesn't mention that type of authentication so I don't know if it's still something to check for.

Add some extra logic to make sure we don't mix and match Windows groups and AAD groups in the same environment (maybe just adding some dialog that tells the admin that they need to delete Windows groups first if they attempt to add AAD security  groups the environment with Windows groups)

How would those get mixed and matched? Isn't Windows Authentication handled differently in the GetAvailableGroups() procedure in Codeunit 9871? As in, wouldn't they only get matched if the authentication type of the instance itself gets changed? But maybe I'm missing something here.
Reference from BC22.1:
if IsWindowsAuthentication() then
  foreach LocalWindowsGroupName in NavUserAccountHelper.GetLocalWindowsGroups() do begin
      SecurityGroupBuffer."Group ID" := CopyStr(SID(LocalWindowsGroupName), 1, MaxStrLen(SecurityGroupBuffer."Group ID"));
      if not GetDisallowedWindowsGroupIds().Contains(SecurityGroupBuffer."Group ID") then begin
          SecurityGroupBuffer.Code := CopyStr(CreateGuid(), 1, MaxStrLen(SecurityGroupBuffer.Code));
          SecurityGroupBuffer."Group Name" := CopyStr(LocalWindowsGroupName, 1, MaxStrLen(SecurityGroupBuffer."Group Name"));
          SecurityGroupBuffer.Insert();
      end;
  end
else begin
  InitializeAadGroups();

  foreach AadGroupId in AadGroups.Keys do begin
      UserProperty.SetRange("Authentication Object ID", AadGroupId);
      if UserProperty.IsEmpty() then begin
          SecurityGroupBuffer.Code := CopyStr(CreateGuid(), 1, MaxStrLen(SecurityGroupBuffer.Code));
          SecurityGroupBuffer."Group ID" := CopyStr(AadGroupId, 1, MaxStrLen(SecurityGroupBuffer."Group ID"));
          SecurityGroupBuffer."Group Name" := CopyStr(AadGroups.Get(AadGroupId), 1, MaxStrLen(SecurityGroupBuffer."Group Name"));
          SecurityGroupBuffer.Insert();
      end;
  end;
  end;

Looking at the authentication documentation provided above I don't see the reason to having a check on IsSaaS().
If a user is trying to create Security Groups, and the IsWindowsAuthentication() check is false, isn't the user guaranteed to be trying to create AAD Security Groups?

@JesperSchulz Is this still not prioritized in the Business Central development team?
At the moment we are using a hybrid solution with AAD for authentication, and Windows AD + User Groups for managing permissions. This has us concerned since User Groups are marked as Obsolete and planned for removal in BC25. Is this still the plan?
As I see it there is no other option than using User Groups if you want to authenticate with AAD on Business Central On-Prem.

You should not loose any functionality when user groups are deprecated. Try tagging along on this Yammer / Viva Engage thread: https://www.yammer.com/dynamicsnavdev/#/threads/show?threadId=2515780681072640.
For convenience, copying the latest statement: permission sets support all the operations and scenarios that user groups did. There is no loss of functionality, and most actions have a direct mapping. If you have some examples scenarios becoming difficult to achieve, please share them.

This issue should not get tracked in this repository. It's tracked both in bcideas and is discussed in various Viva Engage threads. I'm hence closing this issue.
This repository is only for Business Central extensibility requests or code contributions towards the 1st party Business Central apps.


Move acquire/release to right before/after execution call

Add more info to FetchContext
Make AcquireReleaseOperator take the plan instead of the operator for the child, to ensure acquire can happen before buffers accessed in planning
Move acquire release to BaseCombineOperator level, to better ensure release only happens after buffer access are done

To ensure clean acquire/release semantics, dataBuffers can no longer cache DirectByteBuffers. Removing it from BitmapInvertedIndexReader, RangeInvertedIndexReader and NullValueVectorReaderImpl.
Included jmh benchmarks in the change, which shows

while cached bitmaps outperform in a setup with large heap and high repeatability, it performs significantly worse in a setup with tight heap and low repeatability, compared to always building the bitmap.
in the case where alwaysBuild performed slower, the overhead we're talking about here is <1 microseconds per read. In the case where alwaysBuild performed better, it outdid by 4.6 microseconds per read.
These results remain consistent regardless of low/high cardinality

Codecov Report

Merging #7493 (a1e2489) into master (240bcb8) will increase coverage by 37.67%.
The diff coverage is 59.64%.

@@              Coverage Diff              @@
##             master    #7493       +/-   ##
=============================================
+ Coverage     32.19%   69.87%   +37.67%
- Complexity        0     3327     +3327
=============================================
  Files          1514     1129      -385
  Lines         75167    53390    -21777
  Branches      10966     8037     -2929
=============================================
+ Hits          24203    37308    +13105
+ Misses        48870    13441    -35429
- Partials       2094     2641      +547

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
69.87% <59.64%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...operator/AcquireReleaseColumnsSegmentOperator.java
0.00% <0.00%> (ø)

...ore/plan/AcquireReleaseColumnsSegmentPlanNode.java
0.00% <0.00%> (ø)

...pinot/core/plan/maker/InstancePlanMakerImplV2.java
72.17% <0.00%> (+5.21%)
:arrow_up:

...gment/index/readers/BitmapInvertedIndexReader.java
100.00% <ø> (+100.00%)
:arrow_up:

...al/segment/index/readers/RangeIndexReaderImpl.java
89.02% <ø> (+89.02%)
:arrow_up:

...ava/org/apache/pinot/segment/spi/FetchContext.java
0.00% <0.00%> (ø)

...not/core/operator/combine/BaseCombineOperator.java
79.48% <33.33%> (-9.56%)
:arrow_down:

...perator/combine/GroupByOrderByCombineOperator.java
80.95% <79.41%> (+17.66%)
:arrow_up:

...gment/index/readers/NullValueVectorReaderImpl.java
100.00% <100.00%> (+100.00%)
:arrow_up:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1311 more

Continue to review full report at Codecov.

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


Popover - html content for Tooltips

Hey @saadeghi ,
I noticed the new tooltip component was added, which is great.
However, it seems that this tooltip component has the limitation of displaying only plain text without the option to put well formatted html content.
Maybe it makes sense to either consider improving the current tooltip or add a new component named e.g. popover, for displaying html content.
Here is an example of one such implementation.

I think dropdown dropdown-hover is basically doing the same job. Showing an HTML element on hover 🤔

Not that have mentioned it, you right, no need for special component like that 😃  ... closing.

One notable difference is that the tooltips have the arrow/caret attached to the tooltip, indicating the source element. Popovers do not have this same UI. I too would like some kind of option that mimics tooltips but can be any contents, and have the anchored arrow styling.

There's new native HTML popover API and we are planning to add it to daisyUI as soon as Firefox starts supporting it. It would be perfect for this use case:
https://developer.mozilla.org/en-US/docs/Web/API/Popover_API

Since the popover API is relatively new and not widely supported, with only newer browser versions offering compatibility, wouldn't it be more practical to not use it?

Yes, it's not widely supported yet. But it will be in the future.

we are planning to add it

Hey guys, it's already supported in all major browsers, including Safari 🎉

@kuchta Yes. It will be in daisyUI 5.0


crash on header

I get a segfault when parsing my library ...
#10 0x00005555555a5b62 in std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::_M_replace (
    this=0x7fffffffd820, __pos=0, __len1=0,
    __s=0x555555892e32 "Calculates a global bilevel image\n     * @param src input image\n     * @param dest output image\n     * @param threshold threshold value (@see Otsu)\n     * @param flags\n     */\n    template <typename A"...,
    __len2=18446650248713916878) at /usr/include/c++/13.1.1/bits/basic_string.tcc:511
#11 0x00005555555a4d63 in std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::replace (
    this=0x7fffffffd820, __pos=0, __n1=0,
    __s=0x555555892e32 "Calculates a global bilevel image\n     * @param src input image\n     * @param dest output image\n     * @param threshold threshold value (@see Otsu)\n     * @param flags\n     */\n    template <typename A"...,
    __n2=18446650248713916878) at /usr/include/c++/13.1.1/bits/basic_string.h:2209
#12 0x00005555555a4c78 in std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::replace (
    this=0x7fffffffd820, __i1=0 '\000', __i2=0 '\000',
    __k1=0x555555892e32 "Calculates a global bilevel image\n     * @param src input image\n     * @param dest output image\n     * @param threshold threshold value (@see Otsu)\n     * @param flags\n     */\n    template <typename A"..., __k2=0x0)
    at /usr/include/c++/13.1.1/bits/basic_string.h:2418
#13 0x00005555555a3510 in std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >::append<char const*, void> (this=0x7fffffffd820,
    __first=0x555555892e32 "Calculates a global bilevel image\n     * @param src input image\n     * @param dest output image\n     * @param threshold threshold value (@see Otsu)\n     * @param flags\n     */\n    template <typename A"..., __last=0x0)
    at /usr/include/c++/13.1.1/bits/basic_string.h:1520
#14 0x000055555559f7d2 in Parser::translate (this=0x7fffffffda50,
    comment="/**\n     * @brief Calculates a global bilevel image\n     * @param src input image\n     * @param dest output image\n     * @param threshold threshold value (@see Otsu)\n     * @param flags\n     */", entity=...)
    at /home/hartmut/Code/doxide/src/parser/Parser.cpp:226
#15 0x000055555559ddc5 in Parser::parse (this=0x7fffffffda50,
    source="/* vim: set tabstop=4:softtabstop=4:shiftwidth=4:noexpandtab */\n/*\n * SSTT - Simplified Spatial Target Tracker\n *\n * © Copyright 2007-2023 Hartmut Seichter\n */\n#ifndef SSTT_CORE_THRESHOLD_HPP\n#define"..., global=...)
    at /home/hartmut/Code/doxide/src/parser/Parser.cpp:58
#16 0x00005555555eb1b5 in Driver::build (this=0x7fffffffdb40) at /home/hartmut/Code/doxide/src/Driver.cpp:150
#17 0x000055555559c21c in main (argc=2, argv=0x7fffffffde98) at /home/hartmut/Code/doxide/src/doxide.cpp:11

Header looks like this:
/* vim: set tabstop=4:softtabstop=4:shiftwidth=4:noexpandtab */
/*
 * SSTT - Simplified Spatial Target Tracker
 *
 * © Copyright 2007-2023 Hartmut Seichter
 */
#ifndef SSTT_CORE_THRESHOLD_HPP
#define SSTT_CORE_THRESHOLD_HPP

#include <sstt/core/globals.hpp>

namespace sstt::core
{

    /**
     * @brief Calculates a global bilevel image
     * @param src input image
     * @param dest output image
     * @param threshold threshold value (@see Otsu)
     * @param flags
     */
    template <typename ArrayTypeSrc, typename ArrayTypeDst>
    void ThresholdGlobal(ArrayTypeDst &dst,
                         const ArrayTypeSrc &src,
                         auto threshold,
                         auto min_val,
                         auto max_val)

I tried to remove the @Otsu reference with no avail.

Thanks @seichter. The segfault itself should be fixed in 6f0dbb0, but still looking to accommodate the usage of @brief that you have here (currently it's assumed to be followed by a sentence ending in a period, which is a little too strict).

Have also now improved the handling of @brief to accommodate this usage, and I've gone ahead and appended brief descriptions to detailed descriptions, assuming that this is typically the desired behavior (is it?), or at least a good default.
On usage of @see, it is intended that this is used to create a separate paragraph, much like @param, rather than used inline as you've done here. Looks as though Doxygen behaves the same way. What you could do instead is use a link in Markdown, e.g. (see [Otsu](#otsu)).

Tagged a v0.0.1 with these fixes, and the packaging pipeline is running, although will take a little while.
Will close the issue now, but please feel free to reopen if the issue has not been addressed.


[Improvement][APi server]  complement data    The complement can be more flexible

Now the complement data only supports day granularity ,
I think  if Provide a quartz expression and a time interval
like this
{"startTime":"2021-07-21 00:00:00","endTime":"2121-07-21 02:00:00","crontab":"0 5 * * * ? *"}
scheduler time  is

2021-07-21 00:05:00
2021-07-21 01:05:00

I just looked at the code， It is found that this feature has been implemented


Added 5.7 Release Notes

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

Docs Build status updates of commit 98c5cf4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/release-notes/NuGet-5.7.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit db49477:
:clock10: Pending: waiting for processors (5 builds ahead of you)
:warning: Docs Build is busy, currently there are 5 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit db49477:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/release-notes/Index.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-5.7.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit f5417f9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/release-notes/Index.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-5.7.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit baee462:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/release-notes/Index.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-5.7.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ef101ce:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/release-notes/Index.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-5.7.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@zivkan - Should we recognize contributors from the community as part of the Release Notes?

Using git log to find the commits made by people who are not Microsoft employees (or interns) between the 5.6 and 5.7 releases, I found these:

Who
PR
Issue

campersau
3433
NuGet.Protocol does not support getting a version's download count from search #9086

Joseph Musser (jnm2)
3241
Suppress the  element when it is redundant #5134

Volodymyr Shkolka (BlackGad)
3273
NuGet cannot restore from HTTPS sources that require Client Certificates #5773

campersau
3120
HttpRequestMessage gets reused multiple times which is not supported with the SocketHttpHandler #8661

Marius Ungureanu (Therzok)
3357
HttpSourceAuthenticationHandler SemaphoreSlim future proofing #9463

Sunner (SuNNjek)
3088
nuget.exe spec Generates a copyright Tag with Hard-Coded Text of Copyright yyyy Instead of $copyright$ #8696

Olivier Spinelli (olivier-spinelli)
3335
Improve DOTNET_HOST_PATH environment variable handling #7438

goyzhang
3370
Add "latest" option for -MSBuildVersion #8808

@chgill-MSFT Did you want to finalize these notes with the added items & additions & i'll take on 5.8+?

@JonDouglas yep, I'll add the finishing touches and publish these!

Docs Build status updates of commit 8d2d93a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/release-notes/Index.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-5.7.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit e66a32f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/release-notes/Index.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-5.7.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 4871ba2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/release-notes/Index.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-5.7.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Basic Sandbox Level - Level Design

To have a sandbox level to work with and test functions is essential, so we don't mess with each others work.
The point is to have many different small challenges close together to test it and develop on it.


Resolve lint warnings for react hooks dependency arrays

When building the frontend, thee are a lot of lint warnings by the react lint package about dependency array issues which can lead to rendering issues

most of these warnings are not relevant unfortunately. that lint check is not robust.


Add :dev_portal_team_id option to pilot

Addresses https://github.com/fastlane/fastlane/issues/4878
This helps the iTunes Transporter deal with situations involving multiple teams, taking advantage of the work in https://github.com/fastlane/fastlane/pull/4746

this looks good to me! is the option being used anywhere though?

Great question. It is used in the ItunesTransporter through ENV['FASTLANE_TEAM_ID']
https://github.com/fastlane/fastlane/pull/4746/files#diff-03ad193b71c4d81874c287fcd4bedcc5R302

👍


Added: GCSArgumentSetter

Added: GCSArgumentSetterConfig
Added: GCSArgumentSetter-action.md
Added: GCSArgumentSetter-action.json
JIRA:
https://issues.cask.co/browse/PLUGIN-168

lgtm, please squash your commits.

squshed


Symbol completion: Generic type

No Types being offered. Surprisingly, if you enter one letter you do get matches:

Fixed as a side effect of #905


Initialize the repo

At least an empty README.md, i want to give it a try by creating a simple HAL for GPIO pins

Done :). I look forward to seeing what you come up wit.

Are you planning something based on https://github.com/rust-embedded/embedded-hal?

@HarkonenBade yes, I'm planning to use the embedded-hal traits

Awesome! Very interested to see more.

I have been working today on creating a little .svd file by hand for the peripherals, mainly I'm going first to support GPIO, so svd2rust can be used. I'm not very optimistic since it's a lot of hard work, but as peripherals are in need, we can modify the .svd file adding the necessary peripherals and registers.
I've tried to mimic peripheral API by hand directly but it seems like an overkill and error prone.
And now that Espressif announced on the llvm-xtensa repo that they're close to release a new version of the back-end I think that having a functional example is good showing the capabilities of Rust, and possibly get some new hands here.

Would you consider putting what you have so far in a WIP pull request? As I have some time tomorrow and would be happy to add some registers.

Let me finish some registers and I'll add a WIP PR.


server-author-view pom is malformed

The server-author-view pom cannot be distributed to JCenter/Maven Central
It needs to include the scm section for example:
   <scm>
         <connection>scm:git:git://github.com/odpi/egeria.git</connection>
         <developerConnection>scm:git:ssh://github.com/odpi/egeria.git</developerConnection>
         <url>http://github.com/odpi/egeria/tree/master</url>
     </scm>

as do our other poms
Please also include a description
Since this is not yet released I think we can skip this for 2.5 (note it will NOT be there)
This has been repeated for the other child projects, however they do appear to sync ok (odd limitations of bintray)
Please advise if we need to fix this for 2.5. Since it's only the parent project, and a function still in development I think not. (time consuming to fix as requires some manual build/push sync)

Unfortunately these restrictions are non-obvious, and Jfrog/Maven Central don't (as far as I see) document them clearly either. There is no validation in our build,

reassigning - needs to be done for 2.6 release

reassigning - needs to be done for 2.6 release


Move the packages:write permission in the ockam-healthcheck workflow

Move the packages:write permission in the ockam-healthcheck from workflow-scope to the build_ockam_healthcheck job-scope

We love helping new contributors! ❤
If you have questions or need help as you explore, please join us on Discord. If you're looking for other issues to contribute to, please checkout our good first issues.

Hey, I'd like to take this issue up!

Hello @tharun634 I have assigned you to https://github.com/build-trust/ockam/issues/6177, we can assign you to this issue when the former is done and closed

@tharun634 while we review/merge the other PRs, if you want please go ahead and take this one as well. Please keep the PRs for each issue separate


chore: Fix incorrect npm tag for previous version release

See here: https://github.com/TanStack/config/pull/24

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

Comparison is base (a857615) 87.74% compared to head (195cf49) 87.74%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #559   +/-   ##
=======================================
  Coverage   87.74%   87.74%
=======================================
  Files          31       31
  Lines         816      816
  Branches      182      182
=======================================
  Hits          716      716
  Misses         95       95
  Partials        5        5

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


Linux Couse (Bengali)

Noticed that there was no Linux Book in Bengali.
Want to add this Course to the list: https://youtube.com/playlist?list=PLMTKJq4uuKqXVg7S7XujEsONl9ZVT4X0p

This is a video course. Put it in our courses-bn.md files

Got it 👍

PR: https://github.com/EbookFoundation/free-programming-books/pull/7674

Not is necessary open issues if you are be able to send the pull request (PR) :wink:.
Anyway, if you submit a PR, help us to autoclose this issue. How? Read this article about how to linking a PR to an issue

Understood 👌. Will keep that in mind from the next time.
Thanks for letting me contribute :)


Use case: generating random free email adress  based on generated first and last name

Is there possible to generate both last/first name and email for the "same" person?

Duplicate of #557


Consider using nix for building nerdctl-full binaries

Using nix may help reproducible builds of nerdctl-full binaries (nerdctl + containerd + buildkitd + ...).
The binaries must remain to be runnable on non-nix distros.

Maintaining the nix pkgs is also quite complicated, compared to the straightforward current solution...
Podman tried to do it, but gave up (and thus doesn't offer any binaries anymore). Only CRI-O still tries

@AkihiroSuda is this still something we want to pursue?

We may still try, but not with a high priority


Fixes misspelling of “address” on Floating IP list screen.

Purpose or Intent
Fixes a typo on the Floating IP screen.
Before:

After:

Fixes BZ 1344081

@dclarizio @h-kataria please review.

👍 Will merge when green

looks good.


Optimize structs for gas

Goals: Down to ~200k gas estimates on build()
Targets:

[x] Pack Position.Info struct
[x] Pack Roller.Snapshot struct
[ ] Create OverlayV1Market.State struct with oi data + timestamp (uint32)

Base estimates from main:
OverlayV1Market <Contract>
   ├─ constructor -  avg: 2870303  avg (confirmed): 2870303  low: 2870303  high: 2870303
   └─ build       -  avg:  449580  avg (confirmed):  449580  low:  422498  high:  476016

After packing Position.Info struct to
    struct Info {
        uint88 oiInitial; // initial open interest
        uint88 debt; // debt
        bool isLong; // whether long or short
        uint256 entryPrice; // price received at entry
    }

have build() gas estimates reduced by ~ 90k gas:
OverlayV1Market <Contract>
   ├─ constructor -  avg: 2864845  avg (confirmed): 2864845  low: 2864845  high: 2864845
   └─ build       -  avg:  366339  avg (confirmed):  366339  low:  350224  high:  388739

oi values have a max from OverlayV1Factory of uint256 public constant MAX_CAP_OI = 8e24; // 8,000,000 OVL (initial supply). This implicitly imposes a max on debt and collateral values as well. log(8e24) / log(2) = 82.7262742772967 means we have enough cushion to accommodate all values to smallest precision with uint88 for oi and debt (thus the choice).

After packing Roller.Snapshot struct to
    struct Snapshot {
        uint32 timestamp; // time last snapshot was taken
        uint32 window; // window (length of time) over which will decay
        int192 accumulator; // accumulator value which will decay to zero over window
    }

have build() gas estimates reduced by ~ 20k gas:
OverlayV1Market <Contract>
   ├─ constructor -  avg: 2955641  avg (confirmed): 2955641  low: 2955641  high: 2955641
   └─ build       -  avg:  339631  avg (confirmed):  339631  low:  334712  high:  352826

will need to come back and write tests for overflow edge cases of uint32(block.timestamp) and accumulator values that exceed int192.max (if even possible given oi caps).

With position struct, better packing if we add bool liquidated:
    struct Info {
        uint120 oi; // initial open interest
        uint120 debt; // debt
        bool isLong; // whether long or short
        bool liquidated; // whether has been liquidated
        uint256 entryPrice; // price received at entry
    }

This with roller packing (above) produces gas estimates for build() of
OverlayV1Market <Contract>
   ├─ constructor -  avg: 2951317  avg (confirmed): 2951317  low: 2951317  high: 2951317
   └─ build       -  avg:  316104  avg (confirmed):  316104  low:  311479  high:  327167

These estimates are now on par with a Curve swap.

The OverlayV1Market.state approach ended up increasing gas estimates by around 20k from the lows above, so won't include those as changes.

Using tests/markets/test_build.py::test_multiple_build_creates_multiple_positions as a measure of estimates when the system gets going (non-zero storage values), gas is around 260-280k:
OverlayV1Market <Contract>
   ├─ constructor -  avg: 2988677  avg (confirmed): 2988677  low: 2988677  high: 2988677
   ├─ build       -  avg:  264729  avg (confirmed):  264729  low:  255966  high:  325060

This is promising and still on par with a Curve swap on the tricrypto pool.


Mouselook senstivity affected by zoom and crouch.

To reproduce: Use any item that grants the player zoom, or crouch. When they try looking around their sensitivity will be lowered, and when using 30x zoom, you can barely use the mouse (Easily done on Villa on Agent/Special Agent).

Crouching affects sensitivity in the base game and I didn't modify this behavior.
For the zoom though it looks like the problem is that I forgot to multiply it by the FOV ratio.

I think the zoom problem should now be fixed. I'll think about the crouching thing.

Not only crouching, Elvis too - shorter characters have faster sensitivity. This happens on N64, but is fixed in 1964 GEPD so all the sensitivities match.

Alright, if it's fixed in 1964 GEPD, then I might as well fix it here too.

Should be disabled now.

Okay, I'll close this then. Feel free to reopen if anyone finds out that something broke.


Slickgrid CSS file missing on application load.

The application is trying to access: https://localhost:3000/images/header-columns-bg.gif

Blocked until nginx goes in.

instead of replacing value in the main.js add the class that reference the gif in the override css and change the background image:
e.g.
.slick-header-columns {
background-image: none;
}


Add marker cluster from pd.DataFrame or csv

Reference: https://python-visualization.github.io/folium/plugins.html#folium.plugins.MarkerCluster

This feature has been implemented.
ipyleaflet
import leafmap
m = leafmap.Map()
data = 'https://raw.githubusercontent.com/giswqs/leafmap/master/examples/data/us_cities.csv'
m.add_points_from_xy(data)
m

folium
import leafmap.foliumap as leafmap
m = leafmap.Map()
data = 'https://raw.githubusercontent.com/giswqs/leafmap/master/examples/data/us_cities.csv'
m.add_points_from_xy(data)
m


No such file or directory: 'gen/caffe2/mpi/mpi_common.h'

This is the error happened when I typed 'make'. What's the problem?

Could you send me the complete build log? You can get it via (under Bash)
make 2>&1 | tee log.txt

Never mind. I reinstalled the dependencies and the problem is solved.

Thanks - let me know if further problems pop up.


Hot Exit: backupService.ts feedback

Some feedback:
I notice that you have some test methods in here, I typically would rather have a subclass for testing purposes to make it clearer that a method is only used from tests (that is more a personal taste).
You could use process.type to check if you are in the renderer or browser process. Though given my other feedback of splitting services up, I think the renderer should not have a dependency to this backup workspace management service.
You should not call fs.existsSync(), just call fs.readFile() and handle the error. That saves you one fs call.
The setCurrentWorkspace is very ugly. Instead I would pass in the IWorkspaceContextService as optional dependency and get the workspace from there. Though again, given my feedback on splitting up services, I think this method is a good indication that we need 2 services: one for main side to manager backup workspaces and one for the renderer to backup files.

[x] Sub-class test https://github.com/Microsoft/vscode/pull/14115/commits/c328d1b808842070fce7c777096e75f0176eb15e
[ ] Avoid fs.existsSync
[ ] Remove setCurrentWorkspace


Deployment US GOV Virginia

Describe the bug
Deployed solution following Getting started guide. Per the guide all of the pieces are there no errors during deployment.
Being deployed to USGOV Virginia
https://github.com/Azure-Samples/cognitive-services-speech-sdk/blob/master/samples/ingestion/ingestion-client/Setup/guide.md
To Reproduce
Upload files to audio-input but don't get any results back. Files just stay in audio-input container
Appinsight I am seeing the following exception. We deployed this in test subscription that wasnt in GOV space and everything worked as expected. Within USGOV Virginia cannot get it to run properly
Event time | 9/1/2021, 8:24:10.640 PM (Local time) |
-- | -- | --
Message | The listener for function 'StartTranscriptionByTimer' was unable to start. Retry failed after 6 tries. (No such host is known.) (No such host is known.) (No such host is known.) (No such host is known…[show more] |
Exception type | System.Net.Sockets.SocketException |
Failed method | Azure.Core.Pipeline.HttpClientTransport+d__8.MoveNext |
Problem Id | System.Net.Sockets.SocketException at Azure.Core.Pipeline.HttpClientTransport+d__8.MoveNext |
Assembly | Azure.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=92742159e12e44c8 |
Severity level | Error |
Telemetry type | exception |
Device type | PC |
Client IP address | <IP_ADDRESS> |
Role name | starttranscriptionfunction-20210831t193152z |
Role instance | d0126224e64fa271e1cd08a43a27e488b4b1a0efa0476f4d823c3ba26c6cf6b8 |
SDK version | azurefunctions: <IP_ADDRESS> |
Sample rate | 1

Hi @Eceder,
deploying the Ingestion Client to Azure Gov should be possible, but is not supported by the ARM template right now.
There are 2 more things you need to do to make it work. You need to go to your StartTranscriptionFunction and change the following application settings (under Settings->Configuration->Application Settings):
1.) Change the variable "AzureSpeechServicesRegion" to "usgovvirginia"
2.) Add a new variable "IsAzureGovDeployment" and set the value to "true"
Please let me know if that worked for you.
Best,
Henry

I made changes but still getting same failures
Should I also be making these changes to the FetchTranscriptionFunction?

Hi,
the reason why you were seeing this issue was that the storage endpoint was formatted incorrectly for Azure Gov (core.windows.net vs. core.usgovcloudapi.net).
I have updated the ARM template now to include Azure Gov regions (https://github.com/Azure-Samples/cognitive-services-speech-sdk/pull/1248) , please redeploy the Ingestion Client with the updated template and let me know if it worked!
Best,
Henry

Redeployed and is working as expected. Thank you for your time and effort’s

Perfect, thanks!


Show warning on create func app if files already exist

@StephenWeatherford,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


add jshintrc

Fixes #126
Changes proposed:

[X] Add
[ ] Prepare
[ ] Delete
[ ] Fix

Things I've done:

[X] I've pulled the latest master branch
[ ] I've run npm install to install all the dependenies
[ ] I've checked my code, it is clean.
[ ] I've run ESLint.
[X] My pull request fixes an issue, I referenced the issue.
[X] My pull request is not too big, otherwise I've already squashed it.
[ ] I've run npm run build to build the extension. (If I had done something with the extension.)

adds jshintrc
Jetlag from holiday :smile:
I'm done. New icon requests, please.

Let's use something similar to the logo they have in their site https://github.com/jshint

Okey, will update.

ready


plugins/languages: move to by-name

Also just individual plugins that don't share code actually.

Oops looks like I need to add stuff to the new ltex-extra test, real quick.

I was hesitant to move languages/* given how "generic" some of the names will be... But I suppose since the name always matches plugin.<name> from the option path it shouldn't be an issue?
Is there any value in "language" plugins being grouped together in a category vs being in by-name?

I didn't think so, like you said, they are unique by-name already and it doesn't help me to find several of the plugins when they are nested that way.

@mergifyio queue


Do not cache the tracer ref

TODO review and send upstream PR
The original code does not rely on the trace provider to get the tracer, leading to initialization issues. This fixes it by relying on the tracer provider instead.

Submitted the PR over here: https://github.com/XSAM/otelsql/pull/115


Bump wellcomeml to 2.0.0

Following #140 , it would be useful to bump wellcomeml to 2.0.0 so we would have more fine control

I believe this is fixed by the new release


Feature Requeset: Allow appending multiple shell commands to one command function

Would be nice to be able to fire off a list of commands that correspond to a single button command in the script.

if you combine the commands as a string with && you should already be able to do this:
commands={
  'play': f"""curl http://localhost:1234/api/playback/stop && curl http://localhost:1234/api/playback/rewind"""
}

but I'll also be adding support for supplying multiple commands via a tuple. I'll have to double check whether these will be run concurrently or sequentially though... My guess would be that because a subprocess is launched, the requests might not be run in succession, but rather in parallel, in which case the && solution might be more appropriate.

as an aside, can you post an example of the curl command you use to set your volume now?
I have an idea for how you can increase/decrease your volume with a fixed amount, rather than just set it to preset values.
I recall you mention btw that you wanted to just increase your volume with a fixed amount, rather than having preset volumes, and I think you can achieve that with:
basically with a command like: f"""curl http://localhost:1234/api/set/volume/?volume=$(curl 'http://localhost:1234/api/info/status' | jq '.config.volume.percent + 10')"""
which relies on jq to parse the Client.GetStatus response, find the volume percentage, and increase it by ten, and then pass that on to the Client.SetVolume request as variable, using Command Substitution.
(guessing the URLs and the data format here, but I hope you get the gist)
echo '{"config":{"instance":1,"latency":0,"name":"","volume":{"muted":false,"percent":74}},"connected":true,"host":{"arch":"x86_64","ip":"<IP_ADDRESS>","mac":"00:21:6a:7d:74:fc","name":"T400","os":"Linux Mint 17.3 Rosa"},"id":"00:21:6a:7d:74:fc","lastSeen":{"sec":1488026416,"usec":135973},"snapclient":{"name":"Snapclient","protocolVersion":2,"version":"0.10.0"}}' | jq '.config.volume.percent + 10'

I will try using &&.
Yes being able to set volume incrementally to be able to fine tune the volume would be nicer than having to set absolute values.
Volume curl commands.
curl -d '{"id":8,"jsonrpc":"2.0","method":"Client.SetVolume","params":{"id":"7c:c7:09:62:5e:ad","volume":{"muted":false,"percent":30}}}' http://<IP_ADDRESS>:1780/jsonrpc
curl -d '{"id":8,"jsonrpc":"2.0","method":"Client.SetVolume","params":{"id":"b8:27:eb:0f:58:68","volume":{"muted":false,"percent":30}}}' http://<IP_ADDRESS>:1780/jsonrpc

While it can probably be done in a single command inside the python script, I would suggest creating a set of helper shell scripts that you can also run by themselves, and thus can reuse in other scripts.
Start by creating the following shell script, get_volume.sh:
#!/bin/sh
# Error when a variable is not set
set -u
RPC_URL="http://<IP_ADDRESS>:1780/jsonrpc"
# Set the ID to the value of the first commandline argument, or default to the 7c address
DEFAULT_ID="7c:c7:09:62:5e:ad"
ID="${1:-$DEFAULT_ID}"
curl --silent -d '{"id":8,"jsonrpc":"2.0","method":"Client.GetStatus","params":{"id":"'$ID'"}}' "$RPC_URL" | jq '.config.volume.percent'

be sure to make get_volume.sh executable, and put it on your PATH (put it in ~/bin is usually a good idea), or specify the full path to the script in the next script.
and create: increase_volume.sh:
#!/bin/sh
set -u
VOLUME_INCREMENT=10
# Set the ID to the value of the first commandline argument, or default to the 7c address
DEFAULT_ID="7c:c7:09:62:5e:ad"
ID="${1:-$DEFAULT_ID}"
NEW_VOLUME="$(echo "$(get_volume.sh "$ID")" + "$VOLUME_INCREMENT" | bc)"
RPC_URL="http://<IP_ADDRESS>:1780/jsonrpc"
curl --silent -d '{"id":8,"jsonrpc":"2.0","method":"Client.SetVolume","params":{"id":"'$ID'","volume":{"muted":false,"percent":'$NEW_VOLUME'}}}' "$RPC_URL"

(You'll need to have bc installed to do the calculation.)
Now just set the increase_volume.sh script as your command:
commands={
  'vol+': f"""curl increase_volume.sh && curl increase_volume.sh b8:27:eb:0f:58:68"""
}

This command should increase the volume on the default (7c) address, and the b8 address. For decrease volume you could just copy the increase script, and change the + for a - :)
(Cleaner of course might be to further abstract it, and allow specifying the volume change via a command line argument as well.)
I hope it helps, or at least gives you some more ideas about what you can do with it. :)
(NOTE: I don't have snapcast myself, so I've based this on your commands and the examples found at https://github.com/badaix/snapcast/blob/master/doc/json_rpc_api/v2_0_0.md#clientgetstatus)

Ok, I'm getting out of "null" on the get-volume script. I think the issue lies with the jq command. Because if I skip the script and run the curl command directly I still get null
curl --silent -d '{"id":8,"jsonrpc":"2.0","method":"Client.GetStatus","params":{"id":"7c:c7:09:62:5e:ad"}}' http://<IP_ADDRESS>:1780/
jsonrpc | jq '.config.volume.percent


use externalip in kubeconfig when cluster externalip is valid

What type of PR is this?

/kind optimize
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

none

Additional documentation, usage docs, etc.:

/kind optimize

/cc @lyp256 @qinyer

Codecov Report

Merging #605 (e1a992b) into master (8c7f0e0) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master     #605      +/-   ##
==========================================
- Coverage   14.05%   14.04%   -0.01%
==========================================
  Files         138      138
  Lines       19482    19490       +8
==========================================
  Hits         2738     2738
- Misses      16448    16456       +8
  Partials      296      296

Impacted Files
Coverage Δ

pkg/apis/core/v1/handler.go
0.38% <0.00%> (-0.01%)
:arrow_down:

pkg/scheme/core/v1/k8s/kubeadm_step.go
0.00% <0.00%> (ø)

/LGTM


Connection failed (no connection to the remote host). Internal error: 1. Error details: Code: 2543

Describe the bug
I have followed the code example for transcribing an audio file, and I always get this error:
Connection failed (no connection to the remote host). Internal error: 1. Error details: Code: 2543. SessionId: 2caaa438233949fbb5ad24dea9b47a6c

To Reproduce
Steps to reproduce the behavior:

Create a speech_config.yml containing this:

--- # Azure config
subscription: "add your key from azure here" # key 1 from keys
region: "northeurope"
speech_recognition_language: "de-DE"

Run the code from here: https://paste.xinu.at/H6tO/

Expected behavior
I expected this output:
done with recognising. results:
Radio an    # German for "radio on"

Version of the Cognitive Services Speech SDK
1.17.0
Platform, Operating System, and Programming Language

OS: Arch Linux (5.12.6-arch1-1)
Hardware - x86
Programming language: Python
Browser [e.g. Chrome, Safari] (if applicable) - not applicable

@Ablesius thanks for opening this issue. Can you please provide full Speech SDK Log, as per the instructions here? https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/how-to-use-logging
Thanks,
Darren

Hi Darren,
here you go:
speech-sdk-log.txt

Currently, undergoing the same issue.
I noticed this pattern "[507337]: 312ms SPX_TRACE_ERROR: AZ_LOG_ERROR:  tlsio_openssl.c:691 error:1416F086:SSL routines:tls_process_server_certificate:certificate verify failed"
My run situation is a bit special though since we are running with the golang SDK inside a docker from "SCRATCH" container
But def sounds like some certs are missing in my machine so maybe it is also the same issue you are having

@Bob-Thomas Regarding your issue, please open a new GitHub issue in the Speech SDK GO repo . We will try to help you there. Please provide full Speech SDK log. This page may be relevant to your issue, but I'm not sure yet without a full log.

Thanks for the information, but I discovered soon after that the docker container from "SCRATCH" did not contain the openssl/ssl libraries.
After copying them over from my build stage it works like a charm

@dargilco yes, it works, thank you! Funny, because I tried that before and then it didn't work. Must have been a glitch :) Cheers!

@Ablesius, glad to hear that!


Restart Python yum helper before each repo enable/disable

Description
This restarts the yum Python helper before each repo enable/disable in order to work around a bug in the yum API that occurs on CentOS 6.
Issues Resolved
#7540
Check List

[ ] New functionality includes tests
[ ] All tests pass
[ ] RELEASE_NOTES.md has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

Merging this under the obvious fix policy.

I made the suggestion to copy this line of code, so I think we're okay without DCO here.


encoded percent to prevent EZID error

I used similar logic as in preprint EZID to fix percent error.

More testing in progress.


Out of memory

I set up the environment correctly and generated 2 samples with 2 input records.(which means it could handle a small input and my setup should be correct.)
But the OS killed the process since memory error. Either the program consumed too much with my input or  there is memory leak in the program.
My evaluation input is about 200 MB.

To fix the issue. I submitted a PR https://github.com/kentonl/e2e-coref/pull/20

The PR can't be merged now due to some code refactoring, but I've fixed this issue in the latest commit (c7ce6c2894fd75d155323698f7f38ec93c4913c4)


Fix errors in cargo fetch usage guide

This is a recreation of #9785.

By the doc, cargo build will always access the network unless --frozen or --offline is present. Instead, cargo fetch will fetch dependencies, and which will allow running subsequent Cargo commands in offline mode.

A general template was used for --target that is not appropriate for cargo fetch.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
Please see the contribution instructions for more information.

Would it be better to use #if helper to avoid duplicate? Like #9189 did.

I agree with @weihanglo that using #if to keep the --target docs in one spot is probably a good idea, otherwise seems good to me though!

So, should I add 3 flags for:
1. Change "The default is the host platform." to "The default is for {{actionverb}} for all architecture."

2. Add a sentence "Use the value `all` to include _all_ targets."

3. Hide the paragraph "Note that specifying this flag makes Cargo run in a different mode..."

respectively?

I feel like cargo-tree diverges from the common one to some extent, so I vote for keeping cargo-tree as is.

@bors: r=weihanglo

:pushpin: Commit 08734a664362b9ea39ba7d15ef340eff25749af4 has been approved by weihanglo

:hourglass: Testing commit 08734a664362b9ea39ba7d15ef340eff25749af4 with merge c974e3528c9a81bc900a23114302538de293f2be...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing c974e3528c9a81bc900a23114302538de293f2be to master...


zpa_application_server: description field not set

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform Version
Terraform v1.1.9
on linux_amd64

provider zscaler.com/zpa/zpa v2.1.5

Affected Resource(s)

zpa_application_server

Terraform Configuration Files

resource "zpa_application_server" "test1_app_server"{
  name                          = "test1-app-server"
  description                   = "test1-app-server"
  address                       = "<IP_ADDRESS>"
  enabled                       = true
}

Debug Output
https://gist.github.com/cbornman/6a5deff072821a4d691ab95189acab7c
Panic Output
n/a
Expected Behavior
A blank description field in the actual resource should have been set to the value in the Terraform code.
Actual Behavior
Blank description field in resource was not populated with value from Terraform code even though TF said it would make that change.
Steps to Reproduce

Started with empty ZPA config.
Created zpa_application_server resource "test1_app_server" using Terraform.
Blanked out the "description" field manually, from the ZPA GUI.
Re-ran "terraform apply" to restore missing description field.
Although plan contained update to description field, field was not updated in actual resource.

Important Factoids

I am working on admin.zpabeta.net (by overriding the base URL in the source code).
If the description field is set to a non-empty value and I change the description in the TF code it is still not updated in the actual resource (in other words, it's not limited to an empty field being set - it seems like that field can't be updated).
The "enabled" flag exhibits the same behavior.
The "name" field does not - it can be updated after creation from the TF code via "terraform apply".

References
n/a

@cbornman A fix will be released early next week along with a couple other enhancements/fixes. The issue is the the resource update function which needs to have the description field updated. Other fields are not being affected by this.
Thanks for reporting.


Support async predictors rebased

⚠ This is intentionally a draft rebase of #1896 for review and fixing tests prior to replacing the history of #1896.

Not bothering with rebasing and instead moving forward in #2010


108 Gracious Gifts

Need help with the persistent effect. Tried removing the ghost rock check just to see if it would work at sundown and it still didn't allow discarding more than one card.

Pushed a fix for persistent effects of outfits. Your is a first outfit card with a persistent effect :)


src: replace message age in chats with username

Replaces the message age in global and private chat with a username stored in the client's sessionStorage, which is cleared on closing of the tab/window. Adds a prompt before global chat for entering a username.
Known issues:

Opening an existing private chat history when the username has
not been set in this tab sets the username to null
Chat prompt inputs are not being enforced as required

Is this a patch, a minor version change, or a major version change
patch

Looks good to me! Can you change the version number to reflect this update?


Is it ok if I refactor this mod?

Hi,
I would like to make this mod more extensible/pluggable, so that other mods can integrate with it to register their own alert types, sound libraries, etc. But to do that I'm going to have to make significant changes to the code (particularly config.lua and control.lua) so thought I'd ask if it's OK to do that (and send a pull request) or whether you'd prefer me to create a completely separate mod?
Guy

Sure, go ahead!

Hi,
I would like to make this mod more extensible/pluggable, so that other mods can integrate with it to register their own alert types, sound libraries, etc. But to do that I'm going to have to make significant changes to the code (particularly config.lua and control.lua) so thought I'd ask if it's OK to do that (and send a pull request) or whether you'd prefer me to create a completely separate mod?
Guy

yo did you make the new mod from it ?
I really loved the mod but its deprecated atm D:


Update medical safety alerts name

Change the medical saftey alerts name from 'Alerts and recalls for drugs and medical devices'
to 'Alerts, recalls and safety information: drugs and medical devices'.
See https://govuk.zendesk.com/agent/tickets/4587261

There are a few places where this content item is reference, but it looks like this is the only that needs changing.


Jd/remove raw data

This PR uses specific methods for each type. Closes #359

Actually, get_raw_data_type should probably be removed if you're removing get_raw_data

Removing get_raw_data_type requires reimplementing the map as mentioned in #356 so we should defer that to another PR.


add jump layer button

Issue: I add a new type of button where, when the user pass the mouse in, the button jump in layers.

Thank you @printf-ana for the contribution but unfortunately, I don't this idea fits in our project.