chore: Test PR

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Type of change.

[ ] FIX - a PR of this type patches a bug.
[ ] FEATURE - a PR of this type introduces a new feature.
[ ] BUILD - a PR of this type introduces build changes.
[ ] CI - a PR of this type introduces CI changes.
[ ] DOCS - a PR of this type introduces DOCS improvement.
[ ] STYLE - a PR of this type introduces style changes.
[ ] REFACTOR - a PR of this type introduces refactoring.
[ ] PERFORMANCE - a PR of this type introduces performance changes.
[ ] TEST - a PR of this type adds more tests.
[x] CHORE - a PR introduces other changes than the specified above.

Closing this PR as it was opened just for testing purposes to trigger GH-Actions because they started to fail for a lack of permissions.


forge throws core dump when encountering long string literal

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[ ] Foundryup

What version of Foundry are you on?
forge 0.2.0 (d7eac74 2024-06-21T00:16:53.812858792Z)
What command(s) is the bug in?
forge test
Operating System
Linux
Describe the bug
I have to simulate a historic transaction.
I hardcode its calldata in string literal  hex"long calldata......."
forge build throws
Error:
solc exited with signal: 11 (SIGSEGV) (core dumped)
<empty output>

LLM suggested this tweak in foundry.toml
[profile.default]
solc_args = ['--memory-limit', '16384']

but it didn't work.
I am not sure if this is intended or a bug and I am not sure if this originates from forge or underlying solc.

Most likely solc related https://github.com/ethereum/solidity/issues/13496 so not much we can do about.
The hex string is too long, try maybe loading it through a binary file rather than hard coding it.


Fix Windows error because of timers deleted too late

This pull request does in fact not fix the error...

yeah I've tried most of the obvious stuff.


Remove create_dag functions

Current Situation

In many DAG definition files, we're defining a create_dag function and then adding the DAG to the global variables by calling that function once.
Suggested Improvement

In these cases, we don't need to call the function at all and can just define the DAG directly within the file.
Benefit

We reduce the need to make an extra function within the DAG files, and we no longer have to call globals()[DAG_ID] = create_dag().
Additional context

Implementation

[ ] 🙋 I would be interested in implementing this feature.

Oops, this is duplicated by #241


Fix CustomListOutputParser format instructions

This PR contains a little fix on CustomListOutputParser argument handling.
I think there may be a bug in the format instruction of CustomListOutputParser

length constructor arguments controls the number of lines the user want - documentation
it may be number or undefined if the user dont want to control the number of lines - documentation
nevertheless, if length is undefined, the format instructions include "undefined items".

if length constructor arguments is provided and equals to 3, the format instruction will be:
Your response should be a list of 3 items separated by "|" (eg: `foo| bar| baz`)

BEFORE: note the undefined items
if length constructor arguments is not provided, the format instruction will be:
Your response should be a list of undefined items separated by "|" (eg: `foo| bar| baz`)

AFTER: if length constructor arguments is not provided, the format instruction will be:
Your response should be a list of items separated by "|" (eg: `foo| bar| baz`)

my twitter handle is https://twitter.com/jerome_etienne
PS: thanks for the excelent library!

Little test program:
import { CustomListOutputParser } from "langchain/output_parsers";

// With a `CustomListOutputParser`, we can parse a list with a specific length and separator.
const parser = new CustomListOutputParser({
  // length: 3,
  separator: "|||"
});

const formatInstructions = parser.getFormatInstructions();

console.log(`Format Instruction:\n${formatInstructions}`);

Thanks for the catch!


LPAL-253 details element keyboard navigation fixes

Purpose
Ensure that links and inputs inside <details> elements can be accessed via keyboard navigation when the element is expanded.
Fixes LPAL-253
Approach
Rewrite the polyfill to correct its behaviour
Add a cypress test to expand and navigate the content of a <details> element (and the same for a polyfilled element behaving as a details element).
Learning
Possibly should have used the govuk-frontend rewrite of the polyfill, rather than do it myself. This rewrite does have the advantage that you can apply it to elements other than <details>, which makes it easier to functional test (see enclosed cypress test). Also, I don't think the govuk-frontend polyfill attaches ARIA attributes (which this implementation does).
Checklist

[x] I have performed a self-review of my own code
[x] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[x] I have added tests to prove my work
[ ] The product team have tested these changes

Apologies @nickdavis2001, after merging master it dismissed your review. Could you please approve again?


Df 1505 archive events detail

Add Events Archive page
Changes

Updates '_events.html" file to add condition for omitting inset markup when even/archive path.
Updates 'lookups.json" file to display the event archive post type.
Updates 'event-post-macros" file to add condition for displaying image and text.
Added 'archive/_single.html' file to display the event archive page.

Review

@jimmynotjim
@anselmbradford

et al
Preview

LGTM :+1:


Restructuring and extending code based on 'clean code' principles

Sorry it was my comment from an other account.
"I think it should be better if NUM_GHOSTS and NUM_COINS were randomly generated between numbers given by the gamer (by difficulty).
Other than that it looks great."

@NyesteAmbrusPal
I've tried to resolve the problems you spotted, would you be so kind to review the changes committed since? #

Sorry it was my comment from an other account.
"I think it should be better if NUM_GHOSTS and NUM_COINS were randomly generated between numbers given by the gamer (by difficulty).
Other than that it looks great."

@VitayVanda
Firstly, thank you for your kind feedback. On one hand, I can agree with you on that these values could be passed as an argument as a let's say 'difficulty parameter'.
But on the other hand, if we look at the original implementation of the Pacman game, the number of coins and ghosts spawned in each episode are indisputably fixed (the number of ghosts is always equal to 4, and the number of coins is basically a constant of the map layout).
Conclusion:
Therefore, in my opinion, these values can remain in the code as constants.
What do You think about this?


can you provide a lite version not including mousewhell and anything not related for mobile to make size smaller?

Mobile device do not need so many features, I'm using the basic functions only for mobile. and it need  size sensitive, I wonder if you interested on make a lite version? thanks!

You can remove not required modules in gulp.js file at https://github.com/nolimits4web/Swiper/blob/master/gulpfile.js#L42
And do gulp build to build required version


How do we handle resourceGroupName / resource_group definition?

It seems like there's some kind of exception built in and it's not mentioned in current documentation.
In batch account example api.yaml has resourceGroupName but in generated module we have resource_group.
I think the mechanism should be included in api.yaml as an exception.
Also it's a bit confusing in examples: resourceGroup and resource_group_name, but I described this in another issue related to examples as well.

Yes, resourceGroupName is a very special string which will be handled specifically in MM for both Terraform and Ansible. But since you are generating resourceGroup, we will use overrides in terraform.yaml to rename it to resourceGroupName.


Create headless terminal backend bench test using termbench-pro.

https://github.com/contour-terminal/termbench-pro
It's a library. It's headless.
Let's see if we can get other terminals to use termbench-pro for internal backend performance testing, too. WT? ;-)

Great!
Currently I'm using https://github.com/cmuratori/termbench/tree/v3_work.
Unfortunately, it often breaks Contour. :(

Great!
Currently I'm using https://github.com/cmuratori/termbench/tree/v3_work.
Unfortunately, it often breaks Contour. :(

Perfect! So let's fix it! (I know it's breaking on binary output at least, yeah).
I did this, so the benchmarking can be done 100% decoupled from the rendering, to just concentrate on the backend throughput but also be able to easier debug and get more determinism into it.

You'll probably be interested in https://github.com/microsoft/terminal/issues/10362. They've done alot of work and still continuing more as time permits to satisfy termbench for VT processing.

You'll probably be interested in microsoft/terminal#10362. They've done alot of work and still continuing more as time permits to satisfy termbench for VT processing.

You will never ever satisfy what cmuratori claims. The values he claims are made up and based on a fundamental design flaw in refterm's buffer management. refterm is basically benchmarking memcpy(). Congratulations to that. :-)
I am aware of that ticket, but sadly I cannot comment there as I'm not a Windows Terminal contributor, even though I'd like to stay constructive. :)

Yeah I think the big thing is looking at hot spots in Process Explorer regarding VT processing and making improvements when able. It should satisfy termbench a bit more than before. As you might of noted in that issue, they already had compared to what was shown at first since it was based on an older conhost. Keep in mind that any conhost changes will also very likely help Contour as well since it's the backend behind ConPTY, which Contour uses.

https://github.com/microsoft/terminal/issues/10563 is MS's actual official issue for it, without any of the other stuff.

Hi guys. Really happy to see fantastic work is done here for Windows users to have another terminal to use. I would like to point out my issue is NOT anything official. Feel free to comment or anything, but I am only an outside contributor, who happens to work for MS.

The values he claims are made up and based on a fundamental design flaw in refterm's buffer management

Yeah this is one of the reasons why refterm is fast. The buffer implementation benefits the buffer -> renderer side. Another reason is that refterm isn't really trying to be a terminal. It uses something close to raw IO (NamedPipe on Windows) instead of pty interface that the OS provides. This benefits hugely on the program stdio -> buffer side. As a result, refterm is not only faster than every terminal on Windows, it also beats the crap out of every single terminal on Linux & macOS (search twitter and you'll find the evidence).
In conclusion, refterm basically sets a standard that no real terminal can meet, if you're talking about a functional terminal. Sadly a lot of people out there treat refterm as something very close to a real terminal, and cmuratori does not seem to want to clarify the difference.
Also fun fact: all of the members of the terminal team (including me, an unofficial member) are blocked by cmuratori personally on both Github & Twitter. We can't comment, fork or even star the refterm repo (and all his other repos). That being said, I doubt any of us would really bother to poke him anymore, even if we're not blocked. Because that turned into something pretty irrational and emotional before, and I think no one wants one more time of that.

Closing as we have that on master now.


Getter not found: 'LocationCallbackHandler'.

Hi again :),
I am trying to implement your package at new flutter project. Before that I used your project on github.
in startLocator method LocationCallbackHandler gives an error. I could not find what I need to call in to solve it.

      lib/main.dart:150:44: Error: Getter not found: 'LocationCallbackHandler'.
      BackgroundLocator.registerLocationUpdate(LocationCallbackHandler.callback,
                                               ^^^^^^^^^^^^^^^^^^^^^^^
    lib/main.dart:151:21: Error: Getter not found: 'LocationCallbackHandler'.
          initCallback: LocationCallbackHandler.initCallback,
                        ^^^^^^^^^^^^^^^^^^^^^^^
    lib/main.dart:157:24: Error: Getter not found: 'LocationCallbackHandler'.
          disposeCallback: LocationCallbackHandler.disposeCallback,
                           ^^^^^^^^^^^^^^^^^^^^^^^
    lib/main.dart:175:15: Error: Getter not found: 'LocationCallbackHandler'.
                  LocationCallbackHandler.notificationCallback)));
                  ^^^^^^^^^^^^^^^^^^^^^^^

    Command PhaseScriptExecution failed with a nonzero exit code
    note: Using new build system
    note: Building targets in parallel
    note: Planning build
    note: Constructing build description

Could not build the application for the simulator.
Error launching application on iPhone 12 Pro Max.

                  ^^^^^^^^^^^^^^^^^^^^^^^

Also, when the location changes in the github project itself, the notification drops out. Where can I turn it off? And I want to write api here.
I wrote api into port.listen ((dynamic data), but it did not log, I would be glad if you could help me write the exact api to which part.

I have the same problem, what can it can be?

Hello,
i guess your are trying the code from the Wiki which is slightly incomplete, LocationCallbackHandler is just a Class containing all callbacks ( you have to create it yourself ) as you can see in the example here
https://github.com/rekab-app/background_locator/blob/master/example/lib/location_callback_handler.dart
Cheers


Using DAL with formtools (preview function)

Hello.
I am using DAL in combination with the formtools. It works well until I get to the preview page. There the autocomplete widget is gone and just a normal MultipleChoiceField is left (with the previous selected options). If I deselect everything and click preview again, then the autocomplete widget is displayed again.
What can I do?

I know, I am a bit late, but I think I have a working example.
https://github.com/mogoh/django-autocomplete-light/blob/960/test_project/select2_preview/preview-test.rst
https://github.com/mogoh/django-autocomplete-light/commit/2712b0b2fdda81d4d025aa7b7672fe9f66c0087c

seems to me that the footer block isn't outputed and thus scripts aren't executed

it's rendering a tpl without scripts and without footer block, enabling django-debug-toolbar to find more

you just need to override formtools/preview.html and install the scripts there (jquery.js and form.media)
and it should work fine
please reopen if any issue

Thank you very much! You solved this riddle for me and that in no time! 😄

When users make it easy to reproduce then yes, the problem is when we have to recreate a case it takes more time !


[BUG] Flickering during start on some systems

Issue description:
When the app is started it starts to flicker on Desktop (wayland) and android 5.
Environment:

Application version: 0.2.2
Device: Ubuntu with Wayland, ASUS ZenFone 2 Laser

Steps to reproduce:

Run the app
See flickering

Additional context
It's stopped when you switching to Edit tab.

Fixed flickering by set Edit tab point cloud view render to "on demand".


slowness on custom styled radio or checkbox

What is your Test Scenario?

Testing a radio or checkbox that has custom styling.
What is the Current behavior?

Currently, clicking on the element is very slow, even though the element is present.  The click will work, but not until the very end of the selector timeout.
What is the Expected behavior?

Not sure if this is a bug or by design.  If it's a bug, then I suppose it should fail?  if it's by design, then perhaps it should not wait for the timeout to complete?
What is your web application and your TestCafe test code?

Repro can be found here (instructions in the readme):
https://github.com/benmonro/slow-radio-testcafe-issue

Your website URL (or attach your complete example):

Your complete test code (or attach your test files):

Your complete configuration file (if any):

Your complete test report:

Screenshots:

Steps to Reproduce:

Clone repo above
yarn && yarn test
See there are 2 tests, one is slow, one is fast, only difference: one sets a z-index on the custom styled input field, the other doesn't.

Your Environment details:

testcafe version:                   1.9.3
node.js version:                   v12.16.1
command-line arguments:  yarn test
browser name and version:  chrome 85
platform and version:         mac
other:

Hello,
This behavior is correct. TestCafe cannot interact with page elements under other elements. If the target element is not on top when an action is triggered, TestCafe waits for this element to appear in the foreground. If this does not happen within the selector timeout, the action is performed with an overlaying element. Please refer to the 'select-target-elements'  documentation

@AlexanderMoiseev curious what the reason is for this?  Custom radio/check box is a pretty common pattern.

Hello,
To avoid waiting for the selector’s timeout, you can use the “span.checkmark” selector instead of your "input[type=radio]” selector.
As for your question, Testcafe emulates a user’s behavior. A user can’t click an element that is overlapped by another element, so Testcafe waits until this element becomes not overlapped. If this does not happen, the found element can be clicked as a fallback.
We agree with you, since this is a popular pattern, we will consider handling this case more gracefully in the future.

Thank you for that suggestion @AlexanderMoiseev.  And yeah thank you for addressing

You are welcome. Should you have additional questions or require further assistance, feel free to contact us at your convenience. We are here to help.


various new_gallery__transition_perf devicelab tests timing out

For example:
[Linux_android new_gallery__transition_perf](https://ci.chromium.org/p/flutter/builders/prod/Linux_android%20new_gallery__transition_perf)

https://ci.chromium.org/p/flutter/builders/prod/Linux_android new_gallery__transition_perf/4323

[Linux_samsung_s10 new_gallery__transition_perf](https://ci.chromium.org/p/flutter/builders/prod/Linux_samsung_s10%20new_gallery__transition_perf)

https://ci.chromium.org/p/flutter/builders/prod/Linux_samsung_s10 new_gallery__transition_perf/3738
https://ci.chromium.org/p/flutter/builders/prod/Linux_samsung_s10 new_gallery__transition_perf/3724
https://ci.chromium.org/p/flutter/builders/prod/Linux_samsung_s10 new_gallery__transition_perf/3643

Assigning P2 as it flakes infrequently enough that a re-run will usually fix the tree.

That sure does look similar to the logs in https://github.com/flutter/gallery/pull/792.  Unfortunately those Linux tests don't upload a screenshot on failure so it's not obvious why it's not scrolling into view. In the case of my PR I could see that the iPhone was in landscape which was a pretty big hint, then I could reproduce locally.

I'm not sure why it's not uploading screenshots for Linux, it should be...
Executing "/opt/s/w/ir/x/w/recipe_cleanup/tmpo5hlmkw8/flutter sdk/bin/flutter drive --screenshot /opt/s/w/ir/x/w/recipe_cleanup/flutter_logs_dir -...

In the other case FlutterDriverExtension timed out before the task was killed at 30 minutes, which gave it time to take the screenshot.
FlutterDriverExtension: Timeout while executing waitFor: TimeoutException after 0:00:10.000000: Future not completed

However here it's waiting the full 30 minutes and being killed at the task level, so doesn't have a chance to save the screenshot.

Assigning to @christopherfujino to look into the screenshots.

Since LUCI sends SIGTERM to the currently running process when timing it out, I will try registering a handler in the drive command to take a screenshot when SIGTERM is sent.

Hit one of these here:
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android new_gallery__transition_perf/4421/overview

Since LUCI sends SIGTERM to the currently running process when timing it out, I will try registering a handler in the drive command to take a screenshot when SIGTERM is sent.

It appears this didn't work, so I will add a flag to allow explicitly specifying a timeout that will ensure the screenshot is taken.

Linux_android new_gallery__transition_perf has been flaky 5 times out of recent 20 runs:
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android new_gallery__transition_perf/4421/overview
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android new_gallery__transition_perf/4426/overview
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android new_gallery__transition_perf/4434/overview
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android new_gallery__transition_perf/4435/overview
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux_android new_gallery__transition_perf/4443/overview
Marking it as flaky:
https://github.com/flutter/flutter/pull/114429

What I hope is a fix for this issue landed in https://github.com/flutter/gallery/pull/824 and rolled to the framework in https://github.com/flutter/flutter/pull/114537.
Currently waiting to see how the next ~15-20 runs go.

No flakes in the last 20 runs. I'm going to close this. @christopherfujino is there further work to do on the screenshotting side? If so, we should probably open a new (non-p2) bug dedicated to that work.


haskell-backend/src/main/native/haskell-backend: update deps

This contains the latest updates for supporting WASM proofs.

Duplicate of #786.


Missing path in bower.json

Please add "main": "./angular-colorthief.min.js", in bower.json

Closed via ae9de98399ced8351e6020ee6f9137304fbf8641


Updated README.md

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
Fixes #1044
Description
[Please include a brief description of the changes or features added]
Added Total number of PRs and Current ongoing issues (Open & Closed) & Last Commit,stars
Type of PR

[ ] Bug fix
[ ] Feature enhancement
[X] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]

Checklist:

[X] I have performed a self-review of my code
[X] I have read and followed the Contribution Guidelines.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ ] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

@arghadipmanna101 please give the labels like gssoc and level1

I'm GSSOC'24 contributor,Please give labels like gssoc and level
On Mon, 10 Jun, 2024, 00:28 Arghadip Manna, @.***>
wrote:

Merged #1081
https://github.com/arghadipmanna101/Flipkart_Clone/pull/1081 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/arghadipmanna101/Flipkart_Clone/pull/1081#event-13092302388,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A53UULV5EAQEU4FYDPEYIGDZGSQW3AVCNFSM6AAAAABI6WKNISVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGA4TEMZQGIZTQOA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

--
*Engineering *Sciences
BVRIT http://bvrit.ac.in | SVECW
http://svecw.edu.in | VIT http://vishnu.edu.in | *BVRITH *
http://bvrithyderabad.edu.in
Medical Sciences
VDC
http://vdc.edu.in | SVCP http://svcp.edu.in | VIPER
http://viper.ac.in| BVRICE http://bvrice.edu.in*
http://viper.ac.in*


target.__agBeanMetaData 'undefined' is not an object

Hey!
I've upgraded our ag-grid version to 4.1.5 and since then, our karma tests don't run anymore.
We are injecting 'bower_components/ag-grid/dist/ag-grid.noStyle.js' which worked in version 3.3.3.
The error is:
TypeError: 'undefined' is not an object (evaluating 'target.__agBeanMetaData')
at http://localhost:9876/base/bower_components/ag-grid/dist/ag-grid.noStyle.js?a85c4693cb44744c8456bfc0352952bd2f98c0fd:1186

Which refers to: src/ts/context/context.ts:315
Is there a migration step which we forgot about or has anyone come across this?
Thanks in advance!

i've not come across this before. we also don't use karma. __agBeanMetaData is something we use internally for annotations.

@ceolter Is something I can help you with investigating this issue? I'm puzzled that no one else came across this before :S

you could see why the code is not executing in the way it should. what should happen:

we use TypeScript for creating the annotations. the annotations execute when the library loads and create the __agBeanMetaData on the objects on the classes. for example, the Bean function gets called as the @Bean annotation is used.
the __agBeanMetaData is then used by the grid when the classes are used.

so you could put a breakpoint into the Bean function, you can verify it is called before the error method when using the grid normally. then see why that path is not following in your test?
i think the test is not executing the script first and allowing initialisation of the annotations before using the classes defined.

I'll look into it again, when I'll need to take care of tests again. Thanks!

no update, closing.


Not working with my version of Laravel Scout ?

Hi, the search() method does not work on the indexed model. The searchable() method does not either. Here is the query in my controller:
public function search(Request $request){

            $articles = Article::search($request->input('search'))->get();

        return view('Articles.searchResult', compact('articles'));

    }

It always returns all objets in the collection.
I use Laravel 5.3, with Scout 2.0, Elastic Search 5.2 and laravel-scout-elastic 2.0.0
Here is an excerpt of my file composer.json :
"require": {
        "php": ">=5.6.4",
        "elasticsearch/elasticsearch": "^5.2",
        "laravel/framework": "5.3.*",
        "tamayo/laravel-scout-elastic": "~2"
    },
    "require-dev": {
        "fzaninotto/faker": "~1.4",
        "mockery/mockery": "0.9.*",
        "phpunit/phpunit": "~5.0",
        "symfony/css-selector": "3.1.*",
        "symfony/dom-crawler": "3.1.*",
        "laravel/scout" : "^2.0"
    },

Could you help me please ? Thanks in advance

closing due to age


Eventually should support setting both timeout and poll rate in same call

The following case was missed when Eventually was created and is not currently supported.
Eventually(FakeAction()).trying_for(23).seconds().polling(3).seconds()

Thanks for the fix, @bandophahita!


Buttons actions generated by messages sent through the step function, do not generate trigger_id

Another problem I encountered is that in the payload returned by actions generated by the step function, there is no trigger_id.
Due to this, I am having to send messages using another traditional bot's token after the step function is activated.

@WilliamBergamin ,
When the user executes the workflow, they receive an ephemeral message with 2 buttons.
I had done it the way you said, using a traditional bot. But I thought I could do it using the new way.
Thanks.

I see, we don't have docs for this in python yet but the approach should be similar to what is found here
You should be able to pass an interactivity_pointer field instead of a trigger_id this interactivity pointer value should be available in the payload of the actions resulting from a button click when the message with the button is posted using the "function token" not the "traditional bot's token"
Let me know this is solves your issue, we will need to document this


⚠ ClamAV has degraded performance

In 4f02de6, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 182 ms

Resolved: ClamAV performance has improved in f09db99.


:dir support for 1.x and native ShadowDOM 2.x

ShadyCSS supports :dir() to provide different styling for RTL and LTR layouts.
This should be backported to 1.x, and also polyfilled to be supported in Native ShadowDOM for 2.x.
The approach for 2.x is somewhat simple: With a mixin, we can detect the direction of the page and set an attribute on all elements implementing the mixin. Then, we can transform :dir() into a simple attribute selector.

Fixed with #4880 and #4881


Suggested improvements

Add redirects for /favicon.ico, /robots.txt, /humans.txt
Add errorhandlers for 404 and 500
Remove config.py, which only had 1 config item
Ask Talisker for "raven", "prometheus", "flask" by default - for our logging / error handling etc.

See https://github.com/canonical-web-and-design/canonical.com/pull/10/files#diff-bed92ddcb62003ca6b223dfa86df23b3L11 for example of how this would be used.
Fixes #5
QA
Pr for testing this here: https://github.com/canonical-web-and-design/canonical.com/pull/10
Demo here: https://canonical-com-canonical-web-and-design-pr-10.run.demo.haus

[ ] https://canonical-com-canonical-web-and-design-pr-10.run.demo.haus/ works
[ ] https://canonical-com-canonical-web-and-design-pr-10.run.demo.haus/test works
[ ] https://canonical-com-canonical-web-and-design-pr-10.run.demo.haus/missing -> "404"
[ ] https://canonical-com-canonical-web-and-design-pr-10.run.demo.haus/error -> "500"
[ ] https://canonical-com-canonical-web-and-design-pr-10.run.demo.haus/favicon.ico -> Redirect to assets
[ ] https://canonical-com-canonical-web-and-design-pr-10.run.demo.haus/robots.txt -> Redirect to /static/robots.txt
[ ] https://canonical-com-canonical-web-and-design-pr-10.run.demo.haus/humans.txt -> Redirect to /static/humans.txt
[ ] https://canonical-com-canonical-web-and-design-pr-10.run.demo.haus/hello -> Redirect to /world (404)

Have you considered http://flask.pocoo.org/docs/1.0/api/#flask.send_from_directory to server the static files instead of the redirect?

@jpmartinspt Have you considered http://flask.pocoo.org/docs/1.0/api/#flask.send_from_directory to server the static files instead of the redirect?

Yes I considered that. I thought it might be rather confusing if /robots.txt directly serves a file at /static/robots.txt, which would behave differently from all other /static files, and I did check that Google's bot at least will follow the redirect.
However, I'm open to the idea of asking for a file path instead of a URL, and simply loading the contents of that file. Do you think that would be better?

@jpmartinspt Have you considered http://flask.pocoo.org/docs/1.0/api/#flask.send_from_directory to server the static files instead of the redirect?

Yes I considered that. I thought it might be rather confusing if /robots.txt directly serves a file at /static/robots.txt, which would behave differently from all other /static files, and I did check that Google's bot at least will follow the redirect.
However, I'm open to the idea of asking for a file path instead of a URL, and simply loading the contents of that file. Do you think that would be better?
In the case of /favicon.ico I definitely want to use a URL, because the actual favicon will likely live on the assets server. But I don't suppose that would be the case for robots.txt or humans.txt - that would probably be rather confusing.

I was thinking just serving them from the static folder like:
@app.route('/robots.txt')
@app.route('/sitemap.xml')
def static_from_root():
    return send_from_directory(app.static_folder, request.path[1:])

For the favicon:
Could we not just link the the asset server in the templates directly?

I was thinking just serving them from the static folder like:
@app.route('/robots.txt')
@app.route('/sitemap.xml')
def static_from_root():
    return send_from_directory(app.static_folder, request.path[1:])

Yeah that seems fine to me - I'd just wrap them in an if os.path.isfile('robots.txt'). So I'll remove the robots_url and humans_url options, and then document in the README.md that you should create those files at the root of the project if you want them served.

For the favicon:
Could we not just link the the asset server in the templates directly?

We do that already - e.g. https://ubuntu.com has <link rel="icon" type="image/png" href="https://assets.ubuntu.com/v1/49a1a858-favicon-32x32.png" sizes="32x32" />, but the issue here is that most browsers query /favicon.ico anyway, which is one of our highest 404s. So the idea here is to send something back at that path specifically. I think a redirect is the right solution in this case.


Jest config .cjs not found

I have to use .cjs extension for my jest config (cause I use "type": "module" in poackage.json).
Then it's not found and does not work.

was another error on my side....


build(codeql): improve code scanning configuration

exclude .md and .txt files from analysis
enabling scans to run amnually
upgrade codeql to v2 - Current version of the CodeQL (v1) was deprecated on January 18th, 2023, and is no longer updated or supported. For better performance, improved security, and new features, we upgrade it to v2. [More information](https://github.blog/changelog/2023-01-18-code-scanning-codeql-action-v1-is-now-deprecated
comment out the autobuild step, which is not needed with javascript-based code

Let's put this on hold, until we finalize the proper configuration/approach in xlts-dev/angularjs-webpack-babel#29.


Improve regex by removing backtracking, and fix few bugs in pattern.

BugFix:
Detection of non-spaced group starting group:"..."
Improves regex by about 5 fold by removing backtracking and unnecessary groups:

I did not found the regex which detects plugins and etc, but have done some work on optimizing that as well (got the permalink through reddit post):
id\s+['"]([^'"]*)['"](?:\s*version\s*['"]([^'"]*)['"])?

\s* -> \s+ (need to have a space in order to be a valid groovy syntax)
next are same (removing backtracking).

Replaced by 5ca1358


Group - Group.fromJSON always returns null

Description
In the snarky.d.ts TS file, the Group class has a method on it called fromJSON. See here
When this function is called with any type of JSONValue, null is always returned
Reproduce:
The following code reproduces this error:
import { Group, isReady, shutdown } from 'snarkyjs';

await isReady;

// Different enumerations of `JSONValue`
const a = Group.fromJSON(1);
const b = Group.fromJSON(true);
const c = Group.fromJSON('1');
const d = Group.fromJSON([]);
const e = Group.fromJSON([1]);
const f = Group.fromJSON([1, 1]);
console.log(a);
console.log(b);
console.log(c);
console.log(d);
console.log(e);
console.log(f);

await shutdown();

I guess the method should throw an error on these inputs. The proper json representation is {x: string, y: string}, where the string represents a Field. I also think we should stop accepting other types than string in Field.fromJSON, there should always be only one valid json representation, namely the same that toJSON returns


[build-script] Create relative symlinks into the llvm tree.

What's in this pull request?

On several occasions I have run into build failures related to missing headers after running update-checkout. I've discovered that one way to trigger this issue is to run the build script simultaneously from my OS X desktop and from within a virtualized Linux environment which has the same source source tree mounted as a volume at a different location. The build script symlinks clang and compiler-rt into the LLVM tree, but because the symlinks are created with absolute paths, the symlinks are specific to the environment where the build script is currently executing.
For example, when the build script is run from OS X, the generated symlink looks like:
./llvm/tools/clang -> /Users/briancroom/Source/swift.org/clang
whereas when run from Linux, it looks like:
./llvm/tools/clang -> /src/swift.org/clang
This PR changes these symlinks to be created with a relative path, looking like:
./llvm/tools/clang -> ../../clang
which is valid in both environments. Note that I'm using Python's os.path.relpath function to generate the path because I was unable to find a reasonable Bash solution for doing so.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test and merge


Exponential operator returning wrong number

🐛 Bug Report
Exponential operator is returning wrong number in jest.  It works fine with node
Anyone else having this issue?
To Reproduce
In this repo npm install && npm test
Expected behavior
The following test is failing :(
test("exp", () => {
  expect((2 ** 3) ** 3).toEqual(512)
})

 Expected value to equal:
     512
 Received:
     134217728

Link to repl or repo (highly encouraged)
Repository jest-test-exponential
Run npx envinfo --preset jest
  System:
    OS: Linux 4.18 Fedora 27 (Workstation Edition) 27 (Workstation Edition)
    CPU: (4) x64 Intel(R) Core(TM) i7-7500U CPU @ 2.70GHz
  Binaries:
    Node: 9.11.1 - ~/.nvm/versions/node/v9.11.1/bin/node
    npm: 6.4.1 - ~/.nvm/versions/node/v9.11.1/bin/npm
  npmPackages:
    jest: ^23.6.0 => 23.6.0

That's an interesting one.
Jest is transforming test files through babel, which seems to remove parenthesis from (2 ** 3) ** 3. Since the exponential operator is right associative, you get the result of 2 ** (3 ** 3). I don't think that the problem is with Jest's Babel configuration, so probably fix should be implemented by Babel.

Well explained! Thanks.


Add Plivo Notifier.

This is a simple notifier, which rings your phone in case of an exception.
Our use case is the following:
Receiving emails or other notifications are nice. However, in the middle of the night the best (worst?) way to get your attention is by ringing your phone. Plivio has a simple API to make your phone ring and gives you free credit.
Let me know what you think of it.

Thanks for the PR. There's so many services like this out there that I don't think it's practical for us to start supporting all of them. I'd suggest integrating via the Generic Webhook. I believe there's already a project started for the various SMS providers.

I understand your concert, there are indeed many, many services. On the other hand, so far you have no single service that's only ringing your phone (for free), which is why I added it.
Concerning your suggestion to use the webhook, how would you make such a call this via the generic webhook with basic authentication and specific parameters?
curl -u asdfasdf:asdf -H "Content-Type: application/json" -d '{"from":"123456","to":"123456","answer_url":"https://s3.amazonaws.com/static.plivo.com/answer.xml","answer_method":"GET"}' https://api.plivo.com/v1/Account/ABC/Call/

At least I didn't find a way how to do it. But I'm more than happy if you have a good idea.
And finally, where do you draw the lines which services to include and which ones not? But yes, I understand, this is finally your decision. In any case thanks for building prometheus, very nice set of applications.

The idea would be to run a small server that takes in the webhook and sends out a request to Plivo.

I was thinking first about setting up a simply proxy, that does exactly as you say. Would have also been the easiest solution. I rejected it so I don't have to monitor yet another service. And it makes more sense to have it directly in the alertmanager, in my opinion.
Now if you dislike the idea to merge it into the code, I can live with that, I just use my fork of the alertmanager.  This way you can support any notifier. At the end you also make alertmanager more compelling the more notification systems you support. But just my thoughts.

The generic webhook is intended as our plugin for the Alertmanager.

I see. Then how about to make it a bit more flexible, eg add specific values to the data part of the, set headers etc.?

That's getting to reinventing XSLT but for JSON, which would be creating another Turing Complete language within the Prometheus ecosystem. One of those is enough.

:)
I see, we're getting into a Pattsituation :)
Well, anyway, what shall I do? Close the PR? Or you still consider to merge it? Or just leave it open for now?

If you could close the PR that'd be good. We try to limit things to the most popular integrations.


examples/ci: Warn, dont error on websocket example

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

example fail https://github.com/envoyproxy/envoy/actions/runs/5624322366/job/15240919988#step:9:15720

related to #28546

/retest mobile


Not very helpful

Did not find the guidance helpful as it does not tally with what is now required, in addition it appears that either someone has changed the location for publishing the App Owns Data project, or Microsoft expects that businesses will have the clients on site viewing the project, as the destination in the project is localhost. I would not recommend following these instructions if you do not have fairly advanced web app building skills, PowerBI services skills and Azure AD skills.

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

ID: 54016cab-2de5-0d30-2dfa-ee386dcfb881
Version Independent ID: 4e464b91-bc16-270d-cd51-9bfa396c0326
Content: Embed content in your application for your customers - Power BI
Content Source: powerbi-docs/developer/embedded/embed-sample-for-customers.md
Service: powerbi
Sub-service: powerbi-developer
GitHub Login: @KesemSharabi
Microsoft Alias: kesharab

Hi @how-to-data. Thanks for your comments. I am forwarding it along to the doc author for further consideration.

Hi @how-to-data,
Thanks for your feedback.
I'm in the process of updating all the embedded analytics tutorials, as some of the information there is outdated.
Please note that this may take some time.
Kesem


create-testnet-data: fix generation of Conway genesis w.r.t. delegation of voting power

Changelog
- description: |
    Fix that the `initialDReps` fields in the `conway-genesis.json` file generated by `create-testnet-data` was incomplete.
# uncomment types applicable to the change:
  type:
  # - feature        # introduces a new feature
  # - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  # - refactoring    # QoL changes
  - bugfix         # fixes a defect
  - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

Context
While testing https://github.com/IntersectMBO/cardano-cli/issues/911, I noticed that the SPO created by create-testnet-data doesn't delegate its voting power to a DRep.
So I looked at create-testnet-data and found that only the --stake-delegators created by create-testnet-data delegate to DReps (whereas the SPO is created by --pools). But then, starting with --stake-delegators, there is a bug (introduced in https://github.com/IntersectMBO/cardano-cli/pull/940/files#diff-46f6afb73150af450b10d232ebdca10061efd5a69e268f5b0870439aca3b19e0) that causes an incorrect conway-genesis to be created.
This PR fixes that
How to trust this PR
TBD
Checklist

[ ] Commit sequence broadly makes sense and commits have useful messages
[ ] New tests are added if needed and existing tests are updated. See Running tests for more details
[ ] Self-reviewed the diff

This PR is useless, by virtue of the explanation here: https://github.com/IntersectMBO/cardano-ledger/issues/4782#issuecomment-2523609735
This PR was causing the following change:

But it is not required, because, as @lehins explained; the field delegators is derived from delegs; so we don't need to populate it. I'll do another PR to add some comments to the code, so that we keep a trace of this, because this is confusing IMO.


Tutorial: Organize and elaborate

It's currently just a test case; it needs to be an actual tutorial. Consider creating separate chapters for every feature, as well as chapters for errors and how to handle them. Also needs a chapter on book.json.

I think it would be fun to have a Boomy A-Z example book. It could illustrate a lot of the features, and also share fun photos and her pet names.

The tutorial is now more embedded in the editor.


PR loop

Add a PR CI loop definition which builds and tests all windows platforms/architectures.
Fixes #13

Yarn is to run beachball, the version/changelog/release system I just installed.


SNOW-1618380: Not able to configure snowpark as i am using python 3.12.4 version

Please answer these questions before submitting your issue. Thanks!

What version of Python are you using?

Python 3.12.4 (tags/v3.12.4:8e8a4ba, Jun  6 2024, 19:30:16) [MSC v.1940 64 bit (AMD64)]

What are the Snowpark Python and pandas versions in the environment?
grep : The term 'grep' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or
if a path was included, verify that the path is correct and try again.
At line:1 char:25

python -m pip freeze | grep -e "snowpark" -e "pandas"

                    ~~~~

CategoryInfo          : ObjectNotFound: (grep:String) [], CommandNotFoundException
FullyQualifiedErrorId : CommandNotFoundException

What did you do?
If possible, provide a recipe for reproducing the issue.
A complete runnable program is good.

What did you expect to see?
What should have happened and what happened instead? Please describe the expected behavior or output when the same code is run against a Snowflake connection.

@nagpvny Snowpark is going to support Python 3.12 soon but not today. We'll update it here https://docs.snowflake.com/en/developer-guide/snowpark/python/setup#prerequisites

Support coming soon?


Evaluating functions stops at comment block

Here is a MWE:
function mwe()
    a = 0
    #= hello =#
end

Hitting cmd+enter at the line with a = 0 yields an error about missing end.

Fixed with https://github.com/JunoLab/atom-julia-client/commit/35918a6733d15ee542a936362db911140c003254.


GPL-925-1 Further RNA plate barcode & plate map file name formats required by 2nd April [Heron]

User story
GPL-925 | As representative for LH labs (Alan K) I would like to accept covid samples from MK, Glasgow and AP that have further plate barcode changes by 2nd April
Who are the primary contacts for this story
Alan K (PAM)
Acceptance criteria
To be considered successful the solution must allow:
For MK

[ ] RNA CP plate barcode for MK: cpRNANNNNNN eg cpRNA123456
[x] With corresponding change to the plate map filename, cpRNA123456.csv
[x] RNA Eagle plate barcode for MK:  EMKNNNNNN eg EMK123456 (NB may have been changed previously)

For AP:

[ ] RNA CP plate barcode for AP: XX-XXX-NNNNNNNN eg AP-chp-12345678
[x] With corresponding change to the plate map filename, AP-chp-12345678.csv

For Glasgow:

[ ] RNA CP plate barcode for Glasgow: GLS-GP-NNNNNN eg GLS-GP-123456
[x] With corresponding change to the plate map filename, GLS-GP-123456.csv

New barcode formats added to SS, class with regex, valid foreign barcodes (for Sentinel only directly in SS)
Updates to accepted file names in Crawler
Dependencies
Additional context
Alan K wrote (Tuesday, 23 March 2021 at 12:19)
...
We have just been made aware of some further change being implemented at the Lighthouses. I’ve copied updated tables for file names and barcode formats below and highlighted the changes. Note: feel free to ignore Brant’s Bridge in this request as I’ll need to send all new LH data through. I thought I’d include in case it is easier to do these items as batches.
...

Lighthouse
Unconsolidated plate barcode format
Unconsolidated plate barcode example
CP plate barcode format
CP plate barcode example
Eagle Barcode Format
Eagle Barcode Example

Milton Keynes
XXXNNNNNN
RNA123456
cpRNANNNNNN
cpRNA123456
XXXNNNNNN
EMK123456

Alderley Park
XX-XXX-NNNNNNNN
AP-rna-12345678
XX-XXX-NNNNNNNN
AP-rna-00210235, AP-chp-12345678
XXXNNNNNN
APE123456

Glasgow
XXXNNNNNNX
GLA123456R
GLS-GP-NNNNNN
GLS-GP-123456
XXXNNNNNN
EGG123456GLS123456

Cambridge
NNNNNNNNN
123456789
NNNNNNNNN
002220450
XXXNNNNNN
EGC123456CBE123456

Randox
XXX-AA-NNNNNNNN
RDX-J1-123456
XXX-AA-NNNNNNNN
RDX-J1-123456
XXXNNNNNN
RXE123456

Brant's Bridge
NNNNNNNNNNNNNNNNN
95040456000420484

Lighthouse
Surveillance Plate Map file names
Eagle Plate Map file names
Unconsolidated Plate map file names

Milton Keynes
RNA123456, cpRNA123456
EGL123456
MK_sanger_report_210315_0800

Alderley Park
AP-rna-12345678, AP-chp-12345678
APE123456
AP_sanger_report_210315_0130

Glasgow
GLA123456R, GLS-GP-123456
EGG123456GLS123456
GLS_sanger_report_210315_0500

Cambridge
123456789
EGC123456CBE123456
CB_sanger_report_210315_0126

Randox
RDX-J1-123456
RXE123456
lw-randox-box-55-18869

Brant's Bridge

Coupled with SS story GPL-925-2


Improve integrations tests

Allow to run/debug individual tests in IDE

Codecov Report

Merging #403 (7050934) into main (9812c94) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #403   +/-   ##
=======================================
  Coverage   27.85%   27.85%
=======================================
  Files          72       72
  Lines        7669     7669
=======================================
  Hits         2136     2136
  Misses       5299     5299
  Partials      234      234

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


Digital Ocean Instructions and images

Updated the with requested information. Updated config file instructions. Another important link that you might want to include in the resources is https://hub.docker.com/

Merged outside of GitHub to resolve conflicts.


!feat: Coreをv2.0.0にアップデート

Gameのデータ構造がいくつか変更になりました。

Codecov Report

Merging #230 (ed53b05) into main (a67ec86) will decrease coverage by 4.47%.
The diff coverage is 48.04%.

@@            Coverage Diff             @@
##             main     #230      +/-   ##
==========================================
- Coverage   90.98%   86.52%   -4.47%
==========================================
  Files          36       36
  Lines        5748     6040     +292
  Branches      320      322       +2
==========================================
- Hits         5230     5226       -4
- Misses        504      800     +296
  Partials       14       14

Impacted Files
Coverage Δ

client/types.ts
100.00% <ø> (ø)

core/util.ts
68.42% <ø> (ø)

v1/gameStream.ts
12.25% <7.14%> (ø)

v1/parts/ai-list.ts
19.38% <18.00%> (-80.62%)
:arrow_down:

server.ts
62.88% <40.00%> (-10.09%)
:arrow_down:

core/datas.ts
87.36% <75.00%> (ø)

v1/_matches.ts
80.88% <77.77%> (-0.13%)
:arrow_down:

core/firestore.ts
84.46% <83.33%> (-3.04%)
:arrow_down:

tomakomai/_matches.ts
81.14% <88.88%> (ø)

miyakonojo/_matches.ts
75.87% <93.10%> (ø)

... and 4 more

... and 1 file with indirect coverage changes


Restart timer

Is there a way to restart a timer once the countdown is finished or to manually start the countdown (on button press for example)?

@jmaricic You have to pass a new until value to the component, you can do this in onFinish callback, once the timer  finished you assign a new until value to the global state that until value came from.
Like if your until value came from the parent component state, in onFinish callback you reset that value, or if your until value came from redux reducer you need to dispatch an action to update that value.
I think this should do the trick for you.

@jmaricic I got your point
I have pushed a fix at bfcf2a6bc21cff4b55ecc9cfc353831a5ca7f7d8
and published as 1.7.3
Thanks

Hi, I tried to reset the timer, and yes... the timer gets a new until value when the countdown finishes, but it won't start counting down again.

hi,
i have the same problem @aldocano29 , any solution ?

if you still have the problem, this solution it work for me:
https://stackoverflow.com/questions/48674827/react-native-whats-the-best-way-to-remount-reset-reload-a-screen
it is a temporally solution.

@Tatandiaz13 @aldocano29 Will push fix soon

@Tatandiaz13 @aldocano29 this should be fixed with a4140c78f01ff55a0125238bab96e6e69442a76d
But if you pass a same last until value after the counter has finished it wont get restarted

@talalmajali But if I need to pass the same value again? I'm doing a counter for a quiz and I need to reset the counter once the answer was saved.

Try this https://github.com/talalmajali/react-native-countdown-component/issues/92#issuecomment-1467393540


Multienv CloudFormation-template Git Merge Conflicts

Describe the bug
When following the recommended workflow for teams https://aws-amplify.github.io/docs/cli/multienv?sdk=js and merging sandboxes, git conflicts on cloudformation-template files for lambda functions occur and force each sandbox to do a new commit.
To Reproduce
Steps to reproduce the behavior:
Simply follow all the step on the recommended team workflow.
Expected behavior
Test just need to do git merge dev and no git conflicts should occur since test was build from a past commit in dev.
Desktop (please complete the following information):

OS: [e.g. iOS] mac
Browser [e.g. chrome, safari] chrome
Version [e.g. 22] 1.1.5

I see that according to #819 it's safe to ignore awscloudformation/ folder which was also causing git conflicts, but I don't believe it's auto generated from the CLI when initing new envs?

@richardzyx It is safe to include that in the gitignore. it will get automatically created when you're doing an amplify push.

@kaustavghosh06 thanks for the clarification, but I'm wondering if that should be in the default .gitignore or behavior for CLI? It was difficult to track down these unexpected behavior so if the CLI can take care of that out-of-the-box that would be very helpful.
Thanks!

@richardzyx If you initialize a new project using the CLI now, we do add all these files in your .gitignore files so that you don't have to think about it.

@kaustavghosh06 amplify/backend/function//-cloudformation-template.json is not ignored by default and still causes these errors.

@kaustavghosh06 amplify/backend/function//-cloudformation-template.json is not ignored by default and still causes these errors.


Point release for Kotlin 1.9.0 support

Expected Behavior
Can build with Kotlin 1.9.0
Current Behavior
1.23.0 only supports Kotlin 1.8.21
Context
Hello, would it be possible to get a point release with Kotlin 1.9.0 support? This is currently blocking our projects from upgrading to Kotlin 1.9.0.
I along with the community would be very appreciative if we could get a point release with minimal changes quickly, that will allow us to run our kotlin 1.9.0 builds with detekt. Instead of waiting weeks for a new build with proper Kotlin 1.9.0 updates for the new features.

Snapshots can be used by anyone who wishes to use a version that supports Kotlin 1.9 in the meantime: https://detekt.dev/docs/introduction/snapshots/

Hello, would it be possible to get a point release with Kotlin 1.9.0 support? This is currently blocking our projects from upgrading to Kotlin 1.9.0.

Yup I believe we can get this sorted out in the next days 👍 Thanks for your patience

detekt 1.23.1 has been released to Maven Central and has support for Kotlin 1.9


Fix warning on OS X.

When building on OS X (haven't tried others) there is a warning when building htp_util.c.
htp_util.c:2088:79: warning: format specifies type 'void *' but the argument has type 'const unsigned char *' [-Wformat-pedantic]
    fprintf(stream, "\n%s: ptr %p offset %" PRIu64 " len %" PRIu64"\n", name, data, (uint64_t) offset, (uint64_t) len);
                               ~~                                             ^~~~
                               %s
1 warning generated.

This commit fixes it by casting to a void pointer.

Merged through #144, thanks Wesley!


SomeThing Weird About The Yolact SegHead

Hi, When I look the code in mmdet/models/dense_heads/yolact_head.py/
@HEADS.register_module()
class YOLACTSegmHead(nn.Module):
def get_targets(self, segm_pred, gt_masks, gt_labels):
"""Compute semantic segmentation targets for each image.
    Args:
        segm_pred (Tensor): Predicted semantic segmentation map
            with shape (num_classes, H, W).
        gt_masks (Tensor): Ground truth masks for each image with
            the same shape of the input image.
        gt_labels (Tensor): Class indices corresponding to each box.

    Returns:
        Tensor: Semantic segmentation targets with shape
            (num_classes, H, W).
    """
    if gt_masks.size(0) == 0:
        return None
    num_classes, mask_h, mask_w = segm_pred.size()
    with torch.no_grad():
        downsampled_masks = F.interpolate(
            gt_masks.unsqueeze(0), (mask_h, mask_w),
            mode='bilinear',
            align_corners=False).squeeze(0)
        downsampled_masks = downsampled_masks.gt(0.5).float()
        segm_targets = torch.zeros_like(segm_pred, requires_grad=False)
        for obj_idx in range(downsampled_masks.size(0)):
            segm_targets[gt_labels[obj_idx] - 1] = torch.max(
                segm_targets[gt_labels[obj_idx] - 1],
                downsampled_masks[obj_idx])
        return segm_targets

Why using gt_labels[obj_idx]-1? When using coco datasets provided by mmdet, the category label has converted to [0,79], I think there is no need to -1 here. Can someone explain it?

@zzzbosss1997 Hi, Do you have any questions now?


System Common messages support added

Will now pass MTC quarter frame, song position select, song select and tune request messages.

It's very cool.
I'm digging it for MTC, but so hard...


link name without file path

Hi Marc Julian,
Thank you for the plugin, i like it very much!
Is it possible to only have the name of the file in the link without the complete file path? At the moment, I remove the file path manually.
Thank you
mkNucleus

Hi,
thank you for the feedback.
Are you using the plugin on windows? It can be the case that it won't display correctly on windows because I mainly tested it on macOS.
I will make sure to fix that with the next release.

Thanks for your quick response.
Correct, I use it on windows.
Then i look forward to the next release.

Sounds great, thanks again.


Documentation link on readme is broken.

Oh, I forgot about that, this was for the times when hackage's doc builder was having some troubles. Now pointing to the hackage docs directly, thanks :+1:


Change where rooms frontend assets are served

Need to change the location of where the frontend dist files are being served to make it easier for them to be replaced / overwritten by either manually (by someone) or automatic (through ansible)
The idea is that when someone creates a new production build of rooms-frontend (which each individual room will need to do in order to apply custom environment variables to the rooms-frontend), this change makes it a lot easier to overwrite the whole folder, rather than figuring out manually which ones need to be replaced/removed. This will then be a lot easier for the ansible script that will automate that process.
Proposed solution:

instead of serving them at go-ssb-room/web/assets they are served at go-ssb-room/web/assets/app or something similar
need to update the route that points to the assets, to point to this subfolder instead

@chereseeriepa https://github.com/planetary-social/go-ssb-room/pull/3/commits/5e2a2da80877facb1d80a7e8508a82f74799e9d7
I hcanged the way the assets are loaded. First we try to load
assets from web/app and if that fails we load them from web/assets. This
means that files can be in either one of those directories with web/app
taking priority.
So basically you should be able to move all your app files to web/app and keep them there. I moved 3 files for now as I wasn't sure which files belong to the frontend app and which are the part of the rooms.
Let me know if this works for you.

Ok for now I fixed the dev server to also serve assets from web/app and then from web/assets if they are missing. This means that it should behave like the production server.
Do we also want to do the other requirement as part of this ticket ("by default, it should show the old UI when there is no web/app (its currently showing a list of the assets on the landing page instead") or is there a separate one for this?


Enable automatic OperationID conflicts resolving

I'm trying to merge several swagger files from similar APIs. Each swagger file contains the ping endpoint having the same OperationID:
"/ping": {       "get": {         "operationId": "getPing",
This results in a conflict. --continueOnError does not help, the combined file is not generated.
IMHO, my use case is rather common, so can I open a PR that would automatically rename getPing to getPing1 or fix it in any other way?
Suggestions appreciated

In our case, we have conflicts on operation Ids, so we simply removed them before merging
    # Swagger specification https://swagger.io/specification/
    # - the operationId field must be unique, it used for the code generation but does not matter here, get rid of it
    sed -i -e "/^.*operationId:.*$/d" tmp/src/*.yaml

Look at "Renaming Path OperationIds"  in https://www.npmjs.com/package/swagger-combine
if you cannot track all the conflict an automatic renaming option could be nice.


Update Documentation or Expose Constants

Currently documentation seems to imply that constants are on the library:
import Hammer from 'react-hammerjs';

console.log(Hammer.DIRECTION_LEFT) //actually undefined
console.log(window.Hammer.DIRECTION_LEFT) //works

In any case, I would vote that the underlying Hammer object shouldn't be exposed globally and should be contained to the package.

Yes, i need it too ^^
but about window.Hammer, there is a proper way for doing this :
import Hammer from 'react-hammerjs';
import { DIRECTION_LEFT, DIRECTION_RIGHT } from 'hammerjs';

because hammerjs is a dependency of react-hammerjs


Refactor abandon claim based on comments in previous issue

related: #768

Fixed in https://github.com/lbryio/lbry/issues/768#issuecomment-410540424 , closing


Only read up to Content-Length in stream wrapper

This commit updates the stream wrapper to only read up to the number of
bytes returned in the Content-Length header when draining a stream
synchronously.
@Yanacek @Tobion

Note that the test failure is due to an unrelated change in a package Guzzle depends on. @Tobion is addressing this issue.

Good feedback, @Tobion. I've pushed up a commit to address the feedback.

Responses to HEAD requests can sent a Content-Length header that is not actually the size of the response body:

in the case of the HEAD method, the size of the entity-body that would have been sent had the request been a GET.

Is it handled somewhere that certain status codes or HEAD responses must not have a body?

Looking at the implementation of copy_to_stream ( https://github.com/guzzle/psr7/blob/master/src/functions.php#L385 ) it seems that it will require the amount of memory that is given with the maxLength size. So if the response is 1 GB, it will also require 1 GB of memory to copy the stream. This doesn't seem right. Shouldn't it read the stream in batches just as it is done when $maxLen === -1?

Yeah, this is not taking HEAD requests into account. I'll push a fix.
As for copy_to_stream, you're right. The implementation could be improved.

No longer draining the stream for HEAD requests. The poor maxLen implementation can be addressed separately on the PSR-7 repo. Rebased on master, so hopefully the tests will pass now.

createSink still fopens a stream even for HEAD requests. This doesn't seem necessary.

Nice catch. I pushed a commit to no longer create a new sink when sending a HEAD request.

:+1:


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/Greiler21/iobroker.conbee2

[ ] [E003] Repository must have name ioBroker.adaptername, but now io"b"roker is in lowercase
[ ] [E004] No adapter name found in URL: https://raw.githubusercontent.com/Greiler21/iobroker.conbee2/master
[ ] [E020] Name of adapter in package.json must be lowercase and be equal to "iobroker.". Now is "undefined"
[ ] [E103] common.name in io-package.json must be equal to "". Now is conbee2
[ ] [E150] No common.connectionType found in io-package.json
[ ] [E152] No common.dataSource found in io-package.json
[ ] [E125] External icon not found under URL: https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.conbee2/master/admin/deconz.png
[ ] [E200] Not found on npm. Please publish
[ ] [E300] Not found on travis. Please setup travis
[ ] [E801] No repository about text found. Please go to "https://github.com/Greiler21/iobroker.conbee2", press the settings button beside the about title and add the description.
[ ] [E802] No topics found in the repository. Please go to "https://github.com/Greiler21/iobroker.conbee2", press the settings button beside the about title and add some topics.
[ ] [E603] NO "## Changelog" found in README.md
[ ] [E604] No "## License" found in README.md
[ ] [E816] file tsconfig.json found in repository, but not found in .npmignore

I have also found warnings that may be fixed if possible.

[ ] [W400] Cannot find "" in latest repository
[ ] [W505] setTimeout found in "main.js", but no clearTimeout detected
[ ] [W901] .gitignore not found

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Do you need help fixing the bugs?

Do you need help fixing the bugs?

Thanks, that all bugs have been fixed.


generated output don't follow moduleSuffixes option

Bug Report

typescript generated output don't follow moduleSuffixes option,
🔎 Search Terms
searched terms: moduleSuffixes, result none

🕗 Version & Regression Information
<EMAIL_ADDRESS>and typescript@next(4.8.0-dev.20220616)

💻 Code
create 2 ts files
a.web.ts
export const a = "web"

a.node.ts
export const a = "node"

create index.ts
import {a} from './a'

create tsconfig.json with
{
"moduleSuffixes":[".node",""]
}

run tsc -w
stackblitz link

🙁 Actual behavior

generated index.js  directly imports a from './a'
🙂 Expected behavior
generated index.js  directly imports a from './a.node' according to moduleSuffixes

Nothing in TS re-writes import paths. They only inform TS about existing path mappings.

according to moduleSuffixes

What makes you think that's the purpose of moduleSuffixes? I couldn't find any documentation regarding this flag. Related: https://github.com/microsoft/TypeScript-Website/issues/2423

according to moduleSuffixes

What makes you think that's the purpose of moduleSuffixes? I couldn't find any documentation regarding this flag. Related: microsoft/TypeScript-Website#2423

i read that from blog here , so i naively think that that's the purpose

Sorry it wasn't more explicit - we won't do path rewriting. moduleSuffixes is meant to inform TypeScript of how your runtime/resolver will resolve the paths.
So if you have a bundler or resolver plugin that will try to look up .node-suffixed name, moduleSuffixes will allow TypeScript to understand that - but TypeScript will defer to that bundler/resolver to do that later and not touch the paths. React Native's Metro is the most well-known example of a bundler with a resolution strategy like this.

I got the same issue. So it's working as intended or not, maybe should ask @afoxman .
#48189

It will resolve that way, but we won't rewrite the paths when we output JavaScript code. We expect that a loader of some sort will prefer foo.node.js over foo.js.
If something isn't working for you, I'd guess it's probably because Node.js doesn't do that by default. You'll need to find some sort of loader hook for it.

Sorry this isn't clear. moduleSuffixes does appear to be working correctly. As @DanielRosenwasser said, the intent is to teach TypeScript how modules are resolved by apps like Webpack/Metro. Not to change emitted code.
This lets TypeScript build a matching file graph and run type-checking on it. For example, when building a React Native Android app, Metro will only bundle .android.ts files, it won't bundle .ios.ts files. So .ios.ts files should not be considered when type-checking Android builds.

@afoxman @DanielRosenwasser
😂My fault, I misunderstood what the issue publisher meant.
Rewrite is wrong.
But I do have the error of load logic though. Maybe I should publish a new issue.
Thx reply.

I feel that documentation on https://www.typescriptlang.org/tsconfig#compilerOptions is very poor for paths and suffixes if this is the behaviour. It should clearly state that this will not change the output of TSC

I found that here: https://www.typescriptlang.org/docs/handbook/module-resolution.html#additional-module-resolution-flags.
But this should be much more prominent on the tsconfig page, since this is the first entry point in looking for these options.


Issue in amalgamation

~/mxnet/amalgamation$ make
g++ -std=c++11 -Wno-unknown-pragmas -Wall -I/opt/OpenBLAS -fPIC -o mxnet_predict-all.o -c mxnet_predict-all.cc
mxnet_predict-all.cc:31:18: fatal error: cuda.h: No such file or directory
#include <cuda.h>
^
compilation terminated.
make: *** [mxnet_predict-all.o] Error 1
I have built OpenBlas 0.2.16 . OpenBLAS root: /opt/OpenBLAS

I think this should be solved by adding cuda_fp16.h and cuda.h into blacklist in
https://github.com/dmlc/mxnet/blob/master/amalgamation/amalgamation.py#L4
Just sent a PR.

No, I am not using this script. I am getting this issue when I am trying to execute 'make' command

make will call this script. See https://github.com/dmlc/mxnet/blob/master/amalgamation/Makefile#L35

Okay. Now I am getting another error:
Getting following error when trying to execute make
:~/mxnet/amalgamation$ make
g++ -std=c++11 -Wno-unknown-pragmas -Wall -I/opt/openblas -shared -o pwd/../lib/libmxnet_predict.so mxnet_predict-all.o -L/opt/openblas -lopenblas -lrt
/usr/bin/ld: cannot find -lopenblas
collect2: error: ld returned 1 exit status
make: *** [pwd/../lib/libmxnet_predict.so] Error 1

The compiler can't find the openblas, please check the installation of openblas.


Microsoft App Center integration updates

Hi @wswebcreation -- made a couple minor adjustments to the App Center page. Would you like to take a look?

Thank you for contributing to the sauce-docs project!
A well described pull request helps maintainers quickly review and merge your change
Before submitting your PR, please check our contributing
guidelines, applied for this repository. Avoid large PRs, help reviewers by making them as simple
and short as possible.

Description

Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[x] I have read the contributing document.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.

LGTM!

Perfect -- thanks for the quick response!


Segfault During Testing with --with-threads=1

Issue description
While running tarpaulin on an integration test with the --test-threads=1 flag, the test segfaults.  I did try to cut the code base and test code down to a minimal test case.  I suspect I only achieved minimal-ish, but it's much smaller than the original project.
Steps to reproduce the issue

git clone git@github.com:jeffrey-howard/minimal.git
cd minimal
cargo tarpaulin --verbose -- --test-threads=1

What's the expected result?

The expected result is that tarpaulin finishes the test, the test passes, and tarpaulin produces coverage results.  (As a side note, this is exactly what happens when running tarpaulin without the --test-threads=1 command line argument.)

What's the actual result?

The test run concludes in a segmentation fault, as follows:

[INFO tarpaulin] Running Tarpaulin
[INFO tarpaulin] Building project
Finished dev [unoptimized + debuginfo] target(s) in 0.10s
[DEBUG tarpaulin] Processing Target(lib)
[INFO tarpaulin] Launching test
[INFO tarpaulin] running /home/ec2-user/environment/minimal/target/debug/deps/minimal-e862dd173fc29ea3
running 0 tests
test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
[DEBUG tarpaulin] Processing Target(test: server_test)
[INFO tarpaulin] Launching test
[INFO tarpaulin] running /home/ec2-user/environment/minimal/target/debug/deps/server_test-9279f386a85d2063
running 1 test
test server_bad_address ... Error: "Failed to get test coverage! Error: Failed to run tests: A segfault occurred while executing tests"

Additional details
Output from uname -a:

Linux [IP REMOVED] 4.14.146-93.123.amzn1.x86_64 #1 SMP Tue Sep 24 00:45:23 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Tarpaulin version: 0.10.0
Rustc version: rustc 1.40.0 (73528e339 2019-12-16)
As I was removing as much code as possible to try to get to a minimal project showing this behavior, I did try stripping out the error reporting, at which point the test seemed to run fine.  That makes me wonder if it's the thread communication in lib.rs on lines 61 and 94 that might be causing the segfault.

This is a known issue and relates to #190 there was a change in how tests were launched for the the instance where test threads is 1 in cargo test and tarpaulin can't reliably follow the ptrace event. I think I have to find a way to improve the following down of executed binaries but that opens a whole can of worms with instrumenting the binaries when they're launched that needs to be figured out.
Is there a reason you need test-threads 1?

Ah, my apologies for reporting a duplicate issue.  I missed that one when looking through the issue list.
In our full test suite, it's a mix.  We don't need it for the unit tests, but a goodly amount of our coverage is in integration tests that make queries against a database back-end. Setting test-threads=1 forces those integration tests to run serially, so that the database is always in a predictable state during each test case.  That may be a naive approach, though. Until now we haven't had reason to look for a better way.

I have found a work-around.  There's a project at https://github.com/palfrey/serial_test that provides attribute-like procedural macros to force tests to be executed serially.  With it, I can make sure the tests run serially, but without using test-threads=1, so tarpaulin runs just fine.

Another approach I've seen used is a static mutex for tests to acquire. Rust nix uses this for Unix signal based tests

That makes sense.  In fact, I'm pretty sure the serial_test project is a pretty syntactic wrapper around exactly that.

So I just came back and tested this out on the 0.1.18-alpha1 and there was no segfault though I could recreate it in 0.1.16. So looks like this is fixed - although I'll keep the issue open until the alpha is ready to transition away from alpha status and is released in a way most users will use it :grin:


#124 Remove undocumented overwrite parameter

removed 'overwrite'
cleaned code affected by the removal of 'overwrite'

@josephca After you merge with Patrick's changes, we can integrate this PR.


Update querying-data-in-a-system-versioned-temporal-table.md

Line 29: added "When using a table alias with a temporal tables, the FOR SYSTEM_TIME clause must included between the temporal table name and the alias (see "Query for a specific time using the AS OF sub-clause" second example, below)."
If someone can make "Query for a specific time using the AS OF sub-clause" a hyperlink, that would be cool.

@JediSQL : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Vendor embedding for "individual_mouse"

We have an Enzyme object that has the "Vendor" embedded under the name "enzyme source".
I tried embedding "Vendor" for "individual_mouse" object under the name "mouse_vendor". If I put a mouse_vendor under insterts for individual_mouse, it fails to load. If take it out everything is fine.
Tried to find the reason but failed, will try again later.

@aschroed Solved the problem, it was the name validation step in vendor.py that changed the names.


solvaholic/octodns-sync checks refs/pull/NNN/merge rather than head

Description

I think when this action is triggered on the pull_request event it's checking the config in refs/pull/NNN/merge rather than refs/pull/NNN/head.
Expected Behavior

When this action adds a comment to a pull request, I expect the commit SHA it names to match the last commit pushed to the pull request head ref.
Actual Behavior

The commit this action mentions in the PR comment may differ from the most recent one pushed to the PR head:

Possible Fix

Configure this action to check the config in refs/pull/NNN/head when the event name is pull_request.
Install a knob so users can configure this behavior?

Steps to Reproduce

Tell solvaholic/octodns-sync to add a pull request comment
Trigger solvaholic/octodns-sync on the pull_request event
Notice the comment added to PR names refs/pull/NNN/merge rather than refs/pull/NNN/head

Context

The mismatch can be scary until you realize what it's checking.
In case it matters: The PR where I noticed this, its base is NOT the default branch in the repository.
Your Environment

Version used: @main
Link to your project: n/a (it's a private repo)

Is this just an issue with the workflow definition?
https://github.com/solvaholic/octodns-sync/issues/41#issuecomment-780041826
The initial example here was in a private repository. I'll try it out in a public one so I can link it.


fix(Client): user and url flow

Description

Added initial code of conduct
Added initial contribution
Changes som routing
added navigation to context change
Renamed Context-page to Project Page

Checklist

[x] My code follows the style guidelines of this project
[ ] Documentation and comments is added where needed.
[ ] My code is covered by tests.
[ ] UX has reviewed relevant UI contributions.

Definition of Done

[ ] PR title and description are to the point and understandable
[ ] I have performed a self-review of my own code

Two notes/questions

Im seeing a lot of regex, whats up with that? Is there a better way to do this?
Why are we not using some form of router in the portal, im seeing a lot of window.location

Im seeing a lot of regex, whats up with that? Is there a better way to do this?
Why are we not using some form of router in the portal, im seeing a lot of window. Location

Sorry to say but i dont think there is any better way. all code using regex is outside of react scope so routing is not an option


TypeError: accept(): incompatible function arguments.

Hi @KugaMaxx , I have been facing this error when I run python3 demo.py:
starting demo, noise sequence length is 1034210
===================================
Model          Length    Runtime
-----------------------------------
knoise     Traceback (most recent call last):
  File "/home/ubuntu/slocal/cuke-emlb/demo.py", line 67, in <module>
    model.accept(data["events"])
  File "/home/ubuntu/slocal/cuke-emlb/configs/denoisors.py", line 26, in accept
    self.model.accept(events)
TypeError: accept(): incompatible function arguments. The following argument types are supported:
    1. (self: modules.python.khodamoradi_noise.init, events: dv::toolkit::EventStorage) -> None

Invoked with: <modules.python.khodamoradi_noise.init object at 0x7f4ea7c7f930>, Storage containing 1034210 elements within 749994µs duration; time range within [1643505112036837;<PHONE_NUMBER>786831]

When i run eval_denoiser.py:
python3 eval_denoisor.py --file ./data/demo/samples/demo-01.aedat4 --denoisor ynoise
TypeError: Unregistered type : dv::AddressableEventStorage<dv::Event, dv::EventPacket>

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/ubuntu/slocal/cuke-emlb/eval_denoisor.py", line 36, in <module>
    score = metric.evalEventStorePerNumber(data["events"].toEventStore())
                                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: Unable to convert function return value to a Python type! The signature was
        (self: dv_toolkit.lib._lib_toolkit.EventStorage) -> dv::AddressableEventStorage<dv::Event, dv::EventPacket>

Could you help me on how I can solve this?
Best, Shintaro

Hi @shiba24, I cannot reproduce your error😞, as the code runs smoothly on my machine. I would appreciate if you could provide more clues.

I have the same error after smooth installation, when I start demo script as adviced in readme.

@KugaMaxx I am using venv not conda. But if @karaul uses conda then that should be no problem. I will post my environment details later this week.
@karaul could you also share about your environment for this error?

I am working with ubuntu 22 and conda. The old vesrion from July 2023 worked fine. The problem is probably in a new submodule  dv-toolikt (yam-toolkit). Later I'll try to check the suspicious  submodel independently.

@KugaMaxx please remove  commas in readme file (mistype)
sudo apt-get install libboost-dev, libopencv-dev, libeigen3-dev, libopenblas-dev

Hi @KugaMaxx (and cc @karaul ),
my environment is as follows:
Python related:
python 3.11

$ pip3 freeze
contourpy==1.2.0
cycler==0.12.1
dv-processing==1.7.9
dv_toolkit @ file:///home/ubuntu/slocal/cuke-emlb/external/dv-toolkit
fonttools==4.49.0
install==1.3.5
kiwisolver==1.4.5
matplotlib==3.8.3
numpy==1.26.4
packaging==24.0
pillow==10.2.0
plotly==5.20.0
pyparsing==3.1.2
python-dateutil==2.9.0.post0
six==1.16.0
tabulate==0.9.0
tenacity==8.2.3
tqdm==4.66.2

dv-related (checked with apt search):
dv-runtime-dev/jammy,now 1.6.2-2~jammy amd64 [installed]
  C++ event-based processing framework for neuromorphic cameras (development files)

dv-processing/jammy,now 1.7.9-1~jammy amd64 [installed]
  Generic algorithms for event cameras (C++ headers).

dv-processing-utilities/jammy,now 1.7.9-1~jammy amd64 [installed,automatic]
  Generic algorithms for event cameras (CLI utilities).

Others:
$ gcc-10 --version
gcc-10 (Ubuntu 10.5.0-1ubuntu1~22.04) 10.5.0
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

$ g++-10 --version
g++-10 (Ubuntu 10.5.0-1ubuntu1~22.04) 10.5.0
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 22.04.3 LTS
Release:        22.04
Codename:       jammy

$ uname -r
6.5.0-17-generic

I have two questions:

While checking the above, I found out that the following apt package is not installed. Do I need this?

dv-processing-python/jammy 1.7.9-1~jammy amd64
  Generic algorithms for event cameras (Python bindings).

I'm wondering if I installed pybind correctly. Do you know how I can check if the pybind installation is correct?

Anyone managed to solve this issue? I am also facing the same problem when I run demo.py and eval_denoiser.py

I have solved the problem in another way specific for my project, and do not know what is a reason.

@karaul do you have any idea that worked for you?

One needs carefully check version of packages and their mutual interfere, it's a standard headache in python. My plans were to use this software in on order to analyse noise statistical propereties for https://arxiv.org/abs/2404.01948, but because of technical difficulties of the latest version,  I just returned to the earlier version of cuke-emlb published in summer 2023 (see above), that worked fine for me.

Unfortunately, I have met the same question. Is seem like the data struct with dv.toolkit can not be used in the mode.chh files.
It's very nice for you if the problem solved~

I tried to analyze the source code and wrote test code in C++, and the conclusion I got was that the input should be a match: The data type for Storage containing 1034210 elements within 749994 µ s duration; The data type for time range within [1643505112036837;<PHONE_NUMBER>786831] is events: dv:: toolkit:: EventStorage, so I am confused and hope you can provide some help or suggestions.


Aaronhmiller patch 1

NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Kong/getkong.org/blob/master/CONTRIBUTING.md
Summary
SUMMARY_GOES_HERE
Full changelog

[Edited docs to ...]
[Added cool new feature ...]
...

Issues resolved
Fix #XXX

Checklist:

[ ] Spellchecked my documentation
[ ] Documentation
[ ] Ready to be merged

We're getting feedback from the field asking why "Create Developer Account" button doesn't work. Positioning in doc more accurately so we needn't verbally explain each time and make users think something is wrong on first touch.

Any blockers to merging this in?


TextField ignores defaultValue property changes

TextField properly paints defaultValue first time but ignores all further updates of defaulltValue.
See the demo of the bug:

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
TextField should always show current defaultValue.
Current Behavior
Only initial defaultValue is shown, all updates are ignored.
Steps to Reproduce (for bugs)
https://codesandbox.io/s/r50qpkywkp

See that value of text field isn't changed after timeout (when defaultValue is changed).

Context
The real use case is to paint empty form, then load data via AJAX and fill in form.
Some libs (for example https://github.com/jeanlescure/material-ui-schema-form) uses TextField's defaultValue for initial field rendering, so if you update defaultValue property for already mounted TextField, the new value is simply ignored.
Your Environment
The latest version of Material-UI next branch and React. Any browser.

@oliviertassinari  native input defaultValue without value property is always update

@HelloLove Do you have an example that supports this claim? It's not working here https://codesandbox.io/s/q9vlvxjw54.

Hi thanks for your hardwork on this project,
I have this issue is there a solution I can apply to my project.
Thanks.


Make judgements fully opaque

It would be nice if Judgements could be made fully opaque. Currently the stage lighting affects the colors of your judgements slightly, which makes reading them difficult with wild Chroma powered lighting effects.

I can't find any specific commit that might have fixed this, but it looks like with the newest version all judgements are opague now?
I hope that was intended and whatever was changed won't be reverted.
Thanks alot!

My response was kinda late as I didn't know you had created this issue (please also let me know on Discord next time) 😅
That being said, the change was caused by a change in how the basegame displays note cut scores since 1.13.4 and I'm planning on offering both possibilities as an option once I've figured out a way to revert it to the old way.

No problem, the solution sounds good to me!


Voxel size error

Hi,
In our data, the voxel size 8nm, not 5nm. I update this information accordingly in the .json file under incasem/scripts/03_predict/data_configs/ . However, when I run the predict.py script, I receive the error below. Do you know how I can troubleshoot this? Do I need to update the voxel size elsewhere, e.g. during the .tiff to .zarr conversion or in another .json or .yaml file?
ERROR:example_prediction_test_ER:Failed after 0:00:00!
Traceback (most recent calls WITHOUT Sacred internals):
File "/content/incasem/scripts/03_predict/predict.py", line 326, in predict
predictions = multiple_prediction_setup(
File "/content/incasem/scripts/03_predict/predict.py", line 126, in multiple_prediction_setup
prediction_setup(
File "/content/incasem/scripts/03_predict/predict.py", line 144, in prediction_setup
prediction = pipeline_type(
File "/content/incasem/incasem/pipeline/prediction_baseline.py", line 44, in init
self._assemble_pipeline()
File "/content/incasem/incasem/pipeline/prediction_baseline.py", line 71, in _assemble_pipeline
raise ValueError((
ValueError: Specfied voxel size (5, 5, 5) does not match the voxel size of the datasets (8, 8, 8).

Hi Patrick,
Thanks for your quick response. I just did it but I still receive the same error. Is there any other document (.yaml or .json) I need to update? Below is my current .zattrs and .json files:
.zattrs files under both raw_equalized_0.02 and raw folders.
{
"offset": [
0,
0,
0
],
"resolution": [
8,
8,
8
]
}
.json file under incasem/scripts/03_predict/data_configs/
{
"test_example_roi_nickname" : {
"file": "/content/incasem/data/test.zarr",
"offset": [8, 8, 8],
"shape": [204, 204, 204],
"voxel_size": [8, 8, 8],
"raw": "volumes/raw_equalized_0.02"
}
}

I'm using the provided models (1841 in this case). Below are the command lines for tiff to zarr conversion and ER prediction. I'm attaching the .json and .zattrs files.
zattrs content.txt
[test.json](https://github.com/kirchhausenlab/incasem/files/14306310/test.json
python 00_image_sequences_to_zarr.py --resolution 8 8 8 -i /content/incasem/data/test -f /content/incasem/data/test.zarr
python 40_equalize_histogram.py -f /content/incasem/data/test.zarr -d volumes/raw -o volumes/raw_equalized_0.02
python predict.py --run_id 1841 --name example_prediction_test_ER with config_prediction.yaml 'prediction.data=data_configs/test.json' 'prediction.checkpoint=/content/incasem/models/pretrained_checkpoints/model_checkpoint_1841_er_CF.pt'

This is very helpful, thank you so much! I'll try these steps. Thanks for the clarification on model's association with other files.


Add progress state to curriculum index in learning interface

What's the motivation behind the feature? what problem does the feature intend to solve ?
I want to see my progress that is reflected in the curriculum overview page shown below:

be reflected in the curriculum index in learning interface:

Stale and non-relevant for Phase 2 of the platform.


Add input component

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Add an input component
Examples 🌈
The input is one of the most-used components, in MUI and for instance in https://www.youtube.com/watch?v=7GgBnbHF8Rg, is pivotal to be able to search inside a data grid or to create forms.

Motivation 🔦
I'm working on a support tool for MUI X to decode the license keys. It would have been great to have access to this component.

I believe the TextField component already does this. If that's the case, then this is probably a discoverability issue. We could rename it to "Input" and/or work on the design of the component catalog?
If not, what would the Input component do that the TextField doesn't?

Ah right, I didn't thought about searching for the the text field 🙃. We will need a search to find components, I believe, e.g. like Figma is doing.

I also think it could help to group components as the catalog grows. Like for instance in the docs: inputs together, layout together,...

And of course, illustrations 😁. I'm adding the UX label as I think we need to involve a designer on this issue

Same as https://github.com/mui/mui-toolpad/issues/344


Remove references to install-build-deps.sh

Now that we're using sysroot images from Chromium, no libraries
needs to be installed on the system in order to build on Linux.
I also verified that the libpulse-dev package is part of that image.

@KaptenJansson: we don't need reviews for changes on GitHub? I just submitted my #41 (assuming that was OK).

I added info about https://bugs.chromium.org/p/webrtc/issues/detail?id=5578 as well to the instructions.

LGTM


Cache locally results and reuse them (instead of reading from s3)

Coverage decreased (-3.6%) to 81.293% when pulling 66cc2209d3cc4905f1eaf2d519fc31127fbc96ee on fix/cache-locally-for-performance into d818b11eeb11dac7c16ad5a4a994cb2e772fd356 on master.


feat(EbbRef) Centralized and explosed Id to Ref Address calculation

In order to make it easier to extend the object model I unified
the calculation of a Ref Address from an Id in a method:
uintptr_t ebbrt::IdToRefAddr(const EbbId id)

There were the following issues with your Pull Request

Commit: fd306f6ee58a9cbf1c3f72d5fc8d4b7a04481683
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/SESA/EbbRT/blob/master/CONTRIBUTING.md

This message was auto-generated by https://gitcop.com

@BetsyRT r+

:pushpin: Commit bedfaad has been approved by jmcadden

@BetsyRT r+

:pushpin: Commit bedfaad has been approved by jmcadden

:bulb: This pull request was already approved, no need to approve it again.

@BetsyRT try

@BetsyRT clean

@BetsyRT try


Install diener 0.4.4

Feels like it belongs in some Dockerfile...
Do we have a fixed image for this?

I should've marked as draft yesterday. Yes, it's placed in paritytech/ci-linux image but I suggested to @mordamax install diener from here but it's not the final decision. The reason - I updated diener yesterday and it broke comapinon-build (because I honestly didn't know that it could). We'll discuss it later today.


Update translation strings as of 20201030T134930

Please do not manually merge this pull request.  It will either be automatically merged or withdrawn.

Test errors found. See https://jenkins-aws.indexdata.com/job/folio-org/job/ui-plugin-find-package-title/job/PR-30/2/

Test errors found. See https://jenkins-aws.indexdata.com/job/folio-org/job/ui-plugin-find-package-title/job/PR-30/3/

Test errors found. See https://jenkins-aws.indexdata.com/job/folio-org/job/ui-plugin-find-package-title/job/PR-30/4/


refactor(types): make user options extend axios request options

this removes the need for allowing arbitrary additional properties without adding too much axios-specific code
also, adds types to a few other local variables

It occurred to me this morning that, since axios defines their own types, we could use their request options as strict parameters instead of allowing additional properties on the base service constructor options. I think this makes for a much better TypeScript experience, since the additional properties makes the existing types worth little more than documentation.
I also went ahead and added some other types to a few local variables in the request wrapper class. Minor changes but the more types we define the more benefit we get out of using TS.

After some discussion with @mkistler, we are going to close this and re-evaluate the possibility of writing our own interface on top of whatever request library we may be using


Increase test coverage

Codecov Report

Merging #86 into master will increase coverage by 1.03%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master    #86      +/-   ##
========================================
+ Coverage   81.97%    83%   +1.03%
========================================
  Files          47     47
  Lines        1187   1212      +25
========================================
+ Hits          973   1006      +33
+ Misses        214    206       -8

Impacted Files
Coverage Δ

ArithmeticToolsTests/SequenceTypeTests.swift
100% <100%> (ø)
:arrow_up:

ArithmeticToolsTests/RandomTests.swift
100% <100%> (ø)
:arrow_up:

ArithmeticToolsTests/NumberFormattingTests.swift
100% <100%> (ø)
:arrow_up:

...meticTools/BidirectionalCollectionExtensions.swift
88.23% <0%> (+5.88%)
:arrow_up:

ArithmeticTools/StringFormatting.swift
100% <0%> (+28.57%)
:arrow_up:

ArithmeticTools/RandomProducing.swift
100% <0%> (+33.33%)
:arrow_up:

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 9ba8754...725891e. Read the comment docs.

This is excellent!


Define constants or literal operators to deal with MiB, GiB, etc.

Sometimes we need to say things like "2 GiB" and it would be nice to have a literal operator and/or constant to avoid writing 2 * 1024 * 1024 * 1024 or 2 * (1 << 30).

Something like this?
// `unsigned long long` parameter required for integer literal.
long long operator "" _GiB(unsigned long long val) {
  return val * 1024 * 1024 * 1024;
}

or
// `unsigned long long` parameter required for integer literal.
long long operator "" _GiB(unsigned long long val) {
  return val * (1 << 30);
}

std::cout << 1_GiB << '\n'; //1073741824
std::cout << 2_GiB << '\n'; //2147483648

I like user-defined literals, but given the other discussion around them (#109), I think maybe this:
template <typename T /* maybe with a std::enable_if<is_integral<T>::value> */>
std::int64_t GiB(T value) {
  return value * (1LL << 30);
}

or maybe this:
constexpr std::int64_t GiB = std::int64_t(1) << 30;

Or alternatively, just close this bug, I am not sure these constants would be that useful.

I will close it.
Those constants do not seem to be very useful.