Cannot find c++ std library headers (RDT-426)

For example:
/Users/johnboiles/Developer/repos/sindarin-inc/embedded-app/src/Sensors/LightingController.hpp:3:10: error: 'functional' file not found [clang-diagnostic-error]
#include <functional>

/Users/johnboiles/Developer/repos/sindarin-inc/embedded-app/src/Inputs/RemoteInput.hpp:4:10: error: 'cstdint' file not found [clang-diagnostic-error]
#include <cstdint>

It looks like the generated clang-tidy command from the log output is very simplistic and doesn't include anything about include paths or anything like that:
/Users/johnboiles/.espressif/tools/esp-clang/15.0.0-23786128ae/esp-clang/bin/clang-tidy -p=build --config-file=.clang-tidy /Users/johnboiles/Developer/repos/sindarin-inc/embedded-app/src/Misc/PlatformESP32.cpp

Seems like the stock run-clang-tidy.py script mentioned in the docs only uses the "file" key in compile_commands.json and doesn't use the "command" key, so all of the build flags are missing.
So how do we use this when using the c++ stdlib?

Hi @johnboiles, I am very sorry for the late response. I have tried to reproduce the issue, but everything seems to work fine on my end with regards to C++ headers.
I have checked this with the examples/cxx/pthread example from ESP-IDF, applying the following patch to it:
diff --git a/examples/cxx/pthread/main/cpp_pthread.cpp a/examples/cxx/pthread/main/cpp_pthread.cpp
index 232e7f3431f724f98802f9f095b02a9d47074dce..43d79ea7ae5d0efe16260802f6aa5e6c67ac2e0a 100644
--- forkSrcPrefix/examples/cxx/pthread/main/cpp_pthread.cpp
+++ forkDstPrefix/examples/cxx/pthread/main/cpp_pthread.cpp
@@ -17,6 +17,7 @@
 #include <freertos/FreeRTOS.h>
 #include <freertos/task.h>
 #include <esp_log.h>
+#include <functional>

 using namespace std::chrono;

@@ -106,6 +107,8 @@ extern "C" void app_main(void)
         ss << "core id: " << xPortGetCoreID()
            << ", prio: " << uxTaskPriorityGet(nullptr)
            << ", minimum free stack: " << uxTaskGetStackHighWaterMark(nullptr) << " bytes.";
+        volatile int a;
+        ss << a;
         ESP_LOGI(pcTaskGetName(nullptr), "%s", ss.str().c_str());
         std::this_thread::sleep_for(sleep_time);
     }

(to make sure I have include <functional> as your code does, and to introduce an error which should be noticed by clang-tidy)
The steps to test were as follows:
$ cd $IDF_PATH/examples/cxx/pthread
$ rm -rf build sdkconfig
$ IDF_TOOLCHAIN=clang idf.py clang-check --run-clang-tidy-py run-clang-tidy
...
/Users/ivan/.espressif/tools/esp-clang/16.0.1-fe4f10a809/esp-clang/bin/clang-tidy -p=build /Users/ivan/e/esp-idf/examples/cxx/pthread/main/cpp_pthread.cpp
/Users/ivan/e/esp-idf/examples/cxx/pthread/main/cpp_pthread.cpp:111:15: error: variable 'a' is uninitialized when used here [clang-diagnostic-uninitialized]
        ss << a;
              ^
/Users/ivan/e/esp-idf/examples/cxx/pthread/main/cpp_pthread.cpp:110:23: note: initialize the variable 'a' to silence this warning
        volatile int a;
                      ^
                       = 0
command "/Users/ivan/.espressif/python_env/idf5.2_py3.9_env/bin/python /Users/ivan/.espressif/tools/esp-clang/16.0.1-fe4f10a809/esp-clang/bin/run-clang-tidy -p build .*" gives the following warnings with exitcode 1:
3 warnings and 1 error generated.

So it looks like clang-tidy didn't complain about the missing headers, and the error introduced in the code was correctly identified.

Hi @johnboiles,
I followed the setup instructions and made the changes described by @igrr and can reproduce the issue:
/Users/radim/esp/esp-idf/examples/cxx/pthread/main/cpp_pthread.cpp:10:10: error: 'functional' file not found [clang-diagnostic-error]
#include <functional>
         ^~~~~~~~~~~~

But only if gcc is used as a compiler (essentially skipping the IDF_TOOLCHAIN=clang step in @igrr's instructions). If clang is installed and set with IDF_TOOLCHAIN=clang, all works fine.
Are you using the gcc compiler by any chance?

Well spotted @radimkarnis! For this to work with GCC toolchain, we need to either:

Stop using clang-tidy-runner and use clang-tidy via clangd. Then we can pass --query-driver argument to clangd so that it knows how to resolve the cross-compiler toolchain headers.
Wait for --query-driver to be supported in clang-tidy-runner (https://github.com/llvm/llvm-project/issues/53468)

The related documentation has been updated with this information. It will be available soon.
We are now looking into ways to enable this to work with GCC.
Thank you for the report! I am closing this ticket.

@radimkarnis thanks for updating the documentation! I can confirm that this works by adding IDF_TOOLCHAIN=clang (though I have to rm -rf build first).


Allow identical DocumentSnapshot sources to trigger output changes.

Prior to this project changes would trigger re-parses which would then be thrown away because source versions were identical.
Added test to verify new SetOutput behavior.

aspnet/Razor.VSCode#184

@SteveSandersonMS FYI, if you see invalid code issues in VSCode on startup, chances are it's because of this. We'll consume this package once it's available.


Fix the monaco sprite editor for python

Fixes https://github.com/microsoft/pxt-arcade/issues/2079
Three changes:

Emit the python syntax and not typescript when we are editing python
Made the regex a little more robust
Register the folding range provider for python (this is what actually makes the icon show up)

Merging for tomorrow testing.


Update notebooks in documentation build

Currently the notebooks are outdated.
The notebooks submodule needs to be updated and a new build of the documentation triggered.

the new build should be triggered automatically if you update the notebooks submodule on main

I have to update some plots anyway, so I can do this


Update migration guide

Hello! The migration guide for resolvePreviewURL included a token in the v5 example, but omitted it in the v6 example. I assumed that was intentional but turned out that accidentally broke our previews! Just wanted to flag the documentation should be updated so both examples include the token.

Hey @itsjoekent, this is actually intentional like you assumed 🙂 (but of course, it's supposed to work automatically). resolvePreviewURL() should read globalThis.location.search for the token and documentId URL params without needing to pass them manually.
Could you share some details about where you're using resolvePreviewURL()? Is there anything unique about the setup? Such as being on a server or in a specific browser?
For reference, the following preview-related methods are available on a client:

client.enableAutoPreviews() - Configures the client to automatically query content from a preview session. This is the client's default mode.
client.enableAutoPreviewsFromReq(req) - Accepts a server request object containing the request's headers and cookies. Configures the client to automatically query content from a preview session in a server environment.
client.disableAutoPreviews() - Configures the client to ignore preview sessions. Queries will be made against the latest published content or the client's configured release.

Source: https://prismic.io/docs/technical-reference/prismicio-client#repo-methods
You can still provide documentID and previewToken as parameters to resolvePreviewURL() if you need to provide them manually for some reason (such as not being in a browser or Express server-like environment).

Here's the relevant code in case you are curious
https://github.com/prismicio/prismic-client/blob/b141ba78c7f842b4c147419f6d7f26ac3b2bb125/src/client.ts#L1155-L1165

Thanks!

Hi @angeloashmore ! Thanks for that, like I mentioned in the original description, the specific thing I got hung-up on was that the example appears to show the token being intentionally removed (that screenshot is from the migration guide here.

I realize your overall point however that it's supposed to be "automatic", maybe just drop a note in the migration guide that Node environments should still keep the token?

Ah, good point. Yes, a note should be added there like you described. I'll work on getting that updated. 👍

@itsjoekent Thanks again for the suggestion. The migration guide now explains how to upgrade in Express and non-Express server environments.
https://prismic.io/docs/technologies/prismic-client-v6-migration-guide#replace-getpreviewresolver-with-resolvepreviewurl
I'm going to close this issue, but please let me know if you think there's a better way to describe this. Thanks!


Enable range queries on datetime and u64 fields

Description
Per title. I need this feature for the Jaeger integration (filters on span start timestamp or duration).
How was this PR tested?
Added unit tests.

This is ready for review.

@PSeitz can you review?

@PSeitz, PTAL!


add MOM 0705

What type of PR is this?
Add one of the following kinds:

bug
correction
enhancement/feature
cleanup
documentation
subproject management
tests

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:
Changelog input
 release-note

Additional documentation
This section can be blank.
docs

ref to https://github.com/camaraproject/NetworkSliceBooking/pull/36


v0.3: Large-scale refactor.

Will squash and update PR synopsis tomorrow.

Reopening with squashed PR. Will link to this PR for reference.


Deploy project live demo

Important: Do not merge this merge
In this pull request I have been able to deploy the branch with render:

here is the back-end live demo
here is the front-end live demo

Note: Render is sometimes slow so it might take 1-5 min to render.

Thank you for the review.
we will keep the deployed branch separate from the main and dev branch. so I will close this PR now.


Feature: add news page

Adds a mailchimp integration into the newsletterpage

fetches recent newsletters
provides a subscription form
adds a scrollToTop utill component

Requires 3 mailchimp envars namely:
NEXT_PUBLIC_MAILCHIMP_API_KEY=""
NEXT_PUBLIC_MAILCHIMP_REGION=""
NEXT_PUBLIC_NEWSLETTER_AUDIENCE_ID=""

this will require uploading the newsletter covers as ${issueid}.jpg

![Screenshot 2024-12-03 at 09 06 05 (2)](https://github.com/user-attachments/assets/a40263c5-1c68-41f1-b654-11ebf178f7ad)

resolves #1332
@race-of-sloths

@flmel Thank you for your contribution! Your pull request is now a part of the Race of Sloths!
Do you want to apply for monthly streak? Get 8+ score for a single PR this month and receive boost for race-of-sloths!

Current status: waiting for scoring
We're waiting for maintainer to score this pull request with @race-of-sloths score [0,1,2,3,5,8,13] command. Alternatively, autoscoring [1,2] will be applied for this pull request
What is the Race of Sloths

Race of Sloths is a friendly competition where you can participate in challenges and compete with other open-source contributors within your normal workflow
For contributors:

Tag @race-of-sloths inside your pull requests
Wait for the maintainer to review and score your pull request
Check out your position in the Leaderboard
Keep weekly and monthly streaks to reach higher positions
Boast your contributions with a dynamic picture of your Profile

For maintainers:

Score pull requests that participate in the Race of Sloths and receive a reward
Engage contributors with fair scoring and fast responses so they keep their streaks
Promote the Race to the point where the Race starts promoting you
Grow the community of your contributors

Feel free to check our website for additional details!

Bot commands

For contributors

Include a PR: @race-of-sloths include to enter the Race with your PR

For maintainers:

Invite contributor @race-of-sloths invite to invite the contributor to participate in a race or include it, if it's already a runner.
Assign points: @race-of-sloths score [1/2/3/5/8/13] to award points based on your assessment.
Reject this PR: @race-of-sloths exclude to send this PR back to the drawing board.
Exclude repo: @race-of-sloths pause to stop bot activity in this repo until @race-of-sloths unpause command is called


Delegated Harvesting

Delegated harvesting docs: https://nemtech.github.io/concepts/harvesting.html
Activate / deactivate delegated harvesting
AccountLinkTransaction.create(
    Deadline.create(),
    LinkAction.Link, // Option Link or Unlink
    PublicAccount.createWithPublicKey(publicKey)
);

Check how to know if delegated harvesting is enabled/disabled.

PR for AccountLinkTransaction merged in library and SDK.


feat(datepicker): added label prop to datepicker

Datepicker is like an input, and without a label field it would be weird because of inconsistency...

Datepicker have a lot of bugs, but now I have other priorities.
Maybe you should replace the input and use the vue-strap input instead?
When I have time maybe should change this component and convert it to a kind of wrapper, then could be used over input, buttons or whatever you want.

What you meant is a component that inherits from vue-strap input would be
better instead?
On Sun, Sep 11, 2016 at 12:46 AM, Willem<EMAIL_ADDRESS>wrote:

Datepicker have a lot of bugs, but now I have other priorities.
Maybe you should replace the input and use the vue-strap input instead?
When I have time maybe should change this component and convert it to a
kind of wrapper, then could be used over input, buttons or whatever you
want.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/yuche/vue-strap/pull/323#issuecomment-246122038, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMeDEpKUJDK4YdVI6QUSKCiVrXd41yXNks5qot71gaJpZM4J31bI
.


test: don't try to use Xcode SDK for build requirement

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

Previously, running brew test failed if the formula had an Xcode build requirement but the user did not have Xcode installed. brew would see the Xcode requirement and default to the Xcode SDK. However, for formula that only need Xcode to build, this check is unnecessary when simply testing the formula. I ran into this in https://github.com/Homebrew/homebrew-core/pull/67043 with deno:
$ brew test deno
==> Testing deno
Error: deno: failed
An exception occurred within a child process:
  NoMethodError: undefined method `path' for nil:NilClass
Did you mean?  paths
/usr/local/Homebrew/Library/Homebrew/extend/os/mac/extend/ENV/std.rb:88:in `macosxsdk'
/usr/local/Homebrew/Library/Homebrew/extend/os/mac/extend/ENV/std.rb:45:in `setup_build_environment'
/usr/local/Homebrew/Library/Homebrew/test.rb:39:in `<main>'

I don't have code installed, so Xcode.sdk.path failed (Xcode.sdk is nil).
My solution is to, when testing, pass a flag to the macosxsdk method to determine whether to check only runtime requirements. When this is false (the default), any Xcode requirement will require the use of the Xcode SDK. When set to true (only in brew test), build-only Xcode requirements will not necessarily require the Xcode SDK.
I'm marking this PR as critical because I believe it fixes a bug. However, I'm not familiar at all with the files that I'm changing so it's entirely possible that this change will have unintended consequences. I will not merge until I have approvals from other maintainers who are more familiar with this code.

Review period skipped due to critical label.

Tested this locally with the existing deno bottle:
❯ brew test deno
==> Testing deno
==> /usr/local/Cellar/deno/1.6.0/bin/deno run hello.ts
Check file:///private/tmp/deno-test-20201218-47524-x10fau/hello.ts
==> /usr/local/Cellar/deno/1.6.0/bin/deno run --allow-read=/private/tmp/deno-test-20201218-47524-x10fau<EMAIL_ADDRESS>Download<EMAIL_ADDRESS>Check<EMAIL_ADDRESS>
Works perfectly fine.

Updated to simplify/clarify the logic and switch to using keyword arguments instead of **options

Thanks @Rylan12!


wide metopic suture

Add:
wide metopic suture (child of abnormal metopic suture morphology MP:0003842)
DEF: an abnormally increased width of the metopic suture for age-related norms
SYN: widened metopic suture
SYN: broad metopic suture
SYN: wide spaced metopic suture
SYN: large metopic suture
SYN: wide frontal suture
SYN: wide interfrontal suture
SYN: wide median frontal suture
SYN: wide mediofrontal suture
Annotations in J:50164, J:115338, J:163175

Added:
wide metopic suture MP:0030431


Clarified some confusion concerning which taxa of_children queries wi…

…ll show when "all" is false.

Coverage increased (+0.004%) to 90.593% when pulling 2baa7ff7f3f6a9db70fa79fa3c228702936df380 on pellaea:name-with-obs-of-children into a5ef527a7963bcc03b24552ea6037153b2df98cb on MushroomObserver:master.

Probably beyond the scope of this PR, but:
Consider a species Name which has both true subtaxa (subspecies, varieties, forms) and Group(s)/Clade(s).  Example: Cantharellus cibarius Fr.
If I'm understanding this PR, that page will include links to:
Show Observation of, Subtaxa (The PR doesn't change this.)

This shows Observations of subtaxa of the species and of groups/clades. No harm done, but illogical.

Classification, Species

What will this show? (I'm too time-stressed to try it our.). Is there a way to show Names of only the subtaxa? Do we care?

OK, I think I understand what's going on here and it seems reasonable to me.  One bit that's confusing is that Jason's explanation about the "Show Species" and "Show Groups and Clades" links doesn't apply to the Tricholomopsis as mentioned in the email, but does for Agaricus.  I need a better local environment for testing these PRs, but that's a totally separate issue.
Jason or Joe feel free to merge this if you feel Joe's concern doesn't need to be addressed here.

Yeah, this was a bit slap-dash right from the beginning.  And very
confusing, even to me.  Sorry for the long email...
Just to try to be more clear, the "of children" flavor of Query has two
modes.
If "all" is true, then it's nice and easy: it just selects all subtaxa at
all ranks below the given name(s).
If "all" is false, then it attempts to get just "immediate children", that
is children taxa whose direct parent is the given name(s).  This is the
mode used by both links on the show_name page.
The confusion as concerns the UI is how to characterize this latter mode.
In the majority of cases, subtaxa of families should be genera, subtaxa of
genera should be species, and so on.  But it breaks down in certain cases.

Below species it can be a mixture of subspecies, varieties and forms.

Below genus Query skips over subgenera, etc.  And note that
classifications are not presently allowed to include subgenera, etc.  So
there is no way, using the of_children flavor, to select for subgenera, etc.

Further confusing things, groups (and presumably clades) confuse the
algorithm.  "Agaricus group" looks just like a species.  Apparently
"Cantharellus cibarius group" looks like a subspecies/variety/form.

Okay.  To answer your questions, then, Joe:
Cantharellus cibarius -> Observations of -> Subtaxa: Presently shows
observations of subspecies/varieties/forms, as well as (erroneously)
groups.
Cantharellus cibarius -> Classsification -> Show Subtaxa (presently called
"Show Groups and Clades"): Presently and under this PR, shows
subspecies/varieties/forms, as well as (erroneously) groups.
More generally (after this PR):
Kingdom -> Observations of -> Subtaxa: shows mostly phylla but can even
include unclassified genera
Family -> Observations of -> Subtaxa: shows genera
Genus -> Observations of -> Subtaxa: shows species (and groups like
Agaricus group)
Species -> Observations of -> Subtaxa: shows infraspecies (and groups like
Cantharellus cibarius group)
Kingdom -> Classification -> Show Subtaxa: shows mostly phylla but can even
include unclassified genera
Family -> Classification -> Show Subtaxa: shows genera
Genus -> Classification -> Show Species: shows species (and groups like
Agaricus group)
Species -> Classification -> Show Subtaxa: shows infraspecies (and groups
like Cantharellus cibarius group)
(Yes, those are exactly the same sets of names selected for under both
Observations of -> Subtaxa and Classification -> Subtaxa.  I have renamed
the latter to "species" instead of "subtaxa" for genera in order to
emphasize that it's skipping subgenera and other infrageneric subtaxa.
Maybe we should do the same for Genus -> Observations of -> Subtaxa, too?)
If we would like to remove groups from these queries -- and I think we
should, I think it feels like a bug -- then I propose we make that a
separate story.  That will require nontrivial modification of the Query
code, tests, etc.  That's beyond the scope of what I can do this summer.
If we would like to give users the ability to request a list of
infrageneric taxa under genus, or list of (just) groups / clades under any
rank taxon, then we will have to expand the of_children Query flavors or
create new Query flavor(s) or use some other mechanism altogether.  Again,
definitely requires a new story, as it will require considerably more
programming effort than I can manage.
-Jason
On Thu, Aug 16, 2018 at 8:57 AM Nathan Wilson<EMAIL_ADDRESS>wrote:

OK, I think I understand what's going on here and it seems reasonable to
me. One bit that's confusing is that Jason's explanation about the "Show
Species" and "Show Groups and Clades" links doesn't apply to the
Tricholomopsis as mentioned in the email, but does for Agaricus. I need a
better local environment for testing these PRs, but that's a totally
separate issue.
Jason or Joe feel free to merge this if you feel Joe's concern doesn't
need to be addressed here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/412#issuecomment-413595627,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADE2tTyh-ye4LXpF-n-NsgC96nOlq8miks5uRZaAgaJpZM4V_Qwc
.


Revert general data ini when error occur with full run "antares in memory"

Xpansion update the "general_data.ini" file of the study.
During the antares step we backup the file before the edit and we revert it at the end of the step
When performing "in memory" simulation there is no "antares step". We still backup the file before editing it, but there is no revert when finishing.

Moved to internal ticketing system https://gopro-tickets.rte-france.com/browse/ANT-2192


Optionally specify display name with push command.

Since Itch.io game pages don't display the date files were updated, there's no easy way to communicate to (potential) customers which version of your file is available on the site. For this, I use the custom display name of a file to include the version number.
For example:
I have MyGame-Win64.zip is displayed at "MyGame v1.3 Win64" but I have to manually update this display name each time I upload new builds. It would be nice if we could use butler's push command to specify the new display name to use for the new upload. Something like:
butler push MyFolder username/game:channel "My New Display Name"

Setting display name via push is an interesting thought, but I don't think it should be used for versioning — https://github.com/itchio/itch.io/issues/398 is my preferred solution to that.

Having the game/download page display the wharf/butler upload version number works for me.
Thinking about it more, what I'd really like is something that probably belongs in a new feature request "Issue" -- linking an existing upload (before butler, wharf, or the itch.app existed) to a new butler channel. So I guess I'll go create an issue for it and then come back here and add a link to it.

I also would like to be able to set the file display name via the push AND/OR display version numbers as suggested in #398

We're just starting to use itch.io refinery and butler, and we too would find this feature incredibly useful. Currently we have to constantly update the display name manually, which is a big pain.

Closing, as https://github.com/itchio/itch.io/issues/398 has been implemented.


slidesPerView: 'auto', centeredSlides: true and active slide larger width

Check that this is really a bug

[X] I confirm

Reproduction link

Bug description
Hi,
I have a slides with slidesPerView: 'auto', centeredSlides: true. All slides have 20% width but the active slides has 60%. It's working on initial load but after slide change the centered slide isn't centered anymore. Am I missing something or isn't that working?
Thanks,
Patrick
Expected Behavior
No response
Actual Behavior
No response
Swiper version
9.1.0
Platform/Target and Browser Versions
All
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Your issue is closed because it is not filled correctly. Before you open an issue please review the contributing guideline.
Please fill the issue template correctly and:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with isssue

It will save a lot of time for other contributors to check your issue and fix it.


Proposed Breaking: add forgot password feature in the SupaEmailAuth widget

What kind of change does this PR introduce?
This PR combines the forgot email flow, signIn, and signUp into a single widget so that the developer using this library does not have to separately implement it.
The aim for this PR is to make it more similar to how Supabase auth UI for react is implemented.
This PR adds the following

Add forgot password feature
Remove authAction parameter on SupaEmailAuth widget.

Now there is a button to toggle between signIn and signUp in SupaEmailAuth widget, so that there is no need for developers to have two separate SupaEmailAuth in two different places.

Minor internal variable renaming for some cleanup

With this PR, you can have this code
SupaEmailAuth(
  onSignInComplete: (response) {
    // handle signIn
  },
  onSignUpComplete: (response) {
    // handle signUp
  },
),

To get this:

@FatumaA  @Csierram96
I wanted to create this PR as a proposal, and I would like to hear what you think about it. Obviously it is quite a huge change from the current limitation, and there are pros and cons about this implementation. Any comments is appreciated, and I don't intend to merge this if you think this is not a good idea, so please don't hesitate to let me know if you don't like it if you don't!

Will do additional cleanup of example directory if we think this PR is a good idea.

Seems like the next logical step 🙂. I will take a closer look a little later today.
Thank you for continuing to maintain this while I could not!

It looks like a really good addition. Just curious, what are the cons that you see of adding this?

Thanks for all the positive comments!
@FatumaA

I think switching the '/' from SignIn to home is not necessary since it is an auth lib. I'm assuming that the sign in screen is what should be at the root?
If we keep that change we have to make minor changes on how the app switches screen

Thanks for the feedback! Makes sense! Will put it back the way it used to be!

Otherwise it looks fine 👍🏽
I will add a commit to remove the forgot password text as it shows doubly
also, do we still need the update_password screen in the example?
I have noticed a bug on validation: fields stay in error even when you toggle between the screens in supaEmailAuth

Yup, I haven't actually checked the example app thoroughly yet. I just wanted to show this feature to you to get some feedback! I will do some cleanup of the example directory!

Thanks for the positive feedback @Csierram96 !

It looks like a really good addition. Just curious, what are the cons that you see of adding this?

 I know the library is still in v0.x, which means it's still in pre-release mode therefore breaking change is okay, but wanted to get a feel on how you felt about it.
Less flexibility. Before this update, developers were able to have login and sign-up page in two different routes, but now with this update, login and sign-up are in the same widget, so they have to be in the same page. There are some flexibility lost with this update. I guess though at the end, developers will use this supabase_auth_ui to get up and running quickly, so it might be a bad thing to have less flexibility and more to offer out of the box. If they want flexibility, they can always just create their own login and sign up form from scratch.

Thanks for the positive feedback @Csierram96 !

It looks like a really good addition. Just curious, what are the cons that you see of adding this?

 I know the library is still in v0.x, which means it's still in pre-release mode therefore breaking change is okay, but wanted to get a feel on how you felt about it.
Less flexibility. Before this update, developers were able to have login and sign-up page in two different routes, but now with this update, login and sign-up are in the same widget, so they have to be in the same page. There are some flexibility lost with this update. I guess though at the end, developers will use this supabase_auth_ui to get up and running quickly, so it might be a bad thing to have less flexibility and more to offer out of the box. If they want flexibility, they can always just create their own login and sign up form from scratch.

That makes sense. The braking change should be fine as you mentioned as this has not been released as stable. And regarding the flexibility that's actually a good point. Personally I prefer something that just works out of the box for this kinda stuff (firebase ui or supabase ui). And if you want more flexibility you can implement your own UI.

Thanks @Csierram96 for the nice comment! It's nice to receive a sanity check from other developers! I will do some additional clean up on this PR and merge it in!

I think this one is good to be merged!


[question]TypeError: Cannot read property 'currentTime' of null

hi thanks to your good code
i have error when close modal have audio component  while audio playing
or i change another url page from page have audio component while audio playing
but after pause then it hasnt error
error is
TypeError: Cannot read property 'currentTime' of null
(anonymous function)
src/components/state/actions.ts:73
70 |     };
71 | }
72 | function setPlayerDuration(dispatch, player) {

73 |     return function () {
74 |         dispatch({ type: PLAYER_SET_DURATION, duration: player.current.duration });
75 |     };
76 | }
View compiled
HTMLAudioElement.handlePlayerTimeUpdate
src/components/AudioPlayer.tsx:253
250 | React.createElement(Grid_1.default, { item: true, className: classes.commonContainer },
251 |     React.createElement(AudioPlayControl_1.default, { classNames: classNames, icons: icons, playerStatus: state.player.status, playerColors: playerColors, replayAudio: _replayAudio, pauseAudio: _pauseAudio, playAudio: _playAudio })),
252 | download && (React.createElement(AudioDownloadsControl_1.default, { src: src, playerColors: playerColors })),
253 | React.createElement(AudioVolumeControl_1.default, { muteAudio: _muteAudio, unmuteAudio: _unmuteAudio, classNames: classNames, changeAudioVolume: _changeAudioVolume, volume: state.player.volume, playerColors: playerColors }),
| ^  254 | React.createElement(Grid_1.default, { item: true, container: true, spacing: 2, className: classnames_1.default(classes.sliderContainerWrapper) },
255 |     (!isSingleTime || isTimePositionStart) && currentTimeComp,
256 |     React.createElement(Grid_1.default, { item: true, className: classes.sliderContainer },

how can i do ?

I couldn't reproduce your issue. I was removing a component from the page or entering a different page while audio was playing. No error showed up. But, I was able to found what piece of code could cause that error. I added check for the currentTime in 1.5.0 version. Please, check out that release.

i checked and update to 1.5.0 but error still remain

Closed due to inactivity.


2019.32 9d0d19a Update. Offline

After getting the 2019.32 9d0d19a update on our Model 3 I notice the car goes offline

Any other users seeing this?
I get in the car and it’s not in WiFi or cellular in the garage

No Wifi or cellular data -> no connection to Tesla -> no connection from TeslaMate to Tesla thru Tesla Api

Never had the car go offline prior to this update

That's perfectly normal behavior. Maybe offline indicates some kind of deep sleep.


No notebook document for file://*.ipynb (wsl2)

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
When adding cells: No notebook document for file://*.ipynb is prompted. I am using a workspace under WSL2 within the .vhdx as a wsl remote.
The notebook goes unresponsive.
VS Code Version
Version: 1.91.0 (user setup) Commit: ea1445cc7016315d0f5728f8e8b12a45dc0a7286 Date: 2024-07-01T18:52:22.949Z Electron: 29.4.0 ElectronBuildId: 9728852 Chromium: 122.0.6261.156 Node.js: 20.9.0 V8: 12.2.281.27-electron.0 OS: Windows_NT x64 10.0.22631
Jupyter Extension Version
v2024.6.0
Jupyter logs
14:28:18.630 [info] Starting Kernel (Python Path: <repo>/.venv/bin/<interpreter>, Venv, 3.9.19) for '~/git/texconnect/DataTransformationApi/eolt_analysis.ipynb' (disableUI=false)
14:28:19.000 [info] Process Execution: <repo>/.venv/bin/<interpreter> -c "import ipykernel; print(ipykernel.__version__); print("5dc3a68c-e34e-4080-9c3e-2a532b2ccb4d"); print(ipykernel.__file__)"
14:28:19.002 [info] Process Execution: <repo>/.venv/bin/<interpreter> -m ipykernel_launcher --f=/home/~/.local/share/jupyter/runtime/kernel-v2-369SMzJUcplhu2Z.json
    > cwd: //home/~/... # this is unedited after ~
14:28:19.957 [info] Kernel successfully started

Coding Language and Runtime Version
v3.9.19
Language Extension Version (if applicable)
v2024.8.1
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

Similar issue https://github.com/microsoft/vscode/issues/196740

Sorry you are running into this issue, are you still able to replicate this at you end.
If so, please can you

Disable all extensions
Ensure you have latest version of vscode installed
Try to replicate this issue


macOS: Closing window. Can't open again

Why does nobody have this problem?
It is not possible to reopen the window again on macOS.
PR: #45

Hmm this is interesting. I always run Punk on Windows so I have never noticed this. 🙈
I'll go check that PR.

Please test on windows, since i have only tested it on macOS.


Model-based Java mirrors

Complete the model-based mirrors, that will provide a Java view of the model of typechecked Ceylon sources.
This was started, but far from finished  in the com.redhat.ceylon.ide.common.model.asjava package.

@bjansen :  ongoing developments are pushed in the asJava-mirrors branches of the ceylon and ceylon-ide-common projects. Feel free to ask questions.


How to get image locally

Hello
I'm trying to use the builder on the example project and I can't quite work it out
I did a git clone git@github.com:home-assistant/addons-example.git and from the addons-example directory did:
docker run \
  --rm \
  -it \
  --name builder \
  --privileged \
  -v $(pwd)/example:/data \
  -v /var/run/docker.sock:/var/run/docker.sock:ro \
  homeassistant/aarch64-builder \
  --target /data    --aarch64  -d local
[23:41:53] INFO: No crossbuild support on host
[23:41:53] INFO: Using host docker setup with '/var/run/docker.sock'
[23:41:53] INFO: Run addon build for: aarch64
[23:41:53] INFO: Init cache for local/aarch64-addon-example:1.2.0 with tag latest and platform linux/arm64
[23:42:00] WARNING: No cache image found. Disabling cache for this build.
[23:42:00] INFO: Run build for local/aarch64-addon-example:1.2.0 with platform linux/arm64
WARNING: No output specified for docker-container driver. Build result will only remain in the build cache. To push result image into registry use --push or to load image into docker use --load
[+] Building 1.8s (7/7) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 410B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for ghcr.io/home-assistant/aarch64-base:3.15                                                                                0.4s
 => CACHED [1/3] FROM ghcr.io/home-assistant/aarch64-base:3.15@sha256:005f1df70301d335e050c8db253e69a8b7862b6f1f6ffa31f686751bb165a6e2                   0.0s
 => => resolve ghcr.io/home-assistant/aarch64-base:3.15@sha256:005f1df70301d335e050c8db253e69a8b7862b6f1f6ffa31f686751bb165a6e2                          0.0s
 => [internal] load build context                                                                                0.1s
 => => transferring context: 1.70kB                                                                                0.1s
 => [2/3] RUN     curl -sSLf -o /usr/bin/tempio     "https://github.com/home-assistant/tempio/releases/download/2021.09.0/tempio_aarch64"                1.3s
 => [3/3] COPY rootfs /                                                                                0.0s
[23:42:02] INFO: Finish build for local/aarch64-addon-example:1.2.0
[23:42:02] INFO: Create image tag: latest
Error response from daemon: failed to find image local/aarch64-addon-example:1.2.0: docker.io/local/aarch64-addon-example:1.2.0: image not known

I'm on MacOS Monterey 12.6 (M1 2020 Macbook Pro)
Using podman 4.3.1
Is there an option so that the built image is available to my local podman without pushing it to a docker registry?

Pass in --test as an argument

even with --test it does the same thing
╰─ docker run \
  --rm \
  -it \
  --name builder \
  --privileged \
  -v $(pwd)/example:/data \
  -v /var/run/docker.sock:/var/run/docker.sock:ro \
  homeassistant/aarch64-builder \
  --target /data    --aarch64  -d local --test
[03:20:41] INFO: No crossbuild support on host
[03:20:41] INFO: Using host docker setup with '/var/run/docker.sock'
[03:20:41] INFO: Run addon build for: aarch64
[03:20:41] INFO: Init cache for local/aarch64-addon-example:1.2.0 with tag latest and platform linux/arm64
[03:20:46] WARNING: No cache image found. Disabling cache for this build.
[03:20:46] INFO: Run build for local/aarch64-addon-example:1.2.0 with platform linux/arm64
WARNING: No output specified for docker-container driver. Build result will only remain in the build cache. To push result image into registry use --push or to load image into docker use --load
[+] Building 4.7s (7/7) FINISHED
 => [internal] load build definition from Dockerfile                                                                                0.0s
 => => transferring dockerfile: 425B                                                                                0.0s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [internal] load metadata for ghcr.io/home-assistant/aarch64-base:3.15                                                                                0.3s
 => CACHED [1/3] FROM ghcr.io/home-assistant/aarch64-base:3.15@sha256:005f1df70301d335e050c8db253e69a8b7862b6f1f6ffa31f686751bb165a6e2                                                                0.0s
 => => resolve ghcr.io/home-assistant/aarch64-base:3.15@sha256:005f1df70301d335e050c8db253e69a8b7862b6f1f6ffa31f686751bb165a6e2                                                                       0.0s
 => [internal] load build context                                                                                0.2s
 => => transferring context: 1.70kB                                                                                0.1s
 => [2/3] RUN     curl -sSLf -o /usr/bin/tempio     "https://github.com/home-assistant/tempio/releases/download/2021.09.0/tempio_aarch64"                                                             4.3s
 => [3/3] COPY rootfs /                                                                                0.0s
[03:20:51] INFO: Finish build for local/aarch64-addon-example:1.2.0
[03:20:51] INFO: Create image tag: latest
Error response from daemon: failed to find image local/aarch64-addon-example:1.2.0: docker.io/local/aarch64-addon-example:1.2.0: image not known

Hi, it seems even in github actions I have the same problem
https://github.com/delfick/photons/commit/963696928d9de5ac04f324c4584a0c1de8375f21

Having the same problem.
I think the --load flag for buildx is missing and I don't understand why this has been working at all.

Adding the --load flag to the buildx command indeed makes it work for me.


a handful of package and publish chores

@davideast care to review?

hi @jeffbcross
since when angular2 was in beta the package.json was like this
"angular2": ">=2.0.0-beta.14 <2.0.0-rc.1", (include all beta versions from the beta-14 up to the first release candidate)
can't we do something similar for the rc versions like this
"@angular/core": ">=2.0.0-rc.2 <2.0.0" (include all rc versions from the rc.2 up to the final release)
instead of listing all the possible future rc versions?
(I'm trying to learn through https://semver.npmjs.com/  :worried:)

LGTM 👍

I have the same problem as #320 and #325 . but I think the problem is fixed now, So how can I use npm to install the latest changes from the master branch ?

@Khalid-Nowaf I use "angularfire2": "git://github.com/angular/angularfire2.git#master"


[Bug][Config-UI] Error: Miller-columns-select duplicate ID in items

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Received an error message Error: miller-columns-select: Duplicate ID in items when trying to setup a new blueprint.

What do you expect to happen
Wanted to be able to setup the blueprint by selecting the repositories in the list.
How to reproduce
I believe this might be a frontend issue that requires unique IDs among groups/projects. Gitlab does shares IDs between groups & repos, which might be causing the issue. Here is how I reproduced it in my local environment:

Setup Gitlab connection to my self hosted gitlab instance
The PAT has full access to the group & its repos (api/read_api/write_api and even the read_repo/write_repo)
Add project
create Blueprint / add data connection  | Click next step
Add scope
Select the "group", and received the error message.

Anything else
I've posted a full recreation video via Slack in private channel. It is very likely this is a gitlab specific issue, but should be addressable in the code since the miller select column component can use namespaced-IDs to avoid collisions.
Version
main
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @amanpatel , may I ask in which version did you find this bug?

@Startrekzky @amanpatel confirmed in Slack that he was on v0.15.x and he has upgraded to v0.16.0-beta10. This issue has already been fixed in v0.16.0-beta10 and thus can be closed.


mpv.com not present in shims for mpv app

mpv (extras bucket) request\issue:

Mpv works great, but it misses the shims\shortcut for the mpv.com (mpv cli), right now it creates the shortcut just for mpv.exe without the chance to use mpv.com too

mpv.exe from cmd openup the mpv player (already implemented)
mpv.com from cmd gives access to the cli for mpv (missed)

Could be useful to create shims for mpv.com (mpv cli)

The only way to create a shim for mpv.com is to manipulate the mpv.exe shim.
Could you try this?

create a copy of the mpv.json
add the following post_install script to the manifest"post_install": "
    (Get-Content \"$(shimdir $global)\\mpv.ps1\").replace('mpv.exe','mpv.com') | Out-File \"$(shimdir $global)\\mpv.ps1\" -encoding utf8
    (Get-Content \"$(shimdir $global)\\mpv.shim\").replace('mpv.exe','mpv.com') | Out-File \"$(shimdir $global)\\mpv.shim\" -encoding utf8
",

install it via scoop install .\path\to\mpv.json

Does this result in the expected behavior for cmd/powershell?

I played around with the code to see if I could manage to have both mpv.exe and mpv.com work from cli, I tried a lot of different combination but it seems that your hack it's the only one that "kind" of works.
I say kind, because right now with your hack here is what happens:

from cmd\powershell: when you run mpv or mpv.exe it gives you the mpv cli argument list

like when you run mpv.com with the "regular mpv installation" (not the scoop one), but the possibility to run the gui from cmd\powershell is gone now.
At least it creates mpv.exe (real gui executable) in the user\scoop\apps\mpv\current directory so the user can create a shortcut to that in the desktop or whatever to launch the gui too.
Doing the shortcut trick you have both options now to choose, if that what you tried to achieve yes it works and it's better than have no cli choice.
I was thinking though a more "clean" way to do it and have two separate commands in the cmd\powershell like the "real installation" does
so if you run from cmd\powershell mpv or mpv.exe it opens up the mpv gui and if you run mpv.com it gives you the mpv cli instead + shortcut to mpv.exe like now.
I thought that because the zip file already contains both executables (mpv.exe mpv.com) was just a matter of create:
"bin": [
    "mpv.exe",
    "mpv.com"

],
but it doesn't work I was expecting something like what ffmpeg does in it's manifest for ffmpeg ffprobe etc.
Am I missing something here?
Or maybe my mistake is that bin expects a list of only *.exe files and doesn't recognize .com is it?
I'm a bit confused.
Indeed if yours is the only hack possible (and it looks to be from all my tests) is better than the original one, at least you can manage to have both things with the shortcut, I think.
Please tell me if I'm not clear enough with what happened

Yes *.com files are not supported by the shim() function.
Should I add the post_install hack?

Yes thanks Richard to me seems the best solution have a shortcut and have the cmd line is it worth it to my humble opinion.
Especially for all the cmd-fu out there!
Thanks!


A tutorial on using gate noise/error parameters in simulation

Description

qiskit-terra version 0.6 supports setting noise parameters. It will be better if we can provide a tutorial on how to set the parameters, and how to use them to model noises.
Your Environment

QISKit version: terra 0.6
Python version:

Thanks! My colleagues in Keio and I will try to think about it.

I would rather wait until aer is out as we have not decided on the interface.


feat(iroh-net): Add counters for holepunching events

Description

Breaking Changes

Notes & open questions

Change checklist

[ ] Self-review.
[ ] Documentation updates following the style guide, if relevant.
[ ] Tests if relevant.
[ ] All breaking changes documented.

Okay, I think sent_disco_relay basically counts the holepunching attempts, and holepunched with this PR counts successful holepunching events.
sent_disco_relay - holepunched should give us holepunching failures.

I don't think this is the right way to measure this.
We don't really care about individual holepunch successes. We care about how many network configurations can switch to direct connections.
We can get closer to that metric via #2795 instead.
This might be nice to track additionally eventually, but we don't need this number now.


fix/standardise storybook control option and default options

Describe pull-request
This PR standardises the controls and default options for all components.
Solving issue
Fixes: AB#2483
How to test

Go to storybook link below
Check in all components and try out the controls.

NOTE
This PR also replaces svgs with out own icons, and gives the user the option of having either a webcomponent or native one.

Awesome work! Just spotted a few remaining defaultValue, if you fix those i'll approve it

Thanks! Yeah im working on those in a separate branch that I'm gonna merge into this one

I found really tiny things, I tried to go commit by commit.
I found some more stuff but they are not related to your changes. all in all, awesome work on standardizing buddy! 🚀

I found really tiny things, I tried to go commit by commit. I found some more stuff but they are not related to your changes. all in all, awesome work on standardizing buddy! 🚀

Thanks! Great, I re-requested your feedback now, after I fixed your suggested changes 😄


Add Download as ics option

Summary
Users can now download their schedules as an .ics file. This allows them to import a schedule into their calendar app.
Currently only the class schedule is include in the download. Finals and Custom Events will be added later.

Test Plan

Create a variety of schedules
Include various types of courses such as

AM and PM courses
Courses that start AM and end PM
Weekend courses (03095)
Overlapping courses

Attempt to download the schedules
Verify that the .ics file correctly imports to gcal and apple calendars

Future Followup

Add Google Analytics to "Download" Button #138
Add Finals and Custom Events to .ics download #139

I wanted to get this PR out asap but there are a still a few things that need to be done before it is ready to merge.

Include Final Exams and Custom events
Handle classes that occur on Saturday and Sunday

One example of a Saturday class is Drama 100. Haven't been able to find any Sunday classes yet

More testing! Make sure everything is working properly. I am very tired and haven't had time to test everything that I want.

What does this code do when a class's time is TBA? Hopefully doesn't break lol
Upload to google calendar
Verify that different schedules under the same username download properly

Known bugs

parseTime

Just realized am classes don't have an 'a' on the end of the time.  parseTime will return the wrong result.
For classes that start before noon but end after noon, parseTime will return the wrong result.

Closes #109

I think this is ready for review now! Although Finals and Custom Events haven't been implemented yet, the ability to download a class schedule is completed.
I figured we might want to get this out now since people are registering for classes. We can always finish up finals and custom events later.
One thought, should I move all of those helper functions in ExportCalendar.js to a separate file or just leave them as is?

Reviewed all thus far, looking beautiful. Will merge soon ™ . Comment when ready for re-review and second merge.

Comment to test reopen


dwm Integration.

I have a configured dwm with different colors, fonts, keybindings. logging in with ly is successful, but somehow it logs into vanilla dwm with all the defaults. this doesn't make sense. it is probably using the config.def.h somehow.
I have .xinitrc opening dwm in a loop:
while true; do
        dwm 2> ~/.dwm.log
        dwm >/dev/null 2>&1
done

so, I typically just enter my user name, then password and startx in the shell. why would dwm load in with default settings?

I think this is related to #121. Maybe this will be fixed once we handle utmp.

Could you give it another shot now #121 is fixed?

@cylgom The problem persists, unfortunately.

Having the same issues with latest version

Maybe i have found the root cause.
I am in Arch and besides a custom dmenu build i have the default dmenu version installed (as a dependency to clipmenu). In this case dmenu_run is located in /usr/bin.
Arch aliases /bin and /sbin to /usr/bin.
So since ly uses /sbin:/bin:/usr/local/sbin:/usr/local/bin:/usr/bin default path, it finds the default version first, even /usr/local/bin stays before /usr/bin.
The solution is to flip path in config.ini:
path = /usr/local/sbin:/usr/local/bin:/usr/bin:/sbin:/bin
@aliveupstairs Could you check if you have the same root cause?

@cylgom Would it be worth to have a bit more complex logic behind default path?
Like if /sbin and /bin are links, then put them to the end of PATH or even omit?

@FalyPr1nc3 trying ly after some time, there is now the ability to login to shell. I just run startx in ~/.profile if tty2. Since xinit does not work without additional parameters, I'm unable to run xinitrc option successfully (https://github.com/cylgom/ly/issues/181), so, I don't know if your solution is actually working since I'm not relying on ly to start X.
ly with dwm works on my end, @cylgom. Feel free to address Faly's observations or close the issue.

still doesnt work on dwm at least for me i get a permission denied error for .xinitrc

@jamescraft10 Please open a separate issue with the most information you can. Thanks!


wssh.send issues with special characters

First off Want to say great work on this terminal it is amazing.
The issue I'm having is due to the special characters not being escaped and no real way to send it a multiline raw string with special characters. or pass the wssh.send function a variable from Python or
This obviously works nicely.
wssh.send('ls -l');
But when say doing something like the below bash oneliner it looks like the only option is to manually escape.  the ' " special characters manually which is time-consuming and error-prone.
wssh.send('for phpver in $(ls -1 /opt/cpanel/ |grep ea-php | sed \'s/ea-php//g\') ; do echo "PHP $phpver" ; /opt/cpanel/ea-php$phpver/root/usr/bin/php -i |grep -Ei \'memory_limit|post_max_size|upload_max_filesize|max_execution_time\' && echo "" ; done');
undefined

I also tried to send the string as a variable was not able to get that to work.
Is there some way we can add support for escaping or raw strings support?
PHP has a library for doing this for its special characters.
https://www.php.net/manual/en/function.htmlspecialchars.php
echo htmlspecialchars($cPanelPHPVersions, ENT_QUOTES, 'UTF-8');
$cPanelPHPVersions = <<<EOT
'for phpver in $(ls -1 /opt/cpanel/ |grep ea-php | sed \'s/ea-php//g\') ; do echo "PHP $phpver" ; /opt/cpanel/ea-php$phpver/root/usr/bin/php -i |grep -Ei \'memory_limit|post_max_size|upload_max_filesize|max_execution_time\' && echo "" ; done

I also see Python has raw strings and also ''' that can be used to do comments and multiline strings.
'''
https://stackoverflow.com/questions/9589301/python-here-document-without-newlines-at-top-and-bottom
Is there a way that's recommended to pass these complex oneliners to the current session either via native python or passing a python variable to this which has special characters that it will accept?
I'm trying to allow rapid oneliners like the above to be buttons on a widget so a user can click the button from a list and its input into the terminal preformatted.
Just an image for visualizion

Using pysides QWebEngine to show the terminal. If that helps.

Base on my experience, the only special character for wssh.send is "\".
Did you test your oneliner in the terminal?

Command entered normally via browser copy and paste.
Command:
for phpver in $(ls -1 /opt/cpanel/ |grep ea-php | sed 's/ea-php//g') ; do echo "PHP $phpver" ; /opt/cpanel/ea-php$phpver/root/usr/bin/php -i |grep -Ei 'memory_limit|post_max_size|upload_max_filesize|max_execution_time' && echo "" ; done

Command entered normally without escaping via browser console.
wssh.send('for phpver in $(ls -1 /opt/cpanel/ |grep ea-php | sed 's/ea-php//g') ; do echo "PHP $phpver" ; /opt/cpanel/ea-php$phpver/root/usr/bin/php -i |grep -Ei 'memory_limit|post_max_size|upload_max_filesize|max_execution_time' && echo "" ; done');

Command entered via browser console with the inner ' escaped manually:
wssh.send('for phpver in $(ls -1 /opt/cpanel/ |grep ea-php | sed \'s/ea-php//g\') ; do echo "PHP $phpver" ; /opt/cpanel/ea-php$phpver/root/usr/bin/php -i |grep -Ei \'memory_limit|post_max_size|upload_max_filesize|max_execution_time\' && echo "" ; done');

I know the commands are good. I use them daily via copy and past and have an index of these that are all pretested.
Manually escaping them all to send over the console while possible is not exactly ideal.
Suggestion: Can we make the delimiter be like the Python docstring comments? This would make it so no escaping is needed and anything between the set of """ would be passed exactly how it is.
wssh.send("""completxcommandstringhere""");
Example:
wssh.send("""ls -l""");
Example2:
wssh.send("""for phpver in $(ls -1 /opt/cpanel/ |grep ea-php | sed 's/ea-php//g') ; do echo "PHP $phpver" ; /opt/cpanel/ea-php$phpver/root/usr/bin/php -i |grep -Ei 'memory_limit|post_max_size|upload_max_filesize|max_execution_time' && echo "" ; done""");

Use double quotes nested when the whole string is quoted by single quote, vice versa.


Not passing interp kwargs to DataArray.interp

https://github.com/xarray-contrib/pint-xarray/blob/ca4e282dda63c5b3ea39b9743fbf7165506b8e72/pint_xarray/accessors.py#L754
From the documentation it seems the kwargs argument is meant to be passed unmodified to xarray's DataArray.interp method. However from the line above it doesn't seem to do that.
The same goes for the assume_sorted argument by the way.

Thanks for the report, @martijnvandermarel.
This looks like a copy/paste error and is definitely not intentional. Also, Dataset seems to also be affected. I'd be happy to take a PR that fixes that, if you're up for it.


Google Analytic setDomainName

Sufia now includes a _setDomainName for google analytics.  It is currently being set to none.  What is it and what should we be setting it to?

It's used to track statistics from multiple domains. Since we're not doing this, setting is to "none" is considered the traditional approach.
See:
https://developers.google.com/analytics/devguides/collection/gajs/methods/gaJSApiDomainDirectory#_gat.GA_Tracker_._setDomainName


Uneven Footer Section

Description:
The footer sections do not have grid layout, the sections are unevenly placed
Image:

@soumita1234 please assign this issue to me under the label LGM-SOC


Threaded Resource Interactive Loader Errors

Godot version:
89560c2
OS/device including version:
Win10 64-bit
Issue description:
When running a interactive loader in a new thread, and attempting to cancel and clean it up between polling, I get the following errors when setting the loader reference to null.

Steps to reproduce:

Thread a Resource Interactive Loader
Set a cancel flag while it is loading.
Between polls check for cancellation and defer to a thread clean up function.
After waiting for thread to finish, clear references to the resource loader and thread.

Minimal reproduction project:
3.1 Thread Load Errors.zip

No longer reproducible as of 3.2.


feat: implement aggregation and subquery plans to SQL

Which issue does this PR close?
works on #8661
Rationale for this change
See issue
What changes are included in this PR?
Adds support for aggregation and subquery plan nodes converting to a SQL AST
For example, one can now convert
Projection: p1.id, COUNT(*) AS cnt
  Aggregate: groupBy=[[p1.id]], aggr=[[COUNT(*)]]
    Projection: p1.id
      Inner Join:  Filter: p1.id = p2.id
        SubqueryAlias: p1
          TableScan: person
        SubqueryAlias: p2
          TableScan: person

to
SELECT p1.id, COUNT(*) AS cnt
FROM (
    SELECT p1.id
    FROM person AS p1
    JOIN person AS p2
    ON (p1.id = p2.id)
)
GROUP BY p1.id

Are these changes tested?
Yes, new roundtrip tests are added
Are there any user-facing changes?
More plans supported

cc @backkem if you have a moment to review

I plan to review this PR carefully tomorrow

In general, I don't think it will be possible to convert all LogicalPlans to SQL as LogicalPlans can be built programatically and some will perfom calculations that are not possible to express in  SQL
Therefore, I think the best testing strategy is to do round trip tests that verify any pattern the DataFusion SQL planner makes can be converted back to SQL

This case will of course fail in the PR. I think that is OK, as we can work on support for more complex aggregations and additional edge cases we find in future PRs.

I think this is a wise strategy.


Move anti-diff and semigroupoid packages to a dedicated repository

The anti-diff and semigroupoid packages that currently live only on the feature/utxo-hd branch (https://github.com/input-output-hk/ouroboros-network/tree/feature/utxo-hd) are general purpose, so they should be moved out of ouroboros-network to one or more dedicated repositories. Furthermore, we should at least publish these packages in CHaP.
To consider: separation of functionalities
The functionalities that the two packages offer can be put into three categories:

A fingertree that is optimised for specific use cases
Differences/deltas for Data.Maps
Type classes and test utilities for semigroupoids/groupoids

We use these three functionalities to optimise operations on sequences (i.e., fingertrees) of Data.Map differences.

Should we bundle these functionalities into a single package, or should they be separated into three packages?
Regardless of the previous, should the package(s) live in a single repository?

The packages now live in https://github.com/input-output-hk/anti-diffs.

The packages now live in https://github.com/input-output-hk/anti-diffs.


Some places don't understand and need help...

example:
Request mRequest = new Request.Builder().url(url)
.post(sendPostBody(obj)).post(sendPostBody(obj)).post(sendPostBody(obj))//and more post
       .build();

if  i do that.I want to know how this is running,
Is a blocking type with sent doubles and more, or sent the body at once...

This forum is for bug reports with test cases and feature requests only. Please try your question on StackOverflow.


Need to stop Telemetry if socket closes

If the socket connects, then later closes while the AUT is still running, we need to stop the C++ level Telemetry from collecting stats if the socket closes, otherwise telemetry data piles up and eventually kills the app.
Ah, looks like I forgot a __hxcpp_hxt_stop_telemetry -- todo in hxcpp.

Hmm, stopping telemetry is complicated esp due to finalizers (potentially from a different thread? depends if stop is called from writer?) Anyway, for now, after socket closes I disable allocations and continue to drain the other stats to avoid a memory leak: 3780096d9b3


[FR] support setting tls version for target server creation

There is currently no support for configuring the tls version (e.g. v1.2, v1.3) on a target server. We can use REST API for this but would rather use apigeecli if possible.

released in v2.7.0


When will the code be released?

Thank you for your outstanding work. The paper has been online on Axriv for several months now. When will the code be released? Thanks for your attention！

Hi @workliulin, the code has been released!


[AutoPR netapp/resource-manager] Updating NetApp readme to include Java

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5502

(message created by the CI based on PR content)

Codecov Report

Merging #4715 into restapi_auto_netapp/resource-manager will decrease coverage by 3.93%.
The diff coverage is 100%.

@@                           Coverage Diff                            @@
##           restapi_auto_netapp/resource-manager    #4715      +/-   ##
========================================================================
- Coverage                                 53.44%    49.5%   -3.94%
========================================================================
  Files                                     10601     6346    -4255
  Lines                                    221864   177655   -44209
========================================================================
- Hits                                     118567    87946   -30621
+ Misses                                   103297    89709   -13588

Impacted Files
Coverage Δ

azure-mgmt-netapp/azure/mgmt/netapp/version.py
100% <100%> (ø)
:arrow_up:

...afactory/azure/mgmt/datafactory/models/__init__.py
50.67% <0%> (-49.33%)
:arrow_down:

...ights/azure/applicationinsights/models/__init__.py
51% <0%> (-49%)
:arrow_down:

...e/cognitiveservices/vision/face/models/__init__.py
51.11% <0%> (-48.89%)
:arrow_down:

...services/language/textanalytics/models/__init__.py
51.16% <0%> (-48.84%)
:arrow_down:

...iveservices/search/entitysearch/models/__init__.py
51.25% <0%> (-48.75%)
:arrow_down:

azure-eventgrid/azure/eventgrid/models/__init__.py
51.25% <0%> (-48.75%)
:arrow_down:

...ration/azure/mgmt/datamigration/models/__init__.py
51.27% <0%> (-48.73%)
:arrow_down:

...ure/mgmt/recoveryservicesbackup/models/__init__.py
51.46% <0%> (-48.54%)
:arrow_down:

...ce/azure/mgmt/containerinstance/models/__init__.py
51.8% <0%> (-48.2%)
:arrow_down:

... and 4340 more

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 5a6932e...8f0b4ad. Read the comment docs.

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/4727


Show user-friendly region name (WOR-1300).

Ticket: https://broadworkbench.atlassian.net/browse/WOR-1300
For consistency with how we display the region in other parts of the UI, display the user-friendly version at Azure billing project creation time.

Integration test failure is unrelated.


CASSANDRA-9007

I already excluded the new file from jenkins. Does just basic data loss validation. A good prototype to discuss what we want this harness to look like long term.
As is, there is going to be far far too much log parsing for errors. If the harness is going to be in Python, then changes need to be made in stress so the harness can get simple error codes returned.

Looking for review @aboudreault

The intent of reverse_test wasn't clear to me at first -- that's making sure that failed reads are recognized when a node is down, right? That depends on an implicit RF=1, so I think that should be documented. And if I'm totally off-base, then add a docstring explaining the whole test.
Works for me locally on trunk. If that's the intended scope, then +1 after documenting reverse_test.

I was similarly confused by base_test (esp since base has a bunch of
different connotations). Docstrings would be nice esp on the tests. Even if
the code is crystal clear they form a statement about purpose that might
help keep people in the future from perverting your test intentions and
confusing things.
On Thu, Mar 26, 2015 at 11:17 AM, Jim Witschey<EMAIL_ADDRESS>wrote:

The intent of reverse_test wasn't clear to me at first -- that's making
sure that failed reads are recognized when a node is down, right? That
depends on an implicit RF=1, so I think that should be documented. And if
I'm totally off-base, then add a docstring explaining the whole test.
Works for me locally on trunk. If that's the intended scope, then +1
after documenting reverse_test.
—
Reply to this email directly or view it on GitHub
https://github.com/riptano/cassandra-dtest/pull/210#issuecomment-86630149
.

that's making sure that failed reads are recognized when a node is down

I wasn't clear there -- I meant "reads that failed because a node was down are recognized as failures".

Clean up the one sentence I commented on:
https://github.com/riptano/cassandra-dtest/pull/210#discussion_r27245145
And then LGTM :tada:


fix(GitHubActions): migration from TravisCI

GitHub action workflow added
GitHub pull request template, bug report template, and feature request template added.
ITs improved to run on GitHub actions.
Some miscellaneous changes in various scripts to improve CI/CD jobs in GitHub actions.

:tada: This PR is included in version 0.0.11 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix typo in error message

Just fixes a typo in an error message.

Thank you for the pull request!  It appears that this is already fixed in the latest code


IDVA6-2053 - Bug Fix - EditMemberRoleViewData missing properties

Link to Jira:
https://companieshouse.atlassian.net/browse/IDVA6-2052
Changes:

Removed unneeded interface elements that caused errors in deployment

CI: No security warnings found


US01-TK09-Save-Button-Integration

Trello Card Link
https://trello.com/c/Ug4Md4or/81-e03-us01-task-04-criar-o-botão-de-salvar-para-salvar-os-estados-dos-inputs
Added
New Button added for save changes.

Please resolve the conflicts


[1.12] Use Namespace from AdmissionReview rather than Pod

It is not always the case that the metadata.Nameapace of a submitted object to an admission review is populated, and so the injector must use the namespace of the admission request itself when requesting for the Pod's identity certificate from sentry.
See https://github.com/kubernetes/website/issues/30574

For reference, the issue kubernetes/kubernetes#13952 describes precisely the problem (from kubernetes/website#30574)


Test codacy coverage push

Test codacy coverage push to see if secret passing works

/bot run coverage

/bot run coverage


error when using fvecVcf

I'm interested in training diploSHIC using a set of simulated vcfs that I have generated. As a first step I wanted to convert the vcfs to feature vectors using fvecVcf. Below is the command line I'm using, attached is the vcf I'm applying the command to
1_recap_human_demo_neutral_p1_slim.vcf.zip
:
python diploSHIC.py fvecVcf diploid 1_recap_human_demo_neutral_p1_slim.vcf 8 100000 1_recap_human_demo_neutral_p1.fvec --numSubWins 11
I get a long error directing me to a TypeError in allel.util. I've tried a few variations on this command and wound up with similar errors

Okay it looks like the issues are with the command line that you are giving above.

The chromosome name you are giving, 8, is not in the .vcf file.
Also the chromosome length, which you are specifying as 100000 but the vcf file says the chrom length is 1000001

using the following command line works on my system
% python diploSHIC.py fvecVcf diploid 1_recap_human_demo_neutral_p1_slim.vcf 1 1000001 1_recap_human_demo_neutral_p1.fvec --numSubWins 11
/Users/adk/miniconda3/envs/diploshic/bin/python makeFeatureVecsForChrArmFromVcfDiploid.py 1_recap_human_demo_neutral_p1_slim.vcf 1 1000001 None 1100000 11 None 0.25 0.75 None None 1_recap_human_demo_neutral_p1.fvec
Warning: a mask.fa file for the chr arm with all masked sites N'ed out is strongly recommended (pass in the reference to remove Ns at the very least)!
makeFeatureVecsForChrArmFromVcfDiploid.py:123: DeprecationWarning: time.clock has been deprecated in Python 3.3 and will be removed from Python 3.8: use time.perf_counter or time.process_time instead
  startTime = time.clock()
1-100000 num unmasked snps: 361; unmasked frac: 0.998540
100001-200000 num unmasked snps: 290; unmasked frac: 0.998520
200001-300000 num unmasked snps: 408; unmasked frac: 0.998850
300001-400000 num unmasked snps: 362; unmasked frac: 0.998450
400001-500000 num unmasked snps: 305; unmasked frac: 0.998580
500001-600000 num unmasked snps: 400; unmasked frac: 0.998550
600001-700000 num unmasked snps: 334; unmasked frac: 0.998510
700001-800000 num unmasked snps: 232; unmasked frac: 0.998480
800001-900000 num unmasked snps: 358; unmasked frac: 0.999000
900001-1000000 num unmasked snps: 337; unmasked frac: 0.998700
completed in 1.33844 seconds

@kahlquist-brown can you confirm for me that this works for you?
thanks

This works! Thanks for your help


can.mustache & can.stache overly lenient in parsing

I was reviewing some code today with my client and I noticed a line of code that looked like the following:
{{#if someCondition}}
   ...
{{/someCondition}}

And found that it worked. What's weirder is I can do the following like the following
{{^someCondition}}
    ...
{{/each}}

and this would properly parse.
My question is - are we intentionally this lenient with our template parsing? I know if I attempt to parse with Mustache outright I get an error returned for my view. E.g.
// template.mustache
{{#items}}
    {{name}}
{{/if}}

// template.json
{
  "items": [
      {"name": "red"}
  ]
}

// results
Error: Unclosed section "items" at 24

Thoughts? Further example with latest here - http://jsfiddle.net/co48cda2/3/

So, I believe mustache and handlebars support ending with just {{/}}; however, I think they also throw warnings if the ending exists and does not match.  We should probably give a warning.

Closing for: https://github.com/canjs/can-stache/issues/13


Change all dense matrices to sparse

Adjacency matrices representing users and items in an algorithmic system are sparse by nature. Dense matrices are not an efficient representation of these components.
https://docs.scipy.org/doc/scipy/reference/sparse.html

Closed (a while ago) with #88


Update Excel.Range.Find.md

Improved the Description for the SearchDirection parameter.

Modified the Example description to better describe what the code does.

Added Sub/End Sub statements to Example 1 code and added Dim statements for variables so that data type is clear.

Added a 2nd example to show usage for Finding strings/substrings.

Docs Build status updates of commit 39f6985:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Excel.Range.Find.md
:bulb:Suggestion
View
Details

api/Excel.Range.Find.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'. NOTE: This Suggestion will become a Warning in late October, 2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit b103a33:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Excel.Range.Find.md
:bulb:Suggestion
View
Details

api/Excel.Range.Find.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'. NOTE: This Suggestion will become a Warning in late October, 2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 4ec9ca0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Excel.Range.Find.md
:bulb:Suggestion
View
Details

api/Excel.Range.Find.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'. NOTE: This Suggestion will become a Warning in late October, 2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Migrate ResourceId from crc32 to (crc32, filesize)

ArkLib defines ResourceId as 2 values of Long type:

CRC-32 checksum
Size of the file

Navigator and arklib-android defines ResourceId as just CRC-32 checksum:
typealias ResourceId = Long

We need to make Navigator use exactly the same what is defined in ArkLib.

We also need to create a migration procedure in https://github.com/ARK-Builders/ark-cli which would replace old ids in .ark/tags and .ark/meta files with newer ids.

We can perform the migration automatically by introducing a version file in .ark subfolder. If it's absent or older than we need, then backup must be performed (see https://github.com/ARK-Builders/ARK-Navigator/pull/264) and the migration must be performed.


Crash when frames argument passed to blocking stream write method exceeds internal buffer size

Sample code:
let output_settings = try!(pa.default_output_stream_settings::<i16>(CHANNELS, SAMPLE_RATE, FRAMES));
let mut output_stream = try!(pa.open_blocking_stream(output_settings));
try!(output_stream.start());
try!(output_stream.write(FRAMES * 2, |output| {
    for i in 0..2*FRAMES {
        output[i] = 10000;
    }
}));

There are no any check for size in neither Stream::write nor Buffer::slice_mut.
Stream::write design differs significantly from Pa_WriteStream: while the latter accepts external buffer and has no frames limit, first one uses internal buffer and thus has such restriction. Why is that?

This is a bug, and you're right, the design does not match the original and should allow an external buffer to be used.
Feel free to open up a PR, otherwise I'll try to get around to it soon.

Just ran into this bug tonight. Is this crate still being actively developed? I'm thinking of building on it, but it looks like it hasn't had any commits to master in 9 months…

Debian stretch, rust-portaudio 0.7.0, pulseaudio, Haswell box.
Just ran into this bug again tonight. :-1: Boy I'm dumb: I'd forgotten about finding it and reporting it six months ago until I went to make a new issue just now. The internal buffer can't be very big: 16 16-bit samples seems like enough to crash it on my box.
 In debug on my reasonably-fast box, there doesn't seem to be a feasible buffer size: 8 samples occasionally underruns, and 16 samples crashes on drop at the end of my play function. Even when no underrun is reported, at small buffer sizes there are pops that sound an awful lot like underruns to me.
Here's some demo code that fails consistently at various buffer sizes: http://github.com/BartMassey/portaudio-bug
Why does write() take a size (as u32) and a callback? Can't we just have a method that takes a slice of samples to write?
Seriously rethinking my life choices. If I did some fixes and put a pull request in here would anyone look at it?

I feel your pain. While this library is not actively developed, I think @mitchmindtree definitely accepts pull requests and he will make a review once in a while. He said he often misses notifications, so you may end up needing to ping him multiple times.
That said, if you want to go fast the best way is to make and use your own personal fork.

Thanks much for the note! Really appreciate your response.
Given that this bug is likely exploitable, it probably ought to get fixed, but I've switched to portaudio-rs, which seems to work quite well for me, so I probably won't get to it soon.


provide a way to not download tests/, examples and other such common directories

Problem
By default cargo includes everything from the root of the package (or almost, see https://doc.rust-lang.org/cargo/reference/manifest.html?highlight=files#the-exclude-and-include-fields)
This is very nice for archiving purposes by crates.io, it allows example scraping from docs.rs and probably other purposes I haven't encountered, but it has an issue: it makes lots of CI downloads heavier than they need to be.
Looking through the dependencies we use at $work I see around a 100MiB of unused files just counting tests/, but there are also .github, benches/, examples/ and probably more I missed, and we have only ~600 deps out of the 138k crates on crates.io (without even counting all the available versions).
We have a cache to avoid spamming crates.io whenever possible so it's not like we request an extra 100MiB on each CI but still, I don't like putting pressure on it if we can avoid it.
As linked above, https://doc.rust-lang.org/cargo/reference/manifest.html?highlight=files#the-exclude-and-include-fields is intended to help with that but it's not automatic: people need to consider and maintain it for each crate (and from the start, else older versions will still have all the unused files)
Proposed Solution
We may need a way to say to cargo only fetch the absolute necessary to build the crate. Two issues with that:

What is the "absolute necessary": probably at least src/, Cargo.toml, license files, build.rs ? Some -sys crates may need more
Changing that silently in a backward compatible way is probably hard ? I think it would break the hashes so it could not become the default automatically at the very least.

It could take the form of cargo fetch --no-extras (to be bikeshedded) ?
Notes
Note: I'm excluding binary crates here, but they could easily be included too, if only so cargo install downloads less if possible, but I don't think binary crates are downloaded as much as library ones.
I also don't know if the benefits would be worth it, since it could mean one of:

Recomputing the set of files on each download
Storing two archives of the same crate&version

I have no data saying the bandwidth gains would be enough to offset either the compute or space gains, if anyone knows (the infra team ?) I would be happy to be proven wrong!

For cargo vendor, we also have #13474.
Breaking down the extraneous files you mentioned:

.github there is no reason to include and that is on the package for including it.
For examples/, that can tie into the documentation and can be important for including.
For tests/ and benches/, that is important for crater and people wanting to do similar for crater.

The challenge is knowing what is needed for build or not among the files.  A heuristic can work but that can only go so far (much like we provide package.include and package.exclude in case our existing heuristic isn't enough.
There is also the question of "what is essential".  For example, explicitly referenced targets are sometimes required just to parse manifests today (see #13456).
There would need to be figured out to get this to work.

I would like to call out that the #[path] attribute, while not widely used, would cause problems if Cargo excludes files without recognizing where they point to. Besides, changing or disable the heuristics might affect other parts in Carog, like https://github.com/rust-lang/cargo/issues/4587, though this doesn't mean that Cargo cannot change the current rule.

I don't think we re-check hashes but i would not rely on that never changing.

i would not rely on that never changing.

It should be a line or two of bash, it can be deleted if an update breaks it :)

Update: cargo is too well-made and check hashes even in vendored dependencies so my two lines of bash won't work 😞

A simpler way to do this then would be to allow not checking hashes when compiling ?


Write a module to fetch and process MARC

Function fetch raw marc from folio using selected uuids (using folio client)
Functions that apply the rules to the marc reocrd needed for all vendors
Function to write processed marc to file location for vendor

See https://docs.google.com/document/d/1NAbY9L6G16DItIBjtRQKFDcL6NN5cWK8VfsFCT48vTA/edit for marc processing rules.

Breaking out these bullet requirements into separate tickets.


Recover files after reset with paper key

Hi,
I recently reset my account but I lost access to all my files. I have my old paper key. Is it possible to recover my files? I have a folder that is called "...files before account reset.." or something but it says that I do not have access when I try to open it.
Any help would be appreciated.
Thanks

Hi @ag14774, we have not built a way to recover files using a paper key, and I'm not sure it's actually possible.  Even if it were, it would be pretty hard for us to build a way to do that securely since an account reset is effectively saying: "all the keys associated with the old account are being revoked", and trusting a revoked key is bad news.
The "files before account reset" folders are pretty useless for your private folder, I agree.  They are more for the case of shared folders.
We probably need to give you better warnings about potential data loss before letting you reset your account.  Sorry we can't be of more help after the fact.

Ok no problem. Is it possible to reset the filesystem so that those folders disappear. It doesn't let me delete them

You're asking about the "files before account reset" folders?  You can "un-favorite" them so they don't show up in your list anymore with rmdir /keybase/private/ageorgiou (files before account reset ...)" (using the full date in the folder name, I don't know it offhand).  That assumes you're on mac/Linux.  Windows should be similar with the command that deletes directories.

That worked thanks!


Any plans on creating an abstraction package?

Many of the Microsoft.Extensions packages comes with an abstraction package containing the fundamental interfaces for a component. This can be a lifesaver when trying to untangle dependency graphs/verison constraints in larger projects.
With this in mind - and in the light of the dependencies that the feature toggle package have taken - are there any plans to extract IFeatureManager, IFeatureFilter et. al. into a abstraction package?

My proposal would be to implement the following:

Microsoft.FeatureManagement.Abstractions
Microsoft.FeatureManagement.FeatureFilters
Microsoft.FeatureManagement.Core

The existing package would have dependency on all the new packages.
Thoughts @zhiyuanliang-ms on doing this split

@thompson-tomo Thank you. I will discuss your proposal with the team.


Master

This reverts commit f910e6ad4042aa3068d25b213aabc6e8125a385f, reversing
changes made to b54c37e70370fac4546734a53d0235a31a0cd040.

Why is this reverted? I was already using v13 on one of our projects...

Sorry, I created this PR by mistake, I was resolving an issue in one of my projects

Oh sorry, I thought this was merged, but it was just closed. Np! Thx for feedback


Support for TabletopAdudio.com tracks

This is a feature request. My apolgies if this is the wrong spot for it.
Any chance of getting LavaPlayer to play TabletopAudio.com tracks?  The composed tracks specifically, not the sound pads.
I'm currently using Redbot (which uses Lavaplayer) for background audio in Discord chat when running RPGs, and while I can download the files to local and then play them, it would be nice to be able to just play them directly, especially as new songs are added to the site.

Please provide a sample url to a specific track.

https://tabletopaudio.com/download.php?downld_file=196_Crossing_the_Styx.mp3

That can already be played with HTTP source manager. A Feature request only makes sense when you have a page from which links should be extracted (for example https://soundcloud.com/thefatrat/thefatrat-monody-feat-laura-brehm-1/)

Closing as sample link already works with HTTP source.


The "src" option from the configuration file will be ignored if user sends an unknown option to CLI

What is your Scenario?
I am currently creating my own js configuration files according to testCafe Documentation
When I try to run those config files through CLI with a command from root folder
TestCafe gives me a message in console "The "src" option from the configuration file will be ignored."
I am sure that the src path I am using is 100% correct since currently I'm using the same path to specify it in the CLI command and it works correctly, taking only the tests from the path I've provided + applying the configuration file filtering and set ups.
I tried to use default testcaferc.json file instead of creating a custom one but src is also ignored in this case.
Note that providing path in cli works as expected but it doesn't suite my needs since I'm trying to remove this cli dependency and move it to configuration files.
In the TestCafe documentation I didn't find anything about this error or why src is ignored.
StackOverflow:https://stackoverflow.com/questions/76231497/the-src-option-from-the-configuration-file-will-be-ignored
What is the Current behavior?
Using a command like testcafe firefox ./Tests/Desktop/**/*.js --skip-js-errors --env staging --debug-on-fail --config-file customConfig.js
Which includes --env ( unknown option for testCafe ), Test cafe gives a message in console "The "src" option from the configuration file will be ignored."
What is the Expected behavior?
SRC option not to be ignored if it is provided in the config file.
What is your public website URL? (or attach your complete example)
Nope
What is your TestCafe test code?
Doesn't matter
Your complete configuration file
module.exports = {
disableHttp2: true,
reporter: 'spec-time',
disableMultipleWindows: true,
proxy: 'PROXY',
clientScripts: 'SCRIPTSPATH',
src: ['./Tests/Desktop/**/*.js'],
filter: {
testMeta: {
type: 'regression',
priority: 'high',
},
},
};
I also tried same configuration but with JSON format
{
"disableHttp2": true,
"reporter": "spec-time",
"disableMultipleWindows": true,
"proxy": "PROXY",
"clientScripts": "SCRIPTSPATH",
"src": "./Tests/Desktop/**/*.js",
"filter": {
"testMeta": {
"type": "regression",
"priority": "high"
}
}
}
Your complete test report
No response
Screenshots
No response
Steps to Reproduce
1.create a config file with src
2.provide an additional unknown option to cli command
3. run the command
TestCafe version
2.4.0
Node.js version
No response
Command-line arguments
testcafe firefox --skip-js-errors --env staging --debug-on-fail --config-file customConfig.js
Browser name(s) and version(s)
No response
Platform(s) and version(s)
No response
Other
No response

Hello, you observe expected behavior. TestCafe tries to resolve browsers and sources from CLI. Since you are passing an unknown parameter, TestCafe decides that this is your test source. That is why you are getting the warning message about an option override. Could you please clarify why you are passing an unknown option to a TestCafe command and what behavior you are trying to achieve?

Hi again, I just found out that passing node arguments will also result in the same behavior. This behavior is incorrect. We will investigate and fix it.

We tried to reproduce this behavior with a simple example using the following command: "npx testcafe chrome --env".
It is expected that the "--env" flag overwrites the 'src' configuration option because "--env" is a valid name for a test folder.

passing node arguments will also result in the same behavior

As for passing node arguments, Testcafe supports not all node flags, so some of them can be processed as valid folder names.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/103647

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/110497

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/113838

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/115749


on a port using TLS, suppress the TLS alert on unexpected http requests

version 0.8.2
This is the caddy file
<IP_ADDRESS>:9999
tls ./key/csr.pem ./key/key.pem

Calling https got the expected result, but then I call http://[[local IP]]:9999 in Chrome, then i got the following file (I zipped it)
download.zip
How could I do manually redirect from http to https ?

You've asked Chrome to load a page with HTTP (http://[[local IP]]:9999) but the server is serving HTTPS on that port. Your URL needs to start with https for it to work properly.

If the http is not serving, it should not reply anything. That what I expected.
I would like to know how to redirect the user that call http to be redirected to https as well.

Allow me to rephrase your second (feature) request: You want Caddy to redirect visitors that did not send a TLS client hello (https ← http in tls) where one was expected.
Although it's possible for the http server to react to something like that (for example by serving an error page), no browser would accept a subsequent server-initiated »protocol upgrade«, even if we implemented it. ("Upgrade to TLS to proceed." – Chromium used to support it.)
FYI

https://tools.ietf.org/html/rfc2817
WebSockets use HTTP upgrade

Thanks for the correction and enlighten.
By the way, if I tried http://google.com:443 on the browser it returns error and not download anything how do I make Caddy act like that ?

The library Caddy uses for TLS responds with a binary packet, which means an error in the TLS protocol, but is interpreted as being the requested content by the browser.
Google's server does not respond at all, resulting in no download.
I will see if we can suppress that TLS alert easily.

 Its sending is optional according to specs.
We would need to wrap the output of tls.NewListener (server/server.go) and check the very first bytes. If they are the alert, then alter the response (push an empty response), else passthrough.
This would be something like a copy of gracefulListener and  gracefulConn.

Would it be possible to implement behavior similar to nginx using error code 497 to allow users to specify the behavior for HTTP requests to an HTTPS port (http://stackoverflow.com/a/14241127/2163024)?

@mrdziuban, yes to your question.

Thanks @wmark. Would it be best if I opened a new issue to track that request?

The issue is easy (»trivial«) enough for a beginner to tackle. ;-)

@wmark I spent some time looking into it, but I'm completely new to Go development and had trouble figuring it out. I'd be happy to help with more guidance, but otherwise I don't have the bandwidth to implement it myself.

@mrdziuban That's okay, since my work in #1430 steps on your toes a little here. In that PR I implement a Listener that manually reads the Client Hello, so what we can do is maybe build on that. Also, in a few days we're getting rid of the gracefulListener and gracefulConn with Go 1.8 coming out. So maybe revisit this in a week or so?

Thanks @mholt, I'll check back in a bit. I appreciate all the hard work!

Is there any interest in this now? I'm not particularly inclined to do it myself, since, frankly, nothing here is broken, and I'm not sure that a blank page is less confusing. From a technical standpoint, it's almost more confusing because it feels like something worked  and the server just isn't sending any data -- although that would be true, at least we can tell the server is responding with the correct protocol on that port and that the error is in the request.
You may disagree, and that's fine, but it probably won't convince me to work on it myself. This discussion can continue, if desired, and others are welcome to try to submit a PR, but any a solution will have to be thoroughly vetted for elegance and performance (and include tests) since it affects a critical part of Caddy's code.

I don't use Caddy anymore, just to manage expectations.
Afaik, nginx displays a dedicated error page in this case. Google's servers return an empty reply.


Lib exported

https://github.com/andrewthad/impure-containers/blob/c930e221918de21aedfedec68e53946bd406107b/impure-containers.cabal#L19
It's great that you put stuff for others on Hackage, but please, use a bit of time to clean it up :)

This is now fixed. I'll close this once I release a new version to hackage.

http://hackage.haskell.org/package/impure-containers-0.2


Screenshot Update - PRs welcome?

current screenshot is for HMAC setting is:

Would it be ok if I update it and outline in red the new UI with this image?

Or does it need to be full-screen size like this?

Let me know which one, and if it's ok for me to submit a PR.

PRs welcome, but I think I want to hold off on updating this just yet. I don't think this is fully rolled out yet and not everyone get's this new UI. Also, if you have a mac, I like to use a screenshot of a chrome incognito browser with the drop-shadow/rounded corners (⌘+Shift+4 space)

k.  Circle back and tag me if you'd like me to do it.


docs: contributing get started new content

Closes #3621
This issue is a complete re-write of the /contribution/overview page.
Changelog
New
All headers contain new content:

Getting started
Contribution gallery (updates to the Airtable iframe can be tracked #3718 )
Carbon champions (reviewers, please verify names and work are accurate. We are getting permission to display names #3753)
Types of contribution
The process

o