BUG: Yeo-Johnson Power Transformer gives Numpy warning

Describe your issue.
scipy.stats.yeojohnson can overflow for reasonable input.
This is the same issue as https://github.com/scikit-learn/scikit-learn/issues/23319.
Reproducing Code Example
import numpy as np
from scipy.stats import yeojohnson

x = np.array([2003.0, 1950.0, 1997.0, 2000.0, 2009.0, 2009.0, 1980.0, 1999.0, 2007.0, 1991.0])
yeojohnson(x)

Error message
python3.9/site-packages/numpy/core/_methods.py:239: RuntimeWarning: overflow encountered in multiply
  x = um.multiply(x, x, out=x)
python3.9/site-packages/numpy/core/_methods.py:250: RuntimeWarning: overflow encountered in reduce
  ret = umr_sum(x, axis, dtype, out, keepdims=keepdims, where=where)
Out[4]:
(array([1.63003810e+154, 4.59874518e+153, 1.41487438e+154, 1.51873102e+154,
        1.87712777e+154, 1.87712777e+154, 9.45185553e+153, 1.48330911e+154,
        1.79094464e+154, 1.22758933e+154]),
 47.211105636582914)

SciPy/NumPy/Python version and system information
numpy 1.24.1
scipy 1.10.1

@lsorber ping

Those values look reasonable, but in fact, that data set runs into a fundamental limitation of 64 bit floating point when one attempts to find the optimal (i.e. maximum likelihood) Yeo-Johnson parameter.
Using mpmath, one can compute the actual optimal parameter $\hat{\lambda}$ to be approximately 99.26.  (I'm adding this to mpsci, but I haven't pushed it to github yet.) The problem is that for each value in that data set, $(x_i + 1)^\hat{\lambda}$ overflows when computed with 64 bit floats. E.g.
In [15]: x
Out[15]:
array([2003., 1950., 1997., 2000., 2009., 2009., 1980., 1999., 2007.,
       1991.])

In [16]: x**99.26
<ipython-input-16-42b558e52fe2>:1: RuntimeWarning: overflow encountered in power
  x**99.26
Out[16]: array([inf, inf, inf, inf, inf, inf, inf, inf, inf, inf])

In other words, the result of the optimal Yeo-Johnson transform for this data set is not representable with 64 bit floats.

In practice, a user can avoid this problem by using an additional preprocessing step to scale the data. For example, the data can be normalized to have mean 0 and standard deviation 1 before applying yeojohnson. In scikit-learn, one would use sklearn.preprocessing.StandardScaler.  In SciPy, scipy.stats.zscore can be used:
In [23]: from scipy.stats import zscore, yeojohnson

In [24]: z = zscore(x)

In [25]: yeojohnson(z)
Out[25]:
(array([ 0.68110757, -1.01124976,  0.16177483,  0.39752566,  1.40110131,
         1.40110131, -0.54744521,  0.31375023,  1.13770228, -0.18007565]),
 2.387535792418072)

Of course, whether or not such a step is valid for a user depends on why they are transforming the data in the first place and what they are going to do with the transformed data.

@WarrenWeckesser what do you recommend for this issue? Document this limitation?

The proposed fix for sklearn's implementation is here: https://github.com/scikit-learn/scikit-learn/pull/26188

Scikit-learn and scipy have very similar implementations of Yeo-Johnson that are both prone to two sources of overflow:

The first source of overflow is caused by certain invocations of np.power with unbalanced bases and exponents.
The second source of overflow is more difficult and occurs when λ grows for a marginal gain in log likelihood. In this case, applying the Yeo-Johnson transform on the input will cause overflow.

Both types of overflow can be mitigated. The fix that I proposed for scikit-learn (https://github.com/scikit-learn/scikit-learn/pull/26188) resolves them as follows:

For the first source of overflow: instances of np.power are replaced by instances of np.exp so that bases and exponents are combined before being exponentiated.
For the second source of overflow: I proposed to regularize the exponents of the transformed input so that they do not blow up for a marginal gain in log likelihood. The regularization term has a small weight relative to the log likelihood objective, and so generally does not affect the result.

The scikit-learn maintainers proposed to fix the issue in scipy and then have scikit-learn depend on its improved implementation.

The scikit-learn maintainers proposed to fix the issue in scipy and then have scikit-learn depend on its improved implementation.

Yes we did, in the hope to serve the greater community and only have one yeojohnson implementation, i.e. the one in scipy.
I hope the fix/ pr of @lsorber is welcome here.

@WarrenWeckesser
Item 1: Even better, great!
Item 2: First, a minimal working example that illustrates the problem:
import numpy as np
from scipy.stats import yeojohnson, yeojohnson_llf
x = np.array([2003.0, 1950.0, 1997.0, 2000.0, 2009.0])
xt, lmb = yeojohnson(x)
log_likelihood = yeojohnson_llf(lmb, x)  # RuntimeWarning: overflow encountered in power: out[pos] = (np.power(x[pos] + 1, lmbda) - 1) / lmbda

# xt = array([2.01405528e+154, 5.67790578e+153, 1.74805534e+154, 1.87644716e+154, 2.31954974e+154])
# lmb = 47.23900784138352
# log_likelihood = -13.9893342

Notice that the lambda that maximises the log likelihood is rather large, and that the transformed data's average exponent has grown from about 3 to 154. This exponent is already beyond what single precision can represent, and is getting close to what double precision can represent.
By adding a regularisation term that sums the squared exponents of the transformed data xt with a weight of 1e-4, we obtain the following result:
# xt = array([349.48615808,<PHONE_NUMBER>4,<PHONE_NUMBER>5,<PHONE_NUMBER>3,<PHONE_NUMBER>7])
# lmb = 0.7292862554669863
# log_likelihood = -15.2971991

The transformed data are now in a more reasonable range, at the cost of a 10% decrease in log likelihood. The regularisation weight could be zero by default so as not to change the current behaviour. This regularisation term is not standard AFAIK. I proposed it for scikit-learn to address the issues described above.
Another way to deal with the overflow is to change the optimiser from optimize.brent to optimize.fminbound (which also uses Brent's method) and pre-compute bounds on lambda that would prevent overflow from occurring in the optimisation. This is something I would consider regardless of the decision to add the regularisation term above.

Related to this issue: I came across a relatively recent paper [1] that proves that the Yeo-Johnson negative log likelihood objective is convex, and proposes to replace the Brent optimiser with a more robust exponential search. In Figure 2, they also illustrate overflow issues with Brent optimisation of the Yeo-Johnson log likelihood that are similar to the ones described in this issue.
[1] https://arxiv.org/abs/2210.01639

@lsorber could you submit the PR with at least #1 and the change to fminbnd? Perhaps include the regulation idea in a separate commit so it can be considered separately.

@lsorber @lorentzenchr were you interested in submitting the fix?

Yes, I'd like to help implement the fix according to your proposal. I haven't had much time the past few weeks. Will try to fit it in in the coming weeks!

@WarrenWeckesser @mdhaber @lorentzenchr I implemented fixes (1) and (2) in #18852. These should resolve the two sources of overflow.


Enforce LF.

This is actually needed because on Windows with autocrlf: true, which is the default, linting changes the line endings to LF.


OAS 3+ has no property 'definitions'

OAS 3.x is not valid as the property definitions (even though it is added as empty list) is automatically added by flasgger. In case OAS 3.x this has been replaced by components (see here). I am not sure how to properly fix this as it has been used throughout the package.
A dirty hack that works for me is simply removing the definitions from the dict. (https://github.com/IKNL/flasgger/commit/b4451780d801688f49e2a1405e00883502365412)
I assumed (maybe wrong) that OAS3+ was supported as it was used in one of the examples. Although no were else in the documentation it states that OAS3+ is supported.

Any news on this?

same problem here as well

Any news on this?
edit: Apparently you can define the schemas inside the template when you create the Swagger instance. Maybe update documentation to properly show this?

You can define schemas and components (and possibly others) in the template. However it is, as far as I know, not possible to remove the definitions label in the output-swagger. Which results in an invalid OAS 3.0 specification.

yes.. how did you solve the problem? modifying base.py in my virtual environment is not a solution for me.

yes.. how did you solve the problem? modifying base.py in my virtual environment is not a solution for me.

So we removed the part where definitions is added, see: https://github.com/IKNL/flasgger/commit/b4451780d801688f49e2a1405e00883502365412
We also published this hack on pypi, see link above.

ah it's a fork! Thanks!

@Kiolali, @frankcorneliusmartin , @AndreasNesKile - I am using flasgger 0.9.4 - the generated swagger spec contains the "definitions" tag. Is there a fix to this yet?

We made a hack: https://github.com/flasgger/flasgger/issues/267#issuecomment-494401828

The definitions tag is still there but the UI no longer complains about it, it seems. So on that note ill close this issue.

When pasted into the swagger editor (http://editor.swagger.io/), it generates validation error due to the definitions tag.

Good point

It would be great if the issue could be solved.
I still get errors if I try to generate Client stubs using the openapi-generator (Version 4.3.1 and 5.0.0-beta2): "attribute definitions is unexpected"

Another datapoint: the extraneous definitions key trips up https://github.com/p1c2u/openapi-spec-validator


x/tools/cmd/godoc: Reset button for doc examples

example: https://golang.org/pkg/html/template/#example_Template_block
The only way to reset a code example when you mest up trying something is to refresh the page.  Note that ussualy you are just on this url https://golang.org/pkg/html/template without a hash tag to the example directly so your page wil jump up when you refresh.
Possible solutions update the hash tag when you expand the example

Or introduce a reset icon somewhere. (where and how i am not sure yet, feel free to discuss)

I think this can be solved by exposing a pilcrow (¶) symbol besides the examples too. Because once you have the link, you can just click on it and hit refresh and your page will come back to the original position.
This takes care of resetting the example contents and has the added benefit of making the examples more readily shareable. I find it cumbersome to go all the way up the page, find the example anchor link and share it.
@gertcuykens - Does that work for you ?
/cc @andybons , @bradfitz, @ysmolsky  for opinions.

@agnivade The semantics of all the various expanding/collapsing sections are all over the place. It would be nice if they were unified, possibly including all having ¶s for quick linking as you suggest. They need a lot of work for #22171, probably a full rewrite. There are a lot of edge cases to work out.
I could see a reset button being useful, regardless. It might be more user friendly.

but don't know if it would be intuitive for others too?

The original issue was that refreshing the page goes to the top because there is nothing to anchor to the example. And putting an anchor solves it. I don't understand where counter-intuitiveness comes into this ?

I could see a reset button being useful, regardless.

I am a soft negative on that. I think doing a quick refresh is cleaner than maintaining state and adding a button to revert to that state. Also, adding the button would probably need the change to be propagated to playground also to be consistent. I prefer godoc to be clean and minimalistic as it is. No strong opinions though.

Agree on minimalistic. By intuitive I mean that everybody has a tendency to just refresh the page in general when something is wrong on the page, by page I mean any website. So if the url get hash tagged automatically when user scrolls down or in my case when I expand the example, then I don't need to know what (¶) stands for. I just refresh the page.

So if the url get hash tagged automatically when user scrolls down or in my case when I expand the example, then I don't need to know what (¶) stands for. I just refresh the page.

I don't think the url should be hash tagged automatically on expanding the example. That will cause confusion because it will jump to the top of page. The pilcrow symbol will be exposed on mouse-hover, same as the other structs and functions today. Hence, it becomes clear that if you click on the symbol, the example will get anchored and you are then free to refresh the page. I think this is pretty intuitive and explicit behavior.

Technically you can prevent jumps using https://developer.mozilla.org/en-US/docs/Web/API/History_API But in reality it's to much effort needed compared to your solution. And I prefer simplicity and consistency over everything so your solution is better :)

The collapsible section headers (Overview, Index) use the same code as the examples. Would those get ¶ too? Regardless, do they go outside the clickable area of the toggle element?

The collapsible section headers (Overview, Index) use the same code as the examples. Would those get ¶ too?

Hmm ... yea to be consistent, I would guess they need it too.

Regardless, do they go outside the clickable area of the toggle element?

I was thinking, we can show the ¶  beside the text (like Example (Block) ¶) itself on hover so that it is consistent with other structs and functions. And use an event propagation logic through js so that if you click on the ¶, the event does not percolate down to the toggle element, letting it act as an anchor link. Clicking on the toggle element anywhere outside ¶ preserves the hide/show behavior.

You can't nest a tags. The browser'll force close the outer tag when it hits the inner a tag. I'm not sure that would work unless you were doing some wacky positioning to move the ¶ back over into the link area without overlapping the text.
Even if nested a tags worked or there was a reasonable workaround, you could diagram the clickable areas of that set up as [ toggle label ] [ ¶ ] [ invisible continuation of toggle area ]. That doesn't seem like ideal usability. The part that's not the toggle is invisible until you're over it which means accidentally hitting the ¶ when moving too fast. For the examples, I'm not entirely sure the "invisible continuation of toggle area" was intentional or just a result of the cascade. Even if it's okay to get rid of that, the same argument applies to the Index, etc., section headers which are 100% width.
It would be fine if it went ¶ ▹ Example but that looks a bit weird and then either those are flipped from the others which is a bit annoyingly irregular (and would seem random to a casual user until the pattern was worked out) or all of the ¶ need to be moved to the front which would be a change for people accustomed to them coming after the item.

Gotcha. I will look into it.


Add attributes to SearchBar form

Fixes #xxx
This PR does the following:

The SearchBar accepts an attributes prop, but it is currently not being implemented. Seems like a small oversight. I need to use it for SCC to add method and action attributes, which are on existing SCC search bar.

Front End Review:

[ ] View the example in Storybook

Can you create a bug ticket in the DSD board and link it in the description? Just so your contribution is also on the JIRA board.
This is fine but method and action are actual form element attributes that I would prefer to be directly passed rather than in an attributes object. How do others feel about this? This is not a blocker for this PR.

I agree with Edwin - attributes is a fallback we're using for people who need stuff quickly, we can totally keep it but if there are specific form props that you need, let's put them here explicitly

Add method and action to SearchBar as well! Thanks for the suggestions. Also ticket here: https://jira.nypl.org/browse/DSD-112


Danger Swift fails on some PRs

Failures on CircleCI:
One:
#!/bin/bash --login -o pipefail
swift run danger-swift ci
[45/45] Build complete!swiftKwiftt.swiftProvider.swiftt
Too long with no output (exceeded 30m0s): context deadline exceeded
#!/bin/bash --login -o pipefail
Two:
swift run danger-swift ci
[1/1] Planning build
[2/9] Compiling Logger Logger.swiftwift
[5/24] Merging module DangerShellExecutor
[22/26] Compiling RunnerLib String+Path.swiftwiftift
[23/26] Merging module RunnerLib
[25/26] Merging module RunnerLib
[28/38] Compiling DangerDependenciesResolver FileCreator.swift2K
[33/38] Compiling DangerDependenciesResolver Script.swiftwift2K
[38/43] Merging module DangerDependenciesResolverataProvider.swiftt
[41/44] Compiling Runner Edit.swiftManager.swift
[44/45] Merging module Runnerwiftt
[45/45] Build complete!swift
Three:
#!/bin/bash --login -o pipefail
swift run danger-swift ci
[45/45] Build complete!swiftKwiftt.swiftMaker.swift[2Kt
Too long with no output (exceeded 30m0s): context deadline exceeded

are you caching the .build?
I saw that happening on Circle, when the cache is used it sometimes just stops.

yes @f-meloni we are caching the .build but we can avoid that if it fixes the problem. Just curious to know how caching could result in above timeout scenarios since it's happening rarely only for a few PRs. Please explain.

To add some more details to this. The issues occurs repeatedly on a particular PR/commit.  Since the .build folder is caches across all PR (JOBs) . i don't think its something to do with .build cache else we would have either

All CI jobs should get stuck
Totally random occurrences across PRs/ Commits and Jobs

There is something in the particular PR changes which is causing the problem. Also this is not something related to Danger Set up as when i was debugging this. there where no changes to Danger  on the PR/Commit.

We have very similar issue - danger 3.18.0. It hangs on public func diff(forFile file: String, sourceBranch: String) -> Result<FileDiff, Error>


Checking if mirrorful server has been shutdown

This is a solution to issue #116. It uses long polling to check if the server is up. If the server is closed then the cli sends a fetch to an endpoint that sends it back to the polls.

Hey @karanvirsb thanks for all your hard work here. I'm a bit worried a long polling solution might lead to a lot of overheard maintenance for little gain (because it's not THAT often the user accidentally kills it), but will take a peek when I get a second. Thank you so much!

Hey @karanvirsb thanks for all your hard work here. I'm a bit worried a long polling solution might lead to a lot of overheard maintenance for little gain (because it's not THAT often the user accidentally kills it), but will take a peek when I get a second. Thank you so much!

Yeah, no problem. If it is too much overhead we can through this idea out. But long polling is better than using sockets which would ping every few seconds. I did look into server-sent events but it does not work as well due to the server closing and not being able to emit the event in time.

Hey @karanvirsb. getting back to this one because i actually really like this feature and want to merge it in:

i resolved merge conflicts for you. FYI looks like lot of changes, but its really just prettier changes in the examples dir, so we can ignore it.

not sure it works, does it work for you? i got this "error"

we'll have to add a thing that only does the long polling on the package and not on web.

@karanvirsb when you're done working on this lmk and happy to merge it in. feel free to post in #contributors

Closing this because we are going to focus on building out the web version (not the localhost version) instead. Thanks so much for your hard work on this! The localhost version is just really tricky.
Full speed ahead on web! 🎊


How to do something different?

Can I use this to check whether either of my cell phones is connected to my wifi at 11 pm every night, then if it is, trigger the maker channel URL?
For example:
At 11:00 PM, if [MAC ADDRESS 1] or [MAC ADDRESS 2] is connected to [WIFI NETWORK] then [IFTTT MAKER CHANNEL URL]

There are other means to do that in an automated fashion. Check out https://home-assistant.io


Evening Shutdown Script - Reported by Nick Buckingham

Expected Behavior
PPUD DEV and UAT environment shutdown at 21:00 and start back up at 06:00
Actual Behavior
following the daylight saving time adjustment where the clocks have rolled back 1 hour the environments are now shutting down at 20:00 and starting back up at 05:00.
Steps to Reproduce the Problem
No response
Version
No response
Modules
No response
Account
PPUD

This has been corrected by PR https://github.com/ministryofjustice/modernisation-platform/pull/5441
Raising new issue to make a permanent fox

https://github.com/ministryofjustice/modernisation-platform/issues/5449


Stats tracker Backend and Frontend

This PR resolves issue #117
For the Backend:
I integrated the stats to be returned in the request with the user profile data since this is public info and can be returned at once.
Let me know what you think of this approach I will reformat it if it needs to be separated.
For the Fronted:
Since everything was set up on the frontend I only added the data to their fields. I am not familiar with React so please let me know if something should be done differently.

Thanks, I will note that. 👌

Sorry, conflicts appeared after some merges @milensski

Sorry, conflicts appeared after some merges @milensski

Should be ok now.

Other than the small comment, LGTM


Checkpoint?

Hi iGEM Toronto Team,
Really impressive work on building a generative model for plasmid sequences! We want to try out your model and noticed that while there is a training script, the checkpoint file is not provided. Do you mind sharing this?
Thanks!

@rohansachdeva Thank you for your kind words! Unfortunately, we are not releasing the checkpoint after we have approvals from Garton Lab, possibly after iGEM Jamboree.


Missing files when creating packages

It seems that the plugin packager doesn't correctly populate the the client_files when creating the package:
mcollective_agent_shell::client_files:
- application/shell.rb

See https://github.com/puppetlabs/mcollective-shell-agent/tree/master/lib/mcollective/application. The sub-directory was not added.
$ mco shell
error 2016/11/09 17:59:05: pluginmanager.rb:171:in `rescue in loadclass' Failed to load MCollective::Application::Shell: cannot load such file -- mcollective/application/shell/watcher
Application 'shell' failed to load:

   cannot load such file -- mcollective/application/shell/watcher (LoadError)

fixed by https://github.com/ripienaar/puppet-mcollective/pull/65 thanks!


chore: hide labels from API reference

PR summary
This PR modifies the Resource Controller working examples so that the labels displayed in the output will not be included in the API reference.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

@mkistler is this an acceptable approach to displaying labels in the output while ensuring that the code does not appear in the API reference?

:tada: This PR is included in version 0.18.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[boolinq] update to 3.0.3

Library name: boolinq
New version number: 3.0.3
Git tag 3.0.3 was added: https://github.com/k06a/boolinq/tree/3.0.3
Users asked to upgrade, but I am not familiar with vcpkg: https://github.com/k06a/boolinq/issues/49

Requesting the help of @JonLiu1993


platformio not compile MySensors 2.0

Not compiled standard examples.
What is the problem?

PlatformIO 2.0 Build System is not compatible with MySensor 2.0.
See https://forum.mysensors.org/topic/1369/best-ide-to-use-for-mysensors-projects/24
Please switch to PlatformIO 3.0 http://docs.platformio.org/en/stable/installation.html#development-version


Development: Update theme components to use signals as example

Checklist
General

[ ] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[ ] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[ ] I chose a title conforming to the naming conventions for pull requests.

Client

[ ] Important: I implemented the changes with a very good performance, prevented too many (unnecessary) REST calls and made sure the UI is responsive, even with large data (e.g. using paging).
[ ] I strictly followed the principle of data economy for all client-server REST calls.
[ ] I strictly followed the client coding and design guidelines.
[ ] Following the theming guidelines, I specified colors only in the theming variable files and checked that the changes look consistent in both the light and the dark theme.
[ ] I added multiple integration tests (Jest) related to the features (with a high test coverage), while following the test guidelines.
[ ] I added authorities to all new routes and checked the course groups for displaying navigation elements (links, buttons).
[ ] I documented the TypeScript code using JSDoc style.
[ ] I added multiple screenshots/screencasts of my UI changes.
[ ] I translated all newly inserted strings into English and German.

Motivation and Context

We want to migrate to the new way of Angular 18 and potentially use this PR as example PR for the documentation.
Description

This migrates the theme switcher component and the components it affects to the latest Angular development standards with signals.
Issues

[ ] Missing text in theme switcher popover (title and sync with operating system)
[ ] Remaining failing client tests:

[ ] ProgrammingExerciseInstructionComponent

[x] should emit that no instructions are available if there is no problem statement
[x] should NOT update markdown if the problemStatement is changed
[x] should NOT update the markdown if there is no participation and the exercise has changed
[x] should still render the instructions if fetching the latest result fails
[ ] should create the steps task icons for the tasks in problem statement markdown
[ ] should create the steps task icons for the tasks in problem statement markdown with no inserted tests

[x] MonacoEditorService

[x] should correctly handle themes

Steps for Testing

Prerequisites:

1 Instructor
2 Students
1 Programming Exercise with Complaints enabled

Log in to Artemis
Navigate to Course Administration
...

Exam Mode Testing

Prerequisites:

1 Instructor
2 Students
1 Exam with a Programming Exercise

Log in to Artemis
Participate in the exam as a student
Make sure that the UI of the programming exercise in the exam mode stays unchanged. You can use the exam mode documentation as reference.
...

Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance even for very large courses with more than 2000 students.
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance even for very large courses with more than 2000 students.

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Performance Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Screenshots

test


missing gitignore in generator

Hello. First of all thanks for generator.
The bug is quiet simple. Npm by default when you do npm publish ignores .gitignore. As a result file generator-userscript/generators/app/templates/.gitignore is not present in npm package and it cause generator to fail.

Thanks for reporting. I should have tried to actually install and run what I have published to npm.


[BUG]

Reporting an Issue or Missing Feature
Cannot use Get-PnPProperty to get properties of Teams Tabs
Expected behavior
I should be able to hydrate the properties of a Teams Tabs with Get-PnPProperty
Actual behavior
I get this sad error message.

It says, "Get-PnPProperty : Cannot bind parameter 'ClientObject'. Cannot convert the
"PnP.PowerShell.Commands.Model.Teams.TeamTab" value of type
"PnP.PowerShell.Commands.Model.Teams.TeamTab" to type "Microsoft.SharePoint.Client.ClientObject"."
Then it tells me I'm ugly and no one likes me.
Steps to reproduce behavior
Connect-PnPOnline -Url https://m365x995492-admin.sharepoint.com
Get-PnPTeamsTab -Team "SOC Team" -Channel "General" | Tee-Object -Variable c
Get-PnPProperty -ClientObject $c[0] -Property TeamsApp
What is the version of the Cmdlet module you are running?
1.9.0
Which operating system/environment are you running PnP PowerShell on?

[x] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

@ToddKlindt - this is by design. It is designed to work with SharePoint CSOM only.
If you look at the source code, we check whether the CSOM property is available or not , and load it if required.
https://github.com/pnp/powershell/blob/e06c687263f07f3d2908970a2cac42ece6f71d46/src/Commands/Base/GetProperty.cs#L34

Okay, I'll buy that. :) Is there an alternative method for me to get the TeamsApp property for a Teams Tab? I know I can get it with the Graph SDK, but that module is a mess. I avoid it if possible.

You can get it using this commands:
$tabs = Get-PnPTeamsTab -Team "<team -name>" -Channel "<channel-name>"
$teamsTabData = Get-PnPTeamsTab -Identity $tabs[0].Id -Team "<team -name>" -Channel "<channel-name>"
$teamsData.TeamsApp

There are also other properties exposed, you can check them :

That got me where I needed to go. Thanks @gautamdsheth .


[Proposal] Support search rankings

The ts_rank and ts_rank_cd methods are really useful when creating a search.
A proposal to support this:
public enum RankType
{
    Frequency, // ts_rank
    Density // ts_rank_cd
}

public enum RankOrder
{
    Descending,
    Ascending
}

public class SearchOptions<T>
{
    public RankType RankType { get; set; } = RankType.Density;

    public RankOrder OrderBy { get; set; }

    public Expression<Func<double, bool>> Where { get; set; } // only display rank over/equal to X

    public Expression<Func<T, double>> MapRanking { get; set; } // output ranking to this property
}

using (var session = Store.LightweightSession())
{
    session.Query<Document>().Where(x => x.PlainTextSearch("Some title", new SearchOptions<Document>
    {
        MapRanking = x => x.Rank, // Optionally output rank to this property on Document
        RankType = RankType.Frequency, // we wish to use ts_rank instead of ts_rank_cd
        Where = x => x >= 1.0, // Optionally only return results with ranking over or equal to 1.0
        OrderBy  = RankOrder.Ascending // Optionally order by the rank property if you can't output it to a property.
    }));
}

MapRanking only exists because there may be instances where you'd want to display it and instances where you want to and wish to order by the rank, then you could use OrderBy.
More info about ts_rank here

@ColonelBundy that looks good to me.
Would you be willing to provide PR for that? I can try to provide some guidance.


Remove error message on MS Excel 2010

Excel says the file is corrupted if the XML contains empty xml nodes.
Those modifications try to no emit xml nodes if they are empty.

@rds13 Hi, I see this is very old, but the issue with line chart still exists, any idea how to solve that?
I tried using this code, but still getting issues at MS Excel 2013, bar chart works fine though.


Add amqp operations

This patch adds all operations used in the configuration progress of a Thing with the KNoT Cloud. As the patch name implies these operations were implemented using the AMQP protocol.

The commits are in a order that does not make a lot of sense.

This PR should depend on another PR?


Is YOLOStereo3D aware of the stereo baseline?

Hi, I wonder whether the stereo baseline is used in YOLOStereo3D.
If I want to use YOLOStereo3D trained on KITTI to perform object detection in another scenario where the stereo baseline is different from KITTI, will it work? Which part should I modify?
Thanks a lot!

Great question. We do not have the stereo baseline explicitly used in inference. So the network is kind of overfitting to the camera setup of the dataset.
Some other users have reported that YOLOStereo3D does not generalize well when directly moving on to a new personalized camera setup even at 2D detection (but we are not sure about the stereo calibration accuracy of their setups).
From my personal LIMITED experience, I will first check if the 2D detection results transfer well (if it fails here then it is hard to transfer); if so, then I will try to compute a mapping between the predicted depth from the trained network to gt depth at the new scene (like calibrating the scale).


Add queue spilling mechanism to developer documents

Description
Explain how the new structure used to buffer queue messages works in MB.

@Shazni,
Thank you for the suggestion. We are planning to implement a global limit
initially. It will be a bit challenging to implement a queue wise limit. I
mean the configuration part of it. Two options I see is that we can define
a broker argument to be used when declaring queue using AMQP, or we can
define it in the main configuration file and lookup when creating queues.
We will consider this in later iterations. This is tracked with
https://github.com/wso2/message-broker/issues/241.
@Waruna
Currently the limit is based on message count. We will consuder this in a
later iteration. This is tracked with
https://github.com/wso2/message-broker/issues/242.
On Sat, Feb 10, 2018 at 12:09 AM, Waruna Jayaweera<EMAIL_ADDRESS>wrote:

Hi,
On Tue, Feb 6, 2018 at 11:34 AM, Hasitha Hiranya<EMAIL_ADDRESS>wrote:

Hi Asanka,
+1 for the idea.
Better if we can define the queue depth queue wise. If depth is
exceeded,  we have two options

drop the message (Qpid in-memory queues did this)
spill data out.

If we think about message is a bean, it will have fetch metadata and
fetch content methods. For inmemory messages those two methods will get
data from memory (inside that message object itself). For spilled out ones
we keep a id, so when loading to memory when queue moves, same method will
be called and DB is queried. But yeah, there is a performance issue if we
load message my message back to memory as we query DB message by message.

In that case we can load message data back to memory as batches,  so
number of DB calls will be less.
@Asanka is this buffer limit  is based on message count or total messages
size ? IMO total messages size of queue would be better limitation.

Thanks
On Tue, Feb 6, 2018 at 10:11 AM, Asanka Abeyweera<EMAIL_ADDRESS>wrote:

Hi all,
Problem
Currently, all the messages for queues are kept in memory without a
limit. If we accumulate too many messages, the broker can go out of memory
which we need to avoid. Therefore as a solution, we need to keep only a
portion of the messages when a certain limit exceeds for a queue. That is,
there will be two modes of operation.

Default mode - Messages are parallely persisted and put in to a in
memory queue. The message delivery is done directly from messages published
to in-memory queue storage
Spill mode - Published messages are directly persisted to database
without putting in to in memory queue. Separately, messages are read from
the database for delivery.

But this change will introduce following issues.

Since we are keeping all the messages in memory, the reference
counting was easy. But with this change, there can be a situation where we
do not have all copies of the same message in memory and we will have to go
for the database to see if there are any message references, which is not
very efficient.
Since we now have two modes of operation and switching between the
two modes need to be handled carefully. Otherwise, the message order can be
messed up.

Proposed Solution
I am planning to keep the message-queue reference information (queue
attachment info) in memory along with the message ordering for all the
messages for a queue. This will not add much memory footprint since we are
only keeping the message ID and the queue attachment info.

In each queue buffer, we will put message with message data until we
reach the buffer limit. Then only the message ID and queue attachement
information is stored in the queue buffer to avoid filling memory.
When one of the messages in the queue buffer is acknowledged we, will
proceed the buffer limit cursor and send message data read requests to fill
the message data.
When the message data is read from database and filled, the
deliverable limit cursor will be progressed to make that message
deliverable.

When we start a node only the message id and queue attachment
information is loaded. The handling logic will then asynchronously load
message data to memory.
WDYT?

Asanka Abeyweera
Associate Technical Lead
WSO2 Inc.
Phone: +94 712228648 <+94%2071%20222%208648>
Blog: a5anka.github.io
https://wso2.com/signature

--
Hasitha Abeykoon
Associate Technical Lead; WSO2, Inc.; http://wso2.com
cell: +94 719363063
*blog: *abeykoon.blogspot.com http://abeykoon.blogspot.com

Architecture mailing list
<EMAIL_ADDRESS>https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture

--
Regards,
Waruna Lakshitha Jayaweera
Senior Software Engineer
WSO2 Inc; http://wso2.com
phone: +94713255198 <+94%2071%20325%205198>
http://waruapz.blogspot.com/

--
Asanka Abeyweera
Associate Technical Lead
WSO2 Inc.
Phone: +94 712228648
Blog: a5anka.github.io
https://wso2.com/signature


Redis discovery

Description:
Adds discovery mode tests and configs for redis receiver
Link to Splunk idea:
No idea link. Internal contribution
Testing:

Added new integration bundled test and test data
verified that existing redis integration tests work as intended

Documentation:

Come on tests! (especially the windows tests that likely wont work because they require WSL)
Tell me what's broken in our workflow that works on my local!

argle new tests to investigate :D

Mind updating the readme https://github.com/signalfx/splunk-otel-collector/blob/main/internal/confmapprovider/discovery/README.md#discovery-mode?

Mind updating the readme https://github.com/signalfx/splunk-otel-collector/blob/main/internal/confmapprovider/discovery/README.md#discovery-mode?

can do!


[CI] Label commenter workflow added

Signed-off-by: Pulkit Singh<EMAIL_ADDRESS>Description
This PR Fixes #33
Notes for Reviewers
Only added configuration for issue/remind.
Pls, suggest if we need to include any other configs as well!
Signed commits

[x] Yes, I signed my commits.

@PulkitSinghDev: DCO is failing, else looks good! : )

@PulkitSinghDev Thanks for this PR! Hopefully, @amino19 will point out how to fix the DCO issue. You can execute these commands - https://github.com/layer5io/wasm-filters/pull/35/checks?check_run_id=3620870893

@leecalcote The cloned repository from my local system and even have deleted the forked repository
so you all can close this pr, I will create a new one

if you're familiar with gh pr checkout you could work of this branch itself, you'd just have to install the github cli.

I tried @warunicorn19 , but it's not gonna work as I deleted the cloned as well as forked repo

🤔 well that's strange, it should still work, but okay a new PR would do.

@PulkitSinghDev: Okay, no worries!
Pl feel free to shoot another pull request, also remember to make a branch of your own with git command:
git checkout -b <branch-name>

Thanks! :slightly_smiling_face:


scripts: more metrics for lock manager (#1090)

Signed-off-by: youjiali1995<EMAIL_ADDRESS>

LGTM


Fixing WinUI 2.6 store certification error

I had previously thought that the WinMetadata version of Microsoft.UI.Xaml.winmd was necessary for .NET Native compilation when that WinMD's implementation comes from a framework package, but it looks like it isn't.  Since its presence is causing store certification to fail, this change removes it from that location, leaving only the WinMD in the root.
I've tested building and uploading a package with this change to the store as well as building and running the app locally, and all seems well - the store certification passes, and the app runs fine.
Fixes #5298

/azp run

The 2.6 release passed the WACK tests that we run as part of validation. Do you know why they did not catch this issue? Is there a more recent version of WACK that we need to use? Or is this issue something separate from WACK?

This issue is separate from WACK - .NET Native compilation was failing because it detected that the two WinMD files weren't the same (this may be a bug).  I don't know why this only repro'd in the store certification process.


Adapt response handling for /v3/service_credential_bindings/:guid/parameters

This is a fix as part of https://github.com/cloudfoundry/cloud_controller_ng/issues/2637. It contains changes in behaviour for:
GET /v3/service_credential_bindings/:guid/parameters
For GET /v3/service_credential_bindings/:guid/parameters, if the state is not create succeeded, it should not return parameters and it should raise an appropriate error.
When the state is not create succeeded for service credential key bindings (type=create and state=create in service_key_operations table), GET /v3/service_credential_bindings/:guid/parameters will return an appropriate error with information why parameters are not returned, e.g. {"errors":[{"detail":"Creation of service key initial","title":"CF-ResourceNotFound","code":10010}]}

Links to any other associated PRs
#3443

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[ ] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

@kathap - Changes look good! Could you please squash your commits, then I will approve and merge the PR.


Check for already registered names

Hi,
is there any progress regarding the check for names that are already registered?
We had that problem a while ago.
I would like to offer a pull request, but @andres-erbsen is the only one who can create new invites, so how far is your diff? :)

Confirming that an extra invite does get used up when trying to register a taken name:
# fresh invites used to reproduce here: xvebJTMqn7ono8SgoKPzag== NZjhT8ITl09pSLc1Q0Px2g==, D6IxGOvT+locehVgVR26cQ==
$ dnmgr init gQPjuIl3574iTW2BwyqnIA1 xvebJTMqn7ono8SgoKPzag==
$ mv ~/.config/dename/gQPjuIl3574iTW2BwyqnIA1{,.bak}
$ dnmgr init gQPjuIl3574iTW2BwyqnIA1 NZjhT8ITl09pSLc1Q0Px2g==
registration failed: not authorized
$ dnmgr init gQPjuIl3574iTW2BwyqnIA2 NZjhT8ITl09pSLc1Q0Px2g==
registration failed: invite already used

After bb79881dfbd81ebd43c8197518d0e5974a94c501, a proposed fix:
$ mv ~/.config/dename/gQPjuIl3574iTW2BwyqnIA1{,.bak2}
$ dnmgr init gQPjuIl3574iTW2BwyqnIA1 D6IxGOvT+locehVgVR26cQ==
registration failed: name already used

I don't actually remember the issue we had with this earlier, so I am not closing this bug now. Please close it if the issue I fixed actually is the one you reported. And, if you ever need invites for testing or give-away purposes, just shoot me an email. I finally have some time to work on this again.

I thing that should do the trick. It's just to inform the user why the registration of a new name failed.
So, this diff is okay with me and I will close this issue.


How can I make my project adaptable to other languages?

This project is excellent, and it would be great if it could be adapted to other languages. If i want to modify the project yourself to make it compatible with other languages, what are the main modifications i need to make?

Thanks! The code is parsed to "CodeBlocks" using Treesitter, so the logic and prompts are agnostic to the programming language being used. What's needed is to add a new parser for the language and a Treesitter query file with definitions.
I've started on a Java parser which uses these treesitter queries.


Bodsog2

Added fire weapons and fire armor set.

Did not pull from main first, please update branch and try again


Ibax  selam

Clear and concise description of the problem
Ibax
Suggested solution
Ibax
Alternative
No response
Additional context
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Ne zaman borsada olacaj

1 şubatı bekleyelim bakalım


[TW#15300] Station Mode Connection Error (bcn_timout,ap_probe_send_start)

Hi,
I have a simple application to connect ESP32 to a Wifi Network.
After it's connected successfully to the network, this log is kept coming continuously  I (50647) wifi: bcn_timout,ap_probe_send_start and finally it's getting disconnected from the Wifi Network in few minutes. I am facing this issue with the recent update of the esp-idf.
Does anyone have any idea what is causing this issue? Thanks.
Log:
I (785) wifi: Init dynamic tx buffer num: 32
I (785) wifi: Init data frame dynamic rx buffer num: 32
I (790) wifi: Init management frame dynamic rx buffer num: 32
I (796) wifi: wifi driver task: 3ffbd268, prio:23, stack:4096
I (801) wifi: Init static rx buffer num: 10
I (805) wifi: Init dynamic rx buffer num: 32
I (809) wifi: Init rx ampdu len mblock:7
I (812) wifi: Init lldesc rx ampdu entry mblock:4
I (817) wifi: wifi power manager task: 0x3ffc26e0 prio: 21 stack: 2560
I (844) phy: phy_version: 359.0, e79c19d, Aug 31 2017, 17:06:07, 0, 0
I (845) wifi: mode : sta (24:0a:c4:0c:ba:28)
I (846) WIFI: Event Ocured: 2
I (970) wifi: n:10 2, o:1 0, ap:255 255, sta:10 2, prof:10
I (1627) wifi: state: init -> auth (b0)
I (1648) wifi: state: auth -> assoc (0)
I (1657) wifi: state: assoc -> run (10)
I (1706) wifi: connected with TestWifiNetwork, channel 10
I (1709) WIFI: Event Ocured: 4
I (3837) event: ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
I (3837) WIFI: Event Ocured: 7
I (4657) wifi: pm start, type:0

I (50647) wifi: bcn_timout,ap_probe_send_start
I (51148) wifi: bcn_timout,ap_probe_send_start
I (51648) wifi: bcn_timout,ap_probe_send_start
I (59966) wifi: bcn_timout,ap_probe_send_start
I (60467) wifi: bcn_timout,ap_probe_send_start
I (60968) wifi: bcn_timout,ap_probe_send_start
I (61468) wifi: bcn_timout,ap_probe_send_start
............
...........
I (201669) wifi: state: run -> auth (7c0)
I (201669) wifi: n:10 0, o:10 2, ap:255 255, sta:10 2, prof:10
I (201670) wifi: pm stop, total sleep time: 0/197012403

Same to me

anybody resolved this ?

Same to me

This has just started happening to me as well, on what I had considered good hardware/software. My OTA server had changed their root CA, so I added it to my root cert file, otherwise no changes. And I cannot do complete OTA session any more.

I did a short power cycle of my dev board and it still didn't work. Then I did a full rebuild (rm -rf build && make erase_flash flash monitor) and a long power cycle, and it seemed to work. I should note this was transient behaviour -- I didn't change my code between builds.


COMPAT/BLD: int dtype in json tests

Fixes failing windows build
https://ci.appveyor.com/project/jreback/pandas-465/build/job/p8hju341jyhnkogc

so maybe add a note in json docs that deserializstion will go to platform ints

The public json api (pd.read_json) actually coerces everything to int64, it's only the internal methods where you can get a platform int.
In [45]: s = pd.Series([1,2,3])

In [46]: pd.read_json(s.to_json(orient='records'), typ='series').dtype
Out[46]: dtype('int64')

In [47]: from pandas.json import loads

In [48]: loads(s.to_json(orient='records'), numpy=True).dtype
Out[48]: dtype('int32')

ok thanks @chris-b1 ping on green.

Current coverage is 85.26% (diff: 100%)

Merging #14100 into master will not change coverage

@@             master     #14100   diff @@
==========================================
  Files           139        139
  Lines         50502      50502
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits          43063      43063
  Misses         7439       7439
  Partials          0          0

Powered by Codecov. Last update a0151a7...a2825d9

thank you sir!


Other Colour for the OpenIdea Idea

It woud be great if the Idea of OpenIdea could get another Colour than all other Ideas.

I understand your concern is, that new users don't know where to start.
b5448ed implements a redirect to your OpenIdea idea for new users and an info button to go back to that idea at any point.
This issue isn't closed for now, id like to add a extra welcome page which explains our project more in detail and lets the user know that clicking on the info button will bring back that menu.

welcome screen is now presented to new users. As of 216123d2864e11886097b18174c185693217e5b4 it also links to an explanation video. The text could need some improvements but this issue will be closed since the main issue is resolved.


Safe reading of packages.config

When there is no packages.config, tool causes fatal error
When there is a dummy / different packages.config, valid XML but with unexpected tags something like:<project></project>

will also cause the fatal error. It could be a non-xml file, a YAML, JSON or anything.

The reading and processing of packages.config should be in try-catch to prevent from these fatalities.

Problem: Simple helloWorld console app created by VS2013, that doesn't required any nuget package. Converting such csproj is not possible with v1.3.1


Missing brackets in Spark_Dqtaframes

There are closing brackets missing in scala files in prepare_movies_dat/.scala
prepare_ratings_dat/.scala

Please could you be more specific. A code snippet would really help.

Hi, it's no big deal but a little typo
In
https://github.com/Thomas-George-T/Movies-Analytics-in-Spark-and-Scala/blob/0d30bc9e7bfb7b7fb7fed454c30c1a588ed56856/Spark_DataFrames/prepare_movies_dat/prepare_movies.scala
And
https://github.com/Thomas-George-T/Movies-Analytics-in-Spark-and-Scala/blob/0d30bc9e7bfb7b7fb7fed454c30c1a588ed56856/Spark_DataFrames/prepare_ratings_dat/prepare_ratings.scala
Where the number of simple paranthesis are miss numbered
Apologies If you feel it's filthy 😅
Thank you.
On Wed, 19 May, 2021, 12:49 pm Thomas George Thomas, <
@.***> wrote:

Please could you be more specific. A code snippet would really help.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Thomas-George-T/Movies-Analytics-in-Spark-and-Scala/issues/2#issuecomment-843817352,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANAFGZSTYGNPVH3XBA5MR4LTONQ7RANCNFSM45D5YZSA
.

Done.
Thank you for bringing it to my notice.
Appreciate it.


Feature Request: Static IP MAPPING (or best effort) for secure-DHCP

It would be nice if there was a way to guarantee a client got a specific IP address or at least have the DHCP provided in SoftEther 'try' to give each client the ip it had last (likewise it should always try to hand out the one that has not been used for the longest time (or ever at all) for clients it has never seen before.
Without looking into the code to heavily I imagine 'last used ip' as a field on the user profiles?
For literally static mapping I assume something similier could be done with a list dialogue in the DHCP Settings,  |

duplicate of https://github.com/SoftEtherVPN/SoftEtherVPN/issues/16


[DEVX-1033] Use<EMAIL_ADDRESS>

Update to<EMAIL_ADDRESS>Don't support playwright configuration file currently.

@valentyn88 Did you happen to test tunnels with this?

@valentyn88 or @FriggaHel, Did you happen to test tunnels with this?

@alexplischke I did not tried tunnel per-se. (Don't know if this is available with CTR).
I tried to run the bundle (on a VM) with a HTTP_PROXY/HTTPS_PROXY var set, and this is going through it.
IMHO this is ok, that would just require a real e2e test to confirm that, but there is no reason this won't be working.

@valentyn88 or @FriggaHel, Did you happen to test tunnels with this?

Only with changes from @FriggaHel it's possible to test the new playwright-runner and tunnel. We still have this ticket in a backlog.


Laravel 9 needs modern symfony/process

Namely ^6.0

Thank you. Let's see if it passes now 👍🏽

Well done :) Viktor! Thank you very much.


Blazor API Review: Router

tbd

We want to rename all of the render fragment parameters to drop the Content suffix.

superseded by https://github.com/aspnet/AspNetCore/issues/10493


[1.17.1 Fabric] Animations do not transition properly when second animation does not specify default values

Versions

Minecraft: 1.17.1
GeckoLib: 1.17:3.0.14

Disclaimer: I'm not actually sure if this is a bug or intended behavior pretty sure this is intended behavior, but I think a flag should be introduced to prevent it. See my comment at the end for more information.

 A causes a root node to transition up 16 units from 0.0 to 2.0 seconds:
"animation.model.a": {
			"animation_length": 2,
			"bones": {
				"bone": {
					"position": {
						"0.0": {
							"vector": [0, 0, 0]
						},
						"2.0": {
							"vector": [0, 16, 0]
						}
					}
				}
			}
		}

Animation B causes the root node to rotate (the details/timing of the rotation do not matter, the main point is that B starts at the default state of the root node, which is position [0, 0, 0]:
"animation.model.b": {
			"animation_length": 2,
			"bones": {
				"bone": {
					"rotation": {
						"0.0": {
							"vector": [0, 0, 0]
						},
						"1.0": {
							"vector": [180, 180, 0]
						},
						"2.0": {
							"vector": [0, 0, 0]
						}
					}
				}
			}
		}

Given a transition time on a controller of 20, and animations added at the same time (A -> B), I would expect the animation to play out like this: cube moves up to [0, 16, 0] over 2 seconds, transitions down to the default state of B ([0, 0, 0] position), and then plays the spin animation. What actually happens is A instantly snaps to [0, 0, 0] during the transition phase, because B does not manually specify* [0, 0, 0] as a position at frame 0.0. This causes unexpected snapping when transitioning between animations.
* Note that manually specifying the default position of [0, 0, 0] in B fixes the problem.
Video Demo for the following animated code:
 private <T extends IAnimatable> PlayState run(AnimationEvent<T> event) {
        event.getController().setAnimation(new AnimationBuilder()
                .addAnimation("animation.model.a", false) // moves up to [0, 24, 0]
                .addAnimation("animation.model.b", false)); // does not specify position, but [0, 0, 0] should be assumed

        return PlayState.CONTINUE;
}

Without Specifying [0, 0, 0] at 0.0 on B
When Specifying [0, 0, 0] at 0.0 on B

I'm wondering if this is intentional to prevent issues with layered animations - which is why I think even having an explicit flag for resetting back to default would be helpful. The issue becomes problematic when animators don't explicitly reset values in animations, and then things snap when you transition around... and finding areas you would have to "manually reset" when you have a bunch of cubes is a bit of a hassle. For the flag, maybe a boolean in AnimationBuilder#addAnimation to manually set unspecified values to 0?
The main reason I think this might not be intended is because A resets to the state of B anyways, so this problem is just a matter of whether it respects the transition time. Let me know what you think.

Thanks for reading - I would be happy to help look for a solution once I know this is not some sort of an intended feature :)

Please test on the latest 1.18.1 version, 1.17.1 is no longer being updated.

@Draylar Thank you for the detailed issue report. In your video it looks like it is transitioning back to the default state, just very quickly. Have you tried changing the AnimationManager#resetTickLength? If you set that to the same as your transition length it should work fine.

@Draylar Thank you for the detailed issue report. In your video it looks like it is transitioning back to the default state, just very quickly. Have you tried changing the AnimationManager#resetTickLength? If you set that to the same as your transition length it should work fine.

That was exactly it, specifying resetTickLength fixed the issue - thanks so much! I appreciate the help.


Expose upload URLs so that perf can stop instrumenting these URLs.

Prototype reference: https://github.com/FirebasePrivate/firebase-ios-sdk/pull/731

Ideally there should be a CHANGELOG entry for a minor version update documenting the feature that FirebasePerformance will depend upon

Ideally there should be a CHANGELOG entry for a minor version update documenting the feature that FirebasePerformance will depend upon

Thanks for reminding me on this! Please take a look at the CHANGELOG  and see if it looks good to you.


Add at method

Fixes #40695

Up.

This proposal has web combability problem so it's need to be renamed.

Well... https://github.com/tc39/proposal-item-method/issues/34

FYI, per the linked issue, item could not be used in ECMAScript due to web compatibility issues. See https://github.com/microsoft/TypeScript/issues/40695#issuecomment-737471990

FYI, per the linked issue, item could not be used in ECMAScript due to web compatibility issues. See https://github.com/microsoft/TypeScript/issues/40695#issuecomment-737471990

We discussed this in the design meeting and concluded:

at should return T | undefined, as this PR does.
The compiler should add support for negative indices on tuples, along with special support for at to work the same way. Something like

declare let t: [string, number]
let n = t.at(-1) // n: number
// and perhaps even
declare let t2: [...string, number]
let n2 = t.at(-1)

Though the second example would require adding quite a bit more code to the checker.
I expect @DanielRosenwasser will link the notes when he's done preparing them.

@rbuckton Can you take another look and merge if all your comments have been addressed?

It looks like a different PR was merged that already addressed this: #46291


Increase VS Code scroll back by default

Because the spool files keep getting cut off for new users.

@worksofliam what are you thinking?  default value of 10,000 ?
this should probably be a minimum value....incase the user already has the scrollback setting set to something larger.
For example, If we decide on 10,000 and the user has went in and made it 20,000 on their own we wouldn't want to set it back to 10,000.  I can take a look at this issue.

@rbeckham Sounds like you're on the right track!


View related Tutorial(s) link(s) with Tutorials section header if available

I WANT to be able to see available tutorials
SO that I can access that specific form of content
GIVEN I am on the learn page
WHEN I scroll down to Tutorials section
IF there are tutorials available
THEN I can see the Topical Guides header and the specific topical guides

We've redesigned the website completely and this issue does not apply anymore.


E (7416) BT: bta_gattc_cmpl_cback unknown conn_id =  3, ignore data

I am using a modified version for the BLE_client.ino example for the Arduino IDE (Ver 1.8.5). Everything works fine if I only register one notify callback. As soon as I attempt to register a second notify callback on a second BLERemoteCharacteristic object this error message appears;
"E (7416) BT: bta_gattc_cmpl_cback unknown conn_id =  3, ignore data". The main Arduino thread is killed by the error, but the first notification callback is still working (must be a different thread ?). The code that causes the error is running in the Arduino loop() function. The loop() function stops running. I have also seen this error message when using the Expressif Arduino BLE API directly but was not able to pinpoint when it happens. This error did not occur with the previous version of the Expressif SDK. Any ideas/help in resolving the issue would be greatly appreciated. If you need my code that causes the error I will post it for you or email it to you.

Sure, it would be very helpful to have your code. It will speed up debugging.


added markdown viewer of @gwaredd

author: @gwaredd
url: https://github.com/gwaredd/UnityMarkdownViewer

I think we hit something we shouldn't have:
Markdig.dll is included in both this library and UpmGitExtension
Therefore Unity shouts at me:
PrecompiledAssemblyException: Multiple precompiled assemblies with the same name Markdig.dll included for the current platform. Only one assembly with the same name is allowed per platform.
Any ideas on how to fix this problem?
@mob-sakai
@gwaredd

Hi @starikcetin
Does renaming dll file solve the error?

@mob-sakai I'll try that tomorrow, and let you guys know.

I tested it and it worked.
It will be fixed in the next release. 👍

@mob-sakai Interesting, I couldn't get it to work. For me, the names are already different.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
noelkoutlis, @supermomonga, @UgurAldanmaz, @chrjstophoros, @zointblackbriar, @cuongtranba, @kumaranimesh, @dleunji, @wertykevin91, @Anu6is, @pengowray, @FournyP, @JessicaMulein, @muhammedsaidibcl, @mhmdnzr, @Leitee, @SteveSimms, @truongx, @SireMartin, @phuonghd

No one send this individual any information.
No company would ever contact you promising 180k base salary ever. This is a scam and a half.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
gwho, @WindsLeaf, @P0oOOOo0YA, @ziljah, @FlyingSB, @devaale, @csuffyy, @jmoralesv, @ranjancse26, @Kaisen2350, @hosseinadib-hub, @diegocapadi, @AlbertJanBoer, @raxuser100, @hristochr, @SERDAR-AKIN, @jjcollinge, @wikes82, @fanyinghao, @panyang1217


RN 0.49.3 broken change

Hi there!
Upgraded to RN0.49.3 and getting node_module bugs.
in file React-native-parallax-view/parallaxView.js
I needed to add const PropTypes = require('prop-types')
const createReactClass = require('create-react-class');
Replace     React.PropTypes with PropTypes
replace React.createClass with createReactClass
Appreciate any help on this issue.
Thanks!

same problem here

Same problem, fix worked for me too.  Thanks Douglas.