[Beta] Sidebar groups

Adds grouping to sections in the sidebar. This lets us get rid of the "react:" prefix on API pages and gives us some space to add more stuff in the future to Learn.

let's give it a try.

Nice improvement 😄


fix install path in README

It appears all of the formulas are now in a folder named "Formula" but the README was not updated. This fixed the path in the example command.

Whoops. This is what happens when I recycle the README from another of our taps. Well spotted. Merged in https://github.com/Homebrew/homebrew-fuse/commit/285f90f9ce55f4ebaeff491c7b8903926a12a1b7. Thank you @mike-lerch!

:+1:


Kerberos authentication doesn't work with NuGet package 2.1.0

Can't authenticate with SQL Server 2014 using new 2.1.0. Switching back to 1.1.3 works.
Exception:
Cannot authenticate using Kerberos. Ensure Kerberos has been initialized on the client with 'kinit' and a Service Principal Name has been registered for the SQL Server to allow Kerberos authentication.
ErrorCode=InternalError, Exception=Interop+NetSecurityNative+GssApiException: GSSAPI operation failed with error - Unspecified GSS failure.  Minor code may provide more information (Server not found in Kerberos database).
   at System.Net.Security.NegotiateStreamPal.GssInitSecurityContext(SafeGssContextHandle& context, SafeGssCredHandle credential, Boolean isNtlm, SafeGssNameHandle targetName, GssFlags inFlags, Byte[] buffer, Byte[]& outputBuffer, UInt32& outFlags, Int32& isNtlmUsed)
   at System.Net.Security.NegotiateStreamPal.EstablishSecurityContext(SafeFreeNegoCredentials credential, SafeDeleteContext& context, String targetName, ContextFlagsPal inFlags, SecurityBuffer inputBuffer, SecurityBuffer outputBuffer, ContextFlagsPal& outFlags)
   at Microsoft.Data.SqlClient.SNI.SNIProxy.GenSspiClientContext(SspiClientContextStatus sspiClientContextStatus, Byte[] receivedBuff, Byte[]& sendBuff, Byte[] serverName)
   at Microsoft.Data.SqlClient.SNI.TdsParserStateObjectManaged.GenerateSspiClientContext(Byte[] receivedBuff, UInt32 receivedLength, Byte[]& sendBuff, UInt32& sendLength, Byte[] _sniSpnBuffer)
   at Microsoft.Data.SqlClient.TdsParser.SNISSPIData(Byte[] receivedBuff, UInt32 receivedLength, Byte[]& sendBuff, UInt32& sendLength)

Image configuration:
FROM mcr.microsoft.com/dotnet/aspnet:5.0.0-buster-slim AS base
RUN sed -i 's/DEFAULT@SECLEVEL=2/DEFAULT@SECLEVEL=1/g' /etc/ssl/openssl.cnf && \
	sed -i 's/MinProtocol = TLSv1.2/MinProtocol = TLSv1/g' /etc/ssl/openssl.cnf && \
	sed -i 's/DEFAULT@SECLEVEL=2/DEFAULT@SECLEVEL=1/g' /usr/lib/ssl/openssl.cnf && \
	sed -i 's/MinProtocol = TLSv1.2/MinProtocol = TLSv1/g' /usr/lib/ssl/openssl.cnf

Hi @mashbrno ,
Thank you for reporting this. We will look into this soon.
Have you tried with other SqlClient versions such as 2.0.0 and 2.0.1?
Are they working for you?

Hi @karinazhou
yes, I'm using TLSv1.0. I've prepared repro sample https://gitlab.com/jan.zahradnik/sqlclient-sample
I've tested with 2.0.0 and 2.0.1 packages and it works. So it's regression in 2.1.0 package.

Same, regression in 2.1.0

+1, faced with that

@mashbrno  Thank you for the sample app.
@galakt  Thank you for the result table.
We will dig into this and update it later.

@mashbrno   I tried with your sample app on my side but I got the Server not found or not accessible exception with SqlClient 2.0.1. I look into the launch.sh and execute the first line. It looks like the key tab file is missing in my docker container. Did I miss some configuration?
root@1f04b264eefd:/etc# user=`klist -k service.keytab | grep 1 | sed -n -e 's/^.* //p'`
klist: Key table file 'service.keytab' not found while starting keytab scan

@karinazhou Of course I didn't share my Active Directory neither whole SQL server with it. Feel free to generate your valid credentials with this commands:
printf "%b"   "addent -password -p<EMAIL_ADDRESS>-k 1 -e   aes256-cts-hmac-sha1-96\npassword\nwrite_kt service.keytab" |   ktutil

You will also need to modify krb5.conf to match your setup.

@mashbrno   When I go through the changes made in MDS v2.1.0, I find that we have changed the behavior for SPN generation. You can find the commit in PR#629. This is to add default port 1433 to the server SPN when the port number is not provided in the connection string. The original issue is reported in #627 .
Are you using another port instead of 1433? And how it works if you provide a port number in the connection string?
Thanks,

Unfortunately changing connection string from
Server=dike4-ag.ad.data3s.com,1433;Integrated Security=true;Initial Catalog=DATA3S_CZ

to
Server=tcp:dike4-ag.ad.data3s.com,1433;Integrated Security=true;Initial Catalog=DATA3S_CZ

didn't make any change. Works with 2.0.1, fails with 2.1.0.

@karinazhou to izolate the issue even more it was introduced between preview1 and preview2. Your PR#629 is before that.
My guess is src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlSecurityUtility.cs from #699 where aes256-cts was removed. This is exactly the cipher I use in my service.keytab.

@mashbrno   Thank you so much for the information and it does help a lot to narrow down the issue 👍
And I can reproduce this in my testing Kerberos now which gives me the exact authentication error you got.
Let me add the code back and give it a try. I will keep you updated.

I have tested #699 PR and it doesn't break the Kerberos behavior. After some tries between preview1 and preview2, it turns out to be #665  introducing the issue. It was a big commit that involved over 200 files being touched so it will take some time to narrow down the root cause.

@mashbrno  Could you try this nuget package to see if it fix your issue? I test it locally and it works for me.
Nuget.zip

@karinazhou I can confirm, that your patch is working.

Closing issue as v2.1.1 has been released with patch.


feat(MatTooltip): Mimic native tooltip behaviour

Feature Description
If nested elements have MatTooltips applied on them, show the innermost tooltip.
Suppose we have a button with a span inside it.
<button mat-raised-button matTooltip="Button tooltip">
  Mat Tooltip
</button>

<button mat-raised-button title="Button tooltip">
  Native Tooltip
</button>

If both of these elements have the title attribute applied to them and we hover the span, the browser shows title text of the span.
However, if we use the MatTooltip directive, both tooltips are shown to the user.
Minimal Reproduction on StackBlitz
Use Case
Whenever you need to show different tooltips for parent and child elements.

It should be doable by injecting parent instances of MatTooltip and going up the tree to close all of them when a new one is opened. That being said, it would probably have to be configurable since somebody might want tooltips to trigger as they do now.

Yes, you're right. This should be configurable. I had a different idea on how we could implement this feature. We can use @ContentChild(MatTooltip) to see if there are any child MatTooltips. If yes, then don't show the tooltip. If no, then show the tooltip.

How can it be considered a feature? Showing several tooltips at once doesn't make any sense, taking into account these tooltips don't know anything about each other and may intersect. Foy my application it's a bug preventing from using matTooltip at all.


[PE-D][Tester C] UG Bug: Unincluded Documentation of feature floating on command line input

Bug Description
The interface of the input box seems to show a list of commands that are supported, which I think is a really good and cool implementation!
However, there is no mention of it in the documentation that explains it use, which I assume is to list supported commands to aid the user.
However, this might confuse the reader.
Steps to Reproduce
No steps needed.
Expected
Some included information regarding this particular feature in the UG.
Actual
No mention of it.
Screenshot

Suggestions for Improvement
Add it into the UG. Personally think it is a really cool feature! :+1 Can't add it to my own since feature freeze :(

Labels: type.DocumentationBug severity.Medium
original: gerteck/ped#4

Done


Log when Jobs toggle device functions

For debugging purposes, add a log for when the automatic jobs may toggle the device.

Being worked on in log-job-toggles

Add a nice UI for log
CardView
Cards are grouped by major event: screen ON and screen OFF
selecting a card expands it, and shows all events from the last major event up to the next major event exclusive.
Color code based on log level:
D - normal
I - blue
W - orange/yellow
E - red
Clicking the actual entry shows date/time


Support cluster level "related" resources as well as resources from other namespaces

Merged to bring up to date with master and apply cleanly
Fixes #345 (and others)

The last failing test, I'm not sure it should pass.
Anyway this allows cluster resources to be used as additional related resources. Uses * namespace as a wildcard (user is expected to understand the implications and do any manual filtering)

hi @splitice , thank for PR, however introducing support for cluster level resources requires introducing an API versioning scheme, which is not done yet.

I'm interested in this one too, as I also have a use case for cluster scoped relatedResources.
In my case I have namespace scoped parents and children, but include info from a related cluster scoped resource

Closing, https://github.com/metacontroller/metacontroller/issues/496 need to be implemented first, later on work on https://github.com/metacontroller/metacontroller/issues/345


Support uploads for Google Fusion Tables importTable, importRows and replaceRows methods

The Google Fusion Tables API describes the methods importTable, importRows and replaceRows  with two different endpoints. One endpoint is to change metadata only, the other can be used to upload a file (e.g. a CSV) to import a table or update data in a table.
E.g. for importTable:

Upload URI, for media upload requests:
POST https://www.googleapis.com/upload/fusiontables/v2/tables/import
Metadata URI, for metadata-only requests:
POST https://www.googleapis.com/fusiontables/v2/tables/import

Currently this library only supports the "Metadata URI" and can therefore not be used to upload a CSV to Google Fusion Tables.
Please consider to add support for uploads for these methods, as this is their intended use. Without this, you force users to change data using SQL statements, which are bound to rate limits or write HTTP calls to the Fusion Tables API directly without using this library.

Blocked by #44

@metaodi, sorry for the late response here. We should already have support for this, the key is to provide the data, mimeType, and uploadType options.
$service = new Google_Service_Fusiontables($client);
$service->table->importRows(
    '{TABLE_ID}',
    [
        'data' => file_get_contents('/path/to/file.csv'),
        'mimeType' => 'application/octet-stream',
        'uploadType' => 'media'
    ]
);

Please feel free to re-open if there's anything else we can do to help!


Event dispatcher for routing

Use an event dispatcher for controllers to respond to during specific parts of a route matching and dispatching process. This removes the dependency of specifically-named methods in controllers.
It helps remove the need for the Darya\Routing\Dispatcher class, which is essentially a decorator around Router that provides before/after hooks and dependency injection from the service container.
It would be nice to see the Router capable of doing all of these things, most likely optionally (this is probably best raised as another issue).
This could be hand-written or just Symfony's.

Implemented by 5e1565af followed by bug fix commits.


[Install Bug]: Space Station 14: Invalid URL for download due to trailing slash

Panel Version
1.11.5
Wings Version
1.11.9
Service
games/spacestation_14
Modified
No, I did not modify the egg
Expected Behavior
The game server installs
Actual Behavior
We have gotten word from one of our players that the egg for our game servers currently does not install due to the url having an extra trailing slash, causing the download to fail

Steps To Reproduce

Try to install the egg

Install logs
The command hangs, sorry. Heres a pasted copy
container@pterodactyl~ Starting installation process, this could take a few minutes...
Running: curl -sSL -o server_linux.zip https://wizards.cdn.spacestation14.com//fork/wizards/version/7472686aff37b95a83d38c9801d26fd1a04a40ac/file/SS14.Server_linux-x64.zip
Archive:  server_linux.zip
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of server_linux.zip or
        server_linux.zip.zip, and cannot find server_linux.zip.ZIP, period.
chmod: cannot access 'Robust.Server': No such file or directory
-----------------------------------------
Installation completed...
-----------------------------------------
[Pterodactyl Daemon]: Checking server disk space usage, this could take a few seconds...
[Pterodactyl Daemon]: Updating process configuration files...
[Pterodactyl Daemon]: Ensuring file permissions are set correctly, this could take a few seconds...
container@pterodactyl~ Server marked as starting...
[Pterodactyl Daemon]: Pulling Docker container image, this could take a few minutes to complete...
Pulling from parkervcp/yolks
Digest: sha256:85381cc1ed34672fdc5aab3a9541916241571ee32279a5d5cacd918869cbb097
Status: Image is up to date for ghcr.io/parkervcp/yolks:dotnet_8
[Pterodactyl Daemon]: Finished pulling Docker container image
container@pelican~ dotnet --version
8.0.401
:/home/container$ ./Robust.Server
/entrypoint.sh: line 20: ./Robust.Server: No such file or directory
container@pterodactyl~ Server marked as offline...
[Pterodactyl Daemon]: ---------- Detected server process in a crashed state! ----------
[Pterodactyl Daemon]: Exit code: 127
[Pterodactyl Daemon]: Out of memory: false
[Pterodactyl Daemon]: Checking server disk space usage, this could take a few seconds...
[Pterodactyl Daemon]: Updating process configuration files...
[Pterodactyl Daemon]: Ensuring file permissions are set correctly, this could take a few seconds...

I can confirm this, This will be fixt


Incorrect database details yield a fatal error (not an exception)

I've been having major difficulties raising an error page in Symphony (from PHP) to see the new styling. One obvious thing would be to change the database connection details (host for example), which should yield a non-fatal error saying that the connection could be made. Instead I see a standard blank page with a one-line PHP error:

Fatal error: Call to undefined method MySQL::getlastError() in /Users/nick.dunn/Sites/symphony-2/symphony/lib/core/class.symphony.php on line 315

Hmm because there is no longer a getlastError method. It used to be this:
/**
 * Returns the last error that occurred while querying MySQL
 *
 * @return array
 *  An associative array with the last query, error number and
 *  error message from MySQL.
 */
public static function getLastError(){
	return current(self::$_log['error']);
}

If I revert the MySQL class to 2.2.5 then I get the correct error page telling me:

There was a problem whilst attempting to establish a database connection. Please check all connection information is correct. The following error was returned:

On further inspection, I see this could be related to the addition of the PostQueryExecution delegate. The comment usefully informs us that:

Hmm, this looks to be a mistake when merging. I'll have a look but I'm pretty sure getLastError() should still exist!

Oh right, I remember now. When a DatabaseException is thrown, the details about the $error are already available in the Exception, so the extra call to the Database class is unnecessary! Shall update (and add to docs)


cleanup vercel api folder

it seems like vercel needs to know the api folder before build finishes. The api folder cannot be generated programmatically during build. We will have to live with this inconsistency...


Request context does not flow down handler pipeline (handler registered i globalmiddleware) is lost.

Problem statement
I have registered handlers. Some in setupMiddlewares  and setupGlobalMiddleware to be called in configure_xxx.go.
I enrich the http request context inside the handler registed in setupGlobalMiddleware  (NewHttpOpentracingHandle) and intend to retrieve that context value  from a one that is registered in setupMiddlewares (NewAuthMiddlewareHandler).
Since global handlers are called first, i expect teh request context to throw through. but it is nil.
func NewHttpOpentracingHandler(next http.Handler) http.Handler {
	return &httpOpentracingHandler{next: next}
}

func (h *httpOpentracingHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
	rw := tracing.NewHTTPResponseWriter(w)
	ctx := r.Context()
	*********
	// Send the context with the span info downward
        ctx = context.WithValue(ctx, "Username", "hrhrwhrheqrhreqhrh")
	r = r.WithContext(opentracing.ContextWithSpan(r.Context(), span))
    fmt.Printf("request context:; %#v\n", r.Context())
	// serve the real operation.
	h.next.ServeHTTP(rw, r)
	h.next.ServeHTTP(rw, r)

func NewAuthMiddlewareHandler(client APIClient) func(http.Handler) http.Handler {
	return func(next http.Handler) http.Handler {
		return &authMiddlewareHandler{
		******
			next:       next,
		}
	}
}

func (h *authMiddlewareHandler) ServeHTTP(w http.ResponseWriter, req *http.Request) {
    ctx := req.Context()
    fmt.Printf("this omehow returns nil: %#v\n", ctx.Value("Username"))
    operationName := tracing.OperationIDFrom(ctx)
    ***********

func setupMiddlewares(handler http.Handler) http.Handler {
	return logging.NewRequestLoggerHandler(logging.Global(), // create the request logger and add it to context
		logging.NewPanicRequestHandler( // request panic handler logs using request logger
					NewAuthHandler((handler))))))
}

func setupGlobalMiddleware(handler http.Handler) http.Handler {
	return logging.NewPanicHandler( // root panic handler, logs using global logger
					opentracing.NewHttpOpentracingHandler( // opentrace all request
						handler)))))
}

Please remove the sections that don't apply
Swagger specification
Steps to reproduce
Environment
swagger version: 0.16.0"
revision = "dd867fd63c30269ac217004c102f47a1774d4f5a"
go version:  go1.10.2 darwin/amd64
OS:   darwin/amd64

there is bug in my code so closing


Hints on migrating to v2.0.0

Very good job on the v2.0.0 release. It looks much nicer to work with!
We would like to migrate our repository from the v1.xx to the v2.0.0-testing branch. I have been looking at the sample code, but I have some questions and assumptions that I would like to confirm:

In the readme: What does it mean Most dependencies are use-able without any change., what is the criteria for them to be usable?
Is there a reason the v2.0.0 crypto sample code uses the legacy build mode? Would there be any reason for us to use the legacy build mode?
Is there something hidden that might prevent us from migrating, or that we need to consider?

Thank you very much for your answers!

Interested in this as well, it would be great if you could publish a migration guide like document.

@volcano0dr any comment on this would be appreciated


Move favicons into common/app

Close #942

I knew it would be a piece of cake for you. Thank you :-)


Skip preflight check

Description
Currently the following errors appear while compiling:
><EMAIL_ADDRESS>build /home/galr/IdeaProjects/pollen-wallet
> craco build

There might be a problem with the project dependency tree.
It is likely not a bug in Create React App, but something you need to fix locally.

The react-scripts package provided by Create React App requires a dependency:

  "babel-jest": "^24.9.0"

Don't try to install it manually: your package manager does it automatically.
However, a different version of babel-jest was detected higher up in the tree:

  /home/galr/node_modules/babel-jest (version: 26.6.3)

Manually installing incompatible versions is known to cause hard-to-debug issues.

If you would prefer to ignore this check, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
That will permanently disable this message but you might encounter other issues.

To fix the dependency tree, try following the steps below in the exact order:

  1. Delete package-lock.json (not package.json!) and/or yarn.lock in your project folder.
  2. Delete node_modules in your project folder.
  3. Remove "babel-jest" from dependencies and/or devDependencies in the package.json file in your project folder.
  4. Run npm install or yarn, depending on the package manager you use.

In most cases, this should be enough to fix the problem.
If this has not helped, there are a few other things you can try:

  5. If you used npm, install yarn (http://yarnpkg.com/) and repeat the above steps with it instead.
     This may help because npm has known issues with package hoisting which may get resolved in future versions.

  6. Check if /home/galr/node_modules/babel-jest is outside your project directory.
     For example, you might have accidentally installed something in your home folder.

  7. Try running npm ls babel-jest in your project folder.
     This will tell you which other package (apart from the expected react-scripts) installed babel-jest.

If nothing else helps, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
That would permanently disable this preflight check in case you want to proceed anyway.

P.S. We know this message is long but please read the steps above :-) We hope you find them helpful!

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `craco build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/galr/.npm/_logs/2021-05-10T14_28_55_512Z-debug.log

Since we don't use babel-jest directly, we can currently skip preflight checks and solve those problems.

I followed these steps and I dont get these errors in my machine:
To fix the dependency tree, try following the steps below in the exact order:
  1. Delete package-lock.json (not package.json!) and/or yarn.lock in your project folder.
  2. Delete node_modules in your project folder.
  3. Remove "babel-jest" from dependencies and/or devDependencies in the package.json file in your project folder.
  4. Run npm install or yarn, depending on the package manager you use.

without 3. because we dont have babel-jest as a dependency
I guess you also tired it but just wanted to make sure before merging the new env variable 🙏🏼


Add setting to adjust vibration strength

Useful for setting values lower than is allowed in in-built Odin settings. Re-applies on reboot.

Thank you for the opportunity!


Provide Package with just parsing (without HTTP capabilities)

As an Android application developer, I would like a JSoup package with just the parsing capabilities, so that I don't include unnecessary dependencies in my application.
In my case, HTTP fetching is already tightly controlled by OKHttp, so adding in another library that has the ability to create HTTP connections 1) adds to developer confusion and 2) adds a lot of code into the application that won't be used (hopefully R8/Proguard will trim it, but I did notice a pretty big jump in binary size after including this package).

I took a look at what practical difference this would make to the JAR size:

Filename
Size (KB)

Connection.class
3.4

Connection$Request.class
1.9

Connection$Method.class
1.5

Connection$Base.class
1.5

Connection$Response.class
1.1

HttpConnection$Response.class
23.6

HttpConnection$Request.class
10.3

HttpConnection$Base.class
8.3

Misc
2

Total HTTP files (unpacked)
53.6

Total all (unpacked)
860.6

Relative size
6.23%

Compressed total
453

Ratio
1.900

HTTP (packed) size
28.214

Mooted package size
425

Reduction
6.23%

A reduction of 6% of the compressed size of the jar doesn't strike me as worth the complexity of publishing another version of the library. It is a perfect use case for using proguard to strip the files that you don't need for your own specific use case. There are probably other files you're not using which you could also remove for a larger saving.
The actual total size is probably smaller again, as when deployed on Android the dex re-packaging of the jar further minimizes the deployed size.
Also note that the HttpConnection code in jsoup is implemented as a wrapper over the platform's HTTP code, with the goal of providing an API that is more ergonomic to use within the context of jsoup. The fetching and parsing of web content is core functionality for jsoup, hence why I don't consider them seperate packages.


wrong references and depricated blocks

Morning Tom,
My name is Suman Srivastava, I was just running the codes with the new terraform azrerm providers =3.0.0 and found the below two blocks are not working.

azurerm_resource_group is referenced with rg and been called as k8s in subsequent resource blocks.
addon_profile is no more valid block and this needs to be corrected.
network_profile is pointing to SKU="Standard" whch is not valid

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

ID: 1a750d14-2f87-914e-8715-fac805c8ba3b
Version Independent ID: f8d16dbe-e594-408c-18d8-26fc95de6c65
Content: Create a Kubernetes cluster with Azure Kubernetes Service (AKS) using Terraform
Content Source: articles/terraform/create-k8s-cluster-with-tf-and-aks.md
Service: terraform
GitHub Login: @TomArcherMsft
Microsoft Alias: tarcher

@tyadon The code has been completely updated and tested. I'm awaiting engineering-team sign-off, and then the article will be republished. Thanks again for your contribution!

The updated article is now live.


[MM-29501] Migrate changeCSS() to CSS variable in utils/utils.jsx, ln 621 (#15888)

Summary
Remove the line as it is already taken into account unconditionally at line 623 thanks to '.app__body .post .Menu .MenuItem' selector
When this was added in commit https://github.com/mattermost/mattermost-webapp/commit/c8ae25a2816656a7791599de765b1e0a84d9c886 cited selector wasn't present
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/15888
JIRA ticket: https://mattermost.atlassian.net/browse/MM-29501

Hello @KuSh,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

This PR will be QA'd in batch when the campaigns branch is merged to master.


Debug component thrashing

Proposed Error detail page on preactjs.com (hit edit on this PR for source):
---
url: /errors/1
title: Preact Error 1: Component Thrashing
---

Error 1: Component Thrashing
This problem occurs when components are redefined every time Preact renders.
This means that the component and all of its descendants will be destroyed and recreated on every render.
Background
Virtual DOM works by building up a new description of your application's structure every time something changes. To make this work with acceptable performance, Virtual DOM libraries implement a diffing algorithm that compares this new structure to the previous one (the structure of the previous render).
The first step of this process is to go through the tree of nodes created by your JSX - called VNodes - and match them up with the previous ones. In order to do this, it must enforce a definition of what makes two equivalent. This is done by comparing the VNode's type:
<meter value="50">home
  ^-type  ^-props  ^-children
type === "meter"

<Avatar user="1" />
  ^-type  ^-props
type === Avatar

Two VNodes are considered equal when their types are the same. When a VNode's type changes from one render to the next, any components and DOM that has been created for that VNode and all of its descendants must be destroyed in order to ensure output doesn't "bleed" across renders.
The Problem
Now think about how where we define components. When a component (class or function) is defined inside of a function, each time that outer function is called we'll get a new copy of that component:
function outer() {
  const Avatar = ({ src }) =>

  return Avatar
}

outer() !== outer()  // a new Avatar function is created every time

This is the same as what the Virtual DOM library sees when looking at the JSX returned from your render functions or functional components. Take a look at what happens when we define Avatar from inside of another component's render:
function Profile(props) {
    const Avatar = ({ src }) =>

    return <Avatar src={props.src} />
}

const render1 = Profile({ src: '/1.gif' })
const render2 = Profile({ src: '/1.gif' })

render1.type !== render2.type
// They're not equal!
// This means we have to destroy and recreate the image.

This creation of new components on each render is how "Component Thrashing" comes to be.
How to Fix Component Thrashing
Let's take our Profile example from above:
function Profile(props) {
    const Avatar = ({ src }) =>    // 🚨

    return <Avatar src={props.src} />
}

We now know that Avatar is going to be created every time Profile is called by Preact.
To fix this, move the component's definition out of the outer function:
function Profile(props) {
    return <Avatar src={props.src} />
}

const Avatar = ({ src }) =>

Now, every time Profile() is called, it will return the same reference to Avatar! It's generally best to write nested components this way so they can't rely on scoped variables, as this makes them "pure".
This will correct the issue and allow Preact to diff properly once again.

Rebased, ready for re-review + merge.

Coverage decreased (-0.2%) to 99.769% when pulling c0e710fa4b6646f0cb53708a5dc608be3431ce17 on debug-component-thrashing into c696534281a1d964a4023c24b0a4f21d60d539c5 on master.

Coverage increased (+3.4%) to 99.769% when pulling abadb30730ec66fd4ff3764a2754992f6c20da88 on debug-component-thrashing into 8ccbd400474a2a7181f997135c95e25873001e6d on master.

took a stab at adding caching in WeakMap-supporting browsers.

Also fixed an issue where options._diff wasn't chained for null vnodes.


Updated Release Notes for KBM Note

Summary of the Pull Request

PR Checklist

[ ] Closes: #xxx
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

RE: Conversation here https://github.com/microsoft/PowerToys/pull/28097

Please roll this back. This was the start of the 73 notes so we didn’t forget a warning
it has tons of place holders!

Please roll this back. This was the start of the 73 notes so we didn’t forget a warning
it has tons of place holders!

Done. Sorry, merged by mistake.


Added the vm creation script and unit tests

Script to create vms on every hv in every aggregate

We should copy over / adapt one of the workflows to run pylint, black and the unit tests that we now have

Can we close this since we've moved it to the st2 pack?


Problem training with multi GPU scenario.

I have 8 GPUs.  I want to train 2 different variations of Tacotron. First variation in GPUs 1-4 and second in 5-8.

I was able to start training using first 4 GPUs by limiting the gpus in multiproc.py.
My i am not able to start train using 5-8 GPUs only. I did the following modifications in the code.

 1. I set the num_gpus = 4
 2. i assigned an array [4,5,6,7] while initializing in the multiproc.py.
 3.  Also, changed num_gpus in train.py to 4.
 4. CUDA_VISIBLE_DEVICES=4,5,6,7 python -m multiproc train.py --output_directory=outdir --log_directory=logdir --hparams=distributed_run=True,fp16_run=True

the training program starts and hangs after distributed initialization. I am not sure what more i need to change in the code. this is the logs i see..
['train.py', '--output_directory=outdir', '--log_directory=logdir', '--hparams=distributed_run=True,fp16_run=True', '--n_gpus=4', '--group_name=group_2020_04_19-003226', '--rank=4]
['train.py', '--output_directory=outdir', '--log_directory=logdir', '--hparams=distributed_run=True,fp16_run=True', '--n_gpus=4', '--group_name=group_2020_04_19-003226', '--rank=5']
['train.py', '--output_directory=outdir', '--log_directory=logdir', '--hparams=distributed_run=True,fp16_run=True', '--n_gpus=4', '--group_name=group_2020_04_19-003226', '--rank=6']
['train.py', '--output_directory=outdir', '--log_directory=logdir', '--hparams=distributed_run=True,fp16_run=True', '--n_gpus=4', '--group_name=group_2020_04_19-003226', '--rank=7']

...
...

FP16 Run: True
Dynamic Loss Scaling: True
Distributed Run: True
cuDNN Enabled: True
cuDNN Benchmark: False
Initializing Distributed
Done initializing distributed
W0419 00:32:32.122517<PHONE_NUMBER>81088 module_wrapper.py:139] From /workspace/tacotrongst/hparams.py:122: The name tf.logging.info is deprecated. Please use tf.compat.v1.logging.info instead.

I just struck here for ever.. It doesn't start training.
Please can someone help..
Thanks
Kannadaraj.

@kannadaraj
Change
https://github.com/NVIDIA/tacotron2/blob/master/hparams.py#L19
tcp://localhost:54321
to another port to initialize multiple distributed sessions.
e.g
tcp://localhost:54321 -> tcp://localhost:54322

First group should be
        dist_url="tcp://localhost:54321",

n_gpus = 4
GPU_0: rank=0
GPU_1: rank=1
GPU_2: rank=2
GPU_3: rank=3
Second group should be
        dist_url="tcp://localhost:54322",

n_gpus = 4
GPU_4: rank=0
GPU_5: rank=1
GPU_6: rank=2
GPU_7: rank=3

@CookiePPP .. Thanks a lot for the reply. I tried your solution as you said. i definitely moved to next step but throwing an error.. I searched over but couldn't find any right solution. This may be due to the setting we have done in the dist_url.. Do you have any idea why? Thanks a lot for your help
terminate called after throwing an instance of 'std::runtime_error'
  what():  NCCL error in: ../torch/lib/c10d/ProcessGroupNCCL.cpp:30, invalid argument

It seems to work for me.
I'm not sure what's wrong to cause an error like that.
Could you give some more detail of your error? (maybe a screenshot)
Also what pytorch version are you using?

@CookiePPP
Thanks a lot for trying out. I am using pytorch 1.2 and nvidia cuda container 19.10..
Here is the error

@CookiePPP  these nvidia cuda containers a finicky. I was able to make it run when i downgraded to pytorch cuda container version 19.06. I dont understand why it works in some and it doenst in others..

@kannadaraj
它似乎为我工作。
我不知道该错误是什么意思。
您能否提供更多有关错误的详细信息？（也许是屏幕截图）
您还使用的是什么pytorch版本？

hello, i want to do the same work, but when i run it like you, i got the bug, like:
NCCL error in: /pytorch/torch/lib/c10d/ProcessGroupNCCL.cpp:784, unhandled system error, NCCL version 2.7.8
i think it about rank, please tell me rank how to modify?


Writing OGG produces an empty file

I don't know if I'm doing something wrong, but this produces an OGG file with zero frames:
import soundfile as sf
import numpy as np
a = np.empty(100000)
sf.write(a, 'delme.ogg', 44100)

This doesn't seem to depend on my libsndfile binary, since the same error happens both on Debian Linux and on Windows XP 32bit.
To be sure, I compiled a little test program in C, which correctly writes a non-empty OGG file (this I tried only on Linux), so it cannot really be related to my installed version of libsndfile, or can it?
Any ideas what's wrong?
Can anybody write OGG files with PySoundFile?
Reading OGGs works perfectly, BTW ...
Writing other formats, too ...

I can confirm this on OS X.

Thanks for confirming this ... I have still no clue what's going on here ...
I originally thought it was enough to test only one format and rely on libsndfile that all others work, too.
But it looks like this isn't libsndfile's fault ... does that mean we have to test every single supported file format?
I think this would be quite tedious ...
See also #52.
Regardless of this bug, we should probably add tests for FLAC and OGG anyway, since those could mistakenly be disabled in the shared library file.
I guess a test for writing and afterwards reading FLAC is simple, since it's lossless, but I'm not sure how to test the contents of an OGG file.
But even checking only the length of the stored data would have exposed the current issue.

More experiments:
import numpy as np
import soundfile as sf
f = sf.SoundFile('delme.ogg', 'w', 44100, 1)
f.write(np.random.randn(10000))

works fine on my machine (Linux, libsndfile 1.0.25-9.1, pysoundfile HEAD).
However, if I write more than a few seconds, I get a SIGSEGV instead.
Furthermore, if I write less than 10000 frames or so, subsequent long writes segfault as well.
Yet, if I first write something between 10000 frames and a few seconds, then subsequent writes work perfectly fine, but do not write the last half second or so to the file. flush does not help with the last second.
And then all audio data vanishes from the file on close.
To be honest, it seems very unlikely that this is a problem on our end.

That's very interesting!
When I replace np.empty() with np.random.randn() in my original example, the written file is not empty anymore! But there are still a few samples missing, I was writing 100000 frames and only 77504 ended up in the OGG file. However, if I try the same thing in C, I get exactly 100000 frames in the OGG file.
The thing with the segmentation fault is also interesting ... I couldn't find a fixed size where this happens, I had a crash with 1.500.000 samples once, but another time it just worked.
This crash also happens in the C version, so this definitely doesn't have anything to do with us.
Here's the C test file ogg_problem.c:
#include <stdlib.h>
#include <sndfile.h>

static const unsigned int LENGTH = 100000;

int main() {
  float data[LENGTH];
  SNDFILE* sf;
  SF_INFO sfinfo;
  sfinfo.samplerate = 44100;
  sfinfo.channels = 1;
  sfinfo.format = SF_FORMAT_OGG | SF_FORMAT_VORBIS;
  sf = sf_open("delme.ogg", SFM_WRITE, &sfinfo);
  if (!sf)
  {
    printf("error!\n");
    exit(99);
  }
  sf_count_t written = sf_writef_float(sf, data, LENGTH);
  printf("written: %d\n", written);
  sf_close(sf);
}

And a Makefile:
LDLIBS = -lsndfile

all: ogg_problem

Have you tried libsndfile-1.0.26pre6 from here http://www.mega-nerd.com/tmp/ ? I can't reproduce the bug (empty OGG) with this pre on Win x64.

Hello,
Does the issue get fixed please ? I just downloaded the last version of libsndfile 1.0.28, but I always get the .ogg bug.
Cheers,
Louis

I believe we are currently shipping libsndfile 1.0.27 in the Windows and macOS wheels.
At what version was this supposed to be fixed?

I believe we are currently shipping libsndfile 1.0.27 in the Windows and macOS wheels.
Exactly, that's why I tested with the lastest version but with no success.
At what version was this supposed to be fixed?
No one, but since the topic is 2 years old I wanted to know if there was any news.
Anyway thanks for your lib, which is very clean to use.
Cheers,
Louis

It appears that the problem still exists today with libsndfile 1.0.27.

Unfortunately yes, and still with libsndfile 1.0.28.

Still broken apparently.

No ideas to fix this?

This still seems to be broken.  I haven't experienced seg faults, but when I write ogg files, the ends are very often cut off, which makes it sort of unusable.  We care considering switching back to wav, but that would be unfortunate since ogg really saves a lot of space.

I can confirm that this issue depends on the libsndfile version used.
I tested it by creating an ogg files with a duration of 1s using different sampling rates and number of channels. Under libsndfile 1.0.27-3 on Debian it works as expected. With libsndfile 1.0.25-10ubuntu<IP_ADDRESS> on Ubuntu I got the following strange results that show even an interaction with the number of channels:

Number of samples
Number of channels
Written number of samples in ogg file

48000
1
0

8000
1
0

48000
2
0

8000
2
0

48000
8
32320

8000
8
0

48000
255
43584

8000
255
5888

@lcouka and @bastibe: as I was not able to reproduce the error under libsndfile 1.0.27 what did you do to get the error there?

I have this same error.
For me, soundfile even crashes the Python kernel - both when running on Google colab or locally on OSX (via Spyder).
A file is written to disk before the crash, but it's empty: the channels contain no data at all.
So far it only happened with files of a certain length.

You can query soundfile.__libsndfile_version__ to get the version of soundfile you are using.
By the way, you say you are using "PySoundFile", which is severely outdated. Please install "soundfile" instead, which is the current version.

Thanks for the advice!
I am using soundfile not PySoundfile. I corrected that typo.

Any updates on this issue? Using Python 3.7 with the latest version of soundfile.

Which version of libsndfile and file are you using?
As stated above for me it did work with 1.0.27, but not 1.0.25

I seem to have the same problem using libsndfile1 1.0.28-7ubuntu0.1 with soundfile.version: 0.10.3.post1.
I see no easy option for downgrading libsndfile1 to 1.0.27.

i just tested with libsndfile-1.2.0 (as packaged in Debian, and afaict, the issue is gone)

That's wonderful news, thank you for following up on this issue!


Possible change

Greetings Darryn..
Thanks for making changes so quickly.  May I make a suggestion.  Would it be possible to change your code on line 86 of IntentShim.java
File uriAsFile = new File(Environment.getExternalStorageDirectory(), fileName);
to
File uriAsFile = new File(uriAsString.replace("file://", "" ));
That way existing apk install intents would not have to change the file location.  Apparently this will create a valid file from an existing file:///path.
Thank you for your consideration.

I don't think that would work, I may be wrong but the file provider will not accept file:// as the external storage directory even though the java file is created... I think it throws an exception when you create the provider

Hi darryn..
the uriAsString.replace("file://", "" ) removes the file:// from the beginning.. leaving /path...
I tested it and it works..  I am new to java so it took me all day to figure it out..  your solution was a good one but I am using cordova paths to download apk's so it would mean changing my locations
Thanks..

Ah I see! Yes that makes sense. I'll take a look at some point, I'd be concerned that '/' would not always be the correct directory (2 SD cards, adopted storage or some other corner case)... I'm not too sure but is worth looking into

I couldn't get it to work with the path starting with a '/' but I made it possible to specify a path here: https://github.com/darryncampbell/darryncampbell-cordova-plugin-intent/blob/master/src/android/IntentShim.java#L91 as long as you specify file:/// to start the URI the rest of the path will be taken (e.g. https://github.com/darryncampbell/plugin-intent-api-exerciser/blob/master/www/js/index.js#L121).  As long as everything's in the shared storage.  Also added some more error logging if the file is not found or inaccessible.


[Removal]:

Hotspot Name
Energetic wool beaver
Hotspot b58 Address
112CWWvpigBUrFekwrZTrq9JFHon4w2itEU2V333BSKvbFY7jF42
Discord Handle
No response
Hotspot Manufacturer
Heltec Automation
Removal Reason
Hello, it's a real deploy. Could you please remove it from denylist? Thank you in advance. Regards
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
No response

We have taken into account this removal request in the next iteration of the denylist. It may take some time before it's processed. This is an automated comment due to the volume of addition and removal requests. A future tool is in the works to provide more insight into the analysis approach made by the Helium team and community members that maintain this list.


growing block production delay on epoch switch (ep360-362)

Internal/External
External   otherwise.
Area
Other   Any other topic (Delegation, Ranking, ...).
Summary
During the last two epoch transitions (361, 362), the period without new blocks has increased significantly again.
Expected behavior
no extraordinary long delay between new blocks
System info (please complete the following information):
seen on all kind of systems and setups. relays, block producers, node version 1.35.3 and 1.34.1
Screenshots and attachments
ep359>360

ep360>361

ep361>362

Additional context
on all 3 epoch transitions one relay node ran on 1.34.1 and all other blockheight lines come from 1.35.3 nodes.
CPU graph below each epoch transition is from one relay node

You can see it on the explorer as well

by looking at v6 vs v7 blocks produced in the epochs we see
ep359 60% v7   no extraordinary delay towards ep360
ep360 88% v7   5 minutes of delay towards ep361
ep361 94% v7   12 minutes of delay towards ep362

It happens that the team are all together today.
We've had a detailed look at the logs so as to understand whether this was due to a change (eg. due to software version) or an emergent interaction between the higher cpu load that occurs at the start of the epoch and the leadership schedule (basically a "height battle between nodes that are under temporary processing strain").
Our conclusion is that, from those logs (and the other evidence) that it was the latter.

As can can be seen in the block 7728110 the system is diffusing and adopting blocks in its usual time range. The diffusion/adoption time of block 7728109 was around 40s indicative of the extra cpu cost not just of performing the epoch boundary crossing but that the diffusion of such a block was slowed (blocks are only forwarded after having being adopted)
There was more than one candidate for block 7728108 - they were created 16s apart; node logs indicate other blocks (as would be expected) that were not forwarded as they were not better "candidates" for the chain extension (by the normal resolution rules)
Yes, 12 minutes is long, but 5-6 min for the first block adopted in an epoch is not that unusual. We don't see this as a bug.
We have already scheduled a meeting to review the situation at the next epoch boundary to double check our conclusion here.
It should be noted that this is does not represent any risk to the integrity of chain growth

@njd42 If there's a follow-up meeting to discuss / review / confirm the above explanation , prolly best to keep this open and not close?

@njd42 I expanded the details in realtime view a bit for more context.  Perhaps this will be useful for your follow up meeting or for next epoch.

Its important to note that in the height battle, the CFLOW block was only reported by 1 node and actually that node was CFLOW.  So it wasn't really a height battle, the CFLOW block never made it anywhere.
The BASHO block was made at slot 674, but the first report of it was 29 seconds later with median prop time of 52 seconds.  The full story is in the prop histogram for that block:

So under the "heavy load" hypothesis, this may demonstrate when the nodes were coming online after the epoch switch and starting to process blocks again.  note that this means epoch processing takes at minimum 674+29=703 seconds with the bulk of the nodes done by 674+100=774 seconds after epoch switch.  So a little over 1 minute of variability between nodes and over 11 minutes on average.
The follow on AZUR3 block is in slot 742 but had a very interesting prop delay chart with almost all nodes reporting it at the same time at 742+40=780 seconds after epoch switch.

I guess the hypothesis here is that AZUR3 was also finishing epoch processing and produced the block late and it then got distributed quickly since everyone was ready.  Might be good to have a look at AZUR3 logs to see what time they actually produced their block.

By looking into number of established tcp connection at that ep switch time I would have expected some small decline, caused by remote p2p logic dropping and re-establishing connections.
Instead I noted on all my public relays a significant spike in tcp connections starting at 21:45utc

If of interest I can provide more detailed views

@njd42 I expanded the details in realtime view a bit for more context. Perhaps this will be useful for your follow up meeting or for next epoch.

Its important to note that in the height battle, the CFLOW block was only reported by 1 node and actually that node was CFLOW.  So it wasn't really a height battle, the CFLOW block never made it anywhere.
The BASHO block was made at slot 674, but the first report of it was 29 seconds later with median prop time of 52 seconds.  The full story is in the prop histogram for that block:

So under the "heavy load" hypothesis, this may demonstrate when the nodes were coming online after the epoch switch and starting to process blocks again.  note that this means epoch processing takes at minimum 674+29=703 seconds with the bulk of the nodes done by 674+100=774 seconds after epoch switch.  So a little over 1 minute of variability between nodes and over 11 minutes on average.
The follow on AZUR3 block is in slot 742 but had a very interesting prop delay chart with almost all nodes reporting it at the same time at 742+40=780 seconds after epoch switch.

I guess the hypothesis here is that AZUR3 was also finishing epoch processing and produced the block late and it then got distributed quickly since everyone was ready.  Might be good to have a look at AZUR3 logs to see what time they actually produced their block.

AZUR3 produced the block at 2022-09-07T21:57:13.03Z
block71021542.txt

So you produced your block on time, but for whatever reason we had a rollback and then your block was re-applied at 2022-09-07T21:57:53.15Z which corresponds exactly to when everyone else applied your block as well. The "ExceededTimeLimit" and resulting "ErrorPolicySuspendConsumer" seems to cause problems.

I agree. One of my relays even "blacked" out for a minute

When this relay was incapable of reporting metrics CPU was at max

Different view

In comparison this is 2 epochs prior on 28th of August also on 1.35.3:

In the logfiles the relay never crashed but the log is completely saturated with messages like these pretty much for the duration of the blackout
{"app":[],"at":"2022-09-07T21:59:15.26Z","data":{"address":"","event":"ErrorPolicySuspendPeer (Just (ApplicationExceptionTrace (MuxError (MuxIOException Network.Socket.recvBuf: resource vanished (Connection reset by peer)) "....
{"app":[],"at":"2022-09-07T21:58:57.97Z","data":{"kind":"PeersFetch","peers":[{"declined":"FetchDeclineChainNotPlausible","kind":"FetchDecision declined","peer":{"local":{"addr"...
{"app":[],"at":"2022-09-07T21:58:57.05Z","data":{"domain":"""","event":"Failed to start all required subscriptions","kind":"SubscriptionTrace"},"env":"1.35.3:950c4","....
{"app":[],"at":"2022-09-07T21:58:50.77Z","data":{"address":"","event":"ErrorPolicySuspendConsumer (Just (ApplicationExceptionTrace ExceededTimeLimit (Handshake) (ServerAgency TokConfirm))) 20s","kind":"ErrorPolicyTrace"},"env":"1.35...

This is still an issue today. 8m27s epoch boundary pause.

This is still an issue today. 8m27s epoch boundary pause.
I had a lot of DNSErrors in my relay log during the switch to 363. Could this be related to the long pause?
[2022-09-11 20:47:27.71 UTC] Domain: "relays-new.cardano-mainnet.iohk.io" Application Exception: <IP_ADDRESS>:3001 ExceededTimeLimit

This is a ledger task that we are working on. we've benchmarked some things https://github.com/input-output-hk/cardano-ledger/pulls?q=is%3Apr+is%3Aclosed+author%3ATimSheard and thanks to Frisby have a great new plan https://github.com/input-output-hk/cardano-ledger/pull/3141
The reason for closing this was it doesn't pertain directly to the node code base and we're trying to clean up issues at the moment.

Also see https://github.com/input-output-hk/cardano-ledger/issues/3132#issuecomment-1317364564


fix: upgrade deps and setup/format project with eslint-plugin-newrelic

Takes care of:
Issue #24
Issue #25
Issue #27

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


Do not override MultiWorkUnit grouping in Gobblin Yarn

Currently, the GobblinHelixJobLauncher flattens MultiWorkUnits and hope for Helix to do a balanced assignments of tasks to containers. However, both Gobblin Kafka and IDPC HDFS Purger already have their own logic of grouping WorkUnits into MultiWorkUnits. It's good to keep their logic and not to override it.
Signed-off-by: Yinan Li<EMAIL_ADDRESS>

@zliu41 can you review?

LGTM


Tab trapping does not work in Safari

In Safari 15.4 on macOS, the "Trapping tab focus within the modal" does not appear to function; tabbing leaves the modal to the background page even in the "Demo Modal" on the micromodal website.

Did you try alt+tab in Safari (said the demo modal to me)?


Remove legacy configuration for generate pdf

As title. need_generate_pdf is retired and we shouldn't use that.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Verify directory exist before reading logs

Just had to apply it to cmssrv217 due to
2016-03-22 05:40:26,681:140674195314432:ERROR:BossAirAPI:Unhandled exception while tracking jobs for plugin PyCondorPlugin!
[Errno 2] No such file or directory: '/storage/local/data1/cmsdataops/srv/wmagent/v1.0.14.patch2/install/wmagent/JobCreator/JobCache/vlimant_TOP-pp502Fall15wmLHEGS-00001_00002_v0__160321_160606_4883/Production/JobCollection_155_0/job_33064'
2016-03-22 05:40:26,822:140674195314432:ERROR:BaseWorkerThread:Error in worker algorithm (1):
Backtrace:
...

however, I'd rather not patch PyCondorPlugin again, so I'll refrain from applying it to the other agents.
It's still on my todo list to review these "read logs" feature.

Just fixed the JobPrio update. Of course we would not get these failures
>>> sd.edit('WMAgent_JobID =!= "UNDEFINED" && WMAgent_SubTaskName == %s && WMAgent_RequestName == %s && JobPrio != %d' %
        (classad.quote(str(task)), classad.quote(str(workflow)), classad.Literal(int(priority))), "JobPrio",
        classad.Literal(int(priority)))... ...
Traceback (most recent call last):
  File "<stdin>", line 2, in <module>
TypeError: %d format: a number is required, not ExprTree
>>>

they are going to debug logging only... :(
Will test this fix in my VM and if it works, I push it to production today.

@ticoann I think it's time to merge this patch. If we find new Pycondor issues, then we create a new PR.
Thanks


[PREVIEW] DIV-2663: Resume case from CCD when multiple cases found

This PR is the same as the changes from https://github.com/hmcts/div-case-progression-service/pull/133
It was just simpler to create a new branch rather than fix the merge conflicts
Description
https://tools.hmcts.net/jira/browse/DIV-2663
Fixes # (issue)
https://tools.hmcts.net/jira/browse/DIV-2663
Type of change

[x] New feature (non-breaking change which adds functionality)

This is tested and working fine the functionality but there are 2 other issues which are reported separately which are below.
For the above Issues below are the JIRA tickets:
Issue 1: https://tools.hmcts.net/jira/browse/DIV-3210
Issue 2: https://tools.hmcts.net/jira/browse/DIV-3284
This PR is good to go hence merged


[PWN-7497] Dismiss swap screen if it is opened via action panel

Link jira to issue
https://p2pvalidator.atlassian.net/browse/PWN-7497
Description of the changes

Need to also redirect user to home in all Swap scene

even if it is tapped via tab bar ?

https://p2pvalidator.atlassian.net/browse/PWN-7497

I don't quite sure, can you check the documentation?

https://p2pvalidator.atlassian.net/browse/PWN-7497

I don't quite sure, can you check the documentation or ask the product team?

yes you are right, we need to redirect from one tab to another. I have not known. will fix it


Backup: init backup, restore and migrate api

What type of PR is this?
/kind api-change
/kind design
/kind documentation
What this PR does / why we need it:
add backup sync api
see #374
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

add backup sync api

/kind api-change
/kind design
/kind documentation

/kind api-change
/kind documentation

/label tide/merge-method-squash

Please rebase

don'edone

/lgtm

/lgtm


data example

Having some docs or an example csv on the expected format for the data would be helpful!

Sorry for the late reply, I have a set of Youtube videos about my package with an example: I will add a link to the front page.
https://www.youtube.com/watch?v=xyMuhXxv6cc&ab_channel=AIandMLFundamentals


Add CGA errors exporter

The goal would be to write the CGA errors to a text file (along with the CGAReport.txt and CGAPrint.txt) when using one of the 3D format PRT exporters.
When working with the PyEncoder, this is going to be implemented in the issue #37.

merged.


replace node-ffi with node-ffi-napi

node-ffi has not been in active development for a while and there are growing concerns about the future of the project. Would recommend to replace the package with more active repo node-ffi-napi.
This would also solve the node 9 support issue.
https://github.com/node-ffi-napi/node-ffi-napi

Good point, will look into that. PR is welcome though. For sure it will be much faster.
Also, N-API is supported starting from Node 8 bit since it is also the minimal required version for relevant version of Angular it should he fine.

Another option is fastcall


ui: Sweep through UI text, clamping max scale factor for less-important content

When the user has set a very large system text-size setting, the layout can change pretty dramatically. For example, compare (on iOS):

No scaling
Largest setting

Apple's Human Interface Guidelines say:

Prioritize important content when responding to text-size changes. Not all content is equally important. When someone chooses a larger text size, they typically want to make the content they care about easier to read; they don’t always want to increase the size of every word on the screen. For example, when people increase text size to read the content in a tabbed window, they don’t expect the tab titles to increase in size. Similarly, in a game, people are often more interested in a character’s dialog than in transient hit-damage values.

This seems like sensible advice. Here is what's needed to complete this issue:

Set the system text size to be very large
Go through all UI areas of the app
Wherever you see less-important content where we can save space by clamping to a max scale factor, do so.

In my "largest setting" screenshot above, the "Mark all messages as read" button seems like a great candidate for this treatment. It's huge at that size. The content in the message list is important and (usually) new/unfamiliar, so it definitely deserves higher priority for readability. I  think the compose-box inputs, especially the content input, are also higher priority than the mark-as-read button. (It should be reasonably easy to know what message you'll be sending and where.)
For how to implement the clamping, start by reading these docs: MediaQuery.textScalerOf and TextScaler.scale and the scaler-clamping logic we've already implemented for emoji reactions.

Hey, Can I work on this if this is still open?

Welcome @Akshaykumar104! Be sure to look at our documentation on "your first codebase contribution" before you start. If you run into issues, post in #mobile-dev-help.

@zulipbot claim

(@zulipbot is not active in this repository; I have assigned this issue to you)


[DOC] better package description in readme - what are skbase "patterns"

I think the readme and landing page should describe more clearly what patterns skbase provides.
I would go with an abridged version of the BaseObject design document, from here
https://github.com/sktime/skbase/pull/76/files
Here's an unabridged copy-paste:

BaseObject
BaseObjects are base classes with:

scikit-learn style interface to get and set parameters
sktime style interface for working with tags

sktime style interface for cloning and re-instantiation (resetting)

sktime style interface for generating test instances
sktime style interface for retrieving fitted parameters
scikit-learn style interface for representing objects (e.g., pretty printing
and drawing a simple block representation in HTML)
Support simple composition where parameter arguments are other BaseObjects,
including the ability to get and set the parameters of component BaseObjects
Interface for instance-level override of configuration context
Interface for parameter validation.

BaseObjects should also follow certain design patterns and coding practices,
including:

Specify all parameters in the classes __init__ as explicit keyword arguments.
No args or kwargs should be used to set class parameters.
Keyword arguments should be stored in the class as attributes with the same name.
These should be documented in the parameters section of the docstring, and not
documented in the attributes section of the docstring.
All instance attributes should be created in __init__. If the attribute
is not assigned a value until later, initialize it as None.
Attributes that depend on the state of the instance's parameters should end
in an underscore to easily communicate that they are "state" dependent.
Start non-public attributes and methods with an underscore
(per standard Python conventions).
Document all public attributes that are not parameters in the class docstring's
attributes section.

(bit of a mix of features and design contract specification - I would recommend to abridge to self-contained list of features)

Agreed that this will be nice to have. I think we can get this into a usable format and included in v0.4.0 pretty easily.


Handle MediaWiki categories

e.g. [[Category:Cookbook]] (generally as the last line in a page) flagged it for inclusion in the special wiki page .../wiki/Category:Cookbook.
Can we map these to Jekyll tags in the markdown header?

Looks doable, e.g. http://www.minddust.com/post/tags-and-categories-on-github-pages/
We'd likely need something like this in the ```wiki/Category:Cookbook.md`` file,
{% include Category_Cookbook %}

which would need a matching file _includes/Category_Cookbook to generate the list of all pages tagged with this category.

Now that I stopped ignoring the category pages in the conversion script, they are converted and exist as files e.g. wiki/Category:Wiki_Documentation.mediawiki or wiki/Category:Wiki_Documentation.md - and any text on the page is of course converted. What is missing is the automatic links to all the pages tagged with that category which MediaWiki would had added to the end of the page, meaning they are not yet very useful.
However, links on other pages to the category are breaking due to the special colon syntax, which I think is best fixed from pandoc: https://github.com/jgm/pandoc/issues/2849
Failing that, we could pre-process the MediaWiki files before giving them to pandoc, or post-process the MD files from pandoc?

I've been reading http://jekyllbootstrap.com/lessons/jekyll-introduction.html and while Jekyll has categories, they are hierarchical (you can't put a page in two different categories). e.g.
---
title :  Hello World
categories : [lessons, beginner]
---

Here you would get a single nested category "lessons/beginner"
Jekyll tags look like a better match to MediaWiki categories, e.g.:
---
title :  Hello World
tags : [lessons, beginner]
---


怎么修改Indicator高度

已经解决

@huyubao  Can you help me with this, i have the same issue i want to reduce the height of strip. I prefer small circular dot instead of a line. Can you please tell me how you did it ?

Modify source code

Thanks for replying, but that's a lot of code to go through. I have tried making some changes but it's not working for me.  Can you share some code that will be a great help?

请问怎么修改指示器的高度。


changed H1 to sans-serif

Normally this would be bad because you're editing the US Web Standards so it'll be very much more difficult to pull in updates from the government. But since this is just a prototype, it's fine.
+1


Expert review: reclassify metabolic disease terms

spreadsheet
ROBOT Reviewer-Curator Agree - Stay:
sh run.sh make TEMPLATE_URL="https://docs.google.com/spreadsheets/d/e/2PACX-1vSLo_0n23zl5gVIDc1fTlVAx8MYwtkti9-7_U4BMhtPsmylHgaVc7s0c_o2Bl-9wg/pub?gid=1207648125&single=true&output=tsv" merge_template -B
ROBOT Reviewer-Curator Agree - Leave
sh run.sh make TEMPLATE_URL="https://docs.google.com/spreadsheets/d/e/2PACX-1vSLo_0n23zl5gVIDc1fTlVAx8MYwtkti9-7_U4BMhtPsmylHgaVc7s0c_o2Bl-9wg/pub?gid=435958611&single=true&output=tsv" merge_template -B
ROBOT Reviewer-Stay Curator-Leave
sh run.sh make TEMPLATE_URL="https://docs.google.com/spreadsheets/d/e/2PACX-1vSLo_0n23zl5gVIDc1fTlVAx8MYwtkti9-7_U4BMhtPsmylHgaVc7s0c_o2Bl-9wg/pub?gid=116920254&single=true&output=tsv" merge_template -B
ROBOT Reviewer-Leave Curator-Stay
sh run.sh make TEMPLATE_URL="https://docs.google.com/spreadsheets/d/e/2PACX-1vSLo_0n23zl5gVIDc1fTlVAx8MYwtkti9-7_U4BMhtPsmylHgaVc7s0c_o2Bl-9wg/pub?gid=1454236917&single=true&output=tsv" merge_template -B
@sabrinatoro I excluded superclasses for the terms in this list. The term 'myopathy caused by variation in FKRP' is inferred to be a metabolic disease (it is inferred to be a 'disorder of protein O-glycosylation'.)
Also, many of these terms were classified as metabolic diseases based on this article: An international classification of inherited metabolic disorders (ICIMD), so removing these parents is in opposition with this expert report.
address https://github.com/monarch-initiative/mondo/issues/6750

We seem to have a conflict between what the publication (which is not very old) reports and what the expert said.
@nicolevasilevsky : based on this, I think we should keep all the parents for which we have a PMID as a source (sorry, you already excluded these parents). It looks like the safest option.
What do you think?

I updated this and it's ready for your review again

ready to merge as soon as QC checks pass.


System.TypeLoadException: Method 'Build' in type 'Microsoft.Framework.Configuration.ConfigurationBuilder'

Updating to Beta 7 and assuring that all of my defendencies are also beta 7, I get the following error,

System.TypeLoadException: Method 'Build' in type 'Microsoft.Framework.Configuration.ConfigurationBuilder' from assembly
'Microsoft.Framework.Configuration, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null' does not have an implementati
on.
at Microsoft.AspNet.Hosting.Program.Main(String[] args)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Microsoft.Dnx.Runtime.Common.EntryPointExecutor.Execute(Assembly assembly, String[] args, IServiceProvider service
Provider)
at Microsoft.Dnx.ApplicationHost.Program.ExecuteMain(DefaultHost host, String applicationName, String[] args)
at Microsoft.Dnx.ApplicationHost.Program.Main(String[] args)
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Microsoft.Dnx.Runtime.Common.EntryPointExecutor.Execute(Assembly assembly, String[] args, IServiceProvider service
Provider)
at Microsoft.Dnx.Host.Bootstrapper.RunAsync(List`1 args, IRuntimeEnvironment env, FrameworkName targetFramework)
at Microsoft.Dnx.Host.RuntimeBootstrapper.ExecuteAsync(String[] args, FrameworkName targetFramework)
at Microsoft.Dnx.Host.RuntimeBootstrapper.Execute(String[] args, FrameworkName targetFramework)

I've checked the breaking changes list but can't see anything relating to this, but have read an old post suggesting that it is due to framework mismatches.
My Config file has the following,
"dependencies": {
    "Microsoft.Framework.Configuration": "1.0.0-beta7",
    "Microsoft.Framework.Configuration.Json": "1.0.0-beta7",
    "Microsoft.AspNet.Mvc": "6.0.0-beta7",
    "Microsoft.AspNet.Mvc.TagHelpers": "6.0.0-beta7",
    "Microsoft.AspNet.StaticFiles": "1.0.0-beta7",
    "Microsoft.AspNet.Tooling.Razor": "1.0.0-beta7",
    "Microsoft.Framework.Logging": "1.0.0-beta7",
    "Microsoft.Framework.Logging.Console": "1.0.0-beta7",
    "Microsoft.VisualStudio.Web.BrowserLink.Loader": "14.0.0-beta7",
    "Microsoft.AspNet.Mvc.WebApiCompatShim": "6.0.0-beta7",
    "System.Security.Cryptography.DeriveBytes": "4.0.0-beta-23224",
    "MyLibrary": "1.0.0-*",
    "Kestrel": "1.0.0-beta7",
    "dnx-clr-win-x64": "1.0.0-beta7"
},

"frameworks": {
    "dnxcore5": {
        "dependencies": {
            "System.Runtime.Serialization.Json": "4.0.0-beta-23110",
            "System.Runtime.Extensions": "4.0.11-beta-23225",
            "System.Collections": "4.0.11-beta-23225",
            "System.Runtime": "4.0.21-beta-23225",
            "Microsoft.CSharp": "4.0.1-beta-23225",
            "System.Private.Networking": "4.0.1-beta-23225",
            "System.Net.Security": "4.0.0-beta-23301"
        }
    }
},

The one for MyLibrary is
"dependencies": {
  "dnx-clr-win-x64": "1.0.0-beta7"
},

"frameworks": {
"dnxcore5": {
  "dependencies": {
    "System.Collections": "4.0.11-beta-23225",
    "System.Runtime": "4.0.21-beta-23225",
    "Microsoft.CSharp": "4.0.1-beta-23225"
  }
}
}

That's Reflector showing the actual library that is in the approot folder, which I assume it should be using but can't find the Build method that is clearly there (highlighted)

I'm not sure what issue you're running into but this reference in your project.json is bogus     "dnx-clr-win-x64": "1.0.0-beta7"

I was told that I needed that if I want to compile for x64 dnxcore.  I was using the beta 5 one just fine so assumed that as it found a beta 7 reference then that was okay.  I can see now that the dependency folder is empty so it's not actually downloaded the library.
Either way, removing that reference and compiling for x86 still results in the same error.

Can you provide a runnable sample on github? The only possible thing would be a dependency mismatch but there's too little information to tell what's going on. Look at your project.lock.json to see what actual dependency versions got resolved.

Can I invite you to a private repo perhaps?  It's not public yet.

Did you look at the project.lock.json file?

It's over 6000 lines long lol, not entirely sure what I should be looking at in there. Microsoft.Framework.Configuration is listed in there.

Is anything beta6? Is everything beta7? Do a search in the file for the versions...

okay, so i'm doing a text search for beta(n)
beta1 comes up with a few, for example
Microsoft.CodeAnalysis.Analyzers/1.1.0-beta1-20150812-01
Microsoft.CodeAnalysis.Common/1.1.0-beta1-20150812-01
Microsoft.CodeAnalysis.CSharp/1.1.0-beta1-20150812-01
...
Everything else is beta7, I think the above were added at compile time as I don't reference them myself.

I'm pretty sure that's not true. The fact is that:
"Kestrel": "1.0.0-beta7",

Does not exist and has been renamed to Microsoft.AspNet.Server.Kestrel. https://github.com/aspnet/Announcements/issues/53

oh let me try changing that to see what happens, nice it doesn't tell me that.
What do you mean by not true though?  I just searched the document, and nothing came up, it's resolving that dependency and not giving any errors, must be a bug in Visual Studio, but it was doing the same for the 64 bit libs as discovered earlier.

After changing that reference the issue persists unfortunately.

Okay so with that oddity in mind, I'm going through each package folder after publishing to see which ones are missing an actual library in the folder,
Microsoft.Bcl.Build
Microsoft.CodeAnalysis.Analyzers
The above 2 have no lib, and the following ones may be odd, not quite sure
System.Reflection.Metadata\1.1.0-alpha-00009
icrosoft.Net.Http\2.2.22

There are several things in this thread that are conflicting so I'm having a hard time figuring out exactly what you're doing.

I was told that I needed that if I want to compile for x64 dnxcore.

They probably meant run not compile with (I hope).

That's Reflector showing the actual library that is in the approot folder, which I assume it should be using but can't find the Build method that is clearly there (highlighted)

What approot folder? Did you publish?

I just searched the document, and nothing came up, it's resolving that dependency and not giving any errors

What version of Kestrel got resolved?

Well that's the ultimate goal, but the issue at current is after updating to beta 7, I got this error.
That approot that's created by visual studio, it puts an the packages with it, just published to a folder and looked inside.
-----Original Message-----
From: "David Fowler"<EMAIL_ADDRESS>Sent: 08/09/2015 17:25
To: "aspnet/Home"<EMAIL_ADDRESS>Cc: "Nick Pateman"<EMAIL_ADDRESS>Subject: Re: [Home] System.TypeLoadException: Method 'Build' in type'Microsoft.Framework.Configuration.ConfigurationBuilder' (#884)
There are several things in this thread that are conflicting so I'm having a hard time figuring out exactly what you're doing.
I was told that I needed that if I want to compile for x64 dnxcore.
They probably meant run not compile with (I hope).
That's Reflector showing the actual library that is in the approot folder, which I assume it should be using but can't find the Build method that is clearly there (highlighted)
What approot folder? Did you publish?
I just searched the document, and nothing came up, it's resolving that dependency and not giving any errors
What version of Kestrel got resolved?
—
Reply to this email directly or view it on GitHub.

Sorry I missed a few of your questions as I replied from my mobile... Let me try again,
There are several things in this thread that are conflicting so I'm having a hard time figuring out exactly what you're doing.
Compiling for DNXCore x64, as I want to run it on Linux, which displays red error text when attempting to install the coreclr as x86 stating that it must be x64.  I want to run this core project on anything I can, ideally!

They probably meant run not compile with (I hope).
Well yeah, that's just the way I have phrased it.  It needs to be referenced with the project.

What approot folder? Did you publish?
Just the one that Visual Studio created during publishing to a local folder.  That folder works just fine on Windows Server with ASP 5 installed manually, well it did prior to Beta 7.

What version of Kestrel got resolved?
Well now, beta 7, previously it said it had resolved but the published library wasn't there.  No compiler errors etc. which leads me to believe it's a bug with Visual Studio as I had no idea anything was wrong there.

If I may provide some supplemental information, I too am facing the same issue when updating my project.json file to use AspNet.MVC 6.0-beta 7.
To demonstrate the issue I did the following

I created a new default project template.
I opened project.json file, and changed the value of  "Microsoft.AspNet.Mvc": "6.0.0-beta5", to "Microsoft.AspNet.Mvc": "6.0.0-beta7",

Re-build, run and I now received the issue.  This is my entire project.json dependency section for reference. Note: I purposely did not change other items to beta7 from beta5 so I can replicate this error above.
"dependencies": {
"EntityFramework.SqlServer": "7.0.0-beta5",
"EntityFramework.Commands": "7.0.0-beta5",
"Microsoft.AspNet.Mvc": "6.0.0-beta7",
"Microsoft.AspNet.Mvc.TagHelpers": "6.0.0-beta5",
"Microsoft.AspNet.Authentication.Cookies": "1.0.0-beta5",
"Microsoft.AspNet.Authentication.Facebook": "1.0.0-beta5",
"Microsoft.AspNet.Authentication.Google": "1.0.0-beta5",
"Microsoft.AspNet.Authentication.MicrosoftAccount": "1.0.0-beta5",
"Microsoft.AspNet.Authentication.Twitter": "1.0.0-beta5",
"Microsoft.AspNet.Diagnostics": "1.0.0-beta5",
"Microsoft.AspNet.Diagnostics.Entity": "7.0.0-beta5",
"Microsoft.AspNet.Identity.EntityFramework": "3.0.0-beta5",
"Microsoft.AspNet.Server.IIS": "1.0.0-beta5",
"Microsoft.AspNet.Server.WebListener": "1.0.0-beta5",
"Microsoft.AspNet.StaticFiles": "1.0.0-beta5",
"Microsoft.AspNet.Tooling.Razor": "1.0.0-beta5",
"Microsoft.Framework.Configuration.Abstractions": "1.0.0-beta5",
"Microsoft.Framework.Configuration.Json": "1.0.0-beta5",
"Microsoft.Framework.Configuration.UserSecrets": "1.0.0-beta5",
"Microsoft.Framework.Logging": "1.0.0-beta5",
"Microsoft.Framework.Logging.Console": "1.0.0-beta5",
"Microsoft.VisualStudio.Web.BrowserLink.Loader": "14.0.0-beta5"
},

I'd recommend reading the announcements repository while doing updates during these betas. It'll significantly cut down the time you spend here :smile:. Go here https://github.com/aspnet/Announcements/issues and filter by milestone (if you're going from beta5 to beta7 you might also want to see what happened in beta6).
You also need to update your global.json to say beta7.
There's also new visual studio tooling associated with this release: http://www.microsoft.com/en-us/download/details.aspx?id=48738&fa43d42b-25b5-4a42-fe9b-1634f450f5ee=True

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


PreviousTxID as byte?

Is it possible to have tx.GetInputs()[0].PreviousTxIDAsBytes?
Thanks a lot.

Hey @caevv, just out of curiosity, what would you need that for?

I'm storing the tx id as a key on a key/value storage, and as the DB will be huge, by storing the key as bytes, instead of a string, it's saving half of the space.
Now I'm doing this:
key, _ := hex.DecodeString(input.PreviousTxID)

But the implementation on the library seems like it's doing the opposite, so just thought it would save a bit of work.
It's not a blocker for me, just a suggestion.
Thanks.

The main reason we chose to use string instead of bytes was so that we abstract away the reversing of the endianness. We previously had it as ba 32-byte slice but it was annoying with extra boilerplate code so we just swapped it out for string to make things easier. Thanks for raising this actually, you made me realise that we're not handling the error returned in some cases. I will add the PreviousTxIDBytes element to sort that out now. We were considering making all struct elements private and making getters and setters for everything to avoid all this - maybe someday we'll do something like that in v2 or something...


Streaming exporter strategy seems not working properly

Testing a couple times streaming strategy, looks that it's executed only once, like a batch behavior, both on console and output on file

From the provided Configuration it seems it's only a matter of available links to explore.
Using the following one shows the results as expected:
scooby:
  crawl:
    url:
      "https://www.example.com"
    policy:
       hyperlinks
    scrape:
       elements
     exports:
       streaming:
         results get(el => (el.tag, el.text)) output:
           toConsole withFormat json


dubbo2.6.3 服务提供者中使用泛化调用时，绑定attachments时interface错误

[*] I have searched the issues of this repository and believe that this is not a duplicate.
[*] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.6.3
Operating System version: macos
Java version: 1.8.0

Steps to reproduce this issue
dubbo服务提供者中使用泛化调用时，绑定attachments时interface错误，导致thrift encode失败：Could not encode request, the specified interface may be incorrect.

调用方调用 interfaceA.methodA方法
methodA中通过泛化调用interfaceB.methodB方法
attachements中的interface参数为interfaceA，而不是interfaceB

问题出现在AbstractClusterInvoker中的233行：
`
// binding attachments into invocation.
    Map<String, String> contextAttachments = RpcContext.getContext().getAttachments();

    if (contextAttachments != null && contextAttachments.size() != 0) {

        ((RpcInvocation) invocation).addAttachments(contextAttachments);

    }

`

Hi, can u provide a demo to reproduce this problem?
I'd like take a look.

不好意思，之后上传demo，目前我查到的原因如下：
AbstractInvoker.java

Seems like this problem will occur whether use thrift protocol or dubbo protocol right ?
Pls send a simple demo that can reproduce this problem.

Seems like this problem will occur whether use thrift protocol or dubbo protocol right ?
Pls send a simple demo that can reproduce this problem.

@carryxyh  提供了重现demo：
https://github.com/cadeeper/dubbo-thrift-bug-demo/tree/master

@cadeeper
Ok, I'd like have a look.

Hello, I don't know about the thrift protocol. I can try to reproduce this problem myself.

Consumer invoke interface1
in interface1's impl, generic invoke interface2

In this case, the generic invoke will actually call a wrong interface (interface1). Is this what it looks like?

There is a problem.
Thx for your issue, I will fix it soon.

There is a problem.
Thx for your issue, I will fix it soon.
好的

we have met the same problem,


Cannot install dev MNE (macOS + no Anaconda)

Hey there,
I'm trying to install dev version of MNE and I just can't get it to work. I'm getting:   No module named setuptools
Needless to say that not only do I have setuptools installed, but no problems happen when I try to install stable MNE. Two possibly significant factors:

macOS user
not using Anaconda - I couldn't find any other way that would enable keyboard shortcuts in raw.plot()

Any ideas?
Regards,
Maciej Gaca

how did you install python with scipy etc.?
what does
which python
which pip
give you ?

Used pip3 install <package>.
which python3:  /Library/Frameworks/Python.framework/Versions/3.6/bin/python3
which pip3 :  /Library/Frameworks/Python.framework/Versions/3.6/bin/pip3
which python and which pip are connected to Python 2.7 so I suppose they aren't of any use atm.

so are you installing with
python3 setup.py install
?
the dev version too?

Of course I did not. Well that was easy.
Thank you very much, apologies for that.
Regards


Setup doman + github org.

https://github.com/crev-dev +

?

I'm not sure why you don't like -rs :) crev.rs is available, too.

crev.rs would be neat.

I don't want -rs becase someone is working on NPM integration, and there are plans for Julia and so on.

If you create a template crev repo, I can update the new command to suggest it.


Grant admin privileges to maiste

Privileges are required to handle some actions in ocaml-ci.

:fearful:

@MisterDA, do you want to join to? :smile:

In fact, it's not needed.


[Snyk] Upgrade body-parser from 1.20.0 to 1.20.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade body-parser from 1.20.0 to 1.20.1.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2022-10-06.

Release notes

Package name: body-parser

1.20.1 - 2022-10-061.20.1

1.20.0 - 2022-04-03
Fix error message for json parse whitespace in strict
Fix internal error when inflated body exceeds limit
Prevent loss of async hooks context
Prevent hanging when request already read
deps<EMAIL_ADDRESS>
Replace internal eval usage with Function constructor
Use instance methods on process to check for listeners

deps<EMAIL_ADDRESS>
deps<EMAIL_ADDRESS>deps<EMAIL_ADDRESS>

deps<EMAIL_ADDRESS>deps<EMAIL_ADDRESS>deps<EMAIL_ADDRESS>
deps<EMAIL_ADDRESS>

    from body-parser GitHub release notes

Commit messages

Package name: body-parser

830bdfb 1.20.1
ecad1cc build<EMAIL_ADDRESS>03b93cf build<EMAIL_ADDRESS>2c611fc build<EMAIL_ADDRESS>f199e94 perf: remove unnecessary object clone
0123e12 build<EMAIL_ADDRESS>de1e6c2 build<EMAIL_ADDRESS>477ff13 build<EMAIL_ADDRESS>40c3fff deps<EMAIL_ADDRESS>4aa84b7 build<EMAIL_ADDRESS>b274dd9 build<EMAIL_ADDRESS>eb6dd96 build<EMAIL_ADDRESS>fe2f823 docs: fix typo in readme
7013c29 build<EMAIL_ADDRESS>9d02699 build<EMAIL_ADDRESS>b89bdc7 build<EMAIL_ADDRESS>44ffa97 build<EMAIL_ADDRESS>633a78a build<EMAIL_ADDRESS>0936f44 build<EMAIL_ADDRESS>3d9269e build: support Node.js 18.x

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Coverage remained the same at 90.909% when pulling 8e7b2accbe63cf2b63ec51b81c3a1e0ce24e6ee1 on snyk-upgrade-d8780be96ec2029f2d7349e9998a1dab into 97055fec0782091235da5ce942adc73feb23ef43 on main.


add parameter -repo to theme-update

Hi, while using you script for webmin I added an option to change default repo.
Here I provide a backport to youn theme update script, may be you find it useful.
I do translations in githup editor because of better UTF-8 support then in vi on linux.
So I can test changes made on github and only push to your repo when reviewed in my setup.
no more seperate: "adjust translation after reviewing in UI" pull requests :-)
I know I can also use git on command line, but not so used to it.

Kay, hi.
I don't mind multiple commits. I can squash them in one and then merge.
Thank you for your ideas but I don't find useful changing repos. I will keep in mind your effort and maybe use it one day but not just right now.
Regards,
Ilia

it was also not for you more for me because translation in contect of webmin/usermin and authentic can be a challenge ;-)
You solved it very simple by having everything in /lang only and non_UTF files simply linked to UTF-8 Verssion.
With webmin/usermin you have to deal with config.info, config.info.de, mutilple langs in module.info and a mix of lang / ulang dirs wich may or may not contain UTF-8 transaltions :-(
In this mix it was very helpful for me to try the changes made as easy as possible from my fork ..
may be you can think about going with the HOST and/or REPO varibales without -repo parameter

We're working with Jamie to get rid of non-unicode languages. It should happen some time soon.
Thank you.

sounds good! meanwhile I have to deal with it :-), at least I want to finish the german translations for filemin and the modules.info descriptions shown on dashboard

Great. Thank you for contributing.

Kay, by the way, there is authentic-theme/help that you would want to take a look. 😉

I'll have a look :-)

I guess I have to create the file file-mager.de.html ?

I suppose. Besides some modules have .de.UTF-8.


User tree view feature

Hello @tananaev @Abyss777
Some time ago, i see in one open Issue some body are developing the TREE view for user and group list,
That will be great to have this User tree view to can handle and know better the car and the level of the user.
For example, to add a new user to another user can we select the all ready registre user and select ADD, and when we finish, we can have the tree view.
Maybe we can integrate this feature in next release.
Thanks for all your work... Your project make our life better

As far as I know, there was never a tree of users. There was a tree for devices.

Ok, this is my image...
To can see better the car, we can select to expand or select to retract the user, and inside the user are the GPS devices
Is possible to add this feature?
Rigth manager.pdf

That looks like a tree of devices, not users.

Well @tananaev ... User with they devices...
Can we have this?

There are already groups for devices, but there is no plan for adding tree view because there were many issues with it in the past. You can try to find the code in one of the pull request and try it yourself if you want.


docs(samples): added samples and tests for BigQuery export RPCs

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

If you write sample code, please follow the samples format.

Would "SECRET_MANAGER_KEYS" work for multiple keys? (The environment variable is plural but I'm not sure)

The kokoro configuration looks correct. It will work with multiple keys.


SAA-1335 capture the released prison code in the stored event data for clarity.

LOW RISK
Small tweak to include the release prison code in the event review data when capturing and storing releases for event review presented on the change in circumstances screen in the UI.

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

...service/events/handlers/InterestingEventHandler.kt
75.43% <33.33%> (ø)

:loudspeaker: Thoughts on this report? Let us know!.


[Documentation] Difference between vi.spyOn and vi.fn is vague.

Clear and concise description of the problem
From the function Mocking Example:

...However spies can only help you spy on functions, they are not able to alter the implementation of those functions.

As a first-time reader, this sounds to me that

Spies (vi.spyOn) only allow you to monitor a method
Mock functions (vi.fn) allow you to alter the implementation of a method + allows restoring the original implementation with clearAllMocks

Then, the following example has these two lines
    spy.mockImplementationOnce(() => 'access-restricted')
    expect(messages.getLatest()).toEqual('access-restricted')

Wait, so we can replace or add behaviour to a function using vi.spyOn, but also restore the original implementation with clearAllMocks
Mocking and Spying by themselves are already confusing terms in testing, and I would understand as much as
If I'm not mistaken and based on my experience with Jest, I would assume that

vi.fn replaces or adds behaviour to a function
vi.spyOn replaces or adds a behaviour to a function, but allows restoring the original implementation

Suggested solution
I feel there needs to be a better explanation for the differences between vi.spyOn and vi.fn.
What exactly are now the differences between vi.spyOn and vi.fn?
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Sorry, but I'm now even more lost. I'm moving from Jest and on the Vitest homepage is read:

Jest Compatible
Expect, snapshot, coverage, and more - migrating from Jest is straightforward.

I'm trying to connect the similarities or differences between jest.fn/jest.spyOn and vi.fn/vi.spyOn.
Having this method
const myObj = {
   addNumbers(a, b){
   return a + b;
}}

In Jest:

jest.fn mocks a function by overriding it

test('mocks addNumbers using jest.fn', () => {
   const mockAddNumbers = jest.fn();
   mockAddNumbers.mockReturnValue(30);
   myObj.addNumbers = mockAddNumbers;

   expect(myObj.addNumbers(10, 20)).toBe(30);
});

jest.spyOn monitors a function, but also allows function mocking while being able to restore the original implementation.

test('mocks addNumbers using jest.spyOn', () => {
   const spy = jest.spyOn(myObj, 'addNumbers');
   spy.mockImplementation(() => 30);

   expect(myObj.addNumbers(10, 20)).toBe(30);
   expect(spy).toHaveBeenCalledTimes(1);

   spy.mockRestore();
});

However, it seems like above doesn't apply to vitest methods?

However, it seems like above doesn't apply to vitest methods?

it does. They should function the same way.

it does. They should function the same way as they do in Jest.

But apparently they don't.
Let's have a look at these examples:
With Jest:
const intersectionObserverMock = () => ({
  observe: () => null,
  disconnect: () => null,
});
window.IntersectionObserver = jest
   .fn()
   .mockImplementation(intersectionObserverMock);

If you create a new IntersectionObserver and you console.log it you get:
observer { observe: [Getter/Setter], disconnect: [Getter/Setter] }
With Vitest:
const intersectionObserverMock = () => ({
  observe: () => null,
  disconnect: () => null,
});
window.IntersectionObserver = vi
  .fn()
  .mockImplementation(intersectionObserverMock);

If you create a new IntersectionObserver and you console.log it you get:
{ ctaObserver: spy {} }

🤔 ...

+1

It's described in documentation: https://vitest.dev/api/mock.html


[WIP] Add selector to get entry title

Read either from seed or entry model.
REDMINE-15907

Coverage remained the same at 99.205% when pulling 9ad703769cf2955269387387d36a71f0eca86fe6 on tf:entry-title-selector into 0c46f88d3e06cf056c7370c322746bcf2c7dc647 on codevise:master.


Properties to disable built-in filters/predicates

Add a ConditionalOnEnabled{Filter|Predicate} to use the name

Do you mean that use a property to disable all built-in filters or predicates ?


Data from one API to another API of the SDK needs Sanitization and Validation

Improper Input Validation (CWE-20) inside Trust Boundary:
• Data from one API to another API of the SDK needs Sanitization and Validation, for future safety concerns.

All internal APIs should be reviewed for proper input validation

• File : src\azure\core\az_json_writer.c
• API : _az_validate_json( )
• Lines : 717 – 750
• Issue : Improper Input Validation is Missing for:
• json_text
• last_token_kind
• Caller Graph:

There is only one called of this internal method, and it is guaranteed to call it correctly, but we should add a null precondition check on last_token_kind for maintainability and consistency.
The need for json_text validation is interesting. The JSON reader init method does the precondition validation already. Is the ask to duplicate such validation? That's doable, but it seems unnecessary, and looks like a restriction of the detection tool.
Is there a way to opt-out of specific instances of false positives, based on context and code inspection?

The need for json_text validation is interesting. The JSON reader init method does the precondition validation already. Is the ask to duplicate such validation? That's doable, but it seems unnecessary, and looks like a restriction of the detection tool.

After discussing with @ericwol-msft, this is discovered through manual code inspection, so regarding json_text we landed on adding a comment stating that the reader init method does the appropriate validation, and not to duplicate the precondition, unnecessarily.


Port to 1.20.4

Hey, here's a port to 1.20.4!
Since there isn't a 1.20.4 branch and i cannot create one for your repository, i'm not sure how to PR, hopefully this one works regardless.

Thank you for this PR, however I have gone ahead and done the port myself as there were other changes that were required to be made, not simply updating the versions.

Sounds good, I was trying to get a mod depending on this to work, but my lack of experience with Java was proving tedious. Thank you for your work, sorry for wasting your time a bit!


dispatcher: Batch node updates

When setting nodes to ready, down, or disconnected, do so in a batch
instead of one node at a time. This can avoid very slow responses to
network partitions or recovery from network partitions.
Tested manually and with the Docker integration tests (needs a slight tweak to one integration test, PR https://github.com/docker/docker/pull/25342).
Fixes #1289
cc @dongluochen @aluzzardi

cc @LK4D4 as well

Current coverage is 55.01% (diff: 68.75%)

Merging #1297 into master will increase coverage by 0.07%

@@             master      #1297   diff @@
==========================================
  Files            78         78
  Lines         12467      12494    +27
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6850       6874    +24
- Misses         4671       4672     +1
- Partials        946        948     +2

Powered by Codecov. Last update f1d2775...7ae935f

That's pretty cool.
LGTM

I've updated this to make register block until the batch runs, as suggested by @tonistiigi. This avoids the need for changes to Docker integration tests, and it means that when swarm init or swarm join completes, the node information will all be up to date.
I also added some selects around writes to processUpdatesTrigger that check for context cancellation, in case the dispatcher is stopped while multiple RPC invocations are trying to write to this channel (very unlikely).

LGTM


envoy: Bump envoy version to v1.27.3

This is mainly for below security fixes

CVE-2024-23324
CVE-2024-23325
CVE-2024-23322
CVE-2024-23323
CVE-2024-23327

Related build: https://github.com/cilium/proxy/actions/runs/7849793599/job/21423879960
Upstream release: https://github.com/envoyproxy/envoy/releases/tag/v1.27.3

/test-backport-1.15

All required jobs are passed, marking this ready to review for tophat to take a look.

Conformance Gateway API got renamed to Conformance Gateway API (ci-gateway-api) and it passed. shipping.


bring back quickstart

@NetOperatorWibby bring back quickstart link and redirect to https://lbry.tech/playground... based on closes issues #819

We removed this on purpose, that section already leads to lbry.tech


Remove returns of financial wealth

Currently only used to increase bank balances but not in any other decision by the households, which seems incoherent. The decision is to remove the mechanism altogether, in order to remove an unjustified parameter and simplify model and calibration. ** Modify paper accordingly**.

Return on financial wealth mechanism removed from model


[Feature] Extract 3rd party dependencies

Priority
P1-Stopper
OS type
Ubuntu
Hardware type
Xeon-GNR
Running nodes
Single Node
Description
Task 1-1
Extract all of the common 3rd party dependencies into comps/3rd_party folder.

Have done in this branch https://github.com/opea-project/GenAIComps/tree/refactor_helmchart