The QUIC causes the process to crash

envoy：v1.17.0

sorry can you give a bit more detail?  Were you running with one of the sample configs and standard traffic?

sorry can you give a bit more detail? Were you running with one of the sample configs and standard traffic?
Does v1.17.0 support QUIC？

An example config which accepts HTTP/1 requests over TCP and forwards them to google using HTTP/3
admin:
access_log_path: "/dev/null"
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 9901
static_resources:
listeners:

name: listener_0
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 10000
filter_chains:

filters:

name: envoy.filters.network.http_connection_manager
typed_config:
"@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
stat_prefix: ingress_http
route_config:
name: local_route
virtual_hosts:
- name: local_service
domains: ["*"]
routes:
- match:
prefix: "/"
route:

host_rewrite_literal: www.google.com
              cluster: service_google

cluster: inbound|80||
      http_filters:
      - name: envoy.filters.http.router

clusters:

name: inbound|80||
connect_timeout: 0.25s
type: strict_dns
lb_policy: round_robin
load_assignment:
cluster_name: service1
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:
address: <IP_ADDRESS>
port_value: 80

name: service_google
connect_timeout: 30s
type: LOGICAL_DNS
Comment out the following line to test on v6 networks
dns_lookup_family: V4_ONLY
lb_policy: ROUND_ROBIN
load_assignment:
cluster_name: service_google
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:

address: www.google.com
            address: <IP_ADDRESS>

port_value: 443
            port_value: 6121

typed_extension_protocol_options:
envoy.extensions.upstreams.http.v3.HttpProtocolOptions:
"@type": type.googleapis.com/envoy.extensions.upstreams.http.v3.HttpProtocolOptions
explicit_http_config:
http3_protocol_options: {}
common_http_protocol_options:
idle_timeout: 1s
transport_socket:
  name: envoy.transport_sockets.quic
  typed_config:
    "@type": type.googleapis.com/envoy.extensions.transport_sockets.quic.v3.QuicUpstreamTransport
    upstream_tls_context:

sni: www.google.com
      sni: <IP_ADDRESS>

client request： sudo curl <IP_ADDRESS>:10000/
curl: /usr/lib/x86_64-linux-gnu/libcurl.so.4: no version information available (required by curl)
curl: (52) Empty reply from server

`# An example config which accepts HTTP/1 requests over TCP and forwards them to google using HTTP/3
admin:
access_log_path: "/dev/null"
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 9901
static_resources:
listeners:

name: listener_0
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 10000
filter_chains:

filters:

name: envoy.filters.network.http_connection_manager
typed_config:
"@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
stat_prefix: ingress_http
route_config:
name: local_route
virtual_hosts:
- name: local_service
domains: ["*"]
routes:
- match:
prefix: "/"
route:

host_rewrite_literal: www.google.com
              cluster: service_google

cluster: inbound|80||
      http_filters:
      - name: envoy.filters.http.router

clusters:

name: inbound|80||
connect_timeout: 0.25s
type: strict_dns
lb_policy: round_robin
load_assignment:
cluster_name: service1
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:
address: <IP_ADDRESS>
port_value: 80

name: service_google
connect_timeout: 30s
type: LOGICAL_DNS
Comment out the following line to test on v6 networks
dns_lookup_family: V4_ONLY
lb_policy: ROUND_ROBIN
load_assignment:
cluster_name: service_google
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:

address: www.google.com
            address: <IP_ADDRESS>

port_value: 443
            port_value: 6121

typed_extension_protocol_options:
envoy.extensions.upstreams.http.v3.HttpProtocolOptions:
"@type": type.googleapis.com/envoy.extensions.upstreams.http.v3.HttpProtocolOptions
explicit_http_config:
http3_protocol_options: {}
common_http_protocol_options:
idle_timeout: 1s
transport_socket:
  name: envoy.transport_sockets.quic
  typed_config:
    "@type": type.googleapis.com/envoy.extensions.transport_sockets.quic.v3.QuicUpstreamTransport
    upstream_tls_context:

sni: www.google.com
      sni: <IP_ADDRESS>

`
client request：sudo curl <IP_ADDRESS>:10000/
curl: /usr/lib/x86_64-linux-gnu/libcurl.so.4: no version information available (required by curl)
curl: (52) Empty reply from server

**# An example config which accepts HTTP/1 requests over TCP and forwards them to google using HTTP/3
admin:
access_log_path: "/dev/null"
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 9901
static_resources:
listeners:

name: listener_0
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 10000
filter_chains:

filters:

name: envoy.filters.network.http_connection_manager
typed_config:
"@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
stat_prefix: ingress_http
route_config:
name: local_route
virtual_hosts:
- name: local_service
domains: ["*"]
routes:
- match:
prefix: "/"
route:

host_rewrite_literal: www.google.com
              cluster: service_google

cluster: inbound|80||
      http_filters:
      - name: envoy.filters.http.router

clusters:

name: inbound|80||
connect_timeout: 0.25s
type: strict_dns
lb_policy: round_robin
load_assignment:
cluster_name: service1
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:
address: <IP_ADDRESS>
port_value: 80

name: service_google
connect_timeout: 30s
type: LOGICAL_DNS
Comment out the following line to test on v6 networks
dns_lookup_family: V4_ONLY
lb_policy: ROUND_ROBIN
load_assignment:
cluster_name: service_google
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:

address: www.google.com
            address: <IP_ADDRESS>

port_value: 443
            port_value: 6121

typed_extension_protocol_options:
envoy.extensions.upstreams.http.v3.HttpProtocolOptions:
"@type": type.googleapis.com/envoy.extensions.upstreams.http.v3.HttpProtocolOptions
explicit_http_config:
http3_protocol_options: {}
common_http_protocol_options:
idle_timeout: 1s
transport_socket:
  name: envoy.transport_sockets.quic
  typed_config:
    "@type": type.googleapis.com/envoy.extensions.transport_sockets.quic.v3.QuicUpstreamTransport
    upstream_tls_context:

sni: www.google.com
      sni: <IP_ADDRESS>

**

`# An example config which accepts HTTP/1 requests over TCP and forwards them to google using HTTP/3
admin:
access_log_path: "/dev/null"
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 9901
static_resources:
listeners:

name: listener_0
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 10000
filter_chains:

filters:

name: envoy.filters.network.http_connection_manager
typed_config:
"@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
stat_prefix: ingress_http
route_config:
name: local_route
virtual_hosts:
- name: local_service
domains: ["*"]
routes:
- match:
prefix: "/"
route:

host_rewrite_literal: www.google.com
              cluster: service_google

cluster: inbound|80||
      http_filters:
      - name: envoy.filters.http.router

clusters:

name: inbound|80||
connect_timeout: 0.25s
type: strict_dns
lb_policy: round_robin
load_assignment:
cluster_name: service1
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:
address: <IP_ADDRESS>
port_value: 80

name: service_google
connect_timeout: 30s
type: LOGICAL_DNS
Comment out the following line to test on v6 networks
dns_lookup_family: V4_ONLY
lb_policy: ROUND_ROBIN
load_assignment:
cluster_name: service_google
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:

address: www.google.com
            address: <IP_ADDRESS>

port_value: 443
            port_value: 6121

typed_extension_protocol_options:
envoy.extensions.upstreams.http.v3.HttpProtocolOptions:
"@type": type.googleapis.com/envoy.extensions.upstreams.http.v3.HttpProtocolOptions
explicit_http_config:
http3_protocol_options: {}
common_http_protocol_options:
idle_timeout: 1s
transport_socket:
  name: envoy.transport_sockets.quic
  typed_config:
    "@type": type.googleapis.com/envoy.extensions.transport_sockets.quic.v3.QuicUpstreamTransport
    upstream_tls_context:

sni: www.google.com
      sni: <IP_ADDRESS>

`

An example config which accepts HTTP/1 requests over TCP and forwards them to google using HTTP/3
admin:
access_log_path: "/dev/null"
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 9901
static_resources:
listeners:

name: listener_0
address:
socket_address:
protocol: TCP
address: <IP_ADDRESS>
port_value: 10000
filter_chains:

filters:

name: envoy.filters.network.http_connection_manager
typed_config:
"@type": type.googleapis.com/envoy.extensions.filters.network.http_connection_manager.v3.HttpConnectionManager
stat_prefix: ingress_http
route_config:
name: local_route
virtual_hosts:
- name: local_service
domains: ["*"]
routes:
- match:
prefix: "/"
route:
cluster: service_google
http_filters:

name: envoy.filters.http.router
clusters:

name: inbound|80||
connect_timeout: 0.25s
type: strict_dns
lb_policy: round_robin
load_assignment:
cluster_name: service1
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:
address: <IP_ADDRESS>
port_value: 80

name: service_google
connect_timeout: 30s
type: LOGICAL_DNS
dns_lookup_family: V4_ONLY
lb_policy: ROUND_ROBIN
load_assignment:
cluster_name: service_google
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:
address: <IP_ADDRESS>
port_value: 6121
transport_socket:
name: envoy.transport_sockets.quic
typed_config:
"@type": type.googleapis.com/envoy.extensions.transport_sockets.quic.v3.QuicUpstreamTransport
upstream_tls_context:
sni: <IP_ADDRESS>

sorry can you give a bit more detail? Were you running with one of the sample configs and standard traffic?

client request：sudo curl <IP_ADDRESS>:10000/
curl: /usr/lib/x86_64-linux-gnu/libcurl.so.4: no version information available (required by curl)
curl: (52) Empty reply from server

I don't think QUIC went alpha until after the release was cut.  I can't reproduce at HEAD so please try with that.

I don't think QUIC went alpha until after the release was cut. I can't reproduce at HEAD so please try with that.

sorry. I don't understand what you're saying. I use envoy v1.17.0, Does v1.17.0 support QUIC？

It does not, sorry

It does not, sorry

Do I need to use the latest version?

It does not, sorry

Does v1.18.3 support QUIC？I failed to use Envoy v1.18.3.

If you want to use QUIC please build from head, or use the nighly builds.  There is currently no release with QUIC support.


Default value of "supportedSubmitMethods" in springfox.js

In using the supportedSubmitMethods field on UiConfiuguration, I have run into a problem where configuring the Jackson ObjectMapper to only include NON_EMPTY values breaks the desired functionality when using Constants.NO_SUBMIT_METHODS.  Since Constants.NO_SUBMIT_METHODS is an empty array, the field is omitted from the JSON response and the javascript defaults the value back to ['get', 'post', 'put', 'delete', 'patch'].  I feel that since supportedSubmitMethods is either defaulted to ['get', 'post', 'put', 'delete', 'patch'] or required in the constructors of UiConfiguration, having the javascript default to an empty array would cover this scenario without breaking the current functionality.

You could create a mixin to work around the problem

Thanks.


Track unregistered camper progress so if they sign up, they'll still have their progress. Stop asking unregistered campers to register

This is done.


Model training and status

Task
Implement feature to train model and get status of model last trained
Procedure

Comment below that you would like to work on this issue
Write clean code
Make a pr in separate branch

Happy Coding

I want to work on this issue @Swarnimashukla


PushNotification.init is not function (TypeError) - phonegap built

Expected Behaviour
When calling PushNotification.init({..}) i'm expect that to work based on the documentation.
Actual Behaviour
Getting an error that the function is undefined
Did some testing:
alert(typeof PushNotification.init);
alert(typeof PushNotification);
Results in:
"undefined"
"function" -> Note: PushNotication is defined. The init function is not defined though
Looks likes this issue: https://github.com/phonegap/phonegap-plugin-push/issues/1838. I also tried alerting the same tests with timeout of 10 seconds, everyting completely loaded for sure. results in the same problem
I'm also noticing that no Push permission is asked when installing the app. Maybe that is related?
Note: push.js is distributed in the APK. I've unzipped the APK and found the push.js.
Reproduce Scenario (including but not limited to)
Because it's a phonegap built with Mendix back-end it's not so easy to provide the reproduce steps here.
The phonegap built log is cleary stating that it has built the Phonegap-push-plugin
Total time: 1.232 secs
Fetching plugin<EMAIL_ADDRESS>via npm
Installing "phonegap-plugin-push" at "1.9.0" for android
Subproject Path: CordovaLib
Configuration on demand is an incubating feature.
Incremental java compilation is an incubating feature.
:clean
:CordovaLib:clean
BUILD SUCCESSFUL
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
PhoneGap (iOS / Android / Windows)
cli-6.4.0 (4.3.0 / 6.1.0 / 4.4.3)
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Galaxy S7, Android
Plugin version
1.9.0
Used it this way. I also tried 1.10.5 but did go back to 1.9 because that's mentioned in the documentation as a working combination with 6.4.0.
 <plugin name="phonegap-plugin-push" source="npm" spec="1.9.0" >
		<param name="SENDER_ID" value="xxx" />
	</plugin>

Also have this in my config.xml:
        <preference name="android-build-tool" value="gradle" />

@herbertvuijk your build log shows it fetching the push plugin but there is not enough there to show it was included properly. What is a "Mendix back-end"? Is that an on-line build tool for Cordova apps?

Hi Macdonst,
Thanks. If I unzip the APK I see this in cordova_plugins.js, among other plugins:
{
"id": "phonegap-plugin-push.PushNotification",
"file": "plugins/phonegap-plugin-push/www/push.js",
"pluginId": "phonegap-plugin-push",
"clobbers": [
"PushNotification"
]
},
Also the assets\www\plugins\phonegap-plugin-push\www\push.js is there and filled.
Mendix is a application platform that runs the wep-app that is referenced from the Cordova app. I can provide the phonegap built zip and APK if that is of any value?

@herbertvuijk well if the Push Plugin is included I have to assume you are trying to call the init method before you get the deviceready event or from a page in which the plugin is not loaded. If you post up the zip file somewhere I will take a quick look.

Hi Simon,
Thanks. Figured it out. There was a very old version of some push plugin included as well, which was overwriting the PushNotification object / prototype. Removed it, and now it worked.


Version 2.1.2 has a new bug

after you click to open the launcher from the taskbar and minimize it, you won't be able to open ti any more.

fixed


Documentation [v1.1.4]

please add documentation and commit comments (changelog) for the latest package version.

added a .caught() method to catch promises in the console, but not throw them

added a .caught() method to catch promises in the console, but not throw them


Error: Islands must be larger than the quorum.

I'm trying to use the SKATER model, and continue to run into issues with the ValueError:
Islands must be larger than the quorum. If not, drop the small islands and solve for clusters in the remaining field.
I've been following the examples provided on their website with no success. I'm using spopt 0.6.1.
sf_kwds = dict(
    dissimilarity=skm.manhattan_distances,
    affinity=None,
    reduction=np.sum,
    center=np.mean
)

model = Skater(
    gdf = cluster_zips_scaled,
    w = weights,
    attrs_name = ['counter'],
    n_clusters = 6, # k_optimal
    floor = 98,
    trace = False,
    islands = "increase",
    spanning_forest_kwds = sf_kwds)
model.solve()

cluster_zips_scaled[f"skater{k_optimal}_label"] = model.labels_

My weights dictionary has no islands, and there are 636 geometries I'm trying to cluster into 6 clusters, and I want at least 98 geometries in each cluster. Given the weights, all geometries border at least one other geometry.
Any help would be appreciated.

sorry, it's really hard to diagnose without a peek at your data/weights. Any chance you can provide a reprex?

The error is somewhat misleading due to our change in terminology. The error should be warning about disconnected components, not islands.
Are there disconnected subgraphs in weights that have more than 98 observations? You can check the weights.connected_components_, and can dig into it further using scipy.sparse.csgraph.connected_components() on your weights.sparse to investigate.


Move todos to issues.

https://github.com/aaraney/PBenJ/blob/53f4ae3c5c364a8e972d05a9903a4126033b4d43/framework_dev/perturb_py_dev/Parameter_editor.py#L69
It's going to be easier to keep track of things that need to get done if you track them in issues and close them with commits.

Agreed.  Although I sometime do this it is more of a reminder to submit these as issues as soon as I can.


NYT train split size

Hi,
thank you for your valuable work!
I was wondering how come the dataset_triples_train.json file contains 570,084 examples, while in the paper it is indicated 455,771?
Best,
Andrei

Hi,
Apologies for the late reply and thanks for your interest in our work.
The number stated in the paper is the number of sentences while the entities/triples are extracted from the sentence, which may contain more than one triple. So the number of triples could be different.
Hope this helps anyone interested, with similar queries in the future.
Many Thanks


Add margins to print page right and bottom

https://github.com/TechSmith/SnagitLiteWin/issues/537
The license agreement will now fit on the printed page.

👍


Update ffmpeg.py

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

I created a yt-dlp GUI and I use ffmpeg to convert to audio often, recently I used pyinstaller to make a .exe for this GUI because I want to send it to friends who don't have python in their computers, also I don't like the console that appears with python so I just put the configuration to remove the console on pyinstaller, the program works fine, until it has to convert anything with ffmpeg, then because ffmpeg requires a console to open it the program crashes.

All of this is on windows, I didn't test on linux or on mac
This problem only happend to me when I compiled the program to .exe
I tried this solution and it worked perfectly

Fixes #
Added 'creationflags=subprocess.CREATE_NO_WINDOW' so it doesn't try to create a terminal window, this fixes the crashing.
Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[x] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Copilot Summary

🤖 Generated by Copilot at 9269931
Summary
🐛🪟🎞

Fix unwanted windows bug on Windows by adding an argument to suppress console windows in ffmpeg.py postprocessor.

When using the ffmpeg postprocessor
Some windows would pop up and bother
So a new argument
Was added and sent
To suppress them with console_window_handler

Walkthrough

Prevent console window from popping up on Windows when running ffprobe or ffmpeg by adding creationflags=subprocess.CREATE_NO_WINDOW argument to Popen.run calls in ffmpeg.py (link, link, link)

Pls provide a minimal example code that causes the issue.
This should already be handling the console
https://github.com/yt-dlp/yt-dlp/blob/95abea9a03289da1384e5bda3d590223ccc0a238/yt_dlp/utils/_utils.py#L799-L803

it should but it doesn't for some reason, you need to add the argument (creationflags=subprocess.CREATE_NO_WINDOW)
Popen.run(cmd, text=True, stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
I tested the code multiple times and I can tell you that it doesn't work, by the way I am using windows 10 I'm not sure if it changes anything.

Can you share a code example that can be used to reproduce the issue (without your patch), though?

Can you share a code example that can be used to reproduce the issue (without your patch), though?

I'm not sure what you mean I thought you guys could see my changes on my commit, but I will try to put here:
1
https://github.com/yt-dlp/yt-dlp/blob/master/yt_dlp/postprocessor/ffmpeg.py#L253-L254
2
https://github.com/yt-dlp/yt-dlp/blob/master/yt_dlp/postprocessor/ffmpeg.py#L295
3
https://github.com/yt-dlp/yt-dlp/blob/master/yt_dlp/postprocessor/ffmpeg.py#L363-L364
if this is not enough I'm happy to record a video of the issue happening if you guys need it.

We know the yt-dlp code. We need a minimal reproducible example of code from e.g. from your GUI / external script that can reproduce the issue that you described

We know the yt-dlp code. We need a minimal reproducible example of code e.g. from your GUI / external script that can reproduce the issue that you described

I created some code, but I can't reproduce the issue on this code, I presume there's probably something wrong with my program's code then :/

No reproduction given, closing


Draft: Feature/pitasc skill interface

Unrealeased
Bug Fixes

fix setting headless mode (8618f56)
fix setting headless mode (2) (c71f1c3)
use Options instead of Profile (d186415)

Build

compile as C++17 (df778a6)

Chore

clean up package.xml and CMakeLists (ed052a1)
remove unused imports (baec4c5)

Code Refactoring

make browser interaction more pythonic (79867e8)
use with statements in file IO (6a4bc05)

Features

implement pitasc script and skill interface (bf6a662)

Style

apply format black (6f5aec1)
sort imports (503cf07)

The code should be tested with the hardware at least before merging.


Include minio and maildev for local development

These changes should make it easier for contributors to quickly spin up a development environment.
Most changes involved simply adding minio and maildev to the docker-compose.yml file.

minio: This is used for S3 storage. The docker-compose file has things set up conveniently since it creates all the buckets without the user having to do anything.
maildev: This is used for emails, any outgoing emails will be captured through this.

Another minor change was to introduce S3_FORCE_PATH_STYLE, this is needed since if we were to run minio locally, the URL would end up following a path style (e.g. localhost:9000/bucket/key) rather than a virtual host style (e.g. bucket.s3.whatever.com/key).
It's set to false by default (that's what you guys would use in production), but I let it be true in the example env, since that's intended to be used for development.
Finally, I of course updated the readme and mentioned a few things that weren't already documented and also updated the example env. I set default values in the env to match those used in the docker-compose file, so it's easier to work with.
I have tested these changes myself twice, once on my main machine and another time on another machine with a direct clone from my fork.  I've taken some screenshots that showcase the results, in case there's any concerns whether this actually works.
Though do note that in the gallery I sent above, I had to mock cloudflare to get images working (these changes were obviously not committed). So while these changes hopefully are in the right direction, there's still a small barrier to running this "fully" locally, and that would be cloudflare.
But in any case, at least with this, all one would need to do is follow the instructions and have cloudflare credentials (or be mad enough to mock cloudflare themselves)

Thank you so much for taking the time to make this more accessible for the community to pick up and run! 🎉
@koenbeuk Can you please review the docker changes here to ensure there won't be any conflicts with our CD pipeline?

Sweet!! This looks very promising 🔥 there shouldn't be any conflicts on the docker-compose file with our CD pipeline, if I'm not mistaken we use a different config


Add missing include in amalgamated build

This is used in ArrowCharView but the include isn't there.

Codecov Report

Merging #35 (6438a35) into main (04cb604) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #35   +/-   ##
=======================================
  Coverage   91.88%   91.88%
=======================================
  Files           6        6
  Lines        1306     1306
  Branches       47       47
=======================================
  Hits         1200     1200
  Misses         74       74
  Partials       32       32

Impacted Files
Coverage Δ

src/nanoarrow/nanoarrow_types.h
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove deprecated locale service provider

The "COMPAT" locale service provider is deprecated in recent Java versions and
including it in the providers list causes the server to spit out a warning message
when it starts up. The CLDR provider has everything we need anyway, so remove the
COMPAT provider from the providers list.

#2357  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sgrimm and the rest of your teammates on  Graphite


0.1.8 seems to be very different from 0.1.7

Hey there, we are using getIssues from this library for validating our geojsons, however after bumping from 0.1.7 to 0.1.8 we have many unit tests breaking as it seems to behave very differently - do you have an upgrade guide we can follow along?

Hi! same issue here. My tests are failing when I check a geojson geometry, which is a valid use case according to the tests. I have trace the problem to the typescript compilation.
If I replace in the types.ts file
export const GEOJSON_TYPES = new Set<GeoJSON['type']>([
  ...GEOJSON_GEOMETRY_TYPES,
  'Feature',
  'FeatureCollection',
]);

for
export const GEOJSON_TYPES = new Set<GeoJSON['type']>([
  'Point',
  'MultiPoint',
  'Polygon',
  'MultiPolygon',
  'LineString',
  'MultiLineString',
  'GeometryCollection',
  'Feature',
  'FeatureCollection',
]);

works perfectly well.
The weird thing is that the tests are defined in TS and seems that aren't affected by this issue.

Hi all, I ran into this issue when using knex-postgis a while back. Solved it by forcing the transitive check-geojson dependency to 0.1.7, but also happened to stumble upon the root cause. As hinted above, it's related to use of spread operator on sets and transpilation - ES6 transpilation does not handle said case well, as is discussed f.ex. here. The javascript target will look something like this
p=new Set(["Point","MultiPoint","Polygon","MultiPolygon","LineString","MultiLineString","GeometryCollection"]);
m=new Set([].concat(p,["Feature","FeatureCollection"]));

which is evidently wrong.
A naive fix is to define the union as
export const GEOJSON_TYPES = new Set<GeoJSON['type']>([
  ...Array.from(GEOJSON_GEOMETRY_TYPES),
  'Feature',
  'FeatureCollection',
]);

which produces the proper flat set in the transpiled javascript.

Sorry, my notification settings on this repo were borked and I didn't get any notification for this issue.
From what I can tell, this is caused by an upstream transpiler transpiling this module but not handling the 'downlevel iteration' feature. I'll make this code more vanilla in the next release.

OK I can confirm that 0.1.9 is passing all tests again 👍🏻


How to Remove alert validation in JQuery Multifile

I am using jquery multifile upload plugin http://www.fyneworks.com/jquery/multifile can able to select document but when i tried to select wrong file it shows validation message as alert
"   You cannot select a .mp4 file. Try again...  "
instead of alert message how to show as validation text message ? also any possiblities to add "Required" validation for file using this plugin.

You can override the default error function with your own function like this...
$(...).MultiFile({   error: function(error_string){     console.error('an error happened:' + error_string);   } })
and you can disable it completely like this:
$(...).MultiFile({   error: false })


Fix links

Why:
Website was linking to 404 pages.
How:
Change links to absolute links rather than relative links since the website has a different root than the Readme file.

awesome. thanks so much for your help


Push the ballerina OracleDB module to ballerina staging central

Description:
Push the ballerina OracleDB module to ballerina staging central then pull the module from it and run the tests to check the module.
Describe your task(s)
Related Issues (optional):
https://github.com/ballerina-platform/ballerina-standard-library/issues/1894

pushed the oracledb module release to staging central and then pulled back from the the staging. Then perform a  simple test the with pulled module.

Closing the issue as push and pull scenarios were tested for staging site.


Redact consent forms

Closes #357.

Coverage remained the same at 34.543% when pulling 8ac960cd5daf3dcde1657a3ff575e65f9fa619a6 on suchow-consent into dcba26f14ed4c7736c4fce2f18e08a2fe12ada16 on master.


Read The Room - Forced first person

The equip/unequip animations from the mod Read The Room forces first person whilst the scripted event is toggled.
The strange issue that it didn't start that way but started after a few hours of playtime. Initially you stay in third person if the animation is activated from third person.
Has happened to me on two different saves without updating either mod.
I realize turn off for scripted events but then I might miss functionality for other mods.

This happens if you are the closest zoom level in third person, if you zoom out by one click it will play in third person instead. I haven't found a suitable solution to this as of yet, although on my radar.


autocomplete display

@LuckyFBB  update to latest version monaco-editor(0.50.0) and monaco-sql-languages(0.12.2), that helped me

@LuckyFBB update to latest version monaco-editor(0.50.0) and monaco-sql-languages(0.12.2), that helped me

Thanks for your reply, I will try it


[📑 Docs]: Adding logo to storybook page

What Dev Docs changes are you proposing?
We can try adding AsyncAPI's logo and title text in the storybook header instead of the default one, steps for doing the same has been explained here
Right now, the header has the default logo + title text

Code of Conduct

[x] I agree to follow this project's Code of Conduct

Agreed! Let's do it. The logo is in Figma in Visual Design > Brand > Logo
You should be able to export it from there, but let me know if you have trouble 😄

I have added the SVG image in the Navbar and pushed the changes in the same readme-sb PR.


Implement push notifications to announce a new pending operation

For the purpose of the mobile token, we need to be able to send push notifications to the mobile device. The push notifications should contain operationId and operationData in the payload.

Already implemented.


Fix settings bug

fix #942, see https://github.com/meilisearch/MeiliSearch/issues/942#issuecomment-706266440

bors try

rename indexed_map into field_positions
rename insert_and_index into register_field

bors r+


[spi_host] Allow SPI host tests to work with multiple flash chips

Description
The spi_host_winbond_flash_test checks the manufacturer and device IDs of a Winbond flash chip connected to the SPI host.
The device ID is hardcoded but the test should work with many kinds of flashes.
With the new CW340 FPGAs added to CI, we will need to add more flash chips. The kind we have are still manufactured by Winbond but have different IDs. We would like to remove the explicit device ID check from this test to allow it to work with our second Winbond flash part.
Opening this issue since it needs to wait until after the current quiet period.

One way to do this would be to rely on the SFDP to configure the test (but report the JEDEC ID in the log, so you can identify the chip for failures).
General support for all the SFDP fields can be a semi-significant undertaking, though.

One way to do this would be to rely on the SFDP to configure the test (but report the JEDEC ID in the log, so you can identify the chip for failures).

Do you mean that the device ID is also in the SFDP and we can compare the JEDEC ID to that instead of our hardcoded value?

One way to do this would be to rely on the SFDP to configure the test (but report the JEDEC ID in the log, so you can identify the chip for failures).

Do you mean that the device ID is also in the SFDP and we can compare the JEDEC ID to that instead of our hardcoded value?

No, I mean that the SFDP tells us how to drive the SPI flash chip (e.g. what the command opcodes are to do various things, whether a chip supports quad mode, how to enable, etc.).
A table of known JEDEC IDs isn't particularly scalable, so it might not make sense to check it. Instead, we'd know it should fail if you get all zeroes or all ones. We do get the flash memory density from the SFDP, though, so maybe we could check one byte of the JEDEC ID.

No, I mean that the SFDP tells us how to drive the SPI flash chip (e.g. what the command opcodes are to do various things, whether a chip supports quad mode, how to enable, etc.).

@AlexJones0 and I checked the SFDP tables and it looks like you can get the op-code for quad reads but not writes from the standard tables. Our sw/device/tests{/pmod}/spi_host_*_flash_tests hard code these IDs, but reading them from the SFDP tables requires knowledge about the manufacturer-specific tables.
We could embed the tables for a few manufacturers and read the op-codes, but this doesn't help us much.
I agree we should replace the device_id check with a check for all-0 or all-f but we still need to check the manufacturer ID I think.

We do get the flash memory density from the SFDP, though, so maybe we could check one byte of the JEDEC ID.

I'm not sure what you mean by this, sorry.

We do get the flash memory density from the SFDP, though, so maybe we could check one byte of the JEDEC ID.

I'm not sure what you mean by this, sorry.

The command to read the JEDEC ID is a tuple of 3 values: (manufacturer ID, memory type, memory capacity). Potentially, that last byte could be checked with the info from the SFDP.

Thanks, that makes sense.
(Closing was a mis-click)


Naming/BlockForwarding and Performance/BlockGivenWithExplicitBlock combined safe autocorrect results in invalid code

When code violates both Naming/BlockForwarding and Performance/BlockGivenWithExplicitBlock, performing a safe autocorrect on that code can result in incorrect code.

Expected behavior
When running rubocop -a foo.rb where foo.rb is the following file:
# frozen_string_literal: true

def bar
  puts 'bar!'
  yield
end

def foo(&block)
  if block_given?
    bar(&block)
  else
    puts 'no block'
  end
end

foo do
  puts 'the block'
end

foo

And where your .rubocop.yml is as follows:
require:
  - rubocop-performance

AllCops:
  NewCops: enable
  TargetRubyVersion: 3.2

I expect either the Naming/BlockForwarding or the Performance/BlockGivenWithExplicitBlock safe autocorrect to be applied, but not both.
Actual behavior
It safe autocorrects to:
# frozen_string_literal: true

def bar
  puts 'bar!'
  yield
end

def foo(&)
  if block
    bar(&)
  else
    puts 'no block'
  end
end

foo do
  puts 'the block'
end

foo

If you try running this code using ruby foo.rb, it will fail.  The Performance/BlockGivenWithExplicitBlock cop safe autocorrect was applied, so block_given? is changed to just block.  The Naming/BlockForwarding cop safe autocorrect was applied, so usage of &block is changed to just &.  This means that where we are referencing block, we are referencing an undefined variable.  When you run the code you encounter the following error:
foo.rb:9:in `foo': undefined local variable or method `block' for main:Object (NameError)

  if block
     ^^^^^
	from foo.rb:16:in `<main>'

Steps to reproduce the problem

Set your Ruby version to 3.2.
Create an empty directory.
Put the above foo.rb and .rubocop.yml files into that directory.
Run ruby foo.rb, and observe that it runs without failing.
Run rubocop -a foo.rb.
Run ruby foo.rb, and observe that it fails.

RuboCop version
> rubocop -V
1.60.2 (using Parser <IP_ADDRESS>, rubocop-ast 1.30.0, running on ruby 3.2.2) [arm64-darwin22]
  - rubocop-performance 1.20.2

This seems important. Thanks for submitting a PR to fix it!
https://github.com/rubocop/rubocop-performance/pull/445


Added editor.urlClickable setting to enable/disable clickable URL

Issue: #27632

@chewong,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@chewong Thank you for your contribution :heart:


feat: Adding builder script

What’s the focus of this PR
This PR adds a helper script triggered on each pre-commit. This script builds the HTML dynamically according to the contents of the folders in the repo so there is no need to add your pill manually anymore to the index.html
How to review this PR
Check the script and the use of Husky for triggering the build of the index.html and adding it to the files to be committed (ideally, the HTML build would be done in the CI but as this project has no CI we could stick to this approach)
EDIT: I've just seen @Girgetto created another PR for solving this in a different way, feel free to reject this, combine it somehow or whatever.

I would like to take count and discuss some of those changes because I was undecided to do some of those that are in this PR maybe we can chat about this, and make a marge with what I've implemented in #37, is it ok for you @danielherrerohernando ?

I would like to take count and discuss some of those changes because I was undecided to do some of those that are in this PR maybe we can chat about this, and make a marge with what I've implemented in #37, is it ok for you @danielherrerohernando ?

Sure it is! I'll reach you on Slack @Girgetto


Update scalajs-dom to 2.7.0

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

🔍 Files still referring to the old version number
The following files still refer to the old version number (0.9.8).
You might want to review and update them manually.
html/src/main/scala/org/lrng/binding/AttributeFactories.scala

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

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

labels: library-update, early-semver-major, semver-spec-major, version-scheme:early-semver, old-version-remains, commit-count:1

Superseded by #250.


how to implement log_store interface?

I use sqlite3 to store log entires,but I do not understand how to implement log_store interface,include issues:

how compact function work?  inmem_log_store.h  implement it by remove data from map, I implement:

private:
    static ptr<log_entry> make_clone(const ptr<log_entry> &entry);

    // Dummy entry for index 0.
    ptr<log_entry> dummy_entry_;
    db_entry_helper entry_helper; //操作数据库
    std::atomic<ulong> start_idx_; //the start log id
    std::atomic<ulong> last_idx_;  //the last log id

bool indb_log_store::compact(ulong last_log_index) {
    info("compact log from [%u] to [%u], cur last_log_index:[%d]", start_idx_.load(), last_log_index, last_idx_.load());

    entry_helper.removeRange("id", start_idx_, last_log_index);

    // WARNING:
    //   Even though nothing has been erased,
    //   we should set `start_idx_` to new index.
    start_idx_ = last_log_index + 1;
    auto last = entry_helper.selectLast("id");
    if (!last->empty()) {
        last_idx_ = last->at(0).id_;
    } else {
        last_idx_ = last_log_index + 1;
    }
    info("start_idx:[%d], last_indx[%d]", start_idx_.load(), last_idx_.load());
}

The operation log is as follows

方日志 3号日志 leader log
2019-11-12T18:51:13.792_873+08:00 [c313] [INFO] trying to sync snapshot with last index 15 to peer 1, its last log idx 9	[handle_snapshot_sync.cxx:111, create_sync_snapshot_req()]
2019-11-12T18:51:13.794_468+08:00 [7067] [WARN] peer 1 declined snapshot: p->get_next_log_idx(): 10, log_store_->next_slot(): 17	[handle_snapshot_sync.cxx:317, handle_install_snapshot_resp()]
2019-11-12T18:51:13.795_047+08:00 [a3ce] [WARN] declined append: peer 1, prev next log idx 12, resp next 12, new next log idx 11	[handle_append_entries.cxx:676, handle_append_entries_resp()]
2019-11-12T18:51:13.795_317+08:00 [a3ce] [ERRO] bad log_idx 10 for retrieving the term value, will ignore this log req	[raft_server.cxx:1063, term_for_log()]
2019-11-12T18:51:13.795_322+08:00 [a3ce] [ERRO] last snapshot 0x7ff0ec003fe0, log_idx 10, snapshot last_log_idx 15	[raft_server.cxx:1066, term_for_log()]
2019-11-12T18:51:13.795_325+08:00 [a3ce] [ERRO] log_store_->start_index() 11	[raft_server.cxx:1068, term_for_log()]
2019-11-12T18:51:13.795_412+08:00 [c313] [WARN] declined append: peer 1, prev next log idx 11, resp next 12, new next log idx 10

//发送方日志 1号日志 fflower logs
2019-11-12T18:51:13.814_681+08:00 [42d1] [WARN] [LOG XX] req log idx: 11, req log term: 1, my last log idx: 11, my log (11) term: 0	[handle_append_entries.cxx:391, handle_append_entries()]
2019-11-12T18:51:13.814_683+08:00 [42d1] [WARN] deny, req term 1, my term 1, req log idx 11, my log idx 11	[handle_append_entries.cxx:398, handle_append_entries()]
2019-11-12T18:51:13.814_685+08:00 [42d1] [WARN] snp idx 15 term 1	[handle_append_entries.cxx:402, handle_append_entries()]
2019-11-12T18:51:13.814_845+08:00 [8b66] [ERRO] bad log_idx 10 for retrieving the term value, will ignore this log req	[raft_server.cxx:1063, term_for_log()]
2019-11-12T18:51:13.814_848+08:00 [8b66] [ERRO] last snapshot 0x7f213c0013c0, log_idx 10, snapshot last_log_idx 15	[raft_server.cxx:1066, term_for_log()]
2019-11-12T18:51:13.814_850+08:00 [8b66] [ERRO] log_store_->start_index() 16	[raft_server.cxx:1068, term_for_log()]

My trouble

in compact function , how to update  start_idx_ and last_idx_
in compact function, Whether or not to remove log entries from db

Hi @cklxiaocui
The semantic of compact is to remove all logs up to given last_log_index (including itself), so your implementation looks ok.
Isn't it because of race condition between calling removeRange and updating last_idx_? If you increase log level to 5 and get more detailed logs, that will be helpful. Thanks.

+) Also you should return true once compaction is done correctly.

thanks  a lot,  the log_store interface be called by asio In multithreaded mode，There are many boundary values to deal with，I need to see the source code of nuraft .


Enable dimension for Azure Monitor auto-scale rule

Resource Provider

Microsoft.Insights/autoscaleSettings
Description of Feature or Work Requested

https://github.com/Azure/azure-rest-api-specs/pull/9945
Minimum API Version Required

2015-04-01
Swagger Link

https://github.com/Azure/azure-rest-api-specs/pull/9945
Target Date

TBD

Copy @MyronFanQiu for awareness.

add to S173


set default values

Hi,
I try run the camera.py with the arguments discribed in the docs , but fail because inside the code dont have values for args.demo_size and img.copy too
Whats the default values for set these variables?
Thank you

It depends on the gpu. if using a laptop gpu, I usually use 320 as demo_size


OSGify Modules

[Migrated from Assembla ticket 26 - reported by @reid-spencer on 2014-11-15 21:00:16.000]
Instead of having Modules be global singletons, make them loadable dynamically via OSGi.
The goals of this should be:

Drop in a jar file to a directory and automatically see it's module(s) show up in the admin interface
Handle missing jar files gracefully by disabling/removing everything that depended on it from the db
Deal with inter-module versioning and dependency
Validate that modules are both internally consistent (signed?) and can integrate into the running scrupal instance (dependencies are met)

Rather than bring the heavyweight OSGi into the mix, we will wait for JSR 376 (Java Module System): https://jcp.org/en/jsr/detail?id=376


Transactions status (#519)

@marcinbodnar Status is looking good! Re: the loading animation, let's use a similar one to this (using for Recovery Options loading): https://github.com/nearprotocol/near-wallet/pull/381/commits/b6b689f8b5126897f1f1cfc8bb2066478ba39042

Re: the loading animation, let's use a similar one to this (using for Recovery Options loading): b6b689f

@kcole16 I'm afraid will not look good, because we are reloading this content every 10s. The idea with dots is that the content is not blinking every reload, just values are changing - it's smooth and discreet. Also, I feel this background animation works for small spaces only.
Also noticed, that going directly to profile doesn't work:
https://wallet.nearprotocol.com/profile/marcink56.test

@kcole16 I'm afraid will not look good, because we are reloading this content every 10s.

Ah, missed the polling. Ok, that makes sense.

Also noticed, that going directly to profile doesn't work:

Yes, looks like this might have to do with the .test suffix. Notice how https://wallet.nearprotocol.com/profile/emailagain should work. Will open a separate issue.

Yes, looks like this might have to do with the .test suffix. Notice how

Yes, it was working fine before.

@kcole16

Yes, looks like this might have to do with the .test suffix. Notice how https://wallet.nearprotocol.com/profile/emailagain should work. Will open a separate issue.

I've noticed it before and AFAIK it's caused by how URL rewrites work on Render. We can workaround by running own HTTP server vs relying on whatever they provide for free, but they also promised to fix it (just will take some time).

need to figure out with @frol how to get rid of N+1 query problem

I am sorry to say this, but there is no way to solve this at the moment (see my comment above). Explorer operates the same way at the moment.


Enhance login

Codecov Report

Merging #1873 into canary will increase coverage by 0.03%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           canary   #1873      +/-   ##
=========================================
+ Coverage    6.35%   6.38%   +0.03%
=========================================
  Files          93      93
  Lines        4062    4041      -21
=========================================
  Hits          258     258
+ Misses       3804    3783      -21

Impacted Files
Coverage Δ

src/commands/login.js
0% <0%> (ø)
: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 eb2fac7...2945d7c. Read the comment docs.


WIP: add tests and linting

Add linting and tests. WIP
Also closes #73

@mtxr do you still have this on your radar? :)

I do. But I’ve been pretty busy, sorry guys
On Wed, 3 Mar 2021 at 16:43 Jonas Thelemann<EMAIL_ADDRESS>wrote:

@mtxr https://github.com/mtxr do you still have this on your radar? :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jsappme/node-binance-trader/pull/74#issuecomment-790005899,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFMX2MW243C22UP4EGFZ63TB2GONANCNFSM4X72JXQQ
.

--
Matheus
https://mteixeira.dev

Alright, no problem :)

This is now superseded by #111.


[ENHANCEMENT] collect containerd.log in support-bundle-kit

Is your enhancement related to a problem? Please describe.

collect containerd.log in support-bundle-kit
Describe the solution you'd like

the harvester collector in support-bundle-kit needs to be enhanced to also collect containerd.log file.
At times this can be useful to troubleshoot issues with container runtime.
It is available at: /var/lib/rancher/rke2/agent/containerd/containerd.log
Describe alternatives you've considered

Additional context

Note:
Backport the fix to v1.2.1 and v1.1.3

@Yu-Jack I think we can resolve this once we have the correct SBK in 1.2

@bk201  Yeah, it's v0.0.28 now in v1.2. https://github.com/harvester/harvester/blob/d66812e66867f62f1f702d636f7e53d835d6456b/deploy/charts/harvester/values.yaml#L482-L486
I'll open a PR back to v1.2.

Provide test result on v1.2.2-dev-20240329, do have containerd.log under nodes/, close as fixed.
Environment

Harvester

Version: v1.2.2-dev-20240329
Profile: QEMU/KVM, single nodes (16C/32G/500G)
ui-source: Auto

Steps

Download supportbundle and unzip nodes/harvester-node-0.zip

Check there is containerd.log :green_circle:

Check containerd.log has content :green_circle:


rocksdb: add WaitForCompact()

Summary: RocksDB 8.7 was released with C bindings for WaitForCompact() call that was added on our request. Update CGO wrapper with those bindings.
Reviewed By: leoleovich
Differential Revision: D51029439

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

Comparison is base (2da5c5b) 65.39% compared to head (b42bace) 68.05%.

:exclamation: Current head b42bace differs from pull request most recent head a2e6827. Consider uploading reports for the commit a2e6827 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   65.39%   68.05%   +2.66%
==========================================
  Files          62        7      -55
  Lines        6852      335    -6517
==========================================
- Hits         4481      228    -4253
+ Misses       2090      106    -1984
+ Partials      281        1     -280

see 69 files with indirect coverage changes

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


MARC: restriction in China

The current access handling in the MARC export doesn't consider the adm:isRestrictedInChina property but instead uses the deprescated access value. I'm not 100% sure how to deal with the combinations.

the access + restriction in China is encoded in the 506 field. For test references:

W00KG010084 is a totally restricted work
W22084 is an open access work
W3CN21529 is a "fair use" work
W1KG23747 is open access but restricted in China
W4CZ346626 is "fair use" but restricted in China

these are all the possible combinations


Error when using onprocessfile - not valid JSON returned.

Hello! When files uploaded response contains some long string, not JSON, i have added filepond* to except array, tried to disable debugbar, but nothing changed..
In dev console:
caught SyntaxError: Unexpected token 'e', "eyJpdiI6In"... is not valid JSON
request to filepond during upload returns string:
eyJpdiI6InQ1b3hTb2RBVVZGNWdybDRIeC9icGc9PSIsInZhbHVlIjoib1d5RVRNRDFnVzRLNzE2RzRwVklMbGs5eUFnUC96YWJoc2ZxaDRBR0QrVT0iLCJtYWMiOiIyMjAzZTA2NjU5OWMwMjdjMTQzZmRiMzY2ZTBiYWRlMmM1NDRjMzFiNTc3NGQxMGNmMWVhMWE5M2MzYTgxZDgzIiwidGFnIjoiIn0=
edit: Looks like return string is always the same, no matter what file i upload.
Project: Laravel 10 + Livewire, php 8.1
Help me please :(

I think i found reason - server returns encrypted id of file (Crypt::encrypt(['id' => $filepond->id])), but i dont understand how this function from example should work:
const handleFilePondAvatarProcess = (error, file) => {
form.avatar = file.serverId;
};
My code:
filePond = FilePond.create(filePondElement, {
                            allowMultiple: true,
                            instantUpload: true,
                            credits: false,
                            // chunkUploads: true,
                            // chunkForce: true,
                            acceptedFileTypes: ['image/*', 'application/pdf'],
                            server: {
                                url: '/filepond',
                                headers: {
                                    'X-CSRF-TOKEN': '{{ csrf_token() }}'
                                }
                            },
                            onprocessfile: (error, file) => {
                                if (!error) {
                                    const response = JSON.parse(file.serverId);
                                    console.log(file.serverId);
                                }
                            },
                        });

Or i got it wrong and "on" event and onprocessfile is different events ?

I think you've a unique case here. Why don't you add the encrypted string to array without JSON.parse(). Then submit and process the files in the server.

Found another way: in sweetalert2 submit function i call filePond.getFiles().map(x => x.serverId) to get array of serverid's, looks like this is only way, because processfile returns serverid, but removefile does not return serverid (author of filepond removed it some time ago, and didnt fixed it.. )
Thanks for help and this great laravel package !


Antora docs build

JIRA: HACBS-989

Still plenty to do, but it's generally working nicely.

Thanks for the pointers. I have an idea for a refactor, still hammering it out.
(Ps, I will make a Jira for this work - just realized I forgot about it for sprint plan.)

Almost there.

Preview at https://simonbaird.github.io/ec-policies/ec-policies/index.html .

It's just about done, but i want to check if it works with the main HACBS docs build before I move it out of draft.

I've tested the downstream build and it seems to be working.
Changes needed there are in https://gitlab.cee.redhat.com/red-hat-hybrid-application-cloud-build-services-documentation/hacbs-documentation/-/merge_requests/40 .
(Note that is failing currently since we need to first merge this PR.)

Downstream build that I did locally: http://file.bos.redhat.com/~sbaird/hacbs-public/public/index.html

Thanks very much for the notes. I'll finish tidying up nitpicks and likely merge tomorrow.

Okay we're done I think.


Custom absolute login redirect URL

Is it possible to specify an absolute redirect login URL in ASP.NET Core 2? The standard method does not accept absolute URLs
        services.ConfigureApplicationCookie(x => {
            x.LoginPath = new PathString("https://localhost:88/account/login");
        });

Produces a runtime error as the URL is expected to start with "/".
The initial problem is that my app service is served over HTTPS, which terminates at CloudFlare, so when user is not authorised, ASP.NET redirects the user to HTTP address, rather than to HTTPs.

Closing as a duplicate of https://github.com/aspnet/Docs/issues/2384.


Adding noobaa's core and db images to a new environment variables

In order to fix issue #372
Moving the responsibility to setup the noobaa images (core and db from the reconciler using the spec to a new environment variables used by the operator (https://github.com/noobaa/noobaa-operator/pull/194)
also fixed the corresponding test scenarios

/ok-to-test

@jarrpa ping

bump @jackyalbo

/retest

/hold

/hold cancel

@obnoxxx @jarrpa We need this backported to 4.3.
Is there a plan to make a mass backport soon?


Request to add additional calls and locations based on latest forestry part 2 update.

The below calls are the official calls of Forestry CC friends chat that operates on w444. These are published in our discord and shared with, and used by all FC members. Can you please add a way to add some/all of the locations and event types to the plugin? Ideally the below would be included by default with some options for variability, such as incorporating a text/regex based edit option for locations and event type. There are slight variations for each call and some flexibility/edibility will make for a more seamless experience for all.
Additionally, the best teles are included if this info can also be included within the setting menu and/or infobox.
Main Call Locations (With Tele Priorities)
-Arc = Arceuus, magics (Kharedst’s Memoirs 5 / Book of the Dead 5)
-Church = Seers’ Village church, yews (Seers’ Village tele > Diary Cape 5)
-Dray = Draynor, willows (Diary Cape G > Glory 3)
-Dray oak = Draynor, oaks (Glory 3 > Diary Cape G)
-Edge = Edgeville, yews (Glory 1)
-Flax = S of Flax/Beehives, yews (Diary Cape 5 > Kandarin Headgear 4)
-Glade = Hosidius, magics (Xeric's Talisman 2 > Con Cape 5)
-Myth = Myths’ Guild, magics (Myth Cape)
-Ne ge/Vcas = NE of Grand Exchange/N of Varrock Palace, yews (Ring of Wealth 2 > Spirit Tree 4)
-Nmage = N of Sorcerer's Tower, magics (Kandarin Headgear 4 > Combat Bracelet 4)
-Nseer = N of Seers’ Village bank, maples (Seers’ Village tele > Diary Cape 5)
-Nwseer/Seer willow = NW of Seers’ Village bank, willows  (Seers’ Village tele)
-Pmage = Prifddinas, magics (Con Cape 9)
-Pmahog = Prifddinas, mahoganies (Spirit Tree 6)
-Pteak = Prifddinas, teaks (Spirit Tree 6) – NOT compatible with 2t woodcutting in W444
-Rada w/Woodland w = W of Woodland, yews (Rada’s Blessing Kourend Woodland)
-Seer/Seer oak = S of Seers’ Village bank, oaks (Seers’ Village tele)
-Smage = S of Sorcerer's Tower, magics (Kandarin Headgear 4)
-Yak = Neitiznot, pines (Enchanted Lyre 3 + Elite Fremennik diary)
-Zalc = Zalcano, yews (Eternal Teleport Crystal Prifddinas > Con Cape 9)
Uncommon Call Locations (With Tele Priorities)
-Ape = Ape Atoll, teaks (Ape Atoll Dungeon tele) – compatible with 2t woodcutting in W444
-Ape w = Ape Atoll, mahoganies (Ape Atoll Dungeon tele)
-Barb = Barbarian Assault, oaks/willows (Games Necklace 5)
-Battlefront = NW of Battlefront, yews (Battlefront tele > Fairy Ring CIR)
-Cam = S of Camelot Castle, maples (Camelot tele > Diary Cape 5)
-Corsair = Corsair Cove Resource Area, maples/yews (Spirit Tree 5)
-Craft guild = Crafting Guild entrance, oaks/willows (Skills Necklace 3)
-Cw = SW of Castle Wars, teaks (Fairy Ring BKP)
-Fally gate = South Falador entrance, yews (Ring of the Elements 1 > Explorer's Ring > Ring of Wealth 3)
-Heart = Kourend Castle, yews (Xeric’s talisman 4 or Kourend tele)
-Hspirit/Spirit A = Hosidius Spirit Tree, yews (Spirit Tree A > Skills Necklace 5)
-Lgrave = Lumbridge Cemetery, yews (Diary Cape B > Lumbridge tele)
-Lookout = NW of Xeric’s Lookout, yews (Xeric’s Talisman 1)
-Mta e = E of Mage Training Arena, magics (Ring of the Elements 4 > Ring of Dueling 1)
-Mta w = W of Mage Training Arena, magics (Ring of the Elements 4 > Ring of Dueling 1)
-Ne glade = NE of Hosidius, magics (Xeric’s Talisman 2)
-Nieve = Tree Gnome Stronghold Nieve/Steve, magics/yews (Slayer Ring 1 > Diary Cape F > Spirit Tree 2)
-Pisc = S of Fairy Ring AKQ, yews (Fairy Ring AKQ)
-Rada e/Woodland e = E of Woodland, magics (Rada's Blessing Kourend Woodland)
-Rada se/Woodland se = SE of Woodland, yews (Rada's Blessing Kourend Woodland)
-Rim = NW of Rimmington POH, yews (Con Cape 2)
-Sarim =  Port Sarim Spirit Tree, willows (Spirit Tree 7 > Explorers's Ring > Glory 3)
-Se gnome = SE of Tree Gnome Stronghold, magics/yews (Slayer Ring 1 > Diary Cape F > Spirit Tree 2)
-Se prif = SE of Prifddinas, yews (Eternal Teleport Crystal Prifddinas > Con Cape 9)
-Shay = N of Graveyard of Heroes, yews (Kharedst’s Memoirs 3 / Book of the Dead 3)
-Slegend = S of Legends’ Guild, yews (Quest Cape > Fairy Ring BLR)
-Souls = W of Fairy Ring BJP, teaks (Fairy Ring BJP)
-Sw hos = SW of Hosidius player-owned house portal, yews (Con Cape 5 > Xeric’s Talisman 2)
-Wcg/Wc guild = N of WC Guild, yews (Skills Necklace 5 > Spirit Tree A)
-Wlegend = W of Legends’ Guild, maples (Quest Cape > Kandarin Headgear 4)
Event names
Important: If NOT making an event call, avoid using event names due to chat notification settings by others, but use alternatives, e.g., “Drv” for “Dry”, “3gg” for “Egg”, “3nt” for “Ent”, and “F0x” for “Fox”.
-Bee / Hive = Beehive
-Dry (Dryad) / Rit (Ritual) = Enchantment Ritual (alt: Shapes)
-Egg = Pheasant Control (alt: Bird, Pheas)
-Ent = Friendly Ent
-Flowers = Flowering Bush
-Fox = Poachers
-Lep = Woodcutting Leprechaun
-Roots = Rising Roots
-Sap = Struggling Sapling
-Root = Roots
-D = dead event
Note we provide to FC members - "P.S. Add "Late" to the end of a call if you were not there at the time of the event spawning, so others can prioritize the order of events they attend to appropriately - especially if there are multiple of the same events active across different locations."
Would love to partner together with the plugin. Please reach out via GitHub and join our discord if interested. https://discord.gg/t7na6BJNcd
Thank you!

Not sure if this could be of use, but sending through just in case - This is the Chat Notifications Regex script that we use for all events and then edit based on the calls we want:
Chat Notifications Plugin - Regex - Last updated 1/5/2024
Paste the below into Highlight Regex in Chat Notifications and remove the calls from 'Highlight Words', the regex will now only notify when event name and location are present. This means you will NOT be notified for FC chatting mistakes or when picking up the egg during pheasant event.
All Events Regex:
^.(?=.\b(p(rif+)?\s*(teak|mahog(any)?|mag(e|ic))|(n(orth)?|s(outh)?)?\sseer|(n(orth)?|s(outh)?)\smag(e|ic)|(xerics?\b?)?(glade|heart|lookout)|dray(nor)?|rada(\b?(e(ast)?|w(est)?))?|zalc(ano)?|edge|rim(mington)?|ape(\b?atoll)?|lgrave|lumb(y|ridge)?|nege|vcastle|church|myth|arc|yak|flax|MTA)s?).(?=.\b(ent|rit(ual)?|dry(ad)?|fox|egg|pheasant|bird|root|lep|sap(ling)?|flower|bee|hive)s?).$


[AssociatedPressNewsBridge] leadPhotoId not showing up in article card

Describe the bug
When subscribing to a feed via the AP News bridge, the leadPhotoId does not show up in the article "card".  If the article does have a jpeg inside of it (basically everything from the 'photography' topic), the first picture in the article will be used in the article card.  Have tested this on the Android app "Read You", and the Windows app "Fluent Reader" via self hosted FreshRSS (Fever API).
Expected behavior
leadPhotoId to display alongside article title
Screenshots
Read You

Fluent Reader

Desktop (please complete the following information):

OS: Windows
App: Fluent Reader

Smartphone (please complete the following information):

Device: Pixel 6
OS: GrapheneOS
App: Read You

Additional context
Add any other context about the problem here.

@dvikan Yeah sorry, it's not super clear what I am saying.
For the photography topic, in this article the leadPhotoId is https://storage.googleapis.com/afs-prod/media/484b96c45d494af593bb3680b9a34393/2000.jpeg.  It's also the first picture in the article, though it shows up as more of a banner image.  But, the bridge doesn't grab that picture at all, and instead uses the first picture after that (https://storage.googleapis.com/afs-prod/media/cea1010574fa46a3a9aa2cf25cccf75c/2000.jpeg) in the title card.
For the World News topic, in this article, the bridge doesn't display any photo in the title card despite this being the leadPhotoId and having 24 other pictures in a slideshow to choose from.
Does that make more sense?
The JPEG file name/number (2000.jpeg) appears to change depending on the resolution/user agent of the device.

The image that you are talking about I think is present as an attachment to the feed item. Can you verify? In that case we can move it to the beginning of the feed item content.

That appears to be correct, the attachment is the leadPhotoId.
I was going to try moving that around in the bridge config, but I basically don't know any php, heh

I fixed. Please test.

Beautiful, thanks!


Darker comments possibility for Solarized Light

In the Solarized Light theme, base5 is used for comments, and it is defined extremely light to my eyes (Also atypically light compared to standard Solarized Light, I think). I would like to have the option to have the comments use base6 instead. I would also want to use base6 for doc-comments and modeline-fg-alt, but the most crucial is comments, so that's what I'm addressing here.
This uses a bit cruddy short circuit with doom-solarized-light-brighter-comments, but I didn't want to merge the defcustoms or do anything more complicated before I knew if this was any way desirable for anyone else.
This is what comments look like with the original base5:

This is what they would look like with base6:

I agree. The comment colors have poor contrast. I've gone in and done a thorough revision of solarized-light's colorscheme, including the comments. Please update and give it a try.

@hlissner As someone who has used Solarized Light for everything from terminal and window manager up, I feel a bit more at home now.
Btw doom-emacs has become my window manager (with exwm), so I'm thankful.


*_persist.json

Hello,
you're creating '* _persist. json 'files.
Can you either save this to 'hombridge / persist /' or be configurable in the 'config. make json '.
Other plugins have an entry in the config.json for the location.
That would be great, because I want to have the homebridge folder free.
Many Thanks
edit:
should concern this
from readme.md in fakegato-history
``path: '/ place / to / store / my / persistence /' // if empty it wants to use the -U homebridge option if present, or. homebridge in the user's home folder
Thanks again

in the plugin 'https: // github. com / standards / home bridge-bravia '
this entry is usable:
'cookiepath': '/ var / homebridge / plugin-persist / sony-bravia / sonycookie',
in the wol plugin "https://www.npmjs.com/package/homebridge-wol" it is this entry:
'cacheDirectory': '/ var / homebridge / plugin-persist / wol-storage',
The plugin for NukiBridge "https://github.com/benzman81/homebridge-nukiio" has this entry:
'cache_directory': '/ var / homebridge / plugin-persist / nuki-persist / storage',
I hope it helps

No, it doesn’t.  Neither of these plugins use fakegato-history (to mimic the sensor history of Elgato devices in the Eve app, see #277).

I had edited in the first post,
you have to put 'path:'

Hello,
Unfortunately, I am not a programmer.
It has been checked in:
Store persist files in ~ /. homebridge / accessories, see # 298.
Is the folder firmly inside?
My homebridge data folder is:
/var/homebridge/accessories
or is the entry configurable?
Thank you very much

Hi,
have the app updated.
The homebridge stopped.
All _persist. json files deleted in /var/homebridge.
The homebridge started.
Now these files are back in the homebridge folder.
-rw-r--r--  1 root       root   178 Mär  1 18:01 hbridge3_00158d0001b1482a_persist.json -rw-r--r--  1 root       root   178 Mär  1 18:01 hbridge3_00158d0001b14841_persist.json -rw-r--r--  1 root       root   178 Mär  1 18:01 hbridge3_00158d0001dad04c_persist.json -rw-r--r--  1 root       root   178 Mär  1 18:01 hbridge3_00158d0001dad11d_persist.json -rw-r--r--  1 root       root   178 Mär  1 18:01 hbridge3_00158d0001e44c15_persist.json -rw-r--r--  1 root       root   15K Mär  1 18:01 homebridge-hue.json.gz
Thanks

It has been checked in:

I’m working on these (and other) changes, but they haven’t been released yet.

or is the entry configurable?

It will be the accessories folder of the homebridge user directory, specified with -U on startup, or ~/.homebridge by default.

Ok, I'll wait until it's released.

In v0.8.2.  Please be aware that you need to re-do you HomeKit configuration after upgrading to v0.8.x., see #299.

Thank you,
so far okay.
Only one file still appears after a reboot
'homebridge-hue. json. gz'
Can you outsource these to ./accessories?
Thank's

No, that's the debug dump of the bridge(s) and gateway(s) and config.json - see https://github.com/ebaauw/homebridge-hue#debug-dump-file.

OK
Thank you
go now, can be closed


can not get domain if domain with underscore (_)

e.g. http://wt_nbshentong.cn.b2b168.com/
can not get the domain.
{ hostname: 'wt_njxinjiyuan1.cn.b2b168.com',
isValid: false,
isIp: false,
tldExists: false,
publicSuffix: null,
domain: null,
subdomain: null }
do you know why?

Hi @henryhuang2013,
The issue is that unfortunately tld.js is rejecting this hostname as invalid because of the _ (underscore) character it contains. There was some work in the past to add an option to still accept such domains but it was not pursued: https://github.com/oncletom/tld.js/pull/122
Best,


Fix wrap x for custom projections without extent defined

Version 6.4.0 changed the way wrapX works, it stopped checking if given projection can be wrapped and extent exists. This PR restores it, returns unchanged coordinate if doesn't.
BTW. should this function mutate original array?

Duplicate of #11359
canWrapX includes a check for extent (it cannot wrap without it).


To be sure we are aware of this webpage.

@willgray @jovo Just double checking that we are aware of this http://docs.neurodata.io/ndpaper2016/

yah, i made it.
On Tue, May 17, 2016 at 11:14 AM, Jesse Leigh Patsolic <
<EMAIL_ADDRESS>wrote:

@willgray https://github.com/willgray @jovo https://github.com/jovo
Just double checking that we are aware of this
http://docs.neurodata.io/ndpaper2016/
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/neurodata/ndpaper2016/issues/32

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York


Expression length 2048 is too large (2048 bits max) nft: datatype.c:265: verdict_jump_chain_print: Assertion `0' failed. Aborted (core dumped)

I am unable to access pods from a different node than the node where the pod is running on.
To debug, I switched flannel from vxlan to host-gw. Because I think I understand the other mode of operation better. VXLAN is bit of magic to me.

Aborted (core dumped)

I am using RHEL 8, Kubernetes 1.17.3, nftables configured using the attached config file nfproxy.txt.
I'd be happy to provide any additional information. Thing is, I have very little previous experience with iptables and essentially nothing with nftables, so I don't know up front what else might be helpful to provide. I'll post what I think might be useful in a comment.

[root@host-10-0-133-145 ~]# ip r
default via <IP_ADDRESS> dev eth0 proto dhcp metric 100
<IP_ADDRESS>/22 dev eth0 proto kernel scope link src <IP_ADDRESS> metric 100
<IP_ADDRESS>/24 via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/24 dev cni0 proto kernel scope link src <IP_ADDRESS>
<IP_ADDRESS>/24 via <IP_ADDRESS> dev eth0
<IP_ADDRESS>/24 via <IP_ADDRESS> dev eth0
<IP_ADDRESS> via <IP_ADDRESS> dev eth0 proto dhcp metric 100
<IP_ADDRESS>/24 dev virbr0 proto kernel scope link src <IP_ADDRESS> linkdown

Output of nft list is <IP_ADDRESS>_nft_list_chains.txt.
[root@host-10-0-133-159 ~]# iptables -nvL
Chain INPUT (policy ACCEPT 0 packets, 0 bytes)
 pkts bytes target     prot opt in     out     source               destination
65268  113M KUBE-FIREWALL  all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
    0     0 TRACE      tcp  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0
    0     0 TRACE      tcp  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0

Chain FORWARD (policy ACCEPT 0 packets, 0 bytes)
 pkts bytes target     prot opt in     out     source               destination
    0     0 TRACE      tcp  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0

Chain OUTPUT (policy ACCEPT 0 packets, 0 bytes)
 pkts bytes target     prot opt in     out     source               destination
36638 2191K KUBE-FIREWALL  all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0

Chain KUBE-FIREWALL (2 references)
 pkts bytes target     prot opt in     out     source               destination
    0     0 DROP       all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0            /* kubernetes firewall for dropping marked packets */ mark match 0x8000/0x8000

Chain KUBE-KUBELET-CANARY (0 references)
 pkts bytes target     prot opt in     out     source               destination
# Warning: iptables-legacy tables present, use iptables-legacy to see them

I don't have iptables-legacy on my system (RHEL 8 does not ship it), I tried to workaround it by doing
[root@host-10-0-133-159 ~]# podman run --privileged --cap-add=NET_ADMIN --net host --rm -it fedora bash
[root@host-10-0-133-159 /]# dnf install -y iptables-nft
[root@host-10-0-133-159 /]# iptables-legacy -nvL
Chain INPUT (policy ACCEPT 60746 packets, 192M bytes)
 pkts bytes target     prot opt in     out     source               destination

Chain FORWARD (policy ACCEPT 0 packets, 0 bytes)
 pkts bytes target     prot opt in     out     source               destination
1146K  211M ACCEPT     all  --  *      *       <IP_ADDRESS>/16        <IP_ADDRESS>/0
1093K  308M ACCEPT     all  --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/16

Chain OUTPUT (policy ACCEPT 61483 packets, 4762K bytes)
 pkts bytes target     prot opt in     out     source               destination

but that apparently does not work, these are not the tables on the host

Hmm, the bug may be because I am running nft monitor on the packets. I'll go see if I can get patched version of nftables. https://bugzilla.redhat.com/show_bug.cgi?id=1763262

Updating nftables fixes the crash in nft monitor. So it is not caused by nfproxy. Sorry for being mistaken at first.
Before:
[root@host-10-0-133-159 ~]# rpm -qa "nftables"
nftables-0.9.0-14.el8.x86_64

After:
[root@host-10-0-133-159 ~]# rpm -qa "nftables"
nftables-0.9.3-11.el8.x86_64

My original problem still lasts. I am not able to communicate with pods running on other nodes.

Hi, thank you for reporting the issue. Please add the following information:
kubernetes version,  nfproxy log from both nodes around the time of the issue, yaml files for service, backend and client pod/deployment. I need these to try to reproduce the issue.
Please also collect nft list table kube-nfproxy-v4

I got it. Configuration error regarding CNI in Kubernetes on my side!
When removing flannel and installing calico (I wanted to try the tested configuration from README.md) I realized that I followed the advice on https://github.com/cri-o/cri-o/blob/3b788a222ad5631cd6273270eb38af0f700ab873/tutorials/kubeadm.md#configuring-cni and changed crio.network.network_dir in crio.conf to /etc/cni_crio/net.d. I then added --cni-conf-dir=/etc/cni_crio/net.d to /etc/sysconfig/kubelet, but still, the /etc/cni_crio/net.d was not updated automatically. After I copied calico files from etc/cni to etc/cni_crio, and rebooted nodes, everything started working.
I'll try now to go back to flannel (I need flannel, because I need to have Windows nodes in the cluster, and calico does not support that yet) and if it works, and I think it now will, I'll close the issue.
The doc page of crio linked above was updated yesterday with somewhat different instructions, that don't make sense to me as of yet. I'll try to understand them...

Switched back to flannel and everything still works. Sorry for blaming the issues on nfproxy, when iptables-nft in rhel and my CNI config were the real culprits.


Fix 45 - Moving a folder using move(to:) creates an invalid path

Fixes #45

Released in https://github.com/JohnSundell/Files/releases/tag/2.1.0


Optimize images

This pull request optimizes the images found in the repository using lossless compression. This means that the image quality will not be degraded. This will mean less space taken up by images along with faster loading times when viewing them. The overall file size has been reduced by around 20%.

Thanks @JeffResc for opening this PR. Following through your other contributions, came across https://github.com/marketplace/imgbot and have installed it for this repository.

You can sign up for Hacktoberfest swag from MayaData by entering details here: https://github.com/mayadata-io/swag/issues/new/choose


Enable the ESLint no-var rule in gulpfile.js

Please refer to the individual commit messages for additional details.

/botio unittest

/botio-linux preview

Looks good; thanks!


chore: enforce type imports where possible

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

:exclamation: No coverage uploaded for pull request base (main@25ede86). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2348   +/-   ##
=======================================
  Coverage        ?   93.69%
=======================================
  Files           ?      239
  Lines           ?    24030
  Branches        ?     2089
=======================================
  Hits            ?    22514
  Misses          ?     1479
  Partials        ?       37

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


ubiContract calls returning empty

Hi! Thanks for this project! It has been working good for the last few weeks. Today, any ubiContract calls seem to be returning empty:
TypeError: Cannot read properties of null (reading 'toNumber')
    at /home/user/goodtimes/cli.js:43:85
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async claim (/home/user/goodtimes/cli.js:43:23)
    at async /home/user/goodtimes/cli.js:18:3

A quick Google search returns that the most common issue is a wrong ABI, but it was working fine before. Any ideas, please?

It looks like they moved the contract as part of the protocol upgrades for GoodDollarV2. I just pushed out a new version (1.0.6) pointing to the new address. I'll close this after I verify a claim tomorrow morning.

Worked for me too, thanks!


Add backend validations for CSV links

Extends #650
Closes #648

ImportError: No module named django_orm
travis check fails
could you have a look, I think it's because of google-api-client version but you should check to be sure

Resolved django_orm issue. It was related to package version.

lgtm

buggy:

clicking on publish doesnt fire the Dialog Open sometimes, and then if it does the default Task field is empty and publish doesnt work from there

could you have a look?

self.instance.status will fail on create because the instance does not exist yet


[ Ui Bug ]. The UI doesn't fit the Android screen.

Godot version
v3.5.beta4.official [b6968ab06]
System information
Android 10. Oppo A15
Issue description
The button can still be pressed but, you know it's hard and uncomfortable.
I hope in the next update the UI will be fixed soon.

[ Ignore ] other bug :
• When I open the project the screen is just black (That's why I can't give a preview of the project)
• I can't change the language back to English (that's why in the picture I use Indonesian)

Steps to reproduce
Open Godot app | v3.5.beta4.official [b6968ab06] | in Android
Minimal reproduction project
No response

Related to https://github.com/godotengine/godot/issues/31133.
I'll leave this issue open as it's specifically about the project manager, but the underlying issue is the same.


#3435 - System announcements

Add a database table for system announcements, and a way to display them on student and institution dashboards.

Nice work. Please take a look at the comments.
The most important part of API is that, It must not return the messages that are not intended for it's own client type.

Thanks for doing the changes @katrina-aot-git. I just have one outstanding comment on E2E tests. https://github.com/bcgov/SIMS/pull/3678#discussion_r1761409785


Don't use staging buffer for simple resources

This change will make DXVK only use a staging buffer for D3D9 resources that have been locked twice before. It should improve memory usage in games that only copy data to their textures once. (which should be the vast majority of games)
It also contains a tiny optimization for buffer locking because once we hit the switch to use staging buffers, we can omit the CS thread sync point. (Supersedes #2039)

Character rendering issue on Dragon Nest when using d3d9.evictManagedOnUnlock = True
https://imgur.com/a/kKJIHf3

@dnblank123 That only happens with this PR, right?

@dnblank123 That only happens with this PR, right?

Yes, I already tried #2039 and new release (1.9) not issue so far.

Any chance you could make an apitrace?

@K0bin https://drive.google.com/file/d/1vHR24meYzsuoeOQEEYhOLI4OqDLIk9EH/view?usp=sharing

Thanks a lot.


nav mode: fatal error: runtime: out of memory

When access proc file like pagemap.

Not fixed by a80e6d15faf4221e5063dcd4858481e8beb7d886?

@xiaq No, proc file size return by stat is always zero.

@xiaq But it would be nice if nav mode doesn't filter out all proc files. I find nav mode with preview is very useful when check a process's information.


Update Readme to include all latest options.

Updated Readme, but "allowGlobalShortcuts" doesn't currently toggle.
PR Checklist
Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[x] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[x] Documentation changes
[ ] Other - Please describe:

Fixes
Issue Number:
What is the current behavior?
What is the new behavior?
Other information

@jkvyff Thanks :v:


Samples don't work

./tagui samples/1_yahoo  firefox
ERROR - use .gui or .txt or no extension for flow filename
ERROR - automation aborted due to above

Thanks @jefleponot for your feedback! Can you tell me more about your execution environment? I can run on my macOS with below output, and for my CentOS server I can run without firefox option.
START - automation started - Tue Mar 07 2017 16:38:55 GMT+0700 (ICT)

https://www.yahoo.com - Yahoo

enter search-box as github
print search-box - github
click search-button
wait 6.6
snap page
snap logo
snap page to results.png
snap logo to logo.png
https://duckduckgo.com - DuckDuckGo
enter search_form_input_homepage as The search engine that doesn't track you.
wait 4.4 seconds

https://duckduckgo.com/ - DuckDuckGo
FINISH - automation finished - 16.2s

To improve on the robustness of the sample flow, I have also updated the flow to explicitly use ca.yahoo.com instead. I think for some users, using yahoo.com might end up serving country-specific versions and the element identifiers will be different.

@jefleponot I looked at the code leading to this error and issue #19, it looks like you may be trying to run it under Windows. As mentioned in the Set Up section, TA.Gui is still in beta and not yet ported to Windows. I'll close this issue but please let me know if there are other reason for the error message!

Hi @kensoh
I test it on Ubuntu 16.04

Hi @jefleponot, will you be able to kindly create a dummy user / account to your Ubuntu so that I can try debugging? I don't have access to a Ubuntu system now, following is the code block which throws the error. What it does is checking for presence of . after removing relative paths and then validates if the file extension is gui or txt.
// check flow filename for .gui or .txt or no extension
$script = $argv[1]; if ($script=="") die("ERROR - specify flow filename as first parameter\n");
if (strpos(str_replace("./","",str_replace("../","",$script)), '.') !== false) // check for extension
if ((pathinfo($script, PATHINFO_EXTENSION)!="gui") and (pathinfo($script, PATHINFO_EXTENSION)!="txt"))
die("ERROR - use .gui or .txt or no extension for flow filename\n");

In the meantime, if you don't have a test user for me to debug, can you try commenting off for example below, and see if any other error messages appear? Hopefully more clues can lead to isolating the root cause of this error message.
// check flow filename for .gui or .txt or no extension
$script = $argv[1]; if ($script=="") die("ERROR - specify flow filename as first parameter\n");
// if (strpos(str_replace("./","",str_replace("../","",$script)), '.') !== false) // check for extension
// if ((pathinfo($script, PATHINFO_EXTENSION)!="gui") and (pathinfo($script, PATHINFO_EXTENSION)!="txt"))
// die("ERROR - use .gui or .txt or no extension for flow filename\n");

I'm reopening the issue as it is happening on Linux-flavor system.

Adding on, /src/tagui_parse.php is where above code block comes from, it's the main engine of TA.Gui project where natural language syntax gets parsed into working CasperJS JavaScript code.

Hi @jefleponot, I think I might have found the cause. Do you happen to install at a directory path containing a '.' in the directory name? If yes it will break, have committed fix that will work now for directory paths containing '.'. Below is the commit message. Let me know if the error message still happens for you!

error is likely due to directory path having a '.'
eg ta.gui will break while master or tagui is fine
amended code to work for directory paths with '.'

Hi @jefleponot, I might have found the cause. Do you happen to install at a directory path containing a '.' in the directory name? If yes it will break, have committed fix that will work now for directory paths containing '.'. Below is the commit message. Let me know if the error message still happens for you!

error is likely due to directory path having a '.'
eg ta.gui will break while master or tagui is fine
amended code to work for directory paths with '.'


Improve image verification via batch/parallel mechanisms

Make batch calls to the registry to resolve a reference to a digest and get referrers for different subjects at once.  This is currently not supported yet and needs an upstream dependency filed.
Provide multiple subject images in the external data provider request, Ratify processes each one in parallel - today it is serial.

Related/Overlap with https://github.com/deislabs/ratify/issues/153

@FeynmanZhou - is there a related item for item 1.  in ORAS* anywhere?

@FeynmanZhou - is there a related item for item 1. in ORAS* anywhere?

@dtzar Make batch calls to the registry to resolve a reference to a digest is not supported in ORAS. I also confirmed with @shizhMSFT before. We thought item 1 is not in the scope of ORAS so far. Could you pls elaborate on this request?

@FeynmanZhou - is there a related item for item 1. in ORAS* anywhere?

@dtzar Make batch calls to the registry to resolve a reference to a digest is not supported in ORAS. I also confirmed with @shizhMSFT before. We thought item 1 is not in the scope of ORAS so far. Could you pls elaborate on this request?

Make batch calls to the registry to resolve a reference to a digest and get referrers for different subjects at once.

oras-go provides the ability to resolve a tag to a digest and the ability to find the referrers of it, which are concurrent-safe. Therefore, batch/parallel mechanisms can be and should be performed at the callers' side.

@akashsinghal - Shiwei's point is good that the parallelization of the requests being done on the client side, which is what you're doing with #153.  Could you please elaborate more on any potential request to ORAS team?

I think we should close this issue for now. Shiwei's point makes sense and now we have parallelization so batch processing from the registry is not a need anymore.

Agreed. We can reopen with more data if this is an issue.


feat(ReportOptions.onReportCallback): add optional callback to `Repor…

…tOptions`, that is called when errors are reported
fix #81

Turns out that I don't need this PR to cover my needs - see https://github.com/panuhorsmalahti/gulp-tslint/issues/81#issuecomment-253898530


xr.merge with compat="minimal" returns corrupted Dataset and causes len to return wrong and possibly negative values.

What happened?
When merging multiple datasets with the compat="minimal" option, coordinates whose variables are dropped due to incompatibility are still saved in the dataset's _coord_names. I believe the cause for this to originate in line 752 of merge_core, where the coordinate names are based on the datasets in coerced, which is not impacted by the dropping of (coordinate) variables/indexes in the merge_collected function.
This is directly related to the bug described in issue 7405. As seen there, one result is that dropped coordinate still evaluates as being contained in the resulting dataset's coords. The effects of this bug are more widespread, which this issue attempts to dive into.
At least one other (perhaps more severe) result of this bug is connected to the fact that the __len__ function of a DataVariable is implemented as follows:
return len(self._dataset._variables) - len(self._dataset._coord_names)
If a coordinate was dropped as a result of the merge, it is no longer part of the _variables, but still listed in the _coord_names, and as such the result of len() will be off by 1 for each such coordinate. This also means that the result of len() can become negative, which causes python to raise ValueError: __len__() should return >= 0.
One instance where this causes immediate errors is when trying to print the resulting dataset. As part of the __repr__ of a Dataset, a boolean evaluation of the DataVariable is performed (if mapping: in xarray/core/formatting.py in _mapping_repr), calling __len__ to check the truth value and triggering the ValueError.
While this is undoubtedly only one of many places where the incorrect __len__ causes issues, it is a rather pressing one as it even stops one from inspecting the Dataset in the most common way (printing it). The ValueError it produces is also very hard to trace back to the actual cause, likely completely throwing users off from fixing their code.
What did you expect to happen?
To get a Dataset with the correct _coord_names property, and in no circumstance whatsoever to get a Dataset which reports a negative length
Minimal Complete Verifiable Example
import xarray as xr
ds1 = xr.Dataset(coords={"foo": [1, 2, 3], "bar": 4})
ds2 = xr.Dataset(coords={"foo": [1, 2, 3], "bar": 5})

res = xr.merge([ds1, ds2], compat="minimal")  # If the result is not captured in res, this will cause a ValueError as the interpreter attempts to print the result

res.coords
# Coordinates:
#   * foo      (foo) int64 1 2 3

res._coord_names
# {'foo', 'bar'}

"bar" in res.coords  # As shown in issue #7405. Note "bar" is not printed in res.coords, revealing an interesting disconnect in behaviors of different functions targeting a dataset's coordinates
# True

res
# ValueError: __len__() should return >= 0

MVCE confirmation

[x] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[x] Complete example — the example is self-contained, including all data and the text of any traceback.
[x] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[x] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
>>> import xarray as xr
>>> ds1 = xr.Dataset(coords={"foo": [1, 2, 3], "bar": 4})
>>> ds2 = xr.Dataset(coords={"foo": [1, 2, 3], "bar": 5})
>>> res = xr.merge([ds1, ds2], compat="minimal")
>>> res.coords
Coordinates:
  * foo      (foo) int64 1 2 3
>>> res._coord_names
{'bar', 'foo'}
>>> "bar" in res.coords
True
>>> res
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/redacted/.venv/lib/python3.10/site-packages/xarray/core/dataset.py", line 2116, in __repr__
    return formatting.dataset_repr(self)
  File "/usr/lib/python3.10/reprlib.py", line 21, in wrapper
    result = user_function(self)
  File "/home/redacted/.venv/lib/python3.10/site-packages/xarray/core/formatting.py", line 673, in dataset_repr
    summary.append(data_vars_repr(ds.data_vars, col_width=col_width, max_rows=max_rows))
  File "/home/redacted/.lvenv/lib/python3.10/site-packages/xarray/core/formatting.py", line 357, in _mapping_repr
    if mapping:
ValueError: __len__() should return >= 0

Anything else we need to know?
 Do you see a simple fix within merge_core?

When using xr.merge with compat='minimal', the resulting Dataset may have unexpected behavior, including causing len to return wrong and possibly negative values.
This is because compat='minimal' mode allows for the merging of datasets with non-matching dimensions and coordinates. When this happens, the merged dataset may contain "empty" dimensions or coordinates that have lost their original values and attributes.
To avoid this issue, you can either use compat='override' mode, which will overwrite conflicting dimensions and coordinates, or manually align and concatenate the datasets before merging them.
Here's an example of how to manually align and concatenate two datasets before merging them:
import xarray as xr
create two sample datasets with different dimensions
ds1 = xr.Dataset({'foo': (['x', 'y'], [[1, 2], [3, 4]])}, coords={'x': [0, 1], 'y': [0, 1]})
ds2 = xr.Dataset({'bar': (['x', 'z'], [[5, 6], [7, 8]])}, coords={'x': [0, 1], 'z': [0, 1]})
align and concatenate the datasets along the 'x' dimension
ds1_aligned = ds1.sel(x=slice(None, 1))
ds2_aligned = ds2.sel(x=slice(1, None))
merged = xr.merge([ds1_aligned, ds2_aligned])
In this example, we first slice each dataset to remove any non-matching dimensions and coordinates, and then concatenate them along the 'x' dimension. Finally, we merge the aligned datasets to create a single merged dataset without any corrupted dimensions or coordinates.

Thanks for diving into this @Metamess.

At least one other (perhaps more severe) result of this bug is connected to the fact that the len function of a DataVariable is implemented as follows:
return len(self._dataset._variables) - len(self._dataset._coord_names)

Dataset._variables and Dataset._coord_names should always be in sync, in theory. There's internal logic that checks that Dataset._coord_names is a subset of set(Dataset._variables) in almost every of the thousands of tests in Xarray, which minimizes the risk of having them out-of-sync. Unfortunately, it is not caught when tests are missing like it seems to be the case here.

Thank you for the PR @benbovy! And my apologies for not responding further initially; I wasn't familiar enough with the code base of xarray to try my hand at fixing it myself, so I really appreciate you picking it up!


Add digital signing guideline file

Was able to cherry-pick the commit to production and pull back to acceptance and development


Blog module giving error

Hi- All other controllers work well with clean URL and no ID but not with the blog module. I checked the controller index for this module and it is written correctly. How do I correct this as SSL is failing and Google is knowing my score down? TY!

ms Blog
*

@author    Joommasters<EMAIL_ADDRESS>@copyright 2007-2015 Joommasters
@license   license http://www.gnu.org/licenses/gpl-2.0.html GNU/GPL
@Website: http://www.joommasters.com
*/

include_once(PS_MODULE_DIR.'jmsblog/class/JmsBlogHelper.php');
class JmsblogCategoriesModuleFrontController extends ModuleFrontController
{
public $ssl = true;
public $display_column_left = false;
/**
 * @see FrontController::initContent()
 */
public function initContent()
{

	parent::initContent();
	$categories 	= $this->getCategories();
	$jmsblog_setting = JmsBlogHelper::getSettingFieldsValues();
	for($i = 0; $i < count($categories); $i++)
	{
		$categories[$i]['introtext'] = JmsBlogHelper::genIntrotext($categories[$i]['description'], 120);
	}
	$this->context->controller->addCSS($this->module->getPathUri().'views/css/style.css', 'all');
	$this->context->smarty->assign(array('meta_title' => 'Categories'));
	$this->context->smarty->assign(array(
		'categories' => $categories,
		'jmsblog_setting' => $jmsblog_setting,
		'image_baseurl' => $this->module->getPathUri().'views/img/'
	));
	$this->setTemplate('categories.tpl');

Hi,
thanks for reporting.
Sorry but I don't fully understand the issue. What is your PS configuration, the module configuration, the expected behaviour and the obtained one?
Have you other modules installed that are overriding the core?
How can I reproduce and test ?

@codepoppo are you alive?


Babe

Describe the problem

Describe the solution

Alternatives you've considered

Please send me link of this report on Gmail. Properly open and clear.
Ali khan
On Tue, Oct 3, 2023, 1:44 PM Pierre Millot @.***> wrote:

Closed #2056
https://github.com/algolia/api-clients-automation/issues/2056 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/algolia/api-clients-automation/issues/2056#event-10535204372,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCXVXILDDAZGFVGBFIBQWFLX5PF5VAVCNFSM6AAAAAA5MX43OOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQGUZTKMRQGQZTOMQ
.
You are receiving this because you authored the thread.Message ID:
@.***
com>


[YUNIKORN-1369] Shim: Read configuration from configmaps and deprecate legacy configs

What is this PR for?
Read configuration from new ConfigMap entries in addition to legacy values, and deprecate old configuration sources.
[WIP] Based on and depends on from YUNIKORN-1365.
What type of PR is it?

[ ] - Bug Fix
[x] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-1369
How should this be tested?
Screenshots (if appropriate)
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Merged to master, updated Debug() to Warn() as requested.