Subtitles sometimes not showing correctly; one line at bottom, 2nd line in middle of screen

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request

General information
Addon version used
0.14.0~beta19
Description
Subtitles often (but not always) show incorrectly.  If there are two successive sentences, first will show at bottom correctly while 2nd line shows simultaneously in middle of screen and shifted left.
Using Aeon-Nox Slivo skin
Steps to Reproduce

Play a Netflix show with subtitles enabled.  For example "American Crime Story: The People v. O.J. Simpson"

Expected behavior:
subtitles show centered at bottom screen
Actual behavior:
If there are two successive sentences, first will show at bottom correctly while 2nd line shows simultaneously in middle of screen and shifted left.
Context (Environment)
Windows 10, v18 Leia
Aeon-Nox Slivo skin
Installation

[ ] I'm using this Netflix Repo
[ ] I'm using other Netflix Repo (Please specify which)
[x] I'm using a different source (Please tell which)
CastagnaIT beta19 zip

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[x] Windows (x86/x64)
[ ] Linux (ARM)
[ ] Android

Debug log
[Please include a link to your debug log (use something like http://sprunge.us/ or https://pastebin.com/) or similar, please do not paste]
Other information
[e.g. detailed explanation, related issues, suggestions how to fix, links for us to have context, etc.]
Screenshots
[Please add a screenshot if that helps understanding your problem]

[You can erase any parts of this template not applicable to your Issue.]

This is a problem of inputstream.adaptive not related to the netflix addon,
i'm trying to find out if they're going to correct

ok thanks, i see you opened issue here, i will keep an eye on it:
https://github.com/peak3d/inputstream.adaptive/issues/229
a few other issues there seem related:
https://github.com/peak3d/inputstream.adaptive/issues/221
https://github.com/peak3d/inputstream.adaptive/issues/214
i see on Kodi forums xiji had described this issue nicely as "like yoda speaks"
https://forum.kodi.tv/showthread.php?tid=329767&pid=2807079#pid2807079
then, essam315 suggested using a "0.13.21" based modified release, and i dont see whether or not it worked.
anyway, my understanding is 0.13.xx releases would not have nearly the features as these 0.14.xx betas.  when i tried 0.13.xx it didn't even play video.

version 13 is almost totally different from version 14 have little in common
I'll check the modified version to see if there's anything useful.
but I think the best way is fix inputstream.adaptive

@dl16
Try using "simplesdh" profile instead of "webvtt-lssdh-ios8", open "profiles.py" in
C:\Users\Essam\AppData\Roaming\Kodi\addons\plugin.video.netflix\resources\lib\services\msl
and replace "webvtt-lssdh-ios8" with "simplesdh" (by the way i do not get this problem with "webvtt-lssdh-ios8" profile on my Windows 10, Kodi 18 Leia, default settings and skin, 1080p monitor, full screen).

i was on 720p, tested with 1080p and do not have this issue.  thanks for the tip.  haven't tested the "simplesdh" to see that would also fix for 720p

problem fixed, download again the addon,
i hope that in the future my request for a fix will be granted from peak3d

@CastagnaIT
Changing the version in the comparison from 2.3.8 to 2.3.14 will disable the subtitles for many languages that are not supported in "simplesdh", and only supported in "webvtt-lssdh-ios8" like Japaneses, Chinese, Arabic, Hebrew, Korean and many more, if the installed inputstream.adaptive is less than 2.3.14, the reason for choosing 2.3.8 as the minimum inputsream.adpative version for using "webvtt-lssdh-ios8" was because 2.3.8 was the first version to support "webvtt-lssdh-ios8", and the reason for using "webvtt-lssdh-ios8" because all the languages are support in it (using "webvtt-lssdh-ios8" was my idea many thanks for "peak3d" for taking my request and supporting it in inputsream.adpative and in the Netflix plugin 0.13.XX and many thanks for "caphm" for taking my request and supporting it in the Netflix plugin 0.14.0).
This problem will occur only in the English subtitles because Netflix only use lines in the same time with different position on the screen with the English subtitles, the workaround may be something like enabling "simplesdh" and  "webvtt-lssdh-ios8" at the same time and only use "simplesdh" if only the subtitles are in English language or inputsream.adpative version is less then 2.3.8 (not sure if this is possible to work right or not).

@CastagnaIT
Changing the version in the comparison from 2.3.8 to 2.3.14 will disable the subtitles for many languages that are not supported in "simplesdh", and only supported in "webvtt-lssdh-ios8" like Japaneses, Chinese, Arabic, Hebrew, Korean and many more, if the installed inputstream.adaptive is less than 2.3.14, the reason for choosing 2.3.8 as the minimum inputsream.adpative version for using "webvtt-lssdh-ios8" because 2.3.8 was the first version to support "webvtt-lssdh-ios8", and the reason for using "webvtt-lssdh-ios8" because all the languages are supported in it.

@CastagnaIT
Changing the version in the comparison from 2.3.8 to 2.3.14 will disable the subtitles for many languages that are not supported in "simplesdh", and only supported in "webvtt-lssdh-ios8" like Japaneses, Chinese, Arabic, Hebrew, Korean and many more, if the installed inputstream.adaptive is less than 2.3.14, the reason for choosing 2.3.8 as the minimum inputsream.adpative version for using "webvtt-lssdh-ios8" was because 2.3.8 was the first version to support "webvtt-lssdh-ios8", and the reason for using "webvtt-lssdh-ios8" because all the languages are supported in it.

@dl16 awaiting correction i have opted for the addition of a new temporany option in the expert settings
to the next version enable "Disable WebVTT subtitle support".

The last modification makes WebVTT subtitles disabled by default, changing from
def _subtitle_profiles():     isversion = xbmcaddon.Addon('inputstream.adaptive').getAddonInfo('version')     subtitle_profile = ['webvtt-lssdh-ios8']     if g.ADDON.getSettingBool('disable_webvtt_subtitle') \         or common.is_minimum_version(isversion, '2.3.8'):         subtitle_profile = ['simplesdh']     return subtitle_profile
to
def _subtitle_profiles():     isversion = xbmcaddon.Addon('inputstream.adaptive').getAddonInfo('version')     return ['webvtt-lssdh-ios8'             if common.is_minimum_version(isversion, '2.3.8') and not g.ADDON.getSettingBool('disable_webvtt_subtitle')             else 'simplesdh']
makes everything works fine.

@CastagnaIT
The last modification makes WebVTT subtitles disabled by default, changing from
def _subtitle_profiles():     isversion = xbmcaddon.Addon('inputstream.adaptive').getAddonInfo('version')     subtitle_profile = ['webvtt-lssdh-ios8']     if g.ADDON.getSettingBool('disable_webvtt_subtitle') \         or common.is_minimum_version(isversion, '2.3.8'):         subtitle_profile = ['simplesdh']     return subtitle_profile
to
def _subtitle_profiles():     isversion = xbmcaddon.Addon('inputstream.adaptive').getAddonInfo('version')     return ['webvtt-lssdh-ios8'             if common.is_minimum_version(isversion, '2.3.8') and not g.ADDON.getSettingBool('disable_webvtt_subtitle')             else 'simplesdh']
makes everything works fine.

Oh, my oversimplification. Last night, i was melted down from work.. Forgotten an "not" thankyou!!
for the VP9 would be to understand also the reason of caphm decision to have it set active by default unfortunately it's not all well documented

He made VP9 on by default because at this time Netflix made many changes resulted in no playback was  working unless VP9 was enabled, now we do not need this, and here in 0.13.XX you will notice that the developer made VP9 optional again after it became needless.
https://github.com/asciidisco/plugin.video.netflix/commit/fe53bce6063252b021c0716ff635b0d1153c6781
This was the first solution
https://github.com/asciidisco/plugin.video.netflix/pull/519#issuecomment-439276151
then that one
https://github.com/asciidisco/plugin.video.netflix/pull/519#issuecomment-439338258
more information
https://github.com/asciidisco/plugin.video.netflix/pull/519#issuecomment-440480430
here VP9 was optional
https://github.com/asciidisco/plugin.video.netflix/pull/519#issuecomment-440794887
here is the reason for using VP9
https://github.com/asciidisco/plugin.video.netflix/pull/519#issuecomment-441029901

Sorry, but WebVTT still need to be fixed, you need to add "and not" before "g.ADDON.getSettingBool('disable_webvtt_subtitle')", in which WebVTT works if 'disable_webvtt_subtitle' was off, please use this code, i tested it and 100% working.
def _subtitle_profiles():
isversion = xbmcaddon.Addon('inputstream.adaptive').getAddonInfo('version')
return ['webvtt-lssdh-ios8'
if common.is_minimum_version(isversion, '2.3.8') and not g.ADDON.getSettingBool('disable_webvtt_subtitle')
else 'simplesdh']

the setting disable_webvtt_subtitle works in reverse mode, no needed the "not" term, you have wrong settings in your addon

Sorry, everything working fine now.


Which version of flutter to use to build?

I have tried different versions of flutter to build the application but still getting a lot of errors:

ecause every version of flutter_localizations from sdk depends on intl 0.19.0 and thingsboard_app depends on intl ^0.18.1, flutter_localizations from sdk is forbidden.
So, because thingsboard_app depends on flutter_localizations from sdk, version solving failed.
=> update intl: ^0.19.0, form_builder_validators: ^11.0.0
Run: flutter build appbundle --no-tree-shake-icons
You are applying Flutter's app_plugin_loader Gradle plugin imperatively using the apply script method, which is deprecated and will be removed in a future release. Migrate to applying Gradle plugins with the declarative plugins block: https://flutter.dev/go/flutter-gradle-plugin-apply
lib/core/auth/login/two_factor_authentication_page.dart:357:35: Error: The argument type 'String' can't be assigned to the parameter type 'RegExp'.

'RegExp' is from 'dart:core'.
FormBuilderValidators.match(pattern,
^
/C:/Users/HoTien/AppData/Local/Pub/Cache/hosted/pub.dev/flutter_html-3.0.0-alpha.5/lib/html_parser.dart:153:62: Error: The getter 'bodyText2' isn't defined for the class 'TextTheme'.
'TextTheme' is from 'package:flutter/src/material/text_theme.dart' ('/D:/ENV/flutter/packages/flutter/lib/src/material/text_theme.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'bodyText2'.
style: Style.fromTextStyle(Theme.of(context).textTheme.bodyText2!),
^^^^^^^^^
/C:/Users/HoTien/AppData/Local/Pub/Cache/hosted/pub.dev/flutter_html-3.0.0-alpha.5/lib/html_parser.dart:217:72: Error: The getter 'bodyText2' isn't defined for the class 'TextTheme'.
'TextTheme' is from 'package:flutter/src/material/text_theme.dart' ('/D:/ENV/flutter/packages/flutter/lib/src/material/text_theme.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'bodyText2'.
style: Style.fromTextStyle(Theme.of(context).textTheme.bodyText2!),
....

How to build applications?

Hi @Tpro4391,
Use Flutter 3.19.6 and don't change any dependencies version in the pubspec.yaml.

Hi @Tpro4391,
Use Flutter 3.19.6 and don't change any dependencies version in the pubspec.yaml.

This worked for me. Thanks


Crash in CKFVideoSession.swift line 44

Fatal Exception: NSInvalidArgumentException
*** -[AVCaptureSession addInput:] Can't add <AVCaptureDeviceInput: 0x107cfaf10 [Front Camera]> because the device does not support AVCaptureSessionPreset1920x1080 - Use -[AVCaptureDevice supportsAVCaptureSessionPreset:]

it happens when flipping the camera.

The same for me... how to fix?


Support diagonal movement in pixel mode.

We will need to support moving in two directions at once before pixel mode is complete.

Here is an idea for a simple algorithm:

Try moving one pixel up
If success, try moving one pixel right

Try moving one pixel up
Return

Else try moving one pixel right
Return

This is actually a bit of a pain because InputManager wasn't written for it and since we use seconds_passed to calculate lengths of movement, if you try to alternate directions you just twitch in place. Can I ask you to take a look at this?

I'm bumping this to the next release.


error subprocess-exited-with-error

Type: Bug
(venv) PS C:\Users\dell\Desktop\AZUBI CLASS\Career Accelerator\LP1\A-Data-Analysis-Project> python -m pip install dotenv
Collecting dotenv
Using cached dotenv-0.0.5.tar.gz (2.4 kB)
Installing build dependencies ... done
Getting requirements to build wheel ... done
Installing backend dependencies ... error
error: subprocess-exited-with-error
× pip subprocess to install backend dependencies did not run successfully.
│ exit code: 1
╰─> [33 lines of output]
Collecting distribute
Using cached distribute-0.7.3.zip (145 kB)
Installing build dependencies: started
Installing build dependencies: finished with status 'done'
Getting requirements to build wheel: started
Getting requirements to build wheel: finished with status 'done'
Installing backend dependencies: started
Installing backend dependencies: finished with status 'done'
Preparing metadata (pyproject.toml): started
Preparing metadata (pyproject.toml): finished with status 'error'
error: subprocess-exited-with-error
    Preparing metadata (pyproject.toml) did not run successfully.
    exit code: 1

    [6 lines of output]
    usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
       or: setup.py --help [cmd1 cmd2 ...]
       or: setup.py --help-commands
       or: setup.py cmd --help

    error: invalid command 'dist_info'
    [end of output]

    note: This error originates from a subprocess, and is likely not a problem with pip.
  error: metadata-generation-failed

  Encountered error while generating package metadata.

  See above for output.

  note: This is an issue with the package mentioned above, not pip.
  hint: See above for details.
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error
× pip subprocess to install backend dependencies did not run successfully.
│ exit code: 1
╰─> See above for output.
note: This error originates from a subprocess, and is likely not a problem with pip.
 It looks like you may be using an old version of VS Code, the latest stable release is 1.87.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I have upgraded to the latest version of vs code and the error still persists

/causedByExtension

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Image previews suddenly stopped working

It now uses exiftool and stoped using ueberzug
I also tried opening lf in kitty but no image preview there too I did'nt change anything it was working fine before
I use archlinux and alacritty in x11(dwm)

I tried reproducing the issue using X11, dwm and alacritty but image previews worked fine. Do the previews still not work?

I have this issue now as well with archlinux + alacritty. Didn't make any changes to config.
However, if I open lf in a tmux session it works fine.

Same problem for around the last 1-2 weeks, same setup (arch/alacritty/dwm) -- also occured using stpv, st terminal, and switching package from ueberzug to AUR ueberzugpp.
Workarounds which worked:

As above, using tmux (did not work using AUR ueberzugpp)
Uninstall Arch package ueberzug and install same package via pip (as ueberzug-bak)

ueberzug layer (using either package) gave no errors and setting the debug flag in ctpv showed something like:
Running preview: image
Preview failed: image
Exiftool output ...

Just want to mention I am having the same issue when using ueberzug, and it works in tmux as well. As for ueberzug-bak pip package that doesn't work for me at all with or without tmux even though ueberzug binary is in path. So for now I am using kitty icat.

@rEnr3n @Zweihander-Main @sr-prog It seems like it started happening to you pretty recently. What distros do you use? If it's something like Arch, maybe it updated something and broke?

@NikitaIvanovV I'm using Arch, it might have something to do with the python 3.11 upgrade? I'm not sure.

Since the author of Ueberzug has decided to archive it, I suggest that to be the problem. Can anyyone try using chafa support and see if it is working properly? I had some random glyphs pop up instead of a clear image.

It suddenly works now. I'm not sure which package caused the issue.

I switched from ueberzug to ueberzugpp, works just like before. Donno how it works all fine suddenly @rEnr3n .

I just use ueberzug package on archlinux. I never installed ueberzugpp. Maybe an issue with cache?

I updated my system yesterday and it broke again. It's even broken inside tmux. I've switched to ueberzugpp to make it work again.

Yeah. It must be the new default ig

same, broke again for me as well after system update

Same, the newest version update broke everything. Need some stable replacement for ueberzugpp now. Or maybe they'll fix it soon

Can anyone inform me if ueberzug or ueberzugcpp works on your machine?

Ueberzugpp works. It's aliased to ueberzug by default. There are bugs from time to time, the image preview pops up on another position in the terminal. Building it is also a pain since one of its optional dependency doesn't work. But that's what I use. Probably want to switch to another option in the future.

Can anyone inform me if ueberzug or ueberzugcpp works on your machine?

latest version of ueberzug from arch repo  works on my archlinux machine


Best practice of dealing with sporadic FrozenDict conversions?

Hi,
in my project, I have multiple instances of modules that look approximately like this:
class TreeEncoder(nn.Module):
    leaf_encoders: Any  # pytree of nn.Modules

    @nn.compact
    def __call__(self, data: Any):  # data is a pytree of jax.Array
        return jnp.stack(jax.tree_leaves(jax.tree_map(lambda d, enc: enc(d), data, self.leaf_encoders)), axis=-1)

So essentially, this module has one encoder for every leaf of the input pytree and uses them to obtain a vector encoding of the entire tree. Usage could look like this:
encoder = TreeEncoder({"img": ResNet(...), "vector": DenseNN(...)})
output = encoder({"img": jnp.zeros((480, 640, 3)), "vector": jnp.zeros(5)})

The beauty of JAX is that the pytree can be an arbitrary structure and not only dicts are possible. I make heavy use of this fact and sometimes just define my own dataclasses.
But here comes my problem: if I use a dictionary, flax will sometimes convert them into FrozenDicts and then the call to jax.tree_map fails with
Custom node type mismatch: expected type: <class 'flax.core.frozen_dict.FrozenDict'>, value: {'img': ..., 'vector': ...}

It took me a while to understand when these conversions happen, but I am fairly certain now that flax behaves as follows:

The inputs to __call__ are never converted (data is always a dict)
The leaf_encoders are converted iff the TreeEncoder instance is created inside a @nn.compact call

How do I deal with this? I cannot call self.leaf_encoders.unfreeze() because it might not be a FrozenDict (and not even a dictionary). Is there some way I can disable the FrozenDict conversion in general? Or is it possible to make FrozenDicts and dicts compatible as arguments to jax.tree_map?
Thanks a lot in advance!
Best,
Tim

Hmm, are you using the latest Flax? The latest Flax should have flax.config.flax_return_frozendict as False and return normal dict by default.
In other case, you can do an explicit check like hasattr(x, unfreeze) and call x.unfreeze() only when true. But please let me know if you are using latest Flax and still run into this problem.
cc @chiamp

Hi,
I have to double check which flax version I am using, but I installed it from PyPi around 2 months ago, so it should be fairly recent.
To be clear, my problem is not that modules return FrozenDicts, but rather that fields of modules sometimes get converted to FrozenDicts. I think I will go with your suggestion for now, but an option to turn that behavior off fully would be nice for the future.
Best,
Tim


Fastnlight: Watch +1-888~405~7779

Fastnlight requests the watch of the watch_number +1-888~405~7779. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Rejected by Jeff Schaller in Charcoal HQ. Reason: 'a good watch, subsequently watched by a blacklister'


Document support for Razor templates

Fixes #9618
Internal Review Topic (Templated Razor delegates section)

/cc @NTaylorMullen - this is you dog

"dog" ... lol ...
var pets = new List<Pet>
{
    new Pet { Name = "Rin Tin Tin" },
    new Pet { Name = "Mr. Bigglesworth" },
    new Pet { Name = "K-9" },
    new Pet { Name = "NTaylorMullen" }
};

... just remember @NTaylorMullen ... he started it. 😀 lol


Fix and significantly simplify the docker build process.

Changes Unknown when pulling f9cfa2300e8007ae8d1d8a12a86ac96bb7448e0b on mars-lan:docs into ** on linkedin:master**.


Feature/tcomp 112 multiple runtimes and deps

Here are the modifications, that break certain APIs so I changed the version for most most of the modules.

Created One interface RuntimeInfo to allow the Component developer to return the Runtime class name as well as the jar dependencies.
removed all Runtime interface to move to on single ComponentDefinition method called
RuntimeInfo getRuntimeInfo(Properties properties, ConnectorTopology connectorTopology);. This method allow for the CompDev to compute the Runtime info according to the ComponentProperties and the Connector topology (see below).
added a ConnectorTopoly enum so that the component designer can say what kind of connectivity it supports thanks to a new ComponentDefinition method called
Set<ConnectorTopology> getSupportedConnectorTopologies();. Here are the enum values (see the real file for complete javadoc).

public enum ConnectorTopology {
    OUTPUTS,
    INPUTS,
    INPUT_AND_OUTPUT,
    NONE;
}

added a Runtime service in a new dedicated module with some submodules for an example/test case of a Component with 2 versions of the library
I also changed the way the dependencies.txt file is generated to be used during test more easily so it implied a pom change for all components.

WARNING : I did not manage to make the tests run in the OSGI test module. The test are not failing, they are not even run because I did not success to make my test component register in the OSGI service registry. I have not found the reason yet. this is the reason why you have a class called : MultiRuntimeTestITXX.java

Please do not merge until build is ok and the last warning (OSGI tests failure is resolved), But that would be nice you start looking at this and comment.

Pull request test result: failure :x:

Pull request test result: success :star:

Hello!  LGTM.
Optional note remains at https://github.com/Talend/components/pull/224#discussion-diff-76984503R84 but I don't think it's important enough to block this PR!

Pull request test result: success :star:


Extension issue

Issue Type: Bug
Extension Name: html-css-class-completion
Extension Version: 1.19.0
OS Version: Windows_NT x64 10.0.18363
VSCode version: 1.49.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Already reported here: https://github.com/zignd/HTML-CSS-Class-Completion/issues/155


PKIX path building failed: unable to find valid certification path to request to requested target

Describe the bug
A clear and concise description of what the bug is.
The error message javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
To Reproduce
Steps to reproduce the behavior:

Use of self-signed certificate keystone
The TLS is enabled for proxy and broker
The Apache Pulsar is running in Openshift Cluster
The Apache Pulsar version 3.2.0

Expected behavior
When we are communicating directly to broker url it is working, we expect same when we try to connect via proxy.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Ubuntu

Additional context
Add any other context about the problem here.
Error logs:
2172
at sun.security.validator.Validator.validate(Validator.java:264) ~[?:?]
2173
at sun.security.ssl.X509TrustManagerImpl.checkTrusted(X509TrustManagerImpl.java:285) ~[?:?]
2174
at sun.security.ssl.X509TrustManagerImpl.checkServerTrusted(X509TrustManagerImpl.java:144) ~[?:?]
2175
at sun.security.ssl.CertificateMessage$T13CertificateConsumer.checkServerCerts(CertificateMessage.java:1329) ~[?:?]
2176
... 17 more
2177
Caused by: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
2178
at sun.security.provider.certpath.SunCertPathBuilder.build(SunCertPathBuilder.java:148) ~[?:?]
2179
at sun.security.provider.certpath.SunCertPathBuilder.engineBuild(SunCertPathBuilder.java:129) ~[?:?]
2180
at java.security.cert.CertPathBuilder.build(CertPathBuilder.java:297) ~[?:?]
2181
at sun.security.validator.PKIXValidator.doBuild(PKIXValidator.java:434) ~[?:?]
2182
at sun.security.validator.PKIXValidator.engineValidate(PKIXValidator.java:306) ~[?:?]
2183
at sun.security.validator.Validator.validate(Validator.java:264) ~[?:?]
2184
at sun.security.ssl.X509TrustManagerImpl.checkTrusted(X509TrustManagerImpl.java:285) ~[?:?]
2185
at sun.security.ssl.X509TrustManagerImpl.checkServerTrusted(X509TrustManagerImpl.java:144) ~[?:?]
2186
at sun.security.ssl.CertificateMessage$T13CertificateConsumer.checkServerCerts(CertificateMessage.java:1329) ~[?:?]
2187
... 17 more

Could you share your yaml?

Closing since there's no response from reporter. Please retest on latest released Pulsar Helm chart version.


devProxy does not Proxy Websockets

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.7.4
CLI Version:      3.9.1
Nitro Version:    2.7.0
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      routeRules, runtimeConfig, components, dir, imports, css, build, modules, nitro, devtools
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-xwvyvm?file=app.vue
Websockets appear blocked in online setup. If run locally, note that proxy is not attempted to fake http address (error expected).
Describe the bug
Websocket connections are not listened for when configured into Nitro devProxy.
Additional context
No response
Logs
Each WebSocket connection fails without reaching intended target.

Let's track in https://github.com/nuxt/cli/issues/107.


The People list seems to reset early

DCAKen  writes:

The People list seems to reset early. When I refreshed it at 10:45 pm, everyone who had ridden every day was showing as yellow.

@obscurerichard : I verified this at 11:39 PM EST today, see below for a screenshot.
This might have to do with the server keeping UTC time but not adjusting for the local time zone. Also the server. And to top it off, the MySQL server seems to be set to PST!
PS C:\Users\XXXXXXXXX> ssh<EMAIL_ADDRESS>Last login: Thu Jan  3 20:43:07 UTC 2019 from <IP_ADDRESS> on pts/0
Container Linux by CoreOS stable (ZZZZZ.X.Y)
core@freezingsaddles ~ $ date; mysql -e 'select now()'
Fri Jan  4 04:42:21 UTC 2019
+---------------------+
| now()               |
+---------------------+
| 2019-01-03 20:42:21 |
+---------------------+
core@freezingsaddles ~ $

This is probably related to #7, #34, and #68 which are all time zone related.


Use Try/Catch to handle the XML theme engine errors for when an element isn't found in the XML file.

This is to prevent making it necessary to update themes when the UXL Launcher theme engine changes what it's looking for in the XML document.

Actually, just use Try/Catch and set the element color to a default color based on the color the Default Theme uses for that element.

Actually, use a messagebox that has 3 options: Yes, No, and Cancel. That messagebox will ask the user if they want to update their chosen theme settings to the Default theme and use the Default theme for UXL Launcher until they change it, use the default theme for this session only, or close UXL Launcher. If the user chooses "Yes," the settings will be updated and the theme engine will attempt to load the default theme. If they choose "No," the theme engine will attempt to load the default theme only for this session. If the user chooses "Cancel," UXL Launcher will just close.
The full message will be something like:
"Would you like to update your chosen theme settings to the Default theme and attempt to load the Default theme for UXL Launcher? It appears that the chosen theme is missing a  XML element for the  control. Click "Yes" to update your chosen theme settings to the Default theme and attempt to use the Default theme until you change your theme in the Options window. Click "No" to attempt to use the Default theme for this session only. Click "Cancel" to close UXL Launcher. See this page for the current XML elements required by UXL Launcher for themes:

Remember to put in a link to the Wiki article for the required XML elements in the themes.

Another change to this plan: Im just ignoring missing elements and invalid colors now. See issue #88.


Add (configurable) support for GTEU.

Closes #2
Tested in a production environment without GregTech and it didn't cause any issues.

Code looks good to me. Was there additional work you wanted to do on this or are you considering it ready to merge?

Should be fine to merge.


Add autocorrect ability to Style/OptionHash cop

Pretty trivial, and useful for updating large outdated codebases.

I think the autocorrection should be disabled by default because this rewriting is not safe.
For example:
def foo(options = {})
  p options
end

def bar(**options)
  p options
end

foo('a' => 1) # => {'a' => 1}
bar('a' => 1) # => wrong number of arguments (given 1, expected 0) (ArgumentError)

Hm. This just changes from one flavour of options hash to another one? 🤔 The recommendation is to use keyword arguments, e.g.:
def foo(options = {})
  @bar = options[:bar] || 1
  @foo = options[:baz] || 2
end

can be replaced with:
def foo(bar: 1, baz: 2)
  @bar = bar
  @baz = baz
end

@pocke Well, the cop is disabled totally by default config. So I believe if you enable it, you probably will be OK with auto-correct.
@Drenmi well, you question makes sense, of course.
So, this autocorrect is not the best way to do to "just make Rubocop happy", all in all.
What I was thinking, though, is "at least make it more modern" is a good thing too.
Also, probably introducing more advanced cop, which would catch the situation like "You have **options, but only use options[:foo] and options[:bar], please make it bar:, baz: (with autocorrect, if possible), could be a good step forward.

Well, the cop is disabled totally by default config. So I believe if you enable it, you probably will be OK with auto-correct.

I do not think so. If users enable it, they expect this cop detects bad usage of option argument. But they do not expect code to be broken by auto-correction.
BTW, I think this cop should be enabled by default.

So, this autocorrect is not the best way to do to "just make Rubocop happy", all in all.
What I was thinking, though, is "at least make it more modern" is a good thing too.
Also, probably introducing more advanced cop, which would catch the situation like "You have **options, but only use options[:foo] and options[:bar], please make it bar:, baz: (with autocorrect, if possible), could be a good step forward.

👍

OK, all concerns make sense here.


Enable hosted lib generation

The lib workflow now builds and uploads the libs for all targets. This can be used to update those targets without having to build them locally.
I’ve also added a GitHub action to centralize the environment adjustments needed to get Rust to work properly with the targets inside the GitHub VMs.

https://github.com/microsoft/windows-rs/pull/2365/commits/01ce49fef4dd6b0d3d0faba590c3a33b6ec6d825 just calls an API that's recently been added and isn't covered by the stale GNU libs. Once that failure is confirmed I can push the libs generated by the hosted build.

It's not finding the llvm-dlltool in the path. Not sure where to find that. 🤔

@kennykerr That's in C:\msys64\mingw64\bin on the runners. The environment steps put that into GITHUB_PATH but I'm not sure if that works across the new workflows you created.

Looks like updating path via composite actions is indeed not really supported https://github.com/actions/runner/issues/789.
This aligns with my experience with all of GitHub's shiny new workflow tools/modularity efforts. They are so enticing on paper but when you try to actually use them... 😊


Fix android build

The OS_FOO macros are set by build/build_config.h, so be sure to
include that before testing the macros.

lgtm


[Enhancement] Extend XAMLmarkup for CurrentVisualState

Hello,
I have defined a style in my App.cs.xaml and I am using it as following example.
is it possible to extend builtin XAML markup for example as  CurrentVisualState Thank you in advance.
<Style x:Key="StackStyle" TargetType="StackLayout">
            <Setter Property="Padding" Value="32" />
            <Setter Property="VisualStateManager.VisualStateGroups">
                <VisualStateGroupList>
                    <VisualStateGroup x:Name="CommonStates">
                        <VisualState x:Name="Small">
                            <VisualState.Setters>
                                <Setter Property="BackgroundColor" Value="Red" />
                                <Setter Property="Padding" Value="0" />
                            </VisualState.Setters>
                        </VisualState>
                        <VisualState x:Name="Medium">
                            <VisualState.Setters>
                                <Setter Property="BackgroundColor" Value="Yellow" />
                                <Setter Property="Padding" Value="8" />
                            </VisualState.Setters>
                        </VisualState>
                        <VisualState x:Name="Large">
                            <VisualState.Setters>
                                <Setter Property="BackgroundColor" Value="Green" />
                                <Setter Property="Padding" Value="16" />
                            </VisualState.Setters>
                        </VisualState>
                    </VisualStateGroup>
                </VisualStateGroupList>
            </Setter>
        </Style>

HomePage.xaml
<StackLayout Style="{StaticResource StackStyle}">
                    <StackLayout.Behaviors>
                        <behaviors:CurrentVisualStateBehavior State="Medium" />
                    </StackLayout.Behaviors>
                </StackLayout>

                <StackLayout Style="{StaticResource StackStyle}">
                    <StackLayout.Behaviors>
                        <behaviors:CurrentVisualStateBehavior State="Large" />
                    </StackLayout.Behaviors>
                </StackLayout>

CurrentVisualStateBehavior.cs
    public class CurrentVisualStateBehavior : Behavior<View>
    {

        private string state = string.Empty;
        public string State
        {
            get => state;
            set
            {
                state = value;
            }
        }

        protected override void OnAttachedTo(View bindable)
        {
            base.OnAttachedTo(bindable);

            if (!string.IsNullOrEmpty(State))
            {
                VisualStateManager.GoToState(bindable, State);
            }
        }

        protected override void OnDetachingFrom(View bindable)
        {
            base.OnDetachingFrom(bindable);
        }

    }

@PureWeen Hello. Actually I did not know to use StyleClass before have opened issue. I were using this approach for multiple styles. Sorry for keeping you busy. I think this approach may be useful anyway.

@kerberosargos I'm sorry, I didn't get the idea. Are you asking to add that behavior? Or is it something else?

All codes are working. I do not need help about that. I have wrote codes for giving you idea. May be xamarin team can add this feature for next version.

@kerberosargos that will not be added on Forms, can be added here, on XamarinCommunityToolkit (XCT), but for that, we need to understand what's that about if you can expand that

Thanks! However, we are no longer adding new features to Xamarin Community Toolkit, focusing on the .NET MAUI Community Toolkit.
Please open a New Feature Discussion to implement this feature in the .NET MAUI Community Toolkit.
I've posted more information about the Future Of Xamarin Community Toolkit here: https://devblogs.microsoft.com/xamarin/the-future-of-xamarin-community-toolkit/?WT.mc_id=mobile-0000-bramin


Adding //single-line comment on same line as @import causing import to compile incorrectly

@import "testcase" //This should import testcase.sass inline, but will instead "compile" as a normal @import rule

This is expected. Single-line comments must be on their own line in the indented syntax. There is an exception for property values, but not in general.


Create Most Frequent Location View with an image

Add Fragment with endpoint data from: aad15c7dc4bdabfb78771aa4bd6c1bada1096b9e

Closing, we will try adding marker for that on existing map

With problems implementing custom InfoWindow, @slimak7 will create this view


Use 6-character RGB color for wider support

I will create more as I find other improvements.

@hello-smile6 Thanks for PR but The CSS 3-digit hex color notation is shorthand for the 6-digit hex notation. No need to update this. It works just fine in all modern browsers.

@hello-smile6 Thanks for PR but The CSS 3-digit hex color notation is shorthand for the 6-digit hex notation. No need to update this. It works just fine in all modern browsers.


Fix for writing log file to root path

Fixes #1350

Current coverage is 75.18%

Merging #1352 into master will increase coverage by +0.01% as of ad215e6

@@            master   #1352   diff @@
======================================
  Files          267     267
  Stmts        15526   15528     +2
  Branches      1631    1631
  Methods          0       0
======================================
+ Hit          11672   11674     +2
  Partial        383     383
  Missed        3471    3471

Review entire Coverage Diff as of ad215e6

Uncovered Suggestions

+0.09% via ...gingConfiguration.cs#374...387
+0.09% via ...gingConfiguration.cs#340...353
+0.08% via ...argets/FileTarget.cs#1713...1725
See 7 more...

Powered by Codecov. Updated on successful CI builds.


Property 'header' is missing in type 'Request'

Hi.
This should already be fixed in 1.7.0 version however I am still getting this error with 1.9.0 version.
Issue: https://github.com/jameswlane/jest-express/issues/4
Pull: https://github.com/jameswlane/jest-express/pull/84

express version: 4.16.3
jest-express version: 1.9.0
node version: 10.9.0
npm version: 6.2.0

Relevant code or config
authorization.spec.ts
    import auth from './authorization';
    import { Request } from 'jest-express/lib/request';

    test('should let me pass', (done) => {
        propertiesToObject.mockReturnValue({
            'unsecured-default': ['/health'],
            'unsecured-specific': []
        });
        const middleware = auth();
        const next = jest.fn(() => done());
        const req = new Request('/health', {
            headers: {
                'Content-Type': 'application/json'
            }
        });
        middleware(req, null, next);
    });

authorization.ts
  import { Request, Response, NextFunction } from 'express';

  function authMiddleware(req: Request, res: Response, next: NextFunction) {
    ...
  }

What you did:
I have mocked the ExpressJs request and passed it in the middleware function.
What happened:
Error:
src/middleware/authorization.spec.ts:21:20 - error TS2345: Argument of type 'import(".../my-project/node_modules/jest-express/lib/req...' is not assignable to parameter of type 'e.Request'.
      Property 'header' is missing in type 'Request'.

    21         middleware(req, null, next);

Suggested solution:
Add property header in the type Request or even better use express typings.

I will try and dig into this today and get a new release out.

@jameswlane i think the best would be to import express typings and extend those of jest-express, I have tested it manually and I defined header in the Request, but later I was getting more and more Property 'xyz' is missing in type 'Request errors.

If you have something working and would like to do a PR I am more than happy to merge it in.

@daniyel Could you link me to the project or an example project that is having this error. So I can work on getting this resolved.

Hi.
Sorry for not coming sooner back with response. Here I managed to make a small repository with same error: https://github.com/daniyel/jest-express-example. At first the tests were passing, which were odd to me but then later I have installed the "@types/express": "^4.11.1", and then I got the error:
TypeScript diagnostics (customize using `[jest-config].globals.ts-jest.diagnostics` option):
    src/middleware/authorization.spec.ts:22:16 - error TS2345: Argument of type 'import("/Users/hrvacada/Workspace/test/jest-express-example/node_modules/jest-express/lib/request").Request' is not assignable to parameter of type 'import("/Users/hrvacada/Workspace/test/jest-express-example/node_modules/@types/express/index").Request'.
      Type 'Request' is missing the following properties from type 'Request': header, accepted, param, host, and 44 more.

    22     middleware(req, res, next);
                      ~~~

What I mean by extending typing is, that in typing from jest-express you can import for example express-serve-static-core and implement that and then you can see which properties are missing in your class and add them. For example like:
/// <reference types="express-serve-static-core" />

import * as core from "express-serve-static-core";

interface IRequestOptions {
    method?: 'GET' | 'POST' | 'DELETE' | 'PATCH' | 'PUT' | 'HEAD' | 'OPTIONS' | 'CONNECT';
    headers?: any;
}
export declare class Request implements core.Request {

...

By this I get in my VS Code error:
Class 'import("/test/jest-express-example/node_modules/jest-express/lib/request").Request' incorrectly implements interface 'import("/test/jest-express-example/node_modules/@types/express-serve-static-core/index").Request'.
  Property 'header' is missing in type 'Request'. [2420]

@daniyel Thank you for the repo, I will try and dig into this as soon as I can. With it being so close to the holidays it maybe a couple weeks.
I have a some thoughts on solutions, but this may be a trial and error sort of thing.
This repo has had a very active community, so maybe someone can pick this up sooner then I can knock it out.

I am will have some free time this week. I will pull down @daniyel example and see if I can get this fixed.

I started working on this issue I feel I have a solid solution to resolve the issue, it will take me a while to test and build this out. I am hoping to have it done by the end of the week.

@jameswlane looking forward to this release. would be great to pass request mock directly to my middlewares that expect a Request from express

I am still working on this, when I built jest-express it was a bare-bones testing utility for my project. Since then it has grown, it seems the underlying issue is I only built out the APIs for ExpressJS and not NodeJS http module that ExpressJS is built on. So I am building out a jest-node-http and jest-express will extend it like ExpressJS extends http. That should allow it to fully match the real interface of an express app. It taking me a bit to dig through all of the NodeJS http code and build a matching interface that will work. Then updatejest-express to extend it. I have about half of it complete already. So I am hoping to have it complete by the end of the month. 🤞

@jameswlane Thank you very much for your work!

I am hoping to start pushing up the new code in a draft PR this week 🤞, if you want to watch the progress:

jest-node-http
jest-express

Team,
I seem to hit a wall, I been stuck on getting past this type error for the last two days. Reaching out to see if someone would like to take a try at it, or at least give me a second pair of eyes.
    TypeScript diagnostics (customize using `[jest-config].globals.ts-jest.diagnostics` option):
    src/middleware/authorization.spec.ts:22:16 - error TS2345: Argument of type 'import("/Users/jameswlane/Projects/GitHub/daniyel/jest-express-example/node_modules/jest-express/lib/request").Request' is not assignable to parameter of type 'import("/Users/jameswlane/Projects/GitHub/daniyel/jest-express-example/node_modules/@types/express/index").Request'.
      Types of property 'app' are incompatible.
        Type 'Express' is not assignable to type 'Application'.
          Type 'Express' provides no match for the signature '(req: Request | IncomingMessage, res: Response | ServerResponse): any'.

    22     middleware(req, res, next);

I am more then happy to push up my spike if someone want's to give it a try. If not I will keep trying to work my way through the issue, but it may take me a little longer than expected.

Hey @jameswlane ,
first of all, i want to thank you for this awesome package! I really like the use of the Request in order to properly mock respective functionality.
However, I recently stumbled upon the same issue as described above. I think it is of utmost important to solve this issue in order to allow a compliant test-scenario..
Is there any new progress on the issue?
Thank you so much for your time and effort!
All the best

@johannesschobel I have currently hit a brick wall on resolving the issue due to it involves not only mocking out Express but the underlying NodeJS layers.
On top of that, I have been traveling for work (Agile / DevSecOps Coach) since Feb and it will go till May. So most of my free time I have been spending with my wife and daughters.
I plan on jumping back on the issues when this engagement is done in May.
If anyone would like to take a stab at this issue, please do. Pull request are always welcomed.

Has there been any progress/resolution on this?

@muskand I have not had the time to work on this issue due to changes at work and home.
This is open for anyone to jump in and PR's are more welcome to help resolve this, long running issue. So if anyone had some free time to try and tackle this issue, the community and myself would greatly appreciate it.
I personally hit a brick wall trying to resolve it and don't have the time and cycles to try and pick it up and tackle it anytime soon.

Unfortunately, jest-express is virtually unusable with typescript in its current form. I have tried using Partial<> and type assertions in tests but the two just won't play nicely when the express route handlers are explicitly typed as Request, Response and NextFunction.
Is this project still being developed or maintained? Core issues haven't been addressed for almost 2 years (e.g. this issue) and there hasn't been a release for well over 12 months.
It's a great idea and a great project but its quickly become unusable and I've resorted to manually mocking my requests and responses.


Broken link for "Image object definition" in docs/concepts/images.md

Steps To Reproduce
Navigate to docs/concepts/images.md and click on Image object definition hyperlink.
The current behavior
It navigates to a 404 github page.
The expected behavior
It navigates to a page that actually exists, being it a .md (documentation) or .js (code sample) file.

I still lack in knowledge to suggest the right hyperlink address, but might be somewhere around this line of code?


Update To-do-list

The to-do list appears to be at the top. It could be made better by making it in the center.
@selemondev I have made these changes and opened a pull request you can have a look at it.

Please Assign me this issue


1.3.x patch releases

update 1.3 with 1.3.8 patch information #167988384

update 1.3 with 1.3.8 patch information #167988384


name changed from foreman to satellite6 in tower 3.1.x

Latest tower-cli should be updated with satellite6 instead of foreman (broken in Tower CLI 3.1.3)

Changes Unknown when pulling 5f3c25abc50f41d08d500d75a9272de03f7ff271 on ldomb:fix-kind-satellite6-remove-foreman into ** on ansible:master**.


small improvements in diffusion pipeline for openvino

What does this PR do?
This PR propose small improvements for OVStableDiffusionPipeline

When diffusion pipeline compiled it logs message about compiling encoder 3 times, which may confuse users. Added name of model part in logs to make these logs more useful (e.g. in case if one of the models filed during compilation, we can understand which one).
Model with static shape does not allow to change shape during pipeline call, but user should know current height and width to understand when model requires reshaping. this PR proposes properties current_height and current_width  where user can check current pipeline status

@AlexKoff88  @helena-intel  could you please take a look?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thanks @eaidova ! Could you also create a test for this in https://github.com/huggingface/optimum-intel/blob/main/tests/openvino/test_modeling.py ?

@helena-intel, thank you, added

@AlexKoff88  @helena-intel  @echarlaix
I observe failing on ci, because stable diffusion onnx export failed with torch 2.0.1 (aten::scaled_dot_product_attention is not registered for export. It will be added in next release as it is already exists on pytorch master and in nightly packages.
I added registering custom operation for that as workaround, please take a look, if it is not acceptable, I can remove it

@echarlaix, can you please take a look what is wrong with the documentation build?


enforcing no "ragged" 2D conjugacy (Wishart)

This fixes incompatible matrix dimensions in 2D conjugate sampler by simply requiring that all dependencies use the full prior, i.e. no “ragged” uses.

Made a few changes to minimize code repetition, and remove all double-quotes.  Thanks for taking the initiative @perrydv and submitting via PR.


Add Wallet Summary [Samoa]

This screen will be a replacement for the current "Change/Add Wallet".

It will be the first screen you see when launching AlphaWallet (already imported wallets).
There are two ways to enter this screen
a) from the wallet tab, you tap on a blockie
b) You Open Settings -> Change/Add Wallet. So the same as we have now.

Zeplin:
https://zpl.io/aBQO6xK

If you open the summary of all wallets, it will work the same as Watch-only but for all wallets you imported (watch-only and that on-chain)

Updated and ready for the development

Make lines I marked in red in gray colors #dddddd Mike

Instead of USD symbol, add $ prior to the number
Add spacing or commas every 3 numbers

for example: $3 400,22  or $3,456.23
@hboon I think we can unify decimals.
so
$1.50
$233.45
$45,453.33
$000,000,000.00

We don't need to show which wallet you are in (checkmark). Instead, show unified icons for all. https://zpl.io/bAKlY3q

We don't need to show which wallet you are in (checkmark). Instead, show unified icons for all.

Do you mean to show the chevron for the current wallet too? Wouldn't it make usability worse by making it harder for users to go back to the current wallet? And this screen is also accessible in Settings > Change /Add Wallet where we show a checkmark against the current wallet.

Yes, chevron for the current wallet too.
https://invis.io/7TZHRQ8W265
It works fine with Zerion. We have few more indicators such as portfolio value, blockie, wallet name to get the difference.


nomad: 1.7.3 -> 1.7.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hashicorp/nomad/releases.
meta.description for nomad is: A Distributed, Highly Available, Datacenter-Aware Scheduler
meta.homepage for nomad is: https://www.nomadproject.io/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.7.2 with grep in /nix/store/8idm3d2h4vj73vd9qzjxv96lp62wklqg-nomad-1.7.2
found 1.7.2 in filename of file in /nix/store/8idm3d2h4vj73vd9qzjxv96lp62wklqg-nomad-1.7.2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
nomad
nomad_1_6

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8idm3d2h4vj73vd9qzjxv96lp62wklqg-nomad-1.7.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nomad https://github.com/r-ryantm/nixpkgs/archive/bd47fadd395be96812da15c48dbc1a38fcf9f750.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/bd47fadd395be96812da15c48dbc1a38fcf9f750#nomad

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8idm3d2h4vj73vd9qzjxv96lp62wklqg-nomad-1.7.2
ls -la /nix/store/8idm3d2h4vj73vd9qzjxv96lp62wklqg-nomad-1.7.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

nomad (nomad_1_6)

Maintainer pings
cc @rushmorem @pradeepchhetri @endocrimes @amaxine @techknowlogick @cottand for testing.

Converting to draft so this doesn't get reopened again.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hashicorp/nomad/releases.
meta.description for nomad is: A Distributed, Highly Available, Datacenter-Aware Scheduler
meta.homepage for nomad is: https://www.nomadproject.io/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 1.7.4 with grep in /nix/store/wk0bcyy2nq8bj0ibxwqmndrm5hccf7pv-nomad-1.7.4
found 1.7.4 in filename of file in /nix/store/wk0bcyy2nq8bj0ibxwqmndrm5hccf7pv-nomad-1.7.4

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
nomad
nomad_1_7

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wk0bcyy2nq8bj0ibxwqmndrm5hccf7pv-nomad-1.7.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nomad https://github.com/r-ryantm/nixpkgs/archive/afe4c002df412664d6f5130335102bf4145639c1.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/afe4c002df412664d6f5130335102bf4145639c1#nomad

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wk0bcyy2nq8bj0ibxwqmndrm5hccf7pv-nomad-1.7.4
ls -la /nix/store/wk0bcyy2nq8bj0ibxwqmndrm5hccf7pv-nomad-1.7.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

nomad (nomad_1_7)

Maintainer pings
cc @rushmorem @pradeepchhetri @endocrimes @amaxine @techknowlogick @cottand for testing.


SignInManager:ExternalLoginSignInAsync needs flag to bypass 2fa

Currently if a user's account in ASP.NET Identity is configured to require 2fa it's required for both password and external authentication. Often the external provider already does the 2fa. There should be a param on the ExternalLoginSignInAsync to bypass the 2fa step, but still keep the other checks (primarily the call to PreSignInCheck).

fa976e7765db498b7587d4028da76f533d1590ff


How to enable Deterministic Platform Encryption?

Per the Salesforce Security Guide, you need to enable Deterministic Platform Encryption in Setup > Platform Encryption > Advanced Settings before you can create a Deterministic Tenent Secret.  I tried to create one directly (sfdx force:data:record:create -s TenantSecret -v "Description=test Type=DeterministicData") but got the error "You can't manage tenant secrets with the DeterministicData type."  How can I enable Deterministic Platform Encryption from DX?  Logging in to perform a manual step prevents the use of DX for CI.

Also:  after creating the TenantSecret from DX, I cannot enable Deterministic PE manually either.  When I try to flip the switch in Advanced Settings, I get an error, "You must generate or upload key material before enabling Deterministic Encryption."  But when I go to the Key Management page, I see a key.  Looks just like the one my standard DE org where I was able to enabled DPE.  Is there an issue with the way the TenantSecret is created from DX?

@jclark-dot-org  I'm having this problem too.


Added Agility CMS to this list of Headless CMSs

https://agilitycms.com

Hi There!
Is there anything else I can do to help to approve these pull requests?

Sorry for the delay - looks great, thanks!

Great, thank you 💃


error on changing icon

Description
described earlier
Steps To Reproduce
won't replace folder ico with new ico
Requirements

e.g. Fix crash that occurs when...
nothing happens

Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.1928
Log File
debug.log

Is this the same issue as #12876


Set default container security context on OpenShift

What does this PR do?
Set the default value for ContainerSecurityContext to match the security context required by OpenShift's restricted-v2 SCC. This ensures that workspaces don't accidentally use a higher-priority, lower-permission SCC (e.g. one that requires readOnlyRootFilesystem: true)
What issues does this PR fix or reference?
Closes https://github.com/devfile/devworkspace-operator/issues/1170
Is it tested? How?
I haven't had a chance to test it yet (to come, tomorrow)
To verify:

On Kubernetes, behavior is unchanged, containers do not get anything set for container security context
On OpenShift, verify that #1170 cannot be reproduced; pod/container spec should look the same as before as the restricted-v2 SCC sets fields matching the default
Also necessary to verify that this change does not interfere with how Eclipse Che sets the default security context to allow for podman build

PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v8-devworkspace-operator-e2e, v8-che-happy-path to trigger)

[ ] v8-devworkspace-operator-e2e: DevWorkspace e2e test
[ ] v8-che-happy-path: Happy path for verification integration with Che

/retest


Google Analytics: adding more statistics tracking

Using an environmental variable for storing the tracking code.

+1


zilencer: Handle deleted realms nicely at server/analytics.

Accomplishes these points:

[x] Mark it as deactivated; this should allow code like sending push notifications to just have to check deactivated to decide whether to send notifications or do other processing on behalf of such a realm.
[x] Mark it as a new locally_deleted flag.
[x] Don't include locally_deleted realms in the realms response from the bouncer analytics endpoint, so the client self-hosted server doesn't need to process them.

Doesn't yet do:

[ ]  If the UUID shows up again later, we can undo that.
[ ] Log warning or something depending  on plan status relating to the RemoteRealm

Merged, thanks @mateuszmandera!


Cancel observe-k8s WG meetings

Please cancel the currently scheduled observe-k8s WG meetings for Dec 20th and Jan 3rd

Created https://cncfservicedesk.atlassian.net/servicedesk/customer/portal/1/CNCFSD-2088

@kenfinnigan I tried to delete the events from the CNCF Community calendar but we don't have access to delete events on this calendar -  CNCF Community Meetings Created by<EMAIL_ADDRESS>@halcyondude Thanks for opening up a service desk ticket.

They've responded, meetings have been removed, which I've confirmed.


spurious warnings about use of non-AD enabled functions

In dCJS functions in nimbleEcology, we use is.na() on inputs that are the result of ADbreak.
Our checking for non-AD enabled functions is nowhere near sophisticated enough to be able to determine that. And this is a general issue that users could experience when writing AD-enabled nimbleFunctions.
One simple option is to give more nuance to the warning message (saying that this is ok if the function is run on a variable that does not have derivative tracking enabled), and provide an option that allows the warning to be turned off.

@perrydv and I discussed offline and agreed that adding text along the lines of
"foo does not supporte derivatives. Be sure that arguments are either not of the double type or are in the ignore list of the buildDerivs argument to nimbleFunction."
to the warning is probably the way to go.

Doing this on cleanups_v1.3.0 branch.


Synthtech importer hotfix

Extension of #643.
Was going to manually merge the referenced PR but turns out there were more issues than I originally thought so could use some other eyes.

Re-opened. Going to take a look at this again, so mods can keep the site running while we do secret stuff.

Good to go :+1: /cc @NuckChorris
MALImport.new(:anime, 5114).metadata
=> {:external_id=>5114,
 :title=>
  {:canonical=>"Fullmetal Alchemist: Brotherhood",
   :unknown=>["Hagane no Renkinjutsushi (2009)", "Fullmetal Alchemist (2009)", "FMA"],
   :en_us=>nil,
   :ja_jp=>"鋼の錬金術師"},
 :synopsis=>
  "In this world there exist alchemists, people who study and perform the art of alchemical transmutation—to manipulate objects and transform one object into another. They are bounded by the basic law of alchemy: in order to gain something you have to sacrifice something of the same value.\r\n\r\nThe main character is the famous alchemist Edward Elric—also known as the Fullmetal Alchemist—who almost lost his little brother, Alphonse, in an alchemical accident. Edward managed to attach his brother's soul to a large suit of armor. While he did manage to save his brother's life, he paid the terrible price of his limbs.\r\n\r\nTo get back what they've lost, the brothers embark on a journey to find the Philosopher's Stone that is said to amplify the powers of an alchemist enormously; however, on the way, they start uncovering a conspiracy that could endanger the entire nation, and they realize the misfortunes brought upon by the Philosopher's Stone.\r\n\r\nFullmetal Alchemist: Brotherhood is a re-telling of the story from the manga's point of view.",
 :poster_image=>#<URI::HTTP:0x007f89abfc9ba8 URL:http://cdn.myanimelist.net/images/anime/5/47421l.jpg>,
 :type=>"TV",
 :status=>"Finished Airing",
 :genres=>["Action", "Adventure", "Drama", "Fantasy", "Magic", "Military", "Shounen"],
 :dates=>[Sun, 05 Apr 2009, Sun, 04 Jul 2010],
 :producers=>["Bones", "Aniplex", "Square Enix", "FUNimation Entertainment", "Mainichi Broadcasting", "Studio Moriken"],
 :age_rating=>"R17+",
 :age_rating_guide=>"Violence, Profanity",
 :episode_count=>64,
 :episode_length=>24}

LGTM! :thumbsup:
Also, was Brotherhood really 64 episodes!?

It was always 64 episodes ;)


Fix overfow in matmul test

Fixes  #2334

Jenkins, test this please.

Jenkins CI test (for commit c4a9fb4f4a620eb7db70c3f3613003792e304feb, target branch master) failed with status FAILURE.

Not equal to tolerance rtol=0.001, atol=0.001 :pensive:

Hm, how can I fix this? 🤔
Adjusting tolerances?

Fixed by using shaped_random instead of shaped_arange.

Jenkins, test this please.

Jenkins CI test (for commit 72ea4910cff196a8e8bfa3450c4bc13a0ec1b8a1, target branch master) succeeded!

/test
Testing again using the latest master.
cc/ @toslunar


Add .travis.yml, show coverage report

Description
Add .travis.yml
Add codecov dev-dev
Show coverage when testing
Motivation and Context
CI
How Has This Been Tested?
npm test
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[X] All new and existing tests passed.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b729b13). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #16   +/-   ##
=========================================
  Coverage          ?   94.04%
=========================================
  Files             ?        3
  Lines             ?      168
  Branches          ?       15
=========================================
  Hits              ?      158
  Misses            ?       10
  Partials          ?        0

Continue to review full report at Codecov.

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


make clang dynamic layer

enable GCC by making Clang settings bbappends if meta-clang is present

https://github.com/meta-flutter/meta-flutter/pull/303


Future of AFE?

More a strategic question, but is there an indication whether the development on the AFE will be continued, or is it "stale" (the last update to AFE was in January)?
Reason being is that LAMBDA is mainly useful with that tooling and training colleagues in the use of LAMBDA is a waste of time if the AFE gets shuttered. Thanks!

And I love the AFE! It's saved me hours of time debugging other people's formulas and it's saved my sanity because I no longer need to try and distinguish between dark blue, dark red, dark purple, etc. paragraph colors.


chore(deps): update dependency compression-webpack-plugin to v5.0.2

This PR contains the following updates:

Package
Type
Update
Change

compression-webpack-plugin
devDependencies
patch
5.0.1 -> 5.0.2

Release Notes

webpack-contrib/compression-webpack-plugin
v5.0.2
Compare Source
⚠ BREAKING CHANGES

default value of the filename option was changed to "[path][base].gz"
removed the [dir] placeholder, please use the [path] placeholder
the Function type of the filename option should return value with placeholders, please see an example

Features

added [fragment], [base] and [path] placeholders for the filename option

Bug Fixes

caching (#194) (9de2a88)
respect immutable flag for assets

5.0.2 (2020-09-02)
Bug Fixes

do not crash when the algorithm option return non Buffer (#190) (81bf601)

5.0.1 (2020-08-22)
Bug Fixes

compatibility with webpack@5 (84e367b)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

:tada: This PR is included in version 5.4.1 :tada:
The release is available on:

v5.4.1
GitHub release

Your semantic-release bot :package::rocket:


bootstrap command not working for development builds

Describe the bug
I tried setting up glasskube in my local have been getting this error when I run the bootstrap command using the binary .
To Reproduce
Steps to reproduce the behavior:

Clone the project
Run the make command
Go to the dist directory
Run the command ./glasskube bootstrap

Expected behavior
Bootstrapping should be working as expected
Screenshots

Cluster Info (please complete the following information):

Type: kubernetes
Version: 1.27
Provider: Minikube

Glasskube Info (please complete the following information):

Package Operator version: [e.g. v0.0.1]
CLI version: [output of glasskube --version]

Additional context
Add any other context about the problem here.

As commented in the related Discord Thread.

At the moment bootstrapping only works with release versions. you could either set your local version to 0.0.2 or bootstrap glasskube from the last release version.

Another option is to copy the Manifest URL from the latest release and apply it using the --url flag, i.e. glasskube bootstrap --url=xxx.
However I agree that this is annoying and should be fixed.

@pmig can I work on this ?

Hi @yrs147! Yes, you can work on this if you'd like!
The solution we though would be appropriate here would be adding a --latest flag to the bootstrap cmd. If this flag is set, rather than bootstrapping the currently installed version, the CLI should fetch https://glasskube.dev/release.json (see checkforupdate.go) and bootstrap the resulting version.
The --latest flag is false by default in a production build but true in a dev build. To make this distinction, one could check if the config.Version is equal to "dev" or something like that.
Do you think you could do that?

Or, if you have a different idea, let us know!

Sure @kosmoz , would like to give this a try first , will get back if stuck somewhere

Hi @yrs147! Are you still working on this issue?

Hey @kosmoz ! was busy with my exams , will be completing this now and raise a PR soon .

hi @yrs147,
As this issue gets more pressing we have been wondering if you have already started? Else we would re-assign it

Hey @pmig
I have completed the fix but for some reason I am unable to commit , looking into it at the moment


added a 'name' prop

added a 'name' prop to make the star-rating div identifiable. In case we have more than one star-rating component in the code, there is no way to differentiate between them. By adding 'name' prop one can make out which star-rating component is being changed inside the changeRating event. @ekeric13

Let me take a look at it this weekend. Will need to do a minor version but something like this is definitely useful.

@ekeric13 hey, did you get the chance to look at it?

@monikamangal123 merged in =b


Unknown completion 'EXPLAIN'

I did an "EXPLAIN ANALYZE SELECT ..." call in Version 0.0.28:
🔥  SkunkException
🔥
🔥    Problem: Just constructed an unknown completion 'EXPLAIN'.  Note that your
🔥             program has not crashed. This message is here to annoy you.
🔥       Hint: Please open an issue, or open a PR adding a case in Completion.scala
🔥             and a parser in CommandComplete.scala
Side note: The result of that is always a 'text' column, which is listed in the documentation as "not supported", although it apparently is.

Cool, thanks!

@tpolecat just wondering how close it is to "good first issue", seems like some mapping is required
I can try to take a look but need some initial guidance.
Thanks!


Validate Docker 18.06 for v1.6

Validate latest Docker stable release for v1.6

The stable release hasn't been released as of July 9. If it doesn't come out before end of the week, this will have to be done with the next 1.6 release.

Docker 18.06 has been released

Ran automation using Docker 18.06 and no new issues seen.

@deniseschannon I can open a new issue if you like, but I just wanted to report that the rancher-agent-bootstrap container uses hardcoded DNS of <IP_ADDRESS> / <IP_ADDRESS> instead of the host its provisioned to on Ubuntu 18.04 + Docker 18.06.
The effect of this is that anyone using a private zone for their Rancher URL will fail to provision the rancher agent and get the: "Failed to find rancher-agent container"

Opened https://github.com/rancher/rancher/issues/14849


renderer.resize does not resize diffuseTexture and normalsTexture

Hi there,
just played a bit with pixi-lights but after calling the resize method of the renderer it looks like this:
before:

after:

I fixed the problem by resizing diffuseTextureand normalsTexture.
resize: function (width, height)
    {
      PIXI.WebGLRenderer.prototype.resize.call(this, width, height);

      if (this.diffuseTexture && this.normalsTexture) {

        this.diffuseTexture.resize(width, height);
        this.normalsTexture.resize(width, height);
      }
    },

I don't know exactly if this solves the problem entirely and it surely could be made better than checking for this.diffuseTexture && this.normalsTexture so there is no pull request.
i did the check because WebGLRenderer.prototype._initContext calls the resize method before this.diffuseTexture and this.normalsTexture are instantiated.
greeting from berlin,
tststs

not a case for v4.


Kapacitor sizing documentation?

Hi .
I'm looking for basic sizing information, and I can't see nothing on the official documentation site (https://docs.influxdata.com/kapacitor/v1.2/introduction/). I would like to know sizing on some tipical use cases.
My use case will be similar to this:
1.- Subscribing on a InfluxDB with 100K time series, and applying  2 or 3 stream rules for each serie ( 1 point / minute)
2.- ingesting graphite 100k graphite metrics, and also applying 2 or 3 stream rules for each serie ( 1 point / minute)
Greatest monitoring time window should be in both cases no more than 1 hour.
What cpu/memory /I/O networking and disk should be necessary for a good kapacitor performance?
Should I  install 2 separate processes one for each data source? or can I use only one kapacitor process?
Thank you very much

@toni-moreno Thanks for the detailed question, unfortunately I won't be able to give very specific answer as it depends mostly on how complex the tasks you want to run are.
In general Kapacitor's resource needs are as follows:

Disk - very light disk activity, Kapacitor only stores tasks and basic metadata on disk.
RAM - Needs to be enough for all the time series you have and any window nodes you use.
CPU - Kapacitor is highly concurrent so the more CPUs available the more it will use.
Network - Nothing special happening here, you will almost always saturate your CPU/RAM usage before hitting any network bottlenecks.

@nathanielc I have a well-used instance of kapacitor (EC2 c4.8xlarge, 30k points/s).  I notice a constant 2-3% iowait across each of the 36 cores.  I believe it is related to the boltDB file.  Upon closer inspection of the boltDB file, it looks like the alert() node stores a number of keys.  If I had to guess, it looks like for each .crit() of my alert() node, kapacitor may initiate a write to boltDB.  My questions are:

Does my assessment thus far seem to be reasonable? (alert() node writes to kapacitor.db which causes 2-3% iowait)
Can you speak to the write path of the kapacitor.db?  Is it fully asynchronous, or will kapacitor need to wait for the data to be written?
I tried changing my instance type to r4.16xlarge (64 cores), and I noticed the iowait% went up significantly... I believe it was double-digits.  Do you have thoughts on why this is?

I ask in this github issue because for environments that make heavy use of the alert() node, it may make sense to put kapacitor.db on faster disk.


ethernet: smsc91x: rework the device node hierarchy

Hi, various cleanups on the smsc91x driver, mainly reworking the mdio/phy hierarchy so that we can match up the init sequence but also various cleanups I bumped into.
Note that I had to hack some enabled = "okay" to test this, it does not look like it's being build in CI so that's alsomething that should e looked into.

You also need to change these four files, otherwise these samples will report build error on fvp_base_revc_2xaemv8a and fvp_baser_aemv8r platforms.
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/dhcpv4_client/boards/fvp_base_revc_2xaemv8a.overlay
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/dhcpv4_client/boards/fvp_baser_aemv8r.overlay
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/zperf/boards/fvp_base_revc_2xaemv8a.overlay
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/zperf/boards/fvp_baser_aemv8r.overlay

The other part looks good to me.

You also need to change these four files, otherwise these samples will report build error on fvp_base_revc_2xaemv8a and fvp_baser_aemv8r platforms.
https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/dhcpv4_client/boards/fvp_base_revc_2xaemv8a.overlay https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/dhcpv4_client/boards/fvp_baser_aemv8r.overlay https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/zperf/boards/fvp_base_revc_2xaemv8a.overlay https://github.com/zephyrproject-rtos/zephyr/blob/main/samples/net/zperf/boards/fvp_baser_aemv8r.overlay

Thanks @SgrrZhf, I think those are fine since the labels are the same but I did miss out a phy-handle in ./dts/arm64/fvp/fvp-aemv8r.dtsi. Added that up now all of these builds:
west build -p -b fvp_base_revc_2xaemv8a samples/net/zperf
west build -p -b fvp_baser_aemv8r samples/net/zperf
west build -p -b fvp_base_revc_2xaemv8a samples/net/dhcpv4_client
west build -p -b fvp_baser_aemv8r samples/net/dhcpv4_client

I could still move phy under mdio in those files if you want me to, end result is the same, I have no preference.

I could still move phy under mdio in those files if you want me to, end result is the same, I have no preference.

Thanks, you don't need to do that, it's okay for me.

@tbursztyka any chance you could check this out? Got a couple PRs blocked by it. Thanks!


Cyrillic file names are prohibited

File names in Cyrillic are bad tone - project is open and any person in world can use it.
Rename чай.stl, please

Fixed.


twMerge regex doesn`t work if class name includes some arbitrary values

Hi,
tailwind-merge Regex  doesn`t work for classes with expression )]
i.e.
twJoin("w-[calc(1.25rem-6px)]");
twJoin("[&:not(:disabled)]:cursor-pointer");

Created regex101 link:
https://regex101.com/r/MYwru6/1

Hi @musme! Thanks for bringing this up. Can you try  if this works for you?
 [
      "(?:twMerge|twJoin)\\(([\"'`\\[\\{]?[^\\;]*?[\"'`\\]\\}])(?:\\n*[\\);\\}]|,\\)|\n*,\n*\\))",
      "[`'\"`]([^'\"`,;]*)[`'\"`]"
 ]

https://regex101.com/r/h3TFgC/1

Hi, @paolotiu!
Doesn`t work as expected in many cases.
For example if I write classes using line breaks the regex doesnt`t work
twJoin(
  "text-ellipsis",
  "text-clip"
)

But it works if I put some comment like //") in the next line.
https://github.com/paolotiu/tailwind-intellisense-regex-list/assets/11893082/c3e5db43-09a3-4aac-b05a-9f5f798ae74f

Thanks for this! I didn't account for these cases. I'll make a few edits then post it here again

@paolotiu, thank you!

Hello @musme! Sorry for getting back to you really really late!
Can you try if this works for you?
  "tailwindCSS.experimental.classRegex": [
    ["(?:twMerge|twJoin)\\(([^;]*)[\\);]", "[`'\"`]([^'\"`,;]*)[`'\"`]"]
  ]

Hello @musme! Sorry for getting back to you really really late!
Can you try if this works for you?
  "tailwindCSS.experimental.classRegex": [
    ["(?:twMerge|twJoin)\\(([^;]*)[\\);]", "[`'\"`]([^'\"`,;]*)[`'\"`]"]
  ]

@paolotiu, hello! Thank you!!! It works in most cases. By now I found only one case then it doesnt`t work as expected.
Take a look at this code:
const switchClassName = twJoin(
    "pointer-events-none absolute top-[2px] rounded-full transition-[background-color,left] bg-white",
    "left-[2px] h-4 w-4",
    "peer-checked:left-[17px]",
    "peer-[:not(:checked):disabled]:opacity-50"
  );

There is "transition-[background-color,left]" class in the first line. Arbitrary value includes comma. The comma brakes the class hinting. But everything is ok on the next lines.
I think this regex fully covers my needs. Thanks again!
By the way I recorded this behavior.
https://github.com/paolotiu/tailwind-intellisense-regex-list/assets/11893082/ef51698c-c549-4445-98f9-d5ef30267ede

This updated one should fix the behavior!
  "tailwindCSS.experimental.classRegex": [
    ["(?:twMerge|twJoin)\\(([^;]*)[\\);]", "[`'\"`]([^'\"`;]*)[`'\"`]"]
  ]

Thanks for giving the video and code! It was a big help

@paolotiu, now everything is cool! Thanks a lot!


feat: major version bump

fix version bump automation
BREAKING CHANGE: major version bump

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Package Issue]: Microsoft.VCRedist.2015+.x64 14.34.31931.0

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
Microsoft.VCRedist.2015+.x64 14.34.31931.0
Updating all my packages using the winget upgrade --all command updates this product. Then after typing it again, then it upgrades this again. After a day, this updates again.
Weird Issue
Steps to reproduce
Install this Microsoft.VCRedist.2015+.x64 14.34.31931.0
Type winget upgrade --all and let it finish
then
Type winget upgrade --all and let it finish
then
Restart your PC
Type winget upgrade --all and let it finish
then
Shutdown your PC and wait for approximately 10 hours
Type winget upgrade --all and let it finish
Actual behavior
PowerShell 7.3.4
PS C:\Users**********> winget upgrade --all
Name                                                    Id                           Version       Available     Source
Microsoft Visual C++ 2015-2022 Redistributable (x86) -… Microsoft.VCRedist.2015+.x64 14.34.31931.0 14.34.31938.0 winget
1 upgrades available.
1 package(s) have version numbers that cannot be determined. Use --include-unknown to see all results.
(1/1) Found Microsoft Visual C++ 2015-2022 Redistributable (x64) [Microsoft.VCRedist.2015+.x64] Version 14.34.31938.0
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.visualstudio.microsoft.com/download/pr/8b92f460-7e03-4c75-a139-e264a770758d/26C2C72FBA6438F5E29AF8EBC4826A1E424581B3C446F8C735361F1DB7BEFF72/VC_redist.x64.exe
██████████████████████████████  24.3 MB / 24.3 MB
Successfully verified installer hash
Starting package install...
Successfully installed
1 package(s) have version numbers that cannot be determined. Use --include-unknown to see all results.
PS C:\Users**********> winget upgrade --all
Name                                                    Id                           Version       Available     Source
Microsoft Visual C++ 2015-2022 Redistributable (x86) -… Microsoft.VCRedist.2015+.x64 14.34.31931.0 14.34.31938.0 winget
1 upgrades available.
1 package(s) have version numbers that cannot be determined. Use --include-unknown to see all results.
(1/1) Found Microsoft Visual C++ 2015-2022 Redistributable (x64) [Microsoft.VCRedist.2015+.x64] Version 14.34.31938.0
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.visualstudio.microsoft.com/download/pr/8b92f460-7e03-4c75-a139-e264a770758d/26C2C72FBA6438F5E29AF8EBC4826A1E424581B3C446F8C735361F1DB7BEFF72/VC_redist.x64.exe
███▋                            3.00 MB / 24.3 MB
Package download canceled.
Cancelled
1 package(s) have version numbers that cannot be determined. Use --include-unknown to see all results.
Expected behavior
PowerShell 7.3.4
PS C:\Users**********> winget upgrade --all
Name                                                    Id                           Version       Available     Source
Microsoft Visual C++ 2015-2022 Redistributable (x86) -… Microsoft.VCRedist.2015+.x64 14.34.31931.0 14.34.31938.0 winget
1 upgrades available.
1 package(s) have version numbers that cannot be determined. Use --include-unknown to see all results.
(1/1) Found Microsoft Visual C++ 2015-2022 Redistributable (x64) [Microsoft.VCRedist.2015+.x64] Version 14.34.31938.0
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.visualstudio.microsoft.com/download/pr/8b92f460-7e03-4c75-a139-e264a770758d/26C2C72FBA6438F5E29AF8EBC4826A1E424581B3C446F8C735361F1DB7BEFF72/VC_redist.x64.exe
██████████████████████████████  24.3 MB / 24.3 MB
Successfully verified installer hash
Starting package install...
Successfully installed
1 package(s) have version numbers that cannot be determined. Use --include-unknown to see all results.
PS C:\Users**********> winget upgrade --all
1 upgrades available.
1 package(s) have version numbers that cannot be determined. Use --include-unknown to see all results.
Environment
Windows Package Manager v1.4.10173
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22621.1555
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.19.10173.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

User Settings: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\settings.json

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Screenshots and Logs
WinGet-2023-04-23-06-47-11.692.log
WinGet-2023-04-23-06-47-16.475.log
WinGetCOM-2023-04-16-19-23-31.063.log
WinGetCOM-2023-04-20-19-01-41.066.log
WinGet-Microsoft.VCRedist.2015+.x<IP_ADDRESS>938.0-2023-04-20-19-07-58.825.log
WinGet-Microsoft.VCRedist.2015+.x<IP_ADDRESS>938.0-2023-04-20-19-07-58.825_000_vcRuntimeMinimum_x64.log
[WinGet-Microsoft.VCRedist.2015+.x<IP_ADDRESS>938.0-2023-04-20-19-07-58.825_001_vcRuntimeAdditional_x64.log](https://github.com/microsoft/winget-pkgs/files/11302441/WinGet-Microsoft.VCRedist.2015%2B.x<IP_ADDRESS>938.0-2023-04-20-
WinGet-2023-04-21-21-11-52.758.log
WinGet-2023-04-22-19-12-17.072.log
19-07-58.825_001_vcRuntimeAdditional_x64.log)
WinGet-Microsoft.VCRedist.2015+.x<IP_ADDRESS>938.0-2023-04-22-19-14-38.100.log
WinGet-Microsoft.VCRedist.2015+.x<IP_ADDRESS>938.0-2023-04-23-06-42-57.962.log
WinGet-2023-04-17-19-24-50.897.log
WinGet-2023-04-20-19-01-52.887.log
WinGet-2023-04-22-19-13-23.458.log
WinGet-2023-04-22-19-13-34.285.log
WinGet-2023-04-22-19-13-59.448.log
WinGet-2023-04-22-19-14-13.070.log
WinGet-2023-04-22-19-27-07.551.log
WinGet-2023-04-22-19-34-51.995.log
WinGet-2023-04-22-19-39-27.379.log
WinGet-2023-04-22-19-49-13.127.log
WinGet-2023-04-22-19-54-34.900.log
WinGet-2023-04-22-19-56-49.231.log
WinGet-2023-04-22-19-57-34.130.log
WinGet-2023-04-22-20-09-12.139.log
WinGet-2023-04-22-21-16-24.506.log
WinGet-2023-04-22-21-31-34.220.log
WinGet-2023-04-23-05-47-21.444.log
WinGet-2023-04-23-06-42-18.592.log
WinGet-2023-04-23-06-42-47.151.log
WinGet-2023-04-23-06-43-09.463.log
WinGet-2023-04-23-06-47-08.504.log

See https://github.com/microsoft/winget-cli/issues/2995

Duplicate of https://github.com/microsoft/winget-cli/issues/2427


Error: Building package gsoap:x64-windows-static failed

Package: gsoap:x64-windows-static
Vcpkg version: 2020.02.04-nohash
CMake Error at scripts/cmake/vcpkg_fail_port_install.cmake:93 (message):
Architecture 'x64' not supported by gsoap!
Call Stack (most recent call first):
ports/gsoap/portfile.cmake:1 (vcpkg_fail_port_install)
scripts/ports.cmake:90 (include)

Hi @tom2068
Thanks for posting this issue.
Since gsoap only supports x86. The current behavior is intended.

gsoap has been updated to 2.8.102 and has added the support for x64 via PR #11355.
Please update vcpkg and try to rebuild this port.


[OTHER] Update COC email to link

What would you like to share?
Change Code of Conduct's email address to link.
The new link is: http://eddiejaoude.io/contact
Additional information
No response

Hi @badex-ai, yes just change it to http://eddiejaoude.io/contact

If it's okay I'd like to contribute to this issue


Add Empty States for "Add Items" Modal on Shipments and Purchase Orders Page

What is the motivation for adding/enhancing this feature?
Currently, when users open the "Add Items" modal on the Shipments and Purchase Orders page, there is no clear indication of what to do or what items can be added. This lack of information can be confusing for users, especially when they are new to the system.
What are the acceptance criteria?
To enhance user understanding and guide them effectively, we should implement an empty state for the "Add Items" modal. This empty state will provide a clear message or visual representation that there are no items to add, ensuring that users are aware of the context.
Can you complete this feature request by yourself?

[ ] YES
[ ] NO

Additional information
Find UI here:
https://www.figma.com/file/j3l0F2DioUHEuQT53NOQJY/Ionic-6-Material-UI-Kit-(Community)-(Nidhi--2023)?type=design&node-id=25814%3A115756&mode=design&t=vlJFFIZBdh4ynJZS-1)[https://www.figma.com/file/j3l0F2DioUHEuQT53NOQJY/Ionic-6-Material-UI-Kit-(Community)-(Nidhi--2023)?type=design&node-id=25814%3A115756&mode=design&t=vlJFFIZBdh4ynJZS-1](https://www.figma.com/file/j3l0F2DioUHEuQT53NOQJY/Ionic-6-Material-UI-Kit-(Community)-(Nidhi--2023)?type=design&node-id=25814%3A115756&mode=design&t=vlJFFIZBdh4ynJZS-1

hello @Dhiraj1405 sir, i want to contribute in this issue can you assign me this issue?

Sure @Preetvani07 go ahead

hello @Dhiraj1405 sir, i want to contribute in this issue can you assign me this issue?

Sure, go ahead.

sir i mistakely unassigned this issue can you please reassign me this issue


Include tests consisting of unittest, Pytest and Flake8

There is a need to add implementation of unittests, Pytest and Flake8 in this project.
Yes, I would like to work on this issue

Let's stick with unittest since it's in the standard library