no default reduce due to non-trivial cinit error

Hi, i got this error while running Experiments/BuildExperiments.py
Can you please help me out with this problem?
(TestEnv) /home/SafeBound# python Experiments/BuildExperiments.py
concurrent.futures.process._RemoteTraceback:
"""
Traceback (most recent call last):
File "/opt/conda/envs/TestEnv/lib/python3.10/concurrent/futures/process.py", line 208, in _sendback_result
result_queue.put(_ResultItem(work_id, result=result,
File "/opt/conda/envs/TestEnv/lib/python3.10/multiprocessing/queues.py", line 372, in put
obj = _ForkingPickler.dumps(obj)
File "/opt/conda/envs/TestEnv/lib/python3.10/multiprocessing/reduction.py", line 51, in dumps
cls(buf, protocol).dump(obj)
File "stringsource", line 2, in pybloomfilter.BloomFilter.reduce_cython
TypeError: no default reduce due to non-trivial cinit
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/SafeBound/Experiments/BuildExperiments.py", line 37, in
time, size = build_stats_object(method='SafeBound',
File "/home/SafeBound/Source/ExperimentUtils/BuildUtils.py", line 459, in build_stats_object
return build_safe_bound(benchmark, parameters, outputFile)
File "/home/SafeBound/Source/ExperimentUtils/BuildUtils.py", line 261, in build_safe_bound
stats = SafeBound(tables, tableNames, joinColumns,
File "SafeBoundUtils.pyx", line 246, in SafeBoundUtils.SafeBound.init
self.finishOneStatsBuildProcess()
File "SafeBoundUtils.pyx", line 254, in SafeBoundUtils.SafeBound.finishOneStatsBuildProcess
self.tableStatsDict[tableName].addFilterColStats(filterCol, statsFuture.result())
File "/opt/conda/envs/TestEnv/lib/python3.10/concurrent/futures/_base.py", line 439, in result
return self.__get_result()
File "/opt/conda/envs/TestEnv/lib/python3.10/concurrent/futures/_base.py", line 391, in __get_result
raise self._exception
TypeError: no default reduce due to non-trivial cinit

Hey,
Thanks for trying to use the package! I am on vacation this week, but I
would be happy to help next Monday. I recall there being a bit of
trickiness with that pybloomfilter package, specifically it has trouble
pickling. There’s a version of the package ( I think it’s a public branch)
on GitHub that solves this, but I can’t remember quite what it is right
now. I’ll update the readme when I get back.
Thanks,
Kyle
On Sun, Jul 2, 2023 at 1:27 PM leesangoh @.***> wrote:

Hi, i got this error while running Experiments/BuildExperiments.py
Can you please help me out with this problem?
(TestEnv) /home/SafeBound# python Experiments/BuildExperiments.py
concurrent.futures.process._RemoteTraceback:
"""
Traceback (most recent call last):
File
"/opt/conda/envs/TestEnv/lib/python3.10/concurrent/futures/process.py",
line 208, in _sendback_result
result_queue.put(_ResultItem(work_id, result=result,
File "/opt/conda/envs/TestEnv/lib/python3.10/multiprocessing/queues.py",
line 372, in put
obj = _ForkingPickler.dumps(obj)
File
"/opt/conda/envs/TestEnv/lib/python3.10/multiprocessing/reduction.py", line
51, in dumps
cls(buf, protocol).dump(obj)
File "stringsource", line 2, in pybloomfilter.BloomFilter.reduce_cython
TypeError: no default reduce due to non-trivial cinit
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/home/SafeBound/Experiments/BuildExperiments.py", line 37, in
time, size = build_stats_object(method='SafeBound',
File "/home/SafeBound/Source/ExperimentUtils/BuildUtils.py", line 459, in
build_stats_object
return build_safe_bound(benchmark, parameters, outputFile)
File "/home/SafeBound/Source/ExperimentUtils/BuildUtils.py", line 261, in
build_safe_bound
stats = SafeBound(tables, tableNames, joinColumns,
File "SafeBoundUtils.pyx", line 246, in SafeBoundUtils.SafeBound.init
self.finishOneStatsBuildProcess()
File "SafeBoundUtils.pyx", line 254, in
SafeBoundUtils.SafeBound.finishOneStatsBuildProcess
self.tableStatsDict[tableName].addFilterColStats(filterCol,
statsFuture.result())
File "/opt/conda/envs/TestEnv/lib/python3.10/concurrent/futures/_base.py",
line 439, in result
return self.__get_result()
File "/opt/conda/envs/TestEnv/lib/python3.10/concurrent/futures/_base.py",
line 391, in __get_result
raise self._exception
TypeError: no default reduce due to non-trivial cinit
—
Reply to this email directly, view it on GitHub
https://github.com/kylebd99/SafeBound/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGQI2YYSIR66CGADKZ3Q55DXOGVPZANCNFSM6AAAAAAZ3SCN2U
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thank you for reaching out and letting us know about the issue with the pybloomfilter package. I appreciate your willingness to help, and I hope you're enjoying your vacation!!
I'll make a note to check the GitHub repository for the public branch that addresses the pickling problem. In the meantime, I'll keep an eye out for your readme update when you return.

Just tried re-running it and ran into some issues. So, I've re-uploaded the entire pybloomfiltermmap3 package and it seems to have fixed it. Should work after a pull of this repo now.


config dot json file

what all inputs do i give from a txt editor for the root file config dot json for the newest version?

Bugs have been fixed in the new version (v3.0.0). Ensure the config file is configured correctly for Mosint to work properly.


WIP Reactions backend: removing reactions

This PR adds support for removing reactions via DELETE requests to the /reactions endpoint with parameters emoji_name and message_id.
The reaction is deleted from the db and a reaction event is sent out with op set to remove.
Tests are added to check that you can't remove a reaction that does not exist and to check that the event payload and users are correct for a delete request.

Automated message from Dropbox CLA bot
@arpith, it looks like you've already signed the Dropbox CLA. Thanks!

@timabbott, @kracekumar could you take a look at this?

This looks pretty reasonable; my only concern is that I'm not sure that use of DELETE without a specific resource is proper REST API style.  We might want to redo the endpoints for reactions to be of the form: PUT|DELETE /messages/1234/emoji_reactions/banana to follow proper REST style.
What do you think @arpith @kracekumar?

I'm fine with doing the update to the endpoints as an additional commit on top of this, since we need to change the existing ADD endpoint anyway, so I'm going to ahead and merge this.

@arpith one thing I notice in the coverage data is that removing a nonexistent reaction doesn't seem to have coverage -- can you look into why?

Thanks @timabbott :) The removing a nonexistent reaction test is here:
https://github.com/zulip/zulip/blob/1fb9220354ec16feb12e794d1f58bc8b5cd668d0/zerver/tests/test_reactions.py#L144
Will take a look to see why it's not reflected in the coverage data.

I'm late to the party but +1 for REST style DELETE /messages/1234/emoji_reactions/banana. Many HTTP clients don't t support DELETE with the body.

The coverage drop was testing invalid emoji's when removing reactions - I've added a test in https://github.com/zulip/zulip/pull/2570 and also fixed the endpoints

Never too late @kracekumar :) fixed in https://github.com/zulip/zulip/pull/2570

Now I think it's better to use DELETE /messages/1234/emoji_reactions/<reaction_id> since reaction_id is PK and  emoji_name isn't indexed. This means reaction id needs to be added in the API.

Also, we can change what the primary key is in the DB model if needed; we should design the API to be best for the programmers coding against it, not for technical limitations, unless they're really insurmountable.

That's very good point!


Add 2 Factor Authentication (Sesame Stretch Goal)

Add option for 2 factor authentication, ring and password/ring and pin

It would be nice to have the option of using the ring and a PIN or some other non-physical token. There are complicated reasons that someone might want to do this but it affects the USA 5th ammendment and what someone there can be compelled to do. Plus it'd kinda be neat.
After a number of invalid PIN guesses, I'd like to write the service to lock / invalidate the encrypted credential and force the use of the original username and password. This way finding someone's token doesn't automatically grant access.


[AKS] Add az aks nodepool create/update/upgrade --max-surge  mirror commit 32ee9feaa up from az extenstions

Description
Not ready yet but hopign to bring this in november after aks/containerservice release.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

aks

@arrownj fixed up some lint but really having problems with azdev and just trying to use or test the acs/aks module after my edits.  A little frustrated. Anyone I could iterate with?
Here's one issue for example.
https://github.com/Azure/azure-cli-dev-tools/issues/269


Support bzlmod in rules_js family of repos

Tracking issue:

[x] rules_js
[x] rules_ts
[x] rules_swc
[x] rules_esbuild
[x] rules_terser
[ ] rules_webpack
[ ] rules_rollup
[x] rules_jest https://github.com/aspect-build/rules_jest/issues/79
[x] rules_jasmine
[ ] rules_cypress

Yep I can take that for my 20%.

rollup is in https://github.com/aspect-build/rules_rollup/pull/72

I think we've done as many of these as we plan to currently. Can use issues on those repos for any follow-up work.


I was wondering which BM25 model you are using

I was wondering which BM25 model you are using.

We use bm25 implementation in Langchain, from langchain_community.retrievers import BM25Retriever.
Refer to implementation in chunk_rerank.py or our toolkit:
from htmlrag import BM25HTMLPruner
bm25_html_pruner = BM25HTMLPruner()
block_rankings=bm25_html_pruner.calculate_block_rankings(question, simplified_html, block_tree)
print(block_rankings)

Btw, we edited the latest issue.

Thank you for your response！


Server back-end database Improvements

Server gets laggy sometimes, especially (mostly) when generating lots of chunks..
If we are still using the sqlite db engine, we should move to use postgres instead.

[ ] Replace SQLite with PostgreSQL for server db
[ ] Migrate database...
[ ] Test server performance using new db engine

Running on Postgresql now.


Error loading script: frame2frame.py

I have a problem loading the script and can´t figure out a solution for that.
Do you have an idea what the problem is and how to solve it?
ERROR:
Error loading script: frame2frame.py
Traceback (most recent call last):
File "C:\Users\Admin\stable-diffusion-webui\modules\scripts.py", line 248, in load_scripts
script_module = script_loading.load_module(scriptfile.path)
File "C:\Users\Admin\stable-diffusion-webui\modules\script_loading.py", line 11, in load_module
module_spec.loader.exec_module(module)
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "C:\Users\Admin\stable-diffusion-webui\extensions\frame2frame\scripts\frame2frame.py", line 14, in
from moviepy.editor import VideoFileClip
File "C:\Users\Admin\stable-diffusion-webui\venv\lib\site-packages\moviepy\editor.py", line 36, in
from .video.io.VideoFileClip import VideoFileClip
File "C:\Users\Admin\stable-diffusion-webui\venv\lib\site-packages\moviepy\video\io\VideoFileClip.py", line 3, in
from moviepy.audio.io.AudioFileClip import AudioFileClip
File "C:\Users\Admin\stable-diffusion-webui\venv\lib\site-packages\moviepy\audio\io\AudioFileClip.py", line 3, in
from moviepy.audio.AudioClip import AudioClip
File "C:\Users\Admin\stable-diffusion-webui\venv\lib\site-packages\moviepy\audio\AudioClip.py", line 4, in
import proglog
ModuleNotFoundError: No module named 'proglog'

Appears one of the dependencies (proglog) is not installed, which should occur during installation of the script. Have you tried restarting the WebUI prompt yet? I'd be interested to hear of any errors that occur during the "installing requirement" stage of startup:

Mind trying that and letting me know?

Thank you for your fast reply.
Ok yes proglog is not installed i checked it in
C:\Users\Admin\stable-diffusion-webui\venv\Lib\site-packages
The other packages are installed. In StabelDiffusion directory and the normal standard Python directory.
I restarted the WebUI a few times since i tried to solve the problem by myself.
I mean i think i could resolve it if i install that missing package (proglog) manually but i will wait since it should work automatically and i´m available to fix this problem together for other users.


Towards consistent null handling

This PR includes a number of fixes to handle nulls more consistently on the query side.  These fixes were done to support a user who is leveraging the schema-less capabilities of Druid.  It's sufficient for all needs we've currently found, but I am not certain that it is comprehensive just yet.
Fixes #665 in the following manner

return null values for all matching rows when a columns is missing (e.g. for groupBy, topN)

treat empty string and null the same, but always return null

allow specifying null in filters

for multi-value dimensions, if the dimension has multiple values:

dim extraction returns null -> null
missing column -> null
[] -> null
[""] -> [null]
["", "a"] -> [null, "a"]

for single value dimensions

missing column -> null
dim extraction returns null -> null
[""] -> null
[] -> null

Also, on the PR is the addition of a context parameter on timeseries queries that allows it to ignore empty buckets.  I realize that this should've been separated into two PRs, but there is a bit of context behind these commits that makes that difficult.  It's just a few changes in TimeseriesQuery, TimeseriesQueryEngine and TimeseriesQueryRunnerTest

I did a quick test replacing news with empty string in druid.sample.tsv.* and it doesn't seem to return the expected result in quite a few test cases. Sometimes we get back null, sometimes "". This obviously requires changing most test, so maybe we'll want to add new ones to cover those case, but that also changing quite a bit of code.

@cheddar Apparently #1014 is not fixed by this pull request. I am +1 on merging this pull request as it makes progress on how Druid handles nulls, but there are still numerous problems with handling nulls and empty strings in Druid and I think we should consider fixing them sooner rather than later.

Can this get squashed/rebased before merging?

Actually we are having this problems with null handlers answering wrong
data when I query or filter this dimensions with null values.
whenever you merge it let me know and I will test it on our labs
Thanks for all
Pablo Nebrera

Pablo Nebrera Herrera
Chief Architect
redborder.net  /<EMAIL_ADDRESS>Mobile: +34 685 48 39 44
Phone: +34 955 60 11 60
https://twitter.com/redBorder_net
https://plus.google.com/u/0/b/115823750653188478256/115823750653188478256/posts
https://www.linkedin.com/company/redborder
Piénsalo antes de imprimir este mensaje
Este correo electrónico, incluidos sus anexos, se dirige exclusivamente a
su destinatario. Contiene información CONFIDENCIAL cuya divulgación está
prohibida por la ley o puede estar sometida a secreto profesional. Si ha
recibido este mensaje por error, le rogamos nos lo comunique inmediatamente
y proceda a su destrucción.
This email, including attachments, is intended exclusively for its
addressee. It contains information that is CONFIDENTIAL whose disclosure is
prohibited by law and may be covered by legal privilege. If you have
received this email in error, please notify the sender and delete it from
your system.
2015-01-27 23:56 GMT+01:00 Fangjin Yang<EMAIL_ADDRESS>
@cheddar https://github.com/cheddar Apparently #1014
https://github.com/druid-io/druid/issues/1014 is not fixed by this pull
request. I am +1 on merging this pull request as it makes progress on how
Druid handles nulls, but there are still numerous problems with handling
nulls and empty strings in Druid and I think we should consider fixing them
sooner rather than later.
—
Reply to this email directly or view it on GitHub
https://github.com/druid-io/druid/pull/995#issuecomment-71746151.

@drcrallen yes, it can, but only when there's actually some indication that the code could get merged.  I'm not interested in wasting time making it mergeable again only to have more merge conflicts introduced later down the line because the PR cannot be merged for political reasons.


Implement configurable db connection timeout

It would be great if the provider configuration allowed for a configurable timeout for connecting to the DB.
Currently using Azure SQL serverless which goes to sleep after 1 hour...it usually takes it 45-60 seconds to wake up and I get:
Error: unable to read user [xxx].[xxx]: db connection failed after 30s timeout    with mssql_user.xxx,    on xxx.database.tf line 54, in resource "mssql_user" "xxx":    54: resource "mssql_user" "xxx" {

The provider should respect normal resource Operation Timeouts.

@magne can you give some more details on how this should work?  I am running into the same issue with timeouts at 30 seconds.  We tried adding the Operation Timeout to both azurerm_mssql_server and azurerm_sql_database and it does not seem to change it.

It would be great if the provider configuration allowed for a configurable timeout for connecting to the DB. Currently using Azure SQL serverless which goes to sleep after 1 hour...it usually takes it 45-60 seconds to wake up and I get:
Error: unable to read user [xxx].[xxx]: db connection failed after 30s timeout with mssql_user.xxx, on xxx.database.tf line 54, in resource "mssql_user" "xxx": 54: resource "mssql_user" "xxx" {

This is because the resource tries to connect to the instance to do the validation, and when we created the azure_mssql_server we didn't insert our output ip in the Firewall.
If you add your outgoing ip on the sql server firewall, it will run successfully.

I am able to connect to the database locally and I am still getting this message:
Error: unable to read user [Communication].[Communication]: db connection failed after 30s timeout
│
│   with module.cluster.module._api_communication.mssql_user._api_db,
│   on ..\modules\cluster\modules\_api_with_db\main.tf line 67, in resource "mssql_user" "_api_db":
│   67: resource "mssql_user" "_api_db"

For anyone unsure of how to use the "normal resource Operation Timeouts" that magne mentions, here's a snippet:
resource "mssql_user" "users" {
  server {
    host = azurerm_mssql_server.main.fully_qualified_domain_name
    azure_login {
      client_id     = null
      client_secret = null
      tenant_id     = null
    }
  }
  database = azurerm_mssql_database.my_database.name
  username = "..."
  roles    = ["db_owner"]
  **timeouts {
    default = "5m"
  }**
}

For anyone unsure of how to use the "normal resource Operation Timeouts" that magne mentions, here's a snippet:
resource "mssql_user" "users" {
  server {
    host = azurerm_mssql_server.main.fully_qualified_domain_name
    azure_login {
      client_id     = null
      client_secret = null
      tenant_id     = null
    }
  }
  database = azurerm_mssql_database.my_database.name
  username = "..."
  roles    = ["db_owner"]
  timeouts {
    default = "5m"
  }
}

Note the timeouts block (which supports only the property default).

I have the following:
resource "mssql_login" "example" {

  # omit for brevity ...

  timeouts {
    default = "5m"
  }
}

and I'm still getting:
Error: unable to read login [login_name]: db connection failed after 30s timeout
@magne, is this the correct way to use the Resource Timeouts mentioned above? There doesn't seem to be any documentation  for this timeouts property.

@pisethdanh I think you'll find you'll keep getting that error message for 5 minutes. That's all that a resource operation timeout does.

@waltervos , thank you for your response. The error message says it's timing out after 30 seconds, which doesn't respect the 5 minutes override that I specified. Unless I'm misconstruing how this works?

@pisethdanh As far as I know this timeout value just refers to how long terraform will keep trying: https://www.terraform.io/language/resources/syntax#operation-timeouts

@pisethdanh, could you try change the timeouts block to
timeouts {
    read = "5m"
}
If this works, I've misunderstood the terraform internals, assuming that the struct would automatically return the default if no specific timeout was set.

@magne , thank you for your response. When setting create, read, update, or delete, I get this error:

It appears those attributes are not supported.
Speculation here, but Azure SQL MI may be throttling the read requests resulting in transient errors. Are there any plans to implement retry logic in this provider?
Apologies for resurrecting a closed issue. Is it more appropriate to create a new issue?

@nazirakz , no real resolution...I just re-run terraform and by the second time the DB is already awake

@nazirakz , no real resolution...I just re-run terraform and by the second time the DB is already awake

ohh.. have you tried to use the resource "time_sleep"?


Double click moves the mouse to the center

This is a really weird issue, if I double click a window, and that double click causes a selection to be made (possibly related to primary selection/clipboard), then the mouse cursor is moved to the center of that window. This only happens with hyprscroller; importantly, it only happens when the scroller layout is active, master layout with hyprscroller loaded does not cause the problem.
I'll try to bisect when this started later
Edit: Also interesting, in XWayland applications, selection does not cause thus, but instead pressing Ctrl+C to copy does, so I think it's fairly likely to be related to the clipboard

It doesn't seem to happen to me.

I open a terminal
I do ls -l
I double click on some word, and the cursor stays there

I think  I am going to need a minimal config where you can reproduce the problem, with your clipboard manager choice etc. Maybe it opens some window?

I noticed at first due to double-click in Firefox. Will try to reduce later, interesting

I think I'm going crazy. Went home, after boot it doesn't happen anymore. Maybe due to a driver bug? At home I use an eGPU, while I'm away discrete Intel Iris... Closing for now, I'll reopen if it becomes a problem again.

OK, reopen the issue if you see it again tomorrow.
The way things work right now, it could happen if there is a tiled (not floating) window that gets created and is maybe hidden from view. When it closes or changes focus, you could find an unexpected warp. It would be hard to debug without an application that does such a thing, and that's why I was expecting you might be using something like that, clipboard manager maybe because you said it happens when copying text.
I was about to add some cursor_warpoption like you suggested the other day, but I will wait until we see another case where things work in an unexpected way.

You were right, it started happening again, after I opened CopyQ for the first time (it was running all the time though). It stopped happening as soon as I quit it. I would appreciate an option to disable focus switching entirely, I can open a PR if you want.

I will add the option, don't worry about a PR. But now that we know an application that does that, I am going to debug it and try to fix the behavior if it is indeed what I thought.

Thank you very much

I have installed CopyQ and I cannot reproduce the issue. I have tried changing most suspect options in its configuration, but I still cannot reproduce the problem. If you feel like it, export your configuration (with commands etc.) from the Preferences menu, Reset it, test again and then import the saved config again if you don't see any difference. I would really like to know what option is causing this problem.
Anyway, now I am going to add warp_cursor or something like that so you can have that option.

Actually, I see there is a cursor:no_warps option in Hyprland, and hyprscroller respects it (I am simply calling CCompositor::warpCursorTo(), and it checks that option). So if you set that to true (the default is false), the cursor won't warp in most cases. Please, try that. It is always better not to duplicate options if possible.

I have installed CopyQ and I cannot reproduce the issue.

I also cannot reliably reproduce it, it just starts at some point. It seems like there's always copying in an xwayland window involved though. Also, since I'm on Ubuntu my CopyQ is somewhat old (7.1.0), so that may also be related. Thank you and sorry for spending so much time.

Actually, I see there is a cursor:no_warps option in Hyprland

Good to know, I haven't seen that option. It's not quite what I want, because it also does not move the cursor in response to keybinds (e.g. switching monitor), but I can live with it and it fixes this problem. Thank you

I will try to compile version 7.1.0 of CopyQ later today to see if I can find out what's happening.
I used CopyQ in the past, but I removed it for two reasons. The first one was dependencies. On Arch, it requires some KDE stuff. The second and more important one is Wayland doesn't have support for a nice interaction of out-of-app menus floating at the cursor, much needed for dictionaries or this kind of applications, so they open in another independent window. That is not pretty when using a tiling layout. For that reason, I moved to cliphist, which I use with rofi (the Wayland version). In case you want to have a look at it:
exec-once = wl-paste --type text --watch cliphist store #Stores only text data
exec-once = wl-paste --type image --watch cliphist store #Stores only image data

# cliphist
bind = $mainMod, V, exec, cliphist list | rofi -dmenu | cliphist decode | wl-copy
bind = $mainMod SHIFT, V, exec, rofi -modi clipboard:/home/dawsers/.config/rofi/cliphist-rofi-img -show clipboard -show-icons

No, don't, doesn't matter, I also updated to the latest from Github and it didn't change anything
I also tried cb, but that constantly opens windows, which does not work at all with my setup, I'll have a look at cliphist, thanks

You could also give clipse a shot, in the event you don't use Rofi.
I have a tuiTool tag that I use to pop up kitty with clipse (and other TUIs) in the center of my screen. Haven't had any issues with what you're describing nor copying from/to XWayland (sans 1Password, but I believe that's a different issue entirely as it predates my use of clipse).

Yep, I settled on clipse :)
I had a command in CopyQ to copy to xwayland (wl-paste | xclip -sel clip), which I have now bound to a keyboard shortcut, since certain applications (like VirtualBox) don't quite like the shared clipboard

Thumbs up for clipse, I didn't know it. I have converted too.


Minor memory safety / initialization fixes for Infer

Description of changes:
Infer flagged some potential problems, although none of them can actually cause an issue.
Testing:
How is this change tested (unit tests, fuzz tests, etc.)? Existing unit tests.
Is this a refactor change? No.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2347 into main will increase coverage by 5.98%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##             main    #2347      +/-   ##
==========================================
+ Coverage   81.11%   87.10%   +5.98%
==========================================
  Files         267      166     -101
  Lines       18218    13977    -4241
==========================================
- Hits        14778    12175    -2603
+ Misses       3440     1802    -1638


Blacklisting a token

There is a security problem that in this module the whole token is stored in the blacklist. Do you have a solution now?

I'm sure this was discussed in the past, just can't find it, but a suggestion to hash the blacklist tokens was provided in the past. I don't really see a need for it since it's mostly for read-only, not write. I think this is an extensible feature that you can add via a PR, but it's likely to degrade performance without much significant benefit in my opinion.

I had some questions from the document:
What is the reason that sliding tokens are not safe? Please explain
And what is the reason that sliding tokens are low performance?
And it is quite clear that there is a security problem with storing tokens in OutstandingToken. Why not just save the jwt token signature, for example? Do not have any special settings to solve this problem.

I had some questions from the document: What is the reason that sliding tokens are not safe? Please explain And what is the reason that sliding tokens are low performance?
And it is quite clear that there is a security problem with storing JWT tokens in OutstandingToken. Why not just save the jwt token signature, for example? Do not have any special settings to solve this problem.

Can anyone answer my questions? I really stopped using your library. :((((((

When I add the blacklisting app to installed apps and try running the migration, I get this database error:
`Operations to perform:
Apply all migrations: accounts, admin, auth, contenttypes, main, sessions, token_blacklist
Running migrations:
Applying token_blacklist.0008_migrate_to_bigautofield...Not implemented alter command for SQL ALTER TABLE "token_blacklist_blacklistedtoken" ALTER COLUMN "id" TYPE long
Traceback (most recent call last):
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\cursor.py", line 51, in execute
self.result = Query(
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\sql2mongo\query.py", line 784, in init
self._query = self.parse()
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\sql2mongo\query.py", line 876, in parse
raise e
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\sql2mongo\query.py", line 857, in parse
return handler(self, statement)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\sql2mongo\query.py", line 889, in _alter
query = AlterQuery(self.db, self.connection_properties, sm, self._params)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\sql2mongo\query.py", line 425, in init
super().init(*args)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\sql2mongo\query.py", line 84, in init
super().init(*args)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\sql2mongo\query.py", line 62, in init
self.parse()
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\sql2mongo\query.py", line 441, in parse
self._alter(statement)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\sql2mongo\query.py", line 500, in _alter
raise SQLDecodeError(f'Unknown token: {tok}')
djongo.exceptions.SQLDecodeError:
    Keyword: Unknown token: TYPE
    Sub SQL: None
    FAILED SQL: ('ALTER TABLE "token_blacklist_blacklistedtoken" ALTER COLUMN "id" TYPE long',)
    Params: ([],)
    Version: 1.3.6

The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\backends\utils.py", line 85, in _execute
return self.cursor.execute(sql, params)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\cursor.py", line 59, in execute
raise db_exe from e
djongo.database.DatabaseError
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "manage.py", line 22, in
main()
File "manage.py", line 18, in main
execute_from_command_line(sys.argv)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\core\management_init_.py", line 425, in execute_from_command_line
utility.execute()
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\core\management_init_.py", line 419, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\core\management\base.py", line 373, in run_from_argv
self.execute(*args, **cmd_options)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\core\management\base.py", line 417, in execute
output = self.handle(*args, **options)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\core\management\base.py", line 90, in wrapped
res = handle_func(*args, **kwargs)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\core\management\commands\migrate.py", line 253, in handle
post_migrate_state = executor.migrate(
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\migrations\executor.py", line 126, in migrate
state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\migrations\executor.py", line 156, in _migrate_all_forwards
state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\migrations\executor.py", line 236, in apply_migration
state = migration.apply(state, schema_editor)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\migrations\migration.py", line 125, in apply
operation.database_forwards(self.app_label, schema_editor, old_state, project_state)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\migrations\operations\fields.py", line 225, in database_forwards
schema_editor.alter_field(from_model, from_field, to_field)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\backends\base\schema.py", line 620, in alter_field
self._alter_field(model, old_field, new_field, old_type, new_type,
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\backends\base\schema.py", line 777, in _alter_field
self.execute(
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\backends\base\schema.py", line 153, in execute
cursor.execute(sql, params)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\backends\utils.py", line 99, in execute
return super().execute(sql, params)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\backends\utils.py", line 67, in execute
return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\backends\utils.py", line 76, in _execute_with_wrappers
return executor(sql, params, many, context)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\backends\utils.py", line 85, in _execute
return self.cursor.execute(sql, params)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\utils.py", line 90, in exit
raise dj_exc_value.with_traceback(traceback) from exc_value
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\django\db\backends\utils.py", line 85, in _execute
return self.cursor.execute(sql, params)
File "C:\Users\MR.Robot.virtualenvs\fsm-GjGxZg3c\lib\site-packages\djongo\cursor.py", line 59, in execute
raise db_exe from e
django.db.utils.DatabaseError`


ccache: 4.4.1 → 4.4.2

Motivation for this change
Update to the latest version: https://github.com/ccache/ccache/releases/tag/v4.4.2
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@r-burns

Result of nixpkgs-review pr 139818 run on x86_64-linux 1

3 packages built:

ccache
ccacheStdenv
ccacheWrapper


feat: Native promises support in Appsmith

Enables native promise support on Appsmith 🤞
Ability to have promises and other async functions in appsmith to have better workflow design that is easy to maintain.
Rework #4723
Implements #6276
Implements #6277
Fixes #7183
Resolves #6966
Fixes #6963
Fixes #5495
Fixes #6822
Fixes #9317
Examples:
Improved action.run syntax
{{
  UsersApi.run({ org: "Appsmith" })
    .then((response) => showAlert(response) )
    .catch((error) => showAlert(error, 'error'))
}}

// instead of

{{
UsersApi
  .run(
    () => showAlert("Done"),
    () => showAlert('Oops'),
    { org: "Appsmith" }
  )
}}

Loops and Promise.all
{{(function(){
    let calls = [];
    // Start a bunch of calls running in parallel
    for (let step = 0; step < 5; step++) {
      calls.push(MockApi.run({ count: step.toString() }))
    }
    // Wait for all to finish and then use their responses together
    return Promise.all(calls)
      .then((responses) => showAlert(responses.map((res) => res.args.count).join(',')))
})()}}

Context is maintained even in callbacks
{{(function(){
  // declare myVar outside
  const myVar = "Hello";
  return MockApi.run({ count: myVar })
     // use it in the callback
    .then(() => showAlert(myVar))
})()}}

Async / Await
{{(async function(){
  const res = await UsersApi.run();
  const res2 = await MockApi.run({ name: res[0].name });
  await storeValue( "name", res2.args.name );
  await showAlert(appsmith.store.name);
})()}}

Breaking changes:

Params will not be passed in the action.run callbacks

Testing

Automated tests with Jest and Cypress

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: feature/promises-js-editor

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
55.22 (0.08)
37.14 (0.07)
34.05 (0.09)
55.73 (0.09)

:red_circle:
app/client/src/components/editorComponents/Debugger/helpers.tsx
79.69 (-0.31)
79.17 (0)
37.5 (0)
79.63 (-0.37)

:red_circle:
app/client/src/constants/messages.ts
74.1 (0)
100 (0)
23.35 (0)
79.43 (-0.11)

:red_circle:
app/client/src/sagas/EvaluationsSaga.ts
20.38 (-2.45)
2.27 (-0.36)
12 (-0.5)
23.24 (-2.69)

:red_circle:
app/client/src/sagas/JSPaneSagas.ts
17.14 (-0.64)
1.33 (0.11)
6.25 (0.37)
20.13 (-0.65)

:red_circle:
app/client/src/sagas/PostEvaluationSagas.ts
16.55 (-0.34)
0 (0)
0 (0)
20.17 (-0.32)

:green_circle:
app/client/src/sagas/ActionExecution/ActionExecutionSagas.ts
23.16 (0.16)
2 (0)
20 (0)
24.44 (0.23)

:green_circle:
app/client/src/sagas/ActionExecution/CopyActionSaga.ts
54.55 (10.11)
0 (0)
0 (0)
54.55 (10.11)

:red_circle:
app/client/src/sagas/ActionExecution/GetCurrentLocationSaga.ts
14.49 (-0.22)
0 (0)
0 (0)
15.63 (-0.24)

:red_circle:
app/client/src/sagas/ActionExecution/ModalSagas.ts
41.18 (-15.96)
0 (0)
0 (0)
41.18 (-15.96)

:green_circle:
app/client/src/sagas/ActionExecution/PluginActionSaga.ts
17.74 (0.24)
2.75 (-0.28)
10.53 (0)
20.28 (0.18)

:green_circle:
app/client/src/sagas/ActionExecution/ResetWidgetActionSaga.ts
42.11 (4.61)
0 (0)
0 (0)
44.44 (4.44)

:green_circle:
app/client/src/sagas/ActionExecution/ShowAlertActionSaga.ts
31.82 (6.82)
0 (0)
0 (0)
31.82 (6.82)

:green_circle:
app/client/src/sagas/ActionExecution/errorUtils.ts
65.45 (0.64)
0 (0)
42.86 (0)
53.66 (1.16)

:red_circle:
app/client/src/utils/DynamicBindingUtils.ts
85.71 (-0.09)
70.48 (0)
78.57 (0)
85.26 (-0.09)

:green_circle:
app/client/src/utils/WorkerUtil.ts
89.76 (-0.11)
72.55 (1.96)
100 (0)
93.33 (1.02)

:green_circle:
app/client/src/workers/Actions.ts
83.58 (13.51)
66 (16)
79.31 (9.31)
81.36 (12.76)

:green_circle:
app/client/src/workers/DataTreeEvaluator.ts
78.41 (0.77)
61.43 (-0.54)
76.74 (0)
78.2 (0.82)

:sparkles: :new:
app/client/src/workers/PromisifyAction.ts
100
100
100
100

:green_circle:
app/client/src/workers/evaluate.ts
94.69 (6.78)
78.57 (1.9)
90.91 (13.13)
94.23 (7.02)

:x:
app/client/src/sagas/ActionExecution/PromiseActionSaga.ts
17.24
0
0
19.23

/ok-to-test sha=7e00046

/ok-to-test sha=073fe12

/ok-to-test sha=7a3425b

/ok-to-test sha=e2873a1

can you please add comments on the new code? @hetunandu

/ok-to-test sha=74cf8ae

/ok-to-test sha=74cf8ae

/ok-to-test sha= a511342

/ok-to-test sha=a511342

Breaking changes:
Params will not be passed in the action.run callbacks
@hetunandu
Just to confirm again, this means onSuccessCallback and onFailureCallback will no more be supported, correct?

As I tried in local and it showed an error and this is what we discussed earlier too AFAIK.
How are we planning to migrate users' older applications for these changes?
Also, we will need changes in our docs for the same.

/ok-to-test sha=a7b1708

/ok-to-test sha=67bfdbe

/ok-to-test sha=71d4c4f

/ok-to-test sha=3f37475

/ok-to-test sha=6ad4460

/ok-to-test sha=ce00516


Testing triton when C matrix has some initial values. Possible?

Hi, Is it possible to benchmark triton when C matrix is not assumed to be all zeros, i.e., it is loaded from global memory rather than initializing an accumulator tile with zeros?
Thanks!

@ptillet any help would be appreciated. Thanks!

@navdeepkk Sorry for the delay. I was double-checking something. So right now it seems like initialize matmul accumulator with loaded values is buggy. But what you can do instead is to add an accumulation after the matmul loop.
acc = 0
for k in range(K, 0, -BLOCK_K):
  ...
D = ... # construct pointer to the other tensor
acc += tl.load(D)

I will work on fixing bugs so that one can do
D = ... # construct pointer to the other tensor
acc = tl.load(D)
for k in range(K, 0, -BLOCK_K):
  ...

instead

Thanks! I'll try this.

Actually I just checked and this doesn't seem to work anymore now ... But I'll be working on it over the next few days.

Oh Okay, I have some matmul kernels generated, which load C from global memory. I wanted to compare against Triton. I'll get on that when this is fixed. Thanks!

Hey. This should work now on top of master :) Just add for example c = c + tl.load(c_ptrs, mask=c_mask) there: https://github.com/openai/triton/blob/master/python/tutorials/03-matrix-multiplication.py#L253. You can also initialize the accumulator here https://github.com/openai/triton/blob/master/python/tutorials/03-matrix-multiplication.py#L228, but it should be less efficient, and for now it'll only work if you explicitly cast it to float32 after loading. Let me know if you have any more issue


Transmissives PR Edits

This PR includes changes related to my comments on #31.
Also fixes the sampleNumber variable which was set to 1 on the first frame, but should be 0 instead. This caused the first frame to be slightly darker.

All fine. Should we write use_external_buffers, or useExternalBuffers. Me personally, i prefer the second option as it avoids having to type the underscore. If we use the current version we should think about also using this convention on other variables be consistent.

I used to use camelCase variable names a few years back as well. However after working a while with code that used the snake_case style, I think that use_external_buffers is more readable than useExternalBuffers. Especially for longer variable names and I prefer to be rather explicit with variable names.
Here a few arbitrary example just for comparison:

screenSpaceReflectionBuffer
screen_space_reflection_buffer
opaqueClosestHitShader
opaque_closest_hit_shader

Additionally, snake_case fits quite well with the convention of adding a leading underscore to class member variables (_some_member_variable), which to me is the least intrusive way of marking member variables compared to other styles such as a hungarian style prefix mSomeMemberVariable or even m_SomeMemberVariable.

All valid arguments, even though I did not experience any problems with camel case longer names concerning readability...
Concerning the consistency with the member variables this would of course be a bonus. Even though i would argue that it is also beneficial if there is no consistency ther, as then the underscore is more dominant and it can be faster distinguished in code that we do have a member variable.
However these are just small design desicions. The main question remains: Should we keep everything consistent, or should we allow different styles? According to my experience having a view variables in 'snake_case' and 'camelCase' does not affect much. If we use 'snake_case' consistently we have to change all other variables, using only 'camelCase' would be faster as most is already in 'camelCase'.
I do not know if the advantages of 'snake_case' are large enough to weigh up the extra cost of refacturing.
Or just let it be for the time being and use a unified format for the future (there might be eventual refactors which automatically will update the variable naming conventions).

Has a small shader bug which is fixed in #32 .


freezing the app(.py to exe)

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):N/A
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary):  source
TensorFlow version (use command below): 1.15.0
Python version:3.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:N/A
GPU model and memory:

Describe the current behavior
I have tried to freeze the custom app with auto-py-to-exe and it is giving mw the error of "ImportError: No module named 'tensorflow_core.python'"
can anyone please help me with it
Thanks in advance

@daynial132
tensorflow_core only exists in 1.15, 2.x.
Provide the exact sequence of commands / steps that you executed before running into the problem.Thanks!

I have created a virtual environment  in anaconda on 3.5 python
installed the packages specifically used for the my app
then installed the auto-py-to-exe and run it
and finally include the script to script location used one direction option,
in additional files  i added complete folder to it and all the files in the folder
in advance option on runtime-hooks-dir  I added complete site-packages
thats it. and press the convert button on auto-py-to-exe

have i done something wrong ??
or missing some thing

@daynial132 This is a conversion issue, not the tensorflow issue itself. It would be better if you post this issue in stackoverflow where there is a wider community to respond. Thanks!


Weird rendering on windows 11

I tried your website on my edge browser this looks something like this

The console is all clear no message is there

Hi codexsr,
This reminds me of the issue mrdoob had:
https://github.com/Spiri0/Threejs-WebGPU-IFFT-Ocean-V2/issues/4
The picture suggests that the WGSL shaders are not being processed correctly by the system.
In some browsers on certain systems, WebGPU is not yet fully supported. The fact that something like this doesn't even produce an error message in the console indicates that the browser in the system in question does not fully support it, otherwise you would expect an error message because it is obviously not right.
It works in Windows 10 in Chrome and Edge. On my Ubuntu computer with Chrome too. Since it works in general, it is something system-specific. However, the fact that there is no error message in the console is annoying because I could at least identify the cause.
Unfortunately, without a trace, we can only speculate 😔
Have you already been able to test it with Chorme on the same computer? I'm asking specifically about chrome because chrome offers the best support for WebGPU so far.
For some systems, Chrome only started supporting WebGPU this year. I find it exciting to experience this new technology from the very beginning.


'az eventgrid event-subscription create' causes Azure CLI task to fail

Azure CLI task v2 causes the pipeline to fail although everything was successfully deployed. Checking the logs written to stderr by my script showed errors that were supposed to be informational logs from running the command az eventgrid event-subscription create.
Task
Task: AzureCLI@2
...
scriptType: bash
failOnStandardError: true

Command in script
      az eventgrid event-subscription create --name "es-$app_name" \         --source-resource-id $eventgrid_id \         --endpoint $app_id/functions/eventGrid_format --endpoint-type azurefunction
Logs written to stderr

2020-06-09T00:34:02.5424246Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5434374Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5436270Z ##[error]If the provided endpoint does not support subscription validation handshake, navigate to the validation URL that you receive in the subscription validation event, in order to complete the event subscription creation or update. For more details, please visit http://aka.ms/esvalidation
2020-06-09T00:34:02.5437644Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5438645Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5441016Z ##[error]If the provided endpoint does not support subscription validation handshake, navigate to the validation URL that you receive in the subscription validation event, in order to complete the event subscription creation or update. For more details, please visit http://aka.ms/esvalidation
2020-06-09T00:34:02.5442431Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5443524Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5445044Z ##[error]If the provided endpoint does not support subscription validation handshake, navigate to the validation URL that you receive in the subscription validation event, in order to complete the event subscription creation or update. For more details, please visit http://aka.ms/esvalidation
2020-06-09T00:34:02.5446488Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5447512Z ##[error]Script has output to stderr. Failing as failOnStdErr is set to true.

eventgrid

Hello @Sarah-Aly .. Can you please let us know if this issue still persist for you? From the error it seems the the Azure function is not created initially.
If the issue still exist, can you please share the details of a new attempt that fails and provide us with details about SubId/Rg and event subscription Name, Region, exact time of when it happens (you can email me these info directly if you want for privacy reasons at ahamad@microsoft.com). Also, make sure the destination is already created and ready to be used. Thanks.

Hi @ahamad-MS , this was a few months ago & the scenario we were working with has changed. However, I'm pretty sure that the destination was already created & ready to be used. The workaround I did at the time was to make the bash script ignore the logs from running az eventgrid  which is not optimal of course :).
bash       az eventgrid event-subscription create --name "egt-$app_name" \         --source-resource-id $eventgrid_id \         --endpoint $app_id/functions/eventGrid_format --endpoint-type azurefunction &> null
with this, I'm closing this issue & will re-open it in case we get this issue again. Thanks.

Thanks @Sarah-Aly. Please let us know if you are still encountering the issue and provide us with the info to investigate. Thanks.

The topic for using failOnStdErr is now tracked by https://github.com/Azure/azure-cli/issues/18372.


fix(scheduler): copy object-storage creds during RC creation instead …

what difference does it make ? creation during RC phase

got it no worries

Prior to that I had added a check where we only added the object keys when it was the right build type, however create actually gets no information passed to it and actually does not have the build information at the time anyway so it was a dud to begin with

I tested this and it does avoid the Error syncing pod, skipping: secrets "objectstorage-keyfile" not found problem.


Allow IgnoreByNameSuppressorInspection to ignore inspections that target lists

Additionally, this pull request also allows targeting field arguments such as Query.findUser.id.

@owenvoke Can you please take a look at this?

Hi @spawnia, apologies for not seeing this before. I haven't really had much time to focus on GraphLint recently.
I'll take a look shortly.

Thank you for the quick https://github.com/worksome/graphlint/releases/tag/v0.11.3 release 👍


Engineering Enhancement: Build and Clean scripts

We're getting close to the stage where it'd be good to have repeatable ways to create a faithful build without extraneous or old files.
To create a ZIM file we need an icon.png and currently I'm also creating a copy of style.css in a new folder ./A/ so the stylesheet is correctly applied in the ZIM content (this may be addressed separately at some point).
Implementation ideas
We currently use npm start to create the distribution. Could we enhance this to:

clean the ./dist folder,
copy the icon.png,
copy Google Analytics Javascript (currently in ./include),
create ./dist/A/ and duplicate the CSS file style.css into this folder (symlinks cause problems for zimwriterfs (See https://github.com/openzim/libzim/issues/16 and https://github.com/openzim/zimwriterfs/issues/3)

I don't see a ./include, what do you mean by that?

Sorry for mentioned it before I'd committed it. I've just done so.

I believe we've addressed this enhancement, so I'll close this issue. Thanks for your help.


[<IP_ADDRESS>] Client cannot connect to remote carla server

Hi,
I'm running Carla <IP_ADDRESS> in a docker with the following command:
sudo -E docker run --privileged --rm --gpus all -it -d --name carla  --net=host -e DISPLAY=$DISPLAY -v /tmp/.X11-unix:/tmp/.X11-unix:rw carlasim/carla:<IP_ADDRESS> /bin/bash
and it seems to work fine, since it is not crashing or anything suspicious. However, clients cannot connect and get a timeout. When I run the release version without a docker it works fine and connecting is not a problem.
sudo netstat -tulpn | grep LISTEN and docker port carla also don't show any ports.
If I use -p 2000-2002:2000-2002  instead of  --net=host then it shows a docker on those ports, however connection is still not possible.
Does anyone have a solution or experienced the same problem and can help?
Thanks

It seems it is not a docker problem. The machine I am working on is remote and accessvia ssh. When I run
SDL_VIDEODRIVER=offscreen ./CarlaUE4.sh -carla-world-port=2000 -opengl
I get no error message. But when I run
netstat -an | grep "LISTEN "
logged in on the same machine via ssh there is no application listening on port 2000 or 2001
Also no python util or example script can connect.
If I try the same both commands on my local machine everything works fine. Also I see that an application is listening on both required ports.
Is it possible to find out if there is a problem during start up in the docker container or the release version on my remote machine?
The difference between local and remote is that my local machine does not have a nvidia card.

Solved here: Issue 3476


Find and findOne with varchar primary key

If a table has a PK of type Varchar (or anything that in Javascript is a string) the call
db.table.findOne("pkvalue", function(err, res) {});

fails, because the code interprets "pkvalue" as a where condition and not as the value of the PK to find.
Actually, it generates an exception in where.js because it expects an object and finds a string, so the
key.trim().split(/ +/)

tries to trim a 0...
The solution is just to change
if(_.isNumber(args.conditions)){

with
if(_.isNumber(args.conditions) || _.isString(args.conditions)){

in queryable.

nice catch!

@dmfay I've seen that v2.1.0 has just been released but unfortunately doesn't include this pr. Should we expect a release including the fix anytime soon?

@eymengunay @dmfay it looks like we have a few things that should probably be bundled up into a release in the near future. i know @robconery is travelling at the moment, and I also think we should review what's pending a release, and if there is anything else outstanding that should be.
Ultimately that's all up to Rob, but it shoujld be coming, soon
@dmfay Thanks for staying on top of this stuff :-)

Also, if you need to get by in the mean time and are willing to risk a little jankiness, you can track master or even a specific commit in your package.json (link). But be sure to update once the release drops!
@xivSolutions it's no problem, without Massive I'd have had to write my own data layer from the ground up -- this is way better!

@xivSolutions @dmfay Thank you for the explanation. I'm already using the master branch although as you said it is not an ideal solution.
I'm looking forward for a new release, thank you all for this awesome library :)


GH-39577: [C++] Fix tail-word access cross buffer boundary in CompareBinaryColumnToRow

Rationale for this change
Default buffer alignment (64b) doesn't guarantee the safety of tail-word access in  KeyCompare::CompareBinaryColumnToRow. Comment https://github.com/apache/arrow/issues/39577#issuecomment-1889090279 is a concrete example.

What changes are included in this PR?
Make KeyCompare::CompareBinaryColumnToRow tail-word safe.

Are these changes tested?
UT included.

Are there any user-facing changes?
No.

Closes: #39577

Thanks a lot for finding this out @zanmato1984 ! I posted some comments above.

Thanks a lot for finding this out @zanmato1984 ! I posted some comments above.

Thank you for looking and the very helpful comments @pitrou ! Change updated.

@github-actions crossbow submit -g cpp

@raulcd I think this is a good candidate for 15.0.0.

@pitrou is this worth a new Release Candidate? I created RC1 a couple of hours ago and have already generated all binaries (just waiting for one job to finish) for it successfully, see: https://github.com/apache/arrow/pull/39641

@raulcd Not in itself. But if you have to craft a RC2 for other reasons, this would be nice to include.


k8s: add daemonset probe

You can test this by:
k8s:
  probes
  - pod
  - daemonset
  - namespace
  - cluster

You can create a DaemonSet by:
cd tests/k8s
kubectl create -f daemonset.yaml

run tests

run skydive-selenium-tests


File Explorer Add-Ons - hide .* file/folders

Description of the new feature / enhancement
Make file/folders which's name is started with . hidden
Scenario when this would be used?
These folders/files are seldom used, so I don't want to see them any time using file explorer
Supporting information
No response

/dup #8054


MH-12998: Clear conflicts when closing “Edit Scheduled Events” modal

When you have conflicts displayed in the “Edit scheduled events”
modal, then close the modal an reopen it again (possibly with
different events and without changing anything), the previous error
message resides.
This commit clears existing conflicts before the modal is closed.
This work was sponsored by SWITCH.

Travis reports failing tests, but I didn't even change any .java files.

I think greg was still working on the travis build at the moment. lets wait a moment and see what happens

@pmiddend @doofy just ignore Buildbot for now.  If Travis is happy then we're happy.  Buildbot is experiencing #189, which has not been solved yet.

Any update on this?

Oh, thanks for the reminder, will look into it today.

Reviewing now…

Seems to work fine now. Thanks for the patch → merged


memcached → sqlite

From what I see, we use it only to store the access_token and it makes no sense to force the user to install it and have it running.
We can easily replace it with a local sqlite database, which we can later use to also store timers, alarms, etc. The main thing here is to carefully think about where to store the DB file.
@sammachin Is there any particular reason for why you've chosen memcached originally?

Reasons I used memcache were:
)1 I was familiar with it as used it on lots of other projects
2) the access token expires after 1hr so when I set it I set an expiry of
just under (3590s IIRC) so it's a fairly easy way to check for an access
token and if there isn't one or it has expired it requests a new one with
the request token.
I'd rather not have to do our own time calaculations to determine if a
token has expired and making the request to find the token has expired then
generate a new one and repost the request would be a huge increase in delay.
On Saturday, 15 October 2016, René Kliment<EMAIL_ADDRESS>wrote:

From what I see, we use it only to store the access_token and it makes no
sense to force the user to install it and have it running.
We can easily replace it with a local sqlite database, which we can later
use to also store timers, alarms, etc. The main thing here is to carefully
think about where to store the DB file.
@sammachin https://github.com/sammachin Is there any particular reason
for why you've chosen memcached originally?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa-pi/AlexaPi/issues/43, or mute the thread
https://github.com/notifications/unsubscribe-auth/AALc_bRMH8b3MoOB_0gckr5UwjfNqJ4Qks5q0Mj_gaJpZM4KXsNE
.

I see. So it basically makes no sense to store it anywhere (permanently). There is not much reason in doing this so someone that restarts AlexaPi can keep using the existing token. It might make sense for developers, because they might restart it often, but I personally don't have memcached installed on my dev machine so it probably fetches it every time, but it works without a problem.
Let's just store it in a variable together with the timestamp it was acquired. The comparison of the time is easy and we won't have to do requests with an expired token. We'll just expire it locally if the time has passed and get a new one.

Ok makes sense yeah store it locally with the time stamp and check that
it's under 3590s old.
The only scenario where mc helps is if you restart the python app but not
the system it will persist but like you said that's an edge case and
fetching a new access token isn't hugely expensive on its own, although I
expect AVS wouldn't like it if we all did it on every request. Limiting the
amount the refresh token is passed over the wire also helps with security
On Saturday, 15 October 2016, René Kliment<EMAIL_ADDRESS>wrote:

I see. So it basically makes no sense to store it anywhere (permanently).
There is not much reason in doing this so someone that restarts AlexaPi can
keep using the existing token. It might make sense for developers, because
they might restart it often, but I personally don't have memcached
installed on my dev machine so it probably fetches it every time, but it
works without a problem.
Let's just store it in a variable together with the timestamp it was
acquired. The comparison of the time is easy and we won't have to do
requests with an expired token. We'll just expire it locally if the time
has passed and get a new one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa-pi/AlexaPi/issues/43#issuecomment-253983652,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALc_dB2KruaK7ZX8IbJ5CDWAb1fybDMks5q0NH2gaJpZM4KXsNE
.

Yes, I'm aware. My main concern is getting rid of memcached.
I might have a look at the sqlite stuff then. It's probably super easy.

Currently, there is no clear idea how (if) we're gonna use sqlite for storing stuff like timers, etc, so let's not go there now. I don't think it's a problem, getting the token each time AlexaPi restarts. Let's just get rid of memcached and we can do this when we know how we want to do this.

Why not just keep that in a text file somewhere? OS provides timestamping and we don't need no SQL to find that file.

Yeah, I thought about that too and then I thought why bother at all. Do you think there's significant advantage in storing it on the filesystem rather than just in a variable (with a timestamp)? I consider the token run-time.


ARROW-12915: [Release] Build of ubuntu-docs is failing on thrift

This PR resolves building on Ubuntu 20.04 which is our default though the crossbow build in tasks is on 20.10 which has one of these additions, but also a newer protobuff so that's not needed to be bundled.
We also might consider not adding these lines but making clear in the docker compose file that ubuntu-docs will not work with Ubuntu 20.04 and instead should always be build on 20.10

I ran this on crossbow (locally since the GH actions comment bot isn't working right now https://issues.apache.org/jira/browse/ARROW-12919) and the only ubuntu crossbow builds that failed were all ones that have other current nightly failures.
I've also added a defaults docs nightly to catch if we have a mismatch like this in the future.


feat(extras): add yazi theme

This PR is a theme for yazi, which is a terminal file manager written in rust.
It provides some features which are similar to which-key, noice.

Thanks!


(Element-wise) division of APyFixedArrays

Either elementwise or a scalar. It seems like NumPy also supports scalar/array so we may want to implement that as well.

Closed in #342


Fix saving of config.yaml when using multiple loggers

🚀 Feature
Currently, the SaveConfigCallback saves the config.yaml file to trainer.log_dir, which for one logger is trainer.loggers[0].log_dir.
However, if multiple loggers are specified trainer.log_dir == trainer.default_root_dir. In this case the config.yaml is saved ABOVE the version_dirs, which prevents running the experiments multiple times.
Ideally, the config.yaml would in this case still be saved to trainer.default_root_dir/{version_found_at_runtime}/config.yaml.
Motivation
If I have a directory structure like this
runs/
    experiment1/

and initialize the trainer: Trainer(default_root_dir="runs/experiment1", loggers=True)
the config.yaml is safed to
runs/
    experiment1/
        lightning_logs/
            version_1/
                config.yaml

If  I however specify multiple loggers (e.g. loggers = [TensorBoardLogger(...), CSV_Logger(...)]) the config.yaml is saved to

runs/
    experiment1/
        lightning_logs/
            config.yaml
            version_1/

, independent of the log_dirs of the loggers, which hinders multiple executions of the experiment.
As I'm not sure If this is intended behaviour I marked this post as a Feature Request.
Alternatives
As the trainer (as far as I know) does not offer an attribute to the path of the current version_dir another solution could be to save the config.yaml to EVERY log_dir of the loggers.

If you enjoy Lightning, check out our other projects! ⚡

Metrics: Machine learning metrics for distributed, scalable PyTorch applications.

Lite: enables pure PyTorch users to scale their existing code on any kind of device while retaining full control over their own loops and optimization logic.

Flash: The fastest way to get a Lightning baseline! A collection of tasks for fast prototyping, baselining, fine-tuning, and solving problems with deep learning.

Bolts: Pretrained SOTA Deep Learning models, callbacks, and more for research and production with PyTorch Lightning and PyTorch.

Lightning Transformers: Flexible interface for high-performance research using SOTA Transformers leveraging Pytorch Lightning, Transformers, and Hydra.

cc @tchaton @justusschock @awaelchli @borda @edward-io @ananthsub @rohitgr7 @kamil-kaczmarek @Raalsky @Blaizzy

This is caused by the logger's design, it's unrelated to the LightningCLI
from pytorch_lightning import Trainer
from pytorch_lightning.loggers import TensorBoardLogger, CSVLogger

save_dir = "my_save_dir"
trainer = Trainer(logger=[CSVLogger(save_dir), TensorBoardLogger(save_dir)])
print(trainer.log_dir)  # /home/carmocca/git/pytorch-lightning

save_dir = "my_save_dir"
trainer = Trainer(logger=CSVLogger(save_dir))
print(trainer.log_dir)  # my_save_dir

cc'ing @daniellepintz @akashkw for comment

Yes, this is a problem of how the Trainer and the loggers interact.
One potential fix for now (I'm using that myself right now, with an overridden SafeConfigCallback subclass) is to instead safe the config.yaml to the log_dir of every logger, as it is done with the save_hyperparameters() function, that saves the hparams.yaml in a similar way.
It should definitely be fixed, as it hinders training with multiple Loggers and the CLI.

https://github.com/PyTorchLightning/pytorch-lightning/issues/12028


Power Rename inprovements?

Description of the new feature / enhancement
Is it possible add to Power Rename this features?
1- add an extension to a selected files (I came from mac os where extension isn't so important, so I have many files to add the extension to work on windows)
2- add the possibility to delete and/or add characters to the beginning or end of the name (e.g. remove the first 5 characters of the name)
Scenario when this would be used?
Coming from macOs some files don't need extension to work.
When i have to rename files from music library (or other similar scenario) where automatically the used program add track and album number to the file.
Supporting information
I have no other information.
Thanks for the attention

ThanksBut i’m italian 56 old man and I don’t know emglish well and my knowledge of computer Line command is very poor I’lll traing your suggestion .Il giorno 26 gen 2024, alle ore 17:25, PesBandi @.***> ha scritto:
You can put (.+) in the Search for field and $1.ext in the Replace with field. Make sure you have RegEx enabled. Then select the files you want to append an extension to on the right.
You can do ^.{n} or .{n}$ to trim n characters from the beginning or the end (see the last row of the second table here)
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Add GA information in README.md

These changes have been made to the README, except for Storage, which is still Beta. Closing this PR as it is now out of date.


[Feat] Support round_decimal in search page

Describe the feature:
Support round_decimal in search page.
Optional input.
Describe a specific use case for the feature:
User can define the score precision

Done


This would preferably just add to or remove from a list of blocks that can be replaced in such a way.
Options: false, all_flowers, one_tall_flowers

Rule name: replaceableFlowers
Description: placing blocks on flowers will replace them like grass

As for the options, using an enum would be good. Much like how we already use one for the movableEmptyEndPortalFrames rule.


add .npy reader to builtin reader

Description
This PR addresses #2976

Hmm test is failing, not quite sure why - maybe @tlambert03 would see based on a quick look

Hmm test is failing, not quite sure why - maybe @tlambert03 would see based on a quick look

because utils.io.imsave (used in the test here to save a temporary npy file) doesn't know how to save npy files ... use np.save instead?

because utils.io.imsave (used in the test here to save a temporary npy file) doesn't know how to save npy files ... use np.save instead?

Thanks @tlambert03 that did the trick. I will merge this now so I can include it in the next rc, thanks for the contribution @thanushipeiris!!


Do not clear highlights on hover end during insertion

Problem:
When dragging to insert, if you drag past the boundary of the scene you will end up targeting the canvas itself.
Fix:
Although we were correctly preventing highlighting new elements during insertion when hovering over them, we weren't doing the same when ending the hover, making it possible to unintentionally clear the highlighted element and then never replace it.

please add a test for this bugfix!


Observer holding on to old queryClient when client changes in QueryClientProvider

Describe the bug
In our app we allow user to switch between workspaces, and when they do we want an entirely different set of query cache / query client combo to isolate the data between workspaces. However simply changing the client provided to QueryClientProvider does not cause queries to refetch. Even manually calling refetch doesn't fully resolve the issue because the Observer has a constant reference to the previous QueryClient that does not update.
See https://github.com/trpc/trpc/issues/4153#issuecomment-1562071359 for additional context
Your minimal, reproducible example
https://github.com/tonyxiao/next-turbopack-example/blob/main/packages/app1/app/debug/page.tsx
Steps to reproduce

Type into the slug field to cause queryClient to change
Notice that value of res.data stays the same

Expected behavior
I would expect 1) All the queries to refetch and 2) Either new observers to be created or for the existing observer to update their query client reference.
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
All platforms
Tanstack Query adapter
None
TanStack Query version
4.29.7
TypeScript version
No response
Additional context
No response

Looks like a duplicate of https://github.com/TanStack/query/issues/5443 @tonyxiao check this issue.

@artem-malko I don't think they are related.
@tonyxiao I don't think creating a new QueryClient inside useMemo is a good idea. useMemo is a perf optimization, not a semantic guarantee that the code will only be executed if the dependencies changes. So you can wind up with random new QueryClients even though you didn't expect them to be recreated. It's a dangerous path.
I also don't think we support swapping out queryClients like that. What you can do is just call queryClient.clear() to clean it when a workspace switch happens. Otherwise, what also works is to add the workspaceId to all your Queries as a QueryKey, like:
['workspace', id, 'user', id]

etc to avoid sharing data. This is what we do, so that we can keep data cached between workspace switches.
Nuclear option would be to assign a key to your component above the one that creates the QueryKey to have react un-mount and re-mount the whole tree. Like:
<MyApp key={workspaceId}>

This would then create a new QueryClient, and there wouldn't be any references of old ones left.

Got it, thanks so much for the workaround recommendations too. We are using tanstack/query through trpc so there is not as much ability to manually control the query key unfortunately. (see https://github.com/trpc/trpc/issues/4153)
What about using queryClient.invalidateQueries to invalidate all? clear would only get rid of the cache but wouldn't cause any refetch to happen right?
Finally if swaping queryClient is not a supported behavior in the QueryClientProvider, could we add either a warning or throw an error in the provider component?

Oh side note, is there an alternative to useMemo you'd recommend that does have the semantic guarantee? A cursory search found this repo (https://github.com/alexreardon/use-memo-one) but it doesn't seem to be maintained.

What about using queryClient.invalidateQueries to invalidate all? clear would only get rid of the cache but wouldn't cause any refetch to happen right?

invalidate doesn't remove though. it only refetches active queries and marks others as stale. What you are likely looking for is queryClient.resetQueries to reset all back to their initial state, then trigger a refetch of active observers.

Finally if swaping queryClient is not a supported behavior in the QueryClientProvider, could we add either a warning or throw an error in the provider component?

if you want to contribute a dev only warning, sure, as long it doesn't affect bundle size of production builds.

Oh side note, is there an alternative to useMemo you'd recommend that does have the semantic guarantee?

useState: https://tkdodo.eu/blog/use-state-for-one-time-initializations

I also don't think we support swapping out queryClients like that.

Just wanted to note this really is not very good; a React component is expected to support handling a change to its props gracefully. If this is really not supported, it's worth warning about in console — but also it would be great to make this supported.

For posterity, the solution we went with was placing a queryClient in state for the tree below, and forcing a remount with key when the user ID changes so that the entire tree below reinitializes with the new client (https://github.com/bluesky-social/social-app/pull/3333). This avoids this problem at the cost of remounting the UI (which for switching users is probably a good idea anyway — to clear out any lingering state — not sure about switching workspaces).

I want to reopen the discussion on memoization of client in Observer.
Currently, changing a queryClient will result in mix of options from and old client (the one Observer was mounted with) and any rerendered client.
As queryClient in Observer is only used to get cache, all the bugs should be quite similar to the one in this issue.
With a simple fix there seem to be no breaking change* and the bug is eliminated.
// packages/query-core/src/queryObserver.ts#136
setClient(client: QueryClient): void {
  this.#client = client
}
// packages/react-query/src/useBaseQuery.ts#95
React.useEffect(() => {
  observer.setClient(client)
}, [client, observer])

*test 'should be able to persist into multiple clients' in react-query-persist-client fails because restoring happens in original client which is immediately swapped (I suppose that's a bug in tests though a subject to discuss)


⚠ SolarOscp has degraded performance

In 87906ba, SolarOscp ($SOLAROSCP_URL) experienced degraded performance:

HTTP code: 200
Response time: 318 ms

Resolved: SolarOscp performance has improved in 96374ee after 9 minutes.


[leptonica] update to 1.83.0

Library name:leptonica
New version number:1.83.0
Other information that may be useful (release notes, etc...)
Fix jp2k header reader for header offset larger than assumed 44 bytes

This issue has been fixed by PR https://github.com/microsoft/vcpkg/pull/29266, please update your vcpkg and build new version of leptonica.


fixing adding duplicate relation during installation of custom fields

fixed https://github.com/mollie/Shopware6/issues/579
During installation of the plugin the creation of the custom fields fails on a unique constraint because the relation between the custom field set and the customer entity is added twice.
This PR removes the second relation and fixes installation routine.

Shure,
i just install the plugin via the admin ui or the command line in a plain, fresh Shopware 6.
And if i check the custom fields after the plugin installation and activation only the mollie_payments_product custom field set is added. The mollie_payments field set and the fields the CustomFieldsService should add, are missing.
I then startet to dig into the problem and was able to finde the unique constraint violation caused by adding the two relations for the field set and the customer entity.

Hi
alright thank you
got it..so i dont know that CustomFieldService....I only did the CustomFieldsInstaller
the class you have touched uses random hex ids...wouldnt that recreate new fields over and over on additional installations?
thats why i did add fixed IDs for upserts in the other class
i have to re-check this :)
did you try to install a couple of times? do they exist multiple times then?

i get this error with your code when uninstalling and installing again in the admin

I always use a blank installation and have not tried to install/uninstall multiple times. But this could be fixed i think by either removing the fieldset on uninstall, or by checking for existing fieldset before reinstalling.

Ok i got it
there was a try catch, so this wasnt even inserted, and im not quite sure if its even good to have it :)
so ill check it out, thanks a lot for this

frankly i am not sure how this ever worked in the first place.
i found checked the problem was introduced with version 1.2 i think, when the third field (ideal prefered bank) was added.

yeah but that should also work without a custom field set. just plain json storing. and we also have custom displays in the admin. so if that upsert did never work. then nobody needs it 🥹

removed code is good code ;)
maybe the best solution in that case to remove the class completely.
Would also solve the install/uninstall issue.

hey @Lapsus thank you for the PR and showing the issue. as discussed with @boxblinkracer  everything is clear now so i can close the issue.


Publish 03/25/2021, 3:30 PM

Docs Build status updates of commit 259c77e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

:warning:Warning

Details

exchange/exchange-ps/exchange/New-Label.md
:white_check_mark:Succeeded
View (exchange-ps)

teams/teams-ps/teams/Get-Team.md
:white_check_mark:Succeeded
View (teams-ps)

Line 0, Column 0: [Warning-validation-incomplete] Failed to get the validation ruleset and validation was not completed. This happens when there's an issue with the service and continuing to retry the call could cause build delays. You might have content issues that were not reported. To retry validation, close and re-open your PR, or rebuild your branch via Docs Portal (requires admin permissions). If you need admin help or if you continue to see this message, file an issue via https://SiteHelp.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

  Invalid base branch - PR blocked
The target (base) branch - live - you're trying to merge this pull request into only accepts pull requests from the main branch.
If you're trying to publish changes you've made to the live site, you must first merge this pull request into the main branch. After this pull request is merged into the main branch, your changes will be published live at the next scheduled publish time. For information about the Marvel publishing process, see Marvel PubOps publishing process.
To change the target branch from live to main, use the following steps:

Select the Edit button next to the pull request title.

Select base:live.

In the list that appears, select main.

Select Change base.

NOTE Clicking Change base is safe. The message is specific to more complex branching methods that aren't typically used in Marvel repos. Even with those methods, this message only applies to how commits are tracked in the repo's history. There is no risk of data loss.


Init

Updated network.params.yaml
Added a new README.md

Could you please explain why you created a new readme?
What is the point of enabling certain services as default?


Question about password hashes

So I'm unsure where to post this, so this seems like as good of a place as any. I understand that a large amount of passwords are hashed in SHA-1, but you also have NTLM hashes. However, you don't explicitly specify whether these are hashed via LMHash or MD4 (NTHash). I ask because I'd like to use the full compliment of passwords, but to do that I obviously need to know all the hashing algorithms to use. Sorry if this isn't the right forum to use.

It should be LM. Try this:

Check "P@ssw0rd" here and see 83,846 results: https://haveibeenpwned.com/Passwords
Convert "P@ssw0rd" to an MD4 hash and get "E19CCF75EE54E06B06A5907AF13CEF42" here: https://suip.biz/?act=ntlm-hash-generator
Query the API for that hash prefix here: https://api.pwnedpasswords.com/range/E19CC?mode=ntlm
See the hash suffix with 83,846 results

@troyhunt Thanks! What's the :x stuff at the end of hashes? Like :4?

Refer to the documentation here: https://haveibeenpwned.com/API/v3#SearchingPwnedPasswordsByRange


memorize the last choice of auto save

File > Auto Save can switch between off and latest used item instead of afterDelay

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Surya scans HTTP Error 404

Steps to reproduce
Update any entry from Surya Scans
Expected behavior
Should update normally
Actual behavior
Says HTTP Error 404. When you click on webview it says the page is not found. When you go on the extention tab and click webview it redirects you to suryatoons. Maybe change the sourece?
Crash logs
No response
Mihon version
0.16.1
Android version
Andriod 12
Device
Redmi Note 9
Other details
The extensions repository takes me to the tachiyomi one which is closed
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.1.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Mihon doesn't provide support for extensions, go ask the people you got the extensions from.

I just used a backup from tachiyomi and uploaded it to mihon today. And since they have ceased development, what should I do?

I am very new to all this

I did not get it from anyone specific, I just downloaded the extension when I had tachiyomi and when I ported the backup to mihon, it came with it. Could you please give me any advice on how to do this myself? If you cannot.

If this is a legacy Tachiyomi extension, you will need to find a third party repo that provides fixed and up to date extensions, that is all I can tell you.


Add support for RM uneven shard width for FD2 reads/writes

Current code assumes page sizes are always equal. True for any tile sharding, but not for width/block sharding for RM tensors.

@tarafdarTT - is this something you can help with?

@smehtaTT is this blocking someone at the moment? I can work on this if this is priority

I'm currently working on reshard on supporting diff widths RM. Is that related to this?

This is unrelated, this is for FD2 host code reading and writing sharded buffers, which I am familiar with/implemented


Fix view tool product error

fixes https://github.com/DefectDojo/django-DefectDojo/issues/6111
'view_tool_product' not found when I added a tool or clicked on view tools on product dashboard settings.

Not sure what happened with this view, and if there is a testcase missing, but removing this commented out code resolves the error from https://github.com/DefectDojo/django-DefectDojo/issues/6111 @StefanFl @kiblik

It's commented in html, the Django template still is trying to render it.


Disable email button for installation shots

If you're viewing a show's installation shots, you can select images to send in an email but you're not allowed to compose anything. Should be disabled here

Also a problem here with documents

Also disable add-to-album

Think a better approach is to fix selecting documents and install shots, but to make sure the "compose" email button only becomes active when there are artworks or documents

And to disable select-all in installation shots

found a related / different issue in https://github.com/artsy/energy/issues/217 but closing this


Add support for the new Select Entity

Proposed change

Adds support for the new select entity type which provides a dropdown box in the interface for selecting one of the given options.
Would be handy to provide quick ways of modifying the behaviour of apps from the interface.
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (which adds functionalit)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[x] The code change is tested and works locally.
[ ] Local tests pass.
[ ] There is no commented out code in this PR.
[ ] The code has been formatted using Black (make black)
[ ] Tests have been added to verify that the new code works.

Thanks alot. Sorry for missing this one. Been away on vacation. @ludeeus is this OK to merge now?

@Laccolith Can you look at the merge conflicts?

I am merging this now. Think it went enough time passed to allwo 2021.7.0 be the required version,


Update autorest version to prerelease 3.0.0-alpha.20240607.2

Generated by Autorest Regen Preview build 20240607.2Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/4803

API change check
API changes are not detected in this pull request.


What should DNN's training data look like in the end?

Because of the privatization of your data, I understand, but one of my questions is, I don't know what the training data really looks like, a CSV file or image data? If it's a CSV file, what features should be included? Is it convenient to provide an example of training data, even if it's just one data, even if it's a fake number? It doesn't matter.
Thank you very much!!! I believe this is also useful for other people.

@roelWuyts
Ok, thank you!!!
If the input data is an image, then what do I use to divide the label of the image? Supervised learning should correspond to a real label. Finally, how do I predict whether an image is active or inactive?

Sorry, can't help you there since I did not do the machine learning, merely making sure the data was there. You should contact our colleagues from the KU Leuven for this (if this is still regarding the results from the paper).


#323 #121 search modal improvements

Fixes #323
Fixes #121
Scroll works on the list.
Looks more or less as follows (Before I removed a bit of the horizontal padding):

Should test that it works on multiple, different devices. Across different android versions.

The code looks good, but:

All works fine after going in and out of the app though.

also:

Tested with a production build, all work fine apart from login modal

What's exactly wrong? In my testing you could log in okay. There has been an issue where the login modal wouldn't go away, just eternally saying "Logging in", though.

Are you testing on device? Try entering invalid username and password. You should see a screen as per above.. this is the first screen that the user sees, so should be glitch free

Oh sorry yea, I remember what that's depicting now: The black bar at the top is because the modal gets squished and when keyboard dismisses it doesn't grow 😕

I actually fixed this @andreievg, classic flexbox

@Chris-Petty, happy to merge this.

I'll have to do a little today to make sure it's utilised in the new stocktake opening check modal, will bump then.

@andreievg Didn't change much, should be good to go


Inconsistent behaviour on IN operator when used over arrays

Having the following structure and data:
CREATE TABLE test_array_ops(arr Array(Nullable(Int64))) ENGINE = Memory;

INSERT INTO test_array_ops(arr) values ([null, 10, -20]);
INSERT INTO test_array_ops(arr) values ([10, -20]);
INSERT INTO test_array_ops(arr) values ([]);

Observations: we need cast in all following queries because of https://github.com/yandex/ClickHouse/issues/2026
SELECT count(*) FROM test_array_ops where arr < CAST([10, -20] AS Array(Nullable(Int64)))

returns 1 (OK)

SELECT count(*) FROM test_array_ops where arr > CAST([10, -20] AS Array(Nullable(Int64)))

return 1 (OK)

SELECT count(*) FROM test_array_ops where arr >= CAST([10, -20] AS Array(Nullable(Int64)))

return 2 (OK)

SELECT count(*) FROM test_array_ops where arr <= CAST([10, -20] AS Array(Nullable(Int64)))

return 2 (OK)

SELECT count(*) FROM test_array_ops where arr = CAST([10, -20] AS Array(Nullable(Int64)))

return 1 (OK)

SELECT count(*) FROM test_array_ops where arr IN( CAST([10, -20] AS Array(Nullable(Int64))), CAST([null,10, -20] AS Array(Nullable(Int64))))

Code: 53, e.displayText() = DB::Exception: Type mismatch in IN or VALUES section. Expected: Int64. Got: Array, e.what() = DB::Exception

Basically IN (a,b,c) should behave like X = a or X = b or x = c which here is not the case.

6452a3bb768712505b2cd74390cb0727fc29610e
30dd11f8caf9eae20926335f16231132424f6f7f

Fixed in master.
Added your example as a test case.


Toggle bufferline?

Hi- is there a way to toggle the bufferline on and off, on the fly, sort of?
If I set it to off in my vimrc:
let g:bufferline_echo = 0
Can I then toggle it back on when I need it? Only, I have too many buffers open and when the bufferline is displayed I have to stop and press enter (or Esc) to move on to the next one with Ctrl + PageUp/Down. It would be nice to know how to display the bufferline only when I want it.
Cheers!

Someone set updatetime to 400ms but I've found it hard to see command's result. So maybe a toggle options to manually on/off feature maybe good. OR option to set a delay after result of a command like 4s otherwise default to 200ms for example.


Unable to locate Microsoft.CodeAnalysis.CSharp during dnu restore on Mac OS X

Upgraded to the latest unstable build 1.0.0-beta7-15019 and attempted to dnu restore the HelloMvc app in samples\Home. Followed suggestions made here https://github.com/aspnet/Home/issues/810 but I'm still unable to dnu restore on Mac OS X.
On the myget feed, Microsoft.CodeAnalysis.CSharp currently points to a later version, namely v1.1.0-beta1-20150817-04.
Below is the error summary from a dnu restore call:
Errors in /Users/miguellira/Documents/Github/Home/samples/latest/HelloMvc/project.json
    Unable to locate Microsoft.CodeAnalysis.CSharp >= 2.0.0-beta1-20150806-01

NuGet Config files used:
    /Users/miguellira/.config/NuGet/NuGet.Config
    /Users/miguellira/Documents/Github/Home/samples/latest/NuGet.Config

Feeds used:
    https://www.myget.org/F/aspnetvnext/api/v2/
    https://nuget.org/api/v2/
    https://www.myget.org/F/roslyn-nightly/
    https://www.myget.org/F/dotnet-corefx/

Try to change you nuget.config like in this commit: https://github.com/aspnet/dnx/commit/f3de70abcf1c455a3a6723aba680a37091c0f0cf
Also i think you don't need this two feeds:
https://www.myget.org/F/roslyn-nightly/
https://www.myget.org/F/dotnet-corefx/

@causer Thanks. I'm able to restore/build/run HelloMvc. Running into a runtime issue, but I'll dig through the recent Announcements and see what's changed.
For the record, the issue I referenced (https://github.com/aspnet/Home/issues/810) suggested the addition of those two feeds which is why I added them. However, after updating my vNext feed to "aspnetlitedev" as the link you provided suggested I was able to restore.
My NuGet.config is currently set to the following:
<?xml version="1.0" encoding="utf-8"?>
<configuration>
  <packageSources>
    <!--add key="AspNetVNext" value="https://www.myget.org/F/aspnetvnext/api/v2" /-->
    <add key="AspNetVNext" value="https://www.myget.org/F/aspnetlitedev/api/v2" />
    <add key="NuGet" value="https://nuget.org/api/v2/" />
    <!--
    <add key="Roslyn" value="https://www.myget.org/F/roslyn-nightly/" />
    <add key="DotNet" value="https://www.myget.org/F/dotnet-corefx/" />
    -->
  </packageSources>
</configuration>

For the record, looks like beta8 just dropped and it addresses all of the restore/build/run issues I encountered with beta7.
Curiously, at the moment running dnvm upgrade states:
Latest version is 1.0.0-beta6

yet running dnvm upgrade -u brings down 1.0.0-beta8-15120 and with this DNX version I am able to restore/build/run Home/samples/latest/HelloMvc without any issues.
In fact, the restore process only uses the two well known/documented feeds:
Feeds used:
    https://www.myget.org/F/aspnetvnext/api/v2/
    https://nuget.org/api/v2/

No need to add the "aspnetlitedev" feed previously discussed. I'll keep this open for a day in case others stumble upon this scenario.