paused_for_user notification not working

I'm having trouble getting the paused_for_user notification to work.
Al other notifications work just fine, so I know the connection to the iOS app is as it should. I set the notification delay for this type of notifications to 1 minute.
I'm using the Smart Filament Sensor Plugin to trigger an M600 command, and added the M600 as a pausing command to OctoPrint. The printer reacts as it should, and OctoPrint shows the printer being paused and gives the possibility to "resume".
Even after a few minutes, stil no notification is send to the iOS app.
The OctoPrint log also does not show the event.
I'm I forgetting to configure something? Or is the OctoPod plugin somehow not able to catch the M600 command from another plugin?

Hi @pheuts, could you post what you see in the Terminal of OctoPrint? OctoPod plugin is looking for a line that starts with echo:busy: paused for user. Marlin will print that line on the terminal when it pauses the printer (for whatever reason). If you see that line then check your configuration for Printer Paused Notifications. Make sure that you do not have a value of 0 since that will disable the pause notification. If progress is more than 0% and less than 100% then you should see a notification.

I created this sample gcode that does not print but moves the printer and executes an M600. I printed that file from OctoPrint and I got the notification as expected. I know it is now your use case but wonder if that works for you. It will help narrow down the use case.
pause.gcode.zip

Hi @gdombiak,
Thank you for your help. It is much appreciated.
Your first message explained to me that the plugin is triggered by an response from the printer, not a command send to the printer. And that's where the problem lies. My printer's (AnyCubic i3 Mega-S) firmware does not support M600 commands. I added M600 to OctoPrints pausing commands, and use OctoPrints Gcode pausing scripts to make the printer do what i need for a filament change. But this does not make the printer give any response, so no notification from OctoPod is triggered.
I tried triggering a notification in a bit hacky way, by sending "echo:busy: paused for user" from my pausing scripts. Which gives the following in the terminal:
Send: echo:busy: paused for user
Recv: echo:Unknown command: "echo:busy: paused for user"
I hoped you used a contains-conditional. But no luck. I assume you use .startswith() ?
Any chance you are willing to make a change somewhere to make this work? Either change the conditional to something like:
if "echo:busy: paused for user" not in [string]:
continue
Or also check commands send to the printer? Or anything else you think is best :)  (I'm just a noob at programming)
Thanks in advance!

Hi @pheuts, does your printer's firmware support M0 gcode? Marlin implements it, it will stop the printer and send back to OctoPrint the "echo:busy: paused for user" message. I wonder if this other firmware does that as well. We can explore other ideas as you suggested if not. BTW, your logic is very good so not a noob on my book. :)

@gdombiak, as you suggested I've been trying with the M0 (and M1) command. The printer does react to the command, but not the way anybody would want it to. It stops printing, but leaves the hotend on the object, gives no serial respons what-so-ever, and times out on serial. I'm gonna leave those commands in the blocked list from OctoPrint. I guess they are there by default for a reason.
Looking through Marlin's gcode's I've also tried M6, M25, M118 and M226. But al no luck.
Any changes on your side would be much appreciated.

A relatively new feature in the PrusaSlicer UI is to add a pause at a specific layer. It would be nice if these kinds of pauses would result in the pause notification (currently the notifications aren't firing for pauses configured in this way). Better yet, it would be handy to be able to use the similar PrusaSlicer feature where you can do the same thing but for any custom G-Code... it would be nice to be able to insert a special g-code that OctoPod could see and send the notification. For example, I would actually like to be notified a few layers before the pause layer, so I can get to my printer and be ready to quickly, for example, insert magnets into the print.

Hi @andrhamm,
Thanks for the detailed ask. Quick answer before I think deeper into your request. :) If you are using the DisplayLayerProgress plugin then you can set a notification from OctoPod for any layer you want. It is not the same as you requested but was wondering if that will help you until I implement your ask.
Regards,
Gaston

I do have that installed but I was not aware of that feature, I will
check it out, thanks!
On February 27, 2021, GitHub<EMAIL_ADDRESS>wrote:

Hi @andrhamm https://github.com/andrhamm,
Thanks for the detailed ask. Quick answer before I think deeper into

your request. :) If you are using the DisplayLayerProgress
https://plugins.octoprint.org/plugins/DisplayLayerProgress/ plugin

then you can set a notification from OctoPod for any layer you want.

It is not the same as you requested but was wondering if that will
help you until I implement your ask.
Regards,
 Gaston

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/gdombiak/OctoPrint-OctoPod/issues/72#issuecomment-
787122646>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/AAEQNMLQ3DHGZFTAADMQXILTBFBHRANCNFSM4VDMCB2Q>.

I have Octopod running on an iPhone.  I don't see a "main bell" in the Octopod app.  I'm trying to get notifications for a pause at layer to change filament.  I do have the DisplayLayerProgress plugin installed

Hi @kwalkerk,
Have you configured your slicer to include layer information? OctoPod displays layer information and lets you receive notifications on layers when DisplayLayerProgress plugin installed and gcode has layer information which requires slicer configuration. If OctoPrint web ui displays layer information then something else must be going on.
Let me know how it goes.
Thanks,
Gaston

Thanks for the reply.
As usual it was my misunderstanding of how things worked.  I had everything set up properly, but was not using the “bell” on the iOS app properly.
I use the “Pause at Height” plugin.  This plugin must not insert the proper codes into the code, because it does not trigger any notification from OctoPod. I have the ability to add gcode before the layer.  Is there something I can add which will trigger the notification?  What is the trigger?
BTW, thanks for this plug in  I love it

On May 7, 2022, at 9:27 PM, Gaston Dombiak @.***> wrote:
Hi @kwalkerk https://github.com/kwalkerk,
Have you configured your slicer to include layer information? OctoPod displays layer information and lets you receive notifications on layers when DisplayLayerProgress plugin installed and gcode has layer information which requires slicer configuration. If OctoPrint web ui displays layer information then something else must be going on.
Let me know how it goes.
Thanks,
Gaston
—
Reply to this email directly, view it on GitHub https://github.com/gdombiak/OctoPrint-OctoPod/issues/72#issuecomment-1120328913, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADXIYZI2W5SVJRGLOHEOJLVI4J67ANCNFSM4VDMCB2Q.
You are receiving this because you were mentioned.

Gotcha. Yes, these are different functionalities. Describing each one in case any of them help you.
OctoPod + DisplayLayerProgress plugins: Notification when print reaches a certain layer
When both plugins are installed and your slicer is correctly configured according to DisplayLayerProgress plugin, then any newly generated gcode will include layer information. From OctoPod iOS app you can use the bell button next to layer information to receive a notification (with an image) when print job reaches the requested layers
OctoPod plugin: Notification when print is paused waiting for user
If you are using Marlin firmware and your gcode includes M600 or runs out of filament and you use a filament sensor then OctoPod plugin will send you a notification indicating when printer halted and is waiting for user. This is also great for swapping filaments.
If you are using Prusa Mini then this functionality only works when printing from OctoPrint. The firmware of prusa mini does not send proper notifications that OctoPrint can react to.
“Pause at Height” plugin: Notification with custom gcode
I do not use Cura so I never heard of “Pause at Height” plugin. I read its documentation and it seems like you can add whatever gcode at any layer. If you are going to include M600 then what I described in the previous section still applies. If you are including any other gcode then OctoPod plugin will not send any notification since it is "unknown" gcode.
Let me know if you are injecting other gcode and I may improve OctoPod plugin so that you can receive notifications on custom gcode. This was requested once before but there was not much demand for it so it is low in the priority list.
Regards,
Gaston

Thanks so much for the detailed explanation!
The M600 did the trick.    I inserted the M600 and got the behavior I wanted.
Thanks,
Ken

On May 8, 2022, at 2:16 PM, Gaston Dombiak @.***> wrote:
Gotcha. Yes, these are different functionalities. Describing each one in case any of them help you.
OctoPod + DisplayLayerProgress plugins: Notification when print reaches a certain layer
When both plugins are installed and your slicer is correctly configured according to DisplayLayerProgress plugin, then any newly generated gcode will include layer information. From OctoPod iOS app you can use the bell button next to layer information to receive a notification (with an image) when print job reaches the requested layers
OctoPod plugin: Notification when print is paused waiting for user
If you are using Marlin firmware and your gcode includes M600 or runs out of filament and you use a filament sensor then OctoPod plugin will send you a notification indicating when printer halted and is waiting for user. This is also great for swapping filaments.
If you are using Prusa Mini then this functionality only works when printing from OctoPrint. The firmware of prusa mini does not send proper notifications that OctoPrint can react to.
“Pause at Height” plugin: Notification with custom gcode
I do not use Cura so I never heard of “Pause at Height” plugin. I read its documentation and it seems like you can add whatever gcode at any layer. If you are going to include M600 then what I described in the previous section still applies. If you are including any other gcode then OctoPod plugin will not send any notification since it is "unknown" gcode.
Let me know if you are injecting other gcode and I may improve OctoPod plugin so that you can receive notifications on custom gcode. This was requested once before but there was not much demand for it so it is low in the priority list.
Regards,
Gaston
—
Reply to this email directly, view it on GitHub https://github.com/gdombiak/OctoPrint-OctoPod/issues/72#issuecomment-1120464165, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADXIYYKEQXZE6ODW4L3BCDVJAAJJANCNFSM4VDMCB2Q.
You are receiving this because you were mentioned.


Maintain Proxy Voting Totals ($3000)

The purpose of this proposal is to update the graphene code to keep vote totals up to date for all witnesses, committee members, and ultimately every account that has proxy votes.
This will improve transparency and allow people to easily audit the power structure of BitShares to know who has influence and how much.
This proposal would be to provide an API that will return all accounts that have a particular user set as their proxy. It would also update the Account/Voting page for a particular user to display their total influence along with a list of all accounts that are actively proxying through them.
No hard fork will be required, and enabling this extra tracking would be optional for witness nodes looking to save CPU and Memory but can be enabled for block explorers and API nodes looking to feed data to the wallet.
Contractor: Cryptonomex
Cost: $3000
Worker:
Escrow:  BunkerChain Labs

This issue was moved to bitshares/bitshares-core#131


Class limit on semantic segmentation

Hi,
amazing repo, I got my hands on an Edge TPU but I am struggling in finding an actually useful way to use it.
I am interested in doing only instance segmentation (so no classification whatsoever), and I was wondering if in your semantic segmentation approach based on DeepLabV3 you are limited to the same 20 classes listed here.
Thanks

I am interested in doing only instance segmentation (so no classification whatsoever),

I also want to perform instance segmentation (like Mask R-CNN) on an Edge TPU. However, I don't know the model that can be executed. I would like to know if you can.

I was wondering if in your semantic segmentation approach based on DeepLabV3 you are limited to the same 20 classes listed here.

Yes. As you pointed out. The model for this repository and Coral's Deep Lab is PASCAL VOC. If you want to model your original dataset, this link will help.
Thanks

I found a couple of resources to convert Mask R-CNN to TFLite, unfortunately, nothing specifically targeted to the Edge TPU.
This article, and associated colab notebook, give some instruction on how to create the model to TFLite. I downloaded it, but I was unsuccessful in running it on the Edge TPU.
Another option is this repo creates a version of Mask RCNN using MobileNet. I am very new to TensorFlow and deep learning in general, so I am not sure how to run this version on the Edge TPU, but maybe you can find it useful.

@Hi-Zed
See this.
https://github.com/PINTO0309/PINTO_model_zoo/tree/master/074_Yolact
YOLACT (Real-time Instance Segmentation) edge tpu model.

Close the issue. If you have any problems, please open it again.

Close the issue. If you have any problems, please open it again.


Fix HouseRules for 9/7/23 Demeo update

RoomFinder is broken (again) - Will look at that later
There's also a new MelonLoader error logged but HouseRules seems to work fine in all my tests

Looks as if the build dependencies bundle needs updating again as thje HouseRules_Essentials build can't find the new method names.


Configurable Options for Stable Wormholes

Is your feature request related to a problem? Please describe.
I'm attempting to make on-the-go storage a bit more difficult in my modpack while also separating the functionality of the stable wormhole and storage accessor. As-is, it's entirely possible to get most of the functionality of the storage accessor by repeatedly placing & destroying a stable wormhole (if a little tedious). I'd like to find a way to disincentivize this "solution."
Describe the solution you'd like
I'd like a config option to unlink the stable wormhole from the actuator when the wormhole is destroyed (or perhaps after a certain number of times it's destroyed?).
Additional context
N/A

Counting the times it has been destroyed is to be honest a bit too specific a use case for me, but I will add a config to delete link on destruction.
If you need the counting on top of that (maybe as a second setting with a default of "unlink after ... 1")  I am open for a PR


[Cosmos] Test changes for failing pipelines

This PR aims at fixing some of the test failures we've been seeing in the pipelines, which seem to mostly revolve around already existing containers. Seems like this bit of code was not updated to reflect the calls to "create_if_not_exists".

/azp run cosmos - python - tests

/azp run python - cosmos - test

/azp run cosmos - python - test

/azp run python - cosmos - tests

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


feat: create thread macro

Create a macro for automatically defining threads for a function:
#[thread(stacksize = 5123, priority = 1)]
fn some_thread() {}

->
#[distributed_slice(THREADS)]
fn __start_some_thread() {
   let stack = make_static!([0u8; 2048 /*paste stack size here */]);
   thread_create(some_thread, priority, &mut stack, 0 /* for now no arguments */);
}

Implemented in #65.


Fix deployment resource image setting

Changes

:bug: Set the image name in the deployment resources Install

/kind bug

/assign @pierDipi


Fix typo in vars/main.yml

occured -> occurred

@eltociear, welcome to the Omnia community!
You have one commit incorrectly signed off! To fix, first ensure you have a local copy of your branch by checking out the pull request locally via command line.  Next run
git push --force-with-lease origin patch-1
Commit sha: 4e9229d, Author: Ikko Ashimine, Committer: GitHub; The sign-off is missing.

@j0hnL I think I'm going to push this one through without the DCO. The change in question is a typo correction, so no concerns about code ownership.

@all-contributors please add @eltociear for code


Improve constant names

Describe the feature
To improve maintainability, constant names need to be revised. Constants names need to be more descriptive and words in constant names need to follow a grammar:
position (1) what is the constant ultimately? Is it a DEFAULT, MAX, MIN, AVERAGE, MEDIAN, TEST, SOFT_LIMIT etc.?
position (2) may come a word from (1), like we could have a DEFAULT_MAX_..., otherwise (3)
position (3) what's the unit? Is it BYTES, COUNT, LENGTH, RETRIES (retries is a specialised form of count), etc.
position(4) what is the thing that we are measuring? For example GET_POOLED_TRANSACTIONS_REQUEST, GET_POOLED_TRANSACTIONS_REQUEST_BODY, etc. If that thing refers to specific type, please include the whole type's name.
position (5) optionally we may need to describe (4) based on the context it's found in, e.g. INBOUND, OUTBOUND, AS_PENDING, POST_IMPORT, etc.
An example is DEFAULT_MAX_COUNT_HASHES_IN_GET_POOLED_TRANSACTIONS_REQUEST. Skipping the preposition is fine too if it remains unambiguous what the constant value holds DEFAULT_MAX_COUNT_HASHES_GET_POOLED_TRANSACTIONS_REQUEST.
That the constant name is long, is not a problem. If it isn't descriptive enough, it may result in some tedious bugs caused by misinterpretation of the constant from interpreting it based on the context it's found in, as compensation for the lack of descriptiveness in the name.
Additional context
No response

Happy to make a start on this as I become more familiar with the codebase!

Happy to make a start on this as I become more familiar with the codebase!

probably makes sense to do this in parts if you're still keen @JP2308 since otherwise you may find yourself endlessly chasing main branch! maybe crate by crate?

Is there any update on this? If not I can take this one. I can do crate by crate, rather than chasing main branch.

I have found myself chasing the main a bit.
I am making my way though it but slowly given my inexperience. I understand if you'd like others to take this on given they can work faster.

how about writing here which crate your are currently on @JP2308 and @shakeib98 so you're not working on the same one?

Yeah that makes sense. @JP2308 , please mention crates or folders you are taking on, I'll take other then.

Sounds like a plan, I have completed the following crates:

blockchain-tree;
config;
consensus;
ethereum-forks; and
etl.

Maybe @shakeib98 you start from the bottom and we meet somewhere in the middle?

Which crate is etl?
Also, I will take the following crates then

trie
transaction-pool
tracing
tokio-util
task
storage
stages
snapshot
rpc
revm
prune
primitives
payload

In total there are 25 crates at the moment, you can take the upper 12 I'll complete the rest 13.


Power Bi excel export

Hello, I am trying to export the file in xlsx format . while i can download the file the content is corrupted. Because the content is not properly encode.

Thanks for your comment, @Jaya-sys. This issue has been assigned to the doc author for review.

Thanks for your comment, @Jaya-sys.
This looks like a support request or product question rather than documentation feedback.
You might find a solution in the Power BI Community. If not, try our support page.


windowrule workspace empty ignores current workspace

There are some window rules in my config to open certain applications in an empty workspace, e.g. windowrulev2 = workspace emptynm, initialTitle:example
But, if the current workspace is empty, the application gets launched to the next one anyway.
Not sure if a bug or that behavior is correct, anyway I would like to know if there's a way to open the application in the current workspace if it's empty, or if it could be implemented.
Hyprland version is 0.41.2

Dealing with this issue too.. is this expected behavior? If so the docs are incredibly confusing and there seems to be no workaround. Very frustrating


CircleCI Migration - 1.0 to 2.0

This project is currently running on CircleCI 1.0 which will no longer be supported after August 31, 2018. Please start migrating this project to CircleCI 2.0.
Just saw the message above in https://circleci.com/gh/h5io/h5io/11 - I was not even aware the project is using CircleCI.

There isn't even a CircleCI config. Must have been errantly enabled. I couldn't find a way to remove it in a minute of looking, but there is probably no harm in keeping it.


--arch converted to --mtriple too directly?

How to reproduce
Compiler Distort example/ in Dplug with redub build --build=debug --arch=x86_64 --compiler=ldc2 =>

Repo here: https://github.com/AuburnSounds/Dplug/tree/master/examples/distort
DUB uses -march=x86-64, which works here.
Also works if triple is --mtriple=x86_64-windows-msvc is provided.
But redub provides --mtriple=x86_64, which doesn't seem correct.

Fixed in 1.3.15


[CARBONDATA-1265] Fix AllDictionary because it is only supported when single_pass is true

Fix AllDictionary because it is only supported when single_pass is true

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/2898/

Build Success with Spark 1.6, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder/311/

Have merged.


Vundle use of Bundle deprecated

https://github.com/gmarik/Vundle.vim/blob/v0.10.2/doc/vundle.txt#L234-L254
Deprecated Names  | New Names
Bundle            | Plugin
BundleInstall(!)  | PluginInstall(!), VundleInstall(!)
BundleUpdate      | PluginUpdate, VundleUpdate
BundleSearch(!)   | PluginSearch(!), VundleSearch(!)
BundleClean       | PluginClean(!), VundleClean(!)
BundleList        | PluginList
Note: The Bundle commands will be deprecated. You may continue using them,
but they may not get all future updates. For instance, we have enabled
comments on Plugin lines but not Bundle, since it requires a change in
command declaration.

:+1:


Hacks_modify_normals_py fails

Traceback (most recent call last):
File "/builds/workspace/continuous3/examples/Mechanics/Hacks/modify_normals.py", line 7, in
from siconos.mechanics.collision.tools import Contactor
ImportError: No module named mechanics.collision.tools

Fixed in 6265bf795df608f67d3d4d99e9b3d8b26b15a229, but this exposed a bigger problem.
First I will say that the error message reported here doesn't make sense, the script should have made it past the import Contactor line, since all mechanics Python scripts have this import.  For me, it broke at the call to makeBulletR.
But secondly, I needed to update this, since I added q1 and q2 as arguments to the BulletR() constructor, which should be taken from the ds1 and ds2 arguments to makeBulletR().  However, I tried adding the code:
q1, q2 = None, None
if ds1: q1 = ds1.q()
if ds2: q2 = ds2.q()

but this segfaults, because actually ds2 is a null shared_ptr in this case.  However "if ds2" is not correctly detected it, the SWIG director calls the makeBulletR with a valid object, not with None, but the object is a null pointer, so dereferencing it segfaults.  However, since it can't be compared with None or True/False, I don't know how to detect that ds2 is invalid.
I have left this check out of the script for now since ds1 is always valid and ds2 is always null, but it would be best to figure out how to fix the director to pass None when an SP is null, because I assume this affects all the Siconos binding code.

I filed a SWIG bug here:
https://github.com/swig/swig/issues/1029
Closing this issue as the problem is not specific to this issue, but affects all directors, and can likely only be fixed in SWIG.

Hey for the test looking weird,  the siconos install is never updated :( So it looks pretty old. I'll look into it.

Sadly it seems this is a known issue for swig.. http://www.swig.org/Doc3.0/Library.html#Library_std_shared_ptr

Note: There is somewhat limited support for %shared_ptr and the director feature and the degrees of success varies among the different target languages. Please help to improve this support by providing patches with improvements.

It seems to me the solution is to simply check the shared_ptr before generating the swig wrapper object, perhaps it would be a simple patch.


Add polyfill for fetchapi

I implemented a polyfill for fetch(input, opt_init);.
Since some more polyfills are required for fetch to work, it`s not ready to merge yet.

@googlebot I signed it!

While cool, fetch is not part of the ecmascript spec. I don't think we want to auto-inject polyfills for other specifications.

cc @shicks

The more I think about this the more opposed I am.  Since the api isn't tied to a language level, there isn't a clear way to disable the polyfill injection. For instance, I inject polyfills dynamically into my projects based on browser support. I don't want the compiler adding a polyfill when the targeted browser supports it natively.

I'd be in favor of a goog.net.fetch function instead then.

As far as I know, polyfills are dynamically injected if a function is not supported by a browser (or you force inject), aren`t they?
I guess goog.net.fetch which uses the native fetch if available would also be fine.

As far as I know, polyfills are dynamically injected if a function is not supported by a browser (or you force inject), aren`t they?

Polyfills are currently added at compile time based on language level. They won't overwrite existing support in a browser if it exists, but the full polyfill code is still present and increases the file size.

We need to have a general discussion about how to enable "platform" API polyfills.  Perhaps have someway to make the polyfill installation extensible.

While cool, I don't think making the polyfill installation extensible is worth it because this would mean having some sort of closure-polyfill-library.
I think we should either have all of them in the compiler or go with @MatrixFrog's proposal and implement non-ecmascript polyfills in closure-library (e.g. goog.net.fetch).

It seems clear that we won't be accepting this PR, though there is still some discussion to do about how best to provide polyfills for "platform" APIs like this one.
@Yannic perhaps you'd like to start a discussion on our email list?
https://groups.google.com/forum/#!forum/closure-compiler-discuss


Universal listings - Add 'has child pages' filter

Thanks @laymonage! I like the idea of using ChoiceFilter - that saves having to fight against the way BooleanFilter wanted to do things :-) All updated now.


Upgrade membrane_raw_audio_format to 0.11.0

Upgrades the :membrane_raw_audio_format dependency to 0.11.0

Changes from this PR has been implemented in https://github.com/membraneframework/membrane_raw_audio_parser_plugin/pull/5


[ESTSR] ConfidentialClientApplication.acquireToken took 20s to finish after specifying region

I am using the msal4j 1.13.4 package in spark project, ConfidentialClientApplication.acquireToken works well without providing region. I have line by line log to record the time each method is taking:

above log shows it finished in 1 second.
but after add region, all acquireToken is taking 20s to finish. got warning from AadInstanceDiscoveryProvider saying "Exception during call to local IMDS endpoint: java.net.SocketTimeoutException: connect timed out".
below is full log of get token call, acquireToken started from 18:39:27 future.join() finish at 18:39:47
23/02/07 18:39:27 INFO AadAuthByCertCallbackV2: acquireToken started - Tue Feb 07 18:39:27 UTC 2023, region - eastus, authority - https://login.microsoft.com/72f988bf-86f1-41af-91ab-2d7cd011db47, scope - https://eventhubs.azure.net/.default, 13.4
23/02/07 18:39:27 INFO AadAuthByCertCallbackV2: acquireToken started - Tue Feb 07 18:39:27 UTC 2023,  ConfidentialClientApplication build()
23/02/07 18:39:27 INFO AadAuthByCertCallbackV2: acquireToken started - Tue Feb 07 18:39:27 UTC 2023,  ClientCredentialParameters build()
23/02/07 18:39:27 INFO AadAuthByCertCallbackV2: acquireToken started - Tue Feb 07 18:39:27 UTC 2023,  acquireToken(parameters)
23/02/07 18:39:37 WARN AadInstanceDiscoveryProvider [ForkJoinPool.commonPool-worker-1]: Exception during call to local IMDS endpoint: java.net.SocketTimeoutException: connect timed out
23/02/07 18:39:37 WARN AadInstanceDiscoveryProvider [ForkJoinPool.commonPool-worker-2]: Exception during call to local IMDS endpoint: java.net.SocketTimeoutException: connect timed out
23/02/07 18:39:47 WARN AadInstanceDiscoveryProvider [ForkJoinPool.commonPool-worker-1]: Exception during call to local IMDS endpoint: java.net.SocketTimeoutException: connect timed out
23/02/07 18:39:47 INFO AadAuthByCertCallbackV2: acquireToken started - Tue Feb 07 18:39:27 UTC 2023,  future.get()
23/02/07 18:39:47 INFO AadAuthByCertCallbackV2: acquireToken started - Tue Feb 07 18:39:27 UTC 2023,  authResult.accessToken()
23/02/07 18:39:47 INFO AadAuthByCertCallbackV2: got access token, starttime: Tue Feb 07 18:39:27 UTC 2023, env eastus.login.microsoft.com, token length 1512
source code for above log:

do you know what might be the issue?

Hello @kentzhang : Just to confirm some things:

Is your application running in an Azure VM or some other Azure service, or just on your own machine?
Are you on any sort of corporate network that might block a call to that IMDS endpoint and cause a timeout?
Have you made any successful calls to a regional endpoint, or have they all timed out?

My first thought is that if you aren't on an Azure VM, then the 'IMDS endpoint' call should fail immediately rather than time out (because that endpoint is unreachable from outside of Azure) and then proceed to use the region you set instead of the one from the IMDS endpoint. However, if there's some sort of firewall holding up the request, then it could be timing out before it knows it's unreachable.

Thanks for looking into this, our program is a spark job running on Azure HDInsight cluster node, let me check if there is any firewall or other traffic control mechanism in place.

We have released a fix for this issue in 1.13.5 version of msal4j. Please retest.

Closing the issue. Please reopen or open another one if the issue persists.


fix bug for multiclass classification train.py

"pred" is belong to the previous train test.

Codecov Report

Merging #2747 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2747      +/-   ##
============================================
- Coverage     42.88%   42.88%   -0.01%
  Complexity      185      185
============================================
  Files           150      150
  Lines         11315    11315
  Branches       1140     1140
============================================
- Hits           4853     4852       -1
- Misses         6137     6138       +1
  Partials        325      325

Impacted Files
Coverage Δ
Complexity Δ

src/data/sparse_page_raw_format.cc
82.69% <0%> (-1.93%)
0% <0%> (ø)

Continue to review full report at Codecov.

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


Added 'type' to reserved keywords list in swift

Nothing special here, I've just added type to the list of reserved keywords for swift.

Good catch, thank you :)


Can't build for dart 2.8.1

../../../.pub-cache/hosted/pub.dartlang.org/rikulo_commons-4.0.1/lib/src/io/http_wrapper.dart:378:8: Error: The method 'HttpHeadersWrapper.add' has fewer named arguments than those of overridden method 'HttpHeaders.add'.
  void add(String name, Object value) {
       ^
org-dartlang-sdk:///sdk/lib/_http/http.dart:694:8: Context: This is the overridden method ('add').
  void add(String name, Object value,
       ^
../../../.pub-cache/hosted/pub.dartlang.org/rikulo_commons-4.0.1/lib/src/io/http_wrapper.dart:382:8: Error: The method 'HttpHeadersWrapper.set' has fewer named arguments than those of overridden method 'HttpHeaders.set'.
  void set(String name, Object value) {
       ^
org-dartlang-sdk:///sdk/lib/_http/http.dart:703:8: Context: This is the overridden method ('set').
  void set(String name, Object value,
       ^

Process finished with exit code 254

FYI: https://github.com/rikulo/commons/issues/6

Please run pub upgrade to fix this issue.


VisualShader "connections" array doubles in size after external edits

Godot version
4.0.dev (222e489d)
System information
Windows 10
Issue description
When editing a VisualShader externally (e.g. via Git operations) while the VisualShader is also open in Godot, re-saving it in Godot will double the length of the "connections" array.
This does not seem to happen for VisualScript files.
Steps to reproduce

Create a VisualShader file
Add a single node and a connection. In my case, opening the file in a text editor shows following connections: nodes/fragment/connections = PackedInt32Array(2, 0, 0, 0)
Keep the shader editor open, and simply save the shader file in your text editor (no changes are needed)
When focusing Godot Editor again, it should detect a change and reload the file
After reload, press Ctrl+S in Godot
Reload the file in your text editor, it will now contain nodes/fragment/connections = PackedInt32Array(2, 0, 0, 0, 2, 0, 0, 0)
Keep repeating this to rapidly multiply the size of your VisualShader file

Minimal reproduction project
No response

This happens because VisualShader::_set is used when reloading the resource. And inside, connect_nodes_forced is used, which does not check for the connection already existing.
Not sure what the correct solution would be, however.

cc @Chaosus

I guess not much we can do beside adding a check to prevent this from happening.


Support IPRequest to specify IP address for interface

Currently the IP is read from the pod network annotation into the NetworkSelectionElement.IPRequest but is not passed further to DelegateNetConf.IPRequest and then to the CNI_ARGS.
This PR fixes it.
The PR also includes some test fixes for both MacRequest and IPRequest (even the MacRequest was already passed correctly it seems the multus_test didn't actually tested it).
Note: the PR assumes the format of the IP on the network annotation is ips: "ip_address" as it was previously assumed by the code read the IP from the pod network annotation into NetworkSelectionElement.IPRequest.
Although it doesn't comply with the de-facto standard specifies the format is ips: ["ip1", "ip2", ...]

LGTM.  Any chance you'd want to do a follow-up to fix up the multiple IP requests issue?

Sure, we do. I just don't know how multiple ips should be passed over the CNI_ARGS.
Another issue is where does the subnet should be specified? I couldn't find it in the standard. I would expect the ip in the ips list be in the format ip/prefix.

In travis log, golint ./multus/... | grep -v ALL_CAPS has some error.
https://travis-ci.org/intel/multus-cni/builds/459436938#L484
Could you please take care of it?

@AlonaKaplan Thank you for your PR!
In case of IPv4/v6 dual stack case, user may need to specify IPv4 and IPv6 with CNI_ARGS hence we may need multiple ip address. That's the reason.

@AlonaKaplan Thank you for your PR!
In case of IPv4/v6 dual stack case, user may need to specify IPv4 and IPv6 with CNI_ARGS hence we may need multiple ip address. That's the reason.

Thanks for the explanation. How should it be reflected in the CNI_ARGS? Currently, as far as I know the de-facto standard specifies the IP should be passed over the IP key in the CNI_ARGS. If I have both ipv4 and ipv6 how would it look like in the CNI_ARGS?

Pull Request Test Coverage Report for Build 515

18 of 27 (66.67%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.3%) to 49.741%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

types/conf.go
0
3
0.0%

multus/multus.go
18
24
75.0%

Files with Coverage Reduction
New Missed Lines
%

multus/multus.go
1
54.27%

Totals

Change from base Build 509:
0.3%

Covered Lines:
576

Relevant Lines:
1158

💛  - Coveralls

Another issue is where does the subnet should be specified? I couldn't find it in the standard. I would expect the ip in the ips list be in the format ip/prefix.

@AlonaKaplan Interesting point. But every network attachment that wants L3/IP must already have IPAM configured somewhere that is driven by a plugin like host-local, static, or something internal to the plugin. Otherwise there is nothing to handle the IP CNI_ARG that is passed to the plugin. And the thing that handles IPAM therefore already knows what the subnet is, and if the CNI_ARGS static IP does not fall within the range, then it is invalid and should be rejected. If it does fall within the range, then we already know the range and the user doesn't need to specify redundant information.
But that falls down somewhat for a case like Tomo's 'static' IPAM CNI plugin, which doesn't have configuration to accept a subnet via JSON config and then validate an incoming CNI_ARGS static IP against that subnet. Instead it takes the CNI_ARGS static IP as CIDR notation.
So, a question for the NPWG I think, whether we should clarify language around section <IP_ADDRESS>.3 "ips" to allow CIDR notation and say that it should be validated by the IPAM function of the plugin in any way the plugin desires.

@dcbw I agree with you that Spec allows to both, CIDR-notation and non-CIDR. From multus/meta-plugin point of view, these parameter goes to IPAM plugin for now and IPAM should take care of that.
Sometimes I feel that CNI community could come up with standard IPAM argument formats to interoperability point of view... what do you think about it?

@dcbw @s1061123 please re-review, I've added support for CIDR format.  Also found (and fixed) a bug that the macs and ips of all the networks were appended to the CNI_ARGS.

LGTM


New version: Serde v3.0.0

Registering package: Serde
Repository: https://github.com/bhftbootcamp/Serde.jl
Created by: @gryumov
Version: v3.0.0
Commit: 2935a145322fc7bc3e38575f42e2fba7c2ce2284
Reviewed by: @gryumov
Reference: https://github.com/bhftbootcamp/Serde.jl/commit/2935a145322fc7bc3e38575f42e2fba7c2ce2284#commitcomment-140117903
Description: Serde is a Julia library for (de)serializing data to/from various formats. The library offers a simple and concise API for defining custom (de)serialization behavior for user-defined types
Release notes:

## 3.0.0 (22/03/2024)

### Added

- Macro `@serde_pascal_case` to transform field names from PascalCase to snake_case for deserialization ([#26](../../pull/26)).
- Macro `@serde_camel_case` to transform field names from camelCase to snake_case for deserialization ([#26](../../pull/26)).
- Macro `@serde_kebab_case` to transform field names from kebab-case to snake_case for deserialization ([#26](../../pull/26)).
- `Serde.ser_name` to override the default field name serialization ([#26](../../pull/26)).
- `Serde.ser_value` to override the default value serialization ([#26](../../pull/26)).
- `Serde.ser_type` to override the default type serialization ([#26](../../pull/26)).
- `Serde.ser_ignore_field` to determine if a field should be ignored during serialization ([#26](../../pull/26)).

### Changed

- Renamed macro `@ser_json_name` to `@ser_name` for consistency with other serialization macros ([#26](../../pull/26)).
- Renamed `ignore_null` to `ser_ignore_null` to align with serialization function naming conventions ([#26](../../pull/26)).
- Renamed `ignore_field` to `ser_ignore_field` for clarity in serialization customization ([#26](../../pull/26)).

I've been wondering, is it possible to support markdown renders for release notes? It would have made PRs much prettier.
[noblock]


Add Changelog file?

Hi!
I just updated LeaderF and was surprised that ESC didn't work as expected. I finally found that it's now TAB that is used for that.
I think it would be interesting to list the fixes, changes and improvements in a Changelog file.
What do you think about that?

In the future, I'll try my best to keep LeaderF backward compatible, so users don't need to care about the Changelog file then.

I'm not against breaking backward compatibility from time to time. I prefer a bad design to be ironed out once and for all instead of hacking around it for many years.
That being said, maybe I could just read the commit list.. it's not that long indeed.

A good template for CHANGELOG files: https://keepachangelog.com/

THANKS

Added.


"this.configureStart is not a function error" when running on Sharepoint Online workbench.aspx

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Enhancement / Suggestion
[ ] Additional article idea

Expected or Desired Behavior
I can successfully run the react-todo-basic sample application on my local workbench.html file. I expect it would work similarly when using workbench.aspx on Sharepoint Online.
Observed Behavior
When running the webpart on Sharepoint Online/workbench.aspx, the initial webpart view is displayed, but when I click the webpart's "Configure" button to open the property pane, I get a "this.configureStart is not a function" error in the _openPropertyPane method of the TodoWebPart.ts class.
Steps to Reproduce

Start with the react-todo-basic sample application.
Make modifications as described in the "New BaseClientSideWebPart package location" section of the Drop 5 Release Notes (this is required or you get the error described here)
Run npm install
Run gulp serve
Add the Todo webpart to the Sharepoint Online workbench.aspx
Click the "Configure" button in the webpart. Nothing happens on the screen, but the error is visible in console on every click of the "Configure" button.

We apologize for the inconvenience. We know about the issue and we are working on fixing it soon.

Is there any workaround ? this is also happening on a brand new helloworld. Nothing will run on my Developer Site Collection workbench since last Weekend. Works locally. To reiterate this is React and plain javascript sample web-parts. My learning has effectively ceased. Is there any config I can do manually, I have followed the Drop 5 instructions but not helping. Should I go back to an earlier version ?

for anyon with same problem i went back to an old copy of the workbench.aspx and seems to work again

I tried the latest Drop 6 and changed to refreshPropertyPane(). It seems working well on SPO.


Removed implicit conversion in AuthRequest

What the title says.

:+1:

Current coverage is 94.54%

Merging #322 into master will change coverage by -0.01% by f321cab

Powered by Codecov

Looks good to me, thanks!

Merging this in. Thanks!


Azure Managed Grafana is missing in this list

Hi, we use this list for naming our resources. However, I see that Azure Managed Grafana is not in this list yet. Can you please add this?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 8c22eb33-2fde-55b4-021d-e229c857e51b
Version Independent ID: 1f767675-7d50-6ea5-b798-1323cd4a63a1
Content: Abbreviation examples for Azure resources - Cloud Adoption Framework
Content Source: docs/ready/azure-best-practices/resource-abbreviations.md
Service: cloud-adoption-framework
Sub-service: ready
GitHub Login: @stephen-sumner
Microsoft Alias: ssumner

@RHooijer - Thanks for your feedback! We're reviewing now. Based on the following reference, amg is (not surprisingly) the likely candidate. I'll keep you posted.

https://learn.microsoft.com/en-us/cli/azure/grafana?view=azure-cli-latest

https://github.com/MicrosoftDocs/cloud-adoption-framework-pr/pull/4363


Cancelable should back-pressure and be () => Future[A]

This is the biggest change in Cats-Effect 3, with which I don't fully agree with.
For the discussions around the issue, see: https://github.com/typelevel/cats-effect/discussions/1979
But if Cancelable does not turn into () => Future[A] for back-pressure, then Cats-Effect becomes de-facto incompatible with Monix. This goes for CancelableFuture too.
It needs to happen, and it needs to happen ASAP.
Calling all contributors that may want to help, as this is a big task.

@paualarco sorry for the silence. I'm trying to come up with a plan — this week, it will happen 🙂

I am having second thoughts about this issue.
The main problem is that cancelation, in Monix (and RxJava, and Reactive Streams), has an entirely different semantic than that of Cats Effect 3, and I disagree that the cancelation model of CE 3 is the correct choice.
This discussion is still relevant: Let's discuss the cancelation model


fix: A regression with CLS on the Home page

A copy from https://github.com/vtex-sites/nextjs.store/pull/251. See the original PR for more details.

Confirmed the CLS has been fixed:

Before
After

I'm going ahead and merge this as it's already been approved in https://github.com/vtex-sites/nextjs.store/pull/251 and I've confirmed it worked.


Please add all Cybrix.AI GPTs

There are 10 so far.
Source: https://cybrix.ai/gpts.html
gpts = [
    {
        "title": "Context Keeper",
        "description": "Saves and loads prompts on command",
        "link": "https://chat.openai.com/g/g-kreBYcHzs-context-keeper"
    },
    {
        "title": "News Bias Analyzer",
        "description": "Effortlessly Analyze the News: Automatically Fetch and Analyze the Latest Headlines or Paste a News Article's URL or Text! Get Instant Graphs, Summaries, Categories, Tags, and In-Depth Source Analysis.",
        "link": "https://chat.openai.com/g/g-9tbHB4O4r-news-bias-analyzer"
    },
    {
        "title": "SOP Writer Pro",
        "description": "Writes and formats Standard Operating Procedures with diagrams.",
        "link": "https://chat.openai.com/g/g-LcI10C2EH-sop-writer-pro"
    },
    {
        "title": "Crop It!",
        "description": "Quick Image Cropping & More: Experience fast, precise cropping and resizing, along with advanced adjustments for brightness, contrast, and white balance. Perfect for image and logo modifications, favicon generation, and other image enhancements – all in one streamlined tool.",
        "link": "https://chat.openai.com/g/g-3K6u1eRlD-crop-it"
    },
    {
        "title": "Data Only",
        "description": "No small talk. Only cold, hard results.",
        "link": "https://chat.openai.com/g/g-z2ZzMfvqv-data-only"
    },
    {
        "title": "GPT Action Builder",
        "description": "Builds compliant OpenAPI schemas for custom GPT actions",
        "link": "https://chat.openai.com/g/g-mLeQhjN21-gpt-action-builder"
    },
    {
        "title": "Regex Crafter",
        "description": "Professional, concise regex creator.",
        "link": "https://chat.openai.com/g/g-TAANxtI3q-regex-crafter"
    },
    {
        "title": "Screenplay and Script Converter",
        "description": "Converts text to script format, keeps original dialogue, uses markdown.",
        "link": "https://chat.openai.com/g/g-b87soezbf-screenplay-and-script-converter"
    },
    {
        "title": "Chemistry Companion",
        "description": "Professional chemistry assistant, SMILES supported molecule and reaction diagrams, and more!",
        "link": "https://chat.openai.com/g/g-XRqFkyefW-chemistry-companion"
    },
    {
        "title": "RudeGPT",
        "description": "RudeGPT: The Assistant with Attitude.",
        "link": "https://chat.openai.com/g/g-Fz2gm6LcG-rudegpt"
    }
]

all added.


Handle exc_info=True used outside of exception

Fixes: https://github.com/namespace-ee/django-datadog-logger/issues/18
I added a test settings file to make tests start. I couldn't find a better way to setup Django settings. I amended the setup by prefixing test commands with DJANGO_SETTINGS_MODULE=tests.settings. Now make test, make coverage and tox all work.
To make tox work I had to:

set max line length to 120 characters (to match pyproject.toml)
remove an incorrectly escaped hyphen from the request ID matcher

I'm sorry for putting it all in a single PR. I did that because I wasn't able to run all the required commands mentioned from CONTRIBUTING.rst. I may extract the actual fix into a separate one if you prefer. The changes are cleanly split between commits so it would be an easy job.

Due to inactivity I decided to close and focus on maintaining a fork.


Include Smiles Render

Reference: https://smilesdrawer.surge.sh/use.html
Render code block with keyword 'smiles' by smilesdrawer. Edit block with codes, edit smiles by typing or copy from chemdraw js. https://chemdrawdirect.perkinelmer.cloud/js/sample/index.html#

In favor of #2


tx raw upgrade-chain gives 'Failed Tx: no confirmation' error

Crate
relayer
Summary of Bug
Step 3 of the Hermes guide's testing client upgrade procedure gives a Failed Tx: no confirmation error, although the tx itself succeeds ->
$ hermes tx raw upgrade-chain ibc-0 ibc-1 07-tendermint-0 10000000 300
Aug 13 21:06:49.881 DEBUG [ibc-0] waiting for commit of block(s) 3EB2DF97AB3D2FC94918F5FC7969286E357722DECDDD65A24DDEAEE970447E43
Error:
   0: upgrade chain error
   1: failed while submitting the Transfer message to chain ibc-0
   2: Failed Tx: no confirmation

Location:
   /home/hussaini/.cargo/registry/src/github.com-1ecc6299db9ec823/flex-error-0.4.2/src/tracer_impl/eyre.rs:10

Backtrace omitted.

Apparently, this is because the transaction doesn't result in the creation of events and Hermes interprets this as a 'no confirmation' error.
Version
v0.6.2
Steps to Reproduce
Run steps 1 to 3 of the Hermes guide's testing client upgrade.
Acceptance Criteria
Hermes is able to detect that the transaction succeeded and prints a success message.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Also note that governane proposals belong to the SDK, not the IBC module, cf a remark by @hu55a1n1.
GovernanceProposals have a content field.
https://github.com/informalsystems/ibc-rs/blob/ba43f88d6813fb3fe8a60617d797de238df21955/relayer/src/upgrade_chain.rs#L131
For chain upgrades gov proposals that affect the IBC state, this content field is an IBC type from ibc.core.client.v1
https://github.com/informalsystems/ibc-rs/blob/773e50c31fb17e1b6655a1a1d5b44883c679f229/proto/src/prost/std/ibc.core.client.v1.rs#L57
This is all to say that the event we're trying to extract is probably produced by SDK (not the IBC module) unlike ChannelEvent for instance. If we want to augment the logic in from_tx_response_event we might want to add a GovProposal::try_from_tx(event) match arm. The implementation and logic for GovProposal should ideally reside in an Cosmos-SDK Rust implement, not in the ibc crate. For the moment, however, we have no SDK, so we can just add  GovProposal in our ibc crate in some temporary place and add a fixme/todo note.

The semi-reasonable course of action seems to me is to:

Add a cosmos::gov::events module (to match the theme of core::ics02_client::events et al.) to define structures pertaining to Cosmos SDK events, and define a Proposal enum with a SoftwareUpgrade { voting_period_start: Height } variant there.
Extend IbcEvent with a SubmitProposal(cosmos::gov::events::Proposal) variant and add a Proposal::try_from_tx method call to the chain of event parsing calls in from_tx_response_event.
In Proposal::try_from_tx, ignore any "submit_proposal" ABCI event that does not have a "proposal_type" key, and try to translate to the only supported proposal variant if the key's value is "SoftwareUpgrade"

Note that the events captured in https://github.com/informalsystems/ibc-rs/issues/1288#issuecomment-1044384411 were produced by gaia 4 and a legacy upgrade proposal was sent. I'll check if gaia 6 has any difference.

Does this issue affect anything except the user experience in testing client upgrade (i.e. the user will see an error while testing the client upgrade procedure but everything else works fine)? And is the error seen only with tx_confirmation = true?
Not clear if we should complicate the relayer code with handling the governance events as in #1909 .
If it's just for hermes tx raw upgrade-chain.. CLI which is a test command afaik, maybe we can just update the testing instructions and mention that the user should use confirmation false and manually check the tx hash.

And is the error seen only with tx_confirmation = true?

The tx raw upgrade-chain CLI ignores tx_confirmation = true because it calls directly into send_messages_and_wait_commit, so the CLI blocks waiting for events:
https://github.com/informalsystems/ibc-rs/blob/ba43f88d6813fb3fe8a60617d797de238df21955/relayer/src/upgrade_chain.rs#L144

If it's just for hermes tx raw upgrade-chain.. CLI which is a test command afaik, maybe we can just update the testing instructions

Good point! I will double check with ibc-go team to see if upgrade-chain is expected to be used in production or in extended testing scenarios.

manually check the tx hash.

This is still problematic with the simpler approach now implemented in #1979: the original problem is that no recognized events get retrieved for the transaction query, and the same is true for query tx events.


[Dev] Implement ScheduleUnsafeInjectCallArgument Primitive to Hack decoding

This pull request includes several changes to improve the handling of decoding and memory prefetching in the GPU intrinsics and matmul dequantization logic. The key changes involve adding support for offset handling in decoding functions, updating the pass context for TVM transformations, and enhancing the scheduling logic for shared memory prefetching.
Decoding Enhancements:

Added new decoding functions decode_i4_to_f16_scale_offset, decode_i4s_to_f16_scale_offset, and decode_i4u_to_f16_scale_offset to handle offset during decoding (bitblas/gpu/intrin/lop3.py).
Updated get_fast_decode_intrin to append _offset to function names when storage_scope is "warp" and scaling is enabled (bitblas/gpu/intrin/lop3.py).

TVM Transformation Context:

Modified the tvm_callback_cuda_postproc function to include "tir.disable_cse_tir": True in the TVM pass context configuration (bitblas/base/utils.py).

Scheduling and Prefetching:

Enhanced shared memory prefetching logic to handle different reduction depths and weight transform kinds in sch_shared_memory_prefetch_with_config (bitblas/gpu/matmul_mma_dequantize.py).
Updated the get_param_indices and fetch_to_shared functions to support reduction across threads (bitblas/gpu/matmul_mma_dequantize.py). [1] [2]

Intrinsic Definitions:

Added new intrinsic definitions for warp scope in intrin_definitions to support various configurations (bitblas/gpu/intrin/lop3.py).

These changes collectively enhance the functionality and performance of the GPU intrinsics and matmul dequantization processes.

Also fixed codeql warning ref to #121

Add GPTQ Repack Test to checkout the integration correctness.


Heat exchanger tubes not transporting fluid?

Heat exchanger tubes appear to not transport fluid at all, whether part of the multiblock or not. When I do something stupid like add a call to pushFluid() in TileHeatExchangerTube's update method and recompile the mod, they do transport fluid as expected when set to input spread, but the heat exchanger multiblock still does nothing (the input fluids just zip to the end of the chain of tubes and stay there). Obviously, I don't know how exactly this is supposed to work, but is this a bug or just something that's intentionally not implemented yet?

Which version of the mod is this? How you you have the heat exchanger set up?

This is, to my knowledge, the most recent beta version (I pulled it off this repository and compiled it myself, but I might have gotten the wrong branch or something). To test, I tried setting up a heat exchanger that was just a single line of tubes running down it - the multiblock formed, but no fluid entered any except the first tube. I also tried the same thing but with two lines of tubes in contraflow.

So the overhaul branch? If so, heat exchangers aren't implemented yet, so that's why it doesn't seem to work correctly in general :P
Any sort of latent functionality is just a hangover from the pre-overhaul version.

Okay, thanks.

No worries :)


Create a WebSocket client interface and basic UI for debugging

We want to start working on a class interface for the WebSocket connection to the https://github.com/vkottler/runtimepy server.
Once we're ready to actually connect, we want to integrate a workflow for this package so that we can easily just a runtimepy server / script via the Makefile. We need runtimepy to be able to run external scripts, including one that servers data via websocket.
For now this can just be a developmental script inside this repo

Got a basic server script done, now need to create a form to specify host and port


Getting a 400 (Bad Request) error on website

Step to reproduce

Go to url
Inspect page
Console tab will show the error message

From network tab, 4 dir requests are made

Bad request :

Successful request:

Bad request seems doesn't have any container


Change love from local to global

Hi, first of all, thanks for creating this.
I am new to Teal but I have a bit of experience with Love2D from long ago and I was thinking if it wouldn't be better to change local love to global love.
I tried to use your typings and right now to make it work I'd have to either local love = require('love') or global love = require('love') in my code. This makes tl place love = require('love') in the resulting Lua files. I tried running that right now and it works, but I was thinking that maybe it would be better to just make love a global in love.d.tl and then preload it for type checking using tl options.
Please do let me know if what happens currently is actually a good situation, I've been out of the loop for a good while, I've yet to properly catch up with API changes since Love2D 0.9, so I presume best practices might have changed as well.

I think that's good idea. I normally use local love = require("love") idiom for all my LOVE projects but accessing global love is fine as of 11.3.

Note that because Teal lack of class inheritance system, you have to cast in many cases.
local love = require("love")

local mesh: love.graphics.Mesh -- Remember the type of this variable is love.graphics.Mesh

function love.load(arg: {string}, _: {string})
	mesh = love.graphics.newMesh({
		{100, 100, 0, 0, 1, 0, 0, 1},
		{200, 100, 0, 0, 0, 1, 0, 1},
		{100, 200, 0, 0, 0, 0, 1, 1},
	}, "triangles", "static")
end

function love.draw()
	love.graphics.draw(mesh as love.graphics.Drawable) -- Notice that we have to cast it to love.graphics.Drawable or Teal checker will spit errors.
end

Don't worry, casting is there for convenience. The generated Lua code is as follows
local love = require("love")

local mesh

function love.load(arg, _)
   mesh = love.graphics.newMesh({
      { 100, 100, 0, 0, 1, 0, 0, 1 },
      { 200, 100, 0, 0, 0, 1, 0, 1 },
      { 100, 200, 0, 0, 0, 0, 1, 1 },
   }, "triangles", "static")
end

function love.draw()
   love.graphics.draw(mesh)
end


Make partition modularity available in adata.uns

This PR stores modularity metric into adata.uns. I also added key_added as a suffix to uns key "louvain", so that we can run louvain with different parameters and store their quality metrics separately.
import scanpy as sc

sc.settings.verbosity = 3
adata = sc.datasets.pbmc3k()
sc.pp.pca(adata, svd_solver='arpack')
sc.pp.neighbors(adata)
sc.tl.louvain(adata, resolution=2.0, key_added='newkey')
adata.uns

I'm really bad at writing warning messages and naming things, so feel free to edit these :)

I think it's really good to record the key added. I have a couple questions about the quality score.

Should it still be called a modularity score if we aren't using the modularity quality function?
Do you have some use cases for recording the modularity score? My impression was that it may not have much interpretable meaning, especially between different graphs.

Also, should this stuff be mirrored to leiden?

I think a use case for recording modularity would be to assess the quality of a partition on the same network. Louvain or Leiden are both heuristics that are not guaranteed to get the global optimal partition. So if you run with a different random seed, you will get a different result. By outputting the modularity you can compare the quality of the partition from different random seeds.

I think it's really good to record the key added. I have a couple questions about the quality score.

Should it still be called a modularity score if we aren't using the modularity quality function?

If another flavor is used, we do not record or print anything about quality, so it's ok. But there is a chance that user supplies a partition type from Louvain/Leiden that is not using modularity optimization (e.g. CPM or Surprize). In this case, we do not print the term scaled modularity anymore.

Do you have some use cases for recording the modularity score? My impression was that it may not have much interpretable meaning, especially between different graphs.

To me, it's like any statistical measure which gives a rough idea about a concept, like correlation or silhouette coef. It's far from conclusive just by itself, but it gives a "feeling" of how "well-clustered" the data is (and how good we are at finding them). Without complementing it with other measures, it's not more than just a "feeling" :)

Also, should this stuff be mirrored to leiden?

It's done.

If another flavor is used, we do not record or print anything about quality, so it's ok. But there is a chance that user supplies a partition type from Louvain/Leiden that is not using modularity optimization (e.g. CPM or Surprize). In this case, we do not print the term scaled modularity anymore.

By default we don't use modularity as the quality function, we use the RBConfigurationVertexPartition. I believe that we should get a quality score from the leiden and louvain packages regardless of quality function though. Could we use the term quality_score and also store quality_function used in params like: .uns[key_added]["params"]["quality_function"] = partition_type.__name__?

To me, scaled modularity is like any statistical measure which gives a rough idea about a concept, like correlation or silhouette coef. It's far from conclusive just by itself, but it gives a "feeling" of how "well-clustered" the data is (and how good we are at finding them). Without complementing it with other measures, it's not more than just a "feeling" :)

A couple follow up points on this and @LuckyMD's points

I don't actually know how different the quality score can be for different solutions. Any chance you have some stats on quality scores from multiple clusterings? I'm mostly wondering if "good" clusterings are associated with high quality scores.
I think if a user sees a value like "quality" they could ascribe more meaning to it than it deserves. I think we should add some docs about what it means, and how to interpret it.

A couple follow up points on this and @LuckyMD's points:
I don't actually know how different the quality score can be for different solutions. Any chance you have some stats on quality scores from multiple clusterings? I'm mostly wondering if "good" clusterings are associated with high quality scores.
I think if a user sees a value like "quality" they could ascribe more meaning to it than it deserves. I think we should add some docs about what it means, and how to interpret it.

I'm not sure I entirely understand your point. The quality score is modularity, which is optimized. Thus a "good" partition is a high quality score by definition. Or what are you referring to as "good"?
I have looked at some stats for communities in protein-protein interaction networks, and the quality of the communities can change dramatically with louvain output there (could find the link if you think it's relevant). However, modularity as a score is fairly degenerate toward the optimal score and therefore the value often doesn't change that much between the optimized partitions. I'm not sure how this is on the knn graphs though.

By default we don't use modularity as the quality function, we use the RBConfigurationVertexPartition. I believe that we should get a quality score from the leiden and louvain packages regardless of quality function though. Could we use the term quality_score and also store quality_function used in params like: .uns[key_added]["params"]["quality_function"] = partition_type.__name__?

Strictly speaking, you are correct that the quality function of RBConfigurationVertexPartition is not exactly the same as modularity, although it is called unscaled modularity in the code.
There are two main differences between RBConfigurationVertexPartition and ModularityVertexPartition which uses typical modularity optimization. 1) Scaling by the number of edges and 2) the resolution parameter (as it's written here in the note). I account for 1) in the code but using a resolution parameter other than 1.0 would lead to values different than modularity due to 2). Right now, for example, you can get a perfect quality (=1.0) by just setting the resolution to 0.0 :D I don't think that'd mislead users though. After all, that's what the algorithm uses for optimization.
I can think of two solutions. We can report typical modularity regardless of the partition_type, namely:
modularity_part = leidenalg.ModularityVertexPartition(g, initial_membership=part.membership)
q = modularity_part.quality()

or we can report the original quality value as "raw quality" (whatever it is) and the modularity together. It's in the "hint" verbosity level anyway.
Regarding the suggestion to record  partition_type.__name__, I think it's a good idea. I'd record it in the uns[uns_key]['partition_type'] though, not in quality_function.

To me, scaled modularity is like any statistical measure which gives a rough idea about a concept, like correlation or silhouette coef. It's far from conclusive just by itself, but it gives a "feeling" of how "well-clustered" the data is (and how good we are at finding them). Without complementing it with other measures, it's not more than just a "feeling" :)

A couple follow up points on this and @LuckyMD's points

I don't actually know how different the quality score can be for different solutions. Any chance you have some stats on quality scores from multiple clusterings? I'm mostly wondering if "good" clusterings are associated with high quality scores.
I think if a user sees a value like "quality" they could ascribe more meaning to it than it deserves. I think we should add some docs about what it means, and how to interpret it.

That makes sense. Maybe calculating typical modularity and using the term modularity is safe enough, since definition of modularity is available everywhere.

On the definition of modularity, I did go back over some literature and saw that modularity sometimes takes multiple meanings within a paper. It can be either the specific quality function, or when used like "modularity optimization" can refer to the whole class of partition optimizing algorithms (which are generic wrt quality function) like louvain (I like section IV F of this paper for an overview).
@LuckyMD

The quality score is modularity, which is optimized. Thus a "good" partition is a high quality score by definition. Or what are you referring to as "good"?

I think of the quality function/score as being determined by the partition_type.
To me, a good partition is one that seperates data points into discrete groups which reflect some true underlying structure. I put this in quotes since it’s ill-defined, however we can tell when it’s definitely not true. A high quality score for a partitioning is just a high quality score for a partitioning.
@gokceneraslan

we can report the original quality value as "raw quality" (whatever it is) and the modularity together.

Personally, I would just report the quality metric calculated by the quality function used. To me, the point of returning this value would be to know if the optimization went well, which is probably best measured by looking at the optimized value. This would also simplify the code a bunch. I think there's another case for trying to tell if it's a "good" partitioning, but I think that should be handled seperatly.

Regarding the suggestion to record partition_type.name, I think it's a good idea. I'd record it in the uns[uns_key]['partition_type'] though, not in quality_function.

That's reasonable. Just to be sure, we'd keep it in uns[uns_key]["params"]['partition_type'] like it is now?

Personally, I would just report the quality metric calculated by the quality function used. To me, the point of returning this value would be to know if the optimization went well, which is probably best measured by looking at the optimized value.

The quality score returned by RBConfigurationVertexPartition is unscaled modularity, so it's something like 41726.23. So how would one interpret that? I don't think there is any point in reporting raw RBConfigurationVertexPartition quality value.

That's reasonable. Just to be sure, we'd keep it in uns[uns_key]["params"]['partition_type'] like it is now?

We don't store the partition_type anywhere now, but yes we'll after this PR.

After the discussions we had here, I'm reporting the standard modularity now. Please have a look and let me know if it makes sense.

Sorry for the long delay – I was away on a retreat. By the way, congrats on the spatial letter!
I'm not sure I understand why you'd report the standard modularity if the partitioning was done with multi resolution modularity or some other quality function. To me, this makes the metric being pretty disconnected from the computation that was run. It seems likely that there could be non-proportional relationships between the whatever quality function is used and unscaled modularity. For example there could be a case where: partitioning A has higher unscaled modularity than partitioning B, but B has higher multi resolution modularity quality with resolution .8 than A. If the multi resolution modularity is what was run, I think the logging should reflect that run resulted in a "better" optimization.
Separately, if it's meant to assess the quality of the clustering, why not calculate something like silhouette? From my perspective, it would be because that's separate enough from the process of clustering that it should be run separately.

I think the PR is now good as is. I will be quite busy for some time, therefore I won't have time for further discussions. Please feel free to close, update or merge it.

@gokceneraslan did you address @ivirshup’s questions?


Composite resource single file metadata files appear when an unrelated file is unzipped

In the 1.16 release candidate deployed on https://dev-hs-6.cuahsi.org I did the following

Added file (Readme.txt)
Switched to edit mode to set metadata for that file.  This converted the file to a Single file aggregation
Added an unrelated zip file (logan_vrt_small.zip)
Unzipped the zip file.
The content display now shows in addition to the folder from the zip file, the two metadata files from the single file content type aggregation readme.txt.  This should not be the case.  *meta.xml and *resmap.xml should be hidden, and they were until the unrelated zip file is unzipped.

The resource where I showed this is https://dev-hs-6.cuahsi.org/resource/4245b73c5cb54111b276dea5239f755b/ (but this will disappear next time hs-6 is rebuilt.
An additional aspect of this problem is that if after step 2, I put /debug after the resource URL I get the following errors.

This leads me to interpret that the display or hiding of metadata files is being managed by whether they are added to Django or not.  This does not seem to be a reliable approach.

@alvacouch perhaps you could just point to the management command functions that need updating and @aphelionz @phuongdm or @sblack-usu could make the changes.

I have made a fix to the unzip function so that it does not add the aggregation xml files to Django. I can create a PR tonight. That may help in modifying the management commands.

@alvacouch Please check my last commit (see above). I am assuming that should fix the debug_resource.

Could I get some others to weigh in on the implementation here?  While making the changes to unzip files to a new folder, I also made it so the folder index would increment to prevent overwriting existing files.  (https://github.com/hydroshare/django_irods/pull/44#discussion_r203739444)
Turns out there are a couple of tests out there that I didn't see which tests for the action to be blocked if files are overwritten.  (http://ci.hydroshare.org:8080/job/hydroshare-pull-requests/1741/testReport/hs_core.tests/TestResourceFileFolderOprsAPI/test_resource_file_folder_oprs/)
I can update this PR to be compliant with these tests, but I thought I'd get some input before updating the PR.

@sblack-usu What's the status of this?

it's been merged into the main composite resource branch

Fixed with 1.16.


bau: Fix pact upgrade on publicapi side

This fixes https://github.com/alphagov/pay-publicapi/pull/468. Why the PR was failing:

We have a pact defined in publicapi
https://github.com/alphagov/pay-publicapi/blob/master/src/test/resources/pacts/publicapi-connector-get-payment-with-gateway-transaction-id.json#L63-L64 which has

"settlement_summary": {
  "capture_submit_time": null,
  "captured_date": null
}

but this actually gets published as
"settlement_summary": {}

due to the pact upgrade.

On the provider side, i.e. connector, it actually returns

"settlement_summary": {
  "capture_submit_time": null,
  "captured_date": null
}

which causes the matching to fail.
This isn't great as they are equivalent and breaks Postel's Law (see
https://docs.pact.io/getting_started/matching/gotchas):
Be liberal in what you accept  - when verifying a pact in the provider project,
the response body and headers may contain fields that were not defined in the
expectations, on the assumption that any extra field will be ignored by your
consumer. This allows a provider to evolve without breaking existing consumers.
Admittedly this is talking about headers rather than json body.
This commit causes null values to not be returned in the json body which will
fix the issue.

Unfortunately this PR will break selfservice. The best thing is to wait for https://github.com/DiUS/pact-jvm/issues/877 to be fixed.


Help

superpatch_network_constrution.py doesn't seem to work, are you sure root_dir is /Graph_test/?

We will check it out and update the code this week, which includes the minor bug fix

We have tried many ways to run your program, but this seems to be very difficult to succeed. Hope to see a working version.

@zqy396 have you solved the error? it seems that the root_dir should be contains the files from both original and superpatch

I am a beginner. I would like to ask, the article says that distance d and Angle a are embedded as edge features. Take the length,d ={d11,d12,... dNN}, where d11 and patch feature dimensions are consistent, what is the specific calculation method for distance d? Are the corresponding elements of two node eigenvectors subtracted?

I am a beginner. I would like to ask, the article says that distance d and Angle a are embedded as edge features. Take the length,d ={d11,d12,... dNN}, where d11 and patch feature dimensions are consistent, what is the specific calculation method for distance d? Are the corresponding elements of two node eigenvectors subtracted?

i can`t run the whole program,if i run main.py,i will take an error--ZeroDivisionError: No admissable pairs in the dataset.


Question: Migrating to a state that is not "New"

Hi,
I have been using your tool for some VSTS to VSTS work item migrations and its been great :)
Currently I'm working on my own project to take some exported CSV data from third party systems like Jira to import to VSTS. Using the API I create a create a new workitem and then set the state as "Committed" for example. When i do a validate it fails and the state is reported as invalid because the only valid values are "New".
I was intrigued to see how you achieved the result i was looking for so started searching through the code and found where you set the state in WorkItemMigrateContext.cs Ln186. I looked and it doesn't look like your are doing anything special to set the state to a value that inst a start state. The interesting thing is, i found the State field in the workitems field list and the "isDirty" for the workitem and the field itself is true and the fields status property is "InvalidListValue". When i do a validate however the invalid fields are reported as 0.
However i can get the validate to fail in the sync migrator if i leave the application in a break point for a while and come back to it, like there is some sort of race condition in the API.
Any ideas about setting new workitems to a state like Done or Comitted instead of saving as New and then reopening to save the actual state i want?

I would love for you to build your tool into the VSTS Sync Migrator..
You should look to interact directly with the Jira API
When you instantiate the Work Item Store you need to pass the BypassRules property...


[DUOS-2082][risk=no] Update random string generation in tests

Addresses
https://broadworkbench.atlassian.net/browse/DUOS-2082
Usages of RandomStringUtils.random(10) can generate strings that our database cannot store. Use RandomStringUtils.randomAlphabetic(10) instead which restricts the string values to the [a-A,z-Z] range.

Have you read CONTRIBUTING.md lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

Just to be thorough I searched the consent repo to see if there are other places we were using RandomStringUtils.random and it looks like theres a lot of usages in many other tests. I see that the scope of this ticket is to fix the flaky user tests so it makes sense to not address them in this PR but should we make a ticket to address all the other places we are using that method?

I did look through them and all of the other cases look harmless to me. I think we can deal with future cases of these as they come up.


lost connection during installation

Operating system
ubuntu 22.04
Description
lost connection during installation even though connection exist
Docker compose
windows:
image: dockurr/windows
container_name: windows
environment:
VERSION: "11"
devices:
- /dev/kvm
cap_add:
- NET_ADMIN
ports:
- 8006:8006
- 3389:3389/tcp
- 3389:3389/udp
stop_grace_period: 2m
Docker log
❯ Starting Windows for Docker v4.02...
❯ For support visit https://github.com/dockur/windows
❯ CPU: 13th Gen Intel Core TM i7 13850HX | RAM: 22/32 GB | DISK: 673 GB (ext4) | HOST: 6.8.0-48...
❯ Requesting Windows 11 from the Microsoft servers...
❯ ERROR: Failed to resolve Microsoft servers! Is there an Internet connection?
❯ Will retry after 5 seconds...
❯ Requesting Windows 11 from the Microsoft servers...
❯ ERROR: Failed to resolve Microsoft servers! Is there an Internet connection?
❯ Failed to download Windows 11, will try a diferent method now...
❯ Downloading product information from Microsoft server...
❯ ERROR: Failed to download https://go.microsoft.com/fwlink?linkid=2156292 , network failure!
❯ Will retry after 5 seconds...
❯ Downloading product information from Microsoft server...
❯ ERROR: Failed to download https://go.microsoft.com/fwlink?linkid=2156292 , network failure!
❯ Failed to download Windows 11, will try another mirror now...
❯ Downloading Windows 11 from bobpony.com...
❯ ERROR: Failed to download https://dl.bobpony.com/windows/11/en-us_windows_11_23h2_x64.iso , network failure!
❯ Will retry after 5 seconds...
❯ Downloading Windows 11 from bobpony.com...
❯ ERROR: Failed to download https://dl.bobpony.com/windows/11/en-us_windows_11_23h2_x64.iso , network failure!
❯ Failed to download Windows 11, will try another mirror now...
❯ Downloading Windows 11 from massgrave.dev...
❯ Will retry after 5 seconds...
❯ ERROR: Failed to download https://drive.massgrave.dev/en-us_windows_11_consumer_editions_version_24h2_x64_dvd_1d5fcad3.iso , network failure!
❯ Downloading Windows 11 from massgrave.dev...
❯ ERROR: Failed to download https://drive.massgrave.dev/en-us_windows_11_consumer_editions_version_24h2_x64_dvd_1d5fcad3.iso , network failure!
Screenshots (optional)
No response

Using local iso image works.
volumes:
  - /path/to/your.iso:/custom.iso


[help] Silent failure

What doesn't work

:call CocAction('format')
:CocCommand prettier.formatFile
:Prettier

The commands report success, but the file doesn't change. Especially with :call CocAction('format') I get the response [coc.nvim] Formatted by prettier, while with other two commands it just finishes without reporting anything in the output. Ofc, format on save doesn't work.
All of those worked completely fine until a few days ago. I don't know what changed, I did run arch packages update and update of vim plugins and of coc extensions but i've also tried reverting them without fixing the problem.
I tried

reverting my dotfiles to a point where it surely worked (my .vimrc)
reverting the project in question to a point when it worked
trying it out on a different typescript project (the two use node versions v10.20.1 and v12.14.1)
reinstalling older version of coc-prettier (tried 1.1.18, 1.1.19, 1.1.20)
updating prettier dependency to the latest version

What works well

Interesting thing is that in that project I have this script:

"format": "prettier --write \"src/**/*.ts\""

Which works just fine.

Also if I run ./node_modules/prettier/bin-prettier.js src/main.ts from the project's root, the output is properly reformatted file.
Running it on a pure javascript file works well (in any project)

Context (files)
:CocConfig (only global, no local):
{
  "prettier.tslintIntegration": true,
  "coc.preferences.formatOnSaveFiletypes": [
    "css",
    "javascript",
    "typescript",
    "json",
    "jsonc"
  ],
  "coc.preferences.colorSupport": false,
  "diagnostic.hintSign": ">>",
  "diagnostic.infoSign": "ℹ",
  "diagnostic.warningSign": "⚠",
  "diagnostic.errorSign": "💣",
  "diagnostic.enableHighlightLineNumber": true,
  "diagnostic-languageserver.filetypes": {
    "vim": "vint",
    "email": "languagetool",
    "markdown": ["write-good", "markdownlint"],
    "sh": "shellcheck"
  },
  "typescript.preferences.importModuleSpecifier": "relative",
  "markdownlint.config": {
    "line-length": false,
    "MD022": false,
    "MD032": false,
    "MD033": false, // inline HTML shouldn't be forbidden (...[goto](#asdf))
    "MD034": false,
    "MD025": { "front_matter_title": "" }
  },
  "languageserver": {
    "ccls": {
      "command": "ccls",
      "filetypes": ["c", "cpp", "cuda", "objc", "objcpp"],
      "rootPatterns": [".ccls-root", "compile_commands.json"],
      "initializationOptions": {
        "cache": {
          "directory": ".ccls-cache"
        },
        "client": {
          "snippetSupport": true
        }
      }
    }
  },
  "explorer.icon.enableNerdfont": true
}

.config/coc/extensions/package.json:
{
  "dependencies": {
    "coc-db": ">=0.0.43",
    "coc-emoji": ">=1.2.3",
    "coc-explorer": ">=0.9.1",
    "coc-highlight": ">=1.2.6",
    "coc-jest": ">=1.1.1",
    "coc-json": ">=1.3.2",
    "coc-markdownlint": ">=1.7.0",
    "coc-prettier": ">=1.1.18",
    "coc-python": ">=1.2.13",
    "coc-sql": ">=0.4.0",
    "coc-syntax": ">=1.2.4",
    "coc-tsserver": ">=1.6.0",
    "coc-word": ">=1.2.2"
  }
}

.prettierrc:
{
  "useTabs": false,
  "singleQuote": true,
  "trailingComma": "all",
  "tabWidth": 2,
  "printWidth": 120
}

tslint.json:
{
  "defaultSeverity": "error",
  "extends": ["tslint:recommended", "tslint-config-prettier"],
  "jsRules": {
    "no-unused-expression": true
  },
  "rules": {
    "arrow-parens": false,
    "array-type": false,
    "curly": [true, "ignore-same-line"],
    "eofline": true,
    "indent": [true, "spaces", 2],
    "interface-name": false,
    "max-classes-per-file": false,
    "max-line-length": [true, 120],
    "member-access": false,
    "member-ordering": [
      true,
      {
        "order": [
          "public-static-field",
          "public-static-method",
          "protected-static-field",
          "protected-static-method",
          "private-static-field",
          "private-static-method",
          "public-instance-field",
          "protected-instance-field",
          "private-instance-field",
          "public-constructor",
          "protected-constructor",
          "private-constructor",
          "public-instance-method",
          "protected-instance-method",
          "private-instance-method"
        ]
      }
    ],
    "no-empty": false,
    "no-empty-interface": false,
    "no-unused-expression": true,
    "object-literal-sort-keys": false,
    "one-line": true,
    "one-variable-per-declaration": true,
    "ordered-imports": true,
    "quotemark": [true, "single", "avoid-escape"],
    "variable-name": [
      true,
      "ban-keywords",
      "check-format",
      "allow-snake-case",
      "allow-pascal-case",
      "allow-leading-underscore"
    ]
  },
  "rulesDirectory": []
}

I'm running into the same issue, except it only happens on a specific file. Other files get properly formatted.

I'm running into the same issue, except it only happens on a specific file. Other files get properly formatted.

Same here, with an html file

Just out of curiosity, as I had a similar problem. Does add prettier-tslint to your project solve the issue?

It could be using bundled prettier from coc-prettier before.

It could be using bundled prettier from coc-prettier before.

@maricn This is what puzzles me also, as the bundled prettier-tslint is in there (in coc-prettier's node modules/.bin)

Local prettier is used when possible by default, so when you don't have global prettier, local prettier is used.

@chemzqm I think that this is not the same functionality as vs code. In vs code I didn't have to install prettier-tslint in order to make it work with typescript files.
Maybe this is a bug?

I see, seems API used by prettier get changed.

Works as expected for me, don't know how to reproduce.

@chemzqm Without going into the internals of this package, maybe this line from my original post is helpful:

Especially with :call CocAction('format') I get the response [coc.nvim] Formatted by prettier, while with other two commands it just finishes without reporting anything in the output. Ofc, format on save doesn't work.

So, maybe the way the plugin detects failure of the external command is flawed? It reports success, but actually nothing happened. Is the fallback to plugin's prettier working in case there's an error with the local one (if it's how it works anyways)?

Please checkout output from coc-prettier https://github.com/neoclide/coc.nvim/wiki/Debug-language-server#using-output-channel

Please checkout output from coc-prettier neoclide/coc.nvim/wiki/Debug-language-server#using-output-channel

[Error - 6/18/2021, 6:07:32 PM:] Failed to load prettier-tslint from /home/nikola/.dotfiles/.config/coc/extensions/node_modules/coc-prettier

[Error - 6/18/2021, 6:07:32 PM:] Cannot read property 'format' of null

[Trace - 6/18/2021, 6:07:32 PM:] Formatted file: file:///home/nikola/Workspace/ProjectXXX/src/PackageYYY/FileZZZ.ts

[Trace - 6/18/2021, 6:07:32 PM:] Prettier format edits: [
  {
    "range": {
      "start": {
        "character": 0,
        "line": 0
      },
      "end": {
        "character": 0,
        "line": 54
      }
    },
    "newText": "REDACTED"
  }
]

there's some symlinks on that path, but this is what's there:
$> ls -lah /home/nikola/.dotfiles/.config/coc/extensions/node_modules/coc-prettier
.rw-r--r-- nikola nikola 116 B  Tue Mar 30 10:30:48 2021  .prettierrc
drwxr-xr-x nikola nikola 4.0 KB Tue Mar 30 10:30:48 2021  .release/
.rw-r--r-- nikola nikola 460 B  Tue Mar 30 10:30:48 2021  CHANGELOG.md
.rw-r--r-- nikola nikola 481 B  Tue Mar 30 10:30:48 2021  esbuild.js
drwxr-xr-x nikola nikola 4.0 KB Tue Mar 30 10:30:48 2021  lib/
drwxr-xr-x nikola nikola  20 KB Tue Mar 30 10:30:48 2021  node_modules/
.rw-r--r-- nikola nikola 233 B  Tue Mar 30 10:30:34 2021  package-json-schema.json
.rw-r--r-- nikola nikola 204 KB Tue Mar 30 10:30:34 2021  package-lock.json
.rw-r--r-- nikola nikola 7.7 KB Tue Mar 30 10:30:34 2021  package.json
.rw-r--r-- nikola nikola 5.9 KB Tue Mar 30 10:30:48 2021  Readme.md

Not sure what this line means:
[Error - 6/18/2021, 6:07:32 PM:] Cannot read property 'format' of null

There should be prettier-tslint inside node_modules folder of coc-prettier

There should be prettier-tslint inside node_modules folder of coc-prettier

I have prettier-tslint version 0.4.2 (latest, from Jan 2019) inside coc-prettier version 1.1.23.

I'm using<EMAIL_ADDRESS>and<EMAIL_ADDRESS>I was having a similar problem where files were not being formatted and just a new line was being added to the end of the file. After installing prettier-tslint` to my project it started working again.

Having same problem.
where and how can i add prettier-tslint?

Make sure check the Prettier output by :CocCommand workspace.showOutput


Fix #318, crashes in Orma migration engines

Fixed at Orma v2.0.6, which includes https://github.com/gfx/Android-Orma/pull/193
Please :sob:

Awesome!  Can I merge this? Is latest version Orma already updated?

You can merge it!

OK! LGTM!

Of course, you should wait for CI success :wink:


Remove non-vanilla * from helper and denial systems

Introduction

Recently it was pointed out to me that @a would be better to be used in place of the asterisk *  because Vanilla bedrock edition uses the @a selector. I agree.
The plugin PlayerSelectors on poggit can be used to add the @a selector along with many other vanilla selectors. This PR removes the * implementation in a push for users to use the PlayerSelectors plugin instead.
Relevant issues

N/A

Changes
API changes

The asterisk * can no longer be used with Plot::isHelper() or Plot::isDenied()

Behavioral changes

The addhelper subcommand can no longer use the asterisk * to open a plot to everyone
The denyplayer subcommand can no longer use the asterisk * to block everyone from a plot

Backwards compatibility

Upon updating, plots that had everyone blocked and plots that had everyone helping will no longer function as expected.

Follow-up

All Following releases should advertise PlayerSelectors as a replacement for the feature.

Tests
The asterisk* no longer functions to allow or block all players.

The addhelper subcommand can no longer use the asterisk * to open a plot to everyone
The denyplayer subcommand can no longer use the asterisk * to block everyone from a plot
what's a valid new replacement or an alternative for a wildcard allow/block?
assuming @a only applies to currently online players


UITextField Delegate

Great work on this component.
Is there anyway to expose the TextField delegate for input verification?

@robmontesinos I will close this issue, unless you would like to extend TextField functionality. Thanks

Thanks @vikmeup - no problem. Unfortunately I am swamped with work. All the best!


[svgPathPen] Improve writing out numbers

See:
https://github.com/fonttools/fonttools/pull/3543#issuecomment-2140398555

Currently:
fonttools pens.svgPathPen AdobeVFPrototype.otf A

generates output using integers only:
<g transform="translate(0 918) scale(1 -1)"><path d="M5 0H235V40L125 55H105L5 40V0ZM71 0H118L322 599H299L500 0H593L363 675H303L71 0ZM170 219H472L456 265H186L170 219ZM383 0H653V40L529 55H509L383 40V0Z"/></g>

As soon as I set any --variations, even to the default, it switches to float because the variation calculations happen in float:
fonttools pens.svgPathPen AdobeVFPrototype.otf A --variations wght=100

<g transform="translate(0 918) scale(1 -1)"><path d="M12.0 0H210.0V20.0L113.0 26.0H99.0L12.0 20.0V0.0ZM85.0 0.0H114.0L325.0 667.0L308.0 657.0L524.0 0.0H556.0L327.0 684.0H307.0L85.0 0.0ZM173.0 247.0H468.0L460.0 267.0H182.0L173.0 247.0ZM403.0 0.0H651.0V20.0L537.0 26.0H523.0L403.0 20.0V0.0Z"/></g>

And if I set non-default location, I get multiple unnecessary digits of float:
fonttools pens.svgPathPen AdobeVFPrototype.otf A --variations wght=400

<g transform="translate(0 918) scale(1 -1)"><path d="M5.205617204267097 0H234.3831483871987V40.781345376214965L125.24674064512051 55.781345376214965H105.37011096768077L5.205617204267099 40.781345376214965V0.0ZM70.75325935487949 0.0H118.45235784938762L321.7121359140261<PHONE_NUMBER>463453L297.8896670969577<PHONE_NUMBER>86606L497.1624825811141 0.0H594.6038141932834L364.8916782792573<PHONE_NUMBER>603713H302.42427182805216L70.7532593548795 0.0ZM169.91775311829318<PHONE_NUMBER>0671666H472.3289875268274L455.95887655914663<PHONE_NUMBER>012248H185.95887655914663L169.9177531182932<PHONE_NUMBER>067167ZM381.3139389250099 2.842170943040401e-14H653.4934812902411V40.65797505365474L529.2467406451206 55.65797505365474H506.5737169896483L381.31393892500984 40.65797505365474V2.842170943040401e-14Z"/></g>

That's clearly not ideal. Before I committed (accidentally actually) the following change: https://github.com/fonttools/fonttools/commit/42d6b6b4fedf51dd741d3134da74df04339335b4 but it was reverted.
Cosimo wanted me to update the callsites, not change default behavior. I realized now that I didn't actually change the default behavior of the pen, just the cmdline tool. And I still think that was the correct change.
But I also propose adding a new named argument to the pen to Do The Right Thing, without each client having to write this lambda.


Continue redesign splash page

In #374, @skipjack cleaned up the splash page to allow for a redesign. He added a temporary screenshot of @pastelsky's mock-up of the splash page. This screenshot is low quality, so we should preferably fix this soon.
This is a ticket to ensure that we don't forget that.
@pastelsky, do you plan on working on the splash page?

@pastelsky happy to help out with this, if you're very busy this week we could always just add a higher res modules image for the time being. Didn't mean to put pressure on you to jump on this right away.
Just for reference here's @pastelsky's design from #285:

I think we should also quickly talk out what end goal with this is... were you thinking this would be the only section displayed on this page (kind of like Gulp's landing page)? If so we should figure out what the plan for the small example and support sections is. Maybe we keep support there and move the short example to our new getting started section?

Here are the vector assets for homepage. Feel free to use them as you like —
https://www.dropbox.com/sh/fktsoaqcgt6r0mr/AADMSXL0dtzvpn1VWAi8HyGHa?dl=0

Closing as @pastelsky tackled this in #510.


sound issue

Describe your bug here.
i cant here anything when playing pls fix
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

by default you can press the + key to increase the game volume. if that doesn't work here's some other things to try:

Increase the volume of your computer sound.
A bit more complex:
i. Open the start menu and type sndvol. Press enter on the first result.
ii. Scroll until you find Psych Engine in the list. Drag the slider up to the little line. This will make the application volume match your system volume
Have you tried turning it off and on again? Restart your computer.

the sndvol thing work thx so much!