Indentation in lint.yml

I was having a problem when issuing the gulp command. I got the error You need to have Ruby and scss-lint gem installed .
I do have Ruby and scss-lint installed, and after a bit of googling I found this issue plugin not seeing Ruby or scss-lint gem? which points to the issue of indentation in the lint.yml. I edited lint.yml to use 2 spaces for indentation, and this seemed to fix the problem for me.

Good to know. I'll update the file!


microshift-containerized as a systemd service using host's cri-o

Signed-off-by: Parul<EMAIL_ADDRESS>
Which issue(s) this PR addresses:
This PR partially resolves #276
microshift-containerized is running microshift as a systemd service using podman and host's cri-o

This PR

systemd unit file for running microshift-containerized
mounts host's cri-o as a volume on microshift-containerized

@husky-parul I have a PR to update the generated unit file, you'll want to update this PR too. Otherwise upon machine restart the service fails due to container already exists with that name error. The --new flag was missing when we generated the unit file before.
Also, since podman generate systemd --new sets the ExecStart|Stop with the podman cmds, I modified the /usr/bin/microshift-script to only include the setsebool cmd,  recording of the kubeconfig path, & conditional volume create and it should run with the ExecPreStart. Take a look!

/assign @fzdarsky


devise.user_registrations key is not the same in the view

After doing a signup, the <%= flash_messages %> are giving me this error
translation missing: en-US.devise.user_registrations.spree_user.signed_up
When I look into your en.yml file it looks like the spree_user: key is not used https://github.com/solidusio/solidus_auth_devise/blob/master/config/locales/en.yml#L37-L41 but it is under user_passwords. Could it be a bug on your .yml file ? Thanks !

Seems fixed.


Type checking on unsaved files

There is one thing supported out of the box by VS Code that I really miss in Atom: the ability to cmd+N, then select the language of the document to TypeScript, and start hacking:

on Code, TypeScript will be already loaded, so you can see type errors
on Atom, no checks are performed

I have no idea if this is actually do-able with Atom's APIs, and I did not see any matching issue on this repo, but supporting this usecase would be awesome.

Hi. There are a couple challenges with the new files. For one, those don't have any meaningful file path. Atom-TypeScript actually checks if the editor has a well-defined file path (and it also checks file extension to avoid undue weirdness if TypeScript grammar was selected accidentally), since that file path is then used to relate the file to the correct project and whatnot. Consequently, I doubt unnamed files would work as well as we'd like even if I did disable this check.
One recommendation I can make though, try using Advanced Open File package for Atom. It allows you to not only create & save new files faster (well, -ish), but it also sets file path for newly-created files, which will work around this issue.


bugfix/issue 773/Fix Devnet and Testnet nodes connect to each other

Purpose
Fix devnet and testnet nodes connect to each other
For reviewers

Related Github issues
https://github.com/Taraxa-project/taraxa-node/issues/773
Changes
Change devnet tcp/udp to 10003 that's different with testnet 10002
Tests

Deployment Considerations
Version Notes
{ major | minor | patch }

NOTE: Include the version notes in the squash commit message

I don't understand how this solves anything?


Async Model Validation

Is your feature request related to a problem? Please describe.
Model validation is synchronous, but if there's a database call or other awaitable call, it would be nice to have an async option.
See https://github.com/aspnet/Mvc/issues/786.
Describe the solution you'd like
Add virtual methods for ValidationAttribute:
Task<bool> IsValidAsync(object value);
Task<ValidationResult> IsValidAsync(object value, ValidationContext validationContext);

Describe alternatives you've considered
Currently working around by using synchronous DB calls.

I second this featue Request, but not for the ValidationAttribute (because there is no direct DI for the Attribute and accessing a DbContext needs scoped services).
I have to lookup in the database, and would like to not look the thread ;)
For my perspective with the IAsyncEnumerable of C# 8 i should be possible to create an IAsyncModelValidator.
  public interface IAsyncModelValidator
  {
    IAsyncEnumerable<ModelValidationResult> ValidateAsync(ModelValidationContext context);
  }

Currently we use ValidationAttribute for validating commands (if the id of the command already exist in database for instance). Since the IsValid metods are synchronous we are seeing thread pool starvation under high load, since the call chain to db is blocked. So this feature would be highly appreciated.
Our temporary "work around" is to explicitly set the ThreadPool min size, which seem to work. Such as:
var threadCount = 1000;
ThreadPool.GetMaxThreads(out _, out var completionThreads);
ThreadPool.SetMinThreads(threadCount, completionThreads);

Someone "solved" it in another fashion?

1- just synchronous validation
2- if errors in one  in 1 then skip 3
3- asynchronous validations

Thanks for contacting us, @bradlis7.
This is not something we plan to do in the near future.


Add appmap to recommended extensions

Fixes #537
The first time that a user opens a map, add the AppMap extension to the list of recommended extensions for the current project in extensions.json in the .vscode folder.

Modifying or creating .vscode/extensions.json on start up without informing or asking the user seems a little too aggressive IMO. Personally, I'd be likely to throw out any changes to this file without even looking at the diff if I saw it was modified. It doesn't happen often (if at all), and if it's changed, it's likely on accident.
This is purely speculation, but I think if we were to prompt the user after they've received value (closed their first AppMap?) we'd have better results.

At least if the user discards the change, they know how to suggest this
extension to their team.
The work is done now, we must have done it for a reason?
On Tue, Jan 10, 2023 at 4:53 PM Dustin Byrne @.***>
wrote:

Modifying or creating .vscode/extensions.json on start up without
informing or asking the user seems a little too aggressive IMO. Personally,
I'd be likely to throw out any changes to this file without even looking at
the diff if I saw it was modified. It doesn't happen often (if at all), and
if it's changed, it's likely on accident.
This is purely speculation, but I think if we were to prompt the user
after they've received value (closed their first AppMap?) we'd have better
results.
—
Reply to this email directly, view it on GitHub
https://github.com/getappmap/vscode-appland/pull/586#issuecomment-1377939632,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAVC6YHBKV7MZUYENUTRWDWRXK5DANCNFSM6AAAAAATXI6XOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

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

v0.61.0
GitHub release

Your semantic-release bot :package::rocket:


[Android] (DocumentImeSerializer._imeToDocumentPosition) Couldn't map an IME position to a document position - Editor Freezes

We started to get feedback from our first major rollout of SE to android (we've been in production ios/mac os for upwards of a year now with no issues, something we are extremely grateful for, really incredible work).  This error is associated with the crash event, and it is the same base error reported in #1081.
There is common behavior reported throughout every user we have been able to individually contact tagged on this event: the editor stops taking input and it must be closed and then reopened in order to continue editing the document (in our case referred to as an "entry").
This is the average report:

After a short period, text will fail to appear as I'm typing. Fixed by coming out of the editor and going back in. But the problem will repeat resulting in having to go in and out several times to complete the entry.

So far, there doesn't seem to be any one specific action / repro that triggers the crash, but rather a myriad of various common editing actions:
- adding a bulleted list
The user adds a bullet list item and can no longer advance to the next line:
https://github.com/superlistapp/super_editor/assets/37011672/f6415536-b2b0-40c6-9baf-da56b45750f5
- in some cases, adding a new line
The user types "Happy", applies rich-text formatting (header), and then moves to the next line:
https://github.com/superlistapp/super_editor/assets/37011672/35526d0c-718f-401b-8f74-0023444e47fa
- special characters associated with non-English languages

Today I noticed a pattern. The editor freezed after writing "ão" (common in Portuguese words) It happened 2 times and in each of them I had to checkmark, close and open again to continue writing.

- after autocorrection of a typo
The user misspells "waves" (~1:17) and the word is auto-corrected, then the editor freezes.
https://github.com/superlistapp/super_editor/assets/37011672/db03b42c-df0e-41bc-ba1a-a021a4334d70
- manually accepting keyboard text suggestions
The user accepts text suggestion of the word "anonymous"
https://github.com/superlistapp/super_editor/assets/37011672/f7a572aa-8000-4d34-a61b-05fff5bde8ee
The user accepts text suggestion of the word "issues" (~:25)
https://github.com/superlistapp/super_editor/assets/37011672/c13e7a4a-c159-4b28-9ac4-e340772ab565
Below are logs of six separate variants of the same error in order of most to least common:
Variant 1
Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError
Exception: Couldn't map an IME position to a document position. TextEditingValue: '[redacted]' IME position: TextPosition(offset: 251, affinity: TextAffinity.downstream). Error thrown Instance of 'ErrorDescription'.
DocumentImeSerializer._imeToDocumentPosition (document_serialization.dart:271)
DocumentImeSerializer.imeToDocumentSelection (document_serialization.dart:164)
TextDeltasDocumentEditor._applyInsertion (document_delta_editing.dart:133)
TextDeltasDocumentEditor.applyDeltas (document_delta_editing.dart:69)
DocumentImeInputClient.updateEditingValueWithDeltas (document_ime_communication.dart:215)
DeltaTextInputClientDecorator.updateEditingValueWithDeltas (ime_decoration.dart:119)
TextInput._handleTextInputInvocation (text_input.dart:1867)
TextInput._loudlyHandleTextInputInvocation (text_input.dart:1753)
MethodChannel._handleAsMethodCall (platform_channel.dart:555)
MethodChannel.setMethodCallHandler.<fn> (platform_channel.dart:548)
_DefaultBinaryMessenger.setMessageHandler.<fn> (binding.dart:393)

Variant 2
Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError
Exception: Couldn't map an IME position to a document position. TextEditingValue: '[redacted]' IME position: TextPosition(offset: 129, affinity: TextAffinity.downstream). Error thrown Instance of 'ErrorDescription'.
DocumentImeSerializer._imeToDocumentPosition (document_serialization.dart:271)
DocumentImeSerializer.imeToDocumentSelection (document_serialization.dart:164)
TextDeltasDocumentEditor._applyInsertion (document_delta_editing.dart:133)
TextDeltasDocumentEditor.applyDeltas (document_delta_editing.dart:69)
DocumentImeInputClient.updateEditingValueWithDeltas (document_ime_communication.dart:215)
DeltaTextInputClientDecorator.updateEditingValueWithDeltas (ime_decoration.dart:119)
TextInput._handleTextInputInvocation (text_input.dart:1867)
TextInput._loudlyHandleTextInputInvocation (text_input.dart:1753)
MethodChannel._handleAsMethodCall (platform_channel.dart:555)
MethodChannel.setMethodCallHandler.<fn> (platform_channel.dart:548)
_DefaultBinaryMessenger.setMessageHandler.<fn> (binding.dart:393)

Variant 3
Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: Exception: Couldn't map an IME position to a document position.
TextEditingValue: '[redacted] '
IME position: TextPosition(offset: 132, affinity: TextAffinity.downstream). Error thrown Instance of 'ErrorDescription'.
       at DocumentImeSerializer._imeToDocumentPosition(document_serialization.dart:271)
       at DocumentImeSerializer.imeToDocumentSelection(document_serialization.dart:164)
       at TextDeltasDocumentEditor._applyNonTextChange(document_delta_editing.dart:211)
       at TextDeltasDocumentEditor.applyDeltas(document_delta_editing.dart:75)
       at DocumentImeInputClient.updateEditingValueWithDeltas(document_ime_communication.dart:215)
       at DeltaTextInputClientDecorator.updateEditingValueWithDeltas(ime_decoration.dart:119)
       at TextInput._handleTextInputInvocation(text_input.dart:1867)
       at TextInput._loudlyHandleTextInputInvocation(text_input.dart:1753)
       at MethodChannel._handleAsMethodCall(platform_channel.dart:555)
       at MethodChannel.setMethodCallHandler.<fn>(platform_channel.dart:548)
       at _DefaultBinaryMessenger.setMessageHandler.<fn>(binding.dart:393)

Variant 4
Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: Exception: Couldn't map an IME position to a document position.
TextEditingValue: '[redacted] '
IME position: TextPosition(offset: 128, affinity: TextAffinity.downstream). Error thrown Instance of 'ErrorDescription'.
       at DocumentImeSerializer._imeToDocumentPosition(document_serialization.dart:271)
       at DocumentImeSerializer.imeToDocumentSelection(document_serialization.dart:164)
       at TextDeltasDocumentEditor.delete(document_delta_editing.dart:311)
       at TextDeltasDocumentEditor._applyDeletion(document_delta_editing.dart:198)
       at TextDeltasDocumentEditor.applyDeltas(document_delta_editing.dart:73)
       at DocumentImeInputClient.updateEditingValueWithDeltas(document_ime_communication.dart:215)
       at DeltaTextInputClientDecorator.updateEditingValueWithDeltas(ime_decoration.dart:119)
       at TextInput._handleTextInputInvocation(text_input.dart:1867)
       at TextInput._loudlyHandleTextInputInvocation(text_input.dart:1753)
       at MethodChannel._handleAsMethodCall(platform_channel.dart:555)
       at MethodChannel.setMethodCallHandler.<fn>(platform_channel.dart:548)
       at _DefaultBinaryMessenger.setMessageHandler.<fn>(binding.dart:393)

Variant 5
Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: Exception: Couldn't map an IME position to a document position.
TextEditingValue: '[redacted]'
IME position: TextPosition(offset: 4, affinity: TextAffinity.downstream). Error thrown Instance of 'ErrorDescription'.
       at DocumentImeSerializer._imeToDocumentPosition(document_serialization.dart:271)
       at DocumentImeSerializer.imeToDocumentRange(document_serialization.dart:220)
       at TextDeltasDocumentEditor.applyDeltas(document_delta_editing.dart:89)
       at DocumentImeInputClient.updateEditingValueWithDeltas(document_ime_communication.dart:215)
       at DeltaTextInputClientDecorator.updateEditingValueWithDeltas(ime_decoration.dart:119)
       at TextInput._handleTextInputInvocation(text_input.dart:1867)
       at TextInput._loudlyHandleTextInputInvocation(text_input.dart:1753)
       at MethodChannel._handleAsMethodCall(platform_channel.dart:555)
       at MethodChannel.setMethodCallHandler.<fn>(platform_channel.dart:548)
       at _DefaultBinaryMessenger.setMessageHandler.<fn>(binding.dart:393)

Variant 6
Fatal Exception: io.flutter.plugins.firebase.crashlytics.FlutterError: Exception: Couldn't map an IME position to a document position.
TextEditingValue: '[redacted]'
IME position: TextPosition(offset: 316, affinity: TextAffinity.downstream). Error thrown Instance of 'ErrorDescription'.
       at DocumentImeSerializer._imeToDocumentPosition(document_serialization.dart:271)
       at DocumentImeSerializer.imeToDocumentSelection(document_serialization.dart:164)
       at TextDeltasDocumentEditor.replace(document_delta_editing.dart:283)
       at TextDeltasDocumentEditor._applyReplacement(document_delta_editing.dart:184)
       at TextDeltasDocumentEditor.applyDeltas(document_delta_editing.dart:71)
       at DocumentImeInputClient.updateEditingValueWithDeltas(document_ime_communication.dart:215)
       at DeltaTextInputClientDecorator.updateEditingValueWithDeltas(ime_decoration.dart:119)
       at TextInput._handleTextInputInvocation(text_input.dart:1867)
       at TextInput._loudlyHandleTextInputInvocation(text_input.dart:1753)
       at MethodChannel._handleAsMethodCall(platform_channel.dart:555)
       at MethodChannel.setMethodCallHandler.<fn>(platform_channel.dart:548)
       at _DefaultBinaryMessenger.setMessageHandler.<fn>(binding.dart:393)

Thanks for all of the examples. Maybe we need to try to add integration tests so that we run at least some of our tests against a real IME.
@angelosilvestre as a start for this ticket, can you try adding a single integration test that runs on Android, which displays a full-screen editor, types something, and then verifies what was typed?

@matthew-carroll Even using an integration test, wouldn't we be using the same test methods that just send the same platform messages as the IME would do?
We will send messages to the real IME, but I don't think we can type using the virtual keyboard...

Can you poke around the docs and options for that? Can we leverage some Android tooling to send input through the keyboard? Or at least through the IME?


[WIP; DO NOT MERGE] NUC9VXQNX support

Support for peripherals on NUC9VXQNX

Intel AX200 WIFI https://github.com/lf-edge/eve/pull/2035
Watchdog

For the watchdog, pending review on why iTCO_wdt is not loaded automatically, and if this can be fixed, instead of a manual modprobe.
It seems that NUC10i7FNH is in a similar situation (iTCO_wdt module not picked up automatically).
Dump of ACPI tables
NUC9VXQNX
        Device (CWDT)
        {
            Name (_HID, "INTC1028")  // _HID: Hardware ID
            Name (_CID, EisaId ("PNP0C02") /* PNP Motherboard Resources */)  // _CID: Compatible ID
            Method (_STA, 0, Serialized)  // _STA: Status
            {
                Return (0x0F)
            }

            Method (_CRS, 0, NotSerialized)  // _CRS: Current Resource Settings
            {
                Name (RBUF, ResourceTemplate ()
                {
                    IO (Decode16,
                        0x0000,             // Range Minimum
                        0x0000,             // Range Maximum
                        0x04,               // Alignment
                        0x04,               // Length
                        _Y38)
                })
                CreateWordField (RBUF, \_SB.PCI0.LPCB.CWDT._CRS._Y38._MIN, OMIN)  // _MIN: Minimum Base Address
                CreateWordField (RBUF, \_SB.PCI0.LPCB.CWDT._CRS._Y38._MAX, OMAX)  // _MAX: Maximum Base Address
                OMIN = (PMBS + 0x54)
                OMAX = (PMBS + 0x54)
                Return (RBUF) /* \_SB_.PCI0.LPCB.CWDT._CRS.RBUF */
            }
        }
    }

Hey @erkki -- can you please split this PR in two? We can merge the firmware part ASAP.
As for the iTCO_wdt -- I have a few boxes where it loads up as expected -- lets compare ACPI tables. To that point, can you please attach the result of tar -C /sys/firmware/acpi -czvf /tmp/acpi.tgz * to this PR?

Hey @erkki -- can you please split this PR in two? We can merge the firmware part ASAP.
As for the iTCO_wdt -- I have a few boxes where it loads up as expected -- lets compare ACPI tables. To that point, can you please attach the result of tar -C /sys/firmware/acpi -czvf /tmp/acpi.tgz * to this PR?

Yeap, https://github.com/lf-edge/eve/pull/2035 .
tar -C /sys/firmware/acpi -czvf /tmp/acpi.tgz * --exclude hotplug --exclude interrupts --exclude pm_profile
.tar => .zip as github blocked the upload :)
acpi.zip

Thanks for the attachment -- I'll need more time now to go through it

It seems to me the watchdog (current latest bios version QNCFLX70.0059.2020.1130.2122) on this NUC is buggy.
I'm seeing the watchdog timer not counting down, similar to the issue here: https://community.intel.com/t5/Processors/Using-linux-iTCO-wdt-driver-on-i3-7100/td-p/1126815 .
I've submitted a support ticket to Intel, hopefully there will be some reaction.
Watchdog start+stop
(ns: pillar) # rmmod iTCO_wdt
(ns: pillar) # modprobe iTCO_wdt nowayout=1 heartbeat=10
(ns: pillar) # cat /sys/class/watchdog/watchdog0/status
0x0
(ns: pillar) # cat /sys/class/watchdog/watchdog0/timeleft
9
(ns: pillar) # echo "1" >> /dev/watchdog
(ns: pillar) # cat /sys/class/watchdog/watchdog0/timeleft
9
(ns: pillar) # cat /sys/class/watchdog/watchdog0/status
0x8000
(ns: pillar) # cat /sys/class/watchdog/watchdog0/timeleft
9
(ns: pillar) # cat /sys/class/watchdog/watchdog0/timeleft
9

dmesg
[11692.422132] iTCO_wdt: Watchdog Module Unloaded
[11769.323024] iTCO_wdt: Intel TCO WatchDog Timer Driver v1.11
[11769.325138] iTCO_wdt: Found a Intel PCH TCO device (Version=6, TCOBASE=0x0400)
[11769.327104] iTCO_wdt: initialized. heartbeat=10 sec (nowayout=1)
[11788.365213] watchdog: watchdog0: watchdog did not stop!

NUCs in general are really not that interesting of a platform to run EVE beyond basic evaluation -- supporting them as an afterthought is probably OK -- but I don't think they have a chance of a first class citizen support until.

Yep, that is fine (although could be interesting for digital signage usecases)

Update: I've managed to test an updated debug BIOS from Intel that:

Automatically loads the wdat_wdt watchdog module
Actually works and triggers a reset

We can close this PR as this means with an updated bios no need to force load modules. I'll post a backreference here if and when the updated bios becomes publicly available.

Yay! Thanks for seeing it through @erkki !


New User event not visible after first batch event sent

Summary
I used Batch API to send an event with event_type = "track" and self generated device_id. "track" event is visible in Amplitude. But "New User" is not created itself as according to my understanding is created by default when a user is created in amplitude. I created users with Identity API with some user_properties.

@macci10 does the event "track" only appear for a single user ID in amplitude? And are you generating the device_ids yourself or letting the SDK generate them?

@macci10 any update on this one?

I have found that I was querying the users in the wrong way. New User event
was being created when I call batch api. So there was no issue. Thanks for
your time.
On Wed, Jun 30, 2021 at 10:30 PM AJ Horst @.***> wrote:

@macci10 https://github.com/macci10 any update on this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/amplitude/Amplitude-Node/issues/121#issuecomment-871596437,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC5JZFSQAV6UEJOP4WZ3YTDTVNIE5ANCNFSM46UULASA
.

@macci10 glad to hear that you figured it out


Look into partitioner discrepancy

cf #71 for now

closing this for now, as no one has requested a different partitioner, or more specifically the default one - so leaving as is but adding a note of caution to produce's docstring


LfMerge integration tests with real MongoDB container

LfMerge integration tests in #342 currently run against a mock Mongo connection. But for real end-to-end testing, we probably want to set up a real MongoDB instance and populate the test project in Mongo, then run the tests that way. We're not talking about setting up an entire local Language Forge, just a Mongo database.
This will be a good follow-up issue once #342 is merged.

To launch a local Mongo, I'd probably use docker run mongo with appropriate options (-d to detach and run in background, -n lfmerge-test-db to have a predictable name, -p 27017 to forward container port 27017 to a random local port, and so on). Then run docker port lfmerge-test-db 27017 to find out which random local port docker has found available, and configure LfMerge's Mongo connection to use that port.


groupUniqArray() result true or false

Describe the bug
Steps to reproduce
1.i use groupUniqArray(toHour(gettime)) to find out unique hour in gettime ,but in metabase it result [false,false].
However when i use dbeaver,it't show [12,14] ,which mean gettime's unique hour is 12 and 14.
Expected behaviour
true result show in dbeaver [12,14]
error result show in metabase [false,false].
Error log
Configuration
Environment

metabase-clickhouse-driver version:1.2.1
metabase-clickhouse-driver configuration:
Metabase version:0.47.4
OS:docker

ClickHouse server

ClickHouse Server version:<IP_ADDRESS>

colume:  gettime       uid
type:      Datetime64  String
data       2023-10-10 01:01:01 12edada
2023-10-10 03:01:01 12edada
2023-10-10 06:01:01 12edada
sql: select groupUniqArray(gettime) as hour_a, uid  from table group by uid
result in dbeaver: [1,3,4],12edada
result in metabase: [false,false,false],12edada

i run this from the sql query editor?

@jack-reso, fixed in 1.3.1

@slvrtrn thanks！


Fresh install get Emacs blocked after many packages fail to install

Emacs 29.4, (build 2, x86_64-w64-mingw32), installed with Chocolatey.
With a fresh copy of the repository, I get a large number of errors when I respond "yes" to create user settings, at which point I can't even exit Emacs because C-x C-g throws the following error: "Cannot open load file: No such file or directory, vertico-repeat".
Warning buffer:
 ■  Error (use-package): Failed to install vertico: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install orderless: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Cannot load orderless
 ■  Error (use-package): Failed to install embark: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install embark-consult: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install marginalia: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): marginalia/:init: Symbol’s function definition is void: marginalia-mode
 ■  Error (use-package): Failed to install consult: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install consult-dir: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install corfu: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install cape: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install kind-icon: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install yasnippet: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install yasnippet-snippets: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install which-key: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install imenu-list: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install goto-last-change: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Cannot load goto-last-change
 ■  Error (use-package): Failed to install dired-narrow: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install dired-quick-sort: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install diredfl: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install peep-dired: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install dired-ranger: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install deadgrep: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install rg: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install visual-regexp: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install magit: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install git-commit: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install diff-hl: Cannot open load file: No such file or directory, goggles
 ■  Error (use-package): Failed to install vdiff-magit: Cannot open load file: No such file or directory, diff-hl
 ■  Error (use-package): Failed to install tabspaces: Cannot open load file: No such file or directory, diff-hl
 ■  Error (use-package): Failed to install ace-window: https://elpa.gnu.org/devel/avy-<IP_ADDRESS>.20241101.125753.tar: No Data
 ■  Error (use-package): Failed to install svg-tag-mode: https://elpa.gnu.org/devel/svg-tag-mode-<IP_ADDRESS>.20241021.134153.tar: No Data
 ■  Error (use-package): Failed to install highlight-numbers: Cannot open load file: No such file or directory, svg-tag-mode
 ■  Error (use-package): Failed to install hl-todo: Cannot open load file: No such file or directory, highlight-numbers
 ■  Error (use-package): Failed to install goggles: Cannot open load file: No such file or directory, hl-todo

Hi @dc6c -- I haven't been able to reproduce this (installing on emacs 29 from a clean install seems to work fine). Are you continuing to see the problem? If you are can you tell me more about your setup (emacs version, OS, etc.)?

I included my Emacs version in the first line of the report -- I didn't mention that the OS is Windows because I thought the build identifier made it clear (it's Windows 10). The error is still there, I checked again with a fresh install of Lambda-Emacs. After restarting Emacs, it continued to install the missing packages, but even after several rounds of restarts, the following two still fail to install:
 ■  Error (use-package): marginalia/:init: Symbol’s function definition is void: marginalia-mode
 ■  Error (use-package): consult/:config: consult-xref is neither a Command command nor a source

unfortunately I don't have a window machine to test this on (I'm on macOS). Is there something different about how emacs handles package installation on Windows?


Pagination

It would be useful to have the option to paginate the in-editor view of a reorderable list.
For example (and this could be hardcoded), only 100 entries are shown, and if there are more a small page UI is visible next to the +/- at the bottom, which shows "page X / Y" where X is editable by the user.
With 315 entries you would by default see "Page 1 of 4" with the first 3 pages having 100 elements visible and page 4 only 15 entries.
Optionally there could also be an input field for 'elements per page' or 'page size'

Yeah I like this idea. Could add some properties to the list such as:
list.pagination = true
list.pageSize = 100 //0 for custom pagination (UI controlled)

Hey @JohannesMP I've added pagination into the list. I still need to document it but the usage is simple:
for the attribute version:
[Reorderable(paginate = true, pageSize = 0)]
for the object version:
list.paginate = true
list.pageSize = 0
When pageSize = 0, custom pagination is used.
You can also right-click elements and move them to different pages if desired.
Let me know what you think, and if you find any bugs.

It appears that it's impossible to do anything with pagination if the callbacks are implemented. To give an example: If I were to subscribe to OnAddCallback, there is no apparent way to move to the next page if the new element is not on the current page like it would in AddItem().

You're right. I've added some public methods to hopefully help with this.
SetPage
SetPageByIndex
GetPage
GetPageByIndex


breaking out of a loop using a label: stops the loop prematurely using larger dataset

Go version
go version go1.23.3 linux/amd64
Output of go env in your module/workspace:
GOARCH='amd64'
GOCACHE='/home/user/.cache/go-build'
GOENV='/home/user/.config/go/env'
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOMODCACHE='/home/user/go/pkg/mod'
GOOS='linux'
GOPATH='/home/user/go'
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/nix/store/zhq8dnjbhwspzdglyq28j74axvqyk86q-go-1.23.3/share/go'
GOSUMDB='sum.golang.org'
GOTOOLCHAIN='auto'
GOTOOLDIR='/nix/store/zhq8dnjbhwspzdglyq28j74axvqyk86q-go-1.23.3/share/go/pkg/tool/linux_amd64'
GOVERSION='go1.23.3'
GOTELEMETRY='off'
GOTELEMETRYDIR='/home/user/.config/go/telemetry'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='1'
GOMOD='/dev/null'
CGO_CFLAGS='-O2 -g'
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build1389248914=/tmp/go-build -gno-record-gcc-switches'

What did you do?
I was writing some quick and inefficient code to get a problem answer fast for advent of code day5 and noticed that the program exits prematurely when breaking from an inner loop to an outer loop using a label -- though it does so only when using a larger input.
I'm not sure if this is an issue with the runtime or I'm not using the language correctly -- if the latter is true; I'm sorry for wasting your time, otherwise happy to help.
What did you see happen?
Program exits prematurely without errors when using a larger dataset, with a smaller input the program runs as expected.
What did you expect to see?
How to reproduce

not working example: https://go.dev/play/p/uuWKb5CDsxd
working example (replaced break OUTER with a func): https://go.dev/play/p/x8ikpK-VD99

Related Issues

cmd/compile: double error when break is not in a loop #51456 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

I'm pretty sure this is working as expected. The only line of code that should execute after your break OUTER is the fmt.Printf at line 66.
Perhaps you meant continue OUTER? Or put the OUTER label one loop in?

thank you @randall77


Mongodb authentication

How do I authenticate with my DB? Using MongoDb authentication in the url failed to work.
Thanks.

@agozie I just add a new method: NewHookerWithAuth：
func NewHookerWithAuth(mgoUrl, db, collection, user, pass string) (*hooker, error)


[BUG] Server Text won't properly display. 1.18.2

Versions

EasyAuth 2.2.2
Fabric API 0.48.0+1.18.2

Describe the bug
The mod should display or send a message to a player to login or register. however, it send incorrect text format. as it was a placeholder thing. i dont know what it is. it was working fine using easyauth 2.1.0. Probably break in 1.18.2.
It seems that lang inside the config file doesnt work.
To Reproduce
Steps to reproduce the behavior:

add the mod to 1.18.2 server
join the server
See the white text message to login with incorrect format.

Expected behavior
should send a message saying You are not authenticated! /login or /l to authenticate!

I cannot load the mod on 1.18.2:

@Glucy-2
it's seems you are using 1.18 version which are incompatible with 1.18.2

OK, thanks
I've found the 1.18.2 ver


To correct fibonacci

Hi guys,
There is a small issue in ruby code of fibonacci method which returned the wrong answer.
fib(2) should be 1, not 2.
It would lead all answers shifted. i.e. fib(n) equals to corrected fibonacci(n+1)
I tried to fix it, so just check it if its helpful :)

@sinorga , Thanks! You are right according to in mathematical definitions of Fibonacci series.


doublecmd: 1.1.16 -> 1.1.17

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/doublecmd/doublecmd/releases.
meta.description for doublecmd is: Two-panel graphical file manager written in Pascal
meta.homepage for doublecmd is: https://doublecmd.sourceforge.io/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.17 in filename of file in /nix/store/c1vbv9q3fn0sf9kf6hg7hrjxiz88aj6w-doublecmd-1.1.17

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

doublecmd

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/2b28845f003edc5ef2608f9989758aeb024abe62#doublecmd

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

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

1 package built:

doublecmd

Maintainer pings
cc @AndersonTorres for testing.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


C api fix posit8 test really this time

that was silly ( this is what #77 was supposed to be )

I just merged it, positN_reinterpret() is a reinterpret_cast which is really saying "these are the bits, I know what I'm doing" and using the low 8 bits of a double is clearly not going to work


Cleanup of pelican config files

Fixes issues with the 3 config files used with Pelican: defaults.yaml, osdf.yaml, and pelican.yaml. Before this fix there was a bit of a mess of where the configs are set up with viper. This fix moves InitConfig to config.go and sets up all the config files in the priority they need to be with pelican.yaml taking higher precedence than defaults.yaml. Also fixed issue in object_copy.go with some weird issues with the -d flag. Only need to check for the -d flag with stashcp since it does not go through root, any other modes are checked within root/initconfig and trigger debug mode.

Ahh, @joereuss12 I think I figured out what the source of your headache is with the calls to MergeConfig() on the embedded yamls -- viper has to be told how it should interpret the io reader because it can parse many types of configuration files. Adding viper.SetConfigType("yaml") as the very first line of InitConfig solved the issue for me.


change src folder name to skbase

This changes the src folder name from baseobject to skbase.
Also bumps the version number to 0.2.0.

Codecov Report

Merging #47 (a8ac651) into main (ae9f5df) will not change coverage.
The diff coverage is 92.85%.

@@           Coverage Diff           @@
##             main      #47   +/-   ##
=======================================
  Coverage   59.03%   59.03%
=======================================
  Files          19       19
  Lines        1538     1538
=======================================
  Hits          908      908
  Misses        630      630

Impacted Files
Coverage Δ

skbase/_exceptions.py
100.00% <ø> (ø)

skbase/testing/utils/__init__.py
100.00% <ø> (ø)

skbase/testing/utils/_conditional_fixtures.py
79.10% <ø> (ø)

skbase/testing/utils/_dependencies.py
40.54% <ø> (ø)

skbase/testing/utils/inspect.py
57.14% <ø> (ø)

skbase/testing/utils/tests/__init__.py
100.00% <ø> (ø)

skbase/tests/__init__.py
100.00% <ø> (ø)

docs/source/conf.py
73.23% <71.42%> (ø)

skbase/__init__.py
100.00% <100.00%> (ø)

skbase/_base.py
63.36% <100.00%> (ø)

... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Incomplete Migrations

It is possible for a migration to be performed incompletely, where, everything is migrated except for statements that have errors (e.g.) syntax errors.
For instance:
CREATE TABLE schema.blah(foo INT);
GRAaNT SELECT on blah Too someone;
Although the second line has clear errors, the first line does not and successfully migrates (the table gets created but permissions are not granted). Unfortunately, because the second line failed, the migration is identified as a failure. Ideally, failed migrations would not be applied to the database. Otherwise, the user must then manually delete everything that succeeded and then re-migrate.

This is due to your database not supporting DDL transactions.
Works as designed. See http://flywaydb.org/documentation/faq.html#rollback


[chore] Change class name from process definition to workflow

Brief Summary of The Change

we should change the class keyword from process definition to workflow
fix: #22
Pull Request checklist
I confirm that the following checklist has been completed.

[x] Add/Change test cases for the changes.
[x] Add/Change the related documentation, should also change docs/source/config.rst when you change file default_config.yaml.
[x] (Optional) Add your change to UPDATING.md when it is an incompatible change.

Codecov Report

Merging #26 (24e3a57) into main (9171490) will decrease coverage by 0.56%.
The diff coverage is 80.45%.

@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
- Coverage   90.59%   90.03%   -0.57%
==========================================
  Files          56       58       +2
  Lines        2042     2057      +15
==========================================
+ Hits         1850     1852       +2
- Misses        192      205      +13

Flag
Coverage Δ

unittests
90.03% <80.45%> (-0.57%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/pydolphinscheduler/core/process_definition.py
0.00% <0.00%> (-81.87%)
:arrow_down:

src/pydolphinscheduler/tasks/sub_process.py
0.00% <0.00%> (-83.34%)
:arrow_down:

src/pydolphinscheduler/tasks/switch.py
100.00% <ø> (ø)

src/pydolphinscheduler/cli/commands.py
97.14% <50.00%> (ø)

src/pydolphinscheduler/tasks/dependent.py
93.23% <66.66%> (-1.35%)
:arrow_down:

src/pydolphinscheduler/core/workflow.py
81.86% <81.86%> (ø)

src/pydolphinscheduler/tasks/sub_workflow.py
83.33% <83.33%> (ø)

src/pydolphinscheduler/core/task.py
93.67% <88.88%> (ø)

src/pydolphinscheduler/constants.py
100.00% <100.00%> (ø)

src/pydolphinscheduler/core/__init__.py
100.00% <100.00%> (ø)

... and 3 more

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

Should not merged until #15 merged, we need to change its keyword as well

ref: https://github.com/apache/dolphinscheduler/pull/12918

ref: apache/dolphinscheduler#12918

released after apache/dolphinscheduler:3.1.2


Allow removal of env files

If an env variable is specified and there are file in the services env
directory that don't match, delete them.  If an env isn't specified files
are not touched.

Thanks @nhuff I'll look to review this and merge early next week.

I'm :+1:, and thanks for the tests!


New namespaced features implementation.

This is a new implementation for namespaced features (#5565). See the unstable.md docs for a description of the new behavior. This is intended to fix several issues with the existing design, and to make it backwards compatible so that it does not require an opt-in flag.
This also includes tangentially-related changes to the new feature resolver. The changes are:

crate_name/feat_name syntax will now always enable the feature crate_name, even if it is an inactive optional dependency (such as a dependency for another platform). The intent here is to have a certain amount of consistency whereby "features" are always activated, but individual crates will still not be activated.
--all-features will now enable features for inactive optional dependencies. This is more consistent with --features foo enabling the foo feature, even when the foo dep is not activated.

I'm still very conflicted on how that should work, but I think it is better from a simplicity/consistency perspective. I still think it may be confusing if someone has a cfg(some_dep) in their code, and some_dep isn't built, it will error. The intent is that cfg(accessible(some_dep)) will be the preferred method in the future, or using platform cfg expression like cfg(windows) to match whatever is in Cargo.toml.
Closes #8044
Closes #8046
Closes #8047
Closes #8316
Questions

For various reasons, I changed the way dependency conflict errors are collected. One slightly negative consequence is that it will raise an error for the first problem it detects (like a "missing feature"). Previously it would collect a list of all missing features and display all of them in the error message. Let me know if I should retain the old behavior. I think it will make the code more complicated and brittle, but it shouldn't be too hard (essentially Requirements will need to collect a list of errors, and then resolve_features would need to check if the list is non-empty, and then aggregate the errors).

Should cargo metadata show the implicit features in the "features" table? Currently it does not, and I think that is probably best (it mirrors what is in Cargo.toml), but I could potentially see an argument to show how cargo sees the implicit features.

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

This was clearly a lot of work. Thank you!

I'm moved by the argument from @est31 that crate: is dangerously close in syntax to crate:: in Rust, and they mean significantly different things. I'm concerned that crate: would perpetuate more misunderstanding about what the term "crate" means. I think it is an uphill battle that cannot be won, but I don't want to push things in the direction of more confusion. I can revert this if people don't agree.
I have pushed a commit that renames it to dep:. I decided on dep: instead of pkg: because the name is not necessarily the package name (it is the "name in toml"), and I think it more clearly emphasizes that it is referring to a dependency.

@bors: r+
I think it's a good point that crate: can be easily confused, and I also agree that crate/pkg is best to not dive into and dep: feels good here to me!
I'm going to go ahead and r+ since this is all unstable anyway, and we can continue to tweak it. Otherwise what's here looks great, thanks @ehuss!

:pushpin: Commit f4ac82a0f5b03eed60b7363bfe15ff1a41ec22e6 has been approved by alexcrichton

:hourglass: Testing commit f4ac82a0f5b03eed60b7363bfe15ff1a41ec22e6 with merge 963bfe4dd9e3b35738182f4e1667c0197d2ab60c...

:sunny: Test successful - checks-actions
Approved by: alexcrichton
Pushing 963bfe4dd9e3b35738182f4e1667c0197d2ab60c to master...

Thanks for fixing this stuff up, sorry I wasn't able to get to it sooner.


luci-app-opkg page refresh exception. #5696

When #5696 was introduced, the opkg page refreshed abnormally.

Details?


Extract NodePattern to a separated gem

I started the work related to #6655.
The gem extracted is under my namespace and I can move it soon some maintainer transfer it.
https://github.com/jonatas/rubocop-node_pattern
The name of the gem is also in the TODO  list to be discussed and probably we can find a better name since all AST utilities are going to the repo too.
TODO

[ ] Move the related project to rubocop-hq namespace

[ ] Update the Gemfile to rubocop-hq namespace

[ ] Move NodePattern implementation to NodePattern::Parser or something to make the gem clear and avoid requires in the bottom

[ ] Define a better name to the gem?

[ ] Setup RuboCop config for the project

[x] Wrote good commit messages.

[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).

[x] Feature branch is up-to-date with master (if not - rebase it).

[x] Squashed related commits together.

[x] Added tests.

[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.

[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.

[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Notes
I'm not sure how can we maintain the ownership of the current git files that are moving to another repo. Is it something common or we just move and have a new author?

Got it @bbatsov. Looking at the code, we can see node_pattern depends on astx. We can do it in two gems or only one.
@baweaver let me know how can we schedule a pairing session or work together on it. I'm open to both cases. We can go with one or two gems. I'd start with one and we can split the external gem later if needed.

With the movement of extracting parts of RuboCop out of it (e.g. rubocop-performance), while keeping those extracted repositories first-class citizen of RuboCopHQ, it makes sense to think about how a change to main rubocop, and specifically to its node pattern repository would affect the dependent repositories.
An approach to consider is the one taken by RSpec, when all the repositories are checked out, and before submitting a change one would check if the dependent officially supported gems are not broken.
# rspec-expectations' Gemfile
branch = File.read(File.expand_path("../maintenance-branch", __FILE__)).chomp # `master`, or some cross-repository branch
%w[rspec rspec-core rspec-mocks rspec-support].each do |lib|
  library_path = File.expand_path("../../#{lib}", __FILE__)
  if File.exist?(library_path) && !ENV['USE_GIT_REPOS']
    gem lib, :path => library_path
  else
    gem lib, :git => "https://github.com/rspec/#{lib}.git", :branch => branch
  end
end

With that modular yet atomically safe approach, it doesn't matter that much anymore if rubocop-node_pattern is in the same repository or not, and breakages can be spotted early, during local development. That also kind of simplifies the changes that need to be made in several repositories at once to fix a bug/introduce a feature.
WDYT?

Yeah, that's a reasonable approach. We should first finish the isolation/extraction of something though. :-) Right now everything's work in progress...
@jonas054 Any updates here?

@bbatsov It's very nice to hear from you again! :smile: But did you in this case mean @jonatas?

Ops! Sorry about that! I totally meant to ping @jonatas, but I love you so much that I type your name every time I start typing @jona.... 😄

I like the idea and I guess we can try to follow the approach @pirj suggested.
The hardest part is that we have ast/node that is a dependency for node_pattern. I'll try to pair with @pirj tomorrow and see if we can handle it.

Sounds good. Ping me if you need my help with anyone.

Thanks, @bbatsov. Should we keep the same module name for now or already go with astx?

I guess for now it can be the same module name.

Let me know if you need help...

I'm not having the proper time to finish it. I'll close it for now and revisit in a near future.

@marcandre feel free to ping me I can try to pair with you and help to review it.


HIVE-26640:Upgrade JUnit to 4.13.2 due to medium CVEs

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Please review this

@nrg4878 Please see this

Could we remove https://github.com/apache/hive/blob/master/packaging/pom.xml#L174-L178  as well? seems this has transitive dependencies to JUnit, we do not need these in runtime. Others look good to me!


Can not update collection UnknownUpdateOperation(...)

collection
    .update(
        Some(&mk_document! { "_key": key_name }),
        &mk_document! { "value": value },
    )
    .unwrap();

thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: UnknownUpdateOperation("value")', src/db.rs:143:14
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I want to update the entry with the "_key" as key_name's value field, however, it displays the above error. any help?

You need to specific the update operation, so try:
collection
    .update(
        Some(&mk_document! { "_key": key_name }),
        &mk_document! {
          "$set": &mk_document! {
             "value": value
          }
       },
    )

Other operation can be found at docs/en-US/Update.md

Are you saying using collection again or using database again?
It's designed to be allow using collection for multiple times.
Database is designed to be used multiple times too, but limited. Only one instance can read/write database at the same time. Otherwise, the thread will be locked.

Just out of curiosity, is it an issue with database initiation or collection initiation?


[active_job] Make the context available in ActiveJob notifications

Previously, subscribers to 'perform.active_job' events had no access to the tracing context. Only the job and its callbacks had access to it.
This was a result of adding OpenTelemetry's instrumentation callback after the notifications instrumentation that is configured in ActiveJob::Logging (in ActiveJob 6.x).
This pull request fixes the issue by prepending OpenTelemetry's instrumentation callback before all others.
In ActiveJob v7, however, notifications instrumentation happens before any callbacks are called, so the current solution doesn't work, sadly.

@hosamaly Do you have any thoughts on how we might make this work for ActiveJob v6 and v7? I recognize how useful this would be, but I would hate to create a situation where users of the library could come to rely on this behavior on ActiveJob v6 only to have a regression in functionality when they get around to upgrading to ActiveJob v7. 😢

@ahayworth: I made another attempt at fixing this. It should now work with all versions of ActiveJob.
Instead of using a callback to propagate the context, we now override ActiveJob::Base#perform_now, which is the entrypoint to execution logic. It is called right after the job is deserialized.
This is the same pattern that is used in ActiveJob::Instrumentation in v7. Instead of using a callback as it did in previous versions, the instrumentation in v7 overrides #perform_now.
I left the indentation looking a bit ugly to keep the file's history intact. Feel free to correct it if you feel it must be.

@ahayworth: I made another attempt at fixing this. It should now work with all versions of ActiveJob.

Wonderful!

I left the indentation looking a bit ugly to keep the file's history intact. Feel free to correct it if you feel it must be.

Thanks - I appreciate that, but I did go ahead and push a whitespace-fixing commit. Since most of this is my ~fault~ handiwork anyways, the appropriate git-blame information is mostly preserved this way.
Conveniently, that will also allow CI to run - the job seemed stuck in a queued state despite my clicking the "approve" button for CI to run. That's why I closed/re-opened the PR; I was trying to kick off the jobs again. But no matter, fixing the whitespace also does that. 😄


add hosting application set property in support of argocd integration…

… with pull model
Signed-off-by: Xiangjing Li<EMAIL_ADDRESS>Description of changes

Add a new property hostingApplicationSet to indicate the hosting applicationset if the argocd application is deployed by pull model.
The hostingApplicationSet info is from the hosting-applicationset annotation in the argocd application CR on the managed cluster

While the existing applicationSet is to show the applicationset if the argocd app is deployed by push model
The applicationSet info is from the owner reference in the argocd application CR on the hub cluster.

We will work with app UI in 2.8 to see if we can use one applicationSet field to show the parent argocd applicationset, regardless of the argo application being deployed by pull model or push model.

/assign @jlpadilla


Chat heads don't animate to position correctly initially (without moving cursor)

Repro: chat heads, don't move mouse. See that the heads don't animate to 250, 300 even though defaultStyles and styles are set.
Probably has something to do with the stop check (again...).
@bsansouci

Fixed in v0.4.1


"This module cannot be imported in server runtime" error on fresh Nuxt install

Version
nuxt-security: 1.0.0
nuxt: 3.9.3
Reproduction Link
https://github.com/johanalkstal/nuxt-security-repro
Steps to reproduce
Run pnpm install
Run pnpm run dev
What is Expected?
No errors
What is actually happening?
Nuxt 3.9.3 with Nitro 2.8.1                                                                                3:41:14 PM
                                                                                3:41:14 PM
  ➜ Local:    http://localhost:3000/
  ➜ Network:  use --host to expose

  ➜ DevTools: press Shift + Alt + D in the browser (v1.0.8)                                                 3:41:15 PM

ℹ Vite server warmed up in 870ms                                                                           3:41:17 PM
ℹ Vite client warmed up in 1043ms                                                                          3:41:17 PM

[3:41:17 PM]  ERROR  This module cannot be importedin server runtime. [importing @nuxt/kit from<EMAIL_ADDRESS>

@Baroshem
This is the same issue with importing @nuxt/kit in the nitro plugin
@pi0 has now implemented guards against this in Nuxt 3.9.3
Maybe we need to temporarily disable 05-cpsSsgPresets while we look for an alternative ?

Hey @vejja
Yes, it is related to that.
Could you create a fix for that? I will publish a patch version later today that we will fix with 1.1.0

Thank you for your quick action 🙏🏽

@Baroshem It is gone for me now. Thank you!


Improve Unit Tests - NPE in Some Cases

A generic utility method needs to be added to the BaseTest classes to obtain references to jar files used when testing that uses the junit assume methods to ensure that the files exist. This new method should be used in the test cases anywhere code obtains the location of a jar file such as:
File struts = new File(this.getClass().getClassLoader().getResource("struts2-core-2.1.2.jar").getPath());

Additionally, consider adding a PMD rule to flag test cases that directly call getClassLoader().getResource().

I agree that this can be split out in a separate method. However, I am not sure about using assume, since it would mean the tests are marked as ignored if the files are not found.
At least when I ran into problems with this (HintAnalyzerTest), I got it consistently when using openjdk-8 (tried Ubuntu and Arch Linux, both result in null pointer in this test case). But with openjdk-7 all tests passed. So I wonder if there might be some difference between jdk7 and 8 which might cause this.

While the openjdk-8 issue still needs to be researched - generic methods have been added to the BaseTest class. The assume API is used, but a message is written to the console when the assumption fails.


Expose original conn variable to client

It would be very helpful if you could add the original conn to the RethinkDB client? It would allow people to assign their own events to it, including the close event.

Hum, I think there's a misunderstanding on how connections are used.
Connections are created/reused/closed as needed, there's a pool of connections, not a single one. There's also no need to listen to a specific connection. The driver already takes care of that.


Fixing image parsing

Image classes changed to fix parsing issues.
Feel free to ignore the todo list in readme, that's for a personal project

keep the updates coming :)
the TODO is welcome


Shipperctl count command output option

This pr adds to the new count command an output option. This allows the users of this command to also get a list of releases for future actions.
For example shipperctl count contender --decommissionedClusters="decommissioned-a,decommissioned-b"  -o json
will output a json of  the form
[{"namespace": "yfouquet","name": "reserve-backend-test-359dc193-0"},{"namespace": "yfouquet", "name": "reserve-backend-test-359dc193-0" },...]
which allows the user to handle these cases separately.

LGTM 👍

OT comment:
Do these commands make sense for non-decommissioned clusters? If this is the case we could change the flag to --clusters. WDYT?
shipperctl count contender --clusters="..."


Headers returned from function isn't lower cased

Headers returned from a mock function isn't lower cased, compare the two following examples.
scope.head('/').reply(200, '', { 'X-Test': 'Hello' })

scope.head('/').reply(() => [200, '', { 'X-Test': 'Hello' }])

The first one correctly has res.headers['x-test'] set to "Hello", the other example on the other hand still has the key capitalised.

Here is a minimal test case to reproduce the error:
const nock = require('nock')
const http = require('http')
const assert = require('assert')

nock('http://example.com')
  .get('/path')
  .reply(() => [200, 'ok', { 'X-Test': 'Hello' }])

http.request('http://example.com/path', res => {
  const header = Object.keys(res.headers)[0]
  assert.equal(header, 'x-test')
}).end()

who ever wants to tackle this issue, this library might be useful: https://www.npmjs.com/package/caseless

This is still an issue with Nock version 10.0.0

@LinusU can you start a PR with a failing test and we take it from there?

Fixed with #1564.


Implemented aggregation over important mark times.

Implemented aggregation over important mark times.

Namely, 50,90,95 and 99th percentiles are now reported on important marks.

Example output (for a single ! mark between begin-end marks in grpc_tcp_write()):

```
Important marks:
================
99999@src/core/iomgr/tcp_posix.c:545
                           Relative mark:         50th p.         90th p.         95th p.         99th p.
  205 { (src/core/iomgr/tcp_posix.c:541):           0.037           0.057           0.070           0.087
  205 } (src/core/iomgr/tcp_posix.c:556):          15.181          27.021          32.509          41.103

```

For a fabricated example (see https://gist.github.com/dgquintas/026d333815589cc37269) with the same ! mark
in two different frames, the output is:

```
Important marks:
================
999999@src/core/iomgr/tcp_posix.c:5
                           Relative mark:         50th p.         90th p.         95th p.         99th p.
    205 { (src/core/iomgr/tcp_posix.c:1):           9.500          13.900          14.450          14.890
    205 } (src/core/iomgr/tcp_posix.c:6):           3.000           4.600           4.800           4.960

999999@src/core/iomgr/tcp_posix.c:3
                           Relative mark:         50th p.         90th p.         95th p.         99th p.
    205 { (src/core/iomgr/tcp_posix.c:1):           2.500           2.900           2.950           2.990
    205 { (src/core/iomgr/tcp_posix.c:2):           1.500           1.900           1.950           1.990
    205 } (src/core/iomgr/tcp_posix.c:4):           2.000           2.800           2.900           2.980
    205 } (src/core/iomgr/tcp_posix.c:6):          10.000          15.600          16.300          16.860

I've just realized it'd be good to always show {'s before }'s in the output. That's a minor change though.

Updated code fixing the issue with percentiles of negatives (by getting rid of negatives altogether).
Also improved ordering: { marks are always reported prior to their corresponding } ones.
PTAL.

Updated for sort().
Another huuuge performance drain is line 123. I'll optimize this on a followup changeset.


Incorrect padding by trailing excess =

I noticed several .net as well as other languages implementations remove the = at the end. Perhaps this shouldn't be done as some services fail without it (file stack requires a policy with base64).  I haven't yet looked into RFC4648 to see if this allowed or not. But I did find this old issue here: https://stackoverflow.com/questions/1228701/code-for-decoding-encoding-a-modified-base64-url
which lead me to do
string policy = UrlBase64.Encode(Encoding.UTF8.GetBytes(json));
policy = policy.PadRight(policy.Length + (4 - policy.Length % 4) % 4, '=');

Please note here is the JSON string which should have the extra padding: {"call":["pick","read","stat","write","writeUrl","store","convert","remove","exif"],"path":"private/organization/537650f3b77efe23a47914f3/contact/5a677e50dbb7bf20c4e1f431d/attachments/map.gif","expiry":1516827600}
The thing is some online decoders account for this and decode anyways. Perhaps we need to look into strict encoding?

Any chance we could get a release out for this.

Yup. There was some documentation that needed to be written prior to release, but v0.1.2 is now out with this feature.

Awesome thanks! Can you please create a github tag for 0.1.2 :)

https://github.com/neosmart/UrlBase64/releases


Push to bp3 fails...

https://github.com/voedger/voedger/actions/runs/6706102305/job/18222058912

not actual anymore


lib.dom.d.ts seems to not be referenced when any file in the compilation contains no-default-lib=true triple slash

🔎 Search Terms
no-default-lib lib dom ignored cannot resolve HTMLElement paths
🕗 Version & Regression Information
This is an unintuitive error that took me a long time to pin down.
⏯ Playground Link
No response
💻 Code
Given the following 3 files:
worker.d.ts
/// <reference no-default-lib="true"/>
/// <reference lib="webworker" />
/// <reference lib="esnext" />

export declare class MyWorker {}

library.d.ts
import type { MyWorker } from "./worker";

export interface Type1 {
  worker: MyWorker;
}

app.d.ts
interface App {
  prop: HTMLElement;
}

tsconfig.json
{
  "compilerOptions": {
    "lib": [ "ES2020", "DOM" ],
  },
}

🙁 Actual behavior
An error occurs in the app.d.ts file
Cannot find name 'HTMLElement'.ts(2304)

🙂 Expected behavior
Any of the following:

A better error message inside the app.d.ts file
an error in the tsconfig
a better error message when using --traceResolution

Suggesting a reason for why lib: [ "DOM" ] in the tsconfig file is ignored.
Its also possible this is incorrect behaviour, however the docs say document that the use case for no-default-lib is for files that do not include the default library, rather than those (like in my case) that wanted to ignore the default libraries included via the tsconfig.json.
Additional information about the issue
This bug is really here as a helpful hint in case anyone else runs into this same issue. Its particularly difficult to debug the issue, as app.d.ts has no references to anything. In my real project, the situation is a bit more complicated, and it took me a long time to figure out that this was the issue.
As a side note, what is the best way to do typings for a web worker file within an existing project? ~At the moment, I'll be adding a separate tsconfig file for the specific web worker, and exclude that file from the main tsconfig,~ EDIT: This doesn't actually work, as each separate tsconfig file cannot include files from the other when composite is true.

I don't think it was intentional that this directive causes an explicitly-set lib setting to be ignored. @sheetalkamat ?

I think this was intentional per https://www.typescriptlang.org/docs/handbook/triple-slash-directives.html#-reference-no-default-libtrue  means it is as if you passed noLib atleast per documentation.

The docs say:

This directive instructs the compiler to not include the default library (i.e. lib.d.ts)

So by the docs, maybe this is correct behaviour. However, if I change app.d.ts to
/// <reference lib="dom" />

interface App {
  prop: HTMLElement;
}

Then the error goes away. this seems to be inconsistent - what is the difference between referencing the library via lib: [ "DOM" ] in tsconfig vs using /// <reference lib="dom" /> in a file?

From documentation:

This directive marks a file as a default library.

It can be seen both ways but doc makes it clear the file marked by the directive is default lib file and ignore everything else.

The impact here is similar to passing noLib on the command line.

So if you add that directive you would need to include the lib reference directives to files as it is equivalent to noLib

the file marked by the directive is default lib file

no-default-lib affects the entire compilation, not just the specific file its in, as shown with the original reported issue.
But I see what you are saying - when using the triple slash to include the types:

if you use noLib you will be including your own type definitions for these

we are basically doing this as mentioned in the docs.
It still seems weird to me that one way works differently to the other. Including the triple slash for lib="dom" affects the entire compilation just as no-default-lib does.
It also seems like order does not matter. Both of these:
/// <reference lib="dom" />
/// <reference no-default-lib="true" />

/// <reference no-default-lib="true" />
/// <reference lib="dom" />

Have the same behaviour.

This directive marks a file as a default library.

This directive instructs the compiler to not include the default library

Whether a file is itself a default library vs. whether it should include the default libs feel like orthogonal concerns…


Uncaught DOMException: Failed to execute 'define' on 'CustomElementRegistry': the name "*" has already been used with this registry

First of, great work on the library! We are really enjoying using it.
We are experiencing an issue with CustomElementRegistry definitions when using dynamic components.
I believe changing the definition instantiation to the following will fix the issue:
if(!window.customElements.get('bs-*')) window.customElements.define('bs-*'', Bs*);
This would prevent double defining elements.
We are happy to help fix the issue and create a pull request if you are open?

Hi Bernard,
I am really happy that you find the library useful. I initially made this for my own personal pert projects and as a way for to learn. I actually counting on community help & wisdom to push this to stable version, so pull request are most definitely appreciated.

Pull request was merged. closing this issue


PostFX of Rope game object

Version

Phaser Version: 3.60

Operating system:

Browser:

Description

Rope game object has PostPipeline component, but it does not invoke this.initPostPipeline() inside constructor, thus it won't have preFX neither postFX instances, imo.

Fixed. You cannot use PreFX on Rope, but Post FX is fine.


Disable auto play

[Suggestion] It would be nice if there was an option to disable auto-play.

Yes that option is on youtube but it was turned on automatically a couple of times in the past, maybe I was only hallucinating but I thought it would be a good feature.


Unable to print checks for customer payments that do not have a invoice

Can the table relations in the check printing "report" be adjusted to not require any kind of receivables document please? Prepayments and "Unknown" payments do not have a receivables document typically.
Ref #1

Can the table relations be adjusted to not require any kind of receivables document please? Prepayments and "Unknown" payments do not have a receivables document typically.
Ref #1

@busyfritz  in the case of Prepayments wouldn't you just create a CreditMemo to represent the desire to refund the prepayment then attach that AR Doc to the refund? Really that should work for any kind of payment.

That could work but then that leaves the customer with a negative balance right?

That could work but then that leaves the customer with a negative balance right?

Testing that right now. Will let you know :)

That could work but then that leaves the customer with a negative balance right?

looks like the CM is not necessary you just point at the Prepayment as your AR Ref document in Documents To Apply.

That could work but then that leaves the customer with a negative balance right?

looks like the CM is not necessary you just point at the Prepayment as your AR Ref document in Documents To Apply.
FYI. I created and SO and applied a Prepayment and released it.
this creates an AR Payment record that you can reference in the Documents to Apply section of the Customer Refund.

Yes, that part works perfectly but because of how to check form is set up, the check won't print. The check "report" relations are setup to require a CM. It just prints blank if there is none. That's the issue.

ah, I see your issue then, sorry for being a bit slow :)
I will see if I can resolve that.

ah, I see your issue then, sorry for being a bit slow :)
I will see if I can resolve that.

Any success with this?

@busyfritz were you ever able to resolve this?


[Telink] Introduce power management

Change overview

Add power management (disabled by default for all examples)
Rework Factory Reset button from "Hold" (3 s) into "Push" (3 times)
Add BUTTON_MANAGER_IRQ_MODE feature
Add CHIP_ENABLE_APPLICATION_STATUS_LED feature

Testing
Tested manually with chip-tool.
Steps:

Run: $ chip-tool pairing ble-thread <...>
Wait till success
Run: $ chip-tool onoff on ${NODE_ID }1
Wait till success
Run: $ chip-tool onoff off ${NODE_ID }1
Wait till success

@andy31415 @Damian-Nordic Could you process merge of PR if everything ok now?


dart migrate 2.11 updates pubspec.yaml to 2.10 maxver

After running dart migrate on my win32 package and choosing apply migration, pubspec.yaml is updated to:
environment:
  sdk: '>=2.9.0 <2.10.0'

However, I'm running it from:
$ dart --version
Dart SDK version: 2.11.0-203.0.dev (dev) (Thu Oct 8 16:48:33 2020 -0700) on "macos_x64"

which means that the resultant package won't work on the build I ran the migration from.
That seems wrong, no?

Perhaps the upper constraint should be <3.0.0 unless they already have a max that is equal to or greater than their current sdk version.

Duplicate of #43757


Upgrading to latest OpenSearch version

Description

Upgrade to latest OpenSearch version.
Update compatibility matrix.

Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Looking into why the integration tests are failing against OpenSearch 2.6.0 and 2.7.0. When I run the tests on my local, they seem to succeed. Happening in #486 as well.

Looking into why the integration tests are failing against OpenSearch 2.6.0 and 2.7.0. When I run the tests on my local, they seem to succeed. Happening in #486 as well.

They're failing because of security permission, no permissions for [] and User [name=admin, backend_roles=[admin], requestedTenant=null] this is the error I get when I try to run the tests against the latest versions of OpenSearch.

They're failing because of security permission, no permissions for [] and User [name=admin, backend_roles=[admin], requestedTenant=null] this is the error I get when I try to run the tests against the latest versions of OpenSearch.

Could be https://github.com/opensearch-project/security/commit/94db5dbb0e2351b6d2b31b2a0969863acbad9ebe ? (security plugin)


failed to build argo 3.6.0 release

Pre-requisites

[X] I have double-checked my configuration
[X] I have tested with the :latest image tag (i.e. quay.io/argoproj/workflow-controller:latest) and can confirm the issue still exists on :latest. If not, I have explained why, in detail, in my description below.
[X] I have searched existing issues and could not find a match for this bug
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened? What did you expect to happen?
👋 trying to build the latest release, but run into some build issue. The error log is as below:

error build log
  Bundling for production...
  Generating listers for workflow:v1alpha1 at github.com/argoproj/argo-workflows/v3/pkg/client/listers
  Generating informers for workflow:v1alpha1 at github.com/argoproj/argo-workflows/v3/pkg/client/informers
  # Force the timestamp to be up to date
  touch pkg/apis/workflow/v1alpha1/zz_generated.deepcopy.go
  # Delete the link
  [ -e ./v3 ] && rm -rf v3
  make: *** [Makefile:659: pkg/apis/workflow/v1alpha1/zz_generated.deepcopy.go] Error 1
  make: *** Waiting for unfinished jobs....

Version(s)
v3.6.0
Paste a minimal workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
brew install -s https://raw.githubusercontent.com/Homebrew/homebrew-core/cefeab8b49d9b0ac97d63be136c251c744bbe1a3/Formula/a/argo.rb

Logs from the workflow controller
n/a

Logs from in your workflow's wait container
n/a

relates to https://github.com/Homebrew/homebrew-core/pull/197750
full build log, https://github.com/Homebrew/homebrew-core/actions/runs/11875018496/job/33091776717?pr=197750

It looks like https://github.com/Homebrew/homebrew-core/pull/197750 was merged. Is this still an issue?

yeah, looks like build parallelization issue. closing for now.


Ability to define pages then sections

So far i really like Vapid and how simple it is, the main thing i would like to see is the ability to define a page and then the sections on a page.

@cece554 Thanks for the submission. If I understand correctly, you'd like to see something similar to #12, nested sections. Feel free to chime in there.


Refactor fluid-fetch/replay-tool/debugger to support bringing additional functionality to debuger

#4141 tracks integrating the entire document bug repro workflow into the debugger tool (downloading, anonymizing, and previewing a buggy/problematic "third-party" i.e. Bohemia document to share with us).  Currently this functionality is split among fluid-fetch, replay-tool, and debugger.  Typically debugger could take a dependency on fluid-fetch/replay-tool and leverage their functionality that way, but fluid-fetch and replay-tool are currently published as @fluid-internal (because they are internal tools) whereas debugger is published under @fluidframework despite essentially being an internal tool because Bohemia uses it (@fluid-internal packages are not published to npmjs).
The functionality that debugger needs must find a home inside a @fluidframework package, or these @fluid-internal packages need to be published in a location that Bohemia can still consume (e.g. offnet) and Bohemia's package feeds need to be set up to find them.

The sanitizer/anonymizer functionality currently in the debugger tool should probably also be moved as part of refactoring here, though there aren't any direct consumers right now

The sanitizer/anonymizer functionality currently in the debugger tool should probably also be moved as part of refactoring here, though there aren't any direct consumers right now

The tool-utils library package will be a good place to shared these functionalities

The tool-utils library package will be a good place to shared these functionalities

Also #630 is a related issue

Also #630 is a related issue

Noting to also revert the part of #4705 that removes the MergeTreeDeltaType enum usage


PROD SUPPORT: Notice: 223 Notice not  getting generated for a case

Describe the bug
"Not able to do e-sign to sent 223 Notice through RPAD and SMS for a single case as notice is not getting generated. When we are trying to download the PDF, it is showing 400 error code with "Invalid filestore or tenantid"
Filing Number - KL-000287-2024
Screenshots

https://github.com/user-attachments/assets/98f46027-34fc-4529-a4b1-8e7222501c96

Will Ask Courtstaff to generate one order/notice


fix mesh_data_tool vertex's edge error

fix #27508

I do not have any experience with MeshDataTool but I believe that this is not right.

Superseded by #31618, but thanks for the contribution nevertheless!


using global::PowerToys.GPOWrapper namespace errors

Microsoft PowerToys version
0.76 dev
Installation method
Dev build in Visual Studio
Steps to reproduce
Open several .cs files under *src\settings-ui\Settings.UI\ViewModels*
✔ Expected Behavior
No errors
❌ Actual Behavior
CS0400

CS0246 and CS0103

Note: this is seen throughout most of the files in this folder.

@Jay-o-Way
You can ignore this. The real code is autogerated on the first build. So before you see the errors. At the end it is important that it works.


Add array for recording array-wide quality values

Adds a new array for tracking array-wide quality values, in addition to per-antenna values. Issuing a pull request to build tests for added code.

Since you've added new parameters to uvcal, we also need to remake the docs. I can do that once all the code changes are done, or I can walk you through it if you'd like.

This looks good to me now. I'm going to merge it.


Kodi Connection Error

I use raspberry B+ with OSMC + hyperion. LEDS are controlled with ws2811 via arduino.
When I use sudo service hyperion status this is what I get
osmc@osmc:~$ sudo service hyperion status
● hyperion.service - Hyperion Systemd service
Loaded: loaded (/etc/systemd/system/hyperion.service; enabled)
Active: active (running) since Fri 2016-04-01 21:24:58 CEST; 6min ago
Main PID: 757 (hyperiond)
CGroup: /system.slice/hyperion.service
└─757 /opt/hyperion/bin/hyperiond /etc/hyperion.config.json
Apr 01 21:30:19 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
Apr 01 21:30:29 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
Apr 01 21:30:39 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
Apr 01 21:30:49 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
Apr 01 21:30:59 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
Apr 01 21:31:09 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
Apr 01 21:31:19 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
Apr 01 21:31:29 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
Apr 01 21:31:39 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
Apr 01 21:31:49 osmc hyperiond[757]: KODICHECK ERROR: Kodi Connection error (0)
I can't use hyperion because of ( I suppose) this error. Any help, any ideas please?
Hyperion config file
http://pastebin.com/Npnnj3E3

In Kodi
Enable "Allow remote control by programs on this system"
and "Allow remote control by programs on other systems" (you need both, even on the same machine)
navigate ->System->Services->Remote control

@DarkHunterer
News?

I managed to get rid of that error before the first answer, so I doubt if that was the cause.
I think I just removed hyperion and run install script again.
Howerver, I found 2 more things:

I was cursing, because I couldn't make hyperion work. It turned out that baudrate I used was propably too high (460800). So this is propably not because of hyperion, just raspberry/arduino stuff.

I use baudrate 115200 now and it looks like its working perfectly. However, when I run rgb test video, entire screen of color looks decent but when there is something like screen divided to 4 pieces with diagonals, the colors are not right. The "left" color overlaps most of the leds, like it is the most important. So it looks like it is not calibrated right, any help with that?

It seams you have to adjust the colors. Did you calibrated it before? Play around with color settings maybe that helps

This doesn't like color adjustment problem for me.
http://postimg.org/image/kxkvwwd3h/f32ea702/
I would say that the colors are 90 degrees ahead from what it should be.
Any solution for that?

https://github.com/tvdzwan/hyperion/wiki/FAQ-Frequently-Asked-(Problems)-Questions
Point 3, if not may be 5

@brindosch Thank you for you answers.
Well, of course I saw FAQ and played with HyperCon MANY times and I think it can be fault of HyperCon.
This is view of my settings, including offset http://imgur.com/82NXAQE
What I noticed is that left color is the most "important". It overlaps the entire top very fast, and the right color is very easly overlaped by the top.
Can you recommend something? Did you change something in Hyperion and forgot to refresh HyperCon maybe?

hm, no and no. What happen if you play the red dot video, is the position at the right place?
http://insaneboard.de/blog/wp-content/uploads/2014/10/RedDot-720p.zip

Great example. It shows the "power" of the left side.
And here is the effect
V_20160406_230054.mp4.zip

wow, completely screwed. I will have a look.

Thank you for your time, let me know.

How big is your grabbing are of your frame gabber?

"framegrabber" :
{
"width" : 80,
"height" : 45,
"frequency_Hz" : 10.0,
"priority" : 890
},
I think this is it? I didn't change anything, so I think these are defaults

The new Hyperion version contains a new driver for ws2811 for gpio direct connection, could you test this? I don´t get it.

can you try this

"framegrabber" :
{
"width" : 720,
"height" : 576,
"frequency_Hz" : 10.0,
"priority" : 890
},
2. try another direction...
just test this, and than we'll see

@tpmodding it didn't really change anything. And another direction is well... another direction.
@brindosch So what I did is : I used HyperCon with "ws281x" type. It's the one, right? I assume that it gives output on gpio18. However, when I upload file with settings for ws281x there is this output...
● hyperion.service - Hyperion Systemd service
Loaded: loaded (/etc/systemd/system/hyperion.service; enabled)
Active: failed (Result: start-limit) since Fri 2016-04-08 21:22:08 CEST; 7s ago
Process: 652 ExecStart=/opt/hyperion/bin/hyperiond /etc/hyperion.config.json (code=killed, signal=SEGV)
Main PID: 652 (code=killed, signal=SEGV)
Apr 08 21:22:07 osmc hyperiond[652]: "leds" : 66,
Apr 08 21:22:07 osmc hyperiond[652]: "name" : "MyHyperionConfig",
Apr 08 21:22:07 osmc hyperiond[652]: "type" : "ws281x"
Apr 08 21:22:08 osmc hyperiond[652]: }
Apr 08 21:22:07 osmc systemd[1]: hyperion.service: main process exited, code=killed, status=11/SEGV
Apr 08 21:22:07 osmc systemd[1]: Unit hyperion.service entered failed state.
Apr 08 21:22:08 osmc hyperiond[652]: Can't open /dev/mem: Permission denied
Apr 08 21:22:08 osmc systemd[1]: hyperion.service start request repeated too quickly, refusing to start.
Apr 08 21:22:08 osmc systemd[1]: Failed to start Hyperion Systemd service.
Apr 08 21:22:08 osmc systemd[1]: Unit hyperion.service entered failed state.
So hyperion can't start.
Hyperion IS working when I use adalight config. So I assume that the config file for "ws281x" is broken or something.

I´m sorry it is "noticed" if you use the script with usb or ws281x argument it will do the right steps.
https://github.com/tvdzwan/hyperion/wiki/Installation-on-OSMC
osmc has some problems with pwm signal, but not all pwm people are affected
ping @penfold42
Is this a "known" problem with osmc for ws2811 leds? Do he need another distro?

Ok... So the problem is over. You don't have to bother writting back for my problems which I had till this time...
I was stubborn and I found the solution after using analyzer... The solution was so obvious...
I am going to write it tomorrow, because I really believe this is going to save much time to people in the future...
However, I still have to check if the ws281x option is working, because now Im using usb->arduino again.
So thank you very much for trying to help me.
Finally Hyperion working... It looks so amazing. Nice job on this project!

Glad that it is working. pwm and osmc is still.. in some way a problem. But we can´t do anything. It is working with Raspbian and OpenELEC.

Ok, so basically what I did was:

Read the signal with analyzer. It turned out that the PWM indeed is working. I saw that the '1' level takes like 1.0~1.2 usecond.

So I had to check the documentation of ws2811 if it is right period or it is not. So generally it was not exact with the documentation, not even very close but there should be some flashes from time to time, because some of the bits timing were correct.

And the moment I read the documentation was the moment when I realized what I have done... When I mounted the leds I counted physical leds... Not the ws2811 chips... The documentation shows 4 different ways of using ws2811, 5V - 1 led ; 5V - 3 leds parallel ; 12V - 1 led ; 12V - 3 leds parallel.
So basically... I thought I am using 66 leds when I in fact was using 22...

tl;dr:
The ws281x chips can be mounted on strip in 2 different ways : 1 led 1 chip or 3 leds 1 chip. It is really worth checking what is your strip like...
I think it is worth adding to FAQ. Something like "How many leds do I have - that many as you have ws chips"

hey @DarkHunterer
that is realy interesting! didnt know that fact in 3)! but i think it is only on ws2811??
do you have a stripe that likes like this? http://images.google.de/imgres?imgurl=http%3A%2F%2Fg01.a.alicdn.com%2Fkf%2FHTB1rOvcJpXXXXcoXFXXq6xXFXXXK%2F1M-5M-WS2811-RGB-LED-Strip-12V-SMD-5050-Adressable-Pixel-Strip-Black-or-White-PCB.jpg&imgrefurl=http%3A%2F%2Fwww.aliexpress.com%2Fstore%2Fproduct%2F1m-5m-WS2811-RGB-LED-Strip-12V-SMD-5050-Adressable-Pixel-Strip-Black-or-White-PCB%2F1755004_32461755828.html&h=506&w=700&tbnid=sdt_yU1gLVevJM%3A&docid=_TasIXaPBFoiwM&ei=4VkKV7bQIcrB6AShzpyYAw&tbm=isch&iact=rc&uact=3&dur=1194&page=5&start=86&ndsp=24&ved=0ahUKEwj2xOePm4TMAhXKIJoKHSEnBzM4ZBAzCBUoCTAJ
cause the ws2812b has is chip IN the led! so that it can control only one led...
i thought hole time there a nearly "same" as shown here:
http://rgb-123.com/wp-content/uploads/2013/08/WS2811vsWS2812b.jpg
how it is possible to check? if you say hyperion you have 2 leds...and make them for example red with "hyperion-remote -c red" than it shoul only flash 2 or 6 leds? right?

@tpmodding This is exactly the strip I bought. And you clearly cannot control each led, I control them by groups of 3.
If I would know that 2812b has chip inside, I would pick it instead :/
You are right, if you would say hyperion that there are only 2 leds and make them flash, 6 first would turn on.
I can check it tomorrow on hyperion, but I did exactly same thing with FastLed library yesterday.

@DarkHunterer ok thanks! that would be nice to know, to add it to the FAQ.
btw ## i don't recommend ws2811 or ws2812b  or any PWM led!!! always use leds with data AND CLOCK!!
then you have never those problems :+1: :)

Yuk - sounds like they were being cheap.
Just so im clear, is each physical led RGB but wired in parallel in groups of 3 to a single 2811 chip ?

@tpmodding Both options have their pros and cons. It is in fact easier to use chips with clock for hyperion but if you have real-time hardware clock on your board it is easier to just use PWM controlled strip.
I checked the setting with just 2 leds in hyperion.config file. So basicaly I see that the first 2 groups (6 leds) are flashing in desired way. But then there are additonal 6 leds turned on in constant color, and than there is color again.
 When I change color for GREEN, the w w w g g g scheme stays the same. I suspect it is connected with "Ada"+"3 bytes of information" word, there goes 6 bytes with it = information of 2 colors.
(The colors were not calibrated, so it doesn't have to be green for real)
@penfold42 Yes, there are groups of 3 parallel leds connected to single 2811 chip.
And they are in fact cheaper but not that much cheaper than ws2812b, I don't understand why the price be any circumstance here. I picked it because there wasn't clear information about the connection or I just didn't understand it.

tpmodding want to say that you may never opend this issue and invest time in fixing - because of this.
Also OSMC guys screwed something up that still make issue and we have no idea what is going wrong.
So just time eating.

@DarkHunterer the price-thing has nothing to do with you :) but most people try to make his ambilight as cheap as possible and forget the "quality" and than came here with thousend of issues....
but it is "good" to know that you found the problem...
and if you now setup your config knowing 3 leds per chip, does it work without problems?

Yes, hyperion works perfectly now.
So thank you for your time and answers guys, I believe this is solved already :)
You can add a note to FAQ for the future someone like me, who didn't realize that has 3 leds parallely.
Great project.

Good


Refactor crate to be more composable and simple

Right now the crate contains a bunch of types and typedefs for different mutabilty and usecase scenarios.
It would be nice if the API could be simplified by offloading some of the duplication and different behaviors to type paramers instead.
Possible refactors:

Anything using the Erased trait should be made generic over the actual trait, to support varitions like Erased+Send or Erased+Sync

This especially includes the *Ref type defs

OwningRef and OwningRefMut are potentially mergeable by adding a type parameter for the mutabilty: OwningRef<*const T> and OwningRef<*mut T>. Or with specially defined marker types: OwningRef<Shared> and OwningRef<Mutable>.
OwningRef might possibly be implemented in terms of an OwningHandle.

I don't think it's useful to try to merge OwningRef and OwningRefMut, since Rust doesn't provide any good way to abstract over mutability. Even if you try, you'll run into the issue that methods have to change receiver type (&self vs &mut self)


[FR] Clicking bracket should select all content

Issue Type: Feature Request
In Eclipse this is very useful:
On line:
asd= (1,2,34,5)
double clicking between ( and 1 selects everything between brackets.
Same goes for:  [ { " ' ( """ /*
everything that has a pair, in any language.
VS Code version: Code 1.52.1 (ea3859d4ba2f3e577a159bc91e3074c5d85c0523, 2020-12-16T16:34:46.910Z)
OS version: Windows_NT x64 6.1.7601

I am new to opensource and would like to work on this issue.
Can you please assign me this issue?

There is already a command called "Select To Bracket" that does this, but there is no mouse gesture for this.


Hold research sessions with Service Managers and Product Owners to identify needs for the dashboard

Hi @srich-DWP. Have you arranged this yet? It would be good to get the 'Prove you can apply team' and others involved.
cc: @gavinelliott @tsmorgan

Started sessions with Service Managers and Product Managers. Outputs from 3  sessions have been captured, one more session next Wednesday with a Service Manager.

Organised a session with Catherine Hope - analytics in Newcastle, to discuss ways in presenting the data to a dashboard. E.G which colours in graphs, pie charts, etc work well with users

This issue can be closed.
A report from the UR sessions will be put together


Overloading unary operators throws lowering error

Reporting a bug

[x] I am using the latest ~released~ master version of Numba (most recent is visible in
the change log (https://github.com/numba/numba/blob/master/CHANGE_LOG).
[x] I have included below a minimal working reproducer (if you are unsure how
to write one see http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports).

Overloading unary operators invert (~) and pos (+) fails.
import operator
import numba
from numba.extending import (type_callable, models, register_model, overload,
    lower_builtin)

class Foo:
    def __invert__(self):
        return self

class FooType(numba.types.Type):
    def __init__(self):
        super(FooType, self).__init__(name='Foo')

foo_type = FooType()

@type_callable(Foo)
def type_foo(context):
    def typer():
        return foo_type
    return typer

@lower_builtin(Foo)
def impl_foo(context, builder, sig, args):
    return context.get_constant_null(sig.return_type)

register_model(FooType)(models.OpaqueModel)

@overload(operator.invert)
def invert_foo(f):
    if f == foo_type:
        return lambda f: f

# def myfunc(foo):
#     return ~foo

# @overload(myfunc)
# def myfunc_overload(f):
#     return lambda f: ~f

@numba.njit
def example():
    f = Foo()
    #myfunc(f)
    ~f

example()

numba.errors.LoweringError: Failed in nopython mode pipeline (step: nopython mode backend)
No definition for lowering <built-in function invert>(Foo,) -> Foo

File "ss.py", line 54:
def example():
    <source elided>
    #myfunc(f)
    ~f
    ^

[1] During: lowering "$0.4 = unary(fn=<built-in function invert>, value=f)" at ss.py (54)

Thanks for the report. This is sufficient to reproduce:
iimport operator
from numba import types, njit
from numba.extending import overload

@overload(operator.not_)
def ol(x):
    if isinstance(x, types.UnicodeType):
        def impl(x):
            return 10
        return impl

@njit
def foo(a):
    return not a

print(foo("123"))

Fixed in #4174


Monitoring: Bump Grafana image

to a patched version of v6.0.1 and extend configuration.
The patched version of Grafana does not contain phantomjs.
External snapshot sharing is now disabled.
Alerting functionality is now disabled.
Release note:

NONE

FYI: @ThormaehlenFred


fix #3 + typos etc.

fix #3

Hey @DaniMorales97
Could you please make a new PR or edit the same, changing the branch where you want to merge the PR from main to sprites so that we can keep both the approaches in different otherwise i have to merge this branch in the base branch.
So please edit or make a new PR, changing the target branch from main to sprites.

And one more thing i saw the changes which you have made, they are really amazing now the code is  much more readabele and easily understandable.
Couldn't wait to add more amazing features in the game once the PR is merged.

Hey
I have merged your PR.
Thank you so much for your time and efforts @DaniMorales97

No worries, I'm enjoying it and learning along the way so its great


Does not stop recording upon boot

On boot, it won't let you stop recording your command, the LED just keeps pulsing. But when you open the terminal and run the program, it works fine. Seems like a normal annoying, fixable bug right? NO!!! Because when you try to debug it, you can't because it can't run the debugger from boot, and when you turn on voice prompts to see what it's doing, it works fine!!!

Okay I went through each voice prompt to see which one was magically making it work and then I just made it do the voice prompt the first time it boots and then never again and that seems to be working and I don't want to touch it because it might break.