Build failing while installing Linenoise

Hi there,
I am trying to install Linenoise in this docker image.
But the build is failing and throwing this error:
root@6c2270698b37:/# zef install Linenoise
===> Searching for: Linenoise
===> Searching for missing dependencies: LibraryMake
===> Searching for missing dependencies: Shell::Command
===> Searching for missing dependencies: File::Which, File::Find
===> Building: Linenoise:ver('0.1.1'):auth('Rob Hoelz')
===> Building [FAIL]: Linenoise:ver('0.1.1'):auth('Rob Hoelz')
Aborting due to build failure: Linenoise:ver('0.1.1'):auth('Rob Hoelz') (use --force-build to override)
  in code  at /usr/share/perl6/site/sources/A5C13EF833A22B1E85E78D1CD88BE1B0B0B24DA1 (Zef::Client) line 331
  in method build at /usr/share/perl6/site/sources/A5C13EF833A22B1E85E78D1CD88BE1B0B0B24DA1 (Zef::Client) line 296
  in sub  at /usr/share/perl6/site/sources/A5C13EF833A22B1E85E78D1CD88BE1B0B0B24DA1 (Zef::Client) line 522
  in method install at /usr/share/perl6/site/sources/A5C13EF833A22B1E85E78D1CD88BE1B0B0B24DA1 (Zef::Client) line 633
  in sub MAIN at /usr/share/perl6/site/sources/31FF9E8498781BF95868DF98AD861FCF1211BCC1 (Zef::CLI) line 152
  in block <unit> at /usr/share/perl6/site/resources/0031ECC6124802A80F7B7B073C50D0A1694E3020 line 1
  in sub MAIN at /usr/share/perl6/site/bin/zef line 2
  in block <unit> at /usr/share/perl6/site/bin/zef line 2

I couldn't make out anything from error message.
Any help!!
Best,
Suman

It got resolved after installingapt-get build essential.
This was the dockerfile.
FROM debian:sid
WORKDIR /

RUN apt-get update \
  && apt-get install -y --no-install-recommends build-essential \
  git nano libzmq3-dev libreadline-dev ca-certificates\
  && rm -rf /var/lib/apt/lists/* \
  && git clone https://github.com/rakudo/rakudo.git \
  && cd rakudo && perl Configure.pl --prefix=/usr --gen-moar --gen-nqp --backends=moar \
  && make && make install && cd .. && rm -rf rakudo \
  && apt-get purge -y --auto-remove build-essential

RUN git clone https://github.com/ugexe/zef.git && cd zef && perl6 -I. bin/zef install . && cd .. && rm -rf zef
ENV PATH /usr/share/perl6/site/bin:$PATH

CMD ["perl6", "--version"]

Now I am going to modify the docker image.
Thanks,

@sumandoc Since you worked out the problem, is it ok to close this issue?

@hoelzro Yes.
I would highly recommend to mention the exact dependencies of Linenoise in README file, because I get it to working after installing apt-get install build-essential which of course brings many unwanted tools along with it.
Best,
Suman

@sumandoc Thanks for the tip - I'll add a notice about build-essential to the README.  UTF-8 support is present in a branch (see #20), but I need some features in LibraryMake in order to merge it.


Issue #104-Export-SimpleWebAuthn-types

This PR fixes #104.
Description:

exporting the options and response types

Notes:
I initially changed the functions to named imports, instead of default exports, but then changed my mind since these are just type exports
Tested using yarn link with the example application. So run yarn link in ./packages/server and then yarn link "@simplewebauthn/server" in the root of the example app.
Then in the example app, change the code to look similar to this:
  let verification: VerifiedAttestation;
  const verificationResponseOptions: VerifyAttestationOptions = {
    credential: body,
    expectedChallenge: `${expectedChallenge}`,
    expectedOrigin,
    expectedRPID: rpID,
  };
  try {
    verification = await verifyAttestationResponse(verificationResponseOptions);
  } catch (error) {
    console.error(error);
    return res.status(400).send({ error: error.message });
  }

After this is merged, we will be able to import these types like so:
import {
// Registration ("Attestation")
generateAttestationOptions,
verifyAttestationResponse,
// Login ("Assertion")
generateAssertionOptions,
verifyAssertionResponse,
GenerateAssertionOptions,
VerifiedAttestation,
} from '@simplewebauthn/server';

derp, types dont export the same as interfaces

After a bit more thought I think you should use Opts instead of an additional Options in my comments above:
Ex:

GenerateAssertionOptions -> GenerateAssertionOptionsOpts
VerifyAssertionOptions -> VerifyAssertionResponseOpts

These changes have been published in<EMAIL_ADDRESS>


Slow Loading Icon in the App File

Description
I love the App Files, which include the column view like MacOS (great to manage huge folders). The UI is even more native than the current File Explorer of W11. The only thing that it has the loading even when reload the old folder or running in the background for indexing

Steps To Reproduce
Just open and use it
Requirements

e.g. Fix crash that occurs when...
Not showing loading

Files Version
3.3.
Windows Version
22631.3527
Log File
debug.log

Thank you for your feedback

How long does it take for the thumbnails to load?
Where are these items located (eg local drive, OneDrive etc)?

It is local, C and D. It takes like 1 2 sec only but we can see the
animation and how annoying it is
--
Marcus / Manh Quyet (Mr.)
Phone    : (+84) 949 279 995 - Viber/Zalo/Telegram
On Mon, 6 May 2024 at 20:56 Yair @.***> wrote:

Thank you for your feedback

How long does it take for the thumbnails to load?
Where are these items located (eg local drive, OneDrive etc)?

—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/15311#issuecomment-2096167144,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVPX65EPWN6N3OBG5Z6HCL3ZA6HH3AVCNFSM6AAAAABHIROYYOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJWGE3DOMJUGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Improving the performance is high on the todo list, the struggle is balancing performance without affecting stability (loading too many thumbnails at the same time can crash the app). We're currently tracking this in #4180, please subscribe to that issue to be notified when further improvements are available.


added option --adapter to dbt init, to create sample profiles.yml bas…

resolves #2533
Description
Added option '--adapter' to dbt init, to create sample profiles.yml based on chosen adapter.
It gets the sample profiles.yml from the adapter 'includes' folder
Related PRs:

https://github.com/fishtown-analytics/dbt-presto/pull/26
https://github.com/fishtown-analytics/dbt-spark/pull/98

Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

I think the test is failing because the container "fishtownanalytics/test-container:7" doesn't have the samples_profiles.yml on the postgres folder. How can I change that?

I think the test is failing because the container "fishtownanalytics/test-container:7" doesn't have the samples_profiles.yml on the postgres folder. How can I change that?

I might be wrong about this, but I think the issue is that the setup.py files for each plugin needs to add the path to the sample_profiles.yml to package_data. This gets me basically every time I add a static file to dbt.

@beckjake Thanks for the heads up! It all worked now. Ready to be reviewed, I think!

@brunomurino I (finally) got the branch this depends upon merged into dev/marian-anderson. Could you change the target branch for this PR to dev/marian-anderson please?

@brunomurino This is a great change! I have a few comments around keeping the sample profiles consistent between the different adapters.

The top-level profile name should be default for all of them, so that it matches the profile name specified in the starter project and just works from the get-go. I dislike that the starter project has profile: 'default' and name: 'my_new_project', but I think we would need to write a bunch more code to make that dynamic.

Let's include bracketed entries (password: [password]) instead of fake entries (password: pa55word)

Let's always include two outputs, dev and prod. I like how it demonstrates the options and extensibility available, especially for plugins that have multiple connection methods.

Postgres: dev and prod, with different usernames ([dev_username] and [prod_username]), passwords, and schemas
Redshift: dev (password-based), prod (iam)
Snowflake: dev (user-password), prod (keypair)
BigQuery: dev (oauth), prod (service_account) on BigQuery

I'm curious to hear what you think about the above. I'm flexible on the specifics, but definitely set on wanting consistency here and for the plugins as well (fishtown-analytics/dbt-spark#98 + fishtown-analytics/dbt-presto#26)

I agree and have updated the sample profiles!

@jtcohen6 Updated the sample profiles on spark and presto plugins as well, let me me know what do you think.

@jtcohen6 No worries at all and sorry for the delay. Feel free to give any more suggestions!

@jtcohen6 Happy to help! I also checked the presto-spark repos and they're already in line with your suggestions on this repo!


Add reset password route

Password reset is difficult to implement securely without the presence of a front-end to use an email link


Support Qt in namespace

SortFilterProxyModel doesn't currently support Qt built with the -qtnamespace configure option

Thank you for this PR

Sorry, I've rushed accepting this one.
It seems that the forward declaration is not working when using a non namespaced Qt. You have to use QT_FORWARD_DECLARE_CLASS instead.
But are you sure that there is not anymore thing to do to support a namespaced Qt? Shouldn't there be QT_USE_NAMESPACE or QT_PREPREND_NAMESPACE all over the place when referencing Qt classes?

Apologies.
Well, it works for me like it is. QT_USE_NAMESPACE is declared in qglobal.h, so I don't think that's necessary.


Revert "feature/elementwise_check (2)"

Reverts stan-dev/math#1798
This pull request was causing noticeable slowdowns in the release.
We should try to figure out how to make these changes without incurring the slowdown.
See: https://discourse.mc-stan.org/t/cmdstan-2-24-release-candidate-now-available/16818/39

The tests aren't quite right on check_finite here. Will get back to this tomorrow

Thanks!


Upgrading GCF runtime versions

Fixes #642

@ddixit14 Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@ddixit14 Thank you for signing the Contributor License Agreement!

Update README to deploy to --runtime java17, otherwise LGTM.

Update README to deploy to --runtime java17, otherwise LGTM.

LGTM


MonitorUtils instance is strongly referenced in WeakReference<>.Event…

WeakReference<MonitorUtils, DisplayInformation, object>.EventAction strongly references MonitorUtils instance.

Great fix


Display error messages with separate component

I added a new component to display error messages.
The error message is stored within the state - so I added the respective actions and reducer to do so.
The overall process is like:

Set the error message by dispatching the "setErrorMessage" function
the action will set the error message state property and perform the "shake" animation to the form
also, the action will set a timer to remove the error message after 3 seconds.

I'm sorry about the whitespaces - I fixed it now.


Adding stream management API

Adding shiming for some legacy API methods related to stream management: getLocalStreams(), getRemoteStreams(), getStreamById() and removeStream().

good to know

We should add deprecation warnings to all these, or this won't drive convergence toward the spec. Adapter's main mission for a while now has been to eventually disappear.

Is there enough usage of getStreamById() to warrant polyfilling?

getStreamById will be gone in Chrome 62, and that removal is possible because the usage is super low: https://www.chromestatus.com/metrics/feature/timeline/popularity/1645

We should add deprecation warnings to all these, or this won't drive convergence toward the spec. Adapter's main mission for a while now has been to eventually disappear.

Sounds good, I would prefer doing it as a follow-up PR to also add warnings to navigator.getUserMedia and callback-based APIs.

getStreamById will be gone in Chrome 62, and that removal is possible because the usage is super low: https://www.chromestatus.com/metrics/feature/timeline/popularity/1645

I am fine killing it. I would prefer doing it sometimes in this summer.

Added some warning messages  at https://github.com/youennf/adapter/commit/bdf41a26a76fd5e46734bc1762560989cf998c21

Can the PR be merged as is?


Increase the dropdown height to enable a "peek" at the next option again

Years ago I set the max-height of a blockly dropdown to be 250px so that we would get a "peek" at the next option, if there were one, so that users would know it was scrollable on systems that don't have scrollbars.
Over time, this size seems to no longer show a peek, and so here's a change to bump it to 265px:

This change has broad impact, so we should consider its adoption carefully.
This fixes https://github.com/code-dot-org/dance-party/issues/260.

Eyes test might be the easiest?

Should we consider as a longer-term fix making the scrollbar be visible?

@Hamms If we use native scrolling on modern operating systems I think it's normal to not see a scrollbar.
The 265px value was chosen because it is just a little bigger than the existing 250px, and is big enough to get a peek at the next value that it incentivises a scroll.

I'm pretty sure it's only osx that hides the scrollbar; and normal or not, it's clearly causing confusion

Fair point.  Good news is that peek seems a reasonable mitigation for now!
If we're doing an eyes test I guess I'll wait until blockly is updated in our main repo.  (I did ponder a PhantomJS test in this repo to make sure that the dropdown max height is not close to a multiple of the item height, but that seems a little... obscure. :)

Related bug? https://github.com/code-dot-org/blockly/issues/148

Separately, it might also be helpful if the dropdown menu took up more space (height) on the screen to reduce times when the user would have to scroll.


Documention Improvements

Documentation ( If Missing libraries and tools )

CMake >3.17 needed for meson
wget -O - https://apt.kitware.com/keys/kitware-archive-latest.asc 2>/dev/null | sudo apt-key add -
sudo apt-add-repository 'deb https://apt.kitware.com/ubuntu/ bionic main'
sudo apt-get update
sudo apt install cmake
sudo apt install cmake-curses-gui  # Recommended, includes ccmake.

While running ./build-third-party.sh

While compiling add missing lib/headers
sudo apt-get install -y gnutls-bin libgupnp-igd-1.0-dev libgirepository1.0-dev libgnutls.*-dev

In Get Started add "cd pear" step
git clone --recursive https://github.com/sepfy/pear
cd pear
./build-third-party.sh

libnice is no longer used


send help merge conflicts aaaaaaaaaaaaaaaaaaaaaaa

aaaaaaaaaaaaaaaaaaaaaaaaaaa!!!!!!!!!!!!!!

already merged

already merged

why  it's reopened

because github didnt merge that, meaning i'd need to resolve myself

because github didnt merge that, meaning i'd need to resolve myself

how can i help u?


"Go to Definition" and "Go to Type Definition" should not open file (with definition) if that file is already opened in editor

VSCode Version:
Version 1.24.1
Commit 24f62626b222e9a8313213fb64b10d741a326288
Date 2018-06-13T17:51:32.889Z
Shell 1.7.12
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64
OS Version: Windows 7

Steps to Reproduce:

open file A, which uses definition from another file B, in left editor
open file B in right editor
open (or create new empty) file C in right editor where B is
set focus on file C, so file B is not active/focused
in A file in left editor, try "Go to Type Definition" of something from file B
file B is opened again in left editor, but it should focus it in right editor where it is already opened

So, guess would be, this bug happens only if file-that-uses-definition is in another view than file-with-definition and file-with-definition is not active/focused.
Does this issue occur when all extensions are disabled?: Yes
Animation:

Sometimes similar thing happens when clicking file in explorer. It opens clicked file in left editor, although it is open and visible in right editor.


Ideas for major changes in Catalogue API v3

This is a ticket to track any ideas for breaking changes we might like to make in a v3 API. It's not for discussing the changes or making a final decision on what we'll change, but a place to write them down until that API version eventually arrives.

The Location model takes a List[AccessCondition], but in practice we never have more than one – this could be an Option[AccessCondition]. We might also consider renaming this field to something like Option[Access] to reflect the fact that it includes access data that doesn't impose any restrictions on the viewer.
Should the physicalDescription field be a List[String]? See discussion on https://github.com/wellcomecollection/catalogue-pipeline/pull/1784
Ditch the single-letter Sierra codes for workType/format/whatever it's called
Bin our bespoke querying syntax in favour of URL syntax, e.g. workType=books&workType=manuscripts instead of workType=books,manuscripts

The partOf list now contains two fairly distinct types of relationship

The single-parent hierarchy of CALM and TEI records
Series links from Sierra

The original use (single parent hierarchy) has always been a list, but probably shouldn't be.
It might be a good idea to split these two uses into two distinct properties.  hierarchy, (probably containing just one value), and something else.

Currently availabilities and alternativeTitles are both presented on every response, rather than behind an include.
This is inconsistent with how we do other list-able fields; we should decide if this is appropriate and:

Change it if we decide it's wrong
Document it if we decide it's right

Quoting Jonathan on https://github.com/wellcomecollection/docs/pull/81#discussion_r883712183:

If [the concepts work] evolves into a Catalogue v3, then it might be a good chance to simplify the slightly cumbersome xxxType properties and give them more specific names. My suggestion here would be identifierType to scheme.

Change the way we deal with Composite Concept Subjects.
Currently, a Subject is kinda like a Concept, and has an id that corresponds to the subject as a whole.  It then also has a list of Concepts.  If the list of Concepts is exactly one, then the subject as a whole, and the sole Concept are essentially the same thing.  If the list is bigger than one, then the subject is one thing and the concepts are another.
I think it would be better/more consistent, if the subject did not have its own id, and that the first concept in the concepts list always represents the whole subject concept, regardless of how many subconcepts it is composed of.
This would be similar to contributors, which have a single agent but no id of their own.


Feature/allow backup cronjobs to be run by a k8s cronjob ams cp

Description
This PR adds a new entry point (backup.js) to run backups from. This allows us to create a kubernetes cronjob (see https://github.com/openstad/openstad-kubernetes/pull/101) to perform the backups outside of the API pod.
Type of change
New feature
Tests
Tested locally

Note: I think this should be removed from the API and maybe setup as a independent service. Will add that to the Headless todo list.

Agreed! But for now this is a workable solution.


Add ability to download file from file manager

Background
Now we can only upload to flipper
Changes

Add ability to download from flipper file to android

Test plan
Try download file from flipper

Good work


Fix typo in Readme

I have read the CLA Document and I hereby sign the CLA

Thank you for the PR but this actually isn't a typo. The word callee here is correct.


Create davpsvelacherychennai.txt

DAV PS Velachery

@gunsnrifles lib/domains/davpsvelacherychennai.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@gunsnrifles Please check my previous comment to your request and submit a new request with correct file name. Regrettably, it's impossible to guess now what is the domain you wish to add to the trusted list. I have to close this request.


Fix license header in core files

Minor code grooming

Note that CI is red

I know
I don't understand why fmt blames if file has an inner attribute right after /** comment. Maybe I have to change license header format to /* thoughout.
BTW I can add headers to redis-rs files - this PR is a right place. Can/should we do that?


Kklopfenstein/docker in docker azure dns override

gh pr checkout 210

This has already been merged in (https://github.com/devcontainers/features/blob/main/src/docker-in-docker/install.sh#L14).  Why did you reopen this?


Presets (prefabs?) for the easy re-initialization of an entity pattern

As the old README.md stated, we might want to:
Presets (prefabs?) for the easy re-initialization of an entity pattern

I'm not sure what's wrong with the current implementation, or what we'd want to change.
If @paked could clarify, or close this issue, that'd be great.

@EtienneBruines Unity has a system where one can create an Entity and the re initialise it multiple times into new entities. I think a similar system would be pretty cool. I imagine something liek
e := NewEntity([]string{"RenderSystem"}) // Create template entity
// do things to entity

es := e.Duplicate(20) // fill a group with 20 different e's

w.Add(es)

This should be added after groups.

As groups is something the user should probably implement himself, we could allow this:
e := NewEntity([]string{"RenderSystem"}) // Create template entity
// do things to entity

es := e.Duplicate(20) // fill a group with 20 different e's

w.Add(es...)

Where the Duplicate function indeed duplicates the Entitys and all Components? Is that something you'd like to see, @paked?

Closed this. Discussion/development should continue at https://github.com/EngoEngine/ecs/issues/5


09/17/2018 01:38 : Create Pull Request for branch master -> prTesting1_2018091701

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


large submission handling

I am trying the submit about 32500 cells to your public beta, but have not been able to do so. I have received the following error messages

[18:21:55.397] Server returned status code 500, Detail: Vector Search returned a match with 0 neighbors. * [18:21:55.397] Resubmitting chunk #66 (32500, 32555) to CAS ...

I removed chunk 66 (the final chunk), only to receive this error

[00:03:05.038] Unexpected error: HTTPError; Message: Server returned status code 413, Detail:

413 Request Entity Too Large

Despite the cell number being within the 50,000 cell weekly quota. Let me know what I can do to move forward!

Hi @charlottedibiase, thank you for submitting this issue. Would it be possible for you to share your AnnData with a member of our team for further investigation? At the same time, @fedorgrab @KevinCLydon, could you please look at the sentry logs and investigate? Thanks.

Hi @charlottedibiase,
Thank you for reaching out to us and participating in our Beta release! I've reviewed your data, and it seems you are submitting normalized data, which is likely the cause of the errors. CAS only accepts raw counts and performs all the necessary data normalization under the hood.
This is a great use case for us, as it would be helpful for CAS to notify users when normalized data is submitted. I've created an issue for this validation check (#103).
Please let us know if you’ve tried using CAS with raw counts and whether it works as expected.

@charlottedibiase
Side note: I suspect we may be encountering the 413 Request Entity Too Large error because normalized data chunks are often larger than those with raw counts. This might vary depending on the normalization strategy. If, in your case, all zero values are converted to non-zero values, the chunk size could increase significantly.


luci-mod-network: add basic bonding features

This already works well with current netifd. Fully featured this will make luci-proto-bonding obsolete

ping

currently of no interest to anyone.

Well, I was interested but it really got no attention, and this is why most times OpenWRT is missing features...

One could address the criticism and force push the recommendations.
Or ping jow.

I'm infinetly grateful to be part of the project and such little gems. Always working the socks off to give a good advice.

Yeah same, of course, from my side, constructive criticism, I said it because I've been there, making a a pull request and waiting 2 years for no one to do anything, btw, still broken today...

@jow- @plappermaul please merge this so one can manage LACP on rtl based switches.

Well, I was interested but it really got no attention

I requested the change of bonding into bond which didn't happen unfortunately.

Yes, I see from the history - is this the only criticism that this PR must undergo? If so, and @plappermaul fixes it, we can have a bonding GUI....


Support for nested lists?

Does this package support nested lists? I have a  nested inside a  and in the rendered PDF I see that the bullet points are positioned in front of the unordered list items as expected but they are all on a single line. I can provide code and output if you would like.

I don't think nested lists have been tested yet, but that shouldn't be very hard to solve. PRs welcome :-)
There may be a way to do a temporary fix for this with a stylesheet.

As a temporary fix you can add some padding-left to the nested list. For example:

...

...

...

I'm also having the same issue. Puts the nested list's items inline. I can't seem to get CSS to jerryrig it either.

@jacoblippincott this should be fixed in the latest version v1.1.13. It was caused by rendering list item content as inline/text


Improve the error message when source resolver is not resolving

Codecov Report

Merging #1152 (4b81993) into main (c428817) will increase coverage by 0.22%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1152      +/-   ##
==========================================
+ Coverage   69.99%   70.21%   +0.22%
==========================================
  Files         124      123       -1
  Lines        6243     6269      +26
==========================================
+ Hits         4370     4402      +32
+ Misses       1468     1463       -5
+ Partials      405      404       -1

Impacted Files
Coverage Δ

pkg/apis/build/v1alpha2/image_lifecycle.go
0.00% <ø> (ø)

pkg/apis/build/v1alpha2/image_types.go
50.00% <ø> (ø)

pkg/reconciler/image/reconcile_build.go
91.48% <100.00%> (+4.10%)
:arrow_up:

pkg/apis/build/v1alpha1/source_resolver_types.go
0.00% <0.00%> (ø)

pkg/apis/build/v1alpha2/source_resolver_types.go

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


ContentSize bug with zoom

ContentSize seems broken when image is zoomed, especially in fullscreen with landscape pictures.
I'm pretty sure a5e56615b7df6edc42d4bcf0865f4d3976514158 has something to do with it, but I think I already encountered this bug before.
To reproduce: run the example, go fullscreen, double tap and swipe up or down

Thanks for the report, I fixed the issue in the latest 1.3.2 version. Please check it and let me know if you spot any further issues.


ctrl-alt-\ is ignored

Version
Microsoft Windows [Version 10.0.19042.1526]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Debian GNU/Linux 11 (bullseye)
Other Software
Windows Terminal
Version: 1.11.3471.0
Repro Steps

Install ncurses-examples: sudo apt install ncurses-examples
Run demo_altkeys: /usr/lib/ncurses/examples/demo_altkeys
Type ctrl-alt-\

Expected Behavior
Something like: 0.259 Keycode ???, name ESC-^\
Actual Behavior
Nothing
The key sequence is recognized however by PowerShell by doing alt-shift-? followed by ctrl-alt-\
Diagnostic Logs
No response

@jeffchase this appears to be related to Windows Terminal and not WSL. Would you mind filing this issue there?
https://github.com/microsoft/terminal/issues
Thanks!

Thanks. The reason I filed it here is because ctrl-alt-\ does work in Windows Terminal when using PowerShell but it still does not in Command Prompt using WSL.


python 3.8 compatibility

check whether python 3.8 works with current code base

solved with commit d51450e0892e8061dd7396b10162e2c8b1b4a2f7


IrisTest method subclass specialisation.

This PR is to support iris-grib testing.
We want the convenience to reuse all of the iris.tests.IrisTest behaviour but also to be able to point the iris.tests._RESULT_PATH to an iris-grib specific directory.
The problem is that iris.tests.get_result_path function (which uses iris.tests._RESULT_PATH) is local to that module, and also that this module level function is referenced by several iris.tests.IrisTest methods.
Currently, the only way to re-direct iris.tests._RESULT_PATH somewhere other than the IrisTest default, is to overwrite iris.tests._RESULT_PATH, which may not be desirable nor appropriate.
Defining iris.tests.get_result_path as an iris.tests.IrisTest static-method, allows light-weight sub-class specialization to cleanly reference a different _RESULT_PATH in a different module (i.e. iris_grib.tests) without clobbering iris.tests._RESULT_PATH.

@dkillick fancy a peek?

Merge when green from me 👍

@pelson Awesome, thanks!

One of the matrix items went green. I have no reason to believe they wont all go green.

Cool, thanks!


Hero block uses linkHref for it's URL field

Describe the bug
The hero block uses linkHref for the additional content to link to. This field is not supported out-the-box by the resolveuid smart field, leading to the possibility of links breaking if content is moved or renamed (Plone aliases help alleviate this, but it's still not ideal).
To Reproduce
Steps to reproduce the behavior:

Create a page with a hero block in it and another page.
Add a 'Link title' and 'Link to' and link to the other page you created
Change the short-id of the other page
See that the link still

Additional context
it is probably easier to add linkHref to the list of smart fields in plone.restapi rather than have an upgrade step due to the schema rename.

Slight aside from this issue, is there a better way to handling changes in schema other than having a backend package with an upgrade step that needs to be linked to the frontend package somehow?

I keep bringing this idea, maybe at some point the slight inconveniences becomes something major and we do something about it: https://github.com/plone/volto/issues/2252

@JeffersonBledsoe my advice would be to open a PR in plone.restapi or plone.volto with the extra field.


vacuum intents creation

Added vaccuum intents and excluded vacuum domain from generic hassturnon (as it was catching the vacuum turn on incorrectly)

@dziadeck47 This PR is duplicating most of the changes from https://github.com/home-assistant/intents/pull/1996/files isn't it?

Since there was no response at all, I'm declaring this PR stale and closing it.
Feel free to re-open this PR when you feel like that is the appropriate thing to do.
../Frenck


PORT env var

you can control the expose port via env var

To everyone who want to contribute, I've forked it and accept PR.


Remove texcoord from QUADFRAGSRC.

Fixes gl errors if drop_shadows are enabled
texcoord is unused in the rounding part of the textureshaders.
QUADFRAGSRC isn't using that variable inside the non rounding code.

What to do?

I updated it back to head

wlroots to master?

yah it fixado

Did i fix it?

no, I did it for you

What did i do wrong?
I did everything like before, but than i saw the wlroots file inside the commit and tried to take it out before you merge.

u did a git pull and forgor :skull: to do git submodule update --init is my guess

I didn't knew that (i never did that).
so i should probably do that every time i pull right?
Sorry again!

jic you ofc can, but it's only necessary when the submodule is updated in upstream

you'll notice it when your prompt says there are changes right after you git pull

Ok,
got it!


Fix icon spacing

Icons currently shown on the navbar don't have proper spacing with the text. This must also be checked if it occurs in other areas.

Icons spacing for navbar is fixed. Other areas look fine with the icons :slightly_smiling_face:


[FEATURE] Add GitHub links in the website

Is your feature request related to a problem? Please describe.
By adding links to the repository, the owner, and other information on the website, it'll make it easier for other users to find your repository in case they find the website first.
Describe the solution you'd like
Add GitHub links to the website and other important links.
Additional context
None.

There are already a few links at the top and bottom that go to different parts of the GitHub repo from the website

I'm happy to hear suggestions if you think there needs to more or different places it could be added

Ohh, right, I did not see them 😅 adding them in the footer would be great too, I guess.


[REVIEW] dhananjayandayalan

GitHub Profile Link
https://github.com/dhananjayandayalan

To get a github profile review please add a custom profile readme to your profile.
Here is a course of eddie on how to do that.
https://www.eddiejaoude.io/course-github-profile-landing

GitHub Profile Link
https://github.com/dhananjayandayalan

Consider adding readme.md named as same as your username and then try to make it look more attractive :)

Looks great, here are some suggestions https://www.youtube.com/watch?v=iqIFD02OkVE

Add more to your bio's description
Suggest pinning your favourite 2 or 4 repos
Try to do more more code reviews and pull request


Feature Request: Expose Wallet Lock State & Disconnect Event

Background
At Dynamic, we have partnered with StarkWare to build a bridge component as seen on https://starkgate.starknet.io/. We aim to create a smooth login experience for end users. We use get-starknet-core to connect users' starknet wallet, whether it's Braavos or ArgentX.
Problem
There are two scenarios that could have improved UX, and they both begin at a state where the user has already connected a starknet wallet:

The user explicitly locks their wallet while it is connected to a dapp and the dapp is open in a browser tab
The user locks their wallet while the dapp is not open (either via a browser restart, or by closing the dapp window and explicitly locking their wallet)

Additionally, when dealing with wallets in a locked state, we want to avoid uninitiated wallet popups – meaning, there should not be any wallet popups without the user explicitly clicking on a connect button.
The starknet window object (accessed via get-starknet-core) has a lot of useful properties and methods like isConnected and isPreauthorized, however these cannot be used for accurately detecting wallet locked state while avoiding popups. The main reason for this is that isConnected is always false until enable is called. However, when enable is called, it will prompt the user to unlock the wallet if it's locked, so we have a chicken-and-egg problem.
Proposed solution
There are two parts, each addressing the problems above, respectively:

Emit an accountsChanged event with undefined account address when the wallet is locked
Add a flag to enable that states the dapp doesn't want to show a popup/UI in case the wallet is locked and/or not preauthorized

Alternate solution

Emit a disconnect event when the wallet is locked
Expose an isLocked (or isUnlocked, whichever is preferred) value that represents the current locked status of the wallet, whether or not enable has been called.

Disconnect event
When the user explicitly locks their wallet, the wallet should emit a disconnect event that can be handled via the on method, similar to accountsChanged and networkChanged. This will solve the first UX scenario because in our disconnect event handler, we can disconnect the wallet from our app state.
isLocked
There should be a boolean value isLocked accessible via get-starknet-core similar to isConnected that is:

true when the wallet is locked
false when the wallet is locked

Additionally, this boolean should have a value before calling enable – it should not be necessary to call enable for this property to have the correct value. This will solve the second UX scenario because our SDK can read this value and determine if it needs to remove the wallet from its connected state. For a reproduction of the UX problem we aim to solve, see: https://codesandbox.io/s/purple-glade-l3llkm?file=/src/App.js. The problem is with the automatic popup – since we have to call enable first to check if isConnected is true, there is a wallet connection popup on page load. If we were able to check isLocked, then we would know to simply "disconnect" the user (reset their connection state in our app), and then show them the connect button again.

thanks @RyanTimesTen.

Perhaps we could get by with the existing set of events (account/network changed) and just emit, on the wallet side, an undefined account-address once the user locks the wallet?

How about adding a flag to enable that states the dapp doesn't want to show a popup/UI in case the wallet is locked and/or not preauthorized?
This way, you could call it naively once the user chooses a wallet. If it works (e.g. you got an address back), perfect; if not, you'll get undefined for the address and realize that either you aren't preauthorized or the wallet is locked.

i could see that, although a proper disconnect event is be clearer and more explicit IMO. this event could also be emitted when the user removes the dapp connection from their wallet
that would work

Isn't disconnect/connect on L1 more about network connectivity rather than the dapp<->wallet connection?
I think it's a good solution that covers your needs without exposing sensitive information about the wallet's state.

i updated the proposal with your suggestions, thank you

@RyanTimesTen wrt silent-mode, pls see https://github.com/starknet-io/get-starknet/pull/194 (RequestAccountsParameters); will be implemented as part of that PR

thanks @avimak. both Braavos wallet and ArgentX will need to release updates to support this, right?

yes, cc @janek26


update to .net 6?

.net 6 has released 😀.

Definitely and a few changes too.
There will be some proposals soon.
Thanks.

My project is running on .Net 6, not problems until now :)

Please refer to issue #355.


HealthDataRefactoring

Checking with testEnvironment so as to avoid collision with other unittest cases.
Making the preference name more clear
Using Date.now() instead of new Date().getTime()
Remove separate object to store userExtensions instead using ExtensionManager.extensions
Change function name to be more clear

Looks good. Merging. Thanks!


README.md

I added my name to the README.md

It seems that the header on README also changed so it conflicts with master - can you revert the commit and try adding your name again then pushing again?
Also for future, include your name on the branch!


jdk21 release

hi. Can we have another release for Java 21?

Happy to release. But I guess this needs some testing first.
Have your tried or have run into problems?
The current CI only tests up to 19.
https://github.com/tcurdt/jdependency/blob/master/.github/workflows/ci.yml
I would also need to check if this needs an ASM upgrade.

I cloned and set my project shade-plugin to use the latest. No problems.
But I'm confused about something. Your pom already specifies asm 9.5 since 4 months ago
https://github.com/tcurdt/jdependency/commit/09234114ab1316c3d9916ce65267be6211f4b27d
So what's with the 5 open pull requests making the same change?

So what's with the 5 open pull requests making the same change?

That's a very good question. They don't make much sense and I also didn't notice them. It's a bit weird given the release in August. Anyway, I closed them and turned on java 21 for testing.
Unfortunately there is now support for java 21 on github yet.
https://github.com/tcurdt/jdependency/actions/runs/6220534843

Could not find satisfied version for semver 21.
Available versions: 20.0.2+9,

I have added java 20 to the testing matrix at least.

Since 2.9.0 was already released in August with ASM 9.5.
https://github.com/tcurdt/jdependency/blob/jdependency-2.9.0/pom.xml
I am not sure what else needs to happen for java 21 at the moment.

It seems like this is really a release request for the maven shade plugin and we can close this here?


Fix installation code in README.md

Corrected the installation code in README.md.
WHY
When I used the Installation code as is, lazy.nvim caused an installation error.
CHANGE
{
- "huantrinh1802/m_taskwarrior_d.nvim/"
+ "huantrinh1802/m_taskwarrior_d.nvim"
    -- ...
}

It appears that lazy.nvim is sensitive to trailing slashes.

Thanks @eraser5th !!!


How to unit test?

I am adding MonacoEditorModule.forRoot(MONACO_CONFIG) to the imports array and have the ngx-ngx-monaco-editor as part of my template but the editor is never instantiated and onInit never fires....
        <ngx-monaco-editor
            class="text-editor"
            (onInit)="onEditorInit($event)"
            [options]="options"
            [(ngModel)]="model">
        </ngx-monaco-editor>

These are the changes that were needed (in my case) in karma.conf in order to make tests work with monaco:
files: [
    { pattern: './../node_modules/ngx-monaco-editor/assets/monaco/**/*.js', watched: false, included: false, served: true },
    { pattern: './../node_modules/ngx-monaco-editor/assets/monaco/**/*.css', watched: false, included: false, served: true }
],
proxies: {
    "/assets/monaco/": '/absolute' + path.resolve("./node_modules/ngx-monaco-editor/assets/monaco/")
},

The gist is to make sure karma server can serve static assets from /node_modules/ngx-monaco-editor/assets/monaco,


Extending form validation methods

Added support for the following functions:
Between - numeric value should be within a given range
In / Not In - string should be in/not in a list of allowed values
Date - string should be a valid date
Before Date / After Date - date should fall between a given date range

I believe I told you in #3520 that only the 'in' rule may be useful ... This PR also has conflicts with our current code.

I understand. I'm fine with only the 'in' rule making it. I'll close this request and create a new one instead? (or is there s better way of deleting those other commits from here?)

You don't have to submit a new pull request, but you do have to sync your code with changes in the origin repository ... otherwise it's not mergeable.
You'll also have to add a changelog entry and I'll leave some inline comments on the code.

This should cover all the necessary changes.

 Don't take this the wrong way, but doing it now makes me think that you're just looking for an excuse to say something, anything ... you don't have to leave a comment on every single issue.

@narbg
I am sorry for making such an impression, yet, maybe it would be useful for you to have a look at this: https://github.com/bcit-ci/codeigniter3-translations/pull/134#issuecomment-70388488

If you think some language strings could be improved, you can do a pull request. Personal issues do not fit in an open source project.

@ivantcholakov So ping me from the relevant repo/issue/whatever if necessary, don't hijack other threads ...
@Razican There's no personal issue. There's an off-topic issue and you just contributed to it.
I value both of you guys for your contributions and enthusiasm about the project, but you're too chatty at times. Let's keep it on-topic now, shall we?
@foxlance Sorry about that distraction.
Can you rename the rule to 'in_list' and alter the changelog message in a manner similar to the one for 'differs'?

@narfbg I believe this should suffice


Fixes for pandas 1.3.0

The recently released pandas 1.3.0 introduces some changes to how where treats types (not sure if a bug or a feature, in some way both are correct): for example a SQL like CASE WHEN a = 3 THEN 1 END does now return a float64.
This PR therefore turns off the dtype checking for some of the test cases and therefore fixes the CI/CD error seen in #199

Codecov Report

Merging #202 (2ae171d) into main (ed6749d) will decrease coverage by 0.08%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              main     #202      +/-   ##
===========================================
- Coverage   100.00%   99.91%   -0.09%
===========================================
  Files           60       60
  Lines         2314     2323       +9
  Branches       317      319       +2
===========================================
+ Hits          2314     2321       +7
- Misses           0        2       +2

Impacted Files
Coverage Δ

dask_sql/physical/rel/logical/aggregate.py
100.00% <100.00%> (ø)

dask_sql/input_utils/hive.py
90.00% <0.00%> (-10.00%)
:arrow_down:

dask_sql/utils.py
100.00% <0.00%> (ø)

dask_sql/physical/rel/custom/predict.py
100.00% <0.00%> (ø)

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 afa8a8e...2ae171d. Read the comment docs.


Optional levels in the domain model

The RRE domain/data model provides a hierarchy with several levels (corpus, topic, query group, query and metrics).
Although this could be helpful when you need to model a rich domain, sometimes that results in a lot of redundancy (e.g. query groups and queries have the same cardinality).
It would be great if the ratings file would allow to exclude one or more levels, with the edge case where we have only queries.

topics and query_groups are now optional in the RRE domain model. That allows a high level of flexibility when defining the ratings set. Specifically (wiki will be updated asap) the domain model supports one of the following:

n topics -> n query groups -> n variant queries (i.e. logical variants of the same query)
n query groups -> n variant queries
n variant queries

In case one level is missing, RRE will create a unnamed node. So for example, if topics are missing, all defined query groups will belong to an "Unnamed" topic; if query groups are missing queries will belong to an "Unnamed" query group which in turn will belong to an "Unnamed" topic.
In this way the domain model is always consistent, and a report could easily hide those "Unnamed" layers


RPP Test Suite - Replace fstrings and subprocess calls to support old/new versions of Python

This PR adds relevant changes to support RPP test suite on old/new versions of python - tested on python 2.7, 3.5, 3.10

Replaces usage of fstrings with just strings for python backward compatibility
Replaces usage of subprocess.run with just subprocess.call / subprocess.Popen for python backward compatibility
Minor bugfix in test suite help display

@LakshmiKumar23 @kiritigowda This PR now passes CI and should support older and newer versions of Python.

@LakshmiKumar23 @kiritigowda This PR now passes CI and should support older and newer versions of Python.

I will update the ticket so QA can test and confirm


Request for a docker image

Hello,
Firstly, congratulations on your work CUE, it sounds very interesting. We have tried to install it via python environment as well as using Dockerfile you provided however we haven't been able to make it work. Is it possible you to create a docker image and upload it to docker-hub or any other hub?
Cheers,
Sevgin

Hi Sevgin,
Thank you for your suggestion and sorry to hear the installation was a problem. I'm in the process of creating an upgraded release and a docker image is on the todo list for it too. Meanwhile, could you let me know what the issue was with the python environment -- perhaps I can help fix that first.
Best,
Victoria


prefixed elements are not correctly processed in 0.5

demo-library.zip
Hello,
attached a sample project with 1 XSL and 1 XSpec. It's a maven project, your can reproduce the problem with mvn test, or you can execute in oXygen 19.0, the problem occurs too.
For each element, my XSL calculates a new element name, where prefix is unchanged, and element local-name is upper-cased. When applying the XSL on itself, it works correctly.
The xspec fails when constructing the new element.
Best regards,
Christophe

I think Michael wrote something about it.
So, in demo-library/src/main/xsl/demo/uppercase-elements.xsl, changing
    <xsl:element name="{$elementName}">

to
    <xsl:element name="{upper-case(local-name())}" namespace="{namespace-uri()}">

made the test return Success on my end.

Ok, I've frogotten this. Thanks a lot, @AirQuick  !


Martin Suggested Readings

TENSOR TRAIN BASED SAMPLING ALGORITHMS FOR APPROXIMATING
REGULARIZED WASSERSTEIN PROXIMAL OPERATORS
https://arxiv.org/pdf/2401.13125.pdf
https://uq-berlin.slack.com/archives/D0168AT80RY/p1706626173108139


Compiler crashes on simple closure

Description
The compiler  crashes while compiling a simple closure returning an opaque return type.
Maybe, this has to do with the fact that it currently seems to be impossible to write such a closure, but the compiler shouldn't crash anyways.
Steps to reproduce
Create a swift file with the following contents:
public func map<T, U, S>(_ f: @escaping (T) -> U) -> (S) -> some Sequence<U> where S: Sequence<T> {
    {
        let lazy = $0.lazy
        return lazy.map(f)
    }
}

Try to compile this file using swiftc.
Expected behavior
The compiler compiles this file, or at least gives diagnostics about what's wrong with it.
Environment

Swift version:
swift-driver version: 1.62.15 Apple Swift version 5.7.1 (swiftlang-<IP_ADDRESS>.2 clang-14<IP_ADDRESS>)
Target: x86_64-apple-macosx13.0
Xcode version (Shouldn't really matter, as this happens on linux using Swift 5.7 as well)
Xcode 14.1
Build version 14B47
Deployment Target
macOS Ventura 13.0

Additional context
Here is the crash backtrace:
Stack dump:
0.      Program arguments: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/swift-frontend -frontend -c -primary-file foo.swift -target x86_64-apple-macosx13.0 -enable-objc-interop -stack-check -sdk /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX13.0.sdk -color-diagnostics -new-driver-path /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/swift-driver -empty-abi-descriptor -resource-dir /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/lib/swift -module-name foo -disable-clang-spi -target-sdk-version 13.0 -o /var/folders/fc/70lr416d6jx3b0nhv958d7y00000gn/T/TemporaryDirectory.7cbOPp/foo-1.o
1.      Apple Swift version 5.7.1 (swiftlang-<IP_ADDRESS>.2 clang-14<IP_ADDRESS>)
2.      Compiling with the current language version
3.      While evaluating request TypeCheckSourceFileRequest(source_file "foo.swift")
4.      While evaluating request TypeCheckFunctionBodyRequest(foo.(file).map@foo.swift:1:13)
5.      While type-checking statement at [foo.swift:1:99 - line:6:1] RangeText="{
    {
        let lazy = $0.lazy
        return lazy.map(f)
    }
"
6.      While type-checking statement at [foo.swift:2:5 - line:5:5] RangeText="{
        let lazy = $0.lazy
        return lazy.map(f)
    "
7.      While type-checking expression at [foo.swift:2:5 - line:5:5] RangeText="{
        let lazy = $0.lazy
        return lazy.map(f)
    "
8.      While type-checking-target starting at foo.swift:2:5
Stack dump without symbol names (ensure you have llvm-symbolizer in your PATH or set the environment var `LLVM_SYMBOLIZER_PATH` to point to it):
0  swift-frontend           0x00000001145c3ff7 llvm::sys::PrintStackTrace(llvm::raw_ostream&, int) + 39
1  swift-frontend           0x00000001145c3028 llvm::sys::RunSignalHandlers() + 248
2  swift-frontend           0x00000001145c4610 SignalHandler(int) + 288
3  libsystem_platform.dylib 0x00007ff80712bc1d _sigtramp + 29
4  libsystem_platform.dylib 0x00007ff7b0f03e90 _sigtramp +<PHONE_NUMBER>2264450704
5  swift-frontend           0x000000010ff5add0 swift::constraints::ConstraintGraph::Change::undo(swift::constraints::ConstraintGraph&) + 176
6  swift-frontend           0x000000010fecc217 swift::constraints::ConstraintSystem::SolverScope::~SolverScope() + 9799
7  swift-frontend           0x000000010fede4b6 swift::constraints::TypeVariableStep::resume(bool) + 166
8  swift-frontend           0x000000010fecc63d swift::constraints::ConstraintSystem::solveImpl(llvm::SmallVectorImpl<swift::constraints::Solution>&) + 541
9  swift-frontend           0x000000010fecf7d4 swift::constraints::ConstraintSystem::solve(llvm::SmallVectorImpl<swift::constraints::Solution>&, swift::FreeTypeVariableBinding) + 52
10 swift-frontend           0x000000010feceb83 swift::constraints::ConstraintSystem::solve(swift::constraints::SolutionApplicationTarget&, swift::FreeTypeVariableBinding) + 899
11 swift-frontend           0x00000001100a4f46 swift::TypeChecker::typeCheckTarget(swift::constraints::SolutionApplicationTarget&, swift::OptionSet<swift::TypeCheckExprFlags, unsigned int>) + 598
12 swift-frontend           0x00000001100a4c81 swift::TypeChecker::typeCheckExpression(swift::constraints::SolutionApplicationTarget&, swift::OptionSet<swift::TypeCheckExprFlags, unsigned int>) + 417
13 swift-frontend           0x0000000110193131 swift::ASTVisitor<(anonymous namespace)::StmtChecker, void, swift::Stmt*, void, void, void, void>::visit(swift::Stmt*) + 6545
14 swift-frontend           0x0000000110191718 bool (anonymous namespace)::StmtChecker::typeCheckStmt<swift::Stmt>(swift::Stmt*&) + 296
15 swift-frontend           0x000000011018de5b (anonymous namespace)::StmtChecker::typeCheckASTNode(swift::ASTNode&) + 171
16 swift-frontend           0x000000011019181c swift::ASTVisitor<(anonymous namespace)::StmtChecker, void, swift::Stmt*, void, void, void, void>::visit(swift::Stmt*) + 124
17 swift-frontend           0x00000001101903c8 bool (anonymous namespace)::StmtChecker::typeCheckStmt<swift::BraceStmt>(swift::BraceStmt*&) + 296
18 swift-frontend           0x000000011018eeee swift::TypeCheckFunctionBodyRequest::evaluate(swift::Evaluator&, swift::AbstractFunctionDecl*) const + 1806
19 swift-frontend           0x00000001105c289d llvm::Expected<swift::TypeCheckFunctionBodyRequest::OutputType> swift::Evaluator::getResultUncached<swift::TypeCheckFunctionBodyRequest>(swift::TypeCheckFunctionBodyRequest const&) + 669
20 swift-frontend           0x0000000110550a6e swift::TypeCheckFunctionBodyRequest::OutputType swift::evaluateOrDefault<swift::TypeCheckFunctionBodyRequest>(swift::Evaluator&, swift::TypeCheckFunctionBodyRequest, swift::TypeCheckFunctionBodyRequest::OutputType) + 94
21 swift-frontend           0x00000001101d2feb swift::TypeCheckSourceFileRequest::evaluate(swift::Evaluator&, swift::SourceFile*) const + 875
22 swift-frontend           0x00000001101d605d llvm::Expected<swift::TypeCheckSourceFileRequest::OutputType> swift::Evaluator::getResultUncached<swift::TypeCheckSourceFileRequest>(swift::TypeCheckSourceFileRequest const&) + 669
23 swift-frontend           0x00000001101d2c32 swift::performTypeChecking(swift::SourceFile&) + 114
24 swift-frontend           0x000000010f1f85d9 swift::CompilerInstance::performSema() + 217
25 swift-frontend           0x000000010f17bd27 swift::performFrontend(llvm::ArrayRef<char const*>, char const*, void*, swift::FrontendObserver*) + 4711
26 swift-frontend           0x000000010f11202a swift::mainEntry(int, char const**) + 3082
27 dyld                     0x00007ff806dd0310 start + 2432

Is indeed repro in 5.7
But in main looks fixed.
@Zollerboy1 Can you verify using the latest main snapshot?

It doesn't crash anymore (at least on Ubuntu). But it still doesn't compile.

Reduced:
protocol P {}
struct S: P {}

func foo() -> () -> some P {
  return {
    return S()
  }
}

Removing the crash labels since that is no longer relevant.

I actually haven't yet found any way to produce a () -> some P closure, even though SE-0328 clearly states that it should be possible. If it helps, this post summarizes all the ways I've tried.

Should I maybe close this issue since it started as something else and open a new one specifically about creating () -> some P closures instead? What would be the way forward here?

Sure! Please try searching for it first, there might be one already.

https://github.com/apple/swift/issues/60038 has a similar case(the first one). Maybe add this one to it as well and use it to track then together @xedin?

I think #60038 covers this completely!


add onr config secrets bucket

create config secrets holder to align with nomis-combined-reporting approach

Shouldn't that read BIP Configuration. I don't think you need separate secrets for web.

/config is what's used in NCR secrets
have removed the /sap/web//* bucket now


() math mode escapes subscript

Consider the following example:
import matplotlib.pyplot as plt
import numpy as np

plt.style.use("ggplot")

t = np.arange(0.0, 2.0, 0.1)
s = np.sin(2 * np.pi * t)
s2 = np.cos(2 * np.pi * t)
plt.plot(t, s, "o-", lw=4.1)
plt.plot(t, s2, "o-", lw=4.1)
plt.xlabel("$x_t$")
plt.ylabel("\(y_t\)")
plt.grid(True)

import tikzplotlib

tikzplotlib.save("test.tex")

It uses different math environments for labels: $$ for x label and \(\) for y label.
This script produces output:
% This file was created by tikzplotlib v0.9.1.
\begin{tikzpicture}

\definecolor{color0}{rgb}{0.886274509803922,0.290196078431373,0.2}
\definecolor{color1}{rgb}{0.203921568627451,0.541176470588235,0.741176470588235}

\begin{axis}[
axis background/.style={fill=white!89.8039215686275!black},
axis line style={white},
tick align=outside,
tick pos=left,
x grid style={white},
xlabel={\(\displaystyle x_t\)},
xmajorgrids,
xmin=-0.095, xmax=1.995,
xtick style={color=white!33.3333333333333!black},
y grid style={white},
ylabel={\(y\_t\)},
ymajorgrids,
ymin=-1.1, ymax=1.1,
ytick style={color=white!33.3333333333333!black}
]
\addplot [line width=1.64pt, color0, mark=*, mark size=3, mark options={solid}]
table {%
0 0
0.1 0.587785252292473
0.2 0.951056516295154
0.3 0.951056516295154
0.4 0.587785252292473
0.5 1.22464679914735e-16
0.6 -0.587785252292473
0.7 -0.951056516295154
0.8 -0.951056516295154
0.9 -0.587785252292473
1 -2.44929359829471e-16
1.1 0.587785252292474
1.2 0.951056516295154
1.3 0.951056516295154
1.4 0.587785252292473
1.5 3.67394039744206e-16
1.6 -0.587785252292473
1.7 -0.951056516295154
1.8 -0.951056516295154
1.9 -0.587785252292473
};
\addplot [line width=1.64pt, color1, mark=*, mark size=3, mark options={solid}]
table {%
0 1
0.1 0.809016994374947
0.2 0.309016994374947
0.3 -0.309016994374948
0.4 -0.809016994374947
0.5 -1
0.6 -0.809016994374947
0.7 -0.309016994374948
0.8 0.309016994374947
0.9 0.809016994374947
1 1
1.1 0.809016994374947
1.2 0.309016994374947
1.3 -0.309016994374947
1.4 -0.809016994374947
1.5 -1
1.6 -0.809016994374948
1.7 -0.309016994374946
1.8 0.309016994374947
1.9 0.809016994374947
};
\end{axis}

\end{tikzpicture}

Here we can see, that different environments handle subscript symbol _ differently:
xlabel={\(\displaystyle x_t\)},
ylabel={\(y\_t\)},

In ylabel code underscore is escaped \_ which produces literal underscore in math mode which is wrong.

No chance of solving this, we'd have to implement a whole LaTeX parser in tikzplotlib. Labels simply cannot be guaranteed to be correct.
Perhaps there are ways of solving this that I don't see yet; if you do, I'd be very happy about a PR.


DOC: announce new features in README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.005%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
33108

Relevant Lines:
35985

💛  - Coveralls


Make result form accessible when the result has no date or date outside the program period

We have a significant number of results in the system whose result forms are inaccessible to us, because there is no date to click on in the result table. Here's an example from the FARM program 480.

The above example is especially confusing because it says the dates are outside the program period, and yet there are no dates displayed. And because dates are what we use to link to the result form modal, you can't access the result to troubleshoot.
Proposed solution
When the date is unavailable for whatever reason, display the string "Date missing" in place of the date, and make the string a link to open the result modal.

I have not seen anything like this in a long time. I think we can assume the issue is fixed or is an extreme edge case.


Add dataset click event

Emit 'dataset-clicked' event from gallery for 'mapintegratedvuer'.
This PR is connected with https://github.com/ABI-Software/mapcore-gallery/pull/10.
This PR is one of the dependencies of https://github.com/ABI-Software/mapintegratedvuer/pull/207.

Great, I have released the required package for this branch. Can you please install<EMAIL_ADDRESS>
@alan-wu, The gallery version is updated 54a6c69.


 Extra , is left for some of them.
To Reproduce
Execute get images script in repository base directory:
bash tools/get-images.sh
Environment
Charm source code. No additional setting are needed.
Relevant Log Output
The following is the list images produced by the script. Note extra , for some of the images:
docker.io/kubeflowkatib/katib-db-manager:v0.15.0
docker.io/kubeflowkatib/katib-controller:v0.15.0
docker.io/kubeflowkatib/katib-ui:v0.15.0
docker.io/kubeflowkatib/suggestion-darts:v0.15.0
docker.io/kubeflowkatib/suggestion-enas:v0.15.0, # <<<<<< extra comma
docker.io/kubeflowkatib/suggestion-goptuna:v0.15.0
docker.io/kubeflowkatib/suggestion-hyperband:v0.15.0
docker.io/kubeflowkatib/suggestion-hyperopt:v0.15.0
docker.io/kubeflowkatib/suggestion-optuna:v0.15.0
docker.io/kubeflowkatib/suggestion-pbt:v0.15.0, # <<<<<< extra comma
docker.io/kubeflowkatib/suggestion-skopt:v0.15.0
docker.io/kubeflowkatib/suggestion-darts:v0.15.0
docker.io/kubeflowkatib/suggestion-enas:v0.15.0, # <<<<<< extra comma
docker.io/kubeflowkatib/suggestion-goptuna:v0.15.0
docker.io/kubeflowkatib/suggestion-hyperband:v0.15.0
docker.io/kubeflowkatib/suggestion-hyperopt:v0.15.0
docker.io/kubeflowkatib/suggestion-optuna:v0.15.0
docker.io/kubeflowkatib/suggestion-pbt:v0.15.0, # <<<<<< extra comma
docker.io/kubeflowkatib/suggestion-skopt:v0.15.0

Additional Context
N/A

Tried the changes locally and they work as expected. Although the more I see these scripts the more I'm convinced they should be written as Python scripts, to make then more readable and maintainable. But this can be a small push from the next week to organize.
Regarding this PR, can we:

Make the tools/get-images.sh script executable +x
Add a shebang at the beginning of the script

Let's be able to do ./tools/get-images.sh

Fix has been merged. Closing.


start the repo with a skeleton

only the needed files
inspirtion from here
https://github.com/laminas-api-tools/documentation

Initial files from main branch have been backed up into branch https://github.com/dotkernel/api-documentation/tree/initial
main branch overwritten with new documentation files moved from dotkernel/api after https://github.com/dotkernel/api/pull/255.


Add missing pins A6 and A7 to nano

Fixes #1228
Adds the missing A6 and A7 pins to the nano board file and increases the max number of analog pins to 8.
While the two pins work without a firmware change, the firmware also needs to update to have a max analog pin count of 8 for all the pins to be usable at the same time (see https://github.com/MobiFlight/MobiFlight-Connector/issues/1228)

@neilenns could you add the ProMicro on this PR to increase the max. numbers of Analog Input to 9 as this PR is still open?
I have issued a PR for the firmware change on Nano and ProMicro, see PR 248.

@elral Updated the PR with the Pro Micro max increased to 9, and also added names to the analog pins that didn't have analog names attached.
I used this for reference: https://learn.sparkfun.com/tutorials/pro-micro--fio-v3-hookup-guide

I double checked the pin assignemt.
The ProMicro does not have Pin D17, but this one is on the json file.
The "original" board is just named "Micro", and this board has additional pins (11/12/13/17/22/23).
We could

add these additional pins to the json file. If one isn't available on the board, it can not be connected
"just" support the ProMicro, but then we have to delete Pin 17.

@neilenns @DocMoebiuz Any idea how to handle this?
The Nano has Pin 20 and 21, but they can only be used as Anlog In. I guess we do not have a marker to avoid that they show up as digital in. Furthermore these both pins are marked as I2C in this PR, but they aren't.

Removed pin D17 from the ProMicro file.
Fixed the incorrect I2C marking for A6 and A7 on the nano

Regarding Micro vs. Pro Micro, Mobiflight website says we support the pro micro so the file should reflect that. you are correct, there is no way currently to avoid having the analog pins show up as digital.

@neilenns i saw you added analog pins to the Pro Micro, I looked at the sparkfun pinout

And I was confused by the labels and the analog pins that they show compared what our JSON file has.
I always tried to name the labels for the pins the same as they are named on the board silkscreen.
Looks like we have a mistake in our json - the analog pins and pin numbers are not the same as on the pinout.


ffmpeg cannot find the program input point vkBindImageMemory 2

I want to install ffmpeg, cmd executes ffmpeg -version, and then ran out of this window (I found that ffmpeg was changed to an English path and directly clicked on ffmpeg.exe, and the result will run out of this window)
https://answers.microsoft.com/zh-hant/windows/forum/windows_10-hardware/如何修復/716b4fed-3de3 -4e61-8700-98521ebea273
I executed
Windows+X select A and enter the following command,
DISM /Online /Cleanup-Image /RestoreHealth
cmd.exe /c "for %1 in (%windir%\system32*.dll) do regsvr32.exe /s %1"
Try again from boot.
But still not resolved
Can ask what to do to solve
Problems with dynamic link libraries
Normally, ffmpeg -version should run out of information
could you tell me how do I do it?

Either install a properly working Vulkan-Capable graphics driver, or download a non-vulkan variant.
That's the exact reason it's its own variant.


feat(frontend): Add e2e test for primary index scan

Update planner tests
Signed-off-by: Georg Kreuzmayr<EMAIL_ADDRESS>I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Showing that there is a problem with sorting on index scan
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Let's close this one as long time no activity. Feel free to reopen.


Guide to setup verus mining pool

Hello,
I have a question. I created VM debian 10 (Digital ocean) but i cannot create pool successfully.
Could you please share more detail setup instructions?
I follow this instruction: https://github.com/VerusCoin/VerusServicesSetup/blob/master/S-NOMP.md

This is completely unrelated to this repository.
Closing.


Kill legacy templates

Based on #6829
With the merging of #6794, these templates are no longer used.
✂ ✂ ✂ ✂ ✂ ✂ ✂ ✂ ✂ ✂ ✂ ✂

Needs base_update conflict resolved (deleted)


skip results  without videoId

Results without videoId cause pytube.exceptions.RegexMatchError: regex_search: could not find match for (?:v=|\/)([0-9A-Za-z_-]{11}).*
fixes #1193

Done. Sorry for the delay, have been on a trip.


Suggestion on DiscreteAllocation class for regular rebalancing

Hello, Robert
I found your DiscreteAllocation.greedy_portfolio() in  pypfopt while implementing a regular rebalancing task. However, my challenge was that my situation was not SIMPLY convert continuous weights to discrete allocation.
A new situation I had to consider was:

I already have some stocks.

In other words, we had to consider the situation of not only buying the stock, but also selling it.

‘fee’ (a kind of contraction cost) must be considered
(In addition, using ‘total_portfolio_value’,  which is dependent on ‘latest_prices’, felt a little less intuitive to me.)

Thankfully, thanks to your project, I was able to implement the task without difficulty.
I really appreciate this part.
I am using your code by modifying the structure as below.
[Inputs & Outputs in DiscreteAllocation class]
“””

Inputs:

weights - dict
latest_prices - pd.Series or dict
total_portfolio_value - int/float
short_ratio- float

Output:

allocation - dict
“””

[After modifying]
“””

Inputs:

weights - dict
init_state -  dict   # number of existing shares
latest_prices - pd.Series or dict
cash - int/float    # amount of cash currently has
fee_rate - float    # fee rate as contraction cost
short_ratio- float
verbose- bool

Output:

allocation - dict
action - dict    # action for desired weight (+ indicates buying, - indicates selling)
cash_left - dict
“””

If this fixing fits the direction of your project PyPortfolioOpt, I'd be happy to share with you.
Please read the above and reply.
Thank you for your time.

If this can be implemented in a self-contained way with a couple of unit tests, I'm happy to accept a PR!


Typos - References to "RUF" should be "RUA"

Typos - References to "RUF" should actually be "RUA" in describing and in examples for DMARC records

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

ID: 7861a800-7643-dca5-1046-232a51d6e61e
Version Independent ID: d78fc416-516b-15ab-4719-068f7d13ecca
Content: Use DMARC to validate email, setup steps - Office 365
Content Source: microsoft-365/security/office-365-security/use-dmarc-to-validate-email.md
Product: m365-security
Technology: mdo
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@rmanjoney Thank you for your feedback. RUF is for forensic reports and  RUA is for aggregated reports. I could not find any typos referring to RUF. Is there a specific section that you can point out?
Thanks
Sri

@msbemba Thank you very much for the contribution and digging deep into it.  @rmanjoney We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if you see an improvement is required in the specific area of this documentation. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


TypeScript definitions support

Hi, do you have any plans of supporting a TypeScript definitions file, or does one exist that is not registered on typings or not on DefinitelyTyped?
Thanks

@leocaseiro
No existing definition that I'm aware of.
No plans of adding any up to this point.
The module is fairly small so adding definitions shouldn't be a big effort. I might look in to it myself but you're welcome to do a pull request if you don't want to wait.


Add a Twig function to minify CSS and JS

See https://github.com/matthiasmullie/minify.

WIP: #97.

#97 merged.


Add tests to show BufRead can be used after decoding

Copy the trailing data tests from gzip::bufread and gzip::write to demonstrate that they also work for zlib and deflate.
This demonstrates that #401 works with bufread decoders.

I put the tests here for symmetry with the equivalent gzip tests, which are also currently unit tests. While unit tests can access private items, I've never heard that unit tests must access private items. And, due to the additional build costs of integration tests, I tend to prefer unit tests where possible.
Nevertheless, I can move the tests to the integration level. Should I move the equivalent gzip tests too?

I put the tests here for symmetry with the equivalent gzip tests, which are also currently unit tests.

Apologies, I wasn't aware and didn't want to change the way things are done here. It seems integration tests have a specific purpose (here, making use of fixtures) and unit-tests are used for everything else. Let's keep it that way.


Some reablocks components don't work with vitest testing library (Checkbox)

I'm submitting a...

[x ] Bug report

Current behavior
When testing reablocks with vitest, some of the components are working fine, some not.
For example: InlineInput
InlineInput.test.tsx`
describe('Test', () => {
it('Should render', () => {
render();
});
});
The error I'm getting`
Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object
Other components that are throwing error`
Checkbox, Select and probably more
This is how I use Checkbox component`
const TestComponent = () => {
const [checked, setChecked] = useState(false);
return (
<Checkbox
checked={checked}
label="Check me"
onChange={() => setChecked(!checked)}
/>
);
};
describe('Test', () => {
it('Should render', async () => {
render();
});
});
Expected behavior
I expect component to not throw error, when testing with vitest
Minimal reproduction of the problem with instructions
The described components above are giving error, when I try to render them, but some other components from reablocks work just fine, for example  Button, Calendar, Card, DateFormat, Menu and more

What is the motivation / use case for changing the behavior?

Environment

Libs:
  "dependencies": {
    "@testing-library/jest-dom": "^6.4.2",
    "@testing-library/react": "^15.0.1",
    "@testing-library/user-event": "^14.5.2",
    "reablocks": "^5.8.0",
    "react": "^18.2.0",
    "react-dnd": "^16.0.1",
    "react-dnd-html5-backend": "^16.0.1",
    "react-dom": "^18.2.0",
    "react-error-boundary": "^4.0.10",
    "react-fast-compare": "^3.2.2",
    "react-helmet-async": "^1.3.0",
    "react-hook-form": "^7.45.2",
    "react-query": "^3.39.3",
    "react-router-dom": "^6.21.2",
    "react-tabs": "^6.0.2",
    "react-test-renderer": "^18.2.0",
    "react-use": "^17.4.0",
    "react-window": "^1.8.10",
    "react-xarrows": "^2.0.2",
    "reakeys": "^1.3.1",
    "reaviz": "^14.15.2",
    "rison": "^0.1.1",
    "use-query-params": "^2.2.1",
    "yup": "^1.2.0"
  },

Browser:
- [x ] Chrome (desktop) version 123.0.6312.122

For Tooling issues:
- Node version: 20.11.1
- Platform: Windows

@tigran1100 - Could you make a sample repo? That will help us test it much easier.

141-vitest-error.zip

Couldn't push the branch, 403
There are 3 tests (Checkbox.test, ConditionInput.test, SearchInput.test), where only searchInput is working
npm run test

Just a new repo would work

https://github.com/tigran1100/reablocks

Just want to inform that the problem still exists for 'InlineInput' and 'Select' components on the latest "7.3.6" version, but it fixed the problem for "Checkbox" component
Thanks

The problem for "InlineInput" is coming from "react-18-input-autosize" library.
For example when I'm changing
AutosizeInput to 'input' in this part of code
"node_modules/reablocks/dist/index.js"
const InlineInput = forwardRef(
({
inputClassName,
placeholderIsMinWidth = true,
theme: customTheme,
...rest
}, ref) => {
const theme2 = useComponentTheme("input", customTheme);
return jsx(
AutosizeInput,
{
inputRef: ref,
inputClassName: twMerge(theme2.inline, inputClassName),
placeholderIsMinWidth,
...rest
}
);
}
);
The test is passing, but the input is becoming non-resizable.
Also it's fixing the problem with Select component. Idk how they are related, but it's being fixed too

@devconn99 - retry now - we redid that component.

Okay, I will try.
Thank you @amcdnl

@tigran1100 - can you retest and let us know if its stil an issue?

this should be solved now, if its not please reopen. It was the peasky inline input but we redid it.


[stable/metricbeat] updated metricbeat to 6.6.1

Signed-off-by: André Bauer<EMAIL_ADDRESS>
What this PR does / why we need it:

updated metricbeat to 6.6.1

Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @mumoshu

/assign @at-k

Thank you for your PR.
I confirmed that the latest version of metricbeat docker image is certainly 6.6.1, and this PR works.

/lgtm


Add Create/CreateAsync/Update/UpdateAsync to INotificationHubOperations

Currently the INotificationHubOperations supports Delete/Get/GetConnectionDetails operations, need to also add the ability to Create and Update Notification hubs.

please reopen if this is still an issue.


Make presubmit tests to run on all go modules in the repo

The approach I took is the following:

I left the prow file as-is, calling various "commands" (unittest, lint, gosec, ...) of the Makefile in the root of the repo
I modified the Makefile to automatically find all go.mod files and run the commands (unittest, lint, gosec, ...) for all of the modules one-by-one

But I am not familiar with prow usage patterns, so this may be a completely wrong direction

/test presubmit-nephio-go-test

/test presubmit-nephio-golangci-lint

/lgtm

But I am not familiar with prow usage patterns, so this may be a completely wrong direction
There are no good or bad approaches here I feel. As long as it's working and it's comfortable for daily use it's awesome. That being said I like that approach from practical point of view having fairly uniform .prow.yaml file between the projects and adjust things in Makefiles (that will obviously differ from project to project anyway).


New package: reoMS v0.0.1

Registering package: reoMS
Repository: https://github.com/yanjer/reoMS.jl
Created by: @yanjer
Version: v0.0.1
Commit: 51f61c1e1a20091ae44ec8e3f28103b1ef53e3b1
Git reference: HEAD

（1h

（1）

(1) fix the AutoMerge issues and re-trigger Registrator


Convert Custom Yolov4 model

HI! I want to ask that I want to convert Yolov4 which has trained on 12 classes, hence I change some filter values. Will this repo will change my weights to the h5 Keras model successfully ??

Yes, remember to set the right class names file to build model. You can check if you got a error message from here


OLM Integration Package Manifests Quickstart

make packagemanifests is is not in the kubebuilder template anymore.
See > https://sdk.operatorframework.io/docs/olm-integration/quickstart-package-manifests/

If users would like to use make packagemanifests than it needs to be added manually. See the first text of the link added:

It is properly described. If you click on and have added the packagemanifests target to your Makefile as described [here](https://sdk.operatorframework.io/docs/olm-integration/generation/#package-manifests-format). you will find the code for you copy and past.
The reason for make packagemanifests has not been scaffolding by default is because the tool provided the bundle feature. Besides OLM still supporting the packagemanifests (CSV) it is recommended to users move forward to use the bundle instead of it. See: https://sdk.operatorframework.io/docs/olm-integration/quickstart-bundle/
In this way, shows fine we close this one as sorted out. However, please feel free to ping us if you think that it should be re-opened.


gemspec dependencies not loading in bundler (rails 3.0.9)

I added paypal_adaptive to my Gemfile, and bundle install didn't load json or jsonschema... I added those to my gemfile and am now working.
Gemfile:
gem 'json'
gem 'jsonschema'
gem 'paypal_adaptive'

I added json/jsonschema as dependencies rather than dev dependencies


Copy item (feature request)

With the old TQVault I kept all my found uniques in the vault and equipped the heroes with copies of those, so I didn't lose track of where they are. Would be nice to have a function that copies items in the right click menu in TQVaultAE.

We have that function! Check out the configuration


Shorten output for failed package in config mode

Thanks.  Please see CONTRIBUTING.rst and submit a merge request to our GitLab instance instead.  GitHub OAuth can be used to authenticate if preferred.

Sorry, I forgot about your GitLab.


ddev share: unknown shorthand flag: 's' in -subdomain

Is there an existing issue for this?

[X] I have searched the existing issues

Run a Diagnostic and Paste Link Here
No response
Expected Behavior
ddev share --subdomain mysubdomain would work.
Actual Behavior
After upgrading ngrok to version 3.1.0, I get this error when using ddev share --subdomain mysubdomain. This worked with previous ngrok versions.

ERROR:  unknown shorthand flag: 's' in -subdomain=mysubdomain

Looks like shorthand notation for subdomain is not supported by ngrok anymore. Potential fix: use --subdomain flag when passing the subdomain argument to ngrok.
Steps To Reproduce
No response
Anything else?
No response

Thanks so much, fixed in #4393
(Note: You don't need this PR to solve the problem, but instead can just use the technique in the docs, https://ddev.readthedocs.io/en/latest/users/topics/sharing/#setting-up-a-stable-ngrok-subdomain - put the ngrok args in config.yaml)


Update supported-guest-os.md

inclcomplete info on mainstream guest OS support; Enhanced Session Mode does not work with Windows 10 Home Guest OS

Hyper-V in general isn't available on Windows 10 Home Edition.

Fair point.  Can you add guest OS?  Then I'll merge

Hello Sarah,
My discription may not have been complete. Enhanced Session Mode does not work when using Windows 10 Home as a guest OS.
Tested a clean install with latest MSDN bits on both Windows 10 build 17025.rs_prerelease.171020-1626 and Windows 2016 10.0.14393 as being a hypervisor for the client. Did not test on earlier Hyper-V platform, but I assume the same because of next.
Also tested conversion from Windows 10 Home to Windows 10 Pro by using same install (key change). After this (and a reboot) Enhanced Session Mode works. So I assume this limitation is deliberate on Windows 10 Home as being a guest OS.
Above will probably have minimum impact on average production environment (not using W10Home as guest) but out took me >3 hours to find this out while just wanting to test some random thingy (assumed error on Hyper-V config/W10 bits donwload/install etc etc ). Bummer. :(


Changed filters encompass mutated or added components

Code such as that found at https://docs.rs/bevy_rapier3d/0.11.0/src/bevy_rapier3d/physics/rigid_body_component_set.rs.html#65 is redundant and induces pointless performance overhead <3

Thanks Alice, this has been fixed since then :+1:


Add option to limit docker machine ssh ingress access to only the runner

Description
Similar to https://github.com/npalm/terraform-aws-gitlab-runner/pull/140, limit the docker machine ssh ingress access to only the runner via security groups instead of cidr ranges.
I kept backwards compatibility by making this feature optional (defaults to false) as I do not know if people actually need external SSH access to the docker machines. @npalm Do you think external SSH access is necessary? If it doesnt make sense, we could remove the CIDR based security rule and simply the code. Otherwise, this should work to support both cases.
Migrations required
No

@bishtawi I think in only some debug cases there is a need for ssh acces, but in that case you use the runner instance to get into. Alternatively the session manager can be used. So I think you right the CIDR can be removed as well.

@npalm @bishtawi Did you check that this change also works for a public runner (this is also true for the change concerning the docker-machine port #140 ?
I tried to do something similar (https://github.com/roock/terraform-aws-gitlab-runner/commit/5e12e8851d16fe78d45e302f4e439a359fe80f0e), but it wasn't able to get it working using public IPs on the runners (security groups do not work with external IPs).
I think we need the docker-machine option "amazonec2-use-private-address" in this case (see
https://docs.docker.com/machine/drivers/aws/ )

@roock Yup, I tested my changes against a repo hosted on Gitlab.com and had no problems. My Gitlab runners were deployed on a private subnet in my VPC, but I dont see why it would be any different if they were in a public subnet. I dont see any reason why the docker machines need fully open public ingress access on ports 22 and 2376. The machines communicate solely with the runner ec2 box, and not with Gitlab.com. And unless I am mistaken, I am pretty sure the Gitlab Runners poll gitlab.com for CI jobs to run; Gitlab.com doesnt push anything to the runners, so I really dont see why unrestricted ingress access is necessary.
Looking at your changes, the only thing that is different is that I added a rule to allow self traffic through. Not sure if that matters? Or maybe I am misunderstanding what you mean by "public runner"?

but it wasn't able to get it working using public IPs on the runners (security groups do not work with external IPs)

This line confuses me. Even if your gitlab runners were on the public subnet, you should still be able to use security groups to specify allowed ingress traffic. In fact, thats usually when security groups are the most important. Unless your docker machines are hosted in a different VPC than the runner? Or you arnt using a VPC? Hah, maybe I am just misunderstanding your comment.

Welp, just tested it out on a public subnet and set runners_use_private_address = false and you are absolutely correct that it does not work. Did some searching and yup, looks like using security groups as the source only allows private IP traffic and not public. You really do learn something new every day.
@npalm How should we best handle this? Probably the best solution would be to introduce some variables allowing users to both either set CIDRs or enable locking ingress access via security group source.

Alright, updated PR to handle both public and private IP addresses. Now variables docker_machine_docker_cidr_blocks and docker_machine_ssh_cidr_blocks allow users to set custom CIDR ranges, but these ranges are only used when runners_use_private_address is false. If runners_use_private_address is true then we use the security group source to limit ingress.

I think it would be best to use the private IP for communicating between Runner and Docker Machine in any case, also because traffic within the same VPC is cheaper and faster than going over the public IPs
see https://github.com/roock/terraform-aws-gitlab-runner/commit/64f9e7971232e03bdc8d85235d4b073c03a87b6e

@npalm @roock Alright done, ready for review

@bishtawi thanks, wil start checking the changes

Just verified the change

runner default works fine
runner public (example) is broken. The count on the ssh key is not working. Making some small changes was able to get the sample working with running the runner on a public ip and the docker machines on a private or public one. Running both on private wasn't working.


Port buildHTML to @flow.

Looking through this PR, some of these changes to buildHTML...seem ancient, and I was surprised they were not in yet. Looking back, I just realized that the last 21 PRs I sent were just so that this PR could finally finish. O_o
buildMathML should come in soon as well.

Looking through this PR, some of these changes to buildHTML...seem ancient, and I was surprised they were not in yet. Looking back, I just realized that the last 21 PRs I sent were just so that this PR could finally finish. O_o

I appreciate how deliberate you've been about these changes.  I'm glad that we're getting a place where we're able to do larger refactorings safely.

buildMathML should come in soon as well.

Looking forward to it.


feat: add "adeu" audio asset with Git LFS

For fun purposes only. Adding Git LFS to keep repo clean.
The husky files were automatically added when running git lfs track \*.ogg. Noice!
Not adding any CI check about Git LFS given it's just for this fun purpose only

Current dependencies on/for this PR:

main

PR #223   👈

This stack of pull requests is managed by Graphite.


Module not found: Can't resolve

Module not found: Can't resolve 'click-to-react-component'
I am trying this awesome tool, but can not use it as a package. For now, I put src files in my components folder.
If I try import from package directly, I will meet resolve error like above.
Don't know why, seems others do not have this problem.
My webpack is<EMAIL_ADDRESS>

What's your webpack config?  Does it resolve .mjs files?
You can fork the repo and put it in https://github.com/ericclemmons/click-to-component/tree/main/apps as a PR & I'll help troubleshoot it.

Got it!  I'll go ahead & close this then – to support webpack (which powered Next.js & CRA), we'd at least need a reproducible sample under /apps to work off of.

Hi ! Same issue for me, I managed to make the lib works by adding "main": "./src/index.js" in package.json.
Looks like the "exports" alternative is not working for me for some reasons.
Node 16.14.2
Webpack 5.72.0


On uploading retail csv files, step function triggers but get stuck at predictor creation and never ends

Describe the bug
On uploading retail csv files, step function triggers but get stuck at predictor creation and never ends
To Reproduce
Deploy via cloud formation stack, upload forecast config file for retail and upload demand csv files in s3 data bucket, step function will trigger but will stuck forever.
Expected behavior
Step fucntion should have been completed
Please complete the following information about the solution:

[Latest

To get the version of the solution, you can look at the description of the created CloudFormation stack. For example, "(SO0123) Improving Forecast Accuracy with Machine Learning v1.3.0[...]".

[ ] Region: us-east-1
[ ] Was the solution modified from the version published on this repository? - No
[ ] If the answer to the previous question was yes, are the changes available on GitHub?
[ ] Have you checked your service quotas for the sevices this solution uses? - Yes
[ ] Were there any errors in the CloudWatch Logs? - Lambda logs says that resource creation pending and it keep going in loops

Screenshots
If applicable, add screenshots to help explain your problem (please DO NOT include sensitive information).

Additional context
Add any other context about the problem here.

this is resolved automatically


Chatterino2 doesn't correctly handle MacOS special character inputs (via keyboard)

Describe the bug
Currently, on MacOS, attempting to type a special character the MacOS way (holding down a key, and selecting the special deviant from the list) will just spam the key, instead of waiting for the user to select a special character and inputting that
To reproduce
In Chatterino on MacOS, hold down the 'a' key.
Screenshots

Chatterino version
Chatterino 2.1.5 (commit 624c578b0)
Operating system
MacOS Mojave v10.14.6
Additional information

sounds like something Qt should fix honestly, but I can't judge that properly

Perhaps, Telegram Desktop seems to implement this correctly, I wonder how they achieve it

Duplicate of #1425

Duplicate of #1425

that issue is 404?

that issue is 404?

Very interesting

I think that's because of discussion changes, github might have deleted some discussions in our repo after a category change.
29 sie 2022 04:02:10 12932 @.***>:

Duplicate of #1425[https://github.com/Chatterino/chatterino2/issues/1425]
that issue is 404?
—
Reply to this email directly, view it on GitHub[https://github.com/Chatterino/chatterino2/issues/1430#issuecomment-1229656942], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AF62MITOCLKZL2P23HMJW5LV3QKZRANCNFSM4JUSCPIA].
You are receiving this because you are subscribed to this thread. [data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAADkAAAA5CAYAAACMGIOFAAAAAXNSR0IArs4c6QAAAARzQklUCAgICHwIZIgAAAAkSURBVGiB7cExAQAAAMKg9U9tDQ+gAAAAAAAAAAAAAAAA4NQAMv0AAa3l6BAAAAAASUVORK5CYII=###24x24:true###][Obrazek śledzący][https://github.com/notifications/beacon/AF62MITU3VPPAQOKL7SCOG3V3QKZRA5CNFSM4JUSCPIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOJFFRG3Q.gif]Message ID: @.***>

Is this still replicatable on Qt 6? I think this might be a not-our-bug. I looked at our code for split input we don't do special key repeat logic there. I'd like to also know if this is replicatable with other line and text edits all around the app:

Search all emotes or Find in all settings inputs (plain QLineEdit)
commands/highlights/nicknames/ignores (QTableView)

I've just tried this in Chatterino 2.5.1 on macOS 12.7.6 (Monterey) and 14.7 (Sonoma) and the long-press keyboard shortcut to access alternate characters is working for me as expected. It seems likely that Qt 6.x fixed the issue.
Chatterino 2.5.1 (commit eafcb941) built with Qt 6.5.0
Running on macOS Monterey (12.7), kernel: 21.6.0
Chatterino 2.5.1 (commit eafcb941) built with Qt 6.5.0
Running on macOS 14.7, kernel: 23.6.0

@ashugg Thank you for taking the time to test this! Closing this as resolved for now then.


GCP App engine integration

Closes #2

Working when deployed but when using the secret manager it fails.
Asked on SO for some help.

Closed by #18


Wrong resolution recognized as unsupported

Current Behavior
I'm running GI on a dual-monitor setup. The main monitor has a resolution of 2880x1920 scaled at 200% (HiDPI). The second monitor is a standard 1366x768 VGA. The game is running on the second, low-res monitor. Inventory Kamera errors out saying Unsupported resolution 2732x1536.
Expected Behavior
The scanner to supports my device configurations (please specify in additional notes)
Reproducing a Behavior
This is a screenshot showing that it errors out despite the resolution being a standard 1366x768.

Device OS
Windows 11
Genshin Impact Version
3.3
Inventory Kamera Version
1.3.4
Screen resolution, screen mode, and UI scale
1366x768 at 100%
Additional notes and remarks
No response
Evidence
No response
Preflight Checklist

[X] I have checked for similar issues in the issue page, and I have not found any similar to mine.
[X] I am using the latest version of the scanner.
[ ] I tried setting the scanning delay to maximum, and it still doesn't work.
[ ] I have attached the video recording to the bug report.
[ ] I have attached the error log to the bug report.
[ ] I am available to later explains the issue.
[X] I have disabled all screen filtering software or plugins (i.e. HDR, Image Sharpening) while scanning.
[X] I use default in-game keybindings or have configured keybindings for the scanner.

Weird that it seems to be applying the 200% scaling. Window resolution is determined entirely through windows libraries so I'm not sure what I can do on Kamera's end to validate the resolution is correct.
In either case, 1366x768 is not a supported resolution regardless per step 3-2 of the instructions.

Oh, I see... Well, 1366x768 is extremely close to 16 : 9 (it's 16 : 8.99 ) so I thought it was still supported, since the button layout can't be that different from a pure 16:9 (Inventory Kamera clicks in the middle of buttons and things, and I'm pretty sure there isn't even a full single pixel difference from a 16:9 monitor layout in terms of button positioning)... So yeah, I'll keep that in mind and try again by running it in a lower-res windowed mode.
Anyways, this is still weird because my main setup consists of a 4K monitor with 150% scaling, and two separate monitors with lower resolutions (1600x900 and 1920x1080) both at 100% scaling, and Inventory Kamera didn't give me any issue with the scaling on my main display.
But if this is purely related to Windows APIs, and there's no way to fix it nicely (without making hacked-on modifications) then I guess you can just close the issue and we just hope that Microsoft fixes their APIs...
Thanks for taking the time to reply!


Handle start/end layer changes gracefully

Closing this as the side-effect isn't as straightforward as expected. For instance, when setting both props immediately after each other, the effect of the fist prop on the second is negated. This is problematic in scenarios where all settings are set at the same time like when using a view model object to hold settings for the UI.


Setting TLS PRF secret unsets TLS PRF seed

Testing https://github.com/openssl/openssl/pull/6674/files#diff-5cdc677aaa86805a06a5772b83976f38R82 I ran into an issue where KDF would fail if the seed was set before the secret.
This appears to be present in previous releases:
https://github.com/openssl/openssl/blob/OpenSSL_1_1_0-stable/crypto/kdf/tls1_prf.c#L70
Use case: https://github.com/wahern/luaossl/pull/154/commits/228c1b6077d063434174d43217764cee4888010c#diff-ce817d33bbc15430a154eb6954bd86faR11735

This does not seem to be an issue anymore after the implementation was moved to the provider(s).