stream settings: Refactor subscriptions update functions.

There are multiple elements in subscription page(sub button, subscriber list, privacy button, delete button), and whether the user is authorized to click or even see the element, depends on various different conditions. Currently our templates also includes this complex conditions when rendering these elements. After elements are rendered, when subscription data(sub.subscribed, sub.invite_only etc.) is changed, we again hide/show/disable these elements.
This PR refactors the functions which updates/rerenders the subscriptions/stream templates on different events(subscribe/unsubscribe/changes stream privacy) etc.
New file stream_ui_updates.js includes different update-stream-settings-ui-elements functions.
This PR is currently WIP as there is still some functions to migrate. However, starting commits can be merged.
Testing Plan:
As I extract new function for any stream-ui-element, I carefully tests the ui-element behavior in all possible cases(Non admin/Admin/Guest user user subscribed to public, private streams, previously private streams), although if others test the PR locally please left a comment, it would be really helpful!

@timabbott Thanks for review! I have updated the PR and also add some changes to fix inconsistency issues between UI changes and URL changes in stream settings.
Here is the gif, in which following error are fixed:

When I unsubscribe stream by clicking on "check button" it don't open stream settings and also don't change URL.
In current version: It changes the URL but don't open stream settings, which is inconsistent.
When I click on stream-header, or stream-list to dis-select stream, it opens nothing-selected tab and also changes URL to either #streams/subscribed or #streams/all according to selected tab.
In current version: It changes the settings to nothing-selected tab but don't update the URL.

This is great.  Merged as 7a168216ff0056ead3a477ee7055177d7999dc98, after shortening the commit message titles (we never include filename extensions before the :, and often just name the subsystem, not the file, e.g. streams: could have been the prefix for all of these commits).
Thanks @YJDave!


Fix #19030: Add DI container usage to Widget end()

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Fixed issues
#19030

Hm, maybe just drop the @covers - not really needed and looks like it's messing the tests

@bizley test method not match by pattern test{$method}, @covers define tested method(s)

can you help me what should i change? i am not that familiar with test docs
mybe test method need to change to testDIBeginEnd

Not really sure about what WinterSilence said but I'm pretty sure that @covers is not necessary. First remove it, and if it's still failing maybe change the name of the test from testDI to something longer like testDependencyInjection.

Not really sure about what WinterSilence said but I'm pretty sure that @covers is not necessary. First remove it, and if it's still failing maybe change the name of the test from testDI to something longer like testDependencyInjection.

i did it. i hope now it will be ok.

@bizley it's work in any case, tags need to Code Coverage Analysis

@papppeter if your test method not match by pattern testMethodName(), then you can define covered method by @covers, see  phpunit manual

No, tags are not needed for coverage to work, it's just a pointer for the user.

I am happy with both solution. But @covers failed if i see it right without it all checks are passing. maybe i did not set @covers in the right way. I accept any solution you suggest.

Thanks.


The workflows icon is incorrect for build a workflow query option in the entity explorer

The icon used for the option to create a new workflow query has a black fill in the first circle which is not part of the logo.
Incorrect logo

Correct logo

Please add your planning poker estimate with Zenhub @ayushpahwa


Remove .open monkey patching in the Bag class

https://github.com/foxglove/rosbag/blob/main/src/node/index.ts#L88
Rather than having node and web folders that re-export the same things and add a reader by money patching the .open static method on the Bag class, we can have two separate classes NodeFileReader and DomFileReader within src and the user can use the one appropriate for their platform directly when making a Bag instance.
We should also combine the Bag class and the BagReader so that initializing becomes:
const bag = new Bag(new NodeFileReader(filename));
// or maybe
const bag = await Bag.open(new DomFileReader(blobOrFile));

Thus eliminating keeping the entry points in sync and making it explicit at the call site which reader is used.

we can have two separate classes NodeFileReader and DomFileReader within src

If we build these with the same tsconfig, there is no way to have a lib setting that differs between files. It might be preferable to just have multiple build targets in the package, or even just multiple packages as we do with rosbag2-node/web.


Notebook Intellisense crashed 5 times during diffing

Type: Bug
Got an error that notebook-intellisense crashed 5 times. I had a notebook diff open in GH PRI.
TypeError: Cannot read properties of undefined (reading 'range')
    at m.handleOpen (/home/vscode/.vscode-server-insiders/extensions/ms-python.vscode-pylance-2022.11.10/dist/pylance-internal/node_modules/@vscode/lsp-notebook-concat/dist/index.js:1:55649)
    at y.handleOpen (/home/vscode/.vscode-server-insiders/extensions/ms-python.vscode-pylance-2022.11.10/dist/pylance-internal/node_modules/@vscode/lsp-notebook-concat/dist/index.js:1:66284)
    at _0x15e85d.<computed>.handleOpen (/home/vscode/.vscode-server-insiders/extensions/ms-python.vscode-pylance-2022.11.10/dist/server.bundle.js:1:274410)
[Error - 4:32:40 AM] The notebook-intellisense server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.

Extension version: 2022.11.10
VS Code version: Code - Insiders 1.74.0-insider (e764c5b816a99eb8673b2fc4fc028827f202872f, 2022-11-04T05:19:54.097Z)
OS version: Darwin x64 22.1.0
Modes:
Sandboxed: No
Remote OS version: Linux x64 5.15.49-linuxkit

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (16 x 2400)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
6, 6, 5

Memory (System)
64.00GB (0.02GB free)

Process Argv
--enable-proposed-api donjayamanne.kusto --log trace --crash-reporter-id 4cf0a6f0-aaf8-4da3-9b2b-81f6c639c12b

Screen Reader
no

VM
0%

Item
Value

Remote
Dev Container: Python 3

OS
Linux x64 5.15.49-linuxkit

CPUs
Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (8 x 2400)

Memory (System)
11.70GB (9.90GB free)

VM
0%

@rchiodo in my settings sync all are on pre-release builds, but when I open the PR from https://github.com/digitarald/ds-together-demo I occasionally get the latest stable releases as well for a moment.

I think the culprit might be in the python core extension. At least if I downgrade Python to 2002.16.1, it works as expected again.

Marking as dupe of #11938


Rust rewrite?

Yeah, I know, this has become a meme 💀. I'm willing to do the rewrite (maybe with the help of @FusionSid ?), but definitely not now as I'm busy IRL and need to develop some unfinished PRs.
I'm proposing RIIR for 3 reasons:

The project is relatively "small", so a rewrite would be "easy" and not so disruptive.
This would make it more maintainable when we add more features.
We could reduce our dependency on Python

which would allow us to (for example) use an LLVM backend (in the far future) to transpile RR to any lang (or native machine code)

We would have to resolve #74 first, as Python libs would no longer be available

(maybe with the help of @FusionSid ?)

I barely know any rust 😭
Most advanced thing I could probably do is fizz buzz

I barely know any rust 😭

oh 💀
In that case, we should postpone the rewrite to ~1 year (if it actually happens), which should be enough time for everyone to get used to Rust.
If organization members don't know Rust, then we can't reliably maintain the repo.
Should we close this issue, then reopen later? Or should we leave it open, to keep it in mind?

I've just stalked analyzed members at a glance, and collected some Rust stats:

There's 10 total at the time of writing this.
5 members have (at least) basic knowledge
2 (including me) have "enough experience"
3 (including me) are interested in learning more

Don’t worry @Rudxain , I learned Rust two years ago but just didn’t use it for 1 and a half years 😂
Still, I will be able to write a compiler. Maybe we can talk to and learn from each other privately 😁 this is my discord: satinwuker
You can also email me your other contacts if you don’t have a discord

Sure! Here's my public contact info

We forgot 💀 to announce that development is work-in-progress here: https://github.com/Rick-Lang/rickroust
If anyone reading this wants to help, please do!

cargo check
    Checking llvm-sys v140.1.3
error: No suitable version of LLVM was found system-wide or pointed
              to by LLVM_SYS_140_PREFIX.

              Consider using `llvmenv` to compile an appropriate copy of LLVM, and
              refer to the llvm-sys documentation for more information.

              llvm-sys: https://crates.io/crates/llvm-sys
              llvmenv: https://crates.io/crates/llvmenv
   --> /home/rudxain/.cargo/registry/src/index.crates.io-6f17d22bba15001f/llvm-sys-140.1.3/src/lib.rs:487:1
    |
487 | / std::compile_error!(concat!(
488 | |     "No suitable version of LLVM was found system-wide or pointed
489 | |        to by LLVM_SYS_",
490 | |     env!("CARGO_PKG_VERSION_MAJOR"),
...   |
497 | |        llvmenv: https://crates.io/crates/llvmenv"
498 | | ));
    | |__^

error: could not compile `llvm-sys` (lib) due to 1 previous error

llvm-config --version
16.0.6

https://gitlab.com/taricorp/llvm-sys.rs#llvm-compatibility
It seems inkwell is using a version of llvm-sys that's incompatible with llvm-16. But the following line indicates otherwise:
https://github.com/TheDan64/inkwell/blob/89e06af9dd70dc5d6bc5ae42a2a03f680a367d37/Cargo.toml#L152
I guess it's a config issue?

I'm so blind 🤦:
https://github.com/Rick-Lang/rickroust/blob/615f6481202a88df17a9508373693e00e1f7594a/Cargo.toml#L9


python39Packages.oci: 2.56.0 -> 2.60.0, oci-cli: 3.5.0 -> 3.6.0, lexicon: 3.9.0 -> 3.9.4

Motivation for this change
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

oci-cli needs to be updated to version 3.5.3 (check pyproject.toml).
Tests failed for dnspython. Upstream is at 2.2.0, while nixpkgs still has 1.16.0.

Result of nixpkgs-review pr 163010 run on x86_64-linux 1

2 packages failed to build:

lexicon
oci-cli

2 packages built:

python310Packages.oci
python39Packages.oci

Result of nixpkgs-review pr 163010 run on x86_64-linux 1

4 packages built:

lexicon
oci-cli
python310Packages.oci
python39Packages.oci

Result of nixpkgs-review pr 163010 run on x86_64-linux 1

4 packages built:

lexicon
oci-cli
python310Packages.oci
python39Packages.oci


Add class to property selector.

The properties <div> has no class, no id, and a hardcoded maxheight of 160px. It's pretty hard to get to using CSS without a class.

This was merged with c77fc2f
I guess github doesn't match up rebased commits to the original pull request.


extract entire warc file?

I have several multi-gigabyte WARC files that I need to unpack
warccat, warc-extractor, and others I've tried have all failed to do the job adequately so I've turned here as a last resort
is this possible or not?
The requirement to specify an "offset" is a problem because I want to extract everything

Are you trying to run code on each webpage in the file?
Or do you want to leave all of the webpages on disk somehow?
The first way is what you probably should do. That's just a loop like the one in the documentation:
https://github.com/webrecorder/warcio#warc-and-arc-streaming
The second thing, leaving all of the webpages on disk as separate files, is difficult to do, because how will you name the files? And what happens if the WARC file has two webpages with exactly the same URL?

Yes I'm trying to unpack the entire contents to disk similar to warcat's "extract" option, preserving all the directory structure and filenames
Unfortunately warcat apparently has a bug where if it encounters an HTTP header in the WARC that it doesn't like it bombs out with no option to ignore/continue
So I'm exploring other options
but it seems like there's not really anything that does the job

I suggest that you answer the questions that I asked, then I will implement it for you. If the WARC is from wget for a website that has no CGI arguments weirdnesses, then it's not so bad, but I would like you to be clear if this is sufficient.

@catharsis71 ?


Singleton config and multiple consumer configurations

As the ConsumerConfig is a singleton, how to create multiple consumers with eg. different groups?

This would be a really nice improvement. I can work on this myself.
The reason for this to be important is that let's say you use kafka for event sourcing in your application but you also one to use kafka for monitoring or observability, they clearly won't share configuration (and might be probably different instances). At the moment we have to do such trick:
public function __construct(Producer $producer = null, array $config = null, ...)
    {
        $config = $config ?: [
            'broker_list' => '<IP_ADDRESS>:9092',
        ];

        $brokerList = ProducerConfig::getInstance()->getMetadataBrokerList();

        $kConfig = ProducerConfig::getInstance();
        $kConfig->setMetadataBrokerList($config['broker_list']);

         $this->producer = $producer ?: new Producer();
        if (!empty($brokerList)) {
            ProducerConfig::getInstance()->setMetadataBrokerList($brokerList);
        }
    }

(source https://github.com/jcchavezs/zipkin-php-reporter-kafka/blob/master/src/ZipkinReporterKafka/Reporter.php#L35)
I propose that we keep the global config and use it when you don't pass a config in the producer arguments, otherwise use the provided one.

@jcchavezs keeping a global config is quite a mess IHMO, ideally we should be able to have multiple producer and consumer in the same event loop and they should just work properly. This is an edge case, for sure, but solving that would remove completely the messy singleton thing we have now.
I'm still refactoring the connection management area but am doing this together with the other projects I maintain - which makes things a bit harder 😄. Give me a couple days and I'll submit a PR with my progress but if anyone wants to help it would be amazing if we could address #213 and #214!

@lcobucci, hi. Is there any luck dropping singletons? I may be able to help you with this but need to know what is already done, so I can choose the right direction.
The real problem I have faced is the Broker when using producer inside the consumer. As we can specify the topics for the consumer the metadata will be fetched only for those topics, so only those topics will be available for the broker. This breaks the producer that uses the same broker as he will have access only to topics that consumer subscribed to...

@Sevavietl unfortunately I haven't been able to continue my implementation, due to a massive amount of work at my company. However I think that this situation will soon change and I will finally address this.
The main challenge here is that we need to invert the place where we control the EventLoop in order to be able to drop the singleton, but that is not a small change to make since the lib we're using for that controls everything using singletons.
IMHO a more appropriated solution would be replacing amp for reactphp, which always encourage people to inject the event loop. We could even drop our socket classes by the ones provided by reactphp, use promises, and a lot of cool stuff. However we must ensure that #213 and #214 are implemented and work fine before of jumping into that journey.

The real problem I have faced is the Broker when using producer inside the consumer. As we can specify the topics for the consumer the metadata will be fetched only for those topics, so only those topics will be available for the broker. This breaks the producer that uses the same broker as he will have access only to topics that consumer subscribed to...
I've made a workaround in consumer client code, by subscribing to all topics and while consuming ignore all but the target one. But this is bad voodoo)

Ouch! I'm really sorry for this 😢

@lcobucci, no worries -- have the same thing with work too)
Regarding EventLoop, well, I think it is ok that amp uses a singleton for the loop, as there can be only one event loop. We actually using amp for the whole app, so global loop comes handy because we were able to integrate this library without much pain using Loop::defer in the callback of the consumer.
It is a real problem that the Broker and Config are singletons. I conclude this from the situation above where topics for producer and consumer became dependant on one another.
I would be glad to help with what I can (and this is actually in my interests because we use this library takes a crucial part in my app, such as providing communication, so it is not without ulterior motives:))


Responsive Table?

Hey!
Datatable at datatables.net are responsive.
Is it possible to make your datatables responsive too?
Thx

Yes, this just serves as the back end to deliver the JSON to DataTables and search the database using CodeIgniter.  You still download the datatables.net library and can continue to use the responsive plugin.


ShowAll Querystring parameter support

How do I respond to ?showall=true querystring parameter, to show all blog posts instead of paginated?
Is there a way to grab that querystring value inside of the Blog Template Engine?

One way to do this would be to have one page (lets call it /Blog ) that shows paging, copy the module on another page (lets call it /Blog-all and set the options to show the content of the existing blog, then on that page, set the pager number high enough so that there is no pager shown.
Then on the first page, you can put a button to show all that simply points to the other page.

We'll only want to have /Blog show up in Google, not /Blog-all so that wouldn't work that well for us.

You can set the indexing page option for the  blog page to NO.


Implementation of an interface for _as_na_df method for dynamics system and ODESystem class

The goal is to create simple method or property that calls _as_na_df method from ODESystem class and dynamic systems.

Method _as_na_df for dynamics systems has been implemented to dynpy,models.mechanics.principles into ComposedSystem class.
https://cocalc.com/projects/cdcf3856-3e8b-4d1c-bcb2-5c42ba749d83/files/Marek Skwarka/Shared files/modules/dynpy/models/mechanics/principles.py


553 feature refactor segmentationdataset class

Thank you for submitting your PR and helping us with GDL.
Please read the template below, fill it out as appropriate, and make additional changes to your code as needed.
Please feel free to submit your PR even if it doesn't satisfy all of the requirements below.
If you need assistance or review from a maintainer, add the label Status: Help Needed or Status: Review Needed respectively.
After review, a maintainer will add the label Status: Revision Needed if further work is required for the PR to be merged.
Description
Fixes #553
Type of change
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Other

How Has This Been Tested?
added unit test for SegmentionDataset class
Screenshots (if applicable):
Checklist:
If you're unsure about any of these, don't hesitate to ask. We're here to help!

[x] My code follows the code style of this project
[x] I have performed a self-review of my own code
[x] My PR has a descriptive title
[x] My code follows PEP8 (most of it)
[x] My code have the proper documentation in the code
[ ] I have made corresponding changes to the documentation (if applicable)
[ ] My changes generate no new errors
[ ] My code is covered by the existing tests (if applicable)
[ ] My code includes tests that show that my feature works
[ ] My PR passes Travis CI tests

If your PR does not fulfill all of the requirements in the checklist above, that's OK! If you need help, @-mention a maintainer and/or add the Status: Help Needed label.

nicely done. What was the initial motivation for this refactoring? Adding a test :) ?


PYDEV DEBUGGER WARNING:

Eel version
0.14.0
Describe the bug
When running Eel in Debug mode in Vs-Code. I get the following warning in console:
PYDEV DEBUGGER WARNING:
sys.settrace() should not be used when the debugger is being used.
This may cause the debugger to stop working correctly.
If this is needed, please check:
http://pydev.blogspot.com/2007/06/why-cant-pydev-debugger-work-with.html
to see how to restore the debug tracing back correctly.
Call Location:
  File "/Users/freddy/.pyenv/versions/3.10.3/lib/python3.10/site-packages/gevent/threadpool.py", line 157, in _before_run_task
    _sys.settrace(_get_thread_trace())

PYDEV DEBUGGER WARNING:
sys.settrace() should not be used when the debugger is being used.
This may cause the debugger to stop working correctly.
If this is needed, please check:
http://pydev.blogspot.com/2007/06/why-cant-pydev-debugger-work-with.html
to see how to restore the debug tracing back correctly.
Call Location:
  File "/Users/freddy/.pyenv/versions/3.10.3/lib/python3.10/site-packages/gevent/threadpool.py", line 162, in _after_run_task
    _sys.settrace(None)

To Reproduce
Steps to reproduce the behavior:

Run:

import eel

def main() -> None:

    eel.init('web')
    eel.start('login.html')

if __name__ == "__main__":
    main()

In Visual Studio Code in debug mode. F5

View console output

Expected behavior
No warning
Desktop:

OS: macOS 12.4 (21F79)
Version: 0.14.0
Python version: 3.10.3 64-bit
VS Code version: Version: 1.68.1

So is this causing VS Code to misbehave? Or is this issue submission more of an alert to the group that something seems wrong?

It's just a warning that something seemed wrong. Thanks for the reply, I'll check it out 👍

even setting gevent to true in launch.json does not cure the issue


v3.0 suggestion 😉

What amazing awesome work this is!
Thank you very much for your hard work on v3.0 during this time! 😊
I'm already using v3.0 and I'd like to make some suggestions:
1⃣ Bottom sheet background corner and shadow
In v3.0, support the new customBackground() modifier, but I only want to change the background corner like v2.x (Because I wish to preserve the built-in multi-platform compatible blur background effect). The same shadow is also. In this demo I found using .shadow directly to set the sheet shadow, but this causes my entire page to have a shadow as well (which is not expected).
2⃣ [enableSwipeToDismiss] become hard to trigger
In v3.0, you need to swipe down very quickly to close the sheet (still may not dismiss), which in v2.0 there is no problem.
3⃣ About [BottomSheet+ViewModifiers]
I don't feel this way is very good, because I may show multiple bottom sheets on one page with different behavior, which will make the call chain very long, currently my solution is this:

It's also possible that I'm using it incorrectly...
Finally, thank you for creating this awesome library! I love you!
My English is poor, hope to convey my thoughts... 😝

I found that v3.0 is a bit strange display on the ipad or mac. Its width is too small, causing some content to be squeezed and not displayed completely. I think it would be better to set the width to half the screen width, just like SwiftUI's sheet modal. And on the ipad, it pops up from the top corner of the screen, which feels a little weird, maybe it's better to pop up from the bottom like an iPhone, even can use sheet instead of display directly on ipad or mac, these are just some of my suggestions...


Content of resizeable window

Is there any way of forcing the content to resize to match the window size?
Thanks in advance

No worries, i figured something out,
I define SCREEN_SIZE_H = 928 SCREEN_SIZE_W = 522
and then use it when scaling for portrait
if self.scalep:
x = 0
w = SCREEN_SIZE_W
h = SCREEN_SIZE_H


Its not possible to add multiple PaymentTerms when using the extended profile

The Invoice class only allows setting a single PaymentTerm at most, which is fine for all profiles apart from extended:

I need to add several payment terms to add multiple cash discount conditions.

Pull Request: https://github.com/ZUGFeRD/mustangproject/pull/459


Return result

Hi, love this new runtime. Can you recommend the best way to handle return messages?
I was looking to use an error handling function that can be called from anywhere in the script to return a standard json response and then exit the script but tried a few ways and doing an Exit always returns the powershell exit response, e.g.;
Response
{
"errorMessage": "RequestId: b6a48129-9818-420c-a837-c7b03a05a696 Error: Runtime exited with error: exit status 1",
"errorType": "Runtime.ExitError"
}
I'm sure this is super simple but what would be your recommendation on the best way to handle this so that whenver the function execute it always spits out a clean json response?
Many thanks!
Example function below:
function Send-PAPIReturnMessage($status_code=500, $success=$false, $custom_message) {
Return a properly formatted API result
$hash = [ordered]@{
"statusCode" = $status_code
headers = [ordered]@{
"Strict-Transport-Security"="testing";
"Cache-control"= "no-cache, no store";
}
body = [ordered]@{
"success"=$success;
"message"=$custom_message;
}
}
$json = $hash | ConvertTo-Json -Depth 99
$json
Exit 1
}
if (!$LambdaInput.period) {
Write-Host "period variable is null"
Send-CMPAPIReturnMessage -status_code 500 -success $true -custom_message "period is not specified "
}
Write-Host "This is the rest of my script which should never be hit"

Glad you're liking the new runtime!
In terms of your issue, when exit 1 is used, that will tell PowerShell to exit the process using the exit code 1 which is why you're seeing that response/error.
Instead of using exit, I modified the code as below and I believe it works as you'd like.
Using the Lambda Function handler: script.ps1::Handler
Code in the file script.ps1
function Send-PAPIReturnMessage($status_code = 500, $success = $false, $custom_message) {
    # Return a properly formatted API result
    $hash = [ordered]@{
        "statusCode" = $status_code
        headers      = [ordered]@{
            "Strict-Transport-Security" = "testing";
            "Cache-control"             = "no-cache, no store";
        }
        body         = [ordered]@{
            "success" = $success;
            "message" = $custom_message;
        }
    }

    $json = $hash | ConvertTo-Json -Depth 99
    return $json
}

function Handler($LambdaInput, $LambdaContext){
    if (!$LambdaInput.period) {
        Write-Host "period variable is null"
        return (Send-PAPIReturnMessage -status_code 500 -success $true -custom_message "period is not specified")
    }

    Write-Host "This is the rest of my script which should never be hit"
}

Ah perfect, thanks! So literally the exit is the last thing written by the script, so need to ensure the return is in the main handler. Got it! Thanks again, really nice job on the runtime!


Small improvements in GC backup and restore

Closes https://github.com/treeverse/lakeFS/issues/4341

Is this PR still relevant given that backup&restore has a bug for the new storage namespace structure?https://github.com/treeverse/lakeFS/issues/4775

Is this PR still relevant given that backup&restore has a bug for the new storage namespace structure?#4775

@itaiad200 yes. it changes no functionality just refactors stuff. Do you have any concern that I merge it?

Is this PR still relevant given that backup&restore has a bug for the new storage namespace structure?#4775

@itaiad200 yes. it changes no functionality just refactors stuff. Do you have any concerns that I merge it?

On the contrary, let's merge it so it will make the next release :)

Yep, that was my plan :)


Undeclared Jinja2 dependency

Description
pycsw depends on jinja2 for some reason, and this isn't declared in requirements.txt
Environment

operating system: mac/linux
Python version: 3.10
pycsw version: master (c532be3aba731bfee093620c40f64160c3901f8e)

Steps to Reproduce
>>> from pycsw import server
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/cdestigter/code/pycsw/pycsw/server.py", line 48, in <module>
    from pycsw.ogc.api.util import yaml_load
  File "/Users/cdestigter/code/pycsw/pycsw/ogc/api/util.py", line 45, in <module>
    from jinja2 import Environment, FileSystemLoader
ModuleNotFoundError: No module named 'jinja2'

Jinja is pulled in by https://github.com/geopython/pycsw/blob/master/requirements-standalone.txt

hm, confusing; I hadn't considered looking in a nonstandard location for this (and, grepping didn't find it)
I suppose that's fine except that things pycsw actually imports directly should probably be mentioned explicitly there, rather than pulled in as a side-effect of some other dependency

changed my PR to pull it in explicitly in requirements-standalone.txt. thanks


[CARBONDATA-3290] No need to apply MV/Preaggregate rules for Command

MVAnalyzerRule/CarbonPreAggregateQueryRules  is used to rewrite query for MV/pre-agg datamap.
CarbonPreAggregateDataLoadingRules is used to update query plan for building pre-agg datamap.
These rules are NO need to apply to Command
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2561/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10820/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2791/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2564/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10823/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/2567/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10826/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2797/

LGTM

@kevinjmh please handle for ExplainCommand scenario, because when i do explain for the query it will not go to mv rules and always show that, it is hitting main table

LGTM

@kevinjmh you need to handle for !plan.isInstanceOf[DeserializeToObject] also, please refer org.apache.carbondata.mv.datamap.MVAnalyzerRule#isValidPlan

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/3094/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/3292/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/11358/

@kevinjmh agree with ravi, we already have a check in mv analyzer ,  which will take care of this issue and MV will be loaded first time in memory and its one time task, so i think you can check close this PR.

OK. I'll close this


[Feature] doris查询条件区分大小写吗

Search before asking

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

Description

如何所示，不想要doris查询值区分大小写，要如何处理？
Use case
No response
Related issues
No response
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

Code of Conduct

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

doris 2.1

fe配置文件修改参数，但是文件初始化后好像就不生效了，你可以尝试元数据清空后，重新部署下
lower_case_table_names = 1


I directly compile the Flex source file in Demo package to LoginApp.swf, and replace the original LoginApp.swf, but execute automatic test fail

Originally submitted to Google Code by genglz1983 on 21 Feb 2011

1.I directly compile the Flex source file in Demo package without any modification to LoginApp.swf,;
2.replace the original LoginApp.swf,and execute rundemo.py;
3.but execute command "Select Flex Application loginApp" fail.
error:
11:18:14.830 INFO - Command request: isElementPresent[loginApp, ] on session f9d589dbbc8c497e961efe2fad767546
11:18:14.845 INFO - Got result: OK,true on session f9d589dbbc8c497e961efe2fad767546
11:18:14.845 INFO - Command request: waitForFlexReady[loginApp, 5.0] on session f9d589dbbc8c497e961efe2fad767546
11:18:19.861 INFO - Got result: Timed out after 5000.0ms on session f9d589dbbc8c497e961efe2fad767546

Originally submitted to Google Code by genglz1983 on 21 Feb 2011

I use Flex builder 4 to compile.When i am debugging,Flex builder reports error:
SecurityError: Error #2142: 安全沙箱冲突:本地 SWF 文件不能使用 LoaderContext.securityDomain 属性。file:///D:/MyEclipse%208.x/Workspaces/Lego-UI-Tag/src/org/apache/struts2/static/flex/LoginApp.swf 正在尝试加载 file:///D:/MyEclipse%208.x/Workspaces/Lego-UI-Tag/src/org/apache/struts2/static/flex/FlexPilot.swf。

Originally submitted to Google Code by @pekkaklarck on 22 Feb 2011

Are you able to run tests successfully with the original swf?

Did you have FPBootstrap package in source path when you compiled the source? See FlexTesting wiki page for more information about compile and execution time dependencies.

Originally submitted to Google Code by genglz1983 on 22 Feb 2011

1)It can run successful with the original swf;
2)I have added the FPBootstrap package into the source path.I have reference FlexTesting wiki page, but still not successful。Coulld you please provide detailed compiling steps.

Originally submitted to Google Code by @yanne on 22 Feb 2011

The demo is compiled with Flex sdk 3, and uses FlexPilot compiled with Flex sdk 3. If you compile with Flex 4, you need to replace the FlexPilot.swf used in the demo (located in demoapp/flex) with a version compiled with Flex 4 (https://github.com/downloads/mde/flex-pilot/FlexPilot-4-stable.swf). That should do the trick.

Originally submitted to Google Code by @pekkaklarck on 27 Apr 2011

Closin due to lack of information.


Serverless is installed globally.

Current coverage is 95.08% (diff: 100%)

Merging #17 into master will not change coverage

@@             master        #17   diff @@
==========================================
  Files            11         11
  Lines           305        305
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            290        290
  Misses           15         15
  Partials          0          0

Powered by Codecov. Last update c8bc860...5f8256f


Dashscope openapi compatible get 404 status code (no body) error

Which API Provider are you using?
OpenAI Compatible
Which Model are you using?
Dashscope qwen-plus
What happened?
Setting base_url as https://dashscope.aliyuncs.com/api/v1,
Setting API key as the key on dashscope
Setting the Model ID as qwen-plus
which is working for my local, while get  404 status code (no body), no place to check the request id or the query body.
Steps to reproduce

Setting base_url as https://dashscope.aliyuncs.com/api/v1,
Setting API key as the key on dashscope
Setting the Model ID as qwen-plus
start chat get   404 status code (no body)

Relevant API REQUEST output
No response
Additional context
No response

change the base url to https://dashscope.aliyuncs.com/compatible-mode/v1  solve the problem


[PL-46862] PL 1.24.x Release Notes

PL-46862
https://harness.atlassian.net/wiki/spaces/PD/pages/21637005335/NG+Release+SAAS+February+5th+2024+for+1.24.x-+Build
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c1d631f9a38345935b8402--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c348be028ba65c20c167dd--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c34a413bc1e05e1ea982d6--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c34eefc5e60e009b144541--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c35027deda1300843df692--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c35ce4ef9f7b0d40852115--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c4baec2a9fc1099c9c695a--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c4bbc76564570a745269e6--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65c4bc926564570a74526be5--harness-developer.netlify.app


Add GetBalance RPC

The wallets would like to be able to receive their balance for a given t-address by requesting it from LWD.
Requirements:

[ ] Return balance given a single t-address

Correction, the rpc that's needed is getaddressbalance. (The getbalance rpc reports only the balance for addresses that are owned by this wallet.) I verified that getaddressbalance is available with txindex and lightwalletd enabled.


Actually use the solver when starting rewriting

When processing an execute request, we start z3 in JsonRpc.hs and pass it to performRewrite. However, we then actually start rewriting without a solver. This PR makes sire the initial state in performRewrite actually is initialized with a solver.

KEVM performance:
| Test                                                                | georgy-dont-throw-away-solver time | master-613cb4984 time | (georgy-dont-throw-away-solver/master-613cb4984) time
|---------------------------------------------------------------------|------------------------------------|-----------------------|-------------------------------------------------------
| benchmarks/ecrecoverloop02-sig1-invalid-spec.k                      | 114.97                             | 274.06                | 0.419506677369919
| benchmarks/ecrecoverloop02-sigs-valid-spec.k                        | 119.43                             | 281.42                | 0.4243834837609267
| benchmarks/ecrecoverloop02-sig0-invalid-spec.k                      | 92.41                              | 159.5                 | 0.5793730407523511
| mcd/end-cash-pass-rough-spec.k                                      | 221.76                             | 331.79                | 0.6683745742789113
| erc20/ds/transferFrom-success-1-spec.k                              | 69.13                              | 87.53                 | 0.7897863589626413
| erc20/ds/transferFrom-success-2-spec.k                              | 65.74                              | 82.67                 | 0.795209870569735
| mcd/flopper-kick-pass-rough-spec.k                                  | 133.2                              | 164.28                | 0.8108108108108107
| mcd/flapper-yank-pass-rough-spec.k                                  | 198.53                             | 235.58                | 0.842728584769505
| erc20/ds/transfer-success-2-spec.k                                  | 60.21                              | 70.71                 | 0.8515061518879933
| erc20/ds/transfer-success-1-spec.k                                  | 61.01                              | 70.55                 | 0.864776754075124
| mcd/end-pack-pass-rough-spec.k                                      | 162.92                             | 181.61                | 0.8970871648037001
| erc20/hkg/transferFrom-success-2-spec.k                             | 58.96                              | 63.95                 | 0.9219702892885067
| benchmarks/staticloop00-a0lt10-spec.k                               | 50.58                              | 54.45                 | 0.9289256198347107
| mcd/cat-file-addr-pass-rough-spec.k                                 | 57.82                              | 62.24                 | 0.9289845758354756
| erc20/ds/approve-success-spec.k                                     | 55.46                              | 59.3                  | 0.9352445193929174
| erc20/ds/transferFrom-failure-1-b-spec.k                            | 145.94                             | 153.72                | 0.9493884985688265
| erc20/hkg/transferFrom-success-1-spec.k                             | 63.86                              | 67.17                 | 0.9507220485335715
| erc20/ds/transferFrom-failure-1-c-spec.k                            | 113.2                              | 118.57                | 0.9547102977144304
| erc20/hkg/approve-spec.k                                            | 49.74                              | 51.99                 | 0.9567224466243508
| erc20/ds/transfer-failure-1-b-spec.k                                | 92.34                              | 96.45                 | 0.9573872472783825
| erc20/ds/transferFrom-failure-2-a-spec.k                            | 75.53                              | 78.26                 | 0.9651162790697674
| benchmarks/encodepacked-keccak01-spec.k                             | 50.21                              | 48.5                  | 1.0352577319587628
| kontrol/test-storetest-teststoreload-0-spec.k                       | 52.86                              | 50.98                 | 1.0368772067477443
| kontrol/test-countertest-testincrement-0-spec.k                     | 100.18                             | 96.51                 | 1.0380271474458604
| mcd/dsvalue-read-pass-summarize-spec.k                              | 60.17                              | 57.85                 | 1.040103716508211
| examples/storage-spec.md                                            | 92.32                              | 88.65                 | 1.0413987591652565
| kontrol/test-emitcontracttest-testexpectemitdonotcheckdata-0-spec.k | 52.56                              | 50.01                 | 1.0509898020395922
| benchmarks/staticarray00-spec.k                                     | 49.39                              | 46.64                 | 1.0589622641509433
| TOTAL                                                               | 2520.43                            | 3184.94               | 0.7913587069144159

Closed in favor of https://github.com/runtimeverification/haskell-backend/pull/4002


DHCPv6 Information-Request response doesn't read info-refresh-time option

I'm running a test where dhcpcd sends a DHCPv6 Information-Request message, and the server responds with a non-default Information-Refresh-Time (32) option.  Dhcpcd isn't reading that option, and so the refresh time remains at the default.
I think I've tracked it down to a line in dhcp6_bind, but I'm not sure if this is fully correct. The test seems to be working with this patch, at least. I'm finding the option in state->recv, but not in state->new in this function.
diff --git a/src/dhcp6.c b/src/dhcp6.c
index 76a6719..b3246d6 100644
--- a/src/dhcp6.c
+++ b/src/dhcp6.c
@@ -3139,7 +3139,7 @@ dhcp6_bind(struct interface *ifp, const char *op, const char *sfrom)

 		if (state->reason == NULL)
 			state->reason = "INFORM6";
-		o = dhcp6_findmoption(state->new, state->new_len,
+		o = dhcp6_findmoption(state->recv, state->recv_len,
 				      D6_OPTION_INFO_REFRESH_TIME, &ol);
 		if (o == NULL || ol != sizeof(uint32_t))
 			state->renew = IRT_DEFAULT;

Packet capture of the transaction is attached.
Roy, could you confirm?
info-request.zip

dhcpcd debug logs, showing the refresh time remains 86400 secs:
Jan 26 05:36:09 [7337]: dhcpcd-10.0.6 starting
Jan 26 05:36:09 [7339]: spawned manager process on PID 7339
Jan 26 05:36:09 [7339]: udev: starting
Jan 26 05:36:09 [7339]: dev: loaded udev
Jan 26 05:36:09 [7339]: DUID 00:01:00:01:25:d0:44:16:f4:02:70:a4:86:9c
Jan 26 05:36:09 [7337]: forked to background
Jan 26 05:36:09 [7339]: bond0: executing: /etc/sysapps_script/network/dhcpcd-script PREINIT
Jan 26 05:36:09 [7339]: bond0: executing: /etc/sysapps_script/network/dhcpcd-script CARRIER
Jan 26 05:36:09 [7339]: bond0: IAID 70:a4:86:9c
Jan 26 05:36:09 [7339]: bond0: requesting DHCPv6 information
Jan 26 05:36:09 [7339]: bond0: delaying INFORM6 (xid 0x4f4efc), next in 1.1 seconds
Jan 26 05:36:10 [7339]: bond0: multicasting INFORM6 (xid 0x4f4efc), next in 1.0 seconds
Jan 26 05:36:10 [7339]: bond0: REPLY6 received from fe80::200:10ff:fe10:1080
Jan 26 05:36:10 [7339]: bond0: refresh in 86400 seconds
Jan 26 05:36:10 [7339]: bond0: writing lease: /var/lib/dhcpcd/bond0.lease6
Jan 26 05:36:10 [7339]: bond0: executing: /etc/sysapps_script/network/dhcpcd-script INFORM6

@jvfranklin ?


request to add my gc_spyu

gc_spyu is a requestor task app that on its own can do more than golem's yapapi's upcoming scanner api by providing a CLI to obtain specific provider cpu speed information. additionally, it provides more information than the modelname which is expected to be available from the next yagna version by adding speed information. in other words, awesome.

gc_spyu will need to be rewritten for yapapi 0.9. in progress.


Upgrade to @angular-devkit/build-angular 17.0.0 leads to EPIPE error in npm install with node 18

Command
build
Is this a regression?

[x] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
16.2.10
Description
Updating NX Workspace from Angular 15 to 17 and having a problem with installing @angular-devkit/build-angular in Version 17.0.3 / 17.0.0. Works with 15.1.5 and 16.2.10.
Minimal Reproduction
npm install with @angular-devkit/build-angular 17.0.0+
[2023-11-25T07:17:10.623Z] npm verb Linux 4.18.0-477.27.1.el8_8.x86_64
[2023-11-25T07:17:10.623Z] npm verb node v18.12.0
[2023-11-25T07:17:10.623Z] npm verb npm  v8.19.2
Exception or Error
[2023-11-25T07:17:02.861Z] npm timing reifyNode:node_modules/@angular-eslint/builder/node_modules/@nx/nx-darwin-arm64 Completed in 378ms
[2023-11-25T07:17:02.861Z] npm timing reifyNode:node_modules/@angular-devkit/build-angular/node_modules/@esbuild/darwin-arm64 Completed in 378ms
[2023-11-25T07:17:09.012Z] npm timing auditReport:getReport Completed in 6231ms
[2023-11-25T07:17:09.012Z] npm timing reify:audit Completed in 6232ms
[2023-11-25T07:17:10.623Z] npm verb stack Error: write EPIPE
[2023-11-25T07:17:10.623Z] npm verb stack     at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:94:16)
[2023-11-25T07:17:10.623Z] npm verb cwd /misc/jenkins/workspace/erkauf-webapp_feature_KPLUS-6648
[2023-11-25T07:17:10.623Z] npm verb Linux 4.18.0-477.27.1.el8_8.x86_64
[2023-11-25T07:17:10.623Z] npm verb node v18.12.0
[2023-11-25T07:17:10.623Z] npm verb npm  v8.19.2
[2023-11-25T07:17:10.624Z] npm ERR! code EPIPE
[2023-11-25T07:17:10.624Z] npm ERR! syscall write
[2023-11-25T07:17:10.624Z] npm ERR! errno -32
[2023-11-25T07:17:10.624Z] npm ERR! write EPIPE
[2023-11-25T07:17:10.624Z] npm verb exit -32
[2023-11-25T07:17:10.624Z] npm timing npm Completed in 12273ms
[2023-11-25T07:17:10.624Z] npm verb unfinished npm timer command:ci<PHONE_NUMBER>931

Your Environment
-

Anything else relevant?
No response

This seems like a problem with your node/npm and not with Angular CLI.

it seems, but no matter which node version I use, the dependency that causes the error is @angular-devkit/build-angular 17.0.0+.
[2023-11-27T09:15:43.705Z] npm verb reify failed optional dependency /misc/jenkins/workspace/erkauf-webapp_feature_KPLUS-6648/node_modules/@angular-devkit/build-angular/node_modules/@esbuild/android-arm
[2023-11-27T09:15:44.673Z] npm http fetch POST 400
[2023-11-27T09:15:52.866Z] npm verb audit error   pkgid: 'quick'
[2023-11-27T09:15:52.866Z] npm verb audit error }
[2023-11-27T09:15:52.866Z] npm verb stack Error: write EPIPE
[2023-11-27T09:15:52.866Z] npm verb stack     at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:94:16)
[2023-11-27T09:15:52.866Z] npm verb cwd /misc/jenkins/workspace/erkauf-webapp_feature_KPLUS-6648
[2023-11-27T09:15:52.866Z] npm verb Linux 4.18.0-477.27.1.el8_8.x86_64
[2023-11-27T09:15:52.867Z] npm verb node v19.6.0
[2023-11-27T09:15:52.867Z] npm verb npm  v9.4.0
[2023-11-27T09:15:52.867Z] npm ERR! code EPIPE
[2023-11-27T09:15:52.867Z] npm ERR! syscall write
[2023-11-27T09:15:52.867Z] npm ERR! errno -32
[2023-11-27T09:15:52.867Z] npm ERR! write EPIPE
[2023-11-27T09:15:52.867Z] npm verb exit -32


modularize html elements

In the dashboard, a lot of the buttons and switches are handwritten in the dashboard.html template. However, this could be modularized by defining widgets for individual elements, like the ThreeWayToggle button.
Read more on that e.g. here.

The main components I can "widgetize" are the three-way toggle buttons and the custom checkboxes I use. For both, there is the issue that I can not easily incorporate the statistics into the widget itself, since it is rendered out before the statistics are computed.
A solution would be to render a form, compute the stats from the cleaned form and then pass everything to a new form, which - in turn - passes the stats to the respective fields, which then pass it to the respective widget. Sounds unnecessarily complicated, right?
And since some stats (e.g. the ones for the T-stages) are inside the element where the button label is, I see no point in writing more custom widgets.


fix: Always enable allowClarmsValidator

Description

This PR just always enable AllowedCallersClaimsValidator in composer runtime.
We didn't remove isSkill settings in this PR, will have follow ups when we get an agreement on that.
Task Item
fixes #4501

Screenshots

Coverage increased (+0.009%) to 55.798% when pulling d273182d5e337acd64a345092d6206adec5ab947 on luhan/allowClaimsValidator into 952a8412a1f720041e9e4072d987e0cbb43d6ca1 on main.


Problems with the basic stuff

Hi SimeonLukas, first thank you for the plugin is a great idea
Second, I tried adding the SMTP host, domain, user and password, and nothing happens
Using default backend
Any idea about what could be wrong? Just used gmail cloud api, and creates oauth credentials...

Hithere and I'm sorry you have Problems with my Backend..
It is kind of complicated to reproduce the issue.
Do you have the onetime-app-password from Google?
Did you try Port 465?
Greetings
Simeon


Need to show list of Keys

While developing in AL, we need list of index (keys) to be listed.  See below for example

@atoader


failed to allocate a guard page on FreeBSD

Whenever I try to run any Rust program with Cachegrind on FreeBSD, it panics with the message thread '<unnamed>' panicked at 'failed to allocate a guard page', library/std/src/sys/unix/thread.rs:364:17.  It seems that Cachegrind's tricky memory tricks interfere with Rust's tricky memory tricks.  Is there a way at compile-time to disable the guard page allocation?  If so, iai should use it.

Unfortunately, even if such an option exists, Iai has no control over compilation. This is a bug for std or rustc, not Iai.


 I have tweaked some values to push Ambrosia firmly into the desoxy/stimulant with a slight heal identity, and increase the value and use of growing other medical plants. In particular, Poppies currently have the same Bicaridine limits as Ambrosia, making it totally outclassed and even more worthless given the nature of Chemistry's ability to crank out bicaridine.
Ambrosia:

Desoxyephedrine max increased from 2 to 10
Kelotane max lowered from 10 to 5
Bicaridine max lowered from 10 to 5
PotencyDivisor for Kelotane and Bicaridine from 10 to 20
PotencyDivisor for Desoxyephedrine from 50 to 10

Poppy:

Bicaridine max increased from 10 to 20
PotencyDivisor from 10 to 20

add a changelog but otherwise good changes

I think there a little problem with this PR, the goal was to increase both Poppy's bicaridine yield and tweak Ambrosia vulgaris identity.
But now that the potency divisor has been doubles both are now near useless chemical production wise.
You'll need to raise poppy potency at 400 to get those 20 bicar max per flower
Same thing for Ambrosia vulgaris, you'll need 500 potency to reach the maximum of 10 units of Desoxyephedrine

I'll fix this tonight.


EACCESS: permission denied

Describe the bug
Running a simple ag command fails with a permission error
How to Reproduce
$ sudo npm -g install @asyncapi/html-template --unsafe-perm=true
...

$ sudo npm -g install @asyncapi/generator
...

$ ag --debug api_description.yml @asyncapi/html-template
Errors while trying to resolve package location at undefined TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined
    at validateString (internal/validators.js:124:11)
    at Object.join (path.js:1039:7)
    at /usr/local/lib/node_modules/@asyncapi/generator/lib/generator.js:353:41
    at new Promise (<anonymous>)
    at Generator.installTemplate (/usr/local/lib/node_modules/@asyncapi/generator/lib/generator.js:339:12)
    at Generator.generate (/usr/local/lib/node_modules/@asyncapi/generator/lib/generator.js:168:73)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async /usr/local/lib/node_modules/@asyncapi/generator/cli.js:132:9 {
  code: 'ERR_INVALID_ARG_TYPE'
}
Template installation started because the template cannot be found on disk
Something went wrong:
Error: EACCES: permission denied, mkdir '/usr/local/lib/node_modules/@asyncapi/generator/node_modules/@asyncapi/html-template'

Expected behavior
I expected ag to generate some nice pretty HTML, not try to install files in a system location. I expected ag to find the previously installed template. I also expected that if ag did need to download anything, it would not do so into a place that needed root privileges.

I should note that this is with version 1.2.0.

@gavanfantom thanks for the issue. Just to get some more information are you saying this worked in the previous version?

@gavanfantom yes, please specify if the previous version has the same effect for you. From debug I can see that all looks good on the generator side, and looks like the app just doesn't have the right to do any changes in /usr/local/lib/

@derberg does the generator actually check the global node_modules directory for the template? 🤔 i.e. if he had permissions to read in /usr/local/lib/node_modules/ would it use this path /usr/local/lib/node_modules/@asyncapi/html-template or only look in /usr/local/lib/node_modules/@asyncapi/generator/node_modules/?

I have the same problem with version 1.1.11.

Yeap, this use case never worked and won't work with the latest version we released today. We use https://github.com/sindresorhus/resolve-pkg to resolve dependencies, and we look as @jonaslagoni wrote, into /usr/local/lib/node_modules/@asyncapi/generator/node_modules/.
this is why you are getting
Errors while trying to resolve package location at undefined TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined
    at validateString (internal/validators.js:124:11)
    at Object.join (path.js:1039:7)
    at /usr/local/lib/node_modules/@asyncapi/generator/lib/generator.js:353:41
    at new Promise (<anonymous>)
    at Generator.installTemplate (/usr/local/lib/node_modules/@asyncapi/generator/lib/generator.js:339:12)
    at Generator.generate (/usr/local/lib/node_modules/@asyncapi/generator/lib/generator.js:168:73)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async /usr/local/lib/node_modules/@asyncapi/generator/cli.js:132:9 {
  code: 'ERR_INVALID_ARG_TYPE'
}
Template installation started because the template cannot be found on disk

but then we try to install it again, into /usr/local/lib/node_modules/@asyncapi/generator/node_modules/, which is good, just fails on your side.
did you try installing generator with sudo npm install generator --unsafe-perm=true -g or sudo npm install generator --unsafe-perm=true --allow-root -g ?

I have just tried reinstalling with:
sudo npm -g install @asyncapi/generator --unsafe-perm=true --allow-root
(you did mean @asyncapi/generator rather than generator, right?)
I'm still getting the same error.

yeap, sorry, thinking with shortcuts 😄
can you try doing chmod +w /usr/local/lib/node_modules

I was getting some issues when using generator version >=1.2.0. I used yarn but was able to contain the issue using npm with Docker. I simplified this as much I could. Using Node 14:
FROM ubuntu:20.04

RUN apt-get update \
 && apt-get -y install curl git
RUN curl -sL https://deb.nodesource.com/setup_14.x | bash \
 && apt-get update \
 && apt-get -y install nodejs

# Install generator
RUN npm install --global --unsafe-perm=true --allow-root<EMAIL_ADDRESS>
# Generate with dummy yaml
RUN echo "asyncapi: 2.0.0\ninfo:\n  title: dummy\n  version: 0.0.0\nchannels: {}" > dummy.yaml \
 && mkdir html \
 && ag --debug dummy.yaml @asyncapi/html-template -o html

This works and generates fine. However if generator version is changed to 1.2.0 or newer this will fail:
Step 5/5 : RUN echo "asyncapi: 2.0.0\ninfo:\n  title: dummy\n  version: 0.0.0\nchannels: {}" > dummy.yaml  && mkdir html  && ag --debug dummy.yaml @asyncapi/html-template -o html
 ---> Running in 8fbc46b82b94
Errors while trying to resolve package location at undefined TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined
    at validateString (internal/validators.js:124:11)
    at Object.join (path.js:1039:7)
    at /usr/lib/node_modules/@asyncapi/generator/lib/generator.js:353:41
    at new Promise (<anonymous>)
    at Generator.installTemplate (/usr/lib/node_modules/@asyncapi/generator/lib/generator.js:339:12)
    at Generator.generate (/usr/lib/node_modules/@asyncapi/generator/lib/generator.js:168:73)
    at async /usr/lib/node_modules/@asyncapi/generator/cli.js:132:9 {
  code: 'ERR_INVALID_ARG_TYPE'
}
Template installation started because the template cannot be found on disk
Template @asyncapi/html-template successfully installed in /usr/lib/node_modules/@asyncapi/generator/node_modules/@asyncapi/html-template
Version of used template is 0.18.1.
Something went wrong:
Error: Cannot find module '/usr/lib/node_modules/@asyncapi/generator/node_modules/@asyncapi/html-template/node_modules/@asyncapi/generator-filters'
Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:880:15)
    at Function.Module._load (internal/modules/cjs/loader.js:725:27)
    at Module.require (internal/modules/cjs/loader.js:952:19)
    at require (internal/modules/cjs/helpers.js:88:18)
    at /usr/lib/node_modules/@asyncapi/generator/lib/filtersRegistry.js:79:13
    at Array.map (<anonymous>)
    at registerConfigFilters (/usr/lib/node_modules/@asyncapi/generator/lib/filtersRegistry.js:71:32)
    at module.exports.registerFilters (/usr/lib/node_modules/@asyncapi/generator/lib/filtersRegistry.js:14:9)
    at async Generator.generate (/usr/lib/node_modules/@asyncapi/generator/lib/generator.js:177:7)
    at async /usr/lib/node_modules/@asyncapi/generator/cli.js:132:9

Hopefully this helps

drwxr-xr-x 6 root root 4096 Mar  3 12:37 /usr/local/lib/node_modules/
It does have write permission, but only for root. Allowing non-root users write access to this directory can not possibly be the right answer.
I'm also a bit confused as to why "this use case never worked and won't work with the latest version we released today." This use case is literally the one documented in README.md ("Install the CLI"). Is there a different installation method that I should be using?

It does have write permission, but only for root. Allowing non-root users write access to this directory can not possibly be the right answer.

Yea I agree with you here, @gavanfantom what would you like to see it changed to (in case we don't install templates where the generator is installed)?

It does have write permission, but only for root. Allowing non-root users write access to this directory can not possibly be the right answer.

I'm not sure what is your expectation here towards the generator

I'm also a bit confused as to why "this use case never worked and won't work with the latest version we released today." This use case is literally the one documented in README.md ("Install the CLI"). Is there a different installation method that I should be using?

I guess I didn't make myself clear, sorry. I meant that npm -g install @asyncapi/html-template global installation of templates was never supported. Just to imply that it is not a bug that generator cannot access a global template. I'm gonna try to enable support this week https://github.com/asyncapi/generator/issues/482#issuecomment-792601786

I'm not really sure what the ideal solution would be.
I would have thought though, that something like the following should work:

Provide a way to explicitly install a template (and do nothing else). Ideally with npm -g, but if it's a separate ag invocation that is documented to require root, and promises not to do anything other than installation, that would probably be ok.
If a template is not found, and can't be installed, issue a message saying so, to prompt the user to perform the installation manually, and/or
Perform the installation in a user-local directory (but not the current directory, ideally)
Test that after using sudo npm -g install @asyncapi/generator (plus any other documented install steps) that a non-root user can run ag.

let us then first make sure we support globally installed templates too. Once this works, let us reiterate again to see what else could be done on the generator side.

If a template is not found, and can't be installed, issue a message saying so, to prompt the user to perform the installation manually, and/or

I'll make sure to extend logs as much as possible when I will work on enabling global templates support.

Perform the installation in a user-local directory (but not the current directory, ideally)

if you use CLI it will install under user local where the generator is installed globally. Otherwise, it will install in the project where you use the generator as a library

Unfortunately this doesn't entirely resolve the issue, as even with the new version we still get:
$ ag --debug api_description.yml @asyncapi/html-template
Errors while trying to resolve package location at undefined TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined
at validateString (internal/validators.js:124:11)
at Object.join (path.js:1039:7)
at /usr/local/lib/node_modules/@asyncapi/generator/lib/generator.js:353:41
at new Promise ()
at Generator.installTemplate (/usr/local/lib/node_modules/@asyncapi/generator/lib/generator.js:339:12)
at Generator.generate (/usr/local/lib/node_modules/@asyncapi/generator/lib/generator.js:168:73)
at processTicksAndRejections (internal/process/task_queues.js:93:5)
at async /usr/local/lib/node_modules/@asyncapi/generator/cli.js:132:9 {
code: 'ERR_INVALID_ARG_TYPE'
}
Template installation started because the template cannot be found on disk
Something went wrong:
Error: EACCES: permission denied, rename '/usr/local/lib/node_modules/@asyncapi/generator/node_modules/caniuse-lite' -><EMAIL_ADDRESS>This is with @asyncapi/html-template installed globally.

This is with @asyncapi/html-template installed globally.

If the template is installed globally, it should not log Template installation started because the template cannot be found on disk. What do you mean by "installed globally"? Where is the html-template located + are you sure you are on latest generator? Errors while trying to resolve - this error has changed a bit in new version

Not to worry, it turns out that somehow an old version had got installed that npm didn't replace with the latest. I have manually cleaned up after npm, and reinstalled, and it works much better now.

Awesome 🚀

I'm receiving this error when trying to use the asyncapi/cli docker container:
Is this expected?
docker run --rm -it -v asyncapi.json:/app/asyncapi.json -v output:/app/output asynca
pi/cli generate fromTemplate -o output asyncapi.json @asyncapi/html-template
Generation in progress. Keep calm and wait a bit... done

Generator Error: EACCES: permission denied, mkdir '/usr/local/lib/node_modules/@asyncapi/cli/node_modules/@asyncapi/generator/node_modules/node-fetch/node_modules'

I'm receiving this error when trying to use the asyncapi/cli docker container:
Is this expected?
docker run --rm -it -v asyncapi.json:/app/asyncapi.json -v output:/app/output asynca
pi/cli generate fromTemplate -o output asyncapi.json @asyncapi/html-template
Generation in progress. Keep calm and wait a bit... done

Generator Error: EACCES: permission denied, mkdir '/usr/local/lib/node_modules/@asyncapi/cli/node_modules/@asyncapi/generator/node_modules/node-fetch/node_modules'

Adding this fixed it for me: --user=root


Mady translations showing up blank

Hi @guigrpa , I've been trying to use mady in order to translate text for my project, formatted this way:
export const ENGLISH_STATE = {
  lang: 'en',
  messages: {
    'label': 'English',
    'login.password': 'Password',
    'login.confirm': 'Confirm Password',
  },
};

Because my project needs to follow this format, I thought of using the Regex option in order to add the text I want and see its translation:

And, while that works, the translated text appears blank:

I've also tried formatting my text so it looks like a function:

But the same issue persists:

Why would this be happening? Please help me understand if I'm not using it right. Should you need it, here is my config.json file:
{
  "dbVersion": 2,
  "srcPaths": [
    "src/testtranslation"
  ],
  "srcExtensions": [
    ".js",
    ".jsx",
    ".coffee",
    ".cjsx"
  ],
  "langs": [
    "en",
    "es",
    "ca",
    "en-US",
    "en-GB"
  ],
  "msgFunctionNames": [
    "_t"
  ],
  "msgRegexps": [
    "(?<=: ')(.*)(?=')"
  ],
  "fJsOutput": true,
  "fMinify": false,
  "fJsonOutput": true,
  "fReactIntlOutput": true,
  "originalLang": "en"
}

The problem I have is that the translations don't show. Like, wouldn't the es section say label2 Ingles for example?

I'm sorry. I'm afraid I misunderstood. I believed the program was supposed to translate it all for you

No worries, I'm glad you figured it out!
Note however that Mady can help you with automatic translation based on Google Translate.


Colorize Tux

This is just something on the side and no problem with the functionality, but it would be nice to have a gem that would colorize Tux the way Pry can be colored.

Thanks for reporting your issue! This is one of my 12 active issues. Use that link to check how soon your issue will be answered. Don't forget to check your issue against this project's CONTRIBUTING.md. Cheers.

Thanks for the feedback. I'm not looking to have tux pull in colorizing functionality by default. This can be done independently of tux. Since tux is based on ripl, you can create a ripl plugin to colorize output as you'd like. You could look to gems like colorize and ripl-color_streams for inspiration.


docs: Suppress phpstan error in AutoRouterImproved

Description
The following error is a false positive.
See AutoRouterImprovedTest::testFindsControllerAndMethodAndParam().
  ------ ---------------------------------------------------------------
  Line   system/Router/AutoRouterImproved.php
 ------ ---------------------------------------------------------------
  301    Strict comparison using !== between null and null will always
         evaluate to false.
 ------ ---------------------------------------------------------------

https://github.com/codeigniter4/CodeIgniter4/actions/runs/5449192482/jobs/9913186865
Checklist:

[x] Securely signed commits
[] Component(s) with PHPDoc blocks, only if necessary or adds value
[] Unit testing, with >80% coverage
[] User guide updated
[x] Conforms to style guide

Closed by #7650


Orchard.Dashboards should use the existing bootstrap.css from Orchard.Resources. Fixes #7755.

Removed the bootstrap stylesheets from Orchard.Dashboards/Styles, and changed the Content.Dashboard.cshtml to require bootstrap from Orchard.Resources instead.

@Xceno, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot


How to label a specified type of picture

Now, i have a large number of images, including IR images and color images in same folder.
ex: 1_ir.png, 1_color.png; 2_ir.png, 2_color.png.
But I just want to label the color picture. How can I filter it when i use "connected file share" to add images;
BTW, in my project, i have request that it can display the order of label point when draw points;
example, first point is eyes, second point is nose, third point is mouth...
because i need to verify these points are correct.
Thanks!

@YuanNBB , it isn't well implemented in CVAT at the moment. You need to select all *_color.png images manually. There are many ways how to fix it. I will close the issue because most probably our team will not have time to implement the feature in the nearest future. Feel free to contribute!


How to select different page when Pager already presenting

I need to select different page when user clicks on the map pin annotation. I do not see the way how to do it. How I could swipe to different page programmatically?
    var body: some View {
        Pager(data: mapViewModel.provider.items) { item in
            VStack {
                Text("OHO")
            }
        }
        .onPageChanged({ self.mapViewModel.selectPinAt(index: $0) })
        .firstPage(mapViewModel.selectedItemIndexSubject.value ?? 0)
        .itemSpacing(10)
        .itemAspectRatio(0.85, alignment: .end)
    }

Use version 1.3.1. I've changed a state variable to a binding so that you can programatically move it. See the example here an example of how to use it.

Thanks for quick fix and release!


minimum Android SDK Decision

This plugin requires min SDK 16 (4.1) BLE introduced with 18 (4.3) The requirement document says min 4.4 (19)
So, any objection for 19 ?

Actually we ideally need to go all the way back to 16 which gives us 89% of the market.
Also we do not require BLE. It's strictly a nice to have. What we require is Wi-Fi Direct discovery and Bluetooth. Those were added in 16 and 5 respectively. So we really should set our minimum to 16.
@DrJukka any comment on what minimum SDK we should support for our Cordova plugin?

Insecure BT has failed few times with API level 16 device, thus not recommending to use it. I have successfully used all API parts with API level 18 (Nokia X2) device.
BLE I have no opinion yet, though I di remember few people saying that its not Great with API level 18, suggesting that it works better on 19.

BLE is a nice to have and we need to cover as much of the market as possible. Still... if we have to go all the way to 18 then we might as well go to 19. 18 is only 5.2% of the market at this point. Bummer.


Clicking on a category should highlight all category items

clicking a category now filters the activities by that category

Closed by 7b825ea3c21330c60c34c347ee65fcc470d476cb c17d7f0a1ae81251d4ddb3953ddb7057afc680a8 8cbebb69fbd28e89c4c291642c7e6f1218a8f403


TorchScript conversion tutorial not working for pytorch_geometric

🐛 Bug
Hi, I am trying to follow the tutorial at https://pytorch-geometric.readthedocs.io/en/latest/notes/jit.html, but I am running into an error. Code and traceback are provided below.
To Reproduce
Run the following code:
import torch
import torch.nn.functional as F
from torch_geometric.nn import GCNConv
from torch_geometric.datasets import Planetoid
class GNN(torch.nn.Module):
    def __init__(self, in_channels, out_channels):
        super().__init__()
        self.conv1 = GCNConv(in_channels, 64).jittable()
        self.conv2 = GCNConv(64, out_channels).jittable()

    def forward(self, x, edge_index):
        x = self.conv1(x, edge_index)
        x = F.relu(x)
        x = self.conv2(x, edge_index)
        return F.log_softmax(x, dim=1)
dataset = Planetoid(root='/tmp/Cora', name='Cora')
model = GNN(dataset.num_features, dataset.num_classes)
model = torch.jit.script(model)

and the following traceback should arise:
RuntimeError:
Tried to access nonexistent attribute or method 'dtype' of type 'Optional[Tensor]'.:
  File "/Users/rohan/.virtualenvs/coding/lib/python3.7/site-packages/torch_sparse/matmul.py", line 99
        valueA = valueA.to(torch.float)
    if valueB is not None:
        valueB = valueB.to(valueA.dtype)
                           ~~~~~~~~~~~~ <--- HERE
    M, K = src.sparse_size(0), other.sparse_size(1)
    rowptrC, colC, valueC = torch.ops.torch_sparse.spspmm_sum(
'spspmm_sum' is being compiled since it was called from 'spspmm'
  File "/Users/rohan/.virtualenvs/coding/lib/python3.7/site-packages/torch_sparse/matmul.py", line 116
           reduce: str = "sum") -> SparseTensor:
    if reduce == 'sum' or reduce == 'add':
        return spspmm_sum(src, other)
               ~~~~~~~~~~~~~~~~~~~~~ <--- HERE
    elif reduce == 'mean' or reduce == 'min' or reduce == 'max':
        raise NotImplementedError
'spspmm' is being compiled since it was called from 'matmul'
  File "/Users/rohan/.virtualenvs/coding/lib/python3.7/site-packages/torch_sparse/matmul.py", line 139
        return spmm(src, other, reduce)
    elif isinstance(other, SparseTensor):
        return spspmm(src, other, reduce)
               ~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
    raise ValueError
'matmul' is being compiled since it was called from 'GCNConvJittable_20110e.message_and_aggregate'
  File "/Users/rohan/.virtualenvs/coding/lib/python3.7/site-packages/torch_geometric/nn/conv/gcn_conv.py", line 194
    def message_and_aggregate(self, adj_t: SparseTensor, x: Tensor) -> Tensor:
        return matmul(adj_t, x, reduce=self.aggr)
               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
'GCNConvJittable_20110e.message_and_aggregate' is being compiled since it was called from 'GCNConvJittable_20110e.propagate__1'
  File "/var/folders/5v/9njppv2x52q_c237jzkpls7m0000gn/T/rohan_pyg_jit/tmppo0w0nzw.py", line 165
        if self.fuse:
            if isinstance(edge_index, SparseTensor):
                out = self.message_and_aggregate(edge_index, x=in_kwargs.x)
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
                return self.update(out)

'GCNConvJittable_20110e.propagate__1' is being compiled since it was called from 'GCNConvJittable_20110e.forward__0'
  File "/var/folders/5v/9njppv2x52q_c237jzkpls7m0000gn/T/rohan_pyg_jit/tmppo0w0nzw.py", line 213
        # propagate_type: (x: Tensor, edge_weight: OptTensor)
        out = self.propagate(edge_index, x=x, edge_weight=edge_weight,
                             size=None)
                                  ~~~~ <--- HERE

        if self.bias is not None:

Expected behavior
No output is expected. The jit compilation should work as intended.
Environment

OS: Mac OS Catalina
Python version: 3.7.2
PyTorch version: 1.8.0
CUDA/cuDNN version: N/A, CPU only
GCC version: Apple clang version 12.0.0 (clang-1<IP_ADDRESS>), Target: x86_64-apple-darwin19.6.0
Any other relevant information: Perhaps this is related to my PyTorch version? Would it help to update?

Additional context
N/A

Can you try to re-install torch-sparse? This should fix the error. There was a bug in the recently released wheels.

Looks like this was addressed by https://github.com/rusty1s/pytorch_sparse/commit/85ce67e4de51ab72b5b5ae9edc4fc6e377232438, thank you! Closing.


[spaceship] add a check in app_version.setup_screenshots to also check if there is an app_preview already uploaded

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ ] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
When trying to do any update, fastlane will first update the scaled flags of the display_families to true unless they have a screenshot. This doesn't take in account that a trailer (app preview) might be present instead. As a result, the scaled flag is set to true for said display_family and upon saving, the server complains that Your app preview can't be uploaded because you have selected to use an app preview from a larger display size.
This happens when a user tries to do a deliver, even with the skip_screenshots flags set to true. And that is because Fastlane will send the whole configuration again in order to update any part of it. The prerequisites are that there should be only video previews in a display_family that can be set as scalable. It might also happen if the overwrite_screenshots flag is also set to true.
Description
There is one more check added to check if the trailers key is present and the value key of it has contents.

I signed it!

The comment at the beginning of the method might need an update as well.
Can you also explain the bigger picture here? What would a user do to trigger this code?

Hey @xpy, looks like a great PR. What’s the fastest way to test this? Can I directly test it with spaceship?

@max-ott Hello! I have to admit that I only tested around my case and I didn't dive into other cases. What I do is fastlane deliver --skip_binary_upload true --run_precheck_before_submit false --force true --overwrite_screenshots true I also tried with --skip_screenshots and --skip_metadata and still have the same problem. And on the App Store Connect I have only one app preview video on the iPhone 5.5'', but I suspect it would also trigger for any display size that can be checked as "scalable".

hi! we're encountering the same issue, is there any chance this will be merged? thank you!

It needs more tests. We can’t simply merge each PR without enough tests and reviews from maintainers. As there were no other ways presented to easily test and debug this, its still waiting for more input from the author.

@max-ott I already mentioned that I tested for the very case that the PR is about. I also described the way to reproduce, if it seems difficult to reproduce, it probably is because it is indeed a very specific case that is difficult to reproduce. I am not sure what else you expect from me to do to validate this PR.

@xpy as this changes spaceship behaviour it would be useful to have steps to reproduce via spacecship. PRs require tests from other people than the initial contributor and if no one else can test it, it won't get merged.
Easiest way would be to provide steps for the spaceship playground / IRB. (https://github.com/fastlane/fastlane/blob/master/spaceship/docs/AppStoreConnect.md)

I've just had some time to review the code. I'm not sure if it's the right place to put the code there and it solves all issues. The issue at the moment with the app preview upload in general seems to be related to the scaled property.
We need to disabled the scaled property for the display_family, if:

There are screenshots present (probably the case right now)
There are app previews present
If you are trying to upload an app preview
If you are trying to upload a screenshot (probably the case right now)

fixed #15036 for me as well

Hi @joshdholtz,
Could you review this PR please ?

Ah yes yes! I will do some review on this this week!  Thanks for friendly ping ❤

How could we help with this PR ? Did you get time to review it @joshdholtz ?

@mackoj Thanks for ping on this! Got lost in the shuffle 😔 Bumping this to top of list for rest of this week


refactor global_parameter_loader

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I've agreed with the maintainers that I can plan this task.

Description
The global_parameter_loader created some years ago to set common ROS parameters to each node, but now only set two parameters, use_sim_time and vehicle_info.
Purpose
To refactor global_parameter_loader, we discuss how it should be.
Possible approaches
For example, we simply delete this package and set parameters directly in autoware.launch.xml
Definition of done
make the conclusion to refactor global_parameter_loader.

https://github.com/autowarefoundation/autoware.universe/pull/6872

Is merged, closing the issue.


update order

I'm actually not sure we need/want to update this one, since it's just a test fixture (I already updated the bundled demo in demos/, which did need it).

Fine by me – I was just on a roll and noticed it :)

We can keep the tests simpler without this one.

:+1:


add main for checking configurations

Actually this PR now also takes the first step to make the code compile and run without any GPUs. Next step would be to make the CPUONLY flag a global CMake flag (instead of just for ConfCheck), which then disables FindCUDA etc, and also comments out all the GPU backend things.
One way to then add CPU support is to make a few executables compile with CPUONLY, which will then help other executables as well, as the code is made more and more less hardcoded to the GPU backend.


Icons not rendering properly

SVG icons and fonts on windows are not rendering as they are supposed to render, refer to screenshots below.
Windows - https://i.imgur.com/AWcc1CD.png
Ubuntu 19.04 - https://i.imgur.com/xhIAxH0.png

That seems like a browser related issue. What browser are you using on Windows? Is there any errors in the devtools console? Or, maybe you can try to disable all browser extentions.


RFC: Status command, expose context

Proposal
Store the team label and name in config so that it's more readily available.
Add a status command that shows the current context.
Why?
We have an issue of not surfacing the context enough, so that a user knows which area they are operating in.

Having a status command makes it easily viewable
Implementing a --short flag enables us to expose just the label of the context
We can then provide a bash snippet for people to use in their PS1 to have it visible -- ie.  ~/manifoldco<EMAIL_ADDRESS>$ manifold run node index.js

Problems
Our middleware needs to be sped up to make it reasonable to include in someones PS1

yeah they'll hit token expiration when they try to do something with their session, but would be nice if status just had all the info locally.

Needs changes to configuration to properly implement the command.

Okay, I've renamed it to manifold context, we can merge this and address the PS1 viability at a later tdate.


Added generic calls for automate infrastructure retirement methods

Automate - Retirement - Infrastructure added generic calls for pre_retirement and check_pre_retirement methods.
Added tests.
https://bugzilla.redhat.com/show_bug.cgi?id=1302130

@mkanoor
Please review

@gmcculloug
Please review

@billfitzgerald0120 @tinaafitz From what I see you added a common method call to check_pre_retirement/pre_retirement that responds to the "create" message, but the state-machine calling this instance is still passing the vendor as a message which means the new method will not be invoked.  The tests are calling the method instances directly and not passing a message so they appear to work.
Would be help if you could better describe the changes you are trying to make here.  Is the state-machine suppose to call with a vendor message now?  If so, do we need the vendor-specific calls that still exist in the method instances?

@gmcculloug
Please review

@billfitzgerald0120 @tinaafitz The tests are failing because you are not passing the vendor message parameter to automate MiqAeEngine.instantiate.

@gmcculloug
Tests fixed


feat: add git_commit with git commit short hash

Description

Add short git commit hash, prepending immediately next to  git_status.
The display format git line is "${git_branch}${git_status}${git_commit}".

Screenshot

Merge please :)

Will we have this merge?

Any update?

No one cares, it's frustrating.

Already merged in #1438


Change LED patterns

This PR moves the LED handling to a new module and separates the UI state and the derived LED settings so that it is easier to consistently change the LED patterns.  Then it changes the LED patterns so that it is off by default and blinks green during a user confirmation request.
Note that I used green blinking instead of white because we already use white blinking for winking, but this is of course open to discussion.  This PR introduces a slight overhead of additional uptime() calls but this should be negligible.

Note that I used green blinking instead of white because we already use white blinking for winking, but this is of course open to discussion.

I think by the original design these two should be exact in the behavior.
Additionally adding another separate color might introduce confusion, not to mention straying away from the NK FIDO2 UX / colors (users having both models will have to remember two different behaviors).
Do you have strong opinion on why these two should have separate colors?

My thoughts for the different colors for user confirmation requests and blinking:

These are different operations with different consequences for the user.  I think it makes sense to be able to distinguish them.
Currently, we only have two standard operations that are visible to the user – these two.  Using the same color for both just appeared non-intuitive to me.
We won’t be able to replicate much of the FIDO2 UI anyway, so I think we should not be too strict here.

Though it would probably make sense to keep the user confirmation color consistent (white) and change the wink color instead (blue?).  Blue blinking is not used by the FIDO2 according to the docs, and the docs don’t state the wink color either.

Updated so that user confirmation blinks white and winking blinks blue.  I will merge this now, but we can discuss the wink color on Tuesday and change it.


Book ticker sockets added

Book ticker sockets added among websocket API.
Following streams are part of the PR:
https://github.com/binance-exchange/binance-official-api-docs/blob/master/web-socket-streams.md#individual-symbol-book-ticker-streams
https://github.com/binance-exchange/binance-official-api-docs/blob/master/web-socket-streams.md#all-book-tickers-stream

Thanks @nardew


Markdown cleanup and add linkTitles

@cartermp:

This PR does a cleanup of the markdown. Mainly, line-wrapping paragraphs (so that git diffs work better), and adding a blank line where appropriate between markdown elements (like a header or code-block and a paragraph).
There is no change in content
I've added a linkTitle to the instrumentation page that shortens the title (since these pages are already under the Instrumentation section of the website docs.

I'd show you a preview if I could, but I can't 😉
/cc @austinlparker

Oh, I forgot to mention: I removed the toc/tocstop TOC from one of the pages (cf. https://github.com/open-telemetry/opentelemetry.io/issues/980).


Issue 211

Fix for allowing addition of multiple classes via helper method

Thank you @samturrell, but I have done this a little bit nicer - TDD + using of modern elem.classList: https://github.com/limonte/sweetalert2/commit/44e875c79059322821916ebaefaa43cd2d2c9c9a


Adds pt_PT locale

Just adds a new pt_PT locale.
Tested it on my Bangle.js and seems to work just as expected. =-)
PS - Thank you for all your work on Bangle.js. I may not be a very active community member, but your work is highly appreciated.

Thanks! And thanks for testing - the locales especially sometimes seem ok but then don't work on the Bangle because of the limited font.


No_Such_Method.Error cannot be displayed on types that overloaded to_display_text

Related to #6216 / #1538.
Repro
Run:
from Standard.Base import all
from Standard.Table import Value_Type

main =
    Value_Type.nonexistent

Expected result
Execution finished with an error: Method `nonexistent` of Value_Type could not be found.
        at <enso> errorinerror.main(errorinerror.enso:5:...)

Actual result
Execution finished with an error: Type error: expected `str` to be Text, but got Function.
        at <enso> No_Such_Method.to_display_text<arg-0>(C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\Errors\Common.enso:164:28-90)
        at <enso> No_Such_Method.to_display_text(C:\NBO\enso\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\lib\Standard\Base\0.0.0-dev\src\Errors\Common.enso:164:28-113)
        at <enso> errorinerror.main(Internal)

The No_Such_Method.Error that is raised cannot be displayed, because it calls to_display_text on Value_Type. Since that method is overridden to provide nicer display for  instances of Value_Type, this call yields Value_Type.type.to_display_text[Value_Type.enso:280-611] instead of Value_Type as we'd expect.

I'd start by implementing

#6300

and then checked the result on this issue.

Will be solved by #6300 so closing.


GC crash when importing pygame & pyjulia

Hi,
Julia crashes if pygame is imported after importing julia. The MWE that causes the crash is a python application which just loads pygame after JuliaCall (I get the same crash using pyjulia) and then allocates memory from a Julia thread. Swapping the 2 import lines avoids the issue
from juliacall import Main as jl
import pygame

jl.seval("""
function worker()
        for i in 1:10000000
            a = Float64[]
            push!(a, 0.42)
        end
    end
""")
jl.seval("t= Threads.@spawn worker()")
jl.eval("wait(t)")""")

Initially I created a Julia issue but I guess having the issue here is more useful.

That's weird! Why would loading a module but not using it have that effect? Presumably there is some weird threads thing going on (is pygame multithreaded? I don't know what it is) but I don't know how to diagnose it.
PythonCall/JuliaCall in general doesn't support threads (because of Python's GIL) but I would have thought that using Julia's multithreading to do Julia-only things (i.e. not calling out to Python) would be OK.
Unfortunately I can't reproduce it. Does the same thing happen if you combine those 3 seval calls into one? Also I assume the eval should be seval in the MWE.

Closing for lack of response. Feel free to reopen.

Hi, since I had a workaround for this issue, I didn't find the time to follow it up.
I just created #202 for which I don't have a workaround :-( thanks