Fix ContentTypeId conflict

What is the purpose of the change
Fix Serialization ContentTypeId conflict between avro protocol and protocoluff protocol #3926
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #3967 into master will decrease coverage by 0.08%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master    #3967      +/-   ##
============================================
- Coverage     63.94%   63.86%   -0.09%
  Complexity       98       98
============================================
  Files           715      715
  Lines         31493    31494       +1
  Branches       5074     5075       +1
============================================
- Hits          20139    20114      -25
- Misses         9052     9071      +19
- Partials       2302     2309       +7

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/org/apache/dubbo/common/Constants.java
93.33% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../serialize/protostuff/ProtostuffSerialization.java
60% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../apache/dubbo/remoting/transport/CodecSupport.java
77.77% <0%> (-3%)
0 <0> (ø)

...dubbo/common/serialize/kryo/KryoSerialization.java
60% <0%> (ø)
0 <0> (ø)
:arrow_down:

...dubbo/common/serialize/java/JavaSerialization.java
80% <100%> (ø)
0 <0> (ø)
:arrow_down:

...mmon/serialize/hessian2/Hessian2Serialization.java
80% <100%> (ø)
0 <0> (ø)
:arrow_down:

...dubbo/common/serialize/avro/AvroSerialization.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

...mon/serialize/java/CompactedJavaSerialization.java
80% <100%> (ø)
0 <0> (ø)
:arrow_down:

.../serialize/nativejava/NativeJavaSerialization.java
80% <100%> (ø)
0 <0> (ø)
:arrow_down:

...mmon/serialize/fastjson/FastJsonSerialization.java
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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


Question

Good morning, I was looking at the repository and I have a question. Is this system functional?

I have the same question :-)


Code in tutorial not working

Thanks Cam for this great package, it is very useful.
I am currently going through the tutorial and I noticed something strange. When executing:
from lifetimes.plotting import plot_period_transactions
plot_period_transactions(bgf)
I get a chart which is significantly different from what you use in your page (it looks like a worse fit to me):

Do you know what can be the reason?
Also, one other minor problem: the pip install gives you a different version of your package, in which the file cdnow_customers_transactions.csv is not present.

Hmm, the algorithm in the background is random - perhaps you got unlucky? Try running it again, and please report back the findings.

I re-ran it several times, obtaining the same results.
I know it is unlikely, but can this strange behaviour be related to the fact that I am using Python 3.4 (while your tutorial seems to be in Python 2.7)?
I am also experiencing issues with this part of the tutorial:
from lifetimes.datasets import load_summary_data_with_monetary_value
summary_with_money_value = load_summary_data_with_monetary_value()
summary_with_money_value.head()
returning_customers_summary = summary_with_money_value[summary_with_money_value['frequency']>0]
returning_customers_summary.head()
I get an error:
ImportError: cannot import name 'load_summary_data_with_monetary_value'
When I try to import the file cdnow_customers_transactions.csv by hand, the result of returning_customers_summary.head() is:
frequency  recency      T  monetary_value
customer_id
1                    2    30.43  38.86           22.35
2                    1     1.71  38.86           11.77
6                    7    29.43  38.86           73.74
7                    1     5.00  38.86           11.77
9                    2    35.71  38.86           25.55
Which is consistent with the content of the file (but not with the tutorial).

I installed a separate Python 2.7 environment using Anaconda, and I still obtain the same graph. I wonder what the reason for this can be.

I can confirm I get the same thing, investigating now.

The first issue should be solved in https://github.com/CamDavidsonPilon/lifetimes/commit/3eda4116a4f6cfb1dbec6062cb5f7a92ebb0268e

I've upload this bug fix to PyPI too. I image the latest should also have cdnow_customers_transactions.csv - let me know if this is wrong.

Thank you!


How to install openssl 1.1.1n

I had try these steps:

search vcpkg install specify version parameter,  sadly no version parameter
create vcpkg.json,  use overrides  [ { "name": "openssl", "version-string": "1.1.1n" } ]   ,run  vcpkg install  and error occur:  Error: while checking out port openssl with git tree 7e4d802e3bde4154c227c0dd1da75c719be9f07a

The latest openssl version is 3.0.2. Do you really need version 1.1.1n?
You can check your current version via .\vcpkg search openssl.

Yes , I need version 1.1.1n ，  compile an open source project , that need version 1, not 3,  3 is not supported.

Maybe run git pull. Which version of vcpkg are you using?

It's latest vcpkg
git clone --depth 1 https://github.com/microsoft/vcpkg vcpkg
.\vcpkg\bootstrap-vcpkg.bat
echo '{ "name": "dbkg", "version-string": "1.0.0", "dependencies": [ "openssl", "fmt", "rapidjson", "keystone", "unicorn" ], "builtin-baseline": "2ac61f87f69f0484b8044f95ab274038fbaf7bdd", "overrides": [ { "name": "openssl", "version-string": "1.1.1n" } ] }' > vcpkg.json
.\vcpkg\vcpkg install
.\vcpkg\vcpkg integrate install

Hm that works on my machine

@muziling  you can use versioning to overwrite the openssl version to 1.1.1n, please refer to https://github.com/microsoft/vcpkg/issues/16967

@muziling you can use versioning to overwrite the openssl version to 1.1.1n, please refer to #16967

not support less than syntax  ,  "version<": "3.0.2" is illegal

please try to override the version:
  "overrides": [
    {
      "name": "openssl",
      "version": "1.1.1n"
    }
  ]

please try to override the version:
  "overrides": [
    {
      "name": "openssl",
      "version": "1.1.1n"
    }
  ]

version only support input number and dot.    'n' is illegal

git clone --depth 1 https://github.com/microsoft/vcpkg vcpkg

Ah now I see the problem! --depth 1. Don't pass --depth 1 and it will work.

For posterity, I ran into this problem even with the most recent commit. I had to use version-string instead of version since openssl didn't use a semver string (1.1.1n). More details here


Remove NO_MODULE from Eigen test

This task is just to not forget to apply the suggestion from this comment: https://github.com/conda-forge/cgal-cpp-feedstock/pull/2#discussion_r271609355

Since Eigen3 version 3.2, a config file Eigen3Config.cmake is shipped with Eigen3, and FindEigen3.cmake is no longer needed. It might be even harmful.
So NO_MODULE is perfectly fine with Eigen version 3.2 or later. This issue should be closed.


Add more events

Like "before-save", "after-save", ...

Hi @pquerner. I'm not sure why this would be useful?
after-save is already available, that's the on save event.
before-save isn't available. If you want something like that, you can easily save the last state of the preferences, and check during the save event what changed?


Typo in article

Hi, great article! I wanted to create a PR for the article itself, but there's a typo in secutiryCode (security vs secutiry) in the PaymentMethod example. Cheers

Hi @laurentvd, thanks for the issue! Feel free to open a PR to this repo and for the article repo. Let me know if you need anything!

Ah thanks @leandrotk, I didn't realize there was a repo for the articles as well. I've made the PR at https://github.com/leandrotk/tk/pull/83

Great, @laurentvd! Thanks for your contribution!
Feel free to open a PR for this repo too if you have any interest/time.


Infinite Compile Fix

To prevent the infinite compile, when we have transitive invalidations include recomplied classes in the next incremental compile round.
Resolves:

https://github.com/sbt/zinc/issues/1120
https://github.com/sbt/sbt/issues/6183

At least one commit author<EMAIL_ADDRESS>is not linked to a user. See https://help.github.com/en/articles/why-are-my-commits-linked-to-the-wrong-user#commits-are-not-linked-to-any-user


Redundant ButtonType parameter in Link button sample of the BitActionButton demo page

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Remaining ButtonType samples should be removed from BitActionButton from its demo page
Expected Behavior
No response
Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
No response
Anything else?
No response

@zahrabayatt Thanks for filing this issue. I've updated the title for more clarity.
thanks for the PR too 🚀


LL-6574 - Dates should be translatable - Intl setup improvement

Scope: https://github.com/LedgerHQ/ledger-live-mobile/pull/1855#discussion_r686072322
Type
Feature/Bug Fix

Context

Parts of the app affected / Test plan

i think we'll have to postpone this part for now.

i see it will collide with #1837
if we want to move to hermes this would have to change again i think

i see the votes was "1" & was probably underrated too. cc @LedgerHQ/live-squad-yellow

As decided during the yellow sprint meeting this task is postponed.

Hardly postponed.


Fixed typos in french translations

It said "Mount tabs" instead of "Show tabs".
I thought I'd fix it directly instead of creating an issue, I hope that's allright.

Thanks!


docker-compose build fail

Describe the bug
docker-compose build fail
unable to prepare context: unable to 'git clone' to temporary context directory: error fetching: /usr/lib/git-core/git-remote-https: /tmp/_MEIUV0ybH/libcrypto.so.1.1: version `OPENSSL_1_1_1' not found (required by /lib/x86_64-linux-gnu/libssh.so.4)

Screenshots

Desktop (please complete the following information):

OS: Ubuntu 20.04
docker: 20.10.7

Having the same issue. Any workaround for this?

Is this still an issue on the latest release @apmillen @xnng ?

Looks like a docker-compose issue: https://stackoverflow.com/questions/66579446/error-executing-docker-compose-building-webserver-unable-to-prepare-context-un

Does the issue go away if we set the following in the config:
commandTemplates:
  dockerCompose: "docker compose"

If so, this will be fixed by https://github.com/jesseduffield/lazydocker/issues/371

At any rate, given that this is not an issue with lazydocker, I'll close this issue


i3status: 2.12 -> 2.13

Update i3status to 2.13.
I needed to add a little hack to get it to build because for reasons
I can't seem to figure out, the configure script generates a broken
Makefile on this release.
The small sed line fixes the syntax error.
Motivation for this change
Wanted the new read_file feature for i3status
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @globin

@GrahamcOfBorg build i3status

@exi thanks!


[Feature][linkis-website]  Document outline optimization

Search before asking

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

Problem Description
Check all documents for uniform style. It is recommended that the title be prefixed with serial number.
For example, the following places are marked with different styles.

Description
No response
Use case
No response
Solutions
No response
Anything else
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

提供最新版DataSphereStudio全家桶安装服务。
包括（DataSphereStudio, Linkis, Exchangeis(Datax, Sqoop), Scriptis, Streamis(Flink), Visualis, DolphinScheduler)

另外包含 元数据，数据安全，数据质量，数据血缘。（这些都是DataSphereStudio不自带的服务）
包含hadoop集群搭建（hive, spark, flink, kafak, kylin）
提供上述所有系统二次定制开发服务
提供系统搭建运维疑难问题解决

添加备注，DSS大数据
持续为DataSphereStudio开源贡献力量


Removing "fail, error, fatal" verbiage not associated with error in python and C++ tests

From CI WTF discussion with @pgkeller

as per slack discussion:
>90% of the cases i'm seeing are from tests that are intentionally trying to trigger an error, so the message printed is the same as if there was actually a problem, gtest then catches it with EXPECT_THROW or something similar going to ignore these since idt i should be changing those error messages, and gtest currently does not have a way to filter or change verbosity from what i've seen

Yeah, this is a hard problem w/o a quick fix - it's architectural/philosophical: should the runtime print errors like this at all?  Or should that be up to the clients?  If errors are trappable/recoverable, the API shouldn't be printing them IMHO.


Disable Redfish collector when user or password is not passed

With the changes, Redfish configuration for
hardware-exporter-config won't be rendered when missing user or password.

@aieri I've created a new PR to target the first part that you mentioned about detecting and enable on #289.
I think there is enough changes to create a separate PR. Changing this to draft while that one is not merged

In the entire cluster, if only one unit (server) has an incorrect Redfish password, the original behavior would cause that unit to enter blocked mode and prevent the exporter from being installed. With this new implementation, we propose to continue the installation process but disable Redfish on that specific unit only. Is this the behavior you intend to propose?
Previously,  we have the debate whether certain components should be automatically disabled. There are some concerns

Users may be unaware of the situation. However, I see that your implementation includes sending a message to address this concern.
What about other special situations, such as intermittent network connection failures? We should be careful here, if we have more and more exception, it will make the situation become complex.
Will Redfish be dynamically disabled after the exporter been installed? Please avoid this behavior. Once enabled, metrics should be sent out continuously.

I like the behavior you proposed, it appears to give the maximum flexibility when installing hardware-observer. However, it may bring some potential issues that you have to handle it carefully and add more E2E testing.

Another question, did we consider to address #282 by a global setting to enable/disable redfish? The potential issue may be smaller and fullfill user's need.

Closing this PR because I've created a new one on #292


Can the primary regions in an image be  automatically generated?

Hello, I found the primary regions you use in the paper Contextual Action Recognition with R*CNN is annotated data. Is there some method that can automatically generate the primary regions in an input image?   if not, how can we recognize the actions in an input image without annotation information?

For the tasks tackled in our paper, the primary region always refers to the person that is being classified. Usually the boxes of the people are provided (in order to disentangle the task from the task of person detection). However, I do agree that in practice you don't have those boxes. If that is your case, you should probably run a person detector on your dataset and get the highest scoring activations to be the primary regions.


api: verbose response body error messages

Feature Request
Sometimes it's hard to understand why an API request fails, as in the body there are no details about what happened.
Would be useful to have a configuration setting to enable a verbose request body in these cases.
By default this should be off as it is not recommended in a production environment to give such details to the outside.
Which Areas Would Be Affected?
api
Why Is the Feature Desired?
To help debug
Solution Proposal
EdcApiExceptionMapper is a place where all the exceptions thrown in the api context are handled.
Should be enough to put the Result object into the EdcApiExtension and if a particular setting is enabled, the error detail could be included in the body.
Should worth how to make also jakarta-validation exceptions pass through this, as I don't think they currently do this (needs further investigation).
If the "verbose response body" setting is enabled, a warning at startup should be printed.
Type of Issue
improvement
Checklist

[x] assigned appropriate label?
[x] Do NOT select a milestone or an assignee!

@florianrusch-zf @tuncaytunc-zf FYI

I'm not sure about having a separate debug setting for this. If there is a problem, it seems the server should return enough information for the client to understand what the issue is, even in a production deployment.
Is there a concrete example we could walk through where the client would need more information than is appropriate to send back in a production environment?

@jimmarino the idea was to avoid giving information to eventual malicious third party systems about how to get their call more effective.

@jimmarino For example deleting a policy could lead in two cases to a 409 statuscode without any information. A not edc contributor person doesn't know in this case, why he receives a 409 because there is not log entry or any information in the response what's going wrong.
This case was already described here: https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/issues/1332


How do I use a cookie jar?

What versions are you running?
$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.7.2
$ google-chrome --version
Google Chrome 91.0.4472.164
$ go version
go version go1.16.6 linux/amd64

What did you do? Include clear steps.
I have a page that I visit, click on an "Accept Cookies" button, fill in an item number, then click a submit button. Before I hit that final submit button, however, I need to send a cookie that is sent in the "set-cookie" header on initial page load. I have the following, but it doesn't seem to be setting it correctly:
var actions = []chromedp.Action{
		chromedp.Navigate("https://www.example.com"),
		chromedp.WaitVisible(cookieAcceptButton),
		chromedp.Click(cookieAcceptButton, chromedp.BySearch),
		chromedp.WaitVisible(sel),
		chromedp.ActionFunc(func(ctx context.Context) error {
			cookies, err := network.GetAllCookies().Do(ctx)
			if err != nil {
				return err
			}

			for i, cookie := range cookies {
				exp := cdp.TimeSinceEpoch(time.Unix(int64(cookie.Expires), int64(1000)))
				err := network.SetCookie(cookie.Name, cookie.Value).
					WithExpires(&exp).
					WithPath(cookie.Path).
					WithDomain(cookie.Domain).
					WithHTTPOnly(cookie.HTTPOnly).
					WithSameSite(cookie.SameSite).
					WithPriority(cookie.Priority).
					WithSameParty(cookie.SameParty).
					WithSecure(cookie.Secure).
					WithSourceScheme(cookie.SourceScheme).
					WithSourcePort(cookie.SourcePort).
					Do(ctx)
				if err != nil {
					return err
				}
				log.Info.Printf("set chrome cookie %d: %+v", i, cookie)
			}

			return nil
		}),
		chromedp.SendKeys(sel, itemNumber),
		chromedp.Submit(sel),
	}

What did you expect to see?
I expected to see the cookie from the "Set-Cookie" header get set but it doesn't
What did you see instead?
Nothing.

Can you provide a minimal reproducer?
General speaking, The set-cookie header in a response will make the browser keep the cookie and send it in the following requests. So you don't need to call network.SetCookie.

If I am doing it wrong then I don't really see the point of doing a minimal reproducer. The set-cookie method you mention doesn't seem to work. When I set the header manually it seems to work (ala https://github.com/chromedp/examples/blob/9d5ce0471fadc51044b253ae824b9457f93e40be/headers/main.go)

If you want to know whether the set-cookie header works, you can test it directly in a browser (just start a browser, navigate to the url, see whether the set-cookie header is in the response; then check  whether the cookie is sent in following requests).
If it works, then test it with chromedp (write chromedp actions to do the same thing). If it works too, then there is nothing wrong. Otherwise, provide this demo so that we can check what's wrong.


Add JMX APIs to Java prelude

Currently, there is not straightforward way of defining a java.* package outside of Duchess

In general, how do we decide which Java APIs go into the Duchess prelude? So far we've got a rather arbitrary subset of the Java standard library. If we continued adding whichever APIs any user needs, would we eventually end up reflecting the entire Java standard library here?

The GarbageCollectorMXBean part looks good
I'd like the MemoryPoolMXBean interface exposing at least getType(), getUsage and getCollectionUsage, instead of the MemoryMXBean

A thought (not for this PR) is to have duchess export java.lang and java.util (or some set of known packages) and then have ability for other crates to do other stuff. Then we can even have duchess-java-foo crates too for convenience.
On Mon, Jun 3, 2024, at 2:30 PM, Russell Cohen wrote:

I think the long term solution is a way for folks to reflect java.* manually, unfortunately that's not currently possible
—
Reply to this email directly, view it on GitHub https://github.com/duchess-rs/duchess/pull/163#issuecomment-2145861734, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABF4ZQUGG5FAMI3SOWSD43ZFSY5LAVCNFSM6AAAAABITNCYYCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBVHA3DCNZTGQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

@rcoh Makes sense. Assuming that we provide a general way to reflect arbitrary java.* classes in the future, would we be OK doing a breaking change to remove the JMX stuff from the prelude?

maybe we will feature gate them or move them to a separate crate. However, I'd imagine for many folks access to the JMX APIs is going to be a generally useful feature.


Crashing when using the alchemical book

Hey,
I'm playing the modpack "Crazy-Craft-Updated" on version 0.10.0B which includes the Project Expansion mod.
I just got to the point where I could craft a master alchemical book but using it will result in the game crashing instantly.
Here's the log in short, I'm attaching a full version:
The game crashed whilst unexpected error Error: java.lang.IllegalAccessError: tried to access method net.minecraft.entity.LivingEntity.func_226268_ag_()Lnet/minecraft/util/math/BlockPos; from class cool.furry.mc.forge.projectexpansion.gui.GUIAlchemicalBook
crash-2023-11-13_12.38.46-client.txt
Edit: Minecraft Version is 1.16.5 and it appears to be happening with any alchemical book.

I can't reproduce this outside of that modpack, which has well over 300 mods
I'm not taking this on if it only happens within an environment very likely to have incompatibilities


windows paths are generated wrongly

This might need to be fixed in https://github.com/dotansimha/graphql-code-generator

seems to be fixed in 0.0.2


PEX sponge doesn't work

Hi!
I'm trying to run a sponge server with your plugin for permissions. But, I'm in trouble. Indeed, when I push the command I can see that the plugin (PermissionsEx) is loaded (green lightened), but the "/pex" command is unknown.
Report to this pic :
http://puu.sh/lRL5Q/e88b6aac79.png
I'm running the PermissionsEX-Sponge.jar (the lastet I can download on Jenkins).
Thanks for any help you could bring me to fix this issue. :)

In Sponge there is no distinction between enabled and disabled plugins -- so everything is green even if an error occurs while loading (this is a FML limitation).
Make sure you create the necessary files in config/permissionsex -> permissionsex.conf and permissions.json -- those aren't currently automatically created due to a bug.

Ho! Okay! thanks :)
But the cmd which doesn't work IG, is an issue due to this missing configuration(s) ?


Bill split CSS issue (native JS)

Describe the bug
CSS issue with navbar in "Bill split" challenge
Expected behavior
The navbar should look proper like in the other challenges
Screenshots

Possible reason
Conflicts with core CSS

@arpansaha13 hey Arpanaha could you please assign this to me

issue resolved you can check and merge it
https://deploy-preview-241--preeminent-tapioca-4294b2.netlify.app/machine-coding/bill-spilt/

Thanks a lot for fixing the issue
A maintainer will review and merge soon within today


Navigator.with(MainActivity.this).utils().isActualShowing("Fragment TAG") chrash

the application crashes when using the method isActualShowing ,when called several times quickly
(back pressed).
Thank you

update please ;)


User Preferences

Hello,
When I use OHIF Viewer I try to use the keyboard shortcuts and it does not work, then I go to Options, User Preferences and activate the shortcut keys.
It is always necessary to do this with each new study and this is not good.
Do you have this user preference setting already enabled by default? are set automatically without the user having to go to Options, Prefe ...

Yeah I noticed this too...


GCC machine descriptions

The .md file extension is used not only for markdown files, but also GCC machine descriptions.  There are about 111000 of those, so this language should qualify for addition.
The syntax is a Lisp-like variation of RTL, Register Transfer Language.
My intention is to open this issue for discussion, and later turn it into a PR if there are no objections.

It may be hard to find non-GPL sample files!

Write some. Fill it with sarcastic wisecracks. Become a god among men.

As luck would have it, I just remembered that I have written a GCC backend for the PDP-10.  It that doesn't make me a god, then I don't know what.  Anyway, I can add that under another license.

@arfon @pchaigno Please review.
The Bayesian classifier doesn't do well with a set of 1000 .md samples.  I added a heuristic: if a line begins with ;; or (define_, it's almost certainly a GCC machine description.  Almost anything else is a Markdown file.

The Bayesian classifier doesn't do well with a set of 1000 .md samples.

Did you get a chance to test the classifier + heuristic on those samples?

Did you get a chance to test the classifier + heuristic on those samples?

Of course.  There were only Markdown files in that set, so I tuned the heuristic until there were no false positives for GCC Machine Descriptions.

Ooh, 100% then, nice! :tada:

Well, some unusual Markdown files fall through the heuristic and lands in the Bayesian classifier, which doesn't always get it right.

Ok, here are the results from a set ot 999 .md files.  (Don't know where number 1000 went.)
| Language | Detected | Actual |
| --- | --- | --- | --- |
| Markdown | 993 | 996 |
| GCC Machine Description | 6 | 0 |
| Unknown| 0 | 3 |

@larsbrinkhoff - is this good to :ship:?

@arfon Yes.


Add support for propertyNamePrefix on bootstrap.properties/yaml

Is your feature request related to a problem? Please describe.
I often have to load multiple secrets from vault which can contain repeated property names (e.g. host property). To distinguish one property from another, I usually create a configuration bean with @VaultPropertySource and propertyNamePrefix.
It would be great if i could make this configuration on bootstrap.properties, inside spring.cloud.vault section, like I can do on Quarkus.
Describe the solution you'd like
Here is an example of my current solution:
@Configuration
@VaultPropertySource(value = "secrets/db", propertyNamePrefix = "db.")
@VaultPropertySource(value = "secrets/mail", propertyNamePrefix = "mail.")
public class VaultSecrets {
}

Here is an example of desired solution, based on Quarkus configuration:
quarkus:
  vault:
    url: ${vault_url}
    kv-secret-engine-mount-path: secrets
    kv-secret-engine-version: 2
    authentication:
      app-role:
        role-id: ${vault_role_id}
        secret-id: ${vault_secret_id}
    # Here is the prefix configuration. I can load secrets/db and secrets/mail configuring the prefix like secret-config-kv-path.{prefix}
    secret-config-kv-path:
      db: secrets/db
      mail: secrets/mail

where the propertyNamePrefix is any property inside secret-config-kv-path

That feature is available through org.springframework.cloud.vault.config.VaultConfigurer as workaround. We might consider extending the way how to declare mounts, also in combination with #459.

See #488 which introduced a prefix for individual Vault locations that is prepended to property names when using the Config Data API. We do not plan on extending bootstrap support as the Bootstrap Context is deprecated.


Search box does not work

Go to http://example.com/man/ and enter a name in the search box right to the text "Directly jump to manpage". This will lead to /man/jump, but this file does not exist on the server:
The requested URL /man/jump was not found on this server.

I haven't found documentation how to make the search work.

Which webserver are you using? I’m assuming you’ve seen https://github.com/Debian/debiman/tree/master/example.
Have your webserver strip the /man prefix, then reverse proxy the request to the auxserver.
Let me know whether this works out.

I used
	<Directory /srv/man>
		Require all granted

		# To set the correct Content-Type (e.g. text/html).
		RemoveType .gz
		AddEncoding gzip gz
		FilterDeclare gzip CONTENT_SET
		FilterProvider gzip inflate "%{req:Accept-Encoding} !~ /gzip,.*gzip/"
		FilterChain gzip
		Options +Multiviews
</Directory>

from https://github.com/Debian/debiman/tree/master/example. So I also need
	<Location /auxserver/>
		ProxyPass "http://localhost:2431/"
		ProxyPassReverse "http://localhost:2431/"
	</Location>

        ErrorDocument 404 /auxserver/%{REQUEST_URI}?%{QUERY_STRING}

and debiman-auxproxy running on the localhost:2431?

auxserver, not auxproxy.
I only configured nginx, so I can’t help out with an apache syntax fragment right now. Let me know if you can’t get it to work and I can take a look in the evening.

I added the apache settings and tweaked auxserver:
diff --git a/cmd/debiman-auxserver/aux.go b/cmd/debiman-auxserver/aux.go
index d8e6c33..1bc4c33 100644
--- a/cmd/debiman-auxserver/aux.go
+++ b/cmd/debiman-auxserver/aux.go
@@ -88,6 +88,10 @@ func main() {
        http.HandleFunc("/suggest", server.HandleSuggest)
        http.HandleFunc("/", server.HandleRedirect)

+       http.HandleFunc("/man/jump", server.HandleJump)
+       http.HandleFunc("/man/suggest", server.HandleSuggest)
+       http.HandleFunc("/man/", server.HandleRedirect)
+
        log.Printf("Loaded %d manpage entries, %d suites, %d languages from index %q",
                len(idx.Entries), len(idx.Suites), len(idx.Langs), *indexPath)

diff --git a/internal/aux/aux.go b/internal/aux/aux.go
index cf38923..e3a243d 100644
--- a/internal/aux/aux.go
+++ b/internal/aux/aux.go
@@ -11,6 +11,7 @@ import (
        "sort"
        "strings"
        "sync"
+       "log"

        "github.com/Debian/debiman/internal/manpage"
        "github.com/Debian/debiman/internal/redirect"
@@ -80,6 +81,7 @@ func (s *Server) redirect(r *http.Request) (string, error) {
 }

 func (s *Server) HandleRedirect(w http.ResponseWriter, r *http.Request) {
+       log.Printf("HandleRedirect: %q", *r)
        redir, err := s.redirect(r)
        if err != nil {
                if nf, ok := err.(*redirect.NotFoundError); ok {
@@ -115,10 +117,11 @@ func (s *Server) HandleRedirect(w http.ResponseWriter, r *http.Request) {
        // StatusTemporaryRedirect (HTTP 307) means subsequent requests
        // should use the old URI, which is what we want — the redirect
        // target will likely change in the future.
-       http.Redirect(w, r, redir, http.StatusTemporaryRedirect)
+       http.Redirect(w, r, "/man" + redir, http.StatusTemporaryRedirect)
 }

 func (s *Server) HandleJump(w http.ResponseWriter, r *http.Request) {
+       log.Printf("HandleJump: %q", *r)
        q := r.FormValue("q")
        if strings.TrimSpace(q) == "" {
                http.Error(w, "No q= query parameter specified", http.StatusBadRequest)
@@ -153,6 +156,7 @@ func (s *Server) suggest(q string) []string {
 }

 func (s *Server) HandleSuggest(w http.ResponseWriter, r *http.Request) {
+       log.Printf("HandleSuggest: %q", *r)
        q := r.FormValue("q")
        if strings.TrimSpace(q) == "" {
                http.Error(w, "No q= query parameter specified", http.StatusBadRequest)

Would be nice if auxserver would get a parameter for this prefix.

…why not remove the path prefix in Apache? That should work without any code changes.

How do I add the path again for the http.Redirect response?

Perhaps https://serverfault.com/a/690555 helps, if https://serverfault.com/q/561892 doesn’t work.

When using
		ProxyPass /auxserver/man/ "http://localhost:2431/"

I can use an unmodified cmd/debiman-auxserver/aux.go, but the ProxyPassReverse is not used by the redirect:
$ wget http://localhost:2431/jump?q=sl
--2017-08-14 13:43:20--  http://localhost:2431/jump?q=sl
Resolving localhost (localhost)... ::1, <IP_ADDRESS>
Connecting to localhost (localhost)|::1|:2431... failed: Connection refused.
Connecting to localhost (localhost)|<IP_ADDRESS>|:2431... connected.
HTTP request sent, awaiting response... 307 Temporary Redirect
Location: /stretch/sl/sl.6.en.html [following]

The following definition works for me with Apache 2.4.27 to serve manpages under the sub directory /sub:
<VirtualHost mansub.localhost:3080>
    ServerName mansub.localhost

    ServerAdmin webmaster@localhost
    DocumentRoot /srv/

    LogLevel alert

    ErrorLog ${APACHE_LOG_DIR}/error.log
    CustomLog ${APACHE_LOG_DIR}/access.log combined

    # Add gzip to the Accept-Encoding to prevent apache from serving an
    # HTTP 406 Not Acceptable response. We keep the original
    # Accept-Encoding value and later on use mod_deflate to uncompress if
    # necessary.
    RequestHeader set Accept-Encoding "expr=gzip,%{req:Accept-Encoding}" early

    ExpiresActive On
    ExpiresDefault "access plus 1 hours"

    <Files ~ "^rwmap">
        Order allow,deny
        Deny from all
    </Files>

    <Location /sub/auxserver/>
        ProxyPass "http://localhost:2431/"
        ProxyPassReverse "http://localhost:2431/"
        Header edit Location / /sub/
    </Location>

    <Location /sub>
        RewriteEngine on
        RewriteCond %{REQUEST_URI} "^/sub/(.*)"
        RewriteRule .* - [E=REWRITTEN_URI:%1]
        ErrorDocument 404 /sub/auxserver/%{env:REWRITTEN_URI}?%{QUERY_STRING}
    </Location>

    Alias "/sub" "/srv/man"
    <Directory /srv/man>
        Require all granted

        # To set the correct Content-Type (e.g. text/html).
        RemoveType .gz
        AddEncoding gzip gz
        FilterDeclare gzip CONTENT_SET
        FilterProvider gzip inflate "%{req:Accept-Encoding} !~ /gzip,.*gzip/"
        FilterChain gzip
        Options +Multiviews
    </Directory>

</VirtualHost>

For completeness, the following configuration works for me with nginx 1.13.3:
server {
        listen <IP_ADDRESS>:80;

        root /srv;

        keepalive_requests 10000;

        expires 1h;

        location /sub {
                # We cannot use try_files because then gzip_static always will
                # not be effective anymore.
                rewrite (.*)/$ $1/index.html;

                rewrite /sub/(.*) /$1  break;

                # We only have gzip-compressed files:
                gzip_static always;

                # Uncompress files for clients which do not support gzip:
                gunzip on;

                root /srv/man;
                error_page 404 = @auxserver;
        }

        location @auxserver {
                proxy_pass http://localhost:2431;
                proxy_redirect / /sub/;
        }
}

Actually, since we needed to add the -base_url flag to have correct URLs in the CSS, I decided to add path stripping/adding within debiman-auxserver in commit https://github.com/Debian/debiman/commit/35bbbaed1d14f491b4db5101979b7378d581695a
Hence, no rewriting is required in Apache/nginx anymore.

Thanks for the example apache/nginx config and for adding -base_url to debiman-auxserver.


Can't download manifest on a small screen (unless you zoom)

On a small screen, the download manifest submit button is off the bottom of the screen and it there is not scrolling. This prevents downloading unless one zooms out.
I would expect a scroll bar.

Thanks @diekhans I think this should be addressed in the upcoming changes to how we download data.
The specific screen is here:
https://chanzuckerberg.invisionapp.com/d/main#/console/17726124/367521677/preview
You can review the entire flow here:
https://chanzuckerberg.invisionapp.com/d/main#/console/17726124/367521669/preview

Covered by other changes for Get Data.

@MillenniumFalconMechanic confirmed this is covered by Get Data.

Thx @frano-m !

@diekhans we have verified this will be addressed by the upcoming get data flows. Will close this now and update you when the updated get data flows are deployed to dev.


Tracking: Keys that could use "friendly" value map

Is your feature request related to a problem? Please describe.
It would be helpful to provide annotations that are "friendly" - e.g. species : homo sapiens could have species_friendly: human.
Let's use this thread to track other keys that could use either standardized synonyms (e.g. RRID, ATCC id...) or "friendly" alternate values.
species
modelSystemName

This probably should be transferred to schematic repo as a feature request in how the spreadsheet options rendered vs how data is submitted.


Enable to change currency format from dollars to euros

Hello,
I want to show euros values on dashboard. But only the $ format are enable. When i changed the $ symbol to € i obtained an error.
How can I show euros value in a chart ?
I want something like "y_axis_format": "$,.2f" ==> to "y_axis_format": ",.2f€"
Regards,
Zied Sellami

Thank you, Zied for suggesting the enhancement. This feature is not currently on Superset 1.0 roadmap so we won't be able to get to it till early next year likely. However, this looks like a 'good first issue' for any community contributors to work on. If you are interested in contributing, I can point you to the right direction and connect you with you engineers.

Hey @junlincc, I am interested in contributing to this issue. Is it still available to be worked on?

Yup, go ahead and work on it!

Also with specifying a currency it would be nice to have the option to remove decimals eg. $,.2f becomes  $,.0f

Is this issue still open? I would like to help!

Is the issue still open? I would love to join and help out

@sahilkamboj3 I believe it is. If you open a PR, I can ensure it gets reviewed.

@carlosaccp, @sahilkamboj3 @dstadz sorry I missed your comments!!  If you all are still interested in contributing, feel free to reach out to me in Slack. you will need to add this y-axis format to multiple viz plug-ins. We can help you navigate if needed. thanks!


Updates lambda implementation to the terraform-aws-lambda module

Hi @phzietsman, thanks for this awesome tool!
This is a larger (opinionated) PR that refactors the module's lambda resources to make use of the terraform-aws-lambda module. This has the added benefit of including automated requirements.txt packaging as part of the terraform apply operation (more info here). Only caveat is that now python3.9 must be available in the client machine's PATH in order to  function properly.
Additional Changes

Removes layer references in favor of the automated terraform-aws-lambda requirements management which removes the requirement for the use of an external lambda layer and the region variable
Removes cloudwatch log group and IAM permissions associated with it in favor of the terraform-aws-lambda's handling of these resources.
Updates lambda python runtime from 3.8 to 3.9
Addresses issue #6 with the addition of the additional_iam_policy_statements input variable
Adds outputs.tf

userhas404d thanks a lot for this!


Add support to export & import Kibana objects

Docs: https://docs.logz.io/api/#operation/exportSavedObjects
Closes #69

Now we can apply the changes to the provider


Assign licenses to Microsoft 365 Groups

Dear MS
It would be great if we could assign licenses to a Microsoft 365 Group.
Regards,

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

ID: 7d1a3b25-8cbd-29f9-d205-31d320f6d315
Version Independent ID: 6711b6ba-154a-d078-d119-603d2c169453
Content: Assign licenses to a group - Azure Active Directory
Content Source: articles/active-directory/enterprise-users/licensing-groups-assign.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @curtand
Microsoft Alias: curtand

@NavishkarSadheo Thank you for your feedback . We will investigate and update the thread further.

@NavishkarSadheo You can assign license to M365 group as well. Let us know if you are having some issues while assigning it.
Microsoft 365 groups that have securityEnabled=TRUE are eligible for group based licensing.

I've read

@NavishkarSadheo You can assign license to M365 group as well. Let us know if you are having some issues while assigning it. Microsoft 365 groups that have securityEnabled=TRUE are eligible for group based licensing.

How? I've read this in multiple places, but there is no documentation on how to apply securityEnabled=TRUE  to a M365 group.


Fix custom playlist UI

Description

Related Issue

Motivation and Context

For playlists other than Watch Later, the UI doesn't work.
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Closing issues
Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

Hi @avallete, I’d be happy to help with maintaining this project if you’re open to it.


Islandora Manual installation hands on

Crayfish 2.0
Installing Prerequisites....
i receive an error "E: The repository 'https://packages.sury.org/php focal Release' does not have a Release file."
when i run "sudo add-apt-repository -y ppa:lyrasis/imagemagick-jp2
And i receive and error " [ErrorException]  Undefined index: splits "
when i run "sudo -u www-data composer install -d crayfish/Houdini"

@Frazer-Nyambe I do not know that much the manual installation process, but during the weekly Islandora Tech call (which anyone is welcome to attend, more meeting info here) the developers in the meeting wanted to details about your setup for installation.

What is the OS of your server set up?
Is it a virtual machine or is it your main computer?
What version of Islandora are you installing, from what repository?

I may not be the one to help you, but this information will help others that know more than me help you. Good luck

I can't speak to the imagemagick line without spinning up a new box, but there are some folks on Slack that came across that composer error and have a pull request to resolve it.


Mention CustomData parameter in Set-AzVMOperatingSystem cmdlet.

This document should be updated to mention that it is possible to set CustomData using the Set-AzVMOperatingSystem
cmdlet in the Az module.
https://docs.microsoft.com/en-us/powershell/module/az.compute/set-azvmoperatingsystem?view=azps-5.1.0

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

ID: 282b39fe-e8fe-f698-36c2-321f088f6554
Version Independent ID: 2a7c130c-291b-40ff-cae5-43255bcb3bc3
Content: Custom data and Azure Virtual Machines - Azure Virtual Machines
Content Source: articles/virtual-machines/custom-data.md
Service: virtual-machines
GitHub Login: @mimckitt
Microsoft Alias: mimckitt

@bwarren18 Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate. We will update you accordingly.

@mimckitt Can you please check and add your comments on this doc update request as applicable

The goal of this doc is to show how to set custom data during VM creation. Although as you found there are other ways to set it. I created a backlog item with the linux team to investigate if we want this added but for now we are keeping the document as is.
#please-close

The goal of this doc is to show how to set custom data during VM creation. Although as you found there are other ways to set it. I created a backlog item with the linux team to investigate if we want this added but for now we are keeping the document as is.
#please-close


delete_by_query with body

Delete by query with body like this:
client.delete_by_query index: 'test', body: { query: { match_all: {} } }
raises:
Elasticsearch::Transport::Transport::Errors::InternalServerError: [500] {"error":"ActionRequestValidationException[Validation Failed: 1: source is missing;]","status":500}
    /opt/boxen/rbenv/versions/2.0.0-p353/lib/ruby/gems/2.0.0/gems/elasticsearch-transport-1.0.4/lib/elasticsearch/transport/transport/base.rb:132:in `__raise_transport_error'
    /opt/boxen/rbenv/versions/2.0.0-p353/lib/ruby/gems/2.0.0/gems/elasticsearch-transport-1.0.4/lib/elasticsearch/transport/transport/base.rb:224:in `perform_request'
    /opt/boxen/rbenv/versions/2.0.0-p353/lib/ruby/gems/2.0.0/gems/elasticsearch-transport-1.0.4/lib/elasticsearch/transport/transport/http/faraday.rb:20:in `perform_request'
    /opt/boxen/rbenv/versions/2.0.0-p353/lib/ruby/gems/2.0.0/gems/elasticsearch-transport-1.0.4/lib/elasticsearch/transport/client.rb:102:in `perform_request'
    /opt/boxen/rbenv/versions/2.0.0-p353/lib/ruby/gems/2.0.0/gems/elasticsearch-api-1.0.4/lib/elasticsearch/api/actions/delete_by_query.rb:71:in `delete_by_query'
when i do the same with source like this:
client.delete_by_query index: 'test', source: '{ "query": { "match_all": {} } }'
everything works fine.

@karmi don't have the chance to test it anymore - it was during a internship. sry.

@simonmulser No worries, thanks for the report!


[TEMPORARY EXCEPTION MESSAGE] FileLoadErrorGeneric: System.Drawing.Common

Good day!
I run: dotnet publish -r win-x64 -c release
Console application
.Net Core version: 2.2
And I get such an error ... What can I do about it?
Generating native code
EXEC : error : [TEMPORARY EXCEPTION MESSAGE] FileLoadErrorGeneric: System.Drawing.Common

I solved the problem. It turned out that such a reaction to the System.Drawing.Common package and the Nlog 4.5.2 package, which were in dependencies of other packages.


New FOM API for FSP TS

Describe the task
New API.
The API will be public accessible and allow to search for FOM related to (by) FSP ID. Apply naming convention to designate this API as used by external parties.
Acceptance Criteria

[ ] New API is built with search params available (FSP ID)
[ ] Add Auth guard if needed.
[ ] Add/Create/Adjust tests as needed.

Additional context

Design discussion around what the API returns: existing ProjectPublicSummaryResponse as result, or with new response object for limited information.

FSP TS FOM API Request.pdf

Hi @basilv , just to confirm with you to see if the endpoint "path" and return json fit your expectation.

This is to add endpoint into current project.controller.ts.
Although I am thinking to separate it out of "project" into something like GET /api/fsp-tracker without "/project", that will need another controller class (which I feel is fine but a bit more work).

Yes, I want it completely separate to make it clear it is used by external... I haven't decided on a good name (feel free to suggest), but definitely a separate controller class.

Verified on TEST environment, it is good.

I sent an email to Viktor Novoselov to ensure the API meets their requirements. Anticipating to hear back from Viktor on Monday.

Confirmed from FSP TS the api fit the requirement. Close this ticket.


启动报错：ERROR: Invalid interpolation format for "searxng" option in service "services": "SEARXNG_BASE_URL=https://${SEARXNG_HOSTNAME:-localhost}/"

启动报错：ERROR: Invalid interpolation format for "searxng" option in service "services": "SEARXNG_BASE_URL=https://${SEARXNG_HOSTNAME:-localhost}/"
是有要启动其他的服务吗

查一下docker版本是不是太老

同样的问题，更新docker版本没解决

同样的问题 Macbook Air M1 + macOS 14

same question

i

solved using latest version of docker

same question

solved using latest docker version

same question

solved using latest docker version

which version of docker you are using?

同样的问题，大佬们怎么解决的？
Docker version 24.0.2, build cb74dfc
docker-compose version 1.17.1, build 6d101fb

same question

solved using latest docker version

which version of docker you are using?
mine: Docker version 26.0.0, build 2ae903e docker-compose version 1.25.0, build unknown

that's it

Invalid interpolation format for "searxng" option in service "services": "SEARXNG_BASE_URL=https://${SEARXNG_HOSTNAME:-localhost}/"
也是这个问题，请问怎么解决

https://docs.docker.com/engine/install/ubuntu/ use docker official instruction，upgrade to the latest version, for me is 26.0, problem fixed

i upgrade to 26.0.1, the problem still exists

After changing docker-compose to docker compose, the error was gone
docker compose up -d
docker server version: 23.0.5
docker compose version: 2.17.3
docker-compose version: 1.25.0


Upgrade Linux Firmware in main from 20240220 to 20240312

Subject says it all.

Build started

Build succeeded, the hw coverage has to be done in the release tests.


Resolve object replication for classes with 3rd party dependencies

There are several issues with shogun classes that depend on 3rd party libraries, e.g. CMKL or CLibSVM. The named two currently don't have any of their members registered as parameters and for the non-shogun ones (LibSVM, CLPLEX, GLPK, ...) it has to be checked upstream how to implement this first.

Check all shogun classes for Parameter registration. Test cases for clone() and equals() exist for CLibSVM etc, but are useless, since they always check out fine with no parameters!
Check 3rd party dependencies and how to replicate their objects.

The member variables in LibSVM classes could actually be moved to the train_machine scope, so maybe there is not much to do there.

I prepared a patch for MKL in #3730

And a patch for LibSVM classes in #3731


ISPN-10945 Remove protobuf and protoparser dependencies

https://issues.jboss.org/browse/ISPN-10495

LGTM. These deps were never needed. They same to have been added during a POM reorg. Perfectly fine to get rid of them.

integrated in master. thanks @pruivo !


[TwigComponent] Add documentation about passing blocks to embedded components

Q
A

Bug fix?
no

New feature?
no

Tickets
Fix #...

License
MIT

This adds documentation of the feature added in #920

Hey! I just did a quick proof and edit of the docs so far - a team effort to explain these important concepts on a simple way. I've tried to remove the wording "embedded component". Even I mix up "embedded" and "nested" in my mind. It's not as succinct, but saying "when you pass a block to a component" instead of "when you render an embedded component" I think is much more clear.

Hey! I just did a quick proof and edit of the docs so far - a team effort to explain these important concepts on a simple way. I've tried to remove the wording "embedded component". Even I mix up "embedded" and "nested" in my mind. It's not as succinct, but saying "when you pass a block to a component" instead of "when you render an embedded component" I think is much more clear.

Love it. Indeed I also always have to think twice to correctly separate nested vs embedded components. It took a while to get this terminology right in my head, but it will take perhaps equally long to stop referring to them as embedded components :-)
Perhaps on a technical level, calling them that is actually still the most correct, especially seeing their similarity to the embed syntax, and even more so since the ComponentNode is in fact extending the EmbedNode. 🤓
But again, I'm all for simplicity and even more for clarity in the documentation!

Perhaps on a technical level, calling them that is actually still the most correct

I think so yes. So we may sometimes still use that term - but let's see if we can avoid it for the "outside" world :)

@weaverryan As discussed on Slack, I also allowed references to outer blocks from with a regular (non-nested) component.
Note that these blocks probably only make sense within an extending template, because otherwise the block would be rendered where it is defined ànd where it is referred as an outerblock. It even that might be desirable in some specific use cases, who knows 🤷


Roll Engine from 2e5833b7c572 to 72dddb47b44e (19 revisions)

https://github.com/flutter/engine/compare/2e5833b7c572...72dddb47b44e
2021-01-08<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from cLR0CUb-e... to IGtSAREVb... (flutter/engine#23509)
2021-01-08<EMAIL_ADDRESS>Skip the FlEngineTest.SettingsPlugin test (flutter/engine#23516)
2021-01-07<EMAIL_ADDRESS>Update virtual to take sampling (flutter/engine#23476)
2021-01-07<EMAIL_ADDRESS>add arm host builder (flutter/engine#23440)
2021-01-07<EMAIL_ADDRESS>[fuchsia][input] Migrate Flutter to "input3" (flutter/engine#23262)
2021-01-07<EMAIL_ADDRESS>Fix argument specifier for g_warning() (flutter/engine#23138)
2021-01-07<EMAIL_ADDRESS>disable UnassignedIdsAreReused flaky test (flutter/engine#23511)
2021-01-07<EMAIL_ADDRESS>[tests] Normalize SkImage before comparison. (flutter/engine#23489)
2021-01-07<EMAIL_ADDRESS>started sharing GPU contexts between spawned engines (flutter/engine#23435)
2021-01-07<EMAIL_ADDRESS>[web] Draw shadows for text in rich paragraphs (flutter/engine#23470)
2021-01-07<EMAIL_ADDRESS>Provide a runtime switch for selecting SkParagraph text layout (flutter/engine#23474)
2021-01-07<EMAIL_ADDRESS>Roll Skia from d64a3193cd49 to be0b3b7363a9 (3 revisions) (flutter/engine#23487)
2021-01-07<EMAIL_ADDRESS>Roll Dart SDK from 1c14d9bb3528 to 6a6a854523fd (21 revisions) (flutter/engine#23486)
2021-01-07<EMAIL_ADDRESS>[macos] Re-land FlutterOpenGLRenderer isolation (flutter/engine#22607)
2021-01-07<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from 5jKxFxRiQ... to qedaMC5nT... (flutter/engine#23484)
2021-01-07<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from de_114mvQ... to cLR0CUb-e... (flutter/engine#23482)
2021-01-07<EMAIL_ADDRESS>Roll Skia from a7b7964a237a to d64a3193cd49 (5 revisions) (flutter/engine#23479)
2021-01-07<EMAIL_ADDRESS>Roll Skia from 417743f806d1 to a7b7964a237a (19 revisions) (flutter/engine#23464)
2021-01-07<EMAIL_ADDRESS>Call JavaVM::AttachCurrentThread only once per thread (flutter/engine#23188)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from 5jKxFxRiQQBZ to qedaMC5nTQFw
fuchsia/sdk/core/mac-amd64 from de_114mvQcHR to IGtSAREVbeqo
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Closing in favor of a new roll with dart fixes

Closing in favor of a new roll with dart fixes

Commit queue failed; closing this roll.

Commit queue failed; closing this roll.


Install Sveltos as a Provider Template

Description
This fixes #269.

Sveltos is installed (via it's helm chart) as a provider type template.
The reason it is installed as a separate template rather than as a subchart within the HMC template is because Sveltos needs to be installed in the projectsveltos namespace for it to function.
We already tried installing Sveltos in hmc-system namespace but it didn't work properly, the details of which can be seen here.

Manual Testing
Installed with AWS provider and waited for everything to be in running state.
Management Cluster

Verified that Sveltos template has been created:

➜  ~ kubectl get template -A
NAMESPACE    NAME                         TYPE         VALID
hmc-system   aws-hosted-cp                deployment   true
hmc-system   aws-standalone-cp            deployment   true
hmc-system   azure-hosted-cp              deployment   true
hmc-system   azure-standalone-cp          deployment   true
hmc-system   cluster-api                  core         true
hmc-system   cluster-api-provider-aws     provider     true
hmc-system   cluster-api-provider-azure   provider     true
hmc-system   hmc                          core         true
hmc-system   k0smotron                    provider     true
hmc-system   projectsveltos               provider     true

Verified all HelmReleases have been successfully reconciled:

➜  ~ kubectl get helmreleases -A
NAMESPACE    NAME                         AGE   READY   STATUS
hmc-system   cluster-api                  15m   True    Helm install succeeded for release hmc-system/cluster-api.v1 with chart<EMAIL_ADDRESS>hmc-system   cluster-api-provider-aws     15m   True    Helm install succeeded for release hmc-system/cluster-api-provider-aws.v1 with chart<EMAIL_ADDRESS>hmc-system   cluster-api-provider-azure   15m   True    Helm install succeeded for release hmc-system/cluster-api-provider-azure.v1 with chart<EMAIL_ADDRESS>hmc-system   hmc                          15m   True    Helm upgrade succeeded for release hmc-system/hmc.v2 with chart<EMAIL_ADDRESS>hmc-system   k0smotron                    15m   True    Helm install succeeded for release hmc-system/k0smotron.v1 with chart<EMAIL_ADDRESS>hmc-system   projectsveltos               15m   True    Helm install succeeded for release projectsveltos/projectsveltos.v1 with chart<EMAIL_ADDRESS>hmc-system   wali-aws-dev                 10m   True    Helm install succeeded for release hmc-system/wali-aws-dev.v1 with chart<EMAIL_ADDRESS>

Verified that components of Sveltos can be seen running in the projectsveltos namespace:

➜  ~ kubectl get deployments -A
NAMESPACE            NAME                                          READY   UP-TO-DATE   AVAILABLE   AGE
hmc-system           azureserviceoperator-controller-manager       1/1     1            1           12m
hmc-system           capa-controller-manager                       1/1     1            1           13m
hmc-system           capi-controller-manager                       1/1     1            1           14m
hmc-system           capz-controller-manager                       1/1     1            1           12m
hmc-system           helm-controller                               1/1     1            1           15m
hmc-system           hmc-cert-manager                              1/1     1            1           15m
hmc-system           hmc-cert-manager-cainjector                   1/1     1            1           15m
hmc-system           hmc-cert-manager-webhook                      1/1     1            1           15m
hmc-system           hmc-cluster-api-operator                      1/1     1            1           14m
hmc-system           hmc-controller-manager                        1/1     1            1           15m
hmc-system           k0smotron-controller-manager-bootstrap        1/1     1            1           13m
hmc-system           k0smotron-controller-manager-control-plane    1/1     1            1           13m
hmc-system           k0smotron-controller-manager-infrastructure   1/1     1            1           12m
hmc-system           source-controller                             1/1     1            1           15m
kube-system          coredns                                       2/2     2            2           16m
local-path-storage   local-path-provisioner                        1/1     1            1           16m
projectsveltos       access-manager                                1/1     1            1           13m
projectsveltos       addon-controller                              1/1     1            1           13m
projectsveltos       classifier-manager                            1/1     1            1           13m
projectsveltos       conversion-webhook                            1/1     1            1           13m
projectsveltos       event-manager                                 1/1     1            1           13m
projectsveltos       hc-manager                                    1/1     1            1           13m
projectsveltos       sc-manager                                    1/1     1            1           13m
projectsveltos       shard-controller                              1/1     1            1           13m
projectsveltos       sveltos-agent-manager                         1/1     1            1           12m

Verified that Sveltos's default classifier has been installed:

➜  ~ kubectl get classifier
NAME                 AGE
default-classifier   13m

Labeled the target cluster with env=fv:

➜  ~ kubectl get cluster -A --show-labels
NAMESPACE    NAME           CLUSTERCLASS   PHASE         AGE   VERSION   LABELS
hmc-system   wali-aws-dev                  Provisioned   11m             app.kubernetes.io/managed-by=Helm,env=fv,helm.toolkit.fluxcd.io/name=wali-aws-dev,helm.toolkit.fluxcd.io/namespace=hmc-system,sveltos-agent=present

Finally created the following cluster profile:

apiVersion: config.projectsveltos.io/v1beta1
kind: ClusterProfile
metadata:
  name: kyverno
spec:
  clusterSelector:
    matchLabels:
      env: fv
  helmCharts:
    - repositoryURL:    https://kyverno.github.io/kyverno/
      repositoryName:   kyverno
      chartName:        kyverno/kyverno
      chartVersion:     v3.2.5
      releaseName:      kyverno-latest
      releaseNamespace: kyverno
      helmChartAction:  Install

➜  ~ kubectl get clusterprofile
NAME      AGE
kyverno   118m

Target Cluster
As can be seen below, Sveltos is running in projectsveltos namespace and kyverno has also been installed in the target cluster.
➜  ~ kubectl get deployments -A
NAMESPACE        NAME                            READY   UP-TO-DATE   AVAILABLE   AGE
kube-system      calico-kube-controllers         1/1     1            1           7m33s
kube-system      coredns                         2/2     2            2           7m37s
kube-system      ebs-csi-controller              2/2     2            2           7m17s
kube-system      metrics-server                  1/1     1            1           7m28s
kyverno          kyverno-admission-controller    1/1     1            1           40s
kyverno          kyverno-background-controller   1/1     1            1           40s
kyverno          kyverno-cleanup-controller      1/1     1            1           40s
kyverno          kyverno-reports-controller      1/1     1            1           40s
projectsveltos   sveltos-agent-manager           1/1     1            1           7m3s

@Kshatrix @a13x5 Included an empty helm chart and added projectsveltos chart as its dependency and rebased from main branch to pull in changes made in https://github.com/Mirantis/hmc/pull/287.


Update documentation for new features

          We should add documentation for these new endpoints.

Originally posted by @SimonJnsson in https://github.com/Morning-Train/economic/pull/8#pullrequestreview-1870268411

I have added documentation in the README


MultiGPU training + changes to Checkpointing logic

This is WIP to rework checkpointing and the multiGPU training in Renate.
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

There are some clean-ups that are not part of this PR.

Most of the set_* methods can be eliminated. We no longer use these to restore checkpoints.
components need to be handled with a little more care. See #236 .

We need to check the doc strings. I saw that the RenateModule doc string still contains the loss_fn argument. Can you make sure that all doc strings reflect the change? I.e., remove it from RenateModule and its subclasses and add it to Learner and its subclasses.


Pin ubuntu version in azure-pipelines

Summary
On December 15th the runner image for ubuntu-latest upgrade to ubuntu 22.04. Since then we've started seeing job timeouts in the docs builds. In attempt to debug and potentially prevent this (in the short term) failure mode this commit pins the ubuntu image version used to the previous value for ubuntu-latest 20.04. We can use this commit to test CI to see if the timeouts are reproducible and if we do not encounter the docs job timeout after multiple attempts we can merge this to hopefully prevent it in CI.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.03%) to 84.634%

Files with Coverage Reduction
New Missed Lines
%

src/sabre_swap/layer.rs
2
98.95%

qiskit/pulse/library/waveform.py
3
91.49%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
64119

Relevant Lines:
75760

💛  - Coveralls

This did not fix the hanging docs build, we'll have to look for another root cause


[SFS] Allow upper-level hirarchy for CRF-sectors when selecting CRF-sector division

Description of the issue
Selecting CRF sector division in field S7 one is confronted with a myriad of individuals of CRF sectors in the ontology which is far to detailed for most of the users.
Factsheet
Study
Field
S7: Sector divisions
S8: Sectors
Ideas of solution
Add the existing sector/sub-sector hirarchy for CRF-sector individuals (implemented as has-part-/part-of-relations) in the factsheet so that users don't have to go through all 100 plus individuals.

The issue slipped my attention, sorry. Your suggestion would defenitley make sense for me.

@l-emele can you please review the current state of the sectors and sector devisions in oekg? I think you know the structure best.
For me, it looks like still not all sector divisions are selectable.
Also, the questions of structuring the sectors, as is the intention of this issue. The idea of subsectors sounds good, but does not seem to work according to your comment  here?!

The idea of subsectors sounds good, but does not seem to work according to your comment  here?!

I implemented a potential solution and it seems to work!

@l-emele can you please review the current state of the sectors and sector devisions in oekg? I think you know the structure best.
For me, it looks like still not all sector divisions are selectable.

I had a quick look:

Currently, we have seven sector division classes and seven sector division individuals in the OEO, however in the bundle only four sector divisions are selectable.
For NC/BR sector division no sector is selectable.


VAnchor Substrate Withdraw and Transfer

Summary of changes

Fix Tangle script error due to balances.ExistentialDeposit error
Commitments setup, fetching leaves for Polkadot Provider
Implement VAnchor withdraw on the substrate side
Implement VAnchor transfer on the substrate side
UI crash in the Avatar component when no relayer is available
Fix should filter available tokens by token symbol and target typed chain id from notes.

Proposed area of change

[x] apps/bridge-dapp
[ ] apps/stats-dapp
[ ] apps/webbsite
[ ] apps/faucet
[ ] apps/tangle-website
[x] libs/webb-ui-components

Reference issue to close (if applicable)

Closes NaN

Screen Recording
https://user-images.githubusercontent.com/60747384/236666654-fe41d2a4-ee12-4be4-9dc2-8b27da7fe06e.mp4
Next steps to support Substrate <> EVM transactions:

There are still errors present in the withdraw and transfer flows of the local Tangle. Specifically, when constructing the merkle tree using the leaves returned by api.query.merkleTreeBn254.leaves(), the root of the created tree does not match the root returned by api.query.merkleTreeBn254.trees().root.
https://github.com/webb-tools/webb-dapp/issues/1158
Testing again with live Tangle and live Relayers.

@AtelyPham how is this going? Looks like it's possibly going stale?

This is still blocked by the Circom proving work. I'll update this PR with the latest changes.

Depends on https://github.com/webb-tools/webb-dapp/pull/1361.

@AtelyPham what's the ETA for this?

Awesome work @AtelyPham !


Inspect doesn't show anything in Android Studio

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Open Android Studio
import 'dart:developer';
use inspect(); in code

Expected results
That Android Studio Console would show inspected element for deeper investigation
Actual results
There is no sign in Android Studio about inspection. I am aware that most people use print in 90% cases, and am also aware of debug option, but I have cases (mostly) when I need to see detailed information about object.  Pausing on breakpoint isn't always an option.
That is the biggest feature I miss from going from VSC to Android Studio. Of course Flutter DevTools does show inspect in its console, but it would be much more convenient to have it embedded in IDE itself. The best scenario is already existing Console, acceptable scenario - if there is some way to add DevTools console into Android Studio IDE.
Code sample
Code Sample

import 'package:english_words/english_words.dart';
import 'package:flutter/material.dart';
import 'package:provider/provider.dart';
import 'dart:developer';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return ChangeNotifierProvider(
      create: (context) => MyAppState(),
      child: MaterialApp(
        title: 'Namer App',
        theme: ThemeData(
          useMaterial3: true,
          colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepOrange),
        ),
        home: MyHomePage(),
      ),
    );
  }
}

class MyAppState extends ChangeNotifier {
  var current = WordPair.random();
}

class MyHomePage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    var appState = context.watch<MyAppState>();
    inspect(appState);

    return Scaffold(
      body: Column(
        children: [
          Text('A random idea:'),
          Text(appState.current.asLowerCase),
        ],
      ),
    );
  }
}

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
flutter doctor

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.9, on macOS 14.0 23A344 darwin-arm64, locale en-LV)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.84.2)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

Thanks for filing this @ebelevics-midis.
I can see this as well but I think this issue will be best suited/filed in this repo: https://github.com/flutter/flutter-intellij/issues
Kindly file this issue there.
I'll be closing this from here per this, if you disagree, please comment and I'll reopen this.
Thank you!


Have warnings optional

Would it possible to have the creation of Cargo warnings mapped from Conan warnings be optional?
Sometimes this isn't under our control, i.e. we can't do anything about them, and they add noise to the build output.
Happy to submit a PR.

I think it's a good idea, but rather than dropping them in the build script I'd prefer to silence them at the Conan side via -verror.
Currently conan install is run with -vwarning (hardcoded).
I'll add a new builder method for the Conan verbosity configuration. I expect to do this and release a new version tomorrow or later this week.

Sounds great, thanks!


Library does not work in .NET Core 2.1

I create a .NET Core 2.1 console app and followed the examples at:

https://docs.microsoft.com/en-us/azure/app-service-mobile/app-service-mobile-windows-store-dotnet-get-started
https://docs.microsoft.com/en-us/azure/app-service-mobile/app-service-mobile-dotnet-how-to-use-client-library

My code is:
var client = new MobileServiceClient(MobileAppUrl);

// Create a typed table reference
IMobileServiceTable<TodoItem> todoTable = client.GetTable<TodoItem>();

// TODO : this fails on NETCore 2.1 ?

// This query filters out completed TodoItems
// http://***.azurewebsites.net/tables/TodoItem?$filter=not(complete)
List<TodoItem> items = await todoTable.Where(todoItem => !todoItem.Complete).ToListAsync();
Console.WriteLine($"item count = {items.Count}");
foreach (var item in items)
{
    Console.WriteLine($"{item.Text}");
}

However I get an exception:

The server did not provide a response with the expected content.

This StackOverflow Issue also defined this problem.
Problem is in this code:
https://github.com/Azure/azure-mobile-apps-net-client/blob/70d79ce835881d62a8a5a9399aa1175f1457bd85/src/Microsoft.Azure.Mobile.Client/Http/MobileServiceHttpClient.cs#L620
Could it be that gzip response compression is not handled by .NET Core 2.1 ???
Response in fiddler looks like:
HTTP/1.1 200 OK
Cache-Control: no-cache
Pragma: no-cache
Content-Length: 326
Content-Type: application/json; charset=utf-8
Content-Encoding: gzip
Expires: 0
Vary: Accept-Encoding
Server: Microsoft-IIS/10.0
X-Powered-By: ASP.NET
Set-Cookie: ARRAffinity=317fc761243dff568147bab1a7c8eb4e658734b4a4080b4361c90b19a1fcc566;Path=/;HttpOnly;Domain=***.azurewebsites.net
Date: Thu, 18 Oct 2018 08:55:40 GMT

         `I %&/m {J J  t  `$ؐ@      iG#) *  eVe]f@ 흼  {   {   ; N'   ?\fdl  J ɞ!   ?~|?"   ?  e 泏 ge  >Z f } ~ 製

*** FIDDLER: RawDisplay truncated at 128 characters. Right-click to disable truncation. ***

@StefH We don't support .NET Core 2.1 in the Azure Mobile Apps SDK. We are not doing active development in the product, so we won't add any new features. I will close the issue for now but feel free to contact us if you have any questions.

Thanks for the information. Can you tell which new project will have this functionality?

We are in the very early stages of customer research to design the new services, so it's too soon to say what product would be the new home for these services.
Please stay tuned with updates on our documentation and readme. We will share more information when we are ready.

@elamalani - do you have any understanding of why a change from .NET Core 2.0 to .NET Core 2.1 would cause gzip response compression to fail in Azure Mobile Apps?   If you could give us any kind of clue as to what went wrong, then maybe we could seek a workaround?

Hi @JohnPenney,
As @elamalani pointed out above, we don't support .Net Core 2.1 in the SDK today. Unfortunately, we don't have a workaround today and like Ela said, the we don't have an ETA for a new release or other SDK to use for .Net Core 2.1 compatibility yet, so your only workaround is to stick with .Net Core 2.0 for now.

the issue is now here with some workarrounds.
https://github.com/Azure/azure-mobile-apps-net-client/issues/514


Syntax Error on Load

When I try to load the import page in admin I get:
Parse error: syntax error, unexpected '[' in .../craft/plugins/import/services/Import_EntryService.php on line 236

What PHP version are you running?


Go Fiber Support / Example?

Corresponding ticket on Go Fiber's side: https://github.com/gofiber/template/issues/302
Based on some of the conversation going on over there, it looks like there are a few different approaches to integrating both Templ and Fiber. Since there are already examples of Gin and Chi, maybe one could be added for Fiber?

I think that since the Fiber context type implements io.Writer, it should be a case of:
app.Get("/", func(c *fiber.Ctx) error {
  return templConponent("hello").Render(c.Context(), c)
})

But I'm just typing this on a mobile, haven't had chance to try it!

I created a helper that does this:
func Render(c *fiber.Ctx, component templ.Component, options ...func(*templ.ComponentHandler)) error {
	componentHandler := templ.Handler(layout.Main(component))
	for _, o := range options {
		o(componentHandler)
	}
	return adaptor.HTTPHandler(componentHandler)(c)
}

This way you can use your render-function like this, which I found easiest:
app.Get("/", func(c *fiber.Ctx) error {
  return view.Render(c, MyTemplView())
})

I think we can consider this closed. We've got a good example now, thanks to @bastianwegge.