JAVA-2918 Ignore invalid peers in schema agreement check

fixes https://datastax-oss.atlassian.net/browse/JAVA-2918

@adutra PTAL when you have a chance.

@olim7t @adutra Please see if you have a chance to review. Jira ticket https://datastax-oss.atlassian.net/browse/JAVA-2918

@annamalai87 I plan to review this week, don't worry. And thanks for the PR :-)

@annamalai87  sorry to bother you but I need you to sign our CLA please:
https://cla.datastax.com/
Please ping me when you've signed it. Many thanks!

Signed.
Thanks,
Annamalai
On Thu, Apr 29, 2021 at 8:47 AM Alexandre Dutra @.***>
wrote:

@annamalai87 https://github.com/annamalai87 sorry to bother you but I
need you to sign our CLA please:
https://cla.datastax.com/
Please ping me when you've signed it. Many thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/datastax/java-driver/pull/1528#issuecomment-829252499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ7PL6BHTZZI76C47ROCHOTTLFPQDANCNFSM4W3UXZ5A
.


fix(RSC): Always read header in RSC render

Should fix https://github.com/amannn/next-intl/discussions/446
As far as I understand the issue currently, something like this happens:

The middleware receives a request where the NEXT_LOCALE cookie is set
During the RSC render, the NEXT_LOCALE cookie is suddenly not available

If this is the case, this would fix the issue.
If however the middlware doesn't run at all for some reason, this will not make a difference. However, since this doesn't seem to have any downsides (possible since newer Next.js versions), this might simplify the code anyway and is a welcome change.

At this moment I get a new error that I don't know if it has to do with nextjs 13.5.4 or with next-intl. The thing is that in all the tutorials that I have seen and even documentation, everywhere in the rootlayout they use the useLocale() hook and no one has a problem, it does give me a problem since it is a hook and the hooks cannot be server side
The error shown is the following:
 ✓ Compiled in 761ms (1242 modules)
 ⨯ node_modules/next-intl/dist/esm/server/RequestLocale.js (1:580) @ eval
 ⨯ Error: Unable to find `next-intl` locale, have you configured the middleware?`
    at RootLayout (./src/app/[locale]/layout.tsx:29:73)
    at async Promise.all (index 0)
null

My configuration is to the letter with the documentation and on par with other programmers, but no one else gets the same error as me, or at least for the same reason.
I will be really thankful if you or anybody can help me with this problem.

I get the same error as well


223 onboard rfdiffusion

RF Diffusion requires: a protein, a motif description, a min length, a max lenght of the construct as well a number of designs that are supposed to be created

Seeing some issues around the parsing of numeric attributes in the config
(base) rindtorff@niklas plex % ./plex -tool tools/inprogress/rfdiffusion.json -input-dir testdata/design
Plex version (v0.5.1) up to date.
BACALHAU_API_HOST not set, using default host
toolPath tools/inprogress/rfdiffusion.json
Running IPWL tool path
Created job directory:  /Users/rindtorff/github/labdao/plex/031a64e4-28ea-4fd4-b9a2-87ceb2f34084
Reading tool config:  tools/inprogress/rfdiffusion.json
Error: failed to unmarshal JSON: json: cannot unmarshal number into Go struct field ToolInput.inputs.default of type string

Just turn all the int values to integer strings. All of these values end up as strings for argument command line interpolation anyways. Go does support multi-type struct values if you want to take the time to have numbers in the json configs.
Eg
      "number_of_designs": {
        "type": "int",
        "default": "10"
      }

good to know - will make the change. We probably should take some time during the retreat to think about configs, types and validation as well


[$1000] [HOLD #11768] on narrow screens most recently visited chat doesn't get marked as unread

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
On a narrow screen:

in account A navigate to a chat with account B
navigate back to home screen
send message from B to A
notice that chat doesn’t get marked as unread

Expected Result:
Chat should get marked as unread
Actual Result:
Chat stays marked as read.
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.2.42-1
Reproducible in staging?: y
Reproducible in production?: y
Slack conversation: https://expensify.slack.com/archives/C04DC6LU2UB/p1671657937698209
View all open jobs on GitHub
The issue has to do with our use of react-freeze which causes the callback to be called with old props. We are deprecating our use of react-freeze as part of the navigation revamp project, so this bug is on hold until then.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c979037bab43aa70
Upwork Job ID:<PHONE_NUMBER>578034176
Last Price Increase: 2022-12-27

Sending this one out to external.

@davidcardoza this is on hold. Not ready to be assigned yet.

Ah my fault. I removed the job posting.

@davidcardoza taking this one over as part of the navigation reboot BZ member assigned.

⚠ Looks like this issue was linked to a Deploy Blocker here
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

This issue has not been updated in over 15 days. @JmillsExpensify, @rushatgabhane, @roryabraham eroding to Monthly issue.
P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

Still on HOLD

Still on hold.


Minor: Add github link to code that was upstreamed

Which issue does this PR close?
Follow on to https://github.com/apache/datafusion/pull/12654
Rationale for this change
@mustafasrepo  ported take_arrays upstream to arrow-rs in https://github.com/apache/arrow-rs/pull/6475
Let's leave a link to that so our future selves know we can remove the copy in DataFusion
What changes are included in this PR?
Add a doc link

Are these changes tested?

Are there any user-facing changes?

Thanks @jonahgao


update image normalization vectors for migraphx worker tests.  Since …

…inputs to normalization were changed from BGR color channel format to RGB, vectors are reordered to match.  Also updates test gold values.
Summary of Changes

Motivation
When I made the last PR to convert images from BGR to RGB I wasn't sure which format the corresponding normalization vectors were written for.  More investigation shows they needed to be reversed.  This restores the label for the bicycle test image to a better result (although the larger Imagenet validation set gives inconclusive results; this change doesn't seem to make much difference in overall accuracy).

Can one of the admins verify this patch?


[main] Upgrade to latest dependencies

Cron -knative-prow-robot
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping github.com/google/go-cmp 8fa37b4...d103655:
  > d103655 Print as text if mostly text (# 258)
  > 9181d1e Avoid diffing by lines if inefficient (# 260)
  > c5c3378 Cleanup edit groups after coalescing (# 259)
  > 1ee4af8 Fix typo in path.go (# 256)
bumping knative.dev/pkg e749d6a...ea766de:
  > ea766de Bump assorted dependencies (# 2130)
  > e2947db update the multiple label selector implemention to avoid clobber in filtered factory (# 2120)

/lgtm
/approve


Add target .NET Core 3.0

Hello!
Consider adding target .NET Core 3.0.
What for?
Starting from this target System.Text.Encoding.CodePages is present in the framework dotnet/corefx#38357
But I didn't find to be added to .NET Standard 2.1 (see here Oops, it's fork)

Hi!
As far as I can tell, it is not necessary to change anything in ExcelDataReader for this.
It just means, if your app targets netcoreapp3.0 or newer, it is no longer necessary to install the Nuget package for System.Text.Encoding.CodePages, since it is now included in the shared frameworks that come with .NET Core 3.0.
Verified in a simple console app. Still need to call System.Text.Encoding.RegisterProvider(System.Text.CodePagesEncodingProvider.Instance);, but it works without the addional dependency.

Thanks for the answer!

Still need to call System.Text.Encoding.RegisterProvider(System.Text.CodePagesEncodingProvider.Instance);

I wish it was spelled out on the library side (as far as I know, this will not affect the library user). I found only the following mention, but this is only for tests
https://github.com/ExcelDataReader/ExcelDataReader/blob/4bbe8e5693beac3d90ca559ea235193bcaa1c8f0/test/ExcelDataReader.Tests/SetUpFixture.cs#L17-L19

Just to be sure, you wish ExcelDataReader would call RegisterProvider itself, so that users of the library don't have to? (as opposed to f.ex being documented better)

As far as I can tell, in the source code of the library there are places where there is a request for encodings that are not supported by Core without additional code.

being documented better

Why to force the user of the library to write additional is considered the best?

new ExcelReaderConfiguration()
{
// Gets or sets the encoding to use when the input XLS lacks a CodePage
// record, or when the input CSV lacks a BOM and does not parse as UTF8.
// Default: cp1252 (XLS BIFF2-5 and CSV only)
FallbackEncoding = Encoding.GetEncoding(1252),

~But I didn't find to be added to .NET Standard 2.1 (see here)~ Oops, it's fork

I tried turning to CodePagesEncodingProvider in .Net Standart 2.1 and got the following
The type or namespace name 'CodePagesEncodingProvider' does not exist in the namespace 'System.Text' (are you missing an assembly reference?)

Why to force the user of the library to write additional is considered the best?

It's not necessarily considered the best, but at the time, it was easier (for me/us) to leave this to users of the library. F.ex it is not clear if it is safe to call RegisterProvider multiple times, and there could be differences how encodings are implemented and registered across the various .NET runtimes. E.g Core 2.x, Core 3.x, Full Framework and Xamarin all have differences how they deal with encodings. Some users might even want to provide their own encoding implementations.
I agree it seems pointless having to write additional code to use the library, but as long as it appears to apply only to some runtimes, I feel it is out of our hands. It may be worth revisiting again and investigating further, but that is for the backlog.

F.ex it is not clear if it is safe to call RegisterProvider multiple times

Do you know how to call unsupported encoding in another way on the library side? :)
Or do you often get encoding in library code? Because by adding CodePagesEncodingProvider.Instance on the library side, the library user will still not be able to get additional encodings.

there could be differences how encodings are implemented and registered across the various .NET runtimes. E.g Core 2.x, Core 3.x, Full Framework and Xamarin all have differences how they deal with encodings

But, I asked only to consider Core 3.0, because, this is the future :)

True, I just started thinking about how it works for Linux.

In general, I thought and decided that the more interesting question is whether to use System.Text.Encoding.CodePages. This has both pros and cons (we can use CodePagesEncodingProvider.Instance.GetEncoding).

In general, I thought and decided that the more interesting question is whether to use System.Text.Encoding.CodePages. This has both pros and cons (we can use CodePagesEncodingProvider.Instance.GetEncoding).


docs: update troubleshooting reference

PR Summary
Small PR - fixes the troubleshooting.md reference in docs/README.md.

 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.

Small fix but an important one! Thanks for opening a PR when you noticed it 👍


Update license with Apache

Added Copyright based on Apache (CELA requested)
Removed unnecessary space.
Removed personal information
Added comment for future code refactoring

@juhlee-microsoft  AFAIK, there is no license Apache per se, maybe you are referring to Apache License, Version 2.0?


NLog does not like some config sections

My config file looks like this:
<?xml version="1.0" encoding="utf-8" ?>
<configuration>
    <startup>
        <supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.5" />
    </startup>
	<configSections>
		<section name="mongoSettings" type="System.Configuration.NameValueSectionHandler" />
	</configSections>
	<appSettings>
		<add key="AWSAccessKey" value=""/><!-- Amazon Credentials -->
		<add key="AWSSecretKey" value=""/><!-- Amazon Credentials -->
	</appSettings>
	<mongoSettings configSource="mongo.config" />
</configuration>

Seemingly because of that, I get this error:

This is incredibly frustrating as it took me about 7 hours to track down why this application was suddenly refusing to start up. All because I have config sections that NLog does not seem to like, but are perfectly valid.

In which file is nlog configurated? Not in this one?
PS: I have removed your credentials from the config.

Whoops, thanks.
NLog is not configured here, it is configured entirely in code, but yet it searches for a config file. I'm more surprised that it's throwing errors for sections in a config file that it probably shouldn't care about.

Well I'm not sure if nlog is throwing the error (it's exception from system.configuration). We have to check that.

It happens on this call:
            var fileTarget                    = new NLog.Targets.FileTarget ();
So I have to assume that's NLog.

Doesn't seem so:
https://github.com/NLog/NLog/search?utf8=✓&q=ConfigurationErrorsException&type=Code

Well we are reading the config,  and probably as first progress,  because we have to search for possible configuration data.

Yeah, and I see that, but I'm not sure why that would crash the program. Especially since the config sections in my config file are 100% valid (according to stackoverflow).

From https://msdn.microsoft.com/en-us/library/aa903350(v=vs.71).aspx

Remarks
If this element is in a configuration file, it must be the first child element of the  element.

Welp, that would very likely explain my issue. As for why NLog seemed to be triggering it - that was the first function call in the app.
¯_(ツ)_/¯ .NET

Thanks @oipo :)


Check for error after running ApplyPlanPatches

The existing code doesn't fail if the plan patch couldn't be found. This PR fixes that.

ridiculous


About cost in FunctionHO.py

Hi good morning,
I wonder what is cost code in here and what does it do?
cost  = alpha * error + beta * (num_feat / max_feat)
And i know that you use error rate as fitness, but why use cost value as fitness instead of error rate?

Hello 😄  , num_feat is the number of features chosen, max_feat is total features dataset has, (num_feat / max_feat) stands for the ratio of the chosen features to total features. Using beta * (num_feat / max_feat), i suppose, is to control the number of features we choose. If the num_feat is too much, then cost would become bigger.

Hi good morning, I wonder what is cost code in here and what does it do?
cost = alpha * error + beta * (num_feat / max_feat)
And i know that you use error rate as fitness, but why use cost value as fitness instead of error rate?

Hi,
You can also use error rate, there is nothing wrong with it.
The cost function is to ensure a lower feature size while reducing the
error rate. (normally we set alpha = 0.99 and beta = 0.01)
Best Regards,
Jingwei Too
On Wed, Apr 20, 2022 at 5:51 PM liansyyy @.***> wrote:

Hello 😄 , num_feat is the number of features chosen, max_feat is total
features dataset has, (num_feat / max_feat) stands for the ratio of the
chosen features to total features. Using beta * (num_feat / max_feat), i
suppose, is to control the number of features we choose. If the num_feat
is too much, then cost would become bigger.
Hi good morning, I wonder what is cost code in here and what does it do?
cost = alpha * error + beta * (num_feat / max_feat)
And i know that you use error rate as fitness, but why use cost value as
fitness instead of error rate?
—
Reply to this email directly, view it on GitHub
https://github.com/JingweiToo/Wrapper-Feature-Selection-Toolbox-Python/issues/17#issuecomment-1103727930,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHKG2GZPLD5UFNUFO35UVFDVF7HTDANCNFSM5RAU7QWQ
.
You are receiving this because you are subscribed to this thread.Message
ID:
<JingweiToo/Wrapper-Feature-Selection-Toolbox-Python/issues/17/1103727930@
github.com>

Thank you 😄

Can we say that cost function is the upgrade version from error rate? because you say that cost function reduce the error rate.


question about vision encoder

Is the vision encoder used here umt-l or internvideo2-1B?
I saw that the mistral version in internvideo2 had similar results to the one here

Hi! We released UMT-L since it runs faster.

Hi! We released UMT-L since it runs faster.
hi! will you release internvideo2 version in the future?

Hi! We released UMT-L since it runs faster.

Hello! Will you release internvideo2 version in there future? thanks~

Hi! We released UMT-L since it runs faster.

Hello! Will you release internvideo2 version in there future? thanks~

@yepzhang  Sure, You can found the internvideo2-chat in huggingface


CPU temp now missing from Local Diagnostics Screen

CPU temp now missing from Local Diagnostics Screen

What fleet is this on @WayneNebra ?
I think this was a temporary issue that has been resolved.
And did it show on RockPi before?

Oh sorry I should of posted more details about fleets etc. Noticed on the SB fleet this morning, funnily enough it's now gone, also on my RockPi and Outdoor that's in "Testnet" how ever just checked the Indoor 915 and 868 RockPi fleet and it's still missing.
I can check some more fleets and report back if you like?

Could you share some links to some devices that have this issue on Balena, from different fleets/variants?

Sure, here we go.
HELIUM-INDOOR-915-rockpi
https://dashboard.balena-cloud.com/devices/e7eda6ff7fd0dd0d65740308e66e7f8c
HELIUM-INDOOR-868-rockpi
https://dashboard.balena-cloud.com/devices/063bbca77864bdd476427a900ba9282b
HELIUM-TESTNET-ROCKPI
https://dashboard.balena-cloud.com/devices/6d181a5a9b5f9fc313f9872e5a39c644

@robputt can you look at this when you can?

@WayneNebra @robputt is this fixed now? Can we close it?

https://psutil.readthedocs.io/en/latest/#psutil.sensors_temperatures
https://github.com/NebraLtd/hm-diag/blob/94b7acd3931ed5ee7535c73c7ea621ee0706050f/hw_diag/utilities/hardware.py#L339-L352

RockPi seems to have soc-thermal for soc and gpu-thermal for gpu

RockPi:
root@nebra-2D845E:/opt# python3
Python 3.11.1 (main, Dec  8 2022, 10:14:58) [GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import psutil
>>> psutil.sensors_temperatures()
{'soc-thermal': [shwtemp(label='', current=43.333, high=0.00010499999999999999, critical=0.00010499999999999999)], 'gpu-thermal': [shwtemp(label='', current=43.888, high=None, critical=None)]}
>>>

Raspi:
root@b9f3779ff641:/opt# python3
Python 3.11.1 (main, Dec  8 2022, 10:14:58) [GCC 10.2.1 20210110] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import psutil
>>> psutil.sensors_temperatures()
{'cpu_thermal': [shwtemp(label='', current=50.634, high=None, critical=None)]}
>>>


Convert a bunch of small files to JS

Continuing on from https://github.com/atom/atom/pull/13065, this PR rewrites a bunch of small files as JS. 🤘
👀 @atom/feedback

Huh, looks like atomdoc is blowing up on some unknown docstring 😞  – I'll take a look into adding more debug info to the build script to make it easier to find which file has invalid Atomdoc going on 🕵

Great, thanks. https://goo.gl/JBljdR

Sat on this one a bit too long, and I'm preeeetty sure at least some of these files have seen some edits since I opened this PR. Gonna do a new one with fewer files.


I wonder why crop HR image too?

Hi, Thank you for your work and sharing code.
I would like ask to about multiscalesrdata.py code.
I don't understand why you crop HR images in line 157.
Can't we just crop the low resolution image for degradation learning?
Thank you!

Hi @choieq, thanks for your interest in our work.
In our implementation, we generate LR images online in the dataloader using random degradations (see here). So we crop HR image patches in multiscalesrdata.py. Of course we can produce LR images in advance and just  crop LR images for degradation learning. However, this will take up lots of space on the hard device.


depth training problems

When training custom data, the output depth value is strange, and the difference between min and max is very small.

When training custom data, the output depth value is strange, and the difference between min and max is very small.

Is this normal, or what parameters need to be adjusted?


Added timeout option

description

If the kubejob runs for more than 4 hours, it will end without an error. However, there is currently no option for a timeout value. Added option to set timeout value.

issue

https://github.com/goccy/kubejob/issues/40

Thank you for your contribution ! LGTM


Pick timer countdown sound keeps playing after draft is stopped

When you stop a draft, the countdown warning that you are running low on time (last 10s) keeps playing as if the draft was still ongoing. Looks like the logic keeps running in the background?!

Good catch!
Should be fixed with 7348a86d75fe405a758d1bf0d3454d1943e2e351


Change the item icon for Cat/kittens

Similar to herb seeds that have "irit" and "cada" on the photo to help distinguish them apart can we have cats and kittens with a "k" or a "c" or similar in the item icon. The "your kitten has grown into a cat" message sometimes gets lost or just doesnt show. And it is impossible to look at the photo thru the /gear view and see if it has grown or not.
Thanks 🙏

Kittens have small not as bold eyes, as where grown cats have bold dots / slashes for eyes.
kitten:

cat:


feat(contractStorage): introduce rocksdb additionally to sqlite

Closes #17
Introduce rocksdb additionally to sqlite and make it default
opened questions:

should we generalize a solution(create e.g. generic SqlDbError -> StorageDbError,  SQLiteContractHandler -> StorageContractHandler entities and have a specific rocksdb/sqlite "impls" for that? I played with a bit with such generelization and found too many generics there. I'm not sure that we need to do that(maintainability, extensebility, compile times, sqlite probably will be remove and rocksdb will be a single store)?

Here some compile timings:
without statically linked rocksdb; compiled with all features flag
Finished dev [unoptimized + debuginfo] target(s) in 11m 41s
statically linked rocksdb; locutus compiled with default features only
Finished dev [unoptimized + debuginfo] target(s) in 5m 47s
statically linked rocksdb; locutus compiled with all features
Finished dev [unoptimized + debuginfo] target(s) in 6m 47s

should we generalize a solution(create e.g. generic SqlDbError -> StorageDbError, SQLiteContractHandler -> StorageContractHandler entities and have a specific rocksdb/sqlite "impls" for that? I played with a bit with such generelization and found too many generics there. I'm not sure that we need to do that(maintainability, extensebility, compile times, sqlite probably will be remove and rocksdb will be a single store)?

I wanted to get rid of the generic error type and just use a boxed error instead in all the node where a generic was being used before, but haven't got to do it. Is annoying deal with it and totally not worth it, is pretty much a left over of an older version. If you want to tackle this  PR will be welcome.
From memory but the error will have to be probably bounded (impl serialize, deserialize, send, sync etc.) but in practice this should not be a problem.
I will take a look into the PR later next week hopefully. Some open questions regarding building/linking, some of which we may be able to address now:

Should we default to static linking the rocksdb deps? Imo, if possible yes. If it fails fall back to pulling anything from the network. We will need to review how is working re. sqlite and follow a similar strategy.
For platforms where rocksdb may be problematic (if this is a real problem at all), we should fallback to using sqlite instead (sqlite platform support is excellent). We will have to make sure of this when we prepare release artifacts for the different platforms we want to support.
At some point we probably want to look into some performance tests, but not necessary now.

All this will probably complicate the build process/compile config etc. unfortunately, but most likely necessary to do.

I've played with linking of rocksdb a bit. I have several options to provide and I'm not sure which one we should pick up.
So I suggest to provide that as a separate PR if you wouldn't mind(so that it will be easier to test, revert, refer).

Sure, no problem.

@kakoc apparently there are some conflicts, can you solve them and then we can merge the PR as it is.
Will will deal with the linking issues in an other PR/issue and simplifying the error generics.

@iduartgomez, rebased


animation callbacks

Hello, it would be nice to have a callback blocs for animation did start and animation did end.
What do you think? I have to address this scenario:
I've got the switches tableview's rows, and when the switch changes from off to on, I have to reorder the table, but the animation during the reording is not fine.

Hi @Progeny can you attach a video for this scenario, or some screenshots, I might have a better solution for it. Thanks

I also have this issue as well. Now if I save the position, the animation doesn't even work.

Thanks! Having a second look now, I can understand this issue. I'll add some callback for animation did end.

Added animation callback closures in 1.0.5 👍
Let me know if it works for you guys


Update VMs for December 2022

https://dev.azure.com/vcpkg/public/_build/results?buildId=82524&view=results

We are fixing those CI issues.


[EngSys] upgrade dev dependency eslint version to v9

Some generated packages still have v8. This PR upgrades them to v9. A separate
PR is submitted to update the code generator.

API change check
API changes are not detected in this pull request.


refactor(toml): Clean up workspace inheritance

What does this PR try to resolve?
The goal is to simplify the code so we have a better boundary between toml/mod.rs and toml/schema.rs for when we break toml/schema.rs into a separate package for #12801.
This let us

Remove a trait used in some back and forth for error handling
Move a lot of the inheritable bookkeeping and logic out of schema.rs

A lot of these changes were inspired by cargo_toml.  This included some renaming that I felt made the code clearer.
I didn't go as far as cargo_toml, yet.

They derive more Deserializers, producing worse errors
Their equivalent of InheritableField has an inherit function on it.  They eagerly inherit things and then error if anything isn't inherited

I'm still toying with something like this but held off for now
One idea is InheritableField has an inherit_with function, like today, that only passes errors up but doesn't generate an error.  We then have a get function that errors if it isn't inherited.  We could encode the field names, for error reporting, into a type parameter for InheritableField

They flatten InheritableDependency into TomlDependency

By splitting these up, workspace.dependencies and .cargo/config.toml code can directly use TomlDependency without extra error handling
-If we went this rout, I think I'd mergeInheritableDependency::Inherit with DetailedDependency, having callers handle the errors (much like TomlManifest is both a real and virtual)

Some things I'm trying to balance

Complexity
Quality of error messages
Knowing what code needs touching when changes are made

Some more work is needed here

How should we test and review this PR?
Additional information

This seems pretty neat!
I'd like to hand this to @Muscraft to see if they have any feedback.
r? Mustcraft

r? Muscraft

@bors r+

:pushpin: Commit 46d9894014fc769a9247ecfdb0f244464ba7df60 has been approved by Muscraft
It is now in the queue for this repository.

:hourglass: Testing commit 46d9894014fc769a9247ecfdb0f244464ba7df60 with merge 9765a449d9b7341c2b49b88da41c2268ea599720...

:sunny: Test successful - checks-actions
Approved by: Muscraft
Pushing 9765a449d9b7341c2b49b88da41c2268ea599720 to master...


1.2 post execution testing interface change tests

Motivation

Solution

Codecov Report
Base: 99.73% // Head: 99.73% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (897ed42) compared to base (c5e9cd9).
Patch has no changes to coverable lines.

Additional details and impacted files
@@                             Coverage Diff                              @@
##           1.2-post-execution-testing-interface-change     #710   +/-   ##
============================================================================
  Coverage                                        99.73%   99.73%
============================================================================
  Files                                               20       20
  Lines                                             1113     1122    +9
  Branches                                           185      185
============================================================================
+ Hits                                              1110     1119    +9
  Misses                                               2        2
  Partials                                             1        1

Flag
Coverage Δ

reference
99.73% <ø> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

reference/lib/ReferenceOrderCombiner.sol
100.00% <0.00%> (ø)

reference/lib/ReferenceOrderFulfiller.sol
100.00% <0.00%> (ø)

reference/lib/ReferenceZoneInteraction.sol
100.00% <0.00%> (ø)

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

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


Process never get finished

npm --add-python-to-path='true' --debug install --global windows-build-tools
i am trying to install the windows build tool but not getting succeed. Can someone please help on this

Same issue here.
Here are a couple of screenshots.

Thank you for the report! This is now fixed. It's a bit scary that the Visual Studio Build Tools installer doesn't exit on its own, but we at least exit the npm install process when we detected that we're all done.


figure wishlist: basic (nohist) model

[ ] History-dependent chronometric and psychometric functions (already in https://github.com/anne-urai/mouse_history_ddm/tree/main/figures)
[ ] As on CCN poster: model comparison between basic (non-history dependent) models e.g. vanilla DDM, collapsing bounds
[ ] Posterior predictions, model verification etc (think of which plots are the standards)


Allow specifying model in env file

Issue description
New users don't know which model the bee-stack is using. Ideally they should have the flexibility to specify which model to use as well as modify parameters.
Ideal solution

Include model param in env file
Document which model(s) and settings the bee-stack is optimized for and include a warning that switching models may deteriorate experience.

Related discussion: https://github.com/i-am-bee/internal/issues/2

@jezekra1 In the short-term, can we add this as a guide to the bee-stack readme? Running the stack with a different model is a common request. Ideally, there should be a quick way to toggle between the models we optimized for at the framework level (llama and granite) while setting up the stack. There should be also a guide for advanced users to run the stack against models we didn't optimize against.

Already there

In the short-term, can we add this as a guide to the bee-stack readme?
I added the curl command to the advanced section, but I don't want to include the list of available model, as it changes often in the framework and it would get out of sync quickly.

It depends on the presets available in each adapter, for example for watsonx:
https://github.com/i-am-bee/bee-agent-framework/blob/main/src/adapters/watsonx/chatPreset.ts
Should I add this disclaimer to the README?

Ideally, there should be a quick way to toggle between the models we optimized for at the framework level (llama and granite) while setting up the stack.
We can add this setting to API and then to the startup script, here is a PR: https://github.com/i-am-bee/bee-api/issues/76

I don't want to include the list of available model, as it changes often in the framework and it would get out of sync quickly

I am wondering if there is a creative solution here. At the very min, we should link to a section in the bee-framework where we keep an up-to-date list of models we are optimized against (cc: @Tomas2D ). Could we maybe have an agent that listens to changes to that section and updates the bee-stack readme as well?

Discussion: should model switching be done directly in the ui - as opposed to while configuring the bee-stack?


BUILD_SHARED_LIBS as ENABLE_SHA... default

If defined, use BUILD_SHARED_LIBS to populate the default value of
ENABLE_SHARED_LIBS
Fixes LLNL/blt#99
Shamelessly stole solution from @cyrush1

@zbeekman looks good!
Interestingly, when I setup appveyor for windows testing, I mistakenly already used  BUILD_SHARED_LIBS.
Which means we were not testing the shared libs case, and of course it failed.
We will need to resolve, not sure when i will have time to take a pass.

I rebased on latest master, and did some brief hunting for the issue on appveyor. It looks like the visual studio project is trying to link example.exe against libexample.lib instead of libexample.dll, which is the name of the library that will be built if dynamic libraries are enabled. It seems odd to me that this is happening at all, unless the logic to deposit the dll in the right directory has something broken in it. Otherwise, I would guess that CMake should be able to resolve the name of the library created, since target_link_libraries is just passed a target name. I looked through the logic and didn't see any place where the custom dependency resolution logic or target creation logic would be renaming a .dll to a .lib.
TL;DR; I'm a windows newb, and I don't see anything obviously wrong in BLT so @cyrush or someone else my proficient with Windows will likely have to debug this.

This sounds familiar. I think this is a simple issue, but painful to debug in appveyor.
I'll try to carve out some time to investigate on a windows machine.

Resolved the merge conflicts. Still somewhat clueless about fixing the appveyor failures

going to try to poke at it on my windows machine soon, sorry for the delay

No worries, just wanted to make your job merging this as easy as I can... If it doesn't get merged over the next few weeks, my windows + CMake proficiency may level up to the point where I can take care of this myself 😝

@zbeekman  found the issue, a classic windows shared libs issue.
We need to export the symbols in the example lib or else the .lib file wont generate, and we can't link (even though the dll exists). I'll work up a solution soon.

this is still a work in progress, I need to iterate on an actual windows machine vs fighting appveyor.

Alright, got things working!
I tried to make minimal changes to the tutorial source, since it's pulled into the docs (in some cases by line number) Some line numbers did shift,  I updated the docs and checked -- I think we are ok.
Not super happy with the use of the windows define to select the proper export type on windows.
More complex projects would be better served using cmake to configure (generate) a header tied to static vs shared on widows, but for our testing projects and the tutorial that seems like too much to do.
We are ready to merge -- thanks @zbeekman for the help with this!

Thanks @cyrush.
Since the exports header files are boilerplate, somewhat intricate, and necessary to get shared libs working, it seems like a great feature for BLT to take care for us.  E.g. with a simple macro/function that generates the file for us with a specified prefix. That way all blt-based projects can benefit from this.
If it is one config file per-library, perhaps it can be incorporated directly into blt_add_library ?

I have thought about automating it several times.
I am in favor of a helper function that can generate a boiler plate header for you that you still explicitly include.
I don't think it should be directly in blt_add_library:
Why?

While its good form to do it on Linux (and OSX), its not required. Many folks may not care about export headers at all
Folks may not like my style of export headers :-). They many want to use their own pragmas or settings related to exports. While my style has been tested many places, its not a complete solution for all compilers.
In some cases I share export headers across libraries, this requires setting the proper defines (vs the ones that CMake automatically creates for you)
The current windows solution for our tests w/ defines ugly, but thats really my opinion. I would suggest not using this solution, but some folks may want to use defines.


Update default board model.

Arturo changed the default board model from Uno to Mega2560. However, ano list-models incorrectly reports that Arduino Uno is still the default.

In the 2.0 version I'm using the same file the IDE uses for the "current board" which is neat since you can change it in the IDE menu and ano will pick that up and use it the next time you build.

Awesome! Looking forward.


update MARS assets on osmosis

We have migrated the MARS token from the mars-1 network to become a native factory token on neutron-1. This PR updates the Osmosis token list to reflect these changes.

The old token logo for MARS.old is now greyed out
The new ibc denom is updated along with the originDenom to reflect the new token details

@stamper-dev
This is important PR.
Please check carefully.

We have tested the token ibc/B67DF59507B3755EEDE0866C449445BD54B4DA82CCEBA89D775E53DC35664255 and was able to confirmed that it was not transferred from Mars Chain to the Osmosis chain as mentioned.
Instead, we observed that token(factory/neutron1ndu2wvkrxtane8se2tr48gv7nsm46y5gcqjhux/MARS) was first created at the Neutron and transferred to the Osmosis chain. So, with that information, we have edited the information based on the observation


feat(#137): mcw step

In this pull I've implemented mcw step for collecting most common words in each README.
closes #137
History:

feat(#137): to_rast -> mcw
feat(#137): rtext
feat(#137): words
feat(#137): markdownit package, puzzles
feat(#137): transforms readme
feat(#137): most common words test case
feat(#137): in collect.yml

PR-Codex overview
This PR focuses on enhancing the functionality for collecting and processing the most common words from README files. It introduces new tasks, modifies workflows, and adds tests to ensure the correctness of the new features.
Detailed summary

Updated extend-exclude in _typos.toml to include Markdown files.
Modified .github/workflows/markdown-lint.yml to ignore specific directories.
Added a new task mcw in justfile for collecting most common words.
Updated pyproject.toml to include markdown-it-py dependency.
Introduced new mcw task in the Poetry configuration.
Changed workflow in .github/workflows/collect.yml to process most common words.
Updated logging in sr-data/src/sr_data/steps/extract.py for clarity.
Created tests in sr-data/src/tests/test_mcw.py for mcw functionality.
Added example README content in sr-data/src/tests/to-words.md and sr-data/src/tests/to-mcw.csv.
Implemented the to_words, remove_stop_words, lemmatize, and most_common functions in sr-data/src/sr_data/steps/mcw.py.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here.

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 9min).

@rultor release, tag is 0.0.3

@rultor release, tag is 0.0.3

@h1alexbel OK, I will release it now. Please check the progress here.

@rultor release, tag is 0.0.3

@h1alexbel Done! FYI, the full log is here (took me 15min).


'keyof typeof' js-object to zod-schema

Hi,
we are slowly transitioning from a JavaScript code base to TypeScript. Due to the quantity of code and our inexperience with TypeScript, we have to live with both worlds for the foreseeable future.
We used simple js Objects as pseudo enums in the past:
within a js file:
const pseudoEnum = {
  ELECTRIC: {someProp: true, anotherProp: "xxx"},
  PNEUMATIC: {someProp: false, anotherProp: "yyy"},
  HYDRAULIC: {someProp: true, anotherProp: "zzz"},
}

This allows us to type stuff like this:
within a ts file:
type Application: keyof typeof pseudoEnum

Which nicely restricts possible values for elements of type Application:

How can I mimic this behavior in a zod schema?
I spent quite a while playing with z.enum and z.nativeEnum, but I don't get it to resolve properly.
I'd prefer not to rewrite our pseudoEnums since so much of our js code base relies on it...
Thanks in advance
Nice regards
Christian

My apologies if you think that this does not belonging here. Feel free to close it. :)

Since z.enum takes in an array containing the values, you can provide an array with those values.
Two ways immediately come to mind:

Object.keys(pseudoEnum) but then you gotta do some type casting to make it the correct tuple type
Directly create the tuple yourself.

For Object.keys you can do as below
const applications = Object.keys(pseudoEnum) as ["ELECTRIC" , "PNEUMATIC" , "HYDRAULIC"];

This could then be feed into zod z.enum(applications)
A far better approach — in my opinion at least — would be to create the tuple yourself
// Typescript create typing of: const applications: readonly ["ELECTRIC", "PNEUMATIC", "HYDRAULIC"]
const applications = ["ELECTRIC" , "PNEUMATIC" , "HYDRAULIC"] as const;

It will be already be a tuple type const applications: readonly ["ELECTRIC", "PNEUMATIC", "HYDRAULIC"]
You can then base the Application off the new applications tuple
// type Application = "ELECTRIC" | "PNEUMATIC" | "HYDRAULIC"
type Application = (typeof applications)[number];

You can then add a typing to pseudoEnums
const pseudoEnums: Record<Application, {someProp: boolean, anotherProp: string}>  = {
  ELECTRIC: {someProp: true, anotherProp: "xxx"},
  PNEUMATIC: {someProp: false, anotherProp: "yyy"},
  HYDRAULIC: {someProp: true, anotherProp: "zzz"}
};

If you add or remove an entry, it will generate an error.
const pseudoEnums: Record<Application, {someProp: boolean, anotherProp: string}>  = {
  ELECTRIC: {someProp: true, anotherProp: "xxx"},
  PNEUMATIC: {someProp: false, anotherProp: "yyy"},
  HYDRAULIC: {someProp: true, anotherProp: "zzz"},
  NEW_APP: {someProp: true, anotherProp: "zzz"} // Object literal may only specify known properties, and 'NEW_APP' does not exist in type 'Record<"ELECTRIC" | "PNEUMATIC" | "HYDRAULIC", { someProp: boolean; anotherProp: string; }>'.ts(2322)
};

You can then feed this into zod z.enum(applications)

Hello William,
thanks for your reply! I just wanted to let you know, that I read your message!
For the last couple of days I tried to wrap my head around it. I don't get it. The problem probably roots in my insufficient knowledge of typescript and I don't want to misuse the issues section as stackoverflow.
I need some more time to play with your approach and learn how ts in general handles pseudoenums. I promise to get back to you! :)

You can do it like so:
const schema = z.object({
  ELECTRIC: z.object({ someProp: boolean, anotherProp: string }),
  PNEUMATIC: z.object({ someProp: boolean, anotherProp: string }),
  HYDRAULIC: z.object({someProp: true, anotherProp: "zzz"}),
})

type Keys = keyof z.infer<typeof schema> // "ELECTRIC" | "PNEUMATIC" | "HYDRAULIC"

const schema = z.object({
  ELECTRIC: z.object({ someProp: z.boolean(), anotherProp: z.string() }),
  PNEUMATIC: z.object({ someProp: z.boolean(), anotherProp: z.string() }),
  HYDRAULIC: z.object({ someProp: z.boolean(), anotherProp: z.string() }),
});

type Keys = keyof z.infer<typeof schema>; // "ELECTRIC" | "PNEUMATIC" | "HYDRAULIC"
type Keys2 = keyof z.TypeOf<typeof schema>; // "ELECTRIC" | "PNEUMATIC" | "HYDRAULIC"
type Keys3 = keyof typeof schema.shape; // "ELECTRIC" | "PNEUMATIC" | "HYDRAULIC"


Extension Does not Build for DuckDB v0.8.1

When I create an extension repo, and try to build it using DuckDB v0.8.1 (latest release as of this writing), the extension does not get built. I attempt to execute the quack() scalar function and it isn't found. Additionally, the extension doesn't exist at build/release/extension/quack/quack.duckdb_extension. Though the directory build/release/extension/quack/ does.
Steps to reproduce:

Create a new repo from this template.
Update the DuckDB git submodule to point to the v0.8.1 tag.
run make
Run ./build/release/duckdb
Run select quack('Jane') as result;

Expected result:
The query result documented in the README:
D select quack('Jane') as result;
┌───────────────┐
│    result     │
│    varchar    │
├───────────────┤
│ Quack Jane 🐥 │
└───────────────┘

Actual Result:
D select quack('Jane') as result;
Error: Catalog Error: Scalar Function with name quack does not exist!
Did you mean "acos"?
LINE 1: select quack('Jane') as result;
               ^
D load quack;
Error: IO Error: Extension "/home/mark/.duckdb/extensions/v0.8.1/linux_amd64/quack.duckdb_extension" not found.

Candidate extensions: "icu"

edit: formatting

I was able to resolve this issue by reverting changes to the Makefile I found here. When using the old values for EXTENSION_FLAGS, the extension was built as expected.

DuckDB v0.9.0 was release and the template now targets that. I'll close this issue!


Feat/attachments diff

So this is the diff part of incorporating the attachments into the pdf/diff system.
It build upon how the pdf is created, however it gets a lot more complicated with matching attachments together.
I think this solution is ok ... but it is very mad scientist-y ... But it was quite a logical challenge to match attachments to a study.
This PR also assumes some knowledge of how Diffsections work (which is also quite mad scientist-y. A basic rundown:
DiffSections receive two sections, the first for the old_p, the second for the new_p. If either of these objects is missing, it gets replaced with None.
Good luck with this review as well!

Recursive slot matching algorithm. Less concise than a for loop, but I liked making it. No need to use this, just wanted to share

def get_order(slot):
    if type(slot.attached_object) is Proposal:
        return 0
    return slot.attached_object.order

def insert_into_matches(old_slot, new_slot, matches):
    if new_slot:
        order = get_order(new_slot)
    else:
        order = get_order(old_slot)
    if order not in matches.keys():
        matches[order] = []
    matches[order].append(
        (old_slot, new_slot),
    )

def match_slot(slot, slots):
    if slots == []:
        # If we've run out of slots, no match was found
        return False
    potential_match = slots[0]
    if slot.attachment is potential_match.attachment:
        return potential_match
    if slot.attachment.parent is potential_match.attachment:
        return potential_match
    # Continue with the next potential match
    return match_slot(
        slot, slots[1:]
    )

def get_matches_from_slots(old_slots, new_slots, matches=dict()):
    """
    Tail recursive function that returns a dictionary of integers to tuples
    of (old, new) attachment slot sets, either of which may be None, but not
    both.
    """
    if new_slots == []:
        # If we've run out of new slots, start going through the yet
        # unmatched old slots.
        if old_slots == []:
            # If not old slots remain, return the matches
            return matches
        unmatched_old_slot = old_slots[0]
        # We've already run out of new slots, so these must be
        # unmatched slots and we can just insert them.
        insert_into_matches(
            unmatched_old_slot, None, matches,
        )
        # Continue until old slots run out
        return get_matches_from_slots(
            old_slots[1:], new_slots, matches=matches,
        )
    current_slot = new_slots[0]lots)
    if match:
        # If we have a match, insert it and remove the matched
        # old slot from the pool
        insert_into_matches(
            match, current_slot,
        )
        old_slots.remove(match)
    # Continue with the next new slot
    return get_matches_from_slots(
        old_slots, new_slots[1:], matches=matches,
    )


No way to unlink a Cluster from a Cluster Template

Is your feature request related to a problem? Please describe.
Once a cluster is created from a template, there is now way to unlink it from said template. Please asd a way to unlink a cluster from cluster templates
Describe the solution you'd like
When a cluster is linked to a template there is no way to undo it appart from deleting the cluster and recreating it, which is not an acceptable solution,  even more with a Production envrionment. Please add a way to unlink the cluster from the template.
Describe alternatives you've considered
Maybe there is an alternative way doing it via kubectl on the Rancher cluster

This would be a great feature to have

Relevant

Hello,
I still need a solution for this, even if not integrated within Rancher.
Regards,


Test cEUR on mainnet after CR3 and before Oracle Activation

To test:

Should be able to move cEUR
2.Should impossible to mint/burn cEUR
pre-mint addresses should have the balances right balance

Tested!
Exchanges don't work as expected:
buy failed:Error: Error: Your request got reverted with the following reason string: SafeMath: subtraction overflow
increase allowance
exchanging...
sell failed:Error: Transaction has been reverted by the EVM:
{
  "blockHash": "0x635ce867c616a98d3c920d969f69374826bd1df4d8fb8fefed68a62a7892ffbc",
  "blockNumber": 4025902,
  "contractAddress": null,
  "cumulativeGasUsed": 134010,
  "from": "0x489324b266dfb125cc791b91bc68f307ce3f6691",
  "gasUsed": 134010,
  "logsBloom": "0x00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000",
  "status": false,
  "to": "0x997b494f17d3c49e66fafb50f37a972d8db9325b",
  "transactionHash": "0xf6b431742ee783e199afe2edf326963d55e909c33e6659824b07a9d6d950a76c",
  "transactionIndex": 0,
  "events": {}
}

And transfers do work: https://alfajores-blockscout.celo-testnet.org/tx/0x2d21abd40d33602925bbd8da82d78c83a2842a120f6dbb443ed1b8a1c6235afb/token_transfers


Make AbstractFunctionalSuite correctly work with all the browsers.

raised issue from #302.
A test like WelcomePageSuite.scala uses AbstractFunctionalSuite.scala and it's running actual web-browser (like chrome, firefox, etc) for test by using selenium.
The problem is, tests are failing depends on installed browser.
For example, on OSX, test passes when system has Firefox. fails when the system has Safari only.

Selenium + PhantomJS ;)


The "PLACEHOLDER_DO_NOT_REMOVE" string is not being replaced in the generated server file.

Hello,
I'm facing an issue with the implementation of tRPC with NestJS.
I've followed the setup in docs and it generates a server.ts file as expected, but when referencing a test endpoint inside a .router.ts file, the content of the Query in server.ts is not being updated accordingly.
Am I misconfiguring something?
Here are the files:
@generated/server.ts
import { initTRPC } from "@trpc/server";
import { z } from "zod";

const t = initTRPC.create();
const publicProcedure = t.procedure;

const appRouter = t.router({
  test: t.router({
    getHello: publicProcedure.input(z.object({
      name: z.string()
    })).output(z.object({
      name: z.string()
    })).query(async () => "PLACEHOLDER_DO_NOT_REMOVE" as any)
  })
});
export type AppRouter = typeof appRouter;

app.router.ts
import { Get, Req } from '@nestjs/common';
import { AppService } from './app.service';
import { AuthGuard } from './auth/auth.guard';
import { Request } from 'express';
import { Input, Middlewares, Query, Router } from 'nestjs-trpc';
import { z } from 'zod';

@Router({ alias: 'test' })
export class AppRouter {
  constructor(private readonly appService: AppService) {}

  @Query({
    input: z.object({
      name: z.string()
    }),
    output: z.object({
      name: z.string()
    })
  })
  async getHello(@Input('name') name: string) {
    return {
      name
    }
  }
}

app.module.ts
import { Module } from '@nestjs/common';
import { TRPCModule } from 'nestjs-trpc';
import { AppRouter } from './app.router';
import { AppService } from './app.service';
import { EnvModule } from './env/env.module';
import { AuthModule } from './auth/auth.module';
import { UsersModule } from './users/users.module';
import { DrizzleModule } from './drizzle/drizzle.module';
import { UploadModule } from './upload/upload.module';
import { SongsModule } from './songs/songs.module';
import { RedisClientModule } from './redis-client/redis-client.module';
import { RoomsModule } from './rooms/rooms.module';
import { AppContext } from './app.context';

@Module({
  imports: [
    TRPCModule.forRoot({
      autoSchemaFile: './src/@generated',
      context: AppContext
    }),
    EnvModule,
    AuthModule,
    UsersModule,
    DrizzleModule,
    UploadModule,
    SongsModule,
    RedisClientModule,
    RoomsModule,
  ],
  providers: [AppService, AppRouter, AppContext],
})
export class AppModule {}

Thank you!

Hey @BenjaminSpoiden, Thanks for opening a new issue, I will try to clarify.
@generated/server.ts is ment for type inference only during compile-time, it's there to get the trpc dx with typescript support and you shouldn't expect logic to be there, all of the actual logic is done by the adapter when it applies your actual router methods to the trpc sdk during run-time.
Since the client-side trpc does not need the actual implementation details of the when referencing the router types, having placeholders is enough to trick it to reference the actual endpoints during run-time and still provide the great e2e dx type-safety experience.
If you try to actually reach that endpoint, it should work, and you are welcome to open a PR that will update the generated file to include an explanation about the placeholders there for future developers.
Does this make sense?

Thanks for your quick response!
I'm not too sure, for example when I try to test an endpoint out inside a next.js project, I get this error:
Server  Error: No "query"-procedure on path "test.getHello"
    at TRPCClientError.from (TRPCClientError-38f9a32a.mjs:27:20)
    at <anonymous> (httpBatchLink-d0f9eac9.mjs:189:60)
    at resolveErrorDev (react-server-dom-turbopack-client.browser.development.js:1785:63)
    at processFullStringRow (react-server-dom-turbopack-client.browser.development.js:2062:17)
    at processFullBinaryRow (react-server-dom-turbopack-client.browser.development.js:2050:7)
    at progress (react-server-dom-turbopack-client.browser.development.js:2253:17)

Here's my trpc.ts file:
import { createTRPCProxyClient, httpBatchLink } from "@trpc/client";
import { AppRouter } from "@server/@generated/server";

export const trpc = createTRPCProxyClient<AppRouter>({
  links: [
    httpBatchLink({
      url: `${process.env.NEXT_PUBLIC_NESTJS_SERVER}/trpc`,
    }),
  ],
});

and the use case:
import { UserButton } from "@clerk/nextjs";
import { SocketTest } from "@frontend/components/Socket";
import { trpc } from "./trpc";

export default async function Home() {
  const test = await trpc.test.getHello.query({ name: 'test' })

  console.log('test', test)
  return (

      <UserButton />
      <SocketTest />

  );
}

For a bit of context, this a monorepo using pnpm with Next.js 15 on the frontend side.

That should have worked, I am not not seeing anything out of the ordinary with the code you shared.
Have you tried reaching the trpc endpoints via Postman? they should be available at GET http://localhost:{NestJS-Port}/trpc/test.getHello.

Oh yeah Its working fine with Postman, I can access the endpoint. Maybe it has something to do with the last version of Next.js?

Maybe, if you are able to hit the endpoint from Postman it seems that it is a problem with the consumption of the API in the client.
For troubleshooting, I'd recommend you either check if theres a version mismatch between your trpc installation in Nestjs and Nextjs, or check the Next.js example in the repo for implementation details, while it is in Next.js 13, I don't think there were breaking changes.

Thanks for your quick response!
I'm not too sure, for example when I try to test an endpoint out inside a next.js project, I get this error:
Server  Error: No "query"-procedure on path "test.getHello"
    at TRPCClientError.from (TRPCClientError-38f9a32a.mjs:27:20)
    at <anonymous> (httpBatchLink-d0f9eac9.mjs:189:60)
    at resolveErrorDev (react-server-dom-turbopack-client.browser.development.js:1785:63)
    at processFullStringRow (react-server-dom-turbopack-client.browser.development.js:2062:17)
    at processFullBinaryRow (react-server-dom-turbopack-client.browser.development.js:2050:7)
    at progress (react-server-dom-turbopack-client.browser.development.js:2253:17)

Here's my trpc.ts file:
import { createTRPCProxyClient, httpBatchLink } from "@trpc/client";
import { AppRouter } from "@server/@generated/server";

export const trpc = createTRPCProxyClient<AppRouter>({
  links: [
    httpBatchLink({
      url: `${process.env.NEXT_PUBLIC_NESTJS_SERVER}/trpc`,
    }),
  ],
});

and the use case:
import { UserButton } from "@clerk/nextjs";
import { SocketTest } from "@frontend/components/Socket";
import { trpc } from "./trpc";

export default async function Home() {
  const test = await trpc.test.getHello.query({ name: 'test' })

  console.log('test', test)
  return (

      <UserButton />
      <SocketTest />

  );
}

For a bit of context, this a monorepo using pnpm with Next.js 15 on the frontend side.

You connect to the ap router as you would to a page router


Lora issue if lora doesnt exist and tries to use the lora the whole colab stops

Lora issue if lora doesnt exist and tries to use the lora the whole colab stops, no error message, nothing from the colab, or the gradio, it just ceases to work

fixed, use the latest notebook


Looses draft when Today pane is closed for a little bit

If I enter some draft text in the widget, close the Today pane (for instance to copy some text to paste in), and then come back to the Today pane the text will be lost. Anecdotally I need to wait longer than a couple seconds for this to happen, but much less than a minute.

Thanks - I will fix this in the next build


Handle left, right, up, down swipe evenrs

make a new class and override the methods onSwipeLeft(), onSwipeRight(), onSwipeTop(), onSwipeBottom(),

I have combined the code but when we change the image on left or right swipe, we get a misplaced and zoomed image.


moveSelectionToEnd glitches the editor when focus was previously moved to the end

Do you want to request a feature or report a bug?
bug
What is the current behavior?

Create an empty EditorState
moveFocusToEnd
Render the editor state
moveSelectionToEnd and update the editor state
Render the editor state
💥 The editor glitches:

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
Demo code: https://codesandbox.io/s/draftjs-selection-bug-g9xgi?file=/src/index.js:588-606
Demo: https://g9xgi.csb.app/
What is the expected behavior?
moveSelectionToEnd should preserve the focus and not glitch the editor.
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
The demo runs on 0.11.7 but I was also able to reproduce it on 0.10.5.

hiii assign me this issue i want to work on this .

hiii assign me this issue i want to work on this .


KUDU-3007. Support building Kudu on aarch64 platform

This change try to modify and make Kudu can build sucessfully on both
x86_64 and aarch64 platform.


Split adushell consts to util library

This will allow for multiple handlers to be included at the same time

Attempting to port these changes to develop in another branch. Will likely close this one once that is done.

Ported in PR https://github.com/Azure/iot-hub-device-update/pull/403


Localization metadata updates

Docs Build status updates of commit 7255b3e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide/console/error-and-status-codes.md
:white_check_mark:Succeeded

microsoft-edge/extensions/guides/adding-and-removing-extensions.md
:white_check_mark:Succeeded

microsoft-edge/index.yml
:white_check_mark:Succeeded

microsoft-edge/privacy-whitepaper.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off

Docs Build status updates of commit 67bad19:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide.md
:white_check_mark:Succeeded

microsoft-edge/devtools-guide/console/error-and-status-codes.md
:white_check_mark:Succeeded

microsoft-edge/extensions/guides/adding-and-removing-extensions.md
:white_check_mark:Succeeded

microsoft-edge/index.yml
:white_check_mark:Succeeded

microsoft-edge/privacy-whitepaper.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

#sign-off and merged.


doc: release procedure

This PR adds some documentation regarding the procedure for publishing hax.
It also tweaks some settings for cargo release:

adds a hook for asking dune to regenerate opam files
add settings to ignore certain (internal) crates

@W95Psp let's get this in and add a sentence on ocaml releases being github releases.


[octavia-ingress-controller] Add annotations to keep floating IP and/or specify an existing floating IP

What this PR does / why we need it:
Add features:

allow to keep floating IP when delete ingress
allow to use an existing available floating IP

Which issue this PR fixes(if applicable):
none
Special notes for reviewers:
The doc is also updated to indicated how to.
During the ingress creation only add two annotations inspired from openstack-cloud-controller:
annotations:
  octavia.ingress.kubernetes.io/keep-floatingip: "true"        # floating ip will not be deleted when ingress is deleted
  octavia.ingress.kubernetes.io/floatingip: "<IP_ADDRESS>"  # define the floating to use

keep-floatingip:  default value false
If octavia.ingress.kubernetes.io/floatingip is sepcified and an error occurred (fip not available, not exist, not valid) fallback to create new fip.
Release note:

NONE

/easycla

/ok-to-test

Some tiny remarks inline.
I wonder if we could have approached this as we do in the controller for Services. There we don't need the keep-ip annotation because we simply check the description of the FIP and in case it's not created by us, we do not delete it.
This would make the cloud provider more consistent, question is - isn't using implicit annotation better?

Thinking about this more now - if we have two separate annotations now (floatingip and keep-floatingip), we have to consider if it makes sense to use them separately. An Ingress with just floatingip annotation would expect the FIP to exist beforehand and then it will remove that FIP on deletion. Then before it's recreated (K8s reconciliation pattern can trigger that automatically) it would need the FIP to be recreated by the user. This just doesn't fit K8s model.
Then using just keep-floatingip annotation would make the FIPs orphaned on every Ingress recreation. This again doesn't fit K8s model well, when resources might be recreated fairly often and without consequences.
IMO that's a strong argument to just implement the floatingip annotation and instead of relying on keep-floatingip annotation, just check the FIPs description to see if it was created by the Ingress controller or not. See a similar thing in CCM [1].
I now see that CCM has a keep-floatingip annotation support too. I don't think that's a good idea in general, unless there's a use case I don't see.
[1] https://github.com/kubernetes/cloud-provider-openstack/blob/2231e343644690f3e672c024af89d695fe20ac3c/pkg/openstack/loadbalancer.go#L2335-L2339

Okay thanks for precision ! The use case I have in mind is: when you are running a kube cluster on publuci cloud provider based on openstack like OVH or t-system, the first time when you create your ingress, you don't have yet an existing FIP (except if you created it manually before) so you probably want to let your Cloud provider to create it for you.
Now, when you delete your ingress, maybe you want to keep the FIP to not have to recreate DNS record for example and having place the keep annotations can allow you to do that.
You probably right, their is a better implementation possible, juste for the moment I don't have it.

I think that at a minimum I'd add that part about checking if description indicates if FIP was created by Ingress controller before deleting it like it's done in CCM. Then keep-floatingip annotation has a precedent in CCM, so I guess it can be added here too. Thing is - it's much easier to add a new functional annotation than to remove it later, so I'm trying to consider all option.

The logic seems to make sense now, but the code of EnsureFloatingIP() is structured weirdly. IMO it should:

Get list of FIPs attached to the VIP port.

if needDelete.

Delete them and return.

Check if len(fips) > 1, return error if so.

if existingFloatingIP == "":

if len(fips) == 1 return success.

else

Create new FIP and save it into fip

else:

if len(fips) == 1 check if attached FIP has that address and network. If so return success. If not, return error. We might consider if here we shouldn't detach that FIP instead of returning error.

else find FIP by address, check if it's free. If not return error. Save it to fip.

Attach fip to the VIP port.

I let you validate and test or need something else ?

I'm working on #2377 and I think it makes sense to wait until it's merged and then try to sync/reuse the logic in this PR.

I'm working on #2377 and I think it makes sense to wait until it's merged and then try to sync/reuse the logic in this PR.

Ok, thanks for update :)

May we get this merged? I have a scenario where this could be very suitable.

I think this looks good now. @kayrus, do you think we can merge this now? Please note that this modifies how Octavia Ingress Controller handles LBs, so #2377 shouldn't affect this?
/lgtm

I just started with Octavia ingress controller and faced the problem adressed by this issue: use/reuse an externally defined FIP to ensure that a service is exposed with a static address. Any chance to see this feature released soon?

/approve


macOS multiline editing with Option key (alt) not working

Hello, I've looked for other existing issues, but didn't find what I'm suggesting. Maybe it already exists, but I didn't find it both in VSCode or issues.
In text editors, I often use multiline editing through keeping pressed Option button (Alt) and dragging the cursor. E.g. in Sublime Text, I can do something like this.

I think WSCode should absolutely support this. Is it possible? Thank you.
Otherwise, how to add it as custom keybinding?

VSCode Version: 1.28.0
OS Version: macOS HighSierra (10.13.6)

Option + CMD + up / down should do this. Swear I've done it on MacOS.

The key sequence you suggested works with up or down but not with mouse clicks, but if I have an empty row among the selected ones, it will put characters also in there, at the beginning of the line.

I discovered in a completely casual way that in VSCode I have to press also Shift, so the key sequence is Shift+alt and drag the mouse.
Anyway, the sequence brings sometimes to a weird (or buggy, don't know) behavior. A sequence of steps to reproduce follows below the gif.
The drag is done from left to right (and happens too if done vice-versa).

Use the provided keys sequence and select a text
If you try to select another area starting from the same side (outside) of last selected area starting side (e.g. you select first from right to left and then select again starting from right), the text get selected first and then start unselecting up to the start point of the first area.

The same behaviour happens if the second selection starts from top and bottom (outside) of the starting side of the first area.
Doing the same above and below the first selected area, works fine and let you select also text on the other side of the first selected area.

This has been improved in 1.32. See https://code.visualstudio.com/updates/v1_32#_improved-column-selection
Let's track the rest of the gaps in #5402


Fix Athena plugin to script view definitions

SHOW CREATE TABLE does not generate view definitions in the current version of Athena.
This change modifies the getViewDDL definition to run the correct command (SHOW CREATE VIEW) by abstracting out the shared logic to a helper method.

Thank you for the fix!


[BACKPORT] stm32:stm32f40xxx I2C ensure proper error handling.

Injecting data errors would cause the driver to continually re enter the isr with BERR an RxNE.
This fix allows the error to be cleared and  propagated to the waiting task.

Awesome!


Update index.md

just some writing touchups. I am coming from the FCC article on doing hacktoberfest for absolute beginners, and I don't know how to change anything else...yet. so please approve my pull requests and by next year ill be PRing like a pro!
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

i tried changing the title but it doesn't let me. sorry

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


amp swarm command

Create an amp swarm command for local swarm development. The implementation for now will be temporary: the command should start the services in order, or in parallel for services that have no dependencies or else have common dependencies; no service should start until its upstream dependencies are healthy; the entire command should block until the swarm is running and healthy, or it should fail if any service fails to become healthy as specified by its healthchecks. The command should update the terminal to keep the user informed as it progresses.

Concidering Docker will release soon its own version of stack implementation, i propose to do not use amp stack package for that, but just the service package and as suggested above create a command with as few as dependencies with amp.

Yes, I agree ("The implementation for now will be temporary").

It strikes me that these aren't top level commands.
imo they should either be
amp platform start/stop/etc. . .
or
amplifier start/stop/etc. . .


Remove polyfills from loaders and add test

Overview
This PR removes core-js polyfills from top of CDN loader files.  Polyfills are still generated as standalone file.
Testing
A test has been added that runs against sub-es6 browsers with the standalone polyfill added as a script above the loader.  This test just checks that the rum call is made.


x86 compatibility

faiss-rs does not work on the x86 platform. This fix allows you to build faiss-rs on x86 and x86_64 platforms

My edits are aimed only at library building.
I tried using isize. Got a lot of the same errors:

I tried using isizeerror[E0308]: mismatched types
   --> src/macros.rs:38:66
    |
38  |                     faiss_try!(faiss_Index_add(self.inner_ptr(), n as isize, x.as_ptr()));
    |                                                                  ^^^^^^^^^^ expected `i64`, found `isize`
    |
   ::: src/index/flat.rs:128:1
    |
128 | impl_native_index!(FlatIndex);
    | ------------------------------ in this macro invocation
    |
help: you can convert an `isize` to `i64` and panic if the converted value wouldn't fit
    |
38  |                     faiss_try!(faiss_Index_add(self.inner_ptr(), (n as isize).try_into().unwrap(), x.as_ptr()));
    |                                                                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

The library is dependent on our project, which needs to be built on the x86 and x64 platforms. I already realized that these edits will not make work with libfaiss_c.so (simply because it is not going to x86)
I think the library should be built anyway, whether it is used or not

Sorry, but I'm at a loss here. The C API unequivocally defines the idx type to int64_t (at faiss_c.h). Without access and agreement to the C API, this crate cannot function and should indeed not build successfully.
In the event that you have this crate as a dependency and would like to only actually use it on certain platforms, you can make it a platform specific dependency.

I think you're right


Update react-redux to 0.5.0

<EMAIL_ADDRESS>released yesterday and includes many performance improvements along with new API's to use. It would be great to get react-apollo on this new version.
It is backwards compatible so ideally we could just drop it in and be fine with no code change. That said it would be great to try to take advantage of some of the new features. I plan on doing some research to see what all this could do for react-apollo

After looking at more source, I realized that this was prematurely created since react-apollo only uses react-redux for testing purposes.
Closing

I bet if there are some big improvements in react-redux 5, we could use some similar ones in here. Of course the use case is pretty different but maybe they have some generic ideas about improvements for how React HoCs should work.


error: Header must provide a Content-Length property.

I hit the following error:
Client roc-lang: connection to server is erroring.
Header must provide a Content-Length property.
{"{\"jsonrpc\"":"\"2.0\",\"result\":{\"data\":[0,[...],0]},\"id\":1}Content-Length: 38"}
Shutting down server.

Solution
I was able to fix it by updating to the latest vscode

Got the same error. Closing vscode and starting it again "fixed" it :)


chore(package): update @kronos-integration/service to version 5.2.5

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


CASMHMS-6261: Updated dependences for Kubernetes 1.24

Summary and Scope
In CSM 1.6 Kubernetes is being updated to 1.24

Updated docker-kubectl to 1.24.17
Updated the build_and_release_charts.yaml workflow from v2 to v3 so we can continue to build

Created new helm chart 5.0.0 for this.  There was no change in app version.
Issues and Related PRs

Resolves CASMHMS-6261

Testing
Tested on:

fanta (running Kubernetes v1.24.17)

Test description:

Upgraded to new chart with 'helm upgrade'
Because pods don't restart after upgrade if there was no container change, manually restarted them with 'kubectl rollout restart deployment'
Verified no errors in chart upgrade or restart of pods
No CT tests exist for this service, so none were run
Checked logs from service to verify nothing unusual present

Test Checklist:

Were the install/upgrade-based validation checks/tests run (goss tests/install-validation doc)? Y
Were continuous integration tests run? If not, why? N (none exist)
Was upgrade tested? If not, why? Y
Was downgrade tested? If not, why? Y

Pull Request Checklist

[x] Version number(s) incremented, if applicable
[x] Copyrights updated
[x] License file intact
[x] Target branch correct
[x] CHANGELOG.md updated
[x] Testing is appropriate and complete, if applicable

fyi, to make review easier
diff -r charts/v4.0/cray-hms-rts/Chart.yaml charts/v5.0/cray-hms-rts/Chart.yaml
3c3
< version: 4.0.2
---
> version: 5.0.0
diff -r charts/v4.0/cray-hms-rts/values.yaml charts/v5.0/cray-hms-rts/values.yaml
12c12
<     tag: 1.19.15
---
>     tag: 1.24.17


Use non-localized date parsing to fix #1838

This commit uses non-localized date parsing in fontTools.misc.timeTools.timestampFromString. This is consistent with the opposite function fontTools.misc.timeTools.timestampToString, which also uses a non-localized date format.
This function is used when importing a font from XML using fontTools.ttLib.TTFont.importXML via fontTools.ttLib.tables.table__h_e_a_d.fromXML
Depending on the LC_TIME locale setting, the parsing of the created and modified dates of a font might fail, as described in #1838. This fix provides simple code to parse the date string with a fixed format, defined by fontTools.misc.timeTools.asctime. The parsing is now independent of the LC_TIME locale setting.

@ChristofKaufmann thanks for this!
Would you be able to add a test by temporarily doing locale.setlocale and verifying that it works?

I tested this on my machine, but setting the locale requires the locale to be installed on the system. So it's not so easy to test in general.
I also ran run-tests.sh and got:
========= 3407 passed, 6 skipped, 2 xfailed, 20 warnings in 32.35s ========
_________________________________ summary _________________________________
  py: commands succeeded
  congratulations :)

have you tried to see if Travis CI comes with a non-english locale to test with?
If we can't test this easily, we may content ourselves of your manual checks.

Added language-pack-de in Travis CI and added a test function. The test will be skipped, when the locale is not available in the system.
I also tested it with the fixed version of timestampFromString and with the broken version. The test behaves as expected. Seems to be good.
Does the added language-pack-de package influences tests on osx? Did Travis CI check on osx already?

No, only affects the Linux builds. I think this is good, we can merge


Is there any way to run Dawn without Chrome?

Summary
It seems that Dawn needs Chrome to work. However, even with chrome, it doesn't seem to work.
Steps To Reproduce The Bug
I have chrome installed (on Windows), and I try to run the example ts file in the README, app_test.ts:
deno test --allow-run app_test.ts

I get failures for both, even though one should pass (ignore my horribly misconfigured prompt):

According to the first test, it should pass, but it doesn't. If we look at the stack trace, we find three relevant files: app_test.ts, dawn.ts, and headless_browser.ts. Since headless_browser.ts is the relevant file closest to the top of the stack, it is most likely that the error is there. If we take a look at headless_browser.ts:
https://github.com/drashland/dawn/blob/833b6a1d118868289d03129bb9e3cf898f75dd74/src/headless_browser.ts#L101-L108
So it appears that Deno is executing "start chrome", but for some reason, it can't find it? However, I have chrome, and 'start chrome' seems to work when I run it from the command prompt:

Expected Behavior
I expect dawn to run app_test.ts with the first test passing and the second test failing.
Suggested Solution(s)
My best hypothesis is that Deno doesn't inherit the PATH from Command Prompt. I still think it would be really cool to let Dawn run without requiring Chrome.

Hey @MVEMCJSUNPE, See https://github.com/drashland/dawn/issues/20, #19 and the warning in the README.md, Dawn/Cinco (we're renaming this repo to Sinco actually) is still a WIP and doesn't actually work - the code i've added in the v1 PR does work, it's just there's an issue with WebSockets that's preventing the tests from working

Though firefox would be another option, good shout 👍

This is ssomething. we should implement, but it seems there's just a lack of documentation on how to connect to a headless firefox instance
But specific to this issue, i have been reading up on it, and so far i've found:
How to create a headless browser instance
$ /Applications/Firefox.app/Contents/MacOS/firefox --headless --start-debugger-server

For windows, there's only one dash
How to connect to the process
Still unsure. Severe lack of documentation
How to send commands and run actions
No information.

There's literally no information anywhere, and most posts are just like this: https://stackoverflow.com/questions/47874959/firefox-remote-debugging-using-websocket, a perfect question with no answer


Add semicolon

Updated run_stop_run/problem.md to add a missing semicolon.

:fire: Thanks a lot, will be published in 1.7.1.


File close on 6.2.3 does not allow sql file to be closed without saving

When closing a popup launces a save file window and does not have do not save button to dismiss saving file opened for oracle connection.

I cannot reproduce. Is this issue still actual for you?

If I catch you correctly, you could uncheck all the scripts and click Save button - so no scripts will be saved


feat(telemetry-http): add "matched" path to HTTP span names

Yay! There is an extension on the Axum request type that returns the router matching string. This means that if the request path in the Axum router was /api/summary/:workspace_id, then a related request span name would now be GET /api/summary/:workspace_id.
With this implementation detail sorted we can now include the request path without matched ids and other path variables in the span name. This keeps the number of span names lower (in OpenTelemetry jargon "low cardinality") and thus helps to increase the power of querying later on.

bors merge


Significant size of postgresql folder even with no open channels

Background
After starting lnd with postgresql backend, wallet freshly created, no open channels, CPU load spikes to 8 for a long while, and main postgresql folder grows to 900MB and counting. Replica is over 400MB.
sudo du -sch /btc/postgresql/
927M    /btc/postgresql/

sudo du -sch /replica/postgresql/
431M    /replica/postgresql/

Your environment

version of lnd 14.2
which operating system (uname -a on *Nix) 5.10.63-v8+ # 1496 SMP PREEMPT Wed Dec 1 15:59:46 GMT 2021 aarch64 GNU/Linux
version of btcd, bitcoind, or other backend 0.22
any other relevant environment details PostgreSQL 11

Steps to reproduce
Install lnd with postesql backend, create a wallet, do not open any channels and have it stay to mainnet and wait enough time.
Observe the increase in size of the database data folder.
Expected behaviour
All of the above, with expected database folder size up to 50% of lncli describegraph output size
Actual behaviour
Almost up to 1GB of database folder with a bare node

I have a postgres-backed node that has been running on mainnet for 60 days now. The node only ever has one open channel and had performed only a handful of transactions so far, so maybe I can provide my postgres usage as comparison. Not as bad as @asvdf 's node but it is around the 500MB mark, with the most of the size going to the channeldb_kv table.

.

I am not sure if lncli describegraph is indicative of the expected postgres db size, because there is more graph data present on disk than what is reported through describegraph. Signatures for example aren't exposed on the rpc.
A better comparison would be to take the size of channel.db of a fully synced bbolt instance.
My own mainnet postgres instance has a channeldb_kv table size of 293 mb.

What you can do is run VACUUM FULL on the database. Although VACUUM FULL is a heavy operation, and shouldn't be necessary unless you really want to reclaim all possible disk space. In my case, it brought the table size down to 71 mb.
It seems that the table contains unused space because of the heavy updating during the initial graph sync. VACUUM (which runs automatically) makes that space available again for subsequent db operations. Future graph updates can reuse the space. So, according to this theory, your database size should not grow further.

What you can do is run VACUUM FULL on the database. Although VACUUM FULL is a heavy operation, and shouldn't be necessary unless you really want to reclaim all possible disk space. In my case, it brought the table size down to 71 mb.
It seems that the table contains unused space because of the heavy updating during the initial graph sync. VACUUM (which runs automatically) makes that space available again for subsequent db operations. Future graph updates can reuse the space. So, according to this theory, your database size should not grow further.

I knew about the difference between VACUUM; and VACUUM FULL;, I do run a postgres node on a custom signet so I play there as well... on this mainnet node, after the full vacuuming, the postgres folder is 902MB (was 830MB yesterday), so it didn't help much!

so I suppose postgresl's space management is not so good.

Our data schema right now is very naive, it just maps things onto a key-value namespace. As a result, I wager postgres isn't able to properly optimize the on-disk format as it would for a traditional normalized table set.
Closing this as the disk space can be reclaim after initial sync as shown above, and also our format isn't yet space optimized.


added ping module

requires os's ping access

Can we leverage an API call for this?

You mean like pyping? It requires root on linux machines. I'll check it in a week because from today it's vacation time :)

Keeping in mind that it should be cross-platform compatible, we have two options: either use a python library (sockets / requests) to check website availability, or use an external service like Is It Up.
Hit me up after your vacations. Enjoy! :smile:

Hey @sasek333, will you be willing to complete this? :smile:

Actually @swapagarwal something's been bothering me for a while.  The only use of ping module is to check if the internet connection is working or not.
And this condition is already satisfied when we are using messenger right? because it's only then we can use messenger right?
Am I missing something or is there something else to it?

@shanmukhateja Well, this would check whether a website can be reached or not, say is github.com down or not?

Oh OK so it was meant for another website's working state. Good :)

@swapagarwal :If the intention is just to check whether a website is up or down, then we can request for the page from our host server if it returns a response then it is up right else if it times out for say continuously three packets then it must be down.
This way I guess cross compatibility will be maintained and purpose will also be served. If any other use case comes to your mind for which the above does not cover let me know.

@VarunWachaspati Yes, we can implement this on our server itself, but that adds to the maintenance cost!

May I take this up? How about implementing this using isitup.org to check whether a website is down or not?

Added in #197


upgrade to btcec v2

The btcd package was split up and btcec became a new go module. At the same time, it was released as a v2 with major breaking changes.
This will require releasing a v3 of tezosprotocol because the changes in btcec are not backwards compatible and trivial to port to.
This also requires bumping up the minimum go version to 1.16

Thanks for your blessings :) Let's see how this goes after merging and releasing. I'll push a 3.0.1 if necessary.


bug: crashing when ssh i/o timeout happened

Describe the bug

Device Information [optional]
- OS:Ubuntu 22.04
- Swiftwave version: develop
- Docker version: latest

Are you working on this issue?
Yes

Fixed by https://github.com/swiftwave-org/swiftwave/commit/0454d3ed9c4e2d94a59786366fbc0d219486e756


Notify on stale cached records

closes https://github.com/github/licensed/issues/656
This change adds a new configuration option stale_records_action which is used by licensed status to determine if and how to handle stored dependency metadata records that do not match any currently used dependencies.
The configuration option supports the following values:

'error': Treat stale cached records as errors.  Licensed will output errors for any stale metadata files and will cause licensed status to fail.
'warn', '', or unset (default): Treat stale cached records as warnings.  Licensed will output warnings for any stale metadata files but will not cause licensed status to fail.
'ignore', any other value: Ignore stale cached records.  Licensed will not output any notifications about stale metadata files.

The default behavior when the value is unset is warn so that this will not cause any unexpected licensed status failures on version upgrade.

Ignoring the bower failure, this is an issue with downloading the jquery package off bower and not an issue with the source itself.  I've recommended removing that source entirely as most of it's usage has been replaced by npm, yarn, pnpm, deno, etc...


Method for getting credentials of the peer

I'd be useful to have a method for getting peer credentials. I suggest get_peer_cred(&self) -> UCred.
I need this now, so I'll now impl it via extension trait (by getting RawFd) but I plan to submit a proper PR later.

Sounds good to me!


Crossplatform

I know I'm asking something huge,
is there anyone willing to scope the feasibility and/or make this project developable,compilable,runnable on OSX and Linux as well as Window.

The project needs to be converted to CMake (or a similar build system) to be able to build this on other platforms than Windows. As Visual Studio only works on Windows.

Absolute paths needs to be refactored (as the developer mentions in the readme)

Someone needs to take a hard look at dependency management

Rewrite any code that are Windows specific, I havn't looked through the code yet. But the three points over need to be done first.

Working on it :) Also merged some cross platform stuff in


Fix FGS system tests

Following the work done in https://gitlab.diamond.ac.uk/controls/python3/s03_utils/-/merge_requests/4 S03 is now reporting that twice the number of images than expected are being taken.
Acceptance Criteria

The code in the FGS system test that forces the expected images to be twice the expected is removed
System tests still work

System tests inside gridscan may need updating to use latest versions of classes

The Zebra and eiger system tests need updating and fixing too. The others pass when run with s03 working.

this is described in more detail by other open issues


Authentication Problem (ClaimType for "role")

Hello,
I notice that when I create a new Role and I assign it to a specific user, by default the application create the Claim as: "role" -- "RoleName".
This make me a bit confused...
I know that the command:
manager.IsInRole(dn1.Id, "ADMINISTRATOR")
Checks for a claim with Schema: http://schemas.microsoft.com/ws/2008/06/identity/claims/role
So why AspIdentityManager is putting "role" instead of "http://schemas.microsoft.com/ws/2008/06/identity/claims/role" as claim type?
Is there a specific reason?
Hope I was clear...
Thanks to support

Because "role" is the more modern claim type for roles, compared to the old WS-* style of using an XML namespace.

Thanks Brock!
But now... How can I tell to my MVC application, to read "role" as ClaimType instead of "http://schemas.microsoft.com/ws/2008/06/identity/claims/role" ClaimType for the authentication?
Which is the best practice ?
In this moment AspIdentityManager role section works with "role" claimtype, but my application tests "http://schemas.microsoft.com/ws/2008/06/identity/claims/role"...
I'm a bit confused...
Thanks

You use the "roleClaimType" parameter when creating the ClaimsIdentity, or configure the "roleClaimType" properties on the CookieMiddleware in Katana.


Transcode HEVC Source File using VT -- Help Wanted

I have some m2ts source files that are HEVC and when I run other-transcode it is copying them vs transcoding them -- which is what I want because they are in a very high bit rate and I am trying to get them into the default values. I haven't run into this before because most of my source material is not in HEVC. I think the tool is assuming that if it's HEVC, we dont want to transcode again (normally most of the time) and it copies the video.
Is there any way to force it to transcode in accordance with the parameters I have set?
Thanks much and have a happy thanksgiving.

It IS what you want? I’m not trying to be extremely obtuse, just moderately so.
LL

On Nov 24, 2022, at 12:39 PM, mushbuddygoose @.***> wrote:

I have some m2ts source files that are HEVC and when I run other-transcode it is copying them vs transcoding them -- which is what I want because they are in a very high bit rate and I am trying to get them into the default values. I haven't run into this before because most of my source material is not in HEVC. I think the tool is assuming that if it's HEVC, we dont want to transcode again (normally most of the time) and it copies the video.
Is there any way to force it to transcode in accordance with the parameters I have set?
Thanks much and have a happy thanksgiving.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Yes but on review, NVM. It works fine. Not sure what the issue was why it was copying. Closed.

@mushbuddygoose Coolio. Glad it's working now. 👍

Assuming it’s not what you want, have you tried changing the name extension of your input file to “.ts” rather than “.mt2s”?
LL

On Nov 24, 2022, at 12:39 PM, mushbuddygoose @.***> wrote:

I have some m2ts source files that are HEVC and when I run other-transcode it is copying them vs transcoding them -- which is what I want because they are in a very high bit rate and I am trying to get them into the default values. I haven't run into this before because most of my source material is not in HEVC. I think the tool is assuming that if it's HEVC, we dont want to transcode again (normally most of the time) and it copies the video.
Is there any way to force it to transcode in accordance with the parameters I have set?
Thanks much and have a happy thanksgiving.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

For what it’s worth (to anybody), “.m2ts” can always be safely changed to “.ts” (afaik).


InternalCommand with FileBasedLog problem

I was trying to set a test for #176 but found a problem with the serialization of internal commands with the file-based log. When the entry is serialized and written to the log, it ignores the internal flag, leading to a corruption of the state machine. A node that recovers from the log parses the command without the flag, applying the command to the actual state machine, for example, increasing the counter with whatever value the command has.
Here is a diff with a small reproducer: https://github.com/jabolina/jgroups-raft/compare/master...internal-cmd-serde
The fix seems simple enough to get in place. The problem, for me, is how can we be backward compatible in this case?

just do the fix. The file-based log is young and it isn't the default; it is unlikely that it is used in production.

Is this related to [1]?
[1] https://github.com/belaban/jgroups-raft/issues/189

Yes, I'd just apply that change. I do think though that FileBasedLog is the future, not LevelDB...

Is this related to [1]? [1] #189

I think so, without this, #189 will not work properly with the FileBasedLog.
I'll open a PR for this, so it unlocks further progress for dyn membership.


Add link to GitHub repo to the website

Would be nice to let people know that the website is open source and that they're welcome to contribute!!

Good point--looks like the recent PR fixed this :)


Dragonrage: Blacklist<PHONE_NUMBER>72

Dragonrage requests the blacklist of the number<PHONE_NUMBER>72. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Rejected by Jeff Schaller in Charcoal HQ. Reason: 'not yet to this level of spam'


Change content on claim confirmation page

Context
Just a small content change on the claim confirm page.
Changes proposed in this pull request
Translations
Guidance to review
Review Changes
Screenshots

 Update wording on the claim submitted page for prototype and application


WOfS Notebook and installation script for the Hackathon

This PR adds a notebook (labelled 03 because 02 is expected to be TSS) and corresponding installation script for its dependencies. It showcases GA's WOfS algorithm and the use of lpdaac DEM data.
Fixes #8
This branch should be deleted after merging.

One suggestion: Can we rename 03-water-observations-from-space.ipynb to case-studies/water-observations-from-space.ipynb ?

The case-studies directory doesn't exist yet (this would be the first notebook).
There may be implications (relative paths) for the pre-commit hook that need to be updated.

One suggestion: Can we rename 03-water-observations-from-space.ipynb to case-studies/water-observations-from-space.ipynb ?

The case-studies directory doesn't exist yet (this would be the first notebook).
There may be implications (relative paths) for the pre-commit hook that need to be updated.

Done in Commit 9849e32