{$file} ignored when launging python debugger for current file

Type: Bug
As the title says: when I set a launch.json file and try to debug my code (selecting "with launch.json", VScode returns:
[Errno 2] No such file or directory: 'home/[myusername]/.local/lib
This is making debugging code outside mine (justMyCode=false) impossible, and is very annoying. What's going on?
Note: I use conda, and I have correctly selected my interpreter to be conda.
VS Code version: Code 1.91.1 (f1e16e1e6214d7c44d078b1f0607b2388f29d729, 2024-07-09T22:08:12.169Z)
OS version: Linux x64 6.9.11-200.fc40.x86_64
Modes:

System Info

Item
Value

CPUs
13th Gen Intel(R) Core(TM) i7-13700H (20 x 400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
15.25GB (8.81GB free)

Process Argv

Screen Reader
no

VM
0%

DESKTOP_SESSION
gnome

XDG_CURRENT_DESKTOP
GNOME

XDG_SESSION_DESKTOP
gnome

XDG_SESSION_TYPE
wayland

Extensions (32)

Extension
Author (truncated)
Version

markdown-mermaid
bie
1.23.1

catppuccin-vsc
Cat
3.15.0

catppuccin-vsc-icons
Cat
1.13.0

ruff
cha
2024.36.0

file-tree-to-text-generator
d-k
1.4.0

vscode-markdownlint
Dav
0.55.0

EditorConfig
Edi
0.16.4

remotehub
Git
0.62.0

vscode-drawio
hed
1.6.6

compareit
in4
0.0.2

mypy
mat
0.3.1

rainbow-csv
mec
3.12.0

git-graph
mhu
1.30.0

debugpy
ms-
2024.8.0

python
ms-
2024.13.2024073102

vscode-pylance
ms-
2024.7.1

jupyter
ms-
2024.6.0

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.19

vscode-jupyter-cell-tags
ms-
0.1.9

vscode-jupyter-slideshow
ms-
0.1.6

remote-ssh
ms-
0.112.0

remote-ssh-edit
ms-
0.86.0

azure-repos
ms-
0.38.0

remote-explorer
ms-
0.4.3

remote-repositories
ms-
0.40.0

autodocstring
njp
0.6.1

vscode-yaml
red
1.15.0

snakemake-lang
sna
0.1.8

code-spell-checker
str
3.0.1

code-spell-checker-british-english
str
1.4.10

even-better-toml
tam
0.19.2

@irm-codebase Please share your configuration? There is also a setting that you can use, for just my code. you don't have to specify it in the launch.json.

Just to make sure, it should be "program":"${file}" and not "program":"{$file}"

@karthiknadig sorry, that was a typo. I'm using the correct
I tried running the debugger without the  debugger even without the launch.json file, and it still fails.
Oddly enough, just running the file works perfectly... no idea what's going on.
Here is the config I've been using
{
    // Use IntelliSense to learn about possible attributes.
    // Hover to view descriptions of existing attributes.
    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
    "version": "0.2.0",
    "configurations": [
        {
            "name": "Python Debugger: Current File",
            "type": "debugpy",
            "request": "launch",
            "program": "${file}",
            "console": "integratedTerminal",
            "justMyCode": false
        }
    ]
}

Here is a picture of the error. Probably not super useful. I can look for stuff in the callstack if need be.

Found the reason why the debugger was not working: setting "debugpy.debugJustMyCode": false IS the cause of the issue above! Basically:

debugJustMyCode: true ---> the debugger works fine (breakpoints are reached, etc)
debugJustMyCode: false ---> the debugger stops working at all, reaching the Errno 2 state.

Please file the issue on the python debug extension, thanks


Timeslider: Mobile – Slider disappears

If I double click several times on the slider, the slider suddenly disappears on mobile
https://user-images.githubusercontent.com/59613691/139869528-cf91b63b-dd8e-43ff-bdc9-bed9f8ef374e.mp4

Reproducable with this chart https://test.visualize.admin.ch/v/RXbybrdOfCO9


add new

as a user I expect to see a new location marker on the map after signing in.
Normal course:
User on map page is not signed in
Select +add new
user is requested to sign in
Select twitter ect
User is re-directed back to map
User see's a marker on the map

Already pushed to staging, as well as edit

After signing in I am redirected to the world view of the map
I expect to return to the place on the map where I wanted to put the new location
Normal course:

User on map page is not signed in
User finds new place on the map by zooming in, browsing the map
Select +add new
user is requested to sign in
Select twitter ect
User is re-directed back to map < the user needs to returmn to the area on the map they found in step 2 above
User see's a marker on the map

fixed on staging


Fix validateLoaderOptionNames test

Just what the name suggests - I would have edited it on your branch directly but I don't have access

I incorporated the fix directly (also had to update output.txt)


Proxy error reported when uploading folders

Uploading single file seems OK.
However, uploading a sample directory logs ECONNRESET
time="2022-09-28T12:38:37.381Z" level="info" msg="using autobought stamp" stamp={"batchID":"761926b6c7889259c997c5716d02d576bb186072aa8abace5bc330b394b23526","utilization":1,"usable":true,"label":"","depth":22,"amount":"200000","bucketDepth":16,"blockNumber":24362883,"immutableFlag":false,"exists":true,"batchTTL":178815,"expired":false}
time="2022-09-28T12:38:37.396Z" level="info" msg="api access" duration="0.016000000" ip="::ffff:<IP_ADDRESS>" method="POST" size=205165 status=201 uri="/bzz" user-agent="bee-js"
time="2022-09-28T12:38:37.397Z" level="error" msg="[HPM] Error occurred while proxying request localhost:3000/bzz to http://localhost:1633/ [ECONNRESET] (https://nodejs.org/api/errors.html#errors_common_system_errors)

Corresponding swarm-cli logs:
Upload size is approximately 0.19 megabytes
Setting --index-document to index.html
curl -X POST "http://localhost:3000/bzz" -H "accept: application/json, text/plain, */*" -H "content-type: application/x-tar" -H "swarm-collection: true" -H "swarm-index-document: index.html" -H "swarm-postage-batch-id:<PHONE_NUMBER>000000000000000000000000000000000000000000000000000000" -H "user-agent: bee-js" --data <buffer>
Data has been sent to the Bee node successfully!
Swarm hash: 7412133a4237012ace7e4f89b1248a29e4d301a17cca42d044f33e6a83d91491
Waiting for file chunks to be synced on Swarm network...
Uploading was successful!
URL: http://localhost:3000/bzz/7412133a4237012ace7e4f89b1248a29e4d301a17cca42d044f33e6a83d91491/
Bzz.link: https://bah5acgzaoqjbgoscg4asvtt6j6e3cjekfhsnganbptfefuce6m7gva6zcsiq.bzz.link

The folder/website is accessible, it is unclear what the problem is

Can you please copy the bee log as well?

This is always reproducible, this run also had the error:
time="2022-09-28T12:54:00.208Z" level="error" msg="[HPM] Error occurred while proxying request localhost:3000/bzz to http://localhost:1633/ [ECONNRESET] (https://nodejs.org/api/errors.html#errors_common_system_errors)"

Then it seems Bee works correctly, because it returns a 201 and the problem is in the proxy.
"time"="2022-09-28 14:54:00.206873" "level"="info" "logger"="node/api" "msg"="api access" "ip"="<IP_ADDRESS>" "method"="POST" "host"="localhost:1633" "uri"="/bzz" "proto"="HTTP/1.1" "status"=201 "size"=93 "duration"="13.954083ms" "user-agent"="bee-js"

The error does not appear when POST /bzz is proxied to a mock HTTP backend

Note to self - compare bee and bee dev as a next step


Update to GoogleInteractiveMediaAds beta13. Move to using a framework.

I updated GMF to use the new IMA SDK with its new framework.
Also made a couple tiny fixes.

LGTM


Condition overlappability measure on two words being placed at random

Previous overlappability assumed two words were already intersecting and computed based on this + some scaling factor.
Better to assume the following model:

Board is empty
Word 1 is placed at random
Word 2 is placed at random

What is the probability (ignoring edge-effects) that the placements of word1 and word2 are compatible?
Winds up being a function of not just letter overlaps, but also word lengths, since two short words are less likely collide even if they share no letters.
Simplified formula:
p(no_overlap) + p(overlap) x p(letter_match | overlap)
~ (1 - w1_area x w2_area / board_area) + (w1_area x w2_area / board_area) x num_same_letter_pairs / (w1_len x w2_len)
~ (1 - w1_len x w2_len / n^2) + (w1_len x w2_len / n^2) x get_num_overlaps() / (w1_len x w2_len)
= 1 + (get_num_overlaps() - w1_len x w2_len) / n^2

Combine this with https://github.com/jonadsimon/wonder-words-generator/issues/39 for more effective word set optimization. In practice, likely equates to choosing from only among the shortest words because the 1st term dominates.

Done in https://github.com/jonadsimon/wonder-words-generator/commit/6f3e8076679a68bc6e42e3cf87405d0d63749e14


Float Parsing Bug Fix

Fixes #186
Accidentally removed the branch. Now, it is recovered.

@yegor256/z everybody who has role REV is banned at #218; I won't be able to assign anyone automatically; consider assigning someone manually (as in §19), or invite more people (as in §51), or remove the job from the scope (as in §14)

@yegor256 ready for review & merge.
you previously reviewed the PR, but I accidentally deleted the branch. Now, I revoked it again.

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

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

Job gh:cqfn/eo#218 is not assigned, can't get performer


Enchanting Events

There is a lack of Enchanting Events available in SpongeAPI
Needed:
EnchantItemEvent: an event that fires when an item's enchantments are being changed either through an anvil or enchanting table.
PreEnchantEvent an event that fires when the choices are given to a player when they insert an item into the enchanting table or anvil, giving them feedback on what happens next.
Usecase:

On my old Bukkit server I would add lore onto items that had been enchanted, with the players names. This made stealing items more interesting, because you could potentially see who the last person who upgraded or cared for an item was. I don't think this is currently possible without enchanting events.
Limiting higher level enchants to a magical crafters RPG class.

agree

bump


Update fs2-core to 3.1.1

Updates co.fs2:fs2-core from 3.0.6 to 3.1.1.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2", artifactId = "fs2-core" } ]

labels: library-update, semver-minor

Superseded by #102.


[GSSOC'23] Half output statement in Age-Calculator

Is your feature request related to a problem? Please describe.
I'm always frustrated when this type of half-message is shown.

Describe the solution you'd like
I'll make it look like the following: which is more liked and easily readable.

Additional context
I can do this for all the texts. Please assign me this issue. I want it to make user-friendly.

I am good at machine learning and had developed lots of machine learning projects, so adding this feature is not a big task for me. so please assign me this and I am also a GSSOC 2023 contributor.

Go Ahead @Shiv-Expert2503


How to save configurations!

I can set a value of the key, but how to save this value to the file (e.g. ini file)?
var config = ConfigurationBuilder().AddIniFIle("c:\test.ini").Build(); config["test"] = "ok";

In my opinion a redesign is needed. This behavior made sense when using ASP, but not now. Now it is assumed that the library can be used in native applications (WinForms, WPF, Console), but the lack of the ability to save the settings makes it practically useless.

before: type safe and super easy way to save/persist and load settings to applocal data. It was just awesome.
now: quite useless.

Yup, I was just trying to use this in an API (outside of ASP.NET Core app) and need the ability to save updates.  This is a needed feature.

Is there any progress on this? .Net Core is now not only used with asp.net, its just silly we don't have a simple save ability without any reason provided, its even more funny that it is internally using System.Text.Json already for AddJsonFile to load it, a simple one-liner like File.WriteAllText(dir, JsonSerializer.Serialize(appSettings)) is enough to save it, but they still insist to not add writing ability to the whole configuration system, it would have so much more sense to support this with any config files.

I'll also ask if there's been any progress on this.  I'm in the middle of updating a WPF application to .NET core, using a host builder and configuring with appsettings.json.  I love the ability to DI the configuration directly into objects I can load with IOptions, but without the ability to save setting modifications made by users of the application back to the JSON file, the functionality for non-ASP.NET applications is simply incomplete.

I'll also ask if there's been any progress on this.  I'm in the middle of updating a WPF application to .NET core, using a host builder and configuring with appsettings.json.  I love the ability to DI the configuration directly into objects I can load with IOptions, but without the ability to save setting modifications made by users of the application back to the JSON file, the functionality for non-ASP.NET applications is simply incomplete.

Closing because this is not an API doc issue.

Closing because this is not an API doc issue.

I'll also ask if there's been any progress on this. I'm in the middle of updating a WPF application to .NET core, using a host builder and configuring with appsettings.json. I love the ability to DI the configuration directly into objects I can load with IOptions, but without the ability to save setting modifications made by users of the application back to the JSON file, the functionality for non-ASP.NET applications is simply incomplete.

I was facing similar need in WPF project and it boiled down for me into two separate problems to solve:

Make IConfiguration indexer setter write back into persisted configuration source.
Be able to use types used with IOptions to modify configuration in a type-safe way.

Since in my case the application had to store configuration in Windows Registry, for addressing first problem we implemented custom IConfigurationProvider that would write back to Windows Registry on Set().
For addressing 2nd point ended up with this.


Text "toolbar" appearing above custom toolbar

here is the custom toolbar settings.
 customToolbar: [["bold", "italic", "underline","strike"],[{
                color: []
            }, {
                background: []
            }], ["link"],[{list: "ordered"}, {list: "bullet"}],['clean']]

This issue is stale because it has been open for 60 days with no activity.


[FEATURE] Add Harvester node driver

Is your feature request related to a problem? Please describe.
Add Harvester node driver of Rancher to enable stage a k8s cluster directly with the RKE.
Describe the solution you'd like

build a Harvester node driver using the docker-machine mechanism
support metadata/userdata injection, test the target OS to be compatible with the requirements of RKE
default to support Ubuntu/openSUSE

**Additional context **

[optional] considering to support VM clone mode, depends on https://github.com/longhorn/longhorn/issues/1815

related issues: #590

If we want to support opensuse：

[ ] need rancher/install-docker support opensuse
[x] need rancher/machine support opensuse （v0.15.0-rancher53 at least）
[ ] need rancher bump rancher/install-docker and rancher/machine

link to repo https://github.com/harvester/docker-machine-driver-harvester

Validate version in v0.2.0-rc5

Add harvester Node Template  ,the configuration is as follows

Adding a harvester cluster, the configuration is as follows, cluster options default

View cluster creation success

Verify that the cluster was created successfully

Delete cluster successfully


Question

Hi
Awesome script you made here! And sorry for abusing the issue tracker, I couldn't find another way to write you.
I just want to know if it is possible to fetch a group of images or the urls? For example for my site, I have multiple images for the same product and would like to fetch all image urls for a specific product in one take.
Regards,
pboxed

Hi,
sorry for the late response. Currently there is not implemented the multiple URL fetching. You can try to use the add() method this way:
foreach ([<LIST OF YOUR IMAGES URL>] as $url) {
    $meta = new <YOUR IMAGE ENTITY CLASS>();
    $storage->add(new File($url), $meta);
}

It is not tested behavior to download images from the URL, so please let me know if this works for you.  If you want something else please provide a more detailed use case.

Hi,
sorry for the late response. Currently there is not implemented the multiple URL fetching. You can try to use the add() method this way:
foreach ([<LIST OF YOUR IMAGES URL>] as $url) {
    $meta = new <YOUR IMAGE ENTITY CLASS>();
    $storage->add(new File($url), $meta);
}

It is not tested behavior to download images from the URL, so please let me know if this works for you.  If you want something else please provide a more detailed use case.

So it is tested behaviour now. The above snippet should work as expected.


Add feature to change options of each point (fill and stroke color and size)

Also, be able to add CSS class name and styles to each point.


fix: apply notify workaround on m1 mac docker

The test failures seem to be flakiness unrelated to the code changes. The compute-node-image-v14 failures looks like a network outage, it failed to download some dependencies. In the pageserver.log the failed test in 'regress-tests', there's this:
    0: db error: ERROR: Failed to process query for timeline 1d4eedb56cc87408c928b19fefcb83d8: Requested streaming from 0/244C000, which is before the start of the timeline 0/244C018
    1: ERROR: Failed to process query for timeline 1d4eedb56cc87408c928b19fefcb83d8: Requested streaming from 0/244C000, which is before the start of the timeline 0/244C018

(See Allure report / release -> test_branching_with_pgbenc[flat-1-10] -> Tear down -> test_output_dir::0 -> repo/pageserver.log)
That's a known issue, tracked in https://github.com/neondatabase/neon/issues/2528.


在Windows 下面pycharm里面运行报错 提示opencv-python错误 但是我已经安装了

594c074\sklearn-0.0-py2.py3-none-any.whl
ERROR: Could not find a version that satisfies the requirement opencv-python (from -r requirements.txt (line
8)) (from versions: none)
ERROR: No matching distribution found for opencv-python (from -r requirements.txt (line 8))

自己手动去装OpenCV-Python，去https://pypi.org/ 上下OpenCV-Python的离线安装包或者使用命令pip install opencv-python


Stop propagation of mousedown for Leaflet 0.7.[5-7] too

In addition to PR https://github.com/CliffCloud/Leaflet.EasyButton/pull/45 .
In stable branch L.DomEvent.addListener accept only one event:
https://github.com/Leaflet/Leaflet/blob/v0.7.7/src/dom/DomEvent.js

Thanks!


Add recommended extensions of vscode

The recommended vscode extensions are specified in the README file of the repository, but because the extensions.json file is not specified, there is a problem with accessibility in the first contribution.
To solve this problem, I ignored the files in the .vscode folder and excluded the extensions.json file that defines the extension.
Also, by specifying Suggested VSCode Extensions in the extensions.json file, the extension can be installed directly from the IDE.

Wow, neat!


CDM_conversion code needs to include compression on output

Closes #65

we need to check if the qff will be .gz

True - it's not currently set up to be.  Would be an easy change to update though, now. Do we know if the mff's will be too?

I think we should amke sure that the qff are compressed and I can check with Matt on the mff either way I can compress the mff after they are produced if needs be , Make sense to zip them up saves space.

OK.  I can make that edit on my QC code for the qffs (both sets).  If the mff changes, then my code will also need to know about that so that it can find the files.

ok cool, I will send Matt an email but will remind him for next run. Thanks.


Add Indonesian (Bahasa Indonesia)

Thank you! How can I credit you in the repository readme?

if allowed, you can credit me to my personal website MRB4Le, Thanks :D

Will do, thanks!


Caching the item metadata for entities

References

Related to https://github.com/DSpace/DSpace/issues/3125

Description
This branch improves performance of entities when there are many relationships. It's only part of the solution https://github.com/DSpace/DSpace/issues/3125
Instructions for Reviewers
This branch will cache the item's metadata (actual metadata and virtual metadata combined). It will invalidate the cache if the item is adjusted in any way. This avoids having to calculate the item's metadata each time it's requested. The discovery indexing use-case benefits most from this change, where the item's metadata is requested separately for the authors, subjects, dates, … (any facet)
List of changes in this PR:

The ItemService.getMetadata will check whether there's a valid cache of the full item's metadata, and use that cache when present. It will calculate the cache otherwise
The Item object will keep track of whether the cache needs to be invalidated, and the cached metadata. This also implies the cache is at most valid for the duration of the REST call. A subsequent REST call will not be able to access that same cache (no risks of outdated data)
Some general fixes to ensure setMetadataModified(dso) are included, for methods which modify the metadata but didn't call this method yet
Some tests had to be adjusted. The item was updated to store e.g. relationship.isAuthorOfPublication, but it was not returned in all use cases. It was a bug that this info was not returned, and the tests assumed this info wasn't present
Some tests had to be adjusted to reload the Item. This is due to the mixed usage of the item object and REST calls, and is specific to the test scenarios

Impact of this change:

The time to create a relationship is reduced with a factor 5 (only 22% of the time needed) for an item containing 100 relationships
The time to retrieve an item containing 100 relationships via REST is reduced to 68% with this change
The time to retrieve an item containing no relationships via REST remains unchanged (the difference is too small to be statistically relevant)

Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide.
[x] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[x] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

@tdonohue I hope it's clearer now based on Samuel's commit that we do expect the metadata to be copied.
The confusion is primarily because the old tests assumed it was not copied, and were proven to be correct due to a bug which didn't allow us to get to the metadata even though it was copied
There was a bug which caused a request to relation.isAuthorOfPublication to return nothing, while a request to *.* and filtering it on relation.isAuthorOfPublication did return the metadata value. Solving this bug did reveal the incorrect tests, which were corrected in this PR

@tdonohue the tests have been included now

@tdonohue I see MetadataSchemaRestRepositoryIT.findAllPaginationTest is suddenly failing again. This is another example where https://github.com/DSpace/DSpace/pull/3170/commits/535f90e14787400a080c2a89590541959c582dca was successful, and https://github.com/DSpace/DSpace/pull/3170/commits/1b3ace654b6fa2595fa26996b1776ac1f8427c3a fails
So this can be ignored for this PR


close on blur

When the input field loses focus, the suggestions dropdown should go away as there is no way to close it except by editing what's been typed.
Chrome 15/OSX

Good catch, will be in added in a upcoming release.


Could RIFE solve the gap between different parts?

For my perspectives, Dain will output some ugly frames.
itch

I will make a simple check, which can filter most transitions. https://github.com/hzwer/arXiv2020-RIFE/blob/d1eb884a24dd2f7cfd3377738b976364b29f5dda/inference_video.py#L138

Can we use SSIM to identify the various transformations by compared with per frame?
Thx for your help


sort patches for identityproviders

Sort the patches for IdentityProviders by name so that the patch in the created SyncSet is consistent. This keeps the SyncSet from potentially thrashing due to a non-determinstic ordering.
https://issues.redhat.com/browse/CO-1144

/assign @twiest
/cc @abutcher

/lgtm


Blog featured images not centering

I have spent a full hour trying to center the featured images in blog posts, I can't get them to move at all.
The CSS I've been messing with is in section 10.1

Issue fixed. It was an issue with CSS precedence.


Add S3 to the list of implemented file systems in doc

This fix adds S3 to the list of implemented file systems in doc.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Ignored unrelated failure.


Add a default parameter option to the .connections.yaml file

This way SimQLe can know which connection to use when no con_name is specified in either recordset() or execute_sql(), and save the user time, especially when dealing with only one connection in a project.

Hey. so it looks like you've got a number of issues that people can work on.  I've forked your code and am using this branch
https://github.com/ZackBotkin/SimQLe/tree/hack
to kind of play around/hack. One thing I noticed.. you don't really seem to use classes or any sort of object oriented style. I find it really helps the maintainability of the project. You can check out my branch to see what I am doing, and I'll send you a pull request when done.

Once I kind of get my feet wet with the code/idea, I could take any of these issues.

Interesting - if you think it would be easier to maintain then I am all for it! Would these be singleton classes mainly? I'm not entirely sure how you intend to do this so I'm looking forward to seeing your PR.
Regarding this particular issue - it should be a very easy change to simply add a default: true option to the connections file that is selected when no con_name is defined when calling recordset() or execute_sql() - happy for anyone to submit a pr :)

closed in #45 and released in #46


Game instantly crashes when trying to render a dropped item while using Sodium 0.6.0 (Beta 2) for Fabric

Tested on 2.16.2 and 3.0.1 on fabric 0.16.5. Doesn't crash on Sodium 0.6.0 (Beta 1).

sodium 0.6 has breaking change
you can disable overlay/outline for entity for now
I will release new version for 1.21 today or tomorrow

Fixed in 3.1.0

thank you for the quick fix!


fix(azure_audit_info): assign correct logging when no auth

Context
In azure audit info, when we are not getting required env variables we are exiting using
raise Exception

Instead of
logger.critical()
sys.exit(1)

That must be unified
Description
Replace raise Exception for the common logging critical path in prowler/providers/common/audit_info.py
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I would prefer to have all unified prior to start with that, but we can start with this to implement custom Exception classes


Add default_sort_value to configuration

Here's a minor enhancement that I found useful:
When the sort parameter is not given, the controller may
specify a default in the configuration object's default_sort_value
member. This is mostly done to allow the sorting links to render
the default sort link with the proper CSS class and indicator when
a page is first loaded.

Daniel
Thanks for your commitment.


[Prover][Issue] Proving the data invariant over two data fields.

Consider the following example:
struct A {
        v1: vector<u64>,
        v2: vector<u64>,
    }
    spec A {
        invariant len(v1) == len(v2);
    }

    public fun push_A(a: &mut A) {
        let v1 = &mut a.v1;
        let v2 = &mut a.v2;
        vector::push_back(v1, 1);
        vector::push_back(v2, 1);
    }

In push_A , the data invariant len(v1) == len(v2) is violated at the intermediate moment when v1 is updated, but not v2. This pattern of data invariant (i.e., the relation property about multiple fields) is used in practice, but we couldn't find a way to properly verify. This situation occurred as a side effect of this PR https://github.com/move-language/move/pull/792.
The following attempt also failed.
struct A {
        v1: vector<u64>,
        v2: vector<u64>,
    }
    spec A {
        invariant len(v1) == len(v2);
    }

    public fun push_A(a: &mut A) {
        let r = a;
        let v1 = &mut r.v1;
        let v2 = &mut r.v2;
        vector::push_back(v1, 1);
        vector::push_back(v2, 1);
    }

It sounds like you're running into an issue with verifying the data invariant len(v1) == len(v2) in your push_A function. The problem is that in the current implementation, the function is modifying both v1 and v2 independently and in an non-atomic way, which means that at the intermediate moment when v1 is updated, but not v2, the data invariant is temporarily violated.
One possible solution to this issue would be to use a lock or a mutex to ensure that the updates to both v1 and v2 are atomic and happen simultaneously.
Another solution would be to use a single vector that holds a pair of values instead of two separate vectors, and ensure that the invariant holds for the single vector.
It's also worth noting that in your second attempt to fix the problem by creating a reference to a and then modifying it, the issue remains the same, as the two vectors are still being modified independently.
It's good to see that you're aware of this problem and trying to find ways to solve it. It's always a good idea to test the code and see if the problem still persist and report it to the community.

impl std::fmt::Debug for PathedIoError {
fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
f.debug_struct("PathedIoError")
.field("path", &self.path)
.field("inner", &self.inner)
.finish()
}
}
In this example, the second field name is changed from "path" to "inner", ensuring that the two field names are unique and meaningful. This will fix the issue, and when you print the struct using the {:?} or {:#?} format string, it will show both fields, the path and inner fields.

@FrodoBaggins74524 , thanks for your comment. My colleague just merged a fix for this. Please feel to check it out: https://github.com/move-language/move/pull/820.


Support publishing messages with ordering keys in PubSub sink

WHAT
This is a request to support publishing messages with ordering keys in PubSub sink.
https://cloud.google.com/pubsub/docs/publisher#using-ordering-keys
WHY
I thought that the option has not been supported yet.
https://github.com/mercari/DataflowTemplate/blob/master/docs/config/module/sink/pubsub.md#pubsub-sink-module-parameters

A new parameter orderingKeyFields has been added to support orderingkey.
Please feel free to let me know if you find anything different from what you expected.

@orfeon Thanks to support my request immediately!!
I don't often know Java but I was reading diff of the source code and the document. SGTM!

Thanks for confirming!
I tried it with dummy data in my environment and it worked as expected, so I would like to merge it.
(I tried to run it with 100 writes per second and ordering key's cardinality of 100, so I hope this example config will be helpful how to use it.)


Cannot set display options for WVD Client

The WVD client takes over all monitors on a multi-monitor system.  Need to be able to define display options including monitors to use, screen resolution, etc.

@sspage can you provide me with the link to the documentation you are referring to?

https://docs.microsoft.com/en-us/azure/virtual-desktop/connect-windows-7-and-10.

Duplicate of #28258
Will work off of that one.


Add configuration for whether new elements are added to the start of the list instead of the end

Overview
There are differing preferences on whether a new block should be added to the top of the list, or the bottom. For large lists of blocks on a page adding to the top might best sense, while for pages that don't use many blocks (we see an average of 4 blocks per page) it might make more sense adding a new block to the bottom - following the flow of adding new content to a page.
Acceptance criteria

When adding a new block with the main action, the user has the option of choosing where it is added (via a UI 'toggle')

Early design suggestion: https://github.com/dnadesign/silverstripe-elemental/issues/566#issuecomment-526028524

The new UI toggle would not be present when a user adds a block using the inline feature (between blocks)
There is configuration for a Developer to set the default state of the toggle

Old description:
In general content editing it could be argued that’s okay, because we don’t know where a new block is destined to be positioned. I think adding to the end is a better default, as when creating content for new pages you either have to drag+drop blocks around each time you add a new one or create content in reverse order

Found this while looking for a way to add to the bottom myself as it's a very frustrating experience trying to add multiple blocks to a new page using the obvious Add button.
Wasn't aware of the hover action either.
How about just adding a prominent "Add" button to the bottom too? Either a 2nd green "Add" button, or, even better, make the last hover "+" button larger, more prominent and always visible?
Craft CMS's excellent "Matrix Field" has their Add buttons at the bottom only:

I'd be for defaulting to the bottom, or at least have it configurable somehow, or two sets of buttons where above adds to the top and below adds to the bottom — Can be "Add first" and "Add last" or so.
Agree with it being PITA when one needs to move several elements.
Also, neither was I aware of the hover option.

I hacked together a workaround for a project I was working on that needed new elements to be added to the bottom of the list: https://gist.github.com/kinglozzer/a2c27f2a0c3563d8a72f8dfb982fc7fa

Let's make this configurable

Noting that we've recently finished #704 which improves the experience of adding blocks inline - something that can be quickly used to add a block to the bottom, should the default be to add the block to the top.
See: https://www.youtube.com/watch?v=qIgJk0JAfrU

Would be good to make user configurable eventually, something like:

Radical! So this would be user-configurable rather than dev-configurable (or both, in that a dev could configure the default position)?

If we went as far as implementing the UI switch then I don't think there is much need for the dev config? Keep in mind that switch wouldn't be there for the action which inserts blocks in-between others. Design is just a suggestion to gauge what others think.

Just thinking that if you always wanted to add to the bottom you might want that to be the default setting in the add block dialog.
We'd also need to ensure that that toggle wouldn't show when adding a block in between existing blocks.

Design is just a suggestion to gauge what others think.

I think it looks 🔥

Hi all, I've updated the description to reflect the conversation in this issue and added some basic ACs. Please feel free to suggest other points here

After a discussion with the product team I got some feedback that the toggle might not be as intuitive as I had thought, so here is another idea to put forward for consideration.
Essentially an icon (replacing the toggle) with something similar these indicating its placed top or bottom. Showing text when hovered to explain iconography (Add to top, Add to bottom).

If adding between blocks was more intuitive/obvious this issue probably wouldn't exist. There are other alternatives to doing this issue for example, a walkthrough of new UI features highlighting "New" things with tips on how to use functionality. This is probably a more usable feature throughout the UI but also a bigger piece of work.

Ran this issue passed the CMS Squad, summarising some of the feedback.

There needs to be more thought on the use of the CMS UI toggle switch and the label 'Top'. That component usually represents an enable/disable setting, where this action is more about selecting a preference between two options.
This may only be worth implementing in the CMS UI if a user's preference could be stored against their profile, or be otherwise semi-permanent on an individual basis
There's a bit of feedback on this issue to set the default to add to the bottom of the list. That probably works for a good majority of sites that only have a handful of blocks on a page, but if a page has so many blocks that you can't see the bottom of the list, adding to the bottom without indicating this could cause a lot of confusion
A more drastic UI option could be to remove the main 'add block' button and put more emphasis into the inline options, perhaps even more so to the top and the bottom borders

How about duplicating/splitting the Add block button to
Add block  and Add block
or using something like https://github.com/symbiote/silverstripe-gridfieldextensions/blob/3/docs/en/index.md#multi-class-adding to support two (or more) options.
It doesn't need to be all handled within the dropdown mini-modal itself.
Alternatively, if the UI controls for adding the block on top and at the bottom were always visible (not just on hover), that would work IMHO too, as suggested in

A more drastic UI option could be to remove the main 'add block' button and put more emphasis into the inline options, perhaps even more so to the top and the bottom borders

Regarding the icons, I'd find the presence of an arrow a bit confusing and would expect this would either be moving something (to the top/bottom) or something to do with sorting. How about something more like these:

Admittedly these would work better if they could use a lighter gray for the existing blocks to emphasize the position of the "added" block.

Yeah, of course, I just used Paul's images to illustrate the concept.

Has this feature gotten anywhere yet? I have had another request for it this week.


Update main README page

Update main README page to conform to the terraform registry standards for modules.
Example README file: https://github.com/hashicorp/terraform-aws-consul-starter/blob/master/README.md

Updated in #28


Use of white-space: pre makes for uncomfortable Handlebars-ing

Please feel free to close as necessary, I've found the solution for my issue, but it's somewhat non-ideal, and discovering made for some interesting thinking.
Issue (?): Handlebars {{ always handles new lines in strings with preceding partials in mind.
B/c pre tags use white-space: pre the white space inside the tag is treated differently in the browser, so when rendering content within a Handlebars content like the following:

{{codeExample}}

Any newlines in codeExample factors in additional whitespace from their parent partial invocations. See example here:
https://jsfiddle.net/e2kmzn56/
My workaround was to use the white space control operator on all partials leading up to the pre tag. I.e., in reference the example, using {{~> example }} instead of {{> example }} and {{~> article }} instead of {{> article }}.
While this solves the issue, it makes it somewhat harder to understand authorial intent for the templates. The compiled HTML here makes it look very obvious:

var abc = {
  a: 1,
  b: 2,
  c: 3
}

but the template itself looks absolutely benign...

{{text}}

So I'm curious if there's any interest in providing a method to bypass & trim the additional whitespace from parent partials for new lines?
E.g. with { text: "My text with\n  new\n  lines\n" },
        {{!-- within a partial --}}

                         {{~~ text }}

would translate to
        ...

My text with
  new
  lines

There is a compile-option "preventIndent" that should turn of auto-indenting with partials.


Composite gets wrapped in latest build

Salutations,
Installing Jimp from the github repo (npm i oliver-moran/jimp) causes composites to act in an unwanted manner.
I have a relatively simplistic script that does the following:

Create an empty canvas new Jimp(256, 256)
Composites two images with slightly random coordinates (to create a 'jitter' effect)
Repeat the process several times
Turn them into a gif using GIFEncoder.

Output from NPM build:

Output from Github build:

I do not want my composite outputs to wrap when portions go off the screen. Rather, I want them to be cropped to the canvas's size as they used to. Please either resolve this or implement as an optional parameter.

@Ratismal does this still happen? What version did it work in?

I have no idea. I tried making a new script to test with (since the old one no longer exists), but after installing npm i oliver-moran/jimp I get the error Cannot find module 'jimp'. This doesn't happen when installing the NPM release.

The builds are no longer hosted in the repo.  I tried making a new script to test with (since the old
one no longer exists), but after installing npm i oliver-moran/jimp I get
the error Cannot find module 'jimp'. This doesn't happen when installing
the NPM release.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/251#issuecomment-409080255,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIyBM75lEzCVgSdYVqV4XKySX6wlwj9ks5uL8skgaJpZM4MdFv0
.

Can't get it to build, but wrote a script anyways for testing.
const Jimp = require('jimp');

async function main() {
    let parent = new Jimp(100, 100, 0x000000FF);
    let child = await Jimp.read('https://i.imgur.com/S2TMaa9.jpg');

    parent.composite(child, -50, 0);

    parent.write('out.png');
}

main().then(() => {
    console.log('Finished');
}).catch(err => {
    console.error(err);
})

when run, this should be the resulting out.png file:

if the issue still exists, you'll get something like this:

Thanks for the test case! I’m interested to see if you get build errors
though.
Steps to install:
npm i
npm run build
Do you get any errors?

Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/251#issuecomment-409084203,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIyBAS8-ifmG1l3VtSdyFwyFI6nbuWDks5uL9GCgaJpZM4MdFv0
.

D:\JS\jimp-test\node_modules\jimp>npm run build

><EMAIL_ADDRESS>build D:\JS\jimp-test\node_modules\jimp
> npm run build:browser && npm run build:node:production && npm run build:module

><EMAIL_ADDRESS>build:browser D:\JS\jimp-test\node_modules\jimp
> BABEL_ENV=production node tools/browser-build.js prepublish

'BABEL_ENV' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build:browser: `BABEL_ENV=production node tools/browser-build.js prepublish`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:browser script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      D:\AppData\Roaming\npm-cache\_logs\2018-07-31T03_32_24_285Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `npm run build:browser && npm run build:node:production && npm run build:module`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      D:\AppData\Roaming\npm-cache\_logs\2018-07-31T03_32_24_335Z-debug.log

Currently working off of a windows machine because my linux machine is at work, will try building on it tomorrow.

 It looks like the environment
variables work differently on windows. Do you know how to set cli env bars
on windows?  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build:browser script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      D:\AppData\Roaming\npm-cache_logs\2018-07-31T03_32_24_285Z-debug.log
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: npm run build:browser && npm run build:node:production && npm run build:module
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      D:\AppData\Roaming\npm-cache_logs\2018-07-31T03_32_24_335Z-debug.log
Currently working off of a windows machine because my linux machine is at
work, will try building on it tomorrow.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/251#issuecomment-409085738,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIyBIWO6pQ867rDShgcmaWOMBeBVfDgks5uL9QqgaJpZM4MdFv0
.

Googling around a little I might have to put ‘set” in front of all the Babel envs. Can you try that out? Thanks for all the help!

was able to build after converting the scripts to windows-compatible calls. the scripts in question:
  "scripts": {
    "lint": "npm run -s lint:src && npm run -s lint:tests && npm run -s lint:extra && echo Lint All Success",
    "lint:src": "xo src/*.js src/**/*.js !src/modules/*.js",
    "lint:tests": "xo test/*.js --global before",
    "lint:extra": "xo {tools,examples}/*.js",
    "test:browser": "set BABEL_ENV=test && set ENV=browser && set CHROME_BIN=\"$(which chrome || which chromium)\" && karma start --single-run",
    "test:browser:watch": "npm run -s test:browser start -- --auto-watch",
    "test:node": "set BABEL_ENV=test && mocha --require @babel/register",
    "test:node:watch": "npm run test:node -- --reporter min --watch",
    "test:node:coverage": "nyc npm run test:node",
    "test": "npm run -s test:node:coverage && npm run -s test:browser",
    "browser-build": "node tools/browser-build.js test",
    "build": "npm run build:browser && npm run build:node:production && npm run build:module",
    "build:debug": "npm run build:browser:debug && npm run build:node:debug",
    "build:module": "set BABEL_ENV=module && babel src -d es --source-maps",
    "build:node": "babel src -d dist/main --source-maps",
    "build:node:debug": "set BABEL_ENV=development && npm run build:node",
    "build:node:production": "set BABEL_ENV=production && npm run build:node",
    "build:browser": "set BABEL_ENV=production && node tools/browser-build.js prepublish",
    "build:browser:debug": "set BABEL_ENV=development && set ENV=browser && node tools/browser-build.js prepublish"
  },

in addition to using set, I had to separate calls with &&
im not sure how you'd make this compatible with both windows and linux

Thanks! I’m not sure either but I’ll try tonight!

Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/251#issuecomment-409087749,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIyBC3mEFddSAHgmVeQKGI98Bw9wdYAks5uL9eCgaJpZM4MdFv0
.

Does the bug persist?
On Mon, Jul 30, 2018 at 9:06 PM Andrew Lisowski<EMAIL_ADDRESS>wrote:

Thanks! I’m not sure either but I’ll try tonight!

Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/251#issuecomment-409087749,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIyBC3mEFddSAHgmVeQKGI98Bw9wdYAks5uL9eCgaJpZM4MdFv0
.

I’ll be using the cross-env package. Now I know why we use it at work lol
On Mon, Jul 30, 2018 at 9:06 PM Andrew Lisowski<EMAIL_ADDRESS>wrote:

Does the bug persist?
On Mon, Jul 30, 2018 at 9:06 PM Andrew Lisowski<EMAIL_ADDRESS>wrote:

Thanks! I’m not sure either but I’ll try tonight!

Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/251#issuecomment-409087749,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIyBC3mEFddSAHgmVeQKGI98Bw9wdYAks5uL9eCgaJpZM4MdFv0
.

the output looks fine, so i guess this bug is not longer occurring.

Sweet!

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/251#issuecomment-409088850,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIyBAAc5Kz-8WC6IaenC2ANjMm-OAPNks5uL9l6gaJpZM4MdFv0
.

Thanks for all the responses and working with me 😁
On Mon, Jul 30, 2018 at 9:14 PM Andrew Lisowski<EMAIL_ADDRESS>wrote:

Sweet!

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/issues/251#issuecomment-409088850,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIyBAAc5Kz-8WC6IaenC2ANjMm-OAPNks5uL9l6gaJpZM4MdFv0
.

no problem, was a pleasure


nginx-ingress-controller: Route By Query String

I'm looking for the ability to route traffic via query string. We have a bucketed system that supports session persistence with either a query string or cookie. We are currently able to achieve this with HAProxy and ACL url_param. The idea is that I specify a field and a value in my ingress resource and any requests with the matching field and value are routed to the specified service.
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: test
spec:
  rules:
  - host: foo.bar.com
    http:
      fields:
      - <fieldKey>: <fieldValue>
        backend:
          serviceName: s1
          servicePort: 80

Is this something nginx can support? I would be more than happy to work on a PR for this change if so.

Looks like that would be done via regex against $args. +1 on getting this into the ingress controller.
if ($args ~ "^version=v1") {
}

@TWood67 please keep in mind that Ingress is an specification. This means you must add/support things that all the vendor that implement the spec supports.
That said you can add this feature as an annotation (tied to only the nginx ingress controller) like:
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: test
  annotations:
    ingress.kubernetes.io/query-routing: default/query-routing
spec:
  backend:
    serviceName: default-backend
    servicePort: 80
  rules:
  - host: foo.bar.com

---

kind: ConfigMap
apiVersion: v1
metadata:
  name: query-routing
data:
  mapping: |-
    [{
      "field": "version",
      "value": "1",
      "path": "/",
      "service": "default/bar",
      "port": "80"
    },{
      "field": "version",
      "value": "2",
      "path": "/foo",
      "service": "default/foo",
      "port": "80"
    }]

but to be honest I found this too complex to maintain or debug in case of issues


Browse Tab Simple Search and Results Table

Conferred with @astridu regarding the UI for the Browse tab in the Editor. Here is an earlier design that we can use (with red x on parts of the design we are no longer using):

This would have a single input field to do a keyword search in Sinopia's Elasticsearch index and return a list of entities that are ordered by relevancy.

Clarifying this issue:

We will re-label this tab "Search" instead of "Browse"
When first clicking over to the Search tab, it should render only the search bar and, when a user enters a search string with results, then render the table of results below.

Screenshots:


Move to exporters

Moves optimum.neuron.exporter to optimum.exporters.neuron.

You shall add an __init__.py file under exporters folder, otherwise neuron won't be identified.
Despite that LGTM. Tested on INF2, it will work well once __init__.py is added.


Not compatible with sync-settings

Currently it doesn't look as though sync-settings is backing up projects.cson.
Is it feasible to put the settings in a file that sync-settings would back up? Or should sync-settings be backing up a wider range of files?
Thanks for the package!

I haven't used the package myself, but it seems to have a extraFiles setting where I guess you could add the projects.cson file. Doesn't seem to be a way for me to register the projects.cson file as something that could be synced, so for an option it would have to be added directly to the package.
I hope that helped you :)

That worked!
It was all on the other package.
Thanks for responding so promptly, and again for project-manager!


Juicy-Jaguars

The last two commits to fix the linting were made after the deadline, although it's only one and two minutes over respectively.
No merge issues
No flake8 linting errors (but note the two commits after the deadline)


refactor [amo] service

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against 1153bf0ca79c7df6212e74ab2e2c890f18cab554


Potential security issue

Hello 👋
I run a security community that finds and fixes vulnerabilities in OSS. A researcher (@m0kr4n3) has found a potential issue, which I would be eager to share with you.
Could you add a SECURITY.md file with an e-mail address for me to send further details to? GitHub recommends a security policy to ensure issues are responsibly disclosed, and it would help direct researchers in the future.
Looking forward to hearing from you 👍
(cc @huntr-helper)

Hey @psmoros and @m0kr4n3, thanks, please reach out to the maintainers at the same e-mail address as for privacy breaches: https://pathway.com/privacy_gdpr_di/#breach-of-privacy. Happy to be in touch.

Thank you for the clarification, I appreciate the insight into its intended usage and the acknowledgment of the concern raised.
While I understand that the Streamlit UI is primarily intended for demonstration purposes within trusted environments, I firmly believe that ensuring security even in examples is paramount. As you mentioned, some demos are shared on the cloud, which could potentially expose vulnerabilities. I am more than willing to contribute a patch via a pull request if you would like.
As a reputable company like Pathway, maintaining a proactive approach to security, even in example code, reflects positively on the overall reliability and trustworthiness of your products and services.
Additionally, I want to clarify that the issue resides in the logic behind handling uploaded files within the Streamlit app, rather than a flaw specific to Streamlit itself.
Once again, thank you for your time and understanding. I look forward to collaborating to ensure the continued security and integrity of Pathway's products.

Additionally, I want to clarify that the issue resides in the logic behind handling uploaded files within the Streamlit app, rather than a flaw specific to Streamlit itself.

No doubts there. The UI bundled with the unstructured example was designed for single-user use demo only. Setting any avoidable issues aside, the entire flow of data in this specific UI does not make much sense for "non-localhost" use, nor for use by more than one user per server - it essentially acts as a local Python application with web browser controls on it. We hoped this was evident from the logic, however, again, we fully acknowledge that this could be considered misleading for some users, especially given most other examples in this repo have a UI setup with a much cleaner flow of data. We'll probably scrap this UI example at some point soon or move it out to the pathway-labs github organization. I'm keeping the issue open until this is done.

I appreciate your clarification and I'll focus my efforts on exploring vulnerabilities within the core application moving forward.
Regarding the pathway hoodies, that's awesome! Thank you so much, I'll gladly take you up on that offer. And by the way, me and my colleague collaborated on the report. Two pathway hoodies would be perfect! Could you please let me know where I can securely share my postal address and size with you?

Thanks a lot @m0kr4n3, I've reached out by e-mail.

@szymondudycz Following recent repo structure review, would you consider the Streamlit UI separation is now sufficient to consider this issue closed?

The UI for unstructured was changed, so this issue is no longer relevant.


feat(button): accessible button and pointer onOut

An accessible button element for #124. Also add onOut as a function a tracked pointer object can have that will be called when the object loses focus.

Coverage decreased (-0.2%) to 99.121% when pulling 97749ec5ac460268eb638d5cd7782e1aacb05849 on button into d33567d4c3cb66cc2f7646c7ae2515cb90b01f6d on v7.


Lack of documentation for relationship filtering

"Polymorphic associations" entry in the recipe book is the only place where filter: {commentableType: 'Issue'}, feature is mentioned, and even then it is not mentioned that it supports either object, or string or function there. Given how useful this feature is, I believe it belongs somewhere in the core documentation.

@koskimas
I could give writing documentation entry a shot, but would appreciate pointing out what particular part of documentation you would like it to be added to.

It should be added here

@koskimas PR created.

PR merged. Thanks 👍

I am also having trouble findng docs on the filter aspect of relations. The link above now seems defunct with the updated docs.
Is there a way to access the old docs?
Or maybe get an update where the link is suppose to go to?


Support comments in Regex patterns

Related to #1167
Comments in Regex patterns should be supported:

(?# comment) - Inline comment. The comment ends at the first closing parenthesis.
# [to end of line] - X-mode comment. The comment starts at an unescaped # and continues to the end of the line.

X-mode comment can be used by enabling RegexOptions.IgnorePatternWhitespace option:

A number sign (#)marks an x-mode comment, which starts at the unescaped # character at the end of the regular expression pattern and continues until the end of the line. To use this construct, you must either enable the x option (through inline options) or supply the RegexOptions.IgnorePatternWhitespace value to the option parameter when instantiating the Regex object or calling a static Regex method.

54b5c36bfcb36044cfaf84782a78ea82af9b13dd


Custom Editor not triggering blur

Clicking outside the table / on another custom editor dosen't trigger the on blur event
https://react-bootstrap-table.github.io/react-bootstrap-table2/storybook/index.html?selectedKind=Cell Editing&selectedStory=Custom Editor&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel

Solved it by wrapping the editor component with https://github.com/airbnb/react-outside-click-handler
and triggering onUpdate function when clicking outside, if there is another solution or fix I would be more than happy to know

hi @arielshad sorry for lately reply, if you have a custom editor, you have to add onblur event on your component. react-bootstrap-table2 will pass onBlur callback for you:
{
  dataField: 'quality',
  text: 'Product Quality',
  // editorProps.onBlur <<<< you can call it when you catch the blur event on your custom component
  editorRenderer: (editorProps, value, row, column, rowIndex, columnIndex) => (
    <QualityRanger { ...editorProps } value={ value } />
  )
}

Thanks


[cadl][union] cadl-python can't deserialize union type when the union is made of customized model

When the union type is made of model types like: https://github.com/Azure/autorest.python/blob/75af948ac1dde6a563495f5a87834072cfec1a7e/packages/cadl-python/inference/models/completions.create.cadl#L27
Our generated code is like:
https://github.com/Azure/autorest.python/blob/75af948ac1dde6a563495f5a87834072cfec1a7e/packages/cadl-python/inference/generated_code/azure/openai/models/_models.py#L278-L281
Here is the test code:
https://github.com/Azure/autorest.python/blob/75af948ac1dde6a563495f5a87834072cfec1a7e/packages/cadl-python/inference/test.py#L12-L46
A and B part doesn't work which shows we may need to optimize the deserialize logic

It is like oneof in swagger which is very hard to distinguish which kind of model the response is.


Support catching multiple exception types in the same handler

There's currently no convenient way to support catch statements which handle multiple exception types. The catch_ method should be overloaded to support a variable number of types. The type of the returned variable is the closest common superclass.

https://github.com/cojen/Maker/commit/d64d207a67b59d2fc2b9a65e362cc2ddacd1881e


Initial draft files

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

test only


Add Enjin & Matrixchain

We are moving from Efinity to Enjin Matrixchain, this PR updates it and adds Enjin Blockchain. We've already changed our slip044 as well: https://github.com/satoshilabs/slips/pull/1598

Given the intended nature of the Enjin Blockchain ecosystem's growth, I believe that the best course of action is to rather fork this registry and keep one that fully suits your needs, current and future.
I think this stands too for PolkadotJS, so I'll be commenting that on here too.

Hello @SBalaguer, Would you mind to explain what is the "intended nature of the Enjin Blockchain ecosystem growth"? And why is it different from the various standalone chains that are also here and are not parachains of polkadot? Regards

My point is that with the objective of the Enjin Blockchain and the consequent Matrixchain explained on it's whitepaper, the best course of action is to fork this registry and keep a list of the top-level blockchain with all the subsequent connected chains (like Enjin Matrixchain and whatever others might come in the future). This predicted growth future is what makes the essence of Enjin Blockchain and Enjin Matrixchain different than any other solochain, and that it's why forking will be most probably best to suit your needs, current and future.

I understand your point but I don't really agree with it. This is a registry of SS58 prefixes it basically maps any substrate chain that uses SS58 as a standard to the prefix that it uses to make the lives of other developers easier. There is no point in making a fork of a registry it would do the exact opposite, making the life of developers harder.
Also, it doesn't say in the docs, nor the repo, nor anywhere. That relaychains and parachains of other relaychains cannot be registered here. I mean like I said a "ss58registry" not a "polkadot relaychain registry" as we can see for the lots of standalone chains that we have here. I would like your reconsideration, otherwise, I would like to know if that's an official Parity response and not just your personal opinion.

As we don't get any answer, @gavofyork, can you confirm to us that's the official parity decision? To not operate a ss58 registry without bias, as people do with slip044 is operated with an independent party?

Hey, I would be open to approve and merge your pull request. I would just have one requirement before doing this, please open up any code that was forked from us. The node side is GPL3 licensed and requires to open up any changes any way. We are believing in open source and this is also better for the Substrate ecosystem. In the future you should then also help us to identify actual builders in your ecosystem to prevent that the registry is spammed.

Hello @bkchr, thank you for your response.
If you don't mind I would like to make a comment about the following point as I'm not sure you are aware of our work.

We are believing in open source and this is also better for the Substrate ecosystem.

Enjin is very committed to the open-source community and we also believe we can do great things together. Every single project that I work on forked or not is open-source and was done in a way that is generic enough for any substrate chain to integrate with it. Here is a list of a few of them:

https://github.com/enjin/wallet-daemon
https://github.com/enjin/platform
https://github.com/enjin/platform-core
https://github.com/enjin/platform-csharp-sdk
https://github.com/enjin/ledger-enjin
https://github.com/enjin/platform-ui
https://github.com/enjin/platform-beam
https://github.com/enjin/ledger-matrixchain
https://github.com/enjin/platform-marketplace
https://github.com/enjin/platform-fuel-tanks
https://github.com/enjin/platform-cpp-sdk
https://github.com/enjin/php-blockchain-tools

@leonardocustodio where can I find your actual Substrate chain?


xelatex support

compiling with xelatex errors on the microtype, can easily add an if around it.

Fixed in https://github.com/vanatteveldt/ccr-quarto/commit/cac9c5cd1ccff36866851728a1b884254698baf4


Bicep linter/static-analysis is not recognizing error scenarios related to resource name

Bicep version
Bicep CLI version 0.4.1124 (66c84c8ee5)
Describe the bug
I was recently attempting to instantiate a bicep resource with a randomly generated name for the purpose of my ci testing process. The name of the resource is constrained (as a Azure policy) to start with a letter.
I inadvertently set the name using uniqueString(), however it turns out this function doesn't guarantee that the result will start with a letter, and the bicep vscode extension didn't warn me about this fact while authoring the resource.
This results in me creating a deployment that will periodically fail introducing flakiness to my CI environment since every now and then (given that I am recreating the environment on each build) I may get a uniqueString that starts with a number resulting in a bad request error.
I am curious if there's a way to have the vscode linter/static-analyzer detect this during authoring since its unreasonable to expect authors to "just know" what are the constraints on a particular azure resource name.
Here is the resource I was setting:
resource key_vault 'Microsoft.KeyVault/vaults@2021-04-01-preview' = {
name: uniqueString('kv', resourceGroup().id)
location: resourceGroup().location
properties: {
enabledForTemplateDeployment: true
tenantId: subscription().tenantId
enableRbacAuthorization:true
sku: {
name: 'standard'
family: 'A'
}
}

Here is the sporadic error I would get every now and then:
Error: Code="DeploymentFailed" Message="At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details." Details=[{"code":"BadRequest","message":"{\r\n \"error\": {\r\n \"code\": \"VaultNameNotValid\",\r\n \"message\": \"The vault name '2d6lducplqzgu' is invalid. A vault's name must be between 3-24 alphanumeric characters. The name must begin with a letter, end with a letter or digit, and not contain consecutive hyphens. Follow this link for more information: https://go.microsoft.com/fwlink/?linkid=2147742\"\r\n }\r\n}"}]

Here is how I ended up fixing the issue:
resource key_vault 'Microsoft.KeyVault/vaults@2021-04-01-preview' = {
name: 'kv-${uniqueString('kv', resourceGroup().id)}'
location: resourceGroup().location
properties: {
enabledForTemplateDeployment: true
tenantId: subscription().tenantId
enableRbacAuthorization:true
sku: {
name: 'standard'
family: 'A'
}
}

** Expected Behavior **
I would expect the Bicep VSCode extension to underline the stanza name: uniqueString('foo') with a warning or an error that indicates that doing this is unsafe.

Today, naming requirements are not a part of our resource type definitions, though they are sometimes present in swagger, so it would be possible to use that info in our type system. I think we'd also need to add some type information about the type of string that is returned (i.e. string that could start with letter or number).
Related: #1679

One fairly straightforward possibility is to include the naming regex in the generated description during type generation.
e.g. here:
https://github.com/Azure/azure-rest-api-specs/blob/0bca06c5e06c6b12ea8e560c7416a5968136e276/specification/keyvault/resource-manager/Microsoft.KeyVault/stable/2021-10-01/keyvault.json#L34


added logic for different npm json format

Codecov Report

Merging #32 into master will decrease coverage by 0.36%.
The diff coverage is 90%.

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage     100%   99.63%   -0.37%
==========================================
  Files           4        4
  Lines         268      277       +9
==========================================
+ Hits          268      276       +8
- Misses          0        1       +1

Impacted Files
Coverage Δ

f8a_utils/versions.py
98.95% <90%> (-1.05%)
:arrow_down:

Continue to review full report at Codecov.

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

@yzainee Your image is available in the registry: docker pull MISSING_REGISTRY_NAME/MISSING_IMAGE_NAME:SNAPSHOT-PR-32

@yzainee @miteshvp have you realized that the code coverage has been decreased by this PR?


Interactive components should be able to use Link component from react router

[x] I’ve read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I’ve read and agree to the Code of Conduct.
[x] I’ve searched for any related issues and avoided creating a duplicate issue.

What happened
Developers using react router have to do a work around to modify the this.props.history whenever using an interactive link.
This was brought up by @rooby in the community:
https://community.digital.gov.au/t/recommendations-best-practices-for-using-components-with-react-router/275/2
What I expected to happen
We should give them flexibility to use whatever component they want for interactive elements.
Steps to reproduce:

Install link-list
Add react router
Change the history without using the Link component

Do you know what the timeline is on this one?
Do you have a specific approach in mind? If so I can submit a PR if that would be of use.

I was going to work on this and get it done by next wednesday. It's a pretty simple approach just time consuming and needs to be applied and documented for all interactive components.
If you want to do it, I can give you much more detailed information 👍


Not support a quality-button

Subj.
For example code:

480p

480p

360p

240p

auto

Thanks for the report! I haven't had much time to maintain this lately, but will tackle it as soon as I can. PRs welcome. :)

One of my users put together a jsFiddle demonstrating the issue. Maybe this will help.
http://jsfiddle.net/ksrwu6kx/


vscode: JSX element implicitly has type 'any' because no interface 'JSX.IntrinsicElements' exists

I just did a fresh vscode and deno install on a windows machine and created example project using https://fresh.deno.dev/docs/getting-started/create-a-project.
Vscode installed twind and deno plugins, but when opening routes/index.tsx it is full of such errors.
JSX element implicitly has type 'any' because no interface 'JSX.IntrinsicElements' exists

I tried:

Deno: Restart Language Server
Deno: Initialize workspace configuration
Developer Reload Window

What else must be configured to get it going in vscode?

Okay on this machine it started to work after deno task start. Have to check what i going on with the other one.

I'm having the exact same issue, even after deno task start, have you been able to solve this? @andresv

@Benji377 This is a pending open issue in Deno. There is a workaround though: By running deno cache main.ts it should work. Context https://github.com/denoland/deno/issues/20582

Thank you!

There is a workaround though: By running deno cache main.ts it should work.

That didn't work for me in my Fresh project nor did deno task start.

+1


chore: Replace Array by T[] syntax

Thank you for this PR. The build is failing because of a failing test. Can you update the expected test result? https://github.com/supergraphql/graphql-static-binding/blob/1da897d1e96d812433b179d91ccf015715923f20/test/features/graphcool-ts.feature#L39 and https://github.com/supergraphql/graphql-static-binding/blob/1da897d1e96d812433b179d91ccf015715923f20/test/features/graphcool-ts.feature#L49

Coverage decreased (-0.3%) to 47.486% when pulling 2f813c309d9f9d91b1955da07cbb0ea65422d746 on schickling:patch-1 into 6856e1e44ba263c38185ebef94c0e42c411bcb73 on supergraphql:master.


Inline simple filesystem functions

Extracted from #1256.
These functions are one-liners that could be easily inlined.

Do you have meaningful performance numbers for this change? Not putting definitions in headers was an explicit choice, not an accident.


Add extension.name clarification

Addressing issue #2100

Jenkins CI test (for commit ba9d07bd546a578843f60294e168aeaf1a58173a) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

@beam2d What was the motivation to introduce the concept of default_name? (in https://github.com/chainer/chainer/pull/1285/commits/09cc404d1c80a0daac778fbe5a7b3ecea93eb380)

Both name and default_name attributes are undocumented. How about documenting them and linking to them?

@kmaehashi It is the default value provided as a property. I wanted to override the behavior with an attribute, so I used default_name instead of name (note that we cannot override a property with an attribute with the same name due to the resolution order of getattr). It could be resolved in a different manner without using the name default_name (e.g. also writing a property setter to emulate attribute-like behavior, using non-data descriptor), but I did not choose them because I felt them a bit trickier.

@niboshi default_name is mentioned here. Is it enough to link to this page? https://docs.chainer.org/en/stable/reference/generated/chainer.training.Extension.html#chainer.training.Extension

@beam2d Thanks for clarification. I understood that this feature is basically for users who want to override the default name quickly (e.g., by calling ext.name = 'MyExtension' before calling Trainer.extend).
@Crissman I think @niboshi -san's comment includes the following:

Add name = None to Extension class
Add description of name under Attributes: of Extension class
Fix docs to link to both attributes using :attr:`Extension.name`,  :attr:`Extension.default_name`

Jenkins, test this please.

Jenkins CI test (for commit dd32185db648f9d0193bf514c2b73b40f15b8a53) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

LGTM! Test failures are not related to this PR.

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?


illegal Instruction on MerlinWRT armv7

I have tried to execute the v1.2.0 armv7 binary on MerlinWRT armv7 architecture machine, but got an error message of "Illegal Instruction".

ASUS AC-RT68U router running MerlinWRT 384.15

How should I got it work?
# part of config
listen:
  host: <IP_ADDRESS>
  port: 34242
  punch: true

  respond: true

cipher: chachapoly

tun:
  dev: nebula1
  drop_local_broadcast: false
  drop_multicast: false
  tx_queue: 500
  mtu: 1300
  routes:
  unsafe_routes:

firewall:
  conntrack:
    tcp_timeout: 120h
    udp_timeout: 3m
    default_timeout: 10m
    max_connections: 100000

  # The firewall is def
  outbound:
    # Allow all outbound traffic from this node
    - port: any
      proto: any
      host: any

  inbound:
    # Allow icmp between any nebula hosts
    - port: any
      proto: any
      host: any

    # Allow tcp/443 from any host with BOTH laptop and home group
    - port: 443
      proto: tcp
      groups:
        - laptop
        - home

Can you test with the arm-5 binary?

Can you test with the arm-5 binary?

well, I got another error message
INFO[0000] Firewall rule added                           firewallRule="map[caName: caSha: direction:outgoing endPort:0 groups:[] host:any ip:<nil> proto:0 startPort:0]"
INFO[0000] Firewall rule added                           firewallRule="map[caName: caSha: direction:incoming endPort:0 groups:[] host:any ip:<nil> proto:0 startPort:0]"
INFO[0000] Firewall rule added                           firewallRule="map[caName: caSha: direction:incoming endPort:443 groups:[laptop home] host: ip:<nil> proto:6 startPort:443]"
INFO[0000] Firewall started                              firewallHash=7f575bbec56ca8fa6eb9318659edfa5bfa68ccbd046e7a6225a4047adad5cae0
FATA[0000] Failed to get a tun/tap device                error="no such file or directory"

Make sure you are running as root or with sudo. If you are, then it could be that your Linux kernel is not compiled with tuntap support.

You can try to follow the steps in section 2. Configuration here: https://www.kernel.org/doc/Documentation/networking/tuntap.txt

Make sure you are running as root or with sudo. If you are, then it could be that your Linux kernel is not compiled with tuntap support.

I'm sure about running as root :)
And I try to run modprobe tun before nebula.
# Linux RT-AC68U-0510 <IP_ADDRESS>brcmarm
modprobe tun
./nebula -config config.yml

got error again
FATA[0001] Failed to open udp listener                   error="unable to set SO_REUSEPORT: protocol not available"

Ah, you must have a pretty old kernel. Can you tell me what uname -a is? I think we only really need to set SO_REUSEPORT when listen.routines > 1, so I'll try to make a build for you with that change so you can test it.

Ah, you must have a pretty old kernel. Can you tell me what uname -a is? I think we only really need to set SO_REUSEPORT when listen.routines > 1, so I'll try to make a build for you with that change so you can test it.

# uname -a
Linux RT-AC68U-0510 <IP_ADDRESS>brcmarm #1 SMP PREEMPT Sun Mar 22 22:37:44 CST 2020 armv7l Merlin-Koolshare-rtac86u

I really appreciate your help!

Can you test with the arm-5 build here? https://github.com/wadey/nebula/releases/tag/v1.2.0-issue222-1

Can you test with the arm-5 build here? https://github.com/wadey/nebula/releases/tag/v1.2.0-issue222-1

It works as least no error from console. Thx a lot!


Fix HAMi image is too large and uses inappropriate base image

What type of PR is this?
/kind bug

https://github.com/Project-HAMi/HAMi/pull/492
This PR modifies the final base image of hami from nvidia/cuda:12.4.1-base-ubuntu22.04 to nvidia/cuda:12.2.0-devel-ubuntu20.04, and the image size becomes 3.1G.
We only need the image nvidia/cuda:12.4.1-base-ubuntu22.04
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@zoyopei  Can you help me review this pr？

the master branch is unmaintainable


Cross compile for armv7-unknown-linux-gnueabihf

How cross compile for armv7-unknown-linux-gnueabihf?
I don't have sysroot.
mhanusek@EtHanusek:~/work/VersionControl/rs485_example$ cross test --target armv7-unknown-linux-gnueabihf
   Compiling libudev-sys v0.1.4
   Compiling nix v0.14.1
error: failed to run custom build command for `libudev-sys v0.1.4`

Caused by:
  process didn't exit successfully: `/target/debug/build/libudev-sys-fdad08bfef6d059e/build-script-build` (exit code: 101)
--- stderr
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: "Cross compilation detected. Use PKG_CONFIG_ALLOW_CROSS=1 to override"', /cargo/registry/src/github.com-1ecc6299db9ec823/libudev-sys-0.1.4/build.rs:38:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

warning: build failed, waiting for other jobs to finish...
error: build failed

@hanusek You should read the error message.


Add discards to What's new in C# 7

When looking at the code of What's new in C# 7, I noticed this comment:
<!-- Add wildcards here, if they are in C# 7
-->

Since wildcards (renamed to discards) are in C# 7, I think they should be included in the article.
cc: @BillWagner

@svick Yes, this needs to be added.  If you want to add it in your current PR, I would appreciate it.
If not, I'll get to it soon.

Adding @rpetrusha since this is in the deconstruction topic you're adding now. Ping me if you want more information.


Add example for a generic JSON exporter/importer

Great work!  Very good parser.  It would be great if the elaborated parse tree could be written out as JSON and loaded from JSON.  There seems to be some circular references in the model hierarchy that makes this difficult without some code change.

I don't have any plans to officially support a JSON exporter. The main reason is that there is no "standard" way to represent a register model in JSON (unlike the IP-XACT standard for XML). It would certainly be possible to export to a JSON format, but as you pointed out, the systemrdl-compiler's Python object model is much richer than what any JSON structure would be able to represent.
Often, it is more appropriate to see if your application can directly manipulate the python object model to export what you need, rather than using an intermediate JSON format.
Since this is a request I see pretty often, I can put together an example for how to build a generic JSON exporter, and add it to the documentation here: https://systemrdl-compiler.readthedocs.io/en/latest/examples.html
Hopefully this could be used as a starting point to extend to any custom/proprietary JSON serializers you might need.

Exporting is easier that importing.  Is there documentation on how to properly import into the object model?  In PeakRDL-ipxact I can see examples of export/import.  For export, whatever information is needed will be extracted from the model.  But for import, what exactly fields are required and what needs to be done with RDLImporter.  Documentation is missing.

I have added another example that walks through building a JSON exporter here: https://systemrdl-compiler.readthedocs.io/en/latest/examples/json_exporter.html
Regarding documentation for how to build an exporter, I agree that the documentation is missing. I have to productize that process a little bit better into an actual API before I add it to the official docs. The IP-XACT importer is a good starting point, but it is a little bit ad-hoc. I'll be enhancing the RDLImporter helper class a bit more in future releases to provide a more "official" API for this.

Thank you.! That is great information.

Just added an example on how to create your own importer: https://systemrdl-compiler.readthedocs.io/en/latest/examples/json_importer.html
This example uses some new helper functions I added to the importer class, which is now properly documented here: https://systemrdl-compiler.readthedocs.io/en/latest/api/importer.html
Note that this is not yet in published to an official release. Let me know if the tutorial and importer API accomplish what you were looking for, or if you have any recommendations.

Released v1.16.0


Policies not honered (local kvstore)

Hi, I just run into some troubles using the local kvstore.
flags and version:
cilium-agent --kvstore local -t vxlan
Cilium 0.9.90 08c1e0c4 Sat, 17 Jun 2017 21:16:47 +0200 go version go1.8.3 linux/amd64
Inserted the following two policies:
[{
        "endpointSelector": {"matchLabels":{"id":"app1"}},
        "ingress": [{
                "fromEndpoints": [
                        {"matchLabels":{"id":"app2"}}
                ]
        }]
}]

[{
        "endpointSelector": {"matchLabels":{"id":"app2"}},
        "ingress": [{
                "fromEndpoints": [
                        {"matchLabels":{"id":"app1"}}
                ]
        }]
}]

Then created two containers using:
docker run -itd --name cilium1 --net cilium-net -l "id=app1" nbruijn/cilium /bin/bash
docker run -itd --name cilium2 --net cilium-net -l "id=app2" nbruijn/cilium /bin/bash

ENDPOINT   POLICY        IDENTITY   LABELS (source:key[=value])   IPv6                     IPv4             STATUS             ENFORCEMENT
15124      Enabled       264        container:id=app1             f00d::9164:84b6:0:3b14   <IP_ADDRESS>   ready
60670      Enabled       263        container:id=app2             f00d::9164:84b6:0:ecfe   <IP_ADDRESS>   ready

After creating these containers and trying to ping the other node, they are both unreachable. Cilium monitor  shows me that the packet dropped because the policy is not met.
Also:
Tracing From: [container:id=app1] => To: [container:id=app2] * Rule 1 {"matchLabels":{"any:id":"app2"}}: match    Allows from labels {"matchLabels":{"any:id":"app1"}} +     Found all required labels 1 rules matched Result: ALLOWED L3 verdict: allowed Verdict: allowed

and:
{  "addressing": {
    "ipv4": "<IP_ADDRESS>",
     "ipv6": "f00d::9164:84b6:0:ecfe"
},
"container-id": "bf16fa1126ce8e4b98c6f7af2d50a3292d697458b2b6221e4b4efc2001d39f25",
"docker-endpoint-id": "84a5ce6ec34bb221dae00ec16631b7bf33d952c7c0a236ba44ee90de831cf0ae",
"docker-network-id": "4f4aa4fdd5cb183e648a09c1a1c80c486266a2cd804690e02710413756f8ae5a",
"host-mac": "32:71:b3:27:94:4d",
"id": 60670,
identity": {
   "id": 263,
   "labels": [
         "container:id=app2"
    ]
 },
"interface-index": 1149,
"interface-name":
"lxc84a5c",
 "mac": "b2:67:e7:c5:fe:d7",
 "policy": {
           "allowed-consumers": [],
           "build": 17,
            "id": 263,
             "l4": {
                   "egress": [],
                   "ingress": []
           }
     },
"policy-enabled": true,
"state":
"ready",
"status": [
      {
          "code": "OK",
     "message": "Successfully regenerated endpoint program",
      "timestamp": "2017-06-26T10:41:30Z"
},
{
 "code": "OK",
    "message": "Policy regenerated",
                     "timestamp": "2017-06-26T10:41:28Z"
},
 {      "code": "OK",      "message": "Successfully regenerated endpoint program",
 "timestamp": "2017-06-26T10:41:28Z"    },
   {
       "code": "OK",
       "message": "Successfully regenerated endpoint program",
      "timestamp": "2017-06-26T10:41:27Z"
 },
 {
       "code": "OK",
       "message": "Policy regenerated",
      "timestamp": "2017-06-26T10:41:26Z"
},
 {
      "code": "OK",
      "message": "Successfully regenerated endpoint program",
      "timestamp": "2017-06-26T10:41:25Z"    }  ] }

Running cilium with consul as kvstore does not give any problems.

@NBruijn The local kvstore plugin was buggy and incomplete. I've opened a PR to remove it #1235

local kvstore backend has been removed, closing this


fix display update button & change name to nickname

Few things here:
a) Fixing the display of the Update button
b) Changing new field name to nickname, I found it quite confusing as this is very close to username field and so in order to people how to extend the User model it might be good to consider making it clearer.
c) Add first_name and last_name to the user update form

First name and last name is a specific thing for western countries, but there are cultures where there is no such thing as a last name or first name. That's why the more generic name is used:
First Name and Last Name do not cover name patterns	 around the globe.


IntelliJ users are prompted to start instructions before logging in

https://github.com/getappmap/board/issues/293

Please refer to https://github.com/getappmap/board/issues/293#issuecomment-1711732452 for test instructions


partial styles

Pull request template
Checklist
https://github.com/swaponline/MultiCurrencyWallet/issues/3050

[ ] I have read the CONTRIBUTING guide
[ ] branch rebased on master
[ ] styleguide check npm run validate
[ ] good naming
[ ] keep simple
[ ] video or screenshot attached
[ ] testing instruction attached
[ ] check your PR once again

Description

Video or screenshot

What and how to test

To test it locally run git fetch && fit checkout twelveWordsCreateBug where twelveWordsCreateBug is a name of this branch (see under title) OR PRID=2812; git fetch origin refs/pull/$PRID/merge:pr$PRID && git checkout pr$PRID where PRID is number of this pull request
npm i && npm run build:mainnet then run MultiCurrencyWallet/build.mainnet/index.html in browse

не собирается?


fix(n-carousel): the transition not working as expected

closes #5993.

Since lint config is fully changed. Conflict is not solvable in github web page. I just manually committed it in 30da93c91c90


how can i restrict the height of the taglistview?

when i add several tags, which makes taglistview three lines, so how much the height of the taglistview will be ?
i init the height of the frame of the taglistview to be 20px

You can use Auto Layout, then you don’t need to set the frame manually. If you can’t, try intrinsicContentSize(), it should return the correct size.


Problem with OmitWhen executing code when condition is true

Linking MR that adds OmitWhen for reference and it does not have the example about my use case.
In my example below, when the condition is true, is randomFunction supposed to be invoked? My understanding was that when the condition is met, everything in omitWhen will be omitted.
omitWhen(condition, () => {
      randomFunction()
      test("testSomething", ()=>{})
})

Hey @hoangtran-97, thank you for this callout. Yes, this is the expected behavior. I should add it to the documentation.
omitWhen does indeed run everything in its scope. The one thing that doesn't run is the test callback. The reason for it is that Vest keeps track of the tests within it and registers them.
If there's a portion of non vest code that you'd like to omit, you can safely wrap it in a condition (note - do not put Vest api functions in a condition, as it would interfere with this internal registration)

Hi @ealush, thank you for the reply 🙏. Hope this can be added to the documentation soon.


Create Mantine palette

https://mantine.dev/
https://github.com/mantinedev/mantine

Create demo-mantine project with a palette containing Mantine components. See https://github.com/react-buddy/demo-chakra-ui for example.
Create mantine package in this repo

Remove unused imports and Popover from App.tsx
AspectRatio component has wrong documentation link.
Remove variants which are the same as default (e.g. Button - filled, ActionIcon - transparent)
Checkbox - with color/Chip - with color - add checked state by default (to see what is color)
Tabs - contentd ?
Data display -> Data Display
Progress and Divider previews do not work

see notes in doc


Travis CI failing with "HTTP Basic: Access denied" / Browserstack

Seems to be some kind of authentication problem with Browserstack via Karma and Travis ENV.

We've fixed this in https://github.com/cyclejs/cyclejs/commit/1607a945bb3a6388f370cd060bb917fd6e7679c8


Fix content-timestamp decider for symlinks

The base filesystem node class has getmtime() and getsize() functions.  Those were changed in an earlier commit to use the lstat() method if the node represented a symbolic link. However, we actually want the information of the file the symlink points to, or we can't detect changes to the mtime or size of the underlying file, and miss rebuilds if content-timestamp is used.
Added a testcase which shows the failure to rebuild from the symlinked source.
No doc impacts.
Fixes #3880
Signed-off-by: Mats Wichmann<EMAIL_ADDRESS>Contributor Checklist:

[x] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated CHANGES.txt (and read the README.rst)
[ ] I have updated the appropriate documentation

Problems with this, will revisit when I get a chance.

Except... I forgot to exclude Windows from running the test - the test harness quietly just doesn't make the symlink if asked for on that platform, so you end up without the test even being set up. Skip logic pushed.


Clarify rules about $t

(This is split from issue #15).
Norm Megill wanted to clarify the rules for $t; here is my reply:

(1) My to-do note was intended to address whether $t needs to be surrounded by whitespace (I would recommend yes, for simplicity).

That would require rewriting all the $t statements, and current code already handles it.   Personally I'd leave it be, but it's up to you.

(2) Also, it isn't clear whether anything other than whitespace can go between the start of the comment and the $t (I would recommend no, for simplicity). In the current metamath.exe, I believe that neither of these cases are currently enforced, but I can add code to 'verify markup' to enforce them.

I like that idea (idea 2), and set.mm already meets that criteria.
Should I try to add at least "only whitespace before $t rule (idea 2) to the text?

(1) My to-do note was intended to address whether $t needs to be surrounded by whitespace (I would recommend yes, for simplicity).

That would require rewriting all the $t statements, and current code already handles it. Personally I'd leave it be, but it's up to you.

I didn't mean the content of the $t comment, just the $t token itself.  All current .mm files (I checked set.mm, iset.mm, nf.mm, hol.mm, ql.mm) have the $t surrounded by whitespace.  The rest of the $t comment contents will continue to have their somewhat liberal whitespace rules.
Perhaps it is silly to single out the token "$t" for requiring surrounding whitespace, and I'm fine with leaving the spec alone in that regard.

Ah, okay. How about this:

There must be 1 or more white space characters, and only white space characters, between the $( and $t, and the $t must be followed by by 1 or more white space characters.

I think that's a reasonable clarification.

That is fine.  I will add a to-do item on my list to have the 'verify markup' command check this.

Also: I think we should document that currently only one $t is supported in the Metamath program, but say that constraint may be lifted in the future.
If we split things into files, it'd make more sense to have $t settings for that file in the file. It might even make more sense to have $t statements next to the symbol they declare. The set.mm database might never do that, but others might want to.

I agree.  Off and on I have thought about splitting up the $t, say one $t per mathbox or even per $c, but each time I've ended up deciding it's more convenient to have them all in one place, especially when making global changes to the list such as the change to XITS fonts.  So I haven't yet been sufficiently motivated to add the code to split them up.  But it is a future possibility, and I have a to-do item to look into it eventually.

Very good.  I think this issue is completely resolved, so I'm closing it.


fix: allow new services to wait for nucleus restart before install

Issue #, if available:
Description of changes:
While soft shutdown of Nucleus, create new services added to the deployment config and close them before nucleus restart. This prevents nucleus restart (in case of ota deployment/bootstrap steps) to interrupt service lifecycles. As Nucleus resumes default deployment (after all pending bootstrap steps are completed), all services in deployment config will request re-installation.
Why is this change necessary:
Nucleus restart was causing component lifecycles to interrupt
How was this change tested:

[x] Updated or added new unit tests.
[ ] Updated or added new integration tests.
[ ] Updated or added new end-to-end tests.
[ ] If my code makes a remote network call, it was tested with a proxy.

Any additional information or context required to review the change:
Documentation Checklist:

[ ] Updated the README if applicable.

Compatibility Checklist:

[ ] I confirm that the change is backwards compatible.
[ ] Any modification or deletion of public interfaces does not impact other plugin components.
[ ] For external library version updates, I have reviewed its change logs and Nucleus does not consume
any deprecated method or type.

Refer to Compatibility Guidelines for more information.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Can you explain why installation of the new services is being triggered when the service isn't even initialized? Doesn't lifecycle state machine only go from new -> installing when you requestInstall or requestStart, and that must be done explicitly for a new service only after it's been initialized through locate/locateIgnoreError?

Resolved by #1455.


scripts: fix misc-flasher runner

This makes the misc-flasher runner usable by passing the build
directory to the underlying tool.
Fixes: #20658

@chen-png thank you for your feedback. I have implemented your suggestion from https://github.com/zephyrproject-rtos/zephyr/pull/20273.

thanks for your help. I think it's great now.
I ran the sanitycheck with this runner on up squared, it's fine.
[ztest@zserver1 zephyr]$ sanitycheck -p up_squared --device-testing --device-serial /dev/ttyUSB1 -T tests/kernel/threads/ --west-flash="/home/ztest/up_squared.sh"
JOBS: 40
Building initial testcase list...
4 test configurations selected, 0 configurations discarded due to filters.
Adding tasks to the queue...
total complete:    4/   4  100%  skipped:    1, failed:    0
3 of 4 tests passed (100.00%), 0 failed, 1 skipped with 0 warnings in 82.67 seconds
In total 4 test cases were executed on 1 out of total 213 platforms (0.47%)
Hardware distribution summary:
up_squared (None): 3


Make ByTitle, ByLinkTitle and ByParam language aware

Fixes #2180

There are some work left in the above, which I may or may not get to before I merge this:

The template sort function.
Page group sorting
Taxonomy sorting
The title fallback in the default page sort

/cc @jry2 @jmooring @CedricRaison


[Backend] As a user, I can sign in with email and password

https://github.com/Shayokh144/ic-survey-swiftui-combine/issues/6
What happened
Need to post login data to server and receive response.
Insight

Post login data.
Decode response.
Send success and error.
Used dataTaskPublisher

Proof Of Work
https://user-images.githubusercontent.com/13636465/200380114-0e6ca8f1-0caf-4d2d-a15c-749b92a62c3d.mov
Remarks
TODO's will be done at integration task.

@ducbm051291 thank you for noticing branch name.. will be careful for next PR's