fix: Include '.conda' packages for reindexing from package store

Currently only .tar.bz2 packages are considered for reindexing. This PR will allow reindexing to see .conda packages too.
fixes #703

@wolfv, thanks for the commit to fix the CI, seems like some of the workflows are still failing.

I recently started trying out the new .conda packages and they work mostly well - except that they don't appear as "current version" in the Quetz API.
I tracked the bug down to the missing reindexing, which causes multiple packages to have version_order = 0 in the database.
It would be really awesome if you could merge this, so that the issue is solved, because currently the API is pretty broken as soon as .conda packages are involved.

@wolfv, Can we merge this change? Are the CI errors related to the change in the PR?


Philips introduces HueSDK

Philips introduced the HueSDK with a Windows version as well (it is however a DLL wrapped around a JAR it seems), and only working on 86/64.. Looks pretty legacy to me.
Anything we could learn from it? (Like the multi-Bridge approach)?
https://github.com/PhilipsHue/HueSDK
https://developers.meethue.com/documentation/hue-sdk

I had a quick look at it, but it looks like it's a Java SDK that also works with Java Desktop apps. It's not a .Net SDK.
I didn't see anything about the multi Bridge support, can you point a link to that?

Ah, its listed in the Q&A that I forgot to post: https://developers.meethue.com/documentation/whyshows-whats-new-hue-sdk

New features in Hue SDK?
Multi-bridge support with persistent simultaneous bridge connections
Fast connection mode
Persistent Bridge State in DB
Rule Builder – Simplifies rule creation
Rule Parser – Simplifies rule interpretation
Schedule Builder – Simplifies schedule creation
Color Space Conversion – CT<-> RGB RGB<->HSV
Heartbeat Manager – Polling period enforcement
Automatic re-connection to Hue bridges
Automatic start of authentication process when not authenticated
Complete Hue API support, including Hue Remote API with valid credentials
Continued future feature support as new features become available


Update inbox zero to New Years?

✨💥🎇🎆
Not sure on the emoji, but might be nice from Friday or something?

Bug Report Dump (Auto-generated)

Version 1.17.0<PHONE_NUMBER>)
Device: iPhone 6s Plus (iOS 11.2.5)
TestFlight: true

Yup! That’s the plan
Sent with GitHawk

done!


Make configurable compression for L0-L1

Search before asking

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

Motivation
I have high write rate, and data have good compression (coef 8-10). But in kvrocks - L0 and L1 dont compress data. And i have a very big write/read rate for compaction L0-L1.
may be move to config this logic?
    // For the first two levels, it may contain the frequently accessed data,
    // so it'd be better to use uncompressed data to save the CPU.
    std::string compression_levels = "kNoCompression:kNoCompression";
    auto db = srv->storage->GetDB();
    for (size_t i = 2; i < db->GetOptions().compression_per_level.size(); i++) {
      compression_levels += ":";
      compression_levels += compression_option;
    }
    return srv->storage->SetOptionForAllColumnFamilies("compression_per_level", compression_levels);
  };

Solution
add setting in config
 rocksdb.enable_compression_for_l0_and_l1 yes/NO

Are you willing to submit a PR?

[X] I'm willing to submit a PR!

Maybe we can have an option called rocksdb.nocompression_level,
and rocksdb.nocompression_level n means that the first n levels don't need to compress.

Maybe we can have an option called rocksdb.nocompression_level,
and rocksdb.nocompression_level n means that the first n levels don't need to compress.

    int compression_level;
    int nocompression_level;

funny naming :)
iI'll come up with a name and add it

Good to me.

Good to me too, we also have an heavy write rate system


Fix indented markdown (from nunjucks renderer) from being wrapped in code blocks by removing indents

Nunjucks loader is run first and will add indentation to markdown which then the markdown renderer will wrap in code blocks. This uses a regex replacement to remove the indentation before the markdown to html renderer is run.
Resolves #17

 Adding two "code" blocks in markdown breaks next line


[JVM-PKG] Update JNI to include Rabit interface

cc @CodingCat
I tested a simple distributed version as standalone runner and it is included in this PR, let us aim for data flow integrated test in next step.

Step 1: Use the same program, only use mapPartition as job starter and set env correctly.
Step 2: load data from the local data source

https://github.com/tqchen/xgboost/blob/c428a93adcb345f9a08c784aeecf7f7bd8c534f0/jvm-packages/test_distributed.sh


item-reorder returning wrong indexes in some cases

the (ionItemReorder) $event object returns the wrong 'to' index when dragging an item up or down, then while still dragging it, dropping it back to it's original place.
you would expect the 'from' and 'to' index to be the same value in this case, but instead it returns the 'from' index +1 or -1 depending on the last direction.
eg. dragging index 1 to index 3 then back up to index 1 and dropping it. will return, Object {from: 1, to: 2}
dragging index 3 to index 0 then back up to index 3 and dropping it. will return, Object {from: 3, to: 2}
you can also see the item is moved up or down while it should actually be remaining in place.
eg. when using ionics reorderArray(); function
Example,
http://plnkr.co/edit/590zKDfE071e63NYzhTK?p=preview
Cordova CLI: 6.3.1
Gulp version:  CLI version 3.9.1
Gulp local:
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.1.0-beta.1
OS:
Node Version: v5.12.0

Hello, thanks for using Ionic! I am going to close this as a duplicate of https://github.com/driftyco/ionic/issues/7522.

Hello, thanks for the reply, I dont think it is a duplicate, I think he is referring to the way animations on the list items don't always work. the two might be related but I doubt it.
I just fixed this bug by the way,
https://github.com/estyles/ionic/commit/4c8bbdf02d0676dcb65dd98fbfbf8767ffe0a82c
maybe you guys can figure out a better solution but this works great for me.

nvm.. it still doenst work, just replaced it with some other problems, I'll look into it further tomorrow.


Incorrect Tile Rendering at Higher Zoom Levels Due to Buffer Size

Description:
At higher zoom levels on our map at https://stadtplantest.winterthur.ch/basemapav/, some vector tiles are incorrectly rendered. In the browser console, I receive the following warning:
Geometry exceeds allowed extent, reduce your vector tile buffer size

Investigation:
Upon reviewing the documentation, I found the parameter buffer_size which controls the tile buffer in pixels:
buffer_size u32 (optional): Tile buffer size in pixels (None: no clipping)
Question:

How should the buffer_size parameter be correctly configured to resolve this issue?
What is the optimal buffer_size to ensure proper rendering without overloading the tiles?
Do I need to define the buffer_size for each individual layer in my tileset?

The buffer_size has to be defined for each layer. The value depends on the used styles. If the style has a border line witdh of 5 for a polygon layer, then buffer_size should also be 5. For line layers it's half of the line witdh.

Solution:
I managed to solve the issue by configuring the buffer_size parameter as follows:
YAML
Code kopieren
[[tileset.postgis.layer]]
name = "av_bb_boflaeche"
table_name = "public.av_bb_boflaeche"
minzoom = 15
maxzoom = 24
geometry_type = "POLYGON"
geometry_field = "geom"
fid_field = "fid"
srid = 2056
buffer_size = 256
I found useful information about buffer_size in this article: Tile Buffer Size.
What do you think about buffer_size = 256?

Objects spread over multiple tiles are a common problem of vector tiles. One solution is to add a feature ID und use that for feature selections, the other one is avoiding clipping. Most of the time you want clipping, but there are use cases like this one when you want to disable it. Maybe we should have a spearate configuration for clipping to make it independent from buffer_size
QGIS reads vector tiles like regular vector geometry sources using the GDAL driver and renders them like other sources, which can result in major differences to MapLibre GL.


Removed auth0 in favor of firebase

Auth0 auth has always been a thorn in my side. Now the web runs anon firebase auth and I extended it here so we can get a unified view of all users. I don't like google either but this sure is less work.
Removed: Auth0, AppCenter
Added: Firebase auth, non-IDFA analytics, crashlytics
If it's important to people in the future, maybe we can add actual signin methods so people can sync their progress across web and mobile, but for now I don't think it's necessary.

Fixed the bitrise build with some magic, disabled circleci since it's not needed anymore


add events metrics in etcdhttp.

This is a follow up of:
https://github.com/coreos/etcd/pull/3001
@xiang90, @philips  and @yichengq asked to move etcdhttp/metrics.go into a separate PR.
The commit has been cleaned up and made in-sync with the conventions recommended in #3001.
We currently are using these in our custom build of etcd and use the failure/total and latency metrics for alerting purposes. We've found that having etcd latency metrics is incredibly useful when troubleshooting latency issues of third party services that rely on etcd (including stuff we've built ourselves).

Guys, any updates here? :)

@xiang90, @philips now that 2.1.0 is released and metrics are a core feature of the system, can you take another look at these etcdhttpd metrics?
We're relying on them for monitoring the health of our etcd clusters, and would be really nice if we didn't have to rely on our custom builds of etcd with this. If you're worried about the performance impact of these, I'd be happy to put them behind a flag or a config parameter like extended_metrics.
Thoughts? :)

We will think about this after we release 2.2 alpha0.

@mwitkow-io Kindly ping. :)

Apologies, it was a hectic week :)
Moved the latency reporting to include flushing, so that it covers the whole lifecycle of the request. Made it clear what the latency metric does.

@mwitkow-io Make sense to me. Can you please make the naming consistent between docs, code and description? Thanks!

Addressed comments:

removed events, renamed metrics and functions
updated docs
Rebased into the same commit.

@mwitkow-io Thanks a lot! We are almost there!

LGTM.


Create popout box for visual response

Popout Box

Small box that displays responses to the user
Show and Hide

Ideas

Show box at selected text
Show box at top-right
Show box at cursor
Show box at fixed position set by user

Currently trialing chrome API notification system

Settled on chat history box which has now been implemented


fix(development): fix URL of golangci-lint

Fix broken documentation link to golangci-lint
Fixes #3633

[x] Change the broken golangci-lint link from https://golangci-lint.run/usage/install/#local-installation to https://golangci-lint.run/welcome/install/#local-installation.

Hey @gabek. Please have a look at this and let me know if any changes are to be made.

Thanks so much for fixing this!


Fix icon for PWA on iPhone/Android

Home screen icon does not look good on iPhone, from my testing of #107.

@mykhailodanilenko Can you please list the sizes/formats for Mikael to make icons & PWA splash screens? Re-assign it to him and me after.

@mykhailodanilenko Icons attached. Let me know if anything is missing

App icon.zip

Icon.zip


Can I use a ReferenceField in a custom field component?

I have an api that returns something like this :
[{
    "created_date": "2016-12-07T13:57:30.896Z",
    "id": 650,
    "project": 7,
    ...
  },
  {
    "created_date": "2016-12-07T13:57:30.897Z",
      "id": 392,
      "project": 7,
     ....
    },
...
]

Now I have a custom field component to display each of these items and there is a REST endpoint to get /projects where I can get info about each project (here with id 7).
Is there a way I can include such reference in my custom field component as I can only get the 7 value there instead of data from GET /posts/7 that I would like to display :
{
   "id": 7,
    "title": "Something cool",
    "slug": "lorem-ipsum",
    ...
}

Thanks

We don't provide support through GitHub issues. If you need advice on using or debugging your own project based on admin-on-rest, please use StackOverflow.


sort output of module names to be able to execute multiple migrations in order.

Single-branch one-commit merge for https://github.com/ostinelli/gin/pull/9 and https://github.com/ostinelli/gin/pull/10

pls use leite's PR to address this issue


pod install, unable to satisfy the following requirements, MetalBender (~> 0.4) required by Podfile

also, when trying on simulator no warning  but when connecting my iPhone , i have multiple error including couldn't not find metal bend ( import) and multiple " already declared etc" on the proxy

thanks for all btw

iOS 11, xcode 9, podfile(10.0)
Any idea??
thanks

Seems we forgot to release the new version to cocoapods 😅
@mtnbarreto

ahah ok no problem, just saw you added this version ! btw is this version a fix for the problem I had ( with the proxy redeclaration etc)?
thanks

@JeremyGozlan
Yup. Feel free to open another issue if you have any, we are releasing to cocoapods in a few minutes.
Cheers


vterm lighter colors

This is a follow-up to a discussion on discord about using vterm's :background attributes to get lighter colors.
Currently, I have this set to get a gray-ish color in the nord theme:
(set-face-attribute 'vterm-color-black nil
                    :background "#9099AB")) ;; a very bright grey

I'm a bit hazy on the specifics of this discussion. Are you requesting a lighter background for vterm-color-black for the nord theme?

Hmm, that is a good question. Digging up the discussion on discord (searching for vterm-color-black), I remembered it was this:

I'm hoping that maybe someone here can help me with emacs colors not quite looking the same as iTerm2 draws it:

The second one is emacs (vterm-mode, specifically)
I'd like to try and get it to match the first one, if possible, but I don't know how to find out what color that is

flat replied (a few messages down):
(set-face-attribute 'vterm-color-red nil
                    :foreground (get-red-color)
                    :background (get-bright-red-color))

and Gagbo followed up with:

Doom-themes usually only have 1 variant of red/blue/green... And we can't just take doom-lighten or doom-darken for the 'bright' version because it depends on the theme (is it a dark or a light theme ?)

So, I guess my request is to somehow figure out the bright version automatically?

and Gagbo followed up with:

Doom-themes usually only have 1 variant of red/blue/green... And we can't just take doom-lighten or doom-darken for the 'bright' version because it depends on the theme (is it a dark or a light theme ?)

So, I guess my request is to somehow figure out the bright version automatically?

Not following the overall question but it might be relevant to point to doom-blend. If you blend with a high base (eg base8), you get a darker color in a light theme and lighter color in a dark theme. With base0 the opposite will happen. Hope that is helpful.

I have no idea what the current status of this issue is

I don't think this is an issue anymore (just commented out my hack and the colors seem correct) so I'll go ahead and close it


"alwaysHideShield" is not properly reflected in first person

(v1.3.1 for mc 1.20.1)
If "alwaysHideShield" is false, the shield will be hidden when holding a sword (and not other items) in third person, but not first person, where it will still render.

I cannot reproduce this one, please list the other mods you are using.

Couldn't reproduce either. Make sure you don't have any other mods that could conflict with this mod & that you are on the latest version of the mod!


support new authorization features of nats

http://nats.io/documentation/server/gnatsd-authorization/

deprecated in https://github.com/choria-io/puppet-nats/pull/72


doc: Informative README

Closes #148.
It is best viewed if you preview the file in "Files changed" tab, then clicking on the 3 dots, followed by "View file". Below a visual guide on how to view it:

Hi thank you for the initial effort. I know, I am picky about the readme but I feel like the bullet points for each module are too bloated and sound like AI. Think it would be better to reduce them to the essentials.

Ok, agreed.
Much more succinct now and following the description @wz14 provided in #148.


Readme and Example ToDos

Move json-rpc example into its own example_test.go (to get it compiled every time) file and put a link to it from readme.md.
Replace the example in readme.md file with a simple byte ping/pong and a separate sender (server --> client) example.

Moved from: https://github.com/nbusy/neptulon/issues/24

Use both Listener and Client objects in JSON-RPC protocol
Demonstrate raw message exchange within comments
Add example_test file with a //result comment, to be compiled and executed by Travis

All done.


fix: autocomplete experience

Description
Autocomplete experience improvements.
Tab to autocomplete.
Backspace no longer closes autocomplete, instead show new suggestions if available.
Fixed autocomplete flicker on pressing tab and space.
Fixes #5653 #7564 #8052
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manual

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: fix/autocomplete_experience

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
54.72 (0)
36.4 (0.01)
33.49 (-0.02)
55.27 (0)

:red_circle:
app/client/src/components/editorComponents/CodeEditor/generateQuickCommands.tsx
29.33 (-0.4)
6.67 (0)
6.25 (0)
28.38 (-0.39)

:red_circle:
app/client/src/components/editorComponents/CodeEditor/hintHelpers.ts
78.57 (0)
58.33 (-11.67)
40 (0)
77.78 (0)

:red_circle:
app/client/src/components/editorComponents/CodeEditor/index.tsx
70.7 (0.26)
42.76 (0.35)
51.28 (-3.72)
70.63 (-0.86)

:red_circle:
app/client/src/components/editorComponents/form/fields/EmbeddedDatasourcePathField.tsx
30.22 (-0.21)
2.53 (0)
4 (0.15)
32.03 (-0.25)

:green_circle:
app/client/src/selectors/commentsSelectors.ts
79.84 (1.61)
57.35 (2.94)
70 (0)
86.36 (2.27)

/ok-to-test sha=85c5c50

/ok-to-test sha=85c5c50

/ok-to-test sha=2104cfb


chore(queue): add an informative toString() in SqlQueue

The QueueProcessor will now display something like Using queue SqlQueue(queueName=default, poolName=default), which can be useful when there are multiple SqlQueue beans around connected to different sql pools

@marchello2000 the useful toString() I promised you in a different PR


unhandled json parsing exceptions

Noticed that json parsing exception are not handled. if the value is not proper json will end up in:
Uncaught SyntaxError: Unexpected token in JSON
I guess it might be handled later on, but does break the example
https://github.com/robinvdvleuten/vuex-persistedstate/blob/9d79a3e09d63c442b83ee70a6aaeb33726a23931/src/plugin.js#L59

@gaborcorp it is indeed not handled by the plugin to keep the functionality of it by a minimum. But all core functionality is overridable, so you could add your own JSON parsing by overriding the getState method.
The reason why I didn't add validation or other "extended" functionality is that everybody has a different opinion on how to handle those. By making the core functionality - persisting (partial) state into localstorage - just work, one can easily add their own functionality on top of it.

I'd still argue that you probably should handle exceptions raised by your own code (or at least somehow be transparent that it must be done) and then if people disagree in the way you recover they can override that.
Currently if there is an error than it will break your application through the unhandled error which I don't think is anyone's default choice.


samples/subsys/shell/shell_module doesn't work on qemu_x86_64

Describe the bug
When I run this sample under qemu_x86_64 I do not get a shell prompt.
Expected behavior
An interactive shell prompt should be available that accepts commands.
Impact
Not sure if this is a sample issue or the shell subsystem is broken in some way. Could be an issue with the x86_64 arch, or an SMP problem.

I think this is an SMP problem, the sample works fine if I set CONFIG_MP_NUM_CPUS=1

the sample works fine if I set CONFIG_MP_NUM_CPUS=1

@andrewboie  Thanks for this! Can you please help me reproduce this? I added CONFIG_MP_NUM_CPUS=1 to the prj.conf file in samples/subsys/shell/shell_module but it still hangs for me.

CONFIG_MP_NUM_CPUS=1 to the prj.conf file in samples/subsys/shell/shell_module but it still hangs for me.

nvm - this works to reproduce the passing case. Not sure what happened earlier. Anyway, this issue is still on my radar.

btw - I see this same behavior for qemu_x86_64 tests/drivers/uart/uart_basic_api/. The test_uart_fifo_fill fails, but works fine if I set CONFIG_MP_NUM_CPUS=1

update here - I rebased today and this issue still occurs. However, this issue appears to be resolved with cherry-pick of #23325.

update here - I rebased today and this issue is resolved. Above referenced #23325 fixed it, from what I can tell.
I tested today on master 7201c1d - No modification or cherry-pick.
west build -p -b qemu_x86_64 samples/subsys/shell/shell_module
[jennife2@jennife2-desk1 zephyr]$ west build -t run
-- west build: running target run
[3/4] To exit from QEMU enter: 'CTRL+a, x'[QEMU] CPU: qemu64,+x2apic
qemu-system-x86_64: warning: TCG doesn't support requested feature: CPUID.01H:ECX.x2apic [bit 21]
qemu-system-x86_64: warning: TCG doesn't support requested feature: CPUID.01H:ECX.x2apic [bit 21]
SeaBIOS (version rel-1.12.1-0-ga5cab58-dirty-20200214_052440-f7294c49af13-zephyr
)
Booting from ROM..Optimal CONFIG_X86_MMU_PAGE_POOL_PAGES 8

uart:~$


Do you have any guide for angular 5 ?

Hi @gtzinos, there's an improved version of Pt called "Pts", written in typescript and better support for npm workflow. Please take a look at https://ptsjs.org.
We don't have Angular guide yet, but there's a getting-started guide as well as a React example repo. Angular workflow should be fairly straighforward. If you encounter a problem, please find an issue in the Pts repo. Thanks!

Sorry i forgot to close the issue, i had already implement it by npm. Thank you!


StreamBridge works incorrect, if producer is partitioned

StreamBridge cant work with more than one producer correctly.
when service has more than one producer, and any of them has filled partitionKeyExpression property or partitionKeyExtractorName property, then this configuration properties used by another producers too.
Also StreamBridge not thread safe right now.

As you can see on this picture (step 1) streamBridge get function for invocation from fuctionCatalog, and this function used for all producers.
step 2 if producer is partitioned, then function got wrapped.
step 3 function invoked.

Here we can see logic of wrapper.
step 1. initializing of enricher that has partition logic. function here is streamBridge function from fuctionCatalog.
step 2 . before function got invoked, wrapper set enhancer to this function.
step 3. setting enhancer field in streamBridge function.
Last step is a problem step. Because streamBridge function one for all producers, this enhancer used by every one of producers, what means that producer without partitioned settings will be partitioned, but with settings of another producer.
And if streamBridge send messages to two different partitioned producers in multiple threads, will lead to race condition.
version 3.1.3
Expected behavior: settings for different producers  work separately.

So the logic you are referring to has changed recently due to a fix for an unrelated issue. I've modified the test to ensure it validates multiple producers with different partition configurations


Fetch backend for Scala.js

Create a JavaScript backend implementation based on the fetch API. The
fetch API does not allow access to redirect responses so the backend
cannot be wrapped with the FollowRedirectsBackend.
In addition a streaming implentation using Monix has also been created.
Unfortunately no browsers support a stream as a request body so it is
converted into an in memory array before sending.

Thanks! Almost there :)
Two questions upfront though:

I'm not sure I understand the problem with redirects - in the fetch backend implementation, you enable or disable redirects basing on the followRedirects setting. If redirects aren't followed, doesn't the user get a 30x status code?

I'm afraid that exposing a "fake" stream might be misleading to users - looking at the API it's easy to be mistaken and assume that when using the monix backend the data is really streamed. So while it's great to have a Monix backend, I would drop the streaming part.

Browsers don't allow you to access any redirect responses. XHR always follows them while fetch lets you choose to follow, manual or error. Setting it to manual causes the response to have a type of opaqueredirect which contains no information at all (the stauts code is 0). So if the request turns off following redirects and one occurs then the response will return an error.

I see your point but the reason I choose fetch was to get proper streaming response bodies (I didn't realise that no browsers have yet to implement streaming request bodies). Would you accept an exception?

Thanks for the prompt reviews :)

I see, thanks. I would put that in the docs, though, as people might be surprised (like me ;) )

ah I read the code & comment once again, so theres is "true" streaming when receiving response bodies, but there isn't when sending request bodies - am I right? If so, then sure, the Monix impl makes sense :)

I did try building for scala native but you have to use a milestone ScalaTest and I got a compile error.

Ah, well, maybe in some time then ;)

Maybe you have an idea, sometimes when running circe tests they fail with:
[error] org.scalajs.testcommon.RPCCore$RPCException
[error] 	at org.scalajs.testcommon.RPCCore.$anonfun$handleMessage$4(RPCCore.scala:69)
[error] 	at scala.Option.foreach(Option.scala:257)
[error] 	at org.scalajs.testcommon.RPCCore.$anonfun$handleMessage$1(RPCCore.scala:67)
[error] 	at org.scalajs.testcommon.RPCCore.$anonfun$handleMessage$1$adapted(RPCCore.scala:47)
[error] 	at org.scalajs.testcommon.Serializer$.withInputStream(Serializer.scala:48)
[error] 	at org.scalajs.testcommon.RPCCore.handleMessage(RPCCore.scala:47)
[error] 	at org.scalajs.testadapter.ComJSEnvRPC$$anon$1.run(ComJSEnvRPC.scala:31)
[error] Caused by: scala.scalajs.js.JavaScriptException: TypeError: Cannot read property 'prototype' of undefined
[error] 	at <jscode>.$newJSObjectWithVarargs(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:478)
[error] 	at org.scalajs.testinterface.TestUtils$.newInstance(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:8224)
[error] 	at org.scalatest.tools.TaskRunner.executionFuture(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:30187)
[error] 	at org.scalatest.tools.TaskRunner.execute(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:30149)
[error] 	at <jscode>.{anonymous}()(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:8390)
[error] 	at scala.scalajs.runtime.AnonFunction1.apply(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:40621)
[error] 	at <jscode>.{anonymous}()(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:7033)
[error] 	at scala.scalajs.runtime.AnonFunction1.apply(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:40621)
[error] 	at <jscode>.{anonymous}()(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:18605)
[error] 	at scala.scalajs.runtime.AnonFunction1.apply(file:///Users/adamw/projects/sttp/core/js/target/scala-2.12/core-test-fastopt.js:40621)
[error] Could not run test com.softwaremill.sttp.CirceTests: org.scalajs.testcommon.RPCCore$RPCException

Seems to have something to do with them running in parallel. I can't seem to find a config setting that stops that so the best thing might just be to add some aliases:
addCommandAlias("testJS", "; coreJS/test; catsJS/test; monixJS/test; circeJS/test")
addCommandAlias("testJVM", "rootJVM/test")

(kind of like what refined does https://github.com/fthomas/refined/blob/master/build.sbt)
Running sbt ++2.12.6 testJVM && sbt ++2.12.6 testJS && sbt ++2.11.12 testJVM && sbt ++2.11.12 testJS no longer seems to get that error.

Hm but this won't fix running sbt test. I tried couple of methods of disabling parallel tests but nothing helped ...


style: the card component should have more shadow when hover

fix: #335

Pull Request Test Coverage Report for Build 389

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

Totals

Change from base Build 385:
0.0%

Covered Lines:
1232

Relevant Lines:
1528

💛  - Coveralls


feat: differentiate ecrecover with strict and lax check for s

Ethereum Yellow paper defines two initial conditions for ECRecover:

in case of checking transaction signatures, s should be s < N+2 + 1
in case of ECREC precompile, s should be s < N.

This PR updates the current precompile method which takes additional variable which defines the upper bound to compare s against. Also added test cases (strict check, lax check, assumed failure).
This is based on top of fix/ecdsa-v which depends on update in gnark-crypto. So have to wait for other PRs being merged before can merge this one.
CC: @AlexandreBelling, @OlivierBBB -- can you confirm this behaviour is good?

in case of checking transaction signatures, s should be s < N+2 + 1

I assume you mean  s < N/2 + 1?

in case of checking transaction signatures, s should be s < N+2 + 1

I assume you mean s < N/2 + 1?

Yes! Sorry, my typo.

I think 01-ecrecover.go should exclude the bounds checks on s and r as these are done on the zkEVM arithmetization side to avoid junk calls https://github.com/ConsenSys/zkevm-spec/issues/50

I think 01-ecrecover.go should exclude the bounds checks on s and r as these are done on the zkEVM arithmetization side to avoid junk calls ConsenSys/zkevm-spec#50

I would also avoid, but there is a gotcha. This primitive is used in two places - first by the arithmetization when ECRECOVER precompile is called and second to verify the validity of the transaction.
For the first case it is obvious that it would be better to perform the check on the arithmetization side to avoid calling the circuit. But for the second case the check is done by the prover directly on the block data without passing to arithmetization. And there it may be better if we do the check on our side (I'm not fully sure, @AlexandreBelling can confirm?).
Now, we have two options - either implement two different circuits where in one we do not check s (for calling precompile in arithmetization) and in the second one we do (for prover verifying transaction signature). Or we have only a single circuit with a selector to select the bound we check s against. For simplicity I went for a single-circuit route.

So, I asked @AlexandreBelling, and as far as I understood then there is a fixed amount of ECRecover calls within a block. Having a single circuit allows to allocate these calls more efficiently between transaction verification and calling ECREC precompile. So the recommendation was to have a single circuit, even if when calling precompile we perform additional check on s.


Changed record name to dict

Closes https://github.com/quantified-uncertainty/squiggle/issues/2084

I'll rename it in other places (squiggle-lang, etc.) in this PR, shouldn't take long

Btw, it's unfortunate that dict and dist are so similar, might cause many typos. I kinda liked Record a bit more because of that.

Btw, it's unfortunate that dict and dist are so similar, might cause many typos. I kinda liked Record a bit more because of that.

That's a good point. I don't think it's enough for us to change now, but it is unfortunate.


[DOCUMENTATION] Create Design Pattern Diagrams for WADM Documentation

Would be wonderful to include or update diagrams found here: https://github.com/wasmCloud/wadm/issues/15 such that it reflects the new design patterns within WADM.

WIP
Building a map for myself prior to comments so it is easier to create a diagram. All of this is already documented withing directories. Just trying to have it in 1 place as a table of contents for diagram development.
Will be updating this as I progress. This comment will be edited. Will make it complete when done.
Directories
root/

bin/

connections.rs:

A module for connection pools and generators. This is needed because control interface clients (and possibly other things like nats connections in the future) are lattice scoped or need different credentials

logging.rs:

otel tracing code

main.rs:

application

nats.rs:

client and helper code

observers.rs:

Types for observing a nats cluster for new lattices

src

lib.rs:

library root to make underlying crates in src available for bin. Default topics and metadata.

nats_utils.rs

helper utilities for working with nats

publisher.rs

generic nats publisher traits for testing and flexibility

test_util.rs

test utilities

commands/

mod.rs:

Type implementations for commands issued to compensate for state changes

consumers/

mod.rs:

Contains implementions of durable consumers of events that automatically take a message from a consumer and parse it to concrete types for consumption in a scheduler

commands.rs:

A module for creating and consuming a stream of commands from NATS

events.rs:

A module for creating and consuming a stream of events from a wasmcloud lattice

manager.rs:

Contains manager and worker code. A manager of a specific type of Consumer that handles giving out permits to work and managing per lattice consumers. A worker is any type that can perform work based on a given [ScopedMessage]. This worker should contain all state (or handles to state) that it needs to complete its work

events/

mod.rs:

exposing modules

data.rs:

types related to claims,linkdefs, actor and provider identity.

deser.rs:

Custom deserializers for event fields

ser.rs:

Custom serializers for event fields

types.rs:

Common event types expected on wasmbus. These are the inner types generally sent in the data attribute of a cloudevent

mirror/

mod.rs:

This is a temporary workaround to let us combine multiple topics into a single stream so consumers work using it in wasmcloud projects

model/

mod.rs:

Types and implementations for OAM manifest fields

internal.rs

Contains the internal storage definition of a manifest

scaler/

mod.rs
manager.rs
simplescaler.rs
spreadscaler/

mod.rs
link.rs
providers.rs

server/

mod.rs
handlers.rs
notifier.rs
parser.rs
types.rs

storage/

mod.rs
nats_kv.rs
reaper.rs
state.rs

workers/

mod.rs
command.rs
event_helpers.rs
event.rs

@thomastaylor312 What is the purpose of having a simplescaler and a spreadscaler that also has an actor spread scaler? Do they do different things?

@stephenandary not Taylor, but just wanted to let you know the simplescaler was developed just to prove out the Scaler trait and to validate some initial tests. It's definitely more of a test implementation than a fully featured scaler 😄

Ok cool, thanks @brooksmtownsend. Just trying to get to good mental map for diagram. 1 thing I don't need to worry about now woot woot haha

@brooksmtownsend or @thomastaylor312 , can I please make sure I am covering all of the components of wadm that may be relevant to a diagram:
Components:

Server
Nats KV
Nats Event Stream
SpreadScaler
Command Manager
Event Manager
Command Worker
Event Worker

Does this make sense? Am I missing anything or is something incorrect?

I also see there are multiple streams, but I don't know if that should be on a diagram as a mirror is just modifying stream to be usable due to nats limitations in current version.
I also see an observer in Bin, should that be a component?

Also, what is emitting events to the event stream?

Per Taylor: Only events are from wasmbus.rpc and from wadm itself

I am also noticing a lot of similarities between wadm and concordance with language. Is this language internal, or is there some design model that it is based on? Some form of event-driven architecture maybe?

@stephenandary that last one might just be us formalizing on some terminology, but they aren't related per se. If there's a specific example we could elaborate though!

ConsumerManagers is 1 example

Let me know what you want me to do with this issue (keep open or close), moving onto some other work now and no longer have time.

@stephenandary I like this layout of the repo both in filenames and in concepts. I think what we should do is, once we get some time together, pop into a diagram tool while we're all on a call so we can sketch out the architecture. That will be fun!

Hey @brooksmtownsend, I have moved onto other work now so I don't have any time in near future. Hopefully someone else can help with this. Otherwise if you need me to close this issue let me know.

No worries @stephenandary! It's not urgent, just wanted to update the status. I like the issue

Closing issue to remove from my issue log.


[SerialConsole] Change to require custom boot diagnostics

The following commands now check to see if the specified VM uses custom boot diagnostics.
az serial-console connect
az serial-console send nmi
az serial-console send sysrq
az serial-console send reset

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[x] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

SerialConsole


Fix python-client publish GH workflow

The previous PR didn't solve the problem completely so here is a new one.

It's OK to skip code review for testing these tweaks.


[conn_pool] fix use after free in H/1 connection pool

Signed-off-by: Asra Ali<EMAIL_ADDRESS>Commit Message: Fixes (thanks @yanavlasov @antoniovicente) use after free when dispatcher tries to run conn_pool->onUpstreamReady() after the connection pool was destroyed. Reverts back to a schedulable callback per https://github.com/envoyproxy/envoy/pull/13867/files
Risk Level: Medium
Testing: Added test. This test fails with the use after free at head:

+TEST_F(Http1ConnPoolImplTest, RegressionTest) {
+  InSequence s;
+
+  NiceMock<MockResponseDecoder> outer_decoder;
+  ConnPoolCallbacks callbacks;
+  conn_pool_->expectClientCreate();
+  Http::ConnectionPool::Cancellable* handle = conn_pool_->newStream(outer_decoder, callbacks);
+  EXPECT_NE(nullptr, handle);
+
+  NiceMock<MockRequestEncoder> request_encoder;
+  ResponseDecoder* inner_decoder;
+  EXPECT_CALL(*conn_pool_->test_clients_[0].codec_, newStream(_))
+      .WillOnce(DoAll(SaveArgAddress(&inner_decoder), ReturnRef(request_encoder)));
+  EXPECT_CALL(callbacks.pool_ready_, ready());
+  EXPECT_CALL(*conn_pool_->test_clients_[0].connect_timer_, disableTimer());
+  conn_pool_->test_clients_[0].connection_->raiseEvent(Network::ConnectionEvent::Connected);
+
+  EXPECT_TRUE(
+      callbacks.outer_encoder_
+          ->encodeHeaders(TestRequestHeaderMapImpl{{":path", "/"}, {":method", "GET"}}, true)
+          .ok());
+
+  Event::PostCb post_cb;
+  EXPECT_CALL(conn_pool_->mock_dispatcher_, post(_)).WillOnce(SaveArg<0>(&post_cb));
+
+  dispatcher_.deferredDelete(std::move(conn_pool_));
+
+  inner_decoder->decodeHeaders(
+      ResponseHeaderMapPtr{new TestResponseHeaderMapImpl{{":status", "200"}}}, true);
+
+  // Clear deferred delete list and trigger post callback
+  dispatcher_.clearDeferredDeleteList();
+  post_cb();
+
+  conn_pool_->destructAllConnections();
+}

This test had to be modified now that there is a schedulable callback.What I don't like about the fix/test is that the callback is never scheduled after the fix because it is guarded by hasPendingStreams() which ends up false in this test, so there is never a potential for use after free if I add that if condition. I don't know if in production crashes this is exactly the problem or not. Trying to understand how to make this test better..

Working on reproducing the test failure in opt mode (doesn't reproduce locally with bazel)

The new test case is crashing on the gcc build and possibly others.  Please take a look.

Seems to be an issue clearing the deferred delete list which contains a connection pool and a client owned by it.


Add template parameter to get_buffer()

Fixes #1499
Description of the changes:
-Added a c++ template parameter to k4a::image::get_buffer() so that the pointer type returned can be easily controlled. There is a significant probability the image buffer doesn't contain 8-bit data, and a significant probability the caller needs a pointer other than uint8_t*. This is extremely common when interoperating with OpenCV

Before submitting a Pull Request:

[X] I reviewed CONTRIBUTING.md
[X] I built my changes locally
[X] I ran the unit tests
[X] I ran the functional tests with a device
[X] I ran the performance tests with a device

I tested changes on:

[ ] Windows
[X] Linux

/azp run

/azp run


Documentation typo

Documentation on following url contains a typo.
@include button-disable);

should be
@include button-disable;

I too ran into this typo. I'd be happy to fix it, but after some searching I'm unclear where this documentation is generated. Any tips?

After looking into this I found where the mixin is defined and believe it should be
@include button-disabled

This seems to be a bug with SassDoc when it encounter @param [Color]... instead or @param {Color}....


ARC-1790 - added set app properties on config

What's in this PR?
Why
Added feature flags
Affected issues
Jira Issues
How has this been tested?
Include how to test if applicable
Whats Next?

Exists in seperate non spike pr and merged


how to introduce mybaits Interceptor with tk.mybaits in srping boot app

i config my plugin as flollowing, but it doesn't work
@Configuration
public class MybatisConfig {
@Bean
public Interceptor myInterceptor() {
    return new MybatisPlugin();
}

}

What does the interceptor define?
Especially does this has the anotation  @Intercepts and @Signature
Example can be like this.
@Intercepts(
        {
                @Signature(type = Executor.class, method = "query", args = {MappedStatement.class, Object.class, RowBounds.class, ResultHandler.class}),
                @Signature(type = Executor.class, method = "query", args = {MappedStatement.class, Object.class, RowBounds.class, ResultHandler.class, CacheKey.class, BoundSql.class}),
        }
)
public class PageInterceptor implements Interceptor

or
@Intercepts({@Signature(type = StatementHandler.class, method = "prepare", args = {Connection.class, Integer.class})})
public class PrepareInterceptor implements Interceptor


AddDependency will find specified types not already found

AddDependency Recipe either responds to a marker put into the ExecutionContext by other visitors or finds specified types which have not yet been found.  Fixes https://github.com/openrewrite/rewrite-testing-frameworks/issues/116
This change

adds a new afterRecipe method to the Recipe, which is invoked after the recipe has finally completed providing an opportunity to conditionally add another recipe to the recipeList.
removes onlyIfUsing from the AddDependencyVisitor

I think afterRecipe is logically equivalent to getApplicableTest.

I think afterRecipe is logically equivalent to getApplicableTest.

Thanks for the suggestion, @jkschneider.  I have re-factored the code to usegetApplicableTest for this scenario by adding the ExecutionContext as a parameter.  The result is simpler, and validation happens first rather than requiring another full pass over all source files.

Really well done @pway99 !

also Fixes: https://github.com/openrewrite/rewrite/issues/401


[Enhancement] Consider key container memory usages for streaming aggregate

What type of PR is this：

[ ] bug
[ ] feature
[x] enhancement
[ ] refactor
[ ] others

Which issues of this PR relates：

Closes #6212.
Problem Summary：

Streaming Agg decides whether to PreAgg according to the memory usage of Hash Table and the aggregation degree. When the HT memory is less than 256KB, PreAgg will be always used.
For now, HT memory only counts the memory allocated by MemPool. However, in some scenarios, in order to optimize performance, HT's keys don't allocate from MemPool, such as AggHashSetOfSerializedKeyFixedSize, which will cause MemPool to be always 0. Therefore, even if there is no degree of aggregation, PreAgg will always be used.
Therefore, this PR consider the key memory used for streaming aggregate.

If keys are allocated from MemPool, the key memory usage is the pointer of the key.
Otherwise, the key memory usage is the real key memory usage.

Test Result
Environment

3BE with 64GB memory and 16vCPUs.

Query
-- Q1 Stremaing Distinct Aggregate
select count(distinct lo_orderkey*10+lo_linenumber) from lineorder;

-- Q2 Stremaing Distinct Aggregate
select count(*) from (
    SELECT LO_CUSTKEY, LO_SUPPKEY, sum(LO_REVENUE)
    FROM lineorder_flat
    GROUP BY LO_CUSTKEY, LO_SUPPKEY
) AS a;

-- Q3 Stremaing Distinct Aggregate
select count(*) from (
    SELECT LO_PARTKEY, LO_LINENUMBER, LO_DISCOUNT, sum(LO_REVENUE)
    FROM lineorder_flat
    GROUP BY LO_PARTKEY, LO_LINENUMBER, LO_DISCOUNT
) AS a;

-- Q4 Stremaing Aggregate
select sum(LO_REVENUE)
from lineorder
group by LO_PARTKEY, LO_LINENUMBER, LO_DISCOUNT
limit 10;

Result

Name
Before (s)
After (s)
After/Before

query01
10.01
6.36
0.64

query02
9.77
6.17
0.63

query03
5.99
3.51
0.59

query04
4.02
3.72
0.93

SUM
29.79
19.76
0.66

@Mergifyio backport branch-2.3


Create Testnet state machine

What was wrong?
In the cow people programming branch genesis_time, we are accessing eth2_config everywhere in the plugin. This should be avoided because eth2_config is defined fork-by-fork.
The correct way to access eth2_config is to get it from the state machine that we intend to use on a particular slot. If a customize eth2_config is required for a testnet, then we should create a Testnet state machine class who has the customized config as property.
During this PR I also realize that some genesis information, such as genesis_time and genesis_slot, could be organized into a BeaconGenesisData, while some private validator information, such as their private and public key that's specific to the testnet use, could be organized into testing_config.py.
How was it fixed?

mkdir  eth2/beacon/state_machines/forks/testnet/

create TestnetStateMachine in __init__.py
move its eth2_config to configs.py

remove the use of eth2_config everywhere
aggregate genesis data into BeaconGenesisData
aggregate all private validator information to trinity/plugins/eth2/beacon/testing_config.py

cc @hwwhww @mhchia @NIC619
Cute Animal Picture

BeaconGenesisData sounds reasonable. However in a local testing or testnet, genesis_time is likely to change in each run while other genesis data, e.g., genesis_slot aren't. Would it be trivial to automate the process of updating genesis_time in each run of testing/testnet?


Consistent behavior for true/false

Some SVM words use boolean atom types (true and false), while others use Nil for false and anything non-Nil for true. Since the second is the canonical SECD behavior, I should:

[ ] change all words to use non-Nil and Nil for true and false
[ ] remove the boolean atom type
[ ] ensure consistency in documentation

Removed from all tests in 48e9058.
Removed from all comparison operators in dee02d2.


Remove PackedLockTime in favor of absolute::LockTime

This is potentially a controversial PR, but hear me out. My argument is that right now absolute::LockTime and PackedLockTime are basically identical types; they can be converted between each other with From and they have exactly the same semantics except that PackedLockTime has Ord on them. This makes it very confusing to tell which one should be used, for example in PSBT2 where there are now extra locktime-related fields.
The motivation for having PackedLockTime independent of LockTime are:

PackedLockTime is theoretically more efficient because you don't need to unpack the field, don't need to store a enum discriminate, etc. I don't buy this. If you are trying to save individual results in your transaction-parsing there are lots of places you'd look before messing around with locktimes, so we shouldn't privilege this specific thing.
PackedLockTIme has an Ord impl, and removing that will have a cascading effect on transactions, blocks, etc., preventing them from being held in BTreeMaps etc. My proposal, implemented here, is to just manually impl Ord on Transaction and don't impl it on LockTime.

I recall some argument that we need to be able to sort miniscripts, and miniscripts might have locktimes in them, but I think this is wrong -- miniscripts always have explicitly either a Height or a Time, and there is no problem ordering these.

cc @Kixunil @sanket1729 @tcharding

Regarding optimization being kinda pointless I both agree with it and dislike it. :D

just manually impl Ord on Transaction and don't impl it on LockTime.

This sounds reasonable, let me sleep on it first but I'm leaning towards ACK here now.

OK, I think it's concept ACK from me. But I do want some doc explaining all this. :)

Rebased. Watch out for the new commit 1b26aac5dd0cce544f357126c5c459ad38ac5710 which unifies the serde impls, breakingly.

The code looks good to me, I wanted to mention that with this applied deriving Ord for Terminal in rust-miniscript becomes impossible and implementing it manually is painful because Terminal is a big enum and the manual implementation requires a match of ever variant which calls cmp on each variant.

Terminal is a big enum and the manual implementation requires a match statement which calls cmp on each variant.

Sounds like a job for a macro?
(I'm low on time rn so reviewing will take a while :( )

Yes, we can macro-ize it, or just remove the impl entirely in favor of PartialOrd since this type is kinda-sorta internal, or create our own OrdLockTime wrapper just for Terminal.
Good to be aware of, but this seems like a problem that's both solvable and not the responsibility of this library.

Cool, I'm happy with that.

Rebased to remove the derive(Ord).

@Kixunil can you say more about what documentation you want for this?
I'm going to merge this as-is, I"m happy to do a followup documenting things. But the LockTime type itself is already heavily documented, and the conversion to/from u32 that occurs during consensus encoding is self-contained and "documented" to the same degree that anything related to bitcoin transaction serialization formats are.

@apoelstra something in the changelog to help people understand it and some paragraph in both Transaction documentation pointing out that the ordering is based on bytes not semantics. And some paragraph on LockTime as well with mentions of each-other.

@Kixunil there is no byte-based ordering. I'm not sure what you want me to add to LockTime.
I will add a changelog entry, yes.

Huh? How are transactions ordered then?

@Kixunil ah, yes, transactions are ordered using the locktime as sort field and in this case the locktimes are compared using their consensus encoding.
I'm happy to document this, but this is the same ordering that Transaction has ever had, and I'm not aware of any semantic meaning to the Transaction sort order, nor can I imagine people expecting one to exist.

Basically I just want to have a heads up along the lines "this contains a non-Ord field and changes the ordering semantics of it in the PartialOrd implementation" Yeah, it's not something super-weird but having it visible so people can have it in the back of their minds when doing things with ordering is nice.


Bugfix 3.4:  revert recently added condition variable in ClusterCommThread stop

Missed in previous PR that ClusterCommThread destructor was calling Thread::shutdown.  The new code created a deadlock waiting for slow STOPPED state.
The actual target bug, segfault during Agency shutdown, is still unresolved.

http://<IP_ADDRESS>:8080/job/arangodb-matrix-pr-linux/2990/  ... running

http://<IP_ADDRESS>:8080/job/arangodb-matrix-pr-linux/2997/ ... running


Please add RelativePanel here

I'm interested in creating the equivalent RelativePanel control for WPF. I got an implementation close to it, but the layout cycle isn't completely in sync with the UWP one. It would be really helpful if RelativePanel was part of the open sourced controls, so that we can create consistent implementations across the XAML platforms.

Thanks for raising this!  I'm curious are there any changes/improvements folks would like in this feature, too, or just getting access to see the control's source code?

Well my immediate need is to see how the layout cycle is implemented, so that the exact behavior can be duplicated in WPF. Having said that knowing how Panels work is a great learning experience, as they form the basic for a lot of advanced controls.

@kodamaSakanu I have an implementation too. Afaik it matches the behavior of UWP, but there are so many edge cases I likely don't handle: https://github.com/dotMorten/UniversalWPF/tree/master/src/UniversalWPF

Okay thanks.  We’d certainly like to help move all our XAML developers forward and help bridge folks to the latest WinUI / UWP XAML.  In general, there are a number of controls we need to migrate into WinUI - e.g. see recent requests for ComBox features - and as we do so it’s not always just a “copy and paste” from the in-box code as we need to ensure the code is layered over the core os the right way, follows the coding and engineering practices that support good open source collaboration, etc.  One option to get you the code is to wait until we’ve done that migration and moved the feature into WinUI, but given your feedback we’re going to look into if there are ways for us to accelerate that and possibly do more of the migration itself in the open which would let you see more of the code sooner.

This would be included in WinUI 3.0, which we plan to make open source.
So, closing this in favor of the general WinUI 3.0 issue #717


Correct to_bgra docstring for DynamicImage

The docstring says RGBA when it should say BGRA, this PR corrects it.

Looks good to me


'version' key still missing in the manifest

STILL NOT FIXED.
" ERROR (SyncWorker_1) [homeassistant.loader] The custom integration 'sonoff' does not have a valid version key (None) in the manifest file and was blocked from loading. See https://developers.home-assistant.io/blog/2021/01/29/custom-integration-changes#versions for more details"
Home Assistant version 2021.7.3

This component was updated long time ago:
https://github.com/AlexxIT/SonoffLAN/blob/11c6a748404caa6435ca08fe5198bd378188187c/custom_components/sonoff/manifest.json#L9
Are yo usure you are suing SonoffLAN ?
Simone

Fixed long time ago


Improve docstrings, a few tests.

Add docstrings to most things
Add tests for some untested functions
Few small code fixes, mostly to improve consistent behavior.

Looks great; happy to have it merged after the minor typo fixes noted.

BTW, is it possible to make the tests pass before the conda package is built?  I thought it was failing due to dynd, but since that's no longer needed, it must be something else...

If you click on the build failure (red x on the commit), you can see what made the test fail. In this case it was a failure to import bokeh when I added the --doctest-modules flag for python 2.7. All tests on master have been passing, and of course we don't merge code with tests failing.


Running Docker compose up but the app doesn't update to the newest version

Hi there
I run docker compose up command to update serpbear to newest version (I built it on ubuntu server)  but it's still showing version 0.3.4
What am I wrong?

it's showing old version 0.3.4

it's ok now, your instruction is missing. I need to run docker compose pull before re-run docker compose up

Thanks for your input. I have updated the docs.


Fixed typos and improved readability of docs

Fixed some typos like compeltions - completions,  not - now, and others.
Removed unnecessary italics on text 'skills'.
Restructured a few sentences to improve readability

Please fix the conflicts if you'd like us to review.


Respect leading spaces for rendering

Is your feature request related to a problem? Please describe.
I'm trying to align a cursor for menu entries.
                let cursor = if some_condition {
                    RichText::new("> ").color(Color32::LIGHT_GRAY).text_style(TextStyle::Monospace)
                } else {
                    RichText::new("  ").color(Color32::LIGHT_GRAY).text_style(TextStyle::Monospace)
                };

                ui.label(cursor);
                ui.label("Some Text");

Currently, the else condition never renders. There has to be an actual symbol besides just regular spaces. I'd prefer not to mess with the fonts as I like the default ones.
Describe the solution you'd like
If there is a space character it was most likely put there deliberately and thus should be rendered by default.
Describe alternatives you've considered
Using a custom font with some space-equivalent-yet-renderable character.
Additional context
I've tried looking into the source code but unfortunately couldn't figure out where the is_empty call is made that is causing this.

I came across your issue when I ran into similar problem. Was trying to get a label to always be 3 digits wide (using leading spaces) and horizontally centered on a row.
Here's a workaround I came up with that works for me:
// self.val is f32
let val_formatted = format!("{:>3}", self.val);
let text_size = WidgetText::from(val_formatted.clone())
    .into_galley(ui, None, ui.available_width(), TextStyle::Monospace)
    .size();
ui.allocate_ui_with_layout(text_size, Layout::right_to_left(), |ui| {
    ui.label(RichText::new(val_formatted).monospace());
});

This seems like a bug in the text layout code!

FWIW, you can use the non-breaking space encoded \u{a0} to get these examples to work. E.g.:
RichText::new("\u{a0}\u{a0}")


Sass support & @import

Is it possible to get Sass running with this plugin, too? What about @import?

I guess this is not possible, as the page is being opened by PhantomJS and this is a "normal" browser (think of Chrome, Firefox - they also do not support that).
You might want to do your own preprocessing (eg. node-sass) upfront and link that generated stylesheet in the html that gets sent to pdf.create.


FixAll on save doesn't pick up file change from organize imports

From https://github.com/microsoft/vscode/issues/88131#issuecomment-608048683
Repro

Clone https://github.com/JacksonKearl/eslint-organizeImports-codeactions

This sets:
"editor.codeActionsOnSave": [
	"source.organizeImports",
	"source.fixAll.eslint",
],

Open index.ts and save the file a few times

Bug
Notice the trailing command being added and removed on each save

Here's the ESlint trace of what happens:
VS Code update the document using organize imports (removing the trailing comma)
[Trace - 12:35:49 PM] Sending notification 'textDocument/didChange'.
Params: {
    "textDocument": {
        "uri": "file:///Users/matb/projects/san/eslint-organizeImports-codeactions/index.ts",
        "version": 33
    },
    "contentChanges": [
        {
            "range": {
                "start": {
                    "line": 6,
                    "character": 14
                },
                "end": {
                    "line": 6,
                    "character": 15
                }
            },
            "rangeLength": 1,
            "text": ""
        }
    ]
}

VS Code request code actions for fix all
[Trace - 12:35:49 PM] Sending request 'textDocument/codeAction - (98)'.
Params: {
    "textDocument": {
        "uri": "file:///Users/matb/projects/san/eslint-organizeImports-codeactions/index.ts"
    },
    "range": {
        "start": {
            "line": 0,
            "character": 0
        },
        "end": {
            "line": 16,
            "character": 0
        }
    },
    "context": {
        "diagnostics": [],
        "only": [
            "source.fixAll.eslint"
        ]
    }
}

ESlint updates it's diagnostics
[Trace - 12:35:49 PM] Received notification 'textDocument/publishDiagnostics'.
Params: {
    "uri": "file:///Users/matb/projects/san/eslint-organizeImports-codeactions/index.ts",
    "diagnostics": [
        {
            "message": "Insert `,`",
            "severity": 1,
            "source": "eslint",
            "range": {
                "start": {
                    "line": 6,
                    "character": 14
                },
                "end": {
                    "line": 6,
                    "character": 14
                }
            },
            "code": {
                "value": "prettier/prettier",
                "target": "https://github.com/prettier/eslint-plugin-prettier#options"
            }
        }
    ]
}

ESLint then returns no code actions
[Trace - 12:35:49 PM] Received response 'textDocument/codeAction - (98)' in 23ms.
Result: []

I looked into this and for me it looks like this: TypeScript and ESLint do have two conflicting fixes for the organize import.

Typescript wants this (e.g. one long line):

import { LongName, ReallyQuiteSuperLongName, ReallyQuiteSuperMegaLongName, ReallySuperLongName, ReallyVeryQuiteSuperMegaLongName, SuperLongName } from './utils'

and ESLint wants this (one import per line)

import {
	LongName,
	ReallyQuiteSuperLongName,
	ReallyQuiteSuperMegaLongName,
	ReallySuperLongName,
	ReallyVeryQuiteSuperMegaLongName,
	SuperLongName,
} from './utils'

I don't see how ESLint can address this.
@mjbvz and @JacksonKearl can you please outline what in your opinion ESLint should do to avodi this?

This conflicting behavior results in the fact that the import switches on every Save.

There is something else wrong as well since I see this in the console:
[Error - 3:04:36 PM] Failed to apply command: eslint.applyDisableLine
[Error - 3:04:36 PM] Failed to apply command: eslint.applyDisableFile

The problematic piece seems to be
	"editor.codeActionsOnSave": [
		"source.organizeImports",
		"source.fixAll.eslint",
	],

I always thought the format is
	"editor.codeActionsOnSave": {
		"source.organizeImports": true,
		"source.fixAll.eslint": true,
	},

Change the settings to
	"editor.codeActionsOnSave": {
		"source.organizeImports": true,
		"source.fixAll.eslint": true,
	},

basically avoid printing the Failed to apply command: message but make the editor flicker due to the problem that the actions run on save have conflicting goals (e.g. TypeScript and ESLint). I will change the code in ESLint to not fall over
"editor.codeActionsOnSave": [
		"source.organizeImports",
		"source.fixAll.eslint",
]

To make things clear: I fixed the fact that the setting was not read correctly. However the sertup itself still doesn't result in a user friendly behaviour :-). But IMO there is nothing ESLint can do to address this.

The idea with #88131 was to allow "editor.codeActionsOnSave" to be an array so you can configure "organize the imports first, then run the code through eslint":
"editor.codeActionsOnSave": [
		"source.organizeImports",
		"source.fixAll.eslint",
]

If instead you use:
	"editor.codeActionsOnSave": {
		"source.organizeImports": true,
		"source.fixAll.eslint": true,
	},

The fixAll gets run first, then the organize imports is run so the end state isn't eslint conformant.
If however we use the array syntax, it should run organize imports, converting the working copy to have organized imports but not be eslint conformant, then immediately run eslint, preserving the order of the imports but not their formatting, and making the document eslint conformant. And only then should it be persisted to disk.
What I'm observing instead is that I save one time and the organizeImports output is written to disk. Then I save again and the eslint output is written to disk. And it keeps alternating between these two.

@mjbvz and @JacksonKearl can you please outline what in your opinion ESLint should do to avodi this?

All eslint should need to do is the same formatting it has always done, the input will be organizeImports conformant but the output doesn't need to be.
@mjbvz maybe organize imports could be changed to not care about formatting, only order? So if the imports are in correct order it doesn't touch them and we don't get flicker.

Thanks for the details @JacksonKearl. That is correct
@dbaeumer I was surprised to see that changing editor.codeActionsOnSave setting to an array caused issues for eslint. VS Code should ask eslint for code actions for source.fixAll.eslint  on save. Why does eslint need to read this setting itself?

@mjbvz the current reason left is settings migration for which we don't have any store in VS Code except that the extension does it itself.


AST and Serialization support for generic subscripts

Some more baby steps along the way of implementing generic subscripts.
Next up:

Parser and Sema support
SILGen
Code completion

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - cc54361b2a2832885adaea38fb5bcaf6e363a2a8
Test requested by - @slavapestov

@swift-ci Please smoke test macOS


Remove vulnerabilities from the package per npm audit

🐛 Describe the bug

Some critical/high vulnerabilities shows when one build the application.
📜 To Reproduce
Steps to reproduce the behavior:
Run npm install
See output:
% npm install

up to date, audited 2018 packages in 5s

143 packages are looking for funding
  run `npm fund` for details

32 vulnerabilities (1 low, 16 moderate, 12 high, 3 critical)

To address issues that do not require attention, run:
  npm audit fix

To address all issues (including breaking changes), run:
  npm audit fix --force

Run `npm audit` for details.

🕵 Expected behavior

There should not be critical or high vulnerabilities.
📚 Version of Software Used

🩺 Test Data / Additional context

🏞Screenshots

🖥 System Info

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

🦄 Related requirements
⚙ Engineering Details

@tloubrieu-jpl
Running npm audit fix will fix 6 moderate, 1 high and 1 critical
All the left overs are caused by one react dependency<EMAIL_ADDRESS>which is a breaking change.
This one update will break react and all the remaining dependencies and the doi-ui. They need to be fixed by the dependency's developers.
Also just some more details about the vulnerabilities. The vulnerabilities are on the tool side. Not exactly on the built code side. So deployed versions would not have all these vulnerabilities exposed. However without reading into each individual dependency's code we wouldn’t be able to tell.

🕐 Updated dependencies to remove most of the vulnerabilities

Re-opened because of I&T of build 13.0

These vulnerabilities are related to the build process but once the application is build these vulnerabilities are not in the online application.
We (dev team ) need to provide a better way to asses the security of the application for the I&T team to test.

@tloubrieu-jpl which Node/NPM version are we using for this repo?

@eddiesarevalo can you answer Miguel ? Thanks

@eddiesarevalo answered my question in today's breakout:
npm -v
8.19.3
node -v
v19.1.0

@eddiesarevalo @tloubrieu-jpl running npm install  yields 31 vulnerabilities (1 low, 1 moderate, 20 high, 9 critical):
node -v                                                                                ─╯
v19.1.0
npm -v                                                                                ─╯
8.19.3
npm install
npm WARN deprecated<EMAIL_ADDRESS>Please use @jridgewell/sourcemap-codec instead
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-terser
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-babel.
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained
npm WARN deprecated<EMAIL_ADDRESS>Moved to 'npm install @sideway/address'
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained
npm WARN deprecated<EMAIL_ADDRESS>Switch to 'npm install joi'
npm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated
npm WARN deprecated<EMAIL_ADDRESS>This functionality has been moved to @npmcli/fs
npm WARN deprecated<EMAIL_ADDRESS>flatten is deprecated in favor of utility frameworks such as lodash.
npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
npm WARN deprecated<EMAIL_ADDRESS>Use your platform's native performance.now() and performance.timeOrigin.
npm WARN deprecated<EMAIL_ADDRESS>some dependency vulnerabilities fixed, support for node < 10 dropped, and newer ECMAScript syntax/features added
npm WARN deprecated<EMAIL_ADDRESS>babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.
npm WARN deprecated<EMAIL_ADDRESS>This SVGO version is no longer supported. Upgrade to v2.x.x.
npm WARN deprecated<EMAIL_ADDRESS>You can now upgrade to @mui/material. See the guide: https://mui.com/guides/migration-v4/
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade to version 7 or higher. Older versions may use Math.random() in certain circumstances, which is known to be problematic. See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 does not receive security updates since 2019. Upgrade to chokidar 3 with 15x fewer dependencies
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Some versions have web compatibility issues. Please, upgrade your dependencies to the actual version of core-js.
npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
npm WARN deprecated<EMAIL_ADDRESS>fsevents 1 will break on node v14+ and could be using insecure binaries. Upgrade to fsevents 2.
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 does not receive security updates since 2019. Upgrade to chokidar 3 with 15x fewer dependencies
npm WARN deprecated<EMAIL_ADDRESS>See https://github.com/lydell/source-map-resolve#deprecated

added 2034 packages, and audited 2035 packages in 1m

187 packages are looking for funding
run npm fund for details

31 vulnerabilities (1 low, 1 moderate, 20 high, 9 critical)

To address issues that do not require attention, run:
npm audit fix

To address all issues (including breaking changes), run:
npm audit fix --force

Run npm audit for details.

thanks @miguelp1986 , @eddiesarevalo should we run npm audit fix so that the critical vulnerability occurring in the build phase (and which do not apply when the application is deployed) do not show anymore ?

@tloubrieu-jpl Yes, you can run npm audit fix often and then do quick testing to see if it still runs and builds. The audit will apply fixes that are available but it won't fix anything if none are available.

@eddiesarevalo @tloubrieu-jpl npm audix fix didn't fix the issue:
npm audit fix

added 1 package, removed 3 packages, changed 4 packages, and audited 2035 packages in 4s

189 packages are looking for funding
  run `npm fund` for details

# npm audit report

ansi-html  <0.0.8
Severity: high
Uncontrolled Resource Consumption in ansi-html - https://github.com/advisories/GHSA-whgm-jr23-g3j9
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/ansi-html
  @pmmmwh/react-refresh-webpack-plugin  <=0.5.0-rc.6
  Depends on vulnerable versions of ansi-html
  node_modules/@pmmmwh/react-refresh-webpack-plugin
    react-scripts  >=0.10.0-alpha.328cb32e
    Depends on vulnerable versions of @pmmmwh/react-refresh-webpack-plugin
    Depends on vulnerable versions of @svgr/webpack
    Depends on vulnerable versions of optimize-css-assets-webpack-plugin
    Depends on vulnerable versions of react-dev-utils
    Depends on vulnerable versions of webpack
    Depends on vulnerable versions of webpack-dev-server
    Depends on vulnerable versions of workbox-webpack-plugin
    node_modules/react-scripts
  webpack-dev-server  2.0.0-beta - 4.7.2
  Depends on vulnerable versions of ansi-html
  Depends on vulnerable versions of chokidar
  Depends on vulnerable versions of selfsigned
  node_modules/webpack-dev-server

browserslist  4.0.0 - 4.16.4
Severity: moderate
Regular Expression Denial of Service in browserslist - https://github.com/advisories/GHSA-w8qv-6jwh-64r5
fix available via `npm audit fix`
node_modules/react-dev-utils/node_modules/browserslist
  react-dev-utils  0.5.2 - 12.0.0-next.60
  Depends on vulnerable versions of browserslist
  Depends on vulnerable versions of immer
  Depends on vulnerable versions of loader-utils
  Depends on vulnerable versions of recursive-readdir
  Depends on vulnerable versions of shell-quote
  node_modules/react-dev-utils

ejs  <3.1.7
Severity: critical
ejs template injection vulnerability - https://github.com/advisories/GHSA-phwq-j96m-2c2q
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/ejs
  @surma/rollup-plugin-off-main-thread  <=2.1.0
  Depends on vulnerable versions of ejs
  node_modules/@surma/rollup-plugin-off-main-thread
    workbox-build  5.0.0-alpha.0 - 6.3.0
    Depends on vulnerable versions of @surma/rollup-plugin-off-main-thread
    node_modules/workbox-build
      workbox-webpack-plugin  5.0.0-alpha.0 - 5.1.4 || 6.2.2 - 6.3.0
      Depends on vulnerable versions of workbox-build
      node_modules/workbox-webpack-plugin

glob-parent  <5.1.2
Severity: high
glob-parent before 5.1.2 vulnerable to Regular Expression Denial of Service in enclosure regex - https://github.com/advisories/GHSA-ww39-953v-wcq6
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/watchpack-chokidar2/node_modules/glob-parent
node_modules/webpack-dev-server/node_modules/glob-parent
  chokidar  1.0.0-rc1 - 2.1.8
  Depends on vulnerable versions of glob-parent
  node_modules/watchpack-chokidar2/node_modules/chokidar
  node_modules/webpack-dev-server/node_modules/chokidar
    watchpack-chokidar2  *
    Depends on vulnerable versions of chokidar
    node_modules/watchpack-chokidar2
      watchpack  1.7.2 - 1.7.5
      Depends on vulnerable versions of watchpack-chokidar2
      node_modules/watchpack
        webpack  4.44.0 - 4.46.0
        Depends on vulnerable versions of watchpack
        node_modules/webpack

immer  <=9.0.5
Severity: critical
Prototype Pollution in immer - https://github.com/advisories/GHSA-c36v-fmgq-m8hx
Prototype Pollution in immer - https://github.com/advisories/GHSA-33f9-j839-rf8h
fix available via `npm audit fix`
node_modules/react-dev-utils/node_modules/immer

loader-utils  2.0.0 - 2.0.3
Severity: critical
Prototype pollution in webpack loader-utils - https://github.com/advisories/GHSA-76p3-8jx3-jpfq
loader-utils is vulnerable to Regular Expression Denial of Service (ReDoS) - https://github.com/advisories/GHSA-hhq3-ff78-jv3g
loader-utils is vulnerable to Regular Expression Denial of Service (ReDoS) via url variable - https://github.com/advisories/GHSA-3rfm-jhwj-7488
fix available via `npm audit fix`
node_modules/loader-utils

minimatch  <3.0.5
Severity: high
minimatch ReDoS vulnerability - https://github.com/advisories/GHSA-f8q6-p94x-37v3
fix available via `npm audit fix`
node_modules/recursive-readdir/node_modules/minimatch
  recursive-readdir  1.2.0 - 2.2.2
  Depends on vulnerable versions of minimatch
  node_modules/recursive-readdir

node-forge  <=1.2.1
Severity: high
Prototype Pollution in node-forge debug API. - https://github.com/advisories/GHSA-5rrq-pxf6-6jx5
URL parsing in node-forge could lead to undesired behavior. - https://github.com/advisories/GHSA-gf8q-jrpm-jvxq
Improper Verification of Cryptographic Signature in `node-forge` - https://github.com/advisories/GHSA-2r2c-g63r-vccr
Open Redirect in node-forge - https://github.com/advisories/GHSA-8fr3-hfg3-gpgp
Improper Verification of Cryptographic Signature in node-forge - https://github.com/advisories/GHSA-cfm4-qjh2-4765
Improper Verification of Cryptographic Signature in node-forge - https://github.com/advisories/GHSA-x4jg-mjrx-434g
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/node-forge
  selfsigned  1.1.1 - 1.10.14
  Depends on vulnerable versions of node-forge
  node_modules/selfsigned

nth-check  <2.0.1
Severity: high
Inefficient Regular Expression Complexity in nth-check - https://github.com/advisories/GHSA-rp65-9cf3-cjxr
fix available via `npm audit fix --force`
Will install<EMAIL_ADDRESS>which is a breaking change
node_modules/nth-check
  css-select  <=3.1.0
  Depends on vulnerable versions of nth-check
  node_modules/css-select
    svgo  1.0.0 - 1.3.2
    Depends on vulnerable versions of css-select
    node_modules/svgo
      @svgr/plugin-svgo  <=5.5.0
      Depends on vulnerable versions of svgo
      node_modules/@svgr/plugin-svgo
        @svgr/webpack  4.0.0 - 5.5.0
        Depends on vulnerable versions of @svgr/plugin-svgo
        node_modules/@svgr/webpack
      postcss-svgo  4.0.0-nightly.2020.1.9 - 5.0.0-rc.2
      Depends on vulnerable versions of svgo
      node_modules/postcss-svgo
        cssnano-preset-default  <=4.0.8
        Depends on vulnerable versions of postcss-svgo
        node_modules/cssnano-preset-default
          cssnano  4.0.0-nightly.2020.1.9 - 4.1.11
          Depends on vulnerable versions of cssnano-preset-default
          node_modules/cssnano
            optimize-css-assets-webpack-plugin  3.2.1 || 5.0.0 - 5.0.8
            Depends on vulnerable versions of cssnano
            node_modules/optimize-css-assets-webpack-plugin

shell-quote  <=1.7.2
Severity: critical
Improper Neutralization of Special Elements used in a Command in Shell-quote - https://github.com/advisories/GHSA-g4rg-993r-mgx7
fix available via `npm audit fix`
node_modules/shell-quote

xlsx  *
Severity: high
Prototype Pollution in sheetJS - https://github.com/advisories/GHSA-4r6h-8v6p-xvw6
No fix available
node_modules/xlsx

32 vulnerabilities (1 low, 1 moderate, 21 high, 9 critical)

To address issues that do not require attention, run:
  npm audit fix

To address all issues possible (including breaking changes), run:
  npm audit fix --force

Some issues need review, and may require choosing
a different dependency.

npm install                                                                                ─╯

up to date, audited 2035 packages in 1s

189 packages are looking for funding
  run `npm fund` for details

32 vulnerabilities (1 low, 1 moderate, 21 high, 9 critical)

To address issues that do not require attention, run:
  npm audit fix

To address all issues possible (including breaking changes), run:
  npm audit fix --force

Some issues need review, and may require choosing
a different dependency.

Run `npm audit` for details.

npm start                                                                                ─╯

><EMAIL_ADDRESS>start
> react-scripts start

ℹ ｢wds｣: Project is running at http://<IP_ADDRESS>:3000/
ℹ ｢wds｣: webpack output is served from
ℹ ｢wds｣: Content not from webpack is served from /Users/MPena/repositories/doi-ui/public
ℹ ｢wds｣: 404s will fallback to /
Starting the development server...

Browserslist: caniuse-lite is outdated. Please run:
  npx update-browserslist-db@latest
  Why you should do it regularly: https://github.com/browserslist/update-db#readme
/Users/MPena/repositories/doi-ui/node_modules/react-scripts/scripts/start.js:19
  throw err;
  ^

Error: error:0308010C:digital envelope routines::unsupported
    at new Hash (node:internal/crypto/hash:71:19)
    at Object.createHash (node:crypto:140:10)
    at module.exports (/Users/MPena/repositories/doi-ui/node_modules/webpack/lib/util/createHash.js:135:53)
    at NormalModule._initBuildHash (/Users/MPena/repositories/doi-ui/node_modules/webpack/lib/NormalModule.js:417:16)
    at /Users/MPena/repositories/doi-ui/node_modules/webpack/lib/NormalModule.js:452:10
    at /Users/MPena/repositories/doi-ui/node_modules/webpack/lib/NormalModule.js:323:13
    at /Users/MPena/repositories/doi-ui/node_modules/loader-runner/lib/LoaderRunner.js:367:11
    at /Users/MPena/repositories/doi-ui/node_modules/loader-runner/lib/LoaderRunner.js:233:18
    at context.callback (/Users/MPena/repositories/doi-ui/node_modules/loader-runner/lib/LoaderRunner.js:111:13)
    at /Users/MPena/repositories/doi-ui/node_modules/babel-loader/lib/index.js:59:103 {
  opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],
  library: 'digital envelope routines',
  reason: 'unsupported',
  code: 'ERR_OSSL_EVP_UNSUPPORTED'
}

Node.js v19.1.0


docs: update context

structure for context object changed a little

I could be missing more things @ardatan


[Feature]:Add Social Media Icons to the Homepage

Is your proposal related to a problem?
I'm frustrated when users don't have easy access to our social media platforms. Having visible social media icons on the homepage would help users easily connect and engage with us across platforms.
Describe the solution you'd like
I propose adding a vertical sidebar with social media icons (like Instagram, Facebook, YouTube, LinkedIn, GitHub, Discord, and X) on the left side of the homepage. The icons should:

Be fixed in position, so they stay visible as the user scrolls.
Be clickable, redirecting users to the respective social media pages.
Have hover effects for better user interaction.
Be responsive to different screen sizes (desktop, tablet, mobile).

Describe alternatives you've considered

Placing social media links in the footer, but this may reduce visibility as it requires users to scroll down.
Using a popup modal for social links, which could be intrusive.

Additional context
I've attached a screenshot as an example of how the social media icons could be displayed. The design should align with the overall aesthetic of the website and ensure accessibility for all users.

For reference:


[New game]: Puzzle Game

🎮 Game Request
The game logic of the puzzle game involves dragging and dropping puzzle pieces to solve the puzzle.
Point down the features
Game Board:
The game board is a 5x5 grid represented by the  element with the id "board".
Each cell of the grid contains an  element with a blank image initially.
Puzzle Pieces:
The puzzle pieces are represented by the  elements within the  element with the id "pieces".
Each puzzle piece is randomly assigned an image from a set of puzzle images.
The puzzle pieces are draggable, and dragging is enabled using event listeners.
Drag and Drop Functionality:
When a puzzle piece is dragged (dragstart event), the reference to the dragged tile is stored in the currTile variable.
The dragover event is handled to prevent the default behavior.
The dragenter and dragleave events are used to provide visual feedback when dragging a puzzle piece over another.
When a puzzle piece is dropped onto another (drop event), the reference to the target tile is stored in the otherTile variable.
The dragend event is triggered after the drop is completed, and the puzzle pieces' images are swapped.
The turn count is incremented, and the updated turn count is displayed on the page.
Puzzle Generation:
The puzzle pieces are generated randomly by shuffling an array of puzzle image names.
The shuffled array is used to assign images to the puzzle pieces.
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @SJ-Kumar !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @SJ-Kumar!
We are already having a similar game request in #579 👀
Make sure you come up with a cool unique idea 😀
Waiting for your new game idea 💗.

Hey @SJ-Kumar ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Can't Mint or Burn Tokens.

pragma solidity ^0.4.18;
import "./MintableToken.sol";
import "./BurnableToken.sol";
import "./StandardToken.sol";

contract Paradisecoin is StandardToken, MintableToken, BurnableToken {
    string public constant name = "APR Token";
    string public constant symbol = "APR";
    uint8 public constant decimals = 18;
    uint256 public constant INITIAL_SUPPLY = 10000 * (10 ** uint256(decimals));

  function burnFrom(address _from, uint256 _value) public {
    require(_value <= allowed[_from][msg.sender]);
    // Should https://github.com/OpenZeppelin/zeppelin-solidity/issues/707 be accepted,
    // this function needs to emit an event with the updated approval.
    allowed[_from][msg.sender] = allowed[_from][msg.sender].sub(_value);
    _burn(_from, _value);
  }
    constructor() public {
        totalSupply_ = INITIAL_SUPPLY;
        balances[msg.sender] = INITIAL_SUPPLY;
        emit Transfer(0x0, msg.sender, INITIAL_SUPPLY);
    }
}

Linux Ubuntu 16
New Remix IDE
Publishes the contract fine with initial amount of coins.  When I try to run the mint as well as the burn function to my address, it does nothing to the amount of coins.  I'm making the calls through the remix IDE once the contracts deployed.  Is there something wrong with my contract why would this be happening?
Thanks.

Hello @Dycedlp!
Which version of OpenZeppelin are you using? In 2.0 (the latest version), MintableToken has been replaced by ERC20Mintable.

Ok this is 1.12v @nventuro


Network chat / messaging

Once the new plugin system is complete, it should be trivial to add a network chat / messaging plugin. This would be implemented as a custom item type and received messages could be displayed with libnotify or the current platform's native equivalent. (Alternatively, a custom UI could be implemented.)

Great Idea!

This would solve sending text snippets, such urls etc...


Playlist example bugs

Live demo here
Issue 1
If Single Non-linear Inline (Banner) ad is loaded after linear ad was loaded before, banner do not show.
Steps to reproduce
Start Video 3, skip the ad and then start Video 4.
Result
Banner is hidden. If you refresh and start Video 4 directly banner is working.
I think the problem is here
Issue 2
If the ad has a mid-roll, and after mid-roll ends you change to another video with ad that also has mid-roll, the second video ad starts directly at mid-roll and then plays pre-roll.
Steps to reproduce:
Start Video 1, wait mid-roll end, then change to Video 2.
Result
Video 2 ad will start from mid-roll and then play pre-roll.
Questions

Why changing videos while running ad is prevented in the demo?
Is the plugin supposed unload and load new ads properly if source and tag change is made while ad is playing?

I have to some issues with this, but I am not yet sure if its my code or the plugin.

@shawnbuso any feedback please?

Sorry for the delay - I think I have a fix for issue 1 - we were hiding the ad display container on resetIMA, and we shouldn't have been doing that. That should be fixed with PR #350.
Re: issue 2, it looks like this could be an IMA SDK issue. I'm looking into it with the rest of the team and I'll let you know what we find. Thanks!

Tested the fix for issue 1 on my fork. Seems good :)

The second issue here was fixed by #374 (it says it fixes the second issue in #354 but that's a dupe of the second issue here), so we should be good to go :)


Create acm.canonical.data.json

See: https://jobs.acm.org/jobs/results/keyword/internship
Note that a lot of the hits aren't internships, I had to scroll to the second page to find any. So, some post-processing may be necessary to filter out the bad hits.

A potentially better link that only shows internships: https://jobs.acm.org/jobs/results/title/Internship/United+States?radius=5

I was mostly able to make an acm scraper and the acm.canonical.data.json file.


saltpack Sign/Verify implementation

Adding (many) more tests, but PTAL @maxtaco when you get a chance.

I hope to take a look soon, but also @oconnor663 really should take a look too

Thanks @maxtaco and @oconnor663
Will be adding a slew of tests, but the basics working/tested.

I'm about to take off for Memphis, but I should have time to look at this later this afternoon. Very excited to see it in action :)
By the way, let me know if you guys think the nonce part of the signing spec is ridiculous.

Made a first pass, looks really good.  Some comments in anticipation of getting verify test coverage up (i.e., eliminating error cases that we'll never hit).

I hit those error cases during development when things weren't fully implemented, that's why they are in there, but I'll remove them now.

All PR comments should be addressed now.  Will be working on test coverage now in this branch.

Great.  jack should take a look too

Looks great to me. Thanks Patrick! :shipit:

Patrick, how did you generate the test vectors? Are they regeneratable?

The fuzz ones?  I generated some of the corpus, but the rest are generated by go-fuzz.  The author recommends storing them in version control:  "Consider committing the generated inputs to your source control system, this will allow you to restart go-fuzz without losing previous work."  https://github.com/dvyukov/go-fuzz

Super cool


damage is bugged

all weapon damage is fixed to like 100, i would think that damage wasnt implemented yet, but thunder self damage works properly

isnt that like 0.0001$? 👽

isnt that like 0.0001$? 👽

1500 rubles

This project is NOT developed anymore. You will NOT get any support.
Assasans now working at https://araumi.org
Issue closed

This project is NOT developed anymore. You will NOT get any support.
Assasans now working at https://araumi.org Issue closed

just 1500 rubles

This project is NOT developed anymore. You will NOT get any support.
Assasans now working at https://araumi.org Issue closed

ok bro no need to get so mad


Set origin and size correctly in LEF; might break a number of things

This looks like the correct LEF. I don't know what will break downstream.

Doesn't seem to break the CI. We should check it in. Please review.


Fix for Lead to Campaign Member creation bug

Changes

Campaign Members were not being created correctly off of Interactions when Leads already existed in the system. When duplicate Leads were found, we were not setting the Lead__c field on Interaction properly and therefore Campaign Members were never created for these Interactions.
Solve an issue where the SaveResult value of a Lead insert could be a failure, and handle it properly.
Add Unit Test + formatting.

Closing this PR in favor of the newest version.


[Q] How to use 'numFmt'?

💬 Questions and Help
I have read the documentation concerning this feature but I'm not sure how to actually use this.
It seems like the documentation here is pretty lacking. Is this some kind of standardized excel feature that I should be able to find documentation on elsewhere (I've looked)?
What I want to do is take a number like 16.454857719662395 and turn have it be formatted like this:
16.45%
I'm attempting to do something like this:
style: {
  numFmt: '0.00%',
}

But this results in a value like:
1645.49%

Which is obviously not what I want. It seems like the software is thinking that I'm giving it a rate and asking for it to be converted into a percentage. All I really want to do is supply a number that is a percentage and format it with 2 decimal places and a percent sign at the end.
Thanks for the help!

need same question answered! Except I just need to add a % to the value

need same question answered! Except I just need to add a % to the value

You can try directly in excel, then go on "Other format, Custom". You got the string to copy.
To get 16%, use '0_%',
To get 16.45%, use '0.00%'

I have a similar issue
sample value => 15.209773191862164
desired result => 15,21%
by using '0.00%' I get this 1521,00
I tried these setup values:
style : {
    numFmt: #,##0.00;[Red]-#,##0.00'
}

since I want to highlight negative values in red.
Thanks

Write it yourself. The other value must be of type number, so I wrote a compatible conversion function.

const input  = [{
a:'1',
b:'1.2',
c:'123,111.00',
d:'google',
e:'12%',
f:12,
}]

const output  = [{
a: 1,
b:1.2,
c:123111.00,
d:'google',
e:0.12,
f:12,
}]

import numeral from 'numeral'

const  isNumString =input => String(Number(input)) === input ||
      /^-?\d{1,3}(,\d{3})*(\.\d{1,2})?$/.test(input)

const textToNum = (input: string | number | null | undefined) => {
  let output = undefined as any
  if (typeof input === 'string') {
    if (input.endsWith('%')) {
      output = Number(input.replace('%', '')) / 100
    } else if (isNumString(input)) {
      // 判断是否数字类型字符串
       output = numeral(input).value()
    } else {
      // 字符串
      output = input
    }
  } else if (typeof input === 'number') {
    output = input
  } else {
    console.warn('未知类型', input)
  }

  return output
}
export const excelTableFormatted = (tableData) =>
  tableData
    ? tableData.map((row) => {
        return Object.fromEntries(
          Object.entries(row).map(([k, v]) => [k, textToNum(v)])
        )
      })
    : []


Add patch for libcudacxx memory resource.

Description
Adds a patch from https://github.com/NVIDIA/libcudacxx/pull/439/ for libcudacxx 2.1.0, to support ongoing memory resource work.
cc: @miscco
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The cmake-format.json is up to date with these changes.
[ ] I have added new files under rapids-cmake/

[ ] I have added include guards (include_guard(GLOBAL))
[ ] I have added the associated docs/ rst file and update the api.rst

Thanks a lot, totally forget to remove the testing changes from the patch I send you

/merge


Crash Report

Fatal Crash occurred on Thread named 'Thread-7'
Unix Time :<PHONE_NUMBER>279
LocalTime : Aug 17, 2024 22:13:54
ACTIVE_CODENAMES = []
app_version = 2.6.0
device_model = Redmi Note 7
RESOURCES_SDK_INT = 29
SECURITY_PATCH = 2021-10-01
BASE_OS =
RELEASE = 10
CODENAME = REL
manufacturer = Xiaomi
SDK_INT = 29
PREVIEW_SDK_FINGERPRINT = REL
FIRST_SDK_INT = 28
PREVIEW_SDK_INT = 0
SDK = 29
brand = xiaomi
supported_abi = [arm64-v8a,armeabi-v7a,armeabi]
INCREMENTAL = V<IP_ADDRESS>.QFGMIXM
Error Message : listFiles(...) must not be null
Error Cause : null
Error StackTrace :
java.lang.NullPointerException: listFiles(...) must not be null
at com.rk.xededitor.MainActivity.treeview2.TreeViewAdapter$Companion.merge(TreeViewAdapter.kt:214)
at com.rk.xededitor.MainActivity.treeview2.TreeView.init$lambda$3(TreeView.kt:35)
at com.rk.xededitor.MainActivity.treeview2.TreeView.$r8$lambda$JBvSDOOqbKRJBQ0KIXzLdcjWoLM(Unknown Source:0)
at com.rk.xededitor.MainActivity.treeview2.TreeView$$ExternalSyntheticLambda0.run(D8$$SyntheticClass:0)
at java.lang.Thread.run(Thread.java:919)

Fatal Crash occurred on Thread named 'Thread-7' Unix Time :<PHONE_NUMBER>279 LocalTime : Aug 17, 2024 22:13:54
ACTIVE_CODENAMES = [] app_version = 2.6.0 device_model = Redmi Note 7 RESOURCES_SDK_INT = 29 SECURITY_PATCH = 2021-10-01 BASE_OS = RELEASE = 10 CODENAME = REL manufacturer = Xiaomi SDK_INT = 29 PREVIEW_SDK_FINGERPRINT = REL FIRST_SDK_INT = 28 PREVIEW_SDK_INT = 0 SDK = 29 brand = xiaomi supported_abi = [arm64-v8a,armeabi-v7a,armeabi] INCREMENTAL = V<IP_ADDRESS>.QFGMIXM
Error Message : listFiles(...) must not be null Error Cause : null Error StackTrace :
java.lang.NullPointerException: listFiles(...) must not be null at com.rk.xededitor.MainActivity.treeview2.TreeViewAdapter$Companion.merge(TreeViewAdapter.kt:214) at com.rk.xededitor.MainActivity.treeview2.TreeView.init$lambda$3(TreeView.kt:35) at com.rk.xededitor.MainActivity.treeview2.TreeView.$r8$lambda$JBvSDOOqbKRJBQ0KIXzLdcjWoLM(Unknown Source:0) at com.rk.xededitor.MainActivity.treeview2.TreeView$$ExternalSyntheticLambda0.run(D8$$SyntheticClass:0) at java.lang.Thread.run(Thread.java:919)

What path you are trying to access?
accessing /sdcard/Android or /storage/emulated/0/Android is not allowed by android os and if you try it will throw this exception

feels weird trying to open a folder and having a core dump show up - maybe a "android doesn't allow going on this path" and a button to go back would make more sense


Attachable shell without tmux

This is not an issue, just a tip, but I don't know where to put it.
I am using ttyd with abduco cause i don't wanna the multiplexing part of tmux. Maybe you can add that to the readme.

PRs are welcome.