[Question] Code inection like flavours

What version of tinty are you using?
tinty 0.22.0
Is there any plans to implement flavours option to inject code in a file, like
[[item]]
    file = '~/.config/starship.toml'
    template = 'starship'
    rewrite = false
    start = "### Here starts Flavours colors"
    end = "### Here ends Flavours colors"

I still use flavours to inject code to conf files that does not support imports, like for starship and swaylock.
I use tinty to generate schemes from wallpaper, and I have this flavours item in my tinty config file:
[[items]]
name = "base16-flavours"
path = "~/.local/share/tinted-theming/tinty/repos/base16-flavours"
themes-dir = "colors"
hook = "cp -r %f ~/.local/share/flavours/base16/schemes/generated/"
supported-systems = ["base16"]

then use flavours to apply the generated scheme to starship for example
Here is a script I use to to choose a wallpaper and apply generated scheme:
#!/usr/bin/env bash

currwall="$(xdg-user-dir PICTURES)/current-wallpaper"
blurwall="$(xdg-user-dir PICTURES)/blured-wallpaper"
# basedir="path/to/my/wallpapers/directory"
basedir="/usr/share/wallpapers"

function set_bg() {
    if [ "$XDG_CURRENT_DESKTOP" == "sway" ]; then
        swaymsg output "*" background "$currwall" fill
    elif ["$XDG_CURRENT_DESKTOP" == "Hyprland" ]; then
        hyprctl hyprpaper preload "$currwall"
    else
        if command -v swaybg; then
            swaybg -i "$1" &
        elif command -v hyprpaper; then
            hyperpaper "$1"
        else
            exit 2
        fi
    fi

}

image=$(fd -tf --absolute-path --base-directory "$basedir" | fzf --preview 'fzf-preview.sh {}')

if [ -f "$image" ]; then
#     tintyname=$(echo $image | awk '{l=split($0, A, /\//); gsub(/[\. ]/, "-", A[l]); print A[l] }')

    tintyname=$(basename $image | awk '{ gsub(/[\. ]/, "-"); print }')

    rm "$currwall"
    cp "$image" "$currwall"

    set_bg "$image"

    base_sys="base16"
    if [ ! -z "$1" -a "$1" == "base24" ]; then
        base_sys="$1"
    fi

    tinty generate-scheme --name $tintyname --slug $tintyname --system $base_sys --variant dark --save  $image

    tinty apply "$base_sys-$tintyname"

    # for apps that I couldn't theme with tinty
    flavours apply base16-flavours-colors-file

    accentcolor=$(grep base0E ~/.local/share/tinted-theming/tinty/custom-schemes/base16/$tintyname.yaml | awk '{print $2}' | sd "'" "")

#     plasma-apply-colorscheme -a "#$accentcolor" BreezeDark
#     plasma-apply-lookandfeel -a org.kde.breezedark.desktop

    magick "$image" -blur "50x30" "$blurwall"

    so-bg-locker.sh "image" "$currwall"

#     echo "window { background: url('$blurwall'); background-size: cover;}" > ~/.config/wleave/background.css

    exit 0

else
    exit 1
fi

I didn’t have plans to add it but I do see the value. I’ve tended towards doing things in the “hook” property via a shell script where possible to keep things simple, however I do see the value in having this built in.
Do you have any thoughts or suggestions on the api/settings properties for this or feel that copying the Flavours implementation is the best way to add to Tinty?

I'm not sure about the best way to implement, but the flavours is way is tested and working.
But in the end it all depends on tinty and you maintainers in how to implement it if there are any plans to. (I don't have experience in developing and maintaining a project)

Would something like this work for you with tinty and allow you to stop using flavours?
[[items]]
path = "~/projects/tinted-theming/tinted-alacritty"
name = "tinted-alacritty"
hook = """
  # Delete lines between the comments, keeping the comments intact
  sed -i "/^# TINTY START$/,/^# TINTY END$/{//!d}" ~/.config/alacritty/colors.toml

  # Insert the contents of %f (filepath) after # TINTY START
  sed -i "/^# TINTY START$/r %f" ~/.config/alacritty/colors.toml
fi
"""
themes-dir = "colors-256"
supported-systems = ["base16", "base24"]

It didn't work,
I use Flavours to theme starship, swaylock and satty, but there were no modifications to their config files when using it.
Tried sed -i "/^# TINTY START$/,/^# TINTY END$/{//!d}" ~/.config/starship.toml alone and got: bash: !d}: event not found
I'm not familiar with sed, so sorry about this

Hmm I think bash is trying to expand the !. Can you try that specific command with single quotes? sed -i '/^# TINTY START$/,/^# TINTY END$/{//!d}' ~/.config/starship.toml


Dependencies chaos fix; Add @babel/plugin-proposal-class-properties

This PR reverts deps from 6.0 version and freezes typescript at 4.7 (https://github.com/parcel-bundler/parcel/issues/8419) so now its successfully builds.
Add a .babelrc with plugin to fix https://github.com/SortableJS/react-sortablejs/issues/244

can release a new package


Update OBA version and dependencies

Unfortunately, this repository no longer builds as is, as mentioned in #23. It appears that the onebusaway-application-modules repository's expected dependencies are no longer compatible with the dependencies specified in this repository's Dockerfile.
Here's what we need to do:

[ ] Parameterize DB credentials
[ ] Move bundle building out of the image
[x] fix issues with loading the arrivals-and-departures-for-stop API endpoint
[x] Get ACTA web UI loading correctly
[x] Upgrade Maven image - probably to maven:3.8.6-openjdk-11
[x] Upgrade Tomcat image - probably to tomcat:8.5-jdk11-openjdk
[x] Modify the Dockerfile to pull the latest official release of the OBA server software (see below for WAR file URLs)
[x] Modify the project to use MySQL instead of Postgresql

User Story
As a technically sophisticated transit enthusiast, I want to have a simple, 'one click' method to run an OBA server, so that I can easily try out the software and see how it works.
Configuration Guidance
https://github.com/OneBusAway/onebusaway/wiki/Configuration-and-Deployment-Guide-for-v2.x

The remaining open tasks have been split into issues #26 and #27


Docs Update: Add SEO to each page

@it09 I did a site audit to review SEO.
Docusaurus by default installs plugins that help with SEO. I checked the sitemap and it looks great. It is also already submitted to Google. Overall, the site is good. It just needs more content and diagrams (which I will help create).
I only added Alt Tags for images. Added some H1s to improve page visibility, and added a meta tag in its config file. Please review the pull request here: https://github.com/bacalhau-project/docs.bacalhau.org/pull/197

@it09 I did a site audit to review SEO. Docusaurus by default installs plugins that help with SEO. I checked the sitemap and it looks great. It is also already submitted to Google. Overall, the site is good. It just needs more content and diagrams (which I will help create).
I only added Alt Tags for images. Added some H1s to improve page visibility, and added a meta tag in its config file. Please review the pull request here: bacalhau-project/docs.bacalhau.org#197

@yenyrubiano thanks for checking it, i submitted the sitemap to Google. Thank for the additional changes. I'll review them now


Update package config from mosquitto to libmosquitto

This fixes https://github.com/PerfectlySoft/Perfect-Mosquitto/issues/4.
mosquitto provides a default package configuration named libmosquitto.pc: https://github.com/eclipse/mosquitto/pull/213
It also looks like we no longer need to export the package config path:
$ export PKG_CONFIG_PATH="/usr/local/lib/pkgconfig:/usr/lib/pkgconfig"

SPM already searches for this path: https://github.com/apple/swift-package-manager/blob/d58d857c5049a6e0e27896fe9a21f00e42b16527/Sources/SPMUtility/PkgConfig.swift#L54-L57

@kjessup Any thoughts on this approach? I'd love to be able to merge this in and tag a release 🙂


Command line option for selecting audio codecs

This commit introduces optional new syntax for BT profiles selection.
Like for BT audio codecs, from now on it will be also possible to modify
the list of profiles enabled by default instead of overwriting it.

I've pushed revised version of this PR. Right now the -p and -c options work as follows: when using these options default profiles/codecs are not disabled. These options support "+" and "-" prefixes ("+" is not documented because it behaves as if there was no prefix). Codecs enabled by default: SBC, AAC.
Now I will have to sleep with that, so to speak, in order to think whether such change is OK. But I think it will stay like that (it will fit perfectly with configuration file which might be added in the future). Additionally, I will add new properties in the D-Bus API for querying enabled profiles and codecs (continuation of #437).
PS. I've managed to modify bash-completion script, so it completes -p and -c options correctly (with +/- prefixes).

I think that this feature mostly ready. The list of enabled codecs can be queried by bluealsa-cli status. E.g. when running bluealsa -p a2dp-sink -c -aac -c aptx-hd -c ldac the output of bluealsa-cli status will be as follows:
Service: org.bluealsa
Version: v3.1.0-84-g5bae03c
Adapters: hci0
Profiles:
 - A2DP-source : SBC aptX-HD LDAC
 - A2DP-sink   : SBC aptX-HD LDAC
 - HFP-AG      : CVSD mSBC
 - HSP-AG      : CVSD

@borine what do you think abut the output format? You've been using bluealsa-cli a lot with scripting, do you think that such output format will be usable?

what do you think abut the output format?

I think the leading hyphens are not strictly necessary in the Profiles list, it's perfectly readable and parseable without them - just the indentation is sufficient. But if they are there for some other reason then that's OK, the output is stll simple enough to parse.


tools/pd-ctl: fix misspell in README.md

Signed-off-by: Keyi Xie<EMAIL_ADDRESS>
What problem does this PR solve?
Fix misspell.
What is changed and how it works?
Make the misspell correct.
Check List
Tests

No code

This change is

Codecov Report

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

@@            Coverage Diff            @@
##             master    #1499   +/-   ##
=========================================
  Coverage          ?   67.32%
=========================================
  Files             ?      158
  Lines             ?    15459
  Branches          ?        0
=========================================
  Hits              ?    10407
  Misses            ?     4101
  Partials          ?      951

Continue to review full report at Codecov.

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


style(Button): border update to match Secondary button dimensions

closes #688
Changes

Use of a border and with a color matching each appearance's (primary/destructive/etc) background-color

@shindigira
From what I can tell from the screenshots the after shows the primary and secondary buttons as the same height. This is correct.


Slow Animations after waking from sleep or logout

Hyprland Version
v0.28.0
Bug or Regression?
Bug
Description
everything is working fine_until I make it sleep
-animations becomes laggy
-videos takes time to render
-Didn't test games but I am sure it will have low fps also after waking from sleep
tell me if you need any information about my system
intel i5 3470
ram 8gb
amd r5 240
hdd
I use xdg-desktop-portal-hyprland not -wlr
and this is my boot tasks
exec-once = /usr/lib64/polkit-kde-authentication-agent-1
exec-once = killall -q hyprpaper || hyprpaper # Set wallpaper
exec-once = waybar -c $HOME/.config/hypr/waybar/config.jsonc -s $HOME/.config/hypr/waybar/style.css
exec-once = dunst # start notification demon
exec-once = cliphist wipe
exec-once = wl-paste --watch cliphist store
exec-once=dbus-update-activation-environment --systemd WAYLAND_DISPLAY XDG_CURRENT_DESKTOP
exec-once=dbus-update-activation-environment DISPLAY I3SOCK SWAYSOCK WAYLAND_DISPLAY XDG_CURRENT_DESKTOP=Hyprland
exec-once=systemctl --user import-environment DISPLAY WAYLAND_DISPLAY XDG_CURRENT_DESKTOP
exec-once=sleep 3 && systemctl --user restart xdg-desktop-portal.service &

How to reproduce
maybe old amd card and intel cpu
or maybe I did something wrong with the portals
but both xdg-desktop-portal and  xdg-desktop-portal-hyprland are "active (running) "
Crash reports, logs, images, videos
No response

looks like an issue on your end. Possibly some power saving shenanigans? Did you test on e.g. sway? or a different machine?

hmm no I tested openbox kde gnome never had this issue
but I did make garuda linux Performance tweaks
like installing this tools
ananicy-cpp irqbalance dbus-broker preload
and changing this file
sudo nano /usr/lib/udev/rules.d/50-sata.rules
SATA Active Link Power Management
ACTION=="add", SUBSYSTEM=="scsi_host", KERNEL=="host*", ATTR{link_power_management_policy}="max_performance"
I'll make a timeshift backup
and try to revert this tweaks

all of the above likely ran on X11, and neither share a common base. Please test sway.

all of the above likely ran on X11, and neither share a common base. Please test sway.

Done Tested sway but didn't have that issue...but bro I have to say that Hyprland is better 100% than any other window manager

all of the above likely ran on X11, and neither share a common base. Please test sway.

It wasn't a hyprland issue...after updating with pacman -Syu
it updated packages like zen-kernel, arcolinux-hyprland-git, xorg-xwayland
I think one of these 3 was the issue
thanks for your help :+1:

Fixed
it's was a gpu waking up issue
making it on powersave or performance will fix it
install CoreCtrl
and make gpu on performance mode
then restart
add it to startup
cp /usr/share/applications/org.corectrl.corectrl.desktop ~/.config/autostart/org.corectrl.corectrl.desktop
don't ask for password at startup
pkaction --version

Polkit version < 0.106
Create the file /etc/polkit-1/localauthority/50-local.d/90-corectrl.pkla with the following contents:

[User permissions]
Identity=unix-group:your-user-group
Action=org.corectrl.*
ResultActive=yes

Replace //your-user-group// with your user group name.

Polkit version >= 0.106
Create the file /etc/polkit-1/rules.d/90-corectrl.rules with the following contents:

polkit.addRule(function(action, subject) {
    if ((action.id == "org.corectrl.helper.init" ||
         action.id == "org.corectrl.helperkiller.init") &&
        subject.local == true &&
        subject.active == true &&
        subject.isInGroup("your-user-group")) {
            return polkit.Result.YES;
    }
});

Replace //your-user-group// with your user group name.


Device names should be reported within all errors

As a user I am likely to be working with multiple devices as once, sometimes multiple instances of the same device class, and I would like to know which device has raised an error at any given time. This can sometimes be hard to divine from stack traces, especially if device errors are wrapped up in RunEngine errors, for example:
  File "/venv/lib/python3.11/site-packages/bluesky/plan_stubs.py", line 1360, in repeated_plan
    yield from ensure_generator(plan())
  File "/venv/lib/python3.11/site-packages/bluesky/plan_stubs.py", line 1217, in one_shot
    yield from take_reading(list(detectors))
  File "/venv/lib/python3.11/site-packages/bluesky/plan_stubs.py", line 1101, in trigger_and_read
    return (yield from rewindable_wrapper(inner_trigger_and_read(),
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/venv/lib/python3.11/site-packages/bluesky/preprocessors.py", line 729, in rewindable_wrapper
    return (yield from plan)
            ^^^^^^^^^^^^^^^
  File "/venv/lib/python3.11/site-packages/bluesky/plan_stubs.py", line 1075, in inner_trigger_and_read
    yield from wait(group=grp)
  File "/venv/lib/python3.11/site-packages/bluesky/plan_stubs.py", line 504, in wait
    return (yield Msg('wait', None, group=group, timeout=timeout))
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/venv/lib/python3.11/site-packages/bluesky/preprocessors.py", line 251, in msg_mutator
    _s = yield msg
         ^^^^^^^^^
  File "/venv/lib/python3.11/site-packages/bluesky/run_engine.py", line 2284, in _status_object_completed
    raise FailedStatus(ret) from exc
bluesky.utils.FailedStatus: <AsyncStatus, task: <coroutine object StandardDetector.trigger at 0x7fc02987f040>, errored: TypeError("TetrammController.arm() missing 1 required positional argument: 'exposure'")>

In this case I can at least work out that error is coming from a Tetramm thanks to the TetrammController, but I still don't know which Tetramm. There are other cases (where more generic classes are used) that are even less obvious.
Reference: https://github.com/DiamondLightSource/i22-bluesky/issues/35#issuecomment-2160417939
Acceptance Criteria

All errors raised from within device methods report the name of the device raising the error message
That are tests that prove this
There is some prominent documentation explaining this pattern and why it should be kept up

@stan-dot suggests there should be some utility function to generate error messsages

Thinking about it, the example I gave is quite hard, because the error is actually raised from the standard library, and the device is written outside of ophyd-async. Maybe we need to change something about RunEngine or Status error reporting?

FailedStatus should still print the traceback somewhere further up, did this not make it out?

Does that mean AsyncStatus.wrap would no longer be usable on free functions?

the device could be optional, right?

Does that mean AsyncStatus.wrap would no longer be usable on free functions?

The simplest implementation would make this only usable on bound methods:
Device = TypeVar("Device", bound=Device)

class AsyncStatus(AsyncStatusBase):
    @classmethod
    def wrap(cls: Type[AS], f: Callable[Concatenate[Device, P], Awaitable]) -> Callable[Concatenate[Device, P], AS]:
        """Wrap an async function in an AsyncStatus."""

        @functools.wraps(f)
        def wrap_f(self: Device, *args: P.args, **kwargs: P.kwargs) -> AS:
            return cls(f(self, *args, **kwargs), device=self)

        # type is actually functools._Wrapped[P, Awaitable, P, AS]
        # but functools._Wrapped is not necessarily available
        return cast(Callable[Concatenate[Device, P], AS], wrap_f)

Do we have any use case to support free functions?

The simplest implementation would make this only usable on bound methods

Actually, scratch that, we could say that if the first argument is a Device then use its name, that's close enough:
class AsyncStatus(AsyncStatusBase):
    @classmethod
    def wrap(cls: Type[AS], f: Callable[P, Awaitable]) -> Callable[P, AS]:
        """Wrap an async function in an AsyncStatus."""

        @functools.wraps(f)
        def wrap_f(*args: P.args, **kwargs: P.kwargs) -> AS:
            if args and isinstance(args[0], Device):
                name = args[0].name
            else:
                name = None
            return cls(f(*args, **kwargs), name=name)

        # type is actually functools._Wrapped[P, Awaitable, P, AS]
        # but functools._Wrapped is not necessarily available
        return cast(Callable[P, AS], wrap_f)

I don't think this will catch every error though, AsyncStatus.wrap is a convenience function that is used in some places but not all.


Update reference-aadsts-error-codes.md

document  AADSTS76026 publicly to be developer friendly Epic 2364602: [doc] document the error to our public doc AADSTS76026: The request has expired  https://identitydivision.visualstudio.com/Engineering/_workitems/edit/2364602

@rwike77
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@cfzhang6688  Thanks for the update.  #sign-off

Close/reopen to resolve validation warnings.


Make executable version of Mean.js with Enclose.js or other

Hello i try to make a executable version of my project Mean.JS with Enclose.js (http://enclosejs.com/), but i have a problem with the exports of MongoDB and Consolidate, i make a screenshot of this situation.

So there are two issues i have to resolve.

I have refactorize the require in MongoDB, the Enclose need a explicit path and here its loaded dynamically, but there are many methods.

[ 'commands/base_command'
, 'admin'
, 'collection'
, 'connection/read_preference'
, 'connection/connection'
, 'connection/server'
, 'connection/mongos'
, 'connection/repl_set/repl_set'
, 'mongo_client'
, 'cursor'
, 'db'
, 'mongo_client'
, 'gridfs/grid'
,	'gridfs/chunk'
, 'gridfs/gridstore'].forEach(function (path) {
var module = require('./' + path);
for (var i in module) {
exports[i] = module[i];
console.log(exports[i]);
}
});

And for other side in the exportation of modules in Consolidate give the error "Cannot find module -name module- from  path/consolidate/name_module", and when i check the "node-modules" of Consolidate, its empty. I dont know why...

Please use GFM to highlight your code properly.

Sorry i fix it the comment, still unresolved the issue.

Did you npm install? What is require('./' + path) coming out to be?


[Question]: Is log probability available in llm.achat?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I wonder whether log prob can be retrieved through llm.achat (see https://platform.openai.com/docs/api-reference/chat#chat-create-logprobs) for log prob in OpenAI API. If so, is there an example code?

@mw19930312 it already is
llm = OpenAI(..., logprobs=True, top_logprobs=20)

response = await llm.achat([ChatMessage(role='user', content='Hello!')])

print(response.logprobs)

20 is the highest openai lets you select.


Update http4s-blaze-client, ... to 0.21.27

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-dsl

from 0.21.22 to 0.21.27.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #400.


[Flake]: Integration tests timeout.

It seems like the default timeout in Bazel might be too short for the integration tests.
[1,543 / 1,545] 1 / 3 tests; Testing //google/cloud/spanner/integration_tests:spanner_client_database_admin_integration_test; 210s local ... (2 actions running)
[1,543 / 1,545] 1 / 3 tests; Testing //google/cloud/spanner/integration_tests:spanner_client_database_admin_integration_test; 242s local ... (2 actions running)
[1,543 / 1,545] 1 / 3 tests; Testing //google/cloud/spanner/integration_tests:spanner_client_database_admin_integration_test; 279s local ... (2 actions running)
TIMEOUT: //google/cloud/spanner/integration_tests:spanner_client_database_admin_integration_test (Summary)
      /h/.cache/bazel/_bazel_kbuilder/a7ecd8237744645c5d189c197108d6d2/execroot/com_github_googleapis_google_cloud_cpp_spanner/bazel-out/k8-fastbuild/testlogs/google/cloud/spanner/integration_tests/spanner_client_database_admin_integration_test/test.log
INFO: From Testing //google/cloud/spanner/integration_tests:spanner_client_database_admin_integration_test:
==================== Test output for //google/cloud/spanner/integration_tests:spanner_client_database_admin_integration_test:
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from DatabaseAdminClient
[ RUN      ] DatabaseAdminClient.DatabaseBasicCRUD
Terminated
-- Test timed out at 2019-08-08 17:45:53 UTC --
================================================================================

https://source.cloud.google.com/results/invocations/59b261ac-f97d-4857-8160-e479f799823c/targets/cloud-cpp%2Fspanner%2Fdocker%2Fintegration/log

Just had another one:
https://source.cloud.google.com/results/invocations/62bb8b05-c490-4d73-862c-d54a494c4d66/targets

And another one:
https://source.cloud.google.com/results/invocations/0dbcb3f9-248b-4e2f-aab5-31be69bab31c/targets

I increased the timeout values to about 10x the observed maximum time (in my workstation). Not planning any more changes for now.

Two more this morning:
https://source.cloud.google.com/results/invocations/e0ed566e-e248-40e6-a21f-e08f0ba19ba6
https://source.cloud.google.com/results/invocations/ae932cb5-7f42-443f-899e-5cbc1f38c4c6

Note that those last two ran for 900s before timing out.

I just got this on my workstation:
//google/cloud/spanner/integration_tests:spanner_client_database_admin_integration_test PASSED in 516.4s

so it looks like 900s is indeed too short as a timeout.  Curious that I got much smaller numbers when running multiple iterations yesterday (several in parallel too!)

And another one, at 1500s!
https://source.cloud.google.com/results/invocations/5971c719-24ae-46c8-908a-90832776c550

I do not have any further ideas for this one.  The CI builds load balance across more instances now, they may not be enough or there may be other problems.

Sorry, just meant to clear the "assigned" bit.

These continue to happen, mostly in the samples now:
https://source.cloud.google.com/results/invocations/32d4b539-2f68-48a2-ac6f-7b3afcf11ffb/targets

And another:
https://source.cloud.google.com/results/invocations/dc08b3e3-0fae-4a0b-acdf-1d0217e42d03

A couple more in #637
one thing I noted there was that the samples times out after ~15x their normal runtime, yet other integration tests in the same run had normal runtimes.  So, it seems like things just occasionally get "stuck" for some reason; it's not clear to me that any timeout would make them succeed.  So, we should try to find the root cause for that and possibly drop the timeout.

These have not happened for about 3 months.  It seems like load balancing the CI builds across multiple instances has fixed the problem.


Mono/C#: Fix InvalidCast because of attempt to unbox long as int

This one was missing in #39739. Should be the last one.

Thanks!


Topic internalStats/stats get failed: Topic not found

Describe the bug
Pulsar v2.8.2
I craete a topic by using https://pulsar.apache.org/admin/v2/persistent/{tenant}/{namespace}/{topic}, then use https://pulsar.apache.org/admin/v2/persistent/{tenant}/{namespace}/{topic}/internalStats and https://pulsar.apache.org/admin/v2/persistent/{tenant}/{namespace}/{topic}/stats to check it. It will be failed and return "404 "reason": "Topic not found"". But I can successfully request some time after topic is created, so why these two requests return this error when topic is just created? Under what circumstances can I get the correct result?

Could you try the Pulsar 2.8.3?

Could you try the Pulsar 2.8.3?

It still doesn't work. I also test it in v2.9.1 and get the same result. So can you reproduct it? By the way, I uesed pulsar cluster with 3 brokers.
Topic List Response:
[     "persistent://public/lxx-consumer-test/consumer-partition-0" ]
Topic internal stats Response:
{     "reason": "Topic not found" }

topic of consumer-partition-0 and consumer, which one is you use when you request for the topic?

Topic of consumer-partition-0 and consumer, which one is you use when you request for the topic?

Tnx, I have get the right response using  consumer-partition-0 .


Add rbac rules for leader election

Signed-off-by: sahare<EMAIL_ADDRESS>https://github.com/stolostron/backlog/issues/18424

/cherry-pick release-2.4


JS compilers ignore default values of optional parameters of external factories

Dart SDK 2.0.0-dev.60.0
@JS()
library js_bug;

import 'package:js/js.dart';

@JS()
@anonymous
class MyClass {
  external factory MyClass({String foo = 'fooValue', String bar});
}

@JS('console.log')
external void log(v);

void main() {
  log(new MyClass(bar: 'barValue'));
}

Compiled with DDC (fragment):
  main.main = function() {
    dart.global.console.log({bar: "barValue"});  // <---- foo is missing
  };

Compiled with Dart2JS (fragment):
d:function(){var z={bar:"barValue"} // <---- foo is missing
self.console.log(z)},

I think this blanket is not supported, since @JS() is supposed to be a direct mapping to JS-code, which obviously won't have Dart-specific default values (in fact they might have there own based on undefined, which this would break).
We should probably cover this in stuff that @jmesserly wants to specify for JS interop if so.

in fact they might have there own based on undefined, which this would break

Fair point. But defining non-external factories should be safe, right? This example shows the same result as the one above.
@JS()
@anonymous
class MyClass {
  external factory MyClass({String foo, String bar});

  factory MyClass.defaultFoo({String foo = 'fooValue', String bar}) =>
      new MyClass(foo: foo, bar: bar);
}

@JS('console.log')
external void log(v);

void main() {
  log(new MyClass.defaultFoo(bar: 'barValue'));
}

I don't think JS interop officially supports Dart-only code on JS members (it might work).

Could the compilers fail on default arguments in this context since they're not going to work as one would expected?


Add Senti_Lex Dataset

TODO:
Fix feature format issue
Create dataset_info.json file
Run pytests
Make Style

Better will be if you close this PR and make a fresh PR

Feel free to ping me if you also have questions about the dummy data

also it looks like this PR includes changes about dummy_data.zip files in the ./datasets//un_pc folder. Can you remove them ?

Thanks for all the advice @lhoestq. I've implemented the changes you kindly highlighted and have made sure the scripts pass all the test. I've also marked this as ready for review as I believe it's in a good place to be merged now.

Great suggestion I fixed the dummy data and the file paths.


Support relative aliasing

Needed this in a current project. Should fix #36

Unfortunately we can't merge this PR, as the tests are failing in CI. We're going to close this for the migration to https://github.com/rollup/plugins. Please feel free to re-open the PR there once migration is complete.


Support INSERT key in addition to SPACE for selecting items in ListView

See this request here:  https://github.com/PowerShell/GraphicalTools/issues/79#issuecomment-614232936

Multi selection is also non-intuitive - not being able to close on Enter, nor using Insert for marking lines is very annoying behavior :(

I've thought more about this and I think this is a bad idea. I do not know what existing UI framework uses INSERT to select items. INSERT feels very arbitrary to me. It has been a common idiom on Mac and Windows that space selects/deselects UI items.

It would really be a bad option in addition to the fact that this key has already been defined for the activation / deactivation of text insertion or overlay.

INSERT is by default used to select items in both "Midnight Commander" and "Far Commander", which are very popular console file managers for Linux and Windows respectively. Not sure what people are using on Mac these days, assume mc as well?

Shows you what I know.
I'm reopening this because the argument that folks used to those console tools might expect INS to select items is valid.
I'm curious: On MC and FC, what does INS do when in an edit field? Or on other controls like buttons and check boxes?
If someone wants to help contribute and submit a PR for this, we're open to that!

Thanks. I guess i just have too much muscle memory, keep hitting INSERT in console lists in file managers :) I've tried INS on my mc in both wsl and ubuntu vm, and I don't see it doing anything on edit fields or buttons. Neither bottom command nor edit box is reacting to INSERT.
I'll try to look into doing PR for that, will look around the code.


dynamic menu links and pages in laravel

Laravel Version: #.#.#
PHP Version:
Database Driver & Version:

Description:
how to make dyanamic  menu and link and then how can i make pages for the menu in laravel
Steps To Reproduce:

use the forums or slack channel to get your issue resolved


[DiagnosticsQoI] SR-3600: Better recovery for trying to name an initializer, deinitializer, or subscript

Add a diagnostic to remove the name of an initializer, deinitializer, or subscript. The identifier token is consumed and skipped to prevent the parser from emitting additional error messages.
Add tests to verify that the name is removed from initializers, deinitializers, and subscripts declared with a name.
https://bugs.swift.org/browse/SR-3600
class A {
    init x() {} // error: initializers cannot have a name
    deinit y() {} // error deinitializers cannot have a name
    subscript z() -> Int { return 0 } // error subscripts cannot have a name
}

@swift-ci please smoke test

@swift-ci Please smoke test

@CodaFi,
Would you mind reviewing this when you get a chance?
Thanks!

@swift-ci Please smoke test

@jrose-apple, @slavapestov,
I added a check for identifiers that are followed by a left paren, added some tests, and fixed the code style issues.

@swift-ci Please smoke test


Add banner module

See

https://github.com/jhipster/jhipster-lite/issues/4122

Please take a look to see if the package I chose for the class and files is okay.

Until remove the banner module from jhipster-lite it is going to be this way:

One last suggestion: you could add one the banner modules to the generated app (https://github.com/jhipster/jhipster-lite-extension/blob/main/tests-ci/generate.sh#L58). I guess to do that you'll need to also declare some pre-requisites like spring-boot.

Done! Great suggestion! 🙌


NIFI-210: Add ExecuteScript and InvokeScriptProcessor

@mattyb149 wow this set of Processors is quite a feat! :) I provided feedback in the way of several inline comments. The only other thing that I would recommend is to consider factoring out a parent Abstract class or a util class. It seems like both Processors have quite a lot of overlap that could be simplified with either one of those approaches. Very cool to get these Processors in though! THanks for all of the work that you've put into getting this done!

TL;DR Will take a fresh look at this :)
I wanted very badly to have a parent Abstract base class, but the two processors extend from different parent classes (AbstractProcessor and AbstractSessionFactoryProcessor). Now that the behavior of ExecuteScript has changed to let the script handle the session and transfers, perhaps I can refactor that, at which point I will certainly pull the common code out. I thought about a Util class too but the methods looked like they'd need large signatures to get all the dependencies injected.


Update index.md

This unique tutorial on Acceptance Testing will give you a complete overview of the meaning, types, uses and various other factors involved in Acceptance Test in a simple and easy manner for your better understanding.

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Added a link to a tutorial


Icon colors

Icons colors are sometimes not easy to see on dark themes of github ! i use a stylish css script to darken github but that's only a suggestion

There's an optional style for the dark theme on GitHub. Do you turn it on?
https://github.com/xxhomey19/github-file-icon#turn-on-dark-mode


New PR to test SignCLA at 2018-03-02 14:02:18.805246076 -0800 PST m=+4.853303882

Body of PR

indiv-pass-staging: test succeeded


Refactor to kernel hooks & unify Kernel Execution

Codecov Report

Merging #11866 (e611bf6) into main (155604f) will decrease coverage by 0%.
The diff coverage is 80%.

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

@@          Coverage Diff           @@
##            main   #11866   +/-   ##
======================================
- Coverage     62%      62%   -1%
======================================
  Files        490      491    +1
  Lines      34771    34742   -29
  Branches    5652     5651    -1
======================================
- Hits       21873    21828   -45
+ Misses     10818    10814    -4
- Partials    2080     2100   +20

Impacted Files
Coverage Δ

src/kernels/types.ts
100% <ø> (ø)

src/kernels/kernel.ts
77% <64%> (-5%)
:arrow_down:

src/interactive-window/interactiveWindow.ts
75% <66%> (-1%)
:arrow_down:

src/kernels/kernelExecution.ts
89% <86%> (-8%)
:arrow_down:

src/kernels/execution/kernelCompletionPreWarmer.ts
93% <93%> (ø)

src/kernels/kernelStatusProvider.ts
97% <93%> (+7%)
:arrow_up:

...c/kernels/execution/cellExecutionMessageHandler.ts
78% <100%> (+<1%)
:arrow_up:

src/kernels/execution/helpers.ts
61% <100%> (-1%)
:arrow_down:

src/kernels/kernelAutoReConnectMonitor.ts
79% <100%> (ø)

src/kernels/kernelProvider.node.ts
100% <100%> (ø)

... and 46 more


INTERNAL | correcting WSSE logic

Description
Correcting the WSSE logic from this PR: https://github.com/usebruno/bruno/pull/2136
Contribution Checklist:

[ X] The pull request only addresses one issue or adds one feature.
[ X] The pull request does not introduce any breaking changes
[ ] I have added screenshots or gifs to help explain the change if applicable.
[ X] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request.

Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.

Merged! Thanks @dwolter-emarsys !


feat(apis_labels)!: drop apis_labels

This commit removes the apis_labels module:

It deletes the module
It removes the module from the INSTALLED_APPS list
It modifies the detail_generic and edit_generic views in
apis_entities, because they refer to the Label model
It removes the label related code from the merge_with method in the
TempEntityClass
It removes label related codes from the generic serializers
It removes a commented Label import from the metainfo model file
It deletes apis_relations.forms - this only contained the
EntityLabelForm for the Label model
It removes the LabelTableBase and LabelTableEdit classes from
apis_relations.tables
It removes the apis_relations.forms module from the form_module_list
in apis_relations.views, because the forms module does not exist
anymore
It removes the labels related stuff from the urls.py

Closes: #15

I'd like to move this forward, therefore adding the needs-attention label. Maybe everyone in @acdh-oeaw/prosnet could have a look if they use the labels and if so could think of a migration step.

We wait for this until we have a working approach to changes of models, probably using reversion

labels can be removed


Upgrade AOSP tag from r17 to r21

This patch will upgrade the AOSP tag for A14 to r21
Tests-Done: Build and boot android successfully.
Tracked-On: OAM-114686
Change-Id: Idfe3c1788ad54ff08dbcd2cc58dbb94fbb214c4c

Merged internally,
more details, logs, and binaries: /absp/builders/celadon-mergerequest/builds/3446


chore: include PULL_REQUEST_TEMPLATE.md

Status
READY
Breaking Changes
NO
Description
Aims to resolve #31 . Includes a PULL_REQUEST_TEMPLATE.md. The template looks like this PR.
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[X] 🗑 Chore

I agree that we should have a PR template, but I don't think we should introduce a new one, but try to be consistent in the organization and use the one we currently have in Flame.

I agree that we should have a PR template, but I don't think we should introduce a new one, but try to be consistent in the organization and use the one we currently have in Flame.

@spydon true, of course! I'm so used to seeing the template BloC uses that I didn't contemplate the Flame team had its own one 😅. I've used Flame's template instead now.

@spydon should we modify it to exclude "I have read the [Contributor Guide] and followed the process outlined for submitting PRs." from the checklist? Since there is none currently under this repo.
I also think the section: "Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process."
Should be commented out. As the above.

Sounds good, that PR should first be made to Flame also then.

I'll raise the PR to Flame to regarding commenting out that section.
On the other, hand I think the Contributor Guide checklist option makes sense since there is one on the Flame repo but not one in this repo.

This PR is currently blocked.
Waiting for this issue to be resolved first.


Update roadmaps.json

added https://pauljerimy.com/security-certification-roadmap/

@CBID2 @rupali-codes
Closing this as there is no issue linked.

@brodante
We are neglecting this now, but in the future, you need to create an issue and allow us to assign you the issue before you start working and submit a PR. Not following this process will require us to close your pull request.
This is outlined in the contributing guidelines.

You need to follow a predefined flow in order to contribute to Open Source. You may be new, so it is important to understand this. Most of the maintainers will mark it as spam and close it.


Best way to render differently each sections?

Let's say you have several sections for your autocomplete, but the rendering of each one is completely different. What would be the best way to provide different component depending on the section?
Right now the signature of renderSuggestion is function renderSuggestion(suggestion, { query }), so it means that your rendering needs to be based on the content of the suggestion to decide how to render. Is there another way to handle this?

I'd just add the section info to each suggestion, and use it in renderSuggestion, e.g.:
const suggestions = [
  {
    text: 'France',
    sectionType: 'country'
  },
  {
    text: 'Watermelon',
    sectionType: 'fruit'
  }
];

renderSuggestion(suggestion) {
  switch (suggestion.sectionType) {
    case 'country': return <Country name={suggestion.text} />;
    case 'fruit': return <Fruit name={suggestion.text} />;
  }
}

Does this help?

Alright so each suggestions should contain their sectionType.
Thanks!

I'd vote for reopening this issue, for the following reason: even though it's not too complicated to add sectionType to your data, when this data directly comes from an API call result, it forces you to enrich that data if you need to know which section a suggestion belongs to.
For instance, I have to do this:
getSectionSuggestions={section => (hits) => hits.map(h => {
        h.section = section;
        return h;
})}

Which is a bit cumbersome, when the signature of renderSuggestion could simply be function renderSuggestion(suggestion, { query, section }).
Would that be a change you'd consider?
Thanks!

If I may add, the same issue is found with renderSuggestionsContainer: there's no information about which section the container should be rendered for, which makes it hard to customize a specific section.

@olance OK, this makes sense!
Would you like to contribute a PR? (Note it will conflict with https://github.com/moroshko/react-autosuggest/pull/384)

I'm not sure I understand the issue with renderSuggestionsContainer. The children you get in this function are already rendered using renderSuggestion. So, if we add section to renderSuggestion, wouldn't this be enough?

@moroshko Yeah, actually I misunderstood the purpose of the function... it might be nice to have a renderSectionContainer callback as well when you need to customise specific sections... but that's another story! :)
I'm open to sending a PR but I'm short of bandwidth at the moment 😕
I'll note that down for later!

can someone help me how can I render unique elements in render sugestions?
consider default example and suppose there are two languages c in array?
how can we render only on 'c' if there are multiples?

+1 for a new renderSectionContainer (which has the responsibility to render the children, like renderSuggestionsContainer).
@moroshko Would you accept a PR for this principally?

@ulrichb Have been hacking around this for a while now. If @moroshko is open for that change I can try and help with a PR.
My (not sufficient) workaround was to do some CSS-foo and leverage the sectionContainerFirst theme element to apply a different structure to the first section. I then made sure my "specially rendered" section is always first. when it's missing, I omit this prop from the theme manually before rendering. yeah, I know... I have now two specially rendered sections. Busted.
Hopefully, I miss something and there a simpler way.

PR that adds renderSectionContainer: https://github.com/moroshko/react-autosuggest/pull/789


Automattic - just some minor copy edits / updates :)

[X] This pull request adheres to the repository's Code of Conduct
[X] This PR contains housekeeping (URL edits, copy changes etc)
[X] I am an employee of the company mentioned and confirm all details are correct

Cheers for this @codebykat! I should've updated it myself as I'm on a8c trial right now ;)

@dougaitken O RLY


Move checks to model

Moved checks to model
Solved too-many-public-methods for Pylint

Codecov Report

Merging #76 (155c888) into main (180b005) will decrease coverage by 0.51%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
- Coverage   98.19%   97.67%   -0.52%
==========================================
  Files           7        7
  Lines         387      387
==========================================
- Hits          380      378       -2
- Misses          7        9       +2

Impacted Files
Coverage Δ

service/routes.py
97.63% <ø> (-1.60%)
:arrow_down:

service/models.py
98.70% <100.00%> (+0.03%)
:arrow_up:

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


default to git context

follow-up https://github.com/docker/bake-action/pull/181

I put this one back in draft while we wait for auth support before defaulting: https://github.com/docker/buildx/pull/2363


version string in the docs looks wrong

Looks right on my local machine (shows 0.1.0a3), but the CI docs build shows the following:

This is a persistent problem I've had with RTD - never figured out how to fix it.

Oops, this isn't rtd!
This looks like a "shallow" clone, and the tag info isn't available for setuptools_scm.

I think this is just showing a version string when the commit was merged into main. I'm tagging the release after this, and there's probably no CI trigger for tags. But maybe you're right, and something else is going on too. :shrug:

I can't remember which repo it was on, but I definitely hit this issue somewhere. The fix was simple, I think it was just setting the depth to 0 or something.

Yep, #228 did the trick, thanks @jeromekelleher!


Add support for binary_sensor

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
I just discovered your card and I really like it! I hope to replace my custom button-card templates with your card instead. I would like support for binary_sensor, such as motion sensor or door sensors. It could show state and last changed. I made one using button card templates but doesn't work well with themes. See image.
Describe alternatives you've considered
I've used button-card with my own CSS templates but I hard-coded the colors and so it doesn't work well with themes.
Additional context

I am waiting for it!


React-Router and Flask-Admin

I have a setup with ReactJS where the routing is handled client sided by React-Router.
Flask-Admin sets up routes prefixed by '/admin'. However, when trying to use these together, the React-Router is clobbering the calls to Flask-Admin.
Is there a way to selectively excuse certain routes from React-Router's purview?

<Link>s are only for links within your application. Use an anchor (<a>) for anything else, which includes any links to locations on the same server that shouldn't be handled by the router.


Production Environment Javascript Optimizations

We are serving our javascript files and dependencies in production without proper compression/minimization.

additional background from picoCTF-platform-2 as a part of moving that project towards retirement.
Originally reported by @dbrumley:
bootstrap and jquery should be served locally, unless at deployment their is a bandwidth issue. The reason is you cannot do development or deployment without internet access.
At the very least we should modify it such that the development environment serves them locally if not the production environment. The Jekyll configuration already does something similar for analytics.


OBS build support

This is a very much WIP support for building rpms in the open build service. I have so far tested it only on packit via packit build in-obs --targets fedora-rawhide-x86_64,fedora-rawhide-aarch64,opensuse-tumbleweed-x86_64,opensuse-tumbleweed-aarch64 which created this package: https://build.opensuse.org/package/show/home:dancermak:packit/packit (it is unresolvable because my obs wrapper is not packaged).
TODO:

[ ] Write new tests or update the old ones to cover new functionality.
[ ] Update doc-strings where appropriate.
[ ] Update or write new documentation in packit/packit.dev.
[x] Implement waiting for package builds
[x] Find the right obs abstraction library (currently it's using my wrapper py-obs but that one is not packaged anywhere, however osc is rather inconvenient to use as a lib). cc @dmach
[ ] probably much more

RELEASE NOTES BEGIN
Packit now supports building packages in the open build service
RELEASE NOTES END

To hook to the packit-service, I think this part needs to be abstracted.

Oh yeah, you will not be able to build in copr because there is no internet access and all of the tests go through testing-farm :worried:. That is a conundrum. Just to run the tests, maybe you can add that package here with a temporary commit.

Please don't spend too much time on reviewing this yet, I will migrate away from using py-obs to osc, but that will change the looks of the code drastically.

@majamassarini I think this is worth taking a look at when working on https://github.com/packit/packit/issues/2099
Here, there is yet another secondary path we are working with (same as we clone dist-git with Fedora-based jobs.)
We should maybe revisit how we handle the jobs with multiple paths involved.

Here, there is yet another secondary path we are working with (same as we clone dist-git with Fedora-based jobs.) We should maybe revisit how we handle the jobs with multiple paths involved.

I am not sure I understand what you mean for multiple paths.
But I can tell you that I see it in this way.
In the OBS use case, I think that packit needs a PathEntry to initialize the api.dg object (and probably the name is wrong here?), instead in the #2099 use case I have to find a way to iniziale the api.up object.
Our working directories are usually wrapped around a LocalProject instance, both dg and up have one LocalProject reference. So we should teach our LocalProject instance how it can obtain a project from the OBS service and in a similar way, for #2099, we should teach the LocalProject how to get the package sources from the Specfile Source directive.
The problem is that LocalProject thinks that the working dir is a Git repo, and in both cases we should abstract somehow the differences. In the OBS code we should go through the osc.core object, if I get it correctly, and in the #2099 use case we should probably do much more hacks.
This should  allow us to use the packit jobs, almost as they are, with this new resources.
I don't know if this is understandable and you agree with me?

Thanks for taking a look @majamassarini ! You are right that it's getting really complicated since it touches the basic design decisions from the early days when we thought we should always work with two git repositories -- one representing upstream (or source-git) and one downstream.
I agree we can avoid a huge architecture redesign for now and fix those two use cases separately. (As Dan is doing.)

@majamassarini @lachmanfrantisek I am not sure if I understood #2099, but afaik there is no project out there that is actually using OBS as its upstream version control system (except for a few obs specific services). Literally everyone else under the sun is nowadays using git or svn or hg, but not obs. Hence I do not think that this is a path worth exploring, as the 3 projects actually being upstream on OBS are not worth the investment.
Additionally, OBS' version control system is very peculiar. There are no branches, subdirectories are not supported and then there is the highly confusing concept of _link files, which are a "floating merge" (your current repo is merged with the HEAD of another repo, i.e. the current HEAD of your repo is not constant and can change in time :zany_face:). I have worked with it longer than I would like to admit and probably longer than is useful for my sanity. I'd therefore suggest to really touch the version control system as little as possible.

@majamassarini @lachmanfrantisek I am not sure if I understood #2099, but afaik there is no project out there that is actually using OBS as its upstream version control system (except for a few obs specific services). Literally everyone else under the sun is nowadays using git or svn or hg, but not obs. Hence I do not think that this is a path worth exploring, as the 3 projects actually being upstream on OBS are not worth the investment.

The pointer to #2099 is probably misleading. Am I wrong or your downstream metadata are taken from the OBS service?
If yes than what I wanted to say is that right now, the Packit code, when works with both upstream or downstream sources, wraps them around a class that is called LocalProject but this class for the moment is able to play just with plain git repositories. And to make this changes and the #2099 changes be more close to the Packit code style we should probably change the LocalProject behaviour. But this is a big work and I also think that for now your code looks good, we can always refactor it later :)

@majamassarini @lachmanfrantisek I am not sure if I understood #2099, but afaik there is no project out there that is actually using OBS as its upstream version control system (except for a few obs specific services). Literally everyone else under the sun is nowadays using git or svn or hg, but not obs. Hence I do not think that this is a path worth exploring, as the 3 projects actually being upstream on OBS are not worth the investment.

The pointer to #2099 is probably misleading. Am I wrong or your downstream metadata are taken from the OBS service?

Currently I do not take any metadata from OBS. This is just adding support for building in OBS, nothing more.

If yes than what I wanted to say is that right now, the Packit code, when works with both upstream or downstream sources, wraps them around a class that is called LocalProject but this class for the moment is able to play just with plain git repositories. And to make this changes and the #2099 changes be more close to the Packit code style we should probably change the LocalProject behaviour. But this is a big work and I also think that for now your code looks good, we can always refactor it later :)

Sounds good to me :+1:

@lachmanfrantisek I am not sure if I understood #2099,

Sorry for the confusion. Packit has historically worked with two git repositories -- one representing upstream and one downstream. And we rely on this concept across the codebase. From the functional perspective, the #2099 is completely different to what you are working on. My point was that both issues destroy the concept of upstream-git + downstream-git. I just wanted to mention this so we know this is yet another use case that does not fit the original design.

I have submitted osc to epel for CentOS 9, so I hope that the CI should be passing now. How should we proceed from here?


Atomic puppy fails for stream that doesn't start at 0

Let's say we have a stream that starts at 50 because we have deleted it once (so now new events start where the former stream left off). Assume atomic puppy has never been run before for that stream, it does not have a key in redis, it will assume that it should start looking at 0-20:
DEBUG:atomicpuppy.stream-reader@eventstore_reader/goods_in_receipt:No last read event, yielding events from this page http://<IP_ADDRESS>:2113/streams/goods_in_receipt/0/forward/20

It will not find anything there since events start at 50.
A workaround is to create the key manually in redis with the value 50:
<IP_ADDRESS>:6379> set urn:atomicpuppy:INSTANCENAME:STREAMNAME:position 50
OK

I think the best option would be to navigate the links from the point where the counter is to "previous" until we find something. That would be the first event to process. Might be worth increasing the page size when doing this so that we can perform less http requests.

What is the value of the link with rel="first" in this case? If we have never seen a stream before, we should hit the entrypoint URI and then jump to first. If there are no events on first, but there is a next link, we should walk back up the links until we have processed all the events.
If the first link points to 0/forward/20 but there is no next link, then there's something wrong with the paging strategy.

Is this still an issue?


error installing stim under windows 10, python 3.8

tried installing using pip -install stim --user
got this error message :
building 'stim' extension
error: Microsoft Visual C++ 14.0 or greater is required. Get it with "Microsoft C++ Build Tools": https://visualstudio.microsoft.com/visual-cpp-build-tools/
I'll try to install these, but I prefer not to. I have mingwin installed already

I will add this to the "reasons to figure out proper prebuilding of wheels" pile.


API Remove configuration of _resources path

NOTE This will need to be targetted at 5 once that branch has been created (see https://github.com/silverstripe/silverstripe-framework/issues/10349)
BC breaking changes

Removed Module::getResourcesDir()
Removed ManifestFileFinder::RESOURCES_DIR (was already deprecated)

Parent issue

https://github.com/silverstripe/silverstripe-framework/issues/10399

Closed in favor of #10437


I got an error in PagerTabStripViewController.swift

When I run project, I got an error like this.
fatal error: Every view controller provided by PagerTabStripDataSource's viewControllersForPagerTabStrip method must conform to  InfoProvider: file /Users/elaneirz/Documents/FileXcode/XLPagerTabStrip2/Pods/XLPagerTabStrip/Sources/PagerTabStripViewController.swift, line 363
(lldb)
that is this method >
viewControllers.forEach { if !($0 is IndicatorInfoProvider) { fatalError("Every view controller provided by PagerTabStripDataSource's viewControllersForPagerTabStrip method must conform to  InfoProvider") }}
How to fix them?
thankyou

The view controllers you return in your viewControllersForPagerTabStrip must implement the IndicatorInfoProvider protocol, e.g.
extension YourViewController: IndicatorInfoProvider {
    func indicatorInfoForPagerTabStrip(pagerTabStripController: PagerTabStripViewController) -> IndicatorInfo {
        return itemInfo
    }
}

@ryantan : how to implement itemInfo ?

@nhatlee here's an example for a simple text based one:  var itemInfo = IndicatorInfo(title: "Your tab title here")

@ryantan got the same error after migration to swift 3. BUT runtime not compile.
Before migration swift 2.3 works
Podfile
pod 'XLPagerTabStrip', '~> 6.0'
Any idea?

The same error .
Some update?

same error on 5.1.0

Hi,
As it was previously mentioned, the childViewController (the ones returned in the method override func viewControllers(for pagerTabStripController: PagerTabStripViewController) -> [UIViewController]) MUST implement the IndicatorInfoProvider protocol. This is the way that each tab can know its title or icon.
The method it must implement is func indicatorInfo(for pagerTabStripController: PagerTabStripViewController) -> IndicatorInfo. This returns an IndicatorInfo which has several inits. See the documentation or the code for further information about it.
Hope it helps! Regards,

Hi i'm newbie ,
I have a button, how do I press this button to switch to another screen while keeping the menu?

Hey. did you manage to solve this problem? same prob in my project.

Make it helps somebody: I managed to fix the issue the following way (in my case only, of course):

go to storyboard
choose every child view controller
go to the tab you're setting custom class for it
verify the 'inherit module from target' checkbox is checked

When I run project, I got an error like this.
fatal error: Every view controller provided by PagerTabStripDataSource's viewControllersForPagerTabStrip method must conform to InfoProvider: file /Users/elaneirz/Documents/FileXcode/XLPagerTabStrip2/Pods/XLPagerTabStrip/Sources/PagerTabStripViewController.swift, line 363
(lldb)
that is this method >
viewControllers.forEach { if !($0 is IndicatorInfoProvider) { fatalError("Every view controller provided by PagerTabStripDataSource's viewControllersForPagerTabStrip method must conform to InfoProvider") }}
How to fix them?
thankyou

same error how to fix this??

Add it to the checklist:

Cross-check whether you set the StoryBoard ID or not.

If you don't set the StoryBoard ID, it throws the same error as listed above.


/index.js: Could not find module in path: 'react-dom/client' relative to '/index.js'

Bug report
My component library dependency is React 17, but after installing React 17, Sandpack keeps giving me the error "/index.js: Could not find module in path: 'react-dom/client' relative to '/index.js'", how to solve it?
Packages affected

[ ] sandpack-client
[ ✅] sandpack-react

Description of the problem
My component library dependency is React 17, but after installing React 17, Sandpack keeps giving me the error "/index.js: Could not find module in path: 'react-dom/client' relative to '/index.js'", how to solve it?
What were you doing when the problem occurred?
What steps can we take to reproduce the problem?

Link to sandbox: link (optional)
Your Environment

Software
Name/Version

Sandpack-client version

Sandpack-react version
1.19.0

Browser
chrome

Operating System
mac

@xiaohehennuli The problem you're running into is that createRoot was released in React 18. This is the default version of React that's installed when you use the react Sandpack template. Since Sandpack is expecting React 18, it's using the newer syntax in the /index.js file that's automatically created.
In your case, you're explicitly installing React 17 which doesn't have the createRoot export. This is causing the Sandpack /index.js file to throw that error you're seeing.
To get around this you can change the /index.js file that's passed to the Sandpack instance so that it uses the older syntax. Here's a Code Sandbox example that shows you how you can do that: https://codesandbox.io/s/condescending-nash-108pv1?file=/src/App.tsx.
I left some comments in the custom /index.js file that outlines what parts require React 18.

Thank you for your help. I have successfully solved the problem.


resourcemanager: communicate with the server

The server will live onsite in the short term, but we will want to have the option to push this to the cloud somewhere.
How can we communicate to the server and have the server communicate with the resource manager (which will live behind the firewall) with little to no config?
I feel like the address of the server will be known and static.  Maybe we poll the server every once and a while to see if it has updates for us

I thought about UPNP to auto configure a port on the firewall, but I'm not sure if it's worth opening that can of worms

I thought about UPNP to auto configure a port on the firewall, but I'm not sure if it's worth opening that can of worms

I'm leaning more toward mqtt now.
Seems like we can just spin up an mqtt broker and communicate with that.
It may require us to punch a hole in the firewall when running the server offsite though

we now have mqtt


Remove folder from dandiset details and use published dandiset data

Once the size field is available on the version model in the publish app, then the dandiset details can display data per version, instead of always being locked to girder.

We currently display the number of folders in the dandiset, however folders don't really exist in our published archive. To remain consistent, I think we should remove this display from the dandiset details entirely. If folders are supposed to represent individual subjects, then we should handle that explicitly, instead of referring to them as folders.

I think this is fixed; re-open if not, @AlmightyYakob.


vm.clone - Option to specify datastore

It would be nice to be able to specify the datastore that you want the new cloned vm to reside on. You can specify the host and resource pool but not the datastore and this is required for our setup to ensure that we do not over commit a datastore. RVC already has the ability to see each datastore utilization, which will provide the data I need to make the decision of what datastore to put the new vm on.

This would apply to both "vm.clone" and "vm.migrate".

I recently ran into the same issue and would like to see this taken care of too. Should be an easy change.

+1 from me. In fact I think this is vital.


Is dvc init target needed?

Do we really need target file in dvc init?
$ dvc init -h
usage: dvc init [-h] [-q] [-v] [-G] [--data-dir DATA_DIR]
                [--cache-dir CACHE_DIR] [--state-dir STATE_DIR]
                [--target-file TARGET_FILE]

I actually removed it from init in https://github.com/dataversioncontrol/dvc/pull/207 .
Target file, state dir and cache dir don't have to be configurable. We're safe by simply using .dvc/{target, state, cache}.
It might be even worth it to remove target file totally and use an entry in config file for that. Unless I'm missing some scenarios.

I.e. after #207, dvc init -h looks like:
`(dvc) ➜  dvc git:(master) ✗ dvc init -h
usage: dvc init [-h] [-q] [-v] [-G] [--data-dir DATA_DIR]
optional arguments:
-h, --help            show this help message and exit
-q, --quiet           Be quiet.
-v, --verbose         Be verbose.
-G, --no-git-actions  Skip all git actions including reproducibility check
and commits.
--data-dir DATA_DIR   Data directory.
`


bug with RFTools and EC3

the bug i have found involves RFtools storage module and Essential Craft 3
Horizons Daybreaker 1.1.0
Essentials Craft 3 4.6.1710.35
RFTools 3.11
https://www.youtube.com/watch?v=Ltp5En3Ae4Q

This is not a bug in RFTools. The modular storage block implements ISidedInventory and other mods should follow the restrictions of that api. I specifically return false for extraction attempts for the storage module slot. Other mods should under no circumstances extract items from that slot.


Angelfeliz angular2

Create ng2Client project, copy the same rules on web.config from web.config ngClient to web.config ng2Client.
Set-up the environment to work with angular2 in the ng2Client project.

skip some file did not have to be on the commit i will closing the pull request and fix it

@angelfeliz okay, just send the new one.
One detail, use forCrowd/publish as your target branch, not master.
I edit PRs on that branch, if it's okay, I'm publishing them to live and then merging that update with master.


Cannot read property 'onRunDescribeFinish' of null

Description
A minor issue where an error is raised during the cleanup stage of a test
It does not cause the test to fail so it's fine.
It affects running tests for both iOS and Android

[x] I have tested this issue on the latest Detox release and it still reproduces

Reproduction
Run any e2e test with the jest-circus runner
Using log, screenshot & video artifacts plugins
Expected behavior
The error should probably not be raised
Environment (please complete the following information):

Detox: 17.8.5
React Native: 0.61.5
Node: 14.5
Device: iOS Simulator / Android Emulator
Xcode: 11.7
iOS: 13
macOS: 10.15

Logs
Device and verbose Detox logs
There is no stack trace in the trace output for the error raised when calling onRunDescribeFinish
detox[12536] TRACE: [Artifact.js/DISCARD] discarding SimulatorVideoRecording
detox[12536] TRACE: [Artifact.js/STOP] stopping SimulatorLogRecording
detox[12536] DEBUG: [exec.js/KILL] sending SIGINT to [pid = 12932]: /usr/bin/xcrun simctl spawn 667D01B1-2BF7-47EA-8783-A5E16D64E9C2 log stream --level debug --style compact --predicate processImagePath beginsWith "/Users/robaldred/Library/Developer/CoreSimulator/Devices/667D01B1-2BF7-47EA-8783-A5E16D64E9C2/data/Containers/Bundle/Application/07B4E547-4522-4500-95C8-268E8DB56A46/Residently.app"
detox[12536] TRACE: [exec.js/SPAWN_END, #34] /usr/bin/xcrun simctl spawn 667D01B1-2BF7-47EA-8783-A5E16D64E9C2 log stream --level debug --style compact --predicate "processImagePath beginsWith \"/Users/robaldred/Library/Developer/CoreSimulator/Devices/667D01B1-2BF7-47EA-8783-A5E16D64E9C2/data/Containers/Bundle/Application/07B4E547-4522-4500-95C8-268E8DB56A46/Residently.app\"" finished with code = 0
detox[12536] INFO:  :ios: Receiving notifications > when the app is in the background > and there is a zendesk update without a ticket_id: should not open zendesk [OK]
and there is a zendesk update without a ticket_id[12536] TRACE: [ArtifactsManager.js/LIFECYCLE] artifactsManager.onRunDescribeFinish({ name: 'and there is a zendesk update without a ticket_id' })
when the app is in the background[12536] TRACE: [ArtifactsManager.js/LIFECYCLE] artifactsManager.onRunDescribeFinish({ name: 'when the app is in the background' })
:ios: Receiving notifications[12536] TRACE: [ArtifactsManager.js/LIFECYCLE] artifactsManager.onRunDescribeFinish({ name: ':ios: Receiving notifications' })

detox[12536] TRACE: [ArtifactsManager.js/LIFECYCLE] artifactsManager.onBeforeCleanup()
detox[12536] TRACE: [Artifact.js/SAVE] saving SimulatorLogRecording to: artifacts/ios.staging.2020-10-13 13-00-58Z/✓ _ios_ Receiving notifications when the app is in the background and there is a zendesk update without a ticket_id should not open zendesk/process.log
detox[12536] DEBUG: [SimulatorLogRecording.js/MOVE_FILE] moving "/private/var/folders/qs/40qhyqp14zx4r81lqty_cryh0000gn/T/5fd979a8-5343-4917-a3cc-323163e52e5f.detox.log" to artifacts/ios.staging.2020-10-13 13-00-58Z/✓ _ios_ Receiving notifications when the app is in the background and there is a zendesk update without a ticket_id should not open zendesk/process.log
detox[12536] TRACE: [Artifact.js/START] starting FileArtifact
detox[12536] TRACE: [Artifact.js/STOP] stopping FileArtifact
detox[12536] TRACE: [Artifact.js/SAVE] saving FileArtifact to: artifacts/ios.staging.2020-10-13 13-00-58Z/detox_pid_12536.json.log { append: true }
detox[12536] TRACE: [Artifact.js/START] starting FileArtifact
detox[12536] TRACE: [Artifact.js/STOP] stopping FileArtifact
detox[12536] TRACE: [Artifact.js/SAVE] saving FileArtifact to: artifacts/ios.staging.2020-10-13 13-00-58Z/detox_pid_12536.log { append: true }
detox[12536] DEBUG: [Artifact.js/MOVE_FILE] moving "/private/var/folders/qs/40qhyqp14zx4r81lqty_cryh0000gn/T/2586c279-8708-44fb-b339-76497cc34a37.detox.log" to artifacts/ios.staging.2020-10-13 13-00-58Z/detox_pid_12536.log
detox[12536] DEBUG: [Artifact.js/MOVE_FILE] moving "/private/var/folders/qs/40qhyqp14zx4r81lqty_cryh0000gn/T/c7c4ba59-85c8-445c-8fe1-75296687530a.detox.log" to artifacts/ios.staging.2020-10-13 13-00-58Z/detox_pid_12536.json.log
detox[12536] TRACE: [AsyncWebSocket.js/WEBSOCKET_SEND] {"type":"cleanup","params":{"stopRunner":true},"messageId":-49642}
detox[12536] TRACE: [DetoxServer.js/MESSAGE] role=tester action=cleanup (sessionId=89152887-8cf2-4158-a0ab-7c51b3d1e75b)
detox[12536] TRACE: [DetoxServer.js/MESSAGE] role=testee action=cleanupDone (sessionId=89152887-8cf2-4158-a0ab-7c51b3d1e75b)
detox[12536] TRACE: [AsyncWebSocket.js/WEBSOCKET_MESSAGE] {"messageId":-49642,"type":"cleanupDone","params":{}}

detox[12536] DEBUG: [exec.js/EXEC_CMD, #38] /usr/bin/xcrun simctl shutdown 667D01B1-2BF7-47EA-8783-A5E16D64E9C2
detox[12536] DEBUG: [exec.js/EXEC_TRY, #38] Shutting down 667D01B1-2BF7-47EA-8783-A5E16D64E9C2...
detox[12536] DEBUG: [DetoxServer.js/DISCONNECT] role=tester, sessionId=89152887-8cf2-4158-a0ab-7c51b3d1e75b
detox[12536] DEBUG: [DetoxServer.js/DISCONNECT] role=testee, sessionId=89152887-8cf2-4158-a0ab-7c51b3d1e75b
detox[12536] DEBUG: [DetoxServer.js/CANNOT_FORWARD] role=tester not connected, cannot fw action (sessionId=89152887-8cf2-4158-a0ab-7c51b3d1e75b)
detox[12536] TRACE: [exec.js/EXEC_SUCCESS, #38]
detox[12536] DEBUG: [exec.js/EXEC_SUCCESS, #38] 667D01B1-2BF7-47EA-8783-A5E16D64E9C2 shut down
detox[12536] DEBUG: [DetoxServer.js/WS_CLOSE] Detox server connections terminated gracefully
detox[12536] ERROR: TypeError: Cannot read property 'onRunDescribeFinish' of null

Well, it looks interesting and probably has to do with the cleanup phase, but the issue is lacking information.
How does the environment class look like, are there setupFilesAfterEnv in use, etc.?
I don't think I've ever seen this error, so any extra detail is crucial.

@noomorph Hey, I have updated the issue with our config files

@raldred, does it reproduce consistently? Locally, on CI, after npm reinstall, etc?

What versions of jest-* packages you have (jest, jest-circus)?
How does that e2e suite look like? I mean: :ios: Receiving notifications. Do you have some exotic describe statements, anything suspicious? Could you please share its content?
Add this debug statement and let me know if anything is printed there:

diff --git a/detox/runners/jest-circus/listeners/DetoxCoreListener.js b/detox/runners/jest-circus/listeners/DetoxCoreListener.js
index 4f52b3a2..e9778a08 100644
--- a/detox/runners/jest-circus/listeners/DetoxCoreListener.js
+++ b/detox/runners/jest-circus/listeners/DetoxCoreListener.js
@@ -37,6 +37,10 @@ class DetoxCoreListener {

   async run_describe_finish({describeBlock: {name, children}}) {
     if (children.length) {
+      if (!this.detox[onRunDescribeFinish]) {
+        console.log('DEBUG_DETOX', name);
+      }
+
       await this.detox[onRunDescribeFinish]({ name });
     }
   }

FWIW, I starting to get this error as I moved our config towards using the CustomDetoxEnvironment instead of relaying on init.ts. I fixed it by removing an extra call to await detox.cleanup() during afterAll (which you don't seem to have). The main difference I see in your config and mine is that my "behavior" looks like:
"behavior": {
  "init": {
    "launchApp": false
  },
   "cleanup": {
    "shutdownDevice": true
  }
},

@gustavos60 , hm......... not sure, but okay, I'll test locally with those flags.

I am also receiving this error at the moment, the error goes away if I remove the line await detox.cleanup(); from here:
beforeEach(async () => {
  await detox.device.reloadReactNative();
});

afterAll(async () => {
  await detox.cleanup();
});

Should we not be calling it anymore?

@dbilgin , Where is that your cleanup line located? In what file? Do you use custom detox circus environment?

awesome, thanks for the insight @noomorph!


Improve Doobie interface layer. It's ugly right now (particularly inserts)

This is mostly complete now. The only thing that feels clunky is the transition from a where/update builder to a doobie Builder[A]

Still need to decouple doobie from the PSql interpreter

Fixed as of 460999870c9918b4c4ff0f641165083e0634179c


update_s3_htmls is not running after the nightly build

🐛 Bug

Per title. The download link for those wheels won't be available until the next day.
cc @seemethere

Duplicate as https://github.com/pytorch/vision/issues/2348.


gitleaks: 8.11.0 -> 8.11.2

Description of changes
https://github.com/zricethezav/gitleaks/releases/tag/v8.11.2

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 188422 run on x86_64-linux 1

1 package built:

gitleaks

Result of nixpkgs-review pr 188422 run on x86_64-linux 1

1 package built:

gitleaks

This package has shell auto completion.


Anyone should be able to contribute

Right now we have a dropdown of clubs that people must choose from to contribute. As brought up by @paked in hackedu/hackedu#119, i'd be great if anyone could create new clubs as well.

Is there anything specifically that needs to be changed in order to make this happen? In my head all we have to do is change the dropdown to a textfield, but I don't know the codebase.

@zachlatta should we close this issue because we don't really use shipped right now?

How to do this issue:
Basically do what @paked suggested. Have an option for an "Other" club and make it so they can type in the name of their school.

Closing because we're deprecating the repo.


Items start jumping on beds too high

Timings link
Not needed
Expected behavior
Items just lay on the bed, only with vanilla rotation
Observed/Actual behavior
After 5-10 second items start to jump higher and higer. If u wait for a couple minutes, they will be jumping 30+ blocks height
Steps/models to reproduce

Drop any items on the bed
Wait 5-10 seconds
U can see how items start to jump higher and higher

Purpur version
This server is running Purpur version git-Purpur-1454 (MC: 1.18.1) (Implementing API version 1.18.1-R0.1-SNAPSHOT) (Git: 35bf26e on HEAD)
You are 2 version(s) behind
Download the new version at: https://purpurmc.org/downloads
Previous version: git-Purpur-1449 (MC: 1.18.1)
Agreements

[ ] I am running the latest version of Purpur available from https://purpurmc.org/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
It works without any plugins

Closing due to no timings report linked.


Add filetype to python.yaml

I propose to add the "filetype" lib

Please add the following dependency to the rosdep database.
Package name:
filetype
Package Upstream Source:
https://pypi.org/project/filetype/
Purpose of using this:
This dependency is being used to infer file type and MIME type checking the magic numbers signature of a file or buffer. This is why I think it's valuable to be added to the rosdep database.
Distro packaging links:
filetype-1.0.7-py2.py3-none-any.whl

Debian: https://packages.debian.org/

REQUIRED

Ubuntu: https://packages.ubuntu.com/

REQUIRED

Fedora: https://src.fedoraproject.org/browse/projects/

IF AVAILABLE

Arch: https://www.archlinux.org/packages/

IF AVAILABLE

Gentoo: https://packages.gentoo.org/

IF AVAILABLE

macOS: https://formulae.brew.sh/

IF AVAILABLE

Alpine: https://pkgs.alpinelinux.org/packages

IF AVAILABLE

NixOS/nixpkgs: https://search.nixos.org/packages

OPTIONAL

Checks

[x] All packages have a declared license in the package.xml
[x] This repository has a LICENSE file
[x] This package is expected to build on the submitted rosdistro

@bullerbeck friendly ping

There seems to be a formatting issue.


Ruby wrapper

It would be great to have a Ruby wrapper over paratext to bring fast CSV reading to Ruby.
We can use it in daru, for example.

I am not very familiar with the internals of Ruby, but I am happy to provide guidance for the effort.

Works :)
Should we use SWIG or it ok to use the Ruby C API directly? Any specific reasons why you used SWIG for the python bindings?

SWIG will likely make your life easier if you have straightforward type mappings. I may write a Cython binding for paratext (mainly to simplify talking to other Python modules that use Cython -- you can define an internal C API via pxd files) but either way it makes for less work than a hand-coded C extension for Python at least.

Alright. I'll explore all the options and post them here soon. Thanks :)

Hi.
I went with SWIG because it does a lot of the hard work of parsing and validating input for you. If you write your own wrapper, you have to write a lot of boilerplated code yourself. Given a large enough codebase (100K+ lines), custom wrappers become very hard to maintain.
The Python typemap lives here: https://github.com/wiseio/paratext/blob/master/src/python. The interface file (python.i) is pretty rudimentary. It converts std::vector<int/double/string> to NumPy vectors.
%typemap(out) std::vector<int> {
  $result = (PyObject*)::build_array<std::vector<int>>($1);
}

%typemap(out) std::vector<double> {
  $result = (PyObject*)::build_array<std::vector<double>>($1);
}

%typemap(out) std::vector<size_t> {
  $result = (PyObject*)::build_array<std::vector<size_t>>($1);
}

%typemap(out) const std::vector<std::string> & {
  { auto range = std::make_pair($1->begin(), $1->end());
   $result = (PyObject*)::build_array_from_range(range);
  }
}

%typemap(out) std::vector<std::string> {
  $result = (PyObject*)::build_array<std::vector<std::string>>($1);
}

%typemap(out) ParaText::CSV::ColBasedPopulator {
  $result = (PyObject*)::build_populator<ParaText::CSV::ColBasedPopulator>($1);
}

You need to write template functions to convert from C++ containers to Ruby arrays.
Ruby::Value build_array<ContainerType>(const ContainerType &container)

You also need to write a function to convert CSV columns to Ruby arrays:
Ruby::Value build_populator<ParaText::CSV::ColBasedPopulator>(ParaText::CSV::ColBasedPopulator &populator)

This is needed to call the loader.get_column(column_index) function.
Best,
Damian


Detect duplicate plugin dependencies

Should at least detect duplicate versions of bevy_egui, that one tends to cause issues for people on Discord.
Should probably be a lint?

I think this would be challenging to do, since it would require evaluating / inspecting the App after it is built. This should probably be changed upstream in bevy_app, if anything.

I think checking the Cargo.lock would a fairly good and easy proxy, wouldn't it?

Ah, great point! It may be difficult telling the difference between a "Bevy plugin crate" and a "normal library", but it certainly would be possible.

Maybe something that depends on Bevy can be classified as a Bevy crate? Wouldn't catch the crates that depend on bevy's subcrates, but the lint doesn't have to be perfect for a start.

True! We could also use package metadata, which would let plugin authors mark their crates as Bevy plugins.

Just as a note, I edited Jan's original message as part of my issue-pass in https://github.com/TheBevyFlock/bevy_cli/issues/34.


[WIP] Only publish coverage of one target framework

Pull request description
Coverage reports seem... a bit unstable (see https://github.com/NCronJob-Dev/NCronJob/pull/114#issuecomment-2450888724 for instance).
Currently the tests are run for both net8.0 and net9.0. As such two different coverage reports are submitted to coveralls.
Maybe this dual reporting is what generates the "instability".
This proposed change runs the tests once again, for the sole purpose of coverage analysis, but by only targeting net8.0.
PR meta checklist

[ ] Pull request is targeted at main branch for code
[ ] Pull request is linked to all related issues, if any.

Code PR specific checklist

[ ] My code follows the code style of this project and AspNetCore coding guidelines.
[ ] My change requires a change to the documentation.

[ ] I have updated the documentation accordingly.

[ ] I have updated the appropriate sub section in the CHANGELOG.md.
[ ] I have added, updated or removed tests to according to my changes.

[ ] All tests passed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.5%) to 84.104%

Files with Coverage Reduction
New Missed Lines
%

src/NCronJob/Scheduler/ObservablePriorityQueue.cs
1
91.38%

src/NCronJob/Scheduler/JobProcessor.cs
3
73.08%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.5%

Covered Lines:
1025

Relevant Lines:
1170

💛  - Coveralls

This seems to clear up some fog and (hopefully) should provide a less fluctuating baseline.
Before:

With this change:

Interesting. Until now we don't have any. net9.0 exclusive stuff (besides System.Threading.Lock). So let's roll with it


[WinReg] update to v2.2.2

Library name: WinReg
New version number: 2.2.2
Other information that may be useful (release notes, etc...)
winreg_vcpkg_files.zip

Thanks for posting this issue. This issue has been fixed. So I am closing this issue for now.

Thank you @LilyWangL .


[5.5] limit some relation queries by 1 when eager loading

This is an attempt to solve a really old issue: https://github.com/laravel/framework/issues/13387
This PR adds a limit 1 to the end of the query while eager loading a HasOne or a BelongsTo relationship, the limit is applied when you query the relation already (e.g. ->product).

This introduced a bug if you do something like:
$persons = App\Person::with('address')->get();

Collection Before:
[
0 => ['name'=>'Person 1', 'address'=>App\Address],
1 => ['name'=>'Person 2', 'address'=>App\Address],
2 => ['name'=>'Person 2', 'address'=>App\Address],
]

Collection After:
[
0 => ['name'=>'Person 1', 'address'=>App\Address],
1 => ['name'=>'Person 2', 'address'=>null],
2 => ['name'=>'Person 2', 'address'=>null],
]

I hope this is clear enough, I can send a PR rolling back this PR

ping @themsaid

@jmarcher Thanks for the heads up, reverted in https://github.com/laravel/framework/commit/86abc0648b62b52967f8d2f4884bdca982f5f75c


How to repair tvguide.com

Guide
https://iptv-org.github.io/epg/guides/en/tvguide.com.xml
Description
It appears they changed the API URL from cmg to fandom
So by editing the tvguide.com.config.js file and changing the URL to https://fandom-prod.apigee.net/ it started working again
Please change this in your online version as well
Take Care
Ben

hello
i have these, I wonder how it works help please , i used the panel xtream codec
https://iptv-org.github.io/epg/guides/tr/digiturk.com.tr.xml
https://iptv-org.github.io/epg/guides/tr/dsmart.com.tr.xml
https://iptv-org.github.io/epg/guides/tr/tvplus.com.tr.xml

hello
i have these, I wonder how it works help please , i used the panel xtream codec
https://iptv-org.github.io/epg/guides/tr/digiturk.com.tr.xml
https://iptv-org.github.io/epg/guides/tr/dsmart.com.tr.xml
https://iptv-org.github.io/epg/guides/tr/tvplus.com.tr.xml

It looks like these files were no longer updated/created
digiturk seems broken
smart looks ok, but old file (13/02)
tvplus looks ok, but old file (13/02)
This is probably also because of the CI missing problem, the crew on this site are looking for a solution
For now, you can use another site maybe? (I recommend bevy.be)
Take Care
Ben

hi bro
if you are a member, tell them to add , i would be very glad if you have any information ( bevy.be)
thanks thanks thanks

hi bro
if you are a member, tell them to add , i would be very glad if you have any information ( bevy.be)
thanks thanks thanks

done

all thanks bro

all thanks bro

ur welcome


Web: Don't use Monospace font by default

Monospace fonts like the one visible on https://zal.im/wasm/iv/ make the apps look unnecessarily raw and "unfinished":

Web (K/Wasm) - Canvas based API
Web (K/JS) - Canvas based API

While it's already possible to bundle a font and load it in Compose/Wasm, the default visual aesthetics would be much improved by using any sans-serif font available on the target system.

Doesn't Material use Roboto which I think is already a downloadable google font? Guessing this is just an issue of trying to use that font by downloading and setting it.
https://m3.material.io/styles/typography/fonts

The fix was merged. It  will be available in the next build of CfW with Kotlin/Wasm.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


SyntaxError: Unexpected token '?'

Description
Since release 1.18.0 with the lift plugin on every sls command I get following exception:
Node version 12.22.10
NPM version 6.14.16

How to Reproduce

Use Node version 12.22.10
Use NPM version 6.14.16
Install version 1.18.0 of lift plugin

Additional Information
My plugins code in serverless file
plugins:
  - ./vendor/bref/bref
  - ./vendor/bref/extra-php-extensions
  - serverless-lift

I'm having the same issue on version 1.17 and 1.19.

Hi both, it seems you are using Node 12? Lift supports Node 14 and up as Node 12 is now deprecated.
https://github.com/getlift/lift/blob/3ebe40ddb48799ea8edbf4576ac33b82256e5a39/package.json#L58-L60
Could you try upgrading Node?


updated dependencies + added details in readme

Cheers, it was helpful as a base for my own needs.
I had to update a few things

I'm glad that it has helped you. Thank you for your contribution!


Remove licenses from the sample code itself as they are throwing off bayesian results.

The PHP sample, for instance, seems to give very positive results to code that bears the MIT license.
Remove the license from the samples and possibly substitute more sample code.

Done


NBT API outdated - Warnings in Console

[20:43:03 WARN]: [NBTAPI] [NBTAPI] The NBT-API at 'package me.badbones69.crazycrates.nbtapi' seems to be outdated!
[20:43:03 WARN]: [NBTAPI] [NBTAPI] Current Version: '2.5.0' Newest Version: 2.6.0'
[20:43:03 WARN]: [NBTAPI] [NBTAPI] Please update the nbt-api or the plugin that contains the api!
We use Crazycrates v1.10.1
We already have NBTAPI 2.6.0 on our server, but still get the above every restart.

Update your dev build to the newest


feat: Adding DEV, PROD, TEST environment variables

I would like to be able to change the code of my application using env variables, eg change the port based on which mode the FOCA app is running or change the configuration etc. This might seem like a per project level requirement, but having this defined either in config.yaml with strict typing to standardize the variable name might be helpful.
For example being able to do something like:
# Development: Permissive CORS
if app.mode == APIMode.DEV:
    CORS(app, origins="*")

# Production: Strict CORS
if app.mode == APIMode.PROD:
    CORS(app, origins=["https://myapp.com"])

Maybe extra configs for those modes in config.yaml, with which we can overwrite some config fields.
environments:
  dev:
    server:
      debug: True
      port: 8080
      use_reloader: True

    db:
      host: localhost
      port: 27017

  prod:
    server:
      debug: False
      port: 80
      use_reloader: False

    log:
      root:
        level: 20
      handlers:
        console:
          level: 20

  staging:
    server:
      debug: False
      port: 8080

Do you want to have bifurcation on the basis of different config files or a single config file? While I understand the need to such particulars, we might need to think on adding such support from a more holistic perspective as to have a long term solution based on scope and requirements. I like the idea of have a dedicated config for prod staging etc, but do we want to think in some additional directions as well? As to have a boiler plate based on different config files or to move such variables and abstract them out? Any additional thoughts?


Light tutorial not working

Habitat-Sim version
v 0.1.7
🐛 Bug
Launching the lightning_tutorial I get
  File "examples/tutorials/lighting_tutorial.py", line 102, in main
    LightInfo(vector=[0.0, 2.0, 0.6, 0.0], model=LightPositionModel.Global)
AttributeError: type object 'habitat_sim._ext.habitat_sim_bindings.LightPositio' has no attribute 'Global'

Is this a unknown issue? It seems like a type in the C++ bindings
Steps to Reproduce
Steps to reproduce the behavior:

Using habitat 0.1.7 (installed with conda)
python examples/tutorials/lighting_tutorial.py
Please note that without a minimal working example to reproduce the bug, we may not be able to help you.

Expected behavior
I expect it to work as in https://aihabitat.org/docs/habitat-sim/lighting-setups.html

Additional context

This tutorial is working for me on master branch.
Did you install from an old conda build like stable v0.1.7 and then run the script on master? That could be the problem. You can try building master from source, conda installing the nightly build, or cloning the repo at v0.1.7 to get scripts compatible with that version.

This solved my issue @aclegg3 ! Thank you :)


ChefDK issue with aws-sdk gem

Hi folks,
I'm not sure if the issue should be opened here on in ChefDK repo.
I've set up two kitchen suites with logs here: https://github.com/scalp42/chefdk-aws-issue
The example is a simple recipe installing aws-sdk-kms v3 and module requiring aws-sdk-kms (but any other aws gem will break I'm pretty sure):
module Kitchen
  def self.test!
    require 'aws-sdk-kms'

    Chef::Log.info 'It works! YAY'
  end
end

If using the product chef, then everything will work as expected when the module method is called:
     Installing Cookbook Gems:
       Compiling Cookbooks...
       Recipe: chefdk-aws-issue::default
         * chef_gem[aws-sdk-kms] action install[2018-06-13T22:03:39+00:00] INFO: Processing chef_gem[aws-sdk-kms] action install (chefdk-aws-issue::default line 10)
       [2018-06-13T22:03:40+00:00] INFO: chef_gem[aws-sdk-kms] installed aws-sdk-kms at 1.5.0

           - install version 1.5.0 of package aws-sdk-kms
       [2018-06-13T22:03:41+00:00] INFO: It works! YAY
         Converging 1 resources
         * chef_gem[aws-sdk-kms] action install[2018-06-13T22:03:41+00:00] INFO: Processing chef_gem[aws-sdk-kms] action install (chefdk-aws-issue::default line 10)
        (up to date)
       [2018-06-13T22:03:41+00:00] INFO: Chef Run complete in 2.159324895 seconds

If using the product chefdk, then it'll error out with following error:
         Gem::ConflictError
         ------------------
         Unable to activate aws-sdk-kms-1.5.0, because aws-sdk-core-2.11.50 conflicts with aws-sdk-core (~> 3)

         Cookbook Trace:
         ---------------
           /var/tmp/kitchen/cache/cookbooks/chefdk-aws-issue/libraries/aws.rb:3:in `test!'
           /var/tmp/kitchen/cache/cookbooks/chefdk-aws-issue/recipes/default.rb:14:in `from_file'

         Relevant File Content:
         ----------------------
         /var/tmp/kitchen/cache/cookbooks/chefdk-aws-issue/libraries/aws.rb:

           1:  module Kitchen
           2:    def self.test!
           3>>     require 'aws-sdk-kms'
           4:
           5:      Chef::Log.info 'It works! YAY'
           6:    end
           7:  end
           8:

         System Info:
         ------------
         chef_version=14.1.12
         platform=ubuntu
         platform_version=16.04
         ruby=ruby 2.5.1p57 (2018-03-29 revision 63029) [x86_64-linux]
         program_name=/opt/chefdk/bin/chef-client
         executable=/opt/chefdk/bin/chef-client

The only difference on fresh install is that ChefDK has aws-sdk v2 already installed:
root@default-ubuntu-1604:~# /opt/chefdk/bin/chef --version
Chef Development Kit Version: 3.0.36
chef-client version: 14.1.12
delivery version: master (7206afaf4cf29a17d2144bb39c55b7212cfafcc7)
berks version: 7.0.2
kitchen version: 1.21.2
inspec version: 2.1.72

root@default-ubuntu-1604:~# /opt/chefdk/embedded/bin/gem list | grep 'aws-sdk'
aws-sdk (2.11.50)
aws-sdk-core (2.11.50)
aws-sdk-resources (2.11.50)

It may or may not be related to https://github.com/chef-cookbooks/aws/issues/323.
TLDR: when requiring aws-sdk-kms v3, Chef works fine but ChefDK breaks.
Thanks for the help in advance 🤕
cc @tas50

Yep we have v2 already installed due to chef-provisioning. We need to get that onto v3 so we can move inspec to v3 and then v3 will work.

Maybe we should not require the aws-sdk gem but instead depend only on the gems of cookbook-supported functions.  It seems the aws-sdk gem goes out and installs the whole encyclopedia of ruby gems for AWS services.
e.g.
gem 'aws-sdk-ec2'
gem 'aws-sdk-ssm'
...

This should be closed via #358 and released as 8.0.0


updated .gitignore

Signed-off-by: Mike Lloyd<EMAIL_ADDRESS>

Coverage remained the same at 87.642% when pulling db747c11a5db4de714606896dc5f46a67c8b4718 on 1.1 into 4b2d44257cab88bc109c5549e5905111823a3596 on master.


Saving Mask as PNG

Hello,
I am trying to save the mask created on Cellpose as PNG, but it only saves as TIF. Is there a way to convert to PNG? Thanks!

you can use the --save_png flag
if you are running cellpose in 3D it will only save as a TIF because PNG doesn't support multi-page images


Fix typo (quartile vs. percentile)

Quartile refers specifically to the four quantiles that split a dataset into quarters.
It doesn't make sense to talk about a 90th quartile - you can only have 1st, 2nd and 3rd quartiles.
Percentile is more accurate in this case. I think it's also more widely
used / commonly understood than "quartile", so maybe a bit easier to
understand.
Unlikely that this would confuse anyone in practice, but if it's going
to end up in an alert we should make it as clear as possible.

👍  I agree, percentile is better. I also spotted the same thing in the alert name, so I've force pushed an update. Sorry to make you review it again 😓