[C++][Parquet]  Update release-candidate script links to gitbox

Reporter: Uwe Korn / @xhochy
Assignee: Uwe Korn](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=uwe) / @xhochy
PRs and other links:

GitHub Pull Request #439

Note: This issue was originally created as PARQUET-1219. Please see the migration documentation for further details.

Wes McKinney / @wesm:
Issue resolved by pull request 439
https://github.com/apache/parquet-cpp/pull/439


chore(main): release 3.7.0

:robot: I have created a release beep boop
3.7.0 (2024-06-18)
Features

new index header text (e1c91b2)
new index header text (#117) (58eac02)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Hessuew/flamethefreeze/releases/tag/v3.7.0 :sunflower:


Review Primate Resource View Styling and Layout

With resource view layout of VM, could it use any improvement.

@RitchieVincent ping


Breaking update in minor 2.1.4 release?

Hi Jay,
Just wondering if you'd perhaps be open to possibly trying to avoid breaking changes on 0.0.x releases? Might be a convenient way for lib users to know whether an update requires migration or not?
Thanks again for all your work on Expectations! Cheers :-)

Yeah, definitely. Sorry for the breaking change in a minor version, it's laziness not an intentional disregard for common sense. The result is the same though, I'll try to do better going forward.

No problem :-)


Add person status while requesting assistance

Options:

Isolation
Sick
Elderly
Lonely
Normal

Remeber to update all relevant forms, tables and Excel output.

@astar26
How are we going to ask people this? Like
סיבה לבקשת עזרה: בידוד/חולה מאומת/קבוצת סיכון/בדידות
sounds like people wouldn't want to fill out that they need assistance because they're lonely. How can we phrase it to sound better? Or what will the question/answers be? (For example "מבוגרים בקבוצת סיכון" could sound better than "זקנים")

It should be a choice between the states, and possibly make it optional, although it helps quite a lot when trying to understand if the request is legit.
@amirkasher opinion?

@astar26 I just think we need to think of how to word the options here.

Isolation == "בבידוד", this one is the easiest
Is "Sick" a confirmed COVID-19 patient or just someone who's sick in general?
Elderly can be "מבוגרים בקבוצת סיכון"
What is "Lonely"?
We can't have "Normal" as an option (implying the other options are not normal). Maybe something like "None of the above" or "Other" could work

Once we decide on these I can implement it - it should be a small addition.

As i see it, there are only 3 options:

solation == בבידוד, any one who diagnosed with COVID - 19
Sick can be == קבוצת סיכון גבוהה, any one who is in high risk, can be anyone with background illness (can be also 30 year old woman with Asthma or Elderly = person who is above 60 years old)
Other == אחר, with a text input as a description

@amirkasher Alright, cool. Then we don't want "sick" right? only "בבידוד", "קבוצת סיכון גבוהה", "אחר"

Perfect
the אחר doesn't need to be be with input..

We currently have the help request "סיוע לעובדים חיוניים" for essentials workers in need of help.
This sort of collides with the "request reason / person status", because the help request type is basically the request reason. Do we want:

To eliminate the "סיוע לעובדים חיוניים" help request and add a "עובד חיוני" status to the request reason?
To keep the "סיוע לעובדים חיוניים" help request and additionally add the "עובד חיוני" status to the request reason?
Keep everything like it is now

@amirkasher


401 error when using jQuery Mobile and added mobile format.

I added a mobile format using https://github.com/tscolari/mobylette
I get a 401 error when I'm viewing a page that requires a logged in user.
I did the Responder alias.
After tweaking around I got it to work with:
  config.http_authenticatable_on_xhr = false
However I thought just adding :mobile to navigational_formats would do it. It did not. Is doing the on_xhr = false the right thing to do or is there a bug somewhere not recognizing navigational_formats?

Seems like a bug somewhere. I assume your mobile request is not a xhr, so it seems something else is going on. Could you please push an application or a failing test to Devise that reproduces the problem?

Btw, which Devise and Rails versions are you using?

I'm experiencing the same issue, but I'm not using mobylette.
I've manually added an additional mime type handler, and when requesting any restricted page, am given a 401 error with the text "You need to sign in or sign up before continuing" as the only html source sent to the browser.
It does work when requesting a non-restricted devise page (ie, the sign-in page) when I've overridden the devise templates and have provided a .mobile.haml implementation. Even though I've gotten the login page to work, when actually submitting the form (which jQuery Mobile does as an ajax request), I'll get a violent 401 error page if the credentials are invalid, or this if they're good:
Template is missing
Missing template devise/sessions/create, application/create with {:handlers=>[:erb, :builder, :coffee, :haml], :formats=>[:mobile], :locale=>[:en, :en]}. Searched in:
  * '/home/prg04itd/projects/ruby/winter_patrol/app/views'
  * '/home/prg04itd/.bundler/ruby/1.9.1/rails_admin-5483935fd170/app/views'
  * '/usr/lib/ruby/gems/1.9.1/gems/kaminari-0.12.4/app/views'
  * '/usr/lib/ruby/gems/1.9.1/gems/devise-1.4.9/app/views'

I think this is an issue with the way Devise handles rendering its templates when the format is set to "mobile". I'm using a few application controller methods that change the format when the user agent is from a mobile device; when I switch to a desktop one, everything works fine (since the format is back to html).
For the record, I'm running Rails 3.1.1 and Devise 1.4.9.

Also of note, I get a complaint about a missing template when providing valid credentials to the login form in mobile format, saying that a template couldn't be found for the create action. I created a template under devise/sessions/ called create.mobile.haml, after which the login succeeded, and didn't generate errors (although it still didn't re-direct me to the index, or anywhere for that matter).
It looks like Rails is supposed to use html as a fallback format if it can't find templates for the specified format. You would figure that would fix the issue we're seeing above, since devise has html templates it can use.

Can someone provide an application or a failing test case to Devise suite that reproduces the error? Otherwise there is not much we can do. Thanks!

Could someone provide an application that reproduces the issue? Otherwise there is nothing we can do. Thanks.

Closing this due to lack of a way to reproduce it.


Validate env file

Create a function that validates all environment variables when the app starts.
The function should only allow the app to start if all environment variables are verified to be present and of the required type
An error should be thrown instead of the app crashing if a variable isn't present.

Is this resolved?

@The-CodeINN any updates on this??
still working on it..??

Yes, but I was unable to start the application. Here is my approach:

Create an env file with the following content:

APP_VERSION=v1
NODE_ENV=development
PORT=5000
JWT_SECRET=pfkkskwdlqoeka

Run pnpm predev
Run pnpm dev
No response when I tried to access it on localhost:5000

I think the generated files are from vitepress.
I'll give it a try and share my findings...

@The-CodeINN I checked, and based on your steps, it needs to have a database URI before it can even run.
I opened an issue that is probably related and would add it in the config file

Okay.. i think it's better we have an .env.example for reference

I'll add an env.example later today


SSL certificate on website invalid since Mar 15, 2022

Hello, please fix your website's SSL settings on https://cone.jondgoodwin.com/, the SSL certificate is invalid since Mar 15, 2022.

I know about it. Have tried to fix and failed so far, due to upgrade script no longer working.
Work commitments have made it impossible for me to carve out bandwidth to rebuild my server as of yet.
It is my top priority when I do get the time. Thanks for raising the issue!

I fixed this last week.  Sorry for any inconvenience.


Add toAttributes to the Transformer

The toAttributes of a transform are required for build scans in the follow ups:

https://github.com/gradle/gradle/pull/23592
https://github.com/gradle/gradle/pull/23486

Your PR is queued. See the queue page for details.


Missing license

Could you add a license to the repo so it could be used in a compliant way?
maybe MIT as most yocto layers use it.

MIT license has been added to dunfell branch.

MIT license has been added to dunfell branch.


[ENHANCEMENT] Refactor FallDetector and TFImageDetection

Is your feature request related to a problem? Please describe.
Currently FallDetector inherits from TFImageDetection, which introduces several maintainability issues:

confidence_thresholdis initialized in FallDetector as a new instance attribute, even though it is expected in the TFImageDetection constructor and used in the super class as an instance attribute. We need to clean up this redundancy, because it makes the code harder to maintain.
TFImageDetection includes methods with assumptions about the underlying ML model that do not hold true for PoseNet. For example it expects that the model output tensor has a bounding box.

Describe the solution you'd like
We need to take these specific assumptions out and construct a more generic abstract class without them - maybe TFDetectionModel. TFImageDetection and FallDetector can separately inherit from TFDetectionModel. At that point TFImageDetection should be probably renamed to TFBoundingBoxDetection to better represent its purpose.
Describe alternatives you've considered
Open to alternative suggestions that would help keep the code clean and maintainable. Along these lines, we will move FallDetection and other future custom models towards their own independent repos with own packaging and pypi distribution.
Additional context
https://github.com/ambianic/fall-detection
https://github.com/ambianic/fall-detection/issues/1

Closed by #278


Dialog Size

Hi! Thank you for this mod!
After you enable the scroll, close the dialog and open it again the dialog size is smaller then the fields in there. If you click the title you can fix it.
https://user-images.githubusercontent.com/4999783/189525057-d5627ee5-0df0-411c-affc-71e2146f56ad.mp4
Is it possible to make the dialog open in the right size?
Tks

fixed


[4.1][Parse] Disable token receiver while parsing types in editor placeholder

Explanation: The parser has TokReceiver for recording consumed tokens used by SourceKit. When the parser parses types in editor placeholder (e.g. '<#T##String#>'), it should stop the recording so it doesn't record extra tokens following to the placeholder. Although those extra tokens were usually just ignored,  if the placeholder was at the end of the file, it used to cause a SourceKit crash.
Scope: Affects SourceKit for files which has placeholder at the end of it.
Issue: rdar://problem/36081659
Risk: Low, TokReceiver is disabled in normal compilation.
Testing: Added compiler regression test.

@swift-ci Please test


Frozen String Literal Compatibility

Turns on frozen string literals in 2.3.0+ and fixes code for it.

@larskanis I know you've already put thought and work into frozen strings ... can you comment?

Sorry -- I didn't see @larskanis work towards this.  One thing to note is that I used String.new("")  rather than String.new for a new empty string.  One would think they would be identical however it turns out the first produces a string with the encoding of the document  - the second has an encoding of ASCII-8BIT for some reason.

@flavorjones The second commit here fixes the random failures that have been occurring on PRs, caused by test order and when the encoding alias added by default was cleared.  If you don't want the frozen literal changes you may want to cherrypick the second commit separately

I don't think we want to pollute all the nokogiri files with this magic comment.
@twalpole The difference between String.new and String.new("") is because the first one doesn't have a source file context, so it can not get any encoding information. The latter one don't build a string with the encoding of the XML document, but with the encoding of the surrounding source file. However this difference doesn't change the nokogiri-API at all.
IMO we can close this in favor of #1413 .

The "pollution" of the magic comment allows nokogiri to gain the benefits of default frozen literals before every other gem a project (using nokogiri) depends on being updated to allow someone to use the ruby switch.
Either way - the second part of the commit attempts to fix the random invalid encoding test failures that have been occuring, so might be worth cherrypicking
@larskanis Yes, by document I meant the file it was in, not the XML document - updated comment to clarify, and I would say String.new does have a context (the file it's called in) although I guess the default value passed to it is declared in the source file.  It may not change the Capybara API but it does mean if you then immediately concat to the new string there are encoding translations that need to be made and you end up with a string that is not in the encoding one would generally assume (if just creating a literal)

@twalpole Sorry, I missed the second commit, but cherry-picked it to master, now.

Closing, frozen string support is now on master.


Retrieve tags from Bass tracks when available

For use in ppy/osu#21189. Only retrieves a subset of all the tags read by Bass that seemed relevant to osu!.

Also please check code style.


Deduplication doesn't seem to work for ReplicatedMergeTree tables

In the #1406 issue, I asked how to handle retries. After some discussion, I learnt that data blocks are deduplicated for the ReplicatedMergeTree tables. Since we have Distributed tables in front of our local tables, we revised the sharding keys for the tables to use table columns, so that the same data is sent to the same replicas. However, later I found that it's not necessarily required to retry the writes into the same replica for the data block to be deduplicated as per this(https://clickhouse.com/docs/en/engines/table-engines/mergetree-family/replication) as the checksums are stored in ZK:

Data blocks are deduplicated. For multiple writes of the same data block (data blocks of the same size containing the same rows in the same order), the block is only written once. The reason for this is in case of network failures when the client application does not know if the data was written to the DB, so the INSERT query can simply be repeated. It does not matter which replica INSERTs were sent to with identical data. INSERTs are idempotent. Deduplication parameters are controlled by merge_tree server settings.

We have encountered a situation in production again which resulted in more data to be persisted. What caused the writes to fail was a rolling restart of the clickhouse servers(we have 3 shards, 2 replicas each) and our service using clickhouse jdbc client reported that the write wasn't successful, so we retried until it succeeded:
09:54:36.562 [***-writer-0] WARN  *** - Failed to write 69177 of 69177 stats due to recoverable error; will retry
***: java.sql.BatchUpdateException: Code: 242, e.displayText() = DB::Exception: Table is in readonly mode (zookeeper path: /clickhouse/tables/***-2/***) (version <IP_ADDRESS> (official build))
, server ClickHouseNode [uri=https://***:8124/***, options={sslrootcert=***.crt,use_server_time_zone=false,use_time_zone=UTC}]@1814630837
...

We were hoping that even though we retry with the same data, deduplication would safeguard against duplicate data; however, this doesn't seem the case. To check whether deduplication kicked off, we reviewed the logs on the servers and looked for entries similar to these, yet there were none:
..17:52:45.064974.. Block with ID all_7615936253566048997_747463735222236827 already exists locally as part all_0_0_0; ignoring it.

Is it possible that since the table on a certain replica to which the initial write call was directed was in readonly mode, it received the data, saved it(but didn't distribute to other replicas yet) but didn't saved the data block checksum in ZK, so when we retried again and even though the data was directed to the same replica(though as I understood it is not required to use the same replica for retries), the data was written again because technically there was no checksum for this data block? If that's not possible, what else could be the culprit here? Any other suggestions how to handle the retry scenarios to achieve reliable writes, i.e., providing atomicity? We cannot use transactions as I believe they don't support replicated tables, so the only reliable way for us is to retry the whole batch because as we understood from the previous conversation, clickhouse jdbc client doesn't provide strong guarantees on what has succeeded or failed in the batch. I stumbled upon this config(https://clickhouse.com/docs/en/operations/settings/settings#insert_quorum) but I believe it would affect performance for inserts.
Environment

Client version: 0.4.6
Language version: Java 17

ClickHouse server

ClickHouse Server version: 21.8.14

Is there anyone who can attend to this issue, please?

Not related to java. Wrong place to ask this question.
Out of support 21.8.14. You should upgrade. https://github.com/ClickHouse/ClickHouse/issues/27667


Needs update

Because file_picker >=5.0.0 depends on ffi ^2.0.1 and every version of rich_clipboard_windows depends on ffi ^1.1.2, file_picker >=5.0.0 is incompatible with rich_clipboard_windows.
And because rich_clipboard >=1.0.0 depends on rich_clipboard_windows ^1.0.0, file_picker >=5.0.0 is incompatible with rich_clipboard >=1.0.0.
So, because hatchapp depends on both rich_clipboard ^1.0.1 and file_picker ^5.0.1, version solving failed.
pub get failed (1; So, because hatchapp depends on both rich_clipboard ^1.0.1 and file_picker ^5.0.1, version solving failed.)

Thanks for the report. Version 1.0.1 should be available on pub.dev with the fix now.


Bugfix issue #33

@ImJohnMDaniel I've added my comments for further discussion at https://github.com/afawcett/force-di/issues/33#issuecomment-443030039


Errors in Installation by following the Typical Installation  under "Install guide"

After following  the Typical Installation  under "Install guide", then I am trying to verify the installation, I got the following server fatal error:
2021/05/24 21:13:00 [error] 1023#1023: *80063 FastCGI sent in stderr: "PHP message: PHP Fatal error:  Uncaught Error: Class 'Minify\App' not found in //min/bootstrap.php:25
Stack trace:
#0 //min/index.php(10): require()
#1 {main}
thrown in //min/bootstrap.php on line 25"
Can you please help?
Thanks,
James

It seems I was following the wrong install guide - The second option "Installing as a composer dependency" works for me.
Thanks,
James


Installation error: Syntax error, unexpected '?'

I installed HAXcms on a new PHP 5 LAMP (Bitnami) server on AWS Lightsail. I used the hands-off installer where I cloned the repo and then ran install.php. It showed me the HAX art and the username and password I'd need.
Installation seemed to go through OK, the _config folder was created fine and I can see there's all the setup in config.json but when navigating back to the home page I see the following error show up:
Array
(
    [type] => 4
    [message] => syntax error, unexpected '?'
    [file] => /opt/bitnami/apache2/htdocs/vendor/symfony/filesystem/Filesystem.php
    [line] => 560
)

Edit: I have a fix. I'll put it through as a potential PR now.

out of curiosity why did you pick PHP 5? All the 5.x's are beyond LTS unless you've got an older version of ubuntu your running that keeps this supported / other things on the same box that would need PHP 5 we've gotta start moving off of it -- https://www.php.net/supported-versions.php

Your PR I"ll assume is accurate but I'm not sure that we can / should patch the vendor directory since composer managed that (ugh I know... :) ). But I'l be curious to hear more of if Bitnami PHP 5 was intentional or just what you happened to pick.

Hi Bryan, it was just something we were going to install on a (sort of) spare server which is running Ubuntu 14/PHP5 as an internal wiki/Knowledgebase.
I did try on PHP7 and got the same issue.
You’re right about the vendor file, it might be something we just use a patch for internally until we’ve played around with it more. Once the fixes were in place, we weren’t able to login due to a POST 403 issue but I’ll maybe have a proper mess around with it on PHP7 tomorrow and put through a proper PR if there are still issues.

Hmm ok, didn't realize it would be THAT php version specific. Wonder if it's using php 7.2 syntax or something. May need to hang onto patches then for broadest support (ugh). Did you apply the patch and then try in ddev/docker locally? I'm not against patching it just (as usual) gritting my teeth about symphony heh

php 7.1 is required for HAXcms apparently based on Symphony requirements. Kinda hard to patch and maintain when we'll be using more symphony in the future


Dark mode should only detect "dark" class

My website has a button to manually switch between dark and light mode.
The button click event implements the switch by adding or removing the "dark" class to the top level HTML tree.
Why use Dark Mode by adding a "dark" class?
Tailwind CSS recommends doing this. Dark mode in Tailwind CSS:

Now instead of dark:{class} classes being applied based on prefers-color-scheme, they will be applied whenever the dark class is present earlier in the HTML tree.

How you add the dark class to the html element is up to you, but a common approach is to use a bit of JavaScript that reads a preference from somewhere (like localStorage) and updates the DOM accordingly.

<!-- Dark mode not enabled -->
<html>
<body>
  <!-- Will be white -->
</body>
</html>

<!-- Dark mode enabled -->
<html class="dark">
<body>
  <!-- Will be black -->
</body>
</html>

By referring to Tailwind CSS, we can learn some tips.
For dark mode detection, the package should leave the decision to the developer. Package only detecting the dark class of the outer DOM.
Packages should not detect @media (prefers-color-scheme: dark). Since it is read-only, projects cannot change the appearance of components by toggling this value.
What developers do in their projects

Automatic switching: listen to @media (prefers-color-scheme: dark) and switch the dark class.
Manual switching: dark mode switch button.
User preferences: local storage, browser session storage, server storage, etc.

Package only do
The package only needs to add the dark mode CSS to the dark class, no other behavior is required.
More
This issue also resolves #16 .

Hey man. The idea of this component is to mimic or be aligned with the browser UI. This concerns the language and dark mode. Because user experience start from this custom dialog and continues in browser UI, and it's important to keep it consistend.

OK, thanks for the clarification. This plugin is designed to complement the browser's capabilities, not to extend functionality to project. I understand now.

@angelofan you actually extend your project, but keeping the UX consistent between the app and browser.


FEDORA PACKAGING: Statically linking some C dependencies instead of dynamically linking system libraries:

This is a problem because some "bindings" (-sys) crates default to building and statically linking a vendored copy of the wrapped C library.
In Fedora, we address this by patching the crate sources to unconditionally link dynamically. You might need to apply similar patches to some vendored crates.
It looks like this currently only affects the "libz-sys" crate, which bundles both zlib and zlib-ng and falls back to building them from source if the zlib pkg-config file cannot be found. As far as I can tell, bpfman does not link to libz.so, which is probably not what you want. Looking at the build script for libz-sys, you might be able to fix this by adding BuildRequires on "pkgconfig(zlib)".

you might be able to fix this by adding BuildRequires on "pkgconfig(zlib)".

Done


Add support for Defold game engine

Defold's archive data and index files; named "game.arcd" and "game.arci" respectively; are consistent across all supported platforms (e.g. Android, iOS, Linux, macOS, Windows)

Please disregard this (duplicate) request in favor of #17


Replace AbstractTriangular by UpperOrLowerTriangular

This is future-proof, and works around the breakage due to ArrayLayouts v1.0.11
After this, print_matrix_row works again:
julia> I(ℵ₀)
ℵ₀×ℵ₀ Diagonal{Bool, LazyArrays.CachedArray{Bool, 1, Vector{Bool}, Fill{Bool, 1, Tuple{InfiniteArrays.OneToInf{Int64}}}}} with indices OneToInf()×OneToInf():
 1  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  …
 ⋅  1  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅
 ⋅  ⋅  1  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅
 ⋅  ⋅  ⋅  1  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅
 ⋅  ⋅  ⋅  ⋅  1  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅
 ⋅  ⋅  ⋅  ⋅  ⋅  1  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  …
 ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  1  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅
 ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  1  ⋅  ⋅  ⋅  ⋅  ⋅
 ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  ⋅  1  ⋅  ⋅  ⋅  ⋅
 ⋮              ⋮              ⋮        ⋱

Isn't this already fixed on v0.12.14?

Oh indeed, somehow I had not fetched the upstream changes. This PR only adds a test after the rebase.


restore BCF tools per user request

cc: @gschleusner
BCF Toolset restoration.

@gschleusner can you point me to a proper thing that needs to be updated here? I think I started working on the wrong ones (old version of the tool). It has some GDrive functionality that you mentioned that doesn't exist anymore.


Elastic 7

QE instance we use in prow is using elastic search 7.* want to try to get changes Tullio is adding properly back working there.
Seeing if we can get this properly running with older version
The enviornment variable ES_SERVER is a public url that we should be able to test properly on

this change is being added in pr https://github.com/krkn-chaos/krkn/pull/658, closing


In the Game-Lobby the Gamemaster can choose between the Gamemodes "Higher or Lower" and "Guess the Price". Depending on the mode he chooses the corresponding Game-View is rendered once the start game button is pressed.

commit: 2ef404b


Illogical search results (to me!)

Hi Meilisearch, and thanks for the good work.
Could you explain to me why in searching "personnel reprographie" (with CURL, against v0.26), I get
{
  "hits": [
    {
      "lemmatized": "convention collectif national commerce détail papeterie fourniture bureau bureautique informatique librairie 15 décembre 1988 étendre arrêter 14 décembre 1989 jorf 30 décembre 1989 champ application fusionner celui convention collectif national personnel reprographie idcc 706 arrêter ministériel 9 avril 2019",
      "uid": "ebd824693c42d7b9351d934aa0dc000caa7ae6ed53b93472f9c5918ac4ed9d5f"
    },
    {
      "lemmatized": "convention collectif national personnel société coopératif hlm 15 mai 1990 champ application fusionner celui convention collectif national personnel office public habitat désormais intituler convention collectif national personnel office public habitat société coordination idcc 3220 arrêter ministériel 16 novembre 2018",
      "uid": "ba7f340219d108c8d39071cfd883e8922d930f2a964241f872bbf16b51a05953"
    }
  ],
  "nbHits": 2,
  "exhaustiveNbHits": false,
  "query": "personnel reprographie",
  "limit": 20,
  "offset": 0,
  "processingTimeMs": 4
}

and when I search "accord personnel reprographie", I get no hits ?
This seems strange to me:
Result #2 has only 1 word matching ("personnel") and nothing close to the second word ("reprographie"), but I get the result anyway. So far so good.
So why with adding "accord" to the query don't I get at least result #1, which has both "personnel" and "reprographie" ?

Thank you for your time answering this.
Have a nice day.

Thank you @curquiza for your quick and detailed reply.
I understand the rationale and I should have remembered that MeiliSearch implements a prefix search (it's in the docs) .
Would you allow me to share a more global point of view?
This prefix search example -- and many others in milli / MeiliSearch -- indicate an opinionated platform.
As developers, we all know that opinionated is good because it simplifies things and reduces time to deployment, as long as the use case fits.
Now, implementing opinions as default BUT allowing non-defaults (in this case regular vs prefix search) would benefit both you and us: we would have more flexibility to adapt to usecase specificities (ie the real life! [1]) and you would make your platform more universal (ie somehow lift the "if usecase fits" barrier).
In other words, defaults make MeiliSearch a very valuable and efficient open-source Algolia with a great future. Non-defaults alternatives would open MeiliSearch to a lot more usecases based on the likes of ElasticSearch (that many want to dump for more efficiency).
I don't want to sound sound like I'm telling you what to do. But I believe allowing non-defaults is (generally, based on your clean code) a small effort with a great return on time investment. Opinionated is good. Flexibly opinionated is the best of both worlds.
Have a nice day.
[1] Go explain customers and their users that they must enter the most important search tokens first...

Hey @trailerparkdev,
For your information Meilisearch is an open source project and we are open to any feature that would help people using the engine.
Therefore, this is why you can propose any feature you like with an explanation about your use case in this dedicated repository.
There is no reason to explain why what we have done is wrong and that we should implement something or something more, just propose that on the appropriate repository and wait for our feedback. We are doing a lot of work in specifying the settings and default options the engine should have, don't say the contrary.
Have a nice day!

Hi @Kerollmops.
Thanx for the time of your reply.
Duly noted for the repository.
I didn't mean to imply that what you've done so far is "wrong" in any way. If that's what it sounded like, I'm sorry. And yes there are many instances of your specifying settings and default options, and you do that with a remarkable documentation (concise and precise). I don't have the impression of having stated the contrary.
My only point was that, from a peer developer and friendly standpoint, cornering MS to mainly search-as-you-type use cases seemed limitative considering its capabilities and the talent of the team (from what the code reveals). In this respect, allowing for non-prefix searches would immediately open MS to many new use cases, with what I believe is not a inhuman effort, even in a startup's constrained time environment.

Keep up the good work.

Hey @trailerparkdev,
No worries, we will review your feature request, post it in the product repository when you have time 🙂

In this respect, allowing for non-prefix searches would immediately open MS to many new use cases

Have you tried to enclose the query words in double-quotes already? I know it can be cumbersome but it is at least a good enough solution for now.
Have a good day!

Hi @Kerollmops.
Thanks again for responding.
The double-quotes trick may not always give expected results with search patterns whose tokens do not really follow one another in the target documents.
Will post the FR!
Have a nice night.

Thank you @trailerparkdev for posting your feature request!
I close this issue, feel free to re-open if needed!


Caps in titles?

@Muraszko : This seems like a low priority datacleaning issue? There are things that could be done either at index-time or render-time, but I don't want to invest the effort here if there will be a fix upstream. (This was an ingest of only about 10% of the records.)

Please see my response in #291 .  I'll be fixing these in the DB as part of regular grooming.

I'd like to keep this open until it's actually fixed, just so that if someone sees it in the next few weeks they know there is a plan.

Sure thing

@mccalluc
Went through several times, fixed a lot of caps, apostrophes, and other Series duplicates.
I'm sure there are plenty of capital Program titles, but I'll make separate tickets for those as they come up.
I'm fine with you closing this issue.

And you've reingested? I want to be sure that if we see problems on the site, they are things that still need to be done.
(If it is reingested, feel free to close.)


Dead link in WASI tutorial

Hi! The WASI tutorial contains a dead link to the browser polyfill:
https://github.com/bytecodealliance/wasmtime/blob/22a4480e90d2e5bb4933a0bd027bd31d007b8b04/docs/WASI-tutorial.md?plain=1#L323

This is essential. Without the polyfill I don't see any possibility to run WebAssembly in the browser.

Thanks for the report! I've now filed https://github.com/bytecodealliance/wasmtime/pull/9471 to update Wasmtime's WASI documentation.
The old browser polyfill was very limited and hard to use, and has since been surpassed by better alternatives, in particular, Jco provides a toolchain for running Wasm programs in browsers, that includes WASI support.


[5.5] Add isFromApi method to Request object

I think this feature is useful when you are using the same controllers for manage API and web requests, maybe you want to know when a user is requesting from an api and return a different set of data that when the request came from the web.
This method bring a clean way for read and implement. For example if you want to return a set of posts in JSON format when someone make a request from an api but return a view when the request came from the web, you can do this.

public index (Request $request)
{
    $posts = Post::paginate();
    if ($request->isFromApi) {
        return $posts;
    } else {
        return view('posts.index', ['posts' => $posts]);
   }
}

@maxalmonte14 I don't have /api segment in my URL. API is on subdomain, this wouldn't work then.

In my point of view all API endpoints should have special middleware which will define that it's API route.

@a-komarev Well, maybe in your case this not apply at all but when you create an api (using Laravel Passport for example) the default way for access to the endpoints is using your main domain and the segment "api", (http://www.myawesomesite.com/api/users).
I created the method as a little helper function but I believe it is most useful like a Request method.

Eh, holding off on this. Request is macroable.


Fix Bookworm compliancy by using a standalone tomcat9

Fix compliancy with Bookworm (Debian 12) by using a standalone tomcat9 instead of tomcat10 .deb provided by Debian/bookworm
Fix non working logrotate by adding a dedicated config file
Fix permissions by executing _set_permissions after compilation (because compilation is executing by root)
update guacamole to version 1.5.5
update MariaDB connector to version 2.7.12
Change version of the package to 1.5.5-ynh3

Problem

Yunohost package for guacamole was not compliant with bookworm since guacamole is not compliant with tomcat10 and tomcat9 was removed from Debian 12 repositories.

Solution

I configure the installer to use a standalone tomcat9 server instead of tomcat10 from Debian repositories.

PR Status

[X] Code finished and ready to be reviewed/tested
[X] The fix/enhancement were manually tested (if applicable)

Automatic tests
Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

!testme
Thank you for your work :+1: Let's have a look at what the CI says

How to add patches to my pull request? I am a noob, sorry

I don't know how but my last commit is in the pull request. But I think it will not fix the CI. CI is running on yunohost 11 but my changes works only on yunohost 12.

Hi guys how to go ahead? How can I help?


[NFC] [arm64e] Delete arm64e fallback test.

I don't think this test is necessary/useful anymore. We already have a separate arm64e-fallback.swift test checking the current behavior.
Fixes rdar://problem/62334591

@swift-ci please smoke test


Double-bite characters not displayed correctly

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: macOS Mojave 10.14
Hyper.app version: 2.1.0-canary.3
Link of a Gist with the contents of your .hyper.js: vanilla
Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere): N/A
The issue is reproducible in vanilla Hyper.app: Is Vanilla

Issue
I am not sure if this is only applicable to double-byte characters, but at least Japanese characters are not displayed correctly on v2.1.0-canary.3. I have been experiencing this problem after xterm.js was updated to v3.8.0 #3255.

me too.
OS version and name: Windows10 Build18290 (insider preview)
Hyper.App verison: 2.1.0-canary.3

me too.
OS version and name: macOS Mojave 10.14
Hyper.app version: 2.1.0-canary.3

This is probably https://github.com/xtermjs/xterm.js/pull/1758 which will be in<EMAIL_ADDRESS>released either today or tomorrow.

https://github.com/xtermjs/xterm.js/releases/tag/3.9.0

Looks like #3319 fixes the issue!

Thanks for reporting @mrymtsk


Stock Span Problem

DESCRIPTION
Implemented Stock Span Problem in the stack using C++ Language.
Please review this PR.
Issue
#709

https://github.com/Algo-Phantoms/Algo-Tree/tree/main/Stack the link to your problem is not working. Changed the folder and push your code to the Code folder.

@rudrakshi99 You mean i push my solution to Code -> C++ -> mycode ??

@rudrakshi99 You mean i push my solution to Code -> C++ -> mycode ??

yes.

@rudrakshi99 You have closed this PR. Should I make another??

@rudrakshi99 You have closed this PR. Should I make another??

yes

@rudrakshi99 Please check my New PR. Please update me if it is correct or not?


content filter in version 2 ?

Hi,
I'm trying to use the content filter in my version 2 theme (based on https://github.com/picocms/Pico/issues/376) but it renders nothing.
I've looked in Pico's core https://github.com/picocms/Pico/blob/af237c1d672ca254a07d0359bfd9c3a5e0723f4c/lib/Pico.php#L1956 but without result...
{{ item.raw_content }} renders all the .md files, as wanted, but adding the filter produce nothing.
{% for item in pages %}{{ item.raw_content|content }}{% endfor %}

I'm running version 2.0.0-beta.3
What am I doing wrong ? :-/
Thanks in advance !

Please refer to Pico's user docs at http://picocms.org/docs/. It's {{ "sub/page"|content }}.

Hum... I tried this one before posting... but without any success (the page.id return the correct path):
{% for page in pages %}{{ page.id }} // {{ page.id|content }}{% endfor %}

I'm doing a lot of things wrong...

Hum...
I've tried to get back to the basics: using the default theme and the content-sample. It wasn't working because of file permissions...
But then, moving back to my content and my theme, another issue: it seems that the PicoTags plugin (https://github.com/bricebou/PicoTags/blob/master/PicoTags.php) I've tried to rewrite causes the pages array issue :-/
It's strange, cause I have the right {{ pages|length }}and all the{{ page.id }}. It's only the {{ page.id|content }}` that doesn't display anything.

After some more investigations, it was due to the fact that PicoTags rewrited the pages array:
https://github.com/bricebou/PicoTags/blob/823b31b9f264866f449d3de629e779bf52b15aa4/PicoTags.php#L190-L191
Declaring a new array and a new Twig variable does the trick. I might ask later for a code review, once I've tried more this plugin and pushed to GitHub the changes.
Sorry, again...


Integration of supabase returning paseto

http://localhost:8000/api/v1.0/auth/web2

Please correct these:

Use correct casing for json keys - camel casing
Need PASETO for both cases. Can't have different responses for the API - User details can be fetched using other API
Change userType to type and providers to provider

ok !


Impossible to connect to the mssql-server-windows-express from asp.net core application.

I could download and run the microsoft/mssql-server-windows-express So this is works:
docker run -d -p 1433:1433 -e sa_password=MyPassword1 -e ACCEPT_EULA=Y microsoft/mssql-server-windows-express
I can connect it via the powershell and via the Sql server management studio (localhost, 1433 and SA and MyPass1) So I can see the databases and play with them.
However in .NET core API application I cannot connect to the dockerized mssql server express (windows dockerized and not linux)
  "ConnectionStrings": {
    "Default": "Server=<IP_ADDRESS>,1433;Database=master;User=sa;Password=MyPassword1 ;"
  },

Unfortunately I get this sql exception always:

System.Data.SqlClient.SqlException: 'A network-related or instance-specific error occurred while establishing a connection to SQL Server. The server was not found or was not accessible. Verify that the instance name is correct and that SQL Server is configured to allow remote connections. (provider: TCP Provider, error: 0 - No connection could be made because the target machine actively refused it.)'

So the mssql-server-windows-express container is running I can reach it by the powershell command and by the sql server management studio, but when I run the .net core api applicaion in the visual studio 2017 I get this sql exception always.

Hi,
"ConnectionStrings": {
    "Default": "Server=<IP_ADDRESS>,1433;Database=master;User=sa;Password=MyPassword1 ;"
  },

Could you try to change the IP address to the host where you run the container? You can use ipconfig command to find it. Please refer to Connect from outside the container.

Maybe its too late but I'll give an answer.
I had the same issue. The problem is that you're using <IP_ADDRESS> in your connectionString instead of the host machin IP address as @forkazmoden said. That solved my problem.
Don't forget you're inside a container so <IP_ADDRESS> refer to your AspNetCore container but NOT to the host machin. That's why you can login sucessfully from Powershell or SSMS to your MSSQL Server because in this case <IP_ADDRESS> address refer to your host machin.


Calculate the distance from monocular RGB camera to object

Hello! Thank you for your excellent open-source projects. I have a project recently where I need to implement a grass-cutting robot that can recognize and avoid obstacles using vision. I'm using a monocular RGB camera and plan to perform camera calibration using OpenCV and a chessboard pattern to obtain the camera's intrinsic matrix. I have previously worked with the Depth-Anything algorithm, which allows me to obtain depth values for each pixel in the captured frames. I'm also considering trying the Metric3D algorithm. Once I obtain the intrinsic matrix, could you please guide me on how to calculate the actual distance from the camera to an object? Thanks for your help!

I want to use below codes to calculate the camera's intrinsic matrix
import cv2
import numpy as np
import glob

chessboard_size = (9, 6)
square_size = 1.0

objp = np.zeros((chessboard_size[0] * chessboard_size[1], 3), np.float32)
objp[:, :2] = np.mgrid[0:chessboard_size[0], 0:chessboard_size[1]].T.reshape(-1, 2)
objp *= square_size

objpoints = []
imgpoints = []

images = glob.glob('calibration_images/*.jpg')

for image_file in images:
    img = cv2.imread(image_file)
    gray = cv2.cvtColor(img, cv2.COLOR_BGR2GRAY)

    ret, corners = cv2.findChessboardCorners(gray, chessboard_size, None)

    if ret:
        objpoints.append(objp)
        imgpoints.append(corners)

        cv2.drawChessboardCorners(img, chessboard_size, corners, ret)
        cv2.imshow('Chessboard Corners', img)
        cv2.waitKey(500)

cv2.destroyAllWindows()

ret, mtx, dist, rvecs, tvecs = cv2.calibrateCamera(objpoints, imgpoints, gray.shape[::-1], None, None)

print("Camera matrix:\n", mtx)
print("Distortion coefficients:\n", dist)
np.savez('calibration_data.npz', camera_matrix=mtx, dist_coeffs=dist, rvecs=rvecs, tvecs=tvecs)

I believe you could try the code at https://github.com/YvanYin/Metric3D/blob/main/hubconf.py#L174 .
canonical_to_real_scale = intrinsic[0] / 1000.0 # 1000.0 is the focal length of canonical camera
pred_depth = pred_depth * canonical_to_real_scale # now the depth is metric
pred_depth = torch.clamp(pred_depth, 0, 300)

I believe you could try the code at https://github.com/YvanYin/Metric3D/blob/main/hubconf.py#L174 .
canonical_to_real_scale = intrinsic[0] / 1000.0 # 1000.0 is the focal length of canonical camera
pred_depth = pred_depth * canonical_to_real_scale # now the depth is metric
pred_depth = torch.clamp(pred_depth, 0, 300)

Thanks for your help! I try it again.

@YvanYin In the de-canonicalize do we need to scale the intrinsics for the preprocess input resize (e.g. for VIT (616, 1064) )?

@STRIVESS , I have a similar problem where I need to get the distance to an object, and the model outputs wild depth values,
any ideas or leads ?


Google font import fails (404) for some UA

Getting a lot of 404 requests lately in my server logs on the following path:
/assets/css/https://fonts.googleapis.com/css?family=Lato:400,700,400italic,700italic&subset=latin
Looks like the way the fonts are imported (@import 'https://fonts.googleapis.com/css?family=Lato:400,700,400italic,700italic&subset=latin';) might be at fault on some devices, any ideas?

If you have more evidence let me know. Not too much I can do practically with this, but I'll keep my eye out.

@loris, do you know any specific UA for which this happens?

Actually, it looks kinda unrelated to UA, here is the list of the latest 10 UA getting an error:
Mozilla/5.0 (iPhone; CPU iPhone OS 7_1_2 like Mac OS X) AppleWebKit/537.51.2 (KHTML, like Gecko) Version/7.0 Mobile/11D257 Safari/9537.53
Mozilla/5.0 (iPhone; CPU iPhone OS 8_1_2 like Mac OS X) AppleWebKit/600.1.4 (KHTML, like Gecko) Version/8.0 Mobile/12B440 Safari/600.1.4
Mozilla/5.0 (iPhone; CPU iPhone OS 8_2 like Mac OS X) AppleWebKit/600.1.4 (KHTML, like Gecko) Version/8.0 Mobile/12D508 Safari/600.1.4
Mozilla/5.0 (Linux; Android 4.2.2; GT-I9205 Build/JDQ39) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.93 Mobile Safari/537.36
Mozilla/5.0 (Linux; Android 4.4.4; fr-fr; SAMSUNG SM-G901F Build/KTU84P) AppleWebKit/537.36 (KHTML, like Gecko) Version/1.6 Chrome/28.0.1500.94 Mobile Safari/537.36
Mozilla/5.0 (Linux; Android 5.0.1; Nexus 5 Build/LRX22C) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/40.0.2214.109 Mobile Safari/537.36
Mozilla/5.0 (Linux; U; Android 2.3.3; fr-fr; GT-I9100 Build/GINGERBREAD) AppleWebKit/533.1 (KHTML, like Gecko) Version/4.0 Mobile Safari/533.1
Mozilla/5.0 (Linux; U; Android 4.2.2;fr-fr; STARTRAIL4 Build/BQ_ZTEV1.0.0B07) AppleWebKit/534.30 (KHTML, like Gecko) Version/4.0 Mobile Safari/534.30
Mozilla/5.0 (Linux; U; Android 4.3; fr-fr; Ultym5 Build/Ultym5) AppleWebKit/534.30 (KHTML, like Gecko) Version/4.0 Mobile Safari/534.30
Mozilla/5.0 (Mobile; Windows Phone 8.1; Android 4.0; ARM; Trident/7.0; Touch; rv:11.0; IEMobile/11.0; NOKIA; Lumia 625) like iPhone OS 7_0_3 Mac OS X AppleWebKit/537 (KHTML, like Gecko) Mobile Safari/537

Not sure it has anything to do mobile vs desktop neither, my website being targeted for mobile users specifically.

how to use local fonts？

@loris Sounds like the asset path is being prefixed to the import URL. Do see the incorrect path in the CSS itself then? Are you building your own or pulling in the default css components?

The path looks correct in the CSS (the 404 only occurs for a little fraction of requests). I am directly using the node_modules/semantic-ui/dist/semantic.css file in my build flow, the resulting import statement is
@import 'https://fonts.googleapis.com/css?family=Lato:400,700,400italic,700italic&subset=latin';
Could it be something related to enterprise firewall or something? maybe a rule forbidding importing CSS over HTTPS when main site is HTTP?

Perhaps try changing to the relative protocol
``//fonts.etc`
There were about 5 issues on this originally, we settled on https working across the most amount of environments

Thanks @jlukic
I changed the @googleProtocol variable from https:// to // in site.variables file a few hours ago and haven’t seen any 404 since.


merge-conflicts package is with broken styles

This is with the Material Theme:

This is when I switch to Atom Dark:

The problematic elements are put on top of <<<<<<< HEAD, ======= and '>>>>>>> branch'. They are transparent for some reason, that the problem.
Some of the popup modals aren't styled properly as well (really strange layout).

I have the same issue with material theme. Some popups are not styled properly (it seems like all styling is missing for Git Plus package, for example). The same problem with merge-conflicts package. Don't sure what t can be.. Any help would be appreciated.
Here is a screenshot of Git Plus popup window:


avoid creating duplicate file-id rows

comes up, e.g. when using pytest-cov where a test launches a subprocess that also touches covered files
use insert-or-ignore followed by select to ensure the same behavior whether the row already exists or not.
I haven't been able to figure out how to provide a test for this, since I haven't been able to provoke the issue with coverage on its own, only via pytest-cov.
closes #716

Thanks for this, but I'm afraid the problem goes a bit deeper, since sometimes the error is in a different place.  I'll need to do some more investigation into what pytest-cov is doing before I know whether this is the true fix.

No problem! I thought that might be the case. Feel free to close.


Unable to "reload" new image into the cropper - ScrollView get zoomed

First of all, Thank you for this great git!
Hey. i'm having this weird issue. When i'm first initializing the cropper View like this
  self.cropView = AKImageCropperView(frame: self.preview.bounds, image: img!, showOverlayView: true)
                    self.preview.addSubview(self.cropView)

                    self.cropView.refresh()

It's works perfectly. Tho when i'm trying to change the image inside the cropper, like this ;
self.cropView.image = img
                    self.cropView.refresh()

It's changing the image, but the scrollView zoom is almost on max.
I might be missing something, but how can i just replace the image and refresh the image?
re-creating the cropper each time is slow, tried it already. Thanks!

Hi, did you solved it? thx

Yes ))


Sup. Mat.

请问 补充材料在哪下载

/path/of/your/project/magic-animate
创建文件夹pretrained_models
cd pretrained_models
git lfs clone https://huggingface.co/zcxu-eric/MagicAnimate
git lfs clone https://huggingface.co/stabilityai/sd-vae-ft-mse
git lfs clone https://huggingface.co/runwayml/stable-diffusion-v1-5
下载如上三个材料

fully auto installers here if you prefer : https://github.com/magic-research/magic-animate/issues/44

使用git lfs clone下载时报下面的错误，怎么解决？
git lfs clone https://huggingface.co/zcxu-eric/MagicAnimate
WARNING: git lfs clone is deprecated and will not be updated
with new flags from git clone
git clone has been updated in upstream Git to have comparable
speeds to git lfs clone.
Cloning into 'MagicAnimate'...
fatal: unable to access 'https://huggingface.co/zcxu-eric/MagicAnimate/': Failed to connect to huggingface.co port 443 after 21102 ms: Couldn't connect to server
Error(s) during clone:
git clone failed: exit status 128

full tutorial here : https://youtu.be/HeXknItbMM8


Replace deprecated

Just replacing deprecated initialize_all_variables -> global_variables_initializer

cc @andresusanopinto @arnoegw

Thanks @lc0 ! This will be merged upstream.


Added Gesture Based Volume Control Project

Pull Request for PyVerse 💡
Issue Title : Gesture-Based Volume Control

Info about the related issue (Aim of the project) : The aim of this project is to develop a system that allows users to control the volume of a device using hand gestures made with the right hand’s index finger and thumb.
Name: Rohit Motwani
GitHub ID: Rohit131313
Email ID<EMAIL_ADDRESS>Idenitfy yourself: (Mention in which program you are contributing in. Eg. For a WoB 2024 participant it's, WoB Participant) Contributor at GSSoC-extd and Hacktoberfest

Closes: #288
Describe the add-ons or changes you've made 📃
I have developed a Gesture-Based Volume Control system that enables touchless volume control using hand gesture recognition. The key features include:

Gesture Recognition: Detects the thumb and index finger movements.
Real-Time Processing: Processes the video feed in real-time to track the gestures.
Volume Mapping: Maps the gestures (pinching or separation) to volume control functionality.

Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙
The Gesture-Based Volume Control project can be tested locally through the following steps:

Clone the Repository:
After cloning the repository using:
git clone https://github.com/UTSAVS26/PyVerse.git

Navigate to the Data_Science/Gesture_Based_Volume_Control directory to set up the environment.

Install Dependencies:
Install the necessary dependencies using the command:
pip install -r requirements.txt

Run the Project:
Execute the project with:
python VolumeHandTracking.py

This will launch the gesture-based volume control application, which processes real-time video feed to detect hand gestures.

Test Gesture Controls:

Volume Control: Using the right hand’s thumb and index finger, the volume can be controlled by pinching or separating the fingers. Pinching reduces the volume, while separating increases it.
Real-Time Response: Tested how the system tracks gestures in real-time and adjusts the volume accordingly.

These steps confirm that the application works as intended, with accurate hand gesture recognition and smooth volume control based on thumb and index finger movements.
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

@Rohit131313 there is some merge conflicts please sync your repo.

Sir, please merge my pull request as it has been pending for quite some time. I have noticed that you have merged the PRs of others on the same project, even though they opened their issues after mine. The conflicts in my PR regarding the repo-structure.md and repo_structure.txt files seem to be due to the merging of another PR that did not update these files properly or changed them entirely.


Fixed padding for left side on mobile for danger card

Fixed the missing padding on the mobile for the danger card. (issue: https://github.com/appwrite/console/issues/1086)

@stnguyen90, you can check again now. I updated the branch.


Howto: Chopping up very large image into small enough tiles

I have a HD image (4k by 3k) and would like to get all faces in it.  The faces are very small in the distance, so I assume I have to tile the image into overlapping tiles of reasonable (500x500?) size, run the detector on each tile, handle overlaps, recombine into one superset.
Has anyone else already done this?  Or, are there better ways to handle larger images?

Any rough upper bounds (image size) to lower bounds (pixels of face size) that are likely to need tiling?

Not sure actually, it depends on the size of the faces appearing in an image relatively to the image size, since the images will be scaled down to the network input size.
You would have to try out what works for you.

Sorry, I didn't get the question. Do you mean shifting a Rect? You could simply create a new one new faceapi.Rect(oldRect.x + offsetX, oldRect.y + offsetY, oldRect.width, oldRect.height).
Or create a PR / issue at tfjs-image-recognition-base to implement a simple shift method in the Box class.

Clarification of my question: Given that I have an const fullFaceDescriptions:Array<FullFaceDescription>, and need to shift every det inside fullFaceDescriptions by some (x,y) amount - is there an easy way to do it?
There is the example's const detectionsForSize = detections.map(det => det.forSize(input.width, input.height)) that scales all the detections, I'm assuming a shift would be similar.
Where I get a bit tripped up is what I need to shift, vs. what is relative, across

fd.detection
fd.landmarks

Because landmarks code mentions "unshiftedLandmarks", and "IFaceLandmarks" has "shift: Point" and "shiftBy" - which makes me think some parts of it are relative, or shift-able, and I'd be reinventing if I stumbled in blindly and tried to make a new det.forShift()

Ahh I see. So the the face landmark positions are relative to the bounding box of the rectangle, since you predict them on the image patch extracted from the bounding box. Thus I introduced a shift for the landmark classes for the ease of drawing them, because they have to be shifted by the bounding box position in order to retreive the positions in the source image.
So actually what you would have to do, is to calculate the correct bounding box for each fd.detection, e.g. shift them by the (x, y) offset of the image patch they have been detected for. Afterwards you would simply have to shift all landmarks by the shift of their bounding box (notice shiftBy(x, y) should actually be named "setShift", since it sets an offset for the relative landmarks, rather then adding an offset for every call).
Unfortunately currently the utility is missing to easily shift instances of FaceDetection or the utility classes, so you would have to reconstruct these objects manually.

Understood.
I may hack my local script and toss in a "tileLocation=(x,y)" for every detection, because I'd rather spend the time figuring out how to deduplicate across tiles rather than how to correctly shift detections.
for (let imgNum = 0; imgNum < IMAGES.length; imgNum++) {
    const imgName = IMAGES[imgNum];
    console.groupCollapsed(`Image:${imgName}`);
    asyncSrc(input, `images/${imgName}`);
    const detections = [];

    for (let x = 0; x < input.width - (TILE_DIMENSION / 2); x += (TILE_DIMENSION / 2)) {
        for (let y = 0; y < input.height - (TILE_DIMENSION / 2); y += (TILE_DIMENSION / 2)) {
            const tileLocation = {'x': x, 'y': y};
            drawTile(input, smallCanvas, tileLocation.x, tileLocation.y, TILE_DIMENSION, TILE_DIMENSION);

            let fullFaceDescriptions = await faceapi.detectAllFaces(
                smallCanvas,
                new faceapi.SsdMobilenetv1Options({maxResults: 100})
            ).withFaceLandmarks().withFaceDescriptors();
            console.log(`${imgName} (${tileLocation.x},${tileLocation.y}) detected faces:${fullFaceDescriptions.length}`);
            fullFaceDescriptions.forEach(ffd => ffd.tileLocation = tileLocation); // *** THIS PART
            detections.push(...fullFaceDescriptions);
        }
    }
    console.log(`${imgName} saving ${detections.length} detections.`);
    await set(input.src.substring(input.src.lastIndexOf('/') + 1), detections);
    console.groupEnd();
}


Union doesn't merge objects properties' type

TypeScript Version:
This is reproducible both in VSCode 1.0 and in the playground (https://www.typescriptlang.org/play/#src=...)
Code
interface A {
	x: number;
}

interface B {
	x: string;
}

const c: A | B = { x: 1 || "a" };

const d: A | B = { x: 1 } || { x: "a" };

Expected behavior:
Both c and d should be fine
Actual behavior:
TypeScript reports this error for the const c: A | B = { x: 1 || "a" }; statement:
Type '{ x: number | string; }' is not assignable to type 'A | B'.
 Type '{ x: number | string; }' is not assignable to type 'B'.
   Types of property 'x' are incompatible.
     Type 'number | string' is not assignable to type 'string'.
       Type 'number' is not assignable to type 'string'.

{ x: number | string }; is neither A nor B. it becomes easier to see that with more than one property, e.g.

interface A {
    x: number;
    y:  number;
}

interface B {
    x: string;
    y: string;
}

const c: A | B = { x: 1 || "a" , y :  2 || "b"};

I should add that we've gotten this request around single-property types a bunch. It turns out to be very difficult to detect all of these cases properly in the compiler.

@mhegazy Thank you, now I see why the current behavior is correct


AdminController is not published??

AdminController is not published.how to publish it???

Why you need it? you can create your own controller and then just use the middleware of this package.


pip install dependency mathutils fails

Hi!
I now try the tool in another environment and after running
pip install Flask ifcopenshell pyproj pint numpy scipy pandas shapely

got:
  src/mathutils/mathutils.c: In function ‘mathutils_array_hash’:
      src/mathutils/mathutils.c:79:24: error: incompatible type for argument 1 of ‘_Py_HashDouble’
         79 |     y = _Py_HashDouble(NULL, (double)(array[i++]));
            |                        ^~~~
            |                        |
            |                        void *
      In file included from /home/nata/anaconda3/include/python3.9/Python.h:87,
                       from src/mathutils/mathutils.c:7:
      /home/nata/anaconda3/include/python3.9/pyhash.h:10:38: note: expected ‘double’ but argument is of type ‘void *’
         10 | PyAPI_FUNC(Py_hash_t) _Py_HashDouble(double);
            |                                      ^~~~~~
      src/mathutils/mathutils.c:79:9: error: too many arguments to function ‘_Py_HashDouble’
         79 |     y = _Py_HashDouble(NULL, (double)(array[i++]));
            |         ^~~~~~~~~~~~~~
      In file included from /home/nata/anaconda3/include/python3.9/Python.h:87,
                       from src/mathutils/mathutils.c:7:
      /home/nata/anaconda3/include/python3.9/pyhash.h:10:23: note: declared here
         10 | PyAPI_FUNC(Py_hash_t) _Py_HashDouble(double);
            |                       ^~~~~~~~~~~~~~
      error: command '/usr/bin/gcc' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: legacy-install-failure

× Encountered error while trying to install package.
╰─> mathutils

note: This is an issue with the package mentioned above, not pip.
hint: See above for output from the failure.

Python 3.9.13
OS UBUNTU Jammy 22.04

The error message shows there's a problem with the mathutils package during installation, specifically due to a function call mismatch in its C code. This issue isn't related to pip but rather with the mathutils package's compatibility with your system.
If you only want to use the tool, consider accessing it through the website instead of installing it: https://ifcgref.bk.tudelft.nl/


Help send button

hello i have wanted to use your library to send messages with buttons and to send mp3 audios and i have not been able to do it, could you help me with an example?

You cant send button or list, are deprecated


How to import linq-es2015?

in package.json
"dependencies": {
...
"linq-es2015": "2.4.4",
...
in systemjs.config.js
var map = {
...
'linq-es2015': 'node_modules/linq-es2015'
...
var packages = {
...
'linq-es2015': { main: 'index.js', defaultExtension: 'js' }
...
in gulpfile.js
gulp.src('./node_modules/linq-es2015/**/*.js').pipe(gulp.dest('./wwwroot/node_modules/linq-es2015'));
then in xxx.ts file:
import * as Enumerable from "linq-es2015";
ngOnInit() {
var count = Enumerable.asEnumerable([1, 2, 3, 4, 5, 6, 7, 8, 9, 10])
.Where(a => a % 2 == 1)
.Count();
	alert(count);

...
but when run in chrome ,
Error: XHR error (404 Not Found) ...
Error loading http://localhost:5000/node_modules/linq-es2015/index.js as "linq-es2015" from http://localhost:5000/app/xxxx.js
can anyone help me?
why error happied?

Why don't you use one of the provided examples?
To be used in chrome linq-es2015 has to be processed by browserify utility. There are sample project here and here.

Thanks for your response. But the error still happen.

@richerp can you post html page here or point me to your code on github?

Could you tell me your email ? I send these code to you?

This vs project has many files.

Create repo here, on GitHub. Put that page there and send me a link...

https://github.com/richerp/WebApp
unzip WebApp.zip ,open with vs2015, run it . in file App/app.ts .

Please look at this sample for correct configuration.
In short, you need to add mapping for the module to be correctly referenced by the loader. For list of changes you  need to make look at this commit

Thank you very much.


Typo in using oauthConfig variable

in line 111, variable name config was used to capture response from adal.NewOAuthConfig() which should be changed to oauthConfig.
Thanks you for your contribution to the Azure-SDK-for-Go! We will triage and review it as quickly as we can.
As part of your submission, please make sure that you can make the following assertions:

[x] I'm not making changes to Auto-Generated files which will just get erased next time there's a release.

If that's what you want to do, consider making a contribution here: https://github.com/Azure/autorest.go

[x] I've tested my changes, adding unit tests where applicable.
[ ] I've added Apache 2.0 Headers to the top of any new source files.
[ ] I'm submitting this PR to the dev branch, or I'm fixing a bug that warrants its own release and I'm targeting the master branch.
[x] If I'm targeting the master branch, I've also added a note to CHANGELOG.md.
[x] I've mentioned any relevant open issues in this PR, making clear the context for the contribution.

Hi @vishal7201 we updated the README to showcase how to use the new auth helper methods.
Given this, the PR is not relevant anymore so I will close it.
Please have a look and let us know if you see any other issue.


Consider porting cake to coreclr/corefx

One benefit will be that this will make cake portable without requiring global .net or mono. Another benefit is since ms uses mono as a prove of concept and not as a supported target , ms projects that would have used cake are not using it like https://github.com/dotnet/cli/issues/270
Even if we don't consider the above mention scenario still it will be useful to target coreclr which looks like to be the .net target of the future.

@shahid-pk agree long term CoreCLR makes perfect sense, added some comments to
dotnet/cli#270
Main hurdle is CoreCLR isn't currently an supported platform by the Microsoft .NET Compiler Platform ("Roslyn") CSharp scripting package , and that's kinda a deal breaker :wink:
Cake's already 96,54% CoreCLR compatible, so it's more "refactor tweaks" than a rewrite once we get the dependencies in place.
If we were to remove some dependencies more code would likely have to be rewritten or replaced by other dependencies, but that might make perfect sense long term once we analyzed more deeply what's involved in a CoreCLR port.
I might be wrong but reading issues like NuGet/Home#1870 by @ferventcoder makes me think CoreCLR / vNext still is a bit of a moving target, needing some stabilization before 3rd party easily can take a dependency.
That said the promise of CoreCLR really makes it an attractive target to port to and might totally justify some early adopter pain :wink:

How does one determine compat? I should go RTFM somewhere.

@ferventcoder  The .Net portability analyzer Visual Studio extension is a good starting point.
https://visualstudiogallery.msdn.microsoft.com/1177943e-cfb7-4822-a8a6-e56c7905292b

This issue has been superseded by this one: https://github.com/cake-build/cake/issues/1015
Going to close this one.


[Bug]:

Version of the software
0.1.21
Operating system
Windows 10
Browser
Edge (Chrome) Last version
What happened?
Var (Here is: ) is not replaced by it's value inside Headers.
(See the attached lolicode)
The request is then sent with User-Agent: <randomUserAgentOutput> without any replacement by the variable.
Relevant LoliCode if needed
BLOCK:RandomUserAgent
  => VAR @randomUserAgentOutput
ENDBLOCK

BLOCK:HttpRequest
  url = "URL"
  customHeaders = {("User-Agent", "<randomUserAgentOutput>"), ("Pragma", "no-cache"), ("Accept", "*/*"), ("Accept-Language", "en-US,en;q=0.8")}
  TYPE:STANDARD
  $""
  "text/html; charset=UTF-8"
ENDBLOCK

This is not a bug, you have to use interpolated mode. In stacker, click on the button next to the field until the background of the textbox turns dark green. In lolicode simply add a $ at the start of the value of the dictionary.
BLOCK:RandomUserAgent
  => VAR @randomUserAgentOutput
ENDBLOCK

BLOCK:HttpRequest
  url = "URL"
  customHeaders = ${("User-Agent", "<randomUserAgentOutput>"), ("Pragma", "no-cache"), ("Accept", "*/*"), ("Accept-Language", "en-US,en;q=0.8")}
  TYPE:STANDARD
  $""
  "text/html; charset=UTF-8"
ENDBLOCK


Identify plausible security vulnerabilities from unsanitary input

Look into areas where processing user input is insecure.

I'm going to hold off on this for now, for two major reasons. First, I question the ethics of discussing potential exploit paths, particularly for undocumented or unreported vulnerabilities. Second, this matter will be discussed to some detail in the milestone documentation, completing the objective would require more time than is available.

Closing this as unlikely to be investigated in the future.


Reflects/redirects

Skills that should reflect or redirect effects are not fully functional. This is partially due to the problem of whether reflected skills count as "new" or not, and how to indicate such distinctions in the client.

The engine itself seems to be going smoothly, so I'm closing this issue and moving the broader issue of indicating distinction and consequences to #35.


Throwing NPE in android M AOSP

Hi, I have integrated an app in android M AOSP and have used butterknife library 7.0.1. I have enabled proguard and have used the rules that is mentioned in the butterknife site. But still views are not getting initialized and on calling any method it throws NPE. The following is the proguard rules:
`-keep class butterknife.** { ; }
-dontwarn butterknife.internal.*
-keep class **$$ViewBinder { *; }
-keepnames class * {
@butterknife.Bind *;
}
-keepclasseswithmembernames class * {
@butterknife.* ;
}
-keepclasseswithmembernames class * {
@butterknife.* ;
}
`
I have two classes.
public class ErrorLayoutViewHolder extends ViewHolder {

    @Bind(R.id.error_content_container)
    View errorContentLayout;
    @Bind(R.id.error_message)
    TextView errorMessage;
    @Bind(R.id.retry_button)
    View retryButton;

    public ErrorLayoutViewHolder(final View itemView, View.OnClickListener retryListener) {
        super(itemView);
        retryButton.setOnClickListener(retryListener);
        errorContentLayout.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                return;
            }
        });
    }
}

public class ViewHolder {
    final public View itemView;

    public ViewHolder(View itemView) {
        this.itemView = itemView;
        itemView.setTag(this);
        ButterKnife.bind(this, itemView);
    }
}

The line  retryButton.setOnClickListener(retryListener); throws NPE.
Can anyone suggest me what I am doing wrong? It is not working only in android M AOSP.

Can you debug what happens at runtime? ButterKnife will let you step into the generated code when it's doing binding.

Recently I found that Jack compiler is used and that is why it is not working. But I still haven't found the solution.

I have created a small app to test butterknife. I have enabled jack by using "useJack true" in gradle because in AOSP M, jack compiler is the default. The following are the logs:
03-07 13:18:51.264 10277-10277/? D/ButterKnife: Looking up view binder for bktest.binay.com.butterknifetest.MainActivity
03-07 13:18:51.265 10277-10277/? D/ButterKnife: Not found. Trying superclass android.app.Activity
03-07 13:18:51.265 10277-10277/? D/ButterKnife: MISS: Reached framework class. Abandoning search.

I tried Android N with Java8 recently and met NPE problem if Jack is enabled.

Jack doesn't support annotation processors yet. Once it does the library will start working. You can't use Jack if you want to use this library.


Fix broken HTML in Add a dashboard widget

Update HTML anchors to use a consistent id like the others.
Fix invalid self-closing anchor element.
Remove some line trailing whitespace.

Issue is visible in Part 1, Step 3 on https://learn.microsoft.com/en-us/azure/devops/extend/develop/add-dashboard-widget?toc=%2Fazure%2Fdevops%2Fmarketplace-extensibility%2Ftoc.json&view=azure-devops#step-2-your-html-page---hello-worldhtml (scroll down a little).

@microsoft-github-policy-service agree

@chcomley
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


openjdk test pipeline failing on test-azure-win2012r2-x64-1

After tests have run (successfully) on test-azure-win2012r2-x64-1, the Jenkins job fails with the messages:
[Pipeline] junit
18:14:18  Recording test results
18:14:20  Remote call on JNLP4-connect connection from <IP_ADDRESS>/<IP_ADDRESS>:52102 failed

e.g. see https://ci.adoptopenjdk.net/job/openjdk8_j9_openjdktest_x86-64_windows/294/console

Have bounced the jenkins agent - let me know if that resolves it ... It looks like the sort of error that might be cured by this - post any new job link if it shows up again.

It's running again


The table in the last section is not formatted properly

The markdown table under section: https://docs.microsoft.com/en-us/style-guide/punctuation/dashes-hyphens/hyphens#capitalization-in-hyphenated-compound-words is broken.

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

ID: ddbd6c9a-7000-1b08-7b83-e03f2a7781ff
Version Independent ID: 6a183599-bb95-007c-41e4-308605df5638
Content: Hyphens - Microsoft Style Guide
Content Source: styleguide/punctuation/dashes-hyphens/hyphens.md
Product: non-product-specific
GitHub Login: @pallep
Microsoft Alias: pallep


layout/starknet with keccak

starknet_with_keccak layout not runnable in github action (due to resource consumption) suppressing it for now


Translation issue in flexbox documentation

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details
There is a problem in the following code example. the "flex-end" property appears to have been translated to "flex-end"

https://developer.mozilla.org/es/docs/Learn/CSS/CSS_layout/Flexbox

Thanks for reporting. I've fixed this in the source, and the final publication should update soon.


Add custom query options for includeHiddenFolders and includeHiddenMessages

Closes https://github.com/microsoftgraph/msgraph-sdk-dotnet/issues/1814
Closes https://github.com/microsoftgraph/msgraph-sdk-dotnet/issues/1680
Adds a CustomQueryOptionsTemplate to enable adding custom query option to various navigation properties as needed and adds includeHiddenFolders and  includeHiddenMessages query options for relevant endpoints
Ref - https://learn.microsoft.com/en-us/graph/api/user-list-mailfolders?view=graph-rest-1.0&tabs=csharp

@baywet Just to confirm, do you mean the annotations to mailFolders and messages in a group? According to the docs and the metadata, there currently are no such paths in a group. These accessing these collections are tied to the user.
https://learn.microsoft.com/en-us/graph/api/user-list-messages?view=graph-rest-1.0&tabs=http
https://learn.microsoft.com/en-us/graph/api/user-list-mailfolders?view=graph-rest-1.0&tabs=csharp

Yeah that's what I meant but group messages are conversations. Even though under the hood they are technically the same.
https://learn.microsoft.com/en-us/graph/api/resources/group?view=graph-rest-1.0

@baywet @andrueastman Could you please explain what are hidden messages? There is no documentation for includeHiddenMessages query parameter.


Downgrade concurrency-related function type errors in existing code

When in "existing" Swift code that is Swift 5.x and has not adopted
concurrency, allow mismatches in function types that only involve
ABI-neutral concurrency changes (e.g., adding @Sendable or removing
a global actor) by downgrading the diagnostic to a warning. This
improves the story for incremental adoption of concurrency in an
existing code base.
As part of this, generalize the facility for downgrading an error to a
warning when performing diagnostics in the constraint solver, using the
new diagnostic behavior limits rather than duplicating diagnostics.

@swift-ci please smoke test

@swift-ci please smoke test macOS

I'm going to merge this (because there are important semi-related PRs coming on top of it), and come back to the diagnostic tweak afterward


Add error handling

I got panics because I didn't have docker installed and went a bit overboard debugging that.

Should usage just be in the README?
I'll totally admit that the current README contents don't necessarily belong in a README, and in any case it is mostly subsumed by the website I'm running.

We can move it there, but yea, I didn't know what the plan for the README was

mv to a file with a mildly better name would be a good start

If you merge/rebase, this should pick up the CI and I think it will fail the cargo fmt --check step.

do you want me to push my current crates.json, too? or do you prefer running it on your side?

Keep that out of the PR for now. It should probably be .gitignored or sorted by crate name so that it doesn't generate huge diffs. Currently a huge amount of churn is created by the sort changing a bit, but I wouldn't mind seeing a diff when a crate status or latest version changes.
Or become a real database and be stored in LFS.

these are some real diffs from my current state
-{"name":"insta","recent_downloads":293311,"version":"1.13.0","status":{"UB":{"cause":"","status":""}}}
+{"name":"insta","recent_downloads":293311,"version":"1.13.0","status":{"UB":{"cause":[{"kind":"int-to-ptr cast","source_crate":null}],"status":""}},"time":99}
-{"name":"opentelemetry-http","recent_downloads":289922,"version":"0.6.0","status":{"UB":{"cause":"","status":""}}}
+{"name":"opentelemetry-http","recent_downloads":289922,"version":"0.6.0","status":{"UB":{"cause":[{"kind":"uninitialized memory","source_crate":"http-0.2.6"}],"status":""}},"time":99}

Ah, that too. So I'll go with the "I prefer running it on my side" for the moment. The one I have running currently says 7663 insertions(+), 7663 deletions(-), it's almost done all 10k crates. I'll merge it in when it's done and we'll go from there.

Oh :facepalm: I should have looked over the diff first.
Yes, since you're adding a field you should add yours to the file. I've done some unholy munging to bring things back in line already, I'll merge my run into whatever you have if it needs.

It looks to me like there's no rendering at all happening at the moment. I'm not entirely sure why, but as mentioned before I'm not opposed to factoring it out to a greater degree, which might obviate subtle locking behavior. rename is atomic on POSIX, thus the writing to a dotfile then renaming.

It looks to me like there's no rendering at all happening at the moment.

I think I made it wait for locks so much it just never progressed.
It's a separate tool now, which works great!

Hm, I just tried to run render while the main program is running and I get this shockingly unhelpful error:
$ cargo r --release --bin render
    Finished release [optimized + debuginfo] target(s) in 0.18s
     Running `target/release/render`
Error:
   0: No such file or directory (os error 2)

Location:
   /rustc/db9d1b20bba1968c1ec1fc49616d4742c1725b4b/library/core/src/result.rs:1911

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
                                ⋮ 3 frames hidden ⋮
   4: render::main::h56946a33b468f8d6
      at /home/ec2-user/miri-tools/src/bin/render.rs:<unknown line>
   5: core::ops::function::FnOnce::call_once::ha9c8b272c363af4f
      at /rustc/db9d1b20bba1968c1ec1fc49616d4742c1725b4b/library/core/src/ops/function.rs:227
                                ⋮ 14 frames hidden ⋮

Run with COLORBT_SHOW_HIDDEN=1 environment variable to disable frame filtering.
Run with RUST_BACKTRACE=full to include source snippets.

I can't figure out how to get a line number in src/bin/render.rs out of the error. I tried debug = 2 and -Cforce-unwind-tables=yes.
But moreover, I don't think there's any need to access crates.json. The render program can just... walk the logs/ directory right? All the information is in the crate logs, I don't think we actually need the crates.json file at all.

All the information is in the crate logs, I don't think we actually need the crates.json file at all.

Yes, I independently came to a similar conclusion.
Technically the json file could just contain crate name and version and nothing else. The main program uses that to find out what to look at at all and everything else processes the logs directory

I can't figure out how to get a line number in src/bin/render.rs out of the error. I tried debug = 2 and -Cforce-unwind-tables=yes.

Did you try running without --release?

I'll get to this on the 25th, won't be on a computer until then

Did you try running without --release?

Without release I get the line number.

I'll get to this on the 25th, won't be on a computer until then

Then I'll take care of this, enjoy your vacation :)


improve logging of FileFetcher

Pull Request Test Coverage Report for Build 5593

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 56.126%

Totals

Change from base Build 5592:
0.01%

Covered Lines:
15373

Relevant Lines:
27390

💛  - Coveralls


[SPARK-10582][Yarn][Core] Fix AM failure situation for dynamic allocation

Because of AM failure, the target executor number between driver and AM will be different, which will lead to unexpected behavior in dynamic allocation. So when AM is re-registered with driver, state in ExecutorAllocationManager and CoarseGrainedSchedulerBacked should be reset.
This issue is originally addressed in #8737 , here re-opened again. Thanks a lot @KaiXinXiaoLei for finding this issue.
@andrewor14 and @vanzin would you please help to review this, thanks a lot.

Test build #46674 has started for PR 9963 at commit 1f92d27.

Test build #46674 has finished for PR 9963 at commit 1f92d27.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class ReregisterClusterManager(am: RpcEndpointRef) extends CoarseGrainedClusterMessage\n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46674/
Test FAILed.

retest this please

Test build #46695 has started for PR 9963 at commit 1f92d27.

Test build #46695 has finished for PR 9963 at commit 1f92d27.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class ReregisterClusterManager(am: RpcEndpointRef) extends CoarseGrainedClusterMessage\n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/46695/
Test PASSed.

Test build #47051 has started for PR 9963 at commit 4e413b1.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47049/
Test FAILed.

Test build #47051 has finished for PR 9963 at commit 4e413b1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47051/
Test PASSed.

Test build #47191 has started for PR 9963 at commit b432968.

Test build #47191 has finished for PR 9963 at commit b432968.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47191/
Test PASSed.

Hi @vanzin , from my test and understanding so far, I think calling reset() on the first registration should be OK. But here I still don't change to that way, for me I prefer to add a explicit flag to mention whether SchedulerBackend should be reset or not. Please review and suggest, thanks a lot.

Test build #47256 has started for PR 9963 at commit 0e1d796.

Test build #47256 has finished for PR 9963 at commit 0e1d796.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/47256/
Test PASSed.

Merged build finished. Test PASSed.


Performance improvement of battery segment on OSX

I reworked the battery segment for some performance. Mainly this performance is better by avoiding ruby for simple calculations and pulling the data in over and over.
IMHO the linux part could need some more structure as well. I'll do that if I have more time.
/cc @rjorgenson @tritlo

I made a new branch on top of this one, where I added the concept of "states" that a battery can have. This makes the code much clearer IMHO. In there I remove the color-only icon version. I hope that is acceptable for you. Just wanted to discuss that beforehand with you @rjorgenson and @tritlo.
What do you think? Is the code really clearer, or is this just my perception? Do we need the color-only version here as well?
Thanks for your feedback.

I think this is much clearer. Am I correct in reading that this displays the whole segment in the same color?
Additionally:
Since you're already reworking, could you maybe add a "non-verbose" setting? Where only the icon is displayed?

The code looks good to me. I don't like the idea of having the whole string be the color of the battery indicator, but if I'm alone in that then there's no need to have it in the project.

It looks like @Tritlo's request for a non-verbose mode has been incorporated.
What do you think of @rjorgenson's feedback regarding the color, @dritter?

@bhilburn It is a perfectly valid request. I am working on it. :)

Okay. I tried different things on that and came to the conclusion that we don't need an extra implementation.
But first, here is what I tried:

Adding an extra switch to the battery segment. That seems obvious and easy, but I didn't like it very much, because it would make the battery segment quite special over the other ones (all other segments are in one color). Here is the code:

  ...
  # prepare string
  local message="$(print_icon 'BATTERY_ICON' $current_state)"
  if [[ "$POWERLEVEL9K_BATTERY_COLORIZE_ICON_ONLY" == true ]]; then
    message="$message%f%F{$DEFAULT_COLOR_INVERTED}"
  fi

  # Default behavior: Be verbose!
  set_default POWERLEVEL9K_BATTERY_VERBOSE true
  ...

The second try was to introduce "states" of a segment to the print_icon function. In that case we would check if the user defined a variable named like the segment and state. That would add the flexibility to have different icons for the different states. Here is a patch of that code.

The third iteration was to hack the icon variable. Despite it feels like exploiting some SQL injection, it works like a charm. And if we don't start some wild credit card payment stuff in our theme, I would like to leave this hackish workaround there. @rjorgenson try to set a special icon variable POWERLEVEL9K_BATTERY_ICON=$'\UE894'"%F{white}". What this variable does is: print the icon and start printing all text after it in white. The variable gets called when the battery segment is rendered and voilá we have just the icon in the right color.
That should do it. :wink:

Hmm, while that would work, I don't see why we can't do it the way it was done before? Just check for an explicit foreground color, and if it exists, apply it, if not, do nothing and it will inherit the battery color.
[[ -z $POWERLEVEL9K_BATTERY_FOREGROUND ]] && local fg_color="" || local fg_color="%F{$POWERLEVEL9K_BATTERY_FOREGROUND}"
that bit there sets the fgcolor variable to either nothing, or to a user defined color. just include $fgcolor in the output after the battery icon and if the user sets it then it will be applied otherwise whatever color the icon is set to will be inherited.

@rjorgenson Hmm. That trick only worked well when the segment had only one state and you were rebuilding the general color overriding mechanism. Now users could override colors for all states, and we would have to cover all that. To be honest, I think this would make the segment quite complicated.
But I see that the idea of coloring the icons different from the text is a good one. But I would like to have a more general solution (which is not the workaround I mentioned). I thought the right could be the print_icon function. Thinking about it again, why not setting a global COLORIZE_ICON_ONLY variable which leads to ending the foreground color after the icon was printed. That would probably work for all segments..

@dritter Wouldn't that only reset the color to your terminals default, instead of being what the foreground color is set to? Maybe if you thought about it as overriding the color of the icon output with the state of the battery instead of overriding the text output color? That was my original goal when I wrote it, to fit in with the way the rest of the segments handle color, but enable the battery icon color to be set based on the state. The only other way I could think of to do it was to build the option into the print_icon function, which would have made it more general and usable by all segments, but it would then have to keep track somehow of the color that was set before it to set it back to that after printing the icon and I couldn't find a good way to do that. Any thoughts on that approach?

@rjorgenson Yes, it would reset to the terminals default.  Currently the icons get printed at the very beginning or the very end of each segment (with the exception of the vcs segment). So it would always be a game of "disable the color until or after this icon".. But maybe I am missing a point.

The only other way I could think of to do it was to build the option into the print_icon function, which would have made it more general and usable by all segments, but it would then have to keep track somehow of the color that was set before it to set it back to that after printing the icon and I couldn't find a good way to do that. Any thoughts on that approach?

Without the "keeping track of the color" idea (which I don't understand), this is a matter of disabling the foreground color after the icon has been printed, right?
So print_iconin a "first icon then text"-like segment would behave like:

print the icon
disable the foreground color if a colorize icon only variable is present

In a "first text then icon"-like segment, we would need to do more (maybe that is what you meant with "keeping track"):

the outer segment needs to disable the foreground color if a colorize icon only variable is present
we need to provide the color to the print_icon function
print icon

This is a quick shot, but I am sure we will get to a good solution. :)

@dritter So basically my thinking is, say someone want's all the text on their prompt to be blue, but they want their default color to be the standard grey color. If we just reset the color after the icon the text will be the default grey. It would be nice if they could specify the foreground color of the battery icon segment to whatever they wanted and still have the battery icon be colored with the status of the battery.
In terms of "keeping track" I mean we would need to know inside the print_icon function what the color was set to when we called it, so at the end of the icon we could set the color back to what it was before, instead of resetting it to the default. If we could find a way to call print_icon without disrupting the current color, we could theoretically put icons in the segments anywhere we wanted without disrupting any of the previous output. I think that would be the best solution, if it's possible.

@dritter Also I don't think we should be focusing on a COLORIZE_ICON_ONLY type setup, because we still want the ability to colorize both the text -and- the icon, but in this specific case, what the user specifies as their color for the segment should not affect the battery icon, it should always be in one of the states defined.

Hmm. Yes, it might be not the best solution if users want to set the colors of the icon and text individually.
print_icon variant
So, why don't we pass the current foreground color and state to print_icon?
In this case we could

check for a special icon for that state
check for a variable that defines a color for the icon
set the color again, after we printed the icon

So to get the behavior of your example, you need to set:
# Text colors
POWERLEVEL9K_BATTERY_LOW_FOREGROUND='blue'
POWERLEVEL9K_BATTERY_FOREGROUND='blue'
POWERLEVEL9K_BATTERY_CHARGING_FOREGROUND='blue'
POWERLEVEL9K_BATTERY_CHARGED_FOREGROUND='blue'
# Icon colors
POWERLEVEL9K_BATTERY_ICON_LOW_FOREGROUND='red'
POWERLEVEL9K_BATTERY_ICON_FOREGROUND='white'
POWERLEVEL9K_BATTERY_ICON_CHARGING_FOREGROUND='yellow'
POWERLEVEL9K_BATTERY_ICON_CHARGED_FOREGROUND='green'

This is a bit verbose, but flexible.
Another solution could be:
Icon as visual identifier
How about we treat an icon as a visual identifier for the segment. This could be done by passing the icon as a separate parameter to $1_prompt_segment.
In there we could:

Check for a defined variable to change the icon color separately
Position the icon dependent of the element position (at the beginning for left; at the end for right).

This has some drawbacks though:

We need a mechanism for the whitespace around an icon, as some are double width and need more space..
We would eventually need to pass the state of the segment as well to $1_prompt_segment, so that the icon is a flexible as in the above solution.

A good thing about this is that the code would have a better structure and the concept of segments having "state" would become crucial (at least I like it :smile: ).
Btw. I already implemented some of this in my improve_rvm branch.

see #161 for my take on floating point in bash (everyone has "bc" installed, right?) and how I got the power stuff in linux (looks to be similar idea to yours, but I think I'm using a different path under proc - the path I am using has worked for me on every laptop I tried, I think the one you are using might be less universal)
What would be really crazy is to include a "time left" mode as well - I prototyped something like this by keeping some state about the rate of change of the current power level, but it never worked too well so I gave up on it.
Cool changes though!

oh, hmm, looks like your code reads a "time left" out of /proc if it is avaiable, that would probably be less "insane" and more "reliable" =)

@dritter - Looks great! Thanks so much for the improvements!
@terabyte, @rjorgenson, @Tritlo - Thanks for providing excellent feedback and testing for this PR! It is much appreciated.


Support MsgUpdateParams gov proposal (47)

Change param proposals are gone

Resolved in https://github.com/CosmWasm/wasmd/pull/1247


Set the 'X-RateLimit-Reset' header

Set the 'X-RateLimit-Reset' header to the number of seconds until the current throttle expires.
This implements what was proposed here: https://github.com/jbyck/rack-attack-rate-limit/issues/2.

Any news about this PR @davebrace? I'm looking for this kind of feature as well.

@pabuisson No news - waiting for feedback from the project maintainer @jbyck.
We're using a custom fork of this project in production that includes this feature.

OK I'm following this PR closely then... and in case it does not merge soon, I'll do the same as you and use a fork of the project. Maybe we'll get some update from @jbyck !

@pabuisson @davebrace sorry for the lack of communication here. I haven't been monitoring this project very well.
This looks reasonable to me, but the tests aren't failing in Travis.  I'll be happy to merge and make a new release once it goes green.
I actually don't use this myself anymore in any current projects - if either of you are using it, and are interested,  I'd be happy to pass on ownership for someone to maintain more actively.


Error - compendium getEntity is not a function

https://discord.com/channels/170995199584108546/722566083499786352/908358081942020096
https://cdn.discordapp.com/attachments/722566083499786352/908358081254141952/unknown.png

Fixed in v1.2.0


Why not use UUID for PK?

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Now PrimaryKey type is int, for 'drives', 'charges' etc..
When I have deploy  more instance, for example:
On AWS, I run Teslamte sine 2021-2023,
On my NAS, I run Teslate sine 2023-2024.
int PK can't merge those data.
If change to UUID, it can be merge.
Expected Behavior
No response
Steps To Reproduce
No response
Relevant log output
Database issues

Screenshots
No response
Additional data
No response
Type of installation
Docker
Version
v1.24.1

Thanks for reporting.
Can you elaborate on your use case? If you have two instances collecting the same data, why would you want to merge them?

Version
v1.24.1

Even if unrelated, this versions is outdated.

Use of UUIDs everywhere is not going to be a magic bullet that solves all these issues (and may have some other issues I don't want to go into here). For example, you would still end up with two sets of cars, and not sure I see any benefit here over just having two databases.
It is clear however that we really could use some sort of tool that will take two databases, a list of instructions concerning which cars (and maybe other resources) are the same, and output a new merged database.