small bug in the brush "snap" demo

the demo at http://bl.ocks.org/mbostock/6232537 does not snap properly to a full day when you brush a small width

changing line 74 seems to fix it:
-     d1[1] = d3.timeDay.offset(d0[1]);
+    d1[1] = d3.timeDay.ceil(d0[1]);

It was supposed to be:
d1[1] = d3.timeDay.offset(d1[0]);

I’ve fixed it.


Named async functions

Codecov Report

Merging #56 into master will decrease coverage by 0.32%.
The diff coverage is 10.00%.

@@            Coverage Diff             @@
##           master      #56      +/-   ##
==========================================
- Coverage   25.97%   25.65%   -0.33%
==========================================
  Files          32       32
  Lines         485      495      +10
  Branches       61       61
==========================================
+ Hits          126      127       +1
- Misses        335      344       +9
  Partials       24       24

Impacted Files
Coverage Δ

src/components/FeaturedCommunities.js
0.00% <0.00%> (ø)

src/components/FeaturedConferences.js
0.00% <0.00%> (ø)

src/pages/Communities.js
0.00% <0.00%> (ø)

src/pages/CommunityDetail.js
11.11% <0.00%> (-0.43%)
:arrow_down:

src/pages/ConferenceDetail.js
11.11% <0.00%> (-0.43%)
:arrow_down:

src/pages/Conferences.js
0.00% <0.00%> (ø)

src/pages/Search.js
0.00% <0.00%> (ø)

src/pages/SpeakerDetail.js
11.11% <0.00%> (-0.43%)
:arrow_down:

src/pages/Speakers.js
0.00% <0.00%> (ø)

src/components/FeaturedSpeakers.js
47.82% <100.00%> (+2.37%)
:arrow_up:

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 2e651c8...e1c7fb6. Read the comment docs.


For SQL data sources to select data from a different database doesn't reset data preview

Please fix it for this release if this is a regression.


Update the holopin.yaml to use the website badge

Update the .github/holopin.yaml file with the following info to set the default Holopin badge for this repo:
organization: dapr
defaultSticker: clrqffunr242860gjxng34gcru
stickers:
  -
    id: clrqffunr242860gjxng34gcru
    alias: website-badge

Once this is in place, maintainers can award contributors via the Holopin Bot by mentioning the contributors' username in a PR comment when the PR is accepted: @holopin-bot @username

/assign

@marcduiker there is no .github/holopin.yaml is it okay to create the file?

PR - #93


.NET applications are observable and easily diagnosable

By adding primitives to the .NET libraries for distributed tracing, metrics, and logging, .NET applications will be observable and easily diagnosable. These primitives will implement open standards defined the the OpenTelemetry project and will allow for instrumentations by multiple APM vendors.
Work Items

[x] dotnet/runtime#44445
[x] dotnet/runtime#47313
[ ] dotnet/runtime#22336
[ ] dotnet/runtime#48619

[x] dotnet/runtime#36022
[x] dotnet/runtime#45290

[ ] dotnet/runtime#44515
[x] dotnet/runtime#36064
[x] dotnet/dotnet-docker#2725
[x] dotnet/runtime#45518
[ ] dotnet/diagnostics#515
[x] dotnet/runtime#45384
[ ] dotnet/runtime#45383
[x] dotnet/runtime#45382
[x] dotnet/runtime#45590
[ ] dotnet/diagnostics#1794
[x] dotnet/runtime#48664
[x] dotnet/runtime#50658

Bulk closing .NET 6 epics and user stories. If you think this issue was closed in error, please reopen the issue and update it accordingly.


Another Dataset added

Dataset Link: https://github.com/KaniyamFoundation/all_tamil_words/blob/master/words_and_frequency.tar.bz2

Total count: 3080012
Frequency > 5: 464197
Frequency > 100: 55496
Frequency > 1000: 8724


PhantomJS v<IP_ADDRESS> - Could not find an executable 'phantomjs' that matched the requirements '>= 1.8.1', '< 3.0' - Cliver::Dependency::VersionMismatch

Hi Team,
I recently updated to PhantomJS <IP_ADDRESS>. and have run into this error while attempting to run my code using poltergeist
Could not find an executable 'phantomjs' that matched the requirements '>= 1.8.1', '< 3.0'. Found versions were {"C:/Ruby23-x64/bin/phantomjs.BAT"=>"23"}
Please Note: I am using a Windows laptop and my ruby setup is as follows:
Ruby => ruby 2.3.0p0 (2015-12-25 revision 53290) [x64-mingw32]
RubyGems => 2.6.7
Gem List =>

phantomjs (<IP_ADDRESS>)
poltergeist (1.10.0)

Thanks

@Sam-Levene  What is the output of running C:/Ruby23-x64/bin/phantomjs.BAT ??

@Sam-Levene it appears as though the only executable phantomjs that is on your PATH is C:/Ruby23-x64/bin/phantomjs.BAT, and when it is executed with a solitary --version flag, we're parsing out 23 as the first number (which coincidentally is part of the path to the .BAT file).
Can you run the following from your command line?
C:\Ruby23-x64\bin\phantomjs.BAT --version

or
phantomjs --version

If the executable is in fact a supported version, you can force the version detection library to not verify it by setting the CLIVER_NO_VERIFY environment variable.

@yaaui  Very very sorry -- I went to delete my comment and somehow ended up deleting yours instead - could you please repost

This was originally posted by @yaauie  but I somehow accidentally deleted it
@Sam-Levene it appears as though the only executable phantomjs that is on your PATH is C:/Ruby23-x64/bin/phantomjs.BAT, and when it is executed with a solitary --version flag, we're parsing out 23 as the first number (which coincidentally is part of the path to the .BAT file).
Can you run the following from your command line?
C:\Ruby23-x64\bin\phantomjs.BAT --version
or
phantomjs --version
If the executable is in fact a supported version, you can force the version detection library to not verify it by setting the CLIVER_NO_VERIFY environment variable.

Oh sorry didn't notice he was running a bat file. Then it's not the issue

Trying the C:\Ruby23-x64\bin\phantomjs.BAT --version gave the following issue:

and phantomjs --version gave: "'phantonjs' is not recognized as an internal or external command,
operable program or batch file." despite having PhantomJS installed on my computer. Not sure why it's saying not recognised.

It looks like you're using the `phantomjs-binaries'](https://github.com/avaynshtok/phantomjs-binaries) gem, and have a  `phantomjs.BAT` on your [`PATH` pointing to it.
Unfortunately, phantomjs-binaries doesn't support Windows; it uses the uname syscall (which is not on Windows by default) to detect which of the pre-built binaries to select, but even then, it only provides 64-bit builds for linux & darwin.
You will need to install phantomjs normally and add it to your PATH so that command-line programs (like poltergeist) can find it.

Nevermind. Please Close.


py::end_method_list is defined in autofunction but needed in automodule

Which causes confusing errors if you only include automodule

Ah, we should just include that in automodule.h then.


Medical Pouches can't hold things they should be able to

Description
Will need a look over/verification from 13, but things like surgical cases, blood bags, and stasis bags can't be placed in medical pouches.

Reproduction

Get medical pouch
get various medical supplies like the listed above
try and stuff them in

I went to look at the CM code and compared it. First aid pouches can't hold roller beds, stasis bags, or surgical cases. The same is true for other medical pouches, excepting the medical kit pouch which can hold such things. I didn't find any particular discrepancies between our various medical pouches and CMs. Is there a specific item you are certain is supposed to be in these pouches?

I went to look at the CM code and compared it. First aid pouches can't hold stasis bags, blood bags, or surgical cases. The same is true for other medical pouches, excepting the medical kit pouch which can hold such things. I didn't find any particular discrepancies between our various medical pouches and CMs. Is there a specific item you are certain is supposed to be in these pouches?

I don't believe the first aid pouches are missing anything, but the medkit pouch is definitely missing some items which I named above

Well blood should be easy enough to fix, though I'm pretty sure there's a check preventing storage containers other than backpacks/satchels/duffle bags, from holding other storage containers, I'll take a look and see in an hour.


Add proper prompt for Chat Journaling

As a user I want to be able to chat with a proper journal chat bot so that it actually helps me journal and gives me smart insights.
Review the prompt given for the chat journal.

 The chat users now you already - you don’t have to introduce yourself. As users chat with you, you provide guidance for journaling and self-reflection. Your goal is to help users explore their emotions and personal development. By asking simple, non-intrusive questions, you empower users to delve deeper into their thoughts. Always ask one question at a time and await the users answer before asking more questions. Your first answer should be one simple question. Remember to maintain a quiet and supportive presence, akin to a trusted journal. Be very concise in answers.
Prompt 2 for challenge/goal discovery:
You are a friendly and motivating journaling companion within an app. The chat users now you already - you don’t have to introduce yourself. As users chat with you, you provide guidance for journaling and self-reflection. Your goal is to help users explore their emotions and personal development. By asking simple, non-intrusive questions, you empower users to delve deeper into their thoughts. Always ask one question at a time and await the users answer before asking more questions. Your first answer should be one simple question.
Once you identify a SMART goal with the user, you suggest specific daily challenges that align with their aspirations. Ask at least 3 questions one after another before you start identifying goals.
Remember to maintain a quiet and supportive presence, akin to a trusted journal. Be very concise in answers.


(lint) Update parser for the result of cmake-lint

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 79.16667% with 5 lines in your changes missing coverage. Please review.

Project coverage is 19.66%. Comparing base (1a89b76) to head (bb00462).

Files
Patch %
Lines

src/gammar.rs
79.16%
5 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
+ Coverage   19.22%   19.66%   +0.43%
==========================================
  Files          24       24
  Lines        3485     3504      +19
==========================================
+ Hits          670      689      +19
  Misses       2815     2815

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

Thank you


shared api key model

📚  View the storybook of this branch here

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of APIM master. (Same login and password as APIM master)

⚠ The JDBC part has been tested with postgres only, due to compatibility issues with Apple Silicon chips


DOCKER BUILD IMAGES -QUERY

this is my sample dockerfile content:
FROM tomcat:10.0.0-M7-jdk8-openjdk
MAINTAINER vMC
ADD jenkins.war /usr/local/tomcat/webapps/jenkins.war
WORKDIR /opt/vmc/
EXPOSE 8080
CMD ["catalina.sh", "run"]
this is the content im getting while trying to build the image:
ubuntu@ip-172-31-3-193:/opt/vmc/images$ docker image build .
Sending build context to Docker daemon  2.048kB
Step 1/5 : FROM tomcat:10.0.0-M7-jdk8-openjdk
---> aaf96e87b224
Step 2/5 : MAINTAINER vMC
---> Using cache
---> e4ff1a829d8f
Step 3/5 : ADD jenkins.war /usr/local/tomcat/webapps/jenkins.war
ADD failed: stat /var/lib/docker/tmp/docker-builder860780445/jenkins.war: no such file or directory

Add failed due path error or wrong path
Please check your machine
Jenkins war available are not
Enter pwd to check path of the file location

Add failed due path error or wrong path
Please check your machine
Jenkins war available are not
Enter pwd to check path of the file location

in manual , i tried pulling the  tomcat image , runnig the image ans inspected the image with he process id
even there it shows the working drectory as /usr/local/tomcat
but while trying to access the folder, it shows , no such directory is available::
looks like some fishy
PLEASE SOMEONE HELP ME IN THIS

which version of tomcat image you have pulled try using tomcat:9 you can get path

ADD instruction contains or URL and , if u don't specify absolute path at src, it looks at Current directory. check once
WORKDIR means Default working directory of Container eg: when u hit docker container exec -it <cont. Id>. it goes to default directory whatever u specified path in WORKDIR.


[AutoPR apimanagement/resource-manager/alertsmanagement/resource-manager] Swagger support for DDoS protection policy customization feature

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4441

Build output:

dist/com.microsoft.azure.network.v2018_07_01/azure-mgmt-network-1.0.0-beta-javadoc.jar
dist/com.microsoft.azure.network.v2018_07_01/azure-mgmt-network-1.0.0-beta.jar
dist/com.microsoft.azure.network.v2018_07_01/azure-mgmt-network-1.0.0-beta.pom
dist/com.microsoft.azure.network.v2018_07_01/azure-mgmt-network-1.0.0-beta-sources.jar
dist/com.microsoft.azure.network.v2018_08_01/azure-mgmt-network-1.0.0-beta-javadoc.jar
dist/com.microsoft.azure.network.v2018_08_01/azure-mgmt-network-1.0.0-beta.jar
dist/com.microsoft.azure.network.v2018_08_01/azure-mgmt-network-1.0.0-beta.pom
dist/com.microsoft.azure.network.v2018_08_01/azure-mgmt-network-1.0.0-beta-sources.jar

Closing PRs that have been open for more than 1 month.


Envoy protos should not have a dependency on opencensus proto

WARNING: If you want to report crashes, leaking of sensitive information,
and/or other security issues, please consider
reporting them using appropriate channels.
Bug Template
Title: Envoy protos should not have a dependency on opencensus proto
Description:
We use Envoy protos for using the xDS APIs between our clients and management server. Until recently things were okay but now there is a dependency on opencensus/proto/trace/v1/trace_config.proto which is unfortunate. The dependency exists because envoy/config/filter/network/http_connection_manager/v2/http_connection_manager.proto includes third_party/envoy/src/api/envoy/config/trace/v2/trace.proto which now includes third_party/opencensus_proto/trace/v1/trace_config.proto . I think http_connection_manager.proto is a common file included from multiple "core" files.
This has started happening recently probably with the PR https://github.com/envoyproxy/envoy/pull/10324
Repro steps:
Without installing the opencensus protos when I try to compile base.proto I get the following compile errors:
Execution failed for task ':grpc-xds:generateProto'.
  opencensus/proto/trace/v1/trace_config.proto: File not found.
  envoy/config/trace/v2/trace.proto:11:1: Import "opencensus/proto/trace/v1/trace_config.proto" was not found or had errors.
  envoy/config/trace/v2/trace.proto:168:3: "opencensus.proto.trace.v1.TraceConfig" is not defined.
  envoy/config/filter/network/http_connection_manager/v2/http_connection_manager.proto:10:1: Import "envoy/config/trace/v2/trace.proto" was not found or had errors.

@dapengzhang0 can add more comments.

@yskopets @g-easy do you have any thoughts here?
I'm wondering why it is not acceptable to have additional proto dependencies for HCM? I.e. what is the criteria.
@sanjaypujare I imagine this is in the context of gRPC, doesn't gRPC depend on OpenCensus anyway?

I'm wondering why it is not acceptable to have additional proto dependencies for HCM? I.e. what is the criteria.

Couple of things:

compiling the protos becomes more difficult because of the additional dependency
should envoy depend on a particular metric and tracing API?

@sanjaypujare I imagine this is in the context of gRPC, doesn't gRPC depend on OpenCensus anyway?

But not on opencensus protos. Even the dependency on opencensus libraries has been removed recently IIRC. Will add a reference here.

For grpc-java https://github.com/grpc/grpc-java/pull/6577 talks about "dependency on opencensus is at runtime by reflection". Even without this we never had any dependency on opencensus protos.

Why should trace.proto explicitly depend on a particular tracing provider third_party/opencensus_proto/trace/v1/trace_config.proto? Could it use Any type for the trace config and let the implementation select the tracing provider and apply the config dynamically in the runtime?

I think Any should be fine here, @yskopets WDYT?

Part of Envoy's config depends on opencensus-proto's trace_config.proto. We could change the Envoy config to not do this.
However, opencensus-cpp's ocagent exporter also depends on opencensus-proto.
(likewise, our Stackdriver exporter depends on googleapis)
gRPC doesn't strictly depend on OpenCensus, there's an optional grpc-opencensus "plugin" that bridges the two but Envoy doesn't use it.

@htuch
Sorry, I didn't notice this question in time.
I think, the issue here is not that Tracing.OpenCensusConfig message explicitly depends on opencensus.proto.trace.v1.TraceConfig, but rather that dependency HttpConnectionManager => Tracing.Http pulls in Tracing.OpenCensusConfig.
Specifically, Tracing.Http is already defined using Any:
message Tracing {
  message Http {
    string name = 1 [(validate.rules).string = {min_bytes: 1}];

    oneof config_type {
      google.protobuf.Any typed_config = 3;
    }
}

And HttpConnectionManager only depends on Tracing.Http.
So, it should be possible to compile HCM without OpenCensus.
I think, the right solution here is to move Tracing.OpenCensusConfig (and other providers) out of the *.proto file with Tracing.Http.
Regarding the dependency Tracing.OpenCensusConfig => opencensus.proto.trace.v1.TraceConfig, I think that replacing an explicit dependency with dependency on Any hurts expressiveness of the API.
If you're not planning to use OpenCensus, then don't compile Tracing.OpenCensusConfig.
If you choose to compile Tracing.OpenCensusConfig, then it won't have much value without opencensus.proto.trace.v1.TraceConfig.

@htuch btw, why does configuration for tracing provider implementations live in config.trace rather than in extensions.tracers.<provider> ?

@htuch
...
So, it should be possible to compile HCM without OpenCensus.

That's the change being requested. Who can make that change?

@sanjaypujare +1 to @yskopets proposal. Optimally each tracer config should be in its own proto file and in the extension directory. Can we make that happen please?

@mattklein123 we have a workaround for our issue so this is no longer a blocker for us although we would prefer to have this resolved. Not sure I can get it to it soon. How about @yskopets or @g-easy or the person who introduced the dependency?

@sanjaypujare 🙂ok, I'll see what I can do.

@yskopets thanks.

Yes, agreed, we should move trace providers to reflect the regular extension structure.

@yskopets FWIW, it's totally fine to move the protos to distinct files inside existing packages within a major version; this doesn't constitute a breaking API change. Moving across packages does though, so we need to do that only in v4alpha via the move_to_package file-level option.

@sanjaypujare We've refactored *.proto files to remove the dependency on OpenCensus.
Please give it a try when you have time.

@yskopets thanks a lot! When we have some time we'll go ahead and reimport these protos and verify we don't have those dependencies anymore.


How can I keep my torso still when my head is moving

When I move my head, my torso also move, which leads to stitching problems when I finally put the result into the origin source image. How can I keep my torso still?

Seconded. I think the model only works best for the face. Let's see if we get a solution to this.


Is there more detailed information available about the SF internals?

This information doesn't seem to align with the following paper:
Gopal Kakivaya et al. “Service Fabric: A Distributed Platform for Building Microservices in the Cloud”. Proceedings of the 13th EuroSys Conference (Apr. 2018)  [URL: https://dl.acm.org/citation.cfm?doid=3190508.3190546 ]
In this paper the naming service is described as a component of the reliability subsystem. Is this outdated information?
Moreover, the other components of the reliability subsystem seem to have changed slightly.
Thank you in advance

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

ID: 4f44786c-b01a-6add-80f9-99c1114f208a
Version Independent ID: 73ef710c-fdca-3622-d797-0cff0827c8dd
Content: Architecture of Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-architecture.md
Service: service-fabric
GitHub Login: @rishirsinha
Microsoft Alias: rsinha

Thanks for the feedback! We are currently investigating and will update you shortly.

Also, is the Resource Manager from this documentation equal to the Placement and Load Balancer (PLB) from the paper?

@dzoech I looked at the link you provided but it is not an official MSFT source. So I would advise to generally use official documentation instead. That being said, it seems this specific document you open the issue on has not been refreshed in a while.
I am going to assign this to the Service Fabric Documentation team to review and ensure the doc is up to date prior to comparing to the article you referenced.
@athinanthny @peterpogorski can one of you take a look?

Thank you for looking into this.

I looked at the link you provided but it is not an official MSFT source.

Yeah, comparing the publication dates, the linked paper seems more up to date.

@MicahMcKittrick-MSFT sorry for being impatient but have you had a chance to get further information about the correct terminology or more specifically how the different terms of both sources are related?

@athinanthny @peterpogorski any update?

#reassign masnider

Please post any specific SF implementation questions to the Microsoft Q&A Service Fabric channel. That will get you in touch with engineers versed in the current state of SF internals. Many of the authors of the conference paper above no longer work on the team. #please-close


[Feature Request]: Support for arm64 processors

Describe the feature
I wanted to use Vibe on my brand new Copilot+ Microsoft Surface Pro 11 with Snapdragon X Elite processor, but I get an error message saying my CPU is old. Guess it's because arm64 processors aren't supported yet. I think there's a huge potential behind that.
I would like to offer myself as a beta tester if you plan to support the CPU and the NPU on it.

Hey!
Thanks for report about it.
Can you try open it from cmd.exe with log enabled and check for errors?

open cmd.exe
run

taskkill /IM vibe.exe /F
set RUST_BACKTRACE=1
set RUST_LOG=vibe=debug,whisper_rs=debug
%localappdata%\vibe\vibe.exe

install ""MSVC v143 - VS 2022 C++ ARM64/ARM64EC build tools (14.41-17.11)"
it can be found on the individual components on visual studio installer, then run these commands on your project
rustup target add aarch64-pc-windows-msvc
cargo build --target aarch64-pc-windows-msvc --release
It should work for your project assuming the code dosnt do something arch specific etc


Sometimes app has inexplicable database related crash!

When I clicked the save computer button the app crashed. This issue isn't repeatable, I tried many times since then.

Only happened once, couldn't make it happen again. Closed!


What is the Privacy Policy

Is there a privacy policy for utterances? How can I adapt my own privacy policy with the inclusion of the widget?
Should I be including GitHub's privacy policy? Thanks!

I'll update and restore the old privacy-policy.md- the cookies section is out of date.


Add support for Hebrew language

Hi all,
I'm looking to add support for Hebrew as a data language.
I'm using the "intfloat/multilingual-e5-large" or the Amazon titan embedding models. Both support Hebrew.
I've tried to select Arabic or English but it does not work properly. Either I get gibberish or Hebrew in the wrong direction (LTR)
Is it only about adding a language to this list (with the values 'hebrew', 'Hebrew')?
lib/user-interface/react-app/src/common/constants.ts
Anything else?
Thank you

The main issue with the text direction was when uploading PDF files. When uploading docx files or TEXT there was no problem


Release otel-arrow v0.17.0

Includes instrumentation changes, core collector upgrade and some cleanup.
Included PRs:

https://github.com/open-telemetry/otel-arrow/pull/146
https://github.com/open-telemetry/otel-arrow/pull/149
https://github.com/open-telemetry/otel-arrow/pull/150
https://github.com/open-telemetry/otel-arrow/pull/151
https://github.com/open-telemetry/otel-arrow/pull/153

Forgot about https://github.com/open-telemetry/otel-arrow/pull/151, @jmacd can this be merged and included in this new release? I also updated collector to v0.94.1 in https://github.com/open-telemetry/otel-arrow/pull/153

fyi I tested https://github.com/open-telemetry/otel-arrow/pull/152/commits/bde64d97c2957fc941b869b0335de05b616b337c by hand; the Makefile is syntactically correct, but the build step fails b/c of the release state.


dapr message invocation with vpn doesn't work after 20 seconds

Hi,
I have setup a local dapr configuration. Installed dapr and the neccesary containers are running in docker. When running 2 different web api's with visual studio the communicate with dapr message invocation works. When switching to a vpn connection this communication works for max 20 secondes after the last message invocation and after that i get a 500 response from the request. Does anybody know a solution or at least at direction where to look at.
Thanks

Hi @mPoelen,
Can you please elaborate on your local environment setup? How are you performing service invocation, is it through a Dapr SDK?

Yes its trough a dapr SKD in .Net
Here is the class with the method invokation:

Here i have dapr settings from the appsettings.development.json

This is the log console message that appears every 30 seconds without vpn and without doing the invokation.

This is the log message that appears every 30 seconds with vpn and without doing the invokation:

This is a part of the response when doing a invokation with the time out message

Do you need more info?

What type of machine are you using?

yes, its about service to service communication. What network settings do i need to change?

Can you confirm the answers to the question above? What type of machine?

The answer will most likely depend on the type of machine and the policies used. Are you able to replicate these issues with all dapr svc to svc calls on your machine? This problem would be consistent. On my Mac I go to network settings and have to disable a content filter that runs as part of Microsoft defender for Mac. Not sure if that is applicable to you

I have a windows machine and i am able to replicate the issue. The issues also popup with other services and other svc to svc calls. Oke i will look into the content filter.  Thanks, if you have more directions let me know. what i get out of the log it seems that the correct ip address is removed from the mDNS cache.

Were you able to look into this on your side or find any potential running agent that might be blocking proper local resolution?


Branch-merge bot does not work on aspnetcore-tooling due to files that exceed MAX_PATH

The bot that merges release/* branches back to master does not work on https://github.com/aspnet/AspNetCore-Tooling because this repo contains files which exceed MAX_PATH.
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/BindToComponent_EventCallback_SpecifiesValueAndExpression_Generic/TestComponent.mappings.txt: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/BindToComponent_EventCallback_SpecifiesValue_WithMatchingProperties/TestComponent.codegen.cs: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/BindToComponent_EventCallback_SpecifiesValue_WithMatchingProperties/TestComponent.mappings.txt: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/BindToComponent_SpecifiesValueAndChangeEvent_WithMatchingProperties/TestComponent.codegen.cs: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/BindToComponent_SpecifiesValueAndChangeEvent_WithMatchingProperties/TestComponent.mappings.txt: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/BindToComponent_SpecifiesValueAndChangeEvent_WithoutMatchingProperties/TestComponent.codegen.cs: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/BindToComponent_SpecifiesValueAndChangeEvent_WithoutMatchingProperties/TestComponent.mappings.txt: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/ChildComponent_WithGenericChildContent_SetsParameterNameOnComponent/TestComponent.codegen.cs: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/ChildComponent_WithGenericChildContent_SetsParameterNameOnComponent/TestComponent.mappings.txt: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentDesignTimeCodeGenerationTest/EventHandler_OnElement_ArbitraryEventName_WithEventArgsMethodGroup/TestComponent.mappings.txt: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentRuntimeCodeGenerationTest/BindToComponent_SpecifiesValueAndChangeEvent_WithoutMatchingProperties/TestComponent.codegen.cs: Filename too long
error: unable to create file src/Razor/test/Microsoft.AspNetCore.Razor.Language.Test/TestFiles/IntegrationTests/ComponentRuntimeCodeGenerationTest/BindToComponent_SpecifiesValueAndChangeEvent_WithoutMatchingProperties/TestComponent.mappings.txt: Filename too long

https://dev.azure.com/dnceng/internal/_build/results?buildId=104473
cc @dougbu @rynowak

Note this is also blocking these test files from being used on helix

72b939d85573

@dougbu this max path issue also exists for https://github.com/aspnet/AspNetCore/tree/master/src/Mvc/Mvc.Api.Analyzers/test/TestFiles and https://github.com/aspnet/AspNetCore/tree/master/src/Mvc/Mvc.Analyzers/test/TestFiles which causes helix to reject these work items since the combined path is too long, do you think you can apply the same fix to those as well?

@HaoK if there's an issue, I can take it. Just assign it to me.
But, we shouldn't try to reuse this issue because it was about a related issue in the AspNetCore-Tooling repo.


Disable bidirectional models

This PR turns off live synchronization between model artifacts and the model editor in the Rill UI.

This branch was superceded by another one ~ closing for now since this was in service of last thursday's presentation.


Add speculatvie tracing functionality to bpftrace command

Is your feature request related to a problem? Please describe.
This is a feature request.
Describe the solution you'd like
Add the ability to tentatively trace data and decide whether to commit the data to a tracing buffer or discard it.  This would be similar to 'speculative tracing' in dtrace.  This feature would allow for focused investigations as it should only print when the specific trace criteria is met, thereby reducing the amount of data collected.
Describe alternative solutions or features you've considered
Review full trace output.

Seems reasonable. Will need a design before committing to it -- this could complicate the runtime/codegen bits quite a bit if done poorly

The question is what would "tracing buffer" mean in this case. IIUC, buffers in dtrace are used for passing data from kernel to userspace, however, BPF has maps to do that. So, I'm wondering if the same as dtrace does with speculative tracing [1] can be done using BPF maps and if expressions in bpftrace.
FWIW, I'm in touch with the OP, so will post more details if I get some.
[1] https://docs.oracle.com/cd/E18752_01/html/819-5488/gbxxu.html

Sounds reasonable but after years it was the biggest failure in the design of DTrace. Never used to solve a real issue. The overhead was always too high. Please close this request.

I can't speak to the wisdom of implementing it in bpftrace, but I would like to point out that speculative tracing has certainly been used to debug real issues.  Like many things in DTrace, it is arcane (and not needed by most people most of the time), but when it is needed, it is (in my experience) essential.

FWIW used speculation in dtrace many times to solve real issues - it's very much usable as long as you didn't do silly things in the speculation. One example we ran in production to nail down a rather tricky issue:
https://github.com/jlevon/grot/blob/master/joyent/manta/manatee-381/manatee-381.d

I'm not very familiar with DTrace, let alone with the speculative tracing feature but from what I understand, it's something that should be replaceable by using BPF maps. I'll try to explain my view below, please correct me if I'm wrong.
The way I understand speculative tracing is that it allows me (as a user) to add something to the tracing buffer (i.e. the place used for kernel->userspace communication) and then to confirm (or reject) whether the thing should stay in the buffer (i.e. be communicated to the user), possibly from a different event. Traditional workflow would be (each step from a potentially different probe):
start speculation -> add to speculative buffer -> commit buffer (or reject buffer)

If this is correct, then I'd say that maps are the way to approach the same problem in the BPF world. In short, the corresponding bpftrace workflow would be:
create BPF map -> write to map -> print the map (or delete the map)

Or am I missing something and speculative tracing offers a functionality that maps are unable to cover?

There also might be plans to have a bpf ring buffer that overwrites old data, which could be used for this.


[BUG] gcloud not working for dev-test instances

Error log:
ssh: connect to host <IP_ADDRESS> port 22: Operation timed out

Recommendation: To check for possible causes of SSH connectivity issues and get
recommendations, rerun the ssh command with the --troubleshoot option.

gcloud compute ssh dev-test-0 --project=testnet-dev-ground --zone=asia-east1-b --troubleshoot

Or, to investigate an IAP tunneling issue:

gcloud compute ssh dev-test-0 --project=testnet-dev-ground --zone=asia-east1-b --troubleshoot --tunnel-through-iap

ERROR: (gcloud.compute.ssh) [/usr/bin/ssh] exited with return code [255].

It turned out that it was an internet provider's issue (SK Broadband). Now resolved by replacing an internet modem.

ERROR: (gcloud.compute.ssh) [/usr/bin/ssh] exited with return code [255].

Is turned out that 'default-allow-ssh' firewall policy has been disabled since the previous issue.
I just enabled it again, and now it works.


[Help needed] Can't build swift package

I tried to make a swiftlane package, but when I got some errors when trying to build the package

Error message:
me@my-machine swiftlanetest % swift build
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/ApplicationServices.framework/Frameworks/ATSUI.framework/Modules/module.modulemap:2:19: error: umbrella for module 'ApplicationServices.ATSUI' already covers this directory
  umbrella header "ATSUnicode.h"
                  ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/ApplicationServices.framework/Frameworks/ATSUI.framework/Modules/module.modulemap:4:10: error: inferred submodules require a module with an umbrella
  module * { export * }
         ^
<module-includes>:1:9: note: in file included from <module-includes>:1:
#import "Headers/AppKit.h"
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/AppKit.h:15:9: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/AppKit.h:15:
#import <AppKit/NSActionCell.h>
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSActionCell.h:8:9: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSActionCell.h:8:
#import <AppKit/NSCell.h>
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSCell.h:11:9: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSCell.h:11:
#import <AppKit/NSText.h>
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSText.h:9:9: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSText.h:9:
#import <AppKit/NSView.h>
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSView.h:19:9: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSView.h:19:
#import <AppKit/NSResponder.h>
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSResponder.h:10:9: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSResponder.h:10:
#import <AppKit/NSEvent.h>
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSEvent.h:10:9: error: could not build module 'ApplicationServices'
#import <ApplicationServices/ApplicationServices.h>
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSColor.h:51:9: note: while building module 'CoreImage' imported from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSColor.h:51:
#import <CoreImage/CIColor.h>
        ^
<module-includes>:1:9: note: in file included from <module-includes>:1:
#import "Headers/CoreImage.h"
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreImage.framework/Headers/CoreImage.h:8:9: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreImage.framework/Headers/CoreImage.h:8:
#import <CoreImage/CoreImageDefines.h>
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreImage.framework/Headers/CoreImageDefines.h:19:10: error: could not build module 'ApplicationServices'
#include <ApplicationServices/ApplicationServices.h>
         ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSColor.h:51:9: note: while building module 'CoreImage' imported from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSColor.h:51:
#import <CoreImage/CIColor.h>
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreImage.framework/Headers/CIImage.h:10:9: note: while building module 'CoreVideo' imported from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreImage.framework/Headers/CIImage.h:10:
#import <CoreVideo/CoreVideo.h>
        ^
<module-includes>:1:9: note: in file included from <module-includes>:1:
#import "Headers/CoreVideo.h"
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreVideo.framework/Headers/CoreVideo.h:29:10: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreVideo.framework/Headers/CoreVideo.h:29:
#include <CoreVideo/CVPixelBuffer.h>
         ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreVideo.framework/Headers/CVPixelBuffer.h:23:10: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreVideo.framework/Headers/CVPixelBuffer.h:23:
#include <CoreVideo/CVImageBuffer.h>
         ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/CoreVideo.framework/Headers/CVImageBuffer.h:28:10: error: could not build module 'ApplicationServices'
#include <ApplicationServices/ApplicationServices.h>
         ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSOpenGLLayer.h:11:9: note: while building module 'QuartzCore' imported from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/AppKit.framework/Headers/NSOpenGLLayer.h:11:
#import <QuartzCore/CAOpenGLLayer.h>
        ^
<module-includes>:1:9: note: in file included from <module-includes>:1:
#import "Headers/QuartzCore.h"
        ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/QuartzCore.framework/Headers/QuartzCore.h:9:10: note: in file included from /Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/QuartzCore.framework/Headers/QuartzCore.h:9:
#include <QuartzCore/CoreImage.h>
         ^
/Applications/Xcode-13.2.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX11.3.sdk/System/Library/Frameworks/QuartzCore.framework/Headers/CoreImage.h:1:10: error: could not build module 'CoreImage'
#include <CoreImage/CoreImage.h>
         ^
<unknown>:0: error: could not build Objective-C module 'AppKit'
[144/927] Compiling a_i2d_fp.c
me@my-machine swiftlanetest %

My Package.swift file:
// swift-tools-version:5.5
// The swift-tools-version declares the minimum version of Swift required to build this package.

import PackageDescription

let package = Package(
    name: "swiftlanetest",
    platforms: [
        .macOS(.v12),
    ],
    products: [
        .executable(name: "swiftlane_runner", targets: ["SwiftlaneRunner"]),
    ],
    dependencies: [
        // Dependencies declare other packages that this package depends on.
        // .package(url: /* package url */, from: "1.0.0"),
        .package(name: "Swiftlane", url: "https://github.com/onmyway133/Swiftlane", from: "0.1.5"),
    ],
    targets: [
        // Targets are the basic building blocks of a package. A target can define a module or a test suite.
        // Targets can depend on other targets in this package, and on products in packages this package depends on.
        .executableTarget(
            name: "SwiftlaneRunner",
            dependencies: [
                .product(name: "Swiftlane", package: "swiftlane"),
            ]),
    ]
)

And my specs are:
OS: macOS 12.2.1
Chips: Apple M1 Max
Xcode: 13.2.1
What should I do next?

Hi @el-hoshino , thanks for checking out. Can you try example https://github.com/onmyway133/Swiftlane/blob/main/Examples/Script/Package.swift to see if it is working for you?

@el-hoshino Hi, I see, that lib has a new release so I will need to update Swiftlane too

@el-hoshino Updated, you can checkout main branch now

@onmyway133 Thanks a lot!
I tried the lates main branch and now I get this error, both Xcode 13.2.1 and Xcode 13.3:
% swift build
error: the executable 'xcbeautify' requires macos 10.10, but depends on the product 'ArgumentParser' which requires macos 10.15; consider changing the executable 'xcbeautify' to require macos 10.15 or later, or the product 'ArgumentParser' to require macos 10.10 or earlier.

seems like xcbeautify's issue?

I tried the 0.12.0 release of xcbeautify and it works fine... weird 🤔

It works on my machine. I only use the lib from xcbeautify in Swiftlane, not the executable, so I don't think there is a need for ArgumentParser

Agree. My colleagues don't have this problem either 🤔
Are you using Apple Silicon or Intel mac?

@el-hoshino I use both, and it runs fine on both. Does any examples in Examples folder work for you?

I tried CLI and it works fine.
I also tried MyApp and it works fine too but seems like it actually is just a Hello World...?

Ah ja, MyApp is just a sample project to test Swiftlane

@onmyway133 Hi. I'm not sure whether it's related or not, but I updated my OS to macOS 12.3.1 and now it's working 🎉
Although the @main attribute in Script/main.swift seems needing to be fixed under latest Swift environment

@el-hoshino glad you've got it working 🎉
If you have any ideas or PRs to make Swiftlane better, that's more than appreciated

@onmyway133 Sure, my colleague should make a PR to fix the @main attribute problem in Script later I think, and I'll check some more details about the examples!
Since the problem is fixed, should I close the comment now?


Mention of Push in setting up app in App Center link is vague

Step 1 under "Create your app in the App Center Portal to obtain the App Secret" says:
To learn how to get started with Push, read the appcenter.ms documentation, where appcenter.ms is a link. It just links to appcenter.ms. Can it be updated to point somewhere more specific?
Also, is this even the section it should be in? This section is talking about creating an app in App Center to get an app secret.

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

ID: 06ece9ab-9965-2ddf-34b6-44f0f61fe599
Version Independent ID: 16261b98-a85e-060f-ccf4-3a24f5f14c4c
Content: Get Started with UWP - Visual Studio App Center
Content Source: docs/sdk/getting-started/uwp.md
Service: vs-appcenter
GitHub Login: @elamalani
Microsoft Alias: emalani

Closing as this is an internal request.


Ubuntu tweak

Added additional step to Appendix B to help address the Debian specific issue of how to compile 1.4.2 (needed to fix #292 on Debian systems) using instructions documented in #421.  Both issues have been closed, but without the matching PR @Benjamin-Dobell requested.  Hopefully this clears things up for future users.

Ubuntu 16.04.4 LTS
sudo make install works fine for me.


failed to load ksched.ko

Hello, I am trying to reproduce results of this paper. I think I have run commends based on README.md, but some problems happened when I set shenango.
Here is output of my system.
$uname -a && ofed_info -s
Linux poweredge0-PowerEdge-R740 5.0.0-20-generic #21~18.04.1-Ubuntu SMP Thu Jun 27 04:04:37 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
MLNX_OFED_LINUX-4.6-<IP_ADDRESS>:

And output.md is the terminal output when I run sudo ./build.sh in AIFM/shenango.
Thanks in advance for any help.
output.md

Hi can you check if cstate is enabled in your kernel?

Oh, I find intel_idle.max_cstate=0 in the grub file of the server. I will change it into 5, reboot the server and try to set again.
Thanks for quick and kind response.

You are very welcome. Feel free to reopen this issue if you still encounter the same error.


Max Line

How to set max line rather than max line height

Please ask usage questions on Stack Overflow.


Async tile pre-caching

Issue Type

[x] Improvement
[x] Performance

Description and/or steps/code to reproduce the problem
Now that the zoom pinch gesture is smoother (no View.scale anymore but floating point zoom only), there are some side effects when zooming across integer zoom boundaries:

@spyhunter99 mentioned "on mapnik a momentary full screen flicker" (#908)
@miguel-porto mentioned a "slight pause" (#812)

That makes sense, because when we cross the integer zoom boundaries, we need to get at the same time (in the same TilesOverlay display loop, playing in the UI thread) a dozen of new tiles, that we need to extract either from the memory, from the database or from the network.
The screen flickering @spyhunter99 experienced is probably the case when all tiles belonging to the new zoom are in the database but not in the memory yet: before we put them from database to memory, we have nothing to display and therefore we display empty gray tiles.
Async'ly pre-caching tiles from database to memory would solve this issue, assuming the speed of access to memory is not the issue there.
The slight pause @miguel-porto experienced would probably be fixed if we implement the async pre-caching. In his case there seems to be an additional issue regarding access to memory, possibly because the tiles he uses are heavier, memory-wise, but the slight pause doesn't seem to appear when the tiles are already in memory, therefore it should be OK too.
What would be the scope of the pre-cache? In the version of MapTileCache implemented in #908, we cache the tiles currently displayed PLUS the equivalent tiles of zooms +-1. I guess we could use the same tile spans.
Then there's the question of how aggressive we are in pre-caching: should we only rely on stored data or get data from the network?
Version of osmdroid the issue relates to:
6.0.0

What would be the scope of the pre-cache?
This is a fundamental question. I'll also try to think about some proposal on this. I think the +-1 zooms should be combined with some kind of LRU (not strictly LRU, but something to think).
In an ideal scenario, one could have an option to choose the best approach for each use case (plain LRU, +-1 zoom cache or other possibility, like the frequency-based LRU).

@miguel-porto Fair enough. And this flexibility also applies to the memory cache tile list.
I guess we need a new interface (MapTileListDestination?) with a MapTileList computeListFromSource(MapTileList source) method where the list of tiles will be computed from the currently displayed tiles (as a source list).
Classes would implement it as "zoom+-x" (for "zoom+1", "zoom-1") or "border+-x".
Other classes would implement it without caring at all about the source: "lru", "fixed-tile-list".
And an array class would gather all the results, according to the users need: "I want systematically Portugal tiles from zoom 4 to zoom 7, the 2-tile-wide border around the displayed tiles, zoom+1 up to zoom level 15 and zoom-2 up to zoom level 4".

@miguel-porto I'm so cunning: I knew I could persuade you by mentioning a map of Portugal ;)

About when to pre-cache, the zoom pinch gesture seems appropriate - at least as an option.

Fortunately I now fully experience the not-that-slight pause on my own app with the latest osmdroid release.
That gives me pause examples and energy to solve this issue.

Coded in #975, and merged.


add buildTargetJavacOptions for bsp

implement buildTargetJavacOptions, work around workspaceReload, fix #1022

I have a similar change on my laptop but that didn't seems to fix it. I'm still investigating.

So I pushed in https://github.com/lihaoyi/mill/pull/1025/files that thing is ScalaModule is also a JavaModule so I'm not sur what's going to happen.

close #1025


No email content displayed

No content in emails at all
they poll as expected and I will get read/unread counts along with the usual titles and headers with the two line preview (as set in the options)
but when openning the email to read - there is nothing at all present.
Checked this with basic text emails, with/without images, with/without attachments etc but nothing shows.
This has only just started to happen recently and the play store reports that I am using a "test version" so am unable to write a review.
Up until this started happening I've always used this app and have got others to use it instead of their normal applications.
The emails are fine using my desktop pc and through other apps but k9 just isnt having any of it, lol
Also tried to bump up the max fetch size to see if that was the issue and also cleared cache out as suggested by a few others, all to no avail.
running latest updated version on android version 5.1.1 samsung s5 trying to access my googlemail account

Same issue for an e-mail I got today.
I got a stack trace when I hooked up ADB. I've created a gist with the email and stack trace. https://gist.github.com/philipwhiuk/43f98fff2260c67f5f68decd6906143c
Not possible to know whether it's the same cause.
We could choose handle errors in a way that allows us to get the reason why the message wasn't parsed (e.g. by adding a message to K9mail-errors folder) or we could just fix it for the cases we get stack traces for.

that email seems to violate the unique boundary property

it happens with nigh on every single email I get now though with a certain amount of annoyance and irony the email I got telling me I had a reply here seems to open just fine.
Is there any info I could provide to help me get a bit further in the process of getting it fixed as its a massive inconvenience not being able to read emails on my phone at the minute

It would be very helpful if you could provide a debug log, because without that we are flying blind here. The relevant bits will look something like the stack trace @philipwhiuk has in the gist he linked above
On a related note, if you use K-9 productively, it might not be a wise decision to join the tester group. It's cool if you help us figure out bugs so they never make it into an actual release, but if it gets to a point where it's detrimental to your experience you might want to reconsider.

to be completely honest, I hadnt even realised I had a test version of k9 installed and not sure how/why I installed it either (I've used k9 for ages now and certain I hadnt installed anything but the regular playstore version) - it has updated itself recently and was looking to roll it back to a useable state but thought I'd check it out first and had never experienced problems up until recently.
I can certainly post up a debug log, is there anything in particular you would suggest I should do before the log to generate a desired outcome such as checking for new mail, sending myself a test email, openning mail that does "open" etc?

just capture a log right after you opened a mail which shows up blank. hopefully your log contains a stacktrace of the cause then

does it with new and old and I've just gone to email myself the log screenshots from catlog (was having issues with logcat and I have root so tried catlog) and it threw back an error there as well, lol,
This is the error from trying to send an email to myself (I copied to clipboard text using airdroid and just dumped it into a notepad file, there wasnt any personal info there as far as I could see quickly)...
K9error.txt

So that issue is from #1164 I think it should be fixed in the next beta?

Doesn't only happen with "large" emails though bit for now I have had to begrudgingly revert back to the "normal" play store version of the app as I rely on access to my emails too much

Regarding the non-unique boundary issue, the problem arises in LocalFolder: When a message part is saved, its boundary/preamble/postamble fields are only filled if the body is actually a Multipart, which may not be the case regardless of declared mimetype of the part. When the message is recalled though, a MimeMultipart body is always constructed if the mimeType is multipart, and that fails if the boundary field is not set.
This error is triggered by non-unique boundaries: A message such as multipart/mixed(multipart/alternative(text/plain, text/html)) with non-unique boundaries might get parsed by an imap server as multipart/mixed(multipart/alternative,text/plain,text/html) which results in a multipart/alternative MimeBodyPart object with a null body.
Making sure that all MimeBodyPart objects always has a MimeMultipart body seems difficult, I would propose adding a check if the mimeType is multipart/* to missingPartToContentValues, and add a generated a boundary if that's the case.

on current master, email and stacktrace:
From: "test account"<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Message-Id: <test3-E2b0uAD@sifjsdlf>
Subject: second and empty content-type header
Content-Type:
Content-Type: multipart/related; type="text/html"; boundary="__part_M1HhxI3trB/gNWWyh03Y5w95"
Date: Fri, 15 Apr 2016 13:00:30 +0200

Hello!

java.lang.IllegalArgumentException: boundary can't be null
at com.fsck.k9.mail.internet.MimeMultipart.<init>(MimeMultipart.java:28)
at com.fsck.k9.mail.internet.MimeMessage$MimeMessageBuilder.startMultipart(MimeMessage.java:529)
at org.apache.james.mime4j.parser.MimeStreamParser.parse(MimeStreamParser.java:171)
at com.fsck.k9.mail.internet.MimeMessage.parse(MimeMessage.java:113)
at com.fsck.k9.mail.internet.MimeMessage.parse(MimeMessage.java:83)
at com.fsck.k9.mail.store.imap.ImapFolder.fetch(ImapFolder.java:760)
at com.fsck.k9.controller.MessagingController.fetchUnsyncedMessages(MessagingController.java:1328)
at com.fsck.k9.controller.MessagingController.downloadMessages(MessagingController.java:1176)
at com.fsck.k9.controller.MessagingController.synchronizeMailboxSynchronous(MessagingController.java:966)
at com.fsck.k9.controller.MessagingController$8.run(MessagingController.java:749)
at com.fsck.k9.controller.MessagingController.run(MessagingController.java:250)
at java.lang.Thread.run(Thread.java:818)

For an email like this (note the duplicate content-type header), the mime parser gives us a startMultipart() callback, but the content-type header we parse does not have a boundary.
The standards violation we need to deal with here is lack of a boundary parameter. We can't do much about a malformed mail like this I guess, but at least we shouldn't crash.
More generally at the moment, MimeMultipart crashes if it gets handed a null boundary parameter, so the handling must happen at every calling site. This is currently handled in none of the three calling sites (mimeparser, decryptstreamparser, loadpart), so we should probably find a common strategy for this.

We should make sure that our code comes to the same conclusion as the MIME4J parser when it comes to the Content-Type header. That way there's no disconnect between interpretation of the headers and message parts in our message structure.

That's not a call we can make when we get the structure of an email from the imap server, so there is at least one way we can get parts that isn't mime4j-dependent

I looked at this in some more detail, please disregardmy previous statement.
So, the main handling for this is in org.apache.james.mime4j.stream.FallbackBodyDescriptorParser which during parsing is handed all field values and remembers the relevant parts of content-* fields. A content-type with value multipart/* is only regarded as valid if it has a boundary value and otherwise disregarded, if more than one content-type header occurs the last one is preferred.
What we do in getContentType() is return the first header for content-type. This is a particular problem in MimeMessageBuilder.startMultipart() which is called to parse ImapMessage content from ImapFolder and similar places. Now in this particular position, we could simply use the values from the BodyDescriptor parameter we get instead of the ones from the message, and that would probably even be the more correct solution there. However it still leaves the getContentType() problem unsolved.
The simple solution would be to pass all content-type headers to FallbackBodyDescriptorParser, and use the content-type value from there. BUT unfortunately, that class is package-local, which makes this a real pain...

Should be fixed in master.


autoPatchelfHook fails with libmpi.dbg

The debug symbols in libmpi.so for Intel MPI are distributed in a separate file libmpi.dbg with an (wrong?) ELF header. For what I can tell, the auto-patchelf hook finds any file and attempts to determine if it has a valid executable ELF header.
The .dbg file has an incorrect header for readelf:
$ readelf -h libmpi.dbg
ELF Header:
  Magic:   7f 45 4c 46 02 01 01 00 00 00 00 00 00 00 00 00
  Class:                             ELF64
  Data:                              2's complement, little endian
  Version:                           1 (current)
  OS/ABI:                            UNIX - System V
  ABI Version:                       0
  Type:                              DYN (Shared object file)
  Machine:                           Advanced Micro Devices X86-64
  Version:                           0x1
  Entry point address:               0x3ee40
  Start of program headers:          64 (bytes into file)
  Start of section headers:          252744 (bytes into file)
  Flags:                             0x0
  Size of this header:               64 (bytes)
  Size of program headers:           56 (bytes)
  Number of program headers:         8
  Size of section headers:           64 (bytes)
  Number of section headers:         30
  Section header string table index: 29
readelf: Error: the dynamic segment offset + size exceeds the size of the file

Also the patchelf attempt fails:
...
searching for dependencies of /nix/store/br931h7y37rl4kshm5nhgqixk6303ii7-intel-mpi-2019.7.217/lib/libmpi.dbg
strange: no string table
builder for '/nix/store/ca1l0r1gzsj4jb84zc6dhgs4kqfgqxm4-intel-mpi-2019.7.217.drv' failed with exit code 1
error: build of '/nix/store/ca1l0r1gzsj4jb84zc6dhgs4kqfgqxm4-intel-mpi-2019.7.217.drv' failed

I was looking for a way to exclude this file from autoPatchelfHook. Can we just patch files with the executable bit set adding -executable to the find filters? So I could easily chmod -x libmpi.dbg to exclude it.
diff --git a/pkgs/build-support/setup-hooks/auto-patchelf.sh b/pkgs/build-support/setup-hooks/auto-patchelf.sh
index 72970623ed7..7a799d23fcd 100644
--- a/pkgs/build-support/setup-hooks/auto-patchelf.sh
+++ b/pkgs/build-support/setup-hooks/auto-patchelf.sh
@@ -216,7 +216,7 @@ autoPatchelf() {
           [ -n "$(echo "$segmentHeaders" | grep "^ *INTERP\\>")" ] || continue
       fi
       autoPatchelfFile "$file"
-    done < <(find "$@" ${norecurse:+-maxdepth 1} -type f -print0)
+    done < <(find "$@" ${norecurse:+-maxdepth 1} -type f -executable -print0)
 }

 # XXX: This should ultimately use fixupOutputHooks but we currently don't have

I ran into the same issue with debug files while attempting to package julia-bin. Rather than checking if the file is executable (in my case they are), which would require manually chmod'ing, could we do something along of the lines of:

Allow specifying a list of files for autoPatchElf to ignore.
And/or rather than exiting on error, print a warning and continue.

For (2), this change was sufficient:
--- a/pkgs/build-support/setup-hooks/auto-patchelf.sh
+++ b/pkgs/build-support/setup-hooks/auto-patchelf.sh
@@ -189,6 +189,16 @@ addAutoPatchelfSearchPath() {
             \( -name '*.so' -o -name '*.so.*' \) -print0)
 }

+# TODO need to first check that the DYNAMIC section exists before
+# extracting the size.
+hasEmptyDynamic() {
+    file="$1"
+    hex='0[xX][0-9a-fA-F]\{16\}'
+    line1='DYNAMIC[ \t0-9A-z]*'
+    dynsize="$(readelf -l "$file" | grep "$line1" -A1 | grep -v "$line1" | grep -o "$hex" | head -n 1)"
+    [ "$dynsize" = "0x0000000000000000" ]
+}
+
 autoPatchelf() {
     local norecurse=

@@ -217,6 +227,7 @@ autoPatchelf() {

     while IFS= read -r -d $'\0' file; do
       isELF "$file" || continue
+      hasEmptyDynamic "$file" && continue
       segmentHeaders="$(LANG=C $READELF -l "$file")"
       # Skip if the ELF file doesn't have segment headers (eg. object files).
       # not using grep -q, because it can cause Broken pipe

Which just parses the output of readelf to get the size of the DYNAMIC section. If it's 0/empty, then skip, a) because there's nothing to do, but b) because patchelf can't handle empty DYNAMIC sections.
I think you could similarly get rid of the "strange: no string table" messages from auto-patchelf.sh and elsewhere from calling  patchelf on ELF's with empty DYNAMIC sections.
Maybe a cleaner option is for patchelf to exit silently without error if the dynamic section is empty (not sure if that's a feature or bug).
Tagging @DavHau since you seem to familiar with this code :).

I also have problem with autopatchelf choking on debug files
https://github.com/NixOS/patchelf/issues/373
https://github.com/yvt/nixpkgs/commit/49be2c828875522289e2250307e85c48625bccc7#commitcomment-70690616


[FEATURE] Discord Integration with Docbot

Is your feature request related to a problem?
Open up a way to interact with Discord client using python. Be able to send and read/receive messages in appointed channel.
What solution would you like?
Write functions in integration.py that allow receiving and sending messages using discord.py API.

I would like to take this issue

I would also like to work on this issue

I would like to work on this issue with my colleague, Yash Sinhal!

Can somebody assign this issue?

@aryanv175 Hey apologies I missed the assignment on this. This got closed with #54


Unable to refresh google oauth2 token after update python social auth to 0.2.1

Using Python social auth in django project and everything was working fine.
I have just updated python social auth to 0.2.1 and I am getting error with load_strategy when trying to refresh google oauth2 token.
Until now I was using:
strategy = load_strategy(backend='google-oauth2')
user = UserSocialAuth.objects.get(uid=..., provider="google-oauth2")
refresh_token(strategy=strategy, redirect_uri='http://.....')

Now I am getting this error:
TypeError: load_strategy() got an unexpected keyword argument 'backend'

Any kind of help will be appreciated.

It should be invoked like this:
from social.apps.django_app.utils import load_strategy

user = User.objects.get(pk=...)
social = user.social_auth.get(provider='google-oauth2')
social.refresh_token(load_strategy())


fix(vtable): inside getCellValue method of ListTable

inside getCellValue method of ListTable couldn't find out record when rerender groups

[中文版模板 / Chinese template]
🤔 This is a ...

[ ] New feature
[x] Bug fix
[ ] TypeScript definition update
[ ] Bundle size optimization
[ ] Performance optimization
[ ] Enhancement feature
[ ] Refactoring
[ ] Update dependency
[ ] Code style optimization
[ ] Test Case
[ ] Branch merge
[ ] Site / documentation update
[ ] Demo update
[ ] Workflow
[ ] Chore
[ ] Release
[ ] Other (about what?)

🔗 Related issue link

💡 Background and solution

Background
demo link: https://visactor.io/vtable/demo/performance/100W
set groupBy ['sex']
expand and collapse last group and then appears an error

solution
record?.vtableMerge
📝 Changelog

Language
Changelog

🇺🇸 English

🇨🇳 Chinese

☑ Self-Check before Merge
⚠ Please check all items below before requesting a reviewing. ⚠

[ ] Doc is updated/provided or not needed
[ ] Demo is updated/provided or not needed
[ ] TypeScript definition is updated/provided or not needed
[ ] Changelog is provided or not needed

🚀 Summary
copilot:summary
🔍 Walkthrough
copilot:walkthrough

请加入飞书群 领取开源共建的奖品 @onlyreadcodes


Use sbt-github-actions

Supersedes and closes #3825
Can't test publishing in advance (as usual).
It might be a good idea to (eventually) put more of this into sbt-http4s-org for code reuse.

MIMA is disabled as it fails right now, and I don't know why: logs
Also TODO: adapt mergify

The fail-fast behavior is not optimal, will create a PR to sbt-github-actions to allow to disable it.
I am confused about the last build error:
sbt.librarymanagement.ResolveException: Error downloading org.http4s:http4s-core_2.12:1.0.0-M6+9-6595f52a-SNAPSHOT

Wait, I should have not used the GitHub UI branch retarget feature... Will close and reopen


Unable to find definition from index file while on the same folder

Issue Type: Bug
VSCode is not able to navigate to the definition from an index file to a file that is on the same folder.
Here is a little but simple example:
index.js
import Conversation from './Conversation.component';

export default Conversation;

Folder structure
Conversation
 \_ index.js
 |_ Conversation.component.jsx

When you try to go to the definition of the only import on the index file it just goes to the index itself.

Also does this reproduce in the latest VS Code insiders build with all extensions disabled?

I'm running it on any place of the codebase, same result.
I am not running the insiders build, but I cant try with all the extensions disabled to see if that makes a difference.
If it does, I may try to provide a small reproduction repo

It still happens with extensions disabled.
However, I tried to create a small reproduction and it works fine on my example. Is there any way I can debug how this happens in some repos? Some console output explaining the steps tried to find the definition that failed?

Did you have a change to test using the insiders build?
You generally cannot debug this type of issue on your own. The best option is for you to provide a small example project so we can debug it, or you could try sending us detailed logs. Are either of those possible?

As I said I tried to create a small reproduction without success, it only happens on my big project. So I think the best option will be to use the insiders build and provide detailed logs. How can I fetch such logs? What should they include?

To collect the TS Server log:

Set "typescript.tsserver.log": "verbose",
Restart VS Code and reproduce the problem
In VS Code with a ts file open, run the TypeScript: Open TS Server log command
This should open a large log file called tsserver.log

Look through that log file for errors or stack traces. If you can share the log, I can take a look to see if anything stands out

⚠Warning: The TypeScript log may include information from your workspace, including file paths and source code. If you have any concerns about posting this publicly on Github, just let me know and we can arrange something else. On our side, we only use these logs to investigate issues like this

Sorry for the late response.
It is still happening, but on a different fashion. Instead of redirecting you again to the same index file it simple does nothing.
I have the (huge) log. I'm not worried about paths, but I don't want to expose src if there is any.
Is there a way that I can send it to you?

Well, I can always remove it later.
tsserver.log

Please try upgrading your workspace to use typescript@next by installing this extension
Do you still see the issue in that case?


Update README.md

typo
resolves #

Description

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

@cla-bot check

Closing and re-opening to re-trigger CI checks.

Fixed within https://github.com/dbt-labs/dbt-core/pull/9928


Remove version from clang-format

Closes #295

It seems like at least, I can't merge it because the errors on some checks

Thank you @bmagyar and @destogl !


Resolve all modules using Node resolution algorithm, but embed only specific ones

Hello!
Thank you for this great plugin!
I'm using the following imports in my project:
import {foo} from './foo';, which should be resolved to: import {foo} from './foo/index.js'; according to the Node resolution algorithm.
But, at the same time, I have some modules in node_modules, which I want to embed to my bundle and the ones I want to leave as external.
However, if I set the only option to something like: only: ['tslib'], the Rollup stops resolving my local imports like import {foo} from './foo';. And if I omit the only option, the modules are resolved correctly, but it embeds everything it could find to the final bundle (which I don't want to do).
Is there a way to use node resolution algorithm for all imports, but to embed only specific modules to the bundle?
I think these are two separate processes (module resolution and module embedding), and one option shouldn't prevent another from working.
It's OK (and desired) to embed all imports to a bundle, when you build a final application, however, when it comes to library building you need a more granular control of what to actually embed and what to leave as external dependency.
The only workaround I can think of so far is to run Rollup two times: one to resolve all local dependencies and leave everything from node_modules as external and then to embed some external dependencies to the bundle. But I would really want to avoid such approach.
Could you elaborate on this issue please? Thanks!

I've implemented this plugin as a solution to this problem:
https://github.com/slavafomin/rollup-plugin-node-resolve-next
I will be glad for any input on it.

Isn't external rollup option what you need?

I think it can be closed.


Simplify parameter names of all class methods.

We must give all class methods shorter names.
Use this guideline:

Do not use param_ prefix.
Use ent for entity
Use attr for attribute
Use rel for relation
Use type
Use inst for instance
Use ptr for pointer

This has been implemented: https://github.com/inexorgame/entity-system/commit/7e8e98b63296d65b68c26192a9747bb1e82da302.


Add CAN payload len on first frame byte

Related:

https://github.com/eProsima/Micro-XRCE-DDS-Client/pull/298
https://github.com/micro-ROS/micro_ros_renesas2estudio_component/pull/23
https://github.com/micro-ROS/micro_ros_renesas_testbench/pull/61
https://github.com/micro-ROS/micro_ros_renesas_demos/pull/26

Test CI:

https://github.com/eProsima/Micro-XRCE-DDS/pull/122

Build status:

Linux
Windows

Build status:

Linux
Windows

Build status:

Linux
Windows


https POST request has blocked by CORS policy

Hi,
I have an issue with https POST request using this configuarathis nelmio_cor yaml configuaration
    defaults:
        origin_regex: true
        allow_methods: [ 'GET', 'OPTIONS', 'POST', 'PUT', 'PATCH', 'DELETE' ]
        allow_headers: [ 'Content-Type', 'Authorization' ]
        expose_headers: ['Link']
        max_age: 3600
    paths:
        '^/api/':
            allow_methods: ['POST', 'PUT', 'GET', 'DELETE']
            max_age: 3600

Also it's working on GET request using same nelmio configuration. But i'm trying to add allow origin * on this nelmeio yaml the POST request working find but GET Request got multiple header issue.
    defaults:
        origin_regex: true
        allow_origin: ['%env(CORS_ALLOW_ORIGIN)%']
        allow_methods: [ 'GET', 'OPTIONS', 'POST', 'PUT', 'PATCH', 'DELETE' ]
        allow_headers: [ 'Content-Type', 'Authorization' ]
        expose_headers: ['Link']
        max_age: 3600
    paths:
        '^/api/':
            allow_methods: ['POST', 'PUT', 'GET', 'DELETE']
            max_age: 3600

Any solution?
Thanks in advance.

Are you sure your address localhost:8899 is allowed to be access by any address? localhost:3000 and localhost:8899 are different though.

Hi @fd6130
My frontend host is localhost:3000 and backend symfony running on docker host is localhost:8899.

Hi @fd6130 Yes it's work when using the GET request from localhost:8899 but POST is not working.
My frontend host is localhost:3000 and backend symfony running on docker host is localhost:8899.

Can you check if the POST request is working without CORS setup? Sometime CORS occurs because the code got problem.

Hi @fd6130 Yes it's work when using the GET request from localhost:8899 but POST is not working.
My frontend host is localhost:3000 and backend symfony running on docker host is localhost:8899.

Can you check if the POST request is working without CORS setup? Sometime CORS occurs because the code got problem.

No it's not working without CORS setup

Hi @fd6130 Yes it's work when using the GET request from localhost:8899 but POST is not working.
My frontend host is localhost:3000 and backend symfony running on docker host is localhost:8899.

Can you check if the POST request is working without CORS setup? Sometime CORS occurs because the code got problem.

No it's not working without CORS setup

What kind of error you get from the POST request without CORS? And without CORS you should be able to access any resources from your backend server.

Hi @fd6130 Yes it's work when using the GET request from localhost:8899 but POST is not working.
My frontend host is localhost:3000 and backend symfony running on docker host is localhost:8899.

Can you check if the POST request is working without CORS setup? Sometime CORS occurs because the code got problem.

No it's not working without CORS setup

Are you using Symfony 5?

How about change your path to this one  (simply copy paste from default actually) ?
paths:
        '^/api/':
            origin_regex: true
            allow_origin: ['*']
            allow_methods: [ 'GET', 'POST', 'PUT', 'DELETE' ]
            allow_headers: [ 'Content-Type', 'Authorization' ]
            expose_headers: ['Link']
            max_age: 3600

It's because you used * as origin. Explicitly use the name of the domain instead, (or a regex if you want to match several domains). See https://stackoverflow.com/a/19744754/1352334 for some explanations.
Duplicate of https://github.com/api-platform/core/issues/3601.

Sorry to reopen "Dunglas", but this error is different. The CORS with * occurs due to access denial, this refers to the possibility of making the PUT / PATCH request for the API, the other methods work correctly. Even in the old code (which I built directly in Symfony with the integration) it works with the same code. So in my opinion it was a change or something that is done differently from the docker code, in relation to the integrated in Symfony.
And in my case the regex is this:
CORS_ALLOW_ORIGIN=^https?://(localhost|127\.0\.0\.1|MY_ADDRESS)(:[0-9]+)?$


Update to Angular v18

Description
Update the following packages and app to Angular to major version 18:

https://www.npmjs.com/package/@folkehelseinstituttet/angular-components
https://www.npmjs.com/package/@folkehelseinstituttet/angular-highcharts
the demo app (https://designsystem.fhi.no)

Acceptance Criteria
Check that Angular is v18, and that all other dependencies are okay, in these three files:

../Fhi.Frontend.Demo/projects/fhi-angular-components/package.json
../Fhi.Frontend.Demo/projects/fhi-angular-highcharts/package.json
../Fhi.Frontend.Demo/package.json

Management has granted us some help with this one; @robinzaal will help us 🙂

Reopened this since the demo app isn't updated yet.


[tests] Increase the wait duration for sink topic being created at PulsarWorkerAssignmentTest

Motivation
Currently there is a wait strategy for sink topic being created. However the value is a bit small,
so if the jenkins becomes slow, the test will fail.
Changes
Increase the retry count which will increase the wait duration.

run java8 tests

run java8 tests

run java8 tests


When creating multiple built-in scripts without saving the scene, only one can be opened

Godot version:

758bccf / 3.3
Issue description:

Steps to reproduce:

Have a scene with at least 2 nodes
Add built-in script to both of them
Try opening both in the script editor

I wasnt able to reproduce this.

You need to open the scripts without saving the scene.

Yes, I made sure not to save the scene - here's a video
https://user-images.githubusercontent.com/76781125/118267903-b9986600-b4cd-11eb-9dc1-446d91ec9773.mp4

You didn't enable the "Built-in Script" checkbox. It only happens with built-in scripts.

@KoBeWi see https://github.com/godotengine/godot/issues/5535 , this can be closed

I don't see how is that issue related.

Isn't this issue that you can't edit built-in scripts without saving the containing scene?

It is, but the other issue is about not being able to save built-in script when another scene is opened. It's like the opposite issue...

understood


⚠ SceneTime has degraded performance

In dd0422f, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 18059 ms

Resolved: SceneTime performance has improved in ae156a1 after 5 minutes.


Unable to Init smart contract via factory

I have problems with calling factory function on the marketplace contract.
I though first time that I did not set contract hash but the error is still there.
Built with latest main at hash 0xedf9dde8b7c52a23ef3972268217a92f3c356707388aaa68a3952e31973797c3
Failed Txs
https://shibuya.subscan.io/account/Z7rYZFFCFt9B2cv692TSWnxTKM1CYnoukBhxUd39ajsYWTe
Example of call:

The supplied hash of RMRK smart contract
0xd22f1aabf3d04bfba1ef5d0b5a5e1fa4764df7224afd1ee1422a8344a54b490f

At the moment you can't create RMRK through factory. Only Shiden34 (https://github.com/swanky-dapps/nft)

@Maar-io what is the desired outcome?


remove peerDependencies in package.json

In react-native 0.44, this will cause "UNMET PEER DEPENDENCY" warning.

Thanks for this.


Fixed Showing no validation errors when form opens

Fixes
#1001 #https://github.com/opensrp/fhircore/issues/873
Description
The validation will not be triggered if the answers are empty or User hasn't made interaction with the input field yet whenever form opens. However the validation will be triggered if you open questionnaire with pre-defined answers since the condition checks if the answers are there or not.
Type
Choose one: ( Bug fix)
Screenshots (if applicable)
https://user-images.githubusercontent.com/35099184/152525877-d7d52896-3063-4925-903e-159a34d86a32.mp4
Checklist

[x] I have read and acknowledged the Code of conduct
[x] I have read the Contributing page
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[x] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

@santosh-pingle can you please leave some comments

@santosh-pingle can you please leave some comments
sure.

Also, can you please update the description for updated solution?

do we need demo/src/main/assets/new-patient-registration-paginated.json change ?

restored changes with the master

Also, can you please update the description for updated solution?

done.

looks good to me, thanks!
Please code owner review once, thanks!

looks good to me, thanks! Please code owner review once, thanks!

cc @f-odhiambo

is this ready for another review? please reply to individual comments. thanks!

is this ready for another review? please reply to individual comments. thanks!

PR made ready for review. thanks

@jingtang10 , would you be able to review this PR?

looks pretty good - almost ready to merge
last thing - just add some test cases where you have changed the existing test cases by adding modified = true. just make sure that if modify is false, we're not showing validation errors. that's the whole point of this PR now.

added the unit test cases for modify = false. Please review. Thanks

looks pretty good - almost ready to merge
last thing - just add some test cases where you have changed the existing test cases by adding modified = true. just make sure that if modify is false, we're not showing validation errors. that's the whole point of this PR now.

added the unit test cases for modify = false. Please review. Thanks

PR gateways passed.
cc @jingtang10

A 503 status code has been received, will attempt to retry the upload
Begin Diagnostic HTTP information
Status Code: 503
Status Message: Service Unavailable


[PrintAsObjC] Do not output invalid decls when allowing errors

We still want the ObjectiveC header when allowing errors, but make sure
to skip printing out any invalid declarations that may be caused because
of those errors.
In particular, duplicate declarations were causing a crash when sorting
declarations where various assumptions would end up casting a
non-ExtensionDecl to an ExtensionDecl.
Resolves rdar://78023656

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - e38910f9c140289eeac229736f7ed9d003748157

@swift-ci please test

@swift-ci please smoke test


[iMessage 2FA] Codes with hypen not parsed correctly

Extension
https://www.raycast.com/yuercl/imessage-2fa
Description
I often get a 2-factor code in the format of 123-456 but the extension only suggests "123" as my code instead of "123456" or "123-456"
Who will benefit from this feature?
Everyone
Anything else?
No response

That seems like a bug, since there is code for handling those codes. I think it is just an issue with matching order.

Ah, yes I see. For better context, here is the a full example text:
Your Raise verification code is 299-074.
Raise will never contact you for this code. If you did not request this code, please contact Raise Member Services.

I have tested your example with my change and it works as expected - returning 299074. Now we just have to wait until the pull request has been reviewed :-)


[gRPC-JSON transcoder] Support configurable request buffer limit

Title: Support configurable request buffer limit in gRPC-JSON transcoder
Description:
Today, as far as I can tell, the gRPC-JSON transcoder can handle a maximum request size of 64 KB, due to flow control limitations limiting the amount of data a filter can buffer.  The requested feature is to add a configurable buffer limit, similar to the one in the buffer filter.
This will allow the filter to support larger requests when necessary.

From JSON to proto message transcoding, I don't think the filter needs to buffer up the whole request body,  it can do "field-by-field" while the data is coming in.  It may need to buffer up a whole field.  So the current limit may limit the field length.
My only concern with the bigger limits is:  will it break the whole flow-control if this filter buffer up more data than flow-control allowed.
@alyssawilk @nareddyt ?

I think for my use case, there are particular fields that might be larger than 64 KB which is why I'm requesting this feature.
If the flow control allowed limit is available via some API, then the filter could do a minimum of this limit (if specified) and the flow control limit, right?

the gRPC-JSON transcoder can handle a maximum request size of 64 KB, due to flow control limitations limiting the amount of data a filter can buffer

Flow control should not affect the buffer limit in the gRPC JSON transcoder. There are two separate buffer limits the transcoder checks:

for requests, it gets the limit from the per_connection_buffer_limit_bytes field in the Listener config. Default is 1 MiB, but you can change your listener config to have any value.
for responses, it gets the limit from the per_connection_buffer_limit_bytes field in each cluster's config

Implementation of this is located here: https://github.com/envoyproxy/envoy/blob/70a5f295da789610b9d078015f43e39c6a76d4b6/source/extensions/filters/http/grpc_json_transcoder/json_transcoder_filter.cc#L892-L926
We have multiple tests that overwrite the limits to test under/over limit code paths: https://github.com/envoyproxy/envoy/blob/70a5f295da789610b9d078015f43e39c6a76d4b6/test/extensions/filters/http/grpc_json_transcoder/grpc_json_transcoder_integration_test.cc#L1143
You can follow the code in this helper function. The tests above just set the buffer limits in listener and cluster API, which you can do as an Envoy user. https://github.com/envoyproxy/envoy/blob/70a5f295da789610b9d078015f43e39c6a76d4b6/test/config/utility.cc#L925

Thanks for the pointers!
We'll try the listener configurations and validate whether those will allow us to overcome the limits.


no copying gdx files during install and setting of model directory

This PR changes the setup and model-directory config such that gdx files are no longer copied back and forth.

@gidden, does it make sense to remove the logging here?

comments in line. remove which logging?

@danielhuppmann can you rebase this one? @khaeru has fixed up the tests so they all should now pass with recent ixmp master

rebased

re @gidden and logging: recursive_copy() in cli.py writes a line to logger for every file copied. Seems excessive (even when not copying gdx files)...

@khaeru gidden, seems that Python 2.7 tests were failing with pathlib and when I started digging, I got confused between pathlib and pathlib2... So I just went back to the previous file extension approach for now.

thanks for the comments, @gidden - should be good to go!

Looks great @danielhuppmann thanks!


Use circleci dynamic configuration

Description
use circle.ci dynamic configuration capabilities to determine:

which maven modules should be built
if UIs should be built

📚  View the storybook of this branch here

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of APIM master. (Same login and password as APIM master)

🎉 Happy to finally close this one (and #2493) !
We are now using circleci SDK to generate our configuration file.
See -> https://gravitee.atlassian.net/browse/APIM-2635


Thorchain LP: improve handling of cached tx fee

Overview
To avoid flickering load states, a useState variable was added to cache the last successful estimated tx fee. Debug this flow to ensure the necessary edge cases are handled for invalidation so you don't have estimated tx fees that are no longer applicable.
References and additional details
AddLiquidityInput.tsx
Acceptance Criteria

poolAssetTxFeeCrypto is properly cached and invalidated

Need By Date
No response
Screenshots/Mockups
No response
Estimated effort
No response

https://github.com/shapeshift/web/pull/6700/commits/cb8716371c2787bce454e1ca0d582277f2e42911


[Event Request] report 6653 "Combine Return Receipts"

Hi,
We have base report 295 "Combine Shipments" in which we have procedure ShowResults, similar to that we have report 6653 "Combine Return Receipts" where the
procedure for ShowResult is different. In report 6653 "Combine Return Receipts" we don't have the boolean for HideDialog in the event
"OnReturnReceiptHeaderOnAfterFinalizeSalesInvHeader" and code is also different in ShowResult function.
In report 6653 "Combine Return Receipts" I have changed the code as per report 295 "Combine Shipments", can you update the changes-->
local procedure ShowResult()
begin
OnBeforeShowResult(SalesHeader, NoOfSalesInvErrors, PostInv);
    if SalesHeader."No." <> '' then begin // Not the first time
        FinalizeSalesInvHeader();

//ALANSH <<
OnReturnReceiptHeaderOnAfterFinalizeSalesInvHeader(SalesHeader, NoOfSalesInvErrors, PostInv, HideDialog);
if (NoOfSalesInvErrors = 0) and not HideDialog then begin
Message(Text010, NoOfSalesInv)
else
if not HideDialog then
Message(Text007, NoOfSalesInvErrors)
end else
if not HideDialog then
Message(Text008);
end;
//ALANSH >>
//ALANSH<<
procedure SetHideDialog(NewHideDialog: Boolean)
begin
HideDialog := NewHideDialog;
end;
var
HideDialog : Boolean;
[IntegrationEvent(false, false)]
local procedure OnReturnReceiptHeaderOnAfterFinalizeSalesInvHeader(var SalesHeader: Record "Sales Header"; var NoOfSalesCrMemoErrors: Integer; PostInv: Boolean; var HideDialog: Boolean)
begin
end;
//ALANSH>>

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 51294.


Add support for custom callback URL in meshery server

Description
This PR fixes https://github.com/layer5labs/meshery-extensions-packages/issues/8
This package adds support for an environmental variable MESHERY_SERVER_CALLBACK_URL which will allow the user to customize the provider callback URL (eg if meshery is running behind  a proxy).
The support has been added in two places:

Meshery Server if gets MESHERY_SERVER_CALLBACK_URL env var then it will use this as callback URL. The default is `{MESHERY_HOST}/api/user/token
mesheryctl also supports the same environment variable in the following commands:

MESHERY_SERVER_CALLBACK_URL=http://something.com/meshery/api/user/token mesheryctl system start this will modify the k8s manifest (if context platform is k8s) or compose file (if context platform is docker) to pass this env var to the meshery server.
mesheryctl system reset will not preserve the callback URL, however if MESHERY_SERVER_CALLBACK_URL is provided here then that will be respected.
mesheryctl system restart will respect the callback URL set in the start command.

Notes for Reviewers
New utility functions related to YAML transformation and nested map transformation are added in this PR along with unit tests. These utilities can be used in the mesheryctl in general as a lot of mesheryctl performs only YAML transformations making the code too verbose (whenever the maps are nested).
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #3544 (4b803c6) into master (b04208b) will decrease coverage by 5.65%.
The diff coverage is 50.40%.

@@            Coverage Diff             @@
##           master    #3544      +/-   ##
==========================================
- Coverage   17.43%   11.78%   -5.66%
==========================================
  Files          37       37
  Lines        3263     3386     +123
==========================================
- Hits          569      399     -170
- Misses       2551     2898     +347
+ Partials      143       89      -54

Flag
Coverage Δ

gointegrationtests
?

unittests
11.78% <50.40%> (+1.48%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/start.go
1.18% <0.00%> (-52.60%)
:arrow_down:

mesheryctl/pkg/utils/platform.go
0.00% <0.00%> (ø)

mesheryctl/pkg/utils/helpers.go
17.69% <57.27%> (+17.69%)
:arrow_up:

mesheryctl/internal/cli/root/system/stop.go
0.93% <0.00%> (-39.26%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.49% <0.00%> (-22.10%)
:arrow_down:

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 b04208b...4b803c6. Read the comment docs.

Viper doesn’t cover the yaml transformation use cases, huh?

Viper doesn’t cover the yaml transformation use cases, huh?

No. Viper can "get" and "set" only the root level keys, not the nested keys. But a lot mesheryctl actually does that, hence the code is too verbose.
This new utility function was created to allow mutation at any level in the YAML without manually dealing with go maps.

@utkarsh-pro conflict in start.go - https://github.com/meshery/meshery/pull/3544/conflicts

The latest build failures seem to be because of our upgrade to golang 1.116

The latest build failures seem to be because of our upgrade to golang 1.116

@leecalcote https://github.com/meshery/meshery/pull/3545#issuecomment-884006803

Merging this PR...


sync2.4

What changes were proposed in this pull request?
sync 2.4
(Please fill in changes proposed in this fix)
How was this patch tested?
no
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
no
Please review http://spark.apache.org/contributing.html before opening a pull request.
no

merge 2.4