fix(skeleton): Moved skeleton animation to skeleton component import to avoid duplicate css

Resolves #985
Changed
Moved skeleton animation to skeleton component import
Testing / Reviewing
Make sure skeleton states still work

Oh good catch, I didn't think about that. I'd rather not push a breaking change, can we update this to keep the code inside helper-mixins but only have it once in the compiled css.

I see thank you for your thoughts @alisonjoseph! How about using import-once (@include exports) while we keep the content in _helper-mixins.scss?


Terminate Beyla if we can't open a configured Prometheus port

Beyla can fail to register the HTTP Prometheus metrics endpoint because of number of reasons. It can be that the port is already opened or we don't have permission to listen on a specific port.
Currently, when we fail to open the Prometheus port the Beyla process prints an error message, but then successfully runs. Successful run can be misleading, since for the end users there is no indication that the process is unhealthy and the error message is buried in the very first few lines of the logs. The symptom is metrics are not scraped, but Beyla is up and running.
This PR causes Beyla to terminate as soon as we fail to register the port. We can't add early validation for this error scenario, because we can't predict all of the possible reasons we may not be able to open the Prometheus port.

Codecov Report
Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 37.08%. Comparing base (a3ffb2f) to head (2315251).
Report is 1 commits behind head on main.

Files
Patch %
Lines

pkg/internal/connector/prommgr.go
0.00%
3 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (a3ffb2f) and HEAD (2315251). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (a3ffb2f)
HEAD (2315251)

unittests
1
0

k8s-integration-test
1
0

integration-test
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1111       +/-   ##
===========================================
- Coverage   82.03%   37.08%   -44.96%
===========================================
  Files         140      139        -1
  Lines       11514    11506        -8
===========================================
- Hits         9446     4267     -5179
- Misses       1548     6809     +5261
+ Partials      520      430       -90

Flag
Coverage Δ

integration-test
?

k8s-integration-test
?

oats-test
37.08% <0.00%> (+0.01%)
:arrow_up:

unittests
?

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

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


Android ARM64 is completely broken (v4.0.2). Only ARMv7 works. (Unity + IL2CPP + Oculus Quest 2)

I have integrated Steam Audio 4.0.2 in my Unity project by installing the "SteamAudio.unitypackage". I am targeting Android (Oculus Quest 2) with the IL2CPP scripting backend.
By default, the target architecture of my project was initially set to ARMv7, and builds with this configuration ran just fine on the Oculus Quest 2.
However, I just changed the target architecture to ARM64 (recommended by Oculus due to performance) and this causes Steam Audio to completely stop functioning in Android builds.  What can be done to fix Steam Audio for ARM64?
See images below:

(Logs from the Oculus Quest were taken via Android Logcat in Unity)

Similar issue, but the project fails to build with IL2CPP and ARM64. SteamAudio 4.0.2, Unity 2020.3.20, with all XR-related libraries updated to their newest versions.

We are able to reproduce this issue. A fix will be included in an upcoming patch release.
@vanGeck We are, however, unable to reproduce the build failure with IL2CPP and ARM64. Could you provide build logs from the Unity console with the error messages? Thanks.

@lakulish Thank you for the update and for confirming the issue! I am very pleased to hear that the issue will be fixed.
Do you have any estimate (even a rough guess) as to when the patch will be released? Public launch for my VR app is on November 18th (10 days from now). If the patch will not be released by then, would it be possible to directly receive a fixed version ahead of time? I would really appreciate any help, and if so, my email can be found in my profile.
Thank you!

@ForemanDesigns We're aiming to release the patch early next week. Meanwhile, I've uploaded the latest internal build of the Steam Audio SDK to Dropbox at: https://www.dropbox.com/sh/6prxtdgqqt4kqfh/AACrenFM1fSbvEO7DtoiC9vGa?dl=0. It's not fully tested, but should be close to what we eventually ship.
You'll have to unzip steamaudio.zip, and copy the file steamaudio/lib/android-armv8/libphonon.so to Assets/Plugins/SteamAudio/Binaries/Android/arm64 in your Unity project.
Let us know if this build fixes the issue you're encountering. Thanks!

@lakulish Thank you very much! I will let you know once I update it.

This issue should be fixed in the latest patch release: https://github.com/ValveSoftware/steam-audio/releases/tag/v4.0.3. Let us know if this issue persists, we're keeping this issue open pending confirmation.

Heyo, just wanted to say the  libphonon file swap. Did the trick for me was able to load test without issues. Thanks for sharing the release file early made my week at work.

@lakulish Thank you for the update and for the release! I will implement it ASAP and let you know.
Also as a side note, I really appreciate you and the Valve team improving and maintaining this plugin! It doesn't go unnoticed! In my opinion it is superior to other spatializers I've tried, and I'm so delighted I can use it for mobile VR! So thank you. :)

@lakulish I've confirmed that this has fixed the issue! We are now up and running with ARM64 architecture on the Quest 2 with Steam Audio fully-functional. Thank you! :)


Editor features disabled

Summary of problem or feature request
I have a licensed copy of editor. After following your installation instructions step by step, I was able to get a users datatable to display, but the editor functions do not work. The edit / delete buttons are grayed out, and in the "action" column, there is only a text that displays "users.action" for each row. Is this a license key problem with datatables editor, or is something else going on?
It looks like this:

Code snippet of problem
System details
I am using Laragon on Windows to run:
Apache 2.4.35
PHP 7.4.6
Laravel 7.x
laravel-datatables: 9.x

Make sure you have enabled select plugin. Just chain ->select() on the html builder or use other similar options.

Thanks for getting back to me! I've chained ->select() onto the builder,
and that has enabled the edit button, but when pressed, the edit button
produces this error:
"Uncaught Unable to find row identifier For more information, please refer
to https://datatables.net/tn/14." The reference at datatables.net says it's
because the row lacks a unique identifier. Do you know why this would be
happening?
On Wed, Aug 12, 2020 at 11:35 PM Arjay Angeles<EMAIL_ADDRESS>wrote:

Make sure you have enabled select plugin. Just chain ->select() on the
html builder or use other similar options.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yajra/laravel-datatables/issues/2446#issuecomment-673232170,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFZ3G4H3X5VIZ5FVBQ6FOTTSANNPNANCNFSM4P5HGP7A
.

You need to set the row id as the error suggested.
datatables()->eloquent($query)->setRowId('id')


Add exports parameter for artifact sharing

I don't know if this is the best solution but we can use emptyDir to collect artifacts from main containers. Using this feature, we can use the full features of Argo even with the kubelet executor. I actually wanted this required parameter of outputs but I left it optional to keep the compatibility.
Please tell me if we should make it a required parameter and remove unnecessary executor methods of CopyFile and GetFileContents . Or we can do it in another PR.
I think this feature solves the problem of #1003 and #1021.

super duper +1, our org at IBM wants to leverage containerd and argo but this is blocking us :)

I understand your thought. I think the 1st reason really makes sense to me. I think that's the best way to do it. Or, we can improve kubelet/kubernetes API to use docker cp like command internally. There's a related issue in Kubernetes repo.

There is a technique we have used in the past (argo v1), where we mapped in the executor static binary to the main container, then invoked the user defined command/args of the container, which allowed us to gain control of the post-processing phase while we still had direct access to the main container's filesystem.

I don't know if users want to put the binary in their containers and let Argo do something there because containers should be isolated even if it runs as a part of a workflow system.
So, for the time being, should we throw unsupported-feature exceptions in CopyFile and GetFileContents of the kubelet, and coming k8sapi, executors so users don't be confused with the system error?

By the way, the k8sapi executor solves some of the  problems about artifact sharing because it doesn't use exec to print logs. I'd like to make them throw unsupported feature exceptions accordingly.

@jessesuen I remember we discussed how this feature should be implemented in offline and you had the idea of utilizing volumeMounts of the main container to automatically mount the volumes in the corresponded wait container. If you're too busy to implement it, I will do it as I need it in my projects ASAP.


Cassy SRV record

https://scalar-labs.atlassian.net/browse/DLT-5554
Set srv_service_url in cassy.properties file using internal_domain.

I will update docs/CassySetup.md in the next PR that will make it work on Azure.

It should be fine so merging it. @tei-k Please take a look when you get a chance.

LGTM!


Recurring Events don't reuse images

The image used for the initial recurring event is not transferred over to the new one. Essentially this data is dropped from the event.

Recurring events are now deprecated due to it now being a native Discord feature. So this bug is no longer relevant.


Can't get LuckPerms link on console/rcon

Description
Unable to get LuckPerms session/link when using lp editor on console/rcon.
Reproduction Steps
Run lp editor on console/ over rcon
Expected Behaviour
Should output a link to the editor
Server Details
paper-1.20.1-60
LuckPerms Version
v5.4.98
Logs and Configs
https://pastebin.com/4kceDD0p
Extra Details
No response

Duplicate of #1365


Is the unmapped state still an error failing the scanner job?

Hello
I recently discovered that on the final report for one nodejs project there are Issues reported by ORT scanner saying that there are unmapped license found
Like:
PyPI::click:7.1.2 | The declared license 'BSD License' could not be mapped to a valid license or parsed as an SPDX expression.
in the past these kind of status failed the scanner job and I had to put the license remapping in the curations.yml file
Doen it still work like this?
I ask because I have another scan where in the log I see:
Found 6 errors, 0 warnings, 0 hints. There are issues with a severity equal to or greater than the WARNING threshold.
In the logs there are no errors, only 6 WARNINGS
Is this issue WARNING -> ERROR related to this:
00:38:40.589 [main] INFO  org.ossreviewtoolkit.downloader.Downloader - Trying to download 'NPM::make-fetch-happen:5.0.2' sources to '/tmp/ort-ScanCode16308538202973995538/NPM/unknown/make-fetch-happen/5.0.2' from VCS...
00:38:40.589 [main] INFO  org.ossreviewtoolkit.downloader.Downloader - Using processed VcsInfo(type=Git, url=https://github.com/zkat/make-fetch-happen.git, revision=b317c90d4d03367b6dd84ee676b850c11721887f, path=). Original was VcsInfo(type=Git, url=git+https://github.com/zkat/make-fetch-happen.git, revision=b317c90d4d03367b6dd84ee676b850c11721887f, path=).
00:38:40.589 [main] INFO  org.ossreviewtoolkit.downloader.Downloader - Detected VCS type 'Git' from type name 'Git'.
00:38:40.933 [main] INFO  org.ossreviewtoolkit.downloader.vcs.Git - Adding Git revision 'b317c90d4d03367b6dd84ee676b850c11721887f' (taken from package meta-data) as a candidate.
00:38:41.206 [main] INFO  org.ossreviewtoolkit.downloader.vcs.Git - No Git revision for package 'make-fetch-happen' and version '5.0.2' found: IOException: No matching tag found for version '5.0.2' among tags 'v1.3.1', 'v1.4.0', 'v1.5.0', 'v1.2.0', 'v1.2.1', 'v1.3.0', 'v1.5.1', 'v1.6.0', 'v1.7.0', 'v2.4.6', 'v2.4.5', 'v2.4.8', 'v2.4.7', 'v1.0.0', 'v1.0.1', 'v1.1.0', 'v2.4.9', 'v2.0.1', 'v2.1.0', 'v2.2.0', 'v2.0.2', 'v2.0.3', 'v2.2.2', 'v2.4.0', 'v2.0.4', 'v2.2.1', 'v2.3.0', 'v2.2.4', 'v2.4.2', 'v2.6.0', 'v2.2.3', 'v2.4.1', 'v2.5.0', 'v2.2.6', 'v2.4.4', 'v2.2.5', 'v2.4.3', 'v2.0.0', 'v2.4.13', 'v2.4.12', 'v2.4.11', 'v2.4.10', 'v3.0.0', 'v4.0.0', 'v4.0.1'. Please create a tag whose name contains the version.
00:38:41.206 [main] INFO  org.ossreviewtoolkit.downloader.vcs.Git - Trying revision candidate 'b317c90d4d03367b6dd84ee676b850c11721887f' (1 of 1)...
00:38:41.206 [main] INFO  org.ossreviewtoolkit.downloader.vcs.Git - Trying to fetch only revision 'b317c90d4d03367b6dd84ee676b850c11721887f' with depth limited to 50.
00:38:41.208 [main] INFO  org.ossreviewtoolkit.utils.ProcessCapture - Running 'git fetch --depth 50 origin b317c90d4d03367b6dd84ee676b850c11721887f' in '/tmp/ort-ScanCode16308538202973995538/NPM/unknown/make-fetch-happen/5.0.2'...
00:38:41.721 [main] WARN  org.ossreviewtoolkit.downloader.vcs.Git - Could not fetch only revision 'b317c90d4d03367b6dd84ee676b850c11721887f': IOException: Running 'git fetch --depth 50 origin b317c90d4d03367b6dd84ee676b850c11721887f' in '/tmp/ort-ScanCode16308538202973995538/NPM/unknown/make-fetch-happen/5.0.2' failed with exit code 128:
fatal: remote error: upload-pack: not our ref b317c90d4d03367b6dd84ee676b850c11721887f

or it is still a unmapped case?
Thank you in advance

There are multiple issues that come into play here:

The declared license 'BSD License' could not be mapped

We used to have a mapping for 'BSD License' but decided to remove it due to ambiguity.

I had to put the license remapping in the curations.yml file

Yes, the correct way to address this kind of issue is to use a package-specific declared license mapping in a curation.

In the logs there are no errors, only 6 WARNINGS

There is a difference between log output and recorded ORT issues. Log output in general might also come from third-party libraries used by ORT (if they use the same logger as ORT). The issue statistics refer to ORT issues in the ORT result files.

Ok, thanks I will apply the curations.yml file but the still it does not fully resolve my doubts because in my first NodeJS project I had unmapped cases and the scanner job wasn't failed - isseus were reported in the HTML report, no curation was required
(runnning analyse, scan, report in gitlab pipeline job)

I forgot to mention one additional thing that comes into play: There were (and still are) some inconsistencies about what counts as an issue (and at what severity), exit codes, reporter visualization, etc. Also see https://github.com/oss-review-toolkit/ort/issues/4096. Also note the new severeIssueThreshold configuration variable. In any case, please try with the very latest ORT commit (and configure severeIssueThreshold to your needs). Things should have been improved.

Also https://github.com/oss-review-toolkit/ort/issues/3324 seems to be related.

ok, this a vital information what counts as an issue,
thank you very much Senastian

Could you share what are the other things that count as an issue?
At my end I have remapped all unmapped licenses but still the jos fails
No errors either. The scan-results.yml is 18MB so I cannot place it here

Could you share what are the other things that count as an issue?

AFAIK, unmapped licenses are the only "non-issue thing" that count as issues (or better: issues are dynamically created but not serialized as issues, as explained here).

The scan-results.yml is 18MB so I cannot place it here

Not even if you ZIP it?

Hello
I use the commit: 82cf932674669918b97d336854df383d7474b2ec (5 days ago)
the issues which I tracked in the scan-results files are:
issues:
          - timestamp: "2021-06-23T01:38:23.591564Z"
            source: "ScanCode"
            message: "ERROR: Timeout after 300 seconds while scanning file 'third_party/dotnet/nunit-3.6.0/net-2.0/nunit.framework.tests.dll'."
            severity: "ERROR"
          - timestamp: "2021-06-23T01:38:23.591601Z"
            source: "ScanCode"
            message: "ERROR: Timeout after 300 seconds while scanning file 'third_party/dotnet/nunit-3.6.0/net-3.5/nunit.framework.tests.dll'."
            severity: "ERROR"

Yes, a zip file is better
scan-result.zip

So, some general remarks about my observations of the scan result:

Note that each section (from the analyzer, advisor, scanner etc.) should have it's own has_issues flag in the result file.
There is no has_issues flag at all in the analyzer part of your result file, which might be a regression from our switch to a new dependency graph format. We'll investigate.
The scanner section has has_issues: true, which is correct as there are issues in the form of scan timeouts.
Issue resolutions do not affect the serialized has_issues flag, as the resolutions are only applied as a "view" on top of the results by the reporter.

And to finally directly answer your original question:

Is the unmapped state still an error failing the scanner job?

Unmapped licenses never failed the scanner step, but only the analyzer step. And whether unmapped licenses fail the analyzer step now actually depends on the configured severeIssueThreshold (which defaults to Severity.WARNING): Unmapped license issues are of Severity.HINT if a concluded license has been set, and of Severity.WARNING otherwise. So, if you have not set a concluded license for a package, and you have unmapped licenses in that package, then unmapped licenses will fail the analyzer by default.

The scanner section has has_issues: true, which is correct as there are issues in the form of scan timeouts.

Is there a way to resolve these kind of issues? I don't want to change the timeout in the scancode settings because it affects all my previously cached scanned packages (for nodejs it was 1200 -> 4h of processing)
Documentation says:

Resolutions set in resolution.yml file are only taken into account by the reporter, while the analyzer and scanner ignore them.

In my pipeline the scanner ends with exit code 1 so I cannot use resilution.yml file here

Unmapped licenses never failed the scanner step (...) unmapped licenses will fail the analyzer by default.
From my observation the pipeline: analyzer, scanner, reporter is failed on scanner step? When I implement curation to umapped license then all three steps go well

Is there a way to resolve these kind of issues?

You basically gave yourself the answer already: These issues can be resolved (that is, what ORT calls "resolved") by writing issue resolutions. But having those issue resolutions does not affect the exit code of the scanner. This is a valid request, though, and I have filed https://github.com/oss-review-toolkit/ort/issues/4222 about it (I thought we already had an issue about this, but apparently that was not the case).
So, does that answer your original question? Are you good to close this issue in favor of the other existing / created ones?

yes of course
I am closing the issue


build: use libkmod pkgconfig and add with-glfs and with-qcow parameters

Looks good, thanks! :+1:

Maybe also document new build parameters in the readme?

Thanks for the merge. Sure, I'll work on a readme patch to document the new params.


Undeclared identifier calling a mixin with a template parameter.

I've tried this on 0.16.0 and the development branch of Nim.
Here's the code:
import
  emerald, streams

proc mixin_with_params(content: string) {.html_mixin.} =
  p: content

proc genSummaryPage(thingy: string) {.html_templ.} =
  body:
    call_mixin mixin_with_params(thingy)

proc doit() =
  let
    outfs = newFileStream(stdout)
    t  = newGenSummaryPage()

  t.thingy = "blah"
  t.render(outfs)

doit()

Compiling it gives me this error:
hg.nim(7, 37) template/generic instantiation from here
hg.nim(9, 34) Error: undeclared identifier: 'thingy'

It seems specific to passing the thingy parameter to the mixin.  Replacing call_mixin mixin_with_params(thingy) with call_mixin mixin_with_params("a string") works fine, and I can reference thingy in other parts of the template.  Just not as a mixin parameter.

This issue shows that emerald is more a tech demo than an actually useful tool for complex tasks. Sometimes, because something is possible does not mean that it is feasible.
This is a conceptual problem an I think the correct way to solve it is to implement the call-by-name-ALGOL-rule for parameter resolution.
That being said, it is unlikely that I will invest more work into this project, since I came to believe that while it is a pretty nice tech demo, it is not a great tool to tackle actual problems.

I understand, thanks for the response.


add visit method

Codecov Report
Patch coverage: 100.00% and project coverage change: -1.05% :warning:

Comparison is base (0a03493) 98.51% compared to head (ecac7eb) 97.47%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage   98.51%   97.47%   -1.05%
==========================================
  Files           6        6
  Lines         270      277       +7
  Branches       55       56       +1
==========================================
+ Hits          266      270       +4
- Misses          3        5       +2
- Partials        1        2       +1

Files Changed
Coverage Δ

blahblah/_generator.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

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


Add a test verifying that etcd watches work even when there's a bunch of unrelated changes

This passes for me locally. Suggests that #6059's problem is our use of etcd, not etcd itself.

Based on your description, the test itself couldn't reproduce the watch delay at all? Since you suggested that issue should be in the way on our use of etcd, can we make the test simulating how we are using etcd?

The point of this test is to rule out etcd as a source of the problem.
...hm, maybe I could make a more advanced test that checks our entire system. That would be interesting.

Thanks for the effort on reproducing the issue. Now we might have to disable the test by default, so that it can pass, be merged, and everyone can play with it. :-P

Test is disabled by default now.

...and I removed the test that didn't repro the issue. should be good to go now.

LGTM except one nit. Will merge once travis / shippable is green.


HHH-11081 - JPA join with treat causes QuerySyntaxException: Invalid path

https://hibernate.atlassian.net/browse/HHH-11081

Merged. Thanks!


Enhancement: Arrows and edge annotations in plot_graph

Hi @gboeing . If we have a graph and wish to zoom in on a particular junction (by passing a value to bbox) to understand what's going on, it could be useful to have additional arguments to plot the direction of traffic flow and annotate edges, with street names for instance.
I was wondering if you've thought about this enhancement before and if it's worth drafting a pull request.
Cheers.

That's an interesting idea. But I wonder if OSMnx is the right place for that functionality, given that GIS tools can do it relatively easily? Typically when I'm trying to better understand some section of the graph, I'll just dump it to shapefile > open in QGIS > click/explore interactively (to see street names, directionality, etc). Thoughts?

Yes, that is probably the sound thing to do. I'm still very much a rookie at GIS and hence not very familiar with the whole spatial analysis ecosystem, tools and workflow.
But what got me thinking about this, is the fact that I have 200+ traffic cameras, which I'm trying to merge with the road graph, based on location, address and camera orientation. Validating this at scale is a bit tricky and for that I'm generating a close up figure of the surrounding road network for each camera (~200 meter bbox). So far, I generate a pair of figures for every camera (example below):

Unmerged
Merged

At first it seems that camera 25 is being projected to the wrong edge, but after I've looked at the direction of traffic flow in Google Maps, it's actually fine. This would be straightforward to see by annotating the graph with arrows and street names.
But like you've suggested, I'm probably better off using other GIS tools? Do you think I can do this at scale with QGIS instead?

I think this workflow would be best-suited to something like QGIS. One other option would be to save the OSMnx graph to Folium/Leaflet and add your annotations there as an interactive web map: https://osmnx.readthedocs.io/en/stable/osmnx.html#osmnx.plot.plot_graph_folium


[feature request] support latest switch for option -p

We can currently specify specific package collections using the -p option to pack.
It would be nice to also support latest and fetch-latest as options, as we do for pack.toml files.
It might not be a huge thing to change, so even if you don't have the time to do it, feel free to sketch what needs changing and where. It might make a nice first-issue.

Well, latest:<branch> and fetch-latest:<branch> in pack.toml is a syntax for "dynamic" commits of a package, not a syntax for pack collections. latest in pack switch command is indeed used as the pack collection, but only during the fetching.
Am I right, that you would like to use all the newest versions of everything during pack -p fetch-latest build <whatever>? Have you tried using pack -p HEAD build <whatever> for that? Is this satisfactory for your expectations?

I wonder if all of this could be solved by allowing two things:

Specifying the pack.toml to use as a command-line argument
Allowing pack.toml files to explicitly state the parent file from which they inherit their settings

Currently, the following hierarchy is established (stuff on the left-hand side takes precedence):
command-line options <- pack.toml in current dir <- ... pack.toml files in parent dirs ... <- $PACK_DIR/user/pack.toml
The current state is well suited when invoking pack at the command-line when already in the correct directory, but not when calling it from other processes, where we probably don't want to have the settings from the current directory. Implementing the new behavior should not be too hard, but we should try and make it backward compatible so we don't invalidate all existing setups.


Product Groups

Description
Product Groups list
Checklist

[x] The code change is tested and works locally.
[x] tests pass. Your PR won't be merged unless tests pass
[x] There is no commented out code in this PR

Just wait before validation, a commit is missing : to expose groups in product class.


docs: Fix Frame Processor install step and QualityPrioritization Order

fix: switched incorrect property ordering for qualityPrioritization options
fix: added extra step required for creating a frame processing plugin on Android
I couldn't work out how to test the changes using docosaurus, but hopefully they're so small that they're ok!

Hey! Thank you for your PR!
You can test those changes by running:
cd docs
yarn start

or
cd docs
yarn build
yarn serve

but the CI GitHub action also tests the docs build :)

Ah thanks, I actually had to add in a yarn before yarn start to install all the dependencies for docosaurus.
I've also updated the contributing guide with that info to help others who might want to update the docs.

All done :white_check_mark:


Modin df.isnull().sum() slower than Stock Pandas

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 20.08
Modin version (modin.__version__): 0.9
Python version: 3.8
Code we can use to reproduce: https://github.com/oneapi-src/oneAPI-samples/tree/master/AI-and-Analytics/Getting-Started-Samples/IntelModin_GettingStarted and code on screenshot (add at end):

Describe the problem
Tested on 2 separate datasets, and machines, on AI Kit Modin and pip Modin installations - the dataset used in the screenshot code is 1.67 GB, so should be large enough to show performance.
df.isnull().sum() is significantly slower than stock Pandas (Modin performs at approximately 0.17x of stock Pandas speed).
Slow-down performance seen on multiple runs.
Source code / logs
See GitHub and screenshot above^

@aregm please prioritize


fix an import issue

Python complains that:
  File "{...}/.venv/lib/python3.12/site-packages/array_record/beam/arrayrecordio.py", line 10, in <module>
    from apache_beam.io.filesystem.CompressionTypes import AUTO
ModuleNotFoundError: No module named 'apache_beam.io.filesystem.CompressionTypes'; 'apache_beam.io.filesystem' is not a package

and indeed it's not.

lol


chore(package): update reflect-metadata to version 0.1.6

https://greenkeeper.io/

Coverage remained the same at 99.517% when pulling 8f985eb7989378f6da1058a23608552dc3f32a2a on greenkeeper-reflect-metadata-0.1.6 into d0d4455ff11c73095a0acb14e6c94bd10b279f6f on master.

Coverage remained the same at 99.517% when pulling 8f985eb7989378f6da1058a23608552dc3f32a2a on greenkeeper-reflect-metadata-0.1.6 into d0d4455ff11c73095a0acb14e6c94bd10b279f6f on master.


SD + leaf nodes

Currently, the SD code does not work for leaf nodes. It essentially cannot; the leaf nodes live on links DNCP is not active on, and the hncp_sd code uses only data in PA TLVs for it's decisions on what to publish as far as SD is concerned. Published SD data is used (mostly) to configure local dnsmasq/ohybridproxy/minimalist-pcproxy.
Only way this could work if the SD code interacted directly with PA (or iface code?) instead of using HNCP TLVs to learn of PA state in general.
(This opens also a general TODO item - is having 3 different per-interface structures (DNCP, PA, iface) sensible?)

This has been fixed semirecently (2-3 weeks); hnet-ttin has a testcase now too on this.


OnHalt callback

Introduce a callback (async method preferably) in Machine that is called when the machine is about to be halted. This will give the user a chance to clean up state. One can do this currently by handling the Halt event, but this will have to be done for all states of the machine.

My understanding of this feature is that this callback is invoked subsequent to unwinding the entire state stack of the halting machine and setting the "halted" status of the machine to true.  In this interpretation, the callback should be executing only non-P# code, i.e., it should not invoke any APIs for creating machines, receiving events, or sending events.


Adjust service selector

The idea is to align service selector with helm name so that you can have multiple oauth2 proxy running. Right now we have this issue in rfjh2 where we have two oauth2 proxy running in different namespaces.
By this we need to change one of the helm charts with a different name because it has some global resources (e.g. ClusterRole, etc.). Changing the value name is fine however it breaks by selecting the endpoints in the service because of the static label selector.

I am just a little bit worried about an update from the existing app as AFAIK you cannot update a selector

I am just a little bit worried about an update from the existing app as AFAIK you cannot update a selector

Hmm isn't that only counting for apps/v1 deployment that the label selector is immutable? It shouldn't stick for server selector. I did manually change the service selector to fix it and it wasn't complaining, at least core K8s doesn't block me doing this.

I am just a little bit worried about an update from the existing app as AFAIK you cannot update a selector

Hmm isn't that only counting for apps/v1 deployment that the label selector is immutable? It shouldn't stick to service selector. I did manually change the service selector to fix it and it wasn't complaining, at least core K8s doesn't block me doing this.

you are probably right, sorry for confusion


Translation: tools, input organization

I'm separating this topic out of https://github.com/ReactiveX/rxjs-docs/issues/29, cause this is most fundamental thing need to be set before actual work and most important thing for daily work as well.
#29 includes good initial ideas to discuss out
organizing source of trasnlated document

[ ] put under current repo
[ ] separate repo for each translation
[ ] convert current one into monorepo?

type of source we'll use

this depends on what tools we'll use

translation pipeline management tools

[ ] crowdin (https://crowdin.com/)
[ ] smartling (https://www.smartling.com/) free edition
[ ] weblate (https://weblate.org/)
[ ] pontoon (https://pontoon.mozilla.org/)
[ ] gitlocalize (https://gitlocalize.com/)
[ ] transifex (https://www.transifex.com/)
[ ] just manually do everything

contributor process

also depends on what tools we'll use
Main q: should we encourage any contributor works with githunb's PR flow / or better to make it happen even without knowning git / PR process?

/cc @ladyleet

My point of interests are

translation contributor need not know about things like cli, git, pr, etcs.
translation material though, should be manageable via github repo.
translation resources management should be easily doable (diff, tracking version, etcs) to orignal raw (English) material.
some sort of programmatic interface allows auto publish / deploy documents.
few other things I haven't thought of

I can confirm that the Yarn team is using crowdin (http://i18n.yarnpkg.com/project/yarn).
I've helped them in the past. The tool seemed to be easy to use. So I do agree to use such tools for "globalization" (g11n) instead of using git. PRs could be such mess when it comes to reviewing g11n content.

Here are my thoughts on this

has to be easy to contribute
has to be easy to access like the vue translations on the docs.
has to be easy to maintain.
has to be able to be released incrementally (like, if there are no translations it does something or says "not avail yet" or something). because i want to be able to release translated docs immediately.
maybe we can have a bar at the top of each translated and be like, this is X% translated - contribute now.
we just need a proposed solution and path going forward that meets these requirements and any others from the core team and then let's just run with it.

Another tool we can use: http://www.ngx-translate.com
Here is the demo of how it works/looks https://embed.plnkr.co/pYo6bFPRRxVPgRR8toDt/

In case anyone interested, here's the PR for ngx-translate, i'll write a short instruction on how to use translations tomorrow :) https://github.com/ReactiveX/rxjs-docs/pull/144 And here is ngx-translate enabled fork https://github.com/musienkoyuriy/rxjs-docs/  feat-i18n-integration tree @ladyleet


Workflow issue: Can't georeference a newly imported table because it goes to Visualization

Import a csv without any automatically recognizable location data
You get to a table full of the_geoms but there is no menu option to georeference

Yep, with the new workflow we should allow datasets from the map view. I think this is duplicated but I cannot find the other issue.

Well, found (although too late): https://github.com/CartoDB/cartodb/issues/2541
I'm closing this ticket.


Load custom reporter in a plugin component

Direct assignment of our reporter into the global reporter field results in occasional sbt exceptions like:
java.lang.ClassCastException: com.softwaremill.clippy.DelegatingReporter cannot be cast to xsbt.DelegatingReporter
       at xsbt.CachedCompiler0$Compiler.logUnreportedWarnings(CompilerInterface.scala:224)
        at xsbt.CachedCompiler0.processUnreportedWarnings(CompilerInterface.scala:145)
        at xsbt.CachedCompiler0.run(CompilerInterface.scala:117)
        at xsbt.CachedCompiler0.run(CompilerInterface.scala:95)
        at xsbt.CompilerInterface.run(CompilerInterface.scala:26)

It's caused by sbt delegating reporter doing a direct cast of global reporter in the CachedCompiler.logUnreportedWarnings functions.
This PR introduces a custom compilation phase injected between parser and namer. This phase will replace global repoter with Clippy's one and looks like SBT won't do the casting afterwards.
However, our custom phase won't get injected when compilation is started from tests, so I added another "undocumented" plugin option testmode. When true, the reporter assignment will happen as previously.

0.3.6 released


Null Refrence Exception in Encoding Thread

Overview
I basically copied the code from the GUI example (witch was working with the same versions of everything), and when I run it I get a null reference exception when I try to connect to a mumble server. It is connecting to the mumble server, witch is the uMurmur server integrated in CitizenFX Framework (FiveM/RedM), and I have confirmed that it did actuality join with a standard client also in the server. There is also about a 5 second delay from when I run the connect function, and when I get the error, and it receives 7 messages during that time.
Stack Trace
Unhandled exception. System.Exception: BasicMumbleProtocol's _encodingThread was terminated unexpectedly because of a System.NullReferenceException
 ---> System.NullReferenceException: Object reference not set to an instance of an object.
   at MumbleSharp.Audio.Codecs.Opus.OpusDecoder..ctor(Int32 outputSampleRate, Int32 outputChannelCount)
   at MumbleSharp.Audio.Codecs.Opus.OpusCodec..ctor(Int32 sampleRate, Byte sampleBits, Byte channels, UInt16 frameSize)
   at MumbleSharp.Audio.CodecSet.<>c__DisplayClass4_0.<.ctor>b__0()
   at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
   at System.Lazy`1.ExecutionAndPublication(LazyHelper executionAndPublication, Boolean useDefaultConstructor)
   at System.Lazy`1.CreateValue()
   at System.Lazy`1.get_Value()
   at MumbleSharp.Audio.CodecSet.GetCodec(SpeechCodecs codec)
   at MumbleSharp.Audio.AudioEncodingBuffer.Encode(SpeechCodecs codec)
   at MumbleSharp.BasicMumbleProtocol.EncodingThreadEntry(Exception& exception)
   --- End of inner exception stack trace ---
   at MumbleSharp.BasicMumbleProtocol.<Initialise>b__40_1()
   at System.Threading.Thread.StartCallback()

Versions

net6.0-windows
MumbelSharp 2.0.0
NAudio 2.1.0
protobuf-net 3.1.26

Can you please check that opus.dll is copied over in the output Subfolder "\Audio\Codecs\Opus\Libs\32bit"

There is an opus.dll at bin\Debug\net6.0-windows\Audio\Codecs\Opus\Libs\32bit\ as well as one in the 64bit folder. I am running it in VS Code.

The Nuget package has been updated to v2.0.1 so that the needed codecs files opus.dll are automatically copied to the output folder of projects referencing the nuget package, this should solve your issue. Please report if it did solve your issue or not, thanks.


Add version pattern on branchingPolicies

Add customizable patterns on branching policies.
This will allow us to create a custom pattern on release branches for git-flow.
fix #81
If this PR is ok I can do the update on maven and Gradle plugins.

Hi, @McFoggy, great work on jgitver! Any chance this PR by @Zomzog could get merged? Would be wonderful, thanks!


RuntimeError: OffloadedCache can only be used with a GPU

I have a 3090 with 24GB VRAM. i tried to geneate with 2 images and i get this error
Traceback (most recent call last):
File "F:\OmniGen\venv\lib\site-packages\gradio\queueing.py", line 624, in process_events
response = await route_utils.call_process_api(
File "F:\OmniGen\venv\lib\site-packages\gradio\route_utils.py", line 323, in call_process_api
output = await app.get_blocks().process_api(
File "F:\OmniGen\venv\lib\site-packages\gradio\blocks.py", line 2018, in process_api
result = await self.call_function(
File "F:\OmniGen\venv\lib\site-packages\gradio\blocks.py", line 1567, in call_function
prediction = await anyio.to_thread.run_sync(  # type: ignore
File "F:\OmniGen\venv\lib\site-packages\anyio\to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "F:\OmniGen\venv\lib\site-packages\anyio_backends_asyncio.py", line 2441, in run_sync_in_worker_thread
return await future
File "F:\OmniGen\venv\lib\site-packages\anyio_backends_asyncio.py", line 943, in run
result = context.run(func, *args)
File "F:\OmniGen\venv\lib\site-packages\gradio\utils.py", line 846, in wrapper
response = f(*args, **kwargs)
File "F:\OmniGen\app.py", line 22, in generate_image
output = pipe(
File "F:\OmniGen\venv\lib\site-packages\torch\utils_contextlib.py", line 116, in decorate_context
return func(*args, **kwargs)
File "F:\OmniGen\OmniGen\pipeline.py", line 278, in call
samples = scheduler(latents, func, model_kwargs, use_kv_cache=use_kv_cache, offload_kv_cache=offload_kv_cache)
File "F:\OmniGen\OmniGen\scheduler.py", line 156, in call
cache = [OmniGenCache(num_tokens_for_img, offload_kv_cache) for _ in range(len(model_kwargs['input_ids']))] if use_kv_cache else None
File "F:\OmniGen\OmniGen\scheduler.py", line 156, in
cache = [OmniGenCache(num_tokens_for_img, offload_kv_cache) for _ in range(len(model_kwargs['input_ids']))] if use_kv_cache else None
File "F:\OmniGen\OmniGen\scheduler.py", line 14, in init
raise RuntimeError("OffloadedCache can only be used with a GPU")
RuntimeError: OffloadedCache can only be used with a GPU
What needs to be done?

This issue is caused by the GPU not being detected, usually due to an incomplete GPU environment setup. You can refer to the following solution: https://github.com/VectorSpaceLab/OmniGen/issues/67

Got the same error, solved it, by installing:
pip install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/cu118
You have to replace cu118 with the cuda version you have installed.


UI for crosschain modules

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR introduces a new feature for displaying and managing cross-chain contract deployments in the dashboard. It adds a Cross Chain category, a dedicated page for cross-chain information, and a data table for visualizing deployment status across multiple chains.
Detailed summary

Added Cross Chain category in explore.ts.
Created a new page at cross-chain/page.tsx to handle cross-chain contract logic.
Implemented DataTable component in cross-chain/data-table.tsx for displaying deployment status.
Integrated logic to fetch and compare contract code across multiple chains.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5351  👈
#5350
modules

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

Join @GWSzeto and the rest of your teammates on  Graphite


fixed MockWsvCommand

Signed-off-by: Mikhail Boldyrev<EMAIL_ADDRESS>

Description of the Change

Benefits

Possible Drawbacks

Usage Examples or Tests [optional]

Alternate Designs [optional]

ok


How can I get the time and date whenever the time changed?

Whenever the date changes I can get the value but time changes it does not give me the changed value?!

It works correctly.
Please see this sandbox and tell me if there is any problem.
https://codesandbox.io/s/competent-voice-mcz6j?file=/src/App.js

I wanna see value on log at each time the value of time changes, but now it shows me the value when the value of time changes only the first time.

Oh I see !!!

Thank you for taking the trouble to help me.I do appreciate it.

You're welcome.


Update validate-web-part-property-values.md

Added the example listName property mentioned in the instruction just above, but not present in the sample JSON
Category

[x] Content fix
[ ] New article
Related issues: fixes #X, partially #Y, mentioned in #Z

What's in this Pull Request?
The article has instructions to add a property to the json, but the json example does not contain the property that the instructions say to add.
Added the property to the sample text so that the content is consistent
Guidance
Fix to article content

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/spfx/web-parts/guidance/validate-web-part-property-values.md
:white_check_mark:Succeeded
View

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

thx @Joon, appreciated.


Datasource discovery through K8s Services

Is your feature request related to a problem? Please describe.
At Gitpod, we have a Kubernetes cluster and a CI/CD system that creates a new namespace and deploy a new Gitpod installation for every commit pushed to any of our branches. We call these installations a "Preview Environment", that can be used for several reasons, e.g., load testing, UI previews, manual testing. After some time, our CI/CD system will also delete the preview environment and the namespace that was created since they are not needed anymore.
We are starting to step up our game when it comes to reliability and would like to enhance this system to also deploy a Prometheus instance alongside each preview environment. This strategy creates a pretty dynamic Prometheus setup, where our Prometheus instances are constantly being created and deleted throughout the day.
We don't want to set up multiple Grafanas though, we would like to have one single Grafana that would have several Prometheuses as datasources. This way we can simply create a Datasource variable to all dashboards and visualize and benchmark metrics from different preview environments at the same place.
Describe the solution you'd like
I'd like to have a K8s Service that would discover all my Prometheus instances and a CRD that would set up GrafanaDatasources for each endpoint that my K8s Service discovers. The exact same strategy that Prometheus-Operator's ServiceMonitor uses to discover metrics endpoints.

discover all my Prometheus instances

I'm wondering how this discovery would work.
Would it be some logic that understands what a Prometheus CR is and watches for them across the whole cluster?
Or would it be looking for specifically labelled services or pods?

set up GrafanaDatasources for each endpoint that my K8s Service discovers

Yes, this makes sense as the easiest way to interact with the grafana operator & manage datasources.

exact same strategy that Prometheus-Operator's ServiceMonitor

That's interesting.
In this case, the prometheus operator looks for services based on namespace & service label selectors (which are configurable in the Prometheus CR)
The discovery is an endpoint to start scraping in prometheus.
Are you proposing a feature of the grafana operator to watch for services based on namespace & service label selectors, and convert them into a data source?
There's typically a number of fields that make up a data source (e.g. auth config), so where would that config be pulled from? The service endpoint may not be enough

Hey, thanks for the quick response!

I'm wondering how this discovery would work.
Would it be some logic that understands what a Prometheus CR is and watches for them across the whole cluster?
Or would it be looking for specifically labelled services or pods?

No, I don't think it is necessary that Grafana knows if a discovered endpoint is indeed a Prometheus or not. I think that such validation should be done by the user.

exact same strategy that Prometheus-Operator's ServiceMonitor

That's interesting.
In this case, the prometheus operator looks for services based on namespace & service label selectors (which are configurable in the Prometheus CR)
The discovery is an endpoint to start scraping in prometheus.
Are you proposing a feature of the grafana operator to watch for services based on namespace & service label selectors, and convert them into a data source?

Exactly :slightly_smiling_face:! Probably a new CRD that manages this discovery and configures Grafana with new datasources would make sense.

There's typically a number of fields that make up a data source (e.g. auth config), so where would that config be pulled from? The service endpoint may not be enough

Good point! I think that the strategy could be copied from Prometheus Operator as well. When discovering endpoints, all information necessary can be added to an Endpoint struct, and it could be used to configure the datasources inside Grafana.
This strategy assumes that all endpoints can be retrieved and accessed with the same config, changing only the endpoint address. If you'd like to set up datasources with different auth configs, or different types, e.g., Prometheus and MySQL, then a separate "ServiceMonitor" would be necessary.

Would something like this make sense?

Maybe a better alternative is to just create a single CRD, but with a type field. An admission webhook would validate the fields with the provided type.

@ArthurSens how long lived are those dynamic Prometheus instances? One thing to be aware of is that Grafana needs to be restarted when new Datasources are added (or updated/removed). If the frequency is too high, the restarts could become a problem.

They remain up for at least 2 hours, but can be extended if needed :slightly_smiling_face:
And thanks for pointing it out about the restart problem, I wasn't aware of it. Just for curiosity's sake, why does Grafana need to be restarted? I thought that Grafana-Operator created dashboards and datasources with Grafana's autoprovisioning feature, that way grafana can hot-reload everything :thinking:

The operator manages dashboards via the Grafana API, but datasources are still written to a ConfigMap and mounted into the filesystem. Grafana reloads this file only on restart.
But there is a datasource API that we could use: https://grafana.com/docs/grafana/latest/http_api/data_source/#data-source-api

But there is a datasource API that we could use: https://grafana.com/docs/grafana/latest/http_api/data_source/#data-source-api

Oh nice! That is even better than autoprovisioning


failed to import .mts file without extension in .mjs file

Bug description
app.mjs
import { v } from "./lib"
console.log(v)

lib.mts
export const v = "hello"

npx tsx app.mjs

Cannot find module '.../lib' imported from .../app.mjs

Reproduction
https://github.com/trim21/node-4wre7g/tree/main
Environment
System:
    OS: Windows 10 10.0.19045
    CPU: (16) x64 AMD Ryzen 7 5800X 8-Core Processor
    Memory: 11.49 GB / 31.93 GB
  Binaries:
    Node: 18.15.0 - ~\scoop\apps\nodejs-lts\current\node.EXE
    Yarn: 1.22.19 - ~\scoop\apps\nodejs-lts\current\bin\yarn.CMD
    npm: 9.6.2 - ~\scoop\apps\nodejs-lts\current\bin\npm.CMD
  npmPackages:
    tsx: ^3.12.1 => 3.12.6

Can you work on a fix?

[ ] I’m interested in opening a pull request to address this issue.

I don't think this is a bug.
Your tsconfig.json is missing an allowJs, but even with it, it will compile the files app.mjs -> app.mjs, and lib.mts -> lib.mjs. And Node.js wouldn't be able to resolve ./lib to ./lib.mjs.

that make sense.


Ignore renamed Info.plist files

At the moment Info.plist files are treated differently in that they don't get added to the Resources build phase.
We should change this to check a target's various levels of build settings for INFOPLIST_FILE, and ignore what is in that setting as some projects may use a different name than Info.plist

I wonder if it's worth creating an actual infoPlist property on Target for easily setting this build setting?

infoPlist property on Target

I would suggest users to set INFOPLIST_FILE somewhere explicitly, rather than introducing infoPlist property on project.yml. That's how it works anyways.
And this infoPlist property possibly conflicts with INFOPLIST_FILE settings. We're already doing implicit Info.plist detection, so implementability of this part would become worse I guess.


Ownership & Deletion

Every item should have an "Owner" / "OP" (original poster) tag.
These users get the ability to remove the app that was posted.

Yeah you tried to borrow me from Reddit how do you like this one oh wow I'm even older in this account and like I was trying to I am one of the original Android Engineers yes we swear our brains off okay there's no reason to be borrowing people if you're working on Android software and the software you're looking for I can give you the most current copy because the guy I work with in Russia with his Play Store so it's not even in English we have a copy I could give it to you but you already bugged me on Reddit right so why should I help you here think about it why don't you give me a text back huh don't do that stuff I haven't even on Reddit and how many months I'm the breakfast guy okay and that's a two and a half year old account so you can see I don't go on that often I could help right now but why should I cuz yeah your Lucky Patcher is way out of date

Do you have any response for yourself now cuz if you don't I'll just keep it the way it is where it doesn't work for anybody you're lucky patcher that's fine with me peace out I'll give you some minutes here and then I'm going I'll be off read it in the site for quite a long time because I don't pop in that often unless I see some serious issues going on with my operating system I'm going to you know I am Android it's my operating system man designed it when I was a kid


"visible()" returns "true" for an element outside visible viewport when its CSS "display" property is "none"

Can something be done about that?
Note:
This was only reproduced using jquery.visible.min.js (on master). Using jquery.visible.js returns false.

:+1:  It took me hours to find this behavior, it was bugging my WordPress theme.

This can be very easily done by adding .is(':visible') selector before calling this plugin. Docs explain that this is outside of the scope of this plugin.


[Rename] PowerToys Regex syntax is weird

Microsoft PowerToys version
0.53.1
Running as admin

[ ] Yes

Area(s) with issue?
PowerRename
Steps to reproduce
Regex syntax used by PowerRename is weird: \. matches .; and . matches anything - not the other way around, as you are suggesting.
Additionally, your documentation contradicts itself: https://docs.microsoft.com/en-us/dotnet/standard/base-types/regular-expression-language-quick-reference#character-classes

Activating or not the Boost library does not make any difference.
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

True. . means any (single) character and .* means any character for any number of times.

Then why am I getting no preview? 😕

also why is this drop-down so obscenely large for just 9 entries?

Then why am I getting no preview?

Don't know. Can't see your settings, search patterns or file names...

also why is this drop-down so obscenely large for just 9 entries?

CC @niels9001

Works fine for me

Don't know. Can't see your settings, search patterns or file names...

I don't see a problem removing the first 5 characters, filenames are >=26 chars.
Also also, the "Replace with" is funnily-aligned (and also obscenely large too)

It's not a specific issue with a specific file name per se - I just type too quickly or too slowly, and eventually it ends up at that state

In the meantime, you can check and then uncheck Match all occurrences to have correct behavior..

@stdedos Did you try this? Does it work this way?

No, that doesn't help 😕

I see.. Is this the same behavior that you get?

Is what happens at the end of the video what you would expect to happen?
One problem that I see here is that with current PT version Match all occurrences is enabled by default on PowerRename start (even though checkbox is shows different - this I resolved with PR I linked few comments above).. With that behavior Match all occurrences and regex both enabled in this case can be confusing.. e.g. if you type . for regex PowerRename tries to match both regex dots and same characters

Aaaah - so the workaround needs to be "acted", and then I need to empty and re-write the box?
Or empty the box before I apply the  workaround?
... all of that logic sounds very complex to verify the issue/workaround it.
I'll wait for the aforementioned patch to be released first, then come back here.

However, yes - you are very true that your replication looks very much like related to my own

Aaaah - so the workaround needs to be "acted", and then I need to empty and re-write the box?
Or empty the box before I apply the  workaround?
... all of that logic sounds very complex to verify the issue/workaround it.
I'll wait for the aforementioned patch to be released first, then come back here.

However, yes - you are very true that your replication looks very much like related to my own

Or empty the box before I apply the workaround?

Yes. Either before or after.
@stdedos Ok. Makes sense. Thanks for help! I believe your issue will be resolved with next PowerToys version.

This is fixed with 0.561.  Please head over to https://aka.ms/installpowertoys

Verified as fixed


Goo Keeper Template Game Crashing

What you were trying to do
Play the Goo Keeper template game
What actually happened
Loaded Goo Keeper game and within about 30 seconds of coming in close proximity to the "Goos", they began to move back and forth faster and faster until the application crashed.  Could possibly be mem leak/overflow, but not sure.  I'll leave that to the pros :)
How to reproduce
Startup the app, load the Goo Keeper template game
Log details and game version
Alpha v5.1.1
Computer details
Operating System: Ubuntu 20.04.3 LTS
Kernel: Linux 5.11.0-34-generic
Architecture: x86-64

What you were trying to do
Play the Goo Keeper template game
What actually happened
Loaded Goo Keeper game and within about 30 seconds of coming in close proximity to the "Goos", they began to move back and forth faster and faster until the application crashed. Could possibly be mem leak/overflow, but not sure. I'll leave that to the pros :)
How to reproduce
Startup the app, load the Goo Keeper template game
Log details and game version
Alpha v5.1.1
Computer details
Operating System: Ubuntu 20.04.3 LTS
Kernel: Linux 5.11.0-34-generic
Architecture: x86-64

This was me when the game was working previously

@runlevel-4 I wasn't able to reproduce this with Terasology 5.1.1 :thinking:
Can you reliably reproduce this on your machine or was this only a sporadic issue?

Closing this as author didn't respond and Goo Keeper is removed since Terasology 5.3.0-rc.2


change the channel name!!!!!

improve channel section!
3 channels appeared in the changes
!

i can change the name of the channel to "Normal Server", but there's no point on having 3 of them


#3934: Enable DPRINT on N300 Device 0

Changes /fixes to the DPRINT server + tests to enable running on n300 device 0. Multi-device will be left for a separate PR.
Also some general refactoring/cleanup in the print server.

What dprint features are working now? Are you able to enable and see dprint dumped from a remote chip?

What dprint features are working now? Are you able to enable and see dprint dumped from a remote chip?

Currently all dprint features/tests (fast and slow dispatch) are working on n300 on device 0 only (which is what issue #3934 is for). Looking into getting things working/testing for the remote device now, but will be a separate PR.

Passing CI: https://github.com/tenstorrent-metal/tt-metal/actions/runs/7281391145. Will merge


Rbac user editing, and routing.

I have made some modifications to enable users editing via RbacUserForm.
I have also created a demo which includes all rbac components, that are needed for adding/viewing/editing users while using client-side redux routing. It should be reproducible in manageiq-ui-classic and with some API tuning we could replace bunch of old code.
Demo is using hashHistory, so it should not collide with any ruby routing.
Storybook for demo: https://5b34e0ddfdd72a0b134bf4fb--confident-banach-ee0a7a.netlify.com/?selectedKind=Rbac forms&selectedStory=Rbac complex demo&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Fstories%2Fstories-panel

@karelhala sorry for adding demo here, i've noticed it after i've pushed it...

@karelhala @himdel @martinpovolny Please check the storybook in description (and code). I would like to know your opinion on this. Good/bad idea? All components can be user separately, so if this turns out as a horrible idea, we can still use these components for the forms etc.

@PanSpagetka, @karelhala : can you, please, test this one?

I have issues when i try to run it in Firefox (60.0.2 ), when I try chrome it is ok
Screenshot:

I have encountered an issue with react-router-dom package in ui-classic. It seems when you add <Router /> component and render the component it will start an infinite loop... We (me and @karelhala) will look into it and see what can be done about it. If we cannot fix it, the whole idea of client-side routing in ui-classic will be useless and this can be scrapped.

The issue with routing in miq was resolved. See https://github.com/ManageIQ/manageiq-ui-classic/pull/4284. Thanks @karelhala and @himdel for help. It turned out to be very easy fix.

@PanSpagetka the firefix issue is caused by redux dev tools... Removing window.__REDUX_DEVTOOLS_EXTENSION__ && window.__REDUX_DEVTOOLS_EXTENSION__() fixes the issue. I will keep it there for now. It will not make it the final component anyway.


Import segmented ops

We need to add support for segmented representations, which means already importing the segmented ops such that they are visible in Gremlin config YAML files.

Done.


run metrics server as non privileged user and in non rootfs

The PR adds support for running metrics server as non root user and uses /tmp dir for storing certs.
Related issue: https://github.com/kubernetes-incubator/metrics-server/issues/152
cc/ @DirectXMan12

Can this be approved? Kubernetes clusters running with podsecuritypolicies enabled can't start the container:
Events:
  Type     Reason     Age               From                                              Message
  ----     ------     ----              ----                                              -------
  Normal   Scheduled  38s               default-scheduler                                 Successfully assigned metrics-server/metrics-server-5f64dbfb9d-zf6c8 to k8spoc-worker003
  Normal   Pulling    37s               kubelet, k8spoc-worker003  pulling image "gcr.io/google_containers/metrics-server-amd64:v0.3.1"
  Normal   Pulled     35s               kubelet, k8spoc-worker003  Successfully pulled image "gcr.io/google_containers/metrics-server-amd64:v0.3.1"
  Warning  Failed     7s (x4 over 35s)  kubelet, k8spoc-worker003  Error: container has runAsNonRoot and image will run as root
  Normal   Pulled     7s (x3 over 35s)  kubelet, k8spoc-worker003  Container image "gcr.io/google_containers/metrics-server-amd64:v0.3.1" already present on machine

/assign @piosz

metrics-server chart in stable repo now runs as non-root. It also has some additional security improvements that you're welcome to incorporate into this PR.

@vjdhama could you please solve these conflicts?

/remove-lifecycle rotten

/assign @s-urbaniak

/assign @brancz

/lgtm

@brancz "approved" label is missing 😞

/approve

Thank you!

i am using k3s (version 0.10.1) and with this merge, i can't deploy metrics server :/ k8s version 1.16.2 and now this error appears ...

ubuntu@master:~$ kubectl describe pod metrics-server-66f4f4c658-qj5f4 -nkube-system
...
...
...
Events:
  Type     Reason     Age                From               Message
  ----     ------     ----               ----               -------
  Normal   Scheduled  <unknown>          default-scheduler  Successfully assigned kube-system/metrics-server-66f4f4c658-qj5f4 to node-1
  Normal   Pulling    13s (x5 over 54s)  kubelet, node-1    Pulling image "k8s.gcr.io/metrics-server-amd64:v0.3.6"
  Normal   Pulled     13s (x5 over 52s)  kubelet, node-1    Successfully pulled image "k8s.gcr.io/metrics-server-amd64:v0.3.6"
  Warning  Failed     13s (x5 over 52s)  kubelet, node-1    Error: container has runAsNonRoot and image will run as root
ubuntu@master:~$

@ysolis these changes are not already released. You need to wait until the next release to use it.

@ysolis these changes are not already released. You need to wait until the next release to use it.

yes, i know, to avoid this problem i deployed from latest release, not master branch ...


proposal: go/parser: add a SkipObjectResolution mode flag

In #45104, go/parser was refactored to make parsing and object resolution separate passes within ParseFile. As a side effect of this change it is trivial to add a mode that skips object resolution entirely. This issue proposes adding such a mode flag: SkipObjectResolution. (note that this mode was initially added as part of #45104, but that is going to be backed-out pending the discussion here).
Background
Object resolution is a feature of go/parser that attempts to resolve identifiers to their declaration, setting ast.Ident.Obj, for use in tools or basic analyzers that need such information. Notably this information is incomplete and in some cases inaccurate: object resolution does not properly handle selectors or fields in composite literals, and often lacks information about declarations in other files or packages. It is also redundant with the information produced by go/types. For this reason object resolution has limited utility, particularly for tools that either deal only with syntax, or which use the output of go/types for identifier information (go/types itself does not need object resolution in order to function).
Pros and Cons
In my testing, skipping object resolution saves 18-20% of the parsing time, which made gofmt around 10% faster (when not rewriting or simplifying, which make use of object resolution). It also avoids unnecessary allocation.  It's a small but meaningful optimization for tools that know they won't need ast.Ident.Obj. See #45104 for some further discussion.
For completeness, it's worth considering the downside of adding such a mode. For one, it adds a new mode that must be supported. However, I think this is a negligible cost: it's unlikely that in the future we revert the refactoring of #45104, but even so, skipping object resolution can still be achieved by adding guards in the functions that record or resolve identifiers. It's also worth noting that this mode is not entirely orthogonal to existing modes: ReportDeclarationErrors only works if object resolution is not skipped.
I think the strongest argument against adding this new mode is that it's simply not necessary. However, my current expectation is that it would add a small benefit for some users at a relatively smaller cost.
CC @griesemer @mvdan

The object resolution mode is not useful for tools that run a resulting AST through the type checker or which only care about an unadorned syntax tree, e.g. for formatting. These feature is present for historical reasons (the go/parser is one of the first larger go/package that was ever written and still in use).
The fact that not doing object resolution can save up to 20% parse time is significant. Arguably, not doing object resolution should be the default but that would be a non-backward compatible change of behavior. Note also that this mode was implemented and operational well before the Go1.17 freeze.
In short, the suggested new SkipObjectResolution mode provides an easy way to increase performance for many tools simply by setting the flag.
I don't see any good reason for not doing this.

I fully agree with what Robert said. I don't see a reason why this change must be in 1.17, but the code and API have been there for a while, so it's not like we missed the freeze deadline. So I also don't see a reason to push it back to 1.18.
We did not follow the proposal process properly, which perhaps meant some potential objections weren't raised. We can see here if anyone brings any in the next week or two, though I personally doubt it :)

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

Closing as the work was done in #45104.
#46298 tracks documenting this new mode for 1.17


pytest error

Some of the tests are failing with the following error:
Run pipenv run pytest -v
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/site-packages/pipenv/core.py", line 2739, in _launch_windows_subprocess
n subprocess.Popen([command] + script.args, **options)
  File "/opt/hostedtoolcache/Python/3.9.16/x64/lib/python3.9/subprocess.py", line 951, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "/opt/hostedtoolcache/Python/3.9.[16](https://github.com/clowder-framework/clowder2/actions/runs/3707688397/jobs/6284399647#step:9:17)/x64/lib/python3.9/subprocess.py", line 1821, in _execute_child
    raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: '/home/runner/.local/share/virtualenvs/backend-yQwxK-bj/bin/pytest'

During handling of the above exception, another exception occurred:

linked to #563


removed user gradle directory, already added to gitignore

@tyle8552 Another quick look at the .gradle folder removal please.

@doneill Looks good. Just added one change I didn't catch on your other PR.


README

README
USER STORY
As a boot camp student,
I want the prework notes to be structured on a webpage,
So that I can easily find and read the information.
ACCEPTANCE CRITERIA
GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it

GIVEN the prework project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it


Support electrs for Mempool Data

Similar to how we'll support Bitcoin Core or electrs for tx composition (cf. https://github.com/CounterpartyXCP/counterparty-core/issues/1764)

I don't think we actually need this? It'd probably also only be desirable if we could batch requests using the JSON-RPC API.


Request Modifier not reached when chained with ConfigurePrimaryHttpMessageHandler

Any ideas?  I need the ConfigurePrimaryHttpMessageHandler for self signed cert purposes.

            services.AddRestEaseClient<T>(baseAddress,
                client => { },

                 async (request, cancellationToken) => {
                    //do some request changes
                 })
                .ConfigurePrimaryHttpMessageHandler(() => new HttpClientHandler
                 {
                     ClientCertificateOptions = ClientCertificateOption.Manual,
                     ServerCertificateCustomValidationCallback =
                    (httpRequestMessage, cert, cetChain, policyErrors) => {
                        return true;
                    }
                 });

This is kind of related to
Looks like the .ConfigurePrimaryHttpMessageHandler is configured after, but then how do I configure the cert?

In the end I took inspiration in this class  and made it work by introducing the ServerCertificateCustomValidationCallback within the constructor.

Yep it's the same issue. I'll get that fixed in the next release.

(this issue will get closed automatically when the release goes out)


[4.0.2] Creating a new window of already open application sometimes ignores focused window, is created in the wrong place

Situation: window layout like so (using Safari and TextEdit as an example, but not limited to these)
---------
| S | S |
|   |   |
---------

Focusing the right hand window, creating a new TextEdit window with spotlight -> cmd+n sometimes inserts like
---------
| T | S |
|---|   |
| S |   |
---------

Strangely I just went to repeat this action and it inserts in the correct place, but new iTerm windows are now opening in the top left spot instead. This seems to only happen sporadically with new windows of an application which is already open in a different space (such that new windows are created through cmd+n and don't automatically open on spotlight search). It seems to behave as if no window is focussed, but the focus border is still visible on the right hand window.

I too have been experiencing this issue.
To add more context any way a window is spawned can create this behavior. For example I have cmd + enter bound to create a new iTerm window, and regardless of what window I have focused it puts it in an unexpected place.
It's not always the top left, and sometimes the window is created in the correct leaf as expected.

This seems to only happen sporadically with new windows of an application which is already open in a different space (such that new windows are created through cmd+n and don't automatically open on spotlight search). It seems to behave as if no window is focussed, but the focus border is still visible on the right hand window.

So what I think is happening is that the application on the other space is focused by OSX. The cmd+n is then registered by that newly focused application and then spawns the window.
Kwm notices the change in focus and updates its focused, but the border is not updated, because the focused window of the new application is not on the same space, so we have a trailing (out-of-sync) border. Kwm then notices that it has no focused window on the current space and defaults to the top-left split (changed this recently, see #553).

To add more context any way a window is spawned can create this behavior. For example I have cmd + enter bound to create a new iTerm window, and regardless of what window I have focused it puts it in an unexpected place.

Hm any more info on this? iTerm is the one application that always does behave the way I expect it to.

Hm any more info on this? iTerm is the one application that always does behave the way I expect it to.

No more info than it was the exact issue described above. I see you made some commits related to window insertion. I will pull master and give it a shot to see if it changes anything.

@koekeishiya Ok, so I have more information about iTerm.
The window spawns near other iTerm windows always, regardless of what window is focused. However it does always stay on the correct (focused) space. If there are no iTerm windows on the current space the window gets inserted into the correct location.
For what it is worth this is how I am spawning iTerm windows
chrishoage at Constance in ~/Projects/kwm on master
› cat ~/bin/open-iTerm.applescript
#!/usr/bin/env osascript

if application "iTerm2" is running then
    tell application "iTerm2"
        create window with default profile
    end tell
else
    activate application "iTerm2"
end if

Perhaps that applescript has something to do with where the window decides to spawn.


Replace python-jose with joserfc

python-jose had the latest release 3 years ago and it does not seem to be properly maintained
we might want to replace it with newer library such as:

PyJWT: https://github.com/jpadilla/pyjwt - https://pypi.org/project/PyJWT/
joserfc: https://github.com/authlib/joserfc - https://pypi.org/project/joserfc/

switch to pyjwt in https://github.com/lnbits/lnbits/pull/2591


DateTimePicker keyboard  visible

When choosing a date or time, the keyboard is visible, someone solved the problem?? with datetime_picker_formfield this was not

Hi @AlexEAA, @lkloon123
Do we still have this issue in the latest version of this package? I'm unable to reproduce this currently

Hello, @danvick.
I am having this issue with the latest version of the package.


[Spec Update] Rename is_primary to default_agg_time

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward change to existing dbt-semantic-interfaces functionality, rather than a Big Idea better suited to a discussion

Describe the feature
As above so below. Let's do some renaming!
Describe alternatives you've considered
No response
Who will this benefit?
No response
Are you interested in contributing this feature?
No response
Anything else?
No response

Closed in favor of doing it all in #50


[Automation] Generate Fluent Lite from Swagger reservations#package-2022-11

[Automation] Generate Fluent Lite from Swagger reservations#package-2022-11

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.resourcemanager:azure-resourcemanager-reservations


Q-Bench Evaluation Scripts for LLaVA-v1.5.

Hi Haotian,
Here is the pull request as asked in https://github.com/haotian-liu/LLaVA/issues/538. We have carefully tested and debug the code so that it can run seamlessly with the LLaVA repo at present.
It is a great pleasure to participate in this open-source project.
Best
Haoning

Thank you! Sorry for the late response. They look good to me.
Only one thing, I will need to move the QBench and other benchmarks that are not initially in the LLaVA-1.5 paper to a different section, so that people won't get confused by the addtional commands. Thanks!


Convenience function for persisting dataframe to files endpoint

civis.io has a function file_to_dataframe, but no function going in the opposite direction. I find myself rewriting this simple function for myself a lot, and I thought it would make a nice little convenience for others. @keithing, would you be interested in including something like this in the civis.io function suite (I would make a PR)?

(this is essentially a thin wrapper around civis.io.dataframe_to_file, but I find it quite useful and I'm guessing other users would as well)

If I understand correctly, this is just a two liner:
df.to_csv(filename)
file_to_civis(filename)

But if you see the need for the function I'd be happy to review.

The most basic version of it is that two-liner, but it's nicer to wrap it in a temporary directory so that you don't end up with a bunch of local junk files. It's still only a few lines, but it's still convenient to have. It's also just nice to have the reverse functionality, since most of civis.io has functions for both i/o directions, and it's surprising as a user when that's not the case for files <-> pandas.


Any way to define the large image key for rich presence?

I'm trying to define a large image for the discord rich presence but no matter what I did my rich presence always shows up without an image. Any idea how I can go about adding in my own images for the rich presence?

Set app id here https://github.com/pypy-vrc/VRCX/blob/master/Discord.cs#L81
Add line here https://github.com/pypy-vrc/VRCX/blob/master/html/src/app.js#L5858 Discord.SetAssets('largeImageName', hoverText, 'smallImageName', 'hoverText);

Set app id here https://github.com/pypy-vrc/VRCX/blob/master/Discord.cs#L81
Add line here https://github.com/pypy-vrc/VRCX/blob/master/html/src/app.js#L5858 Discord.SetAssets('largeImageName', 'hoverText', 'smallImageName', 'hoverText);

Thank you so much! It works.


TypeError: Invalid URL: 'google.com'

my fresh site got many TypeErrors recently:
TypeError: Invalid URL: 'google.com'
    at getSerialization (ext:deno_url/00_url.js:89:11)
    at new URL (ext:deno_url/00_url.js:380:27)
    at handler (https://deno.land/x/fresh@1.6.1/src/server/context.ts:185:19)
    at https://deno.land/x/fresh@1.6.1/src/server/boot.ts:81:9
    at ext:deno_http/00_serve.js:448:24
    at ext:deno_http/00_serve.js:673:29
    at eventLoopTick (ext:core/01_core.js:183:11)
    at async bootServer (https://deno.land/x/fresh@1.6.1/src/server/boot.ts:78:5)
    at async startServer (https://deno.land/x/fresh@1.6.1/src/server/boot.ts:39:5)
    at async start (https://deno.land/x/fresh@1.6.1/src/server/mod.ts:115:3)

the site has no public domain, only private IP, I have no idea why google can access it.

That error is not the referrer but rather the actual URL that is sent to the Fresh instance. The part in question here that threw the error, runs just after receiving the Request object from deno through Deno.serve. The first thing we do in Fresh is construct a proper URL object out of it. That's were this fails because google.com is not a valid URL.
Do you have an idea who could be sending those invalid requests to your Fresh instance?

I get the same error from time to time in the production logs.
This has been happening for a long time.
It was 'google.com' for a long time but now it's 'www.sneakersnstuff.com'

Do you have an idea who could be sending those invalid requests to your Fresh instance?

I found that MS Edge browser may also send similar invalid requests. (...oh, maybe not invalid, i don't know.)
To reproduce the problem, just set <IP_ADDRESS>:8000 as Windows HTTP proxy, and visit any url in Edge.
TypeError: Invalid URL: 'nav-edge.smartscreen.microsoft.com'
    at getSerialization (ext:deno_url/00_url.js:91:11)
    at new URL (ext:deno_url/00_url.js:382:27)
    at handler (https://deno.land/x/fresh@1.6.1/src/server/context.ts:185:19)
    at https://deno.land/x/fresh@1.6.1/src/server/boot.ts:81:9
    at ext:deno_http/00_serve.js:459:24
    at ext:deno_http/00_serve.js:673:33
    at eventLoopTick (ext:core/01_core.js:178:11)
    at async bootServer (https://deno.land/x/fresh@1.6.1/src/server/boot.ts:78:5)
    at async startServer (https://deno.land/x/fresh@1.6.1/src/server/boot.ts:39:5)
    at async dev (https://deno.land/x/fresh@1.6.1/src/dev/dev_command.ts:58:5)
TypeError: Invalid URL: 'browser.events.data.msn.cn'
    at getSerialization (ext:deno_url/00_url.js:91:11)
    at new URL (ext:deno_url/00_url.js:382:27)
    at handler (https://deno.land/x/fresh@1.6.1/src/server/context.ts:185:19)
    at https://deno.land/x/fresh@1.6.1/src/server/boot.ts:81:9
    at ext:deno_http/00_serve.js:459:24
    at ext:deno_http/00_serve.js:673:33
    at eventLoopTick (ext:core/01_core.js:178:11)
    at async bootServer (https://deno.land/x/fresh@1.6.1/src/server/boot.ts:78:5)
    at async startServer (https://deno.land/x/fresh@1.6.1/src/server/boot.ts:39:5)
    at async dev (https://deno.land/x/fresh@1.6.1/src/dev/dev_command.ts:58:5)


Nice and ThrottleInterval equivalence in brew services

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
As plist has been replaced by service, I would like to know if there is any equivalent to the TrottleInterval and Nice plist keys.
I was not able to find anything in the formula cookbook or in ruby service docs
      <key>ThrottleInterval</key>
      <integer>30</integer>

and
      <key>Nice</key>
      <integer>-20</integer>

What is the motivation for the feature?
Be able to set priority for brew services and limit if needed. Matching the actual plist features.
How will the feature be relevant to at least 90% of Homebrew users?
This parameter can be set in any of the brew services if needed. The services onwers might want to lower priority to some of the homebrew services background tasks.
What alternatives to the feature have been considered?
Creating the plist manually as previously done before the deprecation but then deprecation warnings keep showing whenever running brew commands.

See man launchd.plist.
     ThrottleInterval <integer>
     This key lets one override the default throttling policy imposed on jobs
     by launchd.  The value is in seconds, and by default, jobs will not be
     spawned more than once every 10 seconds.  The principle behind this is
     that jobs should linger around just in case they are needed again in the
     near future. This not only reduces the latency of responses, but it
     encourages developers to amortize the cost of program invocation.

     Nice <integer>
     This optional key specifies what nice(3) value should be applied to the
     daemon.

Sorry, I misread the issue. I don't think this is implemented in the service DSL, but we'll review a pull request that does this.

Here is a PR defining Nice
https://github.com/Homebrew/brew/pull/14936
First time contributing so please let me know what I can improve

Note: I'm just coming by to see what might be the best way to integrate launchd for my own service. It seems that skhd and yabai have recently moved away from brew services ((https://github.com/koekeishiya/yabai/issues/1619#issuecomment-1507686308)), probably due to this issue...

This has stalled out. We'll still consider PRs but I don't think having this issue remain open is going to get more eyes here.


Duplicate record entries caused by RuleSet ref-- keys

Do you want to request a feature or report a bug?
Report a bug.
What is the current behavior?
Duplicate entries are added to records when RuleSet ref-- values are needed and the order of loader config changes. This causes the records file to balloon over time.
If the current behavior is a bug, please provide the steps to reproduce.
Please see https://github.com/rowanoulton/webpack-records-ruleset-ref-bug for an example.
What is the expected behavior?
Duplicate entries should not occur. Perhaps RecordIdsPlugin should strip ref-- values from the loader request to prevent this.
Reproduced on webpack 3.4 and 3.10.0.

webpack gives these option objects a unqiue ident, by default the position in the config. You can provide your own ident with this config:
module.rules: [
{ test: ...,
  use: [
    {
      loader: ...,
      options: ...,
      ident: "some unique id"
    }
  ]
}
]

@sokra Adding an ident property in that way throws a validation error:
Invalid configuration object. Webpack has been initialised using a configuration object that does not match the API schema.
 - configuration.module.rules[0].use should be one of these:
   non-empty string | function | object { loader?, options?, query? } | function | [non-empty string | function | object { loader?, options?, query? }]
   -> Modifiers applied to the module when rule is matched
   Details:
    * configuration.module.rules[0].use should be a string.
    * configuration.module.rules[0].use should be an instance of function
    * configuration.module.rules[0].use should be an object.
    * configuration.module.rules[0].use should be an instance of function
    * configuration.module.rules[0].use[0] should be a string.
    * configuration.module.rules[0].use[0] should be an instance of function
    * configuration.module.rules[0].use[0] has an unknown property 'ident'. These properties are valid:
      object { loader?, options?, query? }

Is it possible that this schema needs to be updated to include the ident property?

I have experimented with the change to schema and it works. Submitted #6089 above for review.

great


Update moderation.go

Update moderation.go as mentioned in https://github.com/sashabaranov/go-gpt3/issues/27#issuecomment-1202149781.

Awesome, thank you for the PR! 🔥