Event when map has stopped moving

Hello - I would LOVE to have an event fire when the user has stopped moving the map, preferably also with some sort of distance involved, so that I can use the map bounds to load up new markers. Forgive me if I haven't found this in the documentation yet, but I can't find a feature like this?

That feature isn't implemented in the plugin that I'm aware of. We'd have to take a look at the native APIs to see how it might be done.


Title overflows div

Firefox mobile 31, iPhone 8

this issue still exist ?

This issue has been fixed


Is there a way to limit the size of the button when you zoom a page?

The button looks nice when the page is in 100% zoom. But when you start to increase the zoom level, the button becomes really large. Is there a way to limit the size of the button somehow, so it would stay the same no matter what zoom you use?

Did you ever figure out a way to do this?


Refactor publisher

Decouple the publisher from party info by moving validation logic into PartyInfo Service. Remove access to find url by key to only be used in the party info service itself.

Codecov Report

Merging #826 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #826      +/-   ##
============================================
- Coverage     99.49%   99.49%   -0.01%
+ Complexity     2032     2021      -11
============================================
  Files           305      304       -1
  Lines          5999     5970      -29
  Branches        299      298       -1
============================================
- Hits           5969     5940      -29
  Misses            5        5
  Partials         25       25

Impacted Files
Coverage Δ
Complexity Δ

...quorum/tessera/partyinfo/PartyInfoServiceImpl.java
100% <100%> (ø)
15 <7> (-1)
:arrow_down:

...rum/tessera/partyinfo/PartyInfoServiceFactory.java
100% <100%> (ø)
2 <1> (ø)
:arrow_down:

...quorum/tessera/partyinfo/PayloadPublisherImpl.java
100% <100%> (ø)
4 <1> (-1)
:arrow_down:

...um/tessera/transaction/TransactionManagerImpl.java
100% <100%> (ø)
37 <0> (ø)
:arrow_down:

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 d94b76f...d3879de. Read the comment docs.


Add LSC 2578539-970714 v1.1.1

Can you also add support for LSC 2578539-970714 v1.1.1 color light bulb?

We can't blindly add support for a device without a firmware dump.  You can try By firmware version and name for the 1.1.1 oem_bk7231s_light_ty_oldDp, but no guarantee it will work.  If it does work, we would still need storage data to support the device directly.
Re-open this issue if you are able to provide a dump.


fixed service duplicate in service list. - [closed]

In GitLab by @sokril on Jan 8, 2018, 08:17
Merges mysql -> dev

In GitLab by @wknet123 on Nov 13, 2019, 10:24
closed


Customizations via metadata

Nice gem!
Can I propose you a small PR to allow more customizations via RSpec metadata?
Combined with the configuration:
RSpec::OpenAPI.description_builder = -> (example) { example.metadata[:openapi_description] || example.description }
A usage example could be:
  describe 'GET /api/v1/posts', {
    openapi_summary: 'list all posts',
    openapi_description: 'list all posts ordered by pub_date',
    openapi_tags: %w[v1 posts],
  } do
    # ...
  end

If you like the idea, I could update the specs and the README.

Using metadata for the purpose sensible to me. Please do update the specs and update README to document it.
How about overloading openapi: option and nest keys under that though? i.e.
  describe 'GET /api/v1/posts', openapi: {
    summary: 'list all posts',
    description: 'list all posts ordered by pub_date',
    tags: %w[v1 posts],
  } do
    # ...
  end

When the option is taking a Hash, it's clear that it doesn't need to be openapi: false, so I think it doesn't conflict.

How about overloading openapi: option and nesting keys under that though? i.e.

Sounds good to me.
I was thinking the same at the beginning but I didn't want to replace your current behaviour.
About description - do you think that I could also add a customization there? Or it's better to avoid overriding the RSpec::OpenAPI.description_builder configuration method?
So this line: https://github.com/k0kubun/rspec-openapi/blob/master/lib/rspec/openapi/record_builder.rb#L53
Could become:
description: example.metadata.dig(:openapi, :description) || RSpec::OpenAPI.description_builder.call(example),

On second thought, I think that it could be good to let override also the description.
So I proceeded in that way.


Raising and Logging exception

logzero version: 1.3.0
Python version: 3.6
Operating System: Ubuntu

Description
I am using logzero and pytest. When an exception occur, I would like to be able to catch it in the log, but also to test that it was raised in my unittest. Currently, the only way I managed to do this is to catch the exception, log it and raise it again. That doesn't feel like the right way (maybe it is).
Do you have a leaner way to recommend?
What I Did
# in main.py
    if publisher:
        try:
            timezone = config.TIMEZONES[publisher]
        except KeyError as e:
            logger.exception({
                "error": e,
                "message": f"Missing timezone ID for publisher: {publisher}"
            })
            raise KeyError

Thanks for the great package, a lot simpler than anything else I have experienced for logging. #

This is exactly the recommended pattern in Python except that you should just use a bare raise instead of raise KeyError. That way, the original exception is raised instead of a new KeyError instance.

Yes, as @thebigmunch says, this is the standard pattern of logging exceptions. Perhaps it would be possible to add a helper a la with logzero.try():, but this is out of scope at the moment. Although I'm open to further discussion about this topic!

Thanks both! If that's the recommended pattern, happy to stick with it :)


[App] Plotly from seaborn violinplot looks nothing like the original

Describe the bug
I have a graph that I regrettably cannot show in a public forum, but it's a dual-series violin plot from seaborn (on top of matplotlib).
The resultant in wandb not only looks nothing like the original plot, but it's also not tracking the series correctly. I have two series of data, but they are the same color in the wandb interface and there is no indication of which series is which, making the plot almost useless unfortunately.
To Reproduce
Steps to reproduce the behavior:

Create a simple violin plot with the examples in the link above
Log to wandb using something like below:

fig, _ = plt.subplots(figsize=[16, 9])
sns.violinplot(
    data=df, y='value', x=key, hue='variable',
    inner="quartile"
)
trainer.logger.experiment.log({
    "Title": fig,
    "global_step": trainer.global_step
}, commit=False)

See error

Expected behavior
I expect some differences because they're different chart libraries, but the chart produced isn't keeping track of the important data so I can't really use it. I'd like to avoid uploading images for these as (1) Images take longer to upload in high-res, and (2) I'd like to be able to have these vectorized.
Screenshots
Sample seaborn plot of this data:

Wandb version:

Operating System

OS: Linux/Windows
Browser: Brave

Additional context
The legend for this data also appears to have just been rendered as a box of text and doesn't indicate any color information.

Right, I understand that is a workaround, but as I mentioned

I'd like to avoid uploading images for these as (1) Images take longer to upload in high-res, and (2) I'd like to be able to have these vectorized.

The data in these charts is a bit unwieldy and is more meaningful if it can be zoomed/sliced as one can with plotly charts. Unfortunately, uploading images only partially solves my issue.


Module not found: Can't resolve '@sitecore-cloudsdk/personalize/server'

Describe the Bug
After upgrading from 21.1.6 to 21.6.0 on a non xm-cloud Next.js application, we are receiving an error trying to load the rendering host stating that the '@sitecore-cloudsdk/personalize/server' module was not found:

package-lock.json
    "node_modules/@sitecore-jss/sitecore-jss-nextjs": {
      "version": "21.6.0",
      "resolved"<EMAIL_ADDRESS>      "integrity": "sha512-movSzJTtu4TngDTvOlch4+jYomo8l4qzXnbjZkqx8fPQ1IVA0vkPrG8KGs1oIHeQML3zUDAsXkS5W4opGyux6w==",
      "dependencies": {
        "@sitecore-jss/sitecore-jss": "21.6.0",
        "@sitecore-jss/sitecore-jss-dev-tools": "21.6.0",
        "@sitecore-jss/sitecore-jss-react": "21.6.0",
        "@vercel/kv": "^0.2.1",
        "node-html-parser": "^6.1.4",
        "prop-types": "^15.8.1",
        "regex-parser": "^2.2.11",
        "sync-disk-cache": "^2.1.0"
      },
      "engines": {
        "node": ">=18"
      },
      "peerDependencies": {
        "@sitecore-cloudsdk/events": "^0.1.3",
        "@sitecore-cloudsdk/personalize": "^0.1.3",
        "next": "^13.4.16",
        "react": "^18.2.0",
        "react-dom": "^18.2.0"
      }
    },

Maybe it has to do with adding some sitecore package dependencies that are personalize related in my package.json?
package.json
  "dependencies": {
    "@sitecore-feaas/clientside": "^0.5.5",
    "@sitecore-jss/sitecore-jss-nextjs": "~21.6.0",
    "@sitecore/components": "~1.0.19",
      ...
   },
  "devDependencies": {
    "@sitecore-jss/sitecore-jss-cli": "~21.6.0",
    "@sitecore-jss/sitecore-jss-dev-tools": "~21.6.0",
      ...
   },

To Reproduce

Upgrade from 21.1.6 to 21.6.0 (add in/update/remove all the file updates) via npx<EMAIL_ADDRESS>and set the templates to Next.js and SXA
Run local docker next.js template

Expected Behavior
Expected behavior would be no rendering host errors regarding the personalize middleware for a non xm-cloud application.
Possible Fix
No response
Provide environment information

Sitecore Version: XM 10.3 (Azure PaaS)
JSS Version: 21.6.0

@bewards Hey, using a fresh "nextjs" + "nextjs-sxa" do you face the same problem?
I verified that it works without any errors. Is it a "potentially" problem related to your upgrade process?
If you can provide more information it could be helpful. Are you using any personalize related functionality?

Getting the same error with fresh install. I'm not using docker.
{
  "name": "local-nextjs",
  "description": "Application utilizing Sitecore JavaScript Services and Next.js",
  "version": "21.6.3",
  "private": true,
  "config": {
    "appName": "hackathon",
    "rootPlaceholders": [
      "jss-main"
    ],
    "sitecoreConfigPath": "/App_Config/Include/zzz",
    "graphQLEndpointPath": "/sitecore/api/graph/edge",
    "language": "en",
    "templates": [
      "nextjs",
      "nextjs-sxa"
    ]
  },
  "author": {
    "name": "Sitecore Corporation",
    "url": "https://jss.sitecore.com"
  },
  "repository": {
    "type": "git",
    "url": "git+https://github.com/sitecore/jss.git"
  },
  "bugs": {
    "url": "https://github.com/sitecore/jss/issues"
  },
  "license": "Apache-2.0",
  "dependencies": {
    "@sitecore-feaas/clientside": "^0.5.5",
    "@sitecore-jss/sitecore-jss-nextjs": "~21.6.3",
    "@sitecore/components": "~1.0.19",
    "bootstrap": "^5.1.3",
    "font-awesome": "^4.7.0",
    "graphql": "~15.8.0",
    "graphql-tag": "^2.12.6",
    "next": "^13.4.16",
    "next-localization": "^0.12.0",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "sass": "^1.52.3",
    "sass-alias": "^1.0.5"
  },
  "devDependencies": {
    "@graphql-codegen/cli": "^1.21.8",
    "@graphql-codegen/import-types-preset": "^2.2.6",
    "@graphql-codegen/plugin-helpers": "^3.1.2",
    "@graphql-codegen/typed-document-node": "^2.3.12",
    "@graphql-codegen/typescript": "^2.8.7",
    "@graphql-codegen/typescript-operations": "^2.5.12",
    "@graphql-codegen/typescript-resolvers": "^2.7.12",
    "@graphql-typed-document-node/core": "^3.1.1",
    "@sitecore-jss/sitecore-jss-cli": "~21.6.3",
    "@sitecore-jss/sitecore-jss-dev-tools": "~21.6.3",
    "@types/node": "^18.11.18",
    "@types/react": "^18.2.22",
    "@types/react-dom": "^18.0.5",
    "@typescript-eslint/eslint-plugin": "^5.49.0",
    "@typescript-eslint/parser": "^5.49.0",
    "chalk": "~4.1.2",
    "chokidar": "~3.5.3",
    "constant-case": "^3.0.4",
    "cross-env": "~7.0.3",
    "dotenv": "^16.0.3",
    "eslint": "^8.32.0",
    "eslint-config-next": "^13.1.5",
    "eslint-config-prettier": "^8.6.0",
    "eslint-plugin-prettier": "^4.2.1",
    "eslint-plugin-react": "^7.32.1",
    "eslint-plugin-yaml": "^0.5.0",
    "graphql-let": "^0.18.6",
    "npm-run-all": "~4.1.5",
    "prettier": "^2.8.3",
    "ts-node": "^10.9.1",
    "tsconfig-paths": "^4.1.2",
    "typescript": "~4.9.4",
    "yaml-loader": "^0.8.0"
  },
  "scripts": {
    "bootstrap": "ts-node --project tsconfig.scripts.json scripts/bootstrap.ts",
    "build": "npm-run-all --serial bootstrap next:build",
    "graphql:update": "ts-node --project tsconfig.scripts.json ./scripts/fetch-graphql-introspection-data.ts",
    "install-pre-push-hook": "ts-node --project tsconfig.scripts.json ./scripts/install-pre-push-hook.ts",
    "jss": "jss",
    "lint": "eslint ./src/**/*.tsx ./src/**/*.ts ./scripts/**/*.ts",
    "next:build": "next build",
    "next:dev": "cross-env NODE_OPTIONS='--inspect' next dev",
    "next:start": "next start",
    "scaffold": "ts-node --project tsconfig.scripts.json scripts/scaffold-component/index.ts",
    "start:connected": "npm-run-all --serial bootstrap --parallel next:dev start:watch-components",
    "start:production": "npm-run-all --serial bootstrap next:build next:start",
    "start:watch-components": "ts-node --project tsconfig.scripts.json scripts/generate-component-builder/index.ts --watch"
  }
}

@bewards, @ashishch-98 The required node version is >= 18. Which version do you use? Does it meet our rules?
May be it's a main cause of this issue?

Current node version within Docker is 18.3.0 (I updated it from previously using v16).

@bewards I see. Potentially, it was your problem. But in any case our team will put some effort to explore this problem

Thank you for providing the details. Unfortunately, the steps suggested by @bewards didn't work in my case. My Node version is v20.10.0, and my npm version is 10.2.3.
Upon inspecting my package-lock.json, I discovered that @sitecore-cloudsdk/personalize is listed as a peerDependency of @sitecore-jss/sitecore-jss-nextjs. However, for reasons still unknown to me, npm install failed to automatically install it, nor did it provide any errors or warnings during installation, as expected for npm >v7. I didn't have the legacy-peer-deps=true flag set in the code, as this is a fresh vanilla Sitecore JSS Nextjs setup.
Installing @sitecore-cloudsdk/personalize as a dev dependency resolved the issue for me.

@ashishch-98 Thank you for the details! We will resolve/guide how to resolve this issue properly

@ashishch-98 Thank you for the details! We will resolve/guide how to resolve this issue properly

@illiakovalenko any updates?
Facing the same, when updated to "next": "^14.1.3"
"@sitecore-jss/sitecore-jss-nextjs": "~21.6.3",

This issue is being caused by the legacy-peer-deps flag being set to true in one of the following .npmrc files. Make sure to set it to false.

per-project configuration file (/path/to/my/project/.npmrc)
npm's built-in configuration file (/path/to/npm/npmrc)
per-user configuration file (defaults to $HOME/.npmrc; configurable via CLI option --userconfig or environment variable $NPM_CONFIG_USERCONFIG)
global configuration file (defaults to $PREFIX/etc/npmrc; configurable via CLI option --globalconfig or environment variable $NPM_CONFIG_GLOBALCONFIG)


added link for check out button

Linked the check it out button to the editor page
Fixes #190

@smaranjitghose Please review and merge.


AttributesProxy

A module that behaves like Literal::Attributes, but proxies all the setting and getting to super. This could be used on ActiveRecord models, for example.

Yeah, that's what this is for.


Interfacing with golang and CPP.

It's a pity there is no support for using protobuf in order to interoperate with other languages.
Method channel looks very bespoke
Can the team think about this use case..
I write tons of logic in golang because I use it on all devices. I want to have two way calling between the flutter GUI and my golang. I don't want to use flutter for anything but display.
What's the options today ?

Method channels.
Could not find a single example with golang.

Protobuf
Gomatcha uses this to achieve the same need very well.

Is it because no one else wants this or that flutter is not ready for this in its roadmap yet ?

+1

run golang inside Flutter:
https://github.com/adieu/flutter_go

this one is awesome because it just uses rpc and so its generic. So you dont have to write any swift or java !

https://github.com/renatoathaydes/go-hash
https://github.com/renatoathaydes/gohash_mobile_app
https://github.com/renatoathaydes/gohash_mobile

this one shows a DB and how to do it in detail.
great long tutorial about how to get everything working: https://sites.google.com/a/athaydes.com/renato-athaydes/posts/buildingamobilefrontendforagoapplicationusingflutter

I don't think json-rpc is the right approach...

@charafau
Can you explain your own reasons ?
My own view:

Its a hack to get stuff done quickly in terms fo development.
I was planning to change over to protobufs for perf reasons and easy codegen

BTW FIDL cant be done at Flutter level because it relied on kernel code from zircon in Fuchsia.

@zoechi
Yeah its similar to #7053. Maybe i should close it ?

@gedw99 I guess so. I eouldn't expect something specific to Go.


Publish SNS event on new alert

Pull Request Template

Fixes Issue

Fixes #198
Changes proposed

Screenshots (Optional)
Note to reviewers

Review Error for epociask @ 2024-02-09 22:12:01 UTC
User failed mfa authentication, public email is not set on your github profile. see go/mfa-help


simulations fail to finish for cocotb python syntax errors and rtl forever loops

If there is a syntax error in any of the core cocotb code or the cocotb testbench code, and you have loops in your code that run forever, then ius simulations will run forever.
NB: We have models in our designs that kick off loops that run forever.
e.g a loop that checks for incorrect powerup sequences on a memory model.
For this combination of cocotb/python syntax errors and loop forever rtl, a cocotb simulation will run forever.
I have a branch here https://github.com/luked80/cocotb/tree/runaway_sim with the combination of errors.
Just do:
cd tests/test_cases/test_cocotb
make SIM=ius

This bug only exists on ius.
icarus, questa and vcs error out as expected.
I think the solution here is to have cocotb call vpi_control(vpiFinish, 1); or equivalents when it fails to load.

Hmmm...the code already calls vpi_control(vpiFinish) when the cocotb python code fails to load and ends up calling gpi_sim_end
I'm guessing since this all happens at the very start of the simulation, there could be some issue with calling vpi_control in a cbStartofSimulation callback.
And from my testing that seems to be the case, at least on ius.
I have a fix for this issue here https://github.com/luked80/cocotb/commit/cd8308f537114cfc9122416d568890eb76f7cf84
And have tested that it works on ius.
Seems that calling vpi_control in a time 0 cbAfterDelay callback works fine.
But doing the same thing in a cbStartOfSimulation callback, doesn't.
Probably need to test this out further on the other simulators but https://github.com/luked80/cocotb/commit/cd8308f537114cfc9122416d568890eb76f7cf84 fixes this issue for me on ius.

This issue only affects ius.
icarus, questa and vcs are not affected.
I don't have access to the aldec simulator.
I've updated the description to reflect this.

Added an ius only fix for issue and pull request.


Onvif discovery Handler incorrectly filter ip address

Describe the bug
A clear and concise description of what the bug is.
set filter ip address to exclude <IP_ADDRESS> will end up filtering all ip addresses start with <IP_ADDRESS>, including <IP_ADDRESS>xx.
To Reproduce
Steps to reproduce the behavior:

Create cluster
Install Akri with the Helm command and deploy the Akri Configuration yaml modified from test/yaml/akri-onvif-video-configuration.yaml (see below).  In the configuration, only exclude cameras with ip address <IP_ADDRESS>, <IP_ADDRESS>.
My Onvif camera ip is <IP_ADDRESS>, after deploying the Configuration, my camera is not detected.

apiVersion: akri.sh/v0
kind: Configuration
metadata:
name: akri-onvif-video
spec:
discoveryHandler:
name: onvif
discoveryDetails: |+
ipAddresses:
action: Exclude
items:
- <IP_ADDRESS>
- <IP_ADDRESS>
macAddresses:
action: Exclude
items: []
scopes:
action: Exclude
items: []
discoveryTimeoutSeconds: 2
brokerSpec:
brokerPodSpec:
containers:
- name: akri-onvif-video-broker
image: "ghcr.io/project-akri/akri/onvif-video-broker:latest-dev"
imagePullPolicy: Always
resources:
limits:
"{{PLACEHOLDER}}" : "1"
instanceServiceSpec:
ports:
- name: grpc
port: 80
targetPort: 8083
configurationServiceSpec:
ports:
- name: grpc
port: 80
targetPort: 8083
brokerProperties: {}
Expected behavior
Onvif camera ip address <IP_ADDRESS> discovered.
Onvif discovery handler should only filter out <IP_ADDRESS> and <IP_ADDRESS>, not others.

The fix causes wrong behavior when filtering scopes.

verify the issue is fixed.


New precommit hooks

Description
Update pre-commit versions and configs (in two commits to simplify merging other branches).
Main changes:

update hooks versions
remove encoding pragma (disabled)
add license headers (disabled)
autoformat ini files
autoformat yaml files
extra check for merge conflicts
extra check for debug statements
extra check for unnecesary #noqa pragmas (applied once but disabled)

bors try

bors try

bors try

bors try

bors try

try
Build failed:

ci/gitlab/full-pipeline

It looks like this was due to a temporary unavailability of the jfrog docker registry.


Bootstrapping the Controller Plane api-server, controller-manager, and scheduler exit-code-1 and endless restarting

Heyo. During this guide I have run into a roadblock that is destroying my sanity. I have tried this guide with no modifications and with the following modifications and have come across this problem in both cases. I have localized the issue to the following components and tried to understand the details as to why this may be occurring but cannot understand.
Machine System:  WSL2 backend Unbuntu partition on Windows OS (which also is utilized by my docker desktop ... I am assuming these are independent if I were to utilize the Unbuntu partition as an isolated environment that won't interfer with any docker needs) and GCloud.
The code ( which I modified slightly to use 1.24.3 kubernetes and updated versions of etcd, controller-manager, scheduler, api-server ) and results are as follows. The same thing happened when I tried the same with the guides original packages.
Utilized Code ---

And results below
sudo systemctl status kube-apiserver.service kube-controller-manager.service kube-scheduler.service etcd.service

Spent about a week on a few days on this and losing my mind. Any help would be appreciated, guys. Even just forwarding me off someplace to look for details.
I know I have an issue with trying to run the verifications and what-not since the replies I get amount to this error
asg4@controller-0:~$ kubectl cluster-info --kubeconfig admin.kubeconfig

To further debug and diagnose cluster problems, use 'kubectl cluster-info dump'.
The connection to the server <IP_ADDRESS>:6443 was refused - did you specify the right host or port?

or
asg4@controller-0:~$ curl -H "Host: kubernetes.default.svc.cluster.local" -i http://<IP_ADDRESS>/healthz

HTTP/1.1 502 Bad Gateway
Server: nginx/1.18.0 (Ubuntu)
Date: Fri, 29 Jul 2022 04:16:39 GMT
Content-Type: text/html
Content-Length: 166
Connection: keep-alive

<title>502 Bad Gateway</title>

502 Bad Gateway

nginx/1.18.0 (Ubuntu)

For me, the log indicates an issue with the encryption key. so I recreated a key and regenerate the encryption-config.yaml file. Once it was copied to /var/lib/kubernetes/ the kube-apiserver was able to start successfully.


Pre-training HuBERT - AssertionError: number of labels does not match

Can anyone help me to fix this one please?
I tried to pre-train the HuBERT on another dataset. When I run the following command:
HYDRA_FULL_ERROR=1 python fairseq_cli/hydra_train.py
--config-dir examples/hubert/config/pretrain
--config-name hubert_base_librispeech
task.data=/tmp/word-discovery/path/to/CommonVoice/ja/manifest/ task.label_dir=/tmp/word-discovery/path/to/CommonVoice/ja/lab task.labels='["km"]' model.label_rate=100
Then, I got this kind of error:

File "/tmp/fairseq/fairseq/data/audio/hubert_dataset.py", line 68, in load_label_offset
assert (
AssertionError: number of labels does not match (3008 != 12034)
I'm still not sure what is that. Anyone, please help me :(

Hi hi I ran into this Error too. Have you solved it in the end?


Prioritize global hooks above modules

Hi there.
Thank you for the cool stuff!
Let me describe the case to make this proposal clear.
I have a global hook self-update.sh (with schedule task)  which every N minutes checks a new addon-operator image and pull it to a cluster. This move brings new modules / patches / etc to a cluster.
Then, I have made some broken module my-module-X, which can not be deployed due to some reasons (just personal mistake, env troubles, etc). And it has stucked in the task queue, trying to be released every 5 seconds.
After I have found an error into the module and have fix it, I pull the new image with addon-operator. So, it seems pretty easy: currently working addon-operator pod should find the new image in a few minutes and redeploy it but it`s not possible, because the tasks queue stucked in the next state:
Queue length 296

TASK_MODULE_RUN 'my-module-X' failed 579 times.
# other modules #
TASK_GLOBAL_HOOK_RUN 'self-update.sh'
TASK_GLOBAL_HOOK_RUN 'self-update.sh'
TASK_GLOBAL_HOOK_RUN 'self-update.sh'
TASK_GLOBAL_HOOK_RUN 'self-update.sh'
TASK_GLOBAL_HOOK_RUN 'self-update.sh'
TASK_GLOBAL_HOOK_RUN 'self-update.sh'
...

The solution here is to redeploy addon-operator pod in all clusters. But is brokes all the automation stuff.
That would be nice if global hooks (or maybe only schedule tasks) come at the top of the queue. It will make self-repairing based on schedule/event possible.
Thank you.

Hello. I can recommend a workaround for blocked hook: use a sidecar container with shell-operator and self-update.sh.
P.S. We have the same pull model with out clusters. Our solution for updates is a golang code that checks for image digest in status section of pod and fetch metadata from registry and restarts a Deployment if they are not matched. There is a plan to release it in a form of sidecar.

Yeah. Like a workaround solution I also made a sidecar with crond and script. Sure it works but addon-operator has all required tools to be self-sufficient. And it looks quite reasonable to execute global hooks in a separate goroutine.At the same time, container with config reloader looks more k8s way. It depends...
Anyway, its up too you, I didnt investigate the code well, maybe this change could break something.
Thank you for the answer. I think this issue could be closed if you regard this kind of changes unnecessary or dangerous.

Global hooks are considered to be a part of addons, that is why they are queued to the same working queue with modules. For now, one working queue — one addon-operator instance. And there is no plans to implement multiple working queues. So «Prioritize global hooks above modules» is a no-go for now.
May be there is a good idea to implement a special kind of hooks with separate working queue. Something like maintenance hooks or service hooks.


Add Cloudflare Pages headers file

PoC - disable the default behaviour of adding access-control-allow-origin: * to pages sites
Enable ACAO only for static /images/
Available at:
https://cf-pages-headers.matrix-to-asy.pages.dev/
The deployment mechanism probably needs looking at (writing into the build.js file).
Reference:
https://github.com/vector-im/sre-internal/issues/2606

@bwindels can we merge this?

@laurencegill this seems like it'll break Netlify deploys? Surely it shouldn't be done by the build script unless its happening for CFP?

Not sure why this repo uses Netlify & CFP for PR previews, likely Netlify can just be disabled a this point

Not sure why this repo uses Netlify & CFP for PR previews, likely Netlify can just be disabled a this point

I'm not sure if  Netlify is still used or not, if not we can disable it


misspell: Correct typos

Correct typos listed by misspell.
As seen on https://goreportcard.com/report/github.com/coreos/rkt#misspell

Thanks for cleaning up! Unfortunately this is changing some files in vendored libs (under  vendor/github.com/shirou/gopsutil/) which we shouldn't touch here. Can you please submit those two fixes directly to gopsutils? Rest of this looks ok, can you just split those fixes out of here and retitle the commit (rkt: is a better prefix for this kind of generic fixes)?

LGTM


Fix doc comments for NodeJS.

This patch contains the following updates:

fix on the parameter types -- now they are Object, with the
link text to the message definition.
add NODEJS_DOC generator type and generate message definitions
in that case.

Fixes #410

Current coverage is 81.83% (diff: 94.93%)

Merging #427 into master will increase coverage by 0.33%

@@             master       #427   diff @@
==========================================
  Files           193        193
  Lines          7140       7196    +56
  Methods           0          0
  Messages          0          0
  Branches        933        948    +15
==========================================
+ Hits           5819       5889    +70
+ Misses         1043       1030    -13
+ Partials        278        277     -1

Powered by Codecov. Last update 8324184...5c9c7af

LGTM if build passes
Note: I reviewed only Java code and the templates, not the actual JS output -- presumably if needed it can be refined in a later PR.


npm severity vulnerability

Bug Details
Describe the bug
A high severity vulnerability has been identified in the codebase
1 high severity vulnerability
To address all issues, run:
npm audit fix
Run npm audit for details.
Steps To Reproduce

Access the code space
Run make deploy
Following warning has been highlighted when we run make deploy

What is the expected behavior?
Screenshots

Additional context
Add any other context about the problem here.
If the bug is confirmed, would you be willing to submit a PR?

X] Yes
[ ] No

Not replicable
Ensure you have a recent version of NPM and/or run "npm audit" and provide those details.


How to use/configure on production.

Please can you tell me how to use/configure this gem on production.

Which part is unclear?

as of development i am runnig from terninal as script/mailman_server. how abt in production.

you'll need to start it up as a service and have something monitor it, like runit or god, etc.  that's outside the scope of this project.


Add docker command completion

On RancherOS 0.7.1 command completion for docker is not working, this would be a nice add on.

Duplicate of #1345


Add PGD to Docker image

Motivation
It would be nice to be able to play with bleeding edge features when using docker, but currently this is not possible. For example, PGD config is missing from docker image, because code in master is cloned.
Modification
Instead of cloning the master branch in Dockerfile, now we just copy the repository contents. In v1.0.0-dev0 branch, we can see PGA in Docker container, because contents of the current branch are copied. Sine copying cannot be done outside docker build context, the build command had to be modified a bit by adding the -f option as shown in the .md-file.
BC-breaking (Optional)
I don't see this as a breaking change. If there are CI builds that try to build the docker image in this repo, then the build commands in CI-yml files may need update. (But only if v1.0.0-dev0 gets merged to master).
Use cases (Optional)

Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

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

Hi @Otteri！First of all, we want to express our gratitude for your significant PR in the MMDetection3D project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤ @Otteri


[Doc] Update gh-pages branch for 23.04.1 patch release [skip ci]

Signed-off-by: liyuan<EMAIL_ADDRESS>Merge changes from the main branch to gh-pages branch for v23.04.1 patch release.
View pages: https://nvliyuan.github.io/yuali-spark-rapids/docs/download.html
This pr will not be merged until the v23.04.1 patch release is completed.

Converted to draft so we do not accidentally merge until release testing and the release is complete.

since v2304.1 is released to maven central, just merge this pr


merge from arlac77/npm-package-template

package.json

chore(package<EMAIL_ADDRESS>chore(package): add acorn-class-fields@https://github.com/arlac77/acorn-class-fields.git
chore(scripts): cover@#overwrite c8 -x 'tests/**/*' --temp-directory build/tmp ava && c8 report -r lcov -o build/coverage --temp-directory build/tmp
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/*-test.mjs' as in template
chore(package): set $.ava.extensions='mjs' as in template

rollup.config.js

chore(rollup): import acornClassFields

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added email functionality

Now user can receive his/her certificate on emails

[ ]  Moved the project to flask for scalability and smooth integration.
[ ] Used smtplib to sent the emails.
[ ] Added a Responsive user interface to the functionality
[ ] Added requirement txt

This PR was made as a part of the participation from hacktoberfest, So if you find it useful you can help me out by making this pr count by the following rules :
A pull request is considered approved once it has an overall approving review from maintainers, or has been merged by maintainers, or has been given the 'hacktoberfest-accepted' label.
Thanks,
It was a fun project

It was checked, and it is working all you need to do just fill in the credentials for a Gmail account

@Ayush0922 kindly merge this under the required labels

Will merge it after review by EOD today

@Ayush0922 kindly merge this under the required labels

yeah still under review..will merge today if passes the needs


Test player communication timeouts.

We want to give players 5 seconds to choose a move before killing them. This sort of thing needs checking - ideally some tests, to protect against regressions in future?

Believed implemented but a test needed.


Update README.md

Spellchecking :beers: :grimacing:

Hi @jeffreyjackson
Thanks for doing a quick review and catching some of my mistakes!  I made one comment on a line that has a few other errors that I noticed.  Would you mind doing a quick patch on that one and I'll merge it in.
Thanks for the contributions!
Best,
Logan

All set :beers:


Add documentation (and screen snippet) of new "Go to page" feature

We forgot to close an old issue and update the documentation for the new "Go to page / Jump to page" feature kindly done by @antoine2711
Current Results
There is no documentation for the new "Go to page - Page number" input box feature in the "Grid" section of docs - https://docs.openrefine.org/manual/running#the-grid
Expected Behavior
Have a paragraph that describes:

Changing numbers to go to a page
Clicking in number input box and then using arrow keys UP/DOWN on keyboard can also advance/reverse page numbers
Include a small screen snippet of the grid header area or better yet, the entire grid (div#right-panel in OpenRefine)

Screenshots

Versions

Operating System: all
Browser Version: all
JRE or JDK Version: n/a
OpenRefine: 3.5+

Datasets

Additional context
As designed, discussed, tested in Issue #2638 and PR #2639

It is not really clear to me how much documentation this actually requires - the UI seems fairly intuitive to me, no? I am not opposed to adding one or two sentences about it but I think we should not over-do it either.

@thadguidry sir, I would like to contribute to this project. I understand that we are trying to document https://github.com/OpenRefine/OpenRefine/issues/2638 this feature in the documentation.

Thanks for volunteering!

I was going through the repo. I think I have to follow https://github.com/OpenRefine/OpenRefine/tree/master/docs these instructions in order to start building the docs. Any advice?

@wetneb I have tried to implement the above in #4469. please check.


Change the result "no-photo"'s image to material design icon

Edit

to

Please check it again~ thank you ~


Add live suggestions

Auto complete feature

Hi, Can you assign me for this issue I would like to work on this issue?

Hi @vishal206 i have assigned the issue to you. Good luck :)

In this issue, by the live suggestion they mean when the user types city name it should give some list of suggestions ri8!

yes, we can autocomplete the name of the city entered by the user.

yes, we can autocomplete the name of the city entered by the user.
Thank you.

Where should i check which one is assigned and which one is not to the participant?

Whether this issue is assigned and solved?

There will be an assigned notification in the comments section. You can also refer to the assignes column.

@Bhavik0902 If an issue is closed, it means that it has been solved already.

Hey, I figured out who to have a live suggestion dropdown and collected cities data for it.
I'm facing some problem with connecting the CSV file, when it's done I will give a PR @Archita01

Cool, no problem @vishal206

The previous PR was reverted as build Failed. @vishal206 pls work on the fix and make a new PR.
@Ishan-001 pls check

sure bro I will work on it. @jazxii

I have made a new PR

The previous PR was reverted as build Failed. @vishal206 pls work on the fix and make a new PR. @Ishan-001 pls check


Handle testcloud problem with the images directory

Should be merged only after the testcloud change is released and the new exception is available.

@psss added a blocked label if you do not mind, to prevent accidental merge

@psss  a test would be nice :)

@psss a test would be nice :)

I'm not sure if it's worth for this particular case. Maybe if there's some extra free time ;-)

Going to merge as testcloud-0.3.6-1 which has the new exception implemented is in stable repos.


GHES HA Offering Improvements

Summary
Most of the GHES customers are set up in HA mode, and many of them (especially larger customers) have geo replication and multiple HA replicas in their configuration. However, we have seen many issues with setup / configure / teardown / failover with multiple replicas. For example: GHES HA failover not working when the primary is unavailable. This impacts the customer experience and deteriorates the perceived reliability of the system.
Intended Outcome
The efforts we have planned this quarter will make HA configuration scripts much more reliable, we will improve error handling, and handle failover scenarios more gracefully.
How will it work?

Improve connection checks and reliability of ghe-repl-* commands
Better error handling for offline replicas
Reliable teardown if primary or any of the replicas are not accessible
Prevent ghe-config-apply from running if replica is not fully setup

🚢 This has shipped with GHES 3.9<EMAIL_ADDRESS>Closing as complete.


Old bossbar messages being displayed

I'm using BossBarAPI, and my view-distance is set to 6.
It seems that when I move around in areas where I punched some mobs and leveled up unarmed, these messages remain there when I return to the area. Consequently, the bossbar message does not get removed. I am using the default config - with alwaysShow set to false.
I believe it happens when you move around an area attacking mobs - my guess is that the old messages are never removed when the message is updated (perhaps with a new bossbar entity). Only the "latest" messages get removed, but older ones that have not expired before a new "update" was sent are not.

I'll look into it.


[Bug]: Secrets with spaces do not work

Is there an existing issue for this?

[X] I have searched the existing issues

Example public repository
coolify
Description
When I create a secret with a value containing spaces, the secret is not registered.
Steps To Reproduce

Create a secret, the name doesn't matter
Put a value with spaces, in my case "462780441594822687<PHONE_NUMBER>11479808"
Deploy the application
Try to get the Secret

Version
3.12.17

Nvm

xD, I though it was something else, but yea, it's a bug

The same problem with newlines - e.g. a public key like the one below is not usable in Coolify:
-----BEGIN PUBLIC KEY-----
abcd
efgh
-----END PUBLIC KEY-----

I can confirm I'm experiencing the same issue, and currently my workaround is to base64 encode the value and decode it within the app. Though that isn't always possible with 3rd party docker images/apps.

@andrasbacsai was this resolved elsewhere? I see the issue was unassigned so I'm just curious. Thanks.

I'm closing this issue, because tracking features will be moved to GitHub discussions. Github issues will be only for bugs.
Version 4 is in full focus. v3 related issues will be closed, if it is not a critical bug.
Thanks for your understanding.


Social Media meta tags

@danielmconrad @pbinggeser Can you review this and approve it?

Is there ever a case when facebook and twitter titles would differ from the standard title? What about description? It would really simplify things if we only had to worry about title/description once per page.

@danielmconrad I agree, but we'll still need the meta tags for both services. Maybe we create a snippet that will do all three (html, fb, and twitter) by us sending it one version?


which_selected function

Hi,
I would like to fill FR for which_selected function to return indices of lists and nested lists. The best if it would return the integer vector of indices which can be used in a way mylist[[indices]] to access selected element in any list of the same structure.
I would prefer to have two lists, one the labels list which is visible on the UI, other one the data list (various data types including language objects) the mirror structure of labels list. Using which_selected on the labels list I could easily access the element from the mirror data list.
Or maybe there is a simple trick to achieve it now?

Hi, I'm not sure I understand the request. Can you provide an example and/or some code samples that describe the format of the data you have now and what you're looking for? Thanks.

Sure.
Consider following tree
  output$tree <- renderTree({
    mylist <- list(
      root1 = structure("123"),
      root2 = list(
        SubListA = list(leaf1 = "", leaf2 = "", leaf3=""),
        SubListB = list(leafA = "", leafB = "")
      )
    )
  })

The which_selected function would return:

nothing selected: NA_integer_
selected root1: 1L
selected root2-SubListA-leaf3: c(2L,1L,3L)
selected root2-SubListB: c(2L,2L)
selected root2-SubListB-leafA: c(2L,2L,1L)

Having such integer vector output I could subset mylist[[int_vec]] or any other lists with exact structure.
In my use case I have a second technical lists of the same structure which contains calls and symbols. Those will be execute on selection. Currently I've manage to do this using get_selected but I had to prefix list names with unique numbers to catch those numbers and reuse on subset technical list. So currently my shinyTree names are cluttered by unique id.

#21 closes this issue.


downloadable font: download failed, cross-site access not allowed

After adding all CDN links font is not loading in Firefox:

downloadable font: download failed (font-family: "FontAwesome" style:normal weight:normal stretch:normal src index:2): bad URI or cross-site access not allowed source: https://api.tiles.mapbox.com/mapbox.js/plugins/leaflet-locatecontrol/v0.43.0/fonts/fontawesome-webfont.woff?v=4.3.0

Reproducible on the example:
https://www.mapbox.com/mapbox.js/example/v1.0.0/leaflet-locatecontrol/

https://github.com/FortAwesome/Font-Awesome/issues/6651

Also, loading almost 1Mb font just for one icon looks like a bad choice to me.

I don't see the problem in firefox with the demo from http://domoritz.de/leaflet-locatecontrol/demo/. Can you please try it. For mapbox, try http://domoritz.de/leaflet-locatecontrol/demo_mapbox/. For the example that you showed me, I see that there is a cross origin policy issue on their side. It's not really a problem with my library.
Concerning the size, I don't see the issue. The web font is 50 KB and not 1 MB. I agree that 1 MB would be bad but it's not the case. I'm closing the issue for now because I cannot reproduce it and it's not an issue that I can fix. Please report the issue at https://github.com/mapbox/mapbox.js-plugins/issues/new

This issue is not specific to Firefox. It can be observed in Chrome.
To reproduce:

launch Chrome
in Chrome, open JS console
go to https://www.mapbox.com/mapbox.js/example/v1.0.0/leaflet-locatecontrol/
observe that:
a. no icon displayed in leaflet locate control
b. console reports cross-site problem.

Note: the sample code from mapbox is as follows:
<!DOCTYPE html>

<meta charset=utf-8 />
<title>Leaflet Locate</title>
<meta name='viewport' content='initial-scale=1,maximum-scale=1,user-scalable=no' />

<link href='https://api.tiles.mapbox.com/mapbox.js/v2.2.0/mapbox.css' rel='stylesheet' />

<link href='https://api.tiles.mapbox.com/mapbox.js/plugins/leaflet-locatecontrol/v0.43.0/L.Control.Locate.mapbox.css' rel='stylesheet' />

<link href='https://api.tiles.mapbox.com/mapbox.js/plugins/leaflet-locatecontrol/v0.43.0/css/font-awesome.min.css' rel='stylesheet' />

Can you report this problem at https://github.com/mapbox/mapbox.js-plugins/issues/new?

Reported.


[stable/suitecr] Use storageClassName for suitecrm.

Fixed suitecrm for #1869
/cc @prydonius

/ok-to-test

/lgtm


RIP gorpc tablet conn. We have a working gRPC version.

@sougou @michael-berlin @enisoc

LGTM

LGTM!

@aaijazi @michael-berlin worker.py is busted here, but I don't think that has anything to do with anything I did. It also fails on my workstation, with a different error it seems.
We've been burned by this guy flakiness for a long time now, is there really nothing that can make this better?

I'm taking a look at this, my first guess is that something really did change. I'm seeing a couple of failures due to "context deadline exceeded". Maybe not this PR, but a previous one changes the expected behavior in some way. Both the failures are in test_reparent_during_worker_copy (the test that runs second), not in test_mysql_down_during_worker_copy.
The third failure is due to something even stranger: "E1109 23:02:21.157272   47173 vtworker.go:81] init() failed: the specified shard test_keyspace/0 is not in any overlapping shard". I'm inclined to dismiss this as a previous teardown not quite cleaning up correctly, so maybe not worth investigating.
@michael-berlin feel free to take this over if you'd like during the week, as I'm not going to get too many chances to work on it.

@aaijazi I'll look into it tomorrow.

worker.py is failing since this PR because it switched the tabletmanager protocol from BSON to gRPC.
We never ran worker.py with --protocol-flavors=grpc. Otherwise, we would have seen the original issue earlier.
The issue is that the current master gets reparented and therefore it (correctly) replies with an error "I'm read-only now. I cannot execute this SUPER LONG query for you." Since we include the full SUPER LONG query in the error,  it grows to 20kB and this makes gRPC choke. I've filed a bug with them: https://github.com/grpc/grpc-go/issues/443
I'll temporarily fix the issue tomorrow on our side by truncating the returned error.
btw: The tablet sends out the error, but vtworker never sees the error and the respective RPC times out after two minutes on vtworker's side (context deadline exceeded). We don't retry on an exceeded deadline yet and therefore the test fails. I'll fix this as well.

Great detective work! I think you mean it was PR #1299 that caused it. The test happened to pass on that one, so we didn't notice.

Yes, you're right. PR https://github.com/youtube/vitess/pull/1299 caused it :)

Nice bit of sleuthing @michael-berlin! Thanks for the follow-up work and filing the bug.
We can limit the number of queries we send per RPC request, which should limit the returned error size. Truncating seems like it's probably better though.
Yep, you're right, we should absolutely retry on exceeded deadline, I don't know why we're not.


run docs tests before npm test

Yea can add that as a separate PR also adding support for the shortcodes.


API Docs for Okta:User misrendered

What happened?
Visit https://www.pulumi.com/registry/packages/okta/api-docs/user/user/ and observe the page rendered sideways.
Example
The truth is in the pudding here.
Output of pulumi about
n/a
Additional context
It is very likely that the schema for Okta:User is sending badly formatted data. However, the misrendering should never happen. We should prefer to not render rather than have the page look as it does today.
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

thanks @guineveresaenger for filing this. Doing some poking around, I suspect this may be due to the markdown example code headings being embedded in the code block. Looks like this was an accident on their part. I suspect this may affect the way the page is parsed, but could be wrong here.
https://github.com/okta/terraform-provider-okta/blob/master/docs/resources/user.md?plain=1#L14

I opened a PR in the upstream to fix up the markdown.

resolved by: https://github.com/pulumi/pulumi-snowflake/issues/683


Wrong file attachments from recorded audio on whatsup

Hi,
If I record an audio file on whatsup with the microphone and after sent in whatsup I select the audio message and share in K9-mail the email is Sended with danaged file. The file audio Sended by Android 5.0 on Samsung galaxy S5 with whatsup share function then K9-mail arrive danaged. For send correctly and share I must use Gmail app with Gmail account.
Please fix this on k-9 mail so I can send by email audio recorded message via WhatsUp to email

Issue #744 contains more information.


Use function babbageMinUTxOValue for faster minimum UTxO calculations.

Issue Number
ADP-2144
Summary
This PR adjusts the {compute,isBelow}MinimumCoinForUTxO functions so that in the Babbage era only, we bypass the Cardano API function calculateMinimumUTxO, and instead directly use the ledger function babbageMinUTxOValue.
This leads to a significant reduction in latency when computing transaction fees:
Before:
    Latencies for 2 fixture wallets scenario
        postTransactionFee  - 3177.0 ms
    Latencies for 10 fixture wallets scenario
        postTransactionFee  - 3555.3 ms

After:
    Latencies for 2 fixture wallets scenario
        postTransactionFee  - 218.6 ms
    Latencies for 10 fixture wallets scenario
        postTransactionFee  - 225.8 ms

bors try

Triggered nightly latency step: https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1467#_

Triggered nightly latency step: https://buildkite.com/input-output-hk/cardano-wallet-nightly/builds/1467#_

Results:
    Latencies for 2 fixture wallets scenario
        postTransactionFee  - 12.5 ms
    Latencies for 10 fixture wallets scenario
        postTransactionFee  - 13.0 ms
    Latencies for 100 fixture wallets
        postTransactionFee  - 23.1 ms
    Latencies for 2 fixture wallets with 10 txs scenario
        postTransactionFee  - 13.3 ms
    Latencies for 2 fixture wallets with 20 txs scenario
        postTransactionFee  - 14.5 ms
    Latencies for 2 fixture wallets with 100 txs scenario
        postTransactionFee  - 16.6 ms
    Latencies for 10 fixture wallets with 10 txs scenario
        postTransactionFee  - 13.9 ms
    Latencies for 10 fixture wallets with 20 txs scenario
        postTransactionFee  - 15.2 ms
    Latencies for 10 fixture wallets with 100 txs scenario
        postTransactionFee  - 18.2 ms
    Latencies for 2 fixture wallets with 100 utxos scenario
        postTransactionFee  - 16.4 ms
    Latencies for 2 fixture wallets with 200 utxos scenario
        postTransactionFee  - 20.6 ms
    Latencies for 2 fixture wallets with 500 utxos scenario
        postTransactionFee  - 24.5 ms
    Latencies for 2 fixture wallets with 1000 utxos scenario
        postTransactionFee  - 46.9 ms

And results from recent master for comparison:
   Latencies for 2 fixture wallets scenario
        postTransactionFee  - 2851.2 ms
    Latencies for 10 fixture wallets scenario
        postTransactionFee  - 3049.6 ms
    Latencies for 100 fixture wallets
        postTransactionFee  - 11185.9 ms
    [this run seem to have errored for the rest of the results]

I'm kind of astonished how we'd be able to see this kind of extreme slowdown from the cardano-api wrapper... and only the babbage era thinking might laziness be involved somehow?

It might very well be!
I did run some experiments that added strictness annotations in certain places, but none of these experiments were fruitful. More analysis is needed: I'm very keen to work out exactly what the root cause is!

bors r+

bors r+


Color Issues

Cool!! ViewPager!

Hey @eastswift, Thanks.
Do you have any issue ? because i saw something mentioned regarding colors in my mail whereas found that you have edited that comment. Is your issue fixed ?
If you are still facing issue, can you elaborate more on that?

There are issues. The background color of the navigation bar and TABBAR does not change. Shadows come and go. What should I do?

@eastswift , can you please explain more? I have tried changing background color for navigation bar and its changing correctly. Also what do you mean "Shadows come and go" ? Can you post relevant screenshots and code of what you are trying to achieve?
Still i will provide you with solutions regarding common scenario.

ViewPager tabs shows behind navigation bar
self.edgesForExtendedLayout = UIRectEdge.init(rawValue: 0). Write it in viewDidLoad method of your view controller where you setup ViewPager.

For changing color of navigation bar , use
self.navigationController?.navigationBar.barTintColor = UIColor.blue.  Use barTintColor instead of tintColor. This should change your navigation bar to blue color

I'm sorry. It was my mistake. Your open source is Cool! Have a nice day :)
Thank you for your reply.


Support multiple contexts

The concept of Talon contexts has been designed into the architecture, but has not been fully implemented yet. We generate a default context. We need to test a second context and ensure everything works with multiple contexts.
The context abstraction is how you will support a talon managed front end and backend admin interface, for example.
Here is how a context works:

A set of talon resources are associated with a context
Routes are scoped to a context
Contexts can share themes or map do different themes

Relatedly, I'd add:

A given schema can be represented by more than one Talon resource providing different perspectives of that schema, even within a given Talon context.
A given Talon resource could be re-used across Talon contexts.

Currently, the context is bound to the resource file. That would have to be changed. Furthermore, there would need to be a run time association instead of the current compile time binding.

I believe I have the organizational structure design worked out. First some nomenclature:

theme - a UI layout including a layout, CRUD Phoenix templates, and UI assets including JS, CSS, Images, and fonts.
concern - a Talon "Endpoint" or interface. For example, a Talon implementation with a separate admin backend and front end interfaces.

The current design is based on Talon managed Web UI with the following requirements:

one or more concerns. For example, Admin and FrontEnd.
one or more installed themes i.e admin-lte, active_admin,  and admin-lte-topnav
theme configuration/customization is done based on concerns.
two separate concerns can use the same theme, while customizing their individual layouts, CRUD templates, and custom actions.
a single concern can have different themes with significantly different structure. For example, one Bootstrap based and the other Semantic UI based.
themes should be selectable at runtime.

The above requirements imposes a directory structure like:

web/templates/talon///

layout
generators - CRUD template generators for all concern resources for the given theme
/generators - CRUD generators for a specific resource

web/views/talon//

Additionally, the following ideas are open for discussion

the talon folder structure is used to make it clear what directories/files are part of talon

this adds an extra level of directories that may be seen as a complication

the Talon project is a web concern

a simplified directory structure could be:

lib/my_app/talon/

resources//

.ex
<resource_name>.ex

templates//
controllers
views//
web.ex

complete and merged


[UBS Admin table] Column names should be visible

Environment: Windows 10 pro, browser Chrome, Version 128.0.6613.120
Reproducible: always
Build found: 19.09.24
Preconditions:

Go to https://www.pick-up.city/#/ubs
Register as admin

Steps to reproduce:

Go to Admoin table
Pay attention to the names of columns

Expected result:
Min width of column should show all name of the table:

The bug has not been fixed


[Bug] Installing BBD loads browser discord

For somewhat reason, after my pc had blue screen, restarted it, then I started discord, I saw BBD was missing, so I re-installed it by downloading the latest version from the first link on google, after that discord was loading a browser version inside the client rather a client version with bbd on it, have tried complete re-install of discord and bbd but no solution, what else could I do?

the latest version from the first link on google

Here's where  you went wrong. Use the installers linked on the README... https://github.com/BetterDiscord/Installer

Yup, got that and it worked, I thought you'd still call it BandagedBetterDiscord rather than just BetterDiscord. Its all working great now!


Support Node.js v20 or higher

Problem/Motivation
I tried to install a module to Node-RED, which didn't work as it requires at least v20 of Node.js:
30 Oct 17:55:30 - [info] Installing module: @danpeig/node-red-meshtastic-msg, version: 1.4.3
30 Oct 17:55:36 - [warn] Installation of module @danpeig/node-red-meshtastic-msg failed:
30 Oct 17:55:36 - [warn] ------------------------------------------
30 Oct 17:55:36 - [warn] npm error code EBADENGINE
npm error engine Unsupported engine
npm error engine Not compatible with your version of node/npm<EMAIL_ADDRESS>npm error notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm error notsup Required: {"node":">=20.x"}
npm error notsup Actual:   {"npm":"10.7.0","node":"v18.20.4"}

It seems that this base image repository specifies the Node.js version of the Node-RED addon, which is currently v18.
I think support for v18 will soon end: https://nodejs.org/en/about/previous-releases
Is there a plan to upgrade the Node.js base image?
Expected behavior
Support a newer version of Node.js, at least v20.
Actual behavior
Base image uses Node.js v18.
Steps to reproduce
Install @danpeig/node-red-meshtastic-msg in Node-RED addon.

The whole point for this repo to exists is tk have an older version…
Newer versions don’t play well with cross building Docker images on certain platforms (like armv7). This is a known issue that is caused by a change in NodeJS, triggering a bug in QEMU (which is used by Doxker buildx).
As long as that isn’t resolved upstream… I am blocked. Heck if it is fixed upstream, I can drop this base image fully!


Remove impls tag for testcases working with all exsiting impls

Remove impls tag for testcases working with all exsiting impls
(openj9,
hotspot, sap).  When there is a new impls tests can run against it
automatically. If test doesn't work with new impls we can explicitly add
impls tag.
Add ProblemList_openjdk10-sap.txt to enable tests on openjdk10-sap

Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

Yes, good.  As the default is to assume if its not explicitly specified, the test case is valid for all implementations, this will allow for even more implementations to be supported without having to update the playlist. 👍


Implement PropertyOfVariantType typehint annotation

Annotation already existing in: PropertyTypeHintAnnotation.kt (already there, needs to be uncommented.
Add typehint and typelimiting support in PropertyHintTypeMapper.kt

Closing in favor of #155


Small FilterChecker cleanup

Let’s clean up FilterChecker a bit before it is split up.
Part of the work on #57.

@bkutil

+1


Fixes #2001 New main UI - progress bar in Nearby card view sometimes runs forever.

Description (required)
Fixes #2001 New main UI - progress bar in Nearby card view sometimes runs forever.
Nearby card notification is only visible if closest nearby item is loaded successfully.
Tests performed (required)
Tested on API 26, prodDebug and betaDebug.

Codecov Report

Merging #2021 into 2.9-release will decrease coverage by <.01%.
The diff coverage is 0%.

@@              Coverage Diff               @@
##           2.9-release   #2021      +/-   ##
==============================================
- Coverage         4.13%   4.12%   -0.01%
==============================================
  Files              223     223
  Lines            11227   11239      +12
  Branches          1032    1027       -5
==============================================
  Hits               464     464
- Misses           10728   10740      +12
  Partials            35      35

Impacted Files
Coverage Δ

...r/free/nrw/commons/contributions/MainActivity.java
0% <0%> (ø)
:arrow_up:

...fr/free/nrw/commons/media/MediaDetailFragment.java
0% <0%> (ø)
:arrow_up:

...w/commons/contributions/ContributionsFragment.java
0% <0%> (ø)
:arrow_up:

.../nrw/commons/nearby/NearbyNoificationCardView.java
0% <0%> (ø)
:arrow_up:

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 ea37245...3199775. Read the comment docs.

@neslihanturan I just tested your PR on a Samsung Galaxy S7 running API 26. The general flow works well if the user agrees to all permissions, and the loading works well also. :) Some changes are needed however:

Please rebase on 2.9-release so we don't have changes from master
Change strings as mentioned in comments
There is an issue that needs fixing - when I select 'no' at the initial "Permission Request" dialog, I do not ever get to the Android-generated permissions request, and therefore there is never an option to say "don't ask again". This means that every time I load Home, this dialog pops up again. There are a few possible ways of fixing it - perhaps we can have a "never ask again" option in the initial dialog, and set a flag that we then check before launching that dialog?

@neslihanturan It works now and does not spam the user. Great job! :)
I will go ahead and merge this, but there is one minor scenario that would be good if we can fix in 2.9.1. If I deny permissions in the dialog, then 'reset' it by enabling location permissions via Android settings and disabling again, instead of displaying the appropriate dialog, it instead displays the dialog "Permission required to display a list of nearby places". Even though I'm on Contributions tab.
However, that is a minor issue (I doubt too many users will be doing that), so it need not block the release. :)


Amplitude,Hacker earth event added

event added for the month of november

ive made the changes but im not able to put up a pr even after going through the fix on the internet please help

@darthgera123 Seems like there are conflicts, please resolve the conflicts without removing existing events


Make Basis singleton

Motivation
The Basis represent an abstract mathematical object that is unique, in the sense that it only provide the evaluations of objects that do not change. In the same sense, they must not have a state. Basis are stateless. I have used mutable private variables to improve performance and avoid creating new objects.
Because its abstract and unique nature, Basis must exists in a unique non writable instance.
Proposed solution
class Base {};

template <typename Current> class Helper {
public:
  template <typename... Ts> static std::shared_ptr<Current> get(Ts &&... args) {
    auto const values{std::make_tuple(args...)};

    static std::map<std::tuple<Ts...>, std::shared_ptr<Current>> database;

    if (database.find(values) == database.end()) {
      database[values] = std::make_shared<Current>(args...);
    }

    return database[values];
  }
};
int main() { return 0; }

See how to implement unordered maps with custom comparison operators. This is linked to #42

We can proceed as follows:

Implement the characterization of a basis with the following elements

Array of double parameters
Array of int parameters
string with name
In this case, we can also implement a downcasting from the base class

Implement a Base


[Question]: Best way to upgrade when using docker-compose?

What is your question?
Hi,
Just wanted to ask. What is the best way to upgrade when using docker-compose? I just deleted the ghcr.io/danny-avila/librechat-dev image in my local docker and it seemed to work.
But, for production use, how can I guarantee that all dependencies are upgraded like database migrations and other dependencies which are in other docker containers?
How can I keep the user and messages data during migration?
Thank you.
More Details
No other details
What is the main subject of your question?
No response
Screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

This: https://www.librechat.ai/docs/local/docker#update-librechat
You won't loose your data


Missing second paragraph of each chapter of Pro Git v2

(progit2 issue #582)
Not only chpater 3, but also chapter 4, 5, 6, 7, 9, 10 have the same problem. All content from second paragraph disappear from web page.

Fixed by #756.


Allow -1 (negative one) in cp.reshape

In NumPy, calling b = np.reshape(a, shape=s) for a tuple s permits one occurrence of -1 in s. The index -1 is automatically replaced by whatever value is needed for s to specify valid axis dimensions in reshaping a.
For example, np.reshape(a, (-1, 1)) will reshape a into a column vector, and np.reshape(a, (1, -1)) will reshape a into a row vector. More generally, np.reshape(a, (-1, k)) will reshape a into a matrix of shape (a.size / k, k) if a.size / k is an integer and raise an error if a.size / k is not an integer.
CVXPY's reshape function should have the same behavior as NumPy's (in this regard).
This is minor syntactic sugar as CVXPY exists today, since we only support Expressions with at most two dimensions. This feature would become more important once CVXPY supports n-dimensional Expressions.

See also #1695

Here is the shape handling of scipy sparse, which might be a relevant starting point for the logic and error checks. It only works for two dimensions, though.
Another quite compact point of reference is the pydata/sparse implementation found here.

This missed v1.3. It would be great to have in 1.4!


Bugish: Food / Consume Not Working

Before I start off, I'm going to just say that it does work, unless you're using Enjin's Minecraft plugin (which I need as well, so uh yeah).
This is marked as "bugish" because I'm not sure if there's an override or not. (or a setting I have to toggle to allow it to work with the plugin).
It's not critical, I just wanted to add an item that's going to be a new food item.

I had to update the plugin and it started working again. :)


Problems running this demo

Hi.
Apologies in advance if this is the wrong location to ask for help.  I'm old and dumb.
Cloned jopenvr, jogl-hello-vr, and related libraries jgli, jglm, and java-unofficial-open-SDK.  Massaged jogl-hello-vr to compile in Eclipse.  Tweaked App::createAllShaders() to read from the correct directory and load the shaders.
When I run the program a window appears on my desktop flashing black/dark-grey very fast.  HTC Vive HDR still shows default view ("this is real").
Any idea what am I doing wrong?

Hi,
the sample has been yet to be finished, I wrote it almost completely without running it a single time
there must be a couple of bugs somewhere I guess

Please: Any ETA on a working version?  I need something that works as a base for my code.

unfortunately not yet

What do you need on the 11th? Is it fine also the OpenVR sample itself?

I need it before the 11th because I need to make https://github.com/MarginallyClever/Robot-Overlord work with VR.

On Jun 2, 2016, at 11:53 PM, Giuseppe Barbieri<EMAIL_ADDRESS>wrote:
What do you need on the 11th? Is it fine also the OpenVR sample itself?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi,
I got to the same point as @i-make-robots, will have a bit more of a tinker and post here if I make progress (although I should also be getting back to some other stuff...)

If you can help, I'll add you

I would help but...
http://img2.rnkr-static.com/list_img_v2/9660/549660/full/very-best-of-the-i-have-no-idea-what-i-m-doing-meme-u1.jpg
On Tue, Jun 14, 2016 at 11:07 AM, Giuseppe Barbieri <
<EMAIL_ADDRESS>wrote:

If you can help, I'll add you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jovr/jogl-hello-vr/issues/1#issuecomment-225966400,
or mute the thread
https://github.com/notifications/unsubscribe/ABZYhrkFhtbnNHhnN6OCrKHwhmU6fFghks5qLu3ogaJpZM4Ir1eg
.

--
Dan Royer, Owner, Marginally Clever Robots
https://www.marginallyclever.com/
Ph: +1<PHONE_NUMBER>
Mo: +1<PHONE_NUMBER>

Well, I'm making some progress I think.  Not quite managed to actually get the thing showing the graphics I want, but close I think (actually, possibly close enough that it might even be easier to get it working with my own app than this sample at this point, but it'd be nice to make a contribution of use to others and my app isn't open source ATM unfortunately).
What I see now is the solid clear color which I've set to animate and different for each eye, so at least I can tell results of our drawing are being used.  I can't actually remember which change it was that made it show that JOGL output on the HMD rather than the default stuff... (stupidly haven't even been using any source control for my changes so far, have probably made enough progress I should address that).
Commenting out the call to drawable.swapBuffers(); in the middle of display gets rid of the flickering on the window, although I note that you were following Valve's original C++ sample where they do the equivalent for performance.
I've added pose updating. Printing out hmdPose to console, the numbers look plausible.  I think whatever bug I'm dealing with now is in the basic gl/scene code, nothing to do with OpenVR as such.  As you can appreciate, I have less appetite for debugging someone else's GL code that I'm never going to use than I do for getting something working with OpenVR, hence it probably makes sense for me to try to integrate my own code as a next step.
I removed the distortion code, following stumbling on this issue https://github.com/ValveSoftware/openvr/issues/42 where it is mentioned distortion is dealt with by the compositor.
I'm not initializing the chaperone stuff or some other bits atm, don't think that's the cause of problems, but I suppose it's possible.

Well, getting the clear color is undoubtedly the proof jovr is working  :scream:
Thanks for all the hints you mentioned, they are extremely interesting. I am gonna try them as soon as I get some spare time to work on this.
Anyway I just added you, you may want to submit your changes with a small comment to help identifying which may be the code parts to get it working

I've noticed that the chaperone box does appear when you move towards edges of space.

So, the call to glDrawArrays in Scene.render() is leading to GL_INVALID_OPERATION error generated. Array object is not active.
Not sure yet what is different about your code compared to the Valve sample such that it fails.  Such are computers.

I'll investigate, thanks for the contribution

I was wrong again in my edited version of above comment; my change to the stride (5 * Float.Bytes rather than VertexDataLens.SIZE) doesn't get rid of the error, I had the draw call itself commented out when I reintroduced the stride change, hence no error.

Well, FWIW, my other app is sort-of working now.  For some reason HMD orientation is fine, translation doesn't seem to be happening and I've got some slightly odd frustum clipping going on, but it doesn't seem likely any of that's jOpenVR related - I'm probably just not accounting for all matrix mangling that's going on in my code.
Back in jogl-hello-vr, I also noticed that setupTextureMaps isn't being called yet.  That wouldn't likely fix the Array object is not active error, but presumably might prove important shortly after.
p.s. there's a few points in the code where I put XXX: garbage comments; not a judgement on code quality, just flagging new objects being made which if the methods were called frequently would lead to excess garbage (actually, at a glance the only place where this really happens is the two new Texture_t objects in display, which is hardly close to being anything worth commenting on, so you really I'm just rambling about nothing much).

This morning I updated jogl-hello-vr, jopenvr, jglm, jgli, and java-unofficial-opengl-SDK.  App says it si running and a black window appears on my PC.  headset still shows default view, floor, and playspace fence.  What am I doing wrong?

Use xinaesthete's commit as a starting base for your app

I might get back to my fork and replace the GL bits with something that at least works, but can confirm that elsewhere I'm using very similar OpenVR code and it is working (well, for some reason still not tracking position properly, but that must be a bug in my code).
I've done a bit of messing with WorldWind in the past - various things I plan to do with that if I make time for it (or ideally raise funds).  I might have a go at hacking an OpenVR layer onto that sometime (just thinking aloud).

It sounds promising :)

xinaesthete, I tried the changes you made, I only get a GL Error 1282 and nothing get rendered on the screen, not even the clear color.
Anyway we updated jovr, may you try if your app still works with it?

We made it, I commented out updateHMDMatrixPose() where compositor.WaitGetPoses.apply() essentially is

I made it, it works  :scream:
Thanks xinaesthete for your tip! :+1:


Custom searches

Allow users to define different searches.
For example, searching through dependency/library files. Additionally configuring custom search implementations should be implemented.

One option for this would be to define a new class, which would load a set of custom searches from the settings and allow selecting one of these with a configurable shortcut. The simple options would still be available as the default use case of the plugin
This would make selecting the correct custom search a chord, where the initial shortcut would open the selector and the second key would select the correct search.
e.g. sc would open the selector and "s" would open the corresponding custom search


Train steps and data process

I want to train this model on nuscense and other datasets, could you please release the training steps and data preprocess steps, many thanks.

Hi, this work is built based on mmdet3d, please refer to the tutorials of mmdet3d to perform data preprocess and training https://mmdetection3d.readthedocs.io/


CLI help text should be more consistent with other Puppet tools

Use Case
The CLI help text for Bolt should be updated to be more consistent with the formatting and content of other Puppet CLI tools like r10k and pdk. Currently, the way Bolt displays available subcommands and actions is inconsistent with these other tools, and also has some inconsistency in how it displays information for its own commands.
Updated CLI help text would help improve the readability of the help text and make it clearer what different subcommands and actions are available and what they do.
Describe the Solution You Would Like
Update the help text to be more consistent with other Puppet tools like r10k and pdk. Help text should have the name of the command, usage of the command, any available subcommands or actions, an example of how to use the command, and any available options.
r10k help text
NAME
    r10k - Killer robot powered Puppet environment deployment

USAGE
    r10k <subcommand> [options]

DESCRIPTION
    r10k is a suite of commands to help deploy and manage puppet code for
    complex environments.

COMMANDS
    deploy         Puppet dynamic environment deployment
    help           show help
    puppetfile     Perform operations on a Puppetfile
    version        Print the version of r10k

OPTIONS
    ...

Proposed Bolt help text
NAME
    bolt

USAGE
    bolt <subcommand> [action] [options]

DESCRIPTION
    Bolt is an orchestration tool that automates the manual work it takes to maintain
    your infrastructure.

SUBCOMMANDS
    apply                  Apply Puppet manifest code
    command                Run a command remotely
    file                   Upload a local file or directory
    group                  Show the list of groups in the inventory
    inventory              Show the list of targets an action would run on
    plan                   Convert, show, and run Bolt plans
    project                Create and migrate Bolt projects
    puppetfile             Install and list modules
    script                 Upload a local script and run it remotely
    secret                 Create encryption keys and encrypt and decrypt values
    task                   Show and run Bolt tasks

GLOBAL OPTIONS
    ...

It is consistent with puppet help text though. I don't strongly care, but I don't think this is an important thing for us to work on.


Remove unused assets

Remove unused assets
Resolves: #1859
Change-type: patch
Signed-off-by: Andrea Rosci<EMAIL_ADDRESS>I wrote a parser that check which assets are unused. Those found seem to me to be really many, so I would like to calmly check if correct.

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

Running the DOCS locally everything works fine, with all the images. I hope no external project uses these images :)

@JSReds This such good PR but you raise a fair point. Is there any way for us to check about that? We can do a team ping for folks to make a note about this?

Hey @JSReds just an update on this PR. I did check on the Heroku dashboard and there isn't a way to check the requests that the server is getting. I am actually thinking to hold this PR for a bit, as we go around generating the getting started guides from contracts. These image might serve to be useful inside contracts to someone else in a later stage. If they are not, we can surely remove them. Atm let's keep them and around revisit this work when that task is done.

@balena-ci rebase

Thanks for all the hard work @JSReds Found something else in the docs.


Make sure all CUTGroup test responses are displayed in Patterns

Over the last 19 tests we have had some issues with make sure that test answers are entered into Patterns due to webhook issues (#84), misspellings in the email address, etc. I want to take the time to go through and make sure all of the tests are listed properly in Patterns so that we are able to count the tests each tester has done (#94).
@shua123 We should also think what makes sense about how to make sure we distinguish the test responses vs other forms. Can we create a different webhook to do this?

Loop through wufoo forms with API
check if entries are in submissions table
add if possible. If not, make list

Looped through and added ~92 submissions. There are still submissions that could not find a person to match to that need manual attention.


Replace OpenSUSE with openSUSE

It is how the name of this distro is usually stylized.
Cheers :)

Thanks :-)


add serve static with basic frontend to display data from Tensorflow

Feature Request
Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Teachability, Documentation, Adoption, Migration Strategy

https://docs.nestjs.com/recipes/serve-static
What is the motivation / use case for changing the behavior?

Already Added.

Already Added.


Error:  error: AnalyticsListenerImplementor is not abstract and does not override abstract method onRenderedFirstFrame(EventTime,Surface) in AnalyticsListener public class AnalyticsListenerImplementor

After using XamarinMediaManager I am getting:

Error:  error: AnalyticsListenerImplementor is not abstract and does not override abstract method onRenderedFirstFrame(EventTime,Surface) in AnalyticsListener
public class AnalyticsListenerImplementor

I added:
public override void onRenderedFirstFrame()
{
       // LogUtil.i(this, "MainActivity.onRenderedFirstFrame.");
}

to my MainActivity.cs but still not fixed.
How can I fix this please?

See https://github.com/martijn00/ExoPlayerXamarin/issues/64

https://github.com/martijn00/ExoPlayerXamarin/issues/64#issuecomment-393505696

This exception still appear on the latest version!
https://github.com/martijn00/ExoPlayerXamarin/issues/64#issuecomment-393505696


Remove pass_context_arg_name argument

I propose to remove the pass_context_arg_name argument and fix the name under which the context is passed. This parameter is passed often between internal functions and classes, which is not worth the benefit of having it configurable.

We'll have to take into account name collisions. E.g. if there is a parameter called context in the spec - then this could cause trouble if we want to pass the request context as context


grid support (1st iteration)

Thanks to the refactored resize functions, it was pretty easy to add grids for resizing a single element. (Drawing grids was harder.)

TODOs:

[x] enable/disable shortcut (disabled by default)
[x] resize a single element
[x] resize multiple elements
[x] drag a single elements
[x] drag multiple elements
[x] create a new elements (rectangles, diamonds, ellipses)

For follow-up PRs:

context menu to toggle and customize grid size
modify multi-point lines
create new elements (two-point lines)
create new elements (multi-point lines)
create new elements (texts)

Cool. I thought it's gonna be much harder than this!

we'll want to expose it under a shortcut (Cmd/Ctrl+' is the standard), not as a default
moving elements should move on the grid, too

And, creating new elements too.

I would like to have a context menu to enable/disable grid and configure the grid size (10, 20, `40), but not sure how to add context sub menus.

I would like to have a context menu to enable/disable grid and configure the grid size (10, 20, 40), but not sure how to add context sub menus.

We'll want submenus sooner or later, anyway. But maybe stick with single grid size for this PR and add the granular support later.

I love this!

Would you help with the snapshot tests? Don't know why it fails other than gridSize: null.

There's one failing case that seems like a regression (but I couldn't figure out what exactly that regression is, playing with the preview):
 ● duplicate element on move when ALT is clicked › rectangle

    expect(received).toEqual(expected) // deep equality

    - Expected
    + Received

      Array [
    -   30,
    -   20,
    +   0,
    +   40,
      ]

      89 |
      90 |     // previous element should stay intact
    > 91 |     expect([h.elements[0].x, h.elements[0].y]).toEqual([30, 20]);
         |                                                ^
      92 |     expect([h.elements[1].x, h.elements[1].y]).toEqual([-10, 60]);
      93 |
      94 |     h.elements.forEach((element) => expect(element).toMatchSnapshot());

      at Object.<anonymous> (src/tests/move.test.tsx:91:48)

And some of the snapshots are failing on similar because there are significant changes to x/y as well (not just gridSize: null).

Ok, I'm not sure if this is the exact regression, but here's one: when you drag an element and press Alt mid-way, the non-duplicated element should revert to orig position.

Thanks! That helped a lot. That's the exact regression.
Fixed.

I've played with the preview and it is amazing!
What I don't like is how the blue selection box also snaps to the grid - makes the app feel non-responsive, glitchy or slow (because the box no longer tightly follows the cursor) - what was the intention with snapping the selection to the grid?

@xixixao Nice catch! It's not intentional. Fixed.

Let's add the shortcut to the shortcut dialog.
As for exposing it on elsewhere (contextmenu etc.), I asked on chat and we should discuss whether adding a settings button isn't a better idea.

For new lines/arrows it doesn't align to grid (witch click - drag, not with click-click-click).
Otherwise it looks great!

I would like to have a context menu to enable/disable grid and configure the grid size (10, 20, 40), but not sure how to add context sub menus.

You could have cmd+' cycle through 10, 20, 40, none. This way no need for configuration.

When drawing a circle, the first few pixels are not aligned to the grid and then they are, this makes it feel like there's a glitch when doing it quickly.

The grid doesn't seem to be propagated in collaboration mode. Is this something that we want to happen or is the current behavior is the expected one?

You could have cmd+' cycle through 10, 20, 40, none. This way no need for configuration.

No, the common use will/could be to quickly toggle it on/off when you just want to align a shape, or draw one with some proportions. If we make it cycle through 4 settings it'll be a nightmare, not to mention you'll always shoot over the setting you wanted and need to cycle again.
If we're adamantly opposed to the settings idea, let's keep one grid size only and keep it simple.

The grid doesn't seem to be propagated in collaboration mode. Is this something that we want to happen or is the current behavior is the expected one?

I can see arguments for either. Maybe someone on the other end wants to draw on grid, but others don't? How would that work then.

collaboration mode

I don't know which is better. Notice the background color is not in sync.

Looks great!
One last thing, I'd store gridSize when exporting to file (cleanAppStateForExport()) so it's restored on open.

Ad the shortcut. Can you try if Cmd+' works for on you on Figma? I think they must be using event.key because there on French keyboard (AZERTY) that shortcut doesn't work (the ' is one key to the right of QUERTY physical position, but Figma doesn't accept that physical key on my QUERTY ISO keyboard either, so it must be using event.key).
I'd really love to get my hands on Mac (keyboard) so I can do proper debugging.

(Never used figma...)
Should we go with this?
diff --git a/src/components/App.tsx b/src/components/App.tsx
index dba59fd..27e18b4 100644
--- a/src/components/App.tsx
+++ b/src/components/App.tsx
@@ -1184,7 +1184,7 @@ class App extends React.Component<any, AppState> {
       this.toggleZenMode();
     }

-    if (event[KEYS.CTRL_OR_CMD] && event.keyCode === KEYS.GRID_KEY_CODE) {
+    if (event[KEYS.CTRL_OR_CMD] && event.key === KEYS.GRID_KEY) {
       this.toggleGridMode();
     }

diff --git a/src/keys.ts b/src/keys.ts
index 10e82bc..d31d0d5 100644
--- a/src/keys.ts
+++ b/src/keys.ts
@@ -16,7 +16,7 @@ export const KEYS = {
   F_KEY_CODE: 70,
   ALT_KEY_CODE: 18,
   Z_KEY_CODE: 90,
-  GRID_KEY_CODE: 222,
+  GRID_KEY: "'",
   G_KEY_CODE: 71,
 } as const;

I dunno TBH. Per discussion on chat, if we go with event.key for this one, it'll take layout into account (which is generally good), but won't work for AZERTY (and maybe other) layout at all :/, whereas what you used previously will work on all layouts, but e.g. on AZERTY the labeling won't be useful for AZERTY users at all (the physical key will be ù).

OK, as there doesn't seem to be a perfect solution, let's leave it.

Actually, keyCode doesn't work on AZERTY at all either, I must have tested it wrong. It seems the ù uses a different keyCode altogether. Anyway, let's leave it for now.

That said, some users wouldn't be able to use this feature until we added it to UI somewhere. I'm wondering if we should add it to contextmenu right now, even if we later decide to move it elsewhere.

For me as a user, I'd be happy to have context menu anyway (not only about grid), even if we had another config UI.

Yea, let's do it.


Introduce header validation as concept

This then paves the way for introducing additional checks
(which this doesn't do yet).

One annoying thing: EBBs (obviously). Block numbers don't always increase. But if we do this generically, then that generic code must be EBB aware :angry:

One annoying thing: EBBs (obviously). Block numbers don't always increase. But if we do this generically, then that generic code must be EBB aware angry

Thinking out loud: what if we make this blk-specific using, e.g., RunNode? The default is increasing, but we can override the implementation for Byron to handle EBBs correctly.

No, it doesn't belong in RunNode. We need an abstraction that captures this. It's part of the ledger definition.

bors r+

bors r+

Rebased and restarted, since Hydra seemed stuck.

bors r+


Download testing datasets

to Download testing datasets ,
how to set the dataset?
could you say the detail of it ?
thanks

Put the VOT2018 in the testing_dataset, also download VOT2018.json and put it in the VOT2018 directory

VOT2018/ants1/color
里面 ants1的 图片，gtroudtruth ，color等的层次关系是什么？


Create partner project Action

Gonna close this for now to avoid confusion. It was part of my exploration for a partner issue reporting workflow, which I'm putting on hold for now.


[Stabilization 2.6.0] HandInteractionExamples scene: Unity UI Slate text filed : text wrapping is not supported but input field is vertically big.

Overview
[Stabilization 2.6.0] HandInteractionExamples scene: Unity UI Slate text filed : text wrapping is not supported but input field is vertically big.
Before & After

Changes

Fixes: #9331

/azp run


implement fmt::Display for Matrix

Users of a linear algebra library might occasionally want to pretty-print a matrix!
use rusty_machine::linalg::matrix::Matrix;

fn main() {
    let m = Matrix::new(3, 3, (1..10).collect::<Vec<_>>());
    println!("{}", m);
}

// outputs:
//
// ⎡1 2 3⎤
// ⎢4 5 6⎥
// ⎣7 8 9⎦

Another super useful contribution, thank you!
Before I merge this one I'd like to get your opinion. You added the test to an internal tests module and I've seen this a lot through other repos. Do you think it is worth migrating the tests I currently have in the tests folder to internal modules as well? If so I'll make an issue for that.
My current opinion is that it probably is useful - it's hard maintaining those separate tests right now (and they need updating!).

I looked a little into display accuracy. We can control precision based on the print statement, e.g.
use rusty_machine::linalg::matrix::Matrix;

fn main() {
    let m = Matrix::new(2, 2, [1.234, 2.3456, 1.1, 2.567]);
    println!("{:.2}", m);
}

// should output:
//
// ⎡1.23 2.34⎤
// ⎢1.10 2.56]

To achieve this we use the following in the fmt function:
if let Some(prec) = f.precision() {
    try!(write!(f, "{:>2$.*}", prec, datum, width));
}

This will require computing the width differently (if a precision is present we will have to compute it using this precision.
It's your call as to whether this is included in this PR - I'm happy to write up a separate ticket.

You added the test to an internal tests module and I've seen this a lot through other repos. Do you think it is worth migrating the tests I currently have in the tests folder to internal modules as well?

The relevant sections of The Book seem to recommend same-file submodules for unit tests and a tests directory for larger-scale integration tests (exercising multiple features of the library the way a client would).

Do you think it is possible to set a display accuracy [...] This will require computing the width differently

Good point; what do you think of 865bc6dc1? (fa221ad0 also adds a test for the one-row special case.)

The new changes are great; thanks for your work! I'll release a new version with these improvements soon.


Publisher and extension name was reversed

The VSCode extensions provided by Nixpkgs are, for example, vscode-extensions.github.copilot, ({publisher}. {ext-name}), but all VSCode extensions provided by this flake are like vscode-xml.redhat, ({ext-name}. {publisher}). Is there some intention behind this?

This was totally a mistake on my part. I will fix it now.


⚠ ESPN has degraded performance

In 9d385b8, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 2192 ms

Resolved: ESPN performance has improved in dbf48ee.


Roll engine f56623d65e2a..2abf1101b482 (1 commits)

git@github.com:flutter/engine.git/compare/f56623d65e2a...2abf1101b482
git log f56623d65e2a..2abf1101b482 --no-merges --oneline
2019-10-24<EMAIL_ADDRESS>Roll src/third_party/dart d43cd7e909..081c01e89a (7 commits)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Auto-roller completed checks. About to merge.


Contacts added via v3 API are never processed

Issue Summary
Tried adding contacts through the /v3/marketing/contacts endpoint mentioned here at the docs. Both the "try it out" tool and the nodejs sdk gave me no problems, a job_id is returned as expected, but the contacts never actually show up on the list, seems like they're never processed. It's been more than six hours since I added the first one, so this seems like an issue.
Steps to Reproduce

Use the Try it out tool with your own list and API key
Get a successful response
Check the list at least half an hour later to see if the new contact has been processed

Here's my code. Again, the API returns an OK response with a job_id, but the new contact never shows up on the list.
const client = require("@sendgrid/client")
client.setApiKey("TOKEN")

const request = {
	method: "PUT",
	url: "/v3/marketing/contacts",
	body: {
		list_ids: ["list-id"],
		contacts: [
			{
				id: "custom_id",
				email<EMAIL_ADDRESS>				first_name: "Alfredo",
				last_name: "Re",
			},
		],
	},
}
client
	.request(request)
	.then(([response, body]) => {
		console.log("response.statusCode")
		console.log(response.statusCode)
		console.log(JSON.stringify(body))
	})
	.catch(err => {
		console.log("error")
		console.log(err)
	})

Technical details:

sendgrid-nodejs Version: 6.4.0 master (latest commit: 8b20079ec82d9bd83254de173ecd3c3c989e5f64)
Node.js Version: 8.11.4

Closed by https://github.com/sendgrid/sendgrid-nodejs/issues/957#issuecomment-559244701

I'm having issues with this on the GET list API endpoint that has conflicting documentation and results differ.

https://github.com/sendgrid/sendgrid-csharp/issues/1013 <-- posted this.
https://api.sendgrid.com/v3/marketing/lists/{list ID}?contact_sample=true <-- works with my authentication and gives me a sample of 50 years. I need all of them, so I looked further down at documentation and found the CONTACT LIST API section.
When I try this ... it does not return what I expect, based on the sample and both in POSTMAN to test the endpoint and also the TRY IT OUT tool on the website.
https://api.sendgrid.com/v3/contactdb/lists/{list ID} <--- I expect this to work
https://api.sendgrid.com/v3/contactdb/lists/{list ID}?list_id=1 <-- why dual list IDs?!?!?! Documentation and sample both give me issues.
I get an access forbidden and know EXPLICITLY my API_KEY has full access and I can indeed get access to that list in SendGrid.
Anyone from SendGrid able to reproduce this issue?

@Buffalonian Someone will follow-up on the new ticket you opened. Let's just keep the conversation there for easier tracking.