runtime: difficulty handling system page aligned stacks

The current runtime stack allocation is based on mheap, which uses its own fixed page size (4096 bytes). On some systems there is a requirement to have stacks be system page aligned (for example, 16KB alignment for OpenBSD/octeon). In this case, if mheap provides memory that is not 16KB aligned various things fail.
There are a couple of options to address this:

Make it possible to system allocate the stacks (i.e. malloc) - this is the simplest option.

Over allocate via allocManual() and round to system page alignment during stack allocation - this should not require changes to mheap, however would require changes to type stack, which would need to track the actual allocation (address and size), in addition to the lo/hi pointers (this is needed in order for stackfree() to correctly return the allocation). Two additional pointers in this struct are probably acceptable, however various assembly knows about its size and offset.

Change mheap to provide system page size aligned allocations - this would be doable, but seems to require a fair amount of effort, however avoids the need to change type stack (and various assembly).

For the time being I plan on using (1) for the openbsd/mips64 port, however we may want to consider alternatives at a later date.

On some systems there is a requirement to have stacks be system page aligned

I'm not sure I understand this requirement. How does the system ensure it? What does the system check?

@cherrymui I assume this is associated with MAP_STACK.

https://man.openbsd.org/mmap.2
https://marc.info/?l=openbsd-tech&m=152035796722258&w=2

How big is the physical page size on the system? If we set in the runtime the physical page size as at least 16KB, then all mmap should be 16K aligned, including MAP_STACK. I assume the actual goroutine stack does not necessarily start at the beginning of the mmap, and it would probably be fine.

How big is the physical page size on the system? If we set in the runtime the physical page size as at least 16KB, then all mmap should be 16K aligned, including MAP_STACK.

@cherrymui - OpenBSD on Octeon runs with 16KB physical pages. Changing the runtime page size to 16KB would be one solution (however that seemed like a larger hammer than might be acceptable).

I assume the actual goroutine stack does not necessarily start at the beginning of the mmap, and it would probably be fine.

I believe this is correct. The failures are when, for example, setting an alternate signal stack and passing a stack that is not physical page aligned.

The runtime should query the physical page size at startup and only do physical-page-aligned mmaps, regardless of runtime's internal page size. If that is not the case, we probably should fix that.
Maybe we just need to round the address when calling sigaltstack?

The runtime should query the physical page size at startup and only do physical-page-aligned mmaps, regardless of runtime's internal page size. If that is not the case, we probably should fix that.

Two mmaps are required on OpenBSD - the first is done for allocation, the second is to mark an allocation as being stack (MAP_STACK). The problem is that due to the way stack allocation is currently performed, we end up in the following situation:

Go allocates memory for mheap via mmap.
A stack allocation is requested from mheap - this allocation is part of the earlier mmap, but rounded to the mheap fixed page size (4096 bytes) (i.e. we're no longer physical page aligned).
On OpenBSD, this stack allocation needs to be marked as MAP_STACK - Go calls mmap with the 4096 byte rounded allocation, which fails as the allocation is not physical page aligned.

I'm not sure how we would fix/avoid this situation without (2) or (3) from the original issue comment.

Can we round the mmap in (3) to physical page size? I guess it would not be a problem if we map more?

Can we round the mmap in (3) to physical page size? I guess it would not be a problem if we map more?

Hrmmm, we can likely round a stack allocation down to get physical page alignment and then round up to get a multiple of physical page size. The problem would then potentially be two back to back stack allocations, that would then have overlapping mmaps - I'll have to see what the kernel says if we do that (and particularly if we remove one of them again).

So I do not think this is going to work - if we do an mmap (96 physical pages) we get:
0000002758FA0000   1536K read/write          [ anon ]

We then mmap again one physical page into the previous allocation and 32 physical pages in size:
0000002758FA0000     16K read/write          [ anon ]
0000002758FA4000    512K read/write          [ anon ]  <- stack mapped a
0000002759024000   1008K read/write          [ anon ]

We then mmap again, 32 physical pages into the original allocation and 32 physical pages in size (resulting in a single physical page overlap with the previous allocation, as would be required in the round down/up scenario):
0000002758FA0000     16K read/write          [ anon ]
0000002758FA4000    496K read/write          [ anon ] <- stack mapped a
0000002759020000    512K read/write          [ anon ] <- stack mapped b
00000027590A0000    512K read/write          [ anon ]

We then stop using the second stack and remove MAP_STACKMAP:
0000002758FA0000     16K read/write          [ anon ]
0000002758FA4000    496K read/write          [ anon ] <- stack mapped a
0000002759020000    512K read/write          [ anon ]
00000027590A0000    512K read/write          [ anon ]

If the top physical page of the first stack mapped allocation is then used and a system call is made (or a signal occurs), the process will now be terminated. So we'd be back to having to over allocate in order to guarantee there was no overlap in the case of rounding down/up. I'll take another look to see if this is possible in combination.

I think basing this off stackFromSystem is the wrong approach, I'm afraid. This approach is going to cause all goroutine stacks to start at 16 KiB on openbsd/mips64, rather than the usual 2 KiB; it's going to throw off stack memory metrics; and it's going to dramatically change scavenging behavior for stack memory. stackFromSystem is really meant for runtime debugging (I'm fine with adding the osStackAlloc on that path, since that does fix that debugging mode on openbsd, but as a separate change).
If I understand the underlying problem, we need to mark MAP_STACK regions on physical page boundaries (which happens to work right now only when the physical page size is <= the runtime page size).
I just discussed this with @cherrymui and @mknyszek and they came up with several ideas. I'll summarize here, but let them weigh in with more details:

If we're switching to libc calls instead of raw syscalls on OpenBSD (are we?), then do we actually need g stacks to be MAP_STACK, or is it enough to have the g0 stacks be MAP_STACK where we do the libc call, and to have the sigaltstacks mapped MAP_STACK? [@cherrymui]
Use a different register as the stack pointer and leave the OS-expected stack pointer register pointing at the g0 stack. [@cherrymui]
Map the Go heap MAP_STACK since Go already mitigates the sorts of C bugs this is meant to defend against.
Add a new spanAllocType, say spanAllocStackPhysPageAligned, and modify allocSpan to recognize this type, skip the page cache, allocate a larger range from the heap, and return any unaligned part back to the heap. [@mknyszek]
Add a way to free part of a span, then in stack allocation ask for a large span, and free the unaligned part.

If the top physical page of the first stack mapped allocation is then used and a system call is made (or a signal occurs), the process will now be terminated. So we'd be back to having to over allocate in order to guarantee there was no overlap in the case of rounding down/up. I'll take another look to see if this is possible in combination.

To close this out, from an operating system perspective this appears to work, however the over allocation required is pretty substantial and there are parts of Go's runtime that seem to dislike this (I've not tracked down exactly why this is yet).

I think basing this off stackFromSystem is the wrong approach, I'm afraid. This approach is going to cause all goroutine stacks to start at 16 KiB on openbsd/mips64, rather than the usual 2 KiB; it's going to throw off stack memory metrics; and it's going to dramatically change scavenging behavior for stack memory. stackFromSystem is really meant for runtime debugging (I'm fine with adding the osStackAlloc on that path, since that does fix that debugging mode on openbsd, but as a separate change).

@aclements thanks for the detailed response - I agree that stackFromSystem is not ideal, but it does allow things to work :)

If I understand the underlying problem, we need to mark MAP_STACK regions on physical page boundaries (which happens to work right now only when the physical page size is <= the runtime page size).

This is correct.

I just discussed this with @cherrymui and @mknyszek and they came up with several ideas. I'll summarize here, but let them weigh in with more details:

If we're switching to libc calls instead of raw syscalls on OpenBSD (are we?), then do we actually need g stacks to be MAP_STACK, or is it enough to have the g0 stacks be MAP_STACK where we do the libc call, and to have the sigaltstacks mapped MAP_STACK? [@cherrymui]

Yes, we need to switch to libc calls (see issue #36435), although this is currently blocked on #39257 (and https://go-review.googlesource.com/c/go/+/249978). This is likely the cleanest and easiest solution, providing that the g0 stacks are always used for libc calls (and therefore system calls). The g0 stack is presumably the OS allocated stack, which will already be MAP_STACK, at which point this entire issue should be moot.

Use a different register as the stack pointer and leave the OS-expected stack pointer register pointing at the g0 stack. [@cherrymui]

Possible, but seems like a lot of effort and a wasted register.

Map the Go heap MAP_STACK since Go already mitigates the sorts of C bugs this is meant to defend against.

This is a pretty big hammer that would allow a ROP stack to be created and run from any part of the heap - while Go mitigates against buffer overflows and makes it harder to gain control flow, it does not necessarily help when cgo or unsafe are in use.

Add a new spanAllocType, say spanAllocStackPhysPageAligned, and modify allocSpan to recognize this type, skip the page cache, allocate a larger range from the heap, and return any unaligned part back to the heap. [@mknyszek]

Hrmmm, that could work reasonably well. I'll have a poke at this.

Add a way to free part of a span, then in stack allocation ask for a large span, and free the unaligned part.

That likely works but seems like more effort the doing it in allocSpan.

Add a new spanAllocType, say spanAllocStackPhysPageAligned, and modify allocSpan to recognize this type, skip the page cache, allocate a larger range from the heap, and return any unaligned part back to the heap. [@mknyszek]

Hrmmm, that could work reasonably well. I'll have a poke at this.

This seems to work reasonably well - https://go-review.googlesource.com/c/go/+/266919

If we're switching to libc calls instead of raw syscalls on OpenBSD (are we?), then do we actually need g stacks to be MAP_STACK, or is it enough to have the g0 stacks be MAP_STACK where we do the libc call, and to have the sigaltstacks mapped MAP_STACK? [@cherrymui]

@cherrymui looked into this a little and determined that this won't work. She points out that the kernel patch mentions "We can also perform this check on standard synchronous traps, for instance page faults." These happen all the time while running on the g stack, so the g stack has to be MAP_STACK.

If we're switching to libc calls instead of raw syscalls on OpenBSD (are we?), then do we actually need g stacks to be MAP_STACK, or is it enough to have the g0 stacks be MAP_STACK where we do the libc call, and to have the sigaltstacks mapped MAP_STACK? [@cherrymui]

@cherrymui looked into this a little and determined that this won't work. She points out that the kernel patch mentions "We can also perform this check on standard synchronous traps, for instance page faults." These happen all the time while running on the g stack, so the g stack has to be MAP_STACK.

Ah, correct - checks are performed on both syscalls and page faults, so a page fault occurring on a g stack would be problematic without it being MAP_STACK. Additionally, sigaltstack() marks the passed memory as MAP_STACK itself so the caller does not actually need to do this ahead of time.


This is a design for the report section

Create A report component that will show the progress of the goals
#352

Please resolve conflict

You don't seem to have any issue attached to this PR, why so please?

It is saying that those with write access has the right to resolve conflict @KelvinAkpobome  and for the issue I didn't link it so the PR does not close automatically......The issue number is #352

Please pull from the staging branch to your local branch and solve the merge conflicts there.
Then add "Closes #352" to your PR's description

Resolve conflicts please


Fix blowing only (Issue 134)

Change "blowingOnly" to "blowingonly" to fix 134.
Not tested, but i think it should work.

@Tonno87 actually the fix is just in the switch/case during command decoding in mqtt_gateway.py
I'd much prefer that as it doesn't break existing HA automations


Adaptation for Telegraph-Influx-Grafana

Hello, been searching for this kind of solution to graph stats of NB6VAC for Telegraph-Influx-Grafana ...
Would this script also work in TIG with some adaptation?
If anyone know how to do, I would be very grateful 😃

I have absolutely no idea. You'll have to write it out and adapt as necessary. Pull requests welcome.


Specifying shaders in PSO

We've been considering including the rasterizer state inside PSO definition. However, there is an elephant in the room - shaders. Currently, passed in as ShaderSet - not the most straightforward or convenient thing to manage. How about specifying the shaders inside PSO definition?

meta
init
data
comment

VertexInput
(Shader<R>, Primitive)
Slice
vertex shader

PixelOutput
(Shader<R>, Rasterizer)
?
pixel shader with the rasterizer

Geometry
(Shader<R>, ..)
?
geometry shader

Tessellator
(Shader<R>, Shader<R>, ..)
?
tessellation

Brief analysis:

(+) removes the concept of ShaderSet
(+) removes all the init and run-time parameters other than the ones associated with PSO definition
(+) nicely extendable to tessellation and compute, exposes vertex/pixel shading being optional
(-) harder to document
(-) doesn't completely isolate the stages. I.e. if PixelOutput is not provided, the RenderTarget, Scissor, BlendTarget, DepthTarget, and other components no longer make sense. Perhaps, these need to be hierarchically under the PixelOutput parent?..

:+1:
I like the idea of listing the components under PixelOutput and friends. Would defining a PSO look roughly like this then?
gfx_pipeline!(terrain {
    VertexInput {
        vbuf: gfx::VertexBuffer<TerrainVertex> = (),
        model: gfx::Global<[[f32; 4]; 4]> = "u_Model",
        view: gfx::Global<[[f32; 4]; 4]> = "u_View",
        proj: gfx::Global<[[f32; 4]; 4]> = "u_Proj"
    },
    PixelOutput {
        light_buf: gfx::ConstantBuffer<grid::LightInfo> = "u_LightData",
        light_grid: gfx::ConstantBuffer<grid::GridTileData> = "u_LightGrid",
        light_index_tex: gfx::ShaderResource<i32> = "u_LightIndexTex",

        out_color: gfx::RenderTarget<Rgba8> = "o_Color",
        out_depth: gfx::DepthTarget<DepthStencil> = gfx::preset::depth::LESS_EQUAL_WRITE
    }
});

This might increase loading time in gfx_graphics, because the shaders need to recompile for every PSO.

@bvssvni Oh no, that's not what's proposed here. We are still going to be compiling shaders as usual. Just instead of providing them in ShaderSet they are gonna have fields inside the PSO definition.

@kvark OK then!
What about shared parameters between shaders? For example, a MVP transform accessed in both vertex and pixel shader.

@bvssvni that's the question of dispute here (among other things).
If the pipeline is split into vertex/pixel/geometry/tessellation/compute pipes, and each have their own parameters, then some of them might need to be repeated. In this case GL backend may be a little pressured into supporting separate shader programs per stage, since otherwise it would have to check that we don't try to set 2 different values to the same parameter.
If all the parameters are shared between stages, there should be no problems. However, it doesn't look as nice when all of them are together. Perhaps, we could have a compromise in the form of Common {} section where you'd put parameters shared between stages, if any.

Still an option for gfx_render

This is fun, but pre-ll has to go, closing.


Fix CPPCheck issues - Known conditional expression is checked

I still need to investigate this one.
/github/workspace/src/TestCPP.cpp:450:31: style: Condition 'testSuiteName.data()' is always true [knownConditionTrueFalse]

This is fixed in #10 with some coincidental changes.


Bug/duplicate links

Codecov Report

Merging #31 into master will increase coverage by 0.15%.
The diff coverage is 72.22%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
+ Coverage   67.11%   67.27%   +0.15%
==========================================
  Files          29       29
  Lines        2725     2738      +13
==========================================
+ Hits         1829     1842      +13
  Misses        896      896

Impacted Files
Coverage Δ

ingest/exporter/ingestexportservice.py
24.46% <72.22%> (+2.73%)
:arrow_up:

Continue to review full report at Codecov.

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


wasm: update V8 to 9.6.152

Before merging this PR, someone with proper access (@PiotrSikora maybe?) should deploy artifacts built by https://storage.googleapis.com/envoyproxy-wee8/wee8-archive.sh to https://storage.googleapis.com/envoyproxy-wee8/wee8-9.6.152.tar.gz
Commit Message: wasm: The current (v<IP_ADDRESS>) contain bug which doesn't let build
envoy on centos 7
Additional Description: none
Risk Level: it's hard to asses by me as I don't have a lot of experience with v8 and couldn't find a good amount of information of changes between these versions
Testing: CI build checks
Docs Changes: none
Release Notes: none
Platform Specific Features: none
Fixes #18365

I'm closing this PR. Thank you for your help!


Choice of port ?

Hello,
As a fellow developper, I noticed that you use the port 5672 for your application.
Could you maybe change that ? This port is commonly used for applications such as RabbitMQ.
Thanks !

Not sure it's documented, so if not it would be great if someone would add this to the Wiki.
https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/blob/master/src/main/features/core/websocketAPI.js#L105
We have the option via environment variable for a non-standard port via GPMDP_API_PORT,  as well as non-standard listening address via GPMDP_API_HOST.


Begin reimplementing emit optimisations

Added precedence to Builder
Partial solution for #99.
pub let main = _ => [(foo + bar) * 3, ((foo + bar) * 3), foo + (bar * 3), foo + bar * 3]
now becomes:
// main :: *
export function main (_) {
    return [(foo + bar) * 3, (foo + bar) * 3, foo + bar * 3, foo + bar * 3]
}

Superceded by commit acab61506019bb4c4531f323000121e005140b60 (on im-so-sorry branch)


Support cross origin in useScript

Add optional attribute crossOrigin to UseScriptOptions with values 'anonymous' | 'use-credentials' | '' | null.
Attach UseScriptOptions.crossOrigin to scriptNode.crossOrigin
Update test cases and demos accordingly

@juliencrn May I have your attention to review it?

@ferrior can you add support for integrity attribute as well?


[tool][Proposal] Allow split-per-abi to use the same version codes for generated apks

flutter build apk --split-per-abi will compile apk with different version code
https://developer.android.com/studio/build/configure-apk-splits#configure-APK-versions
I need the version code to be the same

Hi @git-xiaocao, what is your case for this? According to the playstore guidelines, each apk uploaded needs to have a unique version code.


Updating For Ruby 3.1

Signed-off-by: John McCrae<EMAIL_ADDRESS>Updated RUBY_PLATFORM calls pull in the new ucrt from Ruby 3.1

@johnmccrae Any reason not to use rubygems abstraction to determine whether or not it's a windows platform? see: https://github.com/rubygems/rubygems/blob/19e32079040bc42ba1716d4a2f644fdceddb86f9/lib/rubygems.rb#L993-L1000
For long term support it may make sense to use an abstraction so this sort of thing doesn't need to happen again. Whether it's the Rubygem's abstraction or something specific for Chef.


No rule to make target `kaldi/src/kaldi.mk'.  Stop.

I am having difficulty installing Canetis on my system due to a Kaldi error. Canetis is using a fairly old version of Gentle, which is why I think I am encountering this error. The latest version of Gentle installs on its own, but the version Canetis requires does not install correctly for me.
Makefile:5: kaldi/src/kaldi.mk: No such file or directory
make: *** No rule to make target `kaldi/src/kaldi.mk'.  Stop.

It would be good to update Gentle to the latest version if possible, as it might prevent this issue.

@MysteryPancake I know it has been a while, but were you able to resolve this issue? I have been unable to resolve at all.


Move documentation to documentation

Eventually (when the current docs is updated with httplug content) documentation should go right into the official docs.

Is this already done? If so, can we remove docs from the repo?

Not done

It seems to me that a PR is merged.

Yep keep that open, i want to add a link to the documentation in the readme and remove stuff to avoid duplication

What's the status of this?

done in<PHONE_NUMBER>abe38c05fa64247e1ad0b93d2d1ab8


consider additional security for forms on www.ietf.org

Web forms on www.ietf.org currently do not have any mitigation against automated submissions.
Some amount of "spam form" is currently received.
Current published form pages (and submission emails) are:
https://www.ietf.org/forms/wg-webex-account-request/<EMAIL_ADDRESS>https://www.ietf.org/contact/contact-form/<EMAIL_ADDRESS>https://www.ietf.org/forms/keyword-suggestions/<EMAIL_ADDRESS>Additional information about the volume of spam is needed to understand the urgency of the current situation, but some systemic approach to mitigating form spam should be considered.

HycoAnoxia

HycoAnoxia

Ask contractor to add recaptcha to the base form so that it can be optionally used in any form

Any measures should be aligned with https://www.ietf.org/privacy-statement/ and https://www.ietf.org/policies/web-analytics/ particularly with regard to sharing data with third parties and reliance on javascript.

Note that captcha-style techniques can also have accessibility issues, as 'proving you're human' can be onerous to some users.
Would antispam tech (eg https://akismet.com/ ) also be an appropriate solution to this?

Parking this issue for now, until we find a solution that satisfies privacy concerns and accessibility requirements.

How about a honeypot field in the form? The field would be invisible to the user, but an automated spambot would typically feel obliged to fill all available form fields. The server would then check if the field is filled and silently drop the submission.
It's a simple, privacy and accessibility friendly solution. I've found this SO thread with some ideas to tweak the implementation to make it more effective.


Selecting multiplayer in mod menu bug

Works fine selecting to play modded, then selecting mods. Then comes up the choice between Single Player and Multiplayer. Choosing Single Player works fine, but if I select the Multiplayer option to play modded multiplayer, the 'Configuring game data... Please wait' popup comes up and then it just takes me back to the Main Menu. The multiplayer option is there, I just can't access it.

Same!

any news about it? i'm having the same issue

Duplicate of #54, but it's clearly a common issue, so leaving this open.


bootcfg/http: Stop showing the version hash at path /

Stop showing the version hash at HTTP /. If bootcfg were to be deployed at a public endpoint, the version could be used by attackers to find out-of-date deployments in the future. This was added after v0.3.0 in #174 for convenience, but should be removed before v0.4.0.
cc @chancez

Closed by 53a97f2d1788b3181702b817fbddc936ca4c363c

👍


Failure to connect should result into an error with ftl deploy

> ftl deploy --endpoint=http://localhost:4678/
info:alice:build: Building module
info:alice:build: Module built (1.99s)
info:alice:deploy: Deploying module
◞[alice]
>

It seems that the deploy is a success, even if there is nothing listening to port 4678.
We should give a clear error that the CLI failed to connect.

It seems like wrapped errors aren't getting reported correct when using kctx.FatalIfErrorf(err)
In cli/main.go we have this when calling Run on commands
err = kctx.Run(ctx)
kctx.FatalIfErrorf(err)

If I update this to the following, I then see the messages being printed out:
err = kctx.Run(ctx)
if err != nil {
	fmt.Fprintf(os.Stderr, "ftl: %+v\n", err)
	os.Exit(1)
}


Laravel 11.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 11.x.
Before merging, you need to:

Checkout the l11-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: The latest version of Laravel has a dependency for phpunit/phpunit of ^10.5. If you would like to also upgrade your tests, you may run the PHPUnit 10 Shift for free.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/laravel-digest.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "hmones/laravel-digest": "dev-l11-compatibility",
    }
}

Finally, run: composer update


Signing plugin does not sign Gradle module metadata artifact for Ivy publication

Expected Behavior
The signing plugin should sign all artifacts in a publication.
Current Behavior
The signing plugin does not sign the module metadata artifact for Ivy publications. It does sign the module metadata artifact for Maven publications.
Context

Steps to Reproduce
Run ./gradlew publish in this sample repository: https://github.com/gradle/configuration-cache-dependency-management
The Ivy module in ivy-lib/build does not contain a signature for the metadata artifact. The Maven module in maven-lib/build does contain a signature.
Your Environment

Build scan URL:

It seems the root cause for this issue is that artifacts derived from module metadata are removed in this commit.
@adammurdoch I can create a PR for this if you double confirm the root cause.


JAR Files

The currently published plugin does not include the task for generate Javadoc JARs (I get "Task 'generateDebugJavadocJar' not found in project ...").
Would you publish an updated JAR? Love this plugin.

I'm closing this issue due to inactivity. If you have any further input on the issue, don't hesitate to reopen this issue or post a new one.


How does one import the phabricator unit test report into phabricator?

docs say: PhabricatorReporter produces a JSON array which you can plug directly into the
'arc:unit' diff property in Phabricator.
Anybody with a quick primer on how that works?

My memory is a bit hazy but I think this is correct...
You can write the results via Conduit directly into the diff in Phabricator (for example, from CI machines that maybe don't use arc):
https://secure.phabricator.com/conduit/method/differential.setdiffproperty/
Or, you can create your own UnitTestEngine that plugs into arc unit locally and that will handle sending the results up to the diff in Phabricator:
https://secure.phabricator.com/book/phabricator/article/arcanist_lint_unit/
https://secure.phabricator.com/diffusion/ARC/browse/master/src/unit/engine/

Looks like Christian answered!


Add try catch blocks to pinecone index deletion

Add try catch blocks to pinecone index deletion

torvalds.dev is analyzing the ticket


[Feature]: add quantity of tasks in a group

What kind of feature do you want?
add quantity of completed and uncompleted tasks in a group
Additional questions

[X] I want to work on this feature request

I only added quantity of tasks in a group, I think this is enough for the first time


Migrate flake8_pyi_rules from unspecified to suggested and automatic

Summary
Migrates autofix rules from unspecified to suggested and automatic

Test Plan
Ran all tests

@charliermarsh Would love some 👀 on this . Thanks in advance


Remove the free disk space workaround that prevented the jobs from failing for using too much space for the images.

I believe this isn't necessary anymore
  - name: Free Disk Space (Ubuntu)
        # For some reason the original job (libp2p/test-plans) has enough disk space, but this one doesn't.
        uses<EMAIL_ADDRESS>        with:
          tool-cache: true

@lchenut please approve it again as my approval isn't considered.


upgrade cats effect to rc, replace unqiue with ce3 one

Argh.  I didn't get notifications on this, and just did this myself!


Move serverspec testing to be via ansible

See these repos -
https://github.com/volanja/ansible-sample-tdd
https://github.com/volanja/ansible_spec

moving this to critical as we are not running serverspecs at all after provision since we moved to ansible

Server spec tests?  I love you.

we love serverspec too!


StatusBarRegistration's component field must be optional for backwards compatability

fix bug related to #1598
Signed-off-by: Sebastian Malton<EMAIL_ADDRESS>

Thanks! This broke here too from Dependabot: https://github.com/Mirantis/lens-extension-cc/pull/148


fix(rust): fix get_contact entity worker response type

@mrinalwadhwa @jared-s why this doesn't trigger any CI builds?

@SanjoDeundiak Weird. they ran for your other branch. There is some oddness with Actions this morning. I need to rebase these PRs anyway, so I will ensure CI runs

Merged https://github.com/ockam-network/ockam/pull/1534


add help known_chunks or similar, to list all parsable chunks

I'd love to automate this. It needs to at least show the chunk id and a one sentence summary of that chunk.
Eventually it probably makes sense to lookup all types which implement a specific trait using rustdoc's JSON output. Unfortunately, that's currently nightly only, and I'd prefer to avoid nightly dependencies. When the time comes, rustdoc-json looks like it would be useful in a build.rs step.
In the short term, maybe building rustdoc html and scraping the supported types and docs from there?
Consider deferring this a while?

depends on #37


Feature/factional octave filters

Changes proposed in this pull request:

Add fractional octave filters

Although not needed in the DIN room acoustics context, it might be nice to have a more versatile function for computing fractional octave frequencies with the following parameters:

fraction (e.g., 3=third octaves)
frequency_range (e.g., (20, 16e3))
and returns:
center frequencies (nominal and exact)
lower_cut_off_frequencies
upper_cut_off_frequencies

nominal center frequencies might be None if the input parameters exceed the DIN.
What do you think?

Yes, good points, most of which I'm planning to implement.
This is only a draft by the way, not a final pull request ;)


feat: #82 allow search by user guid

refs: #82

Allow search business bceid user by their userId or userGuid
chore(deps): replace dependency eslint-plugin-node with eslint-plugin-n refs: #78

Please note:  We cannot make the change directly to the existing api /bceid, the parameter change will impact FAM production. So instead, we created a new api route /businessBceid, once FAM finish the changes on FAM side, we'll work on the cleanup to remove the old /bceid api, #84

Search User by userId

Found

Not Found

Search User by userGuid

Found

Not Found

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

The code duplicate is caused by that we can't make the change directly to the existing api path/bceid, the parameter change will impact FAM production. So instead, created a new api route /businessBceid, and once FAM finish the changes on FAM side, we'll work on the cleanup to remove the old /bceid api https://github.com/bcgov/nr-fam-idim-lookup-proxy/issues/84


Pipeline as Code integration for GitLab

Changes

:gift: Pipeline as Code integration for GitLab.
:test_tube: Host names falling under the <IP_ADDRESS>.sslip.io domains are now resolved to cluster node IP inside of the cluster.

/kind enhancement
feat: pipeline as code integration for gitlab

@zroubalik PTAL

/override "Code Style / style / suggester / shell (pull_request)"

/override "style / suggester / shell"

/unhold


Move cache save to runs.post and exit early

Fixes #59
I set up a demo repo showing the difference when using an early exit:
This build ran with https://github.com/oven-sh/setup-bun/commit/f32d88462b6351aea919415b45d01f435ce30fba and took 22s: https://github.com/andyexeter/setup-bun-cache-test/actions/runs/7918709270
This build ran with https://github.com/oven-sh/setup-bun/commit/f5c5efee5778b44fb74ffe8a18e7c071d6057f55 and took 2m 28s: https://github.com/andyexeter/setup-bun-cache-test/actions/runs/7918734742

Thanks for working on this!


Performance and certificate limits

Hello!
We are planning to run this solution for an nginx server that will be used to serve multiple sites. Therefore I wanted to ask the following question:

Is there any limit in regards to the number of certificates this solution can handle? To give you an idea, we plan to have 100+ domains on that server.
In regards to the performance, can we expect any downgrade on performance when using this solution? Is the private key fetched from ACM for every request or is there some caching mechanism implemented that will store the key locally to improve performance?

Thanks in advance for your help! :)

@aleksy-zalenski Hi.

The enclave currently supports one server key and its optional associated certificate chain per token. There are currently maximum 8 tokens and 1024 sessions per token supported. This could be increased in the future if demand arrives. Meanwhile, please see the Using multiple certificates chapter in the docs on how to configure this. Could you provide more info on how you wish to integrate this? i.e. Nginx config file with one stanza file per domain for each private key?

This solution handles the provisioning and renewal of your ACM certificates (and the private key) on your EC2 instance.  After you start the service the private key is provisioned in the enclave and NGINX uses the pkcs#11 engine over AF_VSOCK for local requests between your EC2 instance and the enclave. For example, when a signing operation is requested, the data is sent to the enclave, it gets signed by the local stored private key and the output is provided back. The private key is isolated and never leaves the enclave once provisioned.

You can setup your solution today and do an NGINX benchmark to see your performance areas of interest for up to 8 domains.

Hello @alcioa,
First of all, thank you very much for your quick response on this one. I really appreciate it.
In regards to your answers:

Oh I see, so basically the limit is only 8 certificates per instance, right? In regards to how I plan to integrate it -- basically like you mentioned I was planning to have a separate nginx stanza file for each domain (certificate and private key). Unfortunately the limit of 8 certs is way too low for us and kind of makes this solution unusable. May I ask you what's the reason of only up to 8 certs being supported and if it is possible to somehow increase it?
Got ya, so basically everything is done on the instance level. That's cool. So the singing is basically done in the enclave while NGINX is running on the instance itself and request is being handled there, right?

In case you need any additional input from my end, don't hesitate to let me know.

@aleksy-zalenski currently investigating supporting either 128 tokens or more key-cert pairs per token. Meanwhile you can run some ab to check some benchmarks if curious (keep in mind that x86_64 enclave needs at least 2 vCPUs from your EC2 instance). Are you interested exclusively in NGINX? Apache HTTPD?

Hello @alcioa,
Thanks for the information :)
Do you have any rough estimate on when do you think your investigation regarding supporting more certificates will finish and can be implemented? I assume that supporting 128 tokens means we will be able to support 128 AWS ACM certificates, right?
In regards to the ab testing I've already run some tests last week and the results look promising. Below are the details:
[ec2-user@ip-10-100-1-119 ~]$ ab -n 10000 -c 20 https://test.oneeach.dev/
This is ApacheBench, Version 2.3 <$Revision: 1879490 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/

Benchmarking test.oneeach.dev (be patient)
Completed 1000 requests
Completed 2000 requests
Completed 3000 requests
Completed 4000 requests
Completed 5000 requests
Completed 6000 requests
Completed 7000 requests
Completed 8000 requests
Completed 9000 requests
Completed 10000 requests
Finished 10000 requests

Server Software:        nginx/1.20.0
Server Hostname:        test.oneeach.dev
Server Port:            443
SSL/TLS Protocol:       TLSv1.2,ECDHE-RSA-AES256-GCM-SHA384,2048,256
Server Temp Key:        ECDH P-256 256 bits
TLS Server Name:        test.oneeach.dev

Document Path:          /
Document Length:        3520 bytes

Concurrency Level:      20
Time taken for tests:   43.153 seconds
Complete requests:      10000
Failed requests:        0
Total transferred:      37540000 bytes
HTML transferred:       35200000 bytes
Requests per second:    231.73 [#/sec] (mean)
Time per request:       86.306 [ms] (mean)
Time per request:       4.315 [ms] (mean, across all concurrent requests)
Transfer rate:          849.54 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:        9   69  11.6     67    1071
Processing:     1   17   2.9     17      45
Waiting:        1   17   2.9     17      44
Total:         10   86  12.0     84    1088

Percentage of the requests served within a certain time (ms)
  50%     84
  66%     86
  75%     87
  80%     88
  90%     96
  95%    102
  98%    105
  99%    108
 100%   1088 (longest request)

It was run with only a single certificate installed, but I don't think more certificates will change anything in terms of performance since the requests are anyway handled independently.
In regards to the web server, to be honest for our use case it doesn't matter that much and we can use either Nginx or Apache. I've used Nginx since it was officially supported, but we can use whatever supports more AWS ACM certificates ;)

@aleksy-zalenski anymore info needed here?

Hello @alcioa,
Thank you very much for implementing the fix, I really appreciate it.
Just for the future, in case we will need more than 128 ACM certs, do you think it will be possible to further increase the limit, say to 256?

Hi, yes it can be done as there is no pkcs#11 spec restriction for that, but some more extensive changes might be needed in order to scale better as that number grows from I/O and memory perspective.

@aleksy-zalenski the latest v1.1.2 RPM has been released with support of up to 128 ACM certificates. Will close this.

Hello @alcioa,
We are really close to hitting the 128 ACM certificates limit. As discussed previously, can we have this limit increased to 256 certificates per instance? If needed, I can create a separate Github issue.
Thanks in advance!


Touchscreen input option does not work

The option to use a touchscreen as the input device in Android x86 4.3 and possibly other OS's does not work. A workaround is to use the trackpad input option.

I think touchscreen input will be a future option. But it requires client side support. I don't know if android x86 has it.

There should be.

If it does not work, the touchscreen support should be labeled as experimental or non-working within the app.

Sir the app is labeled as experimental.

Emm. But I’m not a Sir.
| |
彩虹小糖
|
|
<EMAIL_ADDRESS>|
签名由 网易邮箱大师 定制
On 03/10/2020 10:52, osy86 wrote:
Sir the app is labeled as experimental.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Replying to OP.

The touchscreen input option does not work as expected in windows 10 with the SPICE drivers installed. I touch on screen and the input that the OS receives is completely inaccurate. Can provide a video if needed, but i feel this is an issue that should be fixed


Unable to use Control Add-In "PingPong"

Describe the bug
I am unable to use the control add-in "PingPong" on an on-premise installation of Business Central 14.0, due to the compiler not finding the events that are in the library. I have followed the documentation found here: https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-dotnet-controladdins
(The documentation leaves out some crucial steps, and the code samples aren't correct, either, but that's a different issue altogether.)
To Reproduce
Steps and to reproduce the behavior:

Create a new AL Project.
Edit the app.json to ensure that target is set to Internal.
Edit the HelloWorld.al file to look like this:

dotnet
{
    assembly("Microsoft.Dynamics.Nav.Client.PingPong")
    {
        type("Microsoft.Dynamics.Nav.Client.PingPong.PingPongAddIn"; PingPongAddIn)
        {
            IsControlAddIn = true;
        }
    }
}

page 50100 MyPage
{
    PageType = Card;
    ApplicationArea = All;
    UsageCategory = Administration;

    layout
    {
        area(Content)
        {
            usercontrol(PingPongControl; PingPongAddIn)
            {
                trigger Pong()
                begin
                    Message('Pong received.');
                end;

                trigger AddInReady()
                begin
                    Message('Ready');
                end;
            }
        }
    }
}

Create a folder called .netpackages
Copy the Microsoft.Dynamics.Nav.Client.PingPong.dll file from C:\Program Files\Microsoft Dynamics 365 Business Central\140\Service\Add-ins\PingPong\ into .netpackages
Restart VS Code and reopen the project so that it can read the library.
==> The compiler will find the control add-in and won't complain in the dotnet object, but will fail to find the AddInReady and Pong events:

Compilation started for project 'TestControlAddinApp' containing '1' files at '12:19:34.790'.

c:\Users\mt\Documents\AL\TestControlAddinApp\HelloWorld.al(24,25): error AL0280: The event 'Pong' is not found in the target
c:\Users\mt\Documents\AL\TestControlAddinApp\HelloWorld.al(29,25): error AL0280: The event 'AddInReady' is not found in the target

Decompiling the PingPong DLL shows that the methods are shown as [ApplicationVisible] as expected:

Expected behavior
The project should compile and build with no issue. Intelli-sense also does not show the control add-in when attempting to autocomplete the type of usercontrol.
Versions:

AL Language: 3.0.152816
Business Central: 14.0.32600.0

We'll investigate. We are using this add-in internally, so it is quite strange you are hitting the problem.

I should note, this isn't just an issue for myself--several of my colleagues have tried the same steps on their own work machines, with the same problem.
Let me know if I can provide any other information.

I have same problem with my own .NET control add-in. I cannot define any of triggers in Visual Studio Code.
AL: 3.0.168874 (older version published with my version of BC was same)
BC: 14.0.34251.0

@AndreyKorepanov
Can you please explain why in wave 2 the PingPong add-in is still included to refresh the 'My Job Queue' page and the 'Report Inbox' page and how it will eventually be refactored? Is using pagebackgroundtasks the ideal solution to handle a 'RefreshPage every 30 seconds'? In this case, it seems useless to start a child session that will do nothing, just to trigger an autorefresh on page level?

I see in documentation, that .NET variable can have mark [WithEvents], which enables it. I didn't try it, but may be some similar will be work with control add-in.
https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-dotnet-subscribe-to-events


Nechronica: Uploading images to be stored in github

Submission Checklist
Required

[X] The pull request title clearly contains the name of the sheet I am editing.
[X] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[X] The pull request makes changes to files in only one sub-folder.
[X] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description
Uploading images that were hosted on imgur for this character sheet so they can be hosted in a more stable location. There are 19 new images, which will be linked in the character sheet after updating (soon).

Can someone please check on this and let me know why it's broken?

Hey there @mortaine this is just because you are submitting a PR without any changes to sheet files (css, html, json) and validation is considering it invalid. I will go ahead and force the merge.


I can not install package

Npm log:
npm install --save-dev identity-obj-proxy
npm ERR!  Please include the following file with any support request:
npm ERR!     /Users/SVITY/www/yellow.team/pavemint/pavemint-web/npm-debug.log

Hi! This package doesn't have React as a dependency or peerDependency (https://github.com/keyanzhang/identity-obj-proxy/blob/master/package.json#L33-L46). Please take a look at your error log; react-test-renderer requires<EMAIL_ADDRESS>but you only have<EMAIL_ADDRESS>


Date Format

When I display start and end date at the frontpage it looks like this: 2024-06-22 - 2024-06-23
Is there a way you can format that in a non  American style?
Something like: 22.06.2024 - 23.06.2024

The api should always provide the dates / datetimes in RFC 3339 or ISO 8601 format.
The frontend should display it in an appropiate format to the users.

I agree to what @peddermaster2 said. The backend should not be responsible for formatting. If you also agree, please close this issue as "not planned".


can't start open new project in micro:bit beta

Repro steps:

Visit https://makecode.microbit.org/beta
Create a "New project"
give it a name
see that the project doesn't open and inspect console to see error:

pxtapp.js:1

       Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'hideSimButtons')
    at e.SimulatorDriver.createFrame (pxtapp.js:1:566060)
    at e.SimulatorDriver.preload (pxtapp.js:1:566608)
    at ae.internalLoadHeaderAsync (main.js:1:79380)
    at ae.loadHeaderAsync (main.js:1:79166)
    at async ae.createProjectAsync (main.js:1:99790)
createFrame @ pxtapp.js:1
preload @ pxtapp.js:1
internalLoadHeaderAsync @ main.js:1
loadHeaderAsync @ main.js:1

Fixed by https://github.com/microsoft/pxt/commit/deb202367ccb549688366b275f771348ceba6f3f
Will be in /beta microbit in another 30 minutes or so.


Check page's accessibility , Score 100% on lighthouse

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. E.g. "I'm always frustrated when ...".
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Remember to use production not dev build for this


GA - details - no metadata or error info is shown on details view after adding proposal

To reproduce:

Go to dashboard
Add Info Proposal (use inputs like on screenshots below)
Confirm (use json link)
Wait until transaction is completed
Go to GA list
Find newly added Proposal - enter to details page compare with screenshots:

Seems like it might be the db-sync issue not getting the metadata info. @jankun4 Could you have a look?

Answering my previous comment - this data is indeed missing on the db-sync side. However, the validation against the metadata under the url and the hash provided succeeds - that is why we are displaying the metadata.
I wonder whether we should make the 2 step validation:
One - against the metadata and the URL provided to the proposal.
Second one - against the metadata provided directly from the proposal.

@MSzalowski Does this still count as P0 given your recent comment?

We still can vote on the governance action. However, we don't know what the governance action is about. It is something between P0 and P1

P0 to me.

CC @Ryun1
This should be a dbsync bug, so we will retest this after the respin is complete.

@bosko-m
Probably best to raise this as an issue via the DB-Sync repository, or ask via the working group on discord

Raised issue: https://github.com/IntersectMBO/cardano-db-sync/issues/1716

@MSzalowski While testing this on dev (because staging is not yet up and running fully), we have realized that once you create an Info GA, you cant find it on the list of all GAs. The thing that make it worse is that there arent titles showing on the ones recently created. This needs to be solved before the release as its a show stopper. Lets discuss on Monday.

@MSzalowski @m-i-k-e Went through it with @Ryun1 , all seems to be working as they appeared after some period of time which is great, but still missing metadata.

Known issue going to the Beta releaser on 6th June, however, will be fixed in the next few days.
CC @m-i-k-e @Ryun1 @l-br1

OK when validating, showing metadata and info about missing, corrupted
https://jam.dev/c/1d4e44fd-6104-41ce-a18b-bd79c0293918
https://raw.githubusercontent.com/pmbinapps/vva/main/tmp/Info.jsonld

JSONLD file contents:
{
"@context": {
"@language": "en-us",
"CIP100": "https://github.com/cardano-foundation/CIPs/blob/master/CIP-0100/README.md#",
"CIP108": "https://github.com/cardano-foundation/CIPs/blob/master/CIP-0108/README.md#",
"hashAlgorithm": "CIP100:hashAlgorithm",
"body": {
"@id": "CIP108:body",
"@context": {
"references": {
"@id": "CIP108:references",
"@container": "@set",
"@context": {
"GovernanceMetadata": "CIP100:GovernanceMetadataReference",
"Other": "CIP100:OtherReference",
"label": "CIP100:reference-label",
"uri": "CIP100:reference-uri",
"referenceHash": {
"@id": "CIP108:referenceHash",
"@context": {
"hashDigest": "CIP108:hashDigest",
"hashAlgorithm": "CIP100:hashAlgorithm"
}
}
}
},
"title": "CIP108:title",
"abstract": "CIP108:abstract",
"motivation": "CIP108:motivation",
"rationale": "CIP108:rationale"
}
},
"authors": {
"@id": "CIP100:authors",
"@container": "@set",
"@context": {
"name": "http://xmlns.com/foaf/0.1/name",
"witness": {
"@id": "CIP100:witness",
"@context": {
"witnessAlgorithm": "CIP100:witnessAlgorithm",
"publicKey": "CIP100:publicKey",
"signature": "CIP100:signature"
}
}
}
}
},
"authors": [],
"hashAlgorithm": {
"@value": "blake2b-256"
},
"body": {
"abstract": {
"@value": "test"
},
"motivation": {
"@value": "test"
},
"rationale": {
"@value": "test"
},
"references": [
{
"@type": "Other",
"label": {
"@value": "Label"
},
"uri": {
"@value": "https://test.com"
}
}
],
"title": {
"@value": "test"
}
}
}

Working on staging with Info action and your metadata @m-i-k-e

Working on staging with demos, and a treasury action with @m-i-k-e's metadata

and the GAs can be seen via GA explorer
https://staging.govtool.byron.network/connected/governance_actions/28cbfca8afc31df1996ffab4bec23c960121b2a57d30bb5a88ee8265eb5f3a3f#0

Thanks @Ryun1 looks good.

This works for me now. Closing


How to use this code to attack VGG16?

Hello, could you please tell me about how to use? Thank you!

It should be straightforward to use VGG16. I included a vgg16.py file, which you can train. Alternatively, you can use Keras's pretrained model: https://keras.io/applications/#vgg16
Then it should be as simple as including it in the list of models. Let me know if you have any difficulty.

Are you still working on this?

Are you still working on this?

Oh, yes. I'm so sorry. Running statistics result is a bit slow.
And how do I use the GPU to accelerate this process?

You should have CUDA, CuDNN, and tensorflow-gpu installed on your system. Then the library should automatically detect and use your GPU if it is available. To detect if the library is using GPU, see #9.
Alternatively, you can use Google Colab to run the notebook on a free GPU cloud instance.

Yes, that is very slow, shouldn't take longer than an hour. If your system is using a GPU, then that probably means you have a bottleneck somewhere. Have you tried running other code with Keras, like one of their tutorials with GPU?

Yes, that is very slow, shouldn't take longer than an hour. If your system is using a GPU, then that probably means you have a bottleneck somewhere. Have you tried running other code with Keras, like one of their tutorials with GPU?

Ok, I will continue to study the work. And I want to get a statistical success rate of 500 samples on ImageNet.


[tool_crash] FileSystemException: Cannot open file, OS Error: Permission denied, errno = 13

Command
flutter run

Steps to Reproduce

...
...
...

Logs
FileSystemException: Cannot open file, OS Error: Permission denied, errno = 13
#0      _File.throwIfError (dart:io/file_impl.dart:635:7)
#1      _File.openSync (dart:io/file_impl.dart:479:5)
#2      _File.readAsBytesSync (dart:io/file_impl.dart:539:18)
#3      _File.readAsStringSync (dart:io/file_impl.dart:584:18)
#4      ForwardingFile.readAsStringSync (package:file/src/forwarding/forwarding_file.dart:103:16)
#5      ForwardingFile.readAsStringSync (package:file/src/forwarding/forwarding_file.dart:103:16)
#6      _pluginFromPackage (package:flutter_tools/src/plugins.dart:301:65)
#7      findPlugins (package:flutter_tools/src/plugins.dart:332:27)
<asynchronous suspension>
#8      refreshPluginsList (package:flutter_tools/src/plugins.dart:1117:38)
#9      FlutterProject.ensureReadyForPlatformSpecificTooling (package:flutter_tools/src/project.dart:235:11)
#10     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:843:21)
<asynchronous suspension>
#11     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:741:33)
<asynchronous suspension>
#12     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart)
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:29)
#14     _rootRun (dart:async/zone.dart:1190:13)
#15     _CustomZone.run (dart:async/zone.dart:1093:19)
#16     _runZoned (dart:async/zone.dart:1630:10)
#17     runZoned (dart:async/zone.dart:1550:10)
#18     AppContext.run (package:flutter_tools/src/base/context.dart:149:18)
#19     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:730:20)
#20     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
#21     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:332:21)

[✓] Flutter (Channel master, 1.20.0-3.0.pre.126, on Linux, locale pt_BR.UTF-8)
    • Flutter version 1.20.0-3.0.pre.126 at /home/rode/flutter
    • Framework revision 462b0ea76e (9 hours ago), 2020-07-02 16:13:02 -0400
    • Engine revision 51ca1432b3
    • Dart version 2.9.0 (build 2.9.0-20.0.dev 8afe9875a6)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /home/rode/Android/Sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /opt/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[!] Android Studio (version 3.6)
    • Android Studio at /usr/local/android-studio
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] Android Studio (version 4.0)
    • Android Studio at /opt/android-studio
    • Flutter plugin version 47.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.46.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.12.1

[✓] Connected device (1 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86 • Android 6.0 (API 23)
      (emulator)

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 8af6b2f038c1172e61d418869363a28dffec3cb4

Hi @ricardo006
I see there's an open issue addressing the case you described.
Please follow up on that issue, I'm closing the current one as duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


chore(state): extend error handling in core accessor

Overview
Extend error handling in core accessor
Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Codecov Report

Merging #2307 (44ce150) into main (a01d0b5) will decrease coverage by 0.07%.
The diff coverage is 54.05%.

:exclamation: Current head 44ce150 differs from pull request most recent head af7146f. Consider uploading reports for the commit af7146f to get more accurate results

@@            Coverage Diff             @@
##             main    #2307      +/-   ##
==========================================
- Coverage   50.89%   50.83%   -0.07%
==========================================
  Files         150      154       +4
  Lines        9252     9671     +419
==========================================
+ Hits         4709     4916     +207
- Misses       4147     4329     +182
- Partials      396      426      +30

Impacted Files
Coverage Δ

api/gateway/availability.go
0.00% <ø> (ø)

api/gateway/header.go
0.00% <ø> (ø)

api/gateway/state.go
0.00% <0.00%> (ø)

core/testing.go
91.86% <ø> (-0.10%)
:arrow_down:

core/testing_grpc.go
69.84% <ø> (+4.76%)
:arrow_up:

das/daser.go
64.83% <0.00%> (ø)

das/metrics.go
9.91% <0.00%> (ø)

header/headertest/testing.go
71.92% <0.00%> (-6.98%)
:arrow_down:

nodebuilder/module.go
85.71% <ø> (ø)

nodebuilder/p2p/bootstrap.go
6.66% <ø> (ø)

... and 37 more


MSFT_xUserResource.psm1: Fix script analyzer issues

Pull Request (PR) description
Fixes remaining script analyzer issues in MSFT_xUserResource.psm1.
This Pull Request (PR) fixes the following issues

Fixes #492

Task list

[x] Added an entry under the Unreleased section of the change log in
CHANGELOG.md. Entry should say what was changed, and how that affects
users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as
appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See
DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See
DSC Resource Testing Guidelines.
[x] New/changed code adheres to
DSC Resource Style Guidelines and Best Practices

This change is

Codecov Report

Merging #553 into dev will not change coverage.
The diff coverage is 50%.

@@         Coverage Diff         @@
##            dev   #553   +/-   ##
===================================
  Coverage    74%    74%
===================================
  Files        27     27
  Lines      4027   4027
  Branches      4      4
===================================
  Hits       3007   3007
  Misses     1016   1016
  Partials      4      4

Will review once the conflicts are fixed.

Just waiting for the CI to finish before merging.


Add res.json(...) support - /w work around

Almost all of my controllers return res.json(...).
run-middleware does not appear to correctly support this.
In express/lib/response.js it tries to see if Content-Type is set as a header, eventually through the getHeader(...) call.
getHeader is implemented a little poorly, expecting "this[kOutHeaders]" to equal null, but instead it is undefined.
That means all res.json(...) calls will fail.  I haven't looked at your code, but is your response not extending OutgoingMessage ?
As a workaround, after I did runMiddleware(app), I added this little gem:
app.response.getHeader = (name) => {
    return null;
};`

If you wish the create a fix, I will review and merge it.

Facing same problem due to undefined content-type. I tried to add a content-type to the headers in createRes but that didn't work either. Also I'm having a problem where the IP cannot be passed to the other middleware that I'm calling even though I have passed in {ip:...} in the options of runMiddleware.
TypeError: Cannot read properties of undefined (reading 'startsWith')
    at getIPFromReq (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/directus/dist/utils/get-ip-from-req.js:22:15)
    at handler (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/directus/dist/middleware/authenticate.js:25:48)
    at /home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/directus/dist/utils/async-handler.js:3:66
    at Layer.handle [as handle_request] (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/express/lib/router/index.js:328:13)
    at /home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/express/lib/router/index.js:286:9
    at Function.process_params (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/express/lib/router/index.js:346:12)
    at next (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/express/lib/router/index.js:280:10)
    at extractToken (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/directus/dist/middleware/extract-token.js:30:5)
    at Layer.handle [as handle_request] (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/express/lib/router/layer.js:95:5)
node:_http_outgoing:654
  const entry = headers[StringPrototypeToLowerCase(name)];
                       ^

TypeError: Cannot read properties of undefined (reading 'content-type')
    at ServerResponse.getHeader (node:_http_outgoing:654:24)
    at ServerResponse.res.get (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/express/lib/response.js:812:15)
    at ServerResponse.json (/home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/express/lib/response.js:274:13)
    at /home/walfin/Dokumentujo/Gitlab/quasar_upgrade/caswebsite/node_modules/directus/dist/middleware/error-handler.js:93:20
    at processTicksAndRejections (node:internal/process/task_queues:96:5)


DGFIP scope : required value field ?

It seems there are few mistakes in the DGFIP schema.json.
On dgfip_identity_field and dgfip_tax_notice_field types, a field "value" is indicated as "required" whereas there is no value property for these types.

Moreover, the attributes table no mentions the array. The table must be indicate :

Name
Type

declarant1
DGFIP identity

declarant2
DGFIP identity

taxNotices
DGFIP tax notice[ ]

Thanks. I have create PR #11 to fix these issues. Since it does not change the data model, I think it could be merged. @ghislainfabmob, what do you think of merging this PR ?


Cannot debug my app wireless

Hi,
I'm having serious issues when I try to debug my app wireless with Flutter on VS Code. Every time I want to do that, I've to connect my phone with USB cable, but the very moment I disconnect the connection is lost.
Steps to Reproduce

Open Terminal
Run adb tcpip 5555.
Then, run adb connect [IPv4/IPv6 Adresss]
Disconnect the cable and see if the device still appears on VS Code

Expected results:
The device should be ready for debug in VS Code
Actual results:
When I try to reconnect to my IPv4 address I get:
adb server version (41) doesn't match this client (39); killing...
ADB server didn't ACK
Full server startup log: /tmp/adb.1000.log
Server had pid: 6881

--- adb starting (pid 6881) ---
adb I 01-23 11:36:49  6881  6881 main.cpp:57] Android Debug Bridge version 1.0.39
adb I 01-23 11:36:49  6881  6881 main.cpp:57] Version 1:8.1.0+r23-8
adb I 01-23 11:36:49  6881  6881 main.cpp:57] Installed as /usr/lib/android-sdk/platform-tools/adb
adb I 01-23 11:36:49  6881  6881 main.cpp:57]
adb I 01-23 11:36:49  6881  6881 adb_auth_host.cpp:416] adb_auth_init...
adb I 01-23 11:36:49  6881  6881 adb_auth_host.cpp:174] read_key_file '/home/[user_name]/.android/adbkey'...
adb I 01-23 11:36:49  6881  6881 adb_auth_host.cpp:391] adb_auth_inotify_init...
adb server killed by remote request

* failed to start daemon
error: cannot connect to daemon

Logs

[✓] Flutter (Channel stable, 1.22.5, on Linux, locale en_US.UTF-8)
    • Flutter version 1.22.5 at /home/abnilson/.flutter
    • Framework revision<PHONE_NUMBER> (6 weeks ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[!] Android toolchain - develop for Android devices (Android SDK version
    30.0.2-2)
    • Android SDK at /home/abnilson/Android/Sdk
    • Platform android-30, build-tools 30.0.2-2
    • Java binary at: /usr/bin/java
    • Java version OpenJDK Runtime Environment (build 15.0.1+9)
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/linux#android-setup for
      more details.

[!] Android Studio (not installed)
    • Android Studio not found; download from
      https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup
      for detailed instructions).

[✓] VS Code (version 1.52.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.18.1

[!] Connected device
    ! No devices available

! Doctor found issues in 3 categories.

I don't own USB cable, so I rely in Wireless connection.

I believe you have to connect it the first time with USB then start using wireless. Here you can find the steps to do it: https://yazeedalkhalaf.medium.com/debug-your-flutter-app-on-a-real-android-device-over-wifi-bd70a7c25abe

I believe you have to connect it the first time with USB then start using wireless. Here you can find the steps to do it: https://yazeedalkhalaf.medium.com/debug-your-flutter-app-on-a-real-android-device-over-wifi-bd70a7c25abe

I did that first time. The moment I remove USB cable the device is somehow disconnected. And I can't reconnect, unless I plug USB once more.

I did that first time. The moment I remove USB cable the device is somehow disconnected. And I can't reconnect, unless I plug USB once more.

And.... I switched from AS to Flutter + VS Code because of high resources usage... For system with less than 16GB RAM (HDD) and more updates adding many functionalities, AS is turning out to be unstable and unusable.

And.... I switched from AS to Flutter + VS Code because of high resources usage... For system with less than 16GB RAM (HDD) and more updates adding many functionalities, AS is turning out to be unstable and unusable.

i had similar issue
i believe the essense of it is in this line
adb server version (41) doesn't match this client (39); killing...

that means you most definitely have multiple sdk versions installed but the one speicified in PATH mismatches the android studio's
anyways this is not an issue with flutter

i had similar issue
i believe the essense of it is in this line
adb server version (41) doesn't match this client (39); killing...

that means you most definitely have multiple sdk versions installed but the one speicified in PATH mismatches the android studio's
anyways this is not an issue with flutter

By the logs this does not seems an issue with Flutter.

Full server startup log: /tmp/adb.1000.log

Can you please provide this log file?

By the logs this does not seems an issue with Flutter.

Full server startup log: /tmp/adb.1000.log

Can you please provide this log file?

adb.1000.log

adb.1000.log

Hi @UltraBurstXD
This issue does not seems to be related to Flutter so I'm going to close it.
I would suggest you to raise a question on Stack Overflow to get help.
Thank you

Hi @UltraBurstXD
This issue does not seems to be related to Flutter so I'm going to close it.
I would suggest you to raise a question on Stack Overflow to get help.
Thank you


Don't Panic!

Currently code panics on errors in signing and generating keys. Whilst these errors might be unlikely they are not impossible and its not nice behaviour to take down a key-signing daemon just because there is a temporary issue with the entropy source, for example. Let's just return errors and let callers panic if they want to.
Happy to make PR for this.

Yeah, I have the same convictions and not a fan of seeing panics all around, I too would prefer errors. Thank you @silasdavis!

Should we change the Sign method to Sign(msg []byte) (Signature, error) then ?
For certain things, panic seems fine - like errors on marshalling stuff that should not error.
And it looks like we've mostly removed other panics on generating or signing, except for one in Secp - but that will require changing the method signature

I support returning an error. This is in line with the return type of golang's signer interface. (https://github.com/golang/go/blob/master/src/crypto/crypto.go#L115)

there are a few places left were go-crypto code panics but signing is addressed and resolved by #117


Ledger Nano links broken

The links https://support.ledger.com/hc/en-us/articles/360006395233 and https://ledger.com/live do not work on at least page https://nimiq.github.io/tutorials/ledger-guide

Hi there!
Your query will be dealt with by the support team.
Please see the link below to our dedicated support line:
HELP DESK
Ticket id: DR5J
Note: Click on the live chat icon at the bottom corner of the page to start a conversation

Hello, thank you for this report. The update of this guide is on our todo list. We'll keep the dead links in mind, when working on it.


@conda-forge-admin,please add bot automerge

[x] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/opacus-feedstock#3.


Download of cert fails if nodepki is running behind a reverse proxy

Hi there,
If the nodepki service is running behind a reverse proxy (e.g. Traefik) which proxies e.g. ca.example.com to nodepki:8080, the download of the root / intermediate cert fails because of a wrong URL concatenation. The URL then points to http://ca.example.com:8080/public/ca/root/cert instead of http://ca.example.com/public/ca/root/cert where it should point to (note the missing :8080).
https://github.com/aditosoftware/nodepki-webclient/blob/a5f9aca05db9a204ed27ea11e9b0df723fe07b65/controllers/cacerts.js#L41
Any ideas how to fix this so both setups are able to provide a certificate download (with RP or without RP)?
Thanks.
Regards,
Philip

I think the download should use the host name provided by the http headers in the field 'Host'. The port should be included there in case it's not the default port.


UI/translate screen

Description
This is the first iteration on the implementation of the TranslateScreen
Type of Change
This is is a just the implementation of the ui-design
Checklist
[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules
Screenshots (if appropriate)

The design has changed from this but not totally.


Fix the "extra" overrides.

When we switched to the mega module, we lost our sensitivity to these things: https://github.com/chainguard-images/images/blob/b9efef911e8e00ec6e20749fa3c77f148a99e8f2/.github/actions/release-image-terraform/action.yml#L69-L79
This brings it back!

Hmm, I'm not sure why these list arguments are coming through as strings 🤔


Abilities should different icons & descriptions

Not all abilities are the same under the same umbrella. Check if descIDs, iconIDs and titleIDs are valid. They are usually the same. Maybe, alternative icons can be saved in another dict. This should reduce the abilities.json further.

This should be fixed now. Pending review in the app.

This is working as expected now.


Markdown doesn't play nicely with .NET XML documentation

I've noticed a few discrepancies in the rendering of markdown inside swashbuckle/swagger-ui.
The issue stems from the .NET xml comments and how the xml file is produced - each line is usually indented for easier human reading. The leading and trailing whitespace is trimmed, but it leaves any intermediate lines with extra leading whitespace, killing any hopes of an easy markdown conversion.
I've written a javascript workaround script which is added to a custom "index.html". It attempts to calculate the leading padding remove it from each line: https://gist.github.com/davetransom/ddb755a61ad86f0e9690
Though, the xml file could possibly be massaged into a better format via an IDocumentFilter too.
I wasn't sure if I should contribute this as a PR (it would add a .js file to the swashbuckle project), so thought I'd raise an issue for discussion first.
current: partial conversion to html
Note the sentence wrapping and the code block not behaving.

expected:

As an aside, it's worth noting that the swagger-ui version sometimes puts a markdown class on
 elements, which will lead to a nested
 situation. The javascript workaround also averts that.

Between 5.0.4 and 5.1.2, the conversion of XML comments to markdown has worsened considerably.  tags used to be converted to , for example, which allowed for some readability formatting. Now it's rendered as a single paragraph.
I hope this back-step is just temporary while the XmlComment importers are being refactored. For the time being I am sticking with 5.0.4 for applications that have active consumers though.

Unless I'm mistaken, I don't believe there has ever been any code that converts para to p tags so not sure how this was ever working.
The currently supported tags are listed in the readme - https://github.com/domaindrivendev/Swashbuckle#including-xml-comments
Looking at the incoming issues, there is definitely a desire out there for richer support but there is also so many hours in the day and other features are higher on the list of priorities right now.
It's worth noting however, that you can easily implement this yourself with a very simple extension point. Just wire up a custom operation filter to post process the generated Swagger document.
For example:
//SwaggerConfig.cs
c.IncludeXmlComments(GetXmlCommentsPath());
c.OperationFilter<FormatXmlCommentProperties>(); // after above line for correct execution order

//FormatXmlCommentProperties.cs
public class FormatXmlCommentProperties : IOperationFilter
{
    public void Apply(Operation operation, SchemaRegistry schemaRegistry, ApiDescription apiDescription)
    {
        operation.description = Formatted(operation.description);
        operation.summary = Formatted(operation.summary);
    }

    private string Formatted(string text)
    {
        if (text == null) return null;
        var stringBuilder = new StringBuilder(text);

        return stringBuilder
            .Replace("<para>", "
")
            .Replace("</para>", "
")
            .ToString();
    }
}

At some point in the future, we'll get this built in out of the box - still puzzled as to how this was formatting better previously.

Building on @domaindrivendev 's custom formatting sample above, I found that you can get some pretty decent markdown support in XML Comments support by using the following Formatted() method (or something similar to your liking) in your custom FormatXmlCommentProperties.cs class:
        public void Apply(Operation operation, SchemaRegistry schemaRegistry, ApiDescription apiDescription)
        {
            operation.description = Formatted(operation.description);
            operation.summary = Formatted(operation.summary);
        }

        private string Formatted(string text)
        {
            if (text == null) return null;

            // Strip out the whitespace that messes up the markdown in the xml comments
            // and change xml  tags to

            string resultString = Regex.Replace(text, @"^[ \t]+|[ \t]+$", "", RegexOptions.Multiline);
            resultString = Regex.Replace(resultString, @"]*>", "
", RegexOptions.IgnoreCase | RegexOptions.Singleline | RegexOptions.Multiline);
            resultString = Regex.Replace(resultString, @"]*>", "
", RegexOptions.IgnoreCase | RegexOptions.Singleline | RegexOptions.Multiline);
            resultString = Regex.Replace(resultString, @"", "", RegexOptions.Multiline);

            return resultString;
        }

Using this formatter you can have an Xml Comment block in your code that looks like this for example:
        ///
        /// Gets a the list of entities and their descriptions from the Entity Service.
        ///
        /// <remarks>
        /// ### REMARKS ###
        /// Use this call to retrieve a list of entities and their descriptions. This is used when setting up a new ViewSet.
        /// - Returns an array of **EntityDescription** objects.
        /// - The array is `ordered alphabetically` by the Name property.
        ///
        /// </remarks>
        /// <response code="200">OK</response>
        /// <returns>An array of EntityDescription objects.</returns>

Which will yield the following output:

Note that you have to comment out the  block as shown using the   strategy in your XML comments or else the Visual Studio XML compiler will say that it's badly formed and not include it in the output.

Thanks for sharing! It's very helpful.

Thanks Geokaps, I think your on the right course.
We won't be able to do much more without getting at the content as it is loaded from XML (before all the XML documentation tags have been stripped). This happens in the ApplyXmlTypeComments and ApplyXmlActionComments classes in a call to XPathNavigatorExtensiosn.ExtractContent() (typo in original). Since this is a static method there isn't much we can do to affect it's behavior externally.

Just adding a few more thoughts...
I prefer to write just markdown in each content element (albeit XML escaped), so no <para> or <see>, <c>, etc... given its destination/intention is public documentation, and not so much for VS IntelliSense. I also feel plain markdown would give a more consistent transition through to swagger, where it's consumed.
I guess it comes down to how you see Swashbuckle working in your project.

Plain Markdown
A mix of markdown and XML to satisfy both environments

After looking at this more, while the javascript solution worked as a stopgap, I think Swagger should probably have some form of XML comment "padding stripper" built in (perhaps inside XPathNavigatorExtensiosn.ExtractContent) - just to deal with the way VS XML documents are produced (since it's the extra significant whitespace that seems to be causing the issue).
Then perhaps have a separate built-in to "Convert additional elements" similar to what @geokaps has suggested - that can be applied at the configuration level, depending on your preference. Although one thing to be wary of when stripping XML comments e.g. "


Unittests for DeviceMap

We need test the we can't, for example, register multiple devices that service the same io ports. We should also just have more tests to ensure the logic in the device lookup is correct.

I'll look into this

Fixed in 040d72ef808bd6c0e0e92a27e0f53d03412c2e82. Thanks Adam!


about the datasets' "camid "

Excuse me, If I use my own datasets, how should I set the "camid"?
If I set them to the same id, will there be any negative effects?

There will be no negative effects if you don't know the exact camera id.

There will be no negative effects if you don't know the exact camera id.

but i find in
fastreid/evaluation/rank.py
Line 108
it said "Key: for each query identity, its gallery images from the same camera view are discarded."
Does this mean that I need to set the id in query images and gallery  images to be different?

This is in the test phase and there will no negative effect in training phase.

This is in the test phase and there will no negative effect in training phase.

Thanks for your reply,  now I set the camid in query images to "2", and the camid in gallery images to "3". Could I get the correct result in the test phase?  or is there anything else i need to do？

If you don't have camera information, you can set all query images' cameras different from gallery images' cameras. Then you can get the result.

If you don't have camera information, you can set all query images' cameras different from gallery images' cameras. Then you can get the result.

i get it! Thanks a lot!


Fetch URI with query string

When I need to fetch a URI that contains a query string the downloaded file will contain the query string in the name. Is there a way to avoid this? Is it possible to specify the file name?

Hi @lmtjalves,
currently this is not possible, but mesos would allow to specify the file name.  Therefore the marathon API should be enhanced to provide this functionality.
Kind regards
Johannes

Just leaving this here: https://github.com/apache/mesos/blob/master/include/mesos/mesos.proto#L442


Fixed invalid connection ref in handler

When checking for the connection.config.parseJSON flag the coffeescript incorrectly adds "this." to the front even though the connection is the ambient scope variable "connection" -- should also be corrected in tedious.coffee to remove the '@'
Found this when returning JSON column

Thank you.


Add --skip-bids-validator option

As discussed here.

Is this available in the latest docker version? I tried it with the following syntax:
docker run -i --rm -v ~/Volumes/Hanson/NKI_HealthyBrainNetwork/RU/R1/:/bids_dataset -v /home/jamielh/Volumes/Hanson/NKI_HealthyBrainNetwork/RU/R1/derivatives/MRtrix3:/outputs bids/mrtrix3_connectome /bids_dataset /outputs participant --participant_label NDARAG340ERT --skip-bids-validator --parcellation fs_2005 -nocleanup -n 5
But get the following error
Error: unrecognized arguments: --skip-bids-validator Usage: run.py bids_dir output_dir analysis_level [ options ]        (Run run.py -help for more information)
Am I entering the flag wrong (if it is available)?

No, I've not gotten around to implementing it yet; I had a fellowship application due yesterday :-/
One little trick: If you execute the container but do not provide any command-line arguments (i.e. just: "docker run -it --rm bids/mrtrix3_connectome"), you should get a help page describing exactly which command-line options are available for the particular version installed. This is a functionality that's common with the Python scripts in MRtrix3 as I make use of the same libraries.


Cannot npm install in Ubuntu 16.04, fails during preinstall script

I've tried installing from npm as well as building from source, and all installs fail during the preinstall script. I've tried using Node 0.12, 4.4.5, and 6.3.1 (switching with NVM). I think that this issue is different from #1037 (at least as that one was originally reported), because that issue shows up while executing install, not preinstall. Some of the most recent comments in that issue seem to match my problem, though.
The preinstall script seems to throw errors while Configuring libssh2:
configure: error: cannot find OpenSSL or Libgcrypt,

Yeah, theres an issue with our build structure. If you sudo apt-get install libssl that should take care of it. It's supposed to pull its own for a static link, but that doesn't appear to happen properly.

Ah yep, no libssl package on Ubuntu 16.04, but installing libssl-dev did the trick.

The CI build seems to be failing on Linux for the same reason: https://travis-ci.org/nodegit/nodegit/jobs/147599864
It might be good to update the preinstall script, or at least the README.

@mlms13 that CI error looked to be completely unrelated. I've restarted the build since it looked to be network related.

 It'll be interesting to see if it completes this time.

Well it looks like it passed, but the console output is still indicating an error. I dunno?


[Suggestion] Local world map.

ability to load an existing world's data. multi version worlds may be a problem (e.g Etho's Letsplay world). the intent more being to allow more advanced biome/world gen mods to be processed. similar to that... Amidst? Amist? the original seed viewer one that someone made a forge mod compat and in game viewer thing for...
this would allow additional "world presets" to be loaded from an existing world data and be able to map things like "Better Nether/Better End" dimensions (as well as other modded dimensions). more a longterm goal this one...

That is quite out of scope of this tool. There are enough tools to display a map of a generated world, such as https://github.com/Querz/mcaselector or mini-map mods. I don't think there is a need to redo this here; not every tool needs to do everything.


made in webflow js

Displayed for some clients? To be removed.

commented out entire webflow js file with https://github.com/SriSathyaSaiVidyaVahini/SSSVV-landing/commit/acca8a99589e2f139cb6288fae6724365d3447c7
Need to see if this causes any other issues.

The webflow js was needed for carousel views etc. So, changed attrib to SSSVV team instead of webflow. Maybe that will work? https://github.com/SriSathyaSaiVidyaVahini/SSSVV-landing/commit/89c4d5bac0f28ec3dd4ce4c9fb62240fe0d945ec

Sairam Bro.,
The 'Web flow' is no longer appearing. Thanks.
However, now I see in the same bottom RHS, 2 SSSVV logos appearing for
Multimedia page. This appears only for this page and not for another
sections. Also, could you please correct the SSSCHV link in the bottom of
the page? It is given as http://www.ssschv.srisathyasai.org/, whereas it
should be https://ssschv.srisathyasai.org/ This was reported multiple times
but was not corrected by the developer. Thanks.
[image: image.png]
Loving Sairam
V L Subramanian
Planning & Strategy Lead
Sri Sathya Sai Vidya Vahini
Mob: +91 810 5658 625
On Fri, 21 Oct 2022 at 11:30, hn-88 @.***> wrote:

@vlsubramanian https://github.com/vlsubramanian can you please check if
the Made in webflow banner is still displayed? If yes, can you please tell
me on which page and when it is displayed?
—
Reply to this email directly, view it on GitHub
https://github.com/SriSathyaSaiVidyaVahini/SSSVV-landing/issues/4#issuecomment-1286497449,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASR474M2EQAD5P22D4IA7JLWEIWQZANCNFSM6AAAAAARKY7WQ4
.
You are receiving this because you were mentioned.Message ID:
@.***>

ssschv link corrected in all pages with https://github.com/SriSathyaSaiVidyaVahini/SSSVV-landing/commit/80c2486bc2b832a1e5287e0bd8f898d48567c090

https://github.com/SriSathyaSaiVidyaVahini/SSSVV-landing/commit/fbe4d1a65dc8b25060c2c925c85325cd5b1ca4de should stop the popup on bottom right with SSSVV logo.

Thanks a lot.
Loving Sairam
V L Subramanian
Planning & Strategy Lead
Sri Sathya Sai Vidya Vahini
Mob: +91 810 5658 625
On Fri, 21 Oct 2022 at 17:41, hn-88 @.***> wrote:

Closed #4
https://github.com/SriSathyaSaiVidyaVahini/SSSVV-landing/issues/4 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/SriSathyaSaiVidyaVahini/SSSVV-landing/issues/4#event-7640100478,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASR474JUACNXNSOC7XM4JSTWEKB7VANCNFSM6AAAAAARKY7WQ4
.
You are receiving this because you were mentioned.Message ID:
<SriSathyaSaiVidyaVahini/SSSVV-landing/issue/4/issue_event/7640100478@
github.com>


Make goji.ServeListener() return error instead of using log.Fatal internally

Are you running into this in production? What sort of behavior are you looking to perform with the resulting error?
I originally conceived of the top-level goji package as being a quick way to get started with goji, but which advanced users would inline and customize. In this vein, Serve and friends are designed to do opinionated reasonable things in the face of errors, etc.
This has clearly become less true with time, as a great many people (including me) have used the top-level goji package in production, so let me know what pain points you're running in to for which an error return would be helpful, and I'll see what I can do

The main point was to integrate it with own logger. Now I created own web.New() and use it instead of top level goji methods. I think it can be closed.


Create a callback for links and other attachments

In my bot I have implemented unfurling. This is accomplished by scanning every incoming message for links, iterating over them, and then posting adaptive cards. What if this process could be abstracted?
The framework could throw an event "LinkAdded" whose handler would look like this
const handleLinks = async event => {
    const { group, bot, urls } = event
    for (url of urls) {
        // do stuff for url
    }
}

The point is: make it easier for developers to process links. The same could also be said for any type of message:

events
tasks
notes
code snippets
and so forth

Another requirement... the framework when extracting URLs should be able to ignore URLs that are included in a response.
Imagine a user sends a URL in a message, then another responds to that message by quoting it. The first URL should be unfurled, but the second URL (really a duplicate of the first) should be ignore.


Use create_executor block for winrm calls for SCVMM inventory collection.

This is a minor update that uses the create_executor block supported by our current version of winrm. It also eliminates a deprecation warning.

@bronaghs @jerryk55 Please review.

Looks fine other than the Rubocop space warning.

@djberg96 - LGTM too.

@jerryk55 Fixed the rubocop issue (ugh).

@jerryk55 Hang on, this is spewing results to the console for some reason.

Disregard last comment @jerryk55, it was something on my end.

@roliveri @blomquisg should this be backported to Darga?

@chessbyte I think it should. It's a simple change and I think it would be best to keep the winrm/SCVMM
behavior in sync between the branches, if possible.


Analsysis of the 20yrs data

Analysis for 20yrs data done. I think this is a better approach, what do you think?

Never mind, lets keep the original analysis....


Various fixes for Tivi

A collection of fixes for Tivi's usage.

Thanks for testing and fixing the API. This saves me a lot of work :)


Moving to new row in Edit Data doesn't commit refresh previous rows changes

Azure Data Studio Version: master

Steps to Reproduce:

Edit Data on a table
Add a new row that has auto-populated files, such as generated primary keys or columns with default values
Navigate to next row

Expected: Row changes are committed when focus moves to different row and generated columns are refreshed with actual values.
Actual: Generated columns are not refreshed until at least one column in new row is updated and focus moved to another column.  Notice the issue in the below screenshot with <TBD> and getdate values after focus is in next row.

Looking somewhat related to https://github.com/microsoft/azuredatastudio/issues/2574.

Closing as it no longer happens with the latest build


Unable to get environment variables in unit test

It will help me do the assertions, if i can preset some enviroment variables and use them during testing.
Just like Postman's  pm.environment.get().
I just started using it. Is there another way to do this?

Sorry, i didn't find #3218


How to test this model?

Thank you for the amazing repo
However, it appears as if this repo is used for training the model. is it possible to create a file that just takes the input image and generates an output from the trained model?

Hi!
The repo is precisly for evaluating the model not for training. If you want to extract results for a single image you can create your own script using fragments of the code, however, you can get the same results using batch_size=1


Artifact upload

My custom build task uploads the artifact it creates. I tried both approaches, via Artifact Logging Commands and via Publish-BuildArtifact cmdlet. Both way's uploaded my artifacts successfully. However, I do see in the console during the execution of my task the following lines like:
2016-02-23T19:27:53.5176503Z Max Concurrent Uploads 2, Max Creators 1

2016-02-23T19:27:53.5607144Z Found 1 files to upload.

2016-02-23T19:27:53.5646509Z Files found locally 1,

2016-02-23T19:27:53.5656503Z Files evaluated 0,

2016-02-23T19:27:53.5656503Z Files left to evaluate 1.,

2016-02-23T19:27:53.5666503Z Files created without upload 0,

2016-02-23T19:27:53.5666503Z Files uploaded 0

2016-02-23T19:27:53.5666503Z Files left to process 1

2016-02-23T19:27:53.5676504Z ---------------------------

2016-02-23T19:27:55.5720377Z Files found locally 1,

2016-02-23T19:27:55.5730654Z Files evaluated 1,

2016-02-23T19:27:55.5730654Z Files left to evaluate 0.,

2016-02-23T19:27:55.5750601Z Files created without upload 0,

2016-02-23T19:27:55.5761204Z Files uploaded 0

2016-02-23T19:27:55.5770629Z Files left to process 1

2016-02-23T19:27:55.5780589Z ---------------------------

2016-02-23T19:27:57.5799537Z Created 0 files without uploading content. Total files processed 1

Is there a way not to display those lines?
Thanks

That's what those cmds and cmdlets output.  Those just call REST APIs so you could write your own but that would be alot more work.  Seeing the output of what it's doing is a good thing.  If you look at the vso-agent open source repo, you can look into what the commands do if you're interested in writing your own.  I'll caution it's not trivial and alot of work