Missing example of a file passed to --scale-rules param

Hi,
May anyone provide an example of a file with scale rules used for example in an update command: az containerapp update --name $CONTAINER_APP_NAME --resource-group $RESOURCE_GROUP --scale-rules scale.yaml?
I've tried using output from az containerapp show --output yaml, but with no success:
scale:
      maxReplicas: 5
      minReplicas: 0
      rules:
      - azureQueue: null
        custom: null
        http:
          auth: null
          metadata:
            concurrentRequests: '1'
        name: httpscale

Thanks in advance

@L-Sypniewski The file should be just the rules array from https://docs.microsoft.com/en-us/azure/container-apps/scale-app
For example, if you want to configure an http scale rule, you can do
- name: httpscale
  http:
    metadata:
      concurrentRequests: '1'

and you can use
az containerapp update --name $CONTAINER_APP_NAME \
    --resource-group $RESOURCE_GROUP \
    --min-replicas 0 \
    --max-replicas 5 \
    --scale-rules scale.yaml

If you want to add another rule to the same app (so the app will scale if concurrentRequests > 1 OR the average CPU > 75%
- name: httpScale
  http:
    metadata:
      concurrentRequests: '1'
- name: cpuScaleRule
  custom:
    type: cpu
    metadata:
      type: AverageValue
      value: '75'

--dapr-components is also undocumented.

@takekazuomi An example can be found here. According to the docs:

Container Apps does not currently support the native Dapr components schema. The above example uses the supported schema.

@ahmelsayed I've tried using file format you suggested, but I'm getting the following error: --scale-rules: type field required in rules config.
Command I used: az containerapp update --name $CONTAINER_APP_NAME --resource-group $RESOURCE_GROUP --scale-rules scale.yaml
I managed to run the command using the following file:
- name: cpuscalerule
  custom:
  type: cpu
  metadata:
    type: AverageValue
    value: '75'
- name: httpscalerule
  type: http
  metadata:
    concurrentRequests: '4'

Unfortunately CPU rule is not applied as the output after running the command is as follows:
"rules": [
        {
          "azureQueue": null,
          "custom": null,
          "http": {
            "auth": null,
            "metadata": {
              "concurrentRequests": "4"
            }
          },
          "name": "httpscaleyaml"
        }
      ]

/cc @calvinsID
@L-Sypniewski it should be
 - name: cpuscalerule
   custom:
-  type: cpu
-  metadata:
-    type: AverageValue
-    value: '75'
+    type: cpu
+    metadata
+      type: AverageValue
+      value: '75'
 - name: httpscalerule
   type: http
   metadata:
     concurrentRequests: '4'

note the extra indentations. or you can use JSON too
[
  {
    "name": "cpuScaleRule",
    "custom": {
      "type": "cpu",
      "metadata": {
        "type": "AverageValue",
        "value": "75"
      }
    }
  },
  {
    "name": "httpScale",
    "http": {
      "metadata": {
        "concurrentRequests": "1"
      }
    }
  }
]

az containerapp update --name $CONTAINER_APP_NAME --resource-group $RESOURCE_GROUP --scale-rules scale.json

@ahmelsayed thanks for your reply.
As for CPU rules neither .yaml with fixed indentations nor .json work for me - I'm still getting --scale-rules: type field required in rules config error. Regarding http scaling I had to modify the .json file as follows:
[
  {
    "name": "httpscale",
    "type": "http",
    "metadata": {
      "concurrentRequests": "8"
    }
  }
]

Hey @L-Sypniewski can you please try doing the update using the --yaml parameter? Unfortunately the --scale-rules flag is buggy, it will also be deprecated & removed soon in favor of the --yaml parameter

Hi - same issue as @L-Sypniewski .
type field required in rules config with a Json file and --scale-rules.
Trying with the YAML equivalent, and --scale-rules or --yaml:
- name: queue-based-autoscaling
  custom:
    type: azure-servicebus
    metadata:
      queueName: jobs
      messageCount: '5'
    auth:
    - secretRef: servicebusconnectionstring
      triggerParameter: connection

--> AttributeError: 'list' object has no attribute 'get'

With --scale-rules and this content, there is no error, but the scale rule is not created (the revision is).
- name: queue-based-autoscaling
  type: azure-servicebus                  # add this line
  custom:
    type: azure-servicebus                # remove this line
    metadata:
      queueName: jobs
      messageCount: '5'
    auth:
    - secretRef: servicebusconnectionstring
      triggerParameter: connection

Command is:
az containerapp create `
  --name XXX `
  --resource-group XXX `
  --environment XXX `
  --image XXX `
  --secrets "servicebusconnectionstring=$xxx,blobstorageconnectionstring=$xxx" `
  --environment-variables "xxx" `
  --registry-login-server XXX `
  --registry-username XXX `
  --registry-password XXX `
  --min-replicas 1 `
  --max-replicas 1 `
  --scale-rules scale-rules.yaml

Same without --min-replicas and --max-replicas.

closing due to time period that has passed and increased documentation


What does bbox_mAP_copypaste mean?

What does bbox_mAP_copypaste mean?
Thank you！

Any help is greatly appreciated!

Nothing special but for convenience for your copy to elsewhere.

thank you


Debug Launch Not Building Project

Bug
When launching a C# http trigger function created with this project, the "host start" task fails saying it cannot find the specified dll in the bin folder. The bin folder is not there because the build task did not run even though it is clearly depended on in the "host start" task.
Only works if you first do a dotnet build in terminal or manually fire the build task. And you must continue to do these manual steps if you wish to see any code changes take effect.
NOTE: The build task is present and set as a depends on in the func start task. These task and a couple of others are created by this extension.
Steps to Reproduce

Create a new function project with a C# http trigger using this extension
Once the project is created, go to debug and select "Attach to..." and try running it.

Environment

Latest VS Code Preview (although I noticed this prior to the past couple of daily updates
Latest version of this extension v0.22.1
<EMAIL_ADDRESS>

I can reproduce this on VS Code Insiders, but not stable VS Code. @allan-mobley-jr is that what you're seeing as well?

Yes @EricJizbaMSFT.
I use VS Code Insiders exclusively.

Filed an issue on VS Code: https://github.com/microsoft/vscode/issues/101884

Fixed for me on the latest insiders. Thanks for filing @allan-mobley-jr!

Awesome! I'll try it out.


Server(config, callback).start() causes node process to hang

Expected behaviour
When a karma server has finished running, it should clean up so that node can exit, e.g.:
new Server(config, callback).start()

Actual behaviour
The node process hangs, and doesn't exit for upwards of a few minutes.
Environment Details

Latest Karma version (1.7.1/master branch):
Uses the repo karma.conf.js file: test/client/karma.conf.js with ChromeHeadless as the browser

Steps to reproduce the behaviour
Note: This isn't 100% repo (as sometimes writeBuffer is empty when karma server calls .disconnect on the socket, see below for more details).

Create a karma test runner file, e.g.:

var Server = require('./lib/server')

new Server({
    configFile: "...karma-master/test/client/karma.conf.js",
    reporters: ['dots'],
    singleRun: true
}, (exitCode) => {
    console.log(`Exited with code: ${exitCode}`)
}).start()

run from commandline: node runkarma.js
re-run if repo doesn't hit

Cause
From debugging under node --inspect I see an engine.io socket stays alive preventing node from exiting.
Furthermore I see its caused by the writeBuffer containing messages preventing closeTransport from being called.
file: engine.io\lib\socket.js
Socket.prototype.close = function (discard) {
  if ('open' !== this.readyState) return;

  this.readyState = 'closing';

  if (this.writeBuffer.length) {
    this.once('drain', this.closeTransport.bind(this, discard));
    return;
  }

  this.closeTransport(discard);
};

Even more specifically its because the socket needs to cleanup the pingTimeout, which is done inside onClose which is called by closeTransport.
Note
Although the default behavior is to have the callback call process.exit, it doesn't work in our environment because:

We use karma inside a gulp task, which then runs subsequent tasks, so we don't want to exit straight away
We use "gulp watch" to run build/test tasks and this causes the subsequent watch tasks to stall
I see in the main repo they use grunt to spawn a child process, however we use JavaScript to configure the karma config, so that won't be a clean solution, and

Can you create a (failing) test in this repo as a PR to illustrate the problem?

I can attempt to make one, its a little tricky because the bug is in karma not releasing the node instance, so the test would have to verify that the node instance exited (I'm not familiar with writing a test like that).

Heya, I was faced with a similar problem today. As far as I can tell, it's because browser disconnection is setup on single run mode:
https://github.com/karma-runner/karma/blob/master/lib/server.js#L380-L383
But not on watch mode. So on watch mode even if you do something like
const karmaServer = new karma.Server({singleRun: false, configFile: '...'});
karmaServer.start();
karmaServer.emit('exit', () => { });

Karma would never really exit since the browsers are not disconnected.
I tried my hand at a PR for this in https://github.com/karma-runner/karma/pull/2946.

Actually, now that I read the original issue again, I'm not too sure my analysis and fix suit. The original issue is for singleRun mode only, whereas my problem is on watch mode only.


[Request] Soundux

Package Request
Criteria
For a package to be acceptable in the Extras bucket, it should be:

[x] a well-known and widely used tool (e.g. if it's a GitHub project, it should have at least 100 stars and 50 forks) (note: it only has 35 forks at the time of writing, but there are multiple merged PRs that come from forks that don't exist anymore)
[x] the latest stable version of the program

Information
Name: Soundux
Description: A cross-platform soundboard.
Homepage: https://soundux.rocks
Download link(s): https://soundux.rocks/download
Some indication of popularity/repute: Over 830 stars on GitHub

@rasa Would this fit the criteria even though it has under 50 forks despite having over 100 stars?


[iOS] Update Mokoresource to use with iOS

Issue

close #119 and #259

Overview (Required)

In the previous PRs #441 #450 #462 #482, we have migrated all the string usage for Android and now we already using it on the Android side without much of problem.

However, this is not the best outcome yet. In order to fully utilize the moko-resource, we should share the string as a single source of truth for iOS as well.

Changes that I have introduced in this PR
I have migrated the usage of the string resource together with the Android side. The usage on the iOS is the following;
StringsKt.shared.[string_name].desc().localized()

e.g.
StringsKt.shared.about_description.desc().localized()

Comparing to Android side
stringResource(Strings.about_description)

You can check all of the strings that have been declared on the strings.xml here
Last but very important information for using the moko-resource on the iOS.
After build, in order to load the bundle correctly, we need to run this gradle task
"$SRCROOT/../../gradlew" -p "$SRCROOT/../../" :appioscombined:copyFrameworkResourcesToApp \
    -Pmoko.resources.PLATFORM_NAME=$PLATFORM_NAME \
    -Pmoko.resources.CONFIGURATION=$CONFIGURATION \
    -Pmoko.resources.BUILT_PRODUCTS_DIR=$BUILT_PRODUCTS_DIR \
    -Pmoko.resources.CONTENTS_FOLDER_PATH=$CONTENTS_FOLDER_PATH

For iOS to be able to run gradlew as a part of the build phase, we have to tell the script where is the java_home location ...
Since the Java_home location is different for each developer, we have to somehow read it from the safe location (easily modifiable / untrack by source control).
Location is here (/local.properties)

Important
From now, in order to run iOS, we need to set the java_home location on the /local.properties file with key org.gradle.java.home
e.g. my java_home is the following;

For @ry-itto, I have tried to use the SPM plugin but with my inexperience on SPM and iOS in general, I couldn't be able to achieve SPM plugin instead of use the run script directly. I would love to use SPM pre-build plugin instead but I couldn't make it work for a few days so I gave up. I have left the trail here, just in case you want to explore further.
With this proposal being accepted, Strings module on the iOS will be deprecated and should be removed.
Thank you! and let me know if you have question or problem.. @takahirom @ry-itto
Links

Screenshot

A merged PR https://github.com/DroidKaigi/conference-app-2022/pull/615 has String resource change.
Please fix conflicts 🙇🏼

Sure let me work on the conflict tonight then.

@takahirom with the introduction of project folder structure, it makes the iOS CI choked 😢. (In my humble opinion, if you ask me, I prefer a flat structure rather than this folder setup.)
However, I would love to give this a try. I will try to see whether I can mess around with the iOS bundle creation to solve the issue.

Sorry, I didn't notice that 🙇

Conflicts fixed! Still finding a good to get around the upload task though 😓. Please wait a bit.

I have resolved it with using zip instead. Please take a look again.
For next PR, I will clean up Strings for iOS and add some README.md

@takahirom
Please check this PR's Android/Kotlin part 🙏🏼

If possible, in order to avoid AppStore Review rejection, I would like you to create an extension that removes the word Android from the text for iOS project (I think it should be placed in Strings module).

Let me check for that in the next PR but I think ideally we should not have the hardcoded string "Android" somewhere. I will help monitor that too

Sure! Fixed 💪

@ry-itto It's okay, please ping me when you are merging stuff. I can keep merging stuff, no worries.

@kittinunf
I merged that PR 🙏🏼 (This ping is too late..)

I'll merge this PR 🙏🏼  Thank you!
@takahirom
This PR has a risk of conflict, so I'll merge this PR before your review 🙏🏼
If there is something happened, let's do it with issue!

Thanks so much! @ry-itto
Just FYI, iOS build now requires java home to be set on the local.properties file.


reproduce the deadlock described in #15195

Try to reproduce the deadlock described in #15195
DON'T MERGE
Signed-off-by: Shijiang Wei<EMAIL_ADDRESS>

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

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

@mountkin since you don't want this to be merged, I'm going to close it.
If you disagree let us know.
Also feel free to continue the discussion about the deadlock on the issue.
For next time, pasting a link to your branch without opening a PR should be sufficient :)

@tiborvass I think it's not ready for merging so I added the tip. I'll be busy during the weekend.
If it's still not fixed by the next Monday, I'll dig into it deeper.


Replacing the GTL lists

The goal is to replace the GTLs with Agular based and JSON-backed https://github.com/hawkular/hawkular-ui-components
TODOs:

figure out where to put the component (separate repo? together with other components such as the Dialog editor? Under the ManageIQ organization or elsewhere?)
handle Quadicon rendering for the component
figure out toolbars for Angular only pages (either rewrite the final toolbar markup generation to Javascript (app/helpers/toolbar_helper.rb) or find a pattern to request and render the toolbars from Angular.
Implement a JSON endpoint around get_view app/controllers/application_controller.rb L1495 feed the component with these data

I have written function for fetching the toolbar data from app/helpers/toolbar_helper.rb so I am currently rewriting this part, once this is done I'll show it to you and discuss how would be the best way to render these angular pages.

@karelhala @martinpovolny this isn't really stale, right?

We have some "spin offs", but what is written here is done.


Implement Get email template endpoint (Admin)

Description:
This issue involves implementing an API to Get email template. The endpoint should allow Admin to make a GET request to {{baseUrl}}/api/admin/emails/templates/{{templateId}}.
{
    "templateId": 323,
    "content": "
title
"
}

Tasks:

Create a controller for the /controllers/email endpoint in the backend (create the route email if not created).
Parse and validate the request body to ensure it matches the expected format.
Implement appropriate error handling and response status codes for different scenarios (e.g., validation errors, database errors).
Write unit tests to validate the functionality and correctness of the endpoint.

API documentation: https://documenter.getpostman.com/view/27421496/2s93m1a4ac#8744a3ee-970f-489a-853d-8b23fdee8de3
ER diagram: https://drive.google.com/file/d/11KMgdNu2mSAm0Ner8UsSPQpZJS8QNqYc/view
Acceptance Criteria:

The profile API endpoint /controllers/email is implemented via a GET request.
The request body is properly parsed and validated for the required format.
Appropriate error handling is implemented, providing meaningful error messages and correct response status codes.
Unit tests are written to validate the functionality and correctness of the endpoint.

Additional Information:
No
Related Dependencies or References:
No

@anjula-sack  Shall I work on this issue?

@anjula-sack  can you please confirm the issue status

@dsmabulage This issue is no longer valid.


add unified diff separator

fix issue #2295
the purpose of this feature is to make the unified diff
format more readable.  Without it, different portions
of the same actual/expected diff look contiguous even
though they are actually separated by the default
context of four lines.

I know the label said 'semver major' - but I didn't see a major branch to point the PR to.  I figured master was the correct one, but please let me know if that's incorrect.

Coverage remained the same at 88.288% when pulling 8cf22228ca430e651d266ad2edfb41b71d02f9e7 on olsonpm:feature/unified-diff-separator into a054380203d1d7f271ff0649656779915ac28e63 on mochajs:master.

@ScottFreeCode - Just pushed the removal of the unused idx you pointed out.  All we need is a branch to push to now.

Coverage remained the same at 88.395% when pulling 26aaf51f2121c304973277402afc0af7531e910e on olsonpm:feature/unified-diff-separator into f20de56637b2223f614ce40adc0d41a58030f042 on mochajs:master.

i appreciate the time you spent with this.  i know it wasn't a high priority nor highly visible issue.


Support for more complex JsonPath

The StepFunctions implementation of JsonPath allow us to filter elements using the ?() syntax.
When I try to test a state machine that use a filter, the validation fails and if I try to bypass the validation, the engine returns an error.
Step to reproduce
import { StateMachine } from 'aws-local-stepfunctions';

const machineDefinition = JSON.parse(JSON.stringify({
  StartAt: 'GetSubjects',
  States: {
    GetSubjects: {
      Type: 'Task',
      Resource: 'arn:aws:lambda:us-east-1:123456789012:function:GetSubjects',
      Next: 'ReturnResult'
    },
    ReturnResult: {
      Type: 'Pass',
      Parameters: {
        'completed_los.$': '$.Payload[?(@.subject_status==subject_completed)]..subject_id',
      },
      End: true
    }
  },
}));

test('OK', async () => {
  // GIVEN
  const myInput = {};

  // WHEN
  const stateMachine = new StateMachine(machineDefinition, {
    // validationOptions: {
    //   checkPaths: false,
    // }
  });
  const execution = stateMachine.run(myInput, {
    overrides: {
      taskResourceLocalHandlers: {
        "GetSubjects": ((event: any) => {
          return {
            Payload: [
              { subject_id: 1, subject_status: 'subject_completed' },
              { subject_id: 2, subject_status: 'subject_completed' },
              { subject_id: 3, subject_status: 'subject_added' }
            ]
          }
        })
      },
    },
  });
  const result = await execution.result;
  console.log(result);

  // THEN
  expect(result).toStrictEqual([1,2]);
});

Desired outcome
The state machine should process the JsonPath filter.
Error
tate machine definition is invalid, see error(s) below:
     SCHEMA_VALIDATION_FAILED: /States/ReturnResult/Parameters/completed_los.$ is invalid. must match format "asl_payload_template"

By setting checkPaths: false, the state machine is executed, but I get the following error:
ExecutionError: Execution has failed with the following error: jsonPath: subject_completed is not defined: @.subject_status==subject_completed

By changing the definition to wrap the condition with double quotes $.Payload[?(@.subject_status=="subject_completed")]..subject_id, it works, but unfortunately that doesn't work on step function:

Workaround
To make it work, I could pre-process the state machine definition to double quote any jsonpath expression, but that would be quite a good effort, moreover, I'm not keen to disable validations.

Hi @redvex,
This is an issue that has to do with the JSONPath library used in this project, which is jsonpath-plus. It seems this kind of syntax is not supported by that library. I too have noticed some path expressions which are valid in the AWS implementation do not work as expected in aws-local-stepfunctions, and vice versa.
Unfortunately, since JSONPath is not standardized, libraries tend to differ in its implementation. There's an RFC proposal to standardize the JSONPath spec, but that's still a draft.


Impl Delete method for Cacher interface.

Describe the feature
Hi there,
There are three methods of Cacher interface: Get/Store/Invalidate, that is great. Can you impl a new method "Delete" for it, I think "Delete" is important and the Cacher interface would be completed.
Thank you.
Motivation
In some scenarios, the data in Cacher would be delete.
Related Issues
None

Don't need to impl Delete interface.


newest mxnet build failed with operator/tensor/la_op

For bugs or installation issues, please provide the following information.
The more information you provide, the more likely people will be able to help you.
Environment info
Operating System:
Ubuntu 14.04
Compiler: 4.8.4
Package used (Python/R/Scala/Julia):
MXNet version: 0.11.1
Or if installed from source:
MXNet commit hash (git rev-parse HEAD): 3f742d254846fbc14dbaa459191403330425ef83
If you are using python package, please provide
Python version and distribution:
If you are using R package, please provide
R sessionInfo():
Error Message:
Please paste the full error message, including stack trace.
cclplus: all warnings begin treated as errors
make: *** [build/src/operator/tensor/la_op.o] Error

Minimum reproducible example
if you are using your own code, please provide a short script that reproduces the error.
Steps to reproduce
make -j4
only build with gpu will make this error.

What's the error? The last line is a note. Not the error

@piiswrong  error is:
g++ -std=c++11 -c -DMSHADOW_FORCE_STREAM -Wall -Wsign-compare -g -Werror -O3 -DNDEBUG=1 -I/home/wuwei/project/dmlc/mxnet/mshadow/ -I/home/wuwei/project/dmlc/mxnet/dmlc-core/include -fPIC -I/home/wuwei/project/dmlc/mxnet/nnvm/include -I/home/wuwei/project/dmlc/mxnet/dlpack/include -Iinclude -funroll-loops -Wno-unused-variable -Wno-unused-parameter -Wno-unknown-pragmas -Wno-unused-local-typedefs -msse3 -I/usr/local/cuda/include -DMSHADOW_USE_CBLAS=1 -DMSHADOW_USE_MKL=0 -DMSHADOW_RABIT_PS=0 -DMSHADOW_DIST_PS=0 -DMSHADOW_USE_PASCAL=0 -DMXNET_USE_PROFILER=1 -DMXNET_USE_OPENCV=1 -I/usr/local/include/opencv -I/usr/local/include   -fopenmp -DMSHADOW_USE_CUDNN=1  -I/home/wuwei/project/dmlc/mxnet/cub -DMXNET_USE_NVRTC=0 -MMD -c src/executor/attach_op_execs_pass.cc -o build/src/executor/attach_op_execs_pass.o
g++ -std=c++11 -c -DMSHADOW_FORCE_STREAM -Wall -Wsign-compare -g -Werror -O3 -DNDEBUG=1 -I/home/wuwei/project/dmlc/mxnet/mshadow/ -I/home/wuwei/project/dmlc/mxnet/dmlc-core/include -fPIC -I/home/wuwei/project/dmlc/mxnet/nnvm/include -I/home/wuwei/project/dmlc/mxnet/dlpack/include -Iinclude -funroll-loops -Wno-unused-variable -Wno-unused-parameter -Wno-unknown-pragmas -Wno-unused-local-typedefs -msse3 -I/usr/local/cuda/include -DMSHADOW_USE_CBLAS=1 -DMSHADOW_USE_MKL=0 -DMSHADOW_RABIT_PS=0 -DMSHADOW_DIST_PS=0 -DMSHADOW_USE_PASCAL=0 -DMXNET_USE_PROFILER=1 -DMXNET_USE_OPENCV=1 -I/usr/local/include/opencv -I/usr/local/include   -fopenmp -DMSHADOW_USE_CUDNN=1  -I/home/wuwei/project/dmlc/mxnet/cub -DMXNET_USE_NVRTC=0 -MMD -c src/executor/attach_op_resource_pass.cc -o build/src/executor/attach_op_resource_pass.o
In file included from src/operator/tensor/./../linalg.h:167:0,
from src/operator/tensor/./la_op_inline.h:27,
from src/operator/tensor/la_op.cc:25:
src/operator/tensor/./../linalg_impl.h: In function ‘int linalg_gelqf_workspace_query(const mshadow::Tensor<Device, 2, DType>&, mshadow::Stream) [with xpu = mshadow::cpu; DType = float]’:
src/operator/tensor/./../linalg_impl.h:843:37: error: ‘work’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
int ws_size(static_cast(work));
^
src/operator/tensor/./../linalg_impl.h:838:9: note: ‘work’ was declared here
DType work;
^
src/operator/tensor/./../linalg_impl.h:852:1: note: in expansion of macro ‘LINALG_CPU_GELQF_WORKSPACE_QUERY’
LINALG_CPU_GELQF_WORKSPACE_QUERY(s, float)
^
src/operator/tensor/./../linalg_impl.h: In function ‘int linalg_gelqf_workspace_query(const mshadow::Tensor<Device, 2, DType>&, mshadow::Stream) [with xpu = mshadow::cpu; DType = double]’:
src/operator/tensor/./../linalg_impl.h:843:37: error: ‘work’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
int ws_size(static_cast(work));
^
src/operator/tensor/./../linalg_impl.h:838:9: note: ‘work’ was declared here
DType work;
^
src/operator/tensor/./../linalg_impl.h:853:1: note: in expansion of macro ‘LINALG_CPU_GELQF_WORKSPACE_QUERY’
LINALG_CPU_GELQF_WORKSPACE_QUERY(d, double)

@piiswrong  try to fix this as pull request: https://github.com/apache/incubator-mxnet/pull/7819


[Register for Build-a-thon 2022]:

Team name
Team Prime
Total number of team members
4
Contact Details of team member 1
<EMAIL_ADDRESS>Contact Details of team member 2
<EMAIL_ADDRESS>Contact Details of team member 3
<EMAIL_ADDRESS>Contact Details of team member 4
<EMAIL_ADDRESS>Contact Details of mentor
No response
tell us more about yourself
Rahman Alam,Masters Student,IIT Delhi, India
Yogesh Mathpati,Masters Student,IIT Delhi, India
Mahadev,Masters Student,IIT Delhi, India
Kalpesh More,Masters Student,IIT Delhi, India
Code of Conduct

[X] I agree to follow the Challenge Code of Conduct

Is there any update?


KYLIN-4864 Fix .travis.yml syntax

Proposed changes
The commented line about # mvn sonar:sonar -e -Dsonar.host.url=https://sonarcloud.io ... has broken the YAML and because of this TravisCI didn't execute any jobs.
I wasn't able to find a way to preserve the commented line and fix the syntax, so I removed the line. Later it could be get from Git history to re-enable Sonar.
While here I've improved few more things:

install OpenJDK and unzip for both AMD64 and ARM64. This way it uses latest JDK 8 (1.8.0_292) for both CPU architectures
fix more YAML warnings

Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ x ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ x ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
This PR fixes the setup of TravisCI.
But it seems there are failing tests from before which are not related to this PR.

This PR is related to https://github.com/apache/kylin/pull/1558.
// CC @liusheng

Codecov Report

Merging #1700 (d80ab78) into master (0250081) will decrease coverage by 0.31%.
The diff coverage is 2.61%.

@@             Coverage Diff              @@
##             master    #1700      +/-   ##
============================================
- Coverage     25.42%   25.11%   -0.32%
- Complexity     6765     6771       +6
============================================
  Files          1508     1518      +10
  Lines         93918    95184    +1266
  Branches      13158    13294     +136
============================================
+ Hits          23877    23902      +25
- Misses        67662    68897    +1235
- Partials       2379     2385       +6

Impacted Files
Coverage Δ

.../java/org/apache/kylin/common/KylinConfigBase.java
12.39% <0.00%> (-0.08%)
:arrow_down:

...org/apache/kylin/common/restclient/RestClient.java
17.64% <0.00%> (-0.15%)
:arrow_down:

...g/apache/kylin/common/util/CliCommandExecutor.java
31.03% <0.00%> (-1.50%)
:arrow_down:

.../java/org/apache/kylin/common/util/ServerMode.java
0.00% <0.00%> (ø)

...in/java/org/apache/kylin/cube/CubeDescManager.java
32.41% <0.00%> (-1.16%)
:arrow_down:

.../main/java/org/apache/kylin/cube/CubeInstance.java
35.57% <0.00%> (-0.12%)
:arrow_down:

...c/main/java/org/apache/kylin/cube/CubeManager.java
35.83% <0.00%> (-0.81%)
:arrow_down:

...c/main/java/org/apache/kylin/cube/CubeSegment.java
41.97% <0.00%> (-0.31%)
:arrow_down:

.../org/apache/kylin/cube/cuboid/CuboidScheduler.java
85.29% <ø> (ø)

.../java/org/apache/kylin/cube/cuboid/CuboidUtil.java
53.33% <ø> (ø)

... and 143 more

Continue to review full report at Codecov.

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

Thanks @martin-g !

@hit-lacus Welcome!
Where could I find the source code of the website ? I'd like to fix https://kylin.apache.org/development/#ci-and-code-analysis. Currently it points to Jenkins, i.e. error 404.

I think I've found it!
http://svn-master.apache.org/repos/asf/kylin/site/ - is this the one ?

I think I've found it!
http://svn-master.apache.org/repos/asf/kylin/site/ - is this the one ?

The source code of apache kylin is located at https://github.com/apache/kylin . http://svn-master.apache.org/repos/asf/kylin/site/ is the official website .

Following two files should be what you want :

https://github.com/apache/kylin/blob/document/website/_dev/index.md
https://github.com/apache/kylin/blob/document/website/_dev/index.cn.md

Docu update - https://github.com/apache/kylin/pull/1704


Trackpad and Trackpoint went wrong at sometimes.

My trackpad and trackpoint always went wrong at sometimes with your EFI, it happens randomly and can be restored by rebooting.
Thanks for the share of your EFI, it saves a lot of time!

im very very sorry for the late response, what is the update now ?

Haha that's fine. I've gave up your files and found someone else's to ignore the issue😂


No Toolbar Visible

I'm trying to use this gem for my text fields . It appears to be working but there is not toolbar showing above the text field. I can tell its working because the field is resizable using my mouse.

I've never used it for text fields - only textarea elements.
I'd suggest looking at the moxicode web site: http://www.tinymce.com.
This isn't an issue with the gem - it's a TinyMCE issue


Help Command

This is the addition of the kin help command

This is the addition of the kin help command

I've already corrected the typo

From: Murangwa Pacifique @.>
Sent: Tuesday, October 17, 2023 8:15 AM
To: kin-lang/kin @.>
Cc: D Regis @.>; Author @.>
Subject: Re: [kin-lang/kin] Help Command (PR #21)
@pacifiquem requested changes on this pull request.
Check the requested change, for the arc variable in main function.

In src/main.chttps://github.com/kin-lang/kin/pull/21#discussion_r1361704718:

int main(int argc, char const argv[]){
/ entry point of our interpreter */

 if (argc == 1) {
     repl(); /* enter our repl */

}else if(argc == 2) {

}else if(arc == 2) {

isn't this a typo? arc it was supposed to be argc
—
Reply to this email directly, view it on GitHubhttps://github.com/kin-lang/kin/pull/21#pullrequestreview-1681661640, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4EGICRTTXADTJJ2GMNET23X7Y5DHAVCNFSM6AAAAAA6DLNTEKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMOBRGY3DCNRUGA.
You are receiving this because you authored the thread.Message ID: @.***>

This pull request will appear in v0.0.0-pre-release


Rewrite the readme file

Move all contents of previous version to another page and put them to legacy branch.
After updated content in MSLearn, update the doc link in readme.
Release a track1 version to update readme page in pkg.go.dev.

@xboxeer @sandeep-sen @lirenhe @RickWinter for comments.

@tadelesh, could you explain more details about (1),  where are those contents now and what is the legacy branch?

@tadelesh, could you explain more details about (1), where are those contents now and what is the legacy branch?

For current readme, you could see Client: Previous Versions and Management: Previous Versions part that is for track1. We've already moved all track1 code from main branch to legacy branch, thus related doc should be placed in the legacy branch.


Incorrect "smallint" definition when exporting to Laravel

Columns of the type "smallint" currently use a $table->smallint method which doesn't exist in Laravel. This should be $table->smallInteger.

I think this is fixed? in Commit 7864260f66c41f6ee9abe4a9add54c4852732153

Looks like it 👍


RFC: Fix constructor for OrderedDict

When constructing a dictionary literally using Pair syntax, construction would fail if
any of the Pairs were not the same type as the dictionary (including
OrderedDict{Any,Any}.  Loosened this restriction.
Also allow constructing OrderedDict{K,Any} when the keys are the same,
but the values have different types
Note that constructing and OrderedDict{Any,V}} using Pairs is not
currently supported

(Still need to add a test for the second change)

Rather out of date, and hopefully obsoleted by #170.


add .append to TextEditTransaction

Just sugar for:
.append(text) => .edit(original.length, original.length, text)

This issue has been moved to dart-lang/source_maps#13.


Bug: Insert Timestamp Button Crashes

When I right click and then select Insert Timestamp it crashes the client.
Plugin v.1.0.2
Canary 96441 (d18df0a)
Powercord 5d7500d

Also clicking the Button crashes

It was working as soon as I made it what

Still crashing btw,
VM88 C:\Users\Jake\Documents\powercord\src\preload.js:333 TypeError: Cannot read property 'username' of undefined     at get_msg (VM1447 C:\Users\Jake\Documents\powercord\src\Powercord\plugins\send-timestamps\components\Preview.jsx:49)     at Settings (VM1447 C:\Users\Jake\Documents\powercord\src\Powercord\plugins\send-timestamps\components\Preview.jsx:66)     at AsyncComponent.componentDidMount (VM876 C:\Users\Jake\Documents\powercord\src\fake_node_modules\powercord\components\AsyncComponent.js:18)

I'm not sure, it's working for me?


function for cif file IO and possible support for multi-dataset files

Make a record here in case we soon forget about it in the future.
Yesterday I had a discussion with Kevin @kmdalton about how to read structure factor data in cif format. And turns out it can be done with cif parser from gemmi (although not very well documented there).
Mostly Kevin's wisdom:
f = gemmi.cif.read_file("7l84-sf.cif")
refl_blocks = gemmi.as_refln_blocks(f)
refl_block = refl_blocks[0]
mtz = gemmi.CifToMtz().convert_block_to_mtz(refl_block)
ds = rs.DataSet.from_gemmi(mtz)

The above can be easily organized into a function like rs.read_cif().
One thing is that with possible multi datasets cif file, to be general, we have to think about the return as a representation of multi datasets. Maybe a generator? rs.read_cif(cif: str) -> generator ? Or move a step forward, we could implement a new class like DataSetCollection with methods to deal with multi datasets? A lot of decisions to make here.

I like the concept -- I agree that cif format can and should be supported because it is already handled by gemmi.  Regarding multi-dataset formats, a similar issue crops up with mtz files. Unmerged MTZs are currently handled using just a BATCH identifier, but technically the MTZ format supports things like individual unit cell parameters for different batches (particularly useful with serial data).
We do not currently handle this to the full extent that we could, and full support would likely require a new class as you said. I'm not yet sure how to do this in a way that still maintains the feel of pandas in a clean way.

i think we can easily support cif just as @minhuanli demonstrated. honestly, writing the test is the hardest part of that PR.
the multi-dataset object is a tricky one. there's probably a decent enough way to implement it, but it is not immediately obvious to me.

cif support was added in #217


Add CompressionLevel Calculation for PQ

Description
Currently, for product quantization, we set the calculated compression level to NOT_CONFIGURED. The main issue with this is that if a user sets up a disk-based index with PQ, no re-scoring will happen by default.
This change adds the calculation so that the proper re-scoring will happen. The formula is fairly straightforward =>
actual compression = (d * 32) / (m * code_size). Then, we round to the neareste compression level (because we only support discrete compression levels).
One small issue with this is that if PQ is configured to have compression > 32x, the value will be 32x. Functionally, the only issue will be that we may not be as aggressive on oversampling for on disk mode.
Check List

[X] New functionality includes testing.
[X] Commits are signed per the DCO using --signoff.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

One small issue with this is that if PQ is configured to have compression > 32x, the value will be 32x. Functionally, the only issue will be that we may not be as aggressive on oversampling for on disk mode.

should we allow more compression level?

@navneet1v we did here: https://github.com/opensearch-project/k-NN/issues/1779#user-content-appendix-b-baseline-rescore-experiments

@navneet1v we did here: #1779 (comment)

Thanks. I completely forgot about that.


Where to download the STAR videos?

Seems like using Charades_v1 videos does not work. The filename are not matched. Specifically, your filename is
File not found: ./STAR/Charades_v1/Z97SD_18.1_24.1.mp4
while Charades_v1/Z97SD.mp4 does exits.
Should I update the filename?
Also, for TVQA, should I just ask authors for the videos? Can you give me a copy?
Thank you,
Mu

I also need TVQA video, which is not published by official TVQA group. @imagegridworth
and I guess STAR video named Z97SD_18.1_24.1mp4 may be created by clipping the Z97SD.mp4 by [start_time, end_time].
Thanks!


Second Contribution by Tanishka Goel

Name: Tanishka Goel
Projects analyzed: signac | FluxML
Governance Model Links: Project 1 signac governance model | Project 2 FluxML governance model
Governance Model Overview of FluxML
The FluxML governance model follows the approach of contribution-based merit which is not solely based on the code contributions and rather has a holistic view of contributions to the community in both quality and quantity. It divides its roles between Project Contributors, Steering Council, Advisory Committee, and Community. The main features of their model are as follows:

Consensus-based and values majority vote
Strongly believes in inclusivity and diversity
Simple yet effective rules for the community

Governance Model Overview of signac
The governance model of signac follows a Meritocratic Governance Model and divides its major roles between Users (Committers, Contributors, Project Maintainers, and Administrators) and Project Managers (Triage and Community Managers). It has the following features:

Consensus-based and values majority vote
Inclusive
Open to all

Analysis of the two Models
Similarities:

Both models are open-source, community-based, and inclusive of people from all backgrounds.
The decision-making process involves majority voting and is based on the consensus of all.
Each of the models is thorough in defining project roles and responsibilities.
Commitment to the project, alignment with project goals, and quality contributions to the community are valued.
They place importance on the growth and progress of the project.

Differences:

Project Roles:

The FluxML model has an Advisory Committee in addition to Project Contributors and Council.
The signac model has multiple divisions under the User and Manager roles and does not have an Advisory Committee.

Decision-making:

FluxML model is solely based on clear majority voting.
signac model has a 2/3rd voting system for the Project Maintainer role.

Privileges:

FluxML gives no privileges to the Steering Council except for decision-making power for the project.
Project Maintainers have a higher position than Committers and members and can accept or reject them in the signac model.

Merit:

FluxML has a holistic view of merit which considers peer participation, different viewpoints, outreach, etc. apart from code contributions while considering members of the council. Nominations are done by the Council.
Only project contributions and a strong understanding of project strategy are enough for signac. Self-nomination or people with the existing role can nominate for a position.

Special Guidelines/Roles:

Fiscal decisions are taken by the Committee in the FluxML model. There is also a special guideline for when a Conflict of Interest arises.
No role is explicitly specified for making financial decisions and there are no guidelines to resolve Conflicts of Interest in the signac model.

Which Model do I consider better and Why?
I consider the FluxML model to be better than the signac model because of the following reasons:

The model is simple to understand and apply without any complicated rules for nomination or hierarchy.
It implements the concept of inclusivity and diversity in the truest form by having an overall approach and not having any set quantitative merit benchmarks which could play a role in the selection. When considering members, one's participation in the mailing list/chat group, outreach and education, code contributions and code reviews, and community building.
Despite the rules being simple, the model is effective in thorough with the responsibility for the contributors, Council and Committee members, and the community. There is flexibility for being a part of the Council with the availability of a grace period after inactivity and there is no ejection from the community by only a few people being in charge for doing so.
There is a special Advisory Committee which I believe is important for the proper functioning of projects and taking financial decisions. Guidelines for Conflict of Interest are also taken into consideration which could result from people with various life backgrounds.
There is no unnecessary role and the project management is simpler with all the users being allowed to contribute and only the members of the Council delegating the project - from raising issues to merging contributions and working towards the growth of the project.

@arliss-NF I would be glad to receive any modifications/instructions you have for me. :)
Thank you!

@tg-100 - great analysis.  it was very easy to read.  Very concise, so it was a quick read which is appreciated when there are over 57 projects to be analyzed.  Your arguments are solid and exactly what we wanted to see. Well done!

@arliss-NF I am really grateful to you for giving your time and giving me such personalized and encouraging feedback. Thanks so much!


esp_rmaker_param_add_valid_str_list() - possible list update? (MEGH-4202)

Is it possible to update valid string list after it was added?
When is the list sended to server? I can't see it in monitor.
Is it possible to place string list in RAM, not in FLASH?

Yes, it is possible. If you want to add/remove any devices and want the changes to get reflected in the phone app, then you need to call the esp_rmaker_report_node_details() API after you add/remove the device. To see the logs, you have to set the Log Output level to "Debug" from idf.py menuconfig.

Thank You @sanketwadekar
I know and use esp_rmaker_report_node_details() API.
So can implement constesp_rmaker_param_t *param in RAM rather than FLASH and can modify this list before use  esp_rmaker_report_node_details(). It's clear now.
Next question is Can I change 'count' parameter after esp_rmaker_param_add_valid_str_list(const[esp_rmaker_param_t](https://docs.espressif.com/projects/esp-rainmaker/en/latest/c-api-reference/rainmaker_core.html?highlight=valid_st#_CPPv418esp_rmaker_param_t) *param, const char *strs[], uint8_t count)?
Or Can use esp_rmaker_param_add_valid_str_list(const[esp_rmaker_param_t](https://docs.espressif.com/projects/esp-rainmaker/en/latest/c-api-reference/rainmaker_core.html?highlight=valid_st#_CPPv418esp_rmaker_param_t) *param, const char *strs[], uint8_t count) a few times for the same parameter (or is there any API for re-create valid string list)?

If you are increasing/decreasing items in the list of valid strings, then you should also change your count parameter accordingly. You can call the esp_rmaker_param_add_valid_str_list API multiple times even for the same parameter.

thank You @sanketwadekar
that's exactly what I wanted to get

You can call the esp_rmaker_param_add_valid_str_list API multiple times (with the updated list and count) for the same parameter.
I suggest adding this information to esp_rmaker_param_add_valid_str_list API description.

I succesfully implemented dynamic valid_str_list for parameter. Thank You.

Can You confirm if calling  esp_rmaker_param_update_and_report() will report parameter valid_str_list also (earlier updated), or have to call esp_rmaker_report_node_details() ?

Since valid_str_list is part of node config, you will have to call esp_rmaker_report_node_details() to update any changes related to it.

Thank You @shahpiyushv
Is it possible to add some API for valid_str_list report? We're using dynamic valid_str_lists and have to unnecessarily increase traffic on each updating list.

The backend actually just overwrites whatever config is sent by the node. Adding a facility to update smaller snippets would require changes in backend logic including support for new topics for this. We may not consider this in near future. Are the changes really that frequent to cause an impact on network traffic?

Are the changes really that frequent to cause an impact on network traffic?
No, it is not raported very often. Usually while device configurating only. I though about AWS costs rather than network overload.
sometimes x thousands_of_users = bigger_AWS_costs.
But I understand,  it doesn't make sense since it is not simple to add this API to backend and we can live with it.

@jacek12345 , AWS IoT messages are anyways metered in 5 KB increments. So, if your node config is less than 5KB, the optimisation of message size would not help. Moreover, ESP RainMaker now uses basic ingest topics and so, there is no messaging cost, but only processing cost.


bump to contracts alpha 31

Changes proposed in this PR:

remove joinPool & exitPool
use barge env to set contracts, until barge default contract version will be alpha.31

It seems there is still a vesting related test

on it.  strange, because on localhost I have ```
253 passing (1m)


Support for django_pylibmc (Heroku)?

Hi there. I was looking to use django-cachalot with a Django site I've deployed on Heroku. I'm using the third-party django_pylibmc package to make use of Heroku's Memcached addon, because Django's default pylibmc (i.e. django.core.cache.backends.memcached.PyLibMCCache) does not support the binary protocol (see this StackOverflow post and this DjangoProject ticket). With the django_pylibmc package, I am able to get my Heroku site working with Memcachier.
Unfortunately, django_pylibmc does not work with django-cachalot. When I run heroku run python manage.py migrate, I get the following error message:
(cachalot.E001) `django_pylibmc.memcached.PyLibMCCache` is not compatible with django-cachalot.

Are there any plans for supporting django_pylibmc? I'd really love it if I could use django-cachalot on my Heroku site, since it seems to be a very robust and useful package.

Hello @lakshbhasin!
There is no plan for supporting django-pylibmc. I understand that the binary protocol may be useful to some users. But adding a supported backend would add 33 test runs to the test matrix that already contains 165 test runs and takes about an hour to run. Since django-pylibmc is a tiny addition (a few dozens of lines) from the Django pylibmc backend, it’s not worth it. I see django-pylibmc as a patch to django/django#15815. It just needs to be pushed upstream to Django by someone.
In the meantime, I tested locally and everything works. So you can still ignore the error by using SILENCED_SYSTEM_CHECKS. It won’t be officially supported, but it works with django-cachalot 1.1.0 and will work with django-cachalot 1.2.0 (that will be released today or in a few days).

As a side note an hour to run the test suite is excessive, it sounds like
there is considerable room for improvement. Have you tried parallelizing
the tests?
On 12 Jan 2016 11:33, "Bertrand Bordage"<EMAIL_ADDRESS>wrote:

Hello @lakshbhasin https://github.com/lakshbhasin!
There is no plan for supporting django-pylibmc. I understand that the
binary protocol may be useful to some users. But adding a supported backend
would add 33 test runs to the test matrix
https://travis-ci.org/BertrandBordage/django-cachalot that already
contains 165 test runs and takes about an hour to run. Since django-pylibmc
is a tiny addition (a few dozens of lines) from the Django pylibmc backend,
it’s not worth it. I see django-pylibmc as a patch to django/django#15815
https://code.djangoproject.com/ticket/15815. It just needs to be pushed
upstream to Django by someone.
In the meantime, I tested locally and everything works. So you can still
ignore the error by using SILENCED_SYSTEM_CHECKS
https://docs.djangoproject.com/en/1.9/ref/settings/#silenced-system-checks.
It won’t be officially supported, but it works with django-cachalot 1.1.0
and will work with django-cachalot 1.2.0 (that will be released today or in
a few days).
—
Reply to this email directly or view it on GitHub
https://github.com/BertrandBordage/django-cachalot/issues/42#issuecomment-170883550
.

@orf It’s not a single test suite that takes one hour, it’s the whole test matrix that tests django-cachalot against:

Python 2.7, 3.2, 3.3, 3.4, 3.5
Django 1.7, 1.8, 1.9
PostgreSQL, MySQL, SQLite
Redis, python-memcached, pylibmc, locmem, filebased

It therefore runs the test suite 165 times. Each test suite takes about 1 minute 30 seconds, and Travis CI runs 3 test suites in parallel, so we end up with about an hour of execution.
In django 1.9, the tests are run in parallel, but it has nearly no effect on the test suite of django-cachalot, since most of the time on Travis consists in building the environment.

You may want to clear out versions that have reached end-of-life, such as Django 1.7.

@JockeTF: Yes, that’s what I wanted to do for version 1.2.0, but supporting Django 1.7 currently doesn’t require any compatibility code, so I’m in favor of supporting it yet a few months.

SILENCED_SYSTEM_CHECKS only works for WARNINGS. So there is now effective way of ignoring errors except for monkeypatching cachalot.apps.VALID_CACHE_BACKENDS. This also disalbes use of even slightly modified cache backends. Maybe VALID_CACHE_BACKENDS should be configurable through settings?

@imposeren No, it works for all checks, including ERROR. Try or read the docs.


Update isearchpanequerychangedeventargs.md

Whitespace.
This is a test.
***Please do not merge!!!

@MancunianBandit : Thanks for your contribution! The author(s) have been notified to review your proposed change.


bug: store the actually updated field when onlyCareUpdateColumns is true

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Ⅰ. Issue Description
Currently, when onlyCareUpdateColumns is true, and the before/after image will only store the update rows in set condition, but sometimes, it will lost the fields that was auto update, like on update xxx in mysql.
So our idea is to store the fields that is actually being update
If you want to fix it, please comment "Please assign it to me" is this issue.
当前,如果onlyCareUpdateColumns=true, 然后seata的更新的前后镜像中只会记录在set表达式中写了的字段, 有些被自动更新的字段,则不会被记录到, 有可能会导致回滚时校验镜像失败. 因此, 期望在记录更新的前后镜像时, 是去记录真实被更新了的字段(set表达式中包含的字段 & 被数据库自动更新的字段)
如果你有兴趣修复这个问题的话, 请在下方留言 Please assign it to me
Ⅱ. Describe what happened
If there is an exception, please attach the exception trace:
Just paste your stack trace here!

Ⅲ. Describe what you expected to happen
Ⅳ. How to reproduce it (as minimally and precisely as possible)

xxx
xxx
xxx

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

JDK version :
OS :
Others:

Please assign it to me ~


How to declare my attributes for faceting ?

Hi,
I want to know how to declare my attributes for faceting ?
I don't want to declare them in my dashboard, because there are generated.
I see the searchParameters options but i d'ont find how to declare the attributes for faceting with instant Search.
when i do:
var search = instantsearch({
appId: 'myAppId',
apiKey: 'MyKey',
indexName: 'Agoarts',
searchParameters: {
facets: [
'coloris'
]
}
});
There are nothing, i see hts and results, but not my coloris filter.
I have to go through the js api ?
If yes, how to do that with Instant Search and it's widgets ?
Thanks you for your attention :)

Hi Clementvp, you need to declare the attributes for faceting in the dashboard OR use the REST settings call to set them from a script for instance that would run on your infrastructure: https://www.algolia.com/doc/rest#change-index-settings
The JS client has a setSettings method: https://github.com/algolia/algoliasearch-client-js#index-settings
Then once the attributes for faceting are defined in your settings, you can use any refinementList or menu or hierarchicalMenu widget.
Let me know how it goes.

Ok good form me with the REST API.
But when i try with the JS client and the setSettings method, like that:
var client = algoliasearch('APPID', 'APPKEY');
var index = client.initIndex('Agoarts');
index.setSettings({attributesForFaceting: ['coloris', 'dimension', 'prixttc']});
Console give me:  AlgoliaSearchError: This method is only available in HTTPS. Why ?
( my tests are in local).
Thank you for your reactivity.

@clementvp when you run a JS script with the JS client we default to the page protocol for all calls. This is something we can fix for all writes to the API since they need to be done in https.
You can either run this script in node.js OR do this:
var client = algoliasearch('APPID', 'APPKEY', {protocol: 'https:'});

Thanks

Ok thanks :)


Nearby pins hard to see in dark mode

Maybe another color set would work better?
The same color should be used and clearly visible in light mode too.

I have some color ideas to apply. So, may I take this issue?

I had this color in my mind and now it seems to be visible for both themes.
Light Mode

Dark Mode

Please share your thoughts about this color, i.e., Light Sea Green.

Looks good to me! :-)

Should I create a PR or wait for more suggestions?

Yes, please create a PR. :-)


update docs: fix typo, missing comma

I noticed a missing comma in the conf example while following the docs.

Thank you, @mwt!


Resolve/supress issues reported in sonar /compiler warnings

With #124 solved, I want to tackle the reported issues, ideally solve them - or suppress the warnings where I don't agree.
Also, bring down the detekt baselines to be empty in the process.

Closed with several commits w/o referencing this issue - hence closing manually:

bde5208c7f1eba6fcf0168efa2db62d044445c06
ce240e0199249dd88542c00c90d992ab217cbea3
99a99f93bd2b9d9ff1d4e973eb5d08693fff76f9


fix: update rocks for 1.8

Description
Required updates for ROCK for 1.8
Summary of changes:

Updated service command to use env vars.
Added comment on usage of env vars.
Updated sanity tests to use env vars.

Sample sanity test on `jupyter-scipy` ROCK:
$ tox -e sanity
sanity: commands[0]> rockcraft pack
Retrieved base ubuntu:20.04 for amd64
Extracted ubuntu:20.04
Executed: skip pull conda-jupyter (already ran)
Executed: skip pull generate-locale (already ran)
Executed: skip pull kubectl (already ran)
Executed: skip pull nodejs (already ran)
Executed: skip pull non-root-user (already ran)
Executed: skip pull overlay-pkgs (already ran)
Executed: skip pull pebble (already ran)
Executed: skip pull security-team-requirement (already ran)
Executed: skip overlay conda-jupyter (already ran)
Executed: skip overlay generate-locale (already ran)
Executed: skip overlay kubectl (already ran)
Executed: skip overlay nodejs (already ran)
Executed: skip overlay non-root-user (already ran)
Executed: skip overlay overlay-pkgs (already ran)
Executed: skip overlay pebble (already ran)
Executed: skip overlay security-team-requirement (already ran)
Executed: skip build conda-jupyter (already ran)
Executed: skip build generate-locale (already ran)
Executed: skip build kubectl (already ran)
Executed: skip build nodejs (already ran)
Executed: skip build non-root-user (already ran)
Executed: skip build overlay-pkgs (already ran)
Executed: skip build pebble (already ran)
Executed: skip build security-team-requirement (already ran)
Executed: skip stage conda-jupyter (already ran)
Executed: skip stage generate-locale (already ran)
Executed: skip stage kubectl (already ran)
Executed: skip stage nodejs (already ran)
Executed: skip stage non-root-user (already ran)
Executed: skip stage overlay-pkgs (already ran)
Executed: skip stage pebble (already ran)
Executed: skip stage security-team-requirement (already ran)
Executed: skip prime conda-jupyter (already ran)
Executed: skip prime generate-locale (already ran)
Executed: skip prime kubectl (already ran)
Executed: skip prime nodejs (already ran)
Executed: skip prime non-root-user (already ran)
Executed: skip prime overlay-pkgs (already ran)
Executed: skip prime pebble (already ran)
Executed: skip prime security-team-requirement (already ran)
Executed parts lifecycle
Exported to OCI archive 'jupyter-scipy_v1.8.0_20.04_1_amd64.rock'
sanity: commands[1]> bash -c 'NAME=$(yq eval .name rockcraft.yaml) && VERSION=$(yq eval .version rockcraft.yaml) && ARCH=$(yq eval -r ".platforms | keys" rockcraft.yaml | cut -d" " -f2) ROCK="${NAME}_${VERSION}_${ARCH}" && sudo skopeo --insecure-policy copy oci-archive:$ROCK.rock docker-daemon:$ROCK:$VERSION && docker save $ROCK > $ROCK.tar'
Getting image source signatures
Copying blob 96d54c3075c9 done
Copying blob 10f031486a87 done
Copying blob c63838066321 done
Copying blob 6db45539ac8f done
Copying blob 6675d763c1ca done
Copying config e134a8c941 done
Writing manifest to image destination
Storing signatures
sanity: commands[2]> pytest -v --tb native --show-capture=all --log-cli-level=INFO /home/ubuntu/rocks/kubeflow-rocks/jupyter-scipy/tests
===================================================== test session starts =====================================================
platform linux -- Python 3.8.10, pytest-7.4.2, pluggy-1.3.0 -- /home/ubuntu/rocks/kubeflow-rocks/jupyter-scipy/.tox/sanity/bin/python
cachedir: .tox/sanity/.pytest_cache
rootdir: /home/ubuntu/rocks/kubeflow-rocks/jupyter-scipy
plugins: anyio-4.0.0, asyncio-0.21.1, operator-0.29.0
asyncio: mode=strict
collected 1 item

tests/test_rock.py::test_rock
------------------------------------------------------- live log setup --------------------------------------------------------
INFO     pytest_operator.plugin:plugin.py:647 Adding model microk8s-localhost:test-rock-oocd on cloud microk8s
PASSED                                                                                [100%]
------------------------------------------------------ live log teardown ------------------------------------------------------
INFO     pytest_operator.plugin:plugin.py:783 Model status:

Model           Controller          Cloud/Region        Version  SLA          Timestamp
test-rock-oocd  microk8s-localhost  microk8s/localhost  3.1.6    unsupported  12:18:13-04:00

INFO     pytest_operator.plugin:plugin.py:789 Juju error logs:

INFO     pytest_operator.plugin:plugin.py:877 Resetting model test-rock-oocd...
INFO     pytest_operator.plugin:plugin.py:882 Not waiting on reset to complete.
 Did you mean "=="?
  elif num_empty_lines is 2:
/opt/conda/lib/python3.8/site-packages/azure/storage/blob/_serialization.py:189: SyntaxWarning: "is" with a literal. Did you mean "=="?
  if requests is None or len(requests) is 0:
/opt/conda/lib/python3.8/site-packages/azure/storage/blob/_serialization.py:301: SyntaxWarning: "is not" with a literal. Did you mean "!="?
  if len(serialized_query) is not 0:
/opt/conda/lib/python3.8/site-packages/azure/storage/blob/_upload_chunking.py:409: SyntaxWarning: "is" with a literal. Did you mean "=="?
  if n is 0 or self._buffer.closed:
/opt/conda/lib/python3.8/site-packages/azure/storage/blob/baseblobservice.py:1071: SyntaxWarning: "is not" with a literal. Did you mean "!="?
  if lease_duration is not -1 and \
/opt/conda/lib/python3.8/site-packages/azure/storage/blob/baseblobservice.py:2779: SyntaxWarning: "is not" with a literal. Did you mean "!="?
  if lease_duration is not -1 and \
/opt/conda/lib/python3.8/site-packages/azure/storage/common/_connection.py:82: SyntaxWarning: "is" with a literal. Did you mean "=="?
  self.protocol = self.protocol if parsed_url.scheme is '' else parsed_url.scheme
/opt/conda/lib/python3.8/site-packages/azure/storage/blob/blockblobservice.py:608: SyntaxWarning: "is not" with a literal. Did you mean "!="?
  while len(data) < count and len(data_chunk) is not 0:
[I 231011 16:18:23 utils:157] NumExpr defaulting to 4 threads.
[I 231011 16:18:25 font_manager:1443] generated new fontManager
2023-10-11T16:18:27.163Z [pebble] GET /v1/changes/2/wait 7.25122249s 200
2023-10-11T16:18:27.179Z [pebble] GET /v1/changes/1/wait 7.285290881s 200
sanity: commands[4]> python /home/ubuntu/rocks/kubeflow-rocks/jupyter-scipy/tests/test_access.py
Running jupyter-scipy_v1.8.0_20.04_1_amd64:v1.8.0_20.04_1
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4033  100  4033    0     0   437k      0 --:--:-- --:--:-- --:--:--  437k
2023-10-11T16:18:28.014Z [pebble] Started daemon.
2023-10-11T16:18:28.022Z [pebble] POST /v1/services 7.99769ms 202
2023-10-11T16:18:28.023Z [pebble] Started default services with change 1.
2023-10-11T16:18:28.031Z [pebble] Service "jupyter" starting: bash -c './jupyter lab --notebook-dir=${HOME} --ip=<IP_ADDRESS> --no-browser --port=8888 --ServerApp.token="" --ServerApp.password="" --ServerApp.allow_origin="*" --ServerApp.base_url=${NB_PREFIX} --ServerApp.authenticate_prometheus=False'
9666e43cbf3a
9666e43cbf3a
  sanity: OK (113.90=setup[0.39]+cmd[59.94,27.38,12.06,7.91,6.22] seconds)
  congratulations :) (113.95 seconds)

Do you also test the HOME variable?


Feature Request: Support WSL2 paths for Open Folder

As the title says.  I don't want to use the network path because then it doesn't run vs code within WSL.  From powershell, you can run this command to do it: code -n --folder-uri=vscode-remote://wsl+Ubuntu/home/meenie/dev-project

Hey,
Did you try adding the full path to the "open folder" action? It should work.

Can you expand on this?  This doesn't seem to be working for me. I am putting this in the path:  \wsl.localhost\Ubuntu-22.04\home\username\code\
If I try to add vscode-remote://  it says the location does not exist.
It opens up but doesn't up on in a WSL session.

Am I missing something here?  I am putting \wsl.localhost\Ubuntu-22.04\home\username\code\work in my path but it just opens up in a non WSL window like meenie said.  How do you get this to open a folder in a WSL window?


Quick can now take multiple files as input

Usage:
mafia quick -i../disorder-core/src $(find src -name '*.hs')

^ I think with a bit of work we can make mafia repl do this automatically and at the same time prevent the source dependencies we know we will have loaded (in this case disorder-core) from being built.
/cc @markhibberd

For the non-bash non-savages:
mafia quick -i../disorder-core/src **/*.hs

For the non-bash non-savages:
mafia quick -i../disorder-core/src **/*.hs

wow, TIL

@charleso does this look ok now?

:+1:


Cleanup and refactor

Supersedes #1

@deviouspk See the latest commit, hopefully that explains what I was trying to say :stuck_out_tongue:

Re: https://github.com/francislavoie/laravel-websockets/pull/1#issuecomment-515792947
I was wrong, I misread how TriggerEventController works. I think we need to add a call to publish in there.

Alright, c0b6ba0 should hopefully make it work if a message is broadcast via the PusherBroadcaster.

@deviouspk any thoughts on this stuff?

@deviouspk any thoughts? I'll just merge this in a couple days if not.


Command line calls

As it stands, when we run pyspextool (at least this is how I do it), we do,
import pyspextool as ps
and then we do things like,
ps.pyspextool_setup()
ps.extract.load_image()
I really would like people to be able to just do,
import pyspextool
pyspextool_setup()
extract.load_image()
telluric.load_spectra()
etc.
I tried modified the init.py file in src/pyspextool/ to be
from . import extract as extract
from . import combine as combine
from . import telluric as telluric
from pyspextool.setup_utils import pyspextool_setup as pyspextool_setup
hoping the "as XXXX" would do the trick but this doesn't seem to work.  Any ideas?

To do this we would need to change the imports to:
from pyspextool import pyspextool_setup ,extract, telluric, etc.

Then we could run
extract.load_image()
telluric.load_spectra()

I can try playing around with this to see how the initialization needs to work, but in principle the above should work.

Oh I see.  This would be in place of import pyspextool as ps.
Is there a way to do this in the init file?  I am trying to minimize the complexity of getting setup for people.


Security scan

Run SonarCloud less often to reduce build times for normal CI builds.

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


Lint error when trying to install swarmkit

Per https://github.com/docker/swarmkit/blob/master/BUILDING.md, checked it out as noted, and ran into this lint error. Suggested on slack #swarm to file a PR here as this may be due to a new version of golint detecting a new issue. Note that golint is installed by the makefile AFAICT - I did not have it preinstalled.
~/go/src/github.com/docker/swarmkit$ make
:whale: fmt
:whale: bin/swarmd
:whale: bin/swarmctl
:whale: bin/swarm-bench
:whale: bin/swarm-rafttool
:whale: bin/protoc-gen-gogoswarm
:whale: binaries
:whale: vet
:whale: lint
manager/deks.go:244:16: should omit type encryption.Decrypter from declaration of var decrypter; it will be inferred from the right-hand side
manager/deks.go:260:16: should omit type encryption.Encrypter from declaration of var encrypter; it will be inferred from the right-hand side
Makefile:82: recipe for target 'lint' failed
make: *** [lint] Error 1

Thanks for reporting this. The error is strange because the type actually is necessary here. I can't reproduce this with the latest version of golint.
Can you try upgrading golint with go get -u github.com/golang/lint/golint to confirm that the latest version still has this problem?
Do you have any local modifications to swarmkit? (Does git diff show anything?)
If nothing else works, feel free to submit a PR that works around this.

@blsaws I'm closing this issue for now since there hasn't been any update on it, and it seems like the issue doesn't happen with the latest version of golint. If you continue to run into it, please feel free to reopen the issue or submit a PR.


Library name issue

The library's name is too long and complicated to remembre and could not be found on google search for local storage handling with react-native. How about rn-reactive-storage ?
regards,
Ayoub

thanks dear ayoub for your suggestion i think you have right the name is too long how about "rn-rx-storage" ?


Not able to connect to Crm 9.1 getting Authentication failure Error

We are getting this error as spkl is trying to connect using organization service but in microsoft last month went for upgrade all the crm instance and not allowing connection to crm suign orginaztion service instead we need to use crm proxy service.
Error Message "Exception Occured--An unsecured or incorrectly secured fault was received from the other party. See the inner FaultException for the fault code and detail."

Could you try using https://www.nuget.org/packages/spkl/1.0.394-beta

thanks Scott, got fixed by this release.


[5.7] Added fillOrNew method to Eloquent Builder.

This new method will allow you to either create a new instance with the given values or get an existing record that matches the given attributes and fill it with the given values without persisting it to the database.

1+

How is different than firstOrNew?

This also allows for updating values on the model, firstOrNew doesn't do this. This is basically the same as updateOrCreate but without persisting any data to the database.
This can either be created as a separate method or firstOrNew could be updated to fill the model with the values array.


[SPARK-19189] Optimize CartesianRDD to avoid parent RDD's partitions re-computation and re-serialization

What changes were proposed in this pull request?
Current CartesianRDD implementation, suppose RDDA cartisian RDDB, generating RDDC,
each RDDA partition will be reading by multiple RDDC partition, and RDDB has similar problem.
This will cause, when RDDC partition computing, each partition's data in RDDA or RDDB will be repeatedly serialized (then transfer through network), if RDDA or RDDB haven't been persist, it will cause RDD recomputation repeatedly.
In this PR, I change the dependency in CartesianRDD from NarrowDependency into ShuffleDependency, but still keep the way how the parent RDD partitioned. And computing CartesianRDD keep current implementation.
How was this patch tested?
Add a Cartesian test.

Test build #71320 has started for PR 16574 at commit 14ba3b2.

Test build #71320 has finished for PR 16574 at commit 14ba3b2.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71320/
Test FAILed.

Test build #71321 has started for PR 16574 at commit e114eed.

Test build #71321 has finished for PR 16574 at commit e114eed.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71321/
Test FAILed.

Test build #71322 has started for PR 16574 at commit 815063b.

Test build #71322 has finished for PR 16574 at commit 815063b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71322/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, test this please.

Test build #71328 has started for PR 16574 at commit 815063b.

Test build #71328 has finished for PR 16574 at commit 815063b.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71328/
Test FAILed.

This is a behavior change and will break expectations from existing code depending on cartesian to not go through shuffle (particularly when data is already persisted).

@mridulm En...so that still keep NarrowDependency seems better, but I think the recomputation is a serious problem when parents RDD not persisted, I think in this case we should try to print some warning message to remind developer to check their spark application code... how do you think about it ?

Couple of points :
a) Can recomputation be expensive ? Unfortunately, yes if not used properly. For better or for worse, this has been the implementation in spark since early days - pre-0.5; and the costs are known. Particularly given Apache spark's ability to cache/checkpoint data, the assumption is that shuffle is more expensive. This might not hold anymore actually, given improvements since 1.0 - but only redoing benchmarks will give a better picture.
b) If we were to do a shuffle for cartesian, I would implement it differently - take a look at how Apache Pig has implemented it for a more efficient way to do it. (Btw, I dont think the impl in the PR actually works, but I have not looked at it in detail).

@mridulm
Year, I know you are worried about the shuffling cost here. Currently when spark.shuffle.reduceLocality.enabled is true(by default), each shuffling reducer will be launched on the node with the largest outputs. So in this PR implementation it will generate good data-locality so that its network transfer cost is similar to current NarrowDependency implementation, IMO.
BUT, you mention that Cartesian has more efficient way to implement using shuffling... I would like to research about it and consider better solution. Thanks!

I need to make a survey for better Cartesian implementation, especially in shuffle way. Close this PR for now and when the new solution is done I will reopen it.


Add a rosdep key for wxwidgets for artful and bionic.

Signed-off-by: Chris Lalancette<EMAIL_ADDRESS>

Why did we put this instead of #17689 ?

My fault; I didn't realize #17689 was open when I opened this PR.  I don't care which way we go, we just need these keys for melodic.

My bad, I didnt see this was the same key as the other PR when reviewing and the change was valid so I went ahead.
Do you want me to open a new PR with the commits from #17689 ?


Support async requests using underlying Guzzle promises (Datastore)

I'm trying to execute multiple Datastore lookups and queries simultaneously as running them sequentially is too slow.
Looks like the client API uses Guzzle under the hood which supports concurrent requests.
I know I can access the underlying promises like this, but the wrapper API calls (->lookup, ->query, etc) are implemented synchronously.
    $handler = new CurlHandler();
    $stack = HandlerStack::create($handler);
    $stack->push(function (callable $handler) {
        return function (RequestInterface $req, array $options) use ($handler) {
            var_dump("Request Body:");
            echo (string) $req->getBody();

            $promise = $handler($req, $options);
            return $promise->then(function (ResponseInterface $res) {
                var_dump("Response Body:");
                echo (string) $res->getBody();

                return $res;
            });
        };
    });
    $client = new Client(['handler' => $stack]);
    $dsOptions["httpHandler"] = new Guzzle6HttpHandler($client);
    $ds = new DatastoreClient($dsOptions);

    // Is anything like the following planned?
    $promiseA = $ds->lookupPromise($manyKeys);
    $promiseB = $ds->lookupPromise($manyMoreKeys);
    $results = Promise\Utils::settle([$promiseA, $promiseB])->wait();

Is there any way around this?

Not an elegant way yet @calsmith, but we are currently working on a new version of the codebase which will treat async operations as first class elements, please see here to learn more.


feat: add functions for operating on polynomials (PROOF-913)

Rationale for this change
Add functions for operating on polynomials.
These will be used for sumcheck proofs and verifications.
What changes are included in this PR?
Add a new component polynomial_utility with polynomial operations.
Are these changes tested?
Yes

:tada: This PR is included in version 1.82.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[SC-855] SyncswapOracle

Codecov Report
Patch coverage has no change and project coverage change: -0.90 :warning:

Comparison is base (b7c079c) 78.26% compared to head (adf8d65) 77.36%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
- Coverage   78.26%   77.36%   -0.90%
==========================================
  Files          25       26       +1
  Lines         690      698       +8
  Branches      149      152       +3
==========================================
  Hits          540      540
- Misses        150      158       +8

Impacted Files
Coverage Δ

contracts/oracles/SyncswapOracle.sol
0.00% <0.00%> (ø)

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


Support for Mocking functions with layers and Docker

Note: If your feature-request is regarding the AWS Amplify Console service, please log it in the
official AWS Amplify Console forum

Is your feature request related to a problem? Please describe.
When I try to test layers locally I've seen errors which shows that it's not implemented yet. It would be nice if we could install yumda packages using Docker.
Unable to mock function. Mocking a function with layers is not supported. To test in the cloud: run "amplify push" to deploy your function to the cloud and then run "amplify console function" to test your function in the Lambda console.

Describe the solution you'd like
Use Docker to install yumda packages and test it locally, maybe with a flag --docker
Describe alternatives you've considered
Test locally using common mocks

I have the same problem...

Same problem here...

Same, amplify should support testing all the features with Docker, like Sam does at least

Any updates on it ?

same problem i am facing while running this command
amplify mock function storageTrigger
Unable to mock storageTrigger. Mocking a function with layers is not supported. To test in the cloud: run "amplify push" to deploy your function to the cloud and then run "amplify console function" to test your function in the Lambda console.

I wonder what's the challenge with enabling mocks for functions with layers? From what I saw, we just need to setup the correct NODE_PATH to the layer modules during runtime in order to make require work.

I've noticed this only seems to be a soft limitation of Amplify, since by removing the associated layer from the function's dependsOn array within amplify-meta.json, I'm still able to mock these functions as long as the assets for the layer are otherwise provided.
I've been working around this limitation using this method for a while now and haven't found any drawbacks of it as long as the dev env and build process are set up properly, so is there any chance this could be made toggleable with a switch on the mock command or a feature flag?

Any progress on the topic?

@mewtlu Thanks for sharing your solution.
Could please share some code examples? with all the things you tweaked.
Some more context of my case:
I use the Appsync graphql layer (const API = require('@aws-amplify/api').default;) inside a lambda function.
That lambda function is used to handle one route of the API Gateway.

@mewtlu Thanks for sharing your solution.
Could please share some code examples? with all the things you tweaked.
Some more context of my case:
I use the Appsync graphql layer (const API = require('@aws-amplify/api').default;) inside a lambda function.
That lambda function is used to handle one route of the API Gateway.

I'm not sure if there are any further issues related to the specifics of your situation, but to be able to mock any function that has a Lambda layer all that is required is manually editing your local amplify/backend/amplify-meta.json file to remove the Lambda Layer from the dependsOn array within that function's block in the Resource object.
I run the following bash script using jq whilst in in amplify/backend/ to do this automatically for me:
cat amplify-meta.json | jq \". * { function: .function | map_values(if .dependsOn then del(.dependsOn[] | select(.resourceName == \\\"MyLambdaLayerName\\\")) else . end) }\" > amplify-meta.n.json
(replacing MyLambdaLayerName with the name of your Lambda Layer)
Hope this helps.

@mewtlu how does your solution gets the related code stored in the lambda layer folders?

@mewtlu how does your solution gets the related code stored in the lambda layer folders?

If you're asking how we provide the code in the layer to the functions within the dev environment I simply use local npm dependencies, within my frontend application's package.json I have an npm dependency on the package within the monorepo which contains the code used for the Lambda Layer such as:
"@org/shared": "../lib/shared"
and the backend functions have a dependency on the same local package such as:
"@org/shared": "../../../../lib/shared"
As the Lambda layer contains TypeScript code/types we also use TS project references to support this which nicely links up in  a similar manner using relative paths within the monorepo, then we simply have a deployment step that builds and copies the code from lib/shared into amplify/backend/functions/<Name of Lambda Layer>/lib/nodejs ready for Amplify to use for deployment.

@mewtlu So you manually remove these local dependencies before you run amplify push/publish?

@mewtlu Thanks for sharing your solution.
Could please share some code examples? with all the things you tweaked.
Some more context of my case:
I use the Appsync graphql layer (const API = require('@aws-amplify/api').default;) inside a lambda function.
That lambda function is used to handle one route of the API Gateway.

I'm not sure if there are any further issues related to the specifics of your situation, but to be able to mock any function that has a Lambda layer all that is required is manually editing your local amplify/backend/amplify-meta.json file to remove the Lambda Layer from the dependsOn array within that function's block in the Resource object.
I run the following bash script using jq whilst in in amplify/backend/ to do this automatically for me: cat amplify-meta.json | jq \". * { function: .function | map_values(if .dependsOn then del(.dependsOn[] | select(.resourceName == \\\"MyLambdaLayerName\\\")) else . end) }\" > amplify-meta.n.json (replacing MyLambdaLayerName with the name of your Lambda Layer)
Hope this helps.

Thanks a lot @mewtlu . I'll give it shot.

@mewtlu how does your solution gets the related code stored in the lambda layer folders?

If you're asking how we provide the code in the layer to the functions within the dev environment I simply use local npm dependencies, within my frontend application's package.json I have an npm dependency on the package within the monorepo which contains the code used for the Lambda Layer such as: "@org/shared": "../lib/shared" and the backend functions have a dependency on the same local package such as: "@org/shared": "../../../../lib/shared"
As the Lambda layer contains TypeScript code/types we also use TS project references to support this which nicely links up in a similar manner using relative paths within the monorepo, then we simply have a deployment step that builds and copies the code from lib/shared into amplify/backend/functions/MyLambdaLayerName/lib/nodejs ready for Amplify to use for deployment.

How do you handle the layer paths within the lambda function? Ex: /opt/folder/file.js ?


Create Contribution.md

Solved issue #230
kindly review my PR. @Quramy

@Quramy  I have corrected it. now, you can review my PR.

I have corrected my mistake. So, you can review my PR
On Sun, 29 Oct, 2023, 9:48 am Yosuke Kurami, @.***>
wrote:

@.**** commented on this pull request.
In Contribution.md
https://github.com/Quramy/typed-css-modules/pull/232#discussion_r1375365958
:

@@ -0,0 +1,75 @@
+# CSS Modules TypeScript Contribution Guidelines

Project name is not “CSS Modules TypeScript” but “typed-css-modules”
—
Reply to this email directly, view it on GitHub
https://github.com/Quramy/typed-css-modules/pull/232#pullrequestreview-1702894283,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXYMUXHNVZZ2N4SLBXLRVZ3YBXKIRAVCNFSM6AAAAAA6UID2XKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMBSHA4TIMRYGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

@Quramy Please response to my PR

@Parth4git Thanks!


Virtual switch step

During setup the document calls to create an external network for internet access. How necessary is this? Being that i specifically create an 'external' network.
I already have a switch that is configured to internal but it grants internet access my linux vm's seem to have no problem connecting using it. And apparently only 1 switch can be connected to a wireless thingy.

@flyinghotpocket - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.

https://www.microsoft.com/en-us/evalcenter/evaluate-lab-kit Windows off and deployment lab zip. it comes with a setup guide docx.

Yup looks like it is completely required, the installer wont proceed without a specific external network defined.


generateStyle is not valid currently

it can generate an array of style functions, but this is not accepted by OpenLayers
ol.style.Style | Array.<ol.style.Style> | ol.style.StyleFunction | undefined

renderUniqueValue creates a style function
renderClassBreaks creates a style function
renderSimple creates an ol.style.Style
convertLabelingInfo creates an array of style functions, this will not work

So I guess the easiest change is to make everything return a style function and then have a wrapper function that executes all of the sub style functions?


Exhibition related content

An extension of the work on exhibition related pages – this pr now allows articles, books, exhibitions, and events to have a child-to-parent relationship with an exhibition.
We (I) should update the content related to each of the 12 exhibitions in Prismic before this is merged.
TODO:

[ ] Flow
[ ] Update Prismic content

Articles

Seasons

I'm going to suggest that we have some pretty poor performance problems with seasons.

Going to close this until we get time to look at it again.


Offsetting core drawer panel

I would like to know if we can offset the core drawer panel when the toggle button is clicked so that the drawer scrolls after the toggle button but still the toggle button should be active to be clicked on again to hide the drawer panel.
Also is it possible to make the core drawer panel scroll from bottom, say an options bar that when toggled will scroll up from bottom.
Thanks

auto-moving this issue to PolymerElements/paper-drawer-panel#25 and closing this one


Manual testing for 1.5.1 fixes

Installation
istioctl

[ ] default profile with --wait, run bookinfo [ostromart]
[ ] --set profile=demo for same install, run bookinfo [ostromart]

controller

[ ] operator install with init/remove command
[ ] operator install with helm charts in operator/data/operator-charts
[ ] operator install with kubectl -f https://istio.io etc.  [ostromart]
[ ] run bookinfo
[ ] profile to demo for same install, run bookinfo

Enablement

[ ] manifest apply with demo, then iterate through disabling, ingress gateway, prometheus, grafana and ensure resources are pruned.
[ ] same as above with controller

API surface
istioctl

[ ] manifest apply k8s field partial override - ensure this is merged with the builtin defaults. both components and addonComponents.
[ ] ensure enum values can be passed in
[ ] k8s overlays of output manifest [ostromart]

controller

[ ] manifest apply k8s field partial override - ensure this is merged with the builtin defaults. both components and addonComponents.
[ ] ensure enum values can be passed in
[ ] k8s overlays of output manifest [ostromart]

Gateways

[ ] Install three gateways in two namespaces, including ILB per example [ostromart]
[ ] verify bugs in https://github.com/istio/istio/issues/22253 [ostromart]
[ ] ensure user gateway overlays work for --set for both values and IOP [ostromart]
[ ] test included gateway config sample  [ostromart]

@elfinhe add to my mentioned list.

cc @istio/wg-environments-maintainers this is a great list for targeting integration testing with Istio 1.6.

Manually test operator install with init/remove command, it is working, but found another issue about field unmarshal, see:  https://github.com/istio/istio/issues/22280


Missing single quote in routers example

$router.get(-> $request, $response { # Register a GET route on the root of the router
    $response.html('\
Hello World\
);
});

I will gladly take a PR to fix this if you'd like :)

Thanks for the reply. Unfortunately I don't do git (will start soon at my workplace).

No worries, I can fix it.

👍


Anything in FeatMediaNode branch

Dear Team,
Any thing progress on Feat Media Note Branch?
Thanks
Alex

Unfortunately not at the moment.


[REQUEST] Document lack of progressive printing on tables

One issue with rich tables is that they render at the end and there is no way to gradually print tables row by row.
I can understand why this happens but for tools that produce long lists and likely ones that take significant time to produce for each row, this becomes a deal breaker.
I suspect that using a custom logger could be used to partially alleviate this limitation, at the cost of having less control over the final result.
From the rendering perspective, if user could define the column sizes when creating a table, it should be possible to perform gradual printing.
If this cannot be implemented due to current architecture, it may worth documenting this limitation and the logging workaround.

One option would be use the LiveRender class to redraw the table each time. But frankly I think this is a rather niche requirement and I suspect there are better ways of presenting your information.
I don't see the worth in documenting something which I doubt would be expected of a terminal rendering of a table.
Closing this as wontfix. But if you have a concrete proposal of how a progressive table would work / look like I'll listen.

I would love to see this feature too. I have a stream of data that is being received which I want to format as a table. Since I have many rows, I would like to have the ability to print new rows as data is being received without the need to re-render the entire table. As it was said above, we will need to set column widths before printing.
A code example might look like:
from rich.console import Console
from rich.table import ProgressiveTable

console = Console()
table = ProgressiveTable()

table.add_column('test column', width=12)

table.add_row('first row')
console.print(table)

table.add_row('second row')
console.print(table)

table.finish_border()
console.print(table)

Whose output should be
┏━━━━━━━━━━━━━━┓
┃ test column  ┃
┡━━━━━━━━━━━━━━┩
│ first row    │
│ second row   │
└──────────────┘

Such a table would probably work by buffering the newly added rows before printing (only) them.

@giladreti Rich current main goal is to provide the building blocks for creating TUI applications. As my primary use case was for making console application output easier to read, I created a small extension called enrich which focuses on progressive and fluid output (soft wrapping, for both console and logging).
Even so this will may not work for tables because they can work only with a fixed layout and rich needs to decide how much space to give to each column and do the wrapping. I think you could emulate it by styling the table to render as a single row and printing each row as an entire table.
To be honest the rendering of UI elements is a very complex issue for which browsers spent decades improving and I doubt @willmcgugan had the superpowers to address it.
For the moment I avoid printing stuff as tables if progressive display is needed or if the total amount of printed rows is likely to be bigger than 1-2 screens.


JDL with multiple applications fails to import

Describe the bug
Generating an app with a JDL containing multiple applications has a few issues.  This is mainly due to the fact the entity JSON files are generated in app folders rather than at the root, which is how a JDL containing a single application works).
Current workaround is to remove all but one of the applications from the JDL.
Instructions To reproduce
Import the following JDL:
application {
   config {
     baseName gway
     applicationType gateway
  }
  entities *
}

application {
   config {
     baseName ms
     applicationType microservice
  }
  entities *
}

entity A

microservice A with ms

I'm going to not implement this.  It would require either specifying which backend to use, or generating multiple Ignite JHipster apps, which doesn't make much sense to me.  Will add a note to the JDL docs.