Write On Start option not working

Describe the bug
When disabling Write On Start on TMPWriter, the writer still automatically starts writing when on Start()
To Reproduce

Create a gameobject with TextMeshProUGUI component and TMPWriter component
Set some text in the TextMeshProUGUI
Disable Write On Start option on TMPWriter
Start game, observe the writer begins writing

Expected behavior
TMPWriter shouldn't begin Writing automatically
I've put a debug log in the StartWriter() method, and the stack trace is:
StartWriter
TMPWriter.StartWriter() at /External /TMPEffects/Runtime/Components/TMPWriter/TMPWriter.cs:203
TMPWriter.OnTextChanged_Late() at /External /TMPEffects/Runtime/Components/TMPWriter/TMPWriter.cs:605
TMPMediator.TextChangedProcedure() at /External /TMPEffects/Runtime/Components/TMPMediator/TMPMediator.cs:345
TMPMediator.OnTextChanged() at /External /TMPEffects/Runtime/Components/TMPMediator/TMPMediator.cs:297
TMPro.FastAction`1<UnityEngine.Object>.Call() at ./Library/PackageCache/com.unity.textmeshpro@3.0.9/Scripts/Runtime/FastAction.cs:73
TMPro_EventManager.ON_TEXT_CHANGED() at ./Library/PackageCache/com.unity.textmeshpro@3.0.9/Scripts/Runtime/TMPro_EventManager.cs:77
TextMeshProUGUI.GenerateTextMesh() at ./Library/PackageCache/com.unity.textmeshpro@3.0.9/Scripts/Runtime/TMPro_UGUI_Private.cs:4485
TextMeshProUGUI.OnPreRenderCanvas() at ./Library/PackageCache/com.unity.textmeshpro@3.0.9/Scripts/Runtime/TMPro_UGUI_Private.cs:1670
TextMeshProUGUI.ForceMeshUpdate() at ./Library/PackageCache/com.unity.textmeshpro@3.0.9/Scripts/Runtime/TextMeshProUGUI.cs:547
TMPMediator.ForceReprocess() at /External /TMPEffects/Runtime/Components/TMPMediator/TMPMediator.cs:93
TMPWriter.OnEnable() at /External /TMPEffects/Runtime/Components/TMPWriter/TMPWriter.cs:451

You also have to disable "WriteOnNewText", since the first text is also considered a new text.
Which is because TextMeshPro invokes the TEXT_CHANGED_EVENT once for every TMP_Text when first enabled.
Let me know if that solved your issue or if something else is going on ✌


SSL lab grade is B

Hi,
SSl labs report: This server supports TLS 1.0 and TLS 1.1. Grade capped to B.
See https://blog.qualys.com/ssllabs/2018/11/19/grade-change-for-tls-1-0-and-tls-1-1-protocols
I think maybe it is time to remove TLS 1.0 and TLS 1.1 support fron nginx config?

Sounds sane: Chrome removed support for it a year ago.


Add untyped helloworld-literal-bytestring script example

Add a new untyped helloworld example that uses bytestring literal of "Hello World!". This is much smaller than the typed parameterised example when serialised.

could you perhaps provide an example of how to build the on-chain transaction (i.e. what datums are needed)?

You just need to provide datum "Hello World!" as bytes. The cli supports this with its string conversion of the "Hello World!" value, or as file with bytes 48656c6c6f20576f726c6421.


will not install with "bundle install --local"

Hi,
for some reason it keeps trying to fetch it from https://github.com/alexdunae/holidays
"bundle install --local
Fetching https://github.com/alexdunae/holidays
error: Failed connect to github.com:443; Connection timed out while accessing https://github.com/alexdunae/holidays/info/refs
fatal: HTTP request failed"
i'm on Ubuntu 12.04
gem -v
1.8.15

This doesn't seem like a holiday gem specific issue. I'm going to go ahead and close this, feel free to post another update if it is still happening and I would be happy to reopen.


Preview image uses a copyrighted image

The preview image on the readme (https://github.com/fekt/discord-giveaway-bot/blob/master/README.md) uses an example Bot account that is using GiveawayBot#2381's copyrighted profile image. This is misleading, as you were not given permission to use this image to represent your project. I am opening this issue as a courtesy, so that you have an opportunity to rectify this yourself.

Will be fixed asap!

The preview image on the readme (https://github.com/fekt/discord-giveaway-bot/blob/master/README.md) uses an example Bot account that is using GiveawayBot#2381's copyrighted profile image. This is misleading, as you were not given permission to use this image to represent your project. I am opening this issue as a courtesy, so that you have an opportunity to rectify this yourself.

The preview image has been changed and I hope everything is resolved now. If not, please message me via email since I'm not that active on GitHub at the time.


baresip and g722.2

Does baresip support g722.2?
How I can play sound via baresip to phone with amr-wb codec?

g722.2 is the same as AMR-WB, which is supported.
This should be documented in the README.md file

Yes. Should be.

Yes. Should be. (I really didn't see g722.2 module on my PC)

https://github.com/alfredh/baresip/wiki/Guidelines-for-reporting-Issues


Adjust to actually try a server address more than once

Noticed this while testing connection retries across a server name with multiple addresses.  I had the following settings
CONFIG proxy.config.http.connect_attempts_max_retries INT 6
CONFIG proxy.config.http.connect_attempts_rr_retries INT 3

And I had microdns set up to replay with addresses <IP_ADDRESS> and <IP_ADDRESS> for the name "foo".
I had a server listening on port 8888 on <IP_ADDRESS> but a iptables DROP rule on port 8888 for <IP_ADDRESS>.
For my query that maps to http://foo:8888, it was being sent to <IP_ADDRESS> first.  In the original code, it would fail <IP_ADDRESS>, mark it down, and then successfully connect to <IP_ADDRESS>.  So it ignored the connect_attempts_rr_retries count of 3.
With this code change, the same query would try and fail connections to <IP_ADDRESS> three times before marking it down and moving onto  <IP_ADDRESS>.
All of the retries were verified by looking at Debug output.
A note about the name "connect_attempts_rr_retries" implies this is the number of additional tries beyond the first try, but that is not how it is coded.  And reviewing the documentation, that is not what is described either.
Related to issue #7290

@rob05c Can you please take a look at this?

@rob05c Can you please take a look at this?

Cherry-picked to v9.0.x branch.

Cherry-picked to v9.1.x branch.


Linker errors with vcpkg package and systems other than Windows

Git commit reference
DPP v10.0.24 from vcpkg baseline 2f56fdad4bcf43c21ec6f952cf7b91c8c43c5c0d
Describe the bug
I wrote a simple bot using DPP and vcpkg, compiled it on a Windows machine with MSVC 2022, and it works like a charm.
Unfortunately, I can't say the same about the build process using WSL2 (Ubuntu 22.04). So, vcpkg integration does work (all dependencies have been downloaded and configured), object files are generated, but a build fails with a bunch of linker errors:
Something about `ld` unable to find dependant libraries

FAILED: discord_target_replier
: && /usr/bin/c++ -O3 -DNDEBUG  CMakeFiles/discord_target_replier.dir/main.cpp.o CMakeFiles/discord_target_replier.dir/Config.cpp.o -o discord_target_replier  vcpkg_installed/x64-linux/lib/libdpp.a && cd /mnt/c/dev/discord_target_replier/cmake-build-linux-release && /usr/bin/cmake -E copy_if_different /mnt/c/dev/discord_target_replier/config.example.json /mnt/c/dev/discord_target_replier/cmake-build-linux-release
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordclient.cpp.o): in function `dpp::discord_client::setup_zlib()':
discordclient.cpp:(.text+0xc42): undefined reference to `inflateInit_'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordclient.cpp.o): in function `dpp::discord_client::end_zlib()':
discordclient.cpp:(.text+0xc8c): undefined reference to `inflateEnd'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordclient.cpp.o): in function `dpp::discord_client::handle_frame(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)':
discordclient.cpp:(.text+0x7659): undefined reference to `inflate'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordvoiceclient.cpp.o): in function `dpp::discord_voice_client::cleanup()':
discordvoiceclient.cpp:(.text+0x109d): undefined reference to `opus_encoder_destroy'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x10b9): undefined reference to `opus_repacketizer_destroy'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordvoiceclient.cpp.o): in function `dpp::discord_voice_client::encode(unsigned char*, unsigned long, unsigned char*, unsigned long&)':
discordvoiceclient.cpp:(.text+0x36dd): undefined reference to `opus_repacketizer_init'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x3717): undefined reference to `opus_repacketizer_cat'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x3754): undefined reference to `opus_encode'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x377c): undefined reference to `opus_strerror'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x38c6): undefined reference to `opus_repacketizer_out'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x391d): undefined reference to `opus_strerror'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x3c0d): undefined reference to `opus_strerror'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordvoiceclient.cpp.o): in function `dpp::discord_voice_client::discord_voice_client(dpp::cluster*, dpp::snowflake, dpp::snowflake, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)':
discordvoiceclient.cpp:(.text+0x4325): undefined reference to `sodium_init'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x4353): undefined reference to `opus_encoder_create'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x436b): undefined reference to `opus_repacketizer_create'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordvoiceclient.cpp.o): in function `dpp::discord_voice_client::voice_courier_loop(dpp::discord_voice_client&, dpp::discord_voice_client::courier_shared_state_t&)':
discordvoiceclient.cpp:(.text+0x53c9): undefined reference to `opus_decode'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x542e): undefined reference to `opus_decode'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordvoiceclient.cpp.o): in function `dpp::discord_voice_client::read_ready()':
discordvoiceclient.cpp:(.text+0x7d3c): undefined reference to `crypto_secretbox_open_easy'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x81a9): undefined reference to `opus_decoder_create'
/usr/bin/ld: discordvoiceclient.cpp:(.text+0x81ed): undefined reference to `opus_decoder_destroy'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordvoiceclient.cpp.o): in function `dpp::discord_voice_client::send_audio_opus(unsigned char*, unsigned long, unsigned long)':
discordvoiceclient.cpp:(.text+0x931e): undefined reference to `crypto_secretbox_easy'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordvoiceclient.cpp.o): in function `dpp::discord_voice_client::send_audio_opus(unsigned char*, unsigned long)':
discordvoiceclient.cpp:(.text+0x94ee): undefined reference to `opus_packet_get_nb_samples'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordvoiceclient.cpp.o): in function `std::_Function_handler<void (OpusDecoder&), dpp::discord_voice_client::set_user_gain(dpp::snowflake, float)::{lambda(OpusDecoder&)#1}>::_M_invoke(std::_Any_data const&, OpusDecoder&)':
discordvoiceclient.cpp:(.text+0x26): undefined reference to `opus_decoder_ctl'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(discordvoiceclient.cpp.o): in function `dpp::discord_voice_client::read_ready() [clone .cold]':
discordvoiceclient.cpp:(.text.unlikely+0x1288): undefined reference to `opus_decoder_destroy'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(etf.cpp.o): in function `dpp::etf_parser::decode_compressed[abi:cxx11]()':
etf.cpp:(.text+0x2bec): undefined reference to `uncompress'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(sslclient.cpp.o): in function `dpp::ssl_client::write(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&)':
sslclient.cpp:(.text+0x8b): undefined reference to `SSL_write'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(sslclient.cpp.o): in function `dpp::ssl_client::connect()':
sslclient.cpp:(.text+0x372): undefined reference to `SSL_new'
/usr/bin/ld: sslclient.cpp:(.text+0x391): undefined reference to `SSL_set_fd'
/usr/bin/ld: sslclient.cpp:(.text+0x3ab): undefined reference to `SSL_ctrl'
/usr/bin/ld: sslclient.cpp:(.text+0x3f9): undefined reference to `SSL_connect'
/usr/bin/ld: sslclient.cpp:(.text+0x40e): undefined reference to `SSL_get_current_cipher'
/usr/bin/ld: sslclient.cpp:(.text+0x416): undefined reference to `SSL_CIPHER_get_name'
/usr/bin/ld: sslclient.cpp:(.text+0x461): undefined reference to `TLS_client_method'
/usr/bin/ld: sslclient.cpp:(.text+0x469): undefined reference to `SSL_CTX_new'
/usr/bin/ld: sslclient.cpp:(.text+0x48a): undefined reference to `SSL_CTX_ctrl'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(sslclient.cpp.o): in function `dpp::ssl_client::read_loop()':
sslclient.cpp:(.text+0x9e8): undefined reference to `SSL_read'
/usr/bin/ld: sslclient.cpp:(.text+0x9fa): undefined reference to `SSL_get_error'
/usr/bin/ld: sslclient.cpp:(.text+0xa29): undefined reference to `SSL_pending'
/usr/bin/ld: sslclient.cpp:(.text+0xa4d): undefined reference to `SSL_shutdown'
/usr/bin/ld: sslclient.cpp:(.text+0xab8): undefined reference to `SSL_write'
/usr/bin/ld: sslclient.cpp:(.text+0xaca): undefined reference to `SSL_get_error'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(sslclient.cpp.o): in function `dpp::ssl_client::close() [clone .localalias]':
sslclient.cpp:(.text+0x107d): undefined reference to `SSL_free'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(sslclient.cpp.o): in function `dpp::ssl_client::cleanup()':
sslclient.cpp:(.text+0x13bd): undefined reference to `SSL_free'
/usr/bin/ld: vcpkg_installed/x64-linux/lib/libdpp.a(sslclient.cpp.o): in function `dpp::ssl_client::ssl_client(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, bool, bool)':
sslclient.cpp:(.text+0x19f4): undefined reference to `SSL_free'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.

To Reproduce
Steps to reproduce the behavior:

Install vcpkg
Clone https://github.com/t1meshift/discord-target-replier/
Build it:

cd discord_target_replier/
cmake -S . -B build -GNinja -DCMAKE_TOOLCHAIN_FILE=$VCPKG_ROOT/scripts/buildsystems/vcpkg.cmake
cmake --build build

Observe ld errors :(

Expected behavior
Successful executable build
System Details:

OS: WSL2 with Ubuntu 22.04
CMake 3.22+
GCC 11.2.0/Clang 14.0
vcpkg baseline 2f56fdad4bcf43c21ec6f952cf7b91c8c43c5c0d

Additional context
I know that Microsoft products work as expected only on Windows, but I just don't want to build DPP from source and manage dependencies manually, and CLion has vcpkg support.

why not just use the prebuilt dpp for debian, which works fine in 22.04 wsl?

please show link errors.
im guessing though that your use of nlohmann json on line 19 is the issue, as dpp already publically uses nlohmann.

closing due to lack of feedback, we can't identify issue without errors

Just found your errors buried in a collapsed section?
However looks to be user error in that you arent specifying the correct build flags. No -std=c++17, etc. Also possibly building a 32 bit build? Not sure from what i see,
/usr/bin/c++ -O3 -DNDEBUG  CMakeFiles/discord_target_replier.dir/main.cpp.o CMakeFiles/discord_target_replier.dir/Config.cpp.o -o discord_target_replier  vcpkg_installed/x64-linux/lib/libdpp.a
Please try following the guide on dpp.dev.


Release 2-11-0

@JamesSmartCell please use tags in future, you can push a version number change to master and tag it if you desire (just don't put anything else in it)


Move few packages to new repo structure

Moving some packages from packages[@azure](https://github.com/azure) and putting them under an sdk directory. Some of these packages are:

Storage
Keyvault
Eventhubs
Servicebus

As part of this update, I have also updated a few links in some of the README.md and Contributing.MD of these packages, that were pointing to the old repos (from outside our org) to the new repo links.
@Azure/azure-sdk-eng

Adding @ramya-rao-a  @AlexGhiondea and @schaabs to review

The package names in JS contain the [at]azure prefix. Did we intentionally remove that?
/cc @weshaggard @KarishmaGhiya

The package names in JS contain the [at]azure prefix. Did we intentionally remove that?

Yes I did as I didn't feel it added enough value to include and would cause another level of directory structure because it is @azure\.  If you feel that there is a reason it should be include let me know.

What is the plan to deal with all the broken links from npmjs.com?
Are we going to have re-directs?

Any chance we can split this PR into multiple, one for each library?

@ramya-rao-a I was thinking along the same lines of introducing redirects. I will discuss this with @weshaggard

@ramya-rao-a Regarding splitting the PR, I think I will do that from next time - separate PR per service.

@KarishmaGhiya There seems to be quite a few changes to the event hubs area that are more than just the move from packages/@azure to sdk. I tried, but I am still not comfortable in signing off a PR that spans across multiple services this way.
Here is what I suggest:

Have a PR that just makes move from lib to src and examples to samples across all services. Separate PRs for each service. Each owner can sign off on this very quickly
Have a PR that removes the need for the parent folder for event hubs. The client and the processor are each separate libraries and so can live directly under package/@azure. testHub can go under the processor. I and @jeremymeng can review this
Once the above are all merged, update this PR to merge from master. By then, the only change in this PR would be packages/@azure to sdk. At this point, even if we are spanning multiple services, most changes would only be file renames which we will be much more comfortable signing off.

@AlexGhiondea, @jeremymeng, @schaabs Thoughts?

Separate PRs for each service. Each owner can sign off on this very quickly

I agree it makes sense to split this into one PR per service. However I think we can probably handle multiple changes to that service in that PR we should just list out the exact renaming/mapping changes that we are making as part of that PR so it is easier to review.

What is the plan to deal with all the broken links from npmjs.com? Are we going to have re-directs?

I don't think it is worth doing a re-direct as that would keep the old directory structure alive which would cause confusion in the repo as we are trying to transition everything over to the new structure. I do suggest that for the next releases of these packages the URL's should point to the repo root instead of any particular path in the repo. We don't want to have these URL's hamper us in making repo structure changes and as long it is easy to figure out our structure people should be able to work with just a repo pointer to find the source they are looking for.


html can't be styled

Hi, thanks first of all for all the good work. However, I am hitting the inability to override the resulting html with css. It would surely be beneficial to refactor that part, as not being able to style bubbles (or anything for that part) is a big omission imo...
I would try to give it a whirl myself if I had the time and the expertise (I am a react specialist and not familiar with web components).
I surely hope it is not the web component approach that makes this impossible, but I suspect it is. Is there no hope for being able to hack internal html with outside css? If that is the case I am not a big fan of web components...

Web components is an umbrella term for a variety of different type of elements, e.g. shadow DOM, custom elements etc.
shadow DOM is not part of the main DOM, meaning its style will not inherit the one placed in the main DOM (parent app). Custom elements exist alongside the main DOM and they will inherit the parent style.
Due to the beauty of the ability to inject a component inside any application and achieve the advertised result - Deep Chat utilises the shadow DOM. Ofcourse, the downside of this is the difficulty in being able to style its internals.
The Deep Chat website provides many ways to style the chat, and in reference to your message you can style the bubbles here. We also provide examples to help achieve what you need.
You can also use the auxiliaryStyle property to override the existing css (however we recommend using the approach above).
E.g.
auxiliaryStyle = `
    .user-message {
        background-color: yellow;
    }`

Let me know if this helps. Thanks!

I now know what to do :) Thanks for the elaborate reply!

After tinkering with this stuff for waaaaay too long I came to understand that web components the way they are implemented here are making it impossible to style stuff iteratively from a useful dev experience perspective.
@OvidijusParsiunas maybe I missed something? How can we use breakpoints, dark mode, all the goodies etc, to style the container? is that doable with auxiliaryStyle

I agree that editing the styling of Shadow Elements is inconvenient, but as mentioned in my earlier comment this in a way is by design and not much can really be done about it.
The auxiliaryStyle property can be used to define relatively advanced css, such as media queries:
auxiliaryStyle = `
    @media only screen and (max-width: 600px) {
      .ai-message {
            background-color: yellow;
        }
    }`

However I am not fully sure about the extent to how many advanced css features are supported as it is entirely up to how the shadow DOM works and not necessarily how Deep Chat is implemented.
Apologies for not being able to do much more.


galaxis: 1.10 -> 1.11

Description of changes

Closes https://github.com/NixOS/nixpkgs/pull/292918
Many thanks @TheRealKeto for remind me of modernizing another old expression!
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Unless it's necessary to re-generate the manage, I don't think you need asciidoctor, as the tarball already includes a manpage.

Unless it's necessary to re-generate the manage, I don't think you need asciidoctor, as the tarball already includes a manpage.

What are we doing about that?

What are we doing about that?

I forgot that I was reading the GitLab sources instead of untarring the tarball.


Animations

Hello,
On the Semanic UI site, the modal for example zooms and fades in:
http://semantic-ui.com/modules/modal.html
On the angular example, the modal just appears instantly:
http://angularify.github.io/angular-semantic-ui/templates/modal.html
Is there anyway to animate this?
Cheers

I think we should animate it by default, everybody wants it like that.
To animate it we have to add .modal('setting', 'transition', value) in the directive.


Feat-reduce_contributor_UI

For issue #131
Not much of a size reduction. However, let me know if this works :)

How it looked before (For comparison):

The changes I made are:

Made the contributor section a flex display, with column flex-direction and justified content to space evenly
Removed the 'profile' button, and made the circular avatar clickable (with scale on hover) to open the required page on a new tab.
Changed width and heights of the contributor divs, and flex-wrapped the container that has the contributors.


Discuss: ratelimits on /swap_pixel

An endpoint that is being heavily considered is /swap_pixel which would let you swap 2 pixels within a 10-pixel radius of each other. This could be used for dragging nearby pixels into an image or taking them from someone else's image. What we need to figure out is what we should use for the rate limit for this endpoint.
Current suggestions include:

1 per 45 seconds (highly unlikely, will explain further down*)
1 per 10 minutes

*Bear in mind the destructive factor of an endpoint like this for taking away from things other people have made, for this reason, it's likely preferable to have a longer ratelimit.

Bear in mind the destructive factor of an endpoint like this for taking away from things other people have made, for this reason, it's likely preferable to have a longer ratelimit.

Isn't the /set_pixel endpoint as destructive? Although I understand that /swap_pixel would be essentially like two /set_pixel requests, so I suggest a ratelimit of 1 per 2 minutes.

Since swapping 2 pixels is the same as reading the canvas, and then setting 2 pixels manually, why the large rate limit?

I am closing this issue. We will be doing something different for the next event, but that will be planned in secret, as to not spoil the fun!


Less async

If there aren't any _onStarting events; generally the whole startup can bypass the async.
Using ReferenceEquals on TaskCache.CompletedTask rather than doing the bit manipulations that Task does to work out if it has completed sync, as is simpler.
Better bits from https://github.com/aspnet/KestrelHttpServer/pull/1477

Looks like you broke some stuff

Linux issue https://github.com/aspnet/KestrelHttpServer/issues/1632

Failed   ConnectionClosesWhenFinReceivedBeforeRequestCompletes(listenOptions: http://<IP_ADDRESS>:0)
Error Message:
Assert.Equal() Failure
↓ (pos 0)
Expected:
Actual:   HTTP/1.1 400 Bad Request\r\nConnection: clo···
↑ (pos 0)

I'm fixing it

@benaadams any benefit to this change besides more complex codez 😄 . Show me da numberz

Cuts out two functions from the call chain
Pre

Post

Trying to find out why WriteAsync allocates 64 bytes in but Write doesn't

@davidfowl I found with Json.NET async work that async methods are a lot slower to call than normal methods. The async state machine adds a lot of overhead.

@JamesNK I'm well aware, I'm still questioning the gains. We make tons of micro optimizations in kestrel that usually add up but there is a cost of maintaining hard to read code. So there's a point at which the optimization has to be worth the ugly and we have micro benchmarks checked in for jus that reason.

64B allocation is just a measuring artefact; issue raised https://github.com/dotnet/BenchmarkDotNet/issues/415

WriteAsync checks _onStarting under lock, and Write checks it not under lock. Should either be both do or neither. Changing it to both

Complete Results
     Method | WithHeaders | Chunked | OnStarting |        Mean |    StdErr |       Op/s | Allocated |
----------- |------------ |-------- |----------- |------------ |---------- |----------- |---------- |
 WriteAsync |       False |   False |      Async | 143.7179 ns | 1.3632 ns | 6958076.98 |      78 B |
 WriteAsync |       False |   False |       None | 142.8760 ns | 0.4573 ns | 6999077.81 |      78 B |
 WriteAsync |       False |   False |       Sync | 141.6665 ns | 0.1538 ns | 7058832.93 |      78 B |
 WriteAsync |       False |    True |      Async | 140.5505 ns | 0.1393 ns | 7114879.17 |      78 B |
 WriteAsync |       False |    True |       None | 141.8294 ns | 0.3130 ns | 7050726.11 |      78 B |
 WriteAsync |       False |    True |       Sync | 142.3879 ns | 0.3813 ns | 7023069.85 |      78 B |
 WriteAsync |        True |   False |      Async | 657.5599 ns | 0.7676 ns | 1520773.97 |     378 B |
 WriteAsync |        True |   False |       None | 422.1309 ns | 1.9038 ns | 2368933.23 |     178 B |
 WriteAsync |        True |   False |       Sync | 692.7274 ns | 7.1783 ns | 1443569.30 |     306 B |
 WriteAsync |        True |    True |      Async | 643.7326 ns | 1.0632 ns | 1553440.08 |     378 B |
 WriteAsync |        True |    True |       None | 420.6879 ns | 0.2489 ns | 2377059.10 |     178 B |
 WriteAsync |        True |    True |       Sync | 627.1672 ns | 0.7878 ns | 1594471.05 |     306 B |

Before and after of the more interesting portion
Before
     Method | WithHeaders | Chunked | OnStarting |        Mean |    StdErr |       Op/s | Allocated |
----------- |------------ |-------- |----------- |------------ |---------- |----------- |---------- |
 WriteAsync |        True |   False |      Async | 667.5727 ns | 1.3135 ns | 1497964.30 |     378 B |
 WriteAsync |        True |   False |       None | 529.8076 ns | 0.6551 ns | 1887477.81 |     178 B |
 WriteAsync |        True |   False |       Sync | 657.7463 ns | 2.3489 ns | 1520343.12 |     306 B |

After
     Method | WithHeaders | Chunked | OnStarting |        Mean |    StdErr |       Op/s | Allocated |
----------- |------------ |-------- |----------- |------------ |---------- |----------- |---------- |
 WriteAsync |        True |   False |      Async | 657.5599 ns | 0.7676 ns | 1520773.97 |     378 B |
 WriteAsync |        True |   False |       None | 422.1309 ns | 1.9038 ns | 2368933.23 |     178 B |
 WriteAsync |        True |   False |       Sync | 692.7274 ns | 7.1783 ns | 1443569.30 |     306 B |

Hmm... sync and async bit weird; will try local copy of completed task

(Updated) Before and after of the more interesting portion
Before (https://github.com/benaadams/KestrelHttpServer/tree/testwriting for exposed methods)
     Method | WithHeaders | Chunked | OnStarting |        Mean |    StdDev |       Op/s | Allocated |
----------- |------------ |-------- |----------- |------------ |---------- |----------- |---------- |
 WriteAsync |        True |   False |       None | 525.3814 ns | 2.0567 ns | 1903379.13 |     178 B |
 WriteAsync |        True |   False |       Sync | 637.3575 ns | 3.8968 ns | 1568978.12 |     306 B |
 WriteAsync |        True |   False |      Async | 636.1822 ns | 4.5565 ns | 1571876.73 |     306 B |

After (this PR)
     Method | WithHeaders | Chunked | OnStarting |        Mean |    StdDev |       Op/s | Allocated |
----------- |------------ |-------- |----------- |------------ |---------- |----------- |---------- |
 WriteAsync |        True |   False |       None | 421.0196 ns | 7.0106 ns | 2375186.07 |     178 B |
 WriteAsync |        True |   False |       Sync | 543.3438 ns | 4.4737 ns | 1840455.51 |     306 B |
 WriteAsync |        True |   False |      Async | 658.5292 ns | 3.4424 ns | 1518535.49 |     306 B |

/cc @davidfowl da numbers 😄

Added tests as PR https://github.com/aspnet/KestrelHttpServer/pull/1642
     Method | WithHeaders | Chunked | OnStarting |        Mean |    StdDev |      Median |       Op/s | Allocated |
----------- |------------ |-------- |----------- |------------ |---------- |------------ |----------- |---------- |
 WriteAsync |       False |   False |      Async | 141.0204 ns | 0.7899 ns | 140.9354 ns | 7091171.73 |      78 B |
 WriteAsync |       False |   False |       None | 140.9476 ns | 0.7818 ns | 141.0736 ns | 7094835.66 |      78 B |
 WriteAsync |       False |   False |       Sync | 141.6527 ns | 1.5219 ns | 141.5544 ns | 7059519.97 |      78 B |
 WriteAsync |       False |    True |      Async | 140.4693 ns | 0.7723 ns | 140.4000 ns | 7118994.26 |      78 B |
 WriteAsync |       False |    True |       None | 140.1714 ns | 1.0770 ns | 140.0190 ns | 7134121.96 |      78 B |
 WriteAsync |       False |    True |       Sync | 142.6427 ns | 3.2782 ns | 141.0240 ns | 7010522.14 |      78 B |
 WriteAsync |        True |   False |      Async | 625.4775 ns | 4.3903 ns | 625.2574 ns | 1598778.59 |     306 B |
 WriteAsync |        True |   False |       None | 417.3450 ns | 2.5388 ns | 416.4601 ns | 2396098.98 |     178 B |
 WriteAsync |        True |   False |       Sync | 520.6009 ns | 4.0205 ns | 519.5717 ns | 1920857.06 |     306 B |
 WriteAsync |        True |    True |      Async | 624.6187 ns | 5.1475 ns | 624.9212 ns | 1600976.81 |     306 B |
 WriteAsync |        True |    True |       None | 420.0299 ns | 1.5173 ns | 419.6871 ns | 2380782.76 |     178 B |
 WriteAsync |        True |    True |       Sync | 522.6808 ns | 4.8042 ns | 522.3586 ns | 1913213.42 |     306 B |

Re-based and reopened as https://github.com/aspnet/KestrelHttpServer/pull/1644

Hmm; could have branched and done it locally. Mt git-fu is sometimes weak

Rebased, removed benchmark changes (mostly), reduced code churn

[xUnit.net 00:00:08.9682778]     EmptyRequestLoggedAsInformation [FAIL]
[xUnit.net 00:00:08.9713116]       Disposing listeners failed
[xUnit.net 00:00:08.9738906]       Expected: True
[xUnit.net 00:00:08.9739931]       Actual:   False
[xUnit.net 00:00:08.9781634]       Stack Trace:
[xUnit.net 00:00:08.9907507]         /home/travis/build/aspnet/KestrelHttpServer/test/Microsoft.AspNetCore.Server.Kestrel.FunctionalTests/HttpsTests.cs(56,0): at Microsoft.AspNetCore.Server.Kestrel.FunctionalTests.HttpsTests.d__0.MoveNext()
[xUnit.net 00:00:08.9912640]         --- End of stack trace from previous location where exception was thrown ---
[xUnit.net 00:00:08.9919972]            at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
[xUnit.net 00:00:08.9928657]            at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
[xUnit.net 00:00:08.9933869]         --- End of stack trace from previous location where exception was thrown ---
[xUnit.net 00:00:08.9938639]            at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
[xUnit.net 00:00:08.9943484]            at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
[xUnit.net 00:00:08.9950562]         --- End of stack trace from previous location where exception was thrown ---
[xUnit.net 00:00:08.9951328]            at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
[xUnit.net 00:00:08.9951848]            at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)

Rebased

@halter73 could you take a look and make @benaadams work more, or close? :)

I rebased your changes and here are the current before and after numbers on my laptop given the updated micro benchmarks from #1642.
Old dev:
     Method | WithHeaders | Chunked | OnStarting |          Mean |     StdErr |      StdDev |        Median |       Op/s | Allocated |
----------- |------------ |-------- |----------- |-------------- |----------- |------------ |-------------- |----------- |---------- |
 WriteAsync |       False |   False |      Async |   223.1018 ns |  3.8948 ns |  21.3326 ns |   211.0020 ns | 4482258.18 |      78 B |
 WriteAsync |       False |   False |       None |   209.1302 ns |  1.8205 ns |   9.9715 ns |   206.4410 ns | 4781711.13 |      78 B |
 WriteAsync |       False |   False |       Sync |   207.6673 ns |  1.0871 ns |   5.9542 ns |   205.9419 ns | 4815395.51 |      78 B |
 WriteAsync |       False |    True |      Async |   221.9945 ns |  4.4913 ns |  24.5997 ns |   210.8805 ns |    4504616 |      78 B |
 WriteAsync |       False |    True |       None |   221.8296 ns |  3.4974 ns |  19.1559 ns |   215.2639 ns | 4507964.57 |      78 B |
 WriteAsync |       False |    True |       Sync |   217.1743 ns |  2.8331 ns |  15.5176 ns |   209.6381 ns | 4604596.22 |      78 B |
 WriteAsync |        True |   False |      Async | 1,081.5509 ns | 31.7755 ns | 174.0416 ns | 1,134.3591 ns |  924598.21 |     305 B |
 WriteAsync |        True |   False |       None |   685.4649 ns |  6.2189 ns |  34.0625 ns |   673.7863 ns |  1458863.8 |     177 B |
 WriteAsync |        True |   False |       Sync |   891.8900 ns | 15.1367 ns |  82.9071 ns |   857.5039 ns | 1121214.47 |     305 B |
 WriteAsync |        True |    True |      Async |   834.2390 ns |  8.3393 ns |  45.6765 ns |   832.9739 ns | 1198697.28 |     305 B |
 WriteAsync |        True |    True |       None |   720.8754 ns | 15.8111 ns |  86.6010 ns |   688.5577 ns | 1387202.26 |     177 B |
 WriteAsync |        True |    True |       Sync |   818.8223 ns |  3.8527 ns |  21.1021 ns |   813.6414 ns | 1221266.18 |     305 B |

This PR
     Method | WithHeaders | Chunked | OnStarting |        Mean |     StdErr |      StdDev |      Median |       Op/s | Allocated |
----------- |------------ |-------- |----------- |------------ |----------- |------------ |------------ |----------- |---------- |
 WriteAsync |       False |   False |      Async | 227.7008 ns |  7.5025 ns |  41.0929 ns | 202.3973 ns |  4391727.9 |      78 B |
 WriteAsync |       False |   False |       None | 197.3162 ns |  0.6649 ns |   3.6417 ns | 196.8270 ns | 5068006.96 |      78 B |
 WriteAsync |       False |   False |       Sync | 196.7176 ns |  0.5358 ns |   2.9346 ns | 196.6478 ns | 5083428.64 |      78 B |
 WriteAsync |       False |    True |      Async | 202.0139 ns |  0.8840 ns |   4.8418 ns | 201.1500 ns | 4950155.12 |      78 B |
 WriteAsync |       False |    True |       None | 201.4103 ns |  0.5774 ns |   3.1624 ns | 201.3188 ns | 4964988.43 |      78 B |
 WriteAsync |       False |    True |       Sync | 201.8038 ns |  0.6568 ns |   3.5976 ns | 201.0586 ns |  4955307.6 |      78 B |
 WriteAsync |        True |   False |      Async | 830.9345 ns | 19.4620 ns | 106.5979 ns | 781.5924 ns |  1203464.3 |     305 B |
 WriteAsync |        True |   False |       None | 543.3715 ns |  5.2474 ns |  28.7412 ns | 530.7326 ns | 1840361.55 |     177 B |
 WriteAsync |        True |   False |       Sync | 697.5552 ns | 11.1772 ns |  61.2198 ns | 676.5557 ns | 1433578.28 |     305 B |
 WriteAsync |        True |    True |      Async | 916.9603 ns | 35.8063 ns | 196.1193 ns | 802.1534 ns |  1090559.8 |     305 B |
 WriteAsync |        True |    True |       None | 714.5119 ns | 32.0643 ns | 175.6234 ns | 653.1019 ns | 1399556.89 |     177 B |
 WriteAsync |        True |    True |       Sync | 799.3473 ns | 38.1445 ns | 208.9260 ns | 658.3018 ns | 1251020.67 |     305 B |

Seems better overall to me, and all the tests still pass. I reviewed this pretty thoroughly while rebasing and it seems reasonable, so I'm merging.

🎉 😸

442ee80 merged!


Question about variable meaning in dataset loader

Thank author for the awesome work and effort to always keep the repo active!
My question is about meaning of variable in data_loader_frames.py.
    def __init__(self,
                 root,
                 file_input,
                 file_labels,
                 frames_duration,
                 args=None,
                 multi_crop_test=False,
                 sample_rate=2,
                 is_test=False,
                 is_val=False,
                 num_boxes=10,
                 model=None,
                 if_augment=True):
        """
        :param root: data root path
        :param file_input: inputs path
        :param file_labels: labels path
        :param frames_duration: number of frames
        :param multi_crop_test:
        :param sample_rate: FPS
        :param is_test: is_test flag
        :param k_split: number of splits of clips from the video
        :param sample_split: how many frames sub-sample from each clip
        """
        self.in_duration = frames_duration
        self.coord_nr_frames = self.in_duration // 2
        self.multi_crop_test = multi_crop_test
        self.sample_rate = sample_rate
        self.if_augment = if_augment
        self.is_val = is_val
        self.data_root = root
        self.dataset_object = WebmDataset(file_input, file_labels, root, is_test=is_test)
        self.json_data = self.dataset_object.json_data
        self.classes = self.dataset_object.classes
        self.classes_dict = self.dataset_object.classes_dict
        self.model = model
        self.num_boxes = num_boxes

Does the sample_rate here denote FPS? I have the question 'cause I saw @ruiyan1995 mention we need to extract the video in 12 FPS to get the correct performance. But here we are using sample_rate as 2. It's some confusing for me.

Does the frame_duration mean number of frames? Here I find we're setting the frame_duration as num_frames in our args. Do it mean we will have the fixed frames (like 8) for every video?

What does self.coord_nr_frames mean? Why it's a half of frames_duration?

Looking forward your kind reply, thanks! @joaanna @ruiyan1995

@YAOYI626
Extract frames from the original videos with '12 FPS'.  The 'sample rate' is used to sample the images as the input of the model from frames that have been extracted.


Merge 3.2.7 changes into master

Mostly just a merge of the PRs #2451  #2449  and #2440, but I did have to make a change to the Podfile post_install script

          1 Error

:no_entry_sign:
You need to set the App’s plist version to 3.2.7 - use make update_bundle_version

  Generated by :no_entry_sign: Danger

OK, the Danger fail is fine here 👍


Does the client support deep linking?

I want to create an entity and its related entities with a single request using deep linking, but when I try it doesn't work.
I'm trying to send this:
{
"Surname": "Smith",
"Addresses": [
{
"Address1": "End of the road",
"City": "No where",
"Postcode": "SW12"
}
],
"PhoneNumbers": [
{
"Number": "01"
}
]
}
(Which works, by the way, if I POST it using Fiddler2)
But instead it sends this:
{
<EMAIL_ADDRESS>"#KbContext.Customer",
"Surname": "Smith",
<EMAIL_ADDRESS>[
"http://localhost:8080/kb/api/Addresses(0)"
],
<EMAIL_ADDRESS>[
"http://localhost:8080/kb/api/PhoneNumbers(0)"
]
}
I've tried both the typed and none typed "Set" methods of the client, but both bind instead of deep link.
Is this a feature that is supported by the client? Am I doing something wrong?

Hi,
Yes, you can insert multiple entities in a single request, look at batch examples:
https://github.com/object/Simple.OData.Client/blob/master/Simple.OData.Client.Tests.Net40/BatchTypedTests.cs
https://github.com/object/Simple.OData.Client/blob/master/Simple.OData.Client.Tests.Net40/BatchTests.cs
Are you trying something else? Can you paste an example of your code?

Hi,
Thanks for getting back in touch.
I can't upload a sample right now as I'm at work, but I will do later this evening.
However, BatchTypedTests.InsertSingleEntityWithSingleAssociationSingleBatch looks promising. I'll give that a try.
Although, I'm not quite sure that batch is what I'm looking for - its essentially two separate requests sent together. I'd rather it make a single request.
Bind relates the new entity to existing entities (http://docs.oasis-open.org/odata/odata/v4.0/errata02/os/complete/part1-protocol/odata-v4.0-errata02-os-part1-protocol-complete.html#_Toc406398327), but I want them created at once using deep linking (thats not the correct name, its actually called "deep insert"): http://docs.oasis-open.org/odata/odata/v4.0/errata02/os/complete/part1-protocol/odata-v4.0-errata02-os-part1-protocol-complete.html#_Toc406398328
Cheers

Oh I see, you meant deep insert. No, currently Simple.OData.Client doesn't support deep insert - using OData batch is the only way to create multiple entries in a single request. But it doesn't seem that it will take much work to implement it, I will look into it.

Sorry, my mistake. It'll be great if you could get it to work :)

Can you send me at<EMAIL_ADDRESS>your metadata file and the code example that is trying to do a deep insert?

Sure. Should have those for you in about 10-20 minutes.

Hi again,
Thank you for the code example. I looked at it and it seems that the implementation will require API change. Since Simple.OData.Client accepts plain objects, without any annotations, there is no way for it to detect what kind of insert (or update once deep updates are supported) is meant: deep (by value) or by reference. In your example both Address and PhoneNumber can point to a newly created entity or refer to an existing one. My first thought was to reuse Expand clause without changing API, but this will overload standard OData clause with new (and unexpected) semantics. So I believe the best would be to extend Set with new overload, allowing it to specify what referenced entities should be deep-inserted.
I will investigate if this is something that might be implemented as a minor release (4.9) or planned for a major upgrade (5.0) later this year.

Cool - I'm looking forward to it :)

Not sure why it closed...

Is this supported in current version of Simple.OData.Client

No deep inserts are not currently supported. Requires API change, so most likely will be part of version 5.

Hi,
Do you foresee any timeline by which this this would be integrated and Version 5 would be released.
We have requirement for Deep Inserts. So right now we are thinking of using OData for Query and RESTClient for deep Insert, Update, Delete operations.
But I would prefer If we could achieve all CRUD operations using Simple.OData.Client.
In general, I would like to thank you for such a great library and also your quick reply to our issues posted earlier.

Hi Ronak,
My plan for version 5 is Q1 2015, but I will look into deep insert and see if it can be released prior version 5. I will let you know if this happens.

Hi,
any update regarding this issue ?
Do you have any plans, when you are going to release an update for the Deep Insert?
thanks

Hi,
I have looked into this enhancement, but it required too many changes to fit into minor release. So this will be a part of the version 5, originally planned for Q1 but obviously postponed. Since this is my free time project I'd rather not set any date commitments. There a few features (like this one) that needs to be implemented, so it will take some time.

If deep insert is not supported, then this client will not work for the scenario where a Parent entity has a child object that is required and has some sort of validation between the Parent and Child.
For example, I can't POST the Parent, and then POST the Child because of validation asserting that the Parent has a Child object with XYZ property that is required. I can't POST the Child first either, because the Child can't exist without the Parent. Hopefully this makes sense. In summary, if we ever have to POST an object that requires some other child object, this OData client can't support our needs without Deep Insertion.

Hi,
Is this feature implemented in Simple.OData.Client 5.0.0-alpha2 ?
How deep insert will work when keys (of both parent & child entities) are auto-generated at server side?
I'm using Asp.Net WebApi 2 Odata v4 with  Entity Framework.

Hi,
I also would really like this feature. I'm a happy user of this library, and this seems to be the only feature that we need that is missing. We have an object that has required child entities, and our API does not accept a POST of the parent object without any child objects.
Any update on this?

Still would really like to use this client, but am unable to without deep insert support. Any update on expected timing?

@im2cre8iv Since you are one of the few who needs deep insert support, can you think about how you would like API to be extended to support this feature? Also, if you can give some examples of generated payload. I've read the specs, I still need to figure out how to implement the API changes.

Since he doesn't reply and I subscribed to this issue, I'll explain how I'd like it to work instead:

can you think about how you would like API to be extended to support this feature?

Something along the lines of:
client.For<Order>().Set(order, configuration => configuration.Include(o => o.OrderLines)).InsertEntryAsync()
Or maybe:
client.For<Order>().Set(order).Include(o => o.OrderLines).InsertEntryAsync()
Although I think the first one might be less ambiguous, the second one might fit the current idea of Simple OData Client better. It keeps it more simple.

Also, if you can give some examples of generated payload.

Generated Payload would be along the lines of:
{   "OrderId": 34,   "OrderNumber": "    23456",   "Amount": 37.94,   "OrderLines": [     {       "ItemId": 65436,       "Quantity": 5.0,       "Amount": 29.95     },     {       "ItemId": 7356,       "Quantity": 1,       "Amount": 7.99     }   ] }
I hope I was clear enough.
Thanks for your time,
Tim Baart

I looked at this a year or so ago and have come back to it now. I managed to get some test output that looked ok to me but it throws an error. Not sure if it's something I'm doing or if the limitations of the base OData libraries are still causing problems. 😕

Hi Vagif,
I'm just wondering if this Pull Request solves the issue (not sure if it only addresses updates):
https://github.com/OData/odata.net/pull/1585
However, the PR made it to ODataLib 7.6.2 Release:
https://docs.microsoft.com/en-us/odata/changelog/odatalib-7x#odatalib-762-release
What do you think?

The last time I tried deep inserts with the Odata client lib (7.9.3) it didn't work. I was mainly interested in Deep inserts and not Deep updates at the time, so I didn't explicitly try a Deep update.
What I can't seem to find is whether deep inserts/updates were ever implemented in Simple.Odata.Client?
Any idea?


Is there anyway to (de)serialize an OpenAPI object?

For some reason, I'm using swagger-parser to parse Json to OpenAPI object. However, the json-string is pretty large (about 1MB), it takes about half a minute to analyze it every time, so I want to cache the OpenAPI object in some way. But the OpenAPI class itself does not implement serializable, so I can't do in in native way. Is there any tool for me to realise this?

Why don't you cache the json-string yourself?

Jackson supports jdk serialization, so you can try something like this

Why don't you cache the json-string yourself?

I surely can, but the problem is the deserialization procedure from json-string to OpenAPI object cost much longer time than reading the json-string, which is where I want to do the optimization.


Unexpected behavior with skipFiles

Issue Type: Bug
Create two nodejs files, main.js and lib.js, as shown below:
main.js:
const { takesCallback, takesJobAsync } = require('./lib');

//  Repeatedly pressing F11 to "step into"
main().catch(() => {});

async function main() {
    takesCallback(() => {
        console.log('I want to be stopped on.');
    });

    console.log('I, too, want to be stopped on.');

    await takesJobAsync(async () => {
        console.log('I also want to be stopped on.');
    });
}

lib.js:
module.exports = {
    takesCallback(cb) {
        cb();
    },
    async takesJobAsync(job) {
        //  Do setup
        await simulateAsyncSetup();
        try {
            return await job();
        }
        catch (err) {
            //  Do error handling
        }
        finally {
            //  Do tear down
        }
    }
};

function simulateAsyncSetup() {
    return new Promise(resolve => setTimeout(resolve, 1));
}

Create the following launch.json:
{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "node",
            "request": "launch",
            "name": "Launch Program",
            "program": "${workspaceFolder}/main.js",
            "showAsyncStacks": true,
            "stopOnEntry": true,
            "skipFiles": [
                "<node_internals>/**/*.js",
                "${workspaceFolder}/lib.js"
            ]
        }
    ]
}

I'm seeing a few unexpected things:
1. Could not load source 'undefined'

Start the debugger
Press F11

Expected behavior: the debugger stops at line 4.
Observed behavior: a new tab opens titled "undefined" with the following content: Could not load source 'undefined': Could not retrieve content..
2. Stepping into console.log() bypasses the rest of the function

Start the debugger
Press F10 until it stops at line 4
Press F11 to enter main()
Press F11 to stop at line 8
Press F11

Expected behavior: Debugger stops at line 11.
Observed behavior: main() continues until the program exits. The debugger does not stop.
Alternatively, instead of stepping into takesCallback, press F10 to step over takesCallback and stop on line 11. Then press F11. Observe both this issue and the previous.
3. Stepping into async function that takes a callback skips over the callback.

Start the debugger
Press F10 until the debugger stops at line 4.
Press F11 to step into main()
Press F10 until the debugger stops at line 13.
Press F11 to step into takesJobAsync()

Expected behavior: Debugger stops at line 14 because takesJobAsync is defined in lib.js, which is in the list of skipFiles, but the anonymous function it calls is defined in main.js.
Observed behavior: The program finishes. The debugger does not stop.
VS Code version: Code - Insiders 1.38.0-insider (dcc01858d3f75696568eb751238d01b30a1e7d33, 2019-08-30T16:36:37.961Z)
OS version: Linux x64 4.18.0-20-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 3900)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: unavailable_offviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 0, 0

Memory (System)
15.38GB (0.23GB free)

Process Argv
../projects/misc/node-skip-example/

Screen Reader
no

VM
0%

Extensions: none

I can't reproduce this, what version of node are you using?
This feature is totally implemented using the devtools protocol's "blackbox scripts", and I see the same thing using chrome devtools with the pattern I use to identify node files. I can try to report an issue upstream but can't fix it directly.
Same as 2, the runtime is handling async stepping and blackboxed scripts, but apparently can't combine the two.

I'm using node v12.2.0.


💡[Feature]: Add Ola Bike Ride Request Demand Forecast

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The Ola Bike Ride Request Demand Forecast is a predictive feature designed to estimate the future demand for bike ride requests on the Ola platform. Using historical ride request data, weather conditions, time of day, special events, and other relevant factors, this feature forecasts the expected number of ride requests in various locations, helping Ola optimize bike availability and driver allocation.
Use Case
This feature would enhance the project by enabling better resource management and operational efficiency. By accurately predicting demand, Ola can ensure that an adequate number of bikes and drivers are available in high-demand areas, reducing wait times for customers and improving overall service quality. This leads to increased customer satisfaction and more efficient use of Ola's resources.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@sanjay-kv can u please assign me this


AbortErrors when using Cow in FireFox

AbortError on line 564 in cow.js
at
var request = store.put(record);
This occurs when starting cow with the icm-ui client in FireFox

This is reproducable by refreshing firefox within some seconds of opening the app.
Since this situation is not likely to happen often and the effects of it appear to be minimal I close this.


Allow using only lowercase or camelCase in package name.

I now about Oracle Docs and Package naming convention, but...
When package name starts with a capital letter (com.company.appname.UI) I get:
Error:(5, 29) error: cannot find symbol class UI
Error:(9, 51) error: package UI does not exist
But for com.company.appname.uI or com.company.appname.ui all work fine.

@extronovosib, you should simply not do that. Stick to java conventions. By
using package name with upper case you actually make you build depending on
a UNIX* file system, and it's bad.
You can still make a PR to Dart to make your build work, but I strongly
advise you to change your package names to use lower case. And such PR
honestly would have no value except encouraging bad practices..
2016-05-15 3:02 GMT-07:00 ExtroNovosib<EMAIL_ADDRESS>
I now about Oracle Docs and Package naming convention, but...
When package name starts with a capital letter (com.company.appname.UI) I
get:
Error:(5, 29) error: cannot find symbol class UI
Error:(9, 51) error: package UI does not exist
But for com.company.appname.uI or com.company.appname.ui all work fine.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/f2prateek/dart/issues/114


Add autocompletion for xonsh

xonsh is a python-powered shell. It would be nice to have autocompletion for xonsh too!
If you would accept a PR please let me know. :-)

Hi! We need this to be supported in Clap in order to resolve, so I'm going to close in favor of that upstream issue: https://github.com/clap-rs/clap/issues/4693


CI for Node

템플릿을 사용하여 workflow 추가

Contains the compiled JavaScript
Initializes with two players
Starts the game with a random player
Sets the current player to be whoever it is not (1 ms)


Title: PacMan

Is your feature request related to a problem? Please describe.
************************* PacMan ********************************
Creating Pac man Game using JS , PURE CSS AND HTML
Describe the solution you'd like.
1. Win when reaching 274 points
2. eat fruits and the ghosts freeze
3. if ghost eat you you lose
Describe alternatives you've considered.
cosider using keyboard keys
Add any other context or screenshots about the feature request here.

@Mennapika Good Luck 👍🏻


net/dial: ListenPacket does not support modifying socket options

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
newest
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
linux
What did you do?
https://golang.org/pkg/net/#ListenPacket
this function may return a IPConn object through function ListenIP.
a IPConn object contains a socket object, the socket objects comes from  src/net/sock_posix.go,function func socket(ctx context.Context and  set socketopts  through setDefaultSockopts function
https://github.com/golang/go/blob/master/src/net/sock_posix.go#L46
Here is the question: in my case I want to modify the socket's recv buffer size by
syscall.SetsockoptInt(s, syscall.SOL_SOCKET,syscall.SO_RCVBUF,1010241024)
but I can not find any fucntion in go-document to modify the socket's atrribute of the IPConn's socket, So i have to add code
syscall.SetsockoptInt(s, syscall.SOL_SOCKET,syscall.SO_RCVBUF,1010241024)  at here:
https://github.com/golang/go/blob/master/src/net/sockopt_linux.go#L18
It is not good to modify the go source code,
Do you have a function which I can used to modify the IPConn's socket ? Thanks.
I think it is very useful to modify the socket's attribute such as recv buffer size,as the deafult
value can not satisfy every thing
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
What did you expect to see?
What did you see instead?

Closing as dup of #9661.

Here is the question: in my case I want to modify the socket's recv buffer size

c, err := net.ListenPacket("ip4:...", ...)
c.(*net.IPConn).SetReadBuffer(...)
Next time, please take a look at https://github.com/golang/go/wiki/Questions and ask your question on more appropriate forums.

@mikioh: Thanks a lot   (∩_∩)


⚠ Bofrost.fr has degraded performance

In 69350cc, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 885 ms

Resolved: Bofrost.fr performance has improved in 56d961d after 9 minutes.


PHK not decrypting/verifiying data

On 6S+ iOS9.0.2 (jailbroken, for the matter)
Raspberry Pi B2
Latest github commit.
After sending accessories info, the PHK daemon freezes and does not continue.
Ask for accessories info
Reply: HTTP/1.1 200 OK
Content-Type: application/hap+json
Content-Length: 743

{"accessories":[{"aid":1,"services":[{"iid":1,"type":"3E","characteristics":[{"value":"Light 1","perms":["pr"],"type":"23","iid":2,"format":"string"},{"value":"ET","perms":["pr"],"type":"20","iid":3,"format":"string"},{"value":"Light","perms":["pr"],"type":"21","iid":4,"format":"string"},{"value":"12345678","perms":["pr"],"type":"30","iid":5,"format":"string"},{"perms":["pw"],"type":"14","iid":6,"format":"bool"}]},{"iid":7,"type":"43","characteristics":[{"value":"Light","perms":["pr"],"type":"23","iid":8,"format":"string"},{"value":true,"perms":["pr","pw","ev"],"type":"25","iid":9,"format":"bool"},{"value":50,"minValue":0,"maxValue":100,"minStep":1,"perms":["pr","pw","ev"],"type":"8","iid":10,"unit":"percentage","format":"int"}]}]}]}
send: 1
send:<PHONE_NUMBER>00000
send: 2
I??|<QW???9??y?3?f??*bA4'0??Z)ߍ??9
Passed-in data is no-verified!

I assume RPi is little endian?
Anyway, can you try the new push?

It is little endian, according to python -c "import sys; print(sys.byteorder)"
Latest commit not working, same thing. Still: Passed-in data is no-verified!

I'm having the same issue on OS X. I can pair but nothing beyond that.

I'm still trying to figure it out. It works on my Ubuntu x86-64, but not on anything else.
So I will probably finish tracking and fix it this week after I finish handling all the school work.

I track down the problem: My MacBook weirdly pushed an old version of code for a while, which make the anything I push doesn't changed
I'm going to push this alongside with multi characteristic operation this weekend, but I'm going to need some testing in big endian platform before I can say I got all changes merge on this side.

Any luck?

I'm not sure if the the current commit would work on RPi, because mine still haven't arrived, and my friend's was being used as a course project till next week. And given from the output, and the description on RPi endian, the byte swap seems also problematic.
But if you still see the byte swap happening on little endian machine, go to line 90 of PHKNetworkIP.cpp, and change it into the following:
return !e.c[0];
This reverse the result on the big endian detection, until I figure a way to write the endian system
If you wondering if the byte swap is occur, check for the line like this
send: 1
send:<PHONE_NUMBER>00000
send: 2
P.S. Sorry for taking so long. Never through taking 16 credits on MS is such time consuming. I haven't slept normally for a long time.

Latest commit is working fine on my RPI (B2) !
And don't feel bad about it taking time, we are grateful that you chose to share your project with us.


Repetition of client certificate in error string

In the following error string client certificate is repeated. The conditionals leading to the error check for the certificate and the certificate key, so the error likely needs to call out both. Suggest updating the second reference to client certificate key.
https://github.com/versity/versitygw/blob/070406926251e901504172ad14358b81bb4d6895/auth/iam_vault.go#L50
	if clientCert != "" {
			if clientCertKey == "" {
				return nil, fmt.Errorf("client certificate and client certificate should both be specified")
			}

@ndjones This looks like a good fix to me. Do you want to open a PR for this to get credit for the fix?


vcpkg install curl:x86-windows-static FAILED

C:\Users\Dreg\Downloads\vcpkg-master\vcpkg-master>vcpkg update
Using local portfile versions. To update the local portfiles, use git pull.
No packages need updating.
C:\Users\Dreg\Downloads\vcpkg-master\vcpkg-master>vcpkg install curl:x86-windows-static
The following packages will be built and installed:
curl[core]:x86-windows-static

nghttp2[core]:x86-windows-static
Additional packages (*) will be modified to complete this operation.
Starting package 1/2: nghttp2:x86-windows-static
Building package nghttp2[core]:x86-windows-static...
-- CURRENT_INSTALLED_DIR=C:/Users/Dreg/Downloads/vcpkg-master/vcpkg-master/installed/x86-windows-static
-- DOWNLOADS=C:/Users/Dreg/Downloads/vcpkg-master/vcpkg-master/downloads
-- CURRENT_PACKAGES_DIR=C:/Users/Dreg/Downloads/vcpkg-master/vcpkg-master/packages/nghttp2_x86-windows-static
-- CURRENT_BUILDTREES_DIR=C:/Users/Dreg/Downloads/vcpkg-master/vcpkg-master/buildtrees/nghttp2
-- CURRENT_PORT_DIR=C:/Users/Dreg/Downloads/vcpkg-master/vcpkg-master/ports/nghttp2/.
nghttp2 doesn't currently support static library build
CMake Error at ports/nghttp2/portfile.cmake:10 (message):
avoiding building DLL with static CRT.
Call Stack (most recent call first):
scripts/ports.cmake:72 (include)

Error: Building package nghttp2:x86-windows-static failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: nghttp2:x86-windows-static
Vcpkg version: 0.0.103-unknownhash
Additionally, attach any relevant sections from the log files above.

Thanks for reporting this! It should be fixed in the latest master.

The new release works fine, thx!


Escape mustaches symbols

I'm wondering if there is a way to escape double mustaches symbols ({{ }}'). I'm developing a vue-cli template and I want to format the files to the standard.
import Vue from 'vue'{{#if_eq lintConfig "airbnb"}};{{/if_eq}}
{{#vuex}}
import store from '../store'{{#if_eq lintConfig "airbnb"}};{{/if_eq}}
{{/vuex}}

const API_URL = process.env.API_URL{{#if_eq lintConfig "airbnb"}};{{/if_eq}}

Thanks!

Are you linting that file with the {{ symbols included? That won't work. I assume you want to lint it after the template is applied, right?

I'm writing a templete, so i am not liting that file but i want to format it in order to the resulting file follow the standar and could be liting.

Sorry, I don't understand your question. If you're linting the output, after the template has been processed, then you shouldn't have any {{ left in the file. So, what is your question?


[r2.9-rocm-enhanced] r2.9 upstream sync 220429

https://github.com/ROCmSoftwarePlatform/frameworks-internal/issues/1565

jenkins: retest Ubuntu-sanity please


Don't clear hidden elements

Environment
Hi!
When I create a form, the ID hidden input is null.
After I send the form, I want clear it, to a new send a new register.
So, at a first moment I used ....formvue.clear() and after reset etc.
But nothing remove value of id hidden input.
So tried form..$el(#).update('').... clear()... reset()...
And nothing.... :/
Why it's happen?
Reproduction
1 - Using scheme, create a form at least one Hidden element named ID
2 - Fill the form and send it, and dont out of compoment
3 - Only try clear it
4 - Use the inspect element to see. The hidden elemento dont update.
Describe the bug
When I create a form, the ID hidden input is null.
After I send the form, I want clear it, to a new send a new register.
So, at a first moment I used ....formvue.clear() and after reset etc.
But nothing remove value of id hidden input.
Additional context
I'm load a component to create e form to register some data
Logs
Without logs

I wasn't able to reproduce this so I'm closing this. Feel free to reopen with a reproduction.


ARROW-1384: [C++] Add SerializeRecordBatch API for writing a record batch as an IPC message to a new buffer

There's also a little bit of API scrubbing as I went through this code.

+1 LGTM


Build Error: Cannot read property 'Symbol(Symbol.iterator)' of null

Hi, this is working for me with version 2.7, but breaks when using version 3:
TypeError: Cannot read property 'Symbol(Symbol.iterator)' of null
    at Compiler.<anonymous> (/Users/chris/code/xxx/node_modules/webpack-pwa-manifest/dist/WebpackPwaManifest.js:71:51)
    at next (/Users/chris/code/xxx/node_modules/tapable/lib/Tapable.js:140:14)
    at /Users/chris/code/xxx/node_modules/optimize-css-assets-webpack-plugin/index.js:97:27
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:169:7)

It was caused by plugin order; fixed in #8


If  provider is in dependencies, make it optional to add  provider.notifier

Describe the bug
I have recently updated to version 1.0.0 and encountered some issue with using the dependencies parameter when creating provider, I'm not sure if this is a bug or intended but it does seems a bit anti-intuition to have explicitly pass both stateNotifierProvider and stateNotifierProvider.notifier into the dependencies list. It will be great if any clarification that can be provided. Thanks in advance!
To Reproduce
Consider the code below where the MainNotifier makes use of the DependentNotifier's state and function. In order to make the code work and not throw an exception when calling the doSomething function inside the MainNotifier class, both dependentProvider and dependentProvider.notifier are required in the dependencies list for the provider.
class DependentNotifier extends StateNotifier<String> {
  DependentNotifier() : super('dependent');

  String returnSome() {
    return 'some';
  }
}

class MainNotifier extends StateNotifier<String> {
  MainNotifier(this.value, this._ref) : super('main');

  final Ref _ref;
  final String value;
  void doSomething() {
    final a = _ref.read(dependentNotifier);
    final b = _ref.read(dependentNotifier.notifier).returnSome();
    state = state + value + a + b;
  }
}

final valueProvider = Provider<String>((ref) => throw UnimplementedError());

final dependentProvider = StateNotifierProvider<DependentNotifier, String>(
  (ref) => DependentNotifier(),
);

final mainProvider = StateNotifierProvider<MainNotifier, String>(
  (ref) => MainNotifier(ref.watch(valueProvider), ref),
  dependencies: [
    valueProvider,
    dependentProvider,
    dependentProvider.notifier,
  ],
);

Expected behavior
Only need to pass in the dependentProvider.
final mainProvider = StateNotifierProvider<MainNotifier, String>(
  (ref) => MainNotifier(ref.watch(valueProvider), ref),
  dependencies: [
    valueProvider,
    dependentProvider,
  ],
);

That looks like an edge-case I didn't think of.
I'll see if I can change that, but for now yes, you'll need to add boths to your dependencies list

This was fixed in a previous refactoring. So I'll close this


Add Common Name to certificate.

Some applications fail to parse the certificate if the CN is not set,
even if they (correctly) check SANs before the CN when validating a
hostname.  Even though the CN should be ignored if a matching SAN
hostname was found, we can prevent the parse from failing by also
including the hostname as the CN.
Additionally, switch from maps to slices to prevent DNS names and IP
addresses from being reordered when added to the certificate template.

Relevant RFC: https://tools.ietf.org/html/rfc612
The usage of the Common Name is not recommended, but it is ok to still include it for backwards compatibility.

Travis tests failed for an unrelated problem that is fixed in #48.  Will rebase after that is in.

OK after rebase/squash to single commit.


Unable to set SimpleTableDataAdapter inside Fragment

I am unable to set SimpleTableDataAdapter inside Fragment.
tableView.setDataAdapter(new SimpleTableDataAdapter(this, body));
I have tried using the following, but none works.
tableView.setDataAdapter(new SimpleTableDataAdapter(getActivity().getApplicationContext(), body));
tableView.setDataAdapter(new SimpleTableDataAdapter(getActivity(), body));

Hi @arnab3111,
what means "unable". What effect to you see? Is the compiler complaining or do you have issues during runtime?
Best regards,
Ingo

The compiler is complaining.
for getActivity() I am getting the following error
Cannot resolve constructor 'SimpleTableDataAdapter(android.support.v4.app.FragmentActivity, java.lang.String[])
and for getActivity().getApplicationContext() the following error
'Cannot resolve constructor SimpleTableDataAdapter(android.content.Context, java.lang.String[])

Use getContext()

Use TableDataAdapter instead of SimpleTableDataAdapter.


Remove old and unused imex ark algo

This PR does a bit of house cleaning and removes some of the old imex ark algorithm pieces

bors r+

bors r+

bors r+

bors r+

bors r+


Reference '-g myResourceGroup' should be changed to '-g $resourceGroupName'

A reference '-g myResourceGroup' should be changed to '-g $resourceGroupName' in section 'Configure the function app' for Node.js V10 SDK to allow for copy-and-paste to succeed (for when the resource name isn't 'myResourceGroup').

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

ID: cad5909e-e1c6-aa8e-82f1-f2851c1caced
Version Independent ID: 42292330-aa69-9df0-5bd7-e07c9daed995
Content: Use Azure Event Grid to automate resizing uploaded images
Content Source: articles/event-grid/resize-images-on-storage-blob-upload-event.md
Service: event-grid
GitHub Login: @spelluru
Microsoft Alias: spelluru

Hi @ptuton Thank you for your feedback! We will review and update as appropriate.

Hello @ptuton Thanks again for reporting this issue and helping us drive an improvement towards MS Docs! We have updated doc and the changes should reflect within 24hrs.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


learning react website

a react website where i have compiled all the hooks like-

usestate
useeffect
useref
usememo
usecallback
usecontext

a learning react website

Ma'am, why it was closed as completed


Error: Could not locate the bindings file.

Hi, I tried installing libxmljs on a CentOS6.5 server and received the following error:
/opt/node-server-qa/node_modules/libxmljs/node_modules/bindings/bindings.js:91
  throw err
        ^
Error: Could not locate the bindings file. Tried:
 → /opt/node-server-qa/node_modules/libxmljs/build/xmljs.node
 → /opt/node-server-qa/node_modules/libxmljs/build/Debug/xmljs.node
 → /opt/node-server-qa/node_modules/libxmljs/build/Release/xmljs.node
 → /opt/node-server-qa/node_modules/libxmljs/out/Debug/xmljs.node
 → /opt/node-server-qa/node_modules/libxmljs/Debug/xmljs.node
 → /opt/node-server-qa/node_modules/libxmljs/out/Release/xmljs.node
 → /opt/node-server-qa/node_modules/libxmljs/Release/xmljs.node
 → /opt/node-server-qa/node_modules/libxmljs/build/default/xmljs.node
 → /opt/node-server-qa/node_modules/libxmljs/compiled/0.10.29/linux/x64/xmljs.node
    at bindings (/opt/node-server-qa/node_modules/libxmljs/node_modules/bindings/bindings.js:88:9)
    at Object.<anonymous> (/opt/node-server-qa/node_modules/libxmljs/lib/bindings.js:1:99)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/opt/node-server-qa/node_modules/libxmljs/index.js:4:16)
    at Module._compile (module.js:456:26)

Everything seemed to build ok, and I tried rebuilding several times.  (Sorry for the long paste):
$ node-gyp rebuild
gyp info it worked if it ends with ok
gyp info using<EMAIL_ADDRESS>gyp info using<EMAIL_ADDRESS>| linux | x64
gyp info spawn python
gyp info spawn args [ '/usr/bin/gyp',
gyp info spawn args   'binding.gyp',
gyp info spawn args   '-f',
gyp info spawn args   'make',
gyp info spawn args   '-I',
gyp info spawn args   '/opt/node-server-qa/node_modules/libxmljs/build/config.gypi',
gyp info spawn args   '-I',
gyp info spawn args   '/usr/lib/node_modules/node-gyp/addon-rpm.gypi',
gyp info spawn args   '-I',
gyp info spawn args   '/usr/share/node/common.gypi',
gyp info spawn args   '-Dlibrary=shared_library',
gyp info spawn args   '-Dvisibility=default',
gyp info spawn args   '-Dnode_root_dir=/usr/share/node',
gyp info spawn args   '-Dmodule_root_dir=/opt/node-server-qa/node_modules/libxmljs',
gyp info spawn args   '--depth=.',
gyp info spawn args   '--generator-output',
gyp info spawn args   'build',
gyp info spawn args   '-Goutput_dir=.' ]
gyp info spawn make
gyp info spawn args [ 'BUILDTYPE=Release', '-C', 'build' ]
make: Entering directory `/opt/node-server-qa/node_modules/libxmljs/build'
  CC(target) Release/obj.target/libxml/vendor/libxml/buf.o
  CC(target) Release/obj.target/libxml/vendor/libxml/catalog.o
  CC(target) Release/obj.target/libxml/vendor/libxml/chvalid.o
  CC(target) Release/obj.target/libxml/vendor/libxml/dict.o
  CC(target) Release/obj.target/libxml/vendor/libxml/encoding.o
  CC(target) Release/obj.target/libxml/vendor/libxml/entities.o
  CC(target) Release/obj.target/libxml/vendor/libxml/error.o
  CC(target) Release/obj.target/libxml/vendor/libxml/globals.o
  CC(target) Release/obj.target/libxml/vendor/libxml/hash.o
  CC(target) Release/obj.target/libxml/vendor/libxml/HTMLparser.o
  CC(target) Release/obj.target/libxml/vendor/libxml/HTMLtree.o
  CC(target) Release/obj.target/libxml/vendor/libxml/legacy.o
  CC(target) Release/obj.target/libxml/vendor/libxml/list.o
  CC(target) Release/obj.target/libxml/vendor/libxml/parser.o
../vendor/libxml/parser.c: In function ‘xmlParseInNodeContext’:
../vendor/libxml/parser.c:13492: warning: pointer targets in passing argument 1 of ‘xmlFindCharEncodingHandler’ differ in signedness
../vendor/libxml/include/libxml/encoding.h:175: note: expected ‘const char *’ but argument is of type ‘const xmlChar *’
  CC(target) Release/obj.target/libxml/vendor/libxml/parserInternals.o
  CC(target) Release/obj.target/libxml/vendor/libxml/pattern.o
  CC(target) Release/obj.target/libxml/vendor/libxml/relaxng.o
  CC(target) Release/obj.target/libxml/vendor/libxml/SAX2.o
  CC(target) Release/obj.target/libxml/vendor/libxml/SAX.o
  CC(target) Release/obj.target/libxml/vendor/libxml/tree.o
  CC(target) Release/obj.target/libxml/vendor/libxml/threads.o
  CC(target) Release/obj.target/libxml/vendor/libxml/uri.o
  CC(target) Release/obj.target/libxml/vendor/libxml/valid.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xinclude.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xlink.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlIO.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlmemory.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlmodule.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlreader.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlregexp.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlsave.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlschemas.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlschemastypes.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlstring.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlunicode.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlwriter.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xpath.o
../vendor/libxml/trionan.c:218: warning: ‘trio_is_negative’ defined but not used
  CC(target) Release/obj.target/libxml/vendor/libxml/xpointer.o
  AR(target) Release/obj.target/vendor/libxml/xml.a
  CXX(target) Release/obj.target/xmljs/src/libxmljs.o
  CXX(target) Release/obj.target/xmljs/src/xml_attribute.o
  CXX(target) Release/obj.target/xmljs/src/xml_document.o
  CXX(target) Release/obj.target/xmljs/src/xml_element.o
  CXX(target) Release/obj.target/xmljs/src/xml_comment.o
  CXX(target) Release/obj.target/xmljs/src/xml_namespace.o
  CXX(target) Release/obj.target/xmljs/src/xml_node.o
  CXX(target) Release/obj.target/xmljs/src/xml_sax_parser.o
  CXX(target) Release/obj.target/xmljs/src/xml_syntax_error.o
  CXX(target) Release/obj.target/xmljs/src/xml_xpath_context.o
  SOLINK(target) Release/obj.target/xmljs.node
  SOLINK(target) Release/obj.target/xmljs.node: Finished
  COPY Release/lib.target/xmljs.node
make: Leaving directory `/opt/node-server-qa/node_modules/libxmljs/build'
gyp info ok

So I investigated and tried copying cp build/Release/lib.target/xmljs.node build/xmljs.node from within node_modules/libxmljs/ and things started working.  Not sure what the issue is with the bindings file not being copied to the proper location, but thought someone should know.  Thanks.

This is the only solution I could get working: https://github.com/gwicke/libxmljs/commit/7e1ceaf96021926871e07a397d53de63c136a22b

Hi,
When will the fix be available within the official version?
trying an "npm install libxmljs" with current version 0.14.2 gives the error from the initial report.
os: debian 8.1
thanks,
Stefan

I can't use the npm distributed version of libxmljs either because of this. gwicke@7e1ceaf also fixes it for me.

Still can't use libxmljs from npm.  Same problem.  I had to do this to work around it:

npm install libxmljs
(edit node_modules/libxmljs/binding.gyp to remove the 2 offending lines mentioned above)
cd node_modules/libxmljs/
node-gyp rebuild

I'm on Debian Sid (unstable).

I had this problem with the deasync and Node 5.6.0. Solved it with node-gyp rebuild also.

FWIW, this is still happening for me:

Node 4.4
Debian sid
gwicke@7e1ceaf still fixes it.

Is anyone still affected by this issue?

I'm running into it now. Tried on Windows, current versions of Node, NPM, and libxmljs.

Ran into this issue today as well. Thanks for the above context and solutions everyone! gwicke@7e1ceaf fixed this for me.
`node -v`: v.7.6.0`npm -v` 4.1.2CentOS 6.5

I am facing this issue as well. My OS is based on Ubuntu (which is based on Debian)
$ lsb_release -a

No LSB modules are available.
Distributor ID:	elementary
Description:	elementary OS 0.4 Loki
Release:	0.4
Codename:	loki

Myself as well now running in WSL (Ubuntu on Windows 10).
I've also tried git cloneing the source to node_modules and that got me no further.
To borrow from pedzed's wonderful format......
lsb_release -a

No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.2 LTS
Release:        16.04
Codename:       xenial

node --version && npm --version

v6.10.3
5.0.3

rm node_modules/ && yarn

npm install || npm rebuild --update-binary

g++: error: /mnt/c/prj/AZPREP/GIT/WEB/BatchManWeb/node_modules/libxmljs-mt/build/Release/xmljs.node: No such file or directory
node-libxslt.target.mk:140: recipe for target 'Release/obj.target/node-libxslt.node' failed
make: *** [Release/obj.target/node-libxslt.node] Error 1
make: Leaving directory '/mnt/c/prj/AZPREP/GIT/WEB/BatchManWeb/node_modules/libxslt/build'
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Linux 4.4.0-43-Microsoft
gyp ERR! command "/usr/bin/nodejs" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /mnt/c/prj/AZPREP/GIT/WEB/BatchManWeb/node_modules/libxslt
gyp ERR! node -v v6.10.3
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.

I also had this issue when building my project with libxmljs 0.18.4 and I had to use 0.18.3 instead. The node-gyp suggestion above did not solve this issue for me. Node v6.10.2, npm 3.10.10.

I had same problem
just try that:
projectToto > $ sudo rm -rf  node_modules/
projectToto > $ npm install bindings

this on worked for me
projectToto > $ sudo rm -rf node_modules/
projectToto > $ npm install bindings
thanks @masselit

Well, I tried the solutions above but none did work for me. So I solved it my way like this:
I changed node_modules/libxmls/lib/bindings.js from
module.exports = require('bindings')('xmljs');

to
module.exports = require('bindings')({
  "bindings": "xmljs",
  "module_root": "[path-to-my-project-folder]/node_modules/libxmljs",
});

Okay, it's not a bulletproof solution but it works for me. If anybody knows how to make this cleaner I want to hear it.

After once you have done step as done by @MeiKatz  go back to the root project folder i.e.
Drive://
and then run the rebuild command
npm rebuild

What I did was copying cp build/Release/lib.target/xmljs.node build/xmljs.node from within node_modules/libxmljs/ and things started working
as suggested by OP, and now it finally works.................. I wasted an afternoon.


Fix Unix/Linux/Mac detection

I think there was a slight misconception of the usage of isunix.
This actually evaluates to true on mac and linux. As a result, the code required wine on mac machines.
The proposed changes fixes this hopefully. I also remove the dos()/system() distinction as it is unnecessary I think.
I tested Example.m with Matlab 2017 on Windows, Mac and Ubuntu 16.04.

What do you mean?
I didn't add a comment, but I removed two comments that are redundant (imho) because of version control.

What do you mean?
I didn't add a comment, but I removed two comments that are redundant (imho) because of version control.

I meant the lines I reviewed/comented here on github, that you already checked. I wasn't aware that the line comment already was enough to create a comment so I added an additional one to finish the review process.

I force-pushed.. maybe that wasn't the best choice for code review :)


Fix 'cfgov-deployed' queries to not use 'SINCE this quarter'

This corrects a misunderstanding of how SINCE this quarter works in NRQL. I wrongly assumed it would be comparable to SINCE 90 days ago, but it's really since the beginning of the current quarter, which makes sense. But it's not what we want. We just want to go back sufficiently far enough, which is what this change gets us.

merging just for completeness, though we will be getting rid of this code in the near future


Make this Perplexica repo deployable on Vercel

Feature Request: Enable Deployment on Vercel
Is your feature request related to a problem? Please describe.
Currently, the repository can't be deployed directly on Vercel, which limits its accessibility and ease of deployment for users who prefer using Vercel's platform.
Describe the solution you'd like
I'd like to see support added to the repository so that it can be easily deployed on Vercel. This would involve updating the project's configuration files and dependencies to ensure compatibility with Vercel's deployment environment.
Describe alternatives you've considered
One alternative could be deploying the repository on other platforms that support the current configuration. However, adding support for Vercel would offer users more deployment options and flexibility.
Additional context
Providing the option to deploy on Vercel would enhance the accessibility and usability of Perplexica, attracting more users and contributors to the project. Additionally, it would align with the goal of making the project more widely available and user-friendly.

Hi, Vercel doesn't supports websockets and Perplexica uses websockets as its core functionality and hence, Perplexica cannot be fully deployed on Vercel.

Thank you, I've seen all of those, but that is a completely different story. The backend is separate from the frontend in Perplexica and needs to be deployed separately. Specifically, deploying a backend WebSocket server on Vercel is not possible as Vercel primarily supports stateless functions and static sites, which don't maintain persistent connections required by WebSockets.


raven_framework interfaceCheck error

Issue Description
What did you expect to see happen?
Using the following command line to do interface check:
raven_framework inputFile.xml interfaceCheck
Supposed to see if the interface is functional without invoking the executables.
What did you see instead?
IOError: There were failed runs; aborting RAVEN
It seems that RAVEN submits the jobs, ignoring the keyword interfaceCheck.
Do you have a suggested fix for the development team?
A workaround suggested is using:
python ../../../../framework/Driver.py inputFile.xml interfaceCheck
In this way RAVEN recognizes the keyword and tests the interface without invoking the executables of the codes linked by the interface.
Suggestion would be checking the argument processing in the raven_framework.
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.
The input files are from the tests in the codes. Location is:
~/raven/tests/framework/CodeInterfaceTests/Scale/

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[ ] 1. Is it tagged with a type: defect or improvement?
[ ] 2. Is it tagged with a priority: critical, normal or minor?
[ ] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[ ] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[ ] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[ ] 1. If the issue is a defect, is the defect fixed?
[ ] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[ ] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[ ] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
[ ] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?

Out of curiosity, what is the branch (and date of last commit) that you're working with when you see this?

Actually I haven't created a branch but I'm working with Andrea on the interface between RAVEN and SCALE. I did a git pull just before I ran the above tests, so I assume the code is up to date.

@PaulTalbot-INL yes it is in devel...i checked it

I just tried the same thing on (current) devel and it passed for me on the melcor interface ... and I checked and we do preserve arguments in the raven_framework call.  My command was:
~/projects/raven/raven_framework test_melcor_interface.xml interfaceCheck

We have an open issue for that, see #440. I'm not sure if this is fixed or
not. At that time, if you run run_tests with interfaceCheck, the code will
work, and if you run raven_framework, the code will ignore the key word.
Hopefully, this will help.
On Mon, May 21, 2018 at 2:27 PM, Paul Talbot<EMAIL_ADDRESS>wrote:

I just tried the same thing on (current) devel and it passed for me on
the melcor interface ... and I checked and we do preserve arguments in the
raven_framework call. My command was:
~/projects/raven/raven_framework test_melcor_interface.xml interfaceCheck
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_idaholab_raven_issues_659-23issuecomment-2D390772011&d=DwMFaQ&c=54IZrppPQZKX9mLzcGdPfFD1hxrcB__aEkJFOKJFd00&r=vJsbMjnzR6ErmdvVqv2kNhwbKhk1o91w6B8VQ4EHCto&m=AJMwNGMig3Ijri2gl45bOqGSRozXfJjD5QcldVLfuTg&s=2ipKxd4ovOktXhI5OKXI6L2aALLZ4vtauBJgVC2Ny-M&e=,
or mute the thread
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AG-2D1714DE-5FCbkSW8RiRd34Iayx3DJLKDks5t0yMbgaJpZM4UHgRc&d=DwMFaQ&c=54IZrppPQZKX9mLzcGdPfFD1hxrcB__aEkJFOKJFd00&r=vJsbMjnzR6ErmdvVqv2kNhwbKhk1o91w6B8VQ4EHCto&m=AJMwNGMig3Ijri2gl45bOqGSRozXfJjD5QcldVLfuTg&s=Hj8Gsv2CAG8wo6rYM2Di8elfSPXI9g5krIewbNyCRQQ&e=
.

--
Sincerely,
Congjian Wang
PhD, Nuclear Engineering Methods Development, INL
2525 North Fremont Ave., Idaho Falls, ID 83415-3860
Work Phone:<PHONE_NUMBER>

Closure checklist passed...
No email is required since the defect was affecting a "testing" feature that we currently do not expose to the users


Add install instruction using winget

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Banner management #558, workaround PR

Motivation and Context
Workaround PR for https://github.com/ONSdigital/response-operations-ui/pull/558, where GitHub Actions has hit a bug.

What has changed

How to test?

Links

/deploy foxj

Same result as #558


Routing. Header/Footer PR

Also note: every page will looks very similar. So it's a good practice to use reusable Page element as wrapper

Ready for re-check.


WriteTableToMarkdown - need tests for new command

Need tests for new WriteTableToMarkdown and SetTableColumnProperties commands, and update other tests due to enhancements that were required for processing tables.

I added/updated tests.  Close this issue.


How do you type all?

do you have an idea on how to get safely typed generators that are under the all effect?

I proposed a PR about this issue, hope to be merged in next version.
https://github.com/agiledigital/typed-redux-saga/pull/9

@Moejoe90 I'm closing this as @dionshihk 's PR is in the latest version. Let me know how you go.


Bug 2097685: Remove the container at exit

Fix an issue when the agent container keeps restarting indefinitely
if failed to start.

@derekhiggins sorry forgot to change the tests!

/lgtm

/test e2e-metal-ipi

/approve

we will backport this to 4.10 after https://github.com/openshift/image-customization-controller/pull/54 merges

/cherry-pick release-4.10


Add julia 1.11-beta to lookahead and add dependabot

See if we can catch some problems earlier than later.

https://github.com/CliMA/ClimaAtmos.jl/actions/runs/8765896794/job/24057894026?pr=2931
AtmosConfig cannot be inferred/compiled in Julia 1.11-beta1

When I tried on my laptop, compilation would just hang with no errors.
Hopefully things will improve in the next betas

I’m happy to merge this with soft-fail after rebasing


[Improvement][Master] Validate same content of input file when using task cache

Purpose of the pull request

This PR closes https://github.com/apache/dolphinscheduler/issues/13223.

Brief change log

Add function getFileChecksum in FileUtils available for both file and directory. When calculate checksum for a directory, it will calculate all files checksum under the directory and concatenate them together, the order is provided by file-system interface.
Upload checksum file (with suffix .crc) together with output file when uploading for task file transfer.
Download checksum file and replace the value of file property with content of checksum file.

Verify this pull request

Already add some UT.

Codecov Report

Merging #13298 (f6a0a97) into dev (8870464) will increase coverage by 0.06%.
The diff coverage is 82.35%.

@@             Coverage Diff              @@
##                dev   #13298      +/-   ##
============================================
+ Coverage     39.53%   39.59%   +0.06%
- Complexity     4293     4301       +8
============================================
  Files          1072     1072
  Lines         40558    40581      +23
  Branches       4670     4663       -7
============================================
+ Hits          16035    16069      +34
+ Misses        22741    22727      -14
- Partials       1782     1785       +3

Impacted Files
Coverage Δ

...e/dolphinscheduler/common/constants/Constants.java
75.00% <ø> (ø)

...ver/master/consumer/TaskPriorityQueueConsumer.java
0.00% <0.00%> (ø)

...che/dolphinscheduler/dao/utils/TaskCacheUtils.java
86.76% <78.26%> (-4.54%)
:arrow_down:

...pache/dolphinscheduler/common/utils/FileUtils.java
60.22% <83.33%> (+5.94%)
:arrow_up:

...er/server/worker/utils/TaskFilesTransferUtils.java
80.37% <100.00%> (+1.37%)
:arrow_up:

...erver/master/processor/queue/TaskEventService.java
75.00% <0.00%> (-5.36%)
:arrow_down:

...uler/api/controller/ProcessInstanceController.java
68.18% <0.00%> (-2.04%)
:arrow_down:

...java/org/apache/dolphinscheduler/dao/AlertDao.java
28.46% <0.00%> (-0.86%)
:arrow_down:

...pache/dolphinscheduler/common/utils/DateUtils.java
68.30% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/service/alert/ProcessAlertManager.java
41.70% <0.00%> (-0.38%)
:arrow_down:

... and 9 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix package controller client test

This test was failing if you had REGISTRY_USERNAME etc set.

merged in another


added zenodo doi

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.176%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
6324

Relevant Lines:
7172

💛  - Coveralls


Update deep-link-application.md

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/build-and-test/deep-link-application.md
95
7
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/concepts/build-and-test/deep-link-application.md
95
7
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.