Is ObjectAssert(AtomicReference) rendundant?

I noticed there is an ObjectAssert(AtomicReference<ACTUAL>)  constructor which is not used anywhere (including tests):
https://github.com/joel-costigliola/assertj-core/blob/83cd126d183cf55fd3a16cc40c0f5f634ed71dc5/src/main/java/org/assertj/core/api/ObjectAssert.java#L39-L41
At the same time there is an AtomicReferenceAssert(AtomicReference<V>) constructor which is actually used by Assertions:
https://github.com/joel-costigliola/assertj-core/blob/83cd126d183cf55fd3a16cc40c0f5f634ed71dc5/src/main/java/org/assertj/core/api/AtomicReferenceAssert.java#L22-L24
https://github.com/joel-costigliola/assertj-core/blob/83cd126d183cf55fd3a16cc40c0f5f634ed71dc5/src/main/java/org/assertj/core/api/Assertions.java#L939-L941
Clearly they are not the same as the ObjectAssert is getting the actual value before creating the assertion, but I was wondering why there is this shortcut for a particular use case, which must be used directly as it's not accessible via Assertions.
At first glance, it seems to me that the former could be deprecated or removed (breaking change), since actual.get() could be easily done inside assertThat().

AFAIR the idea was to make it easy to get the assertions for the underlying atomic object but it was not possible to expose it in Assertions.assertThat due to the existing atomic assertions.
The https://github.com/joel-costigliola/assertj-core/commit/24f054a2c72988d543f545c801401f1290693962 commit shows the same approach for other atomic types.
I think this was not a great idea as few people would be aware of this, retrospectively it should not have been (moreover there is no tests for it).
The best thing is to deprecate IMO (and remove it in 2 or 3 versions).

Would it help to use a similar approach to:
https://github.com/joel-costigliola/assertj-core/blob/83cd126d183cf55fd3a16cc40c0f5f634ed71dc5/src/main/java/org/assertj/core/api/AbstractOptionalAssert.java#L433-L436
having a get() method which performs some preliminary assertions and then returns the underlying atomic object as the new one under test?


feat: set swapfile size in fly.toml

Closes #239
CC/ @rubys

Good change, but a user has reported essentially this same issue.... even after setting swap size in fly.toml:
https://community.fly.io/t/pci-fatal-no-config-space-access-function-found/19594


Rearrange device events in ReceiveIGTLMessage and add AboutToReceiveEvent

ReceiveIGTLMessage() now has only one exit point where it returns success or failure.
Modified and DeviceXYZModifiedEvents are triggered if fromIGTL() is successful.
ReceiveEvent is invoked whether or not there are issues with fromIGTL() conversion.
AboutToReceiveEvent is useful for applications such as Slicer that would like to be notified before device content is modified.

AboutToReceiveEvent and ReceiveEvent are now invoked in igtlioConnector.
Event is still invoked on the igtlioDevice.


Error when doing undo

Another issue :-)
Please see the file which uploaded here: https://dropfile.to/zW8X3
I pressed Del, then Ctrl+Z after selecting the text.

Thanks, fixed. I also fixed the caret position after undo.


Link broken (Azure Governance monthly call link)

Join the Monthly Call on Azure Governance (register here) -> register here link is broken.

Updated. Thank you!


Support for disk bursting

Does this VM family support disk bursting?
The page https://docs.microsoft.com/en-us/azure/virtual-machines/disk-bursting#virtual-machine-level-bursting simply says that "most VM series that support premium disk, support bursting."

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

ID: 7a4a150d-1617-96a4-c460-3190ed8598b4
Version Independent ID: fd8dd71a-84b6-11f9-077f-80ea27980756
Content: Mv2-series - Azure Virtual Machines
Content Source: articles/virtual-machines/mv2-series.md
Service: virtual-machines
Sub-service: vm-sizes-memory
GitHub Login: @ayshakeen
Microsoft Alias: jushiman

Hello @dohughes-msft,
I am reviewing this internally with the content author. I will update the thread further. Please standby.

This size does not support disk bursting.
@support -- Any VM SKU with a burst column (see here) supports it, while those without such a column don’t.
There is no VM bursting for the larger Msv2 VMs, however, we did just recently (within the last month) update the M-series and Msv2 Mid-size to have bursting.
Hope that helps. The sizes team will be taking a look at our tables and finding a way to make this information more obvious. #please-close


Stability of Update Changed Common Components

The problem
The stability is fair but not good. The updated Workbook still may be closed by Excel after an update. This is not a serious issue since the Workbook may be re-opened and the update process continues. However, when other Workbooks are open things become a bit cumbersome.
The improvement
Unfortunately the stability is still an ongoing issue. Since the updated Workbook is no longer Saved after each individual update, the stability increased significantly but still the Workbook (i.e. the whole Excel Application with all it's open Workbooks) closes.
The better is the enemy of the good
I am still fighting with the issues, in other words I've not given up looking for solutions which further improve the stability. The new Application.Run service provided by the Common VBA Message Services offers something in this direction. A modeless displayed message with buttons which perform an Application.Run statement which updates a single individual Outdated Common Component. This provides several advantages:

Each single update is fully completed which includes the removal of the renamed component
The Workbook may be saved manually after an update
The developer has the full control over the process, which component update first for instance or none at all when the message is closed without performing an update

Required prerequisites
Prevent parallel update processed
Done by indicating Busy when a Component Management Service started.
Collection of to be updated
Each update process when done again collects and presents in a modeless displayed  message the still outdated components to be updated - except when the message is closed.
Public Update service for an individual component
The service is called via Application.Run, configured as action to be performed when a displayed message button is pressed. It finishes with the re-collection of outdated components and re-displaying them.

First experience proofed a significant stability improvement. All renewals/updates of outdated Common Components had been passed without problems. I'll keep my fingers crossed ...


frame_parser.ParseIOSamplePayload throws assertion for "IS" command if no IO configured

Consider the following remote AT request:
{
  "type":23,
  "destination64",
  "0013a200408de1fe",
  "command":"IS",
  "commandParameter": []
}

If the remote XBee radio has IO lines enabled (eg. D1=3), the response will be an IO Sample packet, which ParseIoSamplePayload() parses happily.
But, If the remote XBee radio does not have any IO lines enabled, it still responds with an IO Sample packet, but with commandStatus=1.  Further, the frame is shorter and does not contain the number of samples, which causes:
AssertionError: Out of Original Buffer's Boundary
    at BufferReader.MAKE_NEXT_READER.BufferReader.(anonymous function) [as nextUInt8] (/home/user/node_modules/xbee-api/node_modules/buffer-reader/index.    at Object.frame_parser.ParseIOSamplePayload (/home/user/node_modules/xbee-api/lib/frame-parser.js:207:29)
    at Object.frame_parser.(anonymous function) (/home/user/node_modules/xbee-api/lib/frame-parser.js:109:18)
    at XBeeAPI.parseFrame (/home/user/node_modules/xbee-api/lib/xbee-api.js:113:27)
    at XBeeAPI.parseRaw (/home/user/node_modules/xbee-api/lib/xbee-api.js:203:26)
    at XBeeAPI.<anonymous> (/home/user/node_modules/xbee-api/lib/xbee-api.js:133:10)

I think the fix is to add:
if (frame.commandStatus !== 0) return;

at frame-parser.js, line 206, but I need to test and confirm this.

Thanks for spotting this! See the patch I submitted. I'm also checking if the frame.commandStatus property is set. The ParseIOSamplePayload function is also used to parse 0x92 packets (ZigBee IO Data Sample Rx Indicator), which of course don't have the commandStatus property (it's AT command/response related).
Could you check if this works for your problem?

Forgive the lateness of my reply.  Yes, with this patch IS responses from remote radios with and without IO lines enabled are parsed correctly.

No worries, thanks a lot!


Add entropy of a categorical distribution

Hello again! :grinning:
So, about #49: The entropy formula for the categorical distribution is actually the definition of the information entropy, see e.g. Wikipedia.
If all categories have the same probability, the formula collapses to ln(n), which is exactly the entropy of a discrete uniform distribution.
The only gotcha is to handle the cases when a category has zero probability. The naive implementation (used by Math.NET) would throw a NaN when multiplying 0.0 by Inf. So I added a filter() which maps p * ln(p) only on non-zero values. This case is also added to the unit test.

Close #49

Uhm, I've noticed one more things: The summation ranges in the docstrings were not correct, since Rust's range notation a..b means from a inclusive to b exclusive. So basically the summation was done till k - 2.
I've changed the notation to the strictly matematical one; however, maybe we should go for the one used in the Dirichlet distribution?

So the notation within the Formula blocks in the doc strings are to represent Mathematical formulas rather than actual Rust code, but I do agree that standardizing with the notation found in the Dirichlet file would be good. If you could make that change I'd happily approve this PR. Again, I appreciate the hard work!

Sure, I changed the docstring formula style to match dirichlet.rs.
I also appreciate your rapid feedback (and merging :smiley:)! It's always a great pleasure to contribute to a well-maintained project.


Taxonomy term permalinks expand incorectly when adding custom metadata via a taxonomy term page

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.124.0-629f84e8edfb0b1b743c3942cd039da1d99812b0+extended linux/amd64 BuildDate=2024-03-16T15:44:32Z VendorInfo=brew

Does this issue reproduce with the latest release?
Yes
Description
When combining [permalinks.term] configuration with  adding taxonomy term custom metadata, the permalinks are not being generated correctly.
The structure of my site is pretty straightforward with a posts section and  categories/TERM/_index.md pages to augment taxonomy with custom metadata:
.
└── content
    └── categories
    |   └── viajes-en-familia
    |       └── _index.md
    ├── posts
    |   ├── secretos-de-albarracin-pueblo-mas-bonito-teruel.md
    |   ├── ..
    |   ├── ..

In my hugo.tomlfile, I customize the [permalinks.term] configuration to localize the permalinks to spanish (categories -> categorias). Note that this site is monolingual, but I'd like to keep the standard categories taxonomy in the source files and tweak url generation though configuration:
[taxonomies]
  category = 'categories'

[permalinks]
  [permalinks.page]
    posts = '/:year/:month/:day/:filename'

  # [permalinks.taxonomy]
  #   categories = "/categorias"

  [permalinks.term]
    categories = "/categorias/:slug/"

My typical post file uses the categories taxonomy:
---
title: "Secretos de Albarracín, el pueblo más bonito de Teruel"
date: 2024-01-19
categories:
  - viajes-con-amigas
  - viaje-en-familia
  - viajes-romanticos
  - viajes-para-mujeres
author:
  - Juan Carlos y Mar
---

La gran hoz del río Guadalaviar...

And the custom metadata is added to the taxonomy term vía the /[taxonomy]/[term]/_index.md file. For demonstration purposes  I will augment just one category with custom metadata and leave the rest without a metadata page:
/categories/viaje-en-familia/_index.md
---
name: "En familia"
description: "Lugares para viajar con niños y actividades especiales para familias."
---

Now, when rendering a post summary template, I render all the categories listed in the post file using the name property of the custom metadata, if available, while also linking to the appropriate category page:
/layouts/posts/summary.html

    {{ .LinkTitle }}

  {{ range .Params.categories }}
    {{- $category := . }}
    {{- $path := printf "%s/%s" "categories" ($category  | urlize ) }}
    {{- with $.Site.GetPage "taxonomyTerm" $path }}
      {{ or .Params.name $category }}
    {{- end }}
  {{ end }}

Now, with this setup, when rendering a post summary, the links generated for each category show a very strange behaviour. Note that the category that was customized (viaje-en-familia) correclty shows the customized category name in the link text ('En familia') while incorrectly generating the link as the slug is missing:
output:

     Secretos de Albarracín, el pueblo más bonito de Teruel

       viajes-con-amigas

       En familia

       viajes-romanticos

       viajes-para-mujeres

This seems a bug to me, but as this is my first hugo experience, I'm not really sure if there's something else that I'm missing.
Thanks!
Germán

Yes, sorry for the long post, didn't know how to put it with less ceremony.
Without any [permalinks.term] configuration, the second category ('viaje-en-familia') renders this way:
En familia

It uses the frontmatter to customize the link text and uses the category term to use as the slug for the link.
If I add the configuration, the rendered output for this category is:
En familia

Note the slug part is missing. So the link text is still correct and populated from the frontmatter, but somehow hugo is unable to generate the link in the form /categorias/:slug.

@germanftorres
Take this for a spin. I may be missing a detail, but it seems like what you want.
git clone --single-branch -b hugo-github-issue-12278 https://github.com/jmooring/hugo-testing hugo-github-issue-12278
cd hugo-github-issue-12278
hugo server

The published site structure:
public/
├── 2024/
│   └── 03/
│       └── 19/
│           └── secretos-de-albarracin-pueblo-mas-bonito-teruel/
│               └── index.html
├── categorias/
│   ├── en-familia/
│   │   └── index.html
│   └── index.html
├── posts/
│   └── index.html
├── favicon.ico
└── index.html

Note that layouts/_default/single.html uses the much less fragile GetTerms method to render the list of categories.

@jmooring
Thanks for putting this sample.
For demonstration purposes, using name instead of title in the category frontmatter
/categories/viajes-en-familia/_index.md
---
name: En familia
description: Lugares para viajar con niños y actividades especiales para familias.
---

layouts/_default/single.html:
{{ define "main" }}

{{ .Title }}

  {{ with .GetTerms "categories" }}

Categories:
      {{ range . }}
        {{ or .Params.name .Title }}
      {{ end }}

  {{ end }}

  {{ .Content }}
{{ end }}

And finally, adding another category to the post that doesn't have a backing metadata file:
posts/secretos-de-albarracin-pueblo-mas-bonito-teruel.md
---
title: Secretos de Albarracín, el pueblo más bonito de Teruel
date: 2024-03-19T09:50:07-07:00
categories:
  - viajes-en-familia
  - other-categoria
---

Without permalink configuration:

Secretos de Albarracín, el pueblo más bonito de Teruel

Categories:
        En familia

        Other Categoria

Adding [permalinks.term] configuration:
[permalinks.taxonomy]
categories = "/categorias"

[permalinks.term]
categories = "/categorias/:slug"

Secretos de Albarracín, el pueblo más bonito de Teruel

Categories:

        En familia

        Other Categoria

So, in the default configuration, the raw permalink term (viajes-en-familia) somehow manages to get slug portion of the rendered .RelPermalink.
If configuring [permailnks.term], the raw category term gets ignored, and only slug or title properties in frontmatter appear to be used. This is insconsistent, since other categories not backed by frontmatter file still use the raw category term (other-categoria).
Hope I managed to express it clearly. Thanks!!

using name instead of title in the category frontmatter

Why do you do this?

Why do you do this?

I was trying to use arbitrary metadata to customize the display name of a category. Think or authors taxonomy. I would rather use name property instead of a title property in the frontmatter.
But it's also ok to use title in the frontmatter, or even the slug property if that is more natural o more hugoesque way of setting things up.
Germán

Use the predefined title front matter field.

Yes, using predefined title makes sense and works perfectly with the permalinks generations.
Thanks for the guidance!!


Fix trusted delegate calls migration

Don't trust old multisend contracts. They allow nested delegate calls

Coverage increased (+0.05%) to 88.855% when pulling 3096872c50014aedd8ebad150053925c5da9223f on fix-trusted-delegate into 17ac3282a5ee6ea9fe3bbd2c5b6271de967a04cb on master.


Test folder with a simple test file also needs to be generated (if does not exist or --test-project switch can be used)

This is fixed in the PR


Worked on distanceDrive

In line 139 of WheelSystem.java, the distances should be divided by the meter to feet constant because we are converting feet to meters. Also, we should say that this pull request is for #27.


[Bug]Using the env-vars command froze

% of customers impacted: [100]%
Affected Users: [Engineers / Operators / Customers]
Is revenue impacted: [Yes / No]
Urgency: [High / Medium / Low]
Deadline: [Date]

Description of Bug
I used the env-vars command and it froze.
❯ ./cloudfox aws env-vars --profile reader-admin-hoge
[🦊 cloudfox v1.7.1 🦊 ] AWS Caller Identity: arn:aws:iam::364399834359:user/reader-admin
[env-vars] Enumerating environment variables in all regions for account<PHONE_NUMBER>59.
[env-vars] Supported Services: App Runner, Elastic Container Service, Lambda, Lightsail Containers, Sagemaker
[env-vars] Status: 0/105 tasks complete (0 errors -- For details check /Users/motikan2010/.cloudfox/cloudfox-error.log)
(...freeze..)

What should the expected behavior be

Command completed.
Result files (table/env-vars.txt, csv/env-vars.csv) are output successfully.

Platform Affected
[ ] Dev
[ ] Preview
[ ] Production
Steps to Reproduce
Run aws env-vars command.
ex. ./cloudfox aws env-vars
Additional context
The Goroutines argument is required, but was not passed.
In my environment, the following modifications made it work correctly.
❯ git diff
diff --git a/cli/aws.go b/cli/aws.go
index 662485f..2a45ea7 100644
--- a/cli/aws.go
+++ b/cli/aws.go
@@ -480,6 +480,7 @@ var (
                                        Caller:          *caller,
                                        AWSRegions:      AWSRegions,
                                        AWSProfile:      profile,
+                                       Goroutines:      Goroutines,
                                        ECSClient:       ecs.NewFromConfig(AWSConfig),
                                        AppRunnerClient: apprunner.NewFromConfig(AWSConfig),
                                        LambdaClient:    lambda.NewFromConfig(AWSConfig),

Golang version
$ go version
go version go1.17.6 darwin/amd64

Relevant screenshots
If applicable, add screenshots to help explain your problem.

Thanks so much for catching this @motikan2010. This has been merged into main and is reflected in release v1.7.2


feat: Add code

Will follow up on automating NPM publishing

I'm not sure why component installation works when this library is embedded in an action versus these tests

@averikitsch I tried these locally and beta seemed to pass for me. For gsutil I had to bump the gcloud version, maybe an issue with json parsing in that specific version?

Code is ready for review. I will do another review of the documentation and workflows, then plan for publishing to NPM.


Add Support For GGUFv3

Oh, I've already added support in https://github.com/crabml/crabml/pull/57.
It seems that GGUFv3 primarily introduced support for big endian machines and did not introduce any changes on the little endian machines' side. I guess we can defer the actual support for big endian machines to later and, for now, just ensure it can read v3 without errors.

now i believe crabml can inference the GGUFv3 models without error in the main branch.


ViT shared low PCC - split_query_key_value_and_split_heads support

Describe the bug
The OP split_query_key_value_and_split_heads() in the sharded mode is restricted to Bert tensor shape, so I had to hack and comment out that line https://github.com/tenstorrent-metal/tt-metal/blob/main/tt_eager/tt_dnn/op_library/transformer_tms/transformer_tms.cpp#L22
After that the model was functioning but attention function has PCC=0.65, and I was able to isolate the issue in the split function where PCC=0.01 after that step
I see that Falcon, Bloom, Whisper has their special config for the split function, can you please help with enabling it for ViT? or generalize the main function?
To Reproduce
Steps to reproduce the behavior:

Go to 'branch' mbahnas/vit_ttnn_sharded/
pytest tests/ttnn/integration_tests/vit/test_ttnn_optimized_sharding_vit.py::test_vit_attention

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Please complete the following environment information:

OS: [e.g. Ubuntu 20.04]
Version of software (eg. commit)

Additional context
Add any other context about the problem here.

hey @TT-BrianLiu, don't know who can work on this yet (won't assigned you). But need your input on triaging - is this a bug or feature?

Worth to mention that the L1_interleaved version of the model, has 0.98 PCC, and that split OP went to  NlpCreateHeads implementation, instead of the low PCC sharded one that went to  tt::operations::primary::transformers::SplitFusedQKVAndSplitHeads

It's a feature request. Our sharded versions on of NLP TMs are not that general yet. What are the shapes you are trying to run?

@TT-BrianLiu
My current use case of resolution 3x224x224 and batch=8
There are 12 heads and head size is 64 , 12x64=768
So my fused input tensor to the split OP is [8, 224, 2304]

the 224 in the tensor shape is coming from (input resolution / patch size)^2 = 224/16 = 196, then I contact to 197, so padded to 224
Other use case will be high resolution (1280x800) and batch=1 so hidden layer size = 4032

I started a new issue with a standalone failure: https://github.com/tenstorrent-metal/tt-metal/issues/6805. @TT-BrianLiu can you please take a look?

Is #6805 tracking the same thing as this issue?

PR with new op: https://github.com/tenstorrent-metal/tt-metal/pull/7323
This should work for ViT shapes on Grayskull.

I integrated new split/create heads op into ViT for attention. Attention now has 0.99 PCC


[FEAT] - Separately negotiated version number for NTC queries

Imagine some innovative small company wants to use only the abstract parts of ouroboros-network, ouroboros-consensus, and cardano-ledger. They want to write their own blocks, with their own (Ouroboros-compatible) ledger rules and queries, entirely separate from Byron, Shelley, Allegra, etc.
Moreover, suppose they're not going to maintain forks of our code. They'll use whatever the latest version of Ouroboros is via our packages, and build everything else around those dependencies.
Today's NodeToNodeVersion and NodeToClientVersion data types in ouroboros-network prevent this use case from being sustainable, because these hypothetical devs (who by assumption are using the Diffusion Layer) would be unable to introduce new negotiable handshake versions. (They'd have to wait until Cardano does in order to ever release a feature that requires a bump to the negotiated version🤦.)
Proposal:

We need a second, separately negotiated version number that is not at all restricted by the core Ouroboros code (ie just a natural number). Its semantics is determined solely by the block type.

The two existing data types are irreplaceable for versioning the block-agnostic core Ouroboros behaviors: the mini protocols themselves, eg their codecs, eg block diffusion pipelining (in ChainSync, NTN), eg whether LocalTxMonitor is enabled (NTC), etc.
But they should not affect any block-specific behaviors.
This new version number would govern at least the evolution of blocks' Node-to-Client queries.
I currently suspect the new version number would not be useful for any block-specific aspects of Node-to-Node communication, since I anticipate those are all fully handled by the on-chain data, eg the chain's protocol version.

@amesgen @coot @njd42 @dcoutts @abailly @KtorZ What do you think? Am I overlooking some part of the handshake that already supports this (idealized) use case?

Perhaps this should happen as an initialization phase of LocalStateQuery itself?

I am fully supportive of the use case you envision, eg. making alternative implementations of Cardano network participants first class citizen if I understood correctly. I am not sure why you would need a different number, though. What I would do here is to standardise the protocol separately from the implementation, use whatever versioning scheme makes sense and from that point on allow a separate evolution of the protocol specification and the implementations.
This could apply equally well to N2N and N2C protocols, I think, as some nodes could decide to support exchanging some data which other nodes do not support yet (eg. Mithril signatures, Peras votes) and the network could evolve in a more incremental way.
Perhaps this vision is very naive :)

I also agree that it's a really good idea to support alternative use cases.
If this is really an alternative blockchain they ought to use alternative NodeTo{Node,Client}Version versions and we could generalise our protocol to be parametrised over the version types.  This would allow one to govern their case by themselves, but would require them to modify cardano-{node,cli}, which they'd need to do anyway.
KeepAlive is fundamental for at least the first of the two reasons:

keep the network running,
do measurements which are used by BlockFetch.

@abailly said

making alternative implementations of Cardano network participants first class citizen if I understood correctly

but that's actually not what I meant. I meant using the Ouroboros code to run a different non-Cardano blockchain.
I suspect that misunderstanding supersedes the rest of your comment, unfortunately?

@abailly said
but that's actually not what I meant. I meant using the Ouroboros code to run a different non-Cardano blockchain.

😢

I suspect that misunderstanding supersedes the rest of your comment, unfortunately?

😂

Please ignore my comment then @nfrisby

@coot said

they ought to use alternative NodeTo{Node,Client}Version versions and we could generalise our protocol to be parametrised over the version types

I'll elaborate what I think you're proposing. Please let me know if I'm wrong. (I'm not sure I agree with your proposal :), but I want to first confirm I get what you're saying.)

The hypothetical devs' (using our Ouroboros code to run their own non-Cardano chain) would have to provide types MyVersionNT{N,C}, a function MyVersionNTN -> DiffusionLayer.NodeToNodeVersion, and a function MyVersionNTC -> DiffusionLayer.NodeToNodeClient when invoking the Diffusion/Consensus layers. And we would have adjusted the Diffusion/Consensus code to additionally pipe the MyVersionNTC version into their queries. The Handshake protocol would negotiate MyVersionNTN and MyVersionNTC instead of negotiating NodeTo{Node,Client}Version.

I had a nice chat with @njd42 and @karknu. I think I can better present my proposal now. In particular, they had two pieces of clarifying feedback:

One handshake to rule them all. The Diffusion Layer has had the design goal that the version negotiated via Handshake should determine everything the node needs to know about the peer.

Keep it simple. We'd rather not introduce these abstractions until we have a customer/user/community who actually benefits from them.

(My current proposal would only weaken that design goal to "everything the Diffusion Layer needs to know about the peer". And the scope of my proposal is: "this is how we should think about versioning NTC queries", but it's totally reasonable  continue conflating their version with the Handshake version until we actually have a worthwhile reason to introduce the proposed degree of freedom.
In light of that very helpful feedback, I'll update my proposal's text.

I'm now rewriting the Issue description, so I'm archiving the original one (to which all the above replies were written) in this comment.

Imagine some innovative small company wants to use only the abstract parts of ouroboros-network, ouroboros-consensus, and cardano-ledger. They want to write their own blocks, with their own (Ouroboros-compatible) ledger rules and queries, entirely separate from Byron, Shelley, Allegra, etc.
Moreover, suppose they're not going to maintain forks of our code. They'll use whatever the latest version of Ouroboros is via our packages, and build everything else around those dependencies.
Today's NodeToNodeVersion and NodeToClientVersion data types in ouroboros-network prevent this use case from being sustainable, because these hypothetical devs (who by assumption are using the Diffusion Layer) would be unable to introduce new negotiable handshake versions. (They'd have to wait until Cardano does in order to ever release a feature that requires a bump to the negotiated version🤦.)
Proposal:

We need a second, separately negotiated version number that is not at all restricted by the core Ouroboros code (ie just a natural number). Its semantics is determined solely by the block type.

The two existing data types are irreplaceable for versioning the block-agnostic core Ouroboros behaviors: the mini protocols themselves, eg their codecs, eg block diffusion pipelining (in ChainSync, NTN), eg whether LocalTxMonitor is enabled (NTC), etc.
But they should not affect any block-specific behaviors.
This new version number would govern at least the evolution of blocks' Node-to-Client queries.
I currently suspect the new version number would not be useful for any block-specific aspects of Node-to-Node communication, since I anticipate those are all fully handled by the on-chain data, eg the chain's protocol version.

I have now updated the Issue description with a much more fleshed out (and longer :/) motivation/proposal. cc @coot @njd42 @karknu


Normal Distributions

Hello,
Any plans to add scipy.stats distributions to this package?
Normals, skewed normals, truncated normals etc
Thanks

So far no, some time in the future I'll do it, but I can indicate some package that can help you with statical stuff: https://pub.dev/packages/statistics

Would this be something you're looking for a collaborator for? I'd be interested in contributing to a full probability distributions package.

Sure thing! Go ahead clone, code and do PR for the project, the only requirement is: for each function at least one test.


Update cloudwatch, sdk-core, ses, sns, sqs, ssm to 2.20.124

About this PR
📦 Updates

software.amazon.awssdk:cloudwatch
software.amazon.awssdk:sdk-core
software.amazon.awssdk:ses
software.amazon.awssdk:sns
software.amazon.awssdk:sqs
software.amazon.awssdk:ssm

from 2.20.120 to 2.20.124
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #2838.


Fixed pickle arbitary code execution bug in _recattrs

📊 Metadata *
Pickle code execution
Bounty URL: https://www.huntr.dev/bounties/1-pip-loguru
⚙ Description *
Loguru is a library which aims to bring enjoyable logging in Python.
💻 Technical Description *
Untrusted loading of data by the pickle.load function leading to Arbitrary code execution.
🐛 Proof of Concept (PoC) *
Provide the vulnerability exploit to show the security issue you're fixing.
🔥 Proof of Fix (PoF) *
Replay the vulnerability exploit to show the successful fix and mitigation of the vulnerability.
👍 User Acceptance Testing (UAT)
Run a unit test or a legitimate use case to prove that your fix does not introduce breaking changes.

👋 Hello, @Delgan - @Asjidkalam has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above.
Ultimately, you get to decide if the fix is 👍 or 👎. If you are happy with the fix, please write a new comment (@huntr-helper - LGTM) and we will open a PR to your repository with the fix. All remaining PRs for this vulnerability will be automatically closed.
If you have any questions or need support, come and join us on our community Discord!
@Delgan & @Asjidkalam - thank you for your efforts in securing the world’s open source code! 🎉

👋 Hello, @Delgan - @Asjidkalam has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above.
Ultimately, you get to decide if the fix is 👍 or 👎. If you are happy with the fix, please write a new comment (@huntr-helper - LGTM) and we will open a PR to your repository with the fix. All remaining PRs for this vulnerability will be automatically closed.
If you have any questions or need support, come and join us on our community Discord!
@Delgan & @Asjidkalam - thank you for your efforts in securing the world’s open source code! 🎉

Hi.
Thanks for the security report. However, I'm not sure protecting __reduce__() in this case is really relevant. Sure, I understand that pickle.loads() is dangerous, but only when used with an untrusted source.
The RecordException is only meant to serialize Python errors. It will not be used again arbitrary data coming from network for example.
From what I see, the PoC is not different than calling os.system() in __str__:
import os

class MyClass:
    def __str__(self):
        os.system("xcalc")

logger.info(MyClass())


Please add Elmhurst, IL

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: [*** INSERT UNIQUE SUBJECT LINE ***]
Message: (Don't forget to replace the [x]'s with your information!)🔗
Hello,
My name is [NAME] and I am a resident of [NEIGHBORHOOD/CITY]. I am extremely concerned with the City of Elmhurst’s inordinate investment in the Elmhurst Police Department (EPD) at the expense of critical youth, education, and health services. As demonstrated by ongoing peaceful protests and a 2019 Gallup poll, a majority of individuals living in Illinois view the police negatively.
I am therefore writing to demand that the City Council vote to reapportion funds allocated to the EPD to support evidence-based programs and services that, unlike increased police presence, promote the well-being of citizens and reduce crime. It is morally reprehensible to dedicate 30% of the City’s operating budget towards policing while too many citizens are homeless and many more lack access to essential services, including clean running water and necessary mental health resources.
I also urge the City Council to enact legislation that holds police accountable and to overturn policies that allow police to engage in unlawful behavior with impunity.
People are marching in the streets because they know that these actions will result in a healthier, more just society. I implore you to please listen to the needs of your constituents and take immediate action to address their concerns.
Thank you,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

Thank you for submitting your city to defund12.org! Your request is now in the queue for review and publishing. We’ll reply here if we have any questions.

Hi @kmccracken12, I'm having trouble finding a source for the claim that 30% of the city budget goes towards policing. Would you mind sharing where you got that number from?

Hello,
I got the budget info from www.elmhurst.org:
https://www.elmhurst.org/DocumentCenter/View/16061/Governmental-FundsPDF?bidId=
Sent from my iPhone

On Jun 29, 2020, at 2:14 PM, mahrer<EMAIL_ADDRESS>wrote:

Hi @kmccracken12, I'm having trouble finding a source for the claim that 30% of the city budget goes towards policing. Would you mind sharing where you got that number from?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

on it


Update llm openai ut cassettes

Description

fix: #1751
Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make unit-testing and make integration-testing successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

I think the error was caused by the use of the old version to generate the cassette.
Simple fix, there is no resembles the try many times, if subsequent CI problem again, will add a retry to capture


What does Detailed System and Configuration Information net the admin?

I currently don't meet the 25 device threshold to require Pro licensing. I would love it if I had better telemetry for the devices though.
Please go into detail as far as what the Pro license provides for:

Detailed system and configuration information
Peripheral health management
Remote configuration

This is pretty vital to me in order to justify asking for the budget to increase licensing. Otherwise, it's a black box which won't be very persuasive.
Thank you,
SaladGoldRancher

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

ID: a2070d20-21d1-c8bb-bbb4-697083efcd20
Version Independent ID: f4998e3c-74ed-4a2b-fe7c-c55044b9a6b4
Content: Microsoft Teams Rooms licenses - Microsoft Teams
Content Source: Teams/rooms/rooms-licensing.md
Service: msteams
GitHub Login: @dstrome
Microsoft Alias: dstrome

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Chart does not reflect changed "val" if this is the only change

If the only change between two messages is a changed "val" then the chart will not reflect the changed value.
Work-Around: After Browser refresh the next message will display the changed value.

Works now, don't know, why it did not work in first place.


Fix artman link

Was 404ing.

Codecov Report

Merging #1462 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1462   +/-   ##
=========================================
  Coverage     87.17%   87.17%
  Complexity     4039     4039
=========================================
  Files           387      387
  Lines         16232    16232
  Branches       1770     1770
=========================================
  Hits          14151    14151
  Misses         1619     1619
  Partials        462      462

Continue to review full report at Codecov.

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

Ping? Is anyone watching this repo? This is just a small change to fix some links in a readme.

/cc @garrettjonesgoogle @lukesneeringer


Being able to pass options to the node-fetch (eg. timeout)

This would allow us to specify set a read timeout or controlling redirect behavior and more.
See #101 for implementation.

This issue can be closed now I think? Fixed in https://github.com/agraboso/redux-api-middleware/pull/150

But only merged into next as of now. Any idea of when next will be released?

Released in 2.0.0 @lizell! I'll close this out now 🎋

Superb!


ArgumentNullException on GetDotProduct

I'm not a specialist, so maybe I'm wrong, but for every "real stuff" I'm trying to train (format of UserBehaviour is respected), a ArgumentNullException is thrown on GeDotProduct (second argument is null).
Here is my test file (data has been anonymized but comes from "real world" example)
TestUserBehaviour.txt
Thanks a lot for this work.

Figured it out! I was incorrectly initializing the item feature array to the count of users instead of items. Since my tests happened to have the same number of users as items, I never found this bug. If you get the latest from the master branch and try again, it should work.

Thank you, the program seems to work. I will check and analyze the results soon to provide you a more accurate feedback.


The variable access tip is outdated

See: https://esbench.com/bench/6147653f6c89f600a570117c.
Not caching the .length lookup and using pre-fix ++i is the fastest way.

Can you make a PR?  Just edit the file with your thoughts 🙂
https://github.com/Kikobeats/js-mythbusters/edit/master/docs/workflow/variable-access.md


Adobe Air 64 bit support for Android

Any chance you will add soon 64 bit support for Android?

Yes, if we can get out hands on the AIR 33 SDK, I'd be happy to re-package it with 64-bit support added.

Updated ANE with 64-bit support is here; should be treated as experimental for now.

I've added a pre-release build here, which I'll replace with an official release once AIR 33 is out of beta.
Feel free to buy us a coffee if you find it useful ;-)


Add --fresh to wipe all local data before starting.

This option can be used in case of a database corruption.

I have adjusted documentation to refer to all data created by Routinator as ‘cache.’ What is currently referred to as the cache in the source, i.e., routinator::cache, will be renamed in a separate PR.


Opsworks EngineAttributes should take list

EngineAttributes variable in the Server class is currently taking a single EngineAttribute object, but Cloudformation allows for a list of attributes, as is necessary when using puppet R10K. See https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-opsworkscm-server.html#cfn-opsworkscm-server-engineattributes

I created pull request 1363

Fixed via #1363.


Package bug see old code version

Hello OZ,
📝 Details
I wanted to install openzeppelin-cairo-contracts 0.5.0 version, it worked for me but when I go to the source code directly on my .env all the header show that I have the 0.4 version (first screen attached)

🔢 Code to reproduce bug
First I tried to pip install openzeppelin-cairo-contracts==0.5.0 --no-cache-dir
After I pip show it in order to be sure my version have been updated

The source code:

Thanks for the help :rocket:

Hey there! Contracts are fine, it's an issue we had with the version bump script, to be fixed soon!

Thanks for the answer Martin, don't hesitate to close this issue when it has been fixed :+1:

Closing this since it will not be relevant anymore after the ongoing Cairo 1.0 migration. If you think this is a mistake, feel free to open a new issue.


The combination of pino and pino-clf for logging hides errors

The stdout directly written from webpack/backpack is masked by the combination of pino-clf piped to pino.
This can be bad for the case when a developer is running in dev mode, and a syntax error prevents things like auto restarting the service. There is no output to indicate such.
How best to use pino and pino-clf in combination?

For production docker, I've left Pino formatting completely out. Pino logs JSON directly to stdout of the docker container now. Managing the multiple processes via shell pipes isn't worth it.
Probably a different logger (Winston?) should be used for the project.


Add PrepareSchemaIfNecessary option

Hi,
It would be possible to add a PrepareSchemaIfNecessary option (where the default could be true), that when set to false MongoStorage ctor wouldn't try to do any migration? (similar to what the Hangfire.SqlServerStorage does).
Currently, if MongoDB goes down after the service has been initialized, as soon as MongoDB is up again, the server recovers as expected.
But if MongoDB is down when the service is starting, an exception is thrown during service initialization (because MongoMigrationManager.MigrateIfNeeded is executed) and the service won't start.
This can be problematic in some environments such as k8s.
I can open a PR for this if you would like to.
Thanks

@braca,
Yes, off course, I will make this functionality overridable, so you create a class inheriting MongoStorage and you can override the needed functionality.

fixed in v.07.17


Refine the inference demo

Fix #56

 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.

 All committers have signed the CLA.


[Bug] When consuming mysql binlog, there is data loss

Search before asking

[X] I searched in the issues and found nothing similar.

Flink version
1.14.6
Flink CDC version
Flink-Connector-mysql-cdc (2.3.0)
Database and its version
5.7.37
Minimal reproduce step
Unable to reproduce problems, re-consuming data can consume missing data
What did you expect to see?
Hopefully it will be fixed, and if you lose data, expect to see exceptions.
What did you see instead?
re-consuming data can consume missing data
Anything else?
We consumed several clusters yesterday, and the lost data was all within the same period of time
Then, I reconsume the binlog data and find that the data is ready to be consumed again. Is there any possibility of data loss due to network problems? Because we have consumed several clusters, the lost data are all within the same period, but there is no exception log in relevant flink tasks.
The time when datas are lost is 2023-03-27 09:12:xx

Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Maybe it's our procedure. Let's check it first

Maybe it's our  problem with program， Let us check first

it's our problem with program, flink-cdc yyds!


Wait for natural AP regeneration after AP is depleted

Instead of the script stopping immediately after you run out of AP, could it wait until the AP reaches the amount needed to repeat the quest (like for ember/qp farming its 40), so that we can just afk from the phone and not use apples. There are some potential problems I already see with this though, for example:
-If it goes past refresh time while waiting (for example for NA its 0400GMT), when the script attempts to restart the quest again, it will have to get past the initial logo screen, and then dismiss any windows after that (like login bonus) to get back to the quest.
-If the master is careless and doesn’t stop the script after the event ends, the script may continually attempt to get to the event that is already over and could possibly lead to undesired actions (I can’t confirm this, just a hypothetical situation)
-Quest icons change after the refresh time, like for embers it changes from saber/rider to archer/assassin. The change in icon may lead to the script not being able to recognize the quest it’s supposed to repeat? (assuming you use the same team for all farming situations)

There are several problems with implementing this:

If you just wait for your AP to recover in the Repeat screen and you get enough AP, the dialog will not change, as far as I know. So you have to click on Close to be able to start any quest again.
If you click on Close because your AP ran out, you would have to do click on a quest, check if the apple dialog is there, then close. After some time click on the quest again and check for the apple screen again. Since most quests use up 40 AP, the app is stuck doing that for at least 3 hours every time.
If you do such a mode, the device can't be used for anything else during the day, so you can't use this feature on your personal phone. If you use that feature during the night, it will only get you 2-3 quest runs until you wake up. So it only eliminates the need to start FGA in the morning.
If a server update happens, you will get kicked out into the start screen. Now how do you get back to the previous quest? Was it an event quest? A Free Quest in America or Salem? A daily quest? You would have to tell FGA which quest it should start and even then, most quests involve map navigation, which is impossible to implement.
Running FGA for hours just for checking if the AP are refilled drastically reduces the lifespan of your device.

For those reasons, I would reject this feature request.

And, I'm not going to support things like doing story on its own, logging in by itself and picking the quest again and waiting and using AP again because these things can be potentially misused to create bot accounts which don't require any intervention from users.
DW might not mind auto battle but those things would definitely be red flags.


Example not working on Windows

Just trying out ipfs-iiif-db on Windows 10 in Chrome. It doesn't look like it's working for me. I screen cap'd it here:
https://drive.google.com/file/d/0B64TWB9qdiOMWGZJNThUM2Z5OFU/view
(Download the video for HD). Just seems to hang indefinitely on ipfs.object.get in the second window.

Same issue as: https://github.com/ipfs/js-ipfs/issues/840


esp32 connect to ARM controller (IDFGH-2283)

hello..
i want  data from ARM controller to PC   via esp32 bluetooth
is this possible?how?

For general questions please use esp32.com forum. Closing as not IDF issue as such.


build(deps): upgrade ng-ovh-checkbox-table to v1.0.0

Upgrade ng-ovh-checkbox-table
⬆ Upgrade ng-ovh-checkbox-table to v1.0.0
f9069da - upgrade ng-ovh-checkbox-table to v1.0.0
💥 Breaking Change
replace eplace ovh-angular-checkbox-table by @ovh-ux/ng-ovh-checkbox-table
ref: OM-326

I close this pull request and I will re-open it when we will finalize the telecom migration.

Since this dependency will no longer be used. I delete this branch.


refactor: package name to be @davidlj95/ngx-meta

Given ngx-metadata was eventually taken (see #19). Being a scoped package to my NPM username, no one has registered it for sure, as I own that username :P
Changes

Replace ngx-metadata with @davidlj95/ngx-meta around. Lots of imports
Replace name in package.json
Replace name in angular.json
Replace name in tsconfig.json
Replace name in release config

Current dependencies on/for this PR:

main

PR #20   👈

This stack of pull requests is managed by Graphite.


adding skills section (vincent)

fixing (victor)

https://github.com/section-engineering-education/engineering-education/pull/7656


Removed the Repeated Footer Text

PR Description 📜
Remove the Repated Footer Text
Fixes #167

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for review.

## Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

@Mukesh751 Can you Please Review the pull Request soon

Can you Please Review The PR Soon

@sanjay-kv Can You Please Help Me with this PR


The following classes needed for analysis

Gradle 5.1.1
Hello, the following error message appears when I execute gradlew check. Can you please take a look? Thank you very much
`

Task :library-processors:spotbugsMain FAILED
The following classes needed for analysis were missing:
com.sun.tools.javac.tree.TreeTranslator
com.sun.source.util.TreeScanner
com.sun.tools.javac.api.JavacTrees
com.sun.tools.javac.tree.JCTree
com.sun.tools.javac.tree.JCTree$JCVariableDecl
com.sun.tools.javac.tree.JCTree$JCClassDecl
com.sun.tools.javac.tree.JCTree$JCExpression
com.sun.tools.javac.tree.JCTree$JCModifiers
com.sun.tools.javac.util.List
com.sun.tools.javac.tree.JCTree$JCAnnotation
com.sun.source.tree.Tree$Kind
com.sun.source.tree.VariableTree
com.sun.source.tree.ExpressionTree
com.sun.tools.javac.util.Name
com.sun.tools.javac.util.Context
com.sun.tools.javac.tree.JCTree$JCMethodDecl
com.sun.tools.javac.model.JavacElements
com.sun.tools.javac.processing.JavacProcessingEnvironment
com.sun.tools.javac.tree.TreeMaker
com.sun.tools.javac.util.Names
com.sun.tools.javac.tree.JCTree$JCImport
com.sun.tools.javac.tree.JCTree$JCFieldAccess
com.sun.source.util.Trees
com.sun.source.util.TreePath
com.sun.tools.javac.tree.JCTree$JCCompilationUnit
com.sun.tools.javac.util.ListBuffer
com.sun.tools.javac.util.Pair
com.sun.source.tree.TreeVisitor
com.sun.source.tree.Tree
com.sun.tools.javac.tree.JCTree$JCIdent
com.sun.tools.javac.tree.JCTree$JCExpressionStatement
com.sun.tools.javac.tree.JCTree$JCLiteral
com.sun.tools.javac.tree.JCTree$JCMethodInvocation
com.sun.tools.javac.tree.JCTree$JCBlock
com.sun.source.tree.CompilationUnitTree
com.sun.tools.javac.code.Symbol$ClassSymbol
com.sun.tools.javac.code.Symbol
com.sun.tools.javac.tree.JCTree$JCStatement
com.sun.tools.javac.tree.JCTree$JCReturn
com.sun.tools.javac.code.Flags
com.sun.tools.javac.tree.JCTree$Visitor

....
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':library-processors:spotbugsMain'.

SpotBugs rule violations were found. See the report at: file:///C:/Users/GZS16000/Project/cc/library-processors/build/reports/spotbugs/main.xml
`

My project configuration is as follows
`
apply plugin: "com.github.spotbugs"
buildscript {
repositories {
maven { url 'https://plugins.gradle.org/m2/' }
}
dependencies {
classpath "com.github.spotbugs:spotbugs-gradle-plugin:2.0.1"
}
}
spotbugs {
effort = 'max'
toolVersion = '4.1.4'
}
tasks.withType(JavaCompile) {
options.encoding = "UTF-8"
}
tasks.withType(com.github.spotbugs.SpotBugsTask) {
reports {
html.enabled = true
xml.enabled = false
}
}
`

classpath "com.github.spotbugs:spotbugs-gradle-plugin:2.0.1"

Now 4.6.0 is the latest version, please upgrade it to have a try. If it still reproduces the problem, please share a mcve project to reproduce the issue in our local.
https://plugins.gradle.org/plugin/com.github.spotbugs


Fix GRPCRoute's kind passed to the parent reference solver

Signed-off-by: spacewander<EMAIL_ADDRESS>Please provide a description of this PR:
Previous, the GRPCRoute can't attach to the Gateway when allowedRoutes.kinds.Kind is given.

Hi @spacewander. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

/ok-to-test

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

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
d3270f2a2e56756809985020cc5f4d47b78fe02f
link
true
/test integ-ambient

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

I take a look at the failed tests: https://prow.istio.io/view/gs/istio-prow/pr-logs/pull/istio_istio/48585/integ-ambient_istio/1742232768688951296
The failed tests are in https://github.com/istio/istio/blob/3bc4b94560c8a55c896bab6967286d231c496f2e/tests/integration/pilot/common/routing.go#L3012, which don't use gateway API. Maybe it is not relative to this change.


Fix pub.dev issues link

Hey there. Thanks for this cool pub ! :-)
On your pub.dev page, the issue link is broken. I fixed it, you'll need to republish for changes to be applied.

Package will be updated with next major change like this


building cache key does not normalize input

What steps will reproduce the problem?

Make a cache element that is an array where one member is numeric value from ActiveRecord (or even boolean)
Try to fetch same element using said record(but now that member is string)

What is the expected result?
Since serialize((int) 1) !== serialize((string) 1) it will fail
What do you get instead?
Every array member should be cast to something common, so things are easier to manage
Additional info

Q
A

Yii version
2.0.42

PHP version
7.4

Operating system
Docker

@mikk150 would you please provide a few $cache->set() calls with example values?

For example, if I make
$cache->set(['array', 'member', 'number', 42], 'array_cache_key_with_integer');

In ActiveRecord itself(where numeric value is a number)
and try to get it in some other part of code using
$cache->get(['array', 'member', 'number', '42']);

where 42 is fetched from DB but since everything fetched from DB is string, it fails
I actually made this issue to upstream code that has been working in production for over a year

I now understand what the underlying issue is... I presumed(wrongly) that anything that came from DB was a string, but looking at ActiveRecord::populateRecord() I can see how this assumption was wrong...
Now if we use ActiveRecord::load() from POST it becomes a string, so I invalidate using a string, and later I check for cache dependency using an int(they are not same datatype), it breaks.
I tried to fix it by doing

but I think I cannot do it this way, because we can do
$model = new yii\db\ActiveRecord();
$model->test = new yii\db\Expression('NOW()');

and these obviously cannot be typecasted to PHP objects
so I think my only way is to refresh object from DB before I do cache invalidation

Yes, seems so. So... closing PR and issue?


Test for ARM CPU compablity

Finally back from my final exam hell. Now my friend an I are perching a quite powerful ARM development board for other research and (hopefully) an ARM Navi GPU. So we can test how well Etaler ran on ARM.

Update on the subject. I think I'll get my hand on a Xilinx dev board before my server arrive. Hopefully I can make sure Etaler runs on ARM in the next few months.

It works without any modifications!


Added failing test and fix for #89.

Nice work @gRegorLove! This looks good, I’m happy to merge this.


MB-8901 Return 403 when user is not admin

When checking if an existing user is an admin (in order to grant
access to the admin site), we were comparing with the wrong error
message in the scenario where the admin user was not found, and so the
code went to the next if statement and returned a 500 error.
To fix this, we need to check for RecordNotFoundErrorString
as opposed to ErrFetchNotFound.
While I was in here, I added the missing test for this bug, and I fixed
other places where we were returning a 401 instead of a 403.
Description
Explain a little about the changes at a high level.
Reviewer Notes
Is there anything you would like reviewers to give additional scrutiny?
Setup
Add any steps or code to run in this section to help others prepare to run your code:
echo "Code goes here"

Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.
Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
this article explains more about the approach used.

Screenshots
If this PR makes visible UI changes, an image of the finished UI can help reviewers and casual
observers understand the context of the changes. A before image is optional and
can be included at the submitter's discretion.
Consider using an animated image to show an entire workflow instead of using multiple images. You may want to use GIPHY CAPTURE for this! 📸
Please frame screenshots to show enough useful context but also highlight the affected regions.

Messages

:book:
:link: MB-8901

  Generated by :no_entry_sign: dangerJS against 7e372c8917e194fe6c3622fd0dc31f21919ae987


Node data - Fields to be in one-line if text is short

For all sections
Put propery name and value on the same line, instead of having a line-break in between.

ID and data in one line.

Comment can be in the next line as text is too long.


ValueError: Duplicate node name in graph

I am trying to run following code（which comes from Chollet's Deep Learning with Python) on a kaggle kernel,but get an error
import keras
from keras import layers
from keras import backend as K
from keras.models import Model
import numpy as np
img_shape=(28,28,1)
batch_size=16
latent_dim=2
input_img=keras.Input(shape=img_shape)
x=layers.Conv2D(32,3,padding='same',activation='relu')(input_img)
x=layers.Conv2D(64,3,padding='same',activation='relu',strides=(2,2))(x)
x=layers.Conv2D(64,3,padding='same',activation='relu')(x)
x=layers.Conv2D(64,3,padding='same',activation='relu')(x)
shape_before_flattening=K.int_shape(x)
x=layers.Flatten()(x)
x=layers.Dense(32,activation='relu')(x)
z_mean=layers.Dense(latent_dim)(x)
z_log_var=layers.Dense(latent_dim)(x)
def sampling(args):
    z_mean,z_log_var=args
    epsilon=K.random_normal(shape=(K.shape(z_mean)[0],latent_dim),mean=0,stddev=1)
    return z_mean+K.exp(0.5*z_log_var)*epsilon
z=layers.Lambda(sampling)([z_mean,z_log_var])

the trace back is following
InvalidArgumentError                      Traceback (most recent call last)
/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/framework/ops.py in _create_c_op(graph, node_def, inputs, control_inputs)
   1618   try:
-> 1619     c_op = c_api.TF_FinishOperation(op_desc)
   1620   except errors.InvalidArgumentError as e:

InvalidArgumentError: Duplicate node name in graph: 'lambda_7/random_normal/shape'

During handling of the above exception, another exception occurred:

ValueError                                Traceback (most recent call last)
<ipython-input-12-6c91b1ce5964> in <module>
      3     epsilon=K.random_normal(shape=(K.shape(z_mean)[0],latent_dim),mean=0,stddev=1)
      4     return z_mean+K.exp(0.5*z_log_var)*epsilon
----> 5 z=layers.Lambda(sampling)([z_mean,z_log_var])

/opt/conda/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py in symbolic_fn_wrapper(*args, **kwargs)
     73         if _SYMBOLIC_SCOPE.value:
     74             with get_graph().as_default():
---> 75                 return func(*args, **kwargs)
     76         else:
     77             return func(*args, **kwargs)

/opt/conda/lib/python3.6/site-packages/keras/engine/base_layer.py in __call__(self, inputs, **kwargs)
    504             if all([s is not None
    505                     for s in to_list(input_shape)]):
--> 506                 output_shape = self.compute_output_shape(input_shape)
    507             else:
    508                 if isinstance(input_shape, list):

/opt/conda/lib/python3.6/site-packages/keras/layers/core.py in compute_output_shape(self, input_shape)
    672                     xs = [K.placeholder(shape=shape, dtype=dtype)
    673                           for shape, dtype in zip(input_shape, self._input_dtypes)]
--> 674                     x = self.call(xs)
    675                 else:
    676                     x = K.placeholder(shape=input_shape, dtype=self._input_dtypes)

/opt/conda/lib/python3.6/site-packages/keras/layers/core.py in call(self, inputs, mask)
    714         else:
    715             self._input_dtypes = K.dtype(inputs)
--> 716         return self.function(inputs, **arguments)
    717
    718     def compute_mask(self, inputs, mask=None):

<ipython-input-12-6c91b1ce5964> in sampling(args)
      1 def sampling(args):
      2     z_mean,z_log_var=args
----> 3     epsilon=K.random_normal(shape=(K.shape(z_mean)[0],latent_dim),mean=0,stddev=1)
      4     return z_mean+K.exp(0.5*z_log_var)*epsilon
      5 z=layers.Lambda(sampling)([z_mean,z_log_var])

/opt/conda/lib/python3.6/site-packages/keras/backend/tensorflow_backend.py in random_normal(shape, mean, stddev, dtype, seed)
   4327     with tf_ops.init_scope():
   4328         return tf_keras_backend.random_normal(
-> 4329             shape, mean=mean, stddev=stddev, dtype=dtype, seed=seed)
   4330
   4331

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/keras/backend.py in random_normal(shape, mean, stddev, dtype, seed)
   5600     seed = np.random.randint(10e6)
   5601   return random_ops.random_normal(
-> 5602       shape, mean=mean, stddev=stddev, dtype=dtype, seed=seed)
   5603
   5604

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/ops/random_ops.py in random_normal(shape, mean, stddev, dtype, seed, name)
     67   """
     68   with ops.name_scope(name, "random_normal", [shape, mean, stddev]) as name:
---> 69     shape_tensor = tensor_util.shape_tensor(shape)
     70     mean_tensor = ops.convert_to_tensor(mean, dtype=dtype, name="mean")
     71     stddev_tensor = ops.convert_to_tensor(stddev, dtype=dtype, name="stddev")

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/framework/tensor_util.py in shape_tensor(shape)
    992       # not convertible to Tensors becasue of mixed content.
    993       shape = tuple(map(tensor_shape.dimension_value, shape))
--> 994   return ops.convert_to_tensor(shape, dtype=dtype, name="shape")
    995
    996

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/framework/ops.py in convert_to_tensor(value, dtype, name, as_ref, preferred_dtype, dtype_hint, ctx, accepted_result_types)
   1312
   1313     if ret is None:
-> 1314       ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
   1315
   1316     if ret is NotImplemented:

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/ops/array_ops.py in _autopacking_conversion_function(v, dtype, name, as_ref)
   1366   elif dtype != inferred_dtype:
   1367     v = nest.map_structure(_cast_nested_seqs_to_dtype(dtype), v)
-> 1368   return _autopacking_helper(v, dtype, name or "packed")
   1369
   1370

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/ops/array_ops.py in _autopacking_helper(list_or_tuple, dtype, name)
   1302           elems_as_tensors.append(
   1303               constant_op.constant(elem, dtype=dtype, name=str(i)))
-> 1304       return gen_array_ops.pack(elems_as_tensors, name=scope)
   1305     else:
   1306       return converted_elems

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/ops/gen_array_ops.py in pack(values, axis, name)
   5702   axis = _execute.make_int(axis, "axis")
   5703   _, _, _op, _outputs = _op_def_library._apply_op_helper(
-> 5704         "Pack", values=values, axis=axis, name=name)
   5705   _result = _outputs[:]
   5706   if _execute.must_record_gradient():

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/framework/op_def_library.py in _apply_op_helper(op_type_name, name, **keywords)
    740       op = g._create_op_internal(op_type_name, inputs, dtypes=None,
    741                                  name=scope, input_types=input_types,
--> 742                                  attrs=attr_protos, op_def=op_def)
    743
    744     # `outputs` is returned as a separate return value so that the output

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/framework/func_graph.py in _create_op_internal(self, op_type, inputs, dtypes, input_types, name, attrs, op_def, compute_device)
    593     return super(FuncGraph, self)._create_op_internal(  # pylint: disable=protected-access
    594         op_type, inputs, dtypes, input_types, name, attrs, op_def,
--> 595         compute_device)
    596
    597   def capture(self, tensor, name=None, shape=None):

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/framework/ops.py in _create_op_internal(self, op_type, inputs, dtypes, input_types, name, attrs, op_def, compute_device)
   3320           input_types=input_types,
   3321           original_op=self._default_original_op,
-> 3322           op_def=op_def)
   3323       self._create_op_helper(ret, compute_device=compute_device)
   3324     return ret

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/framework/ops.py in __init__(self, node_def, g, inputs, output_types, control_inputs, input_types, original_op, op_def)
   1784           op_def, inputs, node_def.attr)
   1785       self._c_op = _create_c_op(self._graph, node_def, grouped_inputs,
-> 1786                                 control_input_ops)
   1787       name = compat.as_str(node_def.name)
   1788     # pylint: enable=protected-access

/opt/conda/lib/python3.6/site-packages/tensorflow_core/python/framework/ops.py in _create_c_op(graph, node_def, inputs, control_inputs)
   1620   except errors.InvalidArgumentError as e:
   1621     # Convert to ValueError for backwards compatibility.
-> 1622     raise ValueError(str(e))
   1623
   1624   return c_op

ValueError: Duplicate node name in graph: 'lambda_7/random_normal/shape'

This is quite strange as I can run these codes normally on my laptop.I wonder whether it comes
from difference of environment,but I don't know how to check the environment of kaggle kernel

@AIChuY
Which version of  TensorFlow you are using. I am not seeing any issue with Tensor Flow 1.14 and 1.15. Please, find the gist here.If you are using TF 2.X you should use tf.keras instead.Thanks!

@AIChuY
Which version of TensorFlow you are using. I am not seeing any issue with Tensor Flow 1.14 and 1.15. Please, find the gist here.If you are using TF 2.X you should use tf.keras instead.Thanks!

It seems that kaggle kernel is using a tensorflow 2.1.0. I have managed to rewrite the codes and
everything is ok now.
Thanks for your help!

@AIChuY
Which version of TensorFlow you are using. I am not seeing any issue with Tensor Flow 1.14 and 1.15. Please, find the gist here.If you are using TF 2.X you should use tf.keras instead.Thanks!

It seems that kaggle kernel is using a tensorflow 2.1.0. I have managed to rewrite the codes and
everything is ok now.
Thanks for your help!

I have the seem question as you. Could you please post your new codes?

@AIChuY
Which version of TensorFlow you are using. I am not seeing any issue with Tensor Flow 1.14 and 1.15. Please, find the gist here.If you are using TF 2.X you should use tf.keras instead.Thanks!

It seems that kaggle kernel is using a tensorflow 2.1.0. I have managed to rewrite the codes and
everything is ok now.
Thanks for your help!

I have the seem question as you. Could you please post your new codes?

You need to replace the line you import the keras backend with:
from tensorflow.keras import backend as K


Add min_strike, max_strike and use the OptionsProvider to filter the options instead of option chain and universe

Add min_call_strike, max_call_strike, min_put_strike, max_put_strike to Alpha for filtering and update the DataHandler to use the optionChainProviderFilter method directly instead of going through the chain and universe as it does now using the getOptionContracts method.
Any approach to using this should be taken to Scanner.py as that is what is caling the getOptionContracts method.

done here https://github.com/Chocksy/qc-options-framework/pull/22


chore(ci): properly install recent version of git (needed >= 2.18 by checkout action)

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area CI
What this PR does / why we need it:
Moreover, fixed path to output packages from Falco make package target.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @leogr @LucaGuerra @Andreagit97

/milestone 0.35.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Further fixes on top of #2501 .

/unhold
Lets try!


Backups and statistics

In case the backup is running and the script "statistics.sh" is launched, this script must terminate without doing any action.

Fixed.


matrix-synapse: 1.51.0 -> 1.52.0

Motivation for this change
upstream release
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Needs https://github.com/matrix-org/synapse/pull/11969 to build on latest master

https://github.com/NixOS/nixpkgs/pull/155546 fixed this too.

Probably worth replacing with an upstream patch still.


(maint) Don't include OpenSSH config section in table of contents

This was at the same level as the actual transports, causing it to show
up as an entry in the table of contents under "Transport configuration
options".


Add GEO_DISTANCE function; make minimal happy path work

Geo distance is a search feature that returns vectors within a radius, in meters, from a geo point expressed as a latitude/longitude vector.
Implementation:

Add support for GEO_DISTANCE(vector_column, :query_point) < :radius_in_meters and for GEO_DISTANCE(vector_column, :query_point) <= :radius_in_meters in the WHERE clause of a CQL statement
Added the relevant intermediate classes/methods to support query serialization from coordinator to replicas. This implementation is quite subtle, which might make it brittle. For example, in the relation, the operation starts out as a slice operation of < or <=, but then when we build the restriction/row filter from the relation, we convert it to use the BOUNDED_ANN operation. This seems to work exactly as we want, but it might surprise contributors, which could be a design smell. Given our time constraints, I think this is likely acceptable for now.
Added relevant validation to reject queries with invalid inputs
Wired up logic to vector index. The current logic ignores a user requested limit when querying the jvector index and instead passes 100000. This will likely change.
Added a GeoUtil class to improve search radius

Potential work remaining before merging:

Do a refactor to encapsulate the logic a bit more. The Expression class has a lot of very specific geo search details.
Is it worth using the name BOUNDED_ANN as an operator, or should we rename this as GEO_DISTANCE? It is more generic than geo, which is why I used a different name in the first place.

Just to be clear, my understanding is that it is a prototype for a client and you do not expect that code to ever make it to C* with this syntax.

@jbellis - this is ready for review. Class names might need improving, but the underlying design seems right, and my tests are passing except for the prepared query test. I am working on that now.

We'll have a 2.0.3 jvector that can remove the 100k hack in about 20m.  Up to you if you want to wait for that or do a separate PR afterwards.


 I chose Coinbase Pro since it looks like it would be more popular than Coinbase Prime for users of this module.

Thanks so much for this!


About how to build and battery info scraping

Hey,
I want to use this awesome project in polybar via shell script. But i can't figure it out how to compile cli or ffi. I installed battop via yay battop. It works perfectly.
Actually, i couldn't see a installation guide or it could be a not-well documented. Simply,

When i run this: cargo install battery-ffi

Returns:
Updating crates.io index
  Installing battery-ffi v0.7.0
error: specified package has no binaries

battop is not supporting output args like this:

battop --time-to-empty
It may be really really life saver.

cargo build --release

Returns:
error: failed to resolve patches for `https://github.com/rust-lang/crates.io-index`

Caused by:
  patch for `battery` in `https://github.com/rust-lang/crates.io-index` did not resolve to any crates. If this is unexpected, you may wish to consult: https://github.com/rust-lang/cargo/issues/4678

The bare-minimum thing I want is looks like:
$ battery-cli --time-to-empty
Consequently, I want to write the battery's battery_get_time_to_empty output to the polybar. I am trying to figure it out how to write a script that scrapes about battery informations.

Hey, @Dentrax!
Here is a deal:
battery-ffi is a thin interface for communication with programs written in other languages; for example, you can write a Python script, which will load the compiled battery_ffi.so library, call the battery_get_time_to_empty function and get back the Python int type.
There are examples for C and Python exists, you may want to check them out: https://github.com/svartalf/rust-battery/tree/master/battery-ffi/examples
Also, please note that battery-ffi is a library and not a binary application, so you can't cargo install it.
battop is the TUI interface, like vim or midnight commander. It sounds like a good idea to implement one-time output into JSON or plain text, but at the moment it just does not have that ¯_(ツ)_/¯
What I can suggest right now: take the example and write your own application which will output data in the required format.
Since it would be the shell script, you just need to create the Manager struct, iterate over available batteries and write the data into stdout.
You can also find multiple example here at the docs.rs
P.S. I'm also glad to hear that AUR package is working correctly :)

@svartalf Thank you so much! So in this case, i should write my own wrapper using the battery-ffi lib. Previously repo had the battery-cli but i guess it no longer exists.
My second question is: can i use compiled battery_ffi.so in my .sh scripts?

@Dentrax, there are two options here: either you can write wrapper in Rust or in any other language.
In the former case you will need the battery crate only: add it into Cargo.toml dependencies block and call as any other crate library.
In the latter case you will need to:

Compile battery-ffi crate
Get the compiled .so file (and .h file if you want to)
Link or load them into your wrapper
Call any function available.

I doubt that there is any good way exists to call FFI functions directly from the shell scripts, so you'll probably need to call your wrapper binary instead.

Thank you i will try that. And finally, how can i compile the battery-ffi crate? I am asking this simply question because cargo build --release not works. When i run rustc build.rs, it creates build file.
build: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 3.2.0, BuildID[sha1]=522085c26c0bbc8efc450c3c2e2a9fa3f83b645b, not stripped
Is that correct? Am i missing something? (Yet another newbie question :)) Ty.

@Dentrax what is an error for cargo build --release?
Any chance it fails with the following message?
error: failed to resolve patches for `https://github.com/rust-lang/crates.io-index`

Caused by:
  patch for `battery` in `https://github.com/rust-lang/crates.io-index` did not resolve to any crates. If this is unexpected, you may wish to consult: https://github.com/rust-lang/cargo/issues/4678

@svartalf Exactly same error with mine. How can i build that with correct way?

@Dentrax there is a "0.7.1" branch with a fix for this bug.
I'm planning to merge it into the master tomorrow and publish properly. Before that, you can use either this git branch or the pre-compiled .so file from the previous release.

Hey, @Dentrax, fyi, I've published 0.7.1 version to the crates.io.
Sorry it took so long.
Do you still need this issue to be opened?

@svartalf
Thank you, now it's compiling without any error. :)


Add links to a feedback sharing site

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
When users encounter problems, it would be nice if they could share their interaction history with us so we could examine the agent's behavior and see if there was anything wrong.
Fixes #1802
Give a brief summary of what the PR does, explaining any non-trivial design decisions
This PR:

Exposes a unique feedback ID to the frontend
When feedback is successfully uploaded, provides the user with a link to a Vercel app that I set up to display the feedback: https://od-feedback.vercel.app/show

See this video for how the user can copy-paste this link, view it, and share it with us: https://youtu.be/yBf-mT__yKE
And see this link for an example of the shared feedback: https://od-feedback.vercel.app/show?feedback_id=f2e05ad2a2c039ae9379c2864b25f39804a82a6bda1f509d672dd25e5693dd8d

Very cool! Though maybe we can consider adding a new route in the frontend repo instead of hosting a new page? That would also allow others to contribute to that screen (e.g., add loading indicator, format trajectory, MAYBE even have a button to quick load the session directly to OD 🤯)
What do you think?

Hey @amanape , thanks!

That would also allow others to contribute to that screen.

I made the code public here, so that at least solves that part: https://github.com/neubig/od-feedback
I do kinda want to have a link that we can quickly share (so we can ask users to share the trajectory with us so we can check any bugs). So until we have a hosted version of OD, probably the easiest way is to use this standalone app.

MAYBE even have a button to quick load the session directly to OD 🤯

This would be very cool, but one issue is that the workspace is not saved, so it would potentially be a session with a different workspace.


docs(changelog): version 1.2.2 [citest skip]

Create changelog update and release for version 1.2.2
Signed-off-by: Sergei Petrosian<EMAIL_ADDRESS>

This was a test


[NETBEANS-54] Module Review #191

Update apichanges.xml file.

Review comments are applied. Please review.


ConnectionPool would be better if prepared before used

First of all, we are aware of that we should use one connection because lettuce is non-blocking and reactive. But for applications where there are many redis transactions or redis pipeline commands, we should use connection pool to prevent connection creation for each call.
Secondly, the redis connection activation of lettuce is split into mainly two process:

TCP connection
Handshake, maybe some commands should be sent.

While we are using connection pool, there is a min-idle in the configuration. We expect there are min-idle connections in the connection pool before the application is ready, that is, there would be a  large amount of requests when the  application is ready and we expect all connections are ready before handing request in case connection failed because the handsahke command reach command timeout.

GenericObjectPool doesn't automatically pre-populate its pool up to min-idle size, rather it uses min-idle as lower bound to retain a number of active connections in the pool. Generally speaking, pre-warming the pool is an application-specific requirement that may not be generally valid for all usages. If you feel that you need a pre-warmed pool, then please call GenericObjectPool.prepare() in your application. A GenericObjectPool is a pool with minimal resource allocation semantics and thus it creates connections on demand.

GenericObjectPool doesn't automatically pre-populate its pool up to min-idle size, rather it uses min-idle as lower bound to retain a number of active connections in the pool. Generally speaking, pre-warming the pool is an application-specific requirement that may not be generally valid for all usages. If you feel that you need a pre-warmed pool, then please call GenericObjectPool.prepare() in your application. A GenericObjectPool is a pool with minimal resource allocation semantics and thus it creates connections on demand.

I see, maybe I should add this feature to spring-data-redis, thanks

FWIW, Spring Data Redis doesn't pre-populate connections for Jedis or Lettuce. In any case, an interesting enhancement so closing this ticket here.


Bundler not working with Jspm 0.17.0-beta.22

I am seeing the following in my app:
[10:43:56] TypeError: Path must be a string. Received [ './jspm.browser.js', './jspm.config.js' ]
    at assertPath (path.js:7:11)
    at Object.resolve (path.js:184:7)
    at validateConfig (C:\aurelia-jspm.17-skeleton\node_modules\aurelia-bundler\dist\utils.js:46:51)
    at Object.unbundle (C:\aurelia-jspm.17-skeleton\node_modules\aurelia-bundler\dist\unbundle.js:42:29)
    at Gulp.<anonymous> (C:\aurelia-jspm.17-skeleton\build\tasks\bundle.js:21:18)
    at module.exports (C:\aurelia-jspm.17-skeleton\node_modules\orchestrator\lib\runTask.js:34:7)
    at Gulp.Orchestrator._runTask (C:\aurelia-jspm.17-skeleton\node_modules\orchestrator\index.js:273:3)
    at Gulp.Orchestrator._runStep (C:\aurelia-jspm.17-skeleton\node_modules\orchestrator\index.js:214:10)
    at Gulp.Orchestrator.start (C:\aurelia-jspm.17-skeleton\node_modules\orchestrator\index.js:134:8)
    at runNextSet (C:\aurelia-jspm.17-skeleton\node_modules\run-sequence\index.js:86:16)
    at runSequence (C:\aurelia-jspm.17-skeleton\node_modules\run-sequence\index.js:97:2)
    at Gulp.<anonymous> (C:\aurelia-jspm.17-skeleton\build\tasks\build.js:47:10)
    at module.exports (C:\aurelia-jspm.17-skeleton\node_modules\orchestrator\lib\runTask.js:34:7)
    at Gulp.Orchestrator._runTask (C:\aurelia-jspm.17-skeleton\node_modules\orchestrator\index.js:273:3)
    at Gulp.Orchestrator._runStep (C:\aurelia-jspm.17-skeleton\node_modules\orchestrator\index.js:214:10)
    at Gulp.Orchestrator.start (C:\aurelia-jspm.17-skeleton\node_modules\orchestrator\index.js:134:8)
    at C:\Users\Fathom\AppData\Roaming\npm\node_modules\gulp\bin\gulp.js:129:20
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)
    at Function.Module.runMain (module.js:577:11)
    at startup (node.js:160:18)
    at node.js:456:3

My bundle configuration is as follows:
var config = {
  force: true,
  baseURL: '.',
  configPath: [
    './jspm.browser.js',
    './jspm.config.js'
  ],
  injectionConfigPath: './jspm.config.js',
  bundles: bundles.bundles
};

Even though the bundler has a check to see if it is an array, it seems to be ignored.

The plot thickens. If I just supply a string to configPath like it wants, so I just provide: jspm.config.js instead, I get the following error:
SystemJS.config({
^
ReferenceError: SystemJS is not defined
    at evalmachine.<anonymous>:1:1
    at Object.exports.runInContext (vm.js:44:17)
    at readConfig (C:\aurelia-jspm.17-skeleton\node_modules\aurelia-bundler\dist\config-serializer.js:25:6)
    at getAppConfig (C:\aurelia-jspm.17-skeleton\node_modules\aurelia-bundler\dist\config-serializer.js:37:16)
    at createBuildExpression (C:\aurelia-jspm.17-skeleton\node_modules\aurelia-bundler\dist\bundler.js:53:51)
    at Object.bundle (C:\aurelia-jspm.17-skeleton\node_modules\aurelia-bundler\dist\bundler.js:115:25)
    at _bundle (C:\aurelia-jspm.17-skeleton\node_modules\aurelia-bundler\dist\index.js:87:18)
    at C:\aurelia-jspm.17-skeleton\node_modules\aurelia-bundler\dist\index.js:58:18
    at Array.forEach (native)
    at Object.bundle (C:\aurelia-jspm.17-skeleton\node_modules\aurelia-bundler\dist\index.js:50:25)

Changing my configuration files to just use System. and I get a few other errors now about not being able to find dependency JSON files.
configPath: './jspm.config.js',  will get the me the following error (presumably because it needs the browser config for paths):
[11:26:52] (SystemJS) Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>            at Error (native)
        Error loading<EMAIL_ADDRESS>

I'm using the same version in one of my projects. For me it fails when bundling. #132

@robertsundstrom Can you help identify the problem?

Should work with the latest jspm@beta.  Closing this one.


Auto-export project to file.

Create an auto-exporter that exports each project to a directory as an XMI file.

This issue was moved to CoastalHacking/semiotics-main#15


🐞 Default Admonition Types missing from latest update

Check for existing bug reports before submitting.

[X] I searched for existing Bug Reports and found no similar reports.

Expected Behavior
Default types should have been in the drop down menu and the desired default displayed for selection (e.g. Important) selected when first 3 or characters entered by user
Current behaviour
Upon using the menu short cut to enter an admonition, the Admonition menu open as normal. But as I typed in the Important type required, the plug in returned an error message stating no such admonition exists.
Reproduction

Select Admonition icon ftom Ribbon Menu.
Enter "imp" in Admonition Type selection entry box.
You will receive error msg "No such admonition exists

Which Operating Systems are you using?

[X] Android
[ ] iPhone/iPad
[ ] Linux
[ ] macOS
[ ] Windows

Obsidian Version Check
1.5.12 (134)
Plugin Version
10.3.1
Confirmation

[X] I have disabled all other plugins and the issue still persists.

Possible solution
No response

The same occurs for me, through the activation for insertion by means of shortcut. No Admonition appears, neither the patterns nor those created by the user. I tried to disable the shortcut and create again, opened and closed Obsidian (W11) but did not return. I also tried to export the existing ones and import again, but it didn't work either.

I have the same problem in windows 10, in several computers.

same here

Windows 11, latest obsidian, same here

same, no more dropdown in the menu

Thank-you for the fix and all your hard work getting and keeping Admonitions running. It's the first and only issue I've ever had with it. And that speaks to your diligence and skills.


can't find pretrained model

Dear yang,
I want to ask for help. When I open the google drive to download the pretrained model mentioned in README.md. there is nothing in that folder.  I don't know what to do. Could you share it with me again?
Thank you! Looking forward to your reply.

Dear yang,
I want to ask for help. When I open the google drive to download the pretrained model mentioned in README.md. there is nothing in that folder. I don't know what to do. Could you share it with me again?
Thank you! Looking forward to your reply.

Hi Ziyu,
We appreciate your interest in our work.
I accidentally deleted the checkpoints when I cleaned my google drive (almost full) on May 7th. The checkpoints have been reuploaded, please check.
Best,
DPGN team.


Reformat navigation/robot_localization/doc/conf.py

Currently failing the black format check, so leaving this as a note to get it fixed

fixd


Best practices for integrating to master template

Thank you for sharing this package, looking pretty good so far.
I have tried to integrate to the master template but thought of bouncing few questions off you;
a) is there a way to use the code so that it looks for "metadata" property (if defined) climbing up the tree like the way we used to do by putting underscore (ex: Model._metadata) so it looks at current page or its parent(s) until it finds defined page?
b) when i tried to use the no-follow code i keep getting exception even on the page where i captured the metadata property values;
@if (Model.Content.GetPropertyValue<Epiphany.SeoMetadata.SeoMetadata>("metadata").NoIndex)
the .NoIndex keeps returning null. Am i missing something?
cheers

Good to hear you're liking it so far.

You need to call Model.Content.GetPropertyValue<SeoMetadata>("metadata", true) (the true tells Umbraco to search for the property recursively)

You'll need to use the generic version to make sure you are getting an instance of SeoMetadata. The standard GetPropertyValue() method returns an object

@if (Model.Content.GetPropertyValue<SeoMetadata>("metadata").NoIndex)
{

Go away google...

}

If you want to use it recursively...
@if (Model.Content.GetPropertyValue<SeoMetadata>("metadata", true).NoIndex)
{

Go away google...

}

Check out the docs here.


Move Jedi to use LSP instead of internal registration

This is the second step in making a working intellisense engine for notebooks. Could be based on this:
https://github.com/pappasam/jedi-language-server

@rchiodo What can I do to validate this?

Set your language server to Jedi and then set this experiment to on:
    "python.experiments.optInto": [
        "jediLSP",
    ],

Reload VS code and then attempt intellisense in a normal python file.

Doesn't seem to be working for me. I get this message and no intellisense:

Are you doing this with the vsix bits or with your debug bits? You need to install the server manually if you're using debug bits

I was using debug bits. I'll just swap over to a VSIX seems easier than installing a server manually.

You run this to install the server (as well as other requirements)
python -m pip --disable-pip-version-check install -t ./pythonFiles/lib/python --no-cache-dir --implementation py --no-deps --upgrade --no-user -r ./requirements.txt

Not if I use the VSIX though, right? With the most recent insiders VSIX I get the same result.

Yeah it should work with the VSIX. Maybe something has since broke. There's no tests making sure this continues to work