CLA Assistant doesn't register agreement correctly

URL to the linked Repository or Organization
https://github.com/tldr-pages/tldr/
Steps to reproduce the problem

Read information about contributing to the project: https://github.com/tldr-pages/tldr/blob/main/CONTRIBUTING.md
Click on sign (1st paragraph of page linked above): https://cla-assistant.io/tldr-pages/tldr
Submit pull request: https://github.com/tldr-pages/tldr/pull/9749
Verify status

What is the expected result?
CLA shown as signed.
What happens instead?
CLA is shown as pending:

Any other information? (attach screenshot if possible)
I can sign and have signed several times.

The first time I sign, I get a page with a button to sign, and after clicking I see a brief message "Thank you blueglyph for signing the CLA!" that redirects to the main repo page.
If I try shortly after, the sign page shows "You have agreed to the CLA for tldr-pages/tldr"
If I wait long enough (I don't know how long - one day maybe), I can sign again.

The status hasn't been updated since the creation of  the pull request, 8 January 2023 (despite GitHub showing that as "last year"...).
See the PR that I initially created on their repository for this issue: https://github.com/tldr-pages/tldr/issues/9821
Used browser and browser version?
Chrome, latest version (Version 109.0.5414.120 (Official Build) (64-bit))

Thanks for the detailed issue description. The problem here is that you have used your local git user for some commits and neither GitHub nor CLA assistant can recognise to whom do these commits belong, thus the message from CLA assistant:
redglyph 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](https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/#commits-are-not-linked-to-any-user).
CLA assistant requires every committer of the PR to sign the CLA in order to set the status to green/success

once you have added the email address of your local git user to your GitHub account, you can hit the "recheck" link at the end of the CLA assistant's comment.

I see, that's surprising, I thought it would use the account of the GitHub repo the request was coming from.
There's nothing I can do unfortunately, I'm not giving my actual email address for privacy reasons so I'm using "redglyph@localhost" which is a common pattern. This address is not accepted by GitHub when I try to add it in the settings. Maybe I'll change it to another fake address in case I happen to participate to another project using CLA.
Not a big problem in this specific case though, someone has vetoed the modification with a fake request for change, so it has little chance of going further.

Anyway, thanks for clarifying that to me, @KharitonOff, I see I missed that message from CLA assistant in the pull request.


Introduce property shortenTransactionManagerIdIfNecessary with default false

Solves #140
Update to #132/#136/quarkusio/quarkus#30491

For MariaDB I can confirm that the shorten nodename is causing an issue in command XA START. Generated HEX String is too long > 64 characters.
@geoand: Maybe hashing approach has drawback also for Quarkus. Should be verified!

So if I understand correctly, you are changing the default behavior to not shorten the ID? Why this change of default (which is a breaking change)?

This feature was introduced in the last version 3.1.0 as a back port from Quarkus where it is disabled by default. So I do just add what should be there already. But it seems that this feature has drawbacks anyway in Spring boot and Quarkus which should be evaluated. 😉

Understood, thanks

Well, It seems the patch at Quarkus is wrong. SHA-224 creates a 28 sized byte array but after creating a UTF-8 String from it the corresponding byte array is somewhere around size 40-60 much bigger than 28 :-/

Thanks for investigating.
Please open a relevant issue in the Quarkus repo

In the meantime I would suggest to revert this feature for a 3.1.1 patch release until a better fix is found.
@geoand : okay for you?

Sounds good to me

@geoand : Revert #136 as discussed.
@jacobdotcosta: Could you plz do a 3.1.1 patch release after merge?

I'll release 3.1.1.

Looks good to me, but could really test it tomorrow, okay for you?

@jacobdotcosta: build and tested successfully. Ready to release! ;-)

@jacobdotcosta thx, but don't miss the scm tag 😉

fixed :(


updating phonegap to ionic 3 app making localStorage clear

Bug Report
Updating phonegap app to ionic 3 app. New app making localStorage clear.
Ionic Info
Run ionic info from a terminal/cmd prompt and paste the output below.
Ionic:

   ionic (Ionic CLI)  : 4.1.1 (C:\Users\Ganesh\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework    : ionic-angular 3.9.2
   @ionic/app-scripts : 3.2.0

Cordova:

   cordova (Cordova CLI) : not installed
   Cordova Platforms     : android 7.0.0
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.2, cordova-plugin-ionic-webview 2.1.0, (and 12 other plugins)

System:

   Android SDK Tools : 26.1.1 (C:\\Users\\Ganesh\\AppData\\Local\\Android\\sdk)
   NodeJS            : v8.11.3 (C:\Program Files\nodejs\node.exe)
   npm               : 5.6.0
   OS                : Windows 10

Describe the Bug
Updating phonegap app to ionic 3 app making localStorage clear. so i'm not able to get previous values stored by phonegap app with localStorage. (note: i'm not clearing localStorage anywhere in my app)
Steps to Reproduce
Steps to reproduce the behavior:

Make any phonegap app
Use localStorage to save something, like ('login','true') and install app on mobile device
Make ionic 3 app with same id, use same localStaorage to retrive data .
log or alert data you will get null instead of some vlaue

Expected Behavior
When we update the app , app must persist the localStorage. ionic should not clear the app storage.
We must able to use previously store data.

Ionic 3 has it's own repository -> https://github.com/ionic-team/ionic-v3
This issues are only related to Ionic 4


Failed to retrieve the ms-Mcs-AdmPwd

Hi,
laps_password lookup is giving error "An unhandled exception occurred while running the lookup plugin 'laps_password'. Error was a <class 'ansible.errors.AnsibleError'>, original message: Failed to retrieve the ms-Mcs-AdmPwd attribute for MACHINE01"
-------Playbook-------------

name: get the LAPS managed password for a host
debug:
msg: "{{ lookup('laps_password', 'MACHINE01', domain_server='dc01.example.com'<EMAIL_ADDRESS>password='test@123', search_base='OU=Test,OU=Workstation,OU=Testing,OU=Computers,OU=Objects,DC=dc01,DC=example,DC=com') }}"

While this plugin is really only designed as a test one, the user must have the correct permissions on the object to be able to retrieve the password. Have a read through the section Permission to read the password from AD at https://blogs.msdn.microsoft.com/laps/2015/06/01/laps-and-password-storage-in-clear-text-in-ad/.

As well as this, all traffic sent sent and received by that plugin is in plaintext. It should not be used in a production environment unless it is configured over LDAPS instead of just LDAP.

I also recommend you look at https://github.com/ansible/ansible/pull/52012  which is a PR that implements more of the secure LDAP features that protect the credentials going over the network. I'm hoping to have that included in the Ansible 2.8 release bu there are no guarantees there.

https://github.com/ansible/ansible/pull/52012 has been merged, please use that instead of this testing version.


[Bug] Can't change client configuration (e.g. request network timeout)

OrientDB Version: 3.0.21
Java Version: 1.8.0_211
OS: Windows 10
Expected behavior
Can change client configuration such as network request timeout
Actual behavior
Changing the configuration option and passing it to the db has no effect.
Steps to reproduce
Building config:
return OrientDBConfig.builder()
        .addConfig(OGlobalConfiguration.DB_POOL_MIN, properties.getMinPoolSize())
        .addConfig(OGlobalConfiguration.DB_POOL_MAX, properties.getMaxPoolSize())
        .addConfig(OGlobalConfiguration.NETWORK_REQUEST_TIMEOUT, properties.getNetworkRequestTimeout())
        .build();

Building database:
database = new OrientDB(properties.getUri(), properties.getUsername(),
        properties.getPassword(), config);

The property is ignored though. As far as I can tell the problem is:
(1) The flow is OServerAdmin will create remote storage, which will create connection pool. All client config will come from the OServerAdmin
(2) OServerAdmin will pass null to the configuration
https://github.com/orientechnologies/orientdb/blob/3.0.21/client/src/main/java/com/orientechnologies/orient/client/remote/OServerAdmin.java#L70
and
https://github.com/orientechnologies/orientdb/blob/3.0.21/client/src/main/java/com/orientechnologies/orient/client/remote/OServerAdmin.java#L81
Hence the config will always be null, and will always use the default values.

Any news on this? this is currently a really huge problem in my app.


API documentation generation

Having a built-in REST API documentation builder could be a nice-to-have feature, although I acknowledge it might be out of scope (maybe as some sort of add-on?).
Some may be familiar with documentation explorers like Swagger UI, which uses the OpenAPI spec. Is there any interest in making something like this a built-in feature?

Precursor: Generate OpenAPI schemas. I'd suggest we just do that directly (plain old dicts etc... rather than relying on any library. The only place where we might want to expand on that would be the types. (Eg. apistar has a type system that is designed to map nicely onto JSONSchema which is useful here)
I'd suggest using apistar's schema validation to make sure we're doing so correctly... https://docs.apistar.com/schema-validation/
Once we're generating OpenAPI schemas then we can take our pick of API docs generators. We could lean on apistar.docs(schema) here if wanted - https://docs.apistar.com/api-documentation/

this is now built


Mi 11 Ultra Not Working Issue

So everything appears to work fine. It allows me to select full screen nav gestures in the settings now, but nothing happens upon swipe sadly. I tried to get some logs from the system, but don't know how helpful they'd be. If you want any other logs from another part of the system let me know. Ps: I'm running Xiaomi.EU MIUI 12 ROM.
Btw, thank you for being a hero and making this mod!

What's your Xposed hook framework and your detailed MIUI versio? This module is only tested in MIUI 13 and LSPosed.

Recent task gesture works, while back gestures won't. If you can't activate recent task by swiping from the buttom screen, try using quick switch and settings lawnchair as the provider.

Hey there! Thanks for reaching out :)
Lsposed version: 1.6.5 (6272)
MIUI: 12.5.20 on Xiaomi.EU ROM
I did see you mention that the back function doesn't work, but I wasn't
able to get any actual function, although I can change the settings to
remove the nav bar and stuff like that with you mod.
On Thu, Apr 21, 2022, 3:03 PM mahoshojoHCG @.***> wrote:

Recent task gesture works, while back gestures won't. If you can't
activate recent task by swiping from the buttom screen, try using quick
switch and settings lawnchair as the provider.
—
Reply to this email directly, view it on GitHub
https://github.com/HCGStudio/FuckMIUIGesture/issues/1#issuecomment-1105646088,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYLVQPHN4QEQAUZMBL7TPQTVGGRATANCNFSM5TT36GUA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you also attch logs of LSPosed about this mod?

Here ya go
On Thu, Apr 21, 2022, 3:18 PM mahoshojoHCG @.***> wrote:

Can you also attch logs of LSPosed about this mod?
—
Reply to this email directly, view it on GitHub
https://github.com/HCGStudio/FuckMIUIGesture/issues/1#issuecomment-1105660645,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYLVQPESMOFZYD3CCID5XVTVGGS2BANCNFSM5TT36GUA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hello, I can't see any log attached. You can upload files in GitHub or just paste them here.

Hmm it should have attached as attachment...let me try again.
On Fri, Apr 22, 2022, 1:14 AM mahoshojoHCG @.***> wrote:

Hello, I can't see any log attached. You can upload files in GitHub or
just paste them here.
—
Reply to this email directly, view it on GitHub
https://github.com/HCGStudio/FuckMIUIGesture/issues/1#issuecomment-1106018522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYLVQPDGU3O7AHCRNJZFBJTVGIYT3ANCNFSM5TT36GUA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Let's try this:
https://drive.google.com/file/d/1-tEAyzlT8m-EvlYRcapjy2jdRnd2W44V/view?usp=drivesdk
On Fri, Apr 22, 2022, 6:51 AM Benjamin Moody @.***> wrote:

Hmm it should have attached as attachment...let me try again.
On Fri, Apr 22, 2022, 1:14 AM mahoshojoHCG @.***>
wrote:

Hello, I can't see any log attached. You can upload files in GitHub or
just paste them here.
—
Reply to this email directly, view it on GitHub
https://github.com/HCGStudio/FuckMIUIGesture/issues/1#issuecomment-1106018522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYLVQPDGU3O7AHCRNJZFBJTVGIYT3ANCNFSM5TT36GUA
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm going to try this on my PC because it keeps saying "Message was
deleted" after sending on my phone...
On Fri, Apr 22, 2022 at 1:14 AM mahoshojoHCG @.***>
wrote:

Hello, I can't see any log attached. You can upload files in GitHub or
just paste them here.
—
Reply to this email directly, view it on GitHub
https://github.com/HCGStudio/FuckMIUIGesture/issues/1#issuecomment-1106018522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYLVQPDGU3O7AHCRNJZFBJTVGIYT3ANCNFSM5TT36GUA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Benjamin Moody

Let me know if you get that. Gmail keeps deleting messages automatically
for no reason...
On Fri, Apr 22, 2022 at 6:53 AM Benjamin Moody @.***> wrote:

Let's try this:
https://drive.google.com/file/d/1-tEAyzlT8m-EvlYRcapjy2jdRnd2W44V/view?usp=drivesdk
On Fri, Apr 22, 2022, 6:51 AM Benjamin Moody @.***> wrote:

Hmm it should have attached as attachment...let me try again.
On Fri, Apr 22, 2022, 1:14 AM mahoshojoHCG @.***>
wrote:

Hello, I can't see any log attached. You can upload files in GitHub or
just paste them here.
—
Reply to this email directly, view it on GitHub
https://github.com/HCGStudio/FuckMIUIGesture/issues/1#issuecomment-1106018522,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYLVQPDGU3O7AHCRNJZFBJTVGIYT3ANCNFSM5TT36GUA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Benjamin Moody

OK, got the logs. It seems that the module loaded successfully without error.

Try the following steps:

Disable the module and restart System UI.
Execute settings put global force_fsg_nav_bar 1 in adb shell or su in Termux.

If you still can't activate gesture, the module can do nothing to activate gesture for you.

Thank you for your response. I did in fact try that and the bottom button
navigation disappears after code execute, but doesn't actually allow swipe
gestures. It's alright though! I really appreciate you going through this
with me :)
On Fri, Apr 22, 2022, 7:39 AM mahoshojoHCG @.***> wrote:

Try the following steps:

Disable the module and restart System UI.
Execute settings put global force_fsg_nav_bar 1 in adb shell or su
in Termux.

If you still can't activate gesture, the module can do nothing to activate
gesture for you.
—
Reply to this email directly, view it on GitHub
https://github.com/HCGStudio/FuckMIUIGesture/issues/1#issuecomment-1106429831,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYLVQPEEKS3YPBR5ISJ3GITVGKFYBANCNFSM5TT36GUA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Judgement counter and stuff

let the tean review this before adding

review start

Megalo wants me to push this but i cant due to conflicts


[Spike]Research Rook Operator Ceph cluster backup and restore

Is your feature request related to a problem? Please describe.
We don't have any information about backup and restore procedure for Rook Ceph Operator.
Describe the solution you'd like
We should check how to handle backup/restore of Ceph Cluster (data).
Describe alternatives you've considered
No alternatives.
Additional context
No additional context.

In case of rook ceph cluster will crash, we would like to have data backup of our application/database which use pvc, completely independent from the ceph cluster. After a deep research and tests, I have to confirm that it is no possible to do it using volume snapshots in kubernetes.
Rook Ceph support only two CSI Drivers: RBD CSI driver and CephFS CSI driver. Please note that the source PVC, PV, and VolumeSnapshotClass for a VolumeSnapshot object must point to the same CSI driver. I was trying to test it and find any opportunity to do it but without any success.
Volume snapshots works great using for example AzureDisk CSI for "plain" PVs and snapshots as well but in our case when I would like to do it with Rook Ceph Cluster and  PVc created with RBD/CephFS CSI Driver, error appears:  GRPC error: rpc error: code = InvalidArgument desc = could not get resource group from diskURI(0001-0009-rook-ceph-0000000000000001-14dc1b34-e7cc-11ea-891b-6ac1584e0dfd) with error(invalid disk URI: 0001-0009-rook-ceph-0000000000000001-14dc1b34-e7cc-11ea-891b-6ac1584e0dfd)
Summing up, in order to make a data backup in rook ceph , we can not use for that volume snapshots. It is no possible right now. There is one rook ceph backup tool which we should take into account link. There is always possible to make a snapshot on the cloud provider level of the entire data disk, need consider.

Backup/Restore is possible using RBD kind of storageclass that PVC uses.
Once this is satisfied we need to do the following steps:

Create toolbox pod with rbd tools installed (example yaml file it is located in rook repo examples)
Make some changes on the pod you want to backup (within the mounted pvc location)
Connect rook-ceph-tools container
Check poolname that was created for the storage class that your PVC uses.
rados lspools
Check volume name
rbd ls replicapool
Create snapshot
rbd snap create replicapool/csi-vol-365399b7-e79e-11ea-a8cc-66630cbb1481@test1-snapshot
Make some more changes in the pod.
Scale down application
kubectl scale --replicas=0 deployment/POD_NAME
Rollback from snapshot (again within toolbox container)
rbd snap rollback replicapool/csi-vol-365399b7-e79e-11ea-a8cc-66630cbb1481@test1-snapshot
Expected result:

1ea-a8cc-66630cbb1481@test1-snapshot
Rolling back to snapshot: 100% complete...done.

Scale up application
kubectl scale --replicas=1 deployment/POD_NAME

Changes should be rolled back

Export / Import tested and seems to be working fine.
Created ceph cluster on 2 separate environments: AWS and AZURE.
Exported changed data in postgresql pod using PVC with Ceph block storage class.
Imported the data on the other cluster and observed successfully changed data on the other cluster.

Nothin to do here more, tomorrow demo will be presented.


[PDI-15059] Spoon startup: Multiple and different slf4j-log4j jar fil…

…es in the class path

downgraded slf4j to 1.7.7 to be consistent across all the Pentaho modules

Build Completed
:white_check_mark: This pull request has passed all validations.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/kettle-json-plugin/build.xml clean-all resolve jacoco && ant -f plugins/kettle-json-plugin/build.xml checkstyle publish-local
ant -Dtestreports.dir=bin/reports/unit-test -f plugins/pdi-pur-plugin/build.xml clean-all resolve jacoco && ant -f plugins/pdi-pur-plugin/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  assembly/ivy.xml
  pdi-engine/serializers/pom.xml
  plugins/kettle-json-plugin/ivy.xml
  plugins/pdi-pur-plugin/ivy.xml

@pamval @mbatchelor @duarteteixeira please take a look


ISPN-6482 Allow setup only truststore in HR client config

https://issues.jboss.org/browse/ISPN-6482

Thanks, I've pushed it.


feat: proper archspec detection using archspec-rs

Implements proper archspec detection using https://github.com/prefix-dev/archspec-rs which is a port of https://github.com/archspec/archspec used by conda. This opens the doors to build and use architecture specific packages.
We loose the __archspec virtual package for wasm targets because the archspec standard does not describe these targets.
I set the default archspec for osx-arm64 to m1 because I think thats greatest common denominator for arm based chips on mac.
I also added a subcommand to rattler-bin to print the detected virtual packages.

On my Windows AMD machine:
> cargo run -- virtual-packages
__win=0=0
__archspec=1=x86_64_v3

WSL under windows AMD:
> cargo run -- virtual-packages
__unix=0=0
__linux=<IP_ADDRESS>=0
__glibc=2.35=0
__archspec=1=zen2

Interestingly WSL and normal windows report a different archspec (zen2 vs x86_64_v3), but conda info reports the same thing on this machine.

On my Fedora Linux 37 Workstation, (CPU: 12th Gen Intel i7-12800H (20)) :
> cargo run -- virtual-packages
__unix=0=0
__linux=6.5.12=0
__glibc=2.36=0
__cuda=12.3=0
__archspec=1=skylake

This is what I get :)
__unix=0=0
__osx=14.4=0
__archspec=1=m2

Nice! Then Ill mark this ready for review!

What I got on my Windows on ARM machine
__win=0=0
__archspec=1=aarch64

The repo is big with git-lfs files (the test-data) and took me a hour to finalize the clone to build and run it. I have no idea why, I'm probably encountering issues of git-lfs on Win on ARM.


Feature Request: LangChain

LangChain (code and docs) is a natural language processing (NLP) library that can understand code using an LLM such as OpenAI.
Here are three code-related LangChain use cases:
Code Generation
One way to use LangChain for code generation is to create templates or prompts that can be used to guide the generation process. For example, you can create a template that asks for input from the user, and then use LangChain to generate the code based on that input.
Here's an example of how you can use LangChain to generate a simple HTML page based on user input:
import { OpenAI } from "langchain";

const model = new OpenAI({ temperature: 0.7 });

const template = `
<!DOCTYPE html>

    <title>{title}</title>

{heading}

{paragraph}

`;

async function generateHtmlPage(title: string, heading: string, paragraph: string) {
  const prompt = template
    .replace("{title}", title)
    .replace("{heading}", heading)
    .replace("{paragraph}", paragraph);

  const res = await model.call(prompt);
  return res.res;
}

In this example, we've created an HTML template that contains placeholders for the title, heading, and paragraph. We then use LangChain to generate the HTML code by replacing the placeholders with the user input.
Code Optimization
One way to use LangChain for code optimization is to create a template or prompt that asks for the input code and optimization criteria, and then use LangChain to generate an optimized version of the code based on the input criteria.
Here's an example of how you can use LangChain to generate an optimized function for a specific optimization criteria:
import { OpenAI } from "langchain";

const model = new OpenAI({ temperature: 0.7 });

async function optimizeCode(inputCode: string, optimizationCriteria: string) {
  const prompt = `
  Given the following code:

  \`\`\`javascript
  ${inputCode}
  \`\`\`

  Please optimize the code for ${optimizationCriteria}.
  `;

  const res = await model.call(prompt);
  return res.res;
}

In this example, we've created a function that takes in the user's input code and the optimization criteria. We then use LangChain to generate an optimized version of the code based on the input criteria.
Bug Fixes
Here's an example of how you can use LangChain to generate a code snippet to help with debugging:
import { OpenAI } from "langchain";

const model = new OpenAI({ temperature: 0.7 });

const template = `
// Given the following error message:

// {errorMessage}

// Here are some possible solutions:

// Solution 1: {solution1}
// Solution 2: {solution2}
// Solution 3: {solution3}
`;

async function generateDebuggingSnippet(errorMessage: string) {
  const prompt = template.replace("{errorMessage}", errorMessage);

  const res = await model.call(prompt);
  return res.res;
}

In this example, we've created a template for generating possible solutions to a given error message. We use LangChain to replace the {errorMessage} placeholder with the user input, and then return the final text.

@slavakurilyak great suggestion and thanks for the engagement here! I will think about it but for very basic tasks you can use codex model to instruct the openai to generate some code based on the instructions. See this example:
https://platform.openai.com/playground/p/default-openai-api?model=code-davinci-002


[MRG] Fixed early stopping mechanism

Closes #55

[X] add test coverage for early stopping mechanism
[X] move verbose logic out of _check_early_stopping. Fixes iteration time computation
[X] Early stopping is now checked at the end of the iteration, and not at the beginning. This fixes an issue where the first init predictions would be considered as a proper iteration.
[X] Fixes the handling of scoring parameter. See comment in the code
[X] Renames max_no_improvement to n_iter_no_change
[X] Simplified early stopping logic to early stop iff the current score doesn't improve any of the last n_iter_no_change by tol percent

In scikit learn early stopping is done iff n_iter_no_change is not None. Here we do early stopping iff scoring is not None (there is no scoring in scikit learn).

Hm tests are passing but take too long without numba so travis times out.
Should we remove the NUMBA_DISABLE_JIT="1" jobs?
Anyway, ready for reviews :)

Should we remove the NUMBA_DISABLE_JIT="1" jobs?

No because this is necessary to compute coverage. However we could skip the slowest test when int(os.environ.get("NUMBA_DISABLE_JIT", 0)) == 1.

I see. Skipping this test when NUMBA_DISABLE_JIT="1" would result in much less coverage: this is the only test that covers early stopping and verbose=1 I think.
Should we have a dummy test that we don't disable, just for the coverage?

Yeah!


Horizontal line discontinue

The result is produced by the example code in tutorial

OS

MacOS Big Sur

Terminal Applications

iTerm2
Terminal
Alacritty

env

LANG=zh_TW.UTF-8
LC_CTYPE=zh_TW.UTF-8

TERM

COLORTERM=truecolor
TERM=xterm-256color

I'm closing it.  The issue is probably a disagreement in the font / rendering engine -- likely the font rendering doesn't recognize that this is East Asian locale and the glyphs should be "full width".


Remove third-party blackbox exporter

Repository says it is deprecated and links to the official blackbox exporter.

Thanks!


unable to upload build using jfrog cli

Hi,
I'm unable to upload build using jfrog cli, if run command without interactive=false getting wrong number, can you please take a look and correct me
env JFROG_CLI_OFFER_CONFIG=false CI=false JFROG_CLI_LOG_LEVEL=DEBUG jfrog rt u test-testing-20210520-112236.tar interactive=false --url=https://artifactory.tests.com/artifactory/test-maven-local/test/testing/20210520-112236/ --user=**** --apikey=****
+ env JFROG_CLI_OFFER_CONFIG=false CI=false JFROG_CLI_LOG_LEVEL=DEBUG jfrog rt u test-testing-20210520-112236.tar interactive=false --url=https://artifactory.tests.com/artifactory/test-maven-local/test/testing/20210520-112236/ --user=**** --apikey=****
[Debug] Usage Report: Sending info...
[Debug] Sending HTTP GET request to: https://artifactory.tests.com/artifactory/test-maven-local/test/testing/20210520-112236/api/system/version
[Info] [Thread 2] Uploading artifact: test-testing-20210520-112236.tar
[Debug] Sending HTTP PUT request to: https://artifactory.tests.com/artifactory/test-maven-local/test/testing/20210520-112236/interactive=false/test-testing-20210520-112236.tar
[Debug] Usage Report: Artifactory response: 404 Not Found
{
  "errors": [
    {
      "status": 404,
      "message": "File not found."
    }
  ]
}
[Debug] [Thread 2]  Artifactory response: 201 Created
[Debug] Uploaded 1 artifacts.
{
  "status": "success",
  "totals": {
    "success": 1,
    "failure": 0
  }
}

@yahavi Thank you it worked as expected , appreciate your quick help.


Delete the DepAnn bug pattern.

This bug pattern is harmful. It encourages developers to repeat themselves with
both a helpful @deprecated Javadoc comment (that typically provides useful
information) and a useless @Deprecated annotation (that serves no practical
purpose, and isn't worth the redundancy).

We need this check because tools are not guaranteed to respect the @deprecated javadoc, only the @Deprecated annotation.  See http://docs.oracle.com/javase/7/docs/technotes/guides/javadoc/deprecation/deprecation.html#javadoc_tag.  Specifically, Kythe does not handle the @deprecated javadoc.
You can turn it off with an "-Xep:DepAnn:OFF" command line option.

@eaftan can we just fix Kythe?

If you do decide to drop this bug pattern, can't it be turned off by default, rather than be deleted outright?

@Stephan202 just curious, why would somebody want to turn this on?

@swankjesse, I see @Deprecated as similar to annotations such as @Nullable: meta-information about the code made available to both users of the code and static analysis tools. I believe that using a "proper" annotation for this is more appropriate because (a) the compiler guards against typos and (b) I'd wager that there are more static analysis tools out there that handle "proper" annotations than that handle both types of annotation.
I do see your argument about duplication, but I do not feel it to be a problem (especially because deprecated code represents a tiny minority of the code I interact with on a daily basis).
I suppose this position is not uncommon, judging by the existence of the DepAnn bug pattern we're currently discussing, as well as the MissingDeprecatedCheck provided by both Checkstyle and Sonar Squid.

Consider also JEP 277 which not only will add value to the annotation, but also makes commentary on the Javadoc tag (emphasis mine):

It has been suggested that the @deprecated Javadoc tag be retired in favor of the @Deprecated annotation. They are redundant, in a narrow sense, in that the @deprecated Javadoc tag and the @Deprecated annotation should always both be present or absent; it never makes sense to have one without the other. However, they provide different information to different consumers. The @deprecated Javadoc tag provides descriptive text to be included in documentation. The @Deprecated annotation provides fine-grained information to tooling and to the runtime. Migrating a documentation string to, say, an annotation value would clutter up the runtime image with documentation strings, and it would require additional work for Javadoc to extract the string from the annotation instead of the doc comment where it currently resides.

The @Deprecated annotation is a mistake. Just like public, static and final, deprecated information is already present in both source and .class files for tools to consume as they like. We had @deprecated long before we had annotations, and IDEs can handle either. Probably the best long term fix is to deprecate the @Deprecated annotation.

This check reimplements javac's dep-ann xlint warning. It was duplicated here because we wanted an automated fix.
JDK-4903501 says:

We propose to add a marker annotation java.lang.Deprecated to supplant and
eventually replace the javadoc tag @deprecated. This is exactly the sort
of thing metadata is best at expression. This may improve the performance
of compilers, as eventually they will no longer have to scan doc comments
for @deprecated tags.

As a transitional feature, in -source 1.5 javac will give a warning if
a program element is deprecated in the javadoc tag but not with an annotation.
In 1.6 we might either make this a hard error or simply stop recognizing the
tag in doc comments altogether.

It sounds like the plans to get rid of the javadoc tag have been abandoned, but the annotation isn't going anywhere either. If you want metadata about deprecation in class files, prefer the annotation. Compilers shouldn't have to parse javadoc.

So... continue to require both Javadoc and the annotation forever, because it makes compilation faster / makes it easier to write compilers? I'm not buying it, but it's your project.


Please provide a better error message than "bad substitution"

First, thanks for creating this awesome tool! It is, however, a bit hard to use in practice since the only thing it outputs when an error occurs is Bad substitution. Would it be possible to output a bit more (contextual) information on what the precise problem is? A print of the line of input that caused the error would be very helpful, for instance.

sure, please consider sending a patch.

A patch would be welcome 👍


fix missing backtick in mysql keywords

What is the purpose of the change
Fix the issue of not adding backticks for some MySQL keywords.
Related issues/PRs
Related issues: #4762
Brief change log

Delete the file linkis-bml-server/src/main/resources/mapper/common/TaskMapper.xml;
Add the file linkis-bml-server/src/main/resources/mapper/mysql/TaskMapper.xml;
In the file mysql/TaskMapper.xml, backticks were added to the MySQL keyword "system".

Checklist

[x] I have read the Contributing Guidelines on pull requests.
[x] I have explained the need for this PR and the problem it solves
[x] I have explained the changes or the new features added to this PR
[x] I have added tests corresponding to this change
[x] I have updated the documentation to reflect this change
[x] I have verified that this change is backward compatible (If not, please discuss on the Linkis mailing list first)
[x] If this is a code change: I have written unit tests to fully verify the new behavior.

LGTM.


⚠ Punto de Fuga has degraded performance

In ad73903, Punto de Fuga (http://puntodefugapodcast.com) experienced degraded performance:

HTTP code: 200
Response time: 10151 ms

Resolved: Punto de Fuga performance has improved in e8d3fae.


Starting application from background with notification happens in foreground (iOS)

When trying to test opening an application from the background with a push notification with this code:
await device.sendToHome();
await device.relaunchApp({ newInstance: false, userNotification });

The notification is sent to the application but only after the native UIApplicationState is already changed to UIApplicationStateActive. This makes the application behave as if the notification was opened when the app was already in the foreground.
The same behaviour happens when trying like this:
await device.sendToHome();
await device.sendUserNotification(userNotification);
await device.relaunchApp({newInstance: false});

@LeoNatan

Thanks

Just to prepare you, @yedidyak, you are relying on bad practice in your application. The application states you rely on are Apple implementation detail, and should not be relied upon in production code. For example, you are using an old, deprecated API. Using the new UserNotifications API brings a different implementation and states are different there.
The more I think about the solution to fix this issue (and how much it complicates the system), the more resistant I am to fixing it. Will discuss more with @rotemmiz, but just be prepared for this possible outcome.

Further discussed with @rotemmiz. We have decided to mark this as "wontfix" and close. We plan to remove support of this deprecated API in Detox soon anyway.
All is not lost, because we have a guild task to upgrade react-native-notifications to use the new Apple API instead of the old one. So it will be solved one way or another.

For the sake of history, this was indeed a bug, which was handled in #604, by changing the way notifications and urls are being sent while application is in background.


Super cool project! Thanks :D

Thanks for building this -- it's exactly what I was looking for! If help is ever needed, I'd be more than happy to try!

Thank you so much for the nice words! I'm not planning on focusing on this much, so if there's a feature you think would like, you could always submit a PR. 🙏 I will make an Erato update when I get to subscribe to Opus. :)


Update instructions

@rochoa maybe you think this is pointless, maybe not :_)

While I agree we should provide some kind of information about upgrading: that's not universal because not every time you upgrade you need to reinstall all dependencies.
Using:
make clean all

Equals:
rm -rf node_modules; npm install

NEWS.md contains information about upgraded dependencies.
Anyway, as I said, I'm OK because we need to provide some information, but can we change the pull master branch to something less specific: checkout your commit/branch or something similar.

@rochoa :eyes:

:+1:

Thanks!


Add tox cover task to travis build.

Running the cover task in the travis build process will catch what coveralls.io misses in code branch coverage.

This isn't necessary. Don't do this. coveralls.io does the job already

So this stems from https://github.com/GoogleCloudPlatform/gcloud-python/pull/1870#issuecomment-227229477.
@dhermes LMKWYT

I think this should be closed. Even this PR has a coverage check, making it clear it's not needed

@dhermes the point of the PR is that coveralls doesn't test branch coverage, while tox -e cover does.

I see I made this PR too quickly haha, parallel conversation happening https://github.com/GoogleCloudPlatform/gcloud-python/issues/1872 as well


NullReferenceException: Object reference not set to an instance of an object.

_Layout.cshtml:
@(this.Url.Action<TestController>(x => x.Index()))
results in:

@skorunka Was this working before?

You are right. i forgot to use extension "UseTypedRouting" in Startup.
Thank you.

I will keep this open in order to implement a friendlier exception message when people forget to call that method.


Packbuilder

Obviously this still needs the documentation added and some more tests. I just wanted to get some feedback on using multiple structs for safety. The two things that I've done this for are getting the hash of the pack and thread safety with the callbacks.
Getting the hash is only valid after the pack has been written somewhere, and once you insert a new item to the pack the hash is invalid until you re-write it. I think this could be done as a runtime check against object_count and returning an Option<Oid>, but by instead returning a struct holding a mutable borrow against the builder that is the only way to get the hash you are guaranteed that you can't insert any new objects until the struct is dropped. In the common case of not actually caring about the hash the struct will be dropped straight away (and hopefully the construction and drop could even be optimised out) and you can go on using the builder as you want.
Thread safety is much more important. I spent a while looking at the libgit2 code around this and verified what guarantees it gives around the callbacks. This seems like the sort of thing that libgit2 should actually document to ensure they aren't violated in the future, but anyway, the guarantees as I see them:

If libgit2 was built with threading disabled or the packbuilder has its thread count set to 1 (the default):

All operations and callbacks will run on the thread that calls one of the the write methods.
Returning false from the callback will stop the build and the callback will not be invoked again.

If the packbuilder has its thread count set to anything other than 1:

The progress callback will be called from whichever thread next has some progress to report when the timeout between reports expires.
Access to the progress callback is locked by a mutex so the callback will not be called concurrently on multiple threads.
Returning false from the callback will eventually stop the build, the callback may (by my reading of the code "will almost certainly") be invoked again from other threads until the build has stopped.

The foreach callback will only ever be called from within the main thread that foreach was invoked on.

So the unthreaded case is basically the same as all the other git2 callbacks, the threaded case needs some special handling. The two changes I could think of to make the threaded case safe are:

The callback must be Send as it may be invoked from other threads spawned by libgit2.
If the callback panics we must ensure it doesn't get invoked again. Normally returning false handles this for us, but as all the threads aren't guaranteed to stop together the C callback may be re-invoked so needs to keep track of panicking and avoid re-invoking the rust callback.

The second change is an internal implementation detail. The first however affects the public API for the pack builder. To minimize issues with this I created two "subclasses" of PackBuilder, one threaded on non-threaded. Calling the set_threads method consumes the PackBuilder and returns a ThreadedPackBuilder that only accepts Send callbacks for its progress callback. If you set the progress callback first then that method consumes the PackBuilder and returns a NonThreadedPackBuilder that doesn't have the set_threads method.
I've temporarily pushed the documentation to https://nemo157.com/git2-rs/packbuilder/doc/git2/struct.Repository.html#method.packbuilder if you want to check how it looks with all the Deref implementations.

Also, I've used this and successfully cloned a repo from a pack built with it :smile: now just need https://github.com/rust-lang/cargo/pull/2385 so I can actually build and deploy those changes to my server while I finish this off :wink:.

Thanks for the PR @Nemo157! Always nice to expand the API bindings :)
I wonder if perhaps the state-based transition between packbuilder types may be a bit unergonomic for now? It's a lot of structs to juggle and it seems the primary benefit is that the closure's bounds are either Send or not. Maybe we could go with just Send for now, and if necessary we can add a separate one-off builder for adding a non-Send closure which hardcodes threads to 0?

I wonder if perhaps the state-based transition between packbuilder types may be a bit unergonomic for now?

Yep, that's the main reason I thought a WIP PR would be good for this :smile:. It was an interesting experiment, but you're probably right that it's overkill for this case. I'm tempted to try and write a couple of tests actually using all the types to see how ergonomic it is in use, then I'll fallback to just having a Send bound.

Maybe we could go with just Send for now, and if necessary we can add a separate one-off builder for adding a non-Send closure which hardcodes threads to 0?

Thinking about what would actually be needed to propagate the panic correctly from any of the callbacks through the packbuilder makes me think going the other way initially might be better. Don't require Send and don't expose the set_threads method. If someone wants to put in the effort to write a multi-threaded, multi-callback panic recovery system then they can add it, I'm going to be sticking to single-threaded for my use-case anyway.

Coverage increased (+0.2%) to 54.951% when pulling a6d48783e077736262f34c9b4a65e40c041fb491 on Nemo157:packbuilder into 6385b8e1750d0b56bb304cc70e60f502e1b8c973 on alexcrichton:master.

Coverage increased (+0.2%) to 55.026% when pulling 3d26265b7ace434a2a25c8da1dc8d82e9dd72f2d on Nemo157:packbuilder into 6385b8e1750d0b56bb304cc70e60f502e1b8c973 on alexcrichton:master.

Looking good! I think with that bug fixed and some docs this is good to go!

Sorry I took so long to get back to this. Changed the progress callback to be passed in to methods that actually will cause it to get called to fix lifetime issues with it and added documentation.

Awesome, thanks @Nemo157!


RethinkDB?

https://www.rethinkdb.com/

DynamoDB
CouchDB

RethinkDB - live json push to web pages, https://www.rethinkdb.com/faq/, https://github.com/rethinkdb/rethinkdb
Apache CouchDB - eventual consistency with an http interface, even if the network drops, it will resume CouchDB connectivity/updates when the network comes back, http://couchdb.apache.org/, https://github.com/apache/couchdb, http://guide.couchdb.org/
DynamoDB - Amazon's web NoSQL DB. There is a local version too, I might move to this if I want to.


Convert to TypeScript

The objective of this PR is to help our migration process to TS.
We will be able to use both TS and JS then just get rid of JS when all the code has been ported to TS.

Support TS and JS file (for smooth transition to type script)
Generate JS bundles as before
Generate definition files
Add a .tslint configuration file for editors to nicely pick up changes
Transpile every file (3 files) to TS

I would recommend starting with ts-loader, given it aligns with the official webpack docs...

I've gone with a possible solution.
I did not delve deep enough on refactors and stuff, but it would be a great idea, for example, to bundle everything into one executable, which should be easy to do by properly configuring the ts compiler.
I didn't add webpack as suggested, as I don't think it is necessary altogether.
A possible next step would be to configure ava to test the TS directly.
I know the latest version supports that out of the box.

Although we can start reviewing, I might add a .prettierrc file just to be sure configs are aligned with tslint. I noticed @gpedro probably uses it automatically on his IDE, which is a good thing.
Also, the tests are not currently passing, so I have to check it up before merging. I'll try to have some time to dedicate to it tomorrow, even though it is election day.

I'm closing outdated issues and pull-requests that are no longer relevant given how much time has passed since they were opened.


Problem with multiple source paths

I am not sure if this is a problem with piggieback or weasel and would need some time to narrow it down if this is necessary:
I have a cljs.user namespace in my /src/dev/cljs/cljs/user.cljs. "src/dev/cljs" is part of the cljsbuild source vector if I am in the dev profile.
I also have a myproject.core namespace in /src/cljs/myproject/core.cljs. "src/cljs/" is obviosuly also part of the cljsbuild source vector.
When I start the weasel REPL, it complains that it can't find myproject.core. This is happening with weasel 0.5.0, clojurescript 2665 and cljsbuild-0.1.5-SNAPSHOT. With Clojurescript 2511, piggieback 0.1.3 and weasel 0.4.2 I don't have the problem.

Perhaps this is a duplicate of #28

Seems very likely. But the error directly refers to the ns form instead of specific vars.

@lgrapenthin Right. Currently you get ExceptionInfo No such namespace errors with the latest ClojureScript releases, but previously the same root cause would result in WARNING: Referred var xxx.yyy/zzz does not exist warnings.
I created a reduction in the form of a minimal repository capable of reproducing it: https://github.com/mfikes/weasel-src-paths

Unless this is slurped somewhere, I currently don't get the WARNING (versions specified above).

Also hit me when https://github.com/tomjakubowski/weasel/pull/44 fixed another issue I was having.
The :source-paths workaround doesn't seem to work for me though.

For the record: In order to be able to require your own project's CLJS-Namespaces, you have to add the source-directories to leiningens :source-paths.

Closing as a duplicate of #28. Feel free to ping this issue if a subsequent fix for #28 doesn't fix your issue!


[BUG] Second witness node's role is Management but NOT behave like management node

According to https://github.com/harvester/harvester/issues/4840#issuecomment-1883190132

Even if we try to set up 2 witness nodes, we still have one witness node.

Suppose the 2nd witness node should become and behave like managemant node.
Please help to check it's expected or not.
Environment

Harvester

Version: v1.3.0-rc1
Profile: QEMU/KVM, 2 -> 3 nodes

Default (D) node: 8C/16G/500G
Witness (W) node: 4C/8G/500G

ui-source: Auto
Longhorn: v1.6.0-rc2

Steps

Setup 2 nodes cluster (Default + Witness)

 Cluster is Active :heavy_check_mark:

 node-0 is Management and node-1 is Witness :heavy_check_mark:

node-0

node-1

 Longhorn should has only 1 node :heavy_check_mark:

Create a VM with some data

 VM is Running :heavy_check_mark:

 Volume is Ready, replica should be 1 (LH has 1 node) :heavy_check_mark:

Adding Witness node-2

 Cluster is Active :heavy_check_mark:

 node-2 should be and behave as Management node :x:

Role is Management but shows Hardware-based virtualization is disabled or not supported. Hardware-based virtualization must be enabled before creating any Virtual Machines.

 LH should have 2 nodes :x:

LH still has 1 node only

Volume is Ready, but still one replica since only has 1 node

Supportbundle
supportbundle_dww_2024-01-31T10-11-10Z.zip

Hi @albinsun,
I created a PR and updated the test plan for this issue.

Hi @albinsun, I thought you could test this issue by master-head.


Drag + drop files and folders to terminal to print path

Check for existing issues

[X] Completed

Is your feature request related to a problem?
This is a feature I've used quite a lot in an Atom terminal package which I really like, with a terminal open you can drag files or folders onto it and it will insert the path to it. It is really useful if you have a file browser open or even just the project tree, you can just drag and drop files to quickly set the path of whatever you are trying to run.
I think VSCode also does this by default.
Describe the solution you'd like
Drag a file from either the project pane or a file browser and drop it onto the terminal.
The terminal will then print the path onto the prompt - adding quotes if necessary due to spaces etc. in the path.
Maybe a modifier could be used to execute an action at the same time, for example if you held alt it would automatically cd to that directory (or if a file, the parent directory).
Another modifier could be used to control whether the printed path is the full path or the relative one e.g if my prompt was in ~/dev/project1/src then dragging a cargo.toml from the parent directory would print ../cargo.toml but if modified it would print /home/user/dev/project1/cargo.toml.
If applicable, add mockups / screenshots to help present your vision of the feature
No response

This is blocked on having better file handling all around in Zed, I'll make sure to keep an eye on it when that work happens though :)

This is implemented in v0.119.4 preview.


Update README.md to show links to data table examples

Fixes #394

Not sure why the build is failing. Unfortunately the CI system requires me to give them my email and private repository access to view the build.

Nothing to do with your PR, it's a glitch between twitter (used in the test) and our CI provider. Thank you for the PR!

:tada: This PR is included in version 2.5.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix: Make the add page responsive

Summary
Adding Responsive Layout to add-blog page.
Description
add-blog page is not responsive. Now, adding responsiveness to add-blog page according to Laptop, Tablet, Mobile.
Images
Mobile:

Tablet:

Laptop:

Issue(s) Addressed

Closed #15

Review Checklist
Please make sure to go through this checklist before submitting your review.

[X] Make sure the PR title is of format tag: Issue Title

tag should be one of fix,feat, docs, chore, refactor
Ex: feat: Added responsive design for the home page

[X] Ensure the code follows our coding standards and guidelines

Ex:  naming conventions should be of form file-name.ts not fileName.ts

[X] Ensure that you have staged only the required commits and raised the PR and not included all the files which would be otherwise required for local setup
[X] Check if the PR is rebased to the latest main/development branch.
[X] Make sure there are no unnecessary files or debugging artifacts in the PR.

Thank you @SankalpHaritash21  for the new PR, the desktop and tablet view looks great, but in the mobile we would need some tweeks, i.e the heading seems smaller than the text labels and arrow is too big, can you please fix this by adding new commit to this branch.
Please follow the following font size
Header 22px
Labels 14px
Button text 16px
Arrow 24 x 24 px ideally but you can change as to suit the header, it should be same size of that of header


how to use third party libraries like Vuejs

Hi i cant make it work
http://vuejs.org/guide/events.html#Key_Modifiers
this line does not work
v-on:keyup.13="submit"

Hi @ramstein74, this a bug in SlimFast itself.  Could you file the bug there?


Empty MachinePool configs when creating a new cluster

Setup

Rancher version: 2.7.2 (Prime) Upgraded from 2.7.1 (Prime)
Browser type & version: Firefox 112.0

Describe the bug
After upgrading a Rancher Prime install from 2.7.1 to 2.7.2 and attempting to create a new cluster on an IaaS provider (EC2/vSphere), machinePools element is empty.
To Reproduce

Upgrade a Rancher Prime instance from 2.7.1 to 2.7.2
Attempt to create an RKE2/K3s cluster on a Infrastructure provider like EC2 or vSphere

Result
Machine pools are empty and cannot be configured

Expected Result
Machine pool configuration to be present
Screenshots

Additional context
Attached HAR file
2.7.2-ui-error.zip

Upon discussion with @pgonin, this ticket is over to the Elemental UI team to investigate.

Good morning all.
There were issues already noted on the cluster provisioning part due to some conflicts between files that are on Elemental UI (up until v1.1.0 included) which were wrongly overwriting files from Rancher Dashboard.
looking into the HAR file, you are using v1.0.0: https://rancher.virtualthoughts.co.uk/api/v1/namespaces/cattle-ui-plugin-system/services/http:ui-plugin-operator:80/proxy/elemental/1.0.0/plugin/elemental-1.0.0.umd.min.js
Which means that all of the problems are still not fixed on the Elemental UI side. The fix for this problem on cluster provisioning was a 2 part fix:

1st part was in Dashboard (which will come in 2.7.2)
other was in Elemental UI (which comes with v1.1.0 - to be released on the following days)

So, all problems should be fixed with the combo of Dashboard v2.7.2 + Elemental UI v1.1.0 (upcoming release).
So, unfortunately, the reported issue is still an expected behaviour.
Shall we keep this issue open until we can test the combo of Dashboard v2.7.2 + Elemental UI v1.1.0, or just close it?
@agracey @gaktive @juadk @David-VTUK

Thanks for the detailed feedback @aalves08.
We shouldn't be shipping UI Extensions that knowingly break core functionality.

There are 2 possible workarounds:

remove elemental-ui if you don't need elemental and need provisioning to work on 2.7.2
install Staging version of Elemental 1.1 if you need Elemental
Situation will be fully fixed with release of Elemental in coming days

@David-VTUK I believe this should be fixed by now - version 1.2.0 of Elemental extension + a recent backend version of rancher. Can you confirm? If so, we can close this.
(doing a bit of backlog grooming)

No more feedback after a long time. Closing this issue.


Error: Philips Hue: 8, parameter, linkbutton, is not modifiable

I have this flow:

With this inject node:

Resulting in this msg object;

And this error while pressing it:

am i doing something wrong?

Thanks for the hint! According to the Hue API docs:

Starting  1.31, Writing is only allowed for Portal access via cloud application_key.

So this is no longer possible in this form. Sorry :S


Serverless V2

Will Serverless V2 be available when it goes GA later this year?
https://aws.amazon.com/rds/aurora/serverless/

if it is supported by the Terraform AWS provider - depending on what changes are necessary; yes or yes with a PR to support

ok thank you it should be out to GA end of this month according to AWS Rep. Its currently available in us-east-1

Looks like this just went GA https://aws.amazon.com/about-aws/whats-new/2022/04/amazon-aurora-serverless-v2/

+1 for this.

once this is supported in the provider, we can add support here https://github.com/hashicorp/terraform-provider-aws/issues/24349

+1 please

again - we cannot support until the provider supports. Please cast your 👍🏽 on the upstream issue since issues with the most votes are prioritized by the maintainers there

I opened the PR to support this in the provider. Once it's upstream, I can try integrating this into this module as well, since we're relying on it as well.

Cool - looking forward to testing this.

Looks like the upstream merge was complete 4.12.0

Looks like the upstream merge was complete 4.12.0 hasn’t been released yet. Please be patient, we all want the feature asap, rest assured I’m monitoring the provider releases closely.

4.12.0 has been released! https://github.com/hashicorp/terraform-provider-aws/releases/tag/v4.12.0 🤗

@bryantbiggs @njoerd114 https://github.com/terraform-aws-modules/terraform-aws-rds-aurora/pull/288 I took a stab at a pr for this. Let me know if its not what your looking for.

I am very excited to have this! Thanks for all the hard work getting this released!

This issue has been resolved in version 7.0.0 :tada:

If you enjoy using terraform-aws-modules please give all your favorite modules a ⭐ and tell your friends! If your company/organization are heavy users of terraform-aws-modules, please consider sponsoring through one of the many avenues available - thanks all!


the color detected by FSCapture doesn't match the specified RGB value on Android emulator running on macOS

hi, i have some problem in using colors, pls help me, my view's color is not same with my codes color. this is my code.
import "package:flutter/material.dart";

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'flutter app',
      home: Scaffold(
        body: Column(
          children: <Widget>[
            Container(
              width: 400.0,
              height: 500.0,
              color: Color.fromARGB(255, 19, 27, 45),
              child: Center(
                child: Container(
                  width: 200.0,
                  height: 200.0,
                  color: Color.fromARGB(255, 32, 45, 73),
                ),
              ),
            )
          ]
        )
      ),
    );
  }
}

in my codes, outer container use color Color.fromARGB(255, 19, 27, 45), inner container use Color.fromARGB(255, 32, 45, 73).
this is view in my Simulator

i use FSCaputure to draw color, the outer's rgb result is 21,28,46, the inner is 34,46,72
The changes in this example are small, but sometimes they change a lot. how can i do?
pls forgive my poor english. thanks very much.

I've seen this, too. I'd like to know what's going on here as well.

@jason-simmons might you know what's going on here?

This may be happening because Android emulator configurations typically use a framebuffer with 16-bit color depth by default.
If the AVD virtual device config file specifies hw.lcd.depth = 16 then try changing it to 32.


inconsistent naming of chaincode transaction types across layers

In Transaction proto
deploy is CHAINCODE_NEW
invoke is CHAINCODE_EXECUTE
query is CHAINCODE_QUERY
In OpenchainMessage proto
deploy and invoke are CHAIN_TRANSACTION
query is CHAIN_QUERY
In Chaincode interface
deploy and invoke call Run
// Query is to be used for read-only access to chaincode state
query calls Query
We need to provide meaningful and consistent names to transaction types.

related #373
I would propose the following changes:
Transaction
    enum Type {
        UNDEFINED = 0;

// publish a chaincode to the network and return an addressable name,
// which is used in CHAINCODE_CREATE,CHAINCODE_UPGRADE,
// and CHAINCODE_TERMINATE
        CHAINCODE_PUBLISH = 1;

// create an instance of chaincode and return an addressable name,
// which is used in CHAINCODE_INVOKE and CHAINCODE_S/QUERY
        CHAINCODE_CREATE = 2;

// call a chaincode instance `invoke` function as a transaction
        CHAINCODE_INVOKE = 3;

// call a chaincode instance `query` function
        CHAINCODE_QUERY = 4;

// call a chaincode instance `query` function as a transaction (strong query)
        CHAINCODE_SQUERY = 5;

// upgrade a chaincode; part of chaincode life-cycle mgmt
// redeploy the chaincode with same name; need security
// call chaincode upgrade func to perform and data upgrade
        CHAINCODE_UPGRADE = 6;

// terminate a chaincode; chaincode no longer accepts tx
        CHAINCODE_TERMINATE = 7;
    }

The OpenchainMessage and chaincode interface should follow these enumeration.
Thoughts?

The OpenchainMessage should abstract the message (eg Transaction) such that handler may look at the payload to further discriminate.
message OpenchainMessage {
    enum Type {
        ...
        CHAIN_TRANSACTION = 6;
        ...

This means we need to change consensus framework event handler.
The chaincode interface should define functions according to the transaction types, including:

Create
Invoke
Query
Upgrade

The original PR is closed but not the issue. Reopening the issue. Related new PR is #1012

pr #1012 has been merged, so close this issue


Error opening doc when the os window is small enough for the telescope preview to hide

I'm running into an error that occurs when the OS window containing neovim is small enough that the Telescope preview does not display. When selecting an item through Telescope I get the following error:
E5108: Error executing lua: ...hare/nvim/lazy/nvim-devdocs/lua/nvim-devdocs/pickers.lua:75: Invalid 'replacement': Expected Lua table
stack traceback:
        [C]: in function 'nvim_buf_set_lines'
        ...hare/nvim/lazy/nvim-devdocs/lua/nvim-devdocs/pickers.lua:75: in function 'open_doc'
        ...hare/nvim/lazy/nvim-devdocs/lua/nvim-devdocs/pickers.lua:180: in function 'run_replace_or_original'
        ...re/nvim/lazy/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'key_func'
        ...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:257: in function <...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:256>

Let me know if you aren't able to repro and I'll try to put together a minimal config.

The issue happened because I reused the telescope preview buffer for avoiding extra read operations but I haven't thought of the case where there is no preview... But it should be fixed now, thanks for the report!

Thanks for the quick fix! Unfortunately I'm still getting an error when doing the following:

use a window large enough for the telescope preview to display
Open the devdocs picker and make sure you can see the preview
shrink the window to the point where the preview no longer displays (e.g. create a new neovim window, tmux pane, etc.)
press enter, and see the following error:

E5108: Error executing lua: ...l/share/nvim/lazy/telescope.nvim/lua/telescope/utils.lua:424: Invalid windo
w id: 1004
stack traceback:
        [C]: in function 'nvim_win_set_buf'
        ...l/share/nvim/lazy/telescope.nvim/lua/telescope/utils.lua:424: in function 'win_set_buf_noautocm
d'
        ...scope.nvim/lua/telescope/previewers/buffer_previewer.lua:401: in function '_teardown_func'
        ...zy/telescope.nvim/lua/telescope/previewers/previewer.lua:79: in function 'teardown'
        ...share/nvim/lazy/telescope.nvim/lua/telescope/pickers.lua:1571: in function 'on_close_prompt'
        .../nvim/lazy/telescope.nvim/lua/telescope/actions/init.lua:386: in function 'run_replace_or_origi
nal'
        ...re/nvim/lazy/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'close'
        ...hare/nvim/lazy/nvim-devdocs/lua/nvim-devdocs/pickers.lua:174: in function 'run_replace_or_origi
nal'
        ...re/nvim/lazy/telescope.nvim/lua/telescope/actions/mt.lua:65: in function 'key_func'
        ...hare/nvim/lazy/telescope.nvim/lua/telescope/mappings.lua:257: in function <...hare/nvim/lazy/te
lescope.nvim/lua/telescope/mappings.lua:256>


requirement text file

Hi, thank you for your valuable package,
After clone the mentioned repository, to install requirements(pip install -r requirements.txt), I found this error:
ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'

It's not present on the master branch, but check the other branches and you'll find it.

Hi @sbmubs, while the answer from @FeckNeck is technically correct, those requirements.txt files are outdated and could lead to undesired behaviors: the auton-survival project is now available on the Python Package Index so, if you just want to use auton-survival as a dependency in your own project, you only need to run pip install auton-survival.
If instead you want to contribute a fix or a feature to the project you should clone the repo, install poetry and use that for development. You can install the needed dependencies by running poetry install in the root of the project.
If you prefer working with a requirements.txt file in your own fork, then clone the repo and run poetry export --output requirements.txt. This will generate a version-locked and hashed requirements.txt file.
Hope this helps you out 🙂

Hi @matteo4diani, I was trying to download the package via pip like you suggested doing. However, when doing so it starts out downloading and then encounters an error. I am including an screenshot of the bottom bit of the error. I will happily provide more information in order to get the package working on my system. Thanks!

Hi @Jrhasse, I need these info to help you troubleshoot the problem:

On which OS (and which version) are you running?
Which version of Python are you running?
Are you attempting to install in a virtual environment? If yes, which virtual environment management tool are you using?
Which dependencies are already installed in the Python environment where you are attempting this installation?

While you gather this info, at a glance it is most likely a problem with Python versions, try using a more recent version (3.10/3.11) and let me know if that fixes the problem.

I would rather use option 2 and use the versions that the module is meant to work with. Makes no difference to me as I need to learn it regardless. I created a fresh conda environment using 3.11.7 but now encountered this issue. It looks like pip cant find an old version of pytorch, but I am not sure. See below.

Try a new env with Python 3.10.

Success! For anyone else who runs into similar issues. I ran the following in the anaconda prompt:
conda create --name myenv python=3.10
conda activate myenv
Python -V
pip install auton_survival
Thank you @matteo4diani for your help!

🎉 You're welcome @Jrhasse , happy to help.
Thanks for reaching out and enjoy auton-survival.
If you find anything that needs improvement or you need a missing feature please open a new issue here on GitHub :)

Thank you both! I was going crazy with this installation problem. It worked perfectly (provided one pays attention to run anaconda in myenv too :) ).


How can I get the time of release of a song or an album

I was going through spotipy and its awsome. I was hoping to extract the time of release of an album or song. Any help in this regard would be greatly appreciated

Do you mean the release date?
It's in the Web API for an album and in the return from the Spotipy album call.
In [6]: ret = api.album('spotify:album:3j8VO8KmIeFACN1ZpEeasY')

In [7]: ret['release_date']
Out[7]: '2016-06-10'

In [8]: ret['release_date_precision']
Out[8]: 'day'

Yes I meant release date. However I would like to see it by track instaed of album. Is it possible?

Did you check the docs?
It's not in the track return call but you can get the album ID from the track response, and then get the album data in a separate call.

Thanks. That did it


Diacritic popup: focus after closing

If you close the diacritic popup, focus switches back to the top of the page, it should go to the element that had focus when the diacritic menu was opened.

Diacritics is no longer a modal.


Restrict to HTML pages

The bug was in a HTML parser and only affected web pages, but the addon currently checks headers for all resources.
This results in a non-Cloudflare page like http://stackexchange.com/, which is not affected, being flagged because it includes images from http://stack.imgur.com/, which are behind CF. The same will be true for any pages which include non-HTML resources from a CF-fronted site.

Hi chocoloateboy, thanks for the report.
In my understanding, the bug was triggered by the HTML parser (which indeed only processes web pages), but this parser was running as a module for nginx, and the nginx process was also handling other types of requests, so that actually memory related to those requests could be disclosed. As far as I understand this section of the CF incident report, plenty of non-HTML data was disclosed.
Example scenario: User requests to load an image from a website behind CF, but needs to send his session token/cookie along to have access to the image (think okcupid). The requested resource is an image, but the request headers might have been leaked through cloudbleed.
The false positives you mention is, in my opinion, a different problem. Namely, if any request in the context of a tab is served by CF, the whole tab is "tainted". In a sense, this is correct: the page does use CF because some part of it is using CF. But I agree, this is potentially confusing for the user.
My suggestion is the following:

For each tab, bin the requests served by CF by domain.
If the tab's primary domain is affected, use red icon. If only 3rd party domains affected, use yellow icon.
Show details about affected domains when clicking on the icon.

Let me know if this approach would be acceptable for you.

My suggestion is the following

Sounds good :+1:


Dev 2662 sdk send component seen events to xp api for sticky components

Refactor the NinetailedCorePlugin to be class based
NinetailedCorePlugin extends the common NinetailedAnalyticsPlugin class to reuse the component view handlers
NinetailedCorePlugin selectively send ComponentView events based on sticky configs

@maroun9t I've changed the plugin threshold to a getter function - is this fine then from your POV?


Not able to get started on Windows

Hi, thank you for the great tool. It looks fantastically useful.
I'm having difficulty getting set up on windows and I wonder if that's because a) this plugin doesn't work on windows, or 2) if I need some additional plugins/libraries installed?

I've installed the plugin and it appears in my atom packages lists ✔

I've opened the GitHub project from windows powershell by typing atom me-ica ✔

I've added the following text to the end of the README.md file ✔
  ## Contributors
  <!-- Contributors START
  Kirstie_Whitaker KirstieJane https://whitakerlab.github.io doc
  Elizabeth_DuPre emdupre http://emdupre.me/ doc code testing
  Contributors END -->
  <!-- Contributors table START -->
  <!-- Contributors table END -->
  This project follows the [all-contributors](https://github.com/kentcdodds/all-contributors) specification.

But when I click run or ctrl+alt+p nothing happens ❌

Any thoughts? Thank  you again!

Hey @KirstieJane
Thanks for reaching out!
I don't think think atom packages are OS specific. They are platform agnostic (for most cases)
Can you check whether the shortcut bindings (ctrl+alt+p) are bound to this plugin on your editor?

Hi @mbad0la! I looked through the Keybindings settings and get: 'ctrl-alt-p': 'allcontributors:transform'.
So I think that's right?

The package has dependencies on the following API methods :

repositoryForDirectory
getOriginURL

As these API methods are specific to git repositories, I think the only place the code might break would be because of logic around the above mentioned methods
Thanks for bringing this to my attention, I'll debug this on windows soon!
(You can of-course hack around and let me know if you find the cause 😛 )

Thank you @mbad0la! I don't know anything about building packages for atom so I don't think I'll be much help, but I'm very happy to test out any updates etc if that's helpful 😄.
No great rush, I made a table based on the readme from this repo that was super easy to edit, so you've already helped out a lot!

@KirstieJane @mbad0la Hi, I'm also facing similar issues!  Let me know if they're fixed, thanks.

Someone from the org will take a look at it next weekend. Atom has been notoriously buggy with Windows. I hope it turns out to be an easy fix ...


Can u tell me where to fine tiktok api

I still wonder why people are fine useful tiktok api like that?
https://api16-normal-c-useast1a.tiktokv.com

I think it's from reverse engineering process. But instead of spending time for that, you should use this one to save time https://apify.com/novi/fast-tiktok-api


Add SUPPORT guide

Add a SUPPORT as per https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/adding-support-resources-to-your-project.
If this is generic it can be added to the .github repository as per https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/creating-a-default-community-health-file

A link to create GitHub Issues is helpful. I find issues are helpful because they both create a trigger email (especially if we include @username in the issue templates) and provide a public history on which others can search and comment.

Personally a fan of keeping things in GitHub. That will mean encouraging maintainers to stay on top if issues and discussions (if we enable those).

Kind of agree - one place to focus folk.
We probably do need to do some cleanup then to "close" or "notify" these other older arenas for discussion... ...but not the scope of this ticket.


count syntax bug?

p.DB.GetDB().Model(&SystemInfo{}).Where("Key=?", data.Key).Count(&count).Error
when i run the above sql , get the following error info:
Error 1064: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'Key=?))' at line 1
and when i exec the following one, is exec right, p.DB.GetDB().Model(&SystemInfo{}).Where("Key=?", data.Key).Error
but i want to get the count of the given key,  is this a bug ? how can i do it next?

the above SystemInfo is
type SystemInfo  struct {
gorm.Model
Key string
Value string
}
when I modified it to
type SystemInfo struct {
gorm.Model
Name string
Value string
}
and exec the above sql:   p.DB.GetDB().Model(&SystemInfo{}).Where("Name=?", data.Name).Count(&count).Error,
it works,

type:question

Please follow https://github.com/jinzhu/gorm/blob/master/.github/ISSUE_TEMPLATE.md
Thank you, please reopen another issue if it is still a problem for you.

what is your &count?

var count int64
p.DB.GetDB().Model(&SystemInfo{}).Where("Name=?", data.Name).Count(&count).Error
@Bartuz


iOS - Auth confirmation dialog disappears

When the iOS device tries to show the authentication dialog box it disappears after a few milliseconds. Every other native dialog is working fine. Only this one with the "Continue" button.
I have tried on iOS 13.1.2 and 13.3, with no luck.
Unity 2019.2.17f1
In the editor it works fine. It opens the webpage and authentications is fine. Uploading files works as well.
Everything works fine using iOS 12.0.
Any ideas?

It could be an issue due to an outdated iOS native auth library; make sure you're using the latest one.

I'm also getting stuck on this issue. Sorry, I'm kinda new to these things, but where would the outdated iOS native auth library be located?


[CD] Auto-deploy

Deploy decookie
Backend (inside @marigold-dev/deku)
For the "backend" the contract will be originated manually by myself when we will decide do to it.
Frontend (this repo)
For the frontend, an auto-deploy on a dedicated URL at each commit on main branch (this branch is protected, so each commit on it is fully working and tested before) would be perfect.
An URI containing cookie and deku-c on marigold.dev would be awesome (cc @oteku @d4hines)

It's deployed :

https://cookie.deku-c.gcp.marigold.dev/
https://cookie.deku-c.marigold.dev/

Auto deploy configured when there is a push on main branch.
deku-c is on the url, but I can see deku-p on the UI I think we should


Merge v26.1.4

Merge release v26.1.4

Pull Request Test Coverage Report for Build 5465

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.936%

Totals

Change from base Build 5460:
0.0%

Covered Lines:
1092

Relevant Lines:
1127

💛  - Coveralls


[Snyk] Security upgrade org.owasp.esapi:esapi from <IP_ADDRESS> to <IP_ADDRESS>

Snyk has created this PR to fix 1 vulnerabilities in the maven dependencies of this project.
Snyk changed the following file(s):

pom.xml

Vulnerabilities that will be fixed with an upgrade:

Issue
Score
Upgrade

Cross-site Scripting (XSS) SNYK-JAVA-ORGOWASPESAPI-6091110
  172
org.owasp.esapi:esapi:  <IP_ADDRESS> -> <IP_ADDRESS>  No Known Exploit

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)

🎉 Snyk hasn't found any issues so far.
✅ code/snyk check is completed. No issues were found. (View Details)


Sometimes Monster Cards Show Blank

I have noticed this odd issue with my monster cards that show up as blank until I hit "refresh" when coming from another tab.  I am using Chrome 67 on Windows 10, have also cleared cache but the issue keeps coming back.  This is what it looks like:

This is what my configuration looks like, don't want to crowd the space so linking to my github:
https://github.com/dshokouhi/Home-AssistantConfig/blob/master/ui-lovelace.yaml#L459
associated error:
2:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1 Uncaught (in promise) TypeError: Failed to execute 'fetch' on 'ServiceWorkerGlobalScope': 'only-if-cached' can be set only with 'same-origin' mode
    at Object.<anonymous> (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at Generator.next (<anonymous>)
    at n (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at self.babelHelpers.asyncToGenerator (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at new Promise (<anonymous>)
    at Object.self.babelHelpers.asyncToGenerator (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at Object.workbox.core.j.fetch (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at :8123/frontend_latest/workbox-v3.3.0/workbox-strategies.prod.js:1
    at Generator.next (<anonymous>)
    at n (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
2:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1 Uncaught (in promise) TypeError: Failed to execute 'fetch' on 'ServiceWorkerGlobalScope': 'only-if-cached' can be set only with 'same-origin' mode
    at Object.<anonymous> (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at Generator.next (<anonymous>)
    at n (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at self.babelHelpers.asyncToGenerator (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at new Promise (<anonymous>)
    at Object.self.babelHelpers.asyncToGenerator (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at Object.workbox.core.j.fetch (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)
    at :8123/frontend_latest/workbox-v3.3.0/workbox-strategies.prod.js:1
    at Generator.next (<anonymous>)
    at n (:8123/frontend_latest/workbox-v3.3.0/workbox-core.prod.js:1)

This was a bug corrected some time ago (https://github.com/ciotlosm/custom-lovelace/commit/29d210adea77f29661e554d0cde9e91de2300c63#diff-8b2c45b836e58d0cec1f55baf87a7e80) Make sure you update to the latest version, and add ?v=0.0.1 at the end of your "resources" URL to get latest version.

ok trying that now

thank you for the quick reply :)


Double reference generated when using runtime value for name of existing resource

Repro:

main.bicep:module foo './foo.bicep' = {
  name: 'foo'
}

resource site 'Microsoft.Web/sites@2022-03-01' existing = {
  name: foo.outputs.faName
}

output principalId string = site.identity.principalId

foo.bicep:output faName string = 'blah'

Deploy main.bicep through the VSCode UI, and the following error is returned:
{
    "error": {
        "code": "InvalidTemplate",
        "message": "Deployment template validation failed: 'The template output 'principalId' at line '1' and column '781' is not valid: The template function 'reference' is not expected at this location. Please see https://aka.ms/arm-template-expressions for usage details.. Please see https://aka.ms/arm-template-expressions for usage details.'.",
        "additionalInfo": [
            {
                "type": "TemplateViolation",
                "info": {
                    "lineNumber": 1,
                    "linePosition": 781,
                    "path": "properties.template.outputs.principalId"
                }
            }
        ]
    }
}

Originally reported here: https://twitter.com/marcjacobi/status/1597222500173492225
Feels like a similar root cause to #4815

@anthony-c-martin - I think double reference should be blocked by https://github.com/Azure/bicep/pull/8965. I got the following error with the sample code:
![image](https://user-images.githubusercontent.com/16367959/206310505-59c269cf-c8cd-4446-a489-6dfefeb67026.png)

Thanks for looking into it (and fixing the original issue)! I agree it's most likely an old version of Bicep - closing.


Add timing to the build goal

When building a docker image, then a lot of time can be spent in

pulling images
building the .tar

I would like if the d-m-p could report how long time that task took in seconds.
[INFO] - Pulling images took 49 seconds
[INFO] - Building tar took 37 seconds

And I guess there is also timing for a docker:push could be good to have as well.

This would be very helpful. Good idea!

this shouldn't be too difficult, i'll see about getting this done along side some other things i'm doing.


unable to install client with the given steps.

$ make test  . All tests fail.

i m installing on ubuntu 14.04 . everything is configured but at last when running "$ make test " in src/aerospike . All tests fail.

Great. So it builds fine, then you sudo make install, then you configured your php.ini, right?
Do you have an aerospike server running locally before calling make test?

no, my server is on another machine, but i have tested also on a machine where aerospike server is locally installed. i have tried on both machines.

OK, you will need to define where your server is for the tests.
Create a file in your Aerospike client for PHP directory at src/aerospike/test/aerospike.local.inc. For example my two node cluster has the following:
<?php
    define("AEROSPIKE_CONFIG_NAME", "<IP_ADDRESS>");
    define("AEROSPIKE_CONFIG_PORT", 3000);
    define("AEROSPIKE_CONFIG_NAME2", "<IP_ADDRESS>");
    define("AEROSPIKE_CONFIG_PORT2", 3010);
?>

Your configuration can have a single NAME/PORT pair, too.
Then
cd src/aerospike
./build.sh
# edit the php.ini as described at the end of the build
sudo make install
make test

Beyond the actual tests, which we run before releasing, and which run on Travis-CI, the main question I have is whether the module is installed for you, and whether you can run the example scripts.

all tests pass except :
Put - PUT List on Wrong IP [tests/phpt/Put/CheckListWithWrongIP.phpt]
Reconnect - Check for Basic Reconnect after close for a non-persistent connection. [tests/phpt/Reconnect/TestReconnectNonPersistentConnectionPositive.phpt]
and here is my php snapshot...
 [
							["addr" => "my server address", "port" => 3000]
					]
			];

            try {
                $db = new Aerospike($config);
            }catch (Exception $ex){
                echo $ex;
            }
?>
after running this script,  Fatal error: Class 'Aerospike' not found in /var/www/html/src/xyz.php .

OK, we're getting progress :+1: . Those tests are known problems and in progress.
Let's check if your module is installed:
php -m | grep aerospike
This should show that the aerospike extension installed for this PHP interpreter (the sudo make install step). Sometimes there are multiple PHP interpreters and the one it installed for isn't the one you're running the script with...

yes, it is showing aerospike.
and $ php -i | grep aerospike gives following :
/etc/php5/cli/conf.d/aerospike.ini
aerospike
aerospike support => enabled
aerospike version => 3.0
Registered save handlers => files user aerospike
OLDPWD => /home/ubuntu/vendor/aerospike/aerospike-client-php/src/aerospike/tests
PWD => /home/ubuntu/vendor/aerospike/aerospike-client-php/src/aerospike
_SERVER["OLDPWD"] => /home/ubuntu/vendor/aerospike/aerospike-client-php/src/aerospike/tests
_SERVER["PWD"] => /home/ubuntu/vendor/aerospike/aerospike-client-php/src/aerospike
and how do i get to know whether i am running the correct interpretor.

phpinfo(); in the above script show PHP Version 5.5.9-1ubuntu4.6,
but not showing aerospike in the modules.

Okay, so at least we know the PHP interpreter on your PATH is working and loading the module.
which php should show its path.
If you run the following:
which php
php -m | grep aerospike
php -i | grep _SERVER | grep _

The info variable SERVER[""] shows the path of the executing script.
So one thing could be the case here - if your script is executing in a webserver context it will be calling a different interpreter. The command line one (CLI) has its own config file, and the one used by the webserver is different. For example if it's an Apache (Prefork MPM) with mod_php, or if it's Nginx using PHP-FPM. Please let me know how you're running that script.
Look inside /etc/php5 - there's cli in there, but also different subdirectories. Make sure the config block loading the aerospike module is in both versions. ls /etc/php5 ?

thanks alot. it was apache with mod_php. i have changed the apache.ini with those 3 lines of directives. and now it is working. one more thing $db->get($key, $record, [ ], $option ).  Is $record is the desired result? If not then how can i get those records from the get call.
once again thanks.

No problem. PHP configuration is still confusing at times, after doing this for 16 years.
Yes, that's one way to get it, though you don't need the third and fourth arguments unless you need to limit the bins you get back, or want to change some default behavior of the client.
Take a look at the PHP documentation on the Aerospike site, specifically the Quick Guide. Next, run some of the examples. Finally, the GitHub repository includes the API documentation.
Let me know if you feel like those need improvement. It's a work in progress and I appreciate the feedback.


add spatial navigation

I have a vim plugin that allows me to use the same bindings to move between vim panes and dvtm splits (similar to vim tmux navigator).
I've been using tmux recently but would like to switch back to dvtm, so I've resurrected this code.

Sorry for taking that long to provide some feedback, a couple of comments:

The keybindings (Ctrl-{h,j,k,l}) you introduce aren't really suitable, they should be within the MOD "keyspace".
Your implementation won't work for minimized windows. In this case you should fallback to the current behavior based (on next/prev pointers)
Your indentation is off
Your code is needlessly verbose e.g. you introduce local variables for no apparent purpose

Cleaning these things up results in the following diff:
diff --git a/config.def.h b/config.def.h
index fec230a..16d053f 100644
--- a/config.def.h
+++ b/config.def.h
@@ -82,8 +82,10 @@ static KeyBinding bindings[] = {
 	{ { MOD, 'C',          }, { create,         { NULL, NULL, "$CWD" }      } },
 	{ { MOD, 'x', 'x',     }, { killclient,     { NULL }                    } },
 	{ { MOD, 'j',          }, { focusnext,      { NULL }                    } },
-	{ { MOD, 'J',          }, { focusnextnm,    { NULL }                    } },
-	{ { MOD, 'K',          }, { focusprevnm,    { NULL }                    } },
+	{ { MOD, 'J',          }, { focusdown,      { NULL }                    } },
+	{ { MOD, 'K',          }, { focusup,        { NULL }                    } },
+	{ { MOD, 'H',          }, { focusleft,      { NULL }                    } },
+	{ { MOD, 'L',          }, { focusright,     { NULL }                    } },
 	{ { MOD, 'k',          }, { focusprev,      { NULL }                    } },
 	{ { MOD, 'f',          }, { setlayout,      { "[]=" }                   } },
 	{ { MOD, 'g',          }, { setlayout,      { "+++" }                   } },
diff --git a/dvtm.c b/dvtm.c
index ce43fb8..3474106 100644
--- a/dvtm.c
+++ b/dvtm.c
@@ -185,6 +185,10 @@ static void focusnextnm(const char *args[]);
 static void focusprev(const char *args[]);
 static void focusprevnm(const char *args[]);
 static void focuslast(const char *args[]);
+static void focusup(const char *args[]);
+static void focusdown(const char *args[]);
+static void focusleft(const char *args[]);
+static void focusright(const char *args[]);
 static void killclient(const char *args[]);
 static void paste(const char *args[]);
 static void quit(const char *args[]);
@@ -1237,6 +1241,50 @@ focuslast(const char *args[]) {
 }

 static void
+focusup(const char *args[]) {
+	if (!sel)
+		return;
+	Client *c = get_client_by_coord(sel->x, sel->y - 2);
+	if (c)
+		focus(c);
+	else
+		focusprev(args);
+}
+
+static void
+focusdown(const char *args[]) {
+	if (!sel)
+		return;
+	Client *c = get_client_by_coord(sel->x, sel->y + sel->h + 1);
+	if (c)
+		focus(c);
+	else
+		focusnext(args);
+}
+
+static void
+focusleft(const char *args[]) {
+	if (!sel)
+		return;
+	Client *c = get_client_by_coord(sel->x - 2, sel->y);
+	if (c)
+		focus(c);
+	else
+		focusprev(args);
+}
+
+static void
+focusright(const char *args[]) {
+	if (!sel)
+		return;
+	Client *c = get_client_by_coord(sel->x + sel->w + 1, sel->y);
+	if (c)
+		focus(c);
+	else
+		focusnext(args);
+}
+
+static void
 killclient(const char *args[]) {
 	if (!sel)
 		return;

I'm a bit reluctant to change the current behavior of MOD-{h,j,k,l} because I'm used to them from dwm.
Also did you notice that you can focus a specific window n with MOD+n?

Also did you notice that you can focus a specific window n with MOD+n?

Yeah, I know about this. I have a vim plugin that lets me navigate vim splits w/ ctrl-{h,j,k,l} and its kind of nice to be have those bindings be seamless between dvtm and vim (via a plugin for vim).
I don't care about the bindings at all, I can take them out, but the navigation functions might have some usefulness for some plugins. If you think so, I'll make the suggested changes, else, no worries.

I merged the core functionality but left out the FIFO command names for now.


Implement is_safe_function() under model/filters

    To be implemented is_safe_function()

Originally posted by @Cyber-JiuJiteria in https://github.com/ossf/alpha-omega/pull/55#discussion_r1042851091

To be moved to Triage portal


WIP: Unit Tests for Issue #1605 AWS Signature Version 4

Unit tests related to PR #1946 in a separate PR (including the code that the test run against).
Unit test related files:
plugins/s3_auth/test/aws_auth_v4_test.cc
plugins/s3_auth/test/test_all.cc
This is WIP PR to facilitate the discussion about an ATS "official" unit test framework and eventually convert the tests to whatever framework we agree upon.
I needed an unit test framework so I can bootstrap quickly to do TDD in order to deliver the AWS auth v4. I chose Google Test because it is popular, powerful and easy to use, also License is "BSD 3-clauses"
These unit tests don't do anything fancy or Google Test specific so we can convert them to any other framework of choice.

I definitely think we need to be very cautious about bringing in dependencies on test packages. The potential for proliferation and compatibility is vast.  For this reason any such dependencies need to be integrated to be general and usable by any plugin. I'll make some additional specific comments on that.

@gtenev I'm going to close this for now, we agreed to rewrite this for the Catch testing framework I think? If I'm wrong, please reopen.

@zwoop, you are right, we agreed to rewrite it to whatever unit test platform we agree upon using.
As I mentioned in the PR description:

These unit tests don't do anything fancy or Google Test specific so we can convert them to any other framework of choice.

I was planning to use this PR to push my Catch unit tests.

Cool, thanks @gtenev

Converted the unit-tests to CATCH, please review!


Delay accounts service startup

As it turns out the race condition between accounts <-> storage-metadata still remains. This PR is a hotfix and it should be followed up with a proper fix. Either:

block the accounts initialization until the storage metadata is ready (using the registry) or
allow the accounts service to initialize and use a message broker to signal the accounts the metadata storage is ready to receive requests.

CI keeps of failing with random problems, those hopefully would be fixed by https://github.com/owncloud/ocis/pull/1669


[WIP] just sinc

Trying to debug the x86 remaining failure

Codecov Report
Patch coverage: 62.50% and project coverage change: -53.84 :warning:

Comparison is base (835082e) 77.32% compared to head (57dcf5c) 23.49%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #757       +/-   ##
===========================================
- Coverage   77.32%   23.49%   -53.84%
===========================================
  Files          18       18
  Lines        7365     7049      -316
===========================================
- Hits         5695     1656     -4039
- Misses       1670     5393     +3723

Impacted Files
Coverage Δ

src/compiler.jl
21.01% <62.50%> (-56.83%)
:arrow_down:

... and 12 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


upgrade request dependency & mocha version

Resolves #9

Hmmm not sure - I was able to run all the mocha tests successfully locally for the PR. I don't have any visibility into the Travis CI, but maybe a brief call together would help us get to the bottom of the issue? I actually sent you an email about this some weeks ago, but you can reach me at<EMAIL_ADDRESS>


Update code to run on new OR-Tools

Currently runs on OR-Tools version: 9.3.10497
At one point, need to update the code to run on the updated version of OR-Tools

The changes needed for our script would likely be similar to those applied to the OR-Tools sample code.

Closed by #5


adding the multiplication function

resolves #4

cant see the problem, total amature

Have this resolved the issue?


Creating chart to manage opa policies and an admission webhook

TODO: Going to add tests to this to make sure the policies get applied correctly.