$HOME/.Xresources were not being loaded

I have an old laptop which I wanted to install something lightweight on, but no rolling release (because I don't use it much).
I settled for minimal debian, then i3 on top. As a display manager I choose ly. As terminal I selected urxvt.
I styled ``urxvtin the$HOME/.Xresourcesfile, but on login, it was never loaded. But when I ranxrdb merge ~/.Xresourcesit was being picked. Using a.xinitrc` also was not working.
After manually debugging by running /etc/ly/xsetup.sh I could see that the script was failing with has_option from the XSession scripts was not being found. I went down the rabbit hole and discovered that xsetup.sh does not run Xsession, only everything in the subfolders of /etc/X11/Xsession.d. So I made a custom 10-custom-xsession file in that dir with the has_option function. This improved, but still my settings were not loaded.
Finally I moved the whole section
if [ -d /etc/X11/Xresources]; then
...

from the xsetup.sh file, which actually contains the xrdb merge call, above the "Load Xsession scripts" section.
This now worked. However, it feels pretty much a hack. Can I do this differently?

I went down the rabbit hole and discovered that xsetup.sh does not run Xsession, only everything in the subfolders of /etc/X11/Xsession.d.

It only seems to source it after trying to merge the resources, perhaps that's the sole reason for this issue.

Finally I moved the whole section from the xsetup.sh file, which actually contains the xrdb merge call, above the "Load Xsession scripts" section.

Did you try removing the custom 10-custom-xsession file afterwards to see if it works? If it doesn't, perhaps the other files (namely OPTIONFILE, USERXSESSIONRC and ALTUSERXSESSION) need to be sourced as well, given that they're just set and forgotten later right now.
My guess here is that this condition:
if [ -f "$USERXSESSION" ]; then
  . "$USERXSESSION"
fi

needs to be moved above, right before the script deals with the Xresources files.


Multi-node setting?

https://github.com/KaiyuYue/torchshard/blob/89e21def180bf6063ceb2e312a61631173abc7e7/projects/minGPT/main.py#L150
I have noticed that the group_size is set to world_size in examples, but in fact the group_size can be set to other numbers according to my understanding.
https://github.com/KaiyuYue/torchshard/blob/main/torchshard/distributed/core.py#L18
I have also found that the get_world_size() will return the number of all processes.
The two findings make me confused in a multi-node setting, say 2 nodes with each node with 2 processes.
If the group_size is 2, then there are 2 distinct groups besides the default group (w/ overlap). However, get_world_size() is used without specifying a group can make a layer be splitted to 4 parts, which is expected to be 2 in our case.
Correct me if I am wrong.

The case is actually that a Node can maximally hold 8 GPUs only. If we have 16 GPUs, we have to make them onto 2 Nodes, with 8 GPUs on each Node.
At the same time, synchronization in sharding is known to be slow between Nodes, so we have to make the group size to 8 but not 16.


make it possible to send and receive datagrams for a HTTP/3 client

Depends on #2963.
Most users will just need to call http3.RoundTripper.RoundTrip(). For users that need access to the underlying QUIC connection, they can get a particular client (which owns one QUIC session) using the newly added GetRoundTripperForRequest. This will be heavily used by MASQUE.

This PR is incomplete. H3 Datagrams come with a flow ID, and we should make sure we properly parse it.

This PR is incomplete. H3 Datagrams come with a flow ID, and we should make sure we properly parse it.

@marten-seemann what’s the plan (if any) to resurrect this functionality?
thanks!


fix: correct ITs with centos

Closes #531

Updated dependencies (ppom, asto, vertx)
Updated fedora verstion
Used redhat instead of centos to work with yum, added come extra configs in RpmSliceITCase
Only one rpm file was left in RpmSliceITCase as redhat fails to install any rpms with dependencies (time package does not have any external deps)
Extended RpmSliceDownloadITCase to install packages from repository with subdirectories (this case was moved from RpmSliceITCase)

@dgarus please, review


Remove unused cucumber step definitions

Checked by running the following:
bundle exec cucumber -f usage

Woah cucumber -f usage :heart:


Remove legacy registry provider code

Change the JSON property back to registry_provider. We should guarantee there are no clients still running the 1.3.6 version. Remove related dead code.
https://github.com/dockstore/dockstore/blob/db7890a5b46234612fbf6c8ff40e06a7c1f3e397/dockstore-webservice/src/main/java/io/dockstore/webservice/core/Tool.java#L321
https://github.com/dockstore/dockstore/blob/5fb31336b608713655ce725fcf08a4db23153ad3/dockstore-webservice/src/main/java/io/dockstore/webservice/core/Tool.java#L342
┆Issue is synchronized with this Jira Story
┆Issue Number: DOCK-736
┆Sprint: Sprint 57- Frilled shark
┆Issue Type: Story

To me, it seems like changing it back to registry_provider right now would break 1.7.0

Current behaviour summary (mostly for myself):
Registry is stored as a string in the DB (ex. "quay.io")
It's returned as:

enum (ex. QUAY_IO) from the "registry" property
string (ex. quay.io) from the "registry_string" property

Confirmed that DockstoreTool appears in the OpenAPI -- verifies what the PR does, not the ticket description.
Webservice - 580a271
UI - 2.7.3-72-gd167ef76
Compose Setup - 36c8ddc
Deploy - 1.10-56-gda5d5ab


Bug 2140998: Display correct number of Nodes and NADs in the Overview

📝 Description
Fixes:
https://bugzilla.redhat.com/show_bug.cgi?id=2140998
Display the correct number of Nodes and NADs/Networks in the Virtualization Overview page.
Cause:
The number of Nodes wasn't fetched correctly (namespace was unnecessary).
Regarding NADs, there was a mismatch between "nads" and "networks" in the code.
Note:
When testing/reproducing this bug, make sure the project name/namespace in the Overview page is the same - for "before" and "after" scenarios.
🎥 Screenshots
Before:
Incorrect numbers in the Nodes and Networks cards (after clicking on the number, the correct list is displayed):

After:

@avivtur @metalice @pcbailey @yzamir @vojtechszocs please review

/lgtm


New IAS authentication

IAS has a new api-key based authentication (described in revision 5.0 of IAS spec)

Can you please link the spec here?

Can you please link the spec here?

no public link yet :-(

Closed with PR #49

Can you please link the spec here?

finally there is a public link to v5: https://api.trustedservices.intel.com/documents/sgx-attestation-api-spec.pdf


Accelerate labeling polygons from per-face labels

The main improvement is first removing faces that are not contained by the target polygons, since this is a much faster operation than doing the overlay. This could yield slightly different results, since partially-overlapping faces are discarded. Therefore the heuristic of buffering the input polygon regions before selecting the subset of faces is used.


feat: global scrollbar style

https://sovryn.monday.com/boards/2218344956/pulses/2321968045

added wide scrollbar style

also:

moved scrollbar styles from components to utilities
added postcss-nested to allow sass like nesting when writing custom classes
added postcss-import so custom tailwind classes could be used in any order, without a need to import declaration file in specific order.
removed unused packages

add body { overflow: overlay; } style
add ::-webkit-scrollbar { background-color: transparent; } style
add a small amount of margin-right or padding-right to page content so that the content doesn't touch the scrollbar

Will check what I can do, but as I understood overflow: overlay is deprecated feature and behaves differently across browsers.
Using it, will put scrollbar behind elements that are positioned using fixed - which is our header.
If I will not be able to appear above header within an hour I will just ignore this comment :D


Minor improvements

run-mongod(usage): remove unused optional variables from the output
run-mongod(cleanup): don't try to remove instance from the replica set
run-mongod-replication(usage): add MONGODB_KEYFILE_VALUE to the list
of required parameters
common.sh(mongo_create_user): don't try to show value of empty variable

PTAL @bparees @omron93

lgtm.  @omron93 ?

@bparees @php-coder LGTM. Thanks for cleaning the code.


Add pickle files for holdings/documents directory

Currently the documents directory does not have associated pickle files, which means any access by PdsFile needs to go to the filesystem instead of the pickle files. It would be more consistent to have pickle files for the documents directory as well. This involves updating the scripts in validation and also making any needed modifications to PdsFile.

Issue #75 is a stopgap that was necessary because this is not done.  If and when this is done, consider reversing #75.


ci: add codecov

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:

Merged to make the configuration take effect first.


⚠ HawkeUno has degraded performance

In 5d09cc2, HawkeUno ($UNO) experienced degraded performance:

HTTP code: 403
Response time: 1519 ms

Resolved: HawkeUno performance has improved in 8add4e1 after 6 minutes.


[Snyk] Security upgrade eslint from 8.29.0 to 9.15.0

Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

website/package.json
website/package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Regular Expression Denial of Service (ReDoS) SNYK-JS-CROSSSPAWN-8303230
  170

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

Checkmarx One – Scan Summary & Details – 1676282c-9982-4933-80a8-bf6fc4b3fac7
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2024-4068
Npm-braces-3.0.2
Vulnerable Package

CVE-2024-47831
Npm-next-13.0.6
Vulnerable Package

CVE-2024-47874
Python-starlette-0.22.0
Vulnerable Package

Reflected_XSS
/website/src/pages/api/update_task.ts: 42
Attack Vector

Reflected_XSS
/website/src/pages/api/new_task/[task_type].ts: 28
Attack Vector

CVE-2024-28176
Npm-jose-4.11.1
Vulnerable Package

CVE-2024-28849
Npm-follow-redirects-1.15.2
Vulnerable Package

CVE-2024-35195
Python-requests-2.28.1
Vulnerable Package

CVE-2024-4067
Npm-micromatch-4.0.5
Vulnerable Package

CVE-2024-47764
Npm-cookie-0.5.0
Vulnerable Package

Missing_CSP_Header
/website/src/pages/api/new_task/[task_type].ts: 71
Attack Vector

Fixed Issues

Severity
Issue
Source File / Package

CVE-2022-46175
Npm-json5-1.0.1

CVE-2023-26115
Npm-word-wrap-1.2.3

Cxab55612e-3a56
Npm-braces-3.0.2

Cxca84a1c2-1f12
Npm-micromatch-4.0.5

Reflected_XSS
/website/src/pages/api/new_task/[task_type].ts: 28

Reflected_XSS
/website/src/pages/api/update_task.ts: 25

Reflected_XSS
/website/src/pages/api/new_task/[task_type].ts: 15

Host Namespace is Shared
/docker-compose.yaml: 11

Host Namespace is Shared
/docker-compose.yaml: 5

Host Namespace is Shared
/docker-compose.yaml: 11

Host Namespace is Shared
/docker-compose.yaml: 50

Host Namespace is Shared
/docker-compose.yaml: 19

Host Namespace is Shared
/docker-compose.yaml: 19

Host Namespace is Shared
/docker-compose.yaml: 18

Host Namespace is Shared
/docker-compose.yaml: 27

Host Namespace is Shared
/docker-compose.yaml: 18

Host Namespace is Shared
/docker-compose.yaml: 26

Host Namespace is Shared
/docker-compose.yaml: 27

Host Namespace is Shared
/docker-compose.yaml: 4

Host Namespace is Shared
/docker-compose.yaml: 5

Networks Not Set
/docker-compose.yaml: 27

Networks Not Set
/docker-compose.yaml: 11

Networks Not Set
/docker-compose.yaml: 5

Networks Not Set
/docker-compose.yaml: 18

Networks Not Set
/docker-compose.yaml: 4

Networks Not Set
/docker-compose.yaml: 19

Networks Not Set
/docker-compose.yaml: 5

Networks Not Set
/docker-compose.yaml: 26

Networks Not Set
/docker-compose.yaml: 27

Networks Not Set
/docker-compose.yaml: 18

Networks Not Set
/docker-compose.yaml: 50

Networks Not Set
/docker-compose.yaml: 11

Networks Not Set
/docker-compose.yaml: 19

Missing_CSP_Header
/website/src/lib/poster.ts: 4


Feature Request: Notify gRPC service of upstream response codes - Useful for multi-tenant Rate Limiting and Concurrency Limits

Terminology:
https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/terminology#terminology
Upstream: An upstream host receives connections and requests from Envoy and returns responses.
Background:
We would like to use envoy + a custom gRPC rate limiting service to replace our existing custom proxy stack.
We service requests in a mutli-tenant manner and require custom global rate limits (per tenant) and global concurrency limits (per tenant).
We are also required to omit error response codes from our rate limits. (Cannot be determined by proxy prior to upstream response).
Currently we achieve this with our custom stack by:
-rate limiting / concurrency limiting before sending a request upstream
-storing keys/counts in redis
-if the proxy gets a response from a upstream server it sends a message to a custom service with a payload of [request url, upstream response headers, upstream http response code]
The feature request:
Add the ability to have envoy notify a custom gRPC service of upstream results [request url, upstream response headers, upstream http response code] in a manner similar to the way gRPC rate limiting is done, but AFTER envoy has the upstream response.
This could be async/non-blocking as we have no use cases that would require altering the response at this point.
This would solve per tenant concurrency limits by allowing the rate limit service to increment a concurrency counter when a request is accepted and the post-upstream-notified service could decrement the concurrency counters.
This would also solve the per tenant rate limits with complex requirements by allowing the rate limit service to only check counts (against redis or other datastore) and the post-upstream-notified service to increment the counters.

Will gRPC access logs works for that? Do you need to change the response after envoy receive from upstream?

Will gRPC access logs works for that? Do you need to change the response after envoy receive from upstream?

We don't need to change the response at that point in time. This is an interested idea that may work. Thank you.
I will experiment with it and report back.


this is in memory cache ?

i implemented this cache for my local caching, but did not see any improvement in performance...
the cache is on file or in memory ?

@DharaSingh55 This is indeed an in-memory cache, as per the documentation.

did not see any improvement in performance...

Maybe your workload's bottleneck isn't I/O? Maybe you misused the library? In any case, I can't help you if you don't include more details in your issue. At the very minimum, a brief explanation of the type of workload your application handles, where you implemented mlcache, and how (ideally with example code).

Closing this due to lack of activity. Feel free to come back here when you have more details @DharaSingh55


node 15, eventually.throw ?

This construct works in Node 10,12,14 and breaks in Node 15
expect( func(arg) ).to.eventually.throw(TypeError);

Real test case:
https://github.com/sifive/duh-core/blob/master/test/invalid.js#L26
Node 12 (worked before)
https://github.com/sifive/duh-core/runs/1301279599?check_suite_focus=true#step:4:49
Node 15 (does not work as expected)
https://github.com/sifive/duh-core/runs/1301279658?check_suite_focus=true#step:4:60

I reproduced this with Node 16 as well.

@drom Did you try checking the Changelog for Node between 14 and 15 to see if there is reference to a related breaking change?

Another option is to quite using chai-as-promised to test rejected promises. This simplifies your code, doesn't require a global handler that might have unintended side-effects and is fairly straightforward.
The pattern is to attach a then handler to your promise that asserts failure if called, then attach a catch/reject handler to the promise that does nothing if called, becoming a "successful" promise in the process. Here's an example:
return Promise.reject(new Error()).then(() => assert.fail(), () => {});

That's using a two-argument call to then where the second argument is the do-nothing rejection handler.  If you want to test that a successul promise would fail if used with this patten, here's an example where the assertion will fail because the first promise will successed, triggering assert.fail():
 return Promise.resolve().then(() => assert.fail(), () => {});

This is approach I'm going with. Promises are already complex enough so removing the extra complexity added by the global side-effects of chai-as-promised can be considered an improvement to the code base.


Disable Signaturepad

Hey,
I just added the possibility to disable the pad, so an existing signature can be shown without an extra element, because I needed this feature for my own project. Maybe you're interested to pull this in.

Thanks for the pull request. The feature will be released soon.


Still work in progress?

Is this project discontinued?

Unfortunately it is. I'll archive it to make that more clear. There was no real momentum with the project and I no longer needed it for my own projects so it stalled out before really becoming a fully developed module. Thanks for checking and sorry I didn't have a better answer.


Use Environment Variables

Going through our a few of our .NET docs, it looks like most guides use Environment Variables rather than hard-coding the values. This document should probably do the same for consistency.

Example

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

ID: 2700fc00-4872-6fcf-65e0-ab9fd5fde2a1
Version Independent ID: e828bbb5-2c22-17f2-32aa-aa3773c55806
Content: Build an Azure Cosmos DB .NET Framework, Core application using the Gremlin API
Content Source: articles/cosmos-db/create-graph-dotnet.md
Service: cosmos-db
Sub-service: cosmosdb-graph
GitHub Login: @LuisBosquez
Microsoft Alias: lbosq

@MikeCodesDotNET Thanks for the feedback. We are forwarding the feedback to doc author for the evaluation.
@LuisBosquez Please check the feedback and see if it can be modified to include the env variables.

Fixes https://github.com/MicrosoftDocs/azure-docs-pr/pull/105196

@MikeCodesDotNET Thanks for the feedback, I updated the sample and doc to use environment variables in the above PR. The updates will be live on Monday.
We will close this issue now.

#please-close


Remove out-of-date specification for C# language from Visual Studio install

We are shipping an out-of-date specification for C# with Visual Studio. We should stop doing that and link to the online specification at https://github.com/dotnet/csharplang/blob/master/spec/README.md .
This is also tracked at https://github.com/dotnet/roslyn/issues/18515 and tracked internally at 405168. and 428131.

@gafter We import that online spec here: https://docs.microsoft.com/en-us/dotnet/csharp/language-reference/language-specification/
We're directly pulling in the master branch and the daily publish cycle will pull in any updates made to the csharplang repo.  I have a similar comment at dotnet/roslyn#18515

@BillWagner Linking to that imported version makes a lot of sense.

So is there any action item for us on the docs team here? Or should this be handled by the internal bug to have VS link to our new docs?

Closing since this is being tracked internally and doesn't apply to docs.


ROS2 support

I am looking at the ROS2 (Humble) support for this package, but it looks like it is mainly ROS1? It's fine I'll just use the C++ library then and integrate it with ROS2, but if I am missing the ROS2 support please point it out.
Thank you

Hi @Marnonel6, the ROS support was designed for ROS 1. Either way those messages can be also used in ROS 2. @SteveMacenski is working in a Nav2 Integration (#73), but it's not ready yet.

Thank you for the update! I'll write a ROS2 wrapper for the C++ implementation.


Tumblr.com not accessible using VPN.

add blogs then 'Add to Queue' can't click,if i want "double click on a blog adds it to the queue",it just crash。I also tried to pull the project to the queue, but it only keep a few seconds then disappeared。i try this app on win10 and win7,use the different versions, all cant work. besides,i also use vpn. Do I need to do other operation,hope to get some help .and thank you for the program.and the Screenshots

Blog is shown as offline, that's why it's not crawlable. Check your connection settings, or describe here in detail how you connect to your VPN, what kind of VPN it is, your windows version and what VPN client you use.
Finally, can you access tumblr.com without the VPN, can you access it with your VPN?

Could you also try this version of TumblThree? Remove your blog and re-add it. It could also be that tumblr is blocking its api to your network if you use something like hide.me. It could also be that someone on that network is using the api already, thus your requests are dropped because they are already above the limit.
Did you set anything in the HTTP Proxy of TumblThrees settings?

I'm closing this since I already think this has nothing to do with TumblThree.

Thank you for your help ,i can access it but must use vpn .I'm using lantern now. I also thought it was VPN's problem，i will  try something else. thank you and your work again.

The problem is resolved. I just have to manually set the proxy port.


LibreOffice.LibreOffice version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Please use the wingetcreate update for existing packages so we don't loose metadata.

Oh, I'm sorry, I'll remember that next time


Godot 4.0 crashes on Linux Mint when selecting "Import"

Godot version
4.0 alpha 15
System information
Linux Mint 20.3 Cinnamon 64 bit
Issue description
Testing Godot 4.0 alpha 15 on Linux Mint Cinnamon 64 bit. It crashes after clicking on "Import" option and clicking on "Browse", then the program freezes and then closes. No project selection is possible. It happens with both the 64 bit and 32 bit versions of Godot.
Steps to reproduce
Clicking "Import" then "Browse".
Minimal reproduction project
No response

No crash on Linux Mint 20.3 xfce 64 bit (both on single- and multiple-window mode), so this looks like a desktop environment issue.

Thank you for your reply. OK, keeping in account this is my own problem, I'll try to find a solution.

Can you reproduce this when using the --single-window command line argument?

System:    Kernel: 5.4.0-125-generic x86_64 bits: 64 compiler: gcc v: 9.4.0
Desktop: Cinnamon 5.2.7 wm: muffin dm: LightDM Distro: Linux Mint 20.3 Una
base: Ubuntu 20.04 focal
Machine:   Type: Laptop System: SAMSUNG product: 305V4A/305V5A v: 04PW serial:  Chassis:
type: 9 serial:
Mobo: SAMSUNG model: 305V4A/305V4A v: 04PW serial:  BIOS: American Megatrends
v: 04PW.ME99.20110908.SKK date: 09/08/2011
CPU:       Topology: Quad Core model: AMD A6-3410MX APU with Radeon HD Graphics bits: 64 type: MCP
arch: Fusion L2 cache: 4096 KiB
flags: lm nx pae sse sse2 sse3 sse4a svm bogomips: 12777
Speed: 973 MHz min/max: 800/1600 MHz Core speeds (MHz): 1: 981 2: 847 3: 990 4: 987
Graphics:  Device-1: AMD Sumo [Radeon HD 6520G] vendor: Samsung Co driver: radeon v: kernel
bus ID: 00:01.0 chip ID: 1002:9647
Device-2: AMD Seymour [Radeon HD 6400M/7400M Series] vendor: Samsung Co driver: radeon
v: kernel bus ID: 02:00.0 chip ID: 1002:6760
Display: x11 server: X.Org 1.20.13 driver: ati,radeon unloaded: fbdev,modesetting,vesa
resolution: 1366x768~60Hz

./Godot_v4.0-alpha15_linux.x86_64
Godot Engine v4.0.alpha15.official.432b25d36 - https://godotengine.org
Vulkan API 1.2.131 - Using Vulkan Device #0: Unknown - llvmpipe (LLVM 12.0.0, 128 bits)
WARNING: lavapipe is not a conformant vulkan implementation, testing use only.
/lib/x86_64-linux-gnu/libasound.so.2: undefined symbol: snd_dlpath
Segmentation fault (core dumped)

This happens when I try to create a new project without clicking on "Browse":
/lib/x86_64-linux-gnu/libasound.so.2: undefined symbol: snd_dlpath
WARNING: Blend file import is enabled in the project settings, but no Blender path is configured in the editor settings. Blend files will not be imported.
at: _editor_init (modules/gltf/register_types.cpp:75)
WARNING: FBX file import is enabled in the project settings, but no FBX2glTF path is configured in the editor settings. FBX files will not be imported.
at: _editor_init (modules/gltf/register_types.cpp:99)

I hope I gave you some useful info.

Duplicate of https://github.com/godotengine/godot/issues/58903. Note that your graphics card has no Vulkan support, so you'll have to use the OpenGL renderer as I explained in https://github.com/godotengine/godot/issues/58903#issuecomment-1276812163.


Added bot counter. Fixed bot limit with actual spawn count.

Press F12 check console for bot count after feeder has started. Will implement visual overlay next.

This is great right here.  I have already implemented visual overlay, quite simple.  I'll show you the few lines of code to make it, when I get home.

Thanks, feel free to submit a pull request with your overlay. Didn't have the time to implement the floating div with the counter.

I'm at school atm :P   Will do so, when I get home.

This is a nice update, but it would be better if it actually counted the number of ACTIVE bots instead of the number of bots. I got this on my server by having it add on +1 when the server has found the leader board, and -1 on Disconnect of the bot,
So that means if the bot doesn't spawn at all, it won't count :D

Have you looked at the code? It pretty much does exactly that. Instead of on leaderboard update it increments on successful socket connection once it spawns in. Decrements on disconnect event. If anyone finds it more accurate to have it onLeaderboard it can easily be moved.

Wait what?
Oh lol, I don't know why my browser didn't load the rest of the code. Also, this is great right here. I see how it works now xD
Before I knew it didn't make much sense.

anyone tested this? looks huge!

Sorry about that lol I'm still getting used to git. Could have had a lot less commits! Anyways I tested it myself and it works great. Anyone else want to confirm?

Yep
Working like charms gcsoftware contact me on skype please :nicolasdelito

+1

@MasonBurdette contact me on skype please : nicolasdelito

many commits are much fun, see no issue there! was talking about the feature :+1:


[BUG] Want to modify the text as "You have searched for " which shows when we search

Description
We need to replace the text "SEARCH?QUERY=" with "You have searched for ".
Screenshots

Any additional information?
@rupali-codes Kindly assign me under GSSOC'23
What browser are you seeing the problem on?
Chrome
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

This issue being resolved in #1297. Thanks for your concern. Feel free to raise any other issues you want to :)


IDEA: Open Source Design System

How interested would you and other developers be on having a consistent design system working along with Borealis?
I started last year to build one, called Aurora Ds, but ended up stopping due to a couple of reasons.
You can check out the skeleton here, with a few elements borrowed from a few people for a quicker ramp up :P

https://www.figma.com/file/XryU2KgqF0bgTMbH1Z1grgqy/Aurora-Design-System?node-id=6%3A54

The idea is to document and support all current and future Borealis components and layouts and simplify the work of developers and makers on bringing new apps to life.
Using figma with it's very permissive free tier, allows easy replication and building new UI's from components, that will be mapped to Borealis's code.
Eventually a plugin for figma could be built to export either themes or layouts, with it's simple plugin framework, or directly through API access. This is similar to what I already today with Diez Framework: https://diez.org/.
I think this could vastly simplify building in a very visual way and create richer experiences.
I also loved how attuned we are 😁 , Aurora (design system) Borealis (UI library).

Hey, I think that's awesome! I love that the names match 😄
So the idea would be to implement all of borealis's views in Figma, and eventually be able to design the UI from Figma and output something that the library can automatically use? That would be really dope!
Does Figma support layouts?

That's it in a nutshell!
I also believe it will lower the barrier curve in building well designed, useable UI's since we can also easily have pre-defined patterns just ready to use.
The easiest to output will be just tokens, as there are many easier ways to do it, and quickly match it to a theme.
For components, maybe look into react or html plugins, to understand the best way to marry the two, either import and use those components, which might make them easier to update, or export based on specific object names or heuristics. A bit of investigation an testing will help here.
Figma has a very robust component based approach, to which you apply overrides, so it's quite easy to extract those properties for both layout and theming. I can more or less imagine some buildup of feature increments, but as usual the best is to start with the quick wins.
The available plugins should also make it easy to test translations, connecting to data (csv, json, gsheets), as well as a few other potentially interesting use cases.
I would be creating all the components and documentation on how to use them for now, and obviously testing :P.
I Read a bit through the documentation, and while it seems to allow quite a bit of flexibility, I think the best way to go is to focus on the most important UI patterns and make sure to support each one well at a time. Or optionally collaborate with specific devs using Borealis and growing it along side the needs.
I am unsure if there are other UI libs, or what other homebrewers are using, but it would be great to have wide use and be able to support a growing community.
For now I'll try to start mapping Borealis's built in views to components and make a couple of the most relevant atomic components and patterns, then we can review what works well, what I might have missed and where to go from there.
Let me know were we can have a quick chat or call, you can reach me at<EMAIL_ADDRESS>oh I'm on GMT - Lisbon

The thing is, you will still need to write actual C++ to glue everything together. Will Figma be able to output something that's usable in C++ without being painful, but still powerful enough to be worth using? I fear that Figma will lack flexibility in its output, and users might need to rewrite what Figma gives to make it usable...
Do you have an example of what can Figma give us? To gauge the feasability.
You can message me on Discord if you want, this is where the Switch stuff happens (RetroNX or switchroot servers), DM me directly

@alfatreze I am digging this from its grave - I am currently rewriting the library to use a more sane layout approach. Basically the views would be defined by XML files, like Android. The layouts are also unified under one single flexbox-like layout.
The flexbox layout is done using the Facebook Yoga library, which has a JS version. That means one could theoretically create a web interface to parse any borealis UI XML file and render it exactly as it would in a C++ app (with some work).

@alfatreze I am digging this from its grave - I am currently rewriting the library to use a more sane layout approach. Basically the views would be defined by XML files, like Android. The layouts are also unified under one single flexbox-like layout.
The flexbox layout is done using the Facebook Yoga library, which has a JS version. That means one could theoretically create a web interface to parse any borealis UI XML file and render it exactly as it would in a C++ app (with some work).


Flow

Codecov Report

Merging #90 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #90   +/-   ##
======================================
  Coverage    92.5%   92.5%
======================================
  Files          10      10
  Lines          40      40
  Branches        2       2
======================================
  Hits           37      37
  Misses          3       3

Impacted Files
Coverage Δ

src/components/Header.js
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Not sure what codecov's problem is, but I'm happy to let it slide for this PR because we're going to step up on testing in the coming days.


Error when a pydantic field is a tuple of variable length

First of all, thank you for this library, it is very useful!
I'm encountering an error when building the diagram of pydantic classes where some fields are tuples of variable size.
For instance, if I try running pydantic-2-mermaid on this class:
class Data(BaseModel):
    variables: tuple[Variable, ...] = Field(...)

I run into the following error:
Traceback (most recent call last):
  File "bin/pydantic-mermaid", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "lib/python3.11/site-packages/pydantic_mermaid/__main__.py", line 87, in main
    mg = MermaidGenerator(module_type)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/pydantic_mermaid/mermaid_generator.py", line 12, in __init__
    self.g: MermaidGraph = PydanticParser()(module)
                           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/pydantic_mermaid/pydantic_parser.py", line 94, in __call__
    field_type_name = _get_name(field.annotation)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/pydantic_mermaid/pydantic_parser.py", line 38, in _get_name
    sub_names = [_get_name(sub_type) for sub_type in get_args(v)]
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "lib/python3.11/site-packages/pydantic_mermaid/pydantic_parser.py", line 38, in <listcomp>
    sub_names = [_get_name(sub_type) for sub_type in get_args(v)]
                 ^^^^^^^^^^^^^^^^^^^
  File "lib/site-packages/pydantic_mermaid/pydantic_parser.py", line 31, in _get_name
    return v.__name__
           ^^^^^^^^^^
AttributeError: 'ellipsis' object has no attribute '__name__'. Did you mean: '__ne__'?

I am able to fix this by replacing the type of variables by list[Variable], but ideally this should work with tuples as well.
Thank you again!

Hi Luo,
Thanks for the feedback. This is fixed in 0.7.1


setup precommit to play nice with bidsschema CI

Find a pre-commit config that is compatible with the linting configuration used for the bidsschema tools and does not break anything in CI.
FYI: I am working on my main branch to be able to show the output of pre-commit CI:
for example: https://results.pre-commit.ci/run/github/150988005/1661942215.jTaSkg0fQD2c-OyJfvjqGw
issue corresponding to #1267 that was closed prematurely

@Remi-Gau FWIW I spent time I shouldn't have: https://github.com/effigies/bids-specification/compare/schema/lazy_loading...ci/precommit (I was working on the schema/lazy_loading branch and didn't rebase)
Not sure if this is helpful.

Cool thanks
will have a look but if this only touches the precommit config and not the schema tools linting CI I am not holding my breath


invalid line (no token ";" or ":") (Nextcloud ics)

When I try to use a Nextcloud hosted calendar I get this error in the node log:
 invalid line (no token ";" or ":") "<?xml version="1.0" encoding="utf-8"?>"
The same calendar works with MMM-CalendarExt
Config example:
      {
           module: 'MMM-CalendarExt2',
           position: "top_bar",
           config: {
             calendars : [
             {
                  name: "Demo",
                   url: "https://example.com/remote.php/dav/calendars/family/john?export",
                   auth: {
                       user: "john",
                       pass: "XXXX",
                       method: "XXXX"
                   }
               }
           ],
             views: [
               {
                 mode: "daily",
               },
             ],
             scenes: [
               {
                 name: "DEFAULT",
               },
             ],
           },
         },

Hmmm... that case will happen when the URL was wrong(or a result of your auth is wrong). Without real ical data, I cannot inspect what was the problem.

Thanks for the quick response. I am afraid I can't share the private calendar but I am testing with MMM-CalendarExt2 and MMM-CalendarExt in the same config with the same URL and MMM-CalendarExt fetches the data without issue (both modules were installed today on a fresh install of MagicMirror).
[CALEXT2] calendar:Demo>> Scanning start with interval:1800000
[CALEXT] https://example.com/remote.php/dav/calendars/family/john?export  >> Fetch starts.
[CALEXT2] calendar:Demo>> invalid line (no token ";" or ":") "<?xml version="1.0" encoding="utf-8"?>"
[CALEXT] https://example.com/remote.php/dav/calendars/family/john?export  >>
{ state: 'renewed',
  lastFetchedTime: '19-11-15 13:39:04',
  lastFetchedEvents: 12,
  lastFetchedOldEvents: 11,
  lastServedEvents: 23,
  error: null }

Two modules are using different parser. (CX2 uses better but somewhat strict)
invalid line (no token ";" or ":") "<?xml version="1.0" encoding="utf-8"?> says there is some invalid format in your ics parsing result. But I have no idea unless look inside the real data. sorry.

I've fixed it. update and try again.

Thanks!

I have the same issue with a Nextcloud calendar - fresh install of magicmirror and the module. Do you have any hints how to fix the error?

"Fixed" it myself - i forgot to specify the authorization...


ln: add possibility creating relative links

hi,
in some cases relative links are prefer.
but now ln.js is not provide the feature.
my case: ln('config/dev', 'config/current') and root dir of the config dir may be move on fs.
so, absolute links are not broken, too. using:
ln('/var/www', '/home/www')
excuse my beginner's English

@a-x- thanks for the PR! I think this will be handled by #282. If that PR doesn't cover this case, please ping this thread and let us know and we can take another look.

ok, i trust you)


BUG || Epost fees and recieved date are In-correct in QA

Epost fees and recieved date are In-correct in QA
Steps to Reproduce:

After Courtstaff E-signs record and it's record is shown to Epost admin
Check Amount and Recieved on date

Refer screeenshot

Hi @rajeshcherukumalli
resolved in UAT


Possible time rounding bug?

If the current time is 11:26 and you end a task with goc e, the end time will be set to: 11:30.
If you then start a new task before 10:30 with goc s ..., the start time will be 11:25 and not 11:30.
This should not be problematic since most breaks should last more than 5min.
If you continue to use goc s ... instead of goc e then everything will work as expected, unless you need to take a break for more than 5min, then you can use goc e.

Could be solved by storing the last end time, and use that as start time if lastEndTime > now? of couse, we must then assume that the event is not adjusted in google cal


[Feature Request] Add 'Compact channels.db on restart' to options menu

Channels.db files are growing larger and it is becoming a recommended practice to compact the file using guggero's chan tools. Can an option be added from the raspiblitz menu to select "Compact channels database on restart" ? Initially I was thinking that this would be a selection like the services menu, but it may be prudent to add an options menu where a user can turn on/off keysend, wumbo, channels.db compaction, etc.

Discussed here already, will be part of LND v0.12: https://github.com/rootzoll/raspiblitz/issues/1778
Hope you don't mind me closing the duplicate issue.


no doAnswer with an actual Answer parameter

Why does the doAnswer infix function take an "InvocationOnMock" but not an "Answer" as a parameter? I wish I could replace my thenAnswer(Answer) with an infix doAnswer(Answer)...

Tried to create a pull-request, but I'm not allowed to push my branch. How could I do that?

@speekha You'll have to fork this repository, change your git remote (git remote set-url origin "myfork") and push to that. Then you should be able to make a pull request.


Literally nothing in 360 media iFrames work.

VR viewers just come up black on all my devices. Even the gorilla example does not work.

I'm having the exact same issue. None of the demos work. The only thing I can get working is linking the script from Google servers and not locally as well as a video from Google servers. If I try to use the same exact script and/or video locally no dice.

Any work around for this? I tried adding allow="vr" in vrview.js, but couldn't get demos to work. Not Just video, I couldn't even load 360 images

I tried using new version of vr-polyfill and it seems to be working! was able to load both mono/stereo images

Yea I just noticed this too... Really sucks... I guess we will have to find other ways to do walk throughs

how did u update newer versions of vr-polyfill?

forge.js from go pro... believe it or not... lol

Switched to react 360 for now, but love to get back and fix the vrview-web lib some time


Closure Library deprecation

Our codebase here relies a lot on Closure Library, which got deprecated and is about to be sunset: https://github.com/google/closure-library/issues/1214

In the short term, we don't need to do anything as we pin the Closure Library to a version that we import and store under //third_party/closure-library/. So as long as there's no serious issue discovered (which we cannot fix in our copy) or incompatibility with future browser versions, we can in theory sit indefinitely on top of it.
For the long-term perspective, if we were to start migrating off it, the whole list of functionality used by us is pretty long, but what we rely most heavily upon is:

logging,
message channels,
Disposable,
the unit test and mock framework.

It seems that there's no ready-to-use list of replacement libraries for specific features, so the work would require analyzing these dependencies and finding analogs or reimplementing. As it was pointed out to me, this work might also require switching to more modern technologies like TypeScript, NPM, esbuild, etc.


Thoughts on streaming

Hi there :)
In the future I see many pipelines, any2video, audio+text+image2video
But am wondering about the ability to instead of prerendeing everything, to optimize for realtime streaming, continueing frames, and being able to alter the prompt as the video is generating. Something like realtime SD with last frame being a reference.
Then to achieve SOTA quality, stream to an upscaler, and character/obj consistency layers.

Thanks for your thoughts. We are working hard on what you have thought. Hope everything goes well and the new model can meet you soon.


Migrate Android Gradle build to new declarative plugins block

When building the app for Android, the following warning gets thrown:

You are applying Flutter's app_plugin_loader Gradle plugin imperatively using the apply script method, which is deprecated and will be removed in a future release. Migrate to applying Gradle plugins with the declarative plugins block: https://flutter.dev/go/flutter-gradle-plugin-apply

Fixed in version 0.3.4


Upgrading this and maintaining

I would like to upgrade this addon and like to maintain myself. Are you ok with that or should I fork and do all the upgrades and raise against this repo?

Absolutely. There's more context in this thread (https://github.com/ragnarpeterson/ember-scroll-to/issues/45#issuecomment-574751575), but basically no one has repo admin so we're kind of stuck.
I have merge and publish rights, so if you fork this repo and start working there, we could publish a deprecation warning on this package and point to your new repo.


Add force option for DockerMsftProvider package

I think that -Force may be a bit too strong to recommend as a default. What do you think about the approach in #583?

Thanks for pointing out the omission in our docs. I decided to add update as a separate tip in #583 instead, so I'm closing this one out.


Autocomplete APIs dont show .NET 5.0 prerelease packages

Describe the bug
I am seeing inconsistent results via the web interface and the autocomplete API. Am I doing something wrong?
The web interface shows me pre-release versions of .NET 5 packages: https://www.nuget.org/packages/runtime.linux-x64.Microsoft.NETCore.DotNetAppHost/5.0.0-preview.6.20305.6
The autocomplete API doesn't. I tried:

https://api-v2v3search-0.nuget.org/autocomplete?id=runtime.linux-x64.Microsoft.NETCore.DotNetAppHost&prerelease=true
https://azuresearch-usnc.nuget.org/autocomplete?id=runtime.linux-x64.Microsoft.NETCore.DotNetAppHost&prerelease=true

Neither does the search API:

https://azuresearch-usnc.nuget.org/query?q=runtime.linux-x64.Microsoft.NETCore.DotNetAppHost&prerelease=true

To Reproduce
Steps to reproduce the behavior:

Point your browser to

https://www.nuget.org/packages/runtime.linux-x64.Microsoft.NETCore.DotNetAppHost/5.0.0-preview.6.20305.6
https://api-v2v3search-0.nuget.org/autocomplete?id=runtime.linux-x64.Microsoft.NETCore.DotNetAppHost&prerelease=true

Expected behavior
The website and search API show consistent view of packages. Right now, the webiste shows the pre-release versions. I dont seem them via the nuget APIs

Hello, thanks for reaching out! Just curious, could you explain your scenario a little more? Why are you interacting with the search APIs directly?
To answer your question, the version 5.0.0-preview.6.20305.6 uses semantic versioning 2.0.0 due to the periods in the prerelease label. Older NuGet clients do not support semver 2, so, by default we filter out semver results. For the search API, you will need to include the semVerLevel=2.0.0 query string argument. For example: https://azuresearch-usnc.nuget.org/autocomplete?id=runtime.linux-x64.Microsoft.NETCore.DotNetAppHost&prerelease=true&semVerLevel=2.0.0
For more information, please refer to this documentation: https://docs.microsoft.com/en-us/nuget/api/search-query-service-resource#request-parameters


You should return destination_file here

https://github.com/promptslab/Promptify/blob/01fa4c754c2fed711d12b24c8fd610e443ce3e41/promptify/models/nlp/utils/utils.py#L27

Also in reality you shouldnt download it it was downloaded less than an hour ago


Fix too long resource name

Issue #, if available:
Fix #122
Description of changes:
Nested CDK constructs names get concatenated and if no resource name is specified in the construct the CDK construct name is used. This might cause some resource identifiers, such as ARNs to exceed the allowed length.
This fix reduces the length of few known resources.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Hi,
It's working perfect now.
Thanks.
Best regards :)


Tutorial: Manage a Liquidity order via the APIs

question: does this replace the existing one, or do they coexist?
Background

After discussion with @MM0819 and @TomMcL Tom has agreed to pick this up. He may need some brief @pennyandrews - not sure if you can either do that or whether @MM0819 does it?

After discussion with @MM0819 and @TomMcL Tom has agreed to pick this up. He may need some brief @pennyandrews - not sure if you can either do that or whether @MM0819 does it?

@rebeccajsimmonds, @MM0819 had a vision for what these tutorials could provide, but if he's not available I can probably provide enough background to go on.

What do we think is missing from a text tutorial which is not already covered pretty well here? Or do we just want a video in addition?

Done in #487


Update Presentations.md

@all-contributors please add @12simu for content, example.

@all-contributors please add @Probably-Poorvi for content, example.


local bot server https issue fixed

when using local bot server, the url shouldn't be https

Looks like we need to update tests too.


Empty languages in registry

Correct me if I'm wrong, but reading the code I got the impression that linters/formatters with empty arrays in the languages attribute will not be loaded (ex: https://github.com/mason-org/mason-registry/blob/main/packages/codespell/package.yaml).
I know I can set it manually with the overrides but it would be nice to have them being set for every filetype when empty.

This is correct. mason-bridge does not maintain a full mapping of languages to filetypes as discussed in #5 so not sure how it could set it for all filetypes as I couldn't find an nvim api that returns such thing. I think giving the user the ability to simply manually set the languages is sufficient until I (or someone else) can think of a good solution for generating/writing a complete mappings file.


Rounding error when deriving target resolution

Rounding error causes incorrect aspect ratio for some resolutions when deriving horizontal target resolution from source.
Seems pretty clear the error is in here:
    # Get Resolution
    source_res = [int(x) for x in job["resolution"]]
    logger.info(f"Source Resolution: {source_res}")

    v_res = int(proxy_settings["vertical_res"])
    res_scale = int(source_res[1] / v_res)
    h_res = int(source_res[0] / res_scale)
    logger.info(f"Output Resolution: {[h_res, v_res]}")

In my haste to ensure no decimals, I'm converting all variables to int, when in reality, only the final number ought to be rounded.
There may be alternative methods others have used that cover more cases.

Fixed in #135


General interface for curve points

Libsecp256k1 provides an interface for operations on curve secp256k1. However, following the principle "Be difficult to use insecurely", only a specific API which directly describes operands was exposed. For instance, instead of functions for sum and multiply by scalars on curve points, only specific functions for sum and multiplications for public keys were made available.
However, during development the number of objects which inherently are points on curve has been growing and presently there are public keys, pedersen commitments and generators.
Our application (leer) requires (de)serialization for curve points independent of the nature of those points (we are developing a virtual machine where stack may contain curve points and has native opcodes for curve operations). Besides that, such functionality is required by libsecp256k1 modules by themselves: rangeproof module uses its own point serialization format https://github.com/ElementsProject/secp256k1-zkp/blob/secp256k1-zkp/src/modules/rangeproof/rangeproof_impl.h#L54 .
Hence it looks expedient to provide API for point (de)serialization as well as for bidirectional casting between points and other objects with specific purposes (that are inherently points) .
Chosing of header byte
Currently there are these rules for headers byte.
Header bytes 0b00000000,0b00000001,0b00000110 and 0b00000111 are reserved for special form of public keys for historical compatibility with bitcoin code.
For other object:

Lowest bit always encodes whether y-coordinate of point is odd or even.
0b0000001x header is used for public keys
0b0000101x header is used for generators
0b0000100x header is used for pedersen commitments

Since point object is a uniting type, and there will probably be other specific objects which represent points in the future, it is pertinent to use a distinctive header, such as 0b1000000x or 0b1111111x.

Points module #33 which implements general point interface and uses header 0b1000000x for points serialization.

Thanks for your work but we don't want to go this direction. In our rangeproofs we don't think in terms of "odd" or "even" points and in schnorr signatures our R-coordinates are 32 bytes rather than 33. I don't see how a general API could handle this while still providing any useful generalization.


Update to full SPDX v2.1

Update the SPDX Python libraries to the SPDX 2.1 specification. The SPDX 2.1 specification is a major upgrade from SPDX 1.2 supporting relationships between SPDX documents and SPDX elements.

From @pombredanne :
We should drop support for 1.2 alright and yes focus on new 2.x things.
As far as I am concerned, I would be quite happy with a complete refactoring of the lib internals such that:

tag/value is the primary target
the internal model is NOT based on RDF data structures but for instance models based on attrs or cattrs, e.g. something pythonic and not tied to RDF.
RDF would only be an adapter of sorts feeding in that model and an "after thought" rather than being central to the data model.

e.g. RDF/XML is not something I care too much for. more like a nice to have for me.
If a refactoring could be done to cut the internal model ties from RDF that would be a big win (and a big work too).

@pombredanne @sschuberth @rtgdk It would be great if I could have your inputs on a few things :
Should I focus on the tag/value builders and parsers for the upgrade to spec 2.1 and keep the rdf part under the optional section ?
Also, since our primary focus is on the tagvalue, we might face problems when while conversions in each of the formats (and also while executing validation functions in individual classes)
I went through the docs of attrs, which will help us to get rid of the boilerplate code in classes that we have to define all the time.
We have separate files for builders and parsers for the rdf and tv formats. Can I have a brief idea of how do we want the libraries to be refactored for independence from each other ?
IIUC, this line -> if (not self.error) and (not self.document.validate(validation_messages)): in both the parsers (tv and rdf) will call the validate(...) function as defined in class Document(...). Now, since our primary focus is to take care of the tag/value files, there might be some fields in tag/value which are mandatory and we haven't included them in our rdf model. This will trigger the validate(..) function to return False when when it is called via the rdf parser.
So, this is where we're facing problems since we have a common validate API tied to both the tag/value and rdf models.
We could maybe pass additional fields like: validate(..., rdf=False, tv=True) and then handle things accordingly ? This would be a not so elegant solution, but will somewhat clutter our codebase with if...else statements.

Hi @yash-nisar. I'm sorry, but I'm going to pull myself out of this project as I personally have to need for it anymore, and I'm not a real Python guy anyway. As a result, I will probably not respond to issues or pull requests.

No problem :). Thank you for your reviews.
On Mon, Mar 19, 2018 at 6:42 PM, Sebastian Schuberth <
<EMAIL_ADDRESS>wrote:

Hi @yash-nisar https://github.com/yash-nisar. I'm sorry, but I'm going
to pull myself out of this project as I personally have to need for it
anymore, and I'm not a real Python guy anyway. As a result, I will probably
not respond to issues or pull requests.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spdx/tools-python/issues/47#issuecomment-374206642,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOLKgBG3_FFvJpCSTqQ6bqIOkjHHIne7ks5tf67XgaJpZM4SqEXV
.

--
http://www.somaiya.edu http://www.somaiya.edu/kjsmc
http://www.nareshwadi.org http://www.somaiya.edu
http://www.helpachild.in http://nareshwadi.org

Should I focus on the tag/value builders and parsers for the upgrade to spec 2.1 and keep the rdf part under the optional section ?

I would prefer to maintain support for the RDF.  I know there are several users of RDF for the Java libraries (since they are finding bugs 😉 I assume there will also be interested users for Python as well.  I’m OK with the primary focus on the tag/value since that is the larger community.  I can help provide support for any RDF related questions.

Also, since our primary focus is on the tagvalue, we might face problems when while conversions in each of the formats (and also while executing validation functions in individual classes)

There are some discussions of supporting other formats like JSON, YAML, and JSON-LD.  Having a design which allows for conversion between various formats would be very helpful for forward compatibility.

We have separate files for builders and parsers for the rdf and tv formats. Can I have a brief idea of how do we want the libraries to be refactored for independence from each other ?

Is there a design where we have a core model which is (semi) independent of the format and converts to build and parse different formats?  I have to admit I have not dived deeply into the current Python implementation, but the basic idea of having a model independent of the serialization format seems like it should work.  Where things got messy in the Java code is the validation (as you point out below) and maintaining the information on where in the input file the validation errors occur.

IIUC, this line -> if (not self.error) and (not self.document.validate(validation_messages)): in both the parsers (tv and rdf) will call the validate(...) function as defined in class Document(...). Now, since our primary focus is to take care of the tag/value files, there might be some fields in tag/value which are mandatory and we haven't included them in our rdf model. This will trigger the validate(..) function to return False when when it is called via the rdf parser.
So, this is where we're facing problems since we have a common validate API tied to both the tag/value and rdf models.
We could maybe pass additional fields like: validate(..., rdf=False, tv=True) and then handle things accordingly ? This would be a not so elegant solution, but will somewhat clutter our codebase with if...else statements.

I'll provide an opinion or two on this with the caveat that I have not spent a lot of time in the Python code, so the replies below may not match the current design/structure.
For the Java libraries, we tried to separate out the validation into 2 separate categories - parsing errors and specification related errors.  The tag/value specific requirements could be thought of as parsing errors since they are typically related to positional requirements to allow unambiguous parsing.  The specification related errors should be common for both RDF and Tag/Value.  For the positional requirements in tag/value we just threw exceptions rather than handle them in the validation since the resultant model could not be reliably produced.
If we want the validation to incorporate format specific errors, could we record those errors during parsing and store them in the model rather than passing a flag to the validation method?  We know when we are parsing which format is being used.
One other consideration in the validation is maintaining the location of the error in the input format.  This proved to be a challenge for the Java code design.  The line number and character positions need to be captured during the parsing and reported back during validation.  For the tag/value we only provided this for the parser errors (not the spec related errors).

Hi @yash-nisar! We are currently cleaning up this repo and I came across your issue that is still asking for support for spec version 2.1. Since this issue is already a bit older and many changes have been made in the repo: Are you still missing fields for spec 2.1?
Concerning the discussion above:
Beginning with #244 we are planning to refactor the data model and the builder structure. This will enable us to tackle the refactoring and isolation of the individual formats mentioned here.
In the course of the refactoring, the validation will also have to be checked. I like the approach described by @goneall for the java tools to distinguish between parser-related validation (which is probably less of a validation and more of a basic check if the content is parsable) and spec-related validation. With #212 we have an open issue concerning the validation.
In your view @yash-nisar, are there any outstanding issues from this issue or could it be closed?

Closing this for now, please ping if it should be reopened


one-to-many relationship with BigInt id always null

<EMAIL_ADDRESS>one is able to query by that id, but the field with the relationship is null
# schema.gql excerpt

type Token @entity {
  id: BigInt!
  owner: Account!
  transferEvents: [TransferEvent!]! @derivedFrom(field: "token")
}

type TransferEvent @entity {
  id: String!
  from: Account!
  to: Account!
  token: Token!
  transactionHash: String!
  timestamp: Int!
}

query {
  transferEvents(where:{token:"1"}) {
    id
    token {
      id
    }
  }
}

returns
{
  "data": {
    "transferEvents": [
      {
        "id": "0xee0d6104cff1b2020f357d3621cf0a8ff54ff6fa5dd0545f17052a7552a872ff-0x0",
        "token": null
      }
    ]
  }
}

Should be fixed in 0.0.71


docs: fix typo 'internalization'

Pre-flight checklist

[x] I have read the Contributing Guidelines on pull requests.
[ ] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/

thanks 👍


how to check whether current element is selected or not from Code point of view

Is there any way to check the status of the Selectize on an object is true or false.
Something similar to hasAttribute of Javascript.
Thanks in advance.

Not in the public api. Loosk like i missed that. However you can access everything through private stuff:
var handler = element.remember('_selectHandler')
// here you go
// handler.rectSelection.isSelected
// handler.pointSelection.isSelected

Thanks a lot Mr.Ulrich
Will check the same and in case of any other queries will get back and open this thread again.

Add this feature to public API

feel free to create a PR


Can't parse html correctly despite browser can

Parser is not able parse the url at below despite browser can. It parse the page without throw exception but parsed document contains almost nothing.
here is url http://www.glittercottage.com
It is tested on latest commit on main branch.

This has nothing to do with the parser. Have you ever tried receiving the content via, e.g., curl? The body I receive there is the same I receive / parse with AngleSharp.

What I can tell you is that the site you try to parse is protected by Incapsula. You will need to break this protection to gather the information. AngleSharp.Scripting.JavaScript may be helpful for that; but there is no version of the JS scripting library for v0.9.5 (there will be, however, one for v0.9.6).


Tensor Product Operator

close https://github.com/SciML/SciMLOperators.jl/issues/23

done with functionality. just needs tests now

done

i dont think there are any integration tests. @ChrisRackauckas could you merge and version bump? id like to start using these operators here: https://github.com/vpuri3/PDEInterfaces.jl

good to go

@ChrisRackauckas could you version bump?


Error Passing Post after upgrade to <IP_ADDRESS>1

Describe the bug
Select Enqueue on my Likes Tumblr and fails with Error1: Likes: Error parsing post
Hover on error and you get:
Internal error description
String reference not set to an instance of a string
Parameter name: s
To Reproduce
Set up a https://www.tumblr.com/likes
Authenticate
Activate Crawl and Enqueue this blog
Error appears quite quickly.
No download occurs but it dates the Last Complete Crawl.
Expected behavior
Should have started pulling down the images
With error, it should fail without updating the Last Completed Crawl.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

TumblThree version: [<IP_ADDRESS>1]
OS: [e.g. Windows 10 Pro]
Browser [e.g. Chrome]
Version [e.g. 100.0.4896.75]

Additional context
Log Details:
20220415 10:30:39.060	Err 	TumblrLikedByCrawler:CrawlPageAsync: System.ArgumentNullException: String reference not set to an instance of a String.
Parameter name: s
at System.DateTimeParse.Parse(String s, DateTimeFormatInfo dtfi, DateTimeStyles styles)
at TumblThree.Applications.Crawler.AbstractTumblrCrawler.BuildFileName(String url, Post post, String type, Int32 index) in C:\projects\Tumblthree\src\TumblThree\TumblThree.Applications\Crawler\AbstractTumblrCrawler.cs:line 497
at TumblThree.Applications.Crawler.AbstractTumblrCrawler.BuildFileName(String url, Post post, Int32 index) in C:\projects\Tumblthree\src\TumblThree\TumblThree.Applications\Crawler\AbstractTumblrCrawler.cs:line 506
at TumblThree.Applications.Crawler.AbstractTumblrCrawler.AddPhotoToDownloadList(String url, Post post) in C:\projects\Tumblthree\src\TumblThree\TumblThree.Applications\Crawler\AbstractTumblrCrawler.cs:line 227
at TumblThree.Applications.Crawler.AbstractTumblrCrawler.AddTumblrPhotoUrl(String text, Post post) in C:\projects\Tumblthree\src\TumblThree\TumblThree.Applications\Crawler\AbstractTumblrCrawler.cs:line 191
at TumblThree.Applications.Crawler.TumblrLikedByCrawler.AddPhotoUrlToDownloadList(String document) in C:\projects\Tumblthree\src\TumblThree\TumblThree.Applications\Crawler\TumblrLikedByCrawler.cs:line 317
at TumblThree.Applications.Crawler.TumblrLikedByCrawler.d__16.MoveNext() in C:\projects\Tumblthree\src\TumblThree\TumblThree.Applications\Crawler\TumblrLikedByCrawler.cs:line 265
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at TumblThree.Applications.Crawler.TumblrLikedByCrawler.d__11.MoveNext() in C:\projects\Tumblthree\src\TumblThree\TumblThree.Applications\Crawler\TumblrLikedByCrawler.cs:line 171

Thanks for reporting a problem. We'll have a look at it.

The issue has been fixed and closed. You can still comment. Feel free to ask for reopening the issue if needed.

Thanks so much.


registration successful toast notification

After registering, a success toast notification and redirecting to login page
I tried logging in using wrong password (It's throwing an internal server error)

I can work on this issues

Participant of which program?

Kwoc

Hey there are other things I noticed in your code • Backend modularising using mvc architecture • Login alerts for wrong password and wrong user • Removing register and login buttons from navbar after logging in and replacing them with logout button
=
what do you think?

Hey there are other things I noticed in your code • Backend modularising using mvc architecture • Login alerts for wrong password and wrong user • Removing register and login buttons from navbar after logging in and replacing them with logout button
what do you think?

Create separate issues for these


Update Vhost-device-can dependency on socketcan and move it in the main workspace

Summary of the PR
This PR updates the socketcan-rs dependency of vhost-device-can device and moves it
on the main workspace.
Specifically by updating the socketcan-rs dependency to "v3.3.1" this PR solve the following
issues noted in the Cargo.toml of the vhost-device-can:

socketcan-rs "v3.3.0" includes the following issue related to CANFD messages reception:

https://github.com/socketcan-rs/socketcan-rs/pull/61.

If rust version "1.80" is used, the compiler might complain about "std::mem::size_of":

https://github.com/socketcan-rs/socketcan-rs/pull/72

Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR have Signed-Off-By trailers (with
git commit -s), and the commit message has max 60 characters for the
summary and max 75 characters for each description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[x] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[x] Any newly added unsafe code is properly documented.

I think the two commits can be squashed, there's no reason to update Cargo.lock separately is there?
Generally LGTM!

Thanks for your review @epilys. Yes indeed I agree, I just merged everything in one commit.
SInce the last push I have also modified the coverage value under staging so the CI succeeds after moving
vhost-device-can, let me know if that's ok.

@TimosAmpel looks good. you can drop the Update licenses' symlinks for vhost-device-console commit since it's part of https://github.com/rust-vmm/vhost-device/pull/758

@TimosAmpel fyi you can (and probably should since this is your device) leave reviews in https://github.com/rust-vmm/vhost-device/pull/758 and other PRs in general even if you're not a maintainer. They just don't count towards the "two approving reviews required" limit.

@TimosAmpel fyi you can (and probably should since this is your device) leave reviews in #758 and other PRs in general even if you're not a maintainer. They just don't count towards the "two approving reviews required" limit.

@epilys Thanks for pointing that out. Yes, I will review the changes done the next days


Store next round in import state

Summary
Currently the import state stores the last accounted round which creates ambiguities: we cannot distinguish between the state before accounting round 0 and after. This works because block 0 contains no transactions, and we don't actually do accounting for it. However, with one phase import it will be a problem.
This PR changes the import state to store the next round to account instead, GetMaxRoundAccounted() is replaced with GetNextRoundToAccount(), and a blocking migration is added.
Test Plan
Added tests for the migration, and manually tested the migration when max round accounted is 0 and >0.

Codecov Report

Merging #559 (0c454d0) into develop (7cf9b17) will increase coverage by 0.32%.
The diff coverage is 78.84%.

@@             Coverage Diff             @@
##           develop     #559      +/-   ##
===========================================
+ Coverage    46.39%   46.71%   +0.32%
===========================================
  Files           24       24
  Lines         3897     3930      +33
===========================================
+ Hits          1808     1836      +28
+ Misses        1818     1817       -1
- Partials       271      277       +6

Impacted Files
Coverage Δ

idb/postgres/postgres_migrations.go
73.89% <71.42%> (+0.83%)
:arrow_up:

idb/postgres/postgres.go
39.84% <94.11%> (+0.19%)
:arrow_up:

Continue to review full report at Codecov.

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


Improve for filename with space in it.

I modify the script so it can handle filename with space.
mkdir encrypted_files

echo -e "Moving encrypted $1.vmdk to encrypted_files"
mv "$1.vmdk" "encrypted_files/$1.vmdk"
file_size=$(ls -la "$1-flat.vmdk" | awk '{print $5}')

echo -e "\nCreating copy of $1-flat.vmdk"
vmkfstools -c $file_size -d thin temp.vmdk
rm temp-flat.vmdk

echo -e "\nAdding $1.vmdk"
sed -i "s/temp-flat/${1}-flat/" temp.vmdk
if [ "$#" -ne 1 ]; then
  if [ $2 != "thin" ]; then
    sed -i '/ddb.thinProvisioned/d' temp.vmdk
  fi
fi
mv temp.vmdk "$1.vmdk"

echo -e "\nCopying $1.vmx"
mv "$1.vmx" "encrypted_files/$1.vmx"
cp "$1.vmx~" "$1.vmx"

retVal=$?
if [ $retVal -ne 0 ]; then
    echo -e "Error: unable to find vmx backup. You may be unable to re-register the virtual machine."
fi

echo -e "\nMoving encrypted $1.vmsd to encrypted_files"
mv "$1.vmsd" "encrypted_files/$1.vmsd"

echo -e "\nMoving encrypted $1.nvram to encrypted_files"
mv "$1.nvram" "encrypted_files/$1.nvram"

echo -e ""

echo -e "\nValidating..."
vmkfstools -e "$1.vmdk"

retVal=$?
if [ $retVal -ne 0 ]; then
    echo -e "\nError. Trying to update the file size."
    file_size_num=$(( file_size / 512 ))
    file_size_num_plus_one=$(( file_size_num + 1 ))
    sed -i "s/${file_size_num_plus_one}/${file_size_num}/" "$1.vmdk"
    vmkfstools -e "$1.vmdk"
    retVal=$?
    if [ $retVal -ne 0 ]; then
        echo -e "\nError. Could not decrypt. Please consult CISA's guidance for further assistance."
    else
        echo -e "\nSuccess! Unregister the virtual machine and re-register it and you should be good to go.\n"
    fi
else
    echo -e "\nSuccess! Unregister the virtual machine and re-register it and you should be good to go.\n"
fi
exit $retVal

Thank you!! Incorporated in https://github.com/cisagov/ESXiArgs-Recover/commit/3dba6d8aac56397b139dc71a516a1956a405f5f6


Avoid dropping _cache_card until after a database update is completed.

This will prevent the site from going down for a couple of minutes during
a scryfall update (which are daily when spoilers for new sets are being
releaesed).
Fixes #6345.

I'm adding "do not merge" label here because this makes me nervous. Local testing says it's fine but want someone to say "I can't see anything obviously terrible here" before I merge :D @silasary
?

I tested this a bit locally, including having the update code pause at some key points and making sure the website still worked, and it all looks good to me.


Advanced Search Changes

[ ] Add Year Date Range to form
[ ] Convert Advanced Search to React
[ ] Remove 'Advanced Search' link in upper right hand corner when on Advanced search page as it's confusing
[ ] Hide the advanced search form itself once the user has submitted
[ ] Add 'Clear Form' button to blank out the form
[x] Change label to be "Match all entered search terms below"
[x] Change "Keyword" input to NOT do exact phrase as it is now

@nessita I think you may have already handled a few of these if I'm not mistaken? Or am I wrong?

@frankwiles I only solved the last item which I'm now marking as Completed, the first 5 are still pending

Adjust advanced search button placement

Advanced search form should exist alone on page -- not sit above previous search results.

Move the search buttons!


feat: pr-compare/create design improvements

Description
This pull request brings design updates for pr compare/create page.
Before:

After:

Before:

After:

can we add all the text to the en/views.json file and all text in this pr to it so that means we have to also add the usetranslationstore and pass thru props @andrewgolovanov


Avoid using the zip FSP in SaveTransaction

The SaveTransaction class uses the forbidden (eclipse speak) class com.sun.nio.zipfs.ZipFileSystemProvider.
It should use FileSystems.newFileSystem() instead.
Pinging @flo for potential interest.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/35123792-avoid-using-the-zip-fsp-in-savetransaction?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github).

Interesting, I thought we had evicted everything com.sun long ago. I see we have a couple win32 usages too but maybe they have to work differently for OS specific stuff.
Reminds me of #1637


go/vt/sqlparser: Expose select substring in create view from the sqlparser.

@zachmu https://github.com/liquidata-inc/vitess/pull/9/commits/6b041e97c1d3bdcad5e441fe1d092f2949ab7360 Thoughts?


Text Extractor FREEDOM - Episode 2 "Don't overfeed the dog."

Description of the new feature / enhancement
Use four (4) spaces (a "standard" tab) for excessive spaces between words on the same line of text or contents in the same row of a table instead of linefeed.
Scenario when this would be used?
At this moment Text Extractor inserts linefeeds for every pair of words separated for more than two spaces - excessive linefeed - it is destroying the text or table structure.
Supporting information
Please, do NOT use the tab to implement this improvement.
Text Extractor is useful for renaming (non-OCR) PDF files with the proper title of the document or copying a link for a browser - a tab could cause some problems with these procedures.
Excessive linefeeds make the paste procedure in some fields like filenames in Windows Explorer or browser URL bars to disregard further lines of text.

Hello @AlexDeMoura Text Extractor uses the Windows OCR API to determine line breaks. The API is aggressive as you point out in separating words into lines. I have built a table mode in Text Grab and plan to bring that functionality into Text Extractor as well. I will close this issue and track that work with the Table Mode feature. If that does not solve the issue, please feel free to open a new issue at that time.
reference #20596


Fixed #33532 -- Special-cased dicts being given to CaseInsensitiveMapping

Ticket is 33532 on Trac.
Saves me roughly 0.5µs across a couple of expectedly common instantiations. Applies mainly to HttpHeaders rather than ResponseHeaders, as that massages it's data slightly differently and I'd look to target that as a separate patch which I've mostly done pre-black, so will need fixing ;)
Timings from ticket follow.
Before:
In [3]: %timeit CaseInsensitiveMapping({'Name': 'Jane'})
1.4 µs ± 7.94 ns per loop (mean ± std. dev. of 7 runs, 1,000,000 loops each)
In [4]: %timeit HttpHeaders({"HTTP_EXAMPLE": 1})
2.79 µs ± 20.1 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)
In [5]: %timeit HttpHeaders({"CONTENT_LENGTH": 1})
2.56 µs ± 40 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

After:
In [3]: %timeit CaseInsensitiveMapping({'Name': 'Jane'})
862 ns ± 9.4 ns per loop (mean ± std. dev. of 7 runs, 1,000,000 loops each)
In [4]: %timeit HttpHeaders({"HTTP_EXAMPLE": 1})
2.05 µs ± 31 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)
In [5]: %timeit HttpHeaders({"CONTENT_LENGTH": 1})
1.98 µs ± 18.1 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

Updated the patch, which now doesn't special-case in the __init__ as per the ticket, but gates it behind the _unpack_items, which I'd expected to be some of the overhead being saved, but it doesn't look like it substantially changes the change (and seems more likely to be acceptable as a patch):
In [3]: %timeit CaseInsensitiveMapping({'Name': 'Jane'})
847 ns ± 14.7 ns per loop (mean ± std. dev. of 7 runs, 1,000,000 loops each)
In [4]: %timeit HttpHeaders({"HTTP_EXAMPLE": 1})
2.06 µs ± 31.6 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)
In [5]: %timeit HttpHeaders({"CONTENT_LENGTH": 1})
2.01 µs ± 27.6 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

For background we recently made changes in bc4c7e5d68e01207268c868932eb14ae4206f02c to optimize stuff here and also consolidated all of the unpacking stuff in CaseInsensitiveMapping._unpack_items(). It would be nice if we could avoid moving logic for the dict special case back out into CaseInsensitiveMapping.__init__(). Also note that bb58d76117513b2af274cb030f67d454fff35218 isn't enough as ResponseHeaders.__init__() would also need to be changed, if we were to do this, as it doesn't call super().__init__(). Moving this logic out of _unpack_items() would be less DRY.
The intent of moving the dict.items() call into _unpack_items() was also to minimize iterations over the data. Previously in bc4c7e5d68e01207268c868932eb14ae4206f02c there were dictionary comprehensions in the __init__() methods for non-mapping iterables after which dict.items() had to be called again.

Turns out I either forgot, or just never knew that return in a yielding generator is largely the same as StopIteration and that they don't combine in the way I was banking on.

👍🏻

Yielding as in 539f82c removes much of the gains of the special-casing (costs around 300ns), so I've reverted back to the original form as the only one to consider or reject (I think...)

So, yes, setting up the generator adds some overhead, but, at least in the case of HttpHeaders, your test is atypical. In most cases there are going to be more than one header. So I tried this again with something more realistic for request.META.
In [1]: from django.http.request import HttpHeaders
   ...: from django.utils.datastructures import CaseInsensitiveMapping

In [2]: request_meta = {
   ...:     "SERVER_NAME": "localhost",
   ...:     "GATEWAY_INTERFACE": "CGI/1.1",
   ...:     "SERVER_PORT": "8888",
   ...:     "REMOTE_HOST": "",
   ...:     "CONTENT_LENGTH": "",
   ...:     "SCRIPT_NAME": "",
   ...:     "SERVER_PROTOCOL": "HTTP/1.1",
   ...:     "SERVER_SOFTWARE": "WSGIServer/0.2",
   ...:     "REQUEST_METHOD": "GET",
   ...:     "PATH_INFO": "/",
   ...:     "QUERY_STRING": "",
   ...:     "REMOTE_ADDR": "<IP_ADDRESS>",
   ...:     "CONTENT_TYPE": "text/plain",
   ...:     "HTTP_HOST": "<IP_ADDRESS>:8888",
   ...:     "HTTP_CONNECTION": "keep-alive",
   ...:     "HTTP_CACHE_CONTROL": "max-age=0",
   ...:     "HTTP_SEC_CH_UA": '" Not A;Brand";v="99", "Chromium";v="98"',
   ...:     "HTTP_SEC_CH_UA_MOBILE": "?0",
   ...:     "HTTP_SEC_CH_UA_PLATFORM": '"Linux"',
   ...:     "HTTP_UPGRADE_INSECURE_REQUESTS": "1",
   ...:     "HTTP_USER_AGENT": "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.80 Safari/537.36",
   ...:     "HTTP_ACCEPT": "text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9",
   ...:     "HTTP_SEC_FETCH_SITE": "none",
   ...:     "HTTP_SEC_FETCH_MODE": "navigate",
   ...:     "HTTP_SEC_FETCH_USER": "?1",
   ...:     "HTTP_SEC_FETCH_DEST": "document",
   ...:     "HTTP_ACCEPT_ENCODING": "gzip, deflate, br",
   ...:     "HTTP_ACCEPT_LANGUAGE": "en-GB,en-US;q=0.9,en;q=0.8",
   ...:     "HTTP_COOKIE": "csrftoken=ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+=",
   ...:     "wsgi.input": object(),
   ...:     "wsgi.errors": object(),
   ...:     "wsgi.version": (1, 0),
   ...:     "wsgi.run_once": False,
   ...:     "wsgi.url_scheme": "http",
   ...:     "wsgi.multithread": True,
   ...:     "wsgi.multiprocess": False,
   ...:     "wsgi.file_wrapper": object(),
   ...: }

659d2421c7adbbcd205604002d521d82d6b0b465 (main):
In [3]: %timeit HttpHeaders(request_meta)
13 µs ± 16.6 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

bb58d76117513b2af274cb030f67d454fff35218:
In [3]: %timeit HttpHeaders(request_meta)
12.3 µs ± 20.5 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

It still has an impact. You mentioned the overhead of comparing dict against Mapping going through _abc_instancecheck. We could just attempt to avoid that with the following:
diff --git a/django/utils/datastructures.py b/django/utils/datastructures.py
index b5858b8076..63cbaf743c 100644
--- a/django/utils/datastructures.py
+++ b/django/utils/datastructures.py
@@ -327,7 +327,8 @@ class CaseInsensitiveMapping(Mapping):

     @staticmethod
     def _unpack_items(data):
-        if isinstance(data, Mapping):
+        # Explicitly test for dict first as the common case for performance.
+        if isinstance(data, (dict, Mapping)):
             yield from data.items()
             return
         for i, elem in enumerate(data):

Which did give me the following:
In [6]: %timeit HttpHeaders(request_meta)
12.6 µs ± 30 ns per loop (mean ± std. dev. of 7 runs, 100,000 loops each)

But I can't promise this will be consistent. Sometimes it seemed to make no difference.
Being a micro-optimisation, it'd be interesting to know whether this actually makes a dent in the normal processing of a request as a whole before we go ahead with the change.

As an aside, I noticed that ResponseHeaders._convert_to_charset() could possibly be improved by reducing the number of calls to isinstance().  Perhaps something like:
diff --git a/django/http/response.py b/django/http/response.py
index e40f2d169b..a94519a872 100644
--- a/django/http/response.py
+++ b/django/http/response.py
@@ -42,16 +42,20 @@ class ResponseHeaders(CaseInsensitiveMapping):
         `charset` must be 'ascii' or 'latin-1'. If `mime_encode` is True and
         `value` can't be represented in the given charset, apply MIME-encoding.
         """
-        if not isinstance(value, (bytes, str)):
-            value = str(value)
-        if (isinstance(value, bytes) and (b"\n" in value or b"\r" in value)) or (
-            isinstance(value, str) and ("\n" in value or "\r" in value)
+        if isinstance(value, bytes):
+            type_ = bytes
+        else:
+            type_ = str
+            if not isinstance(value, str):
+                value = str(value)
+        if (type_ is bytes and (b"\n" in value or b"\r" in value)) or (
+            type_ is str and ("\n" in value or "\r" in value)
         ):
             raise BadHeaderError(
                 "Header values can't contain newlines (got %r)" % value
             )
         try:
-            if isinstance(value, str):
+            if type_ is str:
                 # Ensure string is valid in given charset
                 value.encode(charset)
             else:

I wonder whether we could also delay the check that results in BadHeaderError until after the .decode() to remove one branch. 🤔

[...] Being a micro-optimisation, it'd be interesting to know whether this actually makes a dent in the normal processing of a request as a whole before we go ahead with the change [...]

I don't think it makes much of a dent, really. It's ... micro. But that's the 1,000-papercuts route, so we only need to find 999 (I think I have 1, maybe 2...) more and we'll be golden, right? ;) If we can figure out how to keep the change gated behind _unpack_items and have our cake too, I don't see much reason not to do it.
Definitely appreciate you reminding me of the real-world data environ it would be inheriting, as yeah that may well change things and I'd got fixated on the small test cases because of playing with ResponseHeaders prior to this (which is possibly the ticket I should've started with ...). I'll definitely need to revisit the profile & timings based on your notes and thoughts.

As an aside, I noticed that ResponseHeaders._convert_to_charset() could possibly be improved by reducing the number of calls to isinstance() [...]

Oh Nick, you and I, same exact wavelength. I have a branch from a week or so back which implements basically exactly that. Altogether, it worked out well, seeming to save more than this patch does, but I haven't consolidated all the numbers to add to a ticket yet; and that's where the "small" dict instantiation I was using came from, being that by default a ResponseHeaders will inherit zero (or generally speaking, few) keys/values. It shouldn't have carried over to this patch because it's incorrect, as you pointed out, so my bad.

Hi Both -- I've run this on the Pi with timings below. Have I correctly understood this the profile/test you are assessing this patch by?
Main

In [3]: request_meta
Out[3]:
{'SERVER_NAME': 'localhost',
 'GATEWAY_INTERFACE': 'CGI/1.1',
 'SERVER_PORT': '8888',
 'REMOTE_HOST': '',
 'CONTENT_LENGTH': '',
 'SCRIPT_NAME': '',
 'SERVER_PROTOCOL': 'HTTP/1.1',
 'SERVER_SOFTWARE': 'WSGIServer/0.2',
 'REQUEST_METHOD': 'GET',
 'PATH_INFO': '/',
 'QUERY_STRING': '',
 'REMOTE_ADDR': '<IP_ADDRESS>',
 'CONTENT_TYPE': 'text/plain',
 'HTTP_HOST': '<IP_ADDRESS>:8888',
 'HTTP_CONNECTION': 'keep-alive',
 'HTTP_CACHE_CONTROL': 'max-age=0',
 'HTTP_SEC_CH_UA': '" Not A;Brand";v="99", "Chromium";v="98"',
 'HTTP_SEC_CH_UA_MOBILE': '?0',
 'HTTP_SEC_CH_UA_PLATFORM': '"Linux"',
 'HTTP_UPGRADE_INSECURE_REQUESTS': '1',
 'HTTP_USER_AGENT': 'Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.80 Safari/537.36',
 'HTTP_ACCEPT': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.9',
 'HTTP_SEC_FETCH_SITE': 'none',
 'HTTP_SEC_FETCH_MODE': 'navigate',
 'HTTP_SEC_FETCH_USER': '?1',
 'HTTP_SEC_FETCH_DEST': 'document',
 'HTTP_ACCEPT_ENCODING': 'gzip, deflate, br',
 'HTTP_ACCEPT_LANGUAGE': 'en-GB,en-US;q=0.9,en;q=0.8',
 'HTTP_COOKIE': 'csrftoken=ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+=',
 'wsgi.input': <object at 0xb3e36750>,
 'wsgi.errors': <object at 0xb3e36758>,
 'wsgi.version': (1, 0),
 'wsgi.run_once': False,
 'wsgi.url_scheme': 'http',
 'wsgi.multithread': True,
 'wsgi.multiprocess': False,
 'wsgi.file_wrapper': <object at 0xb3e36760>}

In [4]: %timeit HttpHeaders(request_meta)
81.4 µs ± 421 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

In [5]: %timeit HttpHeaders(request_meta)

80.8 µs ± 237 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

In [6]: %timeit HttpHeaders(request_meta)
83.5 µs ± 564 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

In [7]:

In [7]: %timeit HttpHeaders(request_meta)
82.4 µs ± 424 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

In [8]: %timeit HttpHeaders(request_meta)
84.1 µs ± 2.27 µs per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

6585bbab0eba5633f17a085271fb68351cdff340

In [3]: %timeit HttpHeaders(request_meta)
79.7 µs ± 314 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

In [4]: %timeit HttpHeaders(request_meta)
80.7 µs ± 431 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

In [5]: %timeit HttpHeaders(request_meta)
80 µs ± 894 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

In [6]: %timeit HttpHeaders(request_meta)
79.9 µs ± 251 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)

Ah wonderful. I didn't want to expressly summon you to do some gruntwork, but I'm very pleased you stopped by to try it. Cursory glance suggests yep, that's the test both Nick and I have used (subsequent to my using the wrong one 🤦).
I think I'd consider your results a positive ... the numbers are at least more stable and consistent afterwards, by the look of it, and where I was seeing a ~1µs over a ~25µs runtime, yours reports a difference of somewhere between 2-4µs if we're charitable, which I guess seems about right given the difference in overall timing (~80µs). I suppose the corollary to using a "low-end" device is that while it'd make a difference more pronounced, it'd also take longer and be more subject to other processes drawing on the CPU in a notable way.

OK, so I guess we're all happy with the current form of this patch. @kezabelle Am I right? I'm sometimes confused by your "nano(micro)second" discussions :smile:

My meandering verbiage, confusing? I'm shocked, shocked I tell you ;)
Yep, from my point of view this is the minimal patch to be evaluated. Whether it's worth it, I defer to you/Carlton, and Nick [having proposed this variant, and been in this area recently, and concocted a better benchmarking case scenario].
If it's unnecessary churn, that's fine :)


Allow configuration of ulist spacing

This is related to #798 if we get configuration for unordered lists would it be possible to also specify other rendering options? I find the line spacing in lists especially troublesome because they are really really big compared to some other content on the page (the way I have them configured). Currently I wouldn't know how to configure line_height for this. Could you implement some way to do something like:
ulist:
  line_height: $base_line_height

EDIT: Of course this would be wonderful for ordered lists as well.

This is already possible using the outline_list_item_spacing setting. See https://github.com/asciidoctor/asciidoctor-pdf/blob/master/docs/theming-guide.adoc#key-prefix-outline-list. That controls both ordered and unordered list items.

Ok got it now. I didn't know the term outline list. Works now. Thank you.

That's one of the reasons I plan on changing it. It's time to fix the
terminology ;)


remove dead code and update dependencies

The methods that were being added by implementing the Io trait seem to have been removed in tokio_io, so there's no need to implement that trait any more as far as I can tell.

Thanks for the PR!
You don't actually need to update the dependencies here as these are just dependency specifications, so they're already compatible with these newer releases of crates (the versions listed are in theory the minimum versions). The dead code here is also present for backwards compatibility, so we can't remove this just yet.
Regardless, though, thanks!


Create Music Media package

musicmedia_objects, musicmedia_routes, and musicmedia_forms should be in their own package

Should also add route_utilities and remove non-route code from musicmedia_routes into this new file


Android option?

Ist there a way to use it for Androud as well? At best parallel in Windows/Linux?
And THANKS for realising already such a great browser extension!! 🤩😍🤩

Hey @Improver2000,
Thanks for your comment, I'm happy to hear you are enjoying the extension 😁
Regarding Android support, as far as I know, efficiently modifying network requests on native apps (like on Android) is much more complex compared to a browser environment, where requests can be easily intercepted via an extension for a specific website. That’s why I haven’t prioritized it yet, though it’s definitely on my backlog for future exploration.
Right now, my main focus is addressing potential security issues, but I’d love to see contributions from the community if someone is interested in exploring mobile support.
Thanks again for your feedback and support!


[DM-36074] Fix template for S3

The secrets are not set correctly when S3 is used instead of GCS

Whoops, sorry, got S3 right in one place but not in another place. Thanks for the fix! I agree that S3 is better since it may not be actual AWS.


Failed to parse JSON file ~/.proto/temp/node/versions.json when running multiple tasks

What version?
0.14.1
Which command?
node
What happened?
It is similar to what I reported previously.
To reproduce this issue, use the same steps.

https://github.com/moonrepo/proto/issues/85

If you're reporting an issue for a specific language/tool and not for proto itself, please report the issue on their official repository.

Sorry but I don't know. I think this is issue for proto itself.
Any logs?
><EMAIL_ADDRESS>task
> run-p task:*

><EMAIL_ADDRESS>task:f
> node -e "console.log('f')"

Error: json::parse_file

  × Failed to parse JSON file ~/.proto/temp/node/versions.json.
  ╰─▶ EOF while parsing a value at line 1 column 0

Error: json::parse_file

  × Failed to parse JSON file ~/.proto/temp/node/versions.json.
  ╰─▶ EOF while parsing a value at line 1 column 0

Error: json::parse_file

  × Failed to parse JSON file ~/.proto/temp/node/versions.json.
  ╰─▶ EOF while parsing a value at line 1 column 0

Operating system?
MacOS
Architecture?
Arm64

I'll add some file locking around this file as well.

Thanks for the fix!


Mac OS 10.3.4 Drivers

any ideas when NVidia are releasing drivers to support 10.3.4?  I get the following echo.
Could not find a release for your OS.
The latest recommended release is:
3<IP_ADDRESS>.25.106

3<IP_ADDRESS>.30.103 is released now but not a good driver(at least for me). It feels laggy, no cuda support. I force installed 3<IP_ADDRESS>.25.106 and works well.

./nvidia-update.sh -f 3<IP_ADDRESS>.25.106

this is new cuda version
but not a good driver
this name：cudadriver_387.178_macos.dmg

3<IP_ADDRESS>.30.103 released today and still has problems.

Now that external GPU is released and anyone use the new drivers, they would get so disappointed.


⚠ Bofrost.at has degraded performance

In 770ddf4, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 808 ms

Resolved: Bofrost.at performance has improved in 37a6e2d after 7 minutes.