Panning Issue After Renaming

Godot version
4.0dev (4fa6edc88)
System information
Ubuntu 20.04.2 LTS
Issue description
Panning of the 2D view does not work after attempting to rename node
Steps to reproduce

Open the a scene in the 2D viewing tab
Double click to rename any node under the scene tab
click into the 2D view directly
attempt to pan the view through middle-clicking or using the pan tool

Refocusing the editor window will fix it temporarily
Minimal reproduction project
Panning Issue.zip

Can't reproduce on Windows 10.
When the name is being edited, middle click will close the edition, but after releasing and pressing the MMB again the panning will happen normally.

Possibly related to #72074.

Reproduced on Fedora, Gnome on XORG.
I reproduced with number fields in visual shaders, and renaming nodes too.

Please try the latest master if you haven't, this may be related to what #72370 fixed.

Fixed for me :partying_face:

I tried it yesterday and it seemed to work, but I'm experiencing the same issue again now

I've got a similar issue on Linux.
If I exit renaming by clicking outside the LineEdit:

I can't interact with scrollbars.
LineEdits and TextEdits can be focused and edited, but the carets in these controls won't be shown.

It works fine if I exit renaming by hitting enter or ESC.
It's also intreseting that I can only reproduce this with a custom build. The offical beta16 works fine, while custom built 518b9e5801a19229805fe837d7d0cf92920ad413 has this problem. (target=editor tests=yes module_text_server_fb_enabled=yes dev_build=yes use_asan=yes use_lsan=yes linker=mold)


change the default index pattern on elasticsearch

Not sure when, but kibana has defaulted to "filebeat*" since at least 2018 from what I can see for the logs index pattern. Should the default in this project become filebeat instead of logstash ?


Replace calls to deprecated yaml.load(input)

Replace calls to yaml.load(input) [removed] with yaml.full_load(input) in base_options.py
Fixes https://github.com/SsnL/dataset-distillation/issues/51

Thanks!

Happy to help!


Modularize the Code

I break the code into functions or classes to make it more modular. This makes it easier to test, understand, and maintain. For example, you can have functions for checking the alarm input, converting time formats, and setting the alarm.

Go for it no need to open an issue


v2.5.11 gets clobbered by uglifyjs-webpack-plugin run through webpack

Version
2.5.11
Reproduction link
N/A
Steps to reproduce
While running UglifyJS Webpack Plugin (https://www.npmjs.com/package/uglifyjs-webpack-plugin), there is an error thrown for a function in vue. The error reads Uncaught TypeError: s is not a function. In this instance s is referring to the isPlainObject function, found in the normalizeInject method.
What is expected?
Uglify should be able to correctly minify Vue
What is actually happening?
Vue 2.5.11 should be successfully minified by webpack

Locking our build back to 2.5.10 fixed the uglify issue

Just tried a fresh build from vue-cli with 2.5.11 and everything seems to work fine. Please provide a reproduction if possible.


Update README.md

@jxxcarlson  I tried to make the README more friendly for those who don't use/know Lamdera and to provide more general examples. I kept the points from the previous README that still seem relevant to me.
Feel free to add or remove anything!

Hi Matt,
I'm merging this now and will look at the changes locally.


Support node 4, node 5.

This line is preventing installs on node 4, the current LTS version of node. Also, node 5, the latest stable version.

This has been fixed. Will changse publish to npm soon.


Docstrings to public methods and functions.

Created with OpenAI davinchi-text-003 and validated/fixed by myself.
Close #90
On branch feature/90-docstrings_public_functions
Changes to be committed:

modified:   quinn/dataframe_helpers.py
modified:   quinn/dataframe_validator.py
modified:   quinn/extensions/column_ext.py
modified:   quinn/extensions/spark_session_ext.py
modified:   quinn/functions.py
modified:   quinn/transformations.py

Thanks for adding these!!


Remove the skip-tz-utc configuration item

dumpling does not have this parameter, the current behavior is skip-tz-utc: true
This variable exists in other places in the V2 document, please remove, more quantity
Please conduct technical review
@lance6716

First-time contributors' checklist

[ ] I've signed Contributor License Agreement that's required for repo owners to accept my contribution.

What is changed, added, or deleted? (Required)

Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version, including v3.0 changes)
[x] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

/cc @lichunzhu

/lgtm

@glkappe
Please also help update these two documents:
https://docs.pingcap.com/zh/tidb-data-migration/stable/task-configuration-file-full#完整配置文件示例
https://docs.pingcap.com/zh/tidb-data-migration/stable/usage-scenario-shard-merge#迁移任务配置

/label needs-cherry-pick-2.0
/translation doing
/status PTAL
/cc @CharLotteiu

@lance6716 done, Please do a technical review

/merge

/hold

/merge cancel

@glkappe
Please also help update these two documents:
https://docs.pingcap.com/zh/tidb-data-migration/stable/task-configuration-file-full#完整配置文件示例
https://docs.pingcap.com/zh/tidb-data-migration/stable/usage-scenario-shard-merge#迁移任务配置

I think we'd better delete the configuration in these two docs too.

I think it is already done? See https://github.com/pingcap/docs-dm/pull/631 and https://github.com/pingcap/docs-dm/pull/632.

/unhold

ok... I thought these will be done is one PR.

/merge

/remove-translation doing
/translation done


debian install community server with dependency issues

hello,
I'm trying to install the onlyoffice community server in my debian box.
I'm using the doku on https://helpcenter.onlyoffice.com/server/linux/community/linux-installation.aspx which leaves me with several problems where I could not find a solution.
Here the output of my install:
apt-get install onlyoffice-communityserver
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 onlyoffice-communityserver : Depends: mysql-server but it is not installable
                              Depends: mysql-client but it is not installable
                              Depends: mono-webserver-hyperfastcgi but it is not going to be installed
                              Depends: elasticsearch (= 6.5.0) but it is not installable
E: Unable to correct problems, you have held broken packages.

Explanations:
I already installed owncloud with mariadb and apache2.
Also installed the onlyoffice-documentserver with postgresql and ngnix using apache2 as https-proxy.
cat /etc/debian_version
10.3
Why is the installer trying to install mysql-server and mysql-client, although mariadb is installed and supported?
mono-webserver-hyperfastcgi also has dependency problems:
apt-get install mono-webserver-hyperfastcgi
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 mono-webserver-hyperfastcgi : Depends: libevent-2.0-5 (>= 2.0.10-stable) but it is not installable
                               Depends: libevent-pthreads-2.0-5 (>= 2.0.10-stable) but it is not installable
E: Unable to correct problems, you have held broken packages.
root@tsg-server:~/onlyoffice# apt-get install libevent-2.0-5
Reading package lists... Done
Building dependency tree
Reading state information... Done
Package libevent-2.0-5 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'libevent-2.0-5' has no installation candidate
root@tsg-server:~/onlyoffice# apt-get install libevent-pthreads-2.0-5
Reading package lists... Done
Building dependency tree
Reading state information... Done
Package libevent-pthreads-2.0-5 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'libevent-pthreads-2.0-5' has no installation candidate

elasticsearch also is not installable as described in the doku:
apt-get install libevent-pthreads-2.0-5
Reading package lists... Done
Building dependency tree
Reading state information... Done
Package libevent-pthreads-2.0-5 is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'libevent-pthreads-2.0-5' has no installation candidate
root@tsg-server:~/onlyoffice# LANGUAGE=en apt-get -y install elasticsearch-6.5.0.rpm
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package elasticsearch-6.5.0.rpm
E: Couldn't find any package by glob 'elasticsearch-6.5.0.rpm'
E: Couldn't find any package by regex 'elasticsearch-6.5.0.rpm'

An alternative as in https://computingforgeeks.com/how-to-install-elasticsearch-6-x-on-ubuntu-18-04-lts-bionic-beaver-linux/ has a different name and version: elasticsearch-oss 6.8.10

Hello @markusknappfr  this instruction not up to date, we recommend use our script for installation. But why you need to install onlyoffice community server with owncloud?

The script did not help. The first problem of the script is, that it wants free ports that I use for owncloud: 80, 443 used by apache. And I had to manually cleanup my onlyoffice document server.
The script is very radical. It uninstalls mariadb, which destroyed my owncloud installation. It then tries to install mysqlserver, but that didn't work so the script breaks. Sorry, didn't get the error message.
Now I had an half-installed community server. Correcting by running the script won't work, because of the ports and because it finds the previous installed components. I had to manually uninstall everything that the script has installed.
And to repair my owncloud, I had to install mariadb again and attache the database, reconfigure the apache ports.
I don't understand, why you use so many different components (mysql, progresql, ngnix) for onlyoffice and ignore th default components coming with the distribution (mariadb and apache for debian). This makes it hardly impossible to install it with other things.

Our script install ONLYOFFICE Community or Enterprise Edition, that need all of this components. So why you need onlyoffice community server if you use owncloud? Maybe you need to go in this repo?

owncloud does not have an CRM or project management.
and to the components: why two different sql servers for one solution?

And how you want to configure it all at once? You want setup onlyoffice on the different port?
Onlyoffice community server uses mysql, document server uses postgresql.

Yes, different ports.
I had it running with owncloud (apache2, mariadb) and onlyoffice document server (postgresql, nginx on port 81, apache2 as https reverse proxy). The docu for documents server worked very well: https://helpcenter.onlyoffice.com/server/linux/document/linux-installation.aspx
My trouble with the community server is described in my first post. I would be really happy, if these dependency problem would be solved somehow.

Having the same issue here.


Project is not supporting Tf 2.2.0 or higher version

Hi @SimonKohl, did you change your project to support the higher version of tf. Actually, I am working on a project and cannot downgrade my tf 2.2.0 (due to conflict with other work). Any help from your side will be really appreciated. Hope to get a reply from you soon.

Also, the tensorflow 1.15 which is in the requirements file is not compatible with the tensorflow-probability package. How did you came around this limitation?


added 1001: template for Go

Search for pull from upstream. If you pull from this repo to your repo, you will get your changes on the main branch


query performance guide

How to control queries with the limits in the conf file.
How to precalculate common or expensive queries with CQs
How to make queries less intense:

shorten time range
larger buckets
positive match instead of negative match
ask Ben J for more ;)

link to https://docs.influxdata.com/influxdb/v0.13/troubleshooting/query_management/ for the first topic

another one: tags much faster than fields
lower precision writes = faster queries (not sure how much, though). i.e. nanosecond timestamps take longer than second timestamps
Questions for Ben:

I assume positive matches are faster than negative matches, i.e. given 26 possible tag values, how much faster will SELECT * FROM foo WHERE tag =~ /A/ be compared to SELECT * FROM foo WHERE tag !~ /[B-Z]/?
for regexes, is there a performance difference between =~ /A|B|C|D|E|F/ and =~ /[A-F]/?
Is there a performance difference between explicitly calling every column the SELECT clause or just using *?
How much does each additional AND condition add to the processing time? Is it linear, logarithmic, exponential?
How much does each additional OR condition add to the processing time? Is it linear, logarithmic, exponential?
at what point do more AND|OR conditions get ridiculous? 5? 10? 50?
I would assume that WHERE x =~ /[0-9]/ is faster than WHERE x=0 OR x=1 OR x=2...x=9 even though the two are logically equivalent. Is that assumption true?
Does adding ORDER BY ASC have any performance impact?
Does ORDER BY DESC have significant impact? What if the return is 1 million points?
The fewer shard files queried, the better. But, let's say I have 1 million points scattered over a day in my_day and 1 million points scattered over 1 year in my_year. What would be the relative query speed difference between SELECT * FROM my_day and SELECT * FROM my_year?
can a single query ever use more than one CPU core?
I assume that long strings as field values can take time to decompress, and will eat up more RAM when queried as they are uncompressed when returned. True?
is there any way to change the output to make queries faster? e.g. an unsorted option, or a non-JSON return format?
does pretty add much to the query response time?
I assume most aggregations are fast because they don't need to sort the data, and most transformations are slower because the data must be sorted. I assume most selectors are the same basic speed as aggregations. More or less true?

@benbjohnson some query engine questions for you, if you have answers for any of them.
@mark-rushakoff some query performance testing ideas for your copious free time ;)


chore(main): release 1.1.3

:robot: I have created a release beep boop
1.1.3 (2024-03-22)
Bug Fixes

BezierCurve: 方法类型声明错误 (fd5ef0b)
CircleMarker: 方法类型声明错误 (790bbdf)
InfoWindow: 方法 setSize 参数声明错误 (38dace0)
Polyline: 方法 getPath 返回值错误 (e69daed)
Rectangle: 方法 contains 返回值类型错误 (2279053)
Text: 样式类型声明错误 (22c5f92)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/xyy94813/amap-jsapi-v2-types/releases/tag/v1.1.3 :sunflower:


Using tiles.setTileAt does not update the tile in the 3D rendered space

Recently I was helping someone with a game and they were looking to change the texture on a tile when interacting with another element in-game. We got as far as changing the texture using the [set tile at location] block to change the texture of the tile but it only changes in the tilemap mode and not in the 3D render mode. Would it be possible to detect the change in the tilemap and update the 3D rendered tilemap accordingly?

@AqeeAqee I implemented the fix that I thought was best since you can't inject code into a built-in function. I added the function in the 3D Render block menu that matches the one built in the Scene menu. I tried to follow your code style and used similar formatting for the doc page. I did my best to match both your doc pages as well as the original [set tile at location] block on MakeCode's docs page. If there's anything you want changed just let me know and I'd be happy to make the change.
Additionally, I was wondering if it was alright to update the targetVersions in pxt.json. MakeCode Arcade automatically updated the target so I figured it would be good to update the rest as well. I'm not sure if you kept the version at that particular release because of some compatibility reason so I only pushed the change to targetVersions.target that was automatically done and nothing else. If there is no reason for keeping it at the current version, I'd be happy to update that as well.

Fixed in #17 by @megabyte6

@AqeeAqee sorry for asking if this is mentioned somewhere obvious but how are the updates pushed to MakeCode Arcade? From what I found here, it seems that you'd need to add a tag to the current commit (probably something like v0.3.19). I also noticed you add a release with a .uf2 file each time. Is that also necessary or simply something else you decided to do?


Improvements in Tips Page UI [Hacktoberfest]

Current Behaviour: The Tips page UI is more colorful than required and not following the website theme.
Expected Behaviour: It should have a similar design to Covid Page & Tools Page.  Modify the UI and Design of Tips page and make it similar to the design of the Covid Page & Tools Page
Design of Covid Page & Tools Page (follows a white theme and a banner)

Design of Tips Page (follows a colorful theme)

i want to work on it , assign me

Hey @Srinja333! Good to see your enthusiasm to work on the project.
Yes, It understood the issue you raised, you can proceed ahead and work on it.
Click here, and follow the exact steps in order to contribute, otherwise i won't allow myself to accept the pull request.
Happy Hacktoberfest!

i am new in github contribution i want to know how to run project in my computer, how to see changes of webpages by my web browser, please help

i cloned it, and also made branch

Please follow these steps -  https://github.com/skpandey885/HealhUb/blob/main/Documentation/Getting started.pdf

when i chenged in my branch then how to see changes?

PS E:\pr\HealhUb> git remote add upstream https://github.com/skpandey885/HealhUb.git
error: remote upstream already exists.

i got it, started working on it you can believe me

I have got it, started working on it , you can believe me

Ha, work on it! I am not assigning issue to anyone!

@Srinja333 What is the status of your work?

@sumitkr2000 Are you interested to work on it?

@skpandey885 I will be happy to work on this issue. Thank you for considering me for this.

Hey @sumitkr2000! Click here, and follow the exact steps in order to contribute, otherwise i won't allow myself to accept the pull request.
Happy Hacktoberfest!


build: be more clever about path to clang

Setup the clang build to use clang-cl on Windows and clang on other
targets.  This allows libdispatch to build on windows when building with
cl.  libdispatch requires clang-cl to build as it relies on clang
features.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @gwynne @davezarzycki

@swift-ci please test


cmd/server: weight term score by length similarity

Terms are more equal the closer they are in length, so favor closer length terms.

Codecov Report

Merging #512 (a6a16a9) into master (e3971be) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

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

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##           master    #512      +/-   ##
=========================================
- Coverage    8.26%   8.25%   -0.02%
=========================================
  Files          44      44
  Lines        3496    3503       +7
=========================================
  Hits          289     289
- Misses       3184    3191       +7
  Partials       23      23


Fix match maker only match with a maximum of 3 players

 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.Chounry seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 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.Chounry seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I need to correct my email.


Remove price calculation from tx service

What is needed?
Price calculation must be removed from tx service.

Token price endpoint must be removed
Balances usd endpoint should set usd prices always to 0 as a first step, and then it should be removed

@JagoFigueroa could you test if everything is ok?
Token prices doesn't exist and balances with prices should works but returning price 0.

Of course señor, I would never refuse :trollface:
Checking! 😉

All good for me. I will sync with the wallet QAs to see that everything is aligned with the new implementation for prices and mucho bueno 😉


3.0.0 beta 2 value factory error on getedmmodels

ValueFactory attempted to access the Value property of this instance.
this is in startup.cs   -- will try rollback to beta 1 of 3.0.0  which did not get this issue for me.
3.0.0 beta 1 does not throw this error.
Line 203:            var model = Builder.GetEdmModels();
here is the stack trace:
[InvalidOperationException: ValueFactory attempted to access the Value property of this instance.]
System.Lazy1.CreateValue() +411    System.Lazy1.LazyInitValue() +152
System.Lazy1.get_Value() +75    Microsoft.Web.Http.Dispatcher.ApiVersionControllerSelector.GetControllerMapping() +16    System.Web.Http.Routing.AttributeRoutingMapper.AddRouteEntries(SubRouteCollection collector, HttpConfiguration configuration, IInlineConstraintResolver constraintResolver, IDirectRouteProvider directRouteProvider) +48    System.Web.Http.Routing.<>c__DisplayClass1_1.<MapAttributeRoutes>b__1() +57    System.Web.Http.Routing.RouteCollectionRoute.EnsureInitialized(Func1 initializer) +61
System.Web.Http.Routing.<>c__DisplayClass1_0.b__0(HttpConfiguration config) +91
System.Web.Http.HttpConfiguration.ApplyControllerSettings(HttpControllerSettings settings, HttpConfiguration configuration) +70
System.Web.Http.Controllers.HttpControllerDescriptor.InvokeAttributesOnControllerType(HttpControllerDescriptor controllerDescriptor, Type type) +165
System.Web.Http.Controllers.HttpControllerDescriptor.InvokeAttributesOnControllerType(HttpControllerDescriptor controllerDescriptor, Type type) +54
System.Web.Http.Controllers.HttpControllerDescriptor.InvokeAttributesOnControllerType(HttpControllerDescriptor controllerDescriptor, Type type) +54
System.Web.Http.Controllers.HttpControllerDescriptor..ctor(HttpConfiguration configuration, String controllerName, Type controllerType) +105
Microsoft.Web.Http.Dispatcher.ApiVersionControllerSelector.InitializeControllerInfoCache() +377
System.Lazy1.CreateValue() +411    System.Lazy1.LazyInitValue() +152
System.Lazy`1.get_Value() +75
Microsoft.Web.Http.Dispatcher.ApiVersionControllerSelector.GetControllerMapping() +16
Microsoft.AspNet.OData.Builder.VersionedODataModelBuilder.GetApiVersions() +166
Microsoft.AspNet.OData.Builder.VersionedODataModelBuilder.GetEdmModels() +18
ADL.Web.Services.Adl.Startup.Configuration(IAppBuilder app) in C:\TFSAzure\ATRAK Services\ADL.Web.Services.Adl\Startup.cs:203
[TargetInvocationException: Exception has been thrown by the target of an invocation.]
System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor) +0
System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments) +160
System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture) +101
Owin.Loader.<>c__DisplayClass12.b__b(IAppBuilder builder) +66
Owin.Loader.<>c__DisplayClass1.b__0(IAppBuilder builder) +123
Microsoft.Owin.Host.SystemWeb.<>c__DisplayClass2.b__0(IAppBuilder builder) +71
Microsoft.Owin.Host.SystemWeb.OwinAppContext.Initialize(Action1 startup) +462    Microsoft.Owin.Host.SystemWeb.OwinBuilder.Build(Action1 startup) +40
Microsoft.Owin.Host.SystemWeb.OwinHttpModule.InitializeBlueprint() +70
System.Threading.LazyInitializer.EnsureInitializedCore(T& target, Boolean& initialized, Object& syncLock, Func`1 valueFactory) +115
Microsoft.Owin.Host.SystemWeb.OwinHttpModule.Init(HttpApplication context) +106
System.Web.HttpApplication.RegisterEventSubscriptionsWithIIS(IntPtr appContext, HttpContext context, MethodInfo[] handlers) +523
System.Web.HttpApplication.InitSpecial(HttpApplicationState state, MethodInfo[] handlers, IntPtr appContext, HttpContext context) +176
System.Web.HttpApplicationFactory.GetSpecialApplicationInstance(IntPtr appContext, HttpContext context) +220
System.Web.Hosting.PipelineRuntime.InitializeApplication(IntPtr appContext) +303
[HttpException (0x80004005): Exception has been thrown by the target of an invocation.]
System.Web.HttpRuntime.FirstRequestInit(HttpContext context) +658
System.Web.HttpRuntime.EnsureFirstRequestInit(HttpContext context) +89
System.Web.HttpRuntime.ProcessRequestNotificationPrivate(IIS7WorkerRequest wr, HttpContext context) +188

i will see what i can find, i am for sure not calling EnsureInitalized() seen that issue before.
i did have some odd bits of init code and based on your sample i moved it to one place - startup.cs hoping that would solve it but no joy.
only by going back to beta 1 did it stop.
was hoping to get to see a speed fix!!
in other news i found that one of my large api sets will be stuck back on the older packages.
some kind of problem came up where jwt tokens were not working when our angualr 5 client app got a token.  not your issue ....  but what a pain!  the calls just started to fail with no error results and no logged errors so i we left having to guess at what was going on.
something in the OWIN / jwt / authorize pipeline

Anything you are able to share about your setup will be useful. I suspect it is something related to controller settings being applied by an attribute. I can't say what for sure.
The speed fix is now in, but it didn't make the cut for Beta 2. You can see the improvements in PR #398.  @LuukN2 has done some great work. He reported his scenario went from a couple of minutes (I believe) down to 10 seconds! I suspect you'll see equally significant improvements. Unfortunately, I have to draw the line in the sand somewhere or I'll never ship. There were a large number of changes that people have been waiting on for quite some time. Slow isn't nearly as bad as doesn't work. ;) I expect the next round to be RC or even the official release. The performance fix will be there.

Any chance you can up with anything that can be shared for the investigation? I haven't been able to reproduce your scenario - yet.
Based on the stack trace, I've dug in a little bit. This error is occurring when the direct routes (aka attribute routes) are being initialized. I presume this means that you're either mixing OData and vanilla Web API routes or you just haven't removed the attribute route setup. I strongly discourage you from mixing attribute and convention-based routing. There are certain scenarios that simply will not work in API Versioning; it's a limitation of the routing in Web API. If your intent is to only use convention-based routes, then you must still have configuration.MapHttpAttributeRoutes in your setup. Despite OData having attributes, it does not use the direct routing feature at all.

odata api has no attribute routing.
web api has a different prefix and has attributes.
in time i may be able to revise the api routes to remove almost all attributes but that will take time.
ms needs to start telling developers to stop using attributes .... have to look at my apis and see how to redo them, and posibly break them up into different apps  so i can isolate problems.

The issue is specific to Web API and more to do with the OData implementation than anything else. Attributes were an after-thought in Web API. To exacerbate the situation, OData adds its own conventions on top of the routing system using a greedy catch-all route segment (e.g. odataPath*). I've tried to make the choices in routing transparent to versioning, but throwing OData into the Web API mix introduces challenges. Thus far, I haven't encountered such issues in ASP.NET Core, but I had to rework a lot of OData's default implementation.
What I really need is a repro so I can investigate and fix this problem. It shouldn't be happening. Now that I know you're mixing things up, I can try modifying the advanced sample to see if I can't replicate the issue.

Prefix is irrelevant to the routing system in terms of matching a controller. The route template matches a controller name and the controller name maps to a controller descriptor with its backing type. As long as you don't have the same resource (ex: api/resource and odata/resource) using different routing styles, you should be fine. Mixing attribute routing with convention routes, in particular OData, may not behave properly with versioning. I've attempted to resolve that issue more than once and have yet to achieve 100% success in all cases. I call this limitation out in the wiki. I'm happy to entertain a PR or collaboration to address it.

to be sure i am getting things right:
the AspNet Core routing of Web APi and OData is cleaner than Asp.Net 4.xxx
is that right ?
if the EF Core team have the Sql Geography support fixed  ( it looks like they might) then i think we will be looking at moving to Asp.Net Core in the near future.
do you know of any good guidance on this process ? Migrating  from Net 4.xx to .Net Core  for Web API and OData.
many projects are moving to Core and our big hold back has been EF Geography support.

The routing in ASP.NET Core has been completely redesigned and rebuilt from the ground up. It's quite different from the old stack and much better. For one, there is real concept of controllers in ASP.NET Core; it only cares about candidate actions and matching those. The controller base classes are merely a logical container for common functionality used by actions.
This is markedly different from Web API where there was always a controller and set of corresponding actions. I really had to shoehorn things into beat Web API into submission. Web API expects there to be a single controller implementation for every logically registered resource. So a route template like api/resources expects to map to ResourcesController  where Resources is the name by convention. I had to convince Web API there was only a single matching HttpControllerDescriptor, while having an implementation that actually represents multiple descriptors - one for each implementation.
In ASP.NET Core, the design discovers all possible candidates, regardless of defining controller, and asks the IActionSelector to pick the best one. This is much easier to implement for API versioning because there is no different in routing, the implementation only needs to disambiguate duplicate actions by the API version discriminator.
Sadly, the OData team didn't listen to the ASP.NET team and went with an implementation that still expects a single matching action in the configured EDM. This was quite a bit of work for me to rework their design to make it work for API versioning because there can not only be multiple matches, they can be mixed with non-OData matches too.
I'm not suggesting that you switch everything just yet, but the routing and mixing of routing in ASP.NET Core is way better than it was in Web API. There are likely a few more kinks to workout with API versioning (hence the Beta tag), but mixing the two should orders of magnitude more fluid without bizarre routing wonkiness. All the pain thus far has been on the implementation side - e.g. my pain, but seamless for service authors.

I've tried without success to repro your scenario. I have no doubt that it's real, I just can't seem to reproduce a setup that triggers it.
Here's what I know to be true thus far (from the call stack):

The error is being triggered through execution of IControllerConfiguration.Initialize
IControllerConfiguration is typically implemented via an attribute that is applied to a controller
IControllerConfiguration is recursively invoked via the HttpControllerDescriptor constructor
a. 3 calls in the stack trace prove the following: YourController → ApiController → Object
Exception appears to originate from a call to configuration.MapHttpAttributeRoutes to configure attribute routing

This leads me to a few other questions and/or oddities:

It would very unusual to map attribute routes by controller and is normally done for the entire app
Is the name MapAttributeRoutes in the call stack suggesting this might be coming from MVC. You aren't running MVC with Web API and OData too are you?

I added some attribute-routed controllers to the advanced sample app with mixed routing and I could not repro this issue. I tried adding a custom IControllerConfiguration and that alone didn't change anything. I did, however, notice (and vaguely remember) that the order of route registration matters for this type of setup. I almost certain you must call configuration.MapHttpAttributeRoutes before all other routing configurations, including for OData. At least for me, that was the only way I got it to work.
I hope that helps. Any more information or a repro will greatly assist me in helping you iron this out.

thanks for the feedback and info.
i have spoke with the two other developers and we are going to work on changes as we all see that we have a lot of api code and need to plan so that it is manageable and maintainable.
mvc, no we have no mvc views , all our ui is client based.

Great! Thanks. If you happen to resolve the issue or have other feedback or repros, please share. Setting up and testing all the possible application combinations is difficult and there's certainly a chance that there is a bug somewhere. This is why I let the Beta releases burn in a little bit. It's also possible that there may have been an issue in your configuration that didn't previously surface. Regardless, there is clearly some difference between Beta 1 and Beta 2 that is affecting you and I would love to rule out that it's not a bug - if possible.

Given that no one else seems to have reported this and I am unable to repro it myself, it would seem this issue has reached its conclusion. If you or anyone else can share additional information, I'm happy to reopen this and continue investigating.

Hi, I just got the same error

after upgrading to Microsoft.AspNet.OData v 7.1.0 from 6.1.0 and Microsoft.AspNet.OData.Versioning 3.0.0

[InvalidOperationException: ValueFactory attempted to access the Value property of this instance.]
System.Lazy1.CreateValue() +411    System.Lazy1.LazyInitValue() +152
System.Lazy1.get_Value() +75    System.Web.Http.Dispatcher.DefaultHttpControllerSelector.GetControllerMapping() +16    System.Web.Http.Routing.AttributeRoutingMapper.AddRouteEntries(SubRouteCollection collector, HttpConfiguration configuration, IInlineConstraintResolver constraintResolver, IDirectRouteProvider directRouteProvider) +48    System.Web.Http.Routing.<>c__DisplayClass1_1.<MapAttributeRoutes>b__1() +57    System.Web.Http.Routing.RouteCollectionRoute.EnsureInitialized(Func1 initializer) +61
System.Web.Http.Routing.<>c__DisplayClass1_0.b__0(HttpConfiguration config) +91
System.Web.Http.HttpConfiguration.ApplyControllerSettings(HttpControllerSettings settings, HttpConfiguration configuration) +70
System.Web.Http.Controllers.HttpControllerDescriptor.InvokeAttributesOnControllerType(HttpControllerDescriptor controllerDescriptor, Type type) +165
System.Web.Http.Controllers.HttpControllerDescriptor.InvokeAttributesOnControllerType(HttpControllerDescriptor controllerDescriptor, Type type) +54
System.Web.Http.Controllers.HttpControllerDescriptor..ctor(HttpConfiguration configuration, String controllerName, Type controllerType) +105
System.Web.Http.Dispatcher.DefaultHttpControllerSelector.InitializeControllerInfoCache() +400
System.Lazy1.CreateValue() +411    System.Lazy1.LazyInitValue() +152
System.Lazy`1.get_Value() +75
System.Web.Http.Dispatcher.DefaultHttpControllerSelector.GetControllerMapping() +16
Microsoft.AspNet.OData.Builder.VersionedODataModelBuilder.GetApiVersions() +166
Microsoft.AspNet.OData.Builder.VersionedODataModelBuilder.GetEdmModels() +18
IT.Applications.Delivery.Services.Platform.Startup.CreateHttpConfiguration() in C:\Users\p00015230\Source\Repos\IT.Applications.Delivery.Services.Platform\IT.Applications.Delivery.Services.Platform\Startup.cs:97
IT.Applications.Delivery.Services.Platform.Startup.Configuration(IAppBuilder app) in C:\Users\p00015230\Source\Repos\IT.Applications.Delivery.Services.Platform\IT.Applications.Delivery.Services.Platform\Startup.cs:32
[TargetInvocationException: Exception has been thrown by the target of an invocation.]
System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor) +0
System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments) +160
System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture) +101
Owin.Loader.<>c__DisplayClass12.b__b(IAppBuilder builder) +66
Owin.Loader.<>c__DisplayClass1.b__0(IAppBuilder builder) +123
Microsoft.Owin.Host.SystemWeb.<>c__DisplayClass2.b__0(IAppBuilder builder) +71
Microsoft.Owin.Host.SystemWeb.OwinAppContext.Initialize(Action1 startup) +462    Microsoft.Owin.Host.SystemWeb.OwinBuilder.Build(Action1 startup) +40
Microsoft.Owin.Host.SystemWeb.OwinHttpModule.InitializeBlueprint() +70
System.Threading.LazyInitializer.EnsureInitializedCore(T& target, Boolean& initialized, Object& syncLock, Func`1 valueFactory) +115
Microsoft.Owin.Host.SystemWeb.OwinHttpModule.Init(HttpApplication context) +106
System.Web.HttpApplication.RegisterEventSubscriptionsWithIIS(IntPtr appContext, HttpContext context, MethodInfo[] handlers) +536
System.Web.HttpApplication.InitSpecial(HttpApplicationState state, MethodInfo[] handlers, IntPtr appContext, HttpContext context) +173
System.Web.HttpApplicationFactory.GetSpecialApplicationInstance(IntPtr appContext, HttpContext context) +218
System.Web.Hosting.PipelineRuntime.InitializeApplication(IntPtr appContext) +296
[HttpException (0x80004005): Exception has been thrown by the target of an invocation.]
System.Web.HttpRuntime.FirstRequestInit(HttpContext context) +10074976
System.Web.HttpRuntime.EnsureFirstRequestInit(HttpContext context) +95
System.Web.HttpRuntime.ProcessRequestNotificationPrivate(IIS7WorkerRequest wr, HttpContext context) +254

This is nearly impossible for me to troubleshoot without a repro. I have tried unsuccessfully to repro the scenario myself. There is something specific about the setup that causes this - beyond the obvious cyclicular call.
One thing I do notice in the call stack is:
System.Web.Http.Dispatcher.DefaultHttpControllerSelector.GetControllerMapping()
This suggests you are calling VersionedODataModelBuilder.GetEdmModels before configuration.AddApiVersioning. The configuration.AddApiVersioning extension method replaces the default IHttpControllerSelector and IHttpActionSelector so it should be called early in the setup. This might be cause. I would also recommend comparing the sample application setups to your own for other clues about how the configuration might be different.
 I'm can't fix, nor even investigate, what I can't reproduce.

it is was a wrong configuration on my part. not a bug

httpConfig.MapHttpAttributeRoutes() this line is creating the error. I have controllers with different ApiVersion attribute and to use this setting. please advise! thank you!

In my case,  I do have API and Odata Controllers mixed. for odata controllers APIVersion attribute is used and for the WEB API  the version is API specific. If add  httpConfig.AddApiVersioning(), the odata controllers work as expected, but the web api controllers don't ---> error: "API needs version". if add
httpConfig.MapHttpAttributeRoutes(), I get error ---> "ValueFactory attempted to access the Value property..." Is there easy way to solve it, so I can mix api and odata controllers within same web api?

and thank you so much for your attention!

I had to upgrade the Odata package so I can use "IN" clause in my queries.

the versioning was working with prior Odata package for both odata and api controllers with httpConfig.MapHttpAttributeRoutes() setting

Ah … and I forgot one more thing. The order of route registration is important. If you are going to use attribute routing, it needs to be first. Things will not work otherwise (but it still doesn't repro the error described).

Attached is a variant of the Advanced OData Sample, which enables attribute routing and adds a few extra attribute-routed controllers. It doesn't repro the scenario, but perhaps it can help you troubleshoot or generate a repro on your side.
Attachment: Advanced OData Sample with Attribute Routing


Indent setting for wrapped lines

Some styleguides require a different indent for wrapped lines.
For example, the Google JavaScript styleguide  requires a four-space indent for wrapped lines, two-space otherwise.
It would be handy to have a setting for this, e.g. wrap_line_indent.

This would be good.  It is lower priority at this time.  PR's welcome.


Update common files.

Will need an update in test-infra for all checks to pass.

@geeknoid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

build_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test build_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

build_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test build_client-go_release-1.4

gencheck_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test gencheck_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

build_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test build_client-go_release-1.4

gencheck_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test gencheck_client-go_release-1.4

lint_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test lint_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test gencheck_client-go_release-1.4

lint_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test lint_client-go_release-1.4

build_client-go_release-1.4
1b7dc55f3a9022493b497676707adc1b50295587
link
/test build_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test lint_client-go_release-1.4

build_client-go_release-1.4
1b7dc55f3a9022493b497676707adc1b50295587
link
/test build_client-go_release-1.4

gencheck_client-go_release-1.4
1b7dc55f3a9022493b497676707adc1b50295587
link
/test gencheck_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_client-go_release-1.4
a388cb56881063a647558c2074a309920fa56553
link
/test lint_client-go_release-1.4

gencheck_client-go_release-1.4
1b7dc55f3a9022493b497676707adc1b50295587
link
/test gencheck_client-go_release-1.4

build_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test build_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go_release-1.4
1b7dc55f3a9022493b497676707adc1b50295587
link
/test gencheck_client-go_release-1.4

build_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test build_client-go_release-1.4

lint_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test lint_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

build_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test build_client-go_release-1.4

lint_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test lint_client-go_release-1.4

gencheck_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test gencheck_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rcernich Rob, I'm trying to update the common files, which means this is getting a new build-tools container. This results in some code gen differences, which ends up breaking the build.
The build-tools container is defined in istio/tools/docker/build-tools/Dockerfile. The container image being added here was built from the release-1.4 branch of that Dockerfile.
Any ideas?

/retest

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test lint_client-go_release-1.4

gencheck_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test gencheck_client-go_release-1.4

build_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test build_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test gencheck_client-go_release-1.4

build_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test build_client-go_release-1.4

lint_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test lint_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

build_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test build_client-go_release-1.4

lint_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test lint_client-go_release-1.4

gencheck_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test gencheck_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid, are you still having problem?  I suspect the code changes are the result of this.  I've been waiting for a new tools image that contains those changes, so I could regenerate the code.

Yeah, it's still not working. Note that this is in the 1.4 branch, which is using a container image derived from the 1.4 version of the Dockerfile in tools/docker/build-tools.
So do we need to merge that Dockerfile change to the 1.4 branch then?

Looks like there's a dependency issue with the generated code, scheme.Codecs.WitihoutConversion() is undefined.  I suspect this has to do with changes to the k8s dependencies in tools, specifically using new code generators (but I haven't verified).

@geeknoid, feel free to pull this in: https://github.com/rcernich/istio-client-go/commit/59700f742da3c209f56fe17393a46f2b5ff87266

hold on, looks like k8s client-go needs to be updated too.

With these changes, the build succeeds.  Notice, there were some additional updates to the generated files when the newer dependencies were introduced.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

lint_client-go_release-1.4
625d792a65412521abead9f2a37791b2d854ddad
link
/test lint_client-go_release-1.4

gencheck_client-go_release-1.4
8fbf3558d039845a8af03fd34fe069a37ace4457
link
/test gencheck_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_client-go_release-1.4
8fbf3558d039845a8af03fd34fe069a37ace4457
link
/test gencheck_client-go_release-1.4

lint_client-go_release-1.4
8fbf3558d039845a8af03fd34fe069a37ace4457
link
/test lint_client-go_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@geeknoid, for the gencheck job, do you also need to update the prow config to use the latest image?  I wonder if that's part of the problem.

/retest


crosstalk between fmriprep processes (v1.1.8)

So this is a bit of a strange bug, but I'm nearly 100% certain it's not user-error.
I was concurrently processing two 50-60 subject datasets on an HPC, with each subject having their own job in the queue. After completion, some of the freesurfer outputs from dataset1 were found under the derivatives/freesurfer folder of dataset2.
Submitted jobs were executed like this:
singularity exec -B /om:/om -B /om3:/om3 -B /cm:/cm $proj/singularity_images/fmriprep_1_1_8.simg
fmriprep $bids_dir $out_dir participant --participant_label $subject --nthreads 8
--mem_mb 13000 --ignore slicetiming --use-aroma --ignore-aroma-denoising-errors -w $scratch
--fs-license-file /cm/shared/openmind/freesurfer/6.0.0/.license
--template-resampling-grid /om/group/saxelab/OpenAutism/scripts/HarvardOxford-cort-maxprob-thr50-2mm.nii.gz
where the two different datasets had independent $bids_dir and $out_dir arguments.
During execution, a subject from the first dataset (EIB) did not find his pre-existing freesurfer outputs in the EIB/BIDS/derivatives/freesurfer directory (although they was there), and instead created a new freesurfer folder in the second dataset's (TASTI2) tree.
Unfortunately I deleted all of the slurm logs before I noticed this issue, but the first lines in the recon-all.log file, below, highlight the issue. Note that the subject is an EIB (dataset1) subject, and the input is coming from the EIB hierarchy, but the SUBJECTS_DIR is being set to TASTI2 (dataset2). All of the rest of that subject's outputs are correctly found in the EIB hierarchy.
[toddt@node014 scripts]$ more recon-all.log
Wed Oct 10 06:53:10 UTC 2018
/om3/group/saxelab/ASD_TOM/data/TASTI2/BIDS/derivatives/freesurfer/sub-SAXEIB69
/opt/freesurfer/bin/recon-all
-autorecon1 -i /om3/group/saxelab/ASD_TOM/data/EIB/BIDS/sub-SAXEIB69/anat/sub-SAXEIB69_T1w.nii.gz -noskullstrip -openmp
7 -subjid sub-SAXEIB69 -sd /om3/group/saxelab/ASD_TOM/data/TASTI2/BIDS/derivatives/freesurfer
subjid sub-SAXEIB69
setenv SUBJECTS_DIR /om3/group/saxelab/ASD_TOM/data/TASTI2/BIDS/derivatives/freesurfer
F

Is $scratch dataset-dependent? It may be the case that you're reusing a scratch directory that already had the setup node. Can you try a new $scratch directory, just to rule that out?

No, scratch is the same workingdir for both datasets.
When I look in that workingdir, I see an fmriprep_wf directory. And under that directory is a child directory for each subject, along with a single fsdir directory, that is currently pointing to dataset2's output directory.
I would think that node would be regenerated every time the "derivatives" input changed, though, and that that input would change whenever the most recent dataset switched. Perhaps there's a race condition where multiple processes are updating it simultaneously and it ends up executing for some of those processes with incorrect inputs?

Yeah, if you use the same scratch directory, you could get race conditions.

Hi @toddt, this is definitely a race condition on the shared working directory. The easiest fix is to add a dataset folder to the working directory (i.e. -w work/dataset1, -w work/dataset2, etc.)


Do not log sensitive data in logs

This PR changes the behaviour of install command regarding watching installation. The algorithm is taken from is-installed.sh. Following changes are introduced:

Do not print installer logs - they may contains sensitive information
On error print a line explaining how user cen fetch the logs himself
There is a possibility to specify watch timeout
Installation goes indefinitely unless timeout is specified
On timeout errorLog from installation status is printed

Fixes #26

The uninstall is doing exactly the same, could you please apply the same fix


Update commands.md

chore: misspelling

hi!
can you rename your commit message to chore: misspelling? And I'll merge it

done

Hello! Sorry, how i can do that?

@vladislav-sevostyanov
git commit --amend - > chore: misspelling

Thanks. I did it. Achievement of the day

Hm, you did it wrong =(
Look at the number of your commits. It should be one.


adds cli doc

What does this PR do?
adds ddev's cli documentation to the repo
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Superseeded by https://github.com/DataDog/integrations-core/pull/6094
Closing


PMID:11014802

increased cell population growth on gluconate carbon source

increased cell population growth on gluconate carbon source FYPO:0006234
edit file: 884374b24346e939e1e868297533dc49cc5fc21f
release: f08be34c1aaa2f3a32bc54e8fdcd703d89407fe4


Can't search symbols list by mangled name

It feels like a bug to be able to right-click the symbols list, "Show Mangled Names" and then not be able to search by what's being displayed.

This appears to have been fixed in a previous build

Confirmed.


EncodeException "The serialized GlobalReference has type PointerType ...

Hi, developers of llvm-hs-quote.
Environment
llvm-hs-quote:  e2dec47ad2bbf07d6a7d0e0cd8f91ae6c5186d72
stack solver: lts-10.3 (this is the same as llvm-hs-quote)
Error
I had a error with the following code.
module10 :: AST.Module
module10 = [Quote.LLVM.llmod|
  define void @f() {
  entry:
    call void @g()
    ret void
  }
  define void @g() {
  entry:
    ret void
  }
|]

toLLVM :: AST.Module -> IO ()
toLLVM mod = Context.withContext $ \ctx -> do
  llvm <- Module.withModuleFromAST ctx mod Module.moduleLLVMAssembly
  BS.putStrLn llvm

main = toLLVM module10

I think the error occurs in call void @g().
Error message
 EncodeException "The serialized GlobalReference has type PointerType {pointerReferent = FunctionType {resultType = VoidType, argumentTypes = [], isVarArg = False}, pointerAddrSpace = AddrSpace 0} but should have type FunctionType {resultType = VoidType, argumentTypes = [], isVarArg = False}"

Should be fixed in 1497273bf70ae22b76d190bcbc24737cfb50e334

Thank you very much for you quick fixing! You solved my problem.


Run on windows

Hi there, thank you for sharing, good work.
I want to run the code on windows and it says NCCL error.

So i changed the backend from NCCL to GLOO, and an invalid scalar type error pop up.

Do you have any idea why? What is your environment running the code? Mine is python3.10 Cudatoolkit11.3 with torch 1.12.1+cu113
Appreciate!

Hi, thanks for your interest!
I tested the code on Ubuntu20.04 with NCCL backend. Windows does not support the NCCL backend so you have to use gloo. However it does not seem like a Windows-related problem since I got the same error when I run with gloo on Ubuntu 😂
Before I figure this out, a temporary solution would be to use DP instead of DDP which does not require a communication backend. To do this you have to change the parameter of trainer in launch.py from strategy='ddp_find_unused_parameters_false' to strategy='dp' and modify codes related to aggregating multi-gpu outputs in systems/*.py, namely the validation_epoch_end and test_epoch_end function. I'll open a new branch for this DP support very soon and I'll let you know when I figure out this gloo error.

Windows single-GPU training is now supported in my latest commit. Please have a try using the same training command in README.
It is possible to support multi-GPU training on windows using DP, but it requires more code changes:

implement validation_step_end and test_step_end to aggregate results from all GPUs
possible modification in validation_epoch_end and test_epoch_end as the return value structure coule be different

see https://pytorch-lightning.readthedocs.io/en/latest/accelerators/gpu_intermediate.html#dp-caveats for more details.
The Invalid scalar type error encountered when using gloo backend is related to the bool type parameters used in nerfacc. I'll try to fix this issue when I have time.

Thank you very much, that was quick.


[ZEPPELIN-5730] Support time unit setting in zeppelin conf.

What is this PR for?
This PR is to upport time unit setting in zeppelin conf, such as 5s ,3m ,1h.
What type of PR is it?
Feature
Todos

[ ] - Task

What is the Jira issue?

Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN-5730

How should this be tested?

CI passed

Screenshots (if appropriate)
Questions:

Does the licenses files need to update?  No
Is there breaking changes for older versions? No
Does this needs documentation? Yes

Thank you but it looks like braking configuration after upgrading it. Do you have any plan to support old configuration in a new version? Otherwise, do users need to change it by hand?

Thank you but it looks like braking configuration after upgrading it. Do you have any plan to support old configuration in a new version? Otherwise, do users need to change it by hand?

Hi @jongyoul,  Old configuration is also supported in this PR.
I am sorry to forget to add comments for the code.
If the string passed in doesn't include time unit,  method timeUnitToMill  would fail to parse and  return -1.  And the original code logic is then executed

@huage1994 Could you add unit test in ZeppelinConfigurationTest?

@huage1994 Could you add unit test in ZeppelinConfigurationTest?
Sure. I would add it.

LGTM

I do not like this solution and would "veto" it. The readability and simplicity of the code suffers greatly with this change.
I would prefer a conversion directly at the call (e.g. https://github.com/apache/zeppelin/blob/85ed8e2e51e1ea10df38d4710216343efe218d60/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/lifecycle/TimeoutLifecycleManager.java#L51).

I do not like this solution and would "veto" it. The readability and simplicity of the code suffers greatly with this change. I would prefer a conversion directly at the call (e.g.
https://github.com/apache/zeppelin/blob/85ed8e2e51e1ea10df38d4710216343efe218d60/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/lifecycle/TimeoutLifecycleManager.java#L51
).

Hi @Reamer , I see what you mean.
Now I plan to keep the original code of method getLong without changin.
And then add a new method getTime. Maybe like this:
  public long getTime(String envName, String propertyName, long defaultValue) {
    String timeStr = getString(envName, propertyName, defaultValue + "");
    if(timeUnitToMill(timeStr) > 0){
      return timeUnitToMill(timeStr);
    }
    return getLong(envName, propertyName, defaultValue);
  }

Finally ,replace the calls of getLong  with getTime where needed.
How do you think about it?

How do you think about it?

Much better.
Another improvement timeUnitToMill should throw an exception instead of returning -1. This exception should be caught and reacted to.

Another improvement timeUnitToMill should throw an exception instead of returning -1. This exception should be caught and reacted to.

👍  Thanks a lot for your advice!

CI seems to be unstable recently 🤒 .

This PR currently seems to cause the test  K8sStandardInterpreterLauncherTest to fail, though they looks like to be not related. It's ok to run  K8sStandardInterpreterLauncherTest in my local environment. But it failed every time on CI of my fork repository.
I'll take some time to resolve this.

Fixed.

How did it happen? How did you fix the error?

The type of default value of ZEPPELIN_INTERPRETER_CONNECT_TIMEOUT was int before.
When the ZEPPELIN_INTERPRETER_CONNECT_TIMEOUT is not set, we expect to get the default value , but getLong()  in getTime() would get nothing  because intValue  and longValue are two independent fields in ConfVars.
I change the default value type from int to long as follow.
-    ZEPPELIN_INTERPRETER_CONNECT_TIMEOUT("zeppelin.interpreter.connect.timeout", 600000),
+    ZEPPELIN_INTERPRETER_CONNECT_TIMEOUT("zeppelin.interpreter.connect.timeout", 600000L),

And I added  comment for method getTime.
  /**
   * This method is to support time unit like `1s`, `2m`, `3h`.
   *
   * @param {ConfVars} c . Note：The type of default value of `ConfVars  c` should be long.
   * @return {long} Milliseconds
   */
  public long getTime(ConfVars c) {
    try {
      return timeUnitToMill(getString(c.name(), c.getVarName(), ""));
    } catch (Exception e) {
      return getLong(c);
    }
  }


QoS server information

This is a tracking issue for keeping track of information that's currently known about QoS servers
Connection Flow
For this information "client" referrers to the Mass Effect 3 game client
This communication usually happens when the client first reaches the main menu, however it may happen again if the client connection changes or it needs to reconnect
1. HTTP request to Blaze QoS server (qtyp=1)
Client makes HTTP GET request to the QoS servers provided by the server (gossjcprod-qos01.ea.com) on the port specified (17502). This server is the XML HTTP version of a Blaze server.
The URL is always as follows: /qos/qos?vers=1&prpt=3659&qtyp=1
The response body is the following:
<qos>
    <numprobes>0</numprobes> <--- Number of times the client should probe (This first request type ignores it and sends 10 probes)
    <qosport>17499</qosport> <--- This the port of the server we are going to make our connection with
    <probesize>0</probesize> <--- Payload size in bytes for the probes the client should send (This first type ignores it and the probes are always 20 byte sized)
    <qosip>2733913518</qosip> <-- This is the IP of the server we are going to connect with (Convert to big endian bytes and thats the ip parts in this case <IP_ADDRESS>)
    <requestid>1</requestid> <-- Request ID *never* changes for this first request type and will always be one
    <reqsecret>0</reqsecret> <-- Request Secret *never* changes for this first request type and will always be zero
</qos>

The information provided above is the IP and port for the UDP QoS server, this server is responsible for providing the client with its public-facing IP address and port.
2. UDP QoS Probes (qtyp=1)
Once the client has retreived the server information from the HTTP request, the local game port (3659) will start sending UDP probes to the  QoS server obtained in the previous step.
These probes, are always exactly 20 bytes long and consist of the following:

What
Length
Description

Unknown
4 bytes
This value is known but has been seen as 0x2, 0x3, and 0x5

Request ID
4 bytes
This is the ID of the request obtained from the HTTP endpoint

Request Secret
4 bytes
This is the request secret obtained from the HTTP endpoint

Probe number
4 bytes
The probe sequence number (1..2..3...4 etc) doesnt seem to change for this type tho

Timestamp
4 bytes
Appears to be some sort of timestamp, too small to be a unix timestamp though

For each of these probes the server will send back the following probes which are always exactly 30 bytes long:

What
Length
Description

Client probe
20 bytes
An exact copy of the probe that the client sent (Above)

Client IP Address
4 bytes
The bytes of the public facing IP address that accessed the QoS server (From its perspective)

Client Port
2 bytes
The public facing port that the client is connecting from (May differ due to dynamic NAT port mappings)

Padding
4 bytes
Zeros padding the remaining 4 bytes of length

3. HTTP request to Blaze QoS server (qtyp=2)
Client makes HTTP GET request to the QoS servers provided by the server (gossjcprod-qos01.ea.com) on the port specified (17502). This server is the XML HTTP version of a Blaze server.
The URL is always as follows: /qos/qos?vers=1&prpt=3659&qtyp=2
The response body is the following:
<qos>
    <numprobes>10</numprobes> <-- Number of probes to send
    <qosport>17499</qosport> <-- This the port of the server we are going to make our connection with
    <probesize>1200</probesize> <-- Size for the probes (1200 bytes)
    <qosip>2733913518</qosip> <-- This is the IP of the server we are going to connect with (Convert to big endian bytes and thats the ip parts in this case <IP_ADDRESS>)
    <requestid>1040</requestid> <-- Request ID, actually changes this time
    <reqsecret>421</reqsecret> <-- Request secret, actually unique this time
</qos>

4. UDP QoS Probes (qtyp=2)
Once the client has retreived the server information from the HTTP request, the local game port (3659) will start sending UDP probes to the  QoS server obtained in the previous step.
These probes, are always exactly the length provided by the "probesize" field (with the minimum length of 20 for the header):

What
Length
Description

Unknown
4 bytes
This value is known but has been seen as 0x2, 0x3, and 0x5

Request ID
4 bytes
This is the ID of the request obtained from the HTTP endpoint

Request Secret
4 bytes
This is the request secret obtained from the HTTP endpoint

Probe number
4 bytes
The probe sequence number (1..2..3...4 etc) changes for this type

Number of probes
4 bytes
The total number of probes the client intends to send (Bits flipped for response)

Padding
remaining probe length in bytes
Padding of zeros until the desired probe length is reached

For each of these probes the server will send back the following probes which are always exactly the length provided by the "probesize" field:

What
Length
Description

Client probe
20 bytes
An exact copy of the probe that the client sent excluding the padding (Above)

UBPS
4 bytes
Seems to be the Upstream Bits Per Second

Client Port
2 bytes
Appears to be the connecting port (public facing mentioned in previous QoS)

Padding
remaining length to reach 1200 bytes long
Padding of zeros until the desired probe length is reached

5. HTTP request to Blaze QoS server (firewall)

I have not been able to get clients to reach this step with a custom QoS server

Client makes HTTP GET request to the QoS servers provided by the server (gossjcprod-qos01.ea.com) on the port specified (17502). This server is the XML HTTP version of a Blaze server.
The URL is always as follows: /qos/firewall?vers=1&nint=2
The response body is the following:
<firewall>
    <ips>
        <!-- IP adddresses of some servers to send messages to  -->
        <ips>2733913518</ips> <-- <IP_ADDRESS>
        <ips>2733913519</ips> <-- <IP_ADDRESS>
    </ips>
    <numinterfaces>2</numinterfaces>
    <ports>
        <!-- Ports of servers to send messages to  -->
        <ports>17500</ports>
        <ports>17501</ports>
    </ports>
    <!-- ID of the request -->
    <requestid>1043</requestid>
    <!-- Secret of the request -->
    <reqsecret>1871</reqsecret>
</firewall>

6. Firewall UDP packets
Once the client has retreived the server information from the HTTP request,
the local game port (3659) will start sending UDP packets to the servers mentioned
in the response above.
These packets are always exactly 8 bytes have the following structure:

What
Length
Description

Request ID
4 bytes
This is the ID of the request obtained from the HTTP endpoint

Request Secret
4 bytes
This is the request secret obtained from the HTTP endpoint

The server does not respond to these packets, the information is likely stored for the
next request below:
5. HTTP request to Blaze QoS server (firetype)
Client makes HTTP GET request to the QoS servers provided by the server (gossjcprod-qos01.ea.com) on the port specified (17502). This server is the XML HTTP version of a Blaze server.
The URL is as follows: /qos/firetype?vers=1&rqid=654&rqsc=1115&inip=-1062708997&inpt=3659
where rqid is the request ID from the previous HTTP request to firewall
and rqsc is the request secret from the previous HTTP request. inip is
the internal ip address of the client in thise case <IP_ADDRESS>, and inpt is
the internal port of the client.
The response body is the following:
<firetype>
    <firetype>2</firetype>
</firetype>

6. End
That seems to be the end of the communication with the QoS servers.

JS Snippet for decoding IP addresses from 32bit ints
function decodeIp(ip /* int value */) {
    let [a, b, c, d] = [
        (ip >> 24) & 0xff,
        (ip >> 16) & 0xff,
        (ip >> 8) & 0xff,
        ip & 0xff,
    ];
    return `${a}.${b}.${c}.${d}`;
}

Note in regards to the QoS servers from Mass Effect Andromeda: They appear compatible in regards to underlying protocol however they are not acceptable for use with ME3 as the blaze http request is protected with TLS and the game client doesn't support TLS QoS servers
I will need to investigate further to see if there's any differences in the underlying protocol so that once a working QoS server is created it can be ported for MEA aswell

Looks like one of my assumptions about the QoS servers is true, if the QoS server is running locally or on the same private network as the connecting client the client silently fails (it connects and looks completely normal to the server) except when informing the server about its QoS state it marks the latency to that server as 268374015 (Seems to be the highest possible value for the latency which matches that of a server which is completely offline and not connectable). However when the server is running on a remote host outside of the local device and not on the same network the latency appears correctly and things seem to work properly
Local:

Remote:

There is no visible request flow different on the server, the traffic for both appears to be identical, server appears to still need some work as the results for DBPS and UBPS are not being update and the firewall portion of the request flow still hasn't been reached


adding pre-checks based on regex to fail the tests

Refactored run-all.go so that regex checks can be run without running the whole test cases, thus detecting the test failure early.
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #4750

cc: @JorTurFer

Any update?

Any update about this?

Closing due to inactivity for a long period, feel free to re-open if you have time to complete it anyway.


DOC: Update documentation

The install page was carrying old information about supported Python
version.
Page to be updated: http://www.zipline.io/install.html#gnu-linux
I was based on the information provided at: http://www.zipline.io/releases.html#release-1-2-0 https://www.zipline.io/releases.html#release-1-1-1
The Dockerfile also bases my assumption of outdated documentation: https://github.com/quantopian/zipline/blob/master/Dockerfile.

Hey, can I get some feedback on this PR?

@willianpaixao sorry for the long delay on this. This looks good to me.


[KOGITO-5888] Remove namespace option from BDD tests

https://issues.redhat.com/browse/KOGITO-5888
Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[ ] You have read the contributors' guide
[ ] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[ ] Pull Request contains a link to the JIRA issue
[ ] Pull Request contains a description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

/jenkins test

Closing as it is now included into https://github.com/kiegroup/rhpam-kogito-operator/pull/76, due to testing issues, both PRs needs to be in one


chrome android physical web not working

Hello,
anybody else with this problem? I tried everything to make it work. Set up a beacon with a different HTTPS websites. I enabled bluetooth, location and physical web (experimenting with flags, diagnostics, permissions, settings) on my smartphone. I installed the latest version of Chrome, beta, dev. I tested for at least 100 times. I was able to get a notification only a few times, only sometimes when transmitting a new website or when manualy searching in Chrome. With Opera Labs I allways get a notification, even on the fly but I really need this in Chrome, because it is a standard browser. I really need this thing to work, because my web service depends on that. Thanks.
There is also article about this issue posted two months ago:
https://mobiforge.com/design-development/eddystone-beacon-technology-and-the-physical-web
Two weeks ago:
https://github.com/google/physical-web/issues/648
...
Greetings,
V.G.G.

I'm sorry you are not being notified reliably.  I'm curious as to what the specific behavior is on the most recent Dev.  Do you get any notification?  If so, when isn't it notifying you when you expect it to be?
In the future, please file chromium bugs at crbug.com.

For info, which smartphone do you have and which version of Android is running?

thanks for reporting this bug.
I tried the latest Chrome dev but it seems no different than Chrome. I expect notification for every PW available, when somebody comes close to the beacon. I'm experimenting with different HTTPS websites and with Chrome it is very unreliable. Mostly but not so often it is detected, when I restart my smartphone and after I open/close Chrome. Today I managed to detect it once, with no restart after a while, but this is very rare. With Opera Labs it gets through easely, it doesn't have to be HTTPS and it has a special tab just for PW.
Smartphone: Xiaomi Redmi 3 Pro
Android: 5.1.1 LMY47V
Play store: 6.5.08.D-all [0] 2792142
chrome://physical-web
Physical-Web is ON
Android SDK version: Compatible
Data connection: Connected
Location services: Enabled
Physical Web privacy settings: On
Bluetooth: Unknown
Nearby web pages
https://...

Try this combination.
Make sure your google chrome is closed( i.e., not even running in background)
First switch on bluetooth, then your location and then open google chrome.
If there is a beacon nearby transmitting a HTTPS URL, you will definitely see physical web notification.
Let us know your observations.

kanodia, I can get PW if I follow these steps. There is no notification icon, but if I check, it is there. This could be OS related or maybe not. As you would have to know, if the beacon is available. If somebody would like to check manually, he would have to find a PW scanner in the settings, and even this option is not always available, as If you would have to enable some things first.

Physical web notifications are currently low priority notifications. Android does not show notification icon on the status bar if it is a low priority notification.

I'm seeing the same thing, which is unfortunate. I'd been hoping to use virtual physical web beacons for a very time-sensitive thing (UK European Referendum 'remain' campaign), but they seem to rarely show up to people not following the steps above, which makes them unsuitable as they're not discoverable.
I really wish I had access to Nearby Notifications already, that looks like an awesome evolution.

Separate issues: @kanodia is talking about the level of the notification (min) so it is there it just requires the user to pull down to see it. We don't want this to be abused.
@MatthewWilkes seems to have a separate issue: the notification not showing up at all. We get this quite a bit from devs as they don't see the notification show up constantly. As we are trying to save power, we only scan when the user wakes the phone. Once this is working, it should show up regularly.

hi Scott,
big fan of your videos on physical web, and my team is happy that we found this post, because we just started to explore this physical web technology but our experience has been that iOS experience is less of a hassle compare to Android. In fact we made a post on estimote (we purchased a set for testing):
https://forums.estimote.com/t/physical-web-on-android-not-being-detected/4130
The way to get the physical web notification on android leaves a lot to be desired, and the way we finally got it to work was follow @kanodia  suggestion above.
Now if we are to build an experience around that, for android we need tell the target audience that they must make sure "chrome is not running" first.  Most users would not want to force-close Chrome just to engage in a physical web experience that they're not sure if they will like or not. And to ensure chrome is fully closed, you need to get to setting/apps/chrome to hit the force close button.
Edit: I just tested by restarting my phone, bluetooth is on and chrome is active when android loaded, but the physical web notification didnt show up at all. It only showed up when I quickly disable bluetooth and enable it again. Why is the notification so inconsistent with Android? Both colleague and I are using unlocked android phone (Nexus 6 and Moto X pure edition 2015) and have all the requirements met from this page:
https://support.google.com/chrome/answer/6239299?hl=en
From the above page you can see iOS have less steps to deal with it. This is also true when I ask other co-workers with iPhones to engage in the beacon test using eddystone-url. I personally think the amount of steps an android user need to do in order to make sure they are able to engage in physical web experience will turn way more user instead of garnering new ones into this exciting technology.

@funkybudda : W.r.t. : "Now if we are to build an experience around that, for android we need tell the target audience that they must make sure "chrome is not running" first.  Most users would not want to force-close Chrome just to engage in a physical web experience that they're not sure if they will like or not. And to ensure chrome is fully closed, you need to get to setting/apps/chrome to hit the force close button.""

These were the required steps to get physical web when it was only a part of chrome. Ensuring chrome is closed, switching on bluetooth, then location and then opening chrome used to get physical web notification. Now it is a challenge to educate people about this as there will be high chances that there chrome might be running in the background and if that is the case physical web notification will not come.
Now as physical web is a part of nearby API which is a part of latest play services (version 9.2.55), physical web scanning as become better, reliable, and even the steps are reduced and simple. In Fact UI is also better and more contextual.
Now, if the phone has the latest play services, then just by switching on bluetooth and location, a notification is received if there is a beacon transmitting an https URL nearby. The first time notification reads as "website for this location found".
Hence, now physical web notification will come in just two steps, bluetooth and location needs to be on, that's it.
Hence, making people aware of this experience will be easier compared to the previous one.

As scanning has now moved to nearby, closing this issue. If there is anything new to add, please feel free to reopen.

Hi Scott,
 As i understand this article eddy stone android phone notification problem comes because of nearby . When i turn on all Bluetooth,location ,physical web and WiFi , as you know chrome has default browser still Im not getting notification.One mobile notification comes only one time per day ,another mobile not that much frequently per day , another mobile not able to get per day . As i understand based on testing behavior if notification will pushed if i ignore it automatically clear status bar notify after 5 or 10 mins but if i swipe to remove it will not push notification after 24 hours. can you help out to solve this problem

Sorry but I don't have enough information. If you are running on Android 4.4+, and have latest Google Play Services, all you need is Location and Bluetooth turned on. EVERY time you wake up the phone it will scan and show a notification. Can you confirm your phone OS and GPS version?

Hi Scott,
android phone version 5+,and having latest google play service ,chrome version 54,  both location and Bluetooth on. It is in wake up mode only,still facing problem to get notification frequently

OK, it looks like there may be a bug. Sorry, a bit more info. Can you give me your phone?

I have a Moto G4 plus phone running on Android Nougat, the chrome has Physical Web enabled and location permissions granted, Bluetooth and location settings are also on yet I'm unable to receive a notification from a beacon emitting URL frame of https address.

Have you tried your URL in verify.physical-web.org?
Does the beacon show up in the Physical Web app?

@scottjenson Yes, I am using a verified URL and the beacon does show up in the Physical Web app. I manually scanned for beacons in Chrome (See What's Nearby) and it was also there but, there wasn't any notification thrown by it.

Our notification code was originally in Chrome, but that's all moved to Android OS so the Chrome settings aren't necessary.
The biggest issue when testing it understanding that we're only scanning on screen wake up so put your phone to sleep and see if that helps.
however, this is why we're now moving to more deterministic approaches. YOu'll notice that you can also add the nearby scanner to the home screen. That should always work.

Yeah, the beacon scanner being present into a web browser did not make much sense, this looks better to deal with physical web separately(in OS) and then extend its support to other browsers as well.
This may not be the right platform to ask but, I wanted to contribute to physical-web as a GSOC candidate, I could not find any IRC channel for these discussions(on GSOC organisations page), please provide some direction.
Thanks for the help.

Sorry, but we're not involved with the Google Summer of Code so don't know
who to contact.
Scott
On Fri, Jan 27, 2017 at 12:18 AM, Yash Kant<EMAIL_ADDRESS>wrote:

Yeah, the beacon scanner being present into a web browser did not make
much sense, this looks better to deal with physical web separately(in OS)
and then extend its support to other browsers as well.
This may not be the right platform to ask but, I wanted to contribute to
physical-web as a GSOC candidate, I could not find any IRC channel for
these discussions(on GSOC organisations page), please provide some
direction.
Thanks for the help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/physical-web/issues/667#issuecomment-275611691,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAbuodPxdm-MiQVZAERyUV2CeEG-AZQks5rWahdgaJpZM4InRaA
.

however, this is why we're now moving to more deterministic approaches.

Any chance you could share more details about it at this point? (:

We have an experiment in Chrome for iOS (you have to turn on the Physical Web flag) that shows results in the Omnibox. That isn't as lightweight as a notification but it seems to be much easier to explain to people.

Notifications of my beacons do not appear. Only when I go into "Settings" => "Google" => "Nearby" finally found the link and suddenly appears also in the pull-down notifications


Opening fat Mach-O picks wrong architecture

Version and Platform (required):

Binary Ninja Version: 3.1.3740-dev
OS: macOS
OS Version: 12.5.1
CPU Architecture: M1

Bug Description:
Possibly a dupe of #3500; I didn't investigate the cause.
If I try to open the arm64 slice of a Mach-O containing x86_64 and arm64 slices, it opens the x86_64 one and disassembles it as arm64.
Steps To Reproduce:
Please provide all steps required to reproduce the behavior:

File -> Open; pick libbinaryninjacore.1.dylib from the same Binary Ninja version being run.
At the top of the resulting 'Open with Options' window, where it says "Universal binary detected. Select an object to load", switch from the default "Mach-O 64-bit executable x86_64" to "Mach-O 64-bit executable arm64".
Press Open without changing any other settings.
In the disassembly view, notice that __macho_header at the beginning of the file shows CPU_TYPE_X86_64, and all the function disassemblies are nonsense arm64 instructions, seemingly indicating that the wrong slice was opened.

Additional Information:
Something I noticed which is probably unrelated: At the bottom of the 'Open with Options' window is "* Note: Previously modified settings automatically applied for this file. Show modified settings."  But after clicking "Show modified settings", no settings are displayed (the list of settings goes blank).

As a temporary workaround, you can disable the ui.files.restore.viewOptions setting which obviates the bug I'll push a fix shortly and update.

Fixed in 3.1.3747-dev.
Also, the ui.files.restore.viewOptions setting is now disabled by default.


Could not resolve io.github.stewforani:methodtime:1.0.0

Could not resolve io.github.stewforani:methodtime:1.0.0

不好意思，已注释


[ISSUE #3484] fix springcloud client lossless registration

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3529 (97dd692) into master (66d9c13) will decrease coverage by 0.41%.
The diff coverage is 38.46%.

@@             Coverage Diff              @@
##             master    #3529      +/-   ##
============================================
- Coverage     62.81%   62.39%   -0.42%
+ Complexity     5572     5525      -47
============================================
  Files           842      842
  Lines         23343    23353      +10
  Branches       2115     2118       +3
============================================
- Hits          14663    14572      -91
- Misses         7333     7436     +103
+ Partials       1347     1345       -2

Impacted Files
Coverage Δ

...pringcloud/init/SpringCloudClientBeanListener.java
18.47% <33.33%> (ø)

...ingcloud/ShenyuSpringCloudClientConfiguration.java
100.00% <100.00%> (ø)

...plugin/logging/rocketmq/LoggingRocketMQPlugin.java
22.72% <0.00%> (-72.73%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
75.00% <0.00%> (-25.00%)
:arrow_down:

...ogging/rocketmq/body/LoggingServerHttpRequest.java
0.00% <0.00%> (-20.00%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

...org/apache/shenyu/admin/model/dto/AuthPathDTO.java
58.82% <0.00%> (-5.89%)
:arrow_down:

...nyu/plugin/motan/cache/ApplicationConfigCache.java
52.00% <0.00%> (-5.00%)
:arrow_down:

... and 26 more

Continue to review full report at Codecov.

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


Simulations directory error not found

I get an error that the simulations folder is not found when running zenbot from another folder.
Zenbot is installed as a global command. Error message (pastebin.com)

duplicates #317


CFE is missing compile time errors for partial instantiation with statically-known bounds

When the CFE injects a partial instantiation operator, it needs to check that the type arguments applied match the bounds of the target when the target is statically known (tearoffs against super, local functions, and toplevel functions). The Analyzer currently has the right behavior. This is tested by language_2/partial_instantiation_static_bounds_check_test and there are corresponding status entries in language_2.status. For reference, the test is:
topFn<T extends num>(T x) {
  print(T);
}

class C<T> {
  instanceFn<S extends T>(S x) {
    print(S);
  }
}

class D<T> extends C<T> {
  void foo() {
    void Function(int) k = instanceFn;  //# 03: compile-time error
  }
}

void main() {
  localFn<T extends num>(T x) {
    print(T);
  }
  void Function(String) k0 = localFn;  //# 01: compile-time error
  void Function(String) k1 = topFn;  //# 02: compile-time error
}

Note that the VM is currently applying optimizations under the assumption that the CFE does perform this check correctly.

This is a duplicate of #33308.


docs/config: Exclude sds configs from tests

In #28104 the sds configs were separated into yaml files - these seem to fail when tested - not sure how they got through presubmit
testing locally and it reliably fails, but only for the 2 configs excluded here

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

cc @kanurag94

starting a server with the oauth-sds-example.yaml  - i get ...
[2023-07-12 13:52:52.945][638982][warning][config] [source/common/config/grpc_subscription_impl.cc:120] gRPC config: initial fetch timed out for type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.Secret
[2023-07-12 13:52:52.945][638982][warning][config] [source/common/config/grpc_subscription_impl.cc:120] gRPC config: initial fetch timed out for type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.Secret
[2023-07-12 13:52:52.945][638982][info][config] [source/extensions/listener_managers/listener_manager/listener_manager_impl.cc:858] all dependencies initialized. starting workers
[2023-07-12 13:53:11.309][638982][warning][config] [./source/common/config/grpc_stream.h:191] StreamSecrets gRPC config stream to sds_server_uds closed since 33s ago: 14, upstream connect error or disconnect/reset before headers. reset reason: connection failure, transport failure reason: immediate connect error: No such file or directory

[2023-07-12 13:53:20.980][638982][warning][config] [./source/common/config/grpc_stream.h:191] StreamSecrets gRPC config stream to sds_server_uds closed since 43s ago: 14, upstream connect error or disconnect/reset before headers. reset reason: connection failure, transport failure reason: immediate connect error: No such file or directory

arggh - i see why it passed - the test was skipped as docs-only

pushing this through to fix CI

In my local instance, the envoy was not exiting, though the error was there. I was assuming this is correct since these configs do need some endpoint. Thanks @phlax for fixing CI, I will work on fixing the examples


Don't show DIAGNOSTIC message if run with it set

If there is an exception that isn't handled then even when running with DIAGNOSTIC= you get the message:
===> An unknown error was encountered. Run with DIAGNOSTIC=1 for more details.

I think this should be left out if its already enabled.

That one probably goes to rebar3 since that's a generic error line.

I think this is rebar3_hex, so agreed it should see if DIAGNOSTIC or even DEBUG was set, and not emit that detail if so.

Yea, this is in `rebar3_hex.
Rebar3 essage says "rebar_core" in the message.


Allow user and group in vrrp::script

Keepalived version 1.3.0 added options to run vrrp_scripts under different user/groups.  This PR adds the functionality to the vrrp::script define.

Looks good, thanks!


[FEATURE REQUEST] Add documentation or url to stopping and starting PostgreSQL cluster

Is your feature request related to a problem? Please describe.
Add documentation or url to stopping and starting PostgreSQL cluster. Describe shutdown and startup order.
Describe the solution you'd like

Stop secondary node
Stop primary node
...
Start primary node
Start secondary node

Describe alternatives you've considered
Find nice repmgr or postgresql manual and link in documentation

DoD checklist

[ ] Changelog updated
[ ] COMPONENTS.md updated / doesn't need to be updated
[ ] Schema updated / doesn't need to be updated
[ ] Feature has automated tests
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade

[ ] Idempotency tested
[ ] Documentation added / updated / doesn't need to be updated
[ ] All conversations in PR resolved
[ ] Solution meets requirements and is done according to design doc
[ ] Usage compliant with license
[ ] Backport tasks created / doesn't need to be backported

Paste url to repmgr documentation: https://repmgr.org/docs/dev/repmgrd-pausing.html


Use critical-section for heap locking, rename to embedded-alloc.

This uses critical_section::with instead of cortex_m::interrupt::free to acquire a critical section. This allows customizing the critical section implementation, to make it sound for multicore chips for example.
This is a breaking change, so it'll require a 0.5 release.
Interestingly this makes the crate not cortex-m specific anymore. Perhaps it could be renamed to something more general?
TODO

[x] Wait for cortex-m 0.7.6 release https://github.com/rust-embedded/cortex-m/pull/449

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

Maybe it should be embedded-alloc in line with other crates in this org?

Maybe add a critical-section feature upstream to linked_list_allocator instead?

Having customizable critical sections also benefits the use case of the nRF softdevice (which requires users never to turn off interrupts completely).
If this requires an API change, it might be worth considering whether that change could also be used for non-critical-section exclusitivness. A concrete setup I have in mind is always failing attempts to allocate from interrupts (which in setups without context switching would mean that little to no is needed; of course, if critical_section also allows such a null-implementation, all the better).

Discussed on today's WG meeting. chatlogs, it was decided to rename to embedded-alloc.

bors r+

@adamgreig could you add the cortex-m team as an owner on crates.io as well?

Thanks for the reminder, done.


Model tuning

We need to come up with an approach for tuning the model.
@dmontemayor Proposes that we try to tune the hyperparameters such that we maximize the sparsity -- potentially weighted by the reconstruction error

Transferred to https://github.com/Bishop-Laboratory/Cancer-Normal-Transformer-Analysis/issues/27