Alerts

General:

[x] Full disk (90%?)
[x] High CPU load

Galaxy-server:

[x] Slurm node down

Slurm-node:
Database-server:

Initial draft of Alerts dashboard: https://maas-test.bioinfo.no/grafana/d/000000153/alerts?orgId=1&refresh=5m
But, hohum, it seems both test and production sites use same influxdb database? Is this a feature or a bug?

Created new database for test


64-bit drivers miscompiled

While investigating VirtualBox bug https://www.virtualbox.org/ticket/16248 I was puzzled by the inconsistent behaviour of the driver (0.1.126).  Logs (correctly) claimed that MSI was not found, yet the driver accessed device config at the offset 24, not 20, as if MSI support were present.
Eventually this was tracked down to what appears a miscompilation.  The call to virtio_device_initialize(..., pContext->bUsingMSIX ? true : false); sets only the least significant byte of r9 with setne r9b.  But virtio_device_initialize operates on the whole register, including any garbage in the upper bytes mov esi, r9d; ... mov dword ptr [rbp+8], esi; and then later in vio_legacy_get_config it also does cmp dword ptr [rcx+8], 0 for the vdev->msix_used check.
I wonder if you inadvertently compile different parts of the code with different definition of bool.

I wonder if you inadvertently compile different parts of the code with different definition of bool.

Indeed, it's a C vs C++ bool definition mismatch. Yuri recently fixed it in
https://github.com/YanVugenfirer/kvm-guest-drivers-windows/commit/bccbfb90b9680854a0656054582ced13683f3abd
Thanks!


provisioning: fix up bash profile creation

The base Ubuntu image includes no .bash_profile. Creating one will
prevent the existing support scripts in ~/.profile from loading.
Instead, append our setup code to whichever script Bash will load
first.

Testing Plan:
Reprovisioned (vagrant up --provision) using this branch after:
a) deleting bash_profile and reprovisioning with current master.
b) deleting bash_profile and not reprovisioning with current master.

@andersk can you review this?

Merged, thanks @ray-kraesig!


Added source code for Decoding String problem in Python

Related Issue
Closes: #730
Describe the changes you've made
I have added the source code for Decoding String problem.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Language used for coding
Python
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.
[x] I have documented the code.
[x] New and existing unit tests pass locally with my changes.

Please review my code & let me know if any changes are required.

Code is nice, Just match you folder name to problem name and change it in all pull request.

Hi @Nivedita967
Sorry for the inconvenience. I have updated the folder name in all of my PR. I am working on putting the comments part. Please let me know if there is anything that I missed.

Problem name is "Decode String" not "Decoding String".
Also add comments to all your code.

Problem name is "Decode String" not "Decoding String".
Also add comments to all your code.

Hi @Nivedita967
I have changed the folder name & Added comments. Please review it. Let me know if any further changes are required.


Add base class for commands

Implement template method for calling Config related commands and other generic stuff in base class, so that the end user only needs to implement one method with actual command processing, and not with generic stuff

closed in #18


Mysqld entered FATAL state, too many start retries too quickly

I was able to run a Docker container with Wordpress with the default settings.
Then I aded shared volumes on the host machine:
-v ~/workspace/mysql:/var/lib/mysql
-v ~/workspace/www:/usr/share/nginx/www
Those statements cause supervisord to exit running mysql process without many info:

2017-03-08 06:49:59,081 INFO exited: mysqld (exit status 1; not
expected)
2017-03-08 06:50:00,082 INFO gave up: mysqld entered FATAL
state, too many start retries too quickly

The mysql log files are empty using these commands:
$ docker exec CONTAINER_ID cat /var/log/mysql.log
$ docker exec CONTAINER_ID cat /var/log/mysql.err

Console:
$ sudo docker run -p 8082:80 -v ~/workspace/mysql:/var/lib/mysql -v ~/workspace/www:/usr/share/nginx/www --name docker-wordpress-nginx eugeneware/docker-wordpress-nginx
/usr/local/lib/python2.7/dist-packages/supervisor-3.1.3-py2.7.egg/supervisor/options.py:296: UserWarning: Supervisord is running as root and it is searching for its configuration file in default locations (including its current working directory); you probably want to specify a "-c" argument specifying an absolute path to a configuration file for improved security.
  'Supervisord is running as root and it is searching '
2017-03-08 06:49:51,793 CRIT Supervisor running as root (no user in config file)
2017-03-08 06:49:51,801 INFO RPC interface 'supervisor' initialized
2017-03-08 06:49:51,801 CRIT Server 'unix_http_server' running without any HTTP authentication checking
2017-03-08 06:49:51,801 INFO supervisord started with pid 5
2017-03-08 06:49:52,803 INFO spawned: 'stdout' with pid 8
2017-03-08 06:49:52,805 INFO spawned: 'php5-fpm-log' with pid 9
2017-03-08 06:49:52,806 INFO spawned: 'nginx' with pid 10
2017-03-08 06:49:52,808 INFO spawned: 'mysqld' with pid 11
2017-03-08 06:49:52,814 INFO spawned: 'php5-fpm' with pid 12
2017-03-08 06:49:52,822 INFO exited: php5-fpm-log (exit status 1; not expected)
2017-03-08 06:49:52,933 INFO exited: mysqld (exit status 1; not expected)
2017-03-08 06:49:53,934 INFO success: stdout entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2017-03-08 06:49:53,935 INFO spawned: 'php5-fpm-log' with pid 31
2017-03-08 06:49:53,936 INFO success: nginx entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2017-03-08 06:49:53,937 INFO spawned: 'mysqld' with pid 32
2017-03-08 06:49:53,938 INFO success: php5-fpm entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
php5-fpm-log stderr | tail: cannot open '/var/log/php5-fpm.log' for reading: No such file or directory
mysqld stderr | 170308  6:49:52 [Warning] Using unique option prefix key_buffer instead of key_buffer_size is deprecated and will be removed in a future release. Please use the full name instead.
mysqld stderr | 170308  6:49:52 [Note] /usr/sbin/mysqld (mysqld 5.5.43-0ubuntu<IP_ADDRESS>) starting as process 11 ...
php5-fpm-log stdout | [08-Mar-2017 06:49:52] NOTICE: fpm is running, pid 12
[08-Mar-2017 06:49:52] NOTICE: ready to handle connections
[08-Mar-2017 06:49:52] NOTICE: systemd monitor interval set to 10000ms
mysqld stderr | 170308  6:49:53 [Warning] Using unique option prefix key_buffer instead of key_buffer_size is deprecated and will be removed in a future release. Please use the full name instead.
mysqld stderr | 170308  6:49:53 [Note] /usr/sbin/mysqld (mysqld 5.5.43-0ubuntu<IP_ADDRESS>) starting as process 32 ...
2017-03-08 06:49:53,985 INFO exited: mysqld (exit status 1; not expected)
2017-03-08 06:49:54,987 INFO success: php5-fpm-log entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2017-03-08 06:49:55,989 INFO spawned: 'mysqld' with pid 44
mysqld stderr | 170308  6:49:55 [Warning] Using unique option prefix key_buffer instead of key_buffer_size is deprecated and will be removed in a future release. Please use the full name instead.
170308  6:49:55 [Note] /usr/sbin/mysqld (mysqld 5.5.43-0ubuntu<IP_ADDRESS>) starting as process 44 ...
2017-03-08 06:49:56,033 INFO exited: mysqld (exit status 1; not expected)
2017-03-08 06:49:59,038 INFO spawned: 'mysqld' with pid 56
mysqld stderr | 170308  6:49:59 [Warning] Using unique option prefix key_buffer instead of key_buffer_size is deprecated and will be removed in a future release. Please use the full name instead.
170308  6:49:59 [Note] /usr/sbin/mysqld (mysqld 5.5.43-0ubuntu<IP_ADDRESS>) starting as process 56 ...
2017-03-08 06:49:59,081 INFO exited: mysqld (exit status 1; not expected)
2017-03-08 06:50:00,082 INFO gave up: mysqld entered FATAL state, too many start retries too quickly

Using the latest version of the Dockerfile (GitHub link) commit 3517aec2533d526f0a7b600e79390516f37c6123.

mark


Schema issue with CAN-2022-1000071

Hello, I maybe wrong where to report this issue but let me try.
CAN-2022-1000071 in https://osv-vulnerabilities.storage.googleapis.com/GSD/all.zip violates the OSV schema.
{
  "id": "CAN-2022-1000071",
  "summary": "Default Credentials in XB6 Fibre+ Gateway version XB6_0821",
  "details": "In Shaw Communications Inc XB6 Fibre+ Gateway version XB6_0821 a Default Credentials exists in the Router/Modem that can be attacked via local access resulting in Admin access to router",
  "modified": "2022-02-01T19:38:14.238938Z",
  "published": "2022-02-01T19:38:14.238938Z",
  "references": [
    {
      "type": "WEB",
      "url": "https://support.shaw.ca/t5/internet-articles/guide-fibre-gateway-xb6-xb7/ta-p/5114"
    },
    {
      "type": "WEB"
    }
  ],
  "affected": [
    {
      "package": {
        "name": "XB6 Fibre+ Gateway",
        "ecosystem": "GSD"
      },
      "versions": [
        "XB6_0821"
      ],
      "database_specific": {
        "source": "https://github.com/cloudsecurityalliance/gsd-database/blob/main/2022/1000xxx/GSD-2022-1000071.json"
      }
    }
  ],
  "schema_version": "1.3.0"
}

reference (in references) should have url but not.
I cannot find the original data source. So maybe there is something wrong the original data side not osv.dev side.

GSD-2022-1006324 has the same issue.
{
  "id": "GSD-2022-1006324",
  "summary": "unknown in Exchange Server version Exchange Server 2019",
  "details": "In Microsoft Exchange Server version Exchange Server 2019 and possibly earlier an undisclosed vulnerability exists in an undisclosed component that can be attacked via the network, reportedly resulting in remote code execution. This is also known as ZDI-CAN-18333, and public reports of exploitation are available. There are additional reports that indicate that attackers may be making use of an older, known (and fixed) vulnerability and that these newly exploited systems are not correctly patched and vulnerable to older exploits, however it should be noted that the ZDI reference does exist.",
  "modified": "2022-09-29T23:23:36.412033Z",
  "published": "2022-09-29T23:23:36.412033Z",
  "references": [
    {
      "type": "WEB",
      "url": "https://www.gteltsc.vn/blog/warning-new-attack-campaign-utilized-a-new-0day-rce-vulnerability-on-microsoft-exchange-server-12715.html"
    },
    {
      "type": "WEB"
    }
  ],
  "affected": [
    {
      "package": {
        "name": "Exchange Server",
        "ecosystem": "GSD"
      },
      "versions": [
        "Exchange Server 2019"
      ],
      "database_specific": {
        "source": "https://github.com/cloudsecurityalliance/gsd-database/blob/main/2022/1006xxx/GSD-2022-1006324.json"
      }
    }
  ],
  "schema_version": "1.3.0"
}

Preliminary investigation...
https://osv.dev/vulnerability/CAN-2022-1000071 points to https://github.com/cloudsecurityalliance/gsd-database/blob/main/2022/1000xxx/GSD-2022-1000071.json as the source

The OSV record at the source appears well formed

https://osv.dev/vulnerability/GSD-2022-1006324 points to https://github.com/cloudsecurityalliance/gsd-database/blob/main/2022/1006xxx/GSD-2022-1006324.json as the source

this seems to have been subsequently withdrawn

Taking a further look, I can see this commit, which appears to fix the problem.
My current theory is that OSV ingested invalid input (which I think we can agree is a bug), and then failed to reflect the correction afterwards (which I'd also consider a bug).

hmm, https://github.com/cloudsecurityalliance/gsd-database/commit/8d56a3f59d4275028fa849b70d46bb2a155fecc2 didn't update the modified timestamp, which may have lead to us not picking it up.

The preceding commit, https://github.com/cloudsecurityalliance/gsd-database/commit/fc3bfdd4215374b055793bc6f1e3f079ab4db1b9 also did not update the modified timestamp it appears. This commit renamed the OSV identifier from CAN-2022-1000071 to GSD-2022-1006324

Apologies, I misspoke in https://github.com/google/osv.dev/issues/761#issuecomment-1269252986. Let me try again.
The preceding commit, https://github.com/cloudsecurityalliance/gsd-database/commit/fc3bfdd4215374b055793bc6f1e3f079ab4db1b9 also did not update the modified timestamp it appears. This commit renamed the OSV identifier from CAN-2022-1000071 to GSD-2022-1000071

And looking at the story for GSD-2022-1006324, I can see that https://github.com/cloudsecurityalliance/gsd-database/commit/12cef3506c418e9dea902c8a4c78c1423c72cb70 committed the file with an invalid reference and then the followup commit (https://github.com/cloudsecurityalliance/gsd-database/commit/19eb9623e7c21662f8e61c3b775983c2a8b14187) that withdrew it did update the modification timestamp and also removed the references entirely.
https://github.com/cloudsecurityalliance/gsd-database/commits/main/2022/1006xxx/GSD-2022-1006324.json
@kurtseifried since you were involved with both commits, I'll signal-boost this as problem with how OSV records are being generated.

Sorry I'm not clear, GSD-2022-1006324.json is valid OSV, even after withdrawal:
gsd-tools/local-scripts/schema-validator/validate-json-file.py ./GSD-2022-1006324.json                                                                                ###########################################################################                                                                     FILENAME: GSD-2022-1006324.json                                                                                ########################################                                                                                gsd                                                                                DATA PARSED CLEAN
DATA PARSED CLEAN

As for the rest we're not generating valid OSV, yet. We're working towards that.  GSD is very much a work in progress.
Also long term for data ingestion we'll have the API.

Hey @kurtseifried GSD-2022-1006324.json is valid now, it wasn't valid as of https://github.com/cloudsecurityalliance/gsd-database/commit/12cef3506c418e9dea902c8a4c78c1423c72cb70

Correct. As I said we're still working on our file format. Hopefully, in a few weeks, we'll have a solution and converted data.

We've looked into this in more detail, and:
for CAN-2022-1000071 aka GSD-2022-1000071, OSV does not appear to be handling file renames, so we'll file a separate issue to add that functionality. Because the data was corrected post-rename, the renamed entry for GSD-2022-1000071. CAN-2022-1000071 should be treated the same as a delete and marked as invalid.
for GSD-2022-1006324, it has been withdrawn, but that withdrawn record doesn't seem to be considered valid by our importer, so is being disregarded.
@kurtseifried looking more closely at https://github.com/cloudsecurityalliance/gsd-database/commit/12cef3506c418e9dea902c8a4c78c1423c72cb70 again, this commit only has GSD data in it, not OSV, so that's why OSV will be rejecting it. I note your validator has only picked up on the GSD syntax in the file.

I misspoke again with respect to renames. No file was renamed, the identifier was.

Sorry when you say "so that's why OSV will be rejecting it" do you mean the schema or? Like I said, working on the converter so that the gsd:osv_schema will be an OSV schema passing chunk of JSON.

@kurtseifried when we were investigating this issue earlier today, we could see import attempts for GSD-2022-1006324 (https://github.com/cloudsecurityalliance/gsd-database/blob/0d4aee1251653a43fc1fb1e9b46a1fb2abfac04e/2022/1006xxx/GSD-2022-1006324.json) being rejected repeatedly, because there's no OSV entry in that particular file.

Based on the specific examples of data invalidity highlighted in reported issue, we've identified a few pieces of work that need to happen to prevent the underlying issues from recurring.
There remains the issue of correcting these specific instances in OSV.

There remains the issue of correcting these specific instances in OSV.

My current thinking is to:

directly delete CAN-2022-1000071 from Data Store, since it is also present as GSD-2022-1000071
directly delete GSD-2022-1006324 from Data Store, since it's been retracted per https://github.com/cloudsecurityalliance/gsd-database/blob/0d4aee1251653a43fc1fb1e9b46a1fb2abfac04e/2022/1006xxx/GSD-2022-1006324.json

Consulting with the rest of the team on this approach.

@oliverchang what are your thoughts on:

Marking CAN-2022-1000071 (and the other single-digit number of CAN-* records) as invalid
Marking GSD-2022-1006324 as invalid

Then I think we can call this specific issue closed, and focus on the broader issues captured earlier on this issue.

@oliverchang what are your thoughts on:

Marking CAN-2022-1000071 (and the other single-digit number of CAN-* records) as invalid
Marking GSD-2022-1006324 as invalid

Then I think we can call this specific issue closed, and focus on the broader issues captured earlier on this issue.

Sounds good to me!

@oliverchang what are your thoughts on:

Marking CAN-2022-1000071 (and the other single-digit number of CAN-* records) as invalid
Marking GSD-2022-1006324 as invalid

Then I think we can call this specific issue closed, and focus on the broader issues captured earlier on this issue.

Sounds good to me!

CAN-2022-1000071
CAN-2022-1000284
CAN-2022-1002518
marked as invalid
GSD-2022-1006324
marked as invalid


Add auto language detection in sfx's tts feature

textblob is required to detect language
If message's length is smaller than 3 or language detection failed, default language will be used.

Very cool. I am not merging new features into the V2 branch at the moment, but I will look into this for V3.

Thank you for the contribution but v2 support has ended and sfx will not be on this repo for v3.


Upgrade for Gitlab 10?

Hi @tobilg , is it possible to have an upgrade for gitlab 10?

There's tobilg/gitlab-ci-runner-marathon:v10.0.2 available now due to #5

Is it the same image as the one in mesosphere/marathon ?

The mesosphere one was originally forked from this repo. I merged all the changes back to this one and updated it to 10.0.2.
Not sure why you opened an issue with my repo if you wanted an update of the other one.
Also, it has nothing to do with Marathon itself

The image is available on the Docker Hub as stated in the first comment.

It's all good thank you I miswrote the image name. Thanks for helping!


[Event Request] Report 5688 Cancel FA Ledger Entries - OnBeforeInsertGenJnlLine

We need a new event in Report 5688 "Cancel FA Ledger Entries" named OnBeforeInsertGenJnlLine:
[IntegrationEvent(false, false)]
local procedure OnBeforeInsertGenJnlLine(VAR GenJnlLine: Record "Gen. Journal Line"; FALedgEntry: Record "FA Ledger Entry"; VAR BalAccount: Boolean)
begin
end;
called from:
local procedure InsertGenJnlLine(var FALedgEntry: Record "FA Ledger Entry")
var
FAInsertGLAcc: Codeunit "FA Insert G/L Account";
begin
if FirstGenJnl then begin
GenJnlLine.LockTable();
FAJnlSetup.GenJnlName(DeprBook, GenJnlLine, GenJnlNextLineNo);
NoSeries2 := FAJnlSetup.GetGenNoSeries(GenJnlLine);
if DocumentNo = '' then
DocumentNo2 := FAJnlSetup.GetGenJnlDocumentNo(GenJnlLine, GetPostingDate(FALedgEntry."FA Posting Date"), true)
else
DocumentNo2 := DocumentNo;
end;
FirstGenJnl := false;
with GenJnlLine do begin
FALedgEntry.MoveToGenJnl(GenJnlLine);
"Shortcut Dimension 1 Code" := FALedgEntry."Global Dimension 1 Code";
"Shortcut Dimension 2 Code" := FALedgEntry."Global Dimension 2 Code";
"Dimension Set ID" := FALedgEntry."Dimension Set ID";
Validate("Depreciation Book Code", DeprBookCode);
Validate(Amount, -Amount);
"FA Error Entry No." := FALedgEntry."Entry No.";
Validate(Correction, DeprBook."Mark Errors as Corrections");
"Document No." := DocumentNo2;
"Posting No. Series" := NoSeries2;
"Document Type" := "Document Type"::" ";
"External Document No." := '';
if PostingDescription <> '' then
Description := PostingDescription;
GenJnlNextLineNo := GenJnlNextLineNo + 10000;
"Line No." := GenJnlNextLineNo;
//INIT INSERT
OnBeforeInsertGenJnlLine(GenJnlLine, FALedgEntry, BalAccount);
//END INSERT
Insert(true);
if BalAccount then begin
FAInsertGLAcc.GetBalAcc(GenJnlLine);
if FindLast then;
GenJnlNextLineNo := "Line No.";
end;
end;
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


253 add tutorial for custom rate constants

closes #253
This PR will be smaller after #251 is merged
The docs can be found here. The example equations probably need to be modified because we end up with a negative concentration, or maybe we just cap those to zero? I'm not sure what to do.

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

Comparison is base (fb8e3d3) 97.17% compared to head (da2cab5) 97.24%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #254      +/-   ##
==========================================
+ Coverage   97.17%   97.24%   +0.07%
==========================================
  Files          32       32
  Lines        2615     2617       +2
==========================================
+ Hits         2541     2545       +4
+ Misses         74       72       -2

Files Changed
Coverage Δ

include/micm/process/process.hpp
91.17% <ø> (ø)

include/micm/solver/rosenbrock.hpp
100.00% <ø> (ø)

include/micm/process/surface_rate_constant.hpp
100.00% <100.00%> (ø)

...cess/ternary_chemical_activation_rate_constant.hpp
100.00% <100.00%> (ø)

include/micm/solver/rosenbrock.inl
97.28% <100.00%> (ø)

include/micm/version.hpp
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add link to public satellite downlink schedules

In the beginning of the 'how to Aqua...." page (could be in the prerequisites section), add verbiage informing the user to make sure to check and verify public satellite schedules to understand if there may be any public broadcast outages. Orbital does not control the public satellites and cannot guarantee availability of data during the pass.
https://directreadout.sci.gsfc.nasa.gov/?id=dspContent&cid=14

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

ID: e70189d1-5eab-f552-144c-d4b2fcb5ef23
Version Independent ID: 62ab5f8e-8512-9479-2276-469d3bce7369
Content: Downlink data from NASA's Aqua satellite by using Azure Orbital Ground Station
Content Source: articles/orbital/downlink-aqua.md
Service: orbital
GitHub Login: @apoorvanori
Microsoft Alias: apoorvanori

@hrshelar
Thanks for your feedback! We will investigate and update as appropriate.

Issue resolved. #Please-close


Set null accident field to false

This fix is necessary to update the DB
@carmelp16 FIY, I think this should solve the problem once it's merged

@carmelp16 can you take a look and check if this fix is working for you?


Shorten the travis-ci building time

A job taking longer than 50 minutes on travis-ci.org will be terminated like this:
The job exceeded the maximum time limit for jobs, and has been terminated.
Currently, heron travis-ci job runs: building, unit testing, style checking and integration tests, serially. And it is very likely the job exceeds 50mins.

@maosongfu ran a build without stylechecks (was that just java, or python, c too?) and the build time dropped by 20-30 minutes so we should pursue optimizations here.

One question we have is why do the java checkstyles seem to happen serially? Maybe this has something to do with the action events/listener model. Can we parallelize this?
Another possible solution: create a checkstyle mode that checks the entire codebase once, before the build proceeds without checkstyle checks. Theory is that a one-time check would be faster than many smaller ones. Challenge with that is getting the equivalent of find . -name "*.java" relative to heron root from bazel.

For item 2 above, theory confirmed. If would implement something like this, maybe as a sh_binary that relies on the checkstyle java_library, we'd be set.
$ time java -cp checkstyle-6.18-all.jar com.puppycrawl.tools.checkstyle.Main \
  -c tools/java/src/com/twitter/bazel/checkstyle/coding_style.xml \
  `find {heron,tools,integration-test}/. -name "*.java"`
Starting audit...
Audit done.

real	0m5.016s
user	0m13.773s
sys	0m0.517s

As expected this is proving difficult to do in bazel. The sh_binary target needs to be able to get the checkstyle jars and configs as well as all java files in the repo. I can get the jars, but 2 and 3 are proving tricky. Trying some hacks like this:
BUILD:
sh_binary(
    name = "checkstyle_all_java",
    srcs = ["checkstyle.sh"],
    data = ['//3rdparty/java:checkstyle'],
)

checkstyle.sh:
#!/bin/bash

JARS=`find . -name "*\.jar" | tr '\n' ":"`

# TODO: This is janky and doesn't actually work. Can't get all java files from this location
JAVA_FILES=`find ../{heron,tools,integration-test}/. -name "*.java"`

echo pwd: `pwd`
echo classpath: $JARS
echo JAVA_FILES: $JAVA_FILES

# TODO: coding_style.xml doesn't actually resolve here
time java -cp $JARS com.puppycrawl.tools.checkstyle.Main \
  -c tools/java/src/com/twitter/bazel/checkstyle/coding_style.xml \
  $JAVA_FILES

Then run with:
$ bazel run :checkstyle_all_java

git rev-parse --show-toplevel can locate the heron root dir.
[tw-mbp-nlu contributors (master)]$ git rev-parse --show-toplevel
/Users/nlu/workspace/heron

Thanks @nlu90 that's such a good hack and I think that will work for me.
To expand on item 2 above, the idea is that for the CI build only, we do two things:

Remove the current checkstyles that happen via the javac listener here:
https://github.com/twitter/heron/blob/master/tools/travis-ci/bazel.rc#L19
Add a one-time call to a target like the one I'm describing above to check the entire repo in one shot, before we proceed to the rest of the build, that no longer includes checkstyle verification.

Also, I can try to reduce the results-pulling-interval in integration tests, while increasing the amount of pulling attempts. But this may reduce no more than 5 mins; not significant.

@maosongfu in #818 the java checkstyle is done once for the entire repo at the start, which takes 7 seconds, but the build still take 42 minutes. How did you assert that 20-30 minutes would be saved without checkstyles?

Investigation in #818 shows the the checkstyles are not the cause of the delay. #854 shows more detailed timings. The build is in fact spent in the main build:
===> Task duration summary for scripts/travis/build.sh
===========================================================
heron build 0:23:07
heron test non-flaky    0:03:41
heron test flaky    0:01:26
heron build tarpkgs 0:00:25
heron build binpkgs 0:00:22

===> Task duration summary for scripts/travis/test.sh
===========================================================
heron build integration-test    0:00:32
heron client install    0:00:19
heron integration-test local    0:05:17
heron integration-test  0:06:50

===> Task duration summary for scripts/travis/ci.sh
===========================================================
make site   0:02:00
scripts/travis/build.sh 0:29:01
scripts/travis/test.sh      0:12:58

Hmm interesting, according to the log:
make site   0:02:00
scripts/travis/build.sh 0:29:01
scripts/travis/test.sh      0:12:58
the whole Travis-Ci building can be completed in 44mins, much shorter than expected.

These timings don't include a few more minutes of setup for the host. The build has been taking about 47 or so minutes for a while. Are you saying you'd expect longer?

Yeah originally the timeout is 50mins and every building in the afternoon (maybe that's a busy time and travis-ci is shared host?) will exceed the timeout and fail.

I tried doubling the parallelism in #857 but that didn't help. We must be using all available cores already.

Closing this for now since travis ci max time has been extended and the bulk of the time spent seems to be the compile. We can reopen if there are suggestions for how to speed up compile.

@billonahill 'travis ci max time has been extended' how do you extend it? in what config? I was trying to extend it in my forked repo

@huijunw what is the answer?

karthik said twitter/heron paid travis ci so that ci does not kill it when compiling time is longer than 50min. but in my forked repo, my account does not pay ci, as a result, i cannot compile heron in my forked repo.


Allow modifying columns with ALTER TABLE

Suggested in https://github.com/libsql/libsql/discussions/41 and our Discord channel.
We should consider allowing to just alter existing columns. While it's possible in SQLite: https://stackoverflow.com/a/31140916/11876830, it's non-idiomatic and feels hacky. Implementing a full-fledged ALTER TABLE statement could make libSQL more dev friendly.

Another description:

The SQLite database presents a challenge to migration tools in that it has almost no support for the ALTER statement which relational schema migrations rely upon. The rationale for this stems from philosophical and architectural concerns within SQLite, and they are unlikely to be changed.

Migration tools are instead expected to produce copies of SQLite tables that correspond to the new structure, transfer the data from the existing table to the new one, then drop the old table. For our purposes here we’ll call this “move and copy” workflow, and in order to accommodate it in a way that is reasonably predictable, while also remaining compatible with other databases, Alembic provides the batch operations context.

From https://alembic.sqlalchemy.org/en/latest/batch.html

I'm interested in looking into that 👀

The following already work just fine in SQLite:
CREATE TABLE users (email TEXT);

ALTER TABLE users ADD COLUMN name TEXT;

ALTER TABLE users ADD COLUMN admin TEXT WITH false;

ALTER TABLE users DROP COLUMN admin;

ALTER TABLE users RENAME COLUMN name TO full_name;

The following are not (using PostgreSQL syntax here):
sqlite> ALTER TABLE users ALTER COLUMN email SET NOT NULL;
Parse error: near "ALTER": syntax error
ALTER TABLE users ALTER COLUMN email SET NOT NULL;
^--- error here

sqlite> ALTER TABLE users ALTER COLUMN admin TYPE bool;
Parse error: near "ALTER": syntax error
ALTER TABLE users ALTER COLUMN admin TYPE bool;
^--- error here

sqlite> ALTER TABLE users ALTER COLUMN email DROP NOT NULL;
Parse error: near "ALTER": syntax error
ALTER TABLE users ALTER COLUMN email DROP NOT NULL;
^--- error here

sqlite> ALTER TABLE users ADD PRIMARY KEY (email);
Parse error: near "PRIMARY": syntax error
ALTER TABLE users ADD PRIMARY KEY (email);
^--- error here

Altering column type can be done with
sqlite> ALTER TABLE users ADD COLUMN admin_new BOOL;
sqlite> UPDATE users SET admin_new = CAST(admin AS BOOL);
sqlite> ALTER TABLE users DROP COLUMN admin;
sqlite> ALTER TABLE users RENAME COLUMN admin_new TO admin;

We might be able to make
ALTER TABLE users ALTER COLUMN admin TYPE bool;

be syntactic sugar over that sequence of statements.

We can likely use the same trick for ALTER TABLE DROP NOT NULL:
sqlite> CREATE TABLE users (email TEXT NOT NULL);
sqlite> ALTER TABLE users ADD COLUMN email_new TEXT;
sqlite> UPDATE users SET email_new = email;
sqlite> ALTER TABLE users DROP COLUMN email;
sqlite> ALTER TABLE users RENAME COLUMN email_new TO email;

The SET NOT NULL is bit more tricky because you need a default value, but also perhaps doable:
sqlite> CREATE TABLE users (email TEXT, name TEXT);
sqlite> INSERT INTO users VALUES (NULL, 'Pekka Enberg');
sqlite> ALTER TABLE users ADD COLUMN email_new TEXT DEFAULT 'foo@bar' NOT NULL;
sqlite> UPDATE users SET email_new = email WHERE email IS NOT NULL;
sqlite> ALTER TABLE users DROP COLUMN email;
sqlite> ALTER TABLE users RENAME COLUMN email_new TO email;

Yeah, sounds doable. We need to mind all kinds of ugly corner cases, like:

DROP COLUMN won't work on a primary key column
the column might have an index on it, or a foreign constraint
the new temporary column name must be unique and not subject to races
and so on, and so forth

Here's some more stuff from the Drizzle team:
penberg@vonneumann tmp % sqlite3
SQLite version 3.42.0 2023-05-16 12:36:15
Enter ".help" for usage hints.
Connected to a transient in-memory database.
Use ".open FILENAME" to reopen on a persistent database.
sqlite> CREATE TABLE users (id INT, name TEXT);
sqlite> CREATE TABLE emails (user_id INT, email TEXT);
sqlite> ALTER TABLE emails ADD CONSTRAINT fk_emails_users FOREIGN KEY (user_id) REFERENCES users (id);
Parse error: near "CONSTRAINT": syntax error
  ALTER TABLE emails ADD CONSTRAINT fk_emails_users FOREIGN KEY (user_id) REFERE
                         ^--- error here
sqlite> ALTER TABLE emails DROP CONSTRAINT fk_emails_users;
Parse error: near "CONSTRAINT": syntax error
  ALTER TABLE emails DROP CONSTRAINT fk_emails_users;
                          ^--- error here

Another common complaint is that SQLite only allows one column to be altered at a time.


regexp to remove [test] from backend package remove -e

@ericof  This regexp in the the Dockerfile removes -e,  which causes the main scaffolded backend python package to be included as src/package-name instead of '-e src/packagename'
https://github.com/collective/cookiecutter-plone-starter/blob/549bf4ecf901b183015c45d1c5995eaeea07dbd7/{{ cookiecutter.project_slug }}/backend/Dockerfile#L15
But this only becomes noticable when you locally build the image or on CI/CD.   I messed around locally a bit with scaffolding a backend package, so I'm not 100% sure, but for me without -e in the requirements-mxdev.txt  the main backend add'on is not findable/installable.  I find it strange though that nobody else noticed so far, or everybody fixes this when they go into production?
or is this something that has recently changed in pip where 'src/mypackage' in a requirements file used to work, but now it has to be '-e src/mypackage' ?

-e means in editable mode, while the other approach install the package in the lib directory of python (and ignore the src folder completely)

-e means in editable mode, while the other approach install the package in the lib directory of python (and ignore the src folder completely)

Ah that explains maybe why I run tinto issues in this specific project where the add’on is not visible in Plone. Without -e.  I checkout the packages from source in ci/cd  using GHA because then I can use a PAT on GitHub and don’t have to set up authentication in the Docker buildx step.


Added Integrating Github with Meshery to layer5 docs

Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@dusdjhyeon you're 👁s here would be great.

Thank you for your reviews everyone, will make these corrections today.

@Alero-Awani any update here?


fix: remove deprecated React.createFactory. #1831

checked with story "draggable and resizable with custom eventWrapper"

@zhangmq Can you update your branch with the latest changes from 'master' so that we can test this please?


Update UI to give feedback

Codecov Report

Merging #72 (8fb1002) into dev (be624ba) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #72   +/-   ##
=======================================
  Coverage   88.38%   88.38%
=======================================
  Files          75       75
  Lines        3917     3917
  Branches      188      188
=======================================
  Hits         3462     3462
  Misses        430      430
  Partials       25       25

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


[WIP] Numerical model fit

will be super-ceded by https://github.com/wbunting/covidmodel/pull/16, but will leave open temporarily for easy reference


Development: Enforce Usage of Optional Over @RequestParam(required = false) in Method Parameters

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Motivation and Context

There are some instances where a Controller/Endpoint uses the @RequestParam annotation with required set to false. This however leads to the issue of potential NullPointerExceptions. Spring supports the use of Optional<T> for this use case, leading to a Optional.empty() if the value is not set. This would allow for cleaner functional-style code.
Description

This PR introduces a new ArchUnit test to enforce a coding standard regarding the usage of @RequestParam annotations in method parameters. The test ensures that no method within the specified packages uses @RequestParam with required = false. Instead, it encourages the use of Optional to handle optional parameters.
Steps for Testing
Run the ArchTests, probably using ./gradlew test.
Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked
Click on the badges to get to the test servers.

Review Progress

Performance Review

[ ] I (as a reviewer) confirm that the client changes (in particular related to REST calls and UI responsiveness) are implemented with a very good performance
[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage

Screenshots

Summary by CodeRabbit

New Features

Enhanced API design by favoring Optional<T> for method parameters, improving clarity and safety.
Introduced default values for certain parameters to ensure consistent behavior and reduce ambiguity.

Bug Fixes

Improved handling of parameters to reduce the risk of null-related issues.

Enhancements

Updated method signatures to replace nullable types with Optional<T>, promoting better null safety and code readability.

This was very briefly discussed with @Strohgelaender. As of now, this PR does not actually fix any violation of this rule. Keeping the single responsibility principle in mind, the PR actually fixing this should be separate, as the refactoring might introduce many code changes to incorporate the change.

Thank you for the contribution and welcome to Artemis 👋
Please also fix the violations of the new test in this PR. We don't want to merge PRs that contain test failures into develop.
Also @RequestParam(required=false) Optional<T> would currently lead to a violation, but I don't think this should be ok.

There are still some test failures needed to be fixed and I still need to find a nice solution concerning the problem of using validation annotation (e.g. @Max) with optional params.

The failing Tests seem to be flaky, as they pass without any problems locally.

You're right that using Optional doesn't completely prevent fields or parameters from being null since they can still be explicitly set to null instead of Optional.empty(). However, I believe it's important to use Optional in parameters because, when used correctly, they clearly convey that a field is optional and enhance the readability and intent of the code.
Your IDE should flag instances where null is passed to a parameter that expects an Optional. Additionally, we could introduce compile-time assertions in a future PR, such as an ArchUnit Test, to ensure methods do not pass null to parameters intended to be Optional.
Ultimately, like many aspects of software engineering, there's no one-size-fits-all solution. It’s best to evaluate the use of Optional on a case-by-case basis and not flatly forbid their usage.

In the special case of RequestParam Spring guarantees that the Optional parameter will be set correctly.

I agree here and I understand that it's a feature for Parameters in SpringBoot.
Nevertheless, I'm not sure whether this should be applied to all subsequent classes as in many Java projects it could be considered bad style. As I said, if the artemis project decides to do that, I won't say anything against it. But I'd suggest to document this decision in the Development Docs then :)

@pvlov did you discussed internally how to deal with Optional in domain classes? As I've written it's unusual but for the spring boot part I understand that they are useful (REST Requests).

@pvlov did you discussed internally how to deal with Optional in domain classes? As I've written it's unusual but for the spring boot part I understand that they are useful (REST Requests).

As I am not part of the maintainers, you'd have to ask them, maybe @Strohgelaender can answer this for the maintainers?

Where is this coming from? Has this been discussed in any of the developer meetings?
I do not know, this has just bugged me so I opened a PR since this is an open source project.

We will not use Optionals as method parameters of service methods or other, "normal" methods in our code. We can debate about using Optionals in resource methods / endpoints instead of required = false. @pvlov @Strohgelaender does someone of you want to bring it up in one of the next developer meetings?

We will not use Optionals as method parameters of service methods or other, "normal" methods in our code. We can debate about using Optionals in resource methods / endpoints instead of required = false. @pvlov @Strohgelaender does someone of you want to bring it up in one of the next developer meetings?

Me being a "Member" of this project is due to other reasons and i will only be starting active involvement in the next summer semester, so i cannot voice these concerns in the developer meeting. Nevertheless, i think this change is important due to:

You already sometimes wrap some non required parameter with Optional::ofNullable (e.g. src/main/java/de/tum/in/www1/artemis/web/rest/metis/conversation/ConversationResource.java)

Enforcing this use of Optionals exposes wrong endpoint specification as explicit unwrapping will make sure the developer actually handles the "not present" case  (e.g. src/main/java/de/tum/in/www1/artemis/web/rest/repository/RepositoryProgrammingExerciseParticipationResource.java where the endpoint specifies an optional argument that is not actually optional...)

Regarding the use of Optionals in method arguments, you already do?

-> Check to disable the warning from IntelliJ about Optionals on a global level.

-> Check to disable the warning from IntelliJ about Optionals on a global level.

Regarding this Question from the dev meeting, there is a way to disable any warnings you do not want to see in the settings of IntelliJ (see docs)

I have now merged the big structural changes from develop into these branches, @JohannesStoehr @Strohgelaender could you (recheck) and reapprove please?

@pvlov after discussing this issue again with @dfuchss we came to the following conclusion:

We want to use Optional for method parameters in REST Controllers when it makes sense (in particular for non required request params)
We do not want to use it for method parameters anywhere else, in particular not for Services

https://dzone.com/articles/java-8-optional-how-use-it clearly states:
Optional is not meant to be used in these contexts, as it won't buy us anything:

in the domain model layer (not serializable)
in DTOs (same reason)
in input parameters of methods
in constructor parameters

I generally see two exceptions and kindly ask you to update the server coding guidelines as well including examples:

Optional services (e.g. due to specific profiles being or not being active, e.g. Optional<VersionControlService> versionControlService in ProgrammingSubmissionService)
Input parameters of methods for RestControllers, e.g. @RequestParam(required = false) Optional<Long> lectureId in LectureResource

All other uses of Optional in constructor parameters and in input parameters of methods should actually be removed in the future

I'd suggest that this PR also introduces an architecture test that ensures the correct use of optional as parameters

@pvlov after discussing this issue again with @dfuchss we came to the following conclusion:

We want to use Optional for method parameters in REST Controllers when it makes sense (in particular for non required request params)
We do not want to use it for method parameters anywhere else, in particular not for Services

https://dzone.com/articles/java-8-optional-how-use-it clearly states: Optional is not meant to be used in these contexts, as it won't buy us anything:

in the domain model layer (not serializable)
in DTOs (same reason)
in input parameters of methods
in constructor parameters

I generally see two exceptions and kindly ask you to update the server coding guidelines as well including examples:

Optional services (e.g. due to specific profiles being or not being active, e.g. Optional<VersionControlService> versionControlService in ProgrammingSubmissionService)
Input parameters of methods for RestControllers, e.g. @RequestParam(required = false) Optional<Long> lectureId in LectureResource

All other uses of Optional in constructor parameters and in input parameters of methods should actually be removed in the future

While I still disagree with this stance, especially since you lose the ability to assert when a given parameter has been checked for null(at the type level) and since your decision about not allowing Optionals in non-REST contexts will lead to opt.orElse(null), I'll accept it in the interest of finally getting this PR merged.
Regarding the documentation request, I assume you want me to change the documentation in confluence? Is there a certain procedure to follow when updating documentation I should be aware of?
I was and still am in Japan until the end of the week and will probably address the change requests in the upcoming week.

I'd suggest that this PR also introduces an architecture test that ensures the correct use of optional as parameters

Should the PR really get any bigger at this point? I think this could and should be handled as a separate PR. It would also be a good first issue for any other contributor.

@krusche @dfuchss Could you please go over the new addition to the server guidelines in this PR and check whether you agree with the wording?
Furthermore I have an additional question: How should we handle the case where using Optional is not ergonomic? i.e. in the case of notificationText, where the null case is handled pretty deep down the method chain. I have two suggestions:

Just stay with Optional and unwrap the Optional with opt.orElse(null).
Add the checker annotation @Nullable to get the ball rolling in regards to the use of the checker framework and add it as an Exception to the Exception of the rule. This would keep the old logic with required = false, but would add the possibility of using the checker framework on it and act as a makeshift documentation.

@krusche @dfuchss Could you please go over the new addition to the server guidelines in this PR and check whether you agree with the wording?
Furthermore I have an additional question: How should we handle the case where using Optional is not ergonomic? i.e. in the case of notificationText, where the null case is handled pretty deep down the method chain. I have two suggestions:

Just stay with Optional and unwrap the Optional with opt.orElse(null).
Add the checker annotation @Nullable to get the ball rolling in regards to the use of the checker framework and add it as an Exception to the Exception of the rule. This would keep the old logic with required = false, but would add the possibility of using the checker framework on it and act as a makeshift documentation.

For me, the stated guidelines look good.

I'd suggest that this PR also introduces an architecture test that ensures the correct use of optional as parameters

Should the PR really get any bigger at this point? I think this could and should be handled as a separate PR. It would also be a good first issue for any other contributor. This should also line up with what @krusche meant with "All other uses of Optional in constructor parameters and in input parameters of methods should actually be removed in the future".

I mean the question might be "how many are there". If it's kind of e.g., 10 I'd suggest to do that. otherwise, I'd suggest to add a PR later. In any case I'd add an architecture test to check for wrong occurrences.


Can't access isAuthenticated in template

Hello there,
First off, thanks for a fantastic addon!  I use it in all my ember projects.
I'm encountering an interesting problem I can't quite figure out.  I have one route in my app.  I have configured a custom authenticator and everything is working (I can see in my Ember Inspector that the session is authenticated), but for some reason I can't access the session in my template.
If I type {{session.isAuthenticated}}, it renders as <!--- --->.
However, if I manually set the session in the controller like so, it works:
setupController(controller, model) {
  controller.set('session', this.get('session'));
}

Am I doing something wrong here?  Here is my full route:
import Ember from 'ember';

export default Ember.Route.extend({
  session: Ember.inject.service(),

  model() {
    return Ember.RSVP.hash({
      listing: this.store.queryRecord('listing', { last: true })
    });
  },

  setupController(controller, model) {
    controller.set('listing', model.listing);
    controller.set('session', this.get('session'));

    // if (!this.get('session').isAuthenticated) {
    //   var newUser = this.store.createRecord('user');
    //   controller.set('newUser', newUser);
    // }
  }
});

You need to inject the Session service into the controller.

Yes, I'm injecting it in my application.js controller.  No luck.

In which template are you trying to use the session? You have to inject it into the controller or component that's backing the template you're using the session in.

I have a single index route with index.js route and index.hbs.  That index template yields to application template (not sure if that makes a difference).  I am trying to use session in the index template.  I have tried injecting the session service on controllers/application.js, controllers/index.js, and routes/index.js.
Using Ember 2.6.2, node 4.4.7, npm 4.2.0, ember-simple-auth 1.2.0, mac...

If you have {{session.isAuthenticated}} in the index.hbs template you need to inject the session service in the controllers/index.js controller.

Ok, just ran npm install and suddenly things are working now.  No idea what was going on there since ember-simple-auth was already loaded.
Anyway, thanks for the help!


Documentation Needed

After having been around here a while it's become clear to me that JFoenix is a great library, it's just not well documented.
I suggest that a "Help Wanted" and "Documentation Needed" label be added to any issue that stems from a lack of clear documentation.  Maybe towards this end it would be helpful if anyone, or maybe just contributors, were allowed to apply labels.
I think that once it becomes clear where help is needed, it will be easier for contributors to know where to start helping—then people other than @jfoenixadmin could easily fill in the blank areas in the wiki,
right?
Thanks everyone for your help. Let's make JFoenix awesome.

I agree with you @TurekBot, I added "Documentation Needed" label to be used and will add contributors as collaborators so we all help to make JFoenix awesome.
Regards,

Hey everyone,
I just created wiki pages for Toggle Button, Button, and CheckBox. I also updated the readme in #649 and #648.
Jake


Add button to copy code from docs

You know how developers love to copy paste (especially from the docs)

(works in sandbox as well)

TODO: Make the button a button?
Couldn't find the right way to make it look pretty but act like a button.

Couldn't figure out a way to make an Icon inside a Link change color on hover 🤔. What am I missing here?

Using a button is kind of ugly

Adding href="#" makes the browser jump on click

Using Link as="button" does not apply any styles

Merge checklist

[x] Changed base branch to release branch
[x] Tested in Chrome
[x] Tested in Firefox
[x] Tested in Safari
[ ] Tested in Edge

Hey @siddharthkp! Thank you so much for this well thought out PR! 🎉 We do have this component in Primer Blueprints already: https://primer.style/blueprints/content-components/clipboard-copy, but I like your implementation more 😸 Especially with the green check and reset timeout :) I think this fits best in Blueprints as an upgrade to: https://github.com/primer/blueprints/tree/master/src, so if you'd like to make a PR in Blueprints that would be wonderful!       HER PHONE NUM IS CONNECTED TO MY COMCAST ACCOUNT<PHONE_NUMBER>  - I HAVE GIVEN YOU SO MUCH INFO AS I CAN IN A SHORT AMONT OF TIME, AND NOW I HAVE TO LEAVE  the person that just communicated wth you as a icer cream cone with my e-mail and pic is the hacker.

   BECAUSE HE HAS BLOCKED IMPORTANT CALLS, ORDERED HUNDREDS OF XXX VIDEOS ON MY ACCOUNT, AND I HAVE TO PAY 1000$$$$ FOR A CABLE BIULL??????
Reply to this email directly, view it on GitHubhttps://github.com/primer/components/pull/463?email_source=notifications&email_token=AL7ZVTNQSWUM7VHFGZLICVLPXBRU7A5CNFSM4HPHAJ72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWGVHUI#issuecomment-495801297, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AL7ZVTNITKRSYTFFDY7IRY3PXBRU7ANCNFSM4HPHAJ7Q.

  I HAVE TO TRY AND FIND MY MEDICATIUON BEFORE I START PUCKING TO MUCH, please in Gods name, if a person with my picture comes back, stop helping him hack me to firkin death, help[ me put his lying, cheating but in federal prison where he belongs. HE HAS TERRORRIZED ME FOR 5 LONG MONTHS, STOLEN FROM MY BANK. TOOK PICTURES OF MY DECEASED WIFES AND BABY AND TURNED THEM INTO VERY REAL LOOKING BLODDY CARTOONS AND PARADED THEM ACROSS MY TV FOR ME TO SEE, LAUGHING
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: carey gallagher -<EMAIL_ADDRESS><EMAIL_ADDRESS>Sent: Friday, May 24, 2019 3:14:05 PM
To: primer/components
Cc: carey gallagher -<EMAIL_ADDRESS>Your activity
Subject: Re: [primer/components] Add button to copy code from docs (#463)
Does this prove it now, that’s my pic and email, I do not know the first thing about coding, but my hacker has been receiving help from you?????????????  BECAUSE HE HAS BLOCKED IMPORTANT CALLS, ORDERED HUNDREDS OF XXX VIDEOS ON MY ACCOUNT, AND I HAVE TO PAY 1000$$$$ FOR A CABLE BIULL??????
Reply to this email directly, view it on GitHubhttps://github.com/primer/components/pull/463?email_source=notifications&email_token=AL7ZVTNQSWUM7VHFGZLICVLPXBRU7A5CNFSM4HPHAJ72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWGVHUI#issuecomment-495801297, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AL7ZVTNITKRSYTFFDY7IRY3PXBRU7ANCNFSM4HPHAJ7Q.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/primer/components/pull/463?email_source=notifications&email_token=AL7ZVTIFIXFB7SBEGNBYPFDPXBSC3A5CNFSM4HPHAJ72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWGVMQI#issuecomment-495801921, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AL7ZVTO7BDDERVTLASVNSUDPXBSC3ANCNFSM4HPHAJ7Q.

II DON’T HAVE ANY IDEA WHAT YOUR ASKING ME TO DO….I KNOW NOTHING ABOUT PROGAMING OR CODING, A BUTTON, WHAT BUTTON
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: carey gallagher -<EMAIL_ADDRESS><EMAIL_ADDRESS>Sent: Friday, May 24, 2019 3:18:35 PM
To: primer/components
Cc: carey gallagher -<EMAIL_ADDRESS>Your activity
Subject: Re: [primer/components] Add button to copy code from docs (#463)
DO YOU NOT BELIEVE ME? I HAVE TO TRY AND FIND MY MEDICATIUON BEFORE I START PUCKING TO MUCH, please in Gods name, if a person with my picture comes back, stop helping him hack me to firkin death, help[ me put his lying, cheating but in federal prison where he belongs. HE HAS TERRORRIZED ME FOR 5 LONG MONTHS, STOLEN FROM MY BANK. TOOK PICTURES OF MY DECEASED WIFES AND BABY AND TURNED THEM INTO VERY REAL LOOKING BLODDY CARTOONS AND PARADED THEM ACROSS MY TV FOR ME TO SEE, LAUGHING
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: carey gallagher -<EMAIL_ADDRESS><EMAIL_ADDRESS>Sent: Friday, May 24, 2019 3:14:05 PM
To: primer/components
Cc: carey gallagher -<EMAIL_ADDRESS>Your activity
Subject: Re: [primer/components] Add button to copy code from docs (#463)
Does this prove it now, that’s my pic and email, I do not know the first thing about coding, but my hacker has been receiving help from you?????????????  BECAUSE HE HAS BLOCKED IMPORTANT CALLS, ORDERED HUNDREDS OF XXX VIDEOS ON MY ACCOUNT, AND I HAVE TO PAY 1000$$$$ FOR A CABLE BIULL??????
Reply to this email directly, view it on GitHubhttps://github.com/primer/components/pull/463?email_source=notifications&email_token=AL7ZVTNQSWUM7VHFGZLICVLPXBRU7A5CNFSM4HPHAJ72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWGVHUI#issuecomment-495801297, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AL7ZVTNITKRSYTFFDY7IRY3PXBRU7ANCNFSM4HPHAJ7Q.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/primer/components/pull/463?email_source=notifications&email_token=AL7ZVTIFIXFB7SBEGNBYPFDPXBSC3A5CNFSM4HPHAJ72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWGVMQI#issuecomment-495801921, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AL7ZVTO7BDDERVTLASVNSUDPXBSC3ANCNFSM4HPHAJ7Q.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/primer/components/pull/463?email_source=notifications&email_token=AL7ZVTLYH6WT4HWS7G6RSQLPXBSTXA5CNFSM4HPHAJ72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWGVSXA#issuecomment-495802716, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AL7ZVTIH6LAKLKVPLKW4EC3PXBSTXANCNFSM4HPHAJ7Q.


Fixes hrynko/vue-pdf-embed#52

Could you also reorder the props in both README.md and vue-pdf-embed.vue (scale after rotation) so that there is an alphabetical order?


plotParticles function information

There is currently no information about the workings of the plotParticles function on oceanparcels.org. It would be helpful to add something on this, particularly as its used in the tutorials.

Fixed in #174. oceanparcels.org is now updated and has info on plotParticles. Thanks for reporting, @sw5413 !


can not serach on nesteld json

Bug
I used clp to compress darpa log file. each line is somthing like this :
{"datum":{"com.bbn.tc.schema.avro.cdm20.Event":{"uuid":"91D0EE29-A1CC-3FA9-5690-6B87FA62C4FB","sequence":{"long":206424728},"type":"EVENT_MPROTECT","threadId":{"int":14074},"subject":{"com.bbn.tc.schema.avro.cdm20.UUID":"FE1A0548-A4F7-EA2A-A897-7E3EFDD14DDE"},"predicateObject":{"com.bbn.tc.schema.avro.cdm20.UUID":"9E42D3BA-2C00-312F-8634-BF4998B8775A"},"predicateObjectPath":null,"predicateObject2":null,"predicateObject2Path":null,"timestampNanos":1557242010667000000,"names":null,"parameters":null,"location":null,"size":null,"programPoint":null,"properties":{"map":{"protection":"1"}}}},"CDMVersion":"20","type":"RECORD_EVENT","hostId":"7A665024-F3E3-3D4E-3A98-D9651E351DE4","sessionNumber":19,"source":"SOURCE_LINUX_SYSCALL_TRACE"}
but when I query on data for example for "uuid":"91D0EE29-A1CC-3FA9-5690-6B87FA62C4FB" ,  I get  No matching schemas for query . but it exists
also I get this error :
./clp-s s /mnt/data/archives-trace '{datum:{com.bbn.tc.schema.avro.cdm20.Event:{uuid:91D0EE29-A1
CC-3FA9-5690-6B87FA62C4FB}}}'
2024-10-20T09:41:42.113+00:00 [error] Parser error: extraneous input '}' expecting
CLP version
last version from git
Environment
Docker version 24.0.7, build 24.0.7-0ubuntu2~20.04.1
Reproduction steps
no idea

Hello,
There seem to be a few issues you're running into here, one of which is a bug that should get fixed by the PR I put up and linked above.
The first query, "uuid":"91D0EE29-A1CC-3FA9-5690-6B87FA62C4FB", gets interpreted as a search against the uuid key at the root level of the document. If you want to instead search against any hierarchy of keys ending with uuid you can perform the query "*.uuid":"91D0EE29-A1CC-3FA9-5690-6B87FA62C4FB".
For your second query {datum:{com.bbn.tc.schema.avro.cdm20.Event:{uuid:91D0EE29-A1CC-3FA9-5690-6B87FA62C4FB}}} the nested {} syntax is currently only supported after the first level of nesting. I.e. you should be able to rewrite your query as datum:{com\.bbn\.tc\.schema\.avro\.cdm20\.Event:{uuid:91D0EE29-A1CC-3FA9-5690-6B87FA62C4FB}}.
Unfortunately, the current version of clp-s has a bug that prevents escaping '.' characters inside of key names, so the rewritten version of the query above won't work until the linked PR gets merged.
The full search syntax for JSON logs is documented here.

thanks


Add tooltips with keyboard shortcuts to manual navigation buttons

@edolstra - please add tooltips/hovertext to the manual navigation buttons:
Previous (+P)
Next (+N)
Up (+U)
The  element should be replaced by the actual combination as determined by the browser type and version.

We don’t have any paging on the manuals, so I think this can be closed. cc @jagajaga

Closing, ping if this should be reopened. Thanks, @Profpatsch.


[Bug] Get-DbaRegisteredServer Does Not Exclude Sub-Groups

Environmental information

There was no output from the command.

# Replace with output of Get-Culture

SQL Server:

Microsoft SQL Server 2016 (SP2-CU8) (KB4505830) - 13.0.5426.0 (X64)
	Jul 21 2019 21:16:47
	Copyright (c) Microsoft Corporation
	Enterprise Edition: Core-based Licensing (64-bit) on Windows Server 2016 Datacenter 10.0 <X64> (Build 14393: ) (Hypervisor)

/* REPLACE WITH output of @@LANGUAGE */

Report

There's no way to exclude subgroups when using Get-DbaRegisteredServer. The attached Zip file includes a screenshot of the relevant area of my CMS and the Trial_n.txt files show the PowerShell sessions where I try 2 different ways to exclude the subgroups. In both cases, the subgroups are returned.
The sessions are clean launches of PowerShell.exe.
Host used

[X] powershell.exe
[X] ISE
[ ] VS Code
[ ] Other (please specify)

If anything other than powershell.exe was used, please confirm that you can duplicate the issue with powershell.exe

[X] Still buggy in powershell.exe

Errors Received

No errors.
Steps to Reproduce

See Trial_1.txt and Trial_2.txt in the attached Zip file.
Expected Behavior

I expected to get one result, EHRDB-Train-L.
Actual Behavior
The command returned 3 results: the desired item plus the servers in the subgroup.

Bug_Report.zip

It looks like this is happening when -Group and -ExcludeGroup are combined. When using -ExcludeGroup by itself using Training%Nodes is excluded without an issue. I'll take a look at it.


Proof Of Concept

To Kick Off the development of this app:

Package the Unison Python interpreter
Run the latest UPG release from within the Electron App
Start development

UPG5.5.7 release can work on macOS as long as the python interpreter is Python2 (verified on Python 2.7.15 via conda).
This is a good sign for the UPG-App initiative. It means that the developer does not need to do an additional modification for the UPG-Core but only make sure that a compatible python interpreter is available. This needs further study.
Next task to try is to use the existing boilerplate to run a python app in its gui.


feat(zeroscans)!: rewrite

[!CAUTION]
The comic id's logic has changed so migration will be required

Rewrote most of the logic to cache all the comics and use said cache to avoid making extraneous requests when possible.
Filters and deeplinking were also implemented
Checklist:

[x] Updated source's version for individual source changes
[x] Set appropriate nsfw value
[x] Tested the modifications by running it on the simulator or a test device

not doing any pagination seems a little iffy, but it's probably fine since there aren't that many total titles

Their comics api endpoint doesn't have any pagination, it returns all the comics in a single response.

Yeah I know, but like with MangaSee we paginate it ourselves for aidoku since there's no sense sending all of the data to it if the user doesn't scroll down

we paginate it ourselves for aidoku

Oh, didn't know that

but it's probably fine since there aren't that many total titles

Yeah, they have like 100 titles total, so not a big issue.


feat(app): basic password authentication

Description:
This PR adds basic password authentication for the app. The password is stored in plain text in the indexed database so it's not a strong protection, however it does provide a useful basic level of protection.
This is currently an opt-in feature that users can enable from the preferences page
Additional things we should consider:

Store password in more secure way (eg, in the keychain)
Require users to enter existing password in order to change password
Enable password to grant access to all wallets
Auto-lock app after a time limit

Motivation and Context:
Fix https://github.com/LN-Zap/zap-desktop/issues/1387
How Has This Been Tested?
Manually
Screenshots (if appropriate):

Types of changes:
Feature
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage decreased (-0.3%) to 22.567% when pulling 2ad04e8bdae4a7be6bbcb7a52051a8cb087f41be on mrfelton:feat/password into e72d2b8bd7637cc8721b618feb7ca19619dce2f8 on LN-Zap:next.

Updated

Closing in favour of https://github.com/LN-Zap/zap-desktop/pull/2940


How to install the specified version？

Expected Behaviour
Actual Behaviour
Reproduce Scenario (including but not limited to)
Steps to Reproduce
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Cordova CLI version and cordova platform version
cordova --version
cordova platforms

Plugin version
cordova plugin version | grep phonegap-plugin-barcodescanner

Sample Code that illustrates the problem
Logs taken while reproducing problem

cordova plugin add<EMAIL_ADDRESS>


Lowercase name portion of references

This is a different approach to fixing #1433, after some discussion. It does two things:

Revert #1434.
Replace this with a new commit: extend regexps to allow uppercase and lowercase, then lowercase name portion after parsing.

This involves the behavioral change that ref.String() may not equal the originally parsed text under a case-sensitive string comparison.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "uppercase-hostname-take-2" git@github.com:aaronlehmann/distribution.git somewhere
$ cd somewhere
$ git rebase -i HEAD~2
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Current coverage is 52.89%

Merging #1441 into master will decrease coverage by -4.26% as of c5e1e5a

@@            master   #1441   diff @@
======================================
  Files          123     123
  Stmts        11386   11386
  Branches       799     729    -70
  Methods          0       0
======================================
- Hit           6508    6023   -485
+ Partial        799     729    -70
- Missed        4079    4634   +555

Review entire Coverage Diff as of c5e1e5a

Uncovered Suggestions

+0.28% via ...ge/driver/gcs/gcs.go#182...213
+0.24% via ...ge/driver/gcs/gcs.go#85...111
+0.21% via ...ge/driver/gcs/gcs.go#471...494
See 7 more...

Powered by Codecov. Updated on successful CI builds.

LGTM
This PR has some consequences, but, by far, this is the best solution.

I am :-1: on updating the reference package to do this normalization on the core Reference types. If we want a normalized reference then that should be explicitly done through a normalize function. This would mean pushing the ToLower logic into the existing normalize logic done in the Docker engine but I think that is where it belongs. If we want to upstream normalization then this would be part of it.
Expanding the grammar to allow A-Z in components LGTM.

@stevvooe @tonistiigi Are you okay with @dmcgowan's suggestion above?

I do not have a strict preference but I thought we would want to have less exceptions between docker/distribution references. I don't think it is too bad to have it in distribution.

@dmcgowan We really need to stop propagating these differences between docker/distirbution/reference and the engine. The goal of the package is to be able to import it and be compatible with docker. It is also oriented to show the inconsistencies in docker's reference grammar and resolve them. If we don't have enough methods (ie Normalized()), then let's add them here.
The engine is just the wrong place to do this. It is hard to keep things consistent and evolve the format if we require users of the package to use it in a certain manner.

@stevvooe I am in no way advocating keeping the difference between distribution and engine's version of reference. I am only saying that this logic goes along with this https://github.com/docker/docker/blob/master/reference/reference.go#L175. We need to integrate the packages, but that does not mean the existing reference package behavior needs to changed (String() returning something other than what was parsed is a huge behavior change). Sure we can add a Normalize method, that would satisfy my concern. In the short term we can put this normalization next to the existing normalization and focus on the proper long term fix which will understandably expand the reference package interface.


how to change diff in order to work with cloud mining Rentals

Hi,
can you please advice how to change diff in order to work with cloud mining Rentals like MiningRigRentals.com and NiceHash.com
thank's

for those services, I would recommend setting a static diff of 2, closing as its a configuration question


Missing/skipping data during conversion for ngsim us101

BUG REPORT
High Level Description
When using SMARTS to convert NGSIM raw data (.txt files) to .shf files, for some vehicle_ids, corresponding sim_times get skipped or missed, which leads to vehicles teleport in the visualization.
Also, there is a total row number difference between the raw data and the .shf files.
SMARTS version
0.6.1 rc1
Steps to reproduce the bug
scl scenario build --clean --allow-offset-map scenario/NGSIM/us101
Resulting and expected behaviour
The sim_time for all vehicles should be continuous and the row number of data before/after the conversion should be the same.
Error logs and screenshots
For example, the vehicle_id=1936 in us101_0820-0835.shf, the sim_time jumps from 913.7 to 917.2 (see the line 14,15 in screenshot below).

Also with same vehicle_id=1936, the total row number of it in us101_0820-0835.shf is 520, given that in the raw data, the row number of it is 789.

@YuechengLiu We use this issue to record the bug that you reported to us. Please feel free to edit the description for improvement if necessary. Thanks!

In general, "gaps" in a vehicle's trajectory can occur if it drives off of the mapped area for some reason.  However, that's not occurring in this case, so this was a bug.  I have a PR that I'm just wrapping up that fixes this.

I think this is fixed by PR #1403.  Please reopen this issue if it re-occurs though.


[Feature Request] When creating a new segmentation in [Segmentation Mode] there is no way to tell which series is being segmented

What feature or change would you like to see made?
When a user opens any study (with more than one series) in [Segmentation Mode] and adds a new segmentation, that segmentation will be based upon one series.  Many series may share that Frame of Reference and it will display on all of those, however, the segmentation is linked directly to only 1.
It appears whichever panel is selected when the "New Segmentation" is clicked will become the reference image for the segmentation.  When saving the DICOM SEG back to PACS and then loading it in basic mode it will pull up that reference image set.
In this example I made the hair on CTAC (Top left), eyes on CTSTD (top right), Nose on AC256 (bottom left) and Mouth  on NAC (bottom right)

The issue is there is no way to know which segmentation is linked to what series when segmenting in segmentation mode.
Additionally All the Segment 1's seem to be linked by colour.  If I change it for one Segmentation it changes for all.

Perhaps a better example would be in a 4D CT.   A user would want a different segmentation on each breathing phase.  Right now the only way I have any idea which is which is by making sure I have the right one selected when I click new segmentation and then relabelling them.

Somewhere it should really tell you

Which series is this segmentation  linked to and
Selecting a Segment should localize to the segment and highlight the correct panel.

Why should we prioritize this feature?
Right now this feature works okay with a couple series and users who think to label their segmentations.  But when you get to larger series where a user wants to segment on one as opposed to another it's going to get very confusing if the user doesn't know which series a segmentation is linked to.

I think it makes sense

Hi all,
I hope to make things easier by summarizing the requirements of this issue.  These requirements would applies to both basic and segmentation mode.
On the segmentation panel, for each dicom segmentation

For each segment, we should indicate what is the primary series it was originally contoured.  We should follow the same display format as annotation.
When user clicks on a segment, the viewport of the primary series should be highlighted and navigate to the segment.

Say, Segmentation 1 has Segment 1, and Segmentation 2 has Segment 1.  Currently, changing the colour of (Segmentation 1, Segment 1) also changes the colour of (Segmentation 2 Segment 1).  This should not happen.
On the viewport,

Say segment A is contoured on series A.  Then, the segment A should displayed with solid boundary line on series A.
For any series B in the study sharing the same FoR as series A, the segment A should be displayed with dotted boundary line of the same colour.  This gives a clear meaning that the segment can be visualized on series B because it shares the same physical space; but it was not where it was contoured.  This is also what Raystation does.

We will discuss during our design review and provide update design once we have decided how to approach

I think this is related to this issue: https://github.com/OHIF/Viewers/issues/3421

This is the design


CI: azure conditions should require succeeded()

It appears conditions: should contain succeeded() in order to work
properly when jobs are canceled, see
https://github.com/microsoft/azure-pipelines-tasks/issues/11035

Thanks Pauli


coordinator.frames = []

I'm debugging an issue where the coordinator for the proxy has no frames, so I can't scroll to any ID.
Usage:
List {
            VStack(alignment: .leading, spacing:0) {
                Group {
                        ....
                }
            }

            ScrollViewReader {
                 ForEach(..) {
                        someView.id(someUUID)
                   }
             }
}

Looks like the issue might be that my call to id on my cell seems to call the function in SwiftUI, not in this package. Aliasing the function and trying againn

The function is id(_:scrollView:) not id(_:)


Replace WebView login by local Secret Strore login

Login Activity is Introduced.
A large chunk of code have be taken from end of AuthorizationActivity and
placed in LoginActivity. As functionality is divided.
New Secret Store Authz module is being used for authorization.
expiresAt:null in token is handled.

Thank you :+1:

thank you


Change the fav icon

Fav Icon added to the public folder #6

Changes need to made in the index.html file
<link rel="icon" type="image/svg+xml" href="/vite.svg" />
Change from "/vite.svg" to "/logo.png"

Also Change : type="image/svg+xml"
to type="image/png"


Fix RespawnAsTitan not triggering FirstToFall

Spliced out from #425 by @x3Karma

Added First To Fall score event when respawning as Titan besides calling a Titanfall. Fixes issue in https://github.com/R2Northstar/NorthstarMods/issues/264.

@x3Karma do you remember whether first-to-fall flyout would also show on self? I feel like I remember it also showing for self which it currently still does not in Northstar.

@x3Karma do you remember whether first-to-fall flyout would also show on self? I feel like I remember it also showing for self which it currently still does not in Northstar.

It does show for yourself, however during the RespawnAsTitan fall animation your HUD isn't active yet I believe due to you watching a different camera before fully "respawned" as a Pilot.


New read_podspec Action

Added a new action for loading CocoaPods podspecs in a lane. This is useful for only keeping the library's version number in the podspec and everything else loads it from there.
version = read_podspec['version']

This is a great idea, thanks for the pull request. I added 2 notes before it's ready to merge :rocket:

@KrauseFx Yup you're right - fixed, sorry about that :wink:

I needed this for myself mostly :laughing: https://github.com/czechboy0/XcodeServerSDK/blob/swift-2/fastlane/Fastfile#L39

Looks great, thanks! :rocket:


experimental: Add health server support

This implements the HealthCheckNodePort functionality for running a HTTP health server that responds with the number of active backends a service has available.

As with this sort of random port picking there's always a chance for flakes. I checked with stress that we're doing OKish here:
$ stress -p 8 ./experimental.test -test.run TestScript/health
5s: 88 runs so far, 0 failures, 8 active
10s: 176 runs so far, 0 failures, 8 active
15s: 264 runs so far, 0 failures, 8 active
20s: 352 runs so far, 0 failures, 8 active
25s: 440 runs so far, 0 failures, 8 active
30s: 528 runs so far, 0 failures, 8 active
35s: 624 runs so far, 0 failures, 8 active

With enough parallelism one will hit a situation where the $PORT1 becomes reused and the check that it was closed and replaced with $PORT2 will fail, but this should be very unlikely to occur in CI. If anyone has better ideas on how to orchestrate this I'm all ears ;)


Investigate possible memory leak related to AuthenticationState

Possible reoccurrence of #938

duplicate of https://github.com/gchq/stroom-auth/issues/141


KAFKA-8410: Part 1: processor context bounds

Add type bounds to the ProcessorContext, which bounds the types that can be forwarded to child nodes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Hmm, that last commit illustrates a problem with this approach. Adding generics to a previously raw type is only a compiler warning for Java users, but it's a compiler error for Scala users, because Scala insists on having the generic arguments for any type that takes generic parameters.
I think the way to fix this is to propose to add a new ProcessorContext as well as a new Processor interface. Or we can accept this level of source code incompatibility in the Scala API. For context, many changes we've made in minor versions break Scala source compatibility, since Scala generally makes it harder for libraries to implement non-breaking deprecation paths.

Hmm, that last commit illustrates a problem with this approach. Adding generics to a previously raw type is only a compiler warning for Java users, but it's a compiler error for Scala users, because Scala insists on having the generic arguments for any type that takes generic parameters.
I think the way to fix this is to propose to add a new ProcessorContext as well as a new Processor interface. Or we can accept this level of source code incompatibility in the Scala API. For context, many changes we've made in minor versions break Scala source compatibility, since Scala generally makes it harder for libraries to implement non-breaking deprecation paths.

Hmm, that last commit illustrates a problem with this approach. Adding generics to a previously raw type is only a compiler warning for Java users, but it's a compiler error for Scala users, because Scala insists on having the generic arguments for any type that takes generic parameters.
I think the way to fix this is to propose to add a new ProcessorContext as well as a new Processor interface. Or we can accept this level of source code incompatibility in the Scala API. For context, many changes we've made in minor versions break Scala source compatibility, since Scala generally makes it harder for libraries to implement non-breaking deprecation paths.

Hmm, that last commit illustrates a problem with this approach. Adding generics to a previously raw type is only a compiler warning for Java users, but it's a compiler error for Scala users, because Scala insists on having the generic arguments for any type that takes generic parameters.
I think the way to fix this is to propose to add a new ProcessorContext as well as a new Processor interface. Or we can accept this level of source code incompatibility in the Scala API. For context, many changes we've made in minor versions break Scala source compatibility, since Scala generally makes it harder for libraries to implement non-breaking deprecation paths.

Hey, thanks for the review @mjsax ! I resolved the formatting problem and answered your question.

Java 11: `kafka.api.ConsumerBounceTest.testRollingBrokerRestartsWithSmallerMaxGroupSizeConfigDisruptsBigGroup
Java 8:
kafka.api.ConsumerBounceTest.testRollingBrokerRestartsWithSmallerMaxGroupSizeConfigDisruptsBigGroup
kafka.api.PlaintextConsumerTest.testLowMaxFetchSizeForRequestAndPartition

Retest this please.

Filed https://issues.apache.org/jira/browse/KAFKA-9831 for the unrelated failure.

Thanks, @mjsax !


BBC or Acorn Electron Emulation?

Hi,
This is a request for a new emulator. Could I please request the addition of a BBC Micro emulator and Acorn Electron? Had many a fond day playing on these as a child - they got me into computing. I know they are largely keyboard based, but are similar to the Amstrad 464 and C64 - and some of the games are iconic, such as Elite. There are several versions of a BBC Emulator - the main one being Beeb-em which is available on unix \ windows \ mac. However, what really got me interested in this is that someone ported it to the RG350/P/M - and updated the code to provide default key mappings, and the easy ability to map ked to keys in app for each game.
Link to the Beebem sites for source code, versions and supported ports:
http://www.mkw.me.uk/beebem/
http://beebem-unix.bbcmicro.com/
Forum where the the RG350 version was released with key mapping for each game: https://stardot.org.uk/forums/viewtopic.php?t=18518
Github: https://github.com/jeremyrayner/beebem-rg350/
Github for various Beemem projects and systems.
If you take the code from the RS97 or the RG350, make it run on the rk3326 with the ability to configure joypad to key mappings for each game, then this would open an amazing set of games to the chipset and various platform.
Thank you
Rik

I'm still reviewing the possibility of getting a decent working version of this emulator going.  The issue is getting it to build in my dev environment.  I'm going to try adding sdl1.2 and see if it makes a difference.

Great - thank you for looking into it - I really appreciate your efforts and all the work you have done - as do many others. If you get it working: great, if you don't; at least you tried :-)


Bug: IOS mobile safari cursor position problem

In the mobile IOS system, the cursor behind the DecoratorNode node seems to be forced to move to the beginning of the paragraph, making it impossible to focus the cursor behind the DecoratorNode node
Lexical version: newest
Steps To Reproduce

open playground  in ios safari
add some text
insert an image node
input some text

The current behavior
video here:
https://github.com/user-attachments/assets/a439cc0f-8d1c-431a-a78e-94e60e1c27f2
The expected behavior
the cursor can show behind DecoratorNode like mac safari
Impact of fix
Ensure consistent experience across all ends

This looks like a duplicate of #4487


[WIP] Use bytebuddy for proxy generation for better performance.

Still need to actually benchmark to see the difference. Either way, the code is quite a bit cleaner since Object methods are excluded at proxy-generation time, so probably worth it.

SGTM. Please let me know when it's ready.

Closing for now.

I tried to write a simple micro benchmark to wrap up this PR but cannot see any difference between each other.


Add supported browsers script to query user agent for browser JS support

window.Promise is not a comprehensive cutoff check for browser support of ES features. This PR adds browserslist-useragent-regexp to generate a regex script based on the current browsersList, which is used to set a global variable for checking if the visiting browser is in the set or not.
Additions

browserslist-useragent-regexp dependency.
yarn generateBrowserSupportScript script to generate cfgov/cfgov/v1/_includes/templates/supported_browsers.html

Changes

Include cfgov/cfgov/v1/_includes/templates/supported_browsers.html in base template to set a window. cfpbIsBrowserSupported variable, which then controls js/no-js logic.
Generate supported_browsers.html when building JS.
Alphabetize yarn scripts.

How to test this PR

Run yarn build.
Run npx browserslist.
Visit the site in a browser that is in the generated list.
Inspect the page and see that the html element has a js class, or check window.cfpbIsBrowserSupported in dev console.
Visit the site in a browser that is NOT in the generated list.
Inspect the page and see that the html element has a no-js class.
Go to package.json and edit the browserslist string, such as changing > 0.5% to > 80.5% and run yarn build and see that supported_browsers.html is updated.

Notes and todos

Open to better self-documenting ways of generating the browsers regex JS script that doesn't involve manually minified JS.

Closing in favor of https://github.com/cfpb/consumerfinance.gov/pull/7037
@wpears and I discussed this, and decided the infrastructure for this is not worth it. By setting the target to es6 and making the cutoff support of Promises, we can likely can create a browserslist from https://github.com/Fyrd/caniuse/blob/main/features-json/promises.json. Possibly by copying the stats portion of promises.json into browserslist-stats.json in the cfgov repo root, add a browserslist entry into package.json with > 0% in my stats and running npx browserslist.


envvars file containing incorrect path

After installing httpd24 there's a file called envvars in the /usr/local/etc/apache2/2.4 directory. This file contains the following script
if test "x$DYLD_LIBRARY_PATH" != "x" ; then
  DYLD_LIBRARY_PATH="/usr/local/Cellar/httpd24/2.4.18/libexec:$DYLD_LIBRARY_PATH"
else
  DYLD_LIBRARY_PATH="/usr/local/Cellar/httpd24/2.4.18/libexec"
fi
export DYLD_LIBRARY_PATH

The path in this file points to a directory inside the the apache directory. Shouldn't this envvars file be pointing to a lib directory (/usr/local/lib or /usr/lib) just as the stock envvars file that comes with OSX does ?

We just delete this file in the httpd core formula now.


Enable rules in @return statements

function-color-channel (added in https://github.com/stylelint-scss/stylelint-scss/pull/1060) works great, but it doesn't recognize uses in @return statements:
@function color-to-rgb-string($hexColor) {
  @return "#{color.red($hexColor)}, #{color.green($hexColor)}, #{color.blue($hexColor)}";
}

Probably other rules such as function-color-relative also don't work well there. I suggest to add support for @return statements for rules that make sense there. (Please comment which rules are affected.)

Thanks! Having these kinds of examples helps a lot when fixing rules.


Get Started with C Demos

Write demos to help aid the doc site with examples.

[ ] 1-authentication
[ ] 2a-atkey
[ ] 2b-atkey-with-metadata
[ ] 3a-put-self-key
[ ] 3b-get-self-key
[ ] 4a-put-shared-key
[ ] 4b-get-shared-key
[ ] 5a-put-public-key
[ ] 5b-get-public-key
[ ] 6a-notify
[ ] 6b-monitor
[ ] 7a-hooks

@XavierChanth I would like to propose this structure for the demos
 1-authentication
 2a-atkey
 2b-atkey-with-metadata
 3a-put-self-key
 3b-get-self-key
 4a-put-shared-key
 4b-get-shared-key
 5a-put-public-key
 5b-get-public-key
 6a-notify
 6b-monitor
 7a-hooks

Is that cool?

Yeah, you could also do something like this:
The docs aren't numbered but they are in this order
- Get started (mainly would be the CMakeList.txt file)
- Authentication
- atKey Reference
- CRUD ops
- Notifications
- Extras (such as hooks)

e.g. Then you could say on the CRUD page, the CRUD examples are all number 3.
 1a-authentication
 2a-atkey
 2b-atkey-with-metadata
 3a-put-self-key
 3b-get-self-key
 3c-put-shared-key
 3d-get-shared-key
 3e-put-public-key
 3f-get-public-key
 4a-notify
 4b-monitor
 5a-hooks

But whatever you prefer, fine by me.

Found a bug with fetch content, allocating 1 SP to fix


Add Kyma version as label + disclaimer-warning as annotation to all managed resources

Description
For debugging purposes and as notification for customers, we should add for all managed Kyma resources:

label which includes the Kyma version
annotation with a disclaimer that this resources is managed and changes will be reverted automatically

AC:

Each Kubernetes resource which was installed by the reconciler (at least when using the reconciler Kubernetes client) contains

a label including the used Kyma version using the key reconciler.kyma-project.io/origin-version
an annotation with a disclaimer for customers (content states "this resources is managed, changes will be automatically reverted" - final wording has to be aligned with TWs). Annotation key should be 'reconciler.kyma-project.io/...tbc... (<< check how Gardener is annotating their resources)

Reasons
Improve debugging and notify customer about managed resources
Attachments

Wording provided by @NHingerl : DO NOT EDIT - This resource is managed by Kyma. Any modifications are discarded and the resource is reverted to the original state.

Example code how Gardener is marking their resources:
 descriptionAnnotation     = "resources.gardener.cloud/description"
 descriptionAnnotationText = `DO NOT EDIT - This resource is managed by gardener-resource-manager.
       Any modifications are discarded and the resource is returned to the original state.`
 originAnnotation = "resources.gardener.cloud/origin"`