Syntax suggestion: Ignore last void parameters

class X<T>
{
    f(t: T) { return { a: t }; }
}

var x: X<number>;
var t1 = x.f(5);
t1.a = 5; // Should not error: t1 should have type {a: number}, instead has type {a: T}

var x2: X<void>;
var t2a = x.f(void 0); // It works but void 0 it's unnecessary
var t2b = x.f(); // SUGGESTION: ignore syntax check of last argument if it's type is void
t2b.a = void 0; // Should not error: t2 should have type {a: void}, instead has type {a: T}

test case: tests/cases/compiler/genericObjectLitReturnType.ts
maybe the solution is the same for #3356

Would I be correct in saying that the type signature T|void should be equivalent to T?? (Any type which contains the ) This equality is true in the closure compiler type system (but it uses undefined rather than void for the type name).

No @weswigham.
class X<T, U>
{
    f(t: T|void, u: U) { ... }
}

t: T it's mandatory because is not the last function argument and we can't lose u: U. My suggestion it's only for last argument. However, the rule can be last void parameters because if X<void,void> f function can be invoke with f() lossless
I don't understand when T|undefined equality is true in the closure compiler, can you send me a sample?
Thanks

Oh, sorry, I was (completely) mistaken - The equality's not true for optional function arguments, only optional record type members. :S

Chatting with @danquirk, our proposal (roughly) is that at a call expression, we treat the unprovided required arguments as if they were of type void, then check that those are assignable to the parameter types (modulo any). Leave assignability/subtyping of function types unchanged.

Yes @RyanCavanaugh, just for call expressions. I don't know how works typescript checker then I can't say if it's better don't throw error if last parameter is void and missing or if it's treat the unprovided required arguments as if they were of type void.  But it's what you say, just in call expression leaving assignability/subtyping of function types unchanged.
If I can help you just tell what to do.
Thanks.

Approved. Should be useful for Promise<void>.

@RyanCavanaugh What is the status on this issue? I saw that @Kingwl was working on a PR that was abandoned (#20642) with some spec issues raised by mhegazy.
Anyway, I have a fix that works for the given examples as well as generic rest / tuple types here 9f6e28b053f94f71ad49438a9b9822bd4b30bb5e if you want to breathe some life into this issue.
@Kingwl : I didn't mean to conflict with your work (I didn't actually see your old PR until I finished my fix). If you want to take my work and finish the PR I would be fine with that.

@jack-williams take it easy, I have already close the pr

@jack-williams It does not seem to work with constructors. Is this intentional or were constructors just forgotten?

@FrogTheFrog Can you provide an example?

@FrogTheFrog Thanks for the example. The issue is not to do with constructors; the issue is the generics. See this comment here in an older PR I made.
Basically the special casing for void does not apply to arguments that are instantiated as type void at the point the argument is called. Getting it to work with generics caused too many problems to ultimately be worth it.
Here is a workaround if you really want it:
class Test<T1, T2 = void> {
    constructor(input1: T1, input2: T2) {
        // Do something
    }

    static makeFactory<T1, T2 = void>(): (input1: T1, input2: T2) => Test<T1,T2> {
        return (input1: T1, input2: T2) => new Test(input1, input2);
    }
}

const factory = Test.makeFactory<number>();
const factoryTwo = Test.makeFactory<number, number>();
const testOne = factory(3); // ok
const testTwo = factoryTwo(3); // not ok

@jack-williams I'm just nitpicking at this point. Since I mainly use "options" object for public classes, it does not really bother me. I'm using this simple trick fallback to void type when nothing is provided:
interface TestOptions<T1, T2> {
    input1: T1;
    input2?: T2;
}

class Test<T1, T2 = void> {
    constructor(options: TestOptions<T1, T2>) {
        if (options.input2 !== void 0) {
            // Do something with input2
        }
    }
}

const test = new Test({ input1: {} }); //=> test has Test<{}, void> type

I'm just glad that the void 0 nonsense is finally over thanks to you :D

Why does this fail?  playground
function qwe<T = void>(d: T) {}
qwe(); // ERROR: Expected 1 arguments, but got 0. An argument for 'd' was not provided.

Why does this fail?  playground
function qwe<T = void>(d: T) {}
qwe(); // ERROR: Expected 1 arguments, but got 0. An argument for 'd' was not provided.

@garkin

Basically the special casing for void does not apply to arguments that are instantiated as type void at the point the argument is called. Getting it to work with generics caused too many problems to ultimately be worth it.

The checks are done before T is replaced with void. This is tracked here: #29131

Thank you!


Add functions to hold and release mouse buttons

In some cases, i often need to long press the mouse left or right.
so the  mouse_press & mouse_release  is very important.

Has been realized with https://github.com/qb-0/pyMeow/commit/27978bde33fd3fc5d4819ff1cf8ebff4b1a84124


adding ability to change tomcatShutdownPort

When JIRA is ran on a server with existing tomcat instance, the shutdown port conflicts (hardcoded to 8005). This PR adds a param to init.pp to override the default.

Looks okay to me. @disassembler  would you mind adding an appropriate RSpec test as well in jira_config_spec.rb and then doing a force push on your branch?

This is simple enough that I'm willing to merge without a test just to get this in.


Page Navigate. Function SetPostingDate set to global

On page Navigate I suggest make SetPostingDate to global, or add an event where we can change PostingDateFilter based on the NewSourceRecVar variable.
It's needed for a document that was posted 2 times with different posting dates.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Possible spritemap support for 2019+.

Not sure if it supports lua. Will test later when I find someone to volunteer for that.

Why did you add a haxelib library in Project.xml and copy the files to the source folder? Only one of those is neccessary.

Ima test this by importing the code from the repository
to see if this works
(Theres no changes to the Character Editor because it still uses the bb one)

Obviously wont work

I probably was doing this while I was out of it. Let me re-do it. So don't approve for now.

Or close it for now n' stuff.

Actually yea I'll close it for now and redo it later on.

Also the haxelib was added from one of the project.xml samples on the flxanimate download.


Nice Plugin!

Hi I like your plugin and feel that it works quite well.
I had to use it in an iframe as the css breaks my site style which was a little tricky with react.
I would like to see a slightly improved interface as there isn't enough padding between text elements and you should use better grays such as the tailwind grays.
I feel an even three columns would look more appealing than 3 different width columns.
I liked that i could pre-fill the form with the previous inputs from my website.
It would be nice to configure actual times that I am available, rather than the current rules, like instead of 1hr meeting, 30m break before, 5 min break after, i would just provide it a list of time slots. this would help sometimes there is times in the middle of the day where you have regular meetings that need to be blocked out every week.
Love what you're doing and I hope it goes well!

@mrdavidrees You can create multiple events and open the url dynamically.
For example, create 3 events with following eventUrl -

/5min
/15min
/30min

Then open the widget dynamically with selected URL or button click:
const mainUrl  = 'https://vikash.dayschedule.com';
daySchedule.initPopupWidget( {
   url: mainUrl + eventUrl
});

Or you may add multiple duration options in same event as well.

We will review the tailwind styling in our weekly meeting.


feat: adapt settings for mobile (UI)

Summary
⚠  This PR depends on #2251, when merged we should change the base to feat/kickoff-mobile
Changelog
Adapt settings UI to mobile

Relevant Issues
Please list any related issues (e.g. bug, task).
Type of Change
Please select any type below that is applicable to your changes, and delete those that are not.

[ ] Breaking - any change that would cause existing functionality to not work as expected
[ ] Chore - refactoring, build scripts or anything else that isn't user-facing
[ ] Docs - changes to the documentation
[ ] Fix - a change which fixes an issue
[ ] New - a change which implements a new feature
[x] Update - a change which updates existing functionality

Testing
Platforms
Please select any platforms where your changes have been tested.

Desktop

[ ] MacOS
[x] Linux
[ ] Windows

Mobile

[ ] iOS
[x] Android

Instructions
You can test the changes from packages/mobile with the usual requirements for Android or iOS.
Additionally, you can test from packages/desktop changing:

In packages/desktop/App.svelte

import { mobile } from 'shared/lib/app'
mobile.set(true)

In packages/desktop/App.svelte move the popup component to be placed right before the Toast Container

...

{#if settings}
    <Settings locale={$_} handleClose={() => (settings = false)} />
{/if}

{#if $popupState.active}
    <Popup
        type={$popupState.type}
        props={$popupState.props}
        hideClose={$popupState.hideClose}
        fullScreen={$popupState.fullScreen}
        transition={$popupState.transition}
        locale={$_}
    />
{/if}

<ToastContainer />

...

In packages/shared/lib/router.ts force to login using the following script:

const profileId = 'XXXXXXXXXXXXXXXXXXXXXXXX';
const profileName = 'XXXXXX'
const profilePin = 'XXXXXX'
setActiveProfile(profileId)
Platform.PincodeManager.verify(profileId, profilePin)
    .then((verified) => {
        if (verified === true) {
            return Electron.getUserDataPath().then((path) => {
                initialise(profileId, getStoragePath(path, profileName))
                api.setStoragePassword(profilePin, {
                    onSuccess() {
                        login()
                        setRoute(AppRoute.Welcome)
                    },
                    onError(err) {
                        console.error(err)
                    },
                })
            })
        }
    })
    .catch((error) => {
        console.error(error)
    })

⚠ Please beware in desktop the scrollbars are not the same as in mobile, in desktop they take space they dont in mobile
Checklist
Please tick all of the following boxes that are relevant to your changes, and delete those that are not.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[ ] I have verified that new and existing unit tests pass locally with my changes
[x] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

replaced by #2590


Please clarify license

Hi,
could you please state the license under which this project is open sourced?
Cheers
Martin

Hi Martin,
Haven't really thought about it, as I haven't worked on this project in a
while (unfortunately, I didn't have time to really dive in and implement
all of browsermob proxies features, so I decided to keep using browsermob
for the time being)
I can put it at whatever license you'd like.
Thanks,
Roy Arnon
On Mon, Apr 13, 2015 at 11:37 AM, Martin Treusch von Buttlar <
<EMAIL_ADDRESS>wrote:

Hi,
could you please state the license under which this project is open
sourced?
Cheers
Martin
—
Reply to this email directly or view it on GitHub
https://github.com/Hellspam/goharproxy/issues/1.

I would prefer an open one, i.e. MIT License or the Go License.
Cheers
Martin

Added license


model size different

I want to train pysot based on mobilenetV2.
The siamrpn_mobilev2_l234_dwxcorr model that downloaded in ModelZoo was 44.9MB in Ubuntu16 system.
But when I train a new model using config in "experiments/siamrpn_mobilev2_l234_dwxcorr/config.yaml", the model size was 75.6MB.
So is this normal？

there is not optimizer state dict in our provided models. It's normal.


Feature: React Native Layout Model

I'm too busy to start experimenting and contributing into this paradigm shifting project.
I assume this functionality is planned and we're all salivating over the day we can replace all our RN primitives and render our entire apps with react-native-skia. Implementing the layout model used in RN styles will be a big step towards the takeover.
Turns out there's a project that has been working towards similar functionality for react-three-fiber.
It is a bit off-spec and over developed being designed for a 3d canvas context but there is a solid foundation that will give us a head start.
@react-three/flex
Hope this helps!

Awesome! No doubt I might succumb to temptation and start tinkering.
Either way I can guarantee it will be released before React Native Fiber...

Closing this issue - but we'll be keeping a note of this one!


”Moșna”, ”Alma Vii”, ”Nemșa” are listed in the drop-down menu

Description:
When user writes ”Moșna”, ”Alma Vii”, ”Nemșa” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Moșna"

Expected results
The location appears in the drop-down menu as "Sat Moșna, SIBIU (Moșna)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Alma Vii"

Expected results
The location appears in the drop-down menu as "Sat Alma Vii, SIBIU (Moșna)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Nemșa"

Expected results
The location appears in the drop-down menu as "Sat Nemșa, SIBIU (Moșna)".


[Support] How to increase WireGuard speed on Raspberry Pi 3 Model B+? Upgrade to Raspberryr Pi 4 Model B?

In raising this issue, I confirm the following:
{please fill the checkboxes, e.g: [X]}

[] I have read and understood the contributors guide.
[] The issue I am reporting can be replicated.
[] The issue I am reporting can be is directly related to the pivpn installer script.
[] The issue I am reporting isn't a duplicate (see FAQs, closed issues, and open issues).

Issue
Have you searched for similar issues and solutions?
(yes/no / which issues?)

Console output of      curl -L install.pivpn.dev | bash
  Output Here

Console output of      pivpn add or pivpn add nopass
  Output Here

Console output of      pivpn debug
  Output Here

Have you taken any steps towards solving your issue?
  which?

accidentally published


raylib: fix darwin builds

Description of changes
Fixes builds on darwin (see this hydra job for the details on the build failure)
Things done
Only run autoPatchelfHook on linux

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

Is something wrong with the darwin runners?

They usually take 2+ days to build. We have less darwin machines and they can't be emulated afaik

Result of nixpkgs-review pr 337681 run on x86_64-darwin 1

1 package marked as broken and skipped:

eepers

4 packages failed to build:

python311Packages.raylib-python-cffi
python311Packages.raylib-python-cffi.dist
python312Packages.raylib-python-cffi
python312Packages.raylib-python-cffi.dist

2 packages built:

raylib
raylib-games

Result of nixpkgs-review pr 337681 run on x86_64-darwin 1
1 package marked as broken and skipped:
4 packages failed to build:

python311Packages.raylib-python-cffi
python311Packages.raylib-python-cffi.dist
python312Packages.raylib-python-cffi
python312Packages.raylib-python-cffi.dist

2 packages built:
These results are fine; raylib built, which is more than can be said for master. That just uncovered a few dependent packages' build failures.

Let's see if I can fix python311Packages.raylib-python-cffi too

Okay, so, raylib-python-cffi requires raylib to be built as a static library on macOS, which succeeds but fails later trying to import symbols from GLFW:
Executing pythonImportsCheckPhase
Check whether the following modules can be imported: pyray
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "<string>", line 1, in <lambda>
  File "/nix/store/xnm9sbp02pp2704dvx4kb8g0yxn263pn-python3-3.12.4/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/nix/store/d2jz44lcpi7yzqdilqazkgiqi7qyijkm-python3.12-raylib-python-cffi-<IP_ADDRESS>/lib/python3.12/site-packages/pyray/__init__.py", line 18, in <module>
    from raylib import rl, ffi
  File "/nix/store/d2jz44lcpi7yzqdilqazkgiqi7qyijkm-python3.12-raylib-python-cffi-<IP_ADDRESS>/lib/python3.12/site-packages/raylib/__init__.py", line 15, in <module>
    from ._raylib_cffi import ffi, lib as rl
ImportError: dlopen(/nix/store/d2jz44lcpi7yzqdilqazkgiqi7qyijkm-python3.12-raylib-python-cffi-<IP_ADDRESS>/lib/python3.12/site-packages/raylib/_raylib_cffi.abi3.so, 0x0002): symbol not found in flat namespace '_glfwGetProcAddress'

At this point, I went to try and check if the static builds of raylib work on their own and they don't, so yeah, I'm not fixing that

Not to discourage you from trying, but it's worth noting that I am pretty sure no one has tried (successfully, at least) to build python311Packages.raylib-python-cffi before because that package has no accommodations for Darwin right now and it would at least need a handful of Darwin libraries to be included as buildInputs in order to succeed: https://github.com/electronstudio/raylib-python-cffi/blob/df2bbccb1f2451a601209163fd8d297129eb49d5/raylib/build.py#L163..L165. So you'd be making it work for the first time rather than fixing something that broke.
I'm not very familiar with the various tactics folks use to build Python packages for Nixpkgs so I may be off here, but my gut says it'd be worth patching raylib-python-cffi to use the dylibs instead of building raylib statically. I am only guessing that would be easier since in the context of Nixpkgs we can control linking and search paths so dynamic vs. static is less of a concern but I don't know the exact reason why raylib-python-cffi requires a static build of raylib.

Not to discourage you from trying, but it's worth noting that I am pretty sure no one has tried (successfully, at least) to build python311Packages.raylib-python-cffi for Darwin before because that package has no accommodations for Darwin right now and it would at least need a handful of Darwin libraries to be included as buildInputs in order to succeed: https://github.com/electronstudio/raylib-python-cffi/blob/df2bbccb1f2451a601209163fd8d297129eb49d5/raylib/build.py#L163..L165. So you'd be making it work for the first time rather than fixing something that broke.
I'm not very familiar with the various tactics folks use to build Python packages for Nixpkgs so I may be off here, but my gut says it'd be worth patching raylib-python-cffi to use the dylibs instead of building raylib statically. I am only guessing that would be easier since in the context of Nixpkgs we can control linking and search paths so dynamic vs. static is less of a concern but I don't know the exact reason why raylib-python-cffi requires a static build of raylib.

Reporting back with `raylib-python-cffi` working :)

here is fine, just ensure it's a separate commit.

Result of nixpkgs-review pr 337681 run on aarch64-darwin 1

6 packages built:

python311Packages.raylib-python-cffi
python311Packages.raylib-python-cffi.dist
python312Packages.raylib-python-cffi
python312Packages.raylib-python-cffi.dist
raylib
raylib-games

Result of nixpkgs-review pr 337681 run on x86_64-darwin 1

1 package marked as broken and skipped:

eepers

6 packages built:

python311Packages.raylib-python-cffi
python311Packages.raylib-python-cffi.dist
python312Packages.raylib-python-cffi
python312Packages.raylib-python-cffi.dist
raylib
raylib-games

FYI I figured we'd wait a few days for any additional feedback since I don't maintain either of these packages but I'll merge this in another day or two.


macOS Ventura: Crash after closing the Options/Settings Sheet

M1 mac mini,
macOS Ventura Beta 1 22a5226r
two monitors (17" HDMI and 15" VGA)
I believe this is a macOS bug, as it's being seen with various screensavers such as https://github.com/soffes/Clock.saver and https://iscreensaver.com
Reported to Apple Feedback as FB10103112.
Seen both with ARM native and Intel x64 Rosetta emulated screensavers.
I'm trying to find a workaround using my own screensaver software, if I stumble across anything I will report back.
Example crash log:
Process:               Screen Saver [3234]
Path:                  /System/Library/ExtensionKit/Extensions/Screen Saver.appex/Contents/MacOS/Screen Saver
Identifier:            com.apple.ScreenSaver-Settings.extension
Version:               1.0 (1)
Build Info:            DesktopScreenEffectsPref-283000000000000~10
Code Type:             ARM-64 (Native)
Parent Process:        launchd [1]
User ID:               501

Date/Time:             2022-06-08 07:46:13.7537 -0700
OS Version:            macOS 13.0 (22A5266r)
Report Version:        12
Anonymous UUID:        E1E1B825-F9F6-3DB0-A932-7E0490DDEA1D

Time Awake Since Boot: 2100 seconds

System Integrity Protection: enabled

Crashed Thread:        0  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_BREAKPOINT (SIGTRAP)
Exception Codes:       0x0000000000000001, 0x0000000102699b98

Termination Reason:    Namespace SIGNAL, Code 5 Trace/BPT trap: 5
Terminating Process:   exc handler [3234]

Thread 0 Crashed::  Dispatch queue: com.apple.main-thread
0   Screen Saver                  	       0x102699b98 0x102684000 + 88984
1   Screen Saver                  	       0x1026999e8 0x102684000 + 88552
2   Screen Saver                  	       0x10268d80c 0x102684000 + 38924
3   ScreenSaver                   	       0x1c8a3dbd0 -[LegacyScreenSaverModule _requestDidCompleteNotification:] + 360
4   CoreFoundation                	       0x18a87c2d4 __CFNOTIFICATIONCENTER_IS_CALLING_OUT_TO_AN_OBSERVER__ + 148
5   CoreFoundation                	       0x18a9199bc ___CFXRegistrationPost_block_invoke + 88
6   CoreFoundation                	       0x18a919904 _CFXRegistrationPost + 440
7   CoreFoundation                	       0x18a84c42c _CFXNotificationPost + 708
8   Foundation                    	       0x18b72b778 -[NSNotificationCenter postNotificationName:object:userInfo:] + 88
9   ScreenSaver                   	       0x1c8a34ed8 __58-[ScreenSaverExtensionModule initWithExtension:isPreview:]_block_invoke.28 + 288
10  ExtensionFoundation           	       0x1f4d964ac __93-[EXConcreteExtension _completeRequestReturningItems:forExtensionContextWithUUID:completion:]_block_invoke + 132
11  libdispatch.dylib             	       0x18a61daf4 _dispatch_call_block_and_release + 32
12  libdispatch.dylib             	       0x18a61f61c _dispatch_client_callout + 20
13  libdispatch.dylib             	       0x18a62ddd4 _dispatch_main_queue_drain + 928
14  libdispatch.dylib             	       0x18a62da24 _dispatch_main_queue_callback_4CF + 44
15  CoreFoundation                	       0x18a8c854c __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__ + 16
16  CoreFoundation                	       0x18a8863c4 __CFRunLoopRun + 2036
17  CoreFoundation                	       0x18a88547c CFRunLoopRunSpecific + 612
18  HIToolbox                     	       0x193fbe634 RunCurrentEventLoopInMode + 292
19  HIToolbox                     	       0x193fbe478 ReceiveNextEventCommon + 700
20  HIToolbox                     	       0x193fbe1a4 _BlockUntilNextEventMatchingListInModeWithFilter + 72
21  AppKit                        	       0x18dbabd78 _DPSNextEvent + 632
22  AppKit                        	       0x18dbaaf08 -[NSApplication(NSEvent) _nextEventMatchingEventMask:untilDate:inMode:dequeue:] + 728
23  ViewBridge                    	       0x19240c4d0 __75-[NSViewServiceApplication nextEventMatchingMask:untilDate:inMode:dequeue:]_block_invoke + 136
24  ViewBridge                    	       0x19240c2f0 -[NSViewServiceApplication _withToxicEventMonitorPerform:] + 152
25  ViewBridge                    	       0x1923fa304 -[NSViewServiceApplication nextEventMatchingMask:untilDate:inMode:dequeue:] + 168
26  AppKit                        	       0x18dd54470 -[NSWindow(NSEventRouting) trackEventsMatchingMask:timeout:mode:handler:] + 216
27  ViewBridge                    	       0x192460718 -[NSRemoteView _beginTrackingLoop:reply:] + 472
28  ViewBridge                    	       0x19246394c -[NSRemoteViewMarshal beginTrackingLoop:reply:] + 84
29  CoreFoundation                	       0x18a869bc4 __invoking___ + 148
30  CoreFoundation                	       0x18a869a3c -[NSInvocation invoke] + 428
31  ViewBridge                    	       0x1923fb534 __deferNSXPCInvocationOntoMainThread_block_invoke + 132
32  ViewBridge                    	       0x1923f0814 __wrapBlockWithVoucher_block_invoke + 56
33  ViewBridge                    	       0x19248147c kNotRunningOnAppKitCompatibleThread_block_invoke + 360
34  CoreFoundation                	       0x18a8871a4 __CFRUNLOOP_IS_CALLING_OUT_TO_A_BLOCK__ + 28
35  CoreFoundation                	       0x18a8870b8 __CFRunLoopDoBlocks + 368
36  CoreFoundation                	       0x18a886564 __CFRunLoopRun + 2452
37  CoreFoundation                	       0x18a88547c CFRunLoopRunSpecific + 612
38  HIToolbox                     	       0x193fbe634 RunCurrentEventLoopInMode + 292
39  HIToolbox                     	       0x193fbe478 ReceiveNextEventCommon + 700
40  HIToolbox                     	       0x193fbe1a4 _BlockUntilNextEventMatchingListInModeWithFilter + 72
41  AppKit                        	       0x18dbabd78 _DPSNextEvent + 632
42  AppKit                        	       0x18dbaaf08 -[NSApplication(NSEvent) _nextEventMatchingEventMask:untilDate:inMode:dequeue:] + 728
43  ViewBridge                    	       0x19240c4d0 __75-[NSViewServiceApplication nextEventMatchingMask:untilDate:inMode:dequeue:]_block_invoke + 136
44  ViewBridge                    	       0x19240c2f0 -[NSViewServiceApplication _withToxicEventMonitorPerform:] + 152
45  ViewBridge                    	       0x1923fa304 -[NSViewServiceApplication nextEventMatchingMask:untilDate:inMode:dequeue:] + 168
46  AppKit                        	       0x18db9f2cc -[NSApplication run] + 464
47  AppKit                        	       0x18db76838 NSApplicationMain + 880
48  libxpc.dylib                  	       0x18a522840 _xpc_objc_main + 976
49  libxpc.dylib                  	       0x18a5220ec xpc_main + 108
50  Foundation                    	       0x18b79aeb8 -[NSXPCListener resume] + 312
51  ExtensionKit                  	       0x1f4df5170 -[_EXRunningUIViewServiceExtension startWithArguments:count:] + 1148
52  ExtensionFoundation           	       0x1f4db0c4c EXExtensionMain + 232
53  Foundation                    	       0x18b7f80c8 NSExtensionMain + 240
54  dyld                          	       0x216b5fc10 start + 2368

I notice that some of the sample code calls .loadWindow() on the Preferences window controller.
Example:
https://github.com/soffes/Clock.saver/blob/main/Clock/Classes/MainView.swift#L24
Apple says

You should never directly invoke this method. Instead, access the window property so the windowDidLoad() and windowWillLoad() methods are invoked

See https://developer.apple.com/documentation/appkit/nswindowcontroller/1535137-loadwindow
I've played with not calling loadWindow() but it didn't seem to help.

For what it's worth, I don't use loadWindow on either Aerial nor my ScreenSaverMinimal sample.

Historically closing the sheet has been a mess as there are few solutions that work accross multiple macOS versions

Agreed.  In earlier OS's you could do this:
override var hasConfigureSheet: Bool { return true }
override var configureSheet: NSWIndow? {
  // launch a stand-alone app which hosts the configure sheet
         configProcess = Process()
         configProcess?.arguments = arguments
          configProcess?.launchPath = "/path/to/helper/app"
          configProcess?.terminationHandler = configTerminatedHandler
          configProcess?.launch()

 return nil
}

However, in my testsing, it's very fickle:
macOS 10.11 through 10.13 :   configureSheet=nil works OK
macOS 10.14 11.x, 12.x :   configureSheet=nil crashes, so need to provide a dummy configureSheet
macOS 13.0 :   configureSheet crashes when not nil.  Maybe try using nil again?
Since Aerial already has the helper app, I wonder if you should consider doing something similar - hosting the Preferences in a stand-alone app?

Hmm, I had no idea this was possible !
Is your helper somewhere in an app in /Applications ? Despite the sandboxing you can launch stuff easily and I do that a bit for various stuff, including launching some app bundled with companion (although that's not fully baked in right now) but it's mostly command line stuff.
I do it in reverse right now, Aerial Companion loads Aerial.saver and puts it in a window or triggers the configuresheet manually. My biggest issue is I want to sync settings between both and the sandbox makes it hard.
In your situation though, I'm unclear about how settings work ? Your app panel conigureSheet should have it's settings outside the sandbox ? Yet the screensaver proper will read from the legacyScreenSaver sandbox ? How do you work that around, or am I missing something ?
In any case thanks for the info a lot of food for thought !

Interesting.
Here's what I'm doing:

the Helper app is inside the .saver bundle, located at : Example.saver/Contents.Helpers/Helper.app
the helper app reads/writes data to/from  ~/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Preferences/com.example.plist (or legacyScreenSaver.x86-64 when running under Rosetta, since this is an Intel screensaver.
the helper app is not, itself, sandboxed, but seems to be fine working within the sandbox that legacyScreenSaver is running inside.
I'm guessing that this woks because the helper.app is being launched from within the .saver when it's running sandboxed?
I did run into one issue: my helper app uses WKWebView2, and that wasn't happy running inside this stack - I reverted to WebView and it worked fine.  I think this bug was fixed in later macOS 12.x versions

I'll keep an eye on this but considering it's beta 1 and a whole new System Preferences, I think the odds are higher than usual they'll fix this. If needed I'll also file another feedback, that does help sometimes. I haven't checked yet if Obj-C screensavers that are run through legacyScreenSaver also have the same issue. It could very well be a Swift thing (there was a lot of that in Catalina).
Regarding the other stuff:

helper app not sandboxed : I can also confirm that even when you call an app outside the bundle, it runs unsandboxed (which completely undermines the sandbox, but hey...)
prefs file in legacyScreenSaver : Yep, I believe that because your app is inside the .saver bundle, that's why you end up having your plist saved in the sandbox so that makes sense. It's still an interesting idea on how to do things and I'll think about it to see it it makes sense.
Does WKWebView2 require some specific entitlement ? I think I tried using it at some point though that wasn't on Aerial I think, and had issues.
Sharing prefs : Theoretically yes, the unsandboxed version of Aerial could definitely read/write the sandboxed file. It's unclear to me however how to approach this, since we use in screen savers the  the ScreenSaverDefault wrapper thing, I may need to override that in some way, though I'm not sure yet how. I hope to investigate this soon.

Does WKWebView2 require some specific entitlement ? I think I tried using it at some point though that wasn't on Aerial I think, and had issues.

Have not had any issues using it within the actual .saver, however we are only using it to read HTTP data via a localhost server on the same mac.  I wouldn't be surprised if the sandbox might prevent other uses?

Sharing prefs : Theoretically yes, the unsandboxed version of Aerial could definitely read/write the sandboxed file. It's unclear to me however how to approach this, since we use in screen savers the  the ScreenSaverDefault wrapper thing, I may need to override that in some way, though I'm not sure yet how. I hope to investigate this soon.

I believe that ScreenSaverDefault stores data in ~/Library/Preferences/ByHost/com.example.[GUID String].plist (or the equivalent sandboxed location).  I've never liked or understood the ByHost folder - I think it's something that was supposed to enable one person to be logged into the same account using multiple machines?  (Wasn't that an idea of the NextStep era?) but I've always just found it confusing and clumsy.  My screensaver just saves preferences using

CFPreferencesSetValue(key as CFString, val as CFString, self.bundleID as CFString, kCFPreferencesCurrentUser, kCFPreferencesAnyHost)
[...]
CFPreferencesAppSynchronize(self.bundleID as CFString)

which puts data in
~/Library/Preferences/[bundleID].plist
which I find a lot easier to use.

@xmddmx
First sorry for the late reply, missed your message the other day and had a busy week.

Have not had any issues using it within the actual .saver, however we are only using it to read HTTP data via a localhost server on the same mac. I wouldn't be surprised if the sandbox might prevent other uses?

This may be exactly that, I think one issue I had was regarding https being mandatory.

I believe that ScreenSaverDefault stores data in ~/Library/Preferences/ByHost/com.example.[GUID String].plist (or the equivalent sandboxed location). I've never liked or understood the ByHost folder - I think it's something that was supposed to enable one person to be logged into the same account using multiple machines? (Wasn't that an idea of the NextStep era?)

If I remember correctly, the ByHost folder is like a "host temporary" preferences, as in those only apply to this machine and could be recreated easily (things that must be tied to the machine like they have some unique MAC address or something). Which is not really great for a screensaver, but that's where it puts them sadly. Those settings are not always migrated in some scenarios (like machine upgrade), and if I remember correctly settings were lost when legacyScreenSaver.appex was introduced, and also in some other macOS transition (can't remember which). Just a pain.

but I've always just found it confusing and clumsy. My screensaver just saves preferences using

CFPreferencesSetValue(key as CFString, val as CFString, self.bundleID as CFString, kCFPreferencesCurrentUser, kCFPreferencesAnyHost)
[...]
CFPreferencesAppSynchronize(self.bundleID as CFString)

which puts data in ~/Library/Preferences/[bundleID].plist  which I find a lot easier to use.

That's... super super useful, thanks a whole lot. I'll have a look at implementing this as this would solve my biggest issue right now with the desktop/full screen mode of Companion. I had an idea it was probably possible but I was completely unfamiliar with those APIs ! Thanks again 👍
Now, regarding your original issue, I had a look with my ScreenSaverMinimal project and tried a few "other" things to dismiss the panel but none worked.
I also tried two other screensavers to see if I could reproduce just to be sure :

Fliqlo, made with Swift
ElectricSheep, Obj-C (or C, but definitely not swift !)

Both have the exact same issue, so if I had to guess, there's a callback from legacyScreenSaver.appex that seems to crash the new Screen Saver.appex. I'll file a radar with more details just in case.
Couple of other bugs I saw :

Sometimes the settings panel preview keeps restarting the preview for no reason that I can see, throwing it in an unresponsive loop. This may be more of an Aerial specific issue than a general screen saver issue though (could be related to video playback/HDR)
Double clicking a .saver file to install it no longer works
The isPreview bug (a flag that tells you if you are in the tiny preview or fullscreen) is finally fixed ! It's been here since Catalina where it always returned false. Now it correctly returns true in that mode (and false in fullscreen) ! Progress ! (that bug was nagging me a lot)
There's been no changes in entitlements to legacyScreenSaver.appex (so still can't use keyboard binding in the screensaver)

I put a list so I can track stuff in the wiki here : https://github.com/glouel/ScreenSaverMinimal/wiki/Issues-with-macOS-Ventura-betas
Hopefully things will get better in a couple of betas, I'll keep you posted if I find other stuff.

@xmddmx You probably noticed this but the original issue (crashing after closing) has been fixed in Ventura beta3.
Also, double clicking a .saver to install works again.
Last thing, I tried your methods to write and read preferences but I'm running into an issue when running in sandbox, the CFPreferences api will read and write inside the container. I'm wondering if you have a trick to read the specific unsandboxed file. Thanks !

I agree, those 2 issues (crash after close the settings panel, double-click .saver file) seem fixed in Beta 3.
Regarding CFPreferences : I think I'm doing the opposite of what you are.  In my setup, the screensaver itself reads/writes settings from within the sandbox container.   I have an external app, which is not sandboxed, which can also read/write those same settings in the container.
It sounds like you are trying to go the opposite direction, where the .saver is trying to break out of the sandbox?

Thanks for taking the time to answer @xmddmx.
This is where I'm not getting it. If I call CFPreferences from outside the sandbox, I will read and write the one outside the sandbox. I don't see how to specify the sandbox path ?
CFPreferencesAppSynchronize("com.glouel.synctest" as CFString) <- should I pass the path here ???

Perhaps using a relative path with "~" doesn't work?
I had to go back and untangle my code, but here's what I'm doing to read from inside the Sandbox from an app Outside the sandbox:
bundleID = "/Users/username/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Preferences/com.mycompany.myscreensaver"

key = "foobar"

value = CFPreferencesCopyValue(key, bundleID, kCFPreferencesCurrentUser, kCFPreferencesAnyHost)

This works for me.  Writing a value is similar, just uses the CFPreferencesSetValue() method.

Also, if your App and .Saver are Intel, you would need to replace legacyScreenSaver with legacyScreenSaver.x86-64 when running on a M1 system.

Absolute path it is...
With an absolute path, it works, and I can succesfully read write from outside the container to it 🎉
Seriously I can't thank you enough for helping me on this, I've been stuck on this for about 6 months on and off, and this should finally solve it.
Thanks for your patience !

FYI, I did further testing and the absolute path actually also work with the "modern" UserDefault API :
        // Test 2
        let bundleID2 = "/Users/guillaume/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Preferences/com.glouel.synctest2"
        let userDefaults = UserDefaults(suiteName: bundleID2)
        userDefaults?.setValue(time, forKey: "lastRun")

        userDefaults?.synchronize()

This actually works ! I'm working on cleaning it all up as I have a mix of stuff currently in preferences but this is looking very very promising. Again many thanks for sharing all that info !

Glad to be of help!

I'd be interested in pursuing some other quality-of-life issues with screensavers on Ventura.  I've pretty much given up on the ability to capture Keyboard events on Catalina or higher, but I did have a pretty robust way of capturing mouse events, which allows the screensaver to provide on-screen controls / Heads-Up-Display (HUD).   This seems broken in Ventura.
Are you interested/willing to collaborate?  I don't have any github repos, so would you mind if I posted a new Issue here?  Or would it perhaps be better posted on ScreenSaverMinimal ? https://github.com/AerialScreensaver/ScreenSaverMinimal/issues

Sure thing, if we can find ways to improve things, I'm all for it.
Either repo is fine to me, this one may have a tiny bit more visibility although I'm not sure it matters much.


Gas analyzer

About the PR
This PR brings Gas Analyzers into the game, to help engineers deal with atmospherics problems which are hard to assess when you are in a hardsuit (such as: is this area still spaced? Is medbay too hot to clone people?)
Specifically:

the Gas Analyzer bound user interface was ported to xaml (it seems it was an old thing predating the usage of xaml in the code base), which makes it resizable (the old UI wasn't, and was truncating informations when a lot of different gases were present)
the ActivatableUI component was added the the gas analyzer (and its cow tools counterpart) so that the UI closes when the analyzer is not in the active hand, matching the behavior of instruments.
the Gas Analyzer was added to YouTool and tool lockers.

Screenshots

Changelog

:cl:

add: Nanotrasen have provisioned YouTool and tool lockers with Gas Analyzer devices, which can measure the temperature, pressure and composition of the ambient atmosphere, in an effort to make engineers less reliant on human guinea pigs to ascertain whether a particular area is livable.

This PR is a bit misleading, it doesn't add gas analyzers, it just changes them and adds them to vending machines. You should reflect the actual changes in your PR title better in the future.
Aside from that, I think the old gas analyzer window looked much better:

I like the gas color names in your version, but the old version was definitely more pleasant to look at.

Does the Analyzer refresh automatically now then? Maybe this is less desirable than a refresh button since you might want to take a sample and GTFO.
And yes if this window is resizable why does it open so big compared to the length of text? The graph should resize correctly if you shrink it no?
Otherwise pretty good.

This PR is a bit misleading, it doesn't add gas analyzers, it just changes them and adds them to vending machines. You should reflect the actual changes in your PR title better in the future.

Alright, will do. I thought it would be better to keep the title short.

Regarding the window size, I agree that it looks less cool. I will look into getting it to resize automatically to deal with the variation in text length depending on the number of gases.
Should I look at the MeasureOverride method for this?

Does the Analyzer refresh automatically now then? Maybe this is less desirable than a refresh button since you might want to take a sample and GTFO.

It does refresh automatically. The old version also did. I don't know if the refresh button was a holdover from a time when it didn't, or if it was intended as a feature where you could keep the last reading, but I'm not sure I can really see the use case for that.

Regarding the window size, I agree that it looks less cool. I will look into getting it to resize automatically to deal with the variation in text length depending on the number of gases.
Should I look at the MeasureOverride method for this?

Look at the RectClipContent in the SS14Window

Regarding the windows size, I haven't found a way to make the window change its size according to its content. RectClipContent doesn't seem to have any effect when changed on the window itself, but as I understand it it only affects whether a control visual clips its content to its own rectangle, not its resizing behavior.

Ping for re-open.


Improve linter with vectorMatching analyzing

this PR finalized the linter. Hopefully, it should cover every possible error in promQL !

:+1: Amazing, thanks!


Remove some logging

The integration is logging massively. Remove some logging.
Divide messages into info, warning, error, debug!

Implemented in 1.3


transparency without borders :)

Is there a way to show switch wireframe off?
Or show it only for the edge voxels and combine the volumes with the same (transparent) color?

First is possible, if I understand what you're asking correctly. The borders you see are just from the texture images; a texture could easily be fully transparent to remove the wireframe effect.
Combining the adjacent faces (like the Connected Textures Mod for Minecraft, or the feature of the same name in Optifine for Minecraft, I assume), not currently possible with voxel-glass. Each block is rendered independently. voxel-mesh is only for solid vowels (voxel-mesher can handle transparency).


[UWP] UWP renderer is missing mechanism to get current values from a custom input.

This is more of a feature we missed when writing the renderer and parser extensibility. Converting to Task and adding estimate.


No SoundCloud source support

The Fdroid repo description  is misleading, there is no support for SoundCloud

Hello,
There once was an experimental soundcloud support but it is gone now.
I cannot include Soundcloud because they have stopped accepting new apps since forever and i cannot get an API key.
I'm not the one publishing on F-Droid, and I don't control the description there. Sorry !


DICOM is parsed OK by pydicom, but EOF in this library

Hi, I got some issues when parsing DICOM file using this library. They are can be: unexpected EOF error or uint16 error... Do we have any further improvement?

Hi there! What version of the library are you using? Do you have an open source or freely available DICOM that can replicate the issue? Are there any other details you can give on what problems you're running into? Please don't post any real identified health information here, but any additional info you can provide would be helpful. You can also set the debug tag when building to let the library emit debug info that may be useful (-tags debug). Again please don't post any protected health information here, but if there's something interesting you see that you can describe that would be helpful.


Know what'd be cool? Chatheads.

This could be a separate app, but I don't have the time to create another one.


[RO-4125] Increase SSH Timeout

Tempest tests often take longer than 10 minutes to run and fail due to
ssh timeouts.  This increases the timeout to 20 minutes.
(cherry picked from commit 32ae4f5f9e364f981cc067aa7d05b00d2a732751)
Issue: RO-4125

Let's merge this as is so that we're only changing one thing at a time, once this merges I'll test removing the session timeout to see if that changes things. I understand that this will create a variance between master and pike-rc, but it will be short lived.


tests: add default_root_dir=tmpdir

What does this PR do?
some test were influenced by previous runs...
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Not sure why MLFlow fails in CI while locally pass
        file_path = os.path.join(root, file_name)
        if not exists(file_path):
>           raise MissingConfigException("Yaml file '%s' does not exist." % file_path)
E           mlflow.exceptions.MissingConfigException: Yaml file '/tmp/pytest-of-runner/pytest-0/test_loggers_fit_test_MLFlowLo0/default/a7c4907de41943a7b958534289a872a2/meta.yaml' does not exist.

same here #2391 what's going on? I've been trying to debug this for literally almost the whole day.
I think it is a mismatch between the logger dir (weights_save_path) and the checkpoint dir .

@awaelchli so  yo say that weights_save_path and default_root_dir does not like each other?

yes, the bug is somewhere burried in the many if else statements that sets these two variables. There seems to be a mismatch betwen these two paths.
Basically some loggers (e.g. mlflow) save a config file like meta.yaml in the log dir and this seems to be in the wrong folder.
Then when you try to resume from checkpoint  (e.g. in trainer.test) it will not find that file.

yes, the bug is somewhere burried in the many if else statements that sets these two variables. There seems to be a mismatch betwen these two paths.

at first I thought just to skip it and use the passing test maybe it is a chance to find the bug and fix it...

@Borda #2502 got merged, this is now unblocked

@awaelchli thx for fixing missed changes after rebase :]


Collision

Is there any documentation that illustrates how to add collision? Working on a Metaverse solution which requires that Avatars walk throughout the building without falling through.

Collisions are not implemented in the plugin but with some work you can extend it with that capability.
You can also add "invisible" collision meshes, which will be very performant.


License consistency across project

[x]  replace verbose license with shorter form
pointing to top level license file
[x] add license to missing files

CI_BRANCHES:NEURON_BRANCH=master,
Merge after #463

retest this please

retest this please


MSID: 86820 Version: 2 DOI: 10.1101/2<IP_ADDRESS>3018

MSID: 86820
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>3018
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=86820
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>3018

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/86820v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>3018
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/86820/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

@acollings should we have both evidence terms here?

Hi @JGilbert-eLife yes, okay in this case: though note that I've edited to add a full stop to the end :-)


Please add NEI intergration of Alloy Recipes ASAP

No one can craft any alloy without reading code.

I am working on it :)
I have little time and am going into JEI pretty much blind!

NEI is dead; it actually uses JEI to display recipes.  Regardless the author has made it clear in his videos and posts that JEI recipe integration isn't in.  IMHO this is a lower priority than core functionality.

How are things going with this? Any news on possible completion dates?

It would be great if you could somehow either integrate this into one of the recipe systems or document the processes on your wiki. I just spent half an hour trying to figure out how to make graphite dust - much less any of the fuels... My understanding is that you've done some of this with videos, but that's not a usable format for me, and for documentation of this sort it's very cumbersome to use in any event. Thanks.

I was just wondering if Tough Alloy was broken as I couldn't find a recipe and had to Google it (found http://nuclearcraft.wikia.com/wiki/Tough_Alloy). I've no idea how to code Minecraft mods, but I did find https://github.com/mezz/JustEnoughItems/wiki if that will help you. :)

Hi there - thank you all so much for your patience. JEI and MineTeaker support is finished and will feature in the next version of the mod. I am adding a few more machines and once that's done, I will release it :)

JEI support has been added for version 2.4, available to download from Curse. Enjoy :)


Re-work grid layout to prevent vertical-height restrictions

The current layout for all views uses a 2x4 grid. The height of the main content is as tall as the cards (the cards fill up the vertical viewport at minimum),

which causes problems when we need the main content to be taller than the cards and/or viewport.
This is what causes bugs like #264
Let's switch to flex columns.
This complicates the mobile/desktop breakpoint logic, but I can't find a better solution using grid.

fixed in #267


Added metadata on product and page

Added metadata on product page
Meta keywords are only added to page id these are not empty (Seo recomendation)

@mmoreram Indentation fixed


Support modifiers on /

A /<name> uri does not support modifiers, so multiple claims within a channel can't be handled currently if the uri includes the channel name. These claims can only be accessed with uris like lbry://<name>#<claim_id>.
Another option would be to support full <name>[modifier] parsing down a / separated path instead of only extending it one level. Certificate claims can themselves be signed, so this would allow uris like lbry://@root/@level1/somethingcool and lbry://@root_channel#<root_claim_id>/@sub_channel#<sub_claim_id>/content#<content_claim_id>.

three years, no activity

three years, no activity


MacOS Running issue

Describe the bug
I installed FAST on MacOS12.2.1 (21D62) but I can’t run this https://fast.eriksmistad.no/scan_convert_iq_data_8py-example.html example
Please do not use issues for questions. Use the gitter chat for questions and discussions: https://gitter.im/smistad/FAST
To Reproduce

install homebrew
/bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/HEAD/install.sh)"
install required package
brew install openslide libomp
In Anaconda
pip install pyfast
In pycharm run the code provided in this example
import fast
import numpy as np
import matplotlib.pyplot as plt

Create some fake IQ beamspace data
iq_data = fast.Image.createFromArray(np.random.normal(size=(512,512,2)).astype(np.float32))
Add some geometric data used for scan conversion
iq_data.setFrameData('isPolar', 'true')
iq_data.setFrameData('startRadius', '0')
iq_data.setFrameData('stopRadius', '0.12')
iq_data.setFrameData('startTheta', '-0.785398') # Start and stop angle in radians
iq_data.setFrameData('stopTheta', '0.785398')
iq_data.setFrameData('depthSpacing', '0.000235')
iq_data.setFrameData('azimuthSpacing', '0.003074')
Create processing & visualization chain
envelope = fast.EnvelopeAndLogCompressor.create()
.connect(iq_data)
scan_convert = fast.ScanConverter.create()
.connect(envelope)
renderer = fast.ImageRenderer.create()
.connect(scan_convert)
fast.SimpleWindow2D.create()
.connect(renderer)
.run()
Visualize output of scan converter with matplotlib instead of FAST:
plt.imshow(np.asarray(scan_convert.runAndGetOutputData())[..., 0], cmap='gray')
plt.show()
System:

MacBooks Pro 2019. MacOS12.2.1 (21D62)
Radeon Pro Vega 20 4 GB
Intel UHD Graphics 630 1536 MB

Expected behavior
fake IQ data will show up
Screenshots
Error message from pycharm
qt.qpa.window: <QNSWindow: 0x7fac7d9f6bb0; contentView=<QNSView: 0x7fac7d9f68f0; QCocoaWindow(0x6000031a5810, window=QWidgetWindow(0x6000022ed380, name="QGLWidgetClassWindow"))>> has active key-value observers (KVO)! These will stop working now that the window is recreated, and will result in exceptions when the observers are removed. Break in QCocoaWindow::recreateWindowIfNeeded to debug.

The example runs fine on my Mac running macOS 10.15 with system python 3.10.
Can you try running the example without anaconda, just using a clean python environment?

@smistad Thanks for getting to me.
I gave it a try running with using anaconda. but still not working
qt.qpa.window: <QNSWindow: 0x7fa2b8fdb2b0; contentView=<QNSView: 0x7fa2b8fdaca0; QCocoaWindow(0x600001eecfd0, window=QWidgetWindow(0x600000de9f20, name="QGLWidgetClassWindow"))>> has active key-value observers (KVO)! These will stop working now that the window is recreated, and will result in exceptions when the observers are removed. Break in QCocoaWindow::recreateWindowIfNeeded to debug.
WARNING [0x1129a3600] Kernel binary Algorithms/Ultrasound/EnvelopeAndLogCompressor.cl is out of date. Compiling...
WARNING [0x1129a3600] Kernel binary /ImageMinMax.cl is out of date. Compiling...
WARNING [0x1129a3600] Kernel binary Algorithms/Ultrasound/ScanConverter.cl is out of date. Compiling...

Not able to reproduce, closing.

@smistad thanks for looking into it.


telemetry: deflake access log tests

Example failure:
https://prow.istio.io/view/gs/istio-prow/pr-logs/directory/integ-telemetry_istio/1652008195079540736
It looks like the XDS push is just causing too much load and it takes more than 10s to process. We see during XDS push /stats/prometheus also times out.
Please provide a description of this PR:

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
04271ad63950bf4f9001e486c75dac32ed7667c3
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest https://github.com/istio/istio/issues/44640

@howardjohn: The /retest command does not accept any targets.
The following commands are available to trigger required jobs:

/test analyze-tests
/test gencheck
/test integ-ambient
/test integ-basic-arm64
/test integ-cni
/test integ-distroless
/test integ-ds
/test integ-helm
/test integ-ipv6
/test integ-operator-controller
/test integ-pilot
/test integ-pilot-istiodremote
/test integ-pilot-istiodremote-mc
/test integ-pilot-multicluster
/test integ-security
/test integ-security-istiodremote
/test integ-security-multicluster
/test integ-telemetry
/test integ-telemetry-istiodremote
/test integ-telemetry-mc
/test lint
/test release-notes
/test release-test
/test unit-tests
/test unit-tests-arm64

The following commands are available to trigger optional jobs:

/test benchmark
/test integ-assertion

Use /test all to run the following jobs that were automatically triggered:

analyze-tests_istio
gencheck_istio
integ-ambient_istio
integ-basic-arm64_istio
integ-cni_istio
integ-distroless_istio
integ-ds_istio
integ-helm_istio
integ-ipv6_istio
integ-operator-controller_istio
integ-pilot-istiodremote-mc_istio
integ-pilot-istiodremote_istio
integ-pilot-multicluster_istio
integ-pilot_istio
integ-security-istiodremote_istio
integ-security-multicluster_istio
integ-security_istio
integ-telemetry-istiodremote_istio
integ-telemetry-mc_istio
integ-telemetry_istio
lint_istio
release-notes_istio
release-test_istio
unit-tests-arm64_istio
unit-tests_istio

In response to this:

/retest https://github.com/istio/istio/issues/44640

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/retest

In response to a cherrypick label: new pull request created: #44645


BUG: Chatbot Design Issues on Smaller Devices and slow response

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The chatbot has significant design issues on smaller devices. The interface is not optimized for mobile screens, causing the chatbot to go out of the screen, making it difficult to use. Additionally, the chatbot frequently fails to generate responses, leaving users without necessary interactions or information. The CSS implementation needs to be reviewed to ensure proper alignment and consistent styling across devices. A thorough design for mobile responsiveness and an investigation into the backend response mechanism are essential to resolve these problems and enhance user experience.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@anuragverma108 Sir this issue has not been fixed but someone has closed this issue. What to do sir??

@anuragverma108 Sir this issue has not been fixed but someone has closed this issue. What to do sir??

hey 👋 !
i am the mentor! and sorry to not give you enough justification about closing the issue but chatbot is working fine on every small screen! you are using dev tools in the wrong way buddy!


Belgian prices are not fetched because of different resolutions in api response

So since the api changes of 10/04 the response for the Belgian zone contains PT15M (15 min) resolution data mixed with PT60M (60 min) data. The current parsing logic only parses PT60M data because that was al that was needed until now.
It is currently unclear to me what the solution here should be. We could change the parsing logic to also parse PT15M data. The issue with that is that the German zone also returns PT15M data we explicitly don't want to parse. We could program in some logic to only parse PT15M when the Belgian data is requested but that feels like a 'hack'.

We can either skip the 2nd, 3rd, 4th entry to start with.
In a second we can average the 2nd, 3rd, 4th prices to get back to hour prices
Changing the complete integration to support 15min intervals is quite a large change

dear all, entsoe-py maintainer dropping in. Sorry to hear you had to build your own implementation due to pandas issue.
please note that the SDAC is currently on hourly basis and the 15 min data from Belgium is a bug. For my own datalake I have added a hardcoded exception that resamples the belgium prices.
for the future it is important to note that the SDAC will go to 15 min resolution somewhere Q1 or Q2 2025. This means that you will need the 15 min parser eventually since ALL countries will swap their day ahead prices to that resolution at the same time.

this might fix this (for now, not ideal as I don't really care about real 15m resolutions
https://github.com/JaccoR/hass-entso-e/pull/196

Proposing temporary workaround with the change below to api_client.py for mixed resolution in API response :
def query_day_ahead_prices(
    self, country_code: Union[Area, str], start: datetime, end: datetime
) -> str:
    """
    Parameters
    ----------
    country_code : Area|str
    start : datetime
    end : datetime

    Returns
    -------
    str
    """
    area = Area[country_code.upper()]
    params = {
        "documentType": "A44",
        "in_Domain": area.code,
        "out_Domain": area.code,
    }
    response = self._base_request(params=params, start=start, end=end)

    if response.status_code == 200:
        try:
            root = self._remove_namespace(ET.fromstring(response.content))
            _LOGGER.debug(f"content: {root}")
            series = {}

            # Extract TimeSeries data
            for timeseries in root.findall(".//TimeSeries"):
                for period in timeseries.findall(".//Period"):
                    resolution = period.find(".//resolution").text

                    _LOGGER.debug(f"Resolution {resolution}")
                    if (resolution != "PT60M" and resolution != "PT30M" and resolution != "PT15M"):
                        continue

                    response_start = period.find(".//timeInterval/start").text
                    start_time = (
                        datetime.strptime(response_start, "%Y-%m-%dT%H:%MZ")
                        .replace(tzinfo=pytz.UTC)
                        .astimezone()
                    )

                    response_end = period.find(".//timeInterval/end").text
                    end_time = (
                        datetime.strptime(response_end, "%Y-%m-%dT%H:%MZ")
                        .replace(tzinfo=pytz.UTC)
                        .astimezone()
                    )

                    _LOGGER.debug(f"Period found is from {start_time} till {end_time}")

                    for point in period.findall(".//Point"):
                        position = point.find(".//position").text
                        price = point.find(".//price.amount").text
                        if resolution == "PT60M":
                            hour = int(position) - 1
                        elif resolution == "PT30M":
                            hour = ((int(position) - 1) / 2)
                        elif resolution == "PT15M":
                            hour = ((int(position) - 1) / 4)
                        else:
                            continue

                        try:
                            series[start_time + timedelta(hours=hour)] = float(price)
                        except:
                            continue

                    # Now fill in any missing hours
                    current_time = start_time
                    last_price = series[current_time]

                    while current_time < end_time:  # upto excluding! the endtime
                        if current_time in series:
                            last_price = series[current_time]  # Update to the current price
                        else:
                            _LOGGER.debug(f"Extending the price {last_price} of the previous hour to {current_time}")
                            series[current_time] = last_price  # Fill with the last known price
                        current_time += timedelta(hours=1)

            return dict(sorted(series.items()))

        except Exception as exc:
            _LOGGER.debug(f"Failed to parse response content:{response.content}")
            raise exc
    else:
        print(f"Failed to retrieve data: {response.status_code}")
        return None

If it helps I was thinking about making a small library only pulling in prices with no pandas dependency for projects such as these. Would that help?
Get BlueMail for Android
On 7 Oct 2024, 20:45, at 20:45, micvdh @.***> wrote:

Proposing temporary workaround with the change below to api_client.py
for mixed resolution in API response :
def query_day_ahead_prices(
self, country_code: Union[Area, str], start: datetime, end: datetime
) -> str:
"""
Parameters
----------
country_code : Area|str
start : datetime
end : datetime
   Returns
   -------
   str
   """
   area = Area[country_code.upper()]
   params = {
       "documentType": "A44",
       "in_Domain": area.code,
       "out_Domain": area.code,
   }
response = self._base_request(params=params, start=start, end=end)

   if response.status_code == 200:
       try:
    root = self._remove_namespace(ET.fromstring(response.content))
           _LOGGER.debug(f"content: {root}")
           series = {}

           # Extract TimeSeries data
           for timeseries in root.findall(".//TimeSeries"):
               for period in timeseries.findall(".//Period"):
                   resolution = period.find(".//resolution").text

                   _LOGGER.debug(f"Resolution {resolution}")

if (resolution != "PT60M" and resolution != "PT30M" and resolution !=
"PT15M"):
continue
        response_start = period.find(".//timeInterval/start").text
                   start_time = (
              datetime.strptime(response_start, "%Y-%m-%dT%H:%MZ")
                       .replace(tzinfo=pytz.UTC)
                       .astimezone()
                   )

            response_end = period.find(".//timeInterval/end").text
                   end_time = (
                datetime.strptime(response_end, "%Y-%m-%dT%H:%MZ")
                       .replace(tzinfo=pytz.UTC)
                       .astimezone()
                   )

_LOGGER.debug(f"Period found is from {start_time} till {end_time}")
                   for point in period.findall(".//Point"):
                       position = point.find(".//position").text
                       price = point.find(".//price.amount").text
                       if resolution == "PT60M":
                           hour = int(position) - 1
                       elif resolution == "PT30M":
                           hour = ((int(position) - 1) / 2)
                       elif resolution == "PT15M":
                           hour = ((int(position) - 1) / 4)
                       else:
                           continue

                       try:
         series[start_time + timedelta(hours=hour)] = float(price)
                       except:
                           continue

                   # Now fill in any missing hours
                   current_time = start_time
                   last_price = series[current_time]

     while current_time < end_time:  # upto excluding! the endtime
                       if current_time in series:
  last_price = series[current_time]  # Update to the current price
                       else:

_LOGGER.debug(f"Extending the price {last_price} of the previous hour
to {current_time}")
series[current_time] = last_price  # Fill with the last known price
current_time += timedelta(hours=1)
           return dict(sorted(series.items()))

       except Exception as exc:

_LOGGER.debug(f"Failed to parse response content:{response.content}")
raise exc
else:
print(f"Failed to retrieve data: {response.status_code}")
return None
--
Reply to this email directly or view it on GitHub:
https://github.com/JaccoR/hass-entso-e/issues/195#issuecomment-2397637466
You are receiving this because you commented.
Message ID: @.***>

dear all, entsoe-py maintainer dropping in. Sorry to hear you had to build your own implementation due to pandas issue. please note that the SDAC is currently on hourly basis and the 15 min data from Belgium is a bug. For my own datalake I have added a hardcoded exception that resamples the belgium prices.
for the future it is important to note that the SDAC will go to 15 min resolution somewhere Q1 or Q2 2025. This means that you will need the 15 min parser eventually since ALL countries will swap their day ahead prices to that resolution at the same time.

I was wondering if you know on how the SDAC and EXAA will be distinguishable from each other when the SDAC changes to 15 min data.

If it helps I was thinking about making a small library only pulling in prices with no pandas dependency for projects such as these. Would that help?

I think that a lightweight low dependency api library could be useful. It would definitely be easier to wait for a package update when entso-e breaks the api again than fixing and maintaining it in this project. In a perfect world that library could then be the core of a library that provides pandas objects. That would eliminate maintenance of redundant functionality. I just don't know how feasible that all is.
Belgian data for today is in the correct format. So maybe it is fixed. If prices of tomorrow break it again I will look into merging the patch @wonko

Belgian data for today is in the correct format. So maybe it is fixed. If prices of tomorrow break it again I will look into merging the patch @wonko

Keep in mind that the response sent was valid according to the specs as set out in the XML Schema guide. The patch allows to accept these valid responses, doesn't break anything (and even adds tests on the XML, which can be easily extended with other cases in the future). Until there's a uniform solution through some external (or internal) package which deals with everything, it only adds robustness.
But, your call in the end.

Yes I agree and i like the tests. And have the intention on merging the PR. But this just takes the pressure off releasing a rushed version.

PR #202 is now tested and should resolve this issue


Checkboxes not checked correctly.

For the checkboxes, if I pass true or a string value or 1, the checkbox is not checked rather if I pass false or an empty string or 0 it gets checked. So I believe the functionality is working in the reverse order contradictory to the documentation provided. Can you please verify this.

I can't get them to work either. I have tried all sorts of combinations. Can you please provide a working code example so we can use that as a reference? It may be of not that my array has checkboxes, but also text areas that I need to populate and those are being passed along without issue.

Same here. Doesn't matter what I use, the fields simply won't populate checkboxes. If the developer can give us the sample pdf and code they used to verify the checkbox function as working, I'm sure I could pull that apart to understand the proper syntax.
Can we get a little bit more info on the checkboxes, please?

Well, I also have the same problem, but not exactly sure why, I have a PDF with forms and checkboxes, I can fill them using the library (I got it from someone in my company, but he is not here anymore), but once I edit the PDF through Adobe Acrobat DC or a online PDF editor, the checkbox cannot be filled anymore.
So I'm wondering if the PDF needs to be "encoded" in a specific format (or at least the format on checkbox is changing after editing) and that's why it's not working.
By enabling verbose mode in the library
$pdf->verbose = true;
$pdf->verbose_level = 3;
$pdf->useCheckboxParser = true;

With the old checkbox I got this output:
Change checkbox of the field chpb at line 9387 to value [Oui]
Change checkbox of the field chlu at line 9458 to value [Oui]
Change checkbox of the field chim at line 9426 to value [0]
Change checkbox of the field chgl at line 9490 to value [0]

And with the edited one through Adobe Acrobat DC and converted back with pdftk old.pdf output new.pdf it's not recognizing the checkboxes anymore:
Change checkbox value aborted, the field chpb has no checkbox definition.
Change checkbox value aborted, the field chlu has no checkbox definition.
Change checkbox value aborted, the field chim has no checkbox definition.
Change checkbox value aborted, the field chgl has no checkbox definition.

So I'm guessing there is something wrong with the format of checkboxes

Running pdftk will break the "checkbox definition" somehow. I have set tooltip and name correctly in Acrobat CC, turned off fast web view an compression, set it all the way down to Acrobat 5.0+ compatible, and on and on...
The problem here is that this developer got checkboxes working, but didn't give anyone specs on the PDF. If they developer could simply post a working demo that we can download and pull apart, then the rest is not hard to figure out. But there seems to be some sort of specific setup to the document that must be in place for the "checkbox definition" to be present.

Ok so I managed to get it working. (idk why tho)
So I started to look into the PR and the fork, I found this one: https://github.com/cedson61/fpdm
Which seems to work at first glance, but I got weird behavior after, and one of my checkboxes doesn't get recognize anymore. So I rolled back to this repo
So for whatever reason, I edited the PDF with Acrobat DC, I deleted the checkboxes, save the PDF, and I re-added them with the same names and save.
I made sure that I had nothing into the tooltip like said in the repo above:
Just make sure you don't have any alt text on your fields like so (or your checkboxes will come out blank)
And now it's working, the only thing that I noticed is that to have the checkbox correctly checked I needed to inverse the boolean
True or "anything" = checkbox unchecked
False or "" = checkbox checked
To resolve that (idk how I managed to find that first try)
I changed the export Value in the checkbox properties to something else than "Yes", I put "Oui" (french) and now True = checked

I can't share this PDF, I will try to share a working one today.
I'm happy to see someone active on this repo, the odds that you had the same problem as me today are quite low 😅

i started working with fpdm today and ran into the same issue. Is there someone who solved this problem without recreating the checkboxes?
I can see, that if i try to change the value of the check box, that it is slightly changed (a inner border inside the checkbox)
And i found out, that the value fpdm is trying to put into the checkbox is "1" instead of true.
Any help would be great :)
Or has someone an example where it worked?


Fix issues with list & map resource types for RestConf driver

Fix issues with list & map resource types for RestConf-driver

Checked in code with branch release-2.5.0 with PR #110


iOS:: Fails during auth process if TikTok App is not installed

Hey, thank you for this library!
I'm working on a Tiktok integration through your library and the authentication process works perfectly when the TikTok app is installed on the iOS device BUT nothing happens when the TikTok app is not installed on an iOS device:
`-canOpenURL: failed for URL: "tiktokopensdk://" - error: "The operation couldn’t be completed. (OSStatus error -10814.)"
[Presentation] Attempt to present <UINavigationController: 0x10c832800> on <Tiktok: 0x106c2e800> (from <Tiktok: 0x106c2e800>) whose view is not in the window hierarchy.
`
Can you help me?
Best Regards

@danielavelasquezc
Hi, can you try something like that?
import DeviceInfo from 'react-native-device-info';

const isTikTokInstalled = async () => {
  const installedApps = await DeviceInfo.getInstalledApps();
  return installedApps.some(app => app.bundleId === 'com.zhiliaoapp.musically');
}

// Usage
const tiktokInstalled = await isTikTokInstalled();
console.log(tiktokInstalled); // true or false

The problem is launching the tiktok website. It doesn't work. I can detect if the app is installed or not but if it's not installed nothing happen.


Fix usage error in README

[clabot:check]

should be
substrate --chain ~/chainspec.json build-spec --raw > ~/mychain.json

I resubmitted according to your advice.


Double and Float are not correctly converted

Hi,
first of all thanks for your work. Works great so far.
Found only one issue:
If I have got a double or float property and enter a value like 0,5 it gets converted to 5.
I have the same problem in the samples-Basic project.
Maybe it has something to do with the local used language? In this case it is DE.
Are you able to reproduce the issue in the sample project?
Thanks in advanced!

It really should have something to do with localization. But the exception in your screenshot looks like it was thrown by Avalonia. The source code of CellEditor about numerical values is at:
https://github.com/bodong1987/Avalonia.PropertyGrid/blob/main/Avalonia.PropertyGrid/Controls/Factories/Builtins/NumericCellEditFactory.cs
In my local language environment, Avalonia's control returns 5 when I input 0,5. Theoretically, Avalonia should use the system's localization configuration by default to initialize the NumberFormat property of the control.
So you can try setting the NumberFormat property of NumericUpDown directly. You can also provide your own NumericCellEditFactory through expansion, change its ImportPriority to a larger value, and then provide it to the PropertyGrid. Extension method: https://github.com/bodong1987/Avalonia.PropertyGrid/blob/main/Samples/Avalonia.PropertyGrid.Samples/Views/TestExtendPropertyGrid.cs

Thanks for your response.
I checked the code and it looks like it's not the localization (or not only).
The error occures during the transformation from float input to double as you can see in the first red marked part:

The double input conversion works in the first part but crashes during string conversion.
So I added the culture info also in the tryparse method which looks like it is working now.


Allow reading in metadata partially

Reading in all metadata of a data source may cost a lot of time, because in the background datasets need to be accessed. This can significantly slow down the retrieval of meta information if it is requested for many data sources. However, the datasets only need to be accessed for a relatively small portion of the information.
xcube-cci can be made a lot faster when this information is read only when needed and requested.

See https://github.com/CCI-Tools/cate-webui/issues/120


Buffer underflow with path resolution on non-dfs shares

Receiving a buffer_underflow in smbj when performing a path resolution on non-DFS shares.
This occurs within a response to sendDfsReferralRequest, then SMB2IoctlResponse.readMessage.
Somehow the buffer gets smaller when smbj is initialized with SmbConfig.withBufferSize(510241024).
The problem goes away when using smaller buffer sizes (default 1 MB; 4 MB works as well).

Version: 0.1.0


preserve comments with AST

While testing #77 i noticed that adding an agent failed because the # Agent definitions tag had disappeared after the file was modified with AST.
This is a simple fix that records comments in the original and inserts them while arranging the file via AST

I do still think modifying the output file by addressing line numbers and only changing what is new is a more sustainable solution, but this gets us back up and running in the meantime.

totally agreed. in my mind i wanted a hot fix to address #74 which has been reported too many times


[Errno 13] Permission denied for containerised functions

Description
After copying the container example, the docker image builds and creates the lambda function fine. But, when I try to invoke the function I get the following error...
{
  "errorMessage": "[Errno 13] Permission denied: '/var/task/app.py'",
  "errorType": "PermissionError",
  "stackTrace": [
    "  File \"/var/lang/lib/python3.8/imp.py\", line 300, in find_module\n    with open(file_path, 'rb') as file:\n"
  ]
}

My .tf file looks like this (as per the example)...
module "lambda_function_from_container_image" {
  source = "terraform-aws-modules/lambda/aws"

  function_name = "my-lambda2"
  description   = "My awesome lambda function from container image"

  create_package = false

  ##################
  # Container Image
  ##################
  image_uri    = module.docker_image.image_uri
  package_type = "Image"
}

module "docker_image" {
  source = "terraform-aws-modules/lambda/aws//modules/docker-build"

  create_ecr_repo = true
  ecr_repo        = "my-lambda2-repo"
  image_tag       = ""
  source_path     = "../lambda"
}

Versions

Terraform: 1.0.8

Provider(s): aws

Module: docker-build

Reproduction

I've tried using a few different environments with the same results. But, my main workflow is the following...
GitHub actions is triggered to execute terraform (remote state, local operations). The above container build/deploy workflow is executed. After this is complete, I log onto the AWS console to test the lambda function (the current handler just returns "Hello World!" no matter the input). After running a test case with dummy inputs, the above error is displayed.
Expected behavior

The function should not be hitting a permissions error (returning "Hello World!" successfully")
Actual behavior

The function is returning an errno 13

Solution/workaround found...
File access permissions need to be explicitly set in the Dockerfile as per this AWS doc. I assume this bug occurs due to the way this module builds the docker image as it doesn't happen when when I build the image using default commands. An example Dockerfile for a python lambda could be as follows...
FROM public.ecr.aws/lambda/python:3.8

# Copy function code
COPY app.py ${LAMBDA_TASK_ROOT}

# Install the function's dependencies using file requirements.txt
# from your project folder.

COPY requirements.txt  .
RUN  pip3 install -r requirements.txt --target "${LAMBDA_TASK_ROOT}"

# Explicitly set file permissions
RUN chmod 644 $(find . -type f)
RUN chmod 755 $(find . -type d)

# Set the CMD to your handler (could also be done as a parameter override outside of the Dockerfile)
CMD [ "app.handler" ]


Two tests in language/factory_redirection_test.dart tests fail on dart2js

The tests
  factory_redirection_test/01
  factory_redirection_test/07
fails on dart2js

Fixed by https://codereview.chromium.org/1323573002/


value wont change

I want to change the value while pass different number to the value prop,
but  the value of battery wont change if the animated prop is added

I want to change the value while pass different number to the value prop, but the value of battery wont change if the animated prop is added

Thanks for raising it. Will look into this at priority

Fixed here, fix will be available on version 1.0.4

Thx for fixing it !


Fixed 501 notebooks

📝 Description
Changed metadata parameters. Also, I included more outputs for visualization purposes.
🛠 Fixes #1489
✨ Changes
Select what type of change your PR is:

[x] 🐞 Bug fix (non-breaking change which fixes an issue)
[ ] 🔨 Refactor (non-breaking change which refactors the code base)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] 📚 Documentation update

@samet-akcay ready :)

Thanks @adrianboguszewski. If that's alright, I'll choose the target branch v1?


Compile a single file

Can we provide instructions on howto compile a single file instead of the entire book?

If possible, I think this would be useful as the whole build takes a long time -- do you want to make pull request for these instructions?


Fix network generation with bonding

Can you explain reason of this change?

Hi @johnnykeats
bonded interfaces does not have IP specified in the inventory, but an ifcfg file should be generated
Actually, this inventory example is provided in the nic role README :
network_interfaces:
  eth0:
    type: bond-slave
    master: bond0
  eth1:
    type: bond-slave
    master: bond0
  bond0:
    type: bond
    vlan: false
    bond_options: "mode=4 xmit_hash_policy=layer3+4 miimon=100 lacp_rate=1"

but this example do not generate any ifcfg files because of this condition in the tasks :
when: (network_interfaces[item]['ip4'] is defined and network_interfaces[item]['ip4'] is not none)

Yes, I confirm this.


Converting Binance Public Data(or other external data) to hftbacktest's dataformat

I want to backtest in a wide range of time series, so I need to use Binance's open public data(https://github.com/binance/binance-public-data). However, I am not sure about the exact data structure in hftbacktest. Can you provide guidelines or an example code for importing external data?

Could you check https://github.com/nkaz001/hftbacktest/wiki/Data and, if you have any further questions, provide additional details when asking again?


remove java8 from workflow runs, http4s to stable version in examples…

…, make zio dependency provided

I think there's also a ref to java8 branch in the Renovate workflow ;)


Add 'LABEL' to workflow

enarx/bot#160
Adding this LABEL property to the workflow file should allow for distinction between badges on the enarx/bot README.

Thanks for pointing this out @haraldh
Should be fixed up now


Investigate UI lag after replay

Androish reported a UI lag at the end of a replay

Hey.
To clarify. This has to do with live replay. it happens when you are at close to realtime as you allowed to be, and the game you are observing ends. the replay will slow down/freeze for a little(sometimes forever) then usually go on at your max simspeed(or whatever setting you have it at) from there on. I think it must be something with the live replay server not the client?

Thanks for the clarification. I don't know the replay server internals, but there's a good chance it's not very efficient :-)

All players have to leave the game for the live replay to unfreeze, atleast thats what i managed to find out, downlords client most likely isn't the problem here, its probably the server.

Thanks for investigating, I'm closing this then :-)


For prereview Schedule

Hi, this looks like a great start! I did some testing and viewed things on my end... here are some comments
Needs fixing

alignment of the November 4th text. If this is centered then we should center the other text as well to make it flow a bit better.

Mobile compatability:

it doesn't seem this code was testing with the mobile screen sizing. We should be using responsive design to ensure that this schedule looks okay on all screen sizes. Before creating any PR, you should be doing testing on multiple screen sizes.

Screen Testing Instructions (if you aren't sure)

right click on the page and then click Inspect
In the top left of the panel that pops up there is a small icon that looks like this:

Adjust the sizing using that tool to see what it looks like on different devices
In general, look into how to make your component responsive and continuously test how it looks on all screens

Make sure this is reviewed or Final Iteration 2 if it is there.
https://github.com/hack-rpi/HackRPI-Website2023/pull/48/commits/2269e362ebe7cf0bbd1c94fc20c3fb2475b1d4d7

Review https://github.com/hack-rpi/HackRPI-Website2023/pull/48/commits/1d461257cf646519896bc78205e9b751d92f7c61

@maryellenmarino
Please also make sure @everyone in rcos is pinged and for them to update their branch from main immediately after release

Should fix the critical security alert as well if people immediately updates from main after branch is merged. Please inform everyone to do so


Clickable links on blog on mobile?

When I'm viewing the "News & Blog" page on my phone, it seems like the links to blog posts are not clickable. Has anyone else noticed this? Perhaps just something strange about using Chrome on an iPhone?

I'm experiencing the same issue, Chrome on Android as well as a desktop browser's inspect element tool.
I'm leaving this here in case anyone on the website subcommittee wants to try to figure  this out. Otherwise please feel free to assign to me.

It seems like this is working now, so I'm closing the issue.


ARROW-17649: [Python] Remove remaining deprecated APIs from <= 1.0.0

@github-actions crossbow submit -g python


fix user options

First of all thanks for your package: it's easy to use and clear.
This PR is a fix I made when I realized nuxt options in nuxt.config.ts are not really taken into account when using the directive.
This is due to the fact that they where not included in the sanitize-html method call.
Furthermore, the option merge strategy were not doing what expected IMHO since if in the nuxt.config.ts I declare for instance:
sanitize: {
    allowedTags: ['a'],
    allowedAttributes: {
      a: ['href'],
    },
  }

merge result will just replace the first element of default allowedTags with a while it would be more useful to totally replace allowedTags entry with the user-defined one.
Hope this is welcome.
Lorenzo

@lsimone  thanks for your first PR, i will take a look into it.


IE11 focus highlight on every other element

Issue Summary
When using tab to navigate the admin in IE11, every other element is not highlighted.
On tab the body class focus-outline-on changes to focus-outline-off.
Steps to Reproduce

Open the admin in IE11
Open the developer console to inspect the body class
Click in the page and press tab a couple of times
Every other item has the focus outline

Technical details

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: YES
Current master via https://github.com/wagtail/gitpod-wagtail-develop
Browser stack IE11

Closing as IE11 support is being dropped in 2.15.


[DCOS-58017] Mark error-prone packages with hasKnownIssues: true

What changes were proposed in this pull request?
PR updates all the error-prone packages with the new field "hasKnownIssues".
This is to mark all packages that do not deploy successfully with default values.
Resolves Mark error-prone packages with hasKnownIssues: true

Adding @takirala for some more visibility on this

@samvantran  @takirala  Hi Guys, looks last updated field change merged to master with PR https://github.com/mesosphere/universe/pull/2414 , could you confirm if i am good resume with this PR ?
I will raise new PR by introducing new field "hasKnownIssues: true" in package.json with v4

@rathohit311356 yes please go ahead with this PR.
Also, make sure you do not change anything under repo/meta directory.

NEW PR raised : https://github.com/mesosphere/universe/pull/2418
Closing this one as there is change required throughout all the packages which does not includes any of the changes done in the this PR.


xformers installs but could not build wheels for it.

This is the error I receive:
File "H:\Program Files\Python311\Lib\site-packages\setuptools_distutils\util.py", line 139, in convert_path
raise ValueError("path '%s' cannot be absolute" % pathname)
ValueError: path '/__w/xformers/xformers/third_party/flash-attention/csrc/flash_attn/flash_api.cpp' cannot be absolute
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for xformers
Running setup.py clean for xformers
Failed to build xformers
ERROR: Could not build wheels for xformers, which is required to install pyproject.toml-based projects`
I was able to install xformers 0.0.20 and I was able to remove it and install xformers 0.0.22.
pip install xformer==0.0.22
I even installed from the source on pypi.org. They all give the same error making wheels. There is an absolute path somewhere that python hates.

Here is my current PIP List: H:\MUSIC\audiocraft>pip list
Package            Version

certifi            2023.7.22
charset-normalizer 3.3.0
filelock           3.12.4
idna               3.4
Jinja2             3.1.2
MarkupSafe         2.1.3
mpmath             1.3.0
mypy-extensions    1.0.0
networkx           3.1
ninja              <IP_ADDRESS>
numpy              1.26.0
Pillow             10.0.1
pip                23.2.1
pyre-extensions    0.0.29
requests           2.31.0
setuptools         65.5.0
sympy              1.12
torch              2.0.1+cu118
torchvision        0.15.2+cu118
typing_extensions  4.8.0
typing-inspect     0.9.0
urllib3            2.0.6
wheel              0.41.2
xformers           0.0.22

This is on Windows by the way. Python version 3.11

looks like there is an issue with latest xformers,
they did a commit few hours ago, https://github.com/facebookresearch/xformers/commit/16e4245e02f47883b6b43065a0e26dfb12e14e33 so it may be fixed already
if not, an older version (0.0.20) is suggested: https://github.com/facebookresearch/xformers/issues/886#issuecomment-1762922926

looks like there is an issue with latest xformers, they did a commit few hours ago, facebookresearch/xformers@16e4245 so it may be fixed already if not, an older version (0.0.20) is suggested: facebookresearch/xformers#886 (comment)

Thanks. I'll give it a try in a few minutes. I did try the older version. It installs too, but the wheels step still errors our. Lets hope they do have it fixed already.

Its fixed. Thanks!

@zeke-john what worked for me (M1) is installing it though a Conda env, you can also try installing it from source.

I'm having the same issue, and i'm in a conda env

hello Can you please help me install on M3 ? I am facing the same issue

Can we re-open this?

having same issue on an Intel Mac

same issue here intel mac

I'm having the same issue, and i'm in a conda env

I'm trying on conda env, is this problem fix on you side.
I the problem fix on your side please let me know how you have fix this issue?