[terra-flowsheet-data-grid] Added column highlighting for flowsheet grids

Summary

What was changed:

Added column highlighting (green and orange) for flowsheet grids.
Added new prop columnHighlightId to choose column to be highlighted.
Added new prop columnHighlightColor to choose highlight colour (orange or green)

Why it was changed:
Column highlighting feature request from clinical charting.
NOTE
Tests will be updated once styles are finalised.
Testing

This change was tested using:

[ ] WDIO
[ ] Jest
[ ] Visual testing (please attach a screenshot or recording)
[ ] Other (please describe below)
[ ] No tests are needed

Reviews
In addition to engineering reviews, this PR needs:

[x] UX review
[ ] Accessibility review
[ ] Functional review

Additional Details

This PR resolves:
UXPLATFORM-9542

Thank you for contributing to Terra.
@cerner/terra

The new props should only be visible on doc site for flowsheet-data-grid and everywhere else it should be private right?

The new props should only be visible on doc site for flowsheet-data-grid and everywhere else it should be private right?

Yes. Changed in https://github.com/cerner/terra-framework/pull/1985/commits/f316f91d6aa32289ef93a1a41ef44f72e62761e8#diff-64d05a135418b5eaa187214d1d0a5ed8df59d36a646018d75c2cf345b93a30e8

Also hover and select states of cells in highlighted columns should still have the border

Also hover and select states of cells in highlighted columns should still have the border

Fixed in https://github.com/cerner/terra-framework/pull/1985/commits/d16d6e95740427f5611a1f2f37e3d81057eb7105

I would expect the columnHighlightId to be at the FlowsheetDataGrid level.  Is there a new requirement to allow multiple columns to be highlighted?

There don't appear to be any examples of column highlighting with collapsible sections.  That should be part of the example as well as unit testing (WDIO).

I don't think we need separate example pages for orange and green highlights.  You can show that with two Flowsheets in the same page.

I would expect the columnHighlightId to be at the FlowsheetDataGrid level. Is there a new requirement to allow multiple columns to be highlighted?

Yes, they mentioned about multiple columns being highlighted at the same time as mentioned here by eric : https://github.com/cerner/terra-framework/pull/1985#discussion_r1472910944
Hence removed columnHighlightId

I don't think we need separate example pages for orange and green highlights. You can show that with two Flowsheets in the same page.

Combined green and orange doc examples into one. Kept multi-column highlight as separate example: https://github.com/cerner/terra-framework/pull/1985/commits/8a2aac51a800bf755283f26bbdb29313b3703c04
Test examples are kept as separate for individual WDIO testing

There don't appear to be any examples of column highlighting with collapsible sections. That should be part of the example as well as unit testing (WDIO).

Added doc and test example for sections in https://github.com/cerner/terra-framework/pull/1985/commits/8a2aac51a800bf755283f26bbdb29313b3703c04


Add Compliant Apim

Proposed changes:

apim custom policies (1), policy set and assignment

Branch  and pr will be recreated


Comment out home page test elements

Closes https://github.com/reform-org/reform/issues/384

I would have moved it to a /debug route that's not linked anyware

I would have moved it to a /debug route that's not linked anyware

This was just a quick fix before our user testing. Feel free to improve


update dependencies

Codecov Report
Base: 48.18% // Head: 48.18% // No change to project coverage :thumbsup:

Coverage data is based on head (405dcaa) compared to base (eca18ed).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   48.18%   48.18%
=======================================
  Files          11       11
  Lines         276      276
  Branches       45       45
=======================================
  Hits          133      133
  Misses        143      143

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

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


TPS < 10 results in 0

100ms batches are calculated by dividing the provided tps by 10, which results in rounding tps < 10 down to 0, and generally inaccurate rates for other low values.

This can be patched by using some modular arithmetic to generate 10 different batch sizes to iterate over, instead of trying to make them all even.
Examples:
1 tps: [1,0,0,0,0,0,0,0,0,0]
5 tps: [1,1,1,1,1,0,0,0,0,0]
15 pts: [2,2,2,2,2,2,2,1,0,0]
etc.
The order of the batches can be randomized as well (once at first, or before each time through) so as not to front load each period.


rake db:seed error

PGError: ERROR:  relation "schema_migrations" does not exist
LINE 4:              WHERE a.attrelid = '"schema_migrations"'::regcl...
^
:             SELECT a.attname, format_type(a.atttypid, a.atttypmod), d.adsrc, a.attnotnull
FROM pg_attribute a LEFT JOIN pg_attrdef d
ON a.attrelid = d.adrelid AND a.attnum = d.adnum
WHERE a.attrelid = '"schema_migrations"'::regclass
AND a.attnum > 0 AND NOT a.attisdropped
ORDER BY a.attnum
Tasks: TOP => db:seed => db:abort_if_pending_migrations

When you run db:seed rake uses ActiveRecord to check for pending_migrations. Which queries the public.schema_migrations table, which doesn't exist in public, it is in global.

Is this a viable solution?
In config/application.rb
config.active_record.table_name_prefix = 'global'


Add PHP client library

I signed it!

Super cool. Could use some API docs to improve the projects usability to new people. But, with a good code analyzer which most PHP devs use, it's completely workable.
Thank you for the addition to the family!


Disabling/hiding title bar

version: newsboat 2.13.0
I want to disable title. I don't want to change the default format, the default background color or the default foreground color. I just want it disabled both in feed list and in article list. Is it possible? Thanks.

Hi! It's not possible at the moment, and if we add an option for that, it will hide the title in all dialogs, not just feed- and articlelist.
Can you explain why you want it hidden? Are you short on vertical space, and want to save one line?

Just to save one line whose content I never find useful. I open newsboat, I see the feedlist, I enter the first non-empty feed, then I switch to the next non-empty feed. A very basic usage.

Implemented in #478.


Is there a 32-Bit and a 64-Bit exe?

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

Hello, I am using the youtube-dlc.exe on the back-end of a program that I'm building an mpv player for. I have a very simple build, but it looks like the youtube-dlc.exe only executes on my 64-bit vm's. I was just curious to know if there was a 32-bit and 64-bit version of this youtube-dlc.exe?

There is. However the new workflow got some minor flaws so 32 Bit didn't get build this time. But will add it manually (maybe today or later). Do note tho that I will name the 32 Bit version youtube-dlc_x86.exe.

There is. However the new workflow got some minor flaws so 32 Bit didn't get build this time. But will add it manually (maybe today or later). Do note tho that I will name the 32 Bit version youtube-dlc_x86.exe.

Thank you for letting me know. I’ll be on the lookout for it! 😁

@armyc84 Sorry forgot to ping you. The 32 Bit version got added.

@armyc84 Sorry forgot to ping you. The 32 Bit version got added.

Awesome, brotha! Thanks!


Possible ClickBench regressions in DF 33 compared to 28

Describe the bug
See: https://github.com/ClickHouse/ClickBench/pull/145 for queries that regressed between 28 and 33
To Reproduce
Need to check whether it is reproducible first or if it is just from different setup (e.g. machine type).
Expected behavior
No response
Additional context
No response

Seems it was the setup afterall


[FEATURE] Gantt Charts

With the inclusion of Moment.js, it feels odd not to have even the bare bones such a time-oriented chart available. Horizontal bars don't quite do the job, even with a Moment.js-based time axis.
I realize this is a rather large feature request.

Expected Behavior

Gantt should be an available choice for chart type under the config.
Each dataset would represent a row in the Gantt chart.
Datasets would contain the Moment ranges that are being displayed.
Background/border color lists would determine each section of the row in the chart that is filled in.

I think this request is best implemented as an external plugin. The specialized logic for this will make the core too big

Hi guys,
This is exactly the feature I am looking for and am currently trying to create it out of a stacked bar chart, but @JSchrepp is right its a bit cumbersome. Having the ability to have it stackable with milestones would be very useful. So I guess integrating it into mixed charts would probably do this. Just a thought. Cheers guys!

Hey,
+1 for this extension.  I'm also at the point to visualize a Gantt chart. The hack via horizontal bar does work with some limitations:

Multiple bars in one row are very insufficient.
Problems with tooltip visibility.
Regards Thomas

@thomasjoscht
Not sure if this helps but here is what I did based on another post. The line chart made it a lot easier to grab tooltips. Im new to Javascript but it seems to be working out if I hide the Legend. https://jsfiddle.net/c_jarv/cb47aejn/

My use case is an overview of job executions of last 24 hours. With some additional coding I could solve my main problems. But solution still smells. Here is a simple sample https://jsfiddle.net/Lq3aptph/.

Multiple bars are possible. But this requires two additional datasets per bar. In summary there must exist 2 times maximum of bars per row (job in my case).
Tooltip problem with filtering requires also a custom tooltip with adjustment of some styles.

Problems:

Scalability is bad for increasing horizont (from last 24 Hours to 30 days f.e.)
Colorizing bars with some information (f.e. success=green, failed=red) is very had and requires much more datasets.
Label information of overlapped jobs (started out of horizont) could not be displayed correctly. Therefore only jobs started in horizont could be displayed now.

Further considerations could be to use a scatter chart instead. But my first tries were not sufficient.

+1 for a Gantt plugin, it's the only thing missing in my opinion, Google Charts gantt is terribly outdated

I'd like that as well. I don't want to drag in another library.


Extract SocketAdress logic from NameResolver

Motivation:
As discussed in #4529, NameResolver design shouldn't be resolving SocketAddresses (or String name + port) and return InetSocketAddresses. It should resolve String names and return InetAddresses.
This SocketAddress to InetSocketAddresses resolution is actually a different concern, used by Bootstrap.
Modifications:
Extract SocketAddress to InetSocketAddresses resolution concern to a new class hierarchy named AddressResolver.
These AddressResolvers delegate to NameResolvers.
Result:
Better separation of concerns.
Note that new AddressResolvers generate a bit more allocations because of the intermediate Promise and List<InetAddress>.

@trustin ping

Please note that when running DnsNameResolverTest on OSX 10.11.1 with Hotspot 1.8.0_66, I get the following warning:
00:43:58.295 [nioEventLoopGroup-2-1] WARN  io.netty.channel.nio.NioEventLoop - Unexpected exception in the selector loop.
java.io.IOException: Invalid argument
	at sun.nio.ch.KQueueArrayWrapper.kevent0(Native Method) ~[na:1.8.0_66]
	at sun.nio.ch.KQueueArrayWrapper.poll(KQueueArrayWrapper.java:198) ~[na:1.8.0_66]
	at sun.nio.ch.KQueueSelectorImpl.doSelect(KQueueSelectorImpl.java:103) ~[na:1.8.0_66]
	at sun.nio.ch.SelectorImpl.lockAndDoSelect(SelectorImpl.java:86) ~[na:1.8.0_66]
	at sun.nio.ch.SelectorImpl.select(SelectorImpl.java:97) ~[na:1.8.0_66]
	at io.netty.channel.nio.NioEventLoop.select(NioEventLoop.java:621) ~[netty-transport-4.1.0.CR1-SNAPSHOT.jar:4.1.0.CR1-SNAPSHOT]
	at io.netty.channel.nio.NioEventLoop.run(NioEventLoop.java:309) ~[netty-transport-4.1.0.CR1-SNAPSHOT.jar:4.1.0.CR1-SNAPSHOT]
	at io.netty.util.concurrent.SingleThreadEventExecutor$5.run(SingleThreadEventExecutor.java:742) [netty-common-4.1.0.CR1-SNAPSHOT.jar:4.1.0.CR1-SNAPSHOT]
	at io.netty.util.concurrent.DefaultThreadFactory$DefaultRunnableDecorator.run(DefaultThreadFactory.java:137) [netty-common-4.1.0.CR1-SNAPSHOT.jar:4.1.0.CR1-SNAPSHOT]
	at java.lang.Thread.run(Thread.java:745) [na:1.8.0_66]

Thanks! Let me take a look.

@slandelle this exception is nothing to worry about. It seems to happen here as well and can be ignored (most likely a jdk bug)

this exception is nothing to worry about. It seems to happen here as well and can be ignored (most likely a jdk bug)

@normanmaurer jdk bugs do tend to get me worried actually ;)

Haha fair enough... Just not anything related to your change ;)

Am 13.12.2015 um 11:08 schrieb Stephane Landelle<EMAIL_ADDRESS>this exception is nothing to worry about. It seems to happen here as well and can be ignored (most likely a jdk bug)
@normanmaurer jdk bugs do tend to get me worried actually ;)
—
Reply to this email directly or view it on GitHub.

Following InetSocketAddressResolver becoming non abstract, I wonder if it still makes sense to have DefaultAddressResolver and DnsAddressResolver? IMO, we could drop them.

@slandelle Fine by me. How about adding DnsNameResolver.asAddressResolver()?

How about adding DnsNameResolver.asAddressResolver()

Will do

asAddressResolver() pushed

@trustin @normanmaurer Is this PR fine for you now? Please let me know when I can squash the commits.

@trustin AddressResolver caching pushed

Ship it!

@slandelle will pull in once the test run is done... Anything else you want for CR1 ?

@trustin @normanmaurer commits squashed.

Anything else you want for CR1 ?

Yes, absolutely!

I'd like to provide support for hosts file, I can send a PR by tomorrow
I absolutely need to be able to tell appart DnsNameResolver cache hits and cache misses
WebSocket handshaker is pretty useless to me, see #4528. As an example, I have a AHC user who wants Kerberos auth for his WebSockets. If it's too late for fixing it, at least making pipeline manual upgrading easier is a must have. Encoder and Decoder can't be fetched by class as they are private. And fetching by name is very hacky. Would it be possible to make those classes public?

@normanmaurer Netkins timed out. This is unrelated to this PR (I only squashed the commits since last successful build).

Cherry-picked into 4.1 (6393506b97f71ed109548946b86db9b88bbfe44d) and master (541fe86fe01b8296d03acae4bd75fe9cc2e66103)
@slandelle thanks!

@normanmaurer Thanks for merging. More to come.


Can't connect: status code 401

Today I wanted to access the company disk from my home office, but this error has displayed instead of any mounting.
2020/10/15 14:44:39 Failure signing in My Cloud Home account: status code 401 logging in on https://prod1.mycloud.com/authrouter/oauth/token: unexpected status code

Hello matejmosko,
Just to tell you that's the first time I try to use this tool, and I get the same error. I'll investigate if I can help.
Is it the first time that you connect to your WD with this tool ?
I tried to put the credential in the config file and directly in the command line, same error.
Br,
Patrick

I tried also --debug, with no luck...

Hello,
small info, I created a vm in linode (cloud provider) and tried the procedure :
curl -sSfL https://github.com/mnencia/mchfuse/raw/master/install.sh | sudo sh -s -- -b /usr/local/bin
cat > mchfuse.conf << 'EOF'
username = "LOGIN" //of course updated
password = "PSW" //of course updated
EOF
mkdir /mnt/t
chmod 600 mchfuse.conf
mchfuse -c mchfuse.conf DEVICE_NAME /mnt/t
same result :
2020/10/25 19:43:04 Failure signing in My Cloud Home account: status code 401 logging in on https://prod1.mycloud.com/authrouter/oauth/token: unexpected status code
Any idea ? Any help is welcome ;-)
Br,
Patric;k

Same for me. They changed something in the API. I'm investigating.

Hello mnencia,
First thank you for the work you have done, and also for your quick answer.
I was looking to theses documentation (that you know) to see if I can troubleshoot.
https://developer.westerndigital.com/develop/wd-my-cloud-home/api/endpoint/get--v1-config.html
https://developer.westerndigital.com/develop/wd-my-cloud-home/api/authentication/get--v2-auth0-user.html
I'm too tired tonight but I'll try to have a look if I can find some thing (just trying to help) tomorrow.
Have a good day
Patrick

Any news?

can u please fix the api error  bro

I should have fixed it. Unfortunately, the method I've used before to obtain an auth0 client_id is not working anymore. So I had to include a client_id/client_secret pair inside the code. I do not like it, but it is the only way I've found.
I'm going to do a new release this evening, after reviewing the other open issues.


alert some error messages while using one etcd endpoint

{"ts":"2022-09-27T18:22:43.423+0800","logger":"etcd-client","caller":"v3@v3.5.4/retry_interceptor.go:62","msg":"retrying of unary invoker failed","target":"etcd-endpoints://0xc0007101c0/etcd:2379","attempt":0,"error":"rpc error: code = DeadlineExceeded desc = context deadline exceeded"}
panic: context deadline exceeded
func (rs *RpcServer) Start() {
if err := rs.server.Start(rs.register); err != nil {
logx.Error(err)
panic(err)
}
}
panic(err)
How did I solve this problem? Can anyone help fix this? thanks~

From the log message, it's context deadline exceeded. You should check whether the connection is normal first.

From the log message, it's context deadline exceeded. You should check whether the connection is normal first.
Is the host named etcd reachable in your machine ?

It's my mistake, thanks a lot~


Removing Tock 1.0 crates (timing?)

Currently, the repository contains both Tock 1.0 crates (libtock and its dependencies) and Tock 2.0 crates (libtock2 and its dependencies). The Tock 2.0 crates are not finished yet (tracked in #322), so it's not clear whether we should delete the Tock 1.0 crates now or later.
The obvious benefit of leaving the Tock 1.0 crates around is it makes it easier for users to continue to use them while they wait for the 2.0 crates to be ready. However, there's also a drawback to keeping the Tock 1.0 crates: they are slowing down the CI, which currently takes about 17 minutes.
Should we keep the crates until the 2.0 crates are ready, or can I go ahead and remove them? Alternatively, as a halfway measure, I can remove them from CI but keep them around in the repository (they shouldn't break if no-one touches them).
I'm personally in favor of removing them sooner rather than later.

I don't think a 17 minute CI is a reason to remove anything, that is pretty quick.
I think it makes sense to keep the 1.0 crates around until the 2.0 is ready. With efforts like https://github.com/tock/libtock-rs/pull/325 it's worth keeping around

For the record, I don't think I am going to put in the effort to polish / finish #325 . I don't think it is worthwhile to merge in its current state because most drivers have not been ported, and merging it also risks some out-of-tree user coming to depend on that 2.0 API which we do not plan to maintain in the future. I shared it as a PR in case anyone was interested, but it is at the point where it is temporarily usable for my downstream uses, and I am inclined to focus future effort on Johnathan's 2.0 work.

With #325 closed and https://github.com/tock/libtock-rs/issues/322 progressing it might be worth re-visiting this

With #325 closed and #322 progressing it might be worth re-visiting this

I want to update the toolchain to nightly-2022-01-20, or something close to it, in preparation to migrating to stable Rust. Doing so results in a lot of compilation errors because libtock_core still uses llvm_asm!.
I'm inclined to remove libtock_core and everything that depends on it (which is almost all of the Tock 1.0 crates), rather than updating it to use asm!.

Before we delete the crates, I think we should tag the repository version from before their removal. In fact, I think we should tag a version from before https://github.com/tock/libtock-rs/pull/365 is merged. That way, users who want access to the Tock 1.0 libtock-rs crates can grab the tagged version.
I propose creating the following GitHub release on this repository, please let me know what you think:
Tag
last-tock-1
Title
Tock 1 crate archive
Description
This is a snapshot of the source tree from shortly before the Tock 1 crates were removed.
This is a pre-release
Yes

That sounds like a good plan to me

I'm all on board with tagging, I have never had much traction in the past (see https://github.com/tock/libtock-rs/issues/193 and https://github.com/tock/libtock-rs/issues/192).
I think we should tag a 1.6 release, that way it's clear that libtock-rs 1.6 works with Tock 1.6. Then in the future we can tag releases timed up with Tock

I think we should tag a 1.6 release, that way it's clear that libtock-rs 1.6 works with Tock 1.6. Then in the future we can tag releases timed up with Tock

I'm not sure that libtock-rs releases should be named that way.
cargo is pretty tied to semantic versioning. Under semantic versioning, if we version the libtock-rs crates to match the Tock kernel, then we can't make breaking changes to libtock-rs' APIs without a major Tock revision. I don't think we want that limitation. Therefore we need some other versioning methodology, and I don't want to delay this issue on that discussion.
In the future, when we decide on a versioning scheme for libtock-rs, I think we should be able to go back and retroactively rename the release.

We can just do another decimal place for changes to libtock-rs.
So for example, Tock 1.6 is released with libtock-rs 1.6 as well. Then libtock-rs fixes a bug and releases 1.6.1. Then Tock fixes a bug and releases 1.6.1. We only tie in the first two numbers, as Tock won't break API inside a major release
I just think that last-tock-1 is a confusing name, and we are stuck with it forever. While v1.6 or 1.6 or release-1.6 is pretty clear.
Either way, I don't care that much. I still think tagging and removing the old stuff is fine

So for example, Tock 1.6 is released with libtock-rs 1.6 as well. Then libtock-rs fixes a bug and releases 1.6.1. Then Tock fixes a bug and releases 1.6.1. We only tie in the first two numbers, as Tock won't break API inside a major release

The problem isn't bugfixes, it's that libtock-rs will be making breaking changes, while Tock will not be.

I just think that last-tock-1 is a confusing name, and we are stuck with it forever. While v1.6 or 1.6 or release-1.6 is pretty clear.

I don't think we're stuck with it forever. If we decide to rename it, we can create a new release with the same commit ID but a different name and description.

Release created: https://github.com/tock/libtock-rs/releases/tag/last-tock-1
I'll start sending PRs to clean up the Tock 1 crates soon.

#372 removed the Tock 1.0 crates.


Move to JuliaStats?

Then other people can help with the maintenance.

I need admin privileges to do so.

Alex sudo'ed me.


File already exists.

I'm getting a file already exists when trying to run it.

Try doing a fresh resintall also is this on Windows? See instructions here: https://docs.unsloth.ai/get-started/install-update/updating

Yes and I followed those instructions to uninstall and reinstall and I get the same error. File already exists.

Yes and I followed those instructions to uninstall and reinstall and I get the same error. File already exists.

Go to the already exisiting file and delete it. And then try installing it again

Still getting the error even after deleting that temp folder and reinstalling everything. It did give more info this time though.


Add keyboard callbacks to TypeScript type definition

PR for #306.
I've added types for the other callback methods thatI can see defined in the native code. They appear to all take the same arguments, although I couldn't confirm that for Android.

https://github.com/APSL/react-native-keyboard-aware-scroll-view/releases/tag/v0.7.3


Hotkey to reload all inline scripts

When writing/testing inline scripts there is AFAIU two options to load fresh script version from filesystem:

Restart mitmproxy.
Remove and add again script via "Editing scripts menu".

It will be handy to have scripts reloaded with just one keystroke.

Good suggestions. We'd be happy to accept a PR for this! :smiley:


Resolving some bugs when user tries to signup/login.

Describe the bug
When users are signing up or logging in on the website, proper error handling and alert displays are not implemented.
Expected behavior

When I tried logging with a new email id which is not registered. The error is not handled properly, an alert should be prompting saying that the email is not registered.

After a successful log-in, an alert displaying the message "You are logged in" should appear. This alert needs to be designed to be visually appealing and in harmony with the website's overall theme. Currently, there is no alert, and the homepage opens abruptly.

When logging out successfully, there is no alert. However, the user should receive an alert even in this situation.

When a user inputs an incorrect password or captcha, the default browser alert is displayed. I would like to modify this alert to be more appealing and match the website's theme.

Additional context
Kindly assign this issue to me under GSSOC'24.

I add you in channel , then let's have more deep conversation about this issues raised by you !
Thanks

Hey i have already raised this issue before see #629

Hey i have already raised this issue before see #629

Hello @DEV-Soumya-Ranjan-Swain,
I have reviewed the issue you raised regarding the warnings that appear in the console during the signup process.
My concern, however, pertains to implementing appropriate error handling. For instance, when attempting to log in with an "unregistered" email, an error message should be displayed on the UI indicating that the email address is not registered. Additionally, there should be messages on the UI for when a user logs in or out.

@DEV-Soumya-Ranjan-Swain Their was a confusion occur , sorry for that 1
This contributor doing this work !
Kindly look into another problem !
@DEV-Soumya-Ranjan-Swain I request you to raise more major issues

Hi @thestarsahil,
Actually, I am already done with the implementation of the issue. Should I submit the PR now?
Additionally, I believe my issue and @DEV-Soumya-Ranjan-Swain 's issue are somewhat different. If he is also working on implementing "alerts" in the UI, there's no issue; you can merge his PR since he raised the issue before me. However, could you review my PR if he is not specifically working on the "alerts"? In that case, you might consider merging mine.

Hi @thestarsahil, Actually, I am already done with the implementation of the issue. Should I submit the PR now? Additionally, I believe my issue and @DEV-Soumya-Ranjan-Swain 's issue are somewhat different. If he is also working on implementing "alerts" in the UI, there's no issue; you can merge his PR since he raised the issue before me, and I am happy to contribute to more issues. However, could you review my PR if he is not specifically working on the "alerts"? In that case, you might consider merging mine.

Are you trying to implement toaster for something?

Hi @thestarsahil, Actually, I am already done with the implementation of the issue. Should I submit the PR now? Additionally, I believe my issue and @DEV-Soumya-Ranjan-Swain 's issue are somewhat different. If he is also working on implementing "alerts" in the UI, there's no issue; you can merge his PR since he raised the issue before me, and I am happy to contribute to more issues. However, could you review my PR if he is not specifically working on the "alerts"? In that case, you might consider merging mine.

Are you trying to implement toaster for something?

Yes! I have implemented toaster for a when user sign-in /sign-out and when captcha/password is wrong or the email is not registered. Here are some screenshots:

@DEV-Soumya-Ranjan-Swain please let us know if you are working on the same...
@thestarsahil  Should I raise the PR?


Add kube-state-metrics and kube-apiserver example exporter servicemonitor

This adds:

kube-state-metrics deployment, service and servicemonitor as an example
ServiceMonitor for the default/kubernetes kube-apiserver service

Are "examples" like this for specific exporters wanted? I also have other ServiceMonitor (+ deployment) manifests, for ceph-exporter, mysqld-exporter, AlertManager and Prometheus.

@brancz On it. I'm writting a doc for "Running exporters" (and scraping them) using prometheus-operator.

test this please

@brancz I've had some time now and added two documentation pages that cover:

Running Exporters
Monitoring Kubernetes

Note: I'm still working on this, but haven't gotten time for it yet. Will definitely work on this this weekend.

No worries if you can't find time on it! You already put a lot of good work into this, if you want to finish it perfect, if not we're happy to continue it. :slightly_smiling_face:

I added your feedback. For now I removed the exporters and will add my exporters manifests in another PR (at least the ceph-exporter, mysql-exporter and nginx-vts-exporter).

I added your last feedback and squashed everything together.

Awesome! Nice job! Thanks for baring with me :slightly_smiling_face:


feat: blockifier update

Bump blockifier to a version (0.4.0-rc9.2) which supports Cairo 2.4.0

@tarrencev can you help out on dojo-lang side of things ?

Locally I've all the tests that pass, but on the CI, the dojo-lang expand is failing. @tarrencev or @kariy do you have any idea of what's I may be missing?
I've run again with CAIRO_FIX_TEST to be sure.

Lets adjust katana updates just enough to support v0.5 rpc. For now, we can ignore stuff that are only relevant in the v0.6 rpc spec.


Use Sphinx directives in docs

The idea of this is to document all classes in Wagtail using this syntax:
.. class:: MyClass

    What the class does

    .. method:: my_method(a, b)

        What the method does

Using this syntax makes read the docs give it a nicer style, and also allows classes and attributes to be referenced throughout the docs like: :class:MyClass and :method:MyClass.my_method.

Going to break this down a bit (too much cleanup mixed with this commit)

I don't understand good this.

@SalahAdDin It'll lead to better cross-linking in our docs I think


Chef::Exceptions::ResourceNotFound from apt cookbook with 12.10.1

Cookbook version
12.10.1
Chef-client version
12.8.1
Platform Details
> cat /etc/*-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=14.04
DISTRIB_CODENAME=trusty
DISTRIB_DESCRIPTION="Ubuntu 14.04.3 LTS"
NAME="Ubuntu"
VERSION="14.04.3 LTS, Trusty Tahr"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 14.04.3 LTS"
VERSION_ID="14.04"

> uname -mrs
Linux 3.13.0-74-generic x86_64

running in AWS
Scenario:
We use the apt cookbook to install logstash. Today that started failing. Locking the compat_resource cookbook to 12.9.1 fixes the issue.
Steps to Reproduce:

create a cookbook that depends on apt (we have no specified version) so depends 'apt'
add the following to default.rb

apt_repository 'elasticsearch' do
  uri 'http://packages.elastic.co/logstash/2.3/debian'
  components %w(stable main)
  distribution ''
  key 'http://packages.elastic.co/GPG-KEY-elasticsearch'
  action :add
end

Expected Result:
logstash is installed
Actual Result:
ERROR: apt_repository[elasticsearch] (the-cookbook::default line 10) had an error: Chef::Exceptions::ResourceNotFound: resource execute[apt-get update] is configured to notify resource execute[apt-cache gencaches] with action run, but execute[apt-cache gencaches] cannot be found in the resource collection. execute[apt-get update] is defined in /var/chef/cache/cookbooks/apt/providers/repository.rb:156:in `block in class_from_file'

NB if we ssh onto the machine in question and run apt-cache gencaches the command succeeds

dup, fixed by 12.10.2


Missing documentation for publishing to production environment

Hello pwa-studio developers , I want a simple release production documentation. For example, execute the package.json script.

Hi @threescs,
You can add the following command to you storefront package.json "start": "NODE_ENV=production node server.js",


Fix Windows Clang-CL build with Bazel@HEAD

With WORKSPACE being disabled at HEAD, all dependencies must be specified in MODULE.bazel
Fixing https://github.com/google/glog/issues/1127

Thank you for the PR!

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.22%. Comparing base (b3b9eb9) to head (5606652).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1131      +/-   ##
==========================================
+ Coverage   64.03%   67.22%   +3.19%
==========================================
  Files          20       14       -6
  Lines        2580     1907     -673
  Branches      889      696     -193
==========================================
- Hits         1652     1282     -370
+ Misses        663      447     -216
+ Partials      265      178      -87

see 17 files with indirect coverage changes


FAForever client update 2022.8.0

Describe the bug
I installed FAF client according to https://youtu.be/Rv3ZXA4FNFk and it worked like a charm!! A few days ago I tried to play the game and it says "update to version 2022.8.0". The update gets stuck at 100% and nothing else happened. I tried downloading the 2022.8.0 tar from releases and start the game from console but this messages came out " java.lang.ClassNotFoundException: install4j.com.faforever.client.Main"

OS
Linux opensuse Tumbleweed
Things that I tried:
1)changing java version
2) donwloading realese 2022.8.0 and from console launching the game
Any ideas?
Gastón

the Auto download doesn't work well with Linux best thing to do is to do it get the new version from the GitHub

Auto update is broken for linux. Duplicate of #1026 , you need to just download and install from the release here or extract the tar.gz that is downloaded.


Adjust Add Liquidity Proportional to expect referenceAmountIn instead of bptOut

Current implementation of Add Liquidity Proportional expects bptOut as input (since it's an exactOut operation).
This causes a problem on the FE, because users input a referenceAmountIn in the UI.
Based on referenceAmountIn and a poolState (fetched on-chain by the FE), bptOut is estimated and then used as input to the SDK query.
SDK query will then rely on another poolState (fetched on-chain by the SDK) to perform that query.
Any eventual mismatch between both poolStates can cause the amountsIn calculated to be slightly different than the ones provided by the user.
This is an issue in the context of inputs being the max user balance, since any slightly higher amount will cause the transaction to revert.
A proposed solution to reduce the surface area for mistakes is to expect referenceAmountIn as the query input, so a single poolState is used for the whole calculation.

I found a lot of issues after going deeper into this solution.
Proportional math operations have inherent rounding issues and internalizing this complexity does not seem to be ideal.
Taking a step back and thinking about the real proposed solution, which is to prevent the possibility of using different pool states (internally vs externally), I believe that exposing a helper that provides a bptOut without the need to query their own on-chain state should be enough.
From there, bptOut can then be used as input to the AddLiquidityProportional query, without the need to update the input to expect any referenceAmount as initially suggested.
This way we keep this complexity out of the SDK, following the standard of being a straightforward interface to interact with the SC.
I'll move on with that implementation, but I though it would be interesting to share my thoughts here and gather feedback sooner.

After talking to Daniel, we reached the conclusion that the first approach of exposing a referenceAmount as input for AddLiquidityProportional makes more sense because it enforces that all math is done in one single poolState.
Approach 2 would query on-chain state twice, increasing the chances of state changing in between calc and query.
On top of that, SDK is rounding down the input, so the bptOut calculated favors leaving some dust behind instead of risking a tx revert due to amountsIn being slightly above user input.


Phar: 'Allocine\Twigcs\Console\Application' not found

twigcs version: 3.2.2, 3.2.1, 3.2.0 (phar)
testen on version 3.2.2 windows (php 7.3) and linux (php 7.1)
wget https://github.com/friendsoftwig/twigcs/releases/download/v3.2.2/twigcs.phar
php twigcs.phar --version

PHP Fatal error:  Uncaught Error: Class 'Allocine\Twigcs\Console\Application' not found in phar:///home/username/twigcs.phar/bin/twigcs:12
Stack trace:
#0 /home/username/twigcs.phar(12): require()
#1 {main}
  thrown in phar:///home/username/twigcs.phar/bin/twigcs on line 12

Hello and thanks for spotting this! I rebuilt the phar and it seems to be working. Versions affected were 3.2.0, 3.2.1 and 3.2.2.


Contract generation

Hi,
I'm in the planning phase for developing an exporter from OpenCensus.io's Go library to Azure's AppInsights, and have started by looking through this SDK.
One thing that stood out in my first pass is that there are a lot of auto-generated contracts that do things like enforcing these limits:
https://docs.microsoft.com/en-us/azure/application-insights/app-insights-api-custom-events-metrics#limits
How was the contracts package generated? I will likely need to replicate this functionality.
Thanks!

Maybe the solution is to import the contracts package from this SDK.

The contracts are generated by an internal tool from the public endpoint specs.  Why the tool is internal is beyond me, but the transformations are generally fairly straightforward.
You could use the contracts package directly, although I'm not sure why you wouldn't just use the appinsights package so you don't need to reinvent the interface with the data collector, retry, backoff, throttling handling, queueing and so on.

That is a good point, will lean on the appinsights package. Thanks for the input.

Hi @freddygv, curious if you've continued to work on this? We're thinking about how to better support OpenCensus in App Insights. Thanks!
cc @devigned @nikmd23

@freddygv if this is something that is still of interest to you, we'd love your feedback and help as we drive forward.

@joshgav I submitted a proposal for this for the Google Summer of Code but it wasn't accepted, so I never got started on it (and unfortunately cannot make the time to contribute now).
Great to hear that Microsoft joined the project though!


model_ds.time uses start time of period while results are at end time

The time coordinates of model_ds refer to the start times of each stress period, while we often much more interested in the outcomes at the end of each stress period. The times should be in sync with the ones configured in the OC package.

I think the times are already at the end of the stress period, just like you would like...

The steady state stress period is added before the supplied start-time, which may have confused you.

You were right! Thank you so much.


<EMAIL_ADDRESS>download via aria2 failed

ERROR Download failed! (Error 29) The remote server was unable to handle the request due to a temporary overloading or maintenance
ERROR https://celestia.space/cc/celestia-win-161#celestia-161.exe
    referer=https://celestia.space/cc/
    dir=D:\Users\tim\Scoop\cache
    out=celestia#1.6.1#https_celestia.space_cc_celestia-win-161_celestia-161.exe

the website is up, i checked
maybe it has something to do with the cloudflare ddos protection?

More like aria2 just trips up on alot of things. I can't test packages with it enabled cause it often fails to download or grabs the wrong file, if it is not a direct link.
I think you can disable it with
scoop config disable aria2

it still fails with aria2 disabled, the link seems to be dead

3ae42a6f06f1d3d7c4287ea1cb9a94ba58b2a10a


Roll engine 54f7c51339ae..3e6dc02ad62e (1 commits)

https://github.com/flutter/engine/compare/54f7c51339ae...3e6dc02ad62e
git log 54f7c51339ae7d86a983b5850b14dafe4e2bd90a..3e6dc02ad62ef9fa02660f5fb0bb114414e282d4 --no-merges --oneline
3e6dc02ad Roll src/third_party/skia ced2b79d00f9..847d55be4e62 (3 commits) (flutter/engine#8630)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (cbracken@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Non-functional reference-style links

Update: This is no longer the issue, however there is still a problem when the reference is defined in another description block. See https://github.com/apiaryio/snowcrash/issues/55#issuecomment-58488125

Following blueprint:
# API
This is [an example][id] reference-style link.

[id]: http://example.com "x"

renders as:
_version: 1.0
metadata:
name: API
description: "This is [an example][id] reference-style link.\n\n["
resourceGroups:

Refer to Markdown Link Syntax

Migrated to apiaryio/drafter#647


Fix typos

Sorry... should have finished reading before submitting the last one. ;)

Maybe we should write a proposal to introduce the Equitable protocol 😂

😀


Laravle-share using

Hello,
I have some troubles implementing laravel-share. I try using this code:
Share::currentPage()->facebook();
but I'm not sure how and where use it, if I using in controller just pass a var? or I need to use directly in view? when I use like:
  {{ Share::currentPage()->facebook() }}
but show this:

laravel-share::laravel-share-fa4.facebook

thankyou for your help

Wha @whoisthisstud is saying should work.

It doesn't display anything @whoisthisstud @jorenvh

This is what I have and what is working for me - "jorenvanhocht/laravel-share": "^3.2",
{!! Share::page( route('your_route_name'), 'Message Content')->twitter()->facebook()->pinterest() !!}

When I dd() this code the above code is also perfectly displaying result but when I add this code into blade file to render it on page It displays blank (no result)


Check multicopter navigation logic

From here:
https://groups.google.com/forum/#!topic/px4users/5vYicnpcmWo
Is there an error in mc_pos_control_main.cpp?
https://github.com/PX4/Firmware/blob/master/src/modules/mc_pos_control/mc_pos_control_main.cpp#L993
 /* copter is too far from trajectory */
						/* if copter is behind prev waypoint, go directly to prev waypoint */
						if ((pos_sp_s - prev_sp_s) * prev_curr_s < 0.0f) {
							pos_sp_s = prev_sp_s;
						}

						/* if copter is in front of curr waypoint, go directly to curr waypoint */
						if ((pos_sp_s - curr_sp_s) * prev_curr_s > 0.0f) {
							pos_sp_s = curr_sp_s;
						}

In the code , the (pos_sp_s - prev_sp_s) is equal with prev_curr_s  , so the programme while never less than 0.0f.
Is this?

I don't see a problem here. The vector (pos_sp_s - prev_sp_s) will always be parallel to prev_curr_s but it's direction can be opposed in which case the dot product of the two vectors will be negative.

Indeed, closing.

In this line ：https://github.com/PX4/Firmware/blob/master/src/modules/mc_pos_control/mc_pos_control_main.cpp#L930
math::Vector<3> pos_sp_s = curr_sp_s;
And in this line:
https://github.com/PX4/Firmware/blob/master/src/modules/mc_pos_control/mc_pos_control_main.cpp#L940
math::Vector<3> prev_curr_s = curr_sp_s - prev_sp_s;
So, (pos_sp_s - prev_sp_s) * prev_curr_s = (curr_sp_s - prev_sp_s) * prev_curr_s  = prev_curr_s   * prev_curr_s   = 0 .
and
If you want to find where the copter is,I think  the "pos_sp_s"  should be "pos_s“ , is it right?
@tumbili
@LorenzMeier


Per-request timing information

We are interested in logging granular HTTP timing info on a per-request basis (non-aggregated) to a custom log destination along with other request-specific metadata.  We are currently planning on measuring this through a custom filter and logging on destruction.  However, it may be desirable to perform these measurements in a general way in the open source and make them available for consumption by HTTP filters and access logs (maybe through the RequestInfo object).  The durations that we are interested in are all timed from the request start to particular request events:

entire request sent to upstream
entire response received from upstream
response received by the downstream client (acked)

Is there an appetite for this sort of per-request timing information to be included and accessible in Envoy?
cc @htuch

In general I think this would be useful for general open source. Also, depending on what precise definitions you want for these timings, it's not going to be possible to do in a filter. Specifically, filters are not always aware of lack of window updates and/or TCP level pauses that cause buffering.
For the three timings that you have above, can you spell out a bit more how you plan on timing them (setting aside how Envoy works for a second). Also, we already have x-envoy-upstream-time headers and startTime() in RequestInfo (which when access logs are written gives you total end to end to end time, minus end of response stream buffering and TCP buffering).

Great!  Agreed. The filter implementation would've been a pretty inaccurate approach. Here are the definitions we would like to use (please forgive me if I butcher any terminology/concepts here) and my understanding of when the timestamps would be captured:

request sent to upstream - stamped just after the last byte of the request is passed to the kernel's send TCP buffer.
response received from upstream - stamped just after the last byte of the request is read by Envoy from the receive TCP buffer.
response sent to downstream client (updated from above) - I think it makes sense to stamp just after the last response byte is written to the send TCP buffer going to the downstream client rather than trying to get the time when the last ack comes back, which requires polling the kernel.

I've looked into both the x-envoy-upstream-time headers, which will certainly be useful for our timing output.  Currently, we were planning on generating the above durations by differencing their timestamps with the startTime() from RequestInfo.  IIUC, this startTime() is stamped at the time when the first header bytes are received, roughly?  It would be a bit nicer to be able to difference against the time when the TCP handshake has just finished, but I'm not sure if this is easy or possible.
In regards to the current way of doing end-to-end time measurement, I was curious about the time that the log() function is actually called on the AccessLog::Instance object.  It appears to be called in the ActiveStream destructor, but because the ActiveStream is destroyed through deferred deletion, doesn't that mean that its destructor will be called at a later point in time, making it a strange point at which to measure the timestamp marking the end of the request?  Is this timing difference negligible?
Please correct me if anything I've mentioned seems fishy or incorrect.

This is really complicated unfortunately. I will drop some comments in here prior to our meeting later today and then we can update this issue for public visibility of that discussion.

request sent to upstream - stamped just after the last byte of the request is passed to the kernel's send TCP buffer.

Loosely, we track this today via the downstream_request_complete_time_ in the router. This forms the start time for the x-envoy-upstream-time header. Obviously, this is not accurate, as after the request is fully processes and forwarded, a bunch of things can take further time: HTTP/1.1 connection pool acquisition, tail flow control (meaning, we got past flow control for most of the upload, but there might be flow control that blocks the last bit of the request from going on the wire). In order to make this fully accurate to the TCP level, it will be a tremendous amount of work. Basically, it will involve tracking request data all the way down to the byte level, and firing callbacks when a particular byte location in the output byte stream actually gets written. This is technically possible, but it's not clear to me that the better fidelity is worth it in the comment case (especially because we have stats on the # of flow control events).

response received from upstream - stamped just after the last byte of the request is read by Envoy from the receive TCP buffer.

This is effectively the x-envoy-upstream-time header. There is no real flow control on the receive path, so when the last byte comes into the TCP buffer modulo some small delta in instructions running, it will fire the event.

response sent to downstream client (updated from above) - I think it makes sense to stamp just after the last response byte is written to the send TCP buffer going to the downstream client rather than trying to get the time when the last ack comes back, which requires polling the kernel.

Loosely, this is the time that is snapped in the access log today. To get more accuracy, you have all the problems with the first item above. You need per byte tracking and callbacks. Also, if you want to track ACK data, this becomes even more complicated. (As an aside, is this even possible via API? What API? Just curious). Again it's not clear to me that the mountain of work required to make this happen is worth it.

IIUC, this startTime() is stamped at the time when the first header bytes are received, roughly? It would be a bit nicer to be able to difference against the time when the TCP handshake has just finished, but I'm not sure if this is easy or possible.

This is actually when all header bytes are received. This is at stream level and is somewhat unrelated to TCP handshake complete since for HTTP/2 and HTTP/1 keep alive the TCP handshake will have happened a long time previous. Either way, tracking TCP handshake complete and TLS handshake complete times would not be difficult.

doesn't that mean that its destructor will be called at a later point in time, making it a strange point at which to measure the timestamp marking the end of the request? Is this timing difference negligible?

IMO the time delta is negligible. It will happen in the next event loop iteration which is going to be in the microsecond granularity level.
cc @alyssawilk

" Basically, it will involve tracking request data all the way down to the byte level, and firing callbacks when a particular byte location in the output byte stream actually gets written. This is technically possible, but it's not clear to me that the better fidelity is worth it in the comment case "
If it isn't done now, I believe it will be done in a few years.    These metrics are super useful for low-level latency analysis and crazy follow-up optimizations which some companies (cough) care a lot about :-)  So +1 to interest.
Worth noting for HTTP-TCP you can get the last-acked-time fairly easily but for H2 it's virtually impossible.  You can do some clever things with underlying socket rtt measurements and H2 pings to estimate both kernel/network time plus additional buffer bloat - happy to brainstorm options or connect you to the folks who have done this at Google.

If it isn't done now, I believe it will be done in a few years. These metrics are super useful for low-level latency analysis and crazy follow-up optimizations which some companies (cough) care a lot about :-) So +1 to interest.

I agree it's interesting and am not opposed to it at all, was just explaining that it will be a huge amount of work. If we want to go down this route we should do a full design doc.

Meeting resolution:

It is of general interest to add more granular timing info to RequestInfo for consumption by filters and access logs.  We will use Matt's suggested sources to populate the timestamps mentioned above.
More accurate timing information is a somewhat orthogonal problem to the above.  While there does seem to be long-term interest in getting more accurate timestamps, this doesn't seem to be a high priority.

Closing because this additional info has been added to RequestInfo.


[ UI ON WINDOWS ] Table not resizing with multiple parameters

Problem with table not resizing with multiple parameters

This is a layout problem that has been very difficult to fix. All of the normal methods of forcing a layout to re-calculate itself don't seem to work. it's almost like it's a visual rendering bug, but I find it hard to believe that Qt would have issues like this without a reason behind it.


Providing vpc_subnet_id, but  failing one get_default_subnet

I think it's failing because my account has neither a default vpc nor a default subnet - even though I do provide a vpc id and subnet id
My molecule yaml looks like this (redacted IDs):
---
dependency:
  name: galaxy
driver:
  name: ec2
platforms:
  - name: instance
    image_id: ami-abc123abc123
    instance_type: t2.micro
    region: us-east-1
    vpc_id: vpc-abc123abc123
    vpc_subnet_id: subnet-abc123abc123
    tags:
      - Name: molecule_instance
provisioner:
  name: ansible
verifier:
  name: ansible

stderr_lines": [
    "Traceback (most recent call last):",
    "  File \"/Users/nmarks/.ansible/tmp/ansible-tmp-1639218466.1749592-46512-137934100112825/AnsiballZ_ec2_instance.py\", line 107, in <module>",
    "    _ansiballz_main()",
    "  File \"/Users/nmarks/.ansible/tmp/ansible-tmp-1639218466.1749592-46512-137934100112825/AnsiballZ_ec2_instance.py\", line 99, in _ansiballz_main",
    "    invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)",
    "  File \"/Users/nmarks/.ansible/tmp/ansible-tmp-1639218466.1749592-46512-137934100112825/AnsiballZ_ec2_instance.py\", line 47, in invoke_module",
    "    runpy.run_module(mod_name='ansible_collections.amazon.aws.plugins.modules.ec2_instance', init_globals=dict(_module_fqn='ansible_collections.amazon.aws.plugins.modules.ec2_instance', _modlib_path=modlib_path),",
    "  File \"/Users/nmarks/.pyenv/versions/3.9.2/lib/python3.9/runpy.py\", line 210, in run_module",
    "    return _run_module_code(code, init_globals, run_name, mod_spec)",
    "  File \"/Users/nmarks/.pyenv/versions/3.9.2/lib/python3.9/runpy.py\", line 97, in _run_module_code",
    "    _run_code(code, mod_globals, init_globals,",
    "  File \"/Users/nmarks/.pyenv/versions/3.9.2/lib/python3.9/runpy.py\", line 87, in _run_code",
    "    exec(code, run_globals)",
    "  File \"/var/folders/72/cmp8x83n3js_spr_3hvvdgch0000gn/T/ansible_ec2_instance_payload_1wo209qr/ansible_ec2_instance_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_instance.py\", line 1910, in <module>",
    "  File \"/var/folders/72/cmp8x83n3js_spr_3hvvdgch0000gn/T/ansible_ec2_instance_payload_1wo209qr/ansible_ec2_instance_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_instance.py\", line 1886, in main",
    "  File \"/var/folders/72/cmp8x83n3js_spr_3hvvdgch0000gn/T/ansible_ec2_instance_payload_1wo209qr/ansible_ec2_instance_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_instance.py\", line 1794, in build_filters",
    "  File \"/var/folders/72/cmp8x83n3js_spr_3hvvdgch0000gn/T/ansible_ec2_instance_payload_1wo209qr/ansible_ec2_instance_payload.zip/ansible_collections/amazon/aws/plugins/modules/ec2_instance.py\", line 1446, in get_default_subnet",
    "TypeError: 'NoneType' object is not subscriptable"
  ],

I was able to reproduce the problem. I tested the same config on a subnet in an account that has a default vpc/subnet and it worked.  Then I moved back to the subnet in the account without a default vpc/subnet and it failed. then i created a default vpc/subnet and it worked.
I think the AWS best practices recommend deleting default VPC/subnets, so this should probably only look for them as a fallback

I too am running into this issue with the driver. I have created a default vpc which has worked around the problem but it would be great to see if the process can be skipped if VPC information is provided, as it is a best practice to delete all default vpcs (especially if they are not in use).

same issue here.

having the same issue

What I have just found on 0.4 is that you are supposed to only pass vpc_subnet_id if you want to use a custom subnet id. Adding on top vpc_id will make it fail, because it will assume that you want to auto-create it in that case. Just removing vpc_id did work for me.


Use in-memory db for tests (makes them faster)

All commits signed-off

jenkins: test this please


DEV-Panoptic Quality Metric Integration into solafune-tools

Implemented Panoptic Quality metrics for segmentation/mask scoring. This PQ metric Authored by @torumitsutake

Looks good! Can you also update the readme.md file? You could add a line the first paragraph: Oct 2024: Added a panoptic metric and then follow-up at the bottom of the readme with an example of how to call it and what expected output is?

Thank you for your contribution!


group image doesn't appear when unauthenticated

When one is unauthenticated, the group image doesn't appear on the group page. Ostensibly, this is because the group object isn't being made available to the template.

Verified in beta


Bug Report: All Music Is Muted

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
"Windows (x86_64)"
What version of the game are you using? Look in the bottom left corner of the main menu.
All versions, as far as I know.
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
Unknown as to how the bug happened.
Please describe your issue. Provide extensive detail and images if possible.
Whenever I play any FNF mods that use the Kade Engine, all music is muted. Sound effects, most menu music, and even in-game music and voices. It's not the volume mixer, as in-game videos and animations still play properly and have sound. I believe it may have been a keybind of some sort, but I haven't been able to find it. The following video is using the "VS Sonic.exe" mod as a showcase for the audio problems. I would've uploaded it directly, but I didn't want to risk compressing it too much.
https://youtu.be/GhkTvnn9Odk
Thank you for taking the time to read this.

you clicked 0, click it again

you clicked 0, click it again

That wasn't the problem, sorry if I didn't mention that beforehand. As I said before, I used all of the volume related keybinds I knew of, even 0.


Problems with Instructions

The layout of the bicep file disagrees with how MS specify it should be done from https://docs.microsoft.com/en-us/azure/templates/microsoft.storage/storageaccounts?tabs=bicep
Also, if you try to run New-AzResourceGroupDeployment -Name firstbicep -ResourceGroupName myResourceGroup -TemplateFile $bicepFile -verbose -debug  as stated you get an error in PowerShell of
New-AzResourceGroupDeployment: Unexpected character encountered while parsing value: r. Path '', line 0, position 0.
Where as if you use Az in PowerShell but run the command as:
az deployment group create --name firstbicep --resource-group myResourceGroup --template-file $bicepFile
Not as specified in the document it works.

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

ID: f7fb6c84-311d-56ad-f810-2604dda37974
Version Independent ID: ab432d50-83e3-f83b-bb6f-80c8f06a8871
Content: Tutorial - Create & deploy Azure Resource Manager Bicep files - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/bicep-tutorial-create-first-bicep.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

Hi,
Sorry yep realised the order of the properties doesn't make a difference.
However, I have PowerShell version
PS C:\Users\Paul.Jemmett> $PSVersionTable.PSVersion
Major  Minor  Patch  PreReleaseLabel BuildLabel

7      1      3
And PS C:\Users\Paul.Jemmett> bicep --versionBicep CLI version 0.3.255 (589f0375df)
And when I have my own bicep template I created when I run:
az deployment group create it deploys successfully with no issue
When I use:
New-AzResourceGroupDeployment
I get New-AzResourceGroupDeployment: Unexpected character encountered while parsing value: p. Path '', line 0, position 0.
DEBUG: AzureQoSEvent: CommandName - New-AzResourceGroupDeployment; IsSuccess - False; Duration - 00:00:00.0317052; Exception - Newtonsoft.Json.JsonReaderException: Unexpected character encountered while parsing value: p. Path '', line 0, position 0.
at Newtonsoft.Json.JsonTextReader.ParseValue()
at Newtonsoft.Json.Linq.JObject.Load(JsonReader reader, JsonLoadSettings settings)
at Newtonsoft.Json.Linq.JObject.Parse(String json, JsonLoadSettings settings)
at Newtonsoft.Json.Linq.JObject.Parse(String json)
at Microsoft.Azure.Commands.ResourceManager.Cmdlets.SdkClient.ResourceManagerSdkClient.CreateBasicDeployment(PSDeploymentCmdletParameters parameters, DeploymentMode deploymentMode, String debugSetting)
at Microsoft.Azure.Commands.ResourceManager.Cmdlets.SdkClient.ResourceManagerSdkClient.ExecuteDeployment(PSDeploymentCmdletParameters parameters)
at Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation.NewAzureResourceGroupDeploymentCmdlet.b__33_0()
at Microsoft.WindowsAzure.Commands.Utilities.Common.AzurePSCmdlet.ConfirmAction(Boolean force, String continueMessage, String processMessage, String target, Action action, Func1 useShouldContinue)    at Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation.NewAzureResourceGroupDeploymentCmdlet.ExecuteCmdlet()    at Microsoft.WindowsAzure.Commands.Utilities.Common.CmdletExtensions.<>c__31.b__3_0(T c)
at Microsoft.WindowsAzure.Commands.Utilities.Common.CmdletExtensions.ExecuteSynchronouslyOrAsJob[T](T cmdlet, Action`1 executor)
at Microsoft.WindowsAzure.Commands.Utilities.Common.CmdletExtensions.ExecuteSynchronouslyOrAsJob[T](T cmdlet)
at Microsoft.WindowsAzure.Commands.Utilities.Common.AzurePSCmdlet.ProcessRecord();
DEBUG: Finish sending metric.
DEBUG: 12:29:44 - NewAzureResourceGroupDeploymentCmdlet end processing.
DEBUG: 12:29:44 - NewAzureResourceGroupDeploymentCmdlet end processing.

From: Jonathan Gao @.>
Sent: 22 April 2021 16:07
To: MicrosoftDocs/azure-docs @.>
Cc: pauljemmett @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Problems with Instructions (#74205)
@pauljemmetthttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpauljemmett&data=04|01||1311ec9f1d3d4f6c86db08d905a04db9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637547008320567454|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=IzKDVmHioAxEz5kAxIaVzvS5fuT0Yx0yUTzcxVPRNO4%3D&reserved=0 - thank you for your comments. Can you share some details of the layout differences? Are you talking about the order the properties? Regarding the PowerShell deployment problem, can you run "bicep --version" to verify that you have Bicep CLI version 0.3.1 or later? To install the latest, see the "Install manually" section in https://docs.microsoft.com/azure/azure-resource-manager/templates/bicep-install?tabs=azure-powershell#install-bicep-clihttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fazure%2Fazure-resource-manager%2Ftemplates%2Fbicep-install%3Ftabs%3Dazure-powershell%23install-bicep-cli&data=04|01||1311ec9f1d3d4f6c86db08d905a04db9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637547008320567454|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=xK%2FgT6ZIk9JWwtKARJCg244IsnrxEh1TNsc6Il6DkZk%3D&reserved=0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F74205%23issuecomment-824924363&data=04|01||1311ec9f1d3d4f6c86db08d905a04db9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637547008320577453|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=8N7JJW5dAUAEsV7kXVt1e2Kjs85k%2FlwHF8tIvNPkCyU%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALVARJXORN4LTR25BEGXGPDTKA3R3ANCNFSM43MLNHXQ&data=04|01||1311ec9f1d3d4f6c86db08d905a04db9|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637547008320577453|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=bZgFvctq8vg2c8utAXxnol1w5ySamjgj8zdPmzhBvt4%3D&reserved=0.

@pauljemmett - what is the value you passed to -templateFile?  Can you change directory to the folder where the bicep file is located and run the cmdlet with only the bicep file name (without specifying the path to the Bicep file)?

Ok so the issue is PowerShell 7.1.3 if I run in the latest version of PowerShell 5.1 it runs fine without issue
The command I was running was (I've replaced my company name with Company Name place holder:
$networkBicepFile = "C:\Users\Paul.Jemmett\OneDrive - Company Name\Azure\POC\companyName_vnetsubnetnsgTemplate.bicep"
New-AzResourceGroupDeployment -Name "AzureNetworkingDeployment" -ResourceGroupName "abc-uks-prod-rg-networking-001" -Templatefile $networkBicepFile
on 7.1.3 it fails on 5.1 it works - might be an issue on my 7.1 install as when I run:
Install-Module -Name Az -Scope CurrentUser -Repository PSGallery -Force -AllowClobber
I get
… talledPackages = PackageManagement\Install-Package @PSBoundParameters
|                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| Administrator rights are required to install or update. Log on to the computer with an account that
| has Administrator rights, and then try again, or install by adding "-Scope CurrentUser" to your
| command. You can also try running the Windows PowerShell session with elevated rights (Run as
| Administrator).
Even though I am an Administrator and the PowerShell window was run with elevated rights.

From: Jonathan Gao @.>
Sent: 27 April 2021 15:01
To: MicrosoftDocs/azure-docs @.>
Cc: pauljemmett @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Problems with Instructions (#74205)
@pauljemmetthttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpauljemmett&data=04|01||8e132c57ef024a2a4ca408d90984fcb5|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637551289039167043|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EzOy6l118ULzcI%2FD%2B4WugAm8JHGYJ0SEIGuQcstDQTs%3D&reserved=0 - what is the value you passed to -templateFile? Can you change directory to the folder where the bicep file is located and run the cmdlet with only the bicep file name (without specifying the path to the Bicep file)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F74205%23issuecomment-827630584&data=04|01||8e132c57ef024a2a4ca408d90984fcb5|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637551289039177039|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=GvYxrWH%2BsTdGJFfjc7bXsaybm7Ws6NnOIsU%2FZ3FDdFk%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALVARJUURCQAPUQGMJYGNXTTK27UNANCNFSM43MLNHXQ&data=04|01||8e132c57ef024a2a4ca408d90984fcb5|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637551289039177039|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=fs8kH9u6EhcgCI3VR%2FxcKh4Sq%2F74GJTIv9CSVbfgtV0%3D&reserved=0.

Can you double check that you have Az 5.6.0 or later in your PowerShell 7 environment?
Get-InstalledModule -Name Az

@pauljemmett - is this issue resolved?

#please-close


Update limit.py

[ ] If creating a new file :

[ ] added links to it in the README files ?
[ ] included tests with it ?
[ ] added description (overview of algorithm, time and space compleixty, and possible edge case) in docstrings ?

[x] if done some changes :

[x] wrote short description in the PR explaining what the changes do ?
[ ] Fixes #[issue number] if related to any issue

[ ] other

More Pythonic in my view

Pull Request Test Coverage Report for Build 905

3 of 4 (75.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.02%) to 75.629%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

algorithms/arrays/limit.py
3
4
75.0%

Totals

Change from base Build 904:
-0.02%

Covered Lines:
5980

Relevant Lines:
7907

💛  - Coveralls

thanks!


Adding a Dockerfile to create an image with all the dependencies

Hi :v
I was playing with your code.
One of the problems I had was that I don't really wanted to install all those dependencies in my main system, so I create and test a docker image using this Dockerfile.
This worked well on Linux, can't tested it on Mac :I, this depends in the docker container being able to read the /dev/video0 device aka the web camera, and Windows... well.

Thanks!


Fix buffer overflow on empty strings in key.

Sometimes apache segfalult on memory copying when key.dptr is some
kind of empty string and key.dsize seems to be 0.
Investigating sefaults in our apache installation via gdb I get this:
Program terminated with signal SIGSEGV, Segmentation fault.
#0  __memcpy_sse2_unaligned () at ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S:116
116    ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S: No such file or directory.
(gdb) up
#1  0x00007fd26129cd58 in memcpy (__len=18446744073709551615, __src=0xcd7c68, _dest= <optimized out>) at /usr/include/bits/string3.h:51
51      return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
(gdb) up
#2  apr_pstrmemdup (a=<optimized out>, s=0xcd7c68 "", n=18446744073709551615) at strings/apr_strings.c:107
107    strings/apr_strings.c: No such file or directory.
(gdb) up
#3  0x00007fd253956987 in collections_remove_stale (msr=msr@entry=0xcc5b88, col_name=0xccd678 "ip") at persist_dbm.c:629
629    persist_dbm.c: No such file or directory.
(gdb) p s
$1 = <optimized out>
(gdb) down
#2  apr_pstrmemdup (a=<optimized out>, s=0xcd7c68 "", n=18446744073709551615) at strings/apr_strings.c:107
107    strings/apr_strings.c: No such file or directory.
(gdb) p s
$2 = 0xcd7c68 ""
(gdb) p a
$3 = <optimized out>
(gdb) up
#3  0x00007fd253956987 in collections_remove_stale (msr=msr@entry=0xcc5b88, col_name=0xccd678 "ip") at persist_dbm.c:629
629    persist_dbm.c: No such file or directory.
(gdb) up
#4  0x00007fd25394393b in modsecurity_persist_data (msr=0xcc5b88) at modsecurity.c:232
232    modsecurity.c: No such file or directory.
(gdb) up
#5  modsecurity_process_phase_logging (msr=0xcc5b88) at modsecurity.c:640
640    in modsecurity.c
(gdb) up
#6  modsecurity_process_phase (msr=msr@entry=0xcc5b88, phase=phase@entry=5) at modsecurity.c:807
807    in modsecurity.c
(gdb) up
#7  0x00007fd253941cd2 in hook_log_transaction (r=<optimized out>) at mod_security2.c:1239
1239    mod_security2.c: No such file or directory.
(gdb) up
#8  0x000000000042cdd0 in ap_run_log_transaction (r=0xcc1be0) at protocol.c:1748
1748    protocol.c: No such file or directory.
(gdb) up
#9  0x000000000042d8c8 in ap_read_request (conn=conn@entry=0xcbbb00) at protocol.c:1078
1078    in protocol.c

Problem is here:
ModSecurity/apache2/persist_dbm.c:
rc = apr_sdbm_firstkey(dbm, &key);
while(rc == APR_SUCCESS) {
    char *s = apr_pstrmemdup(msr->mp, key.dptr, key.dsize - 1);  // <---- buffer overflow here
    *(char **)apr_array_push(keys_arr) = s;
    rc = apr_sdbm_nextkey(dbm, &key);
}

key.dptr = ""
key.dsize - 1 =<PHONE_NUMBER>3709551615 = 0xFFFFFFFFFFFFFFFF
key.dsize = 0

Optimisation: "key.dsize ? (key.dsize - 1) : 0" instead of "strlen(key.dptr)"
Even better, I suspect that we shouldn't even perform the apr_array_push() at all:
if (key.dsize) {
char *s = apr_pstrmemdup(msr->mp, key.dptr, key.dsize - 1);
*(char **)apr_array_push(keys_arr) = s;
}

Hi @ealekseev,
Thank you for the patch. I have also added the suggesting made by @marcstern. Code being cooked by our buildbots.

Thanks @ealekseev for the solution!

Hi @ealekseev,
Thank you for the patch again. Your code has been merged into your mainline. It will be released as part of 2.9.1.


Fix linux tests to report correctly, get rid of stream destroyed messages on raw kernel shutdown

For #12539

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[x] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

On the first commit of this got a successful test run with Linux / Mac / Windows results all in:
https://dev.azure.com/ms/vscode-python/_build/results?buildId=91149&view=results

One Mac and Linux jupyter server and raw kernel daemon processes were not getting shut down on dispose. The server would be closed, but the proc.kill was not killing the process. They would pile up and start to hold onto more and more ports on the machines which would lead to problems (on linux it was slowing things down so much the machine was stopping reporting, mac tests usually finished, but logs were packed with Port XXXX blocked messages). This reproed on my local mac machine.
nc.","type":"Conda","envPath":"/usr/local/miniconda","envName":"base","displayName":"Python 3.7.7 64-bit ('base': conda)","__store":true}, Arg 3: true 2020-06-29T09:16:36.1498470Z Info 2020-06-29 09:16:36: [I 09:16:36.083 NotebookApp] The port 8888 is already in use, trying another port. 2020-06-29T09:16:36.1499250Z [I 09:16:36.083 NotebookApp] The port 8889 is already in use, trying another port. 2020-06-29T09:16:36.1500140Z [I 09:16:36.084 NotebookApp] The port 8890 is already in use, trying another port. 2020-06-29T09:16:36.1500810Z [I 09:16:36.085 NotebookApp] The port 8891 is already in use, trying another port. 2020-06-29T09:16:36.1501440Z [I 09:16:36.086 NotebookApp] The port 8892 is already in use, trying another port. 2020-06-29T09:16:36.1502070Z [I 09:16:36.086 NotebookApp] The port 8843 is already in use, trying another port. 2020-06-29T09:16:36.1502690Z [I 09:16:36.087 NotebookApp] The port 8822 is already in use, trying another port. 2020-06-29T09:16:36.1503330Z [I 09:16:36.088 NotebookApp] The port 8878 is already in use, trying another port. 2020-06-29T09:16:36.1503970Z [I 09:16:36.088 NotebookApp] The port 8850 is already in use, trying another port. 2020-06-29T09:16:36.1504600Z [I 09:16:36.089 NotebookApp] The port 8918 is already in use, trying another port. 2020-06-29T09:16:36.1505230Z [I 09:16:36.090 NotebookApp] The port 8842 is already in use, trying another port. 2020-06-29T09:16:36.1505850Z [I 09:16:36.091 NotebookApp] The port 8859 is already in use, trying another port. 2020-06-29T09:16:36.1506490Z [I 09:16:36.091 NotebookApp] The port 8829 is already in use, trying another port. 2020-06-29T09:16:36.1507120Z [I 09:16:36.092 NotebookApp] The port 8933 is already in use, trying another port. 2020-06-29T09:16:36.1507750Z [I 09:16:36.093 NotebookApp] The port 8964 is already in use, trying another port. 2020-06-29T09:16:36.1508390Z [I 09:16:36.093 NotebookApp] The port 8909 is already in use, trying another port. 2020-06-29T09:16:36.1509020Z [I 09:16:36.094 NotebookApp] The port 8810 is already in use, trying another port. 2020-06-29T09:16:36.1509800Z [I 09:16:36.095 NotebookApp] The port 8832 is already in use, trying another port. 2020-06-29T09:16:36.1510500Z [I 09:16:36.095 NotebookApp] The port 8817 is already in use, trying another port. 2020-06-29T09:16:36.1511150Z [I 09:16:36.096 NotebookApp] The port 8962 is already in use, trying another port. 2020-06-29T09:16:36.1511780Z [I 09:16:36.097 NotebookApp] The port 8833 is already in use, trying another port. 2020-06-29T09:16:36.1512410Z [I 09:16:36.097 NotebookApp] The port 8976 is already in use, trying another port. 2020-06-29T09:16:36.1513030Z [I 09:16:36.098 NotebookApp] The port 8963 is already in use, trying another port. 2020-06-29T09:16:36.1513660Z [I 09:16:36.099 NotebookApp] The port 8794 is already in use, trying another port. 2020-06-29T09:16:36.1514650Z [I 09:16:36.100 NotebookApp] The port 8788 is already in use, trying another port. 2020-06-29T09:16:36.1515310Z [I 09:16:36.100 NotebookApp] The port 8891 is already in use, trying another port. 2020-06-29T09:16:36.1515950Z [I 09:16:36.101 NotebookApp] The port 8833 is already in use, trying another port. 2020-06-29T09:16:36.1516570Z [I 09:16:36.102 NotebookApp] The port 8800 is already in use, trying another port. 2020-06-29T09:16:36.1517210Z [I 09:16:36.102 NotebookApp] The port 8844 is already in use, trying another port. 2020-06-29T09:16:36.1518180Z [I 09:16:36.103 NotebookApp] The port 8855 is already in use, trying another port. 2020-06-29T09:16:36.1518840Z [I 09:16:36.104 NotebookApp] The port 8964 is already in use, trying another port. 2020-06-29T09:16:36.1519460Z [I 09:16:36.105 NotebookApp] The port 8965 is already in use, trying another port. 2020-06-29T09:16:36.1520100Z [I 09:16:36.105 NotebookApp] The port 8787 is already in use, trying another port. 2020-06-29T09:16:36.1521470Z [I 09:16:36.111 NotebookApp] Serving notebooks from local directory: /var/folders/24/8k48jl6d249_n_qfxwsl6xvm0000gn/T/6bb312c4-e12a-4e18-bd9f-aaab7e9d7bea 2020-06-29T09:16:36.1522330Z [I 09:16:36.111 NotebookApp] The Jupyter Notebook is running at: 2020-06-29T09:16:36.1523330Z [I 09:16:36.111 NotebookApp] http://localhost:8884/?token=785855c39fa5554e995d497dbc3bb3c0b5b6da5c8e17c006 2020-06-29T09:16:36.1524240Z [I 09:16:36.112 NotebookApp]  or http://<IP_ADDRESS>:8884/?token=785855c39fa5554e995d497dbc3bb3c0b5b6da5c8e17c006 2020-06-29T09:16:36.1525520Z [I 09:16:36.112 NotebookApp] Use Control-C to stop this server and shut down all kernels (twice to skip confirmation).

Also on both mac and windows there were lots of messages like this in the error log both when running the live extension (shows when shutting down the interactive window) and running functional test. Both on windows and mac / linux. This update seems to resolve that as well:
rejected promise not handled within 1 second: Error [ERR_STREAM_DESTROYED]: Cannot call write after a stream was destroyed  extensionHostProcess.js:976 stack trace: Error [ERR_STREAM_DESTROYED]: Cannot call write after a stream was destroyed at doWrite (_stream_writable.js:414:19) at writeOrBuffer (_stream_writable.js:402:5) at Socket.write (_stream_writable.js:302:11) at /Users/ianhuff/GitHub/vscode-python-ianhu3/node_modules/vscode-jsonrpc/lib/node/ril.js:134:29 at new Promise (<anonymous>) at WritableStreamWrapper.write (/Users/ianhuff/GitHub/vscode-python-ianhu3/node_modules/vscode-jsonrpc/lib/node/ril.js:124:16) at StreamMessageWriter.doWrite (/Users/ianhuff/GitHub/vscode-python-ianhu3/node_modules/vscode-jsonrpc/lib/common/messageWriter.js:99:33) at /Users/ianhuff/GitHub/vscode-python-ianhu3/node_modules/vscode-jsonrpc/lib/common/messageWriter.js:90:29 at runNextTicks (internal/process/task_queues.js:58:5) at processImmediate (internal/timers.js:412:9)

Note: Will test with a full flake run before submitting.

Nightly test results:
https://dev.azure.com/ms/vscode-python/_build/results?buildId=91309&view=results

Non-required Windows Py3x Single Workspace looks to be timeout and not legit test failure (happening on many PRs) so merging.


Revise 04: Repertoire: Common Music Notation

This issue can be used to discuss chapter 4 of the guidelines.

I spotted the following inconsistencies:

4.2.2.  model.model.eventLike in Guidelines but model.eventLike in specs, this also occurs in   4.2.5., <IP_ADDRESS>., <IP_ADDRESS>., <IP_ADDRESS>.
<IP_ADDRESS>. I wasn't able to find @mRest, @multiRest and @mSpace in specs
4.3.1. I can't find @rendfor  in specs
4.3.3.  model.model.controlEventLike in Guidelines but model.controlEventLike in specs, also occurs two time in 4.3.3. and also in <IP_ADDRESS>
-4.3.4. @dots in Guidelines but @dots.gesin specs
<IP_ADDRESS>. I can't find @rend and @text as attributes of  in specs
<IP_ADDRESS>.  model.model.eventLike.cmn in Guidelines but model.eventLike.cmn in specs
<IP_ADDRESS>. For  I can't find  @rend in specs

@rettinghaus Could you please add a short explanation why the issue was closed (deprecated, resolved, superseded?)? This makes it much easier to trace and evaluate the status of the issue, also from a future perspective, than simply closing it after almost a year in which the issue had no activity. Thank you!

Should have been closed with #107, which was obviously forgotten.

Great, many thanks!


BUG: Sentry is overloaded on LCD failures by 15K and Unclosed Client Session by 24K

Reduce LCD Capture Exception Alerts. 15K in last 24 hours.
https://volume-x3.sentry.io/issues/3930149722/?project=6380489&query=is%3Aunresolved&referrer=issue-stream&statsPeriod=14d&stream_index=0
and
https://volume-x3.sentry.io/issues/4443395347/?project=4505082653573120&query=&referrer=issue-stream&statsPeriod=14d&stream_index=0

disabled node reset on robot server because it's not needed as it's not running lcd.
disabled sentry for /home/rootedbox/app/curve-dca-bot-script
should fix both exception
@taariq

@webelf101 moving this ticket to you:

The Async.io script is overloading Sentry with errors for unclosed Client Session. Will you please update the python script to close all asyncio sessions to remove this error?  The bots creating this error are:
a. sentry for /home/rootedbox/app/curve-dca-bot-script
b. disabled sentry for /home/rootedbox/app/dca_bot_python_scrip

FYI, also filtered out localhost alerts on palomabot

Fixed by fixed by https://github.com/VolumeFi/curve-twap-bot-script/pull/28


NOTE: Your Android app must use a base theme named "AppTheme"

At the following location https://github.com/aparajita/capacitor-biometric-auth?tab=readme-ov-file#usage it is saying:

👉 NOTE: Your Android app must use a base theme named "AppTheme".

...but for me it works fine even though my app is not using "a base theme named "AppTheme"".
I found the following related issue https://github.com/aparajita/capacitor-biometric-auth/issues/4 and it seems it has been fixed.
The question is whether it has really been fixed but the docs have not been updated accordingly or does the requirement related to AppTheme name still applies (which would imply the issue at the link from above is not related to this issue).
As I reported above for me it is working fine when I am using a theme that has name different then AppTheme.
Thanks!

Thanks for the info, I'll check into it.

Honestly I have no idea why I put that note in there. I can't think of any reason why the theme has anything to do with the plugin. 🤷🏽

Thanks!
On Thu, Apr 4, 2024 at 7:01 PM Aparajita Fishman @.***>
wrote:

Honestly I have no idea why I put that note in there. I can't think of any
reason why the theme has anything to do with the plugin. 🤷🏽
—
Reply to this email directly, view it on GitHub
https://github.com/aparajita/capacitor-biometric-auth/issues/24#issuecomment-2037730884,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZ6CNBRZW6SLHJ4GB4DC7TY3WBPDAVCNFSM6AAAAABB6TGDNKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZXG4ZTAOBYGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Uniformity

A tiny problem. It seems that the uniformity score should be divided by the length of test samples.

Thank you for pointing this out. To keep consistent with paper, I would not modify it. You may clarify this in your paper.


Cant save model without fire event

Laravel Version: 5.4.29

Description:
In model:
protected $events = [
    'saved' => Saved::class
];

In listener:
if ($event->selection->phone == $event->source->phone) {
    $event->selection->phone = null;
}

if ($event->selection->date == $event->source->date) {
    $event->selection->date = null;
}

$event->selection->save();

This will call recursion...

Of course it will, you're re firing the event from the listener.

If you want to use an event in this situation, you'll need to opt for a custom event, not an Eloquent event. The 'saved' event fires every time a model is saved. If that's not the behavior you want, create a custom event that is fired only when you need it.

i was trying to do something similar in a project i was working on recently, and the solution i ended up with was to use the saving event rather than saved. that allows you to manipulate values whenever the object is saved, without having to call save again


Expand metagenome pipeline for 16S

NCBI 16S kraken2 database built by following "Basic Protocol 3" in  https://currentprotocols.onlinelibrary.wiley.com/doi/full/10.1002/cpz1.930


Safari TP 180 breaks polyfill

Hi !
After #18 , unfortunately the latest Safari TP180 released today is once again breaking the polyfill, which cause all stores using our theme to break under this version.
The issue can be reproduced here: https://impact-dev-store.myshopify.com (the password to enter the page is rewblo):

Here as well I'm not familiar enough with the internals to properly debug it, but let me know, I would be happy to do the same debugging as last time, and ensure that this get patched by Apple before official release, as this would be quite a disaster for us as we have thousands of stores relying on this.
While I love this polyfill, on our next project we have stopped using it, Safari breaking it every 3-4 TP versions just make it unmanageable for themes dev like us...
Thanks again for your incredible work on this, and really sorry to constantly bring those issues related to Safari...

FWIW I have no errors in console with Web Epiphany TP 45.0-11-g986625975+ that I've just updated out of flathub ... are we sure this is a custom elements story/polyfill ??? :thinking:

This is for the TP180 released today. I am not sure if this has an impact or not, but I am on osX Sonoma as well. I will try to check if someone on my team is on the previous OSX version to try.
But the error is thrown here: https://github.com/ungap/custom-elements/blob/f9010ee553923eafbe7eb36b6113c7007ff5244c/index.js#L94
Interestingly though, I only have the issue on one of two themes using the polyfill this time, so there might be some odd conflicts between libraries here. I will try to do some more debugging tomorrow.

While I love this polyfill, on our next project we have stopped using it, Safari breaking it every 3-4 TP versions just make it unmanageable for themes dev like us...

so ... you folks just ditched entirely builtin extends or did you just implement your own version via MutationObserver? :thinking:
in any case, that's how you kill any community effort to make the Web cross platform and cross vendor and I am this close as well to stop maintaining it as I don't get paid Safari money to do so (it's out of my already inexistent free time) but I also provided various alternatives that offer the same without even using builtin extends native functionality ... super sad to hear you folks decided to abandon the cure, not the symptom :cry:

No, we just ditched entirely built-in extends. I would have been happy to continue using it, but we are selling themes in Shopify. Shopify themes do not update automatically, and we have thousands of stores using our themes. This means that when something like that break, we have suddenly thousands of stores being broken. When you suddenly have 3000+ angry merchants who urge you to fix their store (and considering we can only fix Shopify stores manually), I hope you will understand why I decided to give up built-ins in our next themes :). This is manageable when you manage 10-20 websites, not 3000 :).

Interestingly though, I only have the issue on one of two themes using the polyfill this time

I understand you have history around Safari breaking the poly but they did put effort to not do that earlier this year so maybe this time this is not correlated at all and, if that's the case, I hope your team will take little extra time to investigate errors before blaming the polyfill ... as it's very easy to blame somebody else but ditching the dependency entirely won't fix the underlying issue neither (if an error is unrelated to dependencies, you gotta hunt and fix that error) :sweat_smile:

I hope you will understand why I decided to give up built-ins in our next themes :)

I do ... I myself try to use the least amount of foreign dependencies for anything production-ready but both me and Safari team helped before + shit happens regardless you are using a Web standard or not and you are doing the right thing filing issues around sudden breaks, I just hope this had nothing to do with the poly or, if it does, that Safari actually takes responsibility for breaking what's been fixed already.

anyway, I've added a comment https://bugs.webkit.org/show_bug.cgi?id=258282#c31 as there's nothing I can do, specially because there's not a single issue in my Safari TP (which is based on their Linux TP so maybe that's the one behind).

This issue should be fixed in the next version of STP since the offending code change has already been reverted on trunk.

As a side note, it is one of the roles of Safari Technology Preview, while the WebKit team always tries to minimize breakages, it might happen, and STP is here to catch it early enough so that it is not going into a release. Thanks for reporting.

Thanks a lot, I'm going to close this ticket and try again in TP181. @karlcow , unrelated question: is there a document explaining how Safari release process works? For instance if next Safari version (eg. 17.2) would have freeze on TP180, what would have happen? :)

That's a good question. I do not know the full answer to this. I will ask around.
But the goals is that if something bad is in the build, hopefully it will detected by STP, so it's possible to avoid to have a bad release.


Feeding more data into Vespa container

In the document of [Blog Search Tutorial], it feeding the first 10000 lines to the vespa container by
$ docker exec vespa bash -c 'java -jar /opt/vespa/lib/jars/vespa-http-client-jar-with-dependencies.jar --verbose --file /app/tutorial_feed.json --host localhost --port 8080'
When I try to feed more lines such as 500000 lines, I found here are something wrong.

Do anybody know what the problem is?
And it said the [trainPosts.json] file is around 5GB in size with 1196111 lines, so why do it requires 23GB indexing the full dataset?

The reason you are running into the communication errors here is likely due to not having enough memory in your Docker container. Remember to set the memory to for instance 10GiB in the Advanced tab in the Docker preferences, as the memory argument sent to the docker run command won't exceed this value.
I just tried myself here, and setting to 10GiB I indexed all the data, but when set to 2GiB (the default) I quickly ran into the same errors as you have here.
Regarding index size, it is largely due to the transaction log during feeding:
[root@vespa-tutorial search]# pwd
/opt/vespa/var/db/vespa/search
[root@vespa-tutorial search]# du -h
14G	./cluster.blog_post/n0/tls/tls/blog_post
14G	./cluster.blog_post/n0/tls/tls
14G	./cluster.blog_post/n0/tls
44K	./cluster.blog_post/n0/documents/blog_post/config/config-1
52K	./cluster.blog_post/n0/documents/blog_post/config
4.0K	./cluster.blog_post/n0/documents/blog_post/2.notready/documentmetastore
12K	./cluster.blog_post/n0/documents/blog_post/2.notready/summary
4.0K	./cluster.blog_post/n0/documents/blog_post/2.notready/attribute
24K	./cluster.blog_post/n0/documents/blog_post/2.notready
4.0K	./cluster.blog_post/n0/documents/blog_post/1.removed/documentmetastore
12K	./cluster.blog_post/n0/documents/blog_post/1.removed/summary
20K	./cluster.blog_post/n0/documents/blog_post/1.removed
4.0K	./cluster.blog_post/n0/documents/blog_post/0.ready/documentmetastore
8.8M	./cluster.blog_post/n0/documents/blog_post/0.ready/index/index.flush.4/title
356M	./cluster.blog_post/n0/documents/blog_post/0.ready/index/index.flush.4/content
366M	./cluster.blog_post/n0/documents/blog_post/0.ready/index/index.flush.4
27M	./cluster.blog_post/n0/documents/blog_post/0.ready/index/index.fusion.3/title
1.1G	./cluster.blog_post/n0/documents/blog_post/0.ready/index/index.fusion.3/content
1.2G	./cluster.blog_post/n0/documents/blog_post/0.ready/index/index.fusion.3
1.5G	./cluster.blog_post/n0/documents/blog_post/0.ready/index
5.4G	./cluster.blog_post/n0/documents/blog_post/0.ready/summary
4.0K	./cluster.blog_post/n0/documents/blog_post/0.ready/attribute
6.9G	./cluster.blog_post/n0/documents/blog_post/0.ready
6.9G	./cluster.blog_post/n0/documents/blog_post
6.9G	./cluster.blog_post/n0/documents
21G	./cluster.blog_post/n0
21G	./cluster.blog_post
8.0K	./blog_post/distributor/0
12K	./blog_post/distributor
8.0K	./blog_post/storage/0
12K	./blog_post/storage
28K	./blog_post
21G	.
[root@vespa-tutorial search]# vespa-proton-cmd --local triggerFlush
looking for RTCs matching 'tcp/vespa-tutorial:' (length 19)
found local RTC 'blog_post/search/cluster.blog_post/0/realtimecontroller' with connection spec tcp/vespa-tutorial:19108
OK: flush trigger enabled
[root@vespa-tutorial search]# du -h
46M	./cluster.blog_post/n0/tls/tls/blog_post
46M	./cluster.blog_post/n0/tls/tls
46M	./cluster.blog_post/n0/tls
44K	./cluster.blog_post/n0/documents/blog_post/config/config-1
52K	./cluster.blog_post/n0/documents/blog_post/config
8.0K	./cluster.blog_post/n0/documents/blog_post/2.notready/documentmetastore/snapshot-1198160
16K	./cluster.blog_post/n0/documents/blog_post/2.notready/documentmetastore
16K	./cluster.blog_post/n0/documents/blog_post/2.notready/summary
4.0K	./cluster.blog_post/n0/documents/blog_post/2.notready/attribute
40K	./cluster.blog_post/n0/documents/blog_post/2.notready
8.0K	./cluster.blog_post/n0/documents/blog_post/1.removed/documentmetastore/snapshot-1198160
16K	./cluster.blog_post/n0/documents/blog_post/1.removed/documentmetastore
16K	./cluster.blog_post/n0/documents/blog_post/1.removed/summary
36K	./cluster.blog_post/n0/documents/blog_post/1.removed
32M	./cluster.blog_post/n0/documents/blog_post/0.ready/documentmetastore/snapshot-1198160
32M	./cluster.blog_post/n0/documents/blog_post/0.ready/documentmetastore
40M	./cluster.blog_post/n0/documents/blog_post/0.ready/index/index.fusion.5/title
1.7G	./cluster.blog_post/n0/documents/blog_post/0.ready/index/index.fusion.5/content
1.7G	./cluster.blog_post/n0/documents/blog_post/0.ready/index/index.fusion.5
1.7G	./cluster.blog_post/n0/documents/blog_post/0.ready/index
5.4G	./cluster.blog_post/n0/documents/blog_post/0.ready/summary
4.6M	./cluster.blog_post/n0/documents/blog_post/0.ready/attribute/date/snapshot-1198160
4.6M	./cluster.blog_post/n0/documents/blog_post/0.ready/attribute/date
4.6M	./cluster.blog_post/n0/documents/blog_post/0.ready/attribute
7.1G	./cluster.blog_post/n0/documents/blog_post/0.ready
7.1G	./cluster.blog_post/n0/documents/blog_post
7.1G	./cluster.blog_post/n0/documents
7.2G	./cluster.blog_post/n0
7.2G	./cluster.blog_post
8.0K	./blog_post/distributor/0
12K	./blog_post/distributor
8.0K	./blog_post/storage/0
12K	./blog_post/storage
28K	./blog_post
7.2G	.

As you can see here, in the first du 14Gb is used for the tls directory which is the transaction log directory. After running triggerFlush, the transaction log is guaranteed to be committed and most of the data there is removed.
So the index itself (here under /opt/vespa/var/db/vespa/search/cluster.blog_post/n0/documents) takes roughly 7.1Gb, which isn't that far off the feed of around 5Gb.
Cheers,

Lester

Closing this as question has been answered.


Fix: Created Type declaration dto files for packages

Checklist
Make sure you have

[ ] Added relevant @nestjs/swagger decorators wherever required
[ ] Added test cases (unit and integration) wherever required

Description
Publishing DTO-type declaration files for the controller, interceptor, and services for packages/common

closing this out since this issue is already fixed.


Why should vim-plug be installed through githubusercontent.com not https://github.com/junegunn/vim-plug?

It is advised to install vim-plug using:
curl -fLo ~/.vim/autoload/plug.vim --create-dirs \
    https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim

I would feel better using something like this:
git clone --depth=1 https://github.com/junegunn/vim-plug.git ~/.vim/autoload/plug.vim

What is it the advantage of using this discutable url (raw.githubusercontent.com)?
Of course my way requires some tricks or place plug.vim in a autoload directory.

Type:

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

OS:

[x] All/Other
[ ] Linux
[ ] OS X
[ ] Windows

You only need plug.vim and there is PlugUpgrade to get the latest. Documentation is at the top of plug.vim.
Unless you are submitting patches, you don't need the other files.
If you want to use git clone (or subtree or submodule) without touching autoload/, you have to source plug.vim explicitly in your vimrc.
" assume vim-plug is cloned in ~/.vim/vim-plug
runtime ~/.vim/vim-plug/plug.vim
call plug#begin('~/.vim/plugged')

I finally found a working solution using dotbot. I am using vim-plug as a submodule, but I only link plug.vim into ~/.vim/autoload/. With this solution I explicitly know which version of plug.vim I am using

Of course my way requires some tricks or place plug.vim in a autoload directory.

So you already know the drawback of your approach. I would rather not tell the users to run multiple commands to install vim-plug when a single command can do. raw.githubusercontent.com is where GitHub serves raw git objects, and I can't see what the issue is. vim-plug has been quite stable for a long time and most users really don't have to upgrade it unless they run into some issues.