Fix handling of trailing whitespaces in initials' delimiters

Fixes #137
Some styles, such as vancouver, require there to be nothing between initials, so Astley, Rick Gavin would be formatted as Astley, RG instead of Astley, R. G. (more usual) or Astley, R G for example. However, even when setting initializer-with="" (empty delimiter), Hayagriva was still forcing whitespaces between initials.
It was thought that #150 would have fixed it by only adding whitespaces if a delimiter was indeed specified, but it actually didn't change anything as there is always a delimiter - however, it is sometimes an empty string. While adding an explicit check for empty strings would have fixed this specific problem, it turns out Hayagriva's behavior of forcing a whitespace to always be added after the delimiter, replacing any of its trailing whitespaces if there are any (otherwise, one is added anyways), is not consistent with other CSL processors. Indeed, several citeproc tests demonstrated that the expected way to add whitespaces between initials is through initializer-with=". " instead of initializer-with=".". So, I removed Hayagriva's automatic whitespace behavior, except when we're replacing hyphens (e.g. for initializer-with=". ", Aa-Bb should become A.-B. and not A. -B.).
It is, however, unfortunate that this seems to be somewhat inconsistent with a specific example given by the relevant section of the CSL 1.0.2 spec:
`initialize-with`
    When set, given names are converted to initials. The attribute value is added after each initial (“.” results in “J. J. Doe”).

...but most official styles I've seen appear to assume that whitespaces are preserved, using ". " instead of "." in that case. So this seems to be the way to go.
I've also added some tests. Two more citeproc tests are now passing as well.
I'll be keeping this as a draft as I still need to test the case with initialize="false", initialize-with=".", but in principle the code seems to preserve whitespace here as well, however whitespaces from the name itself are also kept, which is why initialize-with="." is used instead of initialize-with=". " by official styles in this case.

If the other implementations and styles assume/exhibit/rely on this behaviour, we should also add it, so I'm good with it from that perspective.

Taking a look here, it seems that automatic whitespaces should also be removed when initialize="false" but initialize-with is set, according to citeproc tests, whereas we are not doing so. But I'll leave that to a separate PR.


Ness doesn't seem to pickup AWS credentials

Ness can't seem to pick up the AWS credentials that are already configured in ~/.aws/credentials The credentials are correct since I verified it by using aws on my terminal. My user is an IAM user with full privileges.
○ npx ness deploy --domain demo.mydomain.com --dir dist --spa

Please provide your AWS credentials
Where can I find these?

🔑 AWS_ACCESS_KEY_ID:
🤐 AWS_SECRET_ACCESS_KEY:

After trying to find credentials when it prompts me for the credentials, I put in my valid access & secret keys, but it's still prompting me the same info over and over again.
I've tested this with Node v12 & v14

Hi @rushi! Thanks for trying out Ness. Can you tell me which OS you're running this on?

Same here. MacOs Big Sur 11.4

Same problem. Ubuntu 20.10

@rushi @wuwuwu @laurasac any of you mind to tell me how you install Node? Standalone installer? NVM? Something else?

Nvm indeed

with apt install

Same here.
OS: Manjaro 21.0.7
Installation type: pacman -S npm

@rushi @wuwuwu @laurasac any of you mind to tell me how you install Node? Standalone installer? NVM? Something else?

I use nvm to install my node

I'm still looking into this. If I can't reproduce I may just rework the credentials lookup as this seems fairly widespread.

:tada: This issue has been resolved in version 1.7.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@rushi @wuwuwu @laurasac I believe this issue is fixed now. I name the CloudFormation stack with an ID that includes the git branch name of the repository, but I didn't realize that the tool I was using to grab the git branch throws when used in a non-git directory.


Feature/manager profit fee

Adds an optional mgmt fee parameter to PlatformSettings, set at 5% by default.   When a loan is repaid, if excess profits were made with the borrowed escrowed lending token, then 5% of those excess profits will be given to the tToken liquidity pool such that 95% of the excess profits will be claimed back to the borrower.
Unit tests were added to loans.tests.ts.

@admazzola this PR branch can be deleted as well correct?

yes - now we are to use feature/manager-profit-fee-r2.  I did that because starting fresh was easier than rebasing this branch


a simple question about org.apache.ibatis.parsing.XPathParser

org.apache.ibatis.parsing.XPathParser#evalDouble use org.apache.ibatis.parsing.XPathParser#evaluate, not like others ,for example
org.apache.ibatis.parsing.XPathParser#evalFloat(java.lang.Object, java.lang.String) use
org.apache.ibatis.parsing.XPathParser#evalString.
why?

As stated in the issue template, we use GitHub Issues only to track bugs and feature requests.
Please post your question to the mailing list or Stack Overflow.
Thank you!

FYI, these "why do you code like that?" question don't get answers very often because everyone is busy (I guess you are, too).
You might find the answer by using Git's blame feature or searching commit logs in the old subversion repository (that is what developers do when investigating OSS code).


Present curated taxonomy sidebar links

When a content item has related link overrides for the new education
(configured in content-tagger), then govuk_navigation_helpers will
present a hash that contains these curated links, rather than the
Elasticsearch “More like this” links generated by rummager.
Curated links are grouped according to the following rules:

If a related item has a taxon in common with the content, then display
the related item under that taxon in the sidebar (avoiding duplicates)
Otherwise display a related item under the “Elsewhere on GOV.UK”
section
Any external related links are displayed under “Elsewhere on the web”.
External related links currently use the same external links as the
“old” related items sidebar. In order to prevent sidebar overrides
for content items with external related links, but no “new” GOV.UK
related links, we only display curated links if there are GOV.UK
related links present. This means it is currently impossible to
display only external related links in the new navigation.

Before

After

Dependencies

[x] https://github.com/alphagov/static/pull/1065

Trello
https://trello.com/c/J8ofEnrt/167-display-curated-related-links-in-the-sidebar

@carvil changes done.

@carvil markups are done.


How to specify the installation of 32-bit software on a 64-bit system

My Windows system is 64-bit, usually the default installation via scoop install command follows the system 64-bit programs, but I need to install 32-bit programs, what parameters do I need to add to the scoop install command?
After installing officetoolplus, you need to install dotnet-desktop-runtime at the same time to start properly, but 64-bit dotnet installed by scoop install cannot start, only 32-bit dotnet can start properly

if x64 version is not required with you:
scoop install dotnet-desktop-runtime -a 32bit

for installing the dotnet desktop runtime x86 and x64, suggested using the official installer or visual studio.
if you must install them via scoop:
you need to customize the manifest, otherwise, you can only install one of them. (x86 or x64)
e.g.
{ cd to path of dotnet-desktop-runtime.json }
cp dotnet-desktop-runtime.json dotnet-desktop-runtime-x86.json
scoop install dotnet-desktop-runtime-x86 -a 32bit
scoop install dotnet-desktop-runtime

this is not the best solution, but does not require any modifications of the manifest.

@HUMORCE How to install both 32-bit and 64-bit dotnet-desktop-runtime with scoop install, while scoop update * can always automatically upgrade 32-bit and 64-bit dotnet-desktop-runtime?

clearly in the previous comment.

@HUMORCE Thank you


Openstack Resize UI's flavor dropdown should include flavor details.

The list of flavors on the Openstack Resize UI form should include details about the flavor (CPU, Memory, Disks). This was included at one point, but it looks like it got lost in a merge.
Additionally, this limits the list of available flavors to only those with disks equal in size or bigger than the current flavor of the instance.
@Loicavenel

@blomquisg is this for darga?

@chessbyte yeah this should be for darga

@miq-bot add_label darga/yes

:+1: thanks @mansam!


Update to version 0.3.0

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Brought up to date with version 0.3.0 (newest version on pypi)

@conda-forge-admin please rerender

@conda-forge-admin, please rerender

Let's just continue in #2, I'll close this one


non-unicode Active selection

for now there is \U0001F449 used but it would be nice to have option to change this somehow
for example through config, flag or environment variable
it would be useful for those who don't have fonts with this unicode

https://github.com/gennaro-tedesco/gh-s/blob/dd35856aa044749f1a857ee3ae0fd0bc21134ed2/cmd/ui.go#L76

You are completely right, I overlooked it. These days I am away from the computer but I will propose an alternative as I am back (a practical solution could be to check for fonts and default back, so that users do not have to bother with it, although I am not sure how easy it is to do programmatically).


Type error with os.umask call in celerycam command

With celery 2.4.0 and django-celery 2.4.0 (on both osx and ubuntu), when I run ./manage.py celerycam , i get the following stacktrack:
...
File "/home/ubuntu/src/django-celery/djcelery/management/commands/celerycam.py", line 22, in handle
ev.run(*args, **options)
File "/usr/local/lib/python2.7/dist-packages/celery/bin/celeryev.py", line 35, in run
detach=detach)
File "/usr/local/lib/python2.7/dist-packages/celery/bin/celeryev.py", line 64, in run_evcam
with detached(logfile, pidfile, uid, gid, umask, workdir):
File "/usr/local/lib/python2.7/dist-packages/celery/platforms.py", line 211, in open
os.umask(self.umask)
TypeError: an integer is required
Unlike in v2.3.x, there are no more arguments for --umask, etc.
Workaround?

Thanks!  This was fixed in ab9318f828fe9bfdcb4cede962ceff7b07fdc2c3.  2.4.1 will be released shortly!


Unable to set min-clients-per-thread from keydb.conf

Describe the bug
When setting min-clients-per-thread in keydb.conf, after deployment, the value is set to some random value.
For e.g., when I set it to 4, after deployment, when I login to the VM, connect to keydb via keydb-cli, I can see the value set to 28. Not sure why :)
I am able to set this value by directly running config set min-clients-per-thread 4 directly with the keydb-cli, but I have to do this for every master/replica.

@msotheeswaran-sc what does priority 2 means? Can someone have a look at this?
Is this a bug or some mistake from my side?

Could be either, if it is a bug then it is marked as Priority 2 since it doesn't have a significant negative impact on KeyDB's usability.
To understand if it is some mistake from your side I would need more context as to how your deployment is setup.

This is our keydb.conf (main parts):
port 0
cluster-enabled yes
server-threads 4
server-thread-affinity true
min-clients-per-thread 4
When we deploy using this config, and check 'min-clients-per-thread' value it is set to 28. Which is a problem, as the multi threading doesn't really kicks in for us, as we less number of clients.
We can still set te value to 4, by running the command config set min-clients-per-thread 4 in each primary node.

@javedsha works fine for me with
server-threads 4
server-thread-affinity true
min-clients-per-thread 4
can you provide more specific repro instructions, I can't fix this if I can't reproduce it


Can't add record

The script always re-write the last record.

I have problem in adding new records
How can add new record ?

@se8820726 Try this
        $table = new WritableTable($copyTo);
        $record = $table->appendRecord();
        $record->set('name', 'test name');
        $record->set('bio', $newBio = str_pad('', 64 - 8, '-'));
        $table
            ->writeRecord()
            ->save()
            ->close();


GEODE-5982: Synchronized access to CacheLoader and CacheWriter

* Using read-write locks to synchronize the getters and setters for CacheLoader and CacheWriter.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

I don't really know the surrounding context very well, but at face value, I'm not really sure that this change improves anything. For example it seems that 'thread1' could have retrieved the current loader and, before finishing, thread2 changes out the loader and calls close() on the old loader before thread1 completes and is done using that old loader.

It looks like you have a bug to look into
org.apache.geode.cache30.DiskRegionIntegrationTest > testLRUCacheLoader FAILED
java.lang.IllegalMonitorStateException
at java.util.concurrent.locks.ReentrantReadWriteLock$Sync.tryRelease(ReentrantReadWriteLock.java:371)
at java.util.concurrent.locks.AbstractQueuedSynchronizer.release(AbstractQueuedSynchronizer.java:1261)
at java.util.concurrent.locks.ReentrantReadWriteLock$WriteLock.unlock(ReentrantReadWriteLock.java:1131)
at org.apache.geode.internal.cache.AbstractRegion.setCacheLoader(AbstractRegion.java:1165)
at org.apache.geode.cache30.DiskRegionIntegrationTest.testLRUCacheLoader(DiskRegionIntegrationTest.java:551)


[snackpub] read correct remote address behind load balancer

Why
follow up https://github.com/expo/snack/pull/381#discussion_r1080836193
How

read the address from the X-Forwarded-For first then the socket remote address
because cloud run doesn't log the remote address as http access log, this pr also adds remote address to log.

Test Plan

add getRemoteAddress unit test
ci passed

Approving this PR to keep you unblocked but would appreciate it if you could address the comments by either updating the code and including test cases, or explaining why you decided not to update the code.

all comments are followed up. thanks for the neat review. i'm going to merge this pr.


Export getPlayersInRadioChannel not accessible

Did you read the README?: There's a lot of information in the README, if you ask a question that is answered in the README, you will be told to "Read the README"
Describe the bug
A clear and concise description of what the bug is.
local players = exports['pma-voice']:getPlayersInRadioChannel(1)
This is throwing an error. It did it when I tried to use it and then I just tried to use another script and it did the same thing in that script.
SCRIPT ERROR: citizen:/scripting/lua/scheduler.lua:867: No such export getPlayersInRadioChannel in resource pma-voice
To Reproduce
Steps to reproduce the behavior:
Just execute the line above and the error happens
Expected behavior
A clear and concise description of what you expected to happen.
I expect a list of players in the requested channel
Server Info:

Server Version: [e.g. 3456] FXServer-master SERVER v<IP_ADDRESS>02 win32
Client Version: [e.g. production|canary]  production
Client Build: [e.g. b1604|b2060|b2189|b2372] 1604 on my local
Game: [e.g. FiveM|RedM] FiveM

Additional context
Add any other context about the problem here.
I've tried this on a couple of servers, I am not sure of the build versions of the other because it's not mine but it's a live zap hosted server that is kept up to date.

Works fine, but it only works server side.


TypeError: Cannot read properties of undefined (reading 'split')

i'm trying to deploy docker image through sls
functions:
  generatePdf:
    # handler: src/api/generate.pdf
    images:<PHONE_NUMBER>66.dkr.ecr.ap-southeast-1.amazonaws.com/automate-pdf:latest@sha256:98e17f123e538564dff238122cc31ab38ddf1e735e4cc5f7be853cc2cc5e1dee
    description: A lambda handler that generates PDF file from payload data and upload it to S3 bucket.
    memorySize: 2048
    provisionedConcurrency: 2
    events:
      - http:
          path: pdf
          method: POST
          cors: true
          documentation:
            summary: Generates PDF from payload
            tags:
              - generatePdfS3

it works perfectly with handler but the moment i switch to images, it throws this weird error
TypeError: Cannot read properties of undefined (reading 'split')
    at /Users/rijwan/code/second/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/typescript.js:57:33
    at Array.map (<anonymous>)

my sls version is as below:
Framework Core: 3.22.0
Plugin: 6.2.2
SDK: 4.3.2

any update?

still getting error
❯ sls deploy --verbose
Running "serverless" from node_modules

Deploying pdf-api to stage dev (ap-southeast-1)

Packaging
Compiling with Typescript...
Using local tsconfig.json - tsconfig.json

✖ Stack pdf-api-dev failed to deploy (0s)
Environment: darwin, node 12.16.3, framework 3.22.0 (local) 3.22.0v (global), plugin 6.2.2, SDK 4.3.2
Credentials: Local, "default" profile
Docs:        docs.serverless.com
Support:     forum.serverless.com
Bugs:        github.com/serverless/serverless/issues

Error:
TypeError: Cannot read property 'split' of undefined
    at /Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/typescript.js:57:33
    at Array.map (<anonymous>)
    at Object.extractFileNames (/Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/typescript.js:56:10)
    at TypeScriptPlugin.get rootFileNames [as rootFileNames] (/Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/index.js:102:27)
    at TypeScriptPlugin.<anonymous> (/Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/index.js:159:59)
    at Generator.next (<anonymous>)
    at /Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/index.js:8:71
    at new Promise (<anonymous>)
    at __awaiter (/Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/index.js:4:12)
    at TypeScriptPlugin.compileTs (/Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/index.js:143:16)
    at TypeScriptPlugin.<anonymous> (/Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/index.js:58:28)
    at Generator.next (<anonymous>)
    at /Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/index.js:8:71
    at new Promise (<anonymous>)
    at __awaiter (/Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/index.js:4:12)
    at before:package:createDeploymentArtifacts (/Users/rijwan/code/automate-pdf-api/node_modules/serverless-plugin-typescript/dist/src/index.js:57:63)
    at PluginManager.runHooks (/Users/rijwan/code/automate-pdf-api/node_modules/serverless/lib/classes/plugin-manager.js:530:15)
    at PluginManager.invoke (/Users/rijwan/code/automate-pdf-api/node_modules/serverless/lib/classes/plugin-manager.js:563:20)
    at async PluginManager.spawn (/Users/rijwan/code/automate-pdf-api/node_modules/serverless/lib/classes/plugin-manager.js:585:5)
    at async before:deploy:deploy (/Users/rijwan/code/automate-pdf-api/node_modules/serverless/lib/plugins/deploy.js:40:11)
    at async PluginManager.runHooks (/Users/rijwan/code/automate-pdf-api/node_modules/serverless/lib/classes/plugin-manager.js:530:9)
    at async PluginManager.invoke (/Users/rijwan/code/automate-pdf-api/node_modules/serverless/lib/classes/plugin-manager.js:563:9)
    at async PluginManager.run (/Users/rijwan/code/automate-pdf-api/node_modules/serverless/lib/classes/plugin-manager.js:604:7)
    at async Serverless.run (/Users/rijwan/code/automate-pdf-api/node_modules/serverless/lib/serverless.js:174:5)
    at async /Users/rijwan/code/automate-pdf-api/node_modules/serverless/scripts/serverless.js:771:9

2 deprecations found: run 'serverless doctor' for more details

Seems to me node 17 doesn't supported by was

I've run into the same issue, lambda fails when you try to use any import..
( Node 17 is not in the picture at all. I've tried it with 12 and 14. ) Tried both the 2.1.4 and the 2.1.2 version of the plugin...
Switched to serverless-esbuild and the issue went away.

@mdrijwan Running into this myself now, did you ever have any luck resolving this?


Cleanup remaining low-level intrinsics that uses Attrs to use PrimValue

After https://github.com/tlc-pack/relax/pull/392, we cleaned up CallBuiltin.
It would be great migrate the rest of the low-level intrinsics to use PrimValue, these functions include

alloc_storage/tensor
print/unique
Other possible operators that comes with specialized codegen.

Are we removing attributes entirely from the language?

I don't think we will remove Attrs entirely atm, but for some of the low-level ops that have special codegen behaviors, moving them to use PrimValue would be more beneficial(so we can for example leverage same codegen for compiled and vm)

That seems reasonable, thanks.


Unsoundness in ElfParser::get_all_program_headers() function

https://github.com/libbpf/blazesym/blob/fde205278aba4874eda69cdb179988abfb7ec3bd/src/elf/parser.rs#L566-L575
The function creates a mutable reference from a shared context. That means we can end up with multiple "exclusive" references to the same object, which is not allowed.

Similar issue seems to exist here: https://github.com/libbpf/blazesym/blob/fde205278aba4874eda69cdb179988abfb7ec3bd/src/elf/parser.rs#L542


[CELEBORN-1419] Avoid adding shuffle id repeatedly

What changes were proposed in this pull request?
Avoid adding shuffle id repeatedly
Why are the changes needed?
registerShuffle has added the shuffle id to sortShuffleIds, and there is no need to add it repeatedly when calling getWriter
Does this PR introduce any user-facing change?
None
How was this patch tested?

Thanks. I found that it's not removed when unregisterShuffle. Would you mind fixing this in passing？
- if (sortShuffleIds.contains(appShuffleId)) {
+ if (sortShuffleIds.remove(appShuffleId)) {

Done


Delete shopping list

Codecov Report

Merging #17 (ea28ed9) into main (010efba) will decrease coverage by 6.03%.
The diff coverage is 34.14%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@              Coverage Diff              @@
##                main      #17      +/-   ##
=============================================
- Coverage     100.00%   93.97%   -6.03%
- Complexity       111      120       +9
=============================================
  Files             31       35       +4
  Lines            420      448      +28
=============================================
+ Hits             420      421       +1
- Misses             0       27      +27

Impacted Files
Coverage Δ

...opping/Command/DeleteShoppingListByUuidCommand.php
0.00% <0.00%> (ø)

...Command/DeleteShoppingListByUuidCommandHandler.php
0.00% <0.00%> (ø)

.../Fragment/DeleteShoppingListFragmentController.php
0.00% <0.00%> (ø)

...Controller/Shopping/GetShoppingListsController.php
100.00% <ø> (ø)

...ine/Repository/Shopping/ShoppingListRepository.php
92.30% <0.00%> (-7.70%)
:arrow_down:

...roller/Shopping/AbstractShoppingListController.php
100.00% <100.00%> (ø)

...Controller/Shopping/SaveShoppingListController.php
100.00% <100.00%> (ø)

...Controller/Shopping/ShowShoppingListController.php
100.00% <100.00%> (ø)

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


[Bug] Mark beta releases as "pre-release" in GitHub

The problem
I believe when you make a GitHub release and check "Set as a pre-release" it will mark it as a beta release in HACS. This is important because it will then only show up for users who have opted to download beta releases.
It's been a while since I've published a beta in HACS, so correct me if I'm wrong on this, but this is how I recall it working.
Your device model
N/A
Firmware type
Stock
Firmware version
N/A
Integration version with the issue
N/A
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
2023.10.5
Is there anything useful in the logs?
No response
Diagnostics information
No response
Additional information
No response

Hello. Thanks for the information :+1: I have marked both betas as pre-release and checked it. It seems to work as you described


Upgrade Java version to patch 1:17.0.7+7-1.amzn2.1

Thank you for taking the time to help improve Corretto.
If your request concerns a security vulnerability then please report it by email to<EMAIL_ADDRESS>instead of here.
(You can find more information regarding security issues at https://aws.amazon.com/security/vulnerability-reporting/.)
If your proposal is specific to Corretto docker images,
then you are in the right place.
Please proceed with the following.
Is your feature request related to a problem?
Please provide a clear and concise description of what the problem is.
Using the Public ECR release of Corretto Java 17 with this directive in the Dockerfile:
FROM public.ecr.aws/amazoncorretto/amazoncorretto:17
The Corretto 17 image has not been updated to include the patch for a high vulnerability.  I am not able to update it in the dockerfile due to repository priority protections being enabled.
Step 12/22 : RUN yum update -y java-17-amazon-corretto
--
232 | ---> Running in 3a4668419221
233 | Loaded plugins: ovl, priorities
234 | 9 packages excluded due to repository priority protections
235 | Package(s) java-17-amazon-corretto available, but not installed.
236 | No packages marked for update

I would like to be able to get this patched or know when AWS will be updating the base image.  Is there another workaround other than changing this echo "priority=9" >> /etc/yum.repos.d/corretto.repo
Any ideas or suggestions are welcome.
Describe a solution you would like
Please provide a clear and concise description of what you want to happen.
Describe alternatives you have considered
Please provide a clear and concise description
of any alternative solutions or features you have considered.
Additional context
Add any other context or screenshots about the feature request here.

The public.ecr.aws/amazoncorretto/amazoncorretto:17 contains the latest version of Corretto 17 available. We currently do not use the package from the AmazonLinux YUM repository but a generic Linux RPM to ensure we can get our updated docker images out prior to the YUM repository update propagation.
$ docker run -it public.ecr.aws/amazoncorretto/amazoncorretto:17 java -version
openjdk version "17.0.7" 2023-04-18 LTS
OpenJDK Runtime Environment Corretto-<IP_ADDRESS>.1 (build 17.0.7+7-LTS)
OpenJDK 64-Bit Server VM Corretto-<IP_ADDRESS>.1 (build 17.0.7+7-LTS, mixed mode, sharing)

OK, that makes sense @lutkerd.  What is the plan when it comes to security patches and advisories, e.g. https://alas.aws.amazon.com/AL2/ALAS-2023-2025.html
Would that just be rolled into the next latest version above Corretto-<IP_ADDRESS>.1 (build 17.0.7+7-LTS)

https://github.com/corretto/corretto-docker/pull/114 is the next step. We are planning to provide images with the AL2 packages so that the scanners are clean.

could this be pushed upstream faster as the official tomcat images are all failing security scans


i18n(zh-cn): Update 2-pages/3.mdx

Description (required)
Update 2-pages/3.mdx
I commit some bold syntax and indent in doc to change the status of Docs translation.
Related issues & labels (optional)

#9240
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

zh-cn/tutorial/2-pages/3.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Not able to add Group in attribute family.

Framework Version: 0.1.5

Framework Version: 0.1.5

Fixed


🐛🐛  Bug Report: msg.mention_text() 返回值不正常

requirements

puppet-xp as the gateway
0.4.15

Describe your problem

msg.mention_text() 返回值不正常

Reproduce your problem
if msg.room():
            t=msg.text()
            print(msg.mention_text(),t,msg.text())

Error info
Contact <lonely2012> <晓燕>
2022-01-21 10:31:13,364 - Contact - INFO - load contact <Contact <wxid_tnv0hd5hj3rs11> <无空>>
2022-01-21 10:31:13,375 - Contact - INFO - load contact <Contact <wxid_a6xxa7n11u5j22> <大兄弟666>>
2022-01-21 10:31:13,376 - Wechaty - INFO - receive message <Message#message_type_text[🗣 Contact <wx id_tnv0hd5hj3rs11> <无空>@👥 Room <18638045265@chatroom - >]        @晓燕 美 女早>
<coroutine object Message.mention_text at 0x0000022D805D9540> @晓燕 美女早 @晓燕 美女早

Your experiments

只回复群里@bot的消息，

先share一个临时替补方案把，群友 didnhdj给的思路，代码如下：
t=msg.text().replace('@晓燕','')
然后用这个t代替 msg.mention_text()

Yes, there are some differences between puppets, this is the one. And it’s a bug in puppet-xp, so you can use this solution this time.

Yes, there are some differences between puppets, this is the one. And it’s a bug in puppet-xp, so you can use this solution this time.

maybe I should report it on the puppet-xp  page

@WukongZeming  cool, please go ahead.

(https://github.com/wechaty/puppet-xp/issues/59)


fix: #308 include req.readableAborted flag to defaultSuccessfulRequestMessageProvider

Closes #308

@mcollina done

A test is failing on node v14

@mcollina, do you have some plans to drop node v14 support?

Of course, might actually be time to drop v14 and v16.

Could you remove them from CI? I will bump the major.


Cancelling filling animation

It is my understanding that Animation.cancel is supposed to remove any remaining fill effects of KeyframeEffects that where part of the animation even after it has finished.
I am animating page transition and use the fill property to avoid flicker during animation and need to clear the fill effects after the animation finished. Unfortunately it doesn't seem to work reliably in my scenario:
Following examples animate outgoing page from opacity: 1 to opacity: 0 while the incoming page is animated with a transform.
http://jsbin.com/zeyajemuko/edit?html,output
http://jsbin.com/jobavopixo/edit?html,output
First example seems to work fine. Second is a bit more complex with 3 KeyframeEffects being used. After clicking "Next Page" 4 times you should see that both pages get stuck in opacity: 0 filling animation. So it appears that it does work in first 2 page transitions (1 -> 2 and 2->1) but stops after that for some reason.
As you can see I am detaching the outgoing page from dom after the animation - I wonder it this has anything to do with it. Please ignore the polymer bits and just look at _animateTransition methods. Thanks.

I also noticed there might be an issue with animation.onfinish not firing in jsBin for some reason with Script error. (line 0), but it is called when I host it as a stand-alone page.

You can try using anix, a simpler cartoon library
https://github.com/a-jie/AniX


[Feature] Added release version api for clients and RTS

Description
Adds a new API to get current release version of the appsmith server. It also broadcast the version to RTS through redis if server is reloaded. The RTS broadcasts this message to every sockets.
Fixes #685
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Manually

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: feature/add-release-version-api

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
54.89 (0)
36.69 (-0.01)
33.9 (0)
55.46 (-0.01)

:red_circle:
app/client/src/sagas/WebsocketSagas/handleAppLevelSocketEvents.ts
15.79 (-0.57)
0 (0)
0 (0)
16.67 (-0.64)

:red_circle:
app/client/src/selectors/commentsSelectors.ts
78.23 (-1.61)
54.41 (-2.94)
70 (0)
84.09 (-2.27)

@sharat87 If RTS has a build version set, then it's enough. I'll explore more on generating build version in RTS.

Hey @nayan-rafiq, yeah we need to set the version in the following step:
https://github.com/appsmithorg/appsmith/blob/52338878909a7efdc7827a1f73c157a3ce9f236d/.github/workflows/test-build-docker-image.yml#L256-L258
Before the ./build.sh, we just need to stick in the echo command from my earlier screenshot to drop the correct version number into the version.js file.

@sharat87 I've updated the github action. Is there anyway to test it locally?


feat: add binary_sensor for when images are not "no mail"

Proposed change

Add binary_sensors for when the images are not the "no mail" or "no packages" images.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests
[ ] Documentation update
[ ] Adds a new shipper
[ ] Update existing shipper

Additional information

This PR fixes or closes issue: fixes #682

Codecov Report

Merging #684 (82ea8ab) into dev (bf60f7c) will decrease coverage by 0.25%.
The diff coverage is 92.64%.

@@            Coverage Diff             @@
##              dev     #684      +/-   ##
==========================================
- Coverage   98.05%   97.80%   -0.26%
==========================================
  Files           7        8       +1
  Lines        1389     1456      +67
==========================================
+ Hits         1362     1424      +62
- Misses         27       32       +5

Impacted Files
Coverage Δ

custom_components/mail_and_packages/__init__.py
95.90% <ø> (ø)

...stom_components/mail_and_packages/binary_sensor.py
92.30% <92.30%> (ø)

custom_components/mail_and_packages/const.py
100.00% <100.00%> (ø)


fix: resolve parseCDR error and add concatenated CDRs update

Fix Two Issues in CHF When Processing CDR

When the traffic exceeds a certain threshold (around 700KB), the Webconsole displays the error:
[ERRO][WEBUI][BillingLog] parseCDR error when unmarshal with params: offset > next,
and the usage data cannot be displayed on the Webconsole.

According to TS 32.297 Section 5.2 "File Format Principles," a CDR file contains multiple CDRs. However, in CHF, charging data is updated to only one CDR. This causes the cdrBytes length in internal/sbi/processor's dumpCdrFile() to grow increasingly larger, eventually exceeding the uint16 range and resulting in an overflow in cdrHdr.CdrLength.

Hi @TungChiaMing ,
I have tested this code  via this senario: https://free5gc.org/guide/Charging/setting/#3-test-with-ueransim
I used ping -I ueransim0 <IP_ADDRESS> ping -I ueransim0 <IP_ADDRESS> to create traffic, but still got this error:

I'm sorry but can you provide your testing process?😣


Update Core

About the PR

Fixes the vox box and changes the design a bit so that it works more optimally, alongside tweaking some of the signs and adding directional fans. Also fixes med exit button.
Resolves #29074.
Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

I still think the vox box should be in the corner of the room as to not isolate that one cafeteria table, but it looks much better at least.


Getting value after executing command programmatically

I have registered specific commands both on server and client. I am trying to fetch values from lang server when command is executed, but instead value is undefined. Is it possible to do that?

Can you please provide an example of what you want to achieve?

I have defined command in package.json, for e.g.
"commands": [
{
"command": "my_cmd",
"title": "My cmd"
}
]
What I want is to call vscode.commands.executeCommand("my_cmd") and get result returned by language server that I also created.
I have tried something like this:
var returned = vscode.commands.executeCommand('my_cmd')
.then(function(result)
{
// do something with result
});
but the result is undefined. So, is that even possible?
Thank you for your response.

From where do you call the command. From the client or the server. Command execution from the server is currently not possible and would require to specify valid commands which is almost impossible given that a server should work with n clients.

I am calling the command from the client. And on the server I have a function which has a command name and arguments as function parameters. So, I can handle different commands and return different objects from the server.
The flow is something like this:
executeCommand - client  =>  hit function - server  =>  process and return value - server => fetch value - client
and the problem is that I can't fetch value.

Actually that should work. Have you registered the commands from the server using the ServerCapabilities.executeCommandProvider property.

Hello, I am working on the same extension. I have looked at source code of vscode-languageclient module and tried to log server's response.
I have replaced this part:
client.sendRequest(vscode_languageserver_protocol_1.ExecuteCommandRequest.type, params) .then(undefined, (error) => { ... });
with this:
client.sendRequest(vscode_languageserver_protocol_1.ExecuteCommandRequest.type, params) .then((result) => console.log(result) , (error) => { ... });
to be sure that server is returning a value. And it does.
Our activation function:
export function activate(context: ExtensionContext) {
	context.subscriptions.push(startLangServer('...', [], []))

        vscode.commands.executeCommand('my_cmd')
                        .then((result)=>
                        {
                               console.log(result); // undefined
                        });
}

package.json has commands section described in above comment.
Do we need to register command programmatically, too?
Although, we tried that (and removed commands section) and result is still undefined.

@danixeee now I understand. We should continue the discussion in https://github.com/Microsoft/vscode-languageserver-node. In general this should work and it is not a specification problem. Looks more like a bug either in VS Code or in one of the LSP libraries.
Can you try to produce a minimal example using the LSP client and server module I can clone and open a new issue here: https://github.com/Microsoft/vscode-languageserver-node. Then I can definitely have a look and see why it is not working.


Add standard xgboost tests to list of tests.

Only issue is tests are often nose tests and not pytests.  But need to test this since github.com/h2oai/xgboost h2oai branch being used isn't otherwise itself tested, and it's diverged from dmlc/xgboost master branch.

https://docs.pytest.org/en/latest/nose.html
https://wiki.jenkins.io/display/JENKINS/Version+Number+Plugin

Should we run those tests as part of h2o4gpu or make a new build for XGBoost repo? Think we only want to run them when something in XGBoost changes? And eventually have some h2o4gpu+xgboost specific tests here (i.e. testing the interaction between the two).

Well, could jenkins-ify the entire h2oai branch of our xgboost repo, but I'd prefer to keep h2o4gpu as self-contained as possible.  We do have some xgboost specific tests in h2o4gpu, but I prefer that someone can easily test the xgboost side if they want.  It's easy enough and the tests don't take very long.

Do you know how long do they take exactly?

Doing all the python tests takes 5 minutes, but not all pass (on purpose), so have to loop over each one and check if passes or fails in expected way.

5 min should be ok, especially if me and @abal5 will get around to parallelizing the build. Not sure if possible but ideally I'd like:
-3 builds in parallel a) normal build + tests b) nonccl cuda8 build c) nonccl cuda9 build

if all of the above pass then a publishing task that would publish all artifacts from above

For jobs that build but don't run I think we can utilize linux machines with CUDA installed even if there's no GPU present?
ad. tests - I see, you mean like they throw exceptions on purpose etc. or something else? with exceptions, I think we might be able to write tests to check if an exception was thrown and only fail the test if not (and that would be an actual failure).

There are some gpu tests there, so need gpus.  It's possible there should be no errors, but that they don't check all the tests during their dmlc jenkins-like runs.  I'll look into it when I get to this.

Better to have h2oai/xgboost h2oai branch as separate jenkins pipeline


Intermediate results from subscriptions or queries overwrite optimistic updates

This is related to https://github.com/FormidableLabs/urql/discussions/2349, but I'm posting this here as a bug since it's surprising behavior, and I've created a small repro case to illustrate the effect.
urql version & exchanges:
"@urql/core": "^2.4.3",
"@urql/exchange-graphcache": "^4.3.6",

Steps to reproduce

I have a mutation that updates a Todo.
I have a subscription that listens for Todo updates.
I have an optimistic update setup on a completeTodo mutation.

Consider the following sequence of events:

I execute a completeTodo mutation. The todo is optimistically mutated according to my arguments and the request is sent to the server.
I receive an answer from the server with the updated Todo: my todo is now perfectly synced between client and server.
I mutate the same Todo again.
I receive a subscription event with the Todo from (1.). The optimistically updated Todo is rolled back from its state at (3.) to its state at (2.).
I receive an answer from the server  with the updated Todo. The todo is rolled back from its state at (4.) to its state at (3.).

Expected behavior
The Post should not be rolled back to its state at (2.) but instead roll back the optimistic update, apply the update from the subscription, then replay the optimistic update while we have yet to receive word from the server. Hence, the order of states should always be (2.) -> (3.) and not the current (2.) -> (3.) -> (2.) -> (3.).
Actual behavior
The Post is rolled back.
Reproduction
Client: https://codesandbox.io/s/elastic-yalow-x9hctp?file=/src/index.js
Server: https://codesandbox.io/s/beautiful-worker-k7xhgq?file=/schema.js
Demonstration of the effect, which you can observe by toggling a todo in quick successions:
https://user-images.githubusercontent.com/1621758/158698143-3fbd5582-ad35-46c2-b6d0-a886fd022886.mov
The effect is even more pronounced on slow networks and when optimistic updates stack.

Conceptually a subscription reflects an accurate state of the server hence we have opted to not include subscription responses into our layering logic. This because you can't assume ordering mutation --> sub trigger --> query --> ... and other relevant combinations would not be deterministic if subscription responses would be layered.
Hence this isn't really a bug to me but more a server sending an invalid state from another user updating so could this be more of a UI state issue. A user updates a similar entity while also having an update present 😅

We ended up switching to apollo-client because this particular behavior was a deal-breaker for our application. Closing this since it's expected behavior.


effect order dependence when discharging multiple effects

I wrestled with the my top-level run function for a while which boiled down to this:
-- runApp :: Eff '[IOE, Trace] -> IO ()   -- swapped effect order, doesn't compile!
runApp :: Eff '[Trace, IOE] a -> IO a
runApp = runIOE . runTraceStdout

My actual app of course has a bigger list of effects.  It was surprising to see order dependence here when most of Cleff usage has order independence, for example:
myFunc1 :: '[IOE, Trace] :>> es => Eff es ()
myFunc1 = do
  trace "hi"
  liftIO $ putStrLn "there"
myFunc2 :: '[Trace, IOE] :>> es => Eff es ()  -- swapped order is okay here.
myFunc2 = do
  trace "hi"
  liftIO $ putStrLn "there"

Not a huge deal; I can easily work around this now that I know about it.  But, is there a way to specify the effects for runApp in an order independent manner?

You get order independence in the latter two examples because you're using the :> constraint (via the :>> type family).
I think this change should solve your problem:
-runApp :: Eff '[Trace, IOE] a -> IO a
+runApp :: '[Trace, IOE] :>> es => Eff es a -> IO a
 runApp = runIOE . runTraceStdout

Also, the correct order in the first example is defined by the order of your run* functions. You need to handle the Trace effect before you can handle the IOE effect, so Trace gets popped off the head of the type-level list first.
In general you should always try to use :> or :>> to avoid requiring a concrete ordering.

I think this change should solve your problem:
-runApp :: Eff '[Trace, IOE] a -> IO a
+runApp :: '[Trace, IOE] :>> es => Eff es a -> IO a
 runApp = runIOE . runTraceStdout

That fails with "Couldn't match type ‘es’ with ‘'[Trace, IOE]’".  To fully discharge an Eff it needs to be brought down to a concrete Eff '[] or Eff '[IOE] before you runPure/runIOE.

Oh yeah you're right 🤦♂ Sorry about that.
For the case where you're handling effects, you do need to know there are no effects remaining. And :>/:>> doesn't do that for you.

Yeah, unfortunately we can't have that. I think the reason is basically that Haskell doesn't have type-level Sets (except for typeclasses; but it would suck trying to implement effects with typeclasses).


when  vue version 2.1.10 meet error _h is not defined

[Vue warn]: Error when rendering component :
Uncaught ReferenceError: _h is not defined

vue-multiselect version is 2.0.0-beta.13

Weird. Will have to check what might have gone wrong.

@shentao  bro  I reboot the project .  the error is not like content above.
it's like this

Can you show me the markup and data that you pass into multiselect?

@shentao
really sorry.
it's the data 's issue that pass into component because of my incaution.
it's nothing to do with your nice component.


[INLONG-1535]Go SDK should be closed before stopping the event processing goroutine

Signed-off-by: Zijie Lu<EMAIL_ADDRESS>Title Name: [INLONG-XYZ][component] Title of the pull request
where XYZ should be replaced by the actual issue number.
Fixes #1535
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

Codecov Report

Merging #1536 (97d69b6) into INLONG-25 (34ea63c) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             INLONG-25   #1536   +/-   ##
===========================================
  Coverage         7.51%   7.51%
  Complexity         479     479
===========================================
  Files              267     267
  Lines            29500   29500
  Branches          4843    4843
===========================================
  Hits              2216    2216
  Misses           26810   26810
  Partials           474     474

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 34ea63c...97d69b6. Read the comment docs.

I also fix bug about setting the flow check id, PTAL @charlely


Improve Spring Monitor readme

cc @mattmccleary

API change check
API changes are not detected in this pull request.


Issue comparing between Tables Oracle and Tables Db2

Good morning,
i tried to compare the Tables folder from oracle connection and a Tables folder in db2.
but i get this error: "Objects of different types were selected. You may compare only objects of the same type".
db2 Driver Name: DB2 / DB2 LUW 8.x  db2jcc4.jar
ora Driver name: Oracle   ojdbc6.jar
How can i do that? Its a dbeaver issue or my fault?
Thanks
Leonardo

Good morning,
i tried to compare the Tables folder from oracle connection and a Tables folder in db2.
but i get this error: "Objects of different types were selected. You may compare only objects of the same type".
db2 Driver Name: DB2 / DB2 LUW 8.x db2jcc4.jar
ora Driver name: Oracle ojdbc6.jar
How can i do that? Its a dbeaver issue or my fault?
Thanks
Leonardo

Good evening, i found this problem between oracle and sql-server too.
I'm using version 6.1.5
Leonardo

Simple structure compare can't compare objects of different types/structure.
If you need deep table analysis / structured diff / DDL generation then you should use Schema compare tool in DBeaver Enterprise.


Extension uninstall error should be a modal dialog

I clicked to "Uninstall" an extension with dependencies and got an error toast as a result:

I think since I clicked a button explicitly, I would expect a modal dialog as a result.

Sounds low hanging? Should we close or this milestone to fix?

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


ValueError despite ignore_empty flag being True

Hi, firstly thanks for this great package!
I've been using it the last few days to some success but one problem keeps arising. Even though I have the ignore_empty flag set to True, I still get a ValueError: Hypothesis is empty.
I'm not sure why this is happening. The generated/reference summaries are so large that it's hard to look through, and there's no callback from the package to see which example is causing the error.
Here's the code I'm running to get the error:
from rouge import Rouge
rouge = Rouge()
scores = rouge.get_scores(generated.tolist(), references.tolist(), avg=True, ignore_empty=True)

(It's a little hard/useless to show the code used to generate the two lists)
Looking through the source code, I can't see why it's erroring out, as both the hypothesis and reference summaries' lengths are checked for being nonzero with this flag set.
Any suggestions as to what I'm doing wrong here?
Also related: would it be possible to add a callback where it says the list index which causes the error?

I just encountered a similar problem. Apparently, the problem is processing strings consisting of dots.
How to reproduce:
from rouge import Rouge
rouge = Rouge()

hyps = [".", "1"]
refs = ["2", "3"]

rouge.get_scores(hyps, refs, avg=False, ignore_empty=True)

The reason is the splitting sentences by dots and filtering them by zero length: https://github.com/pltrdy/rouge/blob/6d443d9c95d86cce4629ef0410110a76077498fb/rouge/rouge.py#L140
@am-berry, as a quick fix, you can filter all your hypotheses from lines consisting of dots.

@gooppe Thanks for the quick reply!
Looks like that was indeed the problem and filtering for it fixed the problem. Thanks again!
I'm going to leave the issue open for the time being as it's probably worth a fix within the package itself...


Installation script on MacOS fails on Macs with C++17

Preliminary Steps
Please confirm you have...

[X] reviewed How Linguist Works,
[X] reviewed the Troubleshooting docs,
[X] considered implementing an override,
[X] verified an issue has not already been logged for your issue (linguist issues).

Problem Description
On MacOS with (all recent?) versions of xcode, the script/bootstrap fails because Charlock Holmes can't be installed.
This relates to a known issue with Charlock - https://github.com/brianmario/charlock_holmes/issues/146 - namely that it relies on a deprecated setting in C++17.
The bootstrap script attempts to install Charlock 0.7.7 even if it's already there, so fails to get further.
To install Charlock  0.7.7 on a Mac you need to include the following options to the gem build
--with-icu-dir=/usr/local/opt/icu4c --with-cxxflags='-Wno-reserved-user-defined-literal -std=c++11'
... and I'm trying to work out how to add that to the bundle options :-)

My ~/.bundle/config file contains:
$ grep CHAR ~/.bundle/config
BUNDLE_BUILD__CHARLOCK_HOLMES: "--with-icu-dir=/usr/local/opt/icu4c --with-cxxflags=-std=c++11"
$

... which I would have set a long time ago, but probably used something like:
bundle config build.charlock_holmes --with-icu-dir /usr/local/opt/icu4c --with-cxxflags=-std=c++11

... so to get your flags, something like this might do the trick:
bundle config build.charlock_holmes --with-icu-dir /usr/local/opt/icu4c --with-cxxflags='-Wno-reserved-user-defined-literal -std=c++11'

@lildude trying your suggestion didn't work, here is the output
❯ bundler _1.17.3_
Fetching gem metadata from https://rubygems.org/.......
Fetching gem metadata from https://rubygems.org/.
Resolving dependencies...
Using rake 13.0.1
Using public_suffix 4.0.5
Using addressable 2.7.0
Using bundler 1.17.3
Using byebug 11.1.3
Fetching charlock_holmes 0.7.7
Installing charlock_holmes 0.7.7 with native extensions
Gem::Ext::BuildError: ERROR: Failed to build gem native extension.

current directory:
/Users/epokhe/linguist/vendor/gems/ruby/2.6.0/gems/charlock_holmes-0.7.7/ext/charlock_holmes
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby -I
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0 -r
./siteconf20200702-75167-t4l8au.rb extconf.rb --with-icu-dir\ /usr/local/opt/icu4c\
--with-cxxflags\=-Wno-reserved-user-defined-literal\ -std\=c\+\+11
checking for -licui18n... yes
checking for unicode/ucnv.h... yes

...

In file included from
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/include/ruby-2.6.0/ruby.h:33:
In file included from
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/include/ruby-2.6.0/ruby/ruby.h:24:
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/include/ruby-2.6.0/universal-darwin19/ruby/config.h:413:31:
error: invalid suffix on literal; C++11 requires a space between literal and identifier
[-Wreserved-user-defined-literal]
#define RUBY_ARCH "universal-"RUBY_PLATFORM_OS
                              ^

/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/include/ruby-2.6.0/universal-darwin19/ruby/config.h:414:35:
error: invalid suffix on literal; C++11 requires a space between literal and identifier
[-Wreserved-user-defined-literal]
#define RUBY_PLATFORM "universal."RUBY_PLATFORM_CPU"-"RUBY_PLATFORM_OS
                                  ^

/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/include/ruby-2.6.0/universal-darwin19/ruby/config.h:414:55:
error: invalid suffix on literal; C++11 requires a space between literal and identifier
[-Wreserved-user-defined-literal]
#define RUBY_PLATFORM "universal."RUBY_PLATFORM_CPU"-"RUBY_PLATFORM_OS

...

2 warnings and 3 errors generated.
make: *** [transliterator.o] Error 1

make failed, exit code 2

Gem files will remain installed in
/Users/epokhe/linguist/vendor/gems/ruby/2.6.0/gems/charlock_holmes-0.7.7 for inspection.
Results logged to
/Users/epokhe/linguist/vendor/gems/ruby/2.6.0/extensions/universal-darwin-19/2.6.0/charlock_holmes-0.7.7/gem_make.out

An error occurred while installing charlock_holmes (0.7.7), and Bundler cannot continue.
Make sure that `gem install charlock_holmes -v '0.7.7' --source 'https://rubygems.org/'` succeeds
before bundling.

In Gemfile:
  github-linguist was resolved to 7.10.0, which depends on
    charlock_holmes

You got me stumped. Looks like the args are making it through though. Your best bet is to seek help from the upstream charlock_holmes repo. There seem to be quite a few issues like this.

I fixed line 414 manually and saved as sudo, that solved the problem 🤷♂

I fixed lines 413-414 in config.h manually and saved as sudo, that solved the problem 🤷♂

I know I'd missing something obvious, but which config.h ?

Sorry for not being clear @yourpropertyexpert
The errors I got points to a config.h file and line 413-414. The first one from the snippet above:
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/include/ruby-2.6.0/universal-darwin19/ruby/config.h:413:31:
error: invalid suffix on literal; C++11 requires a space between literal and identifier

For each line marked by the errors, I went there and put spaces in between literals and identifiers, changing them from this
#define RUBY_ARCH"universal-"RUBY_PLATFORM_OS

To this
#define RUBY_ARCH "universal-" RUBY_PLATFORM_OS

Hope that helps

It's not your fault... I really appreciate your patience.
I've been programming for 30+ years, but this is my first attempt at Ruby... and I found something like 4 different files called config.h in the repo :-)

I've updated the "Contributing" documentation, and made a PR - #4910
Given it's a problem with an upstream dependency, I don't see a plausible way to fix it in this project (and it has been logged with them.)
But, I thought I could at least try to update the docs.

I did the same to fix the issue:
#define RUBY_PLATFORM_OS "darwin19"
-#define RUBY_ARCH"universal-"RUBY_PLATFORM_OS
-#define RUBY_PLATFORM"universal."RUBY_PLATFORM_CPU"-"RUBY_PLATFORM_OS
+#define RUBY_ARCH "universal-" RUBY_PLATFORM_OS
+#define RUBY_PLATFORM "universal." RUBY_PLATFORM_CPU "-" RUBY_PLATFORM_OS
#endif /* INCLUDE_RUBY_CONFIG_H */

Bump, since there's an open PR.

Closing as we now recommend against using the macOS/XCode supplied version of Ruby in the docs.


Enhance settings by showing user information

Description:
Currently, clicking on the username in the settings does not display any user details. This could be enhanced by showing the user’s information, such as their email address.
Expected Behavior:
Clicking on the username in settings should display user information, including their email and other relevant details.
Steps to Reproduce:

Go to settings.
Click on the username.
Notice that no additional user information is displayed.
Proposed Solution: Enhance the UI to display the user's email and other relevant information when their username is clicked.

Name & show phrase should be moved to Identity.

Work Completed :

Updated the page with the new design requested by @ehab-hassan.

Verified on Android 14,
Threefold-Connect: 4.0.0-184
APK:12-12-2024
The enhancement has been implemented by relocating "The User" and  "Show phrase" under "Identity".

Related Link:

https://app.testlodge.com/a/26076/projects/40893/suites/206114?expand_section=407477#case_4427223


Increase timeout for build step of concurrently in tests

Sometimes the default Jest timeout of 5000ms is reached on GitHub CI.

Coverage remained the same at 100.0% when pulling e028cdf87bd36b84e93596c04e4590581cfa0960 on paescuj:increase-timeout-test-concurrently-build into 2e8eaa5dc8a1e3a562c11f8bd7a77c647fe87f9d on open-cli-tools:main.


upgrade zookeeper due to security issue

What is the purpose of the change
Upgrade due to CVE
https://github.com/apache/dubbo/issues/9798
Brief changelog
Verifying this change

Checklist

[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.
[ ] 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.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] 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 sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #9797 (e6371f8) into 3.0 (dedf281) will decrease coverage by 0.27%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                3.0    #9797      +/-   ##
============================================
- Coverage     65.65%   65.38%   -0.28%
  Complexity      319      319
============================================
  Files          1208     1208
  Lines         52562    52562
  Branches       7983     7983
============================================
- Hits          34511    34366     -145
- Misses        14305    14439     +134
- Partials       3746     3757      +11

Impacted Files
Coverage Δ

...dubbo/remoting/zookeeper/ZookeeperTransporter.java
55.55% <0.00%> (-33.34%)
:arrow_down:

...ng/zookeeper/curator5/Curator5ZookeeperClient.java
43.47% <0.00%> (-28.02%)
:arrow_down:

.../remoting/api/SingleProtocolConnectionManager.java
61.53% <0.00%> (-23.08%)
:arrow_down:

.../apache/dubbo/rpc/filter/ProfilerServerFilter.java
62.85% <0.00%> (-22.86%)
:arrow_down:

...nt/metadata/ServiceInstanceHostPortCustomizer.java
65.78% <0.00%> (-21.06%)
:arrow_down:

...ache/dubbo/rpc/proxy/InvokerInvocationHandler.java
68.00% <0.00%> (-16.00%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
46.15% <0.00%> (-15.39%)
:arrow_down:

...ian2/dubbo/AbstractHessian2FactoryInitializer.java
50.00% <0.00%> (-11.12%)
:arrow_down:

...mmon/threadpool/support/AbortPolicyWithReport.java
73.68% <0.00%> (-10.53%)
:arrow_down:

.../common/threadpool/serial/SerializingExecutor.java
70.37% <0.00%> (-7.41%)
:arrow_down:

... and 24 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 dedf281...e6371f8. Read the comment docs.


⚠ ClamAV has degraded performance

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

HTTP code: 200
Response time: 80 ms

Resolved: ClamAV performance has improved in 8f769d9 after 5 minutes.


Add logging to generation

Feature Request
Description of Problem:
Currently while running the data generator there is no feedback on the status of generation.
Potential Solutions:
Add console logging to provide users with feedback such as the progress of tree walking, which step is being executed etc.

Priority: Low


Remove HHVM support

Composer no longer supports it, so we can no longer support it as well as it won't run in CI

See more at: https://github.com/composer/composer/issues/8127

Coverage remained the same at 100.0% when pulling d6a767cd17b91a037f9056bcee184f414f524a94 on feature/remove-hhvm into cef87787f96d774ac9469a537a32d06b93f2bf72 on master.

Coverage remained the same at 100.0% when pulling d6a767cd17b91a037f9056bcee184f414f524a94 on feature/remove-hhvm into cef87787f96d774ac9469a537a32d06b93f2bf72 on master.


Add ability to replicate connectivity rules

There are many cases where you have a complex connectivity rule that you want to create a similar one with slight modification. Need to add ability to create a new connectivity rule from an existing one. This will be limited to only "vector" and "pattern" type rules. User will select a rule as a base and values will be populated in the editor as a starting point so user can adjust as needed.
The new rule should have its initial name as "old_rule_name_copy"

bumping this


configure exists on fc10

What steps will reproduce the problem?
1. sh autoconf.sh
2. mkdir build
3. cd build; ../configure

What is the expected output? What do you see instead?

checking for swig... swig
checking for java... java
checking for javac... nojavac
configure: error: no javac binary in PATH, bindings will not be built
[oobie@localhost build]$ java -version
java version "1.5.0"
gij (GNU libgcj) version 4.3.2 20081105 (Red Hat 4.3.2-7)

Copyright (C) 2007 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
[oobie@localhost build]$ cat /etc/issue
Fedora release 10 (Cambridge)
Kernel \r on an \m (\l)

[oobie@localhost build]$

Please use labels and text to provide additional information.

./configure --disable-java completes fine, but obviously does not
build java

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 10 Jun 2009 at 2:46

autoconf related. hence closing


Add column-aligner-ads

Added a new little extension I initially developed for VS Code, but it's equally (or even more) useful in ADS, so I made an ADS version as well. (I goofed up and accidentally added an icon for this due to copy/paste error, but can't edit it here since I used GitHub's direct-edit. Sent a different PR for fix that.)
This PR fixes #20936

@richardtallent There's still the two open comments to address before we can merge this in. Let us know if you have any questions!

Count is updated, icon added...

@richardtallent thanks! you should be able to see the new extension in ADS now.


[SRP] Create separate example for enable AD on storage account

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@ChenTanyi Hi, it is example changes. Why would it need ARM feedback?

@zfchen95 the last file is not the examples. It is the api change.

@zfchen95 sorry for mistake the change, it is just a example


Add support for Laravel migration

When loaded in a Laravel project, provide a means to:

[ ] run migrations at a click of a button,
[ ] create migration

@damms005 i would love to work on this!

Sweet! Dig right in and let me know if you have any questions.
You may start with the extension backend code because I am yet to make the frontend Vue code repo public yet.
Cheers! 🥂

No longer a priority/good fit


Mention babel-react-optimize

closes #9

Thanks, great addition!
1. One thought about babel-react-optimize: should we avoid mentioning it? The primary use-case of this repo (as I see it) is:

a reader with a non-optimized app comes here,
they read over the libraries
and they apply plugins/tips from suitable sections to optimize the app.

babel-react-optimize doesn’t fit this use-case (the tip doesn’t help to make the app smaller); moreover, in fact, it advertises this solution (some readers might neglect the warning or skip it and install this preset)
2. Also: should we give an example of the transformation that the transform-react-pure-class-to-function plugin does? E.g., like we do for date-fns.

Hm, I see your point. However, I think people come here to learn about "how can I make my app faster/reduce bundle size?". So to say they are here to learn.
This is why I would teach them "not everything is good".

But I guess this one would be even better: Get a PR merged to babel-react-optimize to warn there, instead of on a 3rd party repo :)

Good idea

Thinking about it again, and I think youʼre right, it makes sense for this repo!
Looking forward to merging this when we add transform-react-... examples :)


Throw an exception when steps are defined which will not be executed

Use case(s)
For example, if a step is defined in a nested context:
"Given something".x(() => "With something nested".x(() => { }));

The nested step will never be executed, and will effectively fail silently.
Description
When a step is defined in a nested context, or in any other place where it will not be executed, such as outside a scenario or background method, an exception is thrown.
Alternatives
An analyzer, but this would be quite tough to pull off and would add more packaging and deployment overhead, etc.

Closed by https://github.com/xbehave/xbehave.net/pull/612.

@BrunoJuchli I've released this enhancement in 2.4.1-rc.1. Would you mind giving it a try to check it works as expected?

@adamralph I should get a chance to do so on Thursday. Will report back.

@adamralph
I've updated our code to use 2.4.1-rc.1. Existing tests work as before. When i try nesting, it works as expected.

I was wondering though whether it would be sensible to have the a more comprehensive exception message to point the user in the right direction:
Notice, though, that English is not my mother-tongue.

Off-topic: do you really live in Flims? Nice place.

@BrunoJuchli that's not a bad idea. Since 2.4.1 has already gone into RC (I probably should have given you a beta first) I'll release RTM as-is, but I'll raise another issue to improve the exception message in 2.4.100.


ISPN-6557 Obtain the Matcher eagerly in QueryEngine and avoid further costs to look it up

Also add a test to measure perf impact of query parameters.
https://issues.jboss.org/browse/ISPN-6557
https://issues.jboss.org/browse/ISPN-6578

@mcimbora This PR has a minor perf improvement for query parsing times. And I've also added a slightly modified form of the perf test you wrote (see NamedParamsPerfTest). The difference is this one runs multiple iterations and averages the results. The query cache is cleared before each iteration so they do not interfere. Running multiple iterations is absolutely necessary to ensure warm-up.
I get these timings on this branch:
NamedParamsPerfTest.testNamedParamPerfComparison t1 (avg, us) = 753.439766
NamedParamsPerfTest.testNamedParamPerfComparison t2 (avg, us) = 589.218402
NamedParamsPerfTest.testNamedParamPerfComparison t3 (avg, us) = 554.143199
If I run the same test on the previous commit before the named param refactorings (750564a5e44405ae47db9ade4399f5a24d1b2215) I get:
NamedParamsPerfTest.testNamedParamPerfComparison t1 (avg, us) = 1436.376647
NamedParamsPerfTest.testNamedParamPerfComparison t2 (avg, us) = 1228.3272339999999
NamedParamsPerfTest.testNamedParamPerfComparison t3 (avg, us) = 1210.666804
Please note the times as in microseconds not millisecond.

And here are the timings on 8.2.x:
NamedParamsPerfTest.testNamedParamPerfComparison t1 (avg, us) = 1512.078813
NamedParamsPerfTest.testNamedParamPerfComparison t2 (avg, us) = 1287.570161
NamedParamsPerfTest.testNamedParamPerfComparison t3 (avg, us) = 1256.965347

Those numbers look very promising! The code also looks fine but as usually I'm picky about commits without JIRA reference...
Could you please squash them or simply add some JIRAs for them? Once this is done I can pull it.

I've created and added a jira ref to the perf test commit. The 3 commits are not related so I do not feel like squashing them. The other one ("Initialisation of arrays of literals ...") is really something not worth a jira.

Ok sounds good. Let's give CI time to process it.

Pushed to master, thanks


not Release 0.36

Description
Please summarise the changes.
Related issues
Please mention any github issues addressed by this PR.
Checklist

[ ] I have performed a self-review of my code.
[ ] I have commented hard-to-understand parts of my code.
[ ] I have made corresponding changes to the public API documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have updated the changelog with any user-facing changes.

the extension is currently not working, I would still like to merge the changes and then not do the release.


Login error (2406): skill id invalid

Can someone help me with. I have followed the information and it doesn’t seem to want to work. When I try and setup the integration I get the error:
Login error (2406): skill id invalid

Please refer - https://github.com/tuya/tuya-home-assistant/blob/main/docs/error_code.md
try choosing the country that you have selected in your tuya/smart life app.(this is not the tuya iot account country)


Add go 1.11 module files so that you can build outside the GOPATH.

In my env it fails with:
go<EMAIL_ADDRESS>unknown revision 1f054f1486df
go: error loading module requirements

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Camel-k-pr/11/


Rust analyzer

This adds Documentation on how to get rust-analyzer working in a Lint library and adds the necessary Lines to all examples.

This is awesome, @MinerSebas.
I made two nit-picky changes regarding where the metadata appears in the Cargo.toml files, and spacing around the =. (I hope you don't mind.)
Thanks a lot!


Wifi motion sensor missing

Hi all,
Would be great if you could enhance your module with the wifi motion sensor.
Thanks

Hi @Hadjimina
Do you have any update on integrating the motion sensor as a component into your library?
Thnaks for your reply!

Hi @Hadjimina
Do you have any update on integrating the motion sensor as a component into your library?
Thnaks for your reply!

Hi,
I developed this module as part of my internship at myStrom which has since finished. You could clone the repo, write it yourself, and create a pull request. You can find the documentation for the motion sensor/PIR on the official myStrom API documentation. Additionally here is the node-RED documentation.
I hope I could be of help and good luck with it. If you run into some issues understanding my code, just send me an email.

Hi,
I developed this module as part of my internship at myStrom which has since finished. You could clone the repo, write it yourself, and create a pull request. You can find the documentation for the motion sensor/PIR on the official myStrom API documentation. Additionally here is the node-RED documentation.
I hope I could be of help and good luck with it. If you run into some issues understanding my code, just send me an email.


Update v1.3.0-alpha.5

Changes to the operation of placeholder {tps}
Changed TPS display, for example: 60.0 -> 60. Just removed tenths.
Updating the auto-updater
Now when updating, the server restarts itself via the rnr command.

I will test it now, if it works I will release v1.3.0-alpha.5

It seems to be working, releasing an update.


✨ Matching tag and bracket pairs

When user types a {, [, (, or <, we should append a matching }, ], ), or > right after the cursor position on the same line.
Related

#78

This one of the core features of this library: https://github.com/ChimeHQ/TextFormation
Might be handy. It is text-system independent, so should be possible to incorporate into any system. But, does have some more straightforward hooks for NSTextView.

Could be implemented using https://github.com/ChimeHQ/TextFormation


types: Describe current state of reaction object in redux.

When messages are fetched from server via GET /messages, then it
contains id as key in user of the reaction object. While reaction
add event has user_id in it.
As we weren't using this attributes anywhere, so there was no problem
and fortunately app never crashed.

Thanks @jainkuniya ! Merged.


REQUEST: any plan to package this for R?

Not an issue at all.
I found the approach interesting, and I was wondering whether there is any plan to package this for R.
Thank you

HI @StfnRomano thank you for your interest.  We don't have plans for an R package - I personally am not fluent with R so I wouldn't be able to maintain it.