#749 Bad error handling for IOException while reading incoming request body

Fixes #749

#749
Fixing issue for status code 404, when content length is large than body size. will be displaying 413 status code ("Entity Payload too large").
Considered scenario when application is hosted on Kestrel , IIS or Http.Sys and have different options configured as per host server.

For example:
builder.WebHost.UseHttpSys(options =>
{
    options.MaxRequestBodySize = 60000000;
});

Above code have MaxRequestBodySize  configured manually (default is ~28.6MB), so changes will handle this case too.
Proposed Changes

I have done manual testing, writing test case to fake server host was tough, as per my research on ASP.NET Core there is no exposed way to directly write this fake server test.
I also faced issue with access level of ToByteArray of RequestMapper, so I could have marked it null and ByteArrayContent constructor have given me error and hence I could have tested it.
Open for discussion how we can test this functionality :D

Tanmay,
The issue #729 is about sticky sessions! Are you sure your PR is related to #729 ?
It seems the number is wrong...

I found it! The issue is #749 😺

#749

I found it! The issue is #749 😺

Bad error handling for IOException while reading incoming request body #749

Sorry! correct its 749. Thank you

@ks1990cn @raman-m  Hello, I took now a bit of time to create an acceptance test, and I realized that the implementation is WAAAAAAAAAAAAAY too complicated (and it's impossible to get the KestrelServer from IServer, because you will get KestrelServerImpl first). Kestrel and other servers will return you a BadHttpRequestException with status code 413. So the issue here, is that the exception is wrapped and hidden in the UnmappableRequestError.  A rethinking of the exception handling is needed...
The issue here can be easily be fixed with:
catch (BadHttpRequestException badHttpRequest)
            {
                return badHttpRequest.StatusCode == 413 ? new ErrorResponse<HttpRequestMessage>(new PayloadTooLargeError(badHttpRequest)) : new ErrorResponse<HttpRequestMessage>(new UnmappableRequestError(badHttpRequest));
            }
            catch (Exception ex)
            {
                return new ErrorResponse<HttpRequestMessage>(new UnmappableRequestError(ex));
            }

Again, it's only a patch, imho, the exception should bubble up to a central exception handler.
The acceptance test:
[Fact]
        public void should_return_status_413_when_exceeding_body_max_size()
        {
            var port = RandomPortFinder.GetRandomPort();

            var configuration = new FileConfiguration
            {
                Routes = new List<FileRoute>
                {
                    new()
                    {
                        DownstreamPathTemplate = "/",
                        UpstreamPathTemplate = "/",
                        UpstreamHttpMethod = new List<string> { "Post" },
                        DownstreamHostAndPorts = new List<FileHostAndPort>
                        {
                            new()
                            {
                                Host = "localhost",
                                Port = port,
                            },
                        },
                        DownstreamScheme = "http",
                    },
                },
            };

            this.Given(x => x.GivenThereIsAServiceRunningOnNoExceptions($"http://localhost:{port}"))
                .And(x => _steps.GivenThereIsAConfiguration(configuration))
                .And(x => _steps.GivenOcelotIsRunningOnKestrelWithCustomBodyMaxSize(1024))
                .When(x => _steps.WhenIPostUrlOnTheApiGateway("/", new ByteArrayContent(File.ReadAllBytes(@"C:\Users\gnaegu\Desktop\test45.txt"))))
                .Then(x => _steps.ThenTheStatusCodeShouldBe(413))
                .BDDfy();
        }

Kestrel Server Implementation
public void GivenOcelotIsRunningOnKestrelWithCustomBodyMaxSize(long customBodyMaxSize)
        {
             var kestrelServer = Host.CreateDefaultBuilder()
                .ConfigureWebHostDefaults(webBuilder =>
                {
                    webBuilder.UseKestrel().ConfigureKestrel((_, options) =>
                        {
                            options.Limits.MaxRequestBodySize = customBodyMaxSize;
                        })
                        .ConfigureAppConfiguration((hostingContext, config) =>
                        {
                            config.SetBasePath(hostingContext.HostingEnvironment.ContentRootPath);
                            var env = hostingContext.HostingEnvironment;
                            config.AddJsonFile("appsettings.json", optional: true, reloadOnChange: false)
                                .AddJsonFile($"appsettings.{env.EnvironmentName}.json", optional: true, reloadOnChange: false);
                            config.AddJsonFile(_ocelotConfigFileName, optional: true, reloadOnChange: false);
                            config.AddEnvironmentVariables();
                        })
                        .ConfigureServices(s =>
                        {
                            s.AddOcelot();
                        })
                        .Configure(app =>
                        {
                            app.Use(async (_, next) =>
                            {
                                await next.Invoke();
                            });
                            app.UseOcelot().Wait();
                        })
                        .UseUrls("http://localhost:5001");
                }).Build();
             kestrelServer.Start();

             _ocelotClient = new HttpClient
             {
                 BaseAddress = new Uri("http://localhost:5001"),
             };
        }

Guys, who is our hero?
I'll get back to this funny thread after current release...
Keep conversation ongoing! ✌😁

Guys, who is our hero?
I'll get back to this funny thread after current release...
Keep conversation ongoing! ✌😁

Well, not really. I'm just tired of this $%^%&*% error handling.

Ah! didn't noticed that it already gives 413 in exception, I remember it was giving me 404 only (Maybe I didn't noticed right). Correct it can be done directly.
 Converting it to draft. Thanks for taking look @ggnaegi .

Open.. close... open... close...
What is the result?


[CELEBORN-707][MASTER] Remove env CELEBORN_MASTER_HOST and CELEBORN_MASTER_PORT

What changes were proposed in this pull request?
Remove environment variables CELEBORN_MASTER_HOST and CELEBORN_MASTER_PORT, and makes CELEBORN_LOCAL_HOSTNAME takes effect on both master and worker.
Why are the changes needed?
There are many different ways to configure the master/worker host and port, which makes the thing complex and inconsistent.
After this change,
master

cli args --host --port takes the highest priority
then lookup env CELEBORN_LOCAL_HOSTNAME
things are different when HA enabled and disabled
3.1. when HA is disabled, lookup configurations celeborn.master.host and celeborn.master.port
3.2. when HA is enabled, each node needs to know the whole cluster info,celeborn.master.ha.node.1.host clb-1
celeborn.master.ha.node.1.port 9097
celeborn.master.ha.node.2.host clb-2
celeborn.master.ha.node.2.port 9097
celeborn.master.ha.node.3.host clb-3
celeborn.master.ha.node.3.port 9097

in addition, celeborn.master.ha.node.id=1 can be used to indicate the node id, otherwise, the master will try to bind each host to match the node id.

worker

cli args --host --port takes the highest priority
then lookup env CELEBORN_LOCAL_HOSTNAME

things are simple than the master case because each worker is not required to know others.
Does this PR introduce any user-facing change?
Yes.
How was this patch tested?
UT.

The further work, I propose adding a new configuration celeborn.bind.preferIpAddress(any naming suggestion is welcome) to control whether to use IP or hostname when CELEBORN_LOCAL_HOSTNAME is absent. This is useful in K8s scenario, given currently Celeborn always uses IP for registering, as Celeborn is deployed as STS, the Pod name will be preserved but Pod IP changed on restarting, it's painful during the rolling upgrade, any restarted worker Pod will be treated as a brand new worker and lost all shuffle data, it's really painful.


Declared but not defined methods

Hi,
I was looking into the SQLPP11 code today and found several methods which are declared but not implemented
Each of the three connectors' connection class (sqlpp::xxxxx::connection) has a method declaration
template <typename Check, typename T>
auto _run(const T& t, Check) -> Check;

These three methods have been declarated but I don't see them being implemented anywhere. For example for the MySQL connector has this method in file
include/sqlpp11/mysql/connection.h on line 427 and it was added with commit 5e16f32e about 2 years ago.
The situation is the same with the other 2 connectors - the same method was declared in them but not implemented about 2 years ago.
What is the reason for the addition of this method? Is it just a copy&paste error or are there any other reasons, e.g. improved error reporting for errors detected at compile time, or maybe something else?
Additionally I was looking into the definition of SQLPP_PORTABLE_STATIC_ASSERT and I found its definition in include/sqlpp11/portable_static_assert.h to be
#define SQLPP_PORTABLE_STATIC_ASSERT(name, message) \
  struct name : std::false_type                     \
  {                                                 \
    template <typename... T>                        \
    constexpr explicit name(T&&...)                 \
    {                                               \
      static_assert(wrong_t<T...>::value, message); \
    }                                               \
    template <typename... T>                        \
    static constexpr void verify(T&&...)                 \
    {                                               \
      static_assert(wrong_t<T...>::value, message); \
    }                                               \
    auto begin() const -> void;                     \
    auto end() const -> void;                       \
  }

Again there are two methods, begin() and end() which are declared but don't seem to be implemented anywhere. They look similar to the methods used to make an object behave like a sequential iterator, e.g. in range-loops, but obviously they are not because they return void. I thought that maybe they could make the static assert definitions display nicer in some IDEs but couldn't find anything about that.  What is the rationale behind these method declarations?
@rbock can you please shed some light on these declarations in the connectors and in SQLPP_PORTABLE_STATIC_ASSERT ?

being and end end:
Having these methods helps suppressing error messages in a number of cases, because compilers often carry on even after hitting static_assert. Try this:
  for (const auto& row : db(select(all_of(t)).from(t))) // missing where condition
  {
  }

And then remove the begin/end from the static assert. On my machine, with clang, the difference is 3.5kB, with gcc it is 9.5kB.
As you noticed, the returning something better than void could help even more (but note that normally stuff will happen in the body of the for-loop and diminish the effect).
template <typename Check, typename T>
auto _run(const T& t, Check) -> Check;

This is similar. It can suppress error messages in case the "correct" _run method is not a match. Interestingly, clang actually has a shorter error message without this construct, while gcc has a longer error message.

@rbock
Thank you for the clarification. Now that you explained it, it makes perfect sense. Also adding a couple of short comments explaining the purpose of those methods will probably help those reading the code.
I am currently getting more familiar with the details of the core code so that is how I came across those methods. Once I am more confident in my knowledge I will continue providing PRs with improvements.

Thanks! I really appreciate your thoroughness, questions and contributions!


review subclasses of disease by subcellular system affected

look for terms in branch 'disease by subcellular system affected' and flag any terms where this is the only parent
this was discussed at the 05/22 Mondo mtg.

Other than the terms whose label starts with "disease by", here's a start:

[ ] MONDO:0021017 synaptopathy -> has only 1 child, that has another parent too;
...
TBC

@paolaroncaglia I don't remember what the specific action item here was, what are we meant to do with these terms? Do we want to reclassify them? Or consider obsoleting them?
Thanks!

Hi @nicolevasilevsky ,

@paolaroncaglia I don't remember what the specific action item here was, what are we meant to do with these terms? Do we want to reclassify them? Or consider obsoleting them?

Some of the terms in that branch may be useful but it's up to Mondo if you'd like to keep the broader grouping classes or not... I tried to help with this, but I progress slowly because I look at terms in OLS (old laptop may not agree with memory settings necessary to display Mondo in Protege). This work is less priority for EFO than the Orphanet-related issue, so I'm ok with leaving it aside, if keeping the ticket open is ok with you please. Thanks!

disease by subcellular system affected has been obsoleted, so I think this is no longer needed. Closing.


Build cache tutorial has outdated content

On https://rushjs.io/pages/maintainer/build_cache/ under "Testing the Build Cache", the docs suggest that "rush rebuild" will pull from the build cache, but that is no longer correct.
(Issue raised by @thekorn in zulipchat.)

This is now fixed.


Question about C++ overloaded functions of segment load

Here are 2 functions in the C++ overloaded functions:
1.void vlseg2e8_v_i8mf8 (vint8mf8_t *v0, vint8mf8_t *v1, const int8_t *base, size_t vl);
2.void vlseg2e8_tu (vint8mf8_t *v0, vint8mf8_t *v1, const int8_t *base, size_t vl);
These 2 functions have the same arguments. Why does one can be overloaded but the other can not be?
I think both of them can not be overloaded, because vint8mf8_t * is a pointer type.

In fact, _tu need to have merge operand just like masked segment operation have maskedoff to align current design.
I will update #137 later to fix all issues in one time.
current known issues in #137:

TA and TAMA vslidedown/vslideup need the dest operand for correctness. https://github.com/riscv-non-isa/rvv-intrinsic-doc/issues/32
segment load and vfclass TU policy need to have merge operand.
vmv.s.x and vfmv.s.f ta policy have no overloaded api.
reduction TAM have no merge operand.

Let me know if anything is missing. I'm really appreciate your time to help and review.

You have remove vqmacc in C intrinsic document but overloaded functions document still have vqmacc. I think you should remove them.
By the way, does the RISC-V community agrees https://github.com/riscv-non-isa/rvv-intrinsic-doc/pull/137 this proposal and merge them into master in the future? I understand the explosion of RVV intrinsics will make LLVM compilation time bad due to
the big <riscv_vector.h>. But this is not a problem in GCC. I hope this proposal can be merged into master branch finally.

@zhongjuzhe https://reviews.llvm.org/D111617 I am working on improve compilation speed for clang/LLVM, I believe that's could resolve that issue.

I have 2 more questions: 1.You have remove vqmacc in C intrinsic document but overloaded functions document still have vqmacc. I think you should also remove them in overloaded functions. 2.tamu/tumu/tuma policy in segment load, you don't have merge or maskoff operand, do you need to add them too?

Thanks, I will remove vqmacc in document later.
for 2, I saw that have merge operand for segment load, maybe you checkout the wrong branch.
https://raw.githubusercontent.com/riscv-non-isa/rvv-intrinsic-doc/new-policy-proposal/policy_funcs/intrinsic_funcs/03_vector_load_store_segment_instructions_zvlsseg.md

By the way, does the RISC-V community agrees #137 this proposal and merge them into master in the future? I understand the explosion of RVV intrinsics will make LLVM compilation time bad due to the big <riscv_vector.h>. But this is not a problem in GCC. I hope this proposal can be merged into master branch finally.


Memory "leaks" in Loop

Hi, I'm using this library to monitor by clipboard and remove unwanted '\r' in silence. But I notice that the CFString are not released in each loop, how can I solve it?
Here is my code:
#include <string>
#include <thread>
#include <chrono>
#include "clip.h"

using namespace std;

int main() {
    while (true) {
        string new_value;
        if (clip::has(clip::text_format())) {
            clip::get_text(new_value);
            if (new_value.find('\r') != string::npos) {
                replace(new_value.begin(), new_value.end(), '\r', ' ');
                clip::set_text(new_value);
            }
        }
        this_thread::sleep_for(chrono::milliseconds(1000));
    }
    return 0;
}

Hi @NekoAsakura thanks a lot for reporting this bug. I've just pushed a fix to main, you can give a try.

Thank you for letting me know! I appreciate your quick response in fixing the bug.


support k8s version and lastAppliedResource

it is require to add support for k8s version and lastAppliedResourceannotation check for outdated-api capabilities to be able to detect converted resources

Great @chen-keinan ! Can you please help me how I can run this analyzes in trivy-operator? I still don‘t get it … sorry!

@jkleinlercher Note: the fix has been merged to upstream but not yet released.
I'll cut an RC tomorrow.
The outdated-api finding will appear as ConfigAuditReport so once you run the operator with an outdated-api resource  you can find it by query
kubectl get ConfigAuditReport  <outdated-api resource report name> -n <outdated-api resource namespace>

example :
apiVersion: aquasecurity.github.io/v1alpha1
kind: ConfigAuditReport
metadata:
  annotations:
    trivy-operator.aquasecurity.github.io/report-ttl: 24h0m0s
  creationTimestamp: "2023-07-10T19:20:44Z"
  generation: 1
  labels:
    plugin-config-hash: 659b7b9c46
    resource-spec-hash: 55b58d8898
    trivy-operator.resource.kind: CronJob
    trivy-operator.resource.name: demo-deprecated-cron-job
    trivy-operator.resource.namespace: default
  name: cronjob-demo-deprecated-cron-job
  namespace: default
  ownerReferences:
  - apiVersion: batch/v1
    blockOwnerDeletion: false
    controller: true
    kind: CronJob
    name: demo-deprecated-cron-job
    uid: 27df88e2-48c2-421b-8428-67517df3251e
  resourceVersion: "1298"
  uid: 0c85d50b-e1ad-43f1-9a91-3708a5d3ad02
report:
  checks:
  - category: Kubernetes Security Check
    checkID: KSV030
    description: The RuntimeDefault/Localhost seccomp profile must be required, or
      allow specific additional profiles.
    messages:
    - Either Pod or Container should set 'securityContext.seccompProfile.type' to
      'RuntimeDefault'
    severity: LOW
    success: false
    title: Default Seccomp profile not set
  - category: Kubernetes Security Check
    checkID: KSV107
    description: 'apiVersion ''batch/v1beta1'' and kind ''CronJob'' has been deprecated
      on: ''v1.21'' and planned for removal on:''v1.25'''
    messages:
    - |-
      apiVersion 'batch/v1beta1' and kind ‘CronJob' should be replaced with the new API 'batch.v1.CronJob'
      See https://github.com/kubernetes/kubernetes/tree/master/staging/src/k8s.io/api/batch/v1beta1/zz_generated.prerelease-lifecycle.go
    severity: LOW
    success: false
    title: Evaluate k8s deprecated and removed APIs

@jkleinlercher trivy-operator v0.15.0-rc is available

I tried it today. CronJob is working but HorizontalPodAutoscaler is not working. I tested with https://github.com/suxess-it/deprecated-k8s-api-testcase/blob/main/hpa-deprecated.yaml .
trivy cli finds it with „trivy conf “ but trivy-operator doesn‘t

@jkleinlercher trivy-operator do not reconcile HorizontalPodAutoscaler can you open an issue for it and I'll add the ability to reconcile custom resources

@chen-keinan I created issue https://github.com/aquasecurity/trivy-operator/issues/1384 . thank you!


Support Bamboo 6.2.1

I cannot find this in the marketplace for Bamboo 6.2.1. Please support this version.

Compatibility updated

Thanks!


Unable to override local configurations using -Dspring.profiles.active=pre on SpringCloud version 2021.0.8.

spring-cloud-dependencies  version 2021.0.8.
Problem Description：
I specified the Spring configuration in bootstrap.yml as follows:
spring:
  profiles:
    active: dev

At the same time, I overrode it during startup using -Dspring.profiles.active=pre. However, when using the Spring Cloud Dependencies version 2021.0.8, the active profile is not overridden correctly. Instead, it outputs the following information: "The following 2 profiles are active: "pre", "dev".
springBoot version is 2.6.15

Can you try Spring Cloud 2022.0.4 and see if there is the same behavior?

I did not use the 2022.0.4 version, but trying with version 2021.0.7 worked. My solution was to change the configuration from:
spring:
  profiles:
    active: dev

to:
spring:
  profiles:
    default: dev

Hello, @Iamdouding, as you can see in the project maintenance site, the only version currently in OSS support is 4.1.x (2023.0.x release train).  Please let us know if the issue is  still present on 2023.0.0.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


Cross Compilation using CGO

I tried to cross compile a project with sqlite driver (github.com/mattn/go-sqlite3) with cgo under Linux for Windows, but failed.
I have mingw-w64 package installed in a debian system.
go version go1.5.2 linux/amd64
CGO_ENABLED=1 GOOS=windows GOARCH=386 CC="i686-w64-mingw32-gcc" gb build all

Failed details as follow:
/tmp/gb399620475/runtime/cgo/_cgo/gcc_libinit_windows.o: In function `cgo_wait_runtime_init_done':
/usr/local/go/src/runtime/cgo/gcc_libinit_windows.c:15: multiple definition of `_cgo_wait_runtime_init_done'
/tmp/gb399620475/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb399620475/runtime/cgo/_cgo/_cgo_main.c:3: first defined here
/tmp/gb399620475/runtime/cgo/_cgo/gcc_windows_386.o: In function `threadentry':
/usr/local/go/src/runtime/cgo/gcc_windows_386.c:60: undefined reference to `crosscall_386'
collect2: error: ld returned 1 exit status
# runtime/cgo

But if I simply use go build as follows, there is no such error.
CGO_ENABLED=1 GOOS=windows GOARCH=386 CC="i686-w64-mingw32-gcc" go build -v main

Is there anything I am missing?

Cross compilation with cgo is not supported by go yet. It's something I want to support but I don't have a timeline for when it will make it onto the road map.

On 7 Jan 2016, at 17:01, KM Tong<EMAIL_ADDRESS>wrote:
I tried to cross compile a project with sqlite driver (githubcom/mattn/go-sqlite3) with cgo under Linux for Windows, but failed
I have mingw-w64 package installed in a debian system
go version go152 linux/amd64
CGO_ENABLED=1 GOOS=windows GOARCH=386 CC="i686-w64-mingw32-gcc" gb build all
Failed details follows:
/tmp/gb399620475/runtime/cgo/_cgo/gcc_libinit_windowso: In function cgo_wait_runtime_init_done': /usr/local/go/src/runtime/cgo/gcc_libinit_windowsc:15: multiple definition of _cgo_wait_runtime_init_done'
/tmp/gb399620475/runtime/cgo/_cgo/_cgo_maino:/tmp/gb399620475/runtime/cgo/_cgo/_cgo_mainc:3: first defined here
/tmp/gb399620475/runtime/cgo/_cgo/gcc_windows_386o: In function threadentry': /usr/local/go/src/runtime/cgo/gcc_windows_386c:60: undefined reference to crosscall_386'
collect2: error: ld returned 1 exit status
runtime/cgo
But if I simply use go build as follows, there is no such error
CGO_ENABLED=1 GOOS=windows GOARCH=386 CC="i686-w64-mingw32-gcc" go build -v main
Is there anything I am missing?
—
Reply to this email directly or view it on GitHub.

I may have a solution to this. At work, thanks to Dave, I've got a program that is cross-compiled for the ARM working. I'm using go 1.6 and there are new limits on using go pointers in C code, but they can be easily overcome. I'll post my solution here tomorrow.
Also, further kudos for Dave: gb supports linking C libraries perfectly. My only hitch was needing to state the library path in the gb build command:
 CGO_LDFLAGS="-L<EMAIL_ADDRESS>-lmylib"  gb build
cgoExample
gb rocks!

I'm running into a similar issue.
Using the latest gb and go1.7 on linux/amd64 trying to cross-compile to linux/arm.
gb build fails while go build works.
The portion with the error from gb build:
/tmp/gb852504266/runtime/cgo/_cgo/gcc_context.o: In function `_cgo_release_context':
/home/mjohn/software/go/src/runtime/cgo/gcc_context.c:11: multiple definition of `_cgo_release_context'
/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.c:4: first defined here
/tmp/gb852504266/runtime/cgo/_cgo/gcc_libinit.o: In function `_cgo_wait_runtime_init_done':
/home/mjohn/software/go/src/runtime/cgo/gcc_libinit.c:32: multiple definition of `_cgo_wait_runtime_init_done'
/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.c:3: first defined here
/tmp/gb852504266/runtime/cgo/_cgo/gcc_linux_arm.o: In function `threadentry':
/home/mjohn/software/go/src/runtime/cgo/gcc_linux_arm.c:54: undefined reference to `crosscall_arm1'
collect2: error: ld returned 1 exit status
# runtime/cgo

Sorry. Gb makes no attempt to support Cross compilation with cgo enabled.
This is unlikely to change because I believe that it is fundamentally a bad
idea.
On Tue, 13 Sep 2016, 00:46 Michael John<EMAIL_ADDRESS>wrote:

I'm running into a similar issue.
Using the latest gb and go1.7 on linux/amd64 trying to cross-compile to
linux/arm.
gb build fails while go build works.
The portion with the error from gb build:
/tmp/gb852504266/runtime/cgo/_cgo/gcc_context.o: In function _cgo_release_context': /home/mjohn/software/go/src/runtime/cgo/gcc_context.c:11: multiple definition of _cgo_release_context'
/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.c:4: first defined here
/tmp/gb852504266/runtime/cgo/_cgo/gcc_libinit.o: In function _cgo_wait_runtime_init_done': /home/mjohn/software/go/src/runtime/cgo/gcc_libinit.c:32: multiple definition of _cgo_wait_runtime_init_done'
/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.c:3: first defined here
/tmp/gb852504266/runtime/cgo/_cgo/gcc_linux_arm.o: In function threadentry': /home/mjohn/software/go/src/runtime/cgo/gcc_linux_arm.c:54: undefined reference to crosscall_arm1'
collect2: error: ld returned 1 exit status
runtime/cgo
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/constabulary/gb/issues/532#issuecomment-246371272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAcA3WNLnBpKuNzvdRdlNjKvnvuUCumks5qpWXCgaJpZM4HAKAF
.

And when I say bad, I understand the desire and the requirement, but
believe Cross compilation with cgo combines all the easy of use of cross
compiling C code with the debugability of a program written in multiple
languages.
On Tue, 13 Sep 2016, 08:45 Dave Cheney<EMAIL_ADDRESS>wrote:

Sorry. Gb makes no attempt to support Cross compilation with cgo enabled.
This is unlikely to change because I believe that it is fundamentally a bad
idea.
On Tue, 13 Sep 2016, 00:46 Michael John<EMAIL_ADDRESS>wrote:

I'm running into a similar issue.
Using the latest gb and go1.7 on linux/amd64 trying to cross-compile to
linux/arm.
gb build fails while go build works.
The portion with the error from gb build:
/tmp/gb852504266/runtime/cgo/_cgo/gcc_context.o: In function _cgo_release_context': /home/mjohn/software/go/src/runtime/cgo/gcc_context.c:11: multiple definition of _cgo_release_context'
/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.c:4: first defined here
/tmp/gb852504266/runtime/cgo/_cgo/gcc_libinit.o: In function _cgo_wait_runtime_init_done': /home/mjohn/software/go/src/runtime/cgo/gcc_libinit.c:32: multiple definition of _cgo_wait_runtime_init_done'
/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.c:3: first defined here
/tmp/gb852504266/runtime/cgo/_cgo/gcc_linux_arm.o: In function threadentry': /home/mjohn/software/go/src/runtime/cgo/gcc_linux_arm.c:54: undefined reference to crosscall_arm1'
collect2: error: ld returned 1 exit status
runtime/cgo
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/constabulary/gb/issues/532#issuecomment-246371272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAcA3WNLnBpKuNzvdRdlNjKvnvuUCumks5qpWXCgaJpZM4HAKAF
.

Dave,
I agree with you. However, I have got a way to do it. Do you want to connect me with the person asking the question?
Regards,
Dave

From: Dave Cheney<EMAIL_ADDRESS>Sent: Monday, September 12, 2016 18:47
To: constabulary/gb
Cc<EMAIL_ADDRESS>Comment
Subject: Re: [constabulary/gb] Cross Compilation using CGO (#532)
And when I say bad, I understand the desire and the requirement, but
believe Cross compilation with cgo combines all the easy of use of cross
compiling C code with the debugability of a program written in multiple
languages.
On Tue, 13 Sep 2016, 08:45 Dave Cheney<EMAIL_ADDRESS>wrote:

Sorry. Gb makes no attempt to support Cross compilation with cgo enabled.
This is unlikely to change because I believe that it is fundamentally a bad
idea.
On Tue, 13 Sep 2016, 00:46 Michael John<EMAIL_ADDRESS>wrote:

I'm running into a similar issue.
Using the latest gb and go1.7 on linux/amd64 trying to cross-compile to
linux/arm.
gb build fails while go build works.
The portion with the error from gb build:
/tmp/gb852504266/runtime/cgo/_cgo/gcc_context.o: In function _cgo_release_context': /home/mjohn/software/go/src/runtime/cgo/gcc_context.c:11: multiple definition of _cgo_release_context'
/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.c:4: first defined here
/tmp/gb852504266/runtime/cgo/_cgo/gcc_libinit.o: In function _cgo_wait_runtime_init_done': /home/mjohn/software/go/src/runtime/cgo/gcc_libinit.c:32: multiple definition of _cgo_wait_runtime_init_done'
/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.o:/tmp/gb852504266/runtime/cgo/_cgo/_cgo_main.c:3: first defined here
/tmp/gb852504266/runtime/cgo/_cgo/gcc_linux_arm.o: In function threadentry': /home/mjohn/software/go/src/runtime/cgo/gcc_linux_arm.c:54: undefined reference to crosscall_arm1'
collect2: error: ld returned 1 exit status
runtime/cgo
�
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/constabulary/gb/issues/532#issuecomment-246371272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAcA3WNLnBpKuNzvdRdlNjKvnvuUCumks5qpWXCgaJpZM4HAKAF
.

�
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/constabulary/gb/issues/532#issuecomment-246519855, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAgSBokFf-pBJ87B_QmCkrDm3GiZcLTkks5qpdZzgaJpZM4HAKAF.


Added My Name in repository

Added My Name in the List in repo

Yeah I Know

Okay

closing due to inactivity feel free to reopen at any time


Model Checkpoints

Question
Is it possible to create checkpoints during training? And/or resume training from a previously saved model?

Hello @Andrea94c
Thank you for the question.
We haven't added a model saving support mainly because the library is still in development, and the internals can frequently change, invalidating the previous caches.
However, we will investigate how to safely cache checkpoints. For which models would you want this kind of support?
Thank you!

Hey @bcebere
I've been doing this manually, training for 50 epochs or so, saving the model to disk, loading it back from disk and continue training.
But I'd love to pass in an argument like n_iter_save and have checkpoints saved every few iterations. Or maybe have it linked to n_iter_print, that's how I assumed it worked initially. I've been trying out GANs, so having that option for it would be nice.
This would presumably go in the /synthcity/plugins/core/models/gan.py file ? What do you think about me submitting a PR for this ?

Thanks for the answers, I'm working with GANs as well.

@pravsels Thank you for your feedback.
Any contribution is more than welcome.
However, caching the models can be tricky. More exactly, any change in the models will break the existing caches.
So, the mechanism should consider the library's version, architecture/OS, and python version.
If you want to tackle this, I can provide feedback on the PR. Otherwise, I will have a look in the following weeks.
Many thanks!
cc @robsdavis

@pravsels Thank you for your feedback.
Any contribution is more than welcome.
However, caching the models can be tricky. More exactly, any change in the models will break the existing caches. So, the mechanism should consider the library's version, architecture/OS, and python version.
If you want to tackle this, I can provide feedback on the PR. Otherwise, I will have a look in the following weeks.
Many thanks!
cc @robsdavis

Yes, I would like to take a crack at it.
Would appreciate any feedback you can provide to help me in the process, maybe even a link to another PR that might've tackled something similar.

@pravsels Thank you very much for working on this.
However, I would suggest to wait a few days before starting, we have a big merge with the image generation support, and most likely will create a ton of merge conflicts in your branch.
After that, there are a few things to keep in mind:

All the plugins have a workspace parameter that needs now to be propagated -> TabularGAN -> GAN, so you save in the same folder.
When there is a patience_metric in the core models, there is a logic for saving the best model version here.
You would need to see if you want to save best_state_dict or just the latest, if you have or not the patience_metric.
When you save best_state_dict/latest_state_dict to a file, that file should be in the workspace path, and with a name that includes

the synthcity version`. If there is a checkpoint or backup from a different synthcity version, it is best to just ignore that, to prevent errors and crashes.
the python version, in case you have multiple conda envs.
a hash of the training set.

Hopefully, these 3 identifiers(synthcity version, python version, dataset hash), are enough to avoid errors in the cached models.
Thank you again for your contribution! Looking forward to the PR. :)


[Distributed] Destroy decoded remote call arguments in reverse order

Follow up to https://github.com/apple/swift/pull/65874
Thanks for review @DougGregor

@swift-ci please smoke test

Why?

@DougGregor asked for it, I guess so it's the same as other things in this func, also in reverse? "Shouldn't matter but" I guess

The other thing is stack allocations which should be in reverse order, this one is just calls to destroy function that’s why I am asking the question here :)

@swift-ci please smoke test

Yeah it really shouldn't matter for this one... I'll ask @DougGregor if he thinks we'd better do it like this anyway or nah.

note to self; add some more tests to check various combinations of types

I was adding more tests and actually see that deinits DON'T run and we have an unbalanced retain here still -- even if the memory is properly destroyed so we don't leak the object itself. We're in the Direct_Guaranteed case and take the param with loadAsTake so that's right... I don't know where else the bad retain could be from 🤔

@swift-ci please smoke test

@swift-ci please smoke test

Fixed formatting

@swift-ci please smoke test and merge


HTTP Logging: how to display [Redacted] request & response info

Here in Microsoft Docs I read "Request headers are logged as soon as the middleware is invoked. Headers are redacted by default with the character '[Redacted]' unless specified in the HttpLoggingOptions.RequestHeaders."
But not how to specify it in HttpLoggingOptions.RequestHeaders. Maybe you could provide an example?
I've looked in the sample code also

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

ID: 7c772b2a-5d37-84af-624c-9022efe0758b
Version Independent ID: b39594cc-9d1c-611c-e3e7-8755d74253a6
Content: HTTP Logging in .NET Core and ASP.NET Core
Content Source: aspnetcore/fundamentals/http-logging/index.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @jkotalik
Microsoft Alias: jukotali

@JeepNL, thanks very much for feedback on this topic.  I see what you mean, it does bring up a scenario that it doesn't really follow through on, and an example would be helpful.

See https://github.com/dotnet/aspnetcore/issues/40981


Support class name completion

When user import Java document, ATM they have to write down FQCN which is inconvenient. We have to provide typeahead search by scanning user uploaded jars.

we did something similar in Groovy LSP, in case you want to share ideas.
To scan a classpath we have used this lib:
https://www.google.com/search?client=firefox-b-d&q=org+reflections
https://github.com/prominic/groovy-language-server/pull/14/files#diff-b1f0a074d4b3cea4a94efc94bdd1e1deR132-R136

Need to add tests - check PaulsFieldAction (may not be related to this push).

@igarashitm - Hey Tomo - In the DM if I import a custom field action jar, the custom action defined by the custom class (io.atlasmap.service.my.MyFieldActionsModel) is automatically loaded into the field actions without having to 'enable' the class.
Is this an error or is the user supposed to enable MyFieldActions?
ref:

Custom transformations doesn't have any enable/disable option, it appears where it's eligible once it's loaded. User just decides where to apply.

Then do we need the Enable/Disable circleplus icon?

I don't personally think so since it's redundant. ImportJar=>applyCustomTransformation and ImportJar=>addJavaDocument looks symmetric and intuitive.

Ok - so I'll bail on adding support for returning a list of loadable FQCNs and turn this into 'Remove Enable/Disable Class' icon from the Source/Target panels.

Importing a jar adds it into the classpath.  The enable class icon is needed to use the class as a document (the jar could contain any number of classes, the user must choose one from them as a document).  The enable class has nothing to do with custom field actions.

In the above case the user would import Bicycle.jar and the enable io.paul.Bicycle.


Format Selection formats everything

VSCode Version: 1.14.1
ProductName: Mac OS X
ProductVersion: 10.12.5
BuildVersion: 16F73

Format Selection formats everything, but not the selected part.

@bsvipas what's the language of that file and what extensions you installed? Formatting selections is related to these two factors.

@rebornix Hi, it is JavaScript (.js) file.

@bsvipas I didn't reproduce this issue. Is it possible that you installed some extensions which might change this behavior? You can try by running code --disable-extensions

@rebornix Yes, you're right this is Prettier problem.


Why should one add a license on a branch??

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
adding-a-license-to-a-repository
What part(s) of the article would you like to see updated?
In adding-a-license-to-a-repository, point 8 states:
Below the commit message fields, decide whether to add your commit
to the current branch or to a new branch.
If your current branch is the default branch, you should choose to create
a new branch for your commit and then create a pull request.
For more information, see "[Creating a new pull request](https://docs.github.com/en/articles/creating-a-pull-request)."

I'm questioning the use of "should" here. Unless the license text needs to be reviewed by several people, it make little-to-no sense to create a branch just for adding the license. Adding a license is a major step forward, so it ought to happen on the default branch.
As it stands, this doc file suggests an (AFAICT) unnecessarily convoluted way of performing a step, and provides no explanation of why it would be better to do it that way.
Additional information
No response

@lars-hellstrom Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

Hi @lars-hellstrom 👋 thank you for opening this issue. ✨ The team decided we're not going to making this change. Thanks so much for taking your time in contributing to the GitHub docs. 💖


Next version release notes don't show from update banner

VSCode Version: 1.9.1
OS Version: Windows 7

Steps to Reproduce:

Open 1.9.1 and wait for update banner to appear.
Click on "Release Notes".

Get error message:
"Unable to open 'Release Notes: 1.10.0': Cannot read property 'replace' of null
1.10 release notes display fine after installing 1.10 VS Code.

Nothing to add really, except I happened to snag a screenshot.

The "Release Notes" open without error if you go through the Help / Release Notes menu item, even without restarting VS Code first.

I saw still the 1.9.1 release note by doing that, instead of 1.10.0

Callstack:
messageService.ts:126 Unable to open 'Release Notes: 1.10.0': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..: Error: Unable to open 'Release Notes: 1.10.0': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..
at t.doHandleSetInputError (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:132:1119)
at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:132:725
at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:18548)
at t.doSetInput (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:132:480)
at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:131:30589
at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:19172)
at t.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:131:30561)
at t.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:131:29827)
at e.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:119:22073)
at e.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:119:21503)

I double-checked and the menu pulls up the 1.9.1 release notes for me too @unional.

Also reproes for me.

The fix for this was pushed in f9c78602c1fc4f55bf6be239fd74f0f4b28faa68, so it's only available in 1.10.

VSCode Version: Code 1.10.1 (653f8733dd5a5c43d66d7168b4701f94d72b62e5, 2017-03-02T00:33:15.706Z)
OS Version: Windows_NT ia32 10.0.15031
Extensions:

Extension
Author
Version

react-toolbox-snippets
alechp
0.12.0

vscode-icontheme-nomo-dark
be5invis
1.2.5

vscode-simpler-icons
davidbabel
1.3.0

vscode-eslint
dbaeumer
1.2.2

dark-plus-plus-theme
DryWolf
0.0.7

vscode-babel-coloring
dzannotti
0.0.4

vscode-great-icons
emmanuelbeziat
1.1.44

vsc-material-theme
Equinusocio
0.0.3

icons-fania
fania
0.4.0

file-icons
file-icons
1.0.0

vscode-flatland-icon-theme
gebilaoxiong
0.0.6

easy-icons
jamesmaj
0.3.1

vsclassic-icon-theme
jez9999
1.0.3

ionic2-vscode
jgw9617
1.0.0

Angular2
johnpapa
0.5.7

vscode-icon-theme
jtlowe
1.4.0

theme-dark-plus-contrast
k3a
0.1.101

vscode-simple-icons
LaurentTreguier
0.10.0

csharp
ms-vscode
1.6.2

debugger-for-chrome
msjsdiag
2.5.3

indent-rainbow
oderwat
0.4.1

material-icon-theme
PKief
1.3.2

seti-icons
qinjia
0.1.3

vscode-icons
robertohuertasm
7.2.0

indenticator
SirTori
0.2.1

react-beautify
taichi
0.3.0

ayu
teabyii
0.2.1

ionic2
Thavarajan
2.1.3

react-redux-es6-snippets
timothymclane
2.0.2

jsx
TwentyChung
0.0.1

ionic2-snippets
uqutub
0.0.4

bootstrap-3-snippets
wcwhitehead
0.0.9

JavaScriptSnippets
xabikos
1.2.0

ReactSnippets
xabikos
1.2.0

Material-theme
zhuangtongfa
1.0.3

Steps to Reproduce:

Open 1.10.1 and wait for update banner to appear.
Click on "Release Notes".

@joaomoreno My comment above also referred to 1.10.1.

@tomahawkzx @fabiorocha
Wait... but if you are on 1.10.1... how are you getting an update notification. There's no later version, after that one. Are you sure about being on 1.10.1?
Can you show me a screenshot of the update notification?
If you open the Dev Tools, and set a breakpoint on vs/workbench/parts/update/electron-browser/update.ts, line 51, before you get the update notification... it should get hit once that update notification comes in... what is the value of the version variable?

@joaomoreno that is where this became weird. Although I already have 1.10.1, the update bar still randomly pops through the day. (sorry to not mention that before, I was in a hurry)
When I select Download Now, nothing happens. Pressing "Release Notes" leads to the second message "Error: Unable to open 'Release Notes: {0}': Cannot read property 'replace' of null".

I don't know if this behavior is related to this issue (nor the actual label for this), but the button Help > "Check for updates" keeps disabled on "Checking for updates..." state.

While I'm figuring how to find update.ts in the Developer Tools, here follows the errors logged at the console:

Overwriting extension <<.jsx>> to now point to mime <<text/x-jsx>>
2/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:31 Overwriting extension <<.jsx>> to now point to mime <<text/javascript>>
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117 [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: ejst.$localShowMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117 [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: lattet.$localShowMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117 [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: tplt.$localShowMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117 [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: tst.$localShowMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117 [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: hamlt.$localShowMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117 [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: erbt.$localShowMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:117
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136 [Extension Host] typescript (vscode.typescript) uses PROPOSED API which is subject to change and removal without notice
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136 [Extension Host] [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: ejsd.logExtensionHostMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136 [Extension Host] [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: latted.logExtensionHostMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136 [Extension Host] [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: tpld.logExtensionHostMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136 [Extension Host] [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: tsd.logExtensionHostMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136 [Extension Host] [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: hamld.logExtensionHostMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136 [Extension Host] [C:/Users/leonardobag/.vscode/extensions/wcwhitehead.bootstrap-3-snippets-0.0.9]: Unknown language in `contributes.snippets.language`. Provided value: erbd.logExtensionHostMessage @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136 [Extension Host] vscode-icons is active!
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:103 Unable to open 'Release Notes: {0}': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..: Error: Unable to open 'Release Notes: {0}': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..
    at t.doHandleSetInputError (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:1119)
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:725
    at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:16796)
    at t.doSetInput (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:480)
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:30715
    at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:17420)
    at t.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:30687)
    at t.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:29953)
    at e.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:122:10371)
    at e.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:122:9801)e.doShow @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:103
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:101 [File Watcher] Warning: Watcher is busy catching up wit 27232 file changes in 60 seconds. Latest path is 'c:\Projetos\AtlasV2\dev\Atlas\Atlas.WebApp\node_modules\.staging\material-design-icons-0f65fbca\hardware\2x_web'
27/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:101 [File Watcher] System.IO.InternalBufferOverflowException: Too many changes at once in directory:c:\Projetos\AtlasV2\dev\Atlas\Atlas.WebApp.
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:103 Unable to open 'Release Notes: {0}': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..: Error: Unable to open 'Release Notes: {0}': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..
    at t.doHandleSetInputError (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:1119)
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:725
    at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:16796)
    at t.doSetInput (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:480)
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:30715
    at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:17420)
    at t.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:30687)
    at t.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:29953)
    at e.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:122:10371)
    at e.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:122:9801)e.doShow @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:103
3/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:71 Cannot execute type because no code editor is focused.
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:103 Unable to open 'Release Notes: {0}': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..: Error: Unable to open 'Release Notes: {0}': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..
    at t.doHandleSetInputError (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:1119)
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:725
    at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:16796)
    at t.doSetInput (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:480)
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:30715
    at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:17420)
    at t.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:30687)
    at t.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:29953)
    at e.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:122:10371)
    at e.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:122:9801)e.doShow @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:103
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:142 Cannot set property 'column' of undefined: TypeError: Cannot set property 'column' of undefined
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:137:21278
    at Object.g [as _notify] (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:10581)
    at Object.enter (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:14159)
    at _run (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:15987)
    at _completed (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:15428)
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:88:1700
    at g.e.dispatch (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:88:3056)
    at g.e.handleData (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:88:2564)
    at Socket.<anonymous> (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:88:1565)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7) (2 errors in total)e.onUnexpectedError @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:142
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32 WARNING: Promise with no error callback:737
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32 Object
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:136 [Extension Host] ApplicationInsights:Sender Array[1]
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:142 getaddrinfo ENOENT vscode-update.azurewebsites.net:443

Error: getaddrinfo ENOENT vscode-update.azurewebsites.net:443
    at Object.exports._errnoException (util.js:1026:11)
    at errnoException (dns.js:33:15)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:76:26): Error: getaddrinfo ENOENT vscode-update.azurewebsites.net:443

Error: getaddrinfo ENOENT vscode-update.azurewebsites.net:443
    at Object.exports._errnoException (util.js:1026:11)
    at errnoException (dns.js:33:15)
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:76:26)
    at Timeout._onTimeout (C:\Program Files (x86)\Microsoft VS Code\resources\app\out\vs\code\electron-main\main.js:5:1308)
    at tryOnTimeout (timers.js:232:11)
    at Timer.listOnTimeout (timers.js:202:5)e.onUnexpectedError @ /C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:142
https://marketplace.visualstudio.com/_apis/public/gallery/extensionquery Failed to load resource: net::ERR_NETWORK_IO_SUSPENDED
/C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:103 Unable to open 'Release Notes: {0}': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..: Error: Unable to open 'Release Notes: {0}': Cannot read property 'replace' of null
Please report this to https://github.com/chjj/marked..
    at t.doHandleSetInputError (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:1119)
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:725
    at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:16796)
    at t.doSetInput (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:135:480)
    at file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:30715
    at then (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:32:17420)
    at t.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:30687)
    at t.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:134:29953)
    at e.doOpenEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:122:10371)
    at e.openEditor (file:///C:/Program Files (x86)/Microsoft VS Code/resources/app/out/vs/workbench/electron-browser/workbench.main.js:122:9801)

After restart VSCode, the Help>"Check for updates..." button became available, and now it seems to work properly.

I'll wait to see if this update request pops randomly along the day. The breakpoint is ready at the file you suggested.

Awesome, remember to keep the Developer Tools window open, otherwise the breakpoint won't be hit.

Now reproing in PROD.

@martellaj Can you use a network inspector like Charles proxy, and inspect the requests done between Code and the update server?

@joaomoreno Hm, I clicked Check For Updates... to try to get the banner to pop again, but now it's telling me there are no updates. I didn't update this morning.

@joaomoreno The banner popped up again. I tried using Fiddler to capture any requests made when I clicked to view release notes, but none were made. I assume the call happens before the banner is displayed, but I didn't have Fiddler running then.

Yeah, I'd be interested in capturing the request which fires up the update notification.

Closing this as dupe of #18714


Paper filter

Is there any way to put a new dropdown list to filter papers by language? I use to publish in English and in Portuguese, so it would be great if I can put a filter on publication page. Of course I will not publish in more than 2 languages, so maybe a dropdown is too much. Maybe it can be buttons or tabs.

This feature is unlikely to be implemented. Please use the forums for help with customization: https://discourse.gohugo.io


It is an empty repository!!!

Are you kidding us?

Dear Kaikai,
Thanks for your interests in our codes. We are still finalizing the codes
and will release it in by the end of this month.
Yours sincerely,
Lingfei (Teddy) Wu
Research Staff Member, AI Foundations Labs - Reasoning
IBM Research AI
IBM T. J. Watson Research Center, Yorktown Heights, NY 10598
Phone: +1<PHONE_NUMBER>
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>On Mon, Sep 10, 2018 at 10:28 AM Kaikai Zhao<EMAIL_ADDRESS>wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/IBM/SpectralClustering_RandomBinning/issues/1, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMWHNb6IP3_5Jz6si7pU5_J8krQnnF_Fks5uZncMgaJpZM4WhhJI
.

Thanks for your nice response! Your work is solid. I've been reading your papers these days.


vscode-extensions.ms-dotnettools: update C# and C# DevKit extensions

Updates the VSCode C# extension with the following changes:

v2.45.20: https://github.com/dotnet/vscode-csharp/releases/tag/v2.45.20
v2.45.25: https://github.com/dotnet/vscode-csharp/releases/tag/v2.45.25
v2.50.25: https://github.com/dotnet/vscode-csharp/releases/tag/v2.50.25
v2.55.28: https://github.com/dotnet/vscode-csharp/releases/tag/v2.55.28
v2.55.29: https://github.com/dotnet/vscode-csharp/releases/tag/v2.55.29

And the VSCode C# DevKit extension with the following changes: https://marketplace.visualstudio.com/items/ms-dotnettools.csdevkit/changelog
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 363156

x86_64-linux

:white_check_mark: 2 packages built:

vscode-extensions.ms-dotnettools.csdevkit
vscode-extensions.ms-dotnettools.csharp

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 363156

x86_64-linux

:white_check_mark: 2 packages built:

vscode-extensions.ms-dotnettools.csdevkit
vscode-extensions.ms-dotnettools.csharp


lwcb: Fix two BTF-related issues

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

Reviewed-by: Wenan Mao


:hammer_and_wrench: Add Harden Runner to improve CI-security (2/2) (#956)

Some action are only run when merged into the main branch. This appiles the recommended configuration of harden-runner from those actions.
Some actions will need to be more strictly configured later.

Cool! So first you can audit to record the allowlist and then block it, right?
Regards,
Aman Sharma
PhD Student
KTH Royal Institute of Technology
School of Electrical Engineering and Computer Science (EECS)
Department of Theoretical Computer Science (TCS)
http://www.kth.sehttps://www.kth.se/profile/amanshahttps://www.kth.se/profile/amansha
https://www.kth.se/profile/amanshahttps://algomaster99.github.io/

From: Elias Lundell @.***>
Sent: Wednesday, October 30, 2024 9:54:11 PM
To: chains-project/maven-lockfile
Cc: Aman Sharma; Comment
Subject: Re: [chains-project/maven-lockfile] :hammer_and_wrench: Add Harden Runner to improve CI-security (2/2) (#956) (PR #958)
@LogFlames commented on this pull request.

In .github/workflows/Lockfile.ymlhttps://github.com/chains-project/maven-lockfile/pull/958#discussion_r1823377917:

       github.com:443

       objects.githubusercontent.com:443

       raw.githubusercontent.com:443

       repo.maven.apache.org:443

       repo1.maven.org:443

       sh.jbang.dev:443

https://app.stepsecurity.io/github/chains-project/maven-lockfile/actions/runs/11597128578
—
Reply to this email directly, view it on GitHubhttps://github.com/chains-project/maven-lockfile/pull/958#discussion_r1823377917, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIMPTOMXSVFTHOEEDZXJH4DZ6FBPHAVCNFSM6AAAAABQ4OC4WWVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDIMBWGI4TANRVGA.
You are receiving this because you commented.Message ID: @.***>

Yes precisely, that's why I split the PR into two as well. The workflow had to run to collect all endpoints. And the publish workflow is still only in audit state. My plan is to record endpoints and hard-code them when I do the next release, although I might keep it in audit mode since it can be quite difficult to fix if the workflow breaks/is blocked half way through. And it can still be useful to check which endpoints were connected to afterwards - It would not prevent a leak but then you can see that you need to rotate the tokens etc
I am missing some option to make an allowlist for file writes as well. Harden-runner currently reports that there are for example 5 file write operations and list the files. But I want to have it report especially if those files changes. Now it's infeasible to check each workflow-run and compare which files were written to.

But it was very easy to add to the project!
And they have a community version which is free for public repos. It doesn't include all features but do the basic and blocks un-allowlisted connections.
They can also create a "security pr" with best practises where they e.g. add depandabot, and setup some other recommended pre-commit hooks etc. I looked through it and dont think it applies to this project, but it could be an easy way to add good defaults.

Although it seems some endpoints are not connected to all the time. For example the repo.maven.apache.org:443 was new for a secondary run (https://app.stepsecurity.io/github/chains-project/maven-lockfile/actions/runs/11600602474?jobid=32301416922&tab=recommendations). It would probably be a good idea to let it run a couple of times and connect endpoints, but I thought github actions would be deterministic :thinking:

For example the repo.maven.apache.org:443 was new for a secondary run (app.stepsecurity.io/github/chains-project/maven-lockfile/actions/runs/11600602474?jobid=32301416922&tab=recommendations). It would probably be a good idea to let it run a couple of times and connect endpoints, but I thought github actions would be deterministic

That is very strange :open_mouth:
Can we know the exact endpoint with path? Why was it queried only in the second run?


Migration error

What steps will reproduce the problem?
migration error
What's expected?
fix
What do you get instead?
Additional info
Array to string conversion

Q
A

Yii version
2.0.13

Yii MongoDB version
^2.1

MongoDB server version
3.2.7

PHP version
5.6.31

Operating system
linux

works!


Fix TQDM error during ingestion

Error that was sent to me from a user over slack
TQDM error when running ingestion:
Process Process-3:
Traceback (most recent call last):
  File "/Users/zhiling/.pyenv/versions/3.7.2/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
    self.run()
  File "/Users/zhiling/.pyenv/versions/3.7.2/lib/python3.7/multiprocessing/process.py", line 99, in run
    self._target(*self._args, **self._kwargs)
  File "/Users/zhiling/Documents/work/dev/feast/sdk/python/feast/feature_set.py", line 465, in _listener
    for _ in iter(q.get, None):
  File "/Users/zhiling/.pyenv/versions/3.7.2/lib/python3.7/multiprocessing/queues.py", line 93, in get
    with self._rlock:
  File "/Users/zhiling/.pyenv/versions/3.7.2/lib/python3.7/multiprocessing/synchronize.py", line 95, in __enter__
    return self._semlock.__enter__()
OSError: [Errno 9] Bad file descriptor

Fixed by #307


Adding new numpy typing conflicts with get_source_schema

Recently numpy has introduced new typing for array and dtypes:
https://numpy.org/devdocs/reference/typing.html
However, our machinery for calculating the source schema finds some bugs because of (at least right now) two reasons:
1 ) They are made of the Union (in the typing sense) of many types and some of those do not have an attribute name which we use in our get_schema_from_method_signature function:
https://github.com/catalystneuro/nwb-conversion-tools/blob/16126dbd3039444a3c74fa8a371cc43a596b07e5/src/nwb_conversion_tools/utils/json_schema.py#L76

Sometimes there are multiple hits with respect to our types to json maping and the following lines fail.
https://github.com/catalystneuro/nwb-conversion-tools/blob/16126dbd3039444a3c74fa8a371cc43a596b07e5/src/nwb_conversion_tools/utils/json_schema.py#L81

I opened PR catalystneuro/nwb-conversion-tools#544 to showcase this issue.

Wow, this issue is so old it refuses to be linked to the active PR to solve it
@h-mayorquin WDYT, can we close this now with https://github.com/catalystneuro/neuroconv/pull/1016?

Let's close it x D


To Reproduce
Steps to reproduce the behavior:

modify samples/subsys/mgmt/mcumgr/smp_svr/overlay-bt.conf ,   delete CONFIG_MCUMGR_SMP_SHELL=y, add CONFIG_NEWLIB_LIBC=y
west build  -b esp32  samples/subsys/mgmt/mcumgr/smp_svr  --   -DOVERLAY_CONFIG=overlay-bt.conf
west flash

Logs and console output
=~=~=~=~=~=~=~=~=~=~=~= PuTTY log 2022.05.28 18:25:44 =~=~=~=~=~=~=~=~=~=~=~=
ets Jun  8 2016 00:22:57
rst:0x10 (RTCWDT_RTC_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff8598,len:4908
ho 0 tail 12 room 4
load:0x40093000,len:13804
load:0x4009b800,len:5976
entry 0x40093448
[esp32] [INF] Enabling RNG early entropy source...
[esp32] [INF] *** Booting MCUboot build v1.7.0-485-ge86f575 ***
[esp32] [INF] Primary image: magic=good, swap_type=0x1, copy_done=0x3, image_ok=0x1
[esp32] [INF] Scratch: magic=unset, swap_type=0x1, copy_done=0x3, image_ok=0x3
[esp32] [INF] Boot source: primary slot
[esp32] [INF] Swap type: none
[esp32] [INF] Disabling RNG early entropy source...
[esp32] [INF] br_image_off = 0x10000
[esp32] [INF] ih_hdr_size = 0x20
[esp32] [INF] Loading image 0 - slot 0 from flash, area id: 1
[esp32] [INF] DRAM segment: start=0x58a6, size=0x1dcc, vaddr=0x3ffbdb60
[esp32] [INF] IRAM segment: start=0x7674, size=0x187dc, vaddr=0x40080000
[esp32] [INF] start=0x4009052c
Fatal exception (28): LoadProhibited
epc1=0x40096f6f, epc2=0x00000000, epc3=0x00000000, excvaddr=0x00000000, depc=0x00000000
Under is the zephyr.map file:
0x0000000040096f64      0x11d /home/sam/zephyr-sdk-0.14.2/xtensa-espressif_esp32_zephyr-elf/xtensa-espressif_esp32_zephyr-elf/lib/./libc.a(lib_a-svfiprintf.o)
0x121 (size before relaxing)
0x0000000040096f64                __ssprint_r
fill         0x0000000040097081        0x3
Environment (please complete the following information):

OS: ubuntu
Toolchain: Zephyr SDK 0.14.2
zephyr ver 3.1 rc2

@sam131208 This error is happening due to a missing PR in zephyr's mcuboot:  https://github.com/mcu-tools/mcuboot/pull/1362`
This should be already synced but it was not. I asked maintainers to take a look. This is same root cause for this: #45349.
So I will close this as duplicated, ok?


http/net exception - in dll context

What version of Go are you using (go version)?
go 1.10 windows / amd64
Does this issue reproduce with the latest release?
i don't know
What operating system and processor architecture are you using (go env)?
windows
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\code34\AppData\Local\go-build
set GOEXE=.exe
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows
set GOPATH=C:\Users\code34\go
set GORACE=
set GOROOT=C:\Go
set GOTMPDIR=
set GOTOOLDIR=C:\Go\pkg\tool\windows_amd64
set GCCGO=gccgo
set CC=gcc
set CXX=g++
set CGO_ENABLED=1
set CGO_CFLAGS=-g -O2
set CGO_CPPFLAGS=
set CGO_CXXFLAGS=-g -O2
set CGO_FFLAGS=-g -O2
set CGO_LDFLAGS=-g -O2
set PKG_CONFIG=pkg-config
set GOGCCFLAGS=-m64 -mthreads -fmessage-length=0 -fdebug-prefix-map=C:\Users\code34\AppData\Local\Temp\go-build638089086=/tmp/go-build -gno-record-gcc-switches
What did you do?
I have a C program wich call a GO dll. The GO dll use net/http to retrieve a distant json. You can retrieve the C program and the go source file at this URL:
https://www.dropbox.com/s/21k6nm75zlxn68q/armaws_x64.zip?dl=0
You only have to call it in powershell, it will load automatically the dll file
.\callextension_x64.exe .\test_script.sqf
What did you expect to see?
The content of my http request. The same code source works when it's a simple .exe file.
What did you see instead?
An exception crash
Exception log file is at this place:
log.txt

@code34 please include the full text of the crash you received (please do not include a screenshot of the text)

done i just at the log file at the end of post

Thank you. I don't understand where this exception is being produced. Is it possible to get the Go stack trace?

yes of course. how can i retrieve it ?

I'm sorry I don't know. Hopefully someone else can help you.

You can send the kill -ABRT signal to the process to get goroutine stack traces.

Looking at your program you are calling
memmove(output, result, len(output))
so if the length of output is larger than the result, memmove will copy
more bytes than is in result and cause an exception.
On 13 August 2018 at 18:22, Nicolas BOITEUX<EMAIL_ADDRESS>wrote:

the body exception message change during each execution. I thought that a
go routine continues to run when the dll finishes its execution. If i add a
fmt.Println(data), i see the result is well returned.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/26952#issuecomment-412443400, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAcA-Vg1FNmXy1OysnOFtYpdlIKAjZnks5uQTc5gaJpZM4V6CB0
.

for the memmove it has been tested, you can for exemple return:
data = "hello"
without http/net code it works, with http there is an exception.
how can i send kill -ABRT on windows ? is it a simple kill throught task manager ?

i test again and effectively you are right. there is something relative to the memmove but i dont understand what. Cause when i use a very large ouput it doesn't create exception.

@code34 is this the test you mean?
//export RVExtension
func RVExtension(output *C.char, outputsize C.size_t, input *C.char) {
	data:= "hello"
	result := C.CString(data)
	defer C.free(unsafe.Pointer(result))
	C.memmove(unsafe.Pointer(output), unsafe.Pointer(result), outputsize - 1)
}

Do you see that C.memmove will attempt to copy outputsize - 1 bytes from result to output irrespective of the size of data?

thanks you davecheney i better understand now 👍
i only manage the long variable size not the short.


The results of evaluate.py about entity

After running evaluate.py, I got the result about entity as follow. It seems different from the result in your paper.
Could you tell me what's wrong with it? Thanks！
Info (entity)
Number of unique entities: 652
Number of malicious entities: 11
Result (entity)
TP: 11
TN: 641
FP: 0
FN: 0

It seems evaluate.py for S1 does not show the right numbers, the evaluate.py shows smaller number than the actual numbers.
The reason for this failure is that if you read the generated evaluation file:
output/eval_seq_graph_testing_preprocessed_logs_S1-CVE-2015-5122_windows.dot.txt.json
You will see all the entities lumped together, thus, many entities can be grouped as one entity like this "Entity1;Entity2; ...":
c:/users/aalsahee/appdata/roaming/microsoft/windows/recent/customdestinations/atd2u3m1o6uzd4q4dugw.temp**;**c:/users/aalsahee/appdata/roaming/mozilla/firefox/profiles/a6gl280b.default/sessionstore.js.tmp**;**c:/users/aalsahee/a....
evaluate.py still counts these lumped entites as one entity! that's way these numbers generated by evaluate.py are smaller than the actual numbers.
During writing the paper I must have implemented another script to consider the numbers of these lumped entities. I don't have it anymore, but evaluate.py can be easily fixed to consider the numbers of these lumped entities. I hope this helps. Thanks.


[Flutter] In Android REST API Calls are slow compared to iOS

Flutter 3.13.9 • channel stable • https://github.com/flutter/flutter.git
Framework • revision d211f42860 (4 weeks ago) • 2023-10-25 13:42:25 -0700
Engine • revision 0545f8705d
Tools • Dart 3.1.5 • DevTools 2.25.0
We are creating business-critical application in flutter for Android and IOS.
This application is mainly based on REST API calls and we are using the http package for network library.
Observed significant performance change while comparing Android and IOS in REST API calls response time. IOS is faster while comparing the Android response time.
As I searched the existing issues there is no relevant information.
So I request your kind attention on this important issue.

The http package is here, but it's likely that its performance characteristics come from dart:io's HTTP implementation.

I am not familiar with the http client implementations, but I was wondering if you could try using https://pub.dev/packages/cronet_http as the underlying package:http client on Android? It is experimental, but might provide a useful point of comparison.

This is not really actionable unless you provide more information and some sort of reproduction. There are so many factors involved here that the problem can be anywhere in the stack (including differences in the networking stack and even hardware).
iOS device might simply be faster than Android device you are comparing against - start by making sure they are comparable.


Add pollution roses plots

After we get the wind roses into MELODIES MONET, it would be nice to connect this to pollution roses too. Gabi from NCAR can help suggest how to set up this analysis if there are questions. Here are some examples:
https://peerj.com/articles/4759/ (Figure 3)
https://agupubs.onlinelibrary.wiley.com/doi/full/10.1002/2016JD025327 (Figure 5)

I've made a plot similar to this using the windrose library described in #230


Integrate search API with faceting

In order of priority @serjoshua

[x] mapping_justification
[x] predicate_id
[ ] mapping_provider
[ ] object_source
[ ] confidence
[ ] mapping_set_title

Dependent on https://github.com/anitacaron/sssom-api/issues/52


rectprops update

Re-forked and new pull request to implement changes for deprecated rectprops with props in matplotlib function call. This version checks for older  versions of matplotlib to continue with old behavior.

@smoia do you know what is this check that failed? .pre-commit-config.yaml is not a file

Hey @afni-dglen ! Could you rebase this PR on the latest master? This should take care of a few issues. After that, @maestroque , do you want to give a try at a PR review? Otherwise @me-pic and @m-miedema could help with it?

Hey @afni-dglen !
I believe the things that remain for this PR are:

rebasing from master branch (this will also take care of pre-commit because it was added in a later patch)
removing the matplotlib constraint from setup.cfg

This PR is currently blocking https://github.com/physiopy/phys2denoise/pull/54, due to the incompatibility of matplotlib throughout peakdet, phys2denoise and physutils, so it would be really appreciated if this could be handled soon.
Btw @smoia I think once this is merged we should do a new release in peakdet, since the current release is already outdated, and clones dependencies such as gooey and wxpython.

Also, another question is that could we just change the version here at >=3.9, as in phys2denoise and physutils and settle with props instead?
Is there any other compatibility issue with other packages I'm missing?

I think I took care of both those issues. Rebasing was problematic, so another fork and a new pull request.
https://github.com/physiopy/peakdet/pull/74
I'll close this one then.


improvement: track supertree, and add Zipper.all_the_way_up/1

I don't really think that all_the_way_up/1 is a great name, and I had to resist the urge to call it Zipper.tippy_top/1 😂
Open to suggestions.

Should we also include topmost_root mimic root but that can be called from a subtree too?

Happy to add topmost_root as well :)


BSD-3-clause variants no longer match properly

At 5cc4f616f8db, a BSD-3-Clause variant like https://github.com/d0ngw/cache/blob/v0.1.0/LICENSE no longer matches well (we are back to a split BSD-2/BSD-3). I think the difference is "names" in the license text vs. "name" in the template.

Changing the relevant part of the template to
Neither ___ be used

fixes many, but there are still three that don't match now but did before.

https://github.com/lwes/lwes-go/blob/f32784a040b3887eebbd4a5f83d33916f12d0767/LICENSE, where clause 3 ends with "... permission of OpenX Technologies, Inc.", matches BSD-3 at 89.3.

https://github.com/smallfish/rabbitmq-http/blob/master/LICENSE.md similarly ends in "permission of Mozilla Foundation" and matches at 89.2%.

https://github.com/AndreyZWorkAccount/XIRR/blob/master/LICENSE changes clause 3 significantly. It may be better that we don't match it.

All fixed. Not worried about the change in AndreyZWorkAccount.


Kill old job if runs too long?

Is there a way to do this. I looked at the strategy and it seems like until_and_while_executing  is the right one with  on_runtime_conflict: :raise .
However, does it raise and kill the old job or simply doesn't start a new job?
Great library btw for us rails users. Thanks for making this 🙏

Hello @Aryk

Kill old job if runs too long?

It is not a responsibility of this gem. Moreover, I recommend to avoid Ruby Timeouts in favor of other approaches.

does it raise and kill the old job or simply doesn't start a new job?

Before the job processing, the strategy unlocks it. In case of the locking conflict the job is not started. A new job is allowed to be enqueued though.
https://github.com/veeqo/activejob-uniqueness/blob/a48b293f3910093ae4aeffe0fe822db791d2213c/lib/active_job/uniqueness/strategies/until_and_while_executing.rb#L24-L31

Great library btw for us rails users. Thanks for making this

Thanks! Cheers :pray:


Abount parameter neighbor_num

Hi, I'm a little confused about the parameter neighbor_num in the JSON file. neighbor_num should control the number of neighboring cells to incorporate when making imputation. the wiki in Higashi-Usage Step3 says that

Train Higashi with cell-dependent GNN, but with k=0

but the wiki in Output-of-Higashi-main  says that

The {k} can be either 1 or the {neighbor_num} parameter specified in the configuration file. When {k}=1, it represents the imputation results without using any neighboring cell information.

which does not agree with the wiki in Higashi-Usage and the paper.
Is it a mistake expression in Output-of-Higashi-main section? can I think the

{chrom_name}_{embedding_name}_nbr_1_impute.hdf5

I get to be the result of 0 nbr in the paper?
another question is that when nbr=0, does the hypergraph built for GNN use only the information from one cell when imputing this cell?
Thanks!

Thanks for your interest in our work and for bringing this problem into my attention. There is some inconsistency for the description of the parameter k. Current neighbor_num=1 is equivalent to k=0 described in the paper.  And "{chrom_name}_{embedding_name}_nbr_1_impute.hdf5" is (0 nbr) results.
And yes, when $k=0$, the information only from 1 cell.
I'll update the code and the wiki to fix this very soon.
We will also have a major update of Higashi very soon which includes improved embedding&imputation results, runtime optimization, etc. Feel free to try the new version when it's released.

Thank you for your reply!
I've done imputation on a dataset containg multiple cells, and I find the nbr=0 imputation result turns out great. Now that nbr=0 means imputation needs no information from other cells, I want to find out what will happen if I impute one cell at a time.
I did this, but I got an exception:

Traceback (most recent call last):
File "/gs/home/zengguangjie/Higashi/Code/main_cell.py", line 867, in
embeddings_initial, attribute_dict, targets_initial = generate_attributes()
File "/gs/home/zengguangjie/Higashi/Code/main_cell.py", line 496, in generate_attributes
a = np.load(os.path.join(temp_dir, "%s_cell_PCA.npy" % c))
File "/gs/home/zengguangjie/anaconda/envs/higashi/lib/python3.7/site-packages/numpy/lib/npyio.py", line 416, in load
fid = stack.enter_context(open(os_fspath(file), "rb"))
FileNotFoundError: [Errno 2] No such file or directory: '/gs/home/zengguangjie/Higashi/TOKI_data/tempdir/chr1_cell_PCA.npy'

when imputing this cell using main_cell.py. Must I put multiple cell data in to get the imputation result? Is there any chance I can impute only one cell a time?
Thanks!

Ah.. I can see why that happens.
When calculating node features for cell nodes, we use PCA or SVD on the contact maps for that cell. And when you only inputing one cell... It causes error for doing PCA or SVD (as you can imagine... there's no covariance matrix to begin with). Hence the error "not found cell_PCA"...
I'll adapt the code a little bit to support inputing one cell at a time in our recent release (probably this weekend to early next week). But generally speaking, Higashi performs better on larger dataset (similar to all deep learning based methods).

Thank you for your help!
I'm looking forward to your update.

Hi, I'm glad to see your update about inputting one cell at a time. I tried with your updated code, but find some exceptions.
I encountered an exception when running Process.py

concurrent.futures.process._RemoteTraceback:
"""
Traceback (most recent call last):
File "/gs/home/zengguangjie/anaconda/envs/higashi/lib/python3.7/concurrent/futures/process.py", line 239, in _process_worker
r = call_item.fn(*call_item.args, **call_item.kwargs)
File "/gs/home/zengguangjie/Higashi/Code/Process.py", line 602, in generate_feats_one
temp1 = np.eye(temp1.shape[0])
UnboundLocalError: local variable 'temp1' referenced before assignment
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/gs/home/zengguangjie/Higashi/Code/Process.py", line 829, in
create_matrix()
File "/gs/home/zengguangjie/Higashi/Code/Process.py", line 403, in create_matrix
temp1, c = p.result()
File "/gs/home/zengguangjie/anaconda/envs/higashi/lib/python3.7/concurrent/futures/_base.py", line 428, in result
return self.__get_result()
File "/gs/home/zengguangjie/anaconda/envs/higashi/lib/python3.7/concurrent/futures/_base.py", line 384, in __get_result
raise self._exception
UnboundLocalError: local variable 'temp1' referenced before assignment

the code is temp1 = np.eye(temp1.shape[0])at line 602 in Process.py. I guess the second "temp1" is a mistake of "temp", so I changed it totemp1 = np.eye(temp.shape[0]) and tried again. But I encountered anoter exception.

File "/gs/home/zengguangjie/Higashi/Code/main_cell.py", line 1172, in
load_first=False, save_embed=True, save_name="_stage1")
File "/gs/home/zengguangjie/Higashi/Code/main_cell.py", line 552, in train
model, loss, training_data_generator, optimizer)
File "/gs/home/zengguangjie/Higashi/Code/main_cell.py", line 137, in train_epoch
batch_edge_weight, batch_chrom, batch_to_neighs, y=batch_y)
File "/gs/home/zengguangjie/Higashi/Code/main_cell.py", line 47, in forward_batch_hyperedge
pred, pred_var, pred_proba = model(x, (batch_chrom, batch_to_neighs))
File "/gs/home/zengguangjie/anaconda/envs/higashi/lib/python3.7/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/gs/home/zengguangjie/Higashi/Code/Higashi_backend/Modules.py", line 737, in forward
dynamic, static, attn = self.get_embedding(x, x_chrom)
File "/gs/home/zengguangjie/Higashi/Code/Higashi_backend/Modules.py", line 708, in get_embedding
raise EOFError

any help will be helpful.
thanks!

I see. The second one is the "nan error". I'll test the code on a dataset with only one cell to see if that triggers the bug.

Hi,
Could you try the latest version again? I can finish training on a dataset with only one cell without any error. If the error raise again, could you share the dataset and the configuration file that can reproduce the error?
Also, when inputing one cell at a time, it would be helpful to reduce the number of epochs and embedding size to prevent potential overfitting.

Hi. I tried the latest version and it workes fine.
Can you give some specific recommendations on the number of epochs and the embedding size when inputing one cell at a time? We want to see how much improvement can Higashi make on our downstream tasks tool.
Thank you for your help!

The choice of epochs can be flexible. I would say 1 for "embedding_epoch", since you are not using the embeddings of that one cell. As for the "no_nbr_epoch", you can set it as something like 10~20 as a start point, the program would terminate if the loss do not improve for 6 epochs. You can then use that as a guidance.
For embedding size, 32-128 should be good.

Thank you for your help!


Add The text "My Changes " To the Artical

[x] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Stop putting the words 'My Changes' in random places.


Bump Rails and govuk_admin_template for security fixes

@fofr has approved this admin gem bump.

You don't necessarily want to do that - that's upgrading everything at once. Get a clean checkout of Gemfile.lock, remove treetop and polyglot manually from it, then bundle update rails.

@boffbowsh Yeah, I'd noticed that and thought that wasn't a great idea. I just tried not doing that, so let's see...

It's fixed now. :-) The multiple polyglots disappeared to learn more languages.


jitsi-videobridge: 2.3-44-g8983b11f -> 2.3-59-g5c48e421

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for jitsi-videobridge is: A WebRTC compatible video router
meta.homepage for jitsi-videobridge is: https://github.com/jitsi/jitsi-videobridge
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.3-59-g5c48e421 in filename of file in /nix/store/c6gh3ndnf5kmr9sh52ip368fjkgfsass-jitsi-videobridge2-2.3-59-g5c48e421

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
jitsi-videobridge

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/c6gh3ndnf5kmr9sh52ip368fjkgfsass-jitsi-videobridge2-2.3-59-g5c48e421 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jitsi-videobridge https://github.com/r-ryantm/nixpkgs/archive/841bd4d99c51eea4722cbf4022f33ce5de673043.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/841bd4d99c51eea4722cbf4022f33ce5de673043#jitsi-videobridge

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/c6gh3ndnf5kmr9sh52ip368fjkgfsass-jitsi-videobridge2-2.3-59-g5c48e421
ls -la /nix/store/c6gh3ndnf5kmr9sh52ip368fjkgfsass-jitsi-videobridge2-2.3-59-g5c48e421/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

jitsi-videobridge

Maintainer pings
cc @cleeyv @ryantm @Lassulus for testing.

tested on my jitsi-instance