Stack overflow with cyclic dependency error

I'm now getting a crash on a project where rust-analyzer worked perfectly before.
VSCode output log:
[ERROR ra_project_model] cyclic dependency test-helpers -> traffic_mesh

thread '<unknown>' has overflowed its stack
fatal runtime error: stack overflow
[Info  - 9:56:00 AM] Connection to server got closed. Server will restart.
[Error - 9:56:00 AM] Request textDocument/semanticTokens/range failed.
Error: Connection got disposed.

The dependency is not actually cyclic btw. The compiler is compiling the project just fine.
This is the graph:
test-helpers -- dependency --> traffic_mesh
traffic_mesh -- test-dependency --> test-helpers

I'm running on Linux in Docker (DevContainers).

The cyclic dependency is not necessarily the problem causing the stack overflow (in fact, that it's logging about it means that it caught the cycle). Does the crash happen if you run rust-analyzer analysis-stats . in the project?

The dependency is not actually cyclic btw. The compiler is compiling the project just fine.

This can happen when a crate has a dev-depencency on itself. When testing Cargo just builds the crate twice. Once without #[cfg(test)] and one with.

Does the crash happen if you run rust-analyzer analysis-stats . in the project?

Nope.
Output:
[ERROR ra_project_model] cyclic dependency traffic_mesh -> test-helpers
Database loaded, 392 roots, 922.5233ms
Crates in this dir: 16
Total modules found: 96
Total declarations: 851
Total functions: 596
Item Collection: 23.670316s, 0b allocated 0b resident
Total expressions: 40616
Expressions of unknown type: 1606 (3%)
Expressions of partially unknown type: 1864 (4%)
Type mismatches: 67
Inference: 32.6445041s, 0b allocated 0b resident
Total: 56.3148737s, 0b allocated 0b resident

The dependency is not actually cyclic btw. The compiler is compiling the project just fine.

This can happen when a crate has a dev-depencency on itself. When testing Cargo just builds the crate twice. Once without #[cfg(test)] and one with.

Ok. is there a fix for it or can it just be ignored?

It can be ignored.

analysis-stats succeeding means the problem is unlikely to be related to the cyclic dependency; rather it's probably somewhere in some IDE code. If you can reproduce the crash, it would be great if you could get a stacktrace with gdb (though I think you'll need to use a self-built rust-analyzer because our releases don't have debug info?)

Closing as non-actionable, as this doesn't have a reproducible example.


Kaspersky catches pacman like a trojan

Today during the execution of the paсman -Suy the following happened:
demyanov-ia@HPN-230 /c/U/demyanov-ia> pacman -Suy
:: Synchronizing package databases...
 mingw32 is up to date
 mingw64 is up to date
 msys is up to date
:: Starting core system upgrade...
 there is nothing to do
:: Starting full system upgrade...
warning: grep: local (3.1-1) is newer than msys (3.0-2)
resolving dependencies...
looking for conflicting packages...

Packages (24) curl-7.63.0-1  gawk-4.2.1-2  gcc-libs-7.4.0-1  git-2.20.1-1  gnupg-2.2.12-1  heimdal-7.5.0-3  heimdal-libs-7.5.0-3  libassuan-2.5.2-1  libcurl-7.63.0-1  libevent-2.1.8-2  libgnutls-3.6.5-1  libgpg-error-1.33-1
              libhogweed-3.4.1-1  libnettle-3.4.1-1  libnghttp2-1.35.1-1  libopenssl-1.1.1.a-1  libssh2-1.8.0-2  nettle-3.4.1-1  openssh-7.9p1-2  openssl-1.1.1.a-1  perl-5.28.1-1  perl-Net-SSLeay-1.85-2  sed-4.7-1  wget-1.20-2

Total Download Size:    32.36 MiB
Total Installed Size:  173.32 MiB
Net Upgrade Size:       49.91 MiB

:: Proceed with installation? [Y/n] y
:: Retrieving packages...
 gcc-libs-7.4.0-1-x86_64
 libopenssl-1.1.1.a-1-x86_64
 heimdal-libs-7.5.0-3-x86_64
 libnghttp2-1.35.1-1-x86_64
 openssl-1.1.1.a-1-x86_64
 sed-4.7-1-x86_64
 libgpg-error-1.33-1-x86_64
 libssh2-1.8.0-2-x86_64
 libcurl-7.63.0-1-x86_64
 curl-7.63.0-1-x86_64
 gawk-4.2.1-2-x86_64
 heimdal-7.5.0-3-x86_64
 openssh-7.9p1-2-x86_64
 perl-5.28.1-1-x86_64
 perl-Net-SSLeay-1.85-2-x86_64
 git-2.20.1-1-x86_64
 libassuan-2.5.2-1-x86_64
 libhogweed-3.4.1-1-x86_64
 libnettle-3.4.1-1-x86_64
 libgnutls-3.6.5-1-x86_64
 nettle-3.4.1-1-x86_64
 gnupg-2.2.12-1-x86_64
 libevent-2.1.8-2-x86_64
 wget-1.20-2-x86_64
(24/24) checking keys in keyring
(24/24) checking package integrity
(24/24) loading package files
(24/24) checking for file conflicts
(24/24) checking available disk space
:: Processing package changes...
( 1/24) upgrading gcc-libs
( 2/24) upgrading libopenssl
( 3/24) upgrading heimdal-libs
( 4/24) upgrading libnghttp2

demyanov-ia@HPN-230 /c/U/demyanov-ia> pacman -Suy
      1 [main] C:\msys64\usr\bin\fish 13788 child_copy: linked dll bss write copy failed, 0x52693B000..0x52693B1B0, done 0, windows pid 14020, Win32 error 998
  14396 [main] C:\msys64\usr\bin\fish 13788 fork: child 5856 - couldn't copy linked dll data/bss, errno 11
      1 [main] C:\msys64\usr\bin\fish 13788 child_copy: linked dll bss write copy failed, 0x52693B000..0x52693B1B0, done 0, windows pid 14020, Win32 error 998
  14396 [main] C:\msys64\usr\bin\fish 13788 fork: child 5856 - couldn't copy linked dll data/bss, errno 11
Failed to execute process '/usr/bin/pkgfile'. Reason:
exec: unknown error (errno was 11)
pacman may be found in the following packages:

demyanov-ia@HPN-230 /c/U/demyanov-ia>

Known because of similar behavior like trojans and malware, but its not gonna be changed, since its a core functionality of pacman (and other tools, especially Ruby, Python and GnuPG get easily captured). Best advice is to put the entire C:\msys64\ onto exclusion list in Kaspersky, works this way in Avast too and maybe other anti-virus.
Development tools and package managers are not liked by most security software.


fix: DEV-1975: Taxonomy multi line items

dd support for super long taxonomy text wrapping multiple lines

/git merge develop


Improve type information of internal functions

These were "fallout" while working on KIP-392 (see #810), but can be treated as independent to hopefully simplify reviewing/merging.

Interesting. I didn't know you could import Typescript types from within JSDoc comments.

While trying to find a reference: There's probably a caveat here that this syntax comes from TypeScript, and right now the JSDoc people haven't implemented it (but it seems they're not actively against it either):

TypeScript reference for the syntax: https://www.typescriptlang.org/docs/handbook/jsdoc-supported-types.html#import-types
JSDoc plugin providing it when building documentation: https://github.com/SoftwareBrothers/better-docs#typedefimport
JSDoc issue tracking the implementation and discussing the approaches: https://github.com/jsdoc/jsdoc/issues/1645

Since we're not generating docs from the JSDoc comments, and only rely on it for IDE support, I think it's fine. VSCode seems to mostly get it (although amusingly hovering over @type shows you this person's github profile 😂)


No visual differentiation between active and disabled links in dark mode dropdown menus

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
In 5.3 Alpha 1, there is no visual differentiation between active and disabled links in dark mode dropdown menus. This has been tested in Safari on iOS and in Chrome/Edge on Windows.
Active links are set to --bs-dropdown-link-color, which itself inherits --bs-body-color. Disabled links are set to --bs-dropdown-link-disabled-color which is set to #adb5bd. In dark mode, --bs-body-color is also set to #adb5bd.
This can be seen on the Bootstrap 5.3 doco page for dropdowns: https://getbootstrap.com/docs/5.3/components/dropdowns/#disabled
Reduced test cases
Issue demo here:
https://codepen.io/Sanx78/pen/YzjjLEy
My personal fix (likely to be hated as it is a large sledgehammer for a very small nut) is here:
https://codepen.io/Sanx78/pen/GRBBdOj
However, what I imagine will be the "correct" way of fixing it is to make --bs-dropdown-link-disabled-color theme aware.
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Microsoft Edge
What version of Bootstrap are you using?
5.3.0 Alpha 1

I would love to work on this issue

Sorry, https://codepen.io/Sanx78/pen/GRBBdOj updated to actually contain the proposed fix.

Oh okay cool
I will work on this now

I'm planning on adding the changes to bootstrap/scss/_dropdown.scss would that be fine to fix this issue

There is a variable that defines the color of the disabled dark dropdown

@Sanx78 Please have a look at my pull request and let me know if it solves your issue


More limitations to uncover and document

Using Hotrestart with an existing project does not work and ends with a native crash. I guess there are more limitations to uncover.
Kudos for this amazing feature though.

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

ID: 8f3eb808-8e6d-4e07-fc75-8d79a7249265
Version Independent ID: dbb2b9a9-87a6-e473-50e6-f88435d49e88
Content: Xamarin Hot Restart - Xamarin
Content Source: docs/xamarin-forms/deploy-test/hot-restart.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @maddyleger1
Microsoft Alias: maleger

Hi @omatrot
Thanks for the feedback!

Hi @omatrot
Thanks for the feedback!


rdfs:subclassOf apparently not honored during SHACL validations

Hello, I'm using the SHACL validator as documented here with RDF4J version 3.4.0.
I'm using the schema.org shapes available here.
I have these triples:
@base <https://emmanueloga.com/> .
@prefix : <https://emmanueloga.com/> .
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .
@prefix rf: <https://eoga.dev/rainbowfish#> .
@prefix schema: <http://schema.org/> .
@prefix xsd: <http://www.w3.org/2001/XMLSchema#> .

<me>
    a schema:Person ;
    schema:name "Emmanuel Oga" ;
    schema:familyName "Oga" ;
    schema:givenName "Emmanuel" ;
    schema:jobTitle "Software Engineer" ;
    schema:knowsLanguage "en-US", "es-ES" ;
    schema:homeLocation :home-location .

<home-location>
    # a schema:ContactPoint ;
    a schema:PostalAddress ;
    schema:addressLocality "San Francisco" ;
    schema:addressRegion "California" ;
    schema:addressCountry "United States" .

... for which RDF4J fails to validate schema:homeLocation (see validation report at the bottom). Note that I commented out a schema:ContactPoint on the data. If I use ContactPoint, the data validates fine. But since the schema.org shapes say:
schema:PostalAddress
  a rdfs:Class ;
  a sh:NodeShape ;
  rdfs:subClassOf schema:ContactPoint ;
  # ... more properties here ...

schema:Person-homeLocation
  a sh:PropertyShape ;
  sh:path schema:homeLocation ;
  sh:description "A contact location for a person's residence."^^rdf:HTML ;
  sh:name "homeLocation" ;
  sh:or (
      [
        sh:class schema:ContactPoint ;
      ]
      [
        sh:class schema:Place ;
      ]
    ) ;
.

... I suspect rdfs:subClassOf  is not being considered. The documentation says "By default the ShaclSail supports the simple rdfs:subClassOf reasoning", and I verified this is enabled on my code (isRdfsSubClassReasoning returns true).
Maybe I'm misunderstanding how this is supposed to work? I was assuming since PostalAddress is defined a subclass of ContactPoint, I should be able to provide a PostalAddress where a ContactPoint is expected.
Thank you!
RD4J Validation Report:
@prefix : <https://emmanueloga.com/> .
@prefix owl: <http://www.w3.org/2002/07/owl#> .
@prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#> .
@prefix rf: <https://eoga.dev/rainbowfish#> .
@prefix schema: <http://schema.org/> .
@prefix sh: <http://www.w3.org/ns/shacl#> .
@prefix xsd: <http://www.w3.org/2001/XMLSchema#> .
@prefix fx: <https://eoga.dev/fishx#> .

_:node1ejqrsupkx1939 a sh:ValidationReport;
  sh:conforms false;
  <http://rdf4j.org/schema/rdf4j#truncated> false;
  sh:result _:node1ejqrsupkx1940 .

_:node1ejqrsupkx1940 a sh:ValidationResult;
  sh:focusNode :me;
  sh:sourceConstraintComponent sh:OrConstraintComponent;
  sh:sourceShape schema:Person-homeLocation;
  sh:resultPath schema:homeLocation;
  sh:detail _:node1ejqrsupkx1941 .

_:node1ejqrsupkx1941 a sh:ValidationResult;
  sh:focusNode :me;
  sh:sourceConstraintComponent sh:ClassConstraintComponent;
  sh:sourceShape _:node1ejqrsup0x1998;
  sh:resultPath schema:homeLocation;
  sh:value :home-location;
  sh:detail _:node1ejqrsupkx1942 .

_:node1ejqrsupkx1942 a sh:ValidationResult;
  sh:focusNode :me;
  sh:sourceConstraintComponent sh:ClassConstraintComponent;
  sh:sourceShape _:node1ejqrsup0x2000;
  sh:resultPath schema:homeLocation;
  sh:value :home-location .

@EmmanuelOga can you try adding the rdfs:subClassOf statements as part of your data instead of the shapes (eg. don't add it to the shacl graph)?

@hmottestad that was exactly what was happening:
                try {
                    conn.add(shapes) // <<-- Adding this line solves the issue!
                    conn.add(shapes, RDF4J.SHACL_SHAPE_GRAPH)
                    conn.commit()
                    println("OK")
                } catch (exception: RepositoryException) { ...

Not sure if there are any lessons that I should get from this... should I always commit the data to both the default graph and the shapes graph in the general case?
thx again

I think we will call this a bug. I should be retrieving rdfs:subClassOf statements from both the base sail and the SHACL graph.

@EmmanuelOga https://rdf4j.org/release-notes/3.4.3/


[Metal] CommandList.UpdateBuffer with a byte size of 0 causes a crash

void CommandList.UpdateBuffer<VertexPositionColor>(DeviceBuffer buffer, uint bufferOffsetInBytes, ref VertexPositionColor source, uint sizeInBytes)
This operation causes a crash with sizeInBytes=0 on Metal but not in OpenGL.
Also, the GraphicsDevice variant seems to work. At least it doesnt crash the application.
Sadly I dont get a debug trace.
The apple crash report has this trace
Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   com.apple.driver.AppleIntelKBLGraphicsMTLDriver	0x00007fff2d804778 CHAL_INTEL::ChalContext::ChalExecuteCopyBufferData(MTLIGAccelBuffer*, unsigned int, MTLIGAccelBuffer*, unsigned int, unsigned int) + 84
1   com.apple.driver.AppleIntelKBLGraphicsMTLDriver	0x00007fff2d804704 IGAccelBlitCommandEncoder::copyBufferToBuffer(MTLIGAccelBuffer*, unsigned int, MTLIGAccelBuffer*, unsigned int, unsigned int) + 28
2   com.apple.driver.AppleIntelKBLGraphicsMTLDriver	0x00007fff2d8045f8 -[MTLIGAccelBlitCommandEncoder copyFromBuffer:sourceOffset:toBuffer:destinationOffset:size:] + 288

Thanks for the info. This should be fixed on master now -- let me know if you're still seeing it after updating.

Hey, copyBuffer is fixed but not updateBuffer :sweat_smile:

@geoeo Hmm, you're still getting a crash? From what I saw, CommandList.UpdateBuffer was only crashing because it internally used CopyBuffer. I'll take a look later today if there's still an issue.

yeah afraid so. Exact same error.

Interesting. I did try that initially, but it didn't crash on my machine. When I looked at your stack trace, it indicated copyBufferToBuffer was causing the problem, which I know will be called in a CommandList.UpdateBuffer call, which DOES crash on my machine. In any case, I think the fix is still to just ignore calls to GraphicsDevice.UpdateBuffer if the size is 0.

Okay, I was able to get a repro after all, and the problem should be fixed on the master branch.

Works fine now.


depth loss computation

Hi,
I use lidar scan to generate depth images. As the lidar is less dense than the image resolution, a large part of my depth images are empty and filled with 0.
In urban_radiance_field_depth_loss and ds_nerf_depth_loss function, a mask is computed to ignore depth estimation where it equal to 0:
depth_mask = termination_depth > 0
...
loss = (expected_depth_loss + line_of_sight_loss) * depth_mask
But at the end we return:
return torch.mean(loss)
So the loss value will depend of the density of the depth image. In my case, with 1024 pixel rays sampled, around twenty depths are available.
Do you think compute the loss as below would be more appropriate ?
return torch.mean(loss[depth_mask])
thanks

I check the original paper for the urban_radiance_field_depth_loss and the loss is given with the expectation of the lidar depth, so I think your method is more appropriate. Anyway the loss is scaled by a constant and the training process may no be greatly influenced.

thanks @KevinXu02 , about the scale sure, I needed to adjust it to get the appropriate convergence.
Do you think I push a PR for that ?


Reintroduce JSLint as a prefered linter

and disable temporary the no-trailing-spaces and eol-last eslint rules
This is based on the discussion in: https://github.com/adobe/brackets/pull/11998
Actually the ESLint checks are ATM weaker than JSLint so there shouldn't be any problems.

Thanks @ficristo .


Services running in WSL 2 no longer accessible from Windows

Discussed in https://github.com/microsoft/WSL/discussions/9744

Originally posted by Trey96 March  6, 2023
Not sure if anyone else is having this problem, but I searched and wasn't able to find anything related to my issue. It seems like the WSL localhost is no longer being shared with the Windows localhost, and I cannot access WSL services from Windows via localhost.
I am running multiple domains on Nginx inside of WSL 2. In my Windows hosts file I am pointing all of those domains to <IP_ADDRESS>. Until recently that was working just fine. However it has stopped working completely, and when I try to connect to the domain from the browser (Chrome) I get a connection refused error. Further investigation reveals there are no ports listening on 80 for <IP_ADDRESS>, in fact nothing appears to be active on <IP_ADDRESS> anymore, especially WSL which used to be active. It seems WSL 2 has stopped binding to localhost but I am not sure why.
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
Windows version: 10.0.22621.1344 (Windows 11)
Distro: Ubuntu 20.04

I recently installed Docker on my computer which runs on WSL. So I am not sure if that might have caused problems. Though I have since uninstalled Docker, but it had not affect. I am not sure if this is a bug with WSL or something I am doing wrong.
Any help identifying what might be the problem or any pointers on how to resolve it (or troubleshoot further) is much appreciated!
Log Files:
WslLogs-2023-03-08_18-58-55.zip

Please use the issue template going forward. Please provide the output of ss -lntp on the guest(wsl), and netstat -an | findstr /c:"80" | findstr /c:"LISTENING" on the host(windows)

@pmartincic, thanks for the quick reply!
The following is the command output:
ss -lntp (WSL)

netstat -an | findstr /c:"80" | findstr /c:"LISTENING" (Windows)

Can you provide /logs?

wsl --shutdown
then start collecting logs
Check that it's broken
Stop the log collection

Smoke test on my VM seems ok?

Got a similar problem.
I've started a grpc service in wsl2, and I can access it from wsl inside. But LAN computors can't access it even after I configured port-forwarding by:
netsh interface portproxy add v4tov4 listenport=50066 listenaddress=<IP_ADDRESS> connectport=50066 connectaddress=<IP_ADDRESS>
The error says:
UNAVAILABLE: Network closed for unknown reason

hi，I had the same problem，can't connect to grpc server written by golang running WSL2, but it fine running windows.
how can I solve it？

I'm sorry, I've gotten routed elsewhere and won't be able to look into this for the time being.

@pmartincic
This seems to be issue for a lot of people - in the end, this render WSL useless in high load scenarios...

@sec, I agree with you. Unfortunately I've gotten routed elsewhere. Your logs have something interesting in them, won't be able to look much right now.
@trey9
Looks like log collection started roughly half a minute after you started the VM. Whatever I'm looking for is probably in that half minute before you started collecting logs.
wsl --shutdown
then start collecting logs
Check that it's broken
Stop the log collection

@pmartincic
I apologize about that, not sure how that happened. But I uploaded the updated logs - Let me know if there is anything else you need

az login redirect fails with 1.1.3, 1.1.5, 1.1.6 & 1.1.7
Had to rollback to 1.0.3
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>-1
WSLg version: 1.0.51
MSRDC version: 1.2.3770
Direct3D version: 1.608.2-61064218
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.23424.1000

Seems to be working for me now, after updating
 Get-Process -Id (Get-NetTCPConnection -LocalPort 80).OwningProcess

Handles  NPM(K)    PM(K)      WS(K)     CPU(s)     Id  SI ProcessName
-------  ------    -----      -----     ------     --  -- -----------
    198      32     3364      10176       0.19  18008   3 wslrelay

netstat -an | findstr /c:"80" | findstr /c:"LISTENING"

  TCP    <IP_ADDRESS>:7680           <IP_ADDRESS>:0              LISTENING
  TCP    <IP_ADDRESS>:8090           <IP_ADDRESS>:0              LISTENING
  TCP    <IP_ADDRESS>:80           <IP_ADDRESS>:0              LISTENING
  TCP    [::]:7680              [::]:0                 LISTENING

WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.51
MSRDC version: 1.2.3770
Direct3D version: 1.608.2-61064218
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.22621.1635

Thanks @Trey96 for the update.
The fix was in 1.2.1

Hi.
@pmartincic  Sorry but this was not fixed. It's still an issue even after recent update to <IP_ADDRESS>.
Network is crashing under high load and the only solution is to restart WSL2, but then again, when docker containers are run and accessed from Windows host, network will be down again.

@pmartincic I have created sample code to repro this issue - https://github.com/sec/wsl-network-crash-test - should I open new issue or can we reopen this one, as it's still the same issue?

Same issue here, API running on wsl2, app running on windows. It worked for a few months, windows system crashed and after restart it doesn't work anymore, throws ERR_CONNECTION_REFUSED instead.


[Question] unable to re add portainer integration

Hi,
Was able to add the integration before and removed it since I was having an issue with duplicate instance on Home Assistant (my fault). now when I try to add it bac it gives me this error:

tried port 9443 and port 9000, https and http uncheck on SSL verify Cert. Tried to generate mutiple API but still the same issue. Is there anything else I can do?
Also both HA and Portainer is running on the same machine. Thanks!

"If you require HTTP port 9000 open for legacy reasons, add the following to your docker run command:
-p 9000:9000"
This is needed to do the "server:9000" the http port is not turned on. Steps below I just did:
docker stop portainer
docker rm portainer
docker pull portainer/portainer-ce:2.21.4
docker run -d -p 8000:8000 -p 9000:9000 -p 9443:9443 --name=portainer --restart=always -v /var/run/docker.sock:/var/run/docker.sock -v portainer_data:/data portainer/portainer-ce:2.21.4

Hi, don't prefix host with http(s):// and try it again.

yea, host should never contain protocol prefix. if that does not work, that means you have networking issue.


'wp-content/uploads' symbolic link creation fails

Attempting to follow the instructions at http://docs.aws.amazon.com/elasticbeanstalk/latest/dg/php-hawordpress-tutorial.html, environment creation consistently fails when attempting to create the symbolic link 'wp-content/uploads'.
Relevant log entries are:
[2017-04-04T16:58:42.918Z] INFO  [2083]  - [Application deployment First<EMAIL_ADDRESS>: Starting activity...
[2017-04-04T16:58:43.648Z] INFO  [2083]  - [Application deployment First<EMAIL_ADDRESS>: Completed activity. Result:
  ++ /opt/elasticbeanstalk/bin/get-config container -k app_user
  + EB_APP_USER=webapp
  ++ /opt/elasticbeanstalk/bin/get-config container -k app_base_dir
  + EB_APP_BASE_DIR=/var/app
  ++ /opt/elasticbeanstalk/bin/get-config container -k app_staging_dir
  + EB_APP_STAGING_DIR=/var/app/ondeck
  ++ /opt/elasticbeanstalk/bin/get-config container -k app_logs_dir
  + EB_APP_LOGS_DIR=/var/app/support/logs
  + cd /var/app/ondeck
  + chown -R webapp:webapp /var/app
  + chown -R webapp:webapp /var/log/httpd
  + '[' -f app/SymfonyRequirements.php ']'
[2017-04-04T16:58:43.648Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/AppDeployPreHook] : Completed activity. Result:
  Successfully execute hooks in directory /opt/elasticbeanstalk/hooks/appdeploy/pre.
[2017-04-04T16:58:43.649Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild] : Starting activity...
[2017-04-04T16:58:44.128Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild/Infra-EmbeddedPostBuild] : Starting activity...
[2017-04-04T16:58:44.130Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild/Infra-EmbeddedPostBuild/postbuild_0_nkwebsite_wordpress] : Starting activity...
[2017-04-04T16:58:44.785Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild/Infra-EmbeddedPostBuild/postbuild_0_nkwebsite_wordpress/Command 1chown] : Starting activity...
[2017-04-04T16:58:44.806Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild/Infra-EmbeddedPostBuild/postbuild_0_nkwebsite_wordpress/Command 1chown] : Completed activity.
[2017-04-04T16:58:45.460Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild/Infra-EmbeddedPostBuild/postbuild_0_nkwebsite_wordpress/Command 2link] : Starting activity...
[2017-04-04T16:58:45.493Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild/Infra-EmbeddedPostBuild/postbuild_0_nkwebsite_wordpress/Command 2link] : Activity execution failed, because: ln: failed to create symbolic link 'wp-content/uploads': No such file or directory
   (ElasticBeanstalk::ExternalInvocationError)

[2017-04-04T16:58:45.493Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild/Infra-EmbeddedPostBuild/postbuild_0_nkwebsite_wordpress/Command 2link] : Activity failed.
[2017-04-04T16:58:45.493Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild/Infra-EmbeddedPostBuild/postbuild_0_nkwebsite_wordpress] : Activity failed.
[2017-04-04T16:58:45.493Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild/Infra-EmbeddedPostBuild] : Activity failed.
[2017-04-04T16:58:45.529Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0/EbExtensionPostBuild] : Activity failed.
[2017-04-04T16:58:45.530Z] INFO  [2083]  - [Application deployment First Release@1/StartupStage0] : Activity failed.
[2017-04-04T16:58:45.530Z] INFO  [2083]  - [Application deployment First Release@1] : Completed activity. Result:
  Application deployment - Command CMD-Startup failed
[2017-04-04T17:00:07.696Z] INFO  [2323]  - [CMD-TailLogs] : Starting activity...
[2017-04-04T17:00:07.696Z] INFO  [2323]  - [CMD-TailLogs/AddonsBefore] : Starting activity...
[2017-04-04T17:00:07.696Z] INFO  [2323]  - [CMD-TailLogs/AddonsBefore] : Completed activity.
[2017-04-04T17:00:07.696Z] INFO  [2323]  - [CMD-TailLogs/TailLogs] : Starting activity...
[2017-04-04T17:00:07.697Z] INFO  [2323]  - [CMD-TailLogs/TailLogs/TailLogs] : Starting activity...

Thanks for documenting this, just made my day 👍


Simplify C# print methods

Extracts the parameters logic to a single method
Simplify the handling of null parameters

That's because in the 3.x branch the TargetFramework is netstandard2.0 while in master it's netstandard2.1.
3.x
https://github.com/godotengine/godot/blob/d9a07d1528a438ddf97b1a9cd37a96cbfa3a92ce/modules/mono/glue/GodotSharp/GodotSharp/GodotSharp.csproj#L7
master
https://github.com/godotengine/godot/blob/769de0c35f10780edc1a670b09f63fc730fefc39/modules/mono/glue/GodotSharp/GodotSharp/GodotSharp.csproj#L7
The AllowNullAttribute is only available from netstandard2.1:
https://docs.microsoft.com/en-us/dotnet/api/system.diagnostics.codeanalysis.allownullattribute?view=net-5.0
I guess I could remove it to make it easier to cherry-pick but at one point I'd like to annotate the  API with nullable attributes to allow users who enable it in their projects get warnings and such.

IIRC @neikeq said that he doesn't want the 3.x and master codebases to diverge too much for the time being.

I ended up removing it, we can annotate the API later.

Thanks!

Cherry-picked for 3.4.


Fixing up merge issues + updating index/map/toc

Docs Build status updates of commit 0f055fb:
:warning: Validation status: warnings

File
Status
Preview URL
Details

advocates/index.html.yml
:warning:Warning
View
Details

advocates/aysegul-yonet.yml
:white_check_mark:Succeeded
View

advocates/map.yml
:white_check_mark:Succeeded
View

advocates/sarah-lean.yml
:white_check_mark:Succeeded
View

advocates/toc.yml
:white_check_mark:Succeeded
View

advocates/index.html.yml

Line 10, Column 8: [Warning-file-not-found] Invalid file link: './map'.
Line 12, Column 8: [Warning-file-not-found] Invalid file link: './tweets'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add Update Dependencies button to Package Dependencies title bar

This is probably the operation I do the most often in relation to swift package manager, making it easy of access would be a good thing
I've included the package-lock.json as it has the wrong package name

Now that resolves happen automatically anyway, maybe we can just change that "Refresh" button to do an update instead of a resolve?

I was going to suggest having a config option to enable/disable the automatic resolve. If you are working with large SPM projects it can be a pain.

Another thing I think the package resolve is not working completely. In that it does unnecessary resolves. The resolve should only be triggered by a change in Package.swift. It seems to trigger on a change to Package.resolved.  I'll add an issue related to this.

The latter was intentional. @0xTim mentioned that users may pull in a newer version of Package.resolved, in which case there should be an additional resolve.
But it shouldn't be too hard to make both features optional.

The latter was intentional. @0xTim mentioned that users may pull in a newer version of Package.resolved, in which case there should be an additional resolve.
But it shouldn't be too hard to make both features optional.

That doesn't make sense swift package resolve updates the Package.resolved file. Why would you run it again if it has been updated by an external process. You can see this happen when you press the swift package update button, it runs an unnecessary resolve immediately after

You will want to rebuild the package view based on changes to Package.resolved

Is it OK if I remove the Resolve button in favor of the Update button? Resolves should mostly happen automatically, so there shouldn't be a need for a button for it.
There's always the Resolve task should a user want to run it without opening a command line.

+1 from me

Is it OK if I remove the Resolve button in favor of the Update button? Resolves should mostly happen automatically, so there shouldn't be a need for a button for it.

done


https://github.com/MicrosoftDocs/microsoft-365-docs/files/11494419/debug.log

[Enter feedback here]

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

ID: e4e2ac78-e08b-d7c8-5eec-baf9e9c13b62
Version Independent ID: 66014f69-00c9-0e9b-fe0a-d0e10f2ca5eb
Content: About registration numbers and under-review notifications
Content Source: microsoft-365/commerce/about-registration-numbers.md
Service: microsoft-365-business
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee
debug.log

@myu1003 Thank you for your feedback. I reviewed the log and if you have any technical queries, I'd recommend you contact support via the need help section on the admin center, so that they can resolve the issue.
Hope this helps!
Thanks
Sri


Ability to remove formatting

At first place, thanks! Very nice library.
I need some features and probably other people will do.
I need to be able to on/off formatCardNumber

Thanks! I'm going to close this as a duplicate of #129.


Add a Gitter chat badge to README.md

Alphasquare/Mxious-src now has a Chat Room on Gitter
@Alphapixels has just created a chat room. You can visit it here: https://gitter.im/Alphasquare/Mxious-src.
This pull-request adds this badge to your README.md:

Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

I'm afraid I already did this :s


No pokemon showing up. Need some update?

Hi guys!
Am I missing something? the map is not showing pokemons in lastest version but when I toggle developer tool it is finding pokemons. I tryied swithing "hide pokemon" in Options but no working for me.
I hope some advise thanks
I'm running Win7 x64
map scanning at UCF

developer console finding pokemons

same no pokemon

I noticed in some pokestops says name of pokemons

I see Pokemon around that area right now.

@MacDogg Are you using 0.1.6? what setup do you have?
Sometimes when I switch PTC user I got javascript error

Dang, now I am not showing any Pokemons at my location.  I wonder if it soft bans us (PTC login) if you bounce around to different places?
I have the latest version.  I also get the javascript error but it still lets me in and scan.

Experiencing the same issue actually. After a certain amount of time, the refreshing does not show any kind of pokemon in the map. Tried to logout and relogin but got a java runtime error. Loging out does not seem to resolve the problem. Is there some kind of cache to be emptied?

getting error after change to a new PTC user

Having the same issues as everyone. No idea why though. Any help at all would be welcome (this isn't the first issue that reports this case)
So hoping a solution would be found. (if anyone can think of a log file that could mention something) that could be helpful and such.
@mchristopher


chore(ci): add coverage code for python

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Codecov Report
Patch coverage has no change and project coverage change: -10.58 :warning:

Comparison is base (4eb0f61) 61.40% compared to head (42d189a) 50.83%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #7681       +/-   ##
===========================================
- Coverage   61.40%   50.83%   -10.58%
===========================================
  Files         188      353      +165
  Lines        4071    35386    +31315
  Branches      949        0      -949
===========================================
+ Hits         2500    17988    +15488
- Misses       1571    17398    +15827

Flag
Coverage Δ

pytest-testIntegration
50.83% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 541 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Timesheet - 14/07/2021 - TristanS

Task 1
Reviewed consolidated timesheets documents
Proof of Work

https://docs.google.com/spreadsheets/d/1I9NEEx0JW6sOcrYXGjzmqqoH7sQyj-SUFFpasOF_14s/edit#gid=0
https://docs.google.com/spreadsheets/d/1TQF-4XKtKCtF6PNptVgSQ8DPBduKPo-V_tvpC1WlDpc/edit#gid=0
https://docs.google.com/spreadsheets/d/1G0jnLpVd5cFxSFABIl75rM7r8-RGdmubkFEWdhFt5G0/edit#gid=0

Time Spent
.5 hours

Task 2
Attended review meeting and prepared for it before hand.
Proof of Work

https://docs.google.com/document/d/1gqJY1o1MYVM7iJaSxsGI59KnujG23tY2Lj3q2l3-7Rs/edit

Time Spent
1 hours

Total Time Spent
1.5 hours
Account Number
6d4e7ec412472b5cfabb234555bc37f9a5a1a75a4926f9fee2a54d3e070f4b3f

@manishram, I assume this means approved?

Haha.. my bad.


[core] Properly use BABEL_ENV test and development

Some scripts were using test when they're not actually testing something but are for development purposes.
This shouldn't change any transpilation but will be important when starting to migrate to ES6 modules.

This broke yarn benchmark:browser


liquibase-core-3.6.3 Vulnerabilities

The dropwizard-bom version: 1.3.16 and 1.3.17 (io.dropwizard:dropwizard-bom:jar:${version}) is using liquibase-core-3.6.3.jar which is having the following vulnerability issues:: CVE-2018-14042, CVE-2019-8331, CVE-2018-14041, CVE-2018-14040, CVE-2015-9251, CVE-2019-11358. Is there any plan to mitigate this issue?
Please suggest!

@beena-yatin-kanyal Dropwizard 2.0.0 is going to be released soon, which is using the latest stable version of Liquibase.
Would upgrading your application to Dropwizard 2.0.0 be an option?

Also, every CVE you've linked has been issued for the CSS framework Bootstrap and not for Liquibase.

@joschi - Yes, I understand. But this is coming from the "liquibase-core" jar present under "dropwizard-jdbi"? Will this be addressed and fixed? Also, I don't see any stable version for Dropwizard 2.0.x? All of them are release candidates.

@beena-yatin-kanyal These are false positives.

Also, I don't see any stable version for Dropwizard 2.0.x? All of them are release candidates.

The release candidates are pretty stable.


Fix updating return type for func with check expr

Purpose
$subject
Fixes #37153
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[x] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Closing in favor of #37308


revise superclassOf for germ cell

close #818

Reasoned diff (on simple file, so direct relations) shows no diff with this change in CL. I'll approve since it is what is agreed by @dosumis and @balhoff - will let @nicolevasilevsky decide on if to merge or wait :) thanks!

Merging in given the amount of time that has passed


Add instructions to enable anypoint monitoring

@sundippatel - PTAL and make sure I didn't introduce any errors here - thank you!

Looks great, thanks!


App crashes with some RSS feed websites.

Steps to Reproduce

Import OPML file
The file is imported partially
Try to add the website rss manually
The app crashes.

Expected behavior
OPML import completed and the website available to read and see on the app.
I used this website as a example: https://www.brasildefato.com.br/rss2.xml
Screenshots or screen recording
https://github.com/user-attachments/assets/eb204173-b42c-4428-b437-1c7609624051
Device, Operating system, and app version
Lenovo Tab P11 Plus, Android 12, 2024.07.1014

@goliv04053 Thanks for the video. I merged a fix for this feed that will be included in the next GitHub release.

@jocmp Thanks

Fix is live in 2024.07.1017-dev

@jocmp Tested here and it's fully working


✨ feat(FlowView): 组件文档

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

Comparison is base (a22ede1) 80.75% compared to head (5fa26bb) 80.75%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #27   +/-   ##
=======================================
  Coverage   80.75%   80.75%
=======================================
  Files           3        3
  Lines         239      239
  Branches       32       32
=======================================
  Hits          193      193
  Misses         46       46

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


Reading PhysicsRayQueryParameters3D.Exclude from C# causes Godot to exit

Godot version
4.0.beta3.mono
System information
Fedora 36
Issue description
Reading PhysicsRayQueryParameters3D.Exclude appears to cause Godot to simply exit. Setting it to a new value doesn't, actually casting rays doesn't, but reading or modifying it does. Godot simply exits without any sign of an error.
This appears to be a C# issue. Doing something equivalent in GDScript works correctly.
Steps to reproduce

Create a 3D scene
Add this to any node:

public override void _PhysicsProcess(double delta)
{
	var rayParams = PhysicsRayQueryParameters3D.Create(
		from: new Vector3(0.0f, 0.0f, 0.0f),
		to: new Vector3(1.0f, 1.0f, 1.0f));
	GD.Print(rayParams.Exclude); // This line is the problem.
}

Run the scene and watch as it immediately exits (without throwing any errors).

Minimal reproduction project
No response

I'm not sure if this is really a physics problem as the same code causes a crash when in _Ready and you never have to actually use the ray parameters. I think this is an issue with the C# code, but I don't know enough about how that works to debug any further.
I checked PhysicsRayQueryParameters2D and it doesn't seem to have this problem.

I looked into this, the get_exclude method returns Vector<RID>:
https://github.com/godotengine/godot/blob/ad3f2a234082f0d62820094199c05855115c1341/servers/physics_server_3d.h#L850
The vector type is a C++ type, in scripting this type does not exist so we use Array. This means that the ptrcall goes through PtrToArg::encode to convert the vector returned by get_exclude into an array that can be returned to a scripting language.
https://github.com/godotengine/godot/blob/ad3f2a234082f0d62820094199c05855115c1341/core/variant/method_ptrcall.h#L306-L313
The PtrToArg::encode method takes a p_ptr parameter which is the pointer to an Array where the contents of the vector will be copied to, in order to do this it first calls Array::resize to ensure the size of the array is the same as the size of the vector. The Array::resize method just resizes the internal vector used by Array to store its elements, the problem is if the array hasn't been initialized this internal data is null.
So, when calling this method from C# it uses ptrcall with a pointer to a godot_array as the return argument but this array is uninitialized so when the method tries to copy the vector to the array it finds the unexpected null data and crashes. The solution would be to initialize the array properly (I see that GDScript seems to use VariantInternal::initialize for this).
    internal static unsafe Godot.Collections.Array godot_icall_0_96(IntPtr method, IntPtr ptr) {
        godot_array ret = default;
        if (ptr == IntPtr.Zero)
            throw new ArgumentNullException(nameof(ptr));
        NativeFuncs.godotsharp_method_bind_ptrcall(method, ptr, null, &ret);
        return Godot.Collections.Array.CreateTakingOwnershipOfDisposableValue(ret);
    }

However, it would be unfortunate to initialize an array that doesn't need to be initialized. The array that is passed to ptrcall doesn't need to be initialized if the method returns Array or TypedArray because in those cases it can just assign the returned value to the return argument pointer directly, it's only needed for Vector because since this type doesn't exist in scripting it needs to be copied to an array first and this array needs to be initialized. Since PtrToArg::encode assumes that it's already initialized, we need to initialize it before that.
In this particular case though, the 2D version exposes get_exclude as a TypedArray<RID> which is why, as you found out, it works there.
https://github.com/godotengine/godot/blob/ad3f2a234082f0d62820094199c05855115c1341/servers/physics_server_2d.h#L635
So I think we should change the bindings in the 3D version to be consistent with 2D and also use a TypedArray<RID> which would have the side-effect of fixing this issue, but that doesn't mean the original issue with Vector<T> parameters should be ignored.


CDI-690 - update assertions and introduce new requestContext tests in

SE.

Merged. Thanks for review.


Contact us

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
In the footer section the contact us page is not opening. I'll make a contact us page that open if the user tap on the contact us by which we can take users data and feedback regarding product or any issues . So please assign me this task
Uploading VID20240512181044.mp4…
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assign me this task I'll do it in the best way I can .

@arghadipmanna101 assign me this task I'll do it in the best and efficient way I can.

Please assign this task to me i 'll do it properly.

plss assign me this issue

i completed this task and facing some issues and want some info like how i can push changes to the original one .


Filter for "not empty" columns

Hi @ErisDS
I'm trying to filter posts based on a column (called video) where I want all the posts where the video column is not empty. Actually, I tried using the video:-null approach, but that won't work for posts where I removed a given video, because the column is empty now instead of null.
Can't find a way to filter only for posts where a given column value isn't empty.
Can you please help?

@marcuspoehls By all means, submit a fix to GQL. However, as video is your own custom field, perhaps you could also fix it by ensuring the field is set back to null?
I'm afraid I'm crazy busy with Ghost 1.0.0 and not going to have any more time for GQL until we next work on the API, which is scheduled for after the release of 1.0.0.

@ErisDS Your hint to set back the field to null was the piece I missed. Thanks for your reply and help!


Tenant service can generate harmful org and bucket IDs

While investigating flaky tests @adrian-thurston bisected a fault down to the following commit:
https://github.com/influxdata/influxdb/commit/d720661e77564105bfe371a392deff165ce49562
It introduced the new tenant service, which in turn introduced a new implementation for the bucket and organizations services.
He observed that the pattern in IDs generated for buckets had changed. Now the IDs being generated were very similar to one another.
This led us to note that the tenant service was using a different influxdb.IDGenerator implementation:

Using snowflake now: https://github.com/influxdata/influxdb/blob/master/tenant/storage.go#L24
Used to use rand implementation : https://github.com/influxdata/influxdb/blob/master/kv/service.go#L62

The rand implementation sanitized away some potentially harmful characters:

\

,

Further investigation showed that this is problematic when the following sequences occur in either a bucket or org ID:

\  (5c20)
\" (5c22)
\= (5c3b)
\, (5c2c)

Since organization and bucket IDs are concatenated to generate the measurement used in the storage engine. This causes an issue within TSI. Which presents itself as empty responses when querying data that was written to the bucket in question.
Data writes successfully, however, it cannot be retrieved again. As it has been stored at an unreachable location.
We need to go back to the original rand.OrgBucketID generator:
https://github.com/influxdata/influxdb/blob/master/rand/id.go#L34

To clarify, is it the sanitation or the lack of sanitation that caused the issue?

@nathanielc lack of sanitation that caused the issue


Improve the performances of ChangeNotifier

Description
This PR improves the performances of ChangeNotifier as described in #71900.
Related Issues
#71900
Tests
I added the following tests:

notifyListener can be called recursively to verify that we can call notifyListeners while we are already notifying.

Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

This seems great. Would it be possible to add a comment somewhere in the file detailing how it works and why, so that people exploring the framework can learn from it as well?

Thanks for the suggestions.
About that, does something like this would be good for you?
// To make sure that listeners removed during this iteration are not called,
// we set them to null, but we don't shrink the list right away.
// By doing this, we can continue to iterate on our list until it reaches
// the last listener added before the call to this method.

// To allow potential listeners to recursively call notifyListener, we track
// the number of times this method is called in _notificationCallStackDepth.
// Once every recursive iteration is finished (i.e. when _notificationCallStackDepth == 0),
// we can safely shrink our list so that it will only contain not null
// listeners.

We should probably also have some comment about when we resize the list outside of iteration.
Other than these very minor nits, this LGTM.

I pushed some comments and I also added this line:

Looks like the analyzer is unhappy with this change now.

Looks like the analyzer is unhappy with this change now.

@letsar When you have a chance can you check out and address the failing checks on this PR? Thanks.

Should be better, sorry for the delay

Should be better, sorry for the delay

It looks like the benchmarks (https://github.com/flutter/flutter/pull/71986) were reverted. Any chance you can take a look at fixing the issue there and resubmit them so we can get this one submitted as well?
Also, it looks the customer_testing shards are still failing due to an outdated master. Can you rebase this?

@letsar This PR is getting pretty stale. Do you still have interest in coming back to this soon-ish? I think it would still be great to finish this up, if you have the time.

Yes sorry, I'm pretty busy at the moment 😔. I will get back to this as soon as I can.

So, what should we do to get the https://github.com/flutter/flutter/pull/71986 merged again?
It was reverted in https://github.com/flutter/flutter/pull/74450
In the referenced issue https://github.com/flutter/flutter/issues/74448
Looks to me like the test was either flaky or the real issue was somewhere else (looks the JSON string got cut off for some reason?). As the referenced issues are all closed now, I think we can just re-land the PR that adds the tests?
According to the Contributing Guide, we should not revert reverts, but making fresh commits instead ( https://github.com/flutter/flutter/wiki/Tree-hygiene#avoid-revert-revert-revert-revert-fix-foo-commit-messages )
If that's okay for you @letsar , I would like to try that (that meaning re-landing https://github.com/flutter/flutter/pull/71986) . If the test is merged, I think we just need another rebase here and can get this merged!

Hi @knaeckeKami, if you have time to do this, it would be great! I'm very busy at the moment and I'm not sure when I would be able to work on this benchmark again. So if you're able to do it sooner than me, it would be great for the community ⚡.
Concerning the issue with the test you're right, the JSON is cut off. One way to quickly fix  it for these tests would be to remove some decimals from the time values, so that the string would be the same size every time.

We got the benchmarks merged, so another
git pull --rebase origin master
git push --force

should get this PR ready to get merged!

I have rebased this. I hope everything will be ok now.

Thanks for rebasing. Looks like the analyzer has some issues now. Can you take a look?

@letsar I fixed it at https://github.com/flutter/flutter/commit/655a836de113b7d98ee1f92f51d91b93ea18d907 , you can use that

Looks like this had a nice positive effect on the benchmarks:
Android

iOS

Pretty awesome 😁


Handle Exception during dab validate when SqlMetadataProvider has not correctly initialized

Why make this change?
Closes https://github.com/Azure/data-api-builder/issues/2184
Closes https://github.com/Azure/data-api-builder/issues/2180
What is this change?
When we don't properly initialize the SqlMetadataProvider, certain functions will not behave as intended. This change adds some defensive programming around this fact so that we have more readable exceptions in these cases, as well as restructuring the validation to avoid the potential of such a circumstance.
We refactor the ValidateEntitiesMetadata function into 2 functions, one called ValidateRelationshipConfigCorrectness that validates the relationships in the config without needing to cross reference database metadata, and leaving the remaining validations that do require such metadata within the original function. This validation is then handled by a mix of the initialization of the metadata provider (this init process does some validation), and the call to ValidateRelationships which happens from within ValidateEntitiesMetadata only when the initialization of the metadata provider resulted in no connection errors.
Therefore, the normal code path when doing dab validate will be

Call ValidateRelationshipConfigCorrectness
Call ValidateEntitiesMetadata
2a. If there were no connection errors Call ValidateRelationships

How was this tested?
Manually, against our test suite, and we add a regression test that looks for the correct error messaging when the config that caused the error from the finding of the bug is used.
Sample Request(s)
This can be reprod by using the command `dab validate -c

/azp run

/azp run

Might not need this clear: https://github.com/Azure/data-api-builder/blob/e895f356f2105132b9ea2cc6e3e396555f09b632/src/Core/Configurations/RuntimeConfigValidator.cs#L1143 based on the resolution of previous comments

/azp run


open-world understanding

Thank you for your excellent work. I have a question about applying a point cloud classification model trained on object datasets to 3D scene segmentation tasks like ScanNet. Does the model require fine-tuning to adapt to the complexities of full scene segmentation, or can the original model be directly applied to this task? Your insights would be very helpful. Thank you!

Hi,
Thanks for you interest on Uni3D! We justify that the the open-world understanding task is to recognize the objects on real-scanned 3D scenes. As described in Sec.4.3, we conduct the experiment with the same settings as CLIP^2 where the ground truth
instant segmentation is available and the target is to recognize the category of each instant of the scene in a zero-shot way. So we do not perform scene segmentation on this task.
For exploring the dense prediction ability of Uni3D, we leverage Uni3D in the open-vocabulary part segmentation with a fine-tuning by texts of unseen parts. You can refer to Sec.4.4 if you are interest in leveraging Uni3D for segmentation.
Best,
Junsheng

I am closing this issue. If you have any more questions, please feel free to reopen it or create new issues. :)


[Documentation] Capacitor 3 - Use Automatic Android Plugin Loading

Sentry's documentation for setting up capacitor projects is a bit outdated for Capacitor 3.
As seen here, Capacitor officially recommends using automatic plugin loading on android. I bumped into a couple issues when using the provided add(SentryCapacitor.class) approach where then a few other plugins no longer worked. Switching to the registerPlugin approach fixed this for me.
This callout may be worth adding to your docs, as Cap2 vs Cap3 setup.

// MainActivity.java
package com.myapp.myapp;

import com.getcapacitor.BridgeActivity;

import android.os.Bundle;
import com.getcapacitor.BridgeActivity;
import io.sentry.capacitor.SentryCapacitor;

public class MainActivity extends BridgeActivity {
    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);

        // Initializes the Bridge
        registerPlugin(SentryCapacitor.class);
    }
}

Thanks for pointing that out, the docs will be updated 🙂

It's definitely worth checking, I don't register other capacitor v3 plugins either.
The Capacitor > Custom Native Android Code docs mention that using registerPlugin method is required for custom plugins.
Might be that Capacitor plugins installed via npm are auto-registered by Capacitor.

Closed as documented at Sentry Docs.

What about capacitor 4 and capacitor 5? Is this still required to add manually?
Documentation: "This step is not needed if you are using Capacitor 3"


Show how to change global settings

I am interested in changing the fontsize globally. I figured our how to do this, relying on the "simple" coding style with #slides:
#let my_template(body) = {
  set text(1.1em)
  slides(body)
}

#show: my_template

I am however not sure if this is the best way. I am also uncertain about how to do with if I was using #slides instead, as I get the error "only element functions can be used in set rules" when I do the following:
#set slide(settings: body=>{
  set text(1.1em)
  body
})

A section in the docs on "How to change global settings" or something to that effect would be great ^_^

You can use #set text(1.1em) to add to the global settings as normal, just put it after #show: init.
It's not usually necessary, but if you really want to change the setting parameters of #slide globally, you can
#let slide = slide.with(setting: body => {
  set text(1.1em)
  body
})

It turns out that #set text(1.2em) has to be placed after #show: init, AND before #show: slides. Fore the formed had no effect, and after the latter ended up making my = Section One show up as a normal heading inside the slide content.
Code used for testing:

#import "@preview/touying:0.2.0": *

#let s = themes.metropolis.register(s, aspect-ratio: "16-9", footer: self => self.info.institution)
#let (init, slide, slides) = utils.methods(s)

#show: init
#set text(1.1em)

#show: slides

= Section one

== Title 3
#utils.fit-to-height(1fr)[
  Test 3
]

Given that there are ways for this to go wrong, and that I managed to misread this section to mean that I had to get fancy to change the style, I think there is some reason to add a short section on how this is done properly, and just briefly mention what will not work.

Yes, you're right, I'll add a more detailed document.

The 0.3.1 release, close the issue by https://touying-typ.github.io/touying/docs/global-settings


[REQUEST] Having the file order configurable, as tab-smart-sort

I use tab-smart-sort (thanks to you actually) and I keep the default order "dir, ext, base" which I find really nice. I'd like my open files to have the same order as my tabs, so maybe you could add the same settings as tab-smart-sort  "dir, ext, base" /  "base, ext, dir (that's kinda what your default here)" / etc. ?

Added this functionality with commits 71aeb2297b7a6e9d5832eaada3fadc47f961aad5 and 306e120f115c80520d600324d02da8090a02e7bb. @misterbridge, it would be great if you could check that it works properly and re-open this bug if it doesn't, since I made a few quick changes that I'm not sure will work in all cases.

Great ! Works perfectly, thanks a lot !

Ha, actually I found a bug : As I sort first by "dir" I saw that you take care of the file's parent dir only, not the full tree. (When tab-smart-sort looks are the full tree => sort by the parent's parent's ... folder first)

Hmm, I'm a little confused @misterbridge . Right now, the sort function alphabetically compares the lower-cased version of the whole path of each file. That means that the higher in the tree, the higher up in the sort order a file will be. For each level in the tree, directories are then ordered alphabetically. I took most of this from tab-smart-sort, and it seems to work for me. Can you give me some more details of how it is not working for you, as well as of your system, so that I can figure out what's going on? Thanks!

Herer you can see it :

application
---- config
-------- configuration.php
test
---- config
-------- configuration.php

Order : appli first, test second (on tab-smart-sort)
Order : test first, appli second (on open-files)

Thanks for the info, @misterbridge . I found what the problem was. This should be fixed with 96db3420ba38c4db03a3ebbd8321079dea980f7e, but please let me know if you are still encountering problems.

Fixed ! Thanks, perfect.


Lower fees as voted plz

Special fee tier for stable needs more thinking so this should come later.
https://dao.projectserum.com/#/proposal/7GxS9nEc4ZiTjgSpTxg7JnsJXfyayoJocSdzAj5PmWR8?programId=AVoAYTs36yB5izAaBkxRG67wL1AMwG3vo41hKtUSb8is

+1 for this. this requires no changes for anyone, maybe the communication to let people know about this. and, we can have the new fees immediately for everyone.

Looks great, thank you so much!


prevent typing in chat from kicking players

Simply move the assertion for decompressed packet contents, because P::decode_packet consumes the packet it is given. It might be a good idea to consider just passing ownership of decompressed rather than &mut decompressed to make this behavior more clear. Not allowing P::decode_packet to modify the packet would also be a good solution.
fixes #24
Test plan
run
cargo r -r --example conway

connect to server on localhost
type in chat
see that you don't get kicked
the chat message does not appear in chat (out of scope for this PR)

Yes, I tried it with conway but not with terrain.

I misunderstood the purpose of this assertion, whoops. I'll do some more digging and update #24, closing this for now.


Core Dump on Arch Linux

Describe the bug
After installing the Insomnia snap, via sudo snap install insomnia per the documentation. It will fail to start on some systems and result in a core dump. The AppImage does not have this problem.
To Reproduce
Steps to reproduce the behavior:

sudo snap install insomnia
insomnia

Expected behavior
I would expect the app to start normally. Like it does with the AppImage.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Arch Linux 5.6.15-arch1-1
Installation Method: snap
App Version: Unknown, but recent

Additional context
$ coredumpctl info
           PID: 128025 (insomnia)
           UID: 1000 (iveelsm)
           GID: 1000 (iveelsm)
        Signal: 5 (TRAP)
     Timestamp: Tue 2020-06-02 20:55:54 MDT (11h ago)
  Command Line: /snap/insomnia/62/insomnia
    Executable: /snap/insomnia/62/insomnia
 Control Group: /user.slice/user-1000.slice/session-2.scope
          Unit: session-2.scope
         Slice: user-1000.slice
       Session: 2
     Owner UID: 1000 (iveelsm)
       Boot ID: 9427af5b1c724002aacb7d9b9e1f7aa5
    Machine ID: 58b909b09b654019949dfe020abf39ab
      Hostname: frogman
       Storage: /var/lib/systemd/coredump/core.insomnia.1000.9427af5b1c724002aacb7d9b9e1f7aa5.128025.1591152954000000000000.lz4
       Message: Process 128025 (insomnia) of user 1000 dumped core.

                Stack trace of thread 128025:
                #0  0x0000000003f7684f n/a (/snap/insomnia/62/insomnia + 0x3d7684f)
                #1  0x0000000003f7778b n/a (/snap/insomnia/62/insomnia + 0x3d7778b)
                #2  0x0000000003f6c653 n/a (/snap/insomnia/62/insomnia + 0x3d6c653)
                #3  0x0000000003f6c53e n/a (/snap/insomnia/62/insomnia + 0x3d6c53e)
                #4  0x0000000003f7f494 n/a (/snap/insomnia/62/insomnia + 0x3d7f494)
                #5  0x0000000003f7cb7f n/a (/snap/insomnia/62/insomnia + 0x3d7cb7f)
                #6  0x0000000003ec6e29 n/a (/snap/insomnia/62/insomnia + 0x3cc6e29)
                #7  0x0000000003ed87ff n/a (/snap/insomnia/62/insomnia + 0x3cd87ff)
                #8  0x0000000003eccf66 n/a (/snap/insomnia/62/insomnia + 0x3cccf66)
                #9  0x0000000003ef6c03 n/a (/snap/insomnia/62/insomnia + 0x3cf6c03)
                #10 0x0000000004b46507 n/a (/snap/insomnia/62/insomnia + 0x4946507)

The Snap image has never been very reliable, unfortunately. I suggest trying out the AUR package instead: https://aur.archlinux.org/packages/insomnia/

@gschier I was able to get it working with the AppImage.
But I just wanted to open this as just as FYI.

Thanks! I appreciate it 💯

@gschier Do you think we should just close this? It seems out of scope. The workaround (using the AppImage) seems to be well understood. So I think we should be good right?

@iveelsm 👋 I'm going to go ahead and close this since you're happy enough with the workaround.
Feel free to open a Discussion in the Ideas category if you (or anyone else who stumbles upon this issue) would rather see a better solution.

@wdawson Sounds good! I think the AppImage is a better approach anyways. :)


add hint in case passlib is not installed

/close due to content no longer valid; the page and content don't exist.
cc @romeokienzler @Arhell
@hbelmiro

/close


Error on Install

From @chidinweke on March 10, 2016 16:32
Note: for support questions, please use one of these channels:

Short description of the problem:
I tried to install iconic on my system and i'm getting this error...see image for error.
Ho do i solve this
Copied from original issue: driftyco/ionic#5796

Thanks for submitting an issue with us! As this is an issue related to the ionic cli i am going to be moving this issue to that repo as it could be answered better over there. Please feel free to continue the conversation on this issue in that repo!

From @BilelKrichen on March 10, 2016 16:44
You must publish this issue on this repository
https://github.com/driftyco/ionic-cli/issues

Hello i've not gotten any response for this issue.

Looks like you may need to set up your proxy, see if https://github.com/npm/npm/issues/3354 and http://stackoverflow.com/questions/7559648/is-there-a-way-to-make-npm-install-the-command-to-work-behind-proxy help.


Enables setting of timeout time of engine script.

The engine script timeout time is now fixed at 50 seconds.
I want to be able to set in the Unit property according to the environment and processing content.

Property name is considered "engine.script.connection.timeout".

Close because it is the same as #71.


Incorrectly setting fixed height on add reviews div causing error

Using a fixed height, even with relative units, is dangerous; fixed height should rarely be used. Here, the div's contents will make it the correct height anyway. If you need to use a height, often you should only be setting ''min height'.

done


Minor fixes

Fixed date format
Fixed donors variable location in document
Fixed getPost donors count

@porames > donors count is working.


Feature/more citation docs

Adds citation documentation for the Times and Dutch Newspapers Public corpora.
close #1322

This is excellent! It gives examples but it also explains very well why a certain way of citing is recommended.
I have not tested whether this is working correctly from a technical perspective, but I assume it does.


Run subscriptions as a different process from a rake task

Is there is a way to run each subscription individually as its own process? For what I see, launching rails will also launch all the subscriptions.

Hi @night91, when you say "launching" the subscriptions, are you referring to registering the subscribers to their topics?
The setup of each subscription/topic is done separately from launching rails. We propose several rake tasks for that purpose:
# Setup publishers (topics) and subscribers (subscriptions) in one go
bundle exec rake cloudenvoy:setup

# Or set them up individually
bundle exec rake cloudenvoy:setup_publishers
bundle exec rake cloudenvoy:setup_subscribers

I was about to ask a similar question, but I think this is what I'm wondering - forgive me if I'm wrong, but I think @night91 is asking whether it's possible to run the subscribers without starting the server on port 3000. For example, in a similar way to how you would start Sidekiq.  Given that you mention much about the microservice architecture, it doesn't seem to fit that you'd have the web server and subscribers running in the same process.

Hi @alachaum, sorry if I didn't transmit my question clearly. Indeed, I was referring to what @stevefolly appointed.
Imagine I wanted to run in a GKE cluster, so I may want to run a deployment for each subscriber or to have a possibility to run a set of subscribers (ex: all subscribers for dead letter topics).
It would be cool if, with a rake task, we could run a subscriber (without staring the server).
One example for running a consumer for Apache Kafka is: https://github.com/zendesk/racecar#running-consumers

Looking through the README again, I'm not sure this will be possible, since the implementation uses the push method, registering a webhook with the Google Cloud.  Google Cloud will then call the webhook for each message received.   What I was looking for (and possibly you as @night91 by the sounds of it) is to use the pull method.
By using this library, I don't think it's possible to use the cloud Pub/Sub API in a local development environment either, since the webhook needs to be accessible from Google Cloud.
@alachaum  can you confirm please?

@stevefolly the pull method is useful for situations such as Cloud Run. But you should still be able to run them without the server. Anyway, the pull method where it really shines. Is there any alternative, besides implementing your own stuffs as docs says?

@stevefolly you're completely right. With Cloudenvoy, you do not need an "active agent" to pull/process messages. The application is configured to subscribe and receive messages via webhooks.
Local development
In terms of local development, it's totally possible to use Cloudenvoy locally. To do so, just install the pubsub emulator via the gcloud cli. This emulator will mimic pub/sub locally and allow you to work in development without the need for an external tunnel (e.g. ngrok). See the the Get started with Rails for an example of setup (I extracted the relevant steps below)
# Install the pub/sub emulator
gcloud components install pubsub-emulator
gcloud components update

# Start the emulator
gcloud beta emulators pubsub start

Pull
At the moment, cloudenvoy does not offer a pull mechanism. But I'll consider it for future developments. I understand the pull approach can be very useful when apps run in a private environment with no accessible URLs.
In the meantime, you can still develop a small ruby agent that pulls messages from Pub/Sub and then use the whole Cloudenvoy logic to declare subscribers and process those messages. After all, the "receiving" part of Cloudenvoy is just a controller - it's a minor part of Cloudenvoy. This part can be replaced by a custom agent.
Dividing processing
Having said that, if you're looking to "divide and conquer" on message processing, it's still possible to divide your app into multiple apps to process certain messages. E.g.

One shared ruby library with core models and logic
One sinatra/rails app using the shared library above and subscribing to topic1, topic2, topic3
One sinatra/rails app using the shared library above and subscribing to topic4, topic5, topic6


PROGRAMMES-6052 findOnNowByService cache issues

Set max cache time to 300s
Add function flushOnNowByService so we can flush it when an stale object still in memory and is not longer valid

Frontend changes to use flushOnNowByService
https://github.com/bbc/programmes-frontend/commit/01d5685fbec0814446ce3797f1527d13919d9d0c

Looks ok 👍


[FLINK-9491] Implement timer data structure based on RocksDB

What is the purpose of the change
This PR is another step towards integrating the timer state with the keyed state backends.
First, the PR generalizes the data structure InternalTimerHeap to InternalPriorityQueue so that the functionality of a heap-set-organized state is decoupled from storing timers. The main reason for this is that state/backend related code lives in flink-runtime and timers are a concept from flink-streaming.
Second, the PR also introduced an implementation of InternalPriorityQueue with set semantics (i.e. the data structure we require to manage timers) that is based on RocksDB. State in RocksDB is always partitioned into key-groups, so the general idea is to organize the implementation as a heap-of-heaps, where each sub-heap represents elements from exactly one key-group, that merges by priority over the key-group boundaries. The implementation reuses the in-memory implementation of InternalPriorityQueue (without set-properties) as the super-heap that holds the sub-heaps. Further more each sub-heap is an instance of CachingInternalPriorityQueueSet, consisting of a "fast", "small" cache (OrderedSetCache) and a "slow", "unbounded" store (OrderedSetStore), currently applying simple write-through synchronization between cache and store. In the current implementation, the cache is based on a an AVL-Tree and restricted in capacity. The store is backed by a RocksDB column family. We utilize caching to reduced read-accesses to RocksDB.
Please note that the RocksDB implementation is currently not yet integrated with the timer service or the backend. This will happen in the next steps.
Brief change log

Refactored InternalTimerHeap to decouple it from timers, moved the data structures from flink-streaming to flink-runtime (-> InternalPriorityQueue).
Split the data-structure into a hierarchy, a heap without set-semantics (HeapPriorityQueue) and a heap extended with set-semantics (HeapPriorityQueueSet).
Introduced an implementation of RocksDB-based InternalPriorityQueue with set-semantics. Starting point is KeyGroupPartitionedPriorityQueue. This class uses a HeapPriorityQueue of CachingInternalPriorityQueueSet elements that each contains elements for exactly one key-group (heap-of-heaps). For RocksDB, we configure each CachingInternalPriorityQueueSet to use a TreeOrderedSetCache and a RocksDBOrderedStore.

Verifying this change
I added dedicated tests for all data structures.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes, fastutil)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (yes)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

CC @azagrebin  @sihuazhou

@StefanRRichter very nice PR, I only had some very minor comments.

@sihuazhou thanks for the fast review. I addressed all your comments.

+1 from my side

@sihuazhou @azagrebin thanks for the reviews! I will merge this once my travis is green.


WIP: [INT-1103] Renew deployment config

Add deployment configuration for sw6
Change deployment to rsync

Closed in favour of #14


Intelligent swapping of strategies

We want to develop a way to swap intelligently between GPU and CPU based strategies.

I've run some tests on matrix multiplication. Here are my results.

This is a graph of two different GPU strategies vs MKL. The crossover point between the copy strategy and MKL is around 170 rows. As for the mapped strategy, the first cross-over point is at 41 rows, with subsequent cross-overs every 64 added rows.

As for the different strategies, memory mapping seems to work better for even-numbered dimensions from 174 rows to 192 rows, where it becomes consistently better until 220 rows. Memory mapping is then better from 256 rows to 268 rows. After this, copying to vRAM is always better.

I tested two different graphics-accelerated BLAS libraries: hipBLAS and rocBLAS. The difference between the two was negligible.

I hope some of these results can help with some discussion about dynamically selecting compute strategies.

I've put the code I used to generate the graphs in https://github.com/cgbriggs99/einsums/tree/main/timing/comparisons for those interested.

One thing to note about these results is that these were treated as if they were in isolation. In reality, the GPU algorithms are asynchronous.

This is super nice! Thanks for doing it.
Let's figure out a strategy and code structure for swapping backends at runtime.

A simple suggestion for the logic could be if the GPU is in use and the operation is small, use a CPU algorithm. If the operation is larger, then copy the memory into vRAM and use the GPU when it is available. If the operation is too large to be held in vRAM, map RAM to the GPU. If the memory is massive and stored on the disk, use a CPU algorithm.

How about this. I basically "stole" the gang of four's structure for a template method and a strategy working together.


Change auth providers and verify user accounts

This includes a number of changes to the auth flow:

Removed the use of Auth0 altogether: we're not using their unified profiles anyway, as we need custom per-provider verifications. On top of that, it imposed artificial rate limits, which could bite us in the future. We now use vanilla github login & vanilla deployment of SIWE (the one that Auth0 used anyway, just without the indirection).
Verifying that a github user was created before a certain date. We could check for commits, but that's more involved and I don't think the return is that big.
Verifying that an ETH address made at least one transaction before a certain block.

If you need the dev secret keys, let me know on Telegram, I can share. I'm using Alchemy for RPC, so for deploy we'll probably need a better key anyway.

Since the PR also edits the format for user_id's, please take a look at this issue: https://github.com/ethereum/kzg-ceremony-sequencer/issues/9
I don't mind which format, we go with, we should document it in that issue and then upstream it to the specs

Verifying that an ETH address made at least one transaction before a certain block.

We initially made a soft decision on nonce > 3 and I think before a certain block, its not a blocker as we can modify the specs to suit this, though the rationale for moving to this should be specified and eventually upstreamed to the specs via a tracking issue or a direct PR

Since the PR also edits the format for user_id's, please take a look at this issue: #9
I don't mind which format, we go with, we should document it in that issue and then upstream it to the specs

Ahh, fair. Haven't seen this issue, will just move to your format, it makes no difference to me.

We initially made a soft decision on nonce > 3 and I think before a certain block, its not a blocker as we can modify the specs to suit this, though the rationale for moving to this should be specified and eventually upstreamed to the specs via a tracking issue or a direct PR

Right, the "certain block" is already a param, I'll just move the min nonce out to a param too (and set it to 4)


Progressbar bars installation

Collecting progressbar>=2.3 (from autosub)
Downloading progressbar-2.3.tar.gz
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\zzj04\AppData\Local\Temp\pip-build-m643aw0q\progressbar\setup.py", line 5, in
import progressbar
File "C:\Users\zzj04\AppData\Local\Temp\pip-build-m643aw0q\progressbar\progressbar_init_.py", line 59, in
from progressbar.widgets import *
File "C:\Users\zzj04\AppData\Local\Temp\pip-build-m643aw0q\progressbar\progressbar\widgets.py", line 121, in
class FileTransferSpeed(Widget):
File "c:\users\zzj04\appdata\local\programs\python\python37-32\lib\abc.py", line 133, in new
cls = super().new(mcls, name, bases, namespace, **kwargs)
ValueError: 'format' in slots conflicts with class variable

tl;dr: Install Python 2. It's not compatible with Python 3.
I just bumped into the same problem. This question/answer says Progressbar is incompatible with Python 3. The Wiki page says to install Python 2.7, so I assume Autosub isn't meant to work with Python 3.
This could be mentioned in README.md. I'm not that familiar with Python stuff, so maybe it's usually assumed everything's for Python 2.


[docs] Vite 1.0 update

I think you meant "3.0" not "1.0".

I believe it's for vite plugin svelte


IllegalStateException if using inner test-classes and ValueProviderExtension been declared on top-level

Having inner classes inside the test and the ValueProviderExtension is been declared on the top-level class, e.g.:
@ExtendWith(ValueProviderExtension.class)
class MyTestClass {

    @Nested
    class MyInnerTestClass1 {
        @Test
        void test() {
        }
    }

    @Nested
    class MyInnerTestClass2 {
        @Test
        void test() {
        }
    }
}

Then the following exception is been thrown:
java.lang.IllegalStateException: Illegal transition, cannot switch from DEFAULT back to TEST_CLASS
	at com.google.common.base.Preconditions.checkState(Preconditions.java:836)
	at com.tngtech.valueprovider.ValueProviderFactory.checkMayFinish(ValueProviderFactory.java:90)
	at com.tngtech.valueprovider.ValueProviderFactory.doFinishTestClassCycle(ValueProviderFactory.java:62)
	at com.tngtech.valueprovider.ValueProviderFactory.finishTestClassCycle(ValueProviderFactory.java:152)
	at com.tngtech.valueprovider.ValueProviderExtension.finishTestClassCycle(ValueProviderExtension.java:115)
	at com.tngtech.valueprovider.ValueProviderExtension.afterAll(ValueProviderExtension.java:99)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

Hello @yobinx again,
according to my tests, support for @Nested test classes works by now on the branch 41-add-support-for-junit-5-lifecycle-per-class. You are welcome to test yourself before a new version of the library is released. This would either give us the opportunity to fix remaining problems, or be more confident that our enhancements work as you expect.


Look at design of the cocktail card

Look at the design of the cocktail card and how it sits with the rest of the home page
See if you can copy the style and design for the favorites page

Create new class
Add styles to the JS file
Make sure new class is added the the JS file for the delete button


[Feature Request] [Low Effort] Add trpc client types for trpc-nuxt

Currently the @zenstackhq/trpc plugin only generates client types for React and Next clients. The types for the Nuxt client trpc-nuxt would be a welcome addition.
It's quite trivial to derive from the Next client and would just require a slightly modified generator template.
Here's the modified version I'm actually using, which is analogous to the Next client:
import type { AnyRouter } from '@trpc/server'
import { createTRPCNuxtClient as _createTRPCNuxtClient } from 'trpc-nuxt/client'
import type { ClientType } from '../routers'
import type { DeepOverrideAtPath } from './utils'

type TRPCNuxtClient<TRouter extends AnyRouter> = ReturnType<typeof _createTRPCNuxtClient<TRouter>>

export function createTRPCNuxtClient<
  TRouter extends AnyRouter,
  TPath extends string | undefined = undefined,
>(...args: Parameters<typeof _createTRPCNuxtClient<TRouter>>) {
  const r: TRPCNuxtClient<TRouter> = _createTRPCNuxtClient<TRouter>(...args)
  return r as DeepOverrideAtPath<TRPCNuxtClient<TRouter>, ClientType<TRouter>, TPath>
}

Would be great if this could make it into the plugin!

+1
Thanks for sharing some code!
What's your plugin definition?
That's what I use with your code, but can't get the include types working (I defined the right ClientType as well from next)
import { httpBatchLink } from "trpc-nuxt/client";
import superjson from "superjson";
import { AppRouter } from "~/server/";
import { createTRPCNuxtClient } from "~/server/routers/client/nuxt";
export default defineNuxtPlugin(() => {
  const trpc = createTRPCNuxtClient<AppRouter>({
    transformer: superjson,
    links: [
      httpBatchLink({
        url: "/api/trpc",
        headers: useRequestHeaders(),
      }),
    ],
  });

  return {
    provide: {
      trpcApi: trpc,
    },
  };
});

I ditched the trpc-plugin for good because it's dead slow (generating) and inefficient. The types generated are apparently very complicated and it slows down my IDE noticeably. Felt very heavy with too little benefit. I needed a few endpoints for an admin interface that weren't too much of a hassle to write manually, if your use case is similar you might want to consider that path as well.
Anyway, I hope you get it going. I didn't do anything fancy really, see my plugin definition below. I did have to install @trpc/react-query since the generated code references that. The query and mutation functions have the same signature as the Nuxt trpc client, so they just worked.
import { httpBatchLink } from 'trpc-nuxt/client'
import { createTRPCNuxtClient } from '~/server/trpc/client'
import transformer from '~/server/trpc/transformer'
import type { AppRouter } from '~/server/trpc/routers'

export const useApi = createSharedComposable(() => {
  return createTRPCNuxtClient<AppRouter, 'admin'>({
    transformer,
    links: [
      httpBatchLink({ url: '/api/trpc' }),
    ],
  })
})


followups for sidebar changes

Screenshot for first commit:

Second commit:

Merged e36971c7e192cb0d4e85dcfd8830855e3f4c0f51 for the first commit, after editing the comment to explain context better.

Merged, thanks @evykassirer!


Precise to add "window.jQuery = window.$ = require('jquery')" in the WEBPACK CONFIG

Has mentioned in this issue (https://github.com/BrockReece/vue-fullcalendar-example/issues/5): you may have to precise to add in the  webpack config : https://github.com/BrockReece/vue-fullcalendar-example/blob/master/build/webpack.base.conf.js#L22
thanks

Hi @trollepierre,
I don't think you need to add jQuery to your project anymore, fullcalendar will do this for us if no jQuery  instance is detected.
I will update the readme later to reflect this.

@trollepierre See PR #110, does this look good to you?

Perfect!


maps

i need a less annoying background map for the vienna page and for the spatial entities page …
https://tiles.stadiamaps.com/tiles/stamen_toner_lite/{z}/{x}/{y}{r}.png
throws error, seems to be related to them changing their urls, but maybe just temp error ??? see https://docs.stadiamaps.com/guides/migrating-from-stamen-map-tiles/
also check https://maps.stamen.com/stadia-partnership/
alternative maybe http://{s}.basemaps.cartocdn.com/dark_all/{z}/{x}/{y}.png](http://{s}.basemaps.cartocdn.com/dark_all/%7Bz%7D/%7Bx%7D/%7By%7D.png)'

check the attributions under the map, should be up to date!


Support for kubernetes clusters with user assigned msi

What this PR does / why we need it:
Support for kubernetes clusters with user assigned msi
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

Support for kubernetes clusters with user assigned msi
Adds the go sdk files for msi apis.
Avoid role assignment retry if the its already present.

Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

Lint errors will help nudge unit tests:
pkg/armhelpers/msi.go:14:1:warning: exported method AzureClient.IsUserAssignedIDPresent should have comment or be unexported (golint)
pkg/armhelpers/msi.go:33:1:warning: exported method AzureClient.CreateUserAssignedID should have comment or be unexported (golint)

Please update Gopkg.toml to reflect the addition of these new dependencies.

This is looking well on its way, thanks @kkmsft! See comments.

@jackfrancis - not able to get the Gopkg.toml updated since this just includes packages (services/preview/msi/mgmt/2015-08-31-preview/msi) from the azure-sdk-for-go project. Will catch you offline and figure out what to do about this..

/lgtm

Thanks @kkmsft!


An Error Occured! Please help!

discord.ext.commands.errors.CommandInvokeError: Command raised an exception: NameError: name 'open_account' is not defined
There is occurring Please help me to solve it!

Re check your code once and see open_account function


subscription implementation

Do you have any plan on implementing this:
https://github.com/graphql/graphql-js/pull/189
http://graphql.org/blog/subscriptions-in-graphql-and-relay/

Sorry, wrong place

PHP used to be bad choice to handle subscriptions. Now things change slowly with some of new libs like ReactPHP. But most of the real-world projects still use node or other tools for websockets / comet.
So the solution for subscriptions in graphql-php will probably be different than in graphql-js. I will evaluate possible approaches, but probably not until next month.

If you ask me (and a little off topic), GraphQL with straight PHP is going to be a difficult mix from the start, as seen from the performance concerns already being found with this new project. I feel it is because of the sequential/ share nothing nature of PHP. There is a reason why Facebook created HHVM with Hack's async() function. It's to run requests to data services in parallel, which is also a need to allow GraphQL to perform well IMHO. I believe it is also why Facebook has turned to the NodeJS crowd for the server side. Why haven't they released a Hack version of a GraphQL type system?
Hmmm....
Scott

@smolinari Yeah, honestly I expected to see PHP / Hack version of GraphQL, but instead they released js version. So had to build this library to integrate with existing PHP project.
As for performance concerns - most of them equally apply to ORMs or custom solutions in PHP. Actually such library has some advantage over custom per-project solutions in that it could be re-implemented as PHP module (as drop-in replacement).
And I think it is a direction to follow eventually. But I suspect that even pure PHP implementation may perform reasonably well if you replace recursive Executor with iterative one (as PHP is slow in handling deep recursion). So that is probably the next step.
As for async - if you're stuck with plain PHP it doesn't matter if you use GraphQL or other tools - you don't have async anyway %)
Targeting for HHVM only is not an option at this point, but I would like to have a version that can handle async features of HHVM for those who can afford to use it.

So had to build this library to integrate with existing PHP project.

And thank you for doing so! :+1:

As for performance concerns - most of them equally apply to ORMs or custom solutions in PHP.

Agreed. PHP7 will help this situation, but I am of the opinion, to scale a lot with PHP, we must break out of the current mold a bit. Facebook did it on their own with HHVM. And amazingly, PHP7 is as good or better as HHVM, even without a JIT compiler. That is really great news and looks promising for the future of PHP. But still, there are things we can do, to get even more performance. Non-blocking I/O and threading for example.

But I suspect that even pure PHP implementation may perform reasonably well if you replace recursive Executor with iterative one (as PHP is slow in handling deep recursion). So that is probably the next step.

Sounds good. I'll be keeping an eye out on these changes myself, as I am going to be attempting to do an appserver.io implementation of this library, where we will try to spread out the resolver work (and I am not certain that is the best cut point yet, but we'll see. If you have a better idea, let me know).

As for async - if you're stuck with plain PHP it doesn't matter if you use GraphQL or other tools - you don't have async anyway %)

True. Thankfully there are other alternatives to plain PHP. :smile:

Targeting for HHVM only is not an option at this point, but I would like to have a version that can handle async features of HHVM for those who can afford to use it.

This is what I was hoping Facebook would give us. I even asked them, what was the reasoning to go with Node instead of an HHVM implementation. I don't expect much of an answer, if any, but you never know.
Scott

FYI. https://github.com/graphql/graphql-js/issues/219#issuecomment-155284851
Scott

@vladar I'm wondering if the way to go about it would be for this package to allow us to define Subscription Types, but leave it up to the developer on how to implement it. If you're in the process of creating functionality to handle subscriptions, maybe it's a separate package?
Not sure what the final spec will look like (seems that they're still flushing out details). But I'm personally thinking that Ratchet with WAMP and ZeroMQ may be the way to go for me. Laravel allows you to easily broadcast events and I'm thinking I will create a custom driver for this setup. Others may find it better to implement some sort of Node/Redis setup. Not sure if something like Pusher would work, but that may another possibility for some (again, depending on what the final spec looks like).
IMO, the way I see it is that GraphQL allows us to define our subscriptions, but doesn't get down into the details of how it's implemented.

@chrissm79 Yes, the goal is to be abstract enough for different backends. Unlike Node PHP doesn't have first-class support for such things, so it doesn't make sense to force concrete implementation.
So I do agree with you: most likely we'll have separate types for subscriptions, but underlying communication / transport layer will be app-specific.
Eventually some backends may be added as optional plugins to simplify initial setup.

If it helps as reference implementation:
https://siler.leocavalcante.com/graphql/subscriptions.html
https://github.com/leocavalcante/siler/tree/master/src/Graphql

Something that could be done here for subscriptions is something kind of like Laravel does for the Broadcast service where the lib would do pub/sub using redis and then a very basic nodejs server to actually handle the websocket connections.
It wouldn't be the best system, I know that, but it would be a workaround that would work.

Closing this as actual subscription implementations may vary depending on the environment. We won't ship out of the box solution until there is no such solution in the PHP core.
This lib already enables execution of subscriptions but each project will have to come up with own implementation. I encourage everyone to build such implementations as separate projects and link here so that we could add them in the docs.
Siler link above is a good example.

Closing this as actual subscription implementations may vary depending on the environment. We won't ship out of the box solution until there is no such solution in the PHP core.

hey, I think that following along with the built in promise adapters, it won't take that much effort to build a similar AsyncIterator adapter system to handle subscriptions. this implementation can remain implementation agnostic; server and transport implementation would remain up to consumer libraries (like a graphql-ws-amphp, graphql-ws-reactphp, etc that handle protocols).
I am currently working on a proof of concept on this approach and hoping to create a draft PR soon for discussion.

@tristanpemble did you already have a chance to work on your proof of concept, which you could share? Thank you very much.

any updates on this ?

any updates on this ?

@mcop1 I realize this is 2.5 years late, and I have no idea what state it is in, but here was the branch I was working from: https://github.com/tristanpemble/graphql-php/tree/subscriptions
I'm not working with PHP these days


Make certain that strain_rate is always displayed in PassSequence Summary

Summary
In analysis of forming processes temperature, strain and strain_rate are the fundamental state variables.
The PassSequence summary should alsways provide these values.
Strain_rate values seem to be available in the reults database but is not added to the Summary when a flowcurve calculation method with no strain_rate dependance is used.

@GRPlan can we close this issue?