Replace glass_knock.ogg

About the PR
Replace glass_knock.ogg with a less clipped version New version has peak compression, reduced high-end frequencies, reverb moved to low-end, and a crossfade.
Why / Balance
The original glass_knock.ogg file was recorded poorly and clipped.  This produces an unpleasant clicking sound that most people will never notice or care about.
The result is a reduced clicking sound to due to clipping, reduced clicking when looping the file, and a less aggressive, somewhat muddier sound. Knocking will likely sound somewhat less frantic.
Technical details
replaced glass_knock.ogg with a newer version
Media
I cannot attach .ogg files directly in discord so here is a link to a discord message with both versions: https://discord.com/channels/310555209753690112/770682801607278632/1308749760798130226
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes
none
Changelog
:cl:

tweak: replaced window knocking sound with a less clipped, more subtle version

Make sure to update the attribution!

I have no idea what this means T_T

I have no idea what this means T_T

There should be an entry in /Resources/Audio/Effects/attributions.yml that lists the copyright information for the sound effect. The spec is here if you're wondering.
I am mildly concerned that I can't seem to find the original attribution for glass_knock.ogg, but there's no time like the present!

I have no idea what this means T_T

There should be an entry in /Resources/Audio/Effects/attributions.yml that lists the copyright information for the sound effect. The spec is here if you're wondering.
I am mildly concerned that I can't seem to find the original attribution for glass_knock.ogg (it's probably from cmss13, as many of the other glass sound effects originated from there, though you should probably find the providence on your own), but there's no time like the present!

Thanks.  I did find the same file in cmss13, but their github doesnt have an attribution for it.  I really have no idea how I would track down the original and first use of the file.  Should I just go ahead with a cmss13 attribution?

Thanks. I did find the same file in cmss13, but their github doesnt have an attribution for it. I really have no idea how I would track down the original and first use of the file. Should I just go ahead with a cmss13 attribution?

Yep! Cite it as originally coming from cmss13, and then cite yourself for editing/modifying it.
Example:
https://github.com/space-wizards/space-station-14/blob/cb246f5d7d79ba8c8b7a250a063abf17f0e6d969/Resources/Audio/Items/attributions.yml#L51-L59

Last I heard a significant number of cm13 files were illegally obtained from AvP or sometimes Alien Isolation so this will need triple checking.

Sounds like we have a bigger situation on our hands...


keep getting modulenotfounderror for onepassword

Scenario & Reproduction Steps
import asyncio
import os
from onepassword.client import Client

async def main():
    # Gets your service account token from the OP_SERVICE_ACCOUNT_TOKEN environment variable.
    token = os.getenv("OP_SERVICE_ACCOUNT_TOKEN")

    # Connects to 1Password. Fill in your own integration name and version.
    client = await Client.authenticate(auth=token, integration_name="My 1Password Integration", integration_version="v1.0.0")

    # Retrieves a secret from 1Password. Takes a secret reference as input and returns the secret to which it points.
    value = await client.secrets.resolve("op://vault/item/field")
    # use value here

if __name__ == '__main__':
    asyncio.run(main())

i am using this in step 4 of readme.md
and i keep getting
 File "/Users/kimsia/Projects/windmill_dev/00_first_try/onepassword.py", line 3, in <module>
    from onepassword.client import Client
  File "/Users/kimsia/Projects/windmill_dev/00_first_try/onepassword.py", line 3, in <module>
    from onepassword.client import Client
ModuleNotFoundError: No module named 'onepassword.client'; 'onepassword' is not a package

i used
pip install git+ssh://git<EMAIL_ADDRESS>
Actual Behavior
 File "/Users/kimsia/Projects/windmill_dev/00_first_try/onepassword.py", line 3, in <module>
    from onepassword.client import Client
  File "/Users/kimsia/Projects/windmill_dev/00_first_try/onepassword.py", line 3, in <module>
    from onepassword.client import Client
ModuleNotFoundError: No module named 'onepassword.client'; 'onepassword' is not a package

Expected Behavior
that it works?
SDK version
pip install git+ssh://git<EMAIL_ADDRESS>Additional information
No response

Hi @simkimsia! Thanks for reporting this bug. We will look into fixing it soon, in the next release of the SDKs

@Marton6 were you able to reproduce? For me this works fine.
@simkimsia What happens when you try python3 -m pip list. Does onepassword show up?

sorry i know why.


Adds a widget to the dashboard

As promised, here's the (surprisingly small) widget PR as mentioned in #1 . Please note however that this is the first time I got my hands on Craft plugins, so maybe I made some rookie errors.
It would be nice to display a success message after purge, or a zone selector.

Thanks for submitting this, @Radiergummi! Would you mind if I merged this into a branch and made a few adjustments? I'd like to work confirmation messages in like you've pointed out, and maybe spiff up the widget UI to visually separate a global purge from individual URLs.

I definitely don't mind, I'm happy if those few lines of poorly cobbled-together code provide you with a starting point 😄

@Radiergummi, I appreciate your jump-start here and just merged your work in with some updates (mostly cosmetic) called out in 492ad92595310216bb81f4d1f50013482538017c. Let me know if you've got an issues or objections!

It's gorgeous. Thank you 😄


file_watch exits with ERROR - Cannot instantiate typing.Optional

After an ansible-galaxy collection install ansible.eda I attempt to use the file_watch source plugin:
- name: Rulebook to do something
  hosts: localhost
  sources:
      - ansible.eda.file_watch:
          path: "files/"
          recursive: false
          ignore_regexes: [ '.*\.o' ]
  rules:
      - name: trigger on range
        condition: event.type == 'FileCreatedEvent'
        action:
           run_module:
               name: debug
               module_args:
                   msg: "I just saw {{ event.src_path }}"

Upon startup, the rulebook fails:
$ ansible-rulebook -i inventory -r r1.yml
2023-08-13 11:26:47,534 - ansible_rulebook.engine - ERROR - Source error Cannot instantiate typing.Optional
2023-08-13 11:26:47,534 - ansible_rulebook.engine - ERROR - Shutting down source: ansible.eda.file_watch error : Cannot instantiate typing.Optional
2023-08-13 11:26:47,551 - ansible_rulebook.app - ERROR - Cannot instantiate typing.Optional
2023-08-13 11:26:47,552 - ansible_rulebook.cli - ERROR - Terminating One of the source plugins failed

With the help of this issue I somewhat fix it with this patch:
--- /tmp/old	2023-08-13 11:29:16.000000000 +0200
+++ /Users/jpm/.ansible/collections/ansible_collections/ansible/eda/extensions/eda/plugins/event_source/file_watch.py	2023-08-13 11:29:08.000000000 +0200
@@ -35,7 +35,7 @@
     root_path = args["path"]

     class Handler(RegexMatchingEventHandler):
-        def __init__(self: "Handler", **kwargs: Optional(list[str])) -> None:
+        def __init__(self: "Handler", **kwargs: Optional[list[str]]) -> None:
             RegexMatchingEventHandler.__init__(self, **kwargs)

         def on_created(self: "Handler", event: dict) -> None:

By "somewhat" fix it, I mean that the watch part seems to work, but I see the following when the rule fires:
$ ansible-rulebook -i inventory -r r1.ym
2023-08-13 11:31:16,348 - asyncio - ERROR - Exception in callback FilteredQueue.put_nowait({'change': 'modified', 'meta': {'received_at': '2023-08-13T09:31:16.348116Z', 'source': {'name': 'ansible.eda.file_watch', 'type': 'ansible.eda.file_watch'}, 'uuid': 'cba22dd7-98a...-112734581631'}, 'root_path': 'files/', 'src_path': '/Users/jpm/tst/files', ...})
handle: <Handle FilteredQueue.put_nowait({'change': 'modified', 'meta': {'received_at': '2023-08-13T09:31:16.348116Z', 'source': {'name': 'ansible.eda.file_watch', 'type': 'ansible.eda.file_watch'}, 'uuid': 'cba22dd7-98a...-112734581631'}, 'root_path': 'files/', 'src_path': '/Users/jpm/tst/files', ...})>
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 80, in _run
    self._context.run(self._callback, *self._args)
  File "/Users/jpm/take/training/rulebook/env.v3/lib/python3.11/site-packages/ansible_rulebook/engine.py", line 93, in put_nowait
    self.queue.put_nowait(data)
  File<EMAIL_ADDRESS>line 143, in put_nowait
    raise QueueFull
asyncio.queues.QueueFull
localhost | SUCCESS => {
    "msg": "I just saw /Users/jpm/tst/files/bla4"
}

Closing as this fix is no longer needed with current codebase.


feat: add robotcode support

Base on: robotcode

thanks please read our rule for new server
https://github.com/neovim/nvim-lspconfig/blob/master/CONTRIBUTING.md#adding-a-server-to-lspconfig

thanks please read our rule for new server https://github.com/neovim/nvim-lspconfig/blob/master/CONTRIBUTING.md#adding-a-server-to-lspconfig

I already read that. Is there something wrong?


[docs] schema docs generation should use the snake_case to avoid confusion

Describe the bug
An example reference elements using UpperCamelCase when the official examples use snake_case in template definition.
Docs:
Definitions

IambicManaged: An enumeration. Must be one of: ["undefined", "read_and_write", "import_only", "enforced", "disabled"].

Template Defintiion:
Link
template_type: NOQ::Okta::Group
iambic_managed: enforced
idp_name: development
properties:
  name: cloud_admins
  description: Cloud Administrators
  members:
    - username<EMAIL_ADDRESS>    - username<EMAIL_ADDRESS>
To Reproduce
Steps to reproduce the behavior:

Check above reference docs vs template.

Expected behavior
Official docs should use snake_case in docs generation.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.
Community Engagement
Your vote counts! Please support this bug report by adding a 👍 reaction to the original issue, which will aid the community and maintainers in addressing this problem.
Please refrain from adding "+1" or "me too" comments, as these create unnecessary noise for issue followers and do not help in prioritizing the issue. If you wish to contribute to solving this issue or have submitted a pull request, please leave a comment.

This commit seems to fixed the issue. Now the docs is consistent using snake_case. (at least for iambic_managed)


Control: added pad_msg to local_view and updated control sub modules

@sjiang2018 @xiaoxq Will merge this one in as hotfix and do a through refine.


Missing TV guides listing

Source
?
Notes
One more thing. There is no EPG guides for those list:
AFV FAMILY*
AMC IFC FILMS PICKS*
AMC PRESENTS*
CONTV ANIME*
CRACKLE*
DEFY TV*
DOVE CHANNEL*
FAIL ARMY (RED BOX)*
FOX WEATHER LIVE *
HI-YAH*
REWIND TV*
THIS TV NETWORK*

Duplicate #1062


CRC-Error / How to use it with UF-Instant?

Hey, great project, thought i won't be able to use the UF Instant at any time again.
I digged a bit into the UF-Instant but then i realised, that changes were not persistant on the UF Instant.
Currently I am facing a CRC error, no Matter what i am trying to change.
Enabling Telnet/SSH is easy.
did the following on my UDM Pro
ifconfig eth9:2 <IP_ADDRESS> netmask <IP_ADDRESS> up
telnet <IP_ADDRESS>
flash get GPON_PLOAM_PASSWD
flash get GPON_SN
ubnt-syscfg service list
ubnt-syscfg service set ssh 1 22

but then i can't apply my config:
UFiber.Configurator --host <IP_ADDRESS> --user ubnt --pw ubnt --port 22 --slid<PHONE_NUMBER>12--vendor SCOM --serial 01234567
or in a reduced way:
UFiber.Configurator --slid<PHONE_NUMBER>12 --vendor SCOM --serial 01234567

Will throw an exeption like this:

Unhandled exception: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation.
---> System.InvalidOperationException: Invalid data, CRC doesn't match

Thanks,
Dominic

Additional Firmware Info
# ubnt-sys cfg firmware releaseTime
1572601200
# ubnt-syscfg firmware version
v4.1.0
# ubnt-syscfg firmware fullString
UF-ONU.UF-INSTANT.v<IP_ADDRESS>9-b87.191101.0940
# ubnt-syscfg firmware string
v<IP_ADDRESS>9-b87.191101.0940
# ubnt-syscfg firmware hash
359-b87

Hello @Dom1n1c has mention in the project's Readme and also in this issue #2 the UF-Instant is unfortunately actually not supported yet
If you are curious and if you want / can help the author you can take a look in this issues #1: there is a bit of discussion with the  @galvesribeiro and he explains at what point he is at on his testing with UF-Instant !

in case you are wondering how to access the original firmware for Instant, Loco, NanoG or UF-Wifi, this is quite simple:
(I don't own a OLT, 4.3.3 seems to be the latest version.)
curl -# https://dl.ui.com/firmwares/ufiber/UF-OLT.v<IP_ADDRESS>46.210622.0853.tar | tar -xv squashfs.tmp
unsquashfs -f squashfs.tmp -e /usr/share/onufw/ && mv squashfs-root/usr/share/onufw/UF-ONU.* ./ && rm -rf squashfs-root squashfs.tmp

But still no luck with the CRC check, but i am wondering, why it should be wrong on a stock firmware:
7a607143  fw-12112021-082051.bin  BAD 7a607143 != 12112021
cc9f48fa  fw-12112021-083545.bin  BAD cc9f48fa != 12112021
958613a4  fw-12112021-084209.bin  BAD 958613a4 != 12112021
958613a4  fw-12112021-084220.bin  BAD 958613a4 != 12112021


Building with Cray compiler fails: no matching function for call to 'to_string'

Description
I get errors from the cray C++ compiler:
/ccs/home/jschuchart/src/slate/slate/src/print.cc:196:30: error: no matching function for call to 'to_string'
  196 |         + ", uplo = "      + to_string( A.uplo() )
      |                              ^~~~~~~~~
/ccs/home/jschuchart/src/slate/slate/src/print.cc:222:6: note: in instantiation of function template specialization 'slate::print<float>' requested here
  222 | void print(
      |      ^

I get similar errors for Layout, Uplo, and Op.
I assume that uplo() returns an enum, which is not automatically cast to int. Implicit promotion of enums to int is only allowed for unscoped enums (I couldn't find the definition of Uplo, Layout, etc.
Steps To Reproduce
On Frontier, build with the Cray compiler.
Environment
$ module list

Currently Loaded Modules:
  1) craype-x86-trento    4) perftools-base/22.12.0                  7) cray-dsmml/0.2.2   10) cmake/3.23.2             13) craype/2.7.23          16) hsi/default        19) cray-libsci/23.12.5
  2) libfabric/<IP_ADDRESS>   5) xpmem/2.6.2-2.5_2.22__gd067c3f.shasta   8) PrgEnv-cray/8.3.3  11) rocm/5.7.1               14) cce/17.0.0             17) lfs-wrapper/0.0.1  20) cray-mpich/8.1.28
  3) craype-network-ofi   6) cray-pmi/6.1.8                          9) boost/1.79.0       12) craype-accel-amd-gfx90a  15) darshan-runtime/3.4.0  18) DefApps/default

This seems to be fixed.


Allow customizing of Switch thumbs

Fixes #2621

@Hixie

travis is angry

I'm going to try this a different way.


Mail stopped coming in or going out

Describe the bug
Mail is not going out or coming in. Here is the notification I have:

Failed to send some messages
Errnoexception: android_getaddrinfo failed: EPERM (Operation not permitted)

I noticed I haven't been getting emails in my K-9 inbox for 5 days, while I can see them fine in Thunderbird.
Environment (please complete the following information):

K-9 Mail version: 5.726
Android version: 11
Device: Pixel 4a
Account type: IMAP

Additional context
I am using PrivateEmail.com.
Logs
k9-log.txt

From the log: java.lang.SecurityException: Permission denied (missing INTERNET permission?)
Looks like the app doesn't have the internet permission. The app does request this permission. And on vanilla Android you can't deny/revoke this permission. So either you're using a custom ROM that provides this as a feature or your installation is somehow broken. In that case try uninstalling, then reinstalling the app.

From the log: java.lang.SecurityException: Permission denied (missing INTERNET permission?)
Looks like the app doesn't have the internet permission. The app does request this permission. And on vanilla Android you can't deny/revoke this permission. So either you're using a custom ROM that provides this as a feature or your installation is somehow broken. In that case try uninstalling, then reinstalling the app.

Oh weird, it's somehow been denied Internet permissions (as well as Sensors)... even though it was working a week ago. Odd. Well, yes, I'm using GrapheneOS, forgot to mention that.
That solved my issue, thanks @cketti!

Oh weird, it's somehow been denied Internet permissions (as well as Sensors)... even though it was working a week ago. Odd. Well, yes, I'm using GrapheneOS, forgot to mention that.
That solved my issue, thanks @cketti!


Add lints include/exclude flag to executable

Added option to include/exclude lints from the command line.
Found it useful for some quick analysis on a large number of certs.

This looks fine overall to me, though I think I might it be better off living in https://github.com/zmap/zcertificate, which handles a number of additional pieces of customizability versus the included (simple) zlint CLI.


Dynamic library loader can't find PromiseKit on tvOS using Carthage integration

When using Carthage to integrate PromiseKit on tvOS, I get a dyld: Library not loaded at app launch when the dynamic library loader tries to load PromiseKit.
I was able to fix the issue by changing some build settings, and have submitted a pull request #408 with those changes. The issue is discussed further in the pull request.

Merged.


Button elements within window element non-functional

This was figured out and fixed (something to do with how the window_area steals input to prevent propagation behind it).


Fix dicom reader to merge dicom slices

Also fix guard check on reader existence in ReaderFactory.

seems ok, but don't know

This at least fixes the raw file load issue. The DICOM stuff works but I may tweak/refine the interface.

cool thx. What was the deal?

https://github.com/Kitware/paraview-glance/pull/226/files#diff-0685cf42d9c2f170eef4fccbbfc61511L168

hum, ok so I broke it... ;-)

Haha, well what matters is that it's fixed now :) (whenever I decide the dicom stuff is good to merge)

Yeah that's perfect.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[e2e]: generic workflow_dispatch main tagname slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11829568911
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.tagname.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.tagname.slsa3.yml
Trigger: schedule
Branch: main
Date: Thu Nov 14 02:30:05 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/11849143018
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.tagname.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.tagname.slsa3.yml
Trigger: schedule
Branch: main
Date: Fri Nov 15 02:33:56 UTC 2024
Tests are passing now. Closing this issue.


feat(#25): Handle multiplication and loading variables

Handle IMUL and ILOAD opcodes.
Closes: #25.

History:

feat(#25): convert multiplication
feat(#25): apply mul handler
feat(#25): use same LoadHandler for all *LOAD opcodes
feat(#25): fix all qulice suggestions

PR-Codex overview
Focus of this PR:
This PR focuses on adding a new Mul class to handle multiplication operations in the opeo module.
Detailed summary:

Added a new Mul class in the opeo module to handle multiplication operations.
Updated the DecompilerMachine class to include a MulHandler for handling the IMUL opcode.
Added tests for the Mul class to ensure proper functionality.

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

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 3min)


Fixes #214. User can select the host from available list in volume provision service

Depends on opensds/orchestration #99
Do not merge this without merging above
After Host management has been introduced, user can select a host to attach to a volume in the volume provision orchestration service instead of entering the Host Info.
Select Host

Prompt user to add a host if no host available

Is to possible to keep the "Add Host" link, even when some hosts are created/available for the user?

Is to possible to keep the "Add Host" link, even when some hosts are created/available for the user?

@joseph-v Yes I can display the "Add Host " link even if hosts are available

LGTM


Limit the number of characters typed

Hello,
More of a suggestion than an issue.
*Limit the number of characters that can be typed. ie. numCharacters = 1000.
*Expose the number of characters typed. So it can be displayed in real time to the user. (observable/published).
kind regards,
Ian.

Looks good. I'll get back to you soon when I'm done.

You can currently limit the character count by implementing the shouldChange closure.
The character count can be a derived publisher based on the text binding.
struct ExampleView: View {
    @State private var text: String = ""
    @State private var isEditing: Bool = false
    private let characterLimit: UInt = 20

    var body: some View {
        VStack {
            Text("Character count: \(text.count)")

            TextView(text: $text, isEditing: $isEditing) { range, characters -> Bool in
                guard let range = Range(range, in: text) else {
                    return true
                }

                let newText = text.replacingCharacters(in: range, with: characters)

                // Prevent the text growing longer than the character limit
                return newText.count <= characterLimit
            }
        }
    }
}

You can currently limit the character count by implementing the shouldChange closure.
The character count can be a derived publisher based on the text binding.
struct ExampleView: View {
    @State private var text: String = ""
    @State private var isEditing: Bool = false
    private let characterLimit: UInt = 20

    var body: some View {
        VStack {
            Text("Character count: \(text.count)")

            TextView(text: $text, isEditing: $isEditing) { range, characters -> Bool in
                guard let range = Range(range, in: text) else {
                    return true
                }

                let newText = text.replacingCharacters(in: range, with: characters)

                // Prevent the text growing longer than the character limit
                return newText.count <= characterLimit
            }
        }
    }
}


HWPV-195: View & Edit individual claim page

Main comments:

Rather than using HTML <br> etc for layout, try using margins/padding etc on surrounding/containing blocks.
CSS rules, better to extend existing rules or a sub-class (in SASS) than naming a class after the rule it's implementing


Fix some nits in the README.md

I've just fixed a link and added a space before a bracket. The rest of the diff (a bit confusing) is due to trailing spaces elimination)

The trailing spaces were read as newlines in Markdown, and were probably intentional.

@RichardLitt I've replaced trailing spaces with <br> where it was appropriate to reduce confusing for the future (many Node.js contributors have their editors and git configured to automatically trim trailing spaces). Trailing spaces before empty lines and on empty lines still seem to be redundant.

Thanks! Awesome.


Zooming linked axes is slow

Zooming the first axis propagates to the others but I'd guess it takes >300ms.
using WGLMakie

fig = Figure()
for i in 1:5
  ax = WGLMakie.Axis(fig[i,1])
  density!(ax, rand(1_000))
end
for a1 in fig.content
  linkaxes!(fig.content[1], a1)
end
fig

The tick-finding algorithm was very slow and should be much better once this is merged https://github.com/JuliaPlots/PlotUtils.jl/pull/136
That should remove most of the slowness here, although there's also this PR https://github.com/JuliaPlots/Makie.jl/pull/1672 which makes it a bit better. If it's still too slow after those two, one could make slightly more complicated optimizations leveraging that the axes are all linked, but it would be nice being able to avoid that for now

@jtrakk, can you update to<EMAIL_ADDRESS>and try out ?

It's much better now, maybe 50 ms (just eyeballing it).


Using an empty string as a verb name causes a crash in player class

#274

https://github.com/JujuAdams/Input/commit/6db1f250816535b2a7112b7de78349c545e28c61


[iOS] TouchEffect: Implemented hover handling

Description of Change
Implemented Hover API for iOS via UIHoverGestureRecognizer.
For testing, please run the test sample on the iPad simulator and enable pointers

Bugs Fixed

Fixes #733

API Changes
None
Behavioral Changes
Hover API work on IOS
PR Checklist

[ ] Has tests (if omitted, state reason in description)
[X] Has samples (if omitted, state reason in description)
[X] Rebased on top of develop at time of PR
[X] Changes adhere to coding standard
[ ] Updated documentation

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

BTW, this is a bug fix, no? If yes should target the main branch

BTW, this is a bug fix, no? If yes should target the main branch

BTW, this is a bug fix, no? If yes should target the main branch

Good question.
I am not sure if we should consider it as bugfix or enhancement
CC @jfversluis

BTW, this is a bug fix, no? If yes should target the main branch

Good question.
I am not sure if we should consider it as bugfix or enhancement
CC @jfversluis

My 10cents, if this works on other platforms and should work on iOS since v1, so it's a bug fix... If iOS didn't have support for this and this PR adds the support, so it's a enhancement

My 10cents, if this works on other platforms and should work on iOS since v1, so it's a bug fix... If iOS didn't have support for this and this PR adds the support, so it's a enhancement

I didn't know that it's possible for iOS before yesterday) So, is it an enhancement or a bug?)

I didn't know that it's possible for iOS before yesterday) So, is it an enhancement or a bug?)

"To be or not to be this is the question" xP

"To be or not to be this is the question" xP

Yeah big fix or not, I think this is small enough to go into main :)

Yeah big fix or not, I think this is small enough to go into main :)

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here


update with master

Codecov Report

Merging #149 (7bb802a) into v0.5.0 (2d14c3d) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           v0.5.0     #149      +/-   ##
==========================================
- Coverage   84.29%   84.27%   -0.03%
==========================================
  Files          22       22
  Lines        3521     3522       +1
==========================================
  Hits         2968     2968
- Misses        553      554       +1

Impacted Files
Coverage Δ

neurodiffeq/losses.py
58.82% <0.00%> (-3.68%)
:arrow_down:

Continue to review full report at Codecov.

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


--bcc strings is not working

Send with --bcc set, but not receieved, and on resend.com logs the bcc is still empty.

BCC recipients

Hey @hellopq, are you using the TUI or the CLI to send bcc?

Hey @hellopq, we now support BCC + CC fields, let me know if everything is working for you:

#65


(WIP) UTY-276: Add the ability to ack authority loss (v2).

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
Currently it is only possible to send authority loss imminent ack only using the C# SDK, added the functionality in the GDK as well.
Its easier to read in commit order to avoid having to scroll through a bunch of generated code.
Example generated code:
            public override void SendAuthorityLossImminentAcknowledgement(ComponentGroup authorityLossComponentGroup, global::Improbable.Worker.Core.Connection connection)
            {
                var componentDataArray = authorityLossComponentGroup.GetComponentDataArray<AuthorityLossImminent<SpatialOSTransform>>();
                var spatialEntityIdData = authorityLossComponentGroup.GetComponentDataArray<SpatialEntityId>();
                for (int i = 0; i < componentDataArray.Length; i++)
                {
                    var component = componentDataArray[i];
                    if (componentDataArray[i].AuthorityLossAcknowledged && !component.AuthorityLossAcknowledgmentSent)
                    {
                        connection.SendAuthorityLossImminentAcknowledgement(spatialEntityIdData[i].EntityId, ComponentId);
                        component.AuthorityLossAcknowledgmentSent = true;
                        componentDataArray[i] = component;
                    }
                }
            }

Tests
How did you test these changes prior to submitting this pull request?
What automated tests are included in this PR?
Documentation
How is this documented (for example: release note, upgrade guide, feature page, in-code documentation)?
Primary reviewers
If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

Before this gets any further we may also want to think if any of the upcoming changes will affect this. Mostly the reactive component stuff that we'd like to get in. I suspect it won't change it much or at all, but it might do.

Will close this given that bridge v2 doesn't support authority loss and none of our code will use this.


android port

What feature do you want to get added on the base engine?
pls can you do an android port of your engine pls
To test your sight, and reliability, please select the option of what should NOT be requested.
A good idea

Test =  'failed';

Test = 'failed';

I have make a android port of os engine 1.5.1 yesterday but it slow in loading songs


Don’t depend on elm reactor

As we’ve found out in https://github.com/elm-lang/elm-reactor/issues/178, elm reactor is broken on some platforms.
No shame on elm reactor – it’s obviously a tremendous piece of code with rocket science under the hood!
I’ve written elm live because I wanted something simple and flexible. Let’s make it even simpler by using elm make directly under the hood. We’ll serve the files ourselves – that opens an easier way for custom static HTML.

Released in v2.0.0.


edge: end-of-candidates can be at session level

a=end-of-candidates is currently only parsed at m-line level but it can be a session-level attribute,
see https://tools.ietf.org/html/draft-ietf-mmusic-trickle-ice-sip-04#section-8.2
kudos to @freeswitch for pointing this out

LGTM


Add support for singleton backends

We'd like to implement our multitenant OVS backend with a singleton object to set up and create the OVS bridge (we only need one bridge for all tenant networks) and maintain some host-wide flows.  To decrease the complexity of a singleton backend we can very slightly increase the complexity of non-singleton backends by splitting Init() into:

Init() which takes properties that won't ever change (ctx, interface properties, etc)
AddNetwork() which tells the backend to actually configure itself for the network

A singleton backend will now only be created once, and have its Init() only called once.  After that, each new Network object will trigger an AddNetwork() call to poke the singleton backend to do whatever it needs to do for the new network.
(once the add/remove networks branch lands, of course we'll also have a RemoveNetwork() call into the backend, which non-singleton backends would ignore...)

I don't like the duality of a backend being a singleton or not, considering there's always AddNetwork method. I think either all backends should be singletons or none of them. If backends are not singletons, it should be pretty easy to just use global variables in the concrete backend package (e.g. ovs) to keep track of any global data.
The reason there's currently both New and Init is that the former does not take ctx and is not interruptable and the latter is. If you want the backend to be a singleton, I would just combine New and Init as both have nothing interruptable. I would then have AddNetwork and Run take ctx as first arg (convention established by context lib authors).

Ok, fair enough.  How about now?

I think the Backend interface is good now. However the implementations need to make sure that any data that is network specific is stored in some map (mapping from network name to network specific data). Stuff like lease, routes, UDP sockets, TUN devices, etc. Otherwise AddNetwork calls will stump over each other. There also needs to a mutex there to protect it as I understand the calls to AddNetwork are not serialized.
This is why it maybe easier to have each backend package just expose something like:
func NewNetwork(ctx context.Context, sm subnet.Manager, network string, config *subnet.Config, iface *Iface) Network`

// need a better name though
type Iface struct {
    extIface *net.Interface
    extIaddr net.IP
    extEaddr net.IP
}

type Network interface {
    Run(ctx context.Context)
}

Data common to all networks in a given backend would be stored in global vars.
I am fine with both designs but I would probably go with the NewNetwork one so that I don't have to keep all those maps in each backend. There would just be one map (with a mutex) in the network package.

probably go with the NewNetwork one

Could you clarify which approach you mean here?

hehe -- I did leave it ambiguous :) I meant the latter one, having the backend expose a global function func NewNetwork(ctx context.Context, sm subnet.Manager, network string, config *subnet.Config, iface *Iface) Network

hehe -- I did leave it ambiguous :) I meant the latter one, having the backend expose a global function func NewNetwork(ctx context.Context, sm subnet.Manager, network string, config *subnet.Config, iface *Iface) Network

Ah, so you mean having the backend take responsibility for creating the Network objects, rather than the Manager?

I think the Backend interface is good now. However the implementations need to make sure that any data that is network specific is stored in some map (mapping from network name to network specific data). Stuff like lease, routes, UDP sockets, TUN devices, etc. Otherwise AddNetwork calls will stump over each other. There also needs to a mutex there to protect it as I understand the calls to AddNetwork are not serialized.

I was attempting modify the existing backends as little as possible here, and have singleton backends be opt-in.  So our in-progress OVS plugin uses Once in the New() call to create the singleton backend, and then protects AddNetwork() with a mutex.  So the backend itself handles storing the individual network info internally.
Existing backends still create a new backend instance for each Network object, so there would be no clashes for AddNetwork() in the existing backends with the branch as it currently stands.

I am sorry -- I didn't read the changes carefully enough. It's fine then with the exception that AddNetwork is now not an ideal name. Unless New() does the Once fanciness, there's a 1-1 mapping between a backend instance and a network. Therefore AddNetwork will be called only once but feels like it will be called many times.
However I think it's ok for now. Once OVS backend is merged, we can revisit the design and refactor it a bit more.
Can you add a comment in backend/common.go to explain it a bit -- I think the semantics may trip people up.

Would SetNetwork() or RegisterNetwork() or even NewNetwork() sound better to you?

Let's go with RegisterNetwork

Updated.

LGTM


Isort cannot handle line magic in jupyter notebook

Isort cannot handle line magic in jupyter notebook
For example, having a cell with %matplotlib widget will cause isort to return:
Syntax error in code: Traceback (most recent call last): File "c:\Users\user\.vscode\extensions\ms-python.isort-2023.10.1\bundled\tool\lsp_server.py", line 293, in is_python ast.parse(code) File "C:\Users\user\AppData\Local\Programs\Python\Python311\Lib\ast.py", line 50, in parse return compile(source, filename, mode, flags, ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "<unknown>", line 1 %matplotlib widget ^ SyntaxError: invalid syntax

This is duplicate of https://github.com/microsoft/vscode-isort/issues/303 . Basically providing Jupyter Support.
We recommend using ruff (Ruff Extension) for import sorting. It implements same rules as isort and has notebook formatting support.

This is duplicate of #303 . Basically providing Jupyter Support.
We recommend using ruff (Ruff Extension) for import sorting. It implements same rules as isort and has notebook formatting support.

I would still argue this is a separate issue considering the error message shows up whenever I open a notebook without explicitly making use of any of the refactor functionality.

That is strange, would it be possible to share the logs?

That is strange, would it be possible to share the logs?

I am not sure exactly where I can find the complete logs, but this is the error message that VSCode shows in the UI.
Syntax error in code: Traceback (most recent call last): File "c:\Users\user\.vscode\extensions\ms-python.isort-2023.10.1\bundled\tool\lsp_server.py", line 293, in is_python ast.parse(code) File "C:\Users\user\AppData\Local\Programs\Python\Python311\Lib\ast.py", line 50, in parse return compile(source, filename, mode, flags, ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "<unknown>", line 1 %matplotlib widget ^ SyntaxError: invalid syntax
Source: isort

It appears to concern a package that is bundled with isort.

In the output panel you can select isort in the drop down. See where it says "Python"  on the right select "isort".

In the output panel you can select isort in the drop down. See where it says "Python" on the right select "isort".

Thanks for detailing this.


Current version does not work.

Running yarn add @react-native-community/slider pulls down the broken version ^1.1.0 even though a later commit reverts the change back to 1.0.4.
Suggest bumping to 1.1.1 or similar to keep moving forward even if the code is the same as 1.0.4 under the hood, as I guess our dependency tools just ain't that smart.
As an aside "@react-native-community/slider": "https://github.com/react-native-community/react-native-slider.git#ede6b7378c06b6ff607d3f51d7d4454b314119e3", in my package.json works fine.

Sorry about that, I just got back from the vacation, and I didn't notice that unpublishing 1.1.0 failed. version 1.1.1 should be bug-free.


Bump user_limit from u8 to u32

Today Discord has been sending "user_limit": 10000. Currently this is stored in a u8 which means deserialization breaks.

Turns out there are fields other than user_limit that have sizes too small for what Discord's sending.
I'll add them to this PR.

JK — they've already been resolved in #1822 and #1838. This is the only one I'm still encountering (though it's probably a good idea to look for more fields that might be problematic as Discord seems to be making changes to their model serialization)

Test failure says the token output for twilight_model/src/template/mod.rs needs to be updated from U8 to U32

Updated


Caching issue when using custom hook built with Context

Since sst 2.25+, the Context.memo(...) is broken, the context doesn't get reset on every request.
This is each SST handlers are using the new context/context2.ts, but the exported context is still context/context.ts
Here is a link to the discord issue: https://discord.com/channels/983865673656705025/1164163304428670976

Fixing PR: https://github.com/sst/sst/pull/3533


export_presets.cfg not found

Hi, thanks for you action.
When I try to build my project, it doesn't find export_presets.cfg and I don't really know why.
I know there are some issues here with the same problem, but I'm using v1.4.1 (same with master).

The error
ERROR: _fs_changed: Invalid export preset name: linux. Make sure `export_presets.cfg` is present in the current directory.
   At: editor/editor_node.cpp:770.

YML file
name: Build
on:
    push:
        branches:
            - main
    pull_request:
        types: [opened, synchronize, reopened]
jobs:
    builds:
        name: Builds
        runs-on: ubuntu-latest
        strategy:
          matrix:
            platform: [ linux, windows ]
        steps:
            - name: Build for ${{ matrix.platform }}
              id: build
              uses<EMAIL_ADDRESS>              with:
                  name: GoGodotJam2021
                  preset: ${{ matrix.platform }}
            - name: Upload ${{ matrix.platform }} artifact
              uses: actions/upload-artifact@v2
              with:
                name: ${{ matrix.platform }} build
                path: ${{ github.workspace }}/${{ steps.build.outputs.build }}

export_presets.cfg located at the root of my project :
[preset.0]

name="windows"
platform="Windows Desktop"
runnable=true
custom_features=""
export_filter="all_resources"
include_filter=""
exclude_filter=""
export_path="Builds/Windows/GoGodotJam2021.exe"
script_export_mode=1
script_encryption_key=""

[preset.0.options]

custom_template/debug=""
custom_template/release=""
binary_format/64_bits=true
binary_format/embed_pck=false
texture_format/bptc=false
texture_format/s3tc=true
texture_format/etc=false
texture_format/etc2=false
texture_format/no_bptc_fallbacks=true
codesign/enable=false
codesign/identity=""
codesign/password=""
codesign/timestamp=true
codesign/timestamp_server_url=""
codesign/digest_algorithm=1
codesign/description=""
codesign/custom_options=PoolStringArray(  )
application/icon=""
application/file_version=""
application/product_version=""
application/company_name="Kale and Lazard"
application/product_name="GoGodotJam2021"
application/file_description=""
application/copyright=""
application/trademarks=""

[preset.1]

name="linux"
platform="Linux/X11"
runnable=true
custom_features=""
export_filter="all_resources"
include_filter=""
exclude_filter=""
export_path="Builds/Linux/GoGodotJam2021.exe.x86_64"
script_export_mode=1
script_encryption_key=""

[preset.1.options]

custom_template/debug=""
custom_template/release=""
binary_format/64_bits=true
binary_format/embed_pck=false
texture_format/bptc=false
texture_format/s3tc=true
texture_format/etc=false
texture_format/etc2=false
texture_format/no_bptc_fallbacks=true

Here is the entire repo

If you need turnkey exporting with GitHub Actions, I would recommend using https://github.com/aBARICHELLO/godot-ci instead. This repository seems more suited to scenarios where you need to build your own Godot export templates.

Thanks, I'll look at it

Hey @QuentinWidlocher, I'll look into this tonight for you. This is actually built on top of aBARICHELLO's godot-ci project, so it has all required export templates.

Oh ok thanks a lot, I didn't get that

Hey @QuentinWidlocher
It looks like you are using C#, in this case you would have to provide your own export templates and compile as a separate step or use the mono version of the godot-ci project. As, by default, it will use default export templates and does not have mono installed.

Though, I will say, I would like this project to have mono-support sooner rather than later. :smile:

Ok thank you very much for your time, I'll look into it !


Declare ArrowPosition with objective-C-compatible format?

This library contains only one enum. What about to declare it with objective-C-compatible format?
The reason is I understand that anyways I need to write swift wrapper over this library to allow to use it in objective-C... But it looks silly to write enum to access another enum which is only one in the library.

#66


400 error when using the create_user api.

I've the similar problem when using the create_user api.
api.create_user('xxxx@kodiary.com','xxx','xxxx','xxxx',                         active=True, roles=['user'],join_default_channels=False, require_password_change=False,                         send_welcome_email=True, verified=False, customFields={})
The error is same when removing customFields and using customFields=None.
HTTPError: 400 Client Error: Bad Request for url: https://chat.kodiary.com/api/v1/users.create
Where as from the logs of rocket chat on my server shows:
{"level":35,"time":"2022-02-18T05:09:11.600Z","pid":8,"hostname":"036b252e8186","name":"","name":"API","method":"POST","url":"/api/v1/login","userAgent":"python-requests/2.27.1","length":"29","host":"chat.kodiary.com","remoteIP":"<IP_ADDRESS>","responseTime":159}
{"level":35,"time":"2022-02-18T05:09:12.778Z","pid":8,"hostname":"036b252e8186","name":"","name":"API","method":"POST","url":"/api/v1/users.create","userId":"u7nopRH2ZfBieBk83","userAgent":"python-requests/2.27.1","length":"256","host":"chat.kodiary.com","remoteIP":"<IP_ADDRESS>","err":{"type":"errorClass","message":"Match error: Missing key 'email'","stack":"Error: Match error: Missing key 'email'    at check (packages/check/match.js:36:17)    at Object.post (app/api/server/v1/users.js:36:4)    at app/api/server/api.js:425:96    at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1257:12)    at Object._internalRouteActionHandler [as action] (app/api/server/api.js:425:39)    at Route._callEndpoint (packages/rocketchat_restivus/lib/route.coffee:150:32)    at packages/rocketchat_restivus/lib/route.coffee:59:33    at packages/simple_json-routes.js:100:9","path":"","sanitizedError":{"type":"errorClass","message":"Match failed [400]","stack":"Error: Match failed [400]    at errorClass. (packages/check/match.js:91:27)    at new errorClass (packages/meteor.js:660:17)    at check (packages/check/match.js:36:17)    at Object.post (app/api/server/v1/users.js:36:4)    at app/api/server/api.js:425:96    at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1257:12)    at Object._internalRouteActionHandler [as action] (app/api/server/api.js:425:39)    at Route._callEndpoint (packages/rocketchat_restivus/lib/route.coffee:150:32)    at packages/rocketchat_restivus/lib/route.coffee:59:33    at packages/simple_json-routes.js:100:9","isClientSafe":true,"error":400,"reason":"Match failed","errorType":"Meteor.Error"},"errorType":"Match.Error"},"status":400,"responseTime":11,"msg":"Match error: Missing key 'email'"}
{"level":35,"time":"2022-02-18T05:09:18.556Z","pid":8,"hostname":"036b252e8186","name":"","name":"API","method":"POST","url":"/api/v1/method.call/license:getModules","userId":"wyQNrrLLPaWRoTsDE","userAgent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/97.0.4692.99 Safari/537.36","length":"96","host":"chat.kodiary.com","remoteIP":"<IP_ADDRESS>","status":200,"responseTime":4}
Originally posted by @warriorbik in https://github.com/Pipoline/rocket-python/issues/44#issuecomment-1043908408

Confirming this, I'll try to open issue in rocketchat project

It seems that I found the issue, I'll inform you about fix

Fixed

Thank you, the issue with the email is fixed but now it shows type cast error for active:
api.create_user(email,                      name,                      password,                      username,                      active=True,                      roles=['user'],                      join_default_channels=True,                      require_password_change=False,                      send_welcome_email=False,                      verified=False,                      customFields=None)
API","method":"POST","url":"/api/v1/users.create","userId":"u7nopRH2ZfBieBk83","userAgent":"python-requests/2.27.1","length":"184","host":"chat.kodiary.com","remoteIP":"<IP_ADDRESS>","err":{"type":"errorClass","message":"Match error: Expected boolean, got string in field active","stack":"Error: Match error: Expected boolean, got string in field active    at check (packages/check/match.js:36:17)    at Object.post (app/api/server/v1/users.js:36:4)    at app/api/server/api.js:425:96    at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1257:12)    at Object._internalRouteActionHandler [as action] (app/api/server/api.js:425:39)    at Route._callEndpoint (packages/rocketchat_restivus/lib/route.coffee:150:32)    at packages/rocketchat_restivus/lib/route.coffee:59:33    at packages/simple_json-routes.js:100:9","path":"active","sanitizedError":{"type":"errorClass","message":"Match failed [400]","stack":"Error: Match failed [400]    at errorClass. (packages/check/match.js:91:27)    at new errorClass (packages/meteor.js:660:17)    at check (packages/check/match.js:36:17)    at Object.post (app/api/server/v1/users.js:36:4)    at app/api/server/api.js:425:96    at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1257:12)    at Object._internalRouteActionHandler [as action] (app/api/server/api.js:425:39)    at Route._callEndpoint (packages/rocketchat_restivus/lib/route.coffee:150:32)    at packages/rocketchat_restivus/lib/route.coffee:59:33    at packages/simple_json-routes.js:100:9","isClientSafe":true,"error":400,"reason":"Match failed","errorType":"Meteor.Error"},"errorType":"Match.Error"},"status":400,"responseTime":18,"msg":"Match error: Expected boolean, got string in field active"}
{"level":35,"time":"2022-03-07T12:17:36.494Z","pid":8,"hostname":"036b252e8186","name":"","name":"API","method":"POST","url":"/api/v1/method.call/license:getModules","userId":"wyQNrrLLPaWRoTsDE","userAgent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.109 Safari/537.36","length":"97","host":"chat.kodiary.com","remoteIP":"<IP_ADDRESS>","status":200,"responseTime":1}

I fixed that now, it’s fixed in v1.3.0 .

On 7 Mar 2022, at 13:21, warriorbik @.***> wrote:
Thank you, the issue with the email is fixed but now it shows type cast error for active:
api.create_user(email, name, password, username, active=True, roles=['user'], join_default_channels=True, require_password_change=False, send_welcome_email=False, verified=False, customFields=None)
API","method":"POST","url":"/api/v1/users.create","userId":"u7nopRH2ZfBieBk83","userAgent":"python-requests/2.27.1","length":"184","host":"chat.kodiary.com","remoteIP":"<IP_ADDRESS>","err":{"type":"errorClass","message":"Match error: Expected boolean, got string in field active","stack":"Error: Match error: Expected boolean, got string in field active
at check (packages/check/match.js:36:17)
at Object.post (app/api/server/v1/users.js:36:4)
at app/api/server/api.js:425:96
at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1257:12)
at Object._internalRouteActionHandler [as action] (app/api/server/api.js:425:39)
at Route._callEndpoint (packages/rocketchat_restivus/lib/route.coffee:150:32)
at packages/rocketchat_restivus/lib/route.coffee:59:33
at packages/simple_json-routes.js💯9","path":"active","sanitizedError":{"type":"errorClass","message":"Match failed [400]","stack":"Error: Match failed [400]
at errorClass. (packages/check/match.js:91:27)
at new errorClass (packages/meteor.js:660:17)
at check (packages/check/match.js:36:17)
at Object.post (app/api/server/v1/users.js:36:4)
at app/api/server/api.js:425:96
at Meteor.EnvironmentVariable.EVp.withValue (packages/meteor.js:1257:12)
at Object._internalRouteActionHandler [as action] (app/api/server/api.js:425:39)
at Route._callEndpoint (packages/rocketchat_restivus/lib/route.coffee:150:32)
at packages/rocketchat_restivus/lib/route.coffee:59:33
at packages/simple_json-routes.js💯9","isClientSafe":true,"error":400,"reason":"Match failed","errorType":"Meteor.Error"},"errorType":"Match.Error"},"status":400,"responseTime":18,"msg":"Match error: Expected boolean, got string in field active"}
{"level":35,"time":"2022-03-07T12:17:36.494Z","pid":8,"hostname":"036b252e8186","name":"","name":"API","method":"POST","url":"/api/v1/method.call/license:getModules","userId":"wyQNrrLLPaWRoTsDE","userAgent":"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.109 Safari/537.36","length":"97","host":"chat.kodiary.com","remoteIP":"<IP_ADDRESS>","status":200,"responseTime":1}
—
Reply to this email directly, view it on GitHub https://github.com/Pipoline/rocket-python/issues/58#issuecomment-1060630570, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJOG6TE6ZLEGUFK7FBDLWTU6XYEXANCNFSM5O5PSJNA.
You are receiving this because you modified the open/close state.


Support partial reads in storage::Client::ReadObject.

We need to support partial reads in storage::Client::ReadObject (i.e. reading a range of bytes from the object). This will allow application developers to download the file "in parallel" with multiple threads reading different sections of the object.
In principle something like:
void F(google::cloud::storage::Client client) {
  // Read the first 32MiB
  auto stream = client.ReadObject(
      "my-bucket", "my-object", google::cloud::storage::Range(0, 32 * 1024, 1024));
  while(not stream.eof()) {
    // Do stuff...
  }
}

Happy to have a look at this one.

@benjaminr that sounds great, just sends us a PR.  As you might expect, you will need to sign a CLA.  We have instructions on how to setup your development environment and how to locally execute builds with the same configuration as the CI builds.
Please do not hesitate to @-mention me in this bug if you need help.

@coryan Would it be sufficient to overload ReadObject to something like the following:
`  template <typename... Options>

ObjectReadStream ReadObject(std::string const& bucket_name,

                          std::string const& object_name,

                          std::int64_t begin,

                          std::int64_t end,

                          Options&&... options) {

internal::ReadObjectRangeRequest request(bucket_name, object_name, begin, end);

request.set_multiple_options(std::forward<Options>(options)...);

return ObjectReadStream(raw_client_->ReadObject(request).value());

}`

Where begin and end are passed to internal::ReadObjectRangeRequest?

Unfortunately that overload will not work, because it is ambiguous to use it like this:
void F(gcs::Client client) {
  auto reader = client.ReadObject("my-bucket", "my-object", 0, 2048);
}

The conversion from int to std::int64_t is worse than binding to the parameter pack in the first overload, and the compiler gets all confused. Of course the user can say:
void F(gcs::Client client) {
  auto reader = client.ReadObject("my-bucket", "my-object", std::int64_t(0), std::int64_t(2048));
}

and that works fine. But it seems too surprising that the overload works with some integral types but not with others.
That is why I thought we would need a new Range option:
struct google::cloud::storage:Range : public internal::ComplexOption<Range, RangeData> {
  Range(std::int64_t begin, std::int64_t end);
  // look at DisableMD5Hash for inspiration...
};

Which would be applied to the ReadObjectRangeRequest:
class ReadObjectRangeRequest
    : public GenericObjectRequest<
          ReadObjectRangeRequest, DisableCrc32cChecksum, DisableMD5Hash,
          EncryptionKey, Generation, IfGenerationMatch, IfGenerationNotMatch,
          IfMetagenerationMatch, IfMetagenerationNotMatch, Range, UserProject> {
// delete the begin_ and end_ fields, use `HasOption<Range>()` and `GetOption<Range>()` instead

I think that would work better, but I probably have some details wrong. Does it make sense?

Yeah I think that makes sense. In the following struct declaration:
struct google::cloud::storage:Range : public internal::ComplexOption<Range, RangeData> {
  Range(std::int64_t begin, std::int64_t end);
  // look at DisableMD5Hash for inspiration...
};

Is RangeData best as a std::pair for begin and end? Apologies if this is obvious, my C++ skills are extremely rusty. Thanks!

Both work for me, a pair looks easier.

@benjaminr I assigned this to you just so we can keep track of who is working on what, if at any point you think you won't be able to work on it anymore just let me know.

@benjaminr no problem, I hope you had fun looking at this issue at least.  After thinking more about it, it is not that easy because we need to disable checksums when reading a section of the file.

@coryan I did, thanks for the opportunity to look at it. I'll probably come back to look at how it ends up being implemented.


Using "Set Defaults" button breaks config

Configs no longer update after pressing the Set Defaults button. The UI updates properly when changing settings, but when fetching the configs it always pulls the pre-defaulted settings. It is broken until a game restart.

Was fixed!


Pinot on Yarn

Placeholder to discuss functionality that would enable Pinot to run on Yarn.
Some initial thoughts- https://groups.google.com/forum/#!topic/pinot_users/RL7QrAdAIf8
I am interested in writing a small yarn app that would deploy Pinot into Yarn.  This would make it easier for my group to deploy pinot into clustered environments with a single command.  I wanted to run this by folks here to gauge the level of interest of having this be a potential contribution.
As a starting point, this is what I envision-

a yarn deploy class, and possibly a script wrapper (pinot-yarn-deploy.sh)

PinotYarnDeploy (and a few additional ancillary classes) would be created and placed inside com.linkedin.pinot.hadoop.yarn from the pinot-hadoop module (or possibly as its own module if that would make more sense: pinot-yarn?)
PinotYarnDeploy would leverage args4j to accept yarn and pinot properties such as numControllers, numBrokers, numServers, and settings for each container type (cpus, memory, container overhead, etc..).

a basic yarn application master with a web ui that would:
a. show where things are running
b. allow one to add/remove: controllers, brokers, servers from the yarn app master ui

Running in yarn all the storage modes should be able to be supported.

hdfs backed- containers would copy segments out of hdfs, which is extra copying and storage but should be fine

nfs backed (we'd probably use the hdfs nfs gateway as that allows storage in hdfs, supports mmapping files directly out of hdfs).

Beyond that I would be interested in adding the following as well:

allow services to bind to random ports if passed 0 as the port number

pinto-admin.sh commands that take -controllerHost -controllerPort would optionally be able to take -zkAddress, which would choose a controller at random, and then connect to it using the published host and port that sit in zookeeper (a.k.a- this just makes it easy to work with random ports)

Any input is appreciated and welcome, thank you!

As I currently see it, this would encompass the following additions-

add yarn deploy related classes, ie- PinotYarnDeploy, PinotYarnAppMaster, ... also possibly a script wrapper (pinot-yarn-deploy.sh)
provide yarn application master that manages container lifecycle, and has a ui component that shows where services are running, ports, etc, ..
allow services to bind to random ports if passed 0 as the port number
change pinot-admin.sh for commands that require a controller host and port, to optionally take -zkAddress. This would discover and choose a random controller to send the command to

I updated the issue with the content of the email. Hope that's ok.
We are definitely interested in this initiative. We were thinking of building something like this for other provisioning systems such as AWS, Azure, Mesos, Kubernetes, Rain (internal to LinkedIn). It will be great if we can build right abstractions so that most of the code can be re-used across different systems.
Another way to look at this problem would be from Helix point of view. It would be cool for Apache Helix project add the support for deploying a Helix based system on any provisioning system.
Some pointers -

@jfim has added the support for Pinot on Kubernetes.
Gobblin on Yarn
Redis auto-scaling with Helix

We can definitely start with a PoC for Yarn and extend it to other systems as the second step. Can you come up an architecture on how this would look when deployed on YARN.

This looks good to me. In step 4, instead of adding tenant, we can model it as add node or add instance which can take instanceType as an argument. InstanceType can be Server,Broker, Minion etc.
This should have set up the cluster. After this users can invoke addTable etc which will take tableConfig as an argument. TableConfig has the tenant info and we automatically assign instances to a tenant. We have all this available as of today.
Thoughts?

Ahhh! Ok yes, I think I follow what you mean.  I believe this should work nicely. I will spend some cycles digging in a bit more in the morning.
One question I have- If instanceType is SERVER, my understanding is that the server type (realtime/offline) won't be determined until the instance is created/assigned to a server.  If we want realtime servers to have say, have a larger heap than offline servers--is there a way to designate the type that a server can be in advance?

@kishoreg I have a very basic prototype that is able to run Pinot entirely in YARN.  Currently the number of controllers/brokers/servers needs to be specified up front, but they can be added/removed during runtime.

You had mentioned the Minion instance type- I'm trying to figure out where exactly this ties in.  Am I correct in thinking that the Minion is work still in progress, but will eventually have a PinotAdministrator StartMinion command?
I'm now looking closer at modeling this all around addInstance per your input.
Originally I was thinking the cluster manager backend would listen as a participant when an instance is needed.  But Pinot can't assign instances that don't already exist.  Given that, would addInstance be a new command within PinotAdministrator that would communicate with the cluster manager backend to create the resources?
Example workflow would be something like-

submit application into yarn
run AddInstance commands-
a. AddInstance -instanceType BROKER -tenantName MyTenant -numInstances 2
b. AddInstance -instanceType SERVER_REALTIME -tenantName MyTenant -numInstances 3
c. AddInstance -instanceType SERVER_OFFLINE -tenantName MyTenant -numInstances 5
d. AddInstance -instanceType MINION -tenantName MyTenant -numInstances 1

After running that you'd have a Pinot cluster with 3 controllers, 2 brokers, 3 offline servers, 5 online servers, and 1 minion.  Is my thinking aligned with what you are envisioning?
Thanks


Properly match on forcenamespace

Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>

Codecov Report

Merging #139 (5d0fc50) into main (9482b36) will increase coverage by 0.81%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##             main     #139      +/-   ##
==========================================
+ Coverage   74.12%   74.94%   +0.81%
==========================================
  Files          13       13
  Lines         831      834       +3
==========================================
+ Hits          616      625       +9
+ Misses        146      141       -5
+ Partials       69       68       -1

Impacted Files
Coverage Δ

pkg/pipelineascode/pipelineascode.go
69.49% <50.00%> (-1.20%)
:arrow_down:

pkg/config/annotation_matcher.go
100.00% <100.00%> (ø)

pkg/config/repo_runinfo_matcher.go
92.00% <0.00%> (+32.00%)
:arrow_up:

Continue to review full report at Codecov.

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


autoinstall yaml ignored, can't find docs

cmd /c ver
Microsoft Windows [Version 10.0.22621.1778]
wsl --status
Default Distribution: Ubuntu-22.04
Default Version: 2
WSL1 is not supported with your current machine configuration.
Please enable the "Windows Subsystem for Linux" optional component to use WSL1.

Did the problem occur during installation?

[X] Yes

What happened?
I have a simple autoinstall yaml file in current directory, containing:

version
identity
packages
late-commands
user-data (mainly for write-files)

I ran this command: ubuntu2204.exe install --root --autoinstall .\simple.yaml --ui=none
(I've tried it without --root with same results.)
It's like the whole file is ignored, I don't know where to look for additional logs. I just see the 10 minutes cloud-init wait gap in the logs and that's it.
I don't know if this feature is working, where the docs are for it, what the current limitations are, plans for the future, etc.
It would be nice to be able to preconfigure wsl Ubuntu environment very similar to my VM and barebone installations - obviously I wouldn't touch some stuff like network or storage.
What was expected?
Ubuntu is installed with configuration according to autoinstall yaml file.
Steps to reproduce
I've used this as contents of autoinstall yaml file:
#cloud-config
autoinstall:
  version: 1
  identity:
    realname: whatever
    username: geth
    password: '$1$Dct4aGTo$p1OettH8tlj74w4g5AVN21'
  packages:
    - mc
    - jq
  late-commands:
    - - curtin
      - in-target
      - --
      - sh
      - -c
      - |
        printf 'TODO late-commands\n' >> /var/tmp/testy
  user-data:
    package_update: true
    package_upgrade: true
    ssh_publish_hostkeys: { enabled: false }
    write_files:
      - path: /etc/skel/.vimrc
        content: |
          let &t_SI .= "\<Esc>[?2004h"
          let &t_EI .= "\<Esc>[?2004l"
          inoremap <special> <expr> <Esc>[200~ XTermPasteBegin()

          function! XTermPasteBegin()
            set pastetoggle=<Esc>[201~
            set paste
            return ""
          endfunction

Additional information
subiquity-server-debug.log

Hello, and thank you for contacting us regarding this issue.
We've identified the origin of the bug and we're working to solve it. There is no workaround currently (other than doing this same thing manually with a normal install) but we'll notify once it has been fixed.
Thank you again for your report.

Hello again!
The cloud-init feature is still considered experimental, but we wrote some documentation around it. You can check it out here in the Wiki: https://github.com/ubuntu/WSL/wiki/Auto-Installation-Support


update master node failover node handle

if master server self dead need remove ZKNodePath , Carry out subsequent processing，add master  to ZK dead node , send alert , record fail tash to DB

There are some inappropriateness in this modification.
the server cannot failover anything when listening itself down. it need other servers to failover the processes.

I am still looking forward your next contribution, thanks


List of sectors

Summary (include Python version)
Python 3.8.6, iexfinance==0.5.0
Hello: I looked through the documentation and could not find an equivalent for producing a list of sectors.  Here is the api call from IEX Cloud:
Call: https://sandbox.iexapis.com/stable/ref-data/sectors?token=
IEX Cloud documentation: https://iexcloud.io/docs/api/#sectors
iexfinance documentation: https://addisonlynch.github.io/iexfinance/stable/market-info.html#collections
And if these kinds of requests are not recommended here, please let me know! I will post to SO.
Date/time of issue
Current
Expected behavior
Looking for list of sectors
Actual behavior
Don't know if it's available

Use refdata.getsectors https://addisonlynch.github.io/iexfinance/stable/refdata.html#sectors

Thanks, Addison. And if you ever need any help with this project, please let me know!


fix Kvret/CamRestBPETextField dataset_path error.

update space-1/galaxy/data/field.py.

@Chen9154 Many thanks !


fix: update testing docs with livewire plugin for pest

Clarify Livewire plugin requirement for Pest testing
Description
This pull request clarifies a dependency for testing Livewire components using Pest in the FilamentPHP documentation.
Currently, the documentation mentions using Pest for testing but doesn't explicitly state the requirement for the Livewire plugin. This can lead to confusion for users unfamiliar with Pest's plugin system.
Issue addressed:
This pull request addresses the potential confusion around the Livewire plugin requirement, as highlighted in issue #12058.
Changes made:

The introduction to the testing section is updated to explicitly mention the need for the Pest Livewire plugin.
A link to the relevant Pest documentation for installing the Livewire plugin is provided for easy reference: Livewire plugin for Pest: https://pestphp.com/docs/plugins#livewire.
The original formatting and structure of the testing text are maintained for consistency.

This change aims to improve the clarity of the documentation and prevent similar confusion for future users.
Visual changes
I could not find out how to launch the docs locally so I don't have any visual changes to share.
Functional changes

[x] Code style has been fixed by running the composer cs command.
[ ] Changes have been tested to not break existing functionality.
[ ] Documentation is up-to-date.

Thanks


Add the open/closed behavior for enums to documentation

Hey there,
first of all, thanks a lot for maintaining this library, what a breath of fresh air to the grpc ecosystem :+1:
We noticed that enum values are decoded "naively" into their integer values, without any bounds check. This may cause out-of-type values on the typescript side, whereas my expectations from protobuf would be to receive the default value for such cases.
I'm not sure what the best way to handle this is. Checking for value correctness comes with a significant performance overhead since it needs to compare against a specific set of known values. But on the other hand, out-of-type values in typescript are very unexpected and can lead to severe consequences. Perhaps a good middle ground is to simply document this behavior? Or make it a configurable performance vs correctness option?
For the record, we are now using this function on all incoming enums to ensure correctness:
function knownEnumOrDefault<T extends Enum>(value: T[keyof T], enumObject: T): T[keyof T] {
  // those next two lines are not cheap, unfortunately
  const enumValues = Object.values(enumObject);
  if (enumValues.includes(value)) {
    return value;
  }
  // Using the protobuf notion of "default", i.e. the first value. By convention, this is
  // a dedicated UNKNOWN value for all our enums.
  // This goes through the reverse mapping to get the actual entry with index 0.
  return enumObject[enumObject[0]] as T[keyof T];
}

This is in line with the official specification for enums (at least for proto3/open enums). See the official and unofficial docs.

Enums have two distinct flavors (open and closed). They behave identically except in their handling of unknown values. Practically, this means that simple cases work the same, but some corner cases have interesting implications.
For the purpose of explanation, let us assume we have the following .proto file (we are deliberately not specifying if this is a syntax = "proto2" or syntax = "proto3" file right now):
enum Enum {
 A = 0;
 B = 1;
}

message Msg {
 optional Enum enum = 1;
}

The distinction between open and closed can be encapsulated in a single question:

What happens when a program parses binary data that contains field 1 with the value 2?

Open enums will parse the value 2 and store it directly in the field. Accessor will report the field as being set and will return something that represents 2.
Closed enums will parse the value 2 and store it in the message’s unknown field set. Accessors will report the field as being unset and will return the enum’s default value.

FWIW you could likely make your function less expensive by changing it to something like:
function knownEnumOrDefault<T extends Enum>(value: T[keyof T], e: T): T[keyof T] {
  return e[e[value]] ?? e[e[0]] as T[keyof T];
}

Thanks for the explanation, that sounds reasonable. It would be great to document this behavior, e.g. under Using enumerations :+1:

Unfortunately the proto3/open enums don't play very well with TS enums (or rather, the developer experience around them). The Most Correct™ type for an field in a message interface would be something like:
enum SomeEnum {
    UNSET = 0,
    FOO = 1,
    BAR = 2
}

interface MyMessage {
    // number is for potentially unknown enum values in a proto3/open enum
    myEnum: SomeEnum | number;
}

Unfortunately because SomeEnum (when used as a type) in a union with number just reduces to number. This makes the type almost useless. Fortunately TS is kind enough to allow one to index into any enum with a number so that can be used to check that the number actually exists in the enum (for runtime purposes), but that check doesn't actually refine the value from SomeEnum | number to just SomeEnum.
declare var m: MyMessage;

if (SomeEnum[m.myEnum]) {
    m.myEnum;
    // ^? (property) MyMessage.myEnum: number
}

So you either have to cast it (🤢) or make a function to cast it (😕):
if (SomeEnum[m.myEnum]) {
    m.myEnum as SomeEnum;
    // ^? (property) MyMessage.myEnum: SomeEnum
}

function isKnownSomeEnumValue(val: number): val is SomeEnum {
    return Boolean(SomeEnum[val]);
}

if (isKnownSomeEnumValue(m.myEnum)) {
    m.myEnum;
    // ^? (property) MyMessage.myEnum: SomeEnum
}

I think the only way to make the DX less bad (at least for performing the refinement) would be to represent protobuf enums as string unions and enum fields as: MyEnum | number. It's debatable whether the DX in general is better with TS enums or string unions.
This allows much easier refinements between known/unknown values for an enum field:
type SomeEnum = 'UNSET' | 'FOO' | 'BAR';

interface MyMessage {
    // number is for potentially unknown enum values in a proto3/open enum
    myEnum: SomeEnum | number;
}

if (typeof m.myEnum === 'string') {
    m.myEnum;
    // ^? (property) MyMessage.myEnum: SomeEnum
}


Deploy operator to Catalog

What can be improved
We have OLM generation, now we should deploy it to the Catalog.
Why
To follow official guidelines of installation in OpenShift and ultimately get in to OperatorHub

ike install should stay as we are not only targeting OpenShift with catalog
we should find a way to automatically test the correctness of the installation
(but also) cover both paths


suggestion feature: autocompletion

Hello,
Do you think it's possible to add autocompletion for show name of function or api in the editor?

yes, why not :)
will try to add. thanks


Can you add the "tag load save" flag, like docker

I think it ’s very practical, what do you guys think?

We can't add it, because CRI doesn't have it.
You may want to use the CLI from the underlying ccontainer runtime for those, e.g. ctr for containerd.

/close


Array of strings as label for data

Would it be possible to configure an array of strings which is displayed in place of the data value in the same way as with the dataLabels.

How? I need an array of values (numeric) to display them on the chart. Strings can't be calculated on chart. You can concert your strings into numeric values, no?

My idea would be that the values are still in the background and responsible for the line but visible are only the strings.

Von: Jonathan Gander @.>
Gesendet: Sonntag, Mai 29, 2022 4:06 PM
An: Jonathan-Gander/LineChartView @.>
Cc: Claudio Cantieni @.>; Author @.>
Betreff: Re: [Jonathan-Gander/LineChartView] Array of strings as label for data (Issue #13)
How? I need an array of values (numeric) to display them on the chart. Strings can't be calculated on chart. You can concert your strings into numeric values, no?
—
Reply to this email directly, view it on GitHubhttps://github.com/Jonathan-Gander/LineChartView/issues/13#issuecomment-1140456774, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYJGUHBMVVKMRWUQ2F46L4DVMN2XZANCNFSM5XIGECBQ.
You are receiving this because you authored the thread.Message ID: @.***>

Aaah! OK, I got it!
I will have a look.

Thank you!

Von: Jonathan Gander @.>
Gesendet: Sunday, May 29, 2022 5:03:32 PM
An: Jonathan-Gander/LineChartView @.>
Cc: Claudio Cantieni @.>; Author @.>
Betreff: Re: [Jonathan-Gander/LineChartView] Array of strings as label for data (Issue #13)
Aaah! OK, I got it!
I will have a look.
—
Reply to this email directly, view it on GitHubhttps://github.com/Jonathan-Gander/LineChartView/issues/13#issuecomment-1140467318, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYJGUHAEBEZCVT73GF775VLVMOBMJANCNFSM5XIGECBQ.
You are receiving this because you authored the thread.Message ID: @.***>

Can you explain or provide me an example of a use case for this? Why would you display values in chart but display other values (as strings)? I understand technically what you need but not why you need it.

I have an app which calculates the shape state for athletes. The value range for that output is between 0 and 1. 0…0.25 would be very low, 0.25…0.5 low, 0.5…0.75 normal, 0.75…1 high. I‘d like to represent the history of the shape state in a graph with these four texts seen. I would code the order. Example: [0.3, 0.5, 0.1, 0.9] represented with -> [”low”, ”medium“, ”very low“, ”high“].

Von: Jonathan Gander @.>
Gesendet: Montag, Mai 30, 2022 8:28 AM
An: Jonathan-Gander/LineChartView @.>
Cc: Claudio Cantieni @.>; Author @.>
Betreff: Re: [Jonathan-Gander/LineChartView] Array of strings as label for data (Issue #13)
Can you explain or provide me an example of a use case for this? Why would you display values in chart but display other values (as strings)? I understand technically what you need but not why you need it.
—
Reply to this email directly, view it on GitHubhttps://github.com/Jonathan-Gander/LineChartView/issues/13#issuecomment-1140751638, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYJGUHESJMOVLT3XU5AEVQLVMRNZDANCNFSM5XIGECBQ.
You are receiving this because you authored the thread.Message ID: @.***>

And I assume you don't want to have something like this?

You only want the Low instead of 0.4, right?

OK.
I've made changes in parameters to simplify your feature. I will try to have a look this week to find a (nice) solution!

Thank you!

Von: Jonathan Gander @.>
Gesendet: Monday, May 30, 2022 6:53:47 PM
An: Jonathan-Gander/LineChartView @.>
Cc: Claudio Cantieni @.>; Author @.>
Betreff: Re: [Jonathan-Gander/LineChartView] Array of strings as label for data (Issue #13)
OK.
I've made changes in parameters to simplify your feature. I will try to have a look this week to find a (nice) solution!
—
Reply to this email directly, view it on GitHubhttps://github.com/Jonathan-Gander/LineChartView/issues/13#issuecomment-1141346707, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYJGUHGPRMOYB7ANA2KE5QDVMTXBXANCNFSM5XIGECBQ.
You are receiving this because you authored the thread.Message ID: @.***>

Hey @claudiocantieni feature is now ready and commited. Check the brand new v1.4.0.
To do what you want, you can simply add this parameter to your LineChartParameters:
displayMode: .noValues

(Add it right after dotsWidth parameter. You can also check updated README.)
I hope you like this feature. Let me know!

It works for me! Thank you very much for your work.

Von: Jonathan Gander @.>
Gesendet: Friday, June 3, 2022 5:42:17 PM
An: Jonathan-Gander/LineChartView @.>
Cc: Claudio Cantieni @.>; Mention @.>
Betreff: Re: [Jonathan-Gander/LineChartView] Array of strings as label for data (Issue #13)
Hey @claudiocantienihttps://github.com/claudiocantieni feature is now ready and commited. Check the brand new v1.4.0.
To do what you want, you can simply add this parameter to your LineChartParameters:
displayMode: .noValues
(Add it right after dotsWidth parameter. You can also check updated README.)
I hope you like this feature. Let me know!
—
Reply to this email directly, view it on GitHubhttps://github.com/Jonathan-Gander/LineChartView/issues/13#issuecomment-1146089689, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AYJGUHAU35SEQD7UKGO4QQDVNIRVTANCNFSM5XIGECBQ.
You are receiving this because you were mentioned.Message ID: @.***>


SIL: Clean up ownership handling in createAsyncTask builtins.

The underlying runtime functions really want to be able to consume the closure being used
to spawn the task, but the implementation was trying to hide this by introducing a retain
at IRGen time, which is not very ARC-optimizer-friendly. Correctly model the builtin operands
as consumed so that the ownership verifier allows them to be taken +1. Also fix a leak here
by making the builtins lower their function argument as @callee_owned; since the runtime
entry points work by setting up the closure's entry point as the initial entry point of the task,
with the context passed in like an argument, nobody else but the closure's own implementation
can really be responsible for releasing that closure context when it's done.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 644e175b4b3c29e31ff222dea4ae7086f4d879d2

@aschwaighofer noted that the partial application forwarder codegen the first take on this might be broken, and probably isn't great for code size in general. I've reworked the non-_f variants of swift_task_create to accept closure values as-is (since the older forms were unused) and unpack the invocation function and initial context on the callee side so IRGen doesn't need to splat out that logic at every use site. When creating a task with a closure context, we now also set up the task's return address to transfer control to a completeTask variant that releases the context for us, fixing the leak without requiring a thunk on the closure itself. @aschwaighofer how does this approach look?

@swift-ci Please test

Build failed
Swift Test OS X Platform
Git Sha - d9798c086865faba79e01072c0443a85c244cecf


Revocation Flows -Credential Revocation

The actual revocation flow allows multiple revocations to be requested at the same time. The Credential Revocation HTTP Request include a revocation_assertion_requests claim.
Within the Response flow, the response MUST include a JSON object with a member named revocation_assertion_responses.
The Revocation Assertion Error object within the response MUST contain an error parameters, among them the credential_updated parameter described as "One or more information contained in the Digital Credential are changed. The error_description field SHOULD contain a human-readable text describing the general parameters updated without specifying each one" is not clear how can be handled as:

The credential to be revoked is not provided in the revocation_assertion_requests claim as it contains only its HASH
In cash the verification would be performed only vs the credential hash, there is no possibility for the Credential Issuer to ensure that the provided Hash differs from the stored one only due to updates of its content
The Credential Issuer is forbidden to store details of the issued credential, thus cannot keep information about specific information changed within it as it could be used to track its contents.

It would be therefore preferable to remove this Error type to avoid Credential Issuers to try to manage it thus reducing related data privacy guarantees

this must be done here: https://github.com/peppelinux/draft-demarco-oauth-status-assertions/issues/85

Done here: https://github.com/peppelinux/draft-demarco-oauth-status-assertions/issues/85


Add gitter badge

+1 it needs a rebase.

Oops, sorry @pilhuhn.  It looks like the changes introduced by @Steve-Fontes requires a re-basing of your change...  Once you take care of that, I'll go ahead with the merge.  Thanks.

@kwsutter Done.


continuous_apr is broken for release

Instead of skipping r000 and using p045, r000 is created, although it is properly replaced with p045 during analysis.

This code

    window_list = []
    phases = ['attach', 'pull', 'release']
    for phase in phases:
        win_counts = []
        for restraint in restraint_list:
            if restraint.phase[phase]['targets'] is not None:
                win_counts.append(len(restraint.phase[phase]['targets']))
            else:
                win_counts.append(0)
        max_count = np.max(win_counts)
        if all(count == 0 or count == max_count for count in win_counts):
            if max_count > 0:
                if all_continuous_apr and phase in ('attach', 'release'):
                    max_count -= 1
                if max_count > 999:
                    log.info('Window name zero padding only applied for windows 0 - 999')
                window_list += [phase[0] + str('{:03.0f}'.format(val)) for val in np.arange(0, max_count, 1)]
        else:
            log.error('Restraints have unequal number of windows during the {} phase.'.format(phase))
            log.debug('Window counts for each restraint are as follows:')
            log.debug(win_counts)
            raise Exception('Restraints have unequal number of windows during the {} phase.'.format(phase))

deals with implemented continuous_apr. It looks like it may not correctly omit r000, but I don't think it is being tested.

I suspect the issue is that the list comprehension for window_list is an np.arange that starts at 0, and so will include r000, even if we correctly identify that should be skipped. What do you think @nhenriksen?

Yeah, it looks like the code assumes my previous approach, i.e. where r000 is the end of the release phase, rather than r014/p045.

Related, even if I pass continuous_apr = False to static restraints, and continuous_apr = False to my regular DAT_restraints, create_window_list says all "All restraints are "continuous_apr" style." and doesn't make the last attach window.

Addressed by #67


Libclang so version different than version

Comment:
Even for say llvm 15 the version of libclang is hardcoded to 13:
https://github.com/conda-forge/clangdev-feedstock/blob/b6cf7124da893bcb57ab1f38c768ea825fabbdb1/recipe/meta.yaml#L11
Is this intentional? What does libclang-13.dll version 15 actually mean?

Found the answer here:
https://github.com/conda-forge/clangdev-feedstock/blob/b6cf7124da893bcb57ab1f38c768ea825fabbdb1/recipe/conda_build_config.yaml#L6
Maybe good to add this comment to the meta.yaml too. abiversion would be less confusing than soversion. Closing otherwise.

Maybe good to add this comment to the meta.yaml too. abiversion would be less confusing than soversion. Closing otherwise.

I find soversion more accurate (and it's also the name of this variable in cmake). Strictly speaking, the ABI can still change in non-breaking ways, i.e. symbols can be added without having to bump the (abi)version.

Thanks for the clarifications (and indeed CMake does use this convention https://cmake.org/cmake/help/latest/prop_tgt/SOVERSION.html )!


test: collect code coverage in the root of the monorepo

Implement scripts to collect coverage reports (JSON format) from all packages in the monorepo that generate such reports. Reports are copied to <root>/.nyc_output/coverage-[pkg-dir-name].json.
Implement scripts to generate a combined html report in <root>/coverage.
Adjust root reset and clean scripts to delete <root>/.nyc_output and <root>/coverage.
Implement a script in <root>/package.json to generate a text-lcov report and upload it to coveralls from CI builds. Remover coveralls from packages/embark.
Supply packages/embark with an nyc configuration in its package.json and have its "test": script generate both json and html reports.
Use nyc with embarkjs's test suite: supply an nyc configuration in its package.json and have its "test": script generate both json and html reports.
Adjust embarkjs's tests for more accurage coverage reporting.

A benefit of these changes is that on coveralls the directory structure of the reports now accurately matches the structure of the monorepo so that the coveralls reports can correctly show the sources.
I'll update this comment with a link to the relevant coveralls report once the CI scripts cause it to be generated.

The motivation for these changes came from my work on a suite of packages (embark-init, embark-cli-utils, etc.) for a future (soon!) PR. I've been aiming for 100% code coverage and wanted to be able to demonstrate that coverage beyond sharing my screen, so I figured out how to leverage coveralls correctly with respect to our monorepo.


add video intro

I think it would be nice to add a link to your video to the readme: https://www.youtube.com/watch?v=yqyQyT8MULk

Updated!


New version: WhereTheWaterFlows v0.5.4

Registering package: WhereTheWaterFlows
Repository: https://github.com/mauro3/WhereTheWaterFlows.jl
Created by: @mauro3
Version: v0.5.4
Commit: 1c3672afc21e2e1f9174f40af1e3aa94ffeea837
Reviewed by: @mauro3
Reference: https://github.com/mauro3/WhereTheWaterFlows.jl/commit/1c3672afc21e2e1f9174f40af1e3aa94ffeea837#commitcomment-56881237

Oops, I should have bumped to 0.6.0

(the correct PR is https://github.com/JuliaRegistries/General/pull/45346)


Help - Custom language and icon

Hi im new to this one.
First this is so amazing work thank you so much.

Is it possible to customize language ?
Is it possible to set custom logo and icon ?
is it possible to change design on login screen ?

Hello @denperss11
The answer is: Kind of.
In fact, everything you mentioned is completely possible, but not very convenient. There isn't any kind of option for setting language, title, logo or things like that (at least currently, this might change in the future).
However you can edit the files you find in server/themes and make PagerMon look like you want it to, but you'll need to have at least a basic understanding of HTML and JavaScript. Simply replace any terms you want to translate, include images you want to be shown or replace the existing files, edit the whole HTML-Structure according to your needs, make some CSS changes (For example to make some kind of a dark mode), ...


Icon request: UseMyCards

URL of the site or app
https://usemycards.com/
Logo resources for the icon
Removed by maintainer (SVG code)
Anything else?
No response

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/UseMyCards.svg

:question: How to add icon(s) to Aegis?
:star: Support this project by giving a star! Just adding like to this comment is also appreciated.


Add mergeable CI gate to check for valid approvals and milestone

Blocked on acceptance of mergeable app permissions on repo

Summary

Add a failed gate check if

There isn't an approved review from a set list of allowed users
There is no PR milestone

Once this PR merges

Add mergeable CI checks as merge blocking
Apply the same file to other ui repos

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[ ] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

Docs
https://mergeable.readthedocs.io/en/latest/validators/approval.html?highlight=limit#approvals
https://mergeable.readthedocs.io/en/latest/validators/milestone.html
First looked in to CODEOWNERS... however that will spam everyone with emails on every PR.


neovim 0.10.3 on Darwin freezes

Describe the bug
Editing a cpp file using the same Neovim configuration:

nixpkgs-unstable.neovim-unwrapped freezes
Neovim installed by Homebrew works well
Neovim from the original repo works well

Steps To Reproduce

nix-shell -p neovim
Create a new .cpp file with the following contents

#include <cstdint>

///\brief Category token.
/// Do not change the enum value, as some old codes use </==/!=/> for comparison (needed_by_csc)
enum class Category : std::uint8_t {
    STOCK = 0,
};

Deleting two / before "Do not change ... "
Neovim stuck

Additional context
No LSP enabled.
Metadata

system: "aarch64-darwin"
host os: Darwin 24.2.0, macOS 15.2
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.24.11
nixpkgs: /nix/store/73cnf2k7mji95fcflvay15pdw4skhhk5-source

Screen recording
The left pane is nvim and the right one is /opt/homebrew/bin/nvim
https://github.com/user-attachments/assets/c4723c4c-20e5-4a07-9120-8b8edab93806

I believe this is linked to those issues:

https://github.com/neovim/neovim/issues/31163
https://github.com/tree-sitter/tree-sitter/issues/3930

It seems to be a bug in tree-sitter 0.24.4, that is fixed in 0.24.5

Fixed by #368952.


Finding dataset for a given error

It would be helpful if the documentation could answer this question:
Probably like most organisations using the Gateway, we have many users refreshing datasets via the Gateway.  I see some errors in the Gateway log files, but I don't know which datasets they relate to.  How can I find the dataset for a given error?
The new Power BI PowerShell commandlets allow me to list all datasets in the tenant, so if I can get the dataset ID for a given error, I can contact the dataset owner, which is what I (in some cases) want to do.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 92f41ca5-f249-e0a8-fb68-d19215f5ca66
Version Independent ID: 74828e70-9e56-afd3-aa11-5bfaab20e384
Content: Troubleshooting the On-premises data gateway - Power BI
Content Source: powerbi-docs/service-gateway-onprem-tshoot.md
Service: powerbi
GitHub Login: @mgblythe
Microsoft Alias: mblythe

Hi, @cbailiss -- thanks for contacting us! The author of this article is out this week, but will be back next week. In the meantime, if you'd like you can try asking in https://community.powerbi.com/.

Thanks @cbailiss -- checking with engineering on functionality.

@cbailiss - unfortunately this is not possible currently but do vote on ideas.powerbi.com so we can work on this feature soon.


vmware-horizon-client-np: Update to version 5.4.0-15805437

VMware Horizon Client 5.4 for Windows Release Notes

Thank you!


Hopefully this shall take care of the issue reported here

https://github.com/RobertDober/earmark_parser/pull/85#issuecomment-1031761529

Is this related to the surface catalogue not showing the components?

@noozo sorry but I am not aware of the context just fixed the bad call in my code base

Hi @RobertDober!
Unfortunately, this only works if the user sets override: true for earmark_parser:
{:earmark_parser, "1.4.18", override: true},

Otherwise, we get conflict with ex_doc when runing mix deps.get:
Failed to use "earmark_parser" (version 1.4.18) because
  earmark (version 1.4.20) requires == 1.4.18
  ex_doc (version 0.28.0) requires ~> 1.4.19
  mix.exs specifies 1.4.18

So it seems users will be forced to adjust the versions manually to get things working for some libs at the cost of potentially breaking other libs.

Hmm maybe you could go with earmark 1.5.0-pre1 ~> earmark 1.4.21  @msaraiva
this might break your lists so
I try to get 1.5.0 out, but the ETA is not near unfortunately

maybe you could go with earmark 1.5.0-pre1 ~> earmark 1.4.21

Hex would not allow us to release a v0.4.0 depending on a *-pre version. I guess we'll have to stick with this version until v1.5.0 is out.
Actually, any project depending on the latest stable Earmark version (v1.14.20) will present the same problem when using recent versions of ex_doc, as the latter requires ~> 1.4.19 and the former only v1.4.18.
I'm closing this issue for now as there's nothing we can do code-wise on our side. The only workaround for now is for the user to explicitly set earmark_parser version to 1.4.18 with override: true.

Hex would not allow us to release a v0.4.0 depending on a *-pre version. I guess we'll have to stick with this version until v1.5.0 is out.

That is why I would like to help out with Earmark 1.4.21 =>  Earmark Parser 1.4.22

Hey @RobertDober!
I've just updated Earmark to 1.4.21 with Earmark Parser 1.4.22 and it works perfectly. Thank you so much! ❤