Rethink type inference for quantization and sparsity

The upstream notion of type inference where: 1) result types are inferred from operands and attributes, 2) TableGen generates builders that allow producers to skip specifying result types and have them inferred - has been working fairly well in the early days of MHLO.
This architecture ran into a difficulty with dynamism because inferred result types could be slightly incompatible with actual result types (e.g. inferred tensor<?xf32> vs actual tensor<2xf32>). However, isCompatibleReturnTypes patched that difficulty.
However, as MHLO started been gaining more features, we discovered that this won't scale. For example, it is totally okay to add two tensors and produce a tensor with a different sparsity encoding or different scale / zero point. Papering over this with isCompatibleReturnTypes stretches the metaphor too much.
As a result, we will need to rethink type inference for StableHLO/MHLO, using the two fundamental requirements that we have heard from stakeholders to date:

Producers shouldn't need to specify redundant information (e.g. AddOp builder in C++ or Python shouldn't require repeating the shape of the lhs/rhs, and specifying quantization/sparsity metadata should be optional).
It should be possible to use type inference logic to specialize dynamically-shaped programs to static shapes (there will be a separate ticket about that shortly. upd: see #622).

Similarly to #620, this is part of the public API track.

Producers shouldn't need to specify redundant information (e.g. AddOp builder in C++ or Python shouldn't require repeating the shape of the lhs/rhs, and specifying quantization/sparsity metadata should be optional).

Type inference in the typical sense is used for inferring the  result types from operands and attributes. One of the primary use-cases of type inference is via the tableGen generated builders that allow producers of an op to skip specifying result types and instead have them inferred. This user view would run into a problem with quantization because the result types can be different from the inferred types based on the constraints related to baseline type (refer to the constraints of subtract operation), where the scales and zero-points of the result-quantized-type can be  different from the operands and cannot be inferred.
As we can se this problem is a bit different than inferring dynamic shapes which can be inferred using methods like this.
My proposal would be no have no special handling for type inference for quantization.
While creating an op, If the result-type is expected to be different (w.r.t say quantization parameters) than that of operand then is will be recommended to use the builder methods with the explicit result type to make sure that result-type is not inferred. The builder methods with result types should be used is the quantization parameters are expected to be the same throughout operands and results.

It should be possible to use type inference logic to specialize dynamically-shaped programs to static shapes (there will be a separate ticket about that shortly. upd: see https://github.com/openxla/stablehlo/issues/622).

This is already covered.


APO cache is not cleared properly!

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the plugin and it is still present.

WordPress version
5.8
Cloudflare-WordPress version
4.5.1
PHP version
8.0.8
Expected result
Up to date version of the website on mobile.
Actual result
A week old version.
Steps to reproduce
How to reproduce (on desktop)
Step 1: fıre up your favourıte web browser ın ıncognıto mode
step 2: press f12 and enable mobıle mode by pressıng ctrl + shıft + m
step 3: open technopat.net
step 4: check the content dates, today ıs july 8 but cache ıs served from july 3!
How to reproduce (on mobıle)
Step 1: fıre up your favourıte web browser ın ıncognıto mode
step 2: open technopat.net
step 3: check the content dates, today ıs july 8 but cache ıs served from july 3!
Additional factoids
I was using litespeed cache before, removed it completely.
References
https://github.com/cloudflare/Cloudflare-WordPress/issues/404

I had to create this ticket as the previous one was closed for no reason and after trying to explain the problem to cloudflare for months, I got this reply:

Thank you for contacting Cloudflare Support.
We have run an automated check and the zones previously mentioned in this ticket have been either moved away from Cloudflare or deleted. This limits the troubleshooting actions our Support Engineers can take to resolve your issue. At this time, we will be marking this ticket as solved.
Alternatives to moving away your nameservers are:
Pause Cloudflare
Go to the Overview tab in the Cloudflare dashboard.
At the bottom right of this page, there is a link under Advanced Actions.
Click Pause Cloudflare on Site
Pausing Cloudflare will stop all the proxying and use our services as DNS only.
Gray Cloud DNS records
If the issue is happening only on specific sub-domains, you could gray cloud DNS records. This will stop proxying for that domain. You can do that in the DNS tab of your Cloudflare dashboard by clicking on the cloud icon.
In both cases, Cloudflare features such as DDoS protection, CDN caching, and free SSL will no longer be available.
This is an automated response, which we hope has answered your question.
If you need further assistance, simply reply to this email to reach a Cloudflare Technical Support Engineer.
You can run tests for common issues on our Diagnostic Center.
Search the Cloudflare Community for advice and insight.

The zones previously mentioned in this ticket have been either moved away from Cloudflare or deleted? Are you kidding me? Even my domain name is on cloudlfare. I'm a Pro customer for years! I have made no change.
See it for yourself:

Either fix this problem or disable the problematic service CF.

This is a duplicate of #404 so keep an eye on that issue for updates.
#404 was locked because the conversation was going off topic and had nothing to do with the issue at hand. If you have useful updates, that's fine however you only bog down efforts when you use it as a spot to complain instead of a move an issue forward.
The response you got from Cloudflare support is completely irrelevant here. Support does not manage or maintain the SDKs or plugins; Engineering and the community does.
Closing as a duplicate and again, watch #404 for updates.


Implement bitwise and for Complex class (refs #36)

Ah - and I can see that due to my tardiness, someone else has produced a patch that is ready to merge.  I hope I haven't lost a potential contributor because of this.

No problem 😄 Something's always bound to fall through the cracks when you have a lot on your plate. I'm a fan of the project so will definitely contribute in future where possible


[Critical] 500 Status code appears when invited user (via LDAP) attempts to login first time

Steps to produce :

Login to zafira ;
Click [Users] button on left sidebar ;
Click [Invitations] button ;
Click [+Invite user] button in the right top corner of the screen ;
Click [Choose Source] drop-down menu , select "LDAP" ;
Click [Choose Group] drop-down menu , select any group ;
Click [Enter an Email] field , input available email ;
Click [Invite] button ;
Logout from current user ;
Go to according email box ;
Click [Join now] button in the "Join the workspace" email ;
Input valid username into "Username" field ;
Input valid password in "Password" field ;
Click [Sign Up] button ;

Expected result :
User able to proceed and finish registration via email invite .
Actual result :
"Failed to sign up" error message appears in the right bottom corner .
User unable to proceed and finish registration via email invite .

Fixed


High memory utilization after router upgrade to 0.91.0

Component(s)
router
Component version
0.91.0
wgc version
latest
controlplane version
latest
router version
0.91.0
What happened?
After upgrading from router version 0.84.4 we observe much higher memory utilization of the router instances. Initially after the upgrade all instances were OOMKilled because the pod hit the memory limits. We are running the router in a k8s cluster so here are the resources before the update which were fine for the older router version:
resources:
   requests:
        cpu: 400m
        memory: 256Mi
   limits:
        cpu: 1000m
        memory: 600Mi

Now memory utilization increased to around 1000Mi. Do you have an explanation for this?
Environment information
Environment
Kubernetes 1.25
Router configuration
No response
Router execution config
No response
Log output
No response
Additional context
No response

We're taking a look

Hi @PeteMac88
Could you check with the 0.90.0 version?

Hey @devsergiy, es definitely higher as the  0.84.4 version but not as high as 0.91.0. Here is a table with the average utilization with same load:
0.84.4 ~ 350Mi
0.90.0 ~ 700 Mi
0.91.0. ~ 1200Mi

Thanks a lot
Could you also check 0.88.0?
We already know the reasons why it changes in 0.90.1
But it is a bit of a surprise that it also spikes before that on 0.90.0

@devsergiy Any update here? I needed to downgrade the router yesterday because of the increasing memory consumption.

Moreover, were you able to check 0.88.0 as requested @PeteMac88?

Are you able to build the router from source like so:
go build -tags=pprof main.go

Then run the router with your regular traffic and run the following command:
go tool pprof http://localhost:6060/debug/pprof/heap

You need to have the go toolchain installed for this.
Once you've got a heap profile, can you upload it here for investigation? Thanks

I also ran into this problem when upgrading to 0.95.6 this week. I observed unbounded memory growth when load testing the router with our batch requests > 100 list items and ~ 150 fields across 2 subgraphs (No deep nesting & max 2 levels).
I did not run into this problem with the same queries that only returned a single item.
Reverting to 0.88.0 definitely helpled and the containers use less memory now.

@PeteMac88 @jfroundjian please reopen the issue if the problem persists with the latest router<EMAIL_ADDRESS>Thank you!


Player IP Collection

What is the issue, e.g. map voting, you would like to solve?
It would be helpful to get the IPs of players that are connected to a Squad server to allow admins to possibly:

IP ban players that could be using multiple accounts to evade bans.
Use geolocation to find the rough location of a player for analytic purposes, etc.
Gain a stronger form of identification to allow admins to report players conducting illegal behaviour to law enforcement, and where possible with the appropriate jurisdiction.

How would you like us to solve this issue?
My thoughts on a possible solution:

Monitor a list of IPs connected to the game server's port.
When a new one has detected insert this into a queue.
When the next player has connected go through this queue to find a relatively recent IP connection.
Assign that IP to the player's stored data.

Concerns:

This will only work on servers we can run commands to gain the connection information from, so not rented instances.
How do we observe when new IPs connect? Do we poll netstat? If so, what frequency is appropriate?
Can we be sure that IP connections to that port will occur in the same order as the player connecting? I.e. if player A then player B connect to the port can we be sure that player A's connection to the Squad server will be detected in SquadJS prior to player B's connection to the server being detected in SquadJS.

This is resolved with https://github.com/Team-Silver-Sphere/SquadJS/pull/318
Squad v7.0 replaced the Steam connection manager with EOS, which also introduced logging of players' IPs


FR: Disable on big files

I noticed that biscuits causes nvim to become quite laggy on larger files (100Kb+). As far as I can tell, there is only an option to disable biscuits on specific filetypes – could there maybe also be an option to disabled based on filesize? (or more generally, a user-supplied function which determines whether biscuits should attach to buffer or not)

@chrisgrieser Check out the latest version. I have added a config setting to allow setting a max file size
https://github.com/code-biscuits/nvim-biscuits?tab=readme-ov-file#configuration-max-file-size
closed by #49


STMG4 Dual Bank Flash support

I have noticed that STMG4 support was added a few days ago:
https://github.com/embassy-rs/embassy/pull/2294
However, this PR is not going to work correctly on dual bank flash MCUs such as STM32G474VE that I work with. The reason is that dual bank mode uses 2KB page size while single bank uses 4K pages.

I worked around this issue by deleting two pages on each call to blocking_erase_sector when I was implementing embassy bootloader several weeks ago. Additionally errata 2.2.2 should be implemented (I reused the same code from f4.rs/save_data_cache_state() as the workaround procedure is the same). Other functions beside erasing the sector work without changes.
(btw I have not tested the PR, however I took the same approach as the author and reused the code from g0.rs so I assume that there are the same issues I faced.)

@tomasbarton-stemcell : I'm using the STM32G491RE, which has the same memory layout and modes. I'm unsure about the meaning of your message. Are you saying that the swap algorithm won't work for these chips? If so, do you have a fix, and could you possibly post it? Thank you!

I have a quick and dirty fix here: https://github.com/embassy-rs/embassy/pull/2946/files
It works, but I haven't had a chance to clean this up yet

@tomasbarton-stemcell : I'm using the STM32G491RE, which has the same memory layout and modes. I'm unsure about the meaning of your message.

The problem is that the current Embassy code assumes single bank mode with 4KB sectors, however the chip by default uses dual bank mode with 2KB pages.
So my solution to get this working quickly was to modify the code in such a way that the erase function erases 2 2KB sectors for each 4KB sector call. The write function does not case about the page size. Other solution might be to switch the chip to single bank mode (I could not do that since I need double bank for something else)

note G4 in dualbank mode has a hole in the address space between the two banks (except in the chips with max amount of flash). You can't pretend the two banks form a single contiguous flash region.
Even worse, all G4 chips are actually the same silicon die, so they actually all have the max amount of flash. The "hole" in the address space is actually working flash that you can use, but is "untested, not guaranteed to work" according to ST. So you might have the wrong memory.x but never notice because it works anyway.
If you don't need dualbank and want one contiguous chunk of flash, you really should actually turn off dualbank.

Thanks for the follow up! Maybe I'm missing something, but in the specific case of the STM32G491RE, which has 512K (2 x 256K in dual-bank config), the flash driver should work just fine in dual-bank mode.
If you look at https://github.com/embassy-rs/stm32-data-generated/blob/main/stm32-metapac/src/chips/stm32g491re/metadata.rs, the erase_size is "correctly" specified as 2048, and since there isn't a gap, the two banks are simply a contiguous memory block of 512K.
So, again for this chip specifically, using dualbank mode will work with the embassy Flash driver.

yeah it happens to work fine if your chip has 512K which is the max amount, because then there's no hole.


Remove unit in the measurement datum

Some measurement datum terms with unit may consider to remove as what we did in clinEpi display terminologies.

ID
Label

EUPATH:0000047
hemoglobin level in g per dL

EUPATH:0000026
land area measured in acres

EUPATH:0010316
stool volume in ml

EUPATH:0000382
total area measured in square meter

EUPATH:0000030
distance to health facility in km

EUPATH:0000753
specimen collection time in year

If we do this, we should deprecate EUPATH:0000753, since we have a term for specimen collection time from OBI.

EUPATH:0000382 is from one project (ICEMR Amazonia Peru) and refers to the total area of a land lot. Should we change the label & definition to reflect that? We could alternatively change it to 'area measurement datum' (which we would want to submit to IAO or OBI).

Discussed and agree that this term should capture the context of the lot area that the dwelling is on and so should relabel and define to be more specific. Make this a new term. Can leave or deprecate EUPATH:0000382 (not urgent).


[RFC] Implement build in chunks to improve import cost

RFC
Overview
Currently, our build generates one single file that embeds all components, fonts, and utils.
This affects the First Load JS size as it will load the entire library in the first render.
Proposal
Configure rollup to use manualChunks or another approach that splits the code in chunks that can be imported invididually.
Examples in other libraries

I haven't found yet an example

Who does this impact? Who is this for?
This will impact anyone that uses the lib by optimizing how the components are imported.
Additional context
Rollup documentation around code splitting and manual chunks.

WIP branch:
https://github.com/paulocfjunior/faency/tree/chore/chunk-build

I just created an issue to study alternatives to one of our (bad) rollup plugins:

https://github.com/traefik/faency/issues/338


[bug] Page contents are shifting when modal is opened [V3]

Link to minimal reproducible example
https://web3modal.com/
Summary

Open the landing page for v3 (https://web3modal.com/)
Click the "CONNECT WALLET" button at the top right
Notice how the page contents shift when the modal opens.

I am testing with a Chrome browser on the desktop.
List of related npm package versions
3.0.0-beta.3

+1, the modal pushes components to the top of the page in my application

it seems walletconnect styles are applied to the html tag, when I moved my css styles off the html tag it fixed the problem for me

Replicated, will be fixed in this release. To give more context when modal get's open we prevent page scrolling as it can lead to a lot of issues, especially on mobile (common practice with modals). Shift is caused by adjusting gutter of scroll-bar incorrectly.

released in beta.4

This still happening for anyone else or just me?
Page contents are still shifting on this website: https://web3modal.com/
@0xAsimetriq shall we re-open?

@pauldemarco Different kind of shift now, if your browser hides the scrollbar when scroll is dissabled and doesn't compensate for that, you'll see content shift. For this one, we can't fo anything. One that was removed is related to "scroll-gutter" css rule that we set before


[NTV-566] Kickstarter Framework Folder Reorganization

📲 What
Our three main frameworks are Kickstarter-Framework-iOS KsAPI and Library.
Right now the folder structure of Kickstarter-Framework-iOS isn’t organized around features/pages of the app.
It’s mostly organized around all views or all view models or all helper classes and their respective test classes.
Lets look through these frameworks and move files around in such a way that existing files in a framework are organized around a feature. Eg. “Comments” in Kickstarter-Framework-IOS would be a folder containing the datasource file, cell view files, view controller files, and so on. Everything inside Kickstarter-Framework-iOS that pertains to “Comments“ but is currently in a large “Views“ folder or “Controllers“ folder can be put together under a “Comments“ folder.
🤔 Why
So the idea is a feature can be worked on in one place within the entire project. Normally we would have view models in the Library framework, views in Kickstarter-Framework-iOS framework and models/networking in the KsApi framework.
This PR only addresses the folder structure of the Kickstarter-Framework-iOS framework not KsApi or Library (those are can be done separately).
🛠 How
Files moved out of top level Views Views/Cells, Controllers, Storyboards, Datasources folders into individual feature folders with the same folder structure as before, just within the context of the feature.
Before:
Datasource
  CommentsDataSource
  CommentsDataSourceTests
  CommentRepliesDataSource
  CommentRepliesDataSourceTests
  ...other datasource mixed in...
Views
    ViewRepliesView
    CommentComposerView
    CommentComposerViewTests
    CommentCellHeaderStackView
    CommentCellHeaderStackViewTests
    CommentTableViewFooterView
    CommentInputContainerView
    CommentInputTextView
    ...other views mixed in...
    Cells
      CommentCell
      CommentViewMoreRepliesFailedCell
      CommentPostFailedCell
      CommentRemovedCell
      CommentsErrorCell
      RootCommentCell
      ViewMoreRepliesCell
      ...other cells mixed in...
    Controllers
      CommentRepliesViewController
      CommentRepliesViewControllerTests
      CommentsViewController
      CommentsViewControllerTests
      CommentDialogViewController
      ...other controllers mixed in...
    Storyboards
      CommentsDialog
      ...other storyboards mixed in...

After:
Comments
  Storyboard
    CommentsDialog
  Controllers
    CommentRepliesViewController
    CommentRepliesViewControllerTests
    CommentsViewController
    CommentsViewControllerTests
    CommentDialogViewController
  Datasource
    CommentsDataSource
    CommentsDataSourceTests
    CommentRepliesDataSource
    CommentRepliesDataSourceTests
  Views
    ViewRepliesView
    CommentComposerView
    CommentComposerViewTests
    CommentCellHeaderStackView
    CommentCellHeaderStackViewTests
    CommentTableViewFooterView
    CommentInputContainerView
    CommentInputTextView
    Cells
      CommentCell
      CommentViewMoreRepliesFailedCell
      CommentPostFailedCell
      CommentRemovedCell
      CommentsErrorCell
      RootCommentCell
      ViewMoreRepliesCell

At high level this looks a little cleaner, and certainly easily navigated by someone who's only working on Comments.
There are still shared files maybe in one feature page that's referenced in another, or in the SharedViews folder. It's the responsibility of the developer to know that any reuse must still be considered before editing an existing file or creating a new one. Our test coverage will help there (modifying a file in one view that's captured by a snapshot test elsewhere will trigger a fail.)
No file names were changed, just moved into subdirectories. Also zero code changes took place, all changes occurred within the .xcodeproj file which is responsible for folder structure.
✅ Acceptance criteria

[x] Ensure tests pass
[x] Smoke test all features moved into their own folder. Do a regression test.

⏰ TODO

[x] Organize all files in the Kickstarter-Framework-iOS framework into respective feature folders.

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger

Ok, so one thing I want to follow up on is doing a bit more grouping for top level directories like "Messages", "MessageBanner", "MessageDialog" and "MessageThreads" --> put these into a directory called "Messages" if they are referencing each other and not referencing other files. Also we should test the creator flow and errored backing flow. At the moment we've got time constraints, and waiting on credentials and as this is a long standing technical issue we wanted to address, since it's 90% there, will merge.


Add linkedin svg logo

Source https://www.svgrepo.com/vectors/linkedin-big-logo/ with some custom modification to fit the theme

+1 for this as I just made the same addition in a fork of this repo.  Would be helpful to add in an upcoming release.

@Himanshu54  Thanks!


Fix:'remove env' wording

Issue #, if available:
#3140
Description of changes:
Change the wording depending on whether it's 'remove env' or 'delete' (remove all env)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #3142      +/-   ##
==========================================
+ Coverage   59.36%   61.86%   +2.49%
==========================================
  Files         254      223      -31
  Lines       12567    11406    -1161
  Branches     2419     2363      -56
==========================================
- Hits         7461     7056     -405
+ Misses       4792     4035     -757
- Partials      314      315       +1

Impacted Files
Coverage Δ

...ync-simulator/src/mqtt-server/abstract-listener.ts
0% <0%> (-85%)
:arrow_down:

...ages/amplify-appsync-simulator/src/server/index.ts
0% <0%> (-64.29%)
:arrow_down:

...amplify-appsync-simulator/src/data-loader/index.ts
0% <0%> (-60%)
:arrow_down:

...amplify-appsync-simulator/src/server/operations.ts
0% <0%> (-28.46%)
:arrow_down:

...plify-appsync-simulator/src/server/subscription.ts
0% <0%> (-26.82%)
:arrow_down:

...c-simulator/src/schema/directives/aws-subscribe.ts
0% <0%> (-26.67%)
:arrow_down:

...simulator/src/data-loader/dynamo-db/utils/index.ts
0% <0%> (-15.39%)
:arrow_down:

packages/amplify-cli/src/context-extensions.ts
25.69% <0%> (ø)
:arrow_up:

...codegen/src/codegen-config/AmplifyCodeGenConfig.js
22.72% <0%> (ø)
:arrow_up:

...lify-graphql-types-generator/src/compiler/index.ts
90.32% <0%> (ø)
:arrow_up:

... and 35 more

Continue to review full report at Codecov.

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

Will this be included in an upcoming release? I'm pretty wary of deleting an environment I used for testing, because the response from amplify env remove backup says that all environments will be deleted as of CLI v4.13.1.


Does not display components when building

According to the readme of the library, a component can be used to display notification toast messages and when using a component in a development environment it is displayed as follows

But when I build the project, and upload it to my production environment it looks like this

I don't understand why if the build is built without errors

I have the same issue


Hint & element text not being properly combined

Description
Hint set on an element without an accessibilityLabel, does not pull in descendant text to use as its label but instead announces the hint only.
This issue is covered in-depth by parent issue #30847, please check there for more details.
React Native version:
v 0.63

Just as a note here, the cause for this is that we are actually adding the hint as part of the contentDescription.  This is because Android has no "accessibilityHint" property like iOS does, and instead its hint system is related to specific actions.
React Native does let you set labels for default actions like click, which will announce the hint "Double tap to ", but since the accessibilityHint prop exists for iOS it is also sort-of supported on Android to have a quasi-iOS-like experience.
Honestly, I see two paths forward here.

Formally make accessibilityHint an iOS-only property, with documentation that states that if you want to set hints on Android, you should use the accessibilityActions prop.

Re-implement accessibilityHint on Android so that rather that concatenate into the contentDescription, it sets the toolTipText property on the AccessibilityNodeInfo (not on the view). This is the closest analog to iOS's hint that Android has, as the text is announced after the contentDescription rather than part of it. It will will not adhere to users preferences on whether they want hints disabled or not, and still has no pause before it like real hints have, but it's far closer than using the contentDescription directly.

Option 2. makes the most sense to me, as removing it from Android entirely will likely break accessibility in unexpected ways across Android.

commenting here to reference https://github.com/facebook/react-native/issues/26739, an existing issue that myself and several others have been hoping to get attention on. does it seem like the issue described here could be the cause of this issue?

@jychiao, I just looked through the attached issue, and I don't think it's directly related, but I do know the cause of the attached issue (which I will comment on that issue with).
Sorry for the delay on this, I work on Android accessibility, but have only recently started working with React Native, so I wasn't aware of many of these problems until recently :)

@jychiao, actually after reading through that whole issue, part of it is in fact related to this issue.  The main cause of the issue here is that accessibilityHint is using the contentDescription property on Android, as "accessibilityHint" is not a concept that is native to that platform.  Due to concatenating text into the contentDescription it causes both this issue, as well as the issue with accessibilityHint in the issue you referenced.
I'm writing up a more thorough explanation in that issue that will explain why it's broken, as well as propose a potential solution.

adding hint text with setTooltipText

https://user-images.githubusercontent.com/24992535/183646208-e074de4c-fc58-40ed-8369-1f5bf980343a.mp4


Random angle in particle seems to be re-randomized somewhere in particle life

Godot version
b7723a01d957e6492a3f20bce8a47d3559afe5c5 <- after RC1
System information
Fedora, RX 590
Issue description
Particles random angles seem to flicker, sometimes.
https://user-images.githubusercontent.com/7917475/218566882-60e552ec-5284-46bd-ac6c-45e6e1dc3d98.mp4
Steps to reproduce
I didn't find yet a way to repro reliably. But definitely random angle is the first step
Minimal reproduction project
See above

Closing as duplicate of https://github.com/godotengine/godot/issues/49767


V3 site page not found

Sorry about that. This is indeed an issue.

https://imagesloaded.desandro.com/v3 now redirects to https://github.com/desandro/imagesloaded/tree/v3.2.0#readme


Update docs to stable channels. Issue template.

Added some comments, but don't think my changes are required. Just trying to give @tvansteenburgh a starting point.

Thank you @hyperbolic2346!


tab autocomplete, exit/quit to exit

@ideoforms
as per your ticket https://github.com/ideoforms/AbletonOSC/issues/36
here's two alterations:

Exit / Quit will now cleanly exit the run-console.py
Tab will attempt to autocomplete what you've written in, and show you the handlers that are currently there.

the list is static generated, not dynamically generated - by just fetching all the add_handlers from all the code.

i think there's a way of making it dynamic instead of static.

the dynamic method ended up with a bunch of %s content. so i'm just dumping the manual there and seeing how big the list gets.

pushing a modification.


为啥这个代码里都是bug 跑chatglm的微调代码根本跑不起来

Describe the bug
Please provide a clear and concise description of what the bug is. If applicable, add screenshots to help explain your problem, especially for visualization related problems.
To Reproduce
Please provide a Minimal, Complete, and Verifiable example here. We hope we can simply copy/paste/run it. It is also nice to share a hosted runnable script (e.g. Google Colab), especially for hardware-related problems.
Describe your attempts

[ ] I checked the documentation and found no answer
[ ] I checked to make sure that this is not a duplicate issue

You should also provide code snippets you tried as a workaround, StackOverflow solution that you have walked through, or your best guess of the cause that you can't locate (e.g. cosmic radiation).
Context

OS [e.g. Windows 10, macOS 10.14]:
Hardware [e.g. CPU only, GTX 1080 Ti]:

如果发泄怨气，请自己写。


Not possible connect Sixfab IoT shield to raspberry pi through serial port

Hello,
I have since October 2018 a Sixfab IoT shield. I have been able to connect shield to raspberry pi with PPP, and get IP adress in Rasapberry Pi, being able to control remotely a robot connected to this raspberry pi through Sixfab shield. I am able to read with python sensors in shield, but when I try to send AT commands from raspberry pi using python to IoT shield to get RSRP, I am getting an error like one below:

from cellulariot import cellulariot
import time
node = cellulariot.CellularIoT()
Sixfab Cellular IoT Hat Class initialized!
node.sendATComm("ATE1","OK\r\n")
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/serial/serialposix.py", line 265, in open
self.fd = os.open(self.portstr, os.O_RDWR | os.O_NOCTTY | os.O_NONBLOCK)
OSError: [Errno 16] Device or resource busy: '/dev/ttyS0'

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.5/dist-packages/sixfab_cellulariot-1.0-py3.5.egg/cellulariot/cellulariot.py", line 152, in sendATComm
File "/usr/local/lib/python3.5/dist-packages/sixfab_cellulariot-1.0-py3.5.egg/cellulariot/cellulariot.py", line 142, in sendATCommOnce
File "/usr/lib/python3/dist-packages/serial/serialposix.py", line 268, in open
raise SerialException(msg.errno, "could not open port {}: {}".format(self._port, msg))
serial.serialutil.SerialException: [Errno 16] could not open port /dev/ttyS0: [Errno 16] Device or resource busy: '/dev/ttyS0'

quit(9
... quit()
File "", line 2
quit()
^
SyntaxError: invalid syntax

Now I am trying to connect Sixfab with Raspberry Pi through USB cable, using usb connection in IoT shield, and one out of four usb ports in raspberry pi, but I do not know how to get usb port in /dev directory of raspberry pi. I am able to connect IoT shield to windows computer through usb cable, and opening a putty serial console I can reach serial port of IoT shield and send AT commands and receive results from IoT shield. At this point I have connected Iot shield with raspberry pi only through usb cable, without insert Iot shield on 40 pins connection of raspberry pi.
I guess that I need drivers in raspberry pi in order to permit to Pi read serial port through usb, but following PDF Quectel_WCDMA&LTE_Linux_USB_Driver_User_Guide_V1.8.pdf is confusing for me since I do not know how to download kernel files.
Please, could you give me some clue about how to follow with this config and be able to connect Pi and Shield through usb serial cable?
Thank you for your help and best regards
Oscar

This product is crap and their support is even worse.I spent the good part of two months on my own time, at my own expense working with Saeed at Sixfab trying to get this POS to work and solve their issues with their CRAPTEL shitty hardware. I incurred out of pocket expenses that were never reimbursed.  We tried multiple antennas, different cell phone carriers. As a software developer with well over 25 years of experience I would say stay well clear of this rubbish!

Agreed mate, absolute rubbish!


Please follow Semantic Versioning in the future

https://semver.org/
Your recent update to version 2.1.0 breaks backwards compatibility by swapping the order of parameters to the constructor and raising an exception in existing code where no such exception was raised on 2.0.x versions of the library.
Per the semantic versioning guidelines, the "1" in 2.1.0 should imply that new features were added which are backwards compatible to previous 2.x.x releases. This breaking change should cause the version number to be upgraded to 3.0.0 as the API is not backwards compatible to a prior 2.x.x release such as 2.0.18

Sorry for the delay in responding and the issues with this update. Originally, I don't think the change was going to include that, but when it did, I forgot to increment the correct number. @charlesthk would you be ok with killing the 2.1.0 version on PyPi and uploading a 3.0.0 instead with appropriate changes?

Hi @stephenross, thanks, The v3.0.0 is live on PyPi.
I haven't deleted the v2.1.0 from PyPi to avoid current projects using it to break because of a missing version (which might resides in requirements.txt).


Update doc

Codecov Report

Merging #81 (1ea7e57) into master (6707ca2) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #81   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           18        18
  Lines         1471      1471
=========================================
  Hits          1471      1471

Impacted Files
Coverage Δ

triad/utils/class_extension.py
100.00% <ø> (ø)

Continue to review full report at Codecov.

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


comments not working properly if we use custom contextmenu.

Hi,
I am using comments with a custom context menu, this issue occurs only if we use a custom context menu.
the issue is, if you try to edit comments and click on the cancel button in the prompt section, comments are getting removed from the cell, I am just canceling the prompt popup, not comments.

i got the solution, i took code to jexcel.js, contextmenu comments code is not up to date in the website


Directory name specification using {owner_id}

I already use {owner_id} with --filename-pattern and would also like to use it with --dirname-pattern. This does not seem to work and the documentation doesn't mention this feature either.
Thanks

dirname_pattern merely defines where the profile picture and the id file are stored. I think we'd have to adjust or rethink and reimplement the find-renamed-profile-by-id logic if the profile id or other custom profile fields could be used in the dirname pattern. Actually there are efforts to use a sqlite3 database (#72) which might give us more flexibility in the future.
Does --dirname-pattern=. --filename-pattern={owner_id}/{date_utc}_UTC work for you?

This works perfectly. Storing the id file and profile picture in the folder above the {owner_id}-subfolder is ideal for me. Thank you for your help.


Spawn Error while cloning powerplatform repo to internal organization

Hi Everyone,
We are trying to use the power-platform action by cloning it internally to our organization.
While doing so we are facing issues while using the action.
Below is the error log that we are encountering.
Can anyone help us here on this?
Run albertsons/esgh-powerplatform/export-solution@v0
loading action yaml file: D:\actions-runner-prd_work_actions\albertsons\esgh-powerplatform\v0\export-solution\action.yml
::group::export-solution:
export-solution:
##[debug]command: D:\actions-runner-prd_work_actions\albertsons\esgh-powerplatform\v0\dist\pac\tools\pac.exe, first arg of 12: auth
Error: failed: spawn UNKNOWN
Error: failed: spawn UNKNOWN
##[debug]command: D:\actions-runner-prd_work_actions\albertsons\esgh-powerplatform\v0\dist\pac\tools\pac.exe, first arg of 2: auth
Error: failed: Error: spawn UNKNOWN
Error: failed: Error: spawn UNKNOWN
::endgroup::
##[debug]Node Action run completed with exit code 1
##[debug]Finishing: export-solution action

@amenaafreenn please check the yaml of your workflow: does its checkout step specify to enable git LFS?
Ensure the checkout step does have lfs: true
    steps:
    - uses: actions/checkout@v2
      with:
        lfs: true

sample

@amenaafreenn also, it appears you've cloned/forked our microsoft/powerplatform-actions repo into your own GH org, but are most likely missing the /dist folder.
Note we STRONGLY advise against cloning of our Power Platform GH Actions repo into your own GH org, as you'll miss out future updates, unless you regularly update your fork.
If you want or need to stick with a particular release of our GH Actions, we recommend you author your GH workflow to not use the floating @v0 or @latest tags after our tasks' names, but instead to pin to a particular version, e.g.:
    - name: create ephemeral test environment
      uses<EMAIL_ADDRESS> # pin to the May/June refresh

check the version tags under Releases: https://github.com/microsoft/powerplatform-actions/releases
On each monthly release we make, we do tag that release first with a unique version tag, e.g. v0.7.1 as above. That specific tag is never moved; even if we had to do a hot fix, we'd increase it to e.g. v0.7.2
The v0 and latest tags, OTOH, we always do move it to the latest released monthly refresh, e.g. right now v0 points at v0.8.1


Adds a canonicalised version of withSystemTempDirectory

This relates to a reported issue with stack that uses this library:
https://github.com/commercialhaskell/stack/issues/1017
And provides a preferred fix in place of a previous stack pull
request:
https://github.com/commercialhaskell/stack/pull/1019

Hey Rob, why did you close this?
I haven't had a chance to look into the details yet. As I understand, this is needed to fix a bug in stack. Should the new function always be preferred to the old one in order to avoid the bug? Or are there partocilar conditions when one wants to use this? It would be nice if the docs provided a bit of guidance on that.

BTW, @Fuuzetsu, do we have a new repository for temporary (that we have control over)? The PR should probably go there.

@feuerbach good point, we should create one. Do you want to do that?

Ok, done: https://github.com/feuerbach/temporary

@Fuuzetsu @feuerbach Hi, and thanks for taking a look. I was about to tweak this stack PR for inclusion into stack: https://github.com/commercialhaskell/stack/pull/1047 .
However, it would make more sense to have this PR be directed at the temporary library. One thing potentially holding that move back is whether @feuerbach has the hackage and stackage upload permissions (and perhaps the nod of approval from previous author at https://github.com/batterseapower/temporary ?). If you do have the upload rights for both, then I'll submit a PR in your direction, and then submit a PR to stack to use the canonicalised versions of withSystemTempDirectory and withTempDirectory from System.IO.Temp .

Yes, @Fuuzetsu and I are the new maintainers; we have hackage upload rights for temporary.
stackage doesn't need any special permissions; it uses hackage as the upstream.


Cleanup logging

Replace string concatenation in logging statements with parameterized messages.

Build failure is nothing to worry about, I need to fix the javadocs deploy to not try running on PRs. Aws token is not available to PR builds for security.

@pau101 sorry it looks like you haven't signed our CLA so I had to roll back this merge. Please re-open it and follow the CLA bot instructions and we will get it merged again ASAP.


paste cannot run in docker environment

amd64  on debian12
root@dev:/home/soft/paste# docker logs -f de0
/docker-entrypoint.sh: /docker-entrypoint.d/ is not empty, will attempt to perform configuration
/docker-entrypoint.sh: Looking for shell scripts in /docker-entrypoint.d/
/docker-entrypoint.sh: Launching /docker-entrypoint.d/10-listen-on-ipv6-by-default.sh
10-listen-on-ipv6-by-default.sh: info: Getting the checksum of /etc/nginx/conf.d/default.conf
10-listen-on-ipv6-by-default.sh: info: /etc/nginx/conf.d/default.conf differs from the packaged version
/docker-entrypoint.sh: Sourcing /docker-entrypoint.d/15-local-resolvers.envsh
/docker-entrypoint.sh: Launching /docker-entrypoint.d/20-envsubst-on-templates.sh
/docker-entrypoint.sh: Launching /docker-entrypoint.d/30-tune-worker-processes.sh
/docker-entrypoint.sh: Configuration complete; ready for start up
2023/09/09 15:52:37 [notice] 1#1: using the "epoll" event method
2023/09/09 15:52:37 [notice] 1#1: nginx/1.25.2
2023/09/09 15:52:37 [notice] 1#1: built by gcc 12.2.1 20220924 (Alpine 12.2.1_git20220924-r10)
2023/09/09 15:52:37 [notice] 1#1: OS: Linux 6.1.0-10-cloud-amd64
2023/09/09 15:52:37 [notice] 1#1: getrlimit(RLIMIT_NOFILE): 1048576:1048576
2023/09/09 15:52:37 [notice] 1#1: start worker processes
2023/09/09 15:52:37 [notice] 1#1: start worker process 29
2023/09/09 15:52:37 [notice] 1#1: start worker process 30
2023/09/09 15:52:37 [notice] 1#1: start worker process 31
2023/09/09 15:52:37 [notice] 1#1: start worker process 32
2023/09/09 15:52:37 [notice] 1#1: start worker process 33
2023/09/09 15:52:37 [notice] 1#1: start worker process 34
2023/09/09 15:52:37 [notice] 1#1: start worker process 35
2023/09/09 15:52:37 [notice] 1#1: start worker process 36
2023/09/09 15:52:37 [notice] 1#1: start worker process 37
2023/09/09 15:52:37 [notice] 1#1: start worker process 38
2023/09/09 15:52:37 [notice] 1#1: start worker process 39
2023/09/09 15:52:37 [notice] 1#1: start worker process 40
2023/09/09 15:52:37 [notice] 1#1: start worker process 41
2023/09/09 15:52:37 [notice] 1#1: start worker process 42
2023/09/09 15:52:37 [notice] 1#1: start worker process 43
2023/09/09 15:52:37 [notice] 1#1: start worker process 44

docker run -p 8080:8080 -d paste:latest

It seems that the project is no longer maintained.


Ability to zoom into d3 charts

With the addition of a sidenav bar, charts have less room on the screen. The charts are smaller and details are more difficult to see at full screen size on standard laptops. Although users can make the screen smaller so that charts appear in a wider, single column, users may not know to do this and it may be inconvenient.

Done.


1525 | Number of Good Ways to Split a String | C++

Title - Number of Good Ways to Split a String
What will change - Add solution for the problem 1525. Number of Good Ways to Split a String  in C++
Type of Issue -

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Can I be assigned this issue?


feat: add getRequestedCredentialsForProofRequest

The method getRequestedCredentialsForProofRequest was present in the API before the PPV2 code was merged but somehow got lost.
It has been restored in this PR

Codecov Report

Merging #1028 (8282df1) into 0.3.0-pre (82a17a3) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##           0.3.0-pre    #1028      +/-   ##
=============================================
- Coverage      88.23%   88.20%   -0.03%
=============================================
  Files            637      637
  Lines          15092    15096       +4
  Branches        2542     2542
=============================================
  Hits           13316    13316
- Misses          1678     1682       +4
  Partials          98       98

Impacted Files
Coverage Δ

packages/core/src/modules/proofs/ProofsApi.ts
83.14% <0.00%> (-1.92%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Comments resolved, a new e2e test added, yarn.lock corrected.


add values option to install

need by: https://github.com/infinyon/fluvio/issues/685

need by: https://github.com/infinyon/fluvio/issues/685


Update enable-sql-azure-hybrid-benefit.md

changed text and provided added additional clarifications. Placement policies apply to both hosts and the VM's running on the host, however the licensing benefit applies to the cores of host.

Docs Build status updates of commit d53735b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-vmware/enable-sql-azure-hybrid-benefit.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@jjaygbay1,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Incorrect data types on documentation for Region

The v2 documentation for Region has some wrong data types. The locations, pokedexes, and version_groups fields should all be list NamedAPIResource, not just NamedAPIResource.

Created PR #105


Cant start (java.lang.NoClassDefFoundError: javafx/application/Application) with Java greater 1.8

Hey,
I cant start the application with higher then Java 1.8. Using the final release.
Why is that so?
Nevertheless its a cool Tool!

Hey,
first of all, thank you for opening an issue and letting me know about it! Also thanks for the feedback! :)
Sorry for having you waiting for so long.
The reason, why you can't start the application with any version higher than Java 8 is that JavaFX is used.
Since Java 11, JavaFX was removed from the SDK, so you would have to add the module separately.
An example for the execution (via CLI) would be something like this:
java --module-path "path/to/javafx-sdk-17/lib" --add-modules=javafx.controls --add-modules=javafx.swing -jar "path/to/LernSoftware.jar"
You can download the required files at the OpenJavaFX Community Site.
If you need further support or when you have any more questions, feel free to reply to this issue here!
I will update the documentation, in the next few days.


First draft of Toxiproxy-based network interruption E2E test.

Issue #, if available:
Description of changes:
Added new ToxiProxy-based Network Interruption E2E test.
Testing performed:
Ran test.  Successful outcome.  Confirmed expected network interruptions were occurring with manual ToxiProxy CLI inspection.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cc @wanyufe

Codecov Report
Base: 33.92% // Head: 34.04% // Increases project coverage by +0.11% :tada:

Coverage data is based on head (1244d64) compared to base (734b333).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #193      +/-   ##
==========================================
+ Coverage   33.92%   34.04%   +0.11%
==========================================
  Files          43       43
  Lines        3873     3892      +19
==========================================
+ Hits         1314     1325      +11
- Misses       2378     2387       +9
+ Partials      181      180       -1

Impacted Files
Coverage Δ

controllers/cloudstackmachine_controller.go
57.64% <0.00%> (-0.59%)
:arrow_down:

pkg/cloud/instance.go
79.59% <0.00%> (+0.49%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

/lgtm


Update reactive-check condition in prepare release script

as right now we get:
Preparing the release ...

+ pushd /var/lib/jenkins/workspace/hibernate-search-release_7.1

/var/lib/jenkins/workspace/hibernate-search-release_7.1 /mnt/workdir/jenkins/workspace/hibernate-search-release_7.1

+ git config --local user.name 'Hibernate CI'

+ git config --local user.email<EMAIL_ADDRESS>
/mnt/workdir/jenkins/workspace/hibernate-search-release_7.1/hibernate-release-scripts/prepare-release.sh: line 59: syntax error near unexpected token `else'

+ cleanup

so maybe let's just print a message.. or simply change to elif [ "$PROJECT" != "reactive" ]; then ?

Ah yes, sorry, but I'm updating them anyway because they are not quite right for Hibernate Reactive

no worries 😃


ValueError: Input tensors to a UTime must come from 'tf.keras.Input'. Received: None (missing previous layer metadata)

This error occurs while evaluating (even when using the SEDF example dataset)

Hi,
Thanks for reporting this. Which version of TensorFlow do you use?
I will look into the issue in 1 or 2 weeks as I am preoccupied with other activities until then.
Cheers,
Mathias

Hi,
Thanks for your reply.
We are using TensorFlow 2.4.0

Just figured out, that this error only occures when using "ut evaluate" with the "--one_shot" argument. Without it works fine (at least does not crash).

This should be fixed in newer versions. Please try to pull the latest version and let me know if you still experience this issue :)


Added support for factory functions

Hello!
This addresses #10.
I've implemented a basic support for factory functions, added tests and updated the documentation.
Could you please take a look and tell me what you think?
I hope the next step will be to introduce factory support for decorators.

I hope the next step will be to introduce factory support for decorators.

yes it can be:
@Service({ name?: string,  factory?: (params?: any[]) => Function  })
class MyService {

}

@pleerock I've pushed some changes according to your comments. However, the last test is not working. I'm not sure what params do I need to pass to the registerService() in order to specify dependencies for the factory function. Could you help with that?
Also, if registerService() is a public API, shouldn't it use Reflect to get metadata of the factory function in order to resolve it's parameters automatically?

If you want to move (<any> Reflect).getMetadata("design:paramtypes", target) to container then go ahead, I don't see reason why it wont work there. This probably will solve your issue with params and how to get Engine instance in tests.

@pleerock I've made another change. Specifically, I've introduced another decorator for factory methods and added separate facility for factories registration. The test is now working correctly, and factory arguments are also properly injected. Could you please take a look and let me know if this implementation makes sense? I'm still looking for other solutions though.

@pleerock I've pushed some changes as we discussed:

I've introduced a common interface for service descriptor.
I've removed separate facilities for factories.

However, I now have the problem which I hope you will be able to help me to solve:
How do I get parameters of the factory function from inside the @Service decorator?
https://github.com/slavafomin/typedi/blob/master/src/decorators.ts#L17
Is it even possible to do so? Or parameters could only be retrieved from the decorator of the method itself? If so, we can't support simple functions as factories, only class methods (please correct me if I'm wrong). And we will still need to use a separate decorator for the factory method.

How do I get parameters of the factory function from inside the @Service decorator?
Is it even possible to do so? Or parameters could only be retrieved from the decorator of the method itself?

It should work we just need to move Reflect.getMetadata.... code to the container I think. Need to check it. Try it and if it won't work I'll take a look by myself.

@pleerock I've already tried and I can't get metadata from the factory function this way. Could you please take a look?
Considering the style guides I would suggest to add your requirements to the linting configuration, so other developers will be able to see errors inside of their IDEs. Also, are you basing this on your own preferences, or are you targeting some community-accepted/popular standard?

@pleerock Could you please also look into overloading of the @Service decorator to support both name string and descriptor object as an input parameter? I'm new to TS and I'm not sure how to correctly implement it.

Style guide is based on my rich experience. JavaScript community is large and multiopinion and you can find popular style guide, unlike php where community is much smaller and less active and there is mostly one style guide dominating. But I'm not sure if there is strong opinion for the things we were talking about because they are case-base. Its good to configure those options in lslint if they are supported.

@pleerock Could you please also look into overloading of the @Service decorator to support both name string and descriptor object as an input parameter? I'm new to TS and I'm not sure how to correctly implement it.

sure. please finish everything you left/need/want to do and I'll finish rest.

@pleerock I've reverted name based decorators in tests to use single string and also updated the README.
The tests are not working now. What needs to be done:
1). Introduce overloading for @Service decorator to support both single string and object calls, this should fix most of the tests.
2). Make service factory decorators work, we need to resolve factory function arguments somehow.
3). Make Container.registerService() work with factories, I don't know how to manually pass the arguments in test.
4). This is closely related to "2" and "3" above: I think it would be better to move arguments reflection code to the central service registration facilit, like we discussed (it will allow to register services both via decorators and via manual call to the container, no need to manually specify arguments like in "3").
I will be glad to assist further if you need my help. Thank you!

Okay, got you, thanks for so many things done. Nevermind on style guide notices, I can change it by myself. I'll work on it tomorrow.


Adding vfg.css breaks styleClasses

When I add the <style src="vue-form-generator/dist/vfg.css"> to my component that wraps VFG I am finding that the styleClasses breaks.
I suspect this is related to the css order and that styleClasses is being added first? Is there ea proper way to include it?

Can you provide a fiddle that reproduce this problem ? Thanks !

One possibility, it may be related to the specificity of you class. When you inspect the element, does your custom class are before those of VFG ? If this is the case, you may want to augment the specificity of your custom class by adding .vue-form-generator .form-control to your CSS.
/* before */
.my-class{
    background-color: red;
}

/* after */
.vue-form-generator .form-control.my-class{
    background-color: red;
}

It doesn't exist in the fiddle, So I guess where is the proper place to add <script>...vfg.css</script> ?
If I add it to my component that calls vfg, the css appear to overwrite my bootstrap.css and the col-md-6 etc widths get corrupted.

Have you tried my proposal ? Have you inspected your code ? What's your level in CSS ?
It may be related to the order in which you import it.
In this fiddle, I added bootstrap along vfg.css and nothing is broken.
Lastly, vfg.css doesn't contain col-md-6 and everything is under the .vue-form-generator so nothing outside should be contaminated.
Sorry, but I'm going to close the issue.
If you provide a fiddle or something with an example of this corruption that we can test then I will reopen. Because without this, it could be anything and maybe unrelated to VFG.
I simply can't help you with the level of information you provided.

Sorry I am coming back to this, thank you for the help.
This Fiddle shows the issue
I would expect to see 1 row with 3 fields in line at 1/3rd width because of the added:
styleClasses : "col-md-4",
Inspecting the field and disabling the width: 100% seems to fix issue. I have experimented with css order also.

Ok, I understand, now ! Thank you for the fiddle and all.
The only solution I found is to add that bit of CSS. It is a bit cumbersome, since you have to do it for every class you want to use, but it work.
.vue-form-generator .form-group.col-md-4 {
    width: 33.33333333%;
}

updated fiddle

That does not seem correct, surely everyone is experiencing this same issue, and if it is just me, then we should try and find out why. I think its only with the Width attribute
There are a lot of classes where its needed, since my forms are dynamic, they could have any bootstrap classes in them.

adding
    width: auto;     min-width: 0;     display:table;
seems to also work
[fiddle](fiddle: https://jsfiddle.net/uma78ekn/)

Changing veg.css and commenting out the width seems to fix this correctly.
.vue-form-generator .form-group {     display: inline-block;     vertical-align: top;     /*width: 100%;*/     margin-bottom: 1rem }

this bit of CSS should prevent that, but it is clearly not working how it is supposed to be.
@icebob Can we remove it ?
@DelfsEngineering You also have the choice of not including vfg.css.

I do need vfg.css for the validation and error classes though.

You can just copy and paste what you really need, and redo that part. It's literally one line to change...

Correct, I have copied the file and edited it locally (out of the modules folder) as a working interim.
Regards
Charles Delfs
www.DelfsEngineering.ca http://www.delfsengineering.ca/
Ph:<PHONE_NUMBER>

On Jul 31, 2017, at 7:18 PM, lionel-bijaoui<EMAIL_ADDRESS>wrote:
You can just copy and paste what you really need, and redo that part. It's literally one line to change...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/icebob/vue-form-generator/issues/227#issuecomment-319223478, or mute the thread https://github.com/notifications/unsubscribe-auth/AKsufWZxm_1FrTbp-cahIt65fnqa-y7Dks5sTmDEgaJpZM4N2fsH.

Would this be caused by processing the css as css and not sass?

icebob please change  in css .vue-form-generator .form-group to  .vue-form-generator .form-group:not([class*="col"])

Hi!
This seems to be quite an old issue, but I had it today and it seems to me that the fixes did not make their way into the CSS. Fixed by adding additional style as suggested above:
.form-group{display:inline-block;vertical-align:top;/width:100%/;margin-bottom:1rem}.vue-form-generator .form-group:not([class*=" col-"]){width:100%}.vue-form-generator
@icebob @lionel-bijaoui Could you make changes in vfg,css file please?
Regards

I am having this same issue as well.
Really disappointed this has not been addressed.

@dastiw1 @DontSeeSharp @averagejoe77 PR welcome !
Just a reminder, we are not paid to do this, we are not at your service, and this is an open source project.
Please stop expecting things, we are very busy on the next version of VFG and we only have so much time in our life.
I would prefer cooperation and thankfulness instead of this condescending attitude some have here.
They were no promise or engagement to fix this issue, and if this is such an annoyance for you, you can contribute and are super welcome to do so.

Closing - the suggested code modifications were added in 2.3.0


enable account_cache

What was wrong?
account_cache was declared but not used causing ~30% of performance degradation when accessing accounts.
How was it fixed?
Starting to use account_cache in get\set_account functions

This looks good to me. Should we include it in the planned release today/tomorrow? /cc @carver @pipermerriam

@cburgdorf I'd say :+1: on inclusion.


Fix Windows 10 box name

Since commit c2a80b2efc the final instead of the technical preview iso is used.

@joefitzgerald can you have a look at this? Do you still maintain this repository?

@joefitzgerald did you already have the time to look at this?


An Issue with an AWS EC2 instance when connecting it to WebSocket (socketo.me)

As I tried to run the chat app from localhost connected to MySQL database which had been coded with PHP via WebSocket it was successful.
Also when I tried to run from the PuTTY terminal logged into SSH credentials, it was displaying as Server Started with the port# 8080
ubuntu@ec3-193-123-96:/home/admin/web/ec3-193-123-96.eu-central-1.compute.amazonaws.com/public_html/application/libraries/server$ php websocket_server.php
PHP Fatal error: Uncaught React\Socket\ConnectionException: Could not bind to tcp://<IP_ADDRESS>:8080: Address already in use in /home/admin/web/ec3-193-123-96.eu-central-1.compute.amazonaws.com/public_html/application/libraries/vendor/react/socket/src/Server.php:29
Stack trace:
#0 /home/admin/web/ec3-193-123-96.eu-central-1.compute.amazonaws.com/public_html/application/libraries/vendor/cboden/ratchet/src/Ratchet/Server/IoServer.php(70): React\Socket\Server->listen(8080, '<IP_ADDRESS>')
#1 /home/admin/web/ec3-193-123-96.eu-central-1.compute.amazonaws.com/public_html/application/libraries/server/websocket_server.php(121): Ratchet\Server\IoServer::factory(Object(Ratchet\Http\HttpServer), 8080)
#2 {main}
thrown in /home/admin/web/ec3-193-123-96.eu-central-1.compute.amazonaws.com/public_html/application/libraries/vendor/react/socket/src/Server.php on line 29
ubuntu@ec3-193-123-96:/home/admin/web/ec3-193-123-96.eu-central-1.compute.amazonaws.com/public_html/application/libraries/server$

So I tried to change the port#8080 to port# 8282, it was successful
ubuntu@ec3-193-123-96:/home/admin/web/ec3-193-123-96.eu-central-1.compute.amazonaws.com/public_html/application/libraries/server$ php websocket_server.php

Keeping the shell script running, open a couple of web browser windows, and open a Javascript console or a page with the following Javascript:
var conn = new WebSocket('ws://<IP_ADDRESS>:8282');
conn.onopen = function(e) {
    console.log("Connection established!");
};

conn.onmessage = function(e) {
    console.log(e.data);
};

From the browser console results:

WebSocket connection to 'ws://<IP_ADDRESS>:8282/' failed: Error in
connection establishment: net::ERR_CONNECTION_TIMED_OUT

I even tried to assign Public IP and Private IP, but no good it resulted in the same old result?
How am I suppose to resolve/overcome while connecting it from the WebSocket especially from the hosted server with the domain name such as
http://ec3-193-123-96.eu-central-1.compute.amazonaws.com/
var conn = new WebSocket('ws://localhost:8383');

From the Security Group

Under Inbound tab

Under Outbound tab

How do we need to set up Virtual Server to run your own services in an AWS EC2 instance?

@nishanth6 I am closing this issue because it is old. I don't run Ratchet on EC2, so I can't help. If you did figure it out please post so it can help others.


python3Packages.ipykernel: fix build on darwin

Motivation for this change
Things done

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

[ ] NixOS
[x] 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 python37Packages.ipykernel
@GrahamcOfBorg build python38Packages.ipykernel

the 2 commits should probably be squashed

Done!

nixpkgs doesn't have a python3.4 interpreter, so the disable change is largely a "noop"

Yes, I thought of it like more of a declaration. I updated the commit to remove the condition instead of updating it.

@GrahamcOfBorg build  python3Packages.ipykernel
@GrahamcOfBorg build  pythonPackages.ipykernel
@GrahamcOfBorg build  python38Packages.ipykernel


consistency with matlabs sapm api

The Matlab project's sapm function signature is:
pvl_sapm(Module, Ee, celltemp)
while ours is
sapm(module, poa_direct, poa_diffuse, temp_cell, airmass_absolute, aoi)
The Matlab project has used that signature since at least version 1.1 (the oldest that I have). The Python signature started as
pvl_sapm(Module,Eb,Ediff,Tcell,AM,AOI)
in commit e74644b and I changed it to the current signature in 5f9bd7c1. If I'm reading the original commit correctly (specifically line 101), I think the units of the irradiance should be W/m**2 despite the docstring's claim of suns.
So that's the current status and history, and I will leave it to you all to decide how you want to go forward.

We chose to use Ee (effective irradiance) as input to SAPM in Matlab so that we had flexibility to replace the reflection and spectral mismatch functions.  Ee can be computed in a variety of ways from POA_direct and POA_diffuse. I would encourage doing similarly in python.

It appears to me that the Matlab library relies on the user to implement the calculations below instead of providing functions such as pvl_sapm_spectral_correction, pvl_sapm_aoi, and pvl_sapm_effective_irradiance. Correct?
F1 = max(0,polyval(ModuleParameters.a,AMa)); %Spectral loss function
F2 = max(0,polyval(ModuleParameters.b,AOI)); % Angle of incidence loss function
Ee = F1.*((Eb.*F2+ModuleParameters.fd.*Ediff)/E0)*SF; %Effective irradiance

Perhaps the Python library should adopt the pvl_sapm api and add some simple functions to standardize the SAPM algorithms. I'm trying to avoid doing any real calculations in ModelChain, so I'd need new functions/methods to call.

Yes, that’s correct.  We felt that approach would offer more flexibility to a modeler who preferred the Martin and Ruiz AOI model, for example, over the Sandia polynomial.  We take the same approach with the single diode models, i.e., the input to these functions is Ee.
From: Will Holmgren<EMAIL_ADDRESS>Sent: Friday, June 24, 2016 11:07 AM
To: pvlib/pvlib-python<EMAIL_ADDRESS>Cc: Hansen, Clifford W<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: [EXTERNAL] Re: [pvlib/pvlib-python] consistency with matlabs sapm api (#198)
It appears to me that the Matlab library relies on the user to implement the calculations below instead of providing functions such as pvl_sapm_spectral_correction, pvl_sapm_aoi, and pvl_sapm_effective_irradiance. Correct?
F1 = max(0,polyval(ModuleParameters.a,AMa)); %Spectral loss function
F2 = max(0,polyval(ModuleParameters.b,AOI)); % Angle of incidence loss function
Ee = F1.*((Eb.*F2+ModuleParameters.fd.*Ediff)/E0)*SF; %Effective irradiance
Perhaps the Python library should adopt the pvl_sapm api and add some simple functions to standardize the SAPM algorithms. I'm trying to avoid doing any real calculations in ModelChain, so I'd need new functions/methods to call.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/pvlib/pvlib-python/issues/198#issuecomment-228402963, or mute the threadhttps://github.com/notifications/unsubscribe/AFJNL3w1FkWrTsWYwpoiHNJCt6Kvq3kAks5qPA6YgaJpZM4I9C8b.

Ok, I'm sold. While I liked the ease of use with sapm taking care of that for me, I disliked the inconsistency with the single diode models. Now that we have PVSystem and ModelChain for the ease of use issues, we should keep the functions simple and consistent.
For what it's worth, I pushed a couple of new-but-old https://github.com/pvlib/pvlib-python/tags to help us keep track of why things are the way that they are and thus better understand the implications of changing things.

Another point on consistency:
pvl_sapm takes Ee in units of suns.
pvl_calcparams_desoto takes S in units of W/m^2.
pvl_calcparams_PVsyst takes Ee in units of W/m^2.
pvl_calcparams_CEC takes S in units of W/m^2.
I'd prefer if the Python functions all take effective_irradiance in units of W/m^2. The discussion in #195 makes me think e_poa_transmitted would work too.


Remove version: BlenderFoundation.Blender.LTS version 2.93.3

No longer part of LTS
Microsoft Reviewers: Open in CodeFlow

URL: https://download.blender.org/release/Blender2.93/blender-2.93.3-windows-x64.msi
OK
(Automated message - build 887)


Make 'timeSpent' Info in Trac App Integration Use Less Space

Something like this would be good enough too:
# TimeSpent
* My Protocol 2 (08:30-15:00) - EWT: 5h
* My Protocol 1 (08:30-15:00) - EWT: 5h

EWT: Effective working time (i.e. w/o breaks)

pushed to master (as described).
✅ CI
✅ readme updated


Memory leak in PowerToys.exe

ℹ Computer information

PowerToys version: 0.27
PowerToy Utility: FancyZones (suspected)
Running PowerToys as Admin: Yes
Windows build number: [run "winver"] 19042.630

📝 Provide detailed reproduction steps (if any)

Create custom zone with >40 squares ( 5 monitors, different DPIs, only one screen has over 40 zones, the rest have <15 )
Use computer as normal (~4-6h)

✔ Expected result
Normal memory use
❌ Actual result
~32GB private, ~2GB working memory used. Suspected trouble when Chrome started flickering and popping up "out of memory" error boxes on a machine with 32GB ram in 2020. :)
📷 Screenshots

I have a full dump and a minidump if anyone would find it helpful, I may take a peek myself what could possibly have taken up 34GB ram...

did this style config work prior to .27?

@whippc
we need repro steps since, creating and using a custom layout with 50 zones doesn't show any memory leak, I would expect a quite significant one to reach 34GB in a few hours.
We need the current FZ settings and also detailed information of all the layouts in use.

I'll need to pull the configs later today. Also, I never tested this before this version. That was the only thing I changed after the update so I assumed that must have caused it, it got a bit slow after editing the zones manually, but I didn't think much of it until later in the day. I gave you all the steps I took to reproduce, so maybe there's something wrong with my config file.
I have 2 dump files I took from Process Explorer before I killed the exe, a mini dump and a 34gb full dump, which I'm hoping to poke at later as well, out of curiosity more than anything. I read we shouldn't post them but send them to Microsoft directly somehow, does anyone know where I can send the dump(s)?
Sent from my phone

From: Enrico Giordani<EMAIL_ADDRESS>Sent: Friday, December 4, 2020 3:36:06 AM
To: microsoft/PowerToys<EMAIL_ADDRESS>Cc: whippc<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: External -Re: [microsoft/PowerToys] After editing fancyzones with many squares on grid, memory usage hit 34GB before choking Chrome out of memory (v0.27) (#8378)
This Email originated outside of Collier County Public Schools and may contain a link. Please do not click links you were not specifically expecting, even from known senders.

@whippchttps://github.com/whippc
we need repro steps since, creating and using a custom layout with 50 zones doesn't show any memory leak, I would expect a quite significant one to reach 34GB in a few hours.
We need the current FZ settings and also detailed information of all the layouts in use.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/8378#issuecomment-738645115, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIPOOJEPD5FF3NDQM4LOZMTSTCNPNANCNFSM4UMO6Z3A.

@whippc
for the minidump you can try to zip it and if it's not too big, GitHub will allow to attach it to a comment.

I am seeing something similar. In particular, I have a 4K monitor with 4 zones, and a second monitor with one zone. Memory just slowly climbs. But I want to double check this is the same problem: I see "commit" size rising. I rebooted last night and I'm at 2GB now (after a few days it was up above 9). But the "Memory" (or what is in RAM) is quite small. So this is something that is allcoated and no longer used (classic leak thing). So it isn't putting pressure on my RAM, but likely is putting pressure on my page file. :-)

I think so too, opening the full dump in a hex editor shows the last 99% of memory is all 0s. Null, nothing there. My setup is similar, 3 4ks and 2 1080p monitors across both the GPU and onboard graphics, I’m also using littlebigmouse to manage mouse movement across screens. Do you use anything similar, or have any other oddities with your setup, just in case we find anything we’re using in common that may have something to do with this somehow.
From: Gordon Watts<EMAIL_ADDRESS>Sent: Friday, December 4, 2020 2:22 PM
To: microsoft/PowerToys<EMAIL_ADDRESS>Cc: Whipple, Chris (Christopher<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: External -Re: [microsoft/PowerToys] After editing fancyzones with many squares on grid, memory usage hit 34GB before choking Chrome out of memory (v0.27) (#8378)
This Email originated outside of Collier County Public Schools and may contain a link. Please do not click links you were not specifically expecting, even from known senders.

I am seeing something similar. In particular, I have a 4K monitor with 4 zones, and a second monitor with one zone. Memory just slowly climbs. But I want to double check this is the same problem: I see "commit" size rising. I rebooted last night and I'm at 2GB now (after a few days it was up above 9). But the "Memory" (or what is in RAM) is quite small. So this is something that is allcoated and no longer used (classic leak thing). So it isn't putting pressure on my RAM, but likely is putting pressure on my page file. :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/8378#issuecomment-738969997, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIPOOJEOYA6V4CEHB3FXI3LSTEZE3ANCNFSM4UMO6Z3A.

PowerToys_memoryusage_mini_12-3-2020.zip

I think PowerToys is about the only "odd" thing about my setup. And I repaved my SurfacePro6 about a week ago, so it is a fairly clean machine. But I do have 2 monitors as well (one is external, the other is the SurfacePro6's screen).

@enricogior, marking this as 0.29 so we can start investigating it

This happened twice on my computer. The first time it took ~10GB working set and the second time it took ~20GB commit size, after using computer for a long time. I don't have many grid squares (actually I didn't use powertoys during that time), and this never happened on v0.25.

Note for the investigation: a memory leak of this size cannot be the result of a user action repeated multiple times. For example, showing the zones cannot be the cause because in a few hours a user might only activate the zones a very limited amount of times. If the leak was in that code, it would have to be in the order of tens if not hundreds of MB for each action.
The possible causes may be:

a recurring code that is not triggered by a user action
a user action that enter a infinite loop

@whippc @gordonwatts @Gallium70
can you please check the size of the %localappdata%\Microsoft\PowerToys folder?
Can you also check the log files in %localappdata%\Microsoft\PowerToys\RunnerLogs?
Thanks.

My PowerToys process has been running for about 24 hours, perhaps a bit more. Memory is at a "reasonable" 300MB, and the commit size is at ~ 6GB.
PowerToys folder: 128 KB
PowerToys\RunnerLogs: 2 zero-length files

Some quick experiments:

Holding down the Windows key and the arrow key and little the key repeat so a window flashes across all zones in my two-monitor setup caused a 20KB allocation for the first move, but then was rock solid for almost 60 seconds of testing.
Alt-space, which calls up the launcher, repeatedly, didn't seem to cause it directly.

I'm playing with deskotp switching with the crtl-Win-left/right keys further.

Restarted PowerToys
Monitored commit size in the Details pane of task manager. At start up ram was about 8 MB, and Commit was about 150 MB
Switched quickly through 3 desktops, and then back to the first desktop
The RAM size had not changed at all - 8 MB, and commit was up to 600 MB.

I don't claim to be an expert on memory models, but it is as if the memory was reserved, but never touched - as the RAM never ticked over in that.

@gordonwatts
thanks for investigating the problem.
I also see a 30 MB increase for each new VD, but only upon creation, not every time VDs are switched.

Try letting it sit for a while, perhaps? After it sleeps? I seem to see it happening there as well, not just when it starts.
I've got some work to do - I'll try running the data collector for a while and see if it captures anything interesting.

Ok - I did some more testing - the act of "going to sleep" seems to trigger a re-allocation. To repro this is what I have to do:

Start power toys, monitor the "Commit Size" or "page file size" depending on the tool (perf mon or task manager)
Switch to a second desktop, switch back
Note that Commit size has gone up.
Switch to the same second desktop, switch back
Note that the Commit size is not going up.
Put the computer to sleep (I have a surface pro 6) usign the start memnu, clicking the power button, and selecting sleep
Wake up, note the Commit size has already increased
Switch to a new desktop, note that it has increased again.

I didn't know how to monitor sleep in the perf monitor, sorry. But, I put the machine to sleep after the first large step (second step). When it wakes up it takes that small step. There is a longer plateau, and then I switched desktop and it takes another step (and then later I went one switch further and there was another step).

It looks like each time you visit a new desktop you get an allocation
If you sleep, the allocations are reset
When you wake, PowerToys makes an immediate allocation for the desktop the machine wakes on
Each subsequent new desktop you switch to costs you again.

The above seems quite repeatable at this point on my machine.

@whippc @gordonwatts @Gallium70
what GPU are you using?

Oh! That is very odd! I'm using a SurfacePro6 - so the GPU is part of the Intel chip. So, also an Intel GPU! I have another machine that has NVIDIA on it, but it is 1000's of miles away and I won't be back near it until January.

My desktop has an Nvidia GTX 1060 6gb, with Studio Driver 456.71 (I want to say EVGA but I’d have to look, out of office at the moment), but it's worth noting I have my motherboard set to allow both that card and the onboard intel graphics, cpu is an i7 6700k, mobo is Asus z170-a.

Thanks for finding the fix! Looking forward to it being released!

@whippc @gordonwatts @Gallium70
if for you it's acceptable to install an unsigned, I've attached an unsigned build that contain the fix for this problem if you want to test it. We are planning to release it in 0.27.1 in the next couple of days.
Thanks.
PowerToysSetup-0.0.1-x64-unsigned.zip

@whippc @gordonwatts @Gallium70
what GPU are you using?
On Intel GPU I can reproduce it.
On AMD or nVidia GPU I cannot reproduce it.

I'm using AMD Vega 8 GPU, integrated with a AMD 4800U CPU.

@whippc @gordonwatts @Gallium70
can you please check the size of the %localappdata%\Microsoft\PowerToys folder?
Can you also check the log files in %localappdata%\Microsoft\PowerToys\RunnerLogs?
Thanks.

The PowerToys folder inside AppData takes ~1.2MB, and RunnerLogs only contains several empty files.

@whippc @gordonwatts @Gallium70
if for you it's acceptable to install an unsigned, I've attached an unsigned build that contain the fix for this problem if you want to test it. We are planning to release it in 0.27.1 in the next couple of days.
Thanks.
PowerToysSetup-0.0.1-x64-unsigned.zip

I'll try this build. Thanks for the fix in advance.

I've installed the .zip file above.

Upon start up it was taking 8MB RAM, and the commit was 62 MB.
Switching back and forth to 4 desktops, and 9MB RAM, and commit was 215MB
After sleep and wake, 91MB of RAM and 94 MB of commit
Switching back and forth to 4 desktops, 91 MB of RAM and 250 MB of commit
Another sleep, after wake, 91 MB of RAM and 94 MB of commit

It appears this is fixed for me. I'll track this for a day and report back if I see anything accumulate over time. Certainly the repo recipe I had above that was effected with the old version is not causing a memory leak this time!
Thanks so much for this work!!
P.S. when a new release comes out will I need to uninstall by hand, or will it auto-detected?

@gordonwatts
thanks for testing it.

when a new release comes out will I need to uninstall by hand, or will it auto-detected?

The 0.0.1 build doesn't auto update, I'll post here the link to the 0.27.1 installer.

@whippc @gordonwatts @Gallium70
if for you it's acceptable to install an unsigned, I've attached an unsigned build that contain the fix for this problem if you want to test it. We are planning to release it in 0.27.1 in the next couple of days.
Thanks.
EDIT: you need to uninstall 0.27 before installing the unsigned build.
PowerToysSetup-0.0.1-x64-unsigned.zip

Uninstall

Will uninstalling remove all zones created? I have four monitors * 8+ Vdesktops. Recreating the zones would be really painful.
If removed upon uninstall, how do I keep a backup? @enricogior

So far so good for me too! It's been running from about 10AM today until about 2PM, and so far it's only hit about 200KB total memory usage, vs last week's 32GB in almost as much time.

@crimsonvspurple

Will uninstalling remove all zones created?

No, all settings remain untouched.

Looks ok so far. Thank you for the fix! I'll report again after 12 hours.

I've openned it up for over 10 hrs and it seems that the fix works for me.

Thanks a lot for the fix!

15 hours later:

No crash. No high RAM/VRAM. 💯

24 hour of use and it looks great! Thanks for this fix!!

Fixed in 0.27.1.  https://github.com/microsoft/PowerToys/releases/tag/v0.27.1

I've openned it up for over 10 hrs and it seems that the fix works for me.  Thanks a lot for the fix!

Please tell me, what is the name of the software in the screenshot? 😁

It's Process Explorer from Sysinternals.


Unified the use of character and string sequences.

Thought this makes the intent a little clearer and makes the code read a little easier.

Is there any strong reason to go one way or the other? Expressions of the form foo << 'a' and foo << "a" are both pretty common in the Swift codebase (904 and 880 instances, respectively).

@cwillmor Either way will work, but I was just thinking in terms of readability and making syntax usage uniform.
If the code is going to use single quote literals in one place and single characters written as double quote literals in another place its better perhaps to use one convention or the other.

Printing a char is theoretically cheaper than printing a string, at least, though I imagine LLVM optimizes away the difference in many case.

@cwillmor @jckarter Thanks for commenting!
Would there be any drawback to doing this change?

This is a legit (nano) optimization, merged.  Thanks!


User's manual data desensitization section parameter error

Question
When using version 4.0.0-RC1, the configuration parameters of the data desensitization part of SpringBoot cannot be started, and the latest version of the document can only use version 4.0.0-RC2. Please add a compatibility note for different versions.

SpringBoot use 2.1,x version

It is suggested to add a full configuration parameter introduction of different versions, some of which are not fully annotated according to the chapter.

Can you show some configuration?

Can you show some configuration?

I have find anawer,think you.
must to use last version ,old version is error by 4.0.0-RC1,you can try ti by youselef


Linters failing for pushes to release/1.7 and release/1.6

It looks like the linter check is failing for pushes to release/1.7 and release/1.6 since https://github.com/containerd/containerd/pull/10575 (and https://github.com/containerd/containerd/pull/10574).  My suspicion is that the only-new-issues: true configuration for golangci/golangci-lint-action@v4 only works on pull requests and not on pushes to the branch.

Thanks @zouyee!


Include hashid in individual models instead of globally

The current implementation includes the Hashid functionality into ActiveRecord::Base. I would like to instead use an explicit include any any models where needed.

Having this same thought myself, wanting to drop this into a sizable application and feels like hitting AR::Base risks breaking things elsewhere in the app.

This has been implemented and is available in the latest release.


[TOSA] Separate left shifting in SQUARED_DIFFERENCE to avoid precision loss

Hi,
This PR moves the left shifting in the tfl.squared_difference TFL -> TOSA legalization in a separate tosa.rescale to avoid any precision loss which could create non bit-exact results.

Hi @Tessil Can you please resolve conflicts? Thank you!

@gbaned Done, thanks.

Hi @jpienaar Can you please review this PR ? Thank you!

Hi @jpienaar Can you please review this PR ? Thank you!

Hi @rdzhabarov Can you please review this PR ? Thank you!

Hi @jpienaar Can you please review this PR ? Thank you!

Hi @jpienaar Can you please review this PR ? Thank you!

Hi @jpienaar Can you please review this PR ? Thank you!

Hi @jpienaar , Can you please review this PR ? Thank you!

Hi @jpienaar , Can you please review this PR ? Thank you!

Hi @jpienaar , Can you please review this PR ? Thank you!

Hi @jpienaar , Can you please review this PR ? Thank you!

Hi @jpienaar , Can you please review this PR ? Thank you!

Hi @jpienaar , Can you please review this PR ? Thank you!


each deprecated in Modules Line 83

Hello,
after Update to PHP 7.2 I become this Message.
Line 83: (is_array($module)) ? list($module, $params) = each($module) : $params = NULL;
How can I resolve this Problem?

Ok, after study the PHP-Manual i solved it.
(is_array($module)) ? list($module, $params) = [key($module), current($module)] : $params = NULL;

may i know which line in specific, sorry for the delay

@alzen8work
https://github.com/alzen8work/CodeIgniter-HMVC/blob/803c6d925d70c3f88cf8d139ccf1424333fd3929/application/third_party/MX/Modules.php#L83


Escape enum members names "nan" in TS as it is an invalid identifier

ts-node in the CI pipeline picked that up in my previous PR. Not sure why this didn't pop up before :shrug:

The CI / Run examples step is failing because the prometheus query schema has been removed from grafana and is no longer published in the next folder of the kind-registry :|
I'll figure out something to work around that in another PR


Transpile only the given file

I need to transpile only the given file and not its imported files. I could not find an option to do so. It seems it transpile all depencies.

Thank you @alan-khosro for reporting this issue. It is a duplicate of https://github.com/denoland/deno_emit/issues/133.
(cc @dsherret to close it)


Fix tests by downgrading colored

cheat.sh stopped highlighting headers two weeks ago https://github.com/chubin/cheat.sh/actions/runs/1393863906

@chubin I hope you're getting all those notifications and track what is going on. )


Service status is "active (exited)"; website not working

After using Uchiwa successfully for a long time, the current service status is:
● uchiwa.service - LSB: Uchiwa, a Sensu dashboard.
   Loaded: loaded (/etc/init.d/uchiwa; bad; vendor preset: enabled)
   Active: active (exited) since Fri 2017-11-17 00:18:17 UTC; 4s ago
     Docs: man:systemd-sysv-generator(8)
  Process: 16059 ExecStop=/etc/init.d/uchiwa stop (code=exited, status=0/SUCCESS)
  Process: 16101 ExecStart=/etc/init.d/uchiwa start (code=exited, status=0/SUCCESS)

Nov 17 00:18:17 sensu-server systemd[1]: Starting LSB: Uchiwa, a Sensu dashboard....
Nov 17 00:18:17 sensu-server uchiwa[16101]: uchiwa started.
Nov 17 00:18:17 sensu-server su[16104]: Successful su for uchiwa by root
Nov 17 00:18:17 sensu-server systemd[1]: Started LSB: Uchiwa, a Sensu dashboard..
Nov 17 00:18:17 sensu-server su[16104]: + ??? root:uchiwa
Nov 17 00:18:17 sensu-server su[16104]: pam_unix(su:session): session opened for user uchiwa by (uid=0)

I looked at the man page above, but wasn't sure what needed to be done
Your Environment

Uchiwa version used: 0.25.2-1
Sensu version used: 0.29.0-11
Operating System and version: Ubuntu 16.04 LTS

Hi @RavDanny,
Have you verified the log files (/var/log/uchiwa.log & /var/log/uchiwa.err)? That should provide us with more insight on what's going on.
Thanks

Hi @palourde,
Not much help, unfortunately. /var/log/uchiwa.err is empty and /var/log/uchiwa.log just contains lines like this: {"timestamp":"2017-11-30T22:08:17.129966543Z","level":"info","message":"Updating the datacenter Sensu"}.
Thanks

@RavDanny This log entry tells me that Uchiwa is still running and continues to query the Sensu APIs. Are you still unable to access the dashboard? And if you restart the service, does it work?

Im facing the same issue. Any updates, @palourde ??

same problem :(


ref(*): sync store changes from upstream

Codecov Report

Merging #13 into master will increase coverage by 1.98%.
The diff coverage is 74.04%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   48.15%   50.14%   +1.98%
==========================================
  Files          26       27       +1
  Lines        1680     1751      +71
==========================================
+ Hits          809      878      +69
- Misses        756      762       +6
+ Partials      115      111       -4

Impacted Files
Coverage Δ

internal/ingress/controller/store/backend_ssl.go
42.04% <50%> (+4.79%)
:arrow_up:

internal/ingress/controller/store/store.go
51.32% <62.96%> (+1.89%)
:arrow_up:

internal/ingress/controller/store/objectref.go
93.75% <93.75%> (ø)

internal/ingress/controller/store/ingress.go
71.42% <0%> (+14.28%)
:arrow_up:

Continue to review full report at Codecov.

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


Add support for URI Templates

As part of the Request Template support, it would be nice to also include a URI template seeing as most requests to a URI come from the same base URI.
http://tools.ietf.org/html/draft-gregorio-uritemplate-04

URI Templates provide a mechanism for abstracting a space of resource
identifiers such that the variable parts can be easily identified and
described.  URI templates can have many uses, including discovery of
available services, configuring resource mappings, defining computed
links, specifying interfaces, and other forms of programmatic
interaction with resources.  For example, the above resources could
be described by the following URI templates:
 http://example.com/~{username}/
 http://example.com/dictionary/{term:1}/{term}
 http://example.com/search{?q,lang}

We define the following terms:

expression - The text between '{' and '}', including the enclosing braces, as defined in Section 2.
expansion - The string result obtained from a template expression after processing it according to its expression type, list of variable names, and value modifiers, as defined in Section 3.
template processor - A program or library that, given a URI Template and a set of variables with values, transforms the template string into a URI-reference by parsing the template for expressions and substituting each one with its corresponding expansion.


Use process_time in test

And comment it at the time as it is not used.

This change has been incorporated in #1248, thanks!


org.joda.time.LocalDate isn't serializable

It's common to use Joda time because Apache Beam uses it instead of java time. Kryo isn't correctly serializing LocalDate, and there no way to register custom coder.
See: https://github.com/spotify/scio/pull/568

fixed in https://github.com/spotify/flo/pull/178


installApkRemotely to parse correct capabilities from opts

Per my previous PR I tested the new release within Appium and I found that my original change is defective
ref: #161
The opts values for localApkPath and pkg parsed in android-helpers.js : line 257 are referred incorrectly.
The tests for this PR passed, but when using official capability arguments passed from appium the appium-android-driver will fail.
I have updated use of variables localApkPath and pkg to their respective capabilities: app and appPackage.

good catch. LGTM


Add transaction mode for inserting and removing items in list view

Hi,
By definition, safe code should not throw while destruction. So by adding
noexception, it will call std::terminate, like you said, and std::abort
finally. The ideia is, the application will crash and should be
investigated...
This was the first time I did a code review in the GitHub, not in the work,
I mean. Just randomly looked for CPP open pr while waiting for my wife to
arrive from her yoga class.
I could improve the CPP code, which is fine, the little bit I've seen. But
could apply some best practices and so on.
But I really don't have experience building with vcpkg and windows.
Do you know if I can build it on Linux?
Thanks and BR,
Luiz
On Thu, Oct 6, 2022, 22:19 Reupen Shah @.***> wrote:

@.**** commented on this pull request.
In list_view/list_view.h
https://github.com/reupen/ui_helpers/pull/92#discussion_r989448307:

@@ -58,6 +58,22 @@ class ListView {
}
};

class ItemTransaction {
public:

   ~ItemTransaction();

Hmm it could throw if one of the gsl::narrow<> casts in one of the called
functions fails, but that would be a bug (or something is very wrong) if
that happens.
As I understand a noexcept function will call std::terminate() if it does
throw (or at least it is supposed), which is fine by me, so I've added it,
thanks.
How did you find this repo anyway? The outer class here (ListView) is
pretty old and a bit of a mess unfortunately.
—
Reply to this email directly, view it on GitHub
https://github.com/reupen/ui_helpers/pull/92#discussion_r989448307, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFGLW4CXJFETKSICCAK57LWB4X4PANCNFSM6AAAAAAQ652EOA
.
You are receiving this because you commented.Message ID:
@.***>

This was the first time I did a code review in the GitHub, not in the work,
I mean. Just randomly looked for CPP open pr while waiting for my wife to
arrive from her yoga class.

Ah, OK!

Do you know if I can build it on Linux?

I'm afraid not, Visual Studio is needed and it would be a waste of time trying to compilation working on Linux really.