QEMU time

Is your question not already answered in the FAQ?

[X] I made sure the question is not listed in the FAQ.

Is this a general question and not a technical issue?

[X] I am sure my question is not about a technical issue.

Question
Windows time is out of sync without ARGUMENTS: "-rtc base=localtime"
Should be handy to create dedicated ENVIRONMENT option for this if not created already.

I have same issue, time difference between my debian 12 host and windows container is 2 hours.
dockurr/windows         latest    d380ee3be346   7 hours ago    371MB
The image I am currently using, it happened on previous versions too.
@kroese

I added this parameter "-rtc base=localtime" @baiomys suggested. So I dont know what more can I do.
Are you sure you set the correct timezone setting? Because that can also cause a difference.

I did set correct timezone. After I do NTP sync in Windows time is ok, but is goes back after Windows reboot.

@baiomys Do you have any idea?

Maybe that fix just needs to be reverted
https://docs.getutm.app/settings-qemu/qemu/

Use local time for base clock

This specifies the -rtc base=localtime option in QEMU. This synchronizes the guest clock to the local clock without any offsets. On some Linux guests, the RTC base is expected to be UTC and so this option should be disabled.

"-rtc base=utc" in ARGUMENTS should help
95% guests run local time, so turning this option ON by default was reasonable

My host is running Linux Mint 21.3 and I have added the ARGUMENTS: "-rtc base=localtime" statement to my existing compose file and restarted the container. But the Windows 11 time still shows a +5 hour difference from my local time which is Central Daylight Time.
I did a test spinning up a new container with the above arguments line and after a new download and setup, the time on the Win11 container showed the correct local time after changing the time zone in the Windows time/date settings. I then shut down the container and started it up again keeping the ARGUMENTS statement intact. After startup, the time was not correct showing the same +5 hour time difference.
Is there any way to make the ARGUMENTS statement to take effect and persist on an existing container or after restart of the container.
Thanks

"-rtc base=localtime" is set by default since version v3.05, did you try "-rtc base=utc"?
Anyway, after changing this option you need to manually RESYNC TIME in windows at least once to make it persistent.

I have resynced the time in the Win11 container and restarted it with no change using "rtc base=localtime" I have also tried "rtc base=utc" and resynced the Win11 time and restarted with no change.

Still does not work - but I have implemented a workaround by invoking a batch file at windows startup to sync time with NTP to get my correct local time.

Still does not work - but I have implemented a workaround by invoking a batch file at windows startup to sync time with NTP to get my correct local time.

can you share that script here?

@the4anoni Sure. Here's the content of the batch file:
@echo off
SC start "W32Time" & W32tm /ReSync

@taylormia Thanks for the idea. I was also having this issue. So I created a Windows scheduled task with the highest level of privileges that gets triggered when any user logs in, and it runs: W32tm /resync /force

heres how I addressed the incorrect timezone issue just prior to the Dockur/Windows container install. This was doen ins a in a build script but the same style of tweaks could be applied after install:
# Set the Windows time zone
	echo -e "\\n    [HKEY_LOCAL_MACHINE\\SYSTEM\\CurrentControlSet\\Control\\TimeZoneInformation]\n    \"TimeZoneKeyName\"=\"Add_timezone_here\"" >> winapps/oem/RDPApps.reg"

# Enable the Windows timezone to update automatically
	 sed -i '$a\\n    [HKEY_LOCAL_MACHINE\\SYSTEM\\CurrentControlSet\\Services\\tzautoupdate]\n    "Start"=dword:00000003' /winapps/oem/RDPApps.reg >/dev/null

# Build a script to sync the time at each startup
	# see issue https://github.com/dockur/windows/issues/465
        {
        echo 'echo @echo off > "C:\ProgramData\Microsoft\Windows\Start Menu\Programs\Startup\timesync.bat"'
        echo 'echo SC start "W32Time" ^& W32tm /ReSync /force >> "C:\ProgramData\Microsoft\Windows\Start Menu\Programs\Startup\timesync.bat"'
        echo 'SC start "W32Time" & W32tm /ReSync /force'
        } >> /winapps/oem/install.bat


Add/Update Badges on README

Similar to what was done on the SendGrid node README:

put the logo at the top of the list
Add License like so
make sure the other badges are available like so:

![SendGrid Logo](https://uiux.s3.amazonaws.com/2016-logos/email-logo%402x.png)

For the following, make sure the correct repo is linked!
For email notifications, you can change the "nodejs" to the language of this repo (e.g. java, python, ruby, php, csharp, go, etc)
[![BuildStatus](https://travis-ci.org/sendgrid/sendgrid-nodejs.svg?branch=master)](https://travis-ci.org/sendgrid/sendgrid-nodejs)
[![Email Notifications Badge](https://dx.sendgrid.com/badge/nodejs)](https://dx.sendgrid.com/newsletter/nodejs)
[![npm](https://img.shields.io/npm/l/express.svg)]()
[![Twitter Follow](https://img.shields.io/twitter/follow/sendgrid.svg?style=social&label=Follow)](https://twitter.com/sendgrid)
[![GitHub contributors](https://img.shields.io/github/contributors/sendgrid/sendgrid-nodejs.svg)](https://github.com/sendgrid/sendgrid-nodejs/graphs/contributors)

Thank you and happy #Hacktoberfest!!

#80

#80 is that okay?


Cannot read property 'panel' of undefined when searching extensions

VSCode Version: 1.18 Insiders dated 2017-10-27
OS Version: Any

Steps to Reproduce:

Search for any extension in the extensions viewlet
The search is successful but an error "Cannot read property 'panel' of undefined" pops up

Call Stack in the console:

Hi Ramya Rao,
@sandy081 fixed it already and closed issue #37026 with commit de42a6d9350e07ece8bb8807528768487aa291fb. The insider builds on the download page are currently outdated and from the 27.10.2017. In my research I found out, that they are released weekly.

Thanks @maxibanki !
I made the mistake of looking for dupes in opened issues instead of also looking at closed ones :)
Closing this as duplicate of #37026


Process doesn't terminate after tests are finished

see video
If I use npm based run then all works right.
Files to reproduce:
simple.js
fixture `Simple`;

test('test', async t => {
	await t.click('body');
});

###run-simple.js
const createTestCafe = require('testcafe');
let runner           = null;

createTestCafe('localhost', 1337, 1338)
    .then(testcafe => {
        runner = testcafe.createRunner();
    })
    .then(() => {
        return runner
            .src('simple.js')
            .browsers(['chrome'])
            .reporter()
            .screenshots('screenshots')
            .run();
    }).catch((err) => console.log(err));

testcafe version: 0.14.0-alpha2
node.js version:7.4.0

It happens because you don't call testcafe.close(). and the process keeps waiting on opened sockets.

Ok. Then we should add more details about this in 'Programmic Interface' section.
Something like that - 'Then your tests are finished you should call testcafe.close() method to close testcafe connection. Otherwise the process will keep waiting for opened sockets.'
@VasilyStrelyaev

Agree, will do it


CHEBI: 82569

Please could CHEBI: 82569 - 4-(Methylnitrosamino)-1-(3-pyridyl)-1-butanol be curated and defined.

Now curated. Thanks

Now curated. Thanks


Add green colour for the pass ticks

Reward the user for having passing tests with a beautiful green hue!

Isn't this (almost) the same as #166?

Please add this! I would love to see nice green check boxes in my logs :smile: :+1:


[feature request] Add function to interpolate series

We need an aggregate to fill in gaps in series if they exist. For example we have this:
SELECT mean(value)
FROM cpu
WHERE time > now() - 24h
GROUP BY time(10m)

Assume that we have host as a tag on those series and they're collected every 10s. That means we'd expect 60 values per host in each group by window. If we wanted to fill in any missing values we'd use the interpolate function:
SELECT interpolate(value, 10s, mean).mean(value)
FROM cpu
WHERE time > now() - 24h
GROUP BY time(10m)

That specifies that we'd fill in any missing values in 10s intervals with whatever the mean is for the group by window. We could also use previous or a specific value as an interpolation value. For instance if we did:
SELECT
  interpolate(value, 10s, 0).count(value)
FROM cpu
WHERE time > now() - 24h
GROUP BY time(10m)

We would just get 60 * number_of_hosts in each 10m group by window.
To support this kind of syntax we'd need to complete #4327 first. The other option is to have it at the trailing end as clause, but then that would make it apply to all fields in the select clause:
SELECT mean(value)
FROM cpu
WHERE time > now() - 24h
GROUP BY time(10m)
INTERPOLATE(10s, previous)

+1

+1
Interpolation is tricky - it will really depend on what you want to get out the other side. For my own cases, I like to have access to previous (we call it "step" resampling), linear (along the line connecting points before/after), and lowess (locally weighted least-squares fitted polynomial) which can be great for upsampling and also act to downsample/smooth while retaining more of the character of first-order discontinuities. Obviously the lowess interpolation needs more parameters like either a fixed time window or fixed number of points to fit.

+1
We also need to be able to resample a series.  For instance when you have two series sampled at different rates, it is necessary to resample one or both of them to the same rate so you can do additional signal processing on them.

+1 for the need for interpolation.
However, I see interpolation a little bit differently. For consistency with others functions, I'd prefer that the GROUP BY time would be with time(10s) if we want one point every 10seconds, and the way to interpolate (last-previous, mean over a 10 minute window) to be parameters of the INTERPOLATION function. Therefore, the interpolation function would return the interpolate if there is no point on the time group, and an aggregate given in parameter if there are 1 or more points on the time group.
Syntax would be :
SELECT interpolate(<field_key>,<function (if there are multiple points in time group)> FROM <measurement_name> WHERE <time_range> GROUP BY time(<time_interval>[,<offset_interval>])
Suppose I have the following points :

time -> value
10:02 -> 3
10:09 -> 5
10:21 -> 7

SELECT interpolate(value, mean)
FROM cpu
WHERE time > now() - 24h
GROUP BY time(10m)

Would return :

For 10:00 -> 4 (doing the mean of 3 and 5 because we have multiple (2) values)
For 10:10 -> 6 computing the interpolation at 10:15, which is 6.
For 10:20 -> 7 (the mean of a single value)

Therefore, there is no need to chain functions and wait for function chaining in #4327
Simple remark : for the second query you mentioned, I think adding a point with the mean of the grouping window and doing a mean will provide exactly the same result as just taking the mean. ;-)

+1

+1


Prevent the show simulator button from stealing focus in some situations.

Closes #893.

I don't like introducing the extra state, but can't think of an alternative way to do it without having to wait for the 'show simulator' button to be available on the page.

I don't like introducing the extra state, but can't think of an alternative way to do it without having to wait for the 'show simulator' button to be available on the page.

Took me a while to understand but I think this makes sense. I think ideally they'd be the same button in the show and hide case (not tried this!) and then no special focus management would be necessary.


Celo:

Describe your issue here.
Hello, I transferred some Celo to my wallet 22 days ago, and up to now I didn't receive them.
The URL of the transaction is : https://explorer.celo.org/tx/0xfe5e48d11a617b998bc57b48d905804af54ce4672aa70c30612c904776ea4880/token_transfers
Thank you.
Anouar.
Environment

Elixir Version: 1.9.0

Erlang Version: 22

BlockScout Version: 330e57bb88472a655cb1b67d70131d8ecef56e49

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_6) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/14.0.2 Safari/605.1.15

Steps to reproduce
Tell us how to reproduce this issue. If possible, push up a branch to your fork with a regression test we can run to reproduce locally.
Expected Behaviour
Tell us what should happen.
Actual Behaviour
Tell us what happens instead.

Hi @anouar32, I came across your issue and wanted to see if I can help in any way.
What is the current status of your issue?


How about adding SSN?

:sparkles::sparkles: Social Security Number :sparkles::sparkles:

Sounds good. I'll add it soon :D

U Korean I Korean & Conversation on english. what a fun

I've added it on https://github.com/mingrammer/commonregex/commit/457f293efbab60507fe32e55787260350c7a6adb
But, it supports weak SSN now. (not strictly)


Would you pls share the xpos / feats annotation protocols/specifications?

Hello, Prof. Koichi,
I guess this model is trained from the corpora here. Did you incorporate all the copora there? Because I find this model performs better against Pre-Qin classical Chinese than Post-Qin ones.
Of course, the most important question is as the title.
Many thanks!!

Hi @fishfree please read our papers below (in Japanese) for the specifications of XPOS and FEATS:

Morphological Analysis of Classical Chinese Texts and Its Application
Designing Universal Dependencies for Classical Chinese and Its Application

Thank you!
So, did you incorporate all the copora here besides 四书? Because I find this model performs better against Pre-Qin classical Chinese than Post-Qin ones.

@fishfree please scrutinize our paper again.

@KoichiYasuoka Thank you, Prof! Sorry for bothering you~


detail page

Describe the feature
user click on card then display their detail page for more information
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hey @Anushkabh ,you can assigned this iisue me ,
i can create like a amazon ,flipkart...
user click on tomato then display tomato related full information


Topics list fail to load if opened from an offline state

Steps to reproduce:

Go offline.
Narrow to any stream.
Tap the 'bullet list' icon to go to the Topics screen.
Turn on mobile data/Wi-Fi.
The list of Topics fail to load even after turning on mobile data.

I think this is related to #2287 .

@pulkonet That issue is similar, but since this deals with the display of topics while that issue deals with the display of messages, I think both need separate issues.

You're right, but I stated the relation just so that anyone trying to fix it could take reference.

Hmm, this issue seems to involve two bugs: One bug is the RN fetch() bug that causes #2287. But even if that bug was resolved, I think this issue would still be valid, since it is about narrowing to a topic while the device is offline and coming back online afterwards. Right now, I don't think we resend requests that have been made while offline. In any case, we probably have to wait until https://github.com/facebook/react-native/issues/19709 gets resolved.


Musixmatch fails after certain requests

Hey @skuill,
I just wanted to code some project using your library. Great library so far! But I just encountered that if I request several lyrics (I'm using Musixmatch as provider because the other ones do not work as expected) I get the following error:
MusixmatchClientLib.API.Model.Exceptions.MusixmatchRequestException: Authentication failed, probably because of invalid/missing API key.
   at MusixmatchClientLib.API.ApiRequestFactory.SendRequestLegacy(ApiMethod method, Dictionary`2 additionalArguments, String data)
   at MusixmatchClientLib.API.ApiRequestFactory.SendRequest(ApiMethod method, Dictionary`2 additionalArguments, Dictionary`2 data)
   at MusixmatchClientLib.MusixmatchClient.SongSearch(TrackSearchParameters parameters, PaginationParameters paginationParameters)
   at LyricsScraperNET.Providers.Musixmatch.MusixmatchProvider.SearchLyric(String artist, String song)
   at LyricsScraperNET.Providers.Abstract.ExternalProviderBase.SearchLyric(SearchRequest searchRequest)
   at LyricsScraperNET.LyricsScraperClient.SearchLyric(SearchRequest searchRequest)

Hi @AutMai !
Could you please provide an example of reproducing the problem? How do you initialize the client, provider, and trigger the lyrics search? If you can attach a solution with an example project, that would be great!

Found the bug. Created an Issue in related project:
https://github.com/Eimaen/MusixmatchClientLib/issues/23

Hi @AutMai!
Published fix in the 1.8.1
Could you please update the package to the latest version and let me know if it works.

A lot of time has passed, I close it, apparently it works.


Add embeddings preprocess helpers and notebook

Description
This moves preprocessing helper functions for embeddings to the experimental namespace accessible in whylogs and updates the notebook example for embeddings.

Cleaning up now that make pre-commit is working for me.


Add Postgres TABLESAMPLE support.

These changes add support for queries like so:
SELECT * FROM my_table TABLESAMPLE BERNOULLI(10)

The syntax, excerpt from the PostgreSQL docs:
TABLESAMPLE sampling_method ( argument [, ...] ) [ REPEATABLE ( seed ) ]

I'm skeptical that these changes are ready to merge. The basic functionality does work for me in a side project. However I have not been able to test this to a level I feel comfortable with.
Note the TODO comment regarding the AppearsInFromClause associated Count type. It is set to Once but I struggled to understand how this is used. If this means once per joinable fragment then it should be Once. However PostgreSQL does allow separate TABLESAMPLE clauses in each joinable fragment. Since it seems meant to pertain to the underlying table, perhaps it should be derived from the S type parameter? If so I'm afraid this is beyond my current understanding of the diesel_derives code :)
I'm not sure whether I've properly exported the TablesampleDsl trait correctly. In my application code, where I call .tablesample(...) I also need to use diesel::dsl::TablesampleDsl. That felt a little unexpected. I looked into re-exporting it into the prelude but I didn't see other backend-specific DSL traits re-exported there so I backed away.
It seems most of the DSL tests are in the doctests. I tried to follow that pattern but I wasn't able to run those. A naive cargo rustdoc --features postgres -- --test led to all tests failing. I'd very much appreciate any direction you can provide to run those tests.

Thank you for all of your feedback. I found it quite helpful! I have updated this branch to:

add a compile test
use bind params
export the dsl symbols more ergonomically
encode the sql specialization into type parameters.

Regarding the type parameters, I'm hoping you can provide some feedback regarding how close my approach is to your suggestion. I don't use this approach much in my own code so I'm likely missing some of the nuance in the existing dsl code. The code works in my own project, however the second of the two doctests fails to compile and I'm struggling to figure out why. The error:
---- diesel/src/pg/expression/extensions/tablesample_dsl.rs - pg::expression::extensions::tablesample_dsl::TablesampleDsl (line 63) stdout ----
error[E0277]: the trait bound `query_source::joins::Join<Tablesample<users::table, diesel::dsl::BernoulliMethod>, posts::table, Inner>: AppearsInFromClause<posts::table>` is not satisfied
   --> diesel/src/pg/expression/extensions/tablesample_dsl.rs:73:6
    |
12  |     .inner_join(posts::table)
    |      ^^^^^^^^^^ the trait `AppearsInFromClause<posts::table>` is not implemented for `query_source::joins::Join<Tablesample<users::table, diesel::dsl::BernoulliMethod>, posts::table, Inner>`
    |
    = help: the trait `AppearsInFromClause<T>` is implemented for `query_source::joins::Join<Left, Right, Kind>`
note: required for `posts::columns::id` to implement `AppearsOnTable<query_source::joins::Join<Tablesample<users::table, diesel::dsl::BernoulliMethod>, posts::table, Inner>>`
   --> diesel/src/pg/expression/extensions/../../../doctest_setup.rs:285:13
    |
285 |             id -> Integer,
    |             ^^
    = note: 2 redundant requirements hidden
    = note: required for `((users::columns::id, users::columns::name), (posts::columns::id, posts::columns::user_id, posts::columns::title))` to implement `AppearsOnTable<query_source::joins::Join<Tablesample<users::table, diesel::dsl::BernoulliMethod>, posts::table, Inner>>`
    = note: required for `query_source::joins::Join<Tablesample<users::table, diesel::dsl::BernoulliMethod>, posts::table, Inner>` to implement `QuerySource`
    = note: 1 redundant requirement hidden
    = note: required for `JoinOn<Join<Tablesample<table, BernoulliMethod>, table, Inner>, Grouped<Eq<Nullable<user_id>, Nullable<id>>>>` to implement `QuerySource`
    = note: the full type name has been written to '/tmp/rustdoctestMSz9fp/rust_out.long-type-14004784566645490992.txt'
    = note: required for `SelectStatement<FromClause<Tablesample<users::table, diesel::dsl::BernoulliMethod>>>` to implement `InternalJoinDsl<posts::table, Inner, diesel::expression::grouped::Grouped<diesel::expression::operators::Eq<NullableExpression<posts::columns::user_id>, NullableExpression<users::columns::id>>>>`

The difference between the two examples is the join, as the error indicates. However I'm having trouble understanding how the type constraints are being applied here. In diesel/src/query_source/joins.rs there is an impl for AppearsInFromClause:
impl<T, Left, Right, Kind> AppearsInFromClause<T> for Join<Left, Right, Kind>
where
    Left: AppearsInFromClause<T> + QuerySource,
    Right: AppearsInFromClause<T> + QuerySource,
    Left::Count: Plus<Right::Count>,
{
    type Count = <Left::Count as Plus<Right::Count>>::Output;
}

Since this impl targets Join, then something about the constraints on the type parameters must cause a mismatch:
Left: Tablesample<users::table, diesel::dsl::BernoulliMethod>
Left: AppearsInFromClause<T> + QuerySource,

In diesel_derives/src/table.rs I added:
impl<TSM> diesel::query_source::AppearsInFromClause<diesel::query_builder::Tablesample<table, TSM>>
    for table
    where TSM:  diesel::query_builder::TablesampleMethod
{
	type Count = diesel::query_source::Once;
}

Since BernoulliMethod is an impl of TablesampleMethod, that constraint is satisfied. Let's look at QuerySource. In src/pg/query_builder/tablesample.rs I added:
impl<S, TSM> QuerySource for Tablesample<S, TSM>
where
    S: Table + Clone,
    TSM: TablesampleMethod,
    <S as QuerySource>::DefaultSelection:
        ValidGrouping<()> + SelectableExpression<Tablesample<S, TSM>>,
{
    type FromClause = Self;
    type DefaultSelection = <S as QuerySource>::DefaultSelection;

It's pretty obvious that users implements Table. In diesel_derives/src/table.rs I added:
impl<TSM> diesel::SelectableExpression<diesel::query_builder::Tablesample<super::table, TSM>>
    for #column_name
    where TSM:  diesel::query_builder::TablesampleMethod {}

So it seems my mistake is likely not satisfying this constraint:
    <S as QuerySource>::DefaultSelection:
        ValidGrouping<()> + SelectableExpression<Tablesample<S, TSM>>

However, with the exception of the TSM type parameter, this follows the Only impl of QuerySource that works with joins.

Thanks for pointing me toward macros/mod.rs! Adding those trait impls did allow my doctest to compile. The latest changes include the builder pattern approach you suggested. I believe all of the test failures are unrelated to my changes. I have a CI run with a few extra unrelated fixes here that show the postgresql tablesample tests all passing.

The fixes for the failing CI tests have been merged, so if you rebase your PR I can do a hopefully final review.

Thanks for the heads up. I've rebased this and marked it as ready for review.

No worries, take the time you need for this I would not have time to do this on my own in the next week or so.


feat: Add support for Vertex AI - WIP

Fixes #1

Replaced by https://github.com/google-deepmind/onetwo/commit/d6fce05f78b26ff9ec9fc316de8c57e9c08acb56


Define pre-release catalog sources from the install command line

QE needs the ability to deploy patterns with a specific set of CatalogSources corresponding to pre-release operator versions.
This PR facilitates that without requiring deployment specific git commits.

Test failures seem unrelated to the PR

If you rebase on top of the main branch the CI fialures should be gone


Installation without using the dockerfile

Is it possible to install Kraken without using the dockerfile ? And if it is can you provide an example on how to do it on something like debian.
It's just nicer to have a full tutorial on how to install it without using the dockerfile (using packages provided by your distribution) instead of decrypting the dockerfile syntax to translate it to a "normal" install.

Currently, this is not possible. But I can add support for that.
The question is what Linux distribution do you want to use?
Debian, Ubuntu, RHEL, Centos, something else?
You need to keep in mind that it will be more complicated.
Docker Compose takes care of spinning up some other supporting services.
So you will need to set up manually:

Postresql
Redis
Clickhouse
MinIO
and some http server for reverse proxy
How do you see it?

Personally I am using an alpine linux, but I think that an explanation for Debian is better.
Because there is a ton of debian-based distributions and once you have a tutorial for one distro it's easy to follow it and just convert the names of the packages to the user's distro.
The tutorial should only focuses on the configurations for the dependencies and the installation part of the Kraken server.
Just say that the user needs to have Postgres, Redis, Clickhouse, MinIO and one http server (the tutorial should probably chose one as example).
Then explain what are the special configurations needed on these services. (Open ports, add user, create database, …)
And finish with the installation of Kraken itself.
Having a tutorial for that can also help to understand how the software works.

OK, this makes sense. I will play with it.

It took a while, but it is ready:
https://kraken.ci/docs/install-manual
Could you have a look?

From what I've seen this is everything that someone can need to setup an instance.
I have one question:
Is PostgreSQL 11 a requirement because you need specific features from it or is it like for MinIO, Redis and Clickhouse and you simply can't guarantee that it will work with a newer version ?
I will try to follow this document and setup an instance, this week or maybe next week.
Thanks for your work

I didn't try newer Postgresql but for 99.9% I'm sure that it will work.
śr., 7 gru 2022, 11:39 użytkownik Eragon @.***> napisał:

From what I've seen this is everything that someone can need to setup an
instance.
I have one question:
Is PostgreSQL 11 a requirement because you need specific features from it
or is it like for MinIO, Redis and Clickhouse and you simply can't
guarantee that it will work with a newer version ?
I will try to follow this document and setup an instance, this week or
maybe next week.
Thanks for your work
—
Reply to this email directly, view it on GitHub
https://github.com/Kraken-CI/kraken/issues/216#issuecomment-1340746994,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABLDN735SIZCZDTD5AUOBTWMBSPXANCNFSM6AAAAAARAGPKFE
.
You are receiving this because you commented.Message ID:
@.***>

I can't get clickhouse to work on Alpine, and I my Debian VM is completely broken.
But outside of that the tutorial seems to work, thanks for it.
This is resolved I think.


Small various bug fixes

What's new?

:bug: Identity Popover is using :squid:
:zap: Carrousel card list should use same path for links
:zap: using correct path in popular collections
:label: permission has duplicate Config type
:label: chainProperties have blockExplorer
:zap: added urls for blockExplorers
:bug: BlockExplorer works again

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[ ] I've posted a screenshot of demonstrated change in this PR

Optional

[x] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

uwu

You deserve one cornetto.

ou deserve one cornetto.

m00d


shell is unusable if used with UART that has flow control (whereby RTS is not asserted at boot time)

Describe the bug
If a board has a UART whereby hardware flow control is enabled and the shell is enabled, if the board boots and sees RTS is de-asserted (i.e. cannot send), the shell is unresponsive, even if RTS then becomes asserted later on. It will not respond to any input.
Seemingly, once it receives enough input to fill an rx buffer and log an error, it then starts responding again.
To Reproduce
Build smp_svr with shell enabled for a board with flow control on UART enabled, boot board without serial port open, then open serial port, send commands or garbage data and press enter.
Expected behavior
Shell to work
Impact
Annoyance
Logs and console output
Output when sending the same thing over and over before it starts working again:
test command
test command
test command
test command
test command
test command
[\1b[0
0
\1b:00:06.105,560] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.105,621] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.374,420] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.374,511] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.374,603] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.374,694] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.374,786] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.374,877] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.374,938] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.375,030] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.375,122] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.375,213] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[1;32muart:~$ \1b[mtest command
\1b[1;31mtest: command not found
\1b[m\1b[1;32muart:~$ \1b[mtest command
\1b[1;31mtest: command not found
\1b[m\1b[1;32muart:~$ \1b[mtest comma[nd
\1b[00:00:06.375,274] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
;31mtest: command not found
\1b[m\1b[1;32muart:~$ \1b[mtest command
\1b[1;31mtest: command not found
\1b[m\1b[1;32muart:~$ \1b[mtest command\1b[20D\1b[J[00:00:06.105,560] \1b[1;33m[<w0>0:00:06.375,366] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
[00:00:06.375,457] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
 shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[mtest command\1b[20D\1b[J[00:00:06.105,621] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[mtest command\1b[20D\1b[J[00:00:06.374,420] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[mtest command\1b[20D\1b[J[00:00:06.374,511] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[mtest commandtest command
\1b[20D\1b[J[00:00:06.374,603] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[mtest command\1b[20D\1b[J[00:00:06.374,694] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[mtest commandtest command
\1b[1;31mtest: command not found
\1b[m\1b[1;32muart:~$ \1b[m\1b[8D\1b[J[00:00:06.374,786] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[m\1b[8D\1b[J[00:00:06.374,877] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[m\1b[8D\1b[J[00:00:06.374,938] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[m\1b[8D\1b[J[00:00:06.375,030] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[m\1b[8D\1b[J[00:00:06.375,122] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[m\1b[8D\1b[J[00:00:06.375,213] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[m\1b[8D\1b[J[00:00:06.375,274] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[m\1b[8D\1b[J[00:00:06.375,366] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[m\1b[8D\1b[J[00:00:06.375,457] \1b[1;33m<wrn> shell_uart: RX ring buffer full.\1b[0m
\1b[1;32muart:~$ \1b[m\1b[8D\1b[J\1b[1;32muart:~$ \1b[m

Environment (please complete the following information):

OS: Linux
Commit SHA or Version used: 67d851fb648acb5bea1f0a26a7c369437684d66f

@jakub-uC  Can you please take a look and see if this is still bug?

@nordicjm  which board did you use?


Build wheels on Windows

shamelessly stole the CI file from rabbitizer.
Don't ask me what the stuff does exactly, I just know it somehow works

Thanks 🥳


Custom Report Custom Layout: Data at the root level is invalid. Line 1, position 1 error

There's some kind of issue with (probably) txt2al causing some problems with rdl files.:
The issue was closed, but it's still happening, as is noted in the thread, but I can't reopen it..
Message: Data at the root level is invalid. Line 1,position 1.
#1513
Actually, the custom report compiles, deploys, and runs fine, so it may be a slightly different issue.
However if I go to Report Layouts and try to change it to "Custom Report" NAV crashes.
The server "DynamicsNAV110" was unable to process the request. The application will close.
Or, from the Custom Report Layouts page, if I click New and select my report, it crashes as well. (Not if I choose the checkbox for the Word layout.)
AL 15.18771
NAV 11.0.22292

You're using an unsupported combination of AL and NAV, please see https://github.com/Microsoft/AL/wiki/Frequently-Asked-Questions#which-version-of-the-al-language-extension-should-i-use

I tried it with CU5, same result.
AL 0.12.17091
NAV 11.0.22292.0

If you can reproduce the issue on the latest developer preview builds (AL 0.17, BC 13.0) we will investigate it further. Otherwise please contact regular support channels.

I haven't had a chance to try that yet.
Anyways. I ran into a bug in your product, and reported it as a courtesy, and as a knowledge base in case it happens to someone else.
I just want to clarify that I will leave it up to you and your team how to proceed.
I am just reporting a bug I ran in to on your product as a courtesy, and to create a knowledge base in case someone else runs into the issue as well.


Error: Fody: Could not find a weaver named 'BasicFodyAddin'

[x] Uploaded a minimal solution that reproduces the problem with the fewest moving pieces. Preferably a public GitHub repository.

Modified a tiny bit the original repository https://github.com/mfkl/BasicFodyAddin
Built it with dotnet build -c Release BasicFodyAddin.sln
Added it to the target project targeting netstandard2.0, xamarin.ios etc. as a PackageReference
Added a PackageReference to fody and a FodyWeavers.xml to the target project with
<?xml version="1.0" encoding="utf-8"?>
<Weavers>
    <BasicFodyAddin />
</Weavers>

No matter what I try, Fody does not find it. BasicFodyAddin does not end up in SolutionDir/packages either, though it is in /Users/Me/.nuget/packages. I can't figure out the part I'm missing to make this work.

Does it worl with un-modified weavers?

No.
I'm on Mac, and to be able to build this repo after a fresh clone I had to remove the net4x target frameworks to make a netstandard only plugin (that will operate on a xamarin.ios assembly later).
I managed to make Fody find the plugin by renaming the folder "netstandardweaver" to "netclassicweaver".
I haven't tried building on Windows, but I suspect I'd not encounter this issue.
Are you sure of these lines?
https://github.com/Fody/Fody/blob/d4ecec3fb87c5e770abeb9bd223478b6494747cd/Fody/AddinFinder/AddinSearchDirectories.cs#L116

Pretty. What do u think it should be?

I don't know. Will let you know if I find out more about this.


(PUP-5428) Add a query string with the environment when using POST

Hello,
This PR is solving issue https://tickets.puppetlabs.com/browse/PUP-5428.
When the length of the uri is greater that 1024, the GET is converted in POST and all data (environment, facts) are stored in the body request.
Adding a query string when posting allows to loadbalance catalog compilation (for example) to specific puppetserver.
Regards
Olivier

This is an intriguing change. It's adding metadata that Puppet will ignore, for use by a load balancer. I assume load balancers can't rely on the POST body because it would be encrypted with HTTPS.
I don't see a problem with this change, but the API documentation should be updated. Since it appears the indirector will convert any GET to a POST, I'd add a section in https://github.com/puppetlabs/puppet/blob/master/api/docs/http_api_index.md about it.
Also, @cprice404 does this present any problems that you can think of?

Hey Mikael,
I didn't see it before, but yes, the documentation says :
Every HTTP endpoint that dispatches to the indirector follows the form: /puppet/v3/:indirection/:key?environment=:environment
In this particular case, when a GET with an uri greater that 1024 is converted in a POST call, this statement becomes false.
Olivier

Ah, that should be sufficient for documentation then.

Hello, any updates ?

Hey @olivierHa, with people out for the holidays we haven't been able to get the eyes we need on this change. We'll get back to you as soon as we can.

Ok, got sufficient thumbs up.


feat(assets): surface the CFN Parameters that Assets create

This is needed in order to override them when deploying the Stack through CodePipeline.

Pull Request Checklist

[x] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[x] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated

[x] Title and Description

Change type: title prefixed with fix, feat will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[x] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

There is a pretty serious issue with this solution. Because Assets check whether the file/directory they point to exists, this forces the build in CodePipeline to happen sequentially - first the packaging of the Lambda source into a deployable artifact, and only then cdk synth NameOfLambdaStack. This is wasteful, as other than checking for that file existing, the CDK process does not use that artifact in any way, so the 2 builds should happen in parallel in the Pipeline.

Because Assets check whether the file/directory they point to exists

Fair point. The Correct Solution (tm) is probably to add a checkForExistence: true parameter to the assets. If left undefined, we use the default, and the default can be based on whether or not we're running in CI.
The toolkit already has a --ci flag, with logic to autodetermine whether it's most likely running on CI. We can pass that value to the CDK app either via context or an ENV_VAR.

Updated according to comments. Let me know if this is better @eladb and @rix0rrr !

Added mention of autoDeploy: false in the ReadMe of Lambda.

Rebased on top of all of the changes (in particular, #2098 , which allows us to depend on the aws-cdk/assets package from the module that contains the Artifact class used in CodePipeline).
@rix0rrr, @eladb , I would appreciate another run through the PR!

Talked with Elad offline about this PR. We decided not to go through with it.


🌱 Update golangci-lint 1.50.1, add more linters

Summary
This PR updates golanci-lint, adds many new linters, and fixes up a number of linter-found issues. For simplicity of review, the commits are split into multiple ones, although happy to squash them if needed.
Related issue(s)
Fixes #

Can we hold that until the workspace refactoring has merged? Fearing rebase hell.

@sttts Absolutely, I forgot to put the hold after talking with Andy, I'll wait and do a rebase on your refactoring once it's ready
/hold

/hold cancel

/lgtm

Let me know if we'd prefer to have a single commit here, or keep them separate, happy to do either way

/retest

Don't think multiple commits will hurt anything - re-triggering the tests to clear flakes

/lgtm

@vincepri if you can fix the remaining lint failure(s) and rebase, we can get this in 😄

/lgtm
/approve


update copyright headers

Revision of Copyright in all files

This is a bit outdated. I will redo it.


Add Russian Peasant Algorithm

I would like to add the Java code for the Russian Peasant Algorithm.
Please assign this to me as a part of GSSOC'21

I would love to contribute in this issue.  My language of code will be C++. Please assign this issue to me.

I would like to add the Java code for the Russian Peasant Algorithm.
Please assign this to me as a part of GSSOC'21

Go ahead and implement in Java

I would love to contribute in this issue. My language of code will be C++. Please assign this issue to me.

Go ahead and implement it in C++


Less command with images demo

I have idea for a new feature for jQuery Terminal
It would be cool to have demo of less command that work with images.
As starting point here is demo that renders image by slices. The slices are height of the lines so I think it will be possible to use less with this image.
The problem is that less accept chunk of text as string and slices demo use echo with raw options.

The problem was resolved by adding @  into formatting that will render images. So now less split images into slices and render then line by line.

released in 2.8.0


Added Feature to Get-VivaInsightsSettings

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-VivaInsightsSettings.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-VivaInsightsSettings.md
:white_check_mark:Succeeded
View (exchange-ps)

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


(Most Wanted/Carbon) Quit Game button on gamepad doesn't work in menu

In the latest version I found an issue with quit game button on gamepad. When I press Back/Veiw button on my Xbox gamepad in menu nothing happens at all, but when I press Q key on keyboard it works. Other front-end actions on my gamepad works correctly. I don't know what's the problem.

there never was a quit button on console so an action to it is currently not mapped
improvements past console parity will come after every version of the script is stable and rumble is implemented

It is mapped.
Xbox 360: Back
PS3: Select
Xbox One: View
PS4/PS5: Share
In MW and Carbon it only works during FrontEnd mode. In Carbon, in addition to that, it only works on the main menu (due to the way its state manager is coded).
Meaning in MW, you can press it anywhere during FrontEnd/menus.
In Carbon, you can only press it at the main menu.
Now my question is - are you pressing it where it's designed to be pressed or not? It is fully working for me.

@xan1242 this feature works up until 1.6 version
Starting from 1.7 it stopped working.

Yeah I see the mistake, it happened during UC definition implementation for the compiler directives...
It'll get fixed very soon.

fixed in 429371b
Closing.


Linux Version Uses System Qt Libs Instead of Embeeded

Ref: https://news.ycombinator.com/item?id=15277894

I am actively working on this by setting the rpath at build time. Running into a few snags, will update when solved.

I believe I have fixed this. I will update this issue when I have cut another release with it fixed.

I have released https://github.com/cretz/doogie/releases/tag/v0.6.1 which I believe fixes this issue. Anyone having this issue, please test. Closing for now, but reopen if issues persist.


ngspice: 37 -> 40

Description of changes
Closes #199545.
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Tested with https://github.com/NixOS/nixpkgs/pull/223378#issuecomment-1536609445.


improve type annotations

Even tough pytype doesn't report any type errors, Pylance reports over 100 typing issues, mostly due to partial type definitions, e.g list should be e.g. list[...].

pytype's doc states that type annotations can be automatically added with merge-pyi -i <filepath>.py .pytype/pyi/<filename>.pyi. If this produces what Rylance requires, it could reduce drastically the effort.

mypy complains that it doesn't have typing stubs for pytype but pytype's doc says they're on typeshed, so shouldn't mypy get them? The make lint script tells mypy to ignore such errors.

Pylance now only reports 3 errors, 2 of them being related to pytype.


use vagrant 0.17

https://rackhd.atlassian.net/browse/RAC-4502
About vagrant 0.17

Has installed current usc-service requirements, consider the possible updates of requirements, so keep the pip install steps in Vagrantfile.
mongodb.conf has been put into this new box.
new pm2 yml has been put into this new box.
remove mongo blacklist, mongodb version is restricted to 2.4.9 in the box
Build with virtualbox v5.0.32

Extra:

Remove unused folder sync. In manifest mechanism all repo src has been prepared to the final status. No more extra sync actions are needed.
Remove dhcpd.conf for esxi. I checked the injected conf and found it removed the dns config. But tests showed that with dns the esxi os installation still succeeded.
Comment out the mongo upgrade. External package version updates introduce a lot of instability to our system. So we disscussed to keep all the external dependency to fixed version. So mongodb version to 2.4.9 in the box. However, this topic is still under discussion, so I just comment out the code snippet.

Packer Modification PR: https://github.com/RackHD/RackHD/pull/749

test this please

BUILD on-build-config #242 : FAILURE

BUILD on-build-config #247 : FAILURE

test this please

test this please

:+1:

BUILD on-build-config #259 : FAILURE

BUILD on-build-config #259  Error Logs  ▼Test Name: post_skupacks
Error Details: Bad Request
-------------------- >> begin captured logging << --------------------
tests.api.v2_0.sel_alert_poller_tests: INFO: ipmi ipmitool_command: ipmitool -I lanplus -H <IP_ADDRESS> -U admin -P admin sel info
tests.api.v2_0.sel_alert_poller_tests: INFO: URL http://localhost:9090/api/2.0/skus/pack
tests.api.v2_0.sel_alert_poller_tests: INFO: Number of attempt to post  the skupack :  0
requests.packages.urllib3.util.retry: DEBUG: Converted retries value: 0 -> Retry(total=0, connect=None, read=None, redirect=None)
requests.packages.urllib3.util.retry: DEBUG: Converted retries value: 0 -> Retry(total=0, connect=None, read=None, redirect=None)
requests.packages.urllib3.connectionpool: INFO: Starting new HTTP connection (1): localhost
requests.packages.urllib3.connectionpool: DEBUG: "POST /api/2.0/skus/pack HTTP/1.1" 201 45
tests.api.v2_0.sel_alert_poller_tests: INFO: ipmi ipmitool_command: ipmitool -I lanplus -H <IP_ADDRESS> -U admin -P admin sel info
tests.api.v2_0.sel_alert_poller_tests: INFO: URL http://localhost:9090/api/2.0/skus/pack
tests.api.v2_0.sel_alert_poller_tests: INFO: Number of attempt to post  the skupack :  0
requests.packages.urllib3.util.retry: DEBUG: Converted retries value: 0 -> Retry(total=0, connect=None, read=None, redirect=None)
requests.packages.urllib3.util.retry: DEBUG: Converted retries value: 0 -> Retry(total=0, connect=None, read=None, redirect=None)
requests.packages.urllib3.connectionpool: INFO: Starting new HTTP connection (1): localhost
requests.packages.urllib3.connectionpool: DEBUG: "POST /api/2.0/skus/pack HTTP/1.1" 400 192
--------------------- >> end captured logging << ---------------------
Stack Trace:   File "/usr/lib/python2.7/unittest/case.py", line 331, in run
    testMethod()
  File "/usr/lib/python2.7/unittest/case.py", line 1043, in runTest
    self._testFunc()
  File "/home/jenkins/workspace/on-build-config/RackHD/test/.venv/on-build-config/local/lib/python2.7/site-packages/proboscis/case.py", line 296, in testng_method_mistake_capture_func
    compatability.capture_type_error(s_func)
  File "/home/jenkins/workspace/on-build-config/RackHD/test/.venv/on-build-config/local/lib/python2.7/site-packages/proboscis/compatability/exceptions_2_6.py", line 27, in capture_type_error
    func()
  File "/home/jenkins/workspace/on-build-config/RackHD/test/.venv/on-build-config/local/lib/python2.7/site-packages/proboscis/case.py", line 350, in func
    func(test_case.state.get_state())
  File "/home/jenkins/workspace/on-build-config/RackHD/test/tests/api/v2_0/sel_alert_poller_tests.py", line 87, in post_skupacks
    assert_equal(201, res.status_code, message=res.reason)
  File "/home/jenkins/workspace/on-build-config/RackHD/test/.venv/on-build-config/local/lib/python2.7/site-packages/proboscis/asserts.py", line 55, in assert_equal
    raise ASSERTION_ERROR(message)
'Bad Request\n-------------------- >> begin captured logging << --------------------\ntests.api.v2_0.sel_alert_poller_tests: INFO: ipmi ipmitool_command: ipmitool -I lanplus -H <IP_ADDRESS> -U admin -P admin sel info\ntests.api.v2_0.sel_alert_poller_tests: INFO: URL http://localhost:9090/api/2.0/skus/pack\ntests.api.v2_0.sel_alert_poller_tests: INFO: Number of attempt to post  the skupack :  0\nrequests.packages.urllib3.util.retry: DEBUG: Converted retries value: 0 -> Retry(total=0, connect=None, read=None, redirect=None)\nrequests.packages.urllib3.util.retry: DEBUG: Converted retries value: 0 -> Retry(total=0, connect=None, read=None, redirect=None)\nrequests.packages.urllib3.connectionpool: INFO: Starting new HTTP connection (1): localhost\nrequests.packages.urllib3.connectionpool: DEBUG: "POST /api/2.0/skus/pack HTTP/1.1" 201 45\ntests.api.v2_0.sel_alert_poller_tests: INFO: ipmi ipmitool_command: ipmitool -I lanplus -H <IP_ADDRESS> -U admin -P admin sel info\ntests.api.v2_0.sel_alert_poller_tests: INFO: URL http://localhost:9090/api/2.0/skus/pack\ntests.api.v2_0.sel_alert_poller_tests: INFO: Number of attempt to post  the skupack :  0\nrequests.packages.urllib3.util.retry: DEBUG: Converted retries value: 0 -> Retry(total=0, connect=None, read=None, redirect=None)\nrequests.packages.urllib3.util.retry: DEBUG: Converted retries value: 0 -> Retry(total=0, connect=None, read=None, redirect=None)\nrequests.packages.urllib3.connectionpool: INFO: Starting new HTTP connection (1): localhost\nrequests.packages.urllib3.connectionpool: DEBUG: "POST /api/2.0/skus/pack HTTP/1.1" 400 192\n--------------------- >> end captured logging << ---------------------'

test this please

BUILD on-build-config #261 : FAILURE

test this please


Request to change package name

Description
You published a the package without changing the name "Trevenant" from the template.
Could you please chang the name of the package as soon as possible
Thanks
Anything else?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thanks for the contribution!


split large audio file [help]

I have successfully used this library for concatenating audio files, but I was wandering if it can be used for splitting files too.
  const reader = new FileReader();
  const blob = file.slice(offset, chunkSize + offset);
  r.onload = readEventHandler;
  r.readAsArrayBuffer(blob);

reading the file in chunks, and then trying to get AudioBuffer in the readEventHandler
const readEventHandler = evt => {
    const arrayBuffer = evt.target.result as ArrayBuffer;
    crunker._context.decodeAudioData(
          arrayBuffer,
          d => console.log(d),
          e => console.log(e)
    );
}

the problem is that only the first part of the file is getting decoded since only it has the necessary headers. For the rest of the parts I am getting
DOMException: Unable to decode audio data.
I tried to manually add the headers to each chunk with modified writeHeaders function , but the audio I get is somehow too fast.
Any suggestions?
const writeHeaders = (arrayBuffer: ArrayBuffer) => {
  // const arrayBuffer = new ArrayBuffer(44 + buffer.length * 2);
  const view = new DataView(arrayBuffer);
  const sampleRate = 44100;

  writeString(view, 0, 'RIFF');
  view.setUint32(4, 32 + arrayBuffer.byteLength * 2, true);
  writeString(view, 8, 'WAVE');
  writeString(view, 12, 'fmt ');
  view.setUint32(16, 16, true);
  view.setUint16(20, 1, true);
  view.setUint16(22, 2, true);
  view.setUint32(24, sampleRate, true);
  view.setUint32(28, sampleRate * 4, true);
  view.setUint16(32, 4, true);
  view.setUint16(34, 16, true);
  writeString(view, 36, 'data');
  view.setUint32(40, arrayBuffer.byteLength * 2, true);

  return floatTo16BitPCM(view, arrayBuffer, 44);
};

@unixxxx If the audio is too fast, it could be that the arrayBuffer is using the original byteLength rather than the new split length?
Did you have any luck with this? I think splitting would be a good method to add to this project.

If you use decodeAudioData() first you can get any part of the AudioBuffer from any offset.
If you do not split the file first you can use AudioWorklet or Media Capture Transform to get the exact time slices expected as Float32Arrays and then merge the data.
For live-streaming you can utilize RTCRtpSender.replaceTrack() with MediaRecorder to dynamically change the track(s).

This https://github.com/guest271314/audioInputToWav  (is not fast) will create a WAV file from an AudioBuffer, or when modified, Float32Array(s).
If you are trying to concantenate specific time slices of of multiple audio (and, or, video) Media Fragment URI can be useful https://github.com/guest271314/MediaFragmentRecorder.

decodeAudioData() requires metadata. It is possible to create files from the output when the entire file is passed to the function.


Possible inaccuracy of SOAP descriptors for periodic systems

Hi,
I am using your package to calculate SOAP vectors for the atoms in my system. It is working very nicely, except that I get small deviations which manifest around the boundary when I try to classify the local structure.
To reproduce this, I have created the following example:

Create a cubic cell with atoms.
Redefine the cubic cell as monoclinic with an angle of 45 degrees, wrap the atoms back inside the cell.
Compute the difference in SOAP descriptors before and after. Ideally, this should be zero (or machine precision) in a periodic system. Instead, I get about 1e-7. The Ewald matrix achieves machine precision.

The error seems to depend on the radial cutoff (which shouldn't matter.) I have attached the script and its output.
Am I interpreting these results correctly?
import numpy as np
from numpy.linalg import norm
from numpy.random import normal
from ase import Atoms
from ase.visualize import view
from ase.io import write
from dscribe.descriptors import SOAP, EwaldSumMatrix

ncells = 5
natoms = 2 * ncells ** 3

# Loop over different radial cutoffs
for rcut in np.linspace(5, 8, 16):

    # Create descriptor generators
    soap_generator = SOAP(
        rcut=rcut, nmax=4, lmax=4, species=["Ni", "Ti"], periodic=True
    )
    ewald_generator = EwaldSumMatrix(natoms, permutation="none", flatten=False)

    # Define unit cell
    a = 2.993
    niti = Atoms(
        "NiTi",
        positions=[[0.0, 0.0, 0.0], [a / 2, a / 2, a / 2]],
        cell=[a, a, a],
        pbc=[1, 1, 1],
    )

    # Replicate system
    niti = niti * ncells
    a *= ncells

    # Add some noise to positions
    positions = niti.get_positions()
    noise = normal(size=positions.shape)
    niti.set_positions(positions + noise)
    niti.wrap()

    # Write positions of orthogonal unit cell
    write("orthogonal.xyz", niti)

    # Evaluate descriptors for orthogonal unit cell
    orthogonal_soaps = soap_generator.create(niti)
    orthogonal_ewalds = ewald_generator.create(niti)

    # Redefine the cubic unit cell as monoclinic
    # with a 45-degree angle,
    # this should not affect the descriptors
    niti.set_cell([[a, 0, 0], [0, a, 0], [a, 0, a]])
    niti.wrap()

    write("nonorthogonal.xyz", niti)

    # Evaluate descriptors for new, monoclinic unit cell
    non_orthogonal_soaps = soap_generator.create(niti)
    non_orthogonal_ewalds = ewald_generator.create(niti)

    # Calculate error for the two descriptors
    soap_error = norm(orthogonal_soaps - non_orthogonal_soaps) / norm(orthogonal_soaps)
    ewald_error = norm(orthogonal_ewalds - non_orthogonal_ewalds) / norm(
        orthogonal_ewalds
    )
    print(
        f"rcut = {rcut:.2e}, SOAP error: {soap_error:.2e}, Ewald error: {ewald_error:.2e}"
    )

"""
Sample output:
rcut = 5.00e+00, SOAP error: 0.00e+00, Ewald error: 1.19e-15
rcut = 5.20e+00, SOAP error: 0.00e+00, Ewald error: 1.14e-15
rcut = 5.40e+00, SOAP error: 0.00e+00, Ewald error: 1.54e-15
rcut = 5.60e+00, SOAP error: 0.00e+00, Ewald error: 7.71e-16
rcut = 5.80e+00, SOAP error: 0.00e+00, Ewald error: 9.89e-16
rcut = 6.00e+00, SOAP error: 1.62e-08, Ewald error: 7.77e-16
rcut = 6.20e+00, SOAP error: 1.27e-08, Ewald error: 7.33e-16
rcut = 6.40e+00, SOAP error: 3.32e-08, Ewald error: 1.03e-15
rcut = 6.60e+00, SOAP error: 5.91e-08, Ewald error: 1.10e-15
rcut = 6.80e+00, SOAP error: 2.20e-07, Ewald error: 1.61e-15
rcut = 7.00e+00, SOAP error: 2.26e-07, Ewald error: 9.01e-16
rcut = 7.20e+00, SOAP error: 2.99e-07, Ewald error: 1.14e-15
rcut = 7.40e+00, SOAP error: 1.13e-06, Ewald error: 1.33e-15
rcut = 7.60e+00, SOAP error: 0.00e+00, Ewald error: 1.28e-15
rcut = 7.80e+00, SOAP error: 0.00e+00, Ewald error: 1.61e-15
rcut = 8.00e+00, SOAP error: 0.00e+00, Ewald error: 9.49e-16
"""

Hi @anjohan!
Thanks for your message. I will look into this and let you know when I
figure out what is going on.

Hi @anjohan!
The error seems to be isolated to the newest pip version 0.2.9. For a quick fix, use an older version, e.g. 0.2.8. I will try to make a hotfix ASAP.

Thank you very much for the quick response and solution!
This does indeed remove the error for my test case. I had noticed that the periodicity code was evolving, so I tried the newest (Github) version as well, but I didn't think to try older versions.
If it helps in your debugging, I noticed that the error goes away if you don't add the noise to the coordinates, i.e. when the structure is a perfect crystal.
There is still some boundary problem for my "real" application, but that is probably on my end now that dscribe handles the periodicity correctly for the test case above.

Brief update: It seems that after wrapping the atoms to the new monoclinic cell (niti.wrap()), the atomic configuration near the boundaries is not completely identical with the cubic case. This is because the wrapping may throw some atoms to the other side of the cell if they near the border (see the eps parameter in atoms.wrap()). The effect is pronounced in your example by the fact that the system is very big == there is a lot of boundary.
I will have to see if it makes sense to increase the default padding that we use to prevent these boundary effects. I wouldn't want to crank this padding too high, because large values incur a performance penalty when extending the system.
For most applications this error is still relatively small. I will have to think about a clever way to decide the padding for the next release.

Thank you for the update and continued effort.
With the downgrade to 0.2.8 and a bugfix in my own handling of triclinic cells, the SOAP vectors now work perfectly for my application!

The master branch is now updated with a version that uses larger padding that also scales according to the provided sigma value. This version is accompanied by a new regression test (roughly based on the code provided above) that checks the accuracy with different cutoffs, sigmas and cell sizes.

The fix to this issue can now be found in version 0.3.0.


Cannot use multiple Bugsnag instances

Expected behavior
Let's say I'm a framework developer who wants to use Bugsnag for tracking crashes. I instantiate Bugsnag in my framework code:
BugsnagConfiguration *configuration = [BugsnagConfiguration new];
configuration.apiKey = @"apiKey1";
[Bugsnag startBugsnagWithConfiguration:configuration];

Separately, an app developer consuming my framework also decides to use Bugsnag:
BugsnagConfiguration *configuration = [BugsnagConfiguration new];
configuration.apiKey = @"apiKey2";
[Bugsnag startBugsnagWithConfiguration:configuration];

I expect Bugsnag to recognize that the API keys are different and internally uses different notifiers and other shared state so that clients with different API keys receive the information that corresponds to each respective configuration.
Observed behavior
The one notifier internal to the Bugsnag class is set to a BugsnagNotifier associated with the API key that was passed to the start method most recently.
This means that if an app developer starts Bugsnag, then initializes my framework (which starts Bugsnag with its own configuration), the Bugsnag application associated with my framework's API key receives all events, and the Bugsnag application associated with the app developer's API key receives no events.
I've attached a sample project which demonstrates this behavior, as seen from the Bugsnag dashboards for two separate Bugsnag applications here:

Steps to reproduce

Start Bugsnag with configuration with API key 1
Start Bugsnag with configuration with API key 2
=> The internal BugsnagNotifier is overwritten by the second call to startBugsnagWithConfiguration, so the app associated with API key 1 no longer receives any events.

Version
master, and all prior releases.
Additional information
This appears to be due to the use of internal singletons within Bugsnag. There may be other related bugs due to internal singletons such as BSG_KSCrash.
Related issue on Bugsnag Android: https://github.com/bugsnag/bugsnag-android/issues/285
Sample project demonstrating this bug:
BugsnagExample.zip

Thanks @gizmosachin for the very detailed report and sample project - I can confirm that this appears to be an issue for any application which has two Bugsnag integrations.
I'll investigate whether there are any more singletons that might get in the way of this use-case (for instance, BugsnagNotifier appears to have at least one). I'll also check whether there are any hidden behaviours (e.g. ensuring reports are saved to separate locations for each Bugsnag integration). We'll get back to you as soon as we have a course of action.

Thanks @fractalwrench!

We've had a think about this internally and we don't think that supporting multiple Bugsnag clients is a good idea. The SDK is relatively large and also hooks into global handlers. Having a non crash detector hook into these global handlers will surprise many developers and so it doesn't feel like a common use case. Happy to talk this through if you like.

This was previously recommended by the Bugsnag support team. As others have discovered, it fails in several ways.
To work around it, I had to fork the Bugsnag (and bugshag-ndk) projects. Not ideal at all, but it does work.

As far as I'm aware the only use case for multiple clients is trying to monitor a library/sdk and not interfere with the application that may also be using Bugsnag. Is there another use case I haven't considered?

Micro front-end architectures where separate apps are docked into a single bigger application need their own bugsnag instances to the respective teams can work independently. This is a big issue for us as it previously worked before the recent updates to the API.

We use a microfrontend architecture. Navigating between different tools owned by different teams in our product does not trigger a full page reload. We are running into issues where errors are showing up in the wrong bugsnag project if bugsnag was initialized in one tool, then the user navigated to another tool, then an uncaught exception occurred. Having separate bugsnag instances is totally valid, and should be supported.

This was previously recommended by the Bugsnag support team. As others have discovered, it fails in several ways.
To work around it, I had to fork the Bugsnag (and bugsnag-ndk) projects. Not ideal at all, but it does work.

Hi,
I am having the same issue with our SDKs too. I would like to know how did you managed to solve the problem please ?


please surpport send input event to extented display on android 9

adb shell input --ext-display tap/swipe... This commend can sending event to extented display,
so it is very useful for scrcpy --display 2 .
it also can solve the issue "Input events are not supported for secondary displays before Android 10".
Because "adb shell input --ext-display keyevent " can send input to secondary displays on android 9.
Thank you very much.

device is   smartisan  nut  pro3


Print error and exit 1 instead of panicking in CLI

This was removed earlier because if a diesel_cli test started failing,
it would abort the entire test suite because it exited with a status
code of 1, which rust interprets as a problem.  This presentation is much nicer than a
panic.

:+1:


Label component bug

Please make sure to target feature and bugfix requests to develop branch
Issue https://github.com/basemate/matestack-ui-core/issues/262:
Update label stand alone component to accept for attribute for correct element binding
Changes

[x] updated docs
[x] updated label (rb)
[x] updated specs (rb)

yessirr


feat: Add a spark.comet.exec.memoryPool configuration for experimenting with various datafusion memory pool setups.

Which issue does this PR close?
This issue relates to https://github.com/apache/datafusion-comet/issues/996 and https://github.com/apache/datafusion-comet/pull/1004
Rationale for this change
This is for investigating various approaches to simplify memory-related configuration and reduce the memory required to run large queries. @andygrove
What changes are included in this PR?
This PR adds a spark.comet.exec.memoryPool configuration for easily running queries using various memory pool setups.

greedy: Each operator has its own GreedyMemoryPool, which is the same as the current situation.
fair_spill: Each operator has its own FairSpillPool
greedy_task_shared (default): All operators for the same task attempt share the same GreedyMemoryPool.
fair_spill_task_shared: All operators for the same task attempt share the same FairSpillPool
greedy_global: All operators in the same executor instance share the same GreedyMemoryPool
fair_spill_global: All operators in the same executor instance share the same FairSpillPool

How are these changes tested?
TODO: add tests running in native memory management mode.

Codecov Report
Attention: Patch coverage is 63.04348% with 17 lines in your changes missing coverage. Please review.

Project coverage is 55.04%. Comparing base (591f45a) to head (4607e8d).
Report is 1 commits behind head on main.

Files with missing lines
Patch %
Lines

...ain/scala/org/apache/comet/CometExecIterator.scala
60.46%
13 Missing and 4 partials :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #1021       +/-   ##
=============================================
+ Coverage     34.30%   55.04%   +20.74%
+ Complexity      887      860       -27
=============================================
  Files           112      109        -3
  Lines         43429    10923    -32506
  Branches       9623     2109     -7514
=============================================
- Hits          14897     6013     -8884
+ Misses        25473     3832    -21641
+ Partials       3059     1078     -1981

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

Thanks for filing this @Kontinuation. I will close my PRs.
Also, there is a suggestion in https://github.com/apache/datafusion-comet/issues/1017 for always using unified memory.

Thank you @Kontinuation Do you mind rebasing onto the latest main?

I have noticed that for now Comet biases towards always using the unified memory manager, so this patch for native memory management may not be relevant, so I prefer closing this PR. If you still want to merge it into the upstream then I'll take some time to rebase it.

I have noticed that for now Comet biases towards always using the unified memory manager, so this patch for native memory management may not be relevant, so I prefer closing this PR. If you still want to merge it into the upstream then I'll take some time to rebase it.

Just to add some context here, we have discovered a need to support existing deployments that are not using offheap memory, so would like to add a non-offheap option back in, and this PR seems like the best starting point for deciding how we want to configure the memory pools in that case.

Just to add some context here, we have discovered a need to support existing deployments that are not using offheap memory, so would like to add a non-offheap option back in, and this PR seems like the best starting point for deciding how we want to configure the memory pools in that case.

OK. I have rebased this branch onto main.


fix(docs): Resend integration code

Fixed Resend Integrations code under docs>integrations>resend.
Under Step 3. Send Email, the code imports Email component but uses EmailTemplate component instead of Email component. So I Updated it to use Email component.

Thanks @Arslan-Soomro!


ARROW-2090: [Python] Add context methods to ParquetWriter

Added the __enter__ and __exit__ methods to the ParquetWriter
Class. This allows you to write code in the following style:
with ParquetWriter(foo, schema) as writer:
    writer.write_table(table)

And the ParquetWriter context object will handle calling the close
method for you when the with block exits.  It propagates errors
in line with previous behavior. I have also updated the module
level write_table method to use the new with style.
And it maintains it's existing behavior of trying to remove
a partial file if an exception is encountered.
The reason for this change, was a bug I encountered while
using the ParquetWriter object, where I wasn't closing
the writer correctly in the event of an exception, which resulted
in partially written parquet files. Adding the ability to use
with syntax will reduce the chances of a similar misuse
while still maintaing backwards compatibility.

I'm not sure why the plasma store tests are failing. As far as I can work out, the code I changed doesn't touch the plasma code at all.

The Plasma failures are likely unrelated, we've been experiencing problems in Travis CI the last couple weeks (see ARROW-2062, ARROW-2071, others)


Build a 100k cover db of all genomes and subtract using that instead of organism-specific databases?

As mentioned in #4, this pipeline is supposed to target any distinct transcriptome sequence, meaning anything that has never been observed in any DNA before. (I don't think we need to worry about RNA contaminating DNA...since rtPCR isn't a step in DNA sequencing, I think any RNA contam would just...disapper? I should check this). If we are targeting truly unique RNA mods (splicing, editing, nucleic acid modifications, ...), I think it would be ok to subtract all DNA sequences ever.
This would also probably remove contamination, which would be nice (see #4).
This would also have the benefit of allowing us to include all of the k-mers from a given species as a cover, which might reduce problems with heterozygosity (see #3). It will also dramatically simplify the encoding of the workflow -- we wouldn't need to know what organism we were working with before doing the subtraction, and we wouldn't need to specify which genome to subtract.
See https://github.com/ctb/2022-database-covers and https://github.com/sourmash-bio/sourmash/issues/1852 for cover DB construction. I would want to work with the latest genbank, as well as Eukaryotic genbank. For Euk genbank, I'd need to download and sketch all of the genomes (except protists and fungi which are already in the genbank db). (Note that here including repeat elements and contamination wouldn't cause problems 🎉 )
Also note that over in https://github.com/greenelab/2022-microberna/, I already sketched all of the microbial transcriptomes on the SRA and all of the signatures are CCBy so...
I don't think this would scale to metagenomes though yet...although I don't know how scaled100k would improve the size of that.

if i do this, I should benchmark running gather with a threshold bp of 0 and outputting unassigned vs. subtracting using the zip of the db. I don't have a good guess as to which would be faster (I'm also not sure if output unassigned keeps the abundances of query signature)
Also it would be fun to have these profiles for everything (although they could be wrong in weird ways with repetitive elements and such).


Fix panic on empty findTraces query

Which problem is this PR solving?
Closes #2996
Short description of the changes
Added validation checks to the gRPC server handler methods that implement the QueryServiceClient interface.
Couple of notes:

I created a gRPC Go client to attempt to test when the request parameters are nil - however the client looks like it panics before it reaches the server.  Is there a way I can write a unit test for this functionality?  As I understand it perhaps not all clients generated in other languages python, java, javascript etc... would fail before sending request to the server.
GetServices service method: the request parameter r *api_v2.GetServicesRequest not used so I didn't add any validation for it.

Questions for the following service methods
GetOperations

GetOperationsRequest parameters are Service and Span which are type String.  Not sure what the behavior should be if they are "". Is it possible that these are valid arguments?

@akuzni2, do you think you can improve the test coverage of this PR?

@akuzni2, do you think you can improve the test coverage of this PR?

@jpkrohling - updated tests to meet code cov. Added tests to pass nil into the GRPCHandler


[MM-46478] - POC for all, channel and mention search

Summary
This PR is a POC for adding the option to search by channel, all and here.
Ticket Link
https://mattermost.atlassian.net/browse/MM-46478
Release Note
NONE

Heads up that as part of our efforts to move to a monorepo, we're closing out a number of older pull requests like this one to help streamline the effort.
If you'd like to preserve these changes -- even if you're not the original author! -- feel free to resubmit the pull request against the monorepo once it's ready. You can subscribe to mattermost-server-issue-22420 for status updates on this effort.


Bazel go rules: remove -w/-s from fastbuild options [phab:D649]

Created by schroederc at 2015-12-14 22:53:13:
These options do not significantly speed up the build with Go 1.5 and
more debug information without a cost is a good deal.

zarko accepted this revision at 2015-12-14 23:45:03

schroederc closed via commit ee78f223e089436f88a694831635b0774dacead0 at 2015-12-15 00:02:12


Button disabled state not updated during onclick function

Describe the bug
Stumbled into this while following documentation here: https://docs.microsoft.com/en-us/aspnet/core/security/blazor/server?view=aspnetcore-3.0#guard-against-multiple-dispatches
While following the guard against multiple dispatches example in the URL above, the button is not disabled  (visibly) to end-user while the forecast data is being retrieved.
To Reproduce
Steps to reproduce the behavior:

Using VS 2019 16.3.5 w/ .NET Core 3.0.
File -> New Project and create a Blazor App (Server) project
Modify Pages/FetchData.razor according to "Guard against multiple dispatches" example
Note: May be helpful to add an artificial delay to the GetForecastAsync service to prove that button is not being disabled.
Run page and click button to see it's not disabled at all

Expected behavior
The button should be disabled while isLoading = true.
Additional context
This is my full source of FetchData.razor:
@page "/fetchdata"

@using BlazorApp_MSBugRepro1.Data
@inject WeatherForecastService ForecastService

Weather forecast

This component demonstrates fetching data from a service.

@if (forecasts == null)
{

Loading...

}
else
{

                Date
                Temp. (C)
                Temp. (F)
                Summary

            @foreach (var forecast in forecasts)
            {

                    @forecast.Date.ToShortDateString()
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

            }

    <button disabled="@isLoading" @onclick="UpdateForecasts">Update</button>
}

@code {
    private bool isLoading;
    WeatherForecast[] forecasts;

    protected override async Task OnInitializedAsync()
    {
        forecasts = await ForecastService.GetForecastAsync(DateTime.Now);
    }

    private async Task UpdateForecasts()
    {
        if (!isLoading)
        {
            isLoading = true;
            //await Task.Delay(1);
            forecasts = await ForecastService.GetForecastAsync(DateTime.Now);
            isLoading = false;
        }
    }
}

You'll notice I left a commented line in there, await Task.Delay(1);. That is the workaround I have found in order to get the button to show as disabled while updating the forecast.

Does the call to GetForecastAsync largely perform work synchronously? The pattern specifically works if work's being done asynchronously. That might explain why the Task.Delay has a visible outcome.

Ah, looks like you're right! I didn't realize the GetForecastAsync function is in fact not asynchronous:
public Task<WeatherForecast[]> GetForecastAsync(DateTime startDate)
{
	var rng = new Random();
	return Task.FromResult(Enumerable.Range(1, 5).Select(index => new WeatherForecast
	{
		Date = startDate.AddDays(index),
		TemperatureC = rng.Next(-20, 55),
		Summary = Summaries[rng.Next(Summaries.Length)]
	}).ToArray());
}

Refactoring to the below seems to have done the trick:
public async Task<WeatherForecast[]> GetForecastAsync(DateTime startDate)
{
	return await Task.Run(() =>
	{
		var rng = new Random();

		return Enumerable.Range(1, 5).Select(index => new WeatherForecast
		{
			Date = startDate.AddDays(index),
			TemperatureC = rng.Next(-20, 55),
			Summary = Summaries[rng.Next(Summaries.Length)]
		}).ToArray();
	});
}

Looks like the default Blazor project template could use an update so those of us following the documentation don't run into this. 👍
Thanks!


TC analysis

New feature: TC analysis based on Balaguru at al. 2020: https://agupubs.onlinelibrary.wiley.com/doi/10.1029/2019MS002024
Fixes https://github.com/E3SM-Project/e3sm_diags/issues/405

[x] Standalone driver script with plotting
[x] Refactor to fit in e3sm-diags code base
[x] Add observation data post-processing and provenance
[x] Add model data post-processing script
[x] Create viewer
[x] Update dependency
[ ] Add Unit tests
[ ] Update documentation

@tomvothecoder Hey, this PR is ready for review.
Would you help to review and refactor if needed. Also add some unit test will be nice.
For testing locally, you will need to download data from an LLNL server:
/p/user_pub/e3sm/e3sm_diags_data/test_model_data_for_acme_diags/tc-analysis
/p/user_pub/e3sm/e3sm_diags_data/obs_for_e3sm_diags/tc-analysis
An example script to run is here: https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/zhang40/tc_anlasis_model_obs/prov/run_tc_analysis.py
The viewer show all the new plots is here: https://web.lcrc.anl.gov/public/e3sm/diagnostic_output/zhang40/tc_anlasis_model_obs/viewer/tc_analysis/index.html
Also note:

a new dependency netcdf4 is added
pre-commit was skipped due to an error i'm not sure to fix
There is a problem in building, which maybe related to cdms2

Please let me know if I can clarify anything. Thank you!

I rebased this branch on master to clean up the diff from ~55 files to 16 files.
The commit message for TC standalone driver script 9a9c458 was also cleaned up to remove content from merging the other commits.

Hi @chengzhuzhang, my refactoring and testing efforts were focused on tc_analysis_driver.py since it contains the core logic for this PR.
I extracted logical chunks of code from (1)generate_tc_metrics_from_te_stitch_files() and (2)generate_tc_metrics_from_obs_files() into smaller, testable modules.
These new functions include:

_calc_num_storms_and_max_len() - used by 2
_get_vars_from_te_stitch() - used by 2
_get_mod_vars() - used by 2
_get_mod_wind() - used by 1
_calc_mean_ace() - used by 1
_calc_seasonal_cycle() - used by 1 and 2

I am unable to write tests to cover the main functions (1 and 2), run_diags, and _get_mod_vars() in time for the v2.5.0 release deadline. These functions are more complex because it involves generating dummy .nc files, mocking cdms2.dataset.DataVariable for ocnfranc, and etc. For now, there are TODO: placeholders in case we want to revisit them in the future.
Next actions for tomorrow

[ ] Perform a diff of generated plots between your last commit a523b10 and my latest refactored commit
[ ] If there is no diff, squash my refactored commits
[ ] Add comments on diff between your last commit and my squashed commit for final review

After these are done, I'll able to move onto #452.

Hey Tom. Thank you for the clarification, and your effort refactoring. I haven't checked out your code, but the logics look good! #452 should be more straightforward to review. We can always add more testing after v2.5.0 release :)

I'm reading the refactored tc_analysis_driver.py again. I understand the benefit of having smaller, testable modules. Somehow having more small units make the codes less readable, especially that generate_tc_metrics_from_te_stitch_files() and generate_tc_metrics_from_obs_files() only have one common unit. I'm not a software engineer by training, but there must be a tricky balance...

I'm reading the refactored tc_analysis_driver.py again. I understand the benefit of having smaller, testable modules. Somehow having more small units make the codes less readable, especially that generate_tc_metrics_from_te_stitch_files() and generate_tc_metrics_from_obs_files() only have one common unit. I'm not a software engineer by training, but there must be a tricky balance...

Hi Jill, here is an excellent and straightforward blog post on why refactoring code into smaller functions/methods is beneficial. Let me know if you find this helpful.
If you're referring to readability in terms of navigating the file and jumping between functions, there's a system that is used. Usually the host functions (e.g., generate_tc_metrics...) is at the top, followed by the helper functions. The helper functions are listed in the same order as they are called in the host functions. I'm not sure what IDE you're using, but having a split screen of the same file helps a lot.
def func_a():
    _help_a()
    _help_b()

def func_b():
    _help_b():
    _help_c():

def _help_a():
     pass

def _help_b():
     pass

def _help_c():
     pass

If you're referring to code readability, functions should be responsible for one well-defined task which is stated in its name. By having many operations performed in one function, it makes it harder to understand how inputs are being transformed. A workaround is having comments over each of the logical groups of code, but most of the time it is better to extract those groups into functions with clear names (e.g., _calc_seasonal_cycle()). The "Naming" section of the blog post also mentions this.
Also as you mentioned, unit testing is significantly easier with smaller units of code!

Hey Tom, thanks for the explanation and pointing to the reading materials. It helps! I'm still using old fashion vim and occasionally reading directly from Github. With this, I guess jumping around code blocks makes reading less efficient. Another point is that a descriptive and precise name will help improve readability. Let me know once you have completed refactoring and I maybe able to try renaming some functions..Thanks!

Hey Tom, thanks for the explanation and pointing to the reading materials. It helps! I'm still using old fashion vim and occasionally reading directly from Github. With this, I guess jumping around code blocks makes reading less efficient. Another point is that a descriptive and precise name will help improve readability. Let me know once you have completed refactoring and I maybe able to try renaming some functions..Thanks!

@chengzhuzhang, I'm glad it helped! I'm always open to other suggestions, ideas, or concepts if they seem to provide more value :smile:
Here's the diff between your commit and my squashed commit. I performed QA by writing a quick script to get the diff between plot outputs. There were no differences so we are good to go! I also added PR review comments that you can look over.
I agree, having clearer names for the functions/variables will help a lot for code readability. You can go ahead and checkout the latest commits.

Hey Tom, I suggested some minor changes. Looking good!

Hey Tom, I suggested some minor changes. Looking good!

Hi Jill, were you going to add review comments in the diff?

@chengzhuzhang, I've incorporated your latest feedback.
Let me know how things look and then we can do a final squash and merge. Also, the build is passing again which I'm assuming was from Jason's CDMS2 fix.

I think this PR is ready. Please feel free to do a squash and merge. :)


get_usage() does not give exact character count

Hi,
If I call translator.get_usage() before and after a translation, I get the same number of character used, an even 1 000 000. Is there a way to get the exact character count?

Hi @jaixan, thanks for creating this issue, we're looking into the problem.

We're having trouble reproducing this issue. Can you please give more information or contact DeepL support?

I’ve played a bit more with the API and I found out the character count increments in steps of 50k. That’s why I couldn’t see the counter increment. Is there a way to have it increment every time a document is translated?

Le 26 sept. 2022 à 07:43, Daniel Jones @.***> a écrit :
We're having trouble reproducing this issue. Can you please give more information or contact DeepL support https://support.deepl.com/hc/en-us/requests/new?
—
Reply to this email directly, view it on GitHub https://github.com/DeepLcom/deepl-python/issues/50#issuecomment-1257904761, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACGX3VJBELPOALQCMM2NDELWAGD5TANCNFSM6AAAAAAQLMHQIA.
You are receiving this because you were mentioned.

Hi @jaixan, I'm assuming you call translate_document() or translate_document_with_filepath(). These functions return a DocumentStatus, you can check the billed_characters property to see how many characters were billed for that document.
Unfortunately the character count returned by get_usage() can be delayed due to technical reasons.


[zh]Resync reference for glossary and sec issues

related: #27673
Work for part of the "Others(1)" items:
1	1	content/en/docs/reference/glossary/index.md
2	3	content/en/docs/reference/glossary/wg.md
1	1	content/en/docs/reference/glossary/cloud-controller-manager.md
1	1	content/en/docs/reference/issues-security/_index.md

This one looks trivial if it contains all the required changes.
/approve

/lgtm


Update README.md

Adding some text around the long tail of self-supervised models as that seems to be a rising problem we should address up front. We can move it to the self-supervised page if we need to.

Sounds good!


Use #9D550F orange for selection - iterm

I tried using it the theme in iterm2 today only to realize I forgot to update the highlight color, based on #5

Thanks!


Transient and scoped services are not disposed when resolved using a compiled call site

This breaks EF DbContext pooling and could lead to a memory leak.

@Eilon @pakrym Heads up: since it blocks a new EF Core feature, we need it looked at for 2.0.


Conditional compilation not works correctly for individual items

My crate browser-app depends on a crate share_lib that supports both wasm32 and native (Linux). In share_lib:
impl SomeStruct {
    // #[cfg()] on an item
    #[cfg(target_arch = "wasm32")]
    pub fn first_method(&self) {    }
}

// #[cfg()] on the whole impl block
#[cfg(target_arch = "wasm32")]
impl SomeStruct {
    pub fn second_method(&self) {    }
}

In browser-app (target=wasm32-unknown-unknown), the second_method is ok, but rust-analyzer complains that no method named first_method found for &share_lib::SomeStruct in the current scope.
I just switch from RLS to rust-analyzer, the whole of my browser-app are mark in red because of this. I try to put .cargo/config (with a [build] that says target="wasm32-unknown-unknown") at the root of browser-app but it has no effect.

Just encountered the same issue.

but rust-analyzer complains that no method named first_method found for &share_lib::SomeStruct in the current scope.

This is error from cargo check, you can set feature for it using rust-analyzer.checkOnSave.extraArgs or rust-analyzer.checkOnSave.overrideCommand.


Add typings to the repo

Do you want to add more comments added? :smile:

Add it to https://github.com/DefinitelyTyped/DefinitelyTyped instead. I'm not interested in maintaining typings for something I don't use. Thanks though :)

No problem. 🌷

And it already exists https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/path-exists

Yes, but having the typings inside the source would help user so that they don't have to manually download and install it (either from DT directly, tsd, typings, or npm install @types/path-exists).
Also, the version I put in here is in external module format. The one in DT is not and need to be converted.
I put it here https://github.com/types/npm-path-exists and register it at https://github.com/typings/registry so that user can install it with typings install path-exists.

by the way, you can see the version in DT is outdated anyway, it's in callback form.


Expand controller information

Use the status API to retrieve controller status information including version numbers, state, etc.

Implemented with version 1.0.7.


Simplify matcher attaching

Marchers are very noisy and clumsy they needs to be simplified.

handled with #86


Add CLI flag for using networking.k8s.io/v1beta1 IngressClass resource

What this PR does / why we need it:
As mentioned in #2563, the Ingress Controller fails when scanning for non-existing resource types. This PR adds a flag to avoid that issue by enabling the user to monitor a more backwards-compatible resource type. The root of the problem can be traced into the fact that networking.k8s.io/v1 was basically hard-coded as the IngressClass namespace, even when it is used with older Ingress resources (from extensions/v1beta1) which are unlikely to exist in a cluster with full networking.k8s.io/v1 support.
Which issue this PR fixes: fixes #2563
Special notes for your reviewer: This PR only addresses the issue for a specific resource type. A more general solution, like not watching non-existent resources can be another option but it would be a more involved effort to implement with a wider surface for possible breakages.
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[X] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

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

Closed in favor of the approach in #2577