Add support for Vector_Editor widget to graph editor

Support node widget that operates on lists of values. It needs to allow:

adding new entry to the list (a + button)
removing existing entries from the list (e.g. by dragging them out of the node)
reordering existing entries in the list (e.g. by dragging them around)
editing existing entries using specified item widget

The engine provided widget type will specify the default value to insert on add action (idem_default), and a widget to use for existing items (item_editor).
Vector_Editor
  item_editor: Widget
  item_default: Text
  label: (Nothing | Text) = Nothing
  display: Display = Display.When_Modified

Reference design:

The vector editor needs to be rendered as part of the node. This will likely require refactoring node input ports rendering to allow inserting interactive UI elements in between fragments of code text.
Acceptance criteria:
A node with Table.select_columns method should receive a vector widget on columns parameter that allows editing a list of selected columns. All of adding, removing, reordering and modifying already present names should be supported.

Depends on #5134

@Frizi you made my day with this design :D


Add uncons

I wrote:
f_uncons :: F.ByteArray -> Maybe (F.Word8, F.ByteArray)
f_uncons x = case x F.! 0 of
    Nothing -> Nothing
    Just c -> Just (c, F.drop 1 x)

But seems like it should be in the library, which is likely more efficient. Naturally generalise to all the things.

definitely. unsnoc comes to mind too

link to #30 relative to unsnoc

I've added uncons and unsnoc, but not plumbed through any export yet

it's all plumbed through, and support for boxed array and String is there too


travis: hhvm added, clear structure

If there is hhvm-nightly, there should be hhvm as well.
Also changed spaces to more readable and standard ones.

First of all, thanks for your contribution!
Because of a bug in the reflection system of HHVM, some tests related to the Hydra documentation fail. Can you add hvvm to the allow_failure section?

Sure, added!

Thank you @TomasVotruba

Thanks


implicit-hie doesn't recognize foreign-library sections in cabal

I was attempting to follow the tutorial at https://nokomprendo.gitlab.io/posts/tuto_075/2021-09-12-fr-README.html
HLS is flagging the Binding module with Multi Cradle: no prefixes matched.  The module is listed in the other-modules section of a foreign-library section in the cabal file, as follows:
foreign-library fibohs
    type:               native-shared
    cc-options:         -std=c++17
    c-sources:          cbits/binding.cpp
    install-includes:   cbits/binding.hpp
    hs-source-dirs:     ffi
    build-depends:      ffi, base
    other-modules:      Binding

Adding this hie.yaml
cradle:
  cabal:

fixes the problem, which indicates this is a problem with implicit-hie.

The fix is published in <IP_ADDRESS>.
This should be fixed once HLS picks up the new version.


Lost commit when running the prepareGhPages and publishGhPages tasks separately

I raised issue #221 and after upgrading and I got what I thought was the same problem. Digging into it, it looks like a later change has introduced a new problem but with the same output, the publish task is skipped.
This PR splits the repo function in GithubPagesPlugin into 3 functions that open the repository, verify it matches the configuration and  resets the repository to the target branch, with only the prepare task calling the reset function.

Hi @rodm, I've released gradle-git-publish to supersede the org.ajoberstar.github-pages plugin. I'll be formally deprecating this plugin in an upcoming release, and won't be making further changes here.
Can you test the new plugin to see if it still presents the same issue?

@ajoberstar I had tried using it but I got an exception, I've created a new issue for that, gitPublishReset throws exception . Given that gradle-git-publish is only at version 0.1 is it ready to replace the github-pages plugin?

It is 0.1 but it's effectively the same code as has been in gradle-git for 5 years. I wanted to give people a few months to validate that it doesn't have any major API problems before going to 1.0.


Fix: stop start build server when getRandomPipepath return empty

Please also consider handle the connection error in java side. Use isResolved() to skip importing from gradle build server


Incorrect terminology used

Hi - you need to change:
"In the new model, endpoints are segregated into three categories, Optimize, REQUIRED and Default, providing a priority-based pivot"
to
"In the new model, endpoints are segregated into three categories, Optimize, ALLOW and Default, providing a priority-based pivot"

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

ID: 6279eb8d-1df4-189d-8952-b6cde4cdf4a3
Version Independent ID: e27f03c9-30f5-7fba-02f4-868abbf5d4ec
Content: Office 365 Network Connectivity Principles
Content Source: Enterprise/office-365-network-connectivity-principles.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@steveianson thank you so much for the feedback. We're looking into this, and an update will be posted here soon.


Join all interfaces to multicast

Fix multicast dgram listen on a multiple net interfaces platform.
If call join_multicast_v4() with <IP_ADDRESS> on Win10, a random interface will be choose. The listen_socket may not be able to receive multicast dgram.
Signed-off-by: fufesou<EMAIL_ADDRESS>

Thanks for opening the PR! Is this an issue only on Win10?  How about Win11?

Is this an issue only on Win10? How about Win11?

I'll try to install a Win11 VM and test it.
By the way, webrtc-rs/mdns also joins all interfaces.

I'll try to install a Win11 VM and test it.

Now I understand more, and this looks like a good fix. Win11 is not a blocking issue.


Unify Arm64 kernels, independent of bitpacking bitwidth

See comment here.
This depends on #350, so will be a draft PR until that is merged.
What do these changes do?
At the moment there are three optimised assembly kernels for Aarch64: a 4x4 uint32 kernel, a 4x4 uint64 kernel, and an 8x4 uint64 kernel.
I can't see a reason to maintain a distinction between uint32 input and uint64 input at a kernel level, because a kernel that's written to accept uint64 input can accept uint32 input as long as some kernel parameters are adjusted (e.g. kernel depth halved) and the Ruy LHS/RHS packing is done correctly.
This PR removes the separate uint32 4x4 kernel, and makes 8x4 kernel and the remaining 4x4 kernels usable with either uint64 or uint32 input. Whereas there were previously three Aarch64 kernel code-paths and three kernel registrations, now there are two Aarch64 kernel code paths (8x4 and 4x4) and four kernel registrations (uint64 and uint32 for each of the code paths).
The main benefit is reduced code and binary size. However, this will also speed up binary convolutions which use uint32 input, as they were previously unable to use a faster 8x4 kernel.
How Has This Been Tested?
CI tests. The 'big' tests that don't run on CI have be run locally (and pass).
Benchmark Results
N/A.
Related issue number
N/A.

Looks like the merge of #346 introduced some merge conflicts here.


when reopen editor , force vscode to refocus all files cache or give us a command for do it.

When re open a editor , we need a way to recache all tabs opened , or also maybe all files.
Is there a feature for this ? for rescan all files to force plugins like eslint to cache all the project warn and error again?
Because when i reopen editor, i need click one by one amd open one by one the full project !
This make no sense , am sure something already exist no, or a option not activated by default  ?
This gif show you a new vscode opened, and a lost all my eslint error !
restart eslint not work, but i need click on all tabs one by one to make it work.

thanks

/duplicate https://github.com/microsoft/vscode/issues/13953


[CockroachDB] Support setting a username/password/database

Fixes #6299
Documentation
No new documentation was added.
Functionality
CockroachDB container now supports setting a username/password/database like:
CockroachContainer cockroach = new CockroachContainer(
                CockroachDBTestImages.FIRST_COCKROACHDB_IMAGE_WITH_ENV_VARS_SUPPORT
            )
                .withUsername("test_user")
                .withPassword("test_password")
                .withDatabaseName("test_database")

Thanks for your contribution, @vcvitaly ! This is now merged in main branch and it will be available in the next release.


authentication error: http error: error issuing the request with reqwest: error sending request: JsValue(TypeError: Only absolute URLs are supported

Scenario & Reproduction Steps
node v22.5.0
authentication error: http error: error issuing the request with reqwest: error sending request: JsValue(TypeError: Only absolute URLs are supported
TypeError: Only absolute URLs are supported
at getNodeRequestOptions (/Users/xxx/Documents/www/xxx/node_modules/node-fetch/lib/index.js:1327:9)
at /Users/xxx/Documents/www/xxx/node_modules/node-fetch/lib/index.js:1450:19
at new Promise ()
at fetch (/Users/xxx/Documents/www/xxx/node_modules/node-fetch/lib/index.js:1447:9)
at module.exports.__wbg_fetch_bc7c8e27076a5c84 (/Users/xxx/Documents/www/xxx/node_modules/@1password/sdk-core/nodejs/core.js:312:17)
at op_js_core.wasm.reqwest::wasm::client::js_fetch::ha27955dbb1f30d16 (wasm://wasm/op_js_core.wasm-014254e6:wasm-function[3158]:0x2e2f95)
at op_js_core.wasm.<op_reqwest_fetcher::ReqwestFetcher as op_http_fetcher::HttpFetcher>::request::{{closure}}::h3a181331326cc5ba (wasm://wasm/op_js_core.wasm-014254e6:wasm-function[107]:0x84076)
at op_js_core.wasm.<op_b5_client::UnauthenticatedSession as op_b5_client::Session>::request::{{closure}}::hff893f425ad048ce (wasm://wasm/op_js_core.wasm-014254e6:wasm-function[686]:0x1d2edd)
at op_js_core.wasm.op_service_account_authentication::lookup_auth::{{closure}}::hefde251e07cc903f (wasm://wasm/op_js_core.wasm-014254e6:wasm-function[221]:0x1029db)
at op_js_core.wasm.<op_service_account_authentication::ServiceAccountAuthenticator as op_b5_client::session_refresher::Authenticator>::new_unverified_session::{{closure}}::he3cc43e42663c1ac (wasm://wasm/op_js_core.wasm-014254e6:wasm-function[78]:0x3d794)
at op_js_core.wasm.<core::pin::Pin as core::future::future::Future>::poll::h8866a0ab15d095a0 (wasm://wasm/op_js_core.wasm-014254e6:wasm-function[89]:0x5e474)
at op_js_core.wasm.wasm_bindgen_futures::future_to_promise::{{closure}}::{{closure}}::h8eb95870477ddaf6 (wasm://wasm/op_js_core.wasm-014254e6:wasm-function[484]:0x190a43)
at op_js_core.wasm.wasm_bindgen_futures::queue::Queue::new::{{closure}}::h40a7ee1c15b8d20a (wasm://wasm/op_js_core.wasm-014254e6:wasm-function[1431]:0x2572c1)
at op_js_core.wasm.<dyn core::ops::function::FnMut<(A,)>+Output = R as wasm_bindgen::closure::WasmClosure>::describe::invoke::h992c8f981e42a5a1 (wasm://wasm/op_js_core.wasm-014254e6:wasm-function[4791]:0x30ca09)
at __wbg_adapter_26 (/Users/xxx/Documents/www/xxx/node_modules/@1password/sdk-core/nodejs/core.js:219:10)
at real (/Users/xxx/Documents/www/xxx/node_modules/@1password/sdk-core/nodejs/core.js:204:20))
My Code:
let client  = await sdk.createClient({
auth: process.env.OP_SERVICE_ACCOUNT_TOKEN,
// Set the following to your own integration name and version.
integrationName: "XXX",
integrationVersion: "v1.0.0",
});
Actual Behavior
i am running the current example on github page
Expected Behavior
conect
SDK version
<EMAIL_ADDRESS>Additional information
No response

Hey @mesaque , thank you for reporting!
That is unexpected - Is your service account token a valid one? You can check this with the 1Password CLI - exporting the token to the OP_SERVICE_ACCOUNT_TOKEN environment variable and running an op whoami command, after installing the CLI.
Additionally, you can check the formatting of your service account to see the actual URL that it makes requests for:
echo $OP_SERVICE_ACCOUNT_TOKEN | base64 -d

You should get back a JSON with a signInAddress field - is the value of that field expected? Is it prefixed with https:// or suffixed by anything after .com/.eu/.ca?
Let us know, happy to debug this further, thank you.


It's hard to discover how to use Transform3D archetype

The archetype Transform3D is confusing to use. It's an archetype and shows up in the archetype docs, but can't be used on it's own. See rendered docs.
The LSP preview is also quite useless:

If it isn't an object that users can't use, we shouldn't expose it to them. In this case though I wonder if it wouldn't be easier make a nice to use init method so we can just use the archetype directly and not have to go down to the component level at all. Perhaps something like:
class Transform3D(Archetype):
    def __init__(self, *, translation, rotation, scale, mat3x3, mat3x4):
        ...

Any Rerun types that the archetype init takes as a parameter should be exposed in the top level rerun (rr) module
That way I can just do:
rr.log("camera", rr.Transform3D(translation=translation, rotation=rotation))
# or
rr.log("camera", rr.Transform3D(translation=translation, mat3x3=rotation))
# or
rr.log("camera", rr.Transform3D(mat3x4=affine))

I gave this a shot, but I don't understand Python enough to do this alone 😭


Flutter Folder Organization

Flutter Folder Organization
Introduction paragraph(s):
Flutter allows one to program an entire application in one file. However, however, as the size of the android application increases, identifying and fixing bugs in a single large file is challenging. This raises the need to identify a preferable folder structure. This article explains the folders and files autogenerated during Flutter project creation and provides a framework for organizing them to enhance the scalability of a project.
Key takeaways:

An explanation to Flutter folders and files
Organisation of Flutter project files and folders for large scale application.
Benefits of maintaining proper folder structure.

References:
N/A

good topic @jerimkaura - approved


2.0.7

Codecov Report
Base: 36.36% // Head: 36.36% // No change to project coverage :thumbsup:

Coverage data is based on head (117639e) compared to base (d510eaa).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #34   +/-   ##
=======================================
  Coverage   36.36%   36.36%
=======================================
  Files           1        1
  Lines          22       22
=======================================
  Hits            8        8
  Misses         14       14

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Upgrade to libsodium 1.0.15

There are a few new releases of libsodium available (current version depends on 1.0.12). Changelog
Upgrading to 1.0.14 or later will also fix #16.

Any update?

This has been addressed in #60 and has been published to crates.io as version 0.9.0 now.


Localizing in 2D from a 3D map

We are creating maps using VLP16s with cartographer in 3D configuration. Is it possible to localize in this map using a 2D planar lidar?
Further, can the map be updated while in localization mode?

yes, cartographer support pure localization through load previously created .pbstream file(local slam state), meanwhile, it can't update submaps but lt can keep 3~5 submaps that scan match with previous submaps.

@shamengjun Localization in 2D works from a pbstream created in 3D?

@ryanmeasel I'm so sorry, this is not known by me currently, I am only focused on 2D currently.

This probably doesn't work because the 2D SLAM grid is completely different to the 3D structure that you saved.


[REEF-237] Add a DriverStarted handler that generates the ClassHierarchy

This adds ClassHierarchyGeneratingDriverStartObserver and registers it as a DriverStartHandler. This new class generates the class hierarchy for the assemblies found in reef/global.
HelloREEF is adapted to no longer generate the class hiearchy.
JIRA: REEF-237

Looking at it.

Can you merge from master? There is little conflict.

I'll resolve conflicts and rebase.


Factor out profiling settings

This factors profiling settings into a settings struct, and then stores settings in that struct instead of a half-dozen fields of other AgentConfig structs.
Additional Notes
Because the core agent uses runtime settings for profiling, instead of using an AgentConfig, we can't store the settings in one place, so the refactor is minimal in that agent (but still cleaner!)
Describe how to test your changes
Enable profiling for each agent, and verify that profiles appear in the app.
For datadog-agent, also enable and disable profiles at runtime.
Checklist

[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] The appropriate team/.. label has been applied, if known.
[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.

Both look like intermittents!


Roundup: [oss-security] Xen Security Advisory 209 (CVE-2017-2620) - cirrus_bitblt_cputovideo does not check if memory region is safe

Here is a vulnerability from the oss-security mailing list
for Vulnerability Roundup 26.
Instructions:
Identification
Identify if we have the software, in 16.09, 17.03, and unstable.
Then determine if we are vulnerable, and make a comment with
your findings. It can also be helpful to specify if you think there is
a patch, or if it can be fixed via a general update.
Example:
unstable: we are not vulnerable (link to the package)
17.03: we are vulnerable (link to the package)
16.09: we don't have it packaged

IMPORTANT: If you believe there are possibly related issues, bring
them up on the parent issue!
Patching
Start by commenting on this issue saying you're working on a patch.
This way, we don't duplicate work.
If you open a pull request, tag this issue and the master issue
for the roundup.
If you commit the patch directly to a branch, please leave a comment
on this issue with the branch and the commit hash, example:
fixed:

release-16.09: abc123

Upon Completion ...

[ ] Update Graham's database

Info
Triage Indicator:
-needs-triage +roundup26 thread:0000000000003de0 # [oss-security] Xen Security Advisory 209 (CVE-2017-2620) - cirrus_bitblt_cputovideo does not check if memory region is safe

File Search: https://search.nix.gsc.io/?q=xen&i=fosho&repos=nixos-nixpkgs
GitHub Search: https://github.com/NixOS/nixpkgs/search?utf8=✓&q=xen+in%3Apath&type=Code

Thu, 23 Feb 2017 15:52:31 +0000 Xen.org security team <security-at-xen.org><EMAIL_ADDRESS>-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

            Xen Security Advisory CVE-2017-2620 / XSA-209
                              version 4

   cirrus_bitblt_cputovideo does not check if memory region is safe

UPDATES IN VERSION 4
====================

Include a prerequisite patch for qemu-upstream, correct statement
regarding the availability of a qemu-traditional patch.

ISSUE DESCRIPTION
=================

In CIRRUS_BLTMODE_MEMSYSSRC mode the bitblit copy routine
cirrus_bitblt_cputovideo fails to check wethehr the specified memory
region is safe.

IMPACT
======

A malicious guest administrator can cause an out of bounds memory
write, very likely exploitable as a privilege escalation.

VULNERABLE SYSTEMS
==================

Versions of qemu shipped with all Xen versions are vulnerable.

Xen systems running on x86 with HVM guests, with the qemu process
running in dom0 are vulnerable.

Only guests provided with the "cirrus" emulated video card can exploit
the vulnerability.  The non-default "stdvga" emulated video card is
not vulnerable.  (With xl the emulated video card is controlled by the
"stdvga=" and "vga=" domain configuration options.)

ARM systems are not vulnerable.  Systems using only PV guests are not
vulnerable.

For VMs whose qemu process is running in a stub domain, a successful
attacker will only gain the privileges of that stubdom, which should
be only over the guest itself.

Both upstream-based versions of qemu (device_model_version="qemu-xen")
and `traditional' qemu (device_model_version="qemu-xen-traditional")
are vulnerable.

MITIGATION
==========

Running only PV guests will avoid the issue.

Running HVM guests with the device model in a stubdomain will mitigate
the issue.

Changing the video card emulation to stdvga (stdvga=1, vga="stdvga",
in the xl domain configuration) will avoid the vulnerability.

CREDITS
=======

This issue was discovered by Gerd Hoffmann of Red Hat.

RESOLUTION
==========

Applying the appropriate attached patch resolves this issue.

xsa209-qemuu/*.patch     qemu-xen, qemu upstream
xsa209-qemut.patch       qemu-xen-traditional

$ sha256sum xsa209* xsa209*/*
167af9ed7163fa7cf4abb52f865290ced3163c7684151bdc1324eb5e534faf13  xsa209-qemut.patch
e698b73d8de24af0fe33968a43561e5e1d094f4caf2443caa447b552677d2683  xsa209-qemuu/0001-display-cirrus-ignore-source-pitch-value-as-needed-i.patch
50c60e45151ef2265cce4f92b204e9fd75f8bc8952f097e77ab4fe1c1446bc98  xsa209-qemuu/0002-cirrus-add-blit_is_unsafe-call-to-cirrus_bitblt_cput.patch

DEPLOYMENT DURING EMBARGO
=========================

Deployment of the patches described above (or others which are
substantially similar) is permitted during the embargo, even on
public-facing systems with untrusted guest users and administrators.

However, deployment of the "stdvga" mitigation (changing the video
card emulation to stdvga) is NOT permitted (except where all the
affected systems and VMs are administered and used only by
organisations which are members of the Xen Project Security Issues
Predisclosure List).  Specifically, deployment on public cloud systems
is NOT permitted.  This is because this produces a guest-visible
change which will indicate which component contains the vulnerability.

Additionally, distribution of updated software is prohibited (except
to other members of the predisclosure list).

Predisclosure list members who wish to deploy significantly different
patches and/or mitigations, please contact the Xen Project Security
Team.

(Note: this during-embargo deployment notice is retained in
post-embargo publicly released Xen Project advisories, even though it
is then no longer applicable.  This is to enable the community to have
oversight of the Xen Project Security Team's decisionmaking.)

For more information about permissible uses of embargoed information,
consult the Xen Project community's agreed Security Policy:
  http://www.xenproject.org/security-policy.html
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQEcBAEBAgAGBQJYrwN/AAoJEIP+FMlX6CvZQoQIAK9UiN9VwXv1I0E7X1TL2TjE
P9SNXkI5wKiwCq22pbz9pjBO//ia3M5UoxpDMwaMAQzn9bEThHnki8x2njRxIEF7
frxm6B8DpHLCoRHiqgwi018JHLLcSbr+KQrZqBns1j5BfOF0in89A8cgBmQrziyX
bj9853Q8dHSUNW1vi8vZkMacIwxMCg4sBLjSRUoqiWmoyfU6XodRwZ3LoglsofTj
/jk/G5OiitqXDBPzvclPRddQ53xiN9eN3fV8IdG6QpX6F+C2qQVDyS8kAqqFmmm6
Vn6yl9UxrmP0OmvQ5CgUw8GWQoY3OqObjiPgfNUdbN+CLjdhdGfF3kGuYIniqd4=
=I92f
-----END PGP SIGNATURE-----

xsa209-qemut.patch
From: Gerd Hoffmann<EMAIL_ADDRESS>Subject: [PATCH 3/3] cirrus: add blit_is_unsafe call to cirrus_bitblt_cputovideo

CIRRUS_BLTMODE_MEMSYSSRC blits do NOT check blit destination
and blit width, at all.  Oops.  Fix it.

Security impact: high.

The missing blit destination check allows to write to host memory.
Basically same as CVE-2014-8106 for the other blit variants.

The missing blit width check allows to overflow cirrus_bltbuf,
with the attractive target cirrus_srcptr (current cirrus_bltbuf write
position) being located right after cirrus_bltbuf in CirrusVGAState.

Due to cirrus emulation writing cirrus_bltbuf bytewise the attacker
hasn't full control over cirrus_srcptr though, only one byte can be
changed.  Once the first byte has been modified further writes land
elsewhere.

[ This is CVE-2017-2620 / XSA-209  - Ian Jackson ]

Fixed compilation by removing extra parameter to blit_is_unsafe. -iwj

Reported-by: Gerd Hoffmann<EMAIL_ADDRESS>Signed-off-by: Gerd Hoffmann<EMAIL_ADDRESS>Signed-off-by: Ian Jackson<EMAIL_ADDRESS>---
diff --git a/hw/cirrus_vga.c b/hw/cirrus_vga.c
index e6c3893..45facb6 100644
--- a/hw/cirrus_vga.c
+++ b/hw/cirrus_vga.c
@@ -900,6 +900,10 @@ static int cirrus_bitblt_cputovideo(CirrusVGAState * s)
 {
     int w;

+    if (blit_is_unsafe(s)) {
+        return 0;
+    }
+
     s->cirrus_blt_mode &= ~CIRRUS_BLTMODE_MEMSYSSRC;
     s->cirrus_srcptr = &s->cirrus_bltbuf[0];
     s->cirrus_srcptr_end = &s->cirrus_bltbuf[0];
@@ -925,6 +929,10 @@ static int cirrus_bitblt_cputovideo(CirrusVGAState * s)
 	}
         s->cirrus_srccounter = s->cirrus_blt_srcpitch * s->cirrus_blt_height;
     }
+
+    /* the blit_is_unsafe call above should catch this */
+    assert(s->cirrus_blt_srcpitch <= CIRRUS_BLTBUFSIZE);
+
     s->cirrus_srcptr = s->cirrus_bltbuf;
     s->cirrus_srcptr_end = s->cirrus_bltbuf + s->cirrus_blt_srcpitch;
     cirrus_update_memory_access(s);

xsa209-qemuu/0001-display-cirrus-ignore-source-pitch-value-as-needed-i.patch
From 52b7f43c8fa185ab856bcaacda7abc9a6fc07f84 Mon Sep 17 00:00:00 2001
From: Bruce Rogers<EMAIL_ADDRESS>Date: Tue, 21 Feb 2017 10:54:38 -0800
Subject: [PATCH 1/2] display: cirrus: ignore source pitch value as needed in
 blit_is_unsafe

Commit 4299b90 added a check which is too broad, given that the source
pitch value is not required to be initialized for solid fill operations.
This patch refines the blit_is_unsafe() check to ignore source pitch in
that case. After applying the above commit as a security patch, we
noticed the SLES 11 SP4 guest gui failed to initialize properly.

Signed-off-by: Bruce Rogers<EMAIL_ADDRESS>Message-id<EMAIL_ADDRESS>Signed-off-by: Gerd Hoffmann<EMAIL_ADDRESS>---
 hw/display/cirrus_vga.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/hw/display/cirrus_vga.c b/hw/display/cirrus_vga.c
index 7bf3707..34a6900 100644
--- a/hw/display/cirrus_vga.c
+++ b/hw/display/cirrus_vga.c
@@ -288,7 +288,7 @@ static bool blit_region_is_unsafe(struct CirrusVGAState *s,
     return false;
 }

-static bool blit_is_unsafe(struct CirrusVGAState *s)
+static bool blit_is_unsafe(struct CirrusVGAState *s, bool dst_only)
 {
     /* should be the case, see cirrus_bitblt_start */
     assert(s->cirrus_blt_width > 0);
@@ -302,6 +302,9 @@ static bool blit_is_unsafe(struct CirrusVGAState *s)
                               s->cirrus_blt_dstaddr & s->cirrus_addr_mask)) {
         return true;
     }
+    if (dst_only) {
+        return false;
+    }
     if (blit_region_is_unsafe(s, s->cirrus_blt_srcpitch,
                               s->cirrus_blt_srcaddr & s->cirrus_addr_mask)) {
         return true;
@@ -667,7 +670,7 @@ static int cirrus_bitblt_common_patterncopy(CirrusVGAState * s,

     dst = s->vga.vram_ptr + (s->cirrus_blt_dstaddr & s->cirrus_addr_mask);

-    if (blit_is_unsafe(s))
+    if (blit_is_unsafe(s, false))
         return 0;

     (*s->cirrus_rop) (s, dst, src,
@@ -685,7 +688,7 @@ static int cirrus_bitblt_solidfill(CirrusVGAState *s, int blt_rop)
 {
     cirrus_fill_t rop_func;

-    if (blit_is_unsafe(s)) {
+    if (blit_is_unsafe(s, true)) {
         return 0;
     }
     rop_func = cirrus_fill[rop_to_index[blt_rop]][s->cirrus_blt_pixelwidth - 1];
@@ -784,7 +787,7 @@ static void cirrus_do_copy(CirrusVGAState *s, int dst, int src, int w, int h)

 static int cirrus_bitblt_videotovideo_copy(CirrusVGAState * s)
 {
-    if (blit_is_unsafe(s))
+    if (blit_is_unsafe(s, false))
         return 0;

     cirrus_do_copy(s, s->cirrus_blt_dstaddr - s->vga.start_addr,
--
2.1.4

xsa209-qemuu/0002-cirrus-add-blit_is_unsafe-call-to-cirrus_bitblt_cput.patch
From 15268f91fbe75b38a851c458aef74e693d646ea5 Mon Sep 17 00:00:00 2001
From: Gerd Hoffmann<EMAIL_ADDRESS>Date: Tue, 21 Feb 2017 10:54:59 -0800
Subject: [PATCH 2/2] cirrus: add blit_is_unsafe call to
 cirrus_bitblt_cputovideo

CIRRUS_BLTMODE_MEMSYSSRC blits do NOT check blit destination
and blit width, at all.  Oops.  Fix it.

Security impact: high.

The missing blit destination check allows to write to host memory.
Basically same as CVE-2014-8106 for the other blit variants.

The missing blit width check allows to overflow cirrus_bltbuf,
with the attractive target cirrus_srcptr (current cirrus_bltbuf write
position) being located right after cirrus_bltbuf in CirrusVGAState.

Due to cirrus emulation writing cirrus_bltbuf bytewise the attacker
hasn't full control over cirrus_srcptr though, only one byte can be
changed.  Once the first byte has been modified further writes land
elsewhere.

[ This is CVE-2017-2620 / XSA-209  - Ian Jackson ]

Reported-by: Gerd Hoffmann<EMAIL_ADDRESS>Signed-off-by: Gerd Hoffmann<EMAIL_ADDRESS>---
 hw/display/cirrus_vga.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/hw/display/cirrus_vga.c b/hw/display/cirrus_vga.c
index 34a6900..5901250 100644
--- a/hw/display/cirrus_vga.c
+++ b/hw/display/cirrus_vga.c
@@ -865,6 +865,10 @@ static int cirrus_bitblt_cputovideo(CirrusVGAState * s)
 {
     int w;

+    if (blit_is_unsafe(s, true)) {
+        return 0;
+    }
+
     s->cirrus_blt_mode &= ~CIRRUS_BLTMODE_MEMSYSSRC;
     s->cirrus_srcptr = &s->cirrus_bltbuf[0];
     s->cirrus_srcptr_end = &s->cirrus_bltbuf[0];
@@ -890,6 +894,10 @@ static int cirrus_bitblt_cputovideo(CirrusVGAState * s)
 	}
         s->cirrus_srccounter = s->cirrus_blt_srcpitch * s->cirrus_blt_height;
     }
+
+    /* the blit_is_unsafe call above should catch this */
+    assert(s->cirrus_blt_srcpitch <= CIRRUS_BLTBUFSIZE);
+
     s->cirrus_srcptr = s->cirrus_bltbuf;
     s->cirrus_srcptr_end = s->cirrus_bltbuf + s->cirrus_blt_srcpitch;
     cirrus_update_memory_access(s);
--
2.1.4

Already applied.


Support Iterative Filling in Direct Mapped Cache Configurations

Currently, the direct-mapped caches are capable of supporting fills of size 512. This means that the downstream serialization/deserialization/buffering logic would have to be 512-bit wide.
We should support iterative filling in direct-mapped caches because -

We are not fully parameterizable until we do.
It has a large area overhead which is not conducive to creating a Tiny BlackParrot.

One possible fix could be to bank the data memories to have 8 banks of 64-bit widths instead of 1 bank with 512-bit width. Modifying the banking scheme this way also has the benefit of creating square memories which have good area and power properties.

Disagree on the shape of the rams being better for 64x64, they’re pretty well optimized at 512x64. If we really wanted to bank them, we could physically bank the hardened swap rams.
I agree that it doesn’t make much sense to have a 512-bit cache line with a direct-mapped cache. Parameterization is good for supporting different use-cases, but parameterization for parameterization’s sake just makes it harder to maintain. Shrinking the line size and matching the fill size is better for TinyParrot, no? That would minimize the number of data registers in the cache.
There’s a prefetching argument for keeping to say, 128-bit lines rather than 64. But that’s obviously workload dependent and you could effectively do the same thing with a simple next-line prefetcher.
In general, I don’t hate the idea. It would require modifying the Cache Engine Interface. But the hardware cost of changing the fill mask to a sub-bank mask is not terrible, assuming we enforce alignment of sub-banks.

Disagree on the shape of the rams being better for 64x64, they’re pretty well optimized at 512x64. If we really wanted to bank them, we could physically bank the hardened swap rams.
I agree that it doesn’t make much sense to have a 512-bit cache line with a direct-mapped cache. Parameterization is good for supporting different use-cases, but parameterization for parameterization’s sake just makes it harder to maintain. Shrinking the line size and matching the fill size is better for TinyParrot, no? That would minimize the number of data registers in the cache.
There’s a prefetching argument for keeping to say, 128-bit lines rather than 64. But that’s obviously workload dependent and you could effectively do the same thing with a simple next-line prefetcher.
In general, I don’t hate the idea. It would require modifying the Cache Engine Interface. But the hardware cost of changing the fill mask to a sub-bank mask is not terrible, assuming we enforce alignment of sub-banks.

Shrinking the line size and matching to fill size would be fine from an area perspective (provided we physically depth-bank the tall memory if we still want a 4KB cache) but will have a large performance hit based on workload. Given that the tiny core is expected to be Linux capable, can't we assume the workloads might be such that a fill width = line width cache makes the core terrible? (Not sure what other tiny linux capable cores have...)
As I see it now, we are restricted to have bank widths to be atleast fill width size and having sub-block filling would be pretty useful if we want to take full advantage of the stream and burst protocols right?

Shrinking the line size and matching to fill size would be fine from an area perspective (provided we physically depth-bank the tall memory if we still want a 4KB cache) but will have a large performance hit based on workload. Given that the tiny core is expected to be Linux capable, can't we assume the workloads might be such that a fill width = line width cache makes the core terrible? (Not sure what other tiny linux capable cores have...)
As I see it now, we are restricted to have bank widths to be atleast fill width size and having sub-block filling would be pretty useful if we want to take full advantage of the stream and burst protocols right?

Shrinking the line size and matching to fill size would be fine from an area perspective

Yeah, the exercise is to make the tiniest BlackParrot possible. Performance tuning comes after we profile the workload.
Analysis of VM-heavy workloads is tricky (and we don't have any set up, other than boot which is not representative). In many workloads, most of the time is spent in PTW, which is fully determined by how much of the PT is resident in the cache (since it's traversing a linked list and has no locality). Having twice of big of a line size at absolute most give you 2x fewer cache misses (in reality it's much smaller, especially for data cache).
Note that doing partial fill also slows down the fill, writeback and notably coherence operations by a several cycles apiece. This is most likely not noticeable in when attached to a slow DRAM device, but with an L2 can be a significant hit to latency-sensitive workloads.

As I see it now, we are restricted to have bank widths to be atleast fill width size and having sub-block filling would be pretty useful if we want to take full advantage of the stream and burst protocols right?

There's not really much point to having a width smaller than your L2 cache word size, or absent L2, your data bus size. For our chips, we usually serialize to off-chip so the size is mostly irrelevant. The L2 is comfortable at 64-bits, but is still pretty efficient at 32-bits.  For attaching to preexisting IP, AXI-buses typically support 64 bits, although if only AXI-lite IP is available, it may be necessary to downshift to 32 bits. I'm not sure what Wishbone buses support.
So, I think it's a fine change in the context of supporting additional off-chip buses. I don't think that it's necessary for analysis of TinyParrot.

Shrinking the line size and matching to fill size would be fine from an area perspective

Yeah, the exercise is to make the tiniest BlackParrot possible. Performance tuning comes after we profile the workload.
Analysis of VM-heavy workloads is tricky (and we don't have any set up, other than boot which is not representative). In many workloads, most of the time is spent in PTW, which is fully determined by how much of the PT is resident in the cache (since it's traversing a linked list and has no locality). Having twice of big of a line size at absolute most give you 2x fewer cache misses (in reality it's much smaller, especially for data cache).
Note that doing partial fill also slows down the fill, writeback and notably coherence operations by a several cycles apiece. This is most likely not noticeable in when attached to a slow DRAM device, but with an L2 can be a significant hit to latency-sensitive workloads.

As I see it now, we are restricted to have bank widths to be atleast fill width size and having sub-block filling would be pretty useful if we want to take full advantage of the stream and burst protocols right?

There's not really much point to having a width smaller than your L2 cache word size, or absent L2, your data bus size. For our chips, we usually serialize to off-chip so the size is mostly irrelevant. The L2 is comfortable at 64-bits, but is still pretty efficient at 32-bits.  For attaching to preexisting IP, AXI-buses typically support 64 bits, although if only AXI-lite IP is available, it may be necessary to downshift to 32 bits. I'm not sure what Wishbone buses support.
So, I think it's a fine change in the context of supporting additional off-chip buses. I don't think that it's necessary for analysis of TinyParrot.


Add a wiki: Russian Minecraft Wiki

Link to origin wiki: https://minecraft.fandom.com/ru
Link to destination (independent) wiki: https://ru.minecraft.wiki

Thank you for the submission! Added in v3.2.0 (now available on Chrome, and should be available on Firefox within a few days after review).


ability to clone Tries?

It looks like this isn't possible at the moment? Would really appreciate it if Trie implemented Clone

Is this possible? How do you update a tree after it has been built?

Trie implements clone now in v0.3.0.


fix: repo dev dependencies of cli components

Description of changes

The recent release of CLI has major version bump for its component packages, which codegen repo has a dev dependency. Bump the dependency to next major version so that the amplify-dev will not break
Related packages:

amplify-cli-core: 1.x to 2.x
graphql-transformer-core: 6.x to 7.x

Issue #, if available

Description of how you validated changes
Checklist

[x] PR description included
[x] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] Breaking changes to existing customers are released behind a feature flag or major version update
[ ] Changes are tested using sample applications for all relevant platforms (iOS/android/flutter/Javascript) that use the feature added/modified

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #304 (11940c7) into master (a9ea050) will decrease coverage by 1.63%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #304      +/-   ##
==========================================
- Coverage   85.23%   83.59%   -1.64%
==========================================
  Files         146      146
  Lines        6798     5827     -971
  Branches     1715     1212     -503
==========================================
- Hits         5794     4871     -923
+ Misses        913      844      -69
- Partials       91      112      +21

Impacted Files
Coverage Δ

packages/appsync-modelgen-plugin/src/utils/sort.ts
75.00% <0.00%> (-25.00%)
:arrow_down:

...l-docs-generator/src/generator/utils/isRequired.ts
57.14% <0.00%> (-14.29%)
:arrow_down:

...ugin/src/languages/typescript-declaration-block.ts
59.82% <0.00%> (-12.78%)
:arrow_down:

...nc-modelgen-plugin/src/utils/process-belongs-to.ts
76.92% <0.00%> (-11.97%)
:arrow_down:

...phql-docs-generator/src/generator/utils/loading.ts
79.16% <0.00%> (-9.73%)
:arrow_down:

.../graphql-docs-generator/src/generator/getFields.ts
68.42% <0.00%> (-9.36%)
:arrow_down:

...delgen-plugin/src/__tests__/utils/validate-java.ts
63.63% <0.00%> (-9.10%)
:arrow_down:

...rc/compiler/visitors/collectFragmentsReferenced.ts
85.71% <0.00%> (-8.74%)
:arrow_down:

packages/graphql-docs-generator/src/index.ts
76.78% <0.00%> (-8.70%)
:arrow_down:

...rator/src/compiler/visitors/generateOperationId.ts
85.71% <0.00%> (-8.04%)
:arrow_down:

... and 107 more

Continue to review full report at Codecov.

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

so that the amplify-dev will not break

Did this package not break based on these as well? If not they may not be proper dev deps.

so that the amplify-dev will not break

Did this package not break based on these as well? If not they may not be proper dev deps.

Only dev deps but not for production


Preview doesn't match with output image

Temporally fixed for iPad Pro 12.9, by increasing size of the ImageView


Frend.co features

[ ] Spec out new features for the component page - stats, revisions, history etc.
[x] Rework CSS styles
[x] Implement CriticalCSS to improve our PageSpeed ranking.
[ ] Rewrite component docs and about page, improve the consistency between components.
[ ] Look into writing a post about Frend, explaining the reasons behind the creation of it, how it works, where we see it heading in the future. Host on site or external?
[ ] SSL/Service Worker
[x] Improve build process in Gulp
[x] Add CSS linting
[ ] Improve code 'link buttons' contrast to meet a11y

Linting & Gulp implemented in #55

Reworked CSS in #56

Critical CSS (but not really) in #58

For the new features on the component page we can access commit data through the API and pass a path to get the relevant component - ?path=_components/accordion
Looks like we're going to need that SSL to retrieve this data...

SSL YO #89


Imlement TextContainerWithBackgroundImage component

Description
create a text container with background image in the fourth theme. So that user can see the text more clearly with the background image because of the overlay.
Changes proposed in the pull request

Implement text container with background image component
Implement the story for the component
Create documentation for the component

Screenshots (In case of UI changes):

Types of changes

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests have passed and it does not give any unexpected error for the same.

Steps to test

npm i
npm run storybook (story will be displayed in HotelManagementSystem/TextContainerWithBackgroundImage )

Resolves https://github.com/scorelab/Webiu/issues/339

@dinithminura can you review the PR please :)


Flutter module missing x86_64 library

Even though Flutter supports x86_64 on Android since ~2019, the APK is missing the respective native library and as such, the Activity crashes with a dlopen error on such an architecture (e.g. while running on an emulator).

Thanks for letting me know! I'll test this out on an x86_64 emulator. I believe there's an entry that needs to be added to the gradle build.config.

Hi! I think I fixed this by specifically including the x86_64 library. Please let me know if the fix worked, thank you!
ndk {
                // Filter for architectures supported by Flutter.
                abiFilters 'armeabi-v7a', 'arm64-v8a', 'x86_64'
            }

Closed issue as fixed.


How to get the mouse click on the curve event

Hi，swharden！
I want to judge which curve the mouse is currently clicking after the mouse clicks on the curve. I obtain the data of the current mouse click through the GetMouseCoordinates method and judge with the bound data. However, this method must ensure that the mouse clicks on the point on the curve, and it will be invalid when the mouse clicks between two points on the curve. I want to be able to click on any drawn part of the curve, so I know which curve I'm currently clicking on.

Looking forward to your reply！

Hi @541728305, it sounds like you are doing everything correctly! Users interested in achieving a similar effect may find useful information at this FAQ page:

https://scottplot.net/faq/mouse-position/

However, you are correct that ScottPlot does not currently have any logic which helps detect if the cursor is over a connecting line between two data points. That calculation could be costly because it has to iterate across all data points every time the mouse moves a pixel. Users could implement this themselves (and it sounds like you're on the right track for this), but you're correct in observing there's no built-in way to select a curve by hovering over it.
Hope it helps!
Scott

Hi @541728305, it sounds like you are doing everything correctly! Users interested in achieving a similar effect may find useful information at this FAQ page:

https://scottplot.net/faq/mouse-position/

However, you are correct that ScottPlot does not currently have any logic which helps detect if the cursor is over a connecting line between two data points. That calculation could be costly because it has to iterate across all data points every time the mouse moves a pixel. Users could implement this themselves (and it sounds like you're on the right track for this), but you're correct in observing there's no built-in way to select a curve by hovering over it.
Hope it helps! Scott

Thank you very much for your reply


obsolete secondary acute transverse myelitis

close #6934

@twhetzel could you please review this when you have a chance? Thanks!


YARRRML - Exception while using grel:string_indexOf function

Hello,
I am working on a case study where I need to do some processing to generate RDF subjects. More precisely, I have the following JSON data source:
{
    "people": [
        {
            "name": "John_Doe"
        },
        {
            "name": "Jane_Smith"
        },
        {
            "name": "Sara_Bladinck"
        }
    ]
}

I have to generate the subjects using the name element value, but I must only use the left part of it when splitting by the _ character. The expected output should be the following:
ex:John rdf:type ex:Person .
ex:Jane rdf:type ex:Person .
ex:Sara rdf:type ex:Person .

I have written the following mappings file:
prefixes:
 ex: "http://example.com/"
 grel: "http://users.ugent.be/~bjdmeest/function/grel.ttl#"

mappings:
  person:
    sources:
      - ['people.json~jsonpath', '$.people[*]']
    s:
      function: grel:array_join
      parameters:
        - parameter: grel:p_array_a
          value: "http://example.com/"
        - parameter: grel:p_array_a
          value:
            function: grel:string_substring
            parameters:
              - [grel:valueParameter, $(name)]
              - [grel:param_int_i_from, "0"]
              - parameter: grel:param_int_i_opt_to
                value:
                  function: grel:string_indexOf
                  parameters:
                    - [grel:valueParameter, $(name)]
                    - [grel:string_sub, "_"]
    po:
      - [a, ex:Person]

With Matey this works as expected, but with Morph it doesn't. I am getting these exceptions:
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "/usr/lib/python3.10/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/usr/lib/python3.10/multiprocessing/pool.py", line 51, in starmapstar
    return list(itertools.starmap(args[0], args[1]))
  File "/home/user/.local/lib/python3.10/site-packages/morph_kgc/materializer.py", line 428, in _materialize_mapping_group_to_file
    data = _materialize_rml_rule(rml_rule, rml_df, fnml_df, config)
  File "/home/user/.local/lib/python3.10/site-packages/morph_kgc/materializer.py", line 391, in _materialize_rml_rule
    data = _materialize_rml_rule_terms(data, rml_rule, fnml_df, config)
  File "/home/user/.local/lib/python3.10/site-packages/morph_kgc/materializer.py", line 274, in _materialize_rml_rule_terms
    results_df = _materialize_fnml_execution(results_df, rml_rule['subject_map_value'], fnml_df, config, 'subject', termtype=rml_rule['subject_termtype'])
  File "/home/user/.local/lib/python3.10/site-packages/morph_kgc/materializer.py", line 190, in _materialize_fnml_execution
    results_df = execute_fnml(results_df, fnml_df, fnml_execution, config)
  File "/home/user/.local/lib/python3.10/site-packages/morph_kgc/fnml/fnml_executer.py", line 76, in execute_fnml
    data = execute_fnml(data, fnml_df, execution_rule['value_map_value'], config)
  File "/home/user/.local/lib/python3.10/site-packages/morph_kgc/fnml/fnml_executer.py", line 76, in execute_fnml
    data = execute_fnml(data, fnml_df, execution_rule['value_map_value'], config)
  File "/home/user/.local/lib/python3.10/site-packages/morph_kgc/fnml/fnml_executer.py", line 89, in execute_fnml
    function = udf_dict[function_id]['function']
KeyError: 'http://users.ugent.be/~bjdmeest/function/grel.ttl#string_indexOf'
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/user/.local/lib/python3.10/site-packages/morph_kgc/__main__.py", line 47, in <module>
    num_triples = sum(pool.starmap(_materialize_mapping_group_to_file,
  File "/usr/lib/python3.10/multiprocessing/pool.py", line 375, in starmap
    return self._map_async(func, iterable, starmapstar, chunksize).get()
  File "/usr/lib/python3.10/multiprocessing/pool.py", line 774, in get
    raise self._value
KeyError: 'http://users.ugent.be/~bjdmeest/function/grel.ttl#string_indexOf'

Do you have any idea about what might be wrong? Thanks a lot.

Thanks. Your are right, ideally all GREL functions would be implemented as built-in functions, but currently we just support a subset of them. I think that taking as reference other GREL function that are already implemented, adding new ones is simple 🙂.


Allow for custom styles

Not sure how the maintainer wants to handle adjusting styles, so I think at the very least accepting a style prop for overwriting styles is good enough... I'd be curious to hear thoughts on this?
Anyways.. just like others who are asking for this feature, I need it! 😃
#37
#52

@isaaccodekill can we get this merged and deployed? There are many issues that this would resolve!


fix(build): patch ngx_lua-1.21.4_07-print-body-double-free bug fix

Summary
This is the backport Openresty patch for https://github.com/openresty/lua-nginx-module/pull/2115
Related: https://github.com/openresty/lua-nginx-module/issues/2184

Checklist

[x] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

FTI-5051

The patch file should start with ngx_lua-0.10.21?

Yes, in OpenResty 1.21.41 ngx_lua is 0.10.21

I tested it locally on mac and it works.


chore: Make settings.settings copyable

Fixes #
Description

Make settings copyable
This change is needed so that we can deep-copy the default settings when we inject them into the context

How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

15 of 17 (88.24%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 79.321%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/apis/settings/zz_generated.deepcopy.go
10
12
83.33%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
6402

Relevant Lines:
8071

💛  - Coveralls


Update sbt-sonatype to 3.9.20

About this PR
📦 Updates org.xerial.sbt:sbt-sonatype from 3.9.5 to 3.9.20
📜 GitHub Release Notes - Release Notes
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" }
}]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #70.


Why builtin chat engine is not being used?

In The Given project is there any specific reason for using custom chat engine by building a chat engine out of query engine and AnyIO memory stream  when there is in built chat engine

@Sai-Rohith-Bobba thanks for the great question!
If you look at the code here in app/chat/engine.py you'll see that we do use the OpenAIAgent class from the LlamaIndex framework which is itself an implementation of a chat engine. The code in app/api/endpoints/conversation.py that creates the anyio channel simply takes the callback events & response tokens from that OpenAIAgent chat engine and sends them to the frontend over the SSE protocol.


Use of IAM User

Many companies won't allow IAM Users to be generated at all. In fact, AWS recommends not to use IAM Users, but to rather use tokens. This can easily end up being a security show-stopper for a Quick Start deploy.

Right now we have two very big Finance customers for that we need to deploy IBM Guardium Insights on AWS and the IAM user creation ( as AWS recommendation ) to not use IAM User rather than Roles and Policy. Please advise what can be done to fix that ASAP.


RStudio: Versioned

See: Trello1, Trello2
Image: https://quay.io/repository/mojanalytics/rstudio?tab=tags
R version: 3.4.2
Modified base image to build from tagged digest for reproducibility. Note we
will not get auto updates to RStudio as a result of this.
Shifted R packages to separate file for readability
Install pinned dependency: PhantomJS from apt (Could not get Webdriver install to
work)
Install R package rGL dependencies mesa-common-dev, libglu1-mesa-dev
Modified R package installs to use install2.r wrapper (It's newer and
used mostly by rocker)
Added CHANGELOG and gitignore
Test

docker container run -d --rm -p 8787:8787 quay.io/mojanalytics/rstudio:v2.0.0

Browse to rstudio

Hi @RobinL,
Going by the comment in the original Dockerfile, I thought they were for setting up rstudio specifically.  Rather than dependencies for users?
I'll look at the base image's heritage for any of the packages mentioned but it might be a good idea to fix forward in this situation.

The following packages will be added to this image:
    rrdtool
    texinfo
    texlive
    texlive-latex-extra
    bzip2
    libglpk-dev
    libpoppler-cpp-dev
    libgeos-dev
    libgdal-dev
    libproj-dev
    lmodern
    curl


Current Bugs

Cannot detect the Level Up screen and freezes
Won't start game on some maps on Easy mode, it is reading 1/40 as "THO" in debug
It is skipping some insctructions, EG: on default chimps instructions, it completely skips R56 and 59 upgrading instructions and then continues with with R61 working fine, and no logs in debug, doesn't attempt to upgrade towers.
Sometimes when the game starts it opens the ESC menu before anything else and breaks the bot, this seems to be random and i cannot repeat it again, but my friend is/was having the same issue. EDIT: It started happening again, i found a temp fix and changed line 408 from time.sleep(0.2) to time.sleep(1). But it's probably not a good fix.

are you using the main branch? If so try the 0.9 release, I didn't have any of the issues you mentioned so far with it.
I have done hundreds of runs with just few rare losses...

yeah im using the latest main, i'll try the 0.9

Okay i pretty much fixed everything, the ocr size radius was messing with some rounds. I just forced the bot to use default round scaling instead of using round_area.png.
As for the leveling up, someone made a new png which fixes that.
And the ESC menu is 99% fixed i just told this self.press_key("esc", timeout=1) to wait 5 seconds instead.
These are probably not good fixes, but it's working fine for me.
Some people are having the same issues where others are having none, i really don't know anything about coding, but maybe system specs or resolution is causing this.


Add Endurance.app latest version

No matter how good your battery is, everyone could use a little more battery power! http://enduranceapp.com/

Thank you for the contribution. It was merged directly as 61ddc60f861d114763a4131fc2872e3f17ee871a to keep commit history cleaner. Your contribution is still credited to you.


🐛 Fixed URL used to fetch Fathom Analytics sites

Description
Fathom Analytics changed their API to use a different URL to fetch sites. A community member reached out to me and asked me to fix it (gotta love OSS).
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @DanielFerguson 👋
Can you split this into two updates since our system can only handle one at a time, otherwise something goes wrong 🙂

Doh! 🤦♂ Absolutely. And thank you for fixing the linting - I thought I remembered to do that pre-commit, but I must have forgotten. I appreciate it. I’ll split the push tomorrow. 😊


How many gpus do you use for aishell dataset?

Hi,  I realize the accum_grad_n_step argument is set to 8 in transformer.yaml, I want to know that how many gpus do you use when you train models for this setting.  Should I change it to 2 when I use 4 gpus to reproduce your results?

Hi @cywang97, I usually use a single GPU on AISHELL-1:)
In general, a good rule of thumb is to reduce accumulation when using multiple GPUs to keep total mini-batch size at a single parameter update step.
Maybe accum8 + 4GPUs leads to too slow convergence.


Adapt library search to accommodate multiple locations

Windows has legacy DLLs dir and newer Library\bin dir as places
to search for libraries in conda packages.

Cross reference: https://github.com/numba/pyculib/issues/19

Passed CUDA smoke test

Also, looks good to me.  :)


[UI] Filter results by tag

Particularly for large benchmarks, it would be useful to be able to filter the list of detailed results based on the result/tag of the run.

probable approach:

pagination for the full results (allows for super fast loading), based on sqlite offset and limit and query parameters containing ?offset
also display a search bar on the paginated full results page, to add filtering on top of that.


[BUG] KeyError: 'shortcuts'

Basic information

Program version: 0.0.8, updated to 0.1.0 in the background while running.
Python version: 3.9.5
Operating system: Arch Linux, kernel 5.13rc5

Describe the bug
A fatal KeyError exception occured while downloading from an airing livestream.
Command/Code used
If running from the command line, provide the following:

The command used (including the verbose tag, -v):

chat_downloader -o "${outname?}".json --message_groups "all" --indent 2 -- "${url?}"

Output from the above command (source doesn't match the running program):

Traceback (most recent call last):
  File "$HOME/.local/bin/chat_downloader", line 8, in <module>
    sys.exit(main())
  File "$HOME/.local/lib/python3.9/site-packages/chat_downloader/cli.py", line 170, in main

  File "$HOME/.local/lib/python3.9/site-packages/chat_downloader/chat_downloader.py", line 297, in run
    output, indent=indent, sort_keys=sort_keys, overwrite=overwrite)
  File "$HOME/.local/lib/python3.9/site-packages/chat_downloader/sites/common.py", line 60, in __iter__
    class Remapper():
  File "$HOME/.local/lib/python3.9/site-packages/chat_downloader/sites/youtube.py", line 1141, in _get_chat_messages
    def _get_chat_messages(self, initial_info, params):
  File "$HOME/.local/lib/python3.9/site-packages/chat_downloader/sites/youtube.py", line 412, in _parse_item

  File "$HOME/.local/lib/python3.9/site-packages/chat_downloader/sites/common.py", line 280, in remap
    _TESTS = [
  File "$HOME/.local/lib/python3.9/site-packages/chat_downloader/sites/youtube.py", line 377, in parse_runs
    index = url.find('=')
KeyError: 'shortcuts'

Expected behavior
I expected the downloader to finish normally.
Additional context/information
It may be related to the KeyError: isCustomEmoji issue as I had personally replaced (just) that field with an instance of .get(). Furthermore, it seems there might be another chance of failure, with name = emoji['shortcuts'][0] just a bit earlier (which matches the line number given when using the 0.0.8 source).

Sure thing let me check this out. Can you provide the URL you tested with?

It's still live for now: https://www.youtube.com/watch?v=b2B4hC6Dh6w

Okay thanks! I'll run it and try to replicate the bug. Either way, it would be an easy fix to just use a try_get on the emoji item, so I will add that.
Although, it would be nice to see what message caused that. From similar issues in the past, this anomaly occurs when the YouTube API returns an incomplete item.
Will keep you updated 👍

I fixed this in the latest commit, and I will push the changes to the latest pip version (i.e., make a new release) once I add a fix for the consent issue (#85 and #94).


Unable to open Gramnetwork.org

We have detected this domain to have malicious intent and have prevented you from interacting with it.
Good day!
I just like to ask. Why is it I can't access gramnetwork.org? It says"We have detected this domain to have malicious intent and have prevented you from interacting with it."
How am I able to get through it?
EAL Warning.pdf

Gramnetwork.org is a phishing/scam site for a Telegram public crowdsale.
There has not been an official announced public sale of Telegram yet.


Feat/litecoin multisig

Description of the change
Add multi-signature payments to coinlib-litecoin. It is similar to the one in coinlib-bitcoin, allowing the user to lock their litecoin into a combined account, and they have to provide enough signatures before transferring the litecoin out.
Add MultiInputTransaction support to coinlib-litecoin.
Fixed a problem in BitcoinishPayments: createMultiInputTransaction when setting the option useAllUtxos to true.

move to new pr "Feat/bitcoinish altcoin multisig"


Public Release v2.3.0

General / Features

Typescript migration
Collectibles Transactions + Add WalletLink Support
Adds copy to clipboard in owners table
Deployment screen improvements
Hide transactions with no confirmations, labels/notifications sync
Safe creation tx details

Fixes

[Contract Interaction] Can't input "false" in a bool while doing contract interactions
[Contract Interaction] Encoded data isn't updated when changing contract interaction input after review
[Contract Interaction] Arguments as array
[Contract Interaction] No ENS resolution for contract address
[Safe Apps] Apps load outside of the container
[Balance Improvements] Fiat balances shown as 0

Safe Apps

Allow to load Safe Apps with ENS names

Desktop App

Update Dialog v2
Logo Background
After Ledger is connected, not able to reconnecting without Ledger being plugged in
Desktop app won't connect to Ledger Nano X
Trezor support

@francovenica @lukasschor I've just created this issue to keep track of v2.3.0 release. moving forward, we could start using releases. but that may required a deeper discussion. so let's use this issue or v.2.3.0,

I assume this is used for QA? I tried to manually gather and consolidate the areas where special attention should be paid during testing. Added the list to the ticket description. However, this is not a 1:1 changelog and not release notes. :)

right, this is for QA and not supposed to be a changelog

Typescript migration (Can't verify, but the guys were working on it all week so is there)
Collectibles Transactions + Add WalletLink Support (done)
Adds copy to clipboard in owners table (done)
Hide transactions with no confirmations, labels/notifications sync (done)
Safe creation tx details (done)
Fixes
[Contract Interaction] Can't input "false" in a bool while doing contract interactions (done)
[Contract Interaction] Encoded data isn't updated when changing contract interaction input after review (done)
[Contract Interaction] Arguments as array (done)
[Contract Interaction] No ENS resolution for contract address (done)
[Safe Apps] Apps load outside of the container (done)
[Balance Improvements] Fiat balances shown as 0 (done)
Safe Apps
Allow to load Safe Apps with ENS names (done)
Desktop App (didn't know we were going to do a new release of this)
After Ledger is connected, not able to reconnecting without Ledger being plugged in (done)
Desktop app won't connect to Ledger Nano X (I know Tobias checked this after it was fixed)
Trezor support (done)
Also check the site in general
Safe loading, safe creationg, send funds, collectibles, transactions in general, Adding/removing owners, req confirmations, Apps (except Request because it doesn't work in low env, maybe because of rinkeby).
Tested with trezor, ledger, MM chrome, MM wallet connect, Walletlink
All these wen't well
@lukasschor This are points that I don't know what they are referring to
Deployment screen improvements
Update Dialog v2
Logo Background

Update Dialog and Logo Background are related to the Desktop App, and as we don't plan to release a Desktop app right now, we don't need to put much focus on this. Deployment screen improvements is about this ticket / this PR

Gotcha. Thanks for that


Order model improvements proposal

In my opinion Order model should have total field with the default store currency value and we should replace anonymous_user_email field by just email (always required filled) and maybe with property is_anonymous (user is None).
Now we don't have possibility to filter, analysis orders by total because get_total is complicated function working on iterator not accessible from database. Mailing campaign and simple sorting will be more simple with email approach.

Yes, we do need to store total at the order level. :+1: on the email change as well. Not sure we need a separate property for checking whether order is anonymous or not. Checking the user should be enough.

Looks like we have it implemented now.


fix nullpointer issue #410

Additions
None.
Changes
Flip the statement in filter so null reference does not occur.
Backwards incompatible changes
None.

Thank you! I'm a little embarassed I've looked over this...


Added support for public/static folder cache control

Fixes #392
Adds support for setting the Cache-Control headers on /public or /static folder assets.
Format:
myNextApplication:
  component: serverless-next.js
  inputs:
    publicAssetCache: true

This uses the default of Cache-Control=public, max-age=31536000, must-revalidate(not sure if must-revalidate is even needed) for most common image formats: /\.(gif|jpe?g|jp2|tiff|png|webp|bmp|svg)$/i.
This can be customized by specifying value and/or test. If one is missing, it will use the default value:
myNextApplication:
  component: serverless-next.js
  inputs:
    publicAssetCache:
      value: public, max-age=604800
      test: /\.(gif|jpe?g|png|txt|xml)$/i

Update:
Based on feedback, Cache-Control is applied by default to the public folder. Setting it to false disables this, using an object allows for configuration:
myNextApplication:
  component: serverless-next.js
  inputs:
    publicDirectoryCache:
      value: public, max-age=604800
      test: /\.(gif|jpe?g|png|txt|xml)$/i

myNextApplication:
  component: serverless-next.js
  inputs:
    publicDirectoryCache: false

Tested with both the default settings:

And using custom settings:

@DanielOrtel Great work thank you 🙏
One minor thing, rather than calling it publicAssetCache could you call it publicDirectoryCache. I think that will be more familiar to next.js devs as is officially named the public directory.

Also, may be worth a test at the serverless-component/ package level to test the integration with s3-static-assets. https://github.com/danielcondemarin/serverless-next.js/tree/master/packages/serverless-component/tests

@danielcondemarin I've pushed a new commit addressing the feedback. I'll squash that one before merging.

@DanielOrtel Thanks for addressing all the feedback and quick turnaround.
I'll merge and publish in a bit 🚀

One minor thing: the Readme of packages/serverless-component was updated but I failed to notice that this is mirrored in the main Readme, that should probably get updated as well.

One minor thing: the Readme of packages/serverless-component was updated but I failed to notice that this is mirrored in the main Readme, that should probably get updated as well.

Thanks for the heads up. I'll remove the one from packages/serverless-component I think and just keep the main one going forwards. Keeping them in sync is a bit of a pain.


Link to the "contributor guidelines" doesn't work

Describe the bug
In the README.md file the link to the "contributor guidelines" doesn't work
https://github.com/orchest/orchest/blob/master/README.md
Expected behavior
the link should work
To Reproduce
Go to README.md and click "contributor guides" link ( see attached screenshots)
Screenshots

Environment
not relevant

Good catch! Fixed it https://github.com/orchest/orchest/commit/49b272da6d68a14cd6e057048291c8a736f963a9


Delete structs in DB when deleted in State

Fixing Issue 306
The Problem
As seen in #306 , renaming and deleting of structs does not actually get rid of the toml file which Binsync uses to track structs.
The Solution
In order to make this somewhat generic, I modified the following files;

binsync/core/client.py
binsync/data/state.py

In state.py, I introduce the deleted_artifacts struct to keep track of any artifact that we want to remove.
https://github.com/binsync/binsync/blob/2af42bd41049c0052863f8398bd0eb2ac2be3c99/binsync/data/state.py#L189
An example of this being used is in the function set_struct which will add an entry like so:
https://github.com/binsync/binsync/blob/2af42bd41049c0052863f8398bd0eb2ac2be3c99/binsync/data/state.py#L505-L516
This then allows us to iterate over the list later on and explicitly all _delete_data from the dump function:
https://github.com/binsync/binsync/blob/2af42bd41049c0052863f8398bd0eb2ac2be3c99/binsync/data/state.py#L289-L301
_delete_data follows the same method as _dump_data but instead will make use of the remove_data function in self.client.
The last thing added was in the client.py file:
https://github.com/binsync/binsync/blob/2af42bd41049c0052863f8398bd0eb2ac2be3c99/binsync/core/client.py#L289
This allows us to ignore trying to pass around deleted files or added files and can just update the current folder with add.  Then when the commit is sent off this will make git also remove the reference so we don't need to manually do repo.index.remove.
Other Changes
I also updated binsync/decompilers/ida/hooks.py to properly pass old_name for the previous name of the changed struct.
TODO
The UI table in QT still has the reference for the previous struct for some reason. Despite it being deleted in the folder, it still is present in the table.

@Pascal-0x90 I'm only getting to reviewing this now. Is this ready for review? I.e. did you test it once and confirm it works as you think it would? Or does it still need dev?

Hey @mahaloz it WOMM for what I described so I would say this is ready for review / testing


Feature request: ForAllTypesContainsNamespace

🚀Feature request
Description: A Method like ForAllTypesInNamespace, but that allows to include all types that belong to a namespace that contains a supperior hierarchy.
Type: Enhancement
Is your proposal related to a problem?
Maybe related with:
#306
Describe the solution you'd like
Namespaces structure:

Pocos
Pocos\PocosAccounting
Pocos\PocosOrders

Method:
ForAllTypesContainsNamespace(ASSEMBLY, "Sample.Pocos");
This method would include all types inside the sub namespaces.
Describe alternatives you've considered
This is possible with ForTypes method and Linq filtering:
ForTypes(typeof(PocoInvoice).Assembly.GetTypes().Where(....).ToArray());
Additional Ideas
This method is too close to this requested functionality. It would be better that have more options.

Cool idea, thank you for suggestion.

thanks @chaowlert for all the fixes and features!


Feature: Fetch existing internal links during editing

Environment
Host

item
version

OS
Ubuntu 18.04

GROWI
3.7.4

node.js
12.16.2

npm
6.14.4

Using Docker
yes

Using growi-docker-compose
yes

(Accessing https://{GROWI_HOST}/admin helps you to fill in above versions)
What is the expected result? (期待される動作)
In the editor, we add an internal by using a syntax such as this: [[this is title>../samplepage]]. However, this can get complicated when we have to add a link which is very far from the current page in hierarchy. A desired workflow would be that a pop-up opens when adding an internal link which searches for existing pages (much like the one that opens when we click on the New button on the top menu.
Here is a demo:

@shivams
Thank you for your request.
In fact, we were talking about a feature within the dev team that was similar to the idea you came up with. (Sorry, the minutes of a meeting is written in Japanese)
/GROWI村議議事録/20200227
https://dev.growi.org/5e5751ad1337da0048e5fc57
However, I think it will be a while before we start considering it. Please wait a moment. Thanks.

@shivams
Thank you for your wonderful request.
The ability to edit links has been released at v4.2.3.
Please see here for the operation.
https://twitter.com/weseek_inc/status/1338496274677510147
Please create an issue if you have other requests.
This issue will be closed.

Wow!! Thanks @itizawa I will check it out. Very happy to hear this.

Wow!! Thanks @itizawa I will check it out. Very happy to hear this.


Transfer koa context to validation rule method

Validation rule method can do more thing by accessing full the request info

I suppose this could be useful some time, but I'm pretty concerned that this could be a breaking change. Is there a strong need for this? Example?

I'm pretty concerned that this could be a breaking change

I think it doesn't break others because the extra parameter is in last location

Is there a strong need for this? Example?

Yes, let's say a request wants to create an object Group but it requires the group name unique under current user tenant. So we want to validate whether the input groupd name on request body existed under the tenant or not. It requires the validation method to see who is on the request (on Authorization header or on a ctx.state.user populated by another middleware).

Please update README. Also bump semver to 1.0.0 because this is a breaking change (yes, it would be rare, but since this change the argument list of call sites, there's always a possibility of something gets broken.)

@buunguyen Thanks, addressed the comment


Update Zico's PPP

Codecov Report
Base: 60.11% // Head: 60.11% // No change to project coverage :thumbsup:

Coverage data is based on head (e370946) compared to base (3be902b).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #256   +/-   ##
=========================================
  Coverage     60.11%   60.11%
  Complexity      595      595
=========================================
  Files           109      109
  Lines          2204     2204
  Branches        267      267
=========================================
  Hits           1325     1325
  Misses          811      811
  Partials         68       68

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Need to add more portfolio templates

I need to add more portfolio templates to the pproject

I have ideas for portfolio Templates.
Please Assign me this task


Adding value_template to weblink

Description:
Adds a template_value to the weblink component
Related issue (if applicable): fixes #18215
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):
weblink:
  entities:
    - name: test
      value_template: "{{ 'https://google.se' if True else 'https://bing.se' }}"

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

Aha! Thank you for clearing this up


warning Two or more files with the name of reply.cpp

I'm working on a Windows build of your library.
The changes to support WIN32 were not bad at all.
However, I'm having this issue:
warning MSB8027: Two or more files with the name of reply.cpp will produce outputs to the same location. This can lead to an incorrect build result.  The files involved are sources\replies\reply.cpp, sources\reply.cpp.
It looks like the 'reply' class exists in both files.
However the top one contains these extra methods:
array_reply& as_array(void);
bulk_string_reply& as_bulk_string(void);
error_reply& as_error(void);
integer_reply& as_integer(void);
simple_string_reply& as_simple_string(void);
The test project needs the extra methods.
Can we unify these so there is only one reply hpp and cpp?
Run a compare tool on the two files so you can see the difference.
Thanks,
Mike M.
P.S Awesome work!  I'm almost ready to run the tests...

Hi,
Thanks for your interest in cpp_redis and for porting the library to Windows! Good job!
Indeed, the reply part is something I have coded a long time ago and by looking again at the implementation this morning, I was really not satisfied with it.
So, I just cleaned this and kept only a single reply.(hpp|cpp) file (the whole replies folder has been removed).
Concerning the tests, I added a bunch of integration tests yesterday as well as configuring Travis. However, I'm still working on a convenient way to run the async integration tests: so don't worry if you see some redis_client/redis_subscriber related tests failing sometimes but succeeding on other runs.. Though all other tests (builders/replies tests) should all pass as they are unit tests.
Thanks


some implementations are different from the original paper

Hi, thanks to your advance code helping me to understand original paper!
Given only global_transformer and local_transformer, I found that your codes are different at some points：
let's set D_G=768, D_L=256, P=4, seq=40,batch_size=1,

then h^global_out shape is (1, 1+10, 768) in your code, but it should be (1, 1+10, 4 * 768) in paper (1+10 means first pad token + following patch tokens). There are two token_emb func in original paper for h^global_in and h^local_in, your code only has a token_emb for h^local_in.
the proj between global and local transformer projects dim 768 to 256, your code is doing this:

h^global_out.shape=(1, 11, 768), choose top10 token, then h^global_out.shape=(1, 10, 768)

h^global_out = proj(h^global_out), then h^global_out.shape=(1, 10, 256)

h^local_in.shape=(10, 4, 256), concat with h^global_out, the final h^local_in has a shape of (10, 5, 256). at the end, drop the first token
in the original paper, it does

h^global_out.shape=(1, 11, 4 * 768), choose top10 token, then h^global_out.shape=(1, 10, 4 * 768)

h^global_out = proj(h^global_out), then h^global_out.shape=(1, 10, 4 * 256)

h^local_in = concat([pad, top3 token in h^local_in]) + h^global_out.reshape(10, 4, 256), which will drop last token in h_local_in.

Please take a look and see if I misunderstand something. Looking forward to your apply!

@ZihaoH
yea you are correct, i didn't do the implementation to the exact detail
number one, this will be negligible difference
i just tried out their #2, but didn't see better results than the way i did it, and code ends up being more complicated. you are welcome to give it a run (or if you can spot a bug, let me know)

@ZihaoH nevermind i found the bug and it converges ok, just released in 0.1.0
their design for number 2 may have slightly better gradients


Histogram views now display rates

Original author: Steven Qiou
Original date: 2018-02-03 18:25:55
Original PR: https://charm.cs.illinois.edu/gerrit/3406

Change-Id: Iafaf460e031c05754a07cb5e0e4fb2f40d4b8e5d

This has already been merged as f4ebff71467fba45520bed832b33a45a669b5273


kv: Remove unused impl of EngineIterator for DBIterator

Signed-off-by: Brian Anderson<EMAIL_ADDRESS>
What problem does this PR solve?
Cleaning up the tikv::storage::kv module to make it eventually generic over engine_traits

cc https://github.com/tikv/tikv/issues/9480
cc https://github.com/tikv/tikv/issues/6402

What is changed and how it works?
This code is unused. Delete it.
Related changes
Check List
Release note

No release note.

/test

/test

/lgtm

/lgtm

/merge

/merge


How to get the name/type of the segment?

Hi,
In the demos it shows, that you can also extract the type of the segment (e.g a door, a human etc.) can it be extracted from the model or do we need an additional library for that?
Cheers

I don't want to duplicate so I'll just wrote I'm also wondering is it possible to get labels or it isn't possible?

This current model does not provide any label, it is a generalised segmentation model
They mention that on the FAQ as well:
Does the model produce mask labels?
No, the model predicts object masks only and does not generate labels.

@krishnaadithya  thanks for the answer and sorry for not seeing it in FAQ.
How did you get the labels classified in your demo:

We offer an automated annotation tool that can support categories in COCO, ADE20K, and open vocabulary annotation. Hopefully this will help you get the category labels for the SA dataset.
Project: Semantic Segment Anything (SSA)
Repo: https://github.com/fudan-zvg/Semantic-Segment-Anything

Wow, nice exactly what I'm looking for. You don't happen to have a Hosted inference API
on Hugging Face?


Dynamic Form to Dev

Q
A

Bug fix?
[ ]

New feature?
[ x ]

New sample?
[ ]

Related issues?
fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
Dynamic form control

Thank you @ravichandran-blog for the PR!
It is a great addition to the library!
The PR has been merged with some additional modifications.
You can review the modifications in the history, but I want to point out some important edits for your future contributions.
Localization
All the texts were hardcoded. We are using standard SPFx localization mechanism to make the controls available on different languages.
Styling
Most of the colors were directly included in scss file. Usually in SPFx solutions it makes sense to use theme variables.
Testing the Functionality
Some of the methods/functions were failing in simple situations.
For example, saveIntoSharePoint in DynamicForm.tsx. It uploads a file using the next request:
let fileresult = await sp.web.getFolderByServerRelativeUrl(this.props.context.pageContext.web.serverRelativeUrl + "/SiteAssets/Lists/" + this.props.listId).files.add(file.fileName, resultcontent, false);

It will fail if SiteAssets/Lists/ListId folder (or any part of it) doesn't exist.
Another example: getLookUpValue in SPService.ts:
const data = await this._context.spHttpClient.get(apiUrl, SPHttpClient.configurations.v1);
if (data.ok) {
  const results = await data.json();
  if (results) {
    return [{ key: results.value[0][fieldName].ID, name: results.value[0][fieldName].Title }];
  }
}

We can pass if (results) condition, but the code will fail if results.value.length === 0 or results.value[0][fieldName] is undefined. And it happens if there is an item but the value for this specific field is not set.
Naming of the Methods
I would recommend to use more clear names for the methods based on what exactly is done.
For example, getListInfo in SPService. It actually requests fields for the list with additional filter based on content type and excluding some additional fields like read-only and hidden. So the name of the method confuses in that case.
Additional Remarks
listFeilds["value"].forEach(async (field) => {
  // do some async stuff
});

// code that should run after all array items are processed

forEach is always synchronous. It will not await for the async callback. You either need to use for loop or other approaches like async-foreach module.
let hiddenname = "";
let TermSetId = "";
let lookupField = "";
let selectedtags: any = [];

You're using different notations for variables names. It makes code harder to read. I would recommend to use camelCase notation everywhere.
Thank you again for the great feature! Hope this code review will be helpful!

Hi @AJIXuMuK,
Many thanks! it is helpful


Bump pipeline version to 4

Currently, string array properties in a pipeline file are just a plain array of strings. However, we need stable IDs for our string/file array form control to safely render/edit values. This is a breaking change, so we need to increment the current pipeline version from 3 to 4.

Could you give an example?

@lresende An example of what?

as per discussion, we won't be adding IDs to list items. Instead we will need to update StringArrayControl to handle plain lists


Corner radius

Hi, setting corner radius applies radius to all four corner of view. I want to have radius only on one side. Is it possible to implement this with carbon???
If yes, then how?? I can't find anything in wiki.

Wiki is pretty outdated, when it comes to details. I don't have enough time to update it.
I started working on it recently. The current version works, but wasn't fully tested. It's in the repo, if you want to play with it. Check out widgets -> buttons.


[BUG] Clicking on games sometimes won't open the card with information

Examples:

Noticed with games 303850 and 391920.
App 67910 is a video; The image is correct, but it shows its name as Farming Simulator 2011 for some reason.
App 488250 is also a video. Log below (from Chrome's console, all of these examples seem to generate similar logs, 4 errors each click):

LogRocket.min.js:1 [Vue warn]: Error in render: "TypeError: Cannot read property 'windows' of null"

(found in <Root>)
e.(anonymous function) @ LogRocket.min.js:1
warn @ bundle.js?v=3-3-23:58966
logError @ bundle.js?v=3-3-23:60178
globalHandleError @ bundle.js?v=3-3-23:60173
handleError @ bundle.js?v=3-3-23:60162
Vue._render @ bundle.js?v=3-3-23:62998
updateComponent @ bundle.js?v=3-3-23:61249
get @ bundle.js?v=3-3-23:61619
run @ bundle.js?v=3-3-23:61694
flushSchedulerQueue @ bundle.js?v=3-3-23:61450
(anonymous) @ bundle.js?v=3-3-23:60279
flushCallbacks @ bundle.js?v=3-3-23:60198
LogRocket.min.js:1 TypeError: Cannot read property 'windows' of null
    at Proxy.eval (eval at createFunction (bundle.js?v=3-3-23:69153), <anonymous>:3:3981)
    at Vue._render (bundle.js?v=3-3-23:62996)
    at Vue.updateComponent (bundle.js?v=3-3-23:61249)
    at Watcher.get (bundle.js?v=3-3-23:61619)
    at Watcher.run (bundle.js?v=3-3-23:61694)
    at flushSchedulerQueue (bundle.js?v=3-3-23:61450)
    at Array.<anonymous> (bundle.js?v=3-3-23:60279)
    at MessagePort.flushCallbacks (bundle.js?v=3-3-23:60198)
e.(anonymous function) @ LogRocket.min.js:1
logError @ bundle.js?v=3-3-23:60182
globalHandleError @ bundle.js?v=3-3-23:60173
handleError @ bundle.js?v=3-3-23:60162
Vue._render @ bundle.js?v=3-3-23:62998
updateComponent @ bundle.js?v=3-3-23:61249
get @ bundle.js?v=3-3-23:61619
run @ bundle.js?v=3-3-23:61694
flushSchedulerQueue @ bundle.js?v=3-3-23:61450
(anonymous) @ bundle.js?v=3-3-23:60279
flushCallbacks @ bundle.js?v=3-3-23:60198
LogRocket.min.js:1 [Vue warn]: Error in event handler for "backlog-modal-current-players": "TypeError: Cannot set property 'current' of null"

(found in <Root>)
e.(anonymous function) @ LogRocket.min.js:1
warn @ bundle.js?v=3-3-23:58966
logError @ bundle.js?v=3-3-23:60178
globalHandleError @ bundle.js?v=3-3-23:60173
handleError @ bundle.js?v=3-3-23:60162
Vue.$emit @ bundle.js?v=3-3-23:61004
./resources/assets/js/vue/vue.backlog-modal.js.window.currentPlayersForModal @ bundle.js?v=3-3-23:72772
(anonymous) @ ?format=json&appid=488250&jsonp=currentPlayersForModal&callback=jQuery33109774839784426799_1553946302770&_=1553946302773:1
LogRocket.min.js:1 TypeError: Cannot set property 'current' of null
    at Vue.<anonymous> (bundle.js?v=3-3-23:72758)
    at Vue.$emit (bundle.js?v=3-3-23:61002)
    at ./resources/assets/js/vue/vue.backlog-modal.js.window.currentPlayersForModal (bundle.js?v=3-3-23:72772)
    at ?format=json&appid=488250&jsonp=currentPlayersForModal&callback=jQuery33109774839784426799_1553946302770&_=1553946302773:1

Hey @MakoSipper
You are completely right. Turns out we were storing more than games and DLC in the database.
After a quick search, steam provides videos, demos, episodes, servers, and other pieces of media.
We are going to clean the database of all this kind of stuff. I stimate a period of three months until everything has been cleared out.
Thanks a lot


Add @daxpedda as web maintainer

Hello @daxpedda,
Given that you've been practically maintaining the web backend for this crate (#76, #77, #87, #101), and that we don't have a web maintainer yet, would you like to be this crate's web maintainer? You seem knowledgeable enough about the web and WASM to fit the bill.
If so, let me know, and we can add you to @rust-windowing/softbuffer.

Absolutely, would love to!

The invite's been sent! Once you've gotten it, feel free to approve/merge this PR :heart:


HTML decoding for Plot

Hi
it will be great if you can add HTML decoding for plot. This is the only field that can have encoded characters:
https://www.themoviedb.org/talk/564ae26bc3a3686031002638
It's no problem to decode it in my code, but i think it will be great to receive all API results in decoded format (like Title is by default).

From what I see, it's the field overview, right?
I wonder how we could do that without depending on something crazy like System.Web.

I can otherwise see the point in ensuring that all fields are "raw"... And not all except a few here and there.

Asked travis if he'll fix it in his end..

Yes, I meant Overview.
Thanks for asking Travis.

@Naliath, reg. what happens if Travis decides to do something. In the talk above, Travis writes:
"Hey Mike,
If there is ever a v4, yes, this would change and be how it gets returned but at this point there's too many clients expecting the data back in the current format. I don't like to make to data changes if I don't have to. It's been this way since 2012 (when v3 launched) so I will leave it be for now."
So .. We should be safe.

@Naliath, reg. what happens if Travis decides to do something. In the talk above, Travis writes:
"Hey Mike,
If there is ever a v4, yes, this would change and be how it gets returned but at this point there's too many clients expecting the data back in the current format. I don't like to make to data changes if I don't have to. It's been this way since 2012 (when v3 launched) so I will leave it be for now."
So .. We should be safe.

Should be fine, worst case is that we would decode already decoded data and might replace characters that should not be replaced. This will only present an issue in very few cases so nothing to worry about.
Date: Thu, 26 Nov 2015 01:23:24 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [TMDbLib] HTML decoding for Plot (#160)
@Naliath, reg. what happens if Travis decides to do something. In the talk above, Travis writes:
"Hey Mike,
If there is ever a v4, yes, this would change and be how it gets returned but at this point there's too many clients expecting the data back in the current format. I don't like to make to data changes if I don't have to. It's been this way since 2012 (when v3 launched) so I will leave it be for now."
So .. We should be safe.
—
Reply to this email directly or view it on GitHub.

Agreed. The moment v4 comes out, if ever, we'll push for unencoded data.
-----Original Message-----
From: "Thomas Vriens"<EMAIL_ADDRESS>Sent: 26-11-2015 11:28
To: "LordMike/TMDbLib"<EMAIL_ADDRESS>Cc: "Michael Bisbjerg"<EMAIL_ADDRESS>Subject: Re: [TMDbLib] HTML decoding for Plot (#160)
Should be fine, worst case is that we would decode already decoded data and might replace characters that should not be replaced. This will only present an issue in very few cases so nothing to worry about.
Date: Thu, 26 Nov 2015 01:23:24 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [TMDbLib] HTML decoding for Plot (#160)
@Naliath, reg. what happens if Travis decides to do something. In the talk above, Travis writes:
"Hey Mike,
If there is ever a v4, yes, this would change and be how it gets returned but at this point there's too many clients expecting the data back in the current format. I don't like to make to data changes if I don't have to. It's been this way since 2012 (when v3 launched) so I will leave it be for now."
So .. We should be safe.
—
Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub.


Autocomplete atom snippet inside Styled-Components template literal

I love the syntax highlighting available when using styled-components as so:
const text = styled.p`
  color: ${palette.primary};
`;

However, I'm trying to get some atom snippets working to save myself repetitive typing. I can't seem to get the snippet to show in autocomplete while between the styled template literal backticks.
In my snippets.cson I've tried
'*':
  # Styled Components
  'SC prop destructure':
    prefix: 'scp',
    body: "${({ $1 }) => $1 && $2 }"

The snippet will show for autocomplete throughout the rest of the file; as soon as I move the caret into the css syntax highlighted area, the snippet won't show. Is there a particular source I need to target to have it show? *, source.js, or even source.css won't seem to apply it within that zone.
Thanks for all your hard work, everything else works like a dream.

I've just pushed 2.60.0 that removes the constraint I placed on snippets from working inside styled-components. I would suggest you use a scope of .source.inside-js.css.styled

Amazing :100:

@DJTB  where you able to make snippets work inside a styled component?

@tenjojeremy
Inside of any of the following it works fine.
css`
  /* snippet tab complete */
`;

styled(Component)`
  /* snippet tab complete */
`;

styled.div`
  /* snippet tab complete */
`;

But not a plain template literal:
const someStyles = `
  color: blue;
  /* snippet won't work */
`;

My snippet (note the source scope):
# Styled Components
'.source.inside-js.css.styled':
  'SC prop destructure':
    prefix: 'scp',
    body: "${({ $1 }) => $1 && $2 }"
```

@DJTB Thank you fellow coder!

@DJTB I want to try it. What do I download?

@c0debreaker Open your snippets in Atom via the command palette and add the snippet?
Then use scp[TAB] in the appropriate scope (see previous comment).


Updating

Summary

Fixes #
Lorem ipsum dolor sit amet, consectetur adipiscing.
List of changes proposed in this PR (pull-request)

Lorem ipsum dolor sit amet, consectetur adipiscing.
Lorem ipsum dolor sit amet, consectetur adipiscing.

What should a reviewer concentrate their feedback on?

[ ] Lorem ipsum dolor sit amet, consectetur adipiscing.
[ ] Everything looks ok?

Sorry, I don't understand the title or description of this PR. Please clarify!

Sorry, I don't understand the title or description of this PR. Please clarify!

Oh I apologise about this.......It is a duplicate PR.

I will close this because it's duplicate PR!

yes please, thank you.


Clip without defined a_max

https://github.com/ElementAI/baal/blob/798f6e3668d73c476013dcdc4aca8f54c3420b28/baal/active/heuristics/stochastics.py#L45
This line is causing me issues.  Will try setting a_max to None, but I'm not sure of the implications relative to the intent.
Traceback (most recent call last):
  File "main.py", line 226, in <module>
    run(model, baal_trainer, baal_data_module, config, experiment_name)
  File "main.py", line 212, in run
    should_continue = baal_trainer.step(model, datamodule=baal_data_module)
  File "/workspace/environments/active-learning/lib/python3.8/site-packages/baal/utils/pytorch_lightning.py", line 202, in step
    to_label = self.heuristic(probs)
  File "/workspace/environments/active-learning/lib/python3.8/site-packages/baal/active/heuristics/heuristics.py", line 258, in __call__
    return self.get_ranks(predictions)[0]
  File "/workspace/environments/active-learning/lib/python3.8/site-packages/baal/active/heuristics/stochastics.py", line 45, in get_ranks
    distributions = np.clip(distributions, 0)
  File "<__array_function__ internals>", line 4, in clip
TypeError: _clip_dispatcher() missing 1 required positional argument: 'a_max'

Will try to submit PR for the string mapping mentioned in previous issue shortly (https://github.com/ElementAI/baal/issues/210).

Closed in #213


bug: Triple left-click selects the first character on the next line

Check the following:

[X] I have tried setting vscode-neovim.neovimClean in VSCode settings and restarting VSCode.
[X] I have read the vscode-neovim docs.
[X] I have searched existing (closed!) issues.

Neovim version (nvim -v)
v0.10.2
Operating system/version
Linux 6.11.5-arch1-1
Problem (including log messages)
Under normal mode, triple left-click (line-wise selection) also selects the first character on the next line. It should only select the current line.
Insert mode doesn't seem to have this issue.
Steps To Reproduce

Switch to normal mode
Triple left-click on any line
The first character on the next line is selected along the current line

Expected Behavior
Only the current line should be selected

This is the behavior of VS Code, which includes the line break at the end of the triple-clicked line. You can set :set selection=exclusive, but I recommend switching to insert mode before selecting with the mouse.