Android compile error occurs in viro v2.17.

[ 0 ] Review the documentation: https://docs.viromedia.com/
[ 0 ] Search for existing issues: https://github.com/viromedia/viro
[ 0 ] Use the latest ViroReact release: https://docs.viromedia.com/docs/releases

Environment
Please provide the following information about your environment:

Development OS: Mac
Device OS & Version: android 10
Version: ViroReact version : 2.17.0 and React Native version : 0.59.9
Device(s): LG Velvet (LM-G900N)

Description
Describe your issue in detail. Include screenshots if needed. If this is a regression, let us know.
Reproducible Demo
Let us know how to reproduce the issue. Include a code sample, screen capture, video recording. The more information you provide, the better we can support you.
I use viro v2.17 source.
However, a compilation error occurs when building an android device as shown below.
The 'npm start' command works normally in the testbed app. Why does an error occur when building directly on the android device?
How to solve this problem?
==> Task 'installGvrDebug' not found in project ':app'. Some candidates are: 'installDebug'.
[Terminal command]
Nuribomui-MacBookPro:ViroSample nuribom$ react-native run-android --variant=gvrDebug
(node:26433) Warning: Accessing non-existent property 'padLevels' of module exports inside circular dependency
(Use node --trace-warnings ... to show where the warning was created)
info Starting JS server...
info Building and installing the app on the device (cd android && ./gradlew app:installGvrDebug)...
FAILURE: Build failed with an exception.

What went wrong:
Task 'installGvrDebug' not found in project ':app'. Some candidates are: 'installDebug'.

Try:
Run gradlew tasks to get a list of available tasks. Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.7/userguide/command_line_interface.html#sec:command_line_warnings
BUILD FAILED in 576ms
error Could not install the app on the device, read the error above for details.
Make sure you have an Android emulator running or a device connected and have
set up your Android development environment:
https://facebook.github.io/react-native/docs/getting-started.html
error Command failed: ./gradlew app:installGvrDebug. Run CLI with --verbose flag for more details.
....

still same problem occurs


Select value option

Thanks so much for developing this tool. It's super useful, easy, and attractive. Really nice work.
An enhancement idea... A setting 'selectInput' for selecting the color value text when the popup opens. This would save extra clicks to type or paste in a color value.
In the bindFields method:
if (settings.selectInput){
    colorValue.select();
}

Thanks for the kind words!
Would you like to submit a pull request that implements this?

@jonkrauss are you still interested in implementing this, or should I close this issue?

Thanks for following up. It’s up to you. I’m not sure how to do what you requested.
Cheers,
Jon

On Jun 1, 2022, at 1:58 PM, Momo Bassit @.***> wrote:

@jonkrauss are you still interested in implementing this, or should I close this issue?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


Formatter for schemas

Category
Other
Describe the feature you'd like to request
The current Display trait for Cedar schemas produces ugly output (see example below). I think we should do some cleanup in the Display trait and also consider building a formatter for schemas, akin to the formatter we have for policies.
Some ideas for improving:

Improve indentation. (In the Display trait in particular, { should always be followed by a newline).
Don’t include = { } in the output, since this can always be omitted.
Don’t use the __cedar prefix if not needed.
Separate entity and action declarations by a newline, and sort entries alphabetically.
If entity type or action declarations have the same body, combine them into one definition.

Example
Here is the CLI output of converting the this JSON schema to the new syntax:
entity Album in [Account, Album] = {  };
entity UserGroup  = {  };
entity AccountGroup  = {  };
entity Video in [Account, Album] = {  };
entity User in [UserGroup] = {  };
entity Photo in [Account, Album] = {  };
entity Administrator  = {  };
entity Account  = {  };
action "edit" appliesTo {  principal: [User],
  resource: [Photo],
  context: { "authenticated": __cedar::Bool,
"confidence_score": __cedar::decimal,
"source_ip": __cedar::ipaddr,
 }
};
action "view" appliesTo {  principal: [User, Administrator],
  resource: [Photo, Video],
  context: { "authenticated": __cedar::Bool,
"confidence_score": __cedar::decimal,
"source_ip": __cedar::ipaddr,
 }
};
action "listPhotos" appliesTo {  principal: [User],
  resource: [Album],
  context: { "authenticated": __cedar::Bool,
"confidence_score": __cedar::decimal,
"source_ip": __cedar::ipaddr,
 }
};
action "delete" appliesTo {  principal: [User],
  resource: [Photo],
  context: { "authenticated": __cedar::Bool,
"confidence_score": __cedar::decimal,
"source_ip": __cedar::ipaddr,
 }
};
action "listAlbums" appliesTo {  principal: [User],
  resource: [Account],
  context: { "authenticated": __cedar::Bool,
"confidence_score": __cedar::decimal,
"source_ip": __cedar::ipaddr,
 }
};
action "addPhoto" appliesTo {  principal: [User],
  resource: [Album],
  context: { "authenticated": __cedar::Bool,
"confidence_score": __cedar::decimal,
"source_ip": __cedar::ipaddr,
 }
};
action "comment" appliesTo {  principal: [User],
  resource: [Photo],
  context: { "authenticated": __cedar::Bool,
"confidence_score": __cedar::decimal,
"source_ip": __cedar::ipaddr,
 }
};

Here is the (manually) prettified form:
entity Account, AccountGroup, Administrator, UserGroup;
entity Album, Photo, Video in [Account, Album];
entity User in [UserGroup];

action addPhoto, listPhotos appliesTo {
    principal: [User],
    resource: [Album],
    context: {
        authenticated: Bool,
        confidence_score: decimal,
        source_ip: ipaddr,
    }
};
action comment, delete, edit appliesTo {
    principal: [User],
    resource: [Photo],
    context: {
        authenticated: Bool,
        confidence_score: decimal,
        source_ip: ipaddr,
    }
};
action listAlbums appliesTo {
    principal: [User],
    resource: [Account],
    context: {
        authenticated: Bool,
        confidence_score: decimal,
        source_ip: ipaddr,
    }
};
action view appliesTo {
    principal: [User, Administrator],
    resource: [Photo, Video],
    context: {
        authenticated: Bool,
        confidence_score: decimal,
        source_ip: ipaddr,
    }
};

Describe alternatives you've considered
No response
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

I think we want a formatter like the Cedar policy formatter. That being said, I think there is a lot of room for improvement.

An item that #780 missed relates to multiple namespaces
namespace NS2 {
}namespace NS1 {
}

was generated from
{
  "NS1": {
    "entityTypes": {},
    "actions": {}
  },
  "NS2": {
    "entityTypes": {},
    "actions": {}
  }
}

Thanks for pointing that out @hakanson (and sorry for the delayed reply!!) I added this fix to #1160, which will be released as part of 4.0.


SQS always tries to SetQueueAttribute

What happened?
We are migrating resources to crossplane and are in testing phase. I am currently trying to take control of existing SQS queues without giving SetQueueAttributes permissions to make sure everything is mapped properly through our existing templates.
This appears to be working as expected for queues without policy declared.
I am not sure I understand this correctly but digging around in the source code  it looks like the string comparison doesn’t take into account ordering or syntax such as spaces between keys and values. Might it make sense to convert both policies into map[string]interface{} and then run the cmp.Equal() function to compare those
How can we reproduce it?
Given the following permissions:
data "aws_iam_policy_document" "crossplane_provider_aws_sqs" {
  statement {
    actions = [
      "sqs:Get*",
      "sqs:List*",
      "sqs:CreateQueue",
      "sqs:TagQueue",
      "sns:UntagQueue"
    ]
    resources = ["*"]
  }
}

What I see in the AWS console policy tab:
{
  "Version": "2008-10-17",
  "Statement": [
    {
      "Sid": "AllowSNSPublish",
      "Effect": "Allow",
      "Principal": {
        "AWS": "*"
      },
      "Action": "sqs:SendMessage",
      "Resource": "<sqs-arn>",
      "Condition": {
        "ArnEquals": {
          "aws:SourceArn": "<sns-arn>"
        }
      }

Crossplane resource (all other attributes match)
apiVersion: sqs.aws.crossplane.io/v1beta1
kind: Queue
metadata:
  name: test-sqs
spec:
  deletionPolicy: Orphan
  forProvider:
    delaySeconds: 0
    fifoQueue: false
    maximumMessageSize: 8192
    messageRetentionPeriod: 604800
    policy: |
      {"Version": "2008-10-17", "Statement": [{"Sid": "AllowSNSPublish","Effect": "Allow","Principal": {"AWS": "*"},"Action": "sqs:SendMessage","Resource": "<sqs-arn>","Condition": {"ArnEquals": {"aws:SourceArn": "<sns-arn>"}}}]}
    receiveMessageWaitTimeSeconds: 20
      maxReceiveCount: 4
    region: <your-region>
    visibilityTimeout: 300
  providerConfigRef:
    name: provider-aws-config

Events:
  Type     Reason                        Age                From                                 Message
  ----     ------                        ----               ----                                 -------
  Warning  CannotUpdateExternalResource  40m (x3 over 51m)  managed/queue.sqs.aws.crossplane.io  failed to update the Queue resource: api error AccessDenied: Access to the resource https://sqs.<region>.amazonaws.com/ is denied.

This issue is happenign with me as well.


Division by 0 error

Hi,
I am trying to test the trained model on my custom image dataset. I modified the config.yaml file as per the suggestions in another issue but I get the following error
Traceback (most recent call last):
File "C:\Program Files\JetBrains\PyCharm\helpers\pydev\pydevd.py", line 2060, in
main()
File "C:\Program Files\JetBrains\PyCharm\helpers\pydev\pydevd.py", line 2054, in main
globals = debugger.run(setup['file'], None, None, is_module)
File "C:\Program Files\JetBrains\PyCharm\helpers\pydev\pydevd.py", line 1405, in run
return self._exec(is_module, entry_point_fn, module_name, file, globals, locals)
File "C:\Program Files\JetBrains\PyCharm\helpers\pydev\pydevd.py", line 1412, in _exec
pydev_imports.execfile(file, globals, locals)  # execute the script
File "C:\Program Files\JetBrains\PyCharm\helpers\pydev_pydev_imps_pydev_execfile.py", line 18, in execfile
exec(compile(contents+"\n", file, 'exec'), glob, loc)
File "C:/Users/Xyedj/PycharmProjects/PolyLaneNet-master/test.py", line 162, in
_, mean_loss = test(model, test_loader, evaluator, exp_root, cfg, epoch=test_epoch, view=args.view)
File "C:/Users/Xyedj/PycharmProjects/PolyLaneNet-master/test.py", line 78, in test
return evaluator, loss / total_iters
ZeroDivisionError: division by zero
I believe it has something to do with the dataset I am using. Can you specify the dataset format to be used with test.py. Do I only need the test images in .jpg or I should have their associated annotations as well?

It seems the dataset loader could not find any data. Are you using the NoLabelDataset as suggested in #2 ? This class loads every image in a directory recursively searching for images with the extension provided (e.g., .jpg), but does not load any annotations.
It would be easier for me to help if you could provide the config.yaml used.

Yes I used that setting. Here is my config file:
Training settings
seed: 1
exps_dir: 'experiments'
iter_log_interval: 1
iter_time_window: 100
model_save_interval: 1
backup:
model:
name: PolyRegression
parameters:
num_outputs: 35 # (5 lanes) * (1 conf + 2 (upper & lower) + 4 poly coeffs)
pretrained: true
backbone: 'efficientnet-b0'
pred_category: false
curriculum_steps: [9000, 9000, 0, 0]
loss_parameters:
conf_weight: 1
lower_weight: 1
upper_weight: 1
cls_weight: 0
poly_weight: 300
batch_size: 16
epochs: 2695
optimizer:
name: Adam
parameters:
lr: 3.0e-4
lr_scheduler:
name: CosineAnnealingLR
parameters:
T_max: 385
Testing settings
test_parameters:
conf_threshold: 0.5
Dataset settings
datasets:
train:
type: LaneDataset
parameters:
dataset: nolabel_dataset
split: train
img_size: [360, 640]
normalize: true
aug_chance: 0.9090909090909091 # 10/11
augmentations:
- name: Affine
parameters:
rotate: !!python/tuple [-10, 10]
- name: HorizontalFlip
parameters:
p: 0.5
- name: CropToFixedSize
parameters:
width: 1152
height: 648
root: "datasets"
test: &test
type: LaneDataset
parameters:
dataset: nolabel_dataset
split: val
max_lanes: 5
img_size: [360, 640]
root: "datasets"
normalize: true
augmentations: []
val = test
val:
<<: *test

I got it working thanks. There was a problem with my dataset path :) I have one question though, how well does your model generalize for line detection in general and not specifically road lane-line detection?

The method was evaluated only on the task of lane detection, so I don't know how it performs in other tasks. Nevertheless, it should perform well depending on the kind of images used.

Hi , i am facing similar issue with below error.
[2023-11-25 18:47:26,144] [INFO] Starting testing.
[2023-11-25 18:47:26,636] [INFO] Testing time: 0.3135
[2023-11-25 18:47:26,637] [INFO]
[2023-11-25 18:47:26,637] [ERROR] Uncaught exception
Traceback (most recent call last):
File "/PolyLaneNet/test.py", line 160, in
_, mean_loss = test(model, test_loader, evaluator, exp_root, cfg, epoch=test_epoch, view=args.view)
File "/PolyLaneNet/test.py", line 77, in test
return evaluator, loss / total_iters
ZeroDivisionError: division by zero
below is my config file used and i verified the image path for test, images are there.
Not getting sure what may be the issue?

Seems like some spacing was the issue, that error is gone, but facing some new error below one.
[2023-11-25 19:27:44,782] [INFO] Starting testing.
[2023-11-25 19:27:47,249] [INFO] Testing iteration: 1/20
[2023-11-25 19:28:18,194] [ERROR] Uncaught exception
Traceback (most recent call last):
File "PolyLaneNet/test.py", line 160, in
_, mean_loss = test(model, test_loader, evaluator, exp_root, cfg, epoch=test_epoch, view=args.view)
File "PolyLaneNet/test.py", line 61, in test
preds = test_loader.dataset.draw_annotation(
File "PolyLaneNet/lib/datasets/lane_dataset.py", line 172, in draw_annotation
cv2.putText(img, str(i), tuple(points[0]), fontFace=cv2.FONT_HERSHEY_COMPLEX, fontScale=1, color=color)
cv2.error: OpenCV(4.8.1) :-1: error: (-5:Bad argument) in function 'putText'

Overload resolution failed:

Layout of the output array img is incompatible with cv::Mat
Expected Ptrcv::UMat for argument 'img'

Any thoughts?


basic plurals (zero, one, two, other)

What it does:
Change messages to fit the right counter passed in by "count", when a { _plural: true } key is included in the translations file.
English and Japanese examples:
friends: [
   _plural: true,
   zero: "no friends",
   one: "1 friend",
   other: "%{count} friends"
 ]

friends: [
  _plural: true,
  other: "%{count}人友達"
]

Strengths: makes your English plurals easier, most European languages, Chinese/Japanese/Korean and many more. Any language whose special rules can be limited to "zero", "one", "two", or "other"
Weaknesses: doesn't work with Arabic or Russian (which have special "few" and "many" plurals), and other languages with their own definitions of "few" and "many". Cannot add new plural rule lambdas with current setup =(
closes #9

@mapmeld Have you taken a look at https://github.com/xavier/linguist/commit/9b7658636015a714ffddb7dddbb3c63ec7934530?
The approach is very straight forward and it's pretty easy to add additional forms from http://www.unicode.org/cldr/charts/27/supplemental/language_plural_rules.html. I have a PR open against the branch for addition of Arabic rules, currently.

@erikreedstrom oh cool - I hadn't seen that before. I like that approach


Request to support slack api_url_file and pagerduty routing_key_file options

Alertmanger supports both configuration options in Slack and Pagerduty to use a file containing the secret rather than injecting credentials in Terraform. This will simplify the setup and management of credentials for complex Alertmanager configurations defining multiple channels and pagerduty teams.
Options mentioned above:
# The Slack webhook URL. Either api_url or api_url_file should be set.
[ api_url_file: [<filepath>](https://prometheus.io/docs/alerting/latest/configuration/#filepath) | default = global.slack_api_url_file ]

# The PagerDuty integration key (when using PagerDuty integration type `Events API v2`).
# Read the Pager Duty routing key from a file.
routing_key_file: [<filepath>](https://prometheus.io/docs/alerting/latest/configuration/#filepath)

It would be nice to support this.

Hi @jnavarrof,
This is not possible because mimir doesn't support it. Many config file parameters are not supported, this is mainly caused by the multi tenants which is difficult to load a specific config file for a given tenant. Every alertmanager config is stored is s3 object storage.
You can check which parameters that are not supported by mimir:
https://github.com/grafana/mimir/blob/main/pkg/alertmanager/api.go

Gi @fgouteroux ,
I see. I guess we will need to find another way.
Many thanks!


New line in UITextButton doesn't work correctly

Describe the bug
On the last version (0.5.7), new lines (using br tag) in UITextButton element are only "line return" and can't produce blank lines.
It works fine on previous version.
To Reproduce
Steps to reproduce the behavior:
Simply run the text test from the examples repository (https://github.com/MyreMylar/pygame_gui_examples/blob/master/text_test.py).
Expected behavior
Multiple br tags should generate blank lines.
Screenshots

Platform and software (please complete the following information):

OS: Windows 10
Pygame GUI version: 0.5.7
Pygame version: 2.1.0

This should be fixed in 0.6.0 - the underlying html parsing, text rendering and layout code has changed completely anyway so there will likely be new exciting bugs.
I'll double check it though.

Checked it out in 0.6.0 and this definitely works again now.


Add action to Resonant Reflection Campaign items.

Created Activate Resonant Reflection action.
Added Item Alteration RE to the corresponding Resonant Reflection items to add the effect description and traits to the  Activate Resonant Reflection action.
Added Grant Item RE to the above items.
Localized the descriptions in re-en.json.

Please review when you get a moment - still having challenges with getting the item to not re-add more than once.

Changes made. Please check when  you get a moment


Add slack notification to Travis

@david-driscoll @davidfowl @DustinCampbell @filipw
Guys, this for the Slack team I set up for OmniSharp.

What was wrong with jabbr?

@Yantrio nothing wrong with Jabbr. But Slack offers more. Like mobile and more extensions. Most importantly, try something new :)

I'm fine with something new, but I'll be on Jabbr for the indefinite future as well. :)

On a personal note, I think gitter as better integration. :wink:

All right. I'll merge once CI passed.


UI improvement for file properties in list view / tile view

Is your feature request related to a problem? Please describe.
When a user browses through the files in a file explorer, they try to identify the file based on two things:

Name of the file
Icon/thumbnail of the file

In the current app, all the properties shown are of the same importance (name, size, file type...)
While yes, the information is helpful and is displayed at all times, they are only necessary when the user wants to know it.
Describe the solution you'd like
Except for the name, other properties can be of a lesser opacity, so that the name and icon/thumbnail stand out better.

Current implementation

Proposed implementation
While the exact color values are up for further discussion, I immediately notice that there is a huge improvement in the visual quality of the screen.

Current implementation

Proposed implementation
The same applies to the light theme as well, though the difference it makes seems lesser compared to the dark theme.

Current implementation

Proposed implementation
Describe alternatives you've considered
This is a generic UI improvement and there are no alternatives I could think of.
Instead of making secondary information less prominent (reducing opacity/grey out), the primary information can be made more prominent (by making the name bolder/bigger) but that doesn't seem visually elegant and results in unnecessarily complicated UI.

I had already tested with a darker color, the problem was that depending on the accent color, when selecting a file it did not look very good:

but if anyone can solve that it would be great

I think this issue is because the text is just 'grey'. If you change the opacity (transparency) instead, the background color bleeds through the text and gives a pinkish tint instead.

This is an approximate recreation, and the opacity percentage is up for discussion and accessibility tests.
Again, I don't know if the code implementation allows for this, but this is a very common UI pattern.

Test

LGTM :)

after more testing:
White 50% looks good in dark mode
Black 50% looks good in light mode

So, I ran some rough accessibility tests with given percentages and the app's colors. Slight change to 54% or 55% opacity for white passes WCAG AA requirements.. since white is too bright on a screen, the text opacity needs to be a bit higher. I think 54% for both dark and light modes is good.

I am now working on this issue, instead of changing opacity or color I am looking at changing the font weight.

After spending some time on this and testing changes to the font weight, I have come to the conclusion that changing the font weight is not the correct path here.

I would suggest using a 60% opacity to try to improve the Color Contrast for accessibility


Formula Recognition script

Firstly you should launch
formula_recogninion_demo/pic2latex.sh
then
formula_recogninion_demo/sample_launch.sh
to test
I'll add python module for easy integration soon
In case of error please refer to orig and install required
PS:
IDK why I can't download vocab.json by downloader. I decided to skip debugging and just add it there

Checked on my ubuntu laptop  - working


Add expected space

          1 Error

:no_entry_sign:
Please provide a summary in the Pull Request description

  Generated by :no_entry_sign: Danger

Nice! Thanks for the fix


Synchronize prefix argument for browse-url in search and show

C-u b now uses generic in view as in search

I like this interface tweak. Is this a browse-url idiom from elsewhere in Emacs?

Did not find (m)any reference inside my elpa dir/emacs source, so probably not that commonly used.
My use-case is mainly, that I have set generic to 'open' and non-generic to eww. For some feeds firefox is just better for reading and for some of my feeds eww is needed to get the actual post content and not only the excerpt. It could be interesting to add configurable default actions for each feed.


HDDS-1610. applyTransaction failure should not be lost on restart.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
46
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

0
mvndep
96
Maven dependency ordering for branch

+1
mvninstall
751
trunk passed

+1
compile
466
trunk passed

+1
checkstyle
86
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
1092
branch has no errors when building and testing our client artifacts.

+1
javadoc
189
trunk passed

0
spotbugs
555
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
847
trunk passed

_ Patch Compile Tests _

0
mvndep
37
Maven dependency ordering for patch

+1
mvninstall
676
the patch passed

+1
compile
489
the patch passed

+1
javac
489
the patch passed

-0
checkstyle
40
hadoop-ozone: The patch generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0)

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
876
patch has no errors when building and testing our client artifacts.

+1
javadoc
179
the patch passed

+1
findbugs
720
the patch passed

_ Other Tests _

+1
unit
330
hadoop-hdds in the patch passed.

-1
unit
2435
hadoop-ozone in the patch failed.

+1
asflicense
38
The patch does not generate ASF License warnings.

9613

Reason
Tests

Failed junit tests
hadoop.hdds.scm.pipeline.TestPipelineClose

hadoop.ozone.client.rpc.TestContainerStateMachine

hadoop.ozone.om.TestScmSafeMode

hadoop.ozone.web.client.TestBuckets

hadoop.ozone.om.TestOMRatisSnapshots

hadoop.ozone.client.rpc.TestContainerStateMachineFailures

hadoop.ozone.client.rpc.TestOzoneRpcClient

hadoop.ozone.om.TestOzoneManagerHA

hadoop.ozone.om.TestContainerReportWithKeys

hadoop.ozone.client.rpc.TestOzoneAtRestEncryption

hadoop.ozone.client.rpc.TestOzoneRpcClientWithRatis

hadoop.ozone.client.rpc.TestCommitWatcher

hadoop.ozone.client.rpc.TestSecureOzoneRpcClient

hadoop.ozone.dn.scrubber.TestDataScrubber

hadoop.ozone.web.TestOzoneVolumes

hadoop.ozone.container.common.transport.server.ratis.TestCSMMetrics

hadoop.ozone.client.rpc.TestBlockOutputStreamWithFailures

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/1/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 48b3f23d31a4 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / f8ea6e1

Default Java
1.8.0_212

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/1/artifact/out/diff-checkstyle-hadoop-ozone.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/1/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/1/testReport/

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

modules
C: hadoop-hdds/container-service hadoop-ozone/integration-test U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

Thanks @mukul1987 . In ratis, as fara as my understanding goes, before taking a snapshot we wait for all the pending applyTrannsaction futures to complete and since now with the patch, the applyTransaction exception is being propagated to Ratis, ideally snapshot creation will fail in Ratis.
I will address the remaining review comments as part of the next patch.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
35
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

0
mvndep
73
Maven dependency ordering for branch

+1
mvninstall
612
trunk passed

+1
compile
387
trunk passed

+1
checkstyle
76
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
916
branch has no errors when building and testing our client artifacts.

+1
javadoc
177
trunk passed

0
spotbugs
467
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
687
trunk passed

_ Patch Compile Tests _

0
mvndep
76
Maven dependency ordering for patch

+1
mvninstall
564
the patch passed

+1
compile
382
the patch passed

+1
javac
382
the patch passed

-0
checkstyle
42
hadoop-ozone: The patch generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0)

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
778
patch has no errors when building and testing our client artifacts.

+1
javadoc
179
the patch passed

+1
findbugs
731
the patch passed

_ Other Tests _

+1
unit
350
hadoop-hdds in the patch passed.

-1
unit
371
hadoop-ozone in the patch failed.

+1
asflicense
36
The patch does not generate ASF License warnings.

6653

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/2/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 81e1c0db1839 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 3cc0ace

Default Java
1.8.0_222

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/2/artifact/out/diff-checkstyle-hadoop-ozone.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/2/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/2/testReport/

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

modules
C: hadoop-hdds/container-service hadoop-ozone/integration-test U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/2/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
43
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

0
mvndep
22
Maven dependency ordering for branch

+1
mvninstall
585
trunk passed

+1
compile
352
trunk passed

+1
checkstyle
66
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
834
branch has no errors when building and testing our client artifacts.

+1
javadoc
146
trunk passed

0
spotbugs
459
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
657
trunk passed

_ Patch Compile Tests _

0
mvndep
29
Maven dependency ordering for patch

+1
mvninstall
574
the patch passed

+1
compile
376
the patch passed

+1
javac
376
the patch passed

-0
checkstyle
37
hadoop-ozone: The patch generated 4 new + 0 unchanged - 0 fixed = 4 total (was 0)

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
638
patch has no errors when building and testing our client artifacts.

+1
javadoc
165
the patch passed

+1
findbugs
667
the patch passed

_ Other Tests _

+1
unit
292
hadoop-hdds in the patch passed.

-1
unit
1955
hadoop-ozone in the patch failed.

+1
asflicense
40
The patch does not generate ASF License warnings.

7657

Reason
Tests

Failed junit tests
hadoop.ozone.container.common.transport.server.ratis.TestCSMMetrics

hadoop.ozone.client.rpc.TestOzoneAtRestEncryption

hadoop.ozone.client.rpc.Test2WayCommitInRatis

hadoop.hdds.scm.pipeline.TestRatisPipelineCreateAndDestory

hadoop.ozone.client.rpc.TestOzoneRpcClient

hadoop.ozone.om.TestKeyManagerImpl

hadoop.ozone.client.rpc.TestMultiBlockWritesWithDnFailures

hadoop.ozone.om.TestScmSafeMode

hadoop.ozone.client.rpc.TestBlockOutputStreamWithFailures

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/3/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux f451f6c48524 4.4.0-157-generic #185-Ubuntu SMP Tue Jul 23 09:17:01 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 00b5a27

Default Java
1.8.0_212

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/3/artifact/out/diff-checkstyle-hadoop-ozone.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/3/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/3/testReport/

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

modules
C: hadoop-hdds/container-service hadoop-ozone/integration-test U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/3/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
142
Docker mode activated.

_ Prechecks _

+1
dupname
1
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

0
mvndep
88
Maven dependency ordering for branch

+1
mvninstall
804
trunk passed

+1
compile
444
trunk passed

+1
checkstyle
86
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
958
branch has no errors when building and testing our client artifacts.

+1
javadoc
166
trunk passed

0
spotbugs
445
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
644
trunk passed

_ Patch Compile Tests _

0
mvndep
32
Maven dependency ordering for patch

+1
mvninstall
595
the patch passed

+1
compile
401
the patch passed

+1
cc
401
the patch passed

+1
javac
401
the patch passed

-0
checkstyle
42
hadoop-ozone: The patch generated 8 new + 0 unchanged - 0 fixed = 8 total (was 0)

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
731
patch has no errors when building and testing our client artifacts.

+1
javadoc
175
the patch passed

+1
findbugs
699
the patch passed

_ Other Tests _

+1
unit
361
hadoop-hdds in the patch passed.

-1
unit
2308
hadoop-ozone in the patch failed.

+1
asflicense
45
The patch does not generate ASF License warnings.

8928

Reason
Tests

Failed junit tests
hadoop.ozone.client.rpc.TestSecureOzoneRpcClient

hadoop.ozone.client.rpc.TestOzoneAtRestEncryption

hadoop.ozone.client.rpc.TestContainerStateMachineFailures

hadoop.ozone.client.rpc.TestBlockOutputStreamWithFailures

hadoop.ozone.client.rpc.TestOzoneRpcClient

hadoop.ozone.container.server.TestSecureContainerServer

hadoop.hdds.scm.pipeline.TestRatisPipelineCreateAndDestory

hadoop.ozone.container.common.transport.server.ratis.TestCSMMetrics

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/4/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle cc

uname
Linux 07f043e4650b 4.15.0-54-generic #58-Ubuntu SMP Mon Jun 24 10:55:24 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 0b507d2

Default Java
1.8.0_222

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/4/artifact/out/diff-checkstyle-hadoop-ozone.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/4/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/4/testReport/

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

modules
C: hadoop-hdds/container-service hadoop-ozone/integration-test hadoop-ozone/tools U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/4/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
70
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

0
mvndep
70
Maven dependency ordering for branch

-1
mvninstall
141
hadoop-ozone in trunk failed.

-1
compile
49
hadoop-ozone in trunk failed.

+1
checkstyle
63
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
915
branch has no errors when building and testing our client artifacts.

+1
javadoc
156
trunk passed

0
spotbugs
201
Used deprecated FindBugs config; considering switching to SpotBugs.

-1
findbugs
101
hadoop-ozone in trunk failed.

_ Patch Compile Tests _

0
mvndep
23
Maven dependency ordering for patch

-1
mvninstall
137
hadoop-ozone in the patch failed.

-1
compile
52
hadoop-ozone in the patch failed.

-1
cc
52
hadoop-ozone in the patch failed.

-1
javac
52
hadoop-ozone in the patch failed.

+1
checkstyle
63
the patch passed

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
721
patch has no errors when building and testing our client artifacts.

+1
javadoc
153
the patch passed

-1
findbugs
101
hadoop-ozone in the patch failed.

_ Other Tests _

+1
unit
339
hadoop-hdds in the patch passed.

-1
unit
329
hadoop-ozone in the patch failed.

+1
asflicense
29
The patch does not generate ASF License warnings.

4445

Reason
Tests

Failed junit tests
hadoop.ozone.s3.TestOzoneClientProducer

Subsystem
Report/Notes

Docker
Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle cc

uname
Linux 960f8c74122e 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 0e4b757

Default Java
1.8.0_212

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/branch-mvninstall-hadoop-ozone.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/branch-compile-hadoop-ozone.txt

findbugs
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/branch-findbugs-hadoop-ozone.txt

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/patch-mvninstall-hadoop-ozone.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/patch-compile-hadoop-ozone.txt

cc
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/patch-compile-hadoop-ozone.txt

javac
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/patch-compile-hadoop-ozone.txt

findbugs
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/patch-findbugs-hadoop-ozone.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/testReport/

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

modules
C: hadoop-hdds/container-service hadoop-ozone/integration-test hadoop-ozone/tools U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/5/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

/retest

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
79
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 3 new or modified test files.

_ trunk Compile Tests _

0
mvndep
24
Maven dependency ordering for branch

+1
mvninstall
665
trunk passed

+1
compile
409
trunk passed

+1
checkstyle
82
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
976
branch has no errors when building and testing our client artifacts.

+1
javadoc
174
trunk passed

0
spotbugs
465
Used deprecated FindBugs config; considering switching to SpotBugs.

+1
findbugs
673
trunk passed

_ Patch Compile Tests _

0
mvndep
34
Maven dependency ordering for patch

+1
mvninstall
626
the patch passed

+1
compile
429
the patch passed

+1
cc
429
the patch passed

+1
javac
429
the patch passed

+1
checkstyle
86
the patch passed

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
825
patch has no errors when building and testing our client artifacts.

+1
javadoc
215
the patch passed

+1
findbugs
802
the patch passed

_ Other Tests _

+1
unit
381
hadoop-hdds in the patch passed.

-1
unit
3032
hadoop-ozone in the patch failed.

+1
asflicense
42
The patch does not generate ASF License warnings.

9719

Reason
Tests

Failed junit tests
hadoop.ozone.container.common.transport.server.ratis.TestCSMMetrics

hadoop.ozone.client.rpc.TestContainerStateMachineFailures

hadoop.ozone.client.rpc.TestBlockOutputStreamWithFailures

hadoop.ozone.container.server.TestSecureContainerServer

hadoop.ozone.client.rpc.TestOzoneRpcClientForAclAuditLog

hadoop.hdds.scm.pipeline.TestRatisPipelineCreateAndDestory

Subsystem
Report/Notes

Docker
Client=19.03.0 Server=19.03.0 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/6/artifact/out/Dockerfile

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle cc

uname
Linux 6314dd3b3d85 4.15.0-52-generic #56-Ubuntu SMP Tue Jun 4 22:49:08 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 9b8359b

Default Java
1.8.0_212

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/6/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/6/testReport/

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

modules
C: hadoop-hdds/container-service hadoop-ozone/integration-test hadoop-ozone/tools U: .

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1226/6/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

The test failures are not related to the patch.

Thanks for working on this @bshashikant. +1 the patch looks good to me. The test failures do not look related.

Thanks @mukul1987 and @supratimdeka for the review. I have committed the change to trunk.


Helm install return 'failed to download' error

Hi,
This error has already perplexed me for a long time.
helm install daily/service-geo
Error: failed to download "daily/service-geo" (hint: running `helm repo update` may help)

Info:
uname -a
Linux daily02 4.4.0-79-generic #100-Ubuntu SMP Wed May 17 19:58:14 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux

helm version
Client: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.9.1", GitCommit:"20adb27c7c5868466912eebdf6664e7390ebe710", GitTreeState:"clean"}

helm repo update
Hang tight while we grab the latest from your chart repositories...
...Skip local chart repository
...Successfully got an update from the "daily" chart repository
...Successfully got an update from the "stable" chart repository
Update Complete. ⎈ Happy Helming!⎈

helm search daily
NAME             	CHART VERSION               	APP VERSION	DESCRIPTION
daily/service-geo	master_180506004437_2c84e673	2.0        	A Helm chart for Kubernetes

helm install daily/service-geo
Error: failed to download "daily/service-geo" (hint: running `helm repo update` may help)

The index.yaml:
apiVersion: v1
entries:
  service-geo:
  - apiVersion: v1
    appVersion: "2.0"
    created: "2018-05-11T08:51:27Z"
    description: A Helm chart for Kubernetes
    digest: 536745de7d53ef4d9a501ce71f1d87959d7e2aca306d0ccf952c6c281dbc56d9
    name: service-geo
    urls:
    - http://chart.daily.net/charts/service-geo-master_180506004437_2c84e673.tgz
    version: master_180506004437_2c84e673
  - apiVersion: v1

The url  http://chart.daily.net/charts/service-geo-master_180506004437_2c84e673.tgz is downloadable with curl.
The chartmuseum is run with docker, the docker compose file is:
version: '2'
services:
 chartmuseum:
  image: "chartmuseum/chartmuseum:latest"
  #volumes:
  ports:
   - "8080"
  container_name: "chartmuseum"
  environment:
   - SERVICE_NAME=chartmuseum
   - SERVICE_TAGS=ha,public
   - PORT=8080
   - DEBUG=1
   - STORAGE=alibaba
   - STORAGE_ALIBABA_BUCKET=bucket
   - STORAGE_ALIBABA_PREFIX=repos/chart/daily
   - STORAGE_ALIBABA_ENDPOINT=oss-cn-hangzhou.aliyuncs.com
   - ALIBABA_CLOUD_ACCESS_KEY_ID=key_id
   - ALIBABA_CLOUD_ACCESS_KEY_SECRET=key_secret
   - CHART_URL=http://chart.daily.net

@rmfish I think this is due to the version of your chart not being proper semantic version.
Can you please try running the following:
helm install --devel daily/service-geo

The --devel flag is documented as
use development versions, too. Equivalent to version '>0.0.0-0'. If --version is set, this is ignored.

Does this solve the issue?

No, still have the same error with --devel  and  --version master_180506004437_2c84e673
But when I modified the chart version to 1.0.0 , it worked fine.
My chart version contains git branch & build time & commit revision. It is automatically created by CI job. It is better to support :).

@rmfish yes I agree. However, it is now apparent that this is an issue with Helm non supporting non-semvar looking versions

There is an related issue in the helm. https://github.com/kubernetes/helm/issues/4008


Use without laravel

I'm able to use this with any rest service that uses laravel pattern for endpoints? I have a restful springboot application and i would like to use together that, If I apply laravel pattern for the endpoints is it supposed to work fine or is there any limitation?

Should work perfectly fine without Laravel. 👍


Isnt there any way to automate the entire authentication process while creating a tweet?

https://github.com/twitterdev/Twitter-API-v2-sample-code/blob/aa131cfa3ee2d440db22fb9f1753374b26b7125b/Manage-Tweets/create_tweet.py#L34
I have to manually visit the authentication link and copy-paste the verifier pin in terminal. Isn't consumer_key and consumer_secret enough to authenticate these whole process?
Do I need an elevated access to do it?

The consumer key and secret identify your app. This is OK for read-only functions like getting a public Tweet, or searching for Tweets. For functions where you need to post, you will need additional values that enable your app to post as an authenticated user.
The PIN-based approach enables the code to do the sign-in and get a usable pair of access token and access token secret. If you were coding this yourself, you could build the sign-in flow and cache/save the access token and access token and secret. This code is just a single example to demonstrate one method of getting tokens to enable you to post a Tweet.

Thanks for pointing it out. I will try it...

Hi @hDmtP! Please, how did you solve this? 🙏  I also wonder the same thing as you, and I'm just starting with this API.

i don't understand it, is it standard to v2 api, puts handly a number get from website?
this is nonsense.


trying to install maui and not seeing android show up on VS2022 tools menu

[Enter feedback here]
HI,
I am trying to install maui preview 6 for the first time.   I have installed vs2022 with the workloads as shown in this page, as well as MSIX package, and maui-check.  (also android studio 20203.1)   I am expecting to see android to show up in the vs2022 tools menu, but it isn't there.  Ultimately I am trying to connect to an android device for app development.
Is there something I am missing in the installation, or is there a different place in vs2022 for accessing android tools, etc.
Thanks,
Bob

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

ID: 7607155a-9a56-d752-c5ac-c3aeb6b128bc
Version Independent ID: 9a62d421-d009-30b3-843e-a92bfc21c061
Content: .NET MAUI installation - .NET MAUI
Content Source: docs/get-started/installation.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

BTW: I tried running maui-check in an adminstrator cmd window, as per another issue recently entered.
In this case, maui-check did NOT find anything new missing.
However, then I opened VS2022 preview again, and this time I found what I was looking for: "framework-new6.0-android"
So I am past that problem and moving on.


feat: ability to override liveness and readiness probe paths

What does this PR do?
Provides the option to override the liveness and readiness probe HTTP GET paths.
Motivation
I wrote a Traefik Plugin that adds some functionality to be used by the readiness probe only. In order to use it, the readiness probe path needs to be different from the liveness probe path. Here is the plugin: https://github.com/livekit/traefik-readiness-plugin
More

[x] Yes, I updated the tests accordingly
[x] Yes, I ran make test and all the tests passed

Hello @matkam,
Thanks for your interest in Traefik. You wrote a plugin, that's cool.
FTM, there is no plugin supported with this chart.
At first glance, I'm not sure to understand why would someone consider Traefik not ready when a threshold on CPU is reached. It's usually managed with HPA or VPA. As explained in the README:

To keep this HelmChart as generic as possible we tend to avoid integrating any third party solutions nor any specific use cases.

So => Would you please detail the use case of this plugin ? (and so why we should consider include it ?)

Hi @mloiseleur!
Nice to see a familiar face. You also reviewed my PR in https://github.com/kubernetes-sigs/external-dns.

I'm not sure to understand why would someone consider Traefik not ready when a threshold on CPU is reached. It's usually managed with HPA or VPA.

This is true under the normal circumstance of short lived HTTP requests, but does not hold true with long running WebSocket connections (multiple hours). The problem is that even after scaling up Traefik Pods, older saturated Pods will continue to receive new connections, and can become overloaded. To get around this problem, we are currently over provisioning Traefik. But if Traefik gets taken out of rotation once it has reached a usage threshold, over provisioning wouldn't be necessary.  Weighted load balancing would probably work better, but not all cloud providers have load balancers that can do that.

There is an open issue on readiness : https://github.com/traefik/traefik/issues/10458
You can also tune your probe using api/rawdata content.

There is an open issue on readiness : traefik/traefik#10458
You can also tune your probe using api/rawdata content.

thanks for the tip @mloiseleur! I've added a rawdata check to my readiness plugin: https://github.com/livekit/traefik-readiness-plugin/commit/af7e8f86e8867c7c84db25fc7431f5a29e505ef7
kind of amazing this isn't built in already.


Information on how to scan QR Code (EXPOSUREAPP-2581)

New PR with 1.6 as base
Description
Added a screen before the QR Code Scan screen in submission flow that explains how the QR Code scanning works.

Created layout XML file
Created Fragment
Created ViewModel
Added required string files
Updated navigation.
Added required illustrations

Test
Begin the submission process, select QR option - you should be presented with the new screen. When next is clicked, you will be taken to the original QR scan screen. Back functionality should work appropriately.

UA reviewed

@d4rken reviewed the request changes and is satisfied.


LoadError: Unable to autoload constant VERSION

I am using Knock, Paperclip and Aws S3 in one of my app. There is some weird conflict happening when i am trying to use S3 as paperclip storage. Getting the following error
LoadError: Unable to autoload constant VERSION, expected RVM_PATH/gems/knock-2.1.1/lib/knock/version.rb to define it
The same error is coming if you fire up the rails console and try to access VERSION

Seems this is nothing to do with Knock but Paperclip & AWS issue.
Description and solutions are discussed here:  https://github.com/thoughtbot/paperclip/issues/2484


WEB_SERVER_* env vars used by web/entity search apps are missing

SENZING_WEB_SERVER_ADMIN_AUTH_MODE
SENZING_WEB_SERVER_ADMIN_AUTH_PATH
SENZING_WEB_SERVER_API_PATH
SENZING_WEB_SERVER_PORT
SENZING_WEB_SERVER_URL

@antaenc To be clear,  where are they missing?
Is it in https://github.com/Senzing/knowledge-base/blob/main/lists/environment-variables.md?


updating to version 1.8.10 causes new error in CommonsChunkPlugin

Version 1.8.9 succeeds but when updating to 1.8.10 I get an error:
ERROR in CommonsChunkPlugin: While running in normal mode it's not allowed to use a non-entry chunk

The bundle js is not output.
My build output from deploying to Heroku (this error is on line 514):
https://gist.github.com/bdefore/838bbe5be65dcb114014

This indicates a wrong configuration.

But even Explicit vendor chunk example from documentation is broken by this change
So, it's indeed either a bug or at least backward-incompatible change.
Anyway, what can we use as a workaround?
Adding children: true seems to work, but is it correct solution?

It's not broken:
I added it to the examples in the repo:
https://github.com/webpack/webpack/commit/300ee8db48da55ce20e4a06629330f97d33005e4
https://github.com/webpack/webpack/tree/master/examples/explicit-vendor-chunk

Previously I was sending two parameters to CommonsChunkPlugin. Has this changed to one object parameter? If so, what is the API for that? I tried changing it to specifying a single object with name and minChunks properties but the error still occurs.
Here's my full configuration object, in case it helps: https://gist.github.com/bdefore/ccc0fe63309e9ff8d11d

yep. You configuration is wrong. Either remove the CommonsChunkPlugin for app or use children: true for it.

It's not broken.
I added it to the examples in the repo:

You're right, sorry.
But it becomes broken if I add second vendor entry like this:
var path = require("path");
var CommonsChunkPlugin = require("../../lib/optimize/CommonsChunkPlugin");
module.exports = {
	entry: {
		vendor: "./vendor",
		vendor1: "./vendor1",
		pageA: "./pageA",
		pageB: "./pageB",
		pageC: "./pageC"
	},
	output: {
		path: path.join(__dirname, "js"),
		filename: "[name].js"
	},
	plugins: [
		new CommonsChunkPlugin({
			name: "vendor",
			minChunks: Infinity
		}),
		new CommonsChunkPlugin({
			name: "vendor1",
			minChunks: Infinity
		})
	]
}
// says:
// ERROR in CommonsChunkPlugin: While running in normal mode it's not allowed to use a non-entry chunk

What I'm trying to achieve is to split my vendor libs into several entries for caching reasons (some of them are getting changed much more often than others). Also different sets of my pages may need different vendor libs.
How to do it right?
PS It looks like using children: true don't do what I want.
PPS
		new CommonsChunkPlugin({
			file: "vendor.js",
			chunks: ["vendor"]
		}),
		new CommonsChunkPlugin({
			file: "vendor1.js",
			chunks: ["vendor1"]
		})

Doesn't work as well - it extracts webpack bootstrap from both vendor files to additional chunk, making them unusable.
PPPS Tried to start from Multiple Commons Chunks example, which uses old-style syntax. It gave me other results, but not what I need anyway.

It's not broken.

You're right, sorry.
I had a modified version with several CommonsChunkPlugins.

I added a shorthand syntax for multiple CommonsChunkPlugins with 1.8.11:
https://github.com/webpack/webpack/tree/master/examples/two-explicit-vendor-chunks

I added a shorthand syntax for multiple CommonsChunkPlugins with 1.8.11

Great, thanks! )

So I'm getting the same error, but everything still works. My goals are somewhat different - I'm trying to get the common bundle into a directory for js in the build folder. I have the following config:
entry: {
  article_editor: './src/article-editor/app.js'
 },
output: {
  path: path.join(__dirname, "./dist/article-editor"),
  filename: 'js/[name].bundle.js',
 ...
},
plugins: [
    new webpack.optimize.CommonsChunkPlugin('js/common.bundle.js'),
  ...
],

I originally had the JS directory in the output path, but I needed to move that in order to get my sass to compile in a folder other than /js. So I moved it to the output filename and it worked for the page-specific JS, but not the common js. So I added it in CommonsChunkPlugin('js/common.bundle.js') and this works, but throws the error mentioned here: 'ERROR in CommonsChunkPlugin: While running in normal mode it's not allowed to use a non-entry chunk (js/common.bundle.js)'
Seems pretty straightforward, but I'm not sure what the solution is (should I just change modes?) or if I'm just doing this terribly wrong. Pretty new to webpack, sincere apologies if I've missed an obvious answer somewhere.

@nerdgrass Could I see the remaining plugins? Your use case is equal to the multiple-entry-points example, which works fine.
The error is thrown when the commons chunks already exists and is not an entry chunk, i. e. when it's an additional chunk or already affected by another CommonsChunkPlugin.

here's my config:
var configuration = {
  entry: {
    article_editor: './src/article-editor/app.js'
  },
  module: {
    loaders: [
      {
        test: /\.js$/,
        exclude: /node_modules/,
        loaders: ["babel-loader"],
      },
      {
        test: /\.scss$/,
        loader: ExtractTextPlugin.extract("style-loader", sassLoaders.join("!")),
      },
    ]
  },
  output: {
    path: path.join(__dirname, "./dist/article-editor"),
    publicPath: 'dist/article-editor/index.html',
    filename: 'js/[name].bundle.js',
    chunkFilename: '[id].chunk.js'
  },
  plugins: [
    new webpack.optimize.CommonsChunkPlugin('js/common.bundle.js'),
    new ExtractTextPlugin("./css/[name].css"),
  ],
  resolve: {
    root: path.join(__dirname, 'js'),
    extensions: ["", ".js", ".scss"],
    modulesDirectories: ["src", "node_modules"],
  }
};

Eventually, there will be multiple entry points (multiple client-side apps in the same repo with shared components), for now its just article-editor. The error goes away when I remove the js/ from js/common.bundle.js, weirdly enough.
And thanks for the example, that's actually what I based this on. The documentation for webpack is pretty extensive and useful :)

See new extract-text-webpack-plugin version.

I'm getting this too, and can't figure out what the problem is.
config:
entry: {
    app: './app/main.js',
    vendor: './app/vendor/vendor.js',
    editor: './app/vendor/editor.js',
    home: './app/vendor/homepage.js'
},
...
plugins: [
    new webpack.optimize.CommonsChunkPlugin( 'common.bundle.js' ),

Error:
ERROR in CommonsChunkPlugin: While running in normal mode it's not allowed to use a non-entry chunk (common.bundle.js)

I require('./common'); in all of the entry point files listed above. I still get the error.

Ok, that error was because I was using an older version of extract-text-webpack-plugin, 0.6.0. Upgrading to 0.8.2 "fixed" it. However, my common bundle only has the webpack boilerplate in it. This process is very hard to debug because it's not obvious what causes what problems or what behavior. The undocumented syntax for CommonsChunkPlugin doesn't help.

Ah, so this is entirely undocumented and confusing, it's because minChunks default is the total number of entry points you have, so if you don't require files in every entry point file, it won't get extracted to to the commons chunk. I don't see any reason why the default shouldn't be 2?

@sokra  Hi, I'm trying to combine the two-explicit-vendor-trunk example with code splitting. But cannot make it to work.
I have two pages:

/ : needs only vendor.js
/d3 : needs both vendor.js and d3.js

The goal is: when visiting /, only app.js and vendor.js are loaded. When the user navigates to /d3, d3.js is loaded as needed. Here is how I try:
webpack config:
entry: {
  vendor: [
    'react',
    'react-dom',
    'react-router',
    ...
  ],
  d3: [
    'd3',
  ],
  app: [
      './src/js/index',
  ],
},
plugins: [
  new webpack.optimize.CommonsChunkPlugin({
    names: ['d3', 'vendor'],
    minChunks: Infinity,
  }),
  ...
],

Router.jsx (I'm using react-router):
export default (
  <Router history={createBrowserHistory()}>
      ...
      <Route path="d3" getComponent={(location, callback) => {
        require.ensure([], (require) => {
          callback(null, require('./pages/D3'));
        });
      }} />
      ...
  </Router>
);

So how can I make it to work? I need this configuration because d3 is too large, and I need to do long-term cache on both vendor.js and d3.js.

As I've learned the hard way, order matters when defining your vendor chunks in the CommonsChunkPlugin. for example...
{
   entry: {
		"vendor1": [
			"jquery",
			"lodash"
		],
		"vendor2" : [
			"angular"
		],
		"bundle": "./src/app.js"
	},
  plugins: [
		new webpack.optimize.CommonsChunkPlugin({
			names: ["vendor2", "vendor1"],
                        minChunks: Infinity
		}),
		new HtmlWebpackPlugin({
			title: 'My App Title',
			template: 'index.html',
			favicon: './assets/images/favicon.ico',
			inject: 'body',
			chunks: [
				"vendor1",
				"vendor2",
				"bundle"
			]
		})
	]
}

This configuration works. Notice how in the new webpack.optimize.CommonsChunkPlugin invocation the vendor chunks are in reverse order. If i swap them I get errors in my resulting index.html
Seems like a bug in the CommonsChunkPlugin internals. Thoughts?

@antoniomercado Thank you so much for sharing this, I was splitting my head over this. Have had any new insights into managing this properly?

@bbottema I ended up scrapping the idea of splitting up the vendor chunks because I had a webpack.config.production.js which had slight variations of this config and even with this "trick" i couldn't get it to sort properly, so i said screw it! Currently I just have one vendor file that I'm packing everything into. I will for sure need to revisit this sometime in the near future because the vendor file is becoming quite large. If anyone else has any insights into this issue or if anyone can post examples of their configs with split up vendor files that would be greatly appreciated.

@antoniomercado Thanks a million!!!
Reversing the order of the chunks will do the trick.

Having to reverse the order makes this very confusing...

I have 4 common trunks output. Writing into the single one CommonChunkPlugin instance is the right solution. Yes, the order matters, just like loader chain, the order should be reversed:
  new webpack.optimize.CommonsChunkPlugin({
    names: ['libScoped', 'lib', 'polyfills', 'vendor'],
    minChunks: Infinity,
    filename: `../${LIB_DEST}/[name].js?[hash]`
  }),

@antoniomercado thanks so much!! Have been searching for a solution for hours...
Don't really understand why Webpack doesn't mention this in the docs or something, because it's not logical to have to reverse the order of the chunks to get the result that you want.
Anyhow.. got finally working now

@antoniomercado  thanks a lot!!!
In case anyone met the same situation. Don't forget to set output config like this.
output: {
	path: './build/',
	filename: '[name].js',
},

According to the docs. output.filename.

multiple entries
If your configuration creates more than a single “chunk” (as with multiple entry points or when using plugins like CommonsChunkPlugin), you should use substitutions below to ensure that each file has a unique name.

So through a lot of trial and error, I've ended up with a rather monolithic config for multiple common chunks with cache busting (truncated where possible…):
import webpack from 'webpack';
import AssetsPlugin from 'assets-webpack-plugin';
import ChunkManifestPlugin from 'chunk-manifest-webpack-plugin';
import WebpackMd5Hash from 'webpack-md5-hash';

export default {
  entry: {
    polyfills: ['babel-polyfill'],
    react: ['react', 'react-dom'],
    page: '(insert path here…)',
    // insert more pages here…
  },
  plugins: [
    // Generates path-based module ids instead of fluctuating integer ones
    new webpack.NamedModulesPlugin(),
    // Without this, the entry chunk hash changes due to the changing manifest
    new WebpackMd5Hash(),
    // Extracts the manifest from the entry chunk
    new ChunkManifestPlugin(),
    // Moves every node module into a vendor chunk,
    // And caches the names of their original chunks for further chicanery
    new webpack.optimize.CommonsChunkPlugin({
      name: 'vendor',
      minChunks: (module) => {
        const { userRequest, chunks } = module;
        const check = typeof userRequest === 'string' &&
          userRequest.indexOf('/node_modules/') >= 0;
        // eslint-disable-next-line no-param-reassign
        module.originalChunkNames = chunks.map(({ name }) => name);
        return check;
      },
    }),
    // Moves the react and react-dom modules (and their deps)
    // back into their original chunk
    new webpack.optimize.CommonsChunkPlugin({
      name: 'react',
      chunks: ['vendor'],
      minChunks: ({ originalChunkNames }) => originalChunkNames.includes('react'),
    }),
    // Moves babel-polyfill (and its deps) back into their original chunk
    new webpack.optimize.CommonsChunkPlugin({
      name: 'polyfills',
      chunks: ['vendor', 'react'],
      minChunks: ({ originalChunkNames }) => originalChunkNames.includes('polyfills'),
    }),
    // Moves the entry (webpack bootstrap) out into its own dedicated chunk.
    // This could be rolled into polyfills but at this point it created
    // less headaches during the debugging of this whole thing…
    new webpack.optimize.CommonsChunkPlugin({
      name: 'entry',
      chunks: ['vendor', 'react', 'polyfills'],
      minChunks: Infinity,
    }),
    // Creates a JSON map of the chunk names to their built files
    new AssetsPlugin(),
    // May be redundant now that NamedModulesPlugin is used…?
    new webpack.optimize.OccurenceOrderPlugin(),
  ],
};

Which enables loading of the manifest (via inline script) followed by entry, polyfills, react, & vendor… before finally loading one of the desired page entry points. Each of these 4 chunks will only change their hash when their contents change. (e.g. only vendor and page will get a new hash if a new vendor module is added to page, and react only changes if the library does.) Each one depends on the others higher-up the chain, so as mentioned earlier, some of them could be rolled together for less complexity (entry & polyfills).
Some observations after going through all this:

The calculation of which chunk became the special "entry" chunk that has the webpack bootstrap was extremely confusing. It took me a long while to grasp that there was some sort of baton-passing or re-thievery of the bootstrap going on between multiple instances of the CommonsChunkPlugin plugin. It wasn't just the order, they also had to be chained together to move the bootstrap down the chain to the last one…
The bug/side effect of the entry chunk's hash changing even after the manifest was removed added another level of headache on top of trying to understand the movements of the bootstrap. Hence why it ended up siphoned off into it's own entry chunk.
It was only through trial and error I managed to get a bit of a handle on how the chunks option works, and the children option is still a mystery to me. The chunks documentation mentions "The chunk must be a child of the commons chunk", and children says "If true all children of the commons chunk are selected", but what exactly is a parent/child relationship in this plugin? And how do these options relate to "While running in normal mode it's not allowed to use a non-entry chunk"? "Normal mode" (or "mode" in general) is not mentioned outside of the error message.
It feels ugly that I've had to do a lot of custom magic via minChunks and module mutation just to be able to create multiple common chunks. It's easy to make examples of multiple static common chunks(via names), or one dynamic common chunk (via a minChunks function), but it feels very hard trying to combine those two configs together in a way that makes sense.
Due to the scale of debugging, I still haven't accounted yet for creating a common chunk for shared internal code…
babel-polyfill modifies globals, so has a special need to be loaded early before other libraries, but there isn't much documentation on how to load special cases like this for non-trivial entry points.

(Sorry for the barrage of words…)

Going to close this since the original question was answered and this became a place to discuss semi-related issues. If you still have an issue, please file a new issue so we can actually do something see it and do something about it. For questions, please refer to StackOverflow.


Remove indication that a failed scheduling attempt removes the Pod

/sig scheduling
I don't know if this was ever true, but today kube-scheduler doesn't remove Pods in this case.

/assign @sftim

/assign @sftim

FYI @Huang-Wei

FYI @Huang-Wei

/lgtm

Page preview:
https://deploy-preview-26097--kubernetes-io-master-staging.netlify.app/docs/concepts/workloads/pods/pod-lifecycle/#pod-lifetime

/approve


[Highlight Events] Because of the super weird event propagation, it is unclear how event.eventPhase works with the events

If event.target stays as the element, and when currentTarget == target, that is supposed to be AT_TARGET phase.
But since propagation goes through Highlight, the examples seem to hint that eventPhase starts to work in some new way and it would probably break sites.

@dandclark FYI..


llmq: Some fixes/improvements

@UdjinM6 See the last 5 commits, i changed/dropped some stuff. Quorums/Commitment caches are now unordered_lru_cache and scanQuorumsCache cache size is fixed by just calculating it based on the params.

@UdjinM6 See the last 5 commits, i changed/dropped some stuff. Quorums/Commitment caches are now unordered_lru_cache and scanQuorumsCache cache size is fixed by just calculating it based on the params.

Okay so, i pushed some more commits to refactor ScanQuorums a bit because IMO it was really hard to read/understand its full logic before (and i hope i didn't break any of its logic haha). Also it seems like it didn't reuse existing cache if more than cache size was requested. So let's say we had 10 quorums in cache and you requested 11 it read all 11 from DB instead of 1 (0603e98)
This recent commits also changes the way the scanQuorumCache works from just one unordered_lru_cache to one unordered_lru_cache for each LLMQ type like with the normal quorums and commitments caches (
4f2a804). The rest should be obvious.

Okay so, i pushed some more commits to refactor ScanQuorums a bit because IMO it was really hard to read/understand its full logic before (and i hope i didn't break any of its logic haha). Also it seems like it didn't reuse existing cache if more than cache size was requested. So let's say we had 10 quorums in cache and you requested 11 it read all 11 from DB instead of 1 (0603e98)
This recent commits also changes the way the scanQuorumCache works from just one unordered_lru_cache to one unordered_lru_cache for each LLMQ type like with the normal quorums and commitments caches (
4f2a804). The rest should be obvious.

LGTM so far, waiting for the "inactive quorum" issue to be discussed (and fixed if needed).

LGTM so far, waiting for the "inactive quorum" issue to be discussed (and fixed if needed).

Okay so in the meeting today it turned out that the reason for querying old quorums is mainly to validate IS locks with verifyislock and chainlocks with quorum verify. But it appears that this only happens frequently during sync of platform because they store the the IS/CL in the platform blocks and they need to validate them on sync. When it's synced it can still happen but unlikely/not often and thats only if someone submits an asset lock older than our singing window to platform. I can't tell you much more details here so if you have more questions i guess @antouhou or @shumkov could leave you an answer.
Soo, from my understanding it wouldn't help a lot then here to have a bigger cache, right? We could maybe still just add a command line which allows to increase the cache sizes...not sure if it makes sense, but we could haha..
And i guess someone (i would but i don't know when :P) should in any case do some research and testing around all this to see how resource consuming fun like this is also given the fact that ivan said they are sometimes at the limit of the RPC queue during sync 🤔

Okay so in the meeting today it turned out that the reason for querying old quorums is mainly to validate IS locks with verifyislock and chainlocks with quorum verify. But it appears that this only happens frequently during sync of platform because they store the the IS/CL in the platform blocks and they need to validate them on sync. When it's synced it can still happen but unlikely/not often and thats only if someone submits an asset lock older than our singing window to platform. I can't tell you much more details here so if you have more questions i guess @antouhou or @shumkov could leave you an answer.
Soo, from my understanding it wouldn't help a lot then here to have a bigger cache, right? We could maybe still just add a command line which allows to increase the cache sizes...not sure if it makes sense, but we could haha..
And i guess someone (i would but i don't know when :P) should in any case do some research and testing around all this to see how resource consuming fun like this is also given the fact that ivan said they are sometimes at the limit of the RPC queue during sync 🤔


New login to Instagram from Unknown browser

Every time I made a login, I received this message

Has anyone already encountered this problem? Is there a way to solve it?

seems it has become impossible to login now, facing the exact issue as @alajon is facing.
Anybody has any solution? Specially, Raiym, you have to say anything, sir?


Downgrade eslint

https://github.com/eslint/eslint/issues/3440

Seems to be fixed already, maybe just upgrade will work?


gdal: 3.7.0 -> 3.7.1 and add package tests

Description of changes

Update GDAL to the latest bug fix release and add package tests.
List of GDAL changes:
https://github.com/OSGeo/gdal/blob/v3.7.1/NEWS.md
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

What's the reason for the backport label?

Also it's not evaluating.

Also it's not evaluating.

Thanks, should be fixed now.

What's the reason for the backport label?

It is bug fix release and it meets backporting criteria.

What's the reason for the backport label?

It is bug fix release and it meets backporting criteria.

as far as I can tell 3.7.0 was not backported, so backporting 3.7.1 only makes sense if 3.7.0 is already in 23.05

What's the reason for the backport label?

It is bug fix release and it meets backporting criteria.

as far as I can tell 3.7.0 was not backported, so backporting 3.7.1 only makes sense if 3.7.0 is already in 23.05

Unfortunately you are right. Thank you. I apologize @risicle . I somehow forgot that 3.7 haven't made it to 23.05.

Thanks everybody involved in review, good comments and merging !


Initialize the react project

The first step is to setup all main files for the project

I cloned the repository and after that ran "npx create-react-app student-life" and tested if the app started.


Created OutlinedLayout

TO-DO
Table of Contents

[x] Identify the component the PR relates to in brackets in the title.
OutlinedLayout
OutlinedHolder

Contributing
has more information and tips for a great pull request.

@zeobot set lc to pass


[stdlib] use @_transparent in an appropriate spot

This function was inconsistently @inlinable, as opposed to its peers which are @_transparent. As a consequence, the detailed diagnostics surrounding Optional with == and != differ strangely based on minute details.

Resolves rdar://46444561

@swift-ci please test

@swift-ci please benchmark

Performance (x86_64): -O

Regression
OLD
NEW
DELTA
RATIO

LessSubstringSubstring
23
29
+26.1%
0.79x

EqualSubstringSubstringGenericEquatable
23
29
+26.1%
0.79x (?)

EqualSubstringString
23
29
+26.1%
0.79x

LessSubstringSubstringGenericComparable
23
29
+26.1%
0.79x (?)

EqualSubstringSubstring
24
30
+25.0%
0.80x (?)

EqualStringSubstring
24
30
+25.0%
0.80x

FlattenListFlatMap
3826
4417
+15.4%
0.87x (?)

DataCreateEmptyArray
1350
1550
+14.8%
0.87x (?)

StringComparison_longSharedPrefix
293
331
+13.0%
0.89x (?)

FindString.Loop1.Substring
272
301
+10.7%
0.90x (?)

SortStringsUnicode
1945
2105
+8.2%
0.92x (?)

Improvement
OLD
NEW
DELTA
RATIO

Dictionary4
217
153
-29.5%
1.42x

UTF8Decode_InitFromCustom_contiguous
159
125
-21.4%
1.27x (?)

UTF8Decode_InitDecoding
159
127
-20.1%
1.25x (?)

Dictionary4OfObjects
259
207
-20.1%
1.25x (?)

UTF8Decode_InitFromCustom_noncontiguous
276
239
-13.4%
1.15x (?)

NormalizedIterator_emoji
396
360
-9.1%
1.10x (?)

DistinctClassFieldAccesses
250
229
-8.4%
1.09x (?)

Set.isStrictSubset.Box0
84
77
-8.3%
1.09x (?)

ArrayInClass
1200
1110
-7.5%
1.08x (?)

UnicodeStringFromCodable
202
187
-7.4%
1.08x (?)

String.replaceSubrange.String.Small
41
38
-7.3%
1.08x (?)

RemoveWhereQuadraticString
213
198
-7.0%
1.08x (?)

DictionaryKeysContainsNative
15
14
-6.7%
1.07x (?)

Code size: -O
Performance (x86_64): -Osize

Regression
OLD
NEW
DELTA
RATIO

EqualSubstringSubstring
23
30
+30.4%
0.77x

LessSubstringSubstring
23
29
+26.1%
0.79x (?)

EqualStringSubstring
23
29
+26.1%
0.79x

EqualSubstringSubstringGenericEquatable
23
29
+26.1%
0.79x (?)

EqualSubstringString
23
29
+26.1%
0.79x (?)

LessSubstringSubstringGenericComparable
23
29
+26.1%
0.79x (?)

DataCreateEmptyArray
1400
1600
+14.3%
0.88x (?)

FlattenListFlatMap
3663
4181
+14.1%
0.88x (?)

FindString.Loop1.Substring
264
300
+13.6%
0.88x (?)

StringAdder
221
250
+13.1%
0.88x (?)

StringComparison_longSharedPrefix
294
332
+12.9%
0.89x (?)

StringWalk
1360
1520
+11.8%
0.89x (?)

StringHasSuffixAscii
1350
1480
+9.6%
0.91x (?)

StringInterpolationSmall
1080
1180
+9.3%
0.92x (?)

SortStringsUnicode
1970
2145
+8.9%
0.92x (?)

Improvement
OLD
NEW
DELTA
RATIO

UTF8Decode_InitDecoding
159
125
-21.4%
1.27x (?)

UTF8Decode_InitFromCustom_contiguous
160
127
-20.6%
1.26x (?)

RemoveWhereFilterInts
28
23
-17.9%
1.22x (?)

UTF8Decode_InitFromCustom_noncontiguous
278
243
-12.6%
1.14x (?)

Array2D
4640
4176
-10.0%
1.11x (?)

Set.isDisjoint.Empty.Box
90
82
-8.9%
1.10x (?)

ArrayInClass
1240
1130
-8.9%
1.10x (?)

Set.isSuperset.Seq.Int0
125
114
-8.8%
1.10x (?)

Set.isStrictSubset.Int0
82
75
-8.5%
1.09x (?)

Set.isStrictSubset.Box0
84
77
-8.3%
1.09x (?)

Set.isDisjoint.Seq.Empty.Box
90
83
-7.8%
1.08x (?)

String.replaceSubrange.String.Small
41
38
-7.3%
1.08x (?)

Code size: -Osize
Performance (x86_64): -Onone

Regression
OLD
NEW
DELTA
RATIO

EqualSubstringString
28
35
+25.0%
0.80x (?)

EqualSubstringSubstringGenericEquatable
26
32
+23.1%
0.81x (?)

LessSubstringSubstringGenericComparable
26
32
+23.1%
0.81x

LessSubstringSubstring
28
34
+21.4%
0.82x (?)

EqualSubstringSubstring
28
34
+21.4%
0.82x (?)

EqualStringSubstring
29
35
+20.7%
0.83x (?)

DictionaryCompactMapValuesOfNilValue
10050
10850
+8.0%
0.93x (?)

Improvement
OLD
NEW
DELTA
RATIO

SubstringTrimmingASCIIWhitespace
659
469
-28.8%
1.41x

UTF8Decode_InitDecoding
169
132
-21.9%
1.28x (?)

UTF8Decode_InitFromCustom_contiguous
169
133
-21.3%
1.27x

StringMatch
28900
23400
-19.0%
1.24x (?)

StackPromo
34800
29100
-16.4%
1.20x

FindString.Rec3.Substring
567
482
-15.0%
1.18x (?)

FindString.Rec3.String
580
498
-14.1%
1.16x (?)

String.replaceSubrange.String.Small
44
40
-9.1%
1.10x (?)

ObjectiveCBridgeStubDateAccess
2168
1975
-8.9%
1.10x (?)

ObjectiveCBridgeStubNSDateRefAccess
2207
2028
-8.1%
1.09x (?)

Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac mini
  Model Identifier: Macmini8,1
  Processor Name: 6-Core Intel Core i7
  Processor Speed: 3.2 GHz
  Number of Processors: 1
  Total Number of Cores: 6
  L2 Cache (per Core): 256 KB
  L3 Cache: 12 MB
  Memory: 32 GB

FYI https://github.com/apple/swift/pull/36309
I don't remember the details any more, but it's potentially worth looking at map and flatMap as well.

This change lgtm!
The benchmark results are probably just noise.


Hardware BoM Data Controller Methods Not Consistently Defined

Expected Behavior
Classes should have identical method names and definitions (e.g., same number/name of parameters).
Actual Behavior
The Hardware BoM data controller has the following methods:
1. request_calculate(self, node_id)
2. request_calculate_all(self)
3. request_delete(self, hardware_id)
4. request_insert(self, revision_id=0, parent_id=0, part=0)
5. request_last_id(self)
6. request_seleect(self, node_id, table)
7. request_update(self, hardware_id)
8. request_update_all(self)

These should be re-defined as:
1. request_calculate(self, **kwargs)
2. request_calculate_all(self, **kwargs)
3. request_delete(self, node_id)
4. request_insert(self, **kwargs)
5. request_last_id(self, **kwargs)
6. request_select(self, node_id, **kwargs)
7. request_update(self, node_id)
8. request_update_all(self, **kwargs)

Steps to Reproduce the Problem
Not applicable for requirements/code quality errors.
Proposed Solution

Rename the Hardware BoM methods as outlined above.
Update the Hardware BoM data controller methods to extract arguments from **kwargs.
Update calls to the Hardware BoM methods to pass arguments as **kwargs.
Update the Hardware BoM data controller tests to reflect the new method definitions.

Operating Environment
Not needed for requirements/code quality errors.

Fixed in commit 34b212a9753a65d405b6ee050df0ab4d1209c73b


Update README.md

Description
I removed outdated information, focused more on developing and building with our Docker image, moved the project layout section further down and put the example CMakeUserPresets.json as well as the directory structure into a collapsible section.
Fixes #85

I cancelled the jobs because only the README file changed.


xmlrpc source

Error: Unable to download xmlrpc source files from https://svn.code.sf.net/p/xmlrpc-c/code/stable
This is most likely a network error, if your network is working, then it is likely a temporary issue with the relevant file server
Run 'bash rtinst.sh -l' for detailed output to rtinst.log
Once issue is resolved the script can be run again to complete installation
The source is not working  since  yesterday.

yes the xmlrpc-c repository is currently down.

It is bask up again. If you run the script now it should work.


Fix run_doc for led plugin

Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
What does the code in this PR do / what does it improve?
I realized that there are run_ids with more than one comment, which will cause an error as it is right now. The first comment is always from the automatic script.
I have fixed that and I have added a check for verifying that the first comments has the right format.
Can you briefly describe how it works?
Can you give a minimal working example (or illustrate with a figure)?
Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

coverage: 90.206% (+0.01%) from 90.196%
when pulling fe763d48f668f4e58eefca7251804126ee11918f on fix_run_doc_led
into 79af57dfb3024f8bedde09c87bc6dd1b1d8251f2 on master.

Here, @dachengx suggested changing the config in the test instead of adding a specific run_id for the led_calibration. Unfortunately, I don't know where to change it. If you prefer me to change the config, I need some help

Ah, of course, the data are not stored so the test are still failing

@dachengx @minzhong98 @MerzJohannes @yuema137 shall we proceed here?

Thank you Dacheng <3


[MNOE-611] Inactive Apps should not be exposed on API

@x4d3 if an app has been launched by a user and is then made inactive, does it still display on the user front-end?

Yes, it should as it's coming from the AppInstance.


In Memory Feature Sets in ArcGIS Pro

Several tools in this repository use In Memory Feature Sets to input parameters, like in the picture below.

While this works great in ArcMap, currently this capability has not been implemented in ArcGIS Pro.
So, my question is: Can/Should we remove the default in_memory\feature_set from these tools in the ArcGIS Pro toolboxes?
After talking with @mfunk about the Pro team's decision making process in regards to implementing this capability, I think it best that we come up with a standard workaround. It seems to me there are three options:

Leave it blank / Have the user add their own data - Definitely the easiest of the three, but goes against the "spirit" of having the in memory feature sets at all. Plus, even if we add another default value here, they can always choose to Browse...
Default to another layer / Have the user create that data in a previous step - The step I'm inclined to choose. Create a step that directs the user to digitize the feature for use in a subsequent step (i.e. "Sketch the area of operations. This will be used later to ____."). Have the geoprocessing tool default to that layer, and it still has the option for the user to browse to and add their own data.
Have the user use Map Notes - This is what was suggested to me as an option, but it is unnecessarily difficult and defeats the purpose in my opinion. Map notes, when it comes to being used as a parameter in a geoprocessing tool, are just like any other feature class, and therefore we might as well use something specific.

I vote for option 2. We can use Map Notes as the source for 2 if it helps but either way, the choice should be deliberate.

Let me talk with a few people on the GP team and see what they suggest. I'd rather not decide now, wait a day or two until I can talk with them, instead of having to rebuild the tools inputs twice.

That's what I was hoping you'd say!

Some changes to Pro 1.1 will address this, not directly, but in a pleasantly unexpected way. Can't really say more now. Sorry. :)


typing error in combination with pylance for factory=list

Hello I'm very excited that attrs now allows to show constructor parameters when used with pylance.
I'm using attrs-21.2.0
However I have a problem when using a class with factory, the factory appears to be not recognized as default value. For example:
@attr.s(auto_attribs=True)
class A:
    my_i: int = 42
    my_list: List[str] = attr.ib(factory=list)

If I try to instantiate this class with a = A() I get error from pylance: 'Argument missing for parameter "my_list"'.
And for the class definition I get the error "Fields without default values cannot appear after fields with default values".
I noticed it works somehow when I use my_list: List[str] = attr.Factory(list) but with this the type is set to Factory and I get problems if I want to use the list a.my_list.append('fu') will then lead to "Instance of 'Factory' has no 'append' member".

Sadly not, maybe pylance could do? Seems like that there is some trouble in finding a aggreement which project will handle default_factory (https://github.com/python-attrs/attrs/issues/795#issuecomment-843409923). Maybe supporting attr.Factory(list) in pylance is an option?

Great news, they added factory!


License: Use the HTTPS link to the Apache 2.0 license in the template

As it is done in the source files.

This needs to update a bunch of tests that use the LICENSE file's hash, though.

Thanks I'll fix them.


Fix homepage icon displacement

Before (just hovering my cursor in place):
https://user-images.githubusercontent.com/22732095/152656536-c3c2f636-f53a-4029-8ab5-bbc979c52a73.mp4
After:

Thanks for catching this! LGTM.
Just as a heads up, I think I might revamp this entire hero section. The layout (and especially the homepage) is far from finished.


Support ESLint 8.x

ESLint v8.0.0 is released 🎉
It would be awesome to have official ESLint 8 support. 👊
I'm happy to help where I can of course 🙂

[ ] https://github.com/import-js/eslint-plugin-import/pull/2191

We won't officially support anything until v8 is actually released (betas don't count any more than rc's do).
See #2191; there was no need for this issue.

ESLint v8 has been released stable. :tada:
It's time to support eslint@8.
See: https://eslint.org/blog/2021/10/eslint-v8.0.0-released

Hi guys,
Is the error
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/util/glob-util' is not defined by "exports"

something you're already aware of? I'm getting this everywhere in my builds, preventing the upgrade to eslint 8.
Thx!
David

The release is blocked on https://help.appveyor.com/discussions/problems/30534-certificate-verification-failed at the moment.


Move language selection under My account / Settings

Also, add a bit more space between the links.

Change the language dropdown to radio group

Dark/light mode toggle hover effect offset

If I have a small browser window, I cannot see the language selection as the settings popup is not scrollable to high enough (centered somehow). However, I can scroll down, so scrolling is working:


Could not find build in build/Release with prebuild on AppVeyor

C:\projects\node-ext2fs\build\Release\bindings.node : warning LNK4088: image being generated due to /FORCE option; image may not run [C:\projects\node-ext2fs\build\bindings.vcxproj]
  bindings.vcxproj -> C:\projects\node-ext2fs\build\Release\\bindings.node
  bindings.vcxproj -> C:\projects\node-ext2fs\build\Release\bindings.pdb (Full PDB)
prebuild ERR! build Error: Could not find build in build/Release
prebuild ERR! build     at Object.exports.noBuild (C:\projects\node-ext2fs\node_modules\prebuild\error.js:2:10)
prebuild ERR! build     at C:\projects\node-ext2fs\node_modules\prebuild\collect-artifacts.js:19:23
prebuild ERR! build     at FSReqWrap.oncomplete (fs.js:135:15)

Fixed in https://github.com/resin-io/node-ext2fs/pull/40


redisClusterAsyncDisconnect does not frees redisClusterAsyncContext

I am using the hiredis vip async api to connect to a 3 node cluster redis and my program is leaking
some memory.
It looks like the redisClusterAsyncContext structure is not being freed after calling the redisClusterAsyncDisconnect on it.
I have just made a simple example in order to reproduce the memory leak.
#include <iostream>
#include <sstream>
#include <event.h>
#include <hiredis-vip/adapters/libevent.h>
#include <hiredis-vip/hircluster.h>

void connect_cluster_cb(const redisAsyncContext *c, int status){
    std::cout << "Connected to redis cluster node " << c->c.tcp.host << ":" << c->c.tcp.port << std::endl;
}

void disconnect_cluster_cb(const redisAsyncContext *c, int status){
    if( status != REDIS_OK){
        std::cout << "Error : Disconnected from redis cluster node " << c->c.tcp.host << ":" << c->c.tcp.port << std::endl;
        exit(1);
    }
    std::cout << "Disconnected from redis cluster node " << c->c.tcp.host << ":" << c->c.tcp.port << std::endl;
}

void get_response_cluster_cb(redisClusterAsyncContext *c, void *r, void *data)
{
    static int count = 0;
    std::cout << "get_response_cluster_cb " << count << std::endl;
    redisClusterAsyncContext* rac = static_cast<redisClusterAsyncContext*>(data);

    if ( count > 8 ){
        std::cout << "Lets disconnect " << std::endl;
        redisClusterAsyncDisconnect(rac);
        // redisClusterAsyncFree(rac); Can't free structure here
    } else {
        count ++;
        std::stringstream ss;
        ss << "key" << count ;
        redisClusterAsyncCommand(c, get_response_cluster_cb, c, "GET %s", ss.str().c_str());
    }
}

int main(){
    std::string host = "localhost:7000";
    struct event_base *base = event_base_new();

    std::cout << "Connecting to redis " << host << std::endl;
    redisClusterAsyncContext *c = redisClusterAsyncConnect(host.c_str(), HIRCLUSTER_FLAG_NULL);
    if (c->err) {
        std::cout << "Redis problem when creating async context : " << c->errstr << std::endl;
    }

    redisClusterLibeventAttach(c, base);
    redisClusterAsyncSetConnectCallback(c, connect_cluster_cb);
    redisClusterAsyncSetDisconnectCallback(c, disconnect_cluster_cb);

    int status = redisClusterAsyncCommand(c, get_response_cluster_cb, c, "GET %s", "dummyping");
    if ( status == REDIS_OK){
        std::cout << "status ok" << std::endl;
    } else {
        std::cout << "status not ok" << std::endl;
    }

    event_base_dispatch(base);

    event_base_free(base);
    // redisClusterAsyncFree(c); If I add this, the program do not leak.
    return 0;
}

Use this line to compile it :
g++ leak_example.cpp  -O0 -g3 -std=c++11 -levent -lhiredis_vip

And when running it with valgrind, i get this :
==5900== Memcheck, a memory error detector
==5900== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==5900== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==5900== Command: ./a.out
==5900==
Connecting to redis localhost:7000
status ok
Connected to redis cluster node <IP_ADDRESS>:7000
get_response_cluster_cb 0
Connected to redis cluster node <IP_ADDRESS>:7001
get_response_cluster_cb 1
get_response_cluster_cb 2
get_response_cluster_cb 3
Connected to redis cluster node <IP_ADDRESS>:7002
get_response_cluster_cb 4
get_response_cluster_cb 5
get_response_cluster_cb 6
get_response_cluster_cb 7
get_response_cluster_cb 8
get_response_cluster_cb 9
Lets disconnect
Disconnected from redis cluster node <IP_ADDRESS>:7000
Disconnected from redis cluster node <IP_ADDRESS>:7002
Disconnected from redis cluster node <IP_ADDRESS>:7001
==5900==
==5900== HEAP SUMMARY:
==5900==     in use at exit: 205,309 bytes in 34 blocks
==5900==   total heap usage: 433 allocs, 399 frees, 250,066 bytes allocated
==5900==
==5900== 132,605 (184 direct, 132,421 indirect) bytes in 1 blocks are definitely lost in loss record 15 of 15
==5900==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==5900==    by 0x4E54337: redisClusterAsyncInitialize (hircluster.c:4256)
==5900==    by 0x4E54337: redisClusterAsyncConnect (hircluster.c:4435)
==5900==    by 0x401B67: main (leak_example.cpp:43)
==5900==
==5900== LEAK SUMMARY:
==5900==    definitely lost: 184 bytes in 1 blocks
==5900==    indirectly lost: 132,421 bytes in 32 blocks
==5900==      possibly lost: 0 bytes in 0 blocks
==5900==    still reachable: 72,704 bytes in 1 blocks
==5900==         suppressed: 0 bytes in 0 blocks
==5900== Reachable blocks (those to which a pointer was found) are not shown.
==5900== To see them, rerun with: --leak-check=full --show-leak-kinds=all
==5900==
==5900== For counts of detected and suppressed errors, rerun with: -v
==5900== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

As I commented in the last part of my code, calling the redisClusterAsyncFree at the end of the main solves the memory leak. However, in my program, this is not possible ( i am connecting and disconnecting from the cluster from time to time).
Doing the redisClusterAsyncFree right after calling the redisClusterAsyncDisconnect is not possible also because it generates invalid reads and writes.
One idea i have is to count the connects and disconnects for a redis cluster and call redisClusterAsyncFree after all disconnects callbacks are called. However i am not able to pass any data to the connect and disconnect callbacks in order to do some counting for a particular connection.
Probably the redisClusterAsyncDisconnect should free the redisClusterAsyncContext ? This is the behaviour of the redisAsyncDisconnect of the hirerdis library.
Thanks in advance,
pablo

Hi Pablo,
The way I have done it , create DAO layer which in turn has redis client layer, but all callbacks registered are of DAO layer,
so connect call back, disconnect call back will get hit on the DAO layer, you can easily add a counter to track the number of connects towards a redis-cluster and call redisClusterAsyncFree when the counter goes to zero from DAO layer.
regards,
badari


Emoji as unicode

How do I get Emojis displayed/entered in TextView/EditText as unicode for sending it to server?
Expected Output
User enter this emoji: 👩
When editText.getText().toString() is performed, it shoud result into: \uD83D\uDC69

This has nothing to do with emojicompat.

Got it.
To display: EmojiTextView can render 👩 by passing \uD83D\uDC69
To get the Emoji from EditText or TextView as \uD83D\uDC69 for sending to server, I am using StringEscapeUtils.
StringEscapeUtils.escapeJava(textView.getText().toString()) // 👩
outputs \uD83D\uDC69