Incorrect error code in CERTIFICATE and DIGESTS

If a responder has advertised that it supports certificates (CERT_CAP == 1) but is unable to obtain the certificates, say due to the corruption of non-volatile memory, then it returns UnsupportedRequest to the GET_DIGESTS and GET_CERTIFICATES requests. In the absence of a better error code it should be Unspecified.
Note that if the responder detects the issue before the CAPABILITIES response, then it can set CERT_CAP == 0, in which case if the requester attempts GET_DIGESTS or GET_CERTIFICATES, then the responder responds with UnsupportedRequest, which is the correct error code.

Approved.

https://github.com/DMTF/libspdm/blob/bc1d1468940defd45645d2b46a097f61ff6f17f4/library/spdm_responder_lib/libspdm_rsp_digests.c#L78-L89
is a runtime check for a condition that shouldn't exist. The integrator needs to populate the Responder's certificates before allowing it to respond to GET_DIGEST requests. This should probably be an ASSERT.

Although if we support https://github.com/DMTF/libspdm/issues/313 then the error response might not arise until the GET_DIGESTS request. I'll just change the error code for now.


Tips overlaid on elements cause flickering

Where a tip actually overlay elements, flicker occurs when hovering over portion of the tip overlaying the element. Presumably this is down to mouseover events being thrown on the tip itself and causing a mouseout on the element to which the tip is applied. I've had a look at issues 42 and 32 and the pointer-events:none solution doesn't resolve it.
I'm using chrome 32.01 but I'm seeing this issue in safari 7.01 and firefox 26 too. I'm using d3-tip v0.6.3 according to bower.
Here's an example (modified from previous ones): http://jsfiddle.net/6wGZB/1/

This worked for me so long as there is no space between none and !important.
So the CSS line would be pointer-events: none!important;


How to set CMAKE_INSTALL_PREFIX?

I have tried:
mkdir build
cd build
mkdir ./INSTALLED
CXX=clang++ CC=clang cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX="$(pwd)/INSTALLED" ..
CXX=clang++ CC=clang cmake --build .

But it didn't work and I wasted lots of time (./build/INSTALLED is empty).
My architecture is Linux amd64.

I have found libtdjson.so in the ./build, seems like what I need.
My first attempt was with -DCMAKE_INSTALL_PREFIX="$(pwd)" so I had to run something like cmake clear before trying second time.


What is the principle and intuition of unsupervised embedding of SGPT?

@Muennighoff Thank you very much!

Principle: Use the probability that the LLM assigns to the words in a document given the query as the score of that doc
Intuition: It's like asking a human how likely is it that this document matters to this query


refactor: move natural isomorphisms involving inverses of equivalences

Move all results from CategoryTheory/Functor/InvIsos.lean to CategoryTheory/Equivalence.lean and delete the former file
Replace use of eqToIso (Functor.comp_id G) with G.rightUnitor
Move into Iso namespace to enable dot notation
Add analogous results stated in terms of Equivalence instead of IsEquivalence

I have to admit that I'm not up to date on the discussion regarding renamings and deprecations. Do I have to add some kind of align statement or deprecation notice due to moving the definitions from Mathlib.CategoryTheory to Mathlib.CategoryTheory.Iso?

It seems that the syntax for deprecation would be like:
@[deprecated] -- since 2024-02-06
alias compInvIso := Iso.compInvIso

Thanks!
bors merge


[Dashboard][Share] Share related links are gone

SHARE links (share url, share in email etc) are hidden in slice header dropdown list in dashboard:
Screenshots

How to reproduce the bug

Go to any dashboard
Click on dropdown list for any chart
Should be able to share chart by link, by email

Environment
latest master
Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Related to https://github.com/apache/superset/pull/14076. According to PR’s test description:
check that sharing is menu option is shown by default for Admin/Gamma/Alpha
But admin/alpha/gamma role didn't see share links.
CC @junlincc @zuzana-vej @amitmiran137

I'm unable to reproduce the issue
my steps:

make sure/remove from Admin/Alpha/Gamma 2 permissions named:
a. can share dashboard on superset
b. can share chart on superset
run superset init
check Admin/Alpha/Gamma and see that the above permission does exist

anyone else able to reproduce?

I can reproduce this way:

checkout open source master branch,
build and compile in your local environment,
open any sample dashboard as Admin role.

Hi @amitmiran137 and @junlincc I found what is wrong here: in airbnb we reverted #13306 and #14148. But actually #14148 fixed a few permission related issues in the original PR #14076...So right now we are running with 14076 without extra permission related fixes. :(

@graceguo-supercat gotcha, glad that you found the cause. not sure what's the best way to fix this issue, but this issue will be added to our test case incase it actually happens. closing issue for now since it's not universal.

Testcase added


Submit error

I have complete "Maximum Length Difference" task. Than I press "Run Test", and receive:
test(MaxDiffLengthTest)
mxdiflg Fixed Tests
Test Passed
But when I press "Submit" I get:
test(MaxDiffLengthTest)
mxdiflg Fixed Tests
0
test1(MaxDiffLengthTest)
100 Random Tests
Test Passed
What does it mean? I have already pass 1st test, but than I get "0".

I'm not following. "Run Test" usually has a smaller set of tests whereas "Submit" runs against the full suite of tests. Is that the issue or is something else happening? Screenshots would be helpful.

I already get it, and complete this task. Thank you


Multi-trait equivalent of Manhattan plot

TBD. Reference: https://github.com/opentargets/postgap-app/issues/54

Closing as duplicate of https://github.com/opentargets/genetics/issues/113.


proptest transactions with randomly generated GasData

Description
Describe the changes or additions included in this PR.
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Updated gas data generation to respect max gas price and max gas budget. @sblackshear


Code Compiler added

Description
i have added html,js,sercer.js and manifest.json file for this
also added reame.md file for contributors and user to how to run this extension
beacause this project using server so server should be run before loading extension
i have mentioned required steps also
used jdoogle api for compiler
Fixes:  #776

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have made this from my own
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
working video of compiler extension
https://github.com/Sulagna-Dutta-Roy/GGExtensions/assets/127788321/da9727b0-4d07-40b2-bbdf-15bb1241fbfd

@Sulagna-Dutta-Roy
added code compiler


Add BL5340PA

This differs from the rebase onto 2.6.1 because of hardware model v2.

I recommend adding a pa variant of the BL5340 board to reduce maintenance and all the duplicate files.

Just flagging that zephyr accidentally broke the display on the bl5340 when they did MIPI DBI changes,
You've copied the breaking change here.
The mipi_dbi interface spidev should be spi4, not spi2.
It was accidentally? changed from spi4 to spi2 when they updated the bl5340 to use a mipi dbi wrapper.
See https://github.com/zephyrproject-rtos/zephyr/pull/64587#issuecomment-2229337258
(I tested bl5340+lvgl and bl5340pa+lvgl before and after)


SDK: Proxy commands generated with  [System.Management.Automation.ProxyCommand]::Create() quietly modify the -OutBuffer common parameter on invocation

When you create a proxy command, the begin block contains the following snippet:
       $outBuffer = $null
        if ($PSBoundParameters.TryGetValue('OutBuffer', [ref]$outBuffer))
        {
            $PSBoundParameters['OutBuffer'] = 1
        }

That is, if a -OutBuffer value is passed, it is quietly reset to 1, which raises two questions:

Why does the -OutBuffer value need to be reset at all for the steppable pipeline created by the proxy (wrapper) command?

Assuming the intent is to reset the value so as to perform NO buffering, why is the value set to 1, which implies paired output (it is -OutBuffer 0 that specifies no buffering)?

Environment data
PowerShell Core 7.0.0-preview.4

I don't know why but I remember this when I first used this to create proxy functions. I have one which adds "-recurse" to select string, which was last modified in 2015, and has this code in it
           # For proxy functions using stepable pipeline Buffer should always be 1
           $outBuffer = $null
           if ($PSBoundParameters.TryGetValue('OutBuffer', [ref]$outBuffer)) {
                $PSBoundParameters['OutBuffer'] = 1
           }

So whatever the logic, it isn't new :-)

@SeeminglyScience do you happen to know anything about this little oddity? 😊

No idea.  I always wondered why:

It doesn't use ContainsKey if it isn't even going to use $outBuffer (maybe that API didn't exist at the time?)
Why do it at all
Why set it to 1

Honestly I generally just remove it.  I don't think I've ever actually used OutBuffer for anything though, so ymmv.

Thanks, @jhoneill - indeed it isn't a new issue, but I was surprised to see the comment in your code snippet - was that auto-generated? If you use [System.Management.Automation.ProxyCommand]::Create() now, this comment is not there anymore.
@SeeminglyScience:
As for .ContainsKey(): the proxy-command class dates back to PSv2, but even then $PSBoundParameters implemented the generic IDicationary<T,T> interface, so I believe that the method was available.
Good point about removing it - it does seem to work OK without it, even when you do use -OutBuffer, that rare best.
Here's my guess as to what happened: The generated code is based on:

a general misconception: since -OutBuffer only applies to the output side and the receiving cmdlet needn't be aware of whether the one providing the input used buffering or not - it still receives the objects one by one, there is no need to modify -OutBuffer at all; and - correct me if I'm wrong - on the output side the buffering is handled automatically by the plumbing.

a specific misconception (which is a moot point): -OutBuffer 1 sends 2 objects at a time through the pipeline (which is an understandable misconception and an unfortunate original design decision for -OutBuffer - too late...)

@SteveL-MSFT:

Do you know anyone who can speak to the original design intent?

Do we feel comfortable removing this part of the generated code? While it is typically benign, it actually thwarts the intentional use of -OutBuffer.

Thanks, @jhoneill - indeed it isn't a new issue, but I was surprised to see the comment in your code snippet - was that auto-generated? If you use [System.Management.Automation.ProxyCommand]::Create() now, this comment is not there anymore.

I don't think any comments are (or were) auto-generated, so it must have been my comment .
I don't understand why it has / always had
    try {
        $steppablePipeline.Process($_)
    } catch {
        throw
    }

Putting throw <> in a catch block doesn't seem very useful to me

I don't understand why it has / always had

Yeah.  I always replace throw with $PSCmdlet.ThrowTerminatingError($PSItem) personally.
Also inconsistent brace, inconsistent white space, and almost always unnecessary variable name escaping. (I think you fixed some of that recently though right @vexx32?)

Good point, @jhoneill; to add to @SeeminglyScience's comment:
While argument-less throw does preserve the original error record (exception details), it converts a statement-terminating error into a script-terminating (runspace-terminating) error - so the wrapper (proxy) function will in fact behave differently than the wrapped cmdlet: it will not just terminate itself, but the entire runspace (unless caught).

cc @JamesWTruher

The OutBuffer code is to help avoid a DOS attack. Proxies are typically generated on servers for RBAC and we didn't want an attacker to easily create large buffers by specifying a large OutBuffer. I wish I could say I remembered this (I added the code), but I had to check my email archives.
IIRC, the try/catch/rethrow was added so errors with a proxy looked as much as possible like they did without the proxy. Any other benefit/downside was unintentional - at least on my part (that was also a change of mine.)

Thanks, @lzybkr, but given that DOS isn't around anymore ... (just kidding).
So it sounds like the following changes are worth making:

Streamline the OutBuffer code and precede it with a comment; e.g.:

# Ignore -OutBuffer arguments to prevent DOS attacks.
# If that is not a concern and you want -OutBuffer support, remove this `if` statement.
if ($PSBoundParameters.ContainsKey('OutBuffer')) { $PSBoundParameters.Remove('OutBuffer') }

Note that the code now removes the OutBuffer  value (tantamount to setting it to 0, i.e. no buffering), which I presume was the original intent.

In the spirit of so errors with a proxy looked as much as possible like they did without the proxy, replace the throw statements in all catch blocks with $PSCmdlet.ThrowTerminatingError($_) as suggested by @SeeminglyScience

A comment was not included largely for performance reasons - the generated code is mostly not meant for humans and Exchange generates many proxies.

I see. Leaving the comments issue aside (see below), do you agree with the proposed changes?

The reason I asked for comments is that I had a different use case  in mind - and I know that this will eventually have to be a separate discussion, but just to get the ball rolling:
End users too can benefit from proxy functions - they're a powerful way to customize existing cmdlets.
For instance, I have a hd function in my profile  (similar to the Unix head utility) that wraps Select-Object -First <n>.
In this scenario, comments are important, because the generated code does require tweaking by humans.
@jpsnover  originally introduced the concept in this blog post from 2009 and even published a PSGallery module with a New-ProxyCommand function - thought it didn't gain much traction, possibly in part because it is rough around the edges.
I think better support for this use case is well worth considering, possibly even with an official cmdlet.

The code changes sound reasonable.
I think it's possible to generate a comment without hurting Exchange like scenarios. If the process is interactive (there are some existing checks already), it seems safe to assume PowerShell isn't being used in a server scenario where performance is more important.


Export UNDEFINED_REFERENCE from glimmer-runtime

Exporting the UNDEFINED_REFERENCE so that it can be used inside of Ember.

Can you export NULL_REFERENCE as well? If it's not already defined in the references package, you can move it.

Also, I just realized this: in https://github.com/tildeio/glimmer/blob/master/packages/glimmer-runtime/lib/references.ts#L8, it probably makes more sense to return an UNDEFINED_REFERENCE now that we have one.

@chancancode Sure, will send another PR :)

PR for NULL_REFERENCE: https://github.com/tildeio/glimmer/pull/157


How to evaluate transformer explainability methods & Compare with other XAI methods

Hey @hila-chefer ,
Congratulations on this great work

Could you pls suggest the evaluation framework for XAI methods for vision transformers?
is there any wrapper in the existing code base for other XAI methods like Integrated gradients/Smoothgrad etc?

Best,
@jaiswati

Hi @jaiswati, thanks for your interest!

I'm not familiar with a framework that includes all methods.
We didn't implement Integrated gradients/Smoothgrad in this work, but all the baselines in the paper are reproducible in this repo (our variation of GradCAM, LRP, partial LRP, raw attention, and rollout).

I hope this helps,
Hila.


FancyZones: When editing, I cannot select edges of overlapped zones

I use a 43" 4k monitor and I'm excited with the prospect of managing my windows with FancyZones. I'm a developer and like to have my main windows large. These main windows vary depending upon what I'm doing, but include IDE's, browsers, office apps, etc. Other windows are typically floating and are fine.
I created a 3 column layout with the middle column wider than the left and right ones - this gives the centre as my main focus. Unfortunately I had to be careful with the ORDER I created them as otherwise I couldn't select the edges of the columns obscured by the centre column.
Creating them Left, Right, Centre meant I couldn't grab all edges. I had to create them Left, Centre, Right.
The editor should allow me to pick any zone edge regardless of z-order.
Versions

PowerToys: v0.18.1
Windows 10 Enterprise 1903, build: 18362.778

Hi @decani
to clarify, in the editor when you select a zone, all four edges can be grabbed to resize it, regardless the creation order.
Or are you asking for a way to make the zone active when dragging a window to the edge of the zone?

No, I cannot interact with a zone obscured by other zones; see attached animated GIF. Notice towards the end that I cannot adjust Zone 3 as it is completely obscured by Zones 1 and 2. I want to make Zone 3 full-height and occupying the centre. I should be able to interact with any zone, regardless of its z-order.

Thanks for the clarification.
The tracking issue for this problem is https://github.com/microsoft/PowerToys/issues/821
Feel free to add you comments there.
I'm closing this one as duplicate.
Thank you.


Update to export-hook 1.0.2 and fix #73

Fixes #73 (and adds a test to verify), so there's now no issue with recursive nesting of types with a mix of derived and non-derived instances:
scala> import io.circe._, io.circe.jawn.decode, io.circe.generic.auto._
import io.circe._
import io.circe.jawn.decode
import io.circe.generic.auto._

scala> case class Foo(o: Option[Foo])
defined class Foo

scala> decode[Foo]("{}")
res0: cats.data.Xor[io.circe.Error,Foo] = Right(Foo(None))

scala> decode[Foo]("""{ "o": null }""")
res1: cats.data.Xor[io.circe.Error,Foo] = Right(Foo(None))

Will update, merge, and publish a new snapshot as soon as export-hook 1.0.2 is published.

:+1: for the removal of the way Priority was used. It wasn't intended to be used this way, only when the return type of the implicit method at hand was the same as the first type arg of Priority (like here).

Current coverage is 68.63%

Merging #77 into master will decrease coverage by -0.10% as of b75f1bc

Powered by Codecov. Updated on successful CI builds.


Don't depend on ffmpeg-full

Issue description
In the process of cleaning up ffmpeg and merging the ffmpeg and ffmpeg-full derivations, I have decided that uses inside Nixpkgs should use the regular ffmpeg; ffmpeg-full being reserved for advanced users to install themselves due to its significantly larger closure:
https://github.com/NixOS/nixpkgs/blob/e2c4c9eafd27af8e5c5e4706834368eda9235470/pkgs/development/libraries/ffmpeg/generic.nix#L7-L22
However, some packages depended and still depend on ffmpeg-full and I have not touched those. This is about to change. I've compiled a list of packages that I will rip ffmpeg-full out of:

liquidsoap
corrscope
obs-studio
vokoscreen
ffcast
escrotum
audiobookshelf
audiobookshelf
jellyfin-ffmpeg
ffmpegthumbnailer
handbrake
imagination
haruna
video-trimmer
peek
restream
tone
tonelib-gfx
tonelib-jam
tonelib-zoom
tonelib-metal
tonelib-noisereducer
MIDIVisualizer
arcanPackages

Of course, some usages of ffmpeg-full might have good reasons behind them that I'm not aware of.
 Please reply why you need ffmpeg-full or, respectively, why regular ffmepg is not enough for your needs such that we can make it fit your needs.
We might also allow some exceptions for ffmpeg-full usage should you require an ffmpeg feature that comes with a huge closure size.
Please either shoot a reply with the reason or react with a rocket if your package doesn't actually require ffmpeg-full. (Or remove yourself from its maintainers if you no longer intend to maintain it.)
While you're at it, please also check whether your package requires ffmpeg's headed playback/capture support (i.e. xorg, wayland, SDL, pipewire) or whether the headless variant is enough. Except for these headed components, ffmpeg-headless should be the exact same.
Additionally, if your package is pinned to i.e. ffmpeg_4 or ffmpeg_5, please also check whether your package actually still requires the older ABI.
You have one month to reply. After that I'll remove ffmpeg-full usages; whether it breaks your package or not.
Maintainer ping
@AndersonTorres @Anton-Latukha @austinbutler @cpcloud @dali99 @dan4ik605743 @deviant @doronbehar @ehmry @ericdallo @jagajaga @jb55 @jojosch @justinas @jvanbruegge @kashw2 @league @materusPL @MP2E @OPNA2608 @orivej @puffnfresh @rasendubi @sikmir @wmertens

Arcan: https://github.com/NixOS/nixpkgs/pull/270301

corrscope: Uses ffplay for export preview, which depends on withFullDeps.
https://github.com/NixOS/nixpkgs/blob/e2c4c9eafd27af8e5c5e4706834368eda9235470/pkgs/development/libraries/ffmpeg/generic.nix#L131

@OPNA2608 thanks. I see no reason why we shouldn't enable ffplay in the small variant. I'll take care of that :)

https://github.com/NixOS/nixpkgs/pull/271876

@doronbehar thanks. Though as I mentioned, I will be moving all uses where the maintainers didn't object anyways ;)

Regarding jellyfin-ffmpeg: I went over what ffmpeg --version in their official Docker images reports being configured with, and summarized below in the expandable table.
A few libraries seem to be missing altogether from Nix's FFmpeg, although some of them might not be essential for normal use of Jellyfin.
Jellyfin should not need the "head" support (so we could build without FFplay, Xcb, etc.), but  it does need some stuff that relates to graphics accelerators (CUDA, OpenCL, etc.) to facilitate hardware-accelerated transcoding.

With cuda we'll have to see how much it bloats closure size. Is it actually required though? HW transcode should only require cuvid, no?
I don't think you need opencl to HW transcode either.
Most of the other stuff is either unimportant, should be enabled in headless too or is proprietary AFAICT.
We could also think about making jellyfin's fork a regular variant of the ffmpeg package.

With cuda we'll have to see how much it bloats closure size. Is it actually required though? HW transcode should only require cuvid, no?
I don't think you need opencl to HW transcode either.

Jellyfin's hardware acceleration covers all the major GPUs, and OpenCL is one of the options for HDR-to-SDR tonemapping on Intel and AMD hardware (I was previously told to add opencl for this reason).
CUDA was also previously requested by the upstream maintainers, and seems to be used for tonemapping, subtitle burn-in, etc.
@nyanmisaka could probably answer any jellyfin-ffmpeg question better than me :slightly_smiling_face:  (hope they don't mind the mention).

With cuda we'll have to see how much it bloats closure size. Is it actually required though? HW transcode should only require cuvid, no?
I don't think you need opencl to HW transcode either.

Jellyfin's hardware acceleration covers all the major GPUs, and OpenCL is one of the options for HDR-to-SDR tonemapping on Intel and AMD hardware (I was previously told to add opencl for this reason).
CUDA was also previously requested by the upstream maintainers, and seems to be used for tonemapping, subtitle burn-in, etc.
@nyanmisaka could probably answer any jellyfin-ffmpeg question better than me 🙂 (hope they don't mind the mention).

CUDA and OpenCL are act as the off-screen 2D image processor to run the programmable kernel/shader to modify (pixel format conversion/HDR tone-mapping/subtitle-burn-in) the HW decoded video before sending it to the HW encoder. Imagine not having OpenGL on a Linux desktop, or DirectX on Windows, it would be a disastrous GUI experience.

@dali99 Do usages of libquidsoap also include interacting with headed components (SDL, pipewire, xorg etc.)? If so, I think it should depend on the regular ffmpeg by default with an option to use ffmpeg-full.
Enabling celt, webp etc. is a no-brainer as long as they don't bloat closure by many megabytes.

@nyanmisaka thanks for the input.
Given that jellyfin appears to have put in a lot of work specifically to make heavy use of accelerated processing, I think a dependence on a fully-featured variant is justified. We'll either make an exception or introduce a jellyfin/hwaccel ffmpeg variant (jellyfin-ffmpeg uses a forked source anyways).
(You can unsubscribe from this thread now if you'd like, we'll ping you directly should further questions arise.)

liquidsoap definitely has functions to just "output" audio to ALSA, JACK, etc. This functionality is commonly suggested to be used during development and testing of liquidsoap pipelines. Now whether that requires headed components, I can’t really say...

@nagisa in that case, I think regular ffmpeg is warranted. Advanced users can override it with ffmpeg-full if they need it.

We might also be able to enable opencv in regular ffmpeg if we switched opencv to ffmpeg-headless. I'm unsure whether opencv needs headed components though. If anyone knows, let me know.

There's also escrotum (maintained by @rasendubi). It uses plain ffmpeg command for screen recording, so I assume that regular ffmpeg will work as well

I've compiled a list of packages that I will rip ffmpeg-full out of

tonelib-* packages do not depend on any variant of ffmpeg.

Okay, so life got in the way but with a year of delay, I think we can finally do this.
Our ffmpeg now has AV1 encoders, webp support, the ffplay binary and will have OpenCL support soon.
If you need any feature from ffmpeg-full that our current non-full ffmpegs currently do not offer, please speak up!
Our only constraint is closure size here which is always weighed against usefulness and may not even increase by much.
I'm planning to remove all usages of ffmpeg-full from Nixpkgs after the 24.11 release. I'll perhaps also look into making ffmpeg-full an alias such that it will throw an eval error when used within Nixpkgs to prevent future usages.
If you absolutely must have ffmpeg-full for some reason, please speak up about that reason so that we can fix it or find a workaround.

We still need cudaLLVM for frigate with nvidia hwaccel. https://github.com/NixOS/nixpkgs/issues/344114
If that is not in the cards, then we must provide ffmpeg-full to users who configure nvidia hwaccel.

I included it in https://github.com/NixOS/nixpkgs/pull/354952.


Fix displaying comment count element

Changes
Summary of changes you made.

Comment count element was visible event if there was no any comments. I've changed it to display it only when comments count is greater than 0.

Test plan
You can use my profile to check changes.

Check and remember space between comment icon and share icon
Run local version with this PR

Demo
pre

after

Thank you for your contribution, we need the cursor pointer here too:

Okay, just doing it


[Dispatcher] Add a query for the TVL-to-APR scale and update the init scale

Currently, the scale passed at the dispatcher contract init is only used internally for calculations. For the sake of UI, we need it to be queried. The subtasks here are:

add a query that returns the scale
update the initial value of the scale with the one provided here below.
update the multiplication factor which is currently x1000, i.e. in mini-LPNs,  to x1000x1000x1000, i.e. in hundreds-LPNs

<500k 15%
500k - 1m 14%
1m - 2m 13%
2m - 3m 12%
3m - 4m 11%
4m - 5m 10%
5m - 7.5m 9%
7.5m - 10m 8%
10m - 15m 7%
15m - 20m 6%
20m - 25m 5%
25m - 30m 4%
30m - 40m 3%
>40m 2%

Closed by 607eaa04264fb9f8b9bc03fa505cdf47f162aa56, 2e6912e8b86fbf6785868af55af677a8f28cc2b6 and d5cffe94f0cf04ca665a92ee8988359b7aebd604


Explanation of branches

Is master the only branch to track here? VCVRack doesn't have branches like this, it just has 0.5 and 0.6 (as far as I can see).
Could you please explain what the three branches or git management is in this repo?

I use branches the traditional way - for developing features that then will get (or not) merged into master. So basically you can ignore other branches most of the time.

Feature branches. Thanks


Fail to download new illust bookmar_new_illust.php

First thanks for your hard work,
then here is the issue:
fails to download new illust after downloading a few ones (2times tested and failed randomly after 5-10 download)
<<<<8. Download new illust from bookmarked members (...)
start page : default
end page : 41 (pages of the latest submission, manually checked on browser)>>>>
multiple errors shown in log but here is the last Message in log:
****2017-04-11 12:15:16,516 - PixivUtil20170411 - ERROR - Unknown Error: ''
Traceback (most recent call last):
File "PixivUtil2.py", line 1876, in main
File "PixivUtil2.py", line 1671, in main_loop
File "PixivUtil2.py", line 1538, in menu_download_new_illust_from_bookmark
File "PixivUtil2.py", line 1081, in process_new_illust_from_bookmark
File "PixivUtil2.py", line 682, in process_image
File "PixivUtil2.py", line 161, in download_image
File "mechanize_mechanize.pyc", line 199, in open_novisit
File "mechanize_mechanize.pyc", line 230, in _mech_open
File "mechanize_opener.pyc", line 193, in open
File "mechanize_urllib2_fork.pyc", line 344, in _open
File "mechanize_urllib2_fork.pyc", line 332, in _call_chain
File "mechanize_urllib2_fork.pyc", line 1170, in https_open
File "mechanize_urllib2_fork.pyc", line 1116, in do_open
File "httplib.pyc", line 1074, in getresponse
File "httplib.pyc", line 415, in begin
File "httplib.pyc", line 379, in _read_status
BadStatusLine: ''
****2017-04-11 12:49:09,882 - PixivUtil20170411 - ERROR - Error at process_new_illust_from_bookmark(): (<type 'exceptions.UnicodeDecodeError'>, UnicodeDecodeError('utf8', 'URLError: <urlopen error [Errno 10054] Une connexion existante a d\xfb \xeatre ferm\xe9e par l\x92h\xf4te distant>\n', 66, 67, 'invalid start byte'), <traceback object at 0x036CE7D8>)
Traceback (most recent call last):
File "PixivUtil2.py", line 1081, in process_new_illust_from_bookmark
File "PixivUtil2.py", line 682, in process_image
File "PixivUtil2.py", line 239, in download_image
File "traceback.pyc", line 128, in print_exception
File "traceback.pyc", line 13, in _print
File "win_unicode_console\streams.pyc", line 254, in write
File "encodings\utf_8.pyc", line 16, in decode
UnicodeDecodeError: 'utf8' codec can't decode byte 0xfb in position 66: invalid start byte
2017-04-11 12:49:09,884 - PixivUtil20170411 - ERROR - Unknown Error: 'utf8' codec can't decode byte 0xfb in position 66: invalid start byte
Traceback (most recent call last):
File "PixivUtil2.py", line 1876, in main
File "PixivUtil2.py", line 1671, in main_loop
File "PixivUtil2.py", line 1538, in menu_download_new_illust_from_bookmark
File "PixivUtil2.py", line 1081, in process_new_illust_from_bookmark
File "PixivUtil2.py", line 682, in process_image
File "PixivUtil2.py", line 239, in download_image
File "traceback.pyc", line 128, in print_exception
File "traceback.pyc", line 13, in _print
File "win_unicode_console\streams.pyc", line 254, in write
File "encodings\utf_8.pyc", line 16, in decode
UnicodeDecodeError: 'utf8' codec can't decode byte 0xfb in position 66: invalid start byte

BadStatusLine: ''
URLError

network issue, looks like the server return empty response.

UnicodeDecodeError

failed to decode the error message from python Une connexion existante a d\xfb \xeatre ferm\xe9e par l\x92h\xf4te distant, looks like also network issue? Google translate said: An existing connection was forcibly closed by the remote host
check you network connection/try different isp?

Thank you very much for quick reply!!!
I tried a different thing : a reboot + full exclusion from avast and spybot + running it alone without µtorrent or any other download
It worked then on the second try successfully from page 1 to 42.
so ""problem solved"", thanks again!!!

De : Nandaka<EMAIL_ADDRESS>Envoyé : mardi 11 avril 2017 14:24:37
À : Nandaka/PixivUtil2
Cc : Saulofein; Author
Objet : Re: [Nandaka/PixivUtil2] Fail to download new illust bookmar_new_illust.php (#221)
BadStatusLine: ''
URLError
network issue, looks like the server return empty response.
UnicodeDecodeError
failed to decode the error message from python Une connexion existante a d\xfb \xeatre ferm\xe9e par l\x92h\xf4te distant, looks like also network issue? Google translate said: An existing connection was forcibly closed by the remote host
check you network connection/try different isp?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Nandaka/PixivUtil2/issues/221#issuecomment-293243901, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AaBDnAnWo-KwOp2Hv1KBvztoWjV8PlP6ks5ru3EFgaJpZM4M57fr.


How to resolve Conflict Between @vueform/nuxt and Nuxt UI Dropdown/Popover After Web Breakpoint?

Discussed in https://github.com/vueform/vueform/discussions/328

Originally posted by almatanjin September 16, 2024
After installing @vueform/nuxt in a Nuxt 3 project, the Nuxt UI dropdown and popover components stop working after reaching the web breakpoint. The dropdowns and popovers worked perfectly before installing the @vueform/nuxt module. Removing the module resolves the issue, but it's required for the project.
nuxt.config.ts
modules: [         "@nuxt/ui",         "nuxt-vuefire",         '@pinia/nuxt',         '@pinia-plugin-persistedstate/nuxt',         "@nuxtjs/i18n",         "@vueform/nuxt"     ],
Here, If I remove @vueform/nuxt all things working but for my project I need to use this module.
vueform.config.ts
import en from '@vueform/vueform/locales/en' import tailwind from '@vueform/vueform/dist/tailwind' import { defineConfig } from '@vueform/vueform' export default defineConfig({   theme: tailwind,   locales: { en },   locale: 'en', })
tailwind.config.ts
module.exports = {   content: [     './vueform.config.ts', // or where vueform.config.jsis located. Change.jsto.ts if required<EMAIL_ADDRESS><EMAIL_ADDRESS>  ],   theme: {     extend: { },   plugins: [     require('@vueform/vueform/tailwind'),   ], };
I use Nuxt 3 version 3.12.1
Steps to Reproduce:

Install @nuxt/ui and @vueform/nuxt modules.
Use any Nuxt UI dropdown or popover in the project.
After the web breakpoint, dropdowns and popovers fail to work.

Expected Behavior:
Nuxt UI dropdowns and popovers should function correctly even after @vueform/nuxt is installed.
Actual Behavior:
After the page crosses the web breakpoint, dropdowns and popovers stop functioning when @vueform/nuxt is installed.
Attempted Solutions:
Removing @vueform/nuxt solves the issue but is not a viable solution for the project requirements.
Additional Information:
This issue only arises after the page reaches the web breakpoint, indicating a potential conflict between the Vueform module's configuration and Nuxt UI's behavior at certain breakpoints.
Screencast from 2024-09-16 00-12-14.webm

Happy to look into this if you provide a reproduction. Closing this as this is a duplicate of #328.


fix pip install for non-UTF-8 locales, fix travis tests

Hi, and thanks for the great package!  No real changes to the core code here, but fixed an installation bug I was running into, as well as some issues with the non-coverage Travis CI tests.
There's a number of ways to handle both of these, and I'd be happy to implement any suggested changes!
Proposed updates:

pip installation broke in some environments after PR #156 (see below).  This PR updates setup.py to fix it, and maintains support for installing with Python 2.7 for now.
black requires Python>=3.6, so Python 2.7 Travis tests were failing when trying to install it.  This PR updates ci_scripts/install.sh to pip install black only for the Python 3.x test builts.
all other changed files are just formatting fixes from running black

pip install bug & setup.py fix
Since #217 was fixed in PR #282, I've been pip-installing UMAP from GitHub (pip install git+git://github.com/lmcinnes/umap.git@ c9aaf8a363e9318fb879f9a350f5dd278d2e7ecf#egg=umap_learn) whenever I use it , without any issues.
Yesterday, I tried to pip-install from a more recent commit hash (9fd18d1bed436d188f48f587a8842881b4e854a8) into a Docker container (Debian 10.2), and it failed with the following error:
root@1c7c270de7bb:/# pip install git+git://github.com/lmcinnes/umap@9fd18d1bed436d188f48f587a8842881b4e854a8#egg=umap_learn
Collecting umap_learn
  Cloning git://github.com/lmcinnes/umap (to revision 9fd18d1bed436d188f48f587a8842881b4e854a8) to /tmp/pip-install-vkhw38vz/umap-learn
  Running command git clone -q git://github.com/lmcinnes/umap /tmp/pip-install-vkhw38vz/umap-learn
  Running command git checkout -q 9fd18d1bed436d188f48f587a8842881b4e854a8
    ERROR: Command errored out with exit status 1:
     command: /opt/conda/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-vkhw38vz/umap-learn/setup.py'"'"'; __file__='"'"'/tmp/pip-install-vkhw38vz/umap-learn/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-install-vkhw38vz/umap-learn/pip-egg-info
         cwd: /tmp/pip-install-vkhw38vz/umap-learn/
    Complete output (9 lines):
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-vkhw38vz/umap-learn/setup.py", line 11, in <module>
        'long_description' : readme(),
      File "/tmp/pip-install-vkhw38vz/umap-learn/setup.py", line 5, in readme
        return readme_file.read()
      File "/opt/conda/lib/python3.6/encodings/ascii.py", line 26, in decode
        return codecs.ascii_decode(input, self.errors)[0]
    UnicodeDecodeError: 'ascii' codec can't decode byte 0xe2 in position 2444: ordinal not in range(128)
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Turns out the issue was due to that container's encoding being ascii, not UTF-8
root@1c7c270de7bb:/# locale charmap
ANSI_X3.4-1968

Python's built-in open() defaults to locale.getpreferredencoding(do_setlocale=False) as the encoding for reading files, and #156 added some non-ascii UTF-8 characters to the README.  I verified I was able to install from the previous commit, but none after that.
In reality, this would likely be a pretty rare issue, as it would only really present when a non-UTF-8 locale is intentionally set.  However, since it didn't crop up at all until recently and the fix was pretty simple and stable, I thought it'd be worth adding.

Hello @paxtonfitzpatrick! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file doc/conf.py:

Line 193:80: E501 line too long (82 > 79 characters)

In the file examples/plot_algorithm_comparison.py:

Line 119:80: E501 line too long (80 > 79 characters)

In the file examples/plot_fashion-mnist_example.py:

Line 28:80: E501 line too long (85 > 79 characters)

In the file examples/plot_feature_extraction_classification.py:

Line 52:80: E501 line too long (86 > 79 characters)

In the file umap/tests/test_umap.py:

Line 922:21: E231 missing whitespace after ','

In the file umap/umap_.py:

Line 223:80: E501 line too long (88 > 79 characters)
Line 228:80: E501 line too long (82 > 79 characters)
Line 237:80: E501 line too long (81 > 79 characters)
Line 251:80: E501 line too long (80 > 79 characters)
Line 283:80: E501 line too long (80 > 79 characters)
Line 486:80: E501 line too long (88 > 79 characters)
Line 633:80: E501 line too long (82 > 79 characters)
Line 1052:80: E501 line too long (80 > 79 characters)
Line 1312:80: E501 line too long (84 > 79 characters)
Line 1314:80: E501 line too long (83 > 79 characters)
Line 1315:80: E501 line too long (81 > 79 characters)
Line 1338:80: E501 line too long (85 > 79 characters)
Line 1381:80: E501 line too long (81 > 79 characters)
Line 1423:80: E501 line too long (81 > 79 characters)
Line 1440:80: E501 line too long (87 > 79 characters)
Line 1649:80: E501 line too long (81 > 79 characters)
Line 1654:80: E501 line too long (82 > 79 characters)
Line 1660:80: E501 line too long (86 > 79 characters)
Line 1664:80: E501 line too long (81 > 79 characters)
Line 1689:80: E501 line too long (85 > 79 characters)
Line 1692:80: E501 line too long (85 > 79 characters)

Thanks for all of this -- I admit that these sorts of issues are definitely not my area of expertise, so I greatly appreciate anyone willing to step in and both find and fix them. Your work is definitely appreciated!


even with a password set, one can connect without (this is a major security issue?)

in docker-entrypoint.sh there is line 252:
``̀`
printf 'host all all all %s\n' "$POSTGRES_HOST_AUTH_METHOD"
} >> "$PGDATA/pg_hba.conf"
That keeps trust! shoud be:

printf 'local all all %s\n' "$POSTGRES_HOST_AUTH_METHOD"
    printf 'host all all all %s\n' "$POSTGRES_HOST_AUTH_METHOD"
} > "$PGDATA/pg_hba.conf"

I can confirm that it works and am submitting a PR.

This is intentionally keeping the PostgreSQL upstream default, which only allows unauthenticated connections locally, and can be modified/disabled via other means (and has been discussed in issues in this repository several times previously).

There are "host" allowed with trust: it is not only local connection. If a user listen all adresses, one can connect from anywhere without password and there is no way to change that apart changing the docket-entrypoint.sh.
Le 29 décembre 2022 06:04:40 GMT-10:00, Tianon Gravi @.***> a écrit :

This is intentionally keeping the PostgreSQL upstream default, which only allows unauthenticated connections locally, and can be modified/disabled via other means (and has been discussed in issues in this repository several times previously).
--
Reply to this email directly or view it on GitHub:
https://github.com/docker-library/postgres/issues/1028#issuecomment-1367432987
You are receiving this because you authored the thread.
Message ID: @.***>

None of the PostgreSQL-provided default host entries in pg_hba.conf allow any remote access. They are all limited to localhost-only IP ranges that are not routable across a network, so none of them are exploitable even if the container is run with host mode networking (--network host).
$ docker run -it --rm -e POSTGRES_PASSWORD="password" --name=pg postgres

$ docker exec -it pg bash
root@ea22e289a6d0:/# grep -vE '^[#]|^\s*$' /var/lib/postgresql/data/pg_hba.conf
local   all             all                                     trust
host    all             all             <IP_ADDRESS>/32            trust
host    all             all             ::1/128                 trust
local   replication     all                                     trust
host    replication     all             <IP_ADDRESS>/32            trust
host    replication     all             ::1/128                 trust
host all all all scram-sha-256
$ # ^ this last one is the one the entrypoint script adds

Without --network=host, these passwordless trust connections can only happen from within the container (like docker exec). If your situation requires them to be different, the file can be controlled via POSTGRES_INITDB_ARGS='--auth-local=scram-sha-256 --auth-host=scram-sha-256' to control the local connection initial default and the host connection initial default values in pg_hba.conf.
$ docker run -it --rm -e POSTGRES_PASSWORD="password" -e POSTGRES_INITDB_ARGS='--auth-local=scram-sha-256 --auth-host=md5' --name=pg --network=host postgres
...
$ docker exec -it pg bash
root@barad-dur:/# grep -vE '^[#]|^\s*$' /var/lib/postgresql/data/pg_hba.conf
local   all             all                                     scram-sha-256
host    all             all             <IP_ADDRESS>/32            md5
host    all             all             ::1/128                 md5
local   replication     all                                     scram-sha-256
host    replication     all             <IP_ADDRESS>/32            md5
host    replication     all             ::1/128                 md5
host all all all scram-sha-256

There is a table I made over in https://github.com/docker-library/postgres/pull/897#issuecomment-944669713 that shows how POSTGRES_INITDB_ARGS="=-auth-host=scram-sha-256", POSTGRES_HOST_AUTH_METHOD=authValue, and -c password_encryption=encValue interact though I am not certain how those also interact with --auth-local in initdb args.


CAMEL-10391 Created a mechanism to configure Camel CDI via CDI event.

See https://issues.apache.org/jira/browse/CAMEL-10391 for details

Would you mind squashing your changes into a single commit. Thanks.

Of course, as soon as we agree that all is done

That looks all good. We'll merge it ASAP once you've squashed it. Thanks!

I think it's correct now

Perfect, we'll merge it ASAP. Thanks!

@davsclaus @oscerd, the PR looks good to me. Would you like me to merge it (if I'm allowed to) or do you want to go through it?

I guess there is no problem! Go for it! Thanks :-)

@astefanutti yeah sure you are the CDI expert.

@sverkera Thanks for the PR! You may want to associate your Git e-mail address to your GitHub account 😉.

Ah @astefanutti next  time you merge a PR, then please run a rebase first so the git tree is a straight line. We always merge on top.
eg what I do is
git pull ....
git --rebase master
And then if that works a git push origin

Ah sorry, I went to quickly on that one 😞. Won't forget next time.


is there a place where we can swap decks?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

what do you mean swap?

if you are alexwagner74 from reddit, for others who wont see the reddit post, here was my response:

if you export your anki deck to plan text you can then add the colon seperators and the required points field to the "cards" but for easy import/export, anki doesnt seem to export directly to csv. could probably download anki decks and export to txt then convert the cards to the format flash.sh uses.
there are bashisms in flash.sh i believe (not 100% about what exactly or the portabiity) but im using bash on linux, not sure if OSx would be compatible with it with their old /bin/bash, plus im unfamiliar with mac file system, do they even have .local/share?

i see that ankiweb flash decks are just .zip files with an sqlite3 database inside, so you can basically do a query and output to csv from there. I am just wondering if you are going create a repo for decks that people can upload if they have taken the time to export or create them and they want to share.

that sounds like a terrific idea! i'll make it tonight in an hour or so

@grokskookum the repo is made and im fleshing out details, readme of flash.sh has also been updated to point to it, thanks again for the idea! :rocket:


Stage pull requests

Staging:

https://github.com/oisf/suricata-verify/pull/1026
https://github.com/oisf/suricata-verify/pull/1204
https://github.com/oisf/suricata-verify/pull/1235
https://github.com/oisf/suricata-verify/pull/1236

Covered in https://github.com/OISF/suricata-verify/pull/1239.


FE: implement fonts + SVGs, general polish on styling

player profile + player compare, decrease the font size
update player profile page to have avg points per game

add white background to dashboard

Add sorting CSS arrows on player stats page

sorting css arrows will be done in #96


Added finer grain capabilities

The functions are only added when using FDB. If you want to do what you showed, use 'FDBGET' EVAL

I know the workaround, but it is not as convenient.
Functions are defined as a standard or by an extension, not by the backend type. And you already check the backend in all these functions, so it won't be a problem to treat them as leveldb ones.
In the current documentation process, it will more easy not to add this kind of exceptions.

Those functions were added in PR#1192 that you approved. We can modify the way they are defined but those types of concerns should be raised within the convo of the appropriate PR.

PTAL


Slow continous integration (github actions)

The CI checks run painfully slow and jobs are often stalled (queued).

For me, this does no longer seem to be the case. Proposing to close this issue.


Resolving to wrong exe for insiders

Should be using Code - Insiders.exe instead of Code.exe.

On Linux should be code-insiders.
On macOS should be Visual Studio Code - Insiders.


Implemented linter in project

SwiftLint included in the project through homebrew to follow some code style patterns.

My suggestion: Include the brew command to install SwiftLint in a step by step in the readme.

good hint @digoreis, i added this step it to Readme.


UnicodeDecodeError

I get the following error when installing this plugin. This is both with PackageControl and manually.
Running on a Windows10 machine
** ERROR: Error from WindowsApps.WindowsApps.on_catalog: <class 'UnicodeDecodeError'>: 'charmap' codec can't decode byte 0x81 in position 106229: character maps to
Traceback (most recent call last):
File "C:\Users[me]\AppData\Roaming\Keypirinha\InstalledPackages\WindowsApps.keypirinha-package\windowsapps.py", line 173, in on_catalog
File "lib\subprocess.py", line 926, in communicate
File "lib\encodings\cp1252.py", line 23, in decode
08:45:55.739 WindowsApps.ModernControlPanel: Cataloged 91 items in 3.9 seconds

Somewhere in the output of the powershell cmdlet Get-AppPackage is an "ü", i think. I use german windows as well and i never encountered any apps that use umlauts in their technical name. Can your run the following in a powershell console and post the output here?
Get-AppPackage | Out-String -Stream | Select-String -Pattern 'ü'

(this searches for "ü" in the same output the WindowsApps-Package fails to decode)
I would like to know which app it is, to be sure my fix works.

Somewhere in the output of the powershell cmdlet Get-AppPackage is an "ü", i think. I use german windows as well and i never encountered any apps that use umlauts in their technical name. Can your run the following in a powershell console and post the output here?
Get-AppPackage | Out-String -Stream | Select-String -Pattern 'ü'

(this searches for "ü" in the same output the WindowsApps-Package fails to decode)
I would like to know which app it is, to be sure my fix works.

This is the output. I have replaced your real name and street with placeholders, just in case. But the output is indeed with the u-umlaut:
Publisher         : CN=[Your name], O=[Your name], STREET=[Your address], L=Nürnberg, S=Bayern, PostalCode=90478,

Well that does not help much because I still don't know which app this is. Can post the output of
Get-AppPackage | Where-Object {$_.Publisher -Like '*L=Nürnberg*'}

here?


✨ Kusama Bot on landing curated list

PR type

[x] Feature

What's new?

[x] Add Kusama Bot Collection in first slider of rmrk landing page

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Screenshot

empowering homies 💪


erasure and wildcards

Our current handling for ? and erasure is not optimal -- see #41 and #55 for an example of the problems we encounter. This issue is for brainstorming a better solution.
Problem 1: returning ?
The getClass method returns Class<?>. We can't model this with 100% fidelity for Rust users. Mapping it to Class<Object> is somewhat reasonable, since the role of the type parameter in this case is primarily covariant -- it's not necessarily suitable as a general purpose solution, consider methods like
class Foo<T> {
    void takes(T t) { /* given Foo<?>, you shouldn't be able to call this with any object */ }
}

It also allows you to invoke Java methods like this one, which ideally wouldn't be allowed...
void sameClass<T>(Class<T> c1, Class<T> c2)

...but this may not be achievable. I think I'm willing to live with that,.
In both cases, the worst that happens is we generate class cast exceptions in Java, not UB.
Problem 2: modeling erased Java types
Scala sometimes intentionally translates to erased Java types. These I think are largely equivalent to ?, except that Java is more lenient and makes "angelic" type assumptions.

cc @fpoli -- does this cover the details, or are there further complications I'm ignoring?

One additional question: Should duchess generate runtime type checks before the JNI calls where it knows to be imprecise?
Since the JVM only verifies bytecode for type safety, not the JNI calls, I suspect that the imprecision of duchess in the type parameters might lead to cases where the JVM receives the wrong type and thus dereferences some bad pointer at runtime. I'll see if I can build an example.

I've been giving this some thought. My current proposal is that we add some special types:

java::Wildcard, where  for all T, T: Upcast<Wildcard> and Wildcard: Upcast<Object>
java::Extends<T>, where for all U, U: Upcast<T> => U: Upcast<Extends<T>> and Extends<T>: Upcast<U>
java::Super<T> etc

and an additional trait java::SpecificClass, which is implemented by all Java objects, but not the three types above. We would add this to any type parameter that appears more than once in a signature.
Then we can translate signatures to use those types and I think we actually get the correct typing rules.
This doesn't account for ? extends A & B, we'd have to do something like java::Extends<(A, B)>, but we can do that (up to some fixed arity).
It's probably also possible to do ?extends Foo super Bar? Also possible to account for.


Returning the image and adding coingecko

Image was probably removed by accident, added coingecko api id

Hi @Kamikaza731 ,
The image reference was actually removed intentionally.
Tracing this asset to its origin, picasso::ppica, the reference to the image does indeed exist. By opting to not reference the image again in this composable::ppica asset object, we are showing that the image (and any potential changes to it) should remain synchronized across both assets.
This makes it easier to update images, because if the name or location of the file happens to change, we longer need to track down and update multiple references to the same image (or risk breaking some forgotten references). Thus, excluding the image reference in this file more appropriately indicates the relationship between this asset and its origin, and is arguably safer and cleaner.
However, not all software reading the chain registry are necessarily sophisticated enough yet to be able to pull properties form the traced origin assets, in which case adding an image reference here could be useful for some cases.
The coingecko id, however, was missing from picasso::ppica, and I think it'd be good to add there (and only there).

Hello @JeremyParish69
I only did this because I noticed some services are not working as intended or are just missing images that should have been there. For example, I noticed in https://restake.app since the last chain reg update to the asset list on the composable directory, for some reason it no longer detects it (for some reason I think because the logo is missing it won't load it which is bizarre if true). Cosmos directory https://cosmos.directory/composable also misses the images. I do not know if some other app/service was affected by this.
I did notice you have all of the necessary data needed in _non-cosmos/picasso, but what about the data needed for Centauri-1? In this case, maybe having a symlink for assetlist.json could work. But I did notice there is a "traces" additional mintage in which case it might not work so I think every chain should have its asset list.
Anyway, I won't interfere any more since you seem to maintain these directories I only thought the logo was removed by accident.

Hi @Kamikaza731 Thanks for pointing out the issues with these missing.
I'm starting to think you're probably right about these needing to be there. It's probably time to separate the data needs from the interface needs so that any API looking for an image doesn't need to be smart enough to crawl up the origin trace.
If interested in joining the discussion and following the development, be sure to join the Telegram working group (linked on the README)

Hey @Kamikaza731 @JeremyParish69 - I'm the REStake/Cosmos.directory dev. Ultimately I think the apps should use the logo URIs defined in the main chain.json file for the chain imagery, rather than the 'first' asset as it was before - I've made this update to cosmos.directory and restake.app so composable is now displaying as expected.
I actually agree with Jeremy's initial suggestion to avoid duplicating the assets and that apps should really look them from the source chain. The Chain Registry is already pretty good from an API point of view, and I think should primarily be a sensible data source rather than jumping through hoops to make it interface suitable. It's not a big lift for apps to lookup this data, so long as all the necessary attributes are available to make the connections easily. Having said that I think both options have pros and cons and I ultimately don't mind which way the decision goes.


Cheque view UI

Done added section for user input and preview/output for cheque view.

But the design will need further refinement,
and for now text boxes to display info in the preview section are not added.

Closes #22.

Screenshot of ChequeView.xaml:

Oh, and previously I tried to do rebase when I had some existing commits pushed, and encountered a few conflicts.
I tried to solve them, hopefully nothing is messed up.

Oh and for the part regarding making Application window always full screen,
at least based on my current knowledge and research,
need to use  instead of .
Not sure if there's any other workaround or not yet.
I think this needs further discussion.

Oh, and previously I tried to do rebase when I had some existing commits pushed, and encountered a few conflicts. I tried to solve them, hopefully nothing is messed up.

I will checked that. Thanks

Oh and for the part regarding making Application window always full screen, at least based on my current knowledge and research, need to use instead of .
Not sure if there's any other workaround or not, I can't figure it out yet.
I think this needs further discussion.

Sure. We can discuss in the standup meeting.

Oh and for the part regarding making Application window always full screen, at least based on my current knowledge and research, need to use  instead of .
Not sure if there's any other workaround or not, I can't figure it out yet.
I think this needs further discussion.

As discussion, you can try on the ChequeWriterMainWindow.xaml in ChequeWriter main project.


No results page

Problem
We need to show a friendly message when there are no results. The page should encourage people to try a different search or browse on the additional source sites.
Description
The mockup is available in the Releases file as both desktop and mobile. I will add the image as an exportable asset ASAP.

Desktop
Mobile

Additional context
On desktop, the source site is a border version of the secondary button, while on mobile is the same as desktop but full-width and 16px top and bottom padding. I was not sure about adding these elements as components since are used only on this page. Any ideas on this?

@panchovm I think even components that appear once should be a part of the component library.
While we are under time constraints we will sometimes make one-off components to quickly get something done but in the long term our idea should be that any component in use across the site can be found in the library.

Perfect. I added the task for the after release. Thanks

Here is attached the image compressed to have a smaller file size.
Depressed musician image.zip


Include Keybase in Finder favorites

We can't reliably create a file list (Finder) favorite directly for /keybase, since it is a remote volume mount, and there is some funkiness there (like the URL property not resolving, or the display name being ignored). If we create a symlink though, all these problems are avoided. So we'll create a symlink to /keybase and add this as the file list favorite item.
Also I'm changing project file to use Automatic Provisioning. We build and sign using a script, and we verify the signing identity in the install scripts.

@chromakode can review?
cc @keybase/react-hackers

bump


Mighty Nit: Standardize torch.Tensor typing

Sometimes we typehint torch.Tensor, sometimes it's Tensor. Wouldn't it be great if it all looked the same?

@SalmanMohammadi Can I work on this?

Hey @varshinimuthukumar1! Thanks for asking. Absolutely. Feel free to put up a PR and ping me on it. : )
cc @RdoubleA @ebsmothers @felipemello1 @pbontrager choose your fighter
🚀 = torch.Tensor
👀 = Tensor

Why make typing take up more space than it already does?

to save an extra import :P

using Tensor requires adding an extra import line from torch import Tensor, you already have torch imported in almost all files just stick with it :)

i think it also bothers me a bit when i see "Tensor". Ok, what type of Tensor? Is it a torch.Tensor? :D

Thanks for assigning this issue to me. I will put up a PR!

Not to throw a wrench in the works here but pytorch also uses Tensor...

they use both

But i just dont know if the time spent on it is worth on it haha. Since @varshinimuthukumar1  is willing to contribute, maybe the time would be better spent in other tickets with "good first issue" and "community help wanted"

I took up this issue to get a first-feel of the code base, and contribute further.
If it is not needed, I will drop this issue, and start elsewhere :D

I think it would be better. The torch.tensor is just cosmetic, but there are plenty of open issues that would have an impact by creating new features or solving small issues that affect users.
Thanks for contributing @varshinimuthukumar1 ! Feel free to share your thoughts and ask questions as you get more familiar with torchtune :)


[Bug]: Heroku deploy button

What happened?
Deploy works fine then vite fails. Running on free dynos in Heroku.
2022-08-06T08:39:38.347888+00:00 app[web.1]:
2022-08-06T08:39:38.347900+00:00 app[web.1]: ><EMAIL_ADDRESS>start
2022-08-06T08:39:38.347901+00:00 app[web.1]: > vite
2022-08-06T08:39:38.347901+00:00 app[web.1]:
2022-08-06T08:39:38.354156+00:00 app[web.1]: sh: 1: vite: not found
2022-08-06T08:39:38.484340+00:00 heroku[web.1]: Process exited with status 127
2022-08-06T08:39:38.700683+00:00 heroku[web.1]: State changed from starting to crashed
2022-08-06T08:41:19.671013+00:00 heroku[router]: at=error code=H10 desc="App crashed" method=GET path="/" host=twingsisterw11.herokuapp.com request_id=53f939a4-7e3d-42d9-b273-cb86f59d7115 fwd="<IP_ADDRESS>" dyno= connect= service= status=503 bytes= protocol=https
2022-08-06T08:41:20.204311+00:00 heroku[router]: at=error code=H10 desc="App crashed" method=GET path="/favicon.ico" host=twingsisterw11.herokuapp.com request_id=944eb4f1-9607-49e2-8bb6-63f2e85dc096 fwd="<IP_ADDRESS>" dyno= connect= service= status=503 bytes= protocol=https
2022-08-06T08:42:30.690935+00:00 heroku[router]: at=error code=H10 desc="App crashed" method=GET path="/" host=twingsisterw11.herokuapp.com request_id=42d49b14-d442-4b36-a6b0-fb795053aeac fwd="<IP_ADDRESS>" dyno= connect= service= status=503 bytes= protocol=https
2022-08-06T08:48:24.478376+00:00 heroku[router]: at=error code=H10 desc="App crashed" method=GET path="/" host=twingsisterw11.herokuapp.com request_id=86ed2e85-0786-4dca-b7ce-ba70e996b6fb fwd="<IP_ADDRESS>" dyno= connect= service= status=503 bytes= protocol=https
To reproduce the behavior?
reproduce!
What browser are you seeing the problem on?
Firefox

Thanks for bringing this issue to notice will look into it.

Can't get this working on Railway, too. There build fails
Using Detected Dockerfile
#1 [internal] load build definition from Dockerfile
#1 sha256:7f88cb51e95e2c56315400d13cca68c88c827740b1ad26af82a4b3a9aacb6b78
#1 transferring dockerfile: 683B done
#1 DONE 0.5s
#2 [internal] load .dockerignore
#2 sha256:34a6bb64a2c5daea0c2ead36d1be05dc194ba6158f043b12f3ea09b4a0055f6d
#2 transferring context: 48B done
#2 DONE 0.6s
#3 [internal] load metadata for docker.io/library/node:14-alpine3.14
#3 sha256:67820c123a43010c00418d79c7a98367573500fc43695d0f8bbe15468138c4d4
#3 DONE 0.7s
#4 [1/5] FROM docker.io/library/node:14-alpine3.14@sha256:f6bd262df57e5cb454ace4f0b2c58b69727bf2ae38d89a5e0a1658d839d7c618
#4 sha256:ccc99e07dd7ccb757af74cb21dea33b9a0e1a4598c200045f2e6194dcd396815
#4 DONE 0.0s
#5 [2/5] WORKDIR /usr/src/windows11/
#5 sha256:2520e0bbfcfd63723e5042b77574a038a83dabb7fd74d5cbb8a1a2e1a98380c6
#5 CACHED
#6 [3/5] RUN npm install -g serve
#6 sha256:18b55ff5ec8660e6b67685064c49a24c1d24230a2c82d3f693ba634fda2ae092
#6 CACHED
#7 [4/5] RUN wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip
#7 sha256:231e3c91e480f00b0cca303b5a82cb1ace619023ff20c7e6c8127c56e5028e75
#7 0.464 Connecting to github.com (<IP_ADDRESS>:443)
#7 0.687 Connecting to codeload.github.com (<IP_ADDRESS>:443)
#7 0.853 wget: server returned error: HTTP/1.1 404 Not Found
#7 ERROR: executor failed running [/bin/sh -c wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip]: exit code: 1

[4/5] RUN wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip:

executor failed running [/bin/sh -c wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip]: exit code: 1

Fails on Railway too
`=========================
Using Detected Dockerfile
#1 [internal] load build definition from Dockerfile
#1 sha256:7f88cb51e95e2c56315400d13cca68c88c827740b1ad26af82a4b3a9aacb6b78
#1 transferring dockerfile: 683B done
#1 DONE 0.5s
#2 [internal] load .dockerignore
#2 sha256:34a6bb64a2c5daea0c2ead36d1be05dc194ba6158f043b12f3ea09b4a0055f6d
#2 transferring context: 48B done
#2 DONE 0.6s
#3 [internal] load metadata for docker.io/library/node:14-alpine3.14
#3 sha256:67820c123a43010c00418d79c7a98367573500fc43695d0f8bbe15468138c4d4
#3 DONE 0.7s
#4 [1/5] FROM docker.io/library/node:14-alpine3.14@sha256:f6bd262df57e5cb454ace4f0b2c58b69727bf2ae38d89a5e0a1658d839d7c618
#4 sha256:ccc99e07dd7ccb757af74cb21dea33b9a0e1a4598c200045f2e6194dcd396815
#4 DONE 0.0s
#5 [2/5] WORKDIR /usr/src/windows11/
#5 sha256:2520e0bbfcfd63723e5042b77574a038a83dabb7fd74d5cbb8a1a2e1a98380c6
#5 CACHED
#6 [3/5] RUN npm install -g serve
#6 sha256:18b55ff5ec8660e6b67685064c49a24c1d24230a2c82d3f693ba634fda2ae092
#6 CACHED
#7 [4/5] RUN wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip
#7 sha256:231e3c91e480f00b0cca303b5a82cb1ace619023ff20c7e6c8127c56e5028e75
#7 0.464 Connecting to github.com (<IP_ADDRESS>:443)
#7 0.687 Connecting to codeload.github.com (<IP_ADDRESS>:443)
#7 0.853 wget: server returned error: HTTP/1.1 404 Not Found
#7 ERROR: executor failed running [/bin/sh -c wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip]: exit code: 1

[4/5] RUN wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip:

executor failed running [/bin/sh -c wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip]: exit code: 1`

Fails on railway, too, during build

=========================
Using Detected Dockerfile
=========================
#1 [internal] load build definition from Dockerfile

#1 sha256:7f88cb51e95e2c56315400d13cca68c88c827740b1ad26af82a4b3a9aacb6b78
#1 transferring dockerfile: 683B done
#1 DONE 0.5s
#2 [internal] load .dockerignore
#2 sha256:34a6bb64a2c5daea0c2ead36d1be05dc194ba6158f043b12f3ea09b4a0055f6d
#2 transferring context: 48B done
#2 DONE 0.6s
#3 [internal] load metadata for docker.io/library/node:14-alpine3.14
#3 sha256:67820c123a43010c00418d79c7a98367573500fc43695d0f8bbe15468138c4d4
#3 DONE 0.7s
#4 [1/5] FROM docker.io/library/node:14-alpine3.14@sha256:f6bd262df57e5cb454ace4f0b2c58b69727bf2ae38d89a5e0a1658d839d7c618
#4 sha256:ccc99e07dd7ccb757af74cb21dea33b9a0e1a4598c200045f2e6194dcd396815
#4 DONE 0.0s
#5 [2/5] WORKDIR /usr/src/windows11/
#5 sha256:2520e0bbfcfd63723e5042b77574a038a83dabb7fd74d5cbb8a1a2e1a98380c6
#5 CACHED
#6 [3/5] RUN npm install -g serve
#6 sha256:18b55ff5ec8660e6b67685064c49a24c1d24230a2c82d3f693ba634fda2ae092
#6 CACHED
#7 [4/5] RUN wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip
#7 sha256:231e3c91e480f00b0cca303b5a82cb1ace619023ff20c7e6c8127c56e5028e75
#7 0.464 Connecting to github.com (<IP_ADDRESS>:443)
#7 0.687 Connecting to codeload.github.com (<IP_ADDRESS>:443)
#7 0.853 wget: server returned error: HTTP/1.1 404 Not Found
#7 ERROR: executor failed running [/bin/sh -c wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip]: exit code: 1

[4/5] RUN wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip:

executor failed running [/bin/sh -c wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip]: exit code: 1

and also netlify
1:59:12 PM: Build ready to start
1:59:17 PM: build-image version: d7b3dbfb0846505993c9a131894d1858074c90b4 (focal)
1:59:17 PM: build-image tag: v4.10.1
1:59:17 PM: buildbot version: fe6512288e75c8fa5aadaebb51ed1f96e9314fd4
1:59:18 PM: Fetching cached dependencies
1:59:18 PM: Failed to fetch cache, continuing with build
1:59:18 PM: Starting to prepare the repo for build
1:59:18 PM: No cached dependencies found. Cloning fresh repo
1:59:18 PM: git clone https://github.com/blueedgetechno/win11React
1:59:20 PM: Preparing Git Reference refs/heads/master
1:59:20 PM: Parsing package.json dependencies
1:59:21 PM: Starting build script
1:59:21 PM: Installing dependencies
1:59:21 PM: Python version set to 2.7
1:59:21 PM: v16.16.0 is already installed.
1:59:21 PM: Now using node v16.16.0 (npm v8.11.0)
1:59:21 PM: Started restoring cached build plugins
1:59:21 PM: Finished restoring cached build plugins
1:59:22 PM: Attempting ruby version 2.7.2, read from environment
1:59:22 PM: Using ruby version 2.7.2
1:59:22 PM: Using PHP version 8.0
1:59:23 PM: No npm workspaces detected
1:59:23 PM: Started restoring cached node modules
1:59:23 PM: Finished restoring cached node modules
1:59:23 PM: Installing NPM modules using NPM version 8.11.0
1:59:23 PM: npm WARN config tmp This setting is no longer used.  npm stores temporary files in a special
1:59:23 PM: npm WARN config location in the cache, and they are managed by
1:59:23 PM: npm WARN config     cacache.
1:59:23 PM: npm WARN config tmp This setting is no longer used.  npm stores temporary files in a special
1:59:23 PM: npm WARN config location in the cache, and they are managed by
1:59:23 PM: npm WARN config     cacache.
1:59:31 PM: npm WARN deprecated<EMAIL_ADDRESS>replaced by i18next-http-backend
1:59:35 PM: npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Some versions have web compatibility issues. Please, upgrade your dependencies to the actual version of core-js.
1:59:38 PM: added 756 packages, and audited 757 packages in 14s
1:59:38 PM: 105 packages are looking for funding
1:59:38 PM:   run npm fund for details
1:59:38 PM: found 0 vulnerabilities
1:59:38 PM: NPM modules installed
1:59:38 PM: npm WARN config tmp This setting is no longer used.  npm stores temporary files in a special
1:59:38 PM: npm WARN config location in the cache, and they are managed by
1:59:38 PM: npm WARN config     cacache.
1:59:38 PM: Started restoring cached go cache
1:59:38 PM: Finished restoring cached go cache
1:59:38 PM: Installing Go version 1.17 (requested 1.17)
1:59:42 PM: unset GOOS;
1:59:42 PM: unset GOARCH;
1:59:42 PM: export GOROOT='/opt/buildhome/.gimme/versions/go1.17.linux.amd64';
1:59:42 PM: export PATH="/opt/buildhome/.gimme/versions/go1.17.linux.amd64/bin:${PATH}";
1:59:42 PM: go version >&2;
1:59:42 PM: export GIMME_ENV="/opt/buildhome/.gimme/env/go1.17.linux.amd64.env"
1:59:42 PM: go version go1.17 linux/amd64
1:59:42 PM: Installing missing commands
1:59:42 PM: Verify run directory
1:59:43 PM:
1:59:43 PM: ────────────────────────────────────────────────────────────────
1:59:43 PM:   Netlify Build
1:59:43 PM: ────────────────────────────────────────────────────────────────
1:59:43 PM:
1:59:43 PM: ❯ Version
1:59:43 PM:   @netlify/build 27.9.1
1:59:43 PM:
1:59:43 PM: ❯ Flags
1:59:43 PM:   baseRelDir: true
1:59:43 PM:   buildId: 62ee579045349f43f83a3da4
1:59:43 PM:   deployId: 62ee579045349f43f83a3da6
1:59:43 PM:   systemLogFile: 3
1:59:43 PM:
1:59:43 PM: ❯ Current directory
1:59:43 PM:   /opt/build/repo
1:59:43 PM:
1:59:43 PM: ❯ Config file
1:59:43 PM:   No config file was defined: using default values.
1:59:43 PM:
1:59:43 PM: ❯ Context
1:59:43 PM:   production
1:59:43 PM:
1:59:43 PM: ────────────────────────────────────────────────────────────────
1:59:43 PM:   1. Build command from Netlify app
1:59:43 PM: ────────────────────────────────────────────────────────────────
1:59:43 PM:
1:59:43 PM: $ npm run build
1:59:43 PM: npm WARN config tmp This setting is no longer used.  npm stores temporary files in a special
1:59:43 PM: npm WARN config location in the cache, and they are managed by
1:59:43 PM: npm WARN config     cacache.
1:59:43 PM: ><EMAIL_ADDRESS>build
1:59:43 PM: > vite build
1:59:44 PM: vite v3.0.4 building for production...
1:59:44 PM: transforming...
1:59:47 PM: Module level directives cause errors when bundled, 'use babel' was ignored.
1:59:47 PM: Module level directives cause errors when bundled, 'use babel' was ignored.
1:59:49 PM: Use of eval is strongly discouraged, as it poses security risks and may cause issues with minification
1:59:50 PM: ✓ 490 modules transformed.
1:59:50 PM: rendering chunks...
1:59:50 PM: build/assets/spotify.2699cb74.woff2         54.23 KiB
1:59:50 PM: build/assets/SettingsIcons.166d2107.woff2   60.55 KiB
1:59:50 PM: build/assets/server.f30e785e.jpg            21.41 KiB
1:59:50 PM: build/assets/spotify.e2d45b27.ttf           184.50 KiB
1:59:50 PM: build/index.html                            1.78 KiB
1:59:50 PM: build/manifest.webmanifest                  0.14 KiB
1:59:50 PM: build/registerSW.js                         0.13 KiB
1:59:50 PM: build/assets/index.a7cbf2af.css             93.90 KiB / gzip: 18.56 KiB
1:59:51 PM: build/assets/index.3b758049.js              2502.79 KiB / gzip: 741.65 KiB
1:59:51 PM:
1:59:51 PM: (!) Some chunks are larger than 500 KiB after minification. Consider:
1:59:51 PM: - Using dynamic import() to code-split the application
1:59:51 PM: - Use build.rollupOptions.output.manualChunks to improve chunking: https://rollupjs.org/guide/en/#outputmanualchunks
1:59:51 PM: - Adjust chunk size limit for this warning via build.chunkSizeWarningLimit.
1:59:53 PM: PWA v0.12.3
1:59:53 PM: mode      generateSW
1:59:53 PM: precache  13 entries (343.60 KiB)
1:59:53 PM: files generated
1:59:53 PM:   build/sw.js
1:59:53 PM:   build/workbox-3ea082d2.js
1:59:53 PM: warnings
1:59:53 PM:   assets/index.3b758049.js is 2.56 MB, and won't be precached. Configure maximumFileSizeToCacheInBytes to change this limit.
1:59:53 PM:
1:59:53 PM: (build.command completed in 9.7s)
1:59:53 PM:
1:59:53 PM: ────────────────────────────────────────────────────────────────
1:59:53 PM:   2. Deploy site
1:59:53 PM: ────────────────────────────────────────────────────────────────
1:59:53 PM:
1:59:53 PM:
1:59:53 PM: ────────────────────────────────────────────────────────────────
1:59:53 PM:   Configuration error
1:59:53 PM: ────────────────────────────────────────────────────────────────
1:59:53 PM:
1:59:53 PM:   Error message
1:59:53 PM:   Deploy did not succeed: Deploy directory 'dist' does not exist
1:59:53 PM:
1:59:53 PM:   Resolved config
1:59:53 PM:   build:
1:59:53 PM:     command: npm run build
1:59:53 PM:     commandOrigin: ui
1:59:53 PM:     publish: /opt/build/repo/dist
1:59:53 PM:     publishOrigin: ui
1:59:53 PM: Caching artifacts
1:59:53 PM: Started saving node modules
1:59:53 PM: Finished saving node modules
1:59:53 PM: Started saving build plugins
1:59:53 PM: Finished saving build plugins
1:59:53 PM: Started saving pip cache
1:59:53 PM: Finished saving pip cache
1:59:53 PM: Started saving emacs cask dependencies
1:59:53 PM: Finished saving emacs cask dependencies
1:59:53 PM: Started saving maven dependencies
1:59:53 PM: Finished saving maven dependencies
1:59:53 PM: Started saving boot dependencies
1:59:53 PM: Finished saving boot dependencies
1:59:53 PM: Started saving rust rustup cache
1:59:53 PM: Finished saving rust rustup cache
1:59:53 PM: Started saving go dependencies
1:59:53 PM: Finished saving go dependencies
1:59:54 PM: Build failed due to a user error: Build script returned non-zero exit code: 2
1:59:54 PM: Creating deploy upload records
1:59:54 PM: Failing build: Failed to build site
1:59:54 PM: Failed during stage 'building site': Build script returned non-zero exit code: 2 (https://ntl.fyi/exit-code-2)
1:59:54 PM: Finished processing build request in 36.892049447s

and also Vercel
[14:04:47.880] Cloning github.com/TWINGSISTER/winreact16 (Branch: main, Commit: e973349)
[14:04:48.759] Cloning completed: 878.295ms
[14:04:49.153] Installing build runtime...
[14:04:51.651] Build runtime installed: 2.498s
[14:04:52.154] Looking up build cache...
[14:04:52.390] Build Cache not found
[14:04:52.624] Detected pnpm-lock.yaml generated by pnpm 7...
[14:04:52.646] Installing dependencies...
[14:04:53.200] Lockfile is up-to-date, resolution step is skipped
[14:04:53.204]  ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up-to-date with package.json
[14:04:53.204]
[14:04:53.205] Note that in CI environments this setting is true by default. If you still need to run install in such cases, use "pnpm install --no-frozen-lockfile"
[14:04:53.214] Error: Command "pnpm install" exited with 1

#7 ERROR: executor failed running [/bin/sh -c wget https://github.com/blueedgetechno/win11React/archive/refs/heads/gh-pages.zip]: exit code: 1

for railway it is trying to get the gh-pages branch which no longer exists

[14:04:53.205] Note that in CI environments this setting is true by default. If you still need to run install in such cases, use "pnpm install --no-frozen-lockfile"

for vercel you need to set CI as false

Thanks for your prompt reply!
Sorry no experience in DevOps, Seemed to be this to be done, yet...

still...
[15:53:51.997] Cloning github.com/TWINGSISTER/reactw11 (Branch: main, Commit: e205f62)
[15:53:52.852] Cloning completed: 854.594ms
[15:53:53.016] Installing build runtime...
[15:53:54.965] Build runtime installed: 1.948s
[15:53:55.756] No Build Cache available
[15:53:56.035] Detected pnpm-lock.yaml generated by pnpm 7...
[15:53:56.055] Installing dependencies...
[15:53:56.588] Lockfile is up-to-date, resolution step is skipped
[15:53:56.591]  ERR_PNPM_OUTDATED_LOCKFILE  Cannot install with "frozen-lockfile" because pnpm-lock.yaml is not up-to-date with package.json
[15:53:56.591]
[15:53:56.591] Note that in CI environments this setting is true by default. If you still need to run install in such cases, use "pnpm install --no-frozen-lockfile"
[15:53:56.601] Error: Command "pnpm install" exited with 1

for netlify i used these settings and it worked fine https://62ef4518ba5a2244993c3526--inquisitive-yeot-fb9676.netlify.app/ see for yourself

for netlify i used these settings and it worked fine https://win11react-test.netlify.app see for yourself.
had to change build command to npm run ghbuild instead of npm run build by default & changed publist directory to build instead of dist .
similarly for other too few settings have to be tweaked to get them to work.
Hopefully this solves your issue


Question about support Sonoff RF

Will support for Sonoff RF be implemented?

Do you have one? Does not work?

I do have one. It doesn't work.
I think I can see the device in .sonoff.json but nothing in HA.

What version of firmware? Is it the last?

I tried it with (i think) 2.6. didn't work, so I updated it to 3.3. Still no joy.

3.3 should work. There may be problems with the multicast in the network.

I have other switches that work, mostly sonoff mini
FYI I previously had it working with this....
https://github.com/peterbuga/HASS-sonoff-ewelink
but I have since removed that.

Try on / off device with physical button.
Are all devices connected to the same router?
I have Sonoff Basic without RF. It works.

yes. All to same router.
The device works fine with ewelink.
like I said, it shows in .sonoff.json

Hi,
I seem to have a similar issue. It shows I think in HA under remote.rf but don't know what to do with it? none of the buttons or sensors connected to it via ewelink app got transferred.

@miketree1 try power on/off your device. And switch on/off with button or RF remote.

@maciak Go to this issue #18

I've just added a TH16 to ewelink, and that show in HA.

"@miketree1 try power on/off your device. And switch on/off with button or RF remote."
Tried that. Still nothing.

The device details are definitely in the .sonoff.json file. But somehow not being pulled trough to Entities

Is there any debugging I can do?

This does not appear to be working. I wish I could tell you what I did, but I can't. Maybe it was working all along and I was getting confied with IDs.
Anyway, thanks for your help.

@miketree1 you can enable debug log and check your device there.


Removed duplicate string; updated German translation

@olivierchatry It already exists further down in the same file.


Any ports this effects module for standard middleware in original Redux?

Hello. This is really interesting module, but works only with Angular2. Will be better have some provider for creating a middleware for original Redux. Any advices?

No, there are no plans to port ngrx/effects to Redux.
However, redux-observable/redux-observable is very comparable to ngrx/effects. I highly recommend taking a look at that project.


Fix telemetry typos

Remove startPerformanceMeasuremeant that is a duplicate of startPerformanceMeasurement.
Rename astsAsyncMeasurement -> atsAsyncMeasurement.

Codecov Report

Merging #5868 (6940564) into dev (81d34b4) will decrease coverage by 1.05%.
The diff coverage is 73.46%.

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

Flag
Coverage Δ

msal-angular
96.61% <100.00%> (+0.10%)
:arrow_up:

msal-browser
?

msal-common
84.16% <81.61%> (-0.38%)
:arrow_down:

msal-core
?

msal-node
78.46% <58.82%> (-4.93%)
:arrow_down:

msal-node-extensions
?

msal-react
94.82% <100.00%> (+0.14%)
:arrow_up:

node-token-validation
86.68% <50.00%> (-2.20%)
:arrow_down:

Impacted Files
Coverage Δ

lib/msal-angular/src/msal.broadcast.service.ts
100.00% <ø> (ø)

lib/msal-angular/src/msal.module.ts
100.00% <ø> (ø)

lib/msal-angular/src/msal.navigation.client.ts
92.85% <ø> (ø)

lib/msal-angular/src/msal.redirect.component.ts
100.00% <ø> (ø)

lib/msal-angular/src/msal.service.ts
100.00% <ø> (ø)

...msal-common/src/cache/entities/CredentialEntity.ts
87.50% <ø> (-4.66%)
:arrow_down:

lib/msal-common/src/error/ClientAuthError.ts
67.92% <0.00%> (-17.34%)
:arrow_down:

...src/telemetry/performance/StubPerformanceClient.ts
88.88% <ø> (-1.59%)
:arrow_down:

lib/msal-common/src/utils/ProtocolUtils.ts
80.76% <0.00%> (-2.57%)
:arrow_down:

lib/msal-node/src/config/Configuration.ts
100.00% <ø> (ø)

... and 38 more

... and 154 files with indirect coverage changes

There was something we implemented to try both spellings when we rolled back the original change. Can we also revert that to only use the correct spelling?
The changes in this PR are what I'm referring to

I have addressed that already here https://github.com/AzureAD/microsoft-authentication-library-for-js/commit/b03430c5841e5caad33d0b928b160af9f1535708#diff-5c2e3cc078d85a3fa0eb4b4596ecee70cc03689d6ac7ab96c69b8628fd8e055fR233.


Document the mapping between OT features and fontspec options

Sometimes it is hard to tell what option used to control what feature. I think a table would suffice, until we have a way to report such info from fontspec directly.

Sorry for the slow turnaround on this :) Finally happening in the next release of the docs.


RFM - vim-tux

Current Maintainer

[x] I am the maintainer of the package and wish to pass it to someone else;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details
Package URL: https://community.chocolatey.org/packages/vim-tux
Package source URL: https://github.com/pauby/ChocoPackages/tree/master/automatic/vim-tux.install

NOTE: The vim-tux package hasn't been updated in a while and there is no source for it. There are also two other packages:

vim-tux.install
vim-tux.portable

hello, @pauby , i see you're the current maintainer of the package, vim-tux, do you want me to take them ?

I am maintaining the package Closing this request.


uncamelify error message for GA invalid settings

now that we're surfacing errors, ugly ones are the worst :)
@f2prateek

lgtm


check: automatic check does not record results

The automatic check which is / should be triggered by releasing a PR does not record the result as comment to the PR.

fixed by using pull_request_target


Implemented new calculation for the first draft. (Parser)

Issue: odota/web#2104

Finally! Was able to remove the extra changes from previous PR. Learned something new about using Git today, thanks!


Optional event parameters are marked as required in the generated SDK

Hey,
I see this is the first issue created, so just wanted to say that we really love your product 👍🏻
I am having a small issue though. I created an event with some optional properties, but when SDK is generated they are typed as required, so typescript throws error when I pass undefined.
I pulled the latest tracking plan and you can see the type of the property is not required:

But then in the code it is typed as a string NOT as a string | undefined

This is really confusing. Am I doing something wrong?

Hi @mptorz thanks for the kind words. I'm glad you are enjoying the product.
I believe the issue that you are seeing is due to a bug with the JSDocs in the generated tracking library.  We have fixed this issue and have it included in our next major release which is targeted for early next week.
In the meantime there are a couple of workarounds.

This bug only exists in the "Browser/Javascript" runtime. If you are using Typescript you could change your runtime to "Browser/Typescript" instead and re-run itly pull to get the new TS tracking library sans errors and with strong types.
The error is caused by the JSDocs in the the generated ./src/itly/index.js. You can (temporarily) fix it by modifying the JSDoc to correctly mark the property as optional by enclosing the property name in brackets. Note that each time you run itly pull this file will be overwritten and you will lose your edits.

  /**
   * Event w optional properties description
   *
   * Owner: Test codegen
   * @param {Object} properties The event's properties.
   * ...
   * @param {string} [properties.medicine] Name of the medicine
   */
  sessionCanceled(properties) { ... }

I will provide an update when we release the next version of the SDK with the fix.
If you would like early access to the new SDK please contact me directly and I can enable it on you account.
Best,
Justin
<EMAIL_ADDRESS>

Hey, thanks a lot for a quick resolution :)
Someone else was setting up our tracking plan and I missed the fact that the runtime was set to Javascript instead of Typescript. Everything works fine after we changed it to Typescript 👍🏻


declare offsetPct with var

This bit of code is causing a runtime error in strict mode, as offsetPct is not declared at the assignment statement.
https://github.com/zurb/foundation-sites/issues/8900

Please can this change get merged into the develop and ultimately the 6.2.4 branch. This bug has literally got me editing my node_modules directory and manually making this change in order to have the slider successfully build and work properly.

Should have been fixed with the merge of #8948.
Can you confirm this?

@Owlbertz Confirmed, that resolves the issue, thanks!


Generic improvements

Description

Checklist

[ ] Targeted PR against correct branch.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit tests.
[ ] Re-reviewed Files changed in the Github PR explorer.

Codecov Report

Merging #21 into master will increase coverage by 0.47%.
The diff coverage is 34.48%.

@@            Coverage Diff             @@
##           master      #21      +/-   ##
==========================================
+ Coverage   63.33%   63.80%   +0.47%
==========================================
  Files           6        6
  Lines         420      420
==========================================
+ Hits          266      268       +2
+ Misses        115      114       -1
+ Partials       39       38       -1

Impacted Files
Coverage Δ

x/staking/types/delegations.go
0.00% <0.00%> (ø)

x/staking/types/validator.go
0.00% <0.00%> (ø)

database/auth.go
78.72% <100.00%> (ø)

database/staking.go
77.93% <100.00%> (+0.71%)
:arrow_up:

Continue to review full report at Codecov.

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


Add test dependencies

Install jest and babel for DOM manipulation
Test the function to check for number of comments

Hi @Booze33  👏🏽,
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Highlights✅✅

Linters are properly setup👍

The test are working great!

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.


fix: adjust contextmenu onclick params to get element

adjust contextmenu onclick params

before
 onClick?: (value: string, target: HTMLElement) => void;

after
 onClick?: (value: string, target: Element) => void;

There are some users who use this parameter normally, and the necessity of this PR and its impact on users need to be considered.


Adding my name

Registering on the ALTSCHOOL repository

Hello! As per the instructions on the readme, please create a working branch different from the main branch. Thank you


can't kekulize molecule from those smiles

Ive got this error message for the folllowing smiles:
Can't kekulize mol
Could not sanitize molecule on line 2435
Can't kekulize mol
188, 'n2c1c(cccc1)nc2c3ncsc3'
316, 'n1cccn1'
333, 'n1cnnc1'
364, 'n1cc(nc1)C'
393, 'n1cnnc1N'
463, '[O-]N+c1ncnn1'
477, 'n1ccnc1C'
746, 'n1c2ccccc2nn1'
1131, 'n2c1ccccc1nc2C'
1229, 'c1cccc2c1c(cn2)C'
2241, 'N#CN\C(=N/C)NCCSCc1ncnc1C'
2436, 'O=C(O)C@@HC(C)C'
I try to generate rdkit descriptors using the following code on rdkit 2015.03.1 version:
from rdkit import Chem
import cPickle,sys
from rdkit.ML.Descriptors import MoleculeDescriptors
from rdkit.Chem import Descriptors
nm=[x[0] for x in Descriptors._descList]
calc = MoleculeDescriptors.MolecularDescriptorCalculator(nm)
suppl = Chem.SmilesMolSupplier('/Users/mbp/Documents/abraham.txt',titleLine=False)
w = open('rdkitdesc.txt',"wb")
errors = open('rdkiterror.txt',"wb")
line = "Molid"
for names in nm:
line = line+ ","+ names
line = line+"\n";
w.write(line)
nDone=0
for mol in suppl:
nDone += 1
line = str(nDone)
if mol is None:
errors.write(line + ",Error\n" )
continue
descrs = calc.CalcDescriptors(mol)
for dec in descrs:
line = line+ "," +str(dec)
line = line+"\n";
w.write(line)
w.close()
errors.close()
Here the complete dataset 2465 molecules work over the complete list of 2674 molecules:
http://onschallenge.wikispaces.com/file/view/20120528CuratedData.xlsx/341079416/20120528CuratedData.xlsx
but CDK can generate the correct molecule.

These are the result of a bad/ambiguous aromatic SMILES from the CDK (I think this has been fixed in recent versions). Googling for the error message will turn up multiple mailing list descriptions of the problem and how to resolve it.


bower reports v1.1.2 as latest version is 1.2.0

Is the library at v1.2.0 now? If so, there's missing github release, that causes bower to fetch only v1.1.2.
Try
bower info pouchdb-collate
Prints
Available versions:
- 1.1.2
- 1.1.1
- 1.1.0
- 0.2.0

fixed, but I'd recommend trying out npm instead of bower, at this point I see bower as an annoyance I'm occasionally forced to interact with.


:four_leaf_clover: Proposal: More Friendly Quickstart Config Template Output

What Would You Like to Add? Why is This Needed?

Quickstart, the simpler and easier to get started, the better. So why not have dtm show config output a quickstart.yaml by default so that users can get started with as little configuration as possible?
Design
dtm show config --template=quickstart -> print quickstart.yaml
Let --template flag default value == quickstart, so dtm show config will also print quickstart.yaml

Approved. Easier for new users to get started without having to copy-paste from the docs.

Thanks @daniel-hutao and @prodanlabs for the discussion in our group and for raising this issue.

/assign @iyear Would you please help to implement this feature?

ok, I will work for it.

What is the template flag used for? There is already a plugin flag. @daniel-hutao

@iyear the plugin flag is used to print config with one specified plugin. The template flag is used to print some combined configs like quickstart/gitops/chatops/...

@daniel-hutao The template files are currently in the examples directory, but show/config/embed.go cannot read the files in the parent directory.
So the directory planning still needs to consider.

@iyear I'm sorry, This proposal depends on #655, so there are some obstacles when you implement it before. But after #655 merged, I think this proposal is simple enough, let's leave it to NEW

Kindly to ask if there is somebody working for this issue? If not, I'm glad to undertake it.
/cc @daniel-hutao

@Thor-wl You got it!!!
Enjoy it!


Unexpected failure parsing device information from adb output

$ flutter start
Unexpected failure parsing device information from adb output:
emulator-5612          host features:shell_2
Please report a bug at https://github.com/flutter/flutter/issues/new
Unexpected failure parsing device information from adb output:
emulator-5644          host features:shell_2
Please report a bug at https://github.com/flutter/flutter/issues/new
Building APK...
...

Command works in the end, just tells me to report an issue.

Weird part is I never started any emulators...


Initial attempt at Android alert areas

Here's the Android version. I did my best to adhere to the flow of iOS. There were a few differences. Android didn't seem to have an origin, so I didn't use that(pointInsideWithOrigin can be safely deleted, but I wasn't sure if we'd need it). Also, on iOS reportAlertEvent happened inside setCenter, but there was no real setCenter on Android that I could find, so I put the call inside onAnimationFrame. Not sure that was the best place, but it seemed right and it seems to work properly. Let me know if anything seems off.
Oh, and I changed setFrictionAreas to write to a frictionAreas array, instead of gravityPoints, though neither of those seem to be consumed anyway

Looks great! Thanks for the contribution :)
There's no set center in Android since we get the coordinates we want using getTranslationX() and getTranslationY().