Support credentials for PSGalleryModule and PSGalleryNuget

We have run into an issue pulling dependencies from a private Nuget repository hosted in Azure DevOps Artifacts.  The Find-NugetPackage function used by the PSGalleryNuget dependency type errors with the following:  TF400813: Resource not available for anonymous access. Client authentication required.  Note: The actual downloading of the package still works because it uses the nuget.exe CLI which can save credentials.  This means that the "latest" version can't be used in this scenario.
The PSGalleryModule dependency type fails with a warning as well: "WARNING: Cannot access 'https://<AccountName>.pkgs.visualstudio.com/_packaging/<RepositoryName>/nuget/v2'. Are you missing 'Credential' parameter in the cmdlet?"
In both cases I think providing credentials for the repository would be useful.  I'm more than happy to submit a PR with those changes if there is interest.

@alphakilo45 Would you mind sharing out what/how you did that? I have that exact scenario I need to overcome.

@johnmccrae Sure - you can see the changes in credentials branch on my fork of the project: https://github.com/alphakilo45/PSDepend/commits/credentials  Beware though, I haven't merged in any changes since creating the fork

@alphakilo45 great suggestion. I've just run into exactly the same requirements. Thanks for sharing your changes.

@RamblingCookieMonster Is it possible to get this baked into the project? I just started using this module and find it very useful but I need to be able to support credentialed feeds. I'd rather not fork it, create a new pipeline, and host the module myself if I can help it. I'm will to contribute on this though looking at @alphakilo45 changes it looks complete.

Arg, missed this issue, code looks reasonable but like you mentioned, there have been changes to that file - if you can get these changes into the current codebase, it looks good to me!!

Absolutely - I can get things merged later this week and submit a PR (unless @rlvandaveer gets around to it first).

That's great. I may have time to tackle this week. I'll look at the branches and see what I think it'll take to resolve this issue given the state of master now.

I'm looking over PSGalleryNuget.ps1 and I'm wondering how much, if anything, needs to change. Nuget supports adding credentialed sources. It seems that Nuget should handle it as long as you have your source setup correctly. Any disagreement? I would think just updating the docs might be enough.
One items that should change in PSGalleryNuget.ps1 is to make it cross platform. I think that all references to nuget.exe should be changed to nuget i.e.,

if(-not (Get-Command Nuget.exe -ErrorAction SilentlyContinue))
{
    Write-Error "PSGalleryNuget requires Nuget.exe.  Ensure this is in your path, or explicitly specified in $ModuleRoot\PSDepend.Config's NugetPath.  Skipping [$DependencyName]"
}

and

	Invoke-ExternalCommand nuget.exe -Arguments $NugetParams

I'm working on a Mac in PowerShell Core and PSDepend works well, but obviously only because I haven't used a nuget dependency yet.

Is Find-NugetPackage.ps1 still necessary? It appears to only exist in order to do a search for all package versions. As of the version I have installed <IP_ADDRESS>35, nuget appears to do this: nuget list -source '<URL here>' -allversions
Why am I asking? I think it would be ideal to avoid configuring credentials in source control if possible. If we can rely on just nuget and credentialed sources configured in nuget.config, then that job becomes easier.

@RamblingCookieMonster, is Find-NugetPackage.ps1 still necessary? It appears to only exist in order to do a search for all package versions. As of the version I have installed, <IP_ADDRESS>35, nuget appears to do this: nuget list -source '<URL here>' -allversions
Why am I asking? I think it would be ideal to avoid configuring credentials in source control if possible. If we can rely on just nuget and credentialed sources configured in nuget.config, then that job becomes easier.

Never mind. I just executed nuget list and I can totally see why that script exists.

So, let's talk about credential strategy. I thought about what @alphakilo45 did and the shortcoming is that it leads to a user likely saving credentials to source code. I think there are some ways to save an encrypted file in Windows PowerShell but my early tests via PowerShell Core haven't been fruitful.
I had a thought that instead of directly committing the credentials to the *depend.psd1 or *requirements.psd1, that we could instead include a reference to the userid and password variables. You could then get the variables using Get-Variable. How you configure them upstream is up to you. E.g.,

        @{
            PSDeploy = @{
                DependencyType = 'PSGalleryNuget'
                Source = 'https://dev.azure.com/orgname/_packages/projectname/nuget/v2'
                Credentials = @{
                                UsernameVariable = 'env:username'
                                PasswordVariable = 'env:password'
                }
            }
        }

Is that naive? Is there a better way?

For Windows you might want to look at BetterCredentials for storing creds
or just export them as clixml, they'll only be usable by the same user on
the same machine. For PSCore it's harder as secure string isn't really
secure on non-windows (its just a string there).
On Mon, Jun 3, 2019, 21:26 rlvandaveer<EMAIL_ADDRESS>wrote:

So, let's talk about credential strategy. I thought about what
@alphakilo45 https://github.com/alphakilo45 did and the shortcoming is
that it leads to a user likely saving credentials to source code. I think
there are some ways to save an encrypted file in Windows PowerShell but my
early tests via PowerShell Core haven't been fruitful.
I had a thought that instead of directly committing the credentials to the
*depend.psd1 or *requirements.psd1, that we could instead include a
reference to the userid and password variables. You could then get the
variables using Get-Variable. How you configure them upstream is up to
you. E.g.,
    @{
        PSDeploy = @{
            DependencyType = 'PSGalleryNuget'
            Source = 'https://dev.azure.com/orgname/_packages/projectname/nuget/v2'
            Credentials = @{
                            UsernameVariable = 'env:username'
                            PasswordVariable = 'env:password'
            }
        }
    }

Is that naive? Is there a better way?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/RamblingCookieMonster/PSDepend/issues/94?email_source=notifications&email_token=AEAOUFSIPLUHJH5Q24HBZY3PYVV6HA5CNFSM4GASDKA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW2N4ZQ#issuecomment-498392678,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEAOUFV2BQTARQF45TAEX6LPYVV6HANCNFSM4GASDKAQ
.

I agree that we don't want credentials in the *.depend.psd1 files.  The intention was was to provide a hash (keyName -> credential) of the credentials to your call to Invoke-PSDepend.  The *.depend.psd1 files then reference the key needed for each module and the matching credential object is passed to the dependency provider.

I agree that we don't want credentials in the *.depend.psd1 files. The intention was was to provide a hash (keyName -> credential) of the credentials to your call to Invoke-PSDepend. The *.depend.psd1 files then reference the key needed for each module and the matching credential object is passed to the dependency provider.

Ah, I see it. I hadn't gotten to the module invocation yet. I read the changes top down and when I saw Dependency.Credentials I thought that it was configured via the dependency declaration. I think that makes sense and is likely enough.

I have created PR #100. It doesn't yet have any unit tests.

I believe PR #100 is done. Unit tests have been added and I have tested retrieving from a private repo using Nuget and PowerShellGet based modules.

It's not clear how the -Credentials parameter is supposed to work (to me at least). And despite trying numerous different ways I simply cannot get it to work.
$d = @{
	module = @{
		Name = 'module-name'
		Parameters = @{
			Repository = 'internal-repo'
		}
	}
}

$d | Invoke-PSDepend -import -install -force -Credentials @{ module = $creds } -v

I have my internal repo on an Azure DevOps Artifacts feed and the error is:
WARNING: Could not get response from query 'https://XXX.pkgs.visualstudio.com/_packaging/XXX/nuget/v2/FindPackagesById()?id='module-name'&$skip=0&$top=40'.
PackageManagement\Find-Package : No match was found for the specified search criteria and module name 'module-name'. Try Get-PSRepository to see all available registered module repositories.
At /opt/microsoft/powershell/6/Modules/PowerShellGet/PSModule.psm1:8850 char:9
+         PackageManagement\Find-Package @PSBoundParameters | Microsoft ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+ CategoryInfo          : ObjectNotFound: (Microsoft.PowerShel\u2026Cmdlets.FindPackage:FindPackage) [Find-Package], Exception
+ FullyQualifiedErrorId : NoMatchFoundForCriteria,Microsoft.PowerShell.PackageManagement.Cmdlets.FindPackage

If I run Find-Package and give it the -Credential $creds parameter it works fine. So the issue seems to be matching something with the -Credentials hash table.
Suggestions?

@pauby  What do get if you execute Gret-PSRepository? I found that for my private repo I had to register it like this. Your URL above is different and I don't know if that's an issue or not.
Register-PSRepository -Name 'MyRepo' -SourceLocation 'https://pkgs.dev.azure.com/my-org/_packaging/stuff/nuget/v2/' -PublishLocation 'https://pkgs.dev.azure.com/my-org/_packaging/stuff/nuget/v2/' -InstallationPolicy Trusted -Credential $credential -Verbose

@johnmccrae I get:
Name                      InstallationPolicy   SourceLocation
----                      ------------------   --------------
internal-repo			  Trusted              https://XXX.pkgs.visualstudio.com/_packaging/XXX/nuget/v2
PSGallery                 Trusted              https://www.powershellgallery.com/api/v2

If I use  find-module -repository internal-repo -credential $creds I get a list of the modules. So it's registered correctly.
Something I didn't mention above (because I simply forgot) is that I am running this in a Docker container locally under PowerShell Core. I am going to go back and update my original message with that and the $PSVersionTable.

Now I gotcha.. ok, so when I try to update from my internal repo I call this:
Update-Module -Name 'internal-repo'  -Credential $credential
The $credential object is the userid and token used above to register the repo.
What I'm finding is that after I create the internal repo i now get warnings and sometimes errors when I install or update modules between PSGallery and Internal-Repo. For all PSGallery or other external modules I now need to specify -Repository on the Install-Module command. Update-Module works correctly after installation

How does that translate into this issue / PSDepend?

OK, I'm a dork. I was responding to the wrong thing. Sorry.

If I run install-package -name 'module-name' -Source 'https://XXX.pkgs.visualstudio.com/_packaging/XXX/nuget/v2' -Credential $creds -providername powershellget it actually installs it.

@pauby I think the piece that's missing is that you need to specify the key for the credentials in your dependency.  For example from the PSDepend tests:
'jenkins' = @{
		DependencyType = 'PSGalleryNuget'
		Version = 'latest'
		Target = 'TestDrive:/PSDependPesterTest'
		Parameters = @{
			Force = $true
		}
		Credential = "imaginaryCreds"
	}

That key (in this case "imaginaryCreds") then needs to match the key in the hashtable passed in on the -Credentials parameter. (ex: $d | Invoke-PSDepend -import -install -force -Credentials @{ 'imaginaryCreds' = $creds })

I had a look at some fo the code. Unfortunately I've not got a lot of familiarity with it and it appears to use a global variables so it's difficult to track stuff down.
I get the error message:
WARNING: No credential found for the specified name . Was the dependency misconfigured?
``

So I [found that](https://github.com/RamblingCookieMonster/PSDepend/blob/master/PSDepend/Public/Get-Dependency.ps1#L435). Iyou look at the error message above and the code, `$Name` is empty. So I traced the code back to where the `Resolve-Credential` function is being called and it's only in one function [here](https://github.com/RamblingCookieMonster/PSDepend/blob/master/PSDepend/Public/Get-Dependency.ps1#L303) and [here](https://github.com/RamblingCookieMonster/PSDepend/blob/master/PSDepend/Public/Get-Dependency.ps1#L414). If we look for `$CredentialName` in the code you can find it [further up](https://github.com/RamblingCookieMonster/PSDepend/blob/master/PSDepend/Public/Get-Dependency.ps1#L254). The `Get-GlobalOption` function is found [further up again](https://github.com/RamblingCookieMonster/PSDepend/blob/master/PSDepend/Public/Get-Dependency.ps1#L163-L197).

Looking at the `Get-GlobalOption` function it looks like `$Output = $Default` would be executed and as `$Default` is `$null` that's what's returned. Because I don't know the code well enough I'm not sure what _should_ be returned here.

I could be reading this code completely wrong and be way off base with the issue I have. It's late and I'm tired so would really appreciate any suggestions.

@alphakilo45 That has fixed it! The above is still an issue as it doesn't fail gracefully there (but tries to display a $null which I'm assuming is because it doesn't match anything in the Credentials hashtable.
I will see if I can raise a PR to update the docs tomorrow as they don't mention Credentials at all and the function help is ambiguous.
Thank you again!


Packaging metadata

Fixes
Fixes #6 by @brylie
Fixes https://github.com/creativecommons/chooser/issues/373
Description
Adds metadata for build and publishing on npm.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main or master).
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin

For the purposes of this DCO, "license" is equivalent to "license or public domain dedication," and "open source license" is equivalent to "open content license or public domain dedication."

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Thanks, @sadn1ck, for your quick and thorough review! Is the change in dist/ file structure very disruptive, or does it seem negligible?
We'll try to merge this once we internally discuss the versioning scheme.

It's not very destructive, just apart from the name change from @creativecommons/vue-vocabulary to @creativecommons/vocabulary-components, we also have to add dist to the end,
ie,
// new
import VueVocab from<EMAIL_ADDRESS>
// old
import VueVocabulary from<EMAIL_ADDRESS>

Out of curiosity, would import VueVocabulary from '@creativecommons/vue-vocabulary work with either setup, since the package.json defines vocabulary-components.common.js as the main script?
...
"main": "dist/vocabulary-components.umd.min.js",
...

dist doesn't add any information (I assume it is a build artifact). Can it be removed without a lot of trouble?

Yes, we can remove the dist folder build artifact by reverting the build script.
echo "vue-vocabulary: Starting build"
npm run build
cp package.json dist/
cp README.md dist/
cp LICENSE dist/
echo "vue-vocabulary: Finished build"

echo "vue-vocabulary: Starting publish"
cd dist
npm publish --access public

Today, I removed the --name and --dest arguments from the build command in package.json. The build output is exactly the same, it places build artifacts in the dist/ directory with file names based on the package name (vocabulary-components).
Going forward, I would like to align with the defaults rather than shadow them in the build command or shuffle files around as was done in the previous publish.sh.
Let's verify that we can have a clean import statement import VueVocabulary from '@creativecommons/vue-vocabulary'; rather than the long and redundant @creativecommons/vocabulary-components/dist/vocabulary-components.common.js, since we have defined the entry point script in the main property of package.json.

@sadn1ck, can you please verify that the clean import statement below will work with the latest changes in this pull request? Specifically, I've added "type": "commonjs" in the package.json.
// Vue.js projects, such as Chooser, should be able to use this
import VueVocabulary from '@creativecommons/vue-vocabulary';

Or in regular JS projects:
// Regular JS projects, such as a generic script.js should be able to use this
var VueVocabulary = require('@creativecommons/vue-vocabulary');

Both of the above import statements should work, depending on the type of downstream project.

first type of import works in chooser

2nd type of import works in a freshly init node project

@brylie looks good!


docs: fix 'npm run build' + run 'npm audit fix'

npm run build would fail because the mdxparser doesn't like the comments at the top.
Also ran npm audit fix as npm was reporting some issues.

Codecov Report

Merging #10995 into master will decrease coverage by 4.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #10995      +/-   ##
==========================================
- Coverage   65.77%   61.73%   -4.05%
==========================================
  Files         815      433     -382
  Lines       38358    14216   -24142
  Branches     3607     3607
==========================================
- Hits        25231     8776   -16455
+ Misses      13019     5254    -7765
- Partials      108      186      +78

Flag
Coverage Δ

#cypress
?

#javascript
61.73% <ø> (ø)

#python
?

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

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

@mistercrunch this appears to need rebase, and for the license check to be fixed.


XML format of mediainfo command has changed

When used with the latest version of mediainfo (17.10), the parser fails with the error "Undefined index: File" because the new XML format has replaced the 'File' element with a 'media' element.
Possible fixes include detecting the new version of mediainfo and changing the command line to use "--Output=OLDXML", or to adapt the parser module to use the new XML format.

Thanks for reporting this issue. I will try to solve this soon as possible.


Please check all of the platforms you are having the issue on (if platform is not listed, it is not supported)

[x] iOS
[ ] Android
[ ] UWP
[ ] .NET Standard

Version of Library
7.0.30
Version of OS(s) listed above with issue
ios 13.1.2
ios 13.1.1
ios 13.1
Steps to Reproduce
1.create a lable with tab gesture recognizer command
2.    add this in the command "        UserDialogs.Instance.TimePrompt(new TimePromptConfig
{ OnAction = result => SetTime(result, type), IsCancellable = true } );"
3. use the settime function to change the label
Expected Behavior
to show time prompt on the screen with 2 places for selecting hours and mins
Actual Behavior
white box is shown only no text is visible on the real device (iphone xs / ios 13.1.1) but it visible on the emulator (ios 13.0)

I have same problem with:
var cfg = new DatePromptConfig
            {
                Title = "Title",
                SelectedDate = DateOfBirth
            };
var result = await UserDialogs.Instance.DatePromptAsync(cfg);

@aritchie is any solution how to fix this problem? or we need wait for new version which will fix this issue?

btw fyi @aritchie @Hackavist this bug is only in Dark mode thats why on Simulator it works. I tried now in my real device Light mode and DatePrompt works fine.

@pinkysek  thank you for this insight it is life saving i was working on my custom time picker it get it working

@Hackavist it is not needed. Add this code into ViewDidLoad() in view when you used it or in BaseViewController.
if (UIDevice.CurrentDevice.CheckSystemVersion(13, 0))
    OverrideUserInterfaceStyle = UIUserInterfaceStyle.Dark;

and add this into Info.plist
<key>UIUserInterfaceStyle</key>
<string>Light</string>

It works for now and just wait till @aritchie fix this issue in lib. Maybe you can use only value in UIUserInterfaceStyle. I didn't try it separately yet. More info https://stackoverflow.com/questions/56546267/ios-13-disable-dark-mode-changes.

So you need set UIUserInterfaceStyle value in Info.plist probably because Userdialogs creating his own view for this.

First off @pinkysek - quit pinging me please.  I'm not here as tech support.
Secondly, guy - this is terrible for OSS

It works for now and just wait till @aritchie fix this issue in lib. Maybe you can use only value in UIUserInterfaceStyle. I didn't try it separately yet. More info

@pinkysek thank you for this insight it is life saving i was working on my custom time picker it get it working

Instead of creating a custom time picker - why not give back and fix the issue here!?  Lastly, don't sit around waiting for me to fix an issue you're having.  I don't personally use Time or Date prompt in any of my active projects.
I'll look into a fix at some point in the future if I can't count on outside assistance, but I'm not making any promises as to when that will be.

Ok sorry for pinging you. I know that's not great solution but I need really fast fix for my app in production for now. If I will have more time at the weekend, I can look at this issue in this library.


upd(git-butler-deb): 0.10.22-703 -> 0.12.12-1093

Failing due to https://github.com/tauri-apps/tauri/issues/9662
what a ****show over there. The current Ubuntu LTS and  Debian testing don’t work with any of the apps compiled with this build system anymore. And they haven’t done anything for 2.5months+

Failing due to tauri-apps/tauri#9662
what a ****show over there. The current Ubuntu LTS and Debian testing don’t work with any of the apps compiled with this build system anymore. And they haven’t done anything for 2.5months+

Oh, understood. Thanks a lot.


Wind unit tests

Testing:

[x] Ran tests and they passed OK

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 3

Complexity decreasing per file
==============================
+ improver_tests/wind_calculations/wind_gust_diagnostic/test_WindGustDiagnostic.py  -2

Clones added
============
- improver_tests/wind_calculations/wind_gust_diagnostic/test_WindGustDiagnostic.py  1

Clones removed
==============
+ improver_tests/metadata/test_probabilistic.py  -1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity decreasing per file
==============================
+ improver_tests/wind_calculations/wind_gust_diagnostic/test_WindGustDiagnostic.py  -2

Clones added
============
- improver_tests/wind_calculations/wind_gust_diagnostic/test_WindGustDiagnostic.py  1

Clones removed
==============
+ improver_tests/metadata/test_probabilistic.py  -1

See the complete overview on Codacy


feat: support deno.jsonc, jsr.jsonc

This adds support for deno.jsonc and jsr.jsonc files.
I have run existing tests with pnpm test, and added a couple of .jsonc specific tests. Published on my personal npm scope @hugojosefson/fork-version for manual testing, and tested successfully with deno like this:
deno --allow-all npm:@hugojosefson/fork-version/cli

Thank you so much for this really good tool!

Good stuff, that jsonc-parser library looks good.
Thanks for you change Hugo!

Thank you, @eglavin 👍


Provide changelog

There was recently a minor uptick 0.1.x to 0.2.x, but it is difficult to tell what the change was without a changelog. Thanks! ...now at 0.4.0 :P

+1

+1 we're up to v0.4.0 and still don't know if the features I need are there.

0.4.1 just got released. what changed?

0.4.1 "Add --noHighlight option"

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

Yes we will do this from now on. Thank you for bringing it to our attention :)


OpenAPI throwing Stack Overflow exception while parsing.

I am parsing the Swagger 2.0 with OpenAPI 3.0 but open API throwing StackOverflow exception. I found that there are multiple refs that are calling itself again and again. open API is not able to terminate the recursive calls in this case.
For Example:  UserSimple definition, it is calling "$ref": "#/definitions/AccountGroup" and AccountGroup is calling again
"$ref": "#/definitions/UserSimple"
"created-by": {
 "$ref": "#/definitions/UserSimple"
        },

You can see hare if it is going to resolve UserSimple again then will call AccountGroup again and this will make an endless loop.
Also, there are multiple refs in the swagger which is also having the same problem.

Please raise tooling-related issues on the relevant repository. This repository is for the OpenAPI specification itself. You may also receive more answers on a general support forum such as StackOverflow. Thanks.


SLC Deployer: Add timeout option to CLI

In the scope of ticket #50 a new ExtractValidator has been implemented.
The current ticket requests adding options to the CLI so that an interactive user can control the timeout duration.
Proposal:
@click.option('--extract-timeout-seconds', type=int, default=10)

Open question:

Should the CLI specify a default-callback?
Should the CLI enable to select a callback?

How would interactive users know what the timeout value should be?
I don't think this option is necessary. It will just clog the documentation.

@ahsimb this option makes actually sense, because the time to wait depends on a lot of factors. And, not all of them we have under control or can detect.
Factors

Size of SLC
Files in SLC
Nodes in Cluster
type of BucketFS (SaaS or on-prem)
hardware (CPU speed, disk speed, network speed)

I agree to both comments: this option makes sense in my eyes, but still the users won't know what the timeout value should be in advance. Bad situation.
I personally would prefer having a way to adjust the timeout than having no way.


Batch commands functionality

Related to discussions in https://github.com/NodeRedis/node_redis/pull/799
It's very common to want to run a batch of commands at the same time and have a single callback for when they are completed. A lot of the time, people use MULTI for this, which is okay but adds noticeable overhead when pushing Redis hard.
An example of this is at GoSquared, where we add a new visitor to our tracking. This involves many redis commands - setting a hash of the visitor details and adding them to relevant lists what we want to run at the same time, but we don't need a multi.
As mentioned in the previous issue, I've got the functionality in redis clustr (https://github.com/gosquared/redis-clustr/blob/master/src/RedisBatch.js.) and it's been incredibly useful. Usage is as follows:
var batch = redis.batch();
batch.set('key', 'value');
batch.get('key', function(err, resp) {
  // "value"
});;
batch.exec(function(err, resp) {
  // [ "OK", "value" ]
});

Trying to use something like async to group the commands together results in awful code, this is familiar and clear (exactly the same usage as multi).
Thoughts?

So, the above isn't atomic like running multi?
I'm interested in an optimized multi-like block that will efficiently run a single command if that is all that is needed. Here is my use case:
var myAtomicBatch = client.atomicBatch();

myAtomicBatch.set('key','value');
if (isSomething === true) {
  myAtomicBatch.set('key-for-is-something','value);
}
myAtomicBatch.exec(function(err, values) {
  //...
});

So the second set command is only called under certain condition(s) but I need both the first and second commands to run together if it is called.
We do something similar to this now but just take the performance hit if the conditional is not true. If I wanted to refactor for to avoid the multi / single command issue, the code would come out fairly messy.

Exactly - this will handle that for you

@simontabor Gotcha - the first callback and the part about using async made me think you were going for a non-atomic series of commands. Also, the link to RedisBatch.js is 404'ing for me.

@stockholmux edited the link, for some reason it had a full stop at the end.
Think I've misunderstood slightly - you're looking for it to run a multi/exec block if it's more than one command but just run the single command if there's only one?
This will simply throw all the commands at redis when you call exec (non-atomic). I understand the use case for an atomic batch - that could quite easily be added too. I've not actually experienced any issues with using the non-atomic batching, especially as we use Redis Cluster and that makes multi's useless anyway.

@simontabor Right - more of an optimization for multi I suppose.
var myAtomicBatch = client.atomicBatch();
myAtomicBatch.set('key','value');
myAtomicBatch.exec(function(err, values) {
  //...
});

is optimized (internally) to something like
client.set('key','value', function(err, value) {
  if (err) { cb(err); } else { cb(err,[values]); }
});

But, if the multi queue is greater than 1, then the multi is called as normal.
It could be shimmed at the application level, but it seems like a simple / smart optimization that could make a fair bit of difference at scale.

Definitely +1, that's what I did in https://github.com/NodeRedis/node_redis/pull/799.
I see value in both an atomic and a non-atomic version - WDYT @stockholmux?

@simontabor If the syntax remains more-or-less the same for optimized multi, 'normal' multi and non-atomic batch, then the end-user of the module could decide what is best without changing much code. That could be a big win.

This is implemented by #878 and will be released in v.2.2.0.


Fixed init animation state

Fixed init animation state

@AtixD
I haven't noticed that currentPageValue needs to be initialized when initialPage is not 0. Thank you for fixing this.

@AtixD
I haven't noticed that currentPageValue needs to be initialized when initialPage is not 0. Thank you for fixing this.


worker,result: create archive_dir for the first job and mark --{seed,subset} optional

@zmc i am jumping the gun and merging it. but the change won't kick in unless we restart the workers.


SQL Query throws divide by zero errors

The fragmentation line of the select statement doesn't handle zeros for total rows, which I run into almost every time I run this.
100*(ISNULL(deleted_rows,0))/NULLIF(total_rows,0) AS 'Fragmentation'   is how I handle it

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

ID: a24a194b-16d9-79e4-8483-95aaf2833c67
Version Independent ID: 108fd3b5-631f-384f-25a0-6badb3a26428
Content: sys.dm_db_column_store_row_group_physical_stats (Transact-SQL) - SQL Server
Content Source: docs/relational-databases/system-dynamic-management-views/sys-dm-db-column-store-row-group-physical-stats-transact-sql.md
Product: sql
Technology: system-objects
GitHub Login: @stevestein
Microsoft Alias: sstein

Fixing with PR 10582.
https://github.com/MicrosoftDocs/sql-docs-pr/pull/10582
@mjlafever Hello, Thank you for this code improvement. It makes good sense. Your improvement should be visible on live Docs within 1-2 business days.
You always have the option of using either an Issue like this Issue 2281, or instead a Pull Request (PR). Today's code improvement would be an excellent fit for a PR, wherein you code edit our source documentation directly, cutting out our task of middleman. To start a PR, just click the 'Edit-pencil' icon at the top-right of the Docs webpage, and then click the similar icon on the GitHub webpage.  Not required of course, but a PR is a nicely direct option.
We hope to hear from you again!
I am now Closing this Issue 2281 as fixed.
CC: @stevestein Steve, I opted to fix this Issue, even though I am not the formal owner.  Just fyi.


Change dependent hw to allow external connections

This is the quick and dirty way that assumes <IP_ADDRESS> is mapped to localhost... we can discuss networkmode: host/using <IP_ADDRESS>/other solutions

@untzag thoughts?

Is there really no better way to do this? Why can't we use network mode host?

I mean we can play around with it... I was just going for working first...


api: Simplify drive hotplugging

The way CreatePod() was implemented, it was making an extra call
to hotplug all the drives. But when StartPod() is called, it leads
eventually to Container.start() to be invoked, resulting into a
hotplug of the drives related to this container.
A mechanism based on a flag was in place to detect if the drive had
been previously hotplugged, but there is no need for such thing
anymore. Indeed, we know that we are hotplugging only from the
container implementation.

lgtm


switch off device automatically at the end of configured running time

sometimes SHM does not send a switch off recommendation, so device stays on

according docu we must not send planningrequests if planned time is already in the past. beside that we need to decrease MinRunningTime and MaxRunningTime when device is on. -> need to implement it...

Will include an option to turn off the device at the end of the time period, regardless of whether there is a recommendation to turn off from the SHM

option added in 0.0.2


cancelOffer doesn't work

Calling cancelOffer will get an empty response like this
{
	"response": {

	}
}

I tried to call the web API CancelTradeOffer (v1) by Send HTTP tool and got the same response.
Is that means the API has been broken and we can't use this method?

Does it cancel the offer you called it for? Steam Web API is a mess and I warn in README that you shouldn't expect any meaningful response from this method.

@Alex7Kom it doesn't cancel the offer.
And it doesn't return any error.
I have read the README and I thought at least the offer can be canceled.
I don't understand if this API doesn't work why it exsits.

Thank you, I'll investigate.

Are you cancelling an offer which you have sent? You can't decline offers with this function.

cancelOffer works fine for me. Haven't checked the return object but the offer is cancelled.

I found the reason, thanks for all the help.
I have passed wrong parameter to the cancelOffer.
The response object from makeOffer is
{tradeofferid:xxxxxxx}
, but the cancelOffer need
{tradeOfferId:xxxxxxx}.
When I pass the right object to the cancelOffer, the method will work.
The offer has been cancelled correctly. But still return
{
"response": {
}

}
Hope this will help other careless people just like me.

Hi i still cant decline/cancel the offers even though i followed the correct building of parameter
These are my code: where tradeofferidA -> is the tradeofferid
var params2 = {
tradeOfferId:tradeofferidA
}
offers[rows[0].bot_steam_id].declineOffer(params2, function (err, callback) {
if (err)
{
console.log("Cancel offer ERROR: " + err);
}
else
{
if (Object.keys(callback).length > 0)
{
console.log("Cancel offer with callback");
console.log(callback);
}
else
{
console.log("Callback error");
}
}
});
If i display the params2 via console.log it is displayed as something like this
{ tradeOfferId: '428073619' }
Kindly help urgent


Amazon Q Connect unable to make API calls when embedded

When Amazon Q Connect is embedded within an iframe the library is unable to make API calls due to a bug in communicationProxy.ts in the subscribeToChannel function.
This line of code tries to check if the event's source.location is not equal to either the window.parent.location or the window.top.location.
if ((e.source as Window)?.location !== ((window.top || window.parent).location)) return;
If it isn't then it exits the function and does not continue to make the API call. The issue is that window.top will never be null so the expression (window.top || window.parent).location will always evaluate to window.top.location. When Amazon Q Connect is embedded e.source.location will never equal window.top.location so the function will always exit early.
A proposed solution would be to individually check e.source.location against both window.top.location and window.parent.location

When Amazon Q Connect is embedded

To add to this, when the app calling the amazon-q-connectjs library is embedded as an iframe App inside of Amazon Connect, it fails to communicate with the API because of these conditions. If the app is opened standalone (not in an iframe), this works without issue.


Move tests to ginkgo

Based of https://github.com/rancher-sandbox/elemental-cli/pull/19

@Itxaka I see you are moving many tests to ginkgo.  Feel free to leave some to me if you feel like.

@Itxaka I see you are moving many tests to ginkgo. Feel free to leave some to me if you feel like.

Can leave you the partition ones maybe as you created them, you already know about them :D


Integrate isomorphic-dompurify additions into dompurify

Re: https://github.com/cure53/DOMPurify/issues/691#issuecomment-1143548009
it may be a good step forward to join forces and merge the isomorphic-dompurify changes with the parent project dompurify

What do you think @NicholasEllul?

The dompurify project received a couple of changes one, two which make me think that we can postpone merging the projects.


feat Support preheating for stats

Proposed changes
Load the most recently used stats as FE booted.
StatsCacheLoader will record the column(tblid-idxid-colname) it loaded to a map. This PR add a shutdown hook to FE so that we could write the records in map to the metadata.
When booting, Stats of columns that recorded previously would be loaded by a single thread.
NOTE
This PR add new operation type of log, so the metadata is not backforward compatitable, once user or dev run their FE on the code of this version they cannot rollback to previous FE version.
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

if there is no history information, can we load some stats on FE startup?

if there is no history information, can we load some stats on FE startup?

No

i think just load top k newest column statistics is good enough, log the cache status is not necessary

and we should load the statistics info after analyze job success immediately

and we should load the statistics info after analyze job success immediately

This logic already exists currently.

if there is no history information, could you just load k stats?


Serial output overlaps floating keyboard buttons/rounded corners

Describe the issue
When connecting to a VM via serial the serial output goes all the way from the very top to the very bottom of the screen and thus is hidden both by the rounded corners at the top of the dipslay and the floating extra keyboard buttons at the bottom. The second is particularly anoying as after filling the screen the cursor is behind the floating keyboard buttons as well making it impossible to see what is being typed without clearing the screen.

Configuration (required)

UTM Version: 4.5.3 build 99
OS Version: 18.0 beta 3
Device Model: 11 inch M1 ipad pro
Is it jailbroken (name jailbreak used)? No
How did you install UTM? Appstore

Debug log
For all issues, including crashes, you should attach a debug log. Open UTM, and open the settings for the VM you wish to launch. Near the bottom of the configuration options (or at the top of the QEMU page) is Debug Log. Turn it on and save the VM. After you experience the issue, quit the VM and re-launch UTM. Open the VM settings again and select Export Log... and attach it here.

+1 for me.
It's great to add some padding or make an item in the menu to customize the horizontal/vertical padding.


Covariance matrix of algebraic parameters associated with the original (unnormalised) coordinate system is inflated.

The covariance matrix of the algebraic parameters associated with the original (unnormalised) coordinate system is incorrect due to an oversight in the implementation. I incorrectly unit normalised a parameter vector when I shouldn't have just before applying a critical formula.
This issue does not affect the covariance matrix of the geometric parameters, nor does it affect the covariance matrix of the algebraic parameters in the normalised coordinate system. Hence, if you used the normalised algebraic covariance matrix or the covariance matrix of the geometric parameters for any subsequent calculations/propagations your results are correct. However, if for some reason you utilised the covariance of the algebraic parameters in the original (unnormalised) coordinate system then you would have had an inflated covariance matrix. That is, your error bars would have been greater than they needed to be.

Fixed the bug in the latest commit. Accidentally logged in with an old GitHub account.


Fixes #2374 IPython does not work in specific project

Fixes #2374 IPython does not work in specific project
Changes script locator to look in interpreter's directory for projects.

👍


Unable to find default nuget package manager path to restore and deploy

Repro steps:
Publish the Dev translator  app
unable to restore the dll with default nuget package manager while deploying.
Action: azureFunctions.deploy
Error type: Error
Error Message: Failed to deploy path that does not exist: c:\Users\username\Documents\IoTWorkbenchProjects\examples\devkit_translator\Functions\bin\Release\netcoreapp2.1\publish
Version: 0.14.0
OS: win32

@dilipprasad does dotnet restore and dotnet build succeed in your project? If not - can you copy the logs here? It seems like you're failing to restore some nuget package, which means the build will fail and the path to deploy won't exist.


fix spacing on match/cosmetics

Currently, each item is weirdly spaced.

@blukai could you take a look at this?  Should be pretty easy fix:
http://hchung.westus.cloudapp.azure.com:8080/matches/2731466093/cosmetics

Ye, I'll do it sometime today


Does the lib support IP address ?

I use OKHttp in my project and wonder whether I could use ip instead domain (wildcards) as below ?
val interceptor = certificateTransparencyInterceptor {
    +"my_ip1",
    +"my_ip2"
}

val client = OkHttpClient.Builder().apply {
    addNetworkInterceptor(interceptor)
}.build()

In theory there's no reason why we couldn't add support, it's just not a scenario we'd thought much about.
Just so I can better understand the use case, is there a particular reason why you'd prefer to use IP address?


plot_arcs_compare

region argument in plot_arcs_compare does not behave as expected. This argument still displays the entire sequence x-axis but only plots interactions that begin prior to the region cutoff. Even if the interaction extends beyond the region cutoff, the entire arc is displayed. For example:
plot = rnav.plot_arcs_compare(
samples=[6kb_RNA1, 6kb_RNA2],
sequence="ss",
plot_kwargs={"dpi": 50},
interactions="pairprob",
region=[1,2000]
)
returns a 6kb x axis with the full sequence displayed. Arcs exist for the first 2kb. Some arcs begin before nucleotide 2000 but extend to nucleotide 2200 and are still fully displayed.
On another note - it would also be very useful to be able to plot annotations on these graphs. They currently don't accept that argument.

Thanks Scott,
This is a duplicate of #29,  I will post another option there.


Inference code to evaluate model on pointcloud files

@xinge008
Do you have Inference code to evaluate model on pointcloud files?

@ChidanandKumarKS  the inference code is shown in train_cylinder_asym.py (see val part). For one point cloud input, you can modify the dataloader to suit your requirements.

@xinge008
Do you have Inference code to evaluate model on pointcloud files?

Hey there!
Did you find the way to perform inference?
i am trying to predict a pointcloud and would really appreciate if you can provide me with your code or some pointers.

@xinge008 i have the inference code shall i create a pull request ??

@xinge008 i have the inference code shall i create a pull request ??

Hey!
could you please share your code on google drive or some similar service?
the pull request might take some time and i really need to have this done by tomorrow.
would really appreciate your help.

@wakasu
For inference on SemanticKITTI or nuScenes, I have provided the code example in https://github.com/xinge008/Cylinder3D/blob/master/train_cylinder_asym.py#L82-L107;
For testing a point cloud from other source, you can modify the dataloader with your own path, like https://github.com/xinge008/Cylinder3D/blob/master/dataloader/pc_dataset.py#L49;
If it is urgent, maybe you can directly use the code from @abhigoku10 ;

@xinge008
Thanks for the pointers.
it will take time to modify the code to use our dataset and that's the thing i don't have right now. lol
will wait for @abhigoku10 to send his code.

@wakasu  share your mail id so that i can send it across

@abhigoku10
Can you also send me to my mail id<EMAIL_ADDRESS>

@abhigoku10
<EMAIL_ADDRESS>Thanks a lot for your help

@wakasu @chowkamlee81  i have uploaded it on git  https://github.com/abhigoku10/Cylinder3D_PC.git

Hi @abhigoku10,
Can you also send your inference code to me?
<EMAIL_ADDRESS>Thx in advance!

Hi @abhigoku10 ,
could you also send me the inference code?
<EMAIL_ADDRESS>Thank you very much.

Hi @abhigoku10 ,
I arrive much later but it would be helpful if you also send me your inference code. Could you?
<EMAIL_ADDRESS>Thank you very much!

Hi @abhigoku10
I arrived late, but could you send me the inference code? it would be really helpful.
<EMAIL_ADDRESS>Thank you very much!


Example with ajax items

Needed an example with fetched items by ajax.

@FDiskas the directive currently doesn't support dynamically changing the media attribute.
Please refer to my comment here.
I have a plunkr that demonstrates the use case, linked in the same comment.


Additional  theming options in the workbench

This is https://github.com/Microsoft/vscode/issues/3112


Smart Editor: When all layer are in update only mode, selecting existing features does not bring up the attribute inspector/editor.

Widget
Smart Editor
Version of widget
1.0 Alpha
Bug or Enhancement
Bug
Repo Steps or Enhancement details

Recent changes seems to introduce thsi issue.  I recalled testing this before and it's working.

User error: I tested with updateOnly + no editable layer.


Binding ViewControllers

Hey Auros, found a new bug regarding UI:

When binding a ViewController with "Container.BindViewController", it isn't curved correctly if it's shown on the left/right (already discussed in #pc-mod-dev)
When binfing a ViewController with "Container.Bind<>().FromNewComponentAsViewController()" it'll be curved correctly but won't be bound again on an app restart, for example when you hit apply in mod settings, resulting in NullReferenceExceptions.

I have already deprecated BindViewConteoller for a while and the next update will now mark it as obsolete. Your second issue is literally impossible unless you're using a persistent singleton or DontDestroyOnLoad somewhere in your code.

I have already deprecated BindViewConteoller for a while and the next update will now mark it as obsolete. Your second issue is literally impossible unless you're using a persistent singleton or DontDestroyOnLoad somewhere in your code.

I indeed used DontDestroyOnLoad. For the next update, you might want to update the readme then. Even though, thanks for the help and for making SiraUtil in general!

I indeed used DontDestroyOnLoad. For the next update, you might want to update the readme then. Even though, thanks for the help and for making SiraUtil in general!


Resolves #693, upgrade ego to v1.3

What this PR does / why we need it:
Upgrade ego to v1.3, current ego version is v1.0
Which issue(s) this PR fixes:
Fixes https://github.com/hyperledger/fabric-private-chaincode/issues/693
Special notes for your reviewer:
Successfully build docker-dev in local
can build local dev and test ego using the code below
cd $FPC_PATH/utils/docker
make build-dev
make run-dev

Here is the result:
root@docker-desktop:/project/src/github.com/hyperledger/fabric-private-chaincode# ego
EGo v1.3.0 (360a6a40836461465fdbd0742dfb0f980b68c638)
Manage and run EGo enclaves.
...

But require FPC developer to help to push the last base-dev image to GitHub so user can easily use make pull-dev
instead of make build-dev

@Flamewind97 I've just found out that there is already ego 1.3. What about checking if this version also works nicely and we then skip 1.2 :) WDYT?
https://github.com/edgelesssys/ego/releases/tag/v1.3.0

ya sure leave it to me

Updated to ego v1.3.0
successfully test and build
fpc-basic-asset-transfer:main
ego sign
EGo v1.3.0 (360a6a40836461465fdbd0742dfb0f980b68c638)
Generating new private.pem
ego uniqueid ecc > mrenclave
EGo v1.3.0 (360a6a40836461465fdbd0742dfb0f980b68c638)
ego bundle ecc ecc-bundle
EGo v1.3.0 (360a6a40836461465fdbd0742dfb0f980b68c638)
Saved bundle as: ecc-bundle
DOCKER_BUILDKIT=1 docker  build  --build-arg FPC_CCENV_IMAGE=hyperledger/fabric-private-chaincode-ccenv:main --build-arg SGX_MODE=SIM --build-arg CAAS_PORT=9999 \
	--build-arg OE_SIMULATION=1 \
	-t fpc/fpc-basic-asset-transfer:main \
	-f /project/src/github.com/hyperledger/fabric-private-chaincode/ecc_go/Dockerfile \
	. \
&& DOCKER_BUILDKIT=1 docker  tag fpc/fpc-basic-asset-transfer:main fpc/fpc-basic-asset-transfer:latest

@Flamewind97 Thanks for the update - can you please fix the DCO issue :)


Unable to run test using plugin in non-gui mode

Hi Team,
It is to request your support regarding using this plugin in non-gui mode.
Scenario: When trying to run this in non-gui mode. Test run gets stuck at 'Citrix Application Launcher'. Works fine in GUI mode.
Please if anybody could assist regarding this issue urgently.
Can reach out to me at<EMAIL_ADDRESS>Thanks
Pritpal Singh

Hi @3dgiordano,
Could you please us know any update on this issue? Running the Jmeter Citrix in non-gui mode.
Its executing till 'Request ICA File' request in non-gui mode, but not executing the 'Citrix Application Launcher' request. However the same script executes fine in GUI mode.
Regards,
Madeppa


Plugins: add pre connected/disconnected monitor events

Describe your PR, what does it fix/add?
This is a very simple PR, it simply adds two events for plugins: preMonitorAdded and preMonitorRemoved. These events are fired when a monitor is connected/disconnected, but before Hyprland handles it. This can be useful for plugins such as hyprland-virtual-desktops, to avoid listening to workspace change events when monitors are being disconnected/connected
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Is it ready for merging, or does it need work?
ready, but this probably needs to be documented in the hyprland wiki. I'll open a PR there once this is approved

@vaxerski here it is https://github.com/hyprwm/hyprland-wiki/pull/875


Fix for no user input

If the property is on a doctype but user dosnt enter anything it is saved as a "string" and not as a an object with empty properties. Here is the simpel fix :)
Btw AWESOME package!

Sweet =D Thanks a lot for the PR! =D

sure thing man ! :)


FEATURE REQUEST: When hovering over a clickable word in a Palauan definition, show a quickview tooltip of the word.

Instead of having to navigate to a new webpage for every word, when hovering the cursor over a Palauan word in a Palauan definition or in the See Also or Synonyms area, show a tooltip of the word, origin, pos, English def, and Palauan def. If the user wants more details, he can click the word.
Ideas:
Use smaller print so it does not take up the whole screen.
Allow it to display multiple entries for homonyms.

Great idea. Don't know how hard this would be to do. I'll look into it.
Nice job following the convention to use 'FEATURE REQUEST' in the title.

This would be easy to do when we build the page by going through every word and doing a query and populating a tooltip for each word. That would be brutal slow to build the page though and would basically destroy the user experience. So we need to populate the tooltip at runtime which is harder.
This page discusses a similar requirement.

Nice fade-in effect

Ok. Super close now. I did all the hard parts that I didn't know how to do. The remaining should be straight-forward.

BOOM! Thanks @ChrisPerrette . This is pretty awesome. Mostly I'm on my phone so I won't appreciate it but it's definitely awesome when on computer. I think this is done so I'm closing it myself. But please re-open if it doesn't work for you or if you have a preference for doing it somehow differently.

Oh, yeah, works for homonyms too:

Hmm, I do wish I could make it wider though.....

It looks awesome, and it is definitely what I pictured! One suggestion, can homonyms be ordered the same way search results are ordered? That way, for example, the state verb diak can precede the derivative possessed noun diak.

Figured out how to change the width.
Fixed the order. Was scared it would be hard but turned out to be super easy.

Do you have an iphone?  But it looks good in chrome. And the long-touch works for me too; neat!
Shoot, looks like I broke something in IOS/Safari. Ugh.

Nevermind. Looks fine in IOS/Safari now. I followed instructions to wipe history but it appears that it was somehow cached. All good!

I have an Android phone, and I use Google Chrome. I had to clear my cache on Chrome to see some of the changes you mentioned.


Textual change on Ramp account.

Sub title should be - “Buy with ApplePay, Bank or Card”

-"Account Details" > "Setup Ramp Account"

@antuz123 would it be better to use “BTC purchased from Ramp” as the account description?

-Verified this on Dev app v 1.4.6(233)


Potential bugs from *_base_slice() on different architectures

AbstractFieldExtension::{as,from}_base_slice() are not really well-defined since they expose the internal choice of representation of the extension. If we have two different machine architectures whose arithmetic for a particular field extension uses two different representations (easy to imagine with CPU vs GPU or vector vs serial), then calling u = fe.as_base_slice() on one machine and then v = FE::from_base_slice(u) on another could result in garbage in v.
IMHO, the guiding principle should be: Any operation that depends on the internal representation of elements of a field extension needs to be defined inside the field extension code. Exposing the internal representation is asking for trouble.
Thoughts?

Hm I see how this would be a concern if Base was an associated type, so different (conditionally compiled) impls could declare different bases. But since it's a generic, there could be multiple impls, e.g. StarkConfig expects two impls -
type Challenge: ExtensionField<Self::Val> + ExtensionField<Self::Domain> + TwoAdicField;

With that in mind, I think the methods seem okay since the caller is specifying the encoding; there could still be ambiguity from certain call sites but I would argue that it's a caller issue.
(Aside: winterfell has a similar trait namedExtensionOf<Base>, maybe that name is more clear to hint that multiple impls are expected?)

Hmm, I think I might not have explained myself clearly (or perhaps I've misunderstood your response). The concern is not about the choice of Base, it's about a (conditionally compiled) choice of extension field model.
For example, suppose we have a base 𝔽_p with p=29 and we want to represent elements of 𝔽_{p^2} = 𝔽[X]/(f(X)). If device A implements f = X^2 - 2 and device B implements f = X^2 + X + 1 (could be CPU and GPU in the same box), then calling u = fe.as_base_slice() on A and then v = FE::from_base_slice(u) on B will result in the wrong value in v. This Sage script shows what I mean by showing how multiplication gives different answers (it "doesn't commute") with an "as/from" base slice:
sage: _.<x> = GF(29)[]
sage: K.<a> = GF(29^2, modulus = x^2 - 2)
sage: L.<b> = GF(29^2, modulus = x^2 + x + 1)
sage: a^2
2
sage: b^2
28*b + 28
sage: s = 23*a + 26  # random element of K
sage: t = 18*a + 14  # random element of K
sage: s * t                 # product in K
7*a + 3
sage: (23*b + 26) * (18*b + 14)   # L::from_base_slice(s.as_base_slice()) * L::from_base_slice(t.as_base_slice())
28*b + 8                   # not equal to L::from_base_slice((s*t).as_base_slice())

Does that make sense?

After further discussion (offline) we have decided that a full resolution of this issue can be postponed until we have some competing field extension models that make it concrete, at which point we'll address how best to handle it. In the meantime, an upcoming PR will add some comments to the affected functions to alert users of their dependence on implementation details.


Reland (2): "Fix how Gradle resolves Android plugin"

Previous PR: #137115,
Revert: #142464
Fixes #141940
Closes #142487
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Tests work as expected for enabled gradle daemon (before the fix): https://github.com/flutter/flutter/pull/142498/checks?check_run_id=21166626702

CC @bartekpacia @camsim99 @reidbaker

@reidbaker small ping if there's time to finish the review :)

I added a teammate who can give this a faster review. Sorry for the delay.

Tests fail as expected for enabled gradle daemon (before the fix): https://github.com/flutter/flutter/pull/142498/checks?check_run_id=21166626702
I reverted the revert with 309767c and added the tests with 851fe0d and added the fix with cddc017

The test added LGTM but can you add some context as to why cddc017
fixes https://github.com/flutter/flutter/issues/141940?
Also pinged @gmackall for a review if you can since you are more of a Gradle expert!

Thank you :)

The test added LGTM but can you give some context as to why cddc017
fixes https://github.com/flutter/flutter/issues/141940?

As stuartmorgan described in https://github.com/flutter/flutter/issues/141940#issuecomment-1912645081 a Singleton persists across builds in gradle daemon. So the Singleton keyword is removed in cddc01714d0a1d6b25788a0347ab2e2b603480d7. To not read the same file twice during the same build, some local variables were introduced (pluginList and pluginDependencies) in flutter.groovy to ensure consistency and push performance.

I would defer to someone who reviewed the original PR (I think the delta here between original and reland is small) but can take a look if no one else is able to. It'll take me a bit though, as I'll have to review (original+reland delta)

Thank you for reviewing!


Added sample for difference schemes based on build variants

Closed #8

Link to http://weibo.com/2263023493/FmHejqyZ9?type=comment


Pkg 4587 - v12.4 to main

Destination channel: defaults
Links

PKG-4587
Upstream repository
Upstream changelog/diff

Explanation of changes:
Previously, this was forked from conda-forge and some changes made for various reasons (see PR 1). Since then, conda-forge's recipe got updated to v12.4.
This rebases the changes we made onto their changes. It also removes the abs.yaml because we want to upload the packages to the main channel now.

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_feyecdzdh3/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:55: license_file_overspecified: Using license_file and license_url is overspecified.

===== ERRORS =====

clone/recipe/meta.yaml:55: missing_dev_url: The recipe is missing a dev_url
clone/recipe/meta.yaml:55: missing_license_family: The recipe is missing the about/license_family key.
===== Final Report: =====
2 Errors and 1 Warning were found

do we always keep .conda-forge.yml when forking from conda-forge?

here we want to minimise changes, see above

do we always keep .conda-forge.yml when forking from conda-forge?

here we want to minimise changes, see above

I had a similar question as well even after reading the description. May I ask why we want to replicate conda-forge when a lot of these changes seem to be useless on our end? It seems like we are adding a bunch of fluff to the feedstock. Is the sole benefit to save time or are there other benefits I'm unaware of for doing this?

@Jrice1317 There is tens of these cuda recipes. If we start to hand-pick which files we shuld update, it'll make the process tedious and slower than it already is.

conda-forge will update the recipes repeatedly for new cuda versions. we want to be able to just pull their changes with as little work our side as possible. Cutting out the fluff makes more changes = more stuff to worry about when they do update.
In general the conda-forge stuff is unsightly, but we only care about stuff in the recipe folder and abs.yaml, so you can just ignore it. A matter of taste really, I have come to live with it

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_15g1dihtsz/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:55: license_file_overspecified: Using license_file and license_url is overspecified.

===== ERRORS =====

clone/recipe/meta.yaml:55: missing_license_family: The recipe is missing the about/license_family key.
clone/recipe/meta.yaml:55: missing_dev_url: The recipe is missing a dev_url
===== Final Report: =====
2 Errors and 1 Warning were found

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_27y72srvce/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:55: license_file_overspecified: Using license_file and license_url is overspecified.

===== ERRORS =====

clone/recipe/meta.yaml:55: missing_license_family: The recipe is missing the about/license_family key.
clone/recipe/meta.yaml:55: missing_dev_url: The recipe is missing a dev_url
===== Final Report: =====
2 Errors and 1 Warning were found


Fixed a few linting issues.

Would you welcome contributions that primarily fix lint issues (this PR being an example)?  Some issues are pretty minor, so feel free to push back on areas where you prefer your style.
I didn't include it here but
counter = 0
train_data.reset()
for batch in train_data:
    counter += 1
print(counter)

can also be written as
print(sum(1 for _ in train_data))

I personally find the 'for batch in train_data' most readable.

I like these stylistic choices and will try to adhere to them. Please keep this feedback coming, I'm grateful to have your eyes on things and keeping the code quality up and the style uniform.


Readme typo

"feed free to join...". I think it should be "feel free to join ..."

Proposed fix: #7


WIP: migrate to miette

currently this produces output like:

(I imagine you'll want that to be better...)
This PR has three fundamental parts (all done together since they have to be done in lockstep):

update to the miette version of axoasset (currently via git, so super don't merge)
update OrandaError to be a proper miette::Diagnostic
do a crapton of stuff in main to register a bunch of hooks and mess with colors and json output

Unlike libraries, applications "should" register hooks to produce final miette output (otherwise they'll behave like normal thiserror Errors). Also note that to emit a miette Diagnostic you need to:

wrap it in a miette::Report
Debug format it ({:?})

This is for... Reasons that I can't remember the full details of. If you Display format it ({}) you'll get thiserror-style output. I forget what the Report does, if anything. Honestly I might be cargo-culting that at this point.
json error output is currently dead code, but I figured I'd toss it in there tentatively since you'll probably want it for Web Services. A lot of this code is messy gunk that's been copied several times. This version is copied from cargo-dist. If there's anything you Hate there's probably a decent argument for it. I think for instance the current panic hook will swallow the actual backtrace..? I thought I fixed this in cargo-dist but I might be wrong. I guess technically it's prettier but... bleh.

I've never bothered to configure it before but there's a bunch of stuff in miette to tune the fancy output that we could probably use to make things more seamless: https://docs.rs/miette/latest/miette/struct.GraphicalTheme.html

Oh yeah, and if you want to wrap another Diagnostic, you need to add #[diagnostic(transparent)] in addition to `#[error(transparent)] to forward everything.
For instance adding that to axoasset changes the screenshot in the first comment to this one:

(note that "help" properly funneled through)

i'm likely gonna integrate the work here into https://github.com/axodotdev/oranda/pull/241 and close it- there's not a super clean way for me to pull commits out of this unfortunately

https://github.com/axodotdev/oranda/pull/241/commits/574bd8033a5055152ff693570dc492cf2b1f3297


print lisp form instead of internal representation

When invalid syntax is reached, print out offending form instead of the internal representation.
Instead of printing out
> (fn <> [form] form)
I did not understand the form `XObj {obj = Lst [XObj {obj = Fn Nothing (fromList []), info = Just (Info {infoLine = 1, infoColumn = 2, infoFile = "REPL", infoDelete = fromList [], infoIdentifier = 0}), ty = Nothing},XObj {obj = Sym <> Symbol, info = Just (Info {infoLine = 1, infoColumn = 5, infoFile = "REPL", infoDelete = fromList [], infoIdentifier = 0}), ty = Nothing},XObj {obj = Arr [XObj {obj = Sym form Symbol, info = Just (Info {infoLine = 1, infoColumn = 9, infoFile = "REPL", infoDelete = fromList [], infoIdentifier = 0}), ty = Nothing}], info = Just (Info {infoLine = 1, infoColumn = 8, infoFile = "REPL", infoDelete = fromList [], infoIdentifier = 0}), ty = Nothing},XObj {obj = Sym form Symbol, info = Just (Info {infoLine = 1, infoColumn = 15, infoFile = "REPL", infoDelete = fromList [], infoIdentifier = 0}), ty = Nothing}], info = Just (Info {infoLine = 1, infoColumn = 1, infoFile = "REPL", infoDelete = fromList [], infoIdentifier = 0}), ty = Nothing}`.

Print the following instead.
> (fn <> [form] form)
I did not understand the form `(fn <> <> [form] form)` at line 1, column 1 in 'REPL'.

This isn't perfect because it doesn't print the exact input form, but it is better than it was.

Thank you!


Make executing commands on a host with sudo optional

@ebd2 this change addresses one of your comments on a previous PR that I forgot to incorporate.

LGTM if everything passes.


Dbks potential cluster bootstrap fix

Resolves #141
What is being addressed
This (hopefully) fixes the repeated issues we've seen when trying to deploy clusters. Following some investigation with the help of a support engineer, the reason why we might be seeing this only in the uksouth region currently is because each region has different IPs for various dependencies Databricks relies on and the routing might be clashing with other resources we have set up.
How is this addressed
As per https://learn.microsoft.com/en-gb/azure/databricks/administration-guide/cloud-configurations/azure/udr?WT.mc_id=Portal-Microsoft_Azure_Support#--configure-user-defined-routes-with-azure-service-tags I've added a routing table with some service tags for these dependencies to ensure the routing happens correctly each time.
In my testing I've encountered no issues in uksouth however only time will tell.

/test

:robot: pr-bot :robot:
:runner: Running tests: https://github.com/UCLH-Foundry/FlowEHR/actions/runs/4547529475 (with refid 9814ea04)
(in response to this comment from @jjgriff93)

are these IPs all static and unchanging!? happy to push this through and let us all feel how we go over the next week or so - maybe if we're getting the same issues still we might revert it? again - nice work getting into this :/

@damoodamoo yes, seems like madness doesn't it but that's what the docs say. We're using service tags wherever possible but they aren't available for extended infrastructure. Yep, think that's the only option - give it a try and hope, tested several times but who knows

The TRE is using purely IPs without service tags so it could be worse: https://github.com/microsoft/AzureTRE/blob/main/templates/workspace_services/databricks/terraform/databricks-udr.json ><

/test-force-approve
Tests passed, destroy failed (fix tracked separately)


ion-refresher: Emit to ionPull on 'cancelling' state.

Short description of what this resolves:
Allow developer have access to 'cancelling' event/state by output events with 'ionPull'.
This event described in documentation, but not available for developers https://github.com/ionic-team/ionic/blob/master/src/components/refresher/refresher.ts#L114
Changes proposed in this pull request:

Emit 'this' object by 'ionPull' on 'cancel' action.

Ionic Version: 3.x

@adamdbradley ping


Change the build script to copy test folders named "resources" in the…

…ir entirety to be able to use any type of files in the tests

LGTM


Fix do.sh to copy warning files + copy the _locales directory properly on OSX

Two changes:

Warning files don't actually get copied to $BUILD_EXT_DIR when running ./do.sh build_extension.
On OS X, cp -fR _locales/  "$BUILD_EXT_DIR" actually copies the contents of _locales instead of the directory itself.

I tested the fix under OS X 10.10.3 and Ubuntu 14.02.

Thanks for fixing this!
It looks like the lint command in do.sh is also probably broken.


Illuminate\Broadcasting\Broadcasters\PusherBroadcaster::broadcast :119

I do not understand why this error appears in https when I want to execute the event, in http it works without problems.

Try to activate log in the pusher connection in broadcasting.php, and see what error is logged in your log files.
    'connections' => [

        'pusher' => [
            // ...
            'log' => true, <-- THIS
            'options' => [
             // ....

Intente activar logla conexión de empuje broadcasting.phpy vea qué error se registra en sus archivos de registro.
    'connections' => [

        'pusher' => [
            // ...
            'log' => true, <-- THIS
            'options' => [
             // ....

How do I solve this?

Can you try this out in the options array of the pusher connection:
'options' => [
        // ...
        'curl_options' => [
             CURLOPT_SSL_VERIFYHOST => 0,
             CURLOPT_SSL_VERIFYPEER => 0,
        ]
 ],

I think to disable it on development should be ok (?).

¿Puedes probar esto en la optionsmatriz de la conexión del empujador?
'options' => [
        // ...
        'curl_options' => [
             CURLOPT_SSL_VERIFYHOST => 0,
             CURLOPT_SSL_VERIFYPEER => 0,
        ]
 ],

Creo que deshabilitarlo en el desarrollo debería estar bien (?).

Thank you very much, I solved it by updating the SSL and adding,
'curl_options' => [
CURLOPT_SSL_VERIFYHOST => 0,
CURLOPT_SSL_VERIFYPEER => 0,
]

Thank you very much, I solved it by updating the SSL and adding,
'curl_options' => [              CURLOPT_SSL_VERIFYHOST => 0,
CURLOPT_SSL_VERIFYPEER => 0,         ]
El jue., 9 ene. 2020 a las 18:59, Viktor Geringer<EMAIL_ADDRESS>escribió:

Can you try this out in the options array of the pusher connection:
'options' => [
// ...
'curl_options' => [
CURLOPT_SSL_VERIFYHOST => 0,
CURLOPT_SSL_VERIFYPEER => 0,
]
],
I think to disable it on development should be ok (?).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/beyondcode/laravel-websockets/issues/298?email_source=notifications&email_token=AHWXPNJPPD76IYCJFLB7XIDQ46M4TA5CNFSM4J55VHYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIR53GI#issuecomment-572775833,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHWXPNI2U5GGV6CIY3PXMMDQ46M4TANCNFSM4J55VHYA
.


Update CodeQL link in README

QL was moved from semmle/ql to github/codeql.

Sure, have changed it now. Also thanks for linking to the style-guide and explaining the rationale, I wasn't aware that this is the US style.


Use voluptuous for uber

Description:
Migration of the configuration check to voluptuous. There is also an upgrade of uber_rides included.
Related issue (if applicable): fixes 127528299
Example entry for configuration.yaml (if applicable):
  platform: uber
  start_latitude: 37.8116380
  start_longitude: -122.2648050
  end_latitude: 37.7768520
  end_longitude: -122.4155500
  server_token: 'BeAPPTDsWZSHLf7fd9OWjZkIezweRw18Q8NltY27'
  product_ids:
    - '04a497f5-380d-47f2-bf1b-ad4cfdcb51f2'

@robbiet480, would be nice if you could take a look at the changes and run a quick test. Thanks.

🐬 !! 👍

LGTM - second set of eyes requested


TypeError: Cannot read property 'build' of undefined

Hello. Getting this error while trying to build factory. Here's definition:
myapp/test/factories/user.js
/* eslint camelcase: [2, {properties: 'never'}] */

import FactoryGuy from 'ember-data-factory-guy';

FactoryGuy.define('user', {
  default: {
    ...omited args...
  }
});

Inside one of my tests -
import { build } from 'ember-data-factory-guy';
....
let user = build("user");
....

This throws me following error:
Source: | TypeError: Cannot read property 'build' of undefined     at FactoryGuy.buildRaw (http://localhost:4243/assets/vendor.js:74915:25)     at FactoryGuy.build (http://localhost:4243/assets/vendor.js:74893:26)

ember version - 3.3.0
ember-data-factory-guy - 3.4.0(tried 3.3.0 as well)
Any ideas how to fix this?

let user = buildList("user",  2);

Sorry, forgot to modify code. I've updated example to latest version where it crashes

sounds like you have something really wrong in your your ember build .. i would have to look at the factory-guy package and see what is going on because this is a very very unusual situation ( never seen it before )

Any else info I can provide to solve this?

I would recommend you start a new ember project ( with ember new test-project )
then
install factory guy, ember-data , ember-source ( the fewest packages possible )
then
make a user model  and make a user factory with no default attributes and write a test where you build a model  just like you did
and you will see that this works =>  build('user')
then
add more models, packages, and code  till it breaks

Ok, will try this way. Thanks!

Just adding for those who will encounter this later. When using new test api, make sure to add setupFactoryGuy(hooks);  after test module declaration. It is mentioned in readme, but in other section than how to use

Hey @avdept, I wonder if there's an opportunity here to open a PR to improve the README?

good point @lorcan ... if you can fix the README @avdept .. I would be thrilled. You would be perfect to fix it because you probably know exactly where to put that info on setting up tests

@lorcan @danielspaniel I will submit PR with readme improvements soon.


Data can be filtered by date

A date filter is provided that affects all charts in the dashboard.
This enables the user to select dates relative to the current one, e.g.:
7d | 30d | 90d | 180d | Max

https://user-images.githubusercontent.com/91760036/177338911-8f30d160-832e-45d1-867c-b3b5d3c4ddb8.mp4


What about tomato or cilantro?

Could be good! Though I understand if you hate cilantro.

Cilantro is the best! I am lucky that I lack the sad genes which make you taste soap.


Switch the repo CI to GitHub Actions?

The workshop CI uses CircleCI and the setup is copied from what Adrian Price-Whelan set up for the astropy-workshop repo. Should we switch to GitHub Actions?

I deferred implementing something for #47 since switching CI schemes might change whether it's needed/desired.  If download_file() timeouts continue to be an issue in CI, then we can consider implementing #47 (or some other workaround).

The pyvo and astropy version updates were handled by #48.

Done in #62. Outstanding todo items to improve the overall CI experience is tracked in #63


Build Web Package Error

Hi, I have a problem with this error:
2016-04-13T11:21:19.9243029Z dnu build for:
2016-04-13T11:21:19.9253033Z     ".\src\..\My.Project.Web"
2016-04-13T11:21:20.0433082Z Microsoft .NET Development Utility Clr-x86-1.0.0-rc1-16231
2016-04-13T11:21:20.1523036Z Building .\My.Project.Web for DNX,Version=v4.6
2016-04-13T11:21:23.5699072Z ##[error]System.Management.Automation.RemoteException: Error: Name contains invalid characters.
2016-04-13T11:21:23.5709070Z ##[error]System.Management.Automation.RemoteException: Parameter name: assemblyName
2016-04-13T11:21:23.5709070Z ##[error]System.Management.Automation.RemoteException:
2016-04-13T11:21:23.6772773Z dnu publish for:
2016-04-13T11:21:23.6782801Z     .\src\..\My.Project.Web
...
...
2016-04-13T11:21:56.6069257Z [11:21:56] Finished 'build-release' after 991 ms
2016-04-13T11:22:56.5349255Z ##[error]System.Management.Automation.RemoteException:
2016-04-13T11:22:56.5359260Z ##[error]stream.js:94
2016-04-13T11:22:56.5369273Z ##[error]      throw er; // Unhandled stream error in pipe.
2016-04-13T11:22:56.5369273Z ##[error]            ^
2016-04-13T11:22:56.5379277Z ##[error]Error: Cannot find module './$.to-object'
2016-04-13T11:22:56.5969250Z ##[error]System.Management.Automation.RemoteException: The 'prepublish' script failed with status code 1.
2016-04-13T11:22:56.5989246Z ##[error]System.Management.Automation.RemoteException:
2016-04-13T11:22:56.6099235Z ##[debug]Leaving script BuildWebPackage.ps1
Do you have any idea?

I think you tried my suggestion to use ".." in your project name. this does obviously not work.
I rewrote the code which handles this and added a parameter where you can specify your folder.
Please have a look here: #3

@lchodkiewicz Is this still a problem with the new version?

Hello Kirkone,
I am also getting the same error...
[error]System.Management.Automation.RemoteException: npm WARN<EMAIL_ADDRESS>No description
[error]System.Management.Automation.RemoteException: npm WARN<EMAIL_ADDRESS>No repository field.
[error]System.Management.Automation.RemoteException:
[error]System.Management.Automation.RemoteException: npm WARN<EMAIL_ADDRESS>No license field.
[error]System.Management.Automation.RemoteException:
any other hack which I should try ?

@narottamgoyal
This seems more like an error on the npm install than the error discribed above.
I think the warnings from npm are interpreted as errors and so the tast will fail.
I will have a look  into this.
so long
KirK

Thank you very much Kirkone for your quick response!!


How do i include image file using assets?

Hi team,
I am trying to get asset url for setting up image url in image tag, but i failed as i don't understand how to get asset url for asset bundle.
I already have assetBundle. now for image tag to display image what i should use? should i user asset path (if i get) or i can also use path alias?
example:
i have assetBundle which set path for @vendor/assets/ and i have image at @vendor/assets/img/[image file].
can i use @vendor/assets/img/[image file] for image so that it will get overwrite after rendering?
regards
Mithun Mandal

@qiangxue what if i put all my assets in the root folder together with the images attached to it , will the images be successfully rendered after I have declared a $sourcePath pointing to the root folder of the asset file?

I had created a theme and placed in vendor->bower folder. I accessed my theme image files as under:


Duplicated a load of tests for the new ConvertTo functions

These tests are pretty much a duplicate of the ones being used to test ConvertToV1*. The functionality of these functions are the same but return either a v1 or v2 view struct.

LGTM


calcFontSize - Should use measureText to get starting size

Maybe increase the padding to 2x or 3x instead of 1.5x to the height only, width should be accurate.  Also only need to calculate the height by checking the bottom of the image data and moving up.

This is already done.


Virtual mode doesn't play well with effects

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/swiper-effect-cards-react-forked-ey5mux?file=/src/App.jsx
Bug description
Virtual mode doesn't play well with some of the effects, for example, the cards effect. The actual behavior is hard to describe. Below is a screen recording which demonstrates the bug.
The realIndex is also wrong. At beginning of the recording, I change from slide 0 to slide 2, and then back to slide 0. Notice that it printed "2 2 0" instead of "2 1 0".
You should move the slides in a quick and short manner. If you slide slowly, it kinda works.
https://user-images.githubusercontent.com/10150604/202657287-9d8fbae5-96bb-4015-ab01-949bd132aa37.mov
Expected Behavior
No response
Actual Behavior
No response
Swiper version
8.4.4
Platform/Target and Browser Versions
macOS Firefox 106
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

+1 for this!

Issue is closed because of outdated/irrelevant/not actual/needed more information/inactivity.
If this issue is still actual and reproducible for latest version of Swiper, please create new issue and fill the issue template correctly:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with issue


Duplicated attribute class when adding custom classes

Hi,
When adding a custom class to an input the attribute class is added twice.
$this->Form->input('name',array('class'=>'input-lg'))

Returns the following

	Name
	<input type="text" name="name" class="form-control input-lg"  class="input-lg" id="name" />

The attribute class exists two times. Most browser don't care about it, but it's not W3C valid and a bit  confusing.

Should be corrected by https://github.com/Holt59/cakephp3-bootstrap-helpers/commit/3177b84d56df5a9ef87f427a098a83b2fc3b9ad3, I thought I had made this change months ago but apparently I did not push it...

works great. thanks for the quick fix.


Versioning

📚 Context
There are several components involved in the development of DApps, each with their own versions:
We have machine-emulator:0.14.0 and machine-emulator-tools:0.11.0, both are part of machine-emulator-sdk:0.15.0. On top of that we have server-manager:0.7.0.
Then we have cartesi rollups 0.9.0 framework, with several different Docker images, all with the same version, but with a strict compatibility with the emulator, and external APIs for GraphQL and Smart Contracts.
The DApp developer needs to deal with some of these versions. The machine-emulator-tools must be part of the application cartesi machine. The frontend uses the GraphQL and Smart Contracts APIs. The node, running locally for testing, must use the correct version.
For the end-user, the DApp developer, it's very hard, or impossible, to keep track of all these versions and compatibility.
✔ Solution
One alternative is to build a compatibility matrix between all components. And try to allow mix and match different components versions.
For example allow a backend built with emulator 0.14.0 to be executed by a node 0.11.0.
The other alternative is to create a "Sunodo SDK" version, that binds together Emulador SDK version and Rollups version.

For the end-user, the DApp developer, it's very hard, or impossible, to keep track of all these versions and compatibility.
One alternative is to build a compatibility matrix between all components. And try to allow mix and match different components versions.

I think all components should start checking their Cartesi dependencies version, to ensure compatibility, otherwise a friendly error message must be raised to the user. For example, if Rollups v0.9.0 depends on Emulator v0.14.x, it should check that emulator is 0.14.x, otherwise raise an error. If there are a lot of components to be checked, it could have a versioning file (such as package.json for npm).
Following semver specifications, we should usually assume that any patch release (x.y.z where only z changes) maintains compatibility. While a feature release (x.y.z where y changes) or a major release, may not be compatible.
In the end, we need somehow to be able to configure a cascade of dependency versioning, with its dependencies versions being checked at runtime by every major component, this way we could make sure the components are compatible with each other and raise friendly error messages otherwise.

Versioning is being resolved in the SDK Docker image level.


Correct date of release 0.4.0 in CHANGELOG

Minor formatting error in date of v0.4.0 in CHANGELOG.md

Done in commit d17e559


Update loader dependencies

This includes a couple major version updates. I've checked the release notes and tested in order to ensure things are compatible. It supersedes a few pull requests from Snyk (#284, #286).
The only package this does not update is Prettier, since it contains changes that would format our code differently (https://prettier.io/blog/2021/05/09/2.3.0.html). I’ll leave that for a later time, and we should probably do it across all projects at once.

Accidentally committed out of sync lockfile.


Gave topk some love

I got a little inspired yesterday and gave the topk package some love. The version that was here before used a map and a pointer to the minimum element, which almost works. Unfortunately it falls apart when you repeatedly update the minimum element so that it's no longer the minumum. I replaced it with a map that indexes into a priority-queue, so that you can always track the minimum element as updates are applied.
There's also a shameless link in there to a blog post I wrote about this paper, but I think it's fairly helpful. ;)

see #16


Aj/filter flags

Add flags for the first four edmf filters to help debug implicit simulations.

bors r+


Get inventory func, how to get object item in json

How to get item object from inventory.items?

Inventory.items is a dictionary using the app Id and the context Id. For example, If you have loaded the trading cards and want to use the result, you'll use
Inventory.items[753][6] and you'll have a list of items.
Btw I'm not sure but I don't think the issues on GitHub are supposed to be used for "I don't know how to use the library queatuons", rather for bugs, enhancements and you know, this kind of things.

Yeah, I'd prefer you stopped asking us here how to use the library. It's still on development so if you want to use it you will have to figure it out yourself.


ReadStateAsync throw stops creating grains when a GrainState changes and no longer

Some changes to GrainState will result in deserialization errors during the SimpleSQLStorageProvider's ReadStateAsync. The resulting throw on SimpleSQLServerStorage.cs line 147 causes a silo fault and stops creation of the grain until the database state is manually cleared (deleting rows from the KeyValueStore table).
Suggest suppressing the throw and ClearStateAsync on any GrainState that cannot be deserialized. Perhaps make the throw a config option.

do we have a unit test for this?


boards: remove pulpino board

This board is unmaintained and unsupported. It is not known to work and
has lots of conditional code across the tree that makes code
unmaintainable.

Since running Pulpino on Zedboards is a rather popular development platform in (my area of) academia the removal is a bit surprising to me. I am aware of the various problems maintaining support for legacy hardware brings but simply dropping support without much (public) discussion(?) does not foster any confidence TBH.
Is/was there any todo list indicating what would have been needed to keep it/update the worst offending parts? I can see some obvious problems/special cases for pulpino in the diff of this commit but some explanations to allow estimating the effort of fixing and re-adding support would be very nice - in case Zephyr is interested in that at all of course.


Broken color styles with animated CircleLayer

Describe the bug
It looks like colors are missing or not applied to animated circles when using expressions styles. The circle color defaults to black.
To Reproduce

Create a new react native app.

npx react-native init AwesomeProject

Install RN mapbox.

npm install @react-native-mapbox-gl/maps --save

Replace App.js with the code snippet below.

Run the app.

Code:
import React, {useState, useEffect} from 'react';
import MapboxGL from '@react-native-mapbox-gl/maps';
import {StyleSheet, Animated} from 'react-native';

const circleStyles = {
  circleColor: [
    'step',
    ['get', 'val'],
    '#6FBF73',
    1.0,
    '#A2CF6E',
    2.0,
    '#FFCD38',
    4.0,
    '#FFAC33',
    5.0,
    '#FF784E',
    6.0,
    '#F6685E',
    7.0,
    '#ED4B82',
    8.0,
    '#E666FB',
    9.0,
    '#AF52BF',
  ],
  circleRadius: 20,
  circleOpacity: 1,
};

const shape = {
  type: 'FeatureCollection',
  features: [
    {
      type: 'Feature',
      properties: {
        val: 2,
      },
      geometry: {
        type: 'Point',
        coordinates: [-105.8203125, 48.69096039092549],
      },
    },
    {
      type: 'Feature',
      properties: {
        val: 6,
      },
      geometry: {
        type: 'Point',
        coordinates: [-88.154296875, 51.28940590271679],
      },
    },
    {
      type: 'Feature',
      properties: {
        val: 8,
      },
      geometry: {
        type: 'Point',
        coordinates: [-90.3515625, 40.44694705960048],
      },
    },
  ],
};

const styles = StyleSheet.create({
  container: {
    flex: 1,
  },
});

const App = () => {
  const [opacity] = useState(new Animated.Value(1));
  const [radius] = useState(new Animated.Value(15));

  const expandAnimation = Animated.parallel([
    Animated.timing(opacity, {
      toValue: 0,
      duration: 1000,
      useNativeDriver: false,
    }),
    Animated.timing(radius, {
      toValue: 15 * 3,
      duration: 1000,
      useNativeDriver: false,
    }),
  ]);

  const circleAnimation = Animated.loop(expandAnimation);

  useEffect(() => {
    circleAnimation.start();
  }, [circleAnimation]);

  return (
    <MapboxGL.MapView
      styleURL={MapboxGL.StyleURL.Light}
      style={styles.container}>
      <MapboxGL.ShapeSource id="shapeSource" shape={shape}>
        <MapboxGL.Animated.CircleLayer
          id="animatedCircle"
          style={[circleStyles, {circleRadius: radius, circleOpacity: opacity}]}
        />
      </MapboxGL.ShapeSource>
    </MapboxGL.MapView>
  );
};

export default App;

Expected behavior
The circles render with animations and correct colors. The colors are based off the Feature's value property in the GeoJSON data.
Actual behavior
The circles render with animations but with black color.
Screenshot

Video
https://user-images.githubusercontent.com/12175684/159401269-d77ef0cf-3c13-4ad5-9c76-6a1f47fe60da.mp4
Versions (please complete the following information):

Platform: Android and iOS
Platform OS: Android 9 and iOS 15.2
Device: all
Emulator/ Simulator: yes
Dev OS: OSX 11.6.4 Win11
Mapbox GL version: 8.6.0-beta.0
React Native Version: 0.67.4

The issue seems to be that RN supports animated styles but styles are different for RN than for rnmapbox, objects are valid as style values in RN  but arrays are non valid in RN style values.
See:
https://github.com/facebook/react-native/blob/main/Libraries/Animated/nodes/AnimatedStyle.js
The solution would be writing our own animated wrappers.
class MyComponent extends React.Component {
  setNativeProps(props) {
    console.log('### props', props);
  }
  render() {
    return <View />;
  }
}

const AnimatedMyComponent = Animated.createAnimatedComponent(MyComponent);

function _nativeRef(ref) {
  return ref.refs.nativeLayer;
}

describe('MyComponent', () => {
  test.only('testSetNativePropsForStyle', () => {
    const baseStyle = {
      array: [1, 2, 3],
      object: { a: 1, b: 2 },
      constantValue: 42,
    };

    let animatedValue = new Animated.Value(10.0);
    Animated.timing(animatedValue, {
      toValue: 20.0,
      duration: 1000.0,
      easing: Easing.linear,
      useNativeDriver: false,
    }).start();

    const setNativeProps = jest.fn();

    let myComponentRef;
    // eslint-disable-next-line no-unused-vars
    const testRenderer = TestRenderer.create(
      <AnimatedMyComponent
        id="my-component"
        style={[baseStyle, { animatedValue }]}
        ref={(ref) => (myComponentRef = ref)}
      />,
    );

    myComponentRef.setNativeProps = setNativeProps;

    expect(setNativeProps).toHaveBeenCalledTimes(0);
    for (let i = 0; i < 5; i++) {
      clock.tick(200);
      clock.fireRequestAnimationFrames();

      expect(setNativeProps).toHaveBeenCalledTimes(1 * (i + 1));
      expect(setNativeProps).toHaveBeenCalledWith({
        style: {
          animatedValue: 10 + (i + 1) * 2,
          object: {},
        },
      });
    }
  });

Valid bug, but not planned do to lack of resources. Closing as not planned.


Consistency: Client and server routes for Modelling and Text exercises

Is your feature request related to a problem?

Both modeling and text exercises have similar routes. However, the naming is usually different and sometimes it doesn't fit our Route Naming Convention.
Describe the solution you'd like

We should implement consistent routes for these types of exercises and have it as an example for any future exercise types that will be added.

We unified client routes when we refactored the angular app structure. Server routes were also unified. This is why I am closing this>


Set supported python version to >=3.9 - 807

[Auto generated]
Number: [#807]

cloud-tools[pull_request][9b72-11ef-9e16-5d9a9c9aefa5][PR 807]: Closing issue for PR: Set supported python version to >=3.9.
PR was closed.


feat(ui): add ability to scroll task output

Description
Adds ability to scroll through task output when the task is not interacted with. This is so we retain the ability for users to enter Ctrl-P/Ctrl-N to the underlying task.
Current keybinds: Ctrl-P for scroll up and Ctrl-N for scroll down. Ctrl-Up and Ctrl-Down are more intuitive, but conflict with default macOS keybinds.
Testing Instructions
https://github.com/vercel/turbo/assets/4131117/3c76c9f4-2e26-4e51-8fde-80d803b708b3
Closes TURBO-2682

I have tried this but in Warp [1] it shows ^P or ^N. Not sure if it's a configuration issue on my side. It's working fine in Terminal.app
[1] https://warp.dev


library: Updating COPY_ARG_xxx() macros to use memcpy().

There are memory alignment issues across architectures, hence
Updating COPY_ARG_xxx() macros to use memcpy() to avoid
failures caused by memory alignment.
Signed-off-by: Aastha Grover<EMAIL_ADDRESS>

Is there another PR to the main zephyr repo that updates the manifest to pull in this module change?

@MaureenHelm There isn't one but I am planning to submit one today which would pull this change.

Functionally, the change looks good.
Stylistically, I would recommend aligning the '' characters at the end of the line to keep the code neat and tidy.

Done. Thanks!!