Client Request Socket timeout issue.

Hi all:
I have seen some strange issue relating to client request timeout.
I have set the socket timeout in ClientConfig and the log output is as follow:
bootstrapTime=1465219076276
context=
deploymentPath=/nfs/public/rw/homes/uni_adm/vold
localHostName=ebi5-039.ebi.ac.uk
sequence=0
storeName=avro-uniprot
updateTime=1465219075861
releaseVersion=null
clusterMetadataVersion=0
max_connections=200
max_total_connections=500
connection_timeout_ms=60000
socket_timeout_ms=60000
routing_timeout_ms=60000
client_zone_id=-1
failuredetector_implementation=voldemort.cluster.failuredetector.ThresholdFailureDetector
failuredetector_threshold=95
failuredetector_threshold_count_minimum=30
failuredetector_threshold_interval=300000
failuredetector_threshold_async_recovery_interval=10000

But when I run the client application, I can see some log errors like:
14:19:56.610 [voldemort-niosocket-client-t8] WARN  v.s.s.c.ClientRequestExecutor - Client request associated with Socket[addr=ves-ebi-ca/<IP_ADDRESS>,port=6666,localport=50337] Destination ves-ebi-ca:6666(vp1) timed out. Start time(ns)<PHONE_NUMBER>7747612 allowed time(ns)<PHONE_NUMBER> elapsed time(ns)<PHONE_NUMBER>
14:19:56.610 [voldemort-niosocket-client-t4] WARN  v.s.s.c.ClientRequestExecutor - Client request associated with Socket[addr=ves-ebi-ba/<IP_ADDRESS>,port=6666,localport=48827] Destination ves-ebi-ba:6666(vp1) timed out. Start time(ns)<PHONE_NUMBER>7490159 allowed time(ns)<PHONE_NUMBER> elapsed time(ns)<PHONE_NUMBER>

The actual allowed time and elapsed time both are about 5 ms. It seems the socket timetout setting doesn't take effect.
Am I missing something ?

What version of voldemort client are you using?
On Jun 7, 2016 3:09 AM, "Wudong Liu"<EMAIL_ADDRESS>wrote:

Hi all:
I have seen some strange issue relating to client request timeout.
I have set the socket timeout in ClientConfig and the log output is as
follow:
bootstrapTime=1465219076276
context=
deploymentPath=/nfs/public/rw/homes/uni_adm/vold
localHostName=ebi5-039.ebi.ac.uk
sequence=0
storeName=avro-uniprot
updateTime=1465219075861
releaseVersion=null
clusterMetadataVersion=0
max_connections=200
max_total_connections=500
connection_timeout_ms=60000
socket_timeout_ms=60000
routing_timeout_ms=60000
client_zone_id=-1
failuredetector_implementation=voldemort.cluster.failuredetector.ThresholdFailureDetector
failuredetector_threshold=95
failuredetector_threshold_count_minimum=30
failuredetector_threshold_interval=300000
failuredetector_threshold_async_recovery_interval=10000
But when I run the client application, I can see some log errors like:
14:19:56.610 [voldemort-niosocket-client-t8] WARN  v.s.s.c.ClientRequestExecutor - Client request associated with Socket[addr=ves-ebi-ca/<IP_ADDRESS>,port=6666,localport=50337] Destination ves-ebi-ca:6666(vp1) timed out. Start time(ns)<PHONE_NUMBER>7747612 allowed time(ns)<PHONE_NUMBER> elapsed time(ns)<PHONE_NUMBER>
14:19:56.610 [voldemort-niosocket-client-t4] WARN  v.s.s.c.ClientRequestExecutor - Client request associated with Socket[addr=ves-ebi-ba/<IP_ADDRESS>,port=6666,localport=48827] Destination ves-ebi-ba:6666(vp1) timed out. Start time(ns)<PHONE_NUMBER>7490159 allowed time(ns)<PHONE_NUMBER> elapsed time(ns)<PHONE_NUMBER>
The actual allowed time and elapsed time both are about 5 ms. It seems the
socket timetout setting doesn't take effect.
Am I missing something ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/voldemort/voldemort/issues/414, or mute the thread
https://github.com/notifications/unsubscribe/AGN-MxioIVbhr3z_DdU4PbjkBAbHxVihks5qJUNggaJpZM4Ivw0j
.

It is branching off 1.9.17, with some minor changes to using avro 1.7

Do you have multiple client factories ? Do you use Admin Client ?
There are some system store poller threads, whose timeouts does not carry
forward from the client timeouts in older versions.
Admin client timeouts were also not correctly set from the client timeouts
in the older versions.
Try upgrading the client to the latest version and see if the problem goes
away. If not, probably you need to instrument the code to see where the
exception is happening and who is not rightly setting the tmieout to
investigate further.
Thanks,
Arun.
On Tue, Jun 7, 2016 at 7:50 AM, Wudong Liu<EMAIL_ADDRESS>wrote:

It is branching off 1.9.17, with some minor changes to using avro 1.7
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/voldemort/voldemort/issues/414#issuecomment-224305306,
or mute the thread
https://github.com/notifications/unsubscribe/AGN-M_3Y_g4VCQVNeD_WQ4KV1W-6TdfMks5qJYVJgaJpZM4Ivw0j
.

From the master branch code i could see that:
in SystemStoreClientFactory, a new ClientConfig is created and of which RoutingTimeout is set from the main ClientConfig.
But the setRoutingTimeout() method in the ClientConfig doesn't update the its TimeoutConfig which is created with the default RoutingTimeout value.
It is the TimeoutConfig that used in system store which explains the problem I'm having.


Support for Promises

Hi @TassosD and @pauliusuza. This is a great library!
I'd be willing to open a PR to add support for Promises.
Would that be a welcome contribution?

Better question is why are there callbacks/promises at all. All of the functions are synchronous.

Hey @boenrobot I hadn't actually looked into the implementation. If there's a way to make this lib synchronous (instead of promisifying it), that would also be an improvement.


[CARBONDATA-3501] Fix update table with varchar column

Problem
Update on table with varchar column will throw exception:
java.lang.Exception: Dataload failed, String length cannot exceed 32000 characters
	at org.apache.carbondata.streaming.parser.FieldConverter$.objectToString(FieldConverter.scala:52)
	at org.apache.carbondata.spark.util.CarbonScalaUtil$.getString(CarbonScalaUtil.scala:70)
	at org.apache.carbondata.spark.rdd.NewRddIterator$$anonfun$next$1.apply$mcVI$sp(NewCarbonDataLoadRDD.scala:357)
	at scala.collection.immutable.Range.foreach$mVc$sp(Range.scala:160)
	at org.apache.carbondata.spark.rdd.NewRddIterator.next(NewCarbonDataLoadRDD.scala:356)
	at org.apache.carbondata.spark.rdd.NewRddIterator.next(NewCarbonDataLoadRDD.scala:333)
	at org.apache.carbondata.processing.loading.steps.InputProcessorStepImpl$InputProcessorIterator.getBatch(InputProcessorStepImpl.java:232)

Analyse
In the loading part of update operation, it gets the isVarcharTypeMapping for each column in the order when table created. And this gives a hint for checking string length. It does not allow to exceeds 32000 char for a column which is not varchar type.
However when changing the plan for updating in CarbonIUDRule, it first deletes the old expression and appends the new one, which makes the order differ to table created. Such that the string length checking fail.
Solution
Keep the order as table created when modify update plan
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/568/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/570/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/574/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/569/

LGTM, test this pr successfully in local.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.1/585/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.2/588/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/590/

@jackylk please help to review, thanks.

LGTM


Is there a document somewhere explain what these mysterious values between 0 (red) and 359?

Or are supposed to just find the color we prefer by trial by error?
Why not hex codes colors as a suggestion for example example? At least there's a sample of the color displayed when you're in a CSS note.

Or are supposed to just find the color we prefer by trial by error?
Why not hex codes colors as a suggestion for example example? At least there's a sample of the color displayed when you're in a CSS note.

Colors HSL and HSLA
couleur.io


/network/peers endpoint

add openAPI documentation to /network/peers rest endpoint and validate against eth2,lighthouse, and prysm.

https://lighthouse-book.sigmaprime.io/http_network.html#networkpeer_id
lighthouse returns an array of string for peer_id, and so does Teku.

add OpenApi annotations
add unit tests (none currently)
add /v1 prefix, pending conversation around that decision

to test, when you run run.sh, set number of nodes to several, eg -n=4. Then you actually get response data :)


Split iOS, Android, support library

@ubamrein proposes:
If you want to go one step further, you could split ios, android and core into three crates. So you don’t have to compile (or clutter the code with #[cfg]s) for different platforms, which means your library gets smaller (as only the symbols of the specific platform is included).
Further, it makes development a bit easier, as the core IR product can be shared between iOS and Android, such that working on the platform specific code, can build faster :slightly_smiling_face:
As a neat bonus, you could also provide the API to desktop apps or CLIs :wink:

Wait, if I use #[cfg, the code is included anyway, even if it's not active?

Wait, if I use #[cfg, the code is included anyway, even if it's not active?
No , I overlooked the #[cfg]s on the ffi imports :)

OK, I'm still new to rust, even after half a year of coding nearly daily I'm still surprised by some stuff. So everything is possible ;)

Closing this, as the #[cfg will take care of this.


pull request for e2e test at 12/31/2016 00:15:08

:white_check_mark: Validation status: passed
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.


Release v3.8 to hackage

I see that release v3.8 is tagged on github, but I don't yet see this version on hackage.
@ndmitchell could you please release it to hackage?

Oops! Fixed.

Thank you :heart_eyes:


Show error and exit on shader/texture file open failure

When craft cannot load shader or texture files (for example, if launched outside of its resource directory), it would crash with a segmentation fault:
native-build $ ./craft
error 78: failed to open file for reading
Segmentation fault: 11

This PR fixes this crash by exiting with an error, as well as printing the file which failed to load:
native-build $ ./craft
load_png_texture textures/texture.png failed, error 78: failed to open file for reading

Thank you, I appreciate small PRs!


Replaced autobind-decorator with bind-decorator

For smaller bundles

This one throws better error messages, already we know there's an ear now.

Can you please take a look at the errors

Also why does materialDom even needs this???
I thought only event listeners will need this

Can you please take a look at the errors

It's a OSX/fsevents error, it seems that they have a broken/at least not prebuild version

Nups, I am taking about this
https://travis-ci.org/prateekbh/preact-material-components/jobs/424755513#L456

Also why does materialDom even needs this???
I thought only event listeners will need this

I could swear I had problems with that, but now it seems to work

Nups, I am taking about this
https://travis-ci.org/prateekbh/preact-material-components/jobs/424755513#L456

I think you can't override methods annotated with@bind.


Fix nested function literal.

This violates strict-mode, and breaks Safari on MacOS/iOS.

Hi @tetious, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

Nice catch @tetious!
@AmazingJaze For a summary of the issue with function declarations in blocks see https://jslinterrors.com/function-statements-should-not-be-placed-in-blocks


must return an object. Instead received [object Object].

hi,I'm new to this
the error message is
browser.js:40 Uncaught Invariant Violation: mapStateToProps must return an object. Instead received [object Object].
1.
		case 'TOGGLE_TODO':

		return	state.todos.map(function(state){
				if(state.id!==action.id){
				return	state
				};

				return {...state,completed:true}
			});

(from egghead )
this will throw the error,but I don't know why.
But if I not using return before state.todos.map it is fine

		case 'TOGGLE_TODO':

		  	state.todos.map(function(state){
				if(state.id!==action.id){
				return	state
				};

				//return state.completed=true}
				return {...state,completed:true}
			});

if I change to this it will not change the store

only
		case 'TOGGLE_TODO':

		  	state.todos.map(function(state){
				if(state.id!==action.id){
				return	state
				};

				//return state.completed=true}
				return state.completed=true
			});

and
		case 'TOGGLE_TODO':

		  	state.todos.map(function(state){
				if(state.id!==action.id){
				return	state
				};

				//return state.completed=true}
				return Object.assign(state,{completed:true})
			});

can successfully update the store and no error message,but this will also change the prev state,
how can I change this code?  Thanks for reply.

If you found the solution please share it in a comment so other people who find this issue know how you solved it. Thanks!

I was using below works good ,but still don't know why others give error when it return [object Object] but still  says must return an object
case 'TOGGLE_TODO':

      return Object.assign({},state,{todos:state.todos.map(function(state){
                if(state.id!==action.id){
                return  state
                };

                return {...state,completed:!state.completed}

			}) }
      )

Why are you posting the reducer code? The error says mapStateToProps returned something that isn’t a plain object. Not the reducer.
My guess is your mapStateToProps returns something like state.todos which will not work. The point of that function is to provide props to component, so it has to return an object with those props. Rather than pass state.todos you would probably want { todos: state.todos }.
If this is the case, we need to make a better error message that detects arrays. Contributions are welcome.

The problem here is that the error message isn't clear enough, an object is being passed in but it isn't a plain object until EasonWang01 uses Object.assign(). checkStateShape calls _.isPlainObject which requires Object.prototype == null or to have been created with Object.constructor. The error message could probably updated to read %sToProps must return a plain object. Instead received %s
The wording for computeMergedProps could be updated as well to reflect this.

the mapStateToProp is look like this
import React, { Component } from 'react'
import TodoInput from './TodoInput.js'
import TodoList from './TodoList.js'
import {connect} from 'react-redux'

class App extends Component {

  render() {
    return (

Todo list

        <TodoInput />
        <TodoList  todos={this.props}/>

    )
  }

}
function  mapStateToProp(state){

	return state
}

export default connect(mapStateToProp)(App)

I guess it might be that I'm not using combineReducer so I need to return the full state to state
( ΄◞ิ .̫.̫ ◟ิ‵)y

Can you provide a full project reproducing the problem on GitHub? These pieces don't quite add up in my mind. For example it is strange you are passing the whole props object as todos prop down.

Hi,  https://github.com/EasonWang01/Redux-tutorial/tree/link  it's a origin version.
in  Redux folder=> reducer.js
I try to change line 18~26 make it back to two days ago,but the error don't show again,don't know.
But I thinks it's not a big problem need to care,maybe just some return object type that I miss.
( ´ ▽ ` )b

update
change reducer.js  line 4~15 to below
switch(action.type){
		case 'ADD_TODO':

			return(	Object.assign({},state,{
				todos:[...state.todos,{
				  text:action.text,
				  completed:false,
				  id:getId++

				}]
			})
			)

and click initial for demo,then the error will appear
( ^ ᴗ ^ )ε

Hi,could I ask other basic question beside this~
that I want to split the reducer function and do something like below,the project is the same as I provide above,only change reducer.js.
But it give me TodoList.js:63Uncaught TypeError: Cannot read property 'map' of undefined
Sorry for this basic question,but I don't know is it miss something.Thanks for reply.
let getId = 1 ;

 export default function reducer (state,action) {
	return Object.assign({},state,{
		visbility:reducer2(state="SHOW_ALL",action),
		todos:reducer1(state=[],action)
	})
}

 function reducer1(state,action){
	switch(action.type){
		case 'ADD_TODO':

			return [...state.todos,{
				  text:action.text,
				  completed:false,
				  id:getId++

				}]

		case 'TOGGLE_TODO':

      return state.todos.map(function(state){
                if(state.id!==action.id){
                return  state
                };

                return {...state,completed:!state.completed}

			});

}}

 function reducer2(state,action){
  switch(action.type){
      	case 'SET_VISBILITY_FILTER':

      	return action.filter

		default:
			return state;

	}

}

But one thing I found is when some error occur in reducer the console.error will always point to some component file rather than reducer.js file.

Normally you return an object like { todos: Array<Todo> }:
        case 'ADD_TODO':

            return( Object.assign({},state,{
                todos:[{
                  text:action.text,
                  completed:false,
                  id:getId++

                },...state.todos]
            })
            )

However when handling TOGGLE_TODO, you return just an Array<Todo>:
        case 'TOGGLE_TODO':

  return  state.todos.map(function(state){
                if(state.id!==action.id){
                return  state
                };

                return {...state,completed:true}
            });

This is missing Object.assign({}, state, { todos: ... }).
Then this state gets returned from mapState() which is where the error fires. Fixing the reducer to return the object would fix the problem. I also suggest you to use reducer composition as Egghead series shows, rather than put all the code into a single big reducer.
Hope it helps!

thanks!


MINOR: Let TestRaftRequestHandler use RequestHandlerHelper

Since we factored out RequestHandlerHelper, we may as well make use of it in TestRaftRequestHandler.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

So what I ended up doing is moving the helpers that TestRaftRequestHandler needed into RequestChannel. Basically this encapsulates the construction of the RequestChannel.Response, which seems a bit nicer since it was confusing to see sendResponse used for throttling events. It is also nice because it ensures that we do not need to rely on the caller to ensure that updateErrorMetrics gets called.

So what I ended up doing is moving the helpers that TestRaftRequestHandler needed into RequestChannel. Basically this encapsulates the construction of the RequestChannel.Response, which seems a bit nicer since it was confusing to see sendResponse used for throttling events. It is also nice because it ensures that we do not need to rely on the caller to ensure that updateErrorMetrics gets called.

Also, looks like there is a compiler error:

02:22:44  [Error] /home/jenkins/jenkins-agent/workspace/Kafka_kafka-pr_PR-9912/core/src/main/scala/kafka/server/KafkaApis.scala:148: value closeConnection is not a member of kafka.server.RequestHandlerHelper

Also, looks like there is a compiler error:

02:22:44  [Error] /home/jenkins/jenkins-agent/workspace/Kafka_kafka-pr_PR-9912/core/src/main/scala/kafka/server/KafkaApis.scala:148: value closeConnection is not a member of kafka.server.RequestHandlerHelper


Notebooks docs are out of date

The documentation on notebooks needs to be updated as we have changed some notebook features:

Preview/Run is gone - we just have run now (it was too confusing and people couldn't tell the difference)
The metric selector is gone - it was too slow for larger buckets. We now have the query builder and the flux editor.
Column editor is gone
Output to bucket is gone (as a standalone panel) - you now do that in the flux editor if that's what you want to do.

Overview could use a description of what a "cell" is.


Add projects subcommands to source command

Add subcommands for different PyDis projects to !source command. For example, SeasonalBot: !source seasonalbot. These subcommands should give short description about project and link to GitHub.
I think these projects should have subcommands:

site
branding
snekbox
public-stats
django-simple-bulma

Sounds nice. Will the descriptions be written and stored in the bot, or will they be fetched from GitHub? In the case of the latter, it may be preferable, but I'd rather not overcomplicate things for such a simple feature. Unless it's a simple unauthorised request to get the description, I don't think we should fetch them from GH.

For some reason I feel seasonalbot had a similar feature actually. Can anyone confirm or deny?

I think this should be inside Markdown files in bot repo and these should be loaded on startup.

SeasonalBot now has a source command - python-discord/seasonalbot#469

@Den4200 Removed SeasonalBot from list

We can fetch all the non-archived repositories using GitHub API and then add a pagination to it. With each repository in the following way maybe:
# Repository Name
- Description (from gh)

We have the .github command on lancebot:

I don't think it's worth keeping separate custom markdown descriptions here. I don't think it would add much extra value to the info from .github, and could take a decent amount of work to keep up to date.


How to share embeddings among features?

I have a dataset where I have 3 columns, created_user, delegated_user, tagged_user which are different users but come from the same population of users. How do I share embeddings among them? I was thinking of using multi-value input like this example - https://deepctr-doc.readthedocs.io/en/latest/Examples.html#multi-value-input-movielens but the order matters in this case. Do I create separate embeddings? Is there a way to share embeddings?

set the same embedding_name

Thank you! What's the difference between name, embedding_name, and group_name?

for example , we have 3 feature group named A B C，if we want to share embedding of feature group A and B wen can set a a same embedding_name for A and B。
group_name only works in DeepFM and FLEN, the model will do feature interactions for feature groups  having same group_name


make build logs quieter by removing the YN0013 log entries which is t…

…he "can't be found in the cache and will be fetched from the remote registry" one

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3642

Relevant Lines:
14249

💛  - Coveralls


DiscriminatorColumn inside Partial Object

When requesting a partial object in a DQL query the discriminator column is returned.
$results = $em->createQuery("SELECT partial.{id, name} FROM Entity:ChildA")->getArrayResult();
The array results contains the keys: id, name and type (type was not requested).

Entity configuration:
`

@InheritanceType("SINGLE_TABLE")
@DiscriminatorColumn(name="type", type="integer")
@DiscriminatorMap({
"1" = "Entity\ChildA",
"2" = "Entity\ChildB",
"3" = "Entity\ChildC"
})
`

I didn't even know that array selection was possible with partial entity queries in DQL: nice catch!

Hi.
Where can I start to analyze the error?


Opening "Open Keyboard Shortcuts" via the command palette using Enter immediately starts editing an item if it was already selected

VSCode Version: 1.19.2
OS Version: Windows 8.1

Steps to Reproduce:

Open the Keyboard Shortcuts list, select any item, and close the tab.
Open the command palette, select "Preferences: Open Keyboard Shortcuts" and press Enter.

The Keyboard Shortcuts tab opens and immediately goes into edit on the item you selected before. It looks like the tab opens on key-down, and the key-up is already handled by the list which causes it to start editing.

Does this issue occur when all extensions are disabled?: Yes

Looks like this doesn't reproduce anymore.


Material texture change not working anymore

Description
The material change which i did through scene graph is not working anymor. When changing color the spa cabinet color should be changed as well by applying jpg texture to the material. It was working for months and no change has been made to the code, suddenly it stopped working.
Code:
Html:  `

`
Javascript:

Live Demo
https://imajinn.live/models/foxspa/dre-1600-2cl/index.html
Version
<EMAIL_ADDRESS>
model-viewer: vX.X.X

Browser Affected

[x ] Chrome, version: xx.x.xxxx.xx
[ x] Edge
[x ] Firefox
[ ] IE
[x ] Safari

OS

[ x] Android
[x ] iOS
[ ] Linux
[x ] MacOS
[x ] Windows

AR

[ x] WebXR
[ ] SceneViewer
[ ] QuickLook

I couldn't put html in description. Though you can easily check the code in the link from chrome dev tools

It sounds like you should probably pin the version of MV you're using, until you can update to createTexture, see https://github.com/google/model-viewer/discussions/3788

Also, the real problem was an accidental push of a pre-release of v2.0 to unpkg, which has since been corrected: #3780.


Add type definitions for router5-transition-path

Implements the transitionPath stub that was in router5's type definitions.

👍


Fix all the default glossaries

Remove common words from city and other dictionaries

Fixed with this commit https://github.com/usc-isi-i2/mydig-webservice/commit/a57a7e7c5cf7594e1248a781e088f10f48ad5f41


Spyder won't start after upgrade to 3.1.0

Description of your problem
What steps will reproduce the problem?

Start spyder from terminal

What is the expected output? What do you see instead?
I expect spyder to start, instead I get an error:

spyder
QML debugging is enabled. Only use this in a safe environment.
Traceback (most recent call last):
File "/usr/lib/python2.7/site-packages/spyder/app/mainwindow.py", line 2979, in main
mainwindow = run_spyder(app, options, args)
File "/usr/lib/python2.7/site-packages/spyder/app/mainwindow.py", line 2883, in run_spyder
main.setup()
File "/usr/lib/python2.7/site-packages/spyder/app/mainwindow.py", line 780, in setup
message=_("Spyder Internal Console\n\n"
File "/usr/lib/python2.7/site-packages/spyder/plugins/console.py", line 79, in init
self.find_widget.set_editor(self.shell)
File "/usr/lib/python2.7/site-packages/spyder/widgets/findreplace.py", line 280, in set_editor
self.refresh()
File "/usr/lib/python2.7/site-packages/spyder/widgets/findreplace.py", line 259, in refresh
self.find()
File "/usr/lib/python2.7/site-packages/spyder/widgets/findreplace.py", line 326, in find
self.editor.find_text('')
File "/usr/lib/python2.7/site-packages/spyder/widgets/mixins.py", line 513, in find_text
found_cursor = self.document().find(pattern, cursor, findflag)
TypeError: arguments did not match any overloaded call:
QTextDocument.find(str, int position=0, QTextDocument.FindFlags options=0): argument 1 has unexpected type 'QRegularExpression'
QTextDocument.find(QRegExp, int position=0, QTextDocument.FindFlags options=0): argument 1 has unexpected type 'QRegularExpression'
QTextDocument.find(str, QTextCursor, QTextDocument.FindFlags options=0): argument 1 has unexpected type 'QRegularExpression'
QTextDocument.find(QRegExp, QTextCursor, QTextDocument.FindFlags options=0): argument 1 has unexpected type 'QRegularExpression'

Please provide any additional information below
Versions and main components

Spyder Version: 3.1.0
Python Version: 2.7.12
Operating system: OpenSuse 13.2

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
I can't get spyder to start, so ...

@rlaverde

What are your qt and pyqt versions?

PyQt5: 5.3.1
PyQt4: 4.11.1
How do I get the Qt-version?
Yast is showing me a bunch of libs (for instance "libqt5-qtbase-devel"), from there:
Qt5: 5.4.2
Qt4: 4.8.6

I think this is a bug of Qt5, you should upgrade to PyQt>=5.5
QRegularExpression was introduced in Qt 5.0 but QTextDocument.find supporting QRegularExpression was introduced in Qt 5.5
@ccordoba12 Since we are supporting PyQt5 5.2+,  Should we implement a workaround for this?

@rlaverde, yes. Please add a check for qt 5.5+ where you introduced this feature.
This is a priority for 3.1.1, so I want a PR as soon as possible :-)

you should upgrade to PyQt>=5.5

I can't. Pip can't and opensuse does not offer an update.
Pip:

pip2 install PyQt5
Downloading/unpacking PyQt5
Could not find any downloads that satisfy the requirement PyQt5
Cleaning up...
No distributions at all found for PyQt5
Storing debug log for failure in /root/.pip/pip.log

you should upgrade to PyQt>=5.5

I can't. Pip can't and opensuse does not offer an update.
Pip:

pip2 install PyQt5
Downloading/unpacking PyQt5
Could not find any downloads that satisfy the requirement PyQt5
Cleaning up...
No distributions at all found for PyQt5
Storing debug log for failure in /root/.pip/pip.log

@BeateBier, this error will be fixed in Spyder 3.1.1. Sorry for hassle.


Both Coinbase Wallet and MetaMask triggered on MetaMask icon click

Hi, both the Coinbase Wallet and MetaMask are triggered when clicking on MetaMask icon.

+1 - Web3Modal gets confused when both the Coinbase and Metamask extensions are enabled at the same time. In fact, when I connect to the injected provider, it randomly chooses from one or the other.
Perhaps instead of having a one Injected slot, we need the option to specify which injects providers we want.

Yeah this is a pain, happy to have found others are experiencing this issue. I haven't been able to find a consistent pattern of when it shows CB Wallet as an option or not.

Hi @Soletiq and @gregegan ,
you can work around this by defining custom options and filter the provider that you want to use.
There are properties inside each providers that give you some info, for instance:

I currently have Coinbase and MetaMask providers injected, and I see that only one of them has "isMetaMask" set to true (obviously MetaMask), the other one has "isWalletLink" set to true (and this is Coinbase):

Below you can see how I'm able to distinguish between providers, or better to capture the right one on web3modal.connect():
"custom-metamask": {
    display: {
      logo: "assets/img/metamask-logo.svg",
      name: "MetaMask Wallet",
      description: "Connect to your MetaMask Wallet"
    },
    package: true,
    connector: async () => {
        if (!isMetaMaskInstalled()) {
            win.location = "https://metamask.app.link/dapp/www.ethbox.org/app/";
            return;
        }

        let provider = null;
        if (typeof win.ethereum !== 'undefined') {
            let providers = win.ethereum.providers;
            provider = providers.find(p => p.isMetaMask); // <-- LOOK HERE
            try {
                await provider.request({ method: 'eth_requestAccounts' });
            } catch (error) {
                throw new Error("User Rejected");
            }
        } else {
            throw new Error("No MetaMask Wallet found");
        }
        console.log("MetaMask provider", provider);
        return provider;
    }
}

I need to connect my Coinbase to Metamask which needs RPC URL and chain ID , i cant find it. Can you please help me with that . I'd appreciate it.

I need to add to web3 modal now. How can I implement it? Can you please help me with that.
I'd really appreciate it. Thanks.
const providerOptions = {       walletconnect: {         package: WalletConnectProvider,         options: {           infuraId: process.env.REACT_APP_INFURA_ID         }       },       walletLink: {         package: WalletLink, // required         // options: {         //   infuraId: process.env.REACT_APP_INFURA_ID // required         // }       },       portis: {         package: Portis,         options: {           id: process.env.REACT_APP_PORTIS_ID         }       },       fortmatic: {         package: Fortmatic,         options: {           key: process.env.REACT_APP_FORTMATIC_KEY         }       },       mewconnect: {         package: MewConnect, // required         options: {           infuraId: process.env.REACT_APP_INFURA_ID // required         }       },       torus: {         package: Torus       }     };

@parisan8 and @DevSmith0213 please do not make comments that are completely unrelated to this issue. Github issues are not for whining and asking for general help. You are detracting from the great work done by @4skinSkywalker. If you have a specific issue, open a new issue and share detailed logs of the errors you are experiencing so that the community can better help you.

On Wed, 20 Apr 2022 at 13:05 zaboomafoo @.***> wrote:

@parisan8 https://github.com/parisan8 and @DevSmith0213
https://github.com/DevSmith0213 please do not make comments that are
completely unrelated to this issue. Github issues are not for whining and
asking for general help. You are detracting from the great work done by
@4skinSkywalker https://github.com/4skinSkywalker. If you have a
specific issue, open a new issue and share detailed logs of the errors you
are experiencing so that the community can better help you.
—
Reply to this email directly, view it on GitHub
https://github.com/Web3Modal/web3modal/issues/316#issuecomment-1104258180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWO6PNJVVSXXZCTP4426JLDVGBBPHANCNFSM5C2ZOLYA
.

Are you trying to make it here uncomfortable for me?
Are you trying to embarrass me? I dont even remember what was my question
or where, but you email me to intimidate me ! I take this personally and a
big hate speech towards me . If i asked a question and i needed help why
you hate me ?
Are you trying to make it here uncomfortable for me?
Are you trying to embarrass me? I dont even remember what was my question or where, but you leave a comment to intimidate me ! I take this personally and a big hate speech towards me . If i asked a question and i needed help why you hate me ? @kyokosdream

I want this solution to work, but has anyone tried it? There's an unexplained function call, window appears to be aliased to win, and most importantly, when I fix all that, it still doesn't work. If there is only one provider (window.ethereum) then the providers list is undefined. And even if I try to control for only using metamask, Coinbase jumps the gun.
I'm having this issue in Vanilla javascript, and from what I gather, web3modal's coinbase simply doesn't work in vanilla javascript (coinbase's fault, since they only have framework sdks). However, Coinbase does get in my way EVEN IF I DON'T WANT TO IMPLEMENT IT.
I think this issue was closed too quickly. Writing custom provider options for injected wallets should not satisfy as a solution. Could we reopen it and see about a real solution? Or if not a real solution, could we get good working guidance? Perhaps with an example implementation that shows the guidance works?

@kyokosdream I used the settings as given above in my vanilla JavaScript solution, and it continues to open Coinbase wallet when I click on Metamask. If both are installed, it doesn't seem to differentiate. Coinbase, as far as I can tell, just does not support vanilla JavaScript.
I believe the problem is in the second condition, where the provider for Metamask just isn't in the providers list at all. The only thing that resolves to Metamask for me is window.ethereum, and the providers list has Coinbase in it. I'm pretty sure some of this is browser specific.
The above is why I think the solution is systematic testing of an implementation that hides this complexity from the users of Web3Modal. People with both Coinbase and JavaScript wallets will increase, and this will become unwieldy quickly.

Thanks! 🙏🏼🌻
On Fri, 29 Apr 2022 at 01:23 Audie Leon Sheridan @.***>
wrote:

@kyokosdream https://github.com/kyokosdream I used the settings as
given above in my vanilla JavaScript solution, and it continues to open
Coinbase wallet when I click on Metamask. If both are installed, it doesn't
seem to differentiate. Coinbase, as far as I can tell, just does not
support vanilla JavaScript.
I believe the problem is in the second condition, where the provider for
Metamask just isn't in the providers list at all. The only thing that
resolves to Metamask for me is window.ethereum, and the providers list
has Coinbase in it. I'm pretty sure some of this is browser specific.
The above is why I think the solution is systematic testing of an
implementation that hides this complexity from the users of Web3Modal.
People with both Coinbase and JavaScript wallets will increase, and this
will become unwieldy quickly.
—
Reply to this email directly, view it on GitHub
https://github.com/Web3Modal/web3modal/issues/316#issuecomment-1112914334,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWO6PNJNAYTJJRK2534MM7TVHN56VANCNFSM5C2ZOLYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

With stable version 2.0.0 of Web3Modal now released, we are officially dropping support for version 1.x
Due to this this issue/pr was marked for closing. It is highly recommended to upgrade as 2.x will be receiving further updates that will enable functionality for some of our newer sdks like auth and push as well as support for WalletConnect v2 (See this post about WalletConnect v1 being deprecated https://medium.com/walletconnect/walletconnect-v1-0-sunset-notice-and-migration-schedule-8af9d3720d2e)
If you need to continue using Web3Modal 1.x and require this feature/fix implemented, we suggest adding it via forking V1 branch.


Set release "Installation Directory" to '/Applications'

Fixes #235

Thanks! Reworded the commit a little and merged it in bc5560a


Dockerfiles to support the -m docker commit equivalent (feature request)

We can not add description to our containers when we create them with docker build

closing as covered by https://github.com/docker/docker/issues/2167

i mean duplicate


Disconnect or return ack if unauthorized publish

Addresses #290.

Thank you very much @thedevop - your expertise is greatly appreciated as always! 💯


Database resources error after upgrade to 0.6.2

Provider Version
0.62.0
Terraform Version
1.4.5
Describe the bug
After upgrading the Terraform plugin from version 0.56.2, I get the following exception.
 Error: unable to scan row for SHOW DATABASES LIKE 'TEST' err = missing destination name resource_group in *[]snowflake.Database
│
│   with snowflake_database.databases["TEST"],
│   on databases.tf line 51, in resource "snowflake_database" "databases":
│   51: resource "snowflake_database" "databases" {
│
╵

Removing the state and re-importing doesn't help.
Terraform used to generate the databases:
locals {
  databases = {
    # Environments
    "PRODUCTION" = {
      data_retention_time_in_days = 14
    }

    "TEST" = {
      data_retention_time_in_days = 1
    }
}

resource "snowflake_database" "databases" {
  provider = snowflake.sysadmin

  for_each = local.databases

  name                        = each.key
  comment                     = lookup(each.value, "comment", null)
  data_retention_time_in_days = lookup(each.value, "data_retention_time_in_days", 3)

  lifecycle {
    prevent_destroy = true
  }
}

We've been struggling ever since version 0.56.3 to upgrade because of issues with the databases. Versions previous to 0.62.0 remove all databases from the state. When trying to re-import them we get the following:
snowflake_database.databases["TEST"]: Refreshing state... [id=TEST]
╷
│ Error: Cannot import non-existent remote object
│
│ While attempting to import an existing object to "snowflake_database.databases[\"TEST\"]", the provider detected that no object exists with the given id.
│ Only pre-existing objects can be imported; check that the id is correct and that it is associated with the provider's configured region or endpoint, or
│ use "terraform apply" to create a new remote object for this resource.

i would like to schedule a call with you so you can show me what you are experiencing because for the life of me I am not able to replicate the error you are seeing. Using the config you provided I had no problem applying, planning, and re-importing the resources.
$ terraform apply
snowflake_database.databases["TEST"]: Creating...
snowflake_database.databases["PRODUCTION"]: Creating...
snowflake_database.databases["TEST"]: Creation complete after 1s [id=TEST]
snowflake_database.databases["PRODUCTION"]: Creation complete after 1s [id=PRODUCTION]

Apply complete! Resources: 2 added, 0 changed, 0 destroyed.

$ terraform state rm snowflake_database.databases["TEST"]
Removed snowflake_database.databases["TEST"]
Successfully removed 1 resource instance(s).
$ terraform import snowflake_database.databases["TEST"] "TEST"
snowflake_database.databases["TEST"]: Importing from ID "TEST"...
snowflake_database.databases["TEST"]: Import prepared!
Prepared snowflake_database for import
snowflake_database.databases["TEST"]: Refreshing state... [id=TEST]
Import successful!
The resources that were imported are shown above. These resources are now in
your Terraform state and will henceforth be managed by Terraform.

@mlavaert (or anyone with the same exact issue) maybe a long shot but ... are you getting this error  err = missing destination name only perhaps when running the plugin against replicated databases ?

our db was not replicated and it was removed from state

I'm having a similar issue with replicated databases. We are getting the error Error: unable to scan row for SHOW DATABASES LIKE '<database_name>' err = missing destination name firewall_configuration in *[]snowflake.Database, we have not made any changes to our snowflake terraform resources - we are seeing this error on the plan when updating resources with a different provider (AWS).

I am getting this error as well with provider version 0.63.0.
Error: unable to scan row for SHOW DATABASES LIKE 'EUROWIND' err = missing destination name resource_group in *[]snowflake.Database

Also facing the same error as @marcincuber with the same 0.63.0 provider version & Terraform v1.4.6

#1758 should fix this issue

@mlavaert (or anyone with the same exact issue) maybe a long shot but ... are you getting this error err = missing destination name only perhaps when running the plugin against replicated databases ?

We're not using replicated databases.

Just testing snowflake-labs/snowflake v0.63.0 from 0.59
│ Error: unable to scan row for SHOW DATABASES LIKE 'AIRBYTE' err = missing destination name firewall_configuration in *[]snowflake.Database
│
│   with snowflake_database.this_db["AIRBYTE"],
│   on main.tf line 1, in resource "snowflake_database" "this_db":
│    1: resource "snowflake_database" "this_db" {
│

If I run show databases like 'AIRBYTE', here is my output
[
  {
    "created_on": "2022-06-08 07:56:18.819 +00:00",
    "name": "AIRBYTE",
    "is_default": "N",
    "is_current": "N",
    "origin": "",
    "owner": "SYSADMIN",
    "comment": "",
    "options": "",
    "retention_time": "1",
    "firewall_configuration": null,
    "share_events_with_provider": ""
  }
]

I do not get the ResourceGroup @mlorek is referring to, but I have the firewall_configuration from @christyvp
Is that some kind of feature available only for some customers?

These issues have been resolved in the 0.64.0 release.

@mlavaert I have a simiar issue as @christyvp
Even on 0.64.0 I get the following error when creating a database from a replica
unable to scan row for SHOW DATABASES LIKE 'TEST_DB' err = missing destination name kind in *[]snowflake.Database

still in 0.65.0 https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/1836

Not sure I am allowed to comment it here but for the sake of other people I will.
We had the same issue today and I opened a ticket with snowflake support. Although they say that terraform provider is not covered by support they gave me the following feedback.

Looking at similar issues, it appears that the issue is related to the most recent release and how it affects the SHOW DATABASES output
Could you please attempt to roll back this behavior change?
select system$disable_behavior_change_bundle(‘2023_03’);

Running that made the plan/apply go back to work. I reckon this is something that has to be addressed on provider eventually.
Although that makes it work for us, it also let us out of any sort of improvement that was released on that bundle.

Hi everyone, this issue should have been fixed in the v0.66 release from last week. Can you upgrade to v0.66.1 and confirm whether that solves your problem?

@douglaszickuhr you'll probably want to re-enable the 2023_03 bundle once you've upgraded, I suggest getting in touch with support to confirm the process.

Hey @sfc-gh-ngaberel thanks for that - we're actually stuck with 0.63.0 because of some breaking changes introduced to masking policies. Once we've managed to get over that we will plan to upgrade it. For now we'll have to stay on v0.63.0


Added IT lang files

Not a real coder but a microG user that want to thank you for this project in its small way. I added translated strings for IT localization, hope there are not missing files or other things to be added, in that case, please, let me know.
Sorry for the inconvenience with the request, hope everything is fixed now.

@Pdevo You can actually keep a PR even if you need to do changes. Normally you create a PR branch in your repository. Every commit into that branch will automatically included into the PR here.
Thank you for contributing!

@chris42 Thank you for your suggestion, not so confident with github. Hope everything is fine now.

@Pdevo you can't break much. ;-D
@mar-v-in will need to review the PR and merge it. I am not sure about his timeline and what is still on his list after he did the refactoring.

In my opinion you should write the translation using a "more direct" form, for example substituting "Toccate qui" with "Tocca qui", "utilizzate" with "utilizza", "vostro dispositivo" with "tuo dispositivo".
A direct form makes the content clearer and I think it's better for a configuration screen.

@ErnyTech I generally think is better not to use such direct forms when the language allow to be more clean and polite (let's say also generic), at least this sounds better in italian language, but it is just a personal opinion ;) I can change that if you find it better

@ErnyTech ok, you got your point, I fixed that

Thanks @Pdevo, this will make it in the next release. :smile:


invite to slack? ;-)

Hey guys! I I got super excited when a friend of mine sent me a link to this project.
I am the found of FuseBox and I am currently working on a brand new version v4 that is (according to my benches) faster than than any bundler there is by the amount of gratitude) and I am looking at alternatives to transpiling TypeScript since the official one is very slow.
As I understood, you do support the latter? Since that's what FuseBox is build on top of. Does it support source maps? How fast is it comparing to the TypeScript compiler?  Can I feed an AST to generate code?
Anyway, I have lots of question, but can't get in. Please send me and invite ;-)
P.S and sorry for the off topic issue ;-)

It's okay, and we support compiling typescript, but we does not support type checking or cross-file compilation like const enum at now.
Slackin link: http://swc-slackin.herokuapp.com


tests #95 error

Python-3.7-PySide6-windows-latest
Run coverage run --source=src --append --branch -m unittest discover -s tests
  coverage run --source=src --append --branch -m unittest discover -s tests
  shell: C:\Program Files\PowerShell\7\pwsh.EXE -command ". '{0}'"
  env:
    pythonLocation: C:\hostedtoolcache\windows\Python\3.7.9\x64
    PKG_CONFIG_PATH: C:\hostedtoolcache\windows\Python\3.7.9\x64/lib/pkgconfig
    Python_ROOT_DIR: C:\hostedtoolcache\windows\Python\3.7.9\x64
    Python2_ROOT_DIR: C:\hostedtoolcache\windows\Python\3.7.9\x64
    Python3_ROOT_DIR: C:\hostedtoolcache\windows\Python\3.7.9\x64
    TEST_QT_MODULE: PySide6
    PYTHONPATH: src

...

======================================================================
ERROR: test_wrapped_static_amethod (test_slot.TestSlot)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "D:\a\qtinter\qtinter\tests\test_slot.py", line 482, in test_wrapped_static_amethod
    result = self._run_once()
  File "D:\a\qtinter\qtinter\tests\test_slot.py", line 234, in _run_once
    with using_asyncio_from_qt():
  File "c:\hostedtoolcache\windows\python\3.7.9\x64\lib\contextlib.py", line 112, in __enter__
    return next(self.gen)
  File "D:\a\qtinter\qtinter\src\qtinter\_contexts.py", line 36, in using_asyncio_from_qt
    loop.start()
  File "D:\a\qtinter\qtinter\src\qtinter\_base_events.py", line 284, in start
    self._qi_loop_startup()
  File "D:\a\qtinter\qtinter\src\qtinter\_base_events.py", line 302, in _qi_loop_startup
    self.__notifier = _create_notifier(self)
  File "D:\a\qtinter\qtinter\src\qtinter\_base_events.py", line 57, in _create_notifier
    return _QiNotifierObject(loop)
AttributeError: __init__

----------------------------------------------------------------------
Ran 59 tests in 0.163s

FAILED (errors=30, skipped=7)
Error: Process completed with exit code 1.

There are 30 error of the same cause AttributeError: __init__.

This is a bug with _create_notifier() in _base_events.py. The function defines a class with a global name. However, the global statement documentation says:

Names listed in a global statement must not be defined as formal parameters, or as targets in with statements or except clauses, or in a for target list, class definition, function definition, import statement, or variable annotation.

Fixed in commit 37e1edb.

Commit 37e1edb didn't fix the issue because the error was reproduced consistently in test runs after the commit:
tests #96: Python-3.7-PySide6-windows-latest
tests #97: Python-3.7-PySide6-windows-latest
tests #98:
Python-3.7-PySide6-windows-latest

Same error occurred again, still with Python 3.7 + PySide6 but on macOS:
Python-3.7-PySide6-macos-latest

Added adhoc workflow to investigate this problem. adhoc runs 2 to 8 show some interesting results.
adhoc#2 passes but adhoc#3 fails. The only difference is that adhoc#3 runs test.test_asyncio as a prior step before running tests. It's unclear why this makes a difference.
adhoc#4 to 7 are a series of runs with increasing logging. adhoc#7 shows that when _QiNotifierObject is first created, _QiNotifierObject.__init__ is <function _create_notifier.<locals>._QiNotifierObjectImpl.__init__ at 0x108dbf0e0>. But when AttributeError is raised later, _QiNotifierObject.__init__ has changed to <method-wrapper '__init__' of Shiboken.ObjectType object at 0x7f8258c9d340>.
This change is unexpected. It might be related to two factors: (1) AttributeError starts after a KeyboardInterrupt test, and (2) _QiNotifierObjectImpl is defined as a function-local class. This suggests that SIGINT might have broken the internal state of a locally-defined QObject-derived class.
adhoc#8 attempts a workaround of this by making _QiNotifierObject a module-level (instead of function-local) class. To keep lazy resolution of Qt binding, a _QiObject class is added to qtinter.bindings.
This appears to have fixed the issue.


Added nuspec and migration pcl to .net standard

Have added nuspec and migration to .net standard 2.0. Needs to be built and tested on iOS to ensure the FormsControl.Touch is being included in the nuget package for iOS.
Resolved obsolete AppCompatAnimationNavRenderer constructor causing exception Android.Content.Res.Resources+NotFoundException: Resource ID #0x7f0a000d

Please run past changes made, let me know if any issues exist.


Fix for missing self sigms

Thanks. I've implemented a slightly different fix that shows which set of sigmas is used. The two sets of sigmas are to support a future option to switch among them.


(#450) Render debug details for platforms

fix #450

@ozy thanks for addressing my concerns! I'll check it out on the next Nothing stream.

@ozy looks ebic! :+1: Thank you for the contribution!


feat(extensions): add --install-as flag to dfx extension install command

Description
Continuation of https://github.com/dfinity/sdk/pull/2968. Allows to install extension under different name, which should help with naming conflicts resolution.

Checklist:

[x] The title of this PR complies with Conventional Commits.
[ ] I have edited the CHANGELOG accordingly.
[ ] I have made corresponding changes to the documentation.

Actually, there have been several occasions where I worked with extensions and wished this was already merged. I'd use that for installing sns extension, even when dfx sns command already exists (i.e. dfx extension install sns --install-as sns-dev) - very useful for testing things. I definitely don't see it as an unnecessary complication

Okay, then let's try some DRY so that there will be fewer differences to maintain.


Add annotation support for buttons

Don't merge quite yet as I am still testing it. This update adds support to callback buttons right within the class where you want them defined. It is pretty cool if I do say so myself!

Hello Technosword!
I'm thankful for all your time and efforts!
However, the reason I didn't add callback button support is because of the number of exploits there have been (for example button id spoofing), and I'm not sure that they have been resolved yet. That said if you wish to maybe not push it to stable, but to another branch, called buttons or something, with a note in readme or something that there is said branch, I'll gladly accept your PR (after you give me the go)
Best Regards,
Nyvil


⚠ Samsung has degraded performance

In e1273b0, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3160 ms

Resolved: Samsung performance has improved in cd3ea0a.


I need your help.. i want to hook DeleteFile API..

Hi, there..
I want to use DeleteFileA, DeleteFileW API...
I made some of codes.. could you check this code?
(i can not find what is the problem...)
Main.txt

I tried that code and it doesn't error out. What is happening for you?
If it is just not being called, then perhaps check with API Monitor or similar to see if that call is actually used in the target application.
I used this with Notepad++ and the DeleteFileWHook successfully fired when I closed it and it tried to cleanup some temporary files.


Feature Request: Allow the entire side panel to scroll (Usability Enhancement)

The side panel is a little difficult to use on fixed mode, or even small resizable, once enough items have been added. This could be resolved by allowing the entire panel to scroll and not just the trips section.

Thanks for the suggestion! I will look into it.

I was going to report the same thing. I got a screenshot of what happens once enough different items have been added.
Since the top section isn't scrollable, the bottom scrollable trips section just keeps getting smaller and smaller until it's unusable.


Feature Request: Maintenance Mode

Allow for a maintenance mode config option that will stop serving web traffic and send it to a static maintenance page.
ex:
$ deis maintenance:set_page http://s3.aws.com/mybucket/mypage.html
$ deis maintenance:on
$ curl http://mysite.com
<html>
We are under maintenance!!!
</html>

Heroku's CLI docs for maintennance mode are here, for reference. I like this because when deploying you often have to migrate your app, which requires stopping the web server. Giving a nice error page is much better than the a 404 error.

Big +1 for this.  I don't see this being a big effort either.

:+1:
FYI for a client, I solved this by creating a maintenance application which is an Nginx serving the maintenance page for any request and then did:
deis domains:remove www.example.com -a example && deis domains:add www.example.com -a maintenace


Replenish new DeploymentDraft class, change the version number of Common to 1.0.17.

Replenish new DeploymentDraft class, change the version number of Common to 1.0.17.

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


Swagger Changes for Access Connector Public Preview

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[x] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
[Subbu] Already service is deployed and in Public Preview starting July 4th
If updating an existing version, please select the specific language SDKs and CLIs that must be refreshed after the swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No refresh required for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following appy to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you are using OpenAPIHub to initialize the PR for adding a new version. More details, refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If any of the following scenarios apply to the PR, request approval from the Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in a stable version
[ ] Removing properties in a stable version
[ ] Removing API version(s) in a stable version
[ ] Updating API in a stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.
These changes are already being reviewed and approved by in private repositories, please check the following PR
https://github.com/Azure/azure-rest-api-specs-pr/pull/6255

/azp run

/azp run

/azp run


Material files, aves gallery, qksms, Tachiyomi, Sony camera, Xbox; Link Amazon india.

https://github.com/LawnchairLauncher/lawnicons/issues/207
https://github.com/LawnchairLauncher/lawnicons/issues/61
https://github.com/LawnchairLauncher/lawnicons/issues/166
https://github.com/LawnchairLauncher/lawnicons/issues/215

Is the sony camera not linked yet?

Sorry for keeping you waiting! Iʼd suggest making the logo in the Tachiyomi icon a bit smaller (tachiyomi.zip). Additionally, because Sony Camera is a system camera app, Iʼd suggest linking the com.sonyericsson.android.camera package to @drawable/camera.

How about sony camera? Is it not linked yet?

Sorry. I forgot this pull request. If anyone want to add these svgs, feel free to do. I don't use themed icons and lawnchair now.


Correct the button name

Why:
Closes #7
What's being changed:

README.md
Insert a suggestion → Add a suggestion
Check off the following:

[x] For workflow changes, I have verified the Actions workflows function as expected.
[x] For content changes, I have reviewed the style guide.

I have added some commits to clarify the instruction.
Cf. discussion comment

I have added the icon of the button in my commits on July 8th.
Are there any problems with it?

Yet it is still difficult to locate that tricky button - Can you add the icon of the "Add a suggestion" button to the README file ? Or even a small screenshot where it is found ?

Gentle reminder
@dihydroJenoxide I would be grateful if you could review this pull request as soon as possible.


Errors from an extension appear in Output instead of JS console

VSCode Version: 1.4.0
OS Version: Windows 10 Version 1607

I'm using an extension that outputs errors fairly frequently (https://github.com/vshaxe/vshaxe). For some reason, these errors go the Output panel for me in a Haxe channel:

This is quite strange, because for everybody else using this extension I've talked to, these errors go to the JS console of the developer tools. This is also the case on my Windows 7 laptop, where I did a clean install of VSCode + the extension (same versions) for testing:

What makes this especially annoying is that when I hide the Output panel, it pops back up automatically when a new error appears and steals focus.
I have tried the following steps to solve the issue on my Win10 PC (to no avail):

reinstall VSCode
remove all extensions except vshaxe
delete %AppData%/Code
start VSCode from the command line with code --user-data-dir=<some-random-directory>

The only reasonable explanation I have is that something must be different about my VSCode installation, maybe some leftover configuration from earlier VSCode versions or anything that would make it behave differently in this regard. Are there other places where VSCode stores configs?

It looks like this was caused by a different vscode-languageclient version on my PC (2.4.2-next.5 vs 2.3.0). When building the extension with 2.3.0, the behavior is consistent with everyone else's.

Specifically https://github.com/Microsoft/vscode-languageserver-node/issues/55.


More specific about CMG supported task sequences

You can run from CMG TSes which don't use WinPE and are deployed with an option: "download all content before starting ts"
Summarize the change in the pull request title
Describe your change, specifically why you think it's needed.
Fixes #Issue_Number (if necessary)

@PanuSaukko : Thanks for your contribution! The author, @aczechowski, has been notified to review your proposed change.

Thanks @PanuSaukko 😃
#sign-off
Once merged, this should be live with the next daily publishing process. Right now that could be today (Tuesday, 26 February) around 11:00 Pacific time. No further action needed from you.


Canvas blend mode compatibility not properly being detected

I opened up a project that relies on canvas's multiply blend mode and noticed that it suddenly wasn't working anymore. After digging around a bit, it looks like most canvas blend modes aren't working in CE in the latest Chrome & FF on Windows 10. (Note: canvas blend modes work fine in Phaser 2.6.2. WebGL blend modes appear fine in CE.)
Here's a codepen example (linked against 2.7.5 via jsdelivr): http://codepen.io/mikewesthad/pen/JWQmdV. Two images are being drawn. The first with the default blend mode (source-over), and the second with multiply. The second image gets drawn with source-over instead of multiply.
What I see:

What I should see:

I tried extracting the code that checks whether multiply blend mode is available (source) and checking it in that codepen test. It appears to properly detect that multiply blend mode is available. A weird quirk - if I force the compatibility check before creating the images, the blend mode seems to work fine... So, do the base64 images need to use onload before they can get pixels and properly check compatibility?

http://codepen.io/samme/pen/gmNJar?editors=0010

Ah, nice visual @samme.
I think my hunch is correct - base64 images aren't loaded synchronously in all browsers. See this codepen: http://codepen.io/mikewesthad/pen/zZgovJ?editors=0010. It uses the Phaser code for checking for multiply blend mode in two ways: checking immediately and checking after img,onload events fire. The image pixels aren't available immediately in the latest Chrome/FF (though they are in Edge), so the multiply check fails unless it is done asynchronously.
I don't have time right now to look into a fix, but I imagine it requires waiting for img.onload events in Device.js and then tweaking initialized & onInitialized so that they allow the canvas check to be carried out asynchronously.

In Safari device.canUseMultiply is also false on first load (or force reload), true after.

#137 skips the pixel test and just checks if the browser accepts the multiply keyword.
If anyone has IE <= 11, please try it and verify that game.device.canUseMultiply is false (like before).


Dynamic empty text

Hoist P/R Checklist
Review and check off the below. Items that do not apply can also be checked off to indicate they
have been considered. If unclear if a step is relevant, please leave unchecked and note in comments.

[x] Up to date with develop branch as of last change.
[x] Ready for review (or open as a draft PR / add wip label).
[x] Added CHANGELOG entry (or N/A)
[x] Reviewed for breaking changes (add breaking-change label + CHANGELOG if so, or N/A)
[x] Updated doc comments / prop-types (or N/A)
[ ] Reviewed and tested on Mobile (or N/A)
[x] Created Toolbox branch / PR (link provided here, or N/A)
https://github.com/xh/toolbox/tree/dynamicEmptyText
Not suggesting we pull these changes in, but might be useful to see.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

@lbwexler what do you think about the use of @bindable here w/o an explicit setter? I'm on the fence re. adding a setter - more boilerplate code, but easier to discover given current (lack of) tooling around those annotations and IDE/doc hints.

Why doesn't bindable define a 'proper' ES6-style setter?

Looking at this with Peter now.  We discussed the fact that we use setXXX() and at this point are unlikely to be able to change that.

Regarding Anselm's question -- would be happy to use an explicit setXXX method rather than bindable in this case, both for the new option and the existing showSummary().
On a related note,  groupBy should be @observable.ref.

@lbwexler I fixed those few things.


Multiple errors in Learning Paths 0 and 10

Learning Paths: 00, 10
Exercise: 02 (LP 0), Exercise 3 (LP 10)
Task: 02 (LP 0), Task 3.2 (LP 10)
Step: 10 (LP 0), Step 1, part 6 (LP 10)
Description of issue
Repro steps:

LP 0, Exercise 02, Task 02, Step 10: The instruction to acquire a PowerApps per user plan isn't available in the Admin environment in Lab 0,

LP 10, Exercise 3, Task 3.2, Step 1, part 6: The status reason value on the Test Permit after completing the prerequisite steps should be set to Status Reason Value locked, but it's set to Active.

Lab 0: Looks like Power Apps per user plan license has been replaced by Power Apps Premium
Will update the lab step
Lab 10: The status reason for the Test Permit should be changed to locked after clicking the lock permit button. Check your code and the trace log


Rename Reflection library to RemoteInspection

In preparation for https://github.com/apple/swift/pull/62819, rename the existing Reflection library here to something more apt like RemoteInspection.
I didn't update any of the type names like swift_reflection_context_t, but if you all think that I should handle that here then let me know. I will have a corresponding llvm-project PR up soon.

@swift-ci please test

https://github.com/apple/llvm-project/pull/5894
@swift-ci please test


global menu: redesign burger menu

Closes #1409 and #1426
Please let me know if you disagree with the design/layout, and if there are any possible/hidden cases I have not considered!
Screenshots
Desktop

Tablet

Mobile

Desktop

Tablet

Mobile

Desktop

Tablet

Mobile

Desktop

Tablet

Mobile

Update: Removed excessive spacing between the link-items.
Screenshots


keeping buttons/images in same position in different resolutions

I'm using CMM 1.10.2-2.0.2.
I'm in the process of creating a new menu for a modpack I'm working on, and I've been decently successful at setting up custom images and buttons for it. However, when the game is launched or resized in a different resolution than what I'm using, the positions of the buttons and images shifts in odd ways. The main problem with this is that I need everything to stay where it is, or else it all looks very wonky. What's the best way to ensure that everything stays in the same relative spot no matter what the resolution?

I've got the background set up as a fill, and I tried futzing around with the alignments, after getting the hint to do so via other closed issues. Either I'm still missing something, or that's not the way to make this work. when the game window gets resized to other aspect ratios, the background fills in and changes resolution, but the buttons stay static based on their alignment, which throws them off depending on how the aspect ratio has changed.

You can make separate configs for each resolution and adjust it for that resolution.
Example,
mainmenu_large.json
mainmenu_auto.json

I did make those various versions, and I set them up to fit my native resolution, but I'm not sure how those will help the other resolution sizes that people tend to use.
One thing I did do that seemed to help a bit was I switched the background mode from "fill" to "stretch". Now, instead of keeping the aspect ratio of the background image and cutting off bits of the image to do so, it just stretches and shrinks the image to get the whole thing in the window. This has the net effect of keeping the buttons almost where they need to be, though they do still wander.
I wish the pages had better documentation of the alignments and what they do. Especially the custom alignments. I have no clue how they work, as the only explanation given is a pastebin link to an example config file, with no notation on what the custom alignment values do. This whole situation would be ideal if I could set the buttons to the prefered positions, set the background mode to stretch, and have the coordinates of the images and buttons adjust themselves according to how the window size has changed. I'd totally be up for using a percentage value instead of an absolute number for the position, if that's what it took. Something like, say, align a button or image with the bottom_center, then keep it 10% of height of the window up from the bottom, and 50% of the width from the left edge, or something.


Is S3 expiration lifecycle rule recommended to delete old Thanos data?

Hello,
I would like to put an S3 lifecycle rule that will delete old time series.
Is this the recommended approach to delete old Thanos data?
Thanks.
Amine

Nope, that's the job of Thanos Compactor


ExamineIndex config for search

How to setting custom field for index in Examine?
Now i can't search with field names in NestedContent field.
Please help this for search content in multi lang site!
Thanks you.

You'll have to hook into the Examine event GatheringNodeData and parse / seperate out the values manually and index them in custom fields. Unfortunately due to the complex nature of the data stored in Vorto / Nested Content, there is not automated way of doing this.


Prettier is not applied to files

The script npm run format only targets pure javascript files (aka *.js) but the source contains only .jsx files.

Sorry, I confused something. The script is fine.


New-NavImage.ps1: Invalid JSON primitive: failed (failed to get console mode for stdout: The handle is invalid)

I'm trying to create a new image, but it fails:
Invalid JSON primitive: failed.
at Get-BcContainerGenericTag, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\6.0.5\ContainerInfo\Get-NavContainerGenericTag.ps1: line 23
at New-BcImage, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\6.0.5\ContainerHandling\New-NavImage.ps1: line 345
Setting a break point and dividing line 23 ($inspect = docker inspect $imageName | ConvertFrom-Json) into
$inspect2 = docker inspect $imageName
$inspect = $inspect2 | ConvertFrom-Json
In $inspect2 I could see that the first line contained "failed to get console mode for stdout: The handle is invalid." which cause the JSON error.
This error again directed me to this thread: https://github.com/docker/compose/issues/8186
If I run the script from Powershell ISE, then I'm not getting the error. I only get it when running it from VSCode.
I assume that the problem started after I update Docker Desktop to version 4.27.2 (137060) yesterday.
I have tried specifying the VSCode setting suggested in the thread:
"terminal.integrated.env.windows": {
"COMPOSE_DOCKER_CLI_BUILD": "0"
}
But this did not work for me.
Have anyone else experienced this error?

Same problem as #3326

I can confirm that updating Docker Desktop to version 4.28.0 has fixed the issue.


Any plan to support Angular 6 schematics?

It would be great to see schematics like the material UI ones here:
https://material.angular.io/guide/schematics

I don't have any plans at the moment, however a PR would be welcome, if you were able?


Confusing warning about settings in ~/.config/mise/config.toml

Edit: looks like the main issue here is already fixed in 2024.1.12. This report also has a short discussion of settings in ~/.config/mise/settings.toml being silently ignored if they use a [settings] table header.

I think I moved my ~/.config/rtx/config.toml to ~/.config/mise/config.toml recently.
This file had the following content:
[settings]
experimental = true

With mise 2024.1.10 and 2024.1.12 I’m getting this warning printed at the start of the shell session:
mise [settings] inside of ~/.config/mise/config.toml is deprecated. A separate ~/.config/mise/config.toml file should be used instead.
mise Removing the [settings] section from ~/.config/mise/config.toml will remove this warning

Note how it's telling me to use a separate ~/.config/mise/config.toml file, which is the exact same file path.
This got me confused and I guessed that maybe the [settings] header had to be removed (I was not super familiar with TOML syntax, but apparently these are called table headers, and the header + key-value pairs it applies to are called a table, with “hash table” and “dictionary” given as synonyms, but not a section). So I changed the file contents to:
experimental = true

Which then triggered this error:
Error:   × error parsing config file: ~/.config/mise/config.toml
  ╰─▶ unknown key: experimental

My guess at this point is that the original warning is wrong, and should have said:
- A separate ~/.config/mise/config.toml file should be used instead.
+ A separate ~/.config/mise/settings.toml file should be used instead.

As a side note, if I create a ~/.config/mise/settings.toml without a table header or with a [settings] table header, I’m not getting a warning or validation error either way.
After testing with mise settings ls, it looks like the version with a [settings] table header results in the settings being ignored. Should that be a warning? If manual migration results in users having a settings.toml file with a [settings] table header, it will fail silently.

I saw the same warning (without the fix) after updating to 2024.1.12, but it looks like that was because it was still in the same shell session, and the code responsible for the warning is loaded once at the start of the session (at least with Fish), so I was still seeing the warning from version 2024.1.10.
I can confirm that the warning is correct in 2024.1.12 in a new terminal session.

Thanks for the heads up @rdonkin.

for some explanation of why this was done (since I doubt it's obvious), I needed to separate the [settings] block out of the config files.
Before, you could have a [settings] section inside of any config file. This was problematic because some of the settings actually affect how config files themselves—and which config files—get loaded. This made the code for loading everything quite complex and some settings were actually impossible to even use in some places.
Changing this logic so there is just one global settings file makes things far more consistent. It's also going to make it possible for me to integrate mise with this other project I'm in the middle of getting ready for an alpha launch of that will bring some features I really like.
You are correct that you don't need the [settings] header. I'll see if I can add that validation so it properly fails on invalid keys.

I considered automating this migration and creating the settings.toml on the user's behalf with the contents of config.toml. I decided against it because it would be complicated enough that it having a bug or 2 wouldn't be surprising. I also thought people might have config.toml as a symlink in a dotfiles repo but creating settings.toml might not also get added which could caue some confusion.


SENNA Interface for NLTK

This module is tested. However, It needs more documentation and polishing. Let me know your comments at this stage.

Thanks.  Yes, please add sphinx docstrings including some doctests, and the test runner to the bottom of the module (see e.g. hunpos.py).  Note that the NLTK tagger package should load without errors even if the senna tagger is not installed, and if someone tries to use it, they should get a helpful error message.


Payload is unable to resolve @floating-ui/react and react-onclickoutside packages.

Describe the Bug
Salam everyone.
I have created a Next.JS application using npx create-next-app@latest then initialised Payload using npx create-payload-app.
When I navigated to /admin an error occurred saying something like Unable to resolve @floating-ui/react.
When I have downloaded @floating-ui/react and react-onclickoutside package, everything worked just fine.
This is the commit for what I have done to fix it in my repository:
https://github.com/shawqicauses/shawqi-stack/commit/8140c6cd0d9bbfc54f8be0f6dc28d1fd8aad0cf0
Thanks.
Link to the code that reproduces this issue
https://github.com/shawqicauses/shawqi-stack
Reproduction Steps

Creating Next.JS application using npx create-next-app@latest
Initialising Payload CMS application using npx create-payload-app

Which area(s) are affected? (Select all that apply)
area: core
Environment Info
Next.JS version 15

What package manager? There is not a lockfile checked into your reproduction. Also please provide the output of payload info.

Also, either you're using an outdated create-payload-app, or create-payload-app wasn't used in creating your reproduction.
Whenever a project is made using cpa, an .npmrc file is written that contains legacy-peer-deps=true in it, which will handle automatically installing peer dependencies.
This is the file tree for blank
.
├── .env.example
├── .eslintrc.cjs
├── .gitignore
├── .npmrc
├── .prettierrc.json
├── .yarnrc
├── Dockerfile
├── README.md
├── docker-compose.yml
├── next.config.mjs
├── package-lock.json
├── package.json
├── src
│   ├── app
│   ├── collections
│   ├── payload-types.ts
│   └── payload.config.ts
└── tsconfig.json

i am having same issue , installed nextjs , run npx create-payload-app

i am having same issue , installed nextjs , run npx create-payload-app

Hey @SaidBahaDev
Run npm install @floating-ui/react react-onclickoutside and it should work.


SAML with Azure AD

do you know how to configure to use with Azure AD?

I added info in the readme pertaining to Azure AD.


Error: read ECONNRESET

Still getting this error in the last two days.
I've tried to uninstall and reinstall the plugin, but the error is still there.

I am having the same issue. I have tried every possible combo of server settings. yours was a year ago...did you work it out.

@RobWellington no. Now I have a dofferent error: "Max connections attempt reached". :(

I just installed this, and have tried multiple server accounts and combinations, Im using GoDaddy Hosting I thought id be cool to have an ssh via Brackets, but this has taken up way too much time to setup. Its 2017 and I guess this is not solved?


Upgrade Spring 5.3.29 -> 5.3.30

This PR contains the following updates:

Package
Type
Update
Change

Spring
import
patch
5.3.29 -> 5.3.30

Release Notes

spring-projects/spring-framework (Spring)
v5.3.30
Compare Source
:star: New Features

Optimize ClassUtils#getMostSpecificMethod #31100
Optimize whitespace checks in StringUtils #31069
Align validation metadata handling in PayloadMethodArgumentResolver #31056
Register an override for an existing adapter in ReactiveAdapterRegistry #31048
Make bean initialization deterministic for multiple @Autowired methods on same bean class #30994
Performance bottlenecks while creating scoped bean instances #30892

:lady_beetle: Bug Fixes

Possible classloader leak through incomplete clearing of annotation caches #31176
Spring LogFactory implementation deviates from original Apache LogFactory in terms of abstract method declarations #31167
Bean injection fails due to nullSafeConciseToString() invoking isEmpty() on a Map/Collection proxy #31156
SpelExpressionParser throws IllegalStateException instead of ParseException for invalid expression #31099
@DynamicPropertySource in @Nested test class cannot override dynamic properties from enclosing class #31085
TransactionalApplicationListenerMethodAdapter should find @TransactionalEventListener on target class method #31037
ScheduledAnnotationBeanPostProcessor: graceful shutdown should not interrupt currently running jobs #31020
Permgen memory leak due to ClassInfo caching in java.beans.Introspector on JDK 11/17 #31005
MethodIntrospector.selectMethods(?) fails to find methods in case of special bridge method arrangement #30907

:notebook_with_decorative_cover: Documentation

Fix documentation: Passing in Lists of Values for IN Clause does not work with JdbcTemplate #31229
Refine CORS documentation for wildcard processing #31168
Propagation REQUIRES_NEW may cause connection pool deadlock #31040
Clarify R2DBC ConnectionAccessor and DatabasePopulator exception declarations #30933
Doc: Avoid deadlock in @PostConstruct through SmartInitializingSingleton or ContextRefreshedEvent #30889

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade Spring 5.3.29 -> 5.3.30

See:
- https://github.com/spring-projects/spring-framework/releases/tag/v5.3.30
- https://github.com/spring-projects/spring-framework/compare/v5.3.29...v5.3.30


Docker fails to pull images

This issue affects all Mac OS X users that are experiencing Docker to get stucked on "Layer already being pulled by another client" message while pulling an image from Docker Hub.
azk uses a VM containing Docker 1.8 for handling containers on Mac OS X.
There's an issue regard this Docker version and the Docker Hub, so it must be updated.
Currently, the latest Docker version is 1.10. However, this version introduces deep changes on images ID, which means more changes on azk (we're working on it, but this can take a little longer).
That said, to solve this issue ASAP, a new ISO image for azk's VM must be created containing Docker 1.9.1.

/cc @hebersonbarbosa


Fixes UT convergence problem  (issue discussed in #1191)

Set the default for "use_memory_as_final_state" flag to False.
Removed the undefined/reduntant flag "use_memory_as_last_state".
NOTE: PR #1192 submitted by @rllin-fathom is basically addressing the same issue, but here I also added the condition of using memory as the final state only when the transition function is an LSTM (to avoid unintentional use of memory for other transition functions).

tl;dr
It was a just a bug and the performance of basic UT should be independent of this parameter.
use_memory_as_final_state is used when we use an LSTM as the transition function (use_memory_as_final_state=True  means we use the memory instead of the output of LSTM as the output of the final step). I, unfortunately, set it to True in another PR and it messed up the code for the basic universal transformer. Sorry for the inconveniences!

@MostafaDehghani - thanks for updating the issues! Merging this now.


Add support for HDFS only iceberg tables

Add support for HDFS only iceberg tables issue: #5571
Added a flag iceberg.hadoopMode to use Iceberg's ability to leverage the file system to store the list of schema and table.
The location of the root directory is specified using a configuration property hive.metastore.warehouse.dir in the the hive-site.xml file.
Some operations are not yet supported:
setSchemaAuthorization
renameSchema
getSchemaProperties
getSchemaOwner

closing


###bug report can not use adb logcat *:E

but if i change to bash it works will

i have find the answer here http://stackoverflow.com/questions/20037364/why-zsh-tries-to-expand-and-bash-does-not
but with my pool english
i really can not understand

* is a globbing character, which means zsh will use it to find files. For example, ls *.jpg will list all files ending with .jpg. In your case, you want to escape the * so zsh doesn't interpret that. You have two alternatives:

Use ' (single quotes): adb logcat '*:E'.
Use noglob and alias so you don't have to quote * every time: alias adb='noglob adb'. After that you can do adb logcat *:E without issues.

thank you
i have add setopt nonomatch to my zshrc
is there any potential problems?

That only disables the no matches found error message, so as far as I know, you are ok :+1:

it really works well,I just worry about potential problems


Upgrade to ~2.0.1

Currently this project is on the rc lets upgrade it to the release. For example in my app when I try to use this I get...
Uncaught Error: No Directive annotation found on ChasingDotsComponent

@jrgleason 1.1.0 is available ;) Please update.

Oops missed this and already issued a pull request, I will cancel it until I confirm the new version works

I also confirmed NPM was pulling the proper version when npm install --save


Does this still work?

I tried logging in but err is null and token is empty.

Yes, it works for me.
Try to get token for the app id in Graph API Explorer.

It no longer works for me. It's using an unsupported api to get at the access token, and that url is now returning an html page and not a json body response.


Enqueue a 'synchronize subscription' job when a new subscription is added

Currently, the user has to wait until the next synchronize happens, which can take quite some time.

Does the functionality exist to synchronise just one subscription, or is the only option to do all of them?


SecureString in Sign-NavContainerApp

Sign-NavContainerApp is a wonderful Cmdlet. One issue I had was an error message:
Sign-NavContainerApp : Cannot process argument transformation on parameter 'pfxPassword'. Cannot convert the "my password with spaces" value of type "System.String"
to type "System.Security.SecureString".
At line:4 char:18

-pfxPassword $MyPassword `

             ~~~~~~~~~~~

CategoryInfo          : InvalidData: (:) [Sign-NavContainerApp], ParameterBindingArgumentTransformationException
FullyQualifiedErrorId : ParameterArgumentTransformationError,Sign-NavContainerApp

I was able to make it work by converting the string to securestring before passing it into the Cmdlet. This worked:
$MyAppFile = "C:\ProgramData\NavContainerHelper\Extensions\Publisher_AppName_<IP_ADDRESS>.app"
$MyPfx = "C:\ProgramData\NavContainerHelper\Extensions\CodeSignCert.pfx"
$MyPassword = ConvertTo-SecureString "my password with spaces" -AsPlainText -Force
$MyContainerName = "MySandbox"
Sign-NavContainerApp     -appFile $MyAppFile
-pfxFile $MyPfx     -pfxPassword $MyPassword
-containerName $MyContainerName

This is very much by design.
All password parameters in NavContainerHelper for all purposes are securestrings.
If not - PowerShellGallery will fail validation.

Ah ok never mind then. Easy enough to send in a securestring. I've closed the issue


Error when hitting /!/zipper/create

Any ideas?

Fixed the issue with release v1.3.0.

Thank you! 🙏


Missing Parameters

[Enter feedback here]
Microsoft.Common.CurrentVersion.targets has this invocation:
    <GenerateApplicationManifest
        AssemblyName="$(_DeploymentApplicationManifestIdentity)"
        AssemblyVersion="$(_DeploymentManifestVersion)"
        ConfigFile="@(AppConfigWithTargetPath)"
        ClrVersion="$(ClrVersion)"
        Dependencies="@(_DeploymentManifestDependencies)"
        Description="$(Description)"
        EntryPoint="@(_DeploymentResolvedManifestEntryPoint)"
        ErrorReportUrl="$(_DeploymentFormattedErrorReportUrl)"
        FileAssociations="@(FileAssociation)"
        Files="@(_DeploymentManifestFiles)"
        HostInBrowser="$(HostInBrowser)"
        IconFile="@(_DeploymentManifestIconFile)"
        InputManifest="$(_DeploymentBaseManifest)"
        IsolatedComReferences="@(ResolvedIsolatedComModules)"
        LauncherBasedDeployment="$(_DeploymentLauncherBased)"
        ManifestType="$(_DeploymentManifestType)"
        MaxTargetPath="$(MaxTargetPath)"
        OutputManifest="@(ApplicationManifest)"
        OSVersion="$(OSVersion)"
        Platform="$(_DeploymentPlatformTarget)"
        Product="$(ProductName)"
        Publisher="$(PublisherName)"
        RequiresMinimumFramework35SP1="$(_DeploymentRequiresMinimumFramework35SP1)"
        SuiteName="$(SuiteName)"
        SupportUrl="$(_DeploymentFormattedSupportUrl)"
        TargetCulture="$(TargetCulture)"
        TargetFrameworkSubset="$(TargetFrameworkSubset)"
        TargetFrameworkProfile="$(TargetFrameworkProfile)"
        TargetFrameworkVersion="$(_DeploymentManifestTargetFrameworkVersion)"
        TrustInfoFile="@(_DeploymentIntermediateTrustInfoFile)"
        UseApplicationTrust="$(UseApplicationTrust)">

This document does not describe "LauncherBasedDeployment"

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

ID: 74c1a488-7180-9404-b96a-50420da0ff05
Version Independent ID: e5252643-45c5-f18e-2a1d-286ae8ef8cbb
Content: GenerateApplicationManifest Task - Visual Studio
Content Source: docs/msbuild/generateapplicationmanifest-task.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@atniomn -- thank you for your feedback.  Please consider these resources:

Visual Studio on Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@ghogen -- please look into this issue.

@atniomn -- thank you for your feedback.  Please consider these resources:

Visual Studio on Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@ghogen -- please look into this issue.

@atniomn Thanks for this issue - we need to add documentation for LauncherBasedDeployment. This feature supports .NET Core executables, which when built are initially named apphost.exe and then renamed to the final executable. See https://github.com/dotnet/msbuild/pull/5745.


Fix dynamic variables in Themes paths + Add new option "GAME MEDIAS DURING FAST SCROLLING"

index > 4 ? why 4?

I hesitated with 3 ;-)