DSA <IP_ADDRESS> Arrays: Suffix Array - Audio/Video

Description
Explain the construction and applications of Suffix Array.
Important: Do not use assign command for this issue.
Task

[ ] Video
[ ] Audio

Note

While requesting the issue to be assigned, please mention your NAME & BATCH NUMBER.
Kindly refer to Issue Guidelines.
Changes should be made inside the DSA/directory & DSA branch.
Name of the file should be same as the issue.
This issue is only for 'GWOC' contributors of 'DSA' Domain.

Please do assign this to me.

Please do assign this to me.

Okay, send in your full name and DSA batch number. @purohitmansi

Please do assign this to me.

Okay, send in your full name and DSA batch number. @purohitmansi

Name: Mansi Rajpurohit
Batch: 4

Name: Mansi Rajpurohit Batch: 4

Assigning this issue to you, all the best!


pyramid levels in reader api

Feature Description
add some notion of multiresolution levels to the reader api
Use Case
ome-zarr supports multiresolution levels.  I believe there is a flavor of ome-tiff also.  And czi.
Solution
This probably requires some extra discussion to design the api but here is an idea:
Add support for ome-zarr first.  Other readers should just always use level 0 (max resolution).
A simple option seems to be to handle it like scenes, where you call set_multiresolution_level, and get_num_multiresolution_levels.
Dimensions for each multiresolution level will be different, of course.
Alternatives
Perhaps a way to fold it in with dimensions, but that seems less ergonomic.  Each data read request has to be for a particular multiresolution level.
It would also be interesting for the ome-zarr reader to have a chunk api so you can know your requests are at the chunk boundaries.

See relevant issue submitted in aicsimageio


Making MAPIE compatible with Tensorflow

It would be great if MAPIE could work with Tensorflow deep learning models. I have an LSTM model developed in Tensorflow using Keras API which only gives point forecasts. I would like to have conformal prediction intervals for the forecasts. Is it possible to use MAPIE for it?

Hi @nilips70, thank you for your interest in MAPIE. There is one way to make your model compatible with Tensorflow, to do so you need to construct a wrapper around your tf model so that you will have the necessary class attributes and methods to use MAPIE. An exemple is given in this notebook : https://github.com/scikit-learn-contrib/MAPIE/blob/master/notebooks/classification/Cifar10.ipynb


docker and docker-compose configuration (fixes #568)

Story / Bug id: https://github.com/CodeForPoznan/volontulo/issues/481 (see comments: https://github.com/stxnext-csr/volontulo/pull/577)
Reference person: @magul
Description:
Providing docker and docker-compose configuration for development environment.
To setup it:

install docker and docker-compose
execute docker-compose up
point your browser to localhost:8000

New imports / dependencies:

N/A

Unit Tests:

N/A

What tests do I need to run to validate this change:
Procedure described above.

Current coverage is 91.26% (diff: 100%)

Merging #502 into master will not change coverage

@@             master       #502   diff @@
==========================================
  Files            53         53
  Lines          2073       2073
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1892       1892
  Misses          181        181
  Partials          0          0

Powered by Codecov. Last update a323cf0...1e04a0c

@riv2q @tomaszwozniak @krzysztofzuraw please review.

@MarekBleschke the docker/vagrant solution is a result of to much time waisted at February hackathon. It last for 8 hours and few participants have problem with setting up dev environment - I could takie 2-3 hours to set up.
I would say that we have right now two supported options to setup dev environmnet (docker/vagrant) and one manually. We will update README I we will see it this is not too much.

@magul Yes, sounds very reasonably :). Just wondering if in the future one solution will suffice (docker or vagrant). One less thing to support :).
From my side that was just general question, not something to block vagrant or docker solution (or both of them).
I'll try to make some time today to run docker locally and see how it goes, cheers.

@MarekBleschke @riv2q please review


Port-forwarding doesn't work in Binder

The port forwarding doesn't work @binderhub

fixed using nbsessionproxy: https://github.com/4QuantOSS/DashIntro/commit/aaa59fa83b577f2a4351a02e57a7b91ce140e96e


Fix missing comma in cognitive-search-custom-skill-interface.md

@iMicknl : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@gmndrg
Can you review the proposed changes? IMPORTANT: 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"
#assign:@MicrosoftDocs/public-repo-pr-review-team

The following users don't have valid GitHub IDs, or are not collaborators on this repo: microsoftdocs/public-repo-pr-review-team

#sign-off

Hello @iMicknl : Your change proposal has been approved. Thank you so much for your help improving our documentation.

This will be published in our next release of documentation.
#please-close


Write a JavaScript program to find weekend days from number of years

Description
Write a JavaScript program to find weekend days from number of years
Weekend means Saturday & Sunday together. In total, we have 52 weeks in a year. So there are 52 weekends in a year. So, in normal we have 104 Weekend Days.
Input  : 1
Output : 104

Contributed by - @Aranarora

How to contribute

Save the solution in FindWeekendDaysFromNumberOfYears.js file
Add FindWeekendDaysFromNumberOfYears.js file in program/find-weekend-days-from-number-of-years folder

!assign


Correct URLs generated when using "prefix" in rodauth

Hi,
It seems when using prefix "/user" in app/misc/rodauth_main.rb, the routes generated for omniauth add an additional /user prefix and do not work.
The generated routes look like the following:
/user/user/auth/facebook/callback
and I get the following error:
'No route matches [GET] "/user/user/auth/facebook/callback"'
perhaps the prefix is also applying somewhere it should not?

I couldn't reproduce this on the Rails demo app; I set prefix "/user" in rodauth_main.rb, and rodauth.omniauth_callback_path(:github) returned /user/auth/github/callback, not /user/user/auth/github/callback. Here is a self-contained example with plain Rodauth showing this behavior:
require "sequel"
require "roda"
require "omniauth-github"
require "rack/test"

DB = Sequel.sqlite

class App < Roda
  plugin :rodauth do
    enable :omniauth
    omniauth_provider :github, "key", "secret"
    prefix "/user"
  end

  route do |r|
    r.root do
      rodauth.omniauth_callback_path(:github)
    end
  end
end

session = Rack::Test::Session.new(App)
session.get("/").body #=> "/user/auth/github/callback"

strange, you are right.. must be something else in my code. Anyway, I was able to get mine working without the prefix.. I can live without the prefix feature. Thanks for this awesome gem to help explore all the features of rodauth!

I have the same issue with the facebook integration and traced it to the following line in the omniauth-facebook gem:
https://github.com/simi/omniauth-facebook/blob/69872e731e3372cf51b8274ce0048139a8102b54/lib/omniauth/strategies/facebook.rb#L88
From what I can see:
full_host -> scheme + domain + port
script_name -> rodauth prefix
callback_path -> rodauth prefix + callback path
When i give the callback_url option to the facebook provider like this:
omniauth_provider :facebook, "client_id", "secret", scope: "email", callback_url: "http://localhost:3000/sso/auth/facebook/callback"

Everything works correctly.

I tested omniauth-github with the path prefix, and it worked correctly – the prefix wasn't duplicated.
It seems this is a bug in omniauth-facebook. It generates the callback URL from full_host + script_name + callback_path, but OmniAuth's callback_path already adds script_name. It appears this was fixed on master, but a new release of omniauth-facebook hasn't yet gone out.
rodauth-omniauth relies on SCRIPT_NAME env variable for its automatic routing of the OmniAuth app, and it seems some reverse proxy setups also use it. So, for the time being I recommend pulling omniauth-facebook from GitHub.


Typings: camel-cases two attributes, alphabetizes, and adds htmlFor

This PR corrects the case of two properties in the JSX.HTMLAttributes interface in the type declarations. It also moves the for property to the appropriate spot (alphabetically) so that it's easier to locate. Finally, it adds the htmlFor property. I'm not sure whether that last change is desired -- since for is in the type declarations, maybe it's intended that we just use that in Preact rather than htmlFor? -- so I'm open to removing it. I'll just note that, IMO, including it provides more consistency with React's typings, as well as with Preact's decision to allow both class and className.

Coverage remained the same at 100.0% when pulling a7f29c2401438149051ceecef11a64580f17ceb4 on jmrog:typescript-update-htmlattributes into 9001436a453d736267359538e2d822ccf23e3115 on developit:master.

Changes Unknown when pulling 598a0727a1ab33b9ff5b355064a8defd2738aa5f on jmrog:typescript-update-htmlattributes into ** on developit:master**.

Coverage remained the same at 100.0% when pulling c032402efeed9099e1df25c5098b345602a02e72 on jmrog:typescript-update-htmlattributes into f3d0047ce748788fa527fe021ccdc2b375afeec4 on developit:master.

Coverage remained the same at 99.598% when pulling c2292da2e2d11bd708ed15cd2a81966eb0dd76c7 on jmrog:typescript-update-htmlattributes into 6756a9c6f74f24dd19002bfa0dff43a78bb78b5c on developit:master.

@jmrog Great find! Thanks a lot for your PR, much appreciated 🎉


Feat: Error Cause Extractor

:scroll: Description

Relates to #1045
Introduce a way for errors to provide a cause to the SDK, so we can extract it.
:bulb: Motivation and Context

This builds on the discussions in #1045. Instead of returning a list, we just return the cause, making it easier to apply the recursion, as we don't have to deal with lists of lists or injecting extractors in other extractors. It also makes the interface simpler to understand.
So the goal is for users to register the cause extractors for their own application errors, or for any other 3rd party object where they want to extract an error cause. The SDK would then try to flatten errors & then transform with error/stacktrace factories. Event processors can check the throwable list in the event and use the data as needed.
:green_heart: How did you test it?

Unit tests.

:pencil: Checklist

[x] I reviewed submitted code
[x] I added tests to verify changes
[ ] I updated the docs if needed
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps
This is pretty much a small POC. Would be great if you (@marandaneto & @ueman) could give me some feedback on this approach, before I'll continue with the "real" implementation.

@denrase sounds like a good option, nice.
Just to double-check, SentryOptions would have a Map<Type, CauseExtractor> where people can add the type and its extractors, so the SDK would be able to do it automatically while manually or automatically capturing errors, right? I don't think users would need to add event processors for that.

Yea I like this as well :)

Codecov Report

:exclamation: No coverage uploaded for pull request base (v7.0.0@20def04). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             v7.0.0    #1198   +/-   ##
=========================================
  Coverage          ?   76.27%
=========================================
  Files             ?       11
  Lines             ?      333
  Branches          ?        0
=========================================
  Hits              ?      254
  Misses            ?       79
  Partials          ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@marandaneto @ueman I have added a check for circularity and also more unit tests. Think we could do a first review of this approach.

@denrase can we try to migrate this event processor to using this approach and see how this works? if it looks like a friendly API for users.

@denrase can we try to migrate this event processor to using this approach and see how this works? if it looks like a friendly API for users.

I just tried that and it's basically not yet possible because of https://github.com/getsentry/sentry-dart/pull/1198#pullrequestreview-1250412668

@denrase can we try to migrate this event processor to using this approach and see how this works? if it looks like a friendly API for users.

I just tried that and it's basically not yet possible because of #1198 (review)

@denrase worth checking that.

@ueman @marandaneto
There's also currently no way of getting the original throwable object of the inner exceptions. That's still only available for the first one.
That makes it impossible to do any processing for inner exceptions in the event processors, because we can't compare based on the type or the name (because of obfuscation).

To get the original throwable we could run the extractor again on the initial throwable. That way we have them in hand. The list of these and the list of sentry exceptions in the event "should" have the same order, so we could match them with the index.
What I'm not sure about is how to "migrate" the dio event processor though, as it does something different.
If we provide an extractor for the DioError, it would just create an additional SentryException with the current approach, but in the processor there is a lot more going on (removint stacktraces, responses, requests, etc).

Can't we just add the throwable to the SentryException?

Can't we just add the throwable to the SentryException?

yeah, I was thinking about this as well, the throwable does not need to be serialized/deserialized but can be used in order to enrich the event with extra info, when needed.

That would make things easier, whats your take on this @marandaneto ?

@ueman @marandaneto
There's also currently no way of getting the original throwable object of the inner exceptions. That's still only available for the first one.
That makes it impossible to do any processing for inner exceptions in the event processors, because we can't compare based on the type or the name (because of obfuscation).

To get the original throwable we could run the extractor again on the initial throwable. That way we have them in hand. The list of these and the list of sentry exceptions in the event "should" have the same order, so we could match them with the index.
What I'm not sure about is how to "migrate" the dio event processor though, as it does something different.
If we provide an extractor for the DioError, it would just create an additional SentryException with the current approach, but in the processor there is a lot more going on (removing/cleaning stacktraces, adding response/request, etc).

Get your point, but the processor does something that the new approach would do, which is converting the error to a sentry exception, also its stack trace, this at least should be possible.
The other thing is that event processors should have access to the inner causes in a way or order, so maybe inner exceptions are held in the SentryException as suggested by https://github.com/getsentry/sentry-dart/pull/1198#issuecomment-1385053554
This is the moment that the Dio event processor would look into the exception list looking for the Dio error and enriching the event with extra context, I guess that's doable.

@ueman @marandaneto I tried to apply the new approach in the dio event processor in combination with an dio error extractor. WDYT?


Remove Bold Style in Button

Describe the chore
Remove Bold Style in Button
see PR for Screenshots
Would you like to work on the issue?

[x] Yes

Why is it necessary ?

@theLimitBreaker Bold Style in Button is not looking good


Fix remote url loader not working

For some reason the request module is not working properly so I used the native http/https module to load a remote URL.
Also eslint is not happy about some things so I incorporated fix of @christianascone on pull request #728 with this. If there is a cleaner way to merge our pull request, let me know.

pull request irrelavant


Scala support for feather format?

I was trying to find any library to for reading/writing feather format, but could not find any.

Now that Feather V2 (coming in Apache Arrow 0.17.0) uses the Arrow IPC format it would be relatively straightforward to read the files in Scala via the Java API


Add logical and & select on host

This PR add the following:

logical and to the edsl and logical dialect
Add a select operation on host. Currently select a Boolean tensor to select a subset of the array, but it could be extended to work with uint tensors.
An example leveraging the output of a psi library (e.g. PrivateID)

Thanks!


Jira 7.10.1 and Service Desk 3.13.1

Description of the Change
Bump versions to:

Jira: 7.10.1
Service Desk: 3.13.1

Verification Process
docker build -t blacklabelops/jira .

Release notes

Jira: https://confluence.atlassian.com/jirasoftware/issues-resolved-in-7-10-1-952059662.html
Service Desk: https://confluence.atlassian.com/servicedesk/issues-resolved-in-3-13-1-952059632.html

Thanks for the update!


v-bind:src=" ....imgurl..." not added to static/img?

Hi,
When I'm using v-bind:src for images they show up as broken because they are not put in the build folder in /static/img/, as opposed to regular src="". Why is this and how do I fix it? I can't manually add stuff to the build folder because they are overwritten on build.
So to make it clear:
this works:

But this do not:

I have the same issue trying to access data from json files, I cant put them anywhere and access the data it seems? Because it's not in the build?

where do you keep those images before build?
The problem is:

For non-bound src attributes Webpack (or more specifically: html-loader) can read those src paths and copies them from the specified path to the assets directory in the build folder.
For dynamic, bound paths, webpack can't do that, of course.

So you will have to put those images into <your project folder>/static/ instead of placing them somewhere in /src/assets., because /static will be copied into /dist/ as-is.

@LinusBorg I see, thanks for clearing that up for me=)


Use mantine use-focus-trap to auto focus answer fields

https://mantine.dev/hooks/use-focus-trap/

Not necessary


Option to view the date a video was uploaded.

Hi, me again. I think the option to view the date a video was uploaded is key piece of information for most video. Especially when working with many video thumbnails.
I thinks the following area:

Would perfectly fit the uploaded date without making the thumbnail look cluttered.
Thanks for your time.

This would be nice along with potentially having the view count there too. Unfortunately, I can't get that information through the way the plugin normally gets video info. Like in issue #9 I could get that information from the Youtube API, but only if the user has an API token set and I don't think many users do. So, since very few users would be able to use this feature, it's not worth making the plugin more complicated. Thanks for the idea though!


[build] Move io-channels.h back into io target

io-channels.h has a circular dependency with io due to its dependency on io-util.h (parsing the header using bazel is expected to fail accordingly). This means any source file depending on it would need to be in a bazel target depending on io to avoid missing header errors, even if the source file doesn't use io itself. Move it back into io in accordance with IWYU.
- Fix dependencies for io:actor-id, util:duration-exceeded-logger
- Fix duration-exceeded-logger.h appearing in two targets

Builds seem to be passing on the eponymous downstream branch.


Refactoring CMake scripts

Beast needs a refactoring of its CML to be like JSON and URL.
The lib and sources should be in their own project and not have the source tree duplicated for every executable target.

@vinniefalco can we close this in favor of #1816  ?

Yes


Added support for [*] and [#] tags

Aww man, I totally forgot I'd not done this.
Travis is reporting a fail - looks like the last character of the first list item is being skipped. Once that's fixed up, I'll merge it ASAP.
Thanks!

Whoops! Mocha passed on my end. Looks like a discrepancy in the newline handling. I'll throw up a fix ASAP

Fixed! I wasn't accounting for Windows \r\n vs. Unix \n (http://www.cs.toronto.edu/~krueger/csc209h/tut/line-endings.html) I also made the code a bit more elegant and added some fancy regex to do the dirty work to keep more in line with your patterns. Looks like the CI is passing now. Merge away!

Ah, of course - that'd do it!
Excellent work, many thanks.


wsltty: Fix installer

closes #2318
Related:
#8006
https://github.com/mintty/wsltty/issues/304

Hmm, as noted in the linked issue, we should wait for the new release which includes the patched installer.


[Dashboard] Keeping refresh frequency value in Dashboard

ref:

#3860
#5628

Codecov Report

Merging #5741 into master will decrease coverage by 1.46%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #5741      +/-   ##
==========================================
- Coverage   77.66%   76.19%   -1.47%
==========================================
  Files          46       46
  Lines        9257     9258       +1
==========================================
- Hits         7189     7054     -135
- Misses       2068     2204     +136

Impacted Files
Coverage Δ

superset/views/core.py
73.73% <100%> (-0.21%)
:arrow_down:

superset/data/countries.py
0% <0%> (-100%)
:arrow_down:

superset/viz.py
75.25% <0%> (-6.11%)
:arrow_down:

superset/db_engine_specs.py
51.86% <0%> (-2.3%)
:arrow_down:

superset/utils.py
87.88% <0%> (-1.27%)
:arrow_down:

superset/models/core.py
84.76% <0%> (-0.34%)
:arrow_down:

Continue to review full report at Codecov.

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

@graceguo-supercat Could you review this pr

@mistercrunch  Could you review this pr

@williaster @graceguo-supercat

Thanks for adding this feature! Very nice work! 🎊  Please see my comment for a performance suggestion.
And it will be super helpful if you can add a unit test for this new feature.

Hi @zhaoyongjie  @graceguo-supercat any chances to include that change into some release? That would be super helpful!

how long since this PR being merged to the superset docker being updated?

@zhaoyongjie was that working for you? I've applied changes on my instance (based on master, had to do that manually, cuz there was a lot of changes) and after setting interval and hitting refresh (even if I save dashboard and overwrite) it looses refresh interval. Same if I set that value in dashboard metadata - nothing happens :(

Sorry, I have been busy recently. I will complete this week. @wowo

@graceguo-supercat could you review code~~. thanks

@graceguo-supercat , I finished the git rebase master branch,  and let the dashboard auto refresh when loading.

@graceguo-supercat @mistercrunch  please review this PR

any news on this one being released?

@graceguo-supercat hi, guo, please review this PR

Thanks This is very important feature. Any idea on when is this expected to be released.?

this feature should be in release 0.32.

but release 0.32 is a bit abstract :) I've read that superset is undergoing ASF release process adjustment and pretty much no releases are done now.

you can build master / 0.32 branch  for you self :)
On Fri, Apr 5, 2019 at 5:01 PM Wojciech Sznapka<EMAIL_ADDRESS>wrote:

but release 0.32 is a bit abstract :) I've read that superset is
undergoing ASF release process adjustment and pretty much no releases are
done now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-superset/pull/5741#issuecomment-480202326,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7FUlJVIU50ksy796yhYS89Tg8Z_p9pks5vdxDXgaJpZM4WNYw4
.


Updated travis to use Xcode 7.2 and SDK 9.2

Updated travis to use Xcode 7.2 and SDK 9.2

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

So the interop tests fail now, for something most probably unrelated to this change:
subchannel_index.c:59] assertion failed: gpr_tls_get(&subchannel_index_exec_ctx) == 0

Hi Jorge,
There is a major change going to 7.2 - we've gone through some of this troubleshooting previously at the following issue in protobuf:
https://github.com/google/protobuf/issues/1253
Maybe running the build via sudo might help to bypass the security checks in 10.11.
Hope it helps,
Paul

Thanks for the heads-up, Paul. Vasilis is finding the same assert error when running ObjC examples locally (#5367), so it looks to me like this needs a fix in the C library. I'm creating an issue for it.

PR #5390 fixed it, so I'll rerun Travis and hopefully merge now :)

:rocket:

Sure thing Jorge, keeping my fingers crossed :)


Update azure-diagnostics-configure-application-insights.md

Adding json equivalents of xml examples.

@nizarq,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@nizarq : Thanks for your contribution to the Azure documentation! The author, @rboucher, has been notified to review your proposed change.

#sign-off


HDDS-9051. Change the log level when no nodes are available in NetworkTopologyImpl

What changes were proposed in this pull request?
We currently have a log message in NetworkTopologyImpl set at the WARN level for when no datanodes are available.  It is better to set it to DEBUG level, and relying on the INFO message at the end as a summary of what happened.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9051
How was this patch tested?
Only log level change. No testing required.

Can you please specify the INFO message which should suffice instead of the WARN messages?

The method chooseNodeInternal() in which this log is present is finally being called by SCMContainerPlacementRackAware#chooseNode() and SCMContainerPlacementRackScatter#chooseNode() . Both these methods have Log.info() messages to log "No satisfied datanode" being found after maxTries are hit. These INFO logs will serve the purpose of showing us the summary

Thinking about it some more, it's actually hard to guarantee what a future user of the class will do, and the implementation details of this class cannot really rely on that. I think we're better off just bringing down the log level from WARN to INFO.

I think we're better off just bringing down the log level from WARN to INFO.

We want to bring it down from WARN to INFO and not DEBUG?

We want to bring it down from WARN to INFO and not DEBUG?

Yes

@siddhantsangwan thank you for the review! I have changed the log level to INFO now. Could you please approve the workflows if the patch is good to go?

Thanks @Tejaskriya for the patch, @siddhantsangwan, @tanvipenumudy for the review.


Custom form attribute name (instead of ngSubmit)

Hi,
I have a custom directive to validate form before submit (validSubmit instead of ngSubmit), it's possible to customize SUBMIT_ATTR to include arbitrary attribute name?
Thank's so much.

Ng-fab-form listens to the forms submit event, so as far as I can see (haven't looked at the source yet), the attribute name should not matter, as long as the forms submit event is triggered.

Oh. Sorry! Wrong repo. Ähem. No currently there is no way to customize the submit attribute, but the functionality should be easy to implement actually. Not sure when I will find time for this, but managing feel free to submit a PR.


Feat: support get secret from project config for helm chart

Description of your changes
Support get secret from project config for helm chart.

-->
Fixes #
I have:

[ ] Read and followed KubeVela's contribution process.
[ ] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[ ] Run yarn lint to ensure the frontend changes are ready for review.
[ ] Run make reviewableto ensure the server changes are ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

Special notes for your reviewer

@caiqi1111 Let's change the frontend code to fix this issue completely.  The related code path packages/velaux-ui/src/extends/Helm*

@caiqi1111 Please rebase the latest code.

@barnettZQG I rebase the latest code now.

LGTM


Memory leak of UIPointerActions 'uidrag'

I've been wanting to use pts.js to draw interactive shapes. But the use of the "uidrag" action (i.e. for translating the shape) results in slowing down drastically the performances. Is this behaviour intended ? Or have I misused the "on" method of UIDragger ?
Fiddle to reproduce:
https://jsfiddle.net/ygov5x2m/ : click and drag on the shape, and open the console to see the issue.
pts version: 0.8.6
using Chromium 75.0.3770.90

Thanks for reporting @cccdupont . We'll investigate as soon as possible.
@tibotiber if you have a moment, would you be able to help us take a look and see if there's a quick fix? Thanks!

Hi @cccdupont, very good catch, thank you :). And you're using on() properly, although note that you also have onDrag() etc if that's a better fit for you.
So I had a quick look, our issue comes from _holds not begin cleared properly.
_holds is an array of event strings being held, elements are added using hold() which returns the index of the added element (see UI.ts L191), and removed using unhold() which removes the element at a given index (see UI.ts L201). The issue is that UIDragger builds uidrag and uidrop events by holding multiple events, it holds move then drop keeping their indices (see UI.ts L442-L443) then unholds them in the same order (see UI.ts L456-L457) so by the moment drop is removed, it has changed index and remains there. Also UIButton is extended by UIDragger and has its own move event holding which probably adds to the indices confusion.
A fix for this would be to use something less volatile for the "id" of the members of _holds. Or maybe we could simply keep a count of how many live holding requests there are for each event in an object? Something like{ move: 2, up: 0, ... } then hold() would increment and unhold() decrement, and checking if an event is held would be like _holds[key] > 0. There might be better ways.
What do you guys think? I can push a PR if you're good with the proposed solution. I'm open to better alternatives.

Good catch, @tibotiber . Agreed that we should use something less volatile than an array index. How about if we store them in an object, and then use delete key instead of splice( index ) ?
I'm open to any solution that you think it's best. Would greatly appreciate it if you have time to push a PR. Thank you so much @tibotiber 🙏

All good, pushed a PR and it seems to solve the memory leak. I went with your idea @williamngan, but I've used a Map instead of an object so keys can be numbers. There was another element to the leak, we were always holding events and creating events handlers even if they already existed, so I now make sure to hold and handle events only once.

Pretty timely report @cccdupont, I've just met with this in my own app :). You saved me a good amount of debugging hours.

@tibotiber @williamngan thank you for the quick fix !

@cccdupont 0.8.7 is published with the fix. Please give it a try and reopen this if you still encounter issues.
@tibotiber thank you so much!

@williamngan It's working perfectly now, thank you again ! Loving the UIDragger :+1:


Float Equality Comparisons

There are a number of -Wfloat-equal warnings that show up in the library.
../printf/src/printf/printf.c:542:26: warning: comparing floating point with == or != is unsafe [-Wfloat-equal]
  542 |     if (scaled_remainder == rounding_threshold) {
      |

I can prepare a PR that uses epsilon comparisons (the better approach), but first I wanted to check whether or not you think that there is a reason these direct float comparisons are actually valid for some reason.

whether or not you think that there is a reason these direct float comparisons are actually valid for some reason.

They are all valid, and safe, in the context in which they are performed. If you believe some specific comparison is problematic, please reopen and say which, and why.

Cool, if they are all valid, I will prepare a change for the top of the source file that will turn off these warnings for GCC and Clang.

Do you mean with #push and #pop directives? or via the CMakeLists.txt ?


(spanish) 5-Clustering/1-Visualize/assignment.md

Translate file 5-Clustering/1-Visualize/assignment.md to spanish

thanks all!


smoke test slack notification on failure

Jira ticket URL

Just add the ticket number to the end:

https://dfedigital.atlassian.net/browse/TEVA-
Changes in this PR:

Review app deployed to https://teaching-vacancies-review-pr-3475.london.cloudapps.digital


Project updates

Hi, Chris.
This PR provides a basic outline for different improvements we have for Slick built in collaboration with @mgajda. We want to create some momentum for the project and have a couple of websites already in production with Slick setup and our corrections.
This PR includes:

Addition of Serve.hs that include functionality for Preview mode that spins out small server for site observation
Additional integration with our shake-watch project that will be released soon which provides continuous rebuilds on either haskell changes or site changes.
Addition of Build.hs file that includes high-level goodies we created while working on multiple sites
Updates example project

Additional keys
Additional Builder.hs file with everything related to builds and update to the main workflow where shakeRules and builder moved to separate functions for further reuse.

Compacts README.md to concentrate on go-to production details (should be improved in later updates) and updates dependencies.

I would like to get your feedback and establish the path to maintainership so I can push continuous updates and move this cool project further. In case you want to split into multiple smaller PRs we can also arrange that.

It's still work in progress and I'll finalize during next couple days and prepare for review.

Hi! The changes look interesting and good; I'd be interested in looking at getting them merged in. Is there a reason you've closed this issue?
I would prefer smaller PRs yes, I know it's a pain to do afterwards, but this is a large change and to do it "all or nothing" is very inconvenient for me.
Other thoughts:

I notice you've changed the base repository in the package.yaml, are you requesting to move this project into your organization?
This brings on a full webserver and websockets implementation as dependencies, I understand that slick already has a lot of dependencies (having lens and pandoc basically brings on everything under the sun) but for that reason I'm actually looking at reducing the number of dependencies I'm using if possible.  The webserver is really great and I'd still love to provide it somehow, to me it seems as though the auto-updating webserver should be implemented as a supplementary package/plugin which can enhance slick. Can we talk about what would be necessary for that to happen? We can look at exposing some hooks for plugins in Slick if it's necessary for the integration.
This rewinds the version number of the project, I suspect that's just an oversight?

Looking forward to hearing from you.

If we can take the websocket stuff out of the equation, the web server part can be simplified quite drastically. This is what I do in my own static site generator as well: https://github.com/srid/rib/blob/master/src/Rib/Server.hs

@ChrisPenner Hi, Chris. Thank you for feedback.

PR was closed due to the decision to simplify changes and get it more polished before you take a look, my mistake was rushing into making a PR. I anticipate to prepare a new PR in 1-2 days with more cleaner changes, stripped down just to 19 files.

Change of version and base repository my mistake as initially, we wanted to keep the project as a fork but then decided it would be nice to contribute back.

Regarding webserver,  Serve.hs is a standalone module and can be easily moved to slick-serve or something similar.

I rushed to touch base as we have more upcoming features and I know it's difficult to work with big PR.

@srid Server side allows not only a local hosting with http server but also live-reload functionality that's why websockets required. We have shake-watch package almost ready for release that sends updates to websocket server and auto-reloads webpage with small js script

@srid Note that live coding is major feature for power users.

Great, thanks for responding so quickly; that all sounds good to me 👍
Looking forward to the new PR. Thanks for your time 😃


Update MSRV to 1.43

Signed-off-by: Otavio Salvador<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 444661704

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.6%) to 75.46%

Files with Coverage Reduction
New Missed Lines
%

../../../../../usr/share/rust/.cargo/registry/src/github.com-1ecc6299db9ec823/futures-channel-0.3.8/src/mpsc/queue.rs
1
97.3%

Totals

Change from base Build 444650317:
-0.6%

Covered Lines:
123

Relevant Lines:
163

💛  - Coveralls


Fix unattached subject inset test

This PR makes /scroll-animations/view-timelines/unattached-subject-inset.html pass.

I'm pretty sure the failure is just a flaky test - I filed #231 to track landing a solution to avoid these failures.


build fails

make all-binaries
go build -v -o output/linux_amd64/azure-cnm-plugin -ldflags "-X main.version=v0.7-2-gf768da9-dirty -s -w" cnm/plugin/*.go
github.com/Azure/azure-container-networking/log
github.com/Azure/azure-container-networking/store
github.com/Azure/azure-container-networking/platform
github.com/Azure/azure-container-networking/log
log/logger.go:44: logger.SetTarget undefined (type Logger has no field or method SetTarget)
log/stdapi.go:19: stdLog.SetTarget undefined (type *Logger has no field or method SetTarget)
make: *** [output/linux_amd64/azure-cnm-plugin] Error 2

@dspillmann2016 I tried to reproduce this error. Confirmed that the repo builds fine on multiple clean build machines. I see that you have made changes to the repo (the build version above is "dirty") before the build. Could it be that something you changes caused this error? Can you please checkout tag v0.7 and build clean?

hi
build env is mac-osx there it fails all the time - it compiles fine on Ubuntu16.4
rm -fr azure-container-networking/
git clone  https://github.com/Azure/azure-container-networking.git
cd azure-container-networking/
make all-binaries
go build -v -o output/linux_amd64/azure-cnm-plugin -ldflags "-X main.version=v0.7-2-gf768da9 -s -w" cnm/plugin/*.go
github.com/Azure/azure-container-networking/log
github.com/Azure/azure-container-networking/store
github.com/Azure/azure-container-networking/platform
github.com/Azure/azure-container-networking/log
log/logger.go:45: logger.SetTarget undefined (type Logger has no field or method SetTarget)
log/stdapi.go:19: stdLog.SetTarget undefined (type *Logger has no field or method SetTarget)
make: *** [output/linux_amd64/azure-cnm-plugin] Error 2

osx is not a supported build target OS. But you should be able to use osx as a build environment to build for a target OS (Linux or Windows) by setting the GOOS env variable. Can you please do this?
GOOS=linux make all-binaries

that worked fine.
Thanks


cmake: private linking of some libraries

Not sure if this is possible, but this would avoid linking to gl if target SoQt::SoQt is used.
@VolkerEnderlein

@VolkerEnderlein ping

@looooo Is this issue still relevant?

@looooo Is this issue still relevant?

I still use this patch yes. But I can also keep it as a patch.


Update District “madison-conservation-district”

Automatically generated by Netlify CMS

👷 Deploy Preview for candid-hamster-c2bfa4 processing.

Name
Link

🔨 Latest commit
d6d5d271bf8c0bc8e8d8ae83b4bdffe4746406f0

🔍 Latest deploy log
https://app.netlify.com/sites/candid-hamster-c2bfa4/deploys/62be565af69abd0008c36f15


Extend .gitignore file for typical things excluded in Python projects.

Here is an example of such a file. We can just take it as is, or be more mindful and select only the things we need.

I would say that we select only the things we need. I would add to the current .gitignore the following:
*.zip
__pycache__/
dist/
*.egg-info/
doc/sphinx/build/


add tension control, simplify curve drawing

Nice! I'm sure this is worth accepting, but can you give a quick blurb explaining the changes?

nm, basic enough. Just updates to correspond to updates in THREE and AFRAME it seems, along with a color option and a tension option and example updates. Thanks!


Make Kafka's message key first-class

Like SQN-3414, we want to hoist up the concept of message key / blend with our concept of grouping. So that messages with the same group are routed to same partition.

One path forward is to extend group_id from ConsumerRecrod to ConsumerEvent - in this case, we can then simply use the group_id as the message_key.Alternatively, we can pass in the full PostgresDatabaseTable and SequenceFilter then dynamically generate the message_key from the sequence_filter.group_column_attnums and the values in the record or event data.This latter path requires fewer changes and also allows the message key to be changed dynamically, ie. without migrating values in the consumer records or consumer events tables.


Out-GridView window sometimes closes when sorting

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
@(
    [pscustomobject]@{Value = 3}
    [pscustomobject]@{Value = 'abc'}
) | Out-GridView

Click on the column header to sort the Value column
Expected behavior
Column is sorted

Actual behavior
Out-GridView window closes

Error details
(No error)

Environment data
Name                           Value
----                           -----
PSVersion                      7.3.1
PSEdition                      Core
GitCommitId                    7.3.1
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

The example above was the smallest example I could come up with that reproduces the bug. I haven't yet figured out what causes the bug, but it seems to happen when you mix values of different type in the same column. But it's not the whole story since you can take the exact same example and swap the values ('abc' comes first) and the bug goes away.

Problem persists in 7.3.2
Name                           Value
----                           -----
PSVersion                      7.3.2
PSEdition                      Core
GitCommitId                    7.3.2
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

PowerShell 5.1 does not have this issue.

I regularly do things like this (with different object types), and this is when I run in to this problem:
(Get-Process)[0].PSObject.Properties | Select-Object Name,Value | Out-GridView
When i try to sort by the Value column, the grid view window closes. As I described in a previous comment, this happens only when you mix different types in the same column.

Still an Issue on Version: 7.4.2

OutGridView uses Microsoft.Management.UI.Internal.PropertyValueComparer to compare list values. PropertyValueComparer deferes the comparison to LanguagePrimitives.Compare if the types don't match.
In Microsoft.Management.UI.Internal.PropertyComparer.CompareData:
Type firstType = firstValue.GetType();
Type secondType = secondValue.GetType();

if (firstType != secondType)
{
    return LanguagePrimitives.Compare(firstValue, secondValue);
}

LanguagePrimitives.Compare tries to convert the second value to a type it can compare with the first value. If it fails it throws an error. If, for example, the first value is a string and the second value is an int, the conversion succeeds, but if it's the other way around (and the string can't be parsed as an int), the conversion will fail.
I realize it's not an easy fix. I was thinking that maybe you could fall back to ToString-ing the values if the conversion fails and let the string comparison later in CompareData take over, but that will lead to unpredictable sorting if ToString doesn't produce the same string as what's displayed in the list.


add schema for ansible-navigator

This is work-in-progress, but if there's something completely wrong I welcome comments.
Closes: https://github.com/ansible/ansible-navigator/issues/188
Related: https://github.com/ansible/vscode-ansible/issues/328

Why do they use dashes instead of underscores for the ansible-navigator configuration file? :(

Ok, I think I got a first version that's ready to be discussed. Tests are green and use the example settings file from https://ansible-navigator.readthedocs.io/en/latest/settings/#the-ansible-navigator-settings-file

That is awesome and we will discuss it for sure, it was already on my todo list so that cames as a really nice xmas present.

Why do they use dashes instead of underscores for the ansible-navigator configuration file? :(

It's quite common and kebab-case is nicer in places where snake_case is not required for things to be compatible with Python identifiers.

Rebased onto current main.

@ziegenberg TY for this btw..... It made my day :)

@cidrblock You are welcome :)


Algebraic datatypes

Just an idea.
Datatypes would be a generalization of tuples. Nonrecursive datatypes would be translated similar to tuples. Recursive datatypes would be translated similar to classes, but with some kind of reference counting to have automated deallocation. This is possible, because immutable datatypes cannot have cycles, and because datatypes will not be castable to int.
Some examples:
// could replace tuples:
data vec2 = vec2(real x, real y)
// can be generic and have cases:
data option<A> = Some(A a) | None
// can be recursive
data list<A> = Cons(A head, list<A> tail) | Nil

Discuss.

I would use it. But what about standard library using options? we are already missing a lot of obvious stl functions and frequently PRs are a pain for user and maintainer.
Option is super nice but it means that pretty much any STL extension function that uses a handle (unit, item, destructable especially) should have another override for Option, I think.
I don't think this is more harm than good - I think it's nice. Just pointing it out.


[tabs][material] Add min-width to the TabPanelRoot component

Added min-width styling to the component of 0.
Closes #37280

where to add the styles?

In the TabPanelRoot definition, inside the styled function. In term of whether we should add the style, I would wait to see if the issue is faced by more developers, currently it has zero upvotes, so probably not a common issue.


Hover zoom

Hi there! I was hoping you could add a feature where the image can zoom/enlarge when the user hovers over it?
Similar to this:

#

st-click-detector is a generalization of st-clickable-images that can be used to achieve this.


testing of new surfacebuilder

by making the biome not always glowdust sand, it means we could keep the sand bright as the dullstone patches counters the brightness of the biome. You can experiment more with the noise for this and see if it is worth add or nah lol

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Inspect all possible module paths when looking for the best one to create a specifier with

This allows us to look at both the real and (potentially multiple) symlink paths available for a module and choose the one that gives the "best" specifier from among all of them (since a symlinked path isn't always the best one!).
In the case of the linked issue, we were finding a symlink from the other module that was symlinked into the project we were compiling, causing us to forgo both the real path (which would give us a nice global specifier) and the  project-local symlink path (which would also yield a global specifier).
Fixes #25511.
Also fixes a bug in our test harness where symlinks and json files weren't carried over into the .d.ts validation build, which was causing some unnecessary .d.ts error baselines.

@weswigham can you also port to release-3.0


This error might come from the limit of the path length.

This may be challenging in an aspect; Windows recommends naming directories and files so that the full path length is shorter than 260 characters. This comes from the tradition that older Windows set the limitation.

This can be resolved by using the UNC path.


c-api: Expose image_range for modules

We're using this to monitor the amount of executable memory each module
needs.

Ooops, yes done!


fix(sonarqube): fix issue_code_blocks_convertor

Summary
Fix ConvertIssueCodeBlocks SQL query.
Add the first code block in cq_issues to cq_issue_code_blocks, so cq_issue_code_blocks can cover all text ranges of the issue.
Previous PR https://github.com/apache/incubator-devlake/pull/5443 didn't fix the split problem, fix it again. (returning page count over 100 will enter the split routine again)
Does this close any open issues?
Closes #5484
Screenshots
Include any relevant screenshots here.
Other Information
Any other information that is important to this PR.

e2e test fail


Multiple responses to get request

Issue
In case the access request is delayed when getting a resource with resource references, it is possible that the client's get/subscribe request will get two responses.
This happens when linked resources are loaded before the subscription.OnReady(...) call is made from wsConn.go.
Example
12:02:10 [Main] Starting server
12:02:10 [Main] Connecting to messaging system
12:02:10 [NATS] <=S system
12:02:10 [NATS] <=S conn.bj5c08l8smgjbc0l8ajg
12:02:10 [bj5c08l8smgjbc0l8ajg] Connected
12:02:10 [bj5c08l8smgjbc0l8ajg] --> {"method":"subscribe.test.model.parent","id":1}
12:02:10 [NATS] <=S event.test.model.parent
12:02:10 [NATS] <== access.test.model.parent: {"token":null,"cid":"bj5c08l8smgjbc0l8ajg"}
12:02:10 [NATS] <== get.test.model.parent: {}
12:02:10 [NATS] ==> get.test.model.parent: {"result":{"model":{"name":"parent","child":{"rid":"test.model"}}}}
12:02:10 [NATS] <=S event.test.model
12:02:10 [NATS] <== get.test.model: {}
12:02:10 [NATS] ==> get.test.model: {"result":{"model":{"string":"foo","int":42,"bool":true,"null":null}}}
12:02:10 [NATS] ==> access.test.model.parent: {"result":{"get":true}}
12:02:10 [bj5c08l8smgjbc0l8ajg] <-- {"result":{"models":{"test.model":{"bool":true,"int":42,"null":null,"string":"foo"},"test.model.parent":{"child":{"rid":"test.model"},"name":"parent"}}},"id":1}
12:02:10 [bj5c08l8smgjbc0l8ajg] <-- {"result":{},"id":1}

Solved in #85


Added WRITE_TO_DISK environment variable. Fixes #1070

This PR will allow us to pass the writeToDisk option to webpack-dev-middleware using a new environment variable WRITE_TO_DISK.
The new option addresses #1070 and use cases such as creating browser extensions.

Well we are just waiting for someone to merge it IIRC. 😃

The interest in the PR is still here, we just need it to be merged or someone to articulate his feedbacks if not suitable for merge already so we can move forward with it.

need this also

also in need for this. Thanks.

Sorry stale bot but no. 😛

Hey,
any news to this PR? Would be awesome to have this merged!
All the best,
Sebastian

It would be great to have this merged, would make chrome extension development much easier.

Pleaaase, merge this PR!

This would be really nice.

doesn't this still suffer from the contents of /public not being written?

Just wanna say this seems to be working fine for me. I've got an npm package @erics-forks/react-scripts that is<EMAIL_ADDRESS>with this PR merged. I'll do my best to keep it updated until this merges.


[562] Increase clock instances memory

Context
This is a quick fix to be able to ssh in qa, staging, sandbox
Changes proposed in this pull request
Increase clock memory in qa, staging, sandbox
Guidance to review
Try make shell
Link to Trello card
https://trello.com/c/tzXpUrgb
Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] Required environment variables have been updated added to the Azure KeyVault

 Apply: increase clock memory


add type script declaration

I am new to ulog and find this is a excellent lib, I want to help with typescript. this will work for all example in readme.

That's great taoqf! Thank you very much for your interest and willingness to help!!
I am always looking for help, because I will never be able to build the ultimate logger alone. And you will be able to help shape what the solutions look like. I have been working on ulog v2 for a long time now, and it changed a lot between v1 and the early betas, but currently I am at a point where I feel the basic design has stabilized and other people can come on board and contribute, so if you are willing to do so then thank you!
About your PR. Can you give me some instructions on how to test this?
And how exactly did you make this? Did you copy-paste some code perhaps? I am asking because I see this:
assert(...args: any[]): void
dir(...args: any[]): void
table(...args: any[]): void
time(...args: any[]): void
timeEnd(...args: any[]): void
verbose(message?: any, ...args: any[]): void
silly(message?: any, ...args: any[]): void

These methods don't actually exist on all loggers.
Also, we have the complex situation that methods and properties are being added to ulog and all loggers dynamically at runtime through mods. E.g. in normal situations (using the default mods), two methods, get() and set(), will be added to ulog by the settings mod. In fact any mod that has an extends key will end up extending the ulog object itself. Looking at the source of the settings mod for example we see this:
module.exports = {
  extend: {
    settings: {},
    get: function() { /* ... */  },
    set: function() { /* ... */ },
  }
}

So this mod adds a settings property and methods get and set to ulog.
I have no idea how to model this with Typescript... Maybe mods could include their own typescript definitions?
So maybe we should start simpler and first make the base typescript definition that extends the one from anylogger and only adds the stuff that is added in core:
ulog.mods = []
ulog.use = function(mod) { /* ... */ }

I can imagine we could maybe make a typescript definition for all standard entries. What I mean by that is that I am planning on providing a bunch of standard configurations of mods:

ulog: The basic config you get when you require('ulog')
full: The config with all mods included you get when you require('ulog/full')
debug: A config designed to closely resemble debug
... (not sure yet)

I have to admit that this is not fully thought out yet. I am still experimenting. So maybe for now we should focus on the ulog entry. I cam imagine we make a typescript definition that has all the properties and methods you get on ulog and all loggers when you require('ulog').

The type declarations are copied from my PR #21, but without preserving authorship! @taoqf has just copied them and committed under his name. This is very disrespectful.

@jirutka Thanks for the heads up!
If you have read my comments here, do you know of any way to dynamically extend a type definition based on which mods are in use? My gut feeling says it's not possible as I think typescript works at build-time right? So then our best option would be to add typescript definitions for the standard entries we provide such as 'ulog', 'ulog/full', 'ulog/debug' etc? If you agree, do you happen to know how we would go about this?
@taoqf Thank you for wanting to help out, but I need you to respect authorship. If Jakub did the work, he should get credit for it. So please keep this in mind when creating PRs.

I use some code of @jirutka 's.
add finished tests code in types/test.ts
I will close this pr if @jirutka minds.
@Download, I am not very familiar with the lib, just add some typescript based on @jirutka 's pr.
It's not a correct type declaration but this is a way that you can add typescript declaration files to your package.
If you want check the types, just put your usage into types/test.ts, the definition will look good If typescript shows no error in your ide(or run tsc -p ./types/tsconfig.json as well, tsc is alias name of typescript which you probably install typescript use npm first.)

@taoqf @jirutka
I opened a new issue #37 for typescript support.
If you both agree, we could maybe make a 'community effort' PR and you both will be credited along with anyone else that steps in to help. Adding the tests is very worthwhile @taoqf . In essence, you did nothing wrong by copy-pasting code; that's why we do Open Source to begin with, so we can use each other's work. Only thing is that most of us want to be credited for the work. So if we sort that out we should be good to go I hope. Do you both agree?


Help with installing ambar on Ubuntu (says it cannot find docker-compose)

So I installed a 64-bit version of Ubuntu server inside VMware with the default settings. The only thing I changed was to install a few additional things (Samba file server and OpenSSH server). Next I run sudo apt-get update and then follow the instructions to install Docker CE. Adding the repository seems to go well, and then I install docker, and I follow step 2 (sudo apt-get install docker-ce). I am able to run the hello-world container that outputs a successful message. docker --version gives me v17.03.1-ce, build c6d412e.
Next I try to install docker-compose. pip is not installed by default it seems, so I run sudo apt install python-pip. pip version is 8.1.1 (python 2.7) if that matters. Next I run pip install docker-compose. It completes and mentions a newer version of pip is available. I execute the wget command on the blog, followed by sudo ./ambar.py install. Here I run into an issue- docker-compose not found. But docker-compose --version returns version 1.12.0, build b31ff33.
Yesterday I had a different error (ERROR: In file 'docker-compose.yml' service 'version' doesn't have any configuration options), but today is different. I'm not proficient with linux and all its permissions and various directories, so if it's obvious something is wrong, I won't see it. Thanks for any help!

Hello!
Try this: sudo chmod +x /usr/local/bin/docker-compose
After, try sudo ./ambar.py install again.

I had this same issue.  I ran
pip uninstall docker-compose
then
sudo pip install docker-compose
and it fixed my issue.  Not sure if the command by fpd4444 would of worked also.

Okay, I ended up having to do the curl command to pull the latest docker-compose and then put it in /usr/bin/docker-compose and chmod +x for that directory. Is that an incorrect way of doing it?
Now ambar installs and I can load the web ui. I'm able to upload some files and it does register that documents have been uploaded under the total MB uploaded. But nothing shows up in search or in statistics panel.
I'm not sure what logs to pull to give you additional information. I rebooted the VM and now it's just stuck on the loading screen with the loading icon ..

@eliluong
Please try sudo ./ambar.py restart, wait a few minutes and post the output of sudo docker logs ambar_webapi_1 here. Thnx

@eliluong does restart helps?

Closing as author did not reply :(


fix(docker): downgrade alpine images

Since the Alpine images went to 3.13 we are experiencing this issue on arm 32-bit. We followed the advise on from the release notes but that didn't resolve the issues.
Downgrading this image now for the release. We can resolve this later on.

We were able to bisect this issue to those 2 commits, which this 2 Drone pipelines:
https://drone.grafana.net/grafana/tanka/641/3/3
https://drone.grafana.net/grafana/tanka/644/3/3
There we can see the alpine version bump from 3.12 to 3.13 and then we started hitting issues.

The ARM builds succeeded, clean'd up the tags/releases in github.


to_gbq should not modify table_schema inplace

Working on a PR...

Problem Description
pandas-gbq version: 0.10.0
GBQ table:

Reproducible example:
from copy import deepcopy
import datetime
import pandas_gbq
import pandas as pd
from google.oauth2.service_account import Credentials

df = pd.DataFrame(
    {
        "field1": ["a", "b"],
        "field2": [1, 2],
        "field3": [datetime.date(2019, 1, 1), datetime.date(2019, 5, 1)],
    }
)

original_schema = [
    {
        "name": "field1",
        "type": "STRING",
        "mode": "REQUIRED",
    },
    {
        "name": "field2",
        "type": "INTEGER",
    },
    {
        "name": "field3",
        "type": "DATE",
    },
]
original_schema_cp = deepcopy(original_schema)

pandas_gbq.to_gbq(
    dataframe=df,
    destination_table="XXXXX.schematest",
    project_id="XXXXX",
    credentials=Credentials.from_service_account_file("XXXXX.json"),
    if_exists="append",
    table_schema=original_schema,
)

Results:
>>> original_schema
[{'name': 'field1', 'type': 'STRING', 'mode': 'REQUIRED'},
 {'name': 'field2', 'type': 'INTEGER', 'mode': 'NULLABLE'},
 {'name': 'field3', 'type': 'DATE', 'mode': 'NULLABLE'}]
>>> original_schema_cp == original_schema
False

Ahhhhh!  Nowhere is it noted in https://pandas-gbq.readthedocs.io/en/latest/api.html that, in some situations, the object passed to table_schema may be modified in place.  Most libraries go out of their way to avoid modifying mutable arguments; pandas-gbq should do the same.
Debugging
A pdb session shows that table_schema is modified in connector.load_data(), which in turn calls load_chunks().  The specific place in load_chunks() where the inadvertant modification happens is here:
# https://github.com/bsolomon1124/pandas-gbq/blob/59228d9c20cee12b24caa5cc41d3f2e6c0337932/pandas_gbq/load.py#L72
    for field in schema["fields"]:
        if "mode" not in field:
field["mode"] = "NULLABLE"

This is because, I think, in table_schema = schema.update_schema(default_schema, dict(fields=table_schema)), the new table_schema now contains a reference to the dict elements of the original table_schema argument.

Good spot @bsolomon1124! 💯 the existing state is not good practice.
I think we could solve this fairly easily by running copy.copy on the incoming schema object. You mentioned at the top you were thinking of a PR - appreciated if so...

I believe we want copy.deepcopy(), since table_schema will be a list of dictionaries.

Opened in https://github.com/pydata/pandas-gbq/pull/278 @max-sixty .  Need to run for the evening, but appreciate a look.  Added a small test also.


Add a company starting a letter between  A - F

Hi @Elijah699 can l work on this issue, l promise deliver it well

I would like to work on this issue


Keep formatting in 'Copy all' option

          Unfortunately copying text using 'Copy all' looses nice formatting.

this formatting is lost:

and copied text looks more like this:

Workaround is to use Developer Tools, copy whole element and convert output from html to markdown. Pasting retrieved transcript directly into the current note would be a nice option.
Originally posted by @gala8y in https://github.com/lstrzepek/obsidian-yt-transcript/issues/1#issuecomment-1516701943

Seeing as this is still unresolved I'd like to solve this


Dependency Injection

Wow! You just took this project out of my mind! With all these 'pure Python webapp' libraries using WebSockets, I wondered when someone would leverage HTMX. Anyway, is there a way to inject dependencies into a parent component? For example, a generic snippet injecting the Alpine.js source into the <head> of the page?
It would be very helpful for creating component libraries that also depend on JavaScript libraries without forcing the final user to worry about setting up those dependencies correctly.

Agreed. We haven't finalised on a design we like yet. I'll leave this issue open and come back to it once we have something implemented.

Agreed. We haven't finalised on a design we like yet. I'll leave this issue open and come back to it once we have something implemented.

It would also be useful not only to inject dependencies into the <head> of the page but also to have the capability of injecting at any level of the tree. I can imagine an infinite number of components using, for instance, Alpine.js, like a form component that injects functions into the x-data attribute of the parent component for validation, serialization, two-way binding, etc. This feature would make implementing a global pub/sub for intra-component communication, among other exciting tools, easily achievable. It would almost give magical superpowers to fasthtml.

You can do that already using oob swaps with HTMX, unless I'm misunderstanding.

You can do that already using oob swaps with HTMX, unless I'm misunderstanding. Here's an example of using Alpine with HTMX btw: https://django-htmx-alpine.nicholasmoen.com/

I'm thinking about more advanced use cases where the states must stay on the client side. There is a barrier that every htmx user will hit, and it's those advanced use cases. The developers themselves recommend using their side project _hyperscript (or Alpine.js, which i prefer), when this happens. The idea I mentioned before was something like a form and field integration (note that this is more pseudocode than real code since I'm not yet fluent in FastHTML):
def TextField(name):
    x_model = f'___form.{name}'
    return Input(type="text", name=name, id=name, x_model=x_model)

def NumberField(name):
    x_model = f'___form.{name}'
    return Input(type="number", name=name, id=name, x_model=x_model)

In order for Alpine.js' two-way binding to work, they must be below a common tag with the x-data attribute set (in addition to the library being added to the <head>). The injection mechanism would inject it into the parent <form>. From the moment x-data exists, all the Alpine.js magic can be attached, such as a serialization function like serialize() { return $el._x_dataStack.reverse().reduce((a, b) => Object.assign(a, b), {}); }, which traverses the entire x-data tree looking for all the data client-side states. The global pub/sub would also work this way, adding the x-data attribute to the <body> tag with some global functions that can be called by any child component.
The use case i have in mind is a personal dashboard i wrote using only htmx. It was a form POST that retrieves data from the database, performs some manipulations with pandas, and returns a Plotly graph over the wire. However, it becomes utterly slow. Turns out i had to rewrite it using the Plotly.js library on the client side only.

I'm just looking at the library today. I came here to ask this question, though more for the reasons that FastAPI uses dependency injection. My endpoint functions typically take a database connection, and the calling user, as an injected parameter. While the latter can be taken from a session, I think it's cleaner to use the FastAPI Depends call.
It would also make porting my application over considerably easier!

Maybe it's better to rename the issue since 'dependency injection' is already a term used in another context within the FastAPI community?

That would seem good to me.

Here is a sketch of my idea for dependency injection that I brainstormed using Claude Sonnet.
I thought of 2 decorators: one to mark the components that need some dependency, and another to apply the dependencies to the final component returned by the routes:
import functools
import starlette.requests

INJECTED_ATTR_NAME = '___deps'
_valid_names = ('parent', 'head')

def register_dep(target, modifier_fn):
    def decorator(func):
        @functools.wraps(func)
        def wrapper(*args, **kwargs):
            result = func(*args, **kwargs)
            if not hasattr(result, INJECTED_ATTR_NAME):
                setattr(result, INJECTED_ATTR_NAME, [])
            getattr(result, INJECTED_ATTR_NAME).append({
                'target': target,
                'modifier_fn': modifier_fn
            })
            return result
        return wrapper
    if target not in _valid_names:
        raise ValueError(f"Invalid target: {target}. Must be one of {_valid_names}")
    return decorator

def apply_deps(func):
    @functools.wraps(func)
    def wrapper(*args, **kwargs):
        def _extract_objects(args_to_test, *types):
            ret = []
            for obj_type in types:
                found = False
                for arg in args_to_test:
                    if isinstance(arg, obj_type):
                        ret.append(arg)
                        found = True
                if not found:
                    ret.append(None)
            return None if not ret else ret[0] if ret and len(ret) == 1 else ret

        request = _extract_objects(args, starlette.requests.Request)

        def traverse_and_apply(element, path=[]):
            if isinstance(element, list) and len(element) == 3:  # FT object
                if hasattr(element, INJECTED_ATTR_NAME):
                    for dep in getattr(element, INJECTED_ATTR_NAME):
                        target = dep['target']
                        modifier_fn = dep['modifier_fn']
                        if target == 'head':
                            if not (request and hasattr(request, 'hdrs')):
                                raise Exception('Target "head" specified, but request.hdrs is not present')
                            modifier_fn(request.hdrs)
                        elif target == 'parent':
                            if path:
                                parent = path[-1]
                                modifier_fn(parent)
                            else:
                                print("Warning: 'parent' modification attempted on root element")
                tag, cs, attrs = element
                new_cs = []
                for child in cs:
                    new_child = traverse_and_apply(child, path + [element])
                    new_cs.append(new_child)
                element[1] = tuple(new_cs)
            elif isinstance(element, (list, tuple)):
                return type(element)(traverse_and_apply(item, path) for item in element)
            return element

        result = func(*args, **kwargs)
        return traverse_and_apply(result)

    return wrapper

First, you register the dependencies on the component definition:
import json
import fasthtml.common as fh

app, rt = fh.fast_app()

def GenericField(_type, name):
    x_model_key = 'x-model'
    x_model = f'___form.{name}'
    if _type == 'number':
        x_model_key += '.number'
    return fh.Input(type=_type, placeholder=name, name=name, id=name, **{x_model_key: x_model})

def add_alpinejs_to_hrds(hrds):
    for i in hrds:
        if i[0] != 'script':
            continue
        if not hasattr(i, 'src'):
            continue
        if 'alpinejs' in i.src:
            return
    hrds.append(fh.Script(src='https://unpkg.com/alpinejs', defer=True))

def init_xdata(t):
    jdata = t.attrs.get('x-data', '{}')
    jdata = json.loads(jdata)
    form = jdata.get('___form', {})
    jdata['___form'] = form
    t.attrs.update({'x-data': json.dumps(jdata)})

@register_dep('head', add_alpinejs_to_hrds)
@register_dep('parent', init_xdata)
def TextField(name):
    return fh.Div(GenericField('text', name))

@register_dep('head', add_alpinejs_to_hrds)
@register_dep('parent', init_xdata)
def NumberField(name):
    return GenericField('number', name)

Note that the modifier_fn can be a simple lambda function, but complete function definitions would work better when avoiding repetitions or solving conflicts:
@register_dep('head',  lambda hrds: hrds.append(fh.Script(src='https://unpkg.com/alpinejs', defer=True)))
@register_dep('parent', init_xdata)
def NumberField(name):
    return GenericField('number', name)

Then, apply the modifications to the FT tree
@rt("/")
@apply_deps
def get(request):
    return fh.Titled("FastHTML", fh.Div(
        fh.Div(x_text="JSON.stringify(___form)"),
        TextField('name'),
        NumberField('age'),
    ))

fh.serve()

Here the full draft:
import functools
import starlette.requests

INJECTED_ATTR_NAME = '___deps'
_valid_names = ('parent', 'head')

def register_dep(target, modifier_fn):
    def decorator(func):
        @functools.wraps(func)
        def wrapper(*args, **kwargs):
            result = func(*args, **kwargs)
            if not hasattr(result, INJECTED_ATTR_NAME):
                setattr(result, INJECTED_ATTR_NAME, [])
            getattr(result, INJECTED_ATTR_NAME).append({
                'target': target,
                'modifier_fn': modifier_fn
            })
            return result
        return wrapper
    if target not in _valid_names:
        raise ValueError(f"Invalid target: {target}. Must be one of {_valid_names}")
    return decorator

def apply_deps(func):
    @functools.wraps(func)
    def wrapper(*args, **kwargs):
        def _extract_objects(args_to_test, *types):
            ret = []
            for obj_type in types:
                found = False
                for arg in args_to_test:
                    if isinstance(arg, obj_type):
                        ret.append(arg)
                        found = True
                if not found:
                    ret.append(None)
            return None if not ret else ret[0] if ret and len(ret) == 1 else ret

        request = _extract_objects(args, starlette.requests.Request)

        def traverse_and_apply(element, path=[]):
            if isinstance(element, list) and len(element) == 3:  # FT object
                if hasattr(element, INJECTED_ATTR_NAME):
                    for dep in getattr(element, INJECTED_ATTR_NAME):
                        target = dep['target']
                        modifier_fn = dep['modifier_fn']
                        if target == 'head':
                            if not (request and hasattr(request, 'hdrs')):
                                raise Exception('Target "head" specified, but request.hdrs is not present')
                            modifier_fn(request.hdrs)
                        elif target == 'parent':
                            if path:
                                parent = path[-1]
                                modifier_fn(parent)
                            else:
                                print("Warning: 'parent' modification attempted on root element")
                tag, cs, attrs = element
                new_cs = []
                for child in cs:
                    new_child = traverse_and_apply(child, path + [element])
                    new_cs.append(new_child)
                element[1] = tuple(new_cs)
            elif isinstance(element, (list, tuple)):
                return type(element)(traverse_and_apply(item, path) for item in element)
            return element

        result = func(*args, **kwargs)
        return traverse_and_apply(result)

    return wrapper

###

import json
import fasthtml.common as fh

app, rt = fh.fast_app()

def GenericField(_type, name):
    x_model_key = 'x-model'
    x_model = f'___form.{name}'
    if _type == 'number':
        x_model_key += '.number'
    return fh.Input(type=_type, placeholder=name, name=name, id=name, **{x_model_key: x_model})

def add_alpinejs_to_hrds(hrds):
    for i in hrds:
        if i[0] != 'script':
            continue
        if not hasattr(i, 'src'):
            continue
        if 'alpinejs' in i.src:
            return
    hrds.append(fh.Script(src='https://unpkg.com/alpinejs', defer=True))

def init_xdata(t):
    jdata = t.attrs.get('x-data', '{}')
    jdata = json.loads(jdata)
    form = jdata.get('___form', {})
    jdata['___form'] = form
    t.attrs.update({'x-data': json.dumps(jdata)})

@register_dep('head', add_alpinejs_to_hrds)
@register_dep('parent', init_xdata)
def TextField(name):
    return fh.Div(GenericField('text', name))

@register_dep('head', add_alpinejs_to_hrds)
@register_dep('parent', init_xdata)
def NumberField(name):
    return GenericField('number', name)

@rt("/")
@apply_deps
def get(request):
    return fh.Titled("FastHTML", fh.Div(
        fh.Div(x_text="JSON.stringify(___form)"),
        TextField('name'),
        NumberField('age'),
    ))

fh.serve()


joss-paper: Performance benchmarks

I think the software package (and the paper) will enormously benefit from a performance benchmark sustaining the performance claims of the software. I think this will also significantly increase the interest in the software and its impact.
I would be very happy to see a plot "number of collocation points vs time", plotting the time required to generate increasingly refined grids for a given function (up to millions of collocation points).
But if you have a better idea for a performance benchmark, please go ahead.
See openjournals/joss-reviews#5003

Hi, @baxmittens, I really appreciate the effort you devoted to this.
I agree that a performance benchmark is not easy to include, so maybe I could do with a more elaborate and precise justification of your performance claim.
I've carefully read your results and discussion. I wonder if what sets this package apart from others and sustain your performance claim are the (1) distributed function evaluation and the (2) usage of arbitrary return types.
When evaluating a function at all grid points, while other packages exploit (at most) shared-memory parallelism, you can exploit distributed-memory parallelism. As a result, your package is clearly more suitable, when a single evaluation is computation- and memory-intensive.
Also, in contrast to other packages, the user can choose the right (performant) return data type for the problem at hand, optimise allocations with inplace operators...
Did I get this right?

@ericneiva
A bit off-topic, but I just read some of your papers. They are very interesting to me. I also work on a similar topic. If you want to, you can check out our paper an hp-hierarchical framework for the finite element exterior calculus where we extended natural refinement (adaptive refinement without hanging nodes based on the refinement of basis function rather than elements) to arbitrary dimensions for all simplicial elements of douglas n. arnolds finite element zoo.

I added some subchapters to the paper. I close this for now and generate a paper proof in the review repo.
Reopen this if necessary.


Sort service standard points numerically

This fixes the bug where we sorted alphabetically and 10 came after 1 rather than 2.

LGTM 👍


Add generic payload typing on dispatch method

Hello 👋
I saw the note in your readme

Note: There is currently no typing support for job payloads.

You can let the user set it's own interface for the payload with a generic on the dispatch method

interface MyJobPayload {
 message: string;
}

Queue.dispatch<MyJobPayload>(
  'App/Jobs/RegisterStripeCustomer',
  {
   message: "Hello job", // 👈 should force MyJobPayload for the payload
  },
);

Hope it's helps

Hey @simlej! 👋🏻
Thanks for your PR!
I was thinking about doing something similar to what it is done in the Adonis Event system.
declare module '@ioc:Adonis/Core/Event' {
  interface EventsList {
    'new:user': { id: number; email: string }
  }
}

What do you think?
📚 https://docs.adonisjs.com/guides/events#making-events-type-safe


Compact menu icon flickering

Steps to Reproduce:

Click compact menu icon to open it
Click again to hide it. Observe icon flickering for a split second

src/vs/base/browser/ui/menu/menubar.ts is removing and adding the button element during  updateOverFlowAction. I don't quite understand the intent of that method but that's why it's behaving this way.

@caiges Thanks for the pointer. I took a look and I fixed it, gonna make a PR.


Symmetric key attestation information is missed

The description of "All devices in the enrollment group present X.509 certificates that have been signed by the same root or intermediate Certificate Authority (CA). Enrollment groups can only use the X.509 attestation mechanism" is incorrect as I can also use symmetric key attestation with group enrollment

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

ID: ec5e18dd-edbd-2978-d8bf-7739a1e54cea
Version Independent ID: ce89eed6-ef5a-bde4-9955-a5e7d99a6945
Content: Service concepts in Azure IoT Hub Device Provisioning Service
Content Source: articles/iot-dps/concepts-service.md
Service: iot-dps
GitHub Login: @nberdy
Microsoft Alias: nberdy

@xiongyu0523 Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@nberdy Please have a look at this issue and update the doc.

Thanks for the feedback, I've submitted a PR to update the definition.

#please-close


The Update staging k8 specs goal can fail if it runs before the image link event from the previous goal has been ingested

There is nothing to control this at the moment - it's just a timing thing. Possibly the goal performance improvements are making this more likely to occur too.
The ideal fix for this is that there is an intermediary goal that is fulfilled by the ImageLinked system event firing. This ensures that the flow is as intended.

timing issue on ImageLink possibly


Cannot find containers 'test_cvat_server_1' and 'test_cvat_db_1' when launching Python tests with default config

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Expected Behaviour

The 'test_cvat_server_1' and 'test_cvat_db_1' containers launch correctly with pytest tests/python command.
Current Behaviour

Unless explicitly editing /cvat/tests/python/shared/fixtures/init.py 'test_cvat_server_1' and 'test_cvat_db_1' container generation lines to include dashes like in attached file these containers fail to launch with the following errors:
(.env) [%USER%@%HOST% cvat]$ python3.9 -m pytest tests/python
===================================== test session starts ======================================
platform linux -- Python 3.9.15, pytest-6.2.5, py-1.11.0, pluggy-1.0.0 -- ~/Development/cvat/.env/bin/python3.9
cachedir: .pytest_cache
rootdir: ~/Development/cvat/tests/python, configfile: pytest.ini
plugins: cov-4.0.0, xdist-3.0.2, Faker-15.1.1
collected 559 items

tests/python/cli/test_cli.py::TestCLI::test_can_create_task_from_local_images WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "http_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "https_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set.
Error message: Error: No such container:path: test_cvat_server_1:/tmp

and
(.env) [%USER%@%HOST% cvat]$ python3.9 -m pytest tests/python
===================================== test session starts ======================================
platform linux -- Python 3.9.15, pytest-6.2.5, py-1.11.0, pluggy-1.0.0 -- ~/Development/cvat/.env/bin/python3.9
cachedir: .pytest_cache
rootdir: ~/Development/cvat/tests/python, configfile: pytest.ini
plugins: cov-4.0.0, xdist-3.0.2, Faker-15.1.1
collected 559 items

tests/python/cli/test_cli.py::TestCLI::test_can_create_task_from_local_images WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "http_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "https_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set. Defaulting to a blank string.
WARN[0000] The "no_proxy" variable is not set.
Error message: Error: No such container:path: test_cvat_db_1:/tmp

Possible Solution

In this stackoverflow question is mentioned either exporting an env variable COMPOSE_COMPATIBILITY=true (tried setting export COMPOSE_COMPATIBILITY=true in .bashrc, didn't help)
Launching docker-compose with --compatibility option (cannot be applied from the cli, since the initial command is different and doesn't have these options)
Downgrading docker-compose to Python-based version 1 (probably unsafe and/or incompatible)
Editing /cvat/tests/python/shared/fixtures/init.py like in the file attached (unmaintainable, hinders git workflow, since it's probably a docker and/or config-related issue)

Steps to Reproduce (for bugs)

Clone the develop cvat repo
Install the dev environment
Launch tests with pytest tests/python

Context

The dev env has been launched on Arch Linux, using packages from this PR.
Also:
[%USER%@%HOST% ~]$ docker --version
Docker version 20.10.21, build baeda1f82a

and
[%USER%@%HOST% ~]$ docker-compose --version
Docker Compose version 2.12.2

There's also this cvat issue, where the problem is mentioned, and the author ended up with manually renaming containers in /cvat/tests/python/shared/fixtures/init.py as well.

Operating System and version (e.g. Linux, Windows, MacOS):
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

init.py.zip

Hi, it looks similar to one other issue, please check the answer.

Thanks! It needs to be noted, however, that it's not enough to simply install v1 docker-compose:
I installed the same 1.29.2 version from here on my Arch setup, reloaded the daemon configs with systemctl daemon-reload and restarted docker.service, but when launching python tests got errors:
tests/python/rest_api/test_load_register.py::TestLoadRegister::test_load_register [27197] Error loading Python lib '/tmp/_MEIcGl0rS/libpython3.7m.so.1.0': dlopen: libcrypt.so.1: cannot open shared object file: No such file or directory

and
! _pytest.outcomes.Exit: Command failed: docker-compose -p test --env-file ~/Development/cvat/tests/python/webhook_receiver/.env -f ~/Development/cvat/docker-compose.dev.yml -f ~/Development/cvat/tests/docker-compose.file_share.yml -f ~/Development/cvat/tests/docker-compose.minio.yml -f ~/Development/cvat/tests/docker-compose.webhook.yml -f ~/Development/cvat/components/analytics/docker-compose.analytics.tests.yml -f ~/Development/cvat/docker-compose.tests.yml up -d .
Error message: see above.
Add `-s` option to see more details !

Then I found a couple possible solutions from this question on SO (yet again mismatching lib name), tested them and they both worked:

Installing libxcrypt-compat:

sudo pacman -S libxcrypt-compat

Symlinking the libs: libcrypt.so to libcrypt.so.1:

$ sudo ln -s /usr/lib/libcrypt.so /usr/lib/libcrypt.so.1

Ok, great! Could you add this info about library manipulations to the Arch linux installation docs?

Ok, great! Could you add this info about library manipulations to the Arch linux installation docs?

Sure, I will.

Opened a related PR.

Wouldn't it be better to determine compose version at runtime and use "_" or "-" in container names, depending on the installed version?

I think we could rename the containers automatically in case - was added during the launch.


Bug: neovim version and other info as soon as nvim is opened is not being displayed

Self Checks

[x] I'm using the latest lualine.
[x] I didn't find the issue in existing issues or PRs.

How to reproduce the problem
require("lualine").setup({})

Expected behaviour
Had to remove lualine to show that without lualine the nvim version and other info is being displayed.

Actual behaviour

Minimal config to reproduce the issue
require("lualine").setup({})

Additional information

FWIW, I get a "flash" of the startup page: https://www.loom.com/share/cda772dd272240dbad05938096a272f4

In neovim version 0.9.0 this seems to be fixed (there was no change in my config)

I don't know if this helps, but I solved the 'flash of startup page' with the following code:
-- If the current buffer is a 'No Name' buffer, display the startup page
if vim.api.nvim_buf_get_name(vim.api.nvim_get_current_buf()) == '' then
    vim.cmd('intro')
end

Note: I am using Neovim version 0.9.0 and lualine is up to date.

I also had the same problem. this has not been patched right?

right

I have the same, still waiting for some resolution

I also have this issue, but there is a workaround; if you change section separators to an empty string, you get the message back.
require('lualine').setup {
    options = {
      -- disabling section separators fixes
      -- the disappearing start screen issue
      section_separators = '',
    }
  }

Personally I don't use this because I like the angled separators, because they match my tmux theme separators, but this is a very curious bug, I really wonder what it is about setting separators which (seems to) cause it.

if you change section separators to an empty string, you get the message back.

I have section_separators = "" in my config but still I do not get the startup screen.

if you change section separators to an empty string, you get the message back.
I have section_separators = "" in my config but still I do not get the startup screen.

Seems even more mysterious!
You may need to disable other plugins too, when I started troubleshooting this I had several which hid the startup screen. I actually just now set some of the others to lazy load and switched my config to feline.nvim which let me style it how I wanted while still showing the startup screen.

You may need to disable other plugins too

yes, in the original post where I mentioned "minimal config" lualine was the only plugin that was active.

I also have this issue, but there is a workaround; if you change section separators to an empty string, you get the message back.
require('lualine').setup {
    options = {
      -- disabling section separators fixes
      -- the disappearing start screen issue
      section_separators = '',
    }
  }

Personally I don't use this because I like the angled separators, because they match my tmux theme separators, but this is a very curious bug, I really wonder what it is about setting separators which (seems to) cause it.

works for me!

Odd, I too can solve it by setting section_separators (using Neovim 0.9.5).
However, flash.nvim also cause the same issue, there is something else going on...

Also experiencing this issue, nvim 0.9.5. As a result of something in this plugin, and a few others.
Issue seems to be in plugins which cause any "redraw" of the nvim UI. This results in the startup info disappearing (:h :intro).
In some cases you can workaround this buy lazy loading the plugin into the VimEnter event, but sadly that doesn't work for me. And, I also have custom separators set, so the above workaround is a no go for me either 😞


Bluetooth: controller: split: Fix DLE duplicate requests

Fix implementation to handle back-to-back and duplicate
LENGTH_REQ PDU reception.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

I am using zephyr, the version is 2.2.0, this bug exists. For testing purposes, I tried to trigger this bug. I repeated LL_LENGTH_REQ twice but no error was triggered.
What conditions need to be met to trigger this bug?What requirements need to be met by the data packet field? Can you explain it in detail？
thank you~~

@JiaDuo Your tester should send two LL_LENGTH_REQ PDU, one after the other in the same connection interval while not receiving the LL_LENGTH_RSP (nack the reception). This is an invalid behavior testing.

Thank you very much!

@cvinayak
Still, I can't trigger this bug. I'm using "peripherals_ HR" as a case. Send connect_ind first, and then send two LL_ LENGTH_ Req, I don't know how to ensure that two REQ are in the same connection interval .
Can you give me more suggestions?