Severe perfomance issues once Deep TabNine Local is enabled in VS Code

For a week or two now, I am experiencing severe performance issues on my TabNine installation once Deep TabNine Local is enabled.
System
Windows 10 Pro, Version	10.0.17763 Build 17763
Intel i7 @ 1.80Ghz with 4 cores, 16 GB RAM, SSD
Visual Studio Code 1.38.0
TabNine 2.1.17
Working on a React app running an un-ejected create-react-app with Webpack watching the directories.
Scenario
Once I enable Deep TabNine Local, for the first couple of minutes everything is ok. After indexing the current project the CPU usage is 0%, the RAM usage is as expected (See #143).
After a couple of minutes however, my entire machine (Visual Studio Code, Firefox, Explorer etc.) is slowing down. I can see two TabNine.exe processes running, one consuming normal amounts of RAM but 25% of CPU power. I assume that it completely takes up the ressources of one of the cores.
The resource consumption is constant, it doesn't go down after I stop typing.
Strange enough if I kill the two TabNine.exe processes, I still experience the slowdown and I have to reboot to get my machine into a workable state again.
Notes

I do not experience this on my MacBook Pro running Mac OS Mojave.
This problem only cropped up for a week or two, I am unable to pin down the exact moment.
Enabling/Disabling semantic completion doesn't have any effect.
I have multiple instances of VSCode open.
Seems to happen on all repos I'm working on. (Example: https://github.com/cstuder/QuoBikeIs/)
I really love Deep TabNine Local and would like to continue using it.

Might be the same as #110 and #124, probably not the same as #24 (That issue is pre-Deep TabNine-time)
What else do you need to know?

Some more observations during a slowdown:

Nothing suspicious in the TabNine logs.
No significant disc activity.
I've checked the handles count with Process Explorer, looks ok (500 handles open) (Relates to #69):

Same here, have the same issue happened couple of times. Both CPU and RAM are highly loaded. Killing the TabNine process and starting VS Code again helps, and no slow downs after.

I'm working with a macbook pro Mojave 10.14.6 and TabNine reaches up to 80% of cpu usage.
Using Visual studio and Deep mode, with local as well.

Came here to say this.
Typing is noticeably slower when tabnine local is enabled. My HUNCH is that this could be fixed by debouncing requests to the engine are only fired after a certain delay (say 200ms). I can't confirm this, though.

My macbook sometimes eat 600% CPU, I like this tool, but 600% will cause very slow to type
this is my laptop info:

Adding here solution from in #24
Hi All, we have a fix in beta (version 2.3.4) for excessive CPU and memory consumption by TabNine.
We'd appreciate it if you try it out and let us know if it solves the issue for you.
To try the fix, you have to enable the beta version. Just open TabNine's config by typing TabNine::config in your editor, and then check both Update TabNine automatically and Receive beta releases of TabNine Then, restart your editor, and press TabNine::config to verify you have TabNine 2.3.4. or newer.  Please let us know if this version solves the problem for you. We plan to make it available to all users in a couple of weeks.

hi @drorweiss， I am using 2.3.5.  The freeze issue still their even I am disable the local modal.

@carlos-wong can you add more detail about the freeze so we can address it?
Specifically, please specify:

what exactly freezes, how often and what causes it
what editor you're using
which OS.

@drorweiss I am use macos 10.15.3.  The freezes will happen after I edit large project which maybe have hundreds source codes. I am wirte java,javascirpt,python,emacs lisp. They all have the same issue. It is ok for some minutes after I am restart tabnine. I am use emacs and https://github.com/TommyX12/company-tabnine.

@carlos-wong kindly try with beta updates enabled and let us know if it's solved (it will be the same version number, but some performance-related code is activated just for beta users)

@drorweiss the issue is still exist

I haven't experienced this issue for some time now.

I am upgrade my mabook pro to 16G RAM , TabNine is work smooth now.


Spi working in the init function but not within a software defined task

I can see the spi is working correctly using a logic analyzer when a spi.write is called within the rtic init function, when I call spi.write within a software task I end up hanging within the spi.write call.
I am using stm32hal2 as my hal.
Initialization
        let cs = Pin::new(Port::G, 12, PinMode::Output);
        let _sck = Pin::new(Port::C, 10, PinMode::Alt(6));
        let _miso = Pin::new(Port::C, 11, PinMode::Alt(6));
        let _mosi = Pin::new(Port::C, 12, PinMode::Alt(6));

        let mode = SpiMode {
            polarity: SpiPolarity::IdleLow,
            phase: SpiPhase::CaptureOnFirstTransition,
        };
        let spi_config = SpiConfig {
            mode,
            ..Default::default()
        };

        // spi_clock_frequency = 80MHz / BaudRate::<divisor>
        let spi = Spi::new(reg, spi_config, BaudRate::Div32);

Here is where execution hangs
// In spi.rs line 557
while !self.regs.sr.read().rxne().bit_is_set() {}

Any thoughts or workarounds to getting a spi.write call to work within a software task?
See also https://github.com/David-OConnor/stm32-hal/issues/81

seems HAL-side (https://github.com/David-OConnor/stm32-hal/issues/81) as far as I can tell

Agreed. It looks like there is a blocking read within the write call.

Sorry I can't share more.

Would it be possible to create a minimum working example? Just an init with the SPI code that executes, and a task that has the SPI code and doesn't execute. Without any code I don't think you'll get any help, unfortunately, as I can't repro the error

Hi, please share av example that we can use for debugging the issue.
Without it it'll be very difficult to help :/

If an example can be provided, feel free to reopen this.


Staged unmarhal, separate marshal for header and partition entries, and many fixes

This fixes #3 and many other things

The change to Gpt.unmarshal is motivated by the fact that you need to read the GPT header before you know how much and where to read the partition entry array (table).
The interface for Gpt.unmarshal is slightly clunky. It returns a pair (`Read_partition_table (lba, sectors), k) signalling that the caller needs to read sectors sectors at lba and pass it to k to finish parsing.


Allow override across different placement types

This change will allow placements to override placementrule defaults and vice versa.
ref: https://issues.redhat.com/browse/ACM-10152

This change will allow placements to override placementrule defaults and vice versa.

Sorry, it looks like the issue was worded poorly. I don't think we should allow overriding Placement for PlacementRule, especially when PlacementRule is deprecated. However, when considering the options for a particular placement Kind (file path, selector, or name), these should be override-able if provided in the policies array.

So based on my understanding, only placement Kinds (name, fp, selector) in the policies array should override the default placement/placementRule Kinds. If a user were to specify a placementRule Kind in a policy in the policies array, but the policy default specifies a placement Kind, would the generated policy bind to a placement or a placementRule?

Sorry, you have it right--I think I got my wires crossed. Let me give this PR another look.

I currently have it so that a placement Kind could override a placementRule default, and vice versa. I could modify it so that only placement Kinds in the policies array could override placementRule defaults if that's the desired behavior.

No, I think it's correct to have it this way. Sorry for my confusion. 😅


Show a Color Bar

You should be able show an image in the sidebar. The code below should give you an idea how to prototype the image that shows a color bar.
# https://stackoverflow.com/questions/48966645/how-can-i-create-a-legend-without-a-plot-in-r
png("del.png")
plot(NULL, xaxt='n', yaxt='n', bty='n', ylab='', xlab='', xlim=0:1, ylim=0:1)
legend("topleft", legend =c('Max Val', 'Min Val', 'Min Val'), pch=15, pt.cex=3, cex=1.5, bty='n',
       col = c('blue', 'white', 'red'), horiz=TRUE)
mtext("Values", at=0.2, cex=2)
dev.off()

Will something like this shown below work?

Yes, that look.

[ ] Use values not just placeholder "Max Val" text
[ ] Reduce margin


Server stops responding post 1 day, only solution start  mycontroller again, Java process also disappears.

Java virtual machine specification
Operating system
Script engines
Java home
/usr/lib/jvm/jdk-8-oracle-arm32-vfp-hflt/jre
Java vendor url
http://java.oracle.com/
VM name
Java HotSpot(TM) Client VM
VM vendor
Oracle Corporation
VM version
25.65-b01
Specification vendor
Oracle Corporation
Specification version
1.8
Management specification version
1.2
Up time
45 minutes 51 seconds
Heap memory
Used: 22 MB, Committed: 38 MB, Init: 32 MB, Max: 247 MB
Non heap memory
Used: 22 MB, Committed: 24 MB, Init: 0 MB, Max: n/a
Threads count
Current: 40, Peak: 41, Total started: 181, Daemon: 20
Class loading details
Loaded: 5293, Unloaded: 36, Total loaded: 5329
Garbage collector - Copy
Collection time: 7759, Collection count: 769, Last GC duration: 45
Garbage collector - MarkSweepCompact
Collection time: 1008, Collection count: 5, Last GC duration: 296

MyController
Version
0.0.3.Final-SNAPSHOT
Database schema revision
1.03.06 - 2017 Mar 03
Database type
H2 database embedded
Database version
1.4.194 (2017-03-10)
Git commit:branch
325447088b374854a8c94856e89550a1fb69f609:development
Built on
2017-06-09T14:20:35+0530
Build JDK
1.8.0_131
Build tool
Apache Maven 3.3.9
Java virtual machine specification
VM vendor
Oracle Corporation
VM name
Java HotSpot(TM) Client VM
VM version
1.8.0_65-b17
Java home
/usr/lib/jvm/jdk-8-oracle-arm32-vfp-hflt/jre
Operating system
Operating system
Linux
Architecture
arm
Version/Kernel
4.4.38-v7+
Location
/home/pi/mycontroller/snapshot/mycontroller/

@hianand do you see any issues on mycontroller.log file?

Nothing very detailed , thats the strange part.
I thought it might be mqtt , i already moved it to raspbian mosquito mqtt,
I had enabled twilio sms setup sometime back.
I am not sure enabling this twilio sms setup is creating this issue, currently the version does not support disabling those entries.
i have setup a cron to restart the service based on your startup script which check if process is active and does not create a new instance.
What i see now is that i could login to the server however My Dashboards freezes , to get it restored, i need to restart the mycontroller.


tests: use single ServiceEntry with multiple addresses in ambient tests

Please provide a description of this PR:
It's a follow-up to https://github.com/istio/istio/pull/51939 and addresses this comment: https://github.com/istio/istio/pull/51939#discussion_r1668976103.

@jewertow: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
44f2b37fcc5d7d792204fa73514551590396646e
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@jewertow: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
fad457bcc9d7d4a1c2224471b6a8a0e32f073d61
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@jewertow: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
fad457bcc9d7d4a1c2224471b6a8a0e32f073d61
link
true
/test lint

integ-ambient_istio
fad457bcc9d7d4a1c2224471b6a8a0e32f073d61
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

/test integ-ambient-ipv6

It fails when request is sent from sidecar:
=== RUN   TestServiceEntryInlinedWorkloadEntry/MESH_INTERNAL_STATIC_0/sidecar
2024-07-19T20:00:04.502045Z	info	tf	=== BEGIN: Test: 'ambient[TestServiceEntryInlinedWorkloadEntry/MESH_INTERNAL_STATIC_0/sidecar]' ===
=== RUN   TestServiceEntryInlinedWorkloadEntry/MESH_INTERNAL_STATIC_0/sidecar/to_captured
2024-07-19T20:00:04.505984Z	info	tf	=== BEGIN: Test: 'ambient[TestServiceEntryInlinedWorkloadEntry/MESH_INTERNAL_STATIC_0/sidecar/to_captured]' ===
    baseline_test.go:1911: 2 errors occurred:
        	* failed calling sidecar (cluster=cluster-0)->'http://[2001:2::f0f0:255]:80': call failed from sidecar (cluster=cluster-0) to http://[2001:2::f0f0:255]:80 (using http): expected no error, but encountered rpc error: code = Unknown desc = request set timed out after 500ms and only 0/1 requests completed (0s avg)
        	* failed calling sidecar (cluster=cluster-0)->'http://[2001:2::f0f0:255]:80': call failed from sidecar (cluster=cluster-0) to http://[2001:2::f0f0:255]:80 (using http): expected no error, but encountered rpc error: code = Unknown desc = request set timed out after 500ms and only 0/1 requests completed (0s avg)

I have to quickly add support for matching multiple VIPs in HTTP routes. So I am reverting this change from baseline_test and will add it again in the PR with HTTP routes.


[WIP] Add action menu for dashboard widgets

Issue: #3717
Old

New

@miq-bot add-label ui, enhancement, wip

@miq-bot remove-label wip

@serenamarie125 @cshinn Please review

Looks great to me! The only change I can think of is to maybe enlarge the "minimize" icon a bit so it's easier to see and the word lines up with the other commands.

@miq-bot add-label wip

@epwinchell vertical ellipses looks good!  Are all 4 icons in action menu from FontAwesome as well?

@serenamarie125 Yes, they are.
@cshinn The words aren't lining up because of a problem with the markup we're generating. We'll probably have that fixed today.

@cshinn added font awesome fixed-width class to better align the text; replaced fa-caret font with the more beefy fa-caret-square-o-down/up

@epwinchell Wonderful, thanks!

@dclarizio Please review

@miq-bot remove-label wip

looks good


Incorrect masquerade rule set

Bug report
The created iptable rule for masquerade is incorrect. Found this (for node CIDR <IP_ADDRESS>/24 and podCIDR <IP_ADDRESS>/16, and node IP in <IP_ADDRESS>/24):
-A CILIUM_POST_nat -s <IP_ADDRESS>/24 ! -d <IP_ADDRESS>/8 ! -o cilium_+ -m comment --comment "cilium masquerade non-cluster" -j MASQUERADE
But according to http://docs.cilium.io/en/latest/_images/kubernetes_iptables.svg it should be:
-A CILIUM_POST_nat -s NODE_CIDR ! -d NODE_CIDR ! -o cilium_+ -j MASQUERADE

General Information

Cilium version v1.6.5
Cilium config:
auto-direct-node-routes: "true"
bpf-ct-global-any-max: "262144"
bpf-ct-global-tcp-max: "524288"
cluster-name: default
container-runtime: none
debug: "false"
enable-ipv4: "true"
enable-ipv6: "false"
enable-node-port: "false"
identity-allocation-mode: crd
install-iptables-rules: "true"
masquerade: "true"
monitor-aggregation: medium
preallocate-bpf-maps: "false"
tofqdns-enable-poller: "false"
tunnel: disabled
wait-bpf-mount: "false"

How to reproduce the issue

create a cluster with podCIDR similar to node IPs e.g. node IPs in 172.16.3.x/24 and podCIDR in 172.28.x.x/16
from any Pod try to ping some local IP (e.g 172.16.3.x/24), that is not part of the cluster in anyway (the ping will fail)

Verified, and same issue is still present with cilium v1.7.2.

@mskrocki I believe there was some discussion today on the nuance of this issue, is there a brief summary that would be useful to note down on this issue here to help guide a developer to resolving this issue?

It probably boils down to why this: https://github.com/cilium/cilium/blob/ae4fbbed82e2e759dc68a83aa0365e26cfe61d7e/pkg/datapath/iptables/iptables.go#L886
ends up being ! -d <IP_ADDRESS>/8 when node podCIDR is <IP_ADDRESS>/24 (from the node.spec.podCIDR).
From the discussion I learned that cilium does not have access to the cluster-wide podCIDR (which is <IP_ADDRESS>/16), but here I don't think it matters.

have been trying to reproduce this for a while. I wasn’t able to do so.
I suspect the cause is the missing flag --set global.k8s.requireIPv4PodCIDR=true
I’ve created a PR to improve the GKE guide: https://github.com/cilium/cilium/pull/11079
It disables tunneling, enables native routing, enables endpoint routes and instructs Cilium to wait until Node.Spec.PodCIDR[s] is populated.
For a node with:
  podCIDR: <IP_ADDRESS>/24

the iptables rule is:
-A CILIUM_POST_nat -s <IP_ADDRESS>/24 ! -d <IP_ADDRESS>/24 ! -o cilium_+ -m comment --comment "cilium masquerade non-cluster" -j MASQUERADE

I have tried the requireIPv4PodCIDR flag and it did not help.
Additionally, I have tried to re-produce this issue on a cluster created with kubeadm v1.17. Using the default https://github.com/cilium/cilium/blob/master/install/kubernetes/quick-install.yaml config from 1.7.2. With tunnel: vxlan, the iptable rule is correct, but when changed to tunnel: disabled, the rule ends with /8 mask (here as well the  requireIPv4PodCIDR flag does not help).

Setting native-routing-cidr fixes this issue.

@mskrocki Can we close this issue?

Yes.


Set message severity according to abaplint.json

Closes https://github.com/abaplint/abaplint-sci-client/issues/177

thanks 👍


The CDN section of the readme doesn't mention the need for jqery

I was trying to use the CDN for jsPDF but my browser would tell me "jspdf is not defined", after seeing the code of someone else randomly on stack overflow I saw that they also included the CDN for jquery, which made jspdf work on my side. I just thought maybe mentioning the need to have jqery imported for the CDN could save the hassle I had for other people haha

jsPDF does not require jQuery! It's probably something else that made it work...

jsPDF does not require jQuery! It's probably something else that made it work...

I was trying to use the CDN for jsPDF but my browser would tell me "jspdf is not defined", after seeing the code of someone else randomly on stack overflow I saw that they also included the CDN for jquery, which made jspdf work on my side. I just thought maybe mentioning the need to have jqery imported for the CDN could save the hassle I had for other people haha

i think you just need this code below to define it, no need of jquary. Then u can do var doc = new jsPDF()
https://github.com/MrRio/jsPDF/blob/63f89d47d1cd60091e62d6d51570f78f27d01a23/index.html#L90

I was trying to use the CDN for jsPDF but my browser would tell me "jspdf is not defined", after seeing the code of someone else randomly on stack overflow I saw that they also included the CDN for jquery, which made jspdf work on my side. I just thought maybe mentioning the need to have jqery imported for the CDN could save the hassle I had for other people haha

i think you just need this code below to define it, no need of jquary. Then u can do var doc = new jsPDF()
https://github.com/MrRio/jsPDF/blob/63f89d47d1cd60091e62d6d51570f78f27d01a23/index.html#L90

I was trying to use the CDN for jsPDF but my browser would tell me "jspdf is not defined", after seeing the code of someone else randomly on stack overflow I saw that they also included the CDN for jquery, which made jspdf work on my side. I just thought maybe mentioning the need to have jqery imported for the CDN could save the hassle I had for other people haha

i think you just need this code below to define it, no need of jquary. Then u can do var doc = new jsPDF()
https://github.com/MrRio/jsPDF/blob/63f89d47d1cd60091e62d6d51570f78f27d01a23/index.html#L90

I've just tried this and javascript var jsPDF = window.jspdf; seems to work, but not javascript var jsPDF = window.jspdf.jsPDF;, and without jquery, thanks a bunch! Is that information available in the documentation?

I was trying to use the CDN for jsPDF but my browser would tell me "jspdf is not defined", after seeing the code of someone else randomly on stack overflow I saw that they also included the CDN for jquery, which made jspdf work on my side. I just thought maybe mentioning the need to have jqery imported for the CDN could save the hassle I had for other people haha

i think you just need this code below to define it, no need of jquary. Then u can do var doc = new jsPDF()
https://github.com/MrRio/jsPDF/blob/63f89d47d1cd60091e62d6d51570f78f27d01a23/index.html#L90

I've just tried this and javascript var jsPDF = window.jspdf; seems to work, but not javascript var jsPDF = window.jspdf.jsPDF;, and without jquery, thanks a bunch! Is that information available in the documentation?


feat(res): rename resources keys if contains unprintable chars or duplicates (#844)

Implement resources key renaming suggested by @jpstotz in #844

Thanks, seems to work perfect for two affected apps!


feat: Add option to not enforce unique module names

Adds a new config key allow-multiple to every module section, by
default it's set to false.
If set to true, the controller will not emit an error when that module
appears multiple times.
There are still some cases where using the same module multiple times is
desirable. For example when you want to have different separators on the
left side than on the right side. adi1090x's polybar-themes does this
in several places.
Since in the future the module name is used to target actions, this will
lead to unspecified behavior when such a module receives an action.
I think when sending actions to modules, we will check allow_multiple()
and either not deliver the action at all or just emit an error.
Ref: https://github.com/adi1090x/polybar-themes

Maybe we should give this a better name. allow-multiple conveyes the purpose of the option but doesn't tell users about what it actually does only the side-effects (except when reading the docs). Something like disable-actions would describe what the module does. And we could then adapt the error message for duplicate modules to explicitly say that modules with actions cannot appear multiple times.
I don't want to dynamically determine which modules actually support actions and which do not because this would make adding actions to a module that hasn't had any before a breaking change.

Alright, I decided it may be better to allow to list the same module multiple times. But I think when actions are delivered, they should be delivered to all modules that can handle them. I think that is much more natural and it allows people to have multiple text modules with the same content (e.g. for separators) and not get any errors or have to add a new key.


The scrollbar on the 'Properties' panel is not aligned to the left after switching from the 'Actions' panel

Storage Explorer Version: 1.37.0-dev (102)
Build Number: 20241106.1
Branch: main
Platform/OS: Windows 11/Windows 11/Linux Ubuntu 22.04/MacOS Sequoia 15.0(Apple M1 Pro)
Architecture: arm64/x64/x64/arm64
How Found: Exploratory testing
Regression From: Not a regression
Steps to Reproduce

Open Storage Explorer and select a storage account.
Observe the scrollbar on the Properties panel (If there isn't a scrollbar, resize the panel until one appears).
Navigate to the 'Actions' panel and click on 'Properties'.
Check whether the scrollbar is aligned to the left.

Expected Experience
The scrollbar is aligned to the left.

Actual Experience
The scrollbar is not aligned to the left.

Additional Context

Not a blocking issue, so we're not going to worry about fixing this unless users are having trouble with it.


Point issues link to this repo

Hi, it looks like you guys used the Angular Full-Stack Generator to scaffold some example Angular apps for this. We have an 'Issues' link in the footer of those apps that points to the GitHub issues for the generator. I think you guys should change that to point to your repos Issues section instead. :)
Reference: https://github.com/angular-fullstack/generator-angular-fullstack/issues/2071

Hi @Awk34 , thanks for the report!  We don't have a repo that developers clone, instead we ask them to start a new project with the angular-fullstack generator, and then we walk them through some steps where they add features to the generated project.  In this workflow I'm not sure how we'd remove the issues link from the default template?

Oh okay, I didn't see anything about using the generator itself when I quickly looked at the docs.
I noticed the two following example apps. That's why I made this issue.
https://github.com/stormpath/stormpath-sdk-angularjs/tree/master/example/dashboard-app
https://github.com/stormpath/stormpath-sdk-angularjs/tree/master/example/ng-route-app
But if users are instructed to just use the latest version of the generator themselves, issues from then on should be reported to us. Thanks :)

Ah sorry I wasn't looking at the example apps :) so we certainly can remove those links.  And we do hope to upgrade to the new generator soon, so that will resolve that use case.  Thanks again for the report, I love the angular generator, thank you for all the work that goes into it!

Per the plan in #166 the Yeomon sample app is going to be removed, so this issue can be closed at that time.


GB Studio Doesn't Work On Windows 11

Describe the bug
Taskbar Icons Disappear, and Windows Button Doesn't Work
To Reproduce
Steps to reproduce the behavior:

Get Windows 11 (Windows Insider Program)
Download newest version of GB Studio (Top one)
Sometimes* when you restart your PC the taskbar will have no icons and the windows button will not work.
And now you just gotta delete GB Studio & Hope that it will fix itself

Screenshots didn't work for me when this happened. But I recreated it!

Platform (please complete the following information):
-Newest Version of Windows 11

Are you sure this isn't an issue with windows 11?
A randomly disappearing taskbar doesn't seem like an app fault, may be other users with this outside of gbstudio.
Gbstudio is built on electron, which is built on chromium.
Depending on the version being used (1.2.1, 1.2.2, 2.0b4, 2.0b5, 3.0alpha from source) will include a different version of electron/chrome, with os specific updates.
We need the gbstudio version in the bug report.

My general idea would be storage or ram are at their limit when gbstudio is open/installed, or how windows 11 handles taskbar previews.
The taskbar used to be a function of explorer.exe, so you should be able to open task manager (ctrl+alt+del, or ctrl+shift+esc) kill explorer.exe process, and then use run (win + r, or from task manager) to type explorer.exe and fix it.

Seems like this is not related to GBStudio.
there are lots of complains about it,
see thread on reddit as well:
https://www.reddit.com/r/windowsinsiders/comments/opxu05/windows_11_taskbar_not_working/

Confirmed issue was with Windows 11 developer preview, unrelated to GB studio
Recommend closing @chrismaltby


flutter.io needs instructions on how to take a trace

https://flutter.io/debugging/ is the closest thing we have.
But I don't have a document to point people to to instruct them to take a trace with Flutter. :)

Can you clarify what it is you would want such a page to say that isn't on the page above?
Specifically under https://flutter.io/debugging/#tracing-any-dart-code-performance

I think I entirely missed this line "Then open your app’s Observatory’s timeline page, check the ‘Dart’ recording option and perform the function you want to measure."  And may be just need some visuals. :)
We also could mention flutter trace assuming that still works.  We could also discuss a little about what one sees in a trace (although that can be a whole separate bug).
I think it would be interesting to have a few folks (like @wmleler) run through this page and see how successful they are at generating a trace file and what they might want added to the docs.

We can take a look at this as we look more at the CPU profiling use-case, cc @a-siva @devoncarew

Woah, an old bug! I do want to do this, but I want the tools to be stable before we write docs.


feat: support macOS.

Support mac OSX.

APNS not work in BigSur 11.3, and find new way for notification.
Ref: https://developer.apple.com/forums/thread/660180

Is this bug fixed in 11.4?
As alternatives, maybe we can set the macos app always in front and connected to the server by websocket? On receiving response, a notification is triggered.

This bug has been fixed in 11.4.
I will restart this feature in 1.1.7.
Thank you very much.

I find the Bark iOS app can be installed on macOS and all the functions work normally. But the chanify iOS version cannot be installed. Maybe you can remove the restrictions as a temporal workaround on macOS?

The Chanify on App store is disabled macos catalyst.
I'm developing macOS native app.
Thx.

The Chanify on App store is disabled macos catalyst.
I'm developing macOS native app.
Thx.

It seems that Mac Appstore does not prohibit catalyst apps. As I know, the note app Craft is developed by catalyst. So, maybe using catalyst is possible and with minimum cost?

@wizyoung version 1.3.0 will support native macOS (11.3 or above).

WOW, a big surprise! Thanks for your effort!

You are welcome. Chanfiy has been released in the Mac App Store. I will continue to improve macOS version in the future.
Thx


add ExpectDo methods

Added example implementations for handling Do() with various return types.  Needs to be fleshed out with all of the various potential return types and tests, but demonstrates the concept.  I may eventually get around to implementing the feature in its entirety, but probably not anytime soon, unfortunately, so this PR exists largely for documentary purposes in the associated issue: https://github.com/go-redis/redismock/issues/25

Any progress on That?? @ideasculptor @ryantate13

I'll update the PR so that it is up to date with go-redis:master but I doubt either of us is going to find ourselves with the time to do the full implementation that would be required to get someone to merge this any time soon.  This fork serves our needs sufficiently as it is.
Check back in a few hours and I should have the PR updated.

Check back in a few hours and I should have the PR updated.

done

client.Do is a special API, its return value is not a fixed data type, if you want to set the return value of int64, please refer to the example:
         // or []interface{}{int64(100), int64(200)}
	mock.ExpectDo("hkeys", "hkey").SetVal([]interface{}{"100", "200"})
	v, err := client.Do(ctx, "hkeys", "hkey").Int64Slice()
	t.Log(v, err)


Sorry, not an issue just a question

Hello! Sorry to post this in issues, but I couldn't find another way to get in touch.
I've been searching for anyone developing a frontend for this project, but haven't come across anything yet.
Do you know of any such projects in development that I could look into?
Thanks, and sorry again.

There's a web front-end in examples/web that you can try out. Is that what you mean?

I completely missed that, thank you I'll have a look at it.
On Mon, Jul 15, 2024 at 11:45 PM Ivo Janssen @.***>
wrote:

There's a web front-end in examples/web
https://github.com/foxxyz/loupedeck/tree/master/examples/web that you
can try out. Is that what you mean?
—
Reply to this email directly, view it on GitHub
https://github.com/foxxyz/loupedeck/issues/35#issuecomment-2230071746,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZV3N33JAY2USFD2TVV2KLZMSXO5AVCNFSM6AAAAABK5YII4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZQGA3TCNZUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

No problem :)


Start building for node v0.12.x instead of v0.11.x

stale, action now over at #43 for new node versions.


output channel: new replaceAll API to improve output channel rendering

This PR fixes https://github.com/microsoft/vscode/issues/132183 and adds a new API method to replaceAll output channel text to improve output channel rendering for extensions that use the output channel for minimal output.
CC: @sandy081, @jrieken

@jrieken and @sandy081 - thank you both very much for the fast turn around on the PR review.
This change is going to make a really big difference to the user experience for our extensions. We had already showed 2 of our users via remote screen share and they were very excited to see that we may be able to have an improved output experience.

@smcenlly Updated the model implementation to incorporate following changes

Updates are completely handled by the abstract model
Replace all / Clear operations will cancel all existing model operations
Append operation is queued after replace operation
Some clean ups

I did some testing with my example. Can you please test with your extension and let me know how it goes.
Thanks.

@sandy081 - I have done some testing with our extension and it works really well. Thanks for making those changes. From a functional perspective, it's good to go.
Are there any other changes to this PR that you need/want me to make? Or anything else you want/need me to do to help with closing this out?
I'm also wondering, what is the process for promoting the changes to the API from vscode.proposed.d.ts to vscode.d.ts (I'm wondering when we might be able to tell our users that they can expect the change)?

Thanks for testing. Next steps:

I will merge the PR
Bring this proposed API to our API discussion meet where we discuss about the API naming/shape/need etc.,
Once it is approved, I will move this to final - vscode.d.ts

I will work to get this to final this milestone but sometimes it will also take time.


Slide control changes

This PR:

Moves the slide control to a new package
Improve the Step model
Adds some missing bootstrap files (I'm not sure why, but this is missing when I'm running in local)

Please @rafaparadela could you take a look? Thanks

Looks good to me! (But add logs to gitignore, please)


Delete database API

Description
API to delete the given database which effectively deletes all the tables under the given database.
The API has a sync behaviour which means partial database deletion is possible in case of failures.
Server side failures are handled silently and further table deletion is resumed. The response will have appropriate information regarding the partial database deletion.
Endpoint : /databases/<databaseName>
Query param : dryRun , required = true
Method : DELETE
Sample output :
{
  "dryRun": false,
  "tablesInDatabase": [
    "db1.table1_REALTIME",
    "db1.table2_OFFLINE"
  ],
  "deletedTables": [
    "db1.table1_REALTIME"
  ],
  "partiallyDeleted": true
}

As the API is of idempotent nature client can repeatedly call the API in-case the deletion has occurred partially
labels
feature release-notes
release-notes
API to delete database and all its underlying tables

Codecov Report
Attention: Patch coverage is 0% with 31 lines in your changes are missing coverage. Please review.

Project coverage is 34.55%. Comparing base (59551e4) to head (71ecee5).
Report is 188 commits behind head on master.

Files
Patch %
Lines

...er/api/resources/PinotDatabaseRestletResource.java
0.00%
31 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12765       +/-   ##
=============================================
- Coverage     61.75%   34.55%   -27.20%
+ Complexity      207        6      -201
=============================================
  Files          2436     2387       -49
  Lines        133233   131086     -2147
  Branches      20636    20266      -370
=============================================
- Hits          82274    45300    -36974
- Misses        44911    82453    +37542
+ Partials       6048     3333     -2715

Flag
Coverage Δ

custom-integration1
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration1
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration2
0.00% <0.00%> (ø)

java-11
34.54% <0.00%> (-27.17%)
:arrow_down:

java-21
34.41% <0.00%> (-27.22%)
:arrow_down:

skip-bytebuffers-false
34.55% <0.00%> (-27.19%)
:arrow_down:

skip-bytebuffers-true
<0.01% <0.00%> (-27.73%)
:arrow_down:

temurin
34.55% <0.00%> (-27.20%)
:arrow_down:

unittests
46.18% <ø> (-15.57%)
:arrow_down:

unittests1
46.18% <ø> (-0.71%)
:arrow_down:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Analytics

Added a new prop to the Link component called analyticsClassName.
This prop adds a class to the resulting <a> providing support for existing adobe tracking.
Files updated are:
Link.js
Link.test.js

Thanks @Shanqua for this!


added terms and condition

fixes #63
is it ok?

@nishantdub37 Thank you very much for your contribution! 🏆

@all-contributors please add @nishantdub37  for content


Turn allowance test back on now that fix is in mirror node.

Turn allowance test back on.
Fixes # https://github.com/hashgraph/hedera-mirror-node/issues/6721

Codecov Report
Patch coverage has no change and project coverage change: +0.95% :tada:

Comparison is base (c4de662) 77.51% compared to head (dcd95ae) 78.47%.

:exclamation: Current head dcd95ae differs from pull request most recent head 72b55d4. Consider uploading reports for the commit 72b55d4 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1685      +/-   ##
==========================================
+ Coverage   77.51%   78.47%   +0.95%
==========================================
  Files          36       23      -13
  Lines        2766     2183     -583
  Branches      561      450     -111
==========================================
- Hits         2144     1713     -431
+ Misses        447      331     -116
+ Partials      175      139      -36

see 14 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add fuzzing test to dioxus-core

This PR adds a fuzzing test to dioxus-core that creates and then diffs random nodes and templates. It is notably missing code to fuzzes keyed diffing. Because this generates random nodes it will hit code paths that are more uncommon. The manual tests we already have can better test diffing nodes that are more similar.

It seems like fuzzing found a bug in core. Are you interested in digging into it, or would you like me to?

I can dig into it

Do you mind resolving the conflicts? Seems like this now passes.

Yayyyyy


This repo has been abandoned by its maintainer.

Issues do not receive responses.  This library does not work with https-proxy-agent, which receives 40 million installs per month.
We need a new repo that will be actively maintained that implements the desired functionality.

Anyone have a good alternative, as this repo does not work with https-proxy-agent?

Hi, I am 3846masa, a maintainer.
I am sorry for not replying for a long time.
Unfortunately, we are very reluctant to be told that maintenance has been abandoned due to a one month or so of inactivity.
As the MIT License originally states, open source software is provided without any warranty, and users should agree to use it.
Making strong demands to fix it because it doesn't work, or to maintain it, is a terribly arrogant and shameful act.
This software may be freely copied under the MIT License. If you disagree with my opinion, please copy and modify this software yourself.
In the future, if you make another issue like this one with no respect for the maintainer, I will ban your account from this repository.

P.S.
This repository has an issue template.
If you create an issue using that template, I will be notified immediately.
I don't know why you didn't use an issue template.
It is clearly wrong for you to contact me by any way other than the one I have provided and then complain that you did not receive a response.


Reset button

Hi itouch2, great module you created! :)
I edited the PhotoTweaksViewController to add a "RESET" button similar to Apple's Photos Edit feature.
At first I tried performing the reset in the PhotoTweakView class, that way I can "animate the reset", to look cool, but then I stumbled up a lot of issues because there are many moving parts (crop rect, grids, scrollview bounds, etc.)
So I ended up just adding the reset button to the PhotoTweaksVC, and drop the photoView from superview and add a new one. Which is ok, but not as cool.
Anyway, it's just a suggestion, if you have time, would be cool to have a reset button.
Thanks!

Thanks for your suggestion. I just added the "RESET" button in the PhotoTweakView class, which allows the user to reset the image similar to Apple's Photos. You can checkout the latest codes. Thanks.

Thanks a lot! Works great :)!


Error: importing VectorStoreIndex

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I have streamlit app that uses llama-index and after the update suddenly it shows error importing VectorStoreIndex. Is there anything I should be taking care of? I am using it on Git and hosting my app on streamlit cloud.
ImportError: cannot import name 'VectorStoreIndex' from 'llama_index' (unknown location)

@paritadave with v0.10.X there is some migration needed.
We have separated the repo into multiple packages, with a core package, plus packages for integration.
There is a migration guide here (with details on a script that will help auto-update!)
https://pretty-sodium-5e0.notion.site/v0-10-0-Migration-Guide-6ede431dcb8841b09ea171e7f133bd77
There is also a package registry here
https://pretty-sodium-5e0.notion.site/ce81b247649a44e4b6b35dfb24af28a6?v=53b3c2ced7bb4c9996b81b83c9f01139

For your example given, it would be from llama_index.core import VectorStoreIndex, since that class is in the core package

Hello. I have a similar problem importing ChromaVectorStore.
I use from llama_index.vector_stores.chroma import ChromaVectorStore and got ModuleNotFoundError: No module named 'llama_index.vector_stores'

@aaronjimv look into the new Doc. I was sitting the whole day on migrating it to the new llama-index (I'm a very novice coder). https://docs.llamaindex.ai/en/stable/examples/vector_stores/ChromaIndexDemo.html
It could be that you need to install the pip dependency anew. The basic python import should be the same as far as the doc is concerned

@aaronjimv look into the new Doc. I was sitting the whole day on migrating it to the new llama-index (I'm a very novice coder). https://docs.llamaindex.ai/en/stable/examples/vector_stores/ChromaIndexDemo.html It could be that you need to install the pip dependency anew. The basic python import should be the same as far as the doc is concerned

Thanks.

Yea for anyone else hitting similar-ish issues, please do create a new venv to ensure there's no lingering references to old installations :)
I usually do
python -m venv venv
source venv/bin/activate
pip install llama-index llama-index-vector-stores-chroma ...

Or similar for poetry or condo

Thanks! @logan-markewich from llama-index.legacy import VectorStoreIndex worked for me


Convert the project to androidx.

Here are the steps I did for the conversion.
Used Android Studio 3.2 canary 17
Auto conversion to androdx

Changed the targetSdkVersion and compileSdkVersion to 28
From Android Studio, navigated to "Refactor -> Refactor to AndroidX"

Manual changes

Trimmed the leading package name (e.g. androidx.recyclerview.widget.RecyclerView -> RecyclerView in files where it's already imported)
Added "androidx.test:rules:1.1.0-alpha3" for a dependency
(com.android.support.test:runner was there before the conversion, but some classes are now provided by a different dependency like "ActivityTestRule")
In build.gradle files, defined duplicate versions through variables.
(e.g. 1.0.0-alpha3 for the androidxCoreVersion)
Handled API changes

LayoutManager.getChildAt is now nullable

These issues need be addressed by the conversion tool. I'll file these
as issues.

double quotations were replaced by single quotations in the dependencies in build.gradle. It's going to be an issue if the versions are defined through variables.
Unneeded leading package names.
e.g.
val navigationView: com.google.android.material.navigation.NavigationView = findViewById(R.id.nav_view)
Should be
val navigationView: NavigationView = findViewById(R.id.nav_view)
Add a missing dependency automatically? Like androidx.test.rules.

Hey guys, can you make a beta release with AndroidX? We like to use FlexBox with AndroidX but our doesn't like it https://travis-ci.org/ebraminio/DroidPersianCalendar/builds/403957063#L1754 apparently.


Create /algo subpage

This page is meant to be the hub for all acmAlgo information. This will provide convenience for the acmAlgo officers who weekly teach about algorithms and data structures.
Changes

[x] Added /algo page with matching infrastructure to the /blog
[ ] Use match routing
[ ] Add hover effect to "Algo" title on /paths that links to /algo with  instructions for going to the acmAlgo page: "Click on one of the paths to find out more" (suggested by @Angel-Armendariz via Discord)

Resolves #420.

This PR is outdated and to be replaced.


Design UI additions to support showing select annotations in Neuroglancer

user is aware that not all annotations will show in NG
user is aware of which annotations are visible in NG?

We added some links to annotations work:
https://www.figma.com/file/gwX2PjfGPfMrbEWMMJq6rW/H1-2024---Annotations-Table-Update?type=design&node-id=561%3A155049&mode=design&t=yRNUdWtNLkrDBrGk-1
We will also look at this again as a part of the tomogram work

This will be on pause until the implementation of annotation visualization in neuroglancer is available.


Documentation Enhancement: Credentials Require Domain

The documentation for the HostConfigurator::RunAs() method does not indicate that the supplied UserID must include a domain or system prefix (e.g., "Muddlehead..."). I discovered this when the credential I supplied were rejected.
Folks experienced in writing services probably know this already. But it would come in handy for people who are still coming down the learning curve.

@markolbert I have raised a PR for this change. Please let me know if you believe it should be worded differently. Cheers :)


Cannot find the 'metadata.csv'

After run the code, I'm sorry to find 'metadata.csv' does not exist. And I could not find it in the homepage of TGS. I was wondering how to solve this problem.

You need to generate it first by running
neptune run --config configs/neptune.yaml main.py prepare_metadata

remember to specify the paths in the neptune.yaml first.
Thanks for spotting this. The Readme.md was missing this part of the instruction.

How to generate metadata.csv using python only?
when I am running below command:
python main.py prepare_metadata
I am getting error:
neptune: Executing in Offline Mode.
neptune: Error: Invalid parameter 'prepare_metadata'. Parameter names must begin with double dash.

Hi - could you clarify where the metadata.csv file should be?
I eventually got the prepare_metadata step running in the cloud, which has put the metadata.csv file into SAL-10\output\metadata.csv
I then changed the neptune.yaml file back to "metadata_filepath: /input/metadata.csv" as per the instructions and ran the line to train the network, but this comes up with an error:
"You cannot use input that is not in Neptune storage."
Does the command line to train the network need to include "SAL-10" somewhere? Plus the metadata.csv file is sitting in the output directory of SAL-10 - does it need to move to an input directory?
Thanks

Hi @LisburnLad
There was a typo-like mistake in the command. It should indeed, contain the reference to SAL-10
neptune send --worker m-p100 \
--environment pytorch-0.3.1-gpu-py3 \
--config configs/neptune.yaml \
--input /SAL-10/output/metadata.csv \
main.py train --pipeline_name unet

Should work in your case.


Does it show(Yes/No) pop up?

Hi
Does it show(Yes/No) pop up?If yes can you make it silent ( without pop up ) please?
Kind regards

Since the project is a POC (proof of concept), it's not at all interesting to do it silently.


Adding a loop to vertex produces two edges

Is it desired behavior? For me it was counter-intuitive. What do you think?
echo $vertex->getEdgesOut()->count(); // will give us 0
echo $vertex->getEdgesIn()->count(); // will give us 0

$vertex->createEdgeTo($vertex);

echo $vertex->getEdgesOut()->count(); // will give us 2
echo $vertex->getEdgesIn()->count(); // will give us 2

Yes, I understand this may seem counterintuitive, but this is actually desired behavior. :+1:
This does make sense from a mathematical perspective, because a vertex with a loop edge is said to have a degree of 2 (i.e. two endpoints of edges reach it).
I hope this helps :+1:

Cool, thanks for pointing this out.
In case I want to have only unique edges, I'd do this:
echo $vertex->getEdgesOut()->getEdgesDistinct()->count(); // will give us 0
echo $vertex->getEdgesIn()->getEdgesDistinct()->count(); // will give us 0

$vertex->createEdgeTo($vertex);

echo $vertex->getEdgesOut()->getEdgesDistinct()->count(); // will give us 1
echo $vertex->getEdgesIn()->getEdgesDistinct()->count(); // will give us 1

This does make sense from a mathematical perspective, because a vertex with a loop edge is said to have a degree of 2 (i.e. two endpoints of edges reach it).

That does make sense if the edge isn't directed. Shouldn't a directed edge only count once?

This does make sense from a mathematical perspective, because a vertex with a loop edge is said to have a degree of 2 (i.e. two endpoints of edges reach it).

That does make sense if the edge isn't directed. Shouldn't a directed edge only count once?

For directed loop edges the same logic applies: The loop edge "points away" from vertex A and "points towards" vertex A, so it is included once in getEdgesOut() and once in getEdgesIn().

Right, but the issue is it's being included twice for getEdgesOut().

@jacobalvarez You're right, what you're describing is indeed a bug in this library, thank you for making us aware of it :+1:
Directed loop edges will indeed be included twice in both getEdgesIn() and getEdgesOut() while they only be included once in each. All other edge combinations seem to work just fine. I'll take a look at this as soon as time permits :shipit:


[REQ] Patching by sha256 digest instead tag only

What kind of request is this?
New feature
What is your request or suggestion?
Hello!
I am testing the tool you have developed and I find it amazing. However, I have seen that currently it only allows to patch with COPA an image with tag. It would be great if you could provide the image with the sha256 hash instead of the tag, as it would be very useful in many occasions.
Is it planned to implement it?
Thank you very much and congratulations for the tool!
Are you willing to submit PRs to contribute to this feature request?

[ ] Yes, I am willing to implement it.

Hi @borja-rivera, patching with only digest is something we can look into. If you want to patch with both the tag and digest however that is possible using the following format: image-name:tag@digest. For example copa patch -i nginx:1.21.6@sha256:2bcabc23b45489fb0885d69a06ba1d648aeda973fae7bb981bafbb884165e514.

agreed, as a workaround you can do the above. tag will be ignored in this case, so it can be anything

Great! I didn't know about this hack. So, in the tag it doesn't matter what I put and what is scanned is the sha256? I could invent a tag and put it there, or the tag must really exist?
Thanks!

@borja-rivera yes, the tag doesn't need to exist and it doesn't matter which tag of the image you scanned for the report - it will still patch the image with the sha256 you entered.

Amazing, thanks for your help!!


Tomn usds/testorder corections phase0

Now part of bigger PR: #686. closing

Now part of bigger PR: #686. closing


Vampire version

Running online-atps --list-atps, I got
[Vampire]
  application: For TF0 FOF CNF
  key: online-vampire
  version: 4.1

[Vampire]
  application: For TF0_CSA TF0_SAT FOF_CSA FOF_SAT CNF_SAT
  key: online-vampire
  version: SAT-4.1

but using the --version-atp option I got:
$ online-atps --version-atp=vampire
Vampire---SAT-4.1

Why the version 4.1 of Vampire isn't reported?

I am getting this
$ online-atps --version
Online-atps version 0.1.1
$ online-atps --version-atp=vampire
Vampire---4.1

and
$ online-atp --list-atps
...
[Vampire]
  application: For TF0 FOF CNF
  key: online-vampire
  version: 4.1

[Vampire]
  application: For TF0_CSA TF0_SAT FOF_CSA FOF_SAT CNF_SAT
  key: online-vampire2
  version: SAT-4.1
...

It seems that you are using and old version, please let me know it.

The problem was fixed in the current master. Thanks!.


New subaperture optic ---NOT READY TO MERGE.---

Here's my first draft of a Shack-Hartmann simulator. Not a finished product, just getting it out there for feedback early.
I've tried to keep it general and make it a "sub_aperture" class which could feed into an IFS or other instrument that samples a wavefront via a regular grid.
Here's a demo notebook showing sampling a wavefront, generating an array of PSFs and a modal retrieval of the input wavefront (this is my first foray into SH wavefront sensing, so it's not at all optimized but it does seem to recover the right Zernikes): https://gist.github.com/douglase/9f70aaffbd683f3af5edf805847a444e
Things needed before merging:

Complete doc strings
I don't know why the logger isn't working

Things that raise a Not Implemented:

more complicated optical systems instead of direct propagation to a detector.
crosstalk between subapertures

Community questions:

What else is missing?
How does the structure seem?
Should it be stored in separate file (as it is here) or just another class within optic.py (or something else?)

What else is missing?

From a software-engineering perspective rather than an optics perspective, I would say documentation and testing. In particular, an example optical system in a notebook would be nice to play with (or just look at) for evaluating the ergonomics of the API. Also, since this models a physical system, a test case that, e.g., shows it really does recover the right Zernikes (to some precision) when used as a SH sensor would be a useful sanity check.

How does the structure seem?

See my line comments for things that stood out.

Should it be stored in separate file (as it is here) or just another class within optic.py (or something else?)

Hmm. optics.py is already almost 2000 lines, so I think it should be its own file. The existing optics in POPPY are exposed in the main namespace through judicious use of __all__ and from .modulename import * (see https://github.com/mperrin/poppy/blob/master/poppy/init.py#L97-L101 for example) so you'll probably want to add a few lines to ensure poppy.Subapertures is a valid identifier.
(It's possible there's a sensible way to split up optics.py into a package with submodules too, but that would probably be a separate PR.)

@josePhoenix, thanks for feedback. Yes, I thinking refactoring into some helper functions is a good idea. I have to think about the mutable arguments, thanks for the warning.
you mentioned wanting to see a notebook, do you want to see something beyond the notebook linked above?  I put it in the gist because I didn't want to clog up the poppy git history with lots of iterations of notebook graphics.

Hahah, I totally overlooked that link. (Maybe I was using my phone?)
That's great!

Coverage decreased (-0.09%) to 64.429% when pulling b50253d8f52d9ae1a18e33574e6c44c6f589dd29 on douglase:new_lenslet_optic into b3852afcd4393d8463a3c0d7d23cd16d3c2a460c on mperrin:master.

Coverage decreased (-0.04%) to 64.475% when pulling a977db8991ea45e17fbab2a7f6f9b4a154033f25 on douglase:new_lenslet_optic into b3852afcd4393d8463a3c0d7d23cd16d3c2a460c on mperrin:master.

Coverage decreased (-0.09%) to 64.475% when pulling 6bd713b1b83b786b55e2b6cfad2e8ebafd6395b9 on douglase:new_lenslet_optic into 5df300a7c8ae481b306429e0c174f1be1b7b477a on mperrin:master.

Coverage decreased (-0.2%) to 64.399% when pulling 559f5e317d30131b284779cf046a931ce125ef6f on douglase:new_lenslet_optic into 5df300a7c8ae481b306429e0c174f1be1b7b477a on mperrin:master.

Coverage decreased (-0.2%) to 64.399% when pulling 4f24c2cd68958c4efe1fe86dc2b80403e67a26b1 on douglase:new_lenslet_optic into 802a249c6e03ffec760a7bfbc960c73be1c6bc2c on mperrin:master.

Coverage decreased (-0.2%) to 64.399% when pulling 15459c38ccbc8a362b300fca9f781bf2425e20e4 on douglase:new_lenslet_optic into 802a249c6e03ffec760a7bfbc960c73be1c6bc2c on mperrin:master.

@douglase is this optic still not ready to merge? I'm hoping to get a poppy release out soon, and my default plan is to leave this out unless you tell me you'd prefer otherwise. In which case, I see there is now a merge conflict with more recent edits in poppy_core that would need to get resolved...

I am still working on test cases for this, please leave it out. If you
prefer to not have the clutter I can close the PR.
On Mon, Jul 10, 2017 at 17:05 Marshall Perrin<EMAIL_ADDRESS>wrote:

@douglase https://github.com/douglase is this optic still not ready to
merge? I'm hoping to get a poppy release out soon, and my default plan is
to leave this out unless you tell me you'd prefer otherwise. In which case,
I see there is now a merge conflict with more recent edits in poppy_core
that would need to get resolved...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mperrin/poppy/pull/185#issuecomment-314274513, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA-nn6fUciKG7UPJSje-HkR8ogaLBbGdks5sMq4cgaJpZM4J5w6w
.

--
Postdoctoral Research Associate, MIT AeroAstro.
<EMAIL_ADDRESS>617-942-1791

No bother about the "clutter" at all!  I'm perfectly happy to leave the PR open as-is. Just wanted to ask you to make sure that was the right thing to do.  Thanks!


Access to event object in onSubmit

I'm trying to access data stored in event in the onSubmit prop. Specifically I want to check event.shiftKey, but with the current parameters, I can only access values, state, and props. Is there a way I can get into that data?
I'm trying to accomplish something like this:
import React from 'react'
import { Form, Text } from 'react-form'

const myForm = (
  <Form
    onSubmit={({values, event}) => { //need access to event here
      if (event.shiftKey){
        console.log('Shift Key Pressed When Clicking on Form Submit Button');
      } else {
        console.log('Just a regular form submission!')
      }
    }}
    validate={({ name }) => {
      return {
        name: !name ? 'A name is required' : undefined
      }
    }}
  >
    {({submitForm}) => {
      return (
        <form onSubmit={submitForm}>
          <Text field='name' />
          <button type='submit'>Submit (Clicking with shift does special things!)</button>
        </form>
      )
    }}
  </Form>
)

@thg303 Any suggestions on this?

I don't know @maxmatthews, maybe it could be done easier with setting a keyboard event handler with onKeyDown attribute.

Would https://github.com/tannerlinsley/react-form/pull/110 fix the issue?
If I do it in a onKeyDown, I have to manage the state which could get messy

@maxmatthews sure that PR would do the job. I don't have write access but @tannerlinsley would take a look at it soon.
If you're in rush you may release a scoped package and use it temporary ;)

Awesome. As long as it's in the queue. Thanks!

Hmm this is a good question. We are working on rolling out V2 of react-form and this is something we can look to add once we make the switch.


Provide onClick handler for resize handle to allow prevent click on resizable parent

[x] Bug
[x] Feature Request
[ ] Question

Problem Report
After resizing, handler and its parent receive click.
If it is sensitive to click, like sortable column header, then it causes a problem.
Solution:
Please provide onClick handler via api to allow to stopPropagation of the click to a parent.
Another solution is to provide API flag e.g. preventClick and call it yourself.
Thanks.

<Resizable
  width={width}
  height={0}
  onResize={onResize}
  axis={'x'}
  minConstraints={[60, 10]}
  maxConstraints={[300, 10]}
  handle={renderResizeHandle}
>

</Resizable>

const renderResizeHandle = resizeHandle => {
return (
 {
e.stopPropagation()
}}
/>
)
}
could solve this problem

<Resizable
  handle={renderResizeHandle}
>
</Resizable>

const renderResizeHandle = resizeHandle => {
return (
 {
e.stopPropagation()
}}
/>
)
}
could solve this problem

I have the same problem and this worked for me.
If you are using typescript and this repo the prop handle is missing.


Compilation error with version 2.0.+

Hi,
Since this commit at the line 26, I get this error
Include of non-modular header inside framework module at build time.
I tried to build a blank project with only ReactiveKit as a dependency in my podfile and the error persists. It works fine with the latest beta though.
I am using Xcode 7.3.1
Regards

Hi @trupin, are you using CocoaPods and do you have the latest version of it? There was the same issue reported in #72 and it was fixed by CocoaPods update.

I am indeed using the version 0.39.0 of cocoapods. I actually cannot use the versions 1.+ since I use some dependencies which are not ready for it yet.
On this project I am stuck with the beta version of ReactiveKit because of this.
Do you think it's doable to make the release compatible with cocoapods v0.39.0 ?

I finally found a way to update to cocoapods v1.0.1 and everything is working as expected now.
Thanks a lot for your help, and this awesome library !


MEmuConsole: This application failed to start because no Qt platform plugin ....

When i package using pyinstaller==6.1.0, run application i get this error. Can you help me?

PyMemuc only uses standard Python libraries so it should compile fine using pyinstaller, maybe check your other dependencies.
Try creating a minimal reproduction with only PyMemuc and pyinstaller to see if this is actually a problem caused by this library.

i used pyinstaller==6.1.0, pyqt5, pyqt6.  But it not work!

Well it looks like an issue with PyQT based on the error message. Try without those to see if this error still appears.
My minimal reproduction would look something like this:
# main.py or entry point for pyinstaller
from pymemuc import PyMemuc

print(PyMemuc.__dict__)


Builder for RHCOS

Motivation
This comment explains well - IMHO - why the community might benefit from having a set of prebuilt Falco drivers for RHCOS.
To have such prebuilt drivers, we need a driverkit builder for RHCOS, first of all.
Feature
Create a builder for RHCOS.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

A builder for RHCOS/RHEL 8 still would be very helpful.

/reopen

@Tamronimus: You can't reopen an issue/PR unless you authored it or you are a collaborator.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/reopen

@FedeDP: Reopened this issue.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

A builder for RHCOS/RHEL 8 still would be very helpful.

I think you can use vanilla builder passing to it a kernel config; see for example what we do for bottlerocket with kernel-crawler: https://falcosecurity.github.io/kernel-crawler/?arch=x86_64&target=BottleRocket

Well, for RHEL 8, you'll need access to a system that is registered to receive RHEL entitlements. I'm not sure how it's actually working now?

Uh i forgot we had a redhat builder now :D it was contributed by @johnbrydon !
Gentle ping :)

Hahaha =)
IIUC, there's no work to be done. If you look at the example builders, there's a sample config for RHEL8 (as well as RHEL9!)

Oh i see; i completely forget about this :laughing:
Then i think we can close this one, right?

I think so! yes

/close

@FedeDP: Closing this issue.

In response to this:

/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Consider cargo fmting the codebase

Totally up to you, but I'm doing a bit of work on this repo and finding the lack of cargo fmt difficult.

The generator code is now cargo fmtd as of https://github.com/tychedelia/kafka-protocol-rs/pull/25
Possibly we also want to cargo fmt the generated code as a final step of code generation.
But I think there is value in terms of debuggability in not doing that.
So I'm going to go ahead and close this issue, but we might still decide one day that cargo fmting the generated code is a good idea.


Customize an existing VM example

Hi,
Can you please show how to customize an existing VM?
For instance, if I have vim.vm object vm
and I want to modify vm.config.hardware.numCPU (which is now == 2),
I wan to set it to 8.
Thanks!

This might help you
https://gist.github.com/snobear/8788977#file-clone-py-L179

spec = vim.vm.ConfigSpec()
spec.numCPUs = 8
task = vm_obj.ReconfigVM_Task(spec=spec)
tasks.wait_for_tasks(si, [task])


Add a timestamp to the directory history

(mentioned in Telegram some time ago, but forgot at the time to file an issue)
It would be nice to add a last-used key to each element in dir-history. This would make it trivial to present a persistent chronological directory history. It should ideally be something easy to sort with - maybe a timestamp of the form YYYYMMDDhhmmss or simply an increasing numeric ID.

maybe a timestamp of the form YYYYMMDDhhmmss or simply an increasing numeric ID.

It should be a value in a timezone and DST/STD agnostic format. For example, the UNIX UTC timestamp or equivalent. A simple integer that represents the time since some epoch. Anything else might be convenient on a single computer if you don't care about DST/STD transitions but is otherwise problematic.


Add internal SPI LIS3MDL support for the R15 Pixracer

This has been tested good on a MRO Pixracer (R15) - (R15 - LIS3MDL replaces the obsolete HMC5983 on SPI).
Also tested on the original R14 Pixracer to confirm there were no ill effects.

I can test and merge this tomorrow morning. What I'm actually wondering is how it went unnoticed in the first place.


HarfBuzz shaping support

HarfBuzz does both shaping and font selection. It seems to be easiest to implement as be a variation of identity-h encoding.
kerning also needs to be taken over by the encoder. It's enabled by turning on the :kern feature in HarfBuzz fonts.

Latest version has a shape() method, which includes a :kern option.


Finish getting photos for the alumni page

Still a couple missing, but I couldn't find their images anywhere on the web


Reduce polution of the global namespace.

Undeclared variables pollute the global namespace and are annoying to track down. This change adds var in-front of x in the for loop and the body variable preventing them from being attached to the global object. A long term solution to making sure a problem like this does not happen in the future could be adding
"use strict";

to the top of the project.

Thanks.


todo specification?

I see that todo.txt is unsupported. What is the syntax for new specification and could you tell why did you decide to drop the support for todo.txt?

The manual is built into the extension.


find_by_headers returned wrong table when using nested tables

find_by_headers would stop to early when scanning for headers, as it would treat headers of nested tables as headers of the top table

ping

Sorry, I'm not sure why I never saw this. Mentioning me (@mk12) might be more reliable.
I'm willing to merge this fix with two changes:

It should be possible to do it directly with CSS selectors. I think :scope > th instead of th would be sufficient. If that doesn't work, there's probably some way to refactor it to use tr > th.
It needs a test.


ZstdOutputStream leaking thread handles

We have a long-lived service process on Windows that periodically uses zstd-jni to create compressed files. We've noticed that this process accumulates thread handles over time that are not released, even after streams are closed and Java garbage collection is triggered. The accumulation of leaked handles is more drastic with higher numbers of concurrent compression threads, but still occurs when configured for a single compression thread. Note that the number actual threads at any given time is constant, but the number of HANDLES does increase over time as repeated compression tasks occur.
This issue is present in the newest release of zstd-jni as of today, which is 1.5.1-1.
How we're measuring handle accumulation
We're using process explorer on Windows to view the accumulation of thread handles in our Java process over time. The count of handles for a given process can be viewed by right clicking the process, going to properties, and then clicking the "environment" tab within the subsequent window that opens. The individual handles for a process are enumerated in the lower pane of the main process explorer window where you select the process and then View --> Lower Pane View --> Handles. From this we can see that the accumulating handles are thread handles and not file handles or some other type.
Sample Code
Below is the simplest example we could come up with to illustrate the problem. If you run the following code against a relatively small file and examine the handles via the process explorer method described above, you should be able to see the growth. It may help to place a breakpoint at the end of the main method to keep the process running while you inspect in process explorer. Note that we are properly closing the streams via try-with-resources.
For reference, when we run this code against a 10MB input file with 16 compression threads, we see an accumulation of about 2000 handles, and this count increases as you increase the number of for loop iterations. If you remove the ZstdOutputStream, the count of handles generally remains constant.
package my.package;

import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.InputStream;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;

import com.github.luben.zstd.ZstdOutputStream;

public class HandleLeakTest {
    public static void main(String[] args) {
        Path inputFile = Paths.get("path/to/input/file");
        Path outputFile = Paths.get("path/to/output/file");
        try {
            for (int i = 0; i < 100; i++) {
                compress(inputFile, outputFile);
                Files.delete(outputFile);
            }
        } catch (Exception e) {
            e.printStackTrace();
            System.exit(1);
        }
    }

    private static void compress(Path inputFile, Path outputFile) throws Exception {
        try(InputStream inputStream = new BufferedInputStream(new FileInputStream(inputFile.toFile()));
            ZstdOutputStream outputStream = new ZstdOutputStream(new BufferedOutputStream(new FileOutputStream(outputFile.toFile())), 16)) {

            outputStream.setWorkers(16);
            byte[] buffer = new byte[4096];
            int n;
            while (-1 != (n = inputStream.read(buffer))) {
                outputStream.write(buffer, 0, n);
            }
        }
    }
}

Hmm, interesting. Does it still leak threads if you add outputStream.close() after the loop?
My hypothesis is that the outputStream is not being closed/GCed fast enough so threads are not released.

Thanks for the response! I'm back to work today.
Yes, the issue still happens if I add an explicit call to close at the end of the try block, which is expected since try-with-resources  does the same thing implicitly.
I also thought the leak may have to do with GC, but now I'm not convinced. Note that although the sample code I supplied is a short lived program, our production code runs continuously as a service process for days/weeks/months and the thread handle count is not reduced until the process is restarted. We've had customers report millions of handles held by our process after multiple days of repeated compression tasks.
I also tried triggering GC manually via jconsole and was able to see the in-use memory drop, but the open thread handle count seems to be unaffected by the cleanup. I also want to re-emphasize that we're not leaking actual threads - just thread handles, which are essentially pointers to threads that no longer exist.

My bad, yes, it's not related to GC - I didn't notice it was in with block. Currently the worker threads are spawned somewhere in the native library that we use from upstream. What do you mean by 'thread handle"? Worker threads  should not be visible to the JVM side as the threads are internal handled by the zstd library. On the JVM side we don't create any threads.

That's a good thought - I was also thinking they might be native thread handles and not related to the zstd-jni java code. It also seems possible that the java code is not properly closing out some native objects, in which case it would be a bug in the java code.
I can try a repro with the native zstd binaries, and I can also see if the issue reproduces on Linux too. It will take a little time to get back to you.

No rush. The Java code calls ZSTD_freeCStream on close and it should release all resources.
I found the Zstd threading implementation for win32 https://github.com/facebook/zstd/blob/12c045f74d922dc934c168f6e1581d72df983388/lib/common/threading.c#L23-L77 and it looks it does not close the handles (CloseHandle) but I have not worked on windows and I am not sure about the semantics and requirements.

I completed a couple of tests on Windows using the native zstd executable (version 1.5.2).
For the first test I generated a list of 323 files to be compressed (total size 51GB) and then compressed those files individually within a single instance of the zstd process using the following command:
zstd -T16 --filelist .\FILE_LIST.TXT --output-dir-flat .\COMPRESSED_FILES
This resulted in the zstd executable opening about 80 handles and staying constant at level for the entire duration. No evidence of thread handles leaking over time, and the count does not seem to vary with the number or size of the files.
For the second test I ran the command suggested by @luben above on a single 24GB file:
zstd -b1 -e9 -i10 -T16 BIG_FILE
This resulted in a total of 250 handles or so by the time the process finished. The number of opened handles did increase over time as it proceeded through the benchmarks, but not as dramatically as it does with the zstd-jni code sample I shared in my original post.
I am not sure why the results are slightly different for these two test scenarios, but I suppose the results from the second test show that the number of open handles can increase over time in the native implementation. This is not a critical issue for my organization at the moment so I don't intend to pursue this further right now, but figured I would share these findings.

Thanks. I will share with upstream if they have any idea how to fix it.

The leak was fixed upstream with https://github.com/facebook/zstd/pull/3147 so next release will have it fixed.

Yes, that's great! Looks like they put out 2-3 releases per year on average.


Fix css-lint warnings in Travis

Reassigning to @Coenego for review.

Coverage remained the same at 100.0% when pulling 60abd939451c485b0ebc3c81b1cf1769e3c52fe0 on bp323:issue-333 into 0011d80bced82dd9aabded11de7b47518829fdab on CUL-DigitalServices:master.


Middleware test

Testing redirecting to tina when authorized

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Suggestion: Guaranteed full alpha-numeric character set?

The randomAlphaNumeric() function doesn't always return a full data set with a minimum of 1 of each possible characters. Basically I want to ensure we get at least 1 of each range: 0-9, a-z, and A-Z.
In some cases I was getting only numbers, or only UPPER with numbers.  I've worked around it with the below function, which I'm sure is probably the wrong way to do this, but it works for now.
// Greg Fischer - guaranteed full data set of a-z A-Z and 0-9
String randomAlphaNumericFullSet(int length){
	String x = randomAlphaNumeric(length);
	bool z = false;
	while (z == false) {
		//print('Assessing X: ${x}');
		if(x.contains(new RegExp(r'[a-z]'))
			&& x.contains(new RegExp(r'[A-Z]'))
			&& x.contains(new RegExp(r'[0-9]'))
		){
			// print('contains a-z A-Z and 0-9 : ${x}');
			z = true;
		}
		else {
			x = randomAlphaNumeric(length);
		}
	}
	return x;
}

Hello @greg-fischer thank you so much for this and I never thought of this helpful case.    For example, I imagine, ok machine give me a random string where at least one digit is lower case, one digit is upper case and another is a special character.  Or, more generally, give me a random string where at least 3 characters are numbers, 2 are lower case and 5 are upper case, etc.  I have an idea how to implement this cleanly but just wanted to see first if my thoughts were aligned with yours.  Thank you again for your idea here.

Yes, in a way that's what I want. Basically, I want to ensure I'm using all of the desired characters by using at least 1 of every type in the set.  Ideally yes, having a better mix would be nice.  In my case I typically wish to provide a unique key or token.  I sometimes want alpha-numeric only, other times I also want special characters.  It ensures the characters used are 62 possible (a-z, A-Z, 0-9) rather than 36 (A-Z and 0-9).  It would be nice to have the option of either though, but in reality if I wanted only upper, that's super easy after the fact.
While I'm asking...
Another thing that I've ran into before is the need to provide "work safe" random characters. I've ran into a few that generated curse words and ended up in customer hands.  It would be nice to have an optional argument to provide a work safe version. Below is what I found per Microsoft suggestion, and in my old PHP code we do this. So and optional "safe" = true.  randomAlphaNumeric(safe: true);
Return only WORK SAFE character output by omitting certain characters and vowels
OMIT Per Microsoft: 0 1 2 5   A E I O U   L N S Z

Also, there may be times it could be helpful to get a random string with a provided character set as an argument.  For example, if I wanted to get a random string with only 12345 and ABCD characters.  So I could call the function: randomStringWithSet(characters: '12345ABCD')
Thanks for helping with this!

Hello @greg-fischer Just writing to let you know I haven't forgotten.  I'm giving a talk next week and once finished, I'll be able to look into this more.  I hope all is well.

Oh no problem at all, and don't go out of your way.  I'm using that sample function for the time being anyway and whenever you happen to fit it into your project work is great.  The update is appreciated though!

@greg-fischer  Thank you for your patience.  I've recovered from my talk finally :).  I'd like to get your feedback on the following to see if this would be helpful.
Return a full data set with a minimum of 1 of each possible characters.
Perhaps the following function might solve this by adding a String alphabet parameter:
String randomFrom(int length, String alphabet,
        {AbstractRandomProvider provider: const DefaultRandomProvider()}
    )
    { ... }

By doing this, then we could fulfill the use case above as:
const NUMERIC = "0123456789"
const UPPER = "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
const LOWER = "abcdefghijklmnopqrstuvwxyz"
const ALPHA = "..." // combined UPPER and LOWER
const ALPHA_NUMERIC = "..." // combine everything
String numberPart = randomFrom(2, NUMERIC);
String lowerPart = randomFrom(5, LOWER)
String upperPart = randomFrom(5, UPPER)

The existing functions would then be special cases of the randomFrom function.
Combining generated sets
Combining without reshuffling is probably a trivial concatenation of the generated random strings.  However, there are situations where we want to shuffle the combined strings.  Perhaps the following could be included in the library for convenience.
String combine(Iterable<String> elements, {AbstractRandomProvider provider = ...}) { ... }

If order matters, then one could simply concatenate strings as follows.
String alphaPart = randomAlpha(...)
String mixPart1 = randomFrom(...)
String mixPart2 = randomFrom(...)
String generated = alphaPart + combine([mixPart1, mixPart2]);

Validated or "work safe"
As to the work safe or validated generated strings, would the following work?  It would return a future because we do not know how long this would take given the restrictions placed by the validators.  I have been meaning to provide an async generator anyway.
class Generator {
    Generator(Iterable<RegExp> validators);
    Future<String> randomFrom(int length, String alphabet, {AbstractRandomProvider ...})
}

@greg-fischer I will close this issue for now.  Please feel free to open it if you still feel this is valuable.  If anyone else reading this feels they'd like to re-open this, please feel free as well.  Thank you, again, for your contribution.

I'm so sorry I never did get back to you on this. I haven't done much in dart/flutter in weeks and have had so many other things to deal with. (mainly php dev and sys admin taking priority right now)  From initial review I think this is helpful. I really appreciate you taking time to help with this.  When we get back into the flutter work I'll definitely test this.  Thanks again!

Hello @greg-fischer Thank you for writing back so quickly.  I've reopen this and will be happy to implement this in time.  I hope you and your family stay safe.


add variable-replace to portfile(7)

this PR add variable-replace extension to portfile(7)

Thanks.

thank you


Can MTR probe be added?

Describe the feature you'd like and the problem it will solve
We would like to run MTR monitoring to discover points of ping failure loss in the network.

MTR probe will require some more thinking, but one interesting way to get MTR on "ping" failure is to use the new alerting feature with command notifier.
This was shared by Nick Zurku in the Cloudprober slack channel:
https://cloudprober.slack.com/archives/C9ZPA5611/p1696540107564699?thread_ts=1696452186.068299&cid=C9ZPA5611
You could add a stanza like this to your probe config:
  alert {
    name: "high_failure_rate_alert"
    condition {}
    notify {
      ...
      command: "/srv/netprobe/send_mtr_report.sh @target@ @target.label.address@ @probe@"
    }
  }

Basically it will run a shall script on any failure.. that shell script can run mtr and send a report to the given address. I found it really interesting.
More on alerting in Cloudprober: https://cloudprober.org/docs/how-to/alerting/.


How to set active hash yml with different environment

I would something like
if(environment=production)

id: 1
code: AAA
currency: usd
deposit: true
withdraw: true
id: 2
code: AAB
currency: usd
deposit: true
withdraw: true
else
id: 1
code: QQQ
currency: usd
deposit: true
withdraw: true
id: 2
code: QQR
currency: usd
deposit: true
withdraw: true
end

How to manage them in one yml?

Maybe impossible 🤔
YAML doesn't have control structure. So you should separate file by each environment, and change loading file depending on environment ( using set_root_path and/or set_filename.


copr: Add vectorized crc32

Signed-off-by: gauss<EMAIL_ADDRESS>PCP #5751
What have you changed?
Add vectorized crc32
What is the type of the changes?

Improvement (a change which is an improvement to an existing feature)

How is the PR tested?

Unit test
Integration test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No

@sre-bot /run-integration-copr-test copr-test=pr/66

@sre-bot /run-integration-copr-test copr-test=pr/66

/merge


Missing fruit bread texture

Hi there! firstly great mod, I really love the depth and amazing work put into this mod, as for the bug: I noticed that some fruit breads are missing textures, not the whole bread/dough just the added fruit parts are see-through, first one I noticed was the dired black currant bread, even while cooking in the oven the textures are missing, hope this helps

!

Hey there, what do you mean? That looks fine to me. Did you put more than just black currants in there and they aren't showing up?

Hi again! I only put the black currant in with the dough, I can't get a video, but the darker parts are actually see through, if you hold a piece of bread and move around you can see whatever is behind the bread, and the dough is also see through while in the oven, those are the oven textures not bread, here's a close up of the dough.

hope this is more clear

Oh no! I never zoomed in that much so I couldn't tell. I assume other fruits are like that too?

Only dried fruits, either mixed or not mixed, normal non dried berries/fruits don't seem to have the effect as far as I can tell. though I haven't checked them all

Also! just checked the muffins in game, they seem to have the same issue, dried berries again, models are super cute though!

That's very puzzling, given that dried fruits and regular fruits have the exact same texture when used in bread or muffins!


How can I contact the author

How can I contact the author？

Either here or email to the address given at https://github.com/giscus/giscus/blob/main/PRIVACY-POLICY.md#how-to-contact-us.


System-tests: framework(clients tests): add comments what value can be set for each parameter

it's not so clear for me: https://github.com/EnMasseProject/enmasse/blob/master/systemtests/src/main/java/io/enmasse/systemtest/executor/client/Argument.java
for example

--timeout //value: (from: 0; to: 2 147 483 647) milliseconds/seconds/minutes/...
--log-msgs //dict/json/...

or just put link to existing documentation please

I't so simple, what about run client wrapper with --help?
answers for your questions:
--timeout = in seconds, e.g.( --timeout 2 -> means that client waits 2 sec in iteration between send/receive, when client cannot send/receive message)
--log-msgs has dict/json/interop, but in systemtests we have to use only json.


Username != user email in sandboxes

https://github.com/trailheadapps/apex-recipes/blob/394e33592c7a9167587bfa4dae04ef2e46e09f95/force-app/main/default/classes/Trigger Recipes/MetadataTriggerService.cls#L49
I'm curious how to address this, since the bypass will fail if we compare running user email to username (since username has a suffix, and SOME user emails will have had that suffix removed).

This is an example we have shown with an assumption that emails are also used as usernames for users in the org!!
If this is not true for your org feel free to simply user Email directly to compare or any other unique Identifier for your user. You might need to use UserInfo.getUserEmail() if you use email for comparison.
Hope that helps!

This is a good opportunity to show best-practices, which means that code should work in any org.
Can you use startswith() somewhere, to at least make sure that the username or email matches?

Sometimes these autofilters of private information are a pain:
My production username might be:
briemers AT redhat.com.gss
Indicating it in RedHat's gss instance.   My e-mail address might be set to:
briemers+gss AT redhat.com
Because I need something to filter email.
A sandbox might have a username and e-mail set respectively as:
briemers AT redhat.com.gss.mysandbox
briemers + gss.mysandbox AT redhat.com
Different users use different conventions for what they set their e-mail
to.  And different organizations use different ways to modify user names to
be unique across production orgs.
It would be hard to compare that even with well written regular expressions.
On Mon, Jul 18, 2022 at 10:16 AM Bill C Riemers @.***>
wrote:

Even starts with is not a good solution.
For example, I will often make my e-mail address in a sandbox indicate the
sandbox name like:
@.***
While my username would often be:
@.***
The reason being is the . syntax is applied automatically by
salesforce. But often, salesforce e-mails do not clearly indicate which
sandbox they are from. If I want to avoid panic from thinking production
is down, I want something that can easily be identified in the e-mail
headers.
At RedHat we actually have multiple salesforce production instances. So
often times even the production usernames maynot be their e-mail address.
Regards,
Bill
On Mon, Jul 18, 2022 at 1:19 AM David Schach @.***>
wrote:

This is a good opportunity to show best-practices, which means that code
should work in any org.
Can you use startswith() somewhere, to at least make sure that the
username or email matches?
—
Reply to this email directly, view it on GitHub
<
https://github.com/trailheadapps/apex-recipes/issues/336#issuecomment-1186778496
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAYFYWAN6BDK7JESWVBXY2TVUTSOJANCNFSM53P4OTFQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Bill C Riemers, PhD, CSM, CSD, A-CSD, SALESFORCE CPD I, SALESFORCE CDL&DA
Principal Software Engineer
Red Hat Canada Ltd https://www.redhat.com/
Enterprise Sales + Services (ESSA)
https://red.ht/sighttps://patents.justia.com/inventor/bill-c-riemers
—
Reply to this email directly, view it on GitHub
https://github.com/trailheadapps/apex-recipes/issues/336#issuecomment-1187547285,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYFYWC7SJAUN22FJEYRLKLVUVRM5ANCNFSM53P4OTFQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Bill C Riemers, PhD, CSM, CSD, A-CSD, SALESFORCE CPD I, SALESFORCE CDL&DA
Principal Software Engineer
Red Hat Canada Ltd https://www.redhat.com/
Enterprise Sales + Services (ESSA)
https://red.ht/sighttps://patents.justia.com/inventor/bill-c-riemers

@dschach, I think every org is unique, and this repo aims not to build code that addresses the needs of every org. I will let the sample apps @pozil lead share his views on this.

I concur with my colleagues. Apex Recipes are samples that are not meant to be deployed to production as-is or fit all needs.
If we were building something generic and fully configurable, it would make the code harder to approach for learners.