[Feature] README Updates

Is there an existing feature request for this?

[X] I have searched the existing issues

Describe the Feature
This packages README is not currently inline with our existing standards. It would be ideal for this repo to see README updates to be in line with our current documentation guidelines.
Describe alternatives you've considered
No response
Are you interested in contributing this feature?

[X] Yes.
[ ] Yes, but I will need assistance and will schedule time during your office hours for guidance.
[ ] No.

Anything else?
During these updates, we should also add the identifier variables and ensure the integration_tests/dbt_project.yml references the identifiers as well.

Update wording of the options to be more clear
Highlight the automations further up and in a better location so users may leverage it

Closing this feature as PR #78 and #80 included the relevant updates that have since been merged.


DFR 3.1.0 has deprecated the pagination parameters

Hmm. we probably need to add DRF 3.1 to the travis build.

it is the official release now.
not talking about Django 1.8 yet, which is for fact working fine with me at the moment.


Login and Log in as user issue

When logging into Cytomine, the page does not always correctly refresh and log you in. Instead it keeps you on the log in screen with no error message or page refresh.
This issue appears in the console of the browser (tested on Chrome and Firefox):

After logging out to test again the page appears to refresh, but after waiting on the page for about five seconds.
There is also an issue as an admin when trying to log in as a user. It does work but initially, it presents this error

but once you refresh the page you have now been logged in as the selected user.

Yes, it's too long to wait.
And without error message if wrong user name or password

completed on 5/12 - tested and approved


[Identity] Creating a sample.env on the samples folder

Fixes #9691

@willmtemple I made this issue to track what you describe: https://github.com/Azure/azure-sdk-for-js/issues/10684 I'll have an update for that later this month.


Don't merge this

We really don't want to change the title any more than @obigriffith already has.

This isn't happening.

Coverage remained the same at 78.48% when pulling 9b33883fe08ab6a306a14a4c5d2b00e76d6a0cd4 on ahwagner:master into 5bc834ab39a74cce02411fdfd0b8a37520a5dfa3 on genome:master.


Support taccumulate

Most of the higher-order iterator functions in Base (sum, prod, any, all, count, minimum, maximum, etc.) can be implemented on top of tmapreduce. The only other fundamental building block that would be useful is taccumulate for cumsum and the like - are there any plans for this?

Parallelizing accumulate is typically very difficult, since the later results require the earlier results. It's possible with lots of re-associations and some clever two-pass algorithms.
If you'd like to take a shot at implementing it, I'd try and take a look and give advice.


Reset not working as expected

Describe the bug
As can be seen in the attached video, the Reset button under Sum of Sequence x,xx,xxx Calculator loads the whole web-app again. But it is not as expected as all the other Reset buttons only reset the value in their respective input fields.
Possible solution
Let me fix this under LGMSOC'21
Screenshots
Makes Math Easy - Google Chrome 2021-06-20 20-16-08.zip

No need to create another issue, just make a commit in your existing branch from which you made a PR

No need to create another issue, just make a commit in your existing branch from which you made a PR

Ok committed. So just had to commit and push in the branch ryt? No need to open a PR for this issue?

No need to create another issue, just make a commit in your existing branch from which you made a PR

Ok committed. So just had to commit and push in the branch ryt? No need to open a PR for this issue?

Exactly


B satish sutar patch 6

Check List (Check all the applicable boxes)

[* ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ *] This PR does not contain plagiarized content.
[ *] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Added few skill sets, result of me upskilling. Please do mind me raising PRs more often, I am upskilling daily and adding more skills, tools etc to my arsenal. Thank you reviewers and maintainers. More power to you..!!

@BSatishSutar I am not sure why you have tried to change the README.md again?  I have already explained in https://github.com/EddieHubCommunity/LinkFree/pull/8761 as to why we are not looking for this change, but also our guidelines if you want to suggest any changes other than to your profile.

@BSatishSutar I am not sure why you have tried to change the README.md again? I have already explained in #8761 as to why we are not looking for this change, but also our guidelines if you want to suggest any changes other than to your profile.

I apologize for any confusion. I just found my GitHub inbox today. I won't modify the readme.md file anymore. Thank you for informing me. Also, I'm unsure why I can't see the changes from my recent PR# 8771 , even though Eddie mentioned it was merged.

Attaching the screenshot for your reference:-

My concern:- The (.) after Cloud Engineer, Cross-platform DevOps Engineer were replaced by a pipe symbol ( | ) and I added a few skills, which still doesn't reflect on my account.

@BSatishSutar as discussed with you in Discord, you have edited your profile with the forms, and once you start using the forms your json file will be ignored. You can now delete your json file via another pull request and continue to use the forms

@BSatishSutar as discussed with you in Discord, you have edited your profile with the forms, and once you start using the forms your json file will be ignored. You can now delete your json file via another pull request and continue to use the forms

No Eddie. I have never used forms to edit anything. As a matter of fact i wasn't even aware that we can edit using forms untill yesterday. I would only resort to using the 'forms' option if circumstances necessitate it. And, even If I did use the forms, I still do not see the changes reflecting on my account.
Took this screenshot at 9 am IST today.

Oh! So, you're saying that I added repositories using the 'add link' + choice. Does that mean I used forms? I went with that choice to add repositories because I couldn't find any other way to do it. Alright, if my changes won't show up, I'll avoid using the Forms choice in the future. I'm avoiding 'Forms' because I don't want to erase my JSON file at any point. My goal was to learn and create a JSON file to make a pull request for linkfree/biodrop. If I have to delete the JSON file and make all edits using 'Forms', it takes away that excitement, motivation, inner drive within me, and I'm not interested in that.
Anyway, I'm really Grateful to you and your team!


Hamming tests are not consistent

There is a test that checks that two empty strands should return Ok 0:
https://github.com/exercism/ocaml/blob/main/exercises/practice/hamming/test.ml#L24-L25
But also two checks that says "disallow * empty strand", expecting an error when given one of the two strands empty:
https://github.com/exercism/ocaml/blob/main/exercises/practice/hamming/test.ml#L38-L41

@Pamplemousse You likely want to re-raise this on the forms, but this is not a specific to the ocaml tests. The problem-specifications repo dictates these tests.
https://github.com/exercism/problem-specifications/blob/main/exercises/hamming/canonical-data.json
https://github.com/exercism/problem-specifications/blob/0b1dfc291099273d56ef77ca71bf7ca8a502b352/exercises/hamming/canonical-data.json#L14-L21
https://github.com/exercism/problem-specifications/blob/0b1dfc291099273d56ef77ca71bf7ca8a502b352/exercises/hamming/canonical-data.json#L116-L125
my guess is that if both inputs are empty than the correct answer is 0. But if only a single input is empty than it is a mistake to error on.


Create client detail page

The client detail page should show information about the company.

This should be at /vms/clients/<client-slug>/

finished this. once u check it we can close it

Closed in #65


Getting Custom Image Sizes

Hi - thanks for making this plugin, in general is making life a lot easier and certainly code cleaner!
Expected behavior
I see this issue has been closed but now I have an issue with the edge case mentioned here: https://github.com/timber/timber/issues/438
Let's say I have some code like this:
{% set image = TimberImage(post.hero_image) %}
  <img src="{{image.src('hero')}}" alt="{{image.alt}} {{post.title}}" width="{{image.sizes.hero.width}}" height="{{image.sizes.hero.height}}"/>

I would expect to correctly get the image src, width and height for that given size.
Actual behavior
I correctly get the src and the sizes for an image, so long as the corresponding size has been generated in Wordpress (i.e., if the original image is large than all the sizes).
However, if the image was a smaller image than say a custom size of 'hero' which might be e.g. 1600px, then I do not get anything for the width and height because in the array of sizes, [hero] does not exist.
I'm pretty sure the default Wordpress behaviour is to give the correct sizes.
Steps to reproduce behavior

What version of WordPress, PHP and Timber are you using?
WordPress 4.5.3, PHP 5.4, Timber Version 1.1.1
How did you install Timber? (for example, from GitHub, Composer/Packagist, WP.org?)
Upgraded to newest version via plugin updater in WordPress dashboard

I'm closing out old issues. Please open a fresh issue if this is still a live issue that we should investigate — thank you!


[FEATURE]: Add A cluster identifier thats human readable in the "Description" field of the created NFS Export

Describe the solution you'd like
when you have multiple Kubernetes clusters but want to keep everything under an access zone, customer wants to be able to either automically add the Kubernetes cluster name or even better, to customize from deployment the "Description" field of the NFS export thats getting created with the CSI deployment
Describe alternatives you've considered
Alternative is to do it manually, but when the description gets updated with something not humaly readable like this from my customer:
CSI_QUOTA_ID:aSxhGQEAAAAAAAAAAAAAQEMAAAAAAAAA
its hard for an powerscale Admin to actually change it but pointing to which cluster owns which export becomes a huge challenge if you do it manually.
Additional context
Add any other context or screenshots about the feature request here.
Current View from a real environment
isi nfs exports view --zone=risess --id=26
                 ID: 26
               Zone: risess
              Paths: /ifs/dvtcisln0004/risess/csi/csivol-6e5d903254
        **Description: CSI_QUOTA_ID:aSxhGQEAAAAAAAAAAAAAQEMAAAAAAAAA**

Desired View
isi nfs exports view --zone=risess --id=26
                 ID: 26
               Zone: risess
              Paths: /ifs/dvtcisln0004/risess/csi/csivol-6e5d903254
        **Description: Openshift Cluster: Cluster01 App: User-App-01**

Thanks for submitting the feature request, @ergonzalezn. We will evaluate the request and add it to our backlog.


You should not be using reflection to add player permission attachments

What code needs changing
In io.github.mdsimmo.bomberman.game.GamePlayer.kt, you have the following code which adds a permission attachment to a player:
// Add a permission group for WorldGuard compatibility (must be done before teleporting)
// Wrap in try loop because it is quite dodgy how other plugins implement
try {
    val playerPermissions = BukkitUtils.getPermissibleBase(player)
    playerPermissions?.addAttachment(plugin, "group.bomberman", true)
} catch (e: Exception) {
    plugin.logger.log(Level.WARNING, "Unable to add permissions", e)
}

This calls io.github.mdsimmo.bomberman.utils.BukkitUtils.kt's getPermissibleBase() function which calls HUMAN_ENTITY_PERMISSIBLE_FIELD which uses reflection. This does not abide by normal plugin coding standards (plugins should not be accessing OBC!), and thus breaks an interaction with one of the plugin APIs that I am maintaining.
What solution would you like
The Player object inherits from Permissible (as shown here at the top under "All Superinterfaces"), meaning you can simply use the following:
player.addAttachment(plugin, "group.bomberman", true)

This avoids the need of:

Using reflection to access perm from CraftHumanEntity
Accessing the OBC class CraftHumanEntity directly
Modifying the perm's access modifiers to be accessible, which is a Java security issue

Thanks in advance!

Oh, that's how it's meant to be done!
I thought it was strange spigot had a built-in permission handler, but no way to access it. Kinda obvious now in hindsight.
Will fix and add to next release.

Added to v0.7.0


readiness matrix: downgrade impls to 🚧 for 7620

Because of the recent spec change.

IIRC, the issue was with the initcode mode validation, and it is now fully specced out (Megaspec & EIPs). Evmone impl is in review, not sure about besu. However, I'm now more inclined to think that if we're doing such feedback-addressing updates, maybe we shouldn't be so strict with downgrading, as the bulk of work is still done.
Also, we had plans to additionally mark cells which are done-done (passing comprehensive reference tests). We can be strict about that.

I'm not sure this update is up-to-date anymore. Close or refresh?

Ping @chfast @pdobacz

Ping @chfast @pdobacz

looks fairly out-of-date by now, closing. I'll look over the readiness tables in general shortly


PIP installation does not work.

PIP installation (for at least Python 3, on Ubuntu Trusty) does not work. In the end it saves a file webkit_server.py, but other modules (such as dryscrape) can't find it properly.
Installing via cloning the repo and then manually running python3 setup.py develop does work though.

I got the same problem with py27

Same issue with OS X 10.10.5, py27, scrapy installer won't find it.

The issue wasn't with the webkit-server, but rather qt. An older installation of qt (mine was 4.7.4) had some path issues. That was fixed by updating the qt installation (mine is included in Anaconda) done via (I wanted to stay in 4.0 so I went with 4.8.6):
conda update conda conda install -c anaconda qt=4.8.6
webkit-server installed fine afterwards. Moreover this all started when I tried to install dryscrape. Hopefully I'm finally out of the wild package chase...

@tristanta any updates on your solution? I tried what you recommended, but I'm still having issues installing webkit-server. Also running into this via the dryscape package install process

I have the same issue with qt 4.8.7, python 2.7(anaconda virtual environment), xcode 8.2.1 when I installed both dryscrape and webkit-server from source code of github.

@cameronkruse Yea the solution worked fine for me. I just had to make sure that the proper version of qt was installed. It got complicated when certain packages required it to be newer than certain versions, while other packages required it to be older (webkit).


Fix build badge

Changes

Fixes build badge - see here  for more information
Fixes build badge link - now takes you to the build.yml workflow results as intended

thanks


Access to the path "C:\whatever\path" is denied.

Tried to do an obfuscation on Rubeus and got the following error:
.\Codecepticon.exe --action obfuscate --module csharp --verbose --path "C:\Users\username\source\repos\Rubeus\Rubeus.sln" --map-file "C:\Users\username\Desktop" --profile rubeus --rename fp --rename-method random --rename-charset abcdefg --rename-length 7 --string-rewrite --string-rewrite-method xor
[18:47:05] Codecepticon v1.2.2 is starting...
[18:47:05] Getting Visual Studio Instance
[18:47:05] Creating MSBuild Workspace
[18:47:06] Loading solution: C:\Users\username\source\repos\Rubeus\Rubeus.sln
[18:47:09] Finished loading solution
[18:47:09]
[18:47:09] Processing project C:\Users\username\source\repos\Rubeus\Rubeus\Rubeus.csproj
[18:47:12]
[18:47:12] Elements Found:
[18:47:12]      Namespaces:     22
[18:47:12]      Classes:        156
[18:47:12]      Enums:          625
[18:47:12]      Functions:      562
[18:47:12]      Properties:     186
[18:47:12]      Parameters:     624
[18:47:12]      Variables:      1512
[18:47:12]      Structs:        89
[18:47:12] Generating mappings...
[18:47:12] Creating mappings for functions
[18:47:12] Creating mappings for properties
[18:47:12] Rewriting code...
[18:47:12] Selected profile is: Rubeus
[18:47:12] Running profile-specific pre-process actions...
[18:47:12] Rewriting assemblies..................
[18:47:12] Removing comments..................
[18:47:12] Rewriting switch statements..................
[18:47:12] Rewriting strings..................
[18:47:13] Renaming 562 functions..................
[18:48:50] Renaming 186 properties..................
[18:49:50] Running profile-specific post-process actions...
[18:49:51] Applying changes to solution...
[18:49:51] Running profile-specific final actions...
[18:49:51] Applying changes (again) to solution...
[18:49:51] Generating mapping file to: C:\Users\username\Desktop
Unhandled Exception: System.UnauthorizedAccessException: Access to the path 'C:\Users\username\Desktop' is denied.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)
at System.IO.StreamWriter.CreateFile(String path, Boolean append, Boolean checkHost)
at System.IO.StreamWriter..ctor(String path, Boolean append, Encoding encoding, Int32 bufferSize, Boolean checkHost)
at System.IO.File.InternalWriteAllText(String path, String contents, Encoding encoding, Boolean checkHost)
at Codecepticon.Modules.DataUnmapping.WriteTemplateFile(String saveAs, Dictionary`2 data) in C:\Users\username\Source\Repos\Codecepticon\Codecepticon\Modules\DataUnmapping.cs:line 183
at Codecepticon.Modules.CSharp.CSharpManager.d__1.MoveNext() in C:\Users\username\Source\Repos\Codecepticon\Codecepticon\Modules\CSharp\CSharpManager.cs:line 165
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Codecepticon.Modules.CSharp.CSharpManager.d__0.MoveNext() in C:\Users\username\Source\Repos\Codecepticon\Codecepticon\Modules\CSharp\CSharpManager.cs:line 43
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Codecepticon.Program.d__0.MoveNext() in C:\Users\username\Source\Repos\Codecepticon\Codecepticon\Program.cs:line 70
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Codecepticon.Program.(String[] args)
Even thought there's no special access required to the path.. I set the HTML mapping file to my desktop.

--map-file "C:\Users\username\Desktop" has to be a file not a directory, change it to --map-file "C:\Users\username\Desktop\something.html"


from Map to json

parse_json was something that I was looking for: now I'm looking at the other direction: I feel like this might be baked in implicitly somehow, but couldn't guess at it.
So: other than coding it by hand, is there an extant way to deserialize a Map into json?

A Map is simply HashMap<String, Dynamic>.  The problem is with Dynamic, which can be a simple dyn Any trait object.  It may be your own type that does not support serialization.
All primitive types built into Dynamic implements Debug.  Therefore, you can try doing a format("{:?}", map) to serialize the HashMap into something that's very similar to JSON.

It may be your own type that does not support serialization
I'm not using my own type, I just used parse_json, manipulate it a bit.

Anything that's a trait object will simply print ?
In my trivial test instance, they all show ?, as all the values are alloc::boxed::Box

Is that not the expected behavior?
json: {"test":"test1","test2":"test2","transmuted":true}
Script: map["test3"] = map["test2"];map["test4"] = map["test"];
Output of the above: {"test3": ?, "test2": ?, "jaims.transmuted": ?, "test4": ?, "test": ?}

Oops, sorry!  I know what the problem is.
I have a mod that I haven't land yet.  It changes the internal representation of primitive types to an enum instead of a trait object for efficiency.  That'd enable the debug print to show up not with ? for primitive types.
You can surf to my branch to check it out.
https://github.com/schungx/rhai/tree/enum-Dynamic

Works great - I noted the () has to get replaced with null, which is fine: I'll definitely use it, but since it's incompatible, maybe it should be wrapped in a feature?

Works great - I noted the () has to get replaced with null, which is fine: I'll definitely use it, but since it's incompatible, maybe it should be wrapped in a feature?

It is never going to be true JSON without a serious serializer.... I can probably put in a serde feature just for that, but that would be too wasteful.  It is going to be pretty easy for you to write your own serializer implementation to serialize a Dynamic anyway -- just defer to the appropriate serialization depending on the actual value type, or bail out if it is a trait object.
That way, you can map a () value with null.


Draft Spec for Cascading Default + User Settings

Summary of the Pull Request
Adds a spec describing how a cascading settings model should work. This spec describes a model where there are two sets of settings, "Default" and "User" settings, and the two are layered to create the runtime settings.

PR Checklist

[x] Specs #754
[x] CLA signed. If not, go over here and sign the CLA
[N/A] Tests added/passed
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments
Just read the spec.

Here is my model of settings, adopted form #1513. It's main goals are:

Graduality and modularity
Reduction of boilerplate
Easy of syncing and usage on large scale

The things I didn't mention below remain as their are now or in above spec/discussion, like default settings. I'd make this an issue or PR but I was instructed to write it here.

There are 2 kinds of settings:

Application/window - e.g. defaultProfile, showOpenTermionalInCurrentDirectory, windowBorderStyle.
Pane - everything else like font, colors/color schema. (Note currently pane = tab).

There are profiles, but thier function is limited to being pretty much just a button. Essentially, they only have following attributes:

guid
name - the text displayed on button
icon

All settings are defined in groups which are JSON elements like the present globals.

Groups have conditions that activate them. They may be:

executiong command line - with support for pattern matching
profile guid that launched the pane - note that terminal/pane could be launched without profile e.g. by directly executing program or script (like double clicking cmd.exe)
user name
machine name / other machine identifier
domain name
no condition -  simply always active ("global")

Groups that were enabled by the condition that is higher in previous list overwrite groups enabled by these that are lower.

All groups can have pane settings, but only these enabled by user name or below can have application settings.

Groups may also include other groups by theirs guid/name. Once included, these would be active regardless to the their condition.

All groups can be in single file, but may also be separated into multiple files. E.g. each user may have file in his own user directory, there may be one on the machine for all users and one shared in domain for all computers. This resolves problem with permissions to edit local settings. Decision on the layout of this files depends on administrator.

Terminal at startup looks only at the local settings file. This file (and others too) may have include element with paths to other files that terminal then loads.

For simplicity, profiles may have settings in their body. Terminal treats it as though it was a group with a condition on the profile's guid.

Landed here after the resolution on #2173, and wanted to ask about some of the intended behavior. In a handful of places in the spec, it sounds like Windows Terminal will sometimes make changes to the user settings file. I see there was some consideration for the case where the user does not want this, e.g. the proposed autoloadWslProfiles and autoloadAzureConnections settings.
Is it possible that there could be some kind of setting that indicates the profiles.json should never be modified, preventing manipulation by any future auto-load features? I'm asking because I'd like to put my profiles.json file into my dotfiles repository, and already deal with a handful of things (cough Visual Studio cough) constantly rewriting git-tracked files just to reorder things or change whitespace.

I'm also confused by defaults.json. So, it's a local json file that we generate from our code, but they can't make changes to it? It sounds like it might just be easier to hardcode these default in our code and refer to them like we do now. Is this supposed to be a stepping stone toward them being able to just define what they want their defaults to be?

@carlos-zamora you better believe there's going to be a much easier doc for this when it comes out. This is more the technical implementation details, when I make an actual PR for it, I'll make sure to include a settings-howto.md with a user-friendly explanation of how the layering works.
Theoretically, everything the user currently has will still work, though some may become redundant. Future user profiles will be simpler to read, with the minimum set of key-value pairs in them.

Just my €0.02: IMVHO this has lower priority than #1770. Of course, the full settings experience needs both.

@miniksa @DHowett-MSFT can I get some additional passes on this?


[textanalytics] Bad error message when sending request to unsupported endpoint region

This issue is to track a bug in the service and verify that a good error message is raised once the bug is fixed.
(update: 1/24 emailed Assaf/Han about this bug)
No body is sent when using an endpoint from a region not supported (e.g. west us, detect languages endpoint). This leads to a stack trace when the error model can't be deserialized correctly since it is missing code/message fields.
The code/message fields in the swagger should not be required here if they are sometimes not sent.

re-gen with autorest v3 fixed this issue


invalid MdSelect should highlight itself on parent form submitted

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Expect md-select to show red (or 'warning' color) if it is invalid and its parent form is submitted.
What is the current behavior?
Currently the select does show warning color on touched and invalid, but not on parent form submitted and invalid.
What are the steps to reproduce?
See this plunk. Both the select and the input are 'required'. Click 'submit' and see that the select does not change its appearance like the input does.
What is the use-case or motivation for changing an existing behavior?
Would like to highlight errors on submit.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
all
Is there anything else we should know?
The MdSelect likely needs to inject an @Optional NgForm/@Optional FormGroupDirective in its constructor like MdInputContainer does so that it can keep track of its submitted state.

The select's 'required' asterisk also is not 'warning' color in all the same states as it is in the input:
When an input is focused (and valid or untouched), the asterisk is 'warning' even though everything else is 'primary'. When a select is focused, everything is 'primary' including the asterisk.


'init' command: ADO Paradocs Artifacts

@johnathon-b whats the vision here for init? does this create a folder structure for documentation?

Originally this was to prep a directory for the paradocs payloads, providing the .order and .override and some config details for docfx.
After some thought I would propose moving from docfx to a nuxtjs/vue. This would allow us to have and static generated content that can be hosted on anything with integrated search and themes.
init would then change to generate, pointing to a content directory. This would also enable us to use either Markdown & YAML within documentation and produce higher quality docs.
Let's discuss this some more.


Specify type definitions as dependencies

We ship type definitions for the engine that contain references to WebXR types. This means that TS projects now have to specify @types/webxr as a dependency:
https://github.com/playcanvas/playcanvas-editor-ts-template/pull/12/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R34
This would be unnecessary if we just made types dependencies instead of devDependencies.
I believe the WebGPU types don't need to be dependencies but, for consistency, I'm moving those to dependencies too. It will avoid any potential future issues around WebGPU type dependencies.
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

This is great, it probably means I no longer need to make module functions private if they use webgpu types in JSDocs.

I believe that's correct, yes.


Only display carrots when there are children

depends on https://github.com/CenterForOpenScience/osf.io/pull/6238
carats now only appear when there is a 3rd row of taxonomy
before

after

ticket
https://trello.com/c/GETV8yQ1/111-preprints-discover-the-caret-does-no-action

It might look nicer if the checkboxes were aligned. Is it possible to add some kind of offset to taxonomy items that don't have children?

okay

@mfraezz the expanded carat width is one pixel wider than the unexpanded carat width :(

victory.


Add FiftyOneTorchDataset

What changes are proposed in this pull request?

New class FiftyOneTorchDataset.
added top-level to_torch method.
some DDP utils.
Recipes for use (still missing DDP example)

How is this patch tested? If it is not, please explain why.

Benchmarked memory footprint
Benchmarked dataset latency
Validated data loaders load correct samples for each trainer
Trained mint classifier w/ new class.
Currently training MAGE in a DDP setup - will report results.

Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[x] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

New to_torch method allows for easy transfer to a fully functional torch.utils.data.Dataset object.
Please see recipes for examples and tutorials.
(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[ ] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[x] Core: Core fiftyone Python library changes
[x] Documentation: FiftyOne documentation changes
[x] Other - fiftyone.utils.torch

Summary by CodeRabbit

New Features

Introduced a script for training and validating models using the MNIST dataset with PyTorch.
Added a new Jupyter notebook for a basic training example with the MNIST dataset.
Implemented a method to convert sample collections into a PyTorch-compatible format.
Added a new class for creating PyTorch datasets from FiftyOne datasets, enhancing integration with distributed training.

Bug Fixes

Improved data handling and efficiency in multi-worker environments.

Documentation

Enhanced clarity with markdown explanations in the new Jupyter notebook.

MAGE training successful. Very coarse test, but nice to see still.


<Section>'s subtitle prop can be PropTypes.node

In https://github.com/SalesforceCommerceCloud/pwa-kit/pull/252/files#diff-e0ea812c2f8e525485d0a8c5864aef81debdc02ab58da8b52b5a6925b90396c0L234-R252, we passed a React node in the subtitle prop of a <Section>.
This causes a big nasty warning when rendering the homepage during local development.
Testing
git checkout -t origin/fix-invalid-prop-type-on-homepage
npm start --prefix packages/pwa

In a browser, open http://localhost:3000
Open the devtools console, observe no giant proptypes warning.
Look at the terminal, observe no giant proptypes warning.

The exact same fix is in release-1.3.x and is soon to be merged back to develop (https://github.com/SalesforceCommerceCloud/pwa-kit/pull/284/files)
To avoid conflicts, I'll close this one...


IE6 CSS hacks breaking Nokia S60

MBP uses star hack a lot to target IE6, but it breaks Symbian S60 3.x browsers, like the N95 one.
The problem occurs when we use a CSS compressor that strips the last semi-colon and when the last rule is a star hack rule. For example:
This original CSS:
button, input { line-height: normal; *overflow: visible; }

Is minified to this:
button,input{line-height:normal;*overflow:visible}

Because the last rule is a start hack one and the semi-collon is missing, S60 browsers will skip the entire line. And since we are minifing everything into one line, it skips the entire stylesheet, rendering an unstyled page.
The solution is simple: just reverse the order so that no star hack rule is the last one:
button, input { *overflow: visible; line-height: normal; }

This line will be minified to this:
button,input{*overflow:visible;line-height:normal}

And that's ok, works fine on Nokia's browsers.
One last note: I think this is not an YUI compressor issue, since it's perfectly valid to strip the last semi-colon. But since that breaks a mobile browser, I think it's a MBP issue.
And of course, the definitive fix is to remove all CSS hacks and target IE6 with safer conditional comments.

FWIW, I filed a bug with YUI Compressor for this, in case they would like to fix it on their end: http://yuilibrary.com/projects/yuicompressor/ticket/2528146
The * hack is used to target IE7 and IE6. Did you try the _ hack for IE6? Does it have the same issue?

The underscore hack works fine. It can be the last one or not, everything works.

I sent a pull request with the proposed changes and GitHub created another issue - #101


Preset "Router" not working

Version
3.0.0-beta.15
Reproduction link
https://github.com/dimensi/test-vue-cli
Steps to reproduce
Start vue-cli ui. Try create new project.
Select npm as package manager.
Select manual preset
Select Babel, Vuex, Router, CSS, Linter presets.
Select Stylus.
Pick linter "Eslint   Standard"
Wait installing and open the project
What is expected?
Vue-router in project
What is actually happening?
Nothing. Installing only vuex.

I can't understand what's happen.
I try create 5 times a new project.
4 times i not got router. 1 time i got router, but not got babel...

Hmm this is really weird, unfortunately, I've never run into this and have no way to reproduce this, so there's really nothing we can do about it.

@yyx990803, I film it :)

Oh this seems to be a UI only problem. @Akryum

Yes, problem in UI.

I can't reproduce with either vue-cli 3.0.0-beta.15 nor the dev branch. I'm using node 10.3.0 and npm 6.1.0 on MacOS.

Yeah, I wasn't able to reproduce either.


kubectl-cnpg completion bash fails when there is no APIserver

When KUBECONFIG is not set, I get:
$ kubectl cnpg completion bash
Error: Get http://localhost:8080/api?timeout=32s: dial tcp <IP_ADDRESS>:8080: connect: connection refused
$ # While it works for `kubectl`:
$ kubectl completion bash
# Copyright 2016 The Kubernetes Authors.
# [...]

This prevents usage of source <(kubectl cnpg completion bash) in ~/.bashrc

The cause is probably:
https://github.com/cloudnative-pg/cloudnative-pg/blob/943c514361e1b947340504b4234990f775cf0a0c/cmd/kubectl-cnpg/main.go#L58-L61
This part should not be run with the completion command.

Hi @sathieu
Yes, following your analysis makes a lot of sense it fails because of that, I'm assigning this ticket to me to work on a fix during the day! will let you know and update the ticket!
Thanks for this report!
Regards!

Hi @sathieu
There's a PR in place, can you test it and provide us your input on this fix?
Regards!


Migrate to compose?

Do you need to migrate to compose?

No need for Compose

No need for Compose


Couldn't fetch parent branch

I have successfully been using git-tfs to convert my TFS repos to git repos. However, one repository provides me some issues. It can't seem to identify the parent branch. When looking in TFS it appears that the branches are successfully parented, however, when trying to clone, it comes up with the error.
I am runnning the latest version <IP_ADDRESS>
Here is a gist of the command run in debug mode.
https://gist.github.com/bmeverett/1b6c08f2dbbb3a5015db49f11a5f9c3c

Yes these branches still exist, and according to TFS are properly parented.
--branches= none returns only one commit from the master branch

Hi,
Any progress on this one please. We are trying to migrate a huge TFVC repository to git and are encountering the same error.

As this is quite an old issue, please retry this with the latest version if it still exists
As we are trying to clean up old/stale issues, I am closing this issue for now. But if there is further information available, we can easily reopen the issue again!


authentication?

This would appear to only work with public git repos, is there a way to use a private repo with a key or user/pass?

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

ID: 7bde2eab-46e1-c7da-9efb-5c0cdd52603a
Version Independent ID: d2bffaed-46cb-9e6c-7893-67a2ebe3fc0d
Content: Mount a gitRepo volume Azure Container Instances
Content Source: articles/container-instances/container-instances-volume-gitrepo.md
Service: container-instances
GitHub Login: @mmacy
Microsoft Alias: marsma

I believe you need to include the access token in the URL. See the following:
https://stackoverflow.com/questions/21907278/github-api-using-repo-scope-but-still-cant-see-private-repos

Hi, Thanks for the response, that's helpful.  Does this mean it only supports github?  How about bitbucket, VSTS, etc?  How about privately hosted git repos?
Thanks!

@jboren specifically related to container instances you mean?

@MicahMcKittrick-MSFT  Thanks for the help, sorry if I'm not being clear.  In addition to the link you posted,  I found a few blog posts and think I understand how to make it work for any repo host that uses an access token, for example, it looks like I could use a PAT to mount a repo from VSTS, or an OAuth token for bitbucket.  It looks like for any repo that uses a username/password I can use<EMAIL_ADDRESS>(correct me if I'm wrong), so I think my question is answered.   It might make sense to clarify the article a little regarding how to construct the gitrepo URL for different authentication methods, at least that would have saved me a bunch of time.   All in all, thanks a ton for the help and clarification on this.
Best regards,

@jboren got it. I believe that you are correct. However I have not used anything other than the git repro for this scenario so I would have to test to confirm. I think it would be good to include some more information around this in the doc. I will assign to the content author to review and see if we can get more information for private repos.
@mmacy can you take a look?

@jboren You are correct. If you can specify a PAT or other auth value in the URL for the Git repository, ACI will be able to perform the pull. For example, to mount a gitRepo volume from a VSTS Git repository, specify the PAT and any username in the URL to the repository. For example, the --git-repo-url for a repository "TestProject01" would look something like:
--gitrepo-url<EMAIL_ADDRESS>
For VSTS, the username in the URL ("vstsuser" in the above example) can be any value--you can use "vstsuser" for your own repo, as long as you specify a valid PAT and the correct URL to your repo.
I've tested this as functional with my own VSTS repo, but have not tested with BitBucket. However, behavior should be the same on any Git repo that accepts creds in the URL.

Hi, reopening this because I'm curious how we can handle private Git repo authentication within the ARM template... is that a viable scenario?


Are requests asynchronous?

I understand that requests should be asynchronous when wrapped in gevent. However, I am not able to get above 16 r/s for one application with extremely slow response times (no matter how many users I use). I am thinking then, that requests are actually waiting for responses before Locust fires off another request.

client requests inside a task are implemented with python-requests... they are synchronous.

My understanding was that gevent made the requests asynchronous, something similar to grequests?


[documentation] Document exotic behaviors like currency.js specifics

Related to #59

Constructor extensions of primitive types with method properties should be console logged as System.Dynamic.ExpandObject but can be properly logged and used in methods using Primitive([object]), like currency from currency.js that will need Number(currency(...).[...?]) to be console logged or passed as argument

Currently, there is no need to document these type of behavior


Logprobs info to completion API

Motivated by attempt to add mlx support to lm-evaluation-harness, this extends the HTTP Model Server's completion API implementation to include logprobs information in the choices result JSON

The log probabilities per token generated
The top_logprobs the tokens with the highest probability at each token position in what the model generated.

To be able to calculate top_logprobs, the  generate_step function was modified to be able to generate a triple instead of a 2-item tuple, where the last item is the step logits, i.e., the logits at each step of the generation for use in capturing and returning the top N tokens at each step of the generated token sequence.  A boolean parameter named return_step_logits was added to this function, which is False by default and causes the function to include the step logits if it is True.
I could not find clarity on what this (legacy) API expects as the data structure the server should return to embed this information.  In addition, some literature and implementations return strings represented by the tokens instead of the tokens themselves, so this just follows the data structure that lm-evaluation-harness expects.
Similarly, it interprets the logprobs OpenAI API parameter as specifying the number of top tokens to return at each token position of what is generated (by default, it doesn't do this).
The current version of lm-evaluation-harness currently from github invokes the completions API with max_tokens set to 0, which causes mlx to not generate anything in response to evaluation requests (see: Issue 1903 ).

@chimezie I made some modifications here. Could you check that the server works for your use case?

@chimezie I made some modifications here. Could you check that the server works for your use case?
If its good we can land this shortly. Though it might be worth having a cleaner solution for the how we take already tokenized prompts.

@awni Yes, this works for the use case.  Thanks! I just made a few small changes to fix the logprobs response format, but nothing else.

Apologize if I missed this, but I had a use case where I needed to return the token text in the server response.
Made a small change locally to do calculate and return text representations of tokens instead of just token IDs, along with text offsets based on the actual token text lengths.
I suppose the client could do this as well, but found it helpful when using a notebook mode UI to alter text.
Happy to add it in if anyone finds it helpful, or if it belongs on the client side, that works too.

In case anyone is interested, I pushed the updated server.py with logprob texts (instead of just token IDs) here: https://github.com/fblissjr/mlx-examples/commit/6eefa51261345c21b5c83cf4d8c937e547b6dd50
@@ -351,6 +351,26 @@ def generate_response(
}
choice = response["choices"][0]
if self.logprobs > 0:
    detokenizer = self.tokenizer.detokenizer
    detokenizer.reset()

    choice["logprobs"]["tokens"] = []
    choice["logprobs"]["text_offset"] = [0]
    accumulated_text = ""

    for token in tokens:
        detokenizer.add_token(token)
        token_text = detokenizer.last_segment
        choice["logprobs"]["tokens"].append(token_text)
        accumulated_text += token_text
        choice["logprobs"]["text_offset"].append(len(accumulated_text))

    choice["logprobs"]["token_logprobs"] = token_logprobs
    choice["logprobs"]["top_logprobs"] = [
        {self.tokenizer.decode([int(token)]): prob for token, prob in top_logprob.items()}
        for top_logprob in top_tokens
    ]

# Add dynamic response
if self.object_type.startswith("chat.completion"):


Has anyone tried this in the browser?

Primary concerns:

Does it actually work? (e.g. with browserify)
What's the size of the complied/minified file?

I would point you towards #25 and #189 seems like there have been varying levels of success


Add fillBlockCache option to bypass RocksDB block cache

Scope & Purpose
https://arangodb.atlassian.net/browse/APM-86
Docs PR: https://github.com/arangodb/docs/pull/703
Added fillBlockCache option to bypass RocksDB block cache

Implemented APM-86: add query option fillBlockCache to control population
of RocksDB block cache with data read by the query. The default value for
this per-query option is true, which mimics the previous behavior.
Setting the option to off allows not storing data in RocksDB's block cache
for queries that are known to read only semi-relevant or unimportant data.

This is currently experimental, and does not have documentation, tests, nor driver support.

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[x] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[x] Backports required for: to be discussed

Testing & Verification

[x] The behavior in this PR was manually tested
[x] This PR adds tests that were used to verify all changes:

[x] Added new integration tests (e.g. in server_parameters)

Just double checking: fillBlockCache is a cursor option, not a query option?
@RESTSTRUCT{fillBlockCache,post_api_cursor_opts,boolean,optional,}
I'm a bit confused that batchSize isn't a cursor option by the look of things:
@RESTBODYPARAM{batchSize,integer,optional,int64}

@Simran-B : I don't know what the difference is, but it can be used as follows:
https://github.com/arangodb/arangodb/pull/14234/files#diff-71885788211640a079769272c37b97fab5c379c088246697cfcab33e063f6f12R62

@mpoeter : it is split into 3 separate files because a test that populates the block cache basically "wastes" the environment. With 3 different files we can at least have 3 separate and clean server instances that don't interfere with each other.


[BUG] helm install csi-hostpath-driver and "/var/lib/kubelet"  is not shared In Windows and Ubuntu

What did you do

How was the cluster created?

k3d cluster create

What did you do afterwards?

helm install my csi-hostpath-driver addons
helm install csi-hostpath-driver kubeblocks/csi-hostpath-driver

What did you expect to happen
csi-hostpath-driver running succeed。
Screenshots or terminal output
helm install succeed

kubectl get pods -A -w

kebuctl describe pod csi-hostpath-driver-0

Which OS & Architecture

output of k3d runtime-info
arch: x86_64
cgroupdriver: cgroupfs
cgroupversion: "1"
endpoint: /var/run/docker.sock
filesystem: extfs
infoname: docker-desktop
name: docker
os: Docker Desktop
ostype: linux
version: 20.10.22

Which version of k3d

output of k3d version
k3d version v5.5.1
k3s version v1.26.4-k3s1 (default)

Which version of docker

docker version
Client:
Cloud integration: v1.0.29
Version:           20.10.22
API version:       1.41
Go version:        go1.18.9
Git commit:        3a2c30b
Built:             Thu Dec 15 22:36:18 2022
OS/Arch:           windows/amd64
Context:           default
Experimental:      true
Server: Docker Desktop 4.16.3 (96739)
Engine:
Version:          20.10.22
API version:      1.41 (minimum version 1.12)
Go version:       go1.18.9
Git commit:       42c8b31
Built:            Thu Dec 15 22:26:14 2022
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.6.14
GitCommit:        9ba4b250366a5ddde94bb7c9d1def331423aa323
runc:
Version:          1.1.4
GitCommit:        v1.1.4-0-g5fd4c4d
docker-init:
Version:          0.19.0
GitCommit:        de40ad0
docker info
Client:
Context:    default
Debug Mode: false
Plugins:
buildx: Docker Buildx (Docker Inc., v0.10.0)
compose: Docker Compose (Docker Inc., v2.15.1)
dev: Docker Dev Environments (Docker Inc., v0.0.5)
extension: Manages Docker extensions (Docker Inc., v0.2.17)
sbom: View the packaged-based Software Bill Of Materials (SBOM) for an image (Anchore Inc., 0.6.0)
scan: Docker Scan (Docker Inc., v0.23.0)

And I notice the similar issue  #1063  and has been fixed in PR https://github.com/k3d-io/k3d/pull/1268.
However it does not work in Ubuntu and Windows.

That's my fault that not use the ENV K3D_FIX_MOUNTS


Android BeaconDidUpdate: Beacon's UniqueID is null

RN Version : 0.62.2
react-native-kontaktio Version: ^2.7.2
to detect beacons I am using this code :
    DeviceEventEmitter.addListener('beaconsDidUpdate', ({ beacons }) => {
      console.log('beaconsDidUpdate', beacons);
    });

    DeviceEventEmitter.addListener('beaconDidAppear', ({ beacon }) => {
      console.log('beaconDidAppear', beacon);
    });

upon detecting the beacon, the uniqueId is null :
beaconDidAppear {"accuracy": 0.12086380610771207, "address": "CD:D4:F2:28:2E:E1", "batteryPower": -1, "firmwareVersion": "-1", "isShuffled": false, "major": 36652, "minor": 28539, "name": null, "proximity": "IMMEDIATE", "rssi": -68, "txPower": -84, "uniqueId": null, "uuid": "f7826da6-4fa2-4e98-8024-bc5b71e0893e"}
is there any setup that needs to be done aside from what is listed on the documents for me to obtain the uniqueId?

I have the same issue, react-native 0.59.10. Any ideas what's wrong?

You should also scan for Secure Profile where you will get the Unique ID. I encourage you to visit the
https://developer.kontakt.io/hardware/packets/secureprofile/

Also, you will get the UniqueID when it came with Secure Response.

Hello everyone. @MKacikKontakt was right!:) To retrieve uniqueId and other meta info (e.g. battery level, firmware revision etc) you need to add SECURE_PROFILE argument into your connect method invokation and listen for "profile*" events.

Thanks for writing this here @sergeylaptev !
Do you want to add it to the documentation, perhaps as a comment in this code-block here right next to uniqueId?
https://github.com/Driversnote-Dev/react-native-kontaktio/blob/master/docs/api.android.md#beacon--eddystone

@johnkennethfurog  I tried running the minimal example code in react-naitve version 0.63, but it did not listen to any beacon , I had a phone act as a alt beacon .but it was not detected.
were you able to detect beacon through "beaconsDidUpdate" method or did you use any other thing for that.
@andrekovac  @sergeylaptev  can you guys help me on this?

@kirantripathi I'm not using the package in my current projects atm so I didn't yet test it with RN 0.63 ... You don't get any errors, do you?

@andrekovac  yes for IOS it gives me error as "no listnere registered" in RN 0.63 . can you  help me on that.

I was trying to run minimal code but it is not returning any beacon information.
@johnkennethfurog  @sergeylaptev   what is your guys basic setup to scan the beacon. I try with minimal code example but got nothing as response

Hello everyone. @MKacikKontakt was right!:) To retrieve uniqueId and other meta info (e.g. battery level, firmware revision etc) you need to add SECURE_PROFILE argument into your connect method invokation and listen for "profile*" events.

Does anyone know wow do we listen for profile events? I'm only seeing listeners for beacons and eddystone appear/update/disappear. What's the name of the listener for onProfilesUpdate? I don't see an example using SECURE_PROFILE in the docs. What am I missing? Thank ya!

Hello guys,
Android 10 has introduced the new requirement of permission, please request "ACCESS_FINE_LOCATION" instead of the "ACCESS_COARSE_LOCATION" in your code. I just made this change and it works on Android 10 and 11 devices again.
Ref: https://altbeacon.github.io/android-beacon-library/requesting_permission.html

@andrekovac @MKacikKontakt  Do you have any more guidance on this? Sounds like the SECURE_PROFILE fix only works with iBeacon Pro, which I do not have. Any other info on getting the uniqueId from the scan response would be much appreciated: https://knowledgebase.kontakt.io/hardware/packets/scanresponse/#classic-scan-response-packet-structure I'm currently not able to know which beacon is broadcasting, which does me no good. I'm happy to reach out to someone else too if neither of you are the right person to ask! Many thanks.

Hello everyone. @MKacikKontakt was right!:) To retrieve uniqueId and other meta info (e.g. battery level, firmware revision etc) you need to add SECURE_PROFILE argument into your connect method invokation and listen for "profile*" events.

Does anyone know wow do we listen for profile events? I'm only seeing listeners for beacons and eddystone appear/update/disappear. What's the name of the listener for onProfilesUpdate? I don't see an example using SECURE_PROFILE in the docs. What am I missing? Thank ya!
@MKacikKontakt can you provide any guidance here?

Hi @noblepaper
I am not sure if you have fixed your issue.
For Android, I am using profileDidAppear/profileDidUpdate/profileDidDisappear events to get the uniqueId,


Update readme.md file

update the website screenshot with the new UI screenshot

If the issue is still open, I'd like to contribute


[$250] DEV : When clicking to send the magic code to verify the secondary email, a blank page appears briefly before the page to enter the code.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: dev
Reproducible in production?: dev
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @brunovjk
Slack conversation (hyperlinked to channel name): ts_external_expensify_bugs
Action Performed:

Log into the account.
Navigate to Account settings > Profile > Contact methods.
Add a secondary login.
Enter the magic code for the existing account to allow the secondary email to be added.
Click to send the magic code to verify the secondary email.

Expected Result:
The user is taken directly to the page to enter the code.
Actual Result:
A blank page appears briefly before navigating to the page to enter the code.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/e39c31a5-04e8-46e3-aa68-d4c9f959b17b
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021867385191571001621
Upwork Job ID:<PHONE_NUMBER>571001621
Last Price Increase: 2024-12-13

Issue OwnerCurrent Issue Owner: @ZhenjaHorbach

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

Proposal
Please re-state the problem that we are trying to solve in this issue.
When entering one of contacts, RHP modal opens twice.
What is the root cause of that problem?
https://github.com/Expensify/App/blob/f8f12c2dcb6d3b62663fc4edea50cd543e44e74d/src/pages/settings/Profile/Contacts/ContactMethodDetailsPage.tsx#L239-L276
Here at details page, though modal is opened other modal is placed by <ValidateCodeActionModal>.
What changes do you think we should make in order to solve the problem?

Instead of using <ValidateCodeActionModal>, we can replace it with <ValidateCodeForm>
Then the fix code would look like:
https://github.com/Expensify/App/blob/f8f12c2dcb6d3b62663fc4edea50cd543e44e74d/src/pages/settings/Profile/Contacts/ContactMethodDetailsPage.tsx#L239-L259
                <View style={[themeStyles.ph5, themeStyles.mt3, themeStyles.mb7, themeStyles.flex1]}>
                   <Text style={[themeStyles.mb3]}>{translate('contacts.enterMagicCode', {contactMethod})}</Text>
                   <ValidateCodeForm
                       validateCodeAction={validateCodeAction}
                       validatePendingAction={loginData.pendingFields?.validateCodeSent}
                       validateError={!isEmptyObject(validateLoginError) ? validateLoginError : ErrorUtils.getLatestErrorField(loginData, 'validateCodeSent')}
                       handleSubmitForm={(validateCode) => User.validateSecondaryLogin(loginList, contactMethod, validateCode)}
                       sendValidateCode={() => User.requestContactMethodValidateCode(contactMethod)}
                       clearError={() => User.clearContactMethodErrors(contactMethod, !isEmptyObject(validateLoginError) ? 'validateLogin' : 'validateCodeSent')}
                       buttonStyles={[themeStyles.justifyContentEnd, themeStyles.flex1]}
                       ref={validateCodeFormRef}
                       hasMagicCodeBeenSent={hasMagicCodeBeenSent}
                   />
                </View>

What alternative solutions did you explore? (Optional)
N/A

Done, what do you think of the proposal above, @hoangzinh ?

Thanks for posting a proposal. Can you share your recording result?

It seems the issue isn't reproducible from my side.

It seems we lost the screen transition with that approach.

@hoangzinh  This cause the transition issue
https://github.com/Expensify/App/blob/1466c022e3bb0ab2ffaaab12f78822a2aca827c8/src/pages/settings/Profile/Contacts/ContactMethodDetailsPage.tsx#L36
hasMagicCodeBeenSent={hasMagicCodeBeenSent}

of
https://github.com/Expensify/App/blob/1466c022e3bb0ab2ffaaab12f78822a2aca827c8/src/pages/settings/Profile/Contacts/ValidateCodeForm/BaseValidateCodeForm.tsx#L145
of
https://github.com/Expensify/App/blob/1466c022e3bb0ab2ffaaab12f78822a2aca827c8/src/components/MagicCodeInput.tsx#L160-L164
But the code form's focus() is already used here
https://github.com/Expensify/App/blob/1466c022e3bb0ab2ffaaab12f78822a2aca827c8/src/pages/settings/Profile/Contacts/ContactMethodDetailsPage.tsx#L240
So I have deleted hasMagicCodeBeenSent.
https://github.com/user-attachments/assets/39a0bbf3-3663-46ad-8141-8b86911bd25e

Btw, we used to use ValidateCodeForm previously, but it has been replaced by ValidateCodeActionModal after this PR https://github.com/Expensify/App/pull/49445
@hoangzinh Then reverting the details page of the PR would be better.

@hoangzinh I meant reverting this:
src/pages/settings/Profile/Contacts/ContactMethodDetailsPage.tsx

Let's use this new component and remove any other usages that could lead people to reintroduce the wrong usage/ component.
@hoangzinh I understood we can't revert.

It seems there are 2 <ValidateCodeForm> at @components/ValidateCodeActionModal and at src/pages/settings/Profile/Contacts each.

Issue not reproducible during KI retests. (First week)

Not overdue, trying to reproduce

Issue not reproducible during KI retests. (Second week)


Create Testimonial 13 white

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

Follow figma design and to make the button use Button Design System
After building the component please raise a PR with a screenshot of the component and add the component in Testimonials/src/testimonial-13-white.html & Please do not add hover effect in the main div
If you need to use any icon please use it from Hero Icons
Good luck.

I want to contribute

I would like to contribute to this issue


LinalgRefactor-SGVector-Rangefill

Only works for cpu backend.

Fine to merge from my side
If you want to, you can send another patch where you remove range_fill from SGVector, and then replace all the compile errors with sg_linalg

lemme see whether i can fix it now. shouldn't be too hard.

@OXPHOS what happening?

@vigsterkr ready to merge


CRITICAL: Impossible to write the driver in /etc/linux-enable-ir-emitter/...

Describe the bug

The driver cannot be written to /etc/linux-enable-ir-emitter
 › sudo result/bin/linux-enable-ir-emitter -v -d /dev/video2 configure
INFO: Ensure to not use the camera during the execution.
INFO: Warning to do not kill the process !
Is the ir emitter flashing (not just turn on) ? Yes/No ? No
YeDEBUG: Computing the resolution control.
DEBUG: Minimum not consistent, it will be ignored.
DEBUG: current: 1 25
DEBUG: maximum: 2 100
DEBUG: resolution: 0 37
DEBUG: unit: 13 selector: 14
DEBUG: new current: 1 62
Is the ir emitter flashing (not just turn on) ? Yes/No ? Yes
Yes/No ? Yes
terminate called after throwing an instance of 'std::ios_base::failure[abi:cxx11]'
  what():  CRITICAL: Impossible to write the driver in /etc/linux-enable-ir-emitter//pci-0000:03:00.4-usb-0:4:1.2-video-index0_emitter0.driver: iostream error
ERROR: The configuration has failed.
INFO: Do not hesitate to visit the GitHub ! https://github.com/EmixamPP/linux-enable-ir-emitter/wiki

I have checked the perms of /etc, there should be no problem writing to it.
drwxr-xr-x - root  7 Mar 23:22  etc

How to reproduce

Run the above command.
Additional info

Computer (or camera) model: Lenovo Yoga Slim 7 13ACN5, Bus 003 Device 003: ID 13d3:5419 IMC Networks Integrated Camera
Version of linux-enable-ir-emitter: 4.5.0

What are the perms of /etc/linux-enable-ir-emitter ?
Did you try a second time ?

I wrote a Nix package that uses meson only to build linux-enable-ir-emitter. I will instruct the respective NixOS module to create that directory then.

Lines 78-81 of the meson; it creates that directory when the install target is called:
install_subdir(
    meson.project_name(),
    install_dir: get_option('sysconfdir'),
)

Lines 78-81 of the meson; it creates that directory when the install target is called:
install_subdir(
    meson.project_name(),
    install_dir: get_option('sysconfdir'),
)

Right. That path gets generated inside the linux-enable-ir-emitter package, but it does not affect the global /etc. This is due to the way NixOS works.
Anyway, it's an easy fix. Thank you!

Ok ok, fine then
With pleasure!


Fix relative links in azdata.*.d.ts

Noticed while working on these that the links were all broken, VS Code expects to use the @link tag instead for code links

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.045%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
28003

Relevant Lines:
62022

💛  - Coveralls


Add support for effects with cleanup?

Should we add effects with cleanup?
There is currently no way to perform an action when a component gets destroyed.

useEffect(() => {
    function handleStatusChange(status) {
      setIsOnline(status.isOnline)
    }

    ChatAPI.subscribeToFriendStatus(props.friend.id, handleStatusChange)

    return function cleanup() {
      ChatAPI.unsubscribeFromFriendStatus(props.friend.id, handleStatusChange)
    }

The use case from the react document looks like when useEffect returns a function, we execute it
The function contains unmouted logic
Am I correct?

Yes:

the function passed to useEffect may return a clean-up function [...] The clean-up function runs before the component is removed from the UI to prevent memory leaks. Additionally, if a component renders multiple times (as they typically do), the previous effect is cleaned up before executing the next effect

(Preact hooks are a good source of reference, as the code is very simple and easy to understand.)

Well, I need to think carefully about the correct implementation here. My previous understanding may be insufficient.~

@mindplay-dk I refactored the useEffect, and now it should be able to resolve these two issues #63 #64

supported Now!

Looks good, just one small deviation from how React works - from the description in the yellow note box on this page:

If you want to run an effect and clean it up only once (on mount and unmount), you can pass an empty array ([]) as a second argument. This tells React that your effect doesn’t depend on any values from props or state, so it never needs to re-run.

So, to summarize:
useEffect(f)       // ✅ effect (and clean-up) every time
useEffect(f, [])   // ❌ effect (and clean-up) only once in a component's life
useEffect(f, [x])  // ✅ effect (and clean-up) when property x changes

Where is the deviation? The summarize is right 😆

@mindplay-dk I tried the use case of react, which would not execute when the array was empty.
useEffect(f, [])   // ✅ return function( such as clean up) should not run.

So fre's current behavior should be consistent with react

I tried the use case of react, which would not execute when the array was empty.

The clean-up function of an effect always runs - the question is only when.
With an empty array, the clean-up function should run only once, when the component is destroyed.

I didn't even notice this detail. I've fixed it and it works now 💯

Looks good!
Still needs tests for useEffect() with no second argument, and with [] - let's leave this issue open until all the tests are in place.

We have tests for this now. Closing!


Add note: floats don't mix with miscellaneous modules

Closes #25829. Could change where we place the sentence, open to suggestions (I just thought creating a new section with caveats would be bad).

I use it with grid system, but i need to add d-block clearfix to row. It's seems to work fine. But i can't use flex utility.

I use it with grid system, but i need to add d-block clearfix to row. It's seems to work fine. But i can't use flex utility.

You shouldn't be doing that. If you're reverting to floats, you should just avoid the grid entirely and roll your own custom stuff. Try out the flexbox approach more before jumping to that.

@mdo
I am trying few days play with flexbox, but without a result. The problem is that flex item can't be in two flexbox line.
https://codepen.io/danijelGombac/pen/yKoXGB

Yeah, for stuff like that, you're relying on float's ability to collapse against content around it. If you changed your markup a bit, it wouldn't be an issue. Also, you could (depending on browser support you need) use grid layout.

@mdo
I tried different markup, but without a good result. On mobile i need the sequence of card 1,2,3. For now i leave floats, I don't want to use a grid layout (browser support), thanks.


Update de.json

minor fix regarding formal grammar.
(just a single "s" i forgot to capitalize, which basically makes the text say "her/they" instead of the formal "you")

i'll look through the rest of the file over the next few days, just in case there are more errors of this nature.

Changes to de.json:

minor formality fix
improved sentence formatting
adjusted capitalization to fit the rest of the translation

Thanks!


retrying sync when is 504

https://github.com/InVisionApp/elroy/issues/218

I am in the process of rewriting this code. Can I add this solution to my branch, otherwise I am going to have a big merge conflict?

@c1freitas  feel free to add it and remove this PR when you do it

done in #54


Crash when attempting to use plant goo as bonemeal

When lobbing plant goo from the goo gauntlet at a plant the server crashes and user is kicked with a stack overflow error. Server crash log attatched.
crash-2021-01-10_12.34.12-server.txt

Forgot to flag this when I saw it.
I haven't seen the log yet, but I'll make it a priority, this is relatively critical. Does it recover from the crash or is it a permanent breakage?

I can load back into the server after the crash I just have to avoid using it.

Looks like I forgot to wrap the spawn bonemeal particles method in a client-only check (it's a client only method, crashes because server doesn't have such a thing)
The stack overflow would just be the thing failing over and over in a loop, but not a recursive one.
I think I've got it fixed in my current build.
Leaving this open for testing.


Should Lease contains a deadline?

We had this discussion with @NiteshKant.
The question is should the lease contains the duration/deadline or not, knowing that a server can overwrite his previous Lease with a new one.
It's a tricky question, the problem with the deadline is that the propagation delay between the server and the client makes the deadline very imprecise. This leads to recommendation about having the window >> network latency.
On the other end, an overloaded server should need to do anything to implicitely reduce the flow of incoming requests. Then just leting the lease expire, is the perfect solution.
I tend to prefer the solution with the duration, but I can be convinced.

Absolutely it is a tricky decision to make!
My reservations around using deadline are the following:

As you specified, using time means that we need to take care of propagation delays.
The client code becomes complex as it has to be considerate of time before making a request. So, it will manifest in two ways:

Check for current time and hence deadline before each request.
Or have a scheduled timer for each connection from the client that resets the request quota for that duration.

Both of the above sounds too heavy a burden to force from the protocol IMO.
I do agree that the lease expiry is a nice side-effect for an overloaded server. An argument against that could be that for a well behaved client and properly implemented flow control from the server, hitting the wall (overloaded) would mean that the server over-subscribed for requests.
Overall, I do not have a concrete reason for either ways but using time does make me nervous!

Just for the record, we ended up choosing to keep a TTL in the lease message.
It has the nice property of automatically applying backpressure when a server is overloaded (it doesn't need to do any extra work to shed load away)


net7

Breaking change

You are now required to use .NET 7 when developing NetDaemon apps.
Proposed change

Upgrades to latest .NET 7 for NetDaemon components. The C# 11 features are not enabled in this PR. There will be a separate PR for this.
Type of change

[X] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[x] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the [development checklist][dev-checklist]
[ ] The code compiles without warnings (code quality chek)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

Codecov Report
Base: 77.16% // Head: 77.92% // Increases project coverage by +0.76% :tada:

Coverage data is based on head (13cc77e) compared to base (3de32f3).
Patch coverage: 75.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##              dev     #786      +/-   ##
==========================================
+ Coverage   77.16%   77.92%   +0.76%
==========================================
  Files         141      152      +11
  Lines        3661     4000     +339
  Branches      365      403      +38
==========================================
+ Hits         2825     3117     +292
- Misses        693      722      +29
- Partials      143      161      +18

Flag
Coverage Δ

unittests
77.92% <75.00%> (+0.76%)
:arrow_up:

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

Impacted Files
Coverage Δ

...mmon/Extensions/IConfigurationBuilderExtensions.cs
93.54% <0.00%> (-3.23%)
:arrow_down:

...n.AppModel/Internal/Config/ConfigurationBinding.cs
78.83% <ø> (ø)

...el/NetDaemon.AppModel/Internal/Config/AppConfig.cs
92.85% <100.00%> (ø)

...aemon.HassModel.CodeGenerator/EntitiesGenerator.cs
98.79% <100.00%> (ø)

...time/Common/Extensions/ServiceBuilderExtensions.cs
100.00% <0.00%> (ø)

...time/Internal/HomeAssistantConnectionExtensions.cs
100.00% <0.00%> (ø)

...e/Common/Extensions/ServiceCollectionExtensions.cs
0.00% <0.00%> (ø)

...Runtime/Common/Extensions/HostBuilderExtensions.cs
34.28% <0.00%> (ø)

...ime/NetDaemon.Runtime/Internal/NetDaemonRuntime.cs
75.23% <0.00%> (ø)

...e/NetDaemon.Runtime/Internal/EntityMapperHelper.cs
88.46% <0.00%> (ø)

... and 12 more

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.


Added rough POC implementation of w3mimgdisplay

This isn't supposed to make it as-is into the master branch. I thought that opening the browser for every image kinda sucked, so I made this little hack. I figured if you were made aware of this cool technology you might want to utilize it. Also, sorry if I did this wrong, I am relatively new to git :p

outdated


Fixing the broken unit test

Fixes #<issue_number_goes_here> (it's a good idea to open an issue first for discussion)

[ ] Tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

@beccca , @alexander-fenster can you please review this PR? This is supposed to fix the broken unit test.


[AST] Imported classes have no stored properties.

Avoid triggering the import of all of the members of an Objective-C
class to determine its stored properties, because we don't import
instance variables of Objective-C classes. rdar://problem/45060773

@swift-ci please smoke test and merge

Can you write a counter-based test for this? See the existing tests in test/NameBinding like the one I recently added


Rewards claim Issue

From last 2 weeks I am not able to claim my IOT rewards. Is there any issue ?
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
App Version [e.g. 3.3.0]

Additional context
Add any other context about the problem here.

When was the last time you were able to claim rewards?

No activity or response on this issue for two weeks. Closing.


Fetch binary tools with build script, hide them with .gitignore

The following bloat the repository and will lead to a slow, agonizing death:

WurstPack\bin\worldedit.exe
WurstPack\wurstscript\wurscript_b.exe
WurstPack\vexorianjasshelper\jasshelper.exe
WurstPack\vexorianjasshelper\clijasshelper.exe
WurstPack\vexorianjasshelper\pjass.exe
WurstPack\vexorianjasshelper\pjass_old.exe
WurstPack\bin\exehack.exe
WurstPack\mpqedit\mpqeditor.exe
WurstPack\wurstscript\MpqCL.exe
WurstPack\wurstscript\wurstscript.exe
WurstPack\grimext\PatchGenerator.exe
WurstPack\grimext\FileExporter.exe
WurstPack\grimext\FileImporter.exe
WurstPack\grimext\TriggerMerger.exe
WurstPack\winmpq\WinMPQ.exe
WurstPack\grimext\ObjectMerger.exe
WurstPack\grimext\ConstantMerger.exe
WurstPack\grimext\grimex.dll
WurstPack\grimext\TileSetter.exe
WurstPack\grimext\PathMapper.exe
WurstPack\tesh\funclist.db
WurstPack\tesh\horus.dll
WurstPack\tesh\tesh.dll

If they are generated by WurstScript (successful builds) gitignore them. If they are tools required by WurstScript (libraries) fetch them with some equivalent of make config (I suppose you use ant), and gitignore them.

They are not generated and as far as I know there is no reliable source to fetch them from, so I would rather have them in the repository. I don't think they will slow down the repository over time, since these are rather static files which are not updated anymore.
But maybe WurstPack can be put into a different repository.

Sadly most of these are c++ binaries which we only use since they're part of newgen and we are too lazy to natively implement all of those.
Cool to see you trying put wurst, but unless you have proper source with makes for all those they will stay.
I didn't have any problems with this "bloated" repo.
Also this isn't uncommon.

@peq there are many reliable sources, for example google code and dropbox are free.
@Frotty definitely not interested in rewriting the binaries

https://github.com/blog/1986-announcing-git-large-file-storage-lfs

@peq returning to this a year later, I'm supposing that this won't get scheduled?
I think it could make sense to put the standard library, wurst compiler, and wurstpack in separate repos, and make a script that bundles stuff. I imagine that it would make it easier to handle wurstscript releases.

Well, it's kind of happening. But slowly.

Frotty has started a new version of the standard library in a separate repo: https://github.com/Frotty/StdLib2
The new editor based on vscode is in a separate repo: https://github.com/peq/wurst4vscode

Once the new editor based on vscode is stable enough, I can ditch the Eclipse plugin (or at least entangle it from the compiler). Without the need for the Eclipse build system it becomes easy to use proper dependency management with gradle.
So check back next year for this issue :)

@peq Is there a roadmap for what's needed to cut stl and eclipse plugin from wurst repo?
Seems like the sort of thing I might be able to contribute with but not sure if will just be disruptive.

Cutting stl: unit tests need to be changed to use stdlib from different
repo.
Eclipse plugin: either deprecate it completely or change it to use the same
interfaces as vscode
On Aug 22, 2016 00:38, "Cokemonkey11"<EMAIL_ADDRESS>wrote:

@peq https://github.com/peq Is there a roadmap for what's needed to cut
stl and eclipse plugin from wurst repo?
Seems like the sort of thing I might be able to contribute with but not
sure if will just be disruptive.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/peq/WurstScript/issues/329#issuecomment-241286812,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBakhJ3yt5lgj0rectkw-kcCpTNaZsxks5qiNNQgaJpZM4DcISg
.

@Frotty Gradle build is now working (on branch https://github.com/peq/WurstScript/tree/towards-gradle)
What is missing is:

[ ] building the jar
[ ] generating the hotdoc pages for stdlib
[ ] changing the jenkins config
[ ] Cleaning up the wurst repository
[ ] Running autoformat on the sourcecode and get rid of tabs :D

I pushed 2 commits to remove local jars and make all tests succeed. Will do further work throwing things away 💃

Wow, very big commit. It's a bit hard to see what happened, could you the
commit up and use single commits for deleting whole subdirs?
Otherwise it's hard to tell what happened here.
On Jun 13, 2017 02:00, "Frotty"<EMAIL_ADDRESS>wrote:

I pushed 2 commits to remove local jars and make all tests succeed. Will
do further work throwing things away 💃
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/peq/WurstScript/issues/329#issuecomment-307968036,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBaknquH69IkH-bovKrH9PhBSEzOM58ks5sDdESgaJpZM4DcISg
.

I agree it's probably cleaner, but I don't really want to revert now. I think the beauty of git is, that nothing is ever lost and if you are missing something, you can always get it back.
 In a decent git interface you can just see that all files are from same folders.

Seems to work


Provide windows scripts for containers in examples

Currently we only have unix based scripts.
See: https://github.com/canoo/dolphin-platform-examples/tree/master/simple-security/keycloak-container

This issue was moved to rico-project/rico#10


Problem to run pg_activity when there's a barman connection

We have a CentOS 7 server running PostgreSQL version 12 and we use Barman (https://www.pgbarman.org/) as one of our backup tools.
When we try to run pg_activity without any additional parameter, we get the following error:
[root@server ~]# sudo -u postgres pg_activity -U postgres
Traceback (most recent call last):
  File "/bin/pg_activity", line 11, in <module>
    load_entry_point('pg-activity==2.1.3', 'console_scripts', 'pg_activity')()
  File "/usr/lib/python3.6/site-packages/pgactivity/cli.py", line 293, in main
    ui.main(term, dataobj, host, options, dsn)
  File "/usr/lib/python3.6/site-packages/pgactivity/ui.py", line 227, in main
    width=width,
  File "/usr/lib/python3.6/site-packages/pgactivity/views.py", line 526, in screen
    width=width,
  File "/usr/lib/python3.6/site-packages/pgactivity/views.py", line 123, in wrapper
    for line in func(term, *args, **kwargs):
  File "/usr/lib/python3.6/site-packages/pgactivity/views.py", line 378, in processes_rows
    cell(getattr(process, field), column)
  File "/usr/lib/python3.6/site-packages/pgactivity/views.py", line 363, in cell
    text_append(f"{color}{column.render(value)}")
  File "/usr/lib/python3.6/site-packages/pgactivity/types.py", line 239, in render
    return self.template_h % self.transform(value)[: self.max_width]
  File "/usr/lib/python3.6/site-packages/pgactivity/utils.py", line 79, in ellipsis
    lv = len(v)
TypeError: object of type 'NoneType' has no len()

To figure out which database name was coming as None and making the function ellipsis to fail, I edited the file types.py to not apply the function ellipsis on the database name. Then, pg_activity ran well and showed a barman connection but with the database name being None in fact.

At this point I don't know where the problem is happening and why that "zombie connection" (with no reads and no writes) shows up without a database name. One possible solution would be to handle the situations when the database name is None. Something like that:
def ellipsis(v: str, width: int) -> str:
    """Shorten a string to specified width with '...' in the middle.
    >>> ellipsis("loooooooooog", 7)
    'lo...og'
    >>> ellipsis("loooooooooog", 6)
    'lo...g'
    >>> ellipsis("short", 10)
    'short'
    """
    if v is None:
        return "Something here"
    lv = len(v)
    if lv <= width:
        return v
    assert width >= 5
    wl = (width - 3) // 2
    return v[: wl + 1] + "..." + v[-wl:]

Thanks in advance for any thought or help in how to fix this issue.

So it seems that the datname field from pg_stat_activity may be NULL in some cases (streaming replication?)?
How did you configure barman? Especially, the conninfo value in pg section described at http://docs.pgbarman.org/release/2.12/#preliminary-steps.

The "zombie" conneciton @DenisMedeirosBBD mentioned is how Postgres does streaming replication (more up-to-date than file-based logs).
Per Postgres documentation, "replication connections do not specify any particular database", which is why there's no database specified in Barman's streaming_conninfo setting.

Right now, I am not sure theses lines should be displayed because they are not "client backends" but "walsenders".
Given that we are displaying queries only client backend en parallel workers make sense.
The line is visible because "state <> 'idle'" filter all processes with a NULL state ... but the walsender who has a status.
It's not completely wrong since walsenders count towards max_connections but I dont like the fact that it's mixed with "normal connexions". Added "bonus" we can kill it with the software.
I think it's a good idea to Fix the null datname nonetheless because I would love to remove this limitation and display all processes in the future since the are gradually added to pg_stat_activity and we have some nice info available for them.

Just released version 2.1.4 including a fix for this issue.

Thank you!


initial tensorrt ep commit

Adds in TensorRT EP
Observed that Encoder Model is much faster with TRT than with CUDA backend
Initialization of onnxrt session takes more time as TRT models are generated and converted

ToDo :

[x] Validate Accuracy of the model inference
[x] Reducing the starting time

Ref : https://github.com/k2-fsa/sherpa-onnx/issues/40, https://github.com/k2-fsa/sherpa-onnx/issues/41
CC : @csukuangfj

I will send a separate PR for handling configs of OnnxRT EP configs separately.
along with TRT config options.
In the middle of something, and it might take sometime

I think the build failures are coming in from the python dependency of nvinfer and similar, I will try to add some libraries and check

https://github.com/k2-fsa/sherpa-onnx/actions/runs/9354678305/job/25748084556#step:5:1121
[ 62%] Linking CXX executable ../../bin/sherpa-onnx
/opt/rh/devtoolset-10/root/usr/libexec/gcc/x86_64-redhat-linux/10/ld: ../../lib/libsherpa-onnx-core.so: undefined reference to `OrtSessionOptionsAppendExecutionProvider_Tensorrt'
collect2: error: ld returned 1 exit status
make[2]: *** [bin/sherpa-onnx] Error 1
make[1]: *** [sherpa-onnx/csrc/CMakeFiles/sherpa-onnx.dir/all] Error 2
make: *** [all] Error 2

I think an extra lib should be linked to for tensorrt.

yes, it seems directly having  OrtSessionOptionsAppendExecutionProvider_Tensorrt() as in interface is not working out
and AppendExecutionProvider_TensorRT_V2 is actually a wrapper around that function.
so it should be alright now

Please merge the latest master into your branch and the CI should pass or you can just ignore the failed tests.
Could you describe what users need to do to build sherpa-onnx with TensorRT support?

please leave a comment if you think it is ready to merge.

Regarding what is required to run TensorRT,
Hardware      : Nvidia GPU 😄 ,
Software lib  : TensorRT requires libnvinfer, libnvinfer-dispatch, libnvinfer-plugin, libnvonnxparsers8 (from my experience)
But seeing that CI/CD's are working fine, I don't think onnxrt is able to handle these,
as long as someone is having a compatible GPU it should work fine.

It can be merged,
after ensuring that basic builds are working fine
Thank you for the review 😄

Regarding what is required to run TensorRT,
Hardware : Nvidia GPU 😄 ,(not sure how compatible AMD GPUs are) Software lib : TensorRT requires libnvinfer, libnvinfer-dispatch, libnvinfer-plugin, libnvonnxparsers8 (from my experience)
But seeing that CI/CD's are working fine, I think onnxrt is able to handle these, as long as someone is having a compatible GPU it should work fine.
Just setting the appropriate provider argument should be enough, if there are any github pages/doc, this can be updated there
--provider=trt

Update this commit, by mistake I had typed ' don't' but edited it

Please merge the latest master into your branch and the CI should pass or you can just ignore the failed tests.
Could you describe what users need to do to build sherpa-onnx with TensorRT support

I think the builds are also good enough.
Shall we merge it @csukuangfj

Thank you for your contribution!


Mix of tabs and spaces in stylesheets

I noticed we have a mix of tabs and spaces in some of the stylesheets. @jordanmccullough what are your thoughts on adding those to spaces?

Gasp! Not sure how, but happy to clean it up, for sure. Thanks for opening the issue, @brntbeer.

haha, not a problem! just figured you worked well with issues, so i thought i'd open this one up!


Use PeoplePicker in WinUI 3

How can use PeoplePicker in WinUI 3 ?

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

ID: 6fa203a5-039a-2696-17c9-74c80f5739f7
Version Independent ID: 0514840b-c9de-8d7c-edab-156ed75f7282
Content: PeoplePicker XAML Control - Windows Community Toolkit
Content Source: docs/graph/controls/PeoplePicker.md
Product: dotnet-communitytoolkit
Technology: windows-community-toolkit
GitHub Login: @michael-hawker
Microsoft Alias: mhawker

Our graph controls haven't been updated to support the Windows App SDK at this time.
The underlying TokenizingTextBox though has, and can be currently found in our new repository https://github.com/CommunityToolkit/Windows - we have a preview package on our "nightly" MainLatest NuGet feed (https://aka.ms/wct/wiki/previewpackages). But should also release a preview package to NuGet soon (and then release).


PrimitiveType.union erases original type

The following ScalaTest test case
  test("merge primitive types") {
    val expected =
      Types.buildMessage()
        .addField(
          Types
            .required(INT32)
            .as(DECIMAL)
            .precision(7)
            .scale(2)
            .named("f"))
        .named("root")

    assert(expected.union(expected) === expected)
  }

produces the following assertion error
message root {
  required int32 f;
}
 did not equal message root {
  required int32 f (DECIMAL(9,0));
}

This is because PrimitiveType.union doesn't handle original type properly. An open question is that, can two primitive types with the same primitive type name but different original types be unioned?
Reporter: Cheng Lian / @liancheng
Assignee: Cheng Lian / @liancheng
Related issues:

Release Parquet-mr 1.9.0 (blocks)
PrimitiveType.union accepts fixed_len_byte_array fields with different lengths when strict mode is on (is related to)
GroupType.union() doesn't merge the original type (is related to)

PRs and other links:

https://github.com/apache/parquet-mr/pull/315

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

Cheng Lian / @liancheng:
So deprecating non-strict schema merging seems to be reasonable? Namely, deprecate MessageType.union(MessageType toMerge, boolean strict), and always set strict to true when we call this method internally.

Julien Le Dem / @julienledem:
[~lian cheng] Did you have a patch for this.

Cheng Lian / @liancheng:
While trying to fix this issue, I got a problem regarding to the strict argument of PrimitiveType.union, and correspondingly, MessageType.union.  Seems that throughout the whole parquet-mr code base (including tests), we always call these methods with strict being true, which means schema primitive types should match.
Maybe I missed something here, but I don't see a sound use case of non-strict schema merging.  Especially, the field types of t1.union(t2, false) is completely determined by t1, rather than the "wider" ones:
message t1 { required int32 f; }
message t2 { required int64 f; }

t1.union(t2, false) =>
  message t3 { required int32 f; }

Basically we can't use such a schema to read actual Parquet files even if we add some sort of automatic "type widening" logic inside Parquet readers since the merged one above loses precision.
So my questions are:

Is there a practical scenario where non-strict schema merging makes sense?
If not, should we deprecate it? (We can't remove it since MessageType.union(MessageType, boolean) is part of the public API.)

Ryan Blue / @rdblue:
I think this is part of a larger issue of handling schema evolution. The main use case I know of for union is merging file schemas into a metadata summary file. Those are no longer really needed because each schema is resolved against the requested schema individually on the reader, which eliminates the bottle-neck that the metadata file was intended to avoid. And as you note, union doesn't really create a union as one might expect: a schema that can be used to read both of the input schemas.

Cheng Lian / @liancheng:
Just opened one for this: https://github.com/apache/parquet-mr/pull/315

Julien Le Dem / @julienledem:
Issue resolved by pull request 315
https://github.com/apache/parquet-mr/pull/315


PHOENIX-6659 RVC query fix for variable column + fixed column

Signed-off-by: Gokcen Iskender<EMAIL_ADDRESS>

@gjacoby126 @stoty Please review

@jpisaac - there are InListIT failures, could you please take a look?

@gjacoby126 This backport (PHOENIX-6659) and the backport of (PHOENIX-6662) are needed for the test failures to be green.


[musecoco] I'm still unable to download the checkpoint file.

My previous issue has been closed as resolved.
But I can't download it.
The permission request input window is still displayed, and clicking the request button only shows blank white window.
Would you check the file permission?

same as me, they forget to make this link as public access

Fixed.

Same problem with Musicbert and Telemelody🥺

Thanks for fixing it.

@heury Hey folk, Can you access the checkpoint file? I still cannot.


Prevent prototype pollution in $.deparam

See https://github.com/cowboy/jquery-bbq/issues/62 for a more in-depth description of how to reproduce the prototype pollution.
Using Object.create(null) prevents prototype pollution because the objects it creates have no prototype/constructor, and therefore does not pollute the global prototype.
You can test/confirm this for yourself by comparing the outputs of:

Object.create(null).__proto__.test = 'polluted' (should simply fail)
vs. ({}).__proto__.test = 'polluted'

jQuery/browser compatability
As a reminder, if you fork this or care about prototype pollution in jQuery BBQ, you must also be on jQuery 3.4.0+, otherwise you're still vulnerable via jQuery's $.extend.
Please also note that Object.create(null) is only supported on IE9+. This fork therefore drops support for IE6-8 (which should be the case anyway if you are on jQuery 3.4.0, which does not support $.browser).

@constancecchen This looks like a good change; I had some trouble getting this to pass the unit tests though.  Could you try running those locally? (it should be a case of opening the unit/index.html file in your preferred browser)

@constancecchen A small reminder about the unit tests here, if & when you have time - or let me know if you're likely to be too busy to take more of a look.

@constancecchen A small reminder about the unit tests here, if & when you have time - or let me know if you're likely to be too busy to take more of a look.

Super sorry about the slow response!! I've been swamped and totally procrastinated on this. It would be awesome to get some help on the unit tests, if anyone's around. For what it's worth also we ran into another issue with prototype pollution even with this PR:

http://benalman.com/code/projects/jquery-bbq/examples/deparam/?a[]=b&a[__proto__][__proto__][test]=polluted

The issue there is when an obj is initially defined as a non-object (e.g. a string or an array), the prototype of the prototype of that string/array can then by still be polluted (yay, JS). I think the issue is somewhere around line 521 of the code but I could be wrong.
With that in mind a more blunt-force solution instead ofObject.create(null) might be to simply return early if someone tries to pass a key of __proto__ at any point - they're probably up to no good 🤷 Thoughts?

@cee-chen This issue was fixed like you suggested in LimeSurvey: https://github.com/LimeSurvey/LimeSurvey/commit/0c4651b1ffe9a3218e642151ef7162709f45b92a