HTTP 204 returned from /ping endpoint is unsuitable for Google Kubernetes Cloud healthcheck

Use case
I want to deploy an InfluxDB instance to the Google Kubernetes Cloud. In order to expose the application to the outer world, I need to create an Ingress.
Problem
The Ingress requires a health check. I can configure the endpoint of the health check but it needs to return a HTTP 200. The /ping endpoint currently returns an HTTP 204 which is unsuitable.
Sugggestion
Consider adding an endpoint that returns a HTTP 200 or make the status code returned from /ping configureable.

@cypressious I think this is a good suggestion, if it's not possible to configure the expected status code on that cloud platform.
In the meantime, you could use /metrics or /debug/vars, which will both return a 200.

Please do!

@cypressious according to the k8s docs found here: https://kubernetes.io/docs/tasks/configure-pod-container/configure-liveness-readiness-probes/#define-a-liveness-http-request

Any code greater than or equal to 200 and less than 400 indicates success. Any other code indicates failure.

Is this issue still relevant?


Carbon v3

Pushed a fix to maintain the int value returned from diffInMilliseconds. This return type changed in Carbon v3.
Although not a major issue, it is a potential change in functionality if the config were to contain a float value.

I would prefer to keep this in draft until v3 is actually released and we know for sure what all the breaking changes are.
Otherwise we merge this and they tag additional breaking changes in future beta release and we have to re-review.

@timacdonald Should only be bug fixes from now on normally.

Stable release out now.


pkg/sdk: combining all pkgs under pkg/sdk into a single pkg

Addresing #162. This PR combines the pkg/sdk/{action,handler,informer,query,types} into one package called sdk.
The new structure:
go doc "github.com/operator-framework/operator-sdk/pkg/sdk"
package sdk // import "github.com/operator-framework/operator-sdk/pkg/sdk"

func Create(object Object) (err error)
func Delete(object Object, opts ...DeleteOption) (err error)
func Get(into Object, opts ...GetOption) error
func Handle(handler Handler)
func List(namespace string, into Object, opts ...ListOption) error
func Run(ctx context.Context)
func Update(object Object) (err error)
func Watch(apiVersion, kind, namespace string, resyncPeriod int)
type Context struct{ ... }
type DeleteOp struct{ ... }
    func NewDeleteOp() *DeleteOp
type DeleteOption func(*DeleteOp)
    func WithDeleteOptions(metaDeleteOptions *metav1.DeleteOptions) DeleteOption
type Event struct{ ... }
type GetOp struct{ ... }
    func NewGetOp() *GetOp
type GetOption func(*GetOp)
    func WithGetOptions(metaGetOptions *metav1.GetOptions) GetOption
type Handler interface{ ... }
    var RegisteredHandler Handler
type Informer interface{ ... }
    func NewInformer(resourcePluralName, namespace string, resourceClient dynamic.ResourceInterface, ...) Informer
type ListOp struct{ ... }
    func NewListOp() *ListOp
type ListOption func(*ListOp)
    func WithListOptions(metaListOptions *metav1.ListOptions) ListOption
type Object runtime.Object

@hasbro17 @ericchiang we might want to think about how we can further improve on this design structure and break it up its more logical modules.

I didn't check your travis logs but if your test seems to be ok, check your spaces/tabs in the test string template, I had the same problem.

All travis tests pass now :+1: Needed a small change in pkg/generator/handler_tmpl.go

Testing this PR out manually with the user-guide example since we don't have an e2e test in place right now.

we might want to think about how we can further improve on this design structure and break it up its more logical modules.

+1
Yeah, definitely "Optimize for Godoc".
I think https://github.com/operator-framework/operator-sdk/issues/169 and https://github.com/operator-framework/operator-sdk/issues/161 will help here. Would also be interesting to know why we need both DeleteOp and DeleteOption, and why thetype Object runtime.Object type declaration exists.
PR lgtm though @hasbro17 should approve.

I've tested out the README and user-guide and the changes seem fine.
@rithujohn191 Can you also please update the CHANGELOG as well. Just mention that the action and query pkgs have been consolidated into pkg/sdk, and so their use in the handler is now sdk.Create(), sdk.Get() etc.
#169 is the next issue we're working on to define all the underlying pieces of the core SDK. Removing all the global state and defining clear structures will hopefully make the design more clear.

Would also be interesting to know why we need both DeleteOp and DeleteOption

@ericchiang DeleteOp is the wrapper around all the possible options(currently only metav1.DeleteOptions{}) and DeleteOption is the functional option type.
https://github.com/tmrts/go-patterns/blob/master/idiom/functional-options.md#options
https://dave.cheney.net/2014/10/17/functional-options-for-friendly-apis

why the type Object runtime.Object type declaration exists

I think in the original design we were anticipating to control all types in the SDK APIs but over time I'm not sure of the benefits of that. We're going to revisit this and  #161 in the upcoming restructuring.

updated the CHANGELOG: https://github.com/operator-framework/operator-sdk/pull/242/files#diff-4ac32a78649ca5bdd8e0ba38b7006a1eR9

lgtm thanks!

LGTM


Add MPR to contributors list

I think it's a great app, but It will be more used and useful if there is an offline version, that's why I tried to use Electon Framework, to convert it. It's my first opensource contribution, be kind please. There is a screenshot of what I obtain when I build the app on my desktop (Windows 10) :

Thanks for your PR, however we already have an Electron app. You can download it from https://about.draw.io/integrations/#integrations_offline

Thanks, but your link is not very visible, and on the website
alternativeto.net , they state that you only have an online application,
thanks for the information, I'm going to fix that, it is my little
contribution ;) .
2018-04-08 20:47 GMT+01:00 Gaudenz Alder<EMAIL_ADDRESS>
Closed #243 https://github.com/jgraph/drawio/pull/243.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jgraph/drawio/pull/243#event-1562734393, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AWFtIZiHDwsEbkrvyjkcuJ19gjk2ec5qks5tmmk_gaJpZM4TLrC0
.


[FEATURE] UI utilities for tutorial-like dialog windows

Introduction
Add UI utilities that show in-game dialog windows. These should be similar to the prompts seen during the tutorial section of the game.
Description of the feature

Feature usage
Would provide a basic method for mod authors to get input from the player. One example is when a mod needs to get the player's confirmation before doing something. Another would be when a mod needs to show the player a message that is worth pausing the game for (when GameMessage.Broadcast isn't enough).
Dependencies and Prerequisites
Does this feature require any new dependencies? No.
Implementation details
My gut feeling is that it is going to be difficult to use the game's tutorial dialog system for this... that code is probably coupled to the "case" and "objectives" systems. We'll probably have to create a hacky solution using the game's interface system.
Two possible approaches below, but we can discuss other ideas also.
// Idea 1: Create dialog objects
var dlg = new Dialog("Title", "Question");
var noBtn = dlg.AddResponse(buttonText: "No", Dialog.LeftButtonPosition);
var yesBtn = dlg.AddResponse(buttonText: "Yes", Dialog.RightButtonPosition);
noBtn.OnClicked += SomeAction;
Lib.Interface.ShowDialog(dialog: dlg, pauseWhileShowing: true, destroyAfterResponse: true);
// Idea 2: Create one helper function that handles everything
ShowDialog(string title, string question, string leftButtonText, Action leftButtonClicked, string centerButtonText, Action ...)

Note: Not a priority for v1.1.0

Should also look into: PopupMessageController.PopupMessage


Wrong response content

What steps will reproduce the problem?

Add middleware: FormatDataResponseAsJson::class
in controller: return $responseFactory->createResponse();

What is the expected result?
We should get empty body in response
What do you get instead?
But we get body with content-length = 4, and with content = null
Additional info

Q
A

Version
1.0.?

PHP version
7.4.9

Operating system
Ubuntu 16.04.6 LTS

Fixed.


Circular dependency between OMONNXOps and OMONNXRewrite?

I am still trying to get HandleLLVMOptions included in the project correctly and one of the things it does by default is build with: -Wl,-z,defs which forces the build of the shared libraries to resolve all symbols. While adding the dependencies for the libraries, I found that OMONNXOps and OMONNXRewrite appear to have a circular dependency as well. OMONNXOps needs the set of mlir::ONNXXXXOp::getCanonicalizationPatterns functions defined in OMONNXRewrite while OMONNXRewrite needs a slew of functions defined by OMONNXOps (the dependency of OMONNXRewrite on OMONNXOps is already in the cmake files). Is this expected?

I can look into this, probably early next week

Surveying a bit the code pattern, here is what I see.
Dialect/ONNX/ONNXOps.td generates the build/src/Dialect/ONNXOps.hpp.inc (included in ), which defines the getCaonicalizationPatterns for the ops that need this, e.g. for the ONNXAddOp
class ONNXAddOp : public ::mlir::Op<ONNXAddOp, ::mlir::OpTrait::ZeroRegion, ::mlir::OpTrait::OneResult, ::mlir::OpTrait::OneTypedResult<::mlir::Type>::Impl, ::mlir::OpTrait::ZeroSuccessor, ::mlir::OpTrait::NOperands<2>::Impl, ::mlir::MemoryEffectOpInterface::Trait, ShapeInference::Trait> {

[...]
  static void getCanonicalizationPatterns(::mlir::RewritePatternSet &results, ::mlir::MLIRContext *context);
[...]
};

These patterns are generated in the Transform/ONNX subdirectory, for example in Combine.cpp
void ONNXAddOp::getCanonicalizationPatterns(
    RewritePatternSet &results, MLIRContext *context) {
  results.insert<MulAddToGemmOptPattern>(context);
}

which are in turn generated by patterns in ONNXCombine.td, which as all ONNX based dialect DDR need the ONNXOps.td file.
So @sstamenova what do you propose? Should we pull basically the patterns that are defined by the ONNXOps.td (namely canonicalization, shape inference,...) all into the same library? Which would reside under the general src/Dialect/ONNX subdirectory?

That would be my expectation based on how the other dialects in mlir are defined. I don't have context, though, into why the decision was made to split some of them between the dialect lib and the transform lib fox ONNX MLIR.

Yes, MLIR has migrated to a model where under a given Dialect, they have multiple folders. We could go to that model too.

Each dialect in MLIR has multiple libraries associated with it (corresponding to each folder) but the libraries do not have circular dependencies. For example, for MLIRAffine, there's libMLIRAffine and libMLIRAffineTransforms with MLIRAffineTransforms strictly dependent on libMLIRAffine, so the definitions for getCanonicalizationPatterns are in libMLIRAffine.

@sstamenova yes, I am arriving at the same conclusion. If we are using DDR to generate basic optimization like canonicalization who are associated with the base dialect (where the ops are defined), then they need to be there too.

@AlexandreEichenberger : Do you think you could make the necessary changes to remove the circular dependency?

@sstamenova yes, let me do it now.


[vcpkg baseline][zeroc-ice] Remove zeroc-ice in ci.baseline.txt

zeroc-ice:x64-windows-static-md=fail was added by https://github.com/microsoft/vcpkg/pull/28522 due to the following post-check problem:
zeroc-ice:x64-windows-static-md install failed with following error on CI pipeline run:
error MSB6006: "slice2cpp.exe" exited with code -1073741515

This problem was random in compilation, in new CI the problem is gone. Later, I will track the error code and continue to analyze the cause of the problem.

[X] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[X] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[ ] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[ ] Only one version is added to each modified port's versions file.

Thanks!


updagre to photo_view 0.15.0

Hello,
I am using PDFX and I find it Awesome! Great job!!!
I need to upgrade my app to version 0.15.0 of the photo_viewer package but I can't because PDFX uses the older 0.14.0 version of photo_viewer.
Do you have plans to upgrade to version 0.15.0 of photo_viewer?
Thank you very much!

+1

+1


Component wishlist

Please comment here if you are lacking a component for a specific task.

Animation driving components, for LEDs etc: https://github.com/microflo/microflo/issues/71

Mean/variance and low-pass filtering http://www.embeddedrelated.com/showarticle/785.php

EVA ExponentialWeightedAverage
Squarewave/triangle/sawtooth
sinewave (using a table)
FIR filters. Lowpass/highpass. Butterworth/Chebychev
Downsample (decimate)
Running median
Could be in its own signals library, together with mean/variance. Should be fixed point.


version update 1.0.12 -> 1.0.13

@KamilSzostak,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Update opentracing java version to 0.22.0

@pavolloffay Version change - could you create a new release after merged?


Missing test images

Hiya, Running cargo test I see
---- test_hamamatsu_properties::case_1 stdout ----
thread 'test_hamamatsu_properties::case_1' panicked at tests/properties.rs:437:42:
called `Result::unwrap()` on an `Err` value: MissingFile("tests/assets/hamamatsu/CMU-3-40x - 2010-01-12 13.57.09.vms")
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- test_mirax_properties::case_1 stdout ----
thread 'test_mirax_properties::case_1' panicked at tests/properties.rs:365:42:
called `Result::unwrap()` on an `Err` value: MissingFile("tests/assets/mirax/CMU-1-Saved-1_16.mrxs")

---- test_leica_properties::case_1 stdout ----
thread 'test_leica_properties::case_1' panicked at tests/properties.rs:205:42:
called `Result::unwrap()` on an `Err` value: MissingFile("tests/assets/leica.scn")

---- test_trestle_properties::case_1 stdout ----
thread 'test_trestle_properties::case_1' panicked at tests/properties.rs:318:42:
called `Result::unwrap()` on an `Err` value: MissingFile("tests/assets/trestle/CMU-1.tif")

Hello,
I need to update instruction but in order to run tests you have to run this command:
make dl-test-images
Note: this can take some time but in the CI test data are always cached https://github.com/AzHicham/openslide-rs/actions/runs/11322570042/workflow#L41


Fix unmanaged branch notification show in case of adding a branch from an external terminal/console (closes #1308)

Seems fixed on my Linux! 🎉


Focus doesn't go to top of the screen/page for "Now you can upload your codes" with VoiceOver on [Global issue]

With VoiceOver on, when the user goes to ""Now you can upload your codes"", the focus is on last paragraph/sentence. It should be on top of the page. Tried it twice and the focus always goes to this area.
Screenshot below shows where the focus land for the VoiceOver user.
iOS 13.5.1 on iPhone 7

#490 #521 #519 GitHub

With VoiceOver on, when the user goes to "Now you can upload your codes" page, the focus now appears on the previous page and not on the header of the current page.
Tested on iPhone XR 13.5.1


[FEATURE] Mass tracking enabled by default

Currently pilots are reporting that mass tracking is not occurring correctly with in the connections logged in pathfinder.
There is an option to toggle "Save Mass" that has to be enabled by default but we need a clear jump log of all hole traffic to track mass for hole longevity and knowing accurately how much mass is left on the hole when taking large fleets out or calculating rolling mass.

There is no mass tracking, it is NOT possible in CREST. Pledge here https://forums.eveonline.com/default.aspx?g=posts&t=490255&find=unread for adding ship info to location endpoint.

Someone from CCP told ship information is available somewhere in the new Fleet Endpoint. I haven´t checked this.... So we could think about adding Fleet - CREST support to Pathfinder...
But this will grant access to lots of  "sensitive data" for Pathfinder. Not sure if people would like that.

This means, that if the character is member of corporation fleet it will show in CREST ship information otherwise not ?

Something like this... I think. Haven´t checked the Endpoint yet. But being in a Fleet is probably a requirement to get it out of the Endpoint.... Not very sexy. And a lot more CREST calls need to be done to get that information ....

Correct, shipType endpoint is only available currently in a fleet, making this much harder to track in Pathfinder.

@InfiniZ https://forums.eveonline.com/default.aspx?g=posts&m=6639828#post6639828

Referencing new ESI support: https://esi.tech.ccp.is/dev/#!/Live/get_characters_character_id_ship
Should be able to calculate an estimate from that, could go a step further and match the ship_item_id with assets from the XML API to get the exact fit and be more accurate.

New Feature for "mass tracking":

Ship mass will be tracked for wormhole jumps
New "Connections" panel was added to the UI
Multiple connections can be selected and compared by its mass status

click at connection to select
ctrl+ click adds more connections to that that panel
new context menu option "select connections" added to systems

Connections that have signature mapped to will show total mass
Your current ship mass can be added to mass calculation
Warnings will be shown in case your current ship:

Does not fit through that type of wormhole
Your ship mass will make the wormhole collapse after jump

Estimation for "mass left" is available (in percent)


RFP - 3utools

Package Details
Software project URL : https://www.3u.com/
Direct download URL for the software / installer : https://url.3u.com/zmAJjyaa
Software summary / short description: This software use for Flash iOS Devices and Other Various Opertions

Yes This Package is Same as You Shared Link

Yes This Package is Same as You Shared Link

Based on this I'm going to close this issue.


Package naming

Should the package be named pony-stubs, types-pony or something else?
@fluffy-critter suggested that

Also, just as a note, a more standard name for the types package would be types-pony.

at https://github.com/ponyorm/pony/issues/479#issuecomment-1100930108
But as far as I could find, this PEP (https://peps.python.org/pep-0561/#type-checker-module-resolution-order) suggests that type-only modules should be named <package_name>-stubs. Personally I don't have an opinion other than that we go as standard as possible.
@fluffy-critter can you find another PEP that would propose the naming you suggested, or is it more based on what is commonly done?

I'm basing it on the various types packages I've seen, for example types-python-dateutil, types-requests, and types-atomicwrites. I wasn't aware of any PEP-provided naming scheme, and I hadn't ever come across a <package>-stubs package before either although it appears there are quite a few with that naming convention as well (e.g. lxml-stubs).

Thanks, now that I look at MyPy docs for example, they claim that

As many packages don’t support type checking yet, you may also have to install a separate stub package, usually named types-<library>.

but afterwards they suggest

If the package is stub-only (not imported at runtime), the package should have a prefix of the runtime package name and a suffix of -stubs

which might be a bit confusing if not contradictory :smile: but I guess we'll keep the name as is for now, at least for Pyright it also enables automatic stub package resolution

Yeah, sounds good to me. Weird that this inconsistency happened, but that's a larger Python ecosystem issue and not something specific to this package. :)


Need example

I'm trying to install Warframe in the most efficient way, without using steam
But I need an example, the only one I could find is this one.
https://www.reddit.com/r/NixOS/comments/sflmgj/wine_issue/
My home is not permanent, that's why being able to move it would suit me
any help would be appreciated, i still have a bit of trouble with some aspect of nix

Please elaborate more on what the issue is.

But I need an example, the only one I could find is this one.

That example is for osu! and it's from this repo. Not sure how it's relevant.

My home is not permanent, that's why being able to move it would suit me

That should be something you take care of. In case of impermanence, you can add paths that won't be deleted on boot.


Add support for rocket_contrib's uuids

This should fix https://github.com/GREsau/okapi/issues/38. I'm creating the PR here as we're using your branch in our application.
It should be noted that FromForm still does not work with rocket_contrib uuids, because they do not implement the JsonSchema trait. This should probably be fixed in schemars by adding support for rocket_contrib.

Hello,
Thanks for contributing and for review @raffomania, @ralpha !
Sorry I didn't took any action sooner, I didn't notice this PR. @raffomania , are you still interested in working on this PR?
Best regards,
Marcin

Awesome, thanks for the review! I will update the PR within the week.

This should be a closed and the rebased and updated PR should be made directly against the now-active-again upstream over at  https://github.com/GREsau/okapi

@svenstaro you're right. @raffomania I'm closing this PR here.

I will open a PR upstream, thanks for the reviews. This might even be supported out of the box, with rocket including uuid now.


Fetch dependency missing

To use I had to add fetch to cod-api/index.js:
const fetch = require('node-fetch');

Fixed https://github.com/jakejrichards/cod-api/commit/8adf0555b6c800cbb44bf8358346b656e4390489


Calibrated MIRI spectra

Generating calibrated MIRI spectra was super easy... just a few lines.

Gah, but for some reason the pytest failed which really can't be related to your changes... Let me see what I can do - probably just need to set an upper limit or something

Looks to me like it is just that your flexo branch is based on an old version of the main branch, and I think merging in the updates from main should let the tests pass without any new edits to the requirements

Codecov Report
Attention: 326 lines in your changes are missing coverage. Please review.

Comparison is base (559f703) 57.02% compared to head (b783b44) 57.60%.
Report is 148 commits behind head on main.

Files
Patch %
Lines

src/eureka/S5_lightcurve_fitting/models/GPModel.py
5.00%
152 Missing :warning:

src/eureka/S5_lightcurve_fitting/s5_fit.py
30.30%
23 Missing :warning:

src/eureka/S3_data_reduction/nirspec.py
28.00%
18 Missing :warning:

src/eureka/S3_data_reduction/niriss_python.py
11.76%
15 Missing :warning:

src/eureka/S5_lightcurve_fitting/plots_s5.py
71.15%
15 Missing :warning:

src/eureka/S5_lightcurve_fitting/likelihood.py
54.83%
14 Missing :warning:

src/eureka/S3_data_reduction/miri.py
57.14%
9 Missing :warning:

src/eureka/S3_data_reduction/s3_reduce.py
80.43%
9 Missing :warning:

src/eureka/S3_data_reduction/nircam.py
61.90%
8 Missing :warning:

...curve_fitting/differentiable_models/PyMC3Models.py
71.42%
8 Missing :warning:

... and 18 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #585      +/-   ##
==========================================
+ Coverage   57.02%   57.60%   +0.58%
==========================================
  Files          94       96       +2
  Lines       11282    11553     +271
==========================================
+ Hits         6433     6655     +222
- Misses       4849     4898      +49

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


66-introduction-for-maxence-guindon

Add Maxence's introduction in both files:

internships.md
the-software-development-team.md

adding @RussellJimmies as a reviewer as I'm a bit dubious on the language in both English and French.

rewrap to 80

Done with rewrap extension (alt+q) could be useful for other intern as well. Thank you @ibrahim-kabir for udpate the doc.

Rebase done


Angular – standalone component compatibility with LucideAngularModule.pick

Prerequisites

Version: 0.219.0
Are you running from source/main: No
Are you using a released build: Yes
Operating system: MacOS

Step to reproduce

Follow the installation guide on the npm page for lucide-angular.

Try to use with a standalone component.

import {Component} from "@angular/core"
import {Box, LucideAngularModule} from "lucide-angular"

@Component({
  imports: [LucideAngularModule.pick({Box})],
  selector: "test",
  standalone: true,
  template: ``,
})
export class TestComponent {}

Actual behavior
The code does not compile.
Compilation error:
TS2322: Type 'ModuleWithProviders ' is not assignable to type 'readonly any[] | Type '
Notes
The current import strategy is not compatible with Angular's standalone components.

Thanks for the bug report, @R-Bower, I'm a bit busy at the moment, but I will get to this as soon as my time allows.

hey, @karsa-mistmere has anything come up on this ticket yet? I want to use the Lucide icons in any case and also just in the combination with angular 16 and standalone components :)

@R-Bower, @Bonus2712 : This should now be resolved in the latest release https://github.com/lucide-icons/lucide/releases/tag/0.269.0

Hi @karsa-mistmere. I appreciate your work on this bug.
I am currently trying to get lucide-angular to work with standalone components as well. I have followed the steps to reproduce and I am still seeing errors in my project. I am using version 0.274.0
I am receiving these errors while running my app:
-NG2012: 'imports' contains a ModuleWithProviders value, likely the result of a 'Module.forRoot()'-style call. These calls are not used to configure components and are not valid in standalone component imports - consider importing them in the application bootstrap instead.
and
error TS2322: Type 'ModuleWithProviders<LucideAngularModule>' is not assignable to type 'readonly any[] | Type<any>'.
Please let me know if there is any other information that will help with your investigation. I would love to use standalone components and lucid-angular!

Hey @seangolob,
The error you get is loud and clear: you cannot use a LucideAngularModule.pick({...}) style import. This is not specific to Lucide, it's how standalone components work.
As a workaround you can do either of the following:

import just LucideAngularModule, no pick() added and then either:

use the LUCIDE_ICONS injection token to provide icons in your standalone component's providers list, see https://lucide.dev/guide/packages/lucide-angular#injecting-multiple-icon-providers
pass LucideIconData objects directly as <lucide-icon [img]="HomeIcon" />

create a custom module that both imports and exports LucideIconModule, with your necessary picks added

(My preferred modus operandi is 1B)

@karsa-mistmere Thank you for your guidance. I am new to the angular space and I am still getting my bearings straight. Thank you!


Method added to get the status code from the last API call

Hi Timo, I've added a new method to the runtastic class to get the response http code from the last API call.
I would like to add it to your repository, so the changes will be added to the composer package.
Best regards.

Merged.
Thanks for the addition. :)


Could not find package elm-lang/core.

Here's what I've done :

npm install -g yeoman
npm install -g<EMAIL_ADDRESS>npm install -g yeoman-elmlang
yo elmlang
gulp :

[20:11:27] Using gulpfile ~/prj/longtail/gulpfile.js
[20:11:27] Starting 'build'...
[20:11:27] Starting 'clean:dev'...
[20:11:27] Starting 'watch'...
[20:11:27] Finished 'watch' after 13 ms
[20:11:27] Finished 'clean:dev' after 24 ms
[20:11:27] Starting 'styles'...
[20:11:27] Starting 'copy:bower'...
[20:11:27] Starting 'elm-init'...
[20:11:27] Finished 'copy:bower' after 20 ms
[20:11:27] Starting 'copy:dev'...
[20:11:28] styles all files 67 B
[20:11:28] Finished 'styles' after 293 ms
[20:11:28] 'elm-init' errored after 105 ms
[20:11:28] Error in plugin 'gulp-elm'
Message:
    Could not find package elm-lang/core.

Maybe your elm-stuff/ directory has been corrupted? You can usually fix stuff
like this by deleting elm-stuff/ and rebuilding your project.

[20:11:28] index.html & CNAME all files 292 B
[20:11:28] Finished 'copy:dev' after 88 ms

Removing elm-stuff did not help.
elm-package install returns without error.
Is this expected ? Am I missing something ?

Did it scaffold all the files properly? What's the content of your elm-package.json?

I have not edited it yet :
{
  "name": "generated-elm",
  "private": false,
  "version": "0.1.0",
  "description": "Generated elm",
  "dependencies": {},
  "devDependencies": {
    "browser-sync": "1.9.2",
    "del": "1.2.1",
    "gulp": "3.9.1",
    "gulp-autoprefixer": "2.3.1",
    "gulp-cache": "0.2.10",
    "gulp-cached": "1.0.1",
    "gulp-changed": "1.0.0",
    "gulp-elm": "0.4.1",
    "gulp-filter": "2.0.2",
    "gulp-gh-pages": "0.4.0",
    "gulp-gzip": "0.0.8",
    "gulp-htmlmin": "0.2.0",
    "gulp-if": "1.2.5",
    "gulp-imagemin": "2.3.0",
    "gulp-jshint": "1.11.2",
    "gulp-load-plugins": "0.8.1",
    "gulp-minify-css": "0.3.13",
    "gulp-plumber": "1.0.1",
    "gulp-rev-all": "0.7.6",
    "gulp-rev-replace": "0.3.4",
    "gulp-sass": "2.1.0",
    "gulp-sequence": "0.4.1",
    "gulp-shell": "0.2.11",
    "gulp-size": "1.3.0",
    "gulp-uglify": "1.4.1",
    "gulp-uncss": "0.5.2",
    "gulp-useref": "1.3.0",
    "jshint-stylish": "1.0.2",
    "merge-stream": "0.1.8",
    "wiredep": "2.2.2"
  },
  "engines": {
    "node": ">0.10.0"
  },
  "scripts": {}
}

No, not the package.json the elm-package.json :)

And after you ran manually elm-package install (which should happen automatically), did it install all the packages?

Now gulp works, but the generated Main.js file can not be loaded. I get a "missing ;" error around those lines :
var _elm_lang$html$Html_Events$Options = F2(
	function (a, b) {
		return {stopPropagation: a, preventDefault: b};
	});

var _phtrivier$longtail.git$Counter$countStyle = _elm_lang$html$Html_Attributes$style(
	_elm_lang$core$Native_List.fromArray(
		[
			{ctor: '_Tuple2', _0: 'font-size', _1: '20px'},
			{ctor: '_Tuple2', _0: 'font-family', _1: 'monospace'},
			{ctor: '_Tuple2', _0: 'display', _1: 'inline-block'},
			{ctor: '_Tuple2', _0: 'width', _1: '50px'},
			{ctor: '_Tuple2', _0: 'text-align', _1: 'center'}
		]));

(The generated file is huge, I have no idea where the error can be :/ , but it's probably not an error in the generator, rather in the example file... so feel free to close this issue if needed... Thanks)

... and it's because the generator uses the github url from the elm-package.json file, and since I typed "http://github.com/phtrivier/longtail.git", for some reason the url in the json file was "http://github.com/phtrivier/longtail.git.git", which caused the error.
Editing the file fixed the problem.
All apologies.


imagick does not work for webp

Hello imagick does not work for webp conversion even with the webp format in the configuration.
I have no error message but cannot have my image in webp format.
On the other hand, if I use GD2 with webp enabled, it works.

config : windows 10 + mamp pro (windows)
Best Regards

Hi,
I have the same problem ::make from webp wont do a thing, black image.

This feature should be available in version 3.


[Port dspace-8_x] Add missing wosPublisherContrib key-ref in wos-integration.xml

Port of https://github.com/DSpace/DSpace/pull/9581, already approved and merged to main and 7.x but I missed the tag out for the autogenerated 8.x port

Merging immediately as this is a port of an approved PR (main, 7.x) #9581


Package request: WebCord

A Discord and SpaceBar electron-based client implemented without Discord API.

Hardened for privacy
Follows the best security practises
Customizable
ARM-friendly and Linux mobile support

Metadata

homepage URL: NaN
source URL: https://github.com/SpacingBat3/WebCord
license: MIT
platforms: Linux, MAC and Windows

Hi @lisbonjoker !
The Webcord package is already available in nixpkgs.
You can search it using search.nixos.org.
Closing the issue now. Thanks !


Where is the chat history stored?

Is it possible to get the chat history? Is there an Azure API to see the history of the chat?

Hi @fpereiramapfrepr, thanks for filling a question. Yes you can get the chat history for a chat thread using ChatSDK for the language of your choosing.
For Javascript ChatSDK, please refer to the example for chatThreadClient.listMessages().
Make sure you are using the ChatClient/ChatThreadClient as a user that is part of a chatThread. For more information refer to concepts


Fix bad interaction between SIL deserialization and weak linking [5.1 08/28]

Description: When using conditionally-available framework declarations from a client app, we're supposed to weak link those declarations. However if the reference came from deserialized SIL code, eg if the client calls a function in the framework that has a default argument, then that reference was not weak linked. This is because SIL functions only stored a single 'weak linked' bit that was computed from the deployment target of the framework together with the declaration's availability. But instead we need to store the version range in the SIL function and make the determination based on the deployment target of the client into which the SIL function is deserialized.

Scope of the issue: This affects anyone that's writing an app that deploys to OS version N, but takes advantage of a Swift-only framework that's available in version N+1.

Origination: The problem has been there ever since weak linking was introduced in Swift 5.0.

Risk: It's a big change, so medium, but mostly it's a straightforward change.

Reviewed by: @jrose-apple

Radar: rdar://problem/52783668

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 80aae9e6a612204d7766f5e650f111860a625ad8

@dcci @shahmishal Is this a known failure on this branch? Looks unrelated to my PR:
14:46:19 Testing Time: 32.56s
14:46:19 ********************
14:46:19 Unresolved Tests (1):
14:46:19     lldb-Suite :: lang/swift/enum_objc/TestEnumObjC.py

@swift-ci Please test Linux


more publications

More publications should show the publications in the same format with authors etc (detailed list), currently, it just shows the names.
I don't want a search functionality through more publications, I just want publications to be shown.
But, I do not want all publications in the main home page, when scrolling down.
How can I do this in hugo academic?

Can't you increase the publications count in the widget?
https://github.com/gcushen/hugo-academic/blob/master/exampleSite/content/home/publications.md

@chasgames - I can and that's what I have done, but what I'd ideally like to do is that *  I do not want all publications * in the main page when one scrolls done. I would like what the "recent publication" feature gives.
However, when I click "more publications", it shows me a list with only titles, and the buttons, the author names, conference, dates etc. dissapear. Also the formatting changes completely.
Here is a snapshot from "more publications"

and here is from "publications"

I would like "more publications" to show the complete list but in the same format. Hope that makes sense?


Scroll resets when ngModel changes?

When I change ngModel programmatically (rather than by typing in the editor), the scroll jumps back up to the top line. Any idea what's causing this? Is it expected behavior?

Looks like setValue() is only meant for when the entire code changes, and you're supposed to use replaceRange() when only a portion of the code changes.
Will close this issue, but I would recommend documenting how to access the internal codemirror instance in your README, i.e.
<codemirror #editor [(ngModel)]="code"></codemirror>
<a (click)="editor.instance.replaceRange('hello world', {line: 0, ch: 0})">click me</a>


Add timeout parameter/option for ConnectPeer(Request)

Background
I could use a timeout=  setting in the ln.ConnectPeerRequest request, and in lncli connect. Now it seems to timeout after 2 minutes [?]. Being able to set this for a certain amount of seconds will make my application a bit smoother.
Your environment

LND 0.9.1-beta commit=v0.9.1-beta
CentOS 8 (4.18.0-147.5.1.el8_1.x86_64)
bitcoind bitcoin-0.18.1

Steps to reproduce
Try to connect to an inactive node:
lncli connect 02596a686579585d52bf1e9f560b470378c64450b4411a86577a200387432cf8cd@<IP_ADDRESS>:9735
Wait around 2 minutes.
See a failure message.
Expected behaviour
If I am able to set a 10 second timeout I do not have to wait long :).
Actual behaviour
It times out after about 2 minutes.

i'm working on this one.

I noticed that for many peers the timeout does not work. I have a 5 second timeout but sometimes I see delays of 3+ minutes
Call
response = stub.ConnectPeer(request = ln.ConnectPeerRequest(addr=address, perm=False, timeout=5))
result
Mon Sep  5 11:07:46 2022 : ... Attempting connection to peer.alias='MidasMulligan' inactive_peer='03fd21fdee8e9adc070b53a0bca8685cbc0b97f27b72f364723dcc54b4164ab21b' host='322e5mbonli3h4r4la6ox2gt2cm534k7uk3cbl6yr34rtxyfnj6mdiid.onion:9735'
Mon Sep  5 11:11:39 2022 : .... Error reconnecting MidasMulligan inactive_peer='03fd21fdee8e9adc070b53a0bca8685cbc0b97f27b72f364723dcc54b4164ab21b' str(e)='<_InactiveRpcError of RPC that terminated with:\n\tstatus = StatusCode.UNKNOWN\n\tdetails = "dial proxy failed: socks connect tcp <IP_ADDRESS>:9050->322e5mbonli3h4r4la6ox2gt2cm534k7uk3cbl6yr34rtxyfnj6mdiid.onion:9735: unknown error general SOCKS server failure"\n\tdebug_error_string = "{"created":"@1662376299.194364800","description":"Error received from peer ipv4:<IP_ADDRESS>:10009","file":"src/core/lib/surface/call.cc","file_line":966,"grpc_message":"dial proxy failed: socks connect tcp <IP_ADDRESS>:9050->322e5mbonli3h4r4la6ox2gt2cm534k7uk3cbl6yr34rtxyfnj6mdiid.onion:9735: unknown error general SOCKS server failure","grpc_status":2}"\n>'


Uncaught Error: Module did not self-register.

[Enter steps to reproduce:]

...
...

Atom: 1.46.0 x64
Electron: 4.2.7
OS: Unknown macOS version
Thrown From: Atom Core
Stack Trace
Uncaught Error: Module did not self-register.
At ELECTRON_ASAR.js:160

Error: Module did not self-register.
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:160:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:722:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:169:18)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:20:18)
    at customRequire (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:1:695134)
    at get_KeyboardLayoutManager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2778162)
    at get_manager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2778264)
    at Object.getCurrentKeyboardLayout (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2778390)
    at e.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1081110)
    at KeymapManager.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1253631)
    at KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1251616)
    at WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:284744)

Commands
     -2:45.9.0 editor:consolidate-selections (input.hidden-input)
     -2:44.3.0 core:select-all (input.hidden-input)
     -2:44.1.0 core:copy (input.hidden-input)
     -0:02.7.0 core:save (input.hidden-input)

Non-Core Packages

Getting exactly the same stack trace.
For me was editing a README.md file and the application just crashed completely - i.e. disappeared from screen as if it hadn't been running.
Re-opened atom, but if I interact with the file I was editing every keypress triggers this error int the IDE.
I can still save the document and edit, but a long list of uncaught errors fill the right hand side of the ide.
Quitting and re-opening things seem to be working again.
For what its worth I reckon the initial crash had left some lock or resource in use, when I first opened the application I reckon this is still in play causing the issue.
Cleanly closing Atom I reckon whatever that garbage is causing the problem its cleaned up properly, as the next restart the application seems to be working fine (so far).

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/20222. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Video Recording Features

This branch will add features for recording video data from participants.

It adds a new plugin initialize-camera that gets permission to record from a camera and allows the user to select which camera to record from.
It adds functions to the pluginAPI to support camera recording.

I am planning to also include an extension that will record video data while trials are running.

Based on current tests it looks like the extension adds the data from the previous trial to the current trial. Posting a note here to remind myself when I get back to working on this.
Also need to think about how to handle the data that's generated from this. Right now I'm generating a Blob for each trial and just storing it in the data structure. This doesn't serialize well, so we probably should do something like base64 or give an option to hook into a server side script for processing video files.
@becky-gilbert have you done any experiments with video recordings that you needed to store as the experiment ran?
@javidalpe any thoughts about what a good API would be for Cognition to consume here?

@becky-gilbert and @bjoluc there's an interesting problem with the video recording extension.
When MediaRecorder.stop() is called, it triggers a dataavailable event. I've got the on_finish event handler of the extension configured like this:
https://github.com/jspsych/jsPsych/blob/1e2df37f8f7ac836ebe132a684cc83fb09156f1c/packages/extension-record-video/src/index.ts#L34-L42
The problem is that the return executes before the final dataavailable event is triggered, so the data stored by the extension is missing the final bit of data. This results in corrupt video files.
Two potential solutions:

We could allow on_finish from an extension to be async, which would require some changes to the core. It also potentially creates some weird situations where a trial finishes but there's a delay due to an extension taking a while to finish. This would be hard for users to diagnose.
I'm not sure if this is possible, but I'm wondering if there is a way to make the return value in the object something that will resolve to the final data when it is ready. Perhaps returning a parameter that is the output of a function, and the function is async?

Any other ideas?

I'm fully on board with solution (1). It's very obvious and rather simple to implement (+maintain).

It also potentially creates some weird situations where a trial finishes but there's a delay due to an extension taking a while to finish. This would be hard for users to diagnose.

By allowing async on_finish callbacks, we empower extension authors to block the experiment for a while (something that is already possible right now using long-running blocking functions). There are certainly reasons why this would be necessary in some extensions and I'm fine with it. If one of our extensions introduces a notable delay, we should mention it in the extension docs to avoid confusion. Extension devs using an async on_finish function will know their reasons, and other's won't see a need to use it. Power to the extension devs :muscle: Well, mostly us at the moment. Power to us :muscle:
Re 2: We could for sure allow Promises in the result data, but that would be way to complex IMHO, given that we would need to (deeply?) check the output data for promises and possibly adapt some data functions to work with promises(?). If we (a) continue with the next trial only when all promises in result data have been resolved, there's no functional advantage over solution (1). If we (b) continue right away, a user cannot be sure anymore whether all result data is actual data – which sounds like trouble to me...
Let me know if you'd like a helping hand here (I could jump in next week) :)

Alright, I'm convinced. And I'd love some help!
💪

I think this is ready for release. @becky-gilbert any interest in looking it over first?

Yes, can you give me a day or two please? I'll review ASAP!

Done with my review and commits. I didn't actually build/run all of the examples/demos, but I can do that tomorrow if it would be helpful? Otherwise I think it's ready! Thanks for waiting for me 👍

Thank you for looking this over! I've tested on my end so I'm gonna go ahead and merge.


support concurrent inference from multiple models

I would like to stream the response from two different LLMs simultaneously

Thanks for the request. Having multiple models in a single engine simultaneously is something we are looking into now. Meanwhile, would having two MLCEngine work for your case?

Yes that should work, assuming the device has enough resources, is this possible today? Is there an example I can play with?

Hi, I found this issue when looking to report a bug.  If you make 2 requests with the same engine, they interleave their outputs.  Example incoming.
I’m down to fix this with just a pointer at where to look.

Hi @mikestaub, from npm 0.2.60, a single engine can load multiple models, and the models can process requests concurrently. However, I have not tested the performance benefit (if any) to process requests simultaneously, as opposed to sequentially. Though being able to load multiple models definitely brings convenience, making the engine behave like an endpoint like OpenAI(), assuming enough resources from the device.
Note: each model can still only process one request at a time (i.e. concurrent batching is not supported).
The two main related PRs are:

https://github.com/mlc-ai/web-llm/pull/542

Main changes needed to support loading multiple models in an engine

https://github.com/mlc-ai/web-llm/pull/546

A patch to the PR above to support simultaneously processing/streaming response

See examples/multi-models for an example, which has the effect below with parallelGeneration():
https://github.com/user-attachments/assets/0c5188de-b7ed-496d-a56a-28af36b11e0a

Closing this issue as completed. Feel free to reopen/open new ones if issues arise!


Add classifications for TagHelper element/attribute quick info tooltips in VS

Summary of the changes

Adds classifications for Razor TagHelper element/attribute quick info tooltips.
Some of the logic here can be reduced once LSP VSCompletionItems support returning ContainerElements instead of just ClassifiedTextElements, tracked by https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1319274.

Before:

After:

Fixes:
https://github.com/dotnet/aspnetcore/issues/30224

🎉


How to inherit resourceTypes by resource?

Hi guys.
First of all, thanks a lot for all you work. Enjoying your parser a lot so far.
When working with resourceTypes I've faced an issue: How do I make resource inherit from one of resourceTypes properly in ramlfications?
Docs section says that

When a resource has a trait and/or type assigned to it, or a resource type has another resource type or a trait assigned to it, it inherits its properties.

I have following RAML file:
#%RAML 0.8
---
title: Example REST API
documentation:
    - title: Home
      content: Welcome to the example API.
baseUri: http://example.com
version: v1
resourceTypes:
    - collection:
        description: The collection of items
        get:
            description: Get all items, optionally filtered
        post:
            description: Create a new items object

/items:
    type: collection

What I expect after parsing this file is:

1 /items resource with method GET
1 /items resource with method POST
Both resources having proper descriptions
/items resources having proper resource description (The collection of items)

But when I parse it and check results in console, that's what I see:
In [1]: import ramlfications

In [2]: api = ramlfications.parse('example.raml')

In [3]: api.resources
Out[3]: [ResourceNode(method=None, path='/items')]

In [4]: resource = api.resources[0]

In [5]: resource.description
Out[5]: ---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
.......
TypeError: __repr__ returned non-string (type NoneType)

I tried defining methods on /items route as follows and was hoping inheritance will have this way but had no luck again.
...
/items:
    type: collection
    get:
    post:

Please give an advice on what is the correct way to achieve what I want using ramlfications?
Thanks for your time! :)

Hey @postatum - so sorry for the delay; was traveling a lot.  (also, slight nitpick: guys, there's only one main dev here, and I'm a woman :smiley: )
I think you've stumbled on a bug; I agree that resource.description should inherit the get description defined above.  Thanks so much for bringing it to my attention!

Hi @econchick. Thanks for answering.

also, slight nitpick: guys, there's only one main dev here, and I'm a woman
Sorry, didn't mean to offend you :) English isn't my mother tongue and for some reason I thought "guys" refers to group of any people, not just men.

I agree that resource.description should inherit the get description defined above.
resource.description was just an example. The more important issue (or maybe it works like this on purpose) is that if resourceType defines methods "a, b, c" and resource uses that resource type, it feels like there should be generated 3 resources - for methods a, b and c.

I've been rereading the raml spec - If a resource type has three methods, and a resource inherits it like the following:
resourceTypes:
  foo:
    get:
      description: get some foo
    post:
      description: post some foo
/foobar
  type: foo

I would expect it to inherit both get and post and both of their descriptions.  However, if it was like this:
resourceTypes:
  foo:
    get?:
      description: get some foo
    post?:
      description: post some foo
/foobar
  type: foo
  get:

I would expect /foobar to only inherit get and its description (notice the ? in the methods of the resourceTypes).
Seems like ramlfications has some wonky behavior when inheriting traits.  Looks like I got some work to do!

Hi again @econchick.
Thanks for working on this issue! We would love to use it :)
I've just tried to use version 0.1.8 and I had troubles with it.
Here is my RAML file https://github.com/postatum/ramses-example/blob/99425204_use_resourcetypes/example.raml. When I look at resources generated from it, here is what I see (formatted to make it easier to see things):
[ResourceNode(method='post', path='/users'),
 ResourceNode(method='get', path='/users'),

 ResourceNode(method='get', path='/users/{username}'),

 ResourceNode(method='get', path='/users/{username}/settings'),
 ResourceNode(method='post', path='/users/{username}/settings'),

 ResourceNode(method='get', path='/users/{username}/groups'),
 ResourceNode(method='post', path='/users/{username}/groups'),

 ResourceNode(method='get', path='/users/{username}/profile'),
 ResourceNode(method='post', path='/users/{username}/profile'),
 ResourceNode(method='patch', path='/users/{username}/profile'),

 ResourceNode(method='post', path='/stories'),
 ResourceNode(method='get', path='/stories'),
 ResourceNode(method='delete', path='/stories'),

 ResourceNode(method='get', path='/stories/{id}')]

As you can see, only the first method from resourcetype is inherited.
So I looked in the code here https://github.com/spotify/ramlfications/blob/0.1.8/ramlfications/parser.py#L824-L849 and I think I found few issues (please excuse me if I didn't understood some code correctly):

resourceType methods are only inherited when resource does not define ANY methods, because line 839 is only run if no methods found in resource. Should inherited methods be generated as well as methods defined in resource?
In lines 824-829 only first method of resourceType is inherited, because _resource_type_lookup returns only first resourceType method.
3 Idea "inherit only if resource doesn't define any methods" breaks because first inherited method is added before check in line 830. E.g. if resource doesn't define any methods but uses resourceType, first method from that resourceType will be added to methods var and check in line 830 will be True though that check seems to be checking that "resource defines methods".

Hey @postatum - thanks for the update!
I'm working on a bit of a rewrite to clean out the spaghetti that the code has become. And I'll definitely write a few test cases for this to fix it.
How high of a priority/issue is this for you folks? I'll adjust my prioritization if necessary (juggling a lot of things at the moment).

A clarification question: w/r/t your example RAML file, would you expect /stories/{id} to inherit all 5 methods since its parent is of the collection type? What about /users/{username}/?

Hi @econchick.

A clarification question: w/r/t your example RAML file, would you expect /stories/{id} to inherit all 5 methods since its parent is of the collection type? What about /users/{username}/?

I expect /stories/{id} and /users/{username} to inherit all methods from item resourceType.
I also expect /stories and /users to inherit all methods from collection resourceType.
PS. @jstoiko will answer your question regarding priorities later.

@postatum I just re-read the RAML spec, and if I understand correctly, I think /stories/{id} and /users/{username} should not inherit resource types.  I'm particularly looking at this line under Resource Types and Traits:

Resource type definitions MUST NOT incorporate nested resources; they cannot be used to generate nested resources when they are applied to a resource, and they do not apply to its existing nested resources.

Do you think it should be otherwise?

Here is how I understood it:

Resource type definitions MUST NOT incorporate nested resources;
Since resourceType is similar to resources, users may think they can define nested resources in resourceType and this says - no you can't.

they cannot be used to generate nested resources when they are applied to a resource,
Same as first, but if nested resource is already defined in resourceType, that resource should not be generated.

and they do not apply to its existing nested resources.
If resourceType is applied to "/stories" it will not be applied to "/stories/{id}" and other nested resources.

But in our case, "/stories" and "/stories/{id}" apply their own resourceTypes. "collection" for "/stories" and "item" for "/stories/{id}".
We can try to ask RAML team to interpret that piece of spec.

Ah sure yea - definitely makes sense. I missed that item was applied to the nested resources.
But If it weren't applied, would you agree that the child resource would not inherit its parent's type?

But If it weren't applied, would you agree that the child resource would not inherit its parent's type?

I agree. I think resource should only inherit resourceType if it's directly applied to it.

awesome thank you :)

It's not a blocker however our RAML files will look a lot nicer once this is fixed :)


controller: Fix app lookup when streaming events

The Events handler is not wrapped in appLookup, so there will never be an app in the context.
The appID is also passed as a query string rather than a param, so we can't use appLookup.

The TestRouteEvents test is failing due to the stream being opened before the app is created. As previous to this fix that worked the same as not passing in the app id, simply removing it from the test seems like a reasonable option.

LGTM pending comment

Is there a test that can be added to verify the fix?

@jvatic

TestRouteEvents test is failing due to the stream being opened before the app is created

Maybe we should just not do the app lookup? I think it's reasonable to want to stream events for an app before you create it to avoid races (although that can be avoided by passing past=true), wdyt?
@titanous

Is there a test that can be added to verify the fix?

I will write a test (create two apps, stream events for one, check I don't get events for the other).

@lmars

Maybe we should just not do the app lookup? I think it's reasonable to want to stream events for an app before you create it to avoid races (although that can be avoided by passing past=true), wdyt?

If we don't do the app lookup we'll need to make sure the app name can be used in place of the id. I think using past=true is the simpler option.

@jvatic @titanous comments addressed

LGTM

LGTM


logger: Display timezone as offset

Change format of time included in log entries to display the timezone
as an offset.
Example of old format:
time="2017-09-28 13:22:58.965483966 +0100 BST"

Example of new format:
time="2017-09-29T13:22:58.965483966+01:00"

Fixes #650.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

lgtm

All tests passed except for coveralls, which is stuck as usual. More oddly than usual, https://coveralls.io/jobs/29948262 claims the unit-test coverage has decreased by 1.8%. However, it's lying and in fact the coveralls report doesn't even list all the files in the repo!?! I've re-run the unit-tests locally on this branch and coverage for logger.go is still 100%.
Merging.


[Windows] Add signature validation

Description
Adding signature validation for several windows packages
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2019, windows2022

/azp run windows2019, windows2022

/azp run windows2019,windows2022

/azp run windows2019,windows2022


Added script for running litefuzz and explanation

This PR adds a script that will invoke litefuzz to test a specific program for crashes. The specific programs in this case could be vpack-to-json, json-to-vpack...
Litefuzz is a program that creates fuzz tests for finding bugs and improving programs. It can be found in this
repository:
litefuzz.

Coverage remained the same at 96.714% when pulling 365f516522aa18c15660cafef8f1930e9752077a on feature/script-run-litefuzz into 0f2aa825f08de5e160a8282430e5c52adf1c2a68 on main.


Refresh from stable to edge

Hello,
I was trying to switch from stable (6.09) to edge (6.55) using snap refresh, but mongodb is no longer starting. I can see the following in the logs:
2022-11-15T17:37:50Z wekan.mongodb[30181]: mongodb bind options:  --bind_ip <IP_ADDRESS> --port 27019
2022-11-15T17:37:50Z wekan.mongodb[30181]: Sending mongodb logs to /dev/null
2022-11-15T17:37:50Z wekan.mongodb[30788]: Error parsing command line: unrecognised option '--bind_ip'
2022-11-15T17:37:50Z wekan.mongodb[30788]: try 'mongo --help' for more information
2022-11-15T17:37:50Z wekan.mongodb[30789]: Error parsing command line: unrecognised option '--bind_ip'
2022-11-15T17:37:50Z wekan.mongodb[30789]: try 'mongo --help' for more information

When reverting back to stable, everything works again. Is switching between channels not supported?
Thank you
Manuel

Switching between channels is not supported.
In theory, you could do mongodump / mongorestore, because they use different MongoDB versions, but there is also other bugs like attachments not visible in newest version, because database structure is different.
I would recommend waiting for official update that has fixed automatic migrations.


Potential interpolant issue

Hi @Nicholaswogan,
Great work on this repository, it looks like you managed a consistent and quite high speed-up compared to SciPy. I have been working with LSODA recently using solve_ivp, and I have encountered an issue relative to the interpolant (PR at SciPy). I am wondering if this implementation of LSODA also has a similar issue, as it looks to me that the root of the problem lies within the FORTRAN implementation. In particular, LSODA::intdy seems very similar to me to DINTDY in ODEPACK, which could well mean that the issue is present. Let me know if you can have a look.

Thanks! This is interesting and seem hard to pin down and understand. I'll spend a bit of time on it. It might be worth opening an issue at LSODA.jl : https://github.com/rveltz/LSODA.jl , if the issue persists there. They might shed some light on the problem that would help this repository.

I have had a look at LSODA.jl, and I can confirm there also looks to be an interpolant problem there - I will open an issue.


error when nr_type=0 in run_tile.sh with hovernet_fast_pannuke_type_tf2pytorch.tar

Hi,
I get the (truncated) error message below when using the run_tile.sh script currently in github (and attached). It runs the pannuke_type model ... but with nr_type = 0.
I suspect the nr_type = 0 is a typo: When I set nr_type = 6, the script runs to successful completion.
Is nr_type = 6 correct for this model? (I copied it from run_wsi.sh, which uses the same model). In general, would we set this parameter by going back to the description of the corresponding data set to see the number of cell types annotated within it? I understand nr_type must be 0 for segmentation only.
Thank you for making your code available!
Best,
Brian
|2021-02-03|17:33:41.748| [INFO] .... Detect #GPUS: 1
Traceback (most recent call last):
File "run_infer.py", line 181, in
infer = InferManager(**method_args)
File "/pod/2/chuang-lab/USERS/whitebr/hover_net/infer/base.py", line 27, in init
self.__load_model()
File "/pod/2/chuang-lab/USERS/whitebr/hover_net/infer/base.py", line 68, in __load_model
net.load_state_dict(saved_state_dict, strict=True)
File "/home/whitebr/anaconda3/envs/hovernet-gpu/lib/python3.6/site-packages/torch/nn/modules/module.py", line 1045, in load_state_dict
self.class.name, "\n\t".join(error_msgs)))
RuntimeError: Error(s) in loading state_dict for HoVerNet:
Unexpected key(s) in state_dict: "decoder.tp.u3.dense.units.7.preact_bna/bn.weight", "decoder.tp.u3.dense.units.7.conv2.weight",
...
run_tile.sh.txt

Yes, another artifact from my latest debug commit.

Is nr_type = 6 correct for this model? (I copied it from run_wsi.sh, which uses the same model). In general, would we set this parameter by going back to the description of the corresponding data set to see the number of cell types annotated within it? I understand nr_type must be 0 for segmentation only.

nr_type is propagated down to netdesc to create typing branch. Segmentation only usually doesnt concern with typing, but the current mechanism when loading model weight enforce all names in model weight must exist netdesc, hence the above error. Strictly speaking you can change this behavior manually if you know what you doing here https://github.com/vqdang/hover_net/blob/be8ae2d621bfbddefd97591ef9df39252e108df9/infer/base.py#L68.


[Bug] - The Design of the product is'nt correctly aligned

"LIKE" button on the product card is not aligned properly, I'd like to change the alignment and properly place the button correctly. If possible I'l try to include a new design for the card

I can complete this task. Please assign me this.

@codervivek5 , Please assign me this issue under GSSoc'23


Disable Thermostat Clusters attribute tests from Darwin

Problem
Thermostat Cluster Server implements constraints per the spec.
The automated test generation for the cluster generates write values that are outside constraints
blocks https://github.com/project-chip/connectedhomeip/pull/11058
Change overview
What's in this PR
Added excluding of Thermostat Cluster Attribute tests to Darwin zapt template file
Testing
Validated that after changes were made and regen:
All ZAP generated files generated successfully
ChipClusterTests.m did not contain any Thermostat Cluster tests

@chrisbeach59 could we instead modify the tests to write values that are within spec range?

@andy31415  We will need to generate manual test cases. They can not be auto generated.  I have a series of tests that were used to validate the pre-attribute-changed-callback.  Different tests need to be run based on the FeatureMap bits.  The test results.  Right now the scripting engine does not support reading an attribute and using it's value as a condition for other tests.  Also the featuremap remains broken for little endian cpu's.  I have some #defines in the thermostat-server.cpp code to work around this.  (There is a PR in the ZAP project to address this that is still pending)  I am hesitant to add the manual test cases to CI until this is addressed for fear of breaking CI, again.

@vivien-apple FIY.
From my side I am approving based on comments from Chris

/rebase


Chapter code and video are mismatched

The code for Chapter 22 and Chapter 23 are swapped. The Deployment Steps code is included in Chapter 22, while Adding Authentication to React Apps is included in Chapter 23. However, the order is reversed in the video.
Links:
https://github.com/PacktPublishing/React---The-Complete-Guide-includes-Hooks-React-Router-and-Redux-Second-Edition/tree/main/Section 22
https://github.com/PacktPublishing/React---The-Complete-Guide-includes-Hooks-React-Router-and-Redux-Second-Edition/tree/main/Section 23

There is a mismatch between the code in Chapter 21 on GitHub and the code being explained in Chapter 22 "Deploying React Apps" video. The video features the Blog Post service, but the code features the Great Quotes service instead.

Hi,
We had a look at the files. The files related to "Section 21: Authentication" are part of the Section 21 folder, and files related to "Section 22: Deploying React Apps" are available in the Section 22 folder.
We could not find any extra or incorrect files in the Section 23 folder; all the resources are for the video "An Introduction to Next.js"
For easy understanding, we have added the course ToC to the readme file with section numbers and names. Also, we have updated the section 21 and 22 folders with the section names.
Happy Learning!


Autocomplete only works for "*Test" class?

Hi there,
i noticed something which i don't know is working as intended but i find it a little - weird to be honest.
The autocomplete for prophecy or mockobject only works if the class ends on Test.
As seen on this screenshots:

Can't this be working if either:

Class ends on Test
Method has the @test annotation
Method starts with test_ or ends with Test
Class extends PHPUnit\TestCase

I would prefer if it also works if the class is derived from TestCase, because I use Prophecy inside Gherkin Feature Contexts that are not Standard PhpUnit tests.

@CiderAndWhisky @icanhazstring there is a way to check for use statements. What is behind "TestCase" indetail that i can whitelist it. still the phpunit implementation?

@see http://ffishnc.reifinger.de/integration-tests-mit-guerkchen-teil-2/ (German language)
Basically, it is something like this:
class FooContext extends \PHPUnit\Framework\TestCase implements \Behat\Behat\Context\Context
Inside this class, Prophecy is not recognized, as the class is not named FooTest.

extened the test detection, hopefully it somehow matches now ;)

Nais, so we don't have to call the classes "SomethingFeatureContextTest" any more :-D

Nice. Thank you ❤


Remove peerDependencies in all official Leaflet plugins

This came up on an Editable issue: https://github.com/Leaflet/Leaflet.Editable/commit/eaaa6cd
I recommend to get rid of peerDependencies in all official Leaflet plugins.

They provide no value at all, no one is going to install a Leaflet
plugin without Leaflet being available (or they might use Leaflet from CDN, etc.).
The canonical place for version compatibility information is in the
README.md, like it has always been for all Leaflet plugins.
npm actually doesn't do anything useful with peer dependencies, it just
shows that warning that peer dependencies are not installed automatically.
On the other hand, we can get crazy build failure situations like https://github.com/Leaflet/Leaflet.Editable/commit/eaaa6cd8c2f626f19b08d5f5950c57ce971d2dc8#commitcomment-17338411
If Leaflet is required for building a plugin, it should be specified in devDependencies, not peer.

=> I recommend removing peerDependencies in all official Leaflet plugins

I think this is fixed. I went through all of the repositories available here https://github.com/Leaflet and could not find peerDependencies in package.json in any of those.


Add support for ArrowFunctionExpression

Technically it should be enough to simply import and call FunctionExpression and return the result, passing in the same arguments.
http://esprima.readthedocs.io/en/latest/syntax-tree-format.html#arrow-function-expression

Hi. I accidentally come across this issue and choose to ask why use Esprima? No more maintenance on that repo, and the parser is full of bugs. Acorn is way better alternative, or even Cherow if you want to support everyting with insane performance.

Arrow function already work. Unclear what needs to be done.


hplip: 3.17.10 -> 3.17.11

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

No attempt on aarch64-linux
The following builds were skipped because they don't evaluate on aarch64-linux: hplip
No log is available.

Success on x86_64-linux (full log)
Attempted: hplip
Partial log (click to expand)
/nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11/share/hplip/upgrade.py: interpreter directive changed from "/usr/bin/python" to "/nix/store/7b2z0vfbs9539ga4pxx5gmli47rz5y3n-python-2.7.14/bin/python"
/nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11/share/hplip/devicesettings.py: interpreter directive changed from "/usr/bin/env python" to "/nix/store/7b2z0vfbs9539ga4pxx5gmli47rz5y3n-python-2.7.14/bin/python"
/nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11/share/hplip/diagnose_queues.py: interpreter directive changed from "/usr/bin/env python" to "/nix/store/7b2z0vfbs9539ga4pxx5gmli47rz5y3n-python-2.7.14/bin/python"
/nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11/share/hplip/makeuri.py: interpreter directive changed from "/usr/bin/env python" to "/nix/store/7b2z0vfbs9539ga4pxx5gmli47rz5y3n-python-2.7.14/bin/python"
/nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11/share/hplip/faxsetup.py: interpreter directive changed from "/usr/bin/env python" to "/nix/store/7b2z0vfbs9539ga4pxx5gmli47rz5y3n-python-2.7.14/bin/python"
/nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11/share/hplip/__init__.py: interpreter directive changed from "/usr/bin/env python" to "/nix/store/7b2z0vfbs9539ga4pxx5gmli47rz5y3n-python-2.7.14/bin/python"
/nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11/lib/cups/backend/hpfax: interpreter directive changed from "/usr/bin/env python" to "/nix/store/7b2z0vfbs9539ga4pxx5gmli47rz5y3n-python-2.7.14/bin/python"
/nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11/lib/cups/filter/pstotiff: interpreter directive changed from "/usr/bin/env python" to "/nix/store/7b2z0vfbs9539ga4pxx5gmli47rz5y3n-python-2.7.14/bin/python"
checking for references to /tmp/nix-build-hplip-3.17.11.drv-0 in /nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11...
/nix/store/6im0n637qrvqbrq79359fa6z6wx8ii0n-hplip-3.17.11


Ignore YAML-metadata

Many of the popular markdown-implementations use YAML for metadata. Currently, the extension thinks that YAML-metadata should be parsed just like the rest of the text. A perfect implementation would be to parse the YAML-metadata correctly, but a sufficient solution to previewing would be to ignore the YAML completely.

Well, YAML metadata are not standard so I'm not considering it for now because they're not implemented by marked. I'm considering switching to markdown-it because their implementation support plugins, so it'll probably possible to write a yaml plugin then.

:+1: for plugins. Could solve other issues like #22 .


Press space key does not work on focused button element

Are you requesting a feature or reporting a bug?
Bug
What is the current behavior?
Pressing the space key on the focused button (the submit one) does not cause any reaction, but it should be the same as clicking on the button. This wrong behavior appears on Chrome and Edge, but it seems that it works on Firefox 90.0.2
What is the expected behavior?
Pressing the space key on focused button (the submit one) should triggers click event which should redirect the user to "thank-you.html" test page displaying the text: "Thank you, " + inserted name from the input with label 'your name'
Steps to reproduce:

Insert some text into the input with label 'your name'
Click on the checkbox with text ' I have tried TestCafe'
Pressing the tab key three times to focus the 'Submit' button
Pressing the space key to simulate click event
Page should displays the text "Thank you, [insert name from the first input]" but it does not happen

Tested page URL:
http://devexpress.github.io/testcafe/example/
Your complete test code (or attach your test files):

test('Should focus the "Submit" button by pressing tab key from previous element, then pressing space key to simulate click event on the button ', async () => {
    await t
        .typeText(Selector('#developer-name'), 'Nick')
        .click(Selector('#tried-test-cafe'))
        .pressKey('tab tab tab')
        .pressKey('space')
        .expect(Selector('#article-header').innerText).eql('Thank you, Nick!');
});

Your complete test report:

Your Environment details:

testcafe version:                  1.15.0
node.js version:                    v14.16.1
browser name and version:  Chrome Version 92.0.4515.131 and Edge Version 92.0.902.67
platform and version:           Windows 10 Enterprise

Hello, I created a new issue with up-to-date information after you closed the old issue: https://github.com/DevExpress/testcafe/issues/2066 after a prolonged period of inactivity. This functionality is still needed so I hope that you are going to implement it as you promised here: https://github.com/DevExpress/testcafe/issues/2066#issuecomment-360395058. Thank you for the understanding.

We apologize for the inconvenience. This issue is re-prioritized and marked as a bug, so it won't be closed automatically anymore.

This bug should be expanded to include the up and down keys as well, both of which can be defined to open a menu. I can provide documentation and a screen video to illustrate the bug, which seems more nuanced than is described here. I can get space and enter  to open the menu but not shift focus to the first menu item. However, the up and down keys do not even open the menu.

@chundhau, thank you for your feedback. We will take it into account. Also, it would be very helpful if you share a working example that illustrates your usage scenario.

I can still see this issue popup and space button is not even working with following hack.
await t.setTestSpeed(0.01);
await t.pressKey('tab');
await t.setTestSpeed(0.01);
await t.pressKey('tab');
await t.setTestSpeed(0.01);
await t.pressKey('space space space space space space space')
await t.wait(1000)
Please take it on priority to fix it.

Hi @onkarb68
We keep this issue in mind. We will update this thread once we have news. However, I cannot give you a precise date, as any estimate may be misleading.

Pressing buttons with space can be vital for testing accessibility and emulating behaviour of vision impaired people. It would be great if you fix it

Hi @yoksel,
Yes, we know about this issue. We don't have any news yet, but we will update this thread once we have any results.

Hello, the issue with pressKey down occurs also at Firefox 105.
testcafe version: 1.18.4
Firefox version: 105.0.3
URL tested: http://devexpress.github.io/testcafe/example/
test('Should select Windows operating system as primary, press down key to select MacOS', async () => {
await t
.click('#windows')
.expect(Selector('#windows').checked).eql(true)
.expect(Selector('#macos').checked).eql(false)
.pressKey('down') // on FF 105.0.3 this does not move to next radio item, on FF 104.0.1 it does and passes
.pressKey('space')
.expect(Selector('#windows').checked).eql(false)
.expect(Selector('#macos').checked).eql(true);
});

Hi @daimoonis,
It isn't the same issue. However, we already know about this case and fixed it. This fix will be included in the nearest release.

Hello,
The .pressKey('enter') still doesn't work for Firefox.
Testcafe version: 2.0.0
Firefox version: 107.0.1
Platform: Windows 10 Enterprise

It seems that this issue is fixed right now


Background color for default vs current window

I find the background color for the default vs current window a bit confusing. It seems that the current window should have the lighter bg, while the default should have the same bg as the whole line. Instead it's inversed, so if two windows are opened it looks like the non-current window is current.

Indeed it was confusing for me at first. I did not want to mess with the original vision of the author and change the default.
The solution here would be to make the background configurable for default and current window.

I had the same initial feeling but, wondered about configuration options too.
but to be honest, I think it had been a mistake to set the active window to be the same as bg color. It just doesn't make sense.
Even your separator icons won't appear if you do so.
Why would one go to the lengths to make the icons configurable if they are not visible?

I am no specialist here but just gave a bash to change background from $thm_bg to $thm_black4 which they provide and the stars seem to align for me:

Closing as https://github.com/catppuccin/tmux/pull/94 was merged, making this user configurable.


The hardware specifications of the GPUs used in the Performance Table.

Hello, first of all, thank you for your wonderful work.
In the Performance Table, what interface is used for the H100 and A100 GPUs, respectively (PCIe or SXM)?
What is the VRAM capacity for each (40GB or 80GB)?
Thanks.

Thanks for your interest with our work.
The perf data collected is based on SXM and 80GB memory.
Thanks
June

@juney-nvidia Thank you for your quick response.

Close this bug because the issue is resolved. Feel free to ask here if you still have question/issue, we will reopen the issue.


mamba install version

Install via mamba, the latest version is still 5.26.1?

bioconda hasn't updated snakemake, see the PR

thanks @qifei9


KSP generates broken code when using expression body syntax

Hi! Strange that you haven't noticed yet that KSP will generate broken destination code when using expression body syntax for the destination declaration.
Example:
@Composable
@Destination
internal fun EditRitual(nav: RespawnNavigator, ritualId: Uuid? = null) = AnalyticsScreen("EditRitual") {
    EditRitualScreen(nav, ritualId)
}

will generate:
internal object EditRitualDestination : NavigationTypedDestination<EditRitualDestination.NavArgs> {

    override fun invoke(navArgs: NavArgs): Direction = with(navArgs) {
        invoke(ritualId)
    }

    public operator fun invoke(
		ritualId: UUID? = null) = AnalyticsScreen("EditRitual"), // does not compile
    ): Direction {
        return Direction(
            route = "$baseRoute" +
					"?ritualId=${uUIDNavType.serializeValue(ritualId)}"
        )
    }
    // ...

Hi @Nek-12 :)
I cannot think of all use cases, especially when my time is so limited.
I rely on helpful people like you to report these things ahah :D So thanks a lot!
Can you please try replacing the order of the arguments of the Destination? Just curious about something :P
So putting the ritualId first and the navigator second.

Also, Nek it's funny you open this issue today, just as I'm finishing a feature that will allow us to easily wrap multiple screens with common functionality. Sounds like that's what you're doing with the AnalyticsScreen.
So stay tunned for that 😋

Anyway, @Nek-12 consider it fixed, I found the issue and fixed it.
It was related to how we read the default values of parameters. Unfortunately, this is our most "glued together" feature because KSP doesn't have this feature, so we're literally parsing the source code manually, and it seems I missed this use case.
I'm expecting to drop a new release during the weekend :)

Hey, just wanted to let you know that the new wrapper feature is absolutely insane. My destinations are defined so neatly now.
@Composable
@Destination(
    wrappers = [AnalyticsScreen::class],
)
internal fun SelectApp(
    nav: RespawnNavigator,
    backNavigator: ResultBackNavigator<String?>
) = SelectAppScreen(nav, backNavigator.navigator)

Any way to provide an injected dependency (using Compose Destinations dependenciesContainerBuilder) in the wrapped object?

Love the progressive mindblowing. Good job @raamcosta!


Normalize path usage to use forward slashes to make things more platform agnostic

For your consideration, this is a different approach that fixes #89 .
I've changed the relevant filepath calls to route through a helper method which calls filepath.ToSlash to convert them to forward slashes.
The path/filepath pkg will convert slashes back and forth dependent upon the underlying OS so converting them all to forward slashes for the purposes of manipulating them within the application should be okay and also passes the tests!
I think this is a better approach than trying to explicitly handle two different path types throughout the app.
There are a few test failures around concurrency with dynamodb and deleting tables that are still in use.

Here's the latest test run with all the changes in this PR thus far.
test_output.txt

test_output.txt

Almost all of these test failures have to do with concurrent modifications to DynamoDB. We generate a random name for the table and the odds of collision should be very low, and yet, the test output shows multiple tests using the exact same name. Here's the code used to generate a unique ID for the table:
func uniqueId() string {
	const BASE_62_CHARS = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz"
	const UNIQUE_ID_LENGTH = 6 // Should be good for 62^6 = 56+ billion combinations

	var out bytes.Buffer

	random := rand.New(rand.NewSource(time.Now().UnixNano()))
	for i := 0; i < UNIQUE_ID_LENGTH; i++ {
		out.WriteByte(BASE_62_CHARS[random.Intn(len(BASE_62_CHARS))])
	}

	return out.String()
}

Perhaps time.Now().UnixNano() doesn't work properly on Windows and returns the same seed for everyone?

BTW, this is an interesting approach! Is the idea here to replace all path separators with a "/"? I assume Windows can handle forward slashes as well as backslashes when loading things from the file system?

BTW, this is an interesting approach! Is the idea here to replace all path separators with a "/"? I assume Windows can handle forward slashes as well as backslashes when loading things from the file system?

Effectively, yes. I did some testing and have had no issues in windows with both slashes. Additionally, the path/filepath pkg in Go does the OS mangling for you already anyway. The path pkg on its own does not.

Effectively, yes. I did some testing and have had no issues in windows with both slashes. Additionally, the path/filepath pkg in Go does the OS mangling for you already anyway. The path pkg on its own does not.

Awesome. In that case, I think this approach makes sense. The only thing left to figure out is why the DynamoDB stuff is failing, including even a panic in some cases. Do those tests pass if you run just one of them? E.g. go test -v -run TestAcquireLockHappyPath.

Awesome. In that case, I think this approach makes sense. The only thing left to figure out is why the DynamoDB stuff is failing, including even a panic in some cases. Do those tests pass if you run just one of them? E.g. go test -v -run TestAcquireLockHappyPath

They seem to, yes. So this definitely has something to do with the parallelization of the tests. I'll take a look at the uniqueid function.

Fixed!
I changed the way the seeding of rand works so it only initializes it once and things now seem to work swimmingly.
This fixes ALL the tests on Windows now.
test_output.txt

@cstavro You rock. This is awesome work! Thank you!
Merging now :)

Just created a new release here: https://github.com/gruntwork-io/terragrunt/releases/tag/v0.9.0. If the build passes, the new binaries should show up in a few minutes.


NEW ALGORITHM

I want to add new allgorithm as well as update few old ones as they are not running

Please assign this to me @AlgoGenesis


Adding additional fields for bindings and validation.

Support additional fields in ServiceRoleBinding as described in #11516.
Add checks to make sure no duplicated definition exists (e.g. a value in ports and destination.port).
Disallow users from using * as a value in names (list of subject.User (deprecated)) or not_names, since using * for source.principal and subject.User means differently. Current subject.User overrides source.principal if the user defines them both at the same time. Use allUsers and allAuthenticatedUsers (naming borrowed from IAM) instead, as discussed with @liminw.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: pitlv2109
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [pitlv2109]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@yangminzhu @liminw @diemtvu I added some TODOs to remove some old code. Let me know what you guys think.

@pitlv2109: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-integ-k8s-tests

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.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-integ-k8s-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-integ-k8s-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-multicluster-e2e

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.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-integ-k8s-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-local-tests.sh
4d894fad598fd9605440de3afbed5b57f105bb73
link
/test istio-integ-local-tests

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.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-local-tests.sh
4d894fad598fd9605440de3afbed5b57f105bb73
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
4d894fad598fd9605440de3afbed5b57f105bb73
link
/test istio-integ-k8s-tests

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.

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-local-tests.sh
4d894fad598fd9605440de3afbed5b57f105bb73
link
/test istio-integ-local-tests

prow/istio-integ-k8s-tests.sh
4d894fad598fd9605440de3afbed5b57f105bb73
link
/test istio-integ-k8s-tests

prow/istio-presubmit.sh
4d894fad598fd9605440de3afbed5b57f105bb73
link
/test istio-presubmit

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.

Looks like most CircleCI tests are suddenly failing this morning with error

Error: Looks like "https://storage.googleapis.com/istio-prerelease/daily-build/master-latest-daily/charts" is not a valid chart repository or cannot be reached: Failed to fetch https://storage.googleapis.com/istio-prerelease/daily-build/master-latest-daily/charts/index.yaml : 404 Not Found

This error is affecting some other recent PRs, too. Where do we check the status of daily charts build (from the URL, looks like we have daily charts build from master)?

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
d7679415f709d38a16a073b7fdacc33c85a9f7dc
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
8f548d279673cacc0288926d5c40117ff8406b3d
link
/test istio-integ-k8s-tests

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.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: pitlv2109, yangminzhu
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [pitlv2109,yangminzhu]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pitlv2109: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
8f548d279673cacc0288926d5c40117ff8406b3d
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
8f548d279673cacc0288926d5c40117ff8406b3d
link
/test istio-pilot-multicluster-e2e

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.


[bug] Cannot build on DeckOS(Arch Linux)

Describe the bug
The prerequisite setup for Arch Linux results in an error on the cargo create-tauri-app step. After installing all Linux pre-reqs and running create-tauri-app, I was presented with this message:
Your system is missing dependencies (or they do not exist in $PATH):
webkit2gtk & rsvg2 │ Visit https://tauri.app/v1/guides/getting-started/prerequisites#setting-up-linux

Reproduction

Open terminal
Install system dependencies with pacman
Run cargo install create-tauri-app

Expected behavior
The script should finish running without reporting missing dependencies.
Platform and versions
create-tauri-app version 3.3.0
cargo 1.68.2
rustup 1.25.2
rustc 1.68.2 (9eb3afe9e 2023-03-27)

Stack trace
No response
Additional context
There weren't any install errors during the system library  installations.

Did you mean SteamOS for the steam deck? I can't find a DeckOS arch derivative 🤔
Also, is only create-tauri-app complaining or does cargo tauri build also not work?
And lastly, do you have homebrew installed? There are known issues with that, see https://tauri.app/v1/guides/faq#build-conflict-with-homebrew-on-linux

Yes, SteamOS, sorry if that wasn't clear. As I understand it, the distro is mostly a modified version of arch(pacman works). I do use linuxbrew and I wasn't aware there is a conflict, so thanks for linking that part of the FAQ, I'll take a look at the proposed solutions.
As for cargo tauri build, I actually can't run it due to needing to install tauri-cli which fails to install because the linker can't find stdint.h. I'd chalk that up to an issue with this particular distro not being intended for development, so I'll close this issue as it seems to be an issue with the platform I'm using and not Tauri.
Thanks for your help!


Debug not working?

Hello - trying to get a simple app up and running and I cannot for the life of me get anything to log.
I've use the default counter example,and tried:

console.log('hi')
Set streamDeck.logger.setLevel(LogLevel.DEBUG);
streamDeck.logger.debug('hi'); and all other log levels
Custom "Debug": "--inspect=<IP_ADDRESS>:12345" in the manifest.json file - chrome does not recognize the inspect instance running.

I just want to see a string in my terminal when I press a button and no combination of the above seems to work.
I'm on Node v23.1.0
I've read the docs through but can't seem to get it working. Am I missing something? Or a bug?

I should note I do see log files being created in the logs folder, but I can't imagine thats the only way to view them? Doesnt seem to log objects either

Hey Wes, the streamDeck.logger console target is currently only available whilst the plugin is being debugged. This is something we'd like to improve, but for now, you should be able to see them via either:
Chrome

Specify a debug port within the manifest ("Debug": "--inspect=<IP_ADDRESS>:12345" is perfect).
On Chrome's inspect page (chrome://inspect/#devices), configure "Discover network targets" to monitor localhost:12345.
Your plugin should show under the "Remote targets", under "Target (v20.18.0)". (this is the version of Node.js that ships with Stream Deck).

VSCode

With your plugin running, press Ctrl / Cmd + P
Search for > Attach to Node Process.
Select the node20 instance that matches your plugin.

In relation to logging objects; the logger is currently capable of logging objects, but utilizes JSON.stringify so does come with some limitations. If you're seeing odd behaviour here, I'd be happy to investigate further — would it be possible to get a code snippet of the object you're wanting to log, and how you're attempting to log it?

Hrmmm Yeah I tried all those things.  Let me make a simple counter example and try again.

...additionally to what @GeekyEggo already mentioned, you can also combine any of the above like so:
export const wrappedLogger = (prefix) => {
	const logger = streamDeck.logger.createScope(`${prefix}`);
	const dbgFn = (...args) => {
		logger.info(wrapMessage(args));
                console.log(...args);
                ...
	}
	return dbgFn;
};

where wrapMessage is your privat stringify method...
If you have large JSON-objects Chrome's console has some nifty features (as you already know)

Okay! Both your suggestions worked. Thank you - I had just expected that the console.log would work in the terminal, but the chrome/VS code logger is a better experience anyways.
Thank you for the quick help - and great job on this SDK!