Further clarification to allowed roles on list items

ARIA in HTML is clarifying the allowed roles on li elements.

No role if the parent list element has an implicit  or explicit list role.
Otherwise, any role.
NOTE: See ul, ol, or menu for allowed roles for list elements.

This update disallows any role on an li element so long as it is a child of a ul, ol or menu element is exposed as an implicit or explicit role=list.
A role of listitem would produce a warning for unnecessary use of a redundant role.
However, if the parent ul, ol or menu has had its implicit list role overwritten, then the li may have any role so long as that role is an allowed role of its ancestor elements.
For example, the following li may have a role=button specified as the parent list element as a role=none

...

However, as the following ul has a role=menu specified, the role=button is not allowed.  The only allowed roles for the li would be menuitem, menuitemcheckbox or menuitemradio.

...</button>

similar to the following, as while the ul has a role=none, it is a child of a role=tablist, so therefor the li may only have a role that is an expected descendant of a tablist.

...

Finally, even though the following would be flagged for invalid HTML, the li is again allowed any role as it is exposed as a generic role when not correctly nested within a list element:

   ....

...

   ...

Additional test cases provided here.

Thanks, @scottaohara Updated to reflect the new rules. I noticed you were using roles from ARIA 1.2 that were not included in the rule set, so I added the following roles:

blockquote
caption
code
deletion
emphasis
generic
insertion
meter
paragraph
strong
subscript
superscript


Standardize api exceptions

Description
Creates a few standard exceptions that our API knows how to catch and apply a standard format for.

All error responses will be JSON objects with at least an "error" key and optionally additional values from now on

Test plan
CI tests

We have a stubbed error folder/module, could we add the code for exception classes to that folder? Then we could nicely import them like from aspen.app.error import ServerException.

i like lucia's idea of adding exceptions to the error folder.

i like lucia's idea of adding exceptions to the error folder.

Yep, I like it too. I'll move these!


Added action filtering by function

This pull request adds a way for the user to filter actions by providing a custom filter function instead of string arrays. The previous functionality of filtering with strings is (of course) retained.
In my case, it proved to be useful when working with namespaced actions to allow me to ignore entire namespaces in a concise way.
Before:
const options: LoggerOptions = {
  filter:  {
    blacklist: ['entity/set-value', 'entity/update-value',
      'entity/do-something', 'entity/some-other-action'
    ]
  }
}

After:
const options: LoggerOptions = {
  filter: (actionType: string) => !actionType.startsWith('entity/');
}

Sorry for the delay on this, lgtm. Would you mind updating the README to reflect this new option? Once that is done I will merge in, thanks! 👍


Deserialize moduleCache beforehand.

Fixes dependency problem in #160.

Merged manually. https://github.com/mzgoddard/hard-source-webpack-plugin/commit/6d583be87c708f547bb985360757513ae4c0e116


Radiation pulses need a shader

The green circle looks like dog poop (as the person who made it) and ideally it'd be a shader instead.

I wrote one and it looks nice [citation needed]. Unfortunately it's based off the singulo code, so multiple instances of it become completely broken, and it includes a wonderful black screen flicker when one spawns remotely near you.
https://user-images.githubusercontent.com/602406/143833341-df18e2ac-0e26-4eea-ae7a-2bd5a643ce06.mp4

(Also there was a second version that's entirely the same, but uses a smooth gradient instead.)
https://user-images.githubusercontent.com/602406/143834173-5a5b286b-870b-4c8d-9baf-c9cdb372fb83.mp4

Update: It's FoV-aware and supports multiple instances now!
https://user-images.githubusercontent.com/602406/143866733-446fff33-a806-4314-9083-db63de01f4f6.mp4


Show layers using a certain token on Multi-Inspector

Is your feature request related to a problem? Please describe.
🟡 The problem is being able to identify the layers where undesired tokens are applied, especially in a frame that has many UI components and tokens used. We can already see the number of layers to which a token is applied to, and I believe thru Figma API we can get the names of those layers too.
Describe the solution you'd like
🟡 Under the Inspect feature, the user is able to use the Multi-Inspector in order to view the layers (e.g. names of layers) using a certain token.
Describe alternatives you've considered
🟡 I currently manually have to guess where broken tokens are applied. Could not figure out a quick way to automatte things.
Additional context
A simple & quick solution can be a tooltip/dialog box that shows up a list of layers' names using that broken color.bg.disabled.

I think this is a duplicate of https://github.com/six7/figma-tokens/issues/513
If you agree, let's close this one?

Yes I agree, let's close this one. Thanks Jan!


Get logging working in ray

Log the incremental output from the server
Handle errors that happen partway through processing
Move ray logging setup into default sycamore init
for testing, add a sycamore.shutdown() to clean up sycamore.init()
Add support for not loading the model when we're only going to call the server
Add assertions to code to check for user error
Switch partitioner to using logger. instead of logging. It's better style, it turned out to
not be necessary.
Remove caching from aryn_config. It caused problems when called with different parameters
If we're going to re-introduce this we can use @cache which will respect parameters.
Make SycamorePartitionerExample notebook runnable. The example failed because of missing
files/directories.

There is a leftover import of the old ray_logging_setup at test_partition.py:70


Error accessing mounted directories using a non-root container user.

Hi @konstruktoid,
I am trying to run an instance of step-ca. As per the Dockerfile of this image the process inside the container is run as the step user which is not the root user. Since my docker imstallation is rootless, The step user inside the container doesn't have write access to the mounted volumes. This issue exists with other images as well where the process inside the container is not run as root.
I also verified that the host directory that is mounted on the container is owned by the dockeruser which is running the docker daemon as rootless.
I saw few articles online that maps the user id from host user to container user but so far I couldn't get it to work.
Could you please provide your 2 cents and assist on what I could be doing wrong?
Thanks & Regards,
Utkarsh Vishnoi

@Utkarsh-vishnoi, I haven't had the time to test yet, but have you tried setting a user using --user=""
(https://docs.docker.com/engine/reference/run/#user)?


Add rpkg template for COPR autobuild

This commit removes .autocopr/netavark.spec in favor of
netavark.spec.rpkg to account for new recommendations for COPR builds.
<EMAIL_ADDRESS>Signed-off-by: Lokesh Mandvekar<EMAIL_ADDRESS>@baude @flouthoc @Luap99 @mheon @rhatdan PTAL . Similar to the change I did for podman

added dependency on aardvark-dns. PTAL

can we get eyes on this please?

LGTM

i'm gonna remove the aardvark dependency for now until COPR is migrated to f35

PTAL once again please

LGTM

/lgtm


can we get validation loss?

Hi, I tried to integrate your fabulous evaluation framework into my training scripts based on HF trainer.
I modified HFLM to support loaded HF model and triggers evaluate with custom callbacks.  Currently, the evaluation works like a charm and I get to see evaluation metrics throughout the training.  And it would be a lot nicer to see validation loss curve.
So, is it possible to get validation loss?

You can use the JsonPerplexity task to evaluate perplexity on an arbitrary text dataset.


bump k8s dependencies

could you bump the k8s dependencies to latest?

@georgettica Thanks for reporting. Can you take a quick look if #3 looks good?


Updates have stopped working

Setup: HASSOS running on bare metal.
Partial Logs: https://pastebin.com/h30j6vKY
Setup on 26/11/2024 and was able to get updates just fine. No longer working morning of 27/11/2024 without making any changes.
In the lgos it appears to be getting data for other stops but not what I want?

Can you provide a copy of the gtfs_rt section of your configuration please so we can see what you want.

Hi Mark, copied the example and added the appropriate lines that pass through my station. (Not sure if there is a better way to concatenate for all trains via my station?)
Example entry for Queensland, Australia

platform: gtfs_rt
trip_update_url: "https://gtfsrt.api.translink.com.au/api/realtime/SEQ/TripUpdates"
vehicle_position_url: "https://gtfsrt.api.translink.com.au/api/realtime/SEQ/VehiclePositions"
route_delimiter: "-"
departures:

name: Redcliffe Train (Bald Hills1)
route: RPBR
stopid: "600452"
icon: mdi:train
service_type: Train
name: Redcliffe Train (Bald Hills2)
route: RPIP
stopid: "600452"
icon: mdi:train
service_type: Train
name: Redcliffe Train (Bald Hills3)
route: RPSP
stopid: "600452"
icon: mdi:train
service_type: Train
name: Redcliffe Train (Central1)
route: BRRP
stopid: "600024"
icon: mdi:train
service_type: Train
name: Redcliffe Train (Central2)
route: IPRP
stopid: "600024"
icon: mdi:train
service_type: Train
name: Redcliffe Train (Central3)
route: RWRP
stopid: "600024"
icon: mdi:train
service_type: Train
name: Redcliffe Train (Central4)
route: SPRP
stopid: "600024"
icon: mdi:train
service_type: Train

Hi, looking at the Translink raw time feed it looks like they are not using platform 1 at Bald Hills today for some reason.  Here is an example of the 11:12am service to Springwood.
      stop_sequence: 11
      arrival {
        delay: 52
        time:<PHONE_NUMBER>
        uncertainty: 30
      }
      departure {
        delay: 82
        time:<PHONE_NUMBER>
        uncertainty: 30
      }
      stop_id: "600453"
      schedule_relationship: SCHEDULED
    }

As you can see the stop id is '600453' (Bald Hills Platform 2) and not '600452' (Bald Hills Platform 1) as per the timetable.  There is a note on the translink app about the trains being "Now on Platform 2" (see below).  I assume that they will return the scheduled service at some point so you can either wait it out or change the stop id in your configuration.  There is no service update about the change so I am not sure what they are doing.

Hi Mark,
Interesting, I got on at Platform1 this morning and it wasn't working then. I do note now that the "Central4" one is getting data but it is the only one. Platform 2 is the only bi-directional at Bald Hills, 1 and 3 are single direction.
Is there any way to either ignore the platform or extract the platform to display? Seems like sometimes more useful way to use these kind of feeds would be "I want any train from X to Y" and then you may have additional information like express or all stops etc.

Unfortunately the data provided by the GTFS Realtime feed is quite limited.  Here is an extract of everything it provides.
entity {
  id: "TU-30981476-QR 24_25-38066-DK30"
  trip_update {
    trip {
      trip_id: "30981476-QR 24_25-38066-DK30"
      start_time: "10:46:00"
      start_date: "20241127"
      schedule_relationship: REPLACEMENT
      route_id: "RPSP-3962"
    }
    stop_time_update {
      stop_sequence: 11
      arrival {
        delay: 52
        time:<PHONE_NUMBER>
        uncertainty: 30
      }
      departure {
        delay: 82
        time:<PHONE_NUMBER>
        uncertainty: 30
      }
      stop_id: "600453"
      schedule_relationship: SCHEDULED
    }
}

So it is route and stop id dependent.  There is no way to link the stop_id provided to the 'Bald Hills' station and it's other platforms.   Theoretically, this can be done by downloading the GTFS static database (which is a ZIP file containing text files that need to be imported into a SQLite DB and is updated at least once per day by Translink) but that would had a whole lot of complexity to the integration.  I have looked at doing this but every transport operator does things slightly differently (because the GTFS and GTFS-RT standards let them and even if they don't comply with the standards there are no consequences for them) so I decided not to progress it.


Aave lesson: Final argument must be a dict of transaction parameters that includes a from field specifying the sender of the transaction

I've tried deploying on both RInkeby and Kovan and received the same error: Here is my code.
get_weth.py
from scripts.helpful_scripts import get_account
from brownie import interface, config, network

def main():
    get_weth()

def get_weth():
    """
    Mints WETH by depositing ETH.
    """
    account = get_account
    weth = interface.IWeth(config["networks"][network.show_active()]["weth_token"])
    tx = weth.deposit()({"from": account, "value": 0.1 * 10 ** 18})
    print("Received 0.1 WETH")
    return tx`

Here is my helpful scripts`from brownie import accounts, network, config

from dotenv import load_dotenv
LOCAL_BLOCKCHAIN_ENVIRONMENTS = ["development", "ganache-local", "mainnet-fork"]

def get_account(index=None, id=None):
    if index:
        return accounts[index]
    if network.show_active() in LOCAL_BLOCKCHAIN_ENVIRONMENTS:
        return accounts[0]
    if id:
        return accounts.load(id)
    if network.show_active() in config["networks"]:
        return accounts.add(config["wallets"]["from_key"])
    return None`

and here is my config file `dependencies:
-<EMAIL_ADDRESS>compiler:
 solc:
   remappings:
    - '<EMAIL_ADDRESS>dotenv: .env
networks:
 kovan:
   weth_token: '0xd0A1E359811322d97991E03f863a0C30C2cF029C'
 mainnet-fork:
   weth_token: '0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2'
wallets:
 from_key: ${PRIVATE_KEY}`

Hello @kjones86
tx = weth.deposit()({"from": account, "value": 0.1 * 10 ** 18})
you have an error here, the "from" and "value" should be inside deposit parenthesis.

Thanks for your response @cromewar the tx line was correct but it looks like I was missing a () from get_account


Support for Launch URLs

https://developers.google.com/maps/documentation/ios-sdk/urlscheme

As per https://developers.google.com/maps/documentation/urls/get-started , We don't need a Google API key to use Maps URLs. This features allows users to specify map configuration and open Google maps application (installed on their phones, nothing to do with maps SDK) with that config .

As per https://developers.google.com/maps/documentation/urls/get-started , We don't need a Google API key to use Maps URLs. This features allows users to specify map configuration and open Google maps application (installed on their phones, nothing to do with maps SDK) with that config .

I'll keep this open to evaluate user interest in this feature. Feel free to ping me here if you feel this will be a good addition!

I'll keep this open to evaluate user interest in this feature. Feel free to ping me here if you feel this will be a good addition!

I think this would be a little out of scope for this project. The Maps URL API is already very simple and elegant. IMHO, it just adds noise and maintenance headaches to this library. Instead I would just link to the Maps URL docs in the readme for example.

@selected-pixel-jameson let's discuss your comment on #85 here. Why would you like this to be implemented in the SDK? In my honest opinion it shouldn't be too difficult to just use the URL api google already provides. Buy maybe I am missing something. Could you please elaborate on that?

This feature will not implemented in the foreseeable future. I have added documentation about this in the (still WIP) docs: https://capacitor-community.github.io/capacitor-googlemaps-native/#/about/out-of-scope-features
I am going to close this issue for now.


Fix: Display prompt when trying to delete folder that's too big for recycle bin

Resolved / Related Issues

Closes #8358
Implemented the siggestion of @BanbanProduction to check the size of the folder being deleted (issue only seemed to occur with folders). The solution involves forcing a permament deletion if the folder might be too big for the bin and may cause the error.
This solution is not very pretty for a couple reasons:
Some overhead to compute the size of the folder (although it seems low - see Validation)
Uses the registry to get the size of the bin, which may hit or miss on different machines
Only guesses if the folder is too big, going by the smallest of the recycle bins on the drives (I know of no way to discern them in the registry)
As a consequence of the previous point can force permanent deletion for big folders unnecessarily to recycle bin size differences

As alternate fixes I can think of adding a prompt for the user to permanently delete files manually in the error message, checking the box to permanently delete for the user, but allowing them to uncheck to account for incorrect guesses, or keeping the same implementation with a better way to query the maximum recycle bin size (which I could not find anywhere).

Adresses #9895
Also cleared some nullable reference type warnings in the related files. Those are in separate commits so can be merged regardless of the rest of pull request

Validation

Confirmed that the issue exists on main
Verified that the new behavior only affects folders
Doesn't slow the program down noticably even when computing the size of a folder with 100 files

Some overhead to compute the size of the folder

I think this is fine as long the check happens after failing to delete the items.

As a consequence of the previous point can force permanent deletion for big folders unnecessarily to recycle bin size differences

We should avoid doing this without confirmation from the user, the main thing is to display a proper error and allow the user to move forward from there.

As a consequence of the previous point can force permanent deletion for big folders unnecessarily to recycle bin size differences

We should avoid doing this without confirmation from the user, the main thing is to display a proper error and allow the user to move forward from there.

Sounds good, I will look into having a proper message for the user. This issue is tricky because the failure seems to happen in OS code, so resolving the issue directly is challenging. I will try to identify the error somehow and add a specific error message similar to "folder could not be moved to the recycling bin".

It might be a bit tricky to intercept the error.


Add test to verify generated files

Also updates generated out-of-date deepcopy file.

@joelddiaz @twiest this should be ready for review now


IIR Filter in android app

Um working with Superpowered SDK in android , but IIR filter in it is of specific degree
so can use your library in this to increase the degree of the filter , for example >> iirfilter_lowPass(300 Hz , degree= 5)

Hi bro I need one example for online low-pass filter for example , I can't find any


[WIP] Graphy graph

Starting to spike out a graph branch..
This PR:

Brings in the d3 library for graphing
Spits out a JSON version of the log data (for easier graphing / manipulation)

Todo:

[ ] Make sure this JSON format will work
[ ] Hook up d3 to actually chart this data

/cc @josh

JSON is formatted like:
[{"dateTime":"Tue May 31 2016 16:39:14 GMT-0400 (EDT)","eventType":"open"},{"dateTime":"Tue May 31 2016 16:39:14 GMT-0400 (EDT)","eventType":"active"}]

:metal:

Trying to figure out a good format for graphing and I think we're going to need to manipulate the JSON a bit to determine the following metrics (to start):

App opens / exits per day
Total length of active time per day
Total length of active time  per hour
Total length of idle time per day
Total length of idle time per hour
Stretches of consecutive active time per day
Stretches of consecutive idle time per day

It kind of hurts my brain to think about how to calculate this and process it. @josh do you have any suggestions for either other metrics we should track or ways to calculate the metrics with the JSON?

Closing this in favor of refactored log format.

Closing this in favor of refactored log format.

🔜


LZ4 support

Output of restic version
restic 0.16.0 compiled with go1.20.6 on linux/amd64

What should restic do differently? Which functionality do you think we should add?
Please add suport for the LZ4 compression algorithm and/or similar light-weight algorithms. LZ4 is way faster and only compresses a wee tad worse, which I can live with.

What are you trying to do? What problem would this solve?
Speed up backups

Did restic help you today? Did it make you happy in any way?
It generally works wel, but everything an be improved :)

Do you have any numbers that back up that claim? What performance region are we talking about? Adding another compression algorithm complicates the repository format.

http://compressionratings.com/ratings.html
If you want to be pedantic for compression speed vs size then you may as well use the real-world ratings table.

@g33kphr33k what do these compression ratings tell me? Based on a quick glance, there seem to be no results using zstd?

LZ4 compression can be much worse than zstd, depending on the data. If a lighter-weight compression algorithm is necessary, I think it would be better to investigate an algorithm that produces the zstd format. E.g., compress/flate has a HuffmanOnly setting that skips the LZ77 (backwards matching) part of compression. Something like that should be possible to implement in klauspost/compress/zstd as well.

Here's an example of where LZ4 is absolutely terrible: lists of hashes. This program creates a list of "hashes", i.e., random hex strings:
package main

import (
	"crypto/sha256"
	"encoding/hex"
	"math/rand"
	"os"
)

func main() {
	r := make([]byte, sha256.Size)
	p := make([]byte, 1+2*sha256.Size)
	p[len(p)-1] = '\n'

	for i := 0; i < 50000; i++ {
		rand.Read(r)
		hex.Encode(p, r)
		os.Stdout.Write(p)
	}
}

The output of this is 3250000 bytes, about 3.1MiB. Let's run the reference implementations of lz4 and zstd on its output:
$ /usr/bin/time lz4 --fast hashes
Compressed filename will be : hashes.lz4
Compressed 3250000 bytes into 3250019 bytes ==> 100.00%
0.00user 0.00system 0:00.01elapsed 93%CPU (0avgtext+0avgdata 8016maxresident)k
0inputs+6352outputs (0major+1692minor)pagefaults 0swaps

$ /usr/bin/time zstd -1 hashes
hashes               : 50.97%   (3250000 => 1656489 bytes, hashes.zst)
0.01user 0.00system 0:00.01elapsed 110%CPU (0avgtext+0avgdata 7796maxresident)k
0inputs+3240outputs (0major+1447minor)pagefaults 0swaps

$ /usr/bin/time lz4 --best hashes
Compressed 3250000 bytes into 2493411 bytes ==> 76.72%
0.17user 0.00system 0:00.18elapsed 99%CPU (0avgtext+0avgdata 7444maxresident)k
0inputs+4872outputs (0major+1580minor)pagefaults 0swaps

On this data, zstd in its fastest mode compresses much better than LZ4's best mode, and runs much faster. I bet the Go implementations' performance will be very similar.

Closing as zstd seems to be the best general choice and support multiple compression algorithms would add complexity.


[web][edge] Launch test for Edge Windows

Run one launch test on Windows Cirrus to launch Edge.
This test will be testing test runner Edge integration and felt windows script.
For the engine unit tests we will be using LUCI.

Our plan is to continue using Chromium as a proxy for all Chromium-based browsers, including Chrome, Edge, Opera, Brave, etc.


Supporting Ethernet connectivity on Olimex PoE board

Merging the Ethernet implementation into main branch, after testing that nothing was broken.


broken (?) typing on JWTStrategy

Describe the bug
mypy can't seem to figure out the type of the JWTStrategy class on its own. The full example in the tutorial doesn't raise any typing error as the class is used within a function with a typed return value.
When assigned to a standalone variable, typing issues occur. Below is an example with a comparison with CookieTransport which doesn't raise any typing errors.
To Reproduce
Steps to reproduce the behavior, in blah.py below:
from fastapi_users.authentication import CookieTransport, JWTStrategy

cookie_transport = CookieTransport(cookie_max_age=3600)
jwt_strategy = JWTStrategy(secret="SECRET", lifetime_seconds=3600)

$ mypy blah.py
blah.py:4: error: Need type annotation for "jwt_strategy"  [var-annotated]
Found 1 error in 1 file (checked 1 source file)

I apparently need to explicitly mention the type to make mypy happy
jwt_strategy: JWTStrategy = JWTStrategy(secret="SECRET", lifetime_seconds=3600)

Disclaimer: I'm a total typing noob.
I tried removing the Generic type on JWTStrategy but that doesn't seem to help. I have no idea what the correct solution is.
Expected behavior
No mypy error, as with CookieTransport.
Configuration

Python version : 3.11.9
FastAPI version : 0.109.2
FastAPI Users version : 13.0.0

FastAPI Users configuration
N/A
Additional context
N/A

Hi. JWTStrategy is a Generic, so you need to provide types:
strategy = JWTStrategy[User, UserIdType](secret="abc", lifetime_seconds=3600)

User is a subclass of SQLAlchemyBaseUserTable
UserIdType is User's id type: UUID or any other type like int / str


Minor documentation update

Missing a comma after: album       : 'Absolution' in the docs

Thanks, I just corrected it.


Can I delete an anonymous Plunker?

Hello there,
I have used Plunker for about 2 days as an anonymous user. And I accidentally delete all of my local temp files (perhaps my security system deleted it). So I lost my deletion authorization of my Plunker. Unfortunately, there is some reason not to open the source code in my anonymous Plunker.
Is there any way to delete anonymous Plunker link? I googled this, but I couldn't find how to. Please help me.

@orlein anonymous plunks can only be deleted while you maintain the session identifier in your browser's cache. This is designed as a convenience so that users can participate without needing to become registered users.
Why do you feel the need to delete your anonymous plunk?

@ggoodman It sounds like I can't do nothing about it... anyway, Because the plunker has some sensitive data from my company, which has some strict security rules. But if it couldn't be searched by Google or something, It seems to be okay to ignore the plunker. Thank you for the reply.

@orlein if you have some string that I could use to locate the plunk in the database, I can delete or sanitize it.

@ggoodman I don't know exactly what data or string snippets are in there. But the code has over 280,000 lines of single JavaScript object and short code for showing the data into grids and charts.


Use embedded databases for testing

For testing, it'd be worth adding support for embedded databases, i.e. ones that run inside the application without setting up and loading an external database.
Possible candidates:

Neo4j, using the ImpermanentGraphDatabase class and my APOC CSV loader procedure)
PostgreSQL, using Yandex's embedded version and the SQL dump of an existing PostgreSQL instance (that was previously filled from CSVs)
Sesame (and RDF4J) have in-memory implementations. Jena also has one.

Closed as wontfix, let's roll with dockerized systems -> #148.


how to get the selected row on checkbox?

@flyingcoder Create a ref to vuetable and use selectedTo data property yo access it.
  this.$refs.vuetable.selectedTo[x]

@ratiw
this.$refs.vuetable.selectedTo is just contain track-by column values.
how do i get all data of selected row?


isPlaying() gives wrong value initially

Bug description
When you check the wavesurfer's isPlaying method directly after starting the playback, you get "false", because the media's "currentTime" is still 0. Why do you check for the currentTime in the isPlaying method?

As clarified here, this is the expected behavior as per the MediaElement spec.


Path returned by external command (wsl.exe) is mishandled by Powershell

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
With WSL2 enabled and at least one linux distribution installed execute from Powershell prompt:
wsl.exe echo "`$(echo `"C:\Windows\System32\wsl.exe`")"

wsl.exe echo "`$(wslpath `"C:\Windows\System32\wsl.exe`")"

Both commands executed in cmd.exe and powershell.exe (PSVersion: 5.1.19041.2364) produce expected output, pwsh.exe (PSVersion: 7.3.2) does not.
Expected behavior
PS> wsl.exe echo "`$(echo `"C:\Windows\System32\wsl.exe`")"
C:\Windows\System32\wsl.exe

PS> wsl.exe echo "`$(wslpath -u `"C:\Windows\System32\wsl.exe`")"
/mnt/c/Windows/System32/wsl.exe

Actual behavior
PS> wsl.exe echo "`$(echo `"C:\Windows\System32\wsl.exe`")"
"C:WindowsSystem32wsl.exe"

PS> wsl.exe echo "`$(wslpath -u `"C:\Windows\System32\wsl.exe`")"
"C:WindowsSystem32wsl.exe"

Error details
No response
Environment data
PS> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.3.2
PSEdition                      Core
GitCommitId                    7.3.2
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

The behavior is as designed: Ultimately, you're executing the following in your WSL shell (which is bash by default):
# -> 'C:WindowsSystem32wsl.exe'
$ echo C:\Windows\System32\wsl.exe

Unquoted \ is the escape character in POSIX-compatible shells, and it is only needed to escape \ itself, ", or '; it has no effect on any other character that follows, but the \ is removed.
What you're looking for is:
wsl -e wslpath 'C:\Windows\System32\wsl.exe'

The behavior is as designed

I am not sure which design you are referring to (considering cmd.exe and powershell.exe behavior), but here is an output from previous version of pwsh.exe:

I will edit the original post to make it clearer that this bug has appeared with a new 7.3.* version(s) of pwsh.exe.

Good point, I forgot about the aspect of the  breaking 7.3 change that corrected the broken-since-v1 argument-passing to external programs with arguments containing embedded double quotes.
To get the old behavior back, use the following.
& {
  $PSNativeCommandArgumentPassing = 'Legacy'
  wsl.exe echo "`$(echo `"C:\Windows\System32\wsl.exe`")"
}

See https://github.com/PowerShell/PowerShell/issues/18568#issuecomment-1315912404 for the backstory, including how the old behavior will probably become the default again, because the change broke too much code.

P.S.: It's a moot point, but it's baffling to me that your commands ever worked; it seems that wsl.exe adds its own quirks:

With Legacy in effect, PowerShell constructs the following raw command line for wsl.exe echo "`$(echo `"C:\Windows\System32\wsl.exe`")", which is broken, because the embedded " chars. lack escaping:

wsl.exe echo "$(echo "C:\Windows\System32\wsl.exe")"

wsl.exe should parse the broken last argument as verbatim $(echo C:\Windows\System32\wsl.exe), with the non-escaped, meant-to-be-embedded " chars. getting removed.

That is, what should ultimately be executed in bash is the equivalent of the following, which does not preserve the \ chars. (-> verbatim C:WindowsSystem32wsl.exe):

echo "$(echo C:\Windows\System32\wsl.exe)"

Thanks for the links with background info. I have actually seen $PSNativeCommandArgumentPassing mentioned in one of 7.3.* releases notes, but the way it was phrased did not raise any red flags for me in terms of path handling.
After reading the info and some experiments it seems the main problem I was having was with paths surrounded by ``"(escaped double quote). Replacing them with single quotes where bash expansion was not needed solved the problem even without changing$PSNativeCommandArgumentPassing`.
Thanks for the help.

Yeah, the magnitude of this breaking change wasn't fully considered and it certainly was inadequately announced.
Using '...'-quoting is certainly a way to avoid the issue, as also used in the
wsl -e wslpath 'C:\Windows\System32\wsl.exe' solution from my earlier comment.
Using -e is generally preferable, because it avoids involvement of the shell on the WSL side, which complicates matters; if you do need to use the shell, call its CLI explicitly via -e, e.g.
wsl -e  bash -c 'wslpath ''C:\Windows\System32\wsl.exe'''

@mklement0
Appreciate your input, thanks.

Glad to hear it was helpful, @wikiped.
I don't necessarily expect you to be interested in the following, but to further my own understanding I
dug deeper into wsl.exe's behavior, and why I think in your particular case two broken behaviors canceled each other out to yield the desired result (it's an amended version of my previous comment, which I've hidden):

With Legacy in effect, PowerShell constructs the following process command line for wsl.exe echo "`$(echo `"C:\Windows\System32\wsl.exe`")", which, is broken, because the embedded " chars. lack escaping:

wsl.exe echo "$(echo "C:\Windows\System32\wsl.exe")"

wsl.exe should parse the broken last argument as verbatim $(echo C:\Windows\System32\wsl.exe), with the non-escaped, meant-to-be-embedded " chars. getting removed - that's how Windows CLIs based on Microsoft's compilers / the Windows API parse such arguments.

However, wsl.exe apparently does not parse its process command line into individual arguments, but passes the relevant part of its process command line as-is to the bash CLI (via its -c parameter), and as part of a Bash command line "$(echo "C:\Windows\System32\wsl.exe")" does work, because the " inside $(...) do not require escaping (sam as as inside PowerShell).

In other words: what is ultimately executed is echo "$(echo "C:\Windows\System32\wsl.exe")" as a Bash command, which happens to work.

By contrast, with Standard in effect, PowerShell correctly produces the following process command line: wsl.exe echo "$(echo \"C:\Windows\System32\wsl.exe\")" - that is, following the most widely observed rules for argument-passing on Windows, the embedded " chars. - meant to be a literal part of the argument, must be \-escaped, with the target CLI then (a) _retaining _ the embedded " as part of the argument and (b) removing \, the escape char.

However, if that is passed on as-is to Bash, broken behavior ensues, because Bash now considers the \" an escaped " to be taken literally, prints it, and considers what follows an unquoted string, resulting in the previously discussed \ removal
In other words: what is ultimately executed in Bash is echo "$(echo \"C:\Windows\System32\wsl.exe\")", which yields verbatim (broken) "C:WindowsSystem32wsl.exe"

In short:

It isn't just PowerShell (in Legacy mode) that is broken; wsl.exe inappropriately relays part of its raw Windows process command line as-is to Bash.

The wsl -e  bash -c '...' technique avoids this problem on the WSL side:

Its works as-is in Standard mode:

wsl -e  bash -c 'echo "$(echo "C:\Windows\System32\wsl.exe")"'

In Legacy mode you have the confusing need to manually \-escape all " chars.:

wsl -e  bash -c 'echo \"$(echo \"C:\Windows\System32\wsl.exe\")\"'

This is super useful - thank you very much.
It should probably be part of WSL Tutorial


Add a way of getting the Horizonite Forge

There was an old recipe you can use if you want. Or you can make a different way of getting it.

Fixed


minor fixes, style tweaks

Replace "" with " in example code, XML element attributes
Adjust some indentation
Remove unprintable character (following "XML bomb")
Remove extra  from Html.Raw() example

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

@kitdotnet : Thanks for your contribution! The author, @RodSan, has been notified to review your proposed change.

#please-close


Docs: Refactor 'get started' guide, refactor 'references', add several index pages

Refactor 'Get started' guide based on David/Joe discussion and notes
Minor updates to 'Folder structure' and 'Docker' files
Add index pages for Create a plugin, Develop a plugin, Extend a plugin, and References so that these could be linked to from 'Get started' guide
Refactor 'References' section to include prompts and CLI commands
Fix links in docs and packages broken by refactoring of 'References'

Note: Upon publication of this PR, links and redirects to plugin.json reference doc in Grafana repo must be updated.

Replaced by https://github.com/grafana/plugin-tools/pull/484 and future PRs.


Fix 1212

Types of Changes
Prerequisites
Please make sure you can check the following two boxes:

[x] I have read the CONTRIBUTING document
[x] My code follows the code style of this project

Contribution Type
What types of changes does your code introduce? Put an x in all the boxes that apply:

[x] Bug fix (non-breaking change which fixes an issue, please reference the issue id)
[ ] New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[x] All new and existing tests passed

Description
I have made the fixes as described in #1212. During testing, I added .NET 9.0 to the TargetFramework, but this change is not included in the commit.

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

In Japan, the new year is just around the corner, but before 2024 ends, I was able to send my first pull request to an OSS project. Wishing you a Happy New Year!

In Japan, the new year is just around the corner, but before 2024 ends, I was able to send my first pull request to an OSS project. Wishing you a Happy New Year!

Happy new year and thanks for your contribution!

Thank you for the review. I’m very happy to have been able to contribute to OSS for the first time!


Using AzCopy return values on Windows CMD files

This is not a bug report, this is a suggestion which would help a lot of people I'm sure.
I use AzCopy in Windows batch files (.CMD) and when operation fails I evaluate the result with an IF statement. So when it fails I start over (IF => GOTO :startcopy). I know this latest version is going to be resilient, so maybe I don't need it anymore. But please return some reliable result values.
An example .CMD script ("AzCopy with SAS.cmd"):
@echo off
SETLOCAL ENABLEEXTENSIONS
REM usage AzCopy with SAS <filename>
SET _fname=%1
SET /A retries = 0

:startover

"%ProgramFiles(x86)%\Microsoft SDKs\Azure\AzCopy\AzCopy" /Source:"./" /Dest:"https://account.blob.core.windows.net/container" /DestSAS:"?sv=2018-03-28&si=..." /Pattern:%_fname% /V:"./AzCopy with SAS.log" /Y
IF %ERRORLEVEL% NEQ 0 (
  SET /A retries = retries + 1
  IF %retries% LEQ 100 (
    ECHO Trying again %retries% ...
    ECHO
    GOTO :startover
  )
)

And the I can just use "AzCopy with SAS" "file-to-upload". And I could sent it to customer so he could easily run it and make sure the file (a HUGE file) will be uploaded successfully.

Thanks for sharing! @alvipeo

I'm not in the habit of adding to close issues, but this was closed with no indication of it being resolved, and if it was resolved what is the result?

Oh sorry @pauby @ianc911, I should have posted a response (not sure what happened).
In short, the old V8 was not very fault tolerant, so it sometimes it needed users to do this sort of user-side "retry". On the other hand, V10 tries to be as fault tolerant as possible, each request is retried at most 20 times. We also added throttling control and various other knobs to make sure most cases succeed. So if something really fails, chances are it's fatal, so the user should not stick the tool in a loop.
I'm not sure whether this is a satisfactory response, please let me know your thoughts.

"I'm not sure whether this is a satisfactory response" - well, in fact you really don't mention the generation of an exit code, which is in fact the subject of this post. I did find mention of it briefly and in a kind of offhand way here:https://stackoverflow.com/questions/37746054/does-azcopy-have-an-exit-code-which-can-be-used-to-throw-error-in-a-powershell-s
Wouldn't it be nice if MS could properly document it?

@ianc911 The original issue was fixed, AzCopy follows the standard of returning 0 for success, and non-zero for failure. But I just wanted to clarify that the pattern of retrying in the script is not advised, for the reason mentioned. V10 also provides a jobs resume command to retry only the failed transfers.
I apologize again for not replying before closing, I should have said it's fixed.

Thanks for the confirm on this . Surprising that it's not officially documented, say here:
https://docs.microsoft.com/en-us/azure/storage/common/storage-use-azcopy-v10
I'll post there asking that it be added. Thanks again!


Improve Game Maker Language file detection for those generated by GM Studio 2.3

Describe the enhancement
The logic currently used for matching Game Maker Language studio .yy and .yyp files don't match the newer format used by Game Maker Studio 2.3 and aren't hiding them or marking them as JSON:
Detecting as generated:
https://github.com/github-linguist/linguist/blob/916bd8f3df802fa98b0ea85539e67bd0b88ef158/lib/linguist/generated.rb#L672-L683
Detecting them as JSON:
https://github.com/github-linguist/linguist/blob/916bd8f3df802fa98b0ea85539e67bd0b88ef158/lib/linguist/heuristics.yml#L810-L814
This results in files with this extension being identified as Yacc.
From a naïve perspective (I know nothing about GML), it appears we can fix this by looking out for "resourceType": "GM* in these two locations too.
Until this is implemented, the effect of the above can be acheived by adding the following override to repositories:
*.yy linguist-language=JSON,linguist-generated

/cc @kt-altyn @RobQuistNL

your assumptions seem to be correct, I've seen that resourceType pointer around everywhere too.

Hi, I'm having this same issue and I don't quite understand how to fix it. Could you simplify it?

Hi, I fixed it by adding a .gitattributes file with "*.yy linguist-language=Game Maker Language" in it. Still, here's a reminder that this is still an issue


Curl Method to download Aspera Share Files and folders using rest API

Can we download Aspera shares using "curl " method. Is there any API for it.

Hi,
No, IBM Aspera Shares does not provide an API or a method to download files using HTTP/S.
Transfers are exclusively with "FASP", IBM Aspera's accelerate file transfer protocol.
This is what this tool provides: ascli shares files download ...
Technically, it is possible to download with HTTPS, but that requires the use of an Aspera HTTP GW, and anyway, to call a few APIs en advance to get a transfer token, which ascli can also do.

Hi Laurent,
One of Our customer is Specifically asking for to download using CURL method  with https using Aspera HTTP GW. Please let us know for any possibilities.

This is described here:
https://github.com/IBM/aspera-cli?tab=readme-ov-file#agent-http-gateway


Made a basic contributor.jsx

I made this basic Contributors.jsx page with the help of co - pilot. Please check and review. Also I used the GitHub API. So this page will be dynamically add Contributors.
#14

Thank you for this opportunity.


leveraging configure options for sns

Hello,
This pull allows one to configure the aws_config options in the same way as the other modules do. This is useful for including the region in the hostname and other use cases.

thank you


[New game]: Valorous Rabbit

🎮 Game Request
Game logic and basic description
Valorous Rabbit is an adventurous game created using HTML, CSS, and JavaScript, where players take control of a brave rabbit on a mission to save its woodland friends from peril. Here's a description of how the game works:
HTML Structure:
The game interface is built using HTML elements. It typically includes a container div to hold the game elements, a canvas element for rendering the game graphics, score and life counters, and buttons for player interactions.
CSS Styling:
The game elements are styled using CSS to create an appealing visual design. The container div and canvas can be styled with appropriate dimensions and positioning to fit the game screen. The buttons, score, and life counters are styled to be easily readable and visually distinct.
JavaScript Functionality:
Initialization: When the game starts, JavaScript initializes the player's score and life counter to a default value and sets up event listeners to track player input.
Player Movement: JavaScript listens for keyboard inputs or button clicks to control the rabbit's movement. It updates the rabbit's position on the canvas accordingly.
Enemy Generation: JavaScript randomly generates enemies, obstacles, or challenges on the canvas to hinder the rabbit's progress. These may include predators, traps, or obstacles that the player must avoid.
Collision Detection: JavaScript detects collisions between the rabbit and enemies or obstacles. If a collision occurs, the player's life counter decreases, and appropriate feedback is provided.
Scoring: JavaScript tracks the player's score based on their progress and achievements within the game. For example, collecting power-ups or saving friends can increase the score.
Game Over: When the player's life counter reaches zero or the game objective is not achieved within a specified time limit, the game ends. The player's final score is displayed, and they may have the option to restart or quit the game.
Point down the features
Game points
The player controls the rabbit using keyboard inputs or button clicks. The objective is to navigate the rabbit through various levels or stages, overcoming obstacles and enemies, and rescuing woodland friends along the way. The player must avoid colliding with enemies or traps to prevent their life counter from depleting. Collecting power-ups or bonuses can enhance the rabbit's abilities or provide extra points. The player's goal is to achieve the highest score and complete the game with the maximum number of rescued friends.
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

/assign

Hey @uzmaansari23 👋!
@kunjgit here,
We have this annoucement to make 📢!!
We have updated our README 🚀
Make sure you follow proper guideline to raise your next issues !!
Make sure you join our Discord🕹 if you haven't 😇
Thank you 💗!

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


Improve Ed25519 Support

Ed25519 is great, and we should support it anywhere we support secp256k1. But right now, there are several places that it's possible to use secp256k1 keys and not Ed25519:

The channel_authorize method can't sign claims on payment channels using Ed25519 keys. (Ripple internal issue: RIPD-1474.)
Validators can't use Ed25519 keys as their ephemeral keys (encoded into the token)
The wallet_propose command's commandline format can't generate Ed25519 keys. I know we view wallet_propose as quasi-deprecated or likely to be deprecated soon, but I still think it would be straightforward and worthwhile to add an optional key type parameter to this command if we can.

Nodes can't really use ed25519 keys as node keys too afaik.

Incidentally, #2647 adds support for Ed25519 in validator keys and node keys. Looks like it might need some updating, though...

With #3116 merged, the channel_authorize method can now sign claims on payment channels using Ed25519 keys.


EEG - confusion between channels.tsv and electrodes.tsv

This dataset for example (https://openneuro.org/datasets/ds002791/versions/1.0.0) has different channel names in its channels.tsv and electrodes.tsv files.
sub-01/ses-naming/eeg/sub-01_ses-naming_run-1_electrodes.tsv
F10h	6.962	5.382	-2.191
F8	6.484	6.404	-0.14
AFF8h	5.699	7.208	1.791
sub-01/ses-naming/eeg/sub-01_ses-naming_run-1_channels.tsv
E1	eeg	microV	1000	n/a	n/a	n/a	n/a	n/a	n/a	n/a
E2	eeg	microV	1000	n/a	n/a	n/a	n/a	n/a	n/a	n/a
E3	eeg	microV	1000	n/a	n/a	n/a	n/a	n/a	n/a	n/a
...
The specification suggests that the name should be the same. It also does not enforce that electrodes.tsv should have the same number of rows as channels.tsv.
https://bids-specification.readthedocs.io/en/latest/04-modality-specific-files/03-electroencephalography.html
Solutions:

Clarity in the specification that electrodes.tsv should have the same number of rows as channels.tsv
Change the validator to enforce that channel names are the same between electrodes.tsv and channels.tsv

electrodes.tsv could have more electrodes than present in channels.tsv for example in the case where an EOG channel (1 channel) is recorded from 2 electrodes with bipolar referencing scheme.

The specification suggests that the name should be the same. It also does not enforce that electrodes.tsv should have the same number of rows as channels.tsv.

actually, could you please cite where you get that idea from? I read the parts of the spec again, and I cannot find that rule.
It also wouldn't make sense in my opinion, apart from the example about bipolar electrodes I listed above. --> Just thinking about a "normal" EEG recording with a reference and a ground electrode --> neither of these two electrodes are "channels" ... but they should be written down in electrodes.tsv.
Unless more information comes up, I would say that we should rather enhance the specification with one or two examles of why the names column in channels.tsv  and electrodes.tsv do NOT have to be exactly the same (in fact, they cannot be the same sometimes)
We hint at that in the intro:

Note that for proper documentation of EEG recording metadata it is important to understand the difference between electrode and channel: An EEG electrode is attached to the skin, whereas a channel is the combination of the analog differential amplifier and analog-to-digital converter that result in a potential (voltage) difference that is stored in the EEG dataset. We employ the following short definitions:

Electrode = A single point of contact between the acquisition system and the recording site (for example, scalp, neural tissue, ...). Multiple electrodes can be organized as caps (for EEG), arrays, grids, leads, strips, probes, shafts, and so on.
Channel = A single analog-to-digital converter in the recording system that regularly samples the value of a transducer, which results in the signal being represented as a time series in the digitized data. This can be connected to two electrodes (to measure the potential difference between them), a magnetic field or magnetic gradient sensor, temperature sensor, accelerometer, and so on.

we could suggest to use the the same names for electrodes and channels

On the topic of adding some examples where it is more difficult to match the positions (in addition to bipolar recordings):
Example 1: a dataset in which an ECG channel is recorded simultaneously with SEEG. The ECG electrode is not present or is n/a in _electrodes.tsv
Example 2: EEG and SEEG are simultaneously recorded and the EEG positions may not be present or are in a different space than the SEEG positions.
However, if a name in _channels.tsv is not present in _electrodes.tsv that maybe should raise a flag?


update hyper and axum

Significant breakage, since hyper 1.0 introduced significant breakage.
Of note:

No more Accept trait, and no more hyper-provided server. Examples are all modeled after the axum serve-with-hyper example.
Conn now implements the hyper read/write traits, because why not?
Our proxy wrapper is ancient and unmaintained, and still uses old hyper :cry:

All in all, not super happy about this change. Hyper used to provide a nice pluggable interface where you could bring-your-own listener type, and that's been entirely done away with. Axum didn't fill in the gap very well, and their serve function explicitly takes a TCP listener. Not sure we want to provide yet another accept-serve loop.
Since hyper is 1.0 and stable though, I feel better about providing implementations of the traits that they do offer. Maybe the rest of the ecosystem will catch up and figure out how to not depend on TcpListener directly.
Resolves #126
Resolves #136

All in all, not super happy about this change. Hyper used to provide a nice pluggable interface where you could bring-your-own listener type, and that's been entirely done away with.

Hopefully, as hyper-utils matures they develop a new pluggable interface.

When is this PR expected to be integrated?

@jrobsonchase @bobzilladev Any chance this will get merge any time soon? I could really use this, since my project is dependent on Hyper 1.0+. Let me know if there is anything I can do to speed this up.


Website is down

Hope you will take him up )

In the meantime, we can use https://web.archive.org/web/20200218041709/http://gaminganywhere.org/

It is back online. You can close this issue

Is it down again?

Hey @keichi not for me. I am using the website. Refresh the webpage and clear the cache.
Ctrl+F5 does the trick

Thanks @chintan-mishra, I can access it now. I thought I refreshed the cash but maybe I didn't.


docs: readme update and plugins

What type of PR is this?
/kind documentation
Any specific area of the project related to this PR?
What this PR does / why we need it:
Adding plugin references to the documentation plus some minor improvements.
Which issue(s) this PR fixes:
Special notes for your reviewer:
I haven't added a link to the plugin-sdk-cpp since it does not yet seem to be mature enough.
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.32.0


Tables: Why is td padding-left only 10px?

Here's the CSS for <td>
td {
  empty-cells: show;
  border-bottom: ((@border-width / @body-font-size) + 0em) @border-style @border-color; /* Current color = color of text (inverted in night mode) */
  @computed-padding: (((@body-font-size * @body-line-height) - (@border-width * 2)) / @body-font-size);
  padding: (@base-margin / 2) 10px ((@computed-padding / 2) + 0em) 0;
  text-align: left;
}

Most of it is computed, but the padding-right value is fixed at 10px.  Why is this?
For example, take a look at this screenshot from Kindle Previewer of a two-column table.  It's extremely cramped, and it's hard to tell where one column begins and another column ends.

That’s the value mapping to the former 2% on smaller screens – % was an issue as it relied on a Reading-System-specific relayout.
Note we don’t officially support KFX (so Kindle Previewer 3) and decided to stop @ KF8 (so up to Kindle Previewer 2.x) given I had to retro-engineer their conversion scripts every time they update, which was unsustainable.

@ptmkenny Maybe try with pt instead of px to see if that improves this result.
I can recall they changed how DPI works in their fork of Webkit – with a comment saying that they don’t respect the W3C standard any longer – so pt units might fix this issue, in which case I could change the default unit.

@JayPanoz Thanks for the suggestion; unfortunately, pt gave me the same result.
For my case, I set the padding-left to @body-font-size + 0px.  For Japanese, this gives a clean one-character indent, but it's probably a good way to calculate padding for English.


metrics: compaction

ref https://github.com/pingcap/tikv/issues/1656
@siddontang @andelf PTAL

PTAL @BusyJay @hhkbp2

LGTM

CI failed
please make format before creating a pr!!!


[1.x] Fix usage documentation

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (0a13765) to head (90bbc8a).
Report is 1 commits behind head on 1.x.

Additional details and impacted files
@@            Coverage Diff            @@
##               1.x       #91   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           59        59
  Lines         2452      2452
=========================================
  Hits          2452      2452

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


npm audit identifies critical vulnerabilities

Whilst investigating https://github.com/RuntimeTools/appmetrics/issues/633, I identified this package as requiring a level of https-proxy-agent with a known Man-In-The-Middle attack. Running npm audit found a total of 57 vulnerabilities (2 low, 1 moderate, 52 high, 2 critical). Please can these vulnerabilities be fixed?

fixed with commit 7200d87bedc69596f35b6d1604046ae7a49fffb9

There are still 57 vulnerabilities (2 low, 1 moderate, 52 high, 2 critical). Please run npm install followed by npm audit to see them all.

[root@joycevm1 buildgreenfield]# npm audit

                       === npm audit security report ===

found 0 vulnerabilities
 in 631 scanned packages

[root@joycevm1 test]# npm install ibmapm-restclient
+<EMAIL_ADDRESS>updated 1 package and audited 826 packages in 2.451s
found 0 vulnerabilities


:technologist: Replace content endpoint with batch endpoint

Changes

Replace content endpoint to support batch operation by default.

I think L13 is still accurate since this endpoint is designed for detectors that work on user prompt or generated text. Its just that instead of 1 document, now the API accepts multiple documents.


add purifier

please

without having this device I will not be able to add it to this solution. I will accept any PRs related to it.


Clear listeners, stop listening

I use this plugin to help with a CI interface I'm building.  When my configurations change I need a way to clear out the listeners so I can re-attach them with the update configurations, then I also would like a way to stop the service altogether.

clearing event listeners is already possible, seeing as this module simply extends a standard node event emitter. just call githubhook.removeAllListeners('eventname') to remove the listeners for a given event.
i'll add a stop function to the server though.


Need enhancement  for configure Fabric L2 MTU / Port MTU size

Hello,
When testing netascode/nac-aci/aci v0.8.0 with below yaml file as input on APIC v4.2, simulator, default Fabric L2 MTU Policy / Port MTU size(bytes) will be updated between 9000 and 9216 every time after running terraform apply with the same input yaml file. That's because below yaml input will be applied by two modules -- terraform-aci-fabric-l2-mtu and terraform-aci-l2-mtu-policy and the two modules are all applied on default Fabric L2 MTU Policy / Port MTU size(bytes), but with different values.
apic:
  fabric_policies:
    l2_port_mtu: 9000
    l2_mtu_policies:
      - name: default
        port_mtu_size: 9216

If module aci_l2_mtu_policy is designed for customized L2 MTU Policy, how about excluding default as showed below?
Line 150 https://github.com/netascode/terraform-aci-nac-aci/blob/eda11599284526188037bfaecb6db17beeec7eca/aci_fabric_policies.tf
from
for_each      = { for policy in try(local.fabric_policies.l2_mtu_policies, []) : policy.name => policy if local.modules.aci_l2_mtu_policy && var.manage_fabric_policies }
to
for_each      = { for policy in try(local.fabric_policies.l2_mtu_policies, []) : policy.name => policy if local.modules.aci_l2_mtu_policy && var.manage_fabric_policies && policy.name != "default" }

I understand that this can be confusing, but I am not convinced it would be less confusing if we would silently ignore explicit configuration of the default policy using l2_mtu_policies. If one adheres to the general best practice of not modifying default policies it should not be an issue. We could add a semantic pre-change validation rule (iac-validate) to flag such an overlapping configuration.

Hi Daniel, thank you very much for your quick response and clarification. It's a good suggestion to add a semantic pre-change validation rule(iac-validate).  I will try to add this validation rule.
Module terraform-aci-fabric-l2-mtu only can be used to configure the default Fabric L2 MTU, whereas terraform-aci-l2-mtu-policy module can be used to configure both default and customized Fabric L2 MTU. The examples on https://netascode.cisco.com/data_model/apic/fabric_policies/l2_mtu is perfect. I don't know if we could add more description for module terraform-aci-fabric-l2-mtu.

Either l2_port_mtu in the schema and the terraform-aci-fabric-l2-mtu module could be deprecated or the terraform-aci-l2-mtu-policy module could have a validation rule on the name variable to disallow a value of default.

Hi Andrew, adding a validation rule on the name variable to disallow a value of default to terraform-aci-l2-mtu-policy module sounds great to me. thank you for the suggestions.


Revert "Revert "Add registry mirror parameter""

This reverts commit 9bbbb5a39c8d17a56dc790801add25c46b96a059.
Change-type: patch

@balena-ci I self-certify!


Authorize.net addon install fail with Composer

As indicated in the summary at the bottom of the addon page, I went ahead and installed Composer on my Mac MAMP install of C5 in the community_store root directory. It downloaded and installed the composer.phar file successfully. Then I downloaded and attempted the install of the Authorize.net addon and received an error:
Concrete\Package\CommunityStoreAuthorizeNet\Controller::on_start(): Failed opening required 'vendor/autoload.php'
I read up on this file from the Composer site and I understand what it does, but since the install failed there is no such file and now my site won't load except to give me that error. Suggestions? I couldn't find a copy of the file to download from the Composer site, is it unique to each install?

This sounds like you've missed the step where you run composer against the add-on folder.
It's not that composer is a prerequisite for install or running it, it's simply a tool used to help download the libraries within the package (and to keep them up to date while developing).
So for reference, you'd need to also run
composer install

as a command/terminal command against the package folder. When you do this, you'll see composer download some resources and you'll end up with these in a vendor folder (which then includes the autoload.php file)
However, you can skip all of that and simply download a 'release' of the add-on from https://github.com/concrete5-community-store/community_store_authorize_net/releases
This already has the libraries installed and ready to go, no need to use composer then.


Table showing features / limits / comparison for Power BI Embedded vs Power BI Pro

Would love to see a basic table that shows the differences in features and limitations between PBI Embedded and PBI Pro. For example: Paginated Reports cannot use Power BI Datasets as a data source in PBI Embedded (works with PBI Pro). A lot of the good information in this FAQ could also be summarized in tabular form.

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

ID: e37c7960-448a-0f10-c84a-f9c531452320
Version Independent ID: 4867daee-c097-cde7-81c9-4f2a44a4f5b3
Content: Frequently asked questions about Power BI Embedded - Power BI
Content Source: powerbi-docs/developer/embedded/embedded-faq.md
Service: powerbi
Sub-service: powerbi-developer
GitHub Login: @KesemSharabi
Microsoft Alias: kesharab

Hi @sethsanu,
Power BI Pro is a user license and Power BI Embedded is a Premium offer solution. Pro doesn't require a capacity, while Embedded requires a capacity. Therefore comparing them doesn't make sense.
You can see a comparison between Power BI Pro and Power BI Premium if you scroll down our pricing page.
Hope this helps,
Kesem


Commit template editor is way too small

When I was working on #719, I used commit template editor for a while, and it's actually awkward. I can only see 3.5 lines at a time, and it is really hard to navigate in such a tiny space. For better user experience I suggest expanding it at least a bit


here, have some base images for older debian versions

I used debootstrap to generate base images for several older debian versions:
https://registry.hub.docker.com/u/mcandre/docker-debian/tags/manage/
Could we tag these into the tianon repository on Docker Hub?

I'm happy to see these available to folks, but since they're not officially supported by upstream, I don't think it'd be appropriate to offer them as official tags.  That being said, I think this issue is a decent pointer for people who are searching for such a thing to where they can find them.  Thanks! :+1:


tests: Anchor integration test and test suites in Kotlin

closes #101

PR Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[x] This PR's title starts with name of package that is most changed in the PR, ex.
paymentservice.stellar, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.

What
[TODO: Short statement about what is changing.]
Why
[TODO: Why this change is being made. Include any context required to understand the why.]
Known limitations
[TODO or N/A]

/azp run


there is no binary when builds end

run   "cargo build --release" , but finnally  there is no binary:
"ls target/release/
build  deps  examples  incremental  libombrac_client.d	libombrac_client.rlib  libombrac_protocol.d  libombrac_protocol.rlib  libombrac_server.d  libombrac_server.rlib"

And build fail in nomarl Ubuntu system (20.04.6)
   Compiling parking_lot_core v0.9.10
   Compiling lock_api v0.4.12
The following warnings were emitted during compilation:

warning<EMAIL_ADDRESS>Building with: CMake
warning<EMAIL_ADDRESS>Symbol Prefix: Some("aws_lc_0_22_0")
warning<EMAIL_ADDRESS>CMAKE environment variable set: cmake
warning<EMAIL_ADDRESS>Generating bindings - internal bindgen. Platform: x86_64-unknown-linux-gnu

error: failed to run custom build command for `aws-lc-sys v0.22.0`

Caused by:
  process didn't exit successfully: `/root/ombrac/target/release/build/aws-lc-sys-d309bfa6b0a1eb80/build-script-main` (exit status: 101)
  --- stdout
  cargo:rerun-if-env-changed=AWS_LC_SYS_NO_PREFIX
  cargo:rerun-if-env-changed=AWS_LC_SYS_PREGENERATING_BINDINGS
  cargo:rerun-if-env-changed=AWS_LC_SYS_EXTERNAL_BINDGEN
  cargo:rerun-if-env-changed=AWS_LC_SYS_NO_ASM
  cargo:rerun-if-env-changed=AWS_LC_SYS_CFLAGS
  cargo:rerun-if-env-changed=AWS_LC_SYS_PREBUILT_NASM
  cargo:rerun-if-env-changed=AWS_LC_SYS_C_STD
  cargo:rerun-if-env-changed=AWS_LC_SYS_CMAKE_BUILDER
  cargo:rerun-if-env-changed=AWS_LC_SYS_STATIC
  cargo:rerun-if-env-changed=CMAKE
  cargo:warning=Building with: CMake
  cargo:warning=Symbol Prefix: Some("aws_lc_0_22_0")
  cargo:rerun-if-env-changed=CMAKE
  cargo:warning=CMAKE environment variable set: cmake
  cargo:warning=Generating bindings - internal bindgen. Platform: x86_64-unknown-linux-gnu
  cargo:rerun-if-env-changed=AWS_LC_SYS_INCLUDES

  --- stderr
  /usr/include/stdlib.h:97:8: error: unknown type name 'size_t'
  /usr/include/stdlib.h:411:4: error: unknown type name 'size_t'
  /usr/include/stdlib.h:441:4: error: unknown type name 'size_t'
  /usr/include/stdlib.h:539:22: error: unknown type name 'size_t'
  /usr/include/stdlib.h:542:22: error: unknown type name 'size_t'
  /usr/include/stdlib.h:542:38: error: unknown type name 'size_t'
  /usr/include/stdlib.h:550:36: error: unknown type name 'size_t'
  /usr/include/stdlib.h:559:41: error: unknown type name 'size_t'
  /usr/include/stdlib.h:559:57: error: unknown type name 'size_t'
  /usr/include/alloca.h:32:22: error: unknown type name 'size_t'
  /usr/include/stdlib.h:574:22: error: unknown type name 'size_t'
  /usr/include/stdlib.h:580:45: error: unknown type name 'size_t'
  /usr/include/stdlib.h:580:65: error: unknown type name 'size_t'
  /usr/include/stdlib.h:586:29: error: unknown type name 'size_t'
  /usr/include/stdlib.h:586:49: error: unknown type name 'size_t'
  /usr/include/stdlib.h:821:9: error: unknown type name 'size_t'
  /usr/include/stdlib.h:821:25: error: unknown type name 'size_t'
  /usr/include/stdlib.h:830:34: error: unknown type name 'size_t'
  /usr/include/stdlib.h:830:50: error: unknown type name 'size_t'
  fatal error: too many errors emitted, stopping now [-ferror-limit=]
  thread 'main' panicked at /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/aws-lc-sys-0.22.0/builder/bindgen.rs:106:10:
  Unable to generate bindings.: ClangDiagnostic("/usr/include/stdlib.h:97:8: error: unknown type name 'size_t'\n/usr/include/stdlib.h:411:4: error: unknown type name 'size_t'\n/usr/include/stdlib.h:441:4: error: unknown type name 'size_t'\n/usr/include/stdlib.h:539:22: error: unknown type name 'size_t'\n/usr/include/stdlib.h:542:22: error: unknown type name 'size_t'\n/usr/include/stdlib.h:542:38: error: unknown type name 'size_t'\n/usr/include/stdlib.h:550:36: error: unknown type name 'size_t'\n/usr/include/stdlib.h:559:41: error: unknown type name 'size_t'\n/usr/include/stdlib.h:559:57: error: unknown type name 'size_t'\n/usr/include/alloca.h:32:22: error: unknown type name 'size_t'\n/usr/include/stdlib.h:574:22: error: unknown type name 'size_t'\n/usr/include/stdlib.h:580:45: error: unknown type name 'size_t'\n/usr/include/stdlib.h:580:65: error: unknown type name 'size_t'\n/usr/include/stdlib.h:586:29: error: unknown type name 'size_t'\n/usr/include/stdlib.h:586:49: error: unknown type name 'size_t'\n/usr/include/stdlib.h:821:9: error: unknown type name 'size_t'\n/usr/include/stdlib.h:821:25: error: unknown type name 'size_t'\n/usr/include/stdlib.h:830:34: error: unknown type name 'size_t'\n/usr/include/stdlib.h:830:50: error: unknown type name 'size_t'\nfatal error: too many errors emitted, stopping now [-ferror-limit=]\n")
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
root@yfn-hk-test:~/ombrac# cat /etc/issue
Ubuntu 20.04.6 LTS \n \l

Under the current workspace, the client and server are set to use the default library.

build the client binary, use cargo build --release --bin ombrac-client --features binary
build the server binary, use cargo build --release --bin ombrac-server --features binary

For more details, you can refer to the Cargo.toml files in each crate.
By the way, you can check the Cross.toml to see what dependencies s2n-tls requires.

./ombrac-server -l <IP_ADDRESS>:4431 --tls-cert /home/bhzhu/.acme.sh/xxxxxx.com_ecc/fullchain.cer --tls-key /home/bhzhu/.acme.sh/xxxxxx.com_ecc/xxxxxx.com.key &

./ombrac-client  --socks <IP_ADDRESS>:1089 --server-address xxxxxx.com:4431
2024-11-14T05:13:56.050120Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:49161) failed to establish connection with xxxxxxxx:4431. The connection was closed because the handshake took longer than the max handshake duration of 3s
2024-11-14T05:13:59.050351Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:49161) failed to establish connection with xxxxxxxx:4431. The connection was closed because the handshake took longer than the max handshake duration of 3s
2024-11-14T05:14:02.051617Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:49161) failed to establish connection with xxxxxxxx:4431. The connection was closed because the handshake took longer than the max handshake duration of 3s
2024-11-14T05:14:05.051714Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:49161) failed to establish connection with xxxxxxxx:4431. The connection was closed because the handshake took longer than the max handshake duration of 3s
2024-11-14T05:14:08.051712Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:49161) failed to establish connection with xxxxxxxx:4431. The connection was closed because the handshake took longer than the max handshake duration of 3s

I can build the binary now . But the client can not connect to the server ,even in the local machine.
openssl req -x509 -newkey rsa:4096 -keyout server.key -out server.crt -days 365 -sha256 -nodes -subj '/US=localhost/'

../ombrac-server --listen <IP_ADDRESS>:5678 --tls-cert ./server.crt  --tls-key server.key

../ombrac-client --server-address <IP_ADDRESS>:5678
2024-11-14T05:19:26.343315Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:35661) failed to establish connection with <IP_ADDRESS>:5678. The connection was closed on the transport level with error HANDSHAKE_FAILURE by the local endpoint
2024-11-14T05:19:26.516484Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:35661) failed to establish connection with <IP_ADDRESS>:5678. The connection was closed on the transport level with error HANDSHAKE_FAILURE by the local endpoint

../ombrac-client --server-address <IP_ADDRESS>:5678
2024-11-14T05:22:53.061976Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:59406) failed to establish connection with <IP_ADDRESS>:5678. The connection was closed on the transport level with error HANDSHAKE_FAILURE by the local endpoint
2024-11-14T05:22:53.231776Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:59406) failed to establish connection with <IP_ADDRESS>:5678. The connection was closed on the transport level with error HANDSHAKE_FAILURE by the local endpoint
2024-11-14T05:22:53.406988Z ERROR ThreadId(02) ombrac_client::transport::quic::connection::impl_s2n_quic: Ok(<IP_ADDRESS>:59406) failed to establish connection with <IP_ADDRESS>:5678. The connection was closed on the transport level with error HANDSHAKE_FAILURE by the local endpoint

This is usually a certificate issue. Please use this command when debugging with a self-signed certificate locally
openssl req -x509 -out localhost.crt -keyout localhost.key \
  -newkey rsa:2048 -nodes -sha256 \
  -subj '/CN=localhost' -extensions EXT -config <( \
   printf "[dn]\nCN=localhost\n[req]\ndistinguished_name = dn\n[EXT]\nsubjectAltName=DNS:localhost\nkeyUsage=digitalSignature\nextendedKeyUsage=serverAuth")

Server
./ombrac-server -l <IP_ADDRESS>:5678 --tls-cert localhost.crt  --tls-key localhost.key

Client
When using a self-signed certificate, the server name must be specified to indicate SNI.
./ombrac-client --server-address <IP_ADDRESS>:5678 --server-name localhost  --tls-cert localhost.crt --socks <IP_ADDRESS>:1081 --tracing-level INFO

This is worked in my local environment

This is usually a certificate issue. Please use this command when debugging with a self-signed certificate locally
openssl req -x509 -out localhost.crt -keyout localhost.key \
  -newkey rsa:2048 -nodes -sha256 \
  -subj '/CN=localhost' -extensions EXT -config <( \
   printf "[dn]\nCN=localhost\n[req]\ndistinguished_name = dn\n[EXT]\nsubjectAltName=DNS:localhost\nkeyUsage=digitalSignature\nextendedKeyUsage=serverAuth")

Server
./ombrac-server -l <IP_ADDRESS>:5678 --tls-cert localhost.crt  --tls-key localhost.key

Client
When using a self-signed certificate, the server name must be specified to indicate SNI.
./ombrac-client --server-address <IP_ADDRESS>:5678 --server-name localhost  --tls-cert localhost.crt --socks <IP_ADDRESS>:1081 --tracing-level INFO

This is worked in my local environment

Ok now , It must be set the "--server-name " to make it working

ur right; the README should be updated to inform others.
However, at this stage, we will still spend more time optimizing performance and anti-censorship.


[PE-D][Tester C] addEvent event name format is inconsistent

On the user guide, the format for event name is n/ but on the application the format for event name is e/.

Labels: type.DocumentationBug severity.Medium
original: ongwenpin/ped#4

Duplicate issue #160


Upgrading path lib in requirements

Remove the path.py from the base.in file.
Added path in the base.in file
Run make upgrade

Coverage increased (+0.2%) to 91.105% when pulling bb043836e07f39696f2c2520ae2f7c57cb1d99d2 on aj/update-deps into 58072c02e05318ab680bcb84f0f32197f29c62ed on master.

@crice100 Thanks, Updated the PR with the version bump.
Do I need to draft a new release on GitHub as well after the merge?


Bug: WebAuthn Error in Browser mallformed

I was trying to catch a specific error from the browser lib and I noticed 2 problems

the FirebaseWebAuthnError does not have a property name set to FirebaseWebAuthnError but just Error, I was thinking that subclasses of Error automatically reflect the name of the constructor/type, mislead by the MDN's

Subclasses like TypeError and SyntaxError provide their own name properties.
-- https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error/name

but unfortunately, it is not the case and it is not set automatically. The FirebaseError sets the name explicitly as a readonly member property. This is not a bug, but I'm writing my app in ReScript and instanceof checks are little bit out of my reach so I was relied on the name,

the second problem is a bug IMHO, the code property contains double-prepended firebaseWebAuthn/firebaseWebAuthn, I believe there should be just one firebaseWebAuthn/, the private firebaseWebAuthnErrorOptions contains the correct form, so it is prepended in the options already and then prepended again at this line.

This feels like a good first issue (for me) 😉

Thank you! I had no idea about the name property. Fix incoming on both


Update github tag action

Signed-off-by: Katrina Rogan<EMAIL_ADDRESS>TL;DR
Signed-off-by: Katrina Rogan<EMAIL_ADDRESS>matches https://github.com/flyteorg/flytetools/pull/62
see https://github.com/anothrNick/github-tag-action/issues/147
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
Remove the 'fixes' keyword if there will be multiple PRs to fix the linked issue
fixes https://github.com/flyteorg/flyte/issues/
Follow-up issue
NA
OR
https://github.com/flyteorg/flyte/issues/

Did something change in this PR with the logging?
We see a lot of unstructured gorm logs in flyteadmin.
example

{"data":{"src":"handlers.go:237"},"message":"Running authentication gRPC interceptor","severity":"DEBUG","timestamp":"2022-05-02T07:56:12Z"}
{"data":{"src":"handlers.go:193"},"message":"gRPC server info in logging interceptor []method [/flyteidl.service.AdminService/ListExecutions]\n","severity":"DEBUG","timestamp":"2022-05-02T07:56:12Z"}
...
2022/05/02 07:56:12 /go/src/github.com/flyteorg/flyteadmin/pkg/repositories/gormimpl/execution_repo.go:103
[18.420ms] [rows:0] SELECT * FROM "executions" WHERE executions.execution_project = 'some-project' AND executions.execution_domain = 'staging' AND executions.phase in ('RUNNING') AND executions.started_at >= '2022-04-08T07:56:12' AND executions.started_at <= '2022-05-02T07:46:12' AND executions.state = 'EXECUTION_ACTIVE' LIMIT 100

resolving in slack.


v2ray: 4.39.2 -> 4.41.0

Motivation for this change
4.39.2 -> 4.41.0
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 128244 run on x86_64-linux 1

1 package built:

v2ray


chore: refactor project structure

closes #143

there's a bug here, some parse errors are now lex errors, that doesn't look right.

i believe that ast should be moved within parser, it doesn't make sense to have ast in root, while token is in lexer.


Extra host lookup triggered on decorate

Would it be possible to add a property flag around adding the IP address and port?
Collecting that information isn't useful for us and triggers an extra (undesirable) host lookup on calls.
Caused by: java.net.UnknownHostException: REDACTED 	at java.net.InetAddress.getAllByName0(InetAddress.java:1281) ~[?:1.8.0_232] 	at java.net.InetAddress.getAllByName(InetAddress.java:1193) ~[?:1.8.0_232] 	at java.net.InetAddress.getAllByName(InetAddress.java:1127) ~[?:1.8.0_232] 	at java.net.InetAddress.getByName(InetAddress.java:1077) ~[?:1.8.0_232] 	at com.mongodb.ServerAddress.getSocketAddress(ServerAddress.java:188) ~[mongo-java-driver-3.11.0.jar:?]
Looking at https://github.com/opentracing-contrib/java-mongo-driver/blob/release-0.1.5/opentracing-mongo-common/src/main/java/io/opentracing/contrib/mongo/common/SpanDecorator.java#L53 it seems like L53-63 could be put in an if-statement

@alawrenc you should be able to provide own SpanDecorator implementation.
flag also is fine. Would you like to submit a PR?


Not working with SQL Server

Laravel Cross Database Subqueries Version: 8.0
PHP Version: 7.4
Database Driver & Version: Sql Server Native Client 11.0

Description:
Trying to perform a cross-database whereHas
The generated SQL looks like this:
(SQL: select top 1 * from [Clients] where exists (select * from [TMACentral-Demo].[Cli_Core] where [Clients].[CLIENTNUM] = [Cli_Core].[SYSID])
When it should be [TMACentral-Demo].[dbo].[Cli_Core]

In QueriesRelationships if I add a ".dbo" like this
$queryFrom = $hasQuery->getConnection()->getTablePrefix().'<-->'.$hasQuery->getQuery()->from.'<-->'.$subqueryConnection."dbo";
then it works but still investigating what I would need to do to make it actually work dynamically so I can do a pull request


[feature] Json schema support

Can ~/.config/asm-lsp/asm-lsp.toml have a json schema and been published in schemastore? Like alacritty?

I wasn't aware of the schemastore until now but this looks really cool! I'll have to look writing/publishing the schema a bit more when I have some free time but on first pass this looks pretty doable :)

This is a reference: https://github.com/cmhughes/latexindent.pl/issues/206
The steps are following:

Create a json schema
Add its URL to schemastore.


Revert "Add support for reference_type"

Reverts nexB/vulnerablecode#1502

@ziadhany I am reverting #1502 , please check it once and re-open it tests are failing

@TG1999 the issue was in the cargo test and I fixed on EPSS #1481

Gotcha! can you please open the PR again for #1502


[Feature Request]: How to modify the KGs when there miss some important relations?

Is there an existing issue for the same feature request?

[X] I have checked the existing issues.

Is your feature request related to a problem?
When I use graphrag to parse some excel files, then checked the KGs, some look likes good but also have some miss important relations between entities.

Describe the feature you'd like
Can you add support for modifying the graph directly? Alternatively, it supports importing and exporting graph files, such as parquet, which can be modified using neo4j and then imported for use.
Describe implementation you've considered
No response
Documentation, adoption, use case
No response
Additional information
No response

mark


SMS -Backup+ unable to log into my email to backup SMS messages

I have been unable to back up my messages for months, say incorrect login or password.  I have changed my email password and verified my IMAP settings, still not working.  any suggestions?  I have an Android phone S20

There's a number of solutions posted in the github thread. I use an app generated password via Google for the app, and I use Tasker to complete the auto backup.


[Bug]: js_run_devserver() sandbox is broken when used with js_library()

What happened?
When used together with js_library(), the sandbox created by js_run_devserver() includes symlinks which point out of the sandbox.
In a small monorepo with two packages, //apps/foo-app and //pkgs/foo-lib, the symlink in _main/apps/foo-app/node_modules/@monorepo/foo-lib that makes @monorepo/foo-lib visible to @monorepo/foo-app points outside of the js_run_devserver sandbox. So even though //pkgs/foo-lib files are copied into the sandbox, they aren't what the bundler sees.
Version
Development (host) and target OS/architectures: MacOS
Output of bazel --version: bazel 7.2.1
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:

bazel_dep(name = "aspect_rules_js", version = "2.0.0-rc6")
bazel_dep(name = "rules_nodejs", version = "6.2.0")
bazel_dep(name = "aspect_rules_ts", version = "3.0.0-rc0")

Language(s) and/or frameworks involved: TypeScript, Vite
How to reproduce
See https://github.com/walkerburgin/js-run-devserver-repro
Any other information?
No response

Fixed by https://github.com/aspect-build/rules_js/pull/1872


Tags from bitbucket are not imported

Hi,
it seems like tags from bitbucket are not auto-imported? 3.3-beta2 is missing.

Hi,
Refined script, thanks!


Array custom field allows Items to be added but not removed.

I have created a custom field that allows the user to edit an array. Initially this is a text box and
insertTemplate converts the array to a string using .join(v)
the insertValue and editValue functions parse the text back into an array using v.split(",").
When I add an element to the textbox it is stored correctly.
When I try to remove an element from the textbox the array is not updated.
I have created a jsfiddle that demonstrates this.  You can edit the Colour field to add an element but you cannot remove one.

I have tracked down the problem and it is in jsgrid.js line 1292
The _updateRow function is taking the original object and the edited object and merging them together using $.extend.  For normal fields this works fine, the old values are replaced with the new values.  For an array however it merges the two arrays together - causing the reported behavious where you can add items to the array but you cannot remove them.
My temporary fix is to remove any array elements from the original object before calling $.extend:
`
_updateRow: function($updatingRow, editedItem) {
var updatingItem = $updatingRow.data(JSGRID_ROW_DATA_KEY);
        // need to remove any arrays from updatingItem
        // $.extend will merge arrays rather than replace them
        // effectively meaning you can't remove items from a property
        // that is an array.
        for (prop in updatingItem) {
            if (Array.isArray(updatingItem[prop])) {
                updatingItem[prop] = [];
            }
        }
        var updatingItemIndex = this._itemIndex(updatingItem);
        var updatedItem = $.extend(true, {}, updatingItem, editedItem);

`

I would appreciate it if you could confirm that this fix is valid and doesn't have any consequences that I might have missed.
Thanks

Yes, that's a tricky case. Actually, if we would do non-deep coping everything works fine. But we need it to support nested fields.
Your hotfix looks valid. While I don't see a proper fix for the moment w/o a breaking change.

Same problem here, I have fixed this by set an id on items and change the value of arrays in updateItem function. Any other ideas?

I just got bit by this too. In my case this behavior of extend causes an inability to delete elements from an array. Would it be possible to add something like this...
var updatedItem = this._mergeObjects(updatingItem, editedItem);
...
_mergeObjects: function(ob1, ob2) {
    return $.extend(true, {}, ob1, ob2);
}

...? Then we API users can replace _mergeObjects with our own code. (As it is, I'm going to have to replace _updateRow.)


[Bug Report] Read timeout (always) on select outE() - Reproducible

OrientDB Version: 3.0.19
Java Version: 1.8.
OS: Alpine Linux 3.9.4 + Ubuntu 19.04 (checked twice, all other configurations are the same)
Expected behavior
Return edges
Actual behavior
Hangs and after a while read timeout
Steps to reproduce
create class person extends V
create property person.name String

create class friend extends E
create property friend.status String

create vertex person set name="Bob1"
create vertex person set name="Bob2"
create vertex person set name="Bob3"
create vertex person set name="Bob4"

create edge friend from (select from person where name="Bob1") to (select from person where name="Bob2") set status="Good"
create edge friend from (select from person where name="Bob2") to (select from person where name="Bob3") set status="Very Good"
create edge friend from (select from person where name="Bob3") to (select from person where name="Bob4") set status="Bad"

orientdb {db=my_db}> select outE() from person

Extra query info
orientdb {db=my_db}> select from person

+----+-----+------+----+----------+---------+
|#   |@RID |@CLASS|name|out_friend|in_friend|
+----+-----+------+----+----------+---------+
|0   |#24:0|person|Bob1|[#25:0]   |         |
|1   |#24:1|person|Bob2|[#25:1]   |[#25:0]  |
|2   |#24:2|person|Bob3|[#25:2]   |[#25:1]  |
|3   |#24:3|person|Bob4|          |[#25:2]  |
+----+-----+------+----+----------+---------+

4 item(s) found. Query executed in 0.014 sec(s).

orientdb {db=my_db}> select from friend

+----+-----+------+-----+-----+---------+
|#   |@RID |@CLASS|out  |in   |status   |
+----+-----+------+-----+-----+---------+
|0   |#25:0|friend|#24:0|#24:1|Good     |
|1   |#25:1|friend|#24:1|#24:2|Very Good|
|2   |#25:2|friend|#24:2|#24:3|Bad      |
+----+-----+------+-----+-----+---------+

Other technical info

The log had no error either
when using list connections in another console, it just shows "Listening" for command.

p.s.
I was trying to do some other queries and it seems they are also causing read timeout.
e.g.
select unionall(tags, [2,3]) from place

where place is a vertex that has 3 records in it, tags is a property containing an embeddedlist, each record contains different values.
Actual Bug
Did some debugging and found the stack trace (which is silented). I tried to see if there were any recent changes around the code but couldn't find any.
java.lang.ClassCastException: com.orientechnologies.common.collection.OMultiCollectionIterator cannot be cast to java.util.Collection
	at com.orientechnologies.orient.core.serialization.serializer.result.binary.OResultSerializerNetwork.serializeValue(OResultSerializerNetwork.java:401)
	at com.orientechnologies.orient.core.serialization.serializer.result.binary.OResultSerializerNetwork.serialize(OResultSerializerNetwork.java:132)
	at com.orientechnologies.orient.core.serialization.serializer.result.binary.OResultSerializerNetwork.toStream(OResultSerializerNetwork.java:651)
	at com.orientechnologies.orient.client.remote.message.OMessageHelper.writeProjection(OMessageHelper.java:411)
	at com.orientechnologies.orient.client.remote.message.OMessageHelper.writeResult(OMessageHelper.java:448)
	at com.orientechnologies.orient.client.remote.message.OQueryResponse.write(OQueryResponse.java:56)
	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.sessionRequest(ONetworkProtocolBinary.java:340)
	at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.execute(ONetworkProtocolBinary.java:205)
	at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:69)

Also, the reason why it's not shown normally is:
https://github.com/orientechnologies/orientdb/blob/develop/core/src/main/java/com/orientechnologies/common/thread/OSoftThread.java#L72
        if (dumpExceptions)
          OLogManager.instance().error(this, "Error during thread execution", e);

dumpExceptions is always false (due to the constructor of ONetworkProtocol)

Hi @matanshukry
I just pushed a fix for this problem, it will be released with v 3.0.20
I'm keeping this issue open just to remember that I have to add some additional checks on result set composition
Thanks
Luigi

Great, I'll work with it locally using develop branch. Thanks!

Hi,
The iterator problem is fixed, and the network blocked as well has been fixed in recent versions.
Closing


Add sbom-destination input variable with .sbom.json file extension

Add support for the final sbom export destination, by replacing the "tar.gz" suffix with ".sbom.json"

/hold

/uncc dorileo
/uncc KarnVadaliya

/lgtm

/unhold


[FEATURE] enable output_margin, pred_contribs in sklearn API

We maintain a slightly adjusted version of xgboost for our own purposes, but one thing that is common are some trivial features that are not exposed.  I've mentioned this to @trivialfis before, but wanted to bring up since it's been several months and we are applying latest changes to ours.
Here's what I'd recommend:

sklearn.py pred_leaf, pred_contribs, approximate_contribs

diffs here:
(some here, but not all): https://github.com/h2oai/xgboost/commit/bfff9ab50084eaf3311379d7ae194cca3b6f94f0
i.e. at: https://github.com/h2oai/xgboost/blob/bfff9ab50084eaf3311379d7ae194cca3b6f94f0/python-package/xgboost/sklearn.py
i.e. API related ones (not label encoder ones):

output_margin de-hard-code and use:

https://github.com/h2oai/xgboost/commit/355e415c1c8a5a148b5a4fef545d0ccc7b44b29c
As-is, output_margin now can no longer be chosen independently.  And pred_contribs etc. are not accessible still.

@pseudotensor Would you like to submit a pull request? Let's get the change into the upstream so that it'd be easier for you to sync your fork with the upstream.

Predict leaf is in the apply method.


Milestone 4: Feature | basic UI

🚩I have made the following changes to complete OOP School Library - Milestone/Project 4
Here is a summary of what has been done
Type of files changed/added

In this new Branch, I have ➕ADDED➕ the succeeding *📂 folders and/or **📄 files to complete this milestone's requirements
*📂 root/

*📂 helpers/

**📄 run_options.rb ➕

This helper is used to run the program options to the console and get the user input
for the option to run.

**📄 setup_options.rb ➕

This method returns an object with all the option's name and methods to run the program.

**📄 validate_inputs.rb ➕

This helper is used to validate the age and name inputs, checking if they are Integer and String respectively.

*📂 ui/

**📄 app.rb ➕

The -App- class will be the console app entry point.

It contains the following methods:

list_all_books
list_all_people
create_teacher_or_student
create_book
create_book_rental
list_all_rentals
exit_program

**📄 dependencies.rb ➕

The dependencies file is used to require all the files needed to run the program.
This will help to avoid requiring files in the main.rb file or elsewhere.

**📄 main.rb ➕

The main method holds the only instance of the -App- class gets the options to run the program and invokes run_options passing the options and the app instance to run the program.
This method is invoked right after its definition.

🛑 Notes on main.rb

"If there ever arises a requirement, even if it's not explicitly outlined in the current specifications, to transform main.rb into an executable file, please feel free to inform me. I'm fully prepared to handle the task. The necessary modifications have already been implemented, and I'm now awaiting the push to proceed."

I have ✏MODIFIED✏ the succeeding *📂 folders and/or 📄** files
*📂 root/

*📂 lib/

**📄 book.rb ✏

I have added :rentals to attr_accessor.
The newly created add_rental method has been added to add a new rental instance to the @rentals array. as long as the rental is not already in the array.

**📄 person.rb ✏

I moved :rentals to attr_reader since it does not need to be modified.
The newly created add_rental method has been added to add a new rental instance to the @rentals array if the rental is not already in the array.

**📄 rental.rb ✏

In order to print the rental's data, such as data, book-tittle, book-author, person-name, I have added the rental_to_s method to the class.

**📄 student.rb ✏

I've modified :classroom attr for only getter since there is no need to be modified.
I replace the class initializer to receive a classroom parameter after the name parameter.
I have also replaced the safe navigation operator with a setter method
including a conditional.

Thank you for taking the time to review this PR ⭐

If you need additional information or have any questions, please don't hesitate to contact me on Slack as Arturo (Arthur) Emanuel Guerra Iturres. I'll be happy to help you out. 🎯

Hi, @Kossi-stack! I appreciate your time in reviewing this PR 👍🏼. Have a fantastic weekend, and may your coding endeavours be filled with joy! 🤠


Dop.blade_rep doesn't do anything

Note the lack of return statement or state modification
https://github.com/pygae/galgebra/blob/7995365bde093c1cec9b3f8523d346e67a46a28c/galgebra/mv.py#L2098-L2106
Also an LGTM error on line 2102

For loop variable 'term' is not used in the loop body.

It seems to be intending to modify the state.
Sent with GitHawk

Does this seem reasonable?
def blade_rep(self):
    return Dop([(coef.blade_rep(), pdiff) for coef, pdiff in self.terms)], cmpflg=self.cmpflg, ga=self.Ga)

Seem reasonable and consistent with the intention.

Alternatively, we could just remove it - we can already spell that operation
some_dop = ga.grad
some_dop_base = some_dop.simpify(Mv.base_rep)
some_dop_blade = some_dop_base .simpify(Mv.base_rep)
assert some_dop_blade == some_dop_base