Make language codes unique for Naver-related websites with multiple subtitles per language

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm requesting a site-specific feature
[X] I've verified that I'm running yt-dlp version 2022.10.04 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Example URLs
https://www.vlive.tv/video/23383
https://www.vlive.tv/video/23861
https://www.vlive.tv/video/908
Provide a description that is worded well enough to be understood
These VLIVE videos have multiple subtitles for the same language. yt-dlp only downloads one per language code, even though these subtitles are different. Sometimes fan subs are more detailed than official subs, or sometimes one of them is unfinished, or multiple fans used the same language code, even though one of them belongs to a different language. The problem lies in the Naver extractor, as the VLIVE extractor inherits from NaverBaseIE.
The captions look like the following in the JSON output:
			{
				"language": "ko",
				"country": "KR",
				"locale": "ko_KR",
				"label": "한국어",
				"source": "http://resources-rmcnmv.pstatic.net/globalv/c/read/v2/VOD_ALPHA/global_v_2017_02_27_1/6ee2Jz2kJsFl7x9cNXNeoiWB6g_rmcvideo_360P_640_1228_128-1488186834268_ko_KR_cp.vtt",
				"type": "cp",
				"fanName": ""
			},
			{
				"language": "ko",
				"country": "KR",
				"locale": "ko_KR",
				"label": "한국어",
				"source": "http://resources-rmcnmv.pstatic.net/globalv/c/read/v2/VOD_ALPHA/global_v_2017_02_27_1/6ee2Jz2kJsFl7x9cNXNeoiWB6g_rmcvideo_360P_640_1228_128-1488282198451_ko_KR_fan.vtt",
				"type": "fan",
				"fanName": "나옹이"
			}

To be able to download all subtitles, the type and fanName could be used to differentiate them, for example ko_KR-fan-나옹이, otherwise they all get added to the same URL list in sub_dict, resulting in only one file being downloaded.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['--list-subs', 'https://www.vlive.tv/video/23383', '--no-playlist', '-vU']
[debug] Portable config "D:\Documents\youtube-dl\yt-dlp.conf": ['-U']
[debug] User config "C:\Users\<user>\AppData\Roaming\yt-dlp\config": []
[debug] Encodings: locale cp1252, fs utf-8, pref cp1252, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version 2022.10.04 [4e0511f] (win32_exe)
[debug] Python 3.8.10 (CPython 64bit) - Windows-10-10.0.19043-SP0
[debug] Checking exe version: ffmpeg -bsfs
[debug] Checking exe version: ffprobe -bsfs
[debug] exe versions: ffmpeg 4.4.1-full_build-www.gyan.dev (setts), ffprobe 4.4.1-full_build-www.gyan.dev, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.15.0, brotli-1.0.9, certifi-2022.09.24, mutagen-1.45.1, sqlite3-2.6.0, websockets-10.3
[debug] Proxy map: {}
[debug] Loaded 1690 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version: 2022.10.04, Current version: 2022.10.04
yt-dlp is up to date (2022.10.04)
[debug] [vlive] Extracting URL: https://www.vlive.tv/video/23383
[vlive] 23383: Downloading officialVideoPost JSON metadata
[vlive] Downloading just the video 23383 because of --no-playlist
[vlive] 23383: Downloading inkey JSON metadata
[vlive] 23383: Downloading JSON metadata
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[info] Available automatic captions for 23383:
Language Formats
es_ES    vtt, ttml
in_ID    vtt, ttml
pt_BR    vtt, ttml
ja_JP    vtt, ttml
[info] Available subtitles for 23383:
Language Formats
en_US    vtt, ttml
vi_VN    vtt, ttml
th_TH    vtt, ttml
zh_CN    vtt, ttml
zh_TW    vtt, ttml
ko_KR    vtt, ttml, vtt, ttml

Do you have an example with multiple fan subs? If not, just language-type should be enough

https://www.vlive.tv/video/908 This one has 3 languages with 2 fan subs (Italian, Russian, Turkish).


An error occurred while attempting to establish an SSL or TLS connection.

Please Help

Thanks for reaching out to us. Is the certificate valid for your email server? Does it support wildcards? It says that the ssl certificate issued doesn't match the host name you are using.

Hi, @knowyi little off topic, but could you share your docker composer YML file?
Thanks in advance!

Hi @knowyi,
did you find the solution to your problem?
I'm getting the same error.
Thanks

I'd check your ssl certificates. You could also take our code and step through it to see if you can get more information / try different things.

@niemyjski  It's settled. Thank you


Front

Changed styling to bootstrap and graphs to D3. No real data is hooked up to this frontend yet, so as of now it is all dummy data.


Update UITK website's help page(s) as needed

Revamp UITK's help section so that:

Users are explicitly directed to the appropriate internal tools or forums for questions, bug reports, etc.
Office hours sessions are mentioned
Available contact email addresses are reduced and de-emphasized
Community chatroom is de-emphasized

(This issue mirrors the internal UITK-10078 one)

May: Shared draft structure with team and got some feedback. Will work on amends.


Fixes 4712: add content override for sslverifystatus

Summary
By default clients registered to stage/prod have  sslsslverifystatus set to 1, which works fine for cdn.redhat.com, but not for s3.   This overrides that setting for each content set.
Testing steps
Register a system and associate it to a template: https://github.com/content-services/content-sources-backend/blob/main/docs/register_client.md
on the client run 'subscription-manager refresh'
# grep sslverifystatus /etc/yum.repos.d/redhat.repo
sslverifystatus = 0

https://issues.redhat.com/browse/HMS-4712

apologies, this will have to be qe'd in stage!


test writes to state before hasTable is true

curently tests wait until hasTable is true before doing sets, should test the usecase where we dont wait for this.

meh


[Long term]: Consolidate into less (public) .bzl files and rules

No action item for now, but we need to start simplifying what user-visible pieces we have in the near future - this is also an issue internally. For example, dart_ddc_bundle - maybe that should just be part of a dart_web_application() macro.

We will still need flags for configuring dart2js output

Creates one or more dart_dev_server rules for you

dart_vm_binary (Doesn't change much/at all)

Agreed that seems reasonable and would reduce confusion

Moving all the DDC stuff into web.bzl seems like a good start.
Keeping platform-specific rules separate has a few significant advantages. Among other issues, implicit dependencies (e.g., on protoc in dart_proto_library) drag in the full transitive closure of the tools they depend on, which is annoying for exoblaze users.
Personally, core.bzl, web.bzl, vm.bzl seem good enough as the full set of developer-facing Skylark files.

Personally I would like to keep the implementations of the actions at least in a separate bzl file, huge python files are not fun to work with imo. The downside being the implementations actually end up being public functions instead of faux-private ones, but maybe we create a src folder or something to further separate the private implementations from the public facing rules?

Im fine with that; it's basically what we did with internal.bzl


Metadata: project status

Expected Behavior
It'd be nice to have metadata that denotes the project status in a way that could be used by gradle and other consumers to produce things like warnings. I think this needs to be a file update-able after a publication
In theory you could publish something like (this is not a well thought out api)
{
   "Status":"Unmaintained",
   "Deprecated": "https://usethisinstead"
}

if a project was using such a dependency it would get a warning about it.
I could see this also being useful for security reporting...
security: {
  "0.1.0": {
     CVE1234: "https://..."
  }

just an idea.
Current Behavior (optional)
No response
Context
I have some old plugins on portal that are deprecated and/or unmaintained. I also have no real ability to publish them with gradle anymore. The gradle was version 5 something, and then one day it stopped working... idk why. Seems there isn't a great way to report this status to anyone who has pulled them down though (probably no one).

This is an ecosystem issue that requires a lot of collaboration between repositories like Maven Central, library authors, build tool developers, etc. This is not something that can be solved solely by Gradle Build Tool.


Update the license file

Rename the file to License.txt, and change name to OData-ValidationTool

Great catch. I agree. Precise makes perfect. :)
The current unprecise version is however learnt from other Microsoft projects: https://github.com/Microsoft/dotnet/blob/master/LICENSE.
I'll update the file according to the feedback.

Just found that the OSS approval requires the file name to be "license.txt" and project name to be "OData-Validation Tool". Will adapt the proposal to it accordingly.

@congysu Please review this PR: https://github.com/OData/ValidationTool/pull/5. After merging you may delete the branch.

License file updated: https://github.com/OData/ValidationTool/blob/master/License.txt


[CASSANDRA-17065] Support settings for entire SSTable outbound streaming

This commit adds support for entire SSTable outbound streaming and entire SSTable
outbound streaming between datacenters. The new configuration options are:

entire_sstable_stream_throughput_outbound_megabits_per_sec for internal throttling, and
entire_sstable_inter_dc_stream_throughput_outbound_megabits_per_sec for inter-DC throttling.

Additionally, nodetool support is added in this commit as well to set entire SSTable
outbound streaming settings. For example, to set the entire SSTable outbound streaming
to 1200 Mbps, new command-line flags (-esst, --set-entire-sstable-throughput,
--print-entire-sstable-throughput) are introduced:
nodetool setstreamthroughput -esst 1200
nodetool setstreamthroughput --set-entire-sstable-throughput 1200

To print the entire SSTable outbound streaming settings, the flag can also be used like this:
nodetool getstreamthroughput -esst
nodetool getstreamthroughput --print-entire-sstable-throughput

The same flags are supported for nodetool's setinterdcstreamthroughput and
getinterdcstreamthroughput operations.
This commit also introduces an optimization when entire SSTable throttling is disabled.
The optimization uses netty's DefaultFileRegion to flush to the channel, as opposed to
the SharedFileChannel implementation, which uses the RateLimiter for the transfer.

Thanks @yifan-c for the review. I have addressed your comments.


Search result order on the hub is counter-intuitive

Hello,
Is your feature request related to a problem? Please describe.
The search results from the hub are sometimes irrelevant with respect to the query. For example below, the first relevant result is shown only on the 5th row.

Describe the solution you'd like
Show results for the user query at the top, based solely on the text, maybe case insensitive. One possibility would be to add search suggestions in case we think the user made typos.
Describe alternatives you've considered
An other possibility would be to support queries with quotation marks, like "mnist", and behave like google search which yields pages with exact match only.

To follow up on this, looks like the search from the homepage is much more consistent:


Remove TPS support

Since we do one tick per frame, we will rely on vsync to manage our tick rate for now.
See #96 for details.

That uses unnecessary CPU cycles. Have you seen how much CPU Driftwood can eat up at higher tps? I think we should keep the setting.

@seisatsu I haven't because it runs with vsync on my computer.
Does your vsync not work?

Actually, hold on a second. I haven't tested Driftwood with no TPS limit yet, and I am assuming it limits itself to one tick per frame.
Let me check this assumption.

If possible, if Driftwood it isn't using vsync on your computer, I'd rather open an issue and fix that as a bug.

One frame, one tick. And the ticks run at exactly the tps given, unless obstructed by load.

If I set the TPS and FPS to 6000, the game still only runs at 60 TPS and 60 FPS on my Mac.
Looks like even with common Linux setups we can't expect this behavior, though. We'll need to keep TPS. I might try to improving the current TPS system sometime.

Here is the behavior I was expecting with vsync on.

@seisatsu Please excuse me for bringing up a closed issue, but could you provide some information about what graphics card & driver that computer has?
I just got a Linux with a Radeon setup today & I can't reproduce what you're describing.


Build job cannot be recognized correctly after upgrade to new  version

Jenkins and plugins versions report

Environment

Jenkins: 2.319.1
OS: Linux - 3.10.0-957.21.3.el7.x86_64
---
ace-editor:1.1
apache-httpcomponents-client-4-api:4.5.13-1.0
bootstrap4-api:4.6.0-3
bootstrap5-api:5.1.3-4
bouncycastle-api:2.25
branch-api:2.7.0
build-with-parameters:1.5
caffeine-api:2.9.2-29.v717aac953ff3
checks-api:1.7.2
cloudbees-folder:6.17
command-launcher:1.6
credentials:1074.v60e6c29b_b_44b_
credentials-binding:1.27.1
data-tables-api:1.11.3-6
display-url-api:2.3.5
durable-task:493.v195aefbb0ff2
echarts-api:5.2.2-2
font-awesome-api:5.15.4-5
generic-webhook-trigger:1.83
git:4.10.3
git-client:3.11.0
git-parameter:0.9.15
git-server:1.10
jackson2-api:2.13.1-246.va8a9f3eaf46a
javadoc:1.6
javax-activation-api:1.2.0-2
javax-mail-api:1.6.2-5
jaxb:2.3.0
jdk-tool:1.5
jersey2-api:2.35-3
jquery:1.12.4-1
jquery-detached:1.2.1
jquery3-api:3.6.0-2
jsch:<IP_ADDRESS>
junit:1.53
localization-support:1.1
localization-zh-cn:1.0.24
mailer:408.vd726a_1130320
matrix-auth:3.0.1
matrix-project:1.20
maven-plugin:3.16
plain-credentials:1.7
plugin-util-api:2.12.0
popper-api:1.16.1-2
popper2-api:2.11.2-1
publish-over:0.22
publish-over-ssh:1.20.1
resource-disposer:0.17
role-strategy:3.2.0
scm-api:595.vd5a_df5eb_0e39
script-security:1131.v8b_b_5eda_c328e
snakeyaml-api:1.29.1
sonar:2.14
ssh:2.6.1
ssh-credentials:1.19
sshd:3.1.0
structs:308.v852b473a2b8c
trilead-api:1.0.13
workflow-api:1136.v7f5f1759dc16
workflow-basic-steps:2.24
workflow-cps:2648.va9433432b33c
workflow-cps-global-lib:552.vd9cc05b8a2e1
workflow-durable-task-step:1121.va_65b_d2701486
workflow-scm-step:2.13
workflow-step-api:622.vb_8e7c15b_c95a_
workflow-support:813.vb_d7c3d2984a_0
ws-cleanup:0.40

What Operating System are you using (both controller, and any agents involved in the problem)?
CentOS 7.6
Reproduction steps

create build jobs in low version 2.222.3, for example, please check the job config.xml （change name to config.txt so that I can upload） in attachment.
config.txt

upgrade jenkins to version 2.319.1

The job cannot be shown correctly in job config page.

same properties are shown repeatly
the value of some properties is wrong, for example, the publish server is Pannong_Test1_142 in config.xml, but in the page is shown as demo_env

Expected Results
Build job cannot be recognized correctly in new version
Actual Results
Build job cannot NOT be recognized correctly in new version
Anything else?
No response

That is nothing to do with the docker image here, see https://www.jenkins.io/doc/developer/views/table-to-div-migration/


Prop's hotspot autotrace

Benefit description
Tracing interaction polygons around props is tedious and time-consuming. This hinders rapid prototyping and iterative development.
Adding an auto-trace feature that does the bulk of the work would speed up the process and allow to move the graphics elements on the scene without perceivable impacts on the workflow.
Solution description
Popochiu can show a "Trace polygon" button (can be placed in the topbar alongside the other node-related tools) for the PopochiuClickable elements that have a texture attached (ex. props, characters), which automatically populates the polygon property in the CollisionPolygon2D node, based on the texture content.
The procedure will have to take into account the value of transparent pixels on the texture. Here are a couple of links that may be relevant as a base to implement this logic:

https://godotengine.org/qa/62440/how-create-collisionpolygon2d-from-sprite-programatically
https://godotengine.org/qa/49947/automatically-create-collisionpolygon2d-gdnative-gdscript

Exclusions

This feature won't affect clickable with no sprites or other texture nodes, like Hotspots, for example.
In the case of a multi-frame animation, we may probably just rely on the first frame and let the dev adjust the collider in case there is a big difference between frames of the animation.

Implications

If #8 is implemented as described, each prop sprite will be as large as the whole background. The logic has to identify non-transparent pixels and ignore them to shape the collider.
Always related to #8, if it's implemented, we should expose a flag (either global or for each prop) in the importer interface to trigger the auto-trace feature during the import, allowing a one-click room creation for those using Aseprite.

A couple relevant links:

https://godotengine.org/qa/62440/how-create-collisionpolygon2d-from-sprite-programatically
https://godotengine.org/qa/49947/automatically-create-collisionpolygon2d-gdnative-gdscript

@drbbgh worked on that and has a working POC.
The result works like a charm but introducing it is dependant on #243 and some changes to the importers interface.
This should make its way into 2.1 anyway.


Explorer window auto expands all projects/folders every time you run VS code

Issue Type: Bug
This is occuring as of version 1.61.0.  Versions before this do not exhibit this problem.
Create an install of VS Code, with only the basic java extensions.
Create a workspace, add some projects with folders.
Collapse all folders/projects
Exit VS Code
Restart VS Code
All projects/folders are auto expanded.
It used to remember your last expanded state of all folders.
VS Code version: Code 1.61.2 (6cba118ac49a1b88332f312a8f67186f7f3c1643, 2021-10-19T14:57:20.575Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 3192)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.96GB (24.04GB free)

Process Argv
--crash-reporter-id 392ddeb3-a168-4a50-824b-15c580a08b57

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

remote-wsl
ms-
0.58.4

quicktype
qui
12.0.46

java
red
1.0.0

vscode-checkstyle
she
1.4.1

vscodeintellicode
Vis
1.2.14

vscode-java-debug
vsc
0.36.0

vscode-java-dependency
vsc
0.18.8

vscode-java-pack
vsc
0.18.6

vscode-java-test
vsc
0.32.0

vscode-maven
vsc
0.34.1

/duplicate and fixed in upcoming 1.62 due to ship later this week.


Call to a member function addFile() on null

Hi, thanks a lot for the package. I'm having some trouble zipping a set of four images. This is done by triggering the saved() event. However, when I do this:
SizeTexture::saved(function($st)
{
	$files = array($st['image_diffuse'], $st['image_nrm'], $st['image_spec'], $st['image_disp']);
	$zipper = new \Chumper\Zipper\Zipper;
	$path = realpath(public_path()) . '/uploads/textures/';

	// We'll create the zip even if we only have one image

	foreach ($files as $file) {
		if ($file != '') {
			$zipper->add($path . $file);
		}
	}

	$zipper->make('file.zip');
});

I get the following error:
Call to a member function addFile() on null

I'm not entirely sure that this is a problem with the plugin, because my $path points to the full path of the file in my disk, not as a web address (Something like h:\folder\public\uploads...). Do you have any clues?
Thanks in advance!

You need to call make() first. That will create the zip package, then you can add Files and later save it.
I merged a PR with documentation, may you give it a try again.


Audio has issues with looping

I'm encountering several issues with audio related to looping.
First: It doesn't do anything other than loop. This seems to be due to an error in the marker system. The _marker property on Audio is an array ([]), but the object attempts to set properties on it by string (this._marker["default"]=...). This fails. The loop property that is set on the default marker is thus lost, and the object defaults to looping even if it was disabled.
Second: It doesn't seem to replay after the first playthrough. This may be a Web Audio API issue.
For sound effects, this is passably OK. The audio plays, loops, and goes silent. The result is that a sound effectively plays once. However, the background resources involved may be problematic.
For music and looping effects, this is not good.

The looping problem was corrected by replacing [] with {}. Further issues not yet resolved.

@BenjaminHarding has fixed several audio errors. This appears to have resolved the issue.


Create LICENSE

Pull over the same license from the main repo just for easy reference.

Ping.

Ping.


Got a traceback after updating to the latest version

Hi, i don't know what has gone wrong but i got this log when ran my script. I hope you can look into it.
Traceback (most recent call last):
  File "/home/runner/work/learn-java/learn-java/main.py", line [9](https://github.com/John4650-hub/learn-java/actions/runs/4149066754/jobs/7177685489#step:4:10), in <module>
    response = reader.instantiate(filters=(filter1, filter2, filter3))  # Apply multiple filters at the same time
  File "/home/runner/.local/lib/python3.[10](https://github.com/John4650-hub/learn-java/actions/runs/4149066754/jobs/7177685489#step:4:11)/site-packages/gmailconnector/read_email.py", line 98, in instantiate
    return_code, messages = self.mail.search(None, filters)
  File "/usr/lib/python3.10/imaplib.py", line 734, in search
    typ, dat = self._simple_command(name, *criteria)
  File "/usr/lib/python3.10/imaplib.py", line [12](https://github.com/John4650-hub/learn-java/actions/runs/4149066754/jobs/7177685489#step:4:13)30, in _simple_command
    return self._command_complete(name, self._command(name, *args))
  File "/usr/lib/python3.10/imaplib.py", line 968, in _command
    raise self.error("command %s illegal in state %s, "
imaplib.IMAP4.error: command SEARCH illegal in state AUTH, only allowed in states SELECTED

@John4650-hub This means that any one of the filter passed is invalid. I see you're passing 3 filters. Can you post the values of it.

These are the 3
filter1 = Condition.since(since=date(year=2023, month=2, day=11))
filter2 = Condition.subject(subject="Job-Logs-for-learn-java")
filter3 = Category.unseen

First i ran it with the last filter and got the same message.
Also i ran the script today.

I fixed the bug in v0.6.8

@thevickypedia  i am still getting the same Traceback
Traceback (most recent call last):
  File "/home/runner/work/learn-java/learn-java/main.py", line [9](https://github.com/John4650-hub/learn-java/actions/runs/4152254739/jobs/7183087260#step:4:10), in <module>
    response = reader.instantiate(filters=(filter1, filter2, filter3))  # Apply multiple filters at the same time
  File "/home/runner/.local/lib/python3.[10](https://github.com/John4650-hub/learn-java/actions/runs/4152254739/jobs/7183087260#step:4:11)/site-packages/gmailconnector/read_email.py", line 91, in instantiate
    return_code, messages = self.mail.search(None, filters)
  File "/usr/lib/python3.10/imaplib.py", line 734, in search
    typ, dat = self._simple_command(name, *criteria)
  File "/usr/lib/python3.10/imaplib.py", line [12](https://github.com/John4650-hub/learn-java/actions/runs/4152254739/jobs/7183087260#step:4:13)30, in _simple_command
    return self._command_complete(name, self._command(name, *args))
  File "/usr/lib/python3.10/imaplib.py", line 968, in _command
    raise self.error("command %s illegal in state %s, "
imaplib.IMAP4.error: command SEARCH illegal in state AUTH, only allowed in states SELECTED

Here is my script that reads the email
from datetime import date

from gmailconnector import ReadEmail, Folder, Condition, Category

reader = ReadEmail(gmail_user='myemail@gmail.com',gmail_pass="myAppPassword",folder="important")  # Folder defaults to inbox
filter1 = Condition.since(since=date(year=2023, month=2, day=11))
filter2 = Condition.subject(subject="Job-Logs-for-learn-java")
filter3 = Category.unseen
response = reader.instantiate(filters=(filter1, filter2, filter3))  # Apply multiple filters at the same time
if response.ok:
    for each_mail in reader.read_mail(messages=response.body, humanize_datetime=False):  # False to get datetime object
        print(each_mail.date_time.date())
        print("[%s] %s" % (each_mail.sender_email, each_mail.sender))
        print("[%s] - %s" % (each_mail.subject, each_mail.body))
else:
    print(response.body)

Also i would like to know about the .env,  i created one file name cred.env and saved my credentials in it but got some error that meant that the credential were missing so now i just
pass them in the way they are.

I'd like to call out three things

Make sure you're on the latest version. I'd suggest running

pip uninstall --no-cache-dir gmail-connector
pip install --no-cache --upgrade gmail-connector
pip freeze | grep gmail-connector

Verify if the output is gmail-connector==0.6.8

The folder you're passing should not be a string.
Change folder="important" to folder=Folder.important

To load the env variables automatically you should rename cred.env to just .env

Let me know how that goes.

It was a success. Thanks.


Add the ability to use lookaside caches for targets

Issue
Give MetricsEndpointProvider the ability to have a lookaside source for targets which is not always in relations/relation data
Solution
Invoke a Callable to get an a point-in-time list of jobs, regardless of where the object is instantiated. Make the method to set the scrape job spec public.
Release Notes
Add the ability to use lookaside caches for targets

Just curious why not leave it for the user to do in charm code:
self._scraping = MetricsEndpointProvider(
            self,
            relation_name="...",
            jobs=self.the_usual_scrape_job + self.that_lookaside_callable(),
            ...,
        )


[11.x] Alternative fix for booted callbacks being called twice

Fixes https://github.com/laravel/framework/issues/53589
This is an alternative implementation of a fix submitted in the PR https://github.com/laravel/framework/pull/53683
In consideration of potential breaking changes in the previous PR, this implementation still allows booted callbacks to be pushed onto the array while not calling the callback immediately if booted callbacks are currently being processed.

I'd only swap the last two lines on the Application@boot() method.
Before:
https://github.com/laravel/framework/blob/8ffd13c6c66d431450a08d78005feac2c332c72b/src/Illuminate/Foundation/Application.php#L1090-L1108
After:
public function boot()
{
    if ($this->isBooted()) {
        return;
    }

    // Once the application has booted we will also fire some "booted" callbacks
    // for any listeners that need to do work after this initial booting gets
    // finished. This is useful when ordering the boot-up processes we run.
    $this->fireAppCallbacks($this->bootingCallbacks);

    array_walk($this->serviceProviders, function ($p) {
        $this->bootProvider($p);
    });

    $this->fireAppCallbacks($this->bootedCallbacks);

    $this->booted = true;
}

As I commented in the issue, it solves the issue, and IMO, it is the less intrusive option.
Also, to me, it makes more sense to mark the application as booted as the very last thing on the Application@boot() method.

@rodrigopedra I wholeheartedly agree that would be the better solution, although my only concern would be that it could be a breaking change for any app that expects App::isBooted() to return true during a callback.

@simonworkhouse that is true.
Although Application@isBooted() is not part of the Illuminate\Contracts\Foundation\Application contract, and it is only called within the Application class within the framework.
Also, IMO, checking Application@isBooted() within a callback registered as a "booted" callback seems very unlikely.
Nonetheless, maybe this could be sent to master, 12.x is around the corner anyway.

I'm just not sure I have an appetite for changing this behavior right now.

@taylorotwell Understandable, although it may be worth having someone update the documentation mentioning this behaviour.


Acr.XamForms.UserDialogs Showloading issue on WinPhone

When using Showloading from acr.userdialogs it gives the issue on WinPhone 8.1
e.ExceptionObject).StackTrace	"   at Microsoft.Phone.Controls.CustomMessageBox.ClosePopup(Boolean restoreOriginalValues)\r\n   at Microsoft.Phone.Controls.CustomMessageBox.<>c__DisplayClass4.<Dismiss>b__1(Object s, EventArgs e)\r\n   at Microsoft.Phone.Controls.Transition.OnCompleted(Object sender, EventArgs e)\r\n   at MS.Internal.CoreInvokeHandler.InvokeEventHandler(Int32 typeIndex, Delegate handlerDelegate, Object sender, Object args)\r\n   at MS.Internal.JoltHelper.FireEvent(IntPtr unmanagedObj, IntPtr unmanagedObjArgs, Int32 argsTypeIndex, Int32 actualArgsTypeIndex, String eventName)"	string
this looks similar to the issue at https://github.com/aritchie/acr-xamarin-forms/issues/26

Please copy the whole exception.  This is missing some key points of the stack trace that is needed.

This is what is get in the complete stack trace. tried to find more.
This happens when using
using(var dialog = Acr.UserDialogs.Instance.Loading("Please wait"))

You've actually cut off part of the stack trace above.  I need the full stack trace please.

That the only info i get from stacktrace from Application_UnhandledException handler in App.xaml.cs

Log the exception yourself then.  Without that whole stack trace, there isn't anything for me to look into.

Without more details on this error, I can't be of much help.  If you get the rest of the exception details, please reply to this thread and I will reopen this.

Hi @aritchie
I have facing same issue
Please have a look below StackTrace:
at Microsoft.Phone.Controls.CustomMessageBox.ClosePopup(Boolean restoreOriginalValues)
at Microsoft.Phone.Controls.CustomMessageBox.<>c__DisplayClass4.b__1(Object s, EventArgs e)
at Microsoft.Phone.Controls.Transition.OnCompleted(Object sender, EventArgs e)
at MS.Internal.CoreInvokeHandler.InvokeEventHandler(Int32 typeIndex, Delegate handlerDelegate, Object sender, Object args)
at MS.Internal.JoltHelper.FireEvent(IntPtr unmanagedObj, IntPtr unmanagedObjArgs, Int32 argsTypeIndex, Int32 actualArgsTypeIndex, String eventName)

@pavan7parekh As with the original poster, you have missed a chunk of the exception that I need to even begin to assess the problem.  As it is, I am preparing to drop any sort of support for WP8


How to set a scope based on the current working directory and git branch

Before this latest update, I was able to achieve a custom scope by using the following setting:
scope = 'git_branch',
  scope = require('grapple.scope').suffix(
    require('grapple').resolvers.directory,
    require('grapple').resolvers.git_branch_suffix,
    {
      persist = true,
    }
),

This would be used to restrict the scopes to child directories in a project that are also restricted to a branch based on the git repository of the parent.
// git repo
~/Projects/foo

// frontend sub-project
~/Projects/foo/frontend

// admin sub-project
~/Projects/foo/admin

Given the above project structure, ideally I would want to open up a separate neovim instance in each of those and the scope above would generate three discrete scopes:
.../foo#<my-feature-branch>
.../foo/frontend#<my-feature-branch>
.../foo/admin#<my-feature-branch>

The goal for something like this would be able to maintain separate scopes within a single repository based on the sub-folder that I'm working in.
How would I be able to achieve something similar to the scope I was using under the new API changes?

Hey @adaviloper! Thanks for following up with this question here.
A couple good starting points:

The README: https://github.com/cbochs/grapple.nvim?tab=readme-ov-file#grapplescope_definition
The definition of git_branch: https://github.com/cbochs/grapple.nvim/blob/main/lua/grapple/settings.lua#L84-L110

In your case, you may be able to (almost verbatim) use git_branch by simply swapping the path used for root. Maybe try something like this in your settings.scope?
{
    name = "cwd_branch",
    desc = "Current working directory and git branch",
    fallback = "cwd",
    cache = {
        event = { "BufEnter", "FocusGained" },
        debounce = 1000, -- ms
    },
    resolver = function()
        local git_files = vim.fs.find(".git", { upward = true, stop = vim.loop.os_homedir() })
        if #git_files == 0 then
            return
        end

        local root = vim.loop.cwd()

        local result = vim.fn.system({ "git", "symbolic-ref", "--short", "HEAD" })
        local branch = vim.trim(string.gsub(result, "\n", ""))

        local id = string.format("%s:%s", root, branch)
        local path = root

        return id, path
    end,
},

@cbochs This seems to work perfectly! Thanks so much for helping with this!


feat: add project site

Proposed Changes
https://influxcommunity.github.io/influxdb3-csharp/

Checklist

[x] Rebased/mergeable
[x] Tests pass
[x] Commit messages are conventional
[x] Sign CLA (if not already signed)

Codecov Report

Merging #21 (4ce5a55) into main (a49388f) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##              main       #21   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          736       736
=========================================
  Hits           736       736


Allow release stage ENV

Currently the release stage is not different from the NODE_ENV. It really should be.

Closing due to inactivity. Let us know if this is causing you an issue.


experimentalDecorators Typescript warning always present - Visual Studio 2017 Professional 15.7.4

Hi,
I've looked through the discussion at https://github.com/Microsoft/TypeScript/issues/9335 and seems that other people report the problem as well, though the original thread has been closed recently.
I have this issue (apart from a dozen of other TS errors I haven't been able to solve so far - maybe they are somehow correlated) in my pet project https://github.com/pablonautic/magiczny-miecz - please fill free to clone and build it.
The problem can be observed in \magiczny-miecz\MagicSword.Core\MagicSword.Core.Web\ClientApp\src\app\game\EventBus.ts line 6-7.

This is not a support forum.
Questions should be asked at StackOverflow or on Gitter.im.
As was repeated multiple times in that issue, your tsconfig.json does not specify files or an include glob that includes the file you are having issues with.  Because people repeatedly were looking for basic support, the core team closed the issue.
Why did you ignore the issue template?

I am very kindly sorry, this is due to my frustration - I've been struggling for the last 4 hours to get anything working… without bigger successes.
To clarify - my tsconfig.json comes directly from the default ASP.NET Core Angular template, so below provide the proper description:
TypeScript Version:  2.9

Search Terms: experimentalDecorators
Code
Under VS 2017 -> Add new project -> ASP.NET Core 2.1 -> Angular application.
Add the file below anywhere in the /src folder:
import { Injectable } from "@angular/core";

@Injectable()
export class Foo {

}

Expected behavior:
Projects builds without problems.
Actual behavior:
Severity	Code	Description	Project	File	Line	Suppression State
Error	TS1219	(TS) Experimental support for decorators is a feature that is subject to change in a future release. Set the 'experimentalDecorators' option to remove this warning.	WebApplicationTest	C:\Users\pawel_000\Documents\Visual Studio 2017\Projects\WebApplicationTest\WebApplicationTest\ClientApp\src\app\home\Foo.ts	4	Active
Playground Link:
Related Issues:
#9335

Doesn't the above imply that the default tsconfig.json should contain such entry? Or maybe it's just the wrong GH repo to report this?
Or maybe it' not an issue at all?
P.S. Adding the include section, as described doesn't seem to affect the problem :(

Finally, I have figured out the problem.
Turned out that it's a side effect of a "feature" present in Visual Studio 2017: ignoring tsconfig.json when not set to "Compile".
That's exactly what happens after creating a new project using the ASP.NET Core Angular template - the build action of tsconfig.json defaults to None which causes this one and a dozen of other ts compilation errors (of course, in runtime, there are no problems).
Ergo, changing the action to Compile and reloading the solution causes the problem to magically disappear.

Thank You pablonautic, this warning was driving me nuts.  A simple change of the build option on tsconfig.json and problem solved !!

Wow thanks, only I needed add

Never

in my .csproj and problem solved!!!

@pablonautic spent hours trying to figure out what was wrong and scouring SO for help. Seems like such a simple thing for the ASP.NET Core template to miss.

Is it that hard to have the Build action for tsconfig.json automatically set to Content?

I'm having the same problem but in a "Folder" instead of a real Project+Solution. Because there's no project file the Properties window is blank and doesn't let me specify a build-action for the tsconfig.json files. I note this problem only started happening recently (perhaps since TypeScript 3.5 shipped?).

Thanks guys, by changing to content these warnings finally went away, after bothering me for weeks

This "Feature" is STILL present in the angular template. Set the build action of tsconfig.json to content, reload project and it now works.

In my case I just needed to set Build Action of tsconfig.json to Content. But I did on the file's Properties window instead of the .njsproj file.

I've been having this issue for over a month and looked it up regularly. Since Google mostly shows VS Code issues, i didn't find the solution.
Thank you, @pablonautic . Setting it as Content solved the issue.

4 years later and this is issue is still present in Visual Studio. Thank you @pablonautic - was driving me nuts.


Strict predicate

This PR adds a config setting strict_predicate_matchers which makes the checks for the dynamic predicate matchers strict. Default is false but for internal specs the setting is changed to true in spec_helper.
It changes the failure messages in the non-strict case from "expected ... to return true/false, ..." to  "expected ... to be truthy/falsey, ...".
It builds on #1195

Converted to draft as it requires pre-requisite PRs to be approved/merged first.

Rebased. CI errors are due to issues in the rspec-core gem, addressed in rspec/rspec-core#2756

Should be all fixed.

Thank you @JonRowe ❤

Thank you, @marcandre !

Yes thank you @marcandre and thank you for your patience :joy:

Will it be possible to make a release?

I'm aiming to find some time this week to release 3.10 which would contain this.

@JonRowe Any hope for a release?

A release was being prepared but is currently blocked by rspec/rspec-expectations#1221, that needs a fix before a minor can be shipped.

Released as 3.10.0

Awesome @JonRowe thanks for letting me know 🎉


old npm module .

npm WARN deprecated<EMAIL_ADDRESS>This package is deprecated in favor of https://github.com/istanbuljs/babel-plugin-istanbul
npm WARN deprecated<EMAIL_ADDRESS>This package is deprecated. Use Object.assign.
npm WARN deprecated<EMAIL_ADDRESS>Renamed to supports-color. If you're using chalk, upgrade to the latest version. https://github.com/chalk/supports-color
npm WARN deprecated<EMAIL_ADDRESS>to-iso-string has been deprecated, use @segment/to-iso-string instead.
npm WARN deprecated<EMAIL_ADDRESS>Jade has been renamed to pug, please install the latest version of pug instead of jade
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue

Yep, I know of this, but those are not direct dependencies, but rather dependencies we are building on. One would need to trace them back to their root module.
I think some of them are coming from yeoman, like the jade and chalk dependency.


clean up tests

This package seems pretty unmaintained now. Since I need to make quite a number of changes, i'm going to maintain my own fork for now.


Can not send message server to client

Hello i am new to watsontcp and its cool thats its simple but for some reason i cant send from my server to my client
but my client sends the message to the server
Server:

Client:

i even tried client.Guid


Add Custom Scale for Line Plot

Background
As soon as #27 is merged, we'll have a simple working line plot available to use! However, we would like programmers to be able to decide the scale for themselves (for example, whether every "x" should represent one, two, five, and so on).
Goal
Modify the line_plot helper to take an additional configuration argument that states the scale. Assume that it will be formatted like this -
{
    scale: 5 # Every "x" equals five of something
}

If the user doesn't specify a scale, then use use some math to determine the scale yourself. Here's the code that does that -
val_per_x = (largest_value.to_f / 10).ceil()

data.each do |object|
    object[:length] = object[:value].to_f / val_per_x
end

However, this isn't ideal because a line plot should be discrete. If you use this math, some marks will need to be fractional. To get around this, it would be ideal to just scale the "x" marks down. For example, if you need every "x" to be worth two, just scale the mark down by half and keep them worth at one. This way...

You won't have fractional marks (which will be difficult to render and confusing for the user), and
You will still be able to fit all the data onto the plot

Good luck!

Claimed for @raj978

Update - since two weeks have expired, this issue is now free to claim again! Several contributors can claim at once, and the "best" PR will be merged (although we may use elements from multiple contributors if it's helpful).

I could give this one a try.


Child view loaded twice when object id is a url (hateos api)

When using a hateos style rest api, the object id's are url's, that the client follows to get more details. In this case, when using ui-sref in the template, the child view is loaded twice, and the resolves also run twice. And the active state is lost.
Plunker demonstrating the issue
I've documented a workaround in the plunker readme, in case anyone else is having this issue.

When using a hateos style rest api

:^5: for good architectural style.
@christopherthielen This looks like a new incarnation of the old encoding/double-transition bug. The problem is the slashes in the URL parameters. I tested the plunkr against master and it's still an issue. The encoding/decoding isn't symmetrical, so $urlRouter is seeing the resulting URL as being different from the one that initiated the transition.

those pesky slash encodings...  I had thought I had the encoding/decoding working symmetrically at one point.
@nateabele got any thoughts on a better way to manage this without the browser's built in decoding getting in the way?  A custom serialization that avoids url encoding in certain cases, or something?
@danielabar note: you can fool around with making a custom Type to make your use case work.  Change your url from "/:id" to use the built-in any type, for example: "/{id:any}" and the url does not get encoded at all, and your plunkr behaves better.

@christopherthielen I merged my PR for raw types to 1.0. I'll set up a test against that later today.

I'd like this to work with string Type. Any string (including those with
slashes) should have symmetrical encoding
On Mon, Sep 21, 2015 at 10:53 AM Nate Abele<EMAIL_ADDRESS>wrote:

@christopherthielen https://github.com/christopherthielen I merged my
PR for raw types to 1.0. I'll set up a test against that later today.
—
Reply to this email directly or view it on GitHub
https://github.com/angular-ui/ui-router/issues/2250#issuecomment-142024161
.

http://plnkr.co/edit/SH4rrZ?p=preview


Cygno libs update to accomodate MANGO and make it more flexible in general

First commit:
Simply added the digitizer correction tables for the MANGO digitizer. These were created by Stefano and me and seem to work well.
Some channels are not very good but at the moment it's enough.
Changed code to see MAN corrections tag.
Second commit:
Changed dgz_full2array to read any number of digitizers (loops on the available digitizers in the data bank).
This makes the code suitable to analyze 1 or 2 digitizers.
The structure is made so that other different digitizers could also be added in the future, although this would require further changes in the reco code.
These modifications were tested with all three detectors: LNGS, LNF, MANGO, with and without PMT readout, in debug and normal modes, for a reasonable amount of runs.

After looking at the code and followed some of these control checks, I believe the PR can be accepted


Please invite me to the Infraform Community

Name
Dylan Hood
Additional Context
Please invite me to the Infraform Community

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is a remarkable open-source solution tailored for serverless platforms, aimed at empowering creators to craft their infrastructure solutions. 💻 We're thrilled to welcome you into our vibrant community! ✨ We eagerly anticipate your valuable contributions! 🤓 Don't forget to explore our community on Github and connect with us on LinkedIn 💜 Feel free to spread the word about our community to like-minded individuals interested in development, DevOps, technology, and collaborative contributions. 👉💜👈


[advriskmin] include chunk (+ deep dive) on proper scoring rules

include material on proper scoring rules, basic idea / definition with video, plus deep-dive mathy proof chunk

moved to wish list


latest daily build on master fails to install istio via istioctl

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities/
To ask questions about how to use Istio, please visit https://discuss.istio.io
)
Bug description
istio-1.6-alpha.8f817062b73d110f81113363bbcaed67fdbb523c bin/istioctl manifest apply

This will install the default Istio profile into the cluster. Proceed? (y/N) y
Error: failed to generate and apply manifests, error: failed to generate manifest: could not read translateConfig file translateConfig/translateConfig-1.6.yaml: Asset translateConfig/translateConfig-1.6.yaml not found

Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)
Additionally, please consider attaching a cluster state archive by attaching
the dump file to this issue.

I think this was resolved, @elfinhe please reopen if that's not the case.


Adding Reader/Writer tests

Adding Reader/Writer tests to Cypress UI automation.

Adding Reader/Writer tests to Cypress UI automation.


Try Windows

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

AppVeyor hook failing. Not sure why. Please hold on this until we get that sorted.

AppVeyor is now running.


"Copy as path" not available on folders

I'd love to be able to copy the folder path as well.

Thanks! I haven't looked at this in a while - sorry about that. I'll look into adding support for that.

Fixed with #5.


[Request] Add AQI sensor

I have a HP04, and the app shows an AQI value calculated from the 4 sensors (PM2.5, PM10, NO2, VOC). Could this be added to the integration?

@Hampo That's a great question! It might be possible, I'll investigate and get back when I have answers.

Thank you :D

Hi @dotvezz , can you implement AQI sensor?
Thanks a lot

Hi everyone. Can you please share the models you use (other than @terzo33 who has already done so, of course.
Additionally, there's a tool you can use to help identify the sensors available on your device:  (not well-documented yet, that's on my to-do list) called opendyson. If you're willing to help debug this, you can use it to get the values directly from your fan.
You can run opendyson devices to list your devices, find the serial number for the specific device you want to pull information from. Then run opendyson listen YOUR-SERIAL-NUMBER and it will start printing information from your device to the screen. Here's an example below.
$ opendyson devices
Use China region for account? (Default is no) [y/n]: n
Email Address<EMAIL_ADDRESS>Code from confirmation email: 123456
Password:
Available devices:
- basedevice:
    name: Dyson Fan
    serial: NM7-US-XXXXXXXX
    model: TP02
    type: "475"
    variant: ""
    productCategory: ec
    connectionCategory: wifiOnly
  mqtt:
    password: REDACTED
    username: NM7-US-XXXXXXXX
    root_topic: "475"
    address: <IP_ADDRESS>
  iot:
    endpoint: REDACTED
    client_id: REDACTED
    custom_authorizer_name: REDACTED
    token_key: REDACTED
    token_signature: REDACTED
    token_value: REDACTED
  firmware:
    version: 21.04.03
    auto_update_enabled: false
    new_version_available: false

This is sensitive information that can be used to remotely control your Dyson devices. Please take caution before sharing it with anyone you don't trust.

$ opendyson listen NM7-US-XXXXXXXX
Subscribing to 475/NM7-US-XXXXXXXX/status/current
Subscribing to 475/NM7-US-XXXXXXXX/status/fault
Subscribing to 475/NM7-US-XXXXXXXX/command
Command:  {"msg": "REQUEST-PRODUCT-ENVIRONMENT-CURRENT-SENSOR-DATA", "time": "2024-07-08T16:32:03Z"}
Status:   {"msg":"ENVIRONMENTAL-CURRENT-SENSOR-DATA","time":"2024-07-08T16:32:03.000Z","data":{"tact":"2970","hact":"0052","pact":"0000","vact":"0000","sltm":"OFF"}}
Command:  {"msg": "REQUEST-PRODUCT-ENVIRONMENT-CURRENT-SENSOR-DATA", "time": "2024-07-08T16:32:33Z"}
Status:   {"msg":"ENVIRONMENTAL-CURRENT-SENSOR-DATA","time":"2024-07-08T16:32:33.000Z","data":{"tact":"2970","hact":"0052","pact":"0000","vact":"0000","sltm":"OFF"}}

If you're able to get this to work, please share two things:

The value of your device's firmware version field (maybe there's a firmware-specific bug to be aware of)
The the output of opendyson listen after letting it run for a while. We can review the values to see if there's something to be worried about.

As mentioned in the OP, I have a HP04. Firmware version is ECG2PF.<IP_ADDRESS>2, and the logs look like this:
Status:   {"msg":"ENVIRONMENTAL-CURRENT-SENSOR-DATA","time":"2024-07-15T12:24:58.000Z","data":{"tact":"2963","hact":"0054","pm25":"0003","pm10":"0001","va10":"0004","noxl":"0004","p25r":"0003","p10r":"0003","sltm":"OFF"}}
Status:   {"msg":"ENVIRONMENTAL-CURRENT-SENSOR-DATA","time":"2024-07-15T12:25:28.000Z","data":{"tact":"2965","hact":"0053","pm25":"0003","pm10":"0002","va10":"0004","noxl":"0003","p25r":"0003","p10r":"0004","sltm":"OFF"}}

Thanks for looking into this

@Hampo thanks! Sorry I missed the HP04 part of your original comment :sweat_smile:.
Interestingly, there's a thread going on in #159 with TP02/475 devices where the MQTT value labled pact is used for the AQI sensor. Interestingly, that's actually labeled as a "particulate" value elsewhere. I wonder if this is another example of slightly wrong assumptions when Shen and the original contributors set up the earlier version of this integration. There's more discussion about it in #4 and a bunch of other threads, but basically some of the Home Assistant entities were set up based on slightly wrong (but very close) assumptions about how some of the physical sensors were reporting raw data.
Getting to your HP04, I see the following environment fields in your message (thanks so much for grabbing that for this issue!)

tact: currently unused in this integration. Unknown what its meaning is.

I'll look into this to see if it's something helpful for us!

hact: humidity
pm25 and p25r: Particulate Matter (PM2.5)
pm10 and p10r: Particulate Matter (PM10)
va10: Volatile Organic Compounds
noxl: Nitrogen Dioxide
sltm: Sleep Timer status

I'll try to find out what tact is, but it's likely that the calculation for AQI is proprietary and happening in the app itself. If that winds up being the case, then this might not be easy to do. Reverse-engineering HTTP and MQTT messages in the MyDyson app has been considerably easier than other things, since there are clearly related landmarks to look for in the decompiled code.

So, according to the APK's source code, tact is meant to be the temperature. My best guess is that it's Kelvin * 10, because 296.5K is 23.35C, which is what my room is.
From the same source, here's all the values:
Code - Type                  - Bounds
tact - Temperature           - 2430-3530
pact - Gen1 Dust             - 0-9
vact - Gen1 VOC              - 0-9
pm25 - PM25                  - 0-149
pm10 - PM10                  - 0-149
va10 - Gen2 VOC              - 0-99
noxl - NO2                   - 0-99
p25r - High Res PM25         - 0-999
p10r - High Res PM10         - 0-999
hact - Humidity              - 0-99
hcho - Formaldehyde          - 0-99
hchr - High Res Formaldehyde - 0-999
co2r - CO2                   - 0-9999

I haven't found the AQI formula offhand, but hopefully these help in the meantime

tact is meant to be the temperature . . . Kelvin * 10,

@Hampo Ahhh yep, you're totally right. I must have fat-fingered my code search because it's even being used by this integration.

I noticed the same with my Dyson Purifier Hot+Cool Formaldehyde: no AQI sensor.
I'm now looking at another interesting integration for home assistant that just does the calculation : https://github.com/Limych/ha-iaquk
Maybe this can be integrated?

@pprazzi interesting... interesting indeed. Looks like that integration uses an index defined here: https://github.com/Limych/ha-iaquk/blob/dev/IAQ_Rating_Index.pdf. I don't really know if that is the same index Dyson uses or not, and unfortunately it looks like the website for the organization isn't around anymore.
But it's something. Maybe I'll just implement that math into an AQI sensor for Dyson devices?
Trying to weigh the pros and cons here. The main con I can think of is possible user confusion if the AQI doesn't match what the app displays.

@dotvezz Hi Ben, here's another calculation of an AQI, more specific targeted to Dyson purifiers : https://aarongodfrey.dev/home automation/dyson-tp04-integration/

Here's a (very rough and ready) Jinja template which tries to derive the Air Quality based on the PDF linked earlier
https://github.com/Limych/ha-iaquk/blob/dev/IAQ_Rating_Index.pdf
Obviously its not optimal, but might aid in developing a custom calc
Ive scaled down the quality intervals, as were only using a subset of measurements.
Also its unclear whether this is even directly translatable, since Nitrogen and a few other values are quoted as being an 'index' value. However, they seem do to correspond to roughly the range of  values youd expect based on that PDF
The Air Quality seems to be corresponding with what the Dyson app is showing (so far, anyway)..
{% set co2 = states('sensor.dyson_carbon_dioxide') | float %}
{% set temperature = states('sensor.dyson_temperature') | float %}
{% set humidity = states('sensor.dyson_humidity') | float %}
{% set no2_index = states('sensor.dyson_nitrogen_dioxide_index') | float %}
{% set voc_index = states('sensor.dyson_volatile_organic_compounds_index') | float %}
{% set hcho = states('sensor.dyson_hcho') | float %}
{% set pm_2_5 = states('sensor.dyson_pm_2_5') | float %}
{% set pm_10 = states('sensor.dyson_pm_10') | float %}

{% set co2_points =
5 if co2 <= 600 else
4 if 601 <= co2 <= 800 else
3 if 801 <= co2 <= 1500 else
2 if 1501 <= co2 <= 1800 else
1
%}

{% set temperature_points =
5 if 18 <= temperature <= 21 else
4 if (17 <= temperature < 18) or (21 < temperature <= 22) else
3 if (16 <= temperature < 17) or (22 < temperature <= 23) else
2 if (15 <= temperature < 16) or (23 < temperature <= 24) else
1
%}

{% set humidity_points =
5 if 40 <= humidity <= 60 else
4 if (30 <= humidity < 40) or (60 < humidity <= 70) else
3 if (20 <= humidity < 30) or (70 < humidity <= 80) else
2 if (10 <= humidity < 20) or (80 < humidity <= 90) else
1
%}

{% set no2_points =
5 if no2_index < 0.2 else
3 if 0.2 <= no2_index <= 0.4 else
1
%}

{% set voc_points =
5 if voc_index < 0.1 else
4 if 0.1 <= voc_index <= 0.3 else
3 if 0.3 < voc_index <= 0.5 else
2 if 0.5 < voc_index <= 1 else
1
%}

{% set hcho_points =
5 if hcho < 0.02 else
4 if 0.02 <= hcho < 0.05 else
3 if 0.05 <= hcho < 0.1 else
2 if 0.1 <= hcho <= 0.2 else
1
%}

{% set pm_2_5_points =
5 if pm_2_5 <= 545 else
3 if 546 <= pm_2_5 <= 1362 else
1
%}

{% set pm_10_points =
5 if pm_10 <= 68 else
3 if 69 <= pm_10 <= 170 else
1
%}

{% set total_score = co2_points + temperature_points + humidity_points + no2_points + voc_points + hcho_points + pm_2_5_points + pm_10_points %}

{% set air_quality_label =
"Excellent" if 36 <= total_score <= 40 else
"Good" if 29 <= total_score <= 35 else
"Fair" if 22 <= total_score <= 28 else
"Poor" if 15 <= total_score <= 21 else
"Inadequate"
%}

Sensor Points:
- CO2: {{ co2_points }}
- Temperature: {{ temperature_points }}
- Humidity: {{ humidity_points }}
- NO2 Index: {{ no2_points }}
- VOC Index: {{ voc_points }}
- HCHO Index: {{ hcho_points }}
- PM 2.5: {{ pm_2_5_points }}
- PM 10: {{ pm_10_points }}

Air Quality Score: {{ total_score }}
Air Quality:  {{ air_quality_label }}

If you're able to get this to work, please share two things:

The value of your device's firmware version field (maybe there's a firmware-specific quirk to be aware of)
The the output of opendyson listen after letting it run for a while. We can review the values to see if there's something available.

Not sure if you still need this, but
Device: BP04
Firmware: 0664PF.<IP_ADDRESS>2
Log: https://gist.github.com/carpii/3e64066a05731132ffb854b121b9cd5b


Grunt contrib compass is giving error - Individual stylesheets must be in the sass directory

I have multiple sass files inside my assetsdirectory. I want to compile a different sass files for different target. I have used specifyoption but still it's showing the error that - 'Individual stylesheets must be in the sass directory'. Below is my grunt task for compass.
compass: {
  options: {
        sassDir: './src/assets',
        cssDir: '<%= build_dir %>/assets/',
        generatedImagesDir: '<%= build_dir %>/assets/images/generated',
        imagesDir: './src/assets/images',
        javascriptsDir: './src/scripts',
        fontsDir: './src/assets/fonts',
        importPath: './vendor',
        httpImagesPath: 'images',
        httpGeneratedImagesPath: 'images/generated',
        httpFontsPath: '<%= build_dir %>/assets/fonts',
        relativeAssets: false,
        assetCacheBuster: false,
        raw: 'Sass::Script::Number.precision = 10\n',
    },
    buildA: { // Target
        options: {
            outputStyle: 'expanded',
            specify: ['./src/assets/appA.scss']
        }
    },
    buildB: { // Target
        options: {
            outputStyle: 'expanded',
            specify: ['./src/assets/appB.scss']
        }
    }
}

What I want to achieve is when I execute grunt compass:buildA, it should compile only appA.scss file not appB.scss. And similarly for grunt compass:buildB it should only compile appB.scss. I hope you can figure out the directory structure.

The problem is the check of the ruby gem compass if the files of buildA.options.specify and buildB.options.specify lie within the sassDir. This check compares the absolute paths of the files with the sassPath (projectPath/sassDir) and expects the sassPath to be the beginning of the file paths. The problem is that the sassPath is not properly constructed for sassDir containing dots.
This should help:

var path = require('path');
..
grunt.initConfig({
..
compass: {
  options: {
        sassDir: path.resolve('./src/assets'),
        ..

Thanks @alexander-nitsche this resolved this issue for me :)


Local bibliographies do not appear in PDF output

Hi!
I introduced some local bibliographies in my Jupyter book by inserting at the end of each chapter,
## Bibliography
```{bibliography}
:filter: docname in docnames
```

The bibliography renders as expected in the HTML output but not in the PDF output, where the bibliography is still collected at the end of the document, while the headers (## Bibliography) keep appearing at the end of each chapter without any content inside.
Is this a known issue? Is there a workaround?
Thanks!

thanks for reporting this @raptistasos. I think the main issue here is that bibliography support in LaTeX relies on Tex (i.e. pdflatex or xelatex etc.) to build the bibliographies through the use of tags. The sphinx.writer is adding  \bibliography at the end of the document to be populated by tex.
Do you know the tex you would use to actually implement this in a LaTeX document?
@AakashGfude this is something we may need to support in this extension (similar to the table of contents issue)

@mmcky @AakashGfude It's me that I must thank you for all the great work that you are doing! In tex, I would use the following template,
https://www.overleaf.com/latex/examples/per-chapter-bibliographies-with-chapterbib/pqzjpxwyyjyd
However, I am not sure if it is compatible with the way that sphinx.writer is adding the bibliography as it dismantles the .bib file and adds any references directly into the .tex file.


Add full synchronous support to transact

Full synchronous support would be nice because it would make setup and debugging of long client-side sequences both simpler and more similar to the sequences in the back-end tests, which (by the time the client-side sequence is under construction) are confirmed working.  transact already does the initial eth_sendTransaction invoke synchronously (if no onSent callback is supplied), but this only supplies the txHash; including the initial fire error check, call return, and mutable return value lookup would allow the whole sequence setup to be written in simple synchronous style (for tests only, of course).

Added in 5cbb84e7157e207dceb9ef4ec9b100f9b68913c3


a single use case question

You edited your whole text away, I don't think that was intentional ?
Looking at the image, I don't think what you want is possible. However, I would add to the question that the slit in the table top (at the bottom of the image) is more concerning to me. I've seen similar artifacts in old VHACD but haven't checked yet whether the new version does the same. If this is done with the new version, is this a known artifact?

I'm away from my computer right now. I will check on this later today or tomorrow and get back to you. Off the top of my head, it might be interesting to try disabling the shrinkwrap option and see if it is the culprit. Use -s false
Let me know if that is the cause.

Woops, I was on my cell phone. I have access to my computer again now. Can you upload the file as a wavefront OBJ and I can take a look at it?

Ok, fixed on 'master'. I changed the '-l' for logging to '-g' so there wouldn't be a name conflict.


whisper_result_to_srt incorrectly rounds start/end time to integers

Hi and thanks for this utiliy.
There is a problem in whisper_result_to_srt, as segments start/end are rounded to integer values, while the original values can be like "3.8999999999999995". This causes unwanted moves of the subtitles.
https://github.com/peterk/srt_equalizer/blob/8972b568227a440bef68f3ecd8bf67ddb231e240/src/srt_equalizer/srt_equalizer.py#L26

Hi! Thx fir the report. I was under the impression that Whisper only produced integer timestamps (see https://github.com/openai/whisper/discussions/809) but will have a look.


Links in ToC broken with 3.13.0 release

Links in ToC broken with 3.13.0 release
Share Link : https://homebrewery.naturalcrit.com/share/fFPgV7Rj1YC2
Clicking a link in the generated ToC opens a new tab instead of jumping to the location.
Worked an hour or so ago, before the new client was pushed out.
Chrome is up to date
Version 126.0.6478.127 (Official Build) (64-bit)

I believe that this is related to the addition of DOMPurify - see https://github.com/cure53/DOMPurify/issues/317

Huh, thought i had filed an issue on this. Wanted it fixed before release, but we got a comunication problem.

Please confirm fix. Thank you.

I believe that this issue was due to the overly restrictive DOMPurify stripping attributes from the ToC link elements, and the issue is no longer present. PR #3548 achieves the same purpose as the original intention of the DOMPurify PR, but without being nearly as restrictive - testing has not shown any of these issues returning.

I'm going to leave this open until #3548 is applied or closed.


Conclude bzlmod migration

This removes the obsolete WORKSPACE file.
Starting with this commit, the MODULE.bazel.lock file is checked in to the repository to lock down the Bazel-side requirements.
Migrates from the Nix-packaged Bazel to a Nix-packaged bazelisk which fetches the correct .bazelversion. This means that we're no longer dependent on the Bazel from nixpkgs and can update this dependency more freely.

This change is

Needs more testing.


wlroots: 0.6.0 -> 0.7.0, sway: 1.1.1 -> 1.2

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Didn't notice any issues and all rebuilds are fine as well :)


remove loc metadata in links

remove loc metadata in links

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azurestackps-1.2.11/overview.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off


automation: Add task for vtctl MigrateServedTypes.

Extended end-to-end test to verify the new task.
@alainjobart @aaijazi

Nice, LGTM. Couple of points, neither of which are blocking.

LGTM


Can't craft Torch variants + Internal Exception on server

I can't seem to craft any of the Torch variants while on a server but I can in Singleplayer. When trying to give myself a Torch variant in creative mod, the client disconnects with the reason
"Internal Exception: io.netty.handler.codec.DecoderException: Failed to decode packet 'serverbound/minecraft:set_creative_mode_slot"
Minecraft version - 1.21
Fabric API - 0.103.0
Quad - 1.3.0-beta.5 & 1.2.7
All of your More mods updated to latest for 1.21

With Quad 1.3.0-beta.5 I keep getting a crash server quad 1.3.0-beta.5.txt when generating new chunks

Sorry about this but good news, very small thing, I'll fix it right away in version 1.0.2.
And thanks again for reporting, I appreciate it a lot. (:
The server-side crash of Quad is directly related (same issue) and as such will be fixed, too.
Btw: Quad 1.3.0's new feature is block replacement during generation meaning that any village (and other structures, too) can have fitting variants for any mod, e.g. a Savanna village having only Acacia Torches, Acacia Barrels, Acacia Smokers etc. instead of the vanilla blocks.
So one thing I would like to know while we're at it: Which of my mods are you using? I can then add the functionality to those ones first. (:

More Torch Variants is working correctly now. Thank ya!
I have most if not all of your more variants mods actually. I love the extra esthetic your mods add the Minecraft.

Quad 1.3.0's block replacement has been added to all my mods where it is applicable.
I just kept this issue open because I was meaning to come back to you on this one but it must've slipped my mind. Thanks for enjoying my mods. (:


Add N. Martin & J. M. Ruiz spectral response mismatch modifiers model

[x] I am familiar with the contributing guidelines
[x] Tests added
[x] Updates entries in docs/sphinx/source/reference for API changes.
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).

Does not apply (to me):
- [ ] Closes #xxxx

[x] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

Hi! As promised, I did a bigger PR.
This proposal adds a new function, pvlib.spectrum.martin_ruiz_spectral_modifier, with the spectral mismatch model proposed in this paper.
Details about the function can be read in the docstring, but to sum it up, it takes the type of material (m-Si, p-Si or a-Si), the absolute airmass and the clearness index to compute a small mismatch for each component's POA irradiance.
This PR has been done with the help of researchers at Instituto de Energía Solar, at Universidad Politécnica de Madrid.
There are a bit of things I'm unsure about, so I'd like to hear your opinion on them:

The name of the function
The tests names
Missing tests, if any comes to mind
Extensiveness of tests
Output keys
Behaviour when using parameter model_parameters (this is almost only to be used if anyone computes its model params)
Behaviour, in general. Hope submitted notebook spectral_mismatch_modifiers.ipynb and tests clarify the use.

Feel free to make comments on everything. English is not my mother tongue, so an in depth inspection of code comments and docs is also welcome.
Idk if I should add something else here, ask whatever you need, I will address it.
Before merging

plot_martin_ruiz_mismatch.py:

[ ] Rename pvlib.atmosphere.first_solar_spectral_correction to pvlib.atmosphere.first_solar for compat with PR #1628

I'm a bit puzzled by the omission of the leading term S_{ef Gbar(lambda)). As coded, this function returns the ratio S_{ef} / S_{ef Gbar(lambda)). Is this the appropriate factor to apply to the broadband irradiance?

Yes, you are right, the func is returning a ratio. In fact, it returns 3 of them, each one for a different POA component; this mismatch is for all the spectrum (with respect to the standard spectrum) response - it is independent from the wavelength.

I'm a bit puzzled by the omission of the leading term S_{ef Gbar(lambda)). As coded, this function returns the ratio S_{ef} / S_{ef Gbar(lambda)). Is this the appropriate factor to apply to the broadband irradiance?

I am also a bit puzzled.  I think, if this paper describes a factor suitable for multiplying with broadband irradiance, it must be this ratio.  But its formulation, especially the integration limit in the denominator of Eq 1, seems strange to me.  Need to chew a bit more on what this equation really represents.

Thank you very much @kanderso-nrel ! I didn't know many of the things you spotted.

First, there is a duplicate of the Bug Fixes section in the original whatsnew for 0.9.5 on line 31, just in case you didn't notice; I did also not want to touch it.

Second, I don't know how to approach the plot_*.py example since the .ipynb tutorials are being deprecated. In the other .py files, I see many times that they replicate a plot from a paper. Should I make a script to mimic some plot in this Martin & Ruiz paper or can I just translate the notebook to this format? Also, if I want to add more code later to this example (e.g., for obtaining the custom model_parameters dict), could I just reuse the file or do I create a different one?

Regarding the questions about the paper, I will talk to my mentors and let you know in following days.

First, there is a duplicate of the Bug Fixes section in the original whatsnew for 0.9.5

Yes, there was a mistake in some other PR that created this duplication.  We will fix that later so don't worry about it in this PR.  For this PR it's fine to just add a new entry to Enhancements.

Should I make a script to mimic some plot in this Martin & Ruiz paper or can I just translate the notebook to this format? Also, if I want to add more code later to this example (e.g., for obtaining the custom model_parameters dict), could I just reuse the file or do I create a different one?

You can mimic a plot from the reference if you like.  It would also be fine to just translate the notebook to this format.  Basically the examples just need to show how to do something useful; it's up to you to decide what "useful" means ;) We do try to keep them relatively short, so if you want to show how to obtain custom model parameters, it may be better to create a new example file.

I've added the sphinx example. I think this is the minimum needed to merge.
About the doubts that have arisen about the paper, I will try to address them this week with help of the mentors I've mentioned.
If I forgot something, don't hesitate to ask about it again.
@kanderso-nrel thank you very much for your guidance.

If the model requires a specific air mass calculation, then perhaps it is better to have zenith angle as input.

Thanks for the input @adriesse .
I wanted to keep the model as accurate as in the paper, to make it easy to understand from the document as possible. In my opinion, that would obfuscate it little more.
Also, this model introduces something like a 1% change in the incident sunlight, and the difference between 'kasten1966' and the default model 'kastenyoung1989' is almost negligible (see this Sandia page). I like to think that anyone using it will have seen the docs or the example at least, or even if they forgot the model, it won't almost have an impact in the results.

I had a meeting with the profs I mentioned some time ago.
They were doubtful about the reliability / trustworthiness of this model. That's why I added the comparison section in the example. See built readthedocs example.
With that info, I'd like you to see it and determine if the Martin & Ruiz proposed method deserves being among PVLIB.

Have you checked the full range of possible kt and am values to see if the model behaves ok?

I started learning about PV almost by myself last summer as these profs asked me to, in order to use my knowledge in Python to try to make this addition possible. So please, bear with me for some of the scientific inconsistencies I might have done.
Answering @adriesse 's question, no, I didn't. That dataset is generated from a spreadsheet, so most tests only check mathematical workflow & I/O works as expected from the M&R paper. In fact, I do not have a clear path of how to approach the tests, if they are not suitable as of now (I mean, origin datasets, where to find them, how much error I should expect come to my mind right now). If the question only wants to address the range of input values, I can solve it, but it won't determine if it yields the values we want as a spectrum mismatch modifier.
The researchers at Instituto de Energía Solar of UPM, the profs, are trying to explain the difference compared to the other models and come to conclusions in following days/weeks. You will hear from me if I'm told anything.
Whether this PR should close, change to draft status or can be merged, is a decision I will leave up to you.

The comparison in the example does look a bit odd.  Anyway, don't give up.  This would be an interesting addition to pvlib if the wrinkles can be sorted out.  If problems remain with the model, then identifying those is scientific progress too!

I had a little conversation with the researchers and I was told the model won't work for this purpose, so I'm closing this issue.


Add new Json driver.

I'd like to add a new Json driver that can read and write JSON files. It's a pretty simple concept and will be a good opportunity to refactor any duplicated code between drivers.
I'd also like to make the driver configurable - specifying the number of spaces for indentation, etc.

Closes by #8.


npm over bower ?

bower install ng-token-auth --save?

lol, did the title of the question change? I am confused by my own answer..


stop duplicating actions for tabs management when using macOS native tabs

I think it fixes https://github.com/Microsoft/vscode/issues/57421

@agurodriguez thank you so much, no idea how I was not seeing this ❤


Made Changes to Code_of_Conduct.md

Description
Fixes issues Code_Of_Conduct.md structure
Type of change

[x] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation

There were some issues in the Code of conduct.Just wanted to edit them.


Add Prometheus monitoring annotations to envoy, for the cases where Prometheus is used.

From a 1:1 Slack conversation on the CNCF slack with @aryan9600:

I wanted to ask if its possible to make a small change to the default knative installation.
since, the whole point of flagger is to do progressive delivery in a metrics driven way, it’d be great if the envoy deployment that is deployed in the kourier-system ns, could have the prometheus annotations as well? that way all the envoy metrics can be automatically scraped by prometheus. if there’s a way to specify this at install time, please let me know. its a bit hacky to tell users to add annotations to the pod template of a deployment, so i wanna keep that as a last resort.

I believe that Sanskar is asking that we add the following annotations to the Pod template (which I can't easily Google, apparently!):
spec:
  template:
    metadata:
      annotations:
        prometheus.io/scrape: "true"
        prometheus.io/port: "8002"
        prometheus.io/path: "/stats/prometheus"

/assign @evankanderson

(where the port is the correct port on the envoy pod where metrics are published, TBD)

I need this.


Result of the block in the 'given' method are ignored

Hello!
In the Grape 1.0.2 the code below works fine:
params do
  requires :payment_type, type: String, values: ['ccard', 'paypal', 'applepay']
  given payment_type: ->(val) { val == 'applepay' } do
    optional :m_card_holder, type: Hash, as: :card_holder do
      optional :first_name, type: String, allow_blank: false, desc: 'Cardholder first name'
      optional :last_name, type: String, allow_blank: false, desc: 'Cardholder last name'
    end
  end
  given payment_type: ->(val) { !(val === 'applepay') } do
    optional :card_holder, type: String, allow_blank: false
  end
end
post '', root: 'payments' do
  # Payments processing
  # ...
end

When the request were from Apple Pay, the card_holder parameter became Hash and were parsed as Hash,
otherwise it been treated as String. It was fine for a time, but then the Grape were upgraded to 1.5.3,
this behavior became broken. In the version 1.5.3, result of the blocks in the 'given' method are ignored
and 'declared' method resturns all the parameters defined in 'params' without conditions. In the case above, the 'cardholder',
passed from request as string in accordance with payment_type='ccard', parsed by Grape as Hash instead of string,
and causes exception
undefined method `any?' for "NA ME":String

in the line https://github.com/ruby-grape/grape/blob/e83e81fe95d71134f2156aa6f31c22fcf954227f/lib/grape/dsl/inside_route.rb#L61
It is unexpected behaviour and I didn't find any notes about changes in the 'given' methods, it's looks like a bug.

Try writing a spec for this that passed in 1.0.2?

The controller:
module API
  module V3
    class Workflow < Grape::API
      include API::V3::Defaults

      resource "workflow" do
        namespace "grapetest" do
          params do
            requires :payment_type, type: String, values: ['ccard', 'applepay']
            given payment_type: ->(val) { val == 'applepay' } do
              optional :m_card_holder, type: Hash, as: :card_holder do
                optional :first_name, type: String, allow_blank: false, desc: 'Cardholder first name'
                optional :last_name, type: String, allow_blank: false, desc: 'Cardholder last name'
              end
            end
            given payment_type: ->(val) { !(val === 'applepay') } do
              optional :card_holder, type: String, allow_blank: false
            end
          end
          get '', root: 'grapetest' do
            declared(params, include_parent_namespaces: true, include_missing: false)
          end
        end
      end
    end
  end
end

The spec (with rspec-grape):
require 'rails_helper'

RSpec.describe ::API::V3::Workflow, type: :api do

  context "GET /v3/workflow/grapetest" do
    it "return card_holder as hash for applepay" do
      call_res = call_api({payment_type: :applepay, card_holder: {first_name: 'xxx', last_name: 'yyy'}})
      expect(call_res.status).to eq(200)
      expect(JSON.parse(call_res.body, symbolize_names: true).dig(:card_holder)).to be_an_instance_of(Hash)
    end
    it "return card_holder as string for ccard" do
     call_res = call_api({payment_type: :ccard, card_holder: 'cardholder name'})
      expect(call_res.status).to eq(200)
      expect(JSON.parse(call_res.body, symbolize_names: true).dig(:card_holder)).to be_an_instance_of(String)
    end
    it "422 if type is ccard and card_holder is a hash" do
      call_res = call_api({payment_type: :ccard, card_holder: {first_name: 'xxx', last_name: 'yyy'}})
      expect(call_res.status).to eq(422)
    end
  end
end

The result of rspec:
API::V3::Workflow
  GET /v3/workflow/grapetest
I, [2021-09-30T11:01:15.423048 #18937]  INFO -- : {:status=>200, :time=>{:total=>12.0, :db=>0.0, :view=>12.0}, :method=>"GET", :path=>"/v3/workflow/grapetest", :params=>{"payment_type"=>"applepay", "card
_holder"=>{"first_name"=>"xxx", "last_name"=>"yyy"}}, :host=>"example.org", :ip=>"<IP_ADDRESS>", :ua=>nil}

    return card_holder as hash for applepay

I, [2021-09-30T11:01:15.427819 #18937]  INFO -- : {:status=>200, :time=>{:total=>1.73, :db=>0.0, :view=>1.73}, :method=>"GET", :path=>"/v3/workflow/grapetest", :params=>{"payment_type"=>"ccard", "card_ho
lder"=>"cardholder name"}, :host=>"example.org", :ip=>"<IP_ADDRESS>", :ua=>nil}

    return card_holder as string for ccard

I, [2021-09-30T11:01:15.439926 #18937]  INFO -- : {:status=>400, :time=>{:total=>10.43, :db=>0.0, :view=>10.43}, :method=>"GET", :path=>"/v3/workflow/grapetest", :params=>{"payment_type"=>"ccard", "card_
holder"=>{"first_name"=>"xxx", "last_name"=>"yyy"}}, :host=>"example.org", :ip=>"<IP_ADDRESS>", :ua=>nil}
D, [2021-09-30T11:01:15.440771 #18937] DEBUG -- : {:code=>422, :user_message=>"Wrong parameters: card_holder is invalid", :internal_message=>"card_holder is invalid"}

    422 if type is ccard and card_holder is a hash

Finished in 0.06557 seconds (files took 3.13 seconds to load)
3 examples, 0 failures, 0 pending

PR the spec - and maybe try a fix?

Unfortunately not, I've looked in the params_scope.rb, it quite complicated. I only found the block calls, but its result doesn't stored or analyzed. But in the Grape 1.0.2 this piece of code looks the same, with different behaviour in results.


Add wasm color and search filter

For https://github.com/davidmarkclements/0x/pull/218
Gives wasm a minty colour, and excludes the [WASM] bit at the end from the search filter. It's not using the same .split('[')[0] way as the cpp filter because I'm sure there's some language out there that will put [] in the middle of a function name

Ooh minty


Pick-up history

Have a visible backlog of pick-ups including the foodsaver who carried out the pick-up and additional info like how much and what kind of food was saved, what went well, potential problems and miscellaneous notes. The additional info could be requested automatically when the user logs in the next time after a pick-up.

This seems to touch #177, sorry didn't see it before.

Yes, that already seems to be covered by the other issues.


Trying to access array offset on value of type null

vendor/sarfraznawaz2005/meter/src/Tables/Table.php:52 and
vendor/sarfraznawaz2005/meter/src/Tables/Table.php:53
in the request the request()->order is not set in request by default

Can you please tell laravel version ? Which page this is happening on ? Also have you modified anything in the package code ?

Laravel version 6.0,  on every page where an datatables table is loaded. I haven't modified the code.
When i changed line 52 and 53 with an isset(request()->order) check then it works but i cannot change columnorder of the datatables on the frontend.

Please try updating to latest version 1.0.7. thx

thnx it's working now as expected


services: possibility to configure index dumper

Coverage increased (+0.007%) to 94.081% when pulling 1ac56e05b0ad6fa550474082d7473b311b9c7596 on lnielsen:index_dumper into 84a7bf7f279def2d671bb72bae3c77804e206840 on inveniosoftware:master.

Coverage increased (+0.007%) to 94.081% when pulling 1ac56e05b0ad6fa550474082d7473b311b9c7596 on lnielsen:index_dumper into 84a7bf7f279def2d671bb72bae3c77804e206840 on inveniosoftware:master.


SCM Graph: Branch selection doesn't persist

Type: Bug

Choose specific branches to display on the graph (in my case main, origin/main and upstream/main).
Restart VS Code, or close and reopen the workspace.
Selector has reverted to 'Auto'

VS Code version: Code - Insiders 1.94.0-insider (6e9d8a281003211220a6732ec538a28c483ff78b, 2024-09-11T05:04:03.124Z)
OS version: Windows_NT x64 10.0.22631
Modes:

even if you move away from the source control tab and come back it is changed back to auto.

even if you move away from the source control tab and come back it is changed back to auto.

I'm not seeing that happen on today's Insiders.

ohh, may be i am on different version than you. (i do not see a pending update though)
Version: 1.94.0-insider
Commit: 6e9d8a281003211220a6732ec538a28c483ff78b
Date: 2024-09-11T05:04:03.124Z
Browser: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Code-Insiders/1.94.0-insider Chrome/124.0.6367.243 Electron/30.4.0 Safari/537.36


On linux, if upgrading node to v9, usb won't build

blinkstick works well with node 8, but stops building when bumping to node 9.
By depending on usb 1.1.1 or  higher, it's fixed.
(There's a deprecated dependency in usb 1.1.1, so on linux with node 9, we need 1.3.1.)
https://github.com/arvydas/blinkstick-node/pull/32 fixes the issue :-)

(For my own dependent project to work, I published a blinkstick-n9 to npm, which may be used until this is resolved on master).


miniflux: 2.0.46 -> 2.0.47

Description of changes

Things done
Updated version and hash

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 250948 run on x86_64-darwin 1

1 package built:

miniflux

Could you please fix:
diff --git i/pkgs/servers/miniflux/default.nix w/pkgs/servers/miniflux/default.nix
index beaa6f9de914..8f2afcb2d6d1 100644
--- i/pkgs/servers/miniflux/default.nix
+++ w/pkgs/servers/miniflux/default.nix
@@ -21,7 +21,7 @@ in buildGoModule {
   checkFlags = [ "-skip=TestClient" ]; # skip client tests as they require network access

   ldflags = [
-    "-s" "-w" "-X miniflux.app/version.Version=${version}"
+    "-s" "-w" "-X miniflux.app/v2/internal/version.Version=${version}"
   ];

   postInstall = ''

See https://github.com/miniflux/v2/commit/168a870c025bfef6efdeb46e166e79a16093c157

-    "-s" "-w" "-X miniflux.app/version.Version=${version}"
+    "-s" "-w" "-X miniflux.app/v2/internal/version.Version=${version}"

fixed

Please squash your 4 commits into a single one titled miniflux: 2.0.46 -> 2.0.47.
See CONTRIBUTING.md#commit-conventions

Superseded by https://github.com/NixOS/nixpkgs/pull/255439


ASP.NET core 2.0 Authentication does persist more than 15 Seconds

I have seen the topic: https://github.com/aspnet/Identity/issues/1389, which seems to be the same issue. But in my case something else (weird) is happening. I have set the authentication and the cookies configuration. Although, the user is logged out almost instantaneously. The weird part refers to the cookies and the expiration of them. When I look at the developer tools on Chrome my cookies are like this:

Does anybody has any ideia on how to set those cookies to the correct expiration date?
My startup looks like this:
`services.AddIdentity<ApplicationUser, IdentityRole>(config =>
{
config.SignIn.RequireConfirmedEmail = false;
})
.AddEntityFrameworkStores()
.AddDefaultTokenProviders();
        services.Configure<SecurityStampValidatorOptions>(options => options.ValidationInterval = TimeSpan.FromMinutes(30));

        services.AddAuthentication()
            .Services.ConfigureApplicationCookie(o =>
        {

            o.LoginPath = "/account/login";
            o.LogoutPath = "/account/logout";
            o.SlidingExpiration = true;
            o.Cookie.Expiration = TimeSpan.FromHours(1);
            o.Cookie = new CookieBuilder
            {
                HttpOnly = true,
                Name = ".BrAssociation.Security.Cookie",
                Path = "/",
                SameSite = SameSiteMode.Lax,
                SecurePolicy = CookieSecurePolicy.SameAsRequest
            };

        });`

You can't mix sliding expiration and absolute expiration at the same time. It's either or, so pick one and see if it still reproduces.

We're closing this issue as no response or updates have been provided in a timely manner. If you have more details and are encountering this issue please add a new reply and re-open the issue.


The userid and pwd of DBB Web Appl server must be externalized

On the current paper the credentials are here: https://github.com/IBM/dbb-zappbuild/blob/development/build-conf/build.properties
They should be at the user repo.

We added an optional option in this document:  https://developer.ibm.com/components/ibmz/tutorials/build-a-pipeline-with-jenkins-dependency-based-build-and-urbancode-deploy/


feature/jenkins build

This MR contains changes to add CI and Docker image for the application

@SergeyDz Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

not relevant anymore


Short solution needed: "How to install Bitnami Redis docker image" (redis)

Please help us write most modern and shortest code solution for this issue:
How to install Bitnami Redis docker image (technology: redis)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to install Bitnami Redis docker image


test with  RFBNet300_VOC_80_7.pth, but the map is 0.

I have the same problem, how do you deal with this issue

no, the problem still exists.

I have the same problem, how do you deal with this issue

I have the same problem, how do you deal with this issue

I have solve this problem, trained more time and change the lr.

FileNotFoundError: [Errno 2] No such file or directory: '03398.xml'， How do I solve it

I have the same problem, how do you deal with this issue

I have solve this problem, trained more time and change the lr.

I directly predict my VOC dataset using the pre-trained model given by the auther, but no boxes are found. Did you have the same issue?


chore: clean up pre-request script sdk objects

Please review this PR (https://github.com/Kong/insomnia/pull/7147) firstly as current one is based on it
Changes:

[x] add simple interpolator for enabling replaceIn methods and cleanups
[x] add private _index field to follow existing method behavior such as one

How to test
Run this script and check if it runs well:
insomnia.environment.set('name', 'jack');
const welcome = pm.environment.replaceIn("Hello {{name}}.");
console.log(welcome);

Close this one as it is too small and a part of this PR: https://github.com/Kong/insomnia/pull/7170


-X is not a valid cp flag, as a result condition always throws exit 1

Description
https://github.com/facebook/react-native/blob/774fe0cf6f22f832b756c166599973d9db6a93de/packages/react-native/scripts/react_native_pods_utils/script_phases.sh#L107
invalid flag for cp, please remove -X
Steps to reproduce
Use script_phases.sh script
React Native Version
na - latest
Affected Platforms
Build - Linux
Output of npx react-native info
System:
  OS: macOS 14.6.1
  CPU: (10) arm64 Apple M2 Pro
  Memory: 1.44 GB / 32.00 GB
  Shell:
    version: 5.2.21
    path: /opt/homebrew/bin/bash
...
npmPackages:
  react-native:
    installed: 0.73.8
    wanted: 0.73.8
...

Stacktrace or Logs
cp: invalid option -- 'X'
Try 'cp --help' for more information.

Reproducer
na
Screenshots and Videos
No response

React Native Version
na - latest

Which version are you using. Here you say latest but in the info box you specified 0.73 @mattmarich


. can't be used in a formatted string

Sample code:
/proc/main()
  . = 5
  world.log << "[.]"

Error:

This was fixed by 283471f8eb7a9387847ce62c3e539356954076ff.


fix: DEV-2432: Allow unfinished polygons to be saved as draft

/frontend build lsf master

/frontend build lsf master


[0.10.0]: Add flag to enable or disable termination

Proposed changes
Adds a flag called use_termination in the configuration, to enable or disable termination. This addresses point (4.) in #1505.
BREAKING CHANGE: the services which are currently using termination will need to set use_termination: True in their configuration in order to continue using it.
Fixes
Point (4.) in #1505.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[ ] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[x] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING doc
[x] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have locally run services that could be impacted and they do not present failures derived from my changes

Further comments
n/a

LGTM. Is the e2e failure expected?

Nope, fixed on fe8a830ce8d373cb8a8a4781af15909a7e83bdda.


use with webpack not working

import "semantic-ui-calendar/dist/calendar.css";
import "semantic-ui-calendar/dist/calendar.js";

I import semantic-ui-calendar like this, however it didn't work, I'm not sure what's wrong?

I'm sorry that it wasn't the calendar problem, in my project, I use both semantic and semantic-ui-calendar, in some places, calendar didn't work, in another place, popup didn't work, I haven't find the reason.

I found that only when the calendar or popup is in the modal, it didn't work.

See #21 #33 #40.

the popup or calendar didn't show in the modal, because the popup has a default setting: observeChanges: true.  when the modal is initialized:

"The modal component always moves a modal’s HTML from its original place in DOM to a certain wrapper during the initialization", and
"The popup’s mutationObserver sees that the modal’s HTML was removed and destroys the popup."

To solve the problem:

setting observeChanges to false in the calendar code at line about 140;
initial the calendar after the modal was initialized.

the answer is from semantic-ui issue 4860#issuecomment-297702810