[BUG] - Quality Assurance projects have user stories related to Information Security

Describe your problem and how to reproduce it:
With the roll-out of the Python curriculum, the full stack curriculum was restructured and Information Security was split off from the Quality Assurance section and given its own section. However, three of the Quality Assurance projects still include user stories that expect features associated with the helmet.js learning - that is now covered under the Information Security section.
This may be something that falls under expecting campers to do their own research, but it seems worth considering the removal of these user stories.
Add a Link to the page with the problem:
Metric Imperial Converter
Issue Tracker
Personal Library

Thanks for bringing this up, Nicholas.  If the info sec certificate came before QA this would be less of a problem, but right now those user stories do seem out of place.
cc @scissorsneedfoodtoo @RandellDawson

Any thoughts on this @RandellDawson @Sky020 @scissorsneedfoodtoo?

I agree with removing the test related to any Helmetjs content

and/or the project you can fork on repl.it. Not sure on those last two.

From what I can tell, the link to a Repl of the boilerplate is just a direct method to clone the GitHub boilerplate. So, just the READMEs and views/index.html would need to be changed as well.

@nhcarrigan, thanks for bringing this up and sorry for the late response.
I'm all for removing these user stories from the projects because Info Sec isn't covered in any of the practice projects. We discussed this internally before breaking the original QA and Infosec certificate up and decided to leave them in at the time, but I always felt a bit iffy about it.

There should be three parts to this:

[ ] Remove tests/text from /learn pages
[ ] Remove text from boilerplates
[ ] Remove text from example projects

I think we might be able to close this. The user stories are removed on master (/learn), and as we implement the changes for https://github.com/freeCodeCamp/freeCodeCamp/issues/39721, they should all be removed.

@nhcarrigan Are you fine with closing this issue and adding any relevant information as a comment to issue #39721?

Sure thing.
@SaintPeter and I worked together on a few of these items and I believe everything has been caught, but I'll make a note on the other issue to ensure it doesn't get lost in the shuffle. 🙂


Unable to build in VS - Visual Studio support for Microsoft.Build.Sql

SqlPackage or DacFx Version: 161
.NET Framework (Windows-only) or .NET Core: Windows
Environment (local platform and source/target platforms):

Steps to Reproduce:

Attempt to load/build a new sqlproj based on template 0.1.7
Load fails
adding this enables the build:

    <TargetDatabaseSet>True</TargetDatabaseSet>
    <TargetFrameworkVersion>v4.8</TargetFrameworkVersion>
    <TargetFrameworkProfile />
    <ProjectGuid>{f1ee546e-2489-4a7b-9d69-7e2849143721}</ProjectGuid>

@ErikEJ can you even build the project in VS?
even with the 4 lines you mentioned, I get
"Error:  : Your project does not reference ".NETFramework,Version=v4.8" framework. Add a reference to ".NETFramework,Version=v4.8" in the "TargetFrameworks" property of your project file and then re-run NuGet restore."
full project contents:
<?xml version="1.0" encoding="utf-8"?>
<Project DefaultTargets="Build">
  <Sdk Name="Microsoft.Build.Sql" Version="0.1.3-preview" />
  <PropertyGroup>
    <Name>My.Database</Name>
    <DSP>Microsoft.Data.Tools.Schema.Sql.SqlAzureV12DatabaseSchemaProvider</DSP>
    <ModelCollation>1033, CI</ModelCollation>
    <TargetDatabaseSet>True</TargetDatabaseSet>
    <TargetFrameworkVersion>v4.8</TargetFrameworkVersion>
    <TargetFrameworkProfile />
    <ProjectGuid>{0e4d9e11-0dd4-4f05-9caa-684d32f16f99}</ProjectGuid>
  </PropertyGroup>
</Project>

@Real-JD-UK you are not using the latest SDK version

Hi @ErikEJ , can you be more specific on how you create the project and where you are trying to build it?
I have created a sample SDK style project on Azure Data Studio (1.40.0) and was able to build it in ADS. Are you trying to open the project in the Visual Studio? If so, the SDK style project is supported in VS yet!
With the project I was able to build "dotnet build "path/sampleproj.sqlproj" /p:NetCoreBuild=true", but msbuild is failing with "our project does not reference ".NETFramework,Version=v4.8" framework"! is this what you are referring about?

Yes, as the title of the issue states this is about building in VS. Its all fine that you can build in limited cross platform tooling  but the circle is not complete until you can build in VS and get the intellisense and table editing experience.

I had a typo in my previous comment and edited. We do have a future enhancement for the VS to support SDK style projects and will support that in future.

Support for Microsoft.Build.Sql sqlprojects is planned for future Visual Studio (SSDT) releases.  Reopening this item until it is shipped

I had a similar issue, and I was able to resolve this by deleting the obj folder for my project. And so, I set up the project to clean the project and to delete the obj folder before clean. It's still a work in progress, but I landed on the code below to both build in Visual Studio 2022 and via the dotnet cli. The dotnet cli seems to build just fine without deleting the obj folder, but it wasn't deleting the folder, so I just added ContinueOnError="True".
<?xml version="1.0" encoding="utf-8"?>
<Project DefaultTargets="Build" ToolsVersion="4.0">
  <Sdk Name="Microsoft.Build.Sql" Version="0.1.7-preview" />
  <PropertyGroup>
    <Name>sample</Name>
    <DSP>Microsoft.Data.Tools.Schema.Sql.Sql160DatabaseSchemaProvider</DSP>
    <ModelCollation>1033, CI</ModelCollation>
    <ProjectGuid>{939b2fc6-e398-4920-a5d4-7b3816c0b3a5}</ProjectGuid>
    <TargetDatabaseSet>True</TargetDatabaseSet>
  </PropertyGroup>
  <ItemGroup>
    <Folder Include="Properties" />
  </ItemGroup>
  <Target Name="CleanOutputPath" BeforeTargets="Clean">
  <!-- Remove obj folder -->
  <RemoveDir Directories="$(BaseIntermediateOutputPath)" Condition="Exists('$(BaseIntermediateOutputPath)')" ContinueOnError="True" />
</Target>
  <Target Name="PreBuildClean" BeforeTargets="Build">
    <Message Text="PreBuildClean - Starting clean"/>
    <CallTarget Targets="Clean" ></CallTarget>
    <Message Text="PreBuildClean - Finished clean" />
  </Target>
</Project>

@ErikEJ can you even build the project in VS?
even with the 4 lines you mentioned, I get "Error: : Your project does not reference ".NETFramework,Version=v4.8" framework. Add a reference to ".NETFramework,Version=v4.8" in the "TargetFrameworks" property of your project file and then re-run NuGet restore."
full project contents:
<?xml version="1.0" encoding="utf-8"?>
<Project DefaultTargets="Build">
  <Sdk Name="Microsoft.Build.Sql" Version="0.1.3-preview" />
  <PropertyGroup>
    <Name>My.Database</Name>
    <DSP>Microsoft.Data.Tools.Schema.Sql.SqlAzureV12DatabaseSchemaProvider</DSP>
    <ModelCollation>1033, CI</ModelCollation>
    <TargetDatabaseSet>True</TargetDatabaseSet>
    <TargetFrameworkVersion>v4.8</TargetFrameworkVersion>
    <TargetFrameworkProfile />
    <ProjectGuid>{0e4d9e11-0dd4-4f05-9caa-684d32f16f99}</ProjectGuid>
  </PropertyGroup>
</Project>

True
what this property does?

For those with Visual Studio issues? Are your files loading properly in Visual Studio? Also, if I have a .sql file in my output directory (as in the one created from a build), it won't load the project. Has anyone seen this issue?

When I try to build in visual studio after building in AzureDataSudio I get this error:
"Error: : Your project does not reference ".NETFramework,Version=v4.8.1" framework. Add a reference to ".NETFramework,Version=v4.8" in the "TargetFrameworks" property of your project file and then re-run NuGet restore."
If I delete the project.assets.json file from the obj folder then it will build fine.

When will have support for SDK-style projects in Visual Studio? It seems like an easy-to-do feature that would be very useful for may of us, but seems to be delayed for more than a year already.

With the new version, "0.1.15-preview", I can no longer build from the command line if I have the fix that CSharpFiasco posted above to build from Visual Studio. However, if I don't have that fix, I still can't build from Visual Studio.
The error:
\Microsoft.PackageDependencyResolution.targets(267,5): error NETSDK1004: Assets file '<filepath>\project.assets.json' not found. Run a NuGet package restore to generate this file.
Running dotnet restore did not work. Only deleting the PreBuild target made it work.

Sharing what my customized .sqproj file ended up being for others who come across this thread while trying to troubleshoot new use of this project sdk:
<?xml version="1.0" encoding="utf-8"?>
<Project DefaultTargets="Build">
  <Sdk Name="Microsoft.Build.Sql" Version="0.1.15-preview" />
  <PropertyGroup>
    <Name>YourProjectName</Name>
    <DSP>Microsoft.Data.Tools.Schema.Sql.Sql160DatabaseSchemaProvider</DSP>
    <ModelCollation>1033, CI</ModelCollation>
    <ProjectGuid>{your-project-guild}</ProjectGuid>
    <TargetDatabaseSet>True</TargetDatabaseSet>
    <!--
      The sdk will target netstandard2.1 automatically when doing a dotnet cli build but we need to target
      netframework when the msbuild type is full (a.k.a. building from within Visual Studio) until the Visual Studio
      SSDT extension supports the new sql project SDK.
    -->
    <TargetFrameworkVersion Condition="'$(MSBuildRuntimeType)' != 'Core'">v4.8</TargetFrameworkVersion>
    <TargetFrameworkProfile Condition="'$(MSBuildRuntimeType)' != 'Core'" />
  </PropertyGroup>

  <!--
    NOTE: The sql project sdk automatically includes a default globbing pattern for sql files but we still need them to be listed
    explicitly for VisualStudio SSDT to display them

    See: https://github.com/microsoft/DacFx/blob/main/src/Microsoft.Build.Sql/docs/Converting-Existing.md#default-sql-files-included-in-build
    See: https://github.com/microsoft/DacFx/blob/a888164f1e83f0b64f130875600d90205adb7c23/src/Microsoft.Build.Sql/sdk/Sdk.props#L35
  -->
  <ItemGroup>
    <Build Include="**/*.sql" />
    <Build Remove="bin/**/*.sql" />
    <Build Remove="obj/**/*.sql" />
    <!-- Also Build Remove any .sql file patterns you want to exclude   e.g. -->
    <Build Remove="StaticDataScriptsReferencedInPostDeploy/**/*.sql" />

    <!-- Explicitly include your folders so that VisualStudio SSDT will display your files -->
    <Folder Include="OtherFolder" />
    <Folder Include="StaticDataScriptsReferencedInPostDeploy" />
    <Folder Include="Tables" />

    <None Include="README.md" />
    <None Include="StaticDataScriptsReferencedInPostDeploy/**/*.sql" />
  </ItemGroup>

  <ItemGroup>
    <PostDeploy Include="Script.PostDeployment.sql" />
  </ItemGroup>

  <!--
    Building through visual studio (targeting net48) will fail if a previous build through the dotnet core cli
    has run and created the obj/project.assets.json file. This custom target should delete
    the file before the visual studio build starts.
  -->
  <Target Name="PreBuild" BeforeTargets="PreBuildEvent" Condition="'$(MSBuildRuntimeType)' != 'Core'">
    <Message Importance="high" Text="Ensuring the $(MSBuildThisFileDirectory)\obj\project.assets.json file is removed, if necessary, so that the database project can be built through VisualStudio SSDT without errors" />
    <Delete Files="$(MSBuildThisFileDirectory)\obj\project.assets.json" />
  </Target>

</Project>

@RyanThomas73 Thanks for sharing!

@EklipZgit
The bottom of the convert existing doc you linked does already state part of the steps needed here Even though build will honor the default globbing pattern, .sql files that are not included explicitly will not be displayed inside Solution Explorer in SSDT. Support for this will be added in a future release of SSDT
That being said, the .md doc could benefit from that callout about using it in conjunction with SSDT

Being moved to the top of the doc
And being made more visually prominent
And including a link to this thread so readers can find the latest and greatest comments on workarounds consumers have found

Tried 1.19-preview with vs2022 17.10.1
get  (d
efault target) (14) ->
(ResolveNuGetPackageAssets target) ->
C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Microsoft\NuGet\17.0\Microsoft.NuGet.targets(198,5):
error : Your project does not reference ".NETFramework,Version=v4.7.2" framework. Add a reference to ".NETFramework,Ve
rsion=v4.7.2" in the "TargetFrameworks" property of your project file and then re-run NuGet restore.
If I try passing "/p:NetCoreBuild=true" to get around, get this isntead
C:\Users\cmire.nuget\packages\microsoft.build.sql\0.1.19-preview\Sdk\Sdk.targets(52,3): error MSB4019: The imported project "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\MSBuild\Sdks\Microsoft.NET.Sdk\targets\Microsoft.NET.DefaultAssemblyInfo.targets" was not found. Confirm that the expression in the Import declaration "C:\Program Files\Microsoft Visual Studio\20
22\Enterprise\MSBuild\Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.DefaultAssemblyInfo.targets" is correct, and that the file exists on disk.

Tried 1.19-preview with vs2022 17.10.2 (Community Edition)
Failed with MSB3030: Could not copy the file "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\normnfd.nlp" because it was not found.
Logs:
Rebuild started at 11:05 pm...
------ Rebuild All started: Project: ProductDatabase, Configuration: Debug Any CPU ------
		Ensuring the C:\Work\SQLExample\ProductDatabase\\obj\project.assets.json file is removed, if necessary, so that the database project can be built through VisualStudio SSDT without errors
		C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\Roslyn\csc.exe /noconfig /nowarn:NU5128,1701,1702,2008 /fullpaths /nostdlib+ /errorreport:prompt /warn:4 /define:DEBUG;TRACE /errorendlocation /preferreduilang:en-US /highentropyva+ /reference:"C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\mscorlib.dll" /debug+ /debug:full /optimize- /out:obj\Debug\ProductDatabase.dll /subsystemversion:6.00 /target:library /warnaserror- /utf8output /langversion:7.3 "C:\Users\andre\AppData\Local\Temp\.NETFramework,Version=v4.7.2.SqlClrAttributes.cs"
		Loading project references...
		Loading project files...
		Building the project model and resolving object interdependencies...
		Validating the project model...
		Writing model to C:\Work\SQLExample\ProductDatabase\obj\Debug\Model.xml...
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(4983,5): Error:  MSB3030: Could not copy the file "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\normnfd.nlp" because it was not found.
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(4983,5): Error:  MSB3030: Could not copy the file "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\normnfc.nlp" because it was not found.
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(4983,5): Error:  MSB3030: Could not copy the file "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\normnfkc.nlp" because it was not found.
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(4983,5): Error:  MSB3030: Could not copy the file "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\normidna.nlp" because it was not found.
C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\amd64\Microsoft.Common.CurrentVersion.targets(4983,5): Error:  MSB3030: Could not copy the file "C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework\.NETFramework\v4.8\normnfkd.nlp" because it was not found.
Done building project "ProductDatabase.sqlproj" -- FAILED.

Using project file based on what @RyanThomas73 provided.
<?xml version="1.0" encoding="utf-8"?>
<Project DefaultTargets="Build">
  <Sdk Name="Microsoft.Build.Sql" Version="0.1.19-preview" />
  <PropertyGroup>
    <Name>Product Database</Name>
    <ProjectGuid>{3ed9c874-1009-4efe-8eb5-00fea496dfd8}</ProjectGuid>
    <DSP>Microsoft.Data.Tools.Schema.Sql.Sql160DatabaseSchemaProvider</DSP>
    <ModelCollation>1033, CI</ModelCollation>
    <TargetDatabaseSet>True</TargetDatabaseSet>

    <!-- Nuget specific tags -->
    <PackageId>My.Databases.Product</PackageId>
    <Version>$(Version)</Version>
    <Authors>Andrew Sumner</Authors>
    <Owners>Andrew Sumner</Owners>
    <Company>Company Name</Company>
    <Product>Product Name</Product>
    <Description>Package for Product dacpac</Description>
    <Copyright>Copyright 2024</Copyright>

    <!-- Support Visual Studio: https://github.com/microsoft/DacFx/issues/180#issuecomment-2010033767 -->
    <TargetFrameworkVersion Condition="'$(MSBuildRuntimeType)' != 'Core'">v4.8</TargetFrameworkVersion>
    <TargetFrameworkProfile Condition="'$(MSBuildRuntimeType)' != 'Core'" />
  </PropertyGroup>
  <ItemGroup>
    <Folder Include="build\" />
    <Folder Include="Tables" />
    <Folder Include="Stored Procedures\" />
  </ItemGroup>
  <ItemGroup>
    <None Include="build\My.Databases.Product.targets" />
    <None Include="Product.publish.xml" />
    <Content Include="build/My.Databases.Product.targets" PackagePath="build/My.Databases.Product.targets" />
  </ItemGroup>
  <Target Name="BeforeBuild">
    <Delete Files="$(BaseIntermediateOutputPath)\project.assets.json" />
  </Target>

  <!-- Support Visual Studio: https://github.com/microsoft/DacFx/issues/180#issuecomment-2010033767 -->
  <ItemGroup>
    <Build Include="Stored Procedures/**/*.sql" />
    <Build Include="Tables/**/*.sql" />
  </ItemGroup>
  <Target Name="PreBuild" BeforeTargets="PreBuildEvent" Condition="'$(MSBuildRuntimeType)' != 'Core'">
    <Message Importance="high" Text="Ensuring the $(MSBuildThisFileDirectory)\obj\project.assets.json file is removed, if necessary, so that the database project can be built through VisualStudio SSDT without errors" />
    <Delete Files="$(MSBuildThisFileDirectory)\obj\project.assets.json" />
  </Target>
</Project>

While this has been an interesting exercise, Visual Studio still lacks nuget package support so guess I'm stuck waiting for an official release :-(

Preview support for the Microsoft.Build.Sql SDK is landing in Visual Studio soon - will update ya'll as soon as the first release goes out the door!

With Visual Studio 17.12 preview 2 - support for Microsoft.Build.Sql SDK-style projects is now available in preview for Visual Studio.
2 key limitations to note:

in preview 2 the file extension for the project is .sqlprojx instead of .sqlproj
to use side-by-side with the original SQL projects in Visual Studio, install separate instances of VS on your machine

blog: https://techcommunity.microsoft.com/t5/azure-sql-blog/preview-release-of-sdk-style-sql-projects-in-visual-studio-2022/ba-p/4240616
docs: https://aka.ms/ssdt-sdk-preview

@dzsquared already?? 😉 - this is amazing!

@dzsquared already?? 😉 - this is amazing!

it seems like just yesterday 0.1.1-alpha was released..... but according to my notes - https://www.nuget.org/packages/Microsoft.Build.Sql/0.1.1-alpha - it was almost 3 years ago 😄


AWS EKS Networking Issue

Hi I have a managed EKS cluster (AWS-EKS) running in a VPC which have a restricted count on the IP. now it have around 150 IP's
my pods will be increased to 1000 in future.
as Amazon EKS supports native VPC networking via the Amazon VPC CNI plugin for Kubernetes. which allows Kubernetes pods to have the same IP address inside the pod as they do on the VPC network.
i will be running out of IP's soon. can any one help me on this note for an enterprised grade K8s
does EKS have any Docker kind of networking which gets the ip with <IP_ADDRESS>/16 kind of range which doesnt exhaust.
i dont want to go with Kops solution but anything on awseks is much appreciated.
Thanks
regards
saikiran (past student)

Hey saikiran
I think This vpc restrictions are over nodes and but not pods for pods we use service

Thanks Bhasker, i raised a request with AWS team They said that amazon vpc cni has an overlay network which takes IPs from the vpc and assign to the pods it cant be changed in any manner when used EKS service hence we are changing the VPC range. adding extra subnets to the cluster,
closing the issue


Handle respawn and mission intro

After connecting there should be the intro cinematic again (if it isn't disabled via mission parameters).
Also a small scene for the deployment.
The known respawn menu also moved to 0.97 but could be redesigned in the further development of the new framework.

Only a reupload of finished code with some smaller tweaks.
That's also the reason for assigning only 1 Story Point.


[SQL][Minor] typo

I couldn't compile the code without the change Files --> Utils, seems the change was not intention.

Test build #29267 has started for   PR 5220 at commit f468358.

This patch merges cleanly.

@chenghao-intel ,I think  #5198 had fixed the problem

Test build #29267 has finished for   PR 5220 at commit f468358.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

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

Yeap it is already fixed. But it is weird that this PR did not make conflict or something.

Oh, thanks @jeanlyn , that's weird, github doesn't show conflict.


submariner-operator pod always restarts

What happened:
submariner-operator pod is always restarted. I have observed this behavior in multiple AWS and KIND setups. This can also be observed in CI with so many of flaky tests failing with cat: /tmp/subctl-gather-output/cluster1_submariner-operator-56f895dd55-slddv.log.prev: No such file or directory make: *** [Makefile:94: e2e] Error 1 as captured in v0.10.0-rc0 test-day spreadsheet
What you expected to happen:
submariner-operator pod should not restart
How to reproduce it (as minimally and precisely as possible):
Set up a cluster in AWS/KIND and run subctl gather after some time.
Anything else we need to know?:
Environment:

Diagnose information (use subctl diagnose all):
subctl_diagnose_both_clusters.log

Gather information (use subctl gather):
clustera_gather.zip
clusterb_gather.zip

Cloud provider or hardware configuration: AWS/KIND

Install tools:

Others:

E0723 10:12:53.071013       1 main.go:205] cmd "msg"="Manager exited non-zero" "error"="no matches for kind \"Gateway\" in version \"submariner.io/v1\""

May be the Gateway CRD is still not ready? ^

Seems like a non severe issue, (not a blocker definitely) but something to look at. May be we should add retries to that, or have something that waits when the CRDs have been installed until they go ready?

Closing old issues. Please re-open if this is still relevant.


Unable to see dashboard password

For security reasons we are hiding the dashboard password on ls command, but we should have an info command on dashboard password node that can be used to see the password.

but we should have an info command on dashboard password node that can be used to see the password.

+1

Note that if the password was not explicitly set, then user will have to look into /var/log/ceph/cephadm.log on bootstrap minion to see which password was generated.
We can simplify this, by generating the random password in ceph-salt.

It's also possible to set a flag instructing the dashboard to force the user to change the password on first login. So, the default behavior could be: "if the user does not set the dashboard username/password explicitly, set it to admin/admin and turn on said flag".


aioredis.exceptions.ResponseError: syntax error

aioredis gives a syntax error when I set and use keepttl:
await redis_db.set(f"identity-{id}", orjson.dumps(rl), ex = 606024, keepttl = True)

The problem with your command is that you can't pass both EX (or PX) with the KEEPTTL option. This is because KEEPTTL keeps the existing TTL, while EX and PX change it, so the options conflict. You'll also get a syntax error in redis-cli:
<IP_ADDRESS>:6379> set a 2 ex 1 keepttl
(error) ERR syntax error

The exclusive nature of the commands is apparent in the autocomplete within redis-cli:
<IP_ADDRESS>:6379> set a value [EX seconds|PX milliseconds|KEEPTTL] [NX|XX]

However, the actual docs on the command don't seem to mention this! https://redis.io/commands/set


extend test setup with puppeteer + WebDriver, incl performance metric

in /examples/ui5-js-app
validate basic performance data, specifically for waitForUI5

see #288


[ALLUXIO-1821][ALLUXIO-1822]Fix a bug in which the metric of RemoteBlockXXXStream is incorrect

Also delete an error logic which is to increment metrics in RemoteBlockInStream because the increment logic is already exist in BufferedBlockInStream.

Can one of the admins verify this patch?

LGTM.

Thanks for the patch

ok to test

Merged build finished. Test PASSed.

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

+1 LGTM. Thanks for fixing the bug!


Implement Options... all of them

I'm collecting all of the options requests under one issue so they can be tackled once and for all! This is one of the last blockers for v0.7 release. Only options that can be reasonably implemented in v0.7 go in this list, anything too intensive is going to have to wait

[x] Enable/disable individual status messages (#89) (c8963130252a8cb90546423564eb6c22911e64ea)
[x] Enable/disable buttons that normally lead to the tech/civic chooser panel to instead lead to the tech/civic tree (#72)
[x] Minimap mode customization (there's two static modes toggled by rmb right now)
[x] Enable/disable toggle yields on save load (32506b803086908a74de9e3f0d386f56d9969781)
[ ] Set default City Details panel tab
[x] Enable/disable production panel small/normal buttons (dbac0ab3829507d0edcc541f51d57a3f60abca15)
[x] Enable/disable production queue
[x] Enable/disable builder lens auto-apply
[x] Enable/disable cultural growth overlay on save load
[x] Enable/disable unit action expando auto-open in unit panel (45d6d2384be13cd6b16ee82b23dcdc600fee66b5)
[ ] Enable/disable transparent citizen icons
[x] Enable/disable city yield info on hover
[x] Enable/disable resource icon dimming

Option to enable/disable tutorial UI (#51)

[ ] Improvements
[ ] Techs/Civics (tree and panel)
[ ] Production
[ ] City Details panel

Option to enable/disable individual smartbanner elements

[x] Unmanaged citizen icon
[x] District icons
[x] Turns to population growth
[x] Amenities
[x] Turns to cultural growth

You had me at "production queue".
I didn't look at the code, whether it's difficult or not, but maybe a toggle of transparent/normal citizen and resource icons would be nice.

I'm working on implementing a status message filter at the moment, seems like one of the more difficult ones to do right though

*A toggle of transparent/normal citizen and resources icons
This would be really nice. From my personal experience, transparent icon/text significantly increase visual clutter, and de-focuses the entire user interface.

Any chance we can get a shortcut key for the unit upgrade button?
Don't know what exactly you have in store for the Mini-Map customization, but i'd personaly love an option to resize it (or revert to vanilla dimensions).
The following issues occur when playing with Fortified Alert, MultiUnit, BuilderMod and Improved Water Yields enabled:

Not sure the Production Queue integration is working as intended, but in Lozenged's version you could queue up buildings after placing a district. In CQUI you have to finish building the district before you can queue any of the respective buildings, which kind of defeats the purpose of having a production queue...
This might've been reported before, apologies in advance, but Next City Plot integration is very buggy. Plots will stay highlighted even outside of city view, with multiple instances of already expanded tiles persisting.
If these are caused by mod conflict i'd ask if you could figure a way to make them compatible and i'd suggest considering integration of Fortified Alert at least (since the other 3 are probably not something you'd welcome in a UI mod).

@lifemare You can toggle the size to original by right clicking in the current build. More customization for the minimap size are definitely on the roadmap for closing this issue. Anyways, in respect to each of your issues: I'll look into that, this should be fixed in the latest nightly, let me know if it's not. I'll look into integrating that fortified alert mod, thanks for letting me know!

Hi, I'm working on "Option to enable/disable individual smartbanner elements". Got the Unassigned citizen and districts done. Just wanted to give a heads up so no else spends time on it too :)

@SpaceOgre Excellent! I'll get to reviewing your PR shortly 👍

Can you add an option to revert back to the original production menu (as opposed to the condensed Civ V one)?

There's an option to make the buttons bigger, if that's what you're asking
On Fri, Feb 3, 2017, 4:42 PM fabikw<EMAIL_ADDRESS>wrote:

Can you add an option to revert back to the original production menu (as
opposed to the condensed Civ V one)?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chaorace/cqui/issues/134#issuecomment-277370057, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGKcgMqTVQdjhJviqTE3v5klkCzGLJuzks5rY59QgaJpZM4K5Ciy
.

I've removed "Set default City Details panel tab" because I don't really think anyone cares about this and I feel like the two or three strings it adds to the LOCs creates additional and almost entirely pointless work for future translators


feat(font): add genjyuu-gothic

This PR adds a Japanese font, genjyuu-gothic (Japanese: 源柔ゴシック).

Thank you for your patience! Looks good to me 👨🍳


problem with update 0.2.14

All my NOUS devices is now unavaliable after update 0.2.14
1821eb6f-4246-4dda-b7a2-30c2f9aa22e9

Same problem with me: All nous smart plugs with metering throw the following error trying to switch the on/off: expected_cluster_id_number

Fixed


Examples and unit tests not working anymore

Hi,
After the last few updates it seems like the examples and unit tests are failing. I installed the latest version with "pip install sureal" and ran the unit tests and they failed. I also tried running the example script
sureal --dataset resource/dataset/lukas_pc_dataset.py --models THURSTONE_MLE BT_MLE \ --plot-dis-videos --output-dir ./output/lukas_pc_dataset
but got the error message
Error: Must have one and only one subclass of SubjectiveModel with type --dataset, but got 0. I have used sureal in the past without issues so it looks like a recent update has caused a failure.

@JoshuaEbenezer I forgot to update the Pypi package. That's why it didn't work since when you run pip install sureal it installed the older version. Now it's been updated to version 0.7.0. You can install by pip install -U sureal and try the cmd again. Let me know if it works.

Oh great! Thank you!

Thanks!
On Thu, Apr 21, 2022, 1:30 AM Zhi Li @.***> wrote:

Closed #43 https://github.com/Netflix/sureal/issues/43.
—
Reply to this email directly, view it on GitHub
https://github.com/Netflix/sureal/issues/43#event-6468292786, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADT5VDG7KV5MZKA7RSVMREDVGDYYPANCNFSM5T2A5ODQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Customers using m6in.2xlarge type instances are not able to get the IP address assigned.

What would you like to be enhanced:
Customers using m6in.2xlarge type instances are not able to get the IP address assigned. Since this instance type is released recently, can you please add the instance type to the allow list.
Why is the change needed and what use case will it solve:
Customers looking for high network performance will be looking to use m6in and r6in instance family and if these instances are  not added to the allow list, the pods that land on these instances will get stuck in pending status.

@77-more Thanks for submitting the issue. The instance type has been added already (#187) and will be released soon. Per discussion offline, sounds this issue happening for a Windows node. We need reassess why we need to check Windows node from the list.

We are planning an effort to support runtime lookup for Windows support. The enhancement will be tracked as #196

m6in family has been added and released to all new clusters. The upgrading existing clusters is in process now. I will update when the global upgrade finishes.

Since the version supporting this instance types has been deployed to all clusters, closing this issue.


Add C# 9 to "The history of C#" article

Add C# 9 to "The history of C#" article:
https://docs.microsoft.com/dotnet/csharp/whats-new/csharp-version-history

Should also have ms.custom: updateeachrelease metadata.


Syntaxerror in variable unpacking 487

Resolves #487
as mentioned in the chat I've tried to minimize the changes, however I chose to make a deoptimization in FinishAssignments so that it always allocates a list at the start. imo the improved readability and reduction in cyclomatic complexity outweigh the cost of the allocation. see what you think.
probably won't be able to make any updates for the next week and a half due to other commitments.

@hackf5 Any other plans for this PR? Otherwise I'm happy with it and will merge it in.

Thanks for the PR!

On Mon, Nov 25, 2019 at 1:06 AM slozier<EMAIL_ADDRESS>wrote:

Thanks for the PR!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/IronLanguages/ironpython3/pull/662?email_source=notifications&email_token=AJK7HWNDKIUANMEG5FYAWHTQVLJSFA5CNFSM4JFEGEOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFATBIA#issuecomment-557920416,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJK7HWILXHNVDHU7QEYF63DQVLJSFANCNFSM4JFEGEOA
.


Regrouped applications, filters, designs

Notes for Reviewers
This PR fixes #7170
https://user-images.githubusercontent.com/103871275/221399363-112a7637-6f84-4045-ada2-139bf981eb59.mp4
Signed commits

[x] Yes, I signed my commits.

Hey @lee. Please review the PR.

@leecalcote @acald-creator How would it be if we regroup other as well?? Like Settings has many components (MesherySettings, MesherySettingsEnvButtons etc). They all can be grouped.

@hs05june Hey thanks for your PR.
At this time, we're focusing on these three, as my next plan to update the code editor, and I wanted to group these so it is easier to debug code editor issues.

@hs05june Let's discuss this on the Meshery Dev call. Please add this as an agenda item in the meeting minutes if you would. :)

:eyes:

@hs05june it'd be great to see a redesign proposal in the Meshery UI figma design file. There are major improvements to be done in Meshery's UI. A mockup will help facilitate traction in this regard.

Instead, you can refractor the entire code which would be great.
Since Applications, Patterns & Filter nearly share the same code reusability would be at peak.
It would save the effort during the muiv5 migration as well.
// @Abhishek-kumar09 @acald-creator @leecalcote @hs05june @Chadha93

You can look at the previous effort of muiV5 to get the context of what I am speaking of.
https://github.com/meshery/meshery/tree/meshery-ui-restructuring


Update MD file on new branch 2017-11-09_07-25-53

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


use pymc_sphinx_theme instead of pydata directly

I tried to abstract most of the common elements between the 3 websites
to an independent theme repo

The preview seems to be working correctly so far, but I want to check the other repos before merging.


Fix navigation URL

Once https://capital-g.github.io/sftm/ is visited it will be changed to https://capital-g.github.io/.
Although the site works this way, one cannot copy, bookmark or share the URL with other because the mandatory /sftm/ will be omitted.
It most probably can be configured here and the build for gh-pages that currently "works" uses
https://github.com/capital-G/sftm/blob/93a9db17feed3570b0318f455b6b9b3533afcc6a/main.js#L23152-L23153
I tried out nearly every configuration and at least this works somehow.

Hey @capital-G can I look at this?

Hey @capital-G can I look at this?

Of course, any PR is welcome :)

Hey @capital-G can I look at this?

Of course, any PR is welcome :)

Great. Have you considered a move to elm 19?


log time to play

Notes
This ended up being a little more complex after realizing we need to reset analytics data when users navigate directly to another file (where the File page won't be re-mounted)
There are three scenarios:

User has already downloaded a file: do nothing
User started downloading but navigated away before it started playing: log file_view with no time_to_start
User stayed on the page until the file started playing: log file_view with time_to_start

Currently outpoint is still not included in the claim information (https://github.com/lbryio/lbry/issues/1306). Previously, we were calling Lbry.get which returns the outpoint and we used that for this api call.
We can't read it from fileInfo because that will not exist if we try to fire the analytics event in response to a user navigating away before the file is downloaded.
I think the simplest approach would be to wait for https://github.com/lbryio/lbry/issues/1306 and grab outpoint along with the rest of the data from the claim information.
@skhameneh @kauffj Thoughts?

Three comments:

I think this overlaps some with @daovist work on media history. Please discuss on stand-up if it makes sense to combine this work or for media history to contain this value.

View tracking may currently supports parameters other than outpoints.

Please don't hesitate to point this issue out to a daemon team member and/or comment on that ticket itself. I suspect this is a fairly small one to fix.

A claim's outpoint is just: ${claim.txid}:${claim.nout}. I don't understand why #1306 is an issue.

@tzarebczan Just updated this to only fire if a user leaves the page after it has started downloading. Before it was firing if they clicked play and left even if it wasn't downloading.

@skhameneh Did you have any concerns before merging?

@seanyesmunt

Before it was firing if they clicked play and left even if it wasn't downloading.

My concern is that we want this information!

@kauffj Tom had a comment
my only concern is that it's now even easier to get a "view" - the file doesn't even have to start playing. Maybe we need to only take into account files with time to play for rewards purposes?
If we change the behavior by adding a view event, we may need to change rewards logic.

@seanyesmunt this is why I had mentioned tagging different events explicitly, isn’t the immediate approach to move forward and change how we track analytics to be more explicit?
We need an approach where we capture a lot of raw data and do transforms to SQL for more specific tables that correlate to data captured.

@seanyesmunt the rewards logic would ideally be a separate issue

We can worry about the rewards logic on the internal-api side. I think this is ready to be merged in right?


[Feature] Font Family for labels and values

could we have a FontFamily property for Entry's labels or values?

There should already be a font family property.


optimization: Accelerates the performance of updating the xDS configuration

What type of PR is this?
Add one of the following kinds:
/kind enhancement
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes https://github.com/kmesh-net/kmesh/issues/324
benchmark test:
before modify:
[root@localhost v2]# go test -bench=. --benchtime=100x
goos: linux
goarch: amd64
pkg: kmesh.net/kmesh/pkg/cache/v2
cpu: Intel(R) Xeon(R) CPU E5-2690 v3 @ 2.60GHz
BenchmarkClusterFlush-16<PHONE_NUMBER>71 ns/op
BenchmarkFlush-16<PHONE_NUMBER>212 ns/op
PASS
ok      kmesh.net/kmesh/pkg/cache/v2    245.370s
[root@localhost v2]#

after modify:
[root@localhost v2]# go test -bench=. --benchtime=100x
goos: linux
goarch: amd64
pkg: kmesh.net/kmesh/pkg/cache/v2
cpu: Intel(R) Xeon(R) CPU E5-2690 v3 @ 2.60GHz
BenchmarkClusterFlush-16             100            600145 ns/op
BenchmarkFlush-16                    100            283194 ns/op
PASS
ok      kmesh.net/kmesh/pkg/cache/v2    4.047s

Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Why BenchmarkFlush(actually listener) is quicker than cluster flush

Why BenchmarkFlush(actually listener) is quicker than cluster flush

Why BenchmarkFlush(actually listener) is quicker than cluster flush

from the prespective of the testcase design:
BenchmarkClusterFlush
Cluster records are continuously added to the map. (the keys of cluster map(cluster name) are different.)
// cluster map define
struct {
    __uint(type, BPF_MAP_TYPE_HASH);
    __uint(key_size, CLUSTER_NAME_MAX_LEN);
    __uint(value_size, sizeof(Cluster__Cluster));
    __uint(map_flags, BPF_F_NO_PREALLOC);
    __uint(max_entries, MAP_SIZE_OF_CLUSTER);
} map_of_cluster SEC(".maps");

BenchmarkFlush:
Only one listener record is updated. (The listener.Name is randomly set, but the key of the listener map is addr.)
// listener map define
struct {
    __uint(type, BPF_MAP_TYPE_HASH);
    __uint(key_size, sizeof(address_t));
    __uint(value_size, sizeof(Listener__Listener));
    __uint(max_entries, MAP_SIZE_OF_LISTENER);
    __uint(map_flags, BPF_F_NO_PREALLOC);
} map_of_listener SEC(".maps");

Good analyze, So we should update the listener  benchmark

Good analyze, So we should update the listener benchmark

may not be required. This case is used to test the scenario where the same record is updated. In deserial_update_elem, the instance tree corresponding to the record will be deleted first.

So we need to test different scenaios. Add/update/delete


Issue 3324: WDA Texts are not taken into account

The WDA texts of the text based entities like labels, buttons, TextView,... have not been included in the ZIP-archive. They are stored as SOTR object and have been added to the github artifacts.

@larshp Would it be possible to create the repo https://github.com/abapGit-tests/SOTR to move the WDYN tests there?

@jfilak I've invited you to the abapGit-tests organization, you should then have access to create repositories there

@larshp  Thanks! I've created the repo https://github.com/abapGit-tests/WDYN because the SOTR class is not an object type handler but a helper class.
@KDS42 We need to move the src/test/zwd_abapgit_test_sotr.wdyn.xml to the new repo. Basically, fork it, create a package $wdyn, clone your fork to your system, move the object to the new package, commit and open a new PR.

@jfilak 👍

@Community: can someone check the scenario? E.G. Export your local WD component (including WD application) OFFLINE from system A.  Afterwards importing ARCHIVE into system B, where the component does not exist - are Labels, Texts correctly displayed at Runtime?

Hi,
after pull of the example repo I get the following diff, need to sort the elements,

Also no SOTR after pull,

I tested again - it worked - please see attachment for the steps:
WDA_SOTR_GITHUB.docx

I think the sorting issue was existing before this PR, so it can be moved to a separate issue

anyhow, lets just go for it 🚢

and thanks 👍 welcome to abapGit as a contributor


Histogram

Codecov Report

Merging #19 (cdb2c00) into dev (60f4a79) will increase coverage by 0.63%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev      #19      +/-   ##
==========================================
+ Coverage   62.62%   63.26%   +0.63%
==========================================
  Files          58       60       +2
  Lines        8500     8648     +148
  Branches     1664     1669       +5
==========================================
+ Hits         5323     5471     +148
  Misses       2728     2728
  Partials      449      449

Flag
Coverage Δ

unittests
63.26% <100.00%> (+0.63%)
:arrow_up:

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

Impacted Files
Coverage Δ

tf_pwa/histogram.py
100.00% <100.00%> (ø)

tf_pwa/tests/test_histogram.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #19 (cdb2c00) into dev (60f4a79) will increase coverage by 0.63%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev      #19      +/-   ##
==========================================
+ Coverage   62.62%   63.26%   +0.63%
==========================================
  Files          58       60       +2
  Lines        8500     8648     +148
  Branches     1664     1669       +5
==========================================
+ Hits         5323     5471     +148
  Misses       2728     2728
  Partials      449      449

Flag
Coverage Δ

unittests
63.26% <100.00%> (+0.63%)
:arrow_up:

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

Impacted Files
Coverage Δ

tf_pwa/histogram.py
100.00% <100.00%> (ø)

tf_pwa/tests/test_histogram.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


update third-party library versions where possible for 2.7

since we are updating to latest vtk for 2.7, thought it might be a good version to update other third party libraries when possible.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1402
Status: Pending
Project: VisIt
Tracker: Feature
Priority: Normal
Subject: update third-party library versions where possible for 2.7
Assigned to: -
Category: -
Target version: -
Author: Kathleen Biagas
Start: 03/28/2013
Due date:
% Done: 0%
Estimated time:
Created: 03/28/2013 02:07 pm
Updated: 04/15/2013 01:43 pm
Likelihood:
Severity:
Found in version: 2.12.3
Impact: 3 - Medium
Expected Use: 3 - Occasional
OS: All
Support Group: Any
Description:
since we are updating to latest vtk for 2.7, thought it might be a good version to update other third party libraries when possible.
Comments:

This is outdated.


Crash in shader compiler on PowerVR devices

Describe the bug
We use the latest version of filament in main branch. And captured some crashes .
The crash log is below:
#00 pc<PHONE_NUMBER>0983c0 /vendor/lib64/libusc.so [arm64-v8a::87dc1f0dd442b4d23860f485a8121ce6]
2
#01 pc<PHONE_NUMBER>0aa678 /vendor/lib64/libusc.so [arm64-v8a::87dc1f0dd442b4d23860f485a8121ce6]
3
#02 pc<PHONE_NUMBER>09fe4c /vendor/lib64/libusc.so [arm64-v8a::87dc1f0dd442b4d23860f485a8121ce6]
4
#03 pc<PHONE_NUMBER>09e2f0 /vendor/lib64/libusc.so [arm64-v8a::87dc1f0dd442b4d23860f485a8121ce6]
5
#04 pc<PHONE_NUMBER>00aa74 /vendor/lib64/libusc.so (PVRUniFlexCompileToHw+4552) [arm64-v8a::87dc1f0dd442b4d23860f485a8121ce6]
6
#05 pc<PHONE_NUMBER>08b9e0 /vendor/lib64/libglslcompiler.so [arm64-v8a::aa9040ec2ed7811792f6329486148a0c]
7
#06 pc<PHONE_NUMBER>09f4c8 /vendor/lib64/libglslcompiler.so (GLSLCompileToUniflex+1464) [arm64-v8a::aa9040ec2ed7811792f6329486148a0c]
8
#07 pc<PHONE_NUMBER>069590 /vendor/lib64/egl/libGLESv2_mtk.so [arm64-v8a::c575894581727aae76bca9337c18e43a]
9
#08 pc<PHONE_NUMBER>069150 /vendor/lib64/egl/libGLESv2_mtk.so [arm64-v8a::c575894581727aae76bca9337c18e43a]
10
#09 pc<PHONE_NUMBER>07ee80 /vendor/lib64/egl/libGLESv2_mtk.so (glCompileShader+124) [arm64-v8a::c575894581727aae76bca9337c18e43a]
11
#10 pc<PHONE_NUMBER>011630 /vendor/lib64/egl/libGLES_meow.so [arm64-v8a::56e1d7e99ff3d2f3846472d7c380c1b6]
12
#11 pc<PHONE_NUMBER>10b114 /lib/arm64/libfilament-jni.so [arm64-v8a::7389f55482942e5e0a5ca0802fe1be4b]
Desktop (please complete the following information):

OS: Android
GPU: PowerVR Rogue GE8320,vendor:Imagination Technologies
Backend: OpenGL ES 3.2 build 1.11@5425693

Smartphone (please complete the following information):

Device: Oppo/COLOROS
OS: Android 10

FYI
We also saw a lot of similar crashes reported by those device with gpu:PowerVR Rogue GE8320,vendor:Imagination Technologies

Yes, unfortunately, Power VR compilers are very buggy.

This is unfortunate. Makes the whole engine not reliable for Android.

Have you tried PVRShaderEditor, is it of any help? Additionally, it might be worth asking Imagination Technologies on their forums, their engineers do answer.
PowerVR has a decent market share on Android, it would be great to improve the compatibility. I tried to switch to Vulkan for those devices but no luck...

Have you tried PVRShaderEditor, is it of any help? Additionally, it might be worth asking Imagination Technologies on their forums, their engineers do answer.
PowerVR has a decent market share on Android, it would be great to improve the compatibility. I tried to switch to Vulkan for those devices but no luck...

Hi Wes. I'm trying to find a solution for this. So you tried to use Vulkan backend and it's still crashing?

@avladgh The samples do work on PowerVR GPU when using Vulkan as a backend.
If you have custom materials it gets tricky to get similar behaviour between OpenGL & Vulkan across devices.

@wes-nz Thx. That's what I want to do, use vulkan backend for these gpus. Luckily that my models are not very complex. However, there are some devices with PowerVR 8320 that doesn't support vulkan either.
Also, I noticed Google maps doesn't support 3d vehicles/models on these devices too, even for devices with PowerVR and vulkan support. Looks like a bigger issue.

Thanks for the update @avladgh, that's good to know.

Dropping in here to say that we see this crash also on the PowerVR GM9446 GPU. The overall breakdown of devices where we this looks like:

I do not have one of these devices to be able to reproduce the crash, but will get one. For now we are going to exclude these SOCs from installing out app, but if we could get a fix in that uses Vulkan backend on these devices that would be great since Oppo devices are pretty popular in latin america.
I would also add that we get a couple other flavors of the crash in e.g. the GLSLCompmileToUniflex, but the associated chipsets that the crash happens on look the same

Is there a way to predict errors in advance? so we can disable the functionality to prevent crashes.
@pixelflinger

The following code causes the crash:
#version 300 es

layout(std140) uniform FrameUniforms {
    vec2 i;
} frameUniforms;

layout(location = 0) in vec4 mesh_position;

void main() {
    gl_Position = mesh_position;
    gl_Position.z = dot(gl_Position.zw, frameUniforms.i);
}

The PowerVR compiler doesn't like when gl_Position is written multiple times. Using a temporary fixes the issue. However, spirv-opt re-creates the problem. Removing the use of dot prevents that.

Thank you for fixing this!


Adding interactive segmentation example

Hi KonstantinKorotaev,
What is the reason for this closed pull request, I am exactly looking for this example to integrate as ML backend. If I would use this to perform interactive segmentation in Label Studio will this work and was this already tested by you? Thanks in advance.

Hi @tommyvtran97
Could you please describe your case in more details?

I was planning to use this repository for interactive segmentation https://github.com/saic-vul/ritm_interactive_segmentation. I think this is the same repository that you used in your pull request. My questions was whether the commit change that you proposed is actually working in Label Studio although it has not been merged.
My goal is to use this interactive segmentation model in the ML backend for performing segmentation labelling, since this is much faster compared to the polygon method


How to add newlines in the message in GhidraScript.askChoice?

I'm trying to show a dialogue asking the user which of two options should be used when importing functions from another file. For legibility, it would be important to show the different options on their own lines. However, just adding newlines to the dialog's message doesn't seem to work.
I'm assuming some code somewhere is filtering out any control characters, but is there some other alternative I can use to create dialogs with multiple lines?
I am running Ghidra version 11.0 with Java version 17.0.9 under Windows 10 22H2.
<A, B> Boolean askBromaConflict(String in, String what, A broma, B ghidra) throws Exception {
    switch (askChoice(
        "Conflict between Broma and Ghidra",
        MessageFormat.format(
            "Conflict between {1}s in {0}: \n" +
            "Broma: {2} \n" +
            "Ghidra: {3} \n" +
            "Should Broma's {1} be used or keep Ghidra's {1}?",
            in, what, broma, ghidra
        ),
        List.of("Use Broma", "Keep Ghidra", "Cancel Script"),
        null
    )) {
        case "Use Broma": return true;
        case "Keep Ghidra": return false;
        case "Cancel Script": throw new Error("Script cancelled");
    }
    return true;
}

It seems that the built-in ask methods do not support newlines.  They make use of a GLabel internally, which extends JLabel.  JLabel's do not support newlines.
The various ask methods will not support this.  However, you can get the effect you desire with this snippet.  The only caveat is if the user cancels the dialog, it will not cancel the script, as do the ask methods.   So, if you need that behavior, you will have to check for null after the dialog returns.

import docking.widgets.dialogs.InputWithChoicesDialog;
import ghidra.app.script.GhidraScript;
import ghidra.util.Swing;

...

	@Override
	public void run() throws Exception {

		String choice = Swing.runNow(() -> {
			InputWithChoicesDialog dialog =
				new InputWithChoicesDialog("Title", "<html>This is<br>my message",
					new String[] { "One", "Two" }, "Two", null);
			state.getTool().showDialog(dialog);
			return dialog.getValue();
		});

		println("Choice: " + choice);

		if (choice == null) {
			// the user cancelled the dialog
			return;
		}

	}

Hello, that works perfectly! Thank you for the response!


Ability to pass command using an URL; small changes

This pull-requests contains three small changes:

Run a StaSh command when clicking an URL (using the pythonista URL-scheme): launch_stash.py now accepts a command as optional argument, which will then be run instead of the totd-command when StaSh is started. This allows the use of StaSh as an utility for easily sharing code, support other users or run installs. I have created a small script which helps to encode a StaSh command into a URL. Also, the StaSh-class now accepts command as an optional argument. If command is None, the default command (totd)will be run on startup. If command is False, no command will be run on startup (This does not show the prompt when StaSh is started because script_will_end will not be called, but is required if you want to use _stash(cmd) after initialization.). If command is a string, it will be interpreted as a command. Example: pythonista3://launch_stash.py?action=run&args=%22selfupdate%20-f%20bennr01%3Arestructure%22 installs pytest from pypi. I am a bit worried someone might use this in combination with the rm -r $HOME2 command and disguises the link, but hopefully no one does this.
optimized ls.py: like discussed in #213, ls.py now checks wether the path it should list is a mounted path. If it is, it will only check the file-extensions to determine the filetype, otherwise it behaves normally.
moved mount_ctrl.py: I think it makes sense to move this file from $STASH_ROOT/lib/mlpatches/ (The path where the monkeypatches are saved) to $STASH_ROOT/lib/stashutils/(The path where the mounting system except the monkeypatches are saved).

@ywangd Sorry for being a bit impatient, but i have two questions:

Should i change some commits or is there any reason this pull-request has not yet been merged? If there is anything wrong with it, just say it and i will change it (I think the most likely reason is that you are/were busy and just did not see this pull-request, which is why i am writing this message).
About the reorganization of the code you mentioned in #213: Are there any new plans? I would like to add a directory for extensions in the documents-folder, which could then be used for command installed by scripts using pip, easily adding custom monkeypatches and for adding FSIs (the abstract interface the mc, mount and umount command use). Are you ok with this?
Thanks

Somehow I thought I have merged this pull request. But apparently I didn't. Sorry for the unnecessary delayed action.
I haven't got any new plans for the code yet. Honestly I haven't been working on it for a while now due to lack of time. If you are still interested in improving it, I could add you as a collaborator of this project so that you have full control of this repository. Is this something you would be interested in? @jsbain Are you OK with being a collaborator as well?

@ywangd dont worry about it. Everyone forgets something from time to time.
Thanks for inviting me to become a collaborator. I am interested in becomming a collaborator, but i do not think it would be necessary. As long as you accept (or deny) my pull-requests every now and then. But thanks for the offer and the answer.


Add ability to lint individual build tasks

Adding the option for individual tasks to be linted during the build. Each task can provide its own tslint.json (using the tslint.json file in the root of the repository as a start, if desired). The existence of a tslint.json and a tsconfig.json file indicates that linting should be run.
The previous discussion talked about having a single tslint.json be applied to all tasks in the repo and creating a plan for all teams to get clean. This is a step in that direction which doesn't require teams to get clean all at once (or at all) but still allows teams that do want to get clean to achieve it.

Updating gradle task to pass linting
Removing tslint from:

NuGet - this task was recently marked deprecated commit.
NuGetCommand - If linting is desired (it currently produces many errors), the file can be re-added.
NuGetPublisher - this task was recently marked deprecated commit.

Hey @ericsciple.  Can you have a look at this change since it affects the build tools (make.js, make-util.js, package.json)?  I ran a complete build of the repo successfully and it'd be great if we could allow linting for individual tasks. (I have some refactoring of the Jenkins tasks to start soon and I'd love to have this ability built-in.)  Note that folks will have to run npm install to get the tslint dependency.  (Notes about all of the changes are in my initial PR notes, above.)

i'll defer to @bryanmacfarlane

closing as this is being tracked by a new re-engineering effort


OMA URI Setting

[Enter feedback here]
./Vendor/MSFT/Policy/Config/Authentication/AllowAadPasswordReset needs to be replaced with ./Device/Vendor/MSFT/Policy/Config/Authentication/AllowAadPasswordReset, The policy is applied only after apending with ./device.

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

ID: c2015c6c-60c7-bb74-469c-e9320301d08a
Version Independent ID: 0ff19de6-c892-10ff-5fe8-f6a70abfa3f5
Content: Self-service password reset for Windows devices - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-sspr-windows.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

@vazhailravi
Thanks for your feedback! We will investigate and update as appropriate.

@vazhailravi
Thank you for pointing this out! I've created a PR to update the documentation and it's currently waiting for the author to review and sign off.
I will now close out this issue. Please allow some time for the author to review and the changes to reflect.
Thank you for your time and patience!

Thanks a lot.
On Wed, 9 Feb 2022 at 11:35 PM, James Tran-MSFT @.***>
wrote:

@vazhailravi https://github.com/vazhailravi
Thank you for pointing this out! I've created a PR to update the
documentation and it's currently waiting for the author to review and sign
off.
I will now close out this issue. Please allow some time for the author to
review and the changes to reflect.
Thank you for your time and patience!
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/87720#issuecomment-1034047312,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXVZ7H5FPMMO563TEDMEWR3U2KUGBANCNFSM5N5C5LPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Regards
Ravi


Update Application Autoscaling tutorial instance type

Issue #, if available:
Sagemaker endpoint no longer supports "t2" type instances for autoscaling, an instance of that type is currently being used in this tutorial. This PR replaces it with a ml.m5.large.
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@RedbackThomson


fix: compatibility with models

The PR makes the square-model-client compatible with the models service in square core. There, several changes have been made unilaterally.

LGTM


Compiler UX Guidelines RFC

This RFC proposes conventions for the user experience of the rust-lang/rust compiler.

Pre-RFC
Reddit Discussion
Rendered

@tshepang I did both changes. The latter does follow though. If developers are spending less time trying to learn their tools due to accidental complexity in the tools, that's less time they can spend working on their customer's actual needs. Of course, they could spend the time somewhere else (e.g. drinking a beer), but that's why the word was "can". But it's fluff nonetheless, so was for the best to be removed anyways.

Why is there no shepherd assigned to this RFC?

@llogiq because the compiler team has only recently started having having regular triage meetings. we're assigning one now. :)

Hey @pnkfelix ;
@llogiq - I added the note about the orthogonality of flags.

Is there any reason to not go into "final comment period"?

A shepard was only even assigned a day ago, so I would think it's a bit early.

cc @wycats

So I gave this a read. I agree that it seems to reflect current practice, which was the goal. I have some thoughts about things we might do to improve, but this is presumably not the place to argue them. Anyway, TL;DR, seems good to me -- that said, I do think it's worth considering just where these guidelines ought to live long term. An RFC doesn't quite seem right. A ux-guidelines.md file in the source repo is perhaps better. (What directory?)

Fixed the missing closing quote;
I originally proposed adding a file directly to the rustc, but that issue was closed in favor of having an RFC to bikeshed and come to a consensus that they were good guidelines.
As for which directory - I'd say just shove it into /docs/ux-guidelines.md or possibly create a /docs/guidelines directory and shove it in as ux.md but also put in other similar RFCs guidelines such as the backwards compatibility RFC. Just tell me where to put it, and I'll put it there.
But I definitely want to be the person who puts it there -- I (selfishly) want my name on the commit so I can point to others exactly how I've contributed to the Rust project.

@Havvy hi
The compiler team discussed this recently.  I think we all agree that the document you've written here codifies our current practice, and that we just need to resolve the bikeshed of where to put it.
Beyond that, here are some personal opinions:
I personally have no problem with putting the document into src/docs/.
The file itself should probably be named rustc-ux-guidelines.md: that is, I would put rustc into the name, since there are libraries elsewhere in the Rust distribution that are not part of the compiler and the several of the guidelines discussed here don't make sense outside the context of the compiler.

That, or the file should have one section for the few guidelines given here that do apply to the project as a whole, and then a separate section for the bulk of the material specific to the rustc user experience.

Anyway, having said all that, I suspect that if no one else chimes in soon (within a day or two) then a great next step would be to just open up a pull request that creates the aforementioned file in src/docs/.  (And in the PR description, just as you did for this pull request, link to this RFC.)

@pnkfelix This can be merged or closed, yes?

@Havvy yah thanks


Add IngressClass for upcoming 2.3 Release

Lock behind a kube version check of 1.18.x

Completed from above


⚠ Avatar API Endpoint has degraded performance

In d183eb8, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 4576 ms

Resolved: Avatar API Endpoint performance has improved in 6d42541 after 3 minutes.


chore(main): release 0.33.0

:robot: I have created a release beep boop
0.33.0 (2024-12-07)
Features

Updates supported resource types (7) (#825) (ab4eeec)

This PR was generated with Release Please. See documentation.

Summary by CodeRabbit

New Features

Version 0.33.0 released, reflecting updates to supported resource types.

Documentation

CHANGELOG.md updated to include details about the new version and changes made.

Chores

Version number updated in the .release-please-manifest.json and Cargo.toml files.

:robot: Release is at https://github.com/udondan/cfn-teleport/releases/tag/v0.33.0 :sunflower:


npm installation error

Hi, folks, great work.
I have issue running npm install:
node-pre-gyp ERR! Tried to download(403): https://storage.googleapis.com/grpc-precompiled-binaries/node/grpc/v1.9.1/node-v64-linux-arm-glibc.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v64 ABI, glibc) (falling back to source compile with node-gyp)
node-pre-gyp ERR! Tried to download(403): https://storage.googleapis.com/grpc-precompiled-binaries/node/grpc/v1.10.0/node-v64-linux-arm-glibc.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v64 ABI, glibc) (falling back to source compile with node-gyp)
I have nodejs version 10.1.0 installed.
Thanks.

Same error here..... how can we setup node to run ODAS? Thank you

OK, working with node 8... Just downgrade and it runs. Thank you!


Fix RecyclerVisited related crashes during marking

(courtesy of Bo Cupp)
Fixing two issues in marking.

If there are only precisely traced objects allocated, the markStack used for non-precisely traced objects will be empty and crash when Pop is called.  An IsEmpty check was added to prevent popping from an empty stack.
A race between allocating and background marking can lead to calling a virtual Trace method on an object that has a null v-table pointer.  It isn't possible to call trace until after a flag has been set by the allocator, so prior to that happening a dummy v-table pointer is now installed until control returns to the caller of the recycler's alloc method where a final v-table pointer will overwrite the dummy one.

Hey @dlibby-, would you mind rebasing this PR on top of https://github.com/Microsoft/ChakraCore/commit/450e9bd79746f90bf3fbfaa49b872c8ff4b82e5e (HEAD of master, at the time of writing) or later? That will fix the ubuntu CI issue. Thanks!

Abandoning in favor of a PR against release/1.8


[Bug]: feedback form is not there

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
issue : after shopping there must be feedback form for the user to know the overall experience of the user after using this website or application
proposition : can add the button of feedback form to get the overall experience of the user.
Add Screenshots / Video Clip
No response
What browsers are you seeing the problem on?
Chrome
Agreements

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC contributor
[X] I'm a Hacktoberfest contributor
[X] I want to work on this issue [Optional]
[X] I'm willing to provide further clarification or assistance if needed.

@SandhyaBadwaik
feedback form already exists in project.


etcd restore fails

What kind of request is this (question/bug/enhancement/feature request):
Failed to restore the backup
Steps to reproduce (least amount of steps as possible):

Backup the etcd using rke cli
copy the snapshot to new target vm /opt/rke/etcd-snapshot/
Restore using rke cli
rke etcd snapshot-restore --name snapshot.db --config ~/tmpw/cluster.yml

Already there is some similar issue but it says it fixed
https://github.com/rancher/rancher/issues/18504
Result:
jana@jvenkata:~/PycharmProjects/k8s$ rke etcd snapshot-restore --name snapshot.db --config ~/tmpw/cluster.yml
INFO[0000] Restoring etcd snapshot snapshot.db
INFO[0000] Successfully Deployed state file at [/home/jana/tmpw/cluster.rkestate]
INFO[0000] [dialer] Setup tunnel for host [*****]
WARN[0006] failed to stop etcd container on host [*****] : Can't stop Docker container [etcd] for host [*****] : Error response from daemon: No such container: etcd
INFO[0006] [etcd] starting backup server on host [*****]
INFO[0006] [etcd] Successfully started [etcd-Serve-backup] container on host [*****]
WARN[0011] failed to start backup server on host [*****] : Failed to run backup server container, container logs: time="2019-04-30T14:45:00Z" level=fatal msg="open /etc/kubernetes/ssl/kube-ca.pem: no such file or directory"

WARN[0012] failed to start etcd container on host [*****] : Failed to start [etcd] container on host [*****] : Error response from daemon: No such container: etcd
FATA[0012] failed to start backup server on all etcd nodes: [Failed to run backup server container, container logs: time="2019-04-30T14:45:00Z" level=fatal msg="open /etc/kubernetes/ssl/kube-ca.pem: no such file or directory"
]

Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
v2.2.2
rke version v0.2.2

Installation option (single install/HA):
Installation in HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
custom
Machine type (cloud/VM/metal) and specifications (CPU/memory):
vm
Kubernetes version (use kubectl version):

v1.11.6

Docker version (use docker version):

Docker version 18.09.5, build e8ff056

@superseb was this issue fixed by one of your recent prs?

Is this only using the RKE CLI? Can you supply a bit more detail on the setup:

RKE CLI or Rancher
What OS is running on the nodes
The complete cluster.yml
Any upgrade path or versions where it worked before

Is this only using the RKE CLI? Can you supply a bit more detail on the setup:

RKE CLI or Rancher
What OS is running on the nodes
The complete cluster.yml
Any upgrade path or versions where it worked before

rke cli
ubuntu

address: *****
internal_address: <IP_ADDRESS>
user: root
role:
- controlplane
- etcd
- worker
ssh_key_path: ~/.ssh/rke92.pub```

Same error here when trying to restore on centos 7, using RKE CLI after having the cluster die and needing to recover. This is a first attempt at doing an etcd restore. Rancher version 2.2.2 RKE version 0.2.2.

Please supply the exact commands executed and the content of the directory (ls -la) where you run the rke command.

It turns out that name convention for file should be same. Restore is working fine

Still facing the same issue with whilst trying to restore from RKE v1.0.0

Restored the snapshot on the running server and downloaded it locally
scp '-ed' it to the target restore server in the folder /opt/rke/etcd-snapshots
Ran the rke restore command,
rke_v1.0.0 etcd snapshot-restore --name /opt/rke/etcd-snapshots/16.12.2019_backup --config rancher-cluster-restore.yaml
The following output is observed:

INFO[0000] Running RKE version: v1.0.0
INFO[0000] Restoring etcd snapshot /opt/rke/etcd-snapshots/16.12.2019_backup
INFO[0000] Successfully Deployed state file at [./rancher-cluster-restore.rkestate]
INFO[0000] [dialer] Setup tunnel for host [xx.xx.xx.xx]
INFO[0001] Checking if container [cert-deployer] is running on host [xx.xx.xx.xx], try #1
INFO[0001] Image [rancher/rke-tools:v0.1.51] exists on host [xx.xx.xx.xx]
INFO[0001] Starting container [cert-deployer] on host [xx.xx.xx.xx], try #1
INFO[0001] Checking if container [cert-deployer] is running on host [xx.xx.xx.xx], try #1
INFO[0006] Checking if container [cert-deployer] is running on host [xx.xx.xx.xx], try #1
INFO[0006] Removing container [cert-deployer] on host [xx.xx.xx.xx], try #1
INFO[0006] Stopping container [etcd] on host [xx.xx.xx.xx] with stopTimeoutDuration [5s], try #1
WARN[0006] Can't stop Docker container [etcd] for host [xx.xx.xx.xx]: Error response from daemon: No such container: etcd
INFO[0006] Stopping container [etcd] on host [xx.xx.xx.xx] with stopTimeoutDuration [5s], try #2
WARN[0006] Can't stop Docker container [etcd] for host [xx.xx.xx.xx]: Error response from daemon: No such container: etcd
INFO[0006] Stopping container [etcd] on host [xx.xx.xx.xx] with stopTimeoutDuration [5s], try #3
WARN[0006] Can't stop Docker container [etcd] for host [xx.xx.xx.xx]: Error response from daemon: No such container: etcd
INFO[0006] [etcd] starting backup server on host [xx.xx.xx.xx]
INFO[0006] Image [rancher/rke-tools:v0.1.51] exists on host [xx.xx.xx.xx]
INFO[0007] Starting container [etcd-Serve-backup] on host [xx.xx.xx.xx], try #1
INFO[0007] [etcd] Successfully started [etcd-Serve-backup] container on host [xx.xx.xx.xx]
INFO[0013] Removing container [etcd-Serve-backup] on host [xx.xx.xx.xx], try #1
WARN[0013] failed to start backup server on host [xx.xx.xx.xx]: Failed to run backup server container, container logs: time="2019-12-16T15:50:44Z" level=fatal msg="open /etc/kubernetes/ssl/kube-ca.pem: no such file or directory"
INFO[0013] Starting container [etcd] on host [xx.xx.xx.xx], try #1
WARN[0013] Can't start Docker container [etcd] on host [xx.xx.xx.xx]: Error response from daemon: No such container: etcd
INFO[0013] Starting container [etcd] on host [xx.xx.xx.xx], try #2
WARN[0013] Can't start Docker container [etcd] on host [xx.xx.xx.xx]: Error response from daemon: No such container: etcd
INFO[0013] Starting container [etcd] on host [xx.xx.xx.xx], try #3
WARN[0013] Can't start Docker container [etcd] on host [xx.xx.xx.xx]: Error response from daemon: No such container: etcd
WARN[0013] failed to start etcd container on host [xx.xx.xx.xx]: Error response from daemon: No such container: etcd
FATA[0013] failed to start backup server on all etcd nodes: [Failed to run backup server container, container logs: time="2019-12-16T15:50:44Z" level=fatal msg="open /etc/kubernetes/ssl/kube-ca.pem: no such file or directory"


[feature template] enable static ip and interface name

Example used:
encapsulation = [
  Encapsulation(EncapType.GRE),
  Encapsulation(EncapType.IPSEC, preference=1, weight=1),
]

template_api = TemplatesAPI(session)
tunnel_1 = Tunnel(
    color = ColorType.G3,
    encapsulation = encapsulation
)

ipv4_eth3 = TypeAddress("static")
vpn_0_eth1 = CiscoVpnInterfaceEthernetModel(
    name="vm81_vpn_0_gigabitethernet2235",
    description="vm81_vpn_0_gigabitethernet3",
    shutdown = False,
    tunnel = tunnel_1,
    interface_name = InterfaceName(InterfaceType.GIGABIT_ETHERNET, "2"),
    ip="<IP_ADDRESS>/24",
    type_address = ipv4_eth3,
    mtu = 1500,
    autonegotiate = True
)

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/145


[language] Native Functions Refactor

This is mostly removing a verification step for native functions that
was not adding anything in terms of safety but may have had some value
in making developers think harder about native functions.
It does not feel the effort, code, maintenence is giving us value.
We should implement at some point signature checks in the proper places
so we have better protection than ever, but for now this is not less
secure than what we have.
The simplification also makes it easier to split the code so that
the declaration of what native functions are accepted now lives inside
the MoveVM.
There is more coming and this is about changes in the
way we do verification/resolution/dispatching.
I think it needs its own PR

for what is worth and for a good laugh removing the resolution at runtime improves performance of a simple test calling natives of ~5%.
Given the number of native functions and the optimized hash map implemented in resolution (a complex switch) that seems a good win :)
Anyway just mentioning that things did not get worse

@tnowacki, @vgao1996 let me know if you have changes or things you'd like to see happening here. I got more PRs coming in this space...

@bors-libra r=tnowacki

:pushpin: Commit a003132 has been approved by tnowacki

:hourglass: Testing commit a003132e1eae403199762dd1cb3f8a604bf15ce0 with merge 0f7bb07247444092441dfb6a5dedfb0bb74fd31b...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: tnowacki
Pushing 0f7bb07247444092441dfb6a5dedfb0bb74fd31b to master...

@dariorussi just FYI it looks good to me! I had some concerns but those were already addressed in our meeting.


feat(hono): enable context storage

Instead of #852 and #884, let's use Hono's getContext instead.

When testing this, I receive the "Context is not available" error when trying to access the hono context within a server component.
I was able to figure out a way to update the dev server plugin to work with the hono context. That is available. I'm not so sure about this solution... I have to reconstruct the hono app to pass the updated env and ctx to the fetch handler. The hono context is immutable once created.
Hono has an adapter system for their vite plugin. The cloudflare adapter returns the env and executionContext and they mix that in when invoking fetch: https://github.com/honojs/vite-plugins/blob/main/packages/dev-server/src/dev-server.ts#L124-L137
I created a new PR with a new fetch adapter option for the dev server. I think this will be much more user-friendly for cloudflare devs than copy and pasting a custom plugin. #892

Assuming this PR itself is fine, let's merge.


Fix tests failures in parquet_write_test.py

FAILED ../../../../integration_tests/src/main/python/parquet_write_test.py::test_hive_timestamp_value
FAILED ../../../../integration_tests/src/main/python/parquet_write_test.py::test_non_empty_ctas
FAILED ../../../../integration_tests/src/main/python/parquet_write_test.py::test_parquet_write_fails_legacy_datetime
FAILED ../../../../integration_tests/src/main/python/parquet_write_test.py::test_parquet_write_roundtrip_datetime_with_legacy_rebase
FAILED ../../../../integration_tests/src/main/python/parquet_write_test.py::test_ts_write_fails_datetime_exception

Some of these tests fail because of the use of a deprecated conf variable:
E               pyspark.errors.exceptions.captured.AnalysisException: The SQL config 'spark.sql.legacy.parquet.datetimeRebaseModeInWrite' was removed in the version 4.0.0
. Use 'spark.sql.parquet.datetimeRebaseModeInWrite' instead.

The failing tests are:

test_hive_timestamp_value
test_parquet_write_roundtrip_datetime_with_legacy_rebase

These should be trivial to fix.

test_non_empty_ctas fails because of creating a table pointing to a non-empty directory:
E               pyspark.errors.exceptions.captured.AnalysisException: CREATE-TABLE-AS-SELECT cannot create table with location to a non-empty directory file:///tmp/pyspark_tests/pop-os-main-3128167-2117921863/CTAS/ctas. To allow overwriting the existing non-empty directory, set 'spark.sql.legacy.allowNonEmptyLocationInCTAS' to true.

I'll need to look at the tests more closely.

Unassigning myself.  I don't think I'll get to this in the next couple of weeks.

Turning ANSI mode off we have the following failures with reasons
test_hive_timestamp_value (The SQL config 'spark.sql.legacy.parquet.datetimeRebaseModeInWrite' was removed in the version 4.0.0. Use 'spark.sql.parquet.datetimeRebaseModeInWrite' instead.)
test_non_empty_ctas (CREATE-TABLE-AS-SELECT cannot create table with location to a non-empty directory file:///tmp/pyspark_tests/a07cb15-lcedt-gw1-206777-597991485/CTAS/ctas. To allow overwriting the existing non-empty directory, set 'spark.sql.legacy.allowNonEmptyLocationInCTAS' to true.)
test_parquet_write_fails_legacy_datetime (The SQL config 'spark.sql.legacy.parquet.datetimeRebaseModeInWrite' was removed in the version 4.0.0. Use 'spark.sql.parquet.datetimeRebaseModeInWrite' instead.)
test_parquet_write_roundtrip_datetime_with_legacy_rebase (The SQL config 'spark.sql.legacy.parquet.int96RebaseModeInWrite' was removed in the version 4.0.0. Use 'spark.sql.parquet.int96RebaseModeInWrite' instead.)
test_ts_write_fails_datetime_exception (The SQL config 'spark.sql.legacy.parquet.datetimeRebaseModeInWrite' was removed in the version 4.0.0. Use 'spark.sql.parquet.datetimeRebaseModeInWrite' instead.)


feat: adds option to remove persistent user data

Provides a native way for Finch to delete a persistent user data disk. Calling finch vm remove --user-data will now cleanup both the VM and any persistent user data.
Signed-off-by: Sam Berning<EMAIL_ADDRESS>Issue #, if available:
Description of changes:
Provides a native way for Finch to delete a persistent user data disk. Calling finch vm remove --user-data will now cleanup both the VM and any persistent user data.
Testing done:
E2E, Unit, and manual testing
$ ./_output/bin/finch vm init
$ ./_output/bin/finch vm stop
$ ./_output/bin/finch vm remove --user-data
$ ls ./_output/lima/data/_disks/finch
ls: ./_output/lima/data/_disks/finch: No such file or directory

[x] I've reviewed the guidance in CONTRIBUTING.md

License Acceptance
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I like the idea behind this PR, but it got me wondering if we need an entire disk lifecycle management command instead of just tacking it on to the vm commands. Like, do we need finch disk remove or finch disk recreate? Or I guess a better question: is there a use-case for deleting a disk without removing the entire VM?

Or I guess a better question: is there a use-case for deleting a disk without removing the entire VM?

Theoretically, users could also remove the disk on finch vm stop safely, and they might want to do that. Another case would be if a user runs finch vm remove without --user-data, they might later want to delete user data.
The reason I chose to attach this to finch vm remove was because persistent disk is invisible to users by design, and having a separate management command for the disk felt kind of counter to that. I think you're right that adding a separate management command would make this more flexible, perhaps as a vm subcommand though, such as finch vm clear-user-data?
The other option would be to also add a similar flag to finch vm stop and changing the logic such that the user data cleanup will still happen even if the VM is already in the state it's supposed to transition to.
Not sure which one I like better, WDYT?


Hotfix lzhang9

Fix to "AL2S OESS handler does not report error when a connection failed to provision #83".

@zlion Looks like the merge was blocked due to unsigned commits. Can you fix that?


Add support for Viacoin [chain-integration]

Description
Add support for Viacoin [chain-integration]
Pretty big code base. Please only review to see if I get the idea on how to support.
I'll try to add the tests later. Might take some time.
Testing instructions

Types of changes

Checklist

[ ] Prefix PR title with [WIP] if necessary.
[ ] Add tests to cover changes as needed.
[ ] Update documentation as needed.

Please add basic unit tests and fix CI (ignore Wallet Core Android now)

@romanornr This pr is 10 days old, any update? Would you mind if we close it now


Fixe Bug 1340

Fixes #1340 and #1322

Hmmm. I made no changes to PostCodeGeo table and I didn't do anything with migrations:

@mgmccarthy This same test failed for one of my recent PRs. @tonysurma was able to retry the build and it works 2nd time. Like you, I was not getting a failure locally.

Thanks @stevejgordon. @tonysurma can you retry this build please?

Will do as well as log an issue to figure out why it fails sometimes


OCM-12479 | fix: Fixed help message for binary builds

OCM-12479

Ran locally as well and this meets criteria for the bug ticket

/lgtm

/test coverage

/cherrypick release_1.2.48


websocket url to long  browser got stuck

Desktop

OS: mac10.14.6
Browser: chrome
Version:docker 1.9.0

ws://<IP_ADDRESS>:17001/action?token=05d3aa444233a13b3254eeff495a5138&some=

How to reproduce the issue?
Possible duplicate of #471


Prepare v0.16.2 release

Signed-off-by: Bridget McErlean<EMAIL_ADDRESS>

Codecov Report

Merging #957 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #957   +/-   ##
=======================================
  Coverage   47.62%   47.62%
=======================================
  Files          76       76
  Lines        5291     5291
=======================================
  Hits         2520     2520
  Misses       2617     2617
  Partials      154      154

Continue to review full report at Codecov.

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


Update example app vulnerabilities

Changelog:

Fixing issues raised by #86 and #87 :

Change to updated react and react native versions
Updated Android app/build.gradle for new react native version
Updated AppDelegate for new signatures in new react native version

General security changes:

Updated jsdoc version due to vulnerability in xml2js
Vulnerability still exists in dependency @expo/config-plugins and will require an update on their side.

Merging changes from main into develop

Codecov Report

Merging #88 (4c2f869) into develop (6efac3b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop      #88   +/-   ##
========================================
  Coverage    91.95%   91.95%
========================================
  Files           10       10
  Lines          373      373
  Branches        75       75
========================================
  Hits           343      343
  Misses          25       25
  Partials         5        5

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[5.3] Align and sort composer keywords

Align and sort composer keywords

Wyy?

The purpose is to inline them: and avoid noise when adding some new ones

This just added noise, and we're not adding new ones right now.


Image from API

Would be nice if images could be inserted from an API

think that should be something you use AlamofireImage or Kingfisher for

@mwanjajoel still interested in this? willing to implement if you give a case for why you would like the library to handle this rather than a separate image downloading library.


Add utility methods to split gradients to GradientParams

Changes
Currently, working with multiple different learning rates in Burn can be a bit of a hassle. The easiest way is to create a different GradientParams for each set of variables, and then step the optimizer for each of these. Eg. in my codebase I had something like:
let mut grad_means = GradientsParams::new();
grad_means.register(
    splats.means.id.clone(),
    splats.means.grad_remove(&mut grads).unwrap(),
);
let mut grad_opac = GradientsParams::new();
grad_opac.register(
    splats.raw_opacity.id.clone(),
    splats.raw_opacity.grad_remove(&mut grads).unwrap(),
);
let mut grad_rest = GradientsParams::new();
grad_rest.register(
    splats.sh_coeffs.id.clone(),
    splats.sh_coeffs.grad_remove(&mut grads).unwrap(),
);
grad_rest.register(
    splats.rotation.id.clone(),
    splats.rotation.grad_remove(&mut grads).unwrap(),
);
grad_rest.register(
    splats.log_scales.id.clone(),
    splats.log_scales.grad_remove(&mut grads).unwrap(),
);

This PR just adds some utility methods to turn this into
let grad_mean = GradientsParams::from_params(&mut grads, &splats, &[splats.means.id]);
let grad_opac = GradientsParams::from_params(&mut grads, &splats, &[splats.raw_opacity.id]);
// Since the first call modifies the grads this will just be the remaining grads.
let grad_rest = GradientsParams::from_module(&mut grads, &splats);

Not entirely sure about the names/shape of the API! It feels like this could be a method on Gradients instead but that's currently not really public.
To make the API slightly simpler I've made ParamId: Copy by making the ID a 64 bit random hash or something instead of a 6(?) byte random string encoded ID.
I've left from_grads in place as is to not break compatibility but also added a from_module, which allows people to extract a GradientParams for a specific module without consuming the gradients, so you could do something like
let linear_grads = GradientParams::from_module(&mut all_grads, &self.model.linear.a);
let rest_grads = GradientParams::from_grads(all_grads, &self.model);

from_grads could also just be the same thing as from_module if it's ok to break compat.
Something that's a bit gnarly that if you were to extract a GradientsParam for each param individually, each optimizer step would map over all parameters and things become O(n^2). I doubt that matters much in practice.

It is a minor but practical change. I think ParamId is fine to be u64 instead of String.

Thanks! Yeah let me just actually include the ID -> u64 change here as well, just so this has a nicer API from the get-go.

Ah great! Thanks for checking the backwards compat. I've added a small test at least for decoding "legacy" 6 bit IDs, doesn't exactly guarantee compatibility but it's something.
I've added a small section to the custom training loop, lmk if that looks good, I'm really not familiar with the "non-custom" training loop so idk how things would work there haha.
I don't know much about how to update the old models so if you're happy to do that that'd be amazing!

Hmmm looks like it doesn't entirely work for backward compatibility.
Currently fails in deserialize when decoding a param id 30b82c23-788d-4d63-a743-ada258d5f13c for the model I'm trying to convert in the examples.
Gonna investigate a bit further.

Oh hurr that's weird :/ 30b82c23-788d-4d63-a743-ada258d5f13c seems like way more than a 6 bytes id! I guess those might be coming from a different source?

Yeah the param ids used to be 128-bit UUIDs before it was changed in #1912, and the mnist-inference-web model I was trying to convert was generated before the switch.
It didn't break before since these were also stored as a string, but this explains the error.
I still think the rest of the changes are good! I'll just have to check for this case since users might still have UUIDs in their records that would otherwise break when trying to load.

Just added additional support for 16-byte uuid. Turns out the binary model doesn't have to be updated, just had to add backward compatibility for the parameter ids 🙂


Create dataset loader for OpenSLR

Dataloader name: openslr/openslr.py
DataCatalogue: http://seacrowd.github.io/seacrowd-catalogue/card.html?openslr

Dataset
openslr

Description
This data set contains transcribed high-quality audio of Javanese, Sundanese, Burmese, Khmer. This data set come from 3 different projects under OpenSLR initiative

Subsets
SLR35 (javanese), SLR36 (sundanese), SLR41 (javanese), SLR42 (khmer), SLR44 (sundanese), SLR80 (burmese)

Languages
mya, jav, sun, khm

Tasks
Automatic Speech Recognition

License
Creative Commons Attribution Share Alike 4.0 (cc-by-sa-4.0)

Homepage
https://www.openslr.org/resources.php

HF URL
https://huggingface.co/datasets/openslr

Paper URL
http://dx.doi.org/10.21437/SLTU.2018-11

#self-assign

Hi @akhdanfadh, we are freeing up this dataloader issue due to the lack of response so other contributors can #self-assign.

#self-assign

#self-assign


Fixed issue with new store indexing

The previous fix somehow worked for the specific test case I was using by coincidence, but is actually wrong as it will only work when the store is defined for the first time. Fixed issue and renamed some vars to avoid confusion.

LGTM


the fine-tuning of language in the content section

Unitable is a powerful recognition tool, but I want to train table content recognition that supports other languages. Have any good suggestions or opinions?

I would suggest finetuning the OCR branch with the targeted language and UniTable should work out-of-the-box.

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

OCR

In the recognition of the bbox section, there may be a large amount of text or gaps in a single bbox, which can lead to content loss or misalignment. Do you have any good suggestions for this situation? What model or debugging method is called for pre training or fine-tuning of unitable_1arge_bbox.pt?


how to  deal with  in one page  have  two  or  more   place  to  need  use  vue-countup-v2

I need to insert a lot of numbers in the carousel, and each one has a digital animation. Do I need to create many instances?
i  think  it  is  so  bad............. please  help....

because  the carousel  must  in  one  page

demo repository?


getTarget stopped working - now fixed

_currentTarget did not take rotation into account.

Cannot merge due to conflicts (mostly because of previous merge:))


Fixing issue #53: preventing a division by zero error

As described in issue #53, in rare occasions you hit a "division by zero" error because there's no check for the value of the divided number. This fix should solve the problem, with a unit test to ensure the issue will be covered.

Fixed by #55