Callback not called when message sending failed by High Level Producer

Questions?

Check the Troubleshooting / FAQ ?
Search the issues ?

Bug Report
Due to some problem with the cluster, which we are trying to locate, the zookeeperd are throwing the following error
ERROR Failed to send last message. Shutting down thread. (org.apache.zookeeper.server.quorum.QuorumCnxManager)
java.net.SocketException: Socket closed
	at java.net.SocketOutputStream.socketWrite(SocketOutputStream.java:118)
	at java.net.SocketOutputStream.write(SocketOutputStream.java:134)
	at java.io.DataOutputStream.writeInt(DataOutputStream.java:197)
	at org.apache.zookeeper.server.quorum.QuorumCnxManager$SendWorker.send(QuorumCnxManager.java:889)
	at org.apache.zookeeper.server.quorum.QuorumCnxManager$SendWorker.run(QuorumCnxManager.java:916)

Environment

Node version: 7.10.1
Kafka-node version:2.3.2
Kafka version:1.0.0

For specific cases also provide

Number of Brokers:3
Number partitions for topic:3

Include Sample Code to reproduce behavior
// include code here
function updateFeedMessage(message){
    producer.send([{topic:'updateFeed',messages:JSON.stringify(message),partition:0}],function(err,data){
        if(err)
            console.log('error in sending update Feed message',err);
        else
            console.log('update feed message sent',data);
    });
}

Include output with Debug turned on
Thanks for your contribution!

Have you tried using KafkaClient instead of Client? KafkaClient does not use zookeeper.

I have tried using that but in that case the client does not detect even when the broker is up after a crash

@haseebnaseem the expectation would be that you would need to handle retrying the send again yourself.

I understand that if the sending failed, there should be a retrying
mechanism however that would only happen if the callback is called with an
error.
On Mon, 9 Apr 2018, 23:02 Daniel Ennis<EMAIL_ADDRESS>wrote:

@haseebnaseem https://github.com/haseebnaseem the expectation would be
that you would need to handle retrying the send again yourself.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SOHU-Co/kafka-node/issues/908#issuecomment-379840819,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ1EQBptj9a-HbzZ-0jSfX7BMMHaDrpFks5tm6I6gaJpZM4S65jD
.

@haseebnaseem that was in reference for if you use KafkaClient, you said you aren't using it since it doesn't detect when its up.
Im saying use KC, and keep retrying.

My kafka set up does not work without zookeeper, so once its down
kafkaClient does not detect its up ever.
On Tue, 10 Apr 2018, 00:09 Daniel Ennis<EMAIL_ADDRESS>wrote:

@haseebnaseem https://github.com/haseebnaseem that was in reference for
if you use KafkaClient, you said you aren't using it since it doesn't
detect when its up.
Im saying use KC, and keep retrying.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SOHU-Co/kafka-node/issues/908#issuecomment-379860941,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ1EQMcPm7BrJrEWgZWcZiPgyIvh_uXfks5tm7HRgaJpZM4S65jD
.


identify .NET single file bundles

.NET single file deployments allow all the dependencies to be included in a single binary.  Other capa rules might not be relevant for this single file bundles. Perhaps a warning could be displayed, same as for MSI. Additional information can be found in https://learn.microsoft.com/en-us/dotnet/core/deploying/single-file/overview?tabs=cli

Cool, can you address the parsing/linter fail and then this should be good to go.

There's also packaged as single-file .NET application which we should consider as part of this update.

will create a new PR for this


Move changelog entry for #1162 to the right version

PR #1162 was merged in d24b26ae2697685e930d863839d7f8834b9ca4a3 which was released in v2.46.0, but its CHANGELOG.me entry had not been moved above the ## v2.45.0 entry in https://github.com/github/view_component/commit/a5f6463f36639e7385a9c905801e18e41b6b6538. Thus, it was incorrectly listed as being included in v2.45.0.
Let's make sure it's listed under the correct version in CHANGELOG.md. And based on #1180, let's make it clear in the release notes what folks need to do to
resolve the deprecation warning after they upgrade.

Thanks @joelhawksley & @Spone. I'll sync this change to the v2.46.0 release notes


Added files for Minecraft v1.12

Tested with current LiteLoader 1.12 and the final Minecraft Client v1.12

Thanks @liach . I also added some missing fixes/files.

Thanks for the PR.
I'm in another country and only have access to my e-mail, so I'll
merge this whenever I can get back on my computer.
--
~Julian Uy

@octan3, I think you're missing a commit in this pull.  I get a crash error after building this 1.12 version.
Error: java.lang.NoClassDefFoundError: bvm

@Goobaroo I downloaded my repo, edited the settings.gradle to:
include 'mcxray1.12', 'mcxrayshare'

and run gradle build. It created a working .litemod file in the folder /mcxray1.12/build/libs/mod-mcxray1.12-1.0.0-mc1.12.litemod.

@octan3 Sorry, it wasn't on build, it was on run that there was an issue.
Tried grabbing the latest liteloader snapshot too from today.  But your build loaded fine for you?
latest.txt

I'm back from abroad.
I will review the code tomorrow, since I'm jet-lagged.

Welcome home.

Thank you @octan3


fix overlapping text

Details

fixed overlapping text and made the view scrollable
Fixed Issues

$ https://github.com/Expensify/App/issues/6826
Tests

QA Steps

Go to app
Create a new workspace if you don't have one
Click "+" > New room

Tested On

[ ] Web
[x] Mobile Web
[ ] Desktop
[ ] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @tgolen in version: 1.1.23-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by @francoisl in version: 1.1.24-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Completed training

@wrandall22 This saves the training and adds them to the map. I got side tracked a couple of times.

setting current ministry is now done with an async task

Looks good to me.

Thanks


Adding Atoti CE Dockerfile

This PR adds a Dockerfile that developers can use to quickly install and access the notebooks in the Atoti repo.

Duplicate PR, see https://github.com/atoti/atoti/pull/874 for successor.


Drop UAT tables and reload data

Required due to issues discussed in https://github.com/podaac/hydrocron/issues/161

Need an AWS CLI command to kick off the lambda so that it can be scripted to run 4 days at a time

4/23/24 - Deleted the 2 tables in Services UAT and ran the load-data Lambda for the Reach table per UAT wiki page:
https://wiki.jpl.nasa.gov/display/PD/Hydrocron+v1.2.0+User+Scenarios+-+Internal+UAT+in+Services+UAT+-+04-23-2024
Only needed to load Jan & Feb 2024 data into Services UAT per Victoria.
The load-data for the Node table will be started on 4/24/24.

4/25/24 - Completed successful dry-run of script to load-data for Node table for Jan & Feb 2024 into Services UAT.
Also ran the load-benchmarking command in Services UAT


pythonPackages.afdko: 3.5.0 → 3.5.1

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python3Packages.nototools

This package is broken due to a flaky (date dependent) test.  See https://github.com/adobe-type-tools/afdko/pull/1232.  Maybe we can either revert this or add a patch from the aforementioned PR?

I would add the patch or disable the offending test. I can look into it but I’m afraid I will not get to it earlier than next week.

On 6. Oct 2020, at 16:48, roconnor<EMAIL_ADDRESS>wrote:

This package is broken due to a flaky (date dependent) test. See adobe-type-tools/afdko#1232. Maybe we can either revert this or add a patch from the aforementioned PR?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


[master] Fix wrong region for China IAM service

Issue: https://github.com/rancher/rancher/issues/29666

Hey @guangbochen, could you (or someone with China credentials) please review/test this fix to confirm it works? I got past the SignatureDoesNotMatch error, but I get a InvalidClientTokenId error now that I suspect is due to my US credentials. Thanks!


Move aware containers

I currently have my own implementations of gc free containers, but it would be nice to switch to a more "widely" used library.
But I would need the containers to be move aware, so that you can have DynamicArray!(Unique!Foo) for example.
Thoughts?

#182 implements this for DynamicArray.


Karma webpack color option

I'm submitting a feature request
It would improve consistency of configs if webpack's color config inherits Karma's color config

What is the motivation / use case for changing the behavior?

Would provide more consistency, similar to the same way that webpack currently uses Karma's singleRun config to determine the watch config:
https://github.com/webpack-contrib/karma-webpack/blob/master/src/karma-webpack.js#L42
Proposed behaviour
As a suggestion, the value could be determined as follows: Taking the next item in the list if the previous one is not defined:

webpack's color config value
karma's color config value
webpack's default color value (same as it is now)

Next step
I could contribute this and open a PR in the near future if the majority of the maintainers agree?
Thanks in advance!
Docs for reference
Webpack stats color docs: https://webpack.js.org/configuration/stats/?sm_au=iVVJFnC5DW5NQLFF#stats
Karma docs for color http://karma-runner.github.io/2.0/config/configuration-file.html

Feel free to send a PR :)


Is the new tracker functionnality implemented ?

Hi,
As mentioned here: https://umami.is/docs/migrate-v1-v2
The tracker no longer uses CSS class names to trigger events. It now uses HTML data attributes. See [Track events](https://umami.is/docs/track-events) for more information.
After testing i noticed that no event is being sent using nuxt-umami ({ version: 2} has been set). So is it on my end or it's not actually implemented in the library ?
Example of usage:
        <button             class="btn btn-sm rounded-full btn-primary mt-4 btn-block"             data-umami-event="sidebar-button-clicked"             :data-umami-event-position="head"             :data-umami-event-tags="Financials, Lifestyle"             @click="awesome"         >             My awesome button         </button>
EDIT: Looking at the library design maybe it's not planned at all ?

Hello @Velka-DEV
This library aims to be very explicit, so no it doesn't track umami attributes.
It shouldn't be too hard to set up, I think. I hope.
onMounted(), get all elements with matching attributes, set up event listeners, and call umTrackEvent().
I could whip up a demo if you need a little help with.

That would be a nice feature, I think. There are interesting ways we could do it, the attributes way, or even a directive.
Sure! Open a PR and we'd work on it.

Closed, #61


UI5 Inspector not detecting UI5

Hi, I have added the UI5 Inspector to Chrome and when I run on browser from SAP Cloud Web IDE it is returning:
There is no OpenUI5/SAPUI5 found on this page
My friend sitting next to me with the same setup is able to use the add-on without issue.
Any ideas?

Hi @kevindeuel,
Can you provide more information about the issue?
Please use https://github.com/SAP/ui5-inspector/blob/master/docs/bugreport_template.txt

Hi @kevindeuel,
this ticket is on 'author action' for more than 2 weeks without reaction.
Therefore this ticket is closed. If the ticket is still relevant please provide
the requested information.
Best regards,
Chariyski


componentWillReceiveProps doesn't always get called?

I'm trying to check some stuff upon every navigation into a certain View. The recommendation was to use componentWillReceiveProps, which I tried - but it seems not to fire upon the route change.
Is there any way to constantly be able to run code upon entry into the View? This seems like it should be trivial, but became insanely hard.

This comes up a lot so I just wrote a guide on it, hope it helps :)
https://github.com/rackt/react-router/blob/master/docs/ComponentLifecycleWhenRouting.md


Reduce allocations

Description
Reduces the number of memory allocations made by modifying the Stanza type to use strings.Builder as opposed to string as the map value. Also, adds the ability for clients to pass their own Stanza buffer object to Fetch and ReadStanza, which further reduces allocations by re-using common map entries.
Performance difference
On Ubuntu 20.04.3 with kernel 5.5.0-050500-generic,GetKernelHeaders allocations in the system-probe decreased from  2.04 GB to 344.49 MB (profiles below). This kernel version/OS version combination is a sort of "worst case" scenario, where aptly fails to find kernel headers and downloads a huge large number of package indices while searching for the headers.
On Ubuntu 20.04.3 with kernel 5.4.0-91-generic, GetKernelHeaders allocations decreased  from 1312.36 MB to 97.70 MB. This kernel version/OS version combination represents a sort of "usual" scenario, where aptly succeeds in finding and downloading kernel headers. Part of the decrease in this scenario is due to other optimizations in nikos, so the profile is not included below.
Before (Ubuntu 20.04.3 with kernel 5.5.0-050500-generic):

After (Ubuntu 20.04.3 with kernel 5.5.0-050500-generic):

@ISauve Is there a way we can have a separate PR for the aptly -> Datadog rename? Otherwise it is hard to find the actual code changes.

Closing in favour of #3


Implement server Stop method

Currently "Stop" function is not implemented in GrpcServer. It leads to the fact, that GrpcServer can't be used inside other processes (like in #23022). I've implemented stop method so that server stops and doesn't abort the process during destructor execution.
@mrry, could you please have a look as the author of this functionality?

I found one more issue, eager service breaks loop immediately after "stop" message. Looks like it's wrong, so I changed behavior to be similar to other services (master and worker).

Nagging Assignee @ymodak: It has been 27 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @ymodak: It has been 42 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Reviewer : You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 14 days with no activity and the awaiting review label has been applied.

Nagging Reviewer : You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 29 days with no activity and the awaiting review label has been applied.

Nagging Reviewer : You have been added as a reviewer to this pull request. Please add your review or reassign. It has been 18 days with no activity and the awaiting review label has been applied.

@dmitrievanthony wondering if this PR still valid , if yes could you please re base your branch.

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

I'm going to go ahead and close this PR, because it seems to have stalled. If you're still interested in pursing this (and responding to my comments), please feel free to reopen!


Cypress testing

trying out testing end to end

hey @kirahsapong @frankhinek thought I would try introducing some e2e testing framework - I know cypus is fairly lightweight - and this really just checks that the website loads.
If this is a useful starting point feel free to merge and can add some concrete (but modest) end to end tests.
Secondly - I changed the run command to use a different docker-compose setup that uses continuously published images of ssi-service (so we don't have to build it each time). Also means we don't need to use a git module dependency.
thoughts?

closing this as we are using playwright


Hc get started

Merging

@Cojacfar,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@SyntaxC4 @cephalin @JennieHubbard - Please review these proposed changes. Thanks.

@Cojacfar are you an internal MS contributor by any chance? if you are, you must use the azure-docs-pr repo.
in any case, thank you much for helping us out, but at the very least, please open separate PRs for different articles. there are multiple issues with your changes, in addition to the fact that there's a merge conflict.

@cephalin I'm a v- here, but work in customer support and don't have access to the private repo. I've been trying to update the documentation when users run into issues following it due to out-of-date references (like BizTalk no longer being used for Hybrid connections).
Looks like I messed up my branches here and included what I meant to be two different branches. I've been trying to keep a branch per topic deal. If you have some documentation on preferred handling of contributing to this repo, I'd be interested.


[BUG] "The VRCSDK build was aborted because the VRCSDKPreprocessAvatarCallback 'AvatarBuildHook' reported a failure."

I came across this error when building for quest and for the life of me I can't figure out what it is.

System.IndexOutOfRangeException: Index was outside the bounds of the array.
  at d4rkAvatarOptimizer.FindAllPhysBoneDependencies () [0x0033f] in .\Packages\d4rkpl4y3r.d4rkavataroptimizer\Editor\d4rkAvatarOptimizer.cs:2251
  at d4rkAvatarOptimizer.FindAllPhysBonesToDisable () [0x00010] in .\Packages\d4rkpl4y3r.d4rkavataroptimizer\Editor\d4rkAvatarOptimizer.cs:2308
  at d4rkAvatarOptimizer.Optimize () [0x0014c] in .\Packages\d4rkpl4y3r.d4rkavataroptimizer\Editor\d4rkAvatarOptimizer.cs:155
  at d4rkpl4y3r.AvatarOptimizer.AvatarBuildHook.OnPreprocessAvatar (UnityEngine.GameObject avatarGameObject) [0x00045] in .\Packages\d4rkpl4y3r.d4rkavataroptimizer\Editor\AvatarBuildHook.cs:37
UnityEngine.Logger:Log (UnityEngine.LogType,object)
(wrapper dynamic-method) UnityEngine.Debug:UnityEngine.Debug.LogError_Patch1 (object)
d4rkpl4y3r.AvatarOptimizer.AvatarBuildHook:OnPreprocessAvatar (UnityEngine.GameObject) (at ./Packages/d4rkpl4y3r.d4rkavataroptimizer/Editor/AvatarBuildHook.cs:42)
VRC.SDKBase.Editor.BuildPipeline.VRCBuildPipelineCallbacks:OnPreprocessAvatar (UnityEngine.GameObject)
nadena.dev.ndmf.ApplyOnPlay:MaybeProcessAvatar (nadena.dev.ndmf.runtime.ApplyOnPlayGlobalActivator/OnDemandSource,UnityEngine.MonoBehaviour) (at ./Packages/nadena.dev.ndmf/Editor/ApplyOnPlay.cs:98)
nadena.dev.ndmf.runtime.AvatarActivator:Awake () (at ./Packages/nadena.dev.ndmf/Runtime/ApplyOnPlayGlobalActivator.cs:187)
UnityEngine.GameObject:AddComponent<nadena.dev.ndmf.runtime.AvatarActivator> ()
nadena.dev.ndmf.runtime.RuntimeUtil:GetOrAddComponent<nadena.dev.ndmf.runtime.AvatarActivator> (UnityEngine.GameObject) (at ./Packages/nadena.dev.ndmf/Runtime/RuntimeUtil.cs:33)
nadena.dev.ndmf.runtime.ApplyOnPlayGlobalActivator:Awake () (at ./Packages/nadena.dev.ndmf/Runtime/ApplyOnPlayGlobalActivator.cs:125)

The VRCSDK build was aborted because the VRCSDKPreprocessAvatarCallback 'AvatarBuildHook' reported a failure.
UnityEngine.Logger:Log (UnityEngine.LogType,object)
(wrapper dynamic-method) UnityEngine.Debug:UnityEngine.Debug.LogError_Patch1 (object)
VRC.SDKBase.Editor.BuildPipeline.VRCBuildPipelineCallbacks:OnPreprocessAvatar (UnityEngine.GameObject)
nadena.dev.ndmf.ApplyOnPlay:MaybeProcessAvatar (nadena.dev.ndmf.runtime.ApplyOnPlayGlobalActivator/OnDemandSource,UnityEngine.MonoBehaviour) (at ./Packages/nadena.dev.ndmf/Editor/ApplyOnPlay.cs:98)
nadena.dev.ndmf.runtime.AvatarActivator:Awake () (at ./Packages/nadena.dev.ndmf/Runtime/ApplyOnPlayGlobalActivator.cs:187)
UnityEngine.GameObject:AddComponent<nadena.dev.ndmf.runtime.AvatarActivator> ()
nadena.dev.ndmf.runtime.RuntimeUtil:GetOrAddComponent<nadena.dev.ndmf.runtime.AvatarActivator> (UnityEngine.GameObject) (at ./Packages/nadena.dev.ndmf/Runtime/RuntimeUtil.cs:33)
nadena.dev.ndmf.runtime.ApplyOnPlayGlobalActivator:Awake () (at ./Packages/nadena.dev.ndmf/Runtime/ApplyOnPlayGlobalActivator.cs:125)

Also getting this after saving the avatar prefab after trying to turn off delete unused objects (with no different of a result.)

Also confirmed my avatar works fine when I remove the d4rk optimizer component completely.
Also fails when the component is added with none of the options except apply on upload is on;

Uh oh. That error means you have a skinned mesh that does something really weird. It has at least one vertex that references a bone id that is larger than the list of bones in the skinned mesh renderer. That should not be a thing a mesh has.
Shouldn't be hard for me to log some warnings instead of producing an error though.

If you could log which skinned mesh has the issue, that'd be super helpful too.

This should now be a warning instead of a crash in 3.9.1


fix test pattern in FormDateTimeSelectTest::testCanRenderTextDelimiters

This fixes a failing test in ZendTest/Form do to trailing format.

Merged to develop for release with 2.4.


Loads for BepinEX <IP_ADDRESS> but Haldor still shows vanilla

I am using the macOS Steam Valheim client. All of my plugins work using the following method:
https://www.reddit.com/r/valheim/comments/1dcko3i/guide_running_mods_on_macos/
The exception is TradersExtended and Configuration Manager...the later is not really an issue.
[Message:   BepInEx] BepInEx <IP_ADDRESS> - Valheim (6/11/2024 9:33:03 AM) [Info   :   BepInEx] Running under Unity vUnknown (post-2017) [Info   :   BepInEx] CLR runtime version: 4.0.30319.42000 [Info   :   BepInEx] Supports SRE: True [Info   :   BepInEx] System platform: Bits64, iOS [Message:   BepInEx] Preloader started [Info   :   BepInEx] Loaded 1 patcher method from [BepInEx.Preloader <IP_ADDRESS>] [Info   :   BepInEx] 1 patcher plugin loaded [Info   :   BepInEx] Patching [UnityEngine.CoreModule] with [BepInEx.Chainloader] [Message:   BepInEx] Preloader finished [Info   :   BepInEx] Detected Unity version: v2022.3.17f1 [Message:   BepInEx] Chainloader ready [Message:   BepInEx] Chainloader started [Info   :   BepInEx] 9 plugins to load [Info   :   BepInEx] Loading [PlantEasily 1.8.1] [Info   :   BepInEx] Loading [PlantEverything 1.18.1] [Info   :   BepInEx] Loading [WieldEquipmentWhileSwimming 1.0.9] [Info   :   BepInEx] Loading [Configuration Manager 18.3] [Info   :   BepInEx] Loading [Valheim Plus <IP_ADDRESS>] ... [Info   :Valheim Plus] ValheimPlus done loading. [Info   :   BepInEx] Loading [Display BepInEx Info In-Game 2.0.2] [Info   :   BepInEx] Loading [Build Camera 1.6.1] [Info   :   BepInEx] Loading [Traders Extended 1.2.5] [Info   :   BepInEx] Loading [Trash Items Mod 1.2.8] [Info   :Trash Items Mod] TrashItems Loaded!  I can't test it on Mac so without at least error text I can't help at all.

And Configuration Manager version 18.3 is not mine but official one.

Also suggested BepInEx v<IP_ADDRESS> is not preconfigured Valheim version but official one and I have no idea what compatibility issues it could cause.


Members only posts are not showing on homepage.

When a post published only for members or paid-members that post is not visible to any user on the whole website.

This theme doesn't support Members, that's why that happens. I'm closing this issue since it was already discussed here: https://github.com/eddiesigner/liebling/issues/70


Support for Viral Tenant

When inviting external users through B2B, and inviting users who have a corporate email address, but do not currently use Azure AD or Office 365 - when they accept their invitation, a Viral Tenant is created for that domain.  This initialization process seems to work for the most part without issue.
However, problems arise for the external user eventually.  We see problems when someone within the external organization verifies that domain, or in some way takes ownership of the domain.  This often happens during some testing, or as part of something completely unrelated and with no consideration for the users in that viral tenant.  When the tenant is verified or someone takes ownership of the tenant, the free SSPR available to the viral tenant no longer works and if users need to reset their password to access a site that they were invited to as an external user, they can no longer do so.
At that point, these external users typically reach out to the owners of the site they were invited to because they can no longer sign in.  Since this is an external user, and an viral tenant, there is nothing that the site owner, or the IT Organization supporting the site owner, can do to help this external user.  And, we have no where to direct them for assistance.
This is causing an alarming number of issues for our site owners and it is beginning to make the B2B invitation process nearly unusable for our site owners.  We do the very best that we can, as the IT Organization that supports the SharePoint site, we can provide them with instructions on how to take ownership of the viral tenant, we can walk them through seeing if their tenant has been verified, and we ultimately have to work one on one with the external users IT Organization - but we are extremely limited in what we can do to help - and most of the time, these organizations don't have a very robust IT support structure internally or at all, and if they do the IT staff are not prepared to be able to handle this support.
So, what support is available for these external users who were initially viral users, but are now unable to manage their accounts?  How can we assist these users?

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

ID: 3f76cc85-8dc0-9be3-4071-217589ddef56
Version Independent ID: e7bdba65-bacc-7fef-2c27-b3425f697b12
Content: Troubleshooting Azure Active Directory B2B collaboration
Content Source: articles/active-directory/b2b/troubleshoot.md
Service: active-directory
GitHub Login: @msmimart
Microsoft Alias: mimart

@SwilliamsIOT If I understand your problem correctly it seems related to an ongoing MSDN thread https://social.msdn.microsoft.com/Forums/azure/en-US/c54844ce-1415-4895-af15-6a5b768131d2/azure-b2b-guest-users-not-getting-password-reset-email-and-way-to-figure-viral-vs-managed-azure-ad?forum=WindowsAzureAD.  Please refer to the same to get latest updates.

Thank you very much for looking into this, however the problem that Dhaval is describing is actually a different problem.  I know this specifically because Dhaval is one of my customers and I have already discussed the problem in the linked thread with him.  He could speak to the issue I am describing as well because he has had many users with the experience I am describing recently.
In the scenario I am describing, the user initially belongs to a viral tenant, and can get in without issues, and can reset their password when needed without issues.
The problems surface when someone else in their organization verifies that domain in some way - which can happen in a number of different scenarios and I am happy to outline those for you if that helps.  But, the end result after the domain is verified or the viral tenant is taken over, is that they can no longer reset their password and receive a message indicating that SSPR is not enabled for their organization, and directs them to their organizations administrator.  However, the end user usually has no idea who to contact to get their password reset, and their IT support organization (if they have one) has no idea that they have taken control of a tenant or verified a domain, they do not understand what impact that has on these viral users, and they don't know how to reset their passwords.

@SwilliamsIOT We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

My client has this same issue with their B2B user who was the first user in their organization, but forgot her password.
The user gets the following error:

Get back into your account
We're sorry
You cannot reset your password at this time because your administrator has not configured password reset for your organization. There is no further action you can take to resolve this situation. Please contact your admin and ask them to configure password reset. To learn more about password reset configuration read the article Quick Start: Azure AD self-service password reset.
If you'd like, we can contact an administrator in your organization to reset your password for you.
Additional details: SSPR_0011: Your organization has not defined a password reset policy. Please contact your admin and ask them to define a password reset policy.

On my customers Azure AAD tenant with all of the guest users, he cannot reset the External Users B2B users password, as he receives this error:

This is not the home directory for<EMAIL_ADDRESS>Either<EMAIL_ADDRESS>or an administrator in their home directory can reset the password for this user.

We are not sure what the B2B customers domain did to restrict access from our tenant (if they have 2 B2B users at their domain, if the B2B company set up O365 now, or what) but we cannot reset the users password, and this user cannot get into their Azure AAD to modify their viral tenant organizations password policy.
We cannot re-invite the user since the account has already been created. My client has Self service password reset enabled on their AAD tenant at the time the B2B users viral tenant was created.

Hi @SwilliamsIOT and @eschrader just wanted to follow up; is the issue resolved or did you find a workaround that we might be able to add to the article?

Microsoft support got back to me, it all worked out. An admin at the B2B guest company took over control of the domain and restricted password reset, so the employee contacted their co-worker and was able to reset the password.
Here is what the MS Support rep said:

Subject:[REG:118102621000069] O365 B2B Viral tenant user forgot password and cannot self service password reset

Body: In addition, please find a summary of the case below:
Problem: A user from a specific tenant cannot reset the password because SSPR is not enabled in the home directory.
Resolution: Home directory has a global administrator that can either enable SSPR or directly reset the password for the user.
I talked with Eric about the inconsistency that B2B feature has when you try to manage a B2B user. The limited options that a user has to manage his own work account in the following scenario:
A user was invited to a another company tenant, at that time a viral tenant is created to host the invited user and is unmanaged. If the user doesn’t have any administrative privileges in the company, he cannot do the admin takeover to enable SSPR and needs to contact a person that can verify TXT records of the domain to perform the admin takeover.
These are big steps to perform just to be able to reset an user password and force the company to manage an Azure since that moment.
Actual documentation suggests that an invited user can get an email to reset the password but this did not work as intended.
As commented, backend team has been working on the feature to avoid these kind of inconsistencies and hopefully they will help with this little management breaches soon.
Best regards,
Support Engineer | Cloud Identity POD Support

That's great, thanks so much for the update. I'll work this into the docs.


Is Tensorflow impossible to predict multiple steps?

Thank you for reading. I'm not good at English.
I am wondering how to predict and get future time series data after model training. I would like to get the values after N steps.
So, I used the time series in the Tensorflow tutorial to practice predicting the model.
a = y_val[-look_back:]
for i in range(N-step prediction): #predict a new value n times.
    tmp = model.predict(a.reshape(-1, look_back, num_feature)) #predicted value
    a = a[1:] #remove first
    a = np.append(a, tmp) #insert predicted value

The result predicted a value from -0.0000035 to -0.000005 very differently from -1.8 to -0.4 of the actual y value.as like below.

How can I get the actual, right solution? or Is Tessorflow impossible to predict multiple steps?
full source  (After the 25th line is my code.)

You did not train the model (no .fit method is called).

You did not train the model (no .fit method is called).

You did not train the model (no .fit method is called).

Thank you. Updated the .fit call.

@Lay4U, Are you happy to close this issue as it is resolved. Thanks!

@Lay4U, Are you happy to close this issue as it is resolved. Thanks!

No.. The problem was updated and not solved. Still predict plots linear.

I am not part of official TF development team, but I believe this issue tracker should be used to report bugs or feature requests. In this case there seems to be nothing wrong with TF, but instead you would like to get help about TF -- there are better places where to get it, for example on https://stackoverflow.com.
BTW, you should probably not use y_val_uni on input, given it contains valid outputs, you should instead use one of x_val_uni. And after 100 steps where the model gets its own predictions (which are not particularly good) on input, the results are not very surprising.

I am not part of official TF development team, but I believe this issue tracker should be used to report bugs or feature requests. In this case there seems to be nothing wrong with TF, but instead you would like to get help about TF -- there are better places where to get it, for example on https://stackoverflow.com.
BTW, you should probably not use y_val_uni on input, given it contains valid outputs, you should instead use one of x_val_uni. And after 100 steps where the model gets its own predictions (which are not particularly good) on input, the results are not very surprising.

Thank you for answer and when I use x_val_uni and after 20 steps result is
I think this method also not right how do you think?

@Lay4U Please go through this tutorial. Closing this issue as it has been resolved. If you have any more questions and are related to bug/performance, build/install, docs or feature request related issues please create a new issue. If they are not related, please post them in stackoverflow. Thanks!


Database Permissions on Tables created subsequent V1

Describe the bug
Database migrations that add tables subsequent to the V1 migration, are not accessible by the ${api_db_username}
To Reproduce
Steps to reproduce the behavior:

hit the user_role_assignment end point deployed to AWS.
user_role_assignment link will return "internal error"
view the cloudwatch logs for the lambda and you will see the following error:
sqlalchemy.exc.ProgrammingError: (psycopg2.errors.InsufficientPrivilege) permission denied for table fam_role_type: link to cloudwatch error
See error

Expected behavior
When the tables are created in migrations ideally they should automatically be viewable by the ${api_db_username}
Additional context
This bug identifies two issues that need to be resolved:

fix api docker-compose as they should be running under the ${api_db_username} locally to expose these issues before they get to AWS

create a new migration that executes the following lines in the db:

GRANT SELECT, UPDATE, DELETE, INSERT ON ALL TABLES IN SCHEMA app_fam TO ${api_db_username};

ALTER default privileges in schema app_fam grant select, insert, delete, update on tables to ${api_db_username};

Done - Updated database migrations management wiki page
Confirmed - a missing grant would be detected when testing the API either directly or from front-end using local docker-compose.


sitToStand --> squatToStand

Brief summary of changes
This PR updates all sitToStand examples and related files to the name "squatToStand", which has become our more recent terminology.
CHANGELOG.md (choose one)

[x] updated
[ ] no need to update because...

This change is

@chrisdembia the Mac build is failing but I can't view the log for some reason. I tried downloading the logs and the Build Moco step (and all following steps) were not included.

The previous commit was building fine, and there's no reason why the latest commit changes would affect a build, so I restarted the CI.

Yeah sometimes GitHub Actions doesn’t show error messages. Pretty frustrating.

Thanks @chrisdembia.


XGBoost default HP num_rounds from 1 to 100

num_rounds =100 is a much better default HP

My policy is generally to match default hyperparameters to what the package documentation declares them to be, because we are not in the business of documenting the third-party models we wrap.
@azev77 Have you checked the XGBoost docs?

1 nrounds is the only HP XGBoost doesn't provide a default value for.
library(xgboost)
data(agaricus.train, package='xgboost')
data(agaricus.test, package='xgboost')
train <- agaricus.train
test <- agaricus.test

b= xgboost(train$data,train$label)
Error in xgb.train(params, dtrain, nrounds, watchlist, verbose = verbose,  :
  argument "nrounds" is missing, with no default

b=xgboost(train$data, train$label, nrounds = 100)

All good then. Thanks for the clarification and this PR!

Technically this is breaking, and I will hold out on merging for now. But have added it to the list #202


Game Maker Studio support?

Hi,
I just noticed that new version of DragonBones can import Spine files.
Game maker studio is supporting Spine json format. Is it possible to make DragonBones to export spine like json format so it can be imported into Game Maker Studio? That would be awesome feature and money saver for us indie game developers!

does game maker studio 2 support spine?

@blurymind Yes.  GM:Studio 2 has native support for importing Spine's json format.  If you go to http://docs2.yoyogames.com/ and then search for Spine you can then see additional details under
Importing Non-Bitmap Sprites
- Skeletal Animation Sprites (Spine)

Can it also import dragonbones json data?
I dont own a spine license
On Thu, Oct 5, 2017 at 4:27 PM, Joiey Seeley<EMAIL_ADDRESS>wrote:

@blurymind https://github.com/blurymind Yes. GM:Studio 2 has native
support for importing Spine's json format. If you go to
http://docs2.yoyogames.com/ and then search for Spine you can then see
additional details under
Importing Non-Bitmap Sprites

Skeletal Animation Sprites (Spine)

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DragonBones/DragonBonesAS/issues/76#issuecomment-334500903,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGMbVVh0s-dFZRoeOU60Th5MSz20xdWfks5spPVJgaJpZM4I6t1Y
.

that is precisely why i think it would be very beneficial if gamemaker
studio 2 adds support for dragonbones json data. It will make it more
afordable for indie devs to use this kind of animation
On Thu, Oct 5, 2017 at 6:34 PM, Joiey Seeley<EMAIL_ADDRESS>wrote:

@blurymind https://github.com/blurymind No. It supports Spine json, but
you need a Spine license due to licensing resctrictions.
It also supports swf.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DragonBones/DragonBonesAS/issues/76#issuecomment-334537081,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGMbVU0m1y1nvUeS7uUUBYYInKh6onj3ks5spRMogaJpZM4I6t1Y
.

even better it would be if godot engine had support for dragon bones..
butalas, there is only a community made spine module
On Thu, Oct 5, 2017 at 7:11 PM, Todor Imreorov<EMAIL_ADDRESS>wrote:

that is precisely why i think it would be very beneficial if gamemaker
studio 2 adds support for dragonbones json data. It will make it more
afordable for indie devs to use this kind of animation
On Thu, Oct 5, 2017 at 6:34 PM, Joiey Seeley<EMAIL_ADDRESS>wrote:

@blurymind https://github.com/blurymind No. It supports Spine json,
but you need a Spine license due to licensing resctrictions.
It also supports swf.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DragonBones/DragonBonesAS/issues/76#issuecomment-334537081,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGMbVU0m1y1nvUeS7uUUBYYInKh6onj3ks5spRMogaJpZM4I6t1Y
.


Disable failing test testOpenWithProperties

Purpose
Disable the failing test case testOpenWithProperties

Disregard this PR, the fix would be included in [1]
[1] https://github.com/ballerina-lang/ballerina/pull/4954/files


How frequently are you updating this bin db?

Hi, how frequently are updates planned to be done?
Thanks for the amazing work!!

Hi,
once per 1-2 months. Going to update again soon.
Thanks!

Is there any chance of doing it with ranges instead of all bins one by one?
Is it extended to 8 digits bin?

Hi @venelinkochev  when shall we expect the update?
Thanks!


Use an ASCII only status line

Update of #415, might fix #414. @amoffat test?

Give it a test locally and see if you can reproduce it.  This is what I did to stop the bleeding for the time being.

I can't reproduce it locally, with or without these changes.

Wait, which PRs is it failing on? For this PR it works fine, look at the status!

Closing this PR as it doesn't appear to help solve the problem.

Maybe it's not working because it's polling old statuses that are already using the invalid characters?

@andrewda Which invalid characters? Remember that the new status is working fine. We also are POSTing when it errors, so we're setting not getting.
Maybe setting the status to unicode is allowed, but getting a unicode status isn't?

@PlasmaPower Good point. That could be, though from the logs it's only crashing on https://api.github.com/repos/chaosbot/Chaos/statuses/fc703c52effdae09d88937721116f23096b94dd9 – all other status requests are working fine.

@andrewda I know that - any idea what's special about that one?

I checked it against some of the others and couldn't find anything obviously out of the ordinary...

Maybe once the PR behind the status gets merged/closed/whatever, it'll stop. IDK.


[MODULE] - VADER sentiment (scores only)

Please describe the module you would like to add to bricks
The VADER sentiment brick will be split into a module only for classification and a generator that returns the scores. The classification one keeps the previous issue ID. This issue is meant to the VADER sentiment module that returns the scores. Please post everything regarding the VADER sentiment score brick here.
Do you already have an implementation?
from vaderSentiment.vaderSentiment import SentimentIntensityAnalyzer
import json

ATTRIBUTE: str = "text" # only text attributes

def vader_sentiment(record):
    analyzer = SentimentIntensityAnalyzer()
    text = record[ATTRIBUTE].text

    vs = analyzer.polarity_scores(text)
    return json.dumps(vs)

Additional context
The issue for the VADER sentiment classifier can be found here -> #157

Part of the bricks restructure. This issue will be closed once the restructure is done.

Implemented with #283


Broken Next.js link in table of contents

It leads nowhere because the section is gone. Remove?

Yes. Can you send PR?

#93


Replace the default JSON consumer

By default, the JSON consumer (reading responses and returning objects to the user) will use json.Number to represent all kinds of numbers (floats, ints, etc)
While it is probably more exact, json.Number is not easily usable because it requires an extra step to convert to int or float
This replaces the default consumer with one that does not set dec.UseNumber()

Changes to ItemDTO and RouteExport are unrelated. Just from regenerating


Yotta version should only output the version number not 'info: 1.3.1'

When trying to include the version number of the main module in CI, it would be beneficial to not get the superfluous 'info:'.
Is:
    $ yotta version
    info: 1.2.1

Expected:
    $ yotta version
    1.2.1

Note, this is not about yotta --version.

This is an area I think we could make some other usability improvements too: sometimes people mistake yotta version and yotta --version. I notice that npm actually lists the versions of all its components (including the module in the current directory, if any) in the output of npm version.
How would you feel about formatted output including similar information, for example:
> yotta version
{
    "your-module-name": "1.2.3",
    "yotta": "0.12.0",
    "python": "2.7.11",
    "requests": "2.8.1",
    "PyGithub": "1.26.0",
    "cryptography": "1.0.2"
}

Definitely there is a usability issue with yotta version and yotta --version.
I'd prefer a plain output, which one can immediately re-use, e.g in CI. How about yotta --plain version outputting 1.2.1?

Closing my yotta issues.


fix fix flaky test test_pk_uk_index_change

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

p0 case don't need such sync, it's not the root cause of the case failure, close it first


Add ISO code for Montenegrin

The ISO language entry for Montenegrin ("cnr: Montenegrin") isn't in dspace yet, as it was only approved in 2017. It needs to be added.

I think I've now added support for this on fado (f74aa9ca0a05a3b008d0692c29a1fc0bc96ecbc4). For example, https://beta.clarin.si/repository/xmlui/handle/11356/1059.
@TomazErjavec: could you verify?

Verified, it works!
Awaiting its release on fido.


13140 - Nova Scotia - Either split or rename

Although this society is called Nova Scotia, some of the songs assigned to it are by Anglo-Canadians from other parts of Canada. I would suggest either of two options:

Rename society "Anglo-Canadians" or something like that, and tweak geographic classification to be broader than Nova Scotia. Society lat-lon is already in Ontario. I think this society used to be called British Canadians or something similar but was renamed Nova Scotia without reassigning any songs.

Split into two societies:
13140 - Nova Scotia - Keep songs 585, 586, 587, 606, 607 assigned to it. Make society lat-lon average of song lat-lons.
New society - Ottawa Valley Anglo-Canadians (or similar) - Assign songs 608 and 609. Make society lat-lon average of song lat-lons.

I defer to @annalwood on which of these options is preferable.
Also, songs 608 and 609 should have homeland lat-lon (51.65, -0.08), since the singer was born in Enfield, England.

Agreed. Ottawa Valley coincides with any other source?  Check please
Sent from my iPhone

On Jan 3, 2022, at 7:20 AM, stellasilbert @.***> wrote:

Decided to go with option 2, splitting into two societies.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were assigned.


Update environment for dem-stitcher with new urls

See https://github.com/ACCESS-Cloud-Based-InSAR/dem-stitcher/pull/49

@jhkennedy - please review at your convenience.
The only change here is to the environment.yml.
All other changes are cosmetic:

assert() is not kosher with flake8.
I also updated the .truffle.txt per your suggestion.


postgres: add an extensions option

This is a convenience option to install a list of PostgreSQL extensions. The set of available extensions is provided as an argument.
{
  services.postgres.enable = true;
  services.postgres.extensions = extensions: [ extensions.postgis ];
}

You could already do this by modifying the package with withPackages and then overriding services.postgres.package. The new option has two advantages over the existing approach:

It's easier to find in the documentation.

Users unfamiliar with nixpkgs are unlikely to be aware of
pkgs.postgresql.withPackages specifically, or the pattern in
general.

The NixOS services calls this option extraPlugins. First of all, that's not what PostgreSQL calls them. Second, you have to manually source the extensions from pkgs.postgresql.pkgs and ensure that the PostgreSQL version matches whatever is specified in services.postgres.package.
The description for the option contains a list of available extensions generated from the current version of PostgreSQL.

@thenonameguy, very nice! 🙌


TAJO-1360: VALUES_ field in OPTIONS table of catalog store should be longer.

I suggest OPTIONS size as 4000, because max size of oracle varchar2 is just 4000
I researched docs for these patch.
maria: https://mariadb.com/kb/en/mariadb/varchar/
mysql: http://dev.mysql.com/doc/refman/5.7/en/char.html
postgres: http://www.postgresql.org/docs/8.3/static/datatype-character.html
derby: https://db.apache.org/derby/docs/10.7/ref/rrefsqlj41207.html
oracle: https://docs.oracle.com/cd/B28359_01/server.111/b28320/limits001.htm#i287903
(here varchar2 limit is just 4000)

@hyunsik @dongjoon-hyun @ykrips
I'm sorry to make you confused. Thank you.

@hyunsik ,
If you are unable to work on this pull request, can I finish up this request?
This issue is related to another pull request, #491 .

@ykrips I would greatly appreciate it if you finish up this request.

Hello All,
+1
This patch was tested with Derby, MySQL 5.1.17, MariaDB 10.0.17, PostgreSQL 8.4.18, and Oracle <IP_ADDRESS>.0, and worked well as expected. I will commit it in a short time.


Eliminate delvewheel mangling and simplify try_unmangle

We have been working on a new package called ocp-addons which is intended to provide separate performance-critical code in C++ and built with pybind11 against OCP libs. We are able to compile wheels and use them on Linux/MacOS without issue. We are able to compile a wheel on Windows and use it with the ocp package from conda-forge, but have had some very confusing issues with the cadquery-ocp. Basically the mangling process from delvewheel seems to introduce something that looks like an ABI/DLL incompatibility. The fix is to simply pass the --no-mangle-all to delvewheel during the repair process and the OCCT libs will be copied like e.g. TKBRep.dll rather than with mangled names. Something about delvewheel is actually modifying the SHA256 sum of the DLLs it mangles.
There is also some redundant/non-working code for in setup.py of the above repo for windows wheel unmangling that is eliminated here.

@jdegenstein I am re-running the wheel build now and you should be able to test the files manually.
https://github.com/CadQuery/ocp-build-system/actions/runs/8757978524


Getting OOM system RAM when fine tuning MBart on TPU

I am need to this industry and little I know. I am tryining to fine tune MBart of TPU colab pro with distributed --distributed-world-size 8. It has 24GB system memory and TPU v2. The model size is around 5.5GB with 600M parameter. I can't understand why I am getting system memory OOM issue no matter how much I minimize batch size?
At the moment, I am also fine tuning the same thing in 1 GPU colab pro with much higher batch size. Everything is fine.
What's your environment?

fairseq Version (e.g., 1.0 or master): 0.10.1
PyTorch Version (e.g., 1.0) 1.7
OS (e.g., Linux):
How you installed fairseq (pip, source): pip
Build command you used (if compiling from source): pip install --editable fairseq
Python version: 3.6
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

TPU support is somewhat experiments currently. What command are you running? Most likely you are running out of TPU memory, not system memory, and need to lower the batch size further. Keep in mind that there is significant extra memory usage during training from things like optimizer state, activations, etc., it’s not just the model size.


What version of the cf cli are you using: cf html5-list

If I use command cf html5-list as you show in the readme it says:
'html5-list' is not a registered command. See 'cf help -a'

I'm on  cf version 7.0.2+17b4eeafd.2020-07-24.

You have to add HTML5 plugin to CF - see https://sap.github.io/cf-html5-apps-repo-cli-plugin/


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Wake up, bedtime images not rounded.

Hi,
I wonder what I miss with the setup. I've already tried to search at Stack Overflow, yet I haven't seen a related issue or answer to the problem.
Currently using xCode8.1\Swift 3
I have the following line of codes, circularSlider is an outlet.
override func viewDidLoad() {
super.viewDidLoad()
self.setUpCircularSlider()
self.onValueChangeUpdateData(self.circularSlider)
}
fileprivate func setUpCircularSlider(){
self.circularSlider.startThumbImage     = UIImage(named: "bedTime")
self.circularSlider.endThumbImage       = UIImage(named: "wakeUp")
let dayInSeconds                        = 24 * 60 * 60
self.circularSlider.maximumValue        = CGFloat(dayInSeconds)
self.circularSlider.startPointValue     = CGFloat(1 * 60 * 60)
self.circularSlider.endPointValue       = CGFloat(8 * 60 * 60)
}
Having the following runtime attributes:

This is the result:

Notice the image, it doesn't look good.
Any input will be greatly appreciated.
Thank you.
Regards,
Jun

Thank you Hamza for acknowledging this as question.
Do you have any idea on this regard.
Thank you.

Hey @asteriomanet,
Firstly, the o'clock has 12 as max value, so you should change the max circular slider value to 12 * 60 * 60 (or change the o'clock) you could see alarm and the time slider examples in the sample code.
About the images, the circular slider take the size contents of the image, for example, if you use 40px as line width, is recommended to use 40,40 for the image, second thing, the background of the image should not be transparent.
If you have any question don't hesitate.
Best regards,
Hamza

Thank you Hamza. It helps a lot.

You are welcome, if you have any other questions don't hesitate :)


SHA-384 support

Hi, I'm working on a project that I'd like to support SHA-384. Any plans to implement this?

There's no plan for that, however if you can provide a good implementation, I'm open to merge it.

According to Wiki, the SHA-384 is just a truncated version of 512: https://en.wikipedia.org/wiki/Secure_Hash_Algorithms

@h2non this issue should be closed as no implementation has been provided within a year...

Right, I don't even think I would support that either. Closing...


Unable to use CacheManager with Redis config in .net core

In .net core ConfigureServices, trying to add instance of cache as singleton
See following code snippet:
 ICacheManager<Model> cache = CacheFactory.Build<Model>(settings =>
            {
                settings
                    //.WithMicrosoftMemoryCacheHandle()
                    //.And
                    .WithRedisConfiguration("redis", config =>
                    {
                        config.WithAllowAdmin()
                            .WithDatabase(0)
                            .WithEndpoint("localhost", 6379);
                    })
                    .WithMaxRetries(1000)
                    .WithRetryTimeout(100)
                    .WithRedisCacheHandle("redis", true);
            });
            services.AddSingleton(cache);

Getting the below exception:
System.InvalidOperationException
  HResult=0x80131509
  Message=Cache handle CacheManager.Redis.RedisCacheHandle`1 requires serialization of cached values but no serializer has been configured.
  Source=CacheManager.Core
  StackTrace:
   at CacheManager.Core.BaseCacheManager`1..ctor(String name, ICacheManagerConfiguration configuration)
   at CacheManager.Core.BaseCacheManager`1..ctor(ICacheManagerConfiguration configuration)
   at CacheManager.Core.CacheFactory.Build[TCacheValue](String cacheName, Action`1 settings)
   at CacheManager.Core.CacheFactory.Build[TCacheValue](Action`1 settings)
   at CalulationService.ServiceFactory.ConfigureServices(IServiceCollection services) in C:\Projects\poc-optimist-calc-engine\server\CalculationService\ServiceFactory.cs:line 48
   at FIS.Risk.Core.Services.AspNetCore.AspNetCoreServiceFactory.Microsoft.AspNetCore.Hosting.IStartup.ConfigureServices(IServiceCollection services)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.EnsureApplicationServices()
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.Initialize()

Then added CacheManager.Serialization.Json nuget package and chaged config settings to have "WithJsonSerializer"
				settings
                    //.WithMicrosoftMemoryCacheHandle()
                    //.And
                    .WithRedisConfiguration("redis", config =>
                    {
                        config.WithAllowAdmin()
                            .WithDatabase(0)
                            .WithEndpoint("localhost", 6379);
                    })
                    .WithJsonSerializer()
                    .WithMaxRetries(1000)
                    .WithRetryTimeout(100)
                    .WithRedisCacheHandle("redis", true);

This started giving theConnection to Redis failed error, Redis service is up and running
System.InvalidOperationException
  HResult=0x80131509
  Message=Connection to 'localhost:6379,allowAdmin=True,connectTimeout=5000,ssl=False,abortConnect=False,connectRetry=10,proxy=None' failed.
  Source=CacheManager.Core
  StackTrace:
   at CacheManager.Core.BaseCacheManager`1..ctor(String name, ICacheManagerConfiguration configuration)
   at CacheManager.Core.BaseCacheManager`1..ctor(ICacheManagerConfiguration configuration)
   at CacheManager.Core.CacheFactory.Build[TCacheValue](String cacheName, Action`1 settings)
   at CacheManager.Core.CacheFactory.Build[TCacheValue](Action`1 settings)
   at CalulationService.ServiceFactory.ConfigureServices(IServiceCollection services) in C:\Projects\poc-optimist-calc-engine\server\CalculationService\ServiceFactory.cs:line 48
   at FIS.Risk.Core.Services.AspNetCore.AspNetCoreServiceFactory.Microsoft.AspNetCore.Hosting.IStartup.ConfigureServices(IServiceCollection services)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.EnsureApplicationServices()
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.Initialize()

Please point us to sample code where .net core is using CacheManager with Redis.
If this is a known bug in CacheManager, do let us know when this will get fixed

No there is absolutely no known bug using .NET Core. I'm using that combination every day/in production.
Both errors are very obvious issues with your configuration/setup.
You figured out the first one with serialization. That's by design, you have to specify a serialization method...
Second one is most likely a invalid connection string. Seems Redis is not running on localhost port 6379?
/closed for now. Will re-open if you can provide a reproducible example

Hi Micha,
Thanks for the quick reply.
I fail to understand how should I fix this?
Could you give me some link for sample app?
Regards,
Vani.
From: MichaC<EMAIL_ADDRESS>Sent: 03 October 2018 15:21
To: MichaCo/CacheManager<EMAIL_ADDRESS>Cc: Kulkarni, Vani<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MichaCo/CacheManager] Unable to use Redis in .net core (#256)
No there is absolutely no known bug using .NET Core. I'm using that combination every day/in production.
Both errors are very obvious issues with your configuration/setup.
You figured out the first one with serialization. That's by design, you have to specify a serialization method...
Second one is most likely a invalid connection string. Seems Redis is not running on localhost port 6379?
/closed for now. Will re-open if you can provide a reproducible example
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMichaCo%2FCacheManager%2Fissues%2F256%23issuecomment-426576168&data=02|01|vani.kulkarni%40fisglobal.com|9501bf2d54e64b661e6908d62915a870|e3ff91d834c84b15a0b418910a6ac575|0|0|636741570326677348&sdata=U0%2FcpFCfYWaQb5F4FzHXp8CxkWm0t214Te2hvL0Rq4o%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAluDJwyniFU-tr4TMJYMNV-0FX2hdfENks5uhIhmgaJpZM4XFpy2&data=02|01|vani.kulkarni%40fisglobal.com|9501bf2d54e64b661e6908d62915a870|e3ff91d834c84b15a0b418910a6ac575|0|0|636741570326687357&sdata=Mz1jOiF7ehMAirVymHd1rSajFFyY5nnr9Lwp3%2BDANYA%3D&reserved=0.
The information contained in this message is proprietary and/or confidential. If you are not the intended recipient, please: (i) delete the message and all copies; (ii) do not disclose, distribute or use the message in any manner; and (iii) notify the sender immediately. In addition, please be aware that any message addressed to our domain is subject to archiving and review by persons other than the intended recipient. Thank you.

Hi Micha,
Thanks for the quick reply.
I fail to understand how should I fix this?
Could you give me some link for sample app?

Hi Micha,
Thanks for the quick reply.
I fail to understand how should I fix this?
Could you give me some link for sample app?
Redis is running on localhost port 6379, when I use StackExchange.Redis directly , it works
I was referring to https://github.com/MichaCo/CacheManager/issues/94 for our implementation
Regards,
Vani.
[Vani]
The information contained in this message is proprietary and/or confidential. If you are not the intended recipient, please: (i) delete the message and all copies; (ii) do not disclose, distribute or use the message in any manner; and (iii) notify the sender immediately. In addition, please be aware that any message addressed to our domain is subject to archiving and review by persons other than the intended recipient. Thank you.

again, your configuration looks fine.
Using that configuration actually works for me if I just run Redis locally.
There is currently no known issue using CacheManager+Redis on .NET Core 1.x, 2 or 2.1
I don't know why you refer to #94, that issue has nothing todo with the code you showed.
(and please stop spamming, thanks)

It worked, thanks a lot.
I was referring to #94 for DI registration
Regards,
Vani.
From: MichaC<EMAIL_ADDRESS>Sent: 03 October 2018 16:00
To: MichaCo/CacheManager<EMAIL_ADDRESS>Cc: Kulkarni, Vani<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [MichaCo/CacheManager] Unable to use CacheManager with Redis config in .net core (#256)
again, your configuration looks fine.
Using that configuration actually works for me if I just run Redis locally.
There is currently no known issue using CacheManager+Redis on .NET Core 1.x, 2 or 2.1
I don't know why you refer to #94https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMichaCo%2FCacheManager%2Fissues%2F94&data=02|01|vani.kulkarni%40fisglobal.com|68f15c91623947d0380d08d6291b3ce8|e3ff91d834c84b15a0b418910a6ac575|0|0|636741594291063755&sdata=%2FMK0PDTYL5K9gN4XwH6Zhe7W07DVnmSJk0D1rHXzLic%3D&reserved=0, that issue has nothing todo with the code you showed.
(and please stop spamming, thanks)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMichaCo%2FCacheManager%2Fissues%2F256%23issuecomment-426587329&data=02|01|vani.kulkarni%40fisglobal.com|68f15c91623947d0380d08d6291b3ce8|e3ff91d834c84b15a0b418910a6ac575|0|0|636741594291063755&sdata=Gm6BDZ20%2FYf9KWPC9wR4c788VKv%2FUN9wBuVGNgAkY48%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAluDJ_VfKVRALBizqAAuF64CJupOYqvzks5uhJHDgaJpZM4XFpy2&data=02|01|vani.kulkarni%40fisglobal.com|68f15c91623947d0380d08d6291b3ce8|e3ff91d834c84b15a0b418910a6ac575|0|0|636741594291063755&sdata=tK3CY66VKer%2FeYzdTeU7hldbyiDAIxIjEhietAkO43Y%3D&reserved=0.
The information contained in this message is proprietary and/or confidential. If you are not the intended recipient, please: (i) delete the message and all copies; (ii) do not disclose, distribute or use the message in any manner; and (iii) notify the sender immediately. In addition, please be aware that any message addressed to our domain is subject to archiving and review by persons other than the intended recipient. Thank you.


Implement Gradient Clipping

Introduce gradient clipping to prevent exploding gradients by limiting the magnitude of gradient updates during backpropagation.
Objective:
Stabilize training for deep networks and improve convergence.
Implementation Suggestions:

Add a parameter for gradient clipping in the optimizer class.
Implement clipping logic in the backpropagation function.
Allow users to customize clipping thresholds.

@ravin-d-27 Hey, I like to work on this can you assign it to me


Press Enter to manage your extensions doesn't do anything

#93641
Version: 1.44.0-insider (user setup)
Commit: d7d1147d6b764a48b789000c03088ee9afabcd8a
Date: 2020-03-31T08:23:56.446Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362

ctrl+p, "ext ", enter
This doesn't do anything, it just keeps flashing?

Does not reproduce for me, but then I wonder why the first item is not selected?

@Tyriar do you have accessibility mode on?

Yes I had it on, I thought it was an informational label. If this is just an action to open the view it should say "Manage extensions", there are more ways to trigger it than just enter (touch, click).

As for the issue of not selecting that item, we have https://github.com/microsoft/vscode/issues/93610
I leave it up to @sandy081 to change this label, it was unchanged from stable:

No plans to change.

Why? We don't tell the user to press enter on any commands in the command palette, right now it's inconsistent with literally every other quick access provider.

Yah this is an information label and I like this way. There are no pickers like in other cases. Also I did not see any users complained about it.


values allowed in properties schema

should be useful put in properties schema a way of configure values allowed

Not sure I quite understand the comment, presumably this is helpful if your intention is to restrict the values to a specific list of items.


Python 3.11

💡 Your Question
I'm getting getting the following conflicting deps for  pip install super-gradients==3.7.1:
The conflict is caused by:
    super-gradients 3.7.1 depends on numpy<=1.23
    onnx 1.15.0 depends on numpy
    onnxruntime 1.15.0 depends on numpy>=1.24.2

Is there any timeline for when python 3.11 will be supported?
Thanks for the great work on the package.
Versions
PyTorch version: N/A
Is debug build: N/A
CUDA used to build PyTorch: N/A
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.3 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: Could not collect
Libc version: glibc-2.35

Python version: 3.11.8 | packaged by conda-forge | (main, Feb 16 2024, 20:53:32) [GCC 12.3.0] (64-bit runtime)
Python platform: Linux-6.5.0-26-generic-x86_64-with-glibc2.35
Is CUDA available: N/A
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: N/A
GPU models and configuration: GPU 0: NVIDIA GeForce RTX 3070 Ti Laptop GPU
Nvidia driver version: 525.147.05
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: N/A

CPU:
Architecture:                       x86_64
CPU op-mode(s):                     32-bit, 64-bit
Address sizes:                      39 bits physical, 48 bits virtual
Byte Order:                         Little Endian
CPU(s):                             20
On-line CPU(s) list:                0-19
Vendor ID:                          GenuineIntel
Model name:                         12th Gen Intel(R) Core(TM) i7-12700H
CPU family:                         6
Model:                              154
Thread(s) per core:                 2
Core(s) per socket:                 14
Socket(s):                          1
Stepping:                           3
CPU max MHz:                        4700.0000
CPU min MHz:                        400.0000
BogoMIPS:                           5376.00
Flags:                              fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf tsc_known_freq pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb ssbd ibrs ibpb stibp ibrs_enhanced tpr_shadow flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid rdseed adx smap clflushopt clwb intel_pt sha_ni xsaveopt xsavec xgetbv1 xsaves split_lock_detect avx_vnni dtherm ida arat pln pts hwp hwp_notify hwp_act_window hwp_epp hwp_pkg_req hfi vnmi umip pku ospke waitpkg gfni vaes vpclmulqdq rdpid movdiri movdir64b fsrm md_clear serialize arch_lbr ibt flush_l1d arch_capabilities
Virtualisation:                     VT-x
L1d cache:                          544 KiB (14 instances)
L1i cache:                          704 KiB (14 instances)
L2 cache:                           11.5 MiB (8 instances)
L3 cache:                           24 MiB (1 instance)
NUMA node(s):                       1
NUMA node0 CPU(s):                  0-19
Vulnerability Gather data sampling: Not affected
Vulnerability Itlb multihit:        Not affected
Vulnerability L1tf:                 Not affected
Vulnerability Mds:                  Not affected
Vulnerability Meltdown:             Not affected
Vulnerability Mmio stale data:      Not affected
Vulnerability Retbleed:             Not affected
Vulnerability Spec rstack overflow: Not affected
Vulnerability Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl
Vulnerability Spectre v1:           Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:           Mitigation; Enhanced / Automatic IBRS, IBPB conditional, RSB filling, PBRSB-eIBRS SW sequence
Vulnerability Srbds:                Not affected
Vulnerability Tsx async abort:      Not affected

Versions of relevant libraries:
[pip3] No relevant packages
[conda] No relevant packages

Looks like it's being addressed in #1946
Until then, I'm finding it easiest to downgrade to Python 3.10 (at least on OSX)

@davesteps 3.8. is scheduled to be released in the next 2-3 weeks, which would reflect the changes in requirements.txt which @HanClinto pointed out.
However, note that we do not officially support 3.11 yet, and theres not targeted timeline when we will at the moment.


Add localforage as an npm dependancies & return module name on import

makes it easier to use in ES6 module:
import angular from 'angular';
import nglocalforage from 'angular-localforage';

angular.module('myModule', [nglocalforage]);

before the PR you have to include both angular-localforage & localforage dependancies manually and do :
import angular from 'angular';
import localforage from 'localforage';
import nglocalforage from 'angular-localforage';

angular.module('myModule', ['LocalForageModule']);

:+1:


Introducing Geomesa Guru on Gurubase.io

Hello @elahrvivaz
I'm the maintainer of Anteon. We have created Gurubase.io with the mission of building a centralized, tool-focused knowledge base. Essentially, each "guru" is equipped with custom knowledge to answer user questions based on collected data related to that tool.
I wanted to update you that I've manually added the Geomesa Guru to Gurubase. Geomesa Guru uses the data from this repo and data from the docs to answer questions by leveraging the LLM.
In this PR, I showcased the "Geomesa Guru" badge, which highlights that Geomesa now has an AI assistant available to help users with their questions. Please let me know your thoughts on this contribution.
Additionally, if you want me to disable Geomesa Guru in Gurubase, just let me know that's totally fine.

thanks for the consideration! I chatted with some of the other maintainers, and we're not comfortable endorsing an LLM yet, just due to their known shortcomings. I think having the GeoMesa guru on your site is fine, as long as it's clear that it's not affiliated with the official project, and hopefully it will help some people! thanks again,


Refactor insight package

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

👍
/lgtm

🚀 🚀 🚀
/approve


Crash on Startup: Windows 7

Reported on Discord - there was a report of a crash on startup:

I downloaded the Windows version, and it crashes on startup. Any way I can get more information to feed back rather than "it crashes"  and what's in the "view details"  windows of the crash reporter?

I saw above about setting ONI2_DEBUG and starting -f, but apart from "creating window", there's still not much of help.

Some additional information that would be helpful:

What version of Windows is the crash occurring?
Is it x86 or x64 Windows?
What graphics card is being used?

In addition, running the editor via the command line with the -f flag would be very helpful in getting the logs - letting us know how far we get:
C:\Program Files\Onivim2\Oni2_editor.exe -f

I've tested on two Windows 10 machines and haven't seen the issue - but I'm wondering if replaces it's occurring on an earlier Windows version? We use some specific APIs for handling DPI awareness and scaling, that may be problematic on earlier Windows versions.

Thanks for Creating the Ticket.
Using Windows 7 Pro,  64-bit,  16Gb RAM.
Graphics Card is Intel(R) HD Graphics 3000 On a Lenovo T520 Laptop
C:\Program Files\Onivim2>set ONI2_DEBUG = 1
C:\Program Files\Onivim2>"C:\Program Files\Onivim2\Oni2_editor.exe" -f
[INFO] [Revery]: Window: Starting window creation...
[INFO] [Revery]: Window: Using vsync: true
[INFO] [Revery]: Window: Creating window Oni2 width: 800 height: 600

From the Windows "has stopped working"  window,
Problem signature:
  Problem Event Name:	BEX64
  Application Name:	Oni2_editor.exe
  Application Version:	<IP_ADDRESS>
  Application Timestamp:	5da287dc
  Fault Module Name:	StackHash_3df4
  Fault Module Version:	<IP_ADDRESS>
  Fault Module Timestamp:	00000000
  Exception Offset:<PHONE_NUMBER>000003
  Exception Code:	c0000005
  Exception Data:<PHONE_NUMBER>000008
  OS Version:	6.1.76<IP_ADDRESS>.256.48
  Locale ID:	2057
  Additional Information 1:	3df4
  Additional Information 2:	3df44c82c3453b9d2ef6eb266b5962c2
  Additional Information 3:	ec83
  Additional Information 4:	ec83d3a522412845c1ec492327d652fe

Downloaded yesterday, think it's v. <IP_ADDRESS>
Oni2_editor.exe digitally signed 13 October 2019 03:11:52

I brought in some additional logging to get some more details; unfortunately I don't have access to a Win7 machine to test, though!

Downloaded Version: <IP_ADDRESS> Date: 10/17/2019, 3:28:45 AM
Still the same
C:\Program Files\Onivim2>set ONI2_DEBUG = 1
C:\Program Files\Onivim2>"C:\Program Files\Onivim2\Oni2_editor.exe" -f
[INFO] [Revery]: Window: Starting window creation...
[INFO] [Revery]: Window: Using vsync: true
[INFO] [Revery]: Window: Creating window Oni2 width: 800 height: 600
If I can help test your version with more logging, I'd be glad to help

Thanks for the update @kevintts ! Interesting - it's failing at creating the SDL2 window - didn't hit any of the new logging.
I saw this thread: https://steamcommunity.com/app/262390/discussions/1/527273452875796792/ where a user hit a similar crash for a game with SDL2 on Windows 7. After updating their drivers - it worked.
Just wondering - what GPU are you using? Maybe that can give us some clues too.
If you're up for building from source - building and running this project: https://github.com/revery-ui/reason-sdl2 could be helpful! We could add logging there to track down the issue.
Still trying to hunt down a Win7 machine to test on

I'm using an intel HD 3000 with latest drivers.
reason-sdl2 crashed in what appeared to be the same way as onivim.
C:\Development\reason-sdl2>esy x test_sdl2
←[0;1mFile "dune", line 1, characters 17-31:←[0m
1 | (ignored_subdirs (node_modules))
                     ^^^^^^^^^^^^^^
←[1;35mWarning←[0m: ignored_subdirs is deprecated in 1.6. Use dirs to specify visible
directories or data_only_dirs for ignoring only dune files.
←[0;1mFile "dune", line 1, characters 17-31:←[0m
1 | (ignored_subdirs (node_modules))
                     ^^^^^^^^^^^^^^
←[1;35mWarning←[0m: ignored_subdirs is deprecated in 1.6. Use dirs to specify visible
directories or data_only_dirs for ignoring only dune files.

Windows 7 machines for testing - https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/ ?
Technically it's for testing different versions of Internet explorer, but they're Windows 7 machines.

Thanks @kevintts for all the help - much appreciated!

Windows 7 machines for testing - https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/ ?
Technically it's for testing different versions of Internet explorer, but they're Windows 7 machines.

Ah I actually saw this... but unfortunately the VMs are only x86 , and Onivim 2 / the toolchain only supports x64. Still working on tracking something down!

I created a branch of reason-sdl2 called windows-7-logging - might give us a bit more info if you're up for trying that. Sorry about the back and forth. Hopefully I can get my hands on a machine to repro soon.

FYI, I added a couple of other prinfs
printf("Calling create window with title: %s width: %d height: %d\n", String_val(vName), width, height);

  SDL_Window *win = (SDL_CreateWindow(
      String_val(vName), SDL_WINDOWPOS_CENTERED, SDL_WINDOWPOS_CENTERED, width,
      height,
      SDL_WINDOW_OPENGL | SDL_WINDOW_ALLOW_HIGHDPI | SDL_WINDOW_RESIZABLE));
  printf("Called create window\n");
  value vWindow = (value)win;
  printf("Set value of Window\n");
  CAMLreturn(vWindow);

and they all worked before the crash:
SDL_INIT returned: 0
Setting SDL_GL_CONTEXT_PROFILE_MASK
Setting SDL_GL_CONTEXT_MAJOR_VERSION
Setting SDL_GL_CONTEXT_MINOR_VERSION
Setting SDL_GL_DOUBLEBUFFER
Calling create window with title: test width: 100 height: 100
Called create window
Set value of Window

Since Windows 7 is EOL now - we'll remove this from our support matrix, and focus on better support for Windows 8/10 (the biggest issue being moving from OpenGL -> DirectX)


Fix Carthage build

I dunno what happened in #6 but this seems to build fine, also this works:
$ echo "github \"ReSwift/ReSwift-Thunk\" \"carthage-fix\"" > Cartfile
$ carthage bootstrap
*** No Cartfile.resolved found, updating dependencies
*** Fetching ReSwift-Thunk
*** Fetching ReSwift
*** Checking out ReSwift at "4.0.1"
*** Checking out ReSwift-Thunk at "a6e02c1651b582d224cde9c558ba22d3618dcc56"
*** xcodebuild output can be found in /var/folders/62/8k21681d08z9lhq8h433z3rh0000gp/T/carthage-xcodebuild.4rYFQw.log
*** Downloading ReSwift.framework binary at "4.0.1"
***  Skipped installing ReSwift.framework binary due to the error:
	"Incompatible Swift version - framework was built with 3.1 (swiftlang-802.0.53 clang-802.0.42) and the local version is 4.2.1 (swiftlang-1000.11.42 clang-1<IP_ADDRESS>)."

    Falling back to building from the source
*** Building scheme "ReSwift-macOS" in ReSwift.xcworkspace
*** Building scheme "ReSwift-watchOS" in ReSwift.xcworkspace
*** Building scheme "ReSwift-tvOS" in ReSwift.xcworkspace
*** Building scheme "ReSwift-iOS" in ReSwift.xcworkspace
*** Building scheme "ReSwift-Thunk" in ReSwift-Thunk.xcodeproj

So, good to go and then try CocoaPods next :)

Cool! Will do it ASAP, haha. I’m glad this worked.
Sent with GitHawk


Switch [f32; _] types in the public api to mint equivalents

This is a larger effort that I'm only just starting. I'm not sure this is the right decision (using language primitives like arrays has a lovely simplicity), but especially for handling sizing in complex layouts, I'm always using a real math lib (basically always glam), and all the .into calls are irksome.
The plan here is that we take mint types as inputs but still return [f32; _] as outputs (for example, in getters).
If anyone has opinions on this, I'd love to hear them

impl Into<[f32; 2]> also makes sense.

It does, but for more complex types, mint support is nice. I'm not sure if that'll come up but it's a good crate to support I think

Hello all -- this has now covered the API. Please let me know if you have any thoughts or opinions -- some of these bounds are a bit rough (especially the InputFloatX and ColorPickerX and ColorEditX widgets). Until mint upgrades, we can't simplify. I added an issue to them here https://github.com/kvark/mint/issues/67
since the above would be a patch we'd be safe to go to 0.5.7.
I'm failing some of the tests, but will fix over the weekend. that mostly is because this includes a breaking change where the ColorPicker and ColorEdit structs where de-genericed to just ColorPicker3 and ColorPicker4, since their generic Into nature was just...a mess.

This looks like a good improvement!
With the caveat that I have almost no idea what I'm talking about: could this potentially have a noticeable impact on binary size due to monomorphization? I have even less idea of how it works in regards to #[inline] methods, but basically could something like
#[inline]
pub fn size(mut self, size: impl Into<crate::math::MintVec2>) -> Self {
    self.size = size.into().into();
    self
}

...mean that user-code like
fn some_giant_function(ui: &imgui::Ui) {
    ui.whatever().size([0.0, 1.0]);
    ui.whatever().size(some_vec);
    ui.whatever().size(some_other_vec);
    // and lots more calls
}

..would end up with lots of copies of some_giant_function with the various permutations of .into().into()?
I had a quick search around and only related-ish thing I could find was https://github.com/rust-lang/rust/issues/77767 so I suspect it probably isn't a big problem

not like that -- it would work the other way around though.
So there will only be one some_giant_function (you can tell since it has no generic type parameters), but there WILL be have at least three (assuming some_other_vec is another T: Into<MintVec2>) implementations of .size on it.
Morever, if you noticed, I actually put the trait bound on the impl block itself, so you can't .size with different concrete types. Actually, supporting that wouldn't be that difficult, but that sounds NEEDLESS.
So that's the say, this wouldn't work...
ui.window("hello")
   .size([10.0, 20.0])
   .size(glam::Vec2::splat(100.0)) // error, expecting [f32; 2]

However, if a user is using two different math libraries, or freely using arrays + a math library, their binary size will definitely increase.
That doesn't overly concern me atm, but what are you thinking? I will say that the compile time increase that this would create somewhat worries me, but since we had #[inline] over most of these functions anyway (and generic code is always a candidate for inlining, even across crate boundaries), it seems like the impact would be too serious.
At least on my end, I've been fairly extensively using the draw API for a node graph editor at work, and being able to use glam instead of arrays will wildly simplify that work

I will say, for a normal math library like glam or vek, we end up at a pretty nice performance situation. We do pay to switch to an from the [f32; 2]` types, though we use some fast instructions on this x86_64 backends: https://godbolt.org/z/sdx4exvMf

Ahh yeh that makes much more sense (for some reason I half-thought inlining a generic function would make the calling function generic, but on actually thinking about it, that.. wouldn't make much sense!)
As long as there isn't much overhead if people exclusively use [f32; 2] then it's hard to see much issue with any potential overhead if someone decides to use 23 different vector types :grin:

At least on my end, I've been fairly extensively using the draw API for a node graph editor at work, and being able to use glam instead of arrays will wildly simplify that work

I've been using euclid for similar purposes and although adding .to_array() each time isn't too bad, removing the need for that would be another minor im_str!-like papercut gone :partying_face:


Aurora compiler warning

A simple thing, but its the ONLY warning i get doing a fresh build on centos!
/home/penfold/github/hyperion.ng/libsrc/leddevice/dev_net/LedDeviceAurora.cpp: In member function ‘virtual int LedDeviceAurora::write(const std::vector<ColorRgb>&)’:
/home/penfold/github/hyperion.ng/libsrc/leddevice/dev_net/LedDeviceAurora.cpp:174:9: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   if (i<udpBufferSize) {
         ^
/home/penfold/github/hyperion.ng/libsrc/leddevice/dev_net/LedDeviceAurora.cpp:183:21: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   if(panelCounter > panelCount) {

Will be fixed in PR #523


isort counts third party imports as local when inside submodules with the same name

repro example
.
├── pyproject.toml
└── src
    └── ruff_test
        └── setuptools
            ├── __init__.py
            └── example.py

pyproject.toml
[project]
name = "ruff-test"
version = "0.1.0"
description = "Add your description here"
authors = [
    { name = "Michael Merickel", email =<EMAIL_ADDRESS>}
]
dependencies = []
readme = "README.md"
requires-python = ">= 3.8"

[build-system]
requires = ["hatchling"]
build-backend = "hatchling.build"

[tool.rye]
managed = true
dev-dependencies = []

[tool.hatch.metadata]
allow-direct-references = true

[tool.hatch.build.targets.wheel]
packages = ["src/ruff_test"]

[tool.ruff]
src = ["src"]

[tool.ruff.lint]
select = ["I"]

example.py
import setuptools

ruff check output
❯ ruff check -v --no-cache src/ruff_test/setuptools/example.py
[2024-03-06][23:01:54][ruff::resolve][DEBUG] Using configuration file (via parent) at: /Users/michael.merickel/scratch/ruff-test/pyproject.toml
[2024-03-06][23:01:54][ruff::commands::check][DEBUG] Identified files to lint in: 2.003584ms
[2024-03-06][23:01:54][ruff::diagnostics][DEBUG] Checking: /Users/michael.merickel/scratch/ruff-test/src/ruff_test/setuptools/example.py
[2024-03-06][23:01:54][ruff_linter::rules::isort::categorize][DEBUG] Categorized 'setuptools' as Known(FirstParty) (SamePackage)
[2024-03-06][23:01:54][ruff::commands::check][DEBUG] Checked 1 files in: 500.583µs

explanation of problem

Categorized 'setuptools' as Known(FirstParty) (SamePackage)

import setuptools here is an absolute import and should be counted as Known(ThirdParty).
The isort logic appears to be counting this as a first party import in a very specific circumstance. The critical detail here is that ruff_test is a namespace package and has no __init__.py file. If you touch src/ruff_test/__init__.py then you get the expected output Categorized 'setuptools' as Known(ThirdParty) (NoMatch).
Instead of adding the __init__.py it is also possible to define a known-third-party section to solve this issue:
[tool.ruff.lint.isort]
known-third-party = ["setuptools"]

Then you will see Categorized 'setuptools' as Known(ThirdParty) (KnownThirdParty).
If you remove the src declaration from tool.ruff it is still a problem, but I think the src declaration should make it extra clear to ruff that the absolute import of the package we created is ruff_test.setuptools, and not setuptools.
If ruff knew the appropriate top-level package roots then import setuptools would be correctly matched to a third party package because "absolute relative imports" are no longer a thing in Python 3.
Another workaround is setting detect-same-package = false. Another option is also namespace-packages = ["src/ruff_test"].
I opened this issue because I think the fact that I set src = ["src"] should have been enough info for ruff to do the right thing in the original pyproject.toml.
I tested this on ruff 0.3.0 and 0.3.1 and observed the same behavior on both.
Possibly related: https://github.com/astral-sh/ruff/issues/5667 https://github.com/astral-sh/ruff/issues/9130

Thanks for the report!
@charliermarsh do we just not support detection of first-party imports in namespace packages in general?


closing connection does not send CONNECTION_CLOSE frame

Hi
We are having issues with some connections that do not close properly.
in ietf_full_conn_ci_tick at closing connection, it does not send pckt with CONNECTION_CLOSE frame (added some prints):
conn: connection is OK to close
conn: generate conn close packet (this print is just before the RETURN_IF_OUT_OF_PACKETS();)
sendctl: send_ctl_can_send: n_out: 2544 (unacked_all: 2501); cwnd: 277536
sendctl: ctl->sc_flags & SC_PACE | send_ctl_can_send -> 0 (this one is the reason for not sending a pckt)
conn: used up packet allowance, sending now (line 8405)
sendctl: packet 7437 has been sent (frame types: ACK)
sendctl: stats: n_total_sent: 7402; n_resent: 0; n_delayed: 0
conn: # ticks: 19482
conn: sent 7402 packets
conn: received 21949 packets, of which 0 were not decryptable, 0 were dups and 0 were errors; sent 7402 packets, avg stream data per outgoing packet is 0 bytes
conn: delayed ACKs settings: (3/1.000/0.800/0.350/0.050/0.050); packet tolerances sent: count: 0, min: 0, max: 0
conn: ACKs: delayed acks on: yes; in: 517; processed: 517; merged: 0
my guess is that pacer do not allow the new pckt.
Quic fix is to disable the RETURN_IF_OUT_OF_PACKETS check in case of closing connection, not sure if this is a good one. pls advise.

This probably related to #281 , maybe need more tweaks handling CONNECTION_CLOSE.

Please check commit (https://github.com/litespeedtech/lsquic/commit/47440e4eaf376676ac658a2b7cfb9470f350e385) .

No comment == No problem. issue closed.


Respect conditions if present

See also https://github.com/facebook/jest/pull/11859
There's probably a better way to do this, but this works for my use case at least. Should fix #5

Hi @vpzomtrrfrt, your changes inspired me to how to aproach the problem and I think I've fixed it at https://github.com/k-g-a/jest-node-exports-resolver/commit/900da2e7eac05090b569e5614bf0b6b6838443de, now conditions are properly enforced and work in (one of) my use cases. Can you update to version 1.1.3 and confirm that it works for you too? If not, please update your PR and I'll merge it :-)

Yes, the latest release appears to work

Yes, the latest release appears to work

Great, good to know :-)


Poor Performance of flan-t5-large Model on MMLU Valid Dataset with Mathematical LoRA

Overview
I have been using the flan-t5-large lora model math_dataset_algebra__linear_1d, which is designed for mathematical tasks, to process and generate results on the MMLU valid dataset. However, the performance of the model on this dataset has been disappointing and the results obtained are not up to the expected standard. Here is the url of the lora: https://huggingface.co/lorahub/flan_t5_large-math_dataset_algebra__linear_1d
Problem Statement
The main issue I'm facing is that despite lora being specifically designed for mathematical tasks, its performance on the MMLU valid dataset is not good as expected.
Steps to Reproduce

Prepare the MMLU valid dataset.
Load and initialize the flan-t5-large model with lora math_dataset_algebra__linear_1d
Input data from the MMLU valid dataset into the model.
Generate results/output using the model.
Evaluate the performance of the model on the MMLU valid dataset.

Expected Behavior
As lora is optimized for mathematical tasks, I expected it to perform well on the MMLU valid dataset. The outputs generated by the model should align with the ground truth data from the dataset, demonstrating accurate and meaningful results.
Experiment Result
Contrary to expectations, the flan-t5-large model's performance on the MMLU valid dataset has been disappointing. The generated outputs often contain errors and inaccuracies, making them unsuitable for my intended use case.
original flan_t5

Subject
Average Accuracy

Abstract Algebra
0.250

College Mathematics
0.280

Elementary Mathematics
0.280

High School Mathematics
0.296

High School Statistics
0.319

Math
0.289

math_dataset_algebra__linear_1d

Subject
Average Accuracy

Abstract Algebra
0.240

College Mathematics
0.300

Elementary Mathematics
0.267

High School Mathematics
0.289

High School Statistics
0.329

Math
0.286

@sunyuhan19981208 Thanks for your detailed report and your interest in our work.
First, I would first clarify that all the upstream modules in lorahub are trained based on the model flan-t5-large using the tasks flan has already seen. In other words, compared to flan-t5-large, we do not see any extra task, and the training tasks are already used in the training of flan-t5-large. Such a setting is to compare our performance with flan-t5-large in an apple-to-apple setting.
Based on the above statement, I would do not expect the model to have a better performance than flan-t5-large on MMLU using the single flan_t5_large-math_dataset_algebra__linear_1d module.
Second, the core idea of lorahub is to integrate modules from different tasks (without human expert design) for the unseen task. And thus, you may want to try to select some potential modules as candidates (or just randomly select some of them), and then use the lorahub learning method. Finally you can evaluate the model performance on MMLU. But honestly, I am not sure if lorahub works well for MMLU since we do have conduct experiments on it.
Last, if you want to improve the MMLU performance, I would suggest you training several new lora modules on different mathematical problems and then use lorahub learning to merge them.
What do you think about it? Does the answer seem reasonable for you? Let me know if you have further feedback.
Thanks,
Qian

@sunyuhan19981208 Thanks for your detailed report and your interest in our work.
First, I would clarify that all the upstream modules in lorahub are trained based on the model flan-t5-large using the tasks flan has already seen. In other words, compared to flan-t5-large, we do not see any extra task, and the training tasks are already used in the training of flan-t5-large. Such a setting is to compare our performance with flan-t5-large in an apple-to-apple setting.
Based on the above statement, I would do not expect the model to have a better performance than flan-t5-large on MMLU using the single flan_t5_large-math_dataset_algebra__linear_1d module.
Second, the core idea of lorahub is to integrate modules from different tasks (without human expert design) for the unseen task. And thus, you may want to try to select some potential modules as candidates (or just randomly select some of them), and then use the lorahub learning method. Finally you can evaluate the model performance on MMLU. But honestly, I am not sure if lorahub works well for MMLU since we do not conduct experiments on it.
Last, if you want to improve the MMLU performance, I would suggest you training several new lora modules on different mathematical problems and then use lorahub learning to merge them.
What do you think about it? Does the answer seem reasonable for you? Let me know if you have further feedback.
Thanks, Qian

Thank you so much for your prompt response!!! I am currently in the process of seeking a suitable code base to train flan-t5-large on the GLUE dataset. However, I've come across some rather outdated code, and I was wondering if you could kindly recommend any updated code that is compatible with these amazing Lora models? Your assistance is highly appreciated!

@sunyuhan19981208 Thanks for your response! What do you mean by train flan-t5-large on the GLUE dataset? You mean lora tuning, or fine-tuning?

@sunyuhan19981208 Thanks for your response! What do you mean by train flan-t5-large on the GLUE dataset? You mean lora tuning, or fine-tuning?

Thanks a lot!!


rustup, adjust error message

@bors r+

:pushpin: Commit 3032224d19394551580a402744ff85b8de5ae99a has been approved by RalfJung

:hourglass: Testing commit 3032224d19394551580a402744ff85b8de5ae99a with merge 177d5f861f278135d0544aea9a30ae13e5fd1e66...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 177d5f861f278135d0544aea9a30ae13e5fd1e66 to master...


Add ocr example

Thanks @fchollet for the review. I will make the necessary changes

@fchollet I have made all the changes except for removing the keras.backend part. I need to check if I need to change anything if I am directly using tf.nn instead of keras.backend

Note that the motivation for using tf.nn APIs is that they're documented and better-maintained. The backend APIs are undocumented, have not been touched in some time, and will eventually be deprecated. But for now, this will do.

Thanks for the review. I have made all the changes.

Note that the motivation for using tf.nn APIs is that they're documented and better-maintained. The backend APIs are undocumented, have not been touched in some time, and will eventually be deprecated. But for now, this will do.

Totally agree with you. My only concern here is that for some APIs, we haven't provided clear cut code examples for their usage (for example CTC related functionalities). So, both things are required: removing the backend and improving the documentation in nn. package

Thanks for the review


Add KMSKeyID option to AWS S3 writer

When KMSKeyID is set objects will be uploaded with server side
encryption using AWS KMS

Looks great, thanks @mwain!