Plotly plots does not show when ipynb is exported to HTML

Type: Bug
Open a new jupyter notebook.
Create a plotly plot.
Export notebook as HTML.
Open HTML file in web browser.
Cells show but not the plots.
(It works if exported in jupyter lab)
VS Code version: Code 1.72.2 (d045a5eda657f4d7b676dedbfa7aab8207f8a075, 2022-10-12T22:16:26.920Z)
OS version: Linux x64 5.4.0-131-generic
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 4000)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
2, 2, 1

Memory (System)
31.01GB (17.73GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

Extensions (22)

Extension
Author (truncated)
Version

jsonl
Alp
0.0.1

format-json
Cle
1.0.2

gitlens
eam
13.0.3

prettier-vscode
esb
9.9.0

unicode-substitutions
Gle
2.3.4

terraform
has
2.24.3

beautify
Hoo
1.5.0

vscode-JS-CSS-HTML-formatter
lon
0.2.3

rainbow-csv
mec
3.2.0

theme-monokai-pro-vscode
mon
1.1.21

python
ms-
2022.16.1

vscode-pylance
ms-
2022.11.10

jupyter
ms-
2022.9.1202862440

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.10

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

color-highlight
nau
2.5.0

vscode-commons
red
0.0.6

vscode-xml
red
0.22.0

vscode-sort-json
ric
1.20.0

markdown-preview-enhanced
shd
0.6.5

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.73.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!


Flexible and Align modifiers doesn't work together

Package version
1.2.0
Flutter version
3.22.2
Steps to reproduce
Run the code sample
Expected results
Box in Align in Flexible. Flexible(child: Align(child: Box()))
Actual results
The Incorrect use of ParentDataWidget error because the Align modifier goes before the Flexible even though it's placed first. That's because of the modifiers default order: Align modifier is listed there but Flexible is not so Flexible is sorted to the end of the modifiers list.
By the way, I found it very misleading that there is a default modifiers order. It's not mentioned anywhere in docs. And I think the default order it's not always good. For example, what if I want to set the box size and then align it's content? To do so I can't use only $with or only $box, I must use the $with.sizedBox and then $box.alignment. $box.height and $with.align won't work!
In my humble opinion, mix shouldn't have the default ordering. The list of modifiers provided by the developer should be the source of truth. And I don't want to override or disable default ordering all the time! 🙂
Full error message:
The ParentDataWidget Flexible(flex: 1) wants to apply ParentData of type FlexParentData to a RenderObject, which has been set up to accept ParentData of incompatible type BoxParentData.
Usually, this means that the Flexible widget has the wrong ancestor RenderObjectWidget. Typically, Flexible widgets are placed directly inside Flex widgets.
The offending Flexible is currently placed inside a Align widget.
The ownership chain for the RenderObject that received the incompatible parent data was:
  SizedBox ← RenderModifiers ← Mix ← Box ← Container ← BoxSpecWidget ← Builder ← Flexible ← Align ← RenderModifiers ← ⋯

Code sample
Code sample
VBox(
  children: [
    Box(
      style: Style(
        $with.expanded(),
        $with.align(alignment: Alignment.center),
      ),
      child: Box(
        style: Style(
          $with.sizedBox.square(50),
          $box.color.red(),
        ),
      ),
    ),
  ],
)

The best solution to this problem seems to be adding a flag that will enable or disable the default order of modifiers

@Maksimka101 You make a good point. We have had discussions about this before, and it seems there would be no perfect solution, but this might be the least imperfect one. I will take a look at the flexible ordering and go from there.


added GiveWorkThanksAction and Tests

Needs to be bulkified.
Your code and my code needs a copyright notice.

merged; going to make one change to the method label so that it doesn't talk about leads.


Include all projects in NuGet package

Adding a separate NPOI.Pack.csproj project that references other projects and thus creates a NuGet package that has all dependencies inside. This changes NPOI.dll to be NPOI.Core.dll (like the project actually is) in order for pack to succeed. Uses Nugetizer reference which fixes project references to be also part of NuGet package contents.
Ideally there would be separate NuGet packages for each on NuGet or as now there's build output (dll and pdb) for each dependency multiple times in package output. The package size is now 11MB which is quite big but that's the way to roll I guess with current setup where Core is/was packaging target and reverse-dependencies are needed.
With Nugetizer there's no snupkg anymore sore pdbs are part of the main NuGet package.

I guess this is now as good as I can get it. Ready for trials.

LGTM


Hard code context menu option to reset to base skin

https://github.com/captbaritone/webamp/commit/587db7a19d76e9cea9fe20c4bcb1cffbf6a73a0b


[opt-tips] Remove -Ounchecked and talk about WMO + 'internal' to reduce dynamic dispatch.

@swift-ci smoke test

@Catfish-Man this one is pretty simple.

Is -Ounchecked being removed?!? It’s quite useful for removing things like collection bounds-checking in situations where you can guarantee indexes will always be valid.

@karwa It is IMO a benchmarking mode that shouldn't really b e used out side of that context. We definitely I don't think want to encourage people to use it.

@swift-ci smoke test

@swift-ci smoke test

@gottesmm It is documented in lots of places, including the standard library‘s documentation: https://developer.apple.com/documentation/swift/1540960-precondition
Deprecating it was suggested once before, but both Ted and Chris agreed that it would need to go through evolution: https://forums.swift.org/t/deprecating-ounchecked/6928/20

@karwa well if you notice I am not suggesting that it be deprecated. These are just optimization tips.

@gottesmm I understand, but the tips are there for swift users to better understand the available options, right? So given that the mode exists, it makes sense to keep the documentation. Otherwise, users might not understand the caveats and over-use it.
Maybe add a bigger warning if you don’t think the existing one is clear enough, but I don’t see any justification to just not document what it does. It’s a supported feature.

@karwa This document is not meant to document optimization options. It is meant to provide tips/directions for improving performance when using standard work flows.
Just b/c the project is not deprecating -Ounchecked (where deprecation means giving notice that -Ounchecked will be removed in a future version of swift) doesn't mean that the project should actively push people towards -Ounchecked and suggest people use -Ounchecked. By putting -Ounchecked in this document, we are doing exactly that.


Remove deprecated requests

These models, corresponding views, forms, templates and tests are deprecated and, honestly, just mess up the codebase:

ProfileUpdateRequests,
EventRequests,
EventSubmissions,
DCSelfOrganizedEventRequests,
InvoiceRequests.

Check if we have a permission to remove them. Remove and forget.

@pbanaszkiewicz   Will this mess up the corresponding data?  Even though all those request forms are deprecated I want to be sure we can view the submissions.

@maneesha yes, my goal was to remove these from the code completely, so that I don't have to maintain it anymore. It's okay if you need it, though.
Are InvoiceRequests needed as well? I think almost no-one used this feature.

I don't think we need to keep InvoiceRequests.
However I would like to keep all the old workshop requests/submissions.

@maneesha okay. What about old ProfileUpdateRequests?

We don't need to keep old ProfileUpdateRequests.


ClientSideAssets missing in the tenant app catalog

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
macOS
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[X] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details
Tenant ID: e4adb445-89ea-4f65-9a90-b147d7fc768a
Describe the bug / error
I've got an older dev tenant that for some reason is missing the ClientSideAssets doclib in the tenant app catalog. This leads to runtime errors when trying to use deployed web parts. What's the supported way to fix this?
Steps to reproduce

In the tenant app catalog deploy an .sppkg
Add web part to the page
Error: failed to load file (ClientSideAssets doclib doesn't exist in the tenant app catalog)

Expected behavior
Web part works as expected.

Few additional questions for engineering to understand that situation

What kind of tenant this is?
When was the tenant created?
Was the ClientSideAssets ever there?

It's pretty impossible to determine what's the root cause of this if it's only an issue in one tenant worldwide.

It's a dev tenant from the dev program (using E3 licenses) created in 2014
The app catalog was created in 2014
I don't know if ClientSideAssets was ever there. Is there a way for me to test it?

@waldekmastykarz could you please check if you can open the /sites/<you_app_catalog>/ClientSideAssets/AllItems.aspx in the browser?

@AJIXuMuK nope, it's a 404

1 - I'm surprised that the package could even be deployed.  I'm assuming that this is a new webpart that you deployed recently.
2 - any chance someone deleted the library?  Can you check the recycle bin?
3 - The easiest way to unblock is going to be to just create the library manually.

Yes, it's a new deployment
The recycle bin is empty
Understood. That's way easier than recreating the whole app catalog.


Remove placeholder text

📝 Changed routes:

…board/wallets/embedded
/dashboard/wallets/local

(view full size)
(view full size)

Commit d9de2e2fd66fad712e49fe14e01546869ac00a13 (https://thirdweb-f9krc2oym.thirdweb-preview.com).


How do I implement breadcrumb items?

How do I implement breadcrumb items?
is there any sample code available for breadcrumb

@nitinjs
for router based breadcrumbs use <cui-breadcrumb> component


uploadFromFile never reaches completion handler

I have modified the sample, as I want to upload a file, rather than text in body of request.
The completion handler on uploadFromFile never occurs. The standard upload uploads a file fine to the container.
There are no errors in the output window in Xcode.
I have tried this with small files (1Mb and larger 20Mb files)
                if let resourcePath = Bundle.main.resourcePath {
                    let imgName = "smallimage1mb"
                    let path = resourcePath + "/" + imgName
                    let localURL = URL(string: path)

                    let storageAccount : AZSCloudStorageAccount;
                    try! storageAccount = AZSCloudStorageAccount(fromConnectionString: connectionString)

                    let blobClient = storageAccount.getBlobClient()
                    let options = blobClient.defaultRequestOptions
                    options?.maximumExecutionTime = 30

                    let blockBlob = container!.blockBlobReference(fromName: name)
                    blockBlob.uploadFromFile(with: localURL!, accessCondition: nil, requestOptions: options, operationContext: nil, completionHandler: {(error: Error?) -> Void in

                        if let _ = error {
                            print(error.debugDescription)
                        } else {
                            print("success")
                        }
                    })

                    blockBlob.upload(fromText: textTextField.text ?? "",  completionHandler: { (error: Error?) -> Void in
                        if (self.viewToReloadOnBlobAdd != nil) {
                            self.viewToReloadOnBlobAdd!.reloadBlobList()
                        }
                    })
                }

Hello! Thank you for submitting this issue; we have added this issue to our backlog for discussion and will get back to you with an update.

Hi @jthake-msft, after experimenting with your code, it looks like the file URL is not correct, please get the URL like following:
let fileURL = Bundle.main.url(forResource:"smallimage1mb", withExtension: "jpg")!

Your url is missing "file:/" at its beginning.

For the record, the completion handler has a known issue.


Request: Moving the reader's view to the centre (macOS)

I'd like to suggest moving the article view to the centre of the app, as its position on the right-hand side is unnatural and uncomfortable when reading. The purpose of the app is the articles we read on it. They should not be somewhere off to the side. Similarly, when we read a book, we don't have it somewhere on the right or left, because it's what we have in the middle that focuses our attention. In the case of this application, our attention is drawn away from the article by the list of articles.
I noticed that instead of reading an article in the app, I open it in the browser for convenience.
Apart from this one shortcoming, I love your app, and thank you in advance for taking what I have written into consideration.

Are you suggesting that flow of selecting an article be changed from left to right to something like left-right-left?

I've taken various things into consideration, but at this point I think the least complicated solution would be as in the graphic.

Do you know of any other apps — any other newsreader or similar (email app, maybe) — that does something similar? It would be great to see some prior art.

That's kind of my thought, but I found another interesting solution that we come across when we go to a news site (such as the NYTimes). This was applied by Newsify (although this app does a poor job of fetching text).

In the first part, we are looking for the article, and this is then the most important part. The text doesn't display by default because we haven't selected it yet.

When we find an article, then it becomes the most important and nothing distracts us from it. In addition, there are options for font and background management (similar to Safari in reader mode).


Feature/wrapper

This should help with #38, but is not yet ready to be pulled. There are a few issues, I'd appreciate help with:

the PackageWrapper is a String for now, which is quite useless. I guess we would need something that can a) query the settings, and b) receive the actual binary name.
I have not figured out how to obtain the GhcSourePath in the Rules () to append to prepend to build the absolute path of the executable.

Also this feels like a very crude hack, hence any advice is appreciated.

@angerman Thanks for giving it a try! Does the current implementation work for you? I guess it doesn't as I don't see any -B stuff yet.

the PackageWrapper is a String for now, which is quite useless. I guess we would need something that can a) query the settings, and b) receive the actual binary name.

What is the intended purpose of PackageWrapper? Is this the full contents of a wrapper? If yes, this could be done similarly to how Generators work. Have a look at Rules.Generators.ConfigHs.

I have not figured out how to obtain the GhcSourePath in the Rules () to append to prepend to build the absolute path of the executable.

It's not possible, because GhcSourePath cannot escape the Action monad. Why do you need an absolute path in Rules ()?
A couple of further comments:

All this wrapper stuff should be disabled on Windows. The relevant condition to check is windowsHost from Oracles.Config.Setting.
We need a data type data Program = UnwrappedProgram | WrappedProgram Wrapper or something similar to forbid cases when pkgType == Library and pkgWrapper /= Nothing.

@snowleopard this only works, if I make the follow non-universal adjustments:
        let top = "/Users/angerman/Projects/haskell/ghc/"
        let wrapper = unlines [ "#!/bin/bash"
                              , "exec " ++ top ++ wrappedProgram ++ " -B" ++ top ++ " ${1+\"$@\"}"
                              ]

It's not possible, because GhcSourePath cannot escape the Action monad. Why do you need an absolute path in Rules ()?
I hope this becomes clear from the snipped above.

What is the intended purpose of PackageWrapper? Is this the full contents of a wrapper? If yes, this could be done similarly to how Generators work. Have a look at Rules.Generators.ConfigHs.

Ideally the PackageWrapper should be in the Action monad, such that one can access the whole environment (e.g. read the GhcSourcePath, and potentially other information relevant to the build system). I envision the PackageWrapper to actually build the wrapped files content (e.g. produce what wrapper is above), that would give sufficient flexibility for all potential wrappers, I think.

All this wrapper stuff should be disabled on Windows. The relevant condition to check is windowsHost from Oracles.Config.Setting.

Yes, this clearly needs to boil down to a no-op on windows. I'm not sure how to do this elegantly.

We need a data type data Program = UnwrappedProgram | WrappedProgram Wrapper or something similar to forbid cases when pkgType == Library and pkgWrapper /= Nothing.

How about the following? Then I'm not so sure what you were debating about cabal package resemblance on IRC, which is why I went ahead with pkgWrapper.
data PackageType = WrappedProgram ProgramWrapper | Program | Library deriving Generic

I hope this becomes clear from the snipped above.

Hmm, why does the above snippet live in Rules? In Rules we only define targets or match targets, but this snippet builds the target, so it should live in Action where you have access to GhcSourePath and friends.

Ideally the PackageWrapper should be in the Action monad, such that one can access the whole environment (e.g. read the GhcSourcePath, and potentially other information relevant to the build system). I envision the PackageWrapper to actually build the wrapped files content

Exactly! So why don't you turn PackageWrapper into an Expr String? Is this because of a potential import cycle?

Yes, this clearly needs to boil down to a no-op on windows. I'm not sure how to do this elegantly.

OK, don't worry about this. I can add conditionals later.

How about the following?

data PackageType = WrappedProgram ProgramWrapper | Program | Library deriving Generic

I think this is semantically wrong: wrapped and unwapped programs are types of programs, not types of packages. We really should contain the wrapping flavour inside a Program.

Then I'm not so sure what you were debating about cabal package resemblance on IRC, which is why I went ahead with pkgWrapper.

Ah, I see. That was about #12. At some point we might change PackageType from Program | Library to something like PackageType [Program] [Library] to reflect the fact that a single cabal package can mix programs and libraries. This shouldn't influence your work on wrappers.

This now correctly generates the binary and wrapper for ghc-stage1. It's still a pretty big hack.

Awesome!
Let me study your code and see whether it can be simplified if we switch to an alternative non-general solution.


Prevent double calls to search.execute! when also querying shards, aggregations, and suggestions'

I was noticing a ton of duplicate search logs in my app, and tracked it down to the Response.response method running a search.execute! in which the raw_response is not being saved, causing subsequent calls to query shards, aggregations, and suggestions to perform another search.execute!. call An example of when this might occur is the following
# Setup search with query and aggregations
response = Post.search('foo')
# Actually execute the search and get results
results = response.results
# Make a second execution because response's `raw_response` is not being cached
aggregations = response.aggregations

By simply storing the search.execute! return in the @raw_response var, we can utilize the cached raw_response in the subsequent calls and prevent double searches.
I have submitted 2 pull requests prior to this one that ended up having pitfalls so I closed them. After some more digging in, I think this simple approach works well. It passes all tests on master, doesn't interfere with pagination (like previous attempt 1) and doesn't add any additional configuration or parameters (like previous attempt 2). Please consider a merge.

Hi @coneybeare, thanks for catching this! We'll definitely look into this — re-using the @raw_response is definitely a good approach, we'll have to check if there are not some thread-safety etc issues.

Hi @coneybeare
I've opened a pull request as well to solve this here
The reason we have the raw_response and response methods is because using HashWrapper can sometimes have large performance implications (see this issue). The raw_response method allows someone to skip creating a HashWrapper and just get the raw response in the form of a Hash. I've taken a slightly different approach to you and just rewritten the #raw_response method so that if:

A HashWrapper has already been created from a search response, the Hash representation of it is returned.
A HashWrapper hasn't been created yet from a search response, it executes the search and returns the raw Hash response.

What do you think?

Closing in favor of https://github.com/elastic/elasticsearch-rails/pull/850


The documentation does not show how to decrypt using a specific certificate

I want to decrypt the XML using a specific certificate which may not be in the local store. Is this possible? It would be nice if the documentation addressed this.

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

ID: 0f05d9eb-6fa2-e2f4-d254-05240c6443ce
Version Independent ID: 1326b2af-21da-9def-c267-43aad4a033c5
Content: How to: Decrypt XML Elements with X.509 Certificates
Content Source: docs/standard/security/how-to-decrypt-xml-elements-with-x-509-certificates.md
Product: dotnet
Technology: dotnet-security
GitHub Login: @IEvangelist
Microsoft Alias: dapine

Associated WorkItem - 72554

I'm not actually sure, let's ask @blowdart.

That's because the xml encryption standards embed a key identifier in the encrypted xml. You don't get a choice to use a different certificate because decryption must use the certificate referred to in the encryption elements. That's part of the xmldigsig standard for the keyinfo elements.

@blowdart in that case, how can I tell EncrpytedXml how to find my cert for decryption? Presumably it has to do some lookup process based on the embedded key identifier.

You can't. It's hard coded to check the certificate stores. And given it's such an old standard, there's no new feature work planned for it.

@blowdart I think that context could be helpful to add to the encryption/decryption docs. It would be nice to make that limitation explicit. Do you agree?

I mean it's there in the doc itself

The code example in this procedure decrypts an XML element using an X.509 certificate from the local certificate store of the current user account.

It's just it doesn't say that's the only way.
And in the encrypt doc

To decrypt the XML element, call the DecryptDocument method, which automatically retrieves the X.509 certificate from the store and performs the necessary decryption.

The docs for the api do show a method where you can force it to use a key you specify, providing it matches the key info
https://learn.microsoft.com/en-us/dotnet/api/system.security.cryptography.xml.encryptedxml.decryptdocument
So it's possible with a lot of manual labour, which puts it ourside of the scope of a simple how-to in my opinion

Based on the feedback here, and the fact that the docs already cover this - I'm going to close this issue, as it was more of a product question. Thanks for posting this @madelson and thank you @blowdart for your feedback/support.


pip check fails with pip==24.2: async_generator is not supported on this platform

Issue
Running a pip check in a fresh conda environment with latest pip==24.2 reports a problem that's likely caused by invalid wheel metadata. More info and discussion can be found on this issue:
https://github.com/pypa/pip/issues/11054
To reproduce
$ conda create -n test pip=22.4 async_generator
$ conda activate test
$ pip check
async_generator 1.10 is not supported on this platform

Note that pip check does not fail if the async_generator package is installed with pip from PYPI.

Just for posterity, it looks like the problem was an incorrect Python tag specified in the WHEEL metadata file. Here's how it was:
Wheel-Version: 1.0
Generator: bdist_wheel (0.31.1)
Root-Is-Purelib: true
Tag: cp37-none-any

And here's the new build, that has correctly switched from cp37 to py3
  Wheel-Version: 1.0
  Generator: setuptools (72.1.0)
  Root-Is-Purelib: true
  Tag: py3-none-any


docs(layout): improve UI/UX of the presentation

Hey @shinokada,
First of all great job for the contribution!
This PR serves the purpose to improve the presentation and UI/UX for the Flowbite Svelte Blocks repository.
📑 Description
I have improved the homepage, the main navbar, content, and you can also check out this page for the main content area too:
http://localhost:5173/application/advanced-tables
The changes I've made here have to be added to 132 files which is not sustainable so we need to create a separate component where the layout itself will be modified in one place and we only pass the actual data as props (ie. title, description, GitHub link, code example).
I've described the issue here: https://github.com/shinokada/flowbite-svelte-blocks/issues/15
I have also updated the navbar in two places already (the main layout and the one for application) but I have to do that for marketing and e-commerce too? Let's just have on navbar if possible just like the footer.

[x] Completed

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information
We should also update the titles to something like this:
Svelte Advanced Tables - Flowbite
Svelte Header - Flowbite
Svelte CRUD Modals - Flowbite
To put an emphasis on Svelte rather than Tailwind CSS.

Before:

After:

This should be applied to all the pages - spacings and margins have been improved.

Please let me know when it is ready to be merged.

I updated and you can see the details.
You need to modify one file to style the section block.
I will update layouts soon.

I updated layouts.
Please proceed updating your PR.

Hey @shinokada,
Thanks for the layout updates - I've finished my part by updating UI/UX related spacings, margins, containers.
I have also updated the expected titles to match a better SEO expectation.
Do you think we could upload this to Flowbite-Svelte.com eventually? I'm thinking we could use a worker or GitHub pages.
Something like:
https://flowbite-svelte.com/blocks/...
Tests have passed, but please double-check if everything works okay.
Cheers,
Zoltan

@zoltanszogyenyi Thanks. Yes, it is a good idea to move to flowbite-svelte.com/blocks.


[BUG][WEBJS] - sendVoice Not Working

Describe the bug

The sendVoice endpoint doesn't work, it hangs and eventually times out with a 503 error

Version
{
  "version": "2024.6.8",
  "engine": "WEBJS",
  "tier": "PLUS",
  "browser": "/usr/bin/chromium"
}

Requests - Responses

sendVoice is working for me in 2024.7.2

sendVoice is working for me in 2024.7.2

Not working here.. 2024.7.2

@borsatti2 could you try to use devlikeapro/waha-plus:chrome version and send the example to POST /api/sendVoice?
{
  "chatId"<EMAIL_ADDRESS>  "file": {
    "mimetype": "audio/ogg; codecs=opus",
    "url": "https://github.com/devlikeapro/waha/raw/core/examples/dev.likeapro.opus"
  },
  "session": "default"
}


Enable copy from the suggest-widget details

VSCode Version: 0.7.0
OS Version: All?

Steps to Reproduce:

Trigger suggestion window
Click on the doc-details icon to move to the details view
Try to mark parts from the documentation for copying

The window is closed (cannot copy parts of it)

#4288


Update linux aarch64 and ppc64le CI providers to default

Default builds on native hardware, which is more reliable.

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

This is derived from the after merge discussion of https://github.com/conda-forge/ripgrep-feedstock/pull/5
@jayfurmanek @jakirkham @isuruf

The native builds are fast compared to azure... even faster than the win azure build ... :)
Travis is triggered twice and one of it is failing ... what to do about that?
The error is:
Status: Downloaded newer image for condaforge/linux-anvil-ppc64le:latest
standard_init_linux.go:190: exec user process caused "exec format error"
The command "if [[ ${PLATFORM} =~ .*linux.* ]]; then ./.travis/run_docker_build.sh; fi" exited with 1.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

What's also interesting is the other Travis CI job seems to be using x86_64; hence, we see a failure there.

@jayfurmanek, does Travis CI only provide ppc64le images on the .org domain or do they provide them on the .com domain as well?

Only on .org (for now). I’m not sure why the .com shows up in some feedstocks. It shouldn’t. In some other feedstocks we just ignored it.

@jayfurmanek Thanks for confirmation.... I am also seeing that travis behaviour in random other feedstocks... mostly ignored that, because it has not consequence.

Ok. ignoring the non-relevant second failing travis run.


Manual mode crashes due to non-int channel

The plugin crashes when run in manual mode because the channel is set to '-' when the only non-int character currently handled is '*'

Fixed with string handling instead of int.


ld: symbol(s) not found for architecture arm64

Please provide a detailed description of your question or the information you seek.
Hello!
I met an error when use astra-sim. My device is Macbook Air with M1 chip. Here is my error:
ld: symbol(s) not found for architecture arm64
ld: symbol(s) not found for architecture arm64
clangclang: : error: error: linker command failed with exit code 1 (use -v to see invocation)linker command failed with exit code 1 (use -v to see invocation)

make[2]: make[2]: *** [bin/AstraSim_Analytical_Congestion_Aware] Error 1
*** [bin/AstraSim_Analytical_Congestion_Unaware] Error 1
make[1]: *** [AstraSim_Analytical/CMakeFiles/AstraSim_Analytical_Congestion_Unaware.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [AstraSim_Analytical/CMakeFiles/AstraSim_Analytical_Congestion_Aware.dir/all] Error 2
make: *** [all] Error 2
I wonder how can I solve it. Thank you so much.

I've address this problem. It's caused by gcc. You are supposed  to use gcc below version 11. So you'd better use Debian 11.x instead of Debian 12.x as there is only gcc-12 for Debian 12.x and gcc-10 | gcc-11 for Debian 11.x.


Enabling gcc complaint HIP memory APIs

hipMemcpyToSymbol , hipMemcpyToSymbolAsync,  hipMemcpyFromSymbol and
hipMemcpyFromSymbolAsync APIs were not gcc complaint hence made them c complaint.

Please ignore those changes, as few of my PRs I had raised on my forked master hence those are showing up here.

Closing this PR as it has got some extra unintended commits. Have raised separate PR with only required changes.


Fix: Display Link instead of button for git issue link in task details page.

Issue:
Main issue: #774
This is a follow-up PR of #827 to resolve review comments.
Review comments:

Use Link instead of button
Write proper CSS.

Description:
Changes :

A link is used instead of a button for displaying the git issue link on the task details page.
Modified the tests accordingly for the link.
Removed unused CSS.

Anything you would like to inform the reviewer about:
A review comment in earlier PR was to use single line. But this formatting is done by Prettier so I could not add it in a single line.
src/components/taskDetails/index.tsx
Dev Tested:

[ ✓] Yes

Images/video of the change:
Before :

After:

gitIssue2.webm
Follow-up Issues (if any)
No
###Test Stat
Before:

After:

can you update proper title of PR please

yes, updated. Thanks.


Question: dlib::type_safe_union::swap()

Why is the swap function not just this:
void swap (
    type_safe_union& item
)
{
    std::swap(mem, item.mem);
    std::swap(type_identity, item.type_identity);
}

If you try it, the unit tests don't pass. But i cannot see why...

Is it that this won't work if whatever object is being constructed with placement-new stores its own underlying storage in a pointer. Possibly std::string does this?

Something like that.

Right. That wouldn't work for objects that are not trivially copyable. Such as std::string.

Yeah I understand why now. But fundamentally, you would think, "why can't I swap the storage, and the thing that tells you how to read the storage". After a few years of C++ing, you still get surprised by subtle things like this.


Buzzword List Dumping

https://www.amanet.org/articles/80-business-buzzwords-now-and-then/
https://www.pointsincase.com/lists/corporate-buzzwords-redefined-for-work-at-home-times
https://andrewessa.com/2020/08/11/119-worst-business-buzzwords-for-2020/
https://topsourcetalentllc.com/60-business-buzzwords-to-delete-from-your-vocabulary/
https://www.platformexecutive.com/news/blog/the-120-most-annoying-business-phrases-and-buzzwords-in-2021/
https://www.thompsonwriting.com/business-buzzwords
https://www.trustradius.com/buyer-blog/119-hated-business-buzzwords
https://www.psychologytoday.com/us/blog/tame-your-terrible-office-tyrant/201603/75-corporate-buzzwords-and-phrases-drive-us-crazy

Are you interested in working on these?

PR welcome!


SURVIVOR stalls when reference FASTA lacks newlines

Hello,
I've been trying to run LRSIM to generate some simulated human reads, and I've noticed that the run is stalling at the initial SURVIVOR stage. (Ie. stuck there for > 12 hours).
After doing some tests, I suspect that it is due to my reference FASTA file lacking newlines for a given sequence.
Ex. for e. coli tests:
SURVIVOR 0 ./ecoli.fa ./test1.hap.parameter 0 ./test1.hap 1000
completes in under a second (fasta has 80 residues per line)
while
SURVIVOR 0 ./ecoli_noNewlines.fa ./test1.hap.parameter 0 ./test1.hap 1000
stalls, and I aborted it after ~4 minutes (fasta has all residues for a sequence on one line).
I'm just wondering if that is a known formatting requirement for running SURVIVOR?
Thanks!
Lauren

Hi Lauren,
thanks for pointing this out. Yes the way I read in files assumes a maximum number of characters per line (200,000 to be precise). I will add that to the requirements.
Thanks
Fritz

I just updated the Readme. I hope that is sufficient.
Thanks
Fritz

Thank you!


Crash on 1.9.8

Crash encountered running PhantomJS 1.9.8 on CircleCI.
Dump here: https://dl.dropboxusercontent.com/u/70299014/3176409c-bf01-cd9f-156cebbd-0d9a92da.dmp
Thanks,
Jeremy

Please upgrade to PhantomJS 2.0. 1.9.x is no longer supported, and known to have many crasher bugs that are fixed in 2.0.


Feat: SpringCloud subscribe strategy and Adapter add enabled config

What this PR does:
Which issue(s) this PR fixes:

Fixes #
#410
Special notes for your reviewer:

SpringCloud firstFetch by subscribe strategy
User control Adapter is or not enable
Does this PR introduce a user-facing change?:

Codecov Report

Merging #425 (2656ff0) into develop (bcc678d) will decrease coverage by 0.03%.
The diff coverage is 25.00%.

@@             Coverage Diff             @@
##           develop     #425      +/-   ##
===========================================
- Coverage    37.58%   37.55%   -0.04%
===========================================
  Files           55       55
  Lines         3720     3723       +3
===========================================
  Hits          1398     1398
- Misses        2166     2168       +2
- Partials       156      157       +1

Impacted Files
Coverage Δ

pkg/server/adapter_manager.go
63.33% <0.00%> (-7.04%)
:arrow_down:

pkg/filter/metric/metric.go
34.21% <100.00%> (ø)

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 bcc678d...2656ff0. Read the comment docs.

Good job


feat: format luxon timezones to their tz identifier

Sentry requires the TZ timezone to be used
I have made an issue with Sentry to see if its something that they would implement. However, a staff member was unsure about it, thus suggesting that it would be something we ourselves would have to deal with.
The issue can be found here: https://github.com/getsentry/sentry-javascript/issues/13820

@kings-world pack

@kings-world pack

@kings-world pack

@kings-world pack

@kings-world pack

@kings-world pack

@kings-world pack

@kings-world pack

@kings-world pack


queryInvoiceData bug?

Hi!
I tried get a invoice with queryInvoiceData method.
I got a SimpleXML object, but the invoiceData property is a base64 encoded XML string.
I need decode manually :
simplexml_load_string(base64_decode($invoiceDataResult->invoiceData));
It's bug or feature?

Hi,
this library doesn't do too much post-processing on the response data (above handling errors), it just returns the whole or the relevant part of the response XML, what is received from NAV.
Personally I have not used this method yet, but indeed, it would be a nice feature to (optionally?) parse the $invoiceDataResult->invoiceData and return as an XML object. However for that we also have to check the $invoiceDataResult->compressedContentIndicator field and apply decompressing on the data if it's needed.
PRs are welcome :) but next month I also planning to work on this topic, since the 3.0 API is here, so I may implement this feature as well.

I've just added this feature in the version 2.0.4: https://github.com/pzs/nav-online-invoice/releases/tag/v2.0.4
Example: https://github.com/pzs/nav-online-invoice/blob/v2.0.4/examples/queryInvoiceData.php#L30-L34


[iOS] monitorCharacteristicForDevice fails with error Characteristic 0000ffe1... notify change failed for device 4221AED1... and service 0000ffe0... on sensor tag

Expected Behavior
To be able to subscribe using monitorCharacteristicForDevice() as expected on iOS.
Current Behavior
On iOS trying to monitor the button pressed characteristic on an Sensor Tag/iTAG fails returning the following BleError: Characteristic 0000ffe1... notify change failed for device 4221AED1... and service 0000ffe0...
Context
The strangest thing is that the very same code DOES WORKS perfectly well on Android, being able to observe Sensor Tag/iTag button presses without getting any issues.

Library version: 2.0.0
Platform: iOS
Formatted code sample or link to a repository:

const subscription = bleManager.monitorCharacteristicForDevice(
  deviceId,
  serviceUuid,
  characteristicUuid,
  (error, result) => {
    if (error) {
      console.log("ERROR ON MONITOR =", error);
      return;
    }
    console.log("MONITORED =", result.value);
  }
);

Am I missing some extra step on iOS that is not required on Android?
Thanks in advance!
Cheers,
Federico

I got a BleError. Now what?

@dariuszseweryn thanks for your answer!
I checked the link you shared, specifically the info regarding the iOS notification emit error but if there is an error with the CCCD wouldn't that also affect the Android build?
On the other hand I tried to subscribe to the onDisconnected as suggested too but I couldn't find any clue since the device is not actually being disconnected while the error is thrown.
So is there anything else I can do to overcome this issue? I mean it shouldn't be impossible to monitor a sensor tag button from iOS isn't it? Specially taking into account that the very same code works perfectly well on Android...
Cheers,
Federico

I checked the link you shared, specifically the info regarding the iOS notifications emit error but if there is a problem with the CCCD, wouldn't that also affect the Android build?

Not necessarily — there are some platform nuances. Without logs as per issue template I cannot do much.

@dariuszseweryn how can I get more logs?
Btw I tried an iOS app for sensor tags and using the same device I was able to detect the button press on it: https://apps.apple.com/us/app/itag-one-bluetooth-key-finder/id1476723114
So I guess there must some missing some setup/option/something on my end while using your lib in order to monitor the very same sensor tag.
If you can pinpoint at me how to retrieve more log about what could be going wrong would be very helpful.
Thanks,
Federico

The issue template which, you ignored, describes how to get logs. setLogLevel(Verbose) and get the native logs.
or at least you could paste the printed error as per what I gave you in my first answer to your post

@dariuszseweryn enabling native logs as per your advise reported this:
error: Optional(Error Domain=CBATTErrorDomain Code=10 "The attribute could not be found." UserInfo={NSLocalizedDescription=The attribute could not be found.}))
Unhandled error happened: Characteristic notify change error has occured: The attribute could not be found.
 subscription called from:

2020-05-06 18:23:09.072 [info][tid:com.facebook.react.JavaScript] 'ERROR ON MONITOR =', [BleError: Characteristic 0000ffe1-0000-1000-8000-00805f9b34fb notify change failed for device 4221AED1-.... and service 0000ffe0-0000-1000-8000-00805f9b34fb]

Any idea?
Thanks!

im seeing it on 1.1.0

@dariuszseweryn @corymsmith @bitcrumb would you please provide some guidance on this issue?

Could you please provide logs? Print all attributes your peripheral has? It looks like your peripheral does not have characteristic you expect

@dariuszseweryn after looping through all the services and characteristics of the iTag by using discoverAllServicesAndCharacteristics() on iOS I get the following attributes:
SERVICE 1
{
  "_manager": {
    "_activePromises": {},
    "_activeSubscriptions": {
      "1": [Object],
      "6": [Object]
    },
    "_errorCodesToMessagesMapping": {
     ...
    },
    "_eventEmitter": {
      "_nativeModule": [Object],
      "_subscriber": [EventSubscriptionVendor]
    },
    "_uniqueId": 9
  },
  "deviceID": "4221AED1-WWWW-XXXX-YYYY-ZZZZZZZZZZZZ",
  "id":<PHONE_NUMBER>4,
  "isPrimary": true,
  "uuid": "0000180f-0000-1000-8000-00805f9b34fb"
}
|
--> CHARACTERISTIC 1
    {
        "_manager": {
        "_activePromises": {},
        "_activeSubscriptions": {
            "1": [Object],
            "6": [Object]
        },
        "_errorCodesToMessagesMapping": {
        ...
        },
        "_eventEmitter": {
            "_nativeModule": [Object],
            "_subscriber": [EventSubscriptionVendor]
        },
        "_uniqueId": 10
        },
        "deviceID": "4221AED1-WWWW-XXXX-YYYY-ZZZZZZZZZZZZ",
        "id":<PHONE_NUMBER>4,
        "isIndicatable": false,
        "isNotifiable": true,
        "isNotifying": false,
        "isReadable": true,
        "isWritableWithResponse": false,
        "isWritableWithoutResponse": false,
        "serviceID":<PHONE_NUMBER>4,
        "serviceUUID": "0000180f-0000-1000-8000-00805f9b34fb",
        "uuid": "00002a19-0000-1000-8000-00805f9b34fb",
        "value": null
    }

SERVICE 2
    {
    "_manager": {
        "_activePromises": {
        "11": [Function anonymous]
        },
        "_activeSubscriptions": {
        "1": [Object],
        "6": [Object]
        },
        "_errorCodesToMessagesMapping": {
        ...
        },
        "_eventEmitter": {
        "_nativeModule": [Object],
        "_subscriber": [EventSubscriptionVendor]
        },
        "_uniqueId": 11
    },
    "deviceID": "4221AED1-WWWW-XXXX-YYYY-ZZZZZZZZZZZZ",
    "id":<PHONE_NUMBER>8,
    "isPrimary": true,
    "uuid": "00001802-0000-1000-8000-00805f9b34fb"
    }
    |
    --> CHARACTERISTIC 1
        {
            "_manager": {
            "_activePromises": {},
            "_activeSubscriptions": {
                "1": [Object],
                "6": [Object]
            },
            "_errorCodesToMessagesMapping": {
                ...
            },
            "_eventEmitter": {
                "_nativeModule": [Object],
                "_subscriber": [EventSubscriptionVendor]
            },
            "_uniqueId": 12
            },
            "deviceID": "4221AED1-WWWW-XXXX-YYYY-ZZZZZZZZZZZZ",
            "id":<PHONE_NUMBER>4,
            "isIndicatable": false,
            "isNotifiable": true,
            "isNotifying": false,
            "isReadable": false,
            "isWritableWithResponse": true,
            "isWritableWithoutResponse": true,
            "serviceID":<PHONE_NUMBER>8,
            "serviceUUID": "00001802-0000-1000-8000-00805f9b34fb",
            "uuid": "00002a06-0000-1000-8000-00805f9b34fb",
            "value": null
        }

SERVICE 3
    {
      "_manager": {
        "_activePromises": {
          "13": [Function anonymous]
        },
        "_activeSubscriptions": {
          "1": [Object],
          "6": [Object]
        },
        "_errorCodesToMessagesMapping": {
         ...
        },
        "_eventEmitter": {
          "_nativeModule": [Object],
          "_subscriber": [EventSubscriptionVendor]
        },
        "_uniqueId": 13
      },
      "deviceID": "4221AED1-WWWW-XXXX-YYYY-ZZZZZZZZZZZZ",
      "id":<PHONE_NUMBER>4,
      "isPrimary": true,
      "uuid": "0000ffe0-0000-1000-8000-00805f9b34fb"
    }
    |
    --> CHARACTERISTIC 1
        {
            "_manager": {
            "_activePromises": {},
            "_activeSubscriptions": {
                "1": [Object],
                "6": [Object]
            },
            "_errorCodesToMessagesMapping": {
                ...
            },
            "_eventEmitter": {
                "_nativeModule": [Object],
                "_subscriber": [EventSubscriptionVendor]
            },
            "_uniqueId": 14
            },
            "deviceID": "4221AED1-WWWW-XXXX-YYYY-ZZZZZZZZZZZZ",
            "id":<PHONE_NUMBER>2,
            "isIndicatable": false,
            "isNotifiable": true,
            "isNotifying": false,
            "isReadable": true,
            "isWritableWithResponse": false,
            "isWritableWithoutResponse": false,
            "serviceID":<PHONE_NUMBER>4,
            "serviceUUID": "0000ffe0-0000-1000-8000-00805f9b34fb",
            "uuid": "0000ffe1-0000-1000-8000-00805f9b34fb",
            "value": null
        }

Out of which the latest characteristic (UUID = 0000ffe1-0000-1000-8000-00805f9b34fb) is the one I expect to use and it's failing with the reported error:
[BleError: Characteristic 0000ffe1-0000-1000-8000-00805f9b34fb notify change failed for device 4221AED1-WWWW-XXXX-YYYY-ZZZZZZZZZZZZ and service 0000ffe0-0000-1000-8000-00805f9b34fb]

Thanks,
Federico

Could you also add info about descriptors?

@dariuszseweryn there aren't...

@dariuszseweryn do you have any update on this?

Having no Client Characteristic Configuration Descriptor is violation of Bluetooth Spec — not sure if iOS in recent versions did show all descriptors. I recall that iOS has been working correctly even for characteristics without CCCD. What phone/OS version you test against? Could you also print the error using JSON.stringify(error)?

Having no Client Characteristic Configuration Descriptor is violation of Bluetooth Spec — not sure if iOS in recent versions did show all descriptors. I recall that iOS has been working correctly even for characteristics without CCCD. What phone/OS version you test against? Could you also print the error using JSON.stringify(error)?

@dariuszseweryn but if that's the case it shouldn't work on the other iOS app I mentioned here:
https://github.com/Polidea/react-native-ble-plx/issues/657#issuecomment-624005624
given I'm using the same iOS device and BLE sensor tag... isn't it?
BTW I'll provide the other details you requested me later on during this week...

@federicomiralles  did you solve it? i am facing the same issue.
{"message":"Characteristic 000036f6-0000-1000-8000-00805f9b34fb notify change failed for device A and service 0000fee7-0000-1000-8000-00805f9b34fb","errorCode":403,"attErrorCode":10,"iosErrorCode":null,"androidErrorCode":null,"reason":"The attribute could not be found.","name":"BleError"}

Same issue [BleError: Characteristic 00002a00-0000-1000-8000-00805f9b34fb notify change failed for device ? and service 00001800-0000-1000-8000-00805f9b34fb]

Same issue [BleError: Characteristic 00002a00-0000-1000-8000-00805f9b34fb notify change failed for device ? and service 00001800-0000-1000-8000-00805f9b34fb]

Has anyone solved this?

anyone fixed this?

@Salva222 very bizzare, I'm hitting the same on ios.
00002a00-0000-1000-8000-00805f9b34fb is not showing up, and I don't know why.
My formatter prints:
formatted BLE error string for human consumption: Error name: "BleError", error message: "Characteristic 00002a00-0000-1000-8000-00805f9b34fb not found", error code: "404", error reason: "null", error code (IOS specific): "null", error code (android specific): "null", error code (ATT specific, whatever that is): "null"

@ariccio
for me, the problem was that I tried to "write with a response" to a characteristic that is "without response" and for some reason android let me do it, but ios did catch my error.
I suggest you use some Bluetooth test app like "nFR Connect" to see what your services and characteristics.

And now I can't read it through other apps?!

Ah, it appears, very strangely, that apple does not expose the GENERIC_ACCESS bluetooth service for some reason.
What's odd, is I thought I saw a bluetooth diagnostic app reading from it yesterday.  I'm wondering if it's worth the MultiPlatformBleAdapter to note this in the docs somewhere?


mdast writes an empty line before EOF

Apparently when I generate a file using mdast, it has two \ns before EOF, one would be better :)

When piping or when using -o?

I'm piping, using it via a Bear in coala.

Shouldn't make a difference should it?

(Note that we capture stderr in an extra stream so it's only the stdout output)

It does, because I opted both to behave differently. For example, all information and warnings are passed to stderr(4)).
When a user is writing to a TTY, I prefer an extra newline for things to look better, but as mentioned before that should’ve been stderr(4). I’ll investigate.


chroot: set exit codes to 125, 126 or 127 for errors from chroot itself

Implements this part of the GNU docs:
   Exit status:
     125 if ‘chroot’ itself fails
     126 if COMMAND is found but cannot be invoked
     127 if COMMAND cannot be found
     the exit status of COMMAND otherwise

I think we can only reliably test the the 125 case, and not the 126 and 127 cases, but if someone has ideas about how to do that I'd be happy to add it.

CI failures seem to be unrelated


Add Pre-commit Hooks for Code Formatting and Linting

To ensure consistent code style and formatting across the entire repository, we need to implement pre-commit hooks that automatically handle formatting and linting for multiple file types. This will improve code quality, catch errors early, and maintain clean code standards.
File Types to be Covered:

Python files (.py)
Jupyter Notebooks (.ipynb)
YAML files (.yml)
Markdown files (.md)
Text files (.txt)

Tasks:

Add a .pre-commit-config.yaml file in the root directory of the repository.
Configure the following pre-commit hooks:

Black: For automatic code formatting for Python and Jupyter Notebooks.
Flake8: For linting Python files and Jupyter Notebooks.
nbQA: To apply Black and Flake8 formatting and linting to Jupyter Notebooks (.ipynb).
Check YAML: To validate the format of YAML files.
Markdownlint: To ensure style consistency in Markdown files.
Trailing Whitespace, End-of-file fixer, and Check-merge-conflict: For cleaning up text files and ensuring proper file formatting.

Install and activate the pre-commit hooks with the pre-commit install command.
Run pre-commit run --all-files to apply the hooks to the entire codebase.

By implementing these hooks, we can maintain consistent formatting across all relevant file types and automatically prevent common formatting issues before code is committed.

Hey @Devanik21, I have already submitted the PR #79 to cover this.
Please crosscheck the pre-commit hooks i have added.

i would like to contribute @Devanik21


Get Client's IP Address

We need this feature for more than a few reasons but it isn't part of the Request model. I think we should just jam it on Server Context probably as clientAddress. Luckily others have gone through this: https://github.com/withastro/astro/issues/3708
So we should add this.

@ryansolid this commit breaks types. see https://github.com/solidjs/solid-start/actions/runs/3917523573/jobs/6697359050
previous commit was green


Bump to version 5.1.5

@stefanpenner, what happened with 5.1.4? There seems to be a published 5.1.4, but the last revision in the repo was 5.1.3. Is it OK to merge this and publish a new revision or will that conflict with changes in Ember CLI?

@gordonkristan - Ya, lets merge and release.  I'm thinking that @stefanpenner forgot to push his version number update commit.


Heatmap ground truth range

The code  heatmaps[i] /= maxi / 255 shows that the maximun value of gt heatmap should be scaled to 255. Is there any reasons for this operation? I found that most projects use [0,1] for heatmap range.

Yes, this operation processes the 0-255 range heatmap to [0,1], which is the same as other works


Custom Shaders

Hi @tchayen! Really interesting project, thanks for working on it.
I've briefly checked some examples and would like to know if there's a way to inject some custom shaders in order to apply some postprocessing after the base layout has been drawn? If not, what do think about this feature? Are there any plans to work on it in the future?

Hi! Currently there's no way to do that. But…
I am working on a big rewrite of almost everything (a bit like version 2.0 but in this case from 0.0.16 it will become uhm, 0.1?). I haven't made any plans to add custom shaders yet but I think it makes a lot of sense. I will keep thinking about it and hopefully when I am done with other things I will have some good idea for it.
Thanks for bringing this up, it's a great idea!

Cool! Looking forward to this v0.1 then. Thanks again for working on it!


how can we append np array in jit

@jit(nopython=True)
def sma(x,d):
	'''
	simple moving average
	x: the vector
	d: the delay window
	'''
	n = len(x)
	x_ = np.lib.stride_tricks.as_strided(x,shape=(n-d+1,d),strides=(x.strides[0],x.strides[0])) #same as view_as_windows
	#ret = np.einsum('ij->i',x_)/d #np.einsum is not supported by numba
	ret = x_.sum(1)/d
	return ret

sma will return a vector of lenght n-d+1, but if I want to return a vector of length n :
@jit(nopython=True)
def sma(x,d):
	'''
	simple moving average
	x: the vector
	d: the delay window
	'''
	n = len(x)
	x_ = np.lib.stride_tricks.as_strided(x,shape=(n-d+1,d),strides=(x.strides[0],x.strides[0])) #same as view_as_windows
	#ret = np.einsum('ij->i',x_)/d #np.einsum is not supported by numba
	ret = x_.sum(1)/d
	ret = np.append(ret,np.zeros(d-1))
	return ret

the second function will give the error:
TypingError: Use of unsupported NumPy function 'numpy.append' or unsupported use of the function.

File "functions.py", line 26:
def sma(x,d):
    <source elided>
	ret = x_.sum(1)/d
	ret = np.append(ret,np.zeros(d-1))
 ^

So how can we extend a np.array in the jit .
numba.__version__
Out[674]: '0.44.0'
python 3.6
spyder 3.3.4

thank you so much

Thanks for the report. I think that the np.append use in the above is the same as concatenate with ravel?
In [14]: a = np.arange(9).reshape((3, 3))

In [15]: b = np.arange(5)

In [16]: np.append(a, b)
Out[16]: array([0, 1, 2, 3, 4, 5, 6, 7, 8, 0, 1, 2, 3, 4])

In [17]: np.concatenate((a.ravel(), b.ravel()))
Out[17]: array([0, 1, 2, 3, 4, 5, 6, 7, 8, 0, 1, 2, 3, 4])

please open a feature request if support for np.append is important to you. Thanks.


[FEAT]: Custom SERP entry

What would you like to see?
Can we get the option to add our own custom SERP to the list of providers?
We do not want existing SERP's to have/use/sell/track our search data and so have built our own SERP internally.
Our custom SERP can use a VPN to search any site, grab results and format it in the appropriate manner.

IMO this is better under a fit as a custom tool/integration

Also, moving/linking the conversation to #1327


Output files are extrodinarily large, should further compress during creation

For each lecture having 50-100 slides, many of the files are taking around 50MB to 200MB.
Tried using ghostscipt to compress the file, which would increase the file size instead.
At last I am settled with imagemagick's convert, as suggested from here link.
convert -density 200x200 -quality 60 -compress jpeg input.pdf output.pdf
This made the files ~10x smaller than original.

Thank you for your solution.
The size of the slide image on the web page is not fixed, so the plug-in will retain its original size and use lossless compression to ensure image quality.
Maybe the slide properties or device compatibility are different. I didn't find any problems during the test.
At present, I prefer to use special tools (like ImageMagick you mentioned) to compress PDF after downloading files.
It is a challenge to implement pdf compression and balance quality and size on the front end.


Wrong GitHub tag is shown

[This](https://github.com/HaxeFlixel/flixel uses ) repo uses this badge:

It shows the latest version as 3.3.9, even though the latest tag / most recent version is 3.3.11.
I suspect this might be caused by the double digit patch number.

Thanks for raising this!
One of the tags is 1.09, which is not semver compliant. Since we can't use semver ordering, we fall back on to the next best thing, ASCII ordering. It is bad, but dealing with arbitrary complex versionning schemes (as are found in the wild) is hard. If you can think of a better algorithm, it would be welcome!

Ah, I see... that's from before we followed semver.
In this case (and most cases I'd imagine?) simply using the most recent tag would work just fine. That's how I thought it works until now, actually.
Different urls to deal with different versioning schemes would be another option.. tag/latest, tag/semver etc.. that might overcomplicate it though?

simply using the most recent tag would work just fine

Hehe, it worked that way until someone told us that they had the wrong version on their badge… They had made a 3.2.14, then a 3.3.0, then a 3.3.1, then performed a bugfix patch tagged 3.2.15.
The use of a different URL is an interesting idea. It is an issue for all badges dealing with versions, though.

We have the same issue here: ![Latest Release](https://img.shields.io/github/tag/robotology/yarp.svg?label=Latest Release)
We have a tag start that is probably causing the issue.
Would it be possible to add a list of tags to skip to the URL? i.e. something like
https://img.shields.io/github/tag/robotology/yarp.svg?label=Latest Release&skip=start;foo;bar

@drdanz We filter out things that don't look like a version. In your case, you have a non-semver v<IP_ADDRESS>, which throws off the semver detection, which makes it fall back to ASCII comparisons.

I intend to switch to a simplified three-tier version analysis: semver / a custom dotted-version comparator / ASCII. The ASCII tier will no longer have non-versions filtered out, while the new custom dotted-version will use integer list comparison (and trailing data ASCII comparison for the -alpha and the like). Initial testing shows that Flixel and Yarp will be fixed by that new design. Versions that follow a different convention than dots (eg, v1-3-10) won't be fixed, and will be handled by the third tier.
(It is irrelevant for https://github.com/badges/shields/issues/261, which now uses the PHP algorithm. The same goes for https://github.com/badges/shields/issues/377, which uses GitHub's /latest version.)


It seems this was handled in another PR, so I'll pass on this one. But thanks for the effort! :coffee:


Jmock 2.6.0 not compatible with JUnit 4.11 due mixed dependencies on hamcrest versions

Solved in 2.8.0, in maven central now.


"no such column" error was thrown when query with predicate created on related model

Describe the bug
Taking schema:
enum PostStatus {
  ACTIVE
  INACTIVE
}

type Post @model {
  id: ID!
  title: String!
  rating: Int!
  status: PostStatus!
  # New field with @hasMany
  comments: [Comment] @hasMany(indexName: "byPost", fields: ["id"])
}

# New model
type Comment @model {
  id: ID!
  postID: ID! @index(name: "byPost", sortKeyFields: ["content"])
  post: Post! @belongsTo(fields: ["postID"])
  content: String!
}

Execute to create post, comment and associate them.
let post = Post(title: "Test post", rating: 10, status: PostStatus.active)
let comment = Comment(post: post, content: "Test comment");

Amplify.DataStore.save(post) { result in
    switch(result) {
    case .success(let savedPost):
        print("saved post: \(savedPost)")
        Amplify.DataStore.save(comment) { result in
            switch(result) {
            case .success(let savedComment):
                print("saved comment: \(savedComment)")
            case .failure(let error):
                print("save comment error: \(error)")
            }
        }
    case .failure(let error):
        print("save post error: \(error)")
    }
}

All good, then run query against the Comment model.
Notice that the predicate is Post.keys.status.eq(PostStatus.active).
Amplify.DataStore.query(
    Comment.self,
    where: Post.keys.status.eq(PostStatus.active)
) { result in
    switch(result) {
    case .success(let comments):
        print("comments: \(comments)")
    case .failure(let error):
        print("query error: \(error)")
    }
}

Below error was thrown
no such column: root.status in "select
  "root"."id" as "id", "root"."content" as "content", "root"."createdAt" as "createdAt",
  "root"."updatedAt" as "updatedAt", "root"."postID" as "postID", "post"."id" as "post.id",
  "post"."createdAt" as "post.createdAt", "post"."rating" as "post.rating", "post"."status" as "post.status",
  "post"."title" as "post.title", "post"."updatedAt" as "post.updatedAt"
from Comment as "root"
inner join Post as "post"
  on "post"."id" = "root"."postID"
where 1 = 1
  and "root"."status" = ?"

Where the predicate on the inner join seems wrong
and "root"."status" = ?"

should be as below?
and "status = ?"

NOTE: The same query predicate works in amplify-android without issues.
Attaching amplify-android generated SQL command for the same query:
SELECT `comment`.`id`      AS `Comment_id`,
       `comment`.`content` AS `Comment_content`,
       `comment`.`postid`  AS `Comment_postID`,
       `post`.`id`         AS `Post_id`,
       `post`.`rating`     AS `Post_rating`,
       `post`.`status`     AS `Post_status`,
       `post`.`title`      AS `Post_title`
FROM   `comment`
       INNER JOIN `post`
               ON `comment`.`postid` = `post`.`id`
WHERE  status = ?;

Steps To Reproduce
Steps to reproduce the behavior:
1. Follow the code example in description
2. Observe the error

Expected behavior
The query should works as the same as how it works in amplify-android.
Amplify Framework Version
1.18.1
Amplify Categories
DataStore
Dependency manager
Swift PM
Swift version
5
CLI version
7.6.5
Xcode version
13.2.1
Relevant log output
2021-12-22 12:17:57.279690-0800 v2Relations[64393:1642091] [logging] no such column: root.status in "select
  "root"."id" as "id", "root"."content" as "content", "root"."createdAt" as "createdAt",
  "root"."updatedAt" as "updatedAt", "root"."postID" as "postID", "post"."id" as "post.id",
  "post"."createdAt" as "post.createdAt", "post"."rating" as "post.rating", "post"."status" as "post.status",
  "post"."title" as "post.title", "post"."updatedAt" as "post.updatedAt"
from Comment as "root"
inner join Post as "post"
  on "post"."id" = "root"."postID"
where 1 = 1
  and "root"."status" = ?"

Is this a regression?
No
Regression additional context
No response
Device
iPhone 13 simulators
iOS Version
15.2
Specific to simulators
No response
Additional context
No response

This bug is affecting us and it is currently a major blocker. Is there an eta on when this will be resolved?

We are currently experiencing this bug as well and it's a major blocker for using Amplify.

It's been quite for a long here, I have the same error with the same schema models configuration but bit simple in terms of absence of the relation 1-to-many, just the field that stores and id of another model

Although this works on Android, i'm not sure other platforms allow the same. iOS currently has not implemented or tested this use case:
Amplify.DataStore.query(
    Comment.self,
    where: Post.keys.status.eq(PostStatus.active)
)

There are some designs around nested predicates which will allow this use case to work, ie. comments.post.status.eq(PostStatus.active), thus this is more of a feature request


Update Configuration.php

In the 4.2 Symfony version the constructor of TreeBuilder requires to enter the name parameter.
After this modify I resolve the error  ArgumentCountError:
Too few arguments to function Symfony\Component\Config\Definition\Builder\TreeBuilder::__construct(), 0

Duplicate with #431

Why is this not merged yet? It fatal blocker for Symfony 5

See #411 (with BC).

This is just mergin PRs and keeping quality. I do it for many dead repos so I'd be happy to do it here. But there is actually no activity with current maintainers in this PR, so I guess there is no way to make it happen.

Replaced by #411, now merged
Thank you for your work on this PR!


Test copy and move semantics

Description
One of SFML 3's specific goals is to support move semantics. One such obstacle to implementing move semantics is not necessary knowing when the compiler is or is not automatically generating the move special member functions. By using <type_traits> we can test to make sure that at a minimum, the special member functions we expect do exist.
These tests will let us catch regressions. If a destructor is added to a class or a member variable is marked const, move semantics are implicitly deleted. These assertions will immediately catch that bug and force the developer to reckon with the bug they have introduced.
These tests also serve to document what types are still lacking move semantics. Search the codebase for static_assert(!std::is_nothrow_move and you will find the tests for all types that fail to implement a move constructor and/or move assignment operator. Once you add move semantics, the test will fail at which point you remove the ! and you're good to go.
By testing copy and move semantics on all remaining SFML types, this PR also knocks out a lot of the boilerplate code required to write additional tests. Future developers will have an easier time improving the test suite when the test file already exists for the file they're testing. They can easily start writing code in that file and see it run which will be quite handy and hopefully encourage writing more tests.
Beyond testing for the existence of move semantics, these tests also assert the lack of copy semantics for types which are non copyable. Because one cannot write a test using a deleted function, this method is the only way to assert that a type is non copyable.
These tests are easily expanded to test for any number of relevant type traits. For example if a type promises to not be constructible or to be convertible from another type or to be abstract, we can write a test for that ensuring that those promises are never broken.

Why don't you just write tests that validate that the move assignment operator and move constructor are correct and skip using type traits?

I already mentioned non copyable types as a justification for testing with type traits in some capacity. As for testing the implementation of move operators, yes that is a good idea and something I'd like to eventually do. However it can be quite difficult (perhaps infeasible for some types) and we're yak shaving so let's take things one step at a time. Besides, most types have implicitly defined move semantics which we know are going to be correct so I don't think it's worth anyone's time to write those tests.

What about putting these assertions in the classes themselves?

I don't hate this idea but let me explain why keeping the assertions in the tests is a better idea. First, these assertions are tests. Just because we can put them in the core library doesn't mean that makes sense. These assertions are only relevant to us developers. If you're merely using the library you do not care about compiling these static asserts. On top of that, putting these assertions in class templates means adding a new transitive include to all users of SFML. This is not a huge cost but it's certain a non-zero cost that we and all SFML users would be incurring.

In this branch I've started enabling move semantics for more types. As expected, the tests detect that move semantics now exist.

Refactored the PR to avoid creating a SUBCASE that contains no runtime code. This means some entire TEST_CASEs go away which has the added benefit of minimizing how often we have to compile doctest.h. We still benefit from the existence of files like test/System/String.cpp so that the boilerplate is already in place for someone to eventually write runtime tests for that type and many others lacking tests.

SFMLBuildMaster: Build this please.

Rebased and fixed merge conflicts

SFMLBuildMaster: Build this please.

Rebased on master and fixed merge conflicts.

SFMLBuildMaster: Build this please.

One last concern before I can proceed: I see that in some tests you check for !is_copy_constructible and !is_nothrow_move_constructible, but you don't check for !is_move_constructible.

I've work been working under the assumption that is_nothrow_move_constructible implies is_move_constructible. If that is not true then I'll need to reevaluate how I'm testing move semantics.

Rebased on master, switched to the _v version of these type traits, and added addition type traits for types which are moveable but not nothrow movable. This is a weird distinction but some types with copy semantics are "movable" in the sense that calling std::move just makes a copy but they're not nothrow movable which means we still have some work to do if we want the performance gains that moving can give us.

SFMLBuildMaster: Build this please.

Rebased to catch up with master. No changes required.

SFMLBuildMaster: Build this please.

@vittorioromeo are the changes what you expected?

Fixed conflicts with master.

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

Fixed conflicts with master.


Eval app improvements

This PR contains the following fixes and improvements:

Updated yaml configuration file with more relevant extraction methodologies to benchmark
Added try catch in unstructured API to prevent Python error in case no table was extracted
Updated eval README and moved data files to run streamlit app to eval/data/ folder to get started more easily
Added eval dependencies to poetry
Eval app improvements (see below)

Eval app improvements

Moved PDF and REF data from right to left tabs for better clarity
New PDF viewer allowing to zoom in/out
Display message when no table got extracted
Fixed display issue for dataframes with multi-row headers (convert header to single row)
Fixed display issue for dataframes with empty headers (fill empty header)
Help tips added

Also @guillaume-millot does it make sense to check the files you sent me by slack in the repo in some well known location for the eval app?  It would make it easier for someone to get started.

Amazing work! You have some pre-commit issues and failing tests to check out.
Don't forget to install the pre-commit git hooks so that you technically can't commit to git before fixing this type of issue.

There are 2 precommit issues:

Test test/test_pdf_downloader.py::TestCBCRScript::test_download_pdf is failing. It seems the issue got introduced by this commit https://github.com/dataforgoodfr/12_taxobservatory/commit/ef3106348cdf817ed47d21ef22ecf4bbd3cd75f8
Vulnerability found in langchain version 0.1.16

I don't think it hurts to add the extra dependencies for the streamlit app directly in poetry.
Ok I'll do that before merging.

I would also merge the eval README with the main README . The eval app should be a first class citizen IMHO.
Great point, but the main README currently is already really long. Let's keep this in mind for the future?

Also @guillaume-millot does it make sense to check the files you sent me by slack in the repo in some well known location for the eval app?
Ok

There are 2 precommit issues:

Test test_pdf_downloader.py::TestCBCRScript::test_download_pdf is failing. It seems the issue got introduced by commit ef31063
Vulnerability found in langchain version 0.1.16

@kaaloo would you know how to fix 2? Re: 1, I'll create an issue so it can be fixed separately if you're ok with that!

@guillaume-millot  since these are pre-commit issues, you will have to skip the failing test by adding a pytest skip decorator.  You can see if updating langchain fixes the vulnerability:
poetry add langchain@latest

I skipped the test and will create an issue.
Running poetry add langchain@latest bumped langchain version to 0.1.17 but the vulnerabilities are still there :(
I added the missing dependencies to poetry and moved the REF CSV & sample pickle file to a new eval/data directory to make it easier to get started (great suggestion!).
@kaaloo are you ok to merge?

I've just added args: ["--ignore=66962"] per our conversation on slack.


Linking error when using tflite::StatefulNnApiDelegate

Click to expand!
Issue Type
Build/Install
Source
source
Tensorflow Version
2.10
Custom Code
No
OS Platform and Distribution
Linux Ubuntu 20.04 - Android Studio for building
Mobile device
Android ARM64
Bazel version
5.1.1
Current Behaviour?
Hi! I am trying to add a delegate to the interpreter from the C++ native side, but am unable to figure out how to make it work. Any help would be greatly appreciated!
I can successfully build the model, interpreter and tensors needed for inference, but attempting to initialize tflite::StatefulNnApiDelegate leads to a linking error when trying to build the apk
C/C++: ld: error: undefined symbol: tflite::StatefulNnApiDelegate::Data::~Data()
What confuses me is that all other tflite functions and objects I needed have been found, I can even declare tflite::StatefulNnApiDelegate::Options without issue.

Am I doing something wrong in the code, from the examples I could find, and looking at the jni code for the java side I believe the code provided should be correct. Could I be missing something completely, like querying for the Nnapi or setting something in the options before creating the delegate?

Could it be an issue happening while building libtensorflowlite.so? I followed the recipe to build it with bazel for ARM chips Build TensorFlow Lite for ARM boards with the command:

bazel build --config=android_arm64 -c opt //tensorflow/lite:libtensorflowlite.so

and also tried
bazel build -c opt --config=android_arm64 //tensorflow/lite/delegates/nnapi:nnapi_delegate

to no avail...
I also included the CMakeLists.txt to make sure I am not doing something wrong there.
Thank you in advance!
Standalone code to reproduce the issue
Failing code.
//! 1) build model
model_ = tflite::FlatBufferModel::BuildFromBuffer(model_buffer_,
                                                  sizeof(char) * model_size_);

//! 2) resolving model
tflite::ops::builtin::BuiltinOpResolver resolver;
tflite::InterpreterBuilder builder(*model_, resolver);

//! 3) create NNApi delegate
tflite::StatefulNnApiDelegate::Options options = tflite::StatefulNnApiDelegate::Options();
tflite::StatefulNnApiDelegate nnapi_delegate(options); // fails with ld error
builder.AddDelegate(&nnapi_delegate);

//! 4) link to interpreter
builder(&interpreter_);
assert(interpreter_ != nullptr);

CMakeLists.txt
cmake_minimum_required(VERSION 3.4.1)

set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=gnu++17")

# Specify where to find the header files for TF Lite C++
set(TF_LITE_DIR ${CMAKE_CURRENT_SOURCE_DIR}/../../../../tflite-api)
set(INCLUDE_DIRS
        ${TF_LITE_DIR}/include
        ${TF_LITE_DIR}/include/flatbuffers/include)
include_directories(
        ${INCLUDE_DIRS})

add_library(tflite SHARED IMPORTED )
set_target_properties(tflite PROPERTIES IMPORTED_LOCATION
        ${TF_LITE_DIR}/lib/${ANDROID_ABI}/libtensorflowlite.so )

add_library(native-lib SHARED
            native-lib.cpp
            native-tf.cpp)

target_include_directories(native-lib
        PUBLIC
        ${CMAKE_CURRENT_SOURCE_DIR})

target_link_libraries(native-lib
        android
        camera2ndk
        mediandk
        log
        tflite
        )

Hi @NicoZobernig !
Sorry for the late respone.
I see you are using CMake 3.4.1  version .
Can you check with CMake 3.14 or hight and post the LD error too.
Thank you!

Hi @mohantym ! Thank you for looking into this.
I am actually using CMake 3.18.1, setting it in the build.gradle file. The minimum version is just a remains from android-studio's template as far as I remember...
So the LD error is the same.
I also tried linking neuralnetworks in target_link_libraries to see if that would change anything:
target_link_libraries(native-lib
        android
        neuralnetworks
        camera2ndk
        mediandk
        log
        tflite
        )

It didn't...
I can make it work is by using the C API instead after getting libtensorflowlite_jni.so from the packaged .aar file provided here.
So my assumption is that the building procedure of the C++ API library is at fault. I cannot say whether the build files have an issue or if I did something wrong there.

@NicoZobernig !
Thanks for sharing your observations.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The TFLite team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the TFLite space.


64bit dll crashes pcsx2

I am trying to figure out what causing this crash no errors or nothing just quits the program so there's a bug somewhere

pcsx2 is a 32bit program. 64bit won't work.


Fix syntax highlighting in README.md

I really hope that the bot is fixed soon. Not only it spams here, but also spams with emails if you have email notifications enabled. I mentioned previously that I would be happy to help with fixing the bot, but it doesn't look like the bot is open-source or if there's any other way to help.
This currently gives an impression that the infrastructure is broken and is not maintained, and maye also be off-putting for new contributors.

@swift-server-bot please test this

@swift-server-bot test this please

Thanks for the PR and comment @ MaxDesiatov. cc @apple/swift-server-admins

@yim-lee Thanks for merging and thanks for the great library!


feat: block level indexing

Removes the notion of transitions and replaces them with just block numbers.
Previously, each change to a storage slot or an account were associated with a transition. A transition could either be a transaction, or a block. Each transaction would always have a transition ID, while only blocks with a block reward (pre-merge blocks) or a block with withdrawals (post-Shanghai blocks) would have transitions.
This presented a few issues:

The amount of changesets we generated were a lot. At least two changesets would be generated per transaction - one for deducting fees from the sender, and one for adding these fees to the block author. Then, an additional block level changeset would be added for the block reward.

This introduced a lot of allocation in the executor, and it cost a lot of disk space

Since transitions could either be blocks or transactions, a lot of additional index tables were needed
There was a lot of logic around figuring out whether a change was part of a block or a transaction etc.

This PR:

Removes the transition related tables, replacing the remaining references to transitions with block numbers
Changes PostState to collect changes per block instead of per transition

This gives us faster sync and less disk space usage, at the (possible) cost of some RPC tracing APIs being slower. This should be benchmarked separately, and we should try to optimize this separately as well.
Additionally this PR changes a lot of block range based APIs with RangeInclusive and does a bit of clean up on stage inputs.

Codecov Report

Merging #2275 (0f5a737) into main (1680630) will decrease coverage by 55.92%.
The diff coverage is 0.15%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #2275       +/-   ##
===========================================
- Coverage   72.05%   16.14%   -55.92%
===========================================
  Files         472      472
  Lines       56734    56354      -380
===========================================
- Hits        40878     9096    -31782
- Misses      15856    47258    +31402

Flag
Coverage Δ

integration-tests
16.14% <0.15%> (-2.24%)
:arrow_down:

unit-tests
?

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

Impacted Files
Coverage Δ

bin/reth/src/dump_stage/hashing_account.rs
0.00% <0.00%> (ø)

bin/reth/src/dump_stage/merkle.rs
0.00% <0.00%> (ø)

bin/reth/src/test_eth_chain/runner.rs
0.00% <0.00%> (ø)

bin/reth/src/utils.rs
0.00% <0.00%> (ø)

crates/consensus/auto-seal/src/lib.rs
0.00% <ø> (-0.97%)
:arrow_down:

crates/consensus/beacon/src/beacon_consensus.rs
0.00% <ø> (-24.00%)
:arrow_down:

crates/executor/src/blockchain_tree/chain.rs
0.00% <0.00%> (-62.59%)
:arrow_down:

crates/executor/src/blockchain_tree/mod.rs
0.00% <0.00%> (-89.84%)
:arrow_down:

crates/interfaces/src/consensus.rs
0.00% <ø> (-100.00%)
:arrow_down:

crates/interfaces/src/provider.rs
0.00% <ø> (ø)

... and 33 more

... and 309 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

It looks like it doesn't introduce any failures:
main: https://github.com/paradigmxyz/reth/actions/runs/4749818600
hive + feat/block_level: https://github.com/paradigmxyz/reth/actions/runs/4750295754

Took a detailed pass:

The RangeInclusive change seems to be unrelated to the core change we're doing in this PR? I find it slightly less readable myself the way it is done right now (vs how it was before), so would rather we keep it as it was, unless there was a functional reason for doing so. WDYT?

Ranges are the main thing that is affected in this PR as changes of transition are made into changes of blocks and a lot of indexing have changed.
As we are now working with blocks, from what I have seen RangeInclusive is a better option as the Range would include a lot of a..b+1.

Before merging let's ensure that this passes on Hive. cc @Rjected please run Hive against this PR.
Let's make sure that the state root value change we made in that test was just as a result of the test vectors changing, and it was not an actual bug
It got effected by merging the changesets, but the main focus that those data are used is to check that commit of block1 and block2 and revert of block2 produces the same state root, tests that confirm that merkle tree are elsewhere.
Will see if I can reuse old values.

There's a few nits I made that we don't need to address in this PR but we can pull in separate issues for 3rd party contributors (e.g. cleaning up the docs).
@rkrasiuk please run this PR on mainnet & testnets and compare performance with your node setups

@rkrasiuk wait a bit, this PR fails syncing on mainnet.

Imo it feels like there are way more + 1 and - 1 scattered all over the place than before, so I am unsure if going RangeInclusive fixed that issue?

We need to be careful here why +1 is introduced, some of +1 are for starting range that would be the same for both Range and RangeInclusive and they are introduced as we have moved from TransitionId to BlockNumber.
For end range, If we used just Range we would introduce a lot more unnecessary +1 in our code that RangeInclusive omits.
For -1, I found 3 places where it was added (mostly with count-1`), and one place where it was removed, so this seems not relevant.


Fixed the problem of white keyboard in TMP_input

When using keyboard inputs with this manifest it was white keyboard which solved the problem

OK, seems good, just please reformat the file using Tabs instead of Spaces, then I will merge your pull request and credit your contribution to this project in the thanks section of the readme.
Use Notepad++ or any other favorite text editor, in the Notepad++ go to View ->Show Symbol -> Show All character.
Thanks.


Improve configuration and add simple REST client

Coverage increased (+3.7%) to 93.496% when pulling b04c140529ef2fae52a86e7e235ee2372e840892 on maretskiy:add-simple-rest-clients into e0397281e03fa8e27da562c75afa63259bd4b741 on seecloud:master.

Coverage increased (+3.5%) to 93.548% when pulling c244229521ddb106f863b6e75caa4d608acb381b on maretskiy:add-simple-rest-clients into 578ad1512e01e1860239cb6d7a8cf8fefb6c2155 on seecloud:master.

Coverage increased (+3.5%) to 93.548% when pulling ed352d953808c5db4576c1971ade85864fc21b36 on maretskiy:add-simple-rest-clients into 578ad1512e01e1860239cb6d7a8cf8fefb6c2155 on seecloud:master.

Coverage increased (+3.5%) to 93.548% when pulling e4bf50b4c6fbfce3b1e0194ef9bca044d24a7670 on maretskiy:add-simple-rest-clients into 578ad1512e01e1860239cb6d7a8cf8fefb6c2155 on seecloud:master.

Coverage increased (+3.5%) to 93.548% when pulling b3429d2e40d5ea094de7dee1f3e2390ff2bf05bf on maretskiy:add-simple-rest-clients into a13e47baef6464d5f4fb40b75af5f4001a72c9f4 on seecloud:master.

Coverage increased (+3.5%) to 93.548% when pulling fe61b03aa5f2f9e9624e0db3c5776f374e11ddf4 on maretskiy:add-simple-rest-clients into 7e87ec7609684e7de217ffa179298cc7f08c70d1 on seecloud:master.


Archive PyCQA/pylint-pytest too

So this repository is a fork of a fork: pylint-dev/pylint-pytest <- PyCQA/pylint-pytest <- reverbc/pylint-pytest.
The original repository reverbc/pylint-pytest is archived but the middle link PyCQA/pylint-pytest is not — it just has issues disabled and a few open PRs that nobody's looking into.
That lets a lot of people land on that PyCQA repo page confused as to where's the active fork. Is it possible to poke somebody from that org to archive that repo too so that there's one obvious alive repository left?

Yeah, I figured as much, reading through the threads elsewhere now. FWIW that pylint-dev/pylint-pytest <- PyCQA/pylint-pytest <- reverbc/pylint-pytest chain is what GitHub itself shows. And when people open PRs from forks, GitHub may offer them to open a PR in one of those, not here. So archiving the mid-upstream component should steer them into the right place. Anyway.. I opened the issue in the proper upstream, I think: https://github.com/PyCQA/meta/issues/60.

I'm assuming they won't risk "removing it", especially given what happened - since this is a "Critical Project" (also according to PyPi).

My understanding is that this repo publishes to the same PyPI project so it's fine. I'm only concerned about the GH-side confusion.

I'm assuming they won't risk "removing it", especially given what happened - since this is a "Critical Project" (also according to PyPi).

My understanding is that this repo publishes to the same PyPI project so it's fine. I'm only concerned about the GH-side confusion.

So was the reverbc repo :upside:

I'm assuming they won't risk "removing it", especially given what happened - since this is a "Critical Project" (also according to PyPi).

My understanding is that this repo publishes to the same PyPI project so it's fine. I'm only concerned about the GH-side confusion.

So was the reverbc repo 🙃

Yep. Which effectively means seamless migration for the end-users that is a good thing so long that the publisher is trusted..


After first install login not possible with initial user "Incorrect credentials"

Describe the bug
docker-compose up "install" produces non working instance, no idea how to login when all i get is Incorrect credentials.
% grep -v '^#' .env
PUBLIC_POCKETBASE_URL=
<EMAIL_ADDRESS>ROOT_ADMIN_PASSWORD=test123
ORIGIN=http://localhost:5173
PUBLIC_HTTPS_ONLY=
PORT=5173
PUBLIC_SIGNUP_DISABLED=false

% docker compose up
[+] Running 2/2
 ✔ Container grimoire-pocketbase  Recreated                                                                                0.1s
 ✔ Container grimoire             Recreated                                                                                0.1s
Attaching to grimoire, grimoire-pocketbase
grimoire-pocketbase  | 2024/01/06 03:26:54 Server started at http://<IP_ADDRESS>:80
grimoire-pocketbase  | ├─ REST API: http://<IP_ADDRESS>:80/api/
grimoire-pocketbase  | └─ Admin UI: http://<IP_ADDRESS>:80/_/
grimoire             | Listening on <IP_ADDRESS>:5173
grimoire             | { invalid: true }
grimoire             | { invalid: true }

To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/goniszewski/grimoire
edit .env
docker compose up
go to http://localhost:5173/login, try to login -> invalid credentials with the pw defined from .env

Expected behavior
be able to login with the email/pw defined in .env
Screenshots

Desktop (please complete the following information):

OS: debian sid
Browser chrome
Version 120.0.6099.129 (Official Build) (64-bit)

Hello @gebi, thank you for bringing this up!
This is not a bug, but a feature (maybe not the best one). It should be stated in a clearer way, but in fact the credentials you are filling in .env file are meant for the Root Admin. The actual Users need to be registered using the Signup form (<GRIMOIRE_URL>/signup).
Perhaps the initial User should be created alongside the Root Admin. Would it be more intuitive (Admins and Users are strictly separated and do not share any features, even use different DB tables)?

oh, ok... i got it working with creating a new user with /signup, strange but ok :).
Having an initial user would be nice, ack especially for testing it out locally.
i've no idea what a root admin is or should do, i tried the docker compose internal 172er ip on port 80 with path /_/ (http://<IP_ADDRESS>/_/#/login where i then get a pocketbase admin sign-in) but there too i just get a invalid credentials
with a 400 bad request on POST to http://<IP_ADDRESS>/api/admins/auth-with-password
Payload {identity: "MY-EMAIL", password: "MY-PASSWORD"}
Response {"code":400,"message":"Failed to authenticate.","data":{}}
i'm really sorry to nag you with that but i just wanted to quickly try out grimoire locally with the suggested docker compose method.

Admins have their designated dashboard (under <GRIMOIRE_URL>/admin), but atm their role is focused around changing the configuration of PocketBase (like setting backups, SMTP etc.). If you cannot log in with the credentials you specified in the .env file, maybe the first migration did proceed with the default values?
If that's the case, please try it again with email<EMAIL_ADDRESS>and password changeme.

no idea what happened i just got this error below.
anyway, i've inserted the bcrypt from the user table in pb_data/data.db to  _admin table and now the expected pw works.
grimoire             |   status: 400,
grimoire             |   response: { code: 400, message: 'Failed to authenticate.', data: {} },
grimoire             |   isAbort: false,
grimoire             |   originalError: {
grimoire             |     url: 'http://pocketbase/api/admins/auth-with-password',
grimoire             |     status: 400,
grimoire             |     data: { code: 400, message: 'Failed to authenticate.', data: {} }
grimoire             |   }

but creating an initial admin and bookmark user and some description to which URL the login works would be awesome :)!


no cpufreq governor available on Asus GA402

Hi,
Just figured out that the cpufreq governors are not available on my ryzen based Asus GA402.
I am on kernel 6.1 and amd-pstate module is builtin. Any idea?
@K900 do you have this problem too ?

schedutil should be available and what you should be using.

Fixed in https://github.com/NixOS/nixos-hardware/pull/517.


Don't switch output pane focus

Sometimes I'm looking at something in my output window - a build warning or something similar - and TFS Auto Shelve switches to its pane to tell me it has shelved. This gets very annoying after a while.
There could be an option to disable this switch - I know you can pause while debugging, but it's not just while debugging that we use the output window.

Hmm, this might actually be the Source Control output pane rather than the TFS Auto Shelve one - not sure there's a way you can control that.

OutputString is called, but the Activate method is not.  I believe this behavior is specific to the VS IDE version.  If you know how to avoid it, you are encouraged to submit a PR.

OutputString is called, Activate is not.  If you know how to avoid unwanted focus, please submit a Pull Request.

I don't, really. After thinking about it some more I think the TFS output window listens for TFS events (like shelve) and pops up to let you know something happened. In theory this would only happen when you clicked a button so maybe that's OK, but that doesn't take into account something automatically doing stuff.
If I can catch it in the act again, I'll see about recording a .gif or something

This is really very annoying. It happens to me in VS 2019. I want to see my Build Output, but TFS auto-shelve extension obstructs it.


Feature/rocket chat users rooms

Adding RC integration for event creation on click of submit
Adding allowed hosts
Changing the model for events
Changing the models for meetup
Supporting creation of migration of model on django without CLI interruption

Yes, there is a way to throw errors and warnings in the same page. I think
we had it integrated somewhere & removed it later
On Sun, Jul 26, 2020, 6:13 PM Shrayas Rajagopal<EMAIL_ADDRESS>wrote:

@shrayasr commented on this pull request.
In guby/settings.py
https://github.com/unnati-xyz/guby/pull/11#discussion_r460522093:

@@ -25,7 +25,7 @@

SECURITY WARNING: don't run with debug turned on in production!
DEBUG = True
-ALLOWED_HOSTS = ['gubylive']
+ALLOWED_HOSTS = ['gubylive', 'localhost', '<IP_ADDRESS>']
Let's keep localhost and remove the 127. I think that's more used than the
IP itself. What do you think>
In guby_backend/views.py
https://github.com/unnati-xyz/guby/pull/11#discussion_r460522237:

 if form.is_valid():

   model = form.save(commit=False)

   meetup = Meetup.objects.get(id=meetup_id)

   model.meetup = meetup

   model.status = Event.STATUS_CREATED

   #TODO handle errors

   model.save()

   return redirect(f'/app/meetups/{meetup_id}/events/')

   try:

Righto.
The way we should be doing error handling in forms is to inform the user
of the errors. There is a way in django to push the errors back to the
form. We need to use that approach instead.
cc @raghothams https://github.com/raghothams right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/unnati-xyz/guby/pull/11#discussion_r460522093, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIYFFCY67RHJJGOE2WEQZLR5QQGLANCNFSM4PH3GAYA
.


hackathon DAO | ZK Podcast - deep dive into zkRollup

To be the main guest speaker of a podcast about zkRollup. The estimated length of the podcast post-editing would be 1-1.5 hour(s) and it should include but not limited to:

Deep dive into zkrollup's proof generation

Bounty: 100 USDT + 1 $HAC
Instructions

Join our Discord
Go to #💰bounties channel
Send “zkRollup Podcast” and our staff will follow up with you bounty details and reviewing.
The bonus will be released after the first PR of the podcast publishing.

#35


improvement: change default global-work to 16k

I think part of the reason why some people find Claymore's miner faster is that they don't change the default work values.  16k global work generally gives a 3-5% improvement in hashrate, and doesn't seem to have any downsides.

Good point.

even though i cleaned my old 7950 and gave it fresh thermal paste, i can't get the card stable with this setting. it's too much for a default setting i'm afraid.

According to dlhenkey and wolf, 16k will give you many more stale shares.