request processing slows down on concurrent requests.

From @rkdrnfds on March 29, 2018 6:43
Request handling extremely slows down on concurrent requests
[Route("Test")]
public class TestController : Controller {
        [HttpGet("DoNothingGet")]
        public void DoNothingGet() { }
}

Against above controller, I tested concurrent request using shell script below
for i in {1..50}
do
curl -X GET "http://localhost:5000/Test/DoNothingGet" -H "Host: <IP_ADDRESS>:5000" -H "X-Debug: true" -H "User-Agent: Terminal" -d "[0, 0]" &
done

For sequential request, there's almost no delay for processing requests. consuming about 1~5ms per request.
but For concurrent request, handling time varies from 100ms ~ 2000ms
There's light middleware attached for logging execution time.
public class ProcessingTimeLoggingMiddleware
    {
        readonly RequestDelegate _next;
        readonly ILogger<RequestLoggingMiddleware> _logger;

        public ProcessingTimeLoggingMiddleware(
            RequestDelegate next,
            ILogger<RequestLoggingMiddleware> logger
        )
        {
            _logger = logger;
            _next = next;
        }

        public async Task Invoke(HttpContext context)
        {
            _logger.LogInformation("Start Middleware");
            var sw = Stopwatch.StartNew();

            await _next.Invoke(context);

            sw.Stop();

            _logger.LogInformation(LoggingEvents.RequestProcessingTime,
                "Processing {Path} [{StatusCode}] took {ElapsedTime}ms.",
                context.Request.Path, context.Response.StatusCode, sw.ElapsedMilliseconds
            );
        }
    }

Below are collected logs for slow request. (one of 50 requests. others also have similar logs)
March 29th 2018, 15:50:20.286 Microsoft.AspNetCore.Hosting.Internal.WebHost
Request starting HTTP/1.1 GET http://<IP_ADDRESS>:5000/Test/DoNothingGet application/x-www-form-urlencoded 6

March 29th 2018, 15:50:20.412 Server.Middlewares.RequestLoggingMiddleware
Start Middleware

March 29th 2018, 15:50:21.445 Server.Middlewares.RequestLoggingMiddleware
Processing /Test/DoNothingGet [200] took 830ms

March 29th 2018, 15:50:22.439 Microsoft.AspNetCore.Hosting.Internal.WebHost
Request finished in 2166.119ms

I tried manipulating ThreadPool.SetMinThreads and KestrelServerOptions.Limits.MaxConcurrentConnection but nothing had any effect.
Environment data
<PackageReference Include="Microsoft.AspNetCore.Hosting" Version="2.0.0" />
    <PackageReference Include="Microsoft.AspNetCore.Mvc" Version="2.1.0-preview1-final" />
    <PackageReference Include="Microsoft.AspNetCore.Owin" Version="2.0.0" />
    <PackageReference Include="Microsoft.AspNetCore.Server.Kestrel" Version="2.0.0" />

Product Information:
 Version:            2.1.4
 Commit SHA-1 hash:  5e8add2190

Runtime Environment:
 OS Name:     Mac OS X
 OS Version:  10.13
 OS Platform: Darwin
 RID:         osx.10.12-x64
 Base Path:   /usr/local/share/dotnet/sdk/2.1.4/

Microsoft .NET Core Shared Framework Host

  Version  : 2.0.5
  Build    : 17373eb129b3b05aa18ece963f8795d65ef8ea54

Copied from original issue: dotnet/cli#8930

@sebastienros - can you take a look at this?

1- Do you have Console logging enabled? If so please try without it.
2- How are you creating the concurrent load? I can see the serial one only, using curl commands.

1- Disabling Console logging changed nothing.
2- By attaching '&' at the end of the curl command, it runs in the background. I tested sequential request without '&', concurrent request with '&' and there's really big performance gap between the results of two.

Changing SetMinThreads to (1024, 1024) which was 100, 100 before gave me some improvements. Now handling time of slow requests ranges 100 ~ 1500ms

We do a lot of perf testing on aspnet, the results are available here: https://aka.ms/aspnet/benchmarks, so I am quite confident there is a flaw in your testing. Can we agree on the methodology and try again your app?

Published the application in Release mode: dotnet publish -c Release -o c:\temp\myapp
Run the published application: dotnet c:\temp\myapp\myapp.dll
Use a load test generator (instead of curl) that will create concurrent requests, please use Apache Bench (Linux and Windows) or Wrk (Linux only) to start a load test

In parallel I will try to reproduce your numbers locally to understand what is happening. Could you share the exact application somewhere (on github for instance) to be sure we are testing the same thing?

Yes, the debugger can add an enormous amount of overhead to the system. Some parts of the system change their behavior when they detect an attached debugger to offer additional diagnostic information.
Given that this was running with the debugger, I'm closing this issue. Thanks!

As a note, your debug performance issue will likely be fixed by https://github.com/dotnet/coreclr/issues/18705 (coming in 2.2)


Add Embrace Open Source

I think @bukinoshita will not mind if I add his project :smiley:

Thanks @lex111, much appreciate! 🍪

We want to keep this list short for now but may consider expanding it in the future. Thanks for the PR, though!

@MikeMcQuaid ok, then can add in opensource.guide? I created PR


urgent

Expected content
Actual content
Link to page:

hello i ned to have the e,mail of the support becsause i cant login in my account

Hello @LEAD1809,
Thanks for submitting a GitHub issue! We are very sorry that you are running into this problem. In order to better serve you, as this does not present itself as a library specific issue, we would like to ask that you reach out to our support team at https://support.sendgrid.com.
Thank you!
SendGrid DX Team


Upgrade Linux Kernel for main from 6.1.38 to 6.1.42

Subject says it all.

See instead https://github.com/flatcar/scripts/pull/1044


Support extracting WhiteSource data through API after the scanning.

This is related to @ryanbogan where we need to extract the data through whitesource API after the scan and use it to build the dependency tree later.
This issue is used to record all the related activities.
Thanks.

[Triage] @peterzhuamazon @ryanbogan - can you add more details to the task? Also, can you add Acceptance Criteria to this as well?
Thank you!

@ryanbogan Can you please confirm if you are actively working on this requirement?

Hey @bbarani he's not working on it -- that's on me.  I think it needs a new owner

Closing this issue for now. We can open a new one if needed.


Feature request: onX for type-level annotations.

https://projectlombok.org/features/experimental/onX.html

The onX parameter is not legal on any type-wide variant. For example, a @Getter annotation on a class does not support onMethod.

Why not, and can it be made to work please?
This is just what I need to avoid repeating Jackson annotations (e.g. @JsonView) for every field.

I'd bet, it does, it's even used on the linked page. What's unsupported is onParam on a @Getter and I guess, you don't need it. :smile_cat:
IOW it's a documentation bug.

No it's not a documentation bug. You also get a compile error.

Error:(16, 1) java: 'onMethod' is not supported for @Getter on a type.


Latest raspistill hangs on current production

Needs firmware update, working on it. Flowdock
See Raspbian workaround mentioned at https://github.com/balena-io-projects/balena-rpi-nodejs-picamera/issues/3#issuecomment-520134846

[zrzka] This issue has attached support thread https://jel.ly.fish/#/support-thread~c7fb4180-69e6-46f6-9ca4-a33cf34aae3d

[zrzka] This issue has attached support thread https://jel.ly.fish/#/support-thread~cc0234b8-e2e7-4c02-805f-0fe7d896789d

Seems to be working on 2.41.0+rev3, but have to wait for some new releases to ensure.

@floion will update the BSP with comes with an updated firmware and kernel. Please try it with those changes.


specification optimization code structure

Seems there is conflict, could you merge main branch?

不好意思我才想起来，我好像没有权限，有一些冲突

我本地重新推一个pr


PARQUET-908: Fix shared library visibility of some symbols in types.h

here's the build from my branch, will keep an eye on https://travis-ci.org/wesm/parquet-cpp/builds/209006672

merging, as this will also fix the arrow builds


Fix multi-GPU training

What does this PR do?

Change the DDP trainer's strategy.
Remove use of trainer's gpus property, since it is deprecated and was affecting the use of devices property.
Update training instructions on GPUs in the README.md file.

Type of change
Please check all relevant options.

[ ] Improvement (non-breaking)
[x] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

[x] Train FasterRCNN model with Carla dataset using 1 GPU:

python -m mart experiment=ArmoryCarlaOverObjDet_TorchvisionFasterRCNN trainer=gpu fit=true

[x] Train FasterRCNN model with Carla dataset on multiple GPUs:

python -m mart experiment=ArmoryCarlaOverObjDet_TorchvisionFasterRCNN trainer=ddp trainer.devices=2 fit=true

Before submitting

[x] The title is self-explanatory and the description concisely explains the PR
[x] My PR does only one thing, instead of bundling different changes together
[ ] I list all the breaking changes introduced by this pull request
[x] I have commented my code
[x] 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
[x] I have run pre-commit hooks with pre-commit run -a command without errors

Did you have fun?
Make sure you had fun coding 🙃

Why we need to bind datamodel.world_size with trainer.devices?

Why we need to bind datamodel.world_size with trainer.devices?

Because batch_size = datamodule.ims_per_batch / datamodule.world_size
https://github.com/IntelLabs/MART/blob/6184ca764c871cc098a4e822e520f476e556a971/mart/datamodules/modular.py#L59-L65

One could also bind trainer.devices to datamodule.world_size. Either way works, but ought to be the same number (assuming single-node multi-gpu training).


Remove "TestProject" sample

I don't understand what the TestProject sample is supposed to capture. It looks like it might have been accidentally checked in.

@qmatteoq Did you mean for this sample to be checked in?

It sounded like it might have been an accident that it's there. Go ahead and remove and we can restore if that's in error.

Yes, feel free to remove, it seems it has been captured by mistake. Sorry for the issue and sorry for the delay in getting back to you, I was on vacation 😊


Why the location of 'if' in 'setExecuting' and 'setFinished' is different？

- (void)setExecuting:(BOOL)executing {
    if (_executing != executing) {  // "if" is here
        [self willChangeValueForKey:@"isExecuting"];
        @synchronized(self) {
            _executing = executing;
        }
        [self didChangeValueForKey:@"isExecuting"];
    }
}

- (void)setFinished:(BOOL)finished {
    [self willChangeValueForKey:@"isFinished"];
    @synchronized(self) {
        if (_finished != finished) {       // "if" is here
            _finished = finished;
        }
    }
    [self didChangeValueForKey:@"isFinished"];
}

It's hard to read unformatted code..

There's no reason for these to be different. I have changed to make them consistent.


Notebooks - Reverting output change in diff does not revert input

VS Code Version: 1.61.0-insider
OS Version: Windows

Steps to Reproduce:

Open notebook and create any output change
Open diff view
Revert the output - input does not revert

Since the output is generated by the input, I would imagine that reverting the output would cause the input to be reverted as well so these are kept in sync. If I just revert the output and go back to the original file, I now have a mismatched cell - input code that does not match the output

This seems like it's by design. Putting back into triage.

If you undo an output, I wouldn't expect the input code to change back too. That should be a secondary undo in my opinion.

I disagree with this feature.
both are two separate actions, adding input is one action, and adding output is a separate action by running the cell (can be done 2 days later)
Hence I'd expect to see just an undo for the input.
To me removing the input when undoing output is like expecting hitting undo on a file file edit to result in the deletion of the file, just because the file was first created and then edited to add contents.

This is a side effect of notebooks having an implicit link to a runtime. Regular files do not have that. To me this is a question that a data scientist could probably answer better than any of us. Does it matter that the outputs of a notebook cell do not match the input code in general? Would users like a warning if they try this at the very least? Would they prefer it to be a setting? etc. etc.

After discussing, we're closing for now. Soojin is going to keep an eye on how people expect reverts to happen with notebooks.


[WIP] Change textures based on Pixeludi pack

https://github.com/SirCmpwn/TrueCraft/blob/7f0e3338d231b8a8f7fba35742b618a4fb3575bf/TrueCraft.Client/Content/terrain.png
Pixeludi pack Creative Commons 2011 by by Wojtek Mroczek

The textures have a 0x7f7f7f background, solid gray:

but Craft uses 0xff01ff magenta for transparent. Changed it with the flood fill tool, substituted in block breaking animation texture section:

but it doesn't look right. The color is too white:

probably it is meant to be an alpha layer over the actual block texture?

The block break texture is meant to be blended, see: glBlendFunc, glBlendEquation. TrueCraft does it this way:
            DestructionBlendState = new BlendState
            {
                ColorSourceBlend = Blend.DestinationColor,
                ColorDestinationBlend = Blend.SourceColor,
                AlphaSourceBlend = Blend.DestinationAlpha,
                AlphaDestinationBlend = Blend.SourceAlpha
            };
...
                Game.GraphicsDevice.BlendState = DestructionBlendState;
                ProgressMesh.Draw(DestructionEffect);
                Game.GraphicsDevice.BlendState = BlendState.AlphaBlend;
                Game.GraphicsDevice.RasterizerState = RasterizerState.CullCounterClockwise;

The C# code translates to in C:
glBlendFuncSeparate(GL_DST_COLOR, GL_SRC_COLOR, GL_DST_ALPHA, GL_SRC_ALPHA);

this looks better, but darkens the block too much:

Magenta-discarding is in shaders/block_fragment.glsl:
    if (color == vec3(1.0, 0.0, 1.0)) {
        discard;
    }

Here's the new look (player can still change by drag-and-drop other texture atlas png):

I made the changes to Pixeludi, but then, found referenced from https://github.com/traverseda/pycraft/issues/60 another pack: "Piehole", CC-BY: http://piehole.alexvoelk.de - it looks good, too, nice and clean. Saving here in case want to use it later (need to convert textures cutting & pasting 16x16 segments by hand - no real "texture pack" parsing support: https://github.com/satoshinm/NetCraft/issues/113), but this is piehole-pattern/terrain.png:

and piehold-plain/terrain.png:

and pixeludi from truecraft terrain.png https://github.com/SirCmpwn/TrueCraft/blob/7f0e3338d231b8a8f7fba35742b618a4fb3575bf/TrueCraft.Client/Content/terrain.png:


Multi-GPU training with mmcls

I've tried to train an image classification model with the tools/dist_train.sh script.
Although there are 4 gpus installed on my machine, only one of them is used during training.
I tried to specify devices with options like " --gpu-ids 0 1 2 3". It doesn't work.
It seems that the world_size value returned by the get_dist_info function determines number of gpus visible by the program.
Is there some example settings which I can refer to ?
Thanks !

Try to use this tool to train.
have you added -m torch.distributed.launch --nproc_per_node=$GPUS in your command.

Yes, I pasted my command as below
python -m torch.distributed.launch --nproc_per_node=1
Perhaps proc_per_node needs to be set to 2 in order to run on 2 gpus

It is recommended to use
CUDA_VISIBLE_DEVICES=0,1 ./tools/dis_train.sh $CFG_PATH

Thanks !
This is very helpful !


feat: add trustd subcommand to export openapi spec

Usage:
trustd openapi export --file test.json

defaults to json format
exports as yaml if file ends with ‘yaml’ or ‘yml’

we might also save openapi.json to root dir and add a check for equiv in github CI action to ensure this openapi.json is always up to date.


chore: fix fstrings

Description
Replace unnecessary f-strings with strings.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Refactor/improvement

How Has This Been Tested?
ui test, unit test
Checklist

[x] My commit message is conventional
[x] I have run pre-commit on all files before creating the PR
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

:tada: This PR is included in version 1.11.0-beta.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Build API Documentation on GitHub Pages

The readme works fairly well for now, but would benefit from being fleshed out like Limebean

Completed today https://nick-lucas.github.io/EntryPoint/


when adding team via API permissions can't be included

In GitLab by @jasonpagetas on Jul 27, 2024, 01:57
Swagger documentation does not provide instruction for adding permissions to a team.
Current examples in /api/organization/{organization_id}/team only show the following:

{
  "team_name": "string"
}

This needs to be updated to include permissions
Additionally it appears that a team id is required (mandatory) to create a team.

In GitLab by @jasonpagetas on Jul 28, 2024, 07:42
marked this issue as related to #74

url to add permissions to an existing team is /api/organization/<int:organization_id>/team/<int:group_ptr_id>/permissions and accepts a list of string.

@jasonpagetas

Additionally it appears that a team id is required (mandatory) to create a team.

what gives that impression?

added 2m of time spent

added 5m of time spent

mentioned in commit 0e426f5a641c24f7e8a7e7e97d751df3ceb169ec

added 2m of time spent

mentioned in commit a45971d2159f5ba27aa103f63f46d25bf97c83d0

removing from milestone v1.0.0 as the docs have been cleaned up for clarity. this issue will be scheduled to a later milestone.

mentioned in commit bb9870806a110dec6f23b42952e75f3510e28af0

mentioned in commit 3a9e4b29b3586531969c7e2677e72fde5f261983

mentioned in commit 9b673f4a0778b774be64ca361e3ebfd8fce628e8

@jasonpagetas, this issue was migrated from gitlab to github. During this migration you were not added as the user whom opened this issue. This message will ensure that you receive further updates on this issue as it progresses.
Further information can be found within #213

closing issue as the crux of the issue is by design and the docs were updated.


Automated cherry pick of #4035: fix: 避免普通用户添加规则到系统安全组

Cherry pick of #4035 on release/2.13.
#4035: fix: 避免普通用户添加规则到系统安全组

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.13@1394187). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.13   #4036   +/-   ##
==============================================
  Coverage                ?   6.92%
==============================================
  Files                   ?     506
  Lines                   ?   85356
  Branches                ?       0
==============================================
  Hits                    ?    5911
  Misses                  ?   78944
  Partials                ?     501

Flag
Coverage Δ

#aFlag
6.92% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/secgrouprules.go
4.34% <0%> (ø)

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 1394187...5579eb7. Read the comment docs.

/lgtm

/approve

/approve


Simplify Arithmetic Priors

Currently, all arithmetic rules using equality grounds two instances: <= and >=.
However, logical priors (logical rules with one atom) only ground one instance.
We should detect when an arithmetic rule only has one atom and make sure we only ground one instance.
(Not that this only applies for non-summation rules.)

I think we need to abstract this one more level.
It is not so much a problem that two ground rule are generated for something like:
1.0: Foo(A, B) = 0 ^2

But rather that we get two rules like:
1.0: Foo('Alice', 'Bob') <= 0.0 ^2
1.0: Foo('Alice', 'Bob') >= 0.0 ^2

Notice that the second rule is trivial, but only when the constant is 0.0.
(The first rule is trivial if the constant is 1.0.)

Essentially, we need something the looks for trivial ground rules right before they are added to the GRS.

Fixed in 42473c4ab43c98b71ee63cfbfb01c61f70e928f7.


chore: Add .go-version file for goenv

Signed-off-by: Ross<EMAIL_ADDRESS>
Summary

How did you test this change?

Issue

I don't see any issues including this file if it helps for people using goenv. Would be nice to have a link to a confluence doc for using goenv with the go-sdk.


Add Rate Limiting Examples

The article has Rate Limiting in the title, but doesn't actually offer any concrete examples of using rate limiting when calling Cosmos DB. .Net 7 introduced a rate limiting classes, so that would be a great starting point in creating code examples of using those for Cosmos DB.

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

ID: cc58d479-bc5b-d33e-1826-7847367d659f
Version Independent ID: 16e2b10e-c518-4dba-68c9-1895306e47f5
Content: Optimize your Azure Cosmos DB application using rate limiting
Content Source: articles/cosmos-db/rate-limiting-requests.md
Service: cosmos-db
GitHub Login: @plasne
Microsoft Alias: pelasne

@jonlave Thank you for reaching out. We will review this and make update as appropriate.

@jonlave
I have submitted a PR to add these links:

https://github.com/mspnp/go-batcher: A GoLang implementation of rate-limiting for Cosmos DB.
https://learn.microsoft.com/en-us/azure/architecture/patterns/rate-limiting-pattern: The original Azure Architecture Center article that discusses rate limiting as a pattern.

I am not aware of any .NET samples. The pattern is for distributed rate-limiting across multiple replicas, so I don't believe the out-of-the-box .NET 7 rate limiters will apply though there are primitives to create your own rate limiters and that could be implemented in keeping with this pattern.
You can reach me on Teams too (Peter Lasne), if you want to discuss further.

#please-close


How to run Dashboard, API and Bot at the same time?

How to run Dashboard, API and Bot at the same time?
npm run majo:bot <- Bot Run
npm run majo:dashboard <- Dashboard Run
npm run majo:api <- API Run
while true; do npm run majo:web; sleep 2; done <- Dashboard and API Run
But I don't know how to run all three in the same command (If I just add --bot it crashes)

Please run npm run start


ci: bump fuel-core to v0.15.0

fuelup passed the same e2e tests within the CI in advance and has published the latest channel, so this should be a safe change to make.

working on a fix right now.

@bingcicle as part of the fix, I had to bump fuel-core to 0.15 in https://github.com/FuelLabs/sway/pull/3581 because of some conflicting types due to the version of fuels-rs.

I shall close this then! @mohammadfawaz


fix(global header animation): story bugs

Overview

testing fixing global header story and/or finding alternative to global header story?

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.14-alpha.85eb50.0@codecademy/gamut-labs@11.2.2-alpha.85eb50.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6049b86cfb3dbf3ad06a6ba0--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.14-alpha.b846f3.0@codecademy/gamut-labs@11.2.2-alpha.b846f3.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.2.14-alpha.3e7cd1.0@codecademy/gamut-labs@11.2.2-alpha.3e7cd1.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://604a31c9887a3308adb538a1--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.14-alpha.d3a614.0@codecademy/gamut-labs@11.2.2-alpha.d3a614.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://604a519e12f7b700ce345c9b--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.14-alpha.b0f749.0@codecademy/gamut-labs@11.2.2-alpha.b0f749.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://604a862d3520193b72228420--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.14-alpha.b28136.0@codecademy/gamut-labs@11.2.2-alpha.b28136.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://604a95fe12f7b7432c346129--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.2.14-alpha.525e63.0@codecademy/gamut-labs@11.2.2-alpha.525e63.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://604a97fc6bb741427a75a2a9--gamut-preview.netlify.appDeploy Logs


Medical Insurance Website

Issue Type:

[ ] Bug Report
[x] Feature Request
[ ] Documentaion

Describe the bug

A Website calculates the medical Insurance bill

Possible solution

Describe the solution you thought of.

Screenshots

#Assign me.


Wait for smooth scrolling animation to add new message

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description
This fixes a bug in smooth scrolling where scrolling at the same time as a new message was added would cause a visual glitch that looks like the text jumps up and back instantly. To test it you can go in a fast chat and once the message queue fill up, and it starts scrolling to keep your place in the chat, you can scroll up and down to see it.
I'm not sure if this is a good fix because I couldn't fully understand the smooth scrolling code, and this seems like a workaround. So if there is a better way to fix this I'll just close this.

Can we fix the merge conflicts so this can be reviewed and hopefully merged soon?

This seems like a fair fix for me.
Basically 'pausing' all message additions during the 'smooth scrolling' effect.

@Pizzabelly could you explain how the other event loop works and if there is race conditions that can happen?


GroupPats: Grouping multiple processes together?

Hi friends, I have a question that I'm guessing has a simple answer (but possibly "sorry that isn't supported").
I have a process ("Parent.exe") I want to analyze that launches many instances of a separate process ("Child.exe"). What I would like to accomplish is a merged view of all of the Child.exe process samples collected, but after several attempts (and several re-readings of the GroupPats documentation) I'm unable to find an incantation that will merge multiple processes together.
If I use the example group pattern Thread -> AllThreads to merge the threads together, it will do as you expect - merging the separate threads under one process together, such that each process has an AllThreads frame.
But when I try what I expect should be the equivalent for processes - Process64 -> AllProcesses - I get a completely empty view (only ROOT remains). I've also tried Process -> AllProcesses (though my processes are all 64-bit) and that yields no change from the default view (it's as if I have no GroupPats specified).
If I switch to Process64 => AllProcesses it (understandably) turns each individual Process64 frame into an AllProcesses frame but doesn't actually merge the processes. Again, this makes sense because each one is a separate entry point from ROOT. But then I'm at a loss why the Process64 -> AllProcesses version doesn't work to merge the frames together?
Is there something obvious I'm missing here? I figured that grouping processes together would be a common use-case, but I haven't been able to figure out a solution in the separate times I've tried over the last few years.
Thanks,
Daniel

@danieljennings, assuming that you know which processes you want to look at, you should be able to do this fairly easily by opening the view of interest, and then selecting all of the processes that you want to see.  For example, if you open CPU Stacks, you'll get something that looks like this:

You can select multiple processes using CTRL or SHIFT and clicking:

If you do this, and then click OK, you'll see that the IncPats field in the stack viewer is limiting the set of samples to just the selected processes.  For this example:
Process% RuntimeInformationStart (22084)|Process% PerfView (33632)|Process% ServiceHub.RoslynCodeAnalysisService (10480)

The most important thing here is that you want to setup the IncPats, also known as Include Paths to include just the samples that you want.  GroupPats or Group Patterns allows you to group frames, such that they are easier to read, but doesn't actually do any filtering.  IncPats should give you what you want.  You can also hand craft the IncPats to be something like Process% Child and that would include any samples that belong to Child.exe.  But, you don't have to hand craft the pattern if you don't want to - multi-selection of processes should do it.  Hope that helps.

Thank you for the response, @brianrob .
I think you've misunderstood what I was asking for - I'm able to see multiple processes already (thanks to an IncPats filter like you've described) but I want to combine all of their frames together (and then combine all of their threads using the normal Thread -> AllThreads GroupPat).
So for example I want to see one root frame in the flame graph view that is shared by all of the processes selected by the IncPats, and then that one frame would have an AllThreads frame, and then the rest would be the normal flame graph (but representing all of the call stacks across all processes unified).
I would have to synthesize a test program to be able to share screenshots of what I mean, but figuring this out would be valuable enough to me that I'd be happy to take that time if it helps clarify what I mean here, so just let me know.
Thanks again for the time,
Daniel

Ok, I think I understand - Is your goal to just look at the processes that you want to look at, but to not have separate Process or Thread frames.  Assuming I have that right, what's happening is that the include path is getting in your way.  What you want is:

Set GroupPats to Process->AllProcesses;Thread -> AllThreads
Set IncPats to the set of processes that you want to look at.

Unfortunately, when you do this, you get no samples, and the reason is because the include patterns don't match anything because the group patterns have renamed all the process frames.  You can fix this with the following steps:

Open the stack viewer that you want (e.g. CPU) and select the processes that you want, so that IncPats gets set properly.

Go to the CallTree, right click on the inclusive value of ROOT frame, and select Drill Into.  This will open a new stack window with the same stacks, but you'll notice that IncPats is blank.  That's because the underlying stack source has been filtered.

Set GroupPats to Process->AllProcesses;Thread -> AllThreads.

Now, you should still be able to see the data, but you'll have a single AllProcesses node, followed by a single AllThreads node.
Let me know if I misunderstood.

YES! Thank you!!
That totally works, I appreciate your help here a ton. I hadn't realized that IncPats would run after GroupPats but in hindsight it makes complete sense and also explains why I'd end up with an empty result set (which felt weird).
That also led me to discover a related approach that I'll share in case someone else comes across this:
I set GroupPats to Process64 {%} (->Process $1;Thread -> AllThreads and then I can set IncPats to Process Child and it'll do the merging, group by process name, and then IncPats will filter it to only the (merged) processes named Child that I care about.
I super appreciate you helping me out here, PerfView has been my go-to CPU profiling tool for the last few years and this was the one piece I couldn't figure out. I hope you have a great weekend! :)


tx.inverseOperations

Implement new tx.inverseOperations, allowing package users to easily globally override some or all of the default update inverse commands

Reference issue #14


Documentation is rather confusing when it comes to installation

The documentation structure has a menu-item: Tutorials, which has a sub item: "Installing django CMS from scratch". However, this tutorial then goes on to instruct someone to use the django-CMS installer to install. I don't think many of us would consider this "from scratch" anymore than making a cake from a box-mix, rather than from raw ingredients.
Also, I don't think many people would consider installation instructions to be a "Tutorial" so much, as rather "Installation instructions". I realise this may be mincing words a bit, but I think a tutorial here would be more like "how to make a plugin", or other things once you have installed the product.
Finally, in a complete different section of the menu, is another menu: "How-to Guides", which includes the item: "Integrate django CMS into an existing project", which is another set of installation instructions (more akin to installing "from scratch", really).
This is confusing (we see comments on this in IRC, G+ on this regularly).
I would like to propose that installation be separate from "Tutorials" and "How-To Guides". A sub-menu could then provide different installation paths including:

djangocms-installer;
manual installation;
integration with existing Django projects.

Pull request https://github.com/divio/django-cms/pull/3947

Thanks @mkoistinen for identifying the issue.

Closed by #3947


Made the application rebar-compliant

Hi Tim,
As a lover of Rebar I decided to make this behave nicely as a rebar app. Perhaps you'd be interested in doing it too.
Comments from the commit:

Moved the oauth.app file to src.
Adjusted the Makefile to use rebar.
Added a .gitignore for some common files and to exclude the
ebin folder.

I haven't added rebar itself to the repo, this is optional. It'll try to find one in your path first before searching for one in the local folder.
If you're not keen on rebarisation, feel free to ignore. Cheers!
OJ

You are not the first (see issue #8), and judging by the network graph you won't be the last. It's nice that you love rebar. I think rebar is a fine piece of software but I do not wish to force it upon others, so I have no interest in "rebarizing" my code, as cool as that might sound.
I do not know what it means to be "rebar compliant". Last time I checked, I could build erlang-oauth just fine with an "out of the box" rebar (i.e. by typing rebar compile in the erlang-oauth directory). Given that the code can already be compiled using make or by using erlang directly (erl -make etc), what problem/issue are you trying to solve?

Fair call if you don't want other's to have it forced on them. You could, should you choose, keep it in another branch (a topic branch) and maintain it off the main one (just like I plan to do). Then at least you can have it there for others to use if they want. Then you're not forcing your users to have it, nor are you saying that you don't "support" it.
The rebar compliant bit was about making it function nicely as a rebar dependency. Sure it compiles out of the box fine, so long as that's all your compiling. Adding it as a dependency to another rebar app is a bit different.
Either way, it doesn't worry me at all if you don't take the request :) It was just a "take it if you want it", not a "you should really take this".
Cheers.

Can you be more specific about why it doesn't "function nicely as a rebar dependency"? What is the problem exactly? Is it the lack of a rebar.config file or is it the location of the oauth.app file that is the issue?


Cloudinary integration failing

Describe the bug
After cloning the repository, everything works fine, except of uploading an image (both with own Cloudinary details and those provided as an example).
System
Browser version: Chrome, 85.0.4183.121
Node.js version: v14.4.0
NPM or Yarn version: 6.14.5
Operating system: Windows
Steps to reproduce

Setup using
npx create-social-network my-network
Updated only the database details
yarn start
Either change the Cloudinary details or leave them as they are

Expected behavior
Image should be uploaded.
Actual behavior
RangeError: Maximum call stack size exceeded error is thrown to the console.

Additional context
I would like to highlight it happens with both: leaving and changing Cloudinary details in .env file. Am I doing something wrong, perhaps?

Found a solution, not an ideal one, but I managed to make it work with both, your Cloudinary details and my own.
to all three package.json files add:
"resolutions": { 		"fs-capacitor": "3.0.0" 	}
And inside api and frontend directories install fs-capacitor packages.

Hi @adreszer, thanks for reporting the issue. I'll try to fix it soon.


tests, network, sriov: Use properties with multi iface spec

What this PR does / why we need it:
When multiple SR-IOV interfaces are defined with properties (e.g. guest
PCI address), these should propagate to the guest.
The existing e2e test that covers multi SR-IOV interfaces is changed to
also check such properties, specifically the guest PCI address.
This change got triggered by a bug [1] which was not detected by the
existing tests.
[1] https://bugzilla.redhat.com/2070050
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Depends on #7496
Release note:
NONE

/test pull-kubevirt-e2e-kind-1.22-sriov

Change: Replace the guest PCI address to 0000:06:00.0 and 0000:07:00.0.
It seems not to work at all with the older values, which may require its own investigation.

/test pull-kubevirt-e2e-kind-1.22-sriov

/test pull-kubevirt-e2e-kind-1.22-sriov

@EdDev  are the first three commits introduce any logic change, or is it just moving code from one file to another?

@EdDev are the first three commits introduce any logic change, or is it just moving code from one file to another?

It is mentioned in the PR that introduced them. This one depends on the other, see the description.

/approve
Thanks!

/hold
Lets wait for https://github.com/kubevirt/kubevirt/pull/7496 to be merged first.

/unhold

/retest

/retest

/retest


Mock GH API to allow testing editorial workflow

- Do you want to request a feature or report a bug?
New feature.
- What is the current behavior?
There is currently no mocking of the GitHub API. The editorial workflow thus has no automated testing.
- What is the expected behavior?
We should be able to make mock requests to the GitHub API to test the editorial workflow.

This is resolved using a recording proxy


core: Fix incorrect regexp

Fixes #1200

NOT fixed in 1.13.1 (from nugget)
If using (for example) an id like "avd.Enabled", it fails to match and adds forever

looks fixed for me in 1.14


shell script action with administrator privileges

I have a few Control Plane actions that require sudo shell commands.  I would like the option to run these with administer privileges (either with or w/out password prompt).  The current workaround is to put it in an applescript "do shell script" wrapper.

I have the same requirement, and it took me a while to understand how to use applescript workaround (as suggested by @swiftster), however the action still fails with an error, and does not display a password prompt when executed from ControlPlane. The following should have been a working example, but it isn;'t.
Let's say, you would like to create a script to change DNS servers when arriving to work/home, to the network's gateway as first, followed by google as fallback.
First create an applescript that accepts command line arguments, and sets the DNS popping a password prompt when required.
# ~/bin/change_dns.scpt

on run first_dns
	set cmd to "networksetup -setdnsservers Wi-Fi " & first_dns & " <IP_ADDRESS> <IP_ADDRESS>"
	do shell script cmd with administrator privileges
end run

Now, since ControlPanel requires the script to be executable with /bin/sh, add a shell script to execute the above script with given arguments.
# ~/bin/change-dns-via-applescript
osascript ~/bin/change_dns.scpt $@

And now, configure the script parameter of your ControlPanel action to:
~/bin/change-dns-via-applescript <IP_ADDRESS>

However the script still fails with an error

22:37:06.826 -[CPController activateContext:] 	Triggering arrival actions, if any, for 'Home downstairs'
22:37:06.829 __56-[ShellScriptAction launchTaskWithLaunchPath:arguments:]_block_invoke 	Failed to execute '/Users/big/bin/change-dns-via-applescript <IP_ADDRESS>' (script terminated with a > non-zero status '127')

It would be great to be able to execute a script with or without a password

"Value 127 is returned by your shell /bin/bash when any given command within your bash script or on bash command line is not found in any of the paths defined by PATH system environment variable.”
Make sure you have explicitly set your path or you include the full path to all commands being run. This should help.

@dustinrue you're right! Setting /Users/big/bin/change-dns-via-applescript as the parameter alone works.
I assumed additional arguments are passed to script, but apparently the whole value is evaluated as the path to script. Can we maybe have ShellScript parameter accept arguments? Alternatively, triggering an action could (optionally) pass arguments with information about the activated context?

I managed to get @73's version working (on Sierra with CP 1.6.6). I used a wrapper script that I call from Controlplane. That file startNfsd.sh contains the sudo command: /usr/bin/sudo /sbin/nfsd start
For some reason ControlPlane fails to call sudo directly.
As far as sudoers go, it is enough to only keep nfsd in the sudoers file. The wrapper script can still run unprivileged.


update \s+tag: (.*) to 0.0.7

UpdateBot pushed regex: \s+tag: (.*) to: 0.0.7

UpdateBot commands:
updatebot push-version --regex \s+tag: (.*) --value 0.0.7 --exclude --previous-line \s+ repository: activiti/activiti-modeling-app charts/activiti-cloud-modeling/values.yaml

/test this
/approve

 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.jenkins-x-bot 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.


Added case insensitive search for project name

Simple change to convert project name to lowercase both in search param as well as column value from DB to do a case insensitive compare. This has been tested in the works dev environment and is working well. Health can also benefit from this change.

Created a tech debt ticket to review the solution later and come up with a more efficient search.


Map not supported

It would be nice that the Map object were supported
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map
https://stackoverflow.com/questions/29085197/how-do-you-json-stringify-an-es6-map

Suggested output format:
new Map([[key, value]...)

This way the output can be used as code to rehydrate if desired.


BZ-1192459, BZ-1206627: general improvements and cleanup on sync and shutdown

http://github.com/droolsjbpm/kie-uberfire-extensions/commit/848c08101


imp!: add encoding type as argument to ICA encoding/decoding functions

Description
This pulls API breaking changes from #3796 to merge in a separate pr. This will reduce review diffs (no logical changes made here) and reduce conflicts when backporting #3796 as the API breaking changes cannot be backported.
In addition, this change will be relevant for #3375
ref #3246
Commit Message / Changelog Entry
imp!: Add encoding type as argument to `SerializeCosmosTx` and `DeserializeCosmosTx`

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Codecov Report

Merging #3967 (aecb47e) into main (3559e58) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3967   +/-   ##
=======================================
  Coverage   78.84%   78.84%
=======================================
  Files         187      187
  Lines       12950    12950
=======================================
  Hits        10211    10211
  Misses       2311     2311
  Partials      428      428

Impacted Files
Coverage Δ

.../apps/27-interchain-accounts/host/keeper/keeper.go
87.39% <ø> (ø)

.../apps/27-interchain-accounts/host/client/cli/tx.go
29.70% <100.00%> (ø)

...s/apps/27-interchain-accounts/host/keeper/relay.go
71.08% <100.00%> (ø)

modules/apps/27-interchain-accounts/types/codec.go
79.54% <100.00%> (ø)


Update current docs to use the new UI

New conda-store UI: https://github.com/Quansight/conda-store-ui

Is this still needed @pavithraes

Yes, this is still needed

Very much needed!

The docs in this area need a general rewrite. Also note that many of the features in the old UI are not available in conda-store ui. Not sure how to address this issue in the docs.

This work is now being tracked in #570 - closing as duplicate. :)


Windows Management Framework 5.0 RTM was released

I've updated download links

switched to https
changed link for Windows Management Framework 5.0 RTM
I've removed notes about preview for WMF 5.0

I'm going to hold off on this as they just pulled WMF 5 RTM. http://blogs.msdn.com/b/powershell/archive/2015/12/23/windows-management-framework-wmf-5-0-currently-removed-from-download-center.aspx


SUBMARINE-494 Design for Notebook

What is this PR for?
SUBMARINE-494 Design for Notebook support
What type of PR is it?
Documentation
Todos

[ ] - Task

What is the Jira issue?
SUBMARINE-494
How should this be tested?

First time? Setup Travis CI as described on https://submarine.apache.org/contribution/contributions.html#continuous-integration
Strongly recommended: add automated unit tests for any new or changed behavior
Outline any manual steps to test the PR here.

Screenshots (if appropriate)
Questions:

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

Attached initial PR, @liuxunorg can you help to review it? Thanks.

Thanks to @wangdatan for contributing the Submarine notebook design document.
LGTM. Let me know more about notebook Implementation in Submarine.


[5.1][CSSolver] Move keypath subscript index Hashable conformance verification to solver

Move checking for index Hashable conformance from CSApply (post-factum)
to the solver and use recorded conformance records to complete subscript
components.
Resolves: rdar://problem/49413561

@swift-ci please test


Accessing objects across multiple threads (Documentation) [Swift 2.0]

How is realm supposed to handle saving and returning objects across multiple threads?
say I have an object that I want to save it without blocking the main thread, so I do something like this:
class Helper {
    let queue = dispatch_queue_create("MyQueue", DISPATCH_QUEUE_CONCURRENT)

    func testSave() {
        let object = Object()
        object.name = "test"

        save(object, { () -> Void in
        }
    }
    func save(object: Object, completion:() -> Void) {

        dispatch_async(self.queue) { () -> Void in
            let realm = try! Realm()

            realm.write {
                realm.add(object, update: true)
            }
            dispatch_async(dispatch_get_main_queue(), { () -> Void in
                completion()
            })
        }
    }
}

My question is this: If I wanted to return this object to another method, how do I return the Results that is used by Realm? Should I be doing a fetch on the main thread after the write is complete?

Yes, if you want to use the object from the main thread you'll need to do a fetch on the main thread after the write is complete. You can add a primary key to your class to make it easy to retrieve the object from the main thread.

We've added more documentation about threading considerations in Realm: https://realm.io/docs/objc/latest/#threading and https://realm.io/docs/swift/latest/#threading.

@bdash following up to this question. If instead of creating the object, I have an instance of the object that is fetched from realm, will I need to refetch that in the background, then update that, and the refetch on main thread?
Updated example:
class Helper {
        let queue = dispatch_queue_create("MyQueue", DISPATCH_QUEUE_CONCURRENT)

        func testSave() {
            let object = Object()
            object.name = "test"

            save(object, { result in
            }
        }

        func testSaveFetchedObject() {
            let realm = try! Realm()
            let object = self.findObjectInMainRealm("1234")
            object.name = "new name"

            save(object, { result in

            })
        }

        func findObjectInMainRealm(id: String) -> Result {
            ////
            dispatch_async(dispatch_get_main_queue(), {
                let realm = try! Realm()
                realm.refresh()

                guard let object = realm.objectForPrimaryKey(SEOrganizer.self, key: originalObject.slug) else {
                    let error = NSError(domain: ShuddleEngineErrorDomain, code: ShuddleEngineErrors.SaveFailed.rawValue, userInfo: nil)
                    return result(.Failure(error))
                }
                return result(.Success(object))
            })
        }

        func save(originalObject: Object, completion:(Result) -> Void) {
            dispatch_async(self.queue) { () -> Void in
                let backgroundRealm = try! Realm()
                backgroundRealm.refresh()

                guard let backgroundObject = backgroundRealm.objectForPrimaryKey(Object.self, key: originalObject.id) else {
                    try! backgroundRealm.write({
                        backgroundRealm.add(originalObject)
                    })

                    completion(findObjectInMainRealm(originalObject.id))

                    return

                }

                try! backgroundRealm.write( {
                    backgroundRealm.add(backgroundObject, update: true)
                })

                completion(findObjectInMainRealm(originalObject.id))

            }
        }
    }

To do this right, you'll need proper thread handover of objects, which is tracked in #3136

@jpsim that looks like it's currently backlogged. If I was shipping this today, what would be the recommended way of doing this?

If I was shipping this today, what would be the recommended way of doing this?

We outline some strategies in the Threading section of our docs.

I'm experiencing some really slow reads / updating UI when scrolling over large data set. Should the reads be done on the background thread? or main thread?

@donaldking reads should be done on the thread that need that data, and since you appear to be using Realm to populate your UI, that implies reads should be done on the main thread.


Question: Google Cloud Platform hosting - exec()

Hi there,
when releasing sentry on GCP I caught an error in the PHP error log file stating that exec is not supported by the platform by default and it seems that there's no way of working around it.
Is there a way to use the sentry package and still get a code version added to the sentry log based on the sha1 of a the version of the code?

I think your best bet is to somehow add the SHA1 hash of the commit in an environment variable. And read it from there, I don't know how deployments work so I am not sure if I can help you with a specific code example. But something like this in config/sentry.php should work:
<?php
return array(
    'dsn' => env('SENTRY_DSN'),

    // capture release as git sha
    'release' => env('SENTRY_SHA'),

    // Capture bindings on SQL queries
    'breadcrumbs.sql_bindings' => true,

    // Capture default user context
    'user_context' => true,
);

Let me know.

hey @stayallive , that seems like a way forward, I've set the release now to env('app.version'), but I think if we add a post-script everything something gets pushed to master which refreshes the SHA1 in the environment file, that would work. It indeed depends on how we deploy, but I think that workflow works in multiple approaches, including ours. Thanks!

That would certainly work. Unfortunatly there is no 1-way for every case :)


Bug when socket.io connects before first HTTP connection

Hi, I discovered  a bug (or maybe just a weird unexpected behavior) with this module.
Description:
When socket.io connects before express.js gets one HTTP request, socket.io gets a new session ID every time it reconnects. As soon as one HTTP request reaches the server, the session ID is static. This leads to unexpected behavior when a socket reconnects after a server restart or network problem.
Code to reproduce:
// bug.js
var app = require('express')(),
    server  = require("http").createServer(app),
    io = require("socket.io")(server),
    session = require("express-session")({
        secret: "my-secret",
        resave: true,
        saveUninitialized: true
    }),
    sharedsession = require("express-socket.io-session");

// Attach session
app.use(session);

app.get('/', function (req, res) {
    console.log('express id: ' + req.session.id);
    res.sendFile(__dirname + '/bug.html');
});

// Share session with io sockets
io.use(sharedsession(session));

io.on("connection", function(socket) {
    console.log('socket.io id: ' + socket.handshake.session.id);
});

server.listen(3000);

<!--bug.html-->
<!DOCTYPE html>
<html>
<head>
    <title></title>
    <script src="/socket.io/socket.io.js"></script>
    <script>
        var socket = io('http://localhost:3000', {
            reconnection: false
        });
        function httpGet() {
            var xmlHttp = new XMLHttpRequest();
            xmlHttp.open( "GET", 'http://localhost:3000', false );
            xmlHttp.send( null );
            return xmlHttp.responseText;
        }
    </script>
</head>
<body>
    <button onclick="socket.disconnect();">disconnect</button>
    <button onclick="socket.connect();">connect</button>
    <button onclick="httpGet();">http</button>
</body>
</html>

Steps to reproduce:

node bug
Load http://localhost:3000 in browser
^C (close node)
node bug
Click on connect
Click on HTTP
Click on disconnect
Click on connect

Example Log Output using the above steps:
>node bug
express id: eesau4KnEaNBu9dxc-sd-AsWxOC2pkmx
socket.io id: eesau4KnEaNBu9dxc-sd-AsWxOC2pkmx
^C
>node bug
socket.io id: unGp5Xct-q0sYJQanm5G4R40kBOad6do
express id: N8eXurfyMeqMsIJDf5azw7ZYCwnJkzuy
socket.io id: N8eXurfyMeqMsIJDf5azw7ZYCwnJkzuy

Thanks! I've noticed that when unit testing using socket.io-client. Somehow I managed to ignore that issue. Do you have an approximate idea of what would be the expected behaviour? Or, I mean, the solution?

Sorry, I closed the issue by error. I reopened it

No, I have no idea how to fix this, but I'd expect socket.io to initialize the session if express didn't already do it.

OK, I'll fix the code to behave as expected. Thanks!

Any news on this?

+1

Sorry. I'll get back to this
On Sat, Nov 7, 2015, 4:25 PM José Ribeiro<EMAIL_ADDRESS>wrote:

+1
—
Reply to this email directly or view it on GitHub
https://github.com/oskosk/express-socket.io-session/issues/6#issuecomment-154741264
.

Weird things happened.
when using var socket = io('http://localhost:3000') or var socket =io('/'),sock.io session id is differing from express and will change after refresh.
however when using var socket=io()  suddenly everything is right in place

@wywzxxz You seem to be right, when providing a connection URL the session ID this module gives is different from the actual session ID. Without a connection string passed it does seem to work as expected however.

Any news about the original bug? Any work-around?

Hi Guys , same issue every time i refresh the page session Id will change and while accessing the session variable it shows "undefined" please help

It is because socket.io doesn't make a new http request therefore the session data isn't available as express doesn't send the session forward. If you don't refresh the http connection socket.io will still connect behind the scene skipping express.
Perhaps if session id doesn't match in the above example, do http() then connect() to refresh the session through express?

I'm having the same problem with the Socket.io sessionID being different from the Express sessionID. @adam4813 would you be so kind to provide a snippet of your suggestion?

When socket.io connects before express.js gets one HTTP request, socket.io gets a new session ID every time it reconnects.

Does this also happen if saveUninitialized: false?

I was having the same issue.  I had to send a dummy ajax request to the express server to trigger the session creation in the browser.  Then in the ajax callback I call socket = io(server_endpoint);

Still an issue especially If you are trying to handle the session exceptionally over socket connections.

still an issue

But orkaround from @islei works!

doesnt work for me sorry.

Hi everybody.
When I use the path
var socketApp = io.of('/socket/app').on('connection', (socket) => {  //Not working
But
var socketApp = **io.of('').**on('connection', (socket) => { // Working
I don't know the reason,


🚧 Changes in packages

🐛 After starting working on implementing the pdf support to download from draft js, I saw that draft js will be archived at the end of this year.
✅ Meta is creating another open-source rich text editor called Lexical, but now is under development.
So I think we have to make changes and not continue to implement something that is going to end soon.
👉 Reference here
We have some plugins that we need to check, here I will list first the features we need that they should support, then I will list some libraries.

❌❌❌
Dropping drafts means that we will need to change some code we wrote, the draft js component, and the toolbar.
The code for OCR and Image cropping will work

✨ Features :

They should support  block styles like

Heading
Lists
Block quote
Code block
Images

Inline styles like

Bold
Italic
Underline
Code
Links

Downloading the note we write into formats like pdf or markdown.
Support json data or data that can be stored into storage.

📝 Libraries

markdown-it  / remarkable / snarkdown 👉 just 1kb

97.2 kB minified 👉 markdown-it
✅ This is not a text editor, this is just a markdown, it has all the features except the pasted image from the clipboard, but needs workarounds.
❌⁉ If we use it as a text editor this needs more work to do because not all people are comfortable using markdowns. So we have to offer some features to press adding headings and stuff.

quilljs

119.6 kB minified
✅ This is a rich text editor, that has all the features we need. Has good support on the browser.
❌ More than 1.1k issues on Github.
❌ Version 2 is waiting to come for around 10 months.
❌ Should implement the export to pdf or markdown
❓ is it trustable or not ?!

slate

84.2 kB minified
✅ This is a framework for building an editor. It has all the features we need. Plus can be edited and added more features.
❌ Should implement the export to pdf or markdown
❌❌ It is not in production mode and can have breaking changes.

editor.js

97.3 kB minified
✅ A rich text editor, with all the required features
❌ Should implement the export to pdf or markdown

summernote

148.9 kB minified
✅ A rich text editor, with all the required features
❌ Uses jQuery
❌ exporting should be implemented

tiptap

210.2 kB minified
✅ headless UI
✅ support JSON format and HTML
❌ paste image from the clipboard needs to be implemented.
❌ exporting to pdf or markdown needs to be implemented

ckeditor5

...
✅ All the features implemented (paste image from clipboard)
❌ not MIT needs checking about license
❌ exporting to pdf/ docs is a premium feature

remirror

189.3 kb minified
✅ Same as tip tap, but it has the paste from clipboard implemented
❌ exporting to pdf or markdown needs to be implemented

⁉ NOTE
Here I listed different text editors, including some basic pros and cons. Also included is just markdown, with markdown, everything should be implemented from scratch including saving state. The others most of them support JSON format, but there are some pros and cons to each other. If you have any idea or any other suggestion, then say it.
As seen above the main problem/issue with the editors is the support of images (paste from clipboard (can be implemented in most of them)), and exported to files.
I think this is important and critical to do first, then continue.
Here is an article that shows how they choose

@bkrmadtya

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

Chrome Web Store
GitHub release

Your semantic-release bot :package::rocket:


nock failing with system proxy set

When I try to use nock in tests like this nock('http://localhost').get('/success').reply(200, 'some data'); and the system which runs the tests has http_proxy set, the requests will be passed to the proxy whatever I try to define in system's proxy exceptions.
Since I did not find any documentation on this, I'd like to know if there's anything I can do to make nock / request not unsing the proxy in the tests.
Thanks
Frank

It may be out of scope of nock, what's the http request module you're using?

I'm using https://www.npmjs.com/package/request

Cool, see https://github.com/request/request#controlling-proxy-behaviour-using-environment-variables,
you can set the no_proxy env var for localhost http request.

Thanks a lot, that's basically it. It took a while for me to figure out that the ical module I'm using relies on a too old version of request (2.40.0) which does not support these variables (since 2.51.0).

Thank's a lot @xavierchow It actually fixed my issue. In windows the proxy is not added in the url with the request module, but on my build machine (linux) it add the proxy before the URL and makes it impossible to NOCK.
I tried to debug the request module, since it does a different behavior depending on the system, but I stopped searching after seeing that I could modify the no_proxy within code.


feat(iroh-dns-server)!: Make http rate limit configurable

Hello,
We are currently testing some new cloud features at Spacedrive, and our implementation relies heavily on iroh. As part of this, we are deploying our own iroh-dns-server. However, since all of our backend services operate behind a reverse proxy, we noticed that the iroh-dns-server was frequently hitting its rate limit because it wasn’t aware of the proxy setup. To address this, I decided to implement a configurable rate limit for the iroh-dns-server.
Description
This PR adds a new entry to the iroh-dns-server TOML file for configuring the HTTP rate limit. The new configuration allows for disabling the rate limit and also supports configuring it to use the SmartIPKeyExtract, making it compatible with reverse proxies.
Breaking Changes

iroh-dns-server's configuration structure now has a new field allowing to choose the rate limiting algorithms.

Notes & open questions
:)
Change checklist

[x] Self-review.
[x] Documentation updates following the style guide, if relevant.
[x] Tests if relevant.
[x] All breaking changes documented.

Hi, I am not sure why the CI is failing. Is this expected? Cargo deny is complaining even though I didn't add any new dependencies, so I am not sure what is going on there. Netsim test is failing making a request to a GitHub API for some reason. The check_semver I imagine is failing because my changes require me to bump a minor version, should I do that?

yes, this is not your fault! we fixed this recently, could you update your branch?

So, I had to add a rate_limit property to [https], because the program crashed with only rate_limit defined in [http], as it requires a port property because it assumes that if you have a [http] block it means you want to enable the http server. So I ended up just implementing an optional rate_limit in [https] which has preference over what is configured in [http] in case both are defined.

Sorry about the conflicts. We have unified the way our imports are done, should be easy to fix.

Hey, thanks for following up on this PR. After the latest changes, it seems clearer the rate limiting applies to the whole dns server and making it present in the http config AND https config might be easily misunderstood in the case where both are defined. You have mentioned a strategy but we should strive to make the meaning of options as obvious as possible.

Yeah, originally I had placed the rate_limit configuration at the top level, but it felt odd since it was the only option without a block. The name also started getting confusing, as it only applies to HTTP, and placing it as a global option might lead users to assume it also applies to the DNS part of the server. I agree that having rate_limit under both [http] and [https] isn’t ideal, though. Maybe a better alternative might be to create a new [rate_limit] block with an http = '<disabled|simple|smart>' option. This would make the configuration clearer to users, avoid repetition, and allow for future expandability if we ever need to support rate limiting for the DNS side as well.

@matheus23 suggested pkarr_put_rate_limit in the top level config. I find this a good option. It's verbose but extremely clear since this is the only place rate limiting applies.


side panel doesn't close in mobile mode in Blazor Server

side panel doesn't close in mobile mode in Blazor Server.
How can we fix

And is not minimized in desktop mode, hides completely

Which mobile device and browser type are you using?

bug video.zip
this is short video, you can try self  http://etest.line.kg
Mobile device is XIAOMI 6A, browser Edge and Chrome, both last version

Problem solved! I forgot change class of body in _Host.cshtml file.
I change

      ....

to

 ......


feature: persist data for CA and couchdb

This PR has following feature:

persists CA data always
persists couchdb data when minifab -s couchdb called

because current latest master persists data for peers and orderers but not for aboves...

CI test error was raised at channel creation phase. it is happenend after my code passed. please run CI test again to double check if the error was caused by accident or my fault.

@itaru2622 please address the comment and make sure that you have just one commit by using --amend, that way, the new push to your own repo will trigger the build again.

@litong01 thank you for your review. I applied your comment. please check it again.

hmm CI test failed again under exactly the same condition ( failed channel creation phase for minifab -i 1.4.1 but other tests passed).  I will send another trigger later if you need.

hmm CI test failed again at the same place by the  same reason ( channel creation phase at minifab up -i 1.4.1, by 'lookup orderer3.example.com: no such host')
I think it may be caused by CI test environment issue, since other versions tests(1.4.6..2.x) passed by CI test and my self test for version 1.4.1 passed as bellow:
#
# myself test passed for minifab -i 1.4.1
#
$ git log -1 --pretty=oneline
bc882f7b383fecaac10795405fd240b1a5e8a744 (HEAD -> persistdata_ca_couchdb, fork/persistdata_ca_couchdb) apply PR review comment.
$ docker build -t hyperledgerlabs/minifab:latest .
:
$ ./minifab up -i 1.4.1
Using spec file: /mnt/minifabric/spec.yaml
Minifab Execution Context:
    FABRIC_RELEASE=1.4.1
    CHANNEL_NAME=mychannel
    PEER_DATABASE_TYPE=golevel
    CHAINCODE_LANGUAGE=go
    CHAINCODE_NAME=simple
    CHAINCODE_VERSION=1.0
    CHAINCODE_INIT_REQUIRED=true
    CHAINCODE_PARAMETERS="init","a","200","b","300"
    CHAINCODE_PRIVATE=false
    CHAINCODE_POLICY=
    TRANSIENT_DATA=
    BLOCK_NUMBER=newest
    EXPOSE_ENDPOINTS=false
    CURRENT_ORG=org0.example.com
    HOST_ADDRESSES=<IP_ADDRESS>
    WORKING_DIRECTORY: /mnt/minifabric
.....
:
# Current Minifabric image ID and created at date time ********
  3184fbd1c506   2021-02-05 14:40:41 +0000 UTC
.....
# Docker node status ******************************************
  a80955269d_cli : Up 7 seconds
  ca1.org1.example.com : Up 8 seconds
  ca1.org0.example.com : Up 9 seconds
  orderer3.example.com : Up 21 seconds
  orderer2.example.com : Up 21 seconds
  orderer1.example.com : Up 22 seconds
  peer2.org1.example.com : Up 23 seconds
  peer1.org1.example.com : Up 24 seconds
  peer2.org0.example.com : Up 25 seconds
  peer1.org0.example.com : Up 25 seconds

# Fabric network peer and orderer node health status **********
  peer1.org0.example.com "OK"
  peer2.org0.example.com "OK"
  peer1.org1.example.com "OK"
  peer2.org1.example.com "OK"
  orderer1.example.com "OK"
  orderer2.example.com "OK"
  orderer3.example.com "OK"
  Network Status: 100%

# Running operation: ******************************************
  channel create
:
# Discover endorsers results **********************************
  Chaincode endorsers file: ./vars/discover/mychannel/simple_endorsers.json
# Discover orderers results ***********************************
  Channel orderers file: ./vars/discover/mychannel/ordererendpoints.json

# STATS *******************************************************
minifab: ok=510	failed=0

real	7m43.253s
user	3m7.384s
sys	0m35.922s

#
# CI test triggered by PR failed.
#
./minifab up -i 1.4.1
:
# Running operation: ******************************************
  channel create
# Run the channel creation script on cli container ************
  non-zero return code
  2021-02-06 00:26:16.107 UTC [common.tools.configtxgen] main -> INFO 001 Loading configuration
  2021-02-06 00:26:16.136 UTC [common.tools.configtxgen.localconfig] Load -> INFO 002 Loaded configuration: /vars/configtx.yaml
  2021-02-06 00:26:16.160 UTC [common.tools.configtxgen.localconfig] completeInitialization -> INFO 003 orderer type: etcdraft
  2021-02-06 00:26:16.161 UTC [common.tools.configtxgen.localconfig] completeInitialization -> INFO 004 Orderer.EtcdRaft.Options unset, setting to tick_interval:"500ms" election_tick:10 heartbeat_tick:1 max_inflight_blocks:5 snapshot_interval_size:20971520
  2021-02-06 00:26:16.161 UTC [common.tools.configtxgen.localconfig] LoadTopLevel -> INFO 005 Loaded configuration: /vars/configtx.yaml
  2021-02-06 00:26:16.161 UTC [common.tools.configtxgen] doOutputChannelCreateTx -> INFO 006 Generating new channel configtx
  2021-02-06 00:26:16.162 UTC [common.tools.configtxgen] doOutputChannelCreateTx -> INFO 007 Writing new channel tx
  Error: failed to create deliver client: orderer client failed to connect to orderer3.example.com:7050: failed to create new connection: connection error: desc = "transport: error while dialing: dial tcp: lookup orderer3.example.com: no such host"

Error: failed to create deliver client: orderer client failed to connect to orderer3.example.com:7050: failed to create new connection: connection error: desc = "transport: error while dialing: dial tcp: lookup orderer3.example.com: no such host"

This is really weird. I do not know what could cause the issue. Each time azure will allocate a new vm to test.

I know sometimes it fails when the nodes are not quite ready for requests, it will fail.

another shot also failed...orz

I've tested the PR locally using 1.4.1 release, and things are working as expected. really not sure why it is failing on Azure. If you please make your PR into one commit, I will try to merge it.

PR 156 is good news, for me. I will push again after PR 156 merged.

@litong01 I rebased my PR and now it has just one commit.

let's wait for this PR https://github.com/hyperledger-labs/minifabric/pull/156 to get merged, then we will trigger this again.


ocamlPackages.spelll: 0.3 → 0.4

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 221124 run on aarch64-linux 1

1 package built:

soupault

@ofborg build soupault

Result of nixpkgs-review pr 221124 run on aarch64-darwin 1

1 package built:

soupault


Wrong inline script hash when the file EOL is CRLF

Description
The hash is wrong when the index.html EOL is CRLF
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
slackhq/csp-html-webpack-plugin version: 5.1.0
node version: v16.15.1
OS version(s): Windows 10
Steps to reproduce:

Change the index.html EOL to CRLF (the default EOL in Windows)
Generate hash for inline script

Expected result:
The hash is generated correctly.
Actual result:
The hash is wrong for CRLF index.html

@chihiroxc I use a forked version of this plugin here and I use Windows and I have no issues. Can you try my forked version: https://github.com/melloware/csp-webpack-plugin


Fails to run react-native run-android (v0.68.0) without yarn

Description
when I create a test project and enable new architecture and Hermes it's giving me this type error below:-
NOTE:- I KNOW I DON'T HAVE YARN (i love using npm that's why I don't install yarn in my system maybe this error is fixed after yarn install but it does not mean we can't run our project through npm)
info Running jetifier to migrate libraries to AndroidX. You can disable it using "--no-jetifier" flag.
Jetifier found 936 file(s) to forward-jetify. Using 8 workers...
info JS server already running.
info Installing the app...

> Configure project :app
Unable to detect AGP versions for included builds. All projects in the build should use the same AGP version. Class name for the included build object: org.gradle.composite.internal.DefaultIncludedBuild$IncludedBuildImpl_Decorated.

> Task :ReactAndroid:generateCodegenSchemaFromJavaScript FAILED

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.

You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.

See https://docs.gradle.org/7.3.3/userguide/command_line_interface.html#sec:command_line_warnings
9 actionable tasks: 1 executed, 8 up-to-date

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':ReactAndroid:generateCodegenSchemaFromJavaScript'.
> A problem occurred starting process 'command 'yarn''

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 6s

error Failed to install the app. Make sure you have the Android development environment set up: https://reactnative.dev/docs/environment-setup.
Error: Command failed: ./gradlew app:installDebug -PreactNativeDevServerPort=8081

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':ReactAndroid:generateCodegenSchemaFromJavaScript'.
> A problem occurred starting process 'command 'yarn''

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 6s

    at makeError (/Users/pushpendersingh/Desktop/Projects/RN/testV68/node_modules/@react-native-community/cli-platform-android/node_modules/execa/index.js:174:9)
    at /Users/pushpendersingh/Desktop/Projects/RN/testV68/node_modules/@react-native-community/cli-platform-android/node_modules/execa/index.js:278:16
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async runOnAllDevices (/Users/pushpendersingh/Desktop/Projects/RN/testV68/node_modules/@react-native-community/cli-platform-android/build/commands/runAndroid/runOnAllDevices.js:109:5)
    at async Command.handleAction (/usr/local/lib/node_modules/@react-native-community/cli/build/index.js:192:9)
info Run CLI with --verbose flag for more details.

Version
0.68.0
Output of npx react-native info
info Fetching system and libraries information...
System:
OS: macOS 12.3
CPU: (8) arm64 Apple M1
Memory: 89.94 MB / 8.00 GB
Shell: 5.8 - /bin/zsh
Binaries:
Node: 16.14.0 - /usr/local/bin/node
Yarn: Not Found
npm: 8.3.1 - /usr/local/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.11.3 - /opt/homebrew/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.4, iOS 15.4, macOS 12.3, tvOS 15.4, watchOS 8.5
Android SDK:
API Levels: 28, 30, 31, 32
Build Tools: 30.0.2, 32.0.0, 32.1.0, 33.0.0
System Images: android-32 | Google APIs ARM 64 v8a
Android NDK: Not Found
IDEs:
Android Studio: 2021.1 AI-211.7628.21.2111.8193401
Xcode: 13.3/13E113 - /usr/bin/xcodebuild
Languages:
Java: 12.0.2 - /Library/Java/JavaVirtualMachines/jdk-12.0.2.jdk/Contents/Home/bin//javac
npmPackages:
@react-native-community/cli: Not Found
react: 17.0.2 => 17.0.2
react-native: 0.68.0 => 0.68.0
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce

react-native init test
cd test
enable newArchEnabled=true (gradle.properties)
enable enableHermes: true (build.gradle)
react-native run-android (when the yarn is not installed on the system)

Snack, code example, screenshot, or link to a repository
Github Repo:- https://github.com/pushpender-singh-ap/testV68

That's not expected. It's a bug in the setup of the New Architecture. Obviously installing yarn would fix it. @danilobuerger is working on a fix for this in https://github.com/facebook/react-native/pull/33530

Except I use yarn already & I'm getting the same error, so it has nothing do do with yarn

Hey @ItsCDubb, are you still seeing the same issue in the 0.68.1 release?

Yes I am & I don't know what it means in order to resolve it

@ItsCDubb I think it would be the most transparent if you created a new issue with reproducible steps instead 🙏

I'm using yarn and still facing the same error in v0.68.1. Any solution to solve the error? Because of this error unable to install the app neither in the emulator nor in android device

version 0.68.1 this issue has been resolved.

@pushpender-singh-ap  I am using 0.68.2 but it's still appearing, debug build not creating due to this

@FayyazAliKhan1 If you could send us screenshots with a sample repository, that would be helpful.

gradlew clean --stacktrace

Configure project :app
Unable to detect AGP versions for included builds. All projects in the build should use the same AGP version. Class name for the included build object: org.gradle.composite.internal.DefaultIncludedBuild$IncludedBuildImpl_Decorated.

Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.
You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.
See https://docs.gradle.org/7.3.3/userguide/command_line_interface.html#sec:command_line_warnings
"react": "17.0.2",
"react-native": "0.68.2",
any help??

getting build failed with the same error when trying to clean or build the app
gradlew clean or gradlew assembleRelease
react: 17.0.2
react-native: 0.68.2
anyone got this error resolved ?

@palavellig did you find any solution ?

Open Android Studio Settings,  then select Gradle JDK as version 11 from the dropdown.

the solution which i answered was worked for me.
On Thu, Oct 20, 2022 at 2:09 PM m7amad-7asan @.***>
wrote:

@palavellig https://github.com/palavellig did you find any solution ?
—
Reply to this email directly, view it on GitHub
https://github.com/facebook/react-native/issues/33525#issuecomment-1285150959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOYW23MBYXBQRDAZNLKCZLTWEEALLANCNFSM5SCZ7SCA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for your response
could you please tell how to get this screen
On Thu, Oct 20, 2022 at 11:43 AM _pal @.***> wrote:

Open Android Studio Settings, then select Gradle JDK as version 11 from
the dropdown.
[image: sckAv]
https://user-images.githubusercontent.com/61959533/196900497-6394c542-f4cf-449f-aecc-a62d0fd3bfe0.png
—
Reply to this email directly, view it on GitHub
https://github.com/facebook/react-native/issues/33525#issuecomment-1285155432,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJGB45HM6CRE7ZGCC2UKBHTWEEAZNANCNFSM5SCZ7SCA
.
You are receiving this because you commented.Message ID:
@.***>

adding this to your eas.json resolves the problem for me: @m7amad-7asan
"build": {
"production": {
"android": {
"buildType": "app-bundle",
"image": "latest"
}
}
}

@Johnhollack
Thanks but I'm not using expo

I'm facing same problem have anyone solved?

@Mmferry This issue fix in latest react native versions

@Mmferry This issue fix in the latest react native versions

Is it worked with expo 45

@Mmferry This issue fix in the latest react native versions

Is it worked with expo 45

Expo 45 is already deprecated use 48 and 47

@Mmferry This issue fix in the latest react native versions

Is it worked with expo 45

Expo 45 is already deprecated use 48 and 47

I'm just upgrading incrementally from expo 43 and wanna make sure it build successfully

@Mmferry This issue fix in the latest react native versions

Is it worked with expo 45

Expo 45 is already deprecated use 48 and 47

I'm just upgrading incrementally from expo 43 and wanna make sure it build successfully

Than upgrade to 46 because deprecated versions not run in expo go application

Still facing the same issue and i am not using expo, i am working react-native cli.
When i run my project on cmd by running this command npx react-native run-android or  yarn react-native run-android,
it gave these errors.

@Malik-Usman17 can you tell us which react native version are you using


authentication error

This one works fine:
client.send_message({
'from': 'xxxxxx',
'to': 'xxxxxx',
'text': 'A text message sent using the Nexmo SMS API',
})
But when I was trying to run client.get_calls(), I got authentication error.

Could you provide more information about your env/OS, and the code you were trying to run?. What kind of authentication error?

Closing due to inactivity.


Support automatic cookie removal on logout

Support of cookie destroying on logout by adding "remove_cookies: { 'cookieName1', 'cookieName2':  { 'path': '/', 'domain': null } }" in your configuration e.g.
$app['security.firewalls'] = array(
[...]
		'logout' => array(
			'logout_path' => '/user/logout',
			'remove_cookies' => array('cookieName1', 'cookieName2' => array('path' => '/', 'domain' => null))
		),
[...]
);

Based on discussion https://groups.google.com/d/topic/silex-php/828UcSi7bks/discussion

:+1:

Can you update the documentation for this new feature? Thanks.

I will update the PR soon.

@CSchulz Any news?

Any update on this?

Closing this old PR as there is no more activity.


Using pd.Panel as a data source does not work as expected.

I am on zipline version 0.8.3 working on windows with python 2.7.  The items in Requirements.txt were installed via conda install and zipline was installed via pip.
I have a pd.Panel of ~500 assets that are not loading correctly into the algorithm.run(pd.Panel) routine.
Basically I was expecting the asset name (string) on the major_index to be converted to an appropriate object and mapped to a sid. This seems to only work about halfway.
The sid and symbol functions from zipline.api do seem to work after the data is loaded, but when I grab prices from the history function the resulting data frame has column names that are a list of integers (not assets?).  So obviously the following does not work:
prices = history(100, '1d', 'price').dropna()

for stock, wt in zip(prices.columnames, wts):
    order_target_percent(stock, wt)

I tried to work around this by casting the integer to its asset using sid(stock).  This pushed me past the initial error, but this still fails as cached price data only has integers as keys (instead of assets?).  So when the order method tires to look up the price it throws a KeyError (all the keys are ints and I'm passing equity assets).  I believe stack trace crashed in protocol.py
So, is  this just user error on my part?  Or, is something else off?
Any help is greatly appreciated.  Thanks!
EDIT:
I went back and tried the routine with data created from the  load_bars_from_yahoo function and I get the same error.  I feel like there is something odd is happening here.

Hi, thanks for the bug report. I think this might be related to an issue I found in TradingAlgorithm.run where sids were getting added to the asset finder twice. I will try to repro this and get back to you with what I find.

Thank you for following up.

I did some additional debugging.  Basically I see bugs based on the transtion from the sid to generic assets design.  Is this correct?
The order function validates that it received an asset object (which is incorrect?) in the validate_order_params function.
So, I was able to backtest with a monkey patched TradingAlgorithm by setting validate_order_params to pass. Ignoring this asset validation, the order goes though based on just the provided sid.
I'm sure this is not the best way to fix this, but it works (for  now).  I guess there are more general design issues to resolve which is complicated by several potential data sources.
Perhaps the construction of the data sources is the correct place to address this?   Would passing an list of assets as the pd.Panel.index resolve this?  I havent' tried it yet.
Please let me know if I'm on the right track.
Thanks!

Hey, I merged a change around asset handling for panel sources, could you retry your code with the code on master?
related pr: https://github.com/quantopian/zipline/pull/942

I'll take a look as soon as I can, thanks.
On Tue, Jan 12, 2016 at 4:56 PM, Joe Jevnik<EMAIL_ADDRESS>wrote:

Hey, I merged a change around asset handling for panel sources, could you
retry your code with the code on master?
related pr: #942 https://github.com/quantopian/zipline/pull/942
—
Reply to this email directly or view it on GitHub
https://github.com/quantopian/zipline/issues/926#issuecomment-171088907.

Sorry, that does not resolve the issue.   As I said above, I believe this has to do with a transition from order methods requiring a SID to and ASSET.  I have provided snippets of my code and the full stack traces as well.
The data should be correct as I am using
data = zipline.data.load_bars_from_yahoo(stocks = ['AAPL', 'CVX'], start='01-01-2010', end='01-01-2015' )

FUNCTION AND STACK TRACE WITH AN ASSEST PASSED
def buy_positions(context, data):
    if context.day < context.lookback:
        return

    price = history(context.lookback, context.freq, field='price')

    # drop the symbols no longer traded
    # drop NA
    price.drop([s for s in price.columns if s not in data], axis = 1, inplace = True)
    price.dropna(axis=1, inplace=True)

    returns = np.log(price).diff(1)[1:].T

    # if returns.shape[1] >= 25:
    wt=opt_max_return(returns)
    print(wt[:10])
    for symbol, amount in zip(price.columns, wt):
        # set_trace()
        order_target_percent(sid(symbol), amount)

Traceback (most recent call last):
File "c:\Users\X\Desktop\Project\myrepo\local_mvo.py", line 178, in
algo_obj.run(data)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 592, in run
for perf in self.gen:
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\gens\tradesimulation.py", line 121, in transform
self.algo.instant_fill,
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\gens\tradesimulation.py", line 298, in _process_snapshot
new_orders = self._call_handle_data()
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\gens\tradesimulation.py", line 327, in _call_handle_data
self.simulation_dt,
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\utils\events.py", line 209, in handle_data
context.trading_environment,
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\utils\events.py", line 228, in handle_data
self.callback(context, data)
File "c:\Users\X\Desktop\Project\myrepo\local_mvo.py", line 126, in buy_positions
order_target_percent(sid(symbol), amount)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\utils\api_support.py", line 51, in wrapped
return getattr(get_algo_instance(), f.name)(*args, **kwargs)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 1186, in order_target_percent
style=style)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 1164, in order_target_value
target_amount = self._calculate_order_value_amount(sid, target)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 850, in _calculate_order_value_amount
last_price = self.trading_client.current_data[asset].price
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\protocol.py", line 528, in getitem
return self._data[name]
KeyError: Equity(0, symbol=u'AAPL', asset_name=None, exchange=None, start_date=Timestamp('1970-01-01 00:00:00+0000', tz='UTC'), end_date=Timestamp('2116-02-20 23:53:38.427387903+00 00', tz='UTC'), first_traded=None)
FUNCTION AND STACK TRACE WITH AN SID PASSED
def buy_positions(context, data):
    if context.day < context.lookback:
        return

    price = history(context.lookback, context.freq, field='price')

    # drop the symbols no longer traded
    # drop NA
    price.drop([s for s in price.columns if s not in data], axis = 1, inplace = True)
    price.dropna(axis=1, inplace=True)

    returns = np.log(price).diff(1)[1:].T

    # if returns.shape[1] >= 25:
    wt=opt_max_return(returns)
    print(wt[:10])
    for symbol, amount in zip(price.columns, wt):
        # set_trace()
        order_target_percent(symbol, amount)

Traceback (most recent call last):
File "c:\Users\X\Desktop\Project\myrepo\local_mvo.py", line 178, in
algo_obj.run(data)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 592, in run
for perf in self.gen:
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\gens\tradesimulation.py", line 121, in transform
self.algo.instant_fill,
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\gens\tradesimulation.py", line 298, in _process_snapshot
new_orders = self._call_handle_data()
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\gens\tradesimulation.py", line 327, in _call_handle_data
self.simulation_dt,
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\utils\events.py", line 209, in handle_data
context.trading_environment,
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\utils\events.py", line 228, in handle_data
self.callback(context, data)
File "c:\Users\X\Desktop\Project\myrepo\local_mvo.py", line 126, in buy_positions
order_target_percent(symbol, amount)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\utils\api_support.py", line 51, in wrapped
return getattr(get_algo_instance(), f.name)(*args, **kwargs)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 1186, in order_target_percent
style=style)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 1168, in order_target_value
style=style)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 1150, in order_target
style=style)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 897, in order
style)
File "C:\Miniconda3\envs\zip_test\lib\site-packages\zipline-0.8.3+232.g0dac2e0-py2.7-win-amd64.egg\zipline\algorithm.py", line 936, in validate_order_params
msg="Passing non-Asset argument to 'order()' is not supported."
zipline.errors.UnsupportedOrderParameters: Passing non-Asset argument to 'order()' is not supported. Use 'sid()' or 'symbol()' methods to look up an Asset.
MONKEY PATCH TO FIX ISSUE PASS A SID TO ORDER_TARGET_PERCENT
def pass_validate_order_params(asset, amount, limit_price, stop_price, style):
   pass

algo_obj = TradingAlgorithm(initialize=initialize, handle_data=handle_data, analyze=analyze, instant_fill = True)
algo_obj.validate_order_params = pass_validate_order_params

Hey, we will try to take a look later this week. Sorry for the delay.

I can reproduce this now - will look into it...

Sounds great.  Thanks.
On Jan 19, 2016 10:00 PM, "Richard Frank"<EMAIL_ADDRESS>wrote:

I can reproduce this now - will look into it...
—
Reply to this email directly or view it on GitHub
https://github.com/quantopian/zipline/issues/926#issuecomment-173078716.

This should be fixed by #959, which is now on master.

I suffer the same problem when using a custom datasource.
my datasource object takes in a custom layout dataframe and implements:
mapping,raw_data_gen,raw_data and worked perfectly with zipline 0.7.0
after finding this issue, I copied what AwesomeExcel did with the Monkey Patch. and now I can run on zipline 0.8.4.
Is it possible to get this fixed for a a custom datasource too? thanks


New version: Xmake-io.Xmake version 2.9.5+20240914 (2.9.5)

Created by 🥟 Dumplings in workflow run #8065.

Log
Updated: 2.9.4 → 2.9.5
No ReleaseNotes (en-US) for version 2.9.5
No ReleaseNotes (zh-CN) for version 2.9.5
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-09-14 16:56:37.336 [FAIL] Installer failed security check. Url: https://github.com/xmake-io/xmake/releases/download/v2.9.5/xmake-v2.9.5.win32.exe Result: 0x80004005
2024-09-14 16:56:13.999 [FAIL] Installer failed security check. Url: https://github.com/xmake-io/xmake/releases/download/v2.9.5/xmake-v2.9.5.win32.exe Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 895.)

@wingetbot run