2.5.0 tf_to_kernel is not compatible with msvc /Ob3

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 20H2
TensorFlow installed from (source or binary): source
TensorFlow version: v2.5.0
Python version: 3.8 (anaconda 2021.05)
Bazel version (if compiling from source): 3.7.2
GCC/Compiler version (if compiling from source): msvc 19.28.29910 (Visual Studio 2019 v16.9.0)
CUDA/cuDNN version: 11.3.0_465.89/<IP_ADDRESS>
GPU model and memory: RTX 3090 24GB

Describe the problem
If the Ob3 parameter is specified during compilation, the compilation will fail.
The error occurs in tf_to_kernel.exe.
2.4.0 is fine with Ob3.
Provide the exact sequence of commands / steps that you executed before running into the problem
C:\Users\User\Source\Repos\tensorflow>python ./configure.py
You have bazel 3.7.2 installed.
Please specify the location of python. [Default is C:\Users\User\anaconda3\python.exe]:
Found possible Python library paths:
  C:\Users\User\anaconda3\lib\site-packages
Please input the desired Python library path to use.  Default is [C:\Users\User\anaconda3\lib\site-packages]
Do you wish to build TensorFlow with ROCm support? [y/N]:
No ROCm support will be enabled for TensorFlow.
Do you wish to build TensorFlow with CUDA support? [y/N]: y
CUDA support will be enabled for TensorFlow.
Found CUDA 11.3 in:
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.3/lib/x64
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.3/include
Found cuDNN 8 in:
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.3/lib/x64
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.3/include
Please specify a list of comma-separated CUDA compute capabilities you want to build with.
You can find the compute capability of your device at: https://developer.nvidia.com/cuda-gpus. Each capability can be specified as "x.y" or "compute_xy" to include both virtual and binary GPU code, or as "sm_xy" to only include the binary code.
Please note that each additional compute capability significantly increases your build time and binary size, and that TensorFlow only supports compute capabilities >= 3.5 [Default is: 3.5,7.0]:
 compute_35

Please specify optimization flags to use during compilation when bazel option "--config=opt" is specified [Default is /arch:AVX]:
/MP /cgthreads8 /Qpar /fp:fast -Ob3

Would you like to override eigen strong inline for some C++ compilation to reduce the compilation time? [Y/n]: n
Not overriding eigen strong inline, some compilations could take more than 20 mins.
Would you like to interactively configure ./WORKSPACE for Android builds? [y/N]:
Not configuring the WORKSPACE for Android builds.
Preconfigured Bazel build configs. You can use any of the below by adding "--config=<>" to your build command. See .bazelrc for more details.
        --config=mkl            # Build with MKL support.
        --config=mkl_aarch64    # Build with oneDNN and Compute Library for the Arm Architecture (ACL).
        --config=monolithic     # Config for mostly static monolithic build.
        --config=numa           # Build with NUMA support.
        --config=dynamic_kernels        # (Experimental) Build kernels into separate shared objects.
        --config=v1             # Build with TensorFlow 1 API instead of TF 2 API.
Preconfigured Bazel build configs to DISABLE default on features:
        --config=nogcp          # Disable GCP support.
        --config=nonccl         # Disable NVIDIA NCCL support.
C:\Users\User\Source\Repos\tensorflow>bazel build --config=opt --incompatible_strict_action_env=false --define=no_tensorflow_py_deps=true //tensorflow/tools/pip_package:build_pip_package

Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
ERROR: tensorflow/tensorflow/core/kernels/mlir_generated/BUILD:871:23: compile tensorflow/core/kernels/mlir_generated/***.o failed (Exit -1073741819): tf_to_kernel.exe failed: error executing command
  cd execroot/org_tensorflow
bazel-out/x64_windows-opt/bin/tensorflow/compiler/mlir/tools/kernel_gen/tf_to_kernel.exe --tile_sizes=1024 --arch=compute_35 --input=bazel-out/x64_windows-opt/bin/tensorflow/core/kernels/mlir_generated/***.mlir --output=bazel-out/x64_windows-opt/bin/tensorflow/core/kernels/mlir_generated/***.o --enable_ftz=False --cpu_codegen=False

@fo40225
The tested build configurations for CUDA/cuDNN for tf2.5 are

Version
Python version
Compiler
Build tools
cuDNN
CUDA

tensorflow_gpu-2.5.0
3.6-3.9
MSVC 2019
Bazel 3.7.2
8.1
11.2

tensorflow_gpu-2.4.0
3.6-3.8
MSVC 2019
Bazel 3.1.0
8.0
11.0

please refer the documentation and let us know if it helps.Thanks

@UsharaniPagadala
This issue is not related to cuda/cudnn version.
I have build the tf 2.5.0 with msvc 2019 16.10/cuda_11.3.0_465.89/cudnn-v<IP_ADDRESS> and still error.
C:\Users\User\Source\Repos\tensorflow>python ./configure.py
You have bazel 3.7.2 installed.
Please specify the location of python. [Default is C:\Users\User\anaconda3\python.exe]:

Found possible Python library paths:
  C:\Users\User\anaconda3\lib\site-packages
Please input the desired Python library path to use.  Default is [C:\Users\User\anaconda3\lib\site-packages]

Do you wish to build TensorFlow with ROCm support? [y/N]:
No ROCm support will be enabled for TensorFlow.

Do you wish to build TensorFlow with CUDA support? [y/N]: y
CUDA support will be enabled for TensorFlow.

Found CUDA 11.2 in:
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.2/lib/x64
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.2/include
Found cuDNN 8 in:
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.2/lib/x64
    C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.2/include

Please specify a list of comma-separated CUDA compute capabilities you want to build with.
You can find the compute capability of your device at: https://developer.nvidia.com/cuda-gpus. Each capability can be specified as "x.y" or "compute_xy" to include both virtual and binary GPU code, or as "sm_xy" to only include the binary code.
Please note that each additional compute capability significantly increases your build time and binary size, and that TensorFlow only supports compute capabilities >= 3.5 [Default is: 3.5,7.0]: compute_35

Please specify optimization flags to use during compilation when bazel option "--config=opt" is specified [Default is /arch:AVX]: /MP /cgthreads8 /Qpar /fp:fast -Ob3

Would you like to override eigen strong inline for some C++ compilation to reduce the compilation time? [Y/n]:
Eigen strong inline overridden.

Would you like to interactively configure ./WORKSPACE for Android builds? [y/N]:
Not configuring the WORKSPACE for Android builds.

Preconfigured Bazel build configs. You can use any of the below by adding "--config=<>" to your build command. See .bazelrc for more details.
        --config=mkl            # Build with MKL support.
        --config=mkl_aarch64    # Build with oneDNN and Compute Library for the Arm Architecture (ACL).
        --config=monolithic     # Config for mostly static monolithic build.
        --config=numa           # Build with NUMA support.
        --config=dynamic_kernels        # (Experimental) Build kernels into separate shared objects.
        --config=v2             # Build TensorFlow 2.x instead of 1.x.
Preconfigured Bazel build configs to DISABLE default on features:
        --config=noaws          # Disable AWS S3 filesystem support.
        --config=nogcp          # Disable GCP support.
        --config=nohdfs         # Disable HDFS support.
        --config=nonccl         # Disable NVIDIA NCCL support.

C:\Users\User\Source\Repos\tensorflow>bazel build --config=opt --incompatible_strict_action_env=false --define=no_tensorflow_py_deps=true //tensorflow/tools/pip_package:build_pip_package

tf 2.4.0 can be compiled with Ob3.
tf 2.5.0 will fail if Ob3 is used, but it will succeed if it is changed to Ob2.


PHP 8.1: fix deprecation notice

As of PHP 8.1, PHP will start throwing deprecation warnings along the lines of:
Deprecated:  Return type of [CLASS]::[METHOD]() should either be compatible with [PHP NATIVE INTERFACE]::[METHOD](): [TYPE], or the #[ReturnTypeWillChange] attribute should be used to temporarily suppress the notice

These type of deprecation notices relate to the Return types for internal methods RFC in PHP 8.1.
Basically, as of PHP 8.1, these methods in classes which implement PHP native interfaces are expected to have a return type declared.
The return type can be the same as used in PHP itself or a more specific type. This complies with the Liskov principle of covariance, which allows the return type of a child overloaded method to be more specific than that of the parent.
The return type for JsonSerializable::jsonSerialize() is supposed to be mixed, but as that is a PHP 8.0 type, it cannot yet be added.
Ref: https://www.php.net/manual/en/class.jsonserializable.php
With that in mind, I've added the annotation instead.
Note: While attributes are a PHP 8.0 feature only, due to the syntax choice for #[], they will ignored in PHP < 8 and can be safely added.

:wave: Hiya all! Anything I can do to move this (and my other PRs) forward ? All relatively small changes and atomic PRs, so would be great if they could get some traction....

I'll try to take a look and get them merged this week, @jrfnl. Thanks so much for contributing!

Thanks @ramsey, appreciated!

Is there anything I can help with to get this PR merged?

Hi @ramsey, is there a plan for the next bugfix release which could contain that change?
Imho it should be planned soon as its currently a blocker for many people for using PHP 8.1. WHats your opinion / the release schedule of this package on that?


Get-DbaDbRoleMember - Changed output to align with ServerRoleMember

Type of Change

[ ] Bug fix (non-breaking change, fixes # )
[ ] New feature (non-breaking change, adds functionality, fixes  )
[x] Breaking change (effects multiple commands or functionality, fixes #4964 )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Changed output to align with Get-DbaServerRoleMember

tests for Get-DbaPermission failed but runs ok locally - seems to be unrelated. Looks like all ran successfully last time as well


cleaner: make the method pure functions by removing dependencies from the class constructor

cleaner: make the method pure functions by removing dependencies from the class constructor

Completed and merged in PR #158 !


Modify Padding navbar menu

Description
Padding is removed for drop down menu in navbar

References # #1841
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How Has This Been Tested?
UI changes are dev tested and compared with figma

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

@ElishaSamPeterPrabhu Are the open comments in this PR resolved?

Yes


Add Docker support

I would love to see "the Dockerfile" for this :)
I read Microsoft discourages Docker for ASP.NET Core with Full .NET Framework, I think that's also the reason you can't right-click "Add > Docker support" in Visual Studio on a aspnet core with full net framework project.
But I started a project during the time when aspnet core was available, but the net core framework was still not/barely released, and I would love to deploy it in Docker anyway.
They do describe how to approach it, but without a working example I haven't been able to get this working yet.
So guidance on how to set this up via this project would be awesome.

Closed


500 ERROR

I had this error
{     "code": 500,     "message": "The token storage contains no authentication token. One possible reason may be that there is no firewall configured for this URL." }
after using
       exclusion = @Hateoas\Exclusion(            excludeIf = "expr(is_granted(['ROLE_ADMIN']))"       )
Can anyone help me please

This is User code error connected to authorization. Not connected to this bundle.
Tip: Read more about symfony auth (error message tells you a lot about possible issue), and see how is_granted works.


Correct link dead link to actionContext details in docs

CLA is valid!

Coverage remained the same at 100.0% when pulling 2f4a1f1030651acc3365b1edecb59ce1d19aa3a7 on Maximilianos:patch-1 into e66931fc62be3196b6a5de6358e7a10794ad1814 on yahoo:master.

Coverage remained the same at 100.0% when pulling 2f4a1f1030651acc3365b1edecb59ce1d19aa3a7 on Maximilianos:patch-1 into e66931fc62be3196b6a5de6358e7a10794ad1814 on yahoo:master.

Coverage remained the same at 100.0% when pulling 2f4a1f1030651acc3365b1edecb59ce1d19aa3a7 on Maximilianos:patch-1 into e66931fc62be3196b6a5de6358e7a10794ad1814 on yahoo:master.

Thanks!


Remove duplicates from the list

Hi @shorodilov I also would like to work on this issue. Please assign it to me. Thankyou


Include queued metic for Sonarr and Radarr

Lidarr exposes this metric:

lidarr_queue_total

I think it will be useful to have the same metrics in Sonarr and Radarr.
My config:

Exportarr v1.1.0
Sonarr <IP_ADDRESS>7
Radarr <IP_ADDRESS>75
Lidarr <IP_ADDRESS>92

This already exists for Radarr and Sonarr
https://github.com/onedr0p/exportarr/blob/master/cmd/exportarr/main.go#L132
https://github.com/onedr0p/exportarr/blob/master/cmd/exportarr/main.go#L151
https://github.com/onedr0p/exportarr/blob/master/internal/collector/shared/queue.go

Ok, the issue is I can't see the metrics in Grafana because it must be at least one item queued in Sonarr/Radarr to see the metric. They doesn't appear in the metrics endpoint.
Could you set 0 as default value?
https://github.com/onedr0p/exportarr/blob/de5176757f3f4caf313a8e850968245ebaa46659/internal/collector/shared/queue.go#L66

Probably, but in the meantime you can have grafana treat NULLs as 0

Going to close this issue as Grafana can be configured to do the above.


Add ASP support

I can't make na aspx work in StaticFileHandler, but why?

Hi @Jaedson33,
Can you tell us a bit more about what you are trying to accomplish? The static file handler will simply post the file back to the client with the mime type based off the extension.
If you want to return the .aspx files as html you can do that by instantiating the StaticFileHandler with a MimeTypeProvider with the .aspx file added to the available mime types.
In any case it's not going to run the .aspx file itself, but will only be able to show it as html, text.
It looks like asp .net core is getting to the state were it's able to run on the pi (see: http://www.devvy.nl/?p=170) and going to be run in a stable fashion.
Hope this helps,
Jark

Closing this issue since there have been no replies for nearly a month.


fixing Gui's Rss Feed

fixing RSS feed for @insidegui

Oh wow, I wonder how long that has been broken. Thanks Leo!


what happened to the previous repo ?

The Wolf-Cola on GitHub was a skelton app; folder structure and a couple of saga actions; main code lives at a private repository
I'm in talks with companies / peoples and MIT licensing isn't in their best interest
Will report back soon


Don't rely on actions/upload-release-asset

That repo is unmaintained and archived.
Instead, use the GitHub CLI that's pre-installed in the GitHub Actions
execution environment and has simple CLI for uploading release assets to
existing releases.
/kind cleanup
Builds on https://github.com/shipwright-io/build/pull/766
cc @SaschaSchwarze0
Submitter Checklist

[n] Includes tests if functionality changed/was added
[n] Includes docs if changes are user-facing
[y] Set a kind label on this PR
[y] Release notes block has been filled in, or marked NONE

See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes
NONE

@HeavyWombat, you are our GitHub CLI expert. Want to take a look?

/approve


display a 0 when there isn't any traffic for a time period on figure 1

Instead of empty lines:

Does that look OK @jread-usgs ?


Unable to trigger the job with github repository.

Issue
I am analysing the devtron and trying to trigger the simple devtron job with GitHub. but it shows 128 status code.

Extra information
Deployed using: Helm
Chart version: v0.6.15
AB#3475

After little dive into found pod devtroncd/git-sensor-0 cannot resolve github.com. I am tried to change but it's read only file system.
Log of git-sensor-0
git-sensor {"level":"error","ts":1685685007.4861348,"caller":"git/GitCliUtil.go:54","msg":"error in git cli operation","msg":"fatal: unable to access 'https://github.com/Bhautik0110/golang-api.git/': Could not resolve host: github.com\n","err":"exit status 128","stacktrace":"github.com/devtron-labs/git-sensor/pkg/git.(*GitUtil).runCommand\n\t/go/src/github.com/devtron-labs/git-sensor/pkg/git/GitCliUtil.go:54

wget github.com
bad address

@Bhautik0110 does your cluster have connectivity to internet? Also, coredns pods are in running state with no error in logs?


Spotify Setup Authentication: Timeout While Waiting for Authentication Callback

What version of Music Assistant has the issue?
2.2.5
What version of the Home Assistant Integration have you got installed?
2024.9.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Unable to add Spotify as a Music Provider. I get the error message: Timeout While Waiting for Authentication Callback
How to reproduce

Settings - > Add Music Provider -> Selected Spotify -> Clicked Authenticate with Spotify
I get the Pop Up, and enter my Spotify credentials -> Agree to allow Spotify to connect to Music Assistant
Get the message "Music Assistant oAuth Redirect Processing your request" on the Pop up screen
In the MA page, after ~30 seconds get the error My-HA-URL says "Timeout while waithing for authentication callback"

Music Providers
Spotify
In my Spotify Account -> Profile ->Managed Apps, I can see a new Music Assistant line created, so the process seems to work outbound from HA/MA -> Spotify. I can replicate this outbound process by clicking the Remove Access button for the Music Assistant App in Spotify, and repeating the steps above to get the Music Assistant App created in Spotify.
Player Providers
No issues - able to add Sonos and Chromecast
Full log output
MA Log.txt
Additional information

MA is installed in the same RPi instance as HA. The RPi is still operating below capacity - CPU is 5%, Memory Utilisation = 42%
MA integration is installed and running, have been able to add player providers and can see my media devices within MA. URL is as per the server address in the MA logs.
Have executed all steps from a local connection of HA - from the MA link in my HA dashboard sidebar
I do not have VPN installed on my network.
I have a paid Spotify Premium Family account
Have gone through all the issue logs on this topic

What version of Home Assistant Core are your running
2024.9.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

Same issue here too. Running on RPi with Music Assistant v2.3.0b26

Did you click the link the popup screen ?

There was a small issue in the oauth redirect page after a recent change. Its fixed now.
You should be able to authenticate to spotify now

Confirmed authentication was successfully completed in a split second. Many thanks. Legend!


shared variables only available within one module.

This is caused by this issue
https://github.com/opencor/opencor/issues/2806
When trying to access shared variables, the variable is only available in one of the shared modules without an apparent rule.

Fix is detailed here.
https://github.com/opencor/opencor/issues/2806#issuecomment-2321881427
I need to modify the generation so boundary_condition variables aren't written as accessible to the module, that way the variable will be accessible in the module it is calculated.

Fixed with this comment
https://github.com/opencor/opencor/issues/2806#issuecomment-2321976395
Now variables are accessible in their own module.

Closing


Support setting untyped null values as parameters on PreparedStatements

Add support for setting untyped null values as parameters for PreparedStatements. Currently, null values must include a SQL type, and using the Types.OTHER constant is not allowed.

Fixed in #1224


[Testing] Make the number of UTXOs during split and merge dynamic

The number of UTXOs being split and merged has been constant so far.
Now it is dynamic in order to add randomness.

This version works for sure.
However, it doesn't work with 1000s of UTXOs yet. I am testing with increased threshold.

Can we merge and try this live?


Full text requirement

Can you provide publish version of your article?

Thanks for your interest in our work, please leave your email address and the PDF will be sent to you.

Thanks for your response, my gmail<EMAIL_ADDRESS>

Hello @cxh0519
How are you?
Thanks for contributing to this project.
Can I get your paper for this project?
My email<EMAIL_ADDRESS>Thanks

Hi,
I sent to you. Pls check your mail.
Vào Th 4, 7 thg 8, 2024 lúc 14:37 rose-jinyang @.***>
đã viết:

Hi @tuantran23012000 https://github.com/tuantran23012000
How are you?
Did you get the paper for this project?
If so, could you share it with me?
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/cxh0519/VTB/issues/1#issuecomment-2272819077, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6KBTSV5AFBO4CG7WODQJLZQHFB7AVCNFSM6AAAAABMDZXFY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZSHAYTSMBXG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Very thank you @tuantran23012000


Creates the initial index.html to style against

This pull request creates an initial html layout taken from the course assignment.  It should provide a sufficient structure to style against.

hi


Add flags for tabsearch and QR generator

This PR contains two commits, the first adds a flag that removes the tabsearch button (and it's keyboard shortcut), the second adds a flag to disable the QR generator for sharing page links.
Fixes #1553 and #1574

Awesome!

Did you avoid removing this empty padding on purpose? Could it be also incorporated into the patch?

I have a separate patch for removing the drag handle space, but I needed to do some more testing before submitting it.
It turns out that the flex layout compresses the tip marquee's space regardless of it's preferred size and the added padding, so the only change needed is the resizing of the handle.
I'll add that patch to the PR soon once I get some time.  @Eloston I hope you don't mind me grouping up these patches like this!  Doing it this way makes it so that their merge order doesn't need to be considered since the flags need to be added sequentially.

With the last patch it looks like this:

@Eloston I hope you don't mind me grouping up these patches like this! Doing it this way makes it so that their merge order doesn't need to be considered since the flags need to be added sequentially.

I don't mind, thanks for letting me know!


add tutorials overview page

Nothing all too fancy, but an overview page. The toc in the sidebar can be expanded.

Closes #244
@wiep would you do the review?

@wiep would you do the review?

I just added a reference to the overview page on the welcome page in the last commit. The PR is ready I think :)


Program exit caused by file descriptor exceeding limit

Describe the bug
Example "simple_reverse_proxy" exited because of fd over the limit.
To Reproduce
Steps to reproduce the behavior:

Turn off the WIFI or Plug out the cable.
Startup the program.
Send HTTP request using short connection.
See Many sockets in CLOSE_WAIT.

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [ubuntu 18.04]
Compiler [gcc 7.5]

Additional context
The reason may be that when 'Socket::connect' return a ENETUNREACH error, the TcpClient will never be destoryed.
HttpClientImpl create a TcpClient and set 'retry_' to false.Then the Connector of TcpClient tries to connect, but gets a ENETUNREACH error. The Connector will not add itself to a new Channel unless it gets EINPROGRESS or else. No one handles the ENETUNREACH error. HTTP requests using the reverse proxy HttpClientImpl will remain in the buffer queue forever.

Hi, @KasdllS
This happens when you create way too many sockets in a short time. The typical solution is to reuse the same HTTP client for all of your requests going to the same host. Unfortunately there's almost nothing we can do in this case. It's the OS stopping the application creating more sockets. And drogon can't just block the application and wait for sockets. That will impact performances elsewhere.

Hi, @KasdllS
This happens when you create way too many sockets in a short time. The typical solution is to reuse the same HTTP client for all of your requests going to the same host. Unfortunately there's almost nothing we can do in this case. It's the OS stopping the application creating more sockets. And drogon can't just block the application and wait for sockets. That will impact performances elsewhere.

I'm sorry my wording made you misunderstand me.If you use a long connection to send an HTTP request, follow the steps I write above. You will find that the HTTP request blocked, because drogon will not return a response forever. If you closed your HTTP client, the socket will turn to the CLOSE_WAIT state and remain in this state until it is reclaimed by the system after hours.

Hi, @KasdllS
This happens when you create way too many sockets in a short time. The typical solution is to reuse the same HTTP client for all of your requests going to the same host. Unfortunately there's almost nothing we can do in this case. It's the OS stopping the application creating more sockets. And drogon can't just block the application and wait for sockets. That will impact performances elsewhere.

Hi, @KasdllS
This happens when you create way too many sockets in a short time. The typical solution is to reuse the same HTTP client for all of your requests going to the same host. Unfortunately there's almost nothing we can do in this case. It's the OS stopping the application creating more sockets. And drogon can't just block the application and wait for sockets. That will impact performances elsewhere.

Hi, @marty1885
I'm sorry my wording made you misunderstand me.If you use a long connection to send an HTTP request, follow the steps I write above. You will find that the HTTP request blocked, because drogon will not return a response forever. If you closed your HTTP client, the socket will turn to the CLOSE_WAIT state and remain in this state until it is reclaimed by the system after hours.

Hi, @KasdllS
This happens when you create way too many sockets in a short time. The typical solution is to reuse the same HTTP client for all of your requests going to the same host. Unfortunately there's almost nothing we can do in this case. It's the OS stopping the application creating more sockets. And drogon can't just block the application and wait for sockets. That will impact performances elsewhere.

Hi, @marty1885
I'm sorry my wording made you misunderstand me.If you use a long connection to send an HTTP request, set 'timeout' to 0, follow the steps I write above. You will find that the HTTP request blocked, because drogon will never return a response. If you closed your HTTP client, the socket will turn to the CLOSE_WAIT state and remain in this state until it is reclaimed by the system after hours.

Describe the bug Example "simple_reverse_proxy" exited because of fd over the limit.
To Reproduce Steps to reproduce the behavior:

Turn off the WIFI or Plug out the cable.
Startup the program.
Send HTTP request using short connection.
See Many sockets in CLOSE_WAIT.

Screenshots If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [ubuntu 18.04]
Compiler [gcc 7.5]

Additional context The reason may be that when 'Socket::connect' return a ENETUNREACH error, the TcpClient will never be destoryed. HttpClientImpl create a TcpClient and set 'retry_' to false.Then the Connector of TcpClient tries to connect, but gets a ENETUNREACH error. The Connector will not add itself to a new Channel unless it gets EINPROGRESS or else. No one handles the ENETUNREACH error. HTTP requests using the reverse proxy HttpClientImpl will remain in the buffer queue forever.

我这边也发现了这个问题。
在wifi网络很差没信号的时候 ， 使用提供的反向代理示例，我这边写了个文件描述符监控脚本，发现会出现大量的处于CLOSE_WAIT状态的socket。在我不去设置tcp_keepalive_time，要等到2个小时之后这些端口才会被系统渐渐回收。这些socket被慢慢累积到进程的默认最大描述符限制数(1024)时，drogon的日志中有sockets::createNonblockingOrDie与大量的too many open files打印，最终导致程序意外退出。


[Hub Generated] Review request for Microsoft.DBforPostgreSQL to add version preview/2023-03-01-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
ARM API Information (Control Plane)
Changes included in this PR:

New api version (2023-03-01-preview) added from the base GA version (2022-12-01) using Open API Hub.
Add support for CMK with geo backup (FlexibleServers.json). New properties added.
Add support for storage autoGrow (FlexibleServers.json). New properties added.
Add support for IOPS scaling properties (FlexibleServers.json). New properties added.
Add API for Advanced Threat protection (ThreatProtection.json).
Add swagger for Migrations API for this new API version (Migrations.json).
Add API for Server Logs (ServerLogs.json).
Add API for Long Term Retention Backup (LongTermRetention.json and LongTermRetentionOperation.json).
Add server capabilities API and update the response for the existing location capabilities API (Capabilities.json). Both these APIs will be used internally by CLI to fetch the desired information for the location capability / server capability and accordingly allow operations to be performed for the Postgres Flexible Server. Swagger is being exposed for these APIs for SDK to be generated which in turn can be used by CLI. There are breaking changes for these APIs to be more ARM compliant and follow ARM API design guidelines.

Breaking change request => https://msazure.visualstudio.com/One/_workitems/edit/17984930
Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

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

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

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

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

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

Change to data plane APIs
Adding new properties
All removals

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

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

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

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

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

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.
Breaking change request => https://msazure.visualstudio.com/One/_workitems/edit/17984930

You must commit to follow the breaking change policy.

/azp run

Please ensure to respond feedbacks from the ARM API reviewer. When you are ready to continue the ARM API review, please remove ARMChangesRequested

Removed ARMCHangesRequested as Breaking changes are now approved

/azp run

Hi @ambrahma please fix the LintDiff failure.

Hi @ambrahma please fix the LintDiff failure.

This is now fixed


Add endpoint to get coordinator DLC channels

Can be useful for debugging, as it is sometimes hard to figure out the state of a DLC channel by looking at the logs (or the UI 😛).

bors r+


Clean Up Registry

Trying to debug another problem I looked at the registry as searched for "MeerK40T"
There was a LOT of stuff from earlier versions, a LOT.
I would like to see a "CLEANUP" tool that cleans up all the garbage left behind.

I guess this might be done in a larger uninstall routine if meerk40t was installing. And I think this could be done with a .reg file if it was really needed.

MK doesn't use registry anymore.


Added ui button to refresh songs for easier usability for most users

adds a UI button just above the settings button that calls the RefreshSongs() method.
Easier for most of the users of the mod as not many people know about the press 'R' to refresh

@xyonico good to pull whenever you're ready


Allow newtype key

See #270

Thanks!
Er but actually now that I've merged it, mind adding a test for this as well?

Sorry I haven't replied. I've got the notice marked as unread in my inbox to remind me. That being said, I noticed that deserialization gets hung up on invalid type: map, expected tuple struct Map. Looks like it's already ticketed as #196 but it was believed to be solved.
Excerpts of the problem code:
/// A path of Identifiers
#[derive(Debug, Display, Eq, Hash, PartialEq, Serialize)]
pub struct Path(String);

#[derive(Debug, Deserialize, Serialize)]
pub struct Map(HashMap<Path, MapEntry>);

Not sure when I'll get to this as it's for a spare-time group project so it's lower priority atm.

Ok no worries, thanks again!


Update [ios.minimum-required-os-version] in gulpfile.js from 13.6 to 13.7

This PR updates the [minimum-required-os-version] in the gulpfile for iOS (13.6 --> 13.7)
Source for iOS 13.7: https://apps.apple.com/de/app/corona-warn-app/id1512595757

@MikeMcC399, @Ein-Tim ok, thanks! We will cross-check this information. Dipankar Sarkar


jmDNS Issue?

I am using the HAP-Java as a simple Bridge. HAP Java is built upon jmDNS 3.5.6 so the latest as I believe. I can register in my Home app on iPhone as soon as the mDNS is visible.
First start often fails but after restart of the Bridge I can see the announcement on my Mac using sd-dns command. I register the Bridge and one switch I have registered. All fine and the Home App shows the Bridge and the Switch.
Now very soon after the registration the Phone/iPad indicates a failure, Can't connect to the Accessory. Then again connection is available. This is ongoing.
If I look at the sd-dns -B output (See below) I can see Add of the announcement. Then 1-3 minutes later Rmv so it is removed. Then again 1-3 minutes later it is Added again. And it is clear that during the period when it is Removed the service is not working but it starts again as soon as the next Add appears.
Is this seen by others? I am running the Bridge on a raspberry pi 4 Buster. I have tried to both stop and as well have Avahi running, as I believe it can interfere.
Output from sd-dns -B:
19:30:44.950 Add 2 17 local. _hap._tcp. MyNewestTestBridge
19:33:54.092 Rmv 0 17 local. _hap._tcp. MyNewestTestBridge
19:36:51.017 Add 2 17 local. _hap._tcp. MyNewestTestBridge
19:39:55.727 Rmv 0 17 local. _hap._tcp. MyNewestTestBridge
19:41:47.931 Add 2 17 local. _hap._tcp. MyNewestTestBridge
19:42:57.181 Rmv 0 17 local. _hap._tcp. MyNewestTestBridge
19:47:35.785 Add 2 17 local. _hap._tcp. MyNewestTestBridge
Output from sd-dns -Z
_hap._tcp PTR MyNewestTestBridge._hap._tcp
MyNewestTestBridge._hap._tcp SRV 0 0 9123 lundqvistpi.local. ; Replace with unicast FQDN of target host
MyNewestTestBridge._hap._tcp TXT "c#=1" "s#=1" "ff=0" "sf=0" "sh=lhjaKg==" "ci=1" "md=MyNewestTestBridge" "id=9a:9b:44:29:77:ee"
/Anders

@alundqvi is this still an issue or can it be closed?

My system works... Hence I have not done any more investigations... So please close it!

Closing

Thanks for closing!


fix(manuals): update manuals to adapt to changed commands

What this PR does / why we need it:
Adapt to #211.
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/assign @ddysher
/assign @zjj2wry

Release note:

NONE

/assign @ddysher
/assign @zjj2wry

LGTM

/lgtm


adjust coprocessor directory

this pr adjust coprocessor directory
PTAL @andelf @AndreMouche @hhkbp2

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.ocean seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Please sign the CLA

LGTM

LGTM


Does not work after ejecting from create-react-app

Im really exited for this plugin. It is exactly what i have been looking for. I just need it to work after ejecting from create react app.

@burtontanner I haven't thought of it at first, but the thing is if CRA doesn't provide any support when you are ejecting that's because they cannot guarantee the changes you'll end up making to the config. The same logic applies for this tool. If you mess around with your webpack config I cannot guarantee that the tool will work and I have no way of knowing what every body will be doing with their config.
I don't think this tool should handle ejection. I directly modify CRA webpack config to make the development build on disk possible and if you touch your config you'll be messing around with what I did. The tool would not be able to work.
I'm closing this for now but I'm open to discussion.

@Nargonath that makes sense. I still need to find a solution to my problem and would like to leverage as much of your code as I can. I am probably going to fork your repo and fiddle around with it until I get it to work. My config files are untouched after ejecting. What do you think I would have to change to get it to work?

If you don't modify your config files, why are you ejecting in the first place?

@Nargonath I have just modified the jest configuration for testing. I doubt that your module will be affected by this so I didn't mention it. Could you point me in the right direction?

Indeed, it shouldn't be affected by it. I'm wondering if I will not implement it in the end. That's a pity CRA isn't built the same as Vue CLI did where they give a way to update the webpack config without having to eject. I could support it but really blindly assuming you didn't touch your webpack config and if ever you did then that's on you afterwards.

:tada: This issue has been resolved in version 1.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Sync red-hat-storage/ceph-csi:devel with ceph/ceph-csi:devel

Sync branches and test the Prow/Tide configuration to automated merging.

Except that there is an unexpected conflict with this PR (on the devel branch?!), we do not have an OWNERS file in the repository yet. This will need to be added, just like a odf directory similar to ceph-csi/ocs

/approve

/cc agarwal-mudit

/lgtm

going to close this and re-create it :crossed_fingers:


0254 add 2021 financial incentives

Context
2021 financial incentives
Changes proposed in this pull request
add 2021 financial incentives
Guidance to review
Affects both
https://github.com/DFE-Digital/find-teacher-training
https://github.com/DFE-Digital/publish-teacher-training/
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Financial Incentives


coverage + celery taskapp + testing = ??

Interesting issue:
I've done a little customization (but nothing crazy) on a basic cookiecutter-django installation.
When I run the normal ./manage.py test, everything is fine.
BUT if I do something like:
coverage run manage.py test
I get:
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 338, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 312, in execute
    django.setup()
  File "/usr/local/lib/python2.7/site-packages/django/__init__.py", line 18, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/usr/local/lib/python2.7/site-packages/django/apps/registry.py", line 85, in populate
    app_config = AppConfig.create(entry)
  File "/usr/local/lib/python2.7/site-packages/django/apps/config.py", line 112, in create
    mod = import_module(mod_path)
  File "/usr/local/Cellar/python/2.7.10_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "/path/to/my/project/project/taskapp/celery.py", line 4, in <module>
    from celery import Celery
ImportError: No module named celery

Context:
OSX Yosemite
Python 2.7.10
pip freeze:
alabaster==0.7.6
amqp==1.4.6
anyjson==0.3.3
appnope==0.1.0
Babel==2.0
billiard==<IP_ADDRESS>
celery==3.1.18
coverage==3.7.1
decorator==4.0.2
Django==1.8.4
django-allauth==0.23.0
django-autoslug==1.8.0
django-braces==1.8.1
django-crispy-forms==1.5.2
django-debug-toolbar==1.3.2
django-environ==0.3.0
django-extensions==1.5.6
django-floppyforms==1.5.2
django-model-utils==2.3.1
django-redis==4.2.0
django-secure==1.0.1
django-test-plus==1.0.9
docutils==0.12
factory-boy==2.5.2
gnureadline==6.3.3
greenlet==0.4.9
ipdb==0.8.1
ipython==4.0.0
ipython-genutils==0.1.0
Jinja2==2.8
kombu==3.0.26
MarkupSafe==0.23
msgpack-python==0.4.6
oauthlib==1.0.3
path.py==8.1.1
pexpect==3.3
pickleshare==0.5
Pillow==2.9.0
psycopg2==2.6.1
Pygments==2.0.2
python-openid==2.2.5
pytz==2015.4
redis==2.10.3
requests==2.7.0
requests-oauthlib==0.5.0
simplegeneric==0.8.1
six==1.9.0
snowballstemmer==1.2.0
Sphinx==1.3.1
sphinx-rtd-theme==0.1.9
sqlparse==0.1.16
traitlets==4.0.0
unicode-slugify==0.1.3
Unidecode==0.4.18
Werkzeug==0.10.4
wheel==0.24.0

Let me know if you need some additional information!
Thanks!

I have tested in my test project and coverage test works with no problem.

Your error seems related to celery, make sure you are running in the right environment, and that environment has installed celery.
Try to install it before running the coverage test:
celery==3.1.18

Thanks for the reply, Celery is installed in my virtual environment. The pip freeze I offered above does include it, although it is quite a large list of packages. Do you have Celery running in a different terminal or as a background process when you run your tests with coverage?

I can't replicate this problem so I'm closing it. If it is still a problem, please reopen.


Dashboard: Make stories list view header sticky to keep in view on scroll

Summary
Updates the stories list view to have a sticky table heading so that when stories are scrolled through in list view the headers stay visible to avoid confusion if focus is on them without disrupting where the user is on the view by moving the page.
Relevant Technical Choices

Uses squishContentHeight and DASHBOARD_TOP_MARGIN to give the table heading its top: value with position: sticky. The top value is passed to the stickyheader component as a prop so that it's not a fixed number.
Because our dashboard layout uses a provider to "squish" the space of the header on scroll I had to tweak the way the header sees it's space a little bit so that the values stay constant regardless of if there's a banner in there or not.

Basically, this boils down to wrapping the banner and setting height to the wrapper so that padding can be taken from the space provided (thus the bump from 161 to 171 since now that 10px margin top is coming from that value).
And then, no more padding for the pageHeader top - that space is now decided on by the heading itself.

Adjusted the z-index so table header is covering the images that scroll past it.

To-do

Combine this with Dillon's update for #4483

User-facing changes

Table header of stories list view should now become "sticky" when you scroll through stories making it stay visible.

Testing Instructions
Scroll through stories in list view, see that table heads stay in view and are interactive. Test with and without telemetry banner.

addresses #4420

Verified in QA


Revert "Improve support for non-master main branches"

This reverts commit 4c02977c219ef79b5eb09cb1fb39c4068de61559.

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

:warning: Warning :warning:

Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.46.0. Please double check that you specified the right target!

As a reminder we'll need to hit the big green button on this one, no bors working for beta

As CI is green...


inbox: Match unread count for stream with left sidebar.

Merged after redoing the comment. I think we still need to refactor this further to make this feel nice and in preparation for moving this file to TypeScript -- that function now has a very weird type, but I think it's correct logically at least, so seems worth merging.
In terms of the refactor: I feel fundamentally, we want to refactor get_stream_count and get_counts together, possibly to end up just having for each stream, the full set of {topic counts requested by include_per_topic_count, plus stream-level totals).
@prakhar1144 FYI.


sensitivity adjustment?

is there a sensitivity setting or console command to adjust sensitivity?
i am finding that no matter how still I hold my head it seems that even my heartbeat seems to be violently shaking the display.

I had a threshold set earlier that ignored tiny movements like heartbeat,
but I temporarily removed this while refactoring for the android changes.
I’m planning to add back the threshold + sliders for look sensitivity
per-axis, but this is low priority. Here is where the threshold was removed
in case you want to git checkout the older commit
https://github.com/3rl-io/spidgets-3dof/commit/914bf4f183b888503fadc52b347249295e30454e#diff-2b58c6f03c86bed2bdac45dcdf7e44fd7f94aa665b294c580ddd75c0c26ea805L25
On Sun, Jun 30, 2024 at 11:09 AM mnjm97 @.***> wrote:

is there a sensitivity setting or console command to adjust sensitivity?
i am finding that no matter how still I hold my head it seems that even my
heartbeat seems to be violently shaking the display.
—
Reply to this email directly, view it on GitHub
https://github.com/3rl-io/spidgets-3dof/issues/8, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ3FPG3SOB7AWHAPAW3NGNDZKBCWTAVCNFSM6AAAAABKEJOTJ6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGM4DEMZWGQ4TMNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I apologize, I am not nearly as smart as I think I am. I can't figure out how to git checkout this without messing up.  I should probably take a course on using git.

I made a stabilizing feature for this. It’s currently bundled with other
changes that are in progress. Will push this week
On Tue, Jul 9, 2024 at 7:45 AM mnjm97 @.***> wrote:

I apologize, I am not nearly as smart as I think I am. I can't figure out
how to git checkout this without messing up. I should probably take a
course on using git.
—
Reply to this email directly, view it on GitHub
https://github.com/3rl-io/spidgets-3dof/issues/8#issuecomment-2217923060,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ3FPG3ZOAM2PAJ3SCLIHD3ZLPZQPAVCNFSM6AAAAABKEJOTJ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJXHEZDGMBWGA
.
You are receiving this because you commented.Message ID:
@.***>


Updated ephemeral-volumes.md

Hello!
This is a documentation fix for https://kubernetes.io/docs/concepts/storage/ephemeral-volumes/#csi-driver-restrictions
Here the word 'NOT' was used on the line 130 in the markdown file of the same page. It has been changed to 'not'
Thanks

@Rohan-Dah Thanks you for your contribution.
Could you please sign the CLA before the PR can be reviewed.
You can follow the steps documented here: https://git.k8s.io/community/CLA.md

@Rohan-Dah Thanks you for your contribution. Could you please sign the CLA before the PR can be reviewed. You can follow the steps documented here: https://git.k8s.io/community/CLA.md

Done signing the CLA

Isn't upper case letters a common form for emphasizing something?
How could people interpret it as an acronym?

Isn't upper case letters a common form for emphasizing something? How could people interpret it as an acronym?

Yes it surely is a way of emphasizing... Also how about making the text bold as it might be a great way to do it as well?

Bold should be fine.
As a side note, do NOT waste your time on these things. You may find a lot more useful things to work on.


Design authentication & authorization for GCP

Describe the issue to be researched
Include any background information and available resources.
Describe the goal of the research
What's the desired outcome of this task? What artifacts should be produced?
Describe the methodology
Where can more information be found? Who should the assignee approach to ask questions? How can a decision be made?
Would you like to work on the issue?

Closing this issue as per discussion with @jingtang10 as it will not be part of Android FHIR SDK.
CC: @fredhersch


As GPIB, I want new IDEM signups to come via the IDEM proxy, so that I don’t get spam accounts

Context

Change the sites.json to include proxy = true
On the app if proxy is set to true, concat their callback URL to the proxy, such as : https://proxy.idem.com.au/users/signup?callback=https://portal.getpaidinbitcoin.com.au/register
Add an end point to the proxy as /user/signup (maybe basic auth from the IDEM app)

User Story

As the GPIB owner,
I want new IDEM signups to come via the IDEM proxy,
So that I don’t get spam accounts

AC

Given
When
Then

Doesn't work https://user-images.githubusercontent.com/50130291/177693990-ea8215f3-276e-46cd-866c-52bb752c3af4.mp4

Request table is being updated when a user signs up to GPIB through IDEM

Used the proxy to sign up even though useproxy=false in sites.json


Merge contracts audit fixes into master

Description
Merge audit fixes into master
Other changes
merge conflicts
Tested
Tests included on a per commit basis
Related issues
https://github.com/celo-org/celo-monorepo/issues/4812
Backwards compatibility
yes

still to fix merge conflicts.

going to do this from a different branch.


mistake structs enums lifetimes

But uncomment let get ... -> But uncomment let mut set ...

whoops nm

Exactly! ;)


Option to apply offset and limit manually in all query APIs for devices that do not support them

This is something that I considered in #243 but did not implement yet because no one wanted it until #252 (thanks to @itinsidenews)
Problem
As discovered in issue #241 and #252, some devices running a modified version of the Android OS such as REALME and OPPO do not support the LIMIT and OFFSET functionality in queries.
This means that queries may return ALL table rows despite the value passed in the limit() and offset() functions for those devices 😞
Solution
Add a forceOffsetAndLimit(Boolean) function to all query APIs. If it is true and when isLimitBreached is true, then we would manually trim the result after the query so that it would contain only the number of entities specified in offset and limit.
This is something that user of this library can do themselves in their app already even without this issue being implemented. However, it adds extra code for library users that can otherwise be a part of the lib.
Support this in;

[x] BroadQuery
[x] Query
[x] DataQuery
[x] BlockedNumbersQuery
[x] GroupsQuery
[x] AccountsRawContactsQuery

Make sure to make the documentation clear;

[x] in code
[x] in docs

Users need to know that manual trimming has occurred.

[x] Update gh-pages

This is included in 0.2.4!


fix toUnfoldable

Hello,
The current toUnfoldable implementation loses whole of mid nodes. This PR just fixes it.

Eek! Thanks for the fix 👍


Add rules for agent

manifestwork rules so work agent can get work
events rules for event recorder

/approve

/lgtm


PowerShell split

Learn Build status updates of commit 392cd25:
:warning: Validation status: warnings

File
Status
Preview URL
Details

tutorials/includes/powershell/app-only/02-register-app.md
:warning:Warning

Details

tutorials/includes/powershell/app-only/03-add-app-only-auth.md
:warning:Warning

Details

tutorials/includes/powershell/app-only/04-get-all-users.md
:warning:Warning

Details

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

tutorials/includes/powershell/01-intro.md
:white_check_mark:Succeeded

tutorials/includes/powershell/02-register-app.md
:white_check_mark:Succeeded

tutorials/includes/powershell/03-add-user-auth.md
:white_check_mark:Succeeded

tutorials/includes/powershell/04-add-user-auth.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

tutorials/includes/powershell/04-get-user.md
:white_check_mark:Succeeded

tutorials/includes/powershell/05-get-inbox-messages.md
:white_check_mark:Succeeded

tutorials/includes/powershell/05-get-user.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

tutorials/includes/powershell/06-get-inbox-messages.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

tutorials/includes/powershell/06-send-mail.md
:white_check_mark:Succeeded

tutorials/includes/powershell/07-extend-app.md
:white_check_mark:Succeeded

tutorials/includes/powershell/07-send-mail.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

tutorials/includes/powershell/08-completed.md
:white_check_mark:Succeeded

tutorials/includes/powershell/08-configure-app-only.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

tutorials/includes/powershell/09-add-app-only-auth.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

tutorials/includes/powershell/10-get-all-users.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

tutorials/includes/powershell/11-extend-app.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

tutorials/includes/powershell/12-completed.md
:white_check_mark:Succeeded
n/a (file deleted or renamed)

tutorials/includes/powershell/app-only/01-intro.md
:white_check_mark:Succeeded

tutorials/includes/powershell/app-only/05-extend-app.md
:white_check_mark:Succeeded

tutorials/includes/powershell/app-only/06-completed.md
:white_check_mark:Succeeded

tutorials/powershell.yml
:white_check_mark:Succeeded

This comment lists only the first 25 files in the pull request.

tutorials/includes/powershell/app-only/02-register-app.md

Line 66, Column 5: [Warning: file-not-found - See documentation] Invalid file link: '../../images/aad-portal-app-registrations.png'.
Line 76, Column 5: [Warning: file-not-found - See documentation] Invalid file link: '../../images/dotnet/aad-application-id.png'.
Line 90, Column 5: [Warning: file-not-found - See documentation] Invalid file link: '../../images/aad-configured-permissions.png'.

tutorials/includes/powershell/app-only/03-add-app-only-auth.md

Line 37, Column 5: [Warning: invalid-code] The code snippet "./src/app-auth/GraphTutorialAppOnly.ps1" could not be found.

tutorials/includes/powershell/app-only/04-get-all-users.md

Line 11, Column 5: [Warning: invalid-code] The code snippet "./src/app-auth/GraphTutorialAppOnly.ps1" could not be found.

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 Learn 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 or suggestions 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 the learn.microsoft.com contributor guidesPost your question in the Learn support channel


GridMap: Ensure the visibility is updated when entering the tree

Fixes #81000
The visibility of GridMap is not updated when entering the tree.

Thanks!

Cherry-picked for 4.1.3.


Username compare failing when usernames match but are not same case

https://github.com/fox-it/BloodHound.py/blob/dea250078b3728cc3bcbe60033138b39c0d34f93/bloodhound/ad/authentication.py#L283
The username comparison if expected_principal != str(tgt_principal): fails when supplied username and ccache username are the same user but not the same case (e.g. USER1 != user1). It looks like tgt_principal may need to be lower-cased so will succeed when compared to the expected_principal which is lowercase if the usernames are effectively the same.

thanks for the report, should be fixed in latest commit on master


[5.5] Bug error context

This PR fixes issue #21041 and provides a test case for PR #21047

duplicate of https://github.com/laravel/framework/pull/21047


Exception: Could not find module! lsasrv.dll

I dump secrets with command:
pypykatz lsa minidump lsass.dmp
I got error with "Could not find module! lsasrv.dll". Can you help me where directory which load file lsarv.dll.
Thanks.

Can we expect 'lsa minidump' to work on Linux if it requires lsasrv.dll? Pywin is not supported on Linux, Pypiwin is too old (syntax errors on Python 3.* in print statements at least). Where would this module come from for non-Windows users?

pypykatz doesn't require lsasrv.dll, the error message means that the dll was not found in the dump file. Most likely the issue is caused by the pypykatz version @thatpham used did not have the matching signature for that particular windows build.
pypykatz parses mindump on any operating system where python works, the entire parsing is implemented in python.

pypykatz doesn't require lsasrv.dll, the error message means that the dll was not found in the dump file. Most likely the issue is caused by the pypykatz version @thatpham used did not have the matching signature for that particular windows build. pypykatz parses mindump on any operating system where python works, the entire parsing is implemented in python.

I think i do encounter same issue on some specific machines whereas in others i dont. How can i handle that situation and how to know which version would work for me ? Please check my issue that i just created


unable to compile release/debug build for x86 (missing curl.h and rapidjson error)

hey all,
i'm trying to compile this launcher from source, and i can't seem to get it to build. the first error is that it's somehow not detecting i have curl installed, as it keeps complaining:
D:\git repos i stole lol\NorthstarLauncher\NorthstarDedicatedTest\masterserver.cpp(6,10): fatal error C1083: Cannot open include file: 'libcurl/include/curl/curl.h': No such file or directory
it's also complaining a lot about rapidjson, as it keeps saying:
C:\src\vcpkg\installed\x86-windows\include\rapidjson\document.h(665,59): error C2248: 'rapidjson::GenericMemberIterator<false,Encoding,Allocator>::Iterator': cannot access private typedef declared in class 'rapidjson::GenericMemberIterator<false,Encoding,Allocator>'
        with
        [
            Encoding=rapidjson::UTF8<char>,
            Allocator=rapidjson::MemoryPoolAllocator<rapidjson::CrtAllocator>
        ]
if i could get some help with this, that'd be amazing.

Since this issue was opened there have been some major changes to the launcher and we now also have build instructions.
Are you able to compile the launcher from source now or is this still and issue?

i recloned the entire repo for good measure, built the solution, and it went off with no errors! the only thing i'm a bit concerned about is that it'll only let me compile in x64, not x86. not sure why, but i'm going to try titanfall 2 with this freshly-compiled build!

i'm in! however, it's not letting me into multiplayer, just saying that it's connecting to respawn servers. how do i fix this?

Hmm, maybe try following steps 3 and onward from the install guide.
Also make sure you got the default mods (Client, Custom, CustomServers)  placed in Titanfall2\R2Northstar\mods\. These are required to run Northstar. They are just script text files, so no compilation there.
Basically you wanna just follow the the install guide](https://r2northstar.gitbook.io/r2northstar-wiki/installing-northstar/basic-setup) but replace binaries from the zip with the stuff you built yourself.

that did the trick! i'm going to close this issue because my issue is solved but i did have one last question: are there any special optimizations i can do if i build northstar myself? i know that gcc has the march options for cpu-specific optimizations, so i wanted to know if that sort of thing exists in the compiler visual studio uses.


String#matchAll is too restrictive in its parameter type

🔎 Search Terms
matchAll
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about RegExps and matchAll

⏯ Playground Link
https://www.typescriptlang.org/play?target=7#code/MYewdgzgLgBAtgTwMIFdojgeQEYCsCmwsAvDAN4CwAUDLTAFQDaAygnNiADYB0cAhlGAALAIKdOAXQAU0AE4BLMAHMAXDDmKlASjUBxfGHyyBIWQB4ASviUBRAB4AHe4RGzjCAHzlqdGAF9qAKpqUEgufG5OECUpRm54gHIAMxAQBN4BYTFOKURUdCw8QigtCS0AbmogA
💻 Code
const myCustomObject = {
    *[Symbol.matchAll](string: string): Generator<RegExpExecArray> {
    }
}

console.log([...'foo'.matchAll(myCustomObject)]);

🙁 Actual behavior
TS rejects this code, because myCustomObject is not an instance of RegExp.
🙂 Expected behavior
Any value matching the following type should be accepted, including myCustomObject. The other parts of the RegExp object are not used.
type MatchAllRegExpLike = {
  [Symbol.matchAll](string: string): IterableIterator<RegExpExecArray>;
  // If the object has a [Symbol.match] property that is truthy, its flags are checked. However, neither
  // must be present, and even if it is present, [Symbol.match] doesn't have to be a function.
  readonly [Symbol.match]?: any;
  readonly flags?: string;
};

Additional information about the issue
This is distinct from #47310. That issue is about string-to-regexp conversion; this is about regexp-like objects.

Same issue also exists for Symbol.match, Symbol.replace, Symbol.search, Symbol.split and their corresponding string methods.
The actual (runtime) types are pretty lenient, but deterministic. E.g. for matchAll, it's really this:
matchAll<T>(pattern: { [Symbol.matchAll](str: string): T; }): T;

And you could easily define a matcher like this where T is number or something crazy:
const matcher = {
    [Symbol.matchAll]: (str) => str.length
}

const x: number = 'abc'.matchAll(matcher)


Programm the comment-point (board message) logic

Board message will link the issue with the masseges, left on a board.

Created a new branch for this issue: comment-point-feature.


This plugins does not support your device (iOS)

Tried install it from Obsidian(iOS), but be prompted with "not support". is it possible to support on iOS device?

Current version of the plugin contains in its public version only the version for desktop PCs, however, in the next version I plan to change this parameter, opening access to the plugin also on mobile devices.

shortly, yes, and it planned, but testing will be on the side of the community.

Extra thanks for the issue!

Changed manifesto of this plugin, now the plugin should be available on all platforms, in case of bugs, errors, problems, create an issue.
https://github.com/Falcion/UnitadeOBSIDIAN/blob/03e30a7352e8f0b3df99b5ccf9414a2c31301766/manifest.json#L9C8-L9C8
Closing due new update of plugin.


Add Azure Blob Storage

Ticket Number

UCD-809
Description of change

Add a local version of Azure Blob Storage to Docker Compose and add to infra
Checklist

[x] Requires infrastructure changes
[x] I have updated the documentation accordingly
[x] I have merged commits to avoid fixing things in this PR
[ ] New features have tests
[ ] Breaking change (team conversation required)

Important
Please do not merge from master (please only rebase). This keeps the history linear and easier to debug.

To test this locally, I've added the following into the Document Service API server.js (it's dirty, but it shows it's working). This will need a service and all that good stuff for prod, but this proves the principal works.
const { BlobServiceClient } = require('@azure/storage-blob');

async function main() {
  const client = BlobServiceClient.fromConnectionString(process.env.BLOB_STORAGE_CONNECTION_STRING);

  const containerName = `newcontainer${new Date().getTime()}`;
  const containerClient = client.getContainerClient(containerName);
  const createContainerResponse = await containerClient.create();
  console.log(`Create container ${containerName} successfully`, createContainerResponse.requestId);

  const containers = client.listContainers();
  let i = 1;
  for await (const container of containers) {
    console.log(`Container ${i++}: ${container.name}`);
  }
}

main().catch((err) => {
  console.log(err.stack);
});

Have also tested in my own cluster and works

Don't deploy before demo today


fix(ngOptions): skip comments when looking for option elements

Closes #12190

LGTM

Woohoo - look at your @Narretz "merging" PRs!

Yeah, it looks nice! Too bad you cannot merge / push multiple commits at once. And thanks for the review!


ROX-23972: Change system. to network. for netpols

Comments in https://github.com/stackrox/stackrox/pull/11586/files justify why
Tested:
go test -v github.com/stackrox/rox/pkg/helm/charts/tests/centralservices
go test -v github.com/stackrox/rox/pkg/helm/charts/tests/securedclusterservices

/retest


Database Issues during import (1.5GB JTL file)

Describe the bug
We have noticed that our most recent tests are currently stuck in an "in progress" state.  This is visible by the UI yellow widget showing the number of processing test runs.
We have viewed the BE and DB logs and its clear the database is having issues.  It seems to get so far through before it throws a segmentation fault, and tries to run a recovery.  In the meantime, the BE shows that its aborted the upload,.
The backend logs show the following:

Notice the "Connection terminated unexpectedly".
Then viewing the DB logs:

Notice service process was terminated by signal 11, segmentation fault.
Presumably the other messages are because the DB was terminated.
It then seems to recover, and we are able to view historical test runs, we can upload smaller tests, but the 1.4GB jtl file is failing.  We have the DB running on its own ECS task now with 2vCPU and 8GB
All other containers are running on another task with 2vCPU and 4GB
Neither of the containers looked exhausted resource wise.

Hi @kierangirvan
This looks like timescaledb issue rather than a problem with the application itself.
Can you double-check the memory allocation? Does the running container have access to all of those 8 gb of ram?
Another thing to consider is the shared_buffers and effective_cache_size of the timescaledb/postgres settings.

@ludeknovy my point above regarding the scheduler not clearing stale test runs can be ignored, I've noticed overnight the scheduler indeed cleared up those staled test runs.

@ludeknovy but regarding the upload issue, we have historically managed to upload far larger files (4GB), so it doesn't appear to be a buffer/cache sizing issue.  We will attempt to upload another test now that the stale test runs have been housekept (maybe clearing those stale runs might have helped?).

@kierangirvan the scheduler has some period set after which it will clean the stale test reports up.

we have historically managed to upload far larger files (4GB), so it doesn't appear to be a buffer/cache sizing issue.
but maybe you've adjusted the DB settings back then? Did you try to adjust those values for the current DB?

anyways, here's a few more things to try:

TimescaleDB sometimes has issues with parallel query plans, especially on hypertables.
Temporarily disable parallel execution for troubleshooting: SET max_parallel_workers_per_gather = 0;

Reindex the hypertable:
REINDEX TABLE jtl.samples;

Check for corrupted chunks:
SELECT chunk_name, table_size_pretty
FROM timescaledb_information.chunks
WHERE hypertable_name = 'jtl.samples';

The configuration of the DB has not changed in months.  The issue looks to have started over the weekend when the DB was stopped (presumably AWS swapping us over to new hardware), by doing so we had 2 tasks (each running the DB) running for a short period, I suspect this is where the corruption has come about.
We are running those queries now and will post the output to help diagnose the issue.

To check for corrupted chunks, the query didn't work as exactly you had put it, but we removed the missing column and confirmed there is no corrupted chunks:
jtl_report=# SELECT chunk_name, table_size_pretty FROM timescaledb_information.chunks WHERE hypertable_name = 'jtl.samples';
ERROR:  column "table_size_pretty" does not exist
LINE 1: SELECT chunk_name, table_size_pretty FROM timescaledb_inform...
                           ^
jtl_report=# \d timescaledb_information.chunks
                       View "timescaledb_information.chunks"
         Column         |           Type           | Collation | Nullable | Default
------------------------+--------------------------+-----------+----------+---------
 hypertable_schema      | name                     |           |          |
 hypertable_name        | name                     |           |          |
 chunk_schema           | name                     |           |          |
 chunk_name             | name                     |           |          |
 primary_dimension      | name                     |           |          |
 primary_dimension_type | regtype                  |           |          |
 range_start            | timestamp with time zone |           |          |
 range_end              | timestamp with time zone |           |          |
 range_start_integer    | bigint                   |           |          |
 range_end_integer      | bigint                   |           |          |
 is_compressed          | boolean                  |           |          |
 chunk_tablespace       | name                     |           |          |
 chunk_creation_time    | timestamp with time zone |           |          |

jtl_report=# SELECT chunk_name FROM timescaledb_information.chunks WHERE hypertable_name = 'jtl.samples';
 chunk_name
------------
(0 rows)

We have now executed the other 2 queries (disable parallel execution and reindex of hypertable).
The upload once again failed, but the error is different, we are no longer seeing the segmentation fault, instead an exit code of 129?


Add python3-googleapi Rule

Please add the following dependency to the rosdep database.
Package name:
python3-googleapi
Package Upstream Source:
https://src.fedoraproject.org/rpms/google-api-python-client
https://packages.debian.org/bullseye/python3-googleapi
https://packages.ubuntu.com/focal/python3-googleapi
https://packages.gentoo.org/packages/dev-python/google-api-python-client
Purpose of using this:
Python3 version is the standard library now for this package

@connoranderson as cottsay pointed out, there is an existing key for python3-googleapi. The key that you've added is a duplicate and rosdep keys in each file should be lexically sorted.

It would be great to update the existing rule with the new target platforms found researching this.


Fix invalide use of encode on bytes object

Within the pywps.inout.inputs.ComplexInput the json convertion function
use encode where decode should be used. The patch fix this issue.
same as #647 but main branch as base
Overview
Related Issue / Discussion
Additional Information
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[x] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

@gschwind Tests are all passing. Would you like to add a test to prove it is doing the right thing?

@gschwind Tests are all passing. Would you like to add a test to prove it is doing the right thing?

@gschwind Tests are all passing. Would you like to add a test to prove it is doing the right thing?

I will have a look to add test, best regards

I did a test but I'm not satisfied because the serialization-deserialition is not symmetric :)
and the test is not symmetric too.

I updated the test to only test the decoding, it should be fine now


How to use pipcorn behind corporate proxy?

I have environment variables http_proxy and https_proxy set.
I get the following error:
{ Error: getaddrinfo ENOTFOUND www.youtube.com www.youtube.com:443     at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)   errno: 'ENOTFOUND',   code: 'ENOTFOUND',   syscall: 'getaddrinfo',   hostname: 'www.youtube.com',   host: 'www.youtube.com',   port: 443 }
Is it possible to use pipcorn behind a proxy?

I am not sure.
Are you able to connect to www.youtube.com via your normal proxy?
We have to figure out how to make the connection work on the Node.js layer, in this library we are just calling youtube.com, not doing anything special.
Leaving it open in case anyone knows how to help.


Difficulty working with Capybara run_server = true

I am testing an OAuth 2 provider using Oauth2 gem, Capybara, RSpec and Poltergeist.
The Capybara server does not seem to respond to the Oauth2 requests:
  visit '/' # Server responds
  save_and_open_page

  get '/' # Server responds
  puts last_response.body

  token = client.auth_code.get_token( @code, redirect_uri:redirect_uri )
 # Server does not respond. Raises:
    Faraday::ConnectionFailed:
      Connection refused - connect(2) for "localhost" port 3000

Capybara setup:
Capybara.app_host = "http://localhost"
Capybara.run_server  = true
Capybara.server_port = 3000

It seems that the server only responds when there is a visit, get or other HTTP request called from within the spec. I guess that the server is started and stopped with each request.
This creates some unexpected behavior (at least for me). I wonder if it would make more sense to start the server at the beginning of all specs, and stop it after? Or perhaps I am missing something?
Alternatively, is there a way to start and stop the Capybara explicitly?

OK, I found the issue. The spec must be tagged as :js. Otherwise it uses Rack::Test and the server is not started.


Fix "text area" text

It may be silly, but I didn't like to see "text area" appear in the UI messages, so I changed them all.

I don't mind this, it's correct

Great!  I'm glad you like it.  I meant to open the PR from branch to master within my fork, but I hadn't noticed that the PR referred to this repo.  I thought such a minor change might not be appreciated, so I meant to merge it into my master and save it to be included with another PR later.  So I closed this PR.
Good thing you reopened it, though.
I'd like to make more contributions to the project.  I have some ideas, but if you know of some issues that should be addressed first, I'm open to suggestions.

Thank you! Appreciated.
There are a few issues but the project is mostly dormant now with the main developer busy. Feel free to open issue to discuss new features to avoid possible disappointment after working on large PRs 😜
Currently the main issue is the incomplete merge of the Firefox extension into the Chrome one, but unless you use Firefox I assume you're not gonna fix that =)


8/25 Updates for CE App Container

Docs Build status updates of commit dc4208a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/develop-your-app/IoTCoreDefaultApp.md
:bulb:Suggestion

Details

windows-iotcore/TOC.yml
:white_check_mark:Succeeded

windows-iotcore/learn-about-hardware/USB-Support.md
:white_check_mark:Succeeded

windows-iotcore/manage-your-device/Overview.md
:white_check_mark:Succeeded

windows-iotcore/tutorials/quickstarter/PrototypeBoards.md
:white_check_mark:Succeeded

windows-iotcore/windows-ce-app-container-getting-started.md
:white_check_mark:Succeeded

windows-iotcore/windows-ce-app-container.md
:white_check_mark:Succeeded

windows-iotcore/develop-your-app/IoTCoreDefaultApp.md

Line 71, Column 1: [Suggestion-image-alt-text-duplicated] Alt text 'How to upload music files' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning in late October, 2020.
Line 77, Column 1: [Suggestion-image-alt-text-duplicated] Alt text 'How to upload music files' is duplicated. Within an article, alt text must be unique. NOTE: This Suggestion will become a Warning in late October, 2020.

For more details, please refer to the build report.
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.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Updated Version Numbers

The old version number seemed to reference data types that are not available in the main branch of the compiler repo, but rather a not-yet merged in feature branch. The new version number references a package made from the main branch of the compiler repo.

This won’t be able to merge. We are in an awkward state where QIR support is in main here, including tests that are part of the build pipeline, while the required changes to the compiler to make QIR generations work are in a working branch of Qsharp-compiler. We need to get those changes merged into main in compiler, but to keep runtime building we need to use those packages here.

How long before the main branch on the compiler repo has the data types needed to work with this?

This won’t be able to merge. We are in an awkward state where QIR support is in main here, including tests that are part of the build pipeline, while the required changes to the compiler to make QIR generations work are in a working branch of Qsharp-compiler. We need to get those changes merged into main in compiler, but to keep runtime building we need to use those packages here.

How long before the main branch on the compiler repo has the data types needed to work with this?

And can those tests be marked as Skip until that happens?

This won’t be able to merge. We are in an awkward state where QIR support is in main here, including tests that are part of the build pipeline, while the required changes to the compiler to make QIR generations work are in a working branch of Qsharp-compiler. We need to get those changes merged into main in compiler, but to keep runtime building we need to use those packages here.

How long before the main branch on the compiler repo has the data types needed to work with this?

And can those tests be marked as Skip until that happens?

It sounds like there will be some time put aside next week to address this and try to get the main of the compiler repo to agree with the main of the runtime repo. If so, this PR may not be needed.

In the meantime, if you are blocked, you can try 0.15.2102129941-alpha, which should hopefully have the changes you need in it.

This was addressed in the compiler PR here (https://github.com/microsoft/qsharp-compiler/pull/893). With this the main branch of the compiler is now compatible with the main branch of this repo. This makes this PR unnecessary now.


fix: missing dependency end-to-end => yarn-project

This is failing on e2e-card-game - but this is the cause of it originally being missed.

@charlielye mind reviewing once e2e-card-game is resolved/reverted?

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 3fd7025a and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
5,700
18,884
36,452

l1_rollup_calldata_gas
66,552
239,348
470,372

l1_rollup_execution_gas
194,500
500,654
909,630

l2_block_processing_time_in_ms
1,189 (+1%)
4,482
8,933

note_successful_decrypting_time_in_ms
204 (+1%)
540
1,004 (+2%)

note_trial_decrypting_time_in_ms
9.15 (+13%)
59.3 (-2%)
35.8 (-70%)

l2_block_building_time_in_ms
11,210 (-1%)
44,650 (-1%)
89,628 (+1%)

l2_block_rollup_simulation_time_in_ms
7,271
29,295 (-1%)
58,841 (+2%)

l2_block_public_tx_process_time_in_ms
3,910 (-1%)
15,290 (-1%)
30,644 (-1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,418 (+1%)
26,894 (-2%)

note_history_successful_decrypting_time_in_ms
1,236
2,388 (+1%)

note_history_trial_decrypting_time_in_ms
83.0 (-3%)
137 (+7%)

node_database_size_in_bytes
18,780,240
35,307,600 (-1%)

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
253
44,736
28,001

private-kernel-ordering
194 (-1%)
52,625
14,627

base-rollup
656
177,932
997

root-rollup
68.7 (-1%)
4,320
825

private-kernel-inner
321
73,715
28,001

public-kernel-app-logic
172
32,254
25,379

merge-rollup
8.52
2,840
997

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
2 leaves
8 leaves
16 leaves
32 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves

batch_insert_into_append_only_tree_16_depth_ms
9.83
10.6 (-8%)
12.4 (+1%)
16.1 (-4%)
21.9
35.3 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.9
17.5
23.0
31.6
47.0
79.0
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.571
0.588 (-8%)
0.528 (+1%)
0.499 (-4%)
0.458
0.440 (-1%)
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
N/A
N/A
N/A
46.1 (+1%)
71.9 (-1%)
231
444
866
1,716

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
N/A
N/A
N/A
96.0
159
543
1,055
2,079
4,127

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.473 (+1%)
0.444 (-1%)
0.421
0.416
0.413
0.412

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
N/A
N/A
N/A
53.7
107
340 (+1%)
662
1,311
2,621

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
N/A
N/A
N/A
104
207
691
1,363
2,707
5,395

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
0.480
0.481
0.462
0.456
0.456
0.453

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
61.1
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
109
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
0.535
N/A
N/A
N/A
N/A
N/A
N/A

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts

tx_size_in_bytes
19,179

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
2,696 (-2%)
1,373

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
0.0289 (-10%)
477


Shall we get revised sails-redis with latest npm redis version 2.4.2

RajRajen:lb4b-api rajrajen$ npm-check -u
npm WARN unmet dependency //lb4b-api/node_modules/sails-redis requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency //lb4b-api/node_modules/redis,
npm WARN unmet dependency which is version 2.4.2

@rasekar2000 yes we shale, it is been long overdue!

@rasekar2000 @Ryanc1256 :+1:
You guys rock!

@mikermcneil rock(s) plural :P and ill try to make a PR with my proposed changes for #80 this weekend and ill update all the dependencies, if I get time to do it... super busy lately lol...

Thanks.

Please reopen .. @Ryanc1256

@rasekar2000 woops done :D

:thumbsup:

@rasekar2000 @Ryanc1256 Hey everyone, thanks for the all the input so far on this one. We're moving all the redis cases to the main Sails repo for a closer eye from the Sails team and some added exposure to the community. I see this issue was re-opened and welcome any further developments!


Remove travel reimbursements section from application

FEATURE REQUEST
What do you want to happen
In the application there is a section for travel reimbursements. This should be removed as we are not paying for travel to/from the hackathon
Anything else we need to know?

Just took a look, this should be possible by just editing hackathon-variables.py


feat(astro): support lessons without parts or chapters

Closes https://github.com/stackblitz/tutorialkit/issues/304

Adds support for defining lessons without chapters or parts.
src/content/tutorial
│
├── meta.md # Tutorial metadata
│
└── lesson-1 # Lesson in root! No part or chapter.
   ├── content.md
   ├── _files
   └── _solution

src/content/tutorial
│
├── meta.md # Tutorial metadata
│
└── part-1
    └── lesson-1 # Lesson in part! No chapter.
        ├── content.md
        ├── _files
        └── _solution

Custom order can be defined using lessons in metadata:
---
type: 'tutorial'
lessons: ['lesson-one', 'lesson-two']
---

---
type: 'part'
lessons: ['lesson-one', 'lesson-two']
---

Mixing structures is not supported and an error is thrown when these are not followed:

A tutorial can have one-or-many part's or one-or-many lesson's
A part can have one-or-many chapter's or one-or-many lesson's

However you can have multiple part's, where some contain just chapter's and some contain just lesson's.

/pkg-pr-new

/pkg-pr-new

/pkg-pr-new

Thanks for review @Nemikolh, let's get this merged! 💯


question to router

where are the available functions on the router? Documentation?
How do I clear history? pop all?

/history/lib/Actions.js?
push/replace/pop?

The documentation for the router can be found in the react-router repository itself. One straight forward example of using the router api directly can be found in modules/Go.

That answer does not really satisfy my question, but I guess that just is what it is.


filesystem.getEntryPath -> filesystem.getWorkingDirectory

depending on what version of Xenon is going to be included in the VM, we may need to filesystem.getEntryPath -> filesystem.getWorkingDirectory
here for example

not relevant anymore, using absolute paths everywhere now


Grid style error in the generated code

When a new grid is added, the generated code has horizontal scrolling.

Solved with Hot Fix.
Searching better solution...
Example grid show the following code:

But adding de 12 grid, the code is the following one:

I appreciate some help to understand it.

Hi sorry I did understand is solved with an Hot fix is correct?

Yes, I solved it in this commit.
The fix solves the problem, but it would be good a deeper revision.


add datatype module

close #3

Codecov Report

Merging #4 (262de4a) into main (adaf556) will increase coverage by 2.44%.
The diff coverage is 86.62%.

@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   86.67%   89.11%   +2.44%
==========================================
  Files          38       35       -3
  Lines        6320     6305      -15
==========================================
+ Hits         5478     5619     +141
+ Misses        842      686     -156

Impacted Files
Coverage Δ

xngin-catalog/src/lib.rs
25.00% <ø> (ø)

xngin-catalog/src/mem_impl.rs
63.63% <ø> (ø)

xngin-compute/src/col/mod.rs
100.00% <ø> (ø)

xngin-compute/src/col/nonsized.rs
0.00% <ø> (ø)

xngin-compute/src/col/sized.rs
89.71% <ø> (ø)

xngin-compute/src/col/typed.rs
94.02% <ø> (ø)

xngin-plan/src/builder/tests.rs
95.00% <ø> (ø)

xngin-datatype/src/lib.rs
42.85% <42.85%> (ø)

xngin-plan/src/resolv.rs
86.17% <47.61%> (-2.11%)
:arrow_down:

xngin-expr/src/lib.rs
64.67% <50.00%> (-1.14%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


Added Danger, PR linter.

          1 Error

:no_entry_sign:
One of the lines below found in CHANGELOG.md doesn’t match the expected format. Please make it look like the other lines, pay attention to periods and spaces.

          1 Warning

:warning:
Unless you’re refactoring existing code, please update CHANGELOG.md.

Here's an example of a CHANGELOG.md entry:
* [#21](https://github.com/artsy/estella/pull/21): Added Danger, PR linter. - [@dblock](https://github.com/dblock).

* [#20](https://github.com/artsy/estella/pull/20): Added ES compaibility note to readme.

  Generated by :no_entry_sign: danger

Coverage remained the same at 99.441% when pulling 3d5bde78f195b197103c26dd481ee10b67f559a8 on dblock:danger into 63054e5bb9b5f95a74a3b4aebedc6411c1f97a5f on artsy:master.


Deposit amount can't be written

Pre conditions:

Being logged at Metamask
-Test on Staging

Steps to reproduce:
1- Enter https://dx.staging.gnosisdev.com
2- Choose a Token pair
3- Click on "Specify amount to deposit" CTA
4- Try to introduce a new amount to deposit
5- Observe
Expected result:
You can add numbers in order to specify amount of tokens for the transaction.
Actual result:
Typing numbers won't affect the gap.
Environment:
Chrome/Firefox/Brave/Edge - Windows 10

It also happens in https://dx.dev.gnosisdev.com/

Note this issue is for dx-react. Move it there pls :)
Thanks,

I'm on that!


[Table] Create a non responsive variation, or make disable responsiveness by default

Forcing tables to stack on mobile makes it impossible to implement more advanced application-specific width-shrinking techniques while keeping the (very nice) semantic-ui styling.  It would be best if the default .ui.table was not responsive and required an extra .stackable variant to enable the stacking behavior.  If that's not possible due to backwards compatibility, then a new .unstackable variant would be fine too -- I implemented this as a workaround in my project, but it's a pain due to all the !important overrides that the media rule brings in.

Is there already a better way?


Source bing webmaster

Tell us what you do here

[x] implementing verified source (please link a relevant issue labeled as verified source)
[ ] fixing a bug (please link a relevant bug report)
[ ] improving, documenting, and customizing existing sources (please link an issue or describe below)
[ ] anything else (please link an issue or describe below)

Relevant issue
issue #310
More PR info
##TODO before merge

[ ] copy unit tests into the test file
[ ] implement test cases for data loading
[ ] write README.md

Hi, could you please give me feedback on:

how to parse the unix timestamp int into a date if the CI bans import of datetime.date. I didn't have time yet to check how to achieve that with dlt.commons.pendulum. So, if it's trivial for you I'd appreciate a pointer.
If you can obtain an API key from Bing Webmaster and share it with me so that I can implement the test suite that calls the API. If you could verify your dlthub.com domain with Bing that would be great! I don't think you would need to submit one or two pages for indexing because I could find your pages already via bing.com.

Should we also add a README.md?

absolutely, I'll type that up together with the integration test suite!
Thank you so much @AstrakhantsevaAA and @sultaniman for your review, I'll work on your points.


Background Color does not change despite i set it for cell layout

Hi dear,
i set a color in cell layout for background but when i run it is still  white?

You need to set a TableView listener to be able to show your cell background color. Actually, It's a bug and it wouldn't be necessary. By the way, I have already fixed this problem. This weekend, I will be published new release. And it will be got this fix as well.

thanks man , as soon as possible.


Compilation error on version 2.2.0

Just recently updated to the latest version (last night?) and it doesn't work. See stacktrace below.
================================================================================
Recipe Compile Error in /var/chef/cache/cookbooks/iptables/resources/rule.rb
================================================================================

NoMethodError
-------------
undefined method `property' for Chef::Resource::IptablesRule:Class

Cookbook Trace:
---------------
  /var/chef/cache/cookbooks/iptables/resources/rule.rb:20:in `class_from_file'

Relevant File Content:
----------------------
/var/chef/cache/cookbooks/iptables/resources/rule.rb:

 13:  #
 14:  # Unless required by applicable law or agreed to in writing, software
 15:  # distributed under the License is distributed on an "AS IS" BASIS,
 16:  # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 17:  # See the License for the specific language governing permissions and
 18:  # limitations under the License.
 19:
 20>> property :name, kind_of: String, name_attribute: true
 21:  property :source, kind_of: String, default: nil
 22:  property :cookbook, kind_of: String, default: nil
 23:  property :variables, kind_of: Hash, default: {}
 24:
 25:  action :enable do
 26:    execute 'rebuild-iptables' do
 27:      command '/usr/sbin/rebuild-iptables'
 28:      action :nothing
 29:    end

I ran into this as well.

This cookbook now requires Chef 12.5+, you are using a version older than that.

Would it make sense to have chef_version in the metadata. Might help with the issue


Move static-site branch to separate repository

Task Description
Title says it all.
Keeps repository size small, avoids issues with hosting

Closing in favor of #5961

Closing in favor of #5961


Add instructions in the README (and elsewhere?) about setup steps, like making secrets

How do we make this accessible to developers on github? Link to README in the console logs? How to make it really pop out? Use the link in a workflow step name?

Interview added at https://github.com/plocket/docassemble-alautomatedtestingtests handles helping people set up steps.


Use uuid for broker id while shortid for client id

@mcollina sorry my bad, we need to change uuid/v5 to uuid/v4 for a different broker id at every run.

can you send another PR?
Il giorno lun 2 set 2019 alle ore 17:53 Gnought<EMAIL_ADDRESS>ha scritto:

@mcollina https://github.com/mcollina sorry my bad, we need to change
uuid/v5 to uuid/v4 for a different broker id at every run.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mcollina/aedes/pull/318?email_source=notifications&email_token=AAAMXY4AFBKBZHKEDAW2LT3QHUZJLA5CNFSM4IS6OQB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5WE7MY#issuecomment-527191987,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAMXY5HKH6YRDPG35DJXITQHUZJLANCNFSM4IS6OQBQ
.