Add interactive terminal UI similar to turborepo, to see logs separately for each task

Turborepo has a really cool UI to see tasks logs separately for each task, here is a video: https://turbo.build/blog/turbo-2-0#new-terminal-ui
the use case is to be able to read logs for each task separately and remove prefixes, the user can select which task they want to see the logs for.
I can work on a PR if you like the idea, it would mean adding the blessed npm dependency and an —interactive option

Hey. This is an interesting take on the problem of handling several long running command at once, and one I'd be happy to integrate into concurrently.
There was some discussion before around the same issue, see #247 / #248.

I can work on a PR if you like the idea

I feel like this task would be large, but I haven't tried tackling it myself. Maybe it's not.
If it turns out actually large, I'd break it up in several PRs.
There certainly are UX considerations to be made which we should discuss, but they can probably be made in each PR.

and an —interactive option

I'm not sure i'm onboard with this flag name, it sounds like --handle-input should've been called, tbh.
Maybe --ui? Not sure either.


[bug] ETL'd collections do not get listed in the institution landing page

The following institutions have ETL'd collections:

Oakland Museum of California: https://calisphere-stage.cdlib.org/institution/94/collections/

https://calisphere-stage.cdlib.org/collections/15685

Corona Public Library: https://calisphere-stage.cdlib.org/institution/63/collections/

https://calisphere-stage.cdlib.org/collections/19967

Harrison Memorial Library: https://calisphere-stage.cdlib.org/institution/279/collections/

https://calisphere-stage.cdlib.org/collections/27423

California State Archives: https://calisphere-stage.cdlib.org/institution/153/collections/

https://calisphere-stage.cdlib.org/collections/27297

Santa Cruz Public Libraries: https://calisphere-stage.cdlib.org/institution/245/collections/

https://calisphere-stage.cdlib.org/collections/27148
https://calisphere-stage.cdlib.org/collections/27143
Note that SCPL does have one collection listed, which was not ETL'd.

These ETL'd collections exist on -stage, but are not listed on the institution landing page. They are found in the collections browse list.

Just checked all these against production calisphere and confirmed that all the Collections are listed in the institution landing page now.


Error when trying to update string to text or keyword

In 2.4.5 the mapping used something like "type":"string".  To make this easier (I guess) ElasticSearch has changed this to "type":"keyword" and "type":"text".
This is all fine - but when you try to post an updated JSON to a mapping which used to look like:
"link": { "type" : "string", "index": "not_analyzed", "store" : "yes", "cql_collection": "singleton" },
"title": { "type" : "string", "index": "analyzed", "store" : "yes", "cql_collection": "singleton"},

and now looks like:
"title": { "type" : "text", "index": true, "store" : "yes", "cql_collection": "singleton"},
"native_guid": { "type" : "keyword", "ignore_above" : 10922, "index": true, "store" : "yes", "cql_collection": "singleton"},

It complains with:
{"error":{"root_cause":[{"type":"illegal_argument_exception","reason":"Automatic downgrade from [keyword] to [string] failed because parameters [cql_collection] are not supported for automatic downgrades."}],"type":"illegal_argument_exception","reason":"Automatic downgrade from [keyword] to [string] failed because parameters [cql_collection] are not supported for automatic downgrades."},"status":400}

I suspect this is because the code which does the "automatic downgrades" is not knowledgable about the added parameters which Elassandra has added?  It seems to be around here:
https://github.com/strapdata/elassandra/blob/dee357d3ddcdca147634699020e9dda0efa4eb31/core/src/main/java/org/elasticsearch/index/mapper/KeywordFieldMapper.java#L135

Yes, you could try to add cql_collection (and all cql_xxxx) to SUPPORTED_PARAMETERS_FOR_AUTO_DOWNGRADE_TO_STRING in KeywordFieldMapper.java, but as explain in #144, i think it's safer to rebuild new indices, eventually with a temporary datacenter.


Implemented async file management, updated all inits with async task return param

Please don't hard-code this into the RenderContext. In the future, the RC.Viewport should be EITHER set directly in Fusee-App-Code (either once per frame in RenderAFrame (recommended) or in the Fusee-App's override Resize method (less recommended)) OR using the new CameraComponent currently in development.

My resize wasn't working anymore after I've replaced the RenderCanvas.cs with the one from the feature/wasm branch.
This line therefore just restores how it had already been implemented in develop before:
https://github.com/FUSEEProjectTeam/Fusee/commit/de44485223fca81eff3200fab72a4efd9e7b3e37#diff-455487616bd8131db144fcb86b923f5fR181
Should I re-add RC.Viewport() to every RenderAFrame (all examples and Player.cs) and delete this line?


[AutoRelease] t2-dynatrace-2023-08-10-67754(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4408
Recording test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=4821

issue link:https://github.com/Azure/sdk-release-request/issues/4408


New Strawberryshake project won't build

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Strawberry Shake
Describe the bug
Following the get started guide (https://chillicream.com/docs/strawberryshake/v13/get-started/console) doesn't work.
I made a fresh solution, followed the guide but I get a build error. The code won't compile.
Steps to reproduce
Reproduction steps:

Installed Visual Studio 2022
Followed this: https://chillicream.com/docs/strawberryshake/v13/get-started/console to create a new solution and console app project, using chilli cream's Conference gql server (targeting .Net 6.0)
Attempted to build solution as per step 5

Expected:
Code compiles
Actual:
Build error:
error MSB4057: The target "GenerateGraphQLCode" does not exist in the project.
Relevant log output
Build started...
1>------ Build started: Project: strawberryTest, Configuration: Debug Any CPU ------
Restored D:\CodeRepos\consoleapp\strawberryTest\strawberryTest\strawberryTest.csproj (in 26 ms).
1>C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\Roslyn\Microsoft.CSharp.Core.targets(57,11): error MSB4057: The target "GenerateGraphQLCode" does not exist in the project.
1>Done building project "strawberryTest.csproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
========== Build started at 6:43 PM and took 01.690 seconds ==========

Additional Context?
none
Version
13.0.5

Can you attach a demo project here showing this issue?

From the error message it looks like the StrawberryShake.Server package is missing in your project.

I have added the reference as per. The issue I think I'm now having is this warning:
Warning	MSB5029	The value "***.cs" of the "Include" attribute in element  in file ".nuget\packages\strawberryshake.server\13.0.5\build\StrawberryShake.Server.targets (23,16)" is a wildcard that results in enumerating all files on the drive, which was likely not intended. Check that referenced properties are always defined.
There is no "Generated" folder, which I think you've done away with, but I also can't now get the project to build. In addition it seems to be including files from outside my project in the build process and throwing errors for them as well as files in my actual project.
I have the .graphqlrc.json now set to:
"documents": "./Documents/.graphql",
with my queries in a Folder called documents, because if I leave them in the project root, with default "documents": "**/.graphql", I get error:
GQL	An item with the same key has already been added. Key: JSON.
But either way I can't seem to get the client to build.


Device management

Attempt to improve backend code for device and ressources management with OpenCL.
Objectives are:

improve device switch
improve code structure
enquiry/solve memory issue when running with dask

Closes clEsperanto/pyclesperanto#283
Closes #387

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 52.38095% with 100 lines in your changes missing coverage. Please review.

Project coverage is 46.68%. Comparing base (5c32a1a) to head (f225442).
Report is 2 commits behind head on master.

Files with missing lines
Patch %
Lines

clic/src/opencldevice.cpp
57.69%
18 Missing and 37 partials :warning:

clic/src/openclbackend.cpp
44.73%
11 Missing and 31 partials :warning:

clic/src/array.cpp
25.00%
0 Missing and 3 partials :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #399      +/-   ##
==========================================
+ Coverage   46.54%   46.68%   +0.13%
==========================================
  Files         199      199
  Lines        3848     3963     +115
  Branches     1767     1804      +37
==========================================
+ Hits         1791     1850      +59
- Misses        692      718      +26
- Partials     1365     1395      +30

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

After a local tests, this PR solve the memory issue when using dask with multi-gpu


AEM 6.3 support for hide dialog values

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: <IP_ADDRESS>
[ ] ACS AEM Commons Version: 3.14.6
[ ] Reproducible on Latest? yes

Expected Behavior
When using conditional dialog fields like described in
https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/master/content/src/main/content/jcr_root/apps/acs-commons/touchui-widgets/showhidedialogfields/source/showhidedialogfields.js
Is should hide field in dialogs based on values of other fields.
Actual Behavior
It works in AEM 6.1 but not in AEM 6.3
Steps to Reproduce
Follow description in
https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/master/content/src/main/content/jcr_root/apps/acs-commons/touchui-widgets/showhidedialogfields/source/showhidedialogfields.js

https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/1400 resolves this issue


 The trailing ] gets included in that operation in error. Should not be.

Does this issue occur when all extensions are disabled?: Yes/No

@mwpowellhtx What programming language are you using? Are those brackets present in the file when the file is opened or were they automatically-inserted when typing [ ?

@alexdima SQF. Concerning the second question, I'm not positive, but I don't think it matters either way.

For overtyping ], we keep track if the ] was automatically inserted when typing [. Pressing backspace does not look at that, and perhaps it should, similar to overtyping.

There is now a new editor option, editor.autoClosingDelete which is auto by default, which means it will delete closing characters only if those characters were automatically inserted by the editor recently. The other options are always which was the current behavior until now, and never which means to never delete closing characters. This is now consistent with editor.atuoClosingOvertype.

@alexdima And by "now" some idea as to which version? Thanks, and good to know, thank you.

next insiders / stable 1.55.x

Perfect, thank you, sir(s).

To verify:

open a file with the following existing text [|], and press backspace. The ] will not be auto-deleted because it was not auto-inserted.
type [ which will insert ] and press backspace. The ] will be auto-deleted because it was auto-inserted.
the setting editor.autoClosingDelete can be configured to always or never to influence this behavior, similar to editor.autoClosingOvertype.

Thanks. 90% positive that any pair will work, AFAIK, I just chose square brackets, in this instance, []. i.e. (), {}, "", '', etc.

This works great -> verified. However I found one case of auto unexpected auto closing behavior. It is already happening in stable however filled this issue https://github.com/microsoft/vscode/issues/119612

There is now a new editor option, editor.autoClosingDelete which is auto by default, which means it will delete closing characters only if those characters were automatically inserted by the editor recently. The other options are always which was the current behavior until now, and never which means to never delete closing characters. This is now consistent with editor.atuoClosingOvertype.

I need to point out that this change, which I consider a 'breaking' change, is not mentioned anywhere in the latest release notes: https://code.visualstudio.com/updates/v1_55. I got here by searching for 'autoClosingDelete' in the issues page,  a config option which I stumbled upon by chance in the settings page while trying to figure out what was wrong with the editor.


'types' parameter is deprecated as of Feb-2016

https://developers.google.com/maps/documentation/javascript/places?csw=1#place_search_requests

Notice: The implementation for types in text search requests is changing. The types parameter is deprecated as of February 16, 2016, replaced by a new type parameter which only supports one type per search request. Additionally, the establishment, finance, food, general_contractor, grocery_or_supermarket, health and place_of_worship types will no longer be supported as search parameters (however these types may still be returned in the results of a search). Requests using the deprecated features will be supported until February 16, 2017, after which all text searches must use the new implementation.

Hmm... Looks like the deprecation notice is related to direct API usage, while google.maps.places.Autocomplete component is not affected.


Add Darklang to showcase

Thanks, LGTM
Can you please sign the CLA? Required by Meta/FB

Can you please sign the CLA? Required by Meta/FB

Done, thanks!


settings: Rework user_group save_discard widget.

Added a consistent save/discard widget inside the
USER GROUP widget and replaces the inconsistent
"Click outside to save or click discard to discard" UI it had.
Allows users to edit user group-settings one user group at a time.
Fixes: #16453

GIFs or screenshots:

@timabbott could you review this PR, while you are free? Thanks


Support for header families

Very recently i came across a certain limitation of typed-api, as title suggests it is about header families, for example all headers starting with l5d-ctx (linkerd context).  Program may not be interested to use all of the headers of that family but it might be required to forward those when doing request to some other service. Unfortunately it is impossible to do this given the current API, but i think it should be pretty simple to add something like the following:
  val MyApi = "header" :> "family" :> Header[String].family("my-hdr-family") :> Get[Json, A]

Given the following definition one can interpret it as to extract all headers starting with my-hdr-family on the server-side, and to provide zero or more headers on the client side.

I am also not sure whether headers with dashes in names are supported at all (one have to use symbol syntax to define their names, while scala does not support dash inside symbol definition)

Hi @wookievx ,
thanks for using typedapi. Regarding your issue. Unfortunately, I don't understand what you try to explain. I have never heard of header families and googling for it didn't bring a meaningful result. Could you maybe give me a bit more context? Maybe an example which shows what the client sends and the server reads.
Furthermore, I am currently preparing 0.1.0 in the 0.1.0-release branch (release end of this week 🤞 ). There I added new features for headers. You can now add static headers in the API :> Header("Accept", "*") which will send this header on every request/response but you don't have to provide it. You can even say you only want to send it from the client (request) :> Client("Accept", "*") or server (response) :> Server("Accept", "*"). I used that to enable CORS.
Would that be helpful for your problem?

Almost forgot about it. In 0.1.0 you are also able to use Strings to define singletons for headers, queries and so on. Therefore, you are able to use dashes.

I might have used a wrong word, what i personally need is ability to extract all headers starting with the given prefix from request(and provide any number of that headers when calling an API). But this is still a bit of a long-shot goal, and  not necessarily a must. I would really like to know how to use this string syntax, i assume it is not yet provided with 0.1.0-RC5?

Okay, now I get a clearer picture. On the server-side, it should like something like this:
val Api = := :> "header" :> Family("expected-header-prefix") :> Get[Json, A]

derive(Api).from(headers: Set[String] => ???)

And on the client-side you would do:
val get = derive(Api)

get(Set(???))

Or do you mean something different?
The project state in 0.1.0-release isn't released yet. I will publish it at the end of the week I hope. Till then you can check out the branch and work with it locally.

#30

Btw. 0.1.0 is released and includes the matching (family) headers.

For me the current state is satisfactory, so i think this issue can be closed

👍


Allow ANSI (color) escape sequences in RTT output

Is your feature request related to a problem? Please describe.
The RTT UI currently strips any ESC/0x1B characters from the text output, making colored output look quite horrible:

Describe the solution you'd like
It would be great if the RTT shell displayed ANSI escape codes as-is. probe-rs/probe-rs#1387 would probably allow this naturally.

Since #2323 cargo embed strips the ANSI sequences from its output. I know this isn't the solution you are looking for, but getting sequences right with line wrapping is pretty tricky.


[HUDI-6719]Fix data inconsistency issues caused by concurrent clustering and delete partition.

Change Logs
Implemented a solution to prevent duplicate key errors in concurrent replace commit operations.
Registered the replace file ID information to the timeline for each replace commit, whether it is submitted or not.
Saved the pending replace commit's partitionToReplaceFileIds information to the inflight commit's extra metadata.
Updated drop partition and clustering operations to read the partitionToReplaceFileIds information in the timeline to ensure no duplicate file IDs.
Removed the modification involved in the related issue for compaction commit.
Impact
No public API or user-facing feature changes.
Risk level (write none, low medium or high below)
low
Documentation Update
Related issue: https://issues.apache.org/jira/browse/HUDI-5553
The specific problem is that when concurrent replace commit operations are executed, two replace commits may point to the same file ID, resulting in a duplicate key error. The existing issue solved the problem of scheduling delete partition while there are pending clustering or compaction operations, which will be prevented in this case. However, this solution is not perfect and may still cause data inconsistency if a clustering plan is scheduled before the delete partition is committed. Because validation is one-way.In this case, both replace commits will still contain duplicate keys, and the table will become inconsistent when both plans are committed. This is very fatal, and there are other similar scenarios that may bypass the validation of the existing issue. Moreover, the existing issue is at the partition level and is not precise enough.
Here is my solution:

As shown in the figure, both drop partition and clustering will go through a period of time that is not registered to the timeline, which is the scenario that the previous issue did not solve. Here, I register the replace file IDs involved in each replace commit to the active timeline (the replace commit timeline that has been submitted has saved partitionToReplaceFileIds, and only pending requests need to be processed). Since in the case of Spark SQL, delete partition creates a requested commit in advance during write, which is inconvenient to handle, I save the pending replace commit's partitionToReplaceFileIds information to the inflight commit's extra metadata. Therefore, each time drop partition or clustering is executed, it only needs to read the partitionToReplaceFileIds information in the timeline after ensuring that the inflight commit information has been saved to the timeline to ensure that there are no duplicate file IDs and prevent this kind of error from occurring.
In simple terms, each replace commit will register the replace file ID information to the timeline whether it is submitted or not, at the same time, each submission will check this information to ensure that it will not be repeated, so that any replace commit containing this file ID will be prevented, ensuring that there are no duplicate keys.
When this idea is also implemented on the compaction commit, the modification involved in the related issue can be removed.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure

TODO： Implement additional validations for insert overwrite, compaction, and these operations on Flink.

@boneanxs @TengHuo @voonhous  have any other comments or questions for this issue?

@bvaradar Your suggestion is reasonable, as it allows detecting most conflict situations at a lower cost.
However, there may still be some issues with this PR. If we check for conflicts before generating the requested files, it won't completely avoid situations where two processes simultaneously reach this point without being aware of each other. On the other hand, if we check for conflicts after generating the requested files, and let the timeline act as a marker, even if we detect conflicts and terminate the final execution process, there will still be failed requested files left to handle, which might not be ideal.
What is your perspective on this issue?

@majian1998 : The conflict resolution happens within the context of LockManager transaction which should serialize the checking. From what I understand, the conflict resolution strategy needs to handle replace commits. We need to incorporate the checking similar to DeletePartitionUtils.checkForPendingTableServiceActions as part of conflict resolution. Currently, DeletePartitionUtils.checkForPendingTableServiceActions is being called outside of transaction scope causing this. Let me know if this makes sense.


Add cli

Have you considered adding a cli for this? I can imagine this being popular as a replacement for rm. If you add a cli for it, you could even provide users with an easy alias rm="wipe" tip in the readme file.

That sounds like a great idea! I have to definitely look into that. That was the initial idea of this module, but it got lost somewhere when I was developing it.
Would it make more sense to separate it to it's own standalone module though? Like 'file-wipe-cli'? Would like to hear your opinion on this.

@simonlovesyou So the whole cli vs non-cli separation is super opinionated. I could care less, but some people don't like the idea of a node module that is packaged with the cli if they never use the global install.
To be the safest split them up into separate repos where the cli does nothing but argument checking, a --help command, and forwards it on to your module.

Actually, I'm not vibing on the code right now, count me out. I'm interested in the space of secure file deletion, but not this method.  Thanks for the merge though on the testing work 👍

What's not vibing with you? I don't want my code to detest other potential contributors in the future, or you for that matter, so a rewrite might be in order.

It's not very modular. So for instance look into doing this with glob.  But it's not setup to be repeatable right now. Does that make sense?


chore: refactor _convert_to_standard_langcode

This PR is one in a series of PRs for refactoring and fixing the languages parameter so it can address incorrect input by users. #2293
This PR adds a dictionary for helping map fully spelled out languages to tesseract language codes

I'm not actually seeing a difference in output between main and this branch - I'm running this code in a temp shreya_test.py, switching out the languages to be "english", "spanish" and "angry" (just as a random non language word)
from unstructured.partition.auto import partition

elements = partition(filename="example-docs/language-docs/eng_spa_mult.doc", languages=["angry"])
print([el.metadata.languages for el in elements])

and am getting
(unst10) shreyanid@Shreyas-MacBook-Pro-2 unstructured % python shreya_test.py
[['eng'], ['eng'], ['eng'], ['eng'], ['eng']]
(unst10) shreyanid@Shreyas-MacBook-Pro-2 unstructured % python shreya_test.py
[['spa'], ['spa'], ['spa'], ['spa'], ['spa']]
(unst10) shreyanid@Shreyas-MacBook-Pro-2 unstructured % python shreya_test.py
[['ang'], ['ang'], ['ang'], ['ang'], ['ang']]

This behavior is the same on both main and this branch. Is this full language name already supported? Could you provide test instructions if there is a different behavior to be testing for this PR?

I'm not actually seeing a difference in output between main and this branch - I'm running this code in a temp shreya_test.py, switching out the languages to be "english", "spanish" and "angry" (just as a random non language word)
from unstructured.partition.auto import partition

elements = partition(filename="example-docs/language-docs/eng_spa_mult.doc", languages=["angry"])
print([el.metadata.languages for el in elements])

and am getting
(unst10) shreyanid@Shreyas-MacBook-Pro-2 unstructured % python shreya_test.py
[['eng'], ['eng'], ['eng'], ['eng'], ['eng']]
(unst10) shreyanid@Shreyas-MacBook-Pro-2 unstructured % python shreya_test.py
[['spa'], ['spa'], ['spa'], ['spa'], ['spa']]
(unst10) shreyanid@Shreyas-MacBook-Pro-2 unstructured % python shreya_test.py
[['ang'], ['ang'], ['ang'], ['ang'], ['ang']]

This behavior is the same on both main and this branch. Is this full language name already supported? Could you provide test instructions if there is a different behavior to be testing for this PR?

This is a refactor with a minor update for mapping languages that are found in the dictionary that's added. It's not intended to change the functionality in a significant way. I did add one test showing it correctly maps "spanish" to "spa"

Ok, thanks for the clarification! Yeah, definitely seeing how the index slice of the input string results in "angry" being reduced to "ang" and interpreted as a language heh, perhaps that can be a future quick fix.

Great.
I don't think we do. This is just to catch common mistakes by users. In a later PR, I'll be adding more warning logs


new feat: Jump to source

❯ ruff format .  --line-length 120
error: Failed to parse graph/node.py:26:5: Expected 'Indent', but got 'llm'
2 files reformatted, 11 files left unchanged

click  "graph/node.py:26:5" go to source,  use can fix it instead  find file and location line no.

I think this would be a feature of your terminal rather than anything that Ruff could do. Like, in my terminal, I can right-click on graph/node.py and it will open the file in my default editor.
Do you know of similar tools that expose this, where you're seeing different behavior than for Ruff?

Currently, there is really not enough context to look if there is something that can be improved.
Lets get this into a reproducible direction:

What directory are you in?
What terminal emulators are you using? What happens in other terminal emulators?
What is the minimal, reproducible output?

Closing for now as it doesn't feel actionable.


dlib is using GPU but the cnn model is still taking too much, something is off somewhere on my setup.

face_recognition version: 1.2.3
Python version: 3.6.9
Operating System:
Ubuntu 18.04.4 LTS
Kernel Version: 4.9.140-tegra
CUDA 10.2.89

Description
Hi.
I've been using the face_recognition library for some time under jetson nano Jetpack 4.3, having a performance of around 500 ms per frame on a 1280 x 720 image using the cnn model, with CUDA support on dlib, and everything working great.
Last night I decided to try Jetpack 4.4 and everything went fine until I saw the performance of the running process. It was around 2000 ms per frame, with the very same setup as before.
The first thing I suspected was  that dlib, for some reason, may have not been compiled with CUDA support, but no, that was not the problem, as you can see below.
>>> import face_recognition
>>> face_recognition.__version__
'1.2.3'
>>> import dlib
>>> dlib.DLIB_USE_CUDA
True
>>> dlib.cuda.get_num_devices()
1
>>> dlib.__version__
'19.19.0'
>>>

Not only that, using jtop I can verify that when running the model GPU usage jumps to almost 100% instantly, meaning that the GPU is actually being used. However, the time it takes to process every frame is around 2 full seconds, a lot compared to the 500 ms I was getting just yesterday when running on Jetpack 4.3.
I've run out of ideas on where to look for the problem.
Any ideas?
Thank you.

You can use the model as 'hog'. It might be faster but I am not entirely sure.

Hi, no, you missed the point.
I'm not looking for alternatives to the cnn detector, I'm just trying to
figure out why would the face_recognition lib, which under the hood uses
dlib (which was compiled for CUDA, which is present, enabled and even being
used according to GPU monitoring tool jtop when running my app), would take
2 full seconds when running in a jetpack 4.4, when on jetpack 4.3 it would
take 500ms for the same input, and as far as I know, the same setup.
Since I posted the original question I went back to jetpack 4.3, and as I
was saying, throughput is exactly as expected, 500ms per frame. Same input
image, same application, same configuration.
I would have expected not to see GPU activity on 4.4 (that would explain
why it takes longer), but in fact there is GPU activity (100%), and it's
still taking 4x the time it takes on 4.3.
I'm just dazzled and trying to find an explanation for it.
On Sat, 2 May 2020 at 01:38, Udit Bansal<EMAIL_ADDRESS>wrote:

You can use the model as 'hog'. It might be faster but I am not entirely
sure.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ageitgey/face_recognition/issues/1130#issuecomment-622667790,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACK2RE53WXIGZIH22KAIYTLRPOPVDANCNFSM4MXFIVDQ
.

Same issue for me

Hi, @mariusmotea, did you have any luck identifying the cause of the increase in the processing times?

I just follow the tutorial from here. Albeit the tutorial is not that old, it is recommending  JetPack 4.2.


Upgrade Guide suggests incorrect type for css function

Description
The Upgrade Guide section of the documentation (https://lit.dev/docs/releases/upgrade/#litelement) notes the change in Lit 2.0 of the css function return type to CSSResultGroup to make it easier to override styles in subclasses.

The type of the css function has been changed to CSSResultGroup and is now the same as LitElement.styles. This avoids the need to cast the styles class field to any when a subclass sets styles to an Array and its super class set a single value (or visa versa).

However, the actual return type of css is CSSResult. Thus, in Typescript, the type of a styles property in the superclass must still be cast or wrapped in an array, if a subclass intends to override styles, adding to the superclass styles.
Steps to Reproduce
Given two LitElement classes, one inheriting from the other:
class MyElement extends LitElement {
    public static override styles = css`
        p {color: red;}
    ';
}

class AnotherElement extends MyElement {
    public static override styles = [
        super.styles,
        css`p {background: #222;}`
    ];
}

This produces the TypeScript error on AnotherElement:

Class static side 'typeof AnotherElement' incorrectly extends base class static side 'typeof MyElement'.
Types of property 'styles' are incompatible.
Type 'CSSResult[]' is missing the following properties from type 'CSSResult': ['_$cssResult$'], cssText, styleSheetts(2417)

To resolve this, the superclass must either wrap the css template literal in an array or assert the type as CSSResultGroup, which is specifically what the documentation says is no longer necessary. If using the array version, the super.styles reference must be spread into the subclass's array to avoid a similar TypeScript error.
Expected Results
LitElement classes should not have to proactively allow for possible subclasses to add style rules. Thus, css should return a CSSResultGroup as suggested in the Upgrade Guide.
Otherwise the documentation should reflect that this cast is still necessary.
Actual Results
A LitElement class must still cast the result of the css template literal expression for sublasses to be able to add additional styles.

static get styles():CSSResult { return css
also run first in header of index
      <!-- Force Polyfills -->
      <script
        async
        Set-Cookie: polyr=poly; SameSite=Lax>
          // Force Polyfills
          if (window.customElements) window.customElements.forcePolyfill = true;
          ShadyDOM = { force: true };
          ShadyCSS = { shimcssproperties: true};
      </script>

Has the guide been updated yet? I'm following the same code in the guide and getting the same kind of error.

In https://github.com/lit/lit.dev/pull/ the line saying that css returns a CSSResultGroup was removed, so that was fixed.
And instructions were added to https://lit.dev/docs/components/styles/#inheriting-styles-from-a-superclass to show how to write a base class where styles can be chained.


MM-T625 Incoming webhook is image only

Summary
Incoming webhook is image only

Ticket Link
MM-T625
mattermost/mattermost-server#15120

Done.


Bluetooth: application notification when MTU updated

Is your enhancement proposal related to a problem? Please describe.
The Bluetooth ATT MTU controls the maximum size of write, read_rsp, notification and indication packets.
Attempting to send more data than the current MTU results in warnings/errors such as:
[00:01:30.406,005] <wrn> bt_att: No ATT channel for MTU 48
[00:01:30.406,005] <wrn> bt_gatt: No buffer available to send indication

The MTU update process is as follows:

The client issues a request for a new MTU value
The server responds with the minimum of the requested MTU and the servers maximum MTU (CONFIG_BT_L2CAP_TX_MTU/CONFIG_BT_L2CAP_RX_MTU)

Therefore the amount of data that the server can transmit can vary over time, even on the same connection.
Currently the connection MTU can only be queried (bt_conn_get_mtu), with no option for the application to be notified when the value changes.
The current options are therefore to query the MTU before each transmit, or to re-query the MTU if a transmission fails due to insufficient space. Neither of these options are great for proactively constructing packets to send based on an expected data size.
Describe the solution you'd like
A callback added to struct bt_conn_cb that notifies the application when the MTU changes.

Having a callback doesn't fix this problem, in fact it would likely result in even more code because it can still cause the above errors while the callback notification is pending. Also the spec is pretty clear regarding not to attempt to exchange the MTU multiple times so it is normally expected the client do the exchange MTU in the very beginning to avoid this sort of problem, or in case of custom protocol on top use L2CAP COC instead of using a GATT attribute as PSM, which is likely much faster and does support segmentation.

4.3.1 Exchange MTU
This sub-procedure is used by the client to set the ATT_MTU to the maximum
possible value that can be supported by both devices when the client supports
a value greater than the default ATT_MTU for the Attribute protocol. This sub-
procedure **shall** only be initiated once during a connection.


RN64.rc3 Xcode build fails when ios hermes enabled and lottie-react-native installed

Description
build error:
Could not build Objective-C module 'React'
#if canImport(React)
import React
#endif

React Native version:
Run react-native info in your terminal and copy the results here.
System:
    OS: macOS 10.15.7
    CPU: (4) x64 Intel(R) Core(TM) i5-4570S CPU @ 2.90GHz
    Memory: 429.56 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 14.15.4 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.10 - /usr/local/bin/npm

    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.1 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.3, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
    Android SDK: Not Found
  IDEs:
    Android Studio: 4.1 AI-201.87<IP_ADDRESS>42882
    Xcode: 12.3/12C33 - /usr/bin/xcodebuild
  Languages:
    Java: 15.0.2 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: ^17.0.1 => 17.0.1
    react-native: ^0.64.0-rc.3 => 0.64.0-rc.3
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

enable ios hermes: :hermes_enabled => true

yarn add lottie-react-native lottie-ios
cd ios && pod install

Open xcode and run project

Expected Results
build successful

solved by installing from master branch yarn add https://github.com/lottie-react-native/lottie-react-native
or checkout https://github.com/lottie-react-native/lottie-react-native/issues/724

Had this issue on 4.0.2... disabling Hermes works


Adding preview CAE workbook

PR Checklist

[x] Explain your changes, so people looking at the PR know what and why, the code changes are the how.
This PR focuses on adding a Workbook for Continuous Access Evaluation feature, whereby admins can view information regarding the users login IP and whether the IP addresses match between IDP and Resource. This workbook will be enhanced in due course of time. The initial query only will supply basic information.

The workbook is currently hidden behind the preview flag
If adding or updating templates:

[ ] post a screenshot of templates and/or gallery changes
[x] ensure all steps have meaningful names
[X] ensure all parameters and grid columns have display names set so they can be localized
[X] ensure that parameters id values are unique or they will fail PR validation (parameter ids are used for localization)
[X] ensure that steps names are unique or they will fail PR validation (step names are used for localization)
[X] grep /subscription/ and ensure that your parameters don't have any hardcoded resourceIds or they will fail PR validation
[X] remove fallbackResourceIds and fromTemplateId fields from your template workbook or they will fail PR validation

@KranthiKiranerusu , @microsoft/azure-ad-workbooks : Requesting review on this PR.


“Save” button and "Revert" button

Dbeaver version:7.0.1
Database:pg or cockroach
Description: I create a new column,and then  I delete it immediately without pressing the "SAVE" or "REVERT" button. I found that the "SAVE" and "REVERT" button are still clickable.
However,when click "SAVE" button,nothing happened.
when click "REVERT" button, the columns I have deleted come back,but I can't operate any more.
As is shown below:

This phenomenon is ubiquitous.I create a new column,and try to modify it without saving.Now,I try to delete it  directly，when click the "SAVE" button,nothing happened.But "REVERT" button is useable.As is shown below:
create:

modify and then delete

revert

I think there must be something wrong  when merging the "modify" and "delete".

thanks for the bug report

It is not a bug. Changes merge has to be improved to support such "mutually collapsing" changes.
Unfortunately it is not that easy to do. Moving forward.

Fixed

verified


关于VMamba中Segmentation的CSM模块

请问为什么classification文件中有csm模块，但segmentation中却没有csm模块了

Hi @ParkMinyoung0304,
The model for segmentation is inited here, and if you trace further, you can find that the Backbone_VSSM is defined here in the classification/models/vmamba.py and as the Backbone_VSSM is a subclass of VSSM the  csm module is defined in that class.


does not have a member named

I created a list of type when I get his and screening, tip me the news. How is this to return a responsibility excuse me

Hi @wodetian, you need to manually create these properties or use mogenerator. I've updated the README to mention this a bit clearer.


Failures in terragrunt render-json

terragrunt render-json fails when run against certain configurations, e.g. the for-production/infrastructure-live folder of the terraform-aws-service-catalog. The error seems to be related to the find_in_parent_folders() function and is a variation of the following:
Call to function "find_in_parent_folders" failed: ParentFileNotFound: Could not find a account.hcl in any of the parent folders of /Users/ana/code/terraform-aws-service-catalog/examples/for-production/infrastructure-live/terragrunt.hcl. Cause: Traversed all the way to the root.

Files account.hcl are located in child directories from infrastructure-live and because of this it is failing on find_in_parent_folders():
terraform-aws-service-catalog $ find . -name "account.hcl"
./examples/for-production/infrastructure-live/security/account.hcl
./examples/for-production/infrastructure-live/logs/account.hcl
./examples/for-production/infrastructure-live/shared/account.hcl
./examples/for-production/infrastructure-live/stage/account.hcl
./examples/for-production/infrastructure-live/dev/account.hcl
./examples/for-production/infrastructure-live/prod/account.hcl


binaural audios received from facing forward and backward are different

Hi  Changan, @ChanganVR
I found that the binaural audios received from facing forward [L1, R1] and facing backward [L2, R2] are different. I meant the L1 and R2 are different. Arn't they should be the same? May I ask what's the possible reason for resulting these difference? Possibly the difference came from when generating the RIR or from generating binaural audios by ambisonic audio?
Thanks!
Best
Lingyu

Hi @ly-zhu,
Yes, this is expected because of binaural audio is different from simple two-channel stereo audio due to the ear shapes and canals.
Changan


[vcpkg] Could not locate a complete toolset.

OS windows

visual studio version: visual surdio 2015

When I want to install a package, the error occured "Could not locate a complete toolset".
Some people suggested to install the English language package for vs, but this is useless for me.

nothing is clear, but very interesting
can you describe in more detail?

@yHsi Thanks for reporting this issue!
Could you please provide the failure logs?
Thanks.

I also have this problem when I use VS2015update3 with English-language-pack on Windows10(1909).
So, How can I find failure log of vcpkg?

@bio-punk please provide the message printed on console.

@bio-punk please provide the message printed on console.
ok, here is my Console when I try to install opencv3

PS C:\dev> chcp 65001                                                                                Active code page: 65001
PS C:\dev> .\vcpkg\vcpkg install opencv3                                                                                Computing installation plan...
The following packages will be built and installed:
  * libjpeg-turbo[core]:x86-windows
  * liblzma[core]:x86-windows
  * libpng[core]:x86-windows
  * libwebp[core,nearlossless,simd,unicode]:x86-windows
    opencv3[core,dnn,flann,jpeg,opengl,png,tiff,webp]:x86-windows
  * opengl[core]:x86-windows
  * protobuf[core]:x86-windows
  * tiff[core]:x86-windows
  * zlib[core]:x86-windows
Additional packages (*) will be modified to complete this operation.
Could not locate a complete toolset.
The following paths were examined:
PS C:\dev> .\vcpkg\vcpkg install opencv3 --debug                                                                                [DEBUG] Feature flag 'binarycaching' unset
[DEBUG] Feature flag 'manifests' unset
[DEBUG] Feature flag 'compilertracking' unset
[DEBUG] Using vcpkg-root: C:\dev\vcpkg
[DEBUG] Using installed-root: C:\dev\vcpkg\installed
[DEBUG] Using buildtrees-root: C:\dev\vcpkg\buildtrees
[DEBUG] Using downloads-root: C:\dev\vcpkg\downloads
[DEBUG] Using packages-root: C:\dev\vcpkg\packages
[DEBUG] Using scripts-root: C:\dev\vcpkg\scripts
[DEBUG] Default binary cache path is: C:\Users\wzk\AppData\Local\vcpkg\archives
Computing installation plan...
[DEBUG] Found path: D:\ProgramData\Miniconda3\Scripts\cmake.exe
[DEBUG] Found path: D:\Program Files\Python36\Scripts\\cmake.exe
[DEBUG] Found path: D:\Program Files\CMake\bin\cmake.exe
[DEBUG] CreateProcessW("D:\ProgramData\Miniconda3\Scripts\cmake.exe" --version)
[DEBUG] cmd_execute_and_stream_data() returned 0 after    88017 us
[DEBUG] CreateProcessW("D:\ProgramData\Miniconda3\Scripts\cmake.exe" -DVCPKG_ROOT_DIR=C:/dev/vcpkg -DPACKAGES_DIR=C:/dev/vcpkg/packages -DBUILDTREES_DIR=C:/dev/vcpkg/buildtrees -D_VCPKG_INSTALLED_DIR=C:/dev/vcpkg/installed -DDOWNLOADS=C:/dev/vcpkg/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P "C:\dev\vcpkg\buildtrees\0.vcpkg_dep_info.cmake")
[DEBUG] cmd_execute_and_stream_data() returned 0 after    81698 us
[DEBUG] CreateProcessW("D:\ProgramData\Miniconda3\Scripts\cmake.exe" -DVCPKG_ROOT_DIR=C:/dev/vcpkg -DPACKAGES_DIR=C:/dev/vcpkg/packages -DBUILDTREES_DIR=C:/dev/vcpkg/buildtrees -D_VCPKG_INSTALLED_DIR=C:/dev/vcpkg/installed -DDOWNLOADS=C:/dev/vcpkg/downloads -DVCPKG_MANIFEST_INSTALL=OFF -P "C:\dev\vcpkg\buildtrees\0.vcpkg_tags.cmake")
[DEBUG] cmd_execute_and_stream_data() returned 0 after    84719 us
The following packages will be built and installed:
  * libjpeg-turbo[core]:x86-windows
  * liblzma[core]:x86-windows
  * libpng[core]:x86-windows
  * libwebp[core,nearlossless,simd,unicode]:x86-windows
    opencv3[core,dnn,flann,jpeg,opengl,png,tiff,webp]:x86-windows
  * opengl[core]:x86-windows
  * protobuf[core]:x86-windows
  * tiff[core]:x86-windows
  * zlib[core]:x86-windows
Additional packages (*) will be modified to complete this operation.
Could not locate a complete toolset.
The following paths were examined:
[DEBUG] ..\..\src\vcpkg\visualstudio.cpp(367)
[DEBUG] Exiting after 270262 us (264373 us)
PS C:\dev>

@bio-punk Does your Visual Studio installed in the default path? And please make sure you installed the English Language Package.

@bio-punk Does your Visual Studio installed in the default path? And please make sure you installed the English Language Package.
No，VS install in D:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE

@bio-punk Does your Visual Studio installed in the default path? And please make sure you installed the English Language Package.
No，VS install in D:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE
Does VS must install in C:\？

@bio-punk
https://github.com/microsoft/vcpkg/blob/ea82e4650890001b6a3177781e0c6096c9e1ca54/toolsrc/src/vcpkg/visualstudio.cpp#L84

I have the same error:
Could not locate a complete toolset.
The following paths were examined:
    f:\gv\VC_2019\Common7\VC\Auxiliary\Build\vcvarsall.bat
[DEBUG] vcpkg/visualstudio.cpp(367)
[DEBUG] Exiting after 276643 us (276478 us)

And I have the latest MSVC-2019. My vcvarsall.bat is in fact here:
f:\gv\vc_2019\VC\Auxiliary\Build\vcvarsall.bat
Why/how does vcpkg look into a path that doesn't exist?

@gvanem That looks like a completely different error since you're talking about a completely different VS. We got the root path f:\gv\VC_2019\Common7 as the output of vswhere.

To help those of us who have problems detecting VS, can you add --debug statements to (I think) https://github.com/microsoft/vcpkg/blob/2020.11/toolsrc/src/vcpkg/visualstudio.cpp#L191-L345 ? Or at least tell us how to do that?
I think it would cut down on the tickets if we had a --debug statement everytime the visualstudio.cpp code rejected a VS instance (or didn't find vswhere.exe, etc).

@yHsi, sorry for reply you so late, does this issue persist in the latest vcpkg?

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


[Bug] Module not found 'six'

Im using Ollama which works sometimes but randomly throws:
For example when I enter sgpt "whats the regex command to print all filenames that start with the word nvim and contain the string lua?"
stack.log

@Ajaymamtora
run this command
pip uninstall six
pip install six
I hope this resolves your issue!


Package size could be reduced

Hi,
The size of the package installed on the disk could be reduced by around 40%.
Specifically, the folder "coverage" takes 423 Kb out of 1.02 Mb. If I understand correctly, it contains results of code test coverage that are not necessary for the end user.
Is there a way to install the package without this folder?
Thanks in advance,
Mark

Hi,
Here I have my main bundle (loaded on every page), the size taken by v-tooltip is a bit heavy.

Can we reopen this feature ?
Have a nice day

Any news on this?

I found that importing a single component instead of all of v-tootip still produces the same bundle size. Normally I think this should reduce the size of my bundle right?
import VTooltip from 'v-tooltip' // 56K

import { VPopover } from 'v-tooltip' // still 56K

@imacrayon should be fixed in v3 alpha

@gertjanwytynck check the update above
@Akryum thank you for the update and all your great work!


Node.js  Loaders Team Meeting 2024-06-04

Time
UTC Tue 04-Jun-2024 18:00 (06:00 PM):

Timezone
Date/Time

US / Pacific
Tue 04-Jun-2024 11:00 (11:00 AM)

US / Mountain
Tue 04-Jun-2024 12:00 (12:00 PM)

US / Central
Tue 04-Jun-2024 13:00 (01:00 PM)

US / Eastern
Tue 04-Jun-2024 14:00 (02:00 PM)

EU / Western
Tue 04-Jun-2024 19:00 (07:00 PM)

EU / Central
Tue 04-Jun-2024 20:00 (08:00 PM)

EU / Eastern
Tue 04-Jun-2024 21:00 (09:00 PM)

Moscow
Tue 04-Jun-2024 21:00 (09:00 PM)

Chennai
Tue 04-Jun-2024 23:30 (11:30 PM)

Hangzhou
Wed 05-Jun-2024 02:00 (02:00 AM)

Tokyo
Wed 05-Jun-2024 03:00 (03:00 AM)

Sydney
Wed 05-Jun-2024 04:00 (04:00 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Loaders Team+Meeting+2024-06-04&iso=20240604T1800
or https://www.wolframalpha.com/input/?i=06PM+UTC%2C+Jun+04%2C+2024+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1QyMYoI9iB9bVsDXskx5ZT3oAwd1H7r9ev4_jQ4kuYCU/edit

Agenda
Extracted from loaders-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
nodejs/node

Proposal for a simple, universal module loader hooks API to replace require() monkey-patching #52219
ESM Hook deadlocks since 21.2.0 a3e09b3 #50948

Invited

Loaders team: @nodejs/loaders

Observers/Guests
Notes
The agenda comes from issues labelled with loaders-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting

link for participants: https://zoom.us/j/97506450082

For those who just want to watch: https://www.youtube.com/c/nodejs+foundation/live

Should we meet? @joyeecheung @dygabo @mcollina @ShogunPanda ?

Yup, I'll try to attend.

I could attend as well

I am at an event so not sure if I can attend or not.


All variables should have descriptive names

Please rename things like df2 to something that describes what it contains

renamed to df_pigs and df_cattle


Listing log hangs fails if filter is invalid

Not sure what's going on here yet - will update with a repro when I have one.

I can't reproduce this - I've tried various invalid filters, and they all end up with an exception. Closing for now; we can reopen if we see it again.


luci-app-watchcat: add support for new features + refactor

Signed-off-by: Nicholas Smith<EMAIL_ADDRESS>
Add support for restarting an interface (including ModemManager interface support)
support for specifying ping packet size
Refactor/rewriting some of the user facing text (will require a re-translation)

Depends on: https://github.com/openwrt/packages/pull/14225

https://github.com/openwrt/packages/pull/14225 has been merged

New watchcat was mergerd so maybe we can also merge this changes ?

New watchcat was mergerd so maybe we can also merge this changes ?

maybe we can also merge this changes ?

At least not when there are conflicts. (apparently some translations have changed while this has been open)

Conflicting files
applications/luci-app-watchcat/po/zh_Hant/watchcat.po

maybe we can also merge this changes ?

At least not when there are conflicts. (apparently some translations have changed while this has been open)

Conflicting files
applications/luci-app-watchcat/po/zh_Hant/watchcat.po

maybe we can also merge this changes ?

At least not when there are conflicts. (apparently some translations have changed while this has been open)

Conflicting files
applications/luci-app-watchcat/po/zh_Hant/watchcat.po

Hey, I have fixed the conflict

maybe we can also merge this changes ?

At least not when there are conflicts. (apparently some translations have changed while this has been open)

Conflicting files
applications/luci-app-watchcat/po/zh_Hant/watchcat.po

Hey, I have fixed the conflict

Hey, I have rewritten it using the JavaScript API to get rid of the luci-compat dependency.

Hey, I have rewritten it using the JavaScript API to get rid of the luci-compat dependency.

From my side, this looks good. However, I still have one question. What happened to the other translations? Why are they deleted? Shouldn't they also be updated with the new content?

From my side, this looks good. However, I still have one question. What happened to the other translations? Why are they deleted? Shouldn't they also be updated with the new content?

From my side, this looks good. However, I still have one question. What happened to the other translations? Why are they deleted? Shouldn't they also be updated with the new content?

My mistake.  I have fixed that now

From my side, this looks good. However, I still have one question. What happened to the other translations? Why are they deleted? Shouldn't they also be updated with the new content?

My mistake.  I have fixed that now

I think we are still waiting for the P/R, then the reload will also work when the configuration is changed.

ping @feckert

Sorry if I'm being annoying, I think this is ready to merge.
But could you still stash the commits so we have one?

Sorry if I'm being annoying, I think this is ready to merge.
No, you're fine, I appreciate the review.
But could you still stash the commits so we have one?
Stash or squash?

Squashed

thanks merged

Hi, are you going to backport this changes to 21.02 ?

Hi, are you going to backport this changes to 21.02 ?

I will if that is allowed
ping @feckert

From my point of view, that's fine.
The openwrt-21.02 branch has not yet been released.
It is still in the stabilization phase.
I would suggest that one of you @nickberry17 @kofec  creates a pull request with the cherry-picked commits.
Please do not forget the package feed ;-) commits

suggestion
on the Mode: Restart Interface
to check if the wireless is in AP or STA mode,
So if we change the wireless interface from STA to AP and we forget to set the watchcat, the watchcat will automatically deactivate the interface restart mode.

@nickberry17 Can you please add these changes to 21.02 as you probably have environment ready  (from trunk)


throw exception when dependency parse when root is unreachable

We found a sentence in the agiga corpus that has a messed up dependency parse.
The issue is that there is a loop (a node pointing to itself) that makes the root unreachable.
This pull request introduces a DependencyUtilsException that is thrown when that happens.

I'll merge this because this is a valid test. But we (or I...) should modify the parser in processors to never allow this to happen.

this is not our parser. It is a sentence from the agiga corpus. I sent you an email with more details.
I can paste it here if you want to

Agreed. But they used CoreNLP to parse this, afaik. So it could happen again.


Update test_numbers for BigInteger

What current issue(s) does this address, or what feature is it adding?
This adds a test for BigInteger response to 0
This PR is in response to @ixje comment
This should help solve issue #47
How did you solve this problem?
Added the test for BigInteger response to 0
How did you make sure your solution works?
Tested it
Did you add any tests?
Yes, see above
Did you run make lint and make coverage?
Yes
Are there any special changes in the code that we should be aware of?
No

Coverage increased (+0.1%) to 96.632% when pulling d3e88090384738289cfacfab3acf0e36a23bc90e on jseagrave21:patch-2 into e7e463d8caf21720e244d33895d544b95905058d on CityOfZion:master.


[QUERY] Changes in local environment affecting deployed workflows

Question

Hello! I built an app following this excellent README and, after completing the development, I ran a slack deploy. Recently, I needed to modify the app and, to my surprise, the changes I made appeared in the workflows whose links were created during the slack deploy. Questions:

Is there any configuration I missed to prevent this from happening?
What exactly happens when slack deploy is executed? I carefully read the documentation for slack run and slack deploy (ref, ref) as well as other documentation and issues in this repository, but it wasn’t clear if/how I can make sure that local environment code does not affect the deployed code. Can you help me with this?

Context

There was a trigger link working normally in a Slack channel, created when I ran slack deploy (I am sure of this because there was no (local) next to the trigger link/workflow name). I needed to modify the app code, so I ran a slack run and chose the local environment (to ensure I was in the local environment, I generated another trigger link to test in a test channel, and it has (local) in the name). However, these changes were affecting the workflows in the channel with deployed trigger links. Observed behaviors:

Completed uses of workflows in the channel with deployed trigger links change from data-support-app to data-support-app (local) when I modify the workflow code, and the log App successfully reinstalled appears (I assumed this reinstalled was about the local environment).
Running slack deploy removes the (local) from all previous uses in the channel with deployed trigger links.
In the terminal where I chose to run slack run in the local environment, logs appear about the slack deploy I ran (with a cloud emoji):

☁  2024-05-09 10:10:50 [info] Application Updated by user 'Uxxx' on team 'Txxx'
File change detected: path/to/file.ts, reinstalling app...
App successfully reinstalled

Environment

"deno-slack-sdk/"<EMAIL_ADDRESS>"deno-slack-api/"<EMAIL_ADDRESS>

deno 1.42.3 (release, x86_64-unknown-linux-gnu)
v8 <IP_ADDRESS>
typescript 5.4.3

#29~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Thu Apr  4 14:39:20 UTC 2
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

Hi @mirianbatista ! Sorry about your confusing experience, let's try to get to the bottom of it.
Your understanding about the isolation of local (slack run) vs. deployed (slack deploy) apps is correct: they are two separate apps. The slack run app that you run locally has its own application ID which is different from the slack deploy app that Slack runs for you. So any triggers and code associated to each context is isolated from the other context.
Let me provide an overview of what happens when you run run vs. deploy.
slack run

An isolated instance of your app is either created or updated. This local version of your app should have a unique application ID (should look like A1234ABCD). This app ID should be surfaced in various places when you use the Slack CLI (like when you are selecting workspaces via the prompts). An application's configuration is almost entirely dictated by the contents of your manifest file. If you run the CLI in verbose mode (via the --verbose flag) you can see this API ID being passed to various Slack HTTP APIs. The creation/updating of an app configuration is mediated by the apps.manifest.* suite of Slack HTTP APIs.
The CLI's process will block your terminal as it establishes a connection with the Slack backend in order to receive events of things happening within the Slack workspace that are relevant to your app (i.e. messages being sent to channels your app is a part of, and so on).
As each such event is sent to your app, the CLI will pick it up and invoke the relevant deno function as applicable. For example, if your app uses a custom function as one of its steps in a workflow that is executed, the CLI will pass the relevant function-specific payload and deno run your custom function code locally.

slack deploy

Same thing here for slack deploy, except that the app ID here will be different than the app ID used for slack run.
The CLI will then bundle your code locally on your machine and zip it up. This gets sent to a Slack API responsible for managing deployed app bundles.
CLI process completes. At this point, as events relevant to your deployed app happen, they will get dispatched to the Slack infrastructure hosting your app bundle.

So, with all this said, I don't think that the following statement is possible:

Recently, I needed to modify the app and, to my surprise, the changes I made appeared in the workflows whose links were created during the slack deploy

Can you expand on "the changes you made"? What slack commands were run? What were the changes made? Did you change function code? Did you change trigger code? What was the order of operations in terms of code changes and slack CLI commands?

@filmaj thank you so much for the detailed explanation of slack run vs slack deploy, this was essential to identify that the slack.json app_id was the same as the slack.dev.json app_id; this was the cause of all the confusion. Thanks again for the clarifications!

Happy to help! Out of curiousity: how did you end up in the situation where the app ID was the same for both? That should basically never happen - want to make sure the tooling isn't somehow getting itself twisted up and causing this situation.

@filmaj At the beginning of the development of the app that is currently working in my company's Slack, I tested the workflow codes in another personal Slack. The app is a support request app that creates support tickets for users to my team and generates a page in Notion for the user to track the progress of their request. Since I didn’t have the permission to create the page in the company’s Notion, I used a personal Slack and Notion until I got the necessary permission to use the company’s Slack and Notion. Probably, when I was removing the information from my personal Slack, I accidentally copied the content of one JSON to another. I don’t remember exactly how it happened, but I suspect that’s what occurred.
On a related note, considering that these files are crucial for the proper separation of development and production environments, do you think we could include a more detailed explanation in the deno-issue-submission documentation (ref) and/or the Team collaboration page (ref) about how each one works? Perhaps adding the valuable detailed information you provided me in this issue about how slack run and slack deploy work? I ask because I am motivated to open a PR about this, but I’m unsure if this problem happened due to my inexperience or if it’s something we can genuinely improve.

Thanks a lot for the info! Super helpful as we look to improve the tooling and see what we can do to eliminate some of the sharp corners. A few follow-ups, if you don't mind?

I accidentally copied the content of one JSON to another.

Just to be clear: do you refer to the .slack/apps.*.json files (which contain app IDs that the CLI reads in order to target the appropriate app)?

do you think we could include a more detailed explanation / documentation

100% agreed, I will raise this with our api.slack.com tech writing team and we will brainstorm ways to make this clearer! I think your suggestion to clarify that in the samples' description of the .slack folder makes a lot of sense too. Will do!
Appreciate your candid feedback 🙇

Just to be clear: do you refer to the .slack/apps.*.json files (which contain app IDs that the CLI reads in order to target the appropriate app)?

This, exactly, somehow (obviously without understanding that those files were the configurations of the dev and prod environments and without being clear about the explanation you gave here in the issue) the app_id of the file .slack/apps.json and .slack/apps.dev.json were the same
Thank you in advance for the improvements in the documentation!!

For what it's worth, I have done the same thing in the past (run cp -r ./my-old-app/* ./my-new-app) and been bitten by this too, so you're not alone.


Avoid suggesting code actions when syntax error exists

Description:
$subject.

Related, https://github.com/ballerina-platform/ballerina-lang/issues/34762

Closed with #35156


Websocket Support broken

Issue Type
Feature Request/Bug
Description
WebSocket support. Request to find solution to fix LLAPI support or add a wrapper to something like websocket-sharp.
Expected Behavior
Expect to be able to add and use LLAPI websocket transport to an MLAPI host, but (possibly) due to issues in LLAPI itself, this doesn't work.
Actual Behavior
If using the a compiled MLAPI dll, you'll get a non-existent method exception when running your client. If including MLAPI source in your Unity project, it will not compile as WebGL.
Stacktrace
Exception: Error: called non-existent method System.Int32 UnityEngine.Networking.NetworkTransport::AddHost(UnityEngine.Networking.HostTopology) UnityLoader.js:4:9380
  at UnityEngine.Networking.NetworkTransport.AddHost (UnityEngine.Networking.HostTopology )

Steps to Reproduce
Build for WebGL and try to connect to server from client
OR
Include MLAPI source in Unity project and attempt to build for webGL.
Screenshot on This Problem (if possible)
{Please write here}
Your Environment

Unity Version: 2018.3.x
MLAPI Version (Commit # or release version):  3ee1b1c

Before any conclusions are drawn. Lets verify that this is a Unity bug and not a MLAPI one. So a quick NetworkTransport project should do the trick.

Confirmed that websockets still work in the Unity LLAPI transport by using the HLAPI:
-Created a new project in 2018.3.4f1.
-Added NetworkHUD to sample scene and build.
-Ran two instances of build, started one as server, one as client to confirm they connected.
-Changed networkmanager setting to "use websockets" and rebuilt.
-Ran two instances of build, started one as server once as client to confirm they DO NOT connect.
-Built again as webgl, ran and successfully connected to localhost server.

Okay, that means that there is a undocumented flow here that the HLAPI uses that we need to mimic...

Fixed in MLAPI v8


fix(csp): apply right csp header when status code is 304

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #8352 (0fab727) into dev (7278b10) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev    #8352   +/-   ##
=======================================
  Coverage   68.55%   68.55%
=======================================
  Files          91       91
  Lines        3876     3876
  Branches     1056     1056
=======================================
  Hits         2657     2657
  Misses        991      991
  Partials      228      228

Flag
Coverage Δ

unittests
68.55% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

packages/server/src/middleware/nuxt.js
97.53% <100.00%> (ø)

Continue to review full report at Codecov.

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

fix


Create student.txt

Bump?

@bondpat17 Thank you for your patience while waiting for a review and processing your request. Pull request merged.

Thank you for your response!
On Wed, Jul 3, 2024, 10:53 PM Philip Torchinsky @.***>
wrote:

@bondpat17 https://github.com/bondpat17 Thank you for your patience
while waiting for a review and processing your request. Pull request merged.
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/21281#issuecomment-2208169254,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJSMCLFH5TUXIO2J3TXIFSDZKTPOPAVCNFSM6AAAAABKGDQHM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBYGE3DSMRVGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


support git repo http.sslverify false

Summary
What change needs making?
Use Cases
i have a gitlab repo with self-signed certificate
``
git config --global http.sslverify false
git clone https://gitlab.example.com/root/test.git

i want git clone use arog-workflows,  and config http.sslverify false, how can i do that?

---
<!-- Issue Author: Don't delete this message to encourage other users to support your issue! -->
**Message from the maintainers**:

Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

@willzhang
I think you need to check your git version or config.
If it does not work, I'd like to know more detail in your case.

Seems like a duplicate of #10762


Support setting compute for init containers

Changes
This commit introduces a flag "enable-init-container-resources" that will
allow a cluster operator to choose whether Tekton should apply compute resources
to its init containers. If this flag is turned on, init container resources will
be set to the maximum values of any of the other containers in the pod. This
provides the init containers with the maximum compute resources possible
without changing the effective resource requirements of the pod.
If LimitRanges are present as well, default values from LimitRanges will be applied
to initContainers only if no Steps or Sidecars have compute resources set.
One potential drawback of this solution is that if Step or Sidecar resource requirements
are set too low, init containers may be OOMkilled or throttled. However, Steps and
Sidecars typically need more compute resources than Tekton init containers do, and this
can be addressed by the user increasing the compute resources of their Steps or Sidecars.
An alternative solution is to provide an installation-level configMap that sets
compute resource requirements for Tekton init containers. (This is the solution pursued in #4552.)
This solution doesn't require anyone to right-size the init containers, so it's easier to maintain.
Setting resource requirements for init containers allows TaskRun pods to be deployed
in namespaces with ResourceQuotas, and provides more flexibility for users who would
like to configure quality of service for Tekton pods without having to use LimitRanges.
However, this doesn't address the problem that ResourceQuotas will consider the
resource usage of the pod to be the sum of the compute resources of its containers
(even though Step containers run sequentially) (#4976).
Closes #2933.
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in
(if there are no user facing changes, use release note "NONE")

Release Notes
Introduce feature flag to set compute resources for Tekton init containers

test failures are due to flakes-- not going to spend time re-running them, for now, just want to get feedback on this solution

/retest

@vdemeester @dibyom @skaegi mind taking a look? this has been open for a bit

Thanks @lbernick The code looks fine to me and we can experiment and see if this is an option we should enable by default.

not actively working on this, closing for now


fix: use new sdi 0.3.6

Fix for broker requirements in dex on track/2.31
Details are in: https://github.com/canonical/dex-auth-operator/issues/136
Summary of changes:

Pinned SDI to 0.3.6 that has a fix. Solution is described in https://github.com/canonical/bundle-kubeflow/issues/648
Regenerayed requirements*
Unpinned controller version in workflow.

@DnPlas Please review. The lease amount of changes to make this work in track/2.31


MudSelect on MultiSelection prints the message Required even if a selection has been made.

Bug type
Component
Component name
MudSelect
What happened?
MudSelect on MultiSelection prints the message Required even if a selection has been made.
Expected behavior
When a selection is made, no "required" error message should appear.
Reproduction link
https://try.mudblazor.com/snippet/QYGmayYPVPCVwXze
Reproduction steps
1.Start and select one or more items.
2.
3.
...
Relevant log output
No response
Version (bug)
6.0.9
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

I took a look under the hood in MudSelect.razor.cs  looks like _value property is always null when doing validation so HasValue is returning false.
I made a little tweak(see below) and it works, but maybe I'm violating the intent of MultiSelect?
public async Task SelectOption(object obj)
        {
            var value = (T)obj;
            if (MultiSelection)
            {
                // multi-selection: menu stays open
                if (!_selectedValues.Contains(value))
                    _selectedValues.Add(value);
                else
                    _selectedValues.Remove(value);

                if (MultiSelectionTextFunc != null)
                {
                    await SetCustomizedTextAsync(string.Join(Delimiter, SelectedValues.Select(x => Converter.Set(x))),
                        selectedConvertedValues: SelectedValues.Select(x => Converter.Set(x)).ToList(),
                        multiSelectionTextFunc: MultiSelectionTextFunc);
                }
                else
                {
                    await SetTextAsync(string.Join(Delimiter, SelectedValues.Select(x => Converter.Set(x))), updateValue: false); //Try to make true(see below)
                }

                UpdateSelectAllChecked();
                _value = value; //<-Add:Setting value make the Required validation pass
                BeginValidate();
            }

I looked at SetTextAsync because the updateValue parameter being false seems a little suspect. I tried setting it to true, but it would never would because this:
        protected override Task UpdateValuePropertyAsync(bool updateText)
        {
            // For MultiSelection of non-string T's we don't update the Value!!!
            if (typeof(T) == typeof(string) || !MultiSelection)
                base.UpdateValuePropertyAsync(updateText);
            else if (MultiSelection)  //<-I added this
                base.UpdateValuePropertyAsync(updateText); //add
            return Task.CompletedTask;
        }

Again,  base.UpdateValuePropertyAsync leads you to SetValueAsync this:
        protected virtual async Task SetValueAsync(T value, bool updateText = true)
        {
            if (!EqualityComparer<T>.Default.Equals(Value, value))
            {
                Value = value;
                if (updateText)
                    await UpdateTextPropertyAsync(false);
                await ValueChanged.InvokeAsync(Value);
                BeginValidate();
            }
        }

But this will never work because Value is null going into it and Value is NEVER set with MultiSelect.(It looks by current design)

Should the Required logic for MultiSelect MudSelect be different? E.g. check bound list of selected items like SelectedItems.Count>0?
Happy to work on a PR, but wanted to get a consensus on how to proceed.

Setting the Value is probably not gonna work. As you have guessed it is intentionally not set for multi-selects. Also it would be a breaking change and would also break the test cases. In any case, it would only be a workaround and I guess we can do better than that.
I think the Required logic needs to be different for a multi-select. Also, like you already discovered, the FluentValidation support  via the For expression should also work differently for multi-select than for a normal select.
This means we need to change MudFormComponent to allow for polymorphism regarding the value which should be validated both for the Required logic and also the Validation funcs.
My idea would be to add an internal object GetValue() function which returns object and which is called in the required logic instead of checking _value. The select can then implement this function like this:
internal object GetValue() {
  if (Multiselection)
    return SelectedItems;
  else
    return _value;
}

And I when calling the validation funcs also GetValue() should be passed to the validation funcs instead of Value. This might cause a problem with the validation funcs which are typed T but we could document that those are not supported for multi-select.
Maybe I am overlooking something here but we'll see soon enough when you do the PR.

@henon
Question, why can't we make a 2nd validation function that works on multiple selections?
We also have the AutoComplete, but there will be the same problem.

@henon how about I just override the HasValue method and pass it in as an optional parameter into the Validate Method? This way it doesn't care about what T is and the component can handle that logic.

@henon how about I just override the HasValue method and pass it in as an optional parameter into the Validate Method? This way it doesn't care about what T is and the component can handle that logic.

Yes, this is a very good solution for the required-problem. The validation problem will have to be dealt with differently then.

@henon Just tested and this works in MudSelect
        protected override bool HasValue(T value)
        {
            if (MultiSelection)
                return SelectedValues?.Count() > 0;
            else
                return base.HasValue(value);
        }

I'll work on creating a test and will submit a PR.
Regarding Validation, Is there an existing issue for that already?
We could add something like  protected virtual object GetValue() => Value to MudBaseInput


Temperature data structure

Right now we're storing temperature data as columns named T5, T10, etc. The problem with this is datasets have completely variable depths, with some having really unique depths (e.g. T16.5). When everything is joined together into a single dataset, this means that T16.5 ends up NA everywhere except that dataset.
Not sure how much space this wastes–R is pretty efficient with data frames–but worth thinking about. Alternatively...could have a separate temperature data frame with fields CSR_DATASET, CSR_TIMESTAMP, CSR_DEPTH, CSR_TEMPERATURE.

Seems like too much work, given that we're keeping datasets in separate data tables.


Non-versioned endpoints not showing up in documentation

If a decorator has 'versions' set all endpoints without 'versions' set do not show up in the documentation. Is this an intended feature?

Hi @cy70sin3,
Thanks for reporting this! No It is not the intended behavior, I'll be working on releasing a fix for this issue soon.
~Timothy

This has been fixed in the just released on PYPI 1.9.6 version of hug.
Thanks!
~Timothy


refactor(module:input-number): refactor input number

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information

Coverage remained the same at 98.348% when pulling 0072c192a92c4572f5496da8f728e78d91ce6da9 on vthinkxie:0.7.0 into 56c2df1948bc51e61f560995b23a654397104162 on NG-ZORRO:0.7.0.


Fix add redirect weirdness

Potentially, this will require shipping source maps to production, which is not super awesome but might be incredibly useful in the short term.

Shipping source maps seems pretty desireable. It'll also richen the utility
of newrelic's JS stuff. We already publicly share the source code, so there
isn't an IP concern.
On Sat, Aug 31, 2019 at 8:59 AM Brenna Switzer<EMAIL_ADDRESS>wrote:

Potentially, this will require shipping source maps to production, which
is not super awesome but might be incredibly useful in the short term.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/switzerb/CookBook/issues/88?email_source=notifications&email_token=AAAUFXVAUPHMJ526FRXQFGDQHKIMPA5CNFSM4ISUBUC2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HIS7Z2A,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAUFXQFSK5KGJO3CGWOUJLQHKIMPANCNFSM4ISUBUCQ
.

A hA HAHA HA! NewRelic refuses to use published sourcemaps. So not that one. But sourcemaps are shipped to production, so  that part's good to go.

I'm not sure what the description and comments are referring to, but the weirdness around the draft add/edit for is spiffier now.

I'm not sure what the description and comments are referring to, but the weirdness around the draft add/edit for is spiffier now.


Black screen after complete.

After complete, this black screen was showed. Without any clicks or relateds, I went redirected automatically.

@felipecastrosales have you faced it multiple times?

@felipecastrosales have you faced it multiple times?

This was shown after I finished the game, and before that I died a few times and the game restarted.
I don't know exactly what could have caused this, but it was after I won.

Experienced the same on Web after I finished the first stage.


Regex rule on array

Versions

vee-validate: 2.1.0-beta.8
vue: 2.5.16

I am trying to validate a field having multiple tags. Every tag should be validated as per a particular regex. If I use a single value then it works fine but fails for multiple values i.e. array field.
I checked the code and found out that regex rule is not handling array field.  Is this by design?

Can I submit a PR for this?

Feel free to PR this and would be much appreciated, Thanks!


Add DECaPS2 to default pywwt surveys

I got a request to expose the DECaPS2 RGB survey to glue-wwt. The most straightforward way to do that is to add it to the default pywwt surveys, and then pywwt users can access it as well. The WTML tag here is taken verbatim from the default webclient surveys.

Closing and reopening to retrigger the CI — I think it was affected by a GitHub service incident yesterday.

@Carifio24 Oh, I see the problem — this PR was asking to merge into the gh-pages branch, not master, for some reason. I think I can change that myself.

Oops, I guess it was supposed to be like that, and GitHub won't let me reopen it. Can you please re-file? I'll try to fix the ReadTheDocs configuration to ignore this branch.

Hmm, as far as I can tell, there's no way to tell the ReadTheDocs integration to ignore PRs to unusual branches like gh-pages. Which is basically OK — we can just ignore the failure. It's just not as tidy as it could be.

@pkgw I've re-filed this as #374.


Generate three.js visual for loose lips logo

Currently on the home page we are using an animated gif as a visual cue for loose lips. This doesn't look great in the light mode of the website. It could be fun to recreate the gif using the three.js library.
docs: https://threlte.xyz/

I've made a repo last year containing the three.js interactive logo (also using tweenjs).
https://github.com/t-scale/looselips
code could use more factoring but tried keeping it functional anyway.
3D assets are in world.js


Add Kafka

As discussed in Add Kafka to TDP #27, we will add Apache Kafka to TDP. This also implies adding a new dedicated ZooKeeper service.
Targeted versions are:

Kafka 3.2
ZooKeeper 3.6.3

Kafka must be an option, for example if we add an Ansible group kafka_server and if there is no host inside this group, the Kafka playbook should do nothing without error. Same for the dedicated ZooKeeper.

As discussed in #340, downgrading targeted Kafka to 2.x and ZooKeeper version accordingly.

Closing this issue as Kafka is moved to tdp-collection-extras. See dicussion#27


Gdt 309 implement and booleantype

Why are these changes being introduced:
Usability testing showed that the OR default for multiword keyword
searches was resulting in many results that were not useful in many
situations, including some situations where many results were returned,
but none of the results was actually useful.
Relevant ticket(s):
https://mitlibraries.atlassian.net/browse/GDT-309
Related ticket(s):
https://mitlibraries.atlassian.net/browse/GDT-281
How does this address that need:

updates TIMDEX graphql schema to include API feature that enables this
functionality
updates our GraphQL queries to include new booleanType
defaults booleanType to AND. If a different option is set in a
user's session, that will take precedence
allows user to set booleanType via query parameter (hidden feature)
adds feature (default off) to allow staff to switch booleanType for
their session via a clunky UI element

Developer
Accessibility

[ ] ANDI or WAVE has been run in accordance to our guide.
[ ] This PR contains no changes to the view layer.
[ ] New issues flagged by ANDI or WAVE have been resolved.
[ ] New issues flagged by ANDI or WAVE have been ticketed (link in the Pull Request details above).
[ ] No new accessibility issues have been flagged.

New ENV

[x] All new ENV is documented in README.
[ ] All new ENV has been added to Heroku Pipeline, Staging and Prod.
[ ] ENV has not changed.

Approval beyond code review

[x] UXWS/stakeholder approval has been confirmed.
[ ] UXWS/stakeholder review will be completed retroactively.
[ ] UXWS/stakeholder review is not needed.

Additional context needed to review
E.g., if the PR includes updated dependencies and/or data
migration, or how to confirm the feature is working.
Code Reviewer
Code

[ ] I have confirmed that the code works as intended.
[ ] Any CodeClimate issues have been fixed or confirmed as
added technical debt.

Documentation

[ ] The commit message is clear and follows our guidelines
(not just this pull request message).
[ ] The documentation has been updated or is unnecessary.
[ ] New dependencies are appropriate or there were no changes.

Testing

[ ] There are appropriate tests covering any new functionality.
[ ] No additional test coverage is required.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

9 of 13 (69.23%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 98.748%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/controllers/static_controller.rb
1
5
20.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.7%

Covered Lines:
552

Relevant Lines:
559

💛  - Coveralls


Changed image-utils to use writeFileSync instead of piping to a str…

…eam.
Streams are async, and without returning the string,
a promise or using a callback, you cannot wait on
the stream response.
I'm using this in unit tests, and need to wait for
the image to be written before comparing it to
a snapshot image.
Should this need to be an async function, it can be
fixed differently.
@jscad/img-utils and @jscad/regl-renderer need to be published to NPM.
All Submissions:

[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 update/change?
[ ] Does your submission pass tests?

I think the tests will pass once these packages are published.

Note: please do NOT include build files (those generate by the build-xxx commands) with your PR,

Thank you for your help in advance, much appreciated !

Hi @johnwebbcole , thanks for the fix !
Two aspects of this:

the change to writeFileSync makes sense in this case, agreed !
the packages however are still in pre-alpha and are not supposed to be published to NPM quite yet:

@jscad/img-utils and @jscad/regl-renderer need to be published to NPM. did you mean this for the unit tests to pass on CI, or in general for usability ?

I think that changing the relative links to package links breaks the tests right now during instal (the img-utils package is not found).
Do you think it's possible to publish a 0.0.1-1 pre-release version?  As long as sem-ver is followed when updated, it should be ok for dependent packages to start using parts of the V2 system.

Coming very soon... I asked the same. Hopefully, we can publish an ‘alpha’ version soon.

I am ok with trying to release a 0.0.1 version, but the error will not be solved, as this seems to indicate a lerna issue , as local deps should be resolved correctly, npm or not
Will take it for a spin locally.
Oh btw , 'latest', should not be used in dependency specifications, that causes issues (we have been bitten by that quite a lot in the past :) ), we now always use exact versions (works well, is reliable and controllable)


Issue: there are no groups to display in Groups and Deployments

Issue: there are no groups to display in Groups and Deployments

Follow this tutorial: https://learn.microsoft.com/en-us/azure/iot-hub-device-update/device-update-simulator
I also setup up these below portions already

Message Routing,
Buit-in endpoints: adum,
IoT Hub AIM (grant device update service principal to Iot hub data contributor)
install DU agent in Ubuntu 18.04
add tags.ADUGroup in module identity

Expected Behavior
Tell us what should happen
Current Behavior
Tell us what happens instead of the expected behavior
Steps to Reproduce
Provide a detailed set of steps to reproduce the bug.
1.
2.
3.
4.
Device Information

Host OS [e.g. Ubuntu 18.04, Windows Server IoT 2019]: Ubuntu 18.04
Architecture [e.g. amd64, arm32, arm64]:  amd64
Provisioning Method: string
*DU Agent Version (run 'sudo -u adu /usr/bin/AducIotAgent -v'): 1.0.2

Logs

Additional Information
Please provide any additional information that may be helpful in understanding the issue.

Few hours later, I figured out there is a sentence on top of adu page: initiate a Device Sync operation from the Diagnostics tab to register any devices provisioned before permission was granted.

we have to go "Diagnostics" page, then in "Find missing devices" session, click "Start a device sync", once it completed go back to "Group and Deployments" page, the device will appear.

The ticket can be closed.


retrieve results based on rating

Hi, I notice that when I do manual queries like http://npmsearch.com/query?q=search&fields=name,description,version,keywords,rating, the results I get back don't factor into account the ratings the same way they do on the website.
Is there a way to form the url so that I only get back results to have them retrieved by highest rated (like the website) or retrieved based on a minimum rating?
Thanks!

ah, I got it working by appending &sort=rating:desc to my search
thx


add auto-generated Regsitry and Protocol to ConfigManager

When user does not specify any ProtocolConfig or RegistryConfig, the auto-generated Regsitry and Protocol should be added to ConfigManager

LGTM

Codecov Report

Merging #5516 into 2.7.5-release will increase coverage by <.01%.
The diff coverage is 100%.

@@                 Coverage Diff                 @@
##             2.7.5-release    #5516      +/-   ##
===================================================
+ Coverage            61.08%   61.09%   +<.01%
+ Complexity             423      422       -1
===================================================
  Files                  921      921
  Lines                37588    37591       +3
  Branches              5465     5465
===================================================
+ Hits                 22961    22966       +5
  Misses               12120    12120
+ Partials              2507     2505       -2

Impacted Files
Coverage Δ
Complexity Δ

...g/apache/dubbo/config/AbstractInterfaceConfig.java
78.63% <100%> (+0.09%)
0 <0> (ø)
:arrow_down:

...ava/org/apache/dubbo/config/ServiceConfigBase.java
57% <100%> (+0.43%)
0 <0> (ø)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0%> (-6.9%)
0% <0%> (ø)

.../org/apache/dubbo/remoting/ExecutionException.java
15.78% <0%> (-5.27%)
0% <0%> (ø)

.../remoting/transport/netty4/NettyClientHandler.java
59.32% <0%> (-5.09%)
0% <0%> (ø)

...he/dubbo/remoting/transport/netty/NettyServer.java
70.17% <0%> (-3.51%)
8% <0%> (-1%)

...bo/rpc/cluster/support/FailbackClusterInvoker.java
67.21% <0%> (-3.28%)
0% <0%> (ø)

...exchange/support/header/HeaderExchangeHandler.java
61.94% <0%> (-2.66%)
0% <0%> (ø)

...g/apache/dubbo/registry/consul/ConsulRegistry.java
61.87% <0%> (-0.63%)
29% <0%> (ø)

...rg/apache/dubbo/common/timer/HashedWheelTimer.java
63.1% <0%> (+0.34%)
0% <0%> (ø)
:arrow_down:

... and 5 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 6249f0f...4a176aa. Read the comment docs.


Changed the style of the 'Old version' callout box

Changed the style of this callout box from red ('danger') to blue ('note') so it is less intrusive for the reader.

Compare:


Avoid IConfigurationService.acquireInstantiationService()

IConfigurationService.acquireInstantiationService() asks for trouble. We should not inject services to hide cyclic dependencies but rather break up these dependencies.

Looks like you already took care of it.

N, I might have moved it, but its still there:
https://github.com/Microsoft/vscode/blob/c6315a3d30771ff83bb6cf56528404859f25a9e4/src/vs/workbench/electron-browser/workbench.ts#L562

Sorry this was mistake. Closed this instead of https://github.com/Microsoft/vscode/issues/69228

@sandy081 maybe this is in reach for next debt week?

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Nudge display brightness on return to cros

For some targets, when croutoncycling back to cros, the display brightness is not restored correctly - the host cros dbus server knows the correct value but the hardware does not (observed on Toshiba B3340, utopic chroot). This admittedly hack-y change nudges the the display brightness down then up so that the old correct value is restored.

This is relevant to #407.
What about using the brightness command to get the current percent, then either re-apply it or jump between two values?  Down then up won't work at minimum brightness, for instance (well, it will, but the end result will be different).

Ah, good point. The brightness command truncates the brightness value it returns - on the assumption that there's a good reason for that, I've left it alone and brought that functionality, without rounding, into croutoncycle, then implemented your suggestion.

Actually, I'd rather you change brightness to have a flag that chooses truncated/untruncated than duplicate the code (probably default to truncated, and add -f to disable truncate or something).  The brightness command is actually very poorly-written (and the "instant" thing doesn't actually work)...

Here's a cleaner version of the brightness fix, complete with a flag to the brightness command rather than a duplicated implementation.

Not sure if people are still hitting this bug.  If they are, this needs to be rebased and corrected.

No longer hitting this bug. I'd suggest just tossing this PR altogether. Thanks very much for your help and followup.

Oh, okay!  Thanks again for submitting a fix.


[interop][SwiftToCxx] add -enable-experimental-cxx-interop-in-clang-h…

…eader flag that lets you enable reverse interop only

This is needed temporarily for now while some clients might have trouble enabling forward interop for various reasons.

@swift-ci please test

@swift-ci please test source compatibility


New worker request: opsec-fbsd15

Username
opsec
GitHub username
No response
Name
Kurt Jaeger
Email address
<EMAIL_ADDRESS>Password status
I will use an existing owner password
Processor architecture
amd64
Operating System
FreeBSD 15-CURRENT
Anything special about the worker?
No response

@zware: Would you mind to have a look at this request? Tell me if you want me to handle it.

What's the status?

Tests pass, builder info completed: all good.
https://buildbot.python.org/all/#/builders/1238/builds/2


"Class Features" error

When I try to add the "Class Features" pre-generated material, it keeps populating it in multiple places throughout the document and creating other coding problems.
Additional Details
Share Link :
or
Brew code to reproduce : Click to expand
PASTE BREW CODE HERE

Duplicate of #413 . A fix is on the way.