System notifications - backend

Develop end-points for following notifications
-internal organization notifications when requests are delegated to another.

Notifications added

Run migrations to add notification table
Frontend integration

A websocket connection must be initialized using following URL
ws://<IP_ADDRESS>:8000/ws/notifications?token=TOKEN

Where TOKEN is the JWT token from auth but without the JWT part

This would continuously push new notifications on the following format

{
 "type": "notification",
  "payload": {
    "id": "3f7ef8cb-aa00-4b9a-b132-c4fc7f44ff6c",
    "notification_type": "INCIDENT_ASSIGNED",
    "is_read": false,
    "send_to": 4,
    "actioned_by": 4,
    "incident": "026d5606-bc0a-448c-972e-e04f2295ecdf",
    "custom_messsage": null,
    "createdDate": "2020-04-15T20:17:11.124665+05:30"
  }
}

Apart from above two HTTP endpoints exist

To get all notifications for a user
GET http://<IP_ADDRESS>:8000/notifications

To mark a notification as read
GET http://<IP_ADDRESS>:8000/notifications/<notification_id>/read

awesome.


Optimize bbolt memory usage

Issue #, if available:
Resolves: #969
Description of changes:

Ensure bbolt insertions happen in chunks of 5k. Bbolt only splits nodes when flushing dirty pages to disk, so a single transaction with many inserts results in a very large node object that ends up eating up a lot of heap memory.

Ensure insertions are "sequential", meaning all KV pairs are inserted in key sorted order.

Testing performed:
make test && make integration
Dockerfile
FROM ubuntu@sha256:c9cf959fd83770dfdefd8fb42cfef0761432af36a764c077aed54bbc5bb25368
RUN mkdir /layer1 && for i in $(seq 20000); do dd if=/dev/random bs=1024 count=1 of=/layer1/${i}; done
RUN mkdir /layer2 && for i in $(seq 50000); do dd if=/dev/random bs=1024 count=1 of=/layer2/${i}; done
RUN mkdir /layer3 && for i in $(seq 100000); do dd if=/dev/random bs=1024 count=1 of=/layer3/${i}; done

Heap In_use before:

Heap In_use after:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@turan18 can you document if there is any performance trade-offs to chunked insertion?

@turan18 can you document if there is any performance trade-offs to chunked insertion?

Added the pull performance benchmarks to the description.


Use of "iam.amazonaws.com/permitted" annotation causes conflicts with existing KIAM setups

I'm submitting a...

[X] bug report

related to roleArn functionality described in issue https://github.com/godaddy/kubernetes-external-secrets/issues/143
and implemented in PR https://github.com/godaddy/kubernetes-external-secrets/pull/144

Use of KIAM iam.amazonaws.com/permitted annotation for roleArn validation causes conflicts with existing KIAM setups.

iam.amazonaws.com/permitted is a KIAM created and managed annotation, and not part of any known standard.
(e.g. kube2iam uses a different annotation: iam.amazonaws.com/allowed-roles )

Issue 1:
What is the current behavior?
Kubernetes external secrets errors out when iam.amazonaws.com/permitted is present, has valid values intended for kiam, but is not provided a roleArn in the ExternalSecret descriptor.
This is a common use case when kiam is handling the IAM for the kubernetes-external-secret operator, and no additional role assumption is required by the individual ExternalSecret resources
To reproduce:
kind: ExternalSecret
metadata:
  annotations:
  name: test-service
secretDescriptor:
 #No Role ARN Provided
  backendType: secretsManager
  data:
  - key: test-service/password
    name: password

Namespace annotated for kiam support only:
kind: Namespace
metadata:
  annotations:
    iam.amazonaws.com/permitted: my-kiam-roles.*

{"level":50,"time":XXX,"pid":16,"hostname":"XXX","type":"Error","stack":"Error: not allowed to fetch secret: test-service: namspace does not allow to assume role undefined
    at Poller._upsertKubernetesSecret (/app/lib/poller.js:110:14)
    at process._tickCallback (internal/process/next_tick.js:68:7)","msg":"failure while polling the secret test-service","v":1}

What's the expected behavior?
Secret sync should be allowed when namespace annotation is present, and no roleArn is provided.
{
     // test undefined
     ns: { metadata: { annotations: { 'iam.amazonaws.com/permitted': 'my-kiam-role.*' } } },
     descriptor: { roleArn: undefined },
     permitted: true
},

Issue 2
What is the current behavior?
kiam or kubernetes-external-secrets will now incorrectly permit (i.e. pass the regex and proceed to attempt an sts call) assumption of roles in a namespace not intended to be used by kiam or kubernetes-external-secrets.  The sharing of an annotation allows for nondeterministic behavior from each operator when used at the same time.
Scenario (AWS ARNs truncated for brevity):

kubernetes external secret controller uses kiam to assume roles for processing, with that role (and all other allowed kiam roles) covered by the iam.amazonaws.com/permitted: kiam-roles.* annotation on the namespace

kubernetes external secrets ExternalSecret has a desired roleArn value my-team-external-secret-role.*

kiam policies are attached to kiam-server pods, and have no knowledge of ExternalSecret level kubernetes-external-secret role / polices

kubernetes-external-secret policies are assumed at runtime from the kubernetes-external-secret controller AFTER it assumes a role via kiam, and have no knowledge of all the other kiam level role / polices

In order to support this, one of the following must now happen:

regex updated to support both patterns: (kiam|my-team-external-secret-role).*

now kiam OR kubernetes-external-secrets is "permitting" roles they can not actually assume

naming convention updated to support a single shared pattern: namespace-iam-roles.*

same result, kiam OR kubernetes-external-secrets is still "permitting" roles they can not actually assume

What's the expected behavior?
One of the following should be done to kubernetes-external-secrets prevent conflicts

Backwards Compatible Break - change the support annotation for  kubernetes-external-secrets to be something under the existing CRD naming of externalsecrets.kubernetes-client.io,

Allow the annotation to be configurable so that both kiam and kubernetes-external-secrets can run without conflict

Explicitly document this issue in the README.md

Thanks @mmcaya for taking the time to do such a good write-up <3
In regard to issue #2:
As far as i understand (correct me if i'm wrong), Kiam is designed to "just" intercept the metadata API. If a pod has already assumed a role, Kiam can do nothing if the pod tries to "escalate" privileges (i.e. assume other roles).
Since there is a 2-way trust relationship between roles needed i don't see a big security issue here. The issue lies in the design/implementation of Kiam.
I personally 100% support documenting this issue and making the externalsecrets.kubernetes-client.io/permitted (or however one will call it) configurable.
I need to take some more time to dig into issue 1 you mentioned.
Just for clarity so no one wastes effort: is someone already working on these issues?

👍 Sorry for the delayed response, thanks for the fixes!


Patch 2 - marriage abroad Ecuador

https://trello.com/c/XW4TQoxW/1580-getting-married-abroad-ecuador-opposite-sex-marriages
On all opposite sex pages:
Replaced Embassy address and contact details and replace with consular appointments link.
Removed content about getting documents translated.

Closing in favour of #3519


[intro-to-pytorch]Part 1 - Tensors in PyTorch (Exercises).ipynb

Add information and example about operator "@" which override for matrix product in numpy and pytorch.
Operation: torch.mm(x, y) equal x @ y. For numpy same: np.matmul(a, b) equal a @ b.
This notation not common but you can see it sometime.
And some trick for weights.view(5,1) you can get using notation: weights.t() - shorter.

We suggest you refer to the Knowledge hub and reach out to your assigned mentor for user-specific issues.
Thanks


alacritty doesn't find and set correct TERM

Which operating system does the issue occur on?
Mageia Cauldron (mga7-dev0) x86_64
If on linux, are you using X11 or Wayland?
X11
On my system, alacritty doesn't set the correct TERM.
env | grep TERM
COLORTERM=truecolor
TERM=alacritty

This means that I don't get colors in neovim. When I set TERM manually to xterm-256colors, it works as expected. My wordaround is the export TERM in .zshrc.
Example

alacritty -V
alacritty 0.2.4

Checked out from git master today.

This is defined in alacritty.yml


Remove deprecated bifurcationfunc.py for v4

Codecov Report

:exclamation: No coverage uploaded for pull request base (v4@f6451f6). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             v4     #1033   +/-   ##
======================================
  Coverage      ?   100.00%
======================================
  Files         ?        35
  Lines         ?      2443
  Branches      ?         0
======================================
  Hits          ?      2443
  Misses        ?         0
  Partials      ?         0


Extensions: Visual Studio Extensions should reference new Nuget packages

we need to make sure we're using SharpGL.WPF etc etc, not the old packages.

Should be done with my PR https://github.com/dwmkerr/sharpgl/pull/91.


Multiple layering policies can be created

Only 1 layering policy should be allowed to be created or else an error should be thrown (409).

Addressed by: https://review.gerrithub.io/#/c/383159/


change in SAS Token management

CRI 51437740
There's a change in the way Azure Storage manages SAS token for Table service
Documented here https://github.com/Azure/azure-cli/issues/4827
If we use "az storage account generate-sas --services bt" - then the generated SAS token will only include Blob service (ss=b).
We need to generate the SAS token for the Table service here, so we need to use "az storage table generate-sas --services bt" and that will generate a SAS token for Table service too (ss=bt).

@axelgMS : Thanks for your contribution to the Azure documentation! The author, @jasonzio, has been notified to review your proposed change.

Hi, @axelgMS . It doesn't look like the same parameters exist if you use az storage table generate-sas. I had ran in to the same issue, and reached out to @jasonzio , but I didn't end up with a working example. Do you have a full command example of how to generate both a Table and Blob storage token with az storage table generate-sas, please?
Here's what shows as being available in Azure CLI 2.0.22:
usage: az storage table generate-sas [-h] [--output {json,tsv,table,jsonc}]
                                     [--verbose] [--debug] [--query JMESPATH]
                                     [--account-name ACCOUNT_NAME]
                                     [--account-key ACCOUNT_KEY]
                                     [--connection-string CONNECTION_STRING]
                                     [--sas-token SAS_TOKEN] --name TABLE_NAME
                                     [--permissions PERMISSION]
                                     [--expiry EXPIRY] [--start START]
                                     [--policy-name ID] [--ip IP]
                                     [--https-only] [--start-pk START_PK]
                                     [--start-rk START_RK] [--end-pk END_PK]
                                     [--end-rk END_RK]

I wouldn't expect az storage table generate-sas to let you generate a single SAS that permits access to multiple services (block and table); the command name alone strikes me as restricting its scope to tables only.

Please reassign followup to @abhijeetgaiha; please check if the latest release of the Azure CLI has restored the required functionality to "az storage".

@sadrat  - can you please reassign to @abhijeetgaiha per Jasonzio?  Thx.

I received an email report this morning which says that Azure CLI 2.0.29 has restored the lost functionality of "az storage".
$ az storage account generate-sas --account-name redactedaccount01 --expiry 2099-01-30T23:59Z --permissions wlacu --resource-types co --services bt -o tsv

se=2099-01-30T23%3A59Z&sp=wlacu&sv=2017-07-29&ss=bt&srt=co&sig=123abc456def789ABC0123abkjLKSOhPhLhddAbCdEfGh%3D

@iainfoulds, @axelgMS, can you confirm? If so, we can just close this as "No Change", which is kind of what I'd hoped we could do.

Thanks, @jasonzio . I can confirm in 2.0.29 that this now works as it originally did. As such, no changes are required. If the CLI changes again in the future, we can revisit this.
#please-close


custom width not work well in react-native-web

Current Behavior
i copy some demo code and use it in react-native-web project,
but it looks not well.

Expected Behavior
work as demo show
Your Environment

Tech
Version

"react-redux-firebase":
"^2.1.9",

"react-swipeable-views":
"^0.13.0",

"react-swipeable-views-core":
"^0.13.0",

"react-swipeable-views-native":
"^0.13.1",

"react-swipeable-views-utils":
"^0.13.0",

"react":
"^16.6.0",

"react-native-web":
"^0.9.5",

ok, i got the Right Behavior
i do something like this:
<Virtualize style={{ padding: 30 }} slideRenderer={slideRenderer}  />
const slideRenderer = ({ index, key }) => (
  <View
    key={key}
    style={{ padding: 15 }}
  >
    <Tile name={key} code={'#1abc9c'} />
  </View>
);


Readme edits

Reasons for making this change
Grammar & clarity in README.md
If this is related to existing tickets, include links to them as well.
Checklist

[x] I'm updating documentation

[x] I've checked the rendering of the Markdown text I've added
[ ] If I'm adding a new section, I've updated the Table of Content

[ ] I'm adding or updating code

[ ] I've added and/or updated tests
[ ] I've updated docs if needed
[ ] I've run npm run cs-format on my branch to conform my code to prettier coding style

[ ] I'm adding a new feature

[ ] I've updated the playground with an example use of the feature

Thanks !


Minor syntax fixes to front page.

Coverage remained the same at 80.849% when pulling 65f190c7b755b3b4d53df976c107a97750f091bf on imallett:master into 06518a5fddd3313bd35e59e56de9358588f24a6e on pygame:master.

Thanks Ian :)


Removal of local admin rights

Removal of local admin rights doesn't make sense when the same Azure AD tenant is used for production devices.  The article is written like the Azure AD tenant is used only for Secure Workstations (!?). In reality you would use the tenant also for other devices and then you might want to have additional device administrators to support your Windows 10 AADJ devices. So this is an advice which needs to be clarified.  Most of the environments I've seen are using the device administrators as this is the only way for device administration currently. The design of this guide is sub-optimal if you use a cloud-only approach. I understand that there is no MS technology currently to solve this, no separation is available, but this should be told here clearly. I hope this will be fixed in future.

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

ID: 80358dc4-a251-b359-e53c-6fd2fed12d1a
Version Independent ID: a5ff6566-b53c-e6c1-5697-bd578c13532d
Content: Deploy Azure-managed workstations - Azure Active Directory
Content Source: articles/active-directory/devices/howto-azure-managed-workstation.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

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

@okieselbach I apologize for the frustration, and understand that there are some issues here that we can address further.
I'm going to look into this and will let you know as soon as I have determined the reasoning behind this,

@okieselbach this is V1 of the guide. It is something we are aware of but wanted to get some guidance out for organizations who have asked for how to do this in a cloud-only scenario. Organizations can make modifications to the guidance in the doc to suit their needs.

#please-close


Logouts too often

app.dbdesigner.id logs me out too often. Is it necessary?

I think this is not necessary. This is bug in backend. Now i change rule for log out if user not login in 24 hours.
@chimit
Thank you...


Update README.md to specify gathered telemetry

PR Summary
The telemetry statement in the README seems out of date.
PR Context
The telemetry statement should fully describe the data we gather.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Pointing to the topic doc is surely better.

@JamesWTruher please fix up the spelling issue, which caused the 'static-analysis' CI failed.

I updated the CI yamls so future PRs to readme and spelling shouldn't trigger product tests, but because the PR now touches those files, we need to run those pipelines to validate those changes.


fixdisable NullSafeEqualToEqual rule

Proposed changes
NullSafeEqualToEqual depends on join conjunct's nullable info. But the nullable value may change after this rule. So convert from <=> to = may be wrong. We disable this rule for now and fix it later

run buildall


Crypto Donation Feature

We are looking for help on creating a cool way for users to tip the project's multi-signature ethereum and/or bitcoin wallet through our website.  Ask @zachdt or @Redoudou for the wallet address when ready to complete this issue.

https://github.com/leaseOTB/ui-components/issues/6
issue transferred to ui-component library


Additional guidance for GCSE grades

Minimum qualification grade
For GCSE or equivalent grades, include guidance around minimum grade and suggestion to contact a provider if currently don‘t meet the criteria.

Multiple English qualifications
For English GCSE subject, add two disclosures:

Only have an English Literature qualification
Have more than one English qualification

"Provider expect you to", shouldn't it be "To train you must"?

@paulrobertlloyd @fofr I prefer 'Providers expect you to' over 'To train you must' because the latter's not strictly true, and I worry it would be off-putting. We know there are cases where people can still train if they have a D (or whatever) because they have other evidence of their skills.
I think this is good to merge now.


[BugFix] Add LzoStreamCompression support text lzo file #19908

What type of PR is this：

[x] BugFix
[] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #19908
Problem Summary(Required) ：

Hive catalog can't read hive table with textfile lzo format
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)
[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto backported to target branch

[ ] 3.0
[ ] 2.5
[ ] 2.4
[ ] 2.3

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wangshifa seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for the contribution, Please sign the CLA

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


Moving to a new package naming

What's done:

Updated naming in packages
Updated tests and resources

@nulls @petertr pls approve and I will merge it


NUI Editor: layoutInfo can be added via context menu to a layoutInfo

What you were trying to do
I was checking out what options the context menu offers me when I right click a layoutInfo node in the NUI Editor.
What actually happened
The context menu offered me the option to add a layoutInfo node to a layoutInfo node.
How to reproduce

Open the NUI editor. To do so press ESC ingame and then click at the bottom right on "Developer Tools". Then select "NUI Editor Screen".
Select a screen (e.g. addServerPopup from the combo box at top left
Navigate to a layoutInfo node
Right click it and checkout what context menu option it offers in the "Add ..." sub menu. It offers to add a layoutInfo which is wrong.

(Should have been closed by #2270, but wasn't - closing manually)


edgecast_dns_zone not working

Hi,
I'm trying to create a dns zone using the provider, but i get AddZone: SubmitRequest: sendRequest failed (HTTP StatusCode:500): {"Message":"Operation Error. Contact Administrator"}
I can create the same zone with the same data just using curl to send the POST for the Add Primary Zone API, so my credentials and the data I'm using are ok.
Resource definition:
resource "edgecast_dns_zone" "main" {
  account_number    = "redacted"
  domain_name       = var.zone_name
  status            = 1    # 1: active, 2: inactive
  zone_type         = 1    # 1: Primary zone. This value should always be 1.
  is_customer_owned = true # This value should always be true

  record_txt {
    name = "test"
    ttl = 3600
    rdata = "testtxt"
  }
}

Tried with Terraform 1.3.9 and 0.13.7 and with provider 1.2.1 and 1.1.1

@cosimomeli - apologies for the delay in my response. We have identified a fix that will go in out in v1.2.4. Thank you for reporting this.

@cosimomeli  - this issue was addressed with our latest release.
https://registry.terraform.io/providers/Edgio/edgecast/1.3.0
Please let us know if any issues. Thanks.


Move gyro calibration from commander to events

This patch moves gyro calibration out of commander to events framework.
Some of the common code shared with other sensor calibration code is moved
to systemlib/calibration
This is one of the part of task mentioned in #8014
Currently the changes are tested on pixhawk using QGC and from commandline and compared the gyro calibration values with and without the changes.

It's hard to review through github, but so far it looks good!
CI failure is a minor clang-tidy issue.

I think the notification stuff should stay private to commander (and needs a cleanup later).
LED control can be done via uORB like this -  https://github.com/PX4/Firmware/blob/master/src/modules/events/temperature_calibration/task.cpp#L343
tones can be written to the tone alarm device like this - https://github.com/PX4/Firmware/blob/master/src/modules/mavlink/mavlink_receiver.cpp#L1510

@sugnanprabhu Could you rebase?

@LorenzMeier I have rebased the changes.

You can keep the calibration routines in with the calibration code for now, I don't think it belongs in systemlib (which should already be split in multiple libraries).

@dagar moved the calibration routines back to commander. As discussed, some of the calibration code is still shared between the other sensor calibration in commander and gyro calibration in events, we can keep this way for now and move calibration routines to events with the movement of all calibration code.

Great! @PX4TestFlights could you give this a try on a few different boards? You don't need to do a full flight test, just set logging from boot and try the gyro calibration. With the logging running from boot we'll be able to compare the params before and after. Thanks!

We'll test this out in the morning... what would be the potential issues
and expected behavior?
On Oct 11, 2017 8:59 PM, "Daniel Agar"<EMAIL_ADDRESS>wrote:

Great! @PX4TestFlights https://github.com/px4testflights could you give
this a try on a few different boards? You don't need to do a full flight
test, just set logging from boot and try the gyro calibration. With the
logging running from boot we'll be able to compare the params before and
after. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PX4/Firmware/pull/8015#issuecomment-336014266, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AXRu8OIGPpl0SD1E_wt8JQQlKxVSD3viks5srY6sgaJpZM4Pj-wK
.

tested on pixracer (V4)
gyro cal failed to start
https://logs.px4.io/plot_app?log=7ee149d6-b61f-43df-add3-ada78acfdad5

@dagar I do not have pixracer to test here.
px4_task_spawn_cmd is failing, any idea what could be the issue?
https://github.com/PX4/Firmware/pull/8015/files#diff-d20167b763bb6f6b6ac105473dfa4931R97
@santiago3dr did you get a chance to test it on other boards?

I'll give it a try on my pixracer.

calibration went through just fine on a pixhawk 3 pro (v4pro)
https://logs.px4.io/plot_app?log=b7dc9c78-b559-4947-b015-10b87fd4462b

@santiago3dr I reduced the stacksize from 5800 to 3000 same as used by commander_low_prio_loop to make sure when other calibration routines are moved in next patches it still works.
Can you please give it a try on pixracer?

@sugnanprabhu
tested on pixracer, calibration didn't work:
https://logs.px4.io/plot_app?log=184c0a00-46d4-4a7d-a7be-e4b9784c4b4f

We should get the error messages fixed here. https://github.com/PX4/Firmware/issues/8100
I have some ideas for things we can do with logger, etc to ensure there's always enough free memory to perform the calibration.

@sugnanprabhu @dagar
The previous calibration was with USB connection, I calibrated again disconnecting USB, with telemetry connection and it worked.
https://logs.px4.io/plot_app?log=6c9378f6-f889-41fb-b71b-5c0155756599

Thanks @r0gelion
This should be acceptable if we restore proper error messages. (again https://github.com/PX4/Firmware/issues/8100)
I'll also add a simple message to suggest disconnecting usb if you hit out of memory during calibration and it's connected.

As moving the calibration incrementally is causing memory issue on pixracer, created a new PR moving all the calibration to events #8200


JAXB dataformat support

This might work already, we are perhaps just missing a test.

I can look at this


Chart.yaml: kubeVersion not interpreted/compared correctly

I tried to constrain the Kubernetes version the chart can run on in my Chart.yaml with the kubeVersion key.
When trying to deploy, the deployment fails due to the comparison not evaluating to true.
Output of helm version:

version.BuildInfo{Version:"v3.7.0", GitCommit:"eeac83883cb4014fe60267ec6373570374ce770b", GitTreeState:"clean", GoVersion:"go1.16.8"}

Output of kubectl version:

Client Version: version.Info{Major:"1", Minor:"19+", GitVersion:"v1.19.6-eks-49a6c0", GitCommit:"49a6c0bf091506e7bafcdb1b142351b69363355a", GitTreeState:"clean", BuildDate:"2020-12-23T22:13:28Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}

Server Version: version.Info{Major:"1", Minor:"19+", GitVersion:"v1.19.13-eks-8df270", GitCommit:"8df2700a72a2598fa3a67c05126fa158fd839620", GitTreeState:"clean", BuildDate:"2021-07-31T01:36:57Z", GoVersion:"go1.15.14", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):

AWS EKS

Chart.yaml (excerpt):

kubeVersion: ">= 1.19.0"

Error:

chart requires kubeVersion: >= 1.19.0 which is incompatible with Kubernetes v1.19.13-eks-8df270

In my understanding, the constraint of >= 1.19.0 should satisfy v1.19.13-eks-8df270 as -eks-8df270 is interpreted as metadata and should not be included in the comparison.
Also I checked an online validator, that also showing this constrain to be valid against the k8s version: [1]

A version number with a dash following the <MAJOR>.<MINOR>.<PATCH> version fields is marked as a pre-release. Pre-releases are excluded from general semantic version lookups unless the constraint includes a pre-release identifier to the constraint.
There are two ways to fix this:

Ask AWS to fix their versioning scheme to match the build metadata schema: v1.19.6+eks-49a6c0
Change your constraint to include pre-release versions as described in the documentation: >= 1.19.0-0

https://helm.sh/docs/chart_template_guide/function_list/#working-with-prerelease-versions

Thanks for your response, this was very useful.
I overlooked the difference between + and -.
I opened up an AWS support ticket and will report back as soon as I got an answer if there is the possibility to fix their versioning scheme.

In the meantime, I'm going to go ahead and close this issue as answered as I think I provided a workaround in case AWS chooses not to change their versioning scheme. Please feel free to comment here and let us know if you have further questions. Thanks!

For anyone who is also interested in the reponse of the AWS team, here is the final answer.
TL;DR: The version scheme of EKS won't be changed in the near feature.

I would like to inform you that the team has come back with their response and explanation to EKS not following the semvar spec.

As I had already assumed, making the version string change is a big change, which is not a priority as yet, and is not likely to be prioritised in the near future either. There have been issues raised previously when EKS accidentally used the build metadata (+) symbol, which was tracked here [1]. In fact, AWS is tracking the importance of this issue here officially, and I would recommend you +1 the ticket to help it gain traction/prioritisation.

One thing to highlight, is that the team believes that we need more than 3 version components (major.minor.patch) that semvar has a strict adherence to. As EKS is a redistributor of Kubernetes which already follows the 3 version components from semvar, we require a 4th version component to be able to distinguish which of our multiple versions of EKS using the same Kubernetes version is newer. When using Build metadata (+), you are unable to use a 4th version component to sort which version is newer. However, with Pre-release metadata as you have already seen, there is a workaround in place to include pre-release metadata in the sorting logic. For example:
Using Build Metadata:               1.20.7+eks.12 == 1.20.7+eks.11
Using Pre-Release Metadata:    1.20.7-eks.12 > 1.20.7-eks.11

Unfortunately, I can't provide an ETA on when EKS team will be revisiting the versioning specifications, but to keep up to date with the latest features announced, take a look at the following websites [2, 3].

[1] Use correct SemVer for Kubernetes version
https://github.com/aws/containers-roadmap/issues/1404#issuecomment-884751824

[2] Containers Road Map
https://github.com/aws/containers-roadmap/projects/1

[3] EKS News Blog
https://aws.amazon.com/blogs/aws/category/compute/amazon-kubernetes-service/

Would it make sense to deprecate the current semverCompare function and introduce a newly named function? As it is there's something of a cottage industry in how to handle the differences in interpreting AWS's semver format vs the standard. For at least Tier 1 providers (e.g. GCP, Azure, AWS) I would hope Helm would handle their versioning scheme correctly even if it's not semantically correct by semvers standards.
In my experience a lot of the feature toggling is done on the minor version as that's typically where Kubernetes will graduate feature flags from off to on. Having natural support baked into Helm such as >= 1.22 that includes knowledge of Tier 1 cloud providers seems reasonable (AWS at last check is larger than the next 10 IaaS providers combined). Are they bad actors? Maybe, but given the response posted above it feels like there is significant technical obstacles to change to an alternative version label.


Instances are only matched with suffix

Need to test the following:
perspective>reboot one,two
Nothing selected for this operation. Exiting.
perspective>reboot one.example.com,two.example.com
Going to reboot 2 instances.

Difference between these cases:
# Without suffix we use REGEX
2016-12-29 19:38:17,327 [             XNIO-1 I/O-2] INFO  org.meridor.perspective.rest.resources.InstancesResource     - Processing query #708 with sql = "select instances.id, instances.real_id, instances.name, projects.id, projects.name, instances.cloud_id, instances.cloud_type, images.name, flavors.name, instances.addresses, instances.state, instances.last_updated from instances inner join projects on instances.project_id = projects.id left join flavors on instances.project_id = flavors.project_id and instances.flavor_id = flavors.id left join images on instances.image_id = images.id where instances.name regexp :instances.name: or instances.name regexp :instances.name1: order by instances.name asc" and parameters = [org.meridor.perspective.sql.Parameter@1d25a2e8[index=<null>, name=instances.name, value=one], org.meridor.perspective.sql.Parameter@34077ef2[index=<null>, name=instances.name1, value=two]]
2016-12-29 19:38:24,254 [             XNIO-1 I/O-2] DEBUG org.meridor.perspective.rest.resources.InstancesResource     - Query #708 results = [0 rows, status = success, message = <empty>]

# With suffix we use exact match
2016-12-29 19:39:47,237 [             XNIO-1 I/O-1] INFO  org.meridor.perspective.rest.resources.InstancesResource     - Processing query #709 with sql = "select instances.id, instances.real_id, instances.name, projects.id, projects.name, instances.cloud_id, instances.cloud_type, images.name, flavors.name, instances.addresses, instances.state, instances.last_updated from instances inner join projects on instances.project_id = projects.id left join flavors on instances.project_id = flavors.project_id and instances.flavor_id = flavors.id left join images on instances.image_id = images.id where instances.name = :instances.name: or instances.name = :instances.name1: order by instances.name asc" and parameters = [org.meridor.perspective.sql.Parameter@71ee79e4[index=<null>, name=instances.name, value=one], org.meridor.perspective.sql.Parameter@6963ed17[index=<null>, name=instances.name1, value=two]]

This was fixed in #138.


Directly use Quartz replase api to bootstrap a schedule

Purpose of the pull request
Optimize the scheduler code in quartz plugin, use quartz replace API to insert/update trigger and job.
Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

It's ok to remove lock when build job details and schedule?

It's ok to remove lock when build job details and schedule?

We don't need to use lock here.

Codecov Report
Attention: Patch coverage is 31.00000% with 69 lines in your changes are missing coverage. Please review.

Project coverage is 39.09%.  Consider uploading reports for the commit 713add8 to get more accurate results

Files
Patch %
Lines

...ler/scheduler/quartz/QuartzCornTriggerBuilder.java
0.00%
26 Missing :warning:

...hinscheduler/scheduler/quartz/QuartzScheduler.java
0.00%
19 Missing :warning:

...duler/scheduler/quartz/QuartzJobDetailBuilder.java
0.00%
15 Missing :warning:

...cheduler/scheduler/quartz/ProcessScheduleTask.java
0.00%
4 Missing :warning:

...lphinscheduler/scheduler/quartz/QuartzJobData.java
75.00%
2 Missing and 2 partials :warning:

...scheduler/quartz/QuartzSchedulerConfiguration.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15781      +/-   ##
============================================
- Coverage     39.16%   39.09%   -0.08%
- Complexity     4867     4878      +11
============================================
  Files          1316     1325       +9
  Lines         44978    45110     +132
  Branches       4803     4790      -13
============================================
+ Hits          17617    17634      +17
- Misses        25477    25590     +113
- Partials       1884     1886       +2

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


Setup process is not intuitive

Expected behavior and actual behavior
For some reasons the password already contains a phantom asterisk symbol from the beginning. When I ran server.sh script for the first time I got distracted by a new email message from the browser window.
When I got back to the terminal, I saw that the password field already contained a symbol. So I thought that I'd already started typing my password for a root user. So I just finished typing the rest of the password (i.e oot in my case) and called it a day.
When I got back to a server the next day and tried to authenticate, I realized that my password didn't match. After some time I figure out that first symbol wasn't part of my password and the real password for the root user is oot not root as I thought before:

Another thing that blew my mind was the fact that OrientDB can generate a new password for the root user if you hit an Enter key. The problem is OrientDB doesn't print this password anywhere. Even in orientdb-server-config the root's password is in encoded format:

In the end, the whole setup process wasn't as smooth as I expected.
Runninng Mode

[x] Remote

OrientDB Version

[x] v2.2.x - Please specify last number: 3

Operating System

[x] MacOSX

Java Version

[x] 8

When the server asks for the password, you didn't press any button and you can see the *, right? If you insert "root", the password is taken. The first * is to protect by showing any digits.
About the auto-generated password, we should print it.

When the server asks for the password, you didn't press any button and you can see the *, right?

That's right.

If you insert "root", the password is taken. The first * is to protect by showing any digits.

What digits? Have you ever seen how the terminal accepts passwords in Linux-based OS or OS X? You don't even see what you're typing. From the security point of view, this approach is even better because nobody will be able to say how many symbols your password consists of by looking at your screen over your shoulder.

Showing the * in place of the characters is a good compromise for the user that shows the console is reading the password. Like when you enter your password in any web forms.

Well, at least, there shouldn't be a starting * symbol because it's very misleading. It looks like I already typed a symbol.

This is a limitation with the way the password protection is managed via console. I'm closing this issue because not really relevant.


Provide completion for command link type

When using Walkthrough/Getting started, a Markdpwn fiel is referenced whcih can contain command link that are executing VS Code commands. it woudl be nice to have completion for available commands and thei rpotential parameters.

Thanks for the request! Unfortunately this is not something we currently support at an structure level and we won't be adding it for this narrow use case


Fixed validation of inline attribute "false"

I noticed, that inline assignments of "false" dit not work because they are given as strings and then strict compared to false.
Example
In file index.html

In file partial.tpl.html

Hello World!

This is text.

Expected result

This is text.

Current Result

Hello World!

This is text.

Addition: Semantic if
We often use partials with some _if statements for things like "project properties". I thought that other may do so as well and integrated an option to enable "semantic" statements using "off"or "no". It is disabled by default.

I added a description for this option to the README. The new code passed all your tests.

Hi David, thanks for the update. I like the addition of the semantic update. Thanks for that. I will merge it now, since it passes all the tests.


Merging an Event with an "Accepted" Workshop Request

@pbanaszkiewicz I just created an event from a URL without accepting the request first. I was trying to go in and match the request to the created event. Once I accepted it (without creating an event) I still couldn't just merge the two. Can we make this an option?

Whoops I figured it out! I'm closing now. Always good when I can solve my own problems!!!!


StaticFile / fromResource: plain code

We rewrite the code of the fromResource method, so as to rely less on
the OptionT wrapper methods.
We avoid introducing the OptionT wrapper until the end, and use for
most of the operations the methods on Option.
We extract a non-optionT method variant for fromUrl.

Withdrawing until the code changes are tackled.


Document how to access tag values

http://rubymotionquery.com/?s=tag&post_type=document
# You can optionally store a value in the tag, which can be super useful in some rare situations
rmq(my_view).tag(your_tag: 22)
rmq(my_view).tag(your_tag: 22, your_other_tag: 'Hello world')

Doesn't mention how to access the 22 and Hello World values. Don't have time to figure it out atm but figured I'd post an issue so I don't forget.

@jamonholmgren I just poked around to figure it out for you - I can document it later, though I'd like to check with @twerth that this is the right API and I'm not just digging into internals here.....
(main)> rmq(my_view).tag(your_tag: 22)
=> RMQ<PHONE_NUMBER>. 1 selected. selectors: [#<UIButton:0x11151d4d0>]. .log for more info
[#<UIButton:0x11151d4d0>]
(main)> my_view.rmq_data.tags[:your_tag]
=> 22

:+1:

Yeah, that's the original way, although I thought we added a nicer way to do it. I'll poke around.

I don't see anything else, be nice to have something nicer, let say:
rmq(my_view).tag_data(:your_tag)

Or some such.

What about:
rmq(my_view).tags(:your_tag) # => value here

I havent actually used tags to store data inside, but I'd almost prefer @jamonholmgren's suggestion so its not one other attribute to remember.  If I am pushing data in with .tag, I want to pull it out with .tag IMHO.

@jamonholmgren @twerth I started throwing something together for this, what do you guys think about tag collisions?  Should we require this to only be on a single selected view? Warn users?  etc.
Example
rmq.append(UIView, :one).tag(important: 10)
rmq.append(UIView, :two).tag(important: 12)

rmq.all.tags(:important)
#or
rmq(UIView).tags(:important)

what do we expect here?  Should we only allow selections of tags when we have one view wrapped, take the latest value?  return an array of values or something to that effect?
My gut feels that the way we would use a tag, would be its something set on one view item, and then we should be pulling it from one view item - rather than a blanket selection, but I'd like others input before I continue on with implementation :)

in other words
rmq.all.tags(:important) should either warn or return an array of the values possibly...
but the expected behavior (maybe forced by us?) would be rmq(:one).tags(:important)

I think returning an array of the values makes the most sense to me.

I just don't know if there is a use case for that. Imagine 10 views with the same tag and requesting row.all.tags what would you do with with that array?
There is no harm in giving them all in an array and it matches what we do with multiple views in a selection. So maybe that makes sense.

Yeah, I don't think I'd ever use that feature, so it doesn't matter too much to me (same with multiple views and .get). Just seems consistent.


The process monitor did not find the connection to the configuration store. The site may not have been initialized. After the site is initialized, the check for database processes will be reattempted.

@cameronkroeker
I get the below error under portal logs. I am using Chef cookbook for ArcGIS 11.0.
<type="VERBOSE" code="217063" source="Portal" process="27884" thread="1" methodName="" machine="FQDN" user="" elapsed="" requestID="">The process monitor did not find the connection to the configuration store. The site may not have been initialized. After the site is initialized, the check for database processes will be reattempted.</Msg>
for some strange reason, I can't get to http://localhost:7080/arcgis or https://localhost:7443/arcgis using curl
both ports are listening already
portal service is up
systemctl status arcgisportal ● arcgisportal.service - Portal for ArcGIS Service  Started Portal for ArcGIS Service.
I also see below under service log file
Exception in thread "main"  java.lang.reflect.UndeclaredThrowableException         at com.sun.proxy.$Proxy0.stop(Unknown Source)         at com.esri.arcgis.portal.impl.Main.stop(Main.java:168)         at com.esri.arcgis.portal.impl.Main.main(Main.java:457)  Caused by: java.rmi.UnmarshalException: Error unmarshaling return header; nested exception is:         java.io.EOFException         at java.rmi/sun.rmi.transport.StreamRemoteCall.executeCall(Unknown Source)         at java.rmi/sun.rmi.server.UnicastRef.invoke(Unknown Source)         at jdk.remoteref/jdk.jmx.remote.internal.rmi.PRef.invoke(Unknown Source)         at java.management.rmi/javax.management.remote.rmi.RMIConnectionImpl_Stub.invoke(Unknown Source)         at java.management.rmi/javax.management.remote.rmi.RMIConnector$RemoteMBeanServerConnection.invoke(Unknown Source)         at java.management/javax.management.MBeanServerInvocationHandler.invoke(Unknown Source)         ... 3 more  Caused by: java.io.EOFException         at java.base/java.io.DataInputStream.readByte(Unknown Source)         ... 9 more  Exception in thread "main"  java.lang.reflect.UndeclaredThrowableException         at com.sun.proxy.$Proxy0.stop(Unknown Source)         at com.esri.arcgis.portal.impl.Main.stop(Main.java:168)         at com.esri.arcgis.portal.impl.Main.main(Main.java:457)  Caused by: java.rmi.UnmarshalException: Error unmarshaling return header; nested exception is:         java.io.EOFException         at java.rmi/sun.rmi.transport.StreamRemoteCall.executeCall(Unknown Source)         at java.rmi/sun.rmi.server.UnicastRef.invoke(Unknown Source)         at jdk.remoteref/jdk.jmx.remote.internal.rmi.PRef.invoke(Unknown Source)         at java.management.rmi/javax.management.remote.rmi.RMIConnectionImpl_Stub.invoke(Unknown Source)         at java.management.rmi/javax.management.remote.rmi.RMIConnector$RemoteMBeanServerConnection.invoke(Unknown Source)         at java.management/javax.management.MBeanServerInvocationHandler.invoke(Unknown Source)         ... 3 more  Caused by: java.io.EOFException         at java.base/java.io.DataInputStream.readByte(Unknown Source)  UTC 2023        ... 9 more

Hi @sgavathe,
This error is expected to show up until a portal site is created, so it can be ignored:
<type="VERBOSE" code="217063" source="Portal" process="27884" thread="1" methodName="" machine="FQDN" user="" elapsed="" requestID="">The process monitor did not find the connection to the configuration store. The site may not have been initialized. After the site is initialized, the check for database processes will be reattempted.</Msg>

I am not quite sure what the second error is. Was there an issue with creating the portal site or what error shows up in the Chef run?
Do these commands return anything?
curl https://<ip>:7443/arcgis/portaladmin/healthCheck?f=json --insecure -v

or
curl https://<ip>:7443/arcgis/portaladmin?f=json --insecure -v

Thanks,
Cameron K.

Thanks @sgavathe. Based on this error sounds like the internal tomcat server that portal uses is not starting properly:
Error checking status of web server. java.util.concurrent.TimeoutException

Let's check to see if there are any errors in: <install dir>/arcgis/portal/framework/runtime/tomcat/logs
One more thing to check is the server.xml file is not corrupted inside:
<install dir>/arcgis/portal/framework/runtime/tomcat/conf

Hi @sgavathe,
Honestly I am not quite sure what the issue could be. Can you share the chef run log and json config file? Scrub out any sensitive information such as passwords.
Have you tried uninstalling and running it again?
Thanks,
Cameron K.

Thanks @sgavathe, one thing I notice off the bat is the json configuration file only contains the installation recipes in the run-list so this will only install. It doesnt authorize, create portal site or server site, doesn't register web adaptors or data store, and doesn't federate. The recipe[arcgis-enterprise::enable_knowledgeserver] will only work if all the above has been completed.

Thanks @sgavathe. Which Linux OS distribution is this running on?


Hi Sir

Great Pyhton script!! How can i do response json data and how host as heroku . Thanks

Thanks! Sorry, I can't understand your question, do you mind being more specific?


[Deploy] Suggestion for Monday.com support

Service
monday.com integration
Link to your repository
https://github.com/WilhelmHjelm/recipe-monday

Oh, great integration. Made the same myself and didn't notice this one before. Yours is by far the better version.
Thanks.


[stdlib] Remove invalid _sanityCheck

A Character should contain only a single grapheme, but we can't formally require it because of grapheme cluster literals and the shifting sands of Unicode.  Fixes https://bugs.swift.org/browse/SR-4955
Note: the stray change in the comment is due to GitHub's editor changing all text from decomposed to precomposed.

@swift-ci Please smoke test and merge

This indeed fixed SR-4955 on my machine.
I don't understand why was only I affected… My configuration should be normal AFAIK: build-script -R. Are there some processor specific optimizations involved somewhere? Only atypical thing is that I'm running on 8 years old Core 2 duo here…

@palimondo The issue is that our normal perf test run builds without assertions. This was caught by a different bot that we have internally that builds the perf test with assertions. I could move it externally, I just have not gotten around to it.

@gottesman And the perf-test I get locally out of build-script -R is with assertions?

@palimondo Did you do a clean build? If not, look in your build directory for CMakeCache.txt and see what SWIFT_STDLIB_ASSERTIONS is set to. Reconfigs sometimes can cause that value to get out of sync.

Um... clean build, when? I have never adjusted assertions AFAIK. I did clean build
(by deleting contests of build directory) when SR-4955 was failing for me. IIRC it looks like Dave was able to reproduce my error on a build with assertions and discovered the invalid _sanityCheck. What I don’t understand how/why I stumbled upon this and how it only manifested on my machine.

@gottesmm While I have your ear... I wrote you an email with request for guidance re: tests and modularization of Benchmark_Driver. Could you respond, plz?

I saw your email. It was a long email, so the response took some thoughts. I kept on getting pulled into things over the weekend so I was unable to. I will finish responding later today.


New: added region selector tests

Before: https://stage--dme-partners--adobecom.hlx.live/channelpartners/?georouting=off
After: https://mwpw-160655-region-selector-tests--dme-partners--adobecom.hlx.live/channelpartners/?georouting=off

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.53%. Comparing base (9759fb2) to head (494bcc6).
Report is 2 commits behind head on stage.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #104      +/-   ##
==========================================
+ Coverage   83.51%   83.53%   +0.01%
==========================================
  Files          10       10
  Lines        3209     3212       +3
==========================================
+ Hits         2680     2683       +3
  Misses        529      529

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


No output when running as part of "podman build" or "docker build"

steps
Given the following Dockerfile which runs a sbt update command as part of the image construction:
FROM eclipse-temurin:11-jre-jammy

# Install git
RUN apt-get update && \
  apt-get install -y git && \
  rm -rf /var/lib/apt/lists/* && \
  git --version

# Mandatory as sbt cannot run from /root anyway
WORKDIR /build

# Install sbt
ARG SBT_VERSION=1.9.7
RUN \
  curl -k -L -o sbt-$SBT_VERSION.deb https://repo.scala-sbt.org/scalasbt/debian/sbt-$SBT_VERSION.deb && \
  dpkg -i sbt-$SBT_VERSION.deb && \
  rm sbt-$SBT_VERSION.deb && \
  apt-get update && \
  apt-get install sbt && \
  rm -rf /var/lib/apt/lists/* && \
  sbt sbtVersion

# Download some sbt project
RUN git clone https://github.com/leanovate/play-mockws.git

# Run a sbt command that should give plenty of output
RUN cd play-mockws && sbt update

Copy the file in a folder and run podman build --no-cache . to build the image from the Dockerfile.
problem
When building the image with podman build --no-cache ., the sbt command doesn't give as much output as it does when running outside of a Dockerfile.
Here's the output:
...
STEP 6/7: RUN git clone https://github.com/leanovate/play-mockws.git
Cloning into 'play-mockws'...
--> 1ae889e2fe5b
STEP 7/7: RUN cd play-mockws && sbt update
[info] welcome to sbt 1.9.7 (Eclipse Adoptium Java 11.0.21)
[info] loading settings for project play-mockws-build from plugins.sbt ...
[info] loading project definition from /build/play-mockws/project
[info] loading settings for project play-mockws from build.sbt ...
[info] set current project to play-mockws (in build file:/build/play-mockws/)
[success] Total time: 14 s, completed Nov 23, 2023, 4:12:50 PM
COMMIT
--> fc9820272ac3

expectation
Doing the same commands outside of the build context/scenario gives plenty of output (like all the stuff that sbt downloads).
For instance, comment the last line (the sbt update command) of the Dockerfile above:
- RUN cd play-mockws && sbt update
+ # RUN cd play-mockws && sbt update

Build the image again and give it a name to then exec into it:
podman build . -t testsbt

podman run -it testsbt bash

> cd play-mockws
> sbt update

Now the output is something like the following:
[info] welcome to sbt 1.9.7 (Eclipse Adoptium Java 11.0.21)
[info] loading settings for project play-mockws-build from plugins.sbt ...
[info] loading project definition from /build/play-mockws/project
https://repo1.maven.org/maven2/com/github/sbt/sbt-github-actions_2.12_1.0/0.19.0/sbt-github-actions_2.12_1.0-0.19.0.pom
  100.0% [##########] 3.0 KiB (4.5 KiB / s)
https://repo1.maven.org/maven2/org/scalameta/sbt-scalafmt_2.12_1.0/2.5.2/sbt-scalafmt_2.12_1.0-2.5.2.pom
  100.0% [##########] 2.9 KiB (4.4 KiB / s)
https://repo1.maven.org/maven2/org/scoverage/sbt-coveralls_2.12_1.0/1.3.11/sbt-coveralls_2.12_1.0-1.3.11.pom
  100.0% [##########] 3.8 KiB (5.7 KiB / s)
https://repo1.maven.org/maven2/org/scoverage/sbt-scoverage_2.12_1.0/1.8.2/sbt-scoverage-1.8.2.pom
  100.0% [##########] 2.3 KiB (47.4 KiB / s)
...

I would expect to have this same output when running sbt update as part of podman build.
notes
I guess this issue is somewhere between sbt and podman, I open it here as I believe I didn't see the same behaviour with Maven for instance but obviously, I may transfer it to podman/docker if you tell me this has nothing to do with sbt.
In comparison with another unrelated tool, the apt-get commands do give some outputs as I would expect.
To be noted that the same happens with Docker, it's not specific to podman.

i don't know anything about podman or docker, but maybe sbt is (probably by design) behaving differently depending on whether it's connected to a terminal or not. is it possible it's as simple as that?

I guess so indeed. Then the question would be: how to get the full output when not connected to a terminal?

https://www.scala-sbt.org/community.html has advice on where to ask questions

Right, here's the question on StackOverflow then for anyone coming here: https://stackoverflow.com/questions/77541385/no-download-progress-shown-when-running-as-part-of-podman-build-or-docker-bui


Add Girls in Tech Catalyst Conference

Hey @linda-lai, thanks for your contribution 🎉

Ugh, should have squashed those commits, sorry for the noise in the Git history!


Consider enabling EPEL in CentOS & RHEL images

To make more packages available and install by default some missing ones, I'm considering enabling EPEL by default in the CentOS & RHEL images.

Prs for that change:

https://github.com/toolbx-images/images/pull/24
https://github.com/toolbx-images/images/pull/20


Plan to integrate kotlinx.serialization

Right now we’re using JsAdapter to encode/decode parameters and responses. This is pretty unpleasant with boilerplate.
Instead I’d like to structure a replacement around kotlinx.serialization.
Our design idea is that function call:
interface Greeter {
  fun greet(person: Person): Greeting
}

Is marshaled as if we had generated these classes:
@Serializable
class GreeterGreetRequest {
  val isSuspend: Boolean,
  val instanceName: String,
  val funName: String,
  val person: Person
}

@Serializable
class GreeterGreetResult {
  val result: Greeting?,
  val failure: Throwable?
}

We can potentially do that – generate these classes – and let Kotlinx.serialization do the work. Or we could encode as parts. Either way, we’ll require that every parameter and return type is encoded and decoded by kotlinx.serialization and we’ll look those the adapters up statically when we generate code.
We’ll add a SerializersModule property to Zipline and require this to be passed in when it is created.

Implemented.
https://github.com/cashapp/zipline/pull/270


Call Back

Hi,
i have an issue with call back url. can you provide an example with mysql

What sort of issue?  MySQL statements are a little beyond the scope of this helper class, you just need to know how to insert and update into a mysql database.
// we pass the $_POST and $_SERVER variables to alleviate any actual dependencies in the class when testing/troubleshooting.
// in the future, methods will be used within CoinPayments:: to grab the $_POST and $_SERVER variables in order to maintain easy of use
// as well as sound pattern design
try {
if($CP->listen($_POST, $_SERVER))
{
    // The payment is successful and passed all security measures
    // you can call the DB here if you want
   Database::updatePayment($_POST['custom']);

}
else
{
    // the payment is pending. an exception is thrown for all other payment errors.
}
}
catch(Exception $e)
{
    // catch the exception and decided what to do with it. (most likely log it)
}

HI

I HAVE problem to use coinpayment class.php


Log output of ShooterGame?

I think we could log the output of the ShooterGame session, so users can see the log and see what the server is currently displaying.
This will be particularly useful once we'll release 1.1 where screen was replaced by nohup

This would be awesome and allow us to use log output to manipulate the server and/or it's webpage (depending on what the server outputs)

I have tested this command:
nohup "$arkserverroot/$arkserverexec" "$arkserveropts" </dev/null >serverlog 2>&1 &

And it seems to print the entire output of the command inside the serverlog file
an example output is:

OK for me, be keep in mind that nohup must work even if we call the iit
script (I'am saying that because I 've removed screen for that reason)
2015-06-24 13:19 GMT+02:00 Fez Vrasta<EMAIL_ADDRESS>
I have tested this command:
nohup "$arkserverroot/$arkserverexec" "$arkserveropts" </dev/null >serverlog 2>&1 &
And it seems to print the entire output of the command inside the
serverlog file
an example output is:
[S_API FAIL] SteamAPI_Init() failed; SteamAPI_IsSteamRunning() failed.
Setting breakpad minidump AppID = 346110
Using binned.
4.5.1-0+UE4 7038 3077 402 3
CtrlCHandler: Signal=15
CtrlCHandler: Signal=15
It could be used to detect unexpected crashes, in this way you know what
the server printed before the crash.
What do you think?
ping @Sispheor https://github.com/Sispheor
—
Reply to this email directly or view it on GitHub
https://github.com/FezVrasta/ark-server-tools/issues/42#issuecomment-114833393
.

--
Nicolas MARCQ

Implemented in 1.2
by the way the nohup was not added by me, I'm not even sure if it's needed (I think no)

You're right - nohup isn't required in a non-interactive shell (i.e. when running a shell script).


handler for create new window

is there a handler before creating new window?
like this: https://github.com/nwjs/nw.js/issues/1162

ohh.. i found it in web-contents, sorry for open this issue.


Release with tar.gz format instead of only gz format (support Ansible unarchive module)

I try to deploy trickster with Ansible, but Ansible doesn't support .gz release file. From Ansible  unarchive docs:

Can handle .zip files using unzip as well as .tar, .tar.gz, .tar.bz2 and .tar.xz files using gtar.
Does not handle .gz files, .bz2 files or .xz files that do not contain a .tar archive.

So I recommend trickster releases with tar.gz format, please consider this request. Example Ansible code

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

Hi xluffy, thanks for this contribution. I think it makes good sense, as it allows us, at some point in the future, to add additional files to the binary distribution (e.g., docs or example configs subdirectories). We are currently doing most of our work against the next branch (which we hope to merge into master soon), so while I'll merge this PR, I would ask if you wouldn't mind submitting a separate PR with the same update on next too. That way, the change is included in the very next beta release (and all going forward); as an added bonus you'll be credit with another contribution to the project on your github profile. Thanks again!


wpa_supplicant: 2.5 -> 2.6

Motivation for this change
Update
Things done

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

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Could somebody sanity-check this? I removed the libressl patch the applied to 2.5 because it is now built into wpa_supplicant.
I have built and run this but I have never done anything with libressl, so I am not sure anything different could happen with that.

Works for me with libressl. Thanks!


Missing courses in the generated image

Obviously, a lot of courses were missing.
I'm sure that I have copied and pasted the whole table, and the parser in step 1 works well.

Could you please save and upload your MIS timetable page as a HTML file here, so we can have a testing sample.

UIC MIS.zip

The root cause for this bug is the original course merge logic can only handle a two hours session, when the course spans three hours, the time merging logic will returns a incorrect structure list.

What it should look like: [1,2,3]
What it return: [[1,2],3]

The new logic will check the length of the array first and apply different merge logic, and the correct image should generates


tap-hold-next-release with macro does not release last character in some apps

Hey,
I have an alias for Shift to type a deadkey-accessed character.
(defalias
    ň-S (tap-hold-next-release 200 #(RS-RA-6 n) sft)
)

However, e.g. in Discord, this causes "n" to repeat until I press another character.
I workaround I found is to add a delay at the end of the macro.
(defalias
    ň-S (tap-hold-next-release 200 #(RS-RA-6 n P5) sft)
)

This was a bug with tap-macros in tap-hold-next-release. The issue is #872. Your particular case was already addressed in #873.


[WiP] Dev v2.7.1

@stevenh I'll be glad to include it if possible. Can you check my comment on the PR?

Will get that sorted ASAP

Just checked, and replied, let me know how you want to proceed.


Restrict Revocation List allowed URL schemes

Revocation List 2020 doesn't say what is a URL. But examples and test vectors use https. This implementation has been allowing http also. For a more secure default, this PR changes it to allow only https.
We could later add some option or mode for allowing HTTP where it is safe and necessary to do so, and/or other protocols/schemes.
If merged, a change similar to this one should be made in the implementation of Status List 2021 in #278.

hold off til override methinks, lots of people use DIDKit in secure environments/PoCs and would need the override!


Fix image description

This is about mobile first design, not progressive enhancement

Deploy preview for openbuildservice ready!
Built with commit 2ecffd1861243dcaca84f1d1174b2f5c35829888
https://deploy-preview-243--openbuildservice.netlify.com


Ethan: Watch hotmart.com

Ethan requests the watch of the watch_keyword hotmart\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

hotmart\.com has been seen in 2 true positives, 1 false positive, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Formatting error when version has trailing dot

lastPart is minor and version format is x. then output string will be x.
Same if lastPart is patch.

.z and x-range are also work badly


Issue 568: Coverage

This PR should discuss the changes introduced to pyrflx to enable the coverage check for messages.
A boolean flag is added to the _next_field method. This is needed in order to determine when the covered link should be stored, as the next_field method is used during message initialization where the used link must not be stored.
In addition to this, a method is added to return all inner messages of a MessageValue. This is needed so that the coverage information of any nested messages can be obtained.
Closes #568

In a separate discussion we decided to use a boolean flag in _next_field. The alternative solutions seem to make the code more complicated.

We will need an additional method (property) to return __type.structure in the MessageValue class. To calculate the percentage of covered links and print the uncovered links in the validation tool, it must know, which links are defined.

I pushed the commit containing the path-recalculation approach (and its tests) to a new branch and removed it from this branch.
It is now on https://github.com/Componolit/RecordFlux/tree/issue_568_path_recalculation .


Bug 2089327: When trying to add network using network modal when no network found is crashing

Signed-off-by: Matan Schatzman<EMAIL_ADDRESS>
📝 Description
When removing all networks from VM and then trying to add a new network is causing the app to crash. It caused as trying to spread undefined.
🎥 Demo
After:

Before:

/retest

/approve


How do I add a loading indicator to the button while the form is being submitted?

Describe the feature
I'm using FormKit with Nuxt 3 and the schema option and I'd like to add an loading indicator while the form is being submitted.
Form.vue
<FormKit type="form" @submit="handleSubmit" :submit-label="submitLabel">
    <FormKitSchema
        :schema="schema"
    />
</FormKit>

I already have a  component - is it possible to replace the submit button text with the component, while it's being submitted?
Or, is it possible to use my  component as a submit button?
Button.vue
<button v-bind="{ type }" class="btn btn-primary" :class="`btn-${size}`" :disabled="isLoading">
    <span v-if="!isLoading">
        <slot></slot>
    </span>
    <LoadingSpinner v-else />
</button>

Thanks in advance

Easiest way to is to put the form into the schema as well. Then you have direct access to all the context properties (of each immediate parent) https://formkit.link/8876041ff5a822fc4d8e0b60eb376ac6
https://formkit.com/api-reference/context


[Improvement] GUI Window

I think You left some details out :
2020-09-08 22:08:38,735 - root - DEBUG - Creating 4 Tesseract API(s) with 'eng' language. 2020-09-08 22:08:38,953 - root - DEBUG - Creating 2 Tesseract API(s) with 'mff+eng' language. 2020-09-08 22:08:39,371 - threading - ERROR - <class 'ModuleNotFoundError'> No module named 'version'   File "app_gui.py", line 5, in <module>     from lib.updater import Updater   File "C:\Users\coold\Downloads\build\lib\updater.py", line 8, in <module>     import version as old_version_module

Also Might wanna add Debug_[timestamp_when_bot_started] + delete old logs file in order to free up space.

Might need to work more on gui part

Yeah, I forgot to add 'version.py` file to build script.

Good suggestion, thanks! I'll add this feature later
What do you mean, which part?

Highlighted the Part which needed to workaround.
Disable Maximize Screen.
Mission Queue should be full Visible.
Tidy up buttons looks messy.
Also Instead of Trying to Figure out How many Quest have left again and again if I checked all then should store in a variable instead of repeating the same task again.

I don't have much exp in python GUI as bit fan of C#

OK I see.

Username, energy, gold and boost labels currently are turned off (here is why https://github.com/tmarenko/mff_auto/blob/master/lib/gui/widgets/main.py#L74)
Other labels and texts don't fit probably because your 125% scale factor in OS.
This is my first time using PyQt (and python GUI in general), sorry. I'll test GUI with scaling later, but it isn't first priority right now.
Ok
Ok. For now you can hover mouse and see full text.
Same as 1)
The point of checking Content Status Board every time before doing the mission is to found out where is the button for mission located. For example, every time when you fully complete one mission => all other buttons rearrange and change their position.
The other thing is that all tasks is made with Python multiprocessing. Multiprocesses don't share memory. If we store all game modes before task execution, then task process would never reload button's position.
This problem requires redesigning all missions and task execution for progress/result callbacks.
Current behavior is much safer if you force stop mission or mission closes with error.

Scaling doesn't affect the Hidden Labels Already tried that one.
BTW please check GUI:

Looks good but I have worries about "Quick Start" and "Game Settings". There are no space for more buttons/settings if they will come in the future.
I did research about flexible layouts in PyQt, so in the next release all hidden labels should be gone.

For space Issue or future button, Game Settings can take place of QuickStart Group and a new Quick Start group can be added to a new column.
Also, I think Tkinter is fine Take a look at that also.

Took Some Liberty to Create Main Window GUI via QT saved as text just import as a .ui
main_window.txt

Also could please add Captain Marvel Level First in Legendary as I only had that Battle fully upgraded.

Took your .ui and made some tweaks here and there:

Sure, I'll add new Legendary Battle

I think check all button or checkbox list would be fine above missions.

Done in https://github.com/tmarenko/mff_auto/releases/tag/0.5.2

Updater doesn't work if Nox isn't running It would be great to add a Window before main and download only those files which have been updated. and delete previous via bat file and restart the application doesn't have to do manually.
`
2020-09-13 09:54:55,818 - root - DEBUG - Creating 4 Tesseract API(s) with 'eng' language.
2020-09-13 09:54:56,175 - root - DEBUG - Creating 2 Tesseract API(s) with 'mff+eng' language.
2020-09-13 09:54:56,722 - updater - INFO - Loaded version: mff_auto=0.5.0, game=6.3.0, updater=1.0.0
2020-09-13 09:54:58,235 - updater - INFO - Found new version of Updater 1.0.1, cannot update. Download last release from project homepage.
2020-09-13 09:54:59,089 - main - INFO - Team number for TimeLine battles : 2
2020-09-13 09:54:59,089 - main - INFO - Team number for missions : 2
2020-09-13 09:54:59,089 - main - INFO - Acquire Heroic Quest rewards: True
2020-09-13 09:54:59,097 - queue_manager - DEBUG - Loading 12 items to queue list.
2020-09-13 09:54:59,140 - main - INFO - Acquire Heroic Quest rewards: True
2020-09-13 09:54:59,158 - threading - ERROR - <class 'TypeError'>
unsupported operand type(s) for *: 'float' and 'NoneType'
File "app_gui.py", line 24, in
main()
File "app_gui.py", line 10, in main
window = MainWindow()
File "C:\Users\coold\Downloads\build\lib\gui\widgets\main.py", line 82, in init
if not self.game.is_main_menu() and not BattleBot(self.game, None).is_battle():
File "C:\Users\coold\Downloads\build\lib\game\game.py", line 132, in is_main_menu
return self.player.is_ui_element_on_screen(self.ui['TEAM']) and self.player.is_ui_element_on_screen(
File "C:\Users\coold\Downloads\build\lib\game\game.py", line 50, in wrapped
if self.is_loading_circle():
File "C:\Users\coold\Downloads\build\lib\game\game.py", line 143, in is_loading_circle
rects=loading_circle_rects)
File "C:\Users\coold\Downloads\build\lib\players\nox_player.py", line 218, in is_color_similar
positions = [self.get_position_inside_screen_rectangle(rect) for rect in rects]
File "C:\Users\coold\Downloads\build\lib\players\nox_player.py", line 218, in
positions = [self.get_position_inside_screen_rectangle(rect) for rect in rects]
File "C:\Users\coold\Downloads\build\lib\players\nox_player.py", line 170, in get_position_inside_screen_rectangle
return int(rect[0] * self.width), int(rect[1] * self.height)
`


[DO NOT MANUAL MERGE] Release Qualification

Trigger release qualification jobs
none

@silverdev: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-e2e-cluster_wide-auth-default.sh
b5fb77a242ef3488b158b8e880735aecffeacec3
link
/test daily-e2e-cluster_wide-auth-default

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@silverdev: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/daily-e2e-cluster_wide-auth-default.sh
b5fb77a242ef3488b158b8e880735aecffeacec3
link
/test daily-e2e-cluster_wide-auth-default

prow/daily-e2e-cluster_wide-auth.sh
b5fb77a242ef3488b158b8e880735aecffeacec3
link
/test daily-e2e-cluster_wide-auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


GH2763: Added GetParent() method to DirectoryPath and associated test

I've implemented a function as discussed in (#2763). To do this I utilized System.IO.DirectoryInfo and it's Parent property. All Cake.Core Tests including the new test specific to this method passed before and after final commit.

Something like that should work, yes! I'll get to it later today 😊
On Sun, May 16, 2021, 7:47 PM Patrik Svensson @.***>
wrote:

@.**** commented on this pull request.
Left a minor comment, otherwise, it looks good to me 👍
@augustoproiete https://github.com/augustoproiete We should probably
think about whether or not we want to bring this into 1.2.0 since it's an
ABI break. Perhaps this PR would be a better candidate for Cake 2.0?
In src/Cake.Core/IO/DirectoryPath.cs
https://github.com/cake-build/cake/pull/3349#discussion_r633156499:

       if (path == @"\\")

       {

           return null;

       }

Couldn't we check the segment length instead before combining the
segments? If there are no parents, segments will be 0, right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cake-build/cake/pull/3349#pullrequestreview-660478540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABONUF7SQEALNMBPTQO3S6TTOBKRPANCNFSM44UA6R7A
.

@LagunaElectric If you haven't seen it, I used an adaptation of your work in https://github.com/cake-build/cake/discussions/3387 - nothing about UNC paths, though.

Yes, sir! Life has been very busy recently, my apologies. I'll try to get this done by the end of the month!

please integrate this

Gentle ping @LagunaElectric are you still working on this?
I'd really like to see this in one of the next releases. 😁
So - if you don't have the resources right now, I can help out with the points I addressed above. Only if that's fine with you, of course.

Hey @nils-a! Yes feel free! I've had a kid and a new job so life is a bit hectic at the moment 😄. I appreciate the assist! 🥰

Great work!

@LagunaElectric your changes have been merged, thanks for your contribution :+1:

Cheers! Sorry for the delay 😅


Add helusers integration

Add helusers / tunnistamo integration to project.
Refs #LIIK-89

Codecov Report

Merging #55 into master will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #55      +/-   ##
==========================================
- Coverage   94.62%   94.56%   -0.06%
==========================================
  Files          60       60
  Lines        3829     3793      -36
  Branches      121      121
==========================================
- Hits         3623     3587      -36
  Misses        192      192
  Partials       14       14

Impacted Files
Coverage Δ

city-infrastructure-platform/urls.py
86.66% <ø> (ø)

traffic_control/admin.py
96.93% <ø> (-0.02%)
:arrow_down:

city-infrastructure-platform/settings.py
80.82% <100.00%> (+3.04%)
:arrow_up:

users/models.py
100.00% <100.00%> (ø)

azure_client_ip/middleware.py
68.75% <0.00%> (-1.84%)
:arrow_down:

traffic_control/views.py
97.86% <0.00%> (-0.15%)
:arrow_down:

traffic_control/models/mount.py
99.19% <0.00%> (-0.01%)
:arrow_down:

traffic_control/models/traffic_sign.py
99.33% <0.00%> (-0.01%)
:arrow_down:

azure_client_ip/test_middleware.py
100.00% <0.00%> (ø)

traffic_control/tests/test_mount_api.py
100.00% <0.00%> (ø)

... and 8 more

Continue to review full report at Codecov.

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


Allow for inMemoryPersistence

Add an option to have inMemoryPersistence for React Native, since browser persistence causes errors.
My solution is not the most elegant, but would preserve the default option and only use inMemoryPersistence when remember is explicitly set to false.

persistence storage fixed according to the platform


Tool relies on an existing nginx container running on the target server

This either needs to be mentioned in the doco, and specified in detail what the requirements are for the tool to be able to configure it.
OR
The config file/release file, if specifying a proxy, should allow you to pull a specified nginx container to run as sidecar.

This issue makes hx_terraform somewhat of a dependency.

I'm not sure what you mean here. hx- deploy-tool starts and manages an nginx proxy in the frontend_proxy container. But it doesn't require one to be existing on the machine?

So confusion do to differences between local and remote state tracking with proxy.
Should add doco to clarify how proxy is used, and when.
i.e. pulls and runs default nginx container only when required, for example when you connect a release to an endpoint.


BufferedSink.writeAll doesn't work well

Java code:
    String src =
        "https://fanyi.baidu.com/gettts?spd=3&lan=jp&text=%E3%81%93%E3%82%93%E3%81%AB%E3%81%A1%E3%81%AF";

    inputStream = new URL(src).openStream();
    outputStream = new FileOutputStream(dst);

    Source source = Okio.source(inputStream);
    BufferedSink buffer = Okio.buffer(Okio.sink(outputStream));
    long total = buffer.writeAll(source);

    System.out.println("total: " + total);
    System.out.println("length: " + dst.length());

Android logcat:
    total: 7344
    length: 0

You are not closing the buffered sink and therefore the buffered writes are not being propagated to the underlying sink.

Thx for replying.
Try the following:
    String src = "https://fanyi.baidu.com/gettts?spd=3&lan=cte&text=%E4%BD%A0%E5%A5%BD";
    ...

Then you will see:
    total: 9504
    length: 9504


Result Page JS file

[ ] When "Get Result" is clicked, Result page is then shown
[ ] Display how many votes selected the movie
[ ] Utilize external movie API to retrieve data on Title, Release Date, Cast, Director, Plot, any rating is available
[ ] Should save information to History Page (localStorage)

Due to plan changed this page is no longer needed - will replace by Favorites page


add community presets

i has  a project unocss-preset-scalpel can add to readme  community presets ?

I have received your email but somehow I can't reply back to you. I have checked your preset and I think it's an interesting idea! Feel free to PR add your preset to readme.

thanks !


F#2806 exploration catalog css bug

Description

exploration css bug
Related Issue :
#2806

How Has This Been Tested?

Exploration -> Data
In LNB check selected catalog's bold feature works properly
Check folding icon toggle properly
Check folding icon's tooltip shows properly
check New or Update and My Favorite list shows properly

Need additional checks?
Types of changes

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

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 4

@thinkcsww

There is a console error.

please change folding button's tooltip
"Fold" / "Unfold"

@ufoscw

I think the reason of error is there is no tags in api result.
Can you check this please?

@thinkcsww
The API contains tags.
Untagged data may exist.
Please enter the defense code for the untagged data.

@ufoscsw
Modified

deploy to 4

@thinkcsww
same console error occurred in "my favorite" list.

@ufoscw
Sorry, Modified again

deploy to 4


vaultwarden-postgresql: 1.30.1 -> 1.30.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for vaultwarden-postgresql is: Unofficial Bitwarden compatible server written in Rust
meta.homepage for vaultwarden-postgresql is: https://github.com/dani-garcia/vaultwarden
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.30.2 with grep in /nix/store/lxc108mshyqskpzh54ajy9dym020ffcp-vaultwarden-1.30.2
found 1.30.2 in filename of file in /nix/store/lxc108mshyqskpzh54ajy9dym020ffcp-vaultwarden-1.30.2

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

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
nixos-install-tools
tests.nixos-functions.nixos-test
vaultwarden
vaultwarden-mysql
vaultwarden-postgresql
vaultwarden-sqlite

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lxc108mshyqskpzh54ajy9dym020ffcp-vaultwarden-1.30.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A vaultwarden-postgresql https://github.com/r-ryantm/nixpkgs/archive/b05f7a0c3bec9e3a77f7a560080265ec524e1afe.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/b05f7a0c3bec9e3a77f7a560080265ec524e1afe#vaultwarden-postgresql

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lxc108mshyqskpzh54ajy9dym020ffcp-vaultwarden-1.30.2
ls -la /nix/store/lxc108mshyqskpzh54ajy9dym020ffcp-vaultwarden-1.30.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package blacklisted:

nixos-install-tools

3 packages built:

vaultwarden
vaultwarden-mysql
vaultwarden-postgresql

Maintainer pings
cc @msteen @ivan for testing.

https://github.com/NixOS/nixpkgs/pull/285225

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for vaultwarden-postgresql is: Unofficial Bitwarden compatible server written in Rust
meta.homepage for vaultwarden-postgresql is: https://github.com/dani-garcia/vaultwarden
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.30.3 with grep in /nix/store/xyvsdfhwsgd28ax19id3wnipa5zpp7c7-vaultwarden-1.30.3
found 1.30.3 in filename of file in /nix/store/xyvsdfhwsgd28ax19id3wnipa5zpp7c7-vaultwarden-1.30.3

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

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
nixos-install-tools
tests.nixos-functions.nixos-test
vaultwarden
vaultwarden-mysql
vaultwarden-postgresql
vaultwarden-sqlite

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/xyvsdfhwsgd28ax19id3wnipa5zpp7c7-vaultwarden-1.30.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A vaultwarden-postgresql https://github.com/r-ryantm/nixpkgs/archive/a0337bd86806e0334e366f9c1cc9ae96ad40e749.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/a0337bd86806e0334e366f9c1cc9ae96ad40e749#vaultwarden-postgresql

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/xyvsdfhwsgd28ax19id3wnipa5zpp7c7-vaultwarden-1.30.3
ls -la /nix/store/xyvsdfhwsgd28ax19id3wnipa5zpp7c7-vaultwarden-1.30.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package blacklisted:

nixos-install-tools

3 packages built:

vaultwarden
vaultwarden-mysql
vaultwarden-postgresql

Maintainer pings
cc @msteen @ivan for testing.


New page at /rental-property-location

@sulthan-ahmed do you foresee  problem here?

I don't foresee a problem.  I think it looks good, just a minor suggested change.


Tokenize ShimmerView

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
Tokenization of ShimmerView. Note that this change changes APIs and is thus a breaking change.

ios/FluentUI.Demo/FluentUI.Demo/Demos.swift

Move ShimmerView demo controller to vNext section

ios/FluentUI.Demo/FluentUI.Demo/Demos/ShimmerLinesViewDemoController.swift

Add new parameters to ShimmerView and ShimmerLinesView inits

ios/FluentUI.xcodeproj/project.pbxproj

Add ShimmerTokens file

ios/FluentUI/Shimmer/ShimmerLinesView.swift

Add lineCount, firstLineFillPercent, and lastLineFillPercent to ShimmerLines init
Update to use tokens where available

ios/FluentUI/Shimmer/ShimmerTokens.swift

Add tokens for Shimmer and ShimmerLines

ios/FluentUI/Shimmer/ShimmerView.swift

Move ShimmerStyle to ShimmerTokens
Add shimmerStyle, shimmersLeafViews, and usesTextHeightForLabels to Shimmer init
Update to use tokens where available

Before
After

https://user-images.githubusercontent.com/31874971/173157760-a5348f0f-c079-4e50-9758-c4b87f79100f.mp4
https://user-images.githubusercontent.com/31874971/173157570-7ac11d4e-d972-48ab-b7df-1b2de91ee3be.mp4

Pull request checklist
This PR has considered:

[x] Light and Dark appearances
[x] iOS supported versions (all major versions greater than or equal current target deployment version)
[x] VoiceOver and Keyboard Accessibility
[x] Internationalization and Right to Left layouts
[x] Different resolutions (1x, 2x, 3x)
[x] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
[ ] iPad Pointer interaction
[ ] SwiftUI consumption (validation or new demo scenarios needed)
[ ] Objective-C exposure (provide it only if needed)

Microsoft Reviewers: Open in CodeFlow
Microsoft Reviewers: Open in CodeFlow

@harrieshin should we have an item for FURN to match tokens?

@harrieshin should we have an item for FURN to match tokens for an eventual "v1" shimmer ?

what the hell is a furn


ARROW-11343: [DataFusion] Simplified example with UDF.

I.e. use collect instead of the builder, which is simpler and faster

Codecov Report

Merging #9290 (4e5220e) into master (c413566) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9290   +/-   ##
=======================================
  Coverage   81.61%   81.61%
=======================================
  Files         215      215
  Lines       52508    52508
=======================================
  Hits        42854    42854
  Misses       9654     9654

Continue to review full report at Codecov.

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

Codecov Report

Merging #9290 (4e5220e) into master (c413566) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #9290   +/-   ##
=======================================
  Coverage   81.61%   81.61%
=======================================
  Files         215      215
  Lines       52508    52508
=======================================
  Hits        42854    42854
  Misses       9654     9654

Continue to review full report at Codecov.

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


[bitmanip] [zbe] bit deposit and bit extract (bdep/bext) instructions

The zbe instruction group implements the generic bit extract and bit deposit functions. bext collects LSB justified bits to rd from rs1 using extract mask in rs2. bdep writes LSB justifie dbits from rs1 to rd using deposit mask in rs2.
The implementation is planned to base upon the butterfly network as described in Hilewitz et al..
Similar reference implemnetations by Claire Wolf indicate a rather large increase in area caused by this architecture. There are several possibilities to trade some latency for a smaller core, but the benefits are limited. However, in a complete RV32B implementation there are several functionalities, which could share the butterfly network, and therefore somewhat mitigate the area increase. For example it should be possible, to reuse zbe structures for bitcount instructions (ctz, clz, pcnt), single-bit instructions (zbs), bit ppermutation (zbp), pack, rotate and cmix instructions.
The plan is to first implement the butterfly network just for bdep and bext. At that point we will see how bad the area increase is. I will then try to incorporate as many of the existing instructions as possible into the structure.

This plan makes sense to me @ganoam . It will be very interesting to see how much can be shared with other instructions.

PR #863 is ready.

This has been merged.


merge select_experiment function into pysweep

pysweep uses:
from pytopo.qctools.dataset2 import select_experiment.
On import the following warning occurs:
C:\ProgramData\Anaconda3\lib\site-packages\qcodes\dataset\sqlite_base.py:51: UserWarning: The module `qcodes.dataset.sqlite_base` is deprecated.
Public features are available at the import of `qcodes`.
Private features are available in `qcodes.dataset.sqlite.*` modules.
  warnings.warn('The module `qcodes.dataset.sqlite_base` is deprecated.\n'

This concerns the functions
from qcodes.dataset.sqlite_base import transaction, one
which moved now to:
from qcodes.dataset.sqlite.queries import one
from qcodes.dataset.sqlite.connection import transaction
A fix is to take care of this in pytopo. However, then one still depends on pytopo.
Therefore, the preferred solution would be to move the function into pysweep/databackend (or something), which would get rid of the deprecation warning and remove the dependency on pytopo
from qcodes.dataset.experiment_container import load_experiment_by_name, \
    new_experiment

def select_experiment(exp_name, sample_name):
    """
    Convenience function that will check if the experiment/sample
    combination already exists in the current database. If so,
    it'll return the existing one. Otherwise it will create a new
    one and return that.

    Potential issue: if multiple experiments with the same
    experiment/sample combination exist, our detection method will
    fail, and another copy of this combination will be created.
    """
    try:
        exp = load_experiment_by_name(exp_name, sample_name)
    except ValueError:
        exp = new_experiment(exp_name, sample_name)
    return exp

... the preferred solution would be to move the function into pysweep/databackend
@LGruenhaupt, sorry, no :) The preferred solution is to use a feature from qcodes that has been there for already long time and does exactly the same, here is the doc about it.

But thanks for calling this out. unfortunately removing these functions from pytopo as noted in this issue on the pytopo repo is still pending.

I will update the example notebook after PR #19 is merged

@LGruenhaupt, sorry, no :) The preferred solution is to use a feature from qcodes that has been there for already long time and does exactly the same, here is the doc about it.
But thanks for calling this out. unfortunately removing these functions from pytopo as noted in this issue on the pytopo repo is still pending.

@astafan8
Fair enough, this is indeed a much better solution :+1:


XAI_MODEL isn't used anywhere

Describe the bug
I can see XAI_MODEL in docs here and there, but from what I can find here https://github.com/search?q=org%3Aai16z+XAI_MODEL&type=code, there's no code using it at all.
Can anyone from the team confirm it? If so, can we remove it from the docs? As it's kind of confusing to users.

To Reproduce

Expected behavior

Screenshots

Additional context

I assume that XAI_MODEL is there to define any custom LLM (for example meta-llama/Meta-Llama-3.1-70B-Instruct-Turbo). The model will be automatically downloaded from HuggingFace.


Can i add more than 5 items?

How can i change the maximum of items from 5 to 6 or seven ?

you can't.
https://material.io/guidelines/components/bottom-navigation.html#