[ISSUE #314] Heartbeat handler use independently thread pool

What is the purpose of the change
thousands of client Sub message, when subscription change, broker notify all client with same cid,
causing related ThreadPoolExecutor pool full, so Heartbeat use independently thread pool
Brief changelog
increase notify   consumerIds changed  semaphore size
Verifying this change
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a JIRA issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [ROCKETMQ-XXX] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

I am wondering why notifying clients of membership change could take up the thread pool? Note notify is performed in an asynchronous non-blocking and one-way manner. Can you share your case details justifying this is indeed the root cause?

some reasons, I can't  say more about it.  @lizhanhui @vongosling

@fuyou001  I agree with @lizhanhui , the others command would not  take up the thread pool. It's useless to isolate the pool.

Coverage increased (+0.05%) to 41.538% when pulling c00c9d6f8a6cca7f34d041c2d25987c2660826d5 on fuyou001:issues-314 into 92f0e1f5abb694c90ced9468e9e1d03edcdcedfe on apache:develop.

Considering heartbeat plays a major role in maintaining cluster members in sync. I'd agree to move it to a dedicated thread-pool so that it won't be delayed too much in case broker is under heavy pressure.

@vsair I talked to @fuyou001 privately via email. The reason he is reluctant to share is that disclosing these technical details may bring about security threats to their commercial service before applying patches to all serving brokers. I'd post updates to share the scenario and explain more at the earliest convenient time.

It seems to be more reasonable to isolate the time-taking notification works with Heartbeat, so +1.

@vongosling

https://github.com/apache/rocketmq/pull/315


BEWLR Page is out of date

https://docs.neoforged.net/docs/items/bewlr/
Now that we are moving away from using the initializeClient event and moving towards using the RegisterClientExtensionsEvent event, the page is not only a little lackluster in it's explanations and could probably use a bit more documentation, but as well we probably want to show devs how to use the event for this, not the method that's being removed in 1.21.2

You probably had an old version of the page open, because we merged a PR fixing this just yesterday. The BEWLR article has been merged into the BER article, and has been updated to reflect the new method of registering.
Thank you for opening this issue regardless!


Person ORCID field?

Currently we have no field in Person (and here) to represent ORCID.
I suggest using identifier, but there might be more appropriate fields.

I think @HLWeil's suggestion in https://github.com/nfdi4plants/isa-ro-crate-profile/issues/16 could also be applied here.

for the workflow ro crate we use identifier for the ORCID, e.g.
{
      "@id": "https://orcid.org/0000-0001-8875-7304",
      "@type": "Person",
      "name": "Thomas Roetzer-Pejrimovsky",
      "identifier": "https://orcid.org/0000-0001-8875-7304"
    }

Yes, that's a good option. It fits with Lukas' suggestion on PubMedIDs or DOIs. Since identifier allows for PropertyValue objects, we can use them to specify that it is an identifier of type ORCID:
{
  "@id": "https://orcid.org/0000-0001-8875-7304",
  "@type": "Person",
  "name": "Thomas Roetzer-Pejrimovsky",
  "identifier": {
    "propertyID": "ORCID",
    "value": "0000-0001-8875-7304"
  }
}

To me, it doesn't really matter which option w choose.

Fixed by PR https://github.com/nfdi4plants/isa-ro-crate-profile/pull/19


Strange Terminal visualization on start up

Issue Type: Bug
Version: 1.30.1 (user setup)
Commit: dea8705087adb1b5e5ae1d9123278e178656186a
Date: 2018-12-18T18:12:07.165Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
OS: Windows_NT x64 10.0.17763
Actual: There is a bug in the integrated terminal on OS Windows 10 Pro, version 1809. The text visualization looks broken. The working path is missing on terminal openning.  The path appears when you press Enter. The bug is reproducible also with vscode 1.29.1 that I'm sure it was working correctly on my computer before the last Windows update.

Expected: It should work as it is working on OS Windows 10 Pro, version 1803.


chore: complete REUSE tool integration

In line with SAP OSPO requirements, this PR integrates the REUSE tool
Notes:

Please register the repository with the REUSE Tool to complete the integration (see #16 and #17)
This PR assumes that no third-party source code was copied in this repository. If this assumption is not correct, please change the .reuse/dep5 file accordingly. Edits by maintainers are enabled in this PR.

Fixes #15

@hackergil Thanks for merging. Please don't forget to register the repository: https://api.reuse.software/register


Update PowerShell ISE to better handle debugging script blocks

The ISE is designed for editing, running, debugging script files.  But many scripts run as script blocks that have no script source files.  In this case the ISE provides a poor command line debugger compared to the PowerShell console.

The ISE command line debugging should be as good as PowerShell console.
The ISE could create temporary files for running script blocks so that they could be debugged in the ISE GUI similar to how script files are currently debugged.

Work on the ISE is on hold and instead effort is being put into VSCode.  The latest PowerShell plugin for VSCode now supports debugging script blocks.  So this issue can be closed.


GIS viewer: improve spatial presentation

We need to improve spatial data presentation for better visualization of multiple objects (per row).

Use different shape colors for different columns
Improve tooltips to show source column on the top
Add column selector (to show geoms for selected columns only)

verified


http version

hello,i want know http version in werkzeug？

if you mean http instead of wsgi: you want something like uwsgi or gunicorn that speaks both http and wsgi.

It doesn't, it removes hop-by-hop headers which are not supposed to be forwarded. See https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers

I have some questions. 1. Why are the headers of the request HTTP/1.1, but the headers of the response are HTTP/1.0?
2. Is there any other way to make the responding headers also follow HTTP/1.1, now I know that using nginx can make the responding headers comply with HTTP/1.1?

How are you accessing werkzeug? If you use the dev server, that one is probably http1.0-only. Use a real webserver if you want HTTP/1.0 or HTTP/2

太感谢了，我用flask开发服务，但是现在的服务是不稳定的。我需要用nginx做个代理了。

Thank you very much, I used flask to develop services, but the service is unstable now. I need to use nginx as a proxy.

Most important you need to use a real WSGI container like uwsgi or gunicorn and DO NOT use flask run or app.run() - those are not suitable for production.

Yes, big brother. I used gunicorn to wrap the flask service. But still not very stable.

Thank you very much


➖ Remove lodash dependency

I might be wrong, but this is the only place in the codebase where we use a lodash function.
lodash is quite big, and even if ember-cli-mirage is only bundled in dev/test builds, this could be refactored with typeof and we could completely drop the lodash dependency…

Prettier gets me every time. Want to add lint-staged to the git commit hook to run it for me 😄

@SergeAstapov I've ran yarn workspace ember-cli-mirage-docs test:ember locally and it passes. How can I fix the CI /Docs action?

@bartocc the stack trace makes me think the issue caused by some transitive dependency that got drifted:
ERROR Summary:

  - broccoliBuilderErrorStack: TypeError: Cannot read property 'program' of undefined
    at DocFactory._inspectExportDefaultDeclaration (/home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/esdoc/out/src/Factory/DocFactory.js:136:40)
    at new DocFactory (/home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/esdoc/out/src/Factory/DocFactory.js:91:10)
    at Function._traverse (/home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/esdoc/out/src/ESDoc.js:232:21)
    at /home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/esdoc/out/src/ESDoc.js:114:25
    at Function._walk (/home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/esdoc/out/src/ESDoc.js:203:9)
    at Function._walk (/home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/esdoc/out/src/ESDoc.js:205:14)
    at Function.generate (/home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/esdoc/out/src/ESDoc.js:98:10)
    at generateESDoc (/home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/ember-cli-addon-docs-esdoc/lib/preprocessors/generate-esdoc.js:53:11)
    at generateESDocJsonApi (/home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/ember-cli-addon-docs-esdoc/lib/preprocessors/generate-esdoc-jsonapi.js:67:14)
    at /home/runner/work/ember-cli-mirage/ember-cli-mirage/node_modules/ember-cli-addon-docs-esdoc/lib/broccoli/generator.js:41:23

however <=3.24 scenarios fail with different error
Uncaught Error: Could not find module `@glimmer/manager` imported from `(require)`

I just saw that exact same error in https://github.com/miguelcobain/ember-css-transitions/runs/6711631246?check_suite_focus=true which makes me think it's again comes from transitive dependency.
I don't have time right to look into root cause, will try to do once I have a chance.

@SergeAstapov any idea how I could help debug the CI?

Thank you @bartocc!


GeneticCNN.py

Make special Setting for boosting performance on MNSIT.
Hi, aqibsaeed,
I'm doing some research about NAS recently.
I noticed your implementation for GeneticCNN, which is awesome.
But I notice that the parameter setting in your Code is not suitable for MNIST（chanel is too small, and kernel size for pooling is very large）, which leads to the result that the performance of it is not ineffective.
So I forked your repo, and made some change for MNSIT.
Thank for your contribution.
Best,
ShipXu


Stack Overflow at api.get_updates in example async_reply_to_message_updates.rs

Hi! I tried running the example async_reply_to_message_updates.rs. It crashed with a stack overflow error:
thread 'main' has overflowed its stack
error: process didn't exit successfully: `target\debug\telegram-bot-test.exe` (exit code: 0xc00000fd, STATUS_STACK_OVERFLOW)

Process finished with exit code -1073741571 (0xC00000FD)

I was able to pinpoint the overflowing call in the example:
https://github.com/ayrat555/frankenstein/blob/1217d29c3b6c8f9866d0c21cbba42517e62ce52e/examples/async_reply_to_message_updates.rs#L18
Running frankenstein = { version = "0.13.0", features = ["async-http-client"] }, rustc 1.60.0 (7737e0b5c 2022-04-04), Win 10.
I will investigate further. Maybe the error is within the example, or my project setup

The following code
let result = api.get_updates(&update_params);
match result.await {

fails at await. so no stack overflow up to the first await

When supplying an invalid token, the stack overflow disappears and nothing unexpected happens. Of course, the request fails, but this is expected.
static TOKEN: &str = "no token provided";

https://github.com/ayrat555/frankenstein/blob/1217d29c3b6c8f9866d0c21cbba42517e62ce52e/src/api/async_telegram_api_impl.rs#L58
The json deserialization seems to be the problem. There is a specific json text coming from telegram, let me see

The json contains some private data. It's a json list of updates, not looking suspicious to me

Like this (ids scambled):
{
    "ok": true,
    "result": [
        {
            "update_id":<PHONE_NUMBER>2,
            "poll": {
                "id": "5345345261554",
                "question": "What would you like to order, master?",
                "options": [
                    {
                        "text": "The Amici for some unhealthy snack",
                        "voter_count": 1
                    },
                    {
                        "text": "The Beef Club for distinguished Ladies and Gentlemen",
                        "voter_count": 0
                    },
                    {
                        "text": "Kaishi Combo for the extraordinarily exotic",
                        "voter_count": 1
                    }
                ],
                "total_voter_count": 2,
                "is_closed": false,
                "is_anonymous": true,
                "type": "regular",
                "allows_multiple_answers": false
            }
        },
        {
            "update_id":<PHONE_NUMBER>,
            "poll": {
                "id": "544757133453451554",
                "question": "What would you like to order, master?",
                "options": [
                    {
                        "text": "The Amici for some unhealthy snack",
                        "voter_count": 1
                    },
                    {
                        "text": "The Beef Club for distinguished Ladies and Gentlemen",
                        "voter_count": 0
                    },
                    {
                        "text": "Kaishi Combo for the extraordinarily exotic",
                        "voter_count": 0
                    }
                ],
                "total_voter_count": 1,
                "is_closed": false,
                "is_anonymous": true,
                "type": "regular",
                "allows_multiple_answers": false
            }
        },
        {
            "update_id":<PHONE_NUMBER>,
            "message": {
                "message_id": 34345,
                "from": {
                    "id":<PHONE_NUMBER>,
                    "is_bot": false,
                    "first_name": "xxxx"
                },
                "chat": {
                    "id": -13453457001,
                    "title": "bot test group",
                    "type": "supergroup"
                },
                "date":<PHONE_NUMBER>,
                "text": "Nice"
            }
        },

    }
}```

and then some more updates. about 500 lines when the json is formatted

example async_get_me.rs works just fine

It also overflows for the following response (ids scrambled):

{
    "ok": true,
    "result": [
        {
            "update_id":<PHONE_NUMBER>45,
            "message": {
                "message_id":<PHONE_NUMBER>45,
                "from": {
                    "id":<PHONE_NUMBER>5,
                    "is_bot": false,
                    "first_name": "Johannes",
                    "username": "cvyxcvxycv",
                    "language_code": "en"
                },
                "chat": {
                    "id":<PHONE_NUMBER>35,
                    "first_name": "Johannes",
                    "username": "cvyxcvxycv",
                    "type": "private"
                },
                "date":<PHONE_NUMBER>45,
                "text": "hi"
            }
        }
    ]
}

@johannesvollmer yes, this issue is happening only on windows. we need to investigate this.
I haven't seen this happening on *nix
cc @EdJoPaTo

I tried to add the following test to async_telegram_api_impls.rs test module
    #[test]
    fn async_send_message_success() {
        let response_string = r#"
        {
            "ok": true,
            "result": [
                {
                    "update_id":<PHONE_NUMBER>45,
                    "message": {
                        "message_id":<PHONE_NUMBER>45,
                        "from": {
                            "id":<PHONE_NUMBER>5,
                            "is_bot": false,
                            "first_name": "XXX",
                            "username": "cvyxcvxycv",
                            "language_code": "en"
                        },
                        "chat": {
                            "id":<PHONE_NUMBER>35,
                            "first_name": "XXX",
                            "username": "cvyxcvxycv",
                            "type": "private"
                        },
                        "date":<PHONE_NUMBER>45,
                        "text": "hi"
                    }
                }
            ]
        }
        "#;

        let json_result: Result<MethodResponse<Vec<Update>>, serde_json::Error> = serde_json::from_str(body);
    }

It does not fail though. Where is the problem

Maybe #8 is related here as it was a windows only problem too? Would be interesting to find out the cause.

After reading https://github.com/rust-lang/rust/issues/80289, I now have a theory:

stack memory is 8MB on unix by default, but only 1MB on Windows
stack memory is exhausted differently when running tests vs when running full examples
json deserialize function uses a lot of stack memory: std::mem::size_of::<Update>() is 30384bytes, which is 30kb! i guess this is allocated a lot, maybe even recursively, in the json deserialization function

If this theory turns out to be true, I see two solutions:

library user increases stack size to a larger number, maybe 10MB. can be done immediately without updating the crate. but how could we document that
the library finds a way of reducing stack memory

use an enum instead of having a ton of optional fields in struct Update and in struct Message
use Option<Box<Message>> in struct Update, instead of inline Option<Message> (may have performance implications)

Boxing all Message fields in struct Update made the example run on my device, finally. The struct Update now has 9872b, which is about 1kb, process. Boxing all the fields in Update makes it about 120 bytes.

I have a fork with the quickfix changes. If the enum solution turns out to take too long, this could serve as a temporary work-around. What do you think? Any doubts about the stack overflow theory?

That's interesting!
Boxing everything seems like horrible code to me. And the storage is not gone, its just elsewhere (on the heap) shifting the problem. Designing the structs differently to improve their memory usage (whereever it is stored) is the best solution to me. This should solve this problem and results in more effective RAM uses on all platforms.
Using an enum for Update seems like an interesting solution here. Due to it being used a lot changes here should have the most impact overall. Even if that requires custom (de)serialization code. Taking a look at teloxide they did exactly that: https://github.com/teloxide/teloxide-core/blob/master/src/types/update.rs (Their code is MIT licensed).
Not sure if we could just use the teloxide-core create for the types in general :thinking:

Even if that requires custom (de)serialization code.

I think we might get away without custom de/serialization, using externally tagged enums and struct flattening. That way, we might just do the following change (field renaming ommited for brevity):
#[derive(Serialize, Deserialize)]
struct Update {
  update_id: u32,

  #[serde(flatten)]
  content: UpdateContent,
}

#[derive(Serialize, Deserialize)]
enum UpdateContent {

  #[serde(rename="message")]
  Message(Message),

  EditedMessage(Message),
  Poll(Poll),
  // ...
}

If I'm not mistaken, this should result in the following json:
{
  "update_id": 32,
  "message": {
    "message_id": 32
  }
}

Which is the json we are currently having, isn't it?

Again, (as I'm unsure how to interpret the official telegram documentation) this will only work if all of these states never occur at the same time, otherwise an enum will not make sense obviously

Again, (as I'm unsure how to interpret the official telegram documentation) this will only work if all of these states never occur at the same time, otherwise an enum will not make sense obviously

The official documentation states exactly that: https://core.telegram.org/bots/api#update

haha didn't find that one. So, any objections about the enum approach? @ayrat555

no objections.
@johannesvollmer great investigation skills. good work!

@EdJoPaTo I'd rather keep teloxide and Frankenstein independent.

Their types look useful. Using the bot api types in two projects would simplify a lot of work. And they will be the same for every Telegram Library. But yeah, teloxide-core seems to do more than just the types.

Also, I noticed that the allowed_updates functionality could be made type safe. Right now, one might pass an invalid string to the function.
 I will happily review them :)

In the message documentation, I can't find any statement as to whether some of the fields will be present at the same time. Nevertheless, teloxide has an enum for some of the fields

I think as the first step we can just migrate Update to enum
btw @johannesvollmer are you working on this?
I can try to pick this up this week (most probably on the weekend) if you're not

have the change done, was actually a task of 5 min, but i feel like some more tests should be added though
see #62

Merged. Awesome, thanks!


Automated cherry pick of #1776: fix: allow access mcclient serviceCatalog

Cherry pick of #1776 on release/2.11.0.
#1776: fix: allow access mcclient serviceCatalog

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11.0@a96e28f). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             release/2.11.0   #1778   +/-   ##
================================================
  Coverage                  ?   7.64%
================================================
  Files                     ?     412
  Lines                     ?   63255
  Branches                  ?       0
================================================
  Hits                      ?    4833
  Misses                    ?   58013
  Partials                  ?     409

Flag
Coverage Δ

#aFlag
7.64% <ø> (?)

Continue to review full report at Codecov.

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

/lgtm
/approve


Visual Studio 2015 - Compiler errors.

Hi,
Have you tried to compile the core with Visual Studio 2015 ( either 32 or 64 bits ) ?
am getting nasty errors within the compiler it goes to error C1001, and C3203 in some classes of the Typepack variadic libraries
/// Turn a class template or alias template \p C into a Alias Class.
template <template <typename...> class C> struct quote {
// Indirection through defer here needed to avoid Core issue 1430
// http://open-std.org/jtc1/sc22/wg21/docs/cwg_active.html#1430
template <typename... Ts> using apply = t_<defer<C, Ts...>>;
};

oh no - I hadn't recently, had hoped that it would be easier, not harder...

Hi, can someone who understand the Typepack library have a look and fix it for visual studio  2015 ?
What's the reason what those complicated libraries has been used to implement a basic wrapper containers ?
Thanks

I will take a look at it in a bit. Fortunately that bit is relatively easy to test on its own.
TypePack is instead of using Boost MPL or Fusion: it uses C++11 features to do metaprogramming. I try to keep its usage minimal - the reason it's used in the container wrapper is because the container wrapper is customizable with what features are exposed publicly from the underlying container.

OK, I've built just the TypePack unit tests in MSVC14, and found two internal compiler errors, which I've isolated in a small repo here: https://github.com/rpavlik/typepack-msvc14-ice
Not sure if they can be worked around easily (if there's some known bug in C++ support that they're hitting - they build fine in many versions of clang and GCC, as well as MSVC 12), or if Microsoft will need to fix the bug on their end but in either case the isolation should be a good step.

(the docs for c1001 mention optimization, but disabling optimization and inlining doesn't fix it.)

Hmm, though it's definitely in c2, the backend, not the c1xx the compiler frontend, because it still goes boom with the new-in-update-1 "Clang-with-C2" toolset.

Err, I take that back. It compiles fine with the clang/c2 thing. The config change just hadn't "taken".
I'm filing a bug.

been there, my feeling is that MSVC  might been more strict than GCC with some of the latest C++ features, this C1001 does not really give so much information, where are you filing the bug ?
about the containers , i see the point, actually it makes sense, if this is being used to transfer data between objects that not need to know about their internal structure.
However i would prefer to have this implementation packed on their own components, and the system just get to the wrapper itself with the typepack as external library,  not so many developers are familiar with meta-programming, and some people might have different ways to implement that part, by instance going back to boost if this is more familiar.

MSVC is just actually less conformant/less complete in its C++11/14 parsing/implementation. (The fact that it builds with clang/c2, which uses the MS codegen but the clang frontend is a pretty good clue here) Filed the issue initially over email to the VC++ team, waiting for a reply to see if that's sufficient or if it should go on MS Connect.
While those are provided for library consumer use, they are intended primarily for internal use: they're not exposed on any "public" APIs (clientkit, pluginkit, or their relatives)

ah Ok, i read somewhere that some changes 14 where reinforcing some characteristics on the language, i guess that might break a lot of things.
About the libraries, seeing from this point of view probably looks ok, but not sure if it's correct based on the fact that the whole platform is open and public, might limit the capability to improve the core itself.

Yeah, no, I'm pretty confident this isn't a correctness issue - given that clang and gcc in multiple versions handle it fine, as well as the EDG frontend used for intellisense, it's just the C1XX frontend that stumbles.
I guess I don't get what you're asking for in the rest of your question/statement. Template metaprogramming is a powerful tool but also challenging to use. It's not expected that many will directly modify that code, and that's OK: it's still substantially simpler than the code it replaced (boost mpl and fusion), and the current container wrapper code is much cleaner and simpler than earlier iterations, and less error prone than one-off duplicate copies all through the codebase. Unfortunately, all the alternative libraries (to typepack/mpl/fusion) are not suitable for the project because they don't even begin to compile on the latest MSVC, much less MSVC 2013, (and many need very recent clang and gcc as well) hence why TypePack started as/exists as a simplified subset/cousin of Eric Niebler's "meta" (the closest fit to the requirements) retaining similar semantics and design but simplifying and adding MSVC support. In any case, this is a bit off-topic from the internal compiler error in MSVC 2015. While we do want to be able to build there at some point, it's not currently a supported platform for building the core, and I imagine it won't be before Update 2 is released, unless MS gets back to me with a workaround for this ICE. (You can still build against the library with 2015, thanks to the C API)
Can you isolate the C3203 errors to add to the reproduction test case? https://github.com/rpavlik/typepack-msvc14-ice

Yes, that's a bit off topic from the compiler error, but is coming along within a more general task on platform code inspection and wider scale adoption exploration.
Thanks you for your support.

Unfortunately, the compiler bug still exists in Visual Studio 2015 Update 1 as well.

Could someone isolate the C3203 errors? The C1001 errors in typepack isolated and reported here https://connect.microsoft.com/VisualStudio/feedback/details/2154536 have fixes checked in, so presumably (hopefully) the fix will be in 2015.2, and I'd like to be able to use 2015.2 for the whole thing.

Added the need info label because I don't have any information on the C3203 errors to either trace them down to an OSVR bug (unlikely given that it works on many other compiler frontends), or file an issue with Microsoft Connect to get a fix in the next MSVC like for the other issue.

OK, there is now a CTP of Update 2 available - that should include the fix for the C1001 errors: https://www.visualstudio.com/en-us/news/vs2015-update2-vs.aspx would be good to find out if any other issues remained with building in 2015.
Note that TypePack itself is modeled on/a subset of Eric Nieblier's meta which is a part of his range_v3 work, which is apparently being used by the MSVC team as a test case, so chances are good that we'll get something usable with 2015 update 2. Whether we'll get something that can compiile in both 2013 and 2015.2, I don't know, but testing and bug reporting (to Microsoft) is the best way to attempt that.


Update honeycomb images to use the jsonapi format

Coverage increased (+0.04%) to 88.17% when pulling 507ce7da824783104cbcc64782434f8ca88844bf on honeypot-jsonapi into 101e1afe5b6041ac3e7c3f6c86dea0fc6480eec0 on master.


Memory Leak

Memory is allocated
https://github.com/IBM/data-broker/blob/5271f74482fa5334dc2f5b13f861b4421f08bdb8/backend/transports/double_buffer.c#L44
Potential leak of memory pointed to by 'buffer '
https://github.com/IBM/data-broker/blob/5271f74482fa5334dc2f5b13f861b4421f08bdb8/backend/transports/double_buffer.c#L50

Thanks for the report. Appreciated. I'll look into it.

I checked the code and it's correct that there's is a bug in that function. However, it's not a memory leak since the allocated memory is correctly freed by calling dbBE_Transport_dbuffer_free().
The actual problem is that there is no check for the failed malloc in line 44. This in turn would cause a segfault in line 46, when buffer is dereferenced. A fix and PR are in progress.

fix merged. Will close issue. Please re-open if you think it's not addressed properly.


runtime: squeeze Hchan into 64 bytes

Currently Hchan is 12 words (on 64-bit machine). It would be nice to squeeze it into 8
words. First it's just memory consumption reduction, and second it will be exactly cache
line (and due to malloc alignment, it will be also cache line aligned) which will
speedup contented synchronization - ping-pong only one line instead of two + no false
sharing.

elemtype goes away as it is not needed.
dataqsiz is a derivative of qcount/sendx/recvx, so it just goes away as well.
recvq/sendq can be reduced to 1 pointer to a head of doubly-linked cycle of waiters.
This gives exactly 8 words.

FTR, here is a CL that lowers OCAP/OLEN for chans into runtime calls (necessary to
remove dataqsiz):
https://golang.org/cl/54570046/

Too late for Go 1.5.


Fixed translation for hue

Fixed a translation spelling for the hue component.

Lokalise is used to provide translations.
More info on adding translations can be found here:
https://developers.home-assistant.io/docs/en/internationalization_translation.html


Recommend removing clean.ts in favor of del plugin

[ X] feature request
Current behavior
Currently there is a clean.ts script that is being maintained by the angular-seed project.  Recommending that we may want to remove it and use del npm plugin instead to cut down on maintenance going forward.
Expected behavior
Recommend updating the seed project to use del npm package below:
del npm plugin - https://www.npmjs.com/package/del
del files and folders example - https://github.com/gulpjs/gulp/blob/master/docs/recipes/delete-files-folder.md

We were using del in the past but it doesn't provide big enough abstraction on top of rimraf in order to switch to it and introduce additional dependency (it relays on rimraf).


[IJ Plugin] Tweak navigation from GraphQL to generated code

The plugin adds elements from the generated models when invoking "Go to declaration" on GraphQL fields, fragment spreads and fragment definitions. In most cases this isn't very useful, and can be an annoyance since going to the GraphQL declaration now requires 2 clicks instead of one.
A setting (defaulting to false) should be added.
Thanks @Demoritas for the heads up!

The fix is included in version 4.0.0-beta.6 of the plugin.


chore(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3197

Questions ?
Please refer to the Backport tool documentation


Added Insertion Sort in Golang

Please do merge this PR and label it hacktoberfest-accepted

PR accepted @francojoem
Successfully merged to HacktoberFest_22 repo.
Need to Star this repo incase not done.
🚀🚀👍


[camel-main] Foundation: BeanTest.parameterTypes  fails with the current Camel main

Important!
Test BeanTest.parameterTypes is disabled. Failure seems to be caused by the fact that fallback converter does not convert the body to String. Problem was first detected when following issue was fixed in >Camel https://issues.apache.org/jira/browse/CAMEL-15105
It seems This issue might be similar as https://issues.apache.org/jira/browse/CAMEL-19160

It's related to this change:
https://github.com/apache/camel/commit/cc58a9ac1df9d4235b6c6200eb976a75f3566756
The fix is very simple. You just need to append .class to the type argument. E.g:
from("direct:parameterTypes")
    .to("bean:parametersBean?method=parameterTypes(String.class)");

thanks for the solution, I missed that.


[DateTimePicke] added prop type

✅ Closes: #364

:tada: This PR is included in version 9.9.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Custom tasks don't work on github.dev

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: VSCode Web 1.70.2 (github.dev)
OS Version: Chrome (MacOS)

Steps to Reproduce:

Using a web extension with a pseudoterminal task provider, e.g. https://github.com/thegecko/web-extension-tasks
Install into vscode.dev and observe the task executes as expected
Install into github.dev and observe the task system hangs

Relates to: https://github.com/microsoft/vscode/issues/157112

This issue seems to have gone away and we now observe behaviour parity with vscode.dev


Cleanup & simplifications

Use NuGet central package management (https://learn.microsoft.com/en-us/nuget/consume-packages/Central-Package-Management)
Obsoleted AssemblyInfo.cs
Reorganized files in test project
Use code analysis in test project

Thanks a lot for reviewing @jonorossi ✌


Using HtmxViewMixin without defining get or post methods raises exception

The HtmxViewMixin defines get and post methods to handle HTMX requests. If the request is not HTMX it calls the super's get or post method. This will raise an exception if the get or post methods are not defined in a super class which is entirely possible.

We can solve this by checking if super has get and post defined before calling it and return HttpResponseNotAllowed if it is defined. Though I'm wondering if we should go with a different approach. We can route HTMX request to separate methods by overriding the dispatch method.


[SPI] Fix for MOT object contentName parameter encoded in utf8

The length was omitted when parsing the utf8 string, causing the names to append garbage data before. This PR simply pass in the calculated length to fromUtf8()
The SPI logos would not display at all in the GUI before, but after this fix they appear.

Thanks!


Update Composer dependencies (2022-01-15-00-03)

Visual regression test passed!
View the visual regression test report


Fix build scan Sarif format and remove redundant scan context message

[ ] All tests passed. If this feature is not already covered by the tests, I added new tests.
[ ] All static analysis checks passed.
[ ] This pull request is on the dev branch.
[ ] I used gofmt for formatting the code before submitting the pull request.

should fix https://github.com/jfrog/jfrog-cli-core/issues/704

LGTM


Customizable gender field

Resolve #3803
Work in Progress, added internal logic and the list and dialog, the genders/gender fields still need to be replaced in the different code positions.
Needs an updated versions of:[

https://github.com/OpenSlides/openslides-autoupdate-service/pull/1025
https://github.com/OpenSlides/openslides-backend/pull/2485

Add subscription, relations, and AppConfig. Open are still filter, sort and import, and perhaps delete prompt.

Needs an updated versions of:

https://github.com/OpenSlides/openslides-meta/pull/116
https://github.com/OpenSlides/openslides-backend/pull/2485

It seems that there is a test failing because of wrong imports in a newly created .spec.ts file.

The autoupdate service gives following:
Error: getting next message: creating first time data: create keys for
keysbuilder: load needed keys: restricting data: getting restiction mode for
gender/A: collection "gender" is not implemented, maybe run go generate ./...
to fetch all fields from the models.yml
I suppose that the autoupdate service needs to implement the new 'gender' collection in the restrictor.
Running go generate doesn't help-

Updated the subscription and disallow edit/delete for gender.id <= 4.

This pr also needs:
https://github.com/OpenSlides/openslides-autoupdate-service/pull/1025

changed requirement in the CR:

the checkboxes should stay where they are
if a default gender is selected then the delete option is (with an explanatory tip) disabled

Worked on the multiselect CR, I have excluded the edit /delete actions, disable the menu delete button if default genders are selected, and add a tooltip then.

Some code cleanup and change of the title.

please leave merging to @rrenkert  or @Elblinator
This PR and some others need merging in the correct order


[WIP] Fixes to Table viewer

DO NOT MERGE YET, WORK IN PROGRESS
Fixes include:

Minor PEP8 fixes to MultiDim
Fixed MultiDim generating multiple thumbnails for the same extension.
Fixed MultiDim not recognizing BinTableHDU as a table.

This is for ejeschke/ginga#370

@ejeschke , I got your table viewer to work. I separated my changes into multiple small commits so you can follow them easily, I hope. As of 826bb38, everything seems to work, except I can't figure out how to get Header to change back to table header when I switch back to it via Thumbs. I'll continue working on this tomorrow.

Sounds great, I'll check out what you've got here so far.

everything seems to work, except I can't figure out how to get Header to change back to table header when I switch back to it via Thumbs

We'd have to take a careful sweep over the plugins to make sure they do sensible things if an AstroTable is set in the channel.  For example, it's not clear to me what Pan and Header should do--maybe be blanked?

Your version is working for me.

@ejeschke , a FITS table extension has its own header, so ideally Header would work for that as well. But I still cannot figure out why Header is not behaving properly for table (it works fine for both image and primary header extensions). I find that when I select an extension from Thumb or Contents (thus bringing data already in memory back to focus), the following happens:

For primary header, Header.set_header() is called once. All is good.
For image extension, Header.set_header() is called twice with the same AstroImage. A bit weird but it still works.
For table extension, Header.set_header() is called twice, once with the correct AstroTable, but then overwritten by a second call with AstroImage. When this happens, the displayed header is not the header of the table. I cannot figure out where the second call is coming from.

As for the other plugins, Histogram is throwing out some error about channel.viewer.onscreen_message. So, I agree that we have to indeed look carefully at how the table interacts with all the other plugins.
As for Pan, yes, it should be blank, or at least do nothing without emitting error message about panset_xy.

@pllim, under the focus_cb method in Header.py, change:
        image = channel.fitsimage.get_image()

to
        image = channel.get_current_image()

As for the other plugins, Histogram is throwing out some error about channel.viewer.onscreen_message

For now you can change that line to:
channel.viewers[0].onscreen_message

and that should take care of the error.

Also, under set_sortable_cb, probably should make the same change as in focus_cb.

Thanks, @ejeschke ! Your suggestions seem to do the trick. It is mostly working now but I still want to look at the rest of the plugins before I call this a success.

I've got some changes I want to make as well.  Can I merge your changes and then we'll go from there?


❓ Question: Slow Upload Speeds

🙋♂ Question
Hello!
I'm currently enjoying using your service but have encountered a technical issue regarding upload speeds. My setup includes a high-capacity server with a 10G Ethernet port and symmetrical 500Mb/s internet connectivity. When uploading a large 10G file, the process only utilizes about 20% of the server's available download bandwidth. This is puzzling because the same server reaches its full upload capacity of 500Mb/s when downloading files. Both the server and the client have similar internet speeds.
I've experimented with different compression settings (0 and 9) but haven't noticed any impact on the upload speed. The server has over 100GB of available RAM and the CPU usage hovers around 5%. Interestingly, when I use multiple tabs for simultaneous uploads (e.g., uploading the 10G file twice), the bandwidth usage scales accordingly (e.g., 20% x 2).
Is there a possibility that Pingvin is intentionally throttling connectivity to balance the network performance for other clients? Any insights or settings recommendations to address this would be greatly appreciated.
server side when client is uploading:

when downloading:

I don't really have network knowledge, but this might happen because the file get uploaded in small parts. Feel free to create a bug report then I can look into it further :)

Thanks,
Not going to raise the bug report yet, I'll make some other tests and see if I can find the bottleneck.
Cheers!

Okay, if you think it's related to Pingvin Share feel free to create a bug report :)


[Manifest] Remove 'Control' as a child element of 'CustomTab'.

Addresses issue #927.

Docs Build status updates of commit 7cfeb4c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/manifest/customtab.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[BUG]Can't Compile

Basic info

OS: [Linux]
VSCode version: [v1.57.1]
Extension version: [v1.12.6]

Further Information
When i want compile sp file, i have error:
fatal error 184: cannot write to file: "compiled//path/shop_buy_granade.smx"

это не баг, тебе нужно создать папку compiled вручную

Post your "sourcepawn.outputDirectoryPath"

@dran1x I've talked with @deff-dev on Discord and I think the next release will fix this issue. I'm waiting for them to test it 👍
The path was correct.

@dran1x I've talked with @deff-dev on Discord and I think the next release will fix this issue. I'm waiting for them to test it 👍
The path was correct.

Ah alright

@deff-dev
Hi again, I wasn't able to reproduce the issue on MacOS. I don't have a Linux VM ready to test this, however, if it works on MacOS, there is no reason it wouldn't run on Linux.
Please test the new version of the extension (v1.12.7) from the marketplace, which will be published in a few hours.
Thanks


minimal UI and clean css..

What does this PR do?
Improves Readability of the code - CSS is written in the css (popup.css) file instead of the js file (inline css -

Can you please refer #35

We have updated the popup page so closing this.


[Bug] [plugin] can not load env/dolphinscheduler_env.sh

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
set export SEATUNNEL_HOME="${SEATUNNEL_HOME:-/opt/seatunnel-2.3.5}" in bin/env/dolphinscheduler_env.sh task log report ：

/tmp/dolphinscheduler/exec/process/default/13865082550688/13865121489568_4/21/23/21_23.sh:行4: /bin/seatunnel.sh: 没有那个文件或目录

What you expected to happen
should execute correct exec seatunnel command
How to reproduce

set export SEATUNNEL_HOME="${SEATUNNEL_HOME:-/opt/seatunnel-2.3.5}" in bin/env/dolphinscheduler_env.sh
start server bin/dolphinscheduler-daemon.sh start standalone-server
create project and seatunnel task
run the job

Anything else
I found the pr Add source env list in common.properties，but not auto load dolphinscheduler_env.sh file, only load the  specify explicitly  env files
Version
3.2.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

这里我直接配置了seatunnel的全局环境变量，这样配置文件在读取到${SEATUNNEL_HOME}时就正常了

这里我直接配置了seatunnel的全局环境变量，这样配置文件在读取到${SEATUNNEL_HOME}时就正常了

@blueflower1 你这是什么版本，我用的3.2.0和3.2.1都需要手动在common配置中配置。 以前3.1.x版本，没有这个配置，可以直接使用环境变量

我使用的是3.2.1，我是直接在/etc/profile.d/env.sh中配置了seatunnel的环境变量

seatunnel和你一样也是2.3.5

你这样配置应该是使用了默认的系统环境变量。 按照文档说在 dolphinscheduler_env.sh 中配置，没有被配置到系统环境变量中，没有读取到

You need to use absolute paths instead of relative paths.

You need to use absolute paths instead of relative paths.

how to config?

Same issue : the bin/env/dolphinscheduler_env.sh is not working.


magic python functions in ndarray.py

I have been reading the code of mxnet for some further understanding of the platform. In optimizer.py, I saw following code:
"from .ndarray import sgd_update, sgd_mom_update, adam_update, rmsprop_update, rmspropalex_update"
However, the python implementations of these functions are absent in ndarray.py. I am sure these functions are written in c but these should at least some cython-wraps for them, but I only get "NNVM_REGISTER_OP(sgd_mom_update)" when scanning the code using ack.

Try to read some codes here: https://github.com/dmlc/mxnet/blob/master/python/mxnet/_ctypes/ndarray.py#L155

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


add no data to legend using CSS

This is sort of forced in using CSS, rather than programmatically getting Vega to add another row with the exact same font, spacing, sizing, color, etc. I'm sure we can do it with Vega with more research

the following json creates the correct legend on the Vega Editor but when converting to vega-react syntax the gray circle disappears.

{
  "$schema": "https://vega.github.io/schema/vega/v5.json",
  "description": "A basic bar chart example, with value labels shown upon mouse hover.",
  "width": 10,
  "height": 10,
  "padding": 50,

  "data": [
    {
      "name": "no-data",
      "values": [
        {"category": "No Data", "amount": 1}
      ]
    }
  ],

  "legends": [
    {
      "fill": "gray",
      "encode": {
        "symbols": {"enter": {"fillOpacity": {"value": 0.25}}},
        "labels": {"update": {"text": {"field": "value"}}}
      }
    }
  ],

  "scales": [
    {
      "name": "gray",
      "type": "band",
      "domain": {"data": "no-data", "field": "category"},
      "padding": 1
    }
  ]
}


Add documentation on how to get remaining countdown if we want to display it to users

Hi,
Great lib and great job, thx !
After some try and errors I finally found that this :
Offline.reconnect.remaining

Combine with the tick event was the way to go if I wanted to display a countdown to the user before the next reconnect attempt. You should definitely mention it somewhere in the documentation as this is a pretty common usecase !
Cheers !

@HadrienPierart Do you have code-snippet you can share using 'tick' ?
I can not figure out a way to use it.

Well, it's actually just a matter of listening to the various events. tick works the same :
      Offline.on('up', connectionUp);
      Offline.on('down', connectionDown);
      Offline.on('reconnect:tick', tick);

Where the second param is a function handling the event.
If you are doing an angularjs application, do remember to unregister the event listener in the onDestroy and to manipulate your scope in a $apply or evalAsync when in an event handler !

@HadrienPierart - here is my working code snippet on using TICK method
<script>
    $(function(){

        var
            $online = $('.online'),
            $offline = $('.offline');

        Offline.on('confirmed-down', function () {
            $online.fadeOut(function () {
                $offline.fadeIn();
                  console.log('Offline state: ',Offline.state);
                  console.log('Offline remaining: ',Offline.reconnect.remaining);
            });
        });

        Offline.on('confirmed-up', function () {
            $offline.fadeOut(function () {
                $online.fadeIn();
                  console.log('Offline state: ',Offline.state);
                  console.log('Offline remaining: ',Offline.reconnect.remaining);
                  Offline.on('reconnect:tick', tick);
            });
        });

        function tick(){
            console.log('Offline remaining: ',Offline.reconnect.remaining);
            console.log('Offline state: ',Offline.state);
        }

    });
</script>


Transpiler?

Hallo,
is it allowed to use .Net in a transpiler?
We planned an opensource rust transpiler, which is for example to translate .NET into Rust.
We planned to transpile the complete .NET SDK to Rust and use this Rust .NET SDK in next Projects.
Is this allowed or not?
We will start our transpiler project after you allowed it.
If you have any further Questions, please ask.
Kind regards
Wolfgang

Please see https://github.com/dotnet/core#licenses for links to the various licenses the project is under.
CC @terrajobst

Thank you
Licenses allow modifications, so I think itˋs allowed.
as soon as we have a good base, we will share it. you can use the knowledge for project Verona.


Clean up the handling of context and public root URLs in our tests

Calling this done, we cleaned it up by removing context URLs entirely.


[Auto-generated] Update dependencies

Update dependencies
Automatically created PR from ci/dependency-updates.
For more information see the "Dependabot updates" workflow.
To-do

[ ] Check that the diff is sensible, and that tests and builds pass with the new dependency versions.
[ ] Make sure that the PR is squash merged, with a sensible commit message.

Codecov Report

Merging #15 (4445659) into main (9c4b41d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   18.91%   18.91%
=======================================
  Files           8        8
  Lines         148      148
=======================================
  Hits           28       28
  Misses        120      120

Flag
Coverage Δ

linux
18.91% <ø> (ø)

windows
?

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

Continue to review full report at Codecov.

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


Obey the usual rules of SSL server name verification when using a private PKI

This line does not do what it thinks it does. If it did do what it thinks it does, then it would be insecure. As it is, it's pretty damn insecure. Actually it's all pretty insecure.
The intended behaviour would not verify the server name when a private PKI is being used. Actually, that's not true, it would not verify any PKI, private or not, if the root certs are being explicitly provided. This means that the TLS does not protect against MITMs if you have any certificate from the PKI (basically removing the P from PKI and making it a rather clumsy shared secret).
The current behaviour of setting ServerName to an empty string causes the ssl to be verified against the Host passed to the transport (as far as I can tell). This is exactly what #6084 is meant to solve. But this breaks it. Here's etcdctl connecting to a cluster where the certs are signed with the SRV domain record, but not the individual records:
▶ etcdctl -D etcd-test.XX.com --key-file ./etcd-test-key.pem --cert-file ./etcd.pem -ca-file ./ca.pem  ls /
Error:  client: etcd cluster is unavailable or misconfigured; error #0: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-0.stg.eu.YY.com.
; error #1: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-4.stg.eu.YY.com.
; error #2: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-2.stg.eu.YY.com.
; error #3: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-3.stg.eu.YY.com.
; error #4: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-1.stg.eu.YY.com.

error #0: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-0.stg.eu.YY.com.
error #1: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-4.stg.eu.YY.com.
error #2: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-2.stg.eu.YY.com.
error #3: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-3.stg.eu.YY.com.
error #4: x509: certificate is valid for etcd-staging.XX.com, etcd-test.XX.com, not etcd-1.stg.eu.YY.com.

Where there is a SRV record etcd-test.XX.com pointing to etcd-[0-4].stg.eu.YY.com

@bluepeppers CI won't pass until clearing the error "Expected commit title format '{", "}: '". Thanks!
Will this cause breakage? That was there to retain old behavior where anything signed by the given ca could be totally trusted (aka the "clumsy shared secret" model). /cc @xiang90

Well, as per my log, that wasn't the behaviour before. If all explicitly passed root CAs are trusted, then I shouldn't be able to get the x509 errors above.
This would correctly implement the behaviour of #6084, so whatever breakages that might cause, this would cause. I think it would cause problems for people using SRV discovery with certs signed for (using my log as an example) signed for the YY.com records, but not the XX.com records. I can't speak to how common that is (the docs on coreos.com don't really mention which name your cert should be fore, though https://coreos.com/os/docs/latest/generate-self-signed-certificates.html takes a rather generous approach to CN assignment)

@lclarkmichalek Sorry about the procrastination on this one. If you fix up the conflict (i.e., remove the new deprecated warning that just went in), this can be merged into master once the 3.1 release branch is cut (probably tomorrow).

Fixed the merge conflict

@lclarkmichalek looks like it's failing on an orphaned import? Squash into a single commit and s/"pkg/transport/listener.go"/"pkg/transport"/ too? Then it should be good to merge. Thanks!

Current coverage is 64.09% (diff: 100%)

No coverage report found for master at 017ea3d.

Powered by Codecov. Last update 017ea3d...70dbca3

Rebased, reworded etc 👍

lgtm. Thanks!


Allow prefixing webjars path for redoc

Describe the feature
Currently webjars path goes to /webjars/redoc. Please allow prefixing it with a string so that swagger docs don't break when Javalin app is deployed with reverse proxy.
Additional context
If the Javalin app is deployed with reverse proxy which prepends a context path to the URL say /fooservice then /redoc docs break because they go to root / instead of /fooservice. Allowing prefixes for webjars will allow us to redirect it to /fooservice/webjars and this not break redoc docs.

Custom base path impl is available for ReDoc plugin in alpha.6:

https://github.com/javalin/javalin-openapi/releases/tag/5.3.0-alpha.6


bugfix: chunk compression should use actual size of chunk

The actual size of a chunk can be smaller than the one reported in the
VolumeChunkSpecification, if the requested chunk was not fully contained
in the volume.

I signed the CLA.

I added the commit e-mail address as alternate e-mail address.


Address SAM issue #1928

LandBOSSE <IP_ADDRESS> is latest PyPi package and supports Python 3.7 only - numpy 1.4.5 requirements do not work with later Python versions. Later LandBOSSE version 2.5.0 has to be installed locally if used with later versions of Python. install_python.ps1 updated to work with Python 3.7 since get-pip.py requires 3.8 or later.

This worked for me on SAMOS, but I still get a Python install callback error when trying to run it from SAM private.

@mjprilliman , please delete you SAM-private\build_windows folder and rebuild SAM-private and test... that worked for me ;-)

Build and tested with SAM-private on RHEL 8.6

Working as universal binary app on MacOS 15.1


Problems generating python-class

Trying to install daphne and running into issues with the python-class generating options. If I try
lein run graph -i programs/arithmetic_circuit.daphne -o output.json

or
lein run desugar -i programs/arithmetic_circuit.daphne -o output.json

both work, and produce the correct output, but if I try
pipenv run lein run python-class -i programs/arithmetic_circuit.daphne -o autogen_arithmetic_circuit.py

I get autogen_arithmetic_circuit.py created, but with only "" in the file. If I try:
lein run python-class -i programs/convolution.daphne -o autogen_convolution.py

I get the error:
Syntax error (NullPointerException) compiling at (/private/var/folders/fr/c2m_v4ss6kjcbgzbqfr_z5fw0000gn/T/form-init14751265169047276845.clj:1:126).
Cannot invoke "java.util.Map$Entry.getValue()" because "e" is null

Full report at:
/var/folders/fr/c2m_v4ss6kjcbgzbqfr_z5fw0000gn/T/clojure-14375094718594195756.edn

I worry that I might be doing something pretty basic wrong...

Thanks for reporting! This should be solved with the latest commit. Please open a new issue if there are still problems.


Contribute cloudant plugin

Add one more plugin to detect Cloudant tokens.

I didn't realize this in the other PR, but it is a little confusing that the numbers referenced in your commit messages map to unrelated issue/PR #'s, presumably b/c they're internal numbers.

Yes, we use Github Enterprise internally, which might mess up with the PR. I can remove the PR id from commit message to avoid confusion.

Pushed again with updated commit message.

Retrigger Travis build.


fix: move analytics token to secrets [FTL-7837]

:tada: This PR is included in version 1.17.2 :tada:
The release is available on:

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


heroku application error

uploading a file of any size causes a Heroku application error. to replicate, upload a file of any size with any browser.
related tickets: #242 #251 #253

@yegor256 let's prioritize this ticket for public beta testing capabilities.

@k0r this is what I see in logs:
sock=backend at=error code=H18 desc="Request Interrupted" method=POST path="/doc/write" host=beta.nerodesk.com request_id=1c4ce399-2ea7-4ac0-a8c3-579c038aaf67 fwd="<IP_ADDRESS>" dyno=web.1 connect=1ms service=428ms status=503 bytes=473

And that's it. No exceptions. No idea why it's happening... investigating now.

@k0r this is what Heroku says about H18 error: "Either the client socket or backend (your app’s web process) socket was closed before the backend returned an HTTP response. The sock field in the log has the value client or backend depending on which socket was closed." see https://devcenter.heroku.com/articles/error-codes#h18-request-interrupted

@k0r and this is happening not every time. Sometimes I can upload a file without problems. I think I know what's going on...

@yegor256 tested again and got the same result.

@k0r this seems to be the problem: https://github.com/yegor256/takes/issues/242 working on it now

@k0r try to upload a very small file, you won't have an error. The problem happens when the file is long enough so that Heroku drops connection with our app

@yegor256 same result with a 1mb file.

@k0r try 1Kb file

@yegor256 works.

@k0r yeah, the problem is in the size of the file... and it has something to do with heroku... I'm looking into it now

@k0r this is what I see in logs:
2015-05-02T02:37:32.090250+00:00 heroku[router]: sock=backend at=error code=H18 desc="Request Interrupted" method=POST path="/doc/write" host=beta.nerodesk.com request_id=5d8507ac-66d5-4919-a824-1f34f2bba521 fwd="<IP_ADDRESS>" dyno=web.1 connect=1ms service=319ms status=503 bytes=473
2015-05-02T02:37:32.058120+00:00 app[web.1]: d88224f com.nerodesk.om.aws.AwsDoc: com.jcabi.s3.retry.ReOcket@82bf62fd written

This means that we interrupt the request and then immediately after that save the file to S3...

@yegor256 your task, 30 mins is budget

@rultor release, tag is 0.3.18

@rultor release, tag is 0.3.18

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.3.18

@yegor256 Done! FYI, the full log is here (took me 8min)

@k0r try now. I can upload files of any size, but those that are bigger than ~100Mb take too much time and heroku sometimes drops connection. It's a known issue in Takes: https://github.com/yegor256/takes/issues/248 I'm not yet sure it's possible to resolve it.. Maybe we'll have to find some workaround.

@yegor256 can't login: oops, something went wrong! java.lang.AssertionError: HTTP response status is not equal to 200: 400 Bad Request

@yegor256 nearly 3 months of development the only thing that works is logging in with facebook.

@k0r confirm, there is a bug in takes: https://github.com/yegor256/takes/issues/249 (fixing now)

@k0r should be fine now, try again

@yegor256 same thing.

@yegor256 logging in is fixed.

@k0r please elaborate on "same thing". what exactly are you doing and what is happening?

@yegor256 logging in is fixed.

@yegor256 oops, something went wrong! org.takes.HttpException: GET http://beta.nerodesk.com/doc/write

@k0r what file size you're trying to upload?

@yegor256 that was from a 150mb file.

@k0r yeah, that's too much for now. I think everything below 50Mb should work fine. Try something smaller, below 50Mb

@yegor256 50mb uploads without a problem.

@k0r yes, that's what I'm saying... I'm not yet sure whether this problem is solvable in general. we have to go through a 10-million byte stream and find the location of a string that stops the file content. that's what takes time... I'll think about some optimization in https://github.com/yegor256/takes/issues/248

@yegor256 this means that editable per file limit parameters isn't currently possible with takes.

@k0r I think we can close this ticket, right?

@yegor256 thanks.

@yegor256 Many thanks! 30 mins were added to your account in Transaction ID 000-8f26d565 (task took 725 hours and 13 mins); +30 added to your rating, current score is: +6991


Can't select Python environment

This issue has been moved from a ticket on Developer Community.

I was trying to create a python web project. But as pythn 3.8 was not supported by VS and i have anaconda and python(which was installed by VS) project creation failed. When i tried to change the python environment from" Tools-->Python-->Python Environments " the application suddenly stopped responding and automatically closed. I am facing this issue related to all type of python project.

Original Comments
Feedback Bot on 9/29/2020, 08:06 PM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

Hi,
I also had this issue while trying to open Tools --> Python --> Python Environments on my VS2017
I am using VS2017 Professional version 15.9.30 on Windows 10
I noticed this might be because of my newly installed Anaconda. I remembered not having this issue before that.
Thanks,

Hi,
I also had this issue while trying to open Tools --> Python --> Python Environments on my VS2017
I am using VS2017 Professional version 15.9.30 on Windows 10
I noticed this might be because of my newly installed Anaconda. I remembered not having this issue before that.
Thanks,


Display tooltip on app status tag and app version on side navigation when the text is truncated

Summary
Add tooltip to app status tag and app version which displays the whole text when it is truncated (because it is too long to be fully displayed).
💬 Description
Currently, when the value provided for the app status tag or the app version is too long for it to be displayed fully and it is truncated (e.g. see image below), there is no way for the user to determine what the whole value is.

A tooltip needs to be added so when the user hovers over the status or version, it shows them the whole text.
💰 Use value
This will allow users to determine what the whole of the text for the app status or app version is when it is not fully visible due to truncation.

Related: increasing max-width for app status tag - #550

Tooltips will no longer be required as the status and version text are being changed to wrap onto multiple lines instead of being truncated as part of #550


CVEs in DD Lambda Extension Layer

Our CVE analysis tools is throwing a bunch of warnings for CVEs coming from the DD Lambda Extension Layer:

Can you guys update the dependencies to handle those?

Hi @luneo7, thanks for reaching out. We currently have a github action set up to run nightly to check for vulnerabilities using a variety of tools, yet we haven't seen any of these show up on the current version of the extension.
In order to investigate, can you please share with us which CVE analysis tool you are using, how you're calling it, and the version of the extension you are analysing? The most recent version of the extension is v65.


Problems while executing DUCE.ipynb

Trying to login
Login Errorcannot unpack non-iterable NoneType object
IDownloadCoupons: 100%|██████████| 25/25 [00:00<00:00, 32676.10it/s]
Course Vania: 100%|██████████| 30/30 [00:39<00:00,  1.31s/it]
Real Discount: 100%|██████████| 36/36 [00:49<00:00,  1.37s/it]
Discudemy: 100%|██████████| 48/48 [00:58<00:00,  1.22s/it]
Tutorial Bar: 100%|██████████| 36/36 [00:53<00:00,  1.48s/it]
Udemy Freebies: 100%|██████████| 24/24 [01:17<00:00,  3.22s/it]
0 SQL Masterclass: SQL for Data Analytics https://www.udemy.com/course/the-complete-sql-masterclass-for-data-analytics/?couponCode=FREMAYSQL
Traceback (most recent call last):
File "", line 536, in main1
auto(links_ls)
File "", line 424, in auto
purchased, amount = course_landing_api(course_id)
File "", line 296, in course_landing_api
r = s.get(
NameError: name 's' is not defined

Fixed thank you so much
Can you try it https://github.com/techtanic/Discounted-Udemy-Course-Enroller/tree/master/G-COLAB#readme
If it works proceed to close the issue


Update Field of Study

The Field of Study (Faculty) listing used is fairly old. There is newer listings which is suppose to be used. This listing must be present at the start of the Exchange Year 2017, which means latest on September 1st, 2016.
Note, that existing Offers must also be corrected to be compliant with the new listing.
Unfortunately, the PDF file cannot be uploaded :-(

Moved from milestone 1.2.1 to 1.2.0, as it will affect the API.

Field of Study Values.pdf

Since IAESTE has not been answering my communication about this feature, it is put on hold. If no communication has been received until the end of August, the task will be closed.

There has been no reaction from IAESTE on this matter. Will close the task as invalid.


Keep getting Pixels not aligned to stride of each line

Hi there,
I am getting Microsoft.Azure.Kinect.Sensor.AzureKinectException: 'Pixels not aligned to stride of each line' exception when i try to call .Color.GetPixels() method.

To Help code:
private void TryGetDepath()
{
var device = Device.Open();
device.StartCameras(new DeviceConfiguration
{
CameraFPS = FPS.FPS5,
ColorResolution = ColorResolution.R1080p,
DepthMode = DepthMode.WFOV_Unbinned,
WiredSyncMode = WiredSyncMode.Standalone,
SynchronizedImagesOnly = true
});
var calibration = device.GetCalibration();
var transformation = calibration.CreateTransformation();
Capture capture = device.GetCapture();
        while (capture == null || capture.Color == null || capture.Depth == null)
        {
            capture = device.GetCapture();
        }

        var colorPixel = capture.Color.GetPixels<abc>();
        Console.WriteLine(colorPixel.Length);
        foreach (var c in colorPixel.Span)
        {
            Console.WriteLine($"{c.a} {c.r} {c.g} {c.b}");

        }

        var depthFromColor = transformation.ColorImageToDepthCamera(capture);
        var depthFromColorPixel = depthFromColor.GetPixels<abc>();
        Console.WriteLine(depthFromColorPixel.Length);

        foreach (var c in depthFromColorPixel.Span)
        {
            Console.WriteLine($"{c.a} {c.r} {c.g} {c.b}");

        }

        var depthPixel = capture.Depth.GetPixels<UInt16>();
        Console.WriteLine(depthPixel.Length);

        foreach (var d in depthPixel.Span)
        {
            if (d > 0)
            {
                Console.WriteLine($"{d} ");

            }

        }

    }

struct abc
{
    public byte a;
    public byte r;
    public byte g;
    public byte b;
}

What image format are you using? MJPEG doesn't have stride.

Found the issue thanks

Found the issue thanks

Hi, can you share the issue solution? Thanks

Hi, can someone share the solution to this issue? I understood it's quite common.

Hi there. I had same problem.
I returned depth image from method which should return color image. Simple misswrote which lead to problem that I then tried to pass returned image into BGRA array which led to "Pixels not aligned to stride of each line error".


Spark Bigquery Job Cannot read from non-public tables

Using the latest branch from master:GHFS version: 1.5.0-hadoop2 and the bigquery connector 0.7.6 from master branch.
Tested on Spark local, Spark 1.6.0/1.6.1 cluster, current dataproc version.
Bigquery dataset permissions for owners, editors, viewers and service account set to "is owner"
Steps to recreate:
When using this example:
https://cloud.google.com/hadoop/examples/bigquery-connector-spark-example
and additional necessary configuration is added
conf.set("google.cloud.auth.service.account.enable", "true");
conf.set("google.cloud.auth.service.account.email", "ABC-compute@developer.gserviceaccount.com");
conf.set("google.cloud.auth.service.account.keyfile", "location.p12");
conf.set("fs.gs.impl","com.google.cloud.hadoop.fs.gcs.GoogleHadoopFileSystem")
conf.set("fs.AbstractFileSystem.gs.impl","com.google.cloud.hadoop.fs.gcs.GoogleHadoopFS")

All works well:
16/07/29 15:19:10 INFO NewHadoopRDD: Input split: gs://tempspark/hadoop/tmp/bigquery/wordcounttmp51001d75-d5d8-4470-865a-0e6e5ea89e26/shard-1/data-*.json[82328 estimated records]
16/07/29 15:19:10 INFO NewHadoopRDD: Input split: gs://tempspark/hadoop/tmp/bigquery/wordcounttmp51001d75-d5d8-4470-865a-0e6e5ea89e26/shard-0/data-*.json[82328 estimated records]
16/07/29 15:19:10 INFO DynamicFileListRecordReader: Initializing DynamicFileListRecordReader with split 'InputSplit:: length:82328 locations: [] toString(): gs://tempspark/hadoop/tmp/bigquery/wordcounttmp51001d75-d5d8-4470-865a-0e6e5ea89e26/shard-0/data-*.json[82328 estimated records]', task context 'TaskAttemptContext:: TaskAttemptID:attempt_201607291519_0000_m_000000_0 Status:'
16/07/29 15:19:10 INFO DynamicFileListRecordReader: Initializing DynamicFileListRecordReader with split 'InputSplit:: length:82328 locations: [] toString(): gs://tempspark/hadoop/tmp/bigquery/wordcounttmp51001d75-d5d8-4470-865a-0e6e5ea89e26/shard-1/data-*.json[82328 estimated records]', task context 'TaskAttemptContext:: TaskAttemptID:attempt_201607291519_0000_m_000001_0 Status:'
16/07/29 15:19:42 INFO DynamicFileListRecordReader: Adding new file 'data-000000000000.json' of size 0 to knownFileSet.
16/07/29 15:19:42 INFO DynamicFileListRecordReader: Moving to next file 'gs://tempspark/hadoop/tmp/bigquery/wordcounttmp51001d75-d5d8-4470-865a-0e6e5ea89e26/shard-1/data-000000000000.json' which has 0 bytes. Records read so far: 0
16/07/29 15:19:42 INFO GoogleCloudStorageReadChannel: Got 'range not satisfiable' for reading gs://tempspark/hadoop/tmp/bigquery/wordcounttmp51001d75-d5d8-4470-865a-0e6e5ea89e26/shard-1/data-000000000000.json at position 0; assuming empty.

etc and the word count and persistence completes.
When the dataset is changed from the public dataset to a private one:
 // Input parameters
val fullyQualifiedInputTableId = "publicdata:samples.shakespeare" to
// Input parameters with valid project, dataset and table
val fullyQualifiedInputTableId = "project:dataset.table"

Shards and records are found but nothing happens and the job stalls at:
16/07/29 16:01:23 INFO NewHadoopRDD: Input split: gs://tempspark/hadoop/tmp/bigquery/wordcounttmp02e3b5bf-5929-4117-a306-45ddadca5896/shard-1/data-*.json[2755 estimated records]
16/07/29 16:01:23 INFO NewHadoopRDD: Input split: gs://tempspark/hadoop/tmp/bigquery/wordcounttmp02e3b5bf-5929-4117-a306-45ddadca5896/shard-0/data-*.json[2755 estimated records]
16/07/29 16:01:23 INFO DynamicFileListRecordReader: Initializing DynamicFileListRecordReader with split 'InputSplit:: length:2755 locations: [] toString(): gs://tempspark/hadoop/tmp/bigquery/wordcounttmp02e3b5bf-5929-4117-a306-45ddadca5896/shard-0/data-*.json[2755 estimated records]', task context 'TaskAttemptContext:: TaskAttemptID:attempt_201607291601_0000_m_000000_0 Status:'
16/07/29 16:01:23 INFO DynamicFileListRecordReader: Initializing DynamicFileListRecordReader with split 'InputSplit:: length:2755 locations: [] toString(): gs://tempspark/hadoop/tmp/bigquery/wordcounttmp02e3b5bf-5929-4117-a306-45ddadca5896/shard-1/data-*.json[2755 estimated records]', task context 'TaskAttemptContext:: TaskAttemptID:attempt_201607291601_0000_m_000001_0 Status:'

without proceeding, or errors in any driver or executor logs. It seems to be a permission or some other problem with private dataset .

This issue is caused by Bigquery not being able to export data between EU and EU-WEST-XYZ buckets. It can be resolved by creating the temp bucket like:
gsutil mb -l EU gs://
as described:
https://code.google.com/p/google-bigquery/issues/detail?id=443


not matching backslash in regex

when I submit my code everything passes except final test.  when debugging I found that the backslash in my regex isn't being matched as it should for some reason.
Challenge Check for Palindromes has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.97 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function palindrome(str) {
  // Good luck!

  str = str.toLowerCase().split("").filter(function(elem){
    return elem.match(/[a-z0-9\-:\/\\()_]/);
  }).join();

  console.log(str);

  return str === str.split("").reverse().join("");
}

palindrome("0_0 (: /-\ :) 0-0");

Note: The instructions ask you to "remove all non-alphanumeric characters (punctuation, spaces and symbols)". This means your filtered string should only have letters and numbers.
If you are stuck on a challenge, please check out the Help Chat first.


add zone.js to implementations list

zone.js has a promise implementation and has passed the promise a+ test. so I make this PR to add zone.js to promise a+ implementation list.

Thanks @JiaLiPassion

@briancavalier , Thank you!


Add Pulumi HCP provider metadata

Adds https://github.com/grapl-security/pulumi-hcp as a provider.
cc: @stack72

Thank you so much for this @christophermaier


 Here is the process of my script:

Launch webcam and init clmtrackr
On button click, drawImage() of webcam in "canvas.image"
Start clmtrackr on "canvas.image" to detect face
Draw in "canvas.mapping"

My problem is that the first time I click on the button clmtrackr detects a face in the upper left corner of the image. If I click on the button again, everything works fine.
Do you have an idea why the first detection is not working as expected ?
Here is my script:
'use strict';

import $ from 'jquery';
import BaseView from 'common/base-view';

export default class Home extends BaseView {
  constructor() {
    super();

    this.el = '.container';

    this.events = {
      'click .js-snapshot' : '_drawImage',
      'click .js-mapping' : '_drawLoop'
    };

    // Define variables
    this._hasUserMedia = this._checkUserMedia();

    this.webcam      = document.getElementById('webcam');
    this.image       = document.getElementById('image');
    this.imageCtx    = this.image.getContext('2d');
    this.mapping     = document.getElementById('mapping');
    this.mappingCtx  = this.mapping.getContext('2d');

    // Init User Media && Canvas
    if (this._hasUserMedia) {
      this._initWebcam();
    }
  }

  /**
   * Initialize Webcam && tracker
   * @private
   */
  _initWebcam() {
    navigator.getUserMedia = this._getUserMedia();

    navigator.getUserMedia({video: true}, (stream) => {
      const object = window.URL.createObjectURL(stream);

      this.webcam.width  = window.innerWidth;
      this.webcam.height = window.innerHeight;

      this.image.width  = window.innerWidth;
      this.image.height = window.innerHeight;

      this.mapping.width  = window.innerWidth;
      this.mapping.height = window.innerHeight;

      // Init clmtrackr
      this.ctrack = new clm.tracker({stopOnConvergence: true});
      this.ctrack.init(pModel);

      // Play stream
      this.webcam.src = object;
    }, (e) => {
      console.log(e);
    });
  }

  /**
   * Draw image from video
   */
  _drawImage() {
    this.imageCtx.clearRect(0, 0, this.image.width, this.image.height);
    this.imageCtx.drawImage(this.webcam, 0, 0);

    this._drawLoop();
  }

   /**
   * Draw mapping from image
   */
  _drawLoop() {
      this.mappingCtx.clearRect(0, 0, this.mapping.width, this.mapping.height);
      this.ctrack.start(this.image);
      this.ctrack.draw(this.mapping);
  }

  /**
   * Check user media
   */
  _checkUserMedia() {
    return !!(navigator.getUserMedia || navigator.webkitGetUserMedia || navigator.mozGetUserMedia || navigator.msGetUserMedia);
  }

  /**
   * Get user media
   */
  _getUserMedia() {
    return navigator.getUserMedia || navigator.webkitGetUserMedia || navigator.mozGetUserMedia || navigator.msGetUserMedia;
  }
}

Thanks a lot !

Hi, it sounds like there might be an issue with the image not being rendered before the clmtrackr tries to detect a face, but that's a guess. It might be that there is a slight delay before the image is rendered to canvas in _drawImage(), but I'd have to see the live example before I could pinpoint what's happening.

I had this issue too at a time. I resolved it by checking if a face is found ( position is evaluated at true) before calling draw method. It seems that it display a default face when you call draw with no face found.


Include examples of how to send form data to e.g. back-ends

Hi guys,
Forge seems to be a very nice and potentially useful project. The fact that the demo apps are included to not only demonstrate Forms but also the application piece (e.g. the routing) is nice.
However, coming from creating monolithic WinForms based applications, I struggle to infer how I would achieve something like adopting the codebase to e.g. do CRUD operations on any kind of storage (a DB, Sharepoint, ...). It is not so much the interaction with these back end that is a problem but rather how to create e.g. a service and then properly register that so the forms can leverage it.
A sort of dummy example here would be extremely helpful.

Hi @KrisJanssen!

The fact that the demo apps are included to not only demonstrate Forms but also the application piece (e.g. the routing) is nice.

I'm glad you found it useful. Though the application "framework" is not something I'd use in production as a dependency because it's stiff and opinionated. As you have noticed, it relies on a DI engine like NInject and other stuff, which is a bad design smell. Instead, I'd take some parts I like and simply add organize those in my project and fit them as needed.
Sadly the same goes for Forge.Forms. It relies on big UI toolkits and is a pain to manage dependencies.

It is not so much the interaction with these back end that is a problem but rather how to create e.g. a service and then properly register that so the forms can leverage it.

Are you using the MVVM pattern for your app? If you're using that, then your form model can raise actions and your ViewModel should handle actions by implementing IActionHandler. It should look like this:
class MyViewModel : INotifyPropertyChanged, IActionHandler
{
    // Suppose we have the DB from somewhere...
    public IDatabase Database { get; set; }

    public void HandleAction(IActionContext actionContext)
    {
        if (actionContext.Action is "save")
        {
            MyFormModel model = (MyFormModel)actionContext.Model;
            this.Database.Save(model);
        }
    }

    ...
}

I'm not sure if this is what you have asked. Please let me know if you need further explanation.


Use culture when convert colors

https://github.com/CommunityToolkit/Maui/pull/552

Docs Build status updates of commit 9eccb99:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/extensions/color-conversion-extensions.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 82029e4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/extensions/color-conversion-extensions.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 3f09c1c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/maui/extensions/color-conversion-extensions.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


[Feature] tree型table有一列checkbox，需要添加一个控制children的checkbox显示隐藏控制项

What problem does this feature solve?
tree型table有一列checkbox，需要添加一个控制children的checkbox显示隐藏控制项
What does the proposed API look like?
tree型table有一列checkbox，需要添加一个控制children的checkbox显示隐藏控制项

感谢你的反馈！我们已经收到你的问题，并将在后续进行处理。
如果有开发者对这个 issue 感兴趣，请在issue下方留言“认领该issue”，留言后开始你的创作提pr给我们！
欢迎大家参与开源共建！我们非常感激每一位贡献者，提交pr后我们将为参与共建的开发者准备一份礼品。
参与共建可参考：https://visactor.io/vtable/guide/Contribution_Guide
与官方开发者交流问题可以下载飞书app扫码加入飞书群：

再次感谢你的支持！
Thank you for your feedback! We have received your question and will deal with it later.
If any developer is interested in this issue, please leave a message "claim this issue" below the issue. Welcome to participate in open source co-construction!
We are very grateful to every contributor. After submitting the PR, we will prepare a gift for the developers who participated in the co-construction.
For participation in co-construction, please refer to: https://visactor.io/vtable/guide/Contribution_Guide
To communicate with official developers, you can download the Feishu app and scan the QR code to join the Feishu group！
Thank you again for your support!


Fix JSONRPCClient.sender(), send_transaction(), new_abi_contract()

Correct import, seemed to throw no test failures

Works when running vs pyethereum develop branch.


fix(cast): do not strip 0x / hex decode message before EIP-191 hashing

Motivation

attempts to use cast hm to create flashbot sig failed, eventually used chisel eval to generate such

chisel eval 'keccak256(abi.encodePacked("\x19Ethereum Signed Message:\n66","'$payload_keccak'"))'

https://github.com/pcaversaccio/white-hat-frontrunning/blob/main/go.sh#L62

fix cast to not hex decode message to hash but use it as it is to create EIP-191 final message "\x19Ethereum Signed Message:\n" + message. length + message which is then hashed

Solution

I think this is fine, however it is a breaking change, and it might be assumed that 0x input is hex decoded like in other commands, we don't really have a consistent behavior in this regard I think

yep it is a breaking change, though the command help reads
Hash a message according to EIP-191
Usage: cast hash-message [MESSAGE]

Arguments:
  [MESSAGE]  The message to hash

so it's confusing as one would expect to just hash without decoding. We could add a cast hash-decode-message for current behavior in case there are people using it as designed now, wdyt?


VT-x detection may show wrong results on MacOS

Last try on Mac OS confirms that this is working now.


how to call includes method before expose

I use grape entity in below way. And got a n+1 warning in expose :roles methods. I need know how can I call includes(:roles) methods before exposed. Thks！
Model
class User
  has_many :roles
end

class Role
   belongs_to :user
end

Entity
class User < Grape::Entity
  expose :roles do |obj, _opts|
    obj.roles.map(&:name)
  end
end

@lazybios did you find how to avoid n+1 queries ?

I think you need to call includes in the controller, when fetching the objects from db.
something like
respond_with User.where(your_params).includes(:roles), with: UserEntity

not sure this can be done at the entity layer.

call includes in the model will solve the N+1 warning
`
class User
has_many :roles
def roles
self.where(your_params).includes(:roles)
end
end
`


How to integerate the tree with animation libraries

Hi.
How can we integrate the tree with animation libraries like react-spring or react-motion. for purpose of adding transition for toggling a node?

I haven't tried to do this yet, but I would love to add it at some point. What would those libraries need from react-arborist to do their thing?

i guess the material tree view component is a good example for integerating external animation library.
https://mui.com/material-ui/react-tree-view/

You can use https://headlessui.com/react/transition


Improve XACML cache clearing logic when User attributes are updated.

According to the current implementation, when a user updates his attributes a listener is fired invalidation the attribute, decision and policy caches.
We need to improve the cache clearing logic to invalidate/remove only affected entries in caches like the policy cache.
OT JIRA: https://wso2.org/jira/browse/IDENTITY-4890

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Propose to add two features

closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/9193
@dstrome Let me know what you think please, thanks
Source:

Docs Build status updates of commit 2efbbcb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/rooms/teams-devices-feature-comparison.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@dstrome PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Docs Build status updates of commit 65fe968:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Teams/rooms/teams-devices-feature-comparison.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

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

File
Status
Preview URL
Details

Teams/rooms/teams-devices-feature-comparison.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Removed managed service entry. MTR Premium features are now part of the Teams Rooms Pro license.


What about the performance on regular classification problems?

Can Augmix boost training on regular classification problems as Mixup or Cutmix do?

In the paper the IID accuracy can increase for a ResNet-50. The more aggressive the augmentation, the higher capacity the model must be. A paper showing that AugMix+other augmentations can help high-capacity ResNeXts is this: https://arxiv.org/pdf/2006.16241.pdf#page=7&zoom=100,424,618

Does the model's uncertainty estimate correlate to the model's OOD detection ability? Are they the same thing?

No, calibration on in-class data is often decoupled from good uncertainty
estimation on out-of-class data.
On Tue, Dec 22, 2020 at 12:01 AM yechenzhi<EMAIL_ADDRESS>wrote:

Does the model's uncertainty estimate correlate to the model's OOD
detection ability? Are they the same thing?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google-research/augmix/issues/16#issuecomment-749404536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACZBITRJHC4567QMKQEX573SWBG6TANCNFSM4TQPFYDA
.

@hendrycks Just curious, is there any evidence for your last comment about calibration on in- and out-class data? I haven't really seen people talking about these two together.

Consider the rightmost two columns of this paper: https://arxiv.org/pdf/2003.03879.pdf#page=3&zoom=100,0,0
I don't see a clear association between Expected Calibration Error and OOD Detection Error.


Example request: Using with-apollo with apollo-cache-persist

We are working on a new NextJS based app based on the with-apollo example.
We'd like to persist client state, and according to this article this should be a simple thing to do. Unfortunately this seems to be more complicated in NextJS, as the intial state is set on the server.
See this issue for the problems I've been facing when trying to do this.
I would love an example on how this could be achieved in NextJS.

@ads1018

does the code work now? the commit is not clear about it


1609 show diff before pushing and pulling changes

Nice! 1 minor thing:

the horizontal scroll is still there in the JSON tab when there's a really long text string in the JSON

On my side, it was fixed already. Could you try again on your side? @six7


Problem with subfolder installs and the MediaManager generating incorrect URLs in places

Reported via email, reproduced here for visibility.

I have installed the WinterCMS 1.2.3 version on one of my testing servers with PHP 8.0 and Mysql 5.7. The installation went smoothly and works very well except for the following thing.
When I upload any image in the media section, The URL omits the subdirectory path and redirects to the root path which results in a 404 error for the image. (please refer the video attached)

https://github.com/wintercms/winter/assets/7253840/c89590c0-a57d-456d-93a8-3787e851e2ee
Have you tried this test on previous versions? The official recommendation when installing Winter in subfolders is to set config/cms.php -> linkPolicy to force. Is this issue new to v1.2.3?

This issue has been there in previous version (1.2.2) as well.
Also setting linkPolicy to force in config/cms.php generates the preview of the image However if we click on the link of the image it still throws "404 Not Found" error. Please refer to the image attached. You can check the image url on the bottom left side where it skips the subdirectory path.

I've attempted to replicate this, and found that while the scenario as described does happen, it might be the result of the config being cached that it initially shows the wrong public URL for the "Click here" link.

I set up Winter in a subdirectory. With cms.linkPolicy set to the default (detect), the media item preview did not show up and the "Click here" link was missing the subdirectory, as described.
By setting app.url to the correct URL and setting cms.linkPolicy to force, the media item preview started displaying correctly, but the "Click here" link was still missing the subdirectory, as described.
Once clearing the cache with php artisan cache:clear (or giving it about 5-10 minutes for the config cache to invalidate), the "Click here" link started showing the correct URL with the subdirectory being included in the URL.

@bennothommo is this something that the Web Installer could detect and configure for the user (or at least provide an option)?

I can't recall if I've asked this prior, but is there any reason we don't set cms.linkPolicy to force by default? I would argue that if we're providing people the ability to define an app URL, the expectation would be that all links should follow the app URL (like defining WP_HOME and WP_SITEURL for WordPress). The only case it should be detected is if the developer knows that more than one URL to the app might be used (eg. multi-sites).

@bennothommo I'm not sure, I wouldn't want to change the default in the code, but we could potentially change the default in the config.
It was first introduced in https://github.com/wintercms/winter/commit/9e500985ff776543e4357e044da1cf0c4c5ce859 and switched to detect in https://github.com/wintercms/winter/commit/47eff923e5b8a7a90bb931fac40628938ff16678. Not really any useful information in either of those commits.
I feel like the main reason against changing it at this point is to avoid having to deal with the inevitable support issues from the people not correctly configuring their app.url values.


[TASK] Context Derives - Defaults and additional options

Overview
Introduce Derive macros for GadgetConfiguration (Context) to simplify the implementation of common traits and functionalities. These derives will automatically handle boilerplate code, such as the implementation of the keystore, Tangle client, and standard defaults, allowing developers to focus on core logic without manual implementation of repetitive tasks.
Checklist

[x] KeystoreContext: Automatically implement the keystore trait for any struct that derives WithKeystore. This will provide out-of-the-box support for secure key management.
[x] SubxtClientContext: Automatically implement the client function that returns a new subxt::OnlineClient instance for interacting with the Tangle. This will streamline integration with the Tangle network.
[ ] StandardDefaults: (Name will change) Automatically derive a set of common derives (e.g., Default, Clone, Debug, PartialEq) for GadgetConfiguration, reducing the need for manual derivations and ensuring consistency.
[x] Specific keystore derives for functions specific to each key type e.g., first_ecdsa_signer
[x] ServiceContext: Automatically gives you access to the current service instance, current operators set, ..etc
[x] Tests for each Derive
[ ] #320
[x] Document the derives - #271

Pseudocode examples would be helpful, but generally I think my first feeling about naming is that I don't like these names or deriving WithKeystore feels off.

This checklist is almost done, the remaining task is caching the clients.

This checklist is now done!