Modal components

a55182e should read "Refactor modal component to use yew-hooks. Add yew-hooks as a dependency"
I would like your views on adding yew-hooks as a dependency.
I tried refactoring the Modal & About components so that they shared more code, but the differences in the exit buttons (verified the mark up differences with paternfly) were causing problems I couldn't find an easy solution. I don't like duplicating the code, but given that this is only used by two components, its a trade off between that and complexity.
Also to note the  About component attributes are significantly different to the patternfly attributes:

brand_src, brand_alt, and the children are required
title is productName in pf and is not required
strapline is trademark in pf and only accepts strings, not html
logo is called  backgroundImageSrc in pf. and is a far better description imho
hero_style allows custom styling, pf does not.

Finally, the About component is called AboutModal in pf

That looks impressive :+1:
Regarding the naming, I agree, it might make sense to align this with PF. Having said that, it would mean a breaking change, unless we add some type alias, which I am not too fond of.
Then again, PF5 is coming up, so maybe that's the right moment to break the API on main, and head for 0.5.x and towards PF5.
When it comes to yew-hooks I am a bit undecided. I love some hooks of that collection, while others are a bit problematic. So, I didn't want to force this dependencies on others. After all, it wasn't that much code. I am still undecided.

Regarding the naming, I agree, it might make sense to align this with PF. Having said that, it would mean a breaking change, unless we add some type alias, which I am not too fond of.

Then again, PF5 is coming up, so maybe that's the right moment to break the API on main, and head for 0.5.x and towards PF5.

Up to you, maybe throw some depreciation warnings in, linking to an issue describing the changes?

When it comes to yew-hooks I am a bit undecided. I love some hooks of that collection, while others are a bit problematic. So, I didn't want to force this dependencies on others. After all, it wasn't that much code. I am still undecided.

I'm in favor, of course as it is my suggestion. Whilst it is minima; in this case, it has potential to reduce maintenance effort, and yew-hooks is going to be more widely used, so better battle tested, At the end of day its down to you, so let me know what to do!

"Keyboard and mouse users shouldn’t be able to interact with the rest of the page outside of an open modal. Set the initial focus on the first focusable element in the modal. The user should be able to Tab to any interactive elements within the modal and use Tab + Shift to move backwards through interactive elements. Keyboard users should be able to leave the modal by pressing Esc and focus should return to the element that invoked the modal."
- https://www.patternfly.org/v4/components/about-modal/accessibility
PF reat appears to use a react wrapper around https://www.npmjs.com/package/focus-trap for this. Any problem with including this? There doesn't appear to be a rust soloution to this...

When it comes to yew-hooks I am a bit undecided. I love some hooks of that collection, while others are a bit problematic. So, I didn't want to force this dependencies on others. After all, it wasn't that much code. I am still undecided.

I'm in favor, of course as it is my suggestion. Whilst it is minima; in this case, it has potential to reduce maintenance effort, and yew-hooks is going to be more widely used, so better battle tested, At the end of day its down to you, so let me know what to do!

I think you just convinced me to add it :)

"Keyboard and mouse users shouldn’t be able to interact with the rest of the page outside of an open modal. Set the initial focus on the first focusable element in the modal. The user should be able to Tab to any interactive elements within the modal and use Tab + Shift to move backwards through interactive elements. Keyboard users should be able to leave the modal by pressing Esc and focus should return to the element that invoked the modal." - https://www.patternfly.org/v4/components/about-modal/accessibility
PF reat appears to use a react wrapper around https://www.npmjs.com/package/focus-trap for this. Any problem with including this? There doesn't appear to be a rust soloution to this...

No, but welcome to JS hell ;-) … You can add this the same way popper.js is added. And that way isn't documented (sorry for that). Basically there are three main steps:

Provide debug/release .js files in js/debug and js/release (if you need, you can create a rollup script in js/hell (sorry for the name) … the code is being committed to the repository.
Create some basic wrapper around it using wasm_bindgen (like src/integration/popperjs.rs). This includes the snippets later on when compiling with trunk.
Use it.

Regarding the naming, considering this for the next release, let's just break some APIs. Choose the names you think are right!

And because I was just peeking at my old code in popperjs.rs, maybe not really to heavily on what I've done there :grin:

This looks good. I think it would be great if you could also fix the button AttrValue (as we now break APIs :wink: ) so that we can get rid to this ToDo. Also for the ToDo list of missing PF react properties, I guess it's ok to open a tracking issue for those, but I would rather not have them inside the code.
Btw, I am planing on upticking the version to 0.5 before merging this, and I would also create a 0.4.x branch to keep the current state available. Does that work for you?

Sounds good boss!

It's good to merge, but it looks like you need to rebase.

Sorry, I'm stuck on what I've got to do now....

Sorry I meant that I didn't know what to to to rebase, but I think i've figured this out. Does the above look correct?

Ah no worries, I could do it manually.
Thanks for putting all the work!

Ah no worries, I could do it manually.
Thanks for putting all the work!

No worries

I had to bring back the hero_style property though, we've been using that, and the new property was a bit too simplistic for our case. But it acts as an override now to the new "background" property.


Remove duplicate (incorrect) fracker mesh declarations

These are redeclared later in the file with correct paths.

Hah, nice catch!


Check AuthenticateResult of IAuthenticateResultFeature is not null.

The authResultFeature.AuthenticateResult may be null in net8.
[19:31:23 INF] Executed endpoint '/_blazor/negotiate'
[19:31:23 ERR] An unhandled exception has occurred while executing the request.
System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.Azure.SignalR.NegotiateHandler`1.BuildClaims(HttpContext context)
   at Microsoft.Azure.SignalR.NegotiateHandler`1.Process(HttpContext context)
   at Microsoft.Azure.SignalR.ServiceRouteHelper.RedirectToService[THub](HttpContext context, IList`1 authorizationData)
   at Microsoft.Azure.SignalR.NegotiateMatcherPolicy.<>c__7`1.<<CreateNegotiateEndpointCore>b__7_0>d.MoveNext()
--- End of stack trace from previous location ---
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|7_0(Endpoint endpoint, Task requestTask, ILogger logger)
   at Volo.Abp.AspNetCore.Serilog.AbpSerilogMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.InterfaceMiddlewareBinder.<>c__DisplayClass2_0.<<CreateMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Volo.Abp.AspNetCore.Auditing.AbpAuditingMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.InterfaceMiddlewareBinder.<>c__DisplayClass2_0.<<CreateMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext)
   at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
   at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
   at Volo.Abp.AspNetCore.Security.Claims.AbpDynamicClaimsMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.InterfaceMiddlewareBinder.<>c__DisplayClass2_0.<<CreateMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Volo.Abp.AspNetCore.Uow.AbpUnitOfWorkMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.InterfaceMiddlewareBinder.<>c__DisplayClass2_0.<<CreateMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Volo.Abp.AspNetCore.ExceptionHandling.AbpExceptionHandlingMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Volo.Abp.AspNetCore.ExceptionHandling.AbpExceptionHandlingMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.InterfaceMiddlewareBinder.<>c__DisplayClass2_0.<<CreateMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Volo.Abp.AspNetCore.MultiTenancy.MultiTenancyMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.InterfaceMiddlewareBinder.<>c__DisplayClass2_0.<<CreateMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Microsoft.AspNetCore.Builder.ApplicationBuilderAbpOpenIddictMiddlewareExtension.<>c__DisplayClass0_0.<<UseAbpOpenIddictValidation>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
   at Volo.Abp.AspNetCore.Security.AbpSecurityHeadersMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.InterfaceMiddlewareBinder.<>c__DisplayClass2_0.<<CreateMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Volo.Abp.AspNetCore.Tracing.AbpCorrelationIdMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.InterfaceMiddlewareBinder.<>c__DisplayClass2_0.<<CreateMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Microsoft.AspNetCore.Localization.RequestLocalizationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.RequestLocalization.AbpRequestLocalizationMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
   at Microsoft.AspNetCore.Builder.UseMiddlewareExtensions.InterfaceMiddlewareBinder.<>c__DisplayClass2_0.<<CreateMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddlewareImpl.Invoke(HttpContext context)
[19:31:23 INF] Request finished HTTP/2 POST https://localhost:44377/_blazor/negotiate?negotiateVersion=1 - 500 null text/plain; charset=utf-8 19.3387ms

I'd love to have this PR completed in the near future. I've been plagued by this same issue.
Is there an update on getting this merged?

oops, interrupted by some random test failure, thanks for the reminder.


grafana: 6.3.3 -> 6.3.4, update url for static source [CVE-2019-15043]

Motivation for this change
Patch relase for CVE-2019-15043.
Things done
Updated URL for the static source.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS (x86_64-linux & aarch64-linux)
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg test grafana

backported in 9fe893156b0ef9e100c78162e8a21767087dfed8


Merge package:pool

[x] Move and fix workflow files, labeler.yaml, and badges in the README.md
[x] Rev the version of the package, so that pub.dev points to the correct site
[x] Add a line to the changelog:

* Move to `dart-lang/tools` monorepo.

[x] Add the package to the top-level readme of the monorepo:

| [pool](pkgs/pool/) | Manage a finite pool of resources. Useful for controlling concurrent file system or network requests. | [![package issues](https://img.shields.io/badge/package:pool-4774bc)](https://github.com/dart-lang/tools/issues?q=is%3Aissue+is%3Aopen+label%3Apackage%3Apool) | [![pub package](https://img.shields.io/pub/v/pool.svg)](https://pub.dev/packages/pool) |

[x] Important! Merge the PR with 'Create a merge commit' (enabling then disabling the Allow merge commits admin setting)
[x] Update the auto-publishing settings on https://pub.dev/packages/pool/admin
[x] Add the following text to https://github.com/dart-lang/pool/:'

> [!IMPORTANT]
> This repo has moved to https://github.com/dart-lang/tools/tree/main/pkgs/pool

[ ] Publish using the autopublish workflow
[x] Push tags to GitHub using

git tag --list 'pool*' | xargs git push origin

[x] Close open PRs in dart-lang/pool with the following message:

Closing as the [dart-lang/pool](https://github.com/dart-lang/pool) repository is merged into the [dart-lang/tools](https://github.com/dart-lang/tools) monorepo. Please re-open this PR there!

[ ] Transfer issues by running

dart run pkgs/repo_manage/bin/report.dart transfer-issues --source-repo dart-lang/pool --target-repo dart-lang/tools --add-label package:pool --apply-changes

[ ] Archive https://github.com/dart-lang/pool/

[x] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.
Most changes should add an entry to the changelog and may need to rev the pubspec package version.
Changes to packages require corresponding tests.

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

I did not publish this yet because of the breaking change introduced by https://github.com/dart-lang/pool/pull/81. Should we roll this back or publish a version 2.0.0?
@kevmoo @devoncarew


Migration from Bootstrap 3 to 5

The migration from Bootstrap 3 to Bootstrap 5 is a huge change, since Bootstrap 3 has been around since 2013. There might be no easy way to convert Bootstrap 3 to Bootstrap 5, due to many major changes and additions.

Hi,
the following security vulnerabilities
https://security.snyk.io/package/npm/bootstrap/3.3.1
was found on the Release 5.2.9 (Docker)
greetings
Thomas

We tested this implementation for more then 2 month now (in the whole Roberta team). Please write new issues if you still find issues.

@Lexorius thank you for your comment. We have almost finished the transition to version 5.


Fix documentation for organization-policy module

Aleks, you can also force push this kind of things ;)


upgrade flutter

We have a few things that would benefit from a newer flutter version or are even currently blocked

Blocked by #139

Here's the WIP: https://github.com/encointer/encointer-wallet-flutter/pull/460

New wip #678

Closed by #678


refactor: removed duplicate storybook code

Refactored duplicate code in progress/meter utils.js

Really love this abstraction, saves a lot of line of duplication. ❤
I did not give much importance since they were stories which may be changed later - so copy pasted it.
Thanks


64 or 32

hey guys, congratulation for the work. There is one small detail which we should ponder: see the screenshot.

I might have incorrectly named the 32 and 64 bit Linux executables. Looking into it now. Thanks!
On Aug 7, 2015 9:36 AM, "Noilson Caio T. de Araújo"<EMAIL_ADDRESS>wrote:hey guys, congratulation for the work. There is one small detail which we should ponder: see the screenshot.
—Reply to this email directly or view it on GitHub.

think nothing of it.

Fixed. Sorry about this. I accidentally switched the 32 & 64 bit hashes.  Not enough sleep at Defcon.


Python Voice Assistant , automated pyautogui

@king04aman
Assistant with more features
I hope you will assign this issue to me , As I will push my project to this repo under  hactoberfest tag.
Thank you.

Assigned!! Happy Coding 🥳


Add encoding param in config.from_pyfile

Enhance: When in windows, and the config file is hard code by utf8, flask.config.from_pyfile will not encoding correctly.
Now give a param to let the user to choose the encoding of the config file

It should honor the encoding markers but i would not be surprised if that function fails on python 3 because it opens in text mode.


How to add Contact without having phone number?

Hi
How can I add a new contact with it's Telegram ID? or Telegram username?
I've used ImportContacts method by leaving phone_number part free and setting the UserID value to TelegramID of the user that I wanted to add but it didn't work

How can I add a new contact?

Have you tried addContact?

How can I add a new contact?

Have you tried addContact?

Yes I've tried that

Yes I've tried that

Then you used it in a wrong way.

Then you used it in a wrong way.

Yes I tried it again I should send SharePhoneNumber parameter true for it to work

Yes I tried it again I should send SharePhoneNumber parameter true for it to work


Make --sign an alias to -s

Thanks!

Thanks @thefloweringash for the review. Can you make a new release with this please?

Sorry for the delay, I've pushed a tag: v0.1.3.

Thanks


Raising a test issue for contributing to the Open-source Community

This is the test issue I have created for testing purpose Please Ignore

Hi. Please don't create spam issues as they create noise for the maintainers. If you want to experiment, the best approach is to do so on your own fork.


Feature/refactor proto

Description
We hade multiple servicers defined in the proto (connector, control, reducer),  that is in the end all implemented in Combiner.  I think it makes the code more understandable if we merge them in combiner, and then if we want we can refactor the server in the future.

Contains SK-658 so that one should be merged first.


Run SonarTsPlugin AND SonarSource/SonarTS

Hi Team,
Is it possible to run together on a same instance your plugin and the new SonarSource/SonarTS one ?
Both plugins have the same name.
When I try to rename it, another problem occurs with the following message:
"java.lang.IllegalArgumentException: Multiple entries with same key: ts=TypeScript and ts=TypeScript "
Thanks

No, you can't. Even if you handle the same key issue, you'll face the same extension error - only one plugin can be registred to analyze ts extension. To make work both plugins, this plugin must be re-written to "custom rules plugin" api, extending the official one.

Now in SonarQube 7.4 seems like SonarTSPlugin does not working. Is that true? I uninstalled the SonarSource/SonarTS and even not worked.

Btw SonarSource/SonarTS now supports import of tslint output.

Yes, but I can not automatically run TSLint through sonar-scanner, and can not manage TSLint core and custom rules, right?


Running XDF.jl

The read_xdf function seems to work without problem for some .xdf files and not others. Listed below is the full error message from my Pluto notebook. Importantly, for the files that fail and the ones that succeed don't seem to have any reliable difference in size (given that they're from the same task performed by different participants) and compile the same number of streams and headers. Any insight would be appreciated.
Error message:
MethodError: no method matching read(::IOStream, ::Nothing)
Closest candidates are:
read(::IOStream)
@ Base iostream.jl:519
read(::IOStream, !Matched::Type{Float64})
@ Base iostream.jl:422
read(::IOStream, !Matched::Type{Float32})
@ Base iostream.jl:421
...
(::Main.var"workspace#18".myXDF.var"#1#3"{Dict{Int64, Int64}, Dict{Int64, Any}})(::IOStream)@my_XDF.jl:130
var"#open#424"(::Base.Pairs{Symbol, Union{}, Tuple{}, @NamedTuple{}}, ::typeof(open), ::Main.var"workspace#18".myXDF.var"#1#3"{Dict{Int64, Int64}, Dict{Int64, Any}}, ::String)@io.jl:396
open(::Function, ::String)@io.jl:393
read_xdf(::String, ::Bool)@my_XDF.jl:36
read_xdf(::String)@my_XDF.jl:33
top-level scope@Local: 8

Can you provide a minimal reproducible example with a file that does not work (i.e. no notebook) and then post the stack trace?
Did you try if e.g. pyxdf can read your files?

Hello,
The datasets that I've seen fail are participant data, so I won't be able to share any of those data and certainly not through a public forum. Perhaps I'll check with my supervisor if it's acceptable to share fully de-identified and encrypted data with you privately so you can take a look.
Otherwise, I have had no issues with pyxdf, but it is significantly slower in my experience than unpacking with Julia, so we're looking to potentially make a switch.

It would be great if you recorded some dummy data to reproduce the problem. Otherwise, it will be almost impossible to determine what the issue is.

How might you suggest sharing the file? The dummy data I have here is >100mb, so I can't attach it here nor push it to a repo.

Do you have Google Drive, OneDrive, Dropbox, etc.? If not, I can share an upload link.

This link doesn't work, it seems to be restricted for internal use.

Yeah, makes sense. Try this link

Makes sense. Try this link

Yes, that link worked!

Something's wrong when reading chunk 9 (stream ID 5, chunk type 3). Somehow, there's a length of 17 bytes where there should be either 1, 4, or 8 bytes. This could be due to two reasons (I can think of): (1) it's a bug or (2) string streams are not supported yet. I'm still trying to figure it out...

Hmm that's interesting. I imagine the issue itself isn't with the datatype being strings since I'm able to unpack our other data of the same format and with string streams.

It definitely has to do with your data containing a regularly sampled string stream. I have never seen this in the wild (and hence I don't have any testing data), so I did not implement support for it (yet). Of course, since these are perfectly valid in XDF, these should be supported, but someone will need to implement it. Currently, I don't have the time to do that, but I'll at least mention this limitation in the README.

Interesting. I'll try collecting a set of dummy data that can be successfully read to see if you can identify any systemic differences between the two datasets.

Oh, I'm just reading this now. Hi @lgarbar I implemented the string markers and can help you debug this if you're still interested.
Let me tell you how I'm picking up this issue: I downloaded the file you provide, a 481.1 Mb xdf. A stream, FaceVideo, with 25344 channels of type float32. Could you please confirm this is the file? I have a sha256 of "836ef458ba94bfa95a5fc1f42093cf671e012632435e5159cb12853335962dbe".
Upon reading chunk by chunk with the debugger I found no issue, but integrating it to tests I get weird error messages about the read() function call in L36
The function `read` exists, but no method is defined for this combination of argument types.

Closest candidates are:
  read(::Any, ::Type{Union{}}, Any...; kwargs...)
   @ Base io.jl:236
  read(::IO, ::Type{Char})
   @ Base io.jl:962
  read(::IO, ::Type{T}) where T<:Enum
   @ Base Enums.jl:22
  ...

These type of errors are common on my linux ditsros, and I usually solve them with a container. I'll test again the file you sent, but It would be of much help if you can tell us more about the origin of the data: was it created by a provider? the LibLSL libraries? Is it working if you use the pylsl?
Thanks for letting us know about the issue.
@cbrnr It might be better to pause #11 if this is a bug in the code. I hadn't seen this issue. Next time please feel free to tag me about bugs, specially if you think they might be in code I implemented. Still, I don't see string streams in the file, just the float32 stream. Is it the same file you revised?


Trainium blog post

Description of your changes:
Add Trainium blog post.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I see we have added this blog post before but we need to revert it because it uses a different scheduler which is not installed as part of Kubeflow. Customer might face issues using the instructions. We need to test it
@krhoyt can you please revert from main as well?


Line chart SideTitles problem

Hello all,
When using a value range below 1 as of [ 0.1 , 0.2 , 0.3, ..etc]  the getTitles function in the SideTitles doesn't work properly.
it only gives the minimum value even when removing the overriding and using the default function it only shows the minimum value.
code sample
leftTitles: SideTitles(
          showTitles: true,
          margin: 8,
          reservedSize: 30,
          getTextStyles: (value) => const TextStyle(
            color: Colors.indigo,
            fontWeight: FontWeight.bold,
            fontSize: 10,
          ),
          getTitles: (value) {
            // the only value i get here is the minmum
            print('value in get titles +$value');
            return value.toString();
          },
        ),

Screenshots

Versions

Flutter 2.2.3 stable channel
fl_chart: ^0.36.3

SideTitles.interval property is 1.0 by default.
Try to set 0.2 in your design.

@momenamiin  Thanks for asking this question, your question has a solution which solved my problem :)


Addition

Addition suggestion: an open foundation model for PPG data.
PaPaGei: Open Foundation Models for Optical Physiological Signals  https://arxiv.org/abs/2410.20542

Thanks for the suggestion! Just added it :)


Increment observation number when sending notification.

Fixes following errros:

IOTCLT-492 - Observe option is not incremented in notification response
IOTCLT-506 - Object/Object instance Notify is plain text in first ACK

@yogpan01


Fixed the service bus client factory resource release bug.

Fixed the service bus client factory resource release bug.

/azp run java - spring - tests


Issue with fsproj-file location

tl dr;
Could it be a reasonable default to look for fsproj-files in the current folder and any parent folders recursively?
According to the plugin documentation, opening an .fs-file will load any fsproj-file in that directory.
And some experimentation will show this to be true :)
With the following project structure:

Project.fsproj
Library.fs
Folder/File1.fs
Folder/File2.fs

Given that Library.fs opens modules from File1 and File2 and
given that File2 opens a module from File1.

I can open a vim-session for Library.fs an it will resolve any modules correctly because if will have loaded Project.fsproj.

If I start a new vim-session and start with opening  File2 it will not be able to resolve the module in File1 because Project.fsproj will not be loaded since it is located in a parent folder.

However if I start a new vim-session and open Library.fs (which will load Project.fsproj) and then open File2 it will resolve the modules in File1 since the project file already is loaded.

Could we solve 2. by letting the plugin search for fsproj-files in parent directories?

Yes within reason. Perhaps limited to 2 or 3 parent directories.
On Fri, 2 Jun 2017 at 08:56, Viktor Andersson<EMAIL_ADDRESS>wrote:

tl dr;
Could it be a reasonable default to look for fsproj-files in the current
folder and any parent folders recursively?
According to the plugin documentation, opening an .fs-file will load any
fsproj-file in that directory.
And some experimentation will show this to be true :)
With the following project structure:

Project.fsproj
Library.fs
Folder/File1.fs
Folder/File2.fs

Given that Library.fs opens modules from File1 and File2 and
given that File2 opens a module from File1.

I can open a vim-session for Library.fs an it will resolve any modules
correctly because if will have loaded Project.fsproj.
2.
If I start a new vim-session and start with opening File2 it will not
be able to resolve the module in File1 because Project.fsproj will not be
loaded since it is located in a parent folder.
3.
However if I start a new vim-session and open Library.fs (which will
load Project.fsproj) and then open File2 it will resolve the modules in
File1 since the project file already is loaded.
Could we solve 2. by letting the plugin search for fsproj-files in parent
directories?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fsharp/vim-fsharp/issues/79, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABIDlNpecqAmmvYHSibwzRCoEiWTMlDKks5r_7IbgaJpZM4Nt5lO
.

actually the limit should be configurable with a default that retains the current behaviour (only scanning the same dir as the file).

Sure, that would work great.
Just for information I ran into the issue when I was using the dotnet cli template for fable-elmish-react. That template is using a project structure with the .fsproj-file in the root-directory and all .fs-files in a src-subfolder. So when opening any .fs-file in the src-folder it never would be able to load the .fsproj-file.

@kjnilsson the search of fsproj is now supported fsac side (ref https://github.com/fsharp/FsAutoComplete/pull/191 ) and support also sln files.
So should help implement this feature (just run workspacepeek at startup).
Vim need to iterate projects and call project. This will be improved by https://github.com/fsharp/FsAutoComplete/issues/192 later, doing whole init fsac side


Create packege from py_module, Separate JS code

@nyirog For your suggestion I tried to add the lines below into init.py (in order to not braking the api):
from ipyvizzu.chart import Chart from ipyvizzu.animation import Data, Config, Style
After that I can import in the previous format in the notebooks:
from ipyvizzu import Chart, Data, Config, Style
But I can not import in the new format (from ipyvizzu.chart import Chart -> ipyvizzu is not a package).
Did I do something wrong or it has to work that way?

If we would import the public classes and functions (interfaces are not required) in ipyvizzu/__init__.py, then we would not break the interface of ipyvizzu.

@nyirog I updated ini.py, and restore import format in the docs. Could you please check init.py

@nyirog For your suggestion I tried to add the lines below into init.py (in order to not braking the api): from ipyvizzu.chart import Chart from ipyvizzu.animation import Data, Config, Style
After that I can import in the previous format in the notebooks: from ipyvizzu import Chart, Data, Config, Style
But I can not import in the new format (from ipyvizzu.chart import Chart -> ipyvizzu is not a package).
Did I do something wrong or it has to work that way?

I checked out the js_separation branch and I could import Chart from ipyvizzu package and from ipyvizzu.chart module, too. I used Python 3.8.12.

If we would import the public classes and functions (interfaces are not required) in ipyvizzu/__init__.py, then we would not break the interface of ipyvizzu.

@nyirog I updated ini.py, and restore import format in the docs. Could you please check init.py

Looks good.
I think in the __init__.py the relative import may be more readable.

@nyirog For your suggestion I tried to add the lines below into init.py (in order to not braking the api): from ipyvizzu.chart import Chart from ipyvizzu.animation import Data, Config, Style
After that I can import in the previous format in the notebooks: from ipyvizzu import Chart, Data, Config, Style
But I can not import in the new format (from ipyvizzu.chart import Chart -> ipyvizzu is not a package).
Did I do something wrong or it has to work that way?

I checked out the js_separation branch and I could import Chart from ipyvizzu package and from ipyvizzu.chart module, too. I used Python 3.8.12.

@nyirog I see, thanks. I think I messed up something previously.


Scroll wheel not working at all.

Microsoft PowerToys version
0.57.2
Running as admin

[X] Yes

Area(s) with issue?
General
Steps to reproduce
i try to instinctively scroll with the mouse wheel as per eons of training hath me acustomed to only to notice that it is disabled on each and every tab on the main program. only the sidebar lets me scroll.
✔ Expected Behavior
wheel scrolls,  window scrolls with it, as any other program in windows.
❌ Actual Behavior
nothing.  though if i use the scroll function of my mouse it kinda bobs up and down. you know the one that makes a grabby hand with a cursor.
Other Software
x-mouse button control
lg hub

x-mouse is the offender here and is a known issue.
/dup https://github.com/microsoft/PowerToys/issues/3400


Feature.adding _f5 and common 5

Issues:
Fixes #5
Problem:
That the _f5.py and the common.py exists in multiple repos; however, it
is not share across a common repo.  This repo, f5-cccl overlaps on the
functionality of these files; thus, we need to consolidate the location
of these modules
Analysis:
By adding this in a shared module of f5-cccl, we offer a single source
of truth for both of these modules.  These modules will be the starting
point moving forward for velcro on porting over to new work in this
repo.
The change I made was:

Migrated the modules over into the f5-cccl
These modules can be imported via import f5_cccl._f5.py
Or import f5_cccl.common

Tests:
There did not exist any unit tests within the two repos that these files
came from; thus, there is no test here yet besides import-ability via
the f5_cccl/init.py file and its associated test_f5_cccl.py py.test.
Reviewer:
@richbrowne
@michaeldayreads

This follows the git commit template but I think there's a shortcoming with the template format.  Because "Issues:" is its own line, the one-liner summary of every commit is "Issues:", which prevents rapid history browsing.
I think we should adjust the commit template (maybe not in this commit, but I'm going to open an issue) so that the headline in this case would be "Fixes #5: Import _f5.py and common.py" or something like that.

Sorry for the pollution, guys...

There is still some cruft from issue#1.

To me I still see the first commit (although it's empty) - we should get rid of that.

@andrewjjenkins @richbrowne The first commit has been removed.  Please let me know if you see otherwise somewhere...  I've got a few other personal branches/forks to clean from this as well...


Update CMakeLists.txt which fixes debug build undefined reference errors

Building with debug
./build/astra_analytical/build.sh -d

gives errors while linking:
ld: [100%] Linking CXX executable ../bin/AstraSim_Analytical_Congestion_Aware
../lib/libAstraSim.a(et_feeder.cpp.o): in function `std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >* absl::lts_20230802::log_internal::MakeCheckOpString<int const&, int const&>(int const&, int const&, char const*)':
env/include/absl/log/internal/check_op.h:292: undefined reference to `absl::lts_20230802::log_internal::CheckOpMessageBuilder::CheckOpMessageBuilder(char const*)'
env/bin/../lib/gcc/x86_64-conda-linux-gnu/11.4.0/../../../../x86_64-conda-linux-gnu/bin/ld: env/include/absl/log/internal/check_op.h:294: undefined reference to `absl::lts_20230802::log_internal::CheckOpMessageBuilder::ForVar2()'
env/bin/../lib/gcc/x86_64-conda-linux-gnu/11.4.0/../../../../x86_64-conda-linux-gnu/bin/ld: env/include/absl/log/internal/check_op.h:295: undefined reference to `absl::lts_20230802::log_internal::CheckOpMessageBuilder::NewString[abi:cxx11]()'

# skipped for brevity

Following https://github.com/protocolbuffers/protobuf/issues/12292 issue discussions:

I believe the issue is that something like find_package(protobuf REQUIRED) uses the embedded FindProtobuf package from CMake:
https://cmake.org/cmake/help/latest/module/FindProtobuf.html
This package has not been updated to know about the Abseil dependency. One needs to use:
find_package(protobuf REQUIRED CONFIG)

and https://github.com/protocolbuffers/protobuf/issues/4806:

You should not need any of these variables. Just link against protobuf::libprotobuf target, it will also find the headers correctly!

This PR fixes the problem in my environment.

Hello, @ChenYuHo.
I am experiencing the following error when I build ASTRA-sim with your PR.
$ git clone --recurse-submodules git@github.com:ChenYuHo/astra-sim.git
$ cd astra-sim
$ git checkout patch-1
$ docker build -t astra-sim . && docker run -it astra-sim
root@5d065abfb9ae:/app/astra-sim# ./build/astra_analytical/build.sh
-- The C compiler identification is GNU 9.4.0
-- The CXX compiler identification is GNU 9.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at ../../CMakeLists.txt:17 (find_package):
  Could not find a package configuration file provided by "Protobuf" with any
  of the following names:

    ProtobufConfig.cmake
    protobuf-config.cmake

  Add the installation prefix of "Protobuf" to CMAKE_PREFIX_PATH or set
  "Protobuf_DIR" to a directory containing one of the above files.  If
  "Protobuf" provides a separate development package or SDK, be sure it has
  been installed.

-- Configuring incomplete, errors occurred!
See also "/app/astra-sim/build/astra_analytical/build/CMakeFiles/CMakeOutput.log".
root@5d065abfb9ae:/app/astra-sim# ./build/astra_analytical/build.sh
CMake Error at ../../CMakeLists.txt:17 (find_package):
  Could not find a package configuration file provided by "Protobuf" with any
  of the following names:

    ProtobufConfig.cmake
    protobuf-config.cmake

  Add the installation prefix of "Protobuf" to CMAKE_PREFIX_PATH or set
  "Protobuf_DIR" to a directory containing one of the above files.  If
  "Protobuf" provides a separate development package or SDK, be sure it has
  been installed.

-- Configuring incomplete, errors occurred!
See also "/app/astra-sim/build/astra_analytical/build/CMakeFiles/CMakeOutput.log".

I'm using an conda environment which is built with
mamba create --strict-channel-priority --yes -p ./env \
gdb fsspec gitpython scipy future numpy nvidia-apex setuptools python-graphviz graphviz networkx pydot pip cuda-cudart-dev cuda-toolkit cuda-nvcc cuda-version=11.8 cuda-libraries-dev make openmpi mpi cmake mkl mkl-include pkg-config git boost-cpp libboost-devel libprotobuf=4  protobuf=4 \
transformers datasets tqdm sentencepiece sentencepiece-python `# for pytorch examples FSDP` \
pytorch=2.1 torchvision torchaudio cuda pytorch-cuda=11.8 compilers sysroot_linux-64==2.17 gcc==11.4 ninja py-cpuinfo libaio ca-certificates certifi openssl python=3 pydantic `#deepspeed` \
-c pytorch -c nvidia/label/cuda-11.8.0 -c conda-forge

I believe the issue stems from how CMake handles protobuf libraries, and since I'm using a newer CMake from conda, maybe this error only happens with newer CMake, and my patch thus applies only to that case.

It seems that ProtobufConfig.cmake or protobuf-config.cmake is not installed by default. When building ASTRA-sim in debug mode, I encountered a different error than what you provided. It appears that the issue isn't confined to debug mode alone but rather pertains to linking errors encountered during the ASTRA-sim build process.

We're closing this PR for now. If the issue still persists, please resubmit a new issue/PR and we'll have another look at a new environment.


fix: apiRoute should have consistent forward slash

windows fix

Codecov Report
Base: 85.81% // Head: 85.83% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (1a0aa21) compared to base (13a77b3).
Patch coverage: 83.33% of modified lines in pull request are covered.

:exclamation: Current head 1a0aa21 differs from pull request most recent head 7fac3e5. Consider uploading reports for the commit 7fac3e5 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #160      +/-   ##
===========================================
+ Coverage    85.81%   85.83%   +0.01%
===========================================
  Files           82       82
  Lines         1445     1447       +2
  Branches       143      134       -9
===========================================
+ Hits          1240     1242       +2
  Misses         203      203
  Partials         2        2

Impacted Files
Coverage Δ

src/faas/v0/context.ts
76.22% <50.00%> (ø)

src/resources/api.ts
85.08% <100.00%> (+0.26%)
:arrow_up:

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.


ObjectDisposedException when trying to call ChatAsync

Description
I have copied the code from the Readme.md fil into a console application, and it works fine. However, I want to use it in a Blazor web app. I don't think Blazor is the issue here, as the same problem would (presumably) occur if I were to use WinForms, WPF, etc.
I used basically the same code, but split off the bit where the user queries the model into a separate method so it could be called from a button click. I ended up with...
using LLama;
using LLama.Common;
using LLama.Sampling;

namespace LlmPlay.Pages;

public partial class Index {
  private string _modelPath = Path.Combine(Directory.GetCurrentDirectory(), "Data", "WizardLM-2-7B-Q5_K_S.gguf");
  private ChatSession _session = null!;
  private InferenceParams _inferenceParams = null!;
  private string _input = "";
  private string _output = "";

  protected override void OnInitialized() {
    //return;
    ModelParams parameters = new(_modelPath) {
      ContextSize = 1024, // The longest length of chat as memory.
      GpuLayerCount = 5 // How many layers to offload to GPU. Please adjust it according to your GPU memory.
    };
    using LLamaWeights model = LLamaWeights.LoadFromFile(parameters);
    using LLamaContext context = model.CreateContext(parameters);
    InteractiveExecutor executor = new(context);

    // Add chat histories as prompt to tell AI how to act.
    ChatHistory chatHistory = new();
    chatHistory.AddMessage(AuthorRole.System, "Transcript of a dialog, ..");
    chatHistory.AddMessage(AuthorRole.User, "Hello, Bob.");
    chatHistory.AddMessage(AuthorRole.Assistant, "Hello. How may I help you today?");

    _session = new(executor, chatHistory);

    _inferenceParams = new InferenceParams() {
      MaxTokens = 256,
      AntiPrompts = new List<string> { "User:" },
      SamplingPipeline = new DefaultSamplingPipeline()
    };
  }

  private async Task Ask() {
    await foreach (string s in _session.ChatAsync(new ChatHistory.Message(AuthorRole.User, _input), _inferenceParams)) {
      _output += $"\n\n{s}";
    }
  }
}

This is a direct copy from the Readme.md file, with a couple of comments removed to avoid too much horizontal scrolling.
The markup looks like this...
<div>
  <input type="text" @bind-value="@_input" @bind-value:event="oninput" /> <button @onclick="Ask">Send</button>
</div>
<textarea style="width:700px; height: 700px">@_output</textarea>

As you can see, _input is bound to an HTML <input> element in the markup, and _output is bound to the content of a <textarea> element.
However, when the Ask method gets called, I get an exception...

System.ObjectDisposedException: Cannot access a disposed object.
Object name: 'Cannot use this SafeLLamaContextHandle - it has been disposed'.

The line of code in question is the first line in Ask.
Anyone able to advise how I would query the model in such a setup? All the samples I saw seem to assume you're using a console app, which is obviously a simple way to demo features, but not what I'd do in a production app.
Thanks

The problem is here:
using LLamaWeights model = LLamaWeights.LoadFromFile(parameters);
using LLamaContext context = model.CreateContext(parameters);

When model and contxt go out of scope (i.e. the end of that method) they are disposed, but then you try to use them in the Ask method. The use-after-free error is turned into an ObjectDisposedException instead.

@martindevans Duh, silly me! Didn't notice the usings there. Thanks


fix: wrap multiple-partition-consumer correctly

For both the Fluvio.multi_partition_consumer and Fluvio.multi_topic_partition_consumer methods, the PyO3-types were wrapped in a PartitionConsumer instead of a MultiplePartitionConsumer type.
This PR fixes this inconsistency.

Thanks @dariogoetz, looks good!


Reference a Case-Study from OpenHIE

Are we looking for a country case study example (which are not simple/trivial to do... b/c we typically need MOH permission)? It may be easier -- rather than a case study -- to prepare an example of how the OpenHIE architecture "works" and the ways it aligns with, for example, MHDS. Would that approach be OK?

Mostly highlevel description of OpenHIE and possibly links to some external descriptions.

@djritz have you made progress on getting an OpenHIE case-study published so that we can reference it?

beyond bringing it up on the most recent (semi-monthly) OpenHIE architecture call, no. i haven't authored anything and won't have time to do that until late august (i'm buried in multiple very-active projects). it may be worthwhile to survey some existing content and see if any of it is on point. @lduncan-intrah -- do you know of any papers (e.g. Instant OpenHIE) that may be applicable?

https://digitalprinciples.org/wp-content/uploads/PDD_CaseStudy-OpenHIE_v3.pdf

@djritz besides pointing at the published CaseStudy on OpenHIE... I think we were wondering if you could write a paragraph that would bridge our paper to that case study.

@djritz please confirm if you are going to write a paragraph introduction to the OpenHIE paper?

@JohnMoehrke -- yes... I will write a paragraph that "introduces" the paper. In truth... a lot has happened since this paper was published, so one of the jobs of that "bridge" will be to point to those differences and to where the most up-to-date info can be found. Too busy to get to that for the next couple of weeks, tho. Sorry... day-job stuff is very heavy lately...

Excellent. Goal is to get this done by end of October, so that it is ready for our November meeting.

@JohnMoehrke -- here is a draft introductory paragraph:
In 2012, an initiative was launched, with support from the US President's Emergency Plan for AIDS Relief (PEPFAR; https://www.state.gov/pepfar/), to develop an open source health enterprise architecture that can be adapted and adopted by low and middle income countries (LMIC) to support national scale health data sharing. The Open Health Information Exchange (OpenHIE; https://ohie.org/) framework describes a comprehensive family of digital health workflows expressed in terms of IHE Profiles. A case study of OpenHIE, and version 1 of its architecture, was published by the Principles for Digital Development: https://digitalprinciples.org/wp-content/uploads/PDD_CaseStudy-OpenHIE_v3.pdf.  The latest version (v2) of the OpenHIE architecture, which is based on the HL7 FHIR specification (as profiled by IHE's Mobile Health Document Sharing (MHDS) specification), can be found here: https://ohie.org/architecture-specification/.

I will put this in place of the text already describing OpenHIE.


Fix warning and error messages

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #38760
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

We should also stop printing the generated JVM executable at https://github.com/ballerina-platform/ballerina-lang/blob/2201.3.0-stage/cli/ballerina-cli/src/main/java/io/ballerina/cli/task/CreateExecutableTask.java#L132


It would be better to adjust the size to remove the scroll bar in the 'Shared Access Signature' dialog when the access policy field has 63 characters

Storage Explorer Version: 1.23.0-dev
Build Number: 20220113.2
Branch: hotfix/1.22.1
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.0.1
How Found: Ad-hoc testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Blob Containers.
Create a blob container.
Right click the blob container -> Click 'Manage Access Policies...'.
An 'Access Policy' dialog pops up -> Click 'Add' -> Type 63 characters to Id field -> Click 'Save'.
Right click the blob container -> Click 'Get Shared Access Signature...'.
A 'Shared Access Signature' dialog pops up.
Check there is no scroll bar.

Expected Experience
There is no scroll bar.
Actual Experience
There is an scroll bar.

Additional Context

This issue doesn't reproduce for tables.
There are two scroll bars for file shares/queues.

There are two scroll bars for blob containers when pulling the dialog horizontally.

I am going to won't fix this issue for now since this is a very edge case.


feat(Header)!: reroute header upsell for upgrade

Overview

This leverages the checkout url both for trial and non-trial upsells in the header (tryProForFree and upgradeToPro) instead of routing upgradeToPro to the pro membership landing page

PR Checklist

[ ] Related to designs:
[x] Related to JIRA ticket: GROW-2331
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

Egg friends who worked on the header :)

📬Published Alpha Packages:@codecademy/gamut-kit@0.3.97-alpha.0b9402.0@codecademy/gamut-labs@16.0.2-alpha.0b9402.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://609c169890061b0f94e33721--gamut-preview.netlify.appDeploy Logs

sorry wasn't ready for review yet just wanted to test it out with an alpha!

📬Published Alpha Packages:@codecademy/gamut-kit@0.3.97-alpha.2879cd.0@codecademy/gamut-labs@16.0.2-alpha.2879cd.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://609c1b2cd984a500c65f3f9d--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.3.97-alpha.221380.0@codecademy/gamut-labs@16.0.2-alpha.221380.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.3.101-alpha.782ef4.0@codecademy/gamut-labs@16.0.6-alpha.782ef4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://609e78a2c6e80d11787c8d9d--gamut-preview.netlify.appDeploy Logs


Version change from 10.1.1 to 10.1.2

Changes

[x] Changed package version to 10.1.2.

Pending release references

#123

Regards.

Codecov Report

Merging #125 (7463d80) into master (194ef70) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #125   +/-   ##
=======================================
  Coverage   92.12%   92.12%
=======================================
  Files         254      254
  Lines        5384     5384
  Branches      868      868
=======================================
  Hits         4960     4960
  Misses        251      251
  Partials      173      173

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Only show finished solutions

As devonfw team, I want the user only to see finished solutions so that the architecture browser makes a good first impression.
There should be a parameter to activate/show the "Maturity level" filter so unfinished solutions can be shown for debugging/preview

To see the "Maturity level" filter you can add the GET parameter show=maturity

If we have no entry for a filter (disabled), then the filter shouldn't be visible.


404 in slugable records

a basic solution
static::getModel()::where(self::getRouteKeyName(), $record)->firstOrFail();
https://github.com/laravel-filament/filament/blob/9414a1aa80ae20b07e29707916c9d408efbe50e9/src/Resources/Pages/EditRecord.php#L58

Sorry, what's the issue you're facing?

Fixed by #167.


Auth::viaRemember() always returns false

Laravel Version: 7.26.0
PHP Version: 7.3.3
Database Driver & Version: 10.1.38-MariaDB

Description:
I'm trying to check if my Keep session active box works.
It does not matter if you check it or not, the Auth::viaRemember() function returns false.
Steps To Reproduce:
use Illuminate\Support\Facades\Auth;
....
 public function home()
    {
        dd(Auth::viaRemember());
          ....
    }

Hey there,
Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to report back and I'll gladly help you out and re-open this issue.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!

I already did, I did not find a solution anywhere, if there is one without modifying the framework. sorry for escaping my attention.

Something I do NOT like about laravel Issues is that they don't even leave it open for discussion, they give a short answer and close it.

@arce701 please understand that we can't provide support for each individual issue that can be asked on a support channel. We have dozens of open source repos to maintain and also need to work on new releases. We have a large and active community out there that can help out.


Exec with the Python binary of currently active virtualenv

This pull requests updates pipdeptree to run itself with the Python binary of the currently active virtualenv. This should fix #2 and #11.
I  have tested it on Python2.7 (Windows and Ubuntu 12.04, Ubuntu 16.04) and Python3.5 (Ubuntu 12.04, Ubuntu 16.04), I haven't written tests, would write them if you agree with the changes. I also think we will need a different strategy to test this change since this involves re-execing the currently running binary, your inputs on the same would be valuable.

Not sure why the render PDF and render SVG tests are failing, will look into them

I got a chance to look into the test failures, they are unrelated to my change. It seems they are failing because pipdeptree adds a newline to package labels while creating graphviz nodes, which in turn generates something like the following
digraph {
    Lookupy [label="Lookupy
             0.1"]
    redis [label="redis
           2.9.1"]
    ...
}

Add graphviz exits with a non-zero error code complaining about syntax error and a lot of warnings (not familiar with dot syntax but I think it is because of new-lines in the labels), here is the output from dot

Warning: test.dot:2: string ran past end of line
Warning: test.dot:3: string ran past end of line
Error: test.dot:4: syntax error near line 4
context:     redis >>>  [ <<< label="redis

Causing test failures, I tested after removing the newline  and it seems to work great.

First of all this is great! Thanks for putting the effort.
I don't see any concerns with the approach, so you may proceed and add tests. IMO, mocking will simplify testing of this function. os.execv can be mocked to assert that it's being called with correct args on different platforms and with different cli args (we should also be able to mock sys.argv)
Besides this, one case I can think should be considered is when the virtualenv is created with a PYTHON_EXE (see the -p option of virtualenv) different from the system default. For eg. if the system wide default python is python2.7 and the virtualenv is created with python3.5. Now if system installed pipdeptree gets run in such a virtualenv, the reexec_in_virtualenv_maybe will run it with python3.5, whereas pipdeptree was originally installed for python2.7. This isn't a problem as long as both versions are supported but any issues arising because of this would be hard to debug. What do you think?

Hi @naiquevin,
Really sorry for the delayed reply, was sort of busy with other things.

Besides this, one case I can think should be considered is when the virtualenv is created with a PYTHON_EXE (see the -p option of virtualenv) different from the system default.

Actually this did cause a bit of a issue especially when because the __file__ pointed to the compiled file in Python 2, to workaround that I explicitly strip the trailing 'c' if the __file__ has a pyc extension. It should not be a problem in later versions since __file__ always points to original file.

This isn't a problem as long as both versions are supported but any issues arising because of this would be hard to debug. What do you think?

Yes that is a valid issue, this was written with the assumption that pipdeptree would be compatible with both Python 2 and Python 3 (in fact the current workaround also depends on this) but I can see that this might get difficult to maintain. Do we have any other option?

I don't see any concerns with the approach, so you may proceed and add tests.

I will add some tests this weekend
Thanks

Closing as this seems to have stalled.


server: Add unit tests from Diffblue Deeptest

Pre-PR review request:
Existing coverage:      42.15%
Increase in coverage: 15.84%
Verfied tests:               2467
Tests for PR:                   15

👍 They are great! Add the licence and commit using difflbue-assistant + John suggestions and you are good to go.

Agreed with John about the DeeptestUtils, but other than that they're gawjus tests :kiss:

Thanks for the comments. All changes made and pushed.
Will file PR.


Os Build: 18945 Can't access local development via domain name in hosts file

I love that localhost has been implimented in 18945 but we also need maping to <IP_ADDRESS> for use with the hosts file as this still isn't working so I still need to get the ip address each time i boot or shutdown wsl.

Hello. We are facing similar problem with wsl2 ports mapped to [::1] and not to <IP_ADDRESS> cause some of our libraries can't work with ipv6.

Is there a way to represent this in the hosts file?

@scrossan-crown You can do this in your hosts file
<IP_ADDRESS> yourdomain.local.com
::1 yourdomain.local.com

@Drakota I have tried that but it didn't work for me

@scrossan-crown
At the moment there's alot of issue with localhost being inconsistent and not working sometimes so I think you're affected by this.
See
#4347
#4340

doesn't work for me
<IP_ADDRESS> yourdomain.local.com
::1 yourdomain.local.com

@Drakota's solution works in 19018

@Drakota's solution works in 19018

Yes it is working but it so slow now. In WSL 1 websites under localhost was really fast but WSL is too slow in this topic.

It is necessary to move website to linux directory etc /home/username and then website is really fast now. But it would be nice to have same speed in /mnt/c

@ZdenekKrcal For WSL 2, it is necessary to move it to a Linux partition. IIRC they are also working on improving this.


[SMALLFIX]Use static imports for standard test utilities in AlluxioFuseFileSystemTest.java

Use static imports for standard test utilities in AlluxioFuseFileSystemTest.java

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17193/
Test PASSed.


Adds input for upload chunk size

Adds input to limit the upload chunk size.  This is useful for self-hosted runners with limited upload speeds, as an upload taking longer than the server timeout will fail.  We used to have this configurable via an environment variable, but that was removed when splitting the cache action out into the cache module.
- name: Limit upload chunk size
  uses: actions/cache@v2
  with:
    path: **/node_modules
    key: ${{ runner.os }}-${{ hashFiles('**/lockfiles') }}
    upload-chunk-size: 4000000

Fixes https://github.com/actions/cache/issues/415

@aiqiaoy No particular reason.  If I had to give a reason, it would be for simplicity.  This puts all arguments under with: instead of splitting some out under env:.  In the end, inputs become environment variables anyway 😄


Downgrade dart sdk minimum version

Hello.
Thank you for your effort in making this package.
it would be great to have the package working on previous versions of Dart.
so instead of
environment:
  sdk: '>=2.12.0 <3.0.0'
  flutter: ">=1.10.0"

would be
environment:
  sdk: '>=2.7.0 <3.0.0'
  flutter: ">=1.10.0"

the issue with that is I need to update flutter to version 2.x to have this package working.

I really don't understand why you still stuck in that problematic dart version. I've done a bigger project with even the latest version as at now which I started at the start of 2019, I really wanted to support null safety.

You can use the previous versions of the library instead

Never mind, I had to update to version 2 anyway.
Thank you.


Missing "encodeURIComponent" when linking to Linkedin and Twitter

Describe the Issue
I think there's an issue where not all info is transfered from freeCodeCamp to Linkedin when trying to add a new certification.
From what I gather in src/client-only-routes/show-certification.tsx you need to add encodeURIComponent for both the twitter(x) and the linkedin links:

  const shareCertBtns = (
    <Row className='text-center'>
      <Col xs={12}>
        <Button
          block={true}
          bsSize='lg'
          bsStyle='primary'
          href={`https://www.linkedin.com/profile/add?startTask=CERTIFICATION_NAME&name=${encodeURIComponent(certTitle)}&organizationId=4831032&issueYear=${certYear}&issueMonth=${
            certMonth + 1
          }&certUrl=${encodeURIComponent(certURL)}`}
          target='_blank'
        >
          {t('profile.add-linkedin')}
        </Button>
        <Spacer size='medium' />
        <Button
          block={true}
          bsSize='lg'
          bsStyle='primary'
          href={`https://twitter.com/intent/tweet?text=${encodeURIComponent(t('profile.tweet', {
            certTitle: certTitle,
            certURL: certURL
          }))}`}
          target='_blank'
        >
          {t('profile.add-twitter')}
        </Button>
      </Col>
      <Spacer size='large' />
    </Row>
  );

Affected Page
https://www.freecodecamp.org/certification
Steps to Reproduce

Finish the C# certification
Try to share it on Linkeding
Notice that a lot of the information is not included
3.5. feel your spidey sense tingle since you spot a '#' in the url

Expected behavior
Expect that all the relevant information is transfered to Linkedin
Screenshots
No response
System
Firefox, Ubuntu
Additional context
No response

Yes, sorry for the slow response. Here are two screenshots showing the difference:
After clicking "Add this certification to my LinkedIn profile"

Copying the link and manually removing # from the link:


Deploy the Project to Server

Hello,
I am trying to Deploy this project to server,
but after i install the pm2 i try to run this command:

pm2 start node_modules/.bin/gulp

    "scripts": {
        "start": "node_modules/.bin/gulp watch"
    },

but i dont success to deploy this
when i try to do

npm start

it is work locally ,
what the problem??

Gulp has been deprecated, this was replaced with npm scripts.


Azazel's quest not appearing in guide post DFF gate

I see that this isn't shown unless you spend an adventure in an area. Got it!


Unmount Container rootfs after successful start

This patch series fixes issue #11113

Re-pushed the branch after taking care of some of the comments of @rhatdan

Hmm.., something is wrong with jenkins? It has been saying "Jenkins build is being scheduled" for last 10 minutes now.

theres a large build queue at the moment and your PR has 9 commits, it tests each individually

Hmmm..., tests failed.
---> Making bundle: test-integration (in bundles/1.5.0-dev/test-integration)

go test -test.timeout=45m github.com/docker/docker/integration

github.com/docker/docker/integration
./utils_test.go:77: not enough arguments in call to c.Unmount
FAIL	github.com/docker/docker/integration [build failed]
There is one more call to UnMount(). This needs to be modifed to pass in "false" as parameter. Will fix it.

Pushed the patches again with the fix for last build failure.

@rhvgoyal it might be better to squash some commits since reducing the commit count may help reducing the stress on CI machines. @jfrazelle what do you think?

yes but since these are already in the queue squshing and force pushing again will just add to the queue more

@ahmetalpbalkan
Personally I like to keep commits small and logically contained unit. Even if take more time to build. that way it helps review patches and in long term one can go back to commits and figure out more easily what went in and why.
But every project is different. So if docker project prefers to combine commits and keep number of commits small, I can do that.

Thank you @rhvgoyal. I understand your taste for separated commit, but that's not what the Docker project encourages. Please squash in 1 if you don't mind!

@icecrime
Ok, I will put all this into one patch and push again.
But I seriously disagree with this methodology. I just can't review a single big patch and not sure how other people can review such changes.  Logically broken down commits with proper commit messages are so much easier to review. People can easily get to the core changes without having to worry about not so much important details.
And I was reading the link you pointed me to. I could not find out where does it say that put everything into a single commit. I could find following though.
" Before you make a pull request, squash your commits into logical units of work using git rebase -i and git push -f. After every commit, make sure the test suite passes. Include documentation changes in the same pull request so that a revert would remove all traces of the feature or fix".
So as per above, you do expect multiple commits in a pull request. Only question is what is considered a "logical unit of work" and I could not find guidelines for what is a logical unit of work. I think I did break down patches into multiple logical unit of work. That's a different thing that somebody might disagree with my organization and say hey i only see 3 logical units and not more.

I'll update the CONTRIBUTING.md to define a "logical unit of work".

Ok, pushed one more time. This time merged all the patches into single patch as asked by @icecrime

@jfrazelle
This "make test-docker-py" failure is happening even without my patches on my local machine. So doe not look like anything related to my patches.

I think it should be fixed now
On Mon, Mar 16, 2015 at 1:14 PM, Vivek Goyal<EMAIL_ADDRESS>wrote:

@jfrazelle https://github.com/jfrazelle
This "make test-docker-py" failure is happening even without my patches on
my local machine. So doe not look like anything related to my patches.
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/11114#issuecomment-81911654.

@jfrazelle
How do I restart my failed job?

restarted the builds

@jfrazelle  @vbatts @cpuguy83
I am wondering how can I make some progress on this pull request. Any thoughts?

@rhvgoyal I believe the Windows failure is genuine - could you have a look please?

This is also the solution for following issue.
https://github.com/docker/docker/issues/10991
But now I prefer the alternate solution of deferring device removal posted here.
https://github.com/docker/docker/pull/12190

closing in favor of #12190


converted to react application

🛠 Issue (2004)

Closes #2004
Description
Title: Convert vanilla HTML, CSS, and JS website to React
In this pull request, I have converted our existing website built with HTML, CSS, and to a React application. This conversion will allow us to take advantage of the many benefits of using a component-based architecture, such as improved code reusability, maintainability, and testability.
Here are some highlights of the changes made:
Created a new React app using create-react-app.
Converted all existing HTML components to React components.
Extracted reusable styles to separate CSS files and imported them into the relevant components.
Refactored existing JavaScript logic to use React hooks and state management.
Please note that due to the conversion, there may be some minor visual differences between the old and new versions of the site. I have tried to maintain the same look and feel, but there may be some discrepancies due to differences in how CSS is applied in React.
some of the projects are not opening due to html parsing error.
I have thoroughly tested all components and functionality to ensure that the new React version of the site works as expected. However, please let me know if you notice any issues or have any questions or concerns.
Please include a summary of the change. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change

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

Checklist:

[ ] I have made this from my own
[ x] I have taken help from some online resourses
[ x] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ x] The title of my pull request is a short description of the requested changes.

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Not required


Sign in request

After install vscode pops up: Mermaid Chart wants to sign in:

What is this?

I have to comment that this popup is a nuisance. I'm sorry, but I don't want to have to sign into MermaidChart. Please provide a way to disable the dialog permanently.

Also, this is a required step, and even after signing in it gives a redirect error and the plugin never finishes installing.

Also, this is a required step, and even after signing in it gives a redirect error and the plugin never finishes installing.

This still isn't supported unfortunately in cursor.


Set-AzureRmStorageAccount is wrong

The --force parameter's information is missing. It shows the info for the access tiers instead. It actually looks as if the information for the -Force parameter and a parameter name after that got accidentally removed.
Also, same thing for -Tag. The information is wrong, and the parameter Kind is missing. (It has the explanation for Kind right after the heading for -Tag.)

@robinsh - sorry for the delay in response.  Can you please take a look at the latest version for this cmdlet and see if the issue is resolved:  https://docs.microsoft.com/en-us/powershell/module/azurerm.storage/set-azurermstorageaccount.
PS:  These values are towards the bottom of the page in the optional parameters section.
Thanks!

Under "-KeyName", what the heck does this mean: "Storage Account encryption keySource KeyVault KeyName" ?
Also, note that "Storage Account" is not a product, and should not be capitalized. Of course if it starts the sentence, it should be "Storage account ... "
Under "-KeyVaultUti", what does "Storage Account encryption keySource KeyVault KeyVaultUri" mean? Same question for "KeyVersion".
Under -Name", why is modify capitalized?
Under "-AccessTier", the link to the hot and cold storage tiers information is messed up.
Under "-DefaultProfile", please capitalize Azure.
Under "-DisableEncryptionService", it's "Storage Service Encryption" -- this is a feature -- and you can remove "on the Storage Service". That's redundant. Also, should be "Azure Blob storage and Azure Files are supported."  They rarely refer to them as services, and Files was rebranched at Ignite last September.
Under "-EnableEncryptionService", should be "Storage Service Encryption" and remove "on the Storage Service". Also, "Azure Blob storage and Azure Files are supported."
Under "-EnableHttpsTrafficOnly", lowercase storage account, and it should be "enables", not "enable". And HTTPS.
Under "

@robinsh - sounds like you need to talk to the devs that created this module; they own the cotent.  I created this pr for you to track in the source repo:  https://github.com/Azure/azure-powershell/issues/4979.

@TaraMeyer Ok, that's fine. Someone told me to post errors as issues to the GitHub, repo, and I didn't look at the other variables at first, hence my comments.

@robinsh - no worries.  If you don't get much traction in the next few weeks on the pr above, let me know and I see if I can get you directly in touch with the engineering team.

@robinsh  - I see this update is still open in the src.  I followed up with you and Wei Wei on the open pr.


feat: Delete the merged branches in stack sync

This is a relatively large change. Previously, av stack sync didn't touch the
already merged branches. This changes this behavior and it always rebases the
merged branches. After the rebase, if it's sure that the changes are fully
merged (no leftovers that haven't merged yet), it automatically deletes the
branch and the child branches are reparented.
This addresses https://github.com/aviator-co/av/issues/78.

This is confusing. I don't fully understand. It seems rather complicated.
I don't think rebasing merged branches is necessary (or a particularly good idea -- we should avoid changing the users history in unexpected ways).
We should be able to determine whether or not there are commits that weren't merged by doing
# check if foo has un-pushed commits
git fetch origin foo
git rev-list origin/foo..foo

If the rev-list returns a non-empty list, then those are the commits that were present in foo that weren't in origin/foo. It's possibly that we'd want to replace the origin/foo with the HEAD commit of the PR (reported by the GH API) but it's still the same idea.

I do not understand why un-pushed commits are relevant to merged commits...?


Cut 2.0.0-beta.0

@Gouthamve @brian-brazil

Looks like there's a vendoring issue.

Fixed

:+1: once CI passes


Bot Unresponsive

I have installed and configured the config.json with my api key and bot key. I run the bot on terminal and it says the bot has started, but when I try to interact with the bot in telegram it does nothing.

I have got the bot somewhat working but it seems when it needs to call gpt it doesnt respond.

some of my logs
Traceback (most recent call last):
File "/home/codespace/.python/current/lib/python3.10/site-packages/telegram/ext/_application.py", line 1104, in process_update
await coroutine
File "/home/codespace/.python/current/lib/python3.10/site-packages/telegram/ext/_handler.py", line 141, in handle_update
return await self.callback(update, context)
File "/workspaces/codespaces-blank/chatgpt-telegram-bot/src/telegram_message_parser.py", line 121, in chat_text
response = self.message_manager.get_response(
File "/workspaces/codespaces-blank/chatgpt-telegram-bot/src/message_manager.py", line 38, in get_response
self.userDict[id].update(t, message, "user")
File "/workspaces/codespaces-blank/chatgpt-telegram-bot/src/chat_session.py", line 34, in update
if (source == "user") and (contactTime - self.__latestTime > ConfigLoader.get("wait_time")) :
TypeError: '>' not supported between instances of 'float' and 'str'
2023-04-24 06:30:07,229 - TelegramMessageParser - INFO - Get a chat message from user:<PHONE_NUMBER>
2023-04-24 06:30:07,635 - telegram.ext._application - ERROR - No error handlers are registered, logging exception.
Traceback (most recent call last):
File "/home/codespace/.python/current/lib/python3.10/site-packages/telegram/ext/_application.py", line 1104, in process_update
await coroutine
File "/home/codespace/.python/current/lib/python3.10/site-packages/telegram/ext/_handler.py", line 141, in handle_update
return await self.callback(update, context)
File "/workspaces/codespaces-blank/chatgpt-telegram-bot/src/telegram_message_parser.py", line 121, in chat_text
response = self.message_manager.get_response(
File "/workspaces/codespaces-blank/chatgpt-telegram-bot/src/message_manager.py", line 38, in get_response
self.userDict[id].update(t, message, "user")
File "/workspaces/codespaces-blank/chatgpt-telegram-bot/src/chat_session.py", line 34, in update
if (source == "user") and (contactTime - self.__latestTime > ConfigLoader.get("wait_time")) :
TypeError: '>' not supported between instances of 'float' and 'str'

@DSwutang Check this

It seems that the issue is caused by the wait_time field in config.json being filled with a string. We currently do not have an automatic check for config types.
You can modify the relevant part of the config field to the following content:
...
"enable_voice": true,
"wait_time": 600,
"enable_dalle": true,
...
The value filled after wait_time should be an integer or a float, adding quotes may cause it to be identified as a string, which could be the cause of the problem you encountered.

@DSwutang answer will solve your problem.


Sources ?

Hello were is the sources of the distrib WSL ? 🤔
Thanks!

I have added a README.md file that should clarify how the build of the image works.


custom bubble which tail not at the corner

New issue checklist

[ ] I have read all of the README, documentation, and FAQ.
[ ] I have reviewed the contributing guidelines. Confirmation: ____
[ ] I have searched existing issues and this is not a duplicate.

General information

Library version(s):
iOS version(s):
Devices/Simulators affected:
Reproducible in the demo project? (Yes/No):
Related issues:

how to customise a bubble use custom image like following image the tail is not on the corner.

And when the message text is more than one line, it  stretchabled like this

I try it use following codes in cellForItemAtIndexPath:
JSQMessagesBubbleImageFactory *bubbleFactory = [[JSQMessagesBubbleImageFactory alloc] initWithBubbleImage:[UIImage imageNamed:@"chat_icon_bubble_min_outgoing"] capInsets:UIEdgeInsetsMake(1.0f, 1.0f, 1.0f, 6.0f)];
        JSQMessagesBubbleImage *bubbleImageDataSource;

        if ([messageRecord.messageSenderUserId isEqualToString:self.senderId]) {
            bubbleImageDataSource = [bubbleFactory outgoingMessagesBubbleImageWithColor:[UIColor jsq_messageBubbleLightGrayColor]];
        } else {
            bubbleImageDataSource = [bubbleFactory incomingMessagesBubbleImageWithColor:[UIColor jsq_messageBubbleGreenColor]];
        }
        cell.messageBubbleImageView.image = [bubbleImageDataSource messageBubbleImage];
        cell.messageBubbleImageView.highlightedImage = [bubbleImageDataSource

But the bubbleImage showing this

Could anyone lend some tips on achieving this?

I'm pretty sure the problem is your linsets. Look at your source image. The full radius of the corners needs to be captured in the capInsets that you pass to JSQMessagesBubbleImageFactory. Read about [UIImage resizableImageWithCapInsets:] if you want to know more about how it works.

Thank you for your reply, after learn more about [UIImage resizableImageWithCapInsets:] , I reset the capInsets to UIEdgeInsetsMake(30.0f, 10.0f, 6.0f, 10.0f) and everything is OK. Perfect!
Thank you very much!

I'm glad it helped! 😄

What is the correct image format? I am trying to implement custom bubble image and it is not working. I get rectangular with the specified color.

@zhouzhongguang Hey I did exactly the same way you did but in swift 3. But I am getting something like this:

Can you please help me out with this. Thanks in advance : )


Added Id property to Button, Checkbox, Text Field & Select...

Added Id property to Button, Checkbox, Text Field & Select.  Added Required & OnChange to select

Hi! Thank you got your PR. Now is too much conflicts with current code. Can you fix it?

Created new one


Angular CLI template does not work with CLI 1.6.5

I updated the CLI on my machine, 1.6.1 to 1.6.5. I created a new sample app and it would not work as the browser showed CANNOT GET.
I downgraded the CLI back to 1.6.1, recreated a new app and it works. The sample app package.json looks for 1.6.3. Not sure if this is an issue or expected behavior. Maybe the global CLI has to be the same as the package.json version or lower.  Anyway, that's what I found so I'm reporting it so maybe if this is expected at least we can document it.

@SteveSandersonMS - thoughts on this?

It's possible there was some breaking change in the ng tooling between 1.6.1 and 1.6.5. However, the Angular CLI template is now updated to 1.6.8 so this should no longer be applicable.


Cannot set property 'labelVersion' of undefined

Leaflet version: 0.7.7
Leaflet-label: 0.2.1-0
I have installed both from npm. When i try to use leaflet-label by using import 'leaflet-label' it gives the following error
Uncaught TypeError: Cannot set property 'labelVersion' of undefined

I import leaflet before importing leaflet-label. Any clues on what might be the issue.

got the same error

But this does not help as well I get "Uncaught TypeError: layer.bindLabel is not a function" inside function onEachFeature(feature, layer) { --- }

@aparij Thanks, do you know what the difference is between https://www.npmjs.com/package/leaflet-label and  https://www.npmjs.com/package/leaflet.label, seems leaflet.label has a more recent version?


Fix /docker/btrfs mount in disk_usage module

/var/lib/docker/btrfs is not accessible as normal user, also the docker dir can be in different locations.
Original Exception:
PermissionError: [Errno 13] Permission denied: '/var/lib/docker/btrfs'

I had the said Exception and fixed it by adding another ignore path, basically only one line touched.
Thank you very much for the nice project!

I'll check it out. Thanks for letting me know


view others quests journal

The ability for staff to see other players quests journal either by doing /quests journal (name of player) or by being able to remove it from their inventory when doin /invsee
at the moment /quests journal (name of player) still brings up my own journal and i can't remove journals from inventories.
( tried doing /quests.journal.others in luckperms with no luck)
We often have people say i wasn't given the reward and this will help staff to see why as normally the player just hasn't done it correctly.

Thanks for the suggestion. There is currently no such functionality beyond giving players the quests.admin.drop permission so that they may drop their journals, but obviously this is not a great solution. Due to how the journal is currently coded, it's more likely I'll resolve this through console logging via https://github.com/PikaMug/Quests/projects/6

I hadn't found that permisson node and actually, that works just fine. Thanks

@elaineandsparky Glad you were able to work out a solution. If you'd like to try the new logging feature, please try the following dev build. Make sure to set console-logging: 2 in config.yml
https://ci.codemc.io/job/PikaMug/job/Quests/152/


Cherry pick master changes to release-0.6.0 branch

Changes Unknown when pulling 29d9bd3e66748e52b61e1f28b5b56205d861473e on aaltay:relfixb into ** on apache:release-0.6.0**.


Support live-migration with allow-two-primaries

This PR allows to live-migrate KubeVirt virtual machines by setting allow-two-primaries flag dynamically.
fixes https://github.com/piraeusdatastore/linstor-csi/issues/159
Changes in this PR:

controllerPublishVolume:

if volume has type: Block and accessMode: ReadWriteMany;
if it is already InUse at one other node;

enables allow-two-primaries flag

controllerUnpublishVolume:

if volume InUse at one one less amount of nodes

removes allow-two-primaries flag

@kvaps If you want, I can take care of the tests + lints

Sure, tests are fixed

Thanks!


Create mboamersfoort.txt

@frontmill Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course

@frontmill You might miss my previous comment. Please react.

@frontmill I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Heuristic rule for Puppet and Pascal .pp files

This pull request adds a heuristic rule for Puppet and Pascal .pp files. Fixes #3605.
/cc @Caged
Checklist:

[x] I am fixing a misclassified language

[ ] I have included a new sample for the misclassified language:

Sample source(s):

[URL to each sample source, if applicable]

Sample license(s):

[x] I have included a change to the heuristics to distinguish my language from others using the same extension.

I've updated the pull request with the new heuristic rule for Puppet.


Streaming issue for ccdv/pubmed-summarization

Link
https://huggingface.co/datasets/ccdv/pubmed-summarization
Description
This was reported by a user of AutoTrain Evaluate. It seems like streaming doesn't work due to the way the dataset loading script is defined?
Status code:   400
Exception:     FileNotFoundError
Message:       https://huggingface.co/datasets/ccdv/pubmed-summarization/resolve/main/train.zip/train.txt

Owner
No

Thanks for reporting @lewtun.
I confirm there is an issue with streaming: it does not stream locally.

Oh, after investigation, the source of the issue is in the Hub dataset loading script.
I'm opening a PR on the Hub dataset.

I've opened a PR on their Hub dataset to support streaming: https://huggingface.co/datasets/ccdv/pubmed-summarization/discussions/2


Some problems and questions.

Hello!
I installed your neural network and ran it in Desktop App mode, but I don't see the "Turn on GPU" switch here. This is the first question.
Second question: How do I use the models from the demo page? GSR_UNet, VF_Unet, Oracle?
Thanks in advance for the answer!

Hi,
Thanks for the interest. When you run the desktop app for the first time, the webpage will leave blank for several minutes, during which the pretrained model is downloaded. May be after giving the app some more time, the "Turn on GPU" button will appear.

VF_Unet stands for the voicefixer model you use by default in this repo.

Oracle stands for the neural vocoder. You can also access it in this repo. Note that neural vocoder cannot perform restoration, it is mainly used to demonstrate the ability of neural vocoder. We open the API of neural vocoder in this repo. You can have a try if you are interested.

GSR_UNet is the baseline of our paper. Unfortunately we do not code GSR_UNet in this repo because its performance is not as good as VF_Unet. If you are interested you can train a GSR_UNet yourself using this repo.

Thanks,
Haohe


Illegal Argumennt Exception?

scrcpy 1.22 https://github.com/Genymobile/scrcpy
C:\scrcpy\scrcpy-server: 1 file pushed, 0 skipped. 84.4 MB/s (40955 bytes in 0.000s)
[server] INFO: Device: OnePlus DE2118 (Android 11)
INFO: Renderer: direct3d
INFO: Initial texture: 1080x2400
WARN: Device disconnected
[server] ERROR: Exception on thread Thread[main,5,main]
java.lang.IllegalArgumentException
at android.media.MediaCodec.native_configure(Native Method)
at android.media.MediaCodec.configure(MediaCodec.java:2128)
at android.media.MediaCodec.configure(MediaCodec.java:2044)
at com.genymobile.scrcpy.ScreenEncoder.configure(ScreenEncoder.java:274)
at com.genymobile.scrcpy.ScreenEncoder.internalStreamScreen(ScreenEncoder.java:94)
at com.genymobile.scrcpy.ScreenEncoder.streamScreen(ScreenEncoder.java:73)
at com.genymobile.scrcpy.Server.scrcpy(Server.java:100)
at com.genymobile.scrcpy.Server.main(Server.java:326)dn
at com.android.internal.os.RuntimeInit.nativeFinishInit(Native Method)
at com.android.internal.os.RuntimeInit.main(RuntimeInit.java:438)
WARN: Killing the server...
Windows 11
Android Version 11
Phone: 1+

Plezse try #2990.

Hello.
I have tried building the server from source from the dev branch (and ran into an interesting error with "Unsupported class file major version 61" but fixed that by using jdk11-openjdk on archlinux and compiled the server successfully).
I also tried using the pre-built server mentioned in #2990 .
I still get a similar error to the one above. It is slightly different, here is my error output:
scrcpy 1.22 <https://github.com/Genymobile/scrcpy> /usr/share/scrcpy/scrcpy-server: 1 file pushed, 0 skipped. 151.1 MB/s (96899 bytes in 0.001s) [server] INFO: Device: motorola moto g stylus 5G (Android 11) INFO: Renderer: opengl INFO: OpenGL version: 4.6 (Compatibility Profile) Mesa 21.3.6 INFO: Trilinear filtering enabled INFO: Initial texture: 1080x2400 WARN: Device disconnected [server] ERROR: Exception on thread Thread[main,5,main] java.lang.IllegalArgumentException         at android.media.MediaCodec.native_configure(Native Method)         at android.media.MediaCodec.configure(MediaCodec.java:2127)         at android.media.MediaCodec.configure(MediaCodec.java:2043)         at com.genymobile.scrcpy.ScreenEncoder.configure(ScreenEncoder.java:278)         at com.genymobile.scrcpy.ScreenEncoder.internalStreamScreen(ScreenEncoder.java:96)         at com.genymobile.scrcpy.ScreenEncoder.streamScreen(ScreenEncoder.java:73)         at com.genymobile.scrcpy.Server.scrcpy(Server.java:100)         at com.genymobile.scrcpy.Server.main(Server.java:326)         at com.android.internal.os.RuntimeInit.nativeFinishInit(Native Method)         at com.android.internal.os.RuntimeInit.main(RuntimeInit.java:399)

The workaround mentioned in #3033
scrcpy -m1920
Works for me.

Then #2990 should work.

/usr/share/scrcpy/scrcpy-server: 1 file pushed, 0 skipped. 151.1 MB/s (96899 bytes in 0.001s)

It didn't uqe the one you built. How did you build and run?

It didn't uqe the one you built. How did you build and run?

I don't really know how arch packages work, but the fact that it does not retry with a lower definition suggests you don't have the change from #2990.

Could you please test the following commands:
git clone https://github.com/Genymobile/scrcpy
cd scrcpy
git checkout dev
meson d -Db_sanitize=address
ninja -Cd
./run d

On which commit SHA1 are you after that?

That was from c4ab65eb

Could you please test the following commands:

Done. Still the same error with latest dev 36c75e15
* daemon not running; starting now at tcp:5037
* daemon started successfully
DEBUG: ADB device found:
DEBUG:     -->   (usb)  ***                      device  moto_g_stylus_5G
DEBUG: Device serial: ***
DEBUG: Using SCRCPY_SERVER_PATH: d/server/scrcpy-server
d/server/scrcpy-server: 1 file pushed, 0 skipped. 437.5 MB/s (60911 bytes in 0.000s)
[server] INFO: Device: motorola moto g stylus 5G (Android 11)
DEBUG: Server connected
DEBUG: Starting controller thread
DEBUG: Starting receiver thread
INFO: Renderer: opengl
INFO: OpenGL version: 4.6 (Compatibility Profile) Mesa 21.3.6
INFO: Trilinear filtering enabled
DEBUG: Using SCRCPY_ICON_PATH: data/icon.png
ERROR: Could not open image codec: data/icon.png
WARN: Could not load icon
INFO: Initial texture: 1080x2400
DEBUG: Starting demuxer thread
[server] DEBUG: Using encoder: 'c2.qti.avc.encoder'
DEBUG: Receiver stopped
[server] DEBUG: Device message sender stopped
[server] DEBUG: Controller stopped
DEBUG: End of frames
WARN: Device disconnected
DEBUG: quit...
[server] ERROR: Exception on thread Thread[main,5,main]
java.lang.IllegalArgumentException
        at android.media.MediaCodec.native_configure(Native Method)
        at android.media.MediaCodec.configure(MediaCodec.java:2127)
        at android.media.MediaCodec.configure(MediaCodec.java:2043)
        at com.genymobile.scrcpy.ScreenEncoder.configure(ScreenEncoder.java:282)
        at com.genymobile.scrcpy.ScreenEncoder.internalStreamScreen(ScreenEncoder.java:97)
        at com.genymobile.scrcpy.ScreenEncoder.streamScreen(ScreenEncoder.java:74)
        at com.genymobile.scrcpy.Server.scrcpy(Server.java:102)
        at com.genymobile.scrcpy.Server.main(Server.java:332)
        at com.android.internal.os.RuntimeInit.nativeFinishInit(Native Method)
        at com.android.internal.os.RuntimeInit.main(RuntimeInit.java:399)
DEBUG: Server disconnected
DEBUG: Server terminated

./run d -m1920 works fine

What was your meson command?
(or post the result of meson configure d)

meson.configure.txt

Thanks!

java.lang.IllegalArgumentException

Oh ok, it's not IllegalStateException. I will catch this one for retry too, thank you.

Please test #3043.

Please test #3043.

Yes, I can confirm that the new change in the illegalargumentexception branch fixes this error by automatically retrying at a smaller video size.

Thank you, merged :+1:

Please test this version to make sure that it does not break this bugfix: https://github.com/Genymobile/scrcpy/issues/3693#issuecomment-1407135509


generic type cannot be inferred from the parameter at the type level

Bug Report

🔎 Search Terms

genericity, inference, typelevel.
🕗 Version & Regression Information

This is the behavior in every version I tried.

⏯ Playground Link

Playground link with relevant code
💻 Code

type AA = <A>(a: A) => A
type BB = AA extends (a: number) => infer x ? x : never

It looks like typescript has recognized that the shape of AA is (a: number) => infer x, then it should know that A for generics is number (Because a has type A, when a is specified as type number, we can be known that the generic type A is number), then the inferred x should be of type A, which is the type number.
I expected BB to be evaluated as number, but in fact, it was evaluated as unknown.

In situations like this, a type parameter is erased to its constraint, so in this case it appears as (a: unknown) => unknown. I believe we've tried to do full inference here before and the results have been, to put it mildly, catastrophic in terms of both overall performance and not hitting various depth/circularity problems.


⚠ BeyondHD has degraded performance

In 8fc52a0, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 16028 ms

Resolved: BeyondHD performance has improved in f9f6e12.


Used to be able to launch VSCode from an Administrator Cmd Prompt but recently cannot?

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

VS Code Version: 1.75.0
OS Version: Windows 11

Steps to Reproduce:

Download/install Visual studio code or Visual studio code insider
Launch admin cmd prompt
run "code-insiders . "

Observe this:

Also experiencing this issue after updating today.

VS Code Version: 1.74.2
Commit: e8a3071ea
OS Version: Windows 11 Enterprise 21H2
OS Build: 22000.1335

Previously, I was able to open VS Code from an Administrator PowerShell without issue. After updating I see this error when I try to run VS Code as Administrator, whether from PowerShell or via the right-click menu of the start menu item:

I have tried closing all instances of VS Code, rebooting the machine, checking for VS Code updates but nothing has fixed the issue. VS Code runs fine if started as non-admin from the start menu.

/duplicate https://github.com/microsoft/vscode/issues/122951

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!

@bpasero I don't think this is a duplicate of the AppLocker issue. I was able to open VSCode as administrator as recently as yesterday morning (before updating). It is only with the latest update that it fails to open. Is there a way I can check whether AppLocker is the source of the issue?

This was a recent change. The link was not working correctly, see https://github.com/microsoft/vscode-docs/blob/vnext/docs/setup/windows.md#unable-to-run-as-admin-when-applocker-is-enabled now

@bpasero the workaround resolved the issue for me. Thanks for the doc link!


DG Add Attribute Feature - Formatting/Wording issue

The word by does not match the period.

repetition

addressed in #364


Fixed spelling and grammar on the EN Georgi page.

Small orthographic and grammar corrections of the English page of the Georgi keyboard.

Thanks!


A common function for sorting ascending and descending order operation.

For products and category sorting endpoints- there was a common logic which used for several times. Just tracked the issue then make a common function following the DRY principle and implement the logic.

Hello Brother,
Thanks to merging.
Can I message you in Facebook?
On Wed, Sep 6, 2023, 9:24 PM Hasanul Banna Khan Abir <
@.***> wrote:

Merged #7 https://github.com/hbkabir004/Mrittik-Server/pull/7 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/hbkabir004/Mrittik-Server/pull/7#event-10297502558,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR4H4LAEOBA54V5PFT6GMXTXZCIUJANCNFSM6AAAAAA4NFFYTE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Redirect url has trailing slash appended during response

Adding a note on reply URL scheme behavior when it only contains a hostname and not a path segment.
In such case, we append a trailing slash as per current behavior. This update is to document this.

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

Hi @msharsur - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you!

Hi @madansr7 and @msharsur - I applied an edit to add the correct Markdown for the note. Are my revisions to the text accurate and OK to merge? Thanks.
-PR review team

Closing per email from @msharsur.


fix README formatting

Fixed minor README formatting:

Missing code block termination
Surrounded url and url template with codeblock for readability

LGTM


Create folder structure

As a Developer, I am going to create separate top level folders for PyTorch and Tensorflow where the codes using the respective tools will be present.

Completed