Checkmark is not visible after first-time authorization

Hey, for some reason the checkmark to confirm your image selection is not visible after authorization to use the photo library/camera.
Also, any way to disable the disabling of the status bar? I think it may be the cause to messing with my collection view, but i'd like to disable it anyway :)
Thanks.

Hey, I have this issue as well. I think its related to the UIAlertController that asks for photos permissions - I've only noticed the issue when it presents. A fix for the time being is simply to ask for photo permissions before presenting Fusuma. I tried this and it seems like it resolves the issue.

Fixed this here: https://github.com/ytakzk/Fusuma/pull/116

Awesome :)
For my second question/problem, for anyone wondering, the UICollectionView automatically sets it's offset to 0, hence the skewed offset. Solved it by setting the offset to the current one through viewDidDisappear, with no animation so it won't be noticed :)


try to give the CI job permission to create PRs

See the discussion here.
@rust-lang/infra just a heads-up that we are doing this -- I have no idea if this could cause any issues elsewhere, or if it is important that we limit the permissions here somehow, or so.

@bors r+

:pushpin: Commit 24c58e6bd61da983f8e898efb5ebd5cb0b0aedc8 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 24c58e6bd61da983f8e898efb5ebd5cb0b0aedc8 with merge 4c3b86d4acaa290c0de3d66e6a0322f90a1563dd...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 4c3b86d4acaa290c0de3d66e6a0322f90a1563dd to master...


Publish Adaptive.Teams package with correct tags

This package needs to be republished with the new tags so that it is available in Package Manager in Composer.
Should be tagged:

msbot-component
msbot-action
msbot-trigger

I think this is already correct: https://github.com/microsoft/botbuilder-dotnet/blob/main/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive.Teams/Microsoft.Bot.Builder.Dialogs.Adaptive.Teams.csproj#L18

Csproj not reflected in nuget: https://www.nuget.org/packages/Microsoft.Bot.Builder.Dialogs.Adaptive.Teams/4.12.2-preview

Reflected here correctly: https://botbuilder.myget.org/feed/botbuilder-v4-dotnet-daily/package/nuget/Microsoft.Bot.Builder.Dialogs.Adaptive.Teams/4.13.0-rc3.preview


OBJ_create: lock is unlocked without ever having been acquired

https://github.com/openssl/openssl/blob/openssl-3.1/crypto/objects/obj_dat.c#L742-L752
These two goto err; will invoke ossl_obj_unlock, even though they
haven't called ossl_obj_write_lock.
At least with pthreads, this is undefined behavior.

Hmm. This seems to have been fixed in master some while ago via #17417 (which covered a whole bunch of different issues). But that only went to master and was not cherry-picked to 3.1. There was a 3.0 backport in #17443, but it was 3.0 specific and did not include the fix for OBJ_create (because it wasn't relevant there). So 3.1 never got that set of fixes. So I think we need a 3.1 backport of #17417 to cover all the fixes there.

The situation is even more complicated actually. It seems that at the time of the merge of #17417 the 3.1 branch did not contain the new thread safe OBJ_create implementation. That was backported later as part of the mass backports of code from master to 3.1 - but the backport of the subsequent fix was somehow missed.

The only fix from #17417 that is missing in 3.1 are actually the obj_dat.c fixes - so yeah, not just the unlocks without lock but also the leak of tmpoid on error.

Fix in #20615


OSSSMC sidebar tests

this PR finalizes smoke tests for sidebar navigation in OCP
tests do verify that Kiali elements are loaded in each view

small edits for npm / yarn scripts
install and uninstall routine
tested on RC build of OSSM 2.5 on OCP

yarn cypress:run
Install and enable OSSMC plugin in OpenShift
✓ Verify oc command is available (8263ms)
✓ Prune OSSMC installation if any (763ms)
✓ Execute OSSMC is installation (2988ms)
✓ Verify that OSSMC is ready to use (9879ms)
✓ Validate OSSMC custom resourceces (12516ms)
Kiali integration with OCP Console
✓ Service mesh buttons are displayed (15474ms)
✓ Overview page is displayed correctly (7672ms)
✓ Graph page is displayed correctly (7413ms)
✓ Istio Config page is displayed correctly (8759ms)
Uninstall OSSMC plugin in OpenShift and all attached resources
✓ run uninstallation scripts to clean OSSMC resources (24632ms)

yarn cypress:run

  Install and enable OSSMC plugin in OpenShift
    ✓ Verify oc command is available (8263ms)
    ✓ Prune OSSMC installation if any (763ms)
    ✓ Execute OSSMC is installation (2988ms)
    ✓ Verify that OSSMC is ready to use (9879ms)
    ✓ Validate OSSMC custom resourceces (12516ms)

 Kiali integration with OCP Console
    ✓ Service mesh buttons are displayed (15474ms)
    ✓ Overview page is displayed correctly (7672ms)
    ✓ Graph page is displayed correctly (7413ms)
    ✓ Istio Config page is displayed correctly (8759ms)

Uninstall OSSMC plugin in OpenShift and all attached resources
    ✓ run uninstallation scripts to clean OSSMC resources (24632ms)


tinymce localization support

I've added language packs for TinyMCE 4.x to assets folder (downloaded from official TinyMCE website http://archive.tinymce.com/i18n/)
There are new config option tinymce => language in config/rapyd.php to be able to specify whether to use default translation (false) or use language pack from assets.
Finally, tinymce language initialization added to Redator class.

@dmdnkv thank you


CORE-11751: Enable crypto-utils publishing

Issue flagged testing CSDE as crypto-utils dependency is required in simulator.

Raised #3399 as this should be targeted to the release branch and not Beta2


Added docker tasks

Added docker cand docker compose tasks

@jitekuma,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@jitekuma,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot


Removal of host-environment-markers breaks backwards compatibility

Commit ecf78424c1740bae3511c731c032a3768154072c removes the host-environment-markers key from the Pipfile.lock _meta section.
This means that Pipfile.lock files generated by >11.0 won't be readable by earlier version of pipenv, because they will fail with a KeyError when trying to reference it.
A concrete form of this issue occurs when using the Heroku python-buildpack, because it is currently pinned to version 11.8.2 (correction: that's what their master branch says now, in production they must be running 10.x for this bug to manifest) which expects host-environment-markers to be there. When you try to git push heroku using the latest pipenv you get a deploy error. (see https://github.com/heroku/heroku-buildpack-python/issues/669)
I'm not sure what the correct solution is here. One possibility would be to add back host-environment-markers for now to preserve backwards compatibility and emit a deprecation warning for a while and then remove it down the road.
Another possible option here is to WONTFIX this, and just say that Pipfile.lock compatibility won't be emphasized, yet. But this means that organizations will have to lock-step their pipenv installations so that everyone is using a similar version. I don't think this is tenable.

This might not be an issue after all. Hour of research later, it turns out my issue was fixed by purging heroku's build cache: https://help.heroku.com/18PI5RSY/how-do-i-clear-the-build-cache
I'm gonna go ahead and close this for now. I'll open it back up if I can prove that the issue as described is really at fault in some way.


fix(dashboard): hash dashboard terminal section caching key to support long commands

Description
This change hashes the caching key for snacks dashboards terminal sections instead of building the key from the command itself. This allows for arbitrary length scripts to be given as a terminal section command instead of relying on existing commands in PATH.
Related Issue(s)
#379
Screenshots

That's awesome, thanks!
TIL about vim.fn.sha256


How to run Prometheus with mesh start without giving an error?

🕸  Mesh - Worker 4 💥 Error: A metric with the name graphql_mesh_fetch_duration has already been registered.
configuração
# ...
serve:
  hostname: <IP_ADDRESS>
  playground: true
  fork: 2
# cors:
#   origin: http://localhost:4001

# ...
sources:
  - name: API-V1
    handler:
      graphql:
        endpoint: ${API_V1:http://localhost:3004/graphql}
  - name: API-V2
    handler:
      graphql:
        endpoint: ${API_V2:http://localhost:3005/graphql}

plugins:
  - prometheus:
      skipIntrospection: true
      requestCount: true
      requestSummary: true
      parse: true
      validate: true
      contextBuilding: true
      execute: true
      errors: true
      delegation: true
      fetch: true
      fetchRequestHeaders: true
      fetchResponseHeaders: true
      http: gateway
      httpRequestHeaders: true
      httpResponseHeaders: true
      deprecatedFields: true
      endpoint: /metrics

I get the same issue. My dependencies are as follows:
"@graphql-mesh/plugin-prometheus":` "^0.98.2",
"prom-client": "^15.1.0"


Downsampling in query service?

It seems we have the wrong scope of these flags, what meaning do they have for the query service?
$ SPAN_STORAGE_TYPE=memory go run ./cmd/query help
...
      --downsampling.hashsalt string               Salt used when hashing trace id for downsampling.
      --downsampling.ratio float                   Ratio of spans passed to storage after downsampling (between 0 and 1), e.g ratio = 0.3 means we are keeping 30% of spans and dropping 70% of spans; ratio = 1.0 disables downsampling. (default 1)
...

I can schedule some time to track this down.

I can schedule some time to track this down.

The query service adds storageFactory flags here:
https://github.com/jaegertracing/jaeger/blob/e677384baeb9a39a8ec1bf7ec55ed1afc1d35323/cmd/query/main.go#L144
which adds the downsampling flags here:
https://github.com/jaegertracing/jaeger/blob/e677384baeb9a39a8ec1bf7ec55ed1afc1d35323/plugin/storage/factory.go#L170
A quick git blame suggests it has been this way for years.  We could split these methods into AddReaderFlags and AddAllFlags?  If that makes sense I'll submit a PR.

We could split these methods into AddReaderFlags and AddAllFlags

+1
However, when archive storage is configured query can also write


Make 'raise_for_status' return the response object

This would make simple usages simpler:
result = requests.get("http://api.server.com/path").raise_for_status().json()

i know this PR is quite old, but i really like it!... mostly because IMO it is completely in sync with the requests package's main objective of reducing clutter. without this change, the following is required:
result = requests.get("http://api.server.com/path")
result.raise_for_status()
result = result.json()

isn't @vmalloc's version much cleaner? if the problem is the naming of raise_for_status, how about renaming it (with a deprecating alias)? e.g.:
result = requests.get("http://api.server.com/path").require_ok().json()

(or whatever you want) or how about a new option to all of the request methods:
result = requests.get("http://api.server.com/path", require_ok=True).json()

btw, i currently always monkey-patch Response.raise_for_status to do exactly this. ugly!

I frankly still don't understand the reason this was rejected. I don't
think this qualifies for "API breakage" per se, and the utility of this
change is enormous.
I suspect the pattern you mentioned is prevalent in 99% of the use cases.
On יום ו׳, 13 במאי 2016 at 0:08 metagriffin<EMAIL_ADDRESS>wrote:

i know this PR is quite old, but i really like it!... mostly because IMO
it is completely in sync with the requests package's main objective of
reducing clutter. without this change, the following is required:
result = requests.get("http://api.server.com/path")
result.raise_for_status()
result = result.json()
isn't @vmalloc https://github.com/vmalloc's version much cleaner? if
the problem is the naming of raise_for_status, how about renaming it
(with a deprecating alias)? e.g.:
result = requests.get("http://api.server.com/path").require_ok().json()
(or whatever you want) or how about a new option to all of the request
methods:
result = requests.get("http://api.server.com/path", require_ok=True).json()
btw, i currently always monkey-patch Response.raise_for_status to do
exactly this. ugly!
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/kennethreitz/requests/pull/1894#issuecomment-218886371

+1 for this change, it would be very handy!
myjson = requests.get("http://api.server.com/path").raise_for_status().json() is exactly what you want to write when dealing with REST API returning JSON objects.
I don't see the point of raise_for_status() renaming, because making it a chainable method does not change its semantics.

@guillp I agree. It's funny to me how this library willy-nilly broke compatibility time and again (e.g. with json turning to json()) but adding a single return value where there was none is a major backward compatibility issue...

Hi @vmalloc.
Please try to extend charity to the maintainers of OSS projects you use. We have not willy-nilly broken compatibility. The move from json to json() was accompanied by a major version bump and a substantial internal refactoring. Other breakages of compatibility have been considered to be bugs and errors and treated as such.
Ironically, I was previously inclined to look much more warmly on this proposal than my fellow maintainers. Now I'm not. The answer is no.


Rendering 3d globes and terrain

This is a major missing item in the WGPU and Rust space IMO. There are lots of complexities here, such as

Hierarchical LOD, with LOD selection based on screen space error
Handling cracks
Mapping textures at different LOD than the underlying geometry
Floating point artifacts due to the scale of the earth
Zooming logic: do you zoom based on altitude or zoom to cursor? How do you make zoom to cursor not janky?
Combining multiple layers of both imagery and terrain

Cesium appears to be state of the art in a lot of these areas, so referencing their code would probably be wise.
This is a good article I found on how they handle floating point artifacts. The book 3D Engine Design for Virtual Globes appears to have a lot of good info as well. Also, Unreal's Nanite LOD tech might be a good reference to improve significantly on state of the art for terrain rendering.

Related to #2


Expose special : commands to Jupyter

I love this project and it already works surprisingly well.
So far only :vars is documented to be used in a Jupyter notebook. However, all the other special :-commands also work and are quite useful. But :vars is the only command shown in the Jupyter notebook, all the others only print to the terminal.
It would be great if the other commands would also show their output in the Jupyter notebook, e.g., to see the output of :help.

Ooops. Thanks for pointing that out. They used to work from Jupyter. Originally evcxr ran everything in one process and the jupyter kernel redirected its own stdout and stderr to pipes that it then read from and passed back to Jupyter. I switched to using a separate process shortly before releasing evcxr, which mean that things printed by the parent process no longer go back to Jupyter. I'll look at fixing it.


[BUG]: node containers crash with "invalid memory address or nil pointer dreference" (CSI driver for PowerStore 2.3.0)

Bug Description
When deploying the CSI driver for PowerStore using Helm onto a freshly installed OpenShift 4.10 setup the controller pods start up ok but the node pods keep crashing showing "panic: runtime error: invalid memory address or nil pointer dreference".
Logs
Log from the driver container inside one of the node pods. All node pods show a similar output:
Read CSI_LOG_FORMAT from log configuration file, format: json Read CSI_LOG_LEVEL from log configuration file, level: debug {"level":"info","msg":"updating array info","time":"2022-07-29T12:21:51.176039557Z"} panic: runtime error: invalid memory address or nil pointer dereference [signal SIGSEGV: segmentation violation code=0x1 addr=0x88 pc=0x13bc2aa] goroutine 1 [running]: github.com/dell/csi-powerstore/pkg/array.(*PowerStoreArray).GetIP(...) /root/redhattest/csi-powerstore/pkg/array/array.go:138 github.com/dell/csi-powerstore/pkg/node.(*Service).updateNodeID(0xc000110dc0) /root/redhattest/csi-powerstore/pkg/node/node.go:1123 +0x2aa github.com/dell/csi-powerstore/pkg/node.(*Service).Init(0xc000110dc0) /root/redhattest/csi-powerstore/pkg/node/node.go:96 +0xe7 main.main() /root/redhattest/csi-powerstore/cmd/csi-powerstore/main.go:124 +0x55c
Screenshots

Additional Environment Information
CSI config yamls.zip
Steps to Reproduce

Install OpenShift 4.10 using Centos 7.7
Install CSI driver 2.3.0 for PowerStore using Helm
See the node pods fail to start

Expected Behavior
node pods becoming "started" with a functional CSI driver.
CSM Driver(s)
CSI driver for PowerStore 2.3.0
Installation Type
Helm3
Container Storage Modules Enabled
None.
Container Orchestrator
v1.23.5+012e945
Operating System
Operating System: CentOS Linux 7 (Core)        CPE OS Name: cpe:/o:centos:centos:7             Kernel: Linux 3.10.0-1160.71.1.el7.x86_64

The issue has been fixed in CSI PowerStore v2.4.


See this

Everyone read the latest version readme file. Work on the designated chapters and commit it in YOUR branch. Then when all of you are done we will merge it in the main branch.

k


Uncaught TypeError: Cannot set property '[Note ID: 8d85110f9423d2339772eb49e6ef71f4]' of null

[Enter steps to reproduce below:]

Create new note. Save note
Send to Evernote

Atom Version: 1.5.3
System: Mac OS X 10.11.3
Thrown From: ever-notedown package, v0.2.18
Stack Trace
Uncaught TypeError: Cannot set property '[Note ID: 8d85110f9423d2339772eb49e6ef71f4]' of null
At /Users/manjari/.atom/packages/ever-notedown/lib/note-helper.coffee:95

TypeError: Cannot set property '[Note ID: 8d85110f9423d2339772eb49e6ef71f4]' of null
    at NoteIndex.addnote (/Users/manjari/.atom/packages/ever-notedown/lib/note-helper.coffee:95:5)
    at StorageManager.exports.StorageManager.StorageManager.addNote (/Users/manjari/.atom/packages/ever-notedown/lib/storage-manager.coffee:197:27)
    at Object.module.exports.saveNewNote (/Users/manjari/.atom/packages/ever-notedown/lib/ever-notedown.coffee:1384:34)
    at atom-text-editor.window.evnd.newNoteDisposables.(anonymous function).atom.commands.add.core:save (/Users/manjari/.atom/packages/ever-notedown/lib/ever-notedown.coffee:1332:12)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:260:29)
    at /Applications/Atom.app/Contents/Resources/app.asar/src/command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:536:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:359:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeydown (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:97:36)
    at HTMLDocument.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/window-event-handler.js:3:61)

Commands
     -7:54.7.0 core:save (atom-text-editor.editor.is-focused)
     -7:48 application:add-project-folder (div.panels-item)
     -6:55.2.0 ever-notedown:toggle (div.panels-item)
  5x -6:40.3.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -6:10.6.0 ever-notedown:new-note (atom-text-editor.editor.mini.is-focused)
     -5:36.7.0 ever-notedown:open-config (div.panels-item)
 18x -4:51.1.0 core:move-right (atom-text-editor#ever-notedown.grammars.editor.mini.is-focused)
     -3:54.9.0 ever-notedown:file-to-evernote (atom-text-editor.editor.is-focused)
     -3:45.8.0 ever-notedown:new-note (atom-text-editor.editor.is-focused)
 12x -3:23.9.0 core:backspace (atom-text-editor.editor.mini.is-focused)
     -2:31 autocomplete-plus:confirm (atom-text-editor.editor.is-focused.autocomplete-active)
 12x -2:10.6.0 core:backspace (atom-text-editor.editor.is-focused)
     -1:24.7.0 core:copy (atom-text-editor.editor.is-focused)
     -1:20.4.0 core:backspace (atom-text-editor.editor.is-focused)
     -1:19.9.0 core:paste (atom-text-editor.editor.is-focused)
     -1:08.5.0 core:save (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "markdown-preview-plus",
      "atom-sync"
    ]
  },
  "ever-notedown": {
    "theme": "Atom"
  }
}

Installed Packages
# User
ever-notedown, v0.2.18
language-ipynb, v1.3.0
language-latex, v0.6.1
language-matlab, v0.2.0
language-pfm, v0.82.0
latex, v0.29.0
nvatom, v0.9.3
pigments, v0.24.0
run-in-terminal, v0.6.0

# Dev
No dev packages

I have the same problem.
I cannot use it any more :(

I had the same issue -- check your index.json in .atom/evnd/ for errors. Mine had a formatting issue, so the note index wasn't ever loaded properly, and so no more could be saved to it either. Not sure the source of this issue though.

@aspin Thanks a lot for that info!
@abruneau @mnarayan I'm sorry to hear that you are having trouble with the package... I think the problem is probably caused by formatting problems in index.json as @aspin mentioned. But that file typically contains many personal info so I cannot offer to check it for you. Could you possibly check it with some kind of JSON validator? Or maybe check for special characters and such. Thanks!

Hi @cs150bf,
I came across the same issue and probably found root cause and solution.
Steps to reproduce

Install your package
Read the doc with help ( ;) )
Create folder with config and symlink it to ~/.atom/evnd
Try to create note -> ERROR

When I installed plugin before creating folder with ENVD storage, Atom asked me to create directory ~/.atom/evnd. Then when I created ENVD storage and tried to symlink it, it didn't work and I didn't notice it. There were index json in ~/.atom/evnd with content 'null'.
When I removed the directory ~/.atom/evnd and symlinked it to 'path/with/my/proper/index/json/git/repo' it worked as a charm!
I can see that more people have the same problem (last 3 issues). Maybe you can add the preparation part of documentation to the plugin description?
By the way, the plugin looks very promising! I haven't chance to use it a lot so far, but it seems that I will. It is exactly what I was looking for, it has many good features and looks nice! Great job! Thank you!

Thank you,
I've been able to to solve it by reseting index.json

@aspin Thanks
@trzeciakp Thanks "When I removed the directory ~/.atom/evnd and symlinked it to 'path/with/my/proper/index/json/git/repo' it worked as a charm!"
@abruneau Thanks
Used the above guidance, I deleted the contents of index.json, replaced with {}. Clean Up. And it does indeed work like a charm!!!
Thank you all,
The path to my storage is /Users/moses/OneDrive/Ever_Notedown/, I did not symlink, added said path to Settings,
@cs150bf Again, thank you for the great work,


Add auto-approve and merge via dependabot

The PR adds auto-approve and auto-merge of PRs of dependabot itself.

Coverage remained the same at 38.88% when pulling 90eebc5a7fda43eb2633f7e5b23ee46a516a78f5 on feature/auto-approve-merge-dependabot into 0072b245a2fd625197249e7fc85a7b2db60b9eeb on main.


Add possibility to register custom fields in error response

Our backend in error responses adds custom fields. For example:
{
  "data": {
    ...
  },
  "errors": [
    {
      "statusCode": 401,
      "error": "Unauthorized",
      "message": "JWT Token not found",
      "timeThrown": "2017-06-06T11:58:54.093Z",
      "data": null
    }
  ]
}

Now, it's impossible to reach statusCode in this situation. It's possible to add feature to the lib that could add fields to the mapped error response?

Closed via https://github.com/apollographql/apollo-android/pull/506


what's the scale between UTM odom and real distance?

When I use utm_odometry_node, it tells 0.823 that I moved,but actually I moved about 9 meters.So,what's the scale between UTM odom and real distance?

The scale varies over each UTM zone, but it is close to 1:1, and it should be within 0.1% of actual within entire UTM zones (see https://en.wikipedia.org/wiki/Universal_Transverse_Mercator_coordinate_system#UTM_zone or http://geokov.com/education/utm.aspx). I suspect you are seeing some other problem with that large of a difference.


bug: cordova: cordova 4.3.0 bug in config.xml while the name has special characters (&)

I have no idea where to post this for apache cordova to see, but in the latest version of cordova (4.3.0) installed via sudo npm install -g cordova having a name in the config.xml that includes a & character written as & gives a very weird error that says that config.xml and info.plist are missing.
to duplicate this:
npm install -g cordova
write config.xml name as "Foo & Bar"
ionic platform rm ios
ionic platform add ios
error!
P.S. only appearing for iOS platform

Greetings @gabrielmuresan!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Cordova specific issues can be found here.
https://issues.apache.org/jira/browse/CB-8250?jql=text ~ "special characters"
Closing this since it is related to cordova and not ionic


 Download QT online installer:
wget https://d13lb3tujbc8s0.cloudfront.net/onlineinstallers/qt-unified-linux-x64-4.6.1-online.run
2. Make it executable:
chmod +x qt-unified-linux-x64-4.6.1-online.run
3. Start Installer:
./qt-unified-linux-x64-4.6.1-online.run
WSL logs:
weston.log
stderr.log
WSL dumps:
No response
Expected behavior:
QT Installer and QT Creator should work without graphic glitch
Actual behavior:
If I start the QT Installer it looks like this

Same with QT Creator

If I move the window or move the mouse over the window the glitch disappears sometimes. The same happens if I make the window fullscreen.
The glitch seems also happen with other, non QT related, apps like Tilix, Audacity and Wireshark

It seems like I fixed this by myself!
I had compiled a custom kernel (version <IP_ADDRESS>) with CAN bus support in the past and loaded this by the .wslconfig. If I do not load this kernel, it works without graphic glitches!
So if you have the same problem, check if you have loaded an old custom kernel. You can check if a custom kernel is loaded by comparing the kernel version from wsl --version (in powershell) and uname -r (in linux wsl shell). If the version in uname -r is lower, a old custom kernel is loaded.
(I also activated the old kernel again and it continuous working without graphic glitches. I did a wsl --shutdown, maybe simply this fixed the glitch. I sadly cannot reproduce this anymore)

I took https://github.com/microsoft/wslg/issues/1130#issuecomment-1813058202 his advice and just updated my nvidia drivers. I then ran wsl --shutdown and rebooted wsl and everything operates fine now. No more application tearing (taring?).


Cannot import nltk after installation

The package is installed in /Library/Python/2.7/lib/python/site-packages but I cannot import nltk. Please advice

Do you use virtualenv? How do you run python? I'm guessing you are on MacOS, is that right?

Thanks for responding. I don't use virtualenv. Yes I am on Mac.

Thanks for the responses. How do you invoke python?
Could you try starting up a session and post the output of the following:
import sys
sys.path

@mmuppidi did you resolve your installation issues?
On a Mac, you can either try Anaconda:
conda install nltk

Or if you would like to install nltk such that the user can use it without messy setup, you could try:
pip install --user nltk


Improvements on the factorial end-to-end test

See #507

Allow tests to return Int32 value
Allow the use of basic asserts in the tests

oh... I merged another PR in this one ... this makes it harder for review ... sorry for that


Failure to read binary archives with large vectors

Hi,
I've been using cereal (great library!) in my project for quite some time (and this one too).  Recently, I upgraded from 1.1.2 to 1.2.2.  However, now I'm experiencing a particular bug.  I write out a BinaryArchive with this code.   However, when I read it in, I consistently get an exception of the form:
Exception : [Failed to read 4936 bytes from input stream! Read 2468]

The numbers are sometimes different, but the key is that it always reads exactly half as many bytes as it expects.  This is suspicious since the bug only occurs when the archive is written from the  largeIndex branch of this conditional.  So, when I'm dealing with int32_t everything seems to work, but with int64_t I consistently see this bug when attempting to read the archive.  Any idea what might be going on here?

Digging in a little more, it looks like the exception occurs when reading in the last of the structures written to the archive (i.e., the serialization at fault occurs here and the de-serialization at fault occurs here).  If I simply don't de-serialize the last structure (and instead fill it in with dummy values etc., the exception goes away and the program executes normally.  Is there a reason that the last thing in the stream might be truncated?  I know that I can't expect the entire archive to be written until the buffer is flushed, but the BinaryArchive is in its own scope, and the destructor for the ifstream will be called when the function exits, and so the stream should always be flushed completely.

I was able to track down this bug, and I think it's safe to blame my code rather than cereal.  However, it did highlight an interesting design decision that I think should be made more explicit in the documentation (and potentially in the exception that is thrown).  The issue is that the archive doesn't seem to record, itself, the size of the object that was written.  My problem was akin to something like the following:
program 1:
serialize vector<uint32_t> to file "data.bin"

program2:
de-serialize vector<uint64_t> from file "data.bin"

Now, clearly, one should try to de-serialize the same thing he serialized.  However, the fact that cereal uses the type of the container being passed in to the de-serialization function to determine how much data to read (and doesn't seem to store much information about the type that was placed in the file in the first place) strikes me as a bit error prone.
Would it be worth it to have the archive also contain information about thing that was serialized into the file (i.e. this was a vector of uint32_t) so that if you try to de-serialize into a container of the wrong type you get a descriptive error message rather than just trying to read too many bytes?  I assume this could be even more nefarious in the other direction (i.e. I have written more data to the archive than I read).  In that case, you could start de-serializing the next thing in the archive from the wrong position in the file and you could be incorrectly interpreting that data and filling up your de-serialized structure with junk.  Having some type of check that what you are attempting to read into is the same as what is written would help prevent these types of errors.


Replicator: support atomic types in inserts/updates

https://docs.scylladb.com/using-scylla/cdc/cdc-basic-operations/#updates
https://docs.scylladb.com/using-scylla/cdc/cdc-basic-operations/#inserts

Fixed by #28


Datagrid leaks JS memory and DOM elements.

Describe the bug
I have a grid in which constantly add new data, update existing one and remove data.
I see that when a row is deleted (data are deleted), row's DOM nodes are not released and the JS memory constantly grows with every added row.

How to reproduce
https://stackblitz.com/edit/clarity-v3-datagrid-memory-leak
Expected behavior
Datagrid not to leak memory and DOM elements.
Versions
App

Angular: 9
Clarity: 3.0.0

Device:
Type: MacBook
OS: iOS
Browser Chrome
Version latest

Second version without *ngFor through the columns in the rows:
https://clarity-v3-datagrid-memory-leak-v2.stackblitz.io/
Strange that comparing both approaches, it looks the first one with second *ngFor without track by is better, the memory leaks are bigger with the second approach

Regarding memory leaks on the heap, is it true it that it grew by 5MB over the course of about 2 minutes?
Worthy of deeper inspection within Clarity datagrid:
Sometimes observables are not cleared or fully unsubscribed. Moreover dangling references are left around to these heavy DOM objects that accumulate.  After 2 minutes, one would expect GC to have been triggered, but it has not. We should see a triangular-shaped heap memory usage over time.
For the use case of recreating the grid's data set periodically, is there any manual need from the developer's code to trigger something else or meddle with ChangeDetectorRef?

Thank for for the report and the reproduction. In Stackblitz, it does seems like a problem with growing memory but in order to better triage and understand the situation it is best to run a local instance to minimize variables. Stackblitz itself modifies the runtime in a way that might be causing the memory leaks in that environment.
Findings
First I started looking at the Stackblitz demo, and it does show increasing memory over time with lots of detached DOM nodes. However, this is hard to debug because the heap size at the start is well over 400 MB, making it very inneficient already.
Second, I tried to reduce the complexity of the demo by removing the unnecessary NgFor loops, suspecting it might cause some issues to have nested iterators. This did not yield significant differences.
Third, I created the reproduction above using the same code logic with a fresh Angular CLI project outside of Stackblitz. This removes the variable of Stackblitz and also lets the memory stay low enough to more easily profile. I ran the sample for 30 minutes, taking heap samples along the way and saw it staying steady between 13.4-13.6 MB (meaning it went up and down between those values, showing it would decrease depending on GC). Also, there is not the increasing detached DOM nodes that I saw in Stackblitz. In short, this means that locally, the memory leak doesn't appear and this particular reproduction appears to be specific to Stackblitz.
Screenshot of 10 minute run of the profiler with memory holding steady and cleaning up at regular, stable intervals.

These findings are correllated with another team member's triage and testing. So based on the reproduction, I cannot conclude that Clarity is the reason that the initial report is growing in memory with detached DOM nodes.
Updated Reproduction
I created https://github.com/gnomeontherun/clarity-issue-4712/ which has the initial reproduction in a standalone Angular app, as well as an updated version that is more isolated by not having ngFor or rxjs. When I run either of these versions locally, I do not see a memory leak. Memory grows and stabilizes, even after 30 minutes of running.
Next steps
Can you please investigate my repository above and see if you get different results? Otherwise, it would seem there is something else at work here and Clarity itself is not holding onto DOM nodes in this scenario. A review of the code also does not immediately surface a place where that might be the case. There might be more details or context that could illuminate the issue further, but I need an updated reproduction to demonstrate it in action.

Also as a follow up, the demo also does not clean up the interval subscription manually so if you are working with the sample code, it will cause a memory leak if the app does a hot module reload to restart the demo. Therefore, you need to either unsubscribe to clean up, or only profile the initial load and not touch the code.

@gnomeontherun Thanks that looking this issue.
I modified a little bit your example to become more near to my use cases and verify that it works. It works without memory leaks according to me.  I see the memory leaks in our app, but it uses older version of Clarity and Angular (Angular 8.2.14, Clarity 2.4.3). I modified your example to use Angular 8.2.14 and Clarity 2.4.3 and I started to see memory leaks again. I just noticed also that the Stackblitz example which I posted here also uses  older version of Clarity and Angular (Clarity 3.0.0, Angular 9.0.4, "@angular/cli": "~7.0.2",  "@angular/compiler-cli": "~7.0.0").
Do you know whether there are some memory leak fixes related to the datagrid in the latest versions of the Clarity library?

Quick update, Clarity v1 and v2 do have a problem with not cleaning up templates, which was fixed during an update to Angular 9 support for Clarity v3 (https://github.com/vmware/clarity/blob/v3/src/clr-angular/data/datagrid/wrapped-row.ts#L29) @Shijir is working on backporting the necessary changes for the next release.

Resolved by #4732


index.html

Let's create it.

Job gh:yegor256/jpeek#22 assigned to @yegor256, please go ahead (policy).

Bug was reported: +15 points just awarded to @yegor256, total is +4340.

@rultor release, tag is 0.4

@rultor release, tag is 0.4

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.4

@yegor256 Done! FYI, the full log is here (took me 12min)

Job gh:yegor256/jpeek#22 is now out of scope.

Order was successfully finished: +30 points just awarded to @yegor256, total is +4425.


An easier way to browse configurations

Problem example:
Today, let's say I want to configure the Item Drop Chance for every quest item.
I have to: open "Full Configuration" > scroll to "Drop item Groups" > browse and find the item I'll edit > clicking edit will open it on the same page, forcing me to go through every step again for the next item. The same applies to other configurations.
Someone has suggested open edits in a modal but I think it'll be great to open it as a new tab, then you can open various tabs and even compare them side by side. A new tab will also be simpler to implement because we'll only alter the button to open it in a new tab, no need to alter the edit page itself. Even enable the middle mouse click as a new tab and leaving everything else as it was would be a great solution.
I'd like us to discuss this feature, if you had any other ideas that would be amazing.

just to clarify, I'll start working on this issue


Add an event commit mechanism to the event registry

This is a work-in-progress to add a checkout/commit mechanism for events in the event state registry.
The mechanism enables "checking out" copies of event objects, which can subsequently changed and "committed" back to the event state registry.
This also adds an observer mechanism, so that observers can be notified about events being committed.  Observers can be used to trigger imminent state dumps or notifications to API clients that have active notification channels open.
Closes #52

CodeCov complains about missing coverage in two changed lines of zino.zino.main.  I don't consider adding coverage to the main method part of this PR, it is already being worked on in #117

Some twitch-twitch from bare integers floating around as timeouts. See also inline comment.

Timeouts? What are you referring to?

As mentioned in this morning's standup meeting, I'm not quite sure how this will play into the get_or_create_event() method.  Most task plugins will use this to get an event to modify when changes are detected.  However, currently, this returns an Event object directly from the events list, which means the task plugin is modifying the "live" object and later committing it.
In all probability, this method should also be changed to use the checkout mechanism for events.  Which, as commented during the meeting, should probably also be documented very specifically in its docstring:  The returned object must be committed for any changes to take effect.

Some twitch-twitch from bare integers floating around as timeouts. See also inline comment.

Timeouts? What are you referring to?

def reschedule_dump_state_on_commit(event_id: int, max_wait=timedelta(seconds=10)): and scheduler.add_job(func=state.state.dump_state_to_log, trigger="interval", seconds=30). The "10" is a timeout, yes? Dunno about the other one. The first one screams to be a config variable eventually.

def reschedule_dump_state_on_commit(event_id: int, max_wait=timedelta(seconds=10)): and scheduler.add_job(func=state.state.dump_state_to_log, trigger="interval", seconds=30). The "10" is a timeout, yes? Dunno about the other one. The first one screams to be a config variable eventually.

That's not how I think of it, @hmpf, but I guess you could construe it that way, if you want.  It is a limit that serves two purposes: If the next scheduled dump job is more than 10 seconds away, we want to reschedule it to run in exactly 10 seconds.  If it's 10 seconds or less until the next job, skip the rescheduling and let things run their course.
In any case, the value is in fact hardcoded in the legacy Tcl code, so I didn't bother to do much about it here, other than accepting an optional argument for it.  It could also be a global constant, of course.

scheduler.add_job(func=state.state.dump_state_to_log, trigger="interval", seconds=30).

And, as discussed elsewhere, this job should be removed. It was put there for debugging purposes before we had a working dump-do-disk mechanism.  The state can get quite large, too large for logs, really.

This isn't part of this PR anyway. And, as discussed elsewhere, this job should be removed. It was put there for debugging purposes before we had a working dump-to-disk mechanism. The state can get quite large, too large for logs, really.

Added #140 to clean up this.


Doc cleanups

Fixes (mostly minor) typos and formatting hiccups to address #134

Hmmm... forgot to sign this one commit, but seems to still be problematic even after reverting it and committing back (with a signature).  Any ideas?

Seems like this DCO signing is buggy -- that last commit (b31d179) is definitely signed, but still marked as not signed (?)


animation nonfluency problem

Hey, I found there was a bit of animation nonfluency when the route was jumping, I was using the latest version and using Qualcomm's 835 CPU phone, was I setting the problem, or was there something that wasn't optimized?Didn't anyone find such a problem, or was it my cell phone problem?
Your Environment

software
version

react-navigation
1.0.0-beta.15

react-native
0.49.3

node
8.1.2

npm
5.0.3

hi @Herryjoeson, unfortunately, your description is not detailed enough for us to see what problem you have. Please feel free to reopen the issue, but make sure add some video or screenshots which clearly show the problem. Thanks.


AgentStat.jvmGcDetailed.gcNewTime error

What version of pinpoint are you using?
master/v1.7.3/v1.7.2 etc.
Describe the bug
From /getAgentStat/jvmGcDetailed.pinpoint, the  gcNewTime always equals to gcNewCount
Expected behavior
gcNewTime & gcNewCount should be different.

Fixed in PR naver/pinpoint#4415.

@yjqg6666 Thanks for reporting and the fix!


Are you able to generate handwriting for special symbols such as @,✓, +, $ and £?

Hello,
I've been exploring different handwriting generators and would like to ask a question about this one.
May I please ask if your model, given the data you used to train it, is able to generate special symbols such as @, +, $ and £, ✓, as well as uppercase Q and Z?
Many thanks! :)

Hi @Grezego,
I hope you're well.
Ah OK, I understand. Thank you very much for your reply, I really appreciate it. :)
I have an additional question I would like to ask please. In your "ReadMe" section, I see that you have used the On-Line dataset. The xml files contain the handwriting data. Do you know how to convert these into a .jpg/.png such that I can visualise the handwriting (i.e. convert the On-Line dataset into "Offline")?
Thank you and have a good weekend!

@AIMLAPP so it seems there is a ready off-line dataset available at "data/lineImages-all.tar.gz". According to this page where you can download and example from it, it contains .tif images that could be converted into .jpg/.png if needed. Hope this helps :)


how to set only custom/user headers as native header

I have to set a custom header in the kafka message my kafka cluster natively supports headers (1.x.x). I am currently using springCloudVersion=Finchley.RELEASE.
When I am setting property
default:
    producer:
      headerMode: none

None of the headers are coming in the output.
On the other hand, If I am setting
default:
    producer:
      headerMode: headers

Many headers including contentType and spring_json_header_types are coming in the header which greatly affects the throughput. Kafka being a language/framework agnostic message delivery mechanism, I feel, spring should provide a way to only include user-provided headers. Is there any work around to get only user-set headers to the kafka topic while suppressing all the spring cloud related headers.

Please use Stack Overflow to ask questions; GitHub issues are for reporting bugs or asking for new features.
You can provide a custom header mapper in the binder configuration.

spring.cloud.stream.kafka.binder.headerMapperBeanName

The bean name of a KafkaHeaderMapper used for mapping spring-messaging headers to and from Kafka headers. Use this, for example, if you wish to customize the trusted packages in a DefaultKafkaHeaderMapper that uses JSON deserialization for the headers.

Default: none.

Implement your own KafkaHeaderMapper and add it as a @Bean and configure the binder to use it.

Thanks for your time and answer Gary. When I opened this github issue, my intention was to discuss a possibility of a 5th HeaderMode (Besides Raw, None, Headers, embeddedHeaders) Something like user-provided or custom.  Apologies for hastily copy pasting and not clarifying my intention.

Resolved https://stackoverflow.com/questions/58615450/how-to-send-only-custom-header-using-spring-cloud-stream/58615736#58615736


Update upgrade.md

See above +  adding a note that old public link to files will break

+1. Tweaked with JB's comments

+1


Difference from no-extend-native unclear

Hello! Some documentation to clarify how this plugin differs from no-extend-native[0] would be most welcome! Thank you!
[0] http://eslint.org/docs/rules/no-extend-native

Hey. Thank you for bringing this up.
ESLint's no-extend-native checks that code is not modifying a prototype.
no-use-extend-native checks that code is not using a a non-native prototype.
These two rules pair well together, but it's not a this rule vs. that rule thing. I'm definitely open to making this clearer in the documentation. I'm open to a PR if you or others have suggestions.
Thanks again!


Test on v17.03.0-ce

Test on latest docker release.

Codecov Report

Merging #677 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master   #677      +/-   ##
==========================================
+ Coverage     63.96%    64%   +0.03%
- Complexity      591    592       +1
==========================================
  Files           134    134
  Lines          2589   2589
  Branches        318    318
==========================================
+ Hits           1656   1657       +1
+ Misses          786    785       -1
  Partials        147    147

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 ab1e007...6f18534. Read the comment docs.

Perhaps we can just drop testing on 1.8.3 which was released 2015-10-12? My motivation for not testing on too many versions is that travis is really flaky and often fails to pull images or, less frequently, install docker. The fewer versions we test on, the less likely a PR is to fail.

Makes sense.

Want to rebase on master? I saw a failure that https://github.com/spotify/docker-client/pull/681 fixes.

This has already been rebased on master with #681. So maybe we do need #666.


Attempting to edit topic names in deactivated streams throws errors

Attempting to edit the name of a topic in a deactivated stream throws the following Blueslip errors:
Error: Failed to update messages undefined is not an object (evaluating 'stream_data.get_sub_by_id(event.stream_id).name')
 at update_messages /static/js/message_events.js:164
 at get_events_success /static/js/server_events.js:140
 at success /static/js/server_events.js:219
 at wrapped_success /static/js/channel.js:93
 at fire jquery/dist/jquery.js:3496
 at fireWith jquery/dist/jquery.js:3626
 at done jquery/dist/jquery.js:9786
 at jquery/dist/jquery.js:10047

Error: Failed to handle get_events success Failed to update messages undefined is not an object (evaluating 'stream_data.get_sub_by_id(event.stream_id).name')
 at construct [native code]:
 at BlueslipError /static/js/blueslip.js:178
 at blueslip_error /static/js/blueslip.js:245
 at get_events_success /static/js/server_events.js:142
 at success /static/js/server_events.js:219
 at wrapped_success /static/js/channel.js:93
 at fire jquery/dist/jquery.js:3496
 at fireWith jquery/dist/jquery.js:3626
 at done jquery/dist/jquery.js:9786
 at jquery/dist/jquery.js:10047

Image

Reproduction

Create a stream as any user.
Deactivate the stream as an admin user.
Filter for the deactivated stream in a search as any member of the stream (i.e. search "stream:name").
Attempt to edit the name of a topic in the stream.

@zulipbot add "bug" "help wanted"

@zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot claim


Support for Activity ID, property names in quotes

I'm submitting a ...  (check one with "x")
[] bug report => search github for a similar issue or PR before submitting
[X] feature request
[] support request
[] general question

Current behavior (how does the issue manifest):
The mapper syntaxes when using the previous activity outputs are like $activity[ACTIVITYID].some_property. Similar case for the $property and $env
Expected behavior:
We want to request for support of $activity["ACTIVITYID"].some_property. It is useful when we want to use some special characters in the ACTIVITYID, similar case for the $property and $env
Minimal steps to reproduce the problem (not required if feature enhancement):
Try to map to previous activity output and observe the expression
What is the motivation / use case for changing the behavior?
The properties can have special characters like "."(dot) in the name, For general javascript like parsers the variables with the special characters should use bracket notation with the special characters variable in quotes
Ex: $property["some.property"]
Please tell us about your environment (Operating system, docker version, browser & web ui version, etc):
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.X.X
Additional information you deem important (e.g. issue happens only occasionally):

One thing to note, Flogo is simply using JSON as a mechanism to persist the apps & flows. It does not have to be JSON, it could be YAML if a new serializer were to be implemented. That said, it does not mean that mapping expressions need to be parsable via a js parsers.
Also, the $activity[id].field is enclosed in quotes:
{
  "type": "assign",
  "value": "$activity[id].input",
  "mapTo": "fieldName"
}

Hence, enclosing id in quotes as well would either force the user to escape the quotes, or a single quote be used...
@fm-tibco thoughts?

@mellistibco : Can both ways be supported? For better validations, someone tend to use parsers (js today something else tomorrow). We either handle values with/out quotes or should let plug-in mechanism to resolve such values.

From a developer (user) point of view, supporting both is kinda tricky as it implies the developer to know the syntax. It also brings up interesting questions as, can I mix the two modes, or how should not using quotes be interpreted if there are quotes in the document (and vice versa). I think we should support one mode, and to be fair I think it should be the one without quotes.


Issue with Lyrics Display

Hi @ClementBeal
I hope this message finds you well. I wanted to reach out regarding the audio metadata plugin for Dart that you've developed. Firstly, I want to express my appreciation for your work on this plugin, as it has been incredibly helpful.
However, I've encountered an issue with the plugin related to the display of lyrics. Specifically, when attempting to access lyrics in Spanish, the text appears as gibberish characters. This suggests a potential problem with character encoding or support for different character sets.

I'm reaching out to bring this to your attention in the hopes that we can find a resolution.
Thank you for your attention to this matter, and I look forward to your response.
Best regards, DomingoMG

Hi @DomingoMG ,
Thank you for the issue!
Could you pass me the audio file that generates this output? I just tried to check the lyrics from my library and I can't see any issue with language using utf-8 character like spanish or french.
Regards,

Yes, of course, I attach the audio.
Adele - Easy On Me.zip
Thanks!

Thank you! I'll try to fix it before Monday. I'll ping you when it's done.

Hello @ClementBeal, I hope you are well.
I was just stopping by to see if you were able to find out anything, thanks

Hi
I have been lazy and forgot the issue, sorry. It's done now with the #27 .
Will be published this week


Make U^pre and U^hcom Kan again!

I was enlightened by the comments given by Andrew Pitts. I believe it also works for U^hcom.

How about ECOM for empty composition types?

I still cannot figure out the elimination rule for this type, but maybe we never need to do so? I'm going to add this type once I'm able to work on RedPRL again.

Closed by #671.


WIP - Add operator seccomp profile

What type of PR is this?
/kind feature
What this PR does / why we need it:
Creates a seccomp profile that is specific to the seccomp-operator.
Which issue(s) this PR fixes:
Fixes #29
Special notes for your reviewer:
For a seccomp profile to be applied it first need to exist on disk, therefore such a task needs to be carried out within the init container.
Does this PR introduce a user-facing change?

Operator now runs under a specific seccomp profile.

/test pull-seccomp-operator-test-e2e

@hasheddan @saschagrunert what are your thoughts about merging this? if we do it now, as we develop features we may experience some problems locally which would mean we would have to amend the profile as we go.
Otherwise I am also happy to just wait until we are feature complete to reassess the profile.

/retest

@saschagrunert I put the profile in "complain mode", the issue may be related to a specific syscall being blocked. I will try to reproduce the error to amend it.

Ah okay there is an issue in the DaemonSet:
The DaemonSet "seccomp-operator" is invalid: spec.template.spec.containers[0].volumeMounts[0].name: Not found: "host-operator-vol"
    suite_test.go:121:

Ah okay there is an issue in the DaemonSet:
The DaemonSet "seccomp-operator" is invalid: spec.template.spec.containers[0].volumeMounts[0].name: Not found: "host-operator-vol"
    suite_test.go:121:

Thank you for the heads up. There were also a few syscalls missing that I added now.

@hasheddan @saschagrunert This profile worked fine on Ubuntu 18.04 and Ubuntu 20.04. PTAL
Are we happy to go ahead with this in "enforce mode"? As we develop new features this may break the build, in which case we may need to check the missing syscalls:
cat /var/log/syslog | grep audit | grep seccomp-operator
I wonder whether is worth adding something to that effect on the E2E tests.

@hasheddan @saschagrunert This profile worked fine on Ubuntu 18.04 and Ubuntu 20.04. PTAL
Are we happy to go ahead with this in "enforce mode"? As we develop new features this may break the build, in which case we may need to check the missing syscalls:
cat /var/log/syslog | grep audit | grep seccomp-operator
I wonder whether is worth adding something to that effect on the E2E tests.

I think that's fine, we have the CI to get early feedback and adjust the profile accordingly during development :+1:

Ah snap this needs another rebase :-/

rebased: PTAL


[Question] How to add new sheet to existing excel file. Add as a different Sheet Tab

            Excel::load(storage_path('excel/Erez_Summary_Datasheet.xls'), function($excel) use($patient_export) {
                            $excel->sheet( "Erez_Summary_Datasheet", function($sheet) use($patient_export) {
                            $sheet->fromArray($patient_export);
                });
                })->store('xls', storage_path('excel'));

Currently not supported.

Any update on this anyone has been done ...the exact I need. Can anybody help me on this?

@raghbendra2015 i need help on this issue i am trying to add extra sheet in existing excel file button every time it's show "Illegal offset type in isset or empty". can you help me?

Has this now been added?  I am facing the exact same issue.  Thank you.

I did in a different way. Ex. In my case three sheets in single excel file. two are static sheet and the third one is dynamic. so in excel generation time create three sheets. not an included third one in existing excel file.

@sujalpatel022 That sounds perfect.  Could you paste the code here?

` Excel::create('ABCReport_' . date("m/d/Y"), function ($excel) use ($data) {
// Start Static Sheet
$excel->sheet('A', function ($sheet) use ($data) {
$sheet->loadView('admin.abcReports.a'); // Sheet one data load using view A
});
            $excel->sheet('B', function ($sheet) use ($data) {
                $sheet->loadView('admin.abcReports.b');  // Sheet one data load using view B
            });

            $excel->sheet('C', function ($sheet) use ($data) {
                $sheet->loadView('admin.abcReports.c');  // Sheet one data load using view C
            });

// End Static Sheet
// Dynamic sheet added
            $excel->sheet('ABCReport', function ($sheet) use ($data) {
                $sheet->loadView('admin.abcReportTemplate')->withdata($data);
            });

// End Dynamic Sheet
        })->export($type);`

@sujalpatel022 Thank you.

welcome.  happy to help you.


fix: resolving relative version stream values files  when specifying a different --helmfile flag and location

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rawlingsj
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [rawlingsj]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Feature/image asset compressions

What kind of change does this PR introduce?
Feature

What is the current behavior?
This is a feature, requested in #36 .

What is the new behavior (if this is a feature change)?
Running adr compress will compress the images located at <adr-base>/assets/
Previously compressed images' names will be recorded as cache and will be skipped in the compression process.

Other information:
The image compression is handled using the sharp

I wanted to write a test for it as well, but I'm not very familiar with the stack that have been used in the library.
As a first step, to generate random images that can be compressed during the test, I installed the @lisnote/image-generator library (it depends on the sharp library as well, which I see as a plus)

Thanks. Benchmark is not a easy works


useMaxSendAmount fails with CELO

Current behavior
Attempting to send the max amount of CELO fails to send.
Does not occur with cUSD, cEUR or cREAL.
Desired behavior
Max send CELO should be produce a valid transaction.
Steps to Reproduce

Navigate to Send
Select CELO
Select Amount in crypto
Select Max
Attempt to send
Observe transaction reverted - example reverted transaction

Versions
Valora 1.24.x and likely earlier.

Hey @MuckT we have verified the above issue on Latest Android Internal Release build and Latest iOS Test Flight Release build V 1.42.0 and observed that issue is not replicated
Observation : We are able to send Max CELO with crypto and without crypto(without swapping places) and transaction are successful
Attachments : celo.mp4
Celo crypto.mp4

#2263 - Related Max Send issue.

Downgrading priority as it is not 100% reproducible.


zsh-syntax-highlighting.zsh:31: invalid alias '=' encountered while printing aliases

When I source the .zshrc, I see below issue:

z-sy-h undefines/redefines aliases so they wouldn't be effective as it's parsed. Aliases that begin with = can't be defined by zsh's API. Thus, the warning you see.
Probably the same issue as #523 except that you use zsh>=5.3.1.
Whoever uses such aliases should get zsh upstream to provide a forward compatible syntax for creating them (or at least help with the aforementioned conversion to autoload).
Excuse brevity.

@danielshahaf I use ‘=‘ as an alias, and it works just fine except this warning?

The warning is generated by zsh itself, not by z-sy-h:
% Src/zsh -fc 'aliases[=]=; alias -L'
zsh:1: invalid alias '=' encountered while printing aliases

With my z-sy-h hat, I'm closing this issue as it's not a bug in z-sy-h (notwithstanding that using autoload will work around this).
With my zsh upstream hat, I suggest that you ask whoever maintains your zshrc to ask zsh-users@ to consider allowing = to be the LHS of an alias.  It should be simple enough to support that, and unlike other aliases of the form =foo, it would not collide with the syntax of the (on-by-default) EQUALS option.
Ideally, whoever approaches zsh-users@ with this idea should be willing to implement it (in C) if it's accepted.


Fix typo and correct proper usage

改正错别字，  对应现版本代码更正了现在的用例代码

Coverage remained the same at 73.564% when pulling 4000ba6b3f769b9be5d461f71cc8acbbdda3cedf on dbFlower:master into 294a6f68733fa1c0abd32e8e587d49eda1f87995 on jabbany:master.


How to use client to load server's native data to server's database ?

I'm using docker for servers, but it's not easy to use client to deal with data remotely. I want to let client to command server to load their native fs data to their database. How to do it just like client cmd

Hello. I have a similar problem.
I tried this way (by analogy with insert from csv):
cat file_name.bin | clickhouse-client --password=my_password --database=some_db --query="INSERT INTO some_table FORMAT Native"
but I got an error:
Code: 210. DB::NetException: Connection reset by peer, while writing to socket (<IP_ADDRESS>:9000)

@yjhmelody @chaelub could you tell about your usecases in more detail?
why your client have direct access to server files in the first place?

In my case there isn't communications between client and server, it's all about the clickhouse-client tool on the same machine where clickhouse-server installed.
Could you explain how to insert data from Native format to a CH table?

@blinkov hi! because of our powerful server and poor net , it's better let data store in server. Also, so many datum shared by server. If client can set a path to server's data, sometime very convenient.
Any way, I have solved my problem in some way.

just checked the logs. So it seems like there is some casting troubles.
`2019.04.26 17:01:07.446018 [ 24 ] {b4a9165c-a36e-484b-868e-8f439aca3403}  executeQuery: Code: 368, e.displayText() = DB::Exception: Bad cast from type DB::ColumnVector to DB::ColumnVector (version <IP_ADDRESS> (official build)) (from <IP_ADDRESS>:51964) (in query: INSERT INTO test.conversions FORMAT Native), Stack trace:

/usr/bin/clickhouse-server(StackTrace::StackTrace()+0x16) [0x73fd046]
/usr/bin/clickhouse-server(std::enable_if<is_reference_v<DB::ColumnVector const&>, DB::ColumnVector const&>::type typeid_cast<DB::ColumnVector const&, DB::IColumn const>(DB::IColumn const&)+0x12d) [0x47ed6fd]
/usr/bin/clickhouse-server(DB::DataTypeNumberBase::serializeBinaryBulk(DB::IColumn const&, DB::WriteBuffer&, unsigned long, unsigned long) const+0x1e) [0x68114fe]
/usr/bin/clickhouse-server(DB::IMergedBlockOutputStream::writeData(std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&, DB::IDataType const&, DB::IColumn const&, std::set<std::__cxx11::basic_string<char, std::char_traits, std::allocator >, std::less<std::__cxx11::basic_string<char, std::char_traits, std::allocator > >, std::allocator<std::__cxx11::basic_string<char, std::char_traits, std::allocator > > >&, bool, std::shared_ptrDB::IDataType::SerializeBinaryBulkState&)+0x259) [0x6bd8f99]
/usr/bin/clickhouse-server(DB::MergedBlockOutputStream::writeImpl(DB::Block const&, DB::PODArray<unsigned long, 4096ul, Allocator, 15ul, 16ul> const*)+0x104e) [0x6bda68e]
/usr/bin/clickhouse-server(DB::MergeTreeDataWriter::writeTempPart(DB::BlockWithPartition&)+0x10b9) [0x6b9c119]
/usr/bin/clickhouse-server(DB::MergeTreeBlockOutputStream::write(DB::Block const&)+0xf1) [0x6b47c71]
/usr/bin/clickhouse-server(DB::PushingToViewsBlockOutputStream::write(DB::Block const&)+0x43) [0x6d4ffb3]
/usr/bin/clickhouse-server(DB::SquashingBlockOutputStream::write(DB::Block const&)+0x454) [0x6d59854]
/usr/bin/clickhouse-server(DB::AddingDefaultBlockOutputStream::write(DB::Block const&)+0x8c) [0x6cecebc]
/usr/bin/clickhouse-server(DB::CountingBlockOutputStream::write(DB::Block const&)+0x2c) [0x6d07eac]
/usr/bin/clickhouse-server(DB::TCPHandler::receiveData()+0x9f) [0x3789ddf]
/usr/bin/clickhouse-server(DB::TCPHandler::receivePacket()+0x135) [0x378aff5]
/usr/bin/clickhouse-server(DB::TCPHandler::readData(DB::Settings const&)+0x1c3) [0x378b493]
/usr/bin/clickhouse-server(DB::TCPHandler::processInsertQuery(DB::Settings const&)+0x23b) [0x378b93b]
/usr/bin/clickhouse-server(DB::TCPHandler::runImpl()+0x7f9) [0x378c2b9]
/usr/bin/clickhouse-server(DB::TCPHandler::run()+0x2b) [0x378c85b]
/usr/bin/clickhouse-server(Poco::Net::TCPServerConnection::start()+0xf) [0x7500d6f]
/usr/bin/clickhouse-server(Poco::Net::TCPServerDispatcher::run()+0xe9) [0x75014a9]
/usr/bin/clickhouse-server(Poco::PooledThread::run()+0x81) [0x75b45e1]
/usr/bin/clickhouse-server(Poco::ThreadImpl::runnableEntry(void*)+0x38) [0x75b07a8]
/usr/bin/clickhouse-server() [0xb27522f]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x76db) [0x7fa57aa996db]
/lib/x86_64-linux-gnu/libc.so.6(clone+0x3f) [0x7fa57a01888f]

2019.04.26 17:01:07.446232 [ 24 ] {}  ServerErrorHandler: Code: 99, e.displayText() = DB::Exception: Unknown packet 96 from client, Stack trace:

/usr/bin/clickhouse-server(StackTrace::StackTrace()+0x16) [0x73fd046]
/usr/bin/clickhouse-server(DB::Exception::Exception(std::__cxx11::basic_string<char, std::char_traits, std::allocator > const&, int)+0x22) [0x377e942]
/usr/bin/clickhouse-server(DB::TCPHandler::receivePacket()+0x250) [0x378b110]
/usr/bin/clickhouse-server(DB::TCPHandler::runImpl()+0x5eb) [0x378c0ab]
/usr/bin/clickhouse-server(DB::TCPHandler::run()+0x2b) [0x378c85b]
/usr/bin/clickhouse-server(Poco::Net::TCPServerConnection::start()+0xf) [0x7500d6f]
/usr/bin/clickhouse-server(Poco::Net::TCPServerDispatcher::run()+0xe9) [0x75014a9]
/usr/bin/clickhouse-server(Poco::PooledThread::run()+0x81) [0x75b45e1]
/usr/bin/clickhouse-server(Poco::ThreadImpl::runnableEntry(void*)+0x38) [0x75b07a8]
/usr/bin/clickhouse-server() [0xb27522f]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x76db) [0x7fa57aa996db]
/lib/x86_64-linux-gnu/libc.so.6(clone+0x3f) [0x7fa57a01888f]
(version <IP_ADDRESS> (official build))`

#数据导出
clickhouse-client --host="host25080" --max_threads="10" --format_csv_delimiter="|" --query="select * from ares.logentity_test2_all FORMAT CSV" > logentity_test2_all.csv
#数据导入
clickhouse-client --host="host25080" --format_csv_delimiter="|" --query="insert into ares.logentity_20181128 FORMAT CSV" </data/clickhouse/10000.txt

@blinkov In my case there isn't communications between client and server, it's all about the clickhouse-client tool on the same machine where clickhouse-server installed.
Could you explain how to insert data from Native format to a CH table?

Native is on-wire format. On-disk format used to store data on server differs from Native.
The only way to manipulate the data in disk format is to do it on PART / PARTITION level. With ATTACH / DETACH and few other commands.


useAuthStateChange fails in @supabase/supabase-js v2+

Bug report
useAuthStateChange fails in @supabase/supabase-js v2+
Description / Observed Behavior
The useAuthStateChange hook throws a n.unsubscribe is not a function error on v2 versions of supabase/supabase-js.
Expected Behavior
Hook does not throw page breaking error and returns documented values.
Repro Steps / Code Example
https://codesandbox.io/s/sharp-snow-vdzivd?file=/src/App.tsx:311-312
Additional Context
react-supabase version - 0.2.0
@supabase/supabase-js - 2.2.3
The authStateChanged listener has since been updated to return a subscription object with the unsubscribe function.
I guess this is a question of compatibility since this library is only rated for v1 but are there plans for v2 support and or are PRs updating the existing hooks welcome?

No plans to support v2 right now. Other projects currently keeping me busy.


No need to use branding for config files

Drop the Solid. Just Cache. It's cleaner.

@dhh, while the recent changes related to handling multiple databases in Rails are welcome, I have a few concerns and suggestions that I think might be worth considering:
Migration Path for Existing Apps: For apps already using a single database setup, transitioning to the new multi-database approach may pose some challenges. While apps with existing databases will continue to work, the "Rails way" moving forward will involve using a separate cache database. For users migrating from a single database to this new setup, what's the recommended approach? Their current schema would still contain the legacy solid_cache tables, which would now need to be removed or moved. Guidance on handling this migration path would be helpful.
Naming and Organization of Schema Files: The introduction of cache_schema.rb as a separate schema file for the cache database seems like an improvement. However, it also introduces a naming inconsistency, as the primary schema remains schema.rb. This could have been an opportunity to better organize schema files by placing them within a dedicated db/schema directory, using names that match their respective databases, such as db/schema/cache.rb and db/schema/primary.rb. This would make it clearer which schema file corresponds to which database. In a single-database setup, the schema file would still be db/schema.rb.
Managing Migrations for Multiple Databases: The new approach also raises concerns about handling future upgrades and migrations. Will updates for new tables in the cache database be handled through traditional migrations? If so, these migrations currently reside in the same db/migrate directory as the primary database migrations. This could lead to confusion when running db:migrate unless explicitly targeted for a specific database. A potential solution would be to introduce separate directories, like db/migrate/cache for all cache database-specific migrations and db/migrate/primary for primary database-specific migrations. This separation would make it clear which migrations belong to which database. In a single-database setup, the migrations would still reside in db/migrate.
I believe these changes would provide better clarity and organization, especially when I see SolidQueue getting the same treatment! What do you think?

Handled by https://github.com/rails/solid_cache/pull/222


Can you install the slack integration so we can get release notifications?

What would you like to be added:
Github slack integration
https://github.com/integrations/slack
Why is this needed:
So we can get release notifications in our slack channels
Current output
/github subscribe awslabs/amazon-eks-ami

I would love to! We have to go through an internal approval process for this integration; I can't give you an ETA, unfortunately -- but I'll update here as things progress.

Sounds good, thanks @cartermckinnon

No need to install any apps to Slack / GitHub, you can also use /feed subscribe https://github.com/awslabs/amazon-eks-ami/releases.atom to get notification about new releases. See also https://slack.com/help/articles/218688467-Add-RSS-feeds-to-Slack .

Added, thanks!


Fix for Issue#209

Changes done in reporter.py  for Issue https://github.com/smartsdk/ngsi-timeseries-api/issues/209

Hi @daminichopra !
If I may give a suggestion here, before working on reporter.py, how about starting with a test to cover the case mentioned in the issue (one entity with a valid type, id but just null values for all its attributes). A good candidate file for the test may be this one: https://github.com/smartsdk/ngsi-timeseries-api/blob/master/src/reporter/tests/test_incomplete_entities.py. Write the test, watch it fail and only then come and edit this file, until you make your test pass :)

Hi @c0c0n3 , i have Incorporated  your comments on PR, kindly review and merge the PR.

Hi @c0c0n3 , i have incorporated review comments on PR. kindly review and close this PR. Thanks :)

hi @daminichopra, I've just added a couple of comments about possible improvements---see above review. If you have time to review/validate/or amend my suggestions and then provide an implementation it'd be fantabulous. On the other hand, if there isn't more time you can dedicate to this PR, just let me know. I think as it stands now it's pretty good anyway so we might just as well merge it in if you agree.

Hie @c0c0n3 , I have updated test case for valid attribute and empty attributes as well. Please close this PR and raise new issue for remaining comments. :)

Hi @daminichopra thank you sooo much for this.

raise new issue for remaining [comments]

Excellent idea! I'll do that. There's just a couple of things still hanging though as far as I can tell.

Your branch needs rebasing since it's behind master GH won't let me merge it.
I have question about the test case you added, see my comment here. Sorry if I sound pedantic, but we need to be sure this test case proves the entity gets inserted. This is critical for us since it could break some of our deployments...

Thanks a stack!!!

@daminichopra I reckon this PR is ready for merging, if you could please rebase onto master. Thanks so much!

@daminichopra can you have a look at the failing tests? Perhaps something in your latest commit? One thing I could pick up is this line in _filter_empty_entities
value = payload[j]['value']

should probably be
value = payload[j].get('value')

as we talked about a while back. Not sure this is what makes the tests unhappy though, like I said if you could please have a look, many thanks!

@daminichopra it looks like this PR is ready for merging, if you give me the green light I'll happily squash it into master.
We should also reimplement properly the tests in test_incomplete_entities.py that you made pass by checking there's no attrs. These tests were meant to check what happens to attrs with no value, so after your change we'll need to update the tests to use an entity with two attrs instead of one---the first attr will have to have a value, the other will be the x attr as it was in the tests before your change.
Since this is just donkey work I'd rather merge your valuable contribution and then do the rest separately along with the few minor improvements we discussed earlier. Like you suggested I'll open a new issue for the boring little bits :-)

Hi @c0c0n3 , Please close this PR if it looks good. Also, Please create a new issue for other minor improvments. Thank you :)

@daminichopra merged :-) kudos to you!


New upgrade generates VS errors

Nuget latest distribution :
Assembly 'TikaOnDotNet,  uses 'IKVM.OpenJDK.Text  Version=8.1.5717.0 which has a higher version than referenced assembly 'IKVM.OpenJDK.Text, Version=7.2.4630.5
Also
Assembly 'TikaOnDotNet.TextExtraction, Version=<IP_ADDRESS> uses 'IKVM.OpenJDK.Core, Version=8.1.5717.0 which has a higher version than referenced assembly 'IKVM.OpenJDK.Core, Version=7.2.4630.5.

I confirmed this and fixed it and will be publishing a nugget shortly. Thanks for the report!

Nuget 1.14.1 is released.


Drop assisted key derivation support

The assisted key derivation functionality is needed to support cards without KeyAgreement.ALG_EC_SVDP_DH_PLAIN_XY. This was implemented because no card actually implemented that and we will wanted to do BIP-32.
However, it is a workaround and it is not strictly necessary anymore. Removing it would allow removing quite a lot of code (making the applet smaller) and test cases.

I agree this is not needed anymore. Latest cards on the market implement KeyAgreement.ALG_EC_SVDP_DH_PLAIN_XY, and a smaller applet will make a quicker upload of the applet either at the factory or through the client side.

Ok, before adding much new code I am going to remove the old cruft then

Closed by e78397c19894d7931fc2ac5c407be441a9fdc32f


created the clone

Amazon Clone #4244

Close #4244

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

@MohitGupta121 Do you want all the images to be uploaded from my drive links

@MohitGupta121 Do you want all the images to be uploaded from my drive links

Drive or directly in github, you got link use that as image source.

@AbhirajBose you not need to add into codespaces simply add them into GitHub comment then you get link automatically

@MohitGupta121 Just check it

still there

@MohitGupta121 Will you please guide me with the steps of how to add the images through drive link.I am not getting how to add the drive link

@MohitGupta121 Please check it once more


Idea of support for RawQuery using the classical class based flow of Django and DRF

When dealing with GeoDjango the need for RawQuery is important as PostGIS queries are inherently difficult to write with the classical Django Queryset pattern.
When trying to bind everything together I have found a simple way to do it, and I share the recipe here because I might not be the only one having this need and there is no obvious documentation about it. At least this issue can last as a documentation.
But I also think it is a good use case to extend this package (which is very convenient by the way). I have the feeling it might be easy to draw a pull request from here. See my stack overflow request for peer reviewing for more details.
With the current version, the best way I have found to perform it, is:
class Location(Model):
    geom = models.PointField()

class CircleSerializer(GeoFeatureModelSerializer):

    id = serializers.IntegerField(required=True)
    circle = GeometrySerializerMethodField()

    def get_circle(self, obj):
        return GEOSGeometry(obj.circle)

    class Meta:
        model = Location
        geo_field = "circle"
        fields = ('id',)

class CircleViewSet(ListAPIView):

    queryset = Location.objects.raw("""
    SELECT id, ST_Buffer(L.geom::Geography, 800)::Geometry AS circle
    FROM location AS L;
    """)
    serializer_class = CircleSerializer

And it works as expected, and it is clean and concise.
But maybe something cleaver can be done to withdraw the need of explicitly cast the GIS column which in this scenario is returned as a str holding the geometry binary representation. Passing it to the GEOS geometry constructor solve the problem.
Maybe creating the field object, it can detect if the QuerySet is actually a RawQuery and add the extra GEOS casting on the fly. That would makes the code even simpler. Something like:
class Location(Model):
    geom = models.PointField()

class CircleSerializer(GeoFeatureModelSerializer):

    id = serializers.IntegerField(required=True)
    circle = GeometrySerializerField()       # <-- Some magic that perhaps could land here
    #circle = RawGeometrySerializerField()   # <-- Or into a new object

    class Meta:
        model = Location
        geo_field = "circle"
        fields = ('id',)

class CircleViewSet(ListAPIView):

    queryset = Location.objects.raw("""
    SELECT id, ST_Buffer(L.geom::Geography, 800)::Geometry AS circle
    FROM location AS L;
    """)
    serializer_class = CircleSerializer

Indeed it must not break the flow for other process. Another kind of isolation could be a totally new field object.
I open the discussion, let me know if it makes sense.
Best regards,
Jean

what about using the query as django model manager?


[Bug]: Object type not displayed when exporting package body and common synonyms

ODC version
odc423
OB version

What happened?

What did you expect to happen?
Display object types
How can we reproduce it (as minimally and precisely as possible)?
Export package body and common synonyms
Anything else we need to know?
No response
Cloud
No response

pass


Patchcore: the value of k in nearest neighbor search

Thanks to your efforts,
I've noticed in the code of patchcore,
doing k nearest neighbor search,
https://github.com/rvorias/ind_knn_ad/blob/5510f0f4bc1fc9688e57d20023985087f90f0f4d/indad/models.py#L286
the value of k is set to 3,
can I ask how to determine the value of k?
can I set k=1?

Thanks for your question. I believe I got 3 from the original paper.
The best k depends on your dataset and your validation performance.
You can indeed set k=1.

Thanks to your explanation,
I'm confused because two other implementations
anomalib and hcw-00
they both set the value of k to 9.
Fig.4 top in the paper, I'm not sure the 'neighborhood size' here is refer to the local neighborhood aggregation part (the Avgpool operation) or the k neighbor search operation...

I've always interpreted the local neighborhood aggregation part as the knn.


VS 2022 Pylance crashes all the time

This issue has been moved from a ticket on Developer Community.

[severity:It's more difficult to complete my work] [regression]
This just started happening today.
All I have to do is create a new file called 'foo.py`, and then I see this error message:
Task Failed - Activating language client Pylance: Operation did not complete successfully because the file contains a virus or potentially unwanted software.  This appears in the Output window under Language Client activation failures.
I can still edit and debug Python code.  However, there is no outlining or completion suggestions, etc, so my work is much harder.
I am going back to VS 2019 Professional, even though it does not officially support Python 3.9.
I will also try downgrading VS 2022 and see if the problem goes away.  If it does, then I'll update this issue.
Details on what I have:
Microsoft Visual Studio Community 2022
Version 17.2.4
VisualStudio.17.Release/17.2.4+32602.215
Microsoft .NET Framework
Version 4.8.04084

Installed Version: Community

Visual C++ 2022   00482-90000-00000-AA239
Microsoft Visual C++ 2022

ASP.NET and Web Tools 2019   17.2.392.13739
ASP.NET and Web Tools 2019

Azure App Service Tools v3.0.0   17.2.392.13739
Azure App Service Tools v3.0.0

C# Tools   4.2.0-4.22281.5+8d3180e5f00d42f0f0295165f756f368f0cbfa44
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Cookiecutter   17.0.22089.1
Provides tools for finding, instantiating and customizing templates in cookiecutter format.

Markdown Editor v2   2.0.111
A full featured Markdown editor with live preview and syntax highlighting. Supports GitHub flavored Markdown.

Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines

NuGet Package Manager   6.2.1
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/

Python - Profiling support   17.0.22089.1
Profiling support for Python projects.

Python with Pylance   17.0.22089.1
Provides IntelliSense, projects, templates, debugging, interactive windows, and other support for Python developers.

Test Adapter for Boost.Test   1.0
Enables Visual Studio's testing tools with unit tests written for Boost.Test.  The use terms and Third Party Notices are available in the extension installation directory.

Test Adapter for Google Test   1.0
Enables Visual Studio's testing tools with unit tests written for Google Test.  The use terms and Third Party Notices are available in the extension installation directory.

Visual Basic Tools   4.2.0-4.22281.5+8d3180e5f00d42f0f0295165f756f368f0cbfa44
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.

Visual Studio IntelliCode   2.2
AI-assisted development for Visual Studio.

Original Comments
Feedback Bot on 6/19/2022, 08:06 PM:
(private comment, text removed)
Michael Rolle on 6/19/2022, 10:17 PM:
(private comment, text removed)
Bill Schnurr [MSFT] on 6/29/2022, 09:43 AM:
(private comment, text removed)

Original Solutions
(no solutions)

Going to close this out. Please take a look at the response in the original post and reopen the ticket if the issue still persists, Thanks😊


bluetooth-udev doesn't seem to execute at all

Running rpi-audio-receiver on a Pi Zero 2 for a headless application and thought I would test to see if its bluetooth chip is acceptable on its own or whether I'd need a dongle.
I uncommented both ifconfig lines in bluetooth-udev, but the wifi doesn't toggle when I connect/disconnect a bluetooth device. To troubleshoot this I tried adding the device-added and device-removed aplay commands to their respective if sections. The sounds don't work either, so it seems as if the bluetooth-udev isn't running at all for some reason.
I also tried uncommenting the first line of the file, which had no effect so I re-commented it.
Finally I tried manually running the ifconfig lines on their own in a terminal, and got an error: "SIOCSIFFLAGS: Operation not permitted". The ifconfig lines run fine in a terminal if preceded by sudo, and adding sudo to the appropriate lines inside bluetooth-udev has no effect.
It seems like maybe bluetooth-udev might be failing to do anything because it is not running with appropriate permissions? Thanks for any help.
Full content of my bluetooth-udev (hopefully without errors, it's manually typed out here as I don't have an easy way to transfer the file to the PC I'm making this issue report from):
#!/bin/bash
if [[ ! $NAME =~ ^/"([0-9A-F]{2}[:-]){5}(0-9A-F{2})"$ ]]; then exit 0; fi
action=$(expr "$ACTION" : "([a-zA-Z]+).*")
if [ "$action" = "add" ]; then
bluetoothctl discoverable off
# disconnect wifi to prevent dropouts
ifconfig wlan0 down &
aplay -q /usr/local/share/sounds/__custom/device-added.wav
fi
if [ "$action" = "remove" ]; then
# reenable wifi
ifconfig wlan0 up &
bluetoothctl discoverable on
aplay -q /usr/local/share/sounds/__custom/device-removed.wav
fi

I also noticed that commands in /usr/local/bin/bluetooth-udev didn't execute. I want to run mpc stop in case my MPD server is currently playing something, but playback didn't stop upon connecting a bluetooth device.
For me removing/commenting the NAME condition did the trick:
https://github.com/nicokaiser/rpi-audio-receiver/blob/2b7ef54b21c4764a53e90e7cf1eedf1dbe48e7a2/install-bluetooth.sh#L60
Now all commands get executed upon a bluetooth connect.
I'm using the RP4 built-in bluetooth (yes, I'm aware of the downsides).


[core] The hidden prop does not always visually hide elements

Im using the hidden prop of Container to show only the current Tab and to hide the rest (copied from the Tabs example)
was working fine until i upgraded to version 4.9.9 from version 4.9.1 (could be higher version but i think 4.9.1)

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

<Container hidden={tabIndex != 1}> </Container>
The Container is not hidden even though tabIndex = 1
Expected Behavior 🤔
Container should not be visible
Steps to Reproduce 🕹
<Container hidden={tabIndex != 1}>sdfsdfsdfds</Container>
https://codesandbox.io/s/empty-browser-n8fwh?fontsize=14&hidenavigation=1&theme=dark

Works on 4.9.1 *

Steps:

upgrade to version 4.9.9
use a Container with hidden prop as True

Tech
Version

Material-UI
v4.9.9

React
v16.12.0

Chrome

I think this is a general issue with overriding display. Setting a default display means that the user agent styling for [hidden] will be overridden.

@shtengel What's the use case for using hidden, why not changing the styles?
I can count the usage of display: 'block', in 50 other Material-UI components.

What's the use case for using hidden, why not changing the styles?

hidden has different semantics. It's also a web standard so supporting that isn't really something that needs to be questioned.

hidden has different semantics. It's also a web standard so supporting that isn't really something that needs to be questioned.

https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/hidden
We had a pass discussion on the topic in #15178, luckily, we solved the issue without even trying :).
However, we also have these components to consider: StepIcon, Slider, InputLabel, CircularProgress, CardMedia, CardActionArea, and SvgIcon that ignore the hidden attribute.

@oliviertassinari what do you mean changing the style?
i am using hidden to hide and show Tabs
if you go to https://codesandbox.io/s/empty-browser-n8fwh?fontsize=14&hidenavigation=1&theme=dark
and change the material-core version to 4.9.1 you'll see that the hidden feature works fine.
its somewhere between 4.9.1 and 4.9.9

@shtengel Any reason for using hidden
<Container hidden={{A ? true : undefined }} />

over
<Container style={{ display: A ? 'none' : undefined }} />

or
{A ? <Container /> : null}

@oliviertassinari Not a major reason, just according to here: https://reactjs.org/docs/faq-styling.html
CSS classes are generally better for performance than inline styles so i try to avoid using style wherever i can. if Container is using styles then it won't make much difference

In https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/hidden, we have:

Appropriate use cases for hidden include:

Content that isn't yet relevant but may be needed later
Content that was previously needed but is not any longer
Content that is reused by other parts of the page in a template-like fashion
Creating an offscreen canvas as a drawing buffer

Inappropriate use cases include:

Hiding panels in a tabbed dialog box
Hiding content in one presentation while intending it to be visible in others

I think that we should ignore 2. 3. and 4. I think that we should also ignore the use case to only set a node invisible (encourage inline style or CSS instead).
However, I have a concern with 1. and especially with React suspense: https://github.com/oliviertassinari/react-swipeable-views/issues/453#issuecomment-417939459.
On the other hand, I haven't seen any progress of the hidden prop in React, developers could always add an extra DOM node, what's the overhead of supporting hidden? Out of this tradeoff equation, I'm leading toward recognizing the problem (it exists) but ignoring it (we have more to lose). @eps1lon What do you think?

@oliviertassinari im using the hidden prop as suggested here https://material-ui.com/components/tabs/
but instead of <Typography> i decided to use <Container>

@shtengel Oh, ok, perfect, so it's simply a leftover from https://github.com/mui-org/material-ui/commit/0369bfc885316a7db89eeb375dcf939f2017ddeb. I will update it, so you don't see this pattern anymore.

@oliviertassinari Ok :) Thanks for the quick reply!
so to use style instead of hidden?

so to use style instead of hidden?

@shtengel Use
<Container style={{ display: A ? 'none' : undefined }} /> or  {A ? <Container /> : null} unless you have a good reason to use hidden.

Ran into this gotcha today. First, the hidden attribute has nothing to do with MUI or React, but is a default HTML attribute from the web standard. Second, the hidden attribute is overridden if a display: ... style is applied to the same element. It doesn't matter if these styles are inline or coming from CSS (the latter surprises me the most since I'd expect the hidden attribute to take precedent over any global CSS). As a result, the hidden attribute is "ignored" by any MUI component that happens to set the display attribute.
You can try this on any of the examples from the official docs. For example, playing around with CodeSandbox I see hidden work on Card, CardContent and Typography, but ignored on CardActionArea, CardMedia, CardActions and Button.
So yea, considering the above it's best to avoid the hidden attribute as its behavior is different on a per-component basis. Moreover, if in any next version of MUI a component is updated to use display internally, it'll change the semantics of hidden for that component and break stuff in existing code bases.

Ran into this gotcha today. First, the hidden attribute has nothing to do with MUI or React, but is a default HTML attribute from the web standard. Second, the hidden attribute is overridden if a display: ... style is applied to the same element. It doesn't matter if these styles are inline or coming from CSS (the latter surprises me the most since I'd expect the hidden attribute to take precedent over any global CSS). As a result, the hidden attribute is "ignored" by any MUI component that happens to set the display attribute.
You can try this on any of the examples from the official docs. For example, playing around with CodeSandbox I see hidden work on Card, CardContent and Typography, but ignored on CardActionArea, CardMedia, CardActions and Button.
So yea, considering the above it's best to avoid the hidden attribute as its behavior is different on a per-component basis. Moreover, if in any next version of MUI a component is updated to use display internally, it'll change the semantics of hidden for that component and break stuff in existing code bases.

Re-opening since https://github.com/mui-org/material-ui/pull/20648 did not address the actual issue

@topazbarziv Regarding #28460, do you have thoughts on https://github.com/mui-org/material-ui/issues/20452#issuecomment-610328925?

@oliviertassinari A little bit strange way, but good enough. Thank you.


fix: handle comments at the end of an action

Got a plan with the line 0.0013:   (T_2_0) [39.0000] ;; InputAct in the generated plan.
The regex raises an AttributeError since no match were found because of the comment at the end of the line.

Can you try now?
Bye

It seems to be working now. Thank you.


chore(main): release cyclistsofmsn-client 1.1.1

:robot: I have created a release beep boop
1.1.1 (2024-10-31)
Bug Fixes

bump @emotion/react from 11.11.4 to 11.13.0 (d5260df)
bump @emotion/react from 11.13.0 to 11.13.3 (25f5efd)
bump @emotion/styled from 11.11.5 to 11.13.0 (a5e4b4b)
bump @fontsource/roboto from 5.0.12 to 5.0.13 (0c8986f)
bump @fontsource/roboto from 5.0.13 to 5.0.14 (f5413eb)
bump @fontsource/roboto from 5.0.14 to 5.1.0 (e5d4f70)
bump @mui/icons-material from 5.15.14 to 5.15.16 (fed2c6b)
bump @mui/icons-material from 5.15.16 to 5.15.19 (4c72174)
bump @mui/icons-material from 5.15.19 to 5.15.21 (c001991)
bump @mui/icons-material from 5.15.21 to 5.16.6 (42f28fe)
bump @mui/icons-material from 5.16.6 to 5.16.7 (67562aa)
bump @mui/material from 5.15.14 to 5.15.16 (ed242e7)
bump @mui/material from 5.15.16 to 5.15.19 (1de73ed)
bump @mui/material from 5.15.19 to 5.15.21 (7e59148)
bump @mui/material from 5.15.21 to 5.16.6 (3558169)
bump @mui/material from 5.16.6 to 5.16.7 (7e38059)
bump @mui/x-data-grid from 7.1.0 to 7.3.1 (5650651)
bump @mui/x-data-grid from 7.12.0 to 7.15.0 (05d0a24)
bump @mui/x-data-grid from 7.15.0 to 7.18.0 (96dd45e)
bump @mui/x-data-grid from 7.3.1 to 7.6.1 (f79b7ab)
bump @mui/x-data-grid from 7.6.1 to 7.8.0 (16ddebf)
bump @mui/x-data-grid from 7.8.0 to 7.12.0 (51b8b31)
bump date-fns from 3.6.0 to 4.1.0 (65d3ffe)
bump react and @types/react (691ede9)
bump react-dom and @types/react-dom (7bd4402)
bump react-hook-form from 7.51.2 to 7.51.3 (c5066a9)
bump react-hook-form from 7.51.3 to 7.51.4 (4c11e88)
bump react-hook-form from 7.51.4 to 7.51.5 (932a8a1)
bump react-hook-form from 7.51.5 to 7.52.0 (b392267)
bump react-hook-form from 7.52.0 to 7.52.1 (5705849)
bump react-hook-form from 7.52.1 to 7.53.0 (4846ce1)
bump react-router-dom from 6.22.3 to 6.23.0 (d9229e7)
bump react-router-dom from 6.23.0 to 6.23.1 (9969987)
bump react-router-dom from 6.23.1 to 6.24.0 (79666e6)
bump react-router-dom from 6.24.0 to 6.26.0 (8924887)
bump react-router-dom from 6.26.0 to 6.26.1 (2a24b26)
bump react-router-dom from 6.26.1 to 6.26.2 (5e56ea0)
bump superagent from 8.1.2 to 9.0.2 (9b4fc3a)
bump superagent from 9.0.2 to 10.1.0 (90ba091)
bump zod from 3.22.4 to 3.23.6 (f6eda8c)
bump zod from 3.23.6 to 3.23.8 (8f906db)
eslint 9 support (bbd9e95)
migrate to tanstack v5 (3f12e2d)
properly handle missing current posts (ea3f1a1)
update to mui 6 (d59d43c)
use new husky script reqs (dbef35e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

cyclistsofmsn-client-v1.1.1
:sunflower:


Capitalized status

Issue:

This issue was assigned to someone else and has been resolved. Please pick issues that are unassigned or that has the label "Always Open". Thank you!


Could you create the extension for the Webstorm IDE?

Could you create the extension for the Webstorm IDE?

Thanks for the recommendation, @MrFiregore! I'll pass this along to our product team.
I'll close this issue (as it's not vscode related) but updates will be posted to our product newsletter and changelog