Switch to react-helmet-async

This is a proposition more than anything, this is very optionnal.
I feel like switching to react-helmet-async would be beneficial and follows the usage trend.
Helmet Trend
Personally, I use react-helmet-async, and using this plugin meant having both versions of Helmet as dependencies.

Hi, thanks for the PR, but I might consider this for the future, since I don't see a particular benefit or use-case for switching the libraries.

Hi, thanks for the PR, but I might consider this for the future, since I don't see a particular benefit or use-case for switching the libraries.


Want to add EDA on Transaction fraud detection

Sure @Nitin-Diwakar

Hey can you please assign this task to me


doc: refresh the components documentation

Updated the comment usage and do some minor adjustements.

:+1:


The width of grades can exceed a single character and may even be non-numeric.

While mentored by Filbo, he revealed to me the idea of generalizing the code to accept grades with width more than one character. I checked it to accept also non-numeric grade values. I adjusted my code to pass this test, and I changed one of the testes to do this kind of test.

the tests come from the exercism/problem-specifications repo: changes to existing tests should be discussed there.

each exercise has an example solution to verify that the exercise can be solved: does your proposed test change pass the example solution?

how does a non-numeric grade affect the sorting of grades? Does "AB" come before 1? After any numeric grade? In Ontario, we have junior kindergarten ("JK"), senior kindergarten ("SK") then grade 1. I don't know what "AB" is supposed to mean.

Hi Glenn,
Here are my answers to your questions
2. when I submitted this request, I didn't consider the example solution, but when I ran it, all tests passed, except the one test that I changed, which title is Students are sorted by grades and then by name in the roster, and if I changed the excepted result to be : Barb,Charlie,Alex,Peter,Zoe,Jim,Anna, it would work.
Now, there is a problem introduced, i.e., the sort function "roster_sort", which sorts by grade, then by name, sees the grades as numerals only and it didn't consider the case of Character grades like "JK" or "SK", the function returns : v1-v2 in case of grades, this would be resolved to zero for any passed character grades, and if there are grades added in the text values as SK then JK, they would be seen as the save value when subtracted, i.e KS - SK, as they both would be resolved to zero
So, and this is a question: would you like me to cancel this pull request and change the example solution to accept characters ?

Based on the function "roster_sort", the "AB" comes first. Also, "AB" was supposed by me as just a character and I didn't think while suggesting it for some real grade in life, but may be there are places that use grades like "A", "B" for baby classes

We can still work in the context of this PR. Just add further commits to it.
If we're going to add non-digit grades:

it must be done as a new test,
the example needs to be updated as required to pass,
a .docs/instructions.append.md file should be added to clearly spell out the sorting requirements.

Exercism specification for markdown files is to have a single top-level heading.
However, it is discarded for the append file.
That means the append file needs to look something like
# ignore

## AWK-specific instructions

There are tests with non-numeric grades.
When generating the roster, these grades need to sort _after_ all numeric grades.

I do appreciate your contribution. Please see the Practice Exercise doc for all the little details about how exercises are constructed within a track.

@IsaacG I'm OK with adding a new test. This exercise gets students to practice sorting and having non-numeric grades adds an interesting new wrinkle to it.

While we may not validate the input is numeric, I don't think we should be adding non-numeric inputs here.
If there is any value to having "wider" grades, I think those should be pushed upstream to the problem specs.

I'm not seeing a justification for these updates to be awk specific and to diverge from the problem specs.

When I initially created this PR, I was do not aware of the example solution, and I didn't make any effort to change it, which needs new/update specs. What prompted me to change the test "Students are sorted by grades and then by name in the roster", was that my initial code for this problem passed all tests, until Filbo advised me to generalize the code to accept more than one digit. I realized that all tests had only one digit in grades. My code in the last iteration accepted numerals and non-numeric values which gave me the idea to change the test to include non-numeric values.
However, following Issac Good comment, I discarded the idea of introducing non-numeric grades and retained the change that used a grade with two digits ( grade numbers before the collage in some countries are counted from 1 to 12).

However, I haven't completely abandoned my idea. I might create a new problem that involves sorting both numeric and non-numeric values.

I'm willing to go forward with this, but it has to be a new test not modifying an existing one.


Add boop

Great package - I use it for most of my projects.
I would love if you could also add support for "boop"?

LOL 😂


Generalize the dotnet wrapper a bit

csharp_binary needs to become a rule which emits a cc_binary that
invokes dotnet <foo>, where <foo> is the actual dll that we
compile, to solve #71. Eventually <foo> will become a wrapper C# exe
that tweaks assembly loading to solve #9. So all of this requires the
wrapper to provide a default argv[1].
For stuff like dotnet vstest (#51) we will have argv[1] be vstest, so we
also need support for baking in argv[2].

Er, had this sitting around on a branch but I forgot to tidy some stuff up.

Ok.

Improvements for future PRs:

Move template params out to a header to make it obvious where they are
Remove the ifdefs in favour of Starlark select() to chose the write helper method impl
Switch to CreateProcess from spawnv on Windows.


Test and release with Python 3.12

This PR updates the CI config to:

Run Tox checks with Python 3.12 (was 3.11)
Run Tox tests with Python 3.12, in addition to the previous versions (3.7+)
Updates actions/checkout@v3 to actions/checkout@v4
Updates actions/setup-python@v3 and actions/setup-python@v4 references to actions/setup-python@v5

P.S. - Thanks for creating this awesome flake8 plugin.

@cooperlees Looks like a format check(?) complained. https://github.com/cooperlees/flake8-async/pull/12/commits/e5466be71024183a5062d39a7c7800fa3422a43b adds a newline, but I did it thru my browser. If a rerun doesn't show the issue as fixed I'll revert the change and address whatever is causing problems.
From CI:
check: commands[1]> flake8 --exclude '.*'
  check: OK (1.74=setup[0.40]+cmd[1.[17](https://github.com/cooperlees/flake8-async/actions/runs/7921123877/job/21630346097#step:5:18),0.17] seconds)
  congratulations :) (1.79 seconds)
diff --git a/tests/test_changelog_and_version.py b/tests/test_changelog_and_version.py
index b11a[21](https://github.com/cooperlees/flake8-async/actions/runs/7921123877/job/21630346097#step:5:22)2..495eecf 100644
--- a/tests/test_changelog_and_version.py
+++ b/tests/test_changelog_and_version.py
@@ -1,4 +1,5 @@
 """Tests for the hypothesmith package metadata."""
+
 import re
 from pathlib import Path
 from typing import NamedTuple, Optional
Error: Process completed with exit code 1.


added attributes to github_user and github_team data sources

updated go-github
added gpg and ssh keys to github_user data source and members to github_team
fixed issues with attributes using pointers instead of functions
documented changes

Looks great! Do we need to change things anywhere else for the vendor update?

@josephholsten nope. should be good to merge.

Hi @mootpt
This LGTM! Thanks for this :)
P.


Issues with reproducibility

I'm trying to work through the 01a_data_import_and_preprocessing.ipynb notebook and I'm running into an issue with the raw data. The first issue I run into deals with this code:
with open("../../../raw_data_path.txt", "r") as file:
    dir_data = file.readline().strip()

The raw_data_path.txt file is not part of this repository. I assumed it was just a txt file that contains the path to the raw data on my local computer, but I didn't want to assume incorrectly.
The second issue deals with actually obtaining the raw data. It isn't clear where I can find the data to download it myself, or even which files are needed in order to complete the code under the import data section of the notebook. Looking at the LCA_file_reading.py script, it seems raw data files need to be organized in a certain way for the functions to execute. For example, in read_file_Banovich_Kropski_2020 it looks for 20210414_VUMC.TGen_control_annotation.csv" in a 01_Metadata directory, while in read_file_Barbry_Leroy_2020 it looks for HCA_Lung_Barbry_Raw_SoupX.h5 and HCA_Lung_Barbry_Raw.h5 to be in a 04_Counts directory.
Would you be able to provide locations for where the raw data can be obtained/downloaded and a structure for how the data should be organized in order to be able to run the notebook to completion?
Best,
Kayla

Hi Kayla,
Yeah that file indeed contained the path to the directory with the raw file on my computer, you can just set dir_data to the directory where you have the data and ignore the file.
And yes I haven't made available the raw data I used at the time, as it has undergone so many rounds of cleaning and gene harmonisation etc. since. I still have it on our cluster though
May I ask what you need it for? It would probably be easiest to just use the raw counts of the final object for most tasks, unless you are trying to exactly reproduce some of our intermediate outputs


(Re-)infer candidate from state

Fixes: #131
This reapplies the changes of #131 starting from a version where tests assure that Hoff is not stuck in an infinite comment loop.  The summary of #131 still holds:

The changes in this PR do not change Hoff behaviour, the high level functionality should be the same.
Instead of maintaining a global candidate, we infer it from the state always.  This is in preparation for merge trains (#77), where we will have multiple builds running in parallel and thus multiple candidates.

The ProjectState datatype is now simpler and easier to maintain (before we could risk it being inconsistent);
Some code paths are now simpler, as we don't have to do the work maintaining the global candidate;
There is a new explicit integration status Promoted to signify that the PR branch is now part of master, before this was signified by being Integrated ... BuildSucceeded but not being the global candidate.

A linear search on the number of open PRs is imposed.  I believe this is not a problem because:

the number of PRs should be at most a few dozen, examples:

there are only 5 open PRs on channable/hoff currently;
a dozen open PRs seems to be a reasonable average (:wink:);
a busy project (:wink:) should have between 100 and 200 open PRs in which a linear search is still reasonable;
... and we could still have way more than that without suffering significant performance impact (1000 or 10000 open PRs).

we already perform linear search for other operations, the candidate was just a special case in where we don't.  If we were to actually get rid of linear search altogether, we perhaps should have a more generic solution that covers all cases.  (If linear search were to be a problem, we would already be suffering from it.)
with #77, we are bound to perform a linear search on the candidate PRs anyway.

TODO

[x] https://github.com/channable/hoff/pull/139

[x] to detect the infinite comment loop

[x] rebase revert of revert of #131 on top of the new master
[x] fix the issue

[x] PRs with BuildFailed are not candidates
[x] Use needsFeedback on handleBuildStatusChanged

[x] test that failing PRs do not "clog" the queue

[x] using GitHub
[x] in the actual test files

@alex-mckenna Since I am sort of restoring my previous changes now, I took the traversePullRequests function and your implementation of handleBuildStatusChanged from your (now-orphan) commit.  This won't be merged soon, as I still need to fix the bug I caused.

@fatho You have previously reviewed #131, which was the PR that had the bug and was reverted.  This PR re-adds the same functionality, so part of the code is the same as in the previous PR.  I think it makes sense for you to review again in this case.
The end result is quite different in some parts, and the code of this new version is IMHO more elegant than the one of #131.

@fatho Thanks for the review.  I will merge this into master now and tag it so that I can have this ready for deploy at an appropriate time this week.
@OpsBotPrime merge

Pull request approved for merge by @rudymatela, rebasing now.

Rebased as 1bd9c99f4f1a0fb08ee00546b4c5958d850857a0, waiting for CI …


local env variable is not set in compose service

VSCode Version: 1.61.2
Local OS Version: wsl2 ubuntu 20.04
Reproduces in:| Remote - Containers |
Name of Dev Container Definition with Issue:

Steps to Reproduce:
HI, I have some variables set in my shell from my bashrc. I am unable to pass them to the devcontainer started from a compose file. It works with regular compose up, as its feature of compose to pick env variables either from shell or a .env file next to the first compose file that is used.
services:
  environment:
    PIP_EXTRA_INDEX_URL:
   # other variables loaded from .env file

Doing it from the devcontainer.json as documented also doesnt work.
  "remoteEnv": {
    "PIP_EXTRA_INDEX_URL": "${localEnv:PIP_EXTRA_INDEX_URL}",
  },

I really don't want to write this in a .env file within the repository, since it's confidential and shouldn't get source controlled. I want to use it as it's set in the shell through my bashrc.

You should just be able to reference "${PIP_EXTRA_INDEX_URL}" in a Docker Compose file.

No it doesn't work. When using normal compose yes, this works, actually then you can even leave the value blank as showing in my question. With devcontainer both are not working. Or do you mean I still should set it in remoteEnv?
WARNING: The PIP_EXTRA_INDEX_URL variable is not set. Defaulting to a blank string.

How are you setting it the env var locally? You would need to set it in the process that is used to start up VS Code.


feat: Make NAT Gateway Routes for Private Route Tables Optional

Description
Added a new variable which will allow for NAT gateway routes on private route tables to be skipped.
Motivation and Context
I have a scenario in my environment where I need to override the <IP_ADDRESS>/0 route for the private route tables.
For example:
I want to deploy the AWS Network Firewall in my VPC that I have built using this module. I need to update my private subnets <IP_ADDRESS>/0 route to use the network firewall endpoint rather than the NAT Gateway endpoint.
I understand that the new version (V4) of terraform-aws-vpc will have network functionality baked in, however in the interim it would be nice to be able to override this route creation.
Breaking Changes
By default this will be a no-op, as the default is set to true, which the count expects in order to create the routes.
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects
[x] I have executed pre-commit run -a on my pull request

👋 Hey @antonbabenko or @bryantbiggs could I get a review on this change?

Ended up forking repository to make the changes, though I do think lack of support network firewall in the module (or providing a mechanism to modify routes to support it as per this PR) does not align with security best practices.


Introduce new random object that acts as a default source of randomness

A thin wrapper around different random number generators is introduced, which should become the new default source of randomness for all methods in scalismo that need random numbers.
The idea is that every method takes an implicit instance of the new Random class as an argument.
The new mechanism has the following behaviour:

If the user does not specify an own implicit instance, the default source in the companion object is used.
If the result should be reproducible, the user can define his/her own implicit instance with a fixed seed.

merged manually due to conflicts


Cliprelatedwork

add the complete clip code including training and inference based on vector database

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Make javafx.base dependency transitive

Would it be possible to modify the requires javafx.base; into requires transitive javafx.base; in module-info.java?
This would avoid excluding the dependency in maven or gradle when facing this type of error:
Package 'com.example' reads package 'javafx.beans' from both 'javafx.base' and 'javafx.base'

The issue is explained in detail in this SO post

Thanks to @overheadhunter this is now fixed. I also removed the os-specific "classifier" in the pom file. This is contained in Release 2.2, which should be available very soon on maven (I just published it).


⚠ ClamAV has degraded performance

In a54507b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 173 ms

Resolved: ClamAV performance has improved in 8e6485b.


Not able to create user in exadmin

Hi,
I have followed every steps for installing coherence and exadmin. it worked very well. I can create other db entries likes roles, countries using exadmin. However when registering user, exadmin neither throws an error nor submitting entries in database. Dont know whats wrong here
Regards,

These are the few debug logs generated by phoenix
iex(1)> [debug] Processing with ExAdmin.AdminResourceController.create/2
Parameters: %{"_csrf_token" => "LXAoYHcQAiUmHyUpDy8uOTUABjw+AAAAoHr/GvdTPXGNCdJsXCcKkA==", "commit" => "Create User", "resource" => "users", "user" => %{"email" =><EMAIL_ADDRESS>"name" => "Suhas", "password_hash" => "[FILTERED]"}, "utf8" => "Γ£ô"}
Pipelines: [:browser]
iex(1)> [info] Sent 200 in 15ms

closing this issue. After bit of research found out that password field is needed in exadmin section in user model. Quite a strange as contact_demo repo which I was referring had this field commented.


Add typescript

Closes #13

Codecov Report

Merging #22 (cf1b25b) into master (1654547) will decrease coverage by 6.06%.
The diff coverage is 77.77%.

@@             Coverage Diff             @@
##            master      #22      +/-   ##
===========================================
- Coverage   100.00%   93.93%   -6.07%
===========================================
  Files            2        1       -1
  Lines           52       33      -19
  Branches         1        3       +2
===========================================
- Hits            52       31      -21
- Partials         0        2       +2

Flag
Coverage Δ

e2e
93.93% <77.77%> (-6.07%)
:arrow_down:

unittests
?

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

Impacted Files
Coverage Δ

src/components/Lumino.vue
93.93% <77.77%> (-6.07%)
:arrow_down:

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 1654547...cf1b25b. Read the comment docs.

Alright. Code OK. Build OK.

The produced bundle still doesn't have a type.
TS7016: Could not find a declaration file for module '@tupilabs/vue-lumino'. <EMAIL_ADDRESS>implicitly has an 'any' type.

Hmm, no luck so far. Still trying to get the Vue3 test project to import the Lumino component correctly. Found a bug that appears to be similar to what I'm getting: https://github.com/vuejs/vue/issues/7392

Alas it also didn't work. Might have to wait until I migrate $work projects to Vue3, and then learn how to build a component for Vue2 & 3, JS & TS. :confused:


MNT: Adding more pre-commit hooks

Change Summary
Overview
Adding additional pre-commit hooks to check for large files and remove bad line endings.
See here for some more options: https://pre-commit.com/hooks.html, https://github.com/pre-commit/pre-commit-hooks

Merging so the other docs PR can be seen more clearly. We can update this later with more/less if we want as well.


Cxf-soap tests fail in FIPS environment (fix for JVM)

fixes https://github.com/apache/camel-quarkus/issues/5482

Native failure has to be investigated further, reported as https://github.com/apache/camel-quarkus/issues/5486

All certficates and keystores have to be generated via openssl to work correctly in the FIPS, therefore I removed the keytool-plugin and added a description, how to generate certificates.
I found several bad passwords in the callbacks and fixed them (I'm not sure why it was working  in non-FIPS environment - probably the passwords were not required by the cxf in non-FIPS) .

For the FIPS-enabled environment, the profile fips has to be used. The reason is that the BC-FIPS is used instead of the BC

The fips Maven profile has:
<properties>
    <security.procide>quarkus.security.security-providers=BCFIPS</security.procide>
</properties>

What does that do?

Sorry I forgot to remove it on all places. It shouldn'tbe there

For the FIPS-enabled environment, the profile fips has to be used. The reason is that the BC-FIPS is used instead of the BC

The fips Maven profile has:
<properties>
    <security.procide>quarkus.security.security-providers=BCFIPS</security.procide>
</properties>

What does that do?

Removed. Thanks for noticing!

Great work, thanks for taking care, @JiriOndrusek!

All certficates and keystores have to be generated via openssl

Do you happen to know what aspect of the generated keystore is not FIPS-compliant with keytool-maven-plugin? At the first sight, the plugin mojos seem to support all we need: keysize, keyalgorithm, storetype...

Great work, thanks for taking care, @JiriOndrusek!

All certficates and keystores have to be generated via openssl

Do you happen to know what aspect of the generated keystore is not FIPS-compliant with keytool-maven-plugin? At the first sight, the plugin mojos seem to support all we need: keysize, keyalgorithm, storetype...

I probably mislead a little bit with my comment. It is possible to run successfully the keytool-maven-plugin, but I haven't found a way of generating such strong certificates to satisfy runtime in FIPS

It looks good.
So we configure openssl with the .cnf configuration file. Is it need for fips purpose ? Or something else please ?

TBH I'm using *.cnf mainly because the default values functionality (so user is not forced to fill all - e.g. CN which is "referenced" from the tests. I'll look into an option with no *.cnf files in the future to be sure whether it is mandatory.

There are 2 open issues:

problem in the native (reported as https://github.com/apache/camel-quarkus/issues/5486) I'll investigate it .
the option of not using BCFIPS in FIPS and use only SunPKCS11-NSS-FIPS provider. I think that it won't be possible, because quarkus-cxf-rt-ws-security brings BC as a transitive dependency, therefore requires BC. But I think that I'll discover the answer during investigation of the native failure. The native issue seems like a BC class is needed, when BC is replaced with BC-FIPS

Do you happen to know what aspect of the generated keystore is not FIPS-compliant with keytool-maven-plugin? At the first sight, the plugin mojos seem to support all we need: keysize, keyalgorithm, storetype...

I probably mislead a little bit with my comment. It is possible to run successfully the keytool-maven-plugin, but I haven't found a way of generating such strong certificates to satisfy runtime in FIPS.
I remember seeing some not-supported algorithms for signing or similar, when I was trying to use keystore only. I don't remember the detail but will save them during investigation of other FIPS issues.

This sounds like an answer: https://stackoverflow.com/a/50398011

PKCS#12 stores the private key encrypted with a password derived key. It looks like keytool uses pbeWithSHAAnd128BitRC2-CBC (pkcs-12PbeIds 5), an PBES1 algorithm for doing so. Even the keytool.exe of Oracle Java 9 does use this algorithm as you can verify by uploading a .p12 file to the online ASN.1 decoder decoding a sample PKCS#12 file.
If I read the PKCS#12 standard correctly PBES1 was long ago superseeded by the "newer" version of the key derivation system named "PBES2" (mostly PBKDF2 based) with should be used instead. But keytool does not make use of it. This is my interpretation of the error message.
Therefore the certificate and the key may be acceptable, but the PKCS#12 container is not acceptable. You may try to extract key and certificate and save them in a new PKCS#12 file using a current software like OpenSSL (or you simply generate the whole PKCS#12 file directly using OpenSSL).

Do you happen to know what aspect of the generated keystore is not FIPS-compliant with keytool-maven-plugin? At the first sight, the plugin mojos seem to support all we need: keysize, keyalgorithm, storetype...

I probably mislead a little bit with my comment. It is possible to run successfully the keytool-maven-plugin, but I haven't found a way of generating such strong certificates to satisfy runtime in FIPS.
I remember seeing some not-supported algorithms for signing or similar, when I was trying to use keystore only. I don't remember the detail but will save them during investigation of other FIPS issues.

This sounds like an answer: https://stackoverflow.com/a/50398011

PKCS#12 stores the private key encrypted with a password derived key. It looks like keytool uses pbeWithSHAAnd128BitRC2-CBC (pkcs-12PbeIds 5), an PBES1 algorithm for doing so. Even the keytool.exe of Oracle Java 9 does use this algorithm as you can verify by uploading a .p12 file to the online ASN.1 decoder decoding a sample PKCS#12 file.
If I read the PKCS#12 standard correctly PBES1 was long ago superseeded by the "newer" version of the key derivation system named "PBES2" (mostly PBKDF2 based) with should be used instead. But keytool does not make use of it. This is my interpretation of the error message.
Therefore the certificate and the key may be acceptable, but the PKCS#12 container is not acceptable. You may try to extract key and certificate and save them in a new PKCS#12 file using a current software like OpenSSL (or you simply generate the whole PKCS#12 file directly using OpenSSL).

Thanks for the explanation. I can confirm the behavior, as I tried JKS keystores at first, but they are not accepted by FIPS-enabled system. I had to move to PKCS12

The change in CXF is prepared as https://github.com/apache/cxf/pull/1660, as soon as the CXF PR is merge, I'll rework this PR to leverage CustomizedAlgorithmSuite

This PR is using BCFIPS, which is not the right call anymore. Once the following change is merged into cxf and leveraged by the CQ, this PR should be reworked.

Superseeded by https://github.com/apache/camel-quarkus/pull/5980


Can't Saw Monthly Data on Dashboard

In GitLab by @Robin-Mao on Aug 23, 2018, 18:18
If you are reporting a problem, please make sure the following information are provided:

Version of board (last git commit number) , docker engine and docker-compose.
Config files of board, you can get them by packaging "board.cfg" and files in the same directory, including subdirectory.
Log files, you can get them by package the /var/log/board/.
Other necessery information.

In GitLab by @liyanq528 on Nov 8, 2019, 07:58
There is nothing record in database table;weiting checked it;

In GitLab by @wknet123 on Nov 8, 2019, 07:58
Fixed with PR #1710

In GitLab by @yhua123 on Nov 8, 2019, 07:58
LGTM

In GitLab by @sokril on Nov 8, 2019, 08:26
closed

In GitLab by @sokril on Nov 8, 2019, 08:26
assigned to @yhua123


repl: make double ctrl-d end REPL shell

This allows quick exit from the REPL shell and is a quite common pattern.

@Bastian-Krause: Great! Thanks for contributing! :)


Align toolchain package versions to SPEC versions

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

What does the PR accomplish, why was it needed?
Ideally, all toolchain package versions should match the SPEC versions. For some SPECS which have been updated due to CVEs, the corresponding toolchain version was not updated. This change aligns the versions for packages: Python3, fontconfig, freetype.
Change Log

Update Python3 version built in toolchain
Update fontconfig version built in toolchain
Update freetype version built in toolchain

Does this affect the toolchain?

YES
NO
Test Methodology

Local toolchain build

Removed update to "file" package due to build issue. Will investigate and fix separately.
/temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../x86_64-pc-linux-gnu/bin/ld: /temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../lib/libbz2.a(bzlib.o): relocation R_X86_64_32S against symbol BZ2_crc32Table' can not be used when making a shared object; recompile with -fPIC /temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../x86_64-pc-linux-gnu/bin/ld: /temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../lib/libbz2.a(compress.o): relocation R_X86_64_32 against .rodata.str1.1' can not be used when making a shared object; recompile with -fPIC
/temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../x86_64-pc-linux-gnu/bin/ld: /temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../lib/libbz2.a(decompress.o): relocation R_X86_64_32S against .rodata' can not be used when making a shared object; recompile with -fPIC /temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../x86_64-pc-linux-gnu/bin/ld: /temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../lib/libbz2.a(blocksort.o): relocation R_X86_64_32 against .rodata.str1.1' can not be used when making a shared object; recompile with -fPIC
/temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../x86_64-pc-linux-gnu/bin/ld: /temptoolchain/lfs/tools/bin/../lib/gcc/x86_64-pc-linux-gnu/9.1.0/../../../../lib/libbz2.a(blocksort.o): warning: relocation against stderr@@GLIBC_2.2.5' in read-only section .text'
collect2: error: ld returned 1 exit status
make[3]: *** [libmagic.la] Error 1
Makefile:499: recipe for target 'libmagic.la' failed
make[3]: Leaving directory '/temptoolchain/lfs/sources/file-5.38/src'


Does not generate some config files

The but does not generate all config files, if i want to start the bot it'll crash because they're missing

Already fixing that bug! wait for the next update (eta 1h)

now i get this error:
[code]Error occurred during initialization of VM
Could not reserve enough space for object heap
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.[/code]

How are you starting the bot?

That happens when I use "java -jar Sprummlbot.jar -setupConfigs"

so the error says, that you don't have enough ram space
look if your server / pc has 512mb ram free

it got like 8gb ram free^^

That's not normal :/ That error comes from java.

any idea how to work around it?

Depends. Do you have java 7 or 8, 32Bit or 64Bit Linux or Windows

Ubuntu15.04-64
and did everything like it's shown in the wiki installation thread for linux

k. try java -version and say if it prints an error

same error
Error occurred during initialization of VM
Could not reserve enough space for object heap
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

Okay then try to reinstall java: apt-get remove openjdk-8-jre && apt-get autoremove && apt-get install openjdk-8-jre

works now, thanks :)

No problem :)

btw whats the default wi password?

btw what's the default wi password?

You need to add your unique Id to the clienrs.ini under Webinterface Login and then write the bot !login
Then you get your login credentials

aight! thanks buddy

No prob

Why is the bot disconnecting almost instantly O.o

Oh umm. Okay... You did something wrong. Try to rejoin?

now it stays connected but the webinterface still doesn't show anything

I know that problem with the WI. How many users do you have on your server at the moment?

2 users are online atm

Lol. I had that problem on servers with many users. :/ I will try to fix this. Sorry for that

My skype: scrumplex (I am also german :))


ci: Add initial CI using GitHub Actions.

This is just a quick initial CI that I use elsewhere in case you're interested in having CI on this repo ...

Ah, this will fail due to clippy warnings from --all-targets ... I'll fix something up there if you're interested, so just let me know.

FYI https://github.com/frewsxcv/earcutr/actions/runs/6569696376

Submitted fix in #8.


Icons doesn't work

I want to work with fontAwesome icons instead of text but it doesn't work
is there any solution to make it possible

Hi @MeReda,
Can you share your code I wanted to see how you're using it.
Actually, I want to use images and text in the texts array, after reading the docs I got to know we can pass useHTML: true in options to use it but it seems not to be working for me.


How exactly Update works?

Hi,
I'm working on the update method where i want to update a document basing on the commentID. So, i created a Unique Index on that as below
postcomment.Collection.CreateIndex(new MongoDB.Driver.Builders.IndexKeysBuilder().Ascending("CommentID"), IndexOptions.SetUnique(true));
However when i use update statement for a document it throwing error as below
ERROR WriteConcern detected an error ''. (Response was { "ok" : 1, "code" : 11000, "err" : "E11000 duplicate key error index: SMAnalytics.FacebookComments.$CommentID_1 dup key: { : "404484049735321_404495519734174" }", "n" : NumberLong(0) }).
The below is the syntax which i'm using to upate
var cmt = new MComments() { PostID = "56772226", Comment = "Testing", FacebookID = "404484049735321", CommentID = "404484049735321_404495519734174", From = "Bhanu", Comment_Date = Convert.ToDateTime(comment.created_time), Like_Count ="10"};
postcomment.Update(cmt);
Please suggest some solution.. I should update the document If i already have that commentID..

How exactly Update works?

You do understand that this project is open source and based on the mongo-csharp-driver. You could just have a look at the update method and see exactly what it does: Call the driver's Save() method. And it is documented perfectly fine:

Save Method

The Save method is a combination of Insert and Update. If the Id member of the document has a value, then it is assumed to be an existing document and Save calls Update on the document (setting the Upsert flag just in case it actually is a new document after all). Otherwise it is assumed to be a new document and Save calls Insert after first assigning a newly generated unique value to the Id member.

Also the message is pretty clear: "E11000 duplicate key error". The id (note: Id) member of the object (in you case "MComments") determines wether the Update should insert or update a document. You created a Unique Index on CommentID. So either make the CommentID the document/entity ID or remove the unique index.

I absolutely understand. As I'm new to this MongoDB I'm just confused here and there.. Sorry if I trouble you much.


remove epoch again

Unintentionally used a "V" prefix in version 1.7.1.
Latest Version 1.7.2 doesn't have it as "most" earlier versions.
I promise to not use a "V" prefix again.

I'm afraid that removing the epoch will make all versions released from now on be counted as older than the version(s) released under the new epoch. Might be able to solve it with an override but I'm not quite sure how.

Ok, I see. It just looks so ugly in CKAN now with the 1: in front. It shows 1:1.7.2 at the moment.

Another question if I may: if I tag a release as a "prerelease" in Github (like V1.7) I don't want them to show up at all on CKAN. Haven't found anything about that topic in the CKAN documentation, only a reference in issue #403: https://github.com/KSP-CKAN/CKAN/issues/403

If you look at which versions of the mod we have indexed so far you will be glad to see that we do not index pre-releases :)

There are a lot of mods on CKAN with epochs making the version numbers look odd. CKAN users are accustomed to them.

AndyMt, are you cool with this PR being closed?

Yes. I wasn't aware that my version of the mod was on CKAN already, because until 1.7.2 it still was work in progress. I specifically asked at some point in the dev thread to NOT put new versions on CKAN. One reason was that I wanted to figure out myself how to do that and what processes and guidelines to follow.
Please don't get me wrong: I very much appreciate the work all of you do and the excellent service you provide. For mod authors and users alike :-).


feat(modules): add depends on to the generated kustomize and helm releases

This adds a dependsOn clause to each of the modules:

flux-git-sync
flux-helm-release
flux-oci-sync

@GeorgeMac can you please rebase with upstream main, tests should be fixed now.

Done, cheers @stefanprodan 👍


Listwriter patch, per proposal IE-0015

This changes the exact results for inventory details and room description details in many common cases, thus would require reblessing a lot of examples' output. I haven't included them in the PR, pending you making a call on whether you want to reduce the changes. The attached is from make check's output.
changed.txt

Pending a fix to I7-2353 this does not add a response (Z) for listwriter.

now with a  rule response 'Z' independent from 'C' thus finally actually addressing https://inform7.atlassian.net/browse/I7-2415


fix: put envs from .env files on process.env

Fixes https://github.com/vitest-dev/vitest/discussions/768

hi folks! Has this been released? I'm trying to make vitest to pick up .env.test file, but can't find any instructions


Add d.ts generation

It would be great if the bootstrapper repository could export its own d.ts file by using the babel-dts-generator plugin.

@EisenbergEffect this can be closed since we now have the dts file available


Merge fastai latest to this branch

I made a mistake here. I thought I was merging from fastai to my fork.  Let me close this


MudTab using Position.Left bug with MudGrid

Bug type
Component
Component name
MudTab
What happened?
Using a <MudTabs Position="Position.Left ..." adding Two  <MudGrid> with a single row <MudItem xs="12" sm="12" md="12"> creates two column instead of two rows.
Postion.Top is functioning correctly.
Attach a trymudblazor link.
Expected behavior
Show two rows.
Reproduction link
https://try.mudblazor.com/snippet/GYcQOSueqjNhyWsF
Reproduction steps
1.See try.mudblazor link
Relevant log output
No response
Version (bug)
6.0.9
Version (working)
No response
What browsers are you seeing the problem on?
Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Am I the only one having this issue?

With only one MudGrid I get two rows:

You can also use MudStack:


⚠ Bofrost.be has degraded performance

In 1bf0489, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2834 ms

Resolved: Bofrost.be performance has improved in e1bb3df.


Need two kinds of qualifiers

Currently there is Qualifiers, which is basically a Map.  That works well for "coloring" other things, i.e. for serving the purpose of an environment in JNDI, for example.
But there are also JSR-330 qualifier annotations, which this existing construct is kind of clumsy at representing.
Consider @Yellow.  Qualifier annotations are often values of qualifier keys whose names are implied.  Here, the qualifier key would be something like color.  So a qualifier annotation has a key, or a name, and a value, i.e. color = yellow.
But it also has notional qualifiers that further qualify it.  So the qualifier represented by color = yellow might also have an rgb qualifier whose value is…is…whatever the RGB code is for yellow 😄 .
Here it's better to say that we have a Qualifier whose name is Yellow, and then, perhaps, that it has meta-qualifiers (rgb = xxxxxx).  So maybe we can merge these two kinds of qualifiers: maybe we can augment the existing Qualifiers class so that it always has two well-known entries: name and value.  name might be set to the empty string by default and value might be set to the empty string by default.  But that would let us use this one class for two common purposes.  Is that good?  Is that bad?  Hard to say.
In the microbean-path case, for example, all of this is basically irrelevant: things that qualify a path element are just key-value pairs, so the existing Qualifiers class works here just fine.  But in the JSR-330 case, the most common kind of qualifier is one with a name and a single value.  Combining many of these things into a single Qualifiers instance gets tricky because then you'd have duplicate name entries in the resulting Qualifiers, which is impossible.
Another option is to do it all by convention: a Qualifiers would have a well-known qualifiers key whose value would be another Qualifiers.

Fixed and implemented.


Book cards not displaying

Cards not displaying on Books view.  Fix this.

Closed with recent commit at 12:38 AM, 12/14/2021.


Transaction Log, Pending Transactions, Query Manager unreadable in Dark Theme

System information:

Windows 10
DBeaver 6.3.0

Connection specification:

PostgreSQL

Describe the problem you're observing:
Light gray row text is unreadable against the green and yellow row-highlighting used by the Transaction Log, Pending Transactions, and Query Manager components.
Steps to reproduce, if exist:
Do a couple of queries.
Enable the dark theme.
Open the Transaction Log, and check 'Show all queries'.
Result:

Similar issue, in the Query Manager component (though the slightly darker text helps a certain amount):

Include any warning/errors/backtraces from the logs
None relevant.

Fixed

verified


Backend resilience during startup

Registrator has backend resilience baked in once things operate normally (-retry-* parameters). However, this does not kick in during startup and the program exits. Would be nice to have this instead of adding a helper like wait-for-it to the image.

Mind contributing a PR to fix this?

I'm not the most experienced Go dev, but in fact, I considered trying to implement this. Decided I first wanted to make sure there is a chance to get this into your fork.


Microsoft/users/nikhilku/fix

Updating DeployAzureResourceGroup and PerformMachine Group as per PM  feedback

Signed off.


disableBinding option

Hi @smnbbrv,
We just found a bug on Drag & drop between two lists.
If we set the property disableBinding to true it works just fine.
The issue was that this.onremove was null on overridenOptions.onRemove

Hi @lbertenasco
the onremove is kinda a dirty hack. The purpose of one before was that it was outside of the component (before it was moved in), so it was global and not depending on the list.
Now it is of course always null because the new item is added to another list and the target list gets the onremove function, although the one that loses an item should get one.
What was the reason of making it a member property instead of simple let? AOT? Would making it a static property solve the issue (if it is a static property it would still work fine)?

@lbertenasco sorry for so complicated text, cannot understand it by myself when re-read it :) So, the idea of onremove is to be global and single for the whole app exactly for drag&drop between the lists. When you moved it inside of the component it does not work anymore. We either should move it back to be a separate variable or make it a static property on SortablejsDirective which is the same thing.
This will fix the issue without any additional parameter and actually this is how it is supposed to be.
Do you see some problems with this?

hi @smnbbrv , I think there will be an issue with the event since it's specific to the list elements.
  this.onremove = (item: any) => {
    if (this.isItemsFormArray) {
        this.items.insert(*event.newIndex*, item);
    } else {
        this.items.splice(*event.newIndex*, 0, item);
    }
  };

I'm not sure on how to fix this, I'll try to find a solution on my free time.
Also I think the user should be able to handle this changes on his end if desired.
That's why I added the disableBinding option which should be explained on the README.md

it was working before :) as long as it is an arrow function it catches this of the current context (the sortable where it gets added) and remembers it until the element of another sortable where it gets deleted actually calls it. If it is not an AOT thing I will return it then and let you know the version to test.
About disableBinding - it is already possible to do it from the user side - just pass nothing (or null or whatsoever) to the [sortablejs], and this is already written in the README

@lbertenasco try<EMAIL_ADDRESS>

@smnbbrv On<EMAIL_ADDRESS>I get:
Uncaught Error: Unexpected value 'undefined' exported by the module 'SortablejsModule'
Ok, I think I've got a clearer picture now. I'm going to test some things and get back to you.

@lbertenasco I pushed the changes etc, so you can have a look at updated things...
and that actually looks like https://github.com/SortableJS/angular-sortablejs/issues/34

I think I close this pull request, feel free to make a new one 👍


doc: Add 2018-11-29 meeting minutes.

Closes: #421

Going to merge this because there's been no requests for changes in nearly a month 👍


Fix Spotlight

Cards don't change after one reload. A Next thing ig. Works on local tho.

useEffect fixed it


Step 390 apple auth update

Checklist

Requires  MINOR version update

Context
Add API Key connection, both API Key and Apple ID input support.
Resolves: https://bitrise.atlassian.net/browse/STEP-390
Changes
Added support for API key (App Store Connect API), from Bitrise Developer Connection and Step inputs.
For Apple ID, now also passing the username/password/app-specific password in addition to the session (FASTLANE_SESSION) as they seem to be required.
Any authentication-related environment that is already set will be overridden, to prevent this set Bitrise Apple Developer Connection (connection) input to off and leave authentication-related inputs empty. This includes DELIVER_USERNAME, DELIVER_PASSWORD, FASTLANE_SESSION, FASTLANE_APPLE_APPLICATION_SPECIFIC_PASSWORD, DELIVER_API_KEY_PATH and PRECHECK_INCLUDE_IN_APP_PURCHASES.
Investigation details

also, the step works without any authentication, right?


Adding rtrim and ltrim to remove null bytes for sql2000

Fixes #
Changes proposed in this pull request:

Adding ltrim and rtrim to filenames to remove null byes in all versions, but specifically to resolve SQL 2000 issues

How to test this code:

Create database
Detach database
Run command Find-DbaOrphanedFile -SqlServer ServerName

Has been tested on minimum requirements:

[ X ]  Powershell 3
[ X ]  Windows 7
[ X ]  SQL Server 2000

Has been tested on maximum requirements:

[ ]  SQL Server vNext
[ ]  Windows 10
[ ]  Azure Database

Tests for tester:

[x] Working/useful help content, including link to command on dbatools web site
[x] All examples work as advertised
[x] Does not contain template content
[x] Does not contain excessive/unnecessary amounts of comments
[x] Works remotely
[x] Works locally
[ ] Works on lower versions or throws error specifying version not supported
[x] Works with named instances
[x] Works with clustered instances
[ ] Handles offline/read only databases
[x] Supports multiple servers (at the command line or piped from Get-SqlRegisteredServerName)
[x] No un-handled errors which stop the command working with multiple servers

Thanks bud!


[production & staging] Folder Picker Unavailable after 2nd Import

Steps
Go to a project's settings page
Select an add-on such as dropbox
Import or Create the access token
Expand the folder picker by clicking Change if not already displayed
Deauthorize the add-on (dropbox)
Reauthorize the add-don (dropbox)
Attempt to expand/collapse the folder picker
Expected
The folder picker expands/collapses upon clicking the Change button.
Actual
The folder picker is never displayed, only a line where it would be. The user must refresh the page to reenable folder picker functionality.

Can I get assigned to this?

Done.

Resolved in: https://github.com/CenterForOpenScience/osf.io/pull/2250

This problem is actually fixed on staging but not yet on production. The fix is possibly included in https://github.com/CenterForOpenScience/osf.io/pull/2014. Once the 31 release is out. Need QA to verify this has been fixed on production.

:+1: ... closing for now.


boundary: 0.13.1 -> 0.14.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for boundary is: Enables identity-based access management for dynamic infrastructure
meta.homepage for boundary is: https://boundaryproject.io/
meta.changelog for boundary is: https://github.com/hashicorp/boundary/blob/v0.14.1/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.14.1 with grep in /nix/store/1cnxp6n5hmidfakm6bmjgis3p1kkx669-boundary-0.14.1
found 0.14.1 in filename of file in /nix/store/1cnxp6n5hmidfakm6bmjgis3p1kkx669-boundary-0.14.1

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
boundary

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/1cnxp6n5hmidfakm6bmjgis3p1kkx669-boundary-0.14.1 \
  --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 boundary https://github.com/r-ryantm/nixpkgs/archive/849a788776ca8a14e8fedfb8e37d2d80f74c90a5.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/849a788776ca8a14e8fedfb8e37d2d80f74c90a5#boundary

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/1cnxp6n5hmidfakm6bmjgis3p1kkx669-boundary-0.14.1
ls -la /nix/store/1cnxp6n5hmidfakm6bmjgis3p1kkx669-boundary-0.14.1/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:

boundary

Maintainer pings
cc @06kellyjac @techknowlogick for testing.

Result of nixpkgs-review pr 263267 run on aarch64-darwin 1

1 package built:

boundary


chore(workflows): add actionlint workflow

@mason-org/triage
 🤖 This is an automated comment.    📖 Source code


Variable Array and Expansion / Dereference

Hi, great work with Grafana 5 and VariableValues.
Now I'm coming with a new request once again.
I'm working with a lot of template variables that I need to pass to the Panels, and they are correctly managed, but I also need to use some of them in the template, and having a single long variable with everything inside is not useful. I need to access the single variables. VariableValues is loaded with a random order, so I can't use it as it is but I need the variables as an independent slice.
This PR handles this in a not so cool way, but it is working.
From the template I can now get a single variable ( var-Desc for example ) using the builtin index function : [[$var := index .VArray "var-Desc"]] or calling a method for it : [[.GetCleanVarKey "var-Desc"]]
I use Title and Description fields of Rows and Panels to put captions under the images. \captionof{figure}{[[.Description]]}, but sometimes they have references to some template variable inside, so I need a further level of variable expansion.
Es. Description may be : "This panel shows the $host load" where $host has to be expanded to the value of var-host that is passed by grafana to the Panel.
This is done using this code in the template :
\captionof{figure}{[[ .GetExpandedVarStr .Description ]]}

Pretty neat, thanks. I might use some of this in to the next minor release if I have time.

This seems pretty specialised. Going to close the PR for now until more people show interest in such a feature.


WS-442 | WMS reusable code

:thinking: What?
Contribute with WMS's reusable code.
:man_shrugging: Why?
Tech debt sprint 2.0
:pushpin: Jira Issue
WS-442
:no_good: Blocked by
Not blocked
:clipboard: Pending

Nothing

:fire: Extra
Nothing

@kress95 what do you think of reusing this major bump, to rename the queries suffixing in the CommonSimulator, so that we have: Http's requests as "#iN", Graphql's queries as #qN and Graphql's mutations as "#mN". This way we'll know what failed in the tests...


alternative keypad?

Hello,
I've been using your gui for a short time and I'm excited so far!
I have a pool project in which I would like to map a timer and my idea would be to create a numeric field where an alternative keypad pops up.
So far I haven't found how to refer to an alternative keypad.
i.e. I would like to define in the Builder that instead of a normal keypad, a keypad with hours from 0 a.m.-11 p.m. should be selected and a keypad with 5 minute steps should open in another field.
Can you help me?
best regards
stefan

Hi stefan --
With the new KeyPad component from #255 that is about to be released, it should be possible to create your own user variant keypads for arbitrary input (like time). I have put together a draft example of a XKeyPad_Time variant that lets the user select the hour, then minute and AM/PM from a single KeyPad instance targeting a single destination field/string. However, if you prefer to have two separate fields (one for hour and another for minutes) and use two separate keypads, that would also be possible.
Once the XKeyPad update has been integrated, I will post a followup here so you can try out the example I created.
cal

Hello,
I just searched for the described versions 0.16 of the API and the builder but couldn't find it.

Hi Stefan — there were a number of last minute changes required so the update took longer than anticipated. Paul and I also spent some time to update the Builder to work with the new APIs.
I hope to have a draft version posted by tomorrow for you to try out. Thanks!

@kellers76  -- the changes for the new XKeyPad have been rolled into the GUIslice core library now. Please download the latest from GitHub (currently showing v0.15.1, which will be officially released later as v0.16, along with the Builder).
I am also attaching the example XKeyPad_Time widget I created for you to demonstrate how one can customize the new KeyPad to address requests such as your time control widget.
As for custom keypads -- if one simply wants to change the keys that appear within one of the existing KeyPad variants (eg. XKeyPad_Alpha), then it is a simple matter to update /src/elem/XKeyPad_Alpha-setup.c as needed. The Builder codegen should still work properly.
However, if you intend to create new keypad variants (such as your time entry widget), note that some customization of the code within the variant will be required -- the Builder won't be able to do this automatically for you. :)
After installing the latest GUIslice, try adding the attached XKeyPad_Time widget and try the example:

WIP278-KeyPadTime.zip
Copy XKeyPad_Time* into /src/elem/
Run example Int278-KeyPadTime1

Some additional information about KeyPad configuration will be posted here: Custom KeyPads
thanks!

Closing as the extended keypad functionality has been integrated. @kellers76 -- if you have any difficulties with using the new API to support your time controller after seeing the "KeyPad_Time" example, please feel free to reply and I'll reopen it.

hi,
unfortunately I have a problem with my keypads.
I could not integrate the keypad Time 1:1 but only make it work by converting the keypad Alpha to the keypad Time.
In my current project I would like to have at least 4 different keypads, Num and Alpha ( standard ), plus my own Time and Speed.
The Speed keypad is a reduced Num keypad with filtered digits from 1-3 (I use it to make a motor speed).
I copied the element Num for the Speed Keypad and renamed all variables from Num to Speed.
In my sketch I copied all the definitions from Num and added them as Speed.
Unfortunately the whole thing doesn't work, when I start the call my program does a reset and restarts.
On the other hand, if I modify the Num keypad, then the call works.
Am I doing something wrong with the call? Can I use more than 2 keypads?
Translated with www.DeepL.com/Translator (free version)

hi,
unfortunately I have a problem with my keypads.
I could not integrate the keypad Time 1:1 but only make it work by converting the keypad Alpha to the keypad Time.
In my current project I would like to have at least 4 different keypads, Num and Alpha ( standard ), plus my own Time and Speed.
The Speed keypad is a reduced Num keypad with filtered digits from 1-3 (I use it to make a motor speed).
I copied the element Num for the Speed Keypad and renamed all variables from Num to Speed.
In my sketch I copied all the definitions from Num and added them as Speed.
Unfortunately the whole thing doesn't work, when I start the call my program does a reset and restarts.
On the other hand, if I modify the Num keypad, then the call works.
Am I doing something wrong with the call? Can I use more than 2 keypads?
Translated with www.DeepL.com/Translator (free version)


HADOOP-17891. Fix compilation error under skipShade

This is a follow up for #3385 to fix the compilation error under skipShade.

cc @sunchao

cc @ayushtkn too

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3441/1/console in case of problems.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 54s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
shelldocs
0m  0s

Shelldocs was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 15s

trunk passed

+1 :green_heart:
compile
0m 17s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
0m 18s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
mvnsite
0m 22s

trunk passed

+1 :green_heart:
javadoc
0m 19s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 19s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
shadedclient
33m 33s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 14s

the patch passed

+1 :green_heart:
compile
0m 12s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
0m 12s

the patch passed

+1 :green_heart:
compile
0m 12s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
0m 12s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 13s

the patch passed

+1 :green_heart:
shellcheck
0m  1s

No new issues.

+1 :green_heart:
xml
0m  1s

The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
0m 12s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 12s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
shadedclient
33m 32s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 14s

hadoop-client-integration-tests in the patch passed.

+1 :green_heart:
asflicense
0m 30s

The patch does not generate ASF License warnings.

106m 57s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3441/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3441

Optional Tests
dupname asflicense mvnsite unit codespell shellcheck shelldocs compile javac javadoc mvninstall shadedclient xml

uname
Linux 8f33bd767048 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 2ea0f25fb77bf864190980082a156309427cd91c

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3441/1/testReport/

Max. process+thread count
523 (vs. ulimit of 5500)

modules
C: hadoop-client-modules/hadoop-client-integration-tests U: hadoop-client-modules/hadoop-client-integration-tests

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3441/1/console

versions
git=2.25.1 maven=3.6.3 shellcheck=0.7.0

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

@viirya I think you should use -DskipShade instead of -PskipShade. The test still doesn't run properly.

Oops! Fixed. Thanks.

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3441/2/console in case of problems.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 52s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
shelldocs
0m  0s

Shelldocs was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
35m 51s

trunk passed

+1 :green_heart:
compile
0m 19s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
0m 18s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
mvnsite
0m 21s

trunk passed

+1 :green_heart:
javadoc
0m 20s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 19s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

-1 :x:
shadedclient
21m 25s

branch has errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 14s

the patch passed

+1 :green_heart:
compile
0m 11s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
0m 11s

the patch passed

+1 :green_heart:
compile
0m 11s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
0m 11s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 14s

the patch passed

+1 :green_heart:
shellcheck
0m  1s

No new issues.

+1 :green_heart:
xml
0m  1s

The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
0m 12s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 12s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
shadedclient
22m  2s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 15s

hadoop-client-integration-tests in the patch passed.

+1 :green_heart:
asflicense
0m 29s

The patch does not generate ASF License warnings.

85m 24s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3441/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3441

Optional Tests
dupname asflicense mvnsite unit codespell shellcheck shelldocs compile javac javadoc mvninstall shadedclient xml

uname
Linux 4a733f0f6333 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / bf80325b20263c58cc79e7112594f6acb38528e6

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3441/2/testReport/

Max. process+thread count
572 (vs. ulimit of 5500)

modules
C: hadoop-client-modules/hadoop-client-integration-tests U: hadoop-client-modules/hadoop-client-integration-tests

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3441/2/console

versions
git=2.25.1 maven=3.6.3 shellcheck=0.7.0

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Seems okay now. patch-shadedclient.txt was passed.

The -1 from shadedclient is for branch-shadedclient.txt. But why branch build uses the updated dev-support/bin/hadoop.sh too?

Thank you @ayushtkn !

Merged to trunk. Thanks @viirya !

Thank you @sunchao @ayushtkn @iwasakims !


Update examples to v1beta1 API

There's a couple of places that use the v1alpha2 API instead of the v1beta1 API.

https://cert-manager.io/docs/usage/certificate/
https://cert-manager.io/docs/configuration/selfsigned/
https://cert-manager.io/docs/configuration/ca/
https://cert-manager.io/docs/configuration/vault/

Looks like most examples are actually showing the old APIs. Is the v1beta1 not the preferred API?

This is on purpose as the v1alpha1 API is the only API version that works for Kubernetes 1.14 and below, using v1beta1 in the examples would break it for them. This will soon change when we release the v1 API however.
/close


Remove debug output in Error constructor

Landed in https://github.com/metarhia/impress/commit/2170c6b9d9d5c329814d50cecf906036bc181fba