Mobile Features

Codecov Report

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

@@           Coverage Diff           @@
##           master     #194   +/-   ##
=======================================
  Coverage   68.61%   68.61%
=======================================
  Files          27       27
  Lines        2141     2141
  Branches      262      262
=======================================
  Hits         1469     1469
  Misses        579      579
  Partials       93       93


Custom renderers receiving no children as params

Is this a bug report or a feature request?
Bug report
Have you read the guidelines regarding bug report?
Yes, I have
Have you read the documentation in its entirety?
Yes, I have
Have you made sure that your issue hasn't already been reported/solved?
Yes, I have
Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
Both platforms
Is the bug reproductible in a production environment (not a debug one)?
Yes, it is
Have you been able to reproduce the bug in the provided example?
Yes, I have
Environment
    React: 16.8.6
    React native: 0.60.0
    react-native-render-html: 4.2.0 (master)

Target Platform:
(They really don't matter, it's a js issue)
Android API 28 (9.0)
iOS (13.3)
Steps to Reproduce

Try to use custom renderers as the documentation states
Provide the parameter for the custom renderer
Log the parameter to the console in order to see what it contains
e.g.

// Copied from the demo project
const CUSTOM_RENDERERS = {
  p: params => {
    console.log(JSON.stringify(params))
    return <Text>{params.children[0]}</Text> // This should be working, but params.children is undefined, thus it will throw an error
  },
}

Expected Behavior
The fields inside params should exist
Actual Behavior

As you can see, the children field is nowhere to be found. The snippets used are the ones provided in the demo. I should see the text nodes as children. I don't even see the strong nodes when I insert them inside the p element
Reproducible Demo
Inside your demo, paste my custom renderer

@nikola-kolosek It is hard to help you since you don't provide the exact html extract that you pass to the HTML component which result in the behavior you are describing.

I'm closing now, feel free to comment if you are still facing this issue.

@nikola-kolosek Looking again at your example, it seems that you are misunderstanding the custom renderer API. This first argument is a node attributes argument, and does not contain any children property. You're looking for the second argument provided to the renderer function.


coreos-kubernetes vagrant up fails with ssl error

[single-node]# vagrant version
Installed Version: 1.8.6
Latest Version: 1.8.6
You're running an up-to-date version of Vagrant!
/opt/vagrant/embedded/bin/openssl: error while loading shared libraries: libssl.so.1.0.0: cannot open shared object file: No such file or directory
failed generating SSL CA artifacts
[single-node]# vagrant up
/opt/vagrant/embedded/bin/openssl: error while loading shared libraries: libssl.so.1.0.0: cannot open shared object file: No such file or directory
failed generating SSL CA artifacts
openssl version:
[single-node]# openssl version
OpenSSL 1.0.1e-fips 11 Feb 2013
Similar to the issue: https://github.com/coreos/coreos-kubernetes/issues/635
Thanks.

Also having this issue on Vagrant 1.8.5 and Fedora 24.

Me too!,
My environment is:

Fedora 24 (Full Updated)
Vagrant 1.8.6

Details:
$ > uname -a
Linux xxxxxx 4.7.9-200.fc24.x86_64 #1 SMP Thu Oct 20 14:26:16 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

$ > vagrant version
Installed Version: 1.8.6
Latest Version: 1.8.6

$ > openssl version
OpenSSL 1.0.2j-fips  26 Sep 2016

$ > openssl version -d
OPENSSLDIR: "/etc/pki/tls"

$ > virtualbox --help
Oracle VM VirtualBox Manager 5.1.4
(C) 2005-2016 Oracle Corporation
All rights reserved.

same problem here, any updates on that issue?
date
Di 15. Nov 12:24:16 CET 2016
vagrant box update
/opt/vagrant/embedded/bin/openssl: error while loading shared libraries: libssl.so.1.0.0: cannot open shared object file: No such file or directory
openssl version
OpenSSL 1.0.1e-fips 11 Feb 2013

bumt too.
1.8.5 Vagrant and Fedora 24

Thanks @Mangirdaz! That's solved it for me as well.

Thanks @Mangirdaz ! It also works for mac

Thanks @Mangirdaz. I also use Mac

great it worked !

Thanks. @mangirdaz
Need adjust openssl source path on mac.
sudo mv /opt/vagrant/embedded/bin/openssl /opt/vagrant/embedded/bin/openssl.orig
sudo ln -sf /usr/bin/openssl /opt/vagrant/embedded/bin/openssl

The same thing that SydOps did on OS X worked on Ubuntu 16.04

Same issue on 16.04. Followed instruction of SydOps as well.

Thanks @SydOps, the fix worked for me too:
$vagrant --version
Vagrant 1.9.3

$sw_vers
ProductName:	Mac OS X
ProductVersion:	10.12.5
BuildVersion:	16F73

Same issue; Arch-Linux with latest-updates installed, Vagrant 2.0.0-2 .
Thanks @ozbillwang , fix worked.


Problems migrating from Identity Server 3 to Identity Server 4 (4.0.4) - Broken introspect endpoint?

I couldn't find a documentation with all the breaking changes, nor any newly added features.
I understood that in the old version, when hitting the token endpoint (to get access/refresh tokens), you could supply the key "scope" with the list of scopes, or supply it as "scopes" and both worked, whereas in the new version only "scopes" now works.
Now I'm stuck at the introspect endpoint. When I hit it, it always returns "isActive": "false" as a response.
I've debugged and it correctly reads the token from the database, but why it returns that 24/7 of the time (if my data is correct) - I don't know. It should return data for the token,as it's not expired.
The token read from the database has all the fields correctly assigned, I even checked if its caused due to an expiration in the date we're saving, but it's not that. Its type is correctly set to "reference_token", the only fields that have no value/are null, are the newly added in PersistedGrant object properties - Description, SessionId, ConsumedTime.
I'm using the resource owner password grant.
My configuration features a simple ApiResource declared with a secret, as well as a list of Client objects configured to my liking.
Example client configuration:

                Enabled = true,
                ClientId = "client",
                ClientSecrets = "secret"
                AllowedGrantTypes = GrantTypes.ResourceOwnerPassword,
                AllowOfflineAccess = true,
                AccessTokenType = AccessTokenType.Reference,
                RequireConsent = false,
                RequirePkce = false,
                UpdateAccessTokenClaimsOnRefresh = true,
                RefreshTokenExpiration = TokenExpiration.Absolute,
                AbsoluteRefreshTokenLifetime = 123456,
                RefreshTokenUsage = TokenUsage.ReUse,
                AccessTokenLifetime = 600000,
                AllowedScopes = { "helloAPI", StandardScopes.OfflineAccess },

The endpoint is being tested by Postman (not API)
EDIT: Just for info - when using JWT instead of reference tokens, everything works correctly.

Not sure. I just tested it with our RO Console sample (I changed it to use reference tokens), and here's the result from introspection:
{"iss":"https://localhost:5001","nbf":1597242427,"exp":1597246027,"aud":["resource1","resource2"],"client_id":"roclient","sub":"88421113","auth_time":1597242427,"idp":"local","amr":"pwd","name":"Bob Smith","email":"BobSmith@email.com","jti":"4A59C1B22EEC41AAE329D473BD25D8B7","iat":1597242427,"active":true,"scope":"resource1.scope1"}

So, I'd suggest try to repro with our samples and then match what's different in your solution.

Not sure. I just tested it with our RO Console sample (I changed it to use reference tokens), and here's the result from introspection:
{
"iss":"https://localhost:5001",
"nbf":1597242427,
"exp":1597246027,
"aud":["resource1","resource2"],
"client_id":"roclient",
"sub":"88421113",
"auth_time":1597242427,
"idp":"local",
"amr":"pwd",
"name":"Bob Smith",
<EMAIL_ADDRESS>"jti":"4A59C1B22EEC41AAE329D473BD25D8B7",
"iat":1597242427,
"active":true,
"scope":"resource1.scope1"
}

Notice it's active, not isActive.
So, I'd suggest try to repro with our samples and then match what's different in your solution.

I found the issue and fixed it - I had to define a scope in the Scopes property of an ApiResource. This was only mandatory for reference tokens, which baffles me.
On another note, could you please help clear out the difference between an identity resource and an api scope?
From what I've gathered, both serve the same purpose, i.e identity resources are scopes, in which you can define and group a lot of claims. What's the distinguishing aspect of an ApiScope then? Do we even need them in the equation?

There also seems to be some weird behavior that doesn't make much sense.
If I have defined an ApiResource with a list of ApIScope's it needs to access, why do I have to wrap it intentionally in a ApiScope with the scope name being the ApiResource's name, and scope and display name the same as ApiResource's?
Why didn't it work as it did before, by simply supplying the ApiResource's name in the scope parameter?
Are we even using the name of an ApIResource right now, seeing that the previous functionality no longer works?

could you please help clear out the difference between an identity resource and an api scope?

Here are our docs: https://identityserver4.readthedocs.io/en/latest/topics/resources.html

Thanks! It seems that an identity resource is a thing tied specifically to identity tokens. Nowhere in the page you linked is that ever clearly stated, hence the major confusion.
What about the above-mentioned problems with wrapping a whole ApiResource's Allowed scopes in an ApiScope?

Nowhere in the page you linked is that ever mentioned, hence the major confusion.

Yea, that's something we generally expect people to know from the protocols.

No worries, we cleared that out of the equation. The elephant in the room right now is the weird wrapping. Perhaps @leastprivilege can answer that?

I don't know/follow what you're trying to do with ApiResources. I'd suggest not using them at all unless you have a protocol reason to do so.

Okay, let's define the following Configuration:
public static IEnumerable<ApiScope> GetApiScopes() =>
	new List<ApiScope>
	{
		new ApiScope(
			name: "Scope1",
			displayName: "scope1 description",
			userClaims: new[] { "claim1" }),
		new ApiScope(
			name: "Scope2",
			displayName: "scope2 description",
			userClaims: new[] { "claim2", "claim3", "claim4"}),
		new ApiScope(
			name: "Scope3",
			displayName: "scope3 description",
			userClaims: new[] { "claim5" }),
		new ApiScope(
			name: "Scope4",
			displayName: "scope4 description",
			userClaims: new[] { "claim6" })
	};

public static IEnumerable<ApiResource> GetApiResources() =>
	new List<ApiResource>
	{
		new ApiResource("MyApi", "MyApi description")
		{
			ApiSecrets = { new Secret("secret").Sha256() },
			Scopes =
			{
				"Scope1",
				"Scope2",
				"Scope3",
				"Scope4"
			}
		}
	};

public static IEnumerable<Client> GetClients() =>
	new List<Client>
	{
		new Client
		{
				Enabled = true,
                ClientId = "client",
                ClientSecrets = "secret"
                AllowedGrantTypes = GrantTypes.ResourceOwnerPassword,
                AllowOfflineAccess = true,
                AccessTokenType = AccessTokenType.Reference,
                RequireConsent = false,
                RequirePkce = false,
                UpdateAccessTokenClaimsOnRefresh = true,
                RefreshTokenExpiration = TokenExpiration.Absolute,
                AbsoluteRefreshTokenLifetime = 123456,
                RefreshTokenUsage = TokenUsage.ReUse,
                AccessTokenLifetime = 600000,
                AllowedScopes = { "MyApi" }, // This previously worked, now it doesn't
		}
	};

Previously this would've worked. When you require the identity provider to issue an access token for the specific resource, they'd just supply the name of the ApiResource in the scope key-value pair of the request (openid profile MyApi)
Now, it doesn't. You need to wrap all of the Scopes present in the ApiResource in another ApiScope and then pass that ApiScope to the ApiResource in place of all those scopes.
For better clarity, here is a working variant in Identity Server 4.0.3:
public static IEnumerable<ApiScope> GetApiScopes() =>
	new List<ApiScope>
	{
	    new ApiScope(
			name: "Scope1",
			displayName: "scope1 description",
			userClaims: new[] { "claim1" }),
		new ApiScope(
			name: "Scope2",
			displayName: "scope2 description",
			userClaims: new[] { "claim2", "claim3", "claim4"}),
		new ApiScope(
			name: "Scope3",
			displayName: "scope3 description",
			userClaims: new[] { "claim5" }),
		new ApiScope(
			name: "Scope4",
			displayName: "scope4 description",
			userClaims: new[] { "claim6" }),
		// Wrapper
		new ApiScope(
			name: "MyApi",
			displayName: "",
			// Manually add all claims from above scopes. If you end up in the future changing one of the above scopes's required claims, well, make sure you do the same here...
			userClaims: new[] { "claim1", "claim2", "claim3", "claim4", "claim5", "claim6"})
	};

public static IEnumerable<ApiResource> GetApiResources() =>
	new List<ApiResource>
	{
		new ApiResource("MyApi", "MyApi description")
		{
			ApiSecrets = { new Secret("secret").Sha256() },
			Scopes =
			{
				"MyApi"
			}
		}
	};

public static IEnumerable<Client> GetClients() =>
	new List<Client>
	{
		new Client
		{
				Enabled = true,
                ClientId = "client",
                ClientSecrets = "secret"
                AllowedGrantTypes = GrantTypes.ResourceOwnerPassword,
                AllowOfflineAccess = true,
                AccessTokenType = AccessTokenType.Reference,
                RequireConsent = false,
                RequirePkce = false,
                UpdateAccessTokenClaimsOnRefresh = true,
                RefreshTokenExpiration = TokenExpiration.Absolute,
                AbsoluteRefreshTokenLifetime = 123456,
                RefreshTokenUsage = TokenUsage.ReUse,
                AccessTokenLifetime = 600000,
                AllowedScopes = { "MyApi" }, // now works because we have a fake "MyApi" scope, encapsulating our previously well-defined structure of scopes
		}
	};

In your first config above, MyApi is not a scope. This is some more context for the changes we made:
https://leastprivilege.com/2020/06/18/resource-access-in-identityserver4-v4-and-going-forward/

I've read it, but I still don't understand the correct way of doing what we previously could do in identity server 3 (first config). It definitely can't be like in my second configuration. Could you share an example of a config showing the right way to grant a client access to a specific api resource?
That'd be super great!

You want to reject requests for scopes unless all scopes that are configured for that ApiResource are requested?

Yes, I think I figured it out:
Use OnTokenValidated and simply reject the token if they don't answer to the required claims. Let me know if this was the correct way to go about it.


ExponentialMatrixCoefficient class

This PR introduces the ExponentialMatrixCoefficient class as well as DenseMatrix::Exponential().
This will be used in a forthcoming MFEM example for the Monge–Ampère equation.
NOTE: Only 2D is supported at this time. 3D support is planned for a future PR.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#4231
@brendankeith
@tzanio
@bslazarov + @psocratis
4/17/24
5/26/24
⌛due 5/8/24

PR Checklist

[ ] Code builds.
[ ] Code passes make style.
[ ] Update CHANGELOG:

[ ] Is this a new feature users need to be aware of? New or updated example or miniapp?
[ ] Does it make sense to create a new section in the CHANGELOG to group with other related features?

[ ] Update INSTALL:

[ ] Had a new optional library been added? If so, what range of versions of this library are required? (Make sure the external library is compatible with our BSD license, e.g. it is not licensed under GPL!)
[ ] Have the version ranges for any required or optional libraries changed?
[ ] Does make or cmake have a new target?
[ ] Did the requirements or the installation process change? (rare)

[ ] Update continuous integration server configurations if necessary (e.g. with new version requirements for each of MFEM's dependencies)

[ ] .github
[ ] .appveyor.yml

[ ] Update .gitignore:

[ ] Check if make distclean; git status shows any files that were generated from the source by the project (not an IDE) but we don't want to track in the repository.
[ ] Add new patterns (just for the new files above) and re-run the above test.

[ ] New examples:

[ ] All sample runs at the top of the example source file work.
[ ] Update examples/makefile:

[ ] Add the example code to the appropriate SEQ_EXAMPLES and PAR_EXAMPLES variables.
[ ] Add any files generated by it to the clean target.
[ ] Add the example binary and any files generated by it to the top-level .gitignore file.

[ ] Update examples/CMakeLists.txt:

[ ] Add the example code to the ALL_EXE_SRCS variable.
[ ] Make sure THIS_TEST_OPTIONS is set correctly for the new example.

[ ] List the new example in doc/CodeDocumentation.dox.
[ ] If new examples directory (e.g.examples/pumi), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add example-specific documentation, see e.g. the src/examples.md.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] In examples.md, list the example under the appropriate categories, add new categories if necessary.
[ ] Add a short description of the example in the "Extensive Examples" section of features.md.

[ ] New miniapps:

[ ] All sample runs at the top of the miniapp source file work.
[ ] Update top-level makefile and makefile in corresponding miniapp directory.
[ ] Add the miniapp binary and any files generated by it to the top-level .gitignore file.
[ ] Update CMake build system:

[ ] Update the CMakeLists.txt file in the miniapps directory, if the new miniapp is in a new directory.
[ ] Add/update the CMakeLists.txt file in the new miniapp directory.
[ ] Consider adding a new test for the new miniapp.

[ ] List the new miniapp in doc/CodeDocumentation.dox
[ ] If new miniapps directory (e.g.miniapps/nurbs), add it to MINIAPP_SUBDIRS in the makefile.
[ ] If new miniapps directory (e.g.miniapps/nurbs), list it in doc/CodeDocumentation.conf.in
[ ] Companion pull request for documentation in mfem/web repo:

[ ] Update or add miniapp-specific documentation, see e.g. the src/meshing.md and src/electromagnetics.md files.
[ ] Add the description, labels and screenshots in src/examples.md and src/img.
[ ] The miniapps go at the end of the page, and are usually listed only under a specific "Application (PDE)" category.
[ ] Add a short description of the miniapp in the "Extensive Examples" section of features.md.

[ ] New capability:

[ ] All new public, protected, and private classes, methods, data members, and functions have full Doxygen-style documentation in source comments. Documentation should include descriptions of member data, function arguments and return values, template parameters, and prerequisites for calling new functions.
[ ] Pointer arguments and return values must specify whether ownership is being transferred or lent with the call.
[ ] Any new functions should include descriptions of their intended use e.g. for internal use only, user-facing, etc., along with references to example code whenever possible/appropriate.
[ ] Consider adding new sample runs in existing examples to highlight the new capability.
[ ] Consider saving cool simulation pictures with the new capability in the Confluence gallery (LLNL only) or submitting them, via pull request, to the gallery section of the mfem/web repo.
[ ] If this is a major new feature, consider mentioning it in the short summary inside README (rare).
[ ] List major new classes in doc/CodeDocumentation.dox (rare).

[ ] Update this checklist, if the new pull request affects it.
[ ] Run make unittest to make sure all unit tests pass.
[ ] Run the tests in tests/scripts.
[ ] (LLNL only) After merging:

[ ] Update internal tests to include the new features.

After talking to @psocratis, I've decided to add one more new function to this PR. I removed the "ready for review" label for now

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

@psocratis and @bslazarov, can you please review?

Merged in next for testing...


Decide if SDK packages should be numbered after the SDK version or runtime version

Problem
For a given .NET release, there will unfortunately be two version numbers, the runtime version (like 8.0.5) used by the runtime, ASP.NET Core runtime, and Desktop runtime; and the SDK version (like 8.0.300).
Current behavior
This repository currently uses the runtime version (8.0.5) for all packages, even SDK packages, to simplify the dependencies and reduce confusion, with the assumption (and lack of official documentation refuting it) that this is harmless because SDKs and runtimes are always released in lockstep, or that runtimeless SDK releases are not useful to Raspberries Pi because they only introduce changes related to Visual Studio, which does not run on Debian.
Runtimeless SDK updates
However, in May 2024, an SDK update was released with no corresponding runtime release. SDK 8.0.300 was released after runtime 8.0.5 was released alongside SDK 8.0.205. This means that the SDK was not released in lockstep with a runtime, so this repository did not update to include SDK 8.0.300 (installing SDK 8.0.5 from this repository will still install SDK 8.0.205).
There is a suspicious lack of release notes for 8.0.300, and people are a little annoyed by this (see dotnet/core issue 9309). I assumed it was only released to make the SDK compatible with Visual Studio 17.10, but there are also breaking changes regarding <ManagePackageVersionsCentrally> and collection initializer type inference.
Question
Should the SDK packages generated by this repository use the SDK versions instead of the runtime versions, since there are actually important (if dumb) differences between them?
Comparison

Runtime version pros: simpler, consistent with runtime packages, dependencies are easier to declare, less state to track
SDK version pros: more faithful to upstream, allows runtimeless SDK updates to be published, runtimeless SDK updates sometimes have important (if dumb) differences from previous versions

Backwards compatibility
I don't think this is a backwards compatibility concern. Old versions of packages have never been preserved in this repository, for either SDKs or runtimes. Nothing really uses the SDK patch version, since no other internal packages depend on SDKs. Package names and minor versions won't change. The latest and latest-lts metapackages will be automatically updated to point to the new patch versions, so anyone with those installed will continue to get their updates transparently.

Changing to SDK versions seems to be a good idea and work well.


Inline editing possible on extended plugins?

I noticed an issue where my plugins subclassing TextPlugin were not rendering the correct template in edit mode. Eventually realized this was due to the conditional in this get_render_template method. The plugins render with the inline.html template even without TEXT_INLINE_EDITING = True.
Seems that a fix may be needed for that template conditional but I decided to just try enabling inline editing and make my own inline template version for the custom plugin following the same structure as the template here:
https://github.com/django-cms/djangocms-text-ckeditor/blob/master/djangocms_text_ckeditor/templates/cms/plugins/inline.html
This doesn't seem to work, however. Inline editing is there for standard text plugins but not my extended plugins. Is there something else needed to get this working?
Using:
django-cms 3.11.3
djangocms-text-ckeditor 5.1.2

Your approach seems reasonable to me. Can you share what goes wrong?

@fsbraun Inline editing is not present for those extended plugins. One thing I've noticed is a conditional in the js explicitly checks for plugin type TextPlugin which would explain at least that portion of it being missing.
Would love to get it working for those but could be that it's just not feasible to add inline editing to extended plugins (or plugins that use HTMLField). In that case, may just need to adjust the conditional I mentioned above in get_render_template to only return self.inline_editing_template when plugin type is TextPlugin and TEXT_INLINE_EDITING=True.

Your analysis makes sense. The JS side does not know about plugin inheritance and can only make decisions on plugin names.

@fsbraun Definitely. I was experimenting with hard-coding the name of my custom plugin in the javascript and it actually works as far as editing and saving though it's buggy as it's wrapping all the fields in the editor as opposed to just the one using ckeditor. I can see that identifying and targeting just the ckeditor field could be a challenge. Let me know any way I can assist.

Thinking about how one might identify which fields should be inline-editable, I don't think it would be unreasonable to require some kind of additional configuration within a custom plugin's inline_editing_template. Just one thought.

@fsbraun , do we have any solution for this feature on extended plugin?


Fix typos and remove duplicated $refs etc.

This PR links to issue #118
@trieloff @kstreete @cmathis
-Fix $ref typo for person-name schema
-Fix $ref typo for person schema
-Remove duplicated $ref for address schema

Looks good. Should merge in immediately.

Thanks @jwen-adobe – much appreciated.


integer check for get_instrument_type fails for int8

In definitions.py the functions get_instrument_type() and get_instrument_number() check for an integer using: isinstance(instrument_numbers, int)
This returns false for np.int8, which is what is you get from:
instrument_numbers = list(np.unique(ds_combined.instrument_type.values))[0]
in the combination code in io.py
Need to work out another comparison... This is separate to issue #77

I am using numpy v1.26.4 in case that is relevant here.

Changing get_instrument_type to this works:
    # If instrument_numbers is a list, return a list of strings
    if isinstance(instrument_numbers, list):
        instrument_type = [k for k, v in instrument_number.items() if v in instrument_numbers]
    else:
        instrument_type = [k for k, v in instrument_number.items() if v == instrument_numbers][0]

    return instrument_type

But then we lose the test, so it might not be ideal

Fixed in #81


Simple S3 Bucket

[x] Define an S3 bucket in Terraform
[x] We are going to use the random resource string for the name
[x] Install the AWS Provider
[x] Configure AWS Provider
[x] Terraform Apply and Terraform Destroy

We tried the random string generator but it had to be configured to be only lower cases and no upper cases.
We also changed the number of generated letters to "32".


Error during Gitea 1.21.x upgrade Add RemoteAddress to mirrors failed No such remote 'origin'

Description
I encountered an error while upgrading Gitea from version 1.20.x to 1.21.x. the upgrade process fails with an error message, whereas the previous version (1.20.x) was working fine.
routers/init.go:69:mustInitCtx() [F] code.gitea.io/gitea/routers/common.InitDBEngine(ctx) failed: migrate: migration[276]: Add RemoteAddress to mirrors failed: get remote origin's address of xxx/xxx-sales failed: exit status 2 - error: No such remote 'origin'
 - error: No such remote 'origin'

Gitea Version
1.21.5
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
docker
Database
MySQL/MariaDB

I'm experiencing the same issue, however I beleive I have found a solution. By navigating to your Gitea's repositories folder and manually adding a remote origin with the name "origin" this resolves the issue. I believe the issue is caused by mirrors without a remote address.
Steps to resolution

Open a terminal on your server and using Docker logs find the repository which is throwing the error (e.g. "xxx/xxx-sales").
Find and navigate to the location gitea/git/repositories (relative to your Gitea volume).
Open the repository which is throwing an error and add a remote with the name origin git remote add origin https://github.com/xxx/xxx.
Repeat from step 1 until no more errors show.

Yes, that's exactly what the error means.
The only weird thing is that there are mirrors without origin in the first place.
This should not be possible…

2. gitea/git/repositories

I'm experiencing the same issue, however I beleive I have found a solution. By navigating to your Gitea's repositories folder and manually adding a remote origin with the name "origin" this resolves the issue. I believe the issue is caused by mirrors without a remote address.
Steps to resolution

Open a terminal on your server and using Docker logs find the repository which is throwing the error (e.g. "xxx/xxx-sales").
Find and navigate to the location gitea/git/repositories (relative to your Gitea volume).
Open the repository which is throwing an error and add a remote with the name origin git remote add origin https://github.com/xxx/xxx.
Repeat from step 1 until no more errors show.

@go-compile @delvh thank you for the response, everything is working properly after adding all the remotes.
previously, I thought it was a bug because there was no "origin" in any of the repository under the gitea/git/repositories path.


Improve installation documentation

Fix wrong instructions to configure Java version
Simplify language around docker/java launcher/fat jar
Pre-install sbt in Docker container

Test plan
n/a

Bazel build failure looks unrelated but still needs to be fixed.


Typed classes and properties not being saves properly

This is a code breaking bug.
Basically if you have something like this:

class Cache extends BaseConfig
{
    /**
     * --------------------------------------------------------------------------
     * Primary Handler
     * --------------------------------------------------------------------------
     *
     * The name of the preferred handler that should be used. If for some reason
     * it is not available, the $backupHandler will be used in its place.
     */
    public string $handler = 'file';

    /**
     * --------------------------------------------------------------------------
     * Backup Handler
     * --------------------------------------------------------------------------
     *
     * The name of the handler that will be used in case the first one is
     * unreachable. Often, 'file' is used here since the filesystem is
     * always available, though that's not always practical for the app.
     */
    public string $backupHandler = 'dummy';
}

In turn this will be changed into this:
class Cache extends BaseConfig
{
    /**
     * --------------------------------------------------------------------------
     * Primary Handler
     * --------------------------------------------------------------------------
     *
     * The name of the preferred handler that should be used. If for some reason
     * it is not available, the $backupHandler will be used in its place.
     */
    stringpublic $handler = 'file';

    /**
     * --------------------------------------------------------------------------
     * Backup Handler
     * --------------------------------------------------------------------------
     *
     * The name of the handler that will be used in case the first one is
     * unreachable. Often, 'file' is used here since the filesystem is
     * always available, though that's not always practical for the app.
     */
    stringpublic $backupHandler = 'dummy';
}

I've located the culprit on the PSR2ModifierVisibilityStaticOrder transformation https://github.com/driade/phpfmt-8/blob/master/fmt.stub.php#L5900

Fixed on https://github.com/driade/phpfmt-8/releases/tag/806.0.0

There's still a bug

Fixed

This still has some issues I found that some types are corrected, but the array still gives an error.
Example.
<?php

namespace Config;

use CodeIgniter\Database\Config;

/**
 * Database Configuration
 */
class Database extends Config
{
    /**
     * The directory that holds the Migrations
     * and Seeds directories.
     */
    public string $filesPath = APPPATH . 'Database' . DIRECTORY_SEPARATOR;

    /**
     * Lets you choose which connection group to
     * use if no other is specified.
     */
    public string $defaultGroup = 'default';

    /**
     * The default database connection.
     */
    public array $default = [
        'DSN'      => '',
        'hostname' => 'localhost',
        'username' => '',
        'password' => '',
        'database' => '',
        'DBDriver' => 'MySQLi',
        'DBPrefix' => '',
        'pConnect' => false,
        'DBDebug'  => true,
        'charset'  => 'utf8',
        'DBCollat' => 'utf8_general_ci',
        'swapPre'  => '',
        'encrypt'  => false,
        'compress' => false,
        'strictOn' => false,
        'failover' => [],
        'port'     => 3306,
    ];

    /**
     * This database connection is used when
     * running PHPUnit database tests.
     */
    public array $tests = [
        'DSN'         => '',
        'hostname'    => '<IP_ADDRESS>',
        'username'    => '',
        'password'    => '',
        'database'    => ':memory:',
        'DBDriver'    => 'SQLite3',
        'DBPrefix'    => 'db_',  // Needed to ensure we're working correctly with prefixes live. DO NOT REMOVE FOR CI DEVS
        'pConnect'    => false,
        'DBDebug'     => true,
        'charset'     => 'utf8',
        'DBCollat'    => 'utf8_general_ci',
        'swapPre'     => '',
        'encrypt'     => false,
        'compress'    => false,
        'strictOn'    => false,
        'failover'    => [],
        'port'        => 3306,
        'foreignKeys' => true,
        'busyTimeout' => 1000,
    ];

    public function __construct()
    {
        parent::__construct();

        // Ensure that we always set the database group to 'tests' if
        // we are currently running an automated test suite, so that
        // we don't overwrite live data on accident.
        if (ENVIRONMENT === 'testing') {
            $this->defaultGroup = 'tests';
        }
    }
}

This turns into:
<?php

namespace Config;

use CodeIgniter\Database\Config;

/**
 * Database Configuration
 */
class Database extends Config
{
    /**
     * The directory that holds the Migrations
     * and Seeds directories.
     */
    public string $filesPath = APPPATH . 'Database' . DIRECTORY_SEPARATOR;

    /**
     * Lets you choose which connection group to
     * use if no other is specified.
     */
    public string $defaultGroup = 'default';

    /**
     * The default database connection.
     */
    arraypublic $default = [
        'DSN' => '',
        'hostname' => 'localhost',
        'username' => '',
        'password' => '',
        'database' => '',
        'DBDriver' => 'MySQLi',
        'DBPrefix' => '',
        'pConnect' => false,
        'DBDebug' => true,
        'charset' => 'utf8',
        'DBCollat' => 'utf8_general_ci',
        'swapPre' => '',
        'encrypt' => false,
        'compress' => false,
        'strictOn' => false,
        'failover' => [],
        'port' => 3306,
    ];

    /**
     * This database connection is used when
     * running PHPUnit database tests.
     */
    arraypublic $tests = [
        'DSN' => '',
        'hostname' => '<IP_ADDRESS>',
        'username' => '',
        'password' => '',
        'database' => ':memory:',
        'DBDriver' => 'SQLite3',
        'DBPrefix' => 'db_', // Needed to ensure we're working correctly with prefixes live. DO NOT REMOVE FOR CI DEVS
        'pConnect' => false,
        'DBDebug' => true,
        'charset' => 'utf8',
        'DBCollat' => 'utf8_general_ci',
        'swapPre' => '',
        'encrypt' => false,
        'compress' => false,
        'strictOn' => false,
        'failover' => [],
        'port' => 3306,
        'foreignKeys' => true,
        'busyTimeout' => 1000,
    ];

    public function __construct()
    {
        parent::__construct();

        // Ensure that we always set the database group to 'tests' if
        // we are currently running an automated test suite, so that
        // we don't overwrite live data on accident.
        if (ENVIRONMENT === 'testing') {
            $this->defaultGroup = 'tests';
        }
    }
}

Basically the arrays still have the old behavior.

Thanks, can you check now?

works now, thanks :)

Sorry mate, another one that's not working.
public ?string $cookieSameSite = 'Lax';
Gets transformed into:
?public string $cookieSameSite = 'Lax';

Thanks!, I've been able to solve it. Please don't hesitate sending me more failing examples.


Polished the appdata.xml file further

A followup of https://github.com/vslavik/poedit/pull/286.

Thanks, merged as 670bf91068263a0a25a425f97972866490ca9e2b (I removed the unused donation stuff I long ago forgot about). And sorry for the delay!


NSInvalidArgumentException in BranchOpenRequest.m

We're seeing a crash with some users.
Fatal Exception: NSInvalidArgumentException
-[__NSDictionaryM length]: unrecognized selector sent to instance 0x16de5790

Thread : Fatal Exception: NSInvalidArgumentException
0  CoreFoundation                 0x20a762eb __exceptionPreprocess
1  libobjc.A.dylib                0x20242dff objc_exception_throw
2  CoreFoundation                 0x20a7bab5 __methodDescriptionForSelector
3  CoreFoundation                 0x20a7970f ___forwarding___
4  CoreFoundation                 0x209a8c88 _CF_forwarding_prep_0
5  FitGurus                       0xbc9d9 -[BranchOpenRequest processResponse:error:] (BranchOpenRequest.m:116)
6  FitGurus                       0xb4b63 __30-[Branch processNextQueueItem]_block_invoke (Branch.m:1240)
7  libdispatch.dylib              0x20613dd7 _dispatch_call_block_and_release
8  libdispatch.dylib              0x20613dc3 _dispatch_client_callout
9  libdispatch.dylib              0x20618671 _dispatch_main_queue_callback_4CF
10 CoreFoundation                 0x20a38fc5 __CFRUNLOOP_IS_SERVICING_THE_MAIN_DISPATCH_QUEUE__
11 CoreFoundation                 0x20a374bf __CFRunLoopRun
12 CoreFoundation                 0x20989bb9 CFRunLoopRunSpecific
13 CoreFoundation                 0x209899ad CFRunLoopRunInMode
14 GraphicsServices               0x21c03af9 GSEventRunModal
15 UIKit                          0x24c75fb5 UIApplicationMain
16 FitGurus                       0x42377 main (main.m:14)
17 libdispatch.dylib              0x2063c873 (Missing)

@mhemmings Thanks for reporting. When was the latest crash log you've received?

@ahmednawar MAR 15 2016 09:50:02 (UTC)
Had 19 crashes so far

More questions to help us determine what might be happening:

When did they start?
What SDK version are you running?
Are you using Crashlytics? Can you send the report link to<EMAIL_ADDRESS>

As far as I can see, all crashes were 15th March.
Using Branch 0.11.16
Crashlytics report sent :)

@mhemmings since we moved the conversation to support, I am closing here. Feel free to open if you have any issues.


Reduce variablity when measuring performance

Currently our code involving filmstirp, network events tracing does not check account for the variablity that could happen due to Network contention, CPU contention, hardware etc.
This issue acts as a placeholder to start tackling those issues in a much broader sense and arrive at a solution which can produce a performane report that would not be influenced.
Lighthouse team has done an amazing job of explaning about various variablity problems in length on their docs - https://github.com/GoogleChrome/lighthouse/blob/master/docs/variability.md
More details to follow up.

Next steps

Create an example app that contains reduced variability when it comes to CPU/Network
Run our synthetics agent to capture the relevant performance information
Capture benchmarks and post it to observability benchmarks cluster.

Next steps

Create an example app that contains reduced variability when it comes to CPU/Network
Run our synthetics agent to capture the relevant performance information
Capture benchmarks and post it to observability benchmarks cluster.


Add tests

Need to add a bunch of tests to check which kinds of URLs are supported, and whether or not the included functions do what they are supposed to.
Most importantly:

Sanitizers
Identifier separators

Added in commit 7bd8544:

3 hyphen identifier handlers (join & separate)
3 named redirect checkers
same 3 tests for named rerouters
5 kebab sanitization tests with long list of bad characters
2 general tests for the URL create() helper

I'm sure these could be more thourough though as there must be cases I haven't tested yet that can break the library.

Added tests in commits
1
2
3
4
See pull request https://github.com/dominic-schmid/svelte-selfheal/pull/8 to review and merge


Fix mypy typing errors attributed to pytorch_lightning/demos/mnist_datamodule.py

What does this PR do?

Fixes mypy typing errors in pytorch_lightning/demos/mnist_datamodule.py in https://github.com/Lightning-AI/lightning/issues/13445.
Does your PR introduce any breaking changes? If yes, please list them.
None. just fixed typing errors.

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃 → Yep! 😊

@otaj @rohitgr7 I addressed your reviews


Fix go version check script for MacOS.

Motivation
This script offers a fix for https://github.com/spacemeshos/go-spacemesh/issues/2097.
MacOS's sed behaves differently from GNU sed, so the script fails to parse go version output on MacOS. This solution rewrites go version parsing part with bash-only features, so sed is not used anymore.
Closes #2097
Changes
Script ./scripts/check-go-version.sh is modified for not using sed.
Test Plan
Tested on Linux and MacOS X by running script manually, and changing minimum minor version value.

though waiting to see what you and @lrettig conclude on in #2097

bors try

I simplified and cleaned this up a bit, @divan could you please check if it still works well for you? If so I think we're ready to merge

bors try

bors try

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge


⬆ Upgrade deps

I was not able to bump https://github.com/gagliardetto/solana-go from v1.4.0 to v1.5.0 because go mod tidy also upgrades https://github.com/streamingfast/logging :-(
For information, we are not able to upgrade https://github.com/streamingfast/logging since a while.
I think the organization renaming from https://github.com/dfuse-io to https://github.com/streamingfast does not help.
Is it possible to upgrade https://github.com/gagliardetto/solana-go?

No we shouldn't.  both package have a common history but diverge quite a bit. I tried to switch at some point but was not able to make everything work. the functions are similar but some outputs are different. Also their are used to make solana smart contracts that were made in Anchor in a Go pkg (I don't remember which one exactly).
So we can put it in stand by for now or close it in the mean time.


New approach and UI for switching between types in sequence mode

Background
Currently, if the user wants to switch a type in sequence mode they have to stop typing, click once to open the drop down menu, select from the menu, and then continue typing.
Solution

Introduce keyboard commands for switching between types of typing:
Ctrl+Alt+R switches to RNA type
Ctrl+Alt+D switches to DNA type
Ctrl+Alt+P switches to peptide type

On macOS instead of "Alt", "Option" is used.

The current drop-down menu should be changed to:

2.1. When hovering over RNA the user should see "RNA (Ctrl+Alt+R)", when hovering over DNA the user should see "DNA (Ctrl+Alt+D)", when hovering over PEP the user should see "Peptides (Ctrl+Alt+P)".
2.2. The default mode for typing is RNA.

Verified.

Ketcher - Version 2.24.0-rc.1 Build at 2024-09-02; 05:32:40
Version 1.24.0-dev.7.0-g59089022-wasm32-wasm-clang-12.0.0
Chrome 128.0.6613.114 (Official Build) (64-bit)
Win10

Tested.

Desktop:

OS: Windows 11
Browser Chrome
Version 128.0.6613.120 (Official Build) (64-bit)

Ketcher version
[Version 2.25.0-rc.1]
Indigo version
[Version 1.24.0-rc.1]


feat(gsoc'24): report as spam and moderation review page

Fixes #17
This PR's covers following functionality

improvised UI for actions like delete, edit, report spam, mark as solution using dropdown.
authenticated user can report any forum post as spam for review
moderators can delete reported spam post from moderators only route. threads/spam_reports

Note: For client side interaction like dropdown toggle, report spam modal form, javascript was needed so I've used hotwire stimulus controllers for this. and currently build files of these javascript assets are not pushed but to use this rails engine in production, we have to push these javascripts assets with the rails engine.
Screenshots:

Could you resolve the conflicts ?  Also not all of these messages are spam. Make spam as a category and rename the text as "Report post".
Also don't push mark as solution to the dropdown, it's visibility will decrease.

Also don't push mark as solution to the dropdown, it's visibility will decrease.

Yeah agreed with your point, but if we add this button in the forum post's footer, it will make post more crowdy with this action button.
Since all action regarding the forum post is now grouped using this dropdown, it just matter of time user will adopt to this feature as they gets used to this forum card layout. wdyt?

@tachyons made that mark as solution and unmark as solution button visible outside of dropdown. Now it will be shown in the forum post's header like this.

@tachyons I've added new tests for new functionality which I added. can you please re-review this PR


ax88772.rb: update to 2.4.0_20151117

Fixes the current Travis failure.

Sorry, didn't see this before. Fixed in caskroom/homebrew-cask@157321e


Fix SimpleChannelPoolTest#testCloseAsync() test flake

Motivation
This test is failing intermittently and upon inspection has a race condition.
Modification
Fix race condition by waiting for async release calls to complete prior to closing the pool.
Result
Hopefully fixed flaky test

@netty-bot test this please

@njhill thanks


missing icons

highly recommend confirming that whatever is in use in the site itself is also available in the actual system, otherwise it feels like the design system is busy ignoring (if not breaking) its own rules while insisting users follow those rules.
frex:
the site uses an icon to indicate an external link, but this icon isn't available in the library.
the site uses tiles in a way that isn't discussed/explained in the grid or tile sections (and the actual tile example doesn't show up on the site, either)

@k2lh the icon we use to indicate an outbound link is called 'Launch' it's available through @carbon/icons or whatever framework you prefer.
The website, like any other application built with Carbon, has some bespoke styles and components. These styles and components are build using primitives provided by Carbon: layout, color, typography, motion, etc.
Gonna close this one out, let me know if that wasn't the icon you were referring to.


Vault: Descriptions are trimmed

Description

The description is cut-off, which is not desired since it can contain useful information

After release:


Upgrade jline 3

We no longer need to use the forked version of jline because they have
merged in our required changes. The latest version of jline does upgrade
jansi, however, and some of the apis we were relying on for windows were
removed so they had to be manually implemented. I verified that console
input still worked on my windows vm after this change.

The latest version of jansi broke binary compatibility with jline 2 (https://github.com/fusesource/jansi/pull/174). This will require a little bit more work to restore the missing apis in our fork of jline2 (https://github.com/sbt/jline2).

I manually tested that the console works on windows with scala 2.10.7, 2.11.12, 2.12.12, 2.13.3 and dotty 0.26.0-RC1 after these changes.

Given the high frequency of JLine and unpredicted result, I feel like this should go into sbt 1.5.0. WDYT?

That seems fine. There is no rush for this. I just wanted to get us off the forked jline 3 and there were unexpected complications due to their breaking binary compatibility in jansi.


[Issue-2385] Add Azero Domains Pool & Validator as default option when users stake A0

Related issue(s)

#2385

Is your feature request related to a problem(s)? Please describe.

Add default pool for Aleph Zero

Describe the solution you'd like

Add default pool for Aleph Zero

Describe alternatives you've considered

No

Additional context

No

🚀 Deployed on https://pr-2388--sw-web-runner.netlify.app


Tasks collision

I need to implement a tasks collision highlighting, do you have an idea how can I implement such a thing?
Is there a way of detecting if 2 tasks (or more) are on the same time range?

Duplicate #77


Update client to download all annotations for a tomogram

Update the client to take a tomogram id and download all annotations

Individual annotation download is captured in https://github.com/chanzuckerberg/cryoet-data-portal/issues/320


Accessibility - Rethink keyboard navigation on page load

When the page loads, a keyboard user has to tab through the entire menu before they can navigate through the main page.
Ideas:

Add a link to the top of the menu to skip to main page content
When page loads, focus on the main page, but allow a user to ctrl-tab or cmd-tab backward to access menu.

https://webaim.org/techniques/skipnav/

Great job! Pressing escape exits out of the navigation, closes it, and focuses on the main body. I believe this closes this issue.


Performance measurements

We need to understand the time it takes various modules to complete a transaction. Starting at the Ledger, we know we can do around 55k tps (1 write), but at the client API level, we see max of around 400 tps. We need to instrument the code (or tests) with time in log INFO using the technique here https://coderwall.com/p/cp5fya/measuring-execution-time-in-go

Check for 15 nodes

I want to contribute something for this, but first we need a list of modules or functions we want to understand the time spend.

maybe we can use this to reduce our coding effort which likes the decorator grammar in python : https://gist.github.com/alehano/8785933


/sys/default/client/scripts/installServerTode2/ should be split out

for folks with existing stone structures, the code between the backups in /sys/default/client/scripts/installServerTode2/ needs to be run ... would be convenient to split that out ahead of time so that a single command can be documented as the conversion method:
jUst for grins here's the code of interest:
# Set up /sys node structure
mount --todeRoot sys/default /sys default
mount --todeRoot sys/local /sys local
mount --todeRoot sys/stones /sys stones
# ensure that --stoneRoot directory structure is present
/sys/default/bin/validateStoneSysNodes --files --repair
mount --stoneRoot / /sys stone
# Define /home and /projects based on a composition of the /sys nodes
mount --stoneRoot homeComposition.ston / home
mount --stoneRoot projectComposition.ston / projects

I just installed a new Tode and the /sys node is not present.
Probably the code mentioned has to be doctumented in the install process or, actually, be run?

The new script file is tode/sys/default/client/scripts/setUpSys ...
@krono all of this work is taking place on the dev branch and will be available for general use when I announce the release GsDevKitHome v1.0.0 and tODE v0.0.3 ... both will be released together


fix: Samples from Docker process now correctly filtered as host processes

This fixes issue #167

Pull Request Test Coverage Report for Build 343678424

60 of 76 (78.95%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.1%) to 57.932%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/metrics/sampler/matcher.go
54
70
77.14%

Files with Coverage Reduction
New Missed Lines
%

pkg/metrics/sampler/matcher.go
3
87.17%

Totals

Change from base Build 341922252:
0.1%

Covered Lines:
11138

Relevant Lines:
19226

💛  - Coveralls


Train SSD 600

HI, I want to train my custom dataset with ssd 600. Is it enough to set
 image_resizer {
      fixed_shape_resizer {
        height: 300
        width: 300
      }
    }

to 600 / 600 instead of 300 / 300 ?
Thank you very much.

yes that is enough, but then you won't be able to use the pre-trained checkpoints.
As the weights only fit a model that takes the same input dimensions!


ThirdwebSDK problem

Hi, I'm beginning with ThirdwebSDK and I got the next problem.
const sdk = new ThirdwebSDK("mumbai");
^
TypeError: ThirdwebSDK is not a constructor
I followed the instructions using first step.
const ThirdwebSDK = require ("@thirdweb-dev/sdk");
const sdk = new ThirdwebSDK("mumbai");

hi @bolisteward I believe that should be
const { ThirdwebSDK } = require("@thirdweb-dev/sdk");
const sdk = new ThirdwebSDK("mumbai");

also where did you find that code-snippet? Would love to hunt it down and fix it / have it fixed as well!

Thank u, jajaj


Finding file paths affected by a commit

I've been trying to work out how to filter file paths affected by a specific commit. Or in other words, show me the files modified by a particular commit, or maybe all the files affected when a particular merge happens.
As far as I can tell, each commit hash in the files table has a record for every file in the tree at that time. I'm struggling to see where in the data model the file paths changed in each commit might be available.
Thanks!

Hi @eddiesholl - I think what you're asking about can be achieved using the stats table-valued function. For instance:
SELECT * FROM commits, stats('', commits.hash)

Will return the stats (additions/deletions) for every file modified in a commit. In this case, since you mainly are looking for the files modified, the stats name is a bit misleading, but the query will give a list of the file paths modified by the commit (and you can ignore the "stats").
SELECT * FROM stats('', 'some-commit-sha')

Will return results only for a known commit hash.
SELECT * FROM commits, stats('', commits.hash)
WHERE commits.parents > 1

Will return stats only for merge commits (a commit with more than 1 parent).
Hope that helps!


feat/reset-button-macro

Related to #51
Adding a Reset Button for each Macro Lane.

Some macro items were misaligned, I think it's worth using the vertical alignment tool on the arrows and buttons


[Bug] Dark mode is not getting applied in our services page.

Is there an existing issue for this?

[X] I have searched the existing issues

Bug Description?
The dark mode feature on the "Our Services" page of the website  is not functioning as expected. When toggling the dark mode switch, the page does not change to the dark theme, and the styles remain in light mode. This issue affects user experience, especially for those who prefer dark mode for better visibility and reduced eye strain. Feel free to modify it as needed to fit any specific guidelines or requirements for your project! If you have any other questions or need further assistance, let me know!
Add Screenshots

What browsers are you seeing the problem on?
No response
Record

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

@Luson045, I would like to work on this issue

@Luson045 I have already solved the problem.


Fix UI and BFF image tags for our make deployment script

Description
We don't tag a dev-latest it should only be latest

How Has This Been Tested?

Merge criteria:

All the commits have been signed-off  (To pass the DCO check)

[x] The commits have meaningful messages; the author will squash them after approval or in case of manual merges will ask to merge with squash.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work.
[x] Code changes follow the kubeflow contribution guidelines.

If you have UI changes

[ ] The developer has added tests or explained why testing cannot be added.
[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Verify that UI/UX changes conform the UX guidelines for Kubeflow.

/lgtm


测试配置通过，但是Job勾选机器人触发不了钉钉消息通知

Describe the bug
JENKIN  dingtalk配置测试通过，但是Job勾选机器人触发不了钉钉消息通知
Screenshots
If applicable, add screenshots to help explain your problem.

Jenkins Info(please complete the following information):

Version [2.289.1]

DingTalk Plugin Info(please complete the following information):

Version [2.4.3]

Additional context
Add any other context about the problem here.

重启完jenkins 就可以了。现关闭issue


deis commands throw errors when being piped to head

When output of deis commands exceeds default number of lines head accepts (10), it starts throwing python exception. For example:
> $ deis logs -a staging -n 11 | head
2015-05-25T21:54:45UTC staging[web.1]:   Rendered shared/_head.html.slim (0.3ms)
2015-05-25T21:54:45UTC staging[web.1]:   Rendered shared/_ga.html.erb (0.2ms)
2015-05-26T08:56:36UTC staging[web.1]: Started GET "/" for <IP_ADDRESS> at 2015-05-26 08:56:36 +0000
2015-05-26T08:56:36UTC staging[web.1]: Processing by WebsiteController#index as HTML
2015-05-26T08:56:36UTC staging[web.1]: Filter chain halted as :authenticate rendered or redirected
2015-05-26T08:56:36UTC staging[web.1]: Completed 401 Unauthorized in 1ms
2015-05-26T22:06:34UTC staging[web.1]: Started GET "/" for <IP_ADDRESS> at 2015-05-26 22:06:34 +0000
2015-05-26T22:06:34UTC staging[web.1]: Processing by WebsiteController#index as HTML
2015-05-26T22:06:34UTC staging[web.1]: Filter chain halted as :authenticate rendered or redirected
2015-05-26T22:06:34UTC staging[web.1]: Completed 401 Unauthorized in 1ms
Traceback (most recent call last):
  File "/var/lib/jenkins/jobs/build-deis-cli-installer-linux/workspace/src/github.com/deis/deis/client/build/deis/out00-PYZ.pyz/logging", line 875, in emit
  File "/var/lib/jenkins/jobs/build-deis-cli-installer-linux/workspace/src/github.com/deis/deis/client/build/deis/out00-PYZ.pyz/logging", line 835, in flush
IOError: [Errno 32] Broken pipe
Logged from file <string>, line 670

but
> $ deis logs -a staging -n 10 | head
2015-05-25T21:54:45UTC staging[web.1]:   Rendered shared/_ga.html.erb (0.2ms)
2015-05-26T08:56:36UTC staging[web.1]: Started GET "/" for <IP_ADDRESS> at 2015-05-26 08:56:36 +0000
2015-05-26T08:56:36UTC staging[web.1]: Processing by WebsiteController#index as HTML
2015-05-26T08:56:36UTC staging[web.1]: Filter chain halted as :authenticate rendered or redirected
2015-05-26T08:56:36UTC staging[web.1]: Completed 401 Unauthorized in 1ms
2015-05-26T22:06:34UTC staging[web.1]: Started GET "/" for <IP_ADDRESS> at 2015-05-26 22:06:34 +0000
2015-05-26T22:06:34UTC staging[web.1]: Processing by WebsiteController#index as HTML
2015-05-26T22:06:34UTC staging[web.1]: Filter chain halted as :authenticate rendered or redirected
2015-05-26T22:06:34UTC staging[web.1]: Completed 401 Unauthorized in 1ms
2015-05-26T22:06:34UTC staging[web.1]: Started GET "/" for <IP_ADDRESS> at 2015-05-26 22:06:34 +0000

Exception output repeats for each line from 11.
This basically means that there is no way to extract first lines of output by head -n 2.

I can reproduce this, not sure yet what's causing it.

Probably related: http://stackoverflow.com/a/30091579/96855
TL;DR: If a sequence of commands appears in a pipeline, and one of the reading commands finishes before the writer has finished, the writer receives a SIGPIPE signal (for example ps -aux | head). Python itself by default traps SIGPIPE and throws IOError.

First attempt at a PR :)

First attempt at a PR :)

:fireworks:


Remove std::merkle::check_membership

Problem
The stdlib function check_membership is pretty trivial and is less readable than performing the equality check in user code imo.
https://github.com/noir-lang/noir/blob/a830ce5c3b247a7a60d7d4de4f8470e3227b8a47/noir_stdlib/src/merkle.nr#L3-L10
Proposed solution
We should remove std::merkle::check_membership and push people towards using compute_merkle_root.
Alternatives considered
Alternatively we can retain this function but update the return type to bool
Additional context
Related to #1258
Submission Checklist

[ ] Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.

Closed in https://github.com/noir-lang/noir/pull/1424


Hebrew SubRip Error

https://itunes.apple.com/il/movie/top-gun-maverick/id1622175422
https://itunes.apple.com/il/movie/monster-trucks/id1181629316

First one is a pre-order (not available yet), second one works fine for me.
You'll have to put more effort when asking for help, I can't guess what the issue is if you just paste links without any context.

The second not work for me..
Config: He with RTL
Same Config work with another movies
Scraping https://itunes.apple.com/il/movie/monster-trucks/id1181629316...
Found movie: Monster Trucks
0/0 matching subtitles for "Monster Trucks" were downloaded from 1 playlists to /tmp".

Please post the config you're using

`# This is the default config.
This file should NOT be altered. For changing settings, use a user-config file.
Any settings set on the user config will override these.
[general]
check-for-updates = false
[downloads]
folder = "/tmp/"
format = "srt"
languages = ["he"]
merge-playlists = false
user-agent = "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/100.0.4896.127 Safari/537.36"
zip = false
[scraping]
user-agent = "iTunes-AppleTV/15.2"
[subtitles]
fix-rtl = true
rtl-languages = ["ar", "he"]
remove-duplicates = true`

The language code for Hebrew in this case (for https://itunes.apple.com/il/movie/monster-trucks/id1181629316)  is he-IL, so you'll have to add it to your list of languages to download.
languages = ["he", "he-IL"]

You can always check your self by removing the languages setting from your config, which will result in downloading all of the available subtitles, where you can see the language code for each downloaded subtitles file.
Also, a small tip:
Don't put settings where you use the default values in your config.
It will help avoid issues in the future for changing one of them by mistake.
You can just use the following config, which will have the same effect:
[general]
check-for-updates = false

[downloads]
folder = "/tmp/"
format = "srt"
languages = ["he", "he-IL"]

[subtitles]
fix-rtl = true


NPN Switch on NeoPixels

Managed to mix up some NPN transistors for the PNP ones I need 😢 Will need to get the right part to test if the switch works correctly, but for now, I've bridged the connection with a 0-ohm resistor and got the examples working correctly.

I think I've got a working hack for the battery monitor.  Have cut the Pin12 line and bridged the pinout for 12 and 34.  Needed the change the code a bit so it actually gets the right voltage
volt = (read * 3.3 / 4096) * 1.115
Have set it up to take readings and see I can see a decrease in the battery over time.  If successful, will hack the other versions I have and update the schematics for V2.2


Different dependencies installed for PG13 s390x, mips64le vs amd64, arm64, ppc64le (has some to PG14 rel. packages)

When I running:
on amd64, arm64, ppc64le
docker run --platform=linux/amd64 -ti --rm postgres:13-bullseye bash -c "dpkg -l | grep -i postgresql"
Status: Downloaded newer image for postgres:13-bullseye
ii  libpq5:amd64              14.2-1.pgdg110+1             amd64        PostgreSQL C client library
ii  pgdg-keyring              2018.2                       all          keyring for apt.postgresql.org
ii  postgresql-13             13.6-1.pgdg110+1             amd64        The World's Most Advanced Open Source Relational Database
ii  postgresql-client-13      13.6-1.pgdg110+1             amd64        front-end programs for PostgreSQL 13
ii  postgresql-client-common  238.pgdg110+1                all          manager for multiple PostgreSQL client versions
ii  postgresql-common         238.pgdg110+1                all          PostgreSQL database-cluster manager

on s390x and mips64le it looks like
docker run --platform=linux/s390x -ti --rm postgres:13-bullseye bash -c "dpkg -l | grep -i postgresql"
ii  libpq5:s390x              13.6-1.pgdg110+1             s390x        PostgreSQL C client library
ii  pgdg-keyring              2018.2                       all          keyring for apt.postgresql.org
ii  postgresql-13             13.6-1.pgdg110+1             s390x        The World's Most Advanced Open Source Relational Database
ii  postgresql-client-13      13.6-1.pgdg110+1             s390x        front-end programs for PostgreSQL 13
ii  postgresql-client-common  238.pgdg110+1                all          manager for multiple PostgreSQL client versions
ii  postgresql-common         238.pgdg110+1                all          PostgreSQL database-cluster manager

Does this make problems?
libpq5:amd64              14.2-1.pgdg110+1 != libpq5:s390x              13.6-1.pgdg110+1
I don't want to try it!
I would expect not to have any postgres14 dependencies for the postgres 13 images

Upstream only has a handful of officially built architectures https://github.com/docker-library/postgres/blob/e483778176ca34bcbe83ee17000820d4f6e64c28/13/bullseye/Dockerfile#L98-L101
Architectures not on the list are built from source https://github.com/docker-library/postgres/blob/e483778176ca34bcbe83ee17000820d4f6e64c28/13/bullseye/Dockerfile#L106-L108

I'm more concerned by the s390x image getting an older version of libpq than I am by the amd64 image getting a newer one -- we're installing these directly from the PostgreSQL repositories in the way that they recommend, and I haven't ever seen a compatibility issue with having "too new" of a libpq instance.

Ah, we're building libpq5 as part of postgresql-XX, so we'd have to do a lot of extra work to build a newer version on these architectures -- https://github.com/docker-library/postgres/pull/800#issuecomment-748195107 is also very relevant.

Who guarantees me that the compatibility between both versions is really permanently given? Although it says on the postgesql website that it should be so, but we all know that such a thing is not 100% guaranteed.
Know I build all source new, to have the packages and created my own apt repo. But this is not the ideal way I think.

Dependencies for PG13 on different architectures (s390x, mips64le vs amd64, arm64, ppc64le) can vary due to architecture-specific optimizations. PG14 packages might be available for newer architectures like amd64 and ppc64le, offering updates. Ensure the correct architecture-specific dependencies are used to avoid conflicts.kineprohub


add screeps docker

Description
A Docker for the Dedicated Screeps server

All Submissions:

[x] Have you ensured there aren't other open Pull Requests for the same update or change?
[x] Have you created a new branch for your changes and PR from that branch and not from your master branch?

New Image Submissions:

[ ] Have you added your image to the Github workflows?
[x] Have you updated the README list to contain your new image?

how do i add the image to the gitlab workflows, do i just add - screeps to the games.yml jobs matrix section?

and will the workflows automatically create the docker image at ghcr.io/parkercvp/games:screeps? I assumed this when updating the readme

is this realy needed? it seems like a normal nodejs image can run this just fine?

Screeps requires nodejs 10, which i didnt see an image of.
It also requires python2 and various build tools to successfully install screeps (on alpine images you would need alpine-sdk & linux-headers)
So all in all, you need a few extra requirements ontop of nodejs for screeps

Screeps requires nodejs 10, which i didnt see an image of.
It also requires python2 and various build tools to successfully install screeps (on alpine images you would need alpine-sdk & linux-headers)
So all in all, you need a few extra requirements ontop of nodejs for screeps

node 10 is long EOL, and building it should be done in the install script, not with the main image. The install container can run regular images.

Testing this with the Alpine yolk and using an older alpine install image (python 2 isnt available on the latest) to see if this can indeed be avoided as its own yolk.
On an AMD 5950X node (assigned 16 cores, node is inside a dedicated VM), the nodejs 10 install takes over 30 minutes. That alone makes me feel like having this as a yolk rather than an install script would be worthwhile... Even though it only has to be done once per install, its a long time for a server to setup
Once installed im having trouble sourcing node into the cmd line aswell in the yolk image, still figuring out why this is happening
The directory node is installed to straight up doesnt even exist after that 30-40 minute install script..., does ptero not mount the same across the install script container and the server container?

Ok, using the Alpine yolk, i can't source or . /home/container/nvm/nvm.sh to load up node v10, looking like I will need at the least a node10 yolk or external docker to run screeps

Ok, using the Alpine yolk, i can't source or . /home/container/nvm/nvm.sh to load up node v10, looking like I will need at the least a node10 yolk or external docker to run screeps

as Quinten said. Node10 is EOL since a long time. Feel free to create your own Image


ci: test choco install no-progress and openssl.light

removes the progress bar when installing OpenSSL via Choco
BTW: We should upgrade Choco/Openssl from 1.1.1 to 3.0.0 ...


Keyboard and Mouse not working?

Question
Keyboard and Mouse not working while follow the doc of running x window
P.S. English is not my native language. Sorry about any grammar or typing mistakes.
Environment
Windows side: 16257.1000, Visual Studio 15.3.2, .NET Framework 4.7.02542
Both Debug and Release build failed on this.
Linux side: Ubuntu 16.04 xenial, x86_64 Linux 4.4.0-43-Microsoft, gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.4), zsh 5.1.1, make 4.1, download from Windows Store.
Build with make command
What did

Run LoWeExposer.exe in Windows.
Run loweagent -e X -o x.log in WSL.
Run unity in another WSL terminal.

What happens
A unity Desktop appear in the FrameBuffer Window, but I can't move the cursor in the window. No responce for keyboard as well.

Output of  loweagent -e X -o x.log
Output of unity
Output of Exposer
x.log file

mplayer can play a simple video, but i don't know how to pause or do something else.
xfce4-terminal failed as well: successfully show the window, but can't input anything.

Hi @EtaoinWu,
Don't worry regarding your English, we sail in the same boat as neither I'm a native English speaker. However we are more in number than the native speakers so we rule the English language not them. Just kidding. :)
As I see your Windows version is newer than mine as according to your version you use some kind of insider version. Mine is 15063 aka Creators Update.
However I think that the problem is not with the Windows version.
For the X/unity issue:
Can it happen that you don't capture the mouse? You should press Left Ctrl + Left Alt and click the left button inside the FrameBuffer Exposer window. The same combination will release the mouse.
For the mplayer issue:
You should hit the controlling keys (like p for pause) not in the FrameBuffer Exposer but in the Bash window where you started mplayer.
Please check if with this explanation you can control you applications and get back to me.
If you say that the explanation solves your problems then I'll clarify the documentation as currently only LoweExposer program explains input device usage.

Hit the keys in Bash window solved my problem about mplayer.
As for the X issue, I found this below:

there is something output in the FrameBuffer Exposer.
I can coltrol neither mouse nor keyboard before I hit Lctrl-Lalt-LMouseClick, There is no output in LoWe Exposer's Mouse log window.
After I hit C-M-click, this is the output like this:X: 717, Y: 291, L: False, R: False
X: 717, Y: 289, L: False, R: False
X: 718, Y: 289, L: False, R: False
X: 718, Y: 288, L: False, R: False
X: 718, Y: 287, L: False, R: False

But I can't control the mouse inside the Exposer as well.

Hi @EtaoinWu,
For the X issue.
Have you correctly created /usr/share/X11/xorg.conf.d/9-lowe.conf with the right content (copy the file from the config folder)?
This is a prerequisite.
Could you please raise the log level of the agent to Debug?
eg. ./loweagent -e X -o x.log -l Debug
If mouse and keyboard are working the you should see fragments like this in the log:
"
[DEBUG][P 46][FD 9]{mice}: -= Before read =-
[DEBUG][P 46][FD 9]{mice}: Read regs. rdi: 9 rsi:<PHONE_NUMBER>86160 rdx: 6144
[DEBUG][P 46][FD 9]{mice}: Read size: 6144
[DEBUG][P 46][FD 9]{mice}: -= After read =-
[DEBUG][P 46][FD 9]{mice}: Read size: 0
[DEBUG][P 46][FD 9]{mice}: regs. rax: 0 rdi: 9 rsi:<PHONE_NUMBER>86160 rdx: 6144 r10: 8 r8:<PHONE_NUMBER>24656 r9: 0
[DEBUG][P 46]{runtime}: ---
[DEBUG][P 46]{runtime}: SYSCALL: 14
[DEBUG][P 46]{runtime}: SYSCALL: 14
[DEBUG][P 46]{runtime}: SYSCALL: 0
[DEBUG][P 46][FD 10]{kbd}: -= Before read =-
[DEBUG][P 46][FD 10]{kbd}: Read regs. rdi: 10 rsi:<PHONE_NUMBER>13216 rdx: 6192
[DEBUG][P 46][FD 10]{kbd}: Read size: 6192
[DEBUG][P 46][FD 10]{kbd}: -= After read =-
[DEBUG][P 46][FD 10]{kbd}: Read size: 0
[DEBUG][P 46][FD 10]{kbd}: regs. rax: 0 rdi: 10 rsi:<PHONE_NUMBER>13216 rdx: 6192 r10: 8 r8:<PHONE_NUMBER>24656 r9: 0
"

Oops, I didn't found the sentence

Copy the 9-lowe.conf from the configs folder of the local git repository to /usr/share/X11/xorg.conf.d/9-lowe.conf.

So the application do work on Windows build 16257. Apologize to you for taking so much of your time.


Add Copyright Policy to Website Home Page

Description:
I would like to request the addition of a copyright policy to the home page of the website. This is to ensure that all users are aware of the copyright policies governing the content and usage of the website. Kindly assign me for it, this request is part of the DevFest AI initiative.
Steps to Implement:

Draft the copyright policy text.
Design and integrate the copyright policy section on the home page.
Ensure the policy is clearly visible and accessible to all users.

Purpose:
To inform users about the copyright policies and protect the intellectual property associated with the website content.
Thank you for considering this request.

You can now use this draft to create a new issue in the mlc-ai/web-llm repository.

Is this an AI-generated issue? The Apache license is clearly mentioned on the Github front page?


feature/file-hosting-updates

This pull request addresses this issue: #100

We deprecated the @staticfiles and @dynamicfiles macros and pointed them to use the new staticfiles() and dynamicfiles() functions. There was no real reason for these to be macros, they could've easily been regular functions from the beginning.
These new filemounting functions can mount files at the root path "/"
The old macros point to the new functions and these macros now support variables

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.30 :tada:

Comparison is base (fa69326) 94.23% compared to head (0b556d3) 94.53%.

:exclamation: Current head 0b556d3 differs from pull request most recent head 8907834. Consider uploading reports for the commit 8907834 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #101      +/-   ##
==========================================
+ Coverage   94.23%   94.53%   +0.30%
==========================================
  Files           9        9
  Lines         780      787       +7
==========================================
+ Hits          735      744       +9
+ Misses         45       43       -2

Impacted Files
Coverage Δ

src/Oxygen.jl
100.00% <ø> (ø)

src/core.jl
86.60% <100.00%> (+0.36%)
:arrow_up:

src/fileutil.jl
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add confirmation before deleting CRIS/JDynA items

Rationale
The basic design of the administrative panel of DSpace CRIS might increase unintentional deletion of CRIS/JDynA items (RPs, OrgUnits, Boxes, Tabs, etc.), since there's no confirmation and buttons are inappropriate arranged. There's merely a gap between editing or deleting an existing box/tab.

The confirmation applies to every button/link with [id^=delete] (CRIS entities) or .jdynaremovebutton (JDynA entities).
*as far as I could identify

I like this patch. I have also added a delete confirmation locally here, but this patch is a bit easier and more universal. So +1 from me to get this into the DSpace CRIS master :+1:

@r3r57 Thank you for this enhancement.


freedv: 1.8.6 -> 1.8.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/drowe67/freedv-gui/releases.
meta.description for freedv is: Digital voice for HF radio
meta.homepage for freedv is: https://freedv.org/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.8.7 with grep in /nix/store/q9pf3xyhvwpydrcxlvfbmddhij6k0h99-freedv-1.8.7
found 1.8.7 in filename of file in /nix/store/q9pf3xyhvwpydrcxlvfbmddhij6k0h99-freedv-1.8.7

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
freedv

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q9pf3xyhvwpydrcxlvfbmddhij6k0h99-freedv-1.8.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A freedv https://github.com/r-ryantm/nixpkgs/archive/0e569a490a083c0e8cfc376a275b6df17d09575b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q9pf3xyhvwpydrcxlvfbmddhij6k0h99-freedv-1.8.7
ls -la /nix/store/q9pf3xyhvwpydrcxlvfbmddhij6k0h99-freedv-1.8.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

freedv

Maintainer pings
cc @illdefined @wegank for testing.

Superseded by #211675.


Form class documentation outdated?

Affected Version
Tested in 4.3, but could affect older versions.
Description
The documentation for the Form class (https://api.silverstripe.org/4/SilverStripe/Forms/Form.html) says:

The form class extends {@link RequestHandler}, which means it can be accessed directly through a URL. This can be handy for refreshing a form by ajax, or even just displaying a single form field. You can find out the base URL for your form by looking at the <form action="..."> value. For example, the edit form in the CMS would be located at "admin/EditForm". This URL will render the form without its surrounding template when called through GET instead of POST.
By appending to this URL, you can render individual form elements through the {@link FormField->FieldHolder()} method. For example, the "URLSegment" field in a standard CMS form would be accessible through "admin/EditForm/field/URLSegment/FieldHolder".

After testing on multiple forms, including the admin/EditForm example, this doesn't actually work and returns This form requires a POST submission instead of the rendered HTML of the form.
Steps to Reproduce
Append the form name to the end of the URL of the page the form is located on. ie:
mysite/admin/EditForm
mysite/contact/ContactForm
Is there a proper way to do this? Or is this just outdated documentation?

Is there a solution by now? Still have the same problem in SS 4.12 - trying to load a form with AJAX (or direct URL input) and still get this error.


Measure and track AS completion by Carrier x Provider x Country x Verify

TODO: walkthrough session with @codyborn to get a sense of how the dashboards currently work cc @arthurgousset

Next steps for the ticket, from AS data landscape walkthrough cc @arthurgousset

make sure necessary fields are accessible in BigQuery, i.e. that metadata crawler is updated + being piped into BigQuery
potentially add fields to an existing model, or likely create a new model with the necessary data
may need to join completion rate (attestation_state_with_user_data issuer JOIN ON  attestation_service_check address or signer? unclear double check this)
figure out how we want to visualize this data/table -- add this to the Mainnet Attestations dashboard, potentially add later to the Mainnet Attestations - Validator Dashboards (public-facing)

(Potential later TODO): alerting based on a specific failure rate for a carrier/provider/country; currently can add an alert based on a specific question (i.e. if anything shows up for a given query, then send an alert at some frequency)


failed to load new config

Hello,I just installed daed using nixosmodules while I can't get it work.
I'm using:
  services.daed = {
    enable = true;
    package = daeuniverse.packages.x86_64-linux.daed; # default
    configDir = "/etc/daed";
    listen = "<IP_ADDRESS>:2023";
    openFirewall = {
      enable = true;
      port = 12345;
    };
  };

when I click proxy switch,I get this:
failed to load new config: ApplyRulesOptimizers error: geoip.dat: file does not exist in [/etc/daed, /root/.local/share/daed, /usr/local/share/daed, /usr/share/daed]; see more in log

Could you please try #23 with this config?
 daed = {
      enable = true;
      configDir = "/etc/daed";
      openFirewall = {
        enable = true;
        port = 12345;
      };
 };

#23 works perfectly.
Thank you.


variety of consistency work

This supersedes #304 and will be split into multiple PRs for review:

#309
#311
#312
#315
#316
#320
#321

Description

Changelog description

Status

[ ] Code base linted
[ ] Ready to go

closed now that contents covered by other PRs


Small fix to 10_tutorial.pets notebook

There was still one reference to Datasource, so I changed it to use the new name Datasets.

Thanks a lot!


fix(modp2p): change the timecache strategy for gossip sub

We observed that sometimes old headers are received by nodes on the p2p network. This PR attempts to increase the time a gossiped message stays in caches by changing the strategy to LastSeen. This assumes there are duplicates on the network which reenter the cache.

Codecov Report

Merging #1873 (280f35f) into main (b8d0203) will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1873      +/-   ##
==========================================
+ Coverage   57.41%   57.53%   +0.11%
==========================================
  Files         242      242
  Lines       16211    16212       +1
==========================================
+ Hits         9308     9327      +19
+ Misses       5957     5937      -20
- Partials      946      948       +2

Impacted Files
Coverage Δ

nodebuilder/p2p/pubsub.go
60.78% <100.00%> (+0.38%)
:arrow_up:

libs/header/p2p/server.go
75.40% <0.00%> (-2.74%)
:arrow_down:

share/eds/byzantine/pb/share.pb.go
34.84% <0.00%> (+1.96%)
:arrow_up:

share/availability/discovery/discovery.go
59.52% <0.00%> (+5.55%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add retry to webapp test

The current test tries to access the webapp immediately after it is deployed, which can result in a 404 or the default page for app service, not the actual app code.
This fix will retry for static text 'Hello, azd. until it finds it or until retries have been exhausted.

Azure Dev CLI Install Instructions
Install scripts
MacOS/Linux

May elevate using sudo on some platforms and configurations

curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/193/uninstall-azd.sh | bash;
curl -fsSL https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/193/install-azd.sh | bash -s -- --base-url https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/193 --version '' --verbose

Windows
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/193/uninstall-azd.ps1' > uninstall-azd.ps1; ./uninstall-azd.ps1;"
powershell -c "Set-ExecutionPolicy Bypass Process; irm 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/193/install-azd.ps1' > install-azd.ps1; ./install-azd.ps1 -BaseUrl 'https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/193' -Version '' -Verbose;"

Standalone Binary

Linux - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/193/azd-linux-amd64.tar.gz
MacOS - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/193/azd-darwin-amd64.zip
Windows - https://azuresdkreleasepreview.blob.core.windows.net/azd/standalone/pr/193/azd-windows-amd64.zip

Container
docker run -it azdevcliextacr.azurecr.io/azure-dev:pr-193


Install-OfficeClickToRun XML handling

Hi,
I have been looking into the Install-OfficeClickToRun script and have found some items which feel like issues, I am only relatively new to PowerShell so I could be wrong.
On execution I am passing a previously generated XML file from http://officedev.github.io/Office-IT-Pro-Deployment-Scripts/XmlEditor.html using the -ConfigurationXML parameter, the .xmlLoad method seems to be looking for a XML string rather than a file path. Once I tested this with the .load method instead, it seemed to work as intended. Is this by design?
if ($TargetFilePath) {
        $ConfigFile.Load($TargetFilePath) | Out-Null
    } else {
        if ($ConfigurationXml)
        {
>            $ConfigFile.LoadXml($ConfigurationXml) | Out-Null
            $global:saveLastConfigFile = $NULL
            $TargetFilePath = $NULL
        }
    }

This issue may well be linked to the one above, but when a XML config file path is passed as the -ConfigurationXML parameter, the configuration.xml file is then overwritten on line 157 with a new Configuration.xml file which contains the parameters path string. E.g. C:\temp\configuration.xml is overwritten and the only file content is "C:\temp\configuration.xml"
    if (!($TargetFilePath)) {
      if ($ConfigurationXML) {
         $TargetFilePath = $scriptRoot + "\configuration.xml"
>         New-Item -Path $TargetFilePath -ItemType "File" -Value $ConfigurationXML -Force | Out-Null
      }
    }

Thanks!

@SubThr33 can you try using the -TargetFilePath parameter with the path to your configuration xml file as the value instead of using the -ConfigurationXML parameter?

@alex182 Sorry for the delay.
I have just tested the install script using -TargetFilePath instead of -ConfigurationXML and it installs correctly.
Is this one by design?


azure-mgmt-resourcegraph 7.0.0 The resource type 'resources' could not be found in the namespace 'Microsoft.ResourceGraph' for api version '2020-04-01-preview'.

Package Name:
azure-mgmt-resourcegraph
Package Version:
7.0.0
Operating System:
Windows 10
Python Version:
3.7
Describe the bug
①Get resourcegraph_client as bellow
resourcegraph_client = ResourceGraphClient(       credential=credential,       base_url="https://management.chinacloudapi.cn/",       credential_scopes = ["https://management.chinacloudapi.cn/.default"]   )
②Build Query and send query
query = QueryRequest(           query='project id, tags, properties | limit 2',           subscriptions=[SUBSCRIPTION_ID]       )   query_response = resourcegraph_client.resources(query)
③ Call failed，error message like this
azure.core.exceptions.ResourceNotFoundError: (InvalidResourceType) The resource type 'resources' could not be found in the namespace 'Microsoft.ResourceGraph' for api version '2020-04-01-preview'. The supported api-versions are '2018-09-01-preview,2019-04-01'.

④ changed the api version，but still call failed
 resourcegraph_client._config.__setattr__("api_version","2019-04-01")
Please help troubleshoot  and solve this issue

Thanks for the feedback, we'll investigate asap.

Hi krista ,  is there any updates about this issue?

@chiragg4u could you take a look at this issue? thanks.

Hi  krista & @chiragg4u, is there any updates about this issue? Please help investigate this issue as soon as possible , Thanks

@msyyc could you take a look?

@RAY-316 Please help on this issue

Hi All，is there any updates about this issue? Please help investigate this issue as soon as possible , Thanks

Hi @RAY-316 @msyyc ，This issue has been going on for some days and there is still no further reply. Please help investigate this issue as soon as possible. thx.

@jackchen1680 Was it work when you used old version of SDK?
And can you try with this new released SDK:https://pypi.org/project/azure-mgmt-resourcegraph/

@00Kai0 yes, i tried use new released 8.0.0,  The error reported is almost the same as the error reported in version 7.0.0。
is work when use version 2.0.0, but because credential system has been completly revamped in new version 7.0.0 and 8.0.0。
so please help me how can i use resourcegraph_client(in V7.0.0  or V8.0.0) to query successfully.

@service team. According to the error message of the issue, it seems that new api-version '2020-04-01-preview' is not supported. The supported api-versions are '2018-09-01-preview, 2019-04-01'. But in swagger repo, '2020-04-01-preview' have been merged into 'master' for a long time. Could you help confirm that?

@msyyc @service is there any updates about this issue?

Hi @00Kai0 @msyyc @service，I haven't received any reply for long time， is there any updates about this issue?

@service team. According to the error message of the issue, it seems that new api-version '2020-04-01-preview' is not supported. The supported api-versions are '2018-09-01-preview, 2019-04-01'. But in swagger repo, '2020-04-01-preview' have been merged into 'master' for a long time. Could you help confirm that?

For Mooncake, we don't have the support for 2020-04-01-preview; we published latest api-version 2021-03-01 instead. Please use the 2021-03-01 api-version.


Link website logo to home page

current logo ( QuickWiKi ) is in text form, convert it to a link which redirect to home URL
Preview:

Please assign this to me.

Hello @kartik3805, Pls assign this issue to me.

@kartik3805, I would like to work on this issue. Please assign it to me.

@kartik3805 hello I had already fixed this issue and created a PR you can review it and  merged if possible

Issue solved