Could not purchase at all

When attempting to purchase I won't take any money or give anything

I were not able to reproduce the bug
Can you try to setup a trade sign see if it works?
Can you check your economy plugin is present and correctly loaded?

A trade sign does work. I am quite sure my economy plugin is working

So you can use /bal and /econ?
Can you give me your plugin list?

Yes, Total Economy,Gp,Holograms,luckperms and nucleus

And a custom

I am unaware if this is the same issue, but when players on my server go to buy some items, despite having enough items in their inventory to sell, it will say "You do not have enough items to sell." It seems to occur randomly, not all the time.
Plugins include: Economy Lite, Enjin Minecraft Plugin, GP, Kai's Commons, LuckPerms, MCClans, Nucleus, SimpleAntiAFK, VoxelSniper

This is a different problem.
About the 1st problem: I do not remember if I tested with totaleconomy. I'll do that asap.
About the 2nd problem, can you open a new issue? Let me know there what the items are.

Please test Total Economy as it is the only good Economy Plugin I found for sponge.

Hi,
I tried with Total Economy 1.7.1 and had no problems
I released v1.17 is it still occuring?
If yes, can you provide step by step actions for me to reproduce the problem?

is the shop's position near the world spawn point and less than 16 block?
the spawn point might be protected by server.

No answer. Closing. Feel free to reopen if you still have troubles


Fix/splashscreen

I moved all of the link tags to the end of the body in index.html so that all of the style sheets and scripts would load after the splash screen is displayed. I think the inconsistency of the splashscreen was caused by some things being loaded before and some things being loaded after the splash screen is shown. It seems to load the splash screen and display the image consistently now.
Also, I fixed the alignment of some tags which is why the diff is so large.
Resolves #331

True that.. merged manually! +5 👍🏻


Location Rule not working

Story
A user on Slack is trying to use the Location Scanner with Forseti v2.25.0 on GKE. The following rule is not producing violations even though they have buckets in the US:
rules:
   - name: All buckets in organization must not be in the US.
     mode: blacklist
     resource:
       - type: organization
         resource_ids:
           - {ORG_ID}
     applies_to:
       - type: 'bucket'
         resource_ids:
           - '*'
     locations:
       - 'us*'

Troubleshooting
Originally the rule was configured at the project level and was using a wildcard for the resource/resource_ids. The user tried changing it to resource/type = organization and using their actual Org Id for the resource/resource_ids. @gkowalski-google was able to reproduce the issue (on GCE), but the changes mentioned above resolved it. I have asked the user to check for any errors in Stackdriver.

Possibly related to #3551

v2.25 is at the pre-release stage, i will follow up with the user and see if he tried out the older version.


Only works in preview

Hi, thanks for the cool plugin!
I noticed that clicking on links in edit mode does not open them in the selected browser
I'm using the latest version of Obsidian on a Mac

Hi, sorry for the late reply to you.
This plugin currently only works in preview mode, and I agree that it needs to work in edit mode as well. But after some attempts, I found it kinda different. In my attempts, the web page opens in both the default browser and the browser configured in the plugin simultaneously after clicking(with modifier key) on the link.
The principle behind this is that the link in edit mode is actually not a hyperlink, obsidian using inner function to handle the click event, which is unable to be hijacked(or disabled) by the plugin. So this function can be implemented, but in a broken way. The alternative solution is to use a different modifier (alt or shift) to start the click event, but this can feel strange and cause potential conflict problems.
Anyway, I'll add the modifier method to the plugin, at least it's better than not having it at all.

By upgrading the plugin to version 0.1.4, the browser's override behavior will work properly in edit mode.

@MamoruDS
With the version 0.1.5, this plugin seems that still only works in preview mode, and it would prevent opening links in edit mode.

Could you help to check that? Not sure if it is a personal problem. BTW, thanks for building this cool plugin.

@MamoruDS
Thanks for your plugins!
Same problem that could not opening links in edit mode(live preview), but I solved it by adding some code based on your main.js code.
I just learned js,  maybe there are some bugs. Share with you @XMB-7. you can try it!

Still doesn't work in the preview mode.
Version 0.1.8.

+1

Hi,
I confirm clicking on links directly has no effect in live preview. However, I noticed that unwrapping the link (with Alt + Click) to show the URL and clicking on the URL does open it, whereas it should not (at least, it does not without the plugin). I'm really no JS specialist, but maybe the event is captured at the wrong moment or something ?...
Thanks for the plugin anyway !

I believe I have a solution to this issue. The root cause of this problem is my insufficient study of Obsidian click actions, from my testing, Obsidian's click actions can be summarized in the following table:

Ctrl
Alt
Shift
Edit
Edit (live)
Read

cursor
open
open

🖱

open
open (new panel)
open (new panel)

🖱

cursor
cursor
open

🖱
cursor
cursor
open

🖱
🖱

open (new split)
open (new split)
open (new split)

🖱

🖱
open (new panel)
open (new panel)
open (new panel)

🖱
🖱
cursor
cursor
open

🖱
🖱
🖱
open (new window)
open (new window)
open (new window)

The appropriate implementation should follow Obsidian's default click behaviors, I will give a quick fix on this

Hey, I just released version 0.1.9 of this plugin which should fix the issue of links not working in live preview mode. And also, if you're using an in-app view as your link opener, all modifiers (combinations) should now work like in Obsidian's default. But keep in mind that, since the plugin's current setting only support one modifier binding and modifiers can be combined in different ways, I'll make it more flexible in the future version. Also, some custom modifier bindings won't work in this version since they are used for cursor move/selection which are treated as non-clickable. Please let me know if there are any other issues

Fix confirmed, thanks for the great work!

It works great indeed, thanks a lot !


For Respeaker 2-mic voice card, added some descriptions and configuration.

Thank you for this fork.
It works great on my Raspberry Pi 3b+ (buster) with MagicMirror 2.8.0 and the Seeed Respeaker 2-Mics hat. I only had to disable the internal sound chip in /boot/config.txt (changed dtparam=audio=on to off) because else the Respeaker's device names "Speaker" and "Playback" would not be recognized.

I'll close this PR(due to different coding styles), but your contributed codes are applied on main branch.
Thanks very much.

I can't maintain this module any more.
Have you interest to take new ownership of this module?
https://forum.magicmirror.builders/topic/12779/wanted-new-ownership-of-modules?page=1

Sorry for bothering. This module found new owner now. In the meantime, thanks for everything. I’m quitting.


HfApi method to fetch "liked" models by current user

Is your feature request related to a problem? Please describe.
The huggingface dashboard seems to only list the last 20 "liked" models and datasets for my account. I use the "like" button extensively for bookmarking models but recently realized I can only view a limited number. Adding an api method would also help for analysing a user's interests.
Describe the solution you'd like
Something like this perhaps?
HfApi.get_liked(
  username: Optional[str] = None,
  auth_token: Optional[str] = None,
  entity_type: "all" | "model" | "space" | "dataset" = "all",
  max_items: int = 100
) -> List[ModelInfo|SpaceInfo|DatasetInfo]

Hi @creatorrr , thanks for creating an issue for this. It is currently not possible to do that from huggingface_hub as you spotted. @osanseviero just created an internal server-side issue.
As for the API design if/when we integrate this to the Hub, I'd be in favor of extending the list_models, list_datasets and list_spaces methods. We can add a ~is_liked boolean argument (defaults to False)  that does the filtering.

@creatorrr that's a good feature (and good to know you're using likes extensively), we'll prioritiez

@Wauplin makes sense. If an API endpoint does become available, please let me know @osanseviero ! I would be happy to use that in the meantime.

Any update on an API endpoint, @osanseviero ? Hate to bother again, would be great if there was a public trello or something. :grimacing:

yes our backlog is LIFO =)
more seriously: coming soon!

Looks like it was a good idea to post a reminder @camenduru ! :stuck_out_tongue: :rocket: => https://github.com/huggingface/huggingface_hub/pull/1254

Thank you guys! Really appreciate this :)

@creatorrr @camenduru It's merged ! Feel free to use it from main branch :)
# !pip install git+https://github.com/huggingface/huggingface_hub.git@main

from huggingface_hub import HfApi

api = HfApi()
api.like("bigcode/the-stack", repo_type="dataset")

likes = api.list_liked_repos()
likes.datasets
# ['bigcode/the-stack', ...]

thanks @Wauplin ❤


Adding changelog for v0.10.1 (#2690)

Adding changelog for v0.10.1

Removing issue fix from change log

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/unhold
/approve
/lgtm

/hold

/unhold

/lgtm


Rails Engine support

I've been experimenting with a rails Engine, and specs for it. So my structure looks like:
/thing/app/models/blah.rb
/thing/spec/spec_helper.rb
/thing/spec/models/blah_spec.rb

/thing/feed/app/models/foo.rb
/thing/feed/spec/spec_helper.rb
/thing/feed/spec/models/foo_spec.rb

When I hit CMD+R from inside foo_spec.rb, with require 'spec_helper' at the top, it loads /thing/spec/spec_helper.rb instead of /thing/feed/spec/spec_helper.rb
Note that ENV['TM_PROJECT_DIRECTORY'] is "/thing" and as far as I can tell, that gets used to decide on load paths of things from thereon out.
What should I do here? Any obvious suggestions or should I do some digging?

Hm … does this work correctly when you run the spec file from the command line?

Yes so if I'm in the /thing/feed folder, and I run rake spec or rspec spec/models/foo_spec.rb then it runs fine.

I see. Do you have the current version of the bundle? Because since https://github.com/rspec/rspec.tmbundle/commit/a81c3f9c7395d72616ddc82ce5963c113334f7b5 RSpec is always run via shelling out, so I’m a bit puzzled that it works when run directly from the command line.
Could you provide a stripped down archive / repo of the file structure causing the problem so that I can easily reproduce it?

https://github.com/rspec/rspec.tmbundle/commit/26d04c is the commit that I'm on, which is the latest.
Remember it runs those specs because in the shell I'm in a sub-folder with its own Rakefile and its own spec_helper.rb.
Here's a trimmed down example: https://dl.dropboxusercontent.com/u/2415512/rspec_engine_test.zip
If you cd into /testapp/feed and run rake spec the specs will pass. Open up /testapp/feed/spec/models/feed/video_spec.rb and run it within TextMate and it will fail because it's trying to load the project root's environment.

Thanks for providing the example!
Currently the bundle assumes that the TextMate project directory is the root of everything RSpec related – so RSpec will always be run from that directory, all paths will be interpreted relative to that directory and so on.
A simple (but rather unpractical) workaround is to set the project directory in TextMate to testapp/feed: Then all the engine’s specs work correctly (but you won’t be able to run the specs of the main Rails app anymore).
A possible solution would be to allow for overriding the “rspec base directory” via .tm_properties: You’d need to create testapp/feed/.tm_properties with the following contents:
TM_RSPEC_DIRECTORY=$CWD

The bundle would be changed to use ENV["TM_RSPEC_DIRECTORY"] as base directory if present (else falling back to ENV["TM_PROJECT_DIRECTORY"] as before).
Do you think this solution is worth implementing?

That would be perfect! We could do lots of clever things to figure out the right base directory, but while it might work for examples we come up with, it won't work for the next person because their situation will likely be too different. This gives exactly the control we need to set it when appropriate without any ambiguity.

I just pushed https://github.com/rspec/rspec.tmbundle/commit/41cf1b2cf84a9579c04da658857560ad1ec20551 – please try it out and let me know if it works for you! (Note that the variable is now named TM_RSPEC_BASEDIR).

There was still a bug that should now be fixed (see https://github.com/rspec/rspec.tmbundle/commit/d229b8e04f30f0fc0e7815eaff376a2de92fb640).

Closing –  please reopen if you still notice something not working as expected!


Clicking the canvas does not always close the details panel

If you do not click perfectly (ie your mouse is not moving at all) on the canvas, the details panel does not go away. Some times it takes a couple clicks in order for the details panel to go away.

The root cause was the isZooming flag. D3 registers drag events as zoom events and was setting the flag if the mouse click wasn't perfect.


Endless "Reading meta files ..." when switching song folder

Switching the song folder to a new folder with some song in it, brings the syncer into endless "Reading meta files ..."
In my case there are 5 songs in the new folder.

What version are you running?
I assume there's a hidden error in the background task. With #254, it should be properly propogated, which should tell us what the underlying issue is.

@RumovZ i use the released 0.5.0 version

2024-07-01 19:34:38 [ERROR] Uncaught exception:
    Traceback (most recent call last):
      File "F:\Games\UltraStar\archiv\usdb_syncer\src\usdb_syncer\gui\mw.py", line 170, in _select_song_dir
    run_with_progress(
      File "F:\Games\UltraStar\archiv\usdb_syncer\src\usdb_syncer\gui\progress.py", line 42, in run_with_progress
    raise cast(Exception, exception)  # pylint: disable=raising-bad-type
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      File "F:\Games\UltraStar\archiv\usdb_syncer\src\usdb_syncer\gui\progress.py", line 32, in wrapped_task
    out = task(dialog)
          ^^^^^^^^^^^^
      File "F:\Games\UltraStar\archiv\usdb_syncer\src\usdb_syncer\gui\mw.py", line 172, in <lambda>
    lambda _: song_routines.synchronize_sync_meta_folder(path),
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
      File "F:\Games\UltraStar\archiv\usdb_syncer\src\usdb_syncer\song_routines.py", line 101, in synchronize_sync_meta_folder
    SyncMeta.upsert_many(to_upsert)
      File "F:\Games\UltraStar\archiv\usdb_syncer\src\usdb_syncer\sync_meta.py", line 175, in upsert_many
    db.upsert_resource_files(
      File "F:\Games\UltraStar\archiv\usdb_syncer\src\usdb_syncer\db\__init__.py", line 573, in upsert_resource_files
    _DbState.connection().executemany(stmt, (p.__dict__ for p in params))
    sqlite3.IntegrityError: NOT NULL constraint failed: resource_file.resource

Thanks!
I think we had a similar report before. An early version of the Syncer didn't enforce a resource name.
How do you go with 2ed328d?

yes that solves the problem


fix build errors

Docs Build status updates of commit bde98de:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: PullRequestMergeConflict] Cannot sync git repo to specified state. Please make sure the pull request is mergeable.

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

Docs Build status updates of commit 5dd44ad:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Language/Reference/User-Interface-Help/addfolders-method.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


IOS compile errors

Hello. I'm using the latest version of ionic framework (1.2) and this plugin does not compile for IOS. Here are the errors:
/Users/vincentcrepin/ionicProjects/BlitzClock/platforms/ios/BlitzClock/Plugins/at.gofg.sportscomputer.powermanagement/PowerManagement.m:39:28: error: no visible @interface for 'CDVPluginResult' declares the selector 'toSuccessCallbackString:'
jsString = [result toSuccessCallbackString:callbackId];
/Users/vincentcrepin/ionicProjects/BlitzClock/platforms/ios/BlitzClock/Plugins/at.gofg.sportscomputer.powermanagement/PowerManagement.m:43:28: error: no visible @interface for 'CDVPluginResult' declares the selector 'toErrorCallbackString:'
jsString = [result toErrorCallbackString:callbackId];
~~~~~~ ^~~~~~~~~~~~~~~~~~~~~
/Users/vincentcrepin/ionicProjects/BlitzClock/platforms/ios/BlitzClock/Plugins/at.gofg.sportscomputer.powermanagement/PowerManagement.m:46:11: error: no visible @interface for 'PowerManagement' declares the selector 'writeJavascript:'
[self writeJavascript:jsString];
~~~~ ^~~~~~~~~~~~~~~
/Users/vincentcrepin/ionicProjects/BlitzClock/platforms/ios/BlitzClock/Plugins/at.gofg.sportscomputer.powermanagement/PowerManagement.m:63:28: error: no visible @interface for 'CDVPluginResult' declares the selector 'toSuccessCallbackString:'
jsString = [result toSuccessCallbackString:callbackId];
~~~~~~ ^~~~~~~~~~~~~~~~~~~~~~~
/Users/vincentcrepin/ionicProjects/BlitzClock/platforms/ios/BlitzClock/Plugins/at.gofg.sportscomputer.powermanagement/PowerManagement.m:67:28: error: no visible @interface for 'CDVPluginResult' declares the selector 'toErrorCallbackString:'
jsString = [result toErrorCallbackString:callbackId];
~~~~~~ ^~~~~~~~~~~~~~~~~~~~~
/Users/vincentcrepin/ionicProjects/BlitzClock/platforms/ios/BlitzClock/Plugins/at.gofg.sportscomputer.powermanagement/PowerManagement.m:70:11: error: no visible @interface for 'PowerManagement' declares the selector 'writeJavascript:'
[self writeJavascript:jsString];

I have same issue.  Any movement on this?

I have same issue too. Any movement on this?

I encountered the same problem and found a solution on this thread: https://stackoverflow.com/questions/28850016/how-do-i-replace-the-deprecated-methods-tosuccesscallbackstring-and-writejavascr

Apparently the code has already been fixed in this repository (see this pull request) but the NPM package hasn't been updated. So many people may have the broken version installed.
I suggest re-installing the plug-in via the cordova command-line instead of npm.
$ cordova plugin add https://github.com/Viras-/cordova-plugin-powermanagement.git

Confirming @andre-arsenault - installing from git repo works.
ionic cordova plugin rm at.gofg.sportscomputer.powermanagement
ionic cordova plugin add https://github.com/Viras-/cordova-plugin-powermanagement.git


Scene is hardcoded (input.txt unused)

Rework scene building and reintroduce a logical scene.txt configuration format

Resolved in 1648976ddc0810725e137a34fa5b0a8cc2b8c822


Add first overpass date as attribute to output

A feature I would find incredibly useful (albeit a static one) is appending an attribute to the output that contains the first date(s) in the science orbit that data is available for (if the waterbody is in SWORD/PLD). This way it would be very easy for folks to simply add the SWOT orbital length to those dates, to easily predict future overpass times, and get a sense of the SWOT coverage for that waterbody (e.g., number of overpasses).  Predicting these dates ahead of time are helpful for planning fieldwork comparing SWOT measurements to in-situ measurements, and helping out with better understanding multi-sensor fusion-- e.g., how often will we get overlapping SWOT/Landsat/Sentinel observations? We are currently storing this date information locally but having it in hydrocron would help us move towards relying on hydrocron for most if not all of our SWOT data needs.

Hi @merrittharlan, we were discussing this today and wondering if just knowing when the science orbit started would be sufficient, so that by including that as the start date in a query to hydrocron, whatever the earliest time returned would be the first date for that feature. Would that be sufficient? We could include a page in the documentation describing how to get those dates using Hydrocron for this use case. Otherwise, since this isn't something we can easily pull from the shapefiles it would be a bit complicated for us to calculate and figure out how to add say a flag to indicate that it's the first record.


feature: Add course to list of Enrolled courses

What does this PR do?

Endpoint to Add course to list of Enrolled courses
Unit tests for adding course to list of Enrolled courses
Endpoint to get subjects for a course
Unit tests for getting subjects for a course

Description of tasks completed
Have the following Endpoints working
GET /api/v1/courses/:courseId/subjects
POST /api/v1/courses/add-course  set headers "token": authToken
How should this be manually tested?

Clone this repository to your local machine
Run npm install to install node packages
Run npm run dev to start Server
Using Postman,  test the endpoint
Run npm run test to run unit test

I have squashed and solved the commit issue.

there is a conflict in auth.test.js. And i can't see that here


Stuck on I/flutter (20447): The Dart VM service is listening on ***** on IQOO Z5 5G [android 13] as in issue #82520

output of flutter doctor -v
l.log
output of  flutter run -v
log.log
as mentioned in the issue   #82520 some devs have solved this with the android 12 update on fontouch os but this error still happening on android 12 (Fountouch Os 13) please.
did not used any thrid party module just flutter create myapp --org com.my_domain
Operating System
OS: Ubuntu 22.10 x86_64
Kernel: 5.19.0-35-generic
output of neofetch

          .-/+oossssoo+/-.               nktech@hackerbook
      `:+ssssssssssssssssss+:`           -----------------
    -+ssssssssssssssssssyyssss+-         OS: Ubuntu 22.10 x86_64
  .ossssssssssssssssssdMMMNysssso.       Host: VivoBook_ASUSLaptop X412FAC_X412FA 1.0
 /ssssssssssshdmmNNmmyNMMMMhssssss/      Kernel: 5.19.0-35-generic
+ssssssssshmydMMMMMMMNddddyssssssss+     Uptime: 3 hours, 59 mins
/sssssssshNMMMyhhyyyyhmNMMMNhssssssss/    Packages: 3104 (dpkg), 36 (flatpak), 11 (snap)
.ssssssssdMMMNhsssssssssshNMMMdssssssss.   Shell: zsh 5.9
+sssshhhyNMMNyssssssssssssyNMMMysssssss+   Resolution: 1920x1080
ossyNMMMNyMMhsssssssssssssshmmmhssssssso   DE: Unity
ossyNMMMNyMMhsssssssssssssshmmmhssssssso   WM: Mutter
+sssshhhyNMMNyssssssssssssyNMMMysssssss+   WM Theme: Dracula-shell-v40
.ssssssssdMMMNhsssssssssshNMMMdssssssss.   Theme: Yaru-blue-dark [GTK2/3]
/sssssssshNMMMyhhyyyyhdNMMMNhssssssss/    Icons: Yaru-blue-dark [GTK2/3]
+sssssssssdmydMMMMMMMMddddyssssssss+     Terminal: vscode
 /ssssssssssshdmNNNNmyNMMMMhssssss/      CPU: Intel i3-10110U (4) @ 4.100GHz
  .ossssssssssssssssssdMMMNysssso.       GPU: Intel CometLake-U GT2 [UHD Graphics]
    -+sssssssssssssssssyyyssss+-         Memory: 3164MiB / 3775MiB

Hi @Nktech-Official, this is an issue with vivo devices and not something the flutter team can fix. See https://github.com/flutter/flutter/issues/117019 for a similar report of this issue.
Closing this based on https://github.com/flutter/flutter/issues/117019#issuecomment-1353683329.
Thank you


question about boundingBoxReference in factsheet topic

Hello, I encountered an issue during development and hope to get some clarification.
In the loadSpecification of the factsheet topic, there is a field called boundingBoxReference, and I am not clear on its purpose in this context. I understand that loadSpecification is used to describe load types, but why would a load type require coordinates?
I would appreciate it if you could explain the use case for this setting or tell me why coordinates need to be specified for load types. Thank you very much.

Hi. I had the same question which was solved in a recent discussion.
In the documentation it says, the boundingBoxReference "is described in coordinates of the AGV's coordinate system." It's easy to assume that the AGV's coordinate system is the same as the warehouse coordinate system because the AGV navigates through the warehouse.
However, what this actually means is: The boundingBoxReference is the position of the load relative to the agv itself.
At least this is what I understood in the discussion. It would make total sense contextwise.
Maybe it would be best to clarify the meaning of "AGV coordinate system" in the documentation to avoid this type of misunderstanding.
Greetings,
Florian Kutz

Thank you for your response.
I think the coordinates of the individual loads make sense (for AGVs). But this loadSpecification should describe the type of load such as plastic boxes, cartons? In my understanding, factsheet should be a place where fixed attributes are uploaded. I think it is more reasonable to upload business-level data, such as how many loads there are and the coordinates of loads, in state.


P2F bash mode does not work for secret names with dashes

##Bug description
See attached image for details.
##Found in version
12.4
##Workaround Complexity
NA
##Workaround Description
##Affects Version/s
##Link to JIRA bug
https://ca-il-jira.il.cyber-ark.com:8443/browse/ONYX-14674

https://ca-il-jira.il.cyber-ark.com:8443/browse/ONYX-14674

Dashes are invalid characters in Bash names, so this behavior is expected.
The error message for this failure was clarified, and this issue is closed by #416.


Is there any example of UVC video capture?

Just wondering if there is any example of standard UVC video capture in USB HOST mode.

Enterprise customization， not opensource.

Iso driver for video is not opensource, but there is a template for parsing video packet(if you can write iso driver), https://github.com/cherry-embedded/CherryUSB/blob/master/demo/usb_host.c

ok please add my qq<PHONE_NUMBER>


Add 'geiser-repl-buffer-name-function' variable

Hello, what about allowing users to change the REPL buffer name?

looks good to me. merged!

thank you!


Documentation for the CHIP jars

I can't find documentation for any of the CHIP API at developers.home.google.com, shouldn't it be listed here?
https://developers.home.google.com/reference/com/google/android/gms/home/matter/commissioning/package-summary
The API all appears to be there, but many of the necessary parameters are completely mysterious and undocumented.

Could you clarify what you mean by the CHIP API?
What's documented at https://developers.home.google.com/reference/com/google/android/gms/home/matter/commissioning/package-summary is the "Google Home Matter APIs".
The Matter SDK that we include with the sample app (under third_party/connectedhomeip) comes straight from the connectedhomeip github repo and documentation of these APIs should hopefully be provided there eventually.

There is no doc at connectedhomeip github for the Matter SDK, I am figuring this stuff out by reading the source code.  Any idea when a real doc team is going to document these APIs?

I'd suggest filing an issue with that very question at the connectedhomeip github repo.


Write-RsRestCatalogItem does not accept xslm files

I can upload a xlsx file with Write-RsRestCatalogItem but when I try it with a xlsm file I receive this error:

I can manually (in PBIRS) upload the xslm without any problems:

Is there a different function than Write-RsRestCatalogItem for xslm files? Or does the function needs an update to support xslm files?

Do you want to request a feature or report a bug?
BUG
What is the current behavior?
Uploading of xlsm files is not possible with Write-RsRestCatalogItem. Manual uploading to PBIRS is possible.
When I rename the file from .xlsm to .xlsx I can upload the file with Write-RsRestCatalogItem.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
PS C:> Write-RsRestCatalogItem -ReportPortalUri 'https://PBIRS-URL/Reports' -Path 'C:\Downloads\004 - Fehlteilliste Einkauf.xlsm' -RsFolder '/LWT'
Write-RsRestCatalogItem : System.Exception: Failed to create catalog item: The remote server returned an error: (400) Bad Request. ---> System.Net.WebException: The remote server returned an error: (400) Bad Request.
at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.GetResponse(WebRequest request)
at Microsoft.PowerShell.Commands.WebRequestPSCmdlet.ProcessRecord()
--- End of inner exception stack trace ---
At line:1 char:1

Write-RsRestCatalogItem -ReportPortalUri 'https://PBIRS-URL ...

  + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
  + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Write-RsRestCatalogItem

PS C:> Write-RsRestCatalogItem -ReportPortalUri 'https://PBIRS-URL/Reports' -Path 'C:\Downloads\004 - Fehlteilliste Einkauf.xlsx' -RsFolder '/LWT'
Same file - just renamed to xlsx gets uploaded successfully
What is the expected behavior?
That the xlsm file gets deployed to the PBIRS.
Which versions of Powershell and which OS are affected by this issue? Did this work in previous versions of our scripts?
PSVersion                      5.1.18362.1801
Script     <IP_ADDRESS>    ReportingServicesTools
PBIRS    Version 1.12.7977.29537 (September 2021)
OS of my laptop/notebook: Windows 10. the server runs windows server (dont know the exact version)

Hello? Any news on this bug? It has been over 1 year since I reported it.

Hello jtarquino, is there a way to get actual support from MS for this project? It is almost 1.5 years since I reported this bug and we still have to update xslm files manually on our PBIRSs, instead of with a Release Pipeline in Devops.

We welcome contributions from the community in order to enhance the functionality of the functions.
I would like to invite you to submit a pull request with the proposed changes and we will review it.

I do not know how to write code for your program.

The solution might be as simple as adding a line for .xlsm to the Get-ItemType function. It is not currently listed there.

The solution might be as simple as adding a line for .xlsm to the Get-ItemType function. It is not currently listed there.

Sounds good to me - I hope the developers find time to look at this eventually. But it seems there is almost no interest in Microsoft to work on this tool.

I know the feeling, it is very much "self-service" right now. I don't have the confidence in my Powershell skills to make pull requests myself just yet.

I had a go at updating the Get-ItemType function and it seemed to fix your issue - I was able to upload an XLSM file without any errors to our PowerBI server. I am going to have a go creating my first Pull Request now!


hotfix(prodpublick8s) migrate mirrorbits PV to native CSI, use a 1000 Gib size and fine-tune mountoptions

Current used disk space is 486 Gb which is over the 80% threshold with a 500 Gib volume => setting it to 1000 Gib
Switch to CSI-premium tier to ensure performances
Use CSI native options to eavoid any kind of "autoamtic" migration
Fine tuned the mount options: only the owner should be able to write

This PR is a hotfix related to https://github.com/jenkins-infra/helpdesk/issues/3525, so no checks as it was applied successuflyy from a local machine


Foreign key indexes are not well changed by prisma

Bug description
If I initially create a 1:1 relation and then need to change it to a many-to-many relation, the foreign key index does not change from unique to non-unique. And need to drop the table and then re-create, the foreign key index can be updated.
How to reproduce

Create a 1:1 relation
Run prisma generate && prisma db push
Change the 1:1 relation to Many-to-Many relation
Run 'prisma generate && prisma db push'
See error, the foreign key is still a unique index

Then I drop the table and run prisma generate && prisma db push, the foreign key can only be changed to non-unique indexes

Expected behavior
No response
Prisma information

1:1 relation(use Official document example)
model Post {
  id         Int                 @id @default(autoincrement())
  title      String
  categories CategoriesOnPosts?
}

model Category {
  id    Int                 @id @default(autoincrement())
  name  String
  posts CategoriesOnPosts?
}

model CategoriesOnPosts {
  post       Post     @relation(fields: [postId], references: [id])
  postId     Int // relation scalar field (used in the `@relation` attribute above)
  category   Category @relation(fields: [categoryId], references: [id])
  categoryId Int // relation scalar field (used in the `@relation` attribute above)
  assignedAt DateTime @default(now())
  assignedBy String

  @@id([postId, categoryId])
}

Then change to Many-to-Many relation(only "?" to "[]")

Environment & setup

OS:
Database:
Node.js version:

Prisma Version

How did you fix this problem @bigTeethLi?

Duplicate at https://github.com/prisma/prisma/issues/12288


System.InvalidCastException: Unable to cast object of type 'Newtonsoft.Json.Linq.JObject' to type 'Newtonsoft.Json.Linq.JArray'

Hi,
I'm unable to run the following code:
var findAccountsArgs = new FindAccountsArgs()
{
    Accounts = new[] { "my_login" }
};
_operationManager.FindAccounts(findAccountsArgs, CancellationToken.None);

It gives the error:

System.InvalidCastException: Unable to cast object of type 'Newtonsoft.Json.Linq.JObject' to type 'Newtonsoft.Json.Linq.JArray'

The error happens in the method:

\Sources\Ditch.Steem\Models\Other\LegacyAsset.cs

public void ReadJson(JsonReader reader, JsonSerializer serializer)
{
->    var arr = serializer.Deserialize<JArray>(reader);
    Amount = arr[0].Value<long>();
    Symbol = new AssetSymbolType(arr[2].Value<string>(), arr[1].Value<byte>());
}

I'm trying to get a single (existing) account as seen here:
https://github.com/Chainers/Ditch/releases
Versions:

Ditch 3.1.3 / 3.2-alpha
Newtonsoft.Json 11.0.2

I'm not sure if I made a mistake or not.
Thank you

Hi! Steemdev team is deploy a new node release on their servers. It has a new format for asset, which leads to the client's downfall :( Until the release becomes stable, there is no sense to updating the current client code. Now, only the condenser_api is stable on the node.

Thank you very much! :)
I highly appreciate your work and help.

v4.0.0 adds BroadcastOperationsSynchronousLikeSteemit function to solve this issue


The evaluation results with the first fashion for TAPIR

Hi, I found the evaluation results in the strided fashion in the paper, and I wonder whether you have the results with the first fashion？

The query first evaluation is shown in table 3. Please check it again.


How to run it on single GPU

I have a server with a single GPU
How can I run it on the server?

What is the memory size of your single GPU?


Variables in Rasa

templates:
utter_greeting:

text: "Hey, {name}. How are you?"
this is my template containing a variable 'name'. How do i get the value of 'name' from the user,so that it displays " Hey, xyz(the name entered).How are you?"

This looks like a Rasa Core issue, not NLU. Can you re-create here.


Allow for using 'python setup.py test' to run test suite.

It would be great if the test suite can be run through 'python setup.py test'. Using 'python setup.py test' seems universal and anyone attempting to run test suites of any Python module would first try running this. Also takes away the need for users to figure out how they need to run the test suite (slight difference when using Django < 1.6).

Why do we need the different codepath for django >= 1.6?


Add NFKD normalization + Fix for Japanese

This library was not using normalization at all, which for Spanish and Japanese would have produced invalid seeds. (So anyone using this library to generate wallets from Spanish or Japanese phrases must use the old non-normalizing version to recover their funds. I doubt anyone is there... but a warning might be necessary? or maybe create a new function for generating non-normalized seed?)
Chinese and English were ok, as their wordlists were pre-normalized (so unorm.nfkd(words) == words) so they should be fine as is.

Also, I added in one change for Japanese, mentioned on the BIP39:
Japanese must be shown to the user being separated by an ideographic space. This is crucial to ensure users don't accidentally view 2 words as 1 word.
Ex.
"a part" // this is a normal ASCII space
"a　part" // this is an ideographic space

It doesn't seem that necessary when letters are so small, but looking at Japanese.
"あさ ごはん" // this is a normal ASCII space
"あさ　ごはん" // this is an ideographic space

It makes a huge difference, and the latter must be shown to the user.
Also notice that ideographic space will be replaced by ASCII space when NFKD normalized, so while the words themselves are NFKD in the wordlist, because Japanese requires non-NFKD ideographic spaces for the "phrase" string, I have placed a catch-all NFKD in the call to pbkdf2 around this.phrase
Users in Japan will also likely input the phrase using ideographic spaces to input it, so I NFKD the mnemonic input to outward facing functions.

My main motivation for this fix is because it seems like Copay might use BIP39 in backups some way, and I was checking to see the ideographic space usage and luckily I did, as Spanish would have been generating bad wallets (non-BIP39-standard).
Also note that Japanese phrases are unique in that ALL Japanese characters are "breakable" and thus textwrap will break a word in the middle on a line break. Care must be taken to ensure a word is not broken on the line break and shown to the user.
Reference: (breadwallet is still trying to get it right... it is difficult)
https://github.com/voisine/breadwallet/issues/231
https://github.com/voisine/breadwallet/commit/dd1bfae75409009f087a72c3d7bd7a10fa845570

Wait a sec, I will add Japanese test vectors.

I tried to alter the test vectors to also use a Japanese passphrase with pbkdf2 instead of just "TREZOR" so that it could test normalizing of passphrase as well.
I realized how long it will take, and I don't have enough time, so I will just submit this PR as is.

LGTM

LGTM


"Invalid date" within weather forecast

This seemed to start after performing latest update. Weather forecast had been working for me for a long time (I built the mirror at least two or three years ago).
Logs don't seem to reveal much:
mm-0 Starting MagicMirror: v2.4.0-dev
mm-0 Loading config ...
mm-0 Loading module helpers ...
mm-0 No helper found for module: alert.
mm-0 Initializing new module helper ...
mm-0 Module helper loaded: MMM-Motion-Detection
mm-0 No helper found for module: clock.
mm-0 No helper found for module: MMM-Nest.
mm-0 Initializing new module helper ...
mm-0 Module helper loaded: calendar
mm-0 No helper found for module: currentweather.
mm-0 No helper found for module: weatherforecast.
mm-0 Initializing new module helper ...
mm-0 Module helper loaded: newsfeed
mm-0 All module helpers loaded.
mm-0 Starting server on port 8080 ...
mm-0 Server started ...
mm-0 Connecting socket for: MMM-Motion-Detection
mm-0 Starting module helper: MMM-Motion-Detection
mm-0 Connecting socket for: calendar
mm-0 Starting node helper for: calendar
mm-0 Connecting socket for: newsfeed
mm-0 Starting module: newsfeed
mm-0 Sockets connected & modules started ...
mm-0 Launching application.
mm-0 Create new calendar fetcher for url: h
mm-0 Create new news fetcher for url: http://feeds.arstechnica.com/arstechnica/index/ - Interval: 300000
mm-0 Create new news fetcher for url: http://www.npr.org/rss/rss.php?id=1001 - Interval: 300000
mm-0 [MMM-Motion-Detection] Motion Detection started...
mm-0 [MMM-Motion-Detection] --------------------
mm-0 [MMM-Motion-Detection] Webcam starting...
mm-0 [MMM-Motion-Detection] motion detected
mm-0 Starting MagicMirror: v2.4.0-dev
mm-0 Loading config ...
mm-0 Loading module helpers ...
mm-0 No helper found for module: alert.
mm-0 Initializing new module helper ...
mm-0 Module helper loaded: MMM-Motion-Detection
mm-0 No helper found for module: clock.
mm-0 No helper found for module: MMM-Nest.
mm-0 Initializing new module helper ...
mm-0 Module helper loaded: calendar
mm-0 No helper found for module: currentweather.
mm-0 No helper found for module: weatherforecast.
mm-0 Initializing new module helper ...
mm-0 Module helper loaded: newsfeed
mm-0 All module helpers loaded.
mm-0 Starting server on port 8080 ...
mm-0 Server started ...
mm-0 Connecting socket for: MMM-Motion-Detection
mm-0 Starting module helper: MMM-Motion-Detection
mm-0 Connecting socket for: calendar
mm-0 Starting node helper for: calendar
mm-0 Connecting socket for: newsfeed
mm-0 Starting module: newsfeed
mm-0 Sockets connected & modules started ...
mm-0 Launching application.
mm-0 Create new news fetcher for url: http://feeds.arstechnica.com/arstechnica/index/ - Interval: 300000
mm-0 Create new news fetcher for url: http://www.npr.org/rss/rss.php?id=1001 - Interval: 300000
mm-0 [MMM-Motion-Detection] Motion Detection started...
mm-0 [MMM-Motion-Detection] --------------------
mm-0 [MMM-Motion-Detection] Webcam starting...
mm-0 [MMM-Motion-Detection] motion detected
and config doesn't show any problems with the config checker:
/* Magic Mirror Config Sample
 *
 * By Michael Teeuw http://michaelteeuw.nl
 * MIT Licensed.
 */

var config = {
	port: 8080,

	language: 'en',
	timeFormat: 12,
	units: 'imperial',

	modules: [
		{
			module: 'alert',
		},
		{
			module: 'MMM-Motion-Detection',
		},
		{
			module: 'clock',
			position: 'top_left'
		},
		{
			module: 'MMM-Nest',
			position: 'bottom_right',
			config: {
				token:"xxx"
			}
		},
		{
			module: 'calendar',
			header: 'Calendar',
			position: 'top_left',
			config: {
				calendars: [
					{
						symbol: 'calendar-check-o ',
						url: 'xxx'
					}

				]
			}
		},

		{
			module: 'currentweather',
			position: 'top_right',
			config: {
				location: 'xxx',
				locationID: 'xxx',  //ID from http://www.openweathermap.org
				appid: 'xxx'
			}
		},
		{
			module: 'weatherforecast',
			position: 'top_right',
			header: 'Weather Forecast',
			config: {
	            location: 'xxx',
				locationID: 'xxx',  //ID from http://www.openweathermap.org
	            appid: 'xxx'
			}
		},
		{
			module: 'newsfeed',
			position: 'bottom_bar',
			config: {
				feeds: [
					{
						title: "ArsTechnica",
						url: "http://feeds.arstechnica.com/arstechnica/index/",
					},
					{
						title: "NPR",
						url: "http://www.npr.org/rss/rss.php?id=1001"
					}
				],
				showSourceTitle: true,
				showPublishDate: true
			}
		},
	]

};

/*************** DO NOT EDIT THE LINE BELOW ***************/
if (typeof module !== 'undefined') {module.exports = config;}

I'm also seeing this issue. Thanks!

Same issue for me today @berlincount .  I am noticing the API json does not have dt_txt.
https://api.openweathermap.org/data/2.5/forecast/daily?id=xxxxx&units=imperial&lang=en&APPID=xxxxx
fill in xxxxx with your values

Just updated with git pull to latest 2.4.0. Same for me.

Same here :/
Any Solutions?

There's a pull request in to correct the problem.  Hopefully that will all clear soon, and then we can update our mirrors confirm the fix and close this issue.

@Hilzerak solution is pretty manual for now:
you edit this file: modules/default/weatherforecast/weatherforecast.js
and change around lines 336-337 back to:
var day = moment(forecast.dt, "X").format("ddd");
var hour = moment(forecast.dt, "X").format("H");

then restart MM.

@GregoryOrciuch
Works like a charm. Much thx!

Ive just merged the fix by @jannekalliola into the develop branch which should solve this issue: https://github.com/MichMich/MagicMirror/pull/1341
Can you guys please test this branch? If this solves the issue, I'll release an intermediate 2.4.1 release.
To tryout the develop branch, use the following git command on your terminal (from within the MagicMirror folder):
git checkout develop
git pull

Thanks for testing!

I believe I'm already on develop branch.  Soon as I get home I'll pull and confirm.

I can confirm that this update fixed this issue!
Thank you everybody!


Sync with AD backend crashed periodically

Bonobo Git Server's sync with AD crashing every several month with next error:

Object reference not set to an instance of an object.
Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.
Exception Details: System.NullReferenceException: Object reference not set to an instance of an object.
Source Error:
An unhandled exception was generated during the execution of the current web request. Information regarding the origin and location of the exception can be identified using the exception stack trace below.
Stack Trace:
[NullReferenceException: Object reference not set to an instance of an object.]
Bonobo.Git.Server.Data.Update.ADBackendUpdate.Pre600UpdateTo600.BackendSubDirectoryNeedsUpdating(String backendDirectory, String subdirectory) +253
Bonobo.Git.Server.Data.Update.ADBackendUpdate.Pre600UpdateTo600.BackEndNeedsUpgrade(String dir) +126
Bonobo.Git.Server.Data.Update.ADBackendUpdate.Pre600UpdateTo600.UpdateADBackend() +236
Bonobo.Git.Server.MvcApplication.Application_Start() +838
[HttpException (0x80004005): Object reference not set to an instance of an object.]
System.Web.HttpApplicationFactory.EnsureAppStartCalledForIntegratedMode(HttpContext context, HttpApplication app) +540
System.Web.HttpApplication.RegisterEventSubscriptionsWithIIS(IntPtr appContext, HttpContext context, MethodInfo[] handlers) +186
System.Web.HttpApplication.InitSpecial(HttpApplicationState state, MethodInfo[] handlers, IntPtr appContext, HttpContext context) +172
System.Web.HttpApplicationFactory.GetSpecialApplicationInstance(IntPtr appContext, HttpContext context) +402
System.Web.Hosting.PipelineRuntime.InitializeApplication(IntPtr appContext) +343
[HttpException (0x80004005): Object reference not set to an instance of an object.]
System.Web.HttpRuntime.FirstRequestInit(HttpContext context) +539
System.Web.HttpRuntime.EnsureFirstRequestInit(HttpContext context) +125
System.Web.HttpRuntime.ProcessRequestNotificationPrivate(IIS7WorkerRequest wr, HttpContext context) +731

After this error needs to recreate ADBackend folder and update all json files of repositories.
Bonobo 6.3.0
Windows Server 2008 R2 SP1

It happened to me as well, I only needed to remove the users from ADBackend, as otherwise I would lose me descriptions/icons for repositories.


Start/stop via ctl should install/uninstall services

See https://keybase.atlassian.net/browse/DESKTOP-1756
Currently we are leaving behind launchd plists on stop. This makes it so it stops the services and removes the plists.
The PR makes some install/uninstall methods public and uses them in ctl start/stop on macOS.
Also:

Adds removal of fuse and helper to keybase uninstall
Lowers launchd status delay from 1s to 200ms
Don't log that we're stopping a service that doesn't exist
Bump installer version

@patrickxb or @maxtaco
cc @keybase/react-hackers

So ctl stop also means "uninstall launch agents"?
Just trying to think if there's a case where you quit keybase menubar (which runs ctl stop) but you are expecting it to start when you reboot...I guess not.

We moved app start to use System Preferences (login item)... So the services no longer start the app (the app starts services on start, and stops them on quit). This simplified everything too.

And yeah, ctl stop means stop (and uninstall) services.

Ok, LGTM


Render method doesn't work as spected

Description
render method not overwritting the existing HTML
Steps to Reproduce

Write this code

import {html, render} from 'lit-html';

const handleClick = () => {
let content = document.getElementById('content')
render(html`New Content`, content)
}

render(html`

<button @click={handleClick}>Click me!</button>

Old Content

`, document.body);

Expected Results
On click, doesn't overwrite the content.

<button @click={handleClick}>Click me!</button>

New Content

Actual Results

<button @click={handleClick}>Click me!</button>

Old Content
New Content

render() appends to the container. This is a change from lit-html 1.x. If you want to old container-clearing behavior you'll need to clear it yourself:
You can either do it eagerly:
import {html, render} from 'lit-html';

const handleClick = () => {
  let content = document.getElementById('content');
  render(html`New Content`, content);
}

render(html`

    <button @click={handleClick}>Click me!</button>

      Old Content

`, document.body);

// clear the container:
let content = document.getElementById('content');
content.innerHTML = '';

Or on-demand by keeping some state:
import {html, render} from 'lit-html';

let containerCleared = false;
const handleClick = () => {
  let content = document.getElementById('content');
  if (!containerCleared) {
    containerCleared = true;
    content.innerHTML = '';
  }
  render(html`New Content`, content);
}

render(html`

    <button @click={handleClick}>Click me!</button>

      Old Content

`, document.body);

We're not inclined to add or change behavior here, which is documented in the upgrade guide (https://lit.dev/docs/releases/upgrade/#lit-html). I think our suggestion would be to create a helper around render to perform the behavior you want.

Hey guys sorry to be a pain!
I have inherited an older project using jquery.   I am modernising the code.
In particular this $(selector).html("
lol
"); which as far as I understand is a full replacement of the selectors content.
I always end up with an error if I try to render to a cleared element.
This code:
// Write TypeScript code!
const appDiv: HTMLElement = document.getElementById('app');
appDiv.innerHTML = `<h1>TypeScript Starter</h1>`;

// trying to replace jquery $(selector).html("
lol
");

appDiv.innerHTML = '';
render(html`<h1>lol</h1>`, appDiv);

appDiv.innerHTML = '';
render(html`<h1>lol</h1>`, appDiv);

Or see my stackblitz
I always get the following error:
Error in<EMAIL_ADDRESS>(93:55)
Cannot read properties of null (reading 'insertBefore')
Do you know what I am missing?  🧐
Thanks!

This is really bad. There must be some way to check if the component needs to be cleared - rather than tracking our own state. Is there a property on the element to check if a render will work as expected?

@robaho are you saying that you have to render to a container that has content and you can't clear it before rendering? I'm curious what creates those conditions. We thought it would be extremely rare.
There is a property that render() adds to an element: _$litPart$, which you can check. It's private API, so not guaranteed to stay the same, but it should be stable for now.

Note: This comment relates to using lit-html's render standalone: https://lit.dev/docs/libraries/standalone-templates/
If you want a component model for composing templates, you want to be defining and using components: https://lit.dev/docs/components/overview/
Case 1: Only using render on a container element
In general, if you are only using lit for rendering HTML content, you never need to think about clearing DOM created by lit in a previous render, because render does that for you.
For example live playground:
render(html`<h1>Hi</h1>`, document.body);
setTimeout(() => render(html``, document.body), 1000);

This example appends
Hi
 into the body, then after 1 second the
Hi
 is cleared from the body. The previously rendered children in document.body that were never rendered by lit remain untouched.
If document.body has multiple children, there is an option renderBefore that can be passed to your render to change where the lit rendered DOM is inserted.
Case 2: Using render to clear existing DOM that wasn't rendered by lit, but then only use lit for subsequent renders.
This case may occur if your initial render was sent from the server. You now want to dynamically update the DOM in a container going forward but clear out the initial contents.
Given this HTML live playground example:

Some existing content

app.textContent = ''; // Clears out "Some existing content".
render(html`<h1>Hi</h1>`, app);

// After 1 second leaves `document.body` empty.
setTimeout(() => render(html``, app), 1000);

Case 3: You can never be sure who has rendered to a container between lit renders.
If DOM mutations can occur on DOM that lit-html is managing without using render(), then the invariants that lit-html uses to be efficient may be broken. This is why we throw an error.
As pointed out in prior comments, if you innerHTML on a DOM that lit is managing, then the next render fails with the error:
This `ChildPart` has no `parentNode` and therefore cannot accept a value. This likely means the element containing the part was manipulated in an unsupported way outside of Lit's control such that the part's marker nodes were ejected from DOM. For example, setting the element's `innerHTML` or `textContent` can do this.

lit must do bookkeeping to keep subsequent renders efficient, and arbitrary DOM mutations that are outside of lit on DOM that lit manages breaks the bookkeeping assumptions.
The reason bookkeeping is important is for the following case:
const template = html`<h1>Hi</h1>`;

render(template, app);
render(template, app);
render(template, app);
render(template, app);

After the first render, the next three renders perform no DOM manipulation and do nothing.
Where is this bookkeeping performed and cached?
The render function does this bookkeeping on a _$litPart$ property that is added on the container element.
So this means that the following code errors:
const template = html`<h1>Hi</h1>`;

render(template, app);
app.textContent = ''; // Breaks bookkeeping that was setup in the line above.
render(template, app); // Errors

What might a very defensive userland render function look like – that tries to do the efficient thing, but falls back on an inefficient clear and re-render?
Live example: https://lit.dev/playground/#gist=e02f6ac344d406b50a97b882483b89fe
// This should only be used if you cannot guarantee some different
// DOM mutation will break lit-html DOM assumptions. This breaks
// optimizations between subsequent re-renders, but only in the
// case where a DOM manipulation happened that was outside Lit's
// control.
function veryDefensiveRender(template, container) {
  try {
    render(template, container);
  } catch {
    // Assume that DOM was manipulated outside of Lit's control.
    // Remove Lit's bookkeeping and clear all the contents in the
    // container.
    container.textContent = '';
    delete container['_$litPart$'];
    render(template, container);
  }
}

If you want to go deep on how lit-html render works – we have a writeup: https://github.com/lit/lit/blob/main/dev-docs/design/how-lit-html-works.md

@robaho are you saying that you have to render to a container that has content and you can't clear it before rendering? I'm curious what creates those conditions. We thought it would be extremely rare.
There is a property that render() adds to an element: _$litPart$, which you can check. It's private API, so not guaranteed to stay the same, but it should be stable for now.

We are not using LitElement - this piece of code uses render(templateresult,element) directly - this is to keep the nodes in the DOM but it is also due to some legacy structuring.
I had to create a method like:
    function rerender(id: string, template: TemplateResult | symbol) {
        const elem = document.getElementById(id);
        if (!elem) { console.log('not found ' + id); return; }
        if (!renderOnce.has(id)) { renderOnce.add(id); elem.replaceChildren(); }
        render(template, elem);
    }

which feels like a hack to work around a lit bug.
Do you agree?

@robaho I get that you're not using LitElement, my question is more about what's requiring you to rendering into a container that has preexisting content that you want to clear.
fwiw, a wrapper over render() that restores the 1.x behavior would be like:
import {render as litRender} from 'lit';

/*
 * Clears the container on the first render. Must render into the whole container, renderBefore is not supported
 */
export const render = (v: unknown, container: HTMLElement | DocumentFragment, options?: RenderOptions) => {
  if (options?.renderBefore !== undefined) {
    throw new Error('renderBefore option not supported');
  }
  if ((container as any)['_$litPart$'] === undefined) {
    // first render
    container.replaceChildren();
  }
  return litRender(v, container, options);
};

You could also keep your own state for tracking the first render.
2.x just has different behavior here than 1.x - we had a lot of feature requests on 1.x to not clear the container, and very few observed cases where container clearing was required. render() is a very hot path and we wanted to optimize it as much as possible, and not keep the bytes for clearing if users could do that when they needed to - which we figured they would be in a position to know.
So that's why I'm asking about your specific case here. What rendered the existing content that you want to clear?

We have complex apps that have 10's thousands of cells. The cells can be grouped into sections. The sections can be collapsed - we don't use 'display: none' because the listeners will still fire - so when the section is collapsed we remove the section from the DOM.
We do this by removing all of the child elements of the section. If the user then re-enables the section we need to render the template back into the section (parent element).
The sections are all rendered from the intial render of the top-level template.
I think the use-case is valid but I am open to other ways of doing this. (I think the same code you provided that checks for the existing of the field is better - thanks).
I guess we could rerender the entire DOM using the top-level template but that seems way more inefficient.

I guess we could rerender the entire DOM using the top-level template but that seems way more inefficient.

That's not necessarily true – depending on your use-case. As I said in https://github.com/lit/lit/issues/2189#issuecomment-1666079323, you can re-render the top level template to the same container changing nothing, and no DOM updates will be performed. lit will only iterate over your dynamic expressions ${} in your template and do a single equality check to see if anything changed. If nothing changed then no updates happen.
There may be a performance problem if you have 10 thousand dynamic bindings, but even then there are tools that can be used. E.g. if you are rendering long lists, the repeat directive has a key function to do the minimum DOM operations.
You could also use the guard directive on the cells. Now the cell will only be re-evaluated by lit-html if a dependency passed into the guard dependency array changes. This means all the work of re-evaluating any DOM within cells can be completely skipped.
And you can check out how lit-html works under the hood if you're interested as well: https://github.com/lit/lit/blob/main/dev-docs/design/how-lit-html-works.md

The entire DOM consists of 270k nodes and a section is typically 1/6 of that.

@robaho I think we should take this to a new issue. What you really want here is multiple coordinated render roots, without components (I would recommend using components for this though - they coordinate this kind of incremental/isolating re-rendering quite well!).
The old render() behavior wasn't really what you wanted either because when you re-render a section that was initially rendered from the top-level render, you do end up clearing the whole section, when you likely would rather incrementally update it. It's possible that a new top-level render would be faster just because it updates the section you'd otherwise throw away and recreate.
If you for some reason can't factor the sections as components, then what you want here is a want to have multiple render roots in a tree. I've considered this before, and it would be possible with a directive that calls render() and somehow returns a reference you can use to re-render just that subtree.
Something like this maybe:

// This creates a new object that manages independently renderable roots
const roots = new RenderRootGroup();

// Render the top-level root of the page
render(html`

Root

  ${repeat(sections, (s) => s.id, (section) => roots.render(section.id, renderSection(section)))}
`, document.body);

// Called to update a specific section
const updateSection = (section) => roots.update(section.id, , renderSection(section));

// Just the template for the section
const renderSection = (section) => html`

Section: ${section.name}

`;

It seems to me you are overcomplicating this. That is not what I want. I want the render() to work as documented. If I clear the children, and call render(template,element) - it crashes. This is simply a bug imo.
The fact that it is a bug because Lit is attempting to optimize behind the scenes has no bearing - the method is not working as documented. If it is easer for Lit to add a method renderReplace() that would be fine, and document the other that you can ONLY use render() to affect the DOM tree for any elements manipulated via render().


Update-Help failing for some modules

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Title
Expected behavior
No error.

Actual behavior
Below

Error details
> Get-Error

Exception             :
    Type    : System.Exception
    Message : Failed to update Help for the module(s) 'ConfigDefenderPerformance, Dism, Get-NetView, Kds, NetQos, PcsvDevice, PKI, PSReadline, StartLayout, Whea, WindowsSearch, WindowsUpdate' with UI
culture(s) {en-US} : One or more errors occurred. (Response status code does not indicate success: 404 (The specified blob does not exist.).).
              English-US help content is available and can be installed using: Update-Help -UICulture en-US.
    HResult : -2146233088
CategoryInfo          : InvalidOperation: (:) [Update-Help], Exception
FullyQualifiedErrorId : UnknownErrorId,Microsoft.PowerShell.Commands.UpdateHelpCommand
InvocationInfo        :
    MyCommand        : Update-Help
    ScriptLineNumber : 1
    OffsetInLine     : 1
    HistoryId        : 2
    Line             : Update-Help -UICulture en-US
    PositionMessage  : At line:1 char:1
                       + Update-Help -UICulture en-US
                       + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : Update-Help
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo :

Environment data
> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.2.5
PSEdition                      Core
GitCommitId                    7.2.5
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

I can verify that I get the same error when updating help with 7.2.5.

The error is expected for those modules. The modules owners must fix the modules. The only module that there is a workaround for is PSReadline. The workaround is documented:

https://devblogs.microsoft.com/powershell/updating-help-for-the-psreadline-module/
https://devblogs.microsoft.com/powershell/updating-help-for-the-psreadline-module-in-windows-powershell-5-1/

This other thing to try for PSReadline is to unload the module, then reload it with the properly-cased name. Then update the help.
Remove-Module PSReadline
Import-Module PSReadLine
Update-Help PSReadLine -Force

@sdwheeler Thanks.  The first blog post fixed it for me.  It appears that the one remaining PSRL still named PSReadline was in my Windows PowerShell Modules folder.

Once I renamed that folder to PSReadLine, Update-Help PSReadLine -Force worked just fine from pwsh.

This seems to be a duplicate of #17067

Adding a summary of answers for this issue is being before closing out comments.
The workaround for the PSReadline module is documented in the following blog post:

https://devblogs.microsoft.com/powershell/updating-help-for-the-psreadline-module/

This other thing to try is to unload the module, then reload it with the properly-cased name. Then update the help.
Remove-Module PSReadline
Import-Module PSReadLine
Update-Help PSReadLine -Force

The error is expected for the following Windows modules.

ConfigDefenderPerformance
Dism
Get-NetView
Kds
NetQos
PcsvDevice
PKI
StartLayout
Whea
WindowsSearch
WindowsUpdate

The Windows modules are not maintained by the PowerShell team. To prevent the error, the module owners on the Windows team must fix the modules. For modules that ship in Windows, open a support case with Windows Support or send feedback via the Windows Feedback Hub. For more information, see Send feedback to Microsoft with the Feedback Hub app.


allow overriding libfuse path using envvar LIBFUSE_PATH

This allows users to manually override the libfuse path determined by find_library.
See https://github.com/terencehonles/fusepy/issues/35#issuecomment-147133806.

Reply on #35


Is this gem still maintained?

Hello 👋
Wondering if this is still being maintained - there doesn't appear to be a commit in over a year and the dependencies are getting a little dated.
For example, the oauth2 gem dependency is locked to a version that is now 2 years out of date...
Happy to throw a PR your way if it will get some consideration!

It will get some consideration

@seashellski we will definitely review a PR if you would like to submit. We're going to prioritize a fix ourselves, but you are welcome to submit a PR if you have one ready.
Note that we'd like to do a bit more formal testing beyond just the units in place to ensure everything works properly on the upgrade, so we'll likely do a .pre release before a full 0.3.4 comes out.
Thanks

We've pushed out our release for 0.3.3. Should be showing up on RubyGems within the next few minutes:
v0.3.3

This has been pushed to RubyGems. Thanks!

Woo!


Crafting pipe can't craft dried kelp block

Okay thanks, I'll fix this

A few of my players also report being unable to craft Paper and Iron Blocks via the Crafting pipe.

Chest can't be crafted as well


MCBrowser redirects into itself.

The config option "Open Links in MCBrowser" set to true makes the button "Open in external browser" open the requested tab in MCBrowser again, instead of an actual external browser. This is technically not incorrect because the config option does what it says on the tin, but it is both unhelpful and unintuitive due to what the button says.
If I press the button, then I already opened the link in MCBrowser and wish to open it in another browser(most likely for missing features like inputting a @ or copying text with a right click), so it should be a very safe change to just make an exception for the button's function to not be redirected.

https://github.com/Blobanium/MCBrowser/releases/tag/v1.1.5


bugfix: Don't attempt to bundle spacers with pin types assigned

Sometimes pin types can get accidentally assigned to spacers. Don't trip up and try to bundle them.

Thanks for the fix. I released it as 1.4.2.


REIL instruction source operands with different sizes

In the CSW’09 paper, see www.zynamics.com/downloads/csw09.pdf, the authors of REIL state:
„Right now there are no limitations on how operand sizes can be combined in one instruction.”
And indeed, in BinNavi you see REIL instructions such as:
or    [DWORD 0, BYTE t3, DWORD edx]
and [DWORD edx, BYTE 255, BYTE t1]
My question: What is the result of such arithmetic or bit-wise instructions where one source operand is smaller than the other? Is the smaller of the two source operands zero-extended or sign-extended before the operation is performed? Or does it depend on the instruction? Here is a third interesting example:
add [WORD 1, BYTE 255, WORD t0]"
Sign- or zero-extend?
I couldn't find any info on this in the specification although I would guess it should be sign-extend here.
Thanks for the clarification!

The code / answer you are looking for can be found here. As the code shows e.g for the OR instruction the result size is currently based on the third operand. This is only half the truth though, within the monotone solver framework you can define your own semantics.


Add code to package-info.java without messing up format

Moved to https://github.com/material-motion/material-motion-team/issues/155.


Not able to use Lazy loading with enableSearchFilter/onFilterSelectAll

What happens is that if lazyLoading is enabled that all items getting selected instead of just the ones matching the filter/search.
https://stackblitz.com/edit/angular2-multiselect-dropdown-lazyloading
Here you can dis/enable the lazyLoading setting, for testing.

@jbrischle fixed. fix available in v4.6.3

I am not able to fire onFilterSelectAll() and onFilterDeSelectAll() if lazyLoading:true
it's working fine if lazyLoading:false
Can you please fix this issue


Add flag of Suriname

Add flag of Suriname

@adavis46 Awesome work! Merging this. Thank you for contributing! :smile:


Updates to Conversion

Updates to Table to Point:

Removed the Copy Features
Set the parameters to the same as the ArcMap tool

@topowright so which tools does this fix?

@topowright I'm not seeing any of the changes you made in Conversion in Military_Tools.tbx. Only one commit? This still sitting on your system? :)

@topowright The parameters should be changed in Conversion tools. You can delete the lw-conversion-updates branch.


Add languages to engine json

/c @brynary

@brynary Provocative question re: possible values for languages. I made a pass here, looking at current and imminently supported languages and frameworks. It seems like supported languages is something that might belong somewhere else in the spec, and referenced here?

@mrb what if we use this list, which GitHub uses to driver their display and source listings:
https://github.com/github/linguist/blob/master/lib/linguist/languages.yml

@brynary WFM!

OK, cool. With that change, this LGTM


Update application-defined-or-object-defined-error.md

Formatting fixes.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Language-Reference-VBA/articles/application-defined-or-object-defined-error.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Language-Reference-VBA/articles/application-defined-or-object-defined-error.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Language-Reference-VBA/articles/application-defined-or-object-defined-error.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

VBA/Language-Reference-VBA/articles/application-defined-or-object-defined-error.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Report inconsistency in the command LRU

This issue is all about ensuring we are as close to Redis as possible. The command in focus for this issue is LRU.
Go through the official documentation of the command LRU on Redis and identify the inconsistencies. The inconsistencies could be in

unhandled edge case
unexpected behavior
unsupported option

Because we are trying to be compatible with Redis v7.2.5, I would recommend you try out different variants of the command with different inputs on that specific version. The instructions on running Redis v7.2.5 locally

from source code
use Docker

Once you find the discrepancy, you can either

raise an issue on Dice repository with details, or
try to fix it yourself and raise a PR

If you are raising the issue, make sure you provide the details such as

use the template and provide the following details
steps to reproduce (series of commands)
observed output on DiceDB
observed output on Redis v7.2.5

Also, feel free to update the documentation and raise the PR in the docs repository.

You will need to go deeper into the command make sure you are covering all cases and reporting the inconsistencies or fixing them. The deeper the work, the better our stability will be. Also, it is possible that we do not find any discrepancies, so please mention the same in the comment on this issue. Mention the PR or issue links that you create under this issue.

Hi @arpitbbhayani
can i take this up? thanks!

Hi @arpitbbhayani @JyotinderSingh  @soumya-codes ,
As I saw in redis docs and in cli I'm not able to find as such regarding this LRU command.
While they implemented a various policy and evict key according to the same.
Refer : https://redis.io/blog/cache-eviction-strategies/.
I'm bit confused why we implemented LRU command? :thinking:
as of now should we go for implement logic of various policies and then based on these policies we will implement this LRU logic?
And as currently on latest master LRU command deletes all key from db

@Abh-ay I guess you're referring to these eviction policies from Redis. I would suggest to wait for till we refurbish the store and maybe then pick one-by-one eviction policies. @soumya-codes your thoughts?

Yes @AshwinKul28 I referred this doc,
Just want to confirm are we having any plan to raise separate issues for different policies or implement one by one in this issue only?
and also this issue for report inconsistency in LRU only but I don't think we need to verify this yet.
and any plan to prioritize some policies to implement or any index which we need to follow for implement mentioned policies?
let me know so we can proceed further...

Hi @arpitbbhayani , I would like to work on this, can you assign me this one.

hi @arpitbbhayani
I would like to work on this, can you please assign this to me?

Hey @arpitbbhayani , can I take this up?

@arpitbbhayani I'm still waiting to get your response, want to take this up

@srikomm @arpitbbhayani @JyotinderSingh can I please take this up?

@shashank-priyadarshi I'm waiting from a long time to take this up :)

@shashank-priyadarshi  Can i take this up?

Hi @arpitbbhayani - Can I pick if no one is working on this?

@berabhishek @AshwinKul28 @srikomm @Tanmay1201
Hi maintainers,
I’ve reviewed the Redis LRU command and tested its behavior on Redis v7.2.5. I’ve identified potential discrepancies and have ideas to improve compatibility with DiceDB. Please assign this issue to me, and I’ll ensure all edge cases are thoroughly addressed.


Doesn't do anything

After installation did not update the settings.json.
[marius@fedora infra-terraform]$ pdm --version
PDM, version 2.4.0

[marius@fedora infra-terraform]$ python --version
Python 3.11.1

After installation did not update the settings.json.

After running pdm plugin add pdm-vscode and pdm use, your settings.json did not update?
There is any other information that can be provided?

The function only contains 10 lines and is very easy to read: https://github.com/frostming/pdm-vscode/blob/45323f42b407a0f9d9a4a307e63dd34c00ecb518/src/pdm_vscode/hook.py#L51-L54
It only updates settings.json if you are NOT using a venv-like python.


navigating state boolean not updating with async component

instance.navigating is not updated when an import/component takes a long time. It only seems to work with a pre async method set.

[!NOTE]
The current example in the readme which shows binding navigating boolean directly seems out of date, I didn't realize I had to bind instance until after reading the example source code.

The current delay example (cc'ed as follows) uses a timeout in the pre option for the route. This works and causes the instance.navigating to update to true and display as expected.
    {
      path: "delayed",
      component: Delayed,
      pre: async (route: Route): Promise<Route> => {
        // Simulate a network delay by returning a promise that resolves after 1.5 seconds:
        return new Promise((resolve) =>
          setTimeout(() => {
            resolve(route);
          }, 1500)
        );
      }
    },

Unexpected behavior
If loading an async component that takes a long time, navigation boolean is not updated. For example:
    {
      path: "/test1$",
      component: async () =>
        new Promise((a) => {
          setTimeout(() => {
            a(import("./test1.svelte"));
          }, 1500);
        }),
    },

Expected Behavior
I think the above async component example should result in the same behavior to the navigating boolean as the pre option with async code and update navigating boolean accordingly.

For anyone else seeing this I moved on to this library, the author took all my suggested changes so now it meets all the needs I had of this router.


min_k8s_version: allow non-numeric characters in versions

fixes #82

@landism should this be merged?


Support of angular 9/10

Hi,
I was wondering if support of angular 9 and angular 10 was planed.
I updated my project from angular 8 to 9 and all went good. No obvious error in leaflet component but my map has a strange behavior. Like it does not fill my page, markers are not displayed. I made no change in the code related to leaflet during the upgrade of angular so I am suspicious about an incompatibility and wanted to ask about here...
Maybe it is just that I was doing something in the wrong way before and that was working by miracle but I really don't known.
Is there someone who had issues with angular 9 and leaflet ?
Thanks for your time.
Wendy

have the same problem!... [leafletLayers]="Layers" load when create map, but not update when you destroy arrat layers or push new markers...
visibility of map i solved with this funtion on ts:
onMapReady(map: Map) {
this.map = map;

// refresh Map

setTimeout(() => {
  console.log('1');
  this.map.invalidateSize();
}, 1);

}
and calling this funtion in html:
(leafletMapReady)="onMapReady($event)"
Please solution for update layers problem!!

Yeah, I'm planning to release an update soon. Likely this week.

Thank you so much ! I look forward to get it !

Hi!
What about ng9 support?

Yeah, working it now. It's a little more complicated than I expected.

v7 is published to npm. Let me know if you have any issues.

@paulolira73 Thanks for the visibility fix!!!
@reblace I still saw the visibility issue mentioned above after upgrading to ng 9.1.0 and ngx-leaflet 7.0.0. I'm also using ngx-leaflet-markercluster 3.0.0. @paulolira73's visibility fix resolved the issue for me (only the top 20% of the map was visible). Interestingly, the issue was not reproducible in development env, only when using deployed artifacts (ng build ...); not sure if that matters.

Do you want to create a separate ticket for the visibility issue and I can
look into it?
On Fri, May 15, 2020 at 3:21 PM Ken<EMAIL_ADDRESS>wrote:

@paulolira73 https://github.com/paulolira73 Thanks for the visibility
fix!!!
@reblace https://github.com/reblace I still saw the visibility issue
mentioned above after upgrading to ng 9.1.0 and ngx-leaflet 7.0.0. I'm also
using ngx-leaflet-markercluster 3.0.0. @paulolira73
https://github.com/paulolira73's visibility fix resolved the issue for
me (only the top 20% of the map was visible). Interestingly, the issue was
not reproducible in development env, only when using deployed artifacts (ng
build ...); not sure if that matters.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Asymmetrik/ngx-leaflet/issues/256#issuecomment-629430113,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADVLPL5HKX4NSOYPOEZ6DTRRWI4VANCNFSM4LUCZOYQ
.


Bounce Type = Expired causing confusion for customers

Hey guys,
Any info on the "Expired" type of bounce you know of?
https://sendgrid.com/docs/API_Reference/Webhooks/event.html#-Bounce

Reopening this to hijack it for creating a glossary entry for "Expired"

Hi @ksigler7, I can take this too.
To clarify the steps to resolve this issue:

Create a new Glossary Entry for "Expired"
Link Bounce Event "Expired", to this new Glossary Entry (related to issue #2734)

Terminology Clarification:
I wanted to clarify, what is the definition of 'Bounce - Expired'? I don't see any information on the SendGrid docs site after searching around.
I find two possible reasons for 'Bounce - Expired' emails
https://support.intouchcrm.com/articles/email/what-are-email-bounces

The message bounced due to an expired, inactive, or disabled recipient address, 5.X.X error.
The message bounced due to not being delivered before the two day window in which the system continues to attempt delivery, 4.X.X error.

How does SendGrid denote a, Bounce - Expired, message?
Thanks

@mbernier @ksigler7
Just pinging to see if you could provide some comments on my previous question?
I wanted to clarify how SendGrid categorizes 'Bounce - Expired'.
Two possible definitions that I found online, relating to message sending, are:

The message bounced due to an expired, inactive, or disabled recipient address, 5.X.X error.
The message bounced due to not being delivered before the two day window in which the system continues to attempt delivery, 4.X.X error.

Thanks!

Hello @duncanleung
Sorry for our delay. Bounce - Expired in general means we were unable to deliver the email in the last 72 hours. Now there are some 5.X.X and 4.X.X that can result in an instance where we try for up to 72 hours to deliver email. So I would say that both your examples are correct.  Additional examples I have seen include:

Domain frequency limited.
Connection frequency limited.
Too many invalid recipients
Ip frequency limited.
Which are all 550 errors.

I hope this helps and please let us know if there are further questions.

This turned out really well @duncanleung, thanks so much for writing this up!
And thanks again for answering the question @kylearoberts :)

@ksigler7 awesome! Glad to hear!
Thanks to @kylearoberts for the support!


[BUG] disallow except +nogil

Describe the bug
We currently allow except +nogil and parse it as except + nogil (with space). However, that is different from except +handlerfunc. We should require it to be spelled except + nogil.
We should also consider rejecting except +nogil all together and not parse it as exception handler function. It's an easy source of typos.
Code to reproduce the behaviour:
cdef func() except +nogil:
    pass

Expected behaviour
No response
OS
No response
Python version
No response
Cython version
0.29.x and master
Additional context
No response

It's quite unfortunate that the ordering of except vs. nogil now makes the white-space after except + semantically relevant. If only to mitigate that, I fully agree that +nogil should be forbidden.
I tried chasing down the decision for the ordering, and AFAICT, it was made directly in the PR. So, for completeness, a comparison of all possible combinations:
Current status: first except qualifier, then gil qualifier
noexcept nogil:
noexcept with gil:
except <val> nogil:
except <val> with gil:
except? <val> nogil:
except? <val> with gil:
except * nogil:
except * with gil:
except + nogil:                # only separated from `except +<handler>` (without gil-qualifier) by a single space
except + with gil:
except +* nogil:
except +* with gil:
except +<PyError> nogil:
except +<PyError> with gil:
except +<handler> nogil:
except +<handler> with gil:

Parallel world: first gil qualifier, then except qualifier
nogil noexcept:
nogil except <val>:
nogil except? <val>:
nogil except *:
nogil except +:
nogil except +*:
nogil except +<PyError>:
nogil except +<handler>:
with gil noexcept:
with gil except <val>:
with gil except? <val>:
with gil except *:
with gil except +:
with gil except +*:
with gil except +<PyError>:
with gil except +<handler>:

Personally I think the second set looks better, aside from the fact that it does not suffer from any ambiguity or semantically relevant whitespace. But I also guess it's too late to change it now that we're in the beta phase already and projects have started responding to the warnings about the ordering.

@scoder @da-woods
Seeing that this is currently one of the main blockers for 3.0 (cannot deprecate something that 0.29.x cannot even do correctly yet), can we decide how to handle this?
I'm not sure if it's worth the churn to change the order of the gil & except qualifiers to avoid the ambiguities in the first place, but if so 3.0 is the only time to do it (and I guess we could say we're not in rc phase yet). If so (and it's a big if), it should IMO be part of the next beta.

I'm not sure if it's worth the churn to change the order of the gil & except qualifiers [...]

I'd probably say no. Although with hindsight I might have argued against fixing down the order.
I guess the long-term options are:

Change except +nogil to mean "use an exception handler called nogil".
Ban except +nogil (with a message that says recommends except + nogil or a different exception handler name).
Leave it as is, and maybe issue a stronger warning if we detect a situation where it's genuinely ambiguous (i.e. a C function called nogil actually exists). Or just always issue a warning recommending the space.

And in principle the first two of those could be done with a deprecation period for the existing behaviour.
I don't think we should do the first option - too much chance of confusion. I think we should either warn about it or ban it for Cython 3. Don't have a strong opinion which.

I've also backported the error to 0.29.35.


Merge OngXinBin-AnimeList

Added

Create Anime list class
Get anime from list with index
Print all anime in animelist
Get the size of animelist
Provide basic usage example in duke.java

I understand your concerns regarding the conflict of the naming convention of AnimeList class and the array list object in the watchlist. I will take up your suggestion and rectify it to "AnimeData" to represent Animes from our offline data.


Add use-text-color

As a border-color CSS value, use-text-color is not added in.
E-mail to reach me<EMAIL_ADDRESS>Ryan Reese
Web Developer

This appears to be a Mozilla-specific extension (-moz-use-text-color) with no un-prefixed or other-vendor-prefixed versions. (See https://developer.mozilla.org/en-US/docs/Web/CSS/Reference/Mozilla_Extensions#border-color).
It's equivalent to currentColor, which is supported all the way back to Firefox 2 per http://caniuse.com/#feat=currentcolor

Thanks!

Just in case anybody stumbles upon this issue:
-moz-use-text-color was removed in Firefox 52 (https://developer.mozilla.org/en-US/Firefox/Releases/52 or bug 1306214).


Fix issue with URLs losing their a's

Fixes #25

Confirmed. It's amazing that no one ran into this before. Please merge! :)


Update to ACK runtime v0.12.0

Update the dependent version of ACK runtime to v0.12.0 and update the ackgenerate version

/lgtm


Allow voting directly from site

Use the Reddit API to allow users to vote on their favorites directly from the site.

After a discussion with /u/bakonydraco, moving to put voting on the site as a replacement for the voting on Reddit. Will add a 1-5 star rating for each flag. User needs to be logged in to their Reddit account to vote.


network: allow to handle GetMPTData command if KeepOnlyLatestState enabled

Depends on #2095.
We need to adjust server's handleGetMPTDataCmd behaviour wrt KeepOnlyLatestState setting. We also need a test for state exchange between two nodes with KeepOnlyLatestState on.

At the moment KeepOnlyLatestState is ignored when RemoveUntraceable is active which kinda solves the problem (we have all the appropriate data). And and it's a fair behaviour to return whatever MPT data we have (if we don't, this command will error out anyway). So we can remove the check, but there are some other things that can be improved here.


Is there a reason for the playbooks/roles symlink?

I wonder if it is necessary. I'd vote for having proper ansible.cfg path setup and removing the symlink.

It's something I thought would be handy, as the playbooks directory is more commonly the root dir, and ansible can find the roles by default when in such root dir. In the testnet inventories we don't actually use it. I'm ok with removing it, if you prefer that.

Hah, maybe just a result of me grep-ing through the code base all the time. I vote for removing. But I am not strong opinionated here.

Cleaning up some ansible things, removed the symlink


Support different CPU architectures

As discussed in webdriverextensions/webdriverextensions-maven-plugin-repository#91

The os.arch / cpu architecture would be good. It would be a new property of the driver object defaulting to "x86" if not set. CPU arch + platform + 64bit = better driver selection.

Hi @sephiroth-j , is there any progress on this topic ? I'd like to help whereever i can to speed up the integration of the Apple Silicon drivers into the webdriveextensions-maven-plugin-repository.

I just released a SNAPSHOT of version 3.4.0 which includes support for AARCH64. It uses a special version of the repository file which includes the following

edgedriver 105.0.1343.53 for MAC and WINDOWS on ARM64
chromedriver 105.0.5195.52 for MAC on ARM64
geckodriver 0.31.0 for MAC on ARM64

It is also no problem to use this repo file with the current stable version 3.3.0 of the plugin. The new property is ignored and the plugin finds the 64-bit variant of the driver twice and selects the first one - i.e. the AMD64 variant.
Anyone who wants to test this version must change the version of the plugin in pom.xml to "3.4.0-SNAPSHOT" and add the Sonatype SNAPSHOT repository with url "https://oss.sonatype.org/content/repositories/snapshots/" as follows.
<repositories>
	<repository>
		<id>sonatype snapshots</id>
		<url>https://oss.sonatype.org/content/repositories/snapshots/</url>
	</repository>
</repositories>

Hi,
just tested this. Works fine for me.


institution_id registration of TEST

institution_id
TEST
institution
Test institution

check if this institution_id is inserted alphabetically not simply in the end.

/register

ok, it seems to work.


Add support for updating ratings with most recent rating

Pull Request: Support Updating Ratings for Movies and TV Shows
Description:
This pull request adds functionality to update user ratings with their most recent ratings for both Movies and TV Shows. However, please note that updating episode ratings is currently not supported due to a limitation in the TMDB API.
Episode Rating Limitation:
At present, the TMDB API (both v3 and v4) does not provide the created_at date for episode ratings, which prevents us from supporting episode rating updates. We have submitted two feature requests to the TMDB API support forum, seeking to add support for obtaining the created_at date for episode ratings. Unfortunately, as of now, we have not received any updates on whether these features will be added in the future.
Feature Requests Submitted:

Feature Request: Add Support for "Rated TV Episodes" in v4 API
Feature Request: Add Support for "created_at" Date in v3 API Endpoints for User Ratings

Until the TMDB API includes support for obtaining created_at dates for episode ratings, only updates for Movie and TV Show ratings will be supported.

New issue opened: Add support for updating episode ratings with most recent rating #22


[Demo] Replacing grpcio with tonic

What is changed and how it works?

Issue Number: Close #xxx

What's Changed:
This is a draft PR to show the current progress of replacing grpcio with tonic.
The current choice is tonic + rust-protobuf because rust-protobuf has better performance over prost. This PR use tonic-protobuf to integrate tonic with rust-protobuf.  Our internal benchmark(mainly with sysbench) show that tonic+rust-protobuf has similar performance with grpcio + rust-protobuf. In some workload tonic shows more than 10% performance better than grpcio.
Currently, most code are finished and the code can be built and deploy in production environment. I have run server benchmarks with this code. But some of the components such as cdc, backup-stream are not widely tested so potential bugs may still exists.
Due to the restriction of standard tonic interface, some feature are hard(or even impossible) to implement. So this PR uses a patched version of tonic. You can see the changes at: https://github.com/hyperium/tonic/compare/master...glorv:tonic:patch-health
Features that are not  supported yet:

TLS Common Name(CN) check. This is because rustls does not expose Common Name or Subject Alternative Name(SAN) to the caller, so we can't validate them.
PdClientV2. The design of PdClientV2 highly depends on the completion based connection state machine of grpc-rs, there is no corresponding feature in tonic.
WriteFlags. WriteFlags is mainly used to control the buffering policy of streaming call, but it is not available in tonic.
CallOption. CallOption is mainly used to control the timeout of gRPC calls. While tonic does not support it, it is possible to support Timeout in tonic's grpc client. Thus, this is doable, but we need to further to investigate the best practice.
MemoryQuota. Tonic does not directly support memory control. Not sure if memory control is doable via a tonic middleware(as tonic service Layer).
grpc_gzip_compression_level, grpc_min_message_size_to_compress. Tonic support message compression, but the compression level and threahold is not configable.

NOTE: This is still a demo to show the possibility of replacing grpcio with tonic. Some part of this PR still need further optimization and is not its final version. We may also change some of the desgin choices.
Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

@hbisheng PTAL if you are interested.

Can we split the PR?

Can we split the PR?

@Connor1996 Seems very hard, because we must choose either grpc-rs or tonic so all component must be changed in 1 PR. maybe in the formal PR, the commits can be better organized so it may make review easier.


Improve handling of user payloads that contain nexus metadata

Description
It has been observed that some defective functionalities of Nexus (e.g. incoming/outgoing links being duplicated) can be caused by the fact that some users submit payloads that contain Nexus metadata. Currently, Delta doesn't have anything in place to handle that situation.

Fixed in #3935


[Feature Request] - Deselect all files

Description
Ctrl+A selects all files. M toggles selection on the hovered file. There is no way to Deselect all (selected) files).
Changes
Select all replaced by toggle selection all
Implementation
Since M toggles the hovered one, probably Ctrl+A should also toggle. If there is a just a subset selected, first Ctrl+A should select all, then if all files are selected, Ctrl+A should deselect all.

I will use CTRL+SHIFT+A


bpf: Add mixed-mode unwinding for JIT sections

This patch introduces support for mixed-mode unwinding for binaries with mixed JIT and AoT(ahead of time) compiled sections with the dwarf-unwinding-mixed flag.
As of now, DWARF-based stackwalking discards or skips JIT sections of binaries resulting in incomplete/incorrect stacks. Currently, we can only unwind JITed sections in binaries that retain frame pointers through FP-based stackwalking mode.
This patch adds a JIT unwinder for walking JITed sections without frame pointers. The agent bpf unwinder can now switch between JIT-unwinding and DWARF-unwinding modes while profiling stripped binaries with both JIT and Ahead-of-Time sections.
Test Plan
The current integration tests make test/profiler pass(both with and without the flag enabled).
$ make test/integration GO=$(which go)
=== RUN   TestCPUProfilerWorks
--- PASS: TestCPUProfilerWorks
PASS
ok  	github.com/parca-dev/parca-agent/test/integration

Stacks are correctly symbolised in Parca for the basic-cpp-jit-no-fp toy jit binary.

Late to the party but awesome work :tada:


Inconsistent license

The README.md file states that the library is LGPLv3 while the rdfstore-js/LICENSE file states the project is licensed under the MIT license.  Which one is correct?

Using MIT now.