add systemd_unit try-restart, reload-or-restart, reload-or-try-restart actions

systemd has some fancy unit commands that do roughly the equivalent of the service resource's "supports" behavior for reloads, where you can say "reload-or-restart" and it'll issue a restart if the unit doesn't support reloading (as defined by the ExecReload directive);
they're defined as:

try-restart: Restart one or more units if active
reload-or-restart: Reload one or more units if possible, otherwise start or restart
reload-or-try-restart: Reload one or more units if possible, otherwise restart if active

i guess it would be too confusing a behaviour change to just default restart to try-restart, but I'm fine with adding these. (And reload-or-try-restart as the default reload)

our general semantics are that :restart falls back to :start and if there's no reload_command i think we revert to :restart don't we?  in general we try to DTRT, and i'd prefer not to add more fussy actions and to just make the actions that we have be more resiliant.

in general we try to DTRT

to my mind this means correctly reflecting the default behaviors of what we're interacting with, without surprises; in the case where the system we're modeling provides explicit commands for the behaviors that other resources are providing as "extra" behaviors that aren't actually in the underlying utilities (i.e. service resource action fallback modes), is there any reason we wouldn't want to expose the "native" facilities directly as resource actions?

Actually that's not what we tend to do.  If the underlying actions are brittle, then we usually wrap those with idempotent actions that are more resilient.  If that result looks really awful then we tend not to.  But for example if a sysv init script doesn't implement :restart we fall back to :start+:stop for the user rather than just bailing with an error and forcing the user to do both of those actions manually.

I don't think we lose anything by including these, and users may well expect them 👍

okay so i guess these are trivial.
for the future record if anyone finds this and agrees with me that this is overly fussy and :restart should just hit try-restart consider this notice that i'd like to see that PR.
👍

@lamont-granquist thanks; i can definitely see some justification for deviating from the default system behavior by falling back to restart if a reload fails or similar rescuing behaviors (maybe even just using reload-or-restart instead of reload). my concern with replacing restart with try-restart in particular is that try-restart is a NOP if the unit isn't already active, which is pretty contrary to what i think most people reasonably expect: that issuing a restart would result in the service running.
[nathwill@wyrd ~]$ systemctl status redis.service
● redis.service - Redis persistent key-value database
   Loaded: loaded (/usr/lib/systemd/system/redis.service; disabled; vendor preset: disabled)
  Drop-In: /etc/systemd/system/redis.service.d
           └─limit.conf
   Active: inactive (dead)
[nathwill@wyrd ~]$ sudo systemctl try-restart redis.service
[nathwill@wyrd ~]$ systemctl status redis.service
● redis.service - Redis persistent key-value database
   Loaded: loaded (/usr/lib/systemd/system/redis.service; disabled; vendor preset: disabled)
  Drop-In: /etc/systemd/system/redis.service.d
           └─limit.conf
   Active: inactive (dead)

that's definitely weird, and i don't care for it, but it makes a little sense in light of systemd being api-driven with the unit having an internal state, instead of being a script that's being executed.


zcbor.py: Do conversion to UINT32_MAX etc. only when doing code generation

It breaks data translation/validation if applied there.
Fixes #297

Most of the changes are tests.


Fix clippy

It seems like this version of Rust does not like to have struct and impl in different files.

I have tried and clippy is not happy:
error: associated function `new` is never used
  --> mobile_verifier/tests/common/mod.rs:26:12
   |
25 | impl MockHexBoostingClient {
   | -------------------------- associated function in this implementation
26 |     pub fn new(boosted_hexes: Vec<BoostedHexInfo>) -> Self {
   |            ^^^
   |
   = note: `-D dead-code` implied by `-D warnings`
   = help: to override `-D warnings` add `#[allow(dead_code)]`

But new is used...


Refactor builder out as an utility

Move the IR builder utility out as a separate utility.
TODO:

Should this be moved into the ir folder?
Should Builder be merged with the Tape class?
Eventually, merge this with trace-mode onnxscript and expose it to end users.

All sounds good!


Scrub parameters before sending exception to Sentry

Instead send the parameters that might be interesting for the exception,
i.e user_id, payload_data etc.

Have we looked into what Raven's sanitizing processors are providing for us? http://raven.readthedocs.org/en/latest/config/#sanitizing-data

@gylaz neither of the options they provided did what we want. So it felt right doing a more "homegrown" solution

What we want is to remove the local parameters from the exception, where we send in things the like user's tokens. Which we don't want stored anywhere

:+1:

Thanks for the feedback @gylaz, merging


Tried with width 100%, not working.

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


Can't download dataset with downloader.py script

When trying to download the dataset using the downloader.py script I hit two issues:

The recommended environment currently seems to install pydantic 2.0 by default, which is incompatible with qcportal. This is easily fixed by pinning the pydantic version to 1.10

For some reason the connection is refused to the qcarchive server, no matter what I do. I'm not too familiar with the qcarchive api, any insights on how to debug this would be appreciated.

The traceback and output are:
Processing SPICE Solvated Amino Acids Single Points Dataset v1.1
Processing SPICE Dipeptides Single Points Dataset v1.2
Traceback (most recent call last):
  File "/spice-dataset/downloader/downloader.py", line 83, in <module>
    recs = ds.get_records(method=spec['method'], basis=spec['basis'], program=spec['program'], keywords=spec['keywords'])
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/envs/spice/lib/python3.11/site-packages/qcportal/collections/dataset.py", line 1606, in get_records
    df = self._get_records(indexer, history, include=include, merge=merge, status=status)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/envs/spice/lib/python3.11/site-packages/qcportal/collections/dataset.py", line 1158, in _get_records
    records.extend(self.client.query_results(**query_set, status=status))
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/envs/spice/lib/python3.11/site-packages/qcportal/client.py", line 718, in query_results
    response = self._automodel_request("result", "get", payload, full_return=True)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/envs/spice/lib/python3.11/site-packages/qcportal/client.py", line 274, in _automodel_request
    r = self._request(rest, name, data=payload.serialize(self.encoding), timeout=timeout)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/envs/spice/lib/python3.11/site-packages/qcportal/client.py", line 233, in _request
    raise ConnectionRefusedError(_connection_error_msg.format(self.address)) from None
ConnectionRefusedError:

Could not connect to server https://api.qcarchive.molssi.org:443/, please check the address and try again.

No idea about the connection refused.  Perhaps QCArchive is having temporary issues?  I'd wait a little while and try again.
The recommended way of getting the dataset is to download it from Zenodo.

@peastman Thank you very much for the prompt response, the zenodo link works correctly. looks like QCArchive is still having issues with my connection.Feel free to close the issue if you want.

Copying in @bennybp, who might be able to help us update the SPICE download scripts for reproducibility.


proxy support

Hello,
I added proxy support for your script.
If you find it necessary, please add my changes to the master.

Thanks for contributing this.


Is 'nanoeX' the same as 'nanoe-G'?

Sorry if this should be obvious. I'm assuming the X might be a placeholder for a number of different options, but the possibility of causing damage to the unit has me hesitant to enable it.

No, nanoeX is a different feature. So far I haven't seen nanoe-G being reported via the wifi adapter so it's currently not possible to toggle it.
You can try enabling it via the IR remote and posting a verbose log with it on and off.

I'll see if I can do that once I actually get this thing installed. Might be a long wait for the right cable to arrive. Feel free to close this in the meantime.

@dixonte Even from the official app, there is no option to toggle nanoe-G. The verbose log does not change when toggling  nanoe-G from IR. This is true for my AC(CS-S13TKH)

@blacklizard Good to know. Just in case I might do my own tests when I am finally able to build this thing, but thanks for the heads up.


Add crud methods to issue boards

These 4 methods were missing from the IssueBoards class:

show (https://docs.gitlab.com/ee/api/boards.html#single-board)
create (https://docs.gitlab.com/ee/api/boards.html#create-a-board)
update (https://docs.gitlab.com/ee/api/boards.html#update-a-board)
remove (https://docs.gitlab.com/ee/api/boards.html#delete-a-board)

I also added the corresponding unit tests

by mistake, this PR also include changes of PR #353


Adds some logic to the configuration provider to default config value…

…s with missing env values to the extension default values.

@chrisdp can you take a look at the failing tests and get them to pass? https://travis-ci.org/rokucommunity/vscode-brightscript-language/builds/626797825?utm_source=github_status&utm_medium=notification


Sphinx docs

Documentation for qgrad

The __call__ method in the Displace class does not document. Any ideas why?

The __call__ method in the Displace class does not document. Any ideas why?

Even when I manually change the .rst file for Displace, it overwrites it to default upon build.


Unsupported operation: Platform._operatingSystem

Describe the bug
when I use this plugin on flutter web, it crashes, the bug description is "Unsupported operation: Platform._operatingSystem
The relevant error-causing widget was: ", it says "flutter_easyloading-2.2.2/lib/src/widgets/loading.dart:62:14", the code as follow
@override
  Widget build(BuildContext context) {
    return Material(
      child: Overlay(
        initialEntries: [
          EasyLoadingOverlayEntry(
            builder: (BuildContext context) => widget.child,
          ),
          _overlayEntry,
        ],
      ),
    );
  }

Screenshots

I am getting following for web when trying to display a toast:
Error: Unsupported operation: Platform.operatingSystem
at Object.throw [as throw] (http://localhost:56903/dart_sdk.js:5347:11)
at Function._operatingSystem (http://localhost:56903/dart_sdk.js:56279:17)
at Function.get operatingSystem [as operatingSystem] (http://localhost:56903/dart_sdk.js:56325:27)
at get _operatingSystem (http://localhost:56903/dart_sdk.js:56238:27)
at Function.desc.get [as _operatingSystem] (http://localhost:56903/dart_sdk.js:5852:17)
at Function.get operatingSystem [as operatingSystem] (http://localhost:56903/dart_sdk.js:56191:26)
at flashlight_off (http://localhost:56903/packages/musicplayer/fourthpage.dart.lib.js:1763:73)
at flashlight_off.next ()
at runBody (http://localhost:56903/dart_sdk.js:39187:34)
at Object._async [as async] (http://localhost:56903/dart_sdk.js:39218:7)
at fourthpage.FourthPage.new.flashlight_off (http://localhost:56903/packages/musicplayer/fourthpage.dart.lib.js:1753:20)
at fourthpage.FourthPage.new. (http://localhost:56903/packages/musicplayer/fourthpage.dart.lib.js:1806:77)
at Generator.next ()
at runBody (http://localhost:56903/dart_sdk.js:39187:34)
at Object._async [as async] (http://localhost:56903/dart_sdk.js:39218:7)
at http://localhost:56903/packages/musicplayer/fourthpage.dart.lib.js:1805:427
at ink_well._InkResponseState.new.[_handleTap] (http://localhost:56903/packages/flutter/src/material/icon_button.dart.lib.js:50329:42)
at tap.TapGestureRecognizer.new.invokeCallback (http://localhost:56903/packages/flutter/src/gestures/recognizer.dart.lib.js:203:18)
at tap.TapGestureRecognizer.new.handleTapUp (http://localhost:56903/packages/flutter/src/gestures/tap.dart.lib.js:417:40)
at tap.TapGestureRecognizer.new.[_checkUp] (http://localhost:56903/packages/flutter/src/gestures/tap.dart.lib.js:223:12)
at tap.TapGestureRecognizer.new.acceptGesture (http://localhost:56903/packages/flutter/src/gestures/tap.dart.lib.js:199:23)
at arena.GestureArenaManager.new.sweep (http://localhost:56903/packages/flutter/src/gestures/arena.dart.lib.js:222:31)
at binding$5.WidgetsFlutterBinding.new.handleEvent (http://localhost:56903/packages/flutter/src/gestures/binding.dart.lib.js:402:27)
at binding$5.WidgetsFlutterBinding.new.dispatchEvent (http://localhost:56903/packages/flutter/src/gestures/binding.dart.lib.js:381:24)
at binding$5.WidgetsFlutterBinding.new.dispatchEvent (http://localhost:56903/packages/flutter/src/rendering/layer.dart.lib.js:6107:13)
at binding$5.WidgetsFlutterBinding.new.[_handlePointerEventImmediately] (http://localhost:56903/packages/flutter/src/gestures/binding.dart.lib.js:352:14)
at binding$5.WidgetsFlutterBinding.new.handlePointerEvent (http://localhost:56903/packages/flutter/src/gestures/binding.dart.lib.js:325:43)
at binding$5.WidgetsFlutterBinding.new.[_flushPointerEventQueue] (http://localhost:56903/packages/flutter/src/gestures/binding.dart.lib.js:314:14)
at binding$5.WidgetsFlutterBinding.new.[_handlePointerDataPacket] (http://localhost:56903/packages/flutter/src/gestures/binding.dart.lib.js:304:65)
at Object.invoke1 (http://localhost:56903/dart_sdk.js:185349:7)
at _engine.EnginePlatformDispatcher..invokeOnPointerDataPacket (http://localhost:56903/dart_sdk.js:165680:15)
at _engine.PointerBinding..[_onPointerData] (http://localhost:56903/dart_sdk.js:166338:49)
at http://localhost:56903/dart_sdk.js:166795:28
at http://localhost:56903/dart_sdk.js:166748:16
at http://localhost:56903/dart_sdk.js:166442:11

same issue

There is a dart native operating system plugin

same issue

sorry, I can not reproduce  the problem, it will be helpfull if you can upload a demo project

same issue

sorry, I can not reproduce the problem, it will be helpfull if you can upload a demo project

Flutter web also generally gives this error. Can you give a sample working with Flutter web?

@ufukhawk  example is OK
https://github.com/kokohuang/flutter_easyloading/tree/develop/example
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.0.3, on macOS 11.0.1 20B29 darwin-x64, locale
    zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.0)
[✓] IntelliJ IDEA Ultimate Edition (version 2020.3.2)
[✓] VS Code (version 1.54.3)
[✓] Connected device (3 available)

Okay, thank you! worked.

Your plugin is awesome, been using it for quite sometime, always loved it and I am so sad to drop it this week. Flutter is so much simpler with "Get" framework plugin, also rates #3 on plugins worldwide, goodbye bloc thankgod. Their built-in snack bar is good enough for my purposes, but some options on it crash web plugin. I really hope you can include this in their project, I really liked look and feel of this, just no point in me using it now that I have built-in support with GetX.
Awesome job, I really hope you submit a pull request on their project.

@syleishere thanks


[Question] Why is opensearch-env different between rpm and tar builds?

I tried to compare rpm and tar builds from the files.
And they apparently have many differences within opensearch-env.
The rpm even hard code to source from /etc/sysconfig/opensearch for env vars

38,42c38,39
< # now set the path to java: OPENSEARCH_JAVA_HOME -> JAVA_HOME -> bundled JDK
< if [ ! -z "$OPENSEARCH_JAVA_HOME" ]; then
<   JAVA="$OPENSEARCH_JAVA_HOME/bin/java"
<   JAVA_TYPE="OPENSEARCH_JAVA_HOME"
< elif [ ! -z "$JAVA_HOME" ]; then
---
> # now set the path to java
> if [ ! -z "$JAVA_HOME" ]; then
84c81
< source /etc/sysconfig/opensearch
---
> if [ -z "$OPENSEARCH_PATH_CONF" ]; then OPENSEARCH_PATH_CONF="$OPENSEARCH_HOME"/config; fi
94c91
< OPENSEARCH_DISTRIBUTION_TYPE=rpm
---
> OPENSEARCH_DISTRIBUTION_TYPE=tar

This kinda pose a problem when producing bundled OpenSearch RPM with these files.
As you need to first remove the sourcing (The file only exist after you install the rpm on customer host).
Then define OPENSEARCH_PATH_CONF so that opensearch-plugin can run properly to install all the plugin zip.
Then add back the sourcing and then repackage into rpm.
However, in practice, even without sourcing /etc/sysconfig/opensearch the variables will still be sourced by systemctl service file with EnvironmentFile=-/etc/sysconfig/opensearch.
Unless someone manually run opensearch bin without using service file for RPM, then it would pose an issue.
Thanks.

Any thoughts, @dblock ?

@peterzhuamazon could you please point out which opensearch-env is used for to build rpm? I suspect this is not the same one (copy?) as used by archive distributions.

@dblock @peterzhuamazon I will take a look shortly

@peterzhuamazon I just did a fresh build of the distributions, there are only 2 differences in opensearch-env between archive and RPM (and DEB as well):

OPENSEARCH_DISTRIBUTION_TYPE=rpm vs OPENSEARCH_DISTRIBUTION_TYPE=tar
and indeed OPENSEARCH_PATH_CONF

if [ -z "$OPENSEARCH_PATH_CONF" ]; then
  source /etc/sysconfig/opensearch
fi

vs
if [ -z "$OPENSEARCH_PATH_CONF" ]; then
  if [ -z "$OPENSEARCH_PATH_CONF" ]; then OPENSEARCH_PATH_CONF="$OPENSEARCH_HOME"/config; fi
fi

It comes from the build script [1] and @dblock is right, we just inherited it from the fork, it seems to be convention the original project has followed for years.
[1]  https://github.com/opensearch-project/OpenSearch/blob/main/distribution/build.gradle#L536

@reta I am using 1.2.0 for the build, sorry for not mentioning that.
I clearly see more changes as in my description.
Not sure if newer versions have less changes tho.
Thanks.

@peterzhuamazon oh ... thanks for clarifying, will rebuild in a sec, the OPENSEARCH_JAVA_HOME presence is very suspicious in 1.2.x though - we did not merge this change [1]. I will come back with the update shortly.
[1] https://github.com/opensearch-project/OpenSearch/pull/2042

@peterzhuamazon I build fresh 1.2.5-SHAPSHOTs
-rw-rw-r-- 1 user user 223M Feb 14 15:06 opensearch-min-1.2.5-SNAPSHOT.x86_64.rpm
-rw-rw-r-- 1 user user 223M Feb 14 15:05 opensearch-min-1.2.5-SNAPSHOT-linux-x64.tar.gz

And the diff is exactly the same, I suspect the archive is not for 1.2.x (the OPENSEARCH_JAVA_HOME should not be present there in any form), could you please double check (or even better, share links to these distributions)?
Thank you!

@reta I use tags/1.2.0 to build but will test again.

Tar: distribution/archives/linux-tar/build/tar-extracted/opensearch-1.2.0/bin/opensearch-env
Rpm: distribution/packages/rpm/build/extracted/rpm-extracted/usr/share/opensearch/bin/opensearch-env
diffs:

% diff -bi ~/env-tar ~/env-rpm
81c81
< if [ -z "$OPENSEARCH_PATH_CONF" ]; then OPENSEARCH_PATH_CONF="$OPENSEARCH_HOME"/config; fi
---
> source /etc/sysconfig/opensearch
91c91
< OPENSEARCH_DISTRIBUTION_TYPE=tar
---
> OPENSEARCH_DISTRIBUTION_TYPE=rpm

Weird enough the diffs are different but anyway the source still present......

I think the source one is understandable as if anyone installed RPM but not using systemd, then there needs to be a way to source the sysconfig.
The OPENSEARCH_PATH_CONF one seems needs to stay for now, as security plugin has some hard dependencies on the config folder. If the folder/link present in OPENSEARCH_HOME/config, then demo installer will use tarball mode, despite you install a version of rpm, else if will link to /etc/opensearch hardcoded.
https://github.com/opensearch-project/security/blob/main/tools/install_demo_configuration.sh#L159-L173

Yeah, the source is "inherited", you definitely have better assumptions why it is there ...

Seems like expected behavior at this point, close issue.


feat(bunotel): add option to enable formatting of queries

Disable formatting by default (breaing change)
and add bunotel option that enables formatting.
Formatting has potential to leak information into the traces
as such not formatting the query by default seems like
safer option.
Fixes: https://github.com/uptrace/bun/issues/546

@vmihailenco would you take a look please? This is breaking change, so if you want the default to stay as it is I will have to invert the logic and have option WithUnfromattedQuery or maybe WithFormatting that would take boolean/flag param.

@matoous thanks for the PR.
Let's change WithFormatting to accept a boolean flag - I have some reasoning here. I am fine with having it disabled by default.
Also, what about renaming WithFormatting to WithFormattedQuery?

@vmihailenco thanks for feedback, updated.

Thanks :+1:


Clear method didn't reset classes in form

Versions:

VueJs: 2.4.2
Vee-Validate: 2.0.0-rc.7

Description:
The method errors.clear() reset error messages but the classes was rendered in fieldset  not reset.

Hi @logaretm,
Thought I'd comment on this ticket. The reset method you mention here is something I've been looking for in the docs but couldn't find! I feel it would be super helpful to have this in the "Guide" section of the docs. Unhelpfully I'm not sure where... If it already is and I couldn't find it apologies!

You are right, I will add it in the docs in a new guide.


chore: revert vmrecolor changes

As discussed in https://github.com/runelite/plugin-hub/pull/6737, we were not interested in accepting plugins inside plugins.
Due to this decision, Dominick went on to submit his plugin to the hub, but nobody ever cleaned up the code in this repo.
Today you attempted to update the plugin as it does not build, and all of those changes were submitted to the hub again.
This is what the for the hub should look like, post merge of this commit & update https://github.com/felanbird/UltimateVolcanicMine/compare/b89038851978b10a4fde5e478616425da7c70a99...felanbird:16f2d4a3a6355dd4d3a6db6914ca3404e9c5e92f

Superseded by #32 and a force push


Too many bundles

Good afternoon. There are about fifty enumerations in my ASP.NET Core Web API project. For each such enumeration, I have allocated ten language ftl files. Plus separate ftl contexts for general translations, separate ftl's for Exceptions. Everything is neatly arranged in folders. This ensures ease of translation management.
How to use Linguini correctly with this project structure?

Perhaps it needs clarifying, but it's one FluentBundle per language. If languages are similar, they can be handled by a single bundle, but I've yet to implement fallback languages.

@eXplowar I hope you haven't missed my reply. I would love to help you, but I need a bit more clarification m

@Ygg01, thank you for your attention to my issue. I creating an example. Later put his her.

what do you mean by enumerations
Enum's

Basic folder structure for translations:

thank you for your attention to my issue.

No worries, I'm here to try to help!
That's a very peculiar pattern you have chosen for your project. Usually, translations are divided per language. For example, SS14 uses it for localization, and they have an entirely different structure.
E.g. their structure goes something like this
Translations
 |
 +-- en-US
 |  |
 |  +-- cities.ftl
 |  |
 |  +-- common.ftl
 |  |
 |  +-- invoiceException.ftl
 |  |
 |  +-- Enum
 |       |
 |       +-- invoice.ftl
 |       |
 |       +-- payment.ftl
 |
 +-- ru-RU
      |
      ... // mirrored structure

At the end of the day, files are just a logical separator, for ease of organization, all resources for a given language are dumped into the same Dictionary.
You could theoretically have a big all in one translation file.
Translations
 |
 +-- en-US
 |  |
 |  +-- all_translations.ftl
 |
 +-- ru-RU
    |
    +-- all_translations.ftl

Like so. But it might be too challenging to navigate. Again, it depends on how you want to structure it.
Another thing to note if you find yourself with following pattern:
Entity = My entity
Entity-description = My entity description

Perhaps the better option would be to use fluent attributes
Entity = My Entity
   .description = My Entity description

@Ygg01, thanks for cool and detailed answer.

That's a very peculiar pattern you have chosen for your project. Usually, translations are divided per language. For example, SS14 uses it for localization, and they have an entirely different structure.

Perhaps. I will try take solusion from SS14. Thanks for alternative example.

At the end of the day, files are just a logical separator, for ease of organization, all resources for a given language are dumped into the same Dictionary.

I see that this is exactly how Linguini is designed. But tell me, won’t there be a problem if I have, say, both Invoice and Payment enums with similar "Paid" values? E.g. Invoice.Paid and Payment.Paid. The fact is that in the ftl-file that I write for them, the key has a value that matches the value of the enum, in our case Paid. I need this so that my extension method can find the translation by enum value. It seems to me that I will have to make a prefix to the key that corresponds to the name Enum... Perhaps this is not a big problem, and I will still make the prefix, it’s just that without it the ftl-file, which relates to some enum, is served a little more conveniently.

Perhaps the better option would be to use fluent attributes

Thanks for the information about fluent attributes.

But tell me, won’t there be a problem if I have, say, both Invoice and Payment enums with similar "Paid" values?

Yes. They would collide, and depending on how you insert a resource, it would either error out or just overwrite.
Keep in mind Fluent is a way to replace .properties like translation resources. Each translation key is expected to be unique in application. I.e. Invoiced.Paid and Payment.Paid would become keys like Invoice-Paid and Payment-Paid. You could use reflection or some other convention to transform a call to Invoice for Paid string into an Invoice-Paid id.
// assume this is a field in some internal helper class
public LinguiniBundle bundle;

public bool TryGetLocalization(string? classPrefix, string loc, IDictionary<string, IFluentType>? args, out string? message)
{

     var id = classPrefix == null ? loc : $"{classPrefix}-{loc}";
     // Note: `out _` is ignoring all errors, don't do this in actual code.
     bundle.TryGetMessage(id, args, out _, out message)
}


Plugin window instances become invalid

Version: 0.8.11
Platforms: All
Description
The histogram, CDF, and CCDF plugins all open windows that display the results in the run_post() method.  When the run_post() method returns, the Python window object must remain valid.  Otherwise, the Python object will be garbage collected leaving the window object in an invalid state causing undefined behavior.  In the existing code, the range tool finishes, which deletes all the python objects.
The range tool object must remain valid until the user closes the window.

Fixed in 0.8.16.


Structured Exploratory Testing of Changed File Detection on Merge

What

Describe at a high level what changes are in this Pull Request.
Delete this callout.

Why

Describe why your are making these changes especially their value.
Delete this callout.

Change Impact Analysis and Testing

Describe the specific impacts of your changes and how those impacts
will be vetted.
Delete this callout.

Testing push changed files example from https://github.com/tj-actions/changed-files?tab=readme-ov-file#on-push-


hpack-based "simple" template.

For when you want a simple project started with a hpack package.yaml file
instead of a pure .cabal file.
Note that this depends on this PR to stack, and that indeed this is an invalid template for older stack versions.
Does it need to somehow work if an older stack tries to use it?

Sounds like we should hold off on this at least until that PR is merged.

that PR has now been merged
.
any thoughts on this one?

LGTM.
I think it is fine that a template which requires a new stack feature only work with new stack.


slicer: Do not write split ID into parent header

Split ID relates to split-chain only, original root object should not carry this information, otherwise, it will be perceived by the system itself as part of a sliced chain that does not exist.

I consider you should discuss this with @carpawell, because we fixed/added the splitID to first header here #448, to fix some bug inside node

@smallhive, the first child obj and the root obj are not the same so that PR is correct.
@cthulhu-rider, do we have a test for "every child has a SplitID"? Slicer code is quite complex such a pair of tests would help (if not yet): (1) every part has a correct and full Split header and (2) a root object has no split info ((2) is already done in that PR).
BTW, is our API description full enough to understand that we should not add splitID to the root object?

@carpawell

do we have a test for "every child has a SplitID"?

yep, i modified place which checks case 1. https://github.com/nspcc-dev/neofs-sdk-go/blob/bc6b51e04d27cddb62c428a3fe50a66d62fb28be/object/slicer/slicer_test.go#L561-L564

API description full enough to understand

full enough? guess no. Strictly speaking, root object is not a part of split-chain cuz it's an autonomous object. When we restore original object, we don't expect to see any split headers, huh? I think so.
if you ask me why this (root split ID absence) is not a recommendation, but a requirement, I will not give an exact answer. There is an open issue https://github.com/nspcc-dev/neofs-node/issues/2451


[new-product] Protractor

Add Protractor Page

Hi @nelsongutidev , why did you discard your PR ? Getting protractor  on eol would be a great catch :pray:

Hi @adriens , thanks! I am working on some minor updates on some links but I should be re-opening the PR for review tomorrow or Friday at the latest

Hi @adriens , thanks! I am working on some minor updates on some links but I should be re-opening the PR for review tomorrow or Friday at the latest

Awesome as we use it and I find very useful to add it among the rest of our stack... my goal is to get our stack fully covered on endoflife :nerd_face:

Hi @adriens , here is the PR ready https://github.com/endoflife-date/endoflife.date/pull/3595

Yup, saw it :zap:


Looks like tools dashboard script is not reading technology accurately

looks like the stuff is pulled, but it shows a wrong technology

while there JSON doesn't have it there
{
  "title": "nestjs-asyncapi",
  "description": "Utilize decorators to generate AsyncAPI document utilizing DTOs (similar to @nestjs/swagger) and a web UI.",
  "links": {
    "repoUrl": "https://github.com/flamewow/nestjs-asyncapi"
  },
  "filters": {
    "language": "Typescript",
    "technology": ["Node.js", "NestJS"],
    "categories": ["code-first"],
    "hasCommercial": false
  }
}

Originally posted by @flamewow in https://github.com/asyncapi/website/issues/1354#issuecomment-1513245278

AsyncAPI specification version used in the tool's implementation. The tool might be using an outdated version of the specification that does not include the correct field for specifying technology. If this is the case, the tool needs to be updated to support the latest AsyncAPI specification.
you can find  it in the AsyncAPI specification file. The AsyncAPI specification file is a YAML or JSON file that describes the structure and contents of a message-based API.
The tool might not be correctly parsing the specification and thus not reading the technology field. In this case, you could try validating the AsyncAPI specification using a tool like the AsyncAPI validator to ensure that it is correctly formatted.


[pulsar-functions] Fix message consume ordering issue of function

Fix #7707
Fix message consume ordering issue of function with EFFECTIVELY_ONCE processing guarantees.
Motivation
Since v2.5.2, function with EFFECTIVELY_ONCE processing guarantees consume message out of ordering when fail, because process a CompletableFuture result by async.
Modifications

Get result from future synchronized when processing guarantees of java instance equals EFFECTIVELY_ONCE.

/pulsarbot run-failure-checks

Rebased from origin/master.

@Tiscs Can you rebase the pull request to latest master again?

@Tiscs Can you rebase the pull request to latest master again?

@sijie Done.

@Tiscs Thanks for the contribution, it seems there are still some conflicts with the master. Would you please help rebase this PR again?

@Tiscs Thanks for the contribution, it seems there are still some conflicts with the master. Would you please help rebase this PR again?

Rebased again, and fixed exception detection in processResult, but some checks were failed.

This pr seems brings too much un-related changes by the merge. Would you please follow this guide to provide a new PR?

Closed by new PR #8026.


the result of ssn_test.py

Hi yaunjun
Your work is so useful, thank you very much for your open source. I'm trying to run your code, and I want to know the meaning of the ssn_test.py result(variable rst). can you help me point them out in this code project?
thanks a lot

The usage of the score file can be found here:
https://github.com/yjxiong/action-detection/blob/master/eval_detection_results.py#L91
Do you have anything you need to use in particular?


I think the accent color of some disabled controls is inappropriate.

ToggleSwitch or Radiobutton with "IsCheckd = true" changes color properly when "IsEnable" is changed.
However, in the same situation, CheckBox only changes the color of the text, and the color of the box does not change.
I checked that it works normally in the demo, is it going to be fixed in the next update?
Will the "IsThreeState" issue also be fixed in the next update?
ToggleButton with "IsEnable = flase" does not change color in light mode and changes color slightly in dark mode compared to WinUI3 when "IsChecked" is changed.
I think the distinction should be greater like WinUI3.
And I have a few suggestions that I want "TimePicker" from WinUI3, "DataGrid" looks like WinUI3, and I want to add a control to "TitleBar".
I'd like you to check if it's possible.
Desktop (please complete the following information):

OS: Windows 10
.NET: .NET 6
Version: 2.0.2

https://github.com/lepoco/wpfui/issues/213 related

I want "TimePicker" from WinUI3

I believe this is coming in 3.0.

I want to add a control to "TitleBar".

You can set the Header property instead of Title to do that. For example, instead of:
        <wpfui:TitleBar Title="Hello world" />

You can do:
            <wpfui:TitleBar>
                <wpfui:TitleBar.Header>
                    <StackPanel Orientation="Horizontal">
                        <TextBlock>
                            <TextBlock.Inlines>
                                <Run FontWeight="DemiBold">Hello</Run>
                                <Run> world</Run>
                            </TextBlock.Inlines>
                        </TextBlock>
                    </StackPanel>
                </wpfui:TitleBar.Header>

This has been resolved in the latest previews


Upgrade node-sass

Requested features:  https://github.com/google/material-design-lite/blob/master/src/button/_button.scss#L17 (actually the file is here: https://github.com/google/material-design-lite/blob/master/src/_variables.scss)

+1

:+1:

:+1:

:+1:

Got it! At this weekend will try to build and upload a release :bomb:

It also can be closed

No, it can't.
@tadatuta try this branch https://github.com/enb-make/enb-sass/tree/imports

There are still some problems in imports branch :(
@veged got a case with broken import resolving and as a result this loop never ended (and error was never caught and logged).
Will try to find some time to provide minimal test case.

I suggest to change here https://github.com/enb-make/enb-sass/commit/fadbe64a19766c34627676d6c41b8d02d842c4c6#diff-09f26496655e6744ced8766d49bdb26bL72 while (!filename && i > 0) {
but there are still cases when var searchString = ': begin */' may be not found in all content, so later https://github.com/enb-make/enb-sass/commit/fadbe64a19766c34627676d6c41b8d02d842c4c6#diff-09f26496655e6744ced8766d49bdb26bL81 filename[1] cause exception


Feature/custom attributes

new option to write custom attributes by usigin the prefix "_"

closes #20


🎨(models) extract invitation converter in a proper method

Improved code readability, by extracting this well-scoped unit of logic in a dedicated method. Also, rename active_invitations to match 'valid' vocabulary used elsewhere in the doc. If no valid invitation exists, early return to avoid nesting.
Also, I wonder @mjeammet should we add try/except to catch any database error, log them but ignore them when creating a new identity?

Improved code readability, by extracting this well-scoped unit of logic in a dedicated method. Also, rename active_invitations to match 'valid' vocabulary used elsewhere in the doc. If no valid invitation exists, early return to avoid nesting.

Absolute +1, both for renaming the invitations and the dedicated private method.

Also, I wonder @mjeammet should we add try/except to catch any database error, log them but ignore them when creating a new identity?

Good idea !


bump-formular-pr --version fails with seemingly equal versions

brew config output
HOMEBREW_VERSION: 3.6.6
ORIGIN: https://github.com/Homebrew/brew
HEAD: 978a452ea7ccdff1ed7377f88add3ff0513ba2b7
Last commit: 5 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: af0056b4243b00dff46f291d0db35180a4729508
Core tap last commit: 3 days ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: ["--no-quarantine"]
HOMEBREW_CLEANUP_PERIODIC_FULL_DAYS: 3650
HOMEBREW_COLOR: set
HOMEBREW_GITHUB_API_TOKEN: set
HOMEBREW_MAKE_JOBS: 3
HOMEBREW_NO_AUTO_UPDATE: set
Homebrew Ruby: 2.6.8 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.8_1/bin/ruby
CPU: 3-core 64-bit ivybridge
Clang: 13.0.0 build 1300
Git: 2.38.1 => /usr/local/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 11.7-x86_64
CLT: <IP_ADDRESS>.1.1638488800
Xcode: 13.2.1 => /Applications/Xcode_13.2.1.app/Contents/Developer

brew doctor output
Your system is ready to brew.

Verification

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
Trying to execute bump-formula-pr with  --version flag to create a pull request for an updated formula.
What happened (include all command output)?
This command didn't work since the release of this line:
https://github.com/Homebrew/brew/blame/master/Library/Homebrew/dev-cmd/bump-formula-pr.rb#L276
Output:
Error: You need to bump this formula manually since the new version
and old version are both 0.11.4.

What did you expect to happen?
Create a PR with the updated content (version 0.12.0).
Step-by-step reproduction instructions (by running brew commands)
brew tap gefyrahq/gefyra
==> Tapping gefyrahq/gefyra
Cloning into '/usr/local/Homebrew/Library/Taps/gefyrahq/homebrew-gefyra'...
Tapped 1 formula (12 files, 40.0KB).
Warning: bump-formula-pr is a developer command, so
Homebrew's developer mode has been automatically turned on.
To turn developer mode off, run brew developer off

brew bump-formula-pr --version=0.12.0 gefyra

==> Installing 'bundler' gem
Fetching gem metadata from https://rubygems.org/.......
Fetching concurrent-ruby 1.1.10
Fetching minitest 5.16.3
Fetching zeitwerk 2.6.1
Fetching public_suffix 5.0.0
Installing zeitwerk 2.6.1
Installing minitest 5.16.3
Installing public_suffix 5.0.0
Installing concurrent-ruby 1.1.10
Fetching ast 2.4.2
Installing ast 2.4.2
Fetching bindata 2.4.12
Fetching msgpack 1.6.0
Installing bindata 2.4.12
Installing msgpack 1.6.0 with native extensions
Using bundler 2.3.23
Fetching byebug 11.1.3
Installing byebug 11.1.3 with native extensions
Fetching connection_pool 2.3.0
Installing connection_pool 2.3.0
Fetching did_you_mean 1.6.1
Installing did_you_mean 1.6.1
Fetching diff-lcs 1.5.0
Installing diff-lcs 1.5.0
Fetching docile 1.4.0
Installing docile 1.4.0
Fetching unf_ext <IP_ADDRESS>
Installing unf_ext <IP_ADDRESS> with native extensions
Fetching regexp_parser 2.6.0
Installing regexp_parser 2.6.0
Fetching hana 1.3.7
Installing hana 1.3.7
Fetching hpricot 0.8.6
Installing hpricot 0.8.6 with native extensions
Fetching simplecov-cobertura 2.1.0
Installing simplecov-cobertura 2.1.0
Fetching http-cookie 1.0.5
Installing http-cookie 1.0.5
Fetching rspec-wait 0.0.9
Installing rspec-wait 0.0.9
Fetching mechanize 2.8.5
Installing mechanize 2.8.5
Fetching ronn 0.7.3
Installing ronn 0.7.3
Fetching rubocop 1.35.1
Installing rubocop 1.35.1
Fetching rubocop-rspec 2.13.2
Fetching rubocop-sorbet 0.6.11
Fetching rubocop-performance 1.15.0
Fetching rubocop-rails 2.16.1
Installing rubocop-sorbet 0.6.11
Installing rubocop-performance 1.15.0
Installing rubocop-rspec 2.13.2
Installing rubocop-rails 2.16.1
Bundle complete! 35 Gemfile dependencies, 77 gems now installed.
Bundled gems are installed into `../../../../../../usr/local/Homebrew/Library/Homebrew/vendor/bundle`
==> Downloading https://github.com/gefyrahq/gefyra/releases/download/0.12.0/gefyra-0.12.0-darwin-universal.zip
==> Downloading from https://objects.githubusercontent.com/github-production-release-asset-2e65be/429510684/ac96d0fb-e502-498e-9f32-2edd927359d9?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20221021%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20221021T140555Z&X-Amz-Expires=300&X-Amz-Signature=255bae04560571ffd2e428d43857cbbd8351520659a0f9ea500d594f3d47a07b&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=429510684&response-content-disposition=attachment%3B%20filename%3Dgefyra-0.12.0-darwin-universal.zip&response-content-type=application%2Foctet-stream
Warning: Cannot verify integrity of 'f6b82ffc6fdae22ab7cbdd90fb766b3fddd76abf3440092b3f432f20efcacd42--gefyra-0.12.0-darwin-universal.zip'.
No checksum was provided for this resource.
For your reference, the checksum is:
  sha256 "a8856133cc86c6ebd41f06c4e9b7508e0a30e600f02916c4a28d0597710a2c14"
==> replace /https:\/\/github\.com\/gefyrahq\/gefyra\/releases\/download\/0\.11\.4\/gefyra\-0\.11\.4\-darwin\-universal\.zip/ with "https://github.com/gefyrahq/gefyra/releases/download/0.12.0/gefyra-0.12.0-darwin-universal.zip"
==> replace "fe26ea81af067d1606b37fab4f8f53518a41fed857416d385c75cea3326114b6" with "a8856133cc86c6ebd41f06c4e9b7508e0a30e600f02916c4a28d0597710a2c14"
Error: You need to bump this formula manually since the new version
and old version are both 0.11.4.
Error: Process completed with exit code 1.

It'll probably work if you do this:
--- gefyra.rb.orig	2022-10-21 23:14:04.085598395 +0800
+++ gefyra.rb	2022-10-21 23:13:27.389323092 +0800
@@ -3,7 +3,6 @@
   homepage "https://gefyra.dev"
   url "https://github.com/gefyrahq/gefyra/releases/download/0.12.0/gefyra-0.12.0-darwin-universal.zip"
   sha256 "a8856133cc86c6ebd41f06c4e9b7508e0a30e600f02916c4a28d0597710a2c14"
-  version "0.12.0"
   license "Apache-2.0"

   def install

Or, if you don't want to do that, also pass --url to brew bump-formula-pr.

Removing version and just working with --url breaks the update capability for existing installations. brew upgrade then claims that there is no new version for gefyra.

I think @carlocab's point is that, after you remove the version line, bump formula-pr --version <new_version> will then automatically rewrite the url line appropriately. That's how the vast majority of the Homebrew core formulae are version-bumped.

Thank you for the workaround.
Just to make sure I got this right - the formula structure of my project is outdated?
Why does the upgrade no work for existing installations after removing the version line?

Why do you say that upgrades don't work for existing installs when you remove the version line?

I am currently maintaining the formula manually since the behaviour of the flag changed in August. Please allow me some time for the next release to check whether it works.
Could you answer my question anyway? Why does this strange behaviour occur when using the --version flag? Did I understand it wrong? Is it a bug? Is the structure of the formula somewhat outdated or deprecated. I just want to understand why the --version flag is causes this issue.

I thought I already explained this -- this is happening because your formula specifies version when it shouldn't. Remove the specification of version and it won't happen.

Okay, I guess I cannot really understand and just have to accept this is how it works.
My intuition was that when passing a newer version through the flag, the version in the formula would be updated as well (this is how it worked before). I just feel like the error message is just kind of misleading:
Error: You need to bump this formula manually since the new version
and old version are both 0.11.4.

When I just passed --version=0.12.0. Just trying to give you my perspective on this. Maybe I am not the only one struggling with strange behaviour :)


Error on submit when removing the last item from a single resource

Pretty sure this is a situation that you fixed yesterday, but the issue is still here for a single resource that holds a list. When you remove the last item from the list and validate, the list is [%{}], instead of []

🚀 Thank you for your contribution! 🚀w


[BUG] This document requires 'TrustedHTML' assignment.

What happened?
Error when use Content Script UI on google play

innerHTML is blocked by CSP

Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
Chrome
Relevant log output
This document requires 'TrustedHTML' assignment.

------

Your site tries to use a plain string in a DOM modification where a Trusted Type is expected. Requiring Trusted Types for DOM modifications helps to prevent cross-site scripting attacks.
To solve this, provide a Trusted Type to all the DOM modifications listed below. You can convert a string into a Trusted Type by:
defining a policy and using its corresponding createHTML, createScript or createScriptURL function.
defining a policy named default which will be automatically called.

(OPTIONAL) Contribution

[ ] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

Merged!
thx

@anuoua I'm cutting a release, will take a bit till it's live!


Adds Subpixel Depth2Space and Space2Depth Functionality.

This adds some new functionality to take stacked outputs from a convolution
and then intersperse them to create an upscaled array (Depth2Space).
It also adds the inverse operation (Space2Depth) for the purpose of building symmetrical
architectures for subpixel autoencoders etc.
Details of the algorithm's underpinnings can be found at https://arxiv.org/abs/1609.05158

Thank you for sending the PR! It seems Depth2Space and Space2Depth are independent, except that they are in the same module. So, is it possible to divide this PR into two smaller PRs so that we can narrow the scope of review? Thank you for your corporation.

Sure, I will submit them as separate PRs. Thanks.

Thanks. Then, I will close this PR when you send the PRs.

Ok added PRs #1748 and #1749 splitting this PR into its component halves.


Create hult.txt

University official website URL:

https://www.hult.edu/

URL of a page on the official website where the university offers a long-term IT-related course:

https://www.hult.edu/mba/one-year-mba/stem-mba/
https://www.hult.edu/masters/business-analytics/
https://www.hult.edu/undergraduate/majors-minors/business-analytics/

Official email domain for the enrolled students:
[first letter of first name][full last<EMAIL_ADDRESS>

@lisa1ynn First stage of verification passed, official domain for Hult International Business School is hult.edu as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@philipto I just tried to apply for a renewal through my account, but it still does not seem to work. This is the error: "Your email address does not belong to any university that we know. You can apply with another method (see tabs above) or add your school to JetBrains' list."
Do you know why this might be?
JetBrains Products for Learning.pdf


No module named 'moviepy.editor'

Thanks for your amazing job!
When I follow the instructions and try to finetune I get the following:
python -m src.main +experiment=dl3dv
Traceback (most recent call last):
  File "/home/sebastian.cavada/micromamba/envs/noposplat/lib/python3.10/runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/sebastian.cavada/micromamba/envs/noposplat/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/sebastian.cavada/scsv/thesis_2025/NoPoSplat/src/main.py", line 24, in <module>
    from src.config import load_typed_root_config
  File "/home/sebastian.cavada/.local/lib/python3.10/site-packages/jaxtyping/_import_hook.py", line 230, in exec_module
    return super().exec_module(module)
  File "/home/sebastian.cavada/scsv/thesis_2025/NoPoSplat/src/config.py", line 13, in <module>
    from .model.model_wrapper import OptimizerCfg, TestCfg, TrainCfg
  File "/home/sebastian.cavada/.local/lib/python3.10/site-packages/jaxtyping/_import_hook.py", line 230, in exec_module
    return super().exec_module(module)
  File "/home/sebastian.cavada/scsv/thesis_2025/NoPoSplat/src/model/model_wrapper.py", line 5, in <module>
    import moviepy.editor as mpy
ModuleNotFoundError: No module named 'moviepy. editor'

This error keeps showing up after I conclude the environment installation. I followed everything, but the version utilized is probably not the correct one, as there is no specified version in the requirements.txt file. Did this happen to others?
When I enforce moviepy==1.0.3, the above problem is solved, but the new one appears:
  File "/home/sebastian.cavada/.local/lib/python3.10/site-packages/jaxtyping/_decorator.py", line 483, in wrapped_fn_impl
    out = fn(*args, **kwargs)
  File "/home/sebastian.cavada/scsv/thesis_2025/NoPoSplat/src/model/model_wrapper.py", line 622, in render_video_generic
    clip = mpy.ImageSequenceClip(list(tensor), fps=value._fps)
AttributeError: 'Video' object has no attribute '_fps'

Probably a version mismatch.
Which version of moviepy is being used?
Thanks!

The training started, the second problem was linked to wandb that I was not using. Once I runned the complete command
python -m src.main +experiment=re10k wandb.mode=online wandb.name=re10k
Then everything went on smooth.


Setting a low flushInterval effectively breaks closeAndFlush()

We're using @segment/analytics-node in a serverless environment that is very strict about ending execution after the main entry point's promise resolves.  The hosting node process is killed immediately after the promise resolves, so if there was anything still happening in the background, it is lost.
We followed the guidance found in the readme that suggests setting flushInterval to 1, but found some events were being dropped, particularly if fired shortly before the serverless function finished and resolved.
So we then added await analytics.closeAndFlush() to the end of our function to ensure events were being flushed before exiting.
To our surprise, this did not help; events were still being lost.
I believe analytics-node.ts#L47 is to blame.  The library sets the default timeout for closeAndFlush() to flushInterval * 1.25, meaning our call to closeAndFlush() was quietly resolving after 1.25ms, which is definitely not enough time to establish a TLS connection to Segment's servers and send any events.
Thus, our function resolved its promise before analytics events could hit the wire, and the process was killed before the events could be sent.
This timeout behavior was not documented anywhere, so it was very surprising to find out that the flush call wasn't working; we had no idea we'd need to set a sane timeout ourselves (await analytics.closeAndFlush({ timeout: 10000 })).

The default should probably be something like this._closeAndFlushDefaultTimeout = Math.max(5000, flushInterval * 1.25) so that a low interval doesn't break flushing, and the docs should better describe this behavior.
This is totally my opinion, but the readme's advice to set flushInterval to 1 and await every track call seems…not great.  Isn't it preferable to keep the default interval, use non-blocking track calls, and then closeAndFlush() before returning from your serverless function?

We followed the guidance found in the readme that suggests setting flushInterval to 1, but found some events were being dropped, particularly if fired shortly before the serverless function finished and resolved.

@daguej a lot to unpack here, but the readme suggests that you set flushAt to 1 (which is the max events in a batch), not flushInterval? flushInterval doesn't come into play if the maxEventsInBatch is 1, since the event queue will always be flushed as soon as it receives an event.

Indeed it does!  Looks like we misread that, and accidentally set flushInterval instead of flushAt.  (I double-checked our codebase; it's definitely set to the former.). Oops, sorry for the confusion.
Mistaken property aside, the core of this issue still stands -- setting a low flushInterval causes flushAndClose to effectively not work.


FR: Smooth only selected tracks

Thanks for this great extension. I really love the old-school look of curved tracks (guess which era of circuit boards I grew up in ;) )
I like the way it creates a copy of the PCB so you can re-edit the original straight-line version, but would it be possible to modify the script so that we have an option to smooth just selected tracks?

Thanks. I'm probably not going to add any new features to this now as I've mostly moved onto using EasyEDA Pro which currently doesn't support extensions.

Hey Matt,
Okay, sure, I can understand that.
Perhaps we can pin this for the day the "pro" version support extensions?
Also, what is it about the Pro version that made you switch?
Take care,
Dax.

Hey again, Matt. Quick question: Do you know of any ill side-effects of running Smooth Tracks on a PCB that has already had Smooth Tracks applied?
I assumed it would either confuse or mess up the layout, but I just tried it and it seems fine. (Though I've not yet tried to edit anything so not sure if, for example, a million superfluous points might have been created.)
Thanks again,
Dax.

I'm not really sure tbh.

Yeah, unfortunately due to the way that tracks are modified when smoothing is applied, it meant I needed to rework the smoothed PCB and at some point, I got sick of updating the original PCB layout and just updated the smoothed one.
IIRC, this was caused by issue #12 which you solved.

The teardrops and fillets (the curves between three-way intersections or places where tracks of differing widths meet) are flagged as 'teardrops' so you can use the standard teardrop tool to remove all of those.  I'm not sure if EasyEDA would throw away any additional properties on tracks that it didn't understand but maybe it would be possible to store information somewhere to be able to "unsmooth" tracks.

Well, let me know if you get around to trying something and if you need a hand testing.


rebrand audio -> media

Since we'll be supporting video (#199) we should probably shift from "audio" to "media" in terms of branding and also rename APIs referencing "audio" to use "video" (don't remove the existing ones, but deprecate them).

I'll also make this a ticket to rebrand everything... since we've got a new name etc.

Remember to change CSS classnames!


[Feature]: Quasi-random search based on low-discrepancy sequences

Description
Quasi-random search based on low-discrepancy sequences is a method for exploring a discrete search space where, intuitively speaking, subsequent hyperparameter trials are as different as possible.
This can be thought of as essentially the opposite of grid search, where parameters are varied one at a time, yet also better at exploration than true random sequences, which by chance may produce subsequent trials which are close in the search space. In quasi-random search, parameters are varied as much as possible within sequences of trials. Also, selecting points quasi-randomly using aforementioned low-discrepancy sequences gives reproducible sequences of trials.
The Deep Learning Tuning Playbook from Google Research lists many advantages of quasi-random search, including reasons to prefer quasi-random to random or grid search, and in some cases, even to Bayesian search. They also lists several reference implementations.
Suggested Solution
The simplest API would probably just be another search option, e.g. method: quasirandom, though of course this is up to you.

Hi Jason, no I don't think there is further information to add. The referenced sections of the Deep Learning Tuning Playbook describe the workflow benefits as well as other pros and cons better than I can.


Resolve UI issue where footer overlapped data table navigation at the…

… bottom of models pages (#28)
Changes Proposed in This PR
This PR fixes a style issue where the footer overlapped with the nav controls for data tables on model pages

closes #28


DPL-282 - ONT - create or edit a Sequencing Run

User story
A Sequencing Run will be made up of one or more libraries or pools which will be assigned to one or more flowcells on either a GridIon or Promethion
Who are the primary contacts for this story
Alex / Emma
Acceptance criteria
To be considered successful the solution must allow:

[x] Page which is similar in structure to the Pacbio run creation page
[ ] Selection of valid pools
[x] Addition of pools onto flowcells
[x] Addition of top level run data e.g. experiment name and instument name
[x] Addition of flowcell id to flowcell
[ ] Adequate recovery from errors e.g. moving pools to other flowcells

References
This story has a relationship with sanger/traction-service#713
Additional context
Minion - 1 flowcell maximum
GridIon - 5 flowcells maximum
Promethion - 24 flowcells maximum (48 positions)

[ ] Create a page to view the sequencing runs - similar to style to Pacbio with run id, run name, state, instrument name and created at (subject to confirmation with Long Read)
[ ] Create a sequencing run view
[ ] Create a component for run information which includes selecting instrument and state. Experiment name is created automatically
[ ] Create a pool search component to allow users to select pools. The identifier will be the pool barcode.
[ ] Create a component to view the currently selected pools
[ ] Create a component to manage flowcells - linked to the maximum number of flowcells?
[ ] Create a component for each flowcell
[ ] Button to create sequencing run

Vuex store for managing state:

[ ] Runs resource
[ ] Instruments resource
[ ] currentRun object (see ONT pools) which would include flowcells, pools and run metadata
[ ] actions - fetchInstruments on page load, fetchPoolByBarcode, createRun, updateRun
[ ] state - currentRun, runs, instrument
[ ] mutations
[ ] getters

(updated run wireframe)

Questions for UAT test

Q: Usage: How/ When will the run be created in relation to the physical creation (upfront/ pending/ plan/ on reflection).
A: xx
Instrument: Is there a default instrument to be selected? Currently we are restricting the abilty to change the instrument type after a run has been created, is this ok?
A: xx
State: Is there a default state to be selected on create? Can the state be changed from and to anything? Should updates be allowed for all states?
A: xx
Flowcell ID: Does the flowcell ID need to be validated in any way? Can it be reused? (between a run, or any previous runs?) Does it always have the same prefix or format that can be validated?
A: xx
Pool Barcode: Do you want to see any pool metadata on the flowcell? (Could add a link to Pools view?) Or is the Pools side bar list helpful.
A: xx
Flowcell: Do they want a clear/reset button from Flowcell ID or Pool Barcode inputs?
A: xx
Run: Do they want a Reset button for the whole run (could just navigate away from page)
A: xx
Run: Might you ever need to create a Draft run
A: xx
Run: Validation, should you be allowed to create a run with no flowcell data
A: xx
Runs: pagination/ sort by?
A: xx
ONTFlowcells: hard to use PromethION, scrolling?
A: xx


addonlist.txt being loaded

I'm not sure if this is intended behaviour.
I would like for my currently set addonlist.txt to be read, to have a starting point when starting to use Funky. Now, after first use, I seem to need to manually activate all the mods I have activated in-game in Funky to achieve parity.
This might be related to #6 though.

#6 is its own separate issue entirely, but this sounds like a nice feature to implement for the onboarding phase.

I'm trying to work out this issue and PR it, I'm currently having issues trying to load the addonlist.txt because while I was reading the source code, I noticed this project was first intended to be a general mod manager rather than a L4D2 one.
Because of this I get caught up with what should I do and has me thinking if the code I write should be as general purpose as that first view of the project and make a game specific workaround section code (Gmod doesn't have a addonlist.txt for example), or code it as this were a L4D2 mod manager first and foremost.
Also something I noticed, is that supporting the addonlist.txt would make it easier to change mod loading order, which is a thing that steam properly supports.

For future reference, it would make it easier, since the addonlist.txt order is defined by what's shown in the steam interface, which then applies that to the addonlist.txt, more details on how load order is managed on the following steam community post

Yup, funky was first designed to work for many source games (with their various differences in terms of mod loading, as you've mentioned) but at some point I just wanted to focus on left 4 dead. The systems still sorta stayed, and I even have full L4D1 support implemented, but I wanted to focus on L4D2 specifically as I knew it best and could focus on it.
I have a "vdf" branch on-going that focuses on reading and writing KeyValue files, such as addonlist.txt, but I haven't been able to work much on it as of late.
For load order support, I'll have to first consider how shuffles and "bundles" (combining multiple mods into one item within Funky) will affect it, so I'd rather focus on bundles first, as it's a more requested feature, and go on from there.
Importing your current addonlist.txt is a nice feature thought, I'd love to add that, I just have to get around to doing it.
Let me know about any future progress / feedback. I appreciate the effort.

I was able to read the addonlist.txt and implement it on a sane-ish way, following your window.api code, I should have a prototype  that enables/disables addons reading the list, tomorrow or soon :tm:, it's a bit messy since I have never used TS and I'm not very familiar with the codebase yet, but it's there

I was able to update the libraryActiveAddons variable and fill it with addons read from the file, but it seems there's something I'm missing since doing so, instantly makes all enabled tags to dissappear, but the number seems to be updating correctly with the in game addon count changes.
https://github.com/user-attachments/assets/609d1454-36b9-4a89-8b5d-919e294bd5f7
(The video showcases the corner number being updated, and the current issues with enabled labels)
It seems I will take a bit more of time, since I'll have to read the database stuff that's going on when clicking stuff and enabled labels, my progress is over this fork on the main branch, I'm also open to code style changes and other comments on how should I improve my code.

Sweet! Unfortunately I won't have time until Saturday to check this stuff out, but big thanks for the work. I'll let you know my thoughts ASAP.
Do let me know how you found navigating the source code... The project was quite rushed in some ways and if you have any tips on making it easier to read, feel free to let me know.

I finally finished it.
Finished product video for those who are interested.
https://github.com/user-attachments/assets/3a301cf6-10a1-482f-a381-5c468df76312

That one shitty comment ended up doing some good in the end lol


Element: update calls to use sdf::Errors output

Signed-off-by: Marco A. Gutierrez<EMAIL_ADDRESS>🎉 New feature
Work towards https://github.com/gazebosim/sdformat/issues/820.
Summary
This is an update for the class Element making the calls use sdf::Errors whenever possible to avoid unwanted console outputs.
Test it
Using any method of the Element class with the errors parameter should not print any error.
Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if the library has them)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Added tests for Get and updated the Set existing ones in https://github.com/gazebosim/sdformat/pull/1141/commits/4c83cdc9aa24303c0170c1b8eb5e294f1bf6bf9a.
The rest of functions changed here seem to only fail when the Element is malformed which I find tricky to create without accessing the private attributes of the class. I modified some of the current tests to make sure they don't return errors nor print any. I guess when the switch is done this should be done for all functions.

The rest of functions changed here seem to only fail when the Element is malformed which I find tricky to create without accessing the private attributes of the class. I modified some of the current tests to make sure they don't return errors nor print any. I guess when the switch is done this should be done for all functions.

Yes, some of the functions are very unlikely to emit an error.

The rest of functions changed here seem to only fail when the Element is malformed which I find tricky to create without accessing the private attributes of the class. I modified some of the current tests to make sure they don't return errors nor print any. I guess when the switch is done this should be done for all functions.

Yes, some of the functions are very unlikely to emit an error, but we might be able to induce some errors by providing ill formed description files. I'll try to look into this.

@marcoag I've pushed some changes to azeey/test_element_error_propagation. Namely, I added some tests that check the propagation sdf::Errors in some of the overloads added in this PR.
I was unable to find a way to cause CountNamedElements to emit an error, but in the process of attempting to do that, I realized that by removing SDF_ASSERT calls, we've made it possible for users to end up with a segfault instead of an exception. This is because as opposed to throwing an exception when there is a critical problem, we now insert a fatal error into an sdf::Errors object  and continue executing the function. We should instead be checking if a fatal error occurred and returning early.  So in the mentioned branch, I have also added a convenience function to check if there are fatal errors after calling a function
https://github.com/gazebosim/sdformat/blob/6af4be6b2dbd13271ef88b2088ac91cea695dfa6/src/Element.cc#L1243-L1246
Would you be able to look through Element.cc and probably also Param.cc and add similar logic to places where returning early is necessary?

Per video call, it would be best to revert all the changes we made code that calls SDF_ASSERT so that exceptions don't cause segfaults. If we remove the console printing in sdf::Exception (https://github.com/gazebosim/sdformat/blob/6af4be6b2dbd13271ef88b2088ac91cea695dfa6/src/Exception.cc#L52), exceptions could be handled by the end user with no errors printed to the console. I think the original goal of #820 was to remove console messages, not eliminating exceptions, but I think we started removing exceptions as well maybe hoping that it would be fairly straightforward. We should still try to remove exceptions from libsdformat, but we should leave that for another time. /cc @scpeters.

Can you resolve the conflicts?


add a schema for querying academic scores

schema "academic" can not run separately now, I used a warning and arbitaryly set the separate as FALSE when you set
schema = "academic" and separate = TRUE.

@Blockhead-yj Try using the latest unit test framework. Before your work, make sure this branch is rebased to develop branch. I think you'd better use git flow.

New logic is applied now. This PR is outdated.

Close because it is outdated.


Add rule to check the markup validity

Fix #28

@qzhou1607 there are some conflicts. Can you please solve them?

@molant In this PR, I use the html-validator dependency, which uses the web service interface of the validator ( I doubled checked the web service and didn't see a limit of requests anywhere.). An alternative is to use the vnu-jar, but using java with node seems complicated according to @alrra . Do you have any opinion on that which one to use?

@qzhou1607 if you are using https://www.npmjs.com/package/html-validator it had at least an option to ignore in the configuration object. That property should be exposed somehow.

@molant In html-validator, currently the setting of ignore is only supported if the output format is set to text. We have to use json here for further parsing. It was not difficult to implement ignore in our code though, so I added that.

@qzhou1607 if you are using https://www.npmjs.com/package/html-validator it had at least an option to ignore in the configuration object. That property should be exposed somehow.

I added documentation also.

I don't see a page with the description of the rule. Remember to use the new-rule parameter you added to dogfood it and adjust if necessary!

I had issues with implementing async iterator. It looks like I need to convert iterables (array) into async iterables first using a generator, and then do the for..await..of loop. But for some reason, the rule can't be exported right whenever I drop the generator in the create function. Am I looking at the wrong direction?

Can we add support for async iterator? They should be working on typescript 2.3 (check the release notes, there might be some hack to do to make it work everywhere).

@molant @sarvaje @alrra I looked into async iterator more, still having trouble with async generators. The async generator function in typescript was directly copied when compiled and throws an error when running in Node. The lib setting I have in tsconfig.json is
        "lib": [
            "es2017",
            "esnext",
            "esnext.asynciterable",
            "dom",
            "dom.iterable"
        ]

I also tried polyfill Symbol.asyncIterator according to this post using core-js, still no luck, same error. So I'll keep the current code for now until I have more information about the async generators.

I also tried polyfill Symbol.asyncIterator according to this post using core-js, still no luck, same error. So I'll keep the current code for now until I have more information about the async generators.

Sounds good to me!

@molant Yep working on it. If we give user this option we should probably ask for the output interface (firstColumn, subType and etc.) as well, otherwise the output can't be consumed by context.report and our formatter.

ce (firstColumn, subType and etc.) as well, otherwise the output can't be consumed by context.report and our formatter.

Not sure I understand why we need that. We are only changing the end point to use.

@molant A different service/endpoint may have a different REST interface (like using  column instead of firstColumn as the property name). This inconsistency could raise error when we process the output from different services. Or should we assume that other service endpoints always have the same output format as the default validator? If not, throws an error?

Not sure I understand why we need that. We are only changing the end point to use.

We will be using the same code as the official one. There are some instructions on how to set it up.

@molant validator option was added.

@sarvaje @AustinKelleher this seems to be ready for another review.

Merged with some modifications in 9f73cc9b6071193a12933d756883738a0308a6b6.


Sync eng/common directory with azure-sdk-tools for PR 7842

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/7842  See eng/common workflow

/check-enforcer override


[WIP]feat: angular and svelte style transform

Description
Add a short description of:

what changes you made,
why you made them, and
any other context that you think might be helpful for someone to better understand what is contained in this pull request.

This sort of information is useful for people reviewing the code, as well as anyone from the future trying to understand why changes were made or why a bug started happening.

@decadef20 did you mean to close this right away?

@decadef20 did you mean to close this right away?

@samijaber  It still works in progress. When totally finished, I'll reopen it.


Combine support

This PR adds support for Combine in Core & CoreData.

I've created a feature/combine branch that copies most of these changes, with some differences:

Avoids type erasure using eraseToAnyPublisher(), it's generally discouraged unless really necessary.
Embeds the Combine functionality into the existing subspecs, simplifying use for everyone.


Response security header X-Content-Type-Options is not set

Fortreum recommends adding a  X-Content-Type-Options: nosniff header to all responses.
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Content-Type-Options:

The X-Content-Type-Options response HTTP header is a marker used by the server to indicate that the MIME types advertised in the Content-Type headers should be followed and not be changed. The header allows you to avoid MIME type sniffing by saying that the MIME types are deliberately configured.

This header is currently returned by the HCA Data Browser, but not the AvVIL Data Browser.

Thanks @dsotirho-ucsc will take a look.

@NoopDog This needs to be completed before the audit starts in mid March.

Got it, thx.


scion-pki: support dummy TRC payload

The dummy TRC payload allows to create a TRC payload without much setup. This is useful for testing access to the necessary cryptographic material, especially in preparation of a TRC ceremony.

This change is


feat!: Remove deprecated features for v5.0.0

The next release will be 4.8 so at the moment this PR is not necessary.


[Docs] RxJS is still too big

Bug Report
Current Behavior
Importing Observable from RxJS pulls in 12kb of code after Uglifying and Gzipping, when compiling to a CommonJS module:
import {Observable} from 'rxjs'

webpack -p && cat ./test-minify/bundle.js | gzip | wc -c # 12234

Importing the internal type instead, the size drops to a more manageable <4kb:
import {Observable} from 'rxjs/internal/Observable'

webpack -p && cat ./test-minify/bundle.js | gzip | wc -c # 3671

Reproduction
https://github.com/bcherny/typed-rx-emitter/commit/50a457d296bad3d32608b257b52f294cec3c8ba1#diff-ed009b6b86b017532ef0489c77de5100R2
yarn test
Question
Reading all the RxJS docs, the canonical way to pull in Observable is straight from rxjs. However, unless you're compiling to ES2015 modules, this surprisingly pulls in a lot of unexpected code.
/internal/Observable does not seem to be documented, and it seems wrong to rely on it. As the path name implies, it's an implementation details that might change in the future.
What this means is that those of us that are still using CommonJS for whatever reason see really big bundle sizes.
So - What's the correct path to import Observable from, if you're using CommonJS?

@cartant would you be able to give feedback to this question?

Answer in here for v6 is let webpack or other bundler properly tree shaking it, i.e in given config (https://github.com/bcherny/typed-rx-emitter/commit/50a457d296bad3d32608b257b52f294cec3c8ba1#diff-d901c9ce0f05c56372ba31dd2215b7b7R17) it should be able to specify webpack / ts-loader uses esNext to enable treeshaking via rxjs's esm module, instead of transpiling whole ES2015 modules. ESM doesn't add burden to transpile module under node_modules but enables tree shaking correctly, which aim to solve this specifics.


RPC-R support plus ip argument and glance_api_local_check rewriting

Changes :

glance_api_local_check: utilize glanceclient now instead of requests
auth_ref['auth_token'] reference has been updated to keystone.auth_token to support v2 and v3 token format
auth_ref['project']['id'] reference has been updated to keystone.tenant_id to support v2 and v3 extraction
IP argument now optional on many checks to fall to query the service endpoint rather than individual IP (except cinder)
This adds support for RPC-R which still uses keystone v2 client/token format

Closes-Bug: #789

recheck
another cirrus image issue....

another cirrus image issue....

recheck

recheck

lint fail is being addressed in #813

:+1:

looks good, I think doing another pass eventually to add support for https might make sense eventually.
:+1:


[SD Correction] Vulnerability Search Sites

Provide the location of the issue
Section A.4
What is the correction request for the cPP? Please describe.
Per RfI202220, exploitsearch.net and securiteam.com should be removed.

References removed.

Issues resolved with PR #209


when I try to use stream. I always got this error: com.fasterxml.jackson.databind.exc.MismatchedInputException: Missing required creator property 'usage' (index 3)

https://github.com/Azure/azure-sdk-for-java/blob/bb37892275404e8a3cac3d4e2c639cbc9b18b9f9/sdk/openai/azure-ai-openai/src/samples/java/com/azure/ai/openai/usage/GetChatCompletionsStreamSample.java#LL53C35-L53C35
Caused by: com.fasterxml.jackson.databind.exc.MismatchedInputException: Missing required creator property 'usage' (index 3)
 at [Source: (String)"{"id":"chatcmpl-7ORI3zwlnUA9cgGa4bU1V1I5XoJO0","object":"chat.completion.chunk","created":1686059131,"model":"gpt-35-turbo","choices":[{"delta":{"role":"assistant"},"index":0,"finish_reason":null}]}"; line: 1, column: 198] (through reference chain: com.azure.ai.openai.models.ChatCompletions["usage"])
	at com.fasterxml.jackson.databind.exc.MismatchedInputException.from(MismatchedInputException.java:59)
	at com.fasterxml.jackson.databind.DeserializationContext.reportInputMismatch(DeserializationContext.java:1755)
	at com.fasterxml.jackson.databind.deser.impl.PropertyValueBuffer._findMissing(PropertyValueBuffer.java:192)

@ZieglerHan
This PR should already fixed the non-azure openai client. And it is already merged into the main branch. https://github.com/Azure/azure-sdk-for-java/pull/35336
You can try to build OpenAI SDK locally and use it.

@ZieglerHan
This PR should fix the non-azure OpenAI client. And it is already merged into the main branch. #35336
You can try to build OpenAI SDK locally and use it.

Thank you！

When will this fix be available via Maven?

When will this fix be available via Maven?

Hi @zizmax . We will release beta.2 which has the fix today, or not late than tomorrow.
Will update the release status here.

https://repo1.maven.org/maven2/com/azure/azure-ai-openai/1.0.0-beta.2/
@zizmax Version 1.0.0-beta.2 released. Can you give a try and see if it solves your issue?

Streaming works now, thank you!


bind a field containing value using ref, it is no work

Vue version
3.3.4
Link to minimal reproduction
https://play.vuejs.org/#eNp9UU1rg0AQ/SvTPaUQDU1uqQptyaGFftDmuBfRiTHRXdkPGxD/e2fXxKY0BDzIm/fevDfbsYemCVuLbMkig3VTpQYTLgCimyAAWG9LDfTJPQTBgOdlm3QdZNIKcxe2aWUR+j6aOXwUnnSolFQXpfN/0mh2FiDSmSobA1UqipgzozkDjcY2NCvrRioDHSjcTEGKV+eHOfSwUbIGzqgPZ/dccJFJoc0xK8ROMOnAL14Sr5AyJ9/+lrhnzPmR6WAuRv/J5BbiBDpX5U+FmKKMnr6xMyWxN45mQ5WETakGrdmURbjTUtDJvRdnmaybskL13piSYnC2HLa4WVpV8vvFY0ZZnJ7wbIvZ/gK+0weHcfahUKNq6RLjzKSqQDOMV19veKD/cVjL3FbEvjL8RC0r6zIOtEcrcop9xvNpn/37lKJY69XBoNCnUi6oY/ae79/p6Ur137iLcOF1dFLW/wD34OO/
Steps to reproduce

What is expected?
no error
What is actually happening?
bind a field containing value using ref, it is no work
System Info
TypeError: Cannot read properties of undefined (reading 'value')

Any additional comments?
null

ou might be missing that refs are unwrapped in the template?
count2' value is unitially undefined, bzr you try to read a property on it. That will necessarily lead to an error.


Allow logging in by pressing Enter key after typing username and password

Fixes #169
Describe your changes

Enter key does the same thing as the Login-button
Disable the Login-button if no username is given and/or password is under the minimum length of 6
Disable the login calls to API with the same logic

Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added thorough tests

Codecov Report

Merging #170 (8916bf7) into dev (db00e4a) will increase coverage by 13.11%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##              dev     #170       +/-   ##
===========================================
+ Coverage   46.54%   59.66%   +13.11%
===========================================
  Files          74       46       -28
  Lines        3246     2375      -871
  Branches      446      194      -252
===========================================
- Hits         1511     1417       -94
+ Misses       1701      926      -775
+ Partials       34       32        -2

Flag
Coverage Δ

backend
59.66% <ø> (ø)

dev
59.66% <ø> (?)

frontend
?

production
?

staging
?

unit
59.66% <ø> (+13.11%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

frontend/src/components/Login.tsx

frontend/src/services/pollService.ts

frontend/src/components/AdminViewPoll.tsx

frontend/src/components/Sidebar.tsx

frontend/src/components/BasicSnackbar.tsx

frontend/src/components/PollCreation.tsx

frontend/src/services/userService.ts

frontend/src/components/Registration.tsx

frontend/src/react-app/reportWebVitals.ts

frontend/src/components/App.tsx

... and 18 more

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


Bot quits with error code 1 on any command

Describe the bug
When I issue any command (/play, /summon, etc.), the bot generates an error in the log and quits.
To Reproduce
Steps to reproduce the behavior:

Install bot
Deploy Docker container
Use slash command in Discord
Bot quits

Expected behavior
I expect container to stay running and music to play
Additional context
[Nest] 28  - 11/02/2024, 9:56:59 PM    WARN [DiscordVoiceService] Current resource is is not playable. This means playback will get stuck. Please report this issue.
node:events:495
      throw er; // Unhandled 'error' event
      ^
Error: Unexpected server response: 404
    at ClientRequest.<anonymous> (/app/node_modules/ws/lib/websocket.js:912:7)
    at ClientRequest.emit (node:events:517:28)
    at HTTPParser.parserOnIncomingClient (node:_http_client:700:27)
    at HTTPParser.parserOnHeadersComplete (node:_http_common:119:17)
    at TLSSocket.socketOnData (node:_http_client:541:22)
    at TLSSocket.emit (node:events:517:28)
    at addChunk (node:internal/streams/readable:368:12)
    at readableAddChunk (node:internal/streams/readable:341:9)
    at Readable.push (node:internal/streams/readable:278:10)
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:190:23)
Emitted 'error' event on WebSocket instance at:
    at emitErrorAndClose (/app/node_modules/ws/lib/websocket.js:1033:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
Node.js v18.20.3
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Hi,

please enable debug mode by setting env DEBUG to true
please post your entire log
please post your environment variables with password and tokens removed. Censor your Jellyfin domain.

Thank you

Log:
yarn run v1.22.19 $ node dist/main [Nest] 28  - 11/03/2024, 3:10:57 PM     LOG [NestFactory] Starting Nest application...  Use emitter.setMaxListeners() to increase limit (Use node --trace-warnings ... to show where the warning was created) [Nest] 28  - 11/03/2024, 3:10:58 PM     LOG [NestApplication] Nest application successfully started +496ms
docker-compose:
version: '3.3' services:   manuel-rw:     ports:       - '3350:3000'     environment:       - DISCORD_CLIENT_TOKEN=VERYSECRET       - JELLYFIN_SERVER_ADDRESS=https://something.something.something       - JELLYFIN_AUTHENTICATION_USERNAME=ALSOSECRET       - JELLYFIN_AUTHENTICATION_PASSWORD=BIGSECRET       - DEBUG:true     image: 'ghcr.io/manuel-rw/jellyfin-discord-music-bot:latest'

Replace DEBUG:true with DEBUG=true. Also, no error in the log. Please reproduce the error and then send the log

Edited in the previous post.

Can you run /help? If yes, can you run /status?

/help works, /status throws an error.
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [NestFactory] Starting Nest application...
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] AppModule dependencies initialized +30ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] DiscordModule dependencies initialized +1ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] DiscoveryModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] PlaybackModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] ConfigHostModule dependencies initialized +1ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] ServeStaticModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] TerminusModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] ScheduleModule dependencies initialized +37ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] EventEmitterModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] ConfigModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] DiscordHostModule dependencies initialized +2ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] UpdatesModule dependencies initialized +1ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] JellyfinClientModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] DiscordModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] DiscordModule dependencies initialized +1ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] HealthModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] DiscordClientModule dependencies initialized +1ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [InstanceLoader] CommandModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [RoutesResolver] HealthController {/health}: +17ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [RouterExplorer] Mapped {/health, GET} route +2ms
[Nest] 28  - 11/03/2024, 3:16:34 PM     LOG [PlayItemCommand] Subscribe to event(on): interactionCreate
(node:28) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 interactionCreate listeners added to [Client]. Use emitter.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)
[Nest] 28  - 11/03/2024, 3:16:35 PM     LOG [NestApplication] Nest application successfully started +559ms
[Nest] 28  - 11/03/2024, 3:17:16 PM     LOG [RegisterCommandService] All global commands are registered!
node:events:495
      throw er; // Unhandled 'error' event
      ^
Error: Unexpected server response: 404
    at ClientRequest.<anonymous> (/app/node_modules/ws/lib/websocket.js:912:7)
    at ClientRequest.emit (node:events:517:28)
    at HTTPParser.parserOnIncomingClient (node:_http_client:700:27)
    at HTTPParser.parserOnHeadersComplete (node:_http_common:119:17)
    at TLSSocket.socketOnData (node:_http_client:541:22)
    at TLSSocket.emit (node:events:517:28)
    at addChunk (node:internal/streams/readable:368:12)
    at readableAddChunk (node:internal/streams/readable:341:9)
    at Readable.push (node:internal/streams/readable:278:10)
    at TLSWrap.onStreamRead (node:internal/stream_base_commons:190:23)
Emitted 'error' event on WebSocket instance at:
    at emitErrorAndClose (/app/node_modules/ws/lib/websocket.js:1033:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
Node.js v18.20.3
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
yarn run v1.22.19
$ node dist/main
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [NestFactory] Starting Nest application...
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] AppModule dependencies initialized +20ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] DiscordModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] DiscoveryModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] PlaybackModule dependencies initialized +1ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] ConfigHostModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] ServeStaticModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] TerminusModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] ScheduleModule dependencies initialized +21ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] EventEmitterModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] ConfigModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] DiscordHostModule dependencies initialized +2ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] UpdatesModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] JellyfinClientModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] DiscordModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] DiscordModule dependencies initialized +1ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] HealthModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] DiscordClientModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [InstanceLoader] CommandModule dependencies initialized +0ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [RoutesResolver] HealthController {/health}: +11ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [RouterExplorer] Mapped {/health, GET} route +3ms
[Nest] 28  - 11/03/2024, 3:23:41 PM     LOG [PlayItemCommand] Subscribe to event(on): interactionCreate
(node:28) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 interactionCreate listeners added to [Client]. Use emitter.setMaxListeners() to increase limit
(Use `node --trace-warnings ...` to show where the warning was created)
[Nest] 28  - 11/03/2024, 3:23:42 PM     LOG [NestApplication] Nest application successfully started +616ms
[Nest] 28  - 11/03/2024, 3:24:12 PM   ERROR [ExceptionsHandler] Received one or more errors
Error: Received one or more errors
    at _ArrayValidator.handle (/app/node_modules/@sapphire/shapeshift/dist/cjs/index.cjs:1205:70)
    at _ArrayValidator.parse (/app/node_modules/@sapphire/shapeshift/dist/cjs/index.cjs:939:90)
    at EmbedBuilder.addFields (/app/node_modules/@discordjs/builders/dist/index.js:228:31)
    at mixin (/app/dist/commands/status.command.js:69:45)
    at DiscordMessageService.buildMessage (/app/dist/clients/discord/discord.message.service.js:43:24)
    at StatusCommand.handler (/app/dist/commands/status.command.js:65:44)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async target (/app/node_modules/@nestjs/core/helpers/external-context-creator.js:74:28)
    at async /app/node_modules/@nestjs/core/helpers/external-proxy.js:9:24
    at async Client.<anonymous> (/app/node_modules/@discord-nestjs/core/dist/explorers/command/command.explorer.js:66:37)
[Nest] 28  - 11/03/2024, 3:24:24 PM     LOG [RegisterCommandService] All global commands are registered!
[Nest] 28  - 11/03/2024, 3:24:48 PM   ERROR [ExceptionsHandler] Received one or more errors
Error: Received one or more errors
    at _ArrayValidator.handle (/app/node_modules/@sapphire/shapeshift/dist/cjs/index.cjs:1205:70)
    at _ArrayValidator.parse (/app/node_modules/@sapphire/shapeshift/dist/cjs/index.cjs:939:90)
    at EmbedBuilder.addFields (/app/node_modules/@discordjs/builders/dist/index.js:228:31)
    at mixin (/app/dist/commands/status.command.js:69:45)
    at DiscordMessageService.buildMessage (/app/dist/clients/discord/discord.message.service.js:43:24)
    at StatusCommand.handler (/app/dist/commands/status.command.js:65:44)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async target (/app/node_modules/@nestjs/core/helpers/external-context-creator.js:74:28)
    at async /app/node_modules/@nestjs/core/helpers/external-proxy.js:9:24
    at async Client.<anonymous> (/app/node_modules/@discord-nestjs/core/dist/explorers/command/command.explorer.js:66:37)

Do you have anything after your Jellyfin URL, like a slash /?
Your Jellyfin URL is probably wrong

I do not. https://something.something.xyz

Can you open a shell inside the container and ping / request your Jellyfin?
https://stackoverflow.com/questions/30172605/how-do-i-get-into-a-docker-containers-shell

I believe I'm in - how do I ping my JF server?

I believe I'm in - how do I ping my JF server?

Using ping, wget and curl

It worked. When I did ping, it only worked with no http:// or https:// - in the environment variable, I have https:// as part of the server address.

It worked. When I did ping, it only worked with no http:// or https:// - in the environment variable, I have https:// as part of the server address.

wget: bad address 'web' when using both IP address and also server URL.

What wget did you execute? Example: wget https://my-jellyfin.com

/app # wget jellyfin.my.domain
Connecting to jellyfin.my.domain (XXX.XX.XXX.XX:80)
Connecting to jellyfin.my.domain (XXX.XX.XXX.XX:443)
wget: bad address 'web'
/app # wget <IP_ADDRESS>:8096
Connecting to <IP_ADDRESS>:8096 (<IP_ADDRESS>:8096)
wget: bad address 'web'

You need the protocol. eg. https://jellyfin.com

root@openmediavault:/# docker exec -it discordmusicbot-manuel-rw-1 sh
/app # wget https://jellyfin.my.domain
Connecting to jellyfin.my.domain (XXX.XX.XXX.XX:443)
wget: bad address 'web'

Your DNS setup is most likely not working: https://stackoverflow.com/a/78929430
See https://docs.docker.com/engine/network/ for more info

Hmm. Okay - this might take me a second...

Is this correct?
The error message you're seeing indicates that a WebSocket connection attempt is resulting in a 404 response, which means that the requested resource could not be found. Here are some steps to help you troubleshoot and fix the issue:


AND / OR  filter selector

It would be interesting to be able to select whether the filters are AND or OR with a button in the filter creation window.

Groups or different views would be awesome! So it would be possible to sqitch quickly between a table with all people from city_a to city_b. And easily activate and change filters,
Like in Notion.

First steps to obtain AND/OR groups

Task as requirements:

[ ] Delete animate filters
[ ] Edit condition for any lvl
[ ] Edit field key for any lvl
[ ] Edit operator for any lvl
[ ] Edit value for any lvl
[ ] Visually distinguish between filter levels
[ ] Option to disable a filter without remove it


nbdev_test fails to run tests when preceded by langchain.embeddings.HuggingFaceEmbeddings

Importing and instantiating langchain.embeddings.HuggingFaceEmbeddings causes nbdev_test to malfunction and not execute subsequent tests.
Steps to Reproduce

In a new repo folder (using nbdev_new) or existing nbdev repo, add the following in a notebook cell (e.g., in 00_core.ipynb):

from langchain.embeddings import HuggingFaceEmbeddings
emb = HuggingFaceEmbeddings()
assert 1==2, 'never executed by nbdev_test'

Run the test: nbdev_test --do_print --file_re core.ipynb.
The nbdev_test returns Success when it should be failing.

Interestingly, it only happens with langchain.embeddings.HuggingFaceEmbeddings, not when using sentence_transformers directly.

Closed as the problem goes away when re-installing from scratch.


Notify LS client of upstream module modifications

When we change a Lean file in the editor, it would be nice to leave a note on dependent open files for the user to refresh the file in order to make the upstream changes visible.
Rough design idea: file workers should send their import closure to the watchdog so it has enough information to notify the client on file edits, and optionally on changes to closed files via a .lean file watcher. The notification could be done as a warning/info message in each affected file, which the client is free to filter out and replace with a more interactive notification (however, having different message lists in the worker and watchdog might spell trouble).

Lean could also simply enable/send over a code action to do this whenever it's detected that this is necessary, right?

I don't think so, code actions are pull, not push


Edit name of category, it's description and order of articles in an index in interface

Should also include deletion of folders and files.