Add .deb support in pacstall

Can we include .deb packages for some packages like Brave Nightly in Pacstall? It will help many users who don't want to just compile packages like a web browser. It will make it even more better and similar to AUR where both git and binaries are available.

You mean -bin packages, yes we have those. I don't understand why you would need .deb packages, as apt already handles that; currently -bin packages get the latest executable from upstream and install it in the appropriate directory.
We are not looking forward to adding scripts for packages which have updated PPAs, it would result in redundancy if we did so.

Ok, I wanted to know how we can add pacstrips for bin packages. I thought .deb was our only choice.

The exact process of making a pacscript differs package to package, for -bin packages you can check out the onefetch-bin pacscript.
Basically, you change the url field to the tar.gz/.zip release, Pacstall automatically extracts the archive and cds into it, then you can mv the binaries/libraries etc to the correct location.
One thing to remember though, is that when you move make sure to use /usr/src/pacstall/"${name}"/ as a prefix, for example if you want to move file A to /usr/bin then you would do this
mv A /usr/src/pacstall/"${name}"/usr/bin

I mean treat the /usr/src/pacstall/"${name}"/ directory as a "fake root" for your packages. Pacstall will symlink whatever you put in that directory to the system's corresponding directories.
For more information read make-a-pacscript, and technical-side

Thanks now I can contribute to the project

I don't understand why you would need .deb packages, as apt already handles that; currently -bin packages get the latest executable from upstream and install it in the appropriate directory.

Because Ubuntu doesn't allow one to simply install the latest packages from the latest repositories - instead some of us are stuck with the Bionic or older...
Reading the documentation of pacstall, i thought the whole point was that PPA's are a floored system and it's better to be able to just reference the source of the package directly and install it, enabling the user to keep the packages up to date.
Forgive me if i'm wrong, but i think based on that, enabling the download of .deb packages is a worthwhile investment, giving the user the flexibility to get packages where a .zip or tar.gz is not available (teams-for-linux as an example)
The way this works in PKGBUILD files is the maintainer simply passes an array of downloaded file names that the packager should not attempt to extract:
noextract=("${pkgname}_${pkgver}_amd64.deb")
Cheers,
LS

Because Ubuntu doesn't allow one to simply install the latest packages from the latest repositories - instead some of us are stuck with the Bionic or older...

I can't say for sure, but Pacstall mainly supports the latest LTS. @Henryws needs to take a look here.

Reading the documentation of pacstall, i thought the whole point was that PPA's are a floored system and it's better to be able to just reference the source of the package directly and install it, enabling the user to keep the packages up to date.

Uh, no. Not all PPAs are bad. Pacstall is a good alternative for software that has no existing updated PPAs, and for software that people may want to compile on their system. Pacstall isn't meant to replace PPAs.

Forgive me if i'm wrong, but i think based on that, enabling the download of .deb packages is a worthwhile investment, giving the user the flexibility to get packages where a .zip or tar.gz is not available (teams-for-linux as an example)

I guess making scripts for software that doesn't have an archive release is a good idea (like teams). @Henryws needs to take a look here.

I don't understand why you would need .deb packages, as apt already handles that; currently -bin packages get the latest executable from upstream and install it in the appropriate directory.

Because Ubuntu doesn't allow one to simply install the latest packages from the latest repositories - instead some of us are stuck with the Bionic or older...
Reading the documentation of pacstall, i thought the whole point was that PPA's are a floored system and it's better to be able to just reference the source of the package directly and install it, enabling the user to keep the packages up to date.
Forgive me if i'm wrong, but i think based on that, enabling the download of .deb packages is a worthwhile investment, giving the user the flexibility to get packages where a .zip or tar.gz is not available (teams-for-linux as an example)
The way this works in PKGBUILD files is the maintainer simply passes an array of downloaded file names that the packager should not attempt to extract:

source=("https://packages.microsoft.com/repos/ms-teams/pool/main/t/teams/${pkgname}_${pkgver}_amd64.deb")

noextract=("${pkgname}_${pkgver}_amd64.deb")

Cheers,
LS

Can't you just use the deb of teams?

Of course people could use the deb, but it's tedious to go the the web site and check if there is a new release to install.
I use FreeTube and they provide a deb file, but they to not have a repository so every user need to check for and install a new release manually.
So I guess the question is, could Pacstall be the tool that download this deb and install it so not everyone need to check for a new release and manually download and install the deb.
What I have done for FreeTube is written a script that checks the Github API for new release of FreeTube, if a new release is available it download the deb and install it.

So perhaps we could extract the deb and make a -bin script

I guess making scripts for software that doesn't have an archive release is a good idea (like teams). @Henryws needs to take a look here.

I think this issue, is basically asking to make pacstall a PPA manager (by adding .deb support, you are basically managing apt/PPAs). Since users are capable of creating their own PPAs, I think this feature is kind of redundant now.

So remove ppa support in 1.4?

So remove ppa support in 1.4?

No I don't mean that, I mean its kind of redundant to make pacscripts that specifically install .debs, as you are basically doing what apt/PPAs do.
The ppa array is useful, and should stay as some pacscripts need additional software from PPAs ( so we can leverage the already existing PPA system)

@wizard-28 It's tedious to manage PPAs. It's one of the criticism of apt over other package managers. So Pacstall can help in this management.


Further improve caching documentation.

Follow-up to #7223 that fixes a doc bug (a result-level cache property
was misspelled), changes the recommended "small cluster" threshold from
20 to 5 servers, and clarifies behavior of the various caching options.

@gianm I'm glad to see that the original PR led to a comprehensive rewrite of the caching doc :)
What I'm curious about is the performance of broker merging in a fully cached scenario. Imagine that all segment results are available in memcached. Then what's the threshold where a single broker getting all results in bulk from the cache then merging them all, starts falling behind farming it out to historicals? Historical caching has its own issues, such as inability of getting results in bulk, and lower local cache hit ratio when replication > 1, so I have a feeling that this threshold could actually be quite high when merging is cheap (simple sums, no HLLs, etc). It's probably really hard to write a good rule of thumb for this in the doc though, since it depends on many factors such as # of historicals, # of segments, result size, result merging cost, etc.
What could be useful, though, is a curated list of holistic caching setups for the entire cluster. It's easy to get lost between all individual settings for using/populating local/remote/hybrid-L1/hybrid-L2 segment-level caches in the brokers, historicals, and MMs. But probably only a handful of distinct combinations of these settings make sense.
For example, a simple setup that would be a good starting point for most users would be:

Broker: useResultLevelCache = true, populateResultLevelCache = true
Historical: useCache = true, populateCache = true
And everything else to default.

A handful of other setups may make sense in more specific scenarios. The discussion in https://github.com/apache/incubator-druid/issues/4947 contains quite elaborate schemes.
I'm not familiar enough with caching perf at the moment to write such a list of recommended setups, but I intend to benchmark a few of these setups later so perhaps I'll give it a shot after that.

Those are all good questions. I'm not sure what the exact tradeoff is between letting the broker use (but not populate) a remote cache. I am sure that for a big enough cluster it's a bad idea, since it would have the same problems that enabling cache population does.
I think the setup you mentioned is a good one for most users, and if you're interested in doing a PR to add the recommendation to the doc, please do!
I also think if we fix #5796 we should add to the recommendation that using an external cache is a good way to boost hit rates, if you don't mind the operational overhead of running the external cache. Doing a bulk get on historicals minimizes the overhead of pulling segments. Cache hit rate is still maximized. And we avoid the broker being a bottleneck.


could not run on mac with error

/ (master) $ thyme track                                                                                [16:07:10]
AppleScript error: exit status 1, output was:
PROCESS 45067:MAMP
264:276: execution error: System Events got an error: osascript is not allowed assistive access. (-25211)
2016/08/27 16:08:29 AppleScript error: exit status 1, output was:
PROCESS 45067:MAMP
264:276: execution error: System Events got an error: osascript is not allowed assistive access. (-25211)

Did you follow the instructions listed in thyme dep?

yes
does this mean my application MAMP can't be stated?


Unify calls to conda, jupyter, etc.

Currently, there're several ways in which conda and jupyter are called within the Dockerfile and the install_gds_py shell script. It'd be good to have them all be the same and, maybe, in a more portable way, if that is a possibility (ie. not /opt/bin/conda, etc.)

Fixed with #19


Potential memory leak?

Describe the bug
After prolonged connection, the player eventually runs out of memory. The problem doesn't exist outside a Raknet connection, and, once the problem happens, it prevents player from further rejoining the server again with Raknet, only restarting the game works.
To Reproduce
Steps to reproduce the behavior:

Join server with Raknet
Idle
Gets kicked

Screenshots

Runtime info (please complete the following information):

OS: Arch Linux
Minecraft version: 1.18.2
Mod version: alpha 4.16

Crash reports / logs
https://0x0.st/oQly.txt
Other mods
https://0x0.st/oQlt.txt
Checklist

[x] I am using the official version of the mod.
[x] I tried the latest development version but the issue persists.
[x] I searched for similar open issues and could not find an existing bug report on this.

ive gotten the same issue in 1.19 versions using the official version of the mod and latest development

Try reproduce with jvm flag  -Dio.netty.leakDetection.level=advanced

common mods:

axes are weapons
blastproof netherite
carpet / extra / tis
cloth config
couplings
fapi
ferritecore
krypton
lazydfu
lithium
starlight
raknetify (obviously)

As the log is extremely big I'll just paste it here https://0x0.st/oQhi.log

Try reproduce again with latest devbuild.

Try reproduce again with latest devbuild.

reproduces, do I need to provide a log again?

reproduces, do I need to provide a log again?

Yes definitely

it will take some time to reproduce a log, brb

Try reproduce with latest devbuild and provide logs if it still leaks.

the log is no longer spammed by things now so I think it's now fine? I'll wait and see

I don't reproduce with the latest build, thx pal.

Remember to remove the flag since the advanced leak detector comes with some performance overhead.


Fix TypeError on changelist_view in django 1.11

List of categories view throws exception in django 1.11 admin. It happens because of changes in Django 1.11 (https://docs.djangoproject.com/en/1.11/releases/1.11/#django-template-backends-django-template-render-prohibits-non-dict-context. So, where is a small changes on redirect_response. RequestContext returned as flatten dict.
Also changes on tox.ini and travis.yml (lines 3-4, not sure it required, but tests didn't run without it with error "Interpreter not found: python 3.5")

I'm sorry, but why is this issue is closed? I still get that error on Django 1.11.1.

context must be a dict rather than RequestContext.

Hi, Coyokto.
Is it happening with "category_detail" view?

Well, it happens when i try to open "Categories" in admin (/admin/categories/category/).
In Traceback changelist_view is highlighted bold, if this is what you asking.

Well, it happens when i try to open "Categories" in admin (/admin/categories/category/).
In Traceback changelist_view is highlighted bold, if this is what you asking.

Coykto, thank you for your quick reply, I'm going to reproduce the problem today.
If you find some time to investigation, please check the code below in the module:
# categories/editor/tree_editor.py
# line 260-254
# method old_changelist_view()

        return render_to_response(self.change_list_template or [
            'admin/%s/%s/change_list.html' % (app_label, opts.object_name.lower()),
            'admin/%s/change_list.html' % app_label,
            'admin/change_list.html'
        ], context=context_instance.flatten())

It must return ontext_instance.flatten() instead of context_instance.

Thank you very much! It helped! There was context_instance, not context_instance.flatten() for some reason. And i installed it with pip just few hours ago. Anyway, thanks again!

Thank you, because I found another one RequestContext which not flatten yet )


chore(main): release 0.31.0

:robot: I have created a release beep boop
0.31.0 (2023-03-28)
Features

heart-rate: keep-alive (8b3509b)
heart-rate: save beat to database (e174f45)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/CCXXXI/ecg_monitor/releases/tag/v0.31.0 :sunflower:


Fully support LaTeX builder

Support for subfigure should be possible via the subcaption package, plus minipage environments for more complex layouts.
Note, from https://latex-tutorial.com/subfigure-latex/, the subfigure package was superseded by the subfig package, which was then superseded by the subcaption package.
Resources:

https://www.overleaf.com/learn/latex/How_to_Write_a_Thesis_in_LaTeX_(Part_3)%3A_Figures%2C_Subfigures_and_Tables
https://tex.stackexchange.com/questions/222656/how-to-align-multiple-subfigures-in-a-grid
https://tex.stackexchange.com/questions/304826/complex-subfigure-layouts
https://tex.stackexchange.com/questions/333249/controlling-subfigure-captions-and-subfigure-placement
https://tex.stackexchange.com/questions/288141/arrange-images-as-tables-or-subfigure-or-some-other-layout
https://tex.stackexchange.com/questions/269301/subfigure-out-of-order-placement-numbering
https://tex.stackexchange.com/questions/404850/subfigures-side-by-side-with-captions-using-minipage-instead-of-subfigures

This would be super cool! Is there any progress on this issue?


Throw an exception if you try to register a module as an internal client

See #93

What if you are running multiple routers in one process? And the object is an internal client of one router and a module of the other?
I am going to close this. If this is still an issue for someone, please open a new issue.


remove default namespace and namespace overrides

Signed-off-by: Utkarsh Srivastava<EMAIL_ADDRESS>Description

This PR removes the namespace overrides. Now it falls back to namespace specified in the manifest if the ApplyOptions has no custom namespace specified.
The ApplyManifest function no longer fails when namespace creation fails due to invalid namespace name (empty string), which may happen for api-resources which are not namespaced.

Signed commits

[x] Yes, I signed my commits.

Great!
Can't quite reconcile This PR removes the namespace overrides. (in this PR's description) and // The namespace specified in ApplyOptions is used doc in the code (great that the doc got updated!!)
As an aside, I wondered why length is compared to  1 instead of 0 here:
if err == nil && len(val) > 1 {
			if len(options.Namespace) > 1

@utkarsh-pro @kumarabd just wondering: are you testing custom operation as well?

Great!
Can't quite reconcile This PR removes the namespace overrides. (in this PR's description) and // The namespace specified in ApplyOptions is used doc in the code (great that the doc got updated!!)
As an aside, I wondered why length is compared to 1 instead of 0 here:
if err == nil && len(val) > 1 {
			if len(options.Namespace) > 1

Just to account for whitespaces because we don't strip it before removing them @mgfeller

@utkarsh-pro @kumarabd just wondering: are you testing custom operation as well?

I didn't, may be @utkarsh-pro did as he said he's testing out almost everything.

I think that we should strip leading and trailing whitespace in the text fields, so that any ambiguity is removed.

Leading and trailing whitespace should also be removed here in MeshKit. Will you confirm that we are stripping leading and trailing whitespace?

Great!
Can't quite reconcile This PR removes the namespace overrides. (in this PR's description) and // The namespace specified in ApplyOptions is used doc in the code (great that the doc got updated!!)
As an aside, I wondered why length is compared to 1 instead of 0 here:
if err == nil && len(val) > 1 {
			if len(options.Namespace) > 1

Just to account for whitespaces because we don't strip it before removing them @mgfeller

Ok.... exactly 1 whitespace, where is that coming from? I agree that leading and trailing whitespaces should always be removed.


fix: additional trait alignment

Due to this: https://github.com/n0-computer/iroh/actions/runs/11909126284/job/33185889868?pr=2932
We need to further clean up our traits and make sure they all fit together.
Currently blocked on figuring out the composite structs here.

No longer required


How to mock ChunkManager for jest tests

Environment
"@callstack/repack": "^2.5.2"
"jest": "^26.6.3",
Description
I have a simple jest test using react-test-renderer:
it('renders correctly', () => {
  renderer.create(<App />)
})

Where ChunkManager.configure() is called in App.tsx. After adding @callstack to transform ignore patterns yarn test is showing:
ReferenceError: __repack__ is not defined.
I have tried many different solutions to mock this, for example:
global.__repack__ = jest.fn()
jest.mock('@callstack/repack/client', () => ({
  ChunkManager: {configure: jest.fn()}
}))

Which times out with the following error:

I am using jest.useFakeTimers(). So what is the proper way mock this to accomplish a simple render test? Im new to this, apologies in advance.

No longer seeing this issue after upgrading jest to
"jest": "^29.0.2"

Hey @bea73693 , I seem to run into a similar issue as well, and updating jest to the latest didn't help
ERROR  ReferenceError: Can't find variable: __repack__
ERROR  TypeError: undefined is not an object (evaluating '_$$_REQUIRE(_dependencyMap[2], "./Root").Root')


Support multi-CQ for N300 MMIO device

We have four dispatch cores defined for each N300
When we init MMIO device we use two cores to service CQ 0 on MMIO and two cores to service CQ 1 for remote device. If we want to init 2 CQs on the MMIO then we won't have enough to program the dispatch cores that service remote CQ. On main we will throw if we hit this case.
We can remove some storage only cores and assign them to dispatch cores (this means losing some l1 banks but this is temporary since we will be moving dispatch to eth cores @davorchap)
Can also update MMIO device initialization to remove programming any dispatch cores that remote FD needs.

Closing this because this has been addressed


Config default persistence

When writing to an assembly-specific config, persist the config as the default for other assemblies.

This is in place as of https://github.com/zspitz/Periscope/commit/83e3d40bc27e27e2b29338bd839f26e9dc5e2b09 . Once we integrate Periscope into the ANTLR visualizer, we can test that this works.


WebAuthenticationBroker from ViewModel?

Hi,
I wonder if there is any documentation on how to handle authentication identity, token, claims, and user, from when the UI logic is decoupled into a ViewModel etc. layer that's UI-agnostic.
Related: https://github.com/microsoft/ProjectReunion/issues/31

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

ID: 3c758cb2-ad03-f28e-6947-21337462753a
Version Independent ID: 3e818dc7-e032-c92b-80ba-0c0773391fec
Content: WebAuthenticationBroker Class (Windows.Security.Authentication.Web) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.Security.Authentication.Web/WebAuthenticationBroker.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @lastnameholiu
Microsoft Alias: alholiu

@weitzhandler - I'm providing a few ideas based on my experience as the MSAL.NET developer. MSAL.NET integrates with WAB.
WAB is deprecated and as far as I know there is no new feature work happening there. I have not investigated if there are win32 projections of the WAB API.
I would suggest instead you look at WAM, which provides much better support especially around device authentication, integration with Windows Hello etc. There are win32 projections of the WAM API. From my experience with integrating with WAM (currently in progress), the difference between UWP and Win32 is:

on Win32, you are required to parent the Authentication controls to the a window handle
on UWP, this is not required, because you can always get the current view
on both, some of Authentication popups require you to be on an UI thread

Thanks @bgavrilMS for your kind and detailed reply,

MSAL.NET

MSAL doesn't cover custom identity management or legacy authentication with username & password against a private authentication provider (https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/issues/1538).

I would suggest instead you look at WAM

Sorry I mixed between WAB and WAM.
Anyway, as I commented here, I wish there was a lower level agnostic abstraction of client-side authentication flow and token management that can be managed in the ViewModel side as well.
Currently it all has to be hand-written down, hence my request for an example for using this with a real life example involving MVVM and UI decoupling.

If any new actionable items arise, let us know either through a new Issue or directly on this thread. In the meantime, I'll be closing this Issue. Thanks once again for submitting your feedback!


Ion-content background image blank issue

Ionic version:

[x] 5.x
I'm submitting a ...

[x] bug report
[ ] feature request

Current behavior:
If a page has a background image set to ion-content, then while navigating to that page is flickering.
Expected behavior:
Flickering should not happen.
Steps to reproduce:
Create a page with ion-content set with background-image like below -
ion-content { --background: url('/assets/images/sky_background.png') 0 0/100% 100% no-repeat;
Actually I want to get an update on the issue If this issue is fixed or we still need to patch it via JS as suggested in the post:
https://github.com/ionic-team/ionic/issues/17494
As it is closed I can't get an update on that
In my case, When I use the background in ion-content for few milliseconds it's showing white and then it shows the background images, so it may be a similar issue
Ionic info:

   Ionic CLI                     : 6.6.0 (/usr/local/lib/node_modules/@ionic/cli)
   Ionic Framework               : @ionic/angular 5.1.1
   @angular-devkit/build-angular : 0.901.7
   @angular-devkit/schematics    : 9.1.7
   @angular/cli                  : 9.1.7
   @ionic/angular-toolkit        : 2.2.0

Cordova:

   Cordova CLI       : 7.1.0
   Cordova Platforms : android 8.1.0, ios 5.1.1
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.2.1, (and 6 other plugins)

Utility:

   cordova-res : 0.14.0
   native-run  : 1.0.0

System:

   Android SDK Tools : 26.1.1 (/Users/hardik.dg/Library/Android/sdk)
   ios-deploy        : 1.9.4
   ios-sim           : 8.0.2
   NodeJS            : v12.16.3 (/usr/local/bin/node)
   npm               : 6.14.4
   OS                : macOS Catalina
   Xcode             : Xcode 11.4.1 Build version 11E503a

Thanks for the issue. What device are you testing this on?

I have checked it in the Web(ionic serve) and iPhone X

Can you provide a repo with the code required to reproduce this issue?

Thanks for the issue! This issue is being closed due to the lack of a code reproduction. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.
Thank you for using Ionic!


protect app CR from change/deletion when in use

"in use" = referenced by existing KubeDirectorCluster
A job for the admission webhook.

This will involve adding v1beta1.Delete to the OperationType slice that we register for in createAdmissionService.
We'll need to pay more attention to ar.Request.Operation in the validation handlers for both cluster and app CRs.
For the app CR we will need to reject Delete if the CR is referenced by any current clusters. We might need to modify the cluster CR metadata to make it more easily searchable by app.
We could also automatically reject any Update for a referenced app CR. Probably we don't need to actually do the diff to check to see if anything is changing.
The "cluster" side of things isn't really related to this particular issue, but once we are registered for Deletes we will need to explicitly ignore those on the cluster side.
Also on the cluster side, we could make some use of the distinction between Create and Update. We shouldn't even try to do "fetch existing" if the operation is Create.

Yeah we'll need to put a label on the cluster CR that identifies the name of the KubeDirectorApp that it is using.
This is already available in the spec.app property, but the ability to search on object field values (as opposed to label values) is very limited.
One consequence of this is that the KubeDirectorApp schema will have to restrict its name values to valid label values:

Valid label values must be 63 characters or less and must be empty or begin and end with an alphanumeric character ([a-z0-9A-Z]) with dashes (-), underscores (_), dots (.), and alphanumerics between.

Not sure if our CR schema will be allowed to express limitations on the object name since that is a standard/generic resource property. If not, we can dynamically check it in the validator.

You know... I've gone to the trouble of figuring out most of the label-related implementation but now I'm thinking that may not be the way to go.
We already maintain an in-memory map of some discovered information for each cluster CR. We should probably just stick the what-app-does-it-use information in there too, so we don't have to query K8s.


Is MultipleImageUploadEditorAttribute file missing?

I want to use MultipleImageUploadEditor attribute but visual studio can not find related attribute file. There are  ImageUploadEditor.cs and ImageUploadEditorAttrbute classes but there is only MultipleImageUploadEditor.cs. Is there anyway to use MultipleImageUploadEditor?

Actually you can use it with EditorType("MultipleImageUpload"), as editor type attributes are just intellisense helpers. But i'm not sure if multiple upload editor works properly, didn't use it for long time, need to have a look.

I tried EditorType attribute usage, editor shown properly but when updating record have an errror:
An exception of type 'Newtonsoft.Json.JsonSerializationException' occurred in Serenity.Data.Entity.dll but was not handled in user code
Additional information: Unexpected token when deserializing row: StartArray
Newtonsoft.Json.JsonSerializationException was unhandled by user code
HResult=-2146233088
Message=Unexpected token when deserializing row: StartArray
Source=Serenity.Data.Entity
StackTrace:
at Serenity.Data.Field.JsonUnexpectedToken(JsonReader reader) in c:\project\Serenity\Serenity.Data.Entity\FieldTypes\Field.cs:line 132
at Serenity.Data.StringField.ValueFromJson(JsonReader reader, Row row, JsonSerializer serializer) in c:\project\Serenity\Serenity.Data.Entity\FieldTypes\StringField.cs:line 83
at Serenity.Data.JsonRowConverter.ReadJson(JsonReader reader, Type objectType, Object existingValue, JsonSerializer serializer) in c:\project\Serenity\Serenity.Data.Entity\Row\JsonRowConverter.cs:line 109
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.DeserializeConvertable(JsonConverter converter, JsonReader reader, Type objectType, Object existingValue)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetPropertyValue(JsonProperty property, JsonConverter propertyConverter, JsonContainerContract containerContract, JsonProperty containerProperty, JsonReader reader, Object target)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
InnerException:

Error you are getting is because MultipleImageUploadEditor sends JSON array to server side by default as field value, and it can't be deserialized to a string field.
Please update Serenity nuget packages to 1.5.1.
I tried making sure MultipleImageUpload works properly in this version.
Then use the ImageUploadEditor attribute like below:
        [ImageUploadEditor(allowMultiple: true, JsonEncodeValue = true)]
        public String ProductImages { get; set; }

And on top of your SaveHandler you should use EmbeddedMultiFileUploadBehaviour instead of EmbeddedFileUploadBehaviour as below:
[EmbeddedMultiFileUploadBehaviour("SonHareketAciklama", fileNameFormat: "ProductImage/{1:00000}/{0:00000000}_{2}", copyFilesToHistory: true)]
private class MySaveHandler : SaveRequestHandler<MyRow> {
}

After this it should work.

I updated serenity packages. I can use ImageUploadEditor . There is no EmbeddedMultiFileUploadBehaviour attribute class. I used EmbeddedFileUploadBehaviour and normally get an error:
Failed to load resource: the server responded with a status of 404 (Not Found)
jquery-2.1.4.js?v=EH--lVW_yI7FyrUkx5D-NA2:8630 POST http://localhost:59837/File/TemporaryUpload 404 (Not Found)

Serenity.Web and Serenity.CodeGenerator 1.5.1 packages somehow didn't make it to Nuget gallery, try to update them again please, and rebuild project.

Please reopen this if problem still persists.


SW-5318 Add createMany function to ParticipantProjectSpeciesStore

This will be used so assign a bulk list of species to any number of projects

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1933
#1932  👈
#1924
#1919
#1918
#1917
#1916
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nickgraz and the rest of your teammates on  Graphite

Merge activity

May 8, 12:09 PM EDT: @nickgraz started a stack merge that includes this pull request via Graphite.


Import failing due to reached ElasticSearch limit

Most of the VSCode extensions that I maintained are having an outdated version number in the search that does not correspond to what was merged.
vscode-extensions.davidanson.vscode-markdownlint for example shows as 0.47.0 in both 22.05 and unstable channels but it should show 0.48.1 as the change was merged (https://github.com/NixOS/nixpkgs/pull/194718) and backported (https://github.com/NixOS/nixpkgs/pull/195381) few days ago.

Seems like we've hit a size limit on the ElasticSearch instance:

Validation Failed: 1: this action would add [2] total shards, but this cluster currently has [3000]/[3000] maximum shards open

cc @garbas

I'm removing old indexes. Will take some time before cleanup is done. I'll manually rerun the last import workflow
I think there are few things we could do to improve this situation in the future (we should probably open separate issues).

report on marketing matrix channel when things fail to import on a daily GitHub workflow

have a garbage collection GitHub workflow which would remove older indexes (the ones with version older the what is in main)

And @ncfavier thank you for pinging me

I've removed the majority of indexes and hourly imports are working again.

Is there maybe a way to tweak the retention policy on ElasticSearch directly?

There is nothing really to tweak since we are using the managed ES.


Importing ordered_set fails on Python 3.5

Error:
Python 3.5.0 (v3.5.0:374f501f4567, Sep 13 2015, 02:27:37) [MSC v.1900 64 bit (AM
D64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import ordered_set
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\Rohan\AppData\Local\Programs\Python\Python35\lib\site-packages\
ordered_set.py", line 55, in <module>
    class OrderedSet(MutableSet[T], Sequence[T]):
  File "C:\Users\Rohan\AppData\Local\Programs\Python\Python35\lib\site-packages\
ordered_set.py", line 84, in OrderedSet
    def __getitem__(self, index: Sequence[int]) -> List[T]:
  File "C:\Users\Rohan\AppData\Local\Programs\Python\Python35\lib\typing.py", li
ne 1181, in overload
    raise RuntimeError("Overloading is only supported in library stubs")
RuntimeError: Overloading is only supported in library stubs

Typing library:
Name: typing
Version: <IP_ADDRESS>
Summary: Type Hints for Python
Home-page: https://docs.python.org/3/library/typing.html

Is there any reason this PR was never merged?

I didn't want to simply comment out the type annotations, but I fixed the issue in https://github.com/LuminosoInsight/ordered-set/commit/bf1a873b455908abc07083e054b83e3b86df4658 .


Improve CGAngle class

I could use Angle structure but it needs iOS 13.0+

I will just close it for now since Mantis still needs to support iOS 11.0+ and currently CGAngle works fine.


Tests should be in TypeScript

One last thing - if the tests were in TypeScript it would be easier to test changes to types. I might convert them if you agree.

I tried to write them in TypeScript, but Jest wouldn't allow me at all, it would just scream and die no matter what I tried. I you can do that it will be great. Also if there's a better way to run the tests that doesn't involve Jest I'm all for it.


Forcing IPv4 when connecting over IPv6

Hi,
I assume it would just be as simple as adding one line into the test-plugin file, but is there a way to force the server to use IPv4 even if the web server is running over IPv6. The server has the ability to communicate via IPv4.
Thanks

You can actually set this through config.php. Just add this line:
$config['curl'] = array(
     CURLOPT_IPRESOLVE=> CURL_IPRESOLVE_V4
);

Wonderful...pushing the update to all my servers now.
Thanks heaps!


What is the use of the quantity in the returned on the resolve endpoint

Can you please clarify why/what is returned for quantity in the response payload for the resolve/list subscription/ get subscription endpoints please?
https://docs.microsoft.com/en-us/azure/marketplace/partner-center-portal/pc-saas-fulfillment-api-v2#get-subscription

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

ID: fe8750da-cb47-7396-8d1b-0abff019a0be
Version Independent ID: b19143ed-64c0-1554-5ac9-610513382609
Content: SaaS Fulfillment API v2
Content Source: articles/marketplace/partner-center-portal/pc-saas-fulfillment-api-v2.md
Service: marketplace
Sub-service: partnercenter-marketplace-publisher
GitHub Login: @qianw211
Microsoft Alias: evansma

Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

@qianw211  - any insights please.?

I got the answer for this, the quantity is basically applicable when there is a per-seat/per-user based plan. This Quantity implies number of users the plan is activated for. And can be changed by the patch call on the Subscription Api, if the user wants to change it. @femsulu You can confirm and close this ticket if you like!

#please-close confirmed that this is correct!


Firefox bug with headers

Firefox returns non-lowercase headers unlike Chrome.

Closed upstream https://github.com/nextgensparx/huawei-router-api/issues/1


chore: update guide to include path to public key for ssh signing

The config file for using SSH paths may not be parsed correctly on some systems.  Add in a hint in the documentation to give an alternative to use the direct path to your ssh pubkey file instead of the contents of your key.

Why:
Closes [issue link]

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

@mitche50 Thanks so much for opening a PR! I'll get this triaged for review :zap:

:wave: hi from the product manager for Git Systems. Thanks for opening this PR!
I'm curious on what systems you've (or the internet at large has) seen a problem with using the literal key in this fashion. I couldn't find much with some searching, including on the Git mailing list, but I'm open to the idea that my search syntax was poor 😅 Could you send any pointers you have?
After doing some deep-diving on this topic, I have two alternate proposals for the docs team:

It looks like we could always suggest pointing to a file rather than a literal key, with no loss of functionality for our customers. Depending on what @mitche50 comes back with, we might want to change to that format and drop the literal syntax from our docs entirely.
Alternatively, we should emphasize the fact that you have to prefix your literal key with key::. That's not completely obvious as-written, and if you "merely" paste your key in without the prefix, you're relying on backcompat behavior in Git which may not be forwards-compatible with future key types.
Something like this should work, assuming the user correctly did the pbcopy step:

$ git config --global user.signingkey "key::$(pbpaste<EMAIL_ADDRESS>
(NB I didn't actually test that syntax, I wrote it from memory.)

The documentation (even with the key:: prefix) did not work on Ubuntu 20.04.02 running on WSL2.  This may be relegated to just whatever internally is happening on the VM that the windows linux kernel is using, but WSL2 is prevalent enough to at least address it.
IIRC (this issue was created a while ago) I had colleagues who had done this successfully with the literal key on macbooks, unsure if I had anyone who was running bare metal linux distribution comment.

Thanks @mitche50, I appreciate the additional detail. What do you think about basically replacing our guidance to use a bare key with advice to use the file path (rather than having one, then the other as fallback)? From my read of the Git mailing list, that seems to be the preferred approach when the feature was designed.

I think this works as well, the file should be compatible with all systems 👍

I've verified on MacOS BigSur
with:
❯ ssh -V
OpenSSH_9.0p1, OpenSSL 1.1.1q  5 Jul 2022
❯ git --version
git version 2.37.3

Using file path only means you don't need to pass the key inline or have an ssh agent running
diff --git a/config/git/config b/config/git/config
index 7aedf53..0df39fb 100755
--- a/config/git/config
+++ b/config/git/config
@@ -3,7 +3,7 @@
 # Please adapt and uncomment the following lines:
        name = Tim Jacomb
        useConfigOnly = true
-       signingkey = ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIDnTfTt8lqJdcucKu538C6EvPNruiO9r67Jvfrbokear (omitted)\n
+       signingkey = ~/.ssh/id_ed25519.pub

Hey @mitche50, are you up for making the changes to this PR that Matt suggested in the previous comment? No worries if not ✨

👋🏻 @mitche50, I'm planning on working on this early next week, but if you've got a chance and the interest before then, feel free to incorporate the changes that @vtbassmatt suggested ☺

Thanks, @mitche50 🙇🏻 We'll merge this at the end of the week after our freeze for GitHub Universe 2022 is over.

This is a nice improvement but it needs review against where the include is used, i.e. some docs don't make sense right now.
https://docs.github.com/en/authentication/managing-commit-signature-verification/telling-git-about-your-signing-key#telling-git-about-your-ssh-key
section 3, asks you to copy a key that isn't needed

@timja 👋🏻 Sorry, I lost track of this. The doc you links to looks alright to me, and doesn't seem to mention copying a key. Can you clarify?

Seems to have been fixed, can't remember exactly what was there I should have taken a screenshot.

Seems to have been fixed, can't remember exactly what was there I should have taken a screenshot.


gcc --version = gcc (Homebrew gcc6 6-20160320) 6.0.0 20160320 (experimental)

So installing this version of gcc, when i do a gcc --version command, it shows up as gcc 6.0.0
According to the documentation, shouldn't it be gcc 6.1?
Thanks

To help us debug this issue can you explain:

What you were trying to do (and why)
What happened
What you expected to happen
Step-by-step reproduction instructions (with as minimal input data as possible)

This is not an actual issue, and more of a request for clarification.
I installed gcc6 via homebrew using the appropriate commands, and added a symbolic link for gcc6
When I run the following command in terminal
gcc --version
I get the following result
gcc (Homebrew gcc6 6-20160320) 6.0.0 20160320 (experimental)
Copyright (C) 2016 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
But according to the documentation on Github, it should be gcc 6.1.0 right?
Just wondering why it is showing up as "experimental." Also, when will a stable version of gcc 6 come to homebrew?

I suspect you have installed the gcc6 formula when it was still in development and before 6.1.0 was released? Unfortunately, because we picked 6-<date> as the version number back then, Homebrew won't automatically detect that it needs to upgrade the formula now, because 6.1.0 compares less than 6-20160320. (It's comparing the second version component and 1 is less than 20160320.)
If you check with brew info gcc6 then it should show that you have indeed version 6-20160320 installed as gcc --version is reporting. The workaround is to brew uninstall gcc6 followed by brew install gcc6. This should lead to an installation of version 6.1.0.

Also, when will a stable version of gcc 6 come to homebrew?

An update of the gcc formula in Homebrew's main formula repository (homebrew/core) is in progress as Homebrew/homebrew-core#676 but is currently blocked by other changes that need to happen first. (You can check out the last few comments in the linked PR to learn more.)

Unfortunately, because we picked 6- as the version number back then, Homebrew won't automatically detect that it needs to upgrade the formula now, because 6.1.0 compares less than 6-20160320.

This was discussed in https://github.com/Homebrew/homebrew-versions/pull/1256 for the sake of historical clarity.


⚠ AGEG website (redirect) has degraded performance

In 0ca1b86, AGEG website (redirect) (https://ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2031 ms

Resolved: AGEG website (redirect) performance has improved in d3c9c45.


[2.2] fix: issue #10295 fixed.

[2.2] fix: issue #10295 fixed.

Checked the slug related concerns and found working fine.

Verify that a slug is correctly generated from a given title, especially handling spaces and special characters.
Checked and found working.

Uniqueness of Slugs
Checked and found working.

Manual Slug Editing
Checked and Found working.

Handling of Non-ASCII Characters
Checked and Found Working

Video
https://github.com/user-attachments/assets/f65ed61f-4f5a-4d3b-8874-76ad8b0a8044
Screenshot
https://prnt.sc/55yXW4CiWt14


chore: Add admin api to resend renew revoke message

Adds an admin api to resent the renew revoke message
Adds an emergency kit to recreate the position from the signed channel and last failed order. This might be needed if the app set the order to fail, which will block the app from recreating the app. I opted to not make the code smarter when processing the RenewRevoke message but rather add an emergency kit feature. That will hopefully be less failure prune.

fixes https://github.com/get10101/meta/issues/413

Since we use PR names to generate the changelog and this PR is adding two feats, we should consider renaming the PR.

I've created a separate PR for the emergency kit https://github.com/get10101/10101/pull/2296


Ztunnel missing writeable /tmp dir for gathering pprof

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
When attempting to gather a debug/pprof/profile for ztunnel on a default install the user is met with an empty response

To resolve this, the following volume and volumeMount is required to add a /tmp directory in the ztunnel daemonset manually, or using helm:
helm upgrade --install ztunnel istio/ztunnel \
-n kube-system \
--version=1.21.0 \
-f -<<EOF
hub: docker.io/istio
tag: 1.21.0
resources:
  requests:
      cpu: 500m
      memory: 2048Mi
istioNamespace: istio-system
# Additional volumeMounts to the ztunnel container
volumeMounts:
  - name: tmp
    mountPath: /tmp
# Additional volumes to the ztunnel pod
volumes:
  - name: tmp
    emptyDir:
      medium: Memory
EOF

Should we be including this missing volume and volumeMount in the default ztunnel deployment templates? or alternatively, if we decide not to fix the template so it "just works" should we upgrade the handler to have a meaningful reply if ztunnel doesn't have the writable location it requires?
cc: @ilrudie @bleggett
Version
% istioctl version
client version: 1.21.0
control plane version: 1.21.0
data plane version: 1.21.0 (25 proxies)

% kubectl version
Client Version: v1.29.0
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.3-gke.1118000

Additional Information
No response

I do think we should fix this and should consider making it the same dir we use in the rest of Istio probably?

I might be missing something here - why do we need a volume mount if we are not sharing the volume with anything else?
Is it just that /tmp is not writable inside the container?

readOnlyRootFileSystem is set for security reasons (is this voided by having an emptyDir? maybe... but I suppose one folder writeable is better than all folders)


Doc: update widget doc to Livewire v3 format

The Livewire dispatch event now expects JSON. See issue #8533 for full details.
Added an example of working code.

[ ] Changes have been thoroughly tested to not break existing functionality.
[x ] New functionality has been documented or existing documentation has been updated to reflect changes.
[ ] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Oh, I have just tried this, it is awesome. Thanks

Thanks for the example, but I did remove it. It just didn't feel like it belonged in the documentation like that. Could be a nice trick on the community section of our website.


[Bug] [Ftp source] java.lang.NoClassDefFoundError: org/apache/hadoop/fs/FSDataInputStream

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
when I use ftp connect,there wa a error:

SeaTunnel Version
2.3.0-beat
SeaTunnel Config
config/ftp.conf :
env {
  execution.parallelism = 1
  job.mode = "BATCH"
}

source {
  FtpFile {
    path = "/var/ftp/pub/1.txt"
    host = "<IP_ADDRESS>"
    port = 21
    user = test
    password = test
    type = "text"
    schema = {
      name = string
      age = int
    }
    delimiter = "#"
  }
}

transform {
}

sink {
  Console {}
}

Running Command
run command:
./bin/seatunnel.sh --config config/ftp.conf

Error Exception
Exception in thread "main" java.lang.NoClassDefFoundError: org/apache/hadoop/fs/FSDataInputStream
	at org.apache.seatunnel.connectors.seatunnel.file.config.FileFormat$2.getReadStrategy(FileFormat.java:55)
	at org.apache.seatunnel.connectors.seatunnel.file.source.reader.ReadStrategyFactory.of(ReadStrategyFactory.java:32)
	at org.apache.seatunnel.connectors.seatunnel.file.ftp.source.FtpFileSource.prepare(FtpFileSource.java:60)
	at org.apache.seatunnel.engine.core.parse.ConnectorInstanceLoader.loadSourceInstance(ConnectorInstanceLoader.java:60)
	at org.apache.seatunnel.engine.core.parse.JobConfigParser.sampleAnalyze(JobConfigParser.java:291)
	at org.apache.seatunnel.engine.core.parse.JobConfigParser.parse(JobConfigParser.java:119)
	at org.apache.seatunnel.engine.client.job.JobExecutionEnvironment.getLogicalDag(JobExecutionEnvironment.java:135)
	at org.apache.seatunnel.engine.client.job.JobExecutionEnvironment.execute(JobExecutionEnvironment.java:127)
	at org.apache.seatunnel.core.starter.seatunnel.command.ClientExecuteCommand.execute(ClientExecuteCommand.java:71)
	at org.apache.seatunnel.core.starter.Seatunnel.run(Seatunnel.java:39)
	at org.apache.seatunnel.core.starter.seatunnel.SeaTunnelClient.main(SeaTunnelClient.java:31)
Caused by: java.lang.ClassNotFoundException: org.apache.hadoop.fs.FSDataInputStream
	at java.net.URLClassLoader.findClass(URLClassLoader.java:387)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:418)
	at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:352)
	at java.lang.ClassLoader.loadClass(ClassLoader.java:351)
	... 11 more

Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Are you add hadoop dependency in your classpath?

I can't add it,how can I do it?

Do you add hadoop dependencies in your classpath?

I can't add it,how can I do it?Do I need to install Hadoop?

You need add hadoop dependency jars such as hadoop-common in classpath of ST.

You can try to add hadoop-hdfs and hadoop-client dependencies to see if this problem can be solved.

You need add hadoop dependency jars such as hadoop-common in classpath of ST.

Thank you ,I have add dependencyies in plugins folder of ST,include hadoop-common-2.6.5.jar etc.
but now the following error occurred:

Is it a dependency version problem or missing other dependencies?

You can try to add hadoop-hdfs and hadoop-client dependencies to see if this problem can be solved.

You need add hadoop dependency jars such as hadoop-common in classpath of ST.

Thank you ,I have add dependencyies in plugins folder of ST,include hadoop-common-2.6.5.jar etc.
but now the following error occurred:

Is it a dependency version problem or missing other dependencies?

You can try to add hadoop-hdfs dependency to see if this problem can be solved.

You need add hadoop dependency jars such as hadoop-common in classpath of ST.

Thank you ,I have add dependencyies in plugins folder of ST,include hadoop-common-2.6.5.jar etc.
but now the following error occurred:

Is it a dependency version problem or missing other dependencies?

我遇到了跟你一样的问题，请问下你后来如何解决了

Dear all, you can download hadoop shade jar from mvnrepository and then put it into folder ${Seatunnel_HOME}/lib


Tech tiles

What do you think about this to manage tech tiles and advanced tech tiles?

I think it's good enough!
Still missing: Federation on top of the navigation research area. I will connect with your federation management


Fix potential nontermination in Observable.zip[Map]3

Fixes #590
This needs to be replicated for other zipMapX operators, but I want some feedback first :)
The behavior expected for zipped observable is to complete as soon as any single of zipped stream completes, which is totally not what current implementation does. Before this fix, it would complete only if one of streams called onComplete not concurrently with onNext or, if neither did, when all streams called onComplete. Waiting for all streams would result on non-termination when backpressuring, as a zipped one would never emit its next result, or would attempt to resolve same promise twice, concurrently.
Tests didn't catch this because they only use equally sized observables.

 We need a backport to 2.x as well 🙁

Could you also do a backport on the series/2.x branch?


[d3d9] Disable exclusive fullscreen for KiriKiri Z games

KiriKiri Z games rely on GDI for top menu bar. If exclusive FS enabled, display or hide menu will result black flicker. So we disable exclusive FS for KiriKiri Z games. Tested on 9-nine- series and Corona Blossom series on Windows 11 and Kali latest with Nvidia GPU.
Edit: Some other galgames also have similar behavior to krkrz, maybe we also need to add them into this new function.
I didn't add krkr2 because krkr2 relies on gdi heavily and only use d3d9 for limited features lick video playback which is nearly impossible to handle well in dxvk.
Edit2: CatSystem2 games have the same behavior, but I have difficulty to detect it. (example: Nekopara Vol.3, Love's Sweet Garnish)

Just how many KiriKiri Z games are there and why can't we simply use executable-name based config overrides, which is like what we almost universally do for everything else?

Just how many KiriKiri Z games are there and why can't we simply use executable-name based config overrides, which is like what we almost universally do for everything else?

Approx 100-200 titles on Steam, while a large amount of titles are not released on Steam. It will be a difficult work to collect file names for all these titles.
Also, galgames usually have a lot of fan made translated version with different file name, which makes it more difficult to write rules for all of them.
That't why I prefer to use universal fix for krkrz games just like what we do for UE4 games.
While for CatSystem2 games (only approx less than 30 titles and have no explicit dll handle to detect), I approve that rules can be better.


Added new/extra tests for converters

Summary
Added new tests or extra tests for converters of OptimizationProblem
I will add more tests.
Details and comments
new tests

[x] test_inequality_mode_integer
[x] test_inequality_mode_continuous
[x] test_inequality_mode_auto
[x] test_binary_to_integer
[x] test_optimizationproblem_to_operator
[x] test_valid_variable_type
[x] test_quadratic_constraints

extra tests for existing tests

[x] test_integer_to_binary

Hi @a-matsuo, there were some updates on the optimization-stack branch, could you update your branch and resolve the one conflict?

Thank you for the comments. I added docstrings and fixed all complains of pylint. I also changed warnings of unsupported quadratic constraints to error. Could you check them please?


Rename trans to codegen

This commit is a followup of changes from b63d7e2b1c4019e40051036bcb1fd5f254a8f6e2 in the Rust source to rename trans to codegen.

@mark-i-m thanks! I kept the trans definition with a note that it's been renamed, and added a codegen definition.

Thanks!


SQL Server Driver

Hi,
I've added a SQL Server driver, think it covers everything. Don't use all the functionality myself (99% of my db stuff is stored procedures, so I execute raw SQL), but generally it seems to work fine :)

I will add some unit tests later to make sure that works in the same way as other drivers.
Thanks.

Apologies for not adding them myself - bit overrun as present, but I'll add some as soon as I get chance to (if you've not found time before me!!!)


uid-map gets corrupted by Unicode usernames

/var/lib/nixos/uid-map seems to get corrupted by usernames containing Unicode characters. With every boot and nixos-rebuild switch, the size of the file increases. On my live system, this file grew to 2.1GB.
While running nixos-rebuild switch, update-users-groups.pl will rapidly increase in memory usage (using up all ~62GB of RAM available) until triggering the OOM killer, thus terminating the build process.

In my case, I had created a user with the name of 😛 (U+1F61B), but quickly removed it with the next build. Several rebuilds and reboots later, my system became increasingly slow to boot (1+ min). After trying to fix the issue (several rebuilds later), booting/rebuilding would take 5+ minutes.
When trying to boot, the system would hang on "running activation script..." at the beginning of stage 2.
Artifacts:
/var/lib/nixos/uid-map after running nixos-rebuild build-vm then launching the VM a couple times (4-5):

hexdump -C /var/lib/nixos/uid-map: uid-map-corrupted.hexdump.bin
raw file: uid-map-corrupted.raw.bin

Steps to reproduce:

Add user 😛 to users.users:

{ lib, config, pkgs, ... }:

{
  # ...
  users.users.myuser = { uid = 1000; extraGroups = [ "wheel" "docker" "libvirtd" ]; };
  users.users."😛" = { uid = 1001; };
}

nixos-rebuild build-vm
Execute the generated run-vm script
In the VM, observe the file size with du -b /var/lib/nixos/uid-map
Repeat from step 2 until system/build becomes unresponsive

Thanks to @cleverca22 for helping me figure this out.

Still relevant

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/running-activation-script-takes-several-minutes/9085/7

just cc'ing some people @Mic92 @Ma27 @aanderse  @volth  as I don't know perl

ping @stigtsp as well.

In long-term we should consider using https://systemd.io/USERDB_AND_DESKTOPS/ which is close to the concept of our database and also compatible with the rest of the Linux ecosystem.

I noticed when testing the example configuration of https://github.com/NixOS/nixpkgs/issues/80426#issue-566802577 in nixos-shell that it tries to remove the user on every activation:
% /nix/store/hk1np5bxy22awzdkb1hclvqiai5y23dm-nixos-system-nixos-21.03.git.3a3da0950d5/activate test
removing user ‘😛’
setting up /etc...
$ /nix/store/hk1np5bxy22awzdkb1hclvqiai5y23dm-nixos-system-nixos-21.03.git.3a3da0950d5/activate test
removing user ‘😛’
setting up /etc...

For simplicity it should tried the user name as a series of bytes instead of trying to decode it.

Fixed by https://github.com/NixOS/nixpkgs/pull/98544


Logo image and Home button breaks for baseURL with subpaths (redirects to wrong address)  when landingPageURL is present and MultiLingual mode isn't supported

Breaking behavior:
When baseURL contains subpath, landingPageURL is defined but there is no support for multiple languages
both logo and Home button redirects to "/" which will not match baseURL when it contains subpaths
More details:
Because .Site.IsMultiLingual variable is requirement for home button to use .Site.Params.landingPageURL
https://github.com/matcornic/hugo-theme-learn/blob/master/layouts/partials/menu.html#L17
https://github.com/matcornic/hugo-theme-learn/blob/master/layouts/partials/logo.html#L1
It's not possible change landingPageURL without defining multiple languages (adding section for single language doesn't work either). Fallback value is hardcoded as "/" for both, button and logo URLs.
Proposed solutions:

I did some quick check and found out that removing this in favor of
(cond (ne .Site.Params.landingPageURL nil) .Site.Params.landingPageURL "/")
works just fine and allows for landingPageURL to be defined in [params] section.
But that was very shallow checking (I didn't check this change for multiple languages etc.) so it requires further examination

If there is any reason to enforce multilngual mode requirement,
default fallback value could be changed from hardcoded  "/" to .Site.Params.baseURL.

I had this error probably due to lack of canonifyurls = true


Annotation labels: labels overlap at some zoom levels

The current browser displays all annotation labels at a certain zoom level, which the user can specify. At some zoom levels no labels are displayed at all and at some, there are overlapping labels. Ideally more labels could be displayed when the annotations are sparse and fewer could be displayed at high density, or the display of labels or annotations could be spread out vertically.

Some more feedback about the annotation features from Ben Hitz: in general the annotation features fade out too quickly and they should be displayed at more zoom levels. This might cause more overlapping issues. Would clustering the labels help? Or using a force layout??


Requirements.txt missing from pip package

/kind bug
pip install kubeflow-fairing
Collecting kubeflow-fairing
Downloading https://files.pythonhosted.org/packages/ab/7a/45aa727f59386aa0bccd1314c4c3837352eb04c43836c25871fbc372abca/kubeflow-fairing-0.7.2.tar.gz (98kB)
100% |████████████████████████████████| 102kB 913kB/s
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-build-HVs6FX/kubeflow-fairing/setup.py", line 3, in
with open('requirements.txt') as f:
IOError: [Errno 2] No such file or directory: 'requirements.txt'

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-HVs6FX/kubeflow-fairing/
Within the tar.gz the requirements.txt is missing and not packaged together for the pip release.

/assign
I going to add that.

@bomerzz which version you used? I tried that, the package can be installed.
(python36) [root@jinchi1 ~]# pip install kubeflow-fairing
Collecting kubeflow-fairing
  Downloading kubeflow_fairing-0.7.3-py3-none-any.whl (161 kB)
     |################################| 161 kB 824 kB/s
Requirement already satisfied: google-cloud-storage>=1.13.2 in /opt/python/python36/lib/python3.6/site-packages (from kubeflow-fairing) (1.26.0)

.....

Requirement already satisfied: pycparser in /opt/python/python36/lib/python3.6/site-packages (from cffi!=1.11.3,>=1.8->cryptography>=1.1.0->adal>=1.2.2->kfserving>=<IP_ADDRESS>->kubeflow-fairing) (2.19)
Installing collected packages: kubeflow-fairing
Successfully installed kubeflow-fairing-0.7.3

But yeah, we should add the requirements.txt, so that user can download tar source code from Pypi.

At the time of posting this issue, I was using version 0.7.2


In Godot, the assert will not execute after the game is exported as a release version.

Program execution statements should not be placed inside an alert; otherwise, the program will not execute in the release version.
Expected Behavior

The item images are displayed in the release version of the game.

Current Behavior

I found that the "item" images in my game disappeared in the release version, while other images displayed normally.
After debugging, I discovered that in the parse function of ItemProtoset, test_json_conv.data was not null in the debug version but was null in the release version.
Then, outputting test_json_conv.parse(json) == OK returned true, indicating that the JSON parsed correctly.

Possible Solution

Next, I changed the original code snippet at item_protoset.gd
assert(test_json_conv.parse(json) == OK, "Failed to parse JSON!")

to the revised code snippet:
var error = test_json_conv.parse(json)
assert(error == OK, "Failed to parse JSON!")

The game then displayed item images correctly in the release version.

I apologize for the typo; please consider "alert" as "assert".

Many inventory operations were also affected by this issue, and I manually replaced all assert statements with the revised operation mentioned above. The game is now running smoothly.

This is a very good point, I think there's a lot of bad assert usage in the code. I'll have to go through all of them and make corrections.

Fixed in https://github.com/peter-kish/gloot/pull/221


Set RUST_BACKTRACE by default

I think it will be a little easier for everyone if we set RUST_BACKTRACE by default for our executables.
This can be useful for regular GUI users (but only if we also use file logging and log crashes).

IMO the "rust_" part of the module name is redundant, I'd remove it.

It's helpful to keep it to recognize that this is something that's language specific, and not application specific.

IMO the "rust_" part of the module name is redundant, I'd remove it.

It's helpful to keep it to recognize that this is something that's language specific, and not application specific.

Well, by this logic we'll probably have to use this prefix for anything that is a wrapper for standard library stuff, which sounds like an overkill.

Well, by this logic we'll probably have to use this prefix for anything that is a wrapper for standard library stuff, which sounds like an overkill.

Not really. I understand why you think like that, but when looking at utils, it's then obvious that this is something different. It's not really a wrapper, it's a configuration of the compiler.
This is my opinion, and quite frankly doesn't matter that much.


Needs updating 1.21.1

I changed the crafting configuration and was able to get the knowledge book, but no mailbox. I don't know enough about datapacks to fix this myself.

Hi.
No.
This pack is discontinued as updating it to 1.20.5 would be a lot of work and we are also humans, with lives to live. We recently had to do this to Greenery, our latest pack, and it delayed it with about half a year.
This is because Mojang changed a major way in how items handle data, which meant we had to redo the whole of Greenery. We're not doing this for Mailboxes. I dearly apologize for any inconvenience.
Just a little notice, the pack version is on the pack page and Minecraft even says that it isn't for this version.
Closing this as invalid & as a wont fix.


Add a mapping property

Use PyMappingProxy to make dict and return
Add a property called mapping to dict.rs. Its return type -> PyMappingProxy

Please run cargo fmt --all 😊


Android and iOS have different landing pages and allows "Continue in your browser" for iOS

You can test with just the device emulation mode in desktop Chrome to change the user-agent.
https://matrix.to/#/#Wgpu:matrix.org

Android
iOS

On iOS, you can even "Continue in your browser" which isn't really supported in Element web. Mobile web is still listed as experimental and there are lots of bugs, https://github.com/vector-im/element-web/issues/1633. While I love mobile web over apps any time when I just want to check something out, this can lead to bad experiences like https://twitter.com/ChevyRay/status/1512211448277078019

Potential solutions
Show the iOS app icons like we do for Android. This appears to be a thing as pictured in https://github.com/matrix-org/matrix.to/issues/255

I tried this out the iOS flow in a fresh Xcode simulator instead of a browser emulation because I noticed we use more than just the user-agent to test whether a device is iOS.
It goes straight from "Continue" to https://app.element.io on mobile web iOS

Step 1
Step 2

We think this tweet is an example of a user confused by this


Multiple issues

Hi,
I tried to make this library work with my rails 5 project without success.
I think the first issue is the folder name in assets which are not named apitome_slate but apitome.
The second one prevents the app to start. No setup method on Apitome::Module. I removed the initializer in this engine and change config.layout = "apitome_slate/application" configuration in the initializer in my app instead.
The third issue is a template/css issue. It's looks broken.

Regards,

Closing for inactivity


internal/cmd: allow exec commands using gRPC server

This PR adds an option for TUI to run commands through a gRPC server.
First, start the gRPC server: ./runme server -a localhost:7890. Then pass the address of the server: ./runme tui --server localhost:7890.
It also introduces internal/runner/{local,remote} which are high-level abstractions to run commands either locally or using RPC.
Closes https://github.com/stateful/runme/issues/135

@sourishkrout
Ready for another review!

@sourishkrout
Fixed the segfault that's causing this, now the error message is:
failed to create remote runner: failed to create session: rpc error: code = Unimplemented desc = unknown service runme.runner.v1.RunnerService
exit status 1

Let me know if you're ok with that error message. I can probably make it nicer to handle this specific case, but it would be inconsistent throughout the codebase.

@sourishkrout
Fixed the segfault that's causing this, now the error message is:
failed to create remote runner: failed to create session: rpc error: code = Unimplemented desc = unknown service runme.runner.v1.RunnerService
exit status 1

Let me know if you're ok with that error message. I can probably make it nicer to handle this specific case, but it would be inconsistent throughout the codebase.

This works 👍 . Thank you!


The column of "similarity" meaning?

I want to know, how is the similarity between text and images calculated here? Is cosine similarity used? The cosine similarity calculated by me is inconsistent with the values stored in the data:
there is one data like this:
('blond woman wearing jewelry stock photo',
0.5808565020561218,
'http://media.istockphoto.com/photos/blond-woman-wearing-jewelry-picture-id157528193?k=6&m=157528193&s=612x612&w=0&h=oVM194R_VeLK5BOwIBkpI3n-NW30StYIQjuiMFgIyX8=')
but, i calculated  the similarity, it's not 0.5808565020561218:
def similarity_cos(a,b):
  return a.dot(b) / (np.linalg.norm(a) * np.linalg.norm(b))

i_e = get_image_emb("http://media.istockphoto.com/photos/blond-woman-wearing-jewelry-picture-id157528193?k=6&m=157528193&s=612x612&w=0&h=oVM194R_VeLK5BOwIBkpI3n-NW30StYIQjuiMFgIyX8=")
t_e1 = get_text_emb("blond woman wearing jewelry stock photo")
s1 = similarity_cos(i_e,t_e1)

the ans is 0.34096664
can you tell me the reason,please?

It's approximate similarity, computed by faiss
For some pairs it might be different than absolute value but overall keeping a similar ordering

It's approximate similarity, computed by faiss For some pairs it might be different than absolute value but overall keeping a similar ordering

thank you, i understand it.


Skip scheduling for empty cold creation.

Description
This is to skip scheduling of cold creation if there is no message.
It's actually harmless but keeps printing an error log.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Scheduler
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[x] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #5316 (dd1a9ce) into master (3354314) will decrease coverage by 18.16%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master    #5316       +/-   ##
===========================================
- Coverage   80.90%   62.74%   -18.17%
===========================================
  Files         239      239
  Lines       14241    14242        +1
  Branches      598      604        +6
===========================================
- Hits        11522     8936     -2586
- Misses       2719     5306     +2587

Impacted Files
Coverage Δ

...sk/core/scheduler/container/ContainerManager.scala
0.00% <0.00%> (-95.84%)
:arrow_down:

...scala/org/apache/openwhisk/common/time/Clock.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...che/openwhisk/core/invoker/LogStoreCollector.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...nwhisk/core/scheduler/queue/ContainerCounter.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...hisk/core/scheduler/message/ContainerMessage.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...core/database/cosmosdb/RxObservableImplicits.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedConsumer.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...core/scheduler/queue/SchedulingDecisionMaker.scala
0.00% <0.00%> (-97.60%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0.00% <0.00%> (-95.85%)
:arrow_down:

... and 78 more

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


resolve with tag should use the full reference

If I do a ko resolve with -t/--tags, we should be able to use the full reference in the resolve yaml files, aka something like gcr.io/tekton-releases/github.com/tektoncd/pipeline/cmd/kubeconfigwriter:v0.6.0@sha256:43050320e994156a2683ab5d30551a52a4a59fe20b0ee50161af5d524b9aa5c6 where both the tag and the digest is present.
Few potential questions:

what to do if there is multiple tags provided -t is an array flag :angel: ?
should it do it by default or should we have an option for it ?

/cc @jonjohnsonjr @bobcatfish @ImJasonH
Related issue https://github.com/tektoncd/pipeline/issues/1232

I so, so desperately hate this syntax and wish it would go away 😄

I so, so desperately hate this syntax and wish it would go away :smile:
Lol, I hear you :stuck_out_tongue_closed_eyes:

We could do it if and only if there is one, non-latest tag specified? It would be nice for releases...

That was kinda my thought (and the use case we have for tekton :angel: ).

Took a stab at it, LMK what you think.


display auto-merge in pr view

Describe what this PR does / why we need it
Displays "Auto-merge" section for PR if it was enabled, it's nice to see if I need to merge PR or it's already scheduled for auto-merge.

Does this pull request fix one issue?

NONE
Describe how to verify it
Apply auto-merge to blocked PR with gh pr merge --auto and see this section.
Checklist

[ ] Passing tests and linting standards
[ ] Documentation updates in README.md and doc/octo.txt

@macovsky thanks for yet another great contribution. If you want to become a collaborator of the project just let me know!


no such module 'SwiftyUserDefaults'

Hi,
Installed via Cocoapods and Podfile , (gem install cocoapods --pre Fetching: cocoapods-core 0.37.0.beta.1.gem (100%)) xCode Version 6.3 (6D570)
source 'https://github.com/CocoaPods/Specs.git'
platform :ios, '8.0'
use_frameworks!
pod 'SwiftHTTP', '~> 0.9.4'
pod 'JSONJoy-Swift', '~> 0.9.2'
pod 'SwiftyUserDefaults'
SwiftHTTP and other is working , when i add import SwiftyUserDefaults that time gives no such module SwiftyUserDefaults
Thank you

I appologize for not replying before. I couldn't reproduce your problem.
Could you try it again with CocoaPods 0.37.1?

I solved it with cocoapods 0.37 also i forgot to post here, thank you

Great, glad to hear that!


Server crash

] Saved data
00:06:37: ACTION[main]: IRC: Disconnected.
00:06:37: ACTION[main]: [interact] Blacklist saved
00:06:37: ACTION[main]: [mana] Server shuts down. Rescuing data into mana.mt
00:06:37: ACTION[main]: [mana] Wrote mana data into /home/quentinbd/mff/worlds/minetestforfun//mana.mt.
00:06:37: ACTION[main]: compassgps writing settings
00:06:37: ACTION[main]: Wrote quests to file
00:06:37: ACTION[main]: [PClasses] Saved data
00:06:37: ACTION[main]: [tsm_pyramids] Chests saved
00:06:37: ACTION[main]: [fishing] Server shuts down. saving trophies table
00:06:37: ERROR[main]: ERROR: An unhandled exception occurred: ...home/quentinbd/mff/bin/../mods/tsm_pyramids/room.lua:62: bad argument #1 to 'ipairs' (table expected, got nil)
00:06:37: ERROR[main]: stack traceback:
00:06:37: ERROR[main]: 	[C]: in function 'ipairs'
00:06:37: ERROR[main]: 	...home/quentinbd/mff/bin/../mods/tsm_pyramids/room.lua:62: in function '?'
00:06:37: ERROR[main]: 	/home/quentinbd/mff/bin/../builtin/game/register.lua:341: in function </home/quentinbd/mff/bin/../builtin/game/register.lua:329>

In thread 7fe801165740:
/home/quentinbd/minetest/src/main.cpp:882: int main(int, char**): Assertion '0' failed.
Debug stacks:
DEBUG STACK FOR THREAD 7fe7f93c8700:
#0  virtual void* CurlFetchThread::Thread()
DEBUG STACK FOR THREAD 7fe801165740:
#0  int main(int, char**)
(Leftover data: #1  Dedicated server branch)
(Leftover data: #2  virtual void ServerMap::save(ModifiedState))
(Leftover data: #3  void Server::step(float))
/home/quentinbd/script/start-mff.sh : ligne 26 :  4553 Abandon                 /home/quentinbd/mff/bin/minetestserver --world /home/quentinbd/mff/worlds/minetestforfun/ --config /home/quentinbd/mff/minetest.conf --gameid minetestforfun_game --port 30001
----------------------
Server restarted at jeudi 6 août 2015, 00:07:07 (UTC+0200)
----------------------

I seriously do not know what happened, but there you have it.
https://github.com/MinetestForFun/minetest-minetestforfun-server/commit/21d9cc420396d57d59603220aba193936342ac52

It's old fixed crash.
https://github.com/MinetestForFun/minetest-minetestforfun-server/commit/25a876e788226bbabf47623a910619f0b357ca88


detectJsStringBuiltins() returns incorrect result

The function detectJsStringBuiltins in https://github.com/dart-lang/sdk/blob/3bbcf1be963a93f716828399c39c62cf3b3e64ba/pkg/dart2wasm/lib/js/runtime_blob.dart#L7 is missing a boolean inversion of its result: those module bytes will fail to validate when the Wasm engine implements support for String Builtins (because the opt-in causes type checks to be applied to imports that in classic Wasm would be allowed to have any type they want), so the function should return !WebAssembly.validate(...) (note the !).
To verify, you can copy-paste that function into the DevTools console of any recent-ish Chrome build: it should return true. E.g.:
> let bytes = [
    0,   97,  115, 109, 1,   0,   0,  0,   1,   4,   1,   96,  0,
    0,   2,   23,  1,   14,  119, 97, 115, 109, 58,  106, 115, 45,
    115, 116, 114, 105, 110, 103, 4,  99,  97,  115, 116, 0,   0
  ];
< undefined
> WebAssembly.validate(new Uint8Array(bytes))
< true
> WebAssembly.validate(new Uint8Array(bytes), {builtins: ["js-string"]})
< false

Looking at how the result of that feature detection is used in compileStreaming and compile a couple of lines further down in the same file, detectJsStringBuiltins() could also be dropped entirely: it never hurts to request {builtins: ['js-string']} in compile/compileStreaming, if the feature isn't present then that parameter will just silently be ignored.
(I noticed this when looking at https://github.com/mkustermann/wasm_gc_benchmarks/blob/main/micro-benchmarks-out/non_devirtualized_typed_data_access.dart2wasm.mjs.)

CC @mkustermann

That's extremely helpful, thank you @jakobkummerow ! (We're soon going to re-evaluate what perf/size would look like when switching to JS strings, so we'd get the wrong conclusion without this fix)

It seems the extra parameter to e.g. WebAssembly.compileStreaming isn't in docs, e.g. here. @jakobkummerow where is the official documentation and how does one know that calling with more arguments won't cause issues?

The compileOptions parameter is new, so most existing documentation still needs to be updated.
It's a general property of JavaScript and Web API functions that you can pass additional parameters to function calls, and they'll just get ignored: Math.sqrt(4, "ignore me") still just returns 2.
If you did have reason to be worried about passing the extra parameter to .compile, then the same worry would have to apply to .validate, where the existing code already demonstrates that it's fine 🙂
Wasm's JS String Builtins were specifically designed to be backwards-compatible: specifying the extra parameter doesn't bother old implementations. And the extra parameter is an object so that it'll be easy to extend it further in the future when the need arises. E.g. perhaps some day we'll have W.compile(..., {builtins: ["js-string", "some other builtin"], other_option: [other_value, maybe_more]}), and again old implementations will naturally ignore what they don't understand.


Unnecessary numpy version restriction

Currently the numpy version is restricted to be <1.14 as h5py was incompatible with it.
This has been fixed in h5py>=2.8: Release Notes / github issue

This is going to be fixed in https://github.com/QCoDeS/Qcodes/pull/1120 PR.

Fixed


support engagement create direct url to support comment period creation in epic

To create comment period from EPIC , we should provide a direct url to engagement create page which supports a project ID in the query param
https://epic-engage-web-dev.apps.gold.devops.gov.bc.ca/eao/engagement/projectId={id} should take to an engagement create page and when the engagement is submitted , the project id should get assosicated
When clicking on the create comment period button from epic , user wont be logged in , so we should add a login check for this page and redirect them to IDIR login

Verified using the url like : https://epic-engage-web-test.apps.gold.devops.gov.bc.ca/engagements/create/form/?projectId=AB5546451


store appium log to file appium.log

-store appium log to appium.log for tracing error

@nurdians Could you add cleaner before starting the server, to make sure the log file is clean every run ..

@nurdians Could you add cleaner before starting the server, to make sure the log file is clean every run ..

@aldochristiaan The file is always re-created and replace existing file when the server starts (at line 82)


[Feature]: Allow Appsmith devs to expose utility functions in the global context for the client editor

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Hello,
I want to provide a function that can be accessed from any input widget through the widget properties panel. I would like the function to appear in the context list of available functions/objects accessed through mustache syntax:

I was able to expose a function as an action in this list, but it cannot be invoked:

Note: I am aware of JSObjects but do not want to create one. This function needs to be available for all end-users of the appsmith framework we provide. I also do not want to expose the function through the JSObject tab for modification.
Why should this be worked on?
In general, I believe it can be useful for appsmith devs to provide global functions to appsmith end-users.
Specifically, I am trying to integrate i18n into the appsmith we plan to make available to users. I've seen an example of this working for an individual page, but want to provide a solution for all pages without exposing the implementation details through JSObjects.

@jtaylor3rd global functions can be provided via our packages feature on the business plan


[Feat] - Removed zh-tw locale

Description:
This PR removes zh-tw locale from docusaurus config

Hi  @adithyaakrishna
Thank you for the PR.
After negotiating with the maintainers, I think we can keep the zh-tw version on the new docs. Because even if we don't translate the content to zh-tw, the users still can see English content, which won't bother the readers.  So I'm going to merge this PR.
Sorry for the misunderstanding.


Cypress/E2E: Add E2E eol-last lint rule

Summary

Enforce "eol-last": ["error", "always"] to e2e folder and fix failures when npm run check

Ticket Link
NA

@deanwhillier @hmhealey This EOL lint rule is applied to E2E folder in this PR. However, could this be applied to the entire webapp codebase too?

I went ahead and applied the rule to the whole codebase.


RP2040 I2C_RESET

I have been working on adding RP2040 hardware support to PX4-Autopilot which uses Nuttx as the RTOS. The only thing missing in this PR right now is the I2C_RESET.
The datasheet of RP2040 states that the MCU has Bus Clear Feature (Page 478) which can handle the problem of SDA stuck to LOW. Thus, it makes sense that the rp2040_i2c_reset() function in rp2040_i2c.c file just returns OK since the hardware is taking care of the problem automatically, I presume.
I would like to know if I am thinking in the right direction or not. If yes, then I think the rp2040_i2c_reset() should rather return the state of the bus, that is whether it is stuck or not. This way, the i2c implementation of PX4, or any other app for that matter, will know whether the bus clearing attempt was successful or not. Then select ARCH_HAVE_I2CRESET could be added to the Kconfig file to allow defining CONFIG_I2C_RESET. It is important to define CONFIG_I2C_RESET since a some parts of i2c implementation in PX4-Autopilot depends on this.

@yunkya2 do you have any suggestion?

What is unclear to me is if 4.3.13. Bus Clear Feature, can be requested via a command register. If it can be, that is what should be defined as the reset function. If not, since the CONFIG_I2C_RESET interface is optional and in this case not needed. What about not having the CONFIG_I2C_RESET option in the Kconfig?

@davids5 I also had the same question regarding how to initiate the bus clear process. Surprisingly, the datasheet does not provide any explanation on how to do that nor any of the I2C registers have anything related bus clear to my understanding. I am presuming that the MCU somehow detects that the SDA line is stuck to low and automatically initiates the Bus Clear process.

@vxj9800 - you could try to ask Synopsis or look as some other i2c drivers like xtensa, esp that have something similar but I have no clue if it is the same IP.

ESP32-C3 has a hardware reset option.  There are a bunch of registers to write to in order to initiate the procedure.
https://github.com/apache/incubator-nuttx/blob/master/arch/risc-v/src/esp32c3/esp32c3_i2c.c#L1138-L1160

I have developed the driver by referring the cxd56's implementation because I found that both SoC have same I2C IP.
As @vxj9800 points out, there are no further descriptions about the Bus Clear Feature in the datasheet and I have no information how this feature works.
@Ouss4 ESP32_C3 seems to have a different I2C IP because the register defintions in hardware/esp32c3_i2c.h is different to RP2040's one.

Yes, I'd actually be surprised if they had the same IP, I just wanted to mention that this feature exist on ESP32-C3.
In any case, if there are no information for the RPi it can be done in software like most of the other chips.

I have incorporated the bus recovery sequence like another I2C_RESET implementation and issued MR #4786.
The code sequence seems to be working as expected, but I cannot confirm whether  the sequence recovers from the bus stuck condition because it is difficult to reproduce.

An update on the existence of I2C Bus Clear feature on the hardware side. It is discussed in the documentation of the RP2040 but is not active in the silicon as per the discussion on the pi pico forum and as per the issue raised on github.

@vxj9800 Thank you for the interesting information. I understood that our only choice is the software solution like #4786.


Is the key added to SSH agent?

The page does not give details regarding how is the private key actually deployed on the instance running the SSH Key task.
Is it made available via ssh-agent? (which would be invaluably useful for agent forwarding in subsequent steps)

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

ID: 9ce9d017-fc43-c904-d94b-87cf91265ffc
Version Independent ID: ecf3b0b7-6c2c-1cd9-33a9-2cf562932e2b
Content: Install SSH Key task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/install-ssh-key.md
Product: devops
Technology: devops-cicd
GitHub Login: @DavidStaheli
Microsoft Alias: dastahel

@Adnn  Thank you for the question.
This topic is currently under discussion in https://github.com/MicrosoftDocs/vsts-docs/issues/5327.
Please refer there for more information on how keys are added and for pending updates to the documentation.

Thanks for your question. It looks like your question isn't an issue with the documentation, but an issue or question specific to your implementation or a bug in the product itself.
For help with a specific task, here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot

If you've encountered an issue with the product itself, to make sure it gets in front of the right people, please submit your bug here.
If you have an idea for improving the product, please submit your idea here.

@dermaaarkus Apologies for the delay.
One of the items listed in the referenced issue states sshKeySecureFile - this is a secret/file, how to securely reference this? which is along the same lines.
Please comment there if you'd like additional information to be added.


Add some documentation to the README.md

Thanks, @willingc! I have addressed your comments.

This is great!


Location of cached dat file

The cached file is downloaded to the current or active folder of the user. This can be problematic.
Can you put it in the users %Temp% folder? (Path.GetTempPath)

We use it to get the domain name and the tld from the url but I'm not sure this is relevant.
It seems to me the user Current or Working folder is not constant so the cache file might end up in multiple places.
The cache is by it very nature a temporary file and should be in a constant place, ie. the users temp folder.
Just my thoughts.

I have add the temp path folder logic to the current code

A new nuget package is available now


Replace packageInfo.version with packageInfo.versionCode to detect app update

Currently in the WhatsNewHelper class, we're checking previous packageInfo.version that saved in prefes with new installed packageInfo.version . If users changes the pubspec.yaml version only for name and code then the code gets changed on both native android & ios side  but the previous version remains the same. So, we need to replace this pacakgeInfo.version with packageInfo.versionCode so if user only changes the version in the pubspec file from version: 1.0.6+8 to version: 1.0.7+9 , we can detect the new install

@riadshuvo please check this one bro


chore: Oink

Added a very important comment

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 8bdb9213 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
5,668
18,820
36,356

l1_rollup_calldata_gas
66,268
239,080
469,676

l1_rollup_execution_gas
666,949
953,990
1,337,033

l2_block_processing_time_in_ms
1,320 (+1%)
4,960 (+3%)
9,000 (-6%)

note_successful_decrypting_time_in_ms
150 (+6%)
540 (+13%)
1,036 (+5%)

note_trial_decrypting_time_in_ms
56.2 (+22%)
56.3 (-43%)
63.0 (-47%)

l2_block_building_time_in_ms
18,374 (-1%)
69,488
138,202

l2_block_rollup_simulation_time_in_ms
8,267 (-1%)
29,246
56,907 (-1%)

l2_block_public_tx_process_time_in_ms
10,084 (-1%)
40,186
81,194 (+1%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,612 (+3%)
27,175 (-2%)

note_history_successful_decrypting_time_in_ms
1,255 (-1%)
2,504 (+2%)

note_history_trial_decrypting_time_in_ms
96.9 (+1%)
201 (+35%)

node_database_size_in_bytes
19,079,248
35,655,760

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
283 (-1%)
44,370
27,700

private-kernel-ordering
217
52,324
14,326

base-parity
1,845 (+1%)
128
311

base-rollup
729
177,083
925

root-parity
1,607 (-1%)
1,244
311

root-rollup
68.6
4,487
789

private-kernel-inner
636 (-1%)
73,229
27,700

public-kernel-app-logic
452
35,262
28,215

public-kernel-tail
178 (+1%)
40,926
28,215

merge-rollup
8.31 (+1%)
2,696
925

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1 (+1%)
16.1 (+1%)
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.586
0.496
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
46.2 (+1%)
72.2
233 (+1%)
446 (+1%)
867 (-1%)
1,729
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.474 (+1%)
0.446
0.426 (+1%)
0.417 (+1%)
0.413 (-1%)
0.413
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.4 (+2%)
107 (+1%)
339 (+1%)
662 (+1%)
1,319 (+1%)
2,630 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
104
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.483 (+1%)
0.481
0.461 (+1%)
0.455
0.456 (+1%)
0.458 (+1%)
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.3

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.536

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes

tx_size_in_bytes
22,014

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes

tx_pxe_processing_time_ms
3,257
1,741

Metric
0 public data writes
1 public data writes

tx_sequencer_processing_time_ms
12.3 (-6%)
1,264


[ADD ME TO THE ORGANIZATION]

Name: Sandeep Kumar
Location: Hyderabad
Interest: python
Discord username(optional):- SANDEEP KUMAR#7546
Why do you want to join?

I want to join because

You can also join our Discord community here
Feel free to check out other cool repositories of the Open Source Community here.

Hello there!👋, Welcome to OpenSourceHub! 🚀⚡Thank you and Congratulations🎉 for opening a issue in this project. Please make sure not to start working on this issue, unless you are assigned to it.😄

i did't get any invite email

@sandeepvinnu Thanks for your interest. Invitations have been sent to you. Kindly accept it and make your membership public.


Descriptors.NPR1 generates different value after #7535

Describe the bug
Generating Descriptors.NPR1 on a fixed seed conformer of ethane changed the value after merge of #7535.
To Reproduce
    from rdkit import Chem
    from rdkit.Chem import AllChem, Descriptors3D

    m2 = Chem.AddHs(Chem.MolFromSmiles("CC"))
    AllChem.EmbedMolecule(m2, randomSeed=1)
    assert round(Descriptors3D.NPR1(m2), 10) == 0.2553516286

Taken from what rdkit-pypi used to do:
https://github.com/kuelumbus/rdkit-pypi/pull/108/files#diff-41d180f6f7233d175c2dfe36c45c4ea80eed4754fa7ed4eed46ccde1b2a778c7R11-R21
The value generated now is 0.2532392741.
Expected behavior
@kuelumbus changed the test -- but I'd like to understand what exactly changed about the conformer generation. Is it that the merged PR totally changed what the random seeds will end up generating?
The values did match on Release_2024_03_4.
Configuration (please complete the following information):

RDKit version: 2024.3.4, 2024.3.5, and git bisect between them
OS: [e.g. Ubuntu 20.04]: Ubuntu 20.04
Python version (if relevant): 3.11
Are you using conda? miniconda, but only for installation of dependencies
If you are using conda, which channel did you install the rdkit from? N/A
If you are not using conda: how did you install the RDKit? make; cmake

ref: https://github.com/kuelumbus/rdkit-pypi/pull/108

Great, thank you for checking and clarifying.


Enhance an ABAP Core Data Services (CDS) View in SAP Business Technology Platform (BTP) ABAP Environment

Tutorials: https://developers.sap.com/tutorials/abap-environment-enhance-cds-view.html
Write here how you think we can improve the tutorial ...
I noticed that the code snippet from Step 4.4, is not present in Step 5 where we are supposed to check our code.

I think that including it there would help learners ensure consistency and accuracy when reviewing the final result.

Hi Fransisco,
You're absolutely right. Many thanks for all this feedback. I have incorporated the change. It should be visible soon.
Best wishes,
Julie.


sb1-ba39go

Describe the bug
I'm importing your StackBlitz project into Bolt. This may take a moment as I set everything up. Once it's ready, you'll be able to explore and interact with your code.
Importing Stackblitz Project
sb1-ba39go
Unexpected token 'U', "Unexpected"... is not valid JSON
An unknown error occured while running an action.
i try to backup with save , now i lost my project
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-ba39go
Steps to reproduce
sb1-ba39go
Unexpected token 'U', "Unexpected"... is not valid JSON
An unknown error occured while running an action.
Expected behavior
sb1-ba39go
Unexpected token 'U', "Unexpected"... is not valid JSON
An unknown error occured while running an action.
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 53512828,
  "usedJSHeapSize": 48836156,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = Extra330
Chat ID = 2a032cd71e02

bolt.new
Additional context
Error:
An unknown error occured while running an action.

@extra330ffefz Thanks for the report and apologies for the frustrating experience! The fix is explained in #1471