Week 05/01

Remainder of year:
@ebridge2

[ ] run lots of data. EC2 works, so the sky is the limit. Visualize the normal way for each dataset. Start with BNU1, BNU2, DC1, HNU1, SWU4. 05/01 goal.
[ ] run one subject through CPAC for QA comparison. 05/01 goal.
[ ] finish up preliminary documentation for Ewok to merge into main documentation, and produce demonstration that what we are doing is working. Discriminability analysis for entire majority of CoRR; come up with better ways to visualize outputs. 05/08 goal.

@02agarwalt

[ ] compare CPAC outputs that Eric generates to FNGS qa.
[ ] finish up cloud deployment scripts.
[ ] get website back up to speed, so it can accept a folder from a user, upload it to S3, and anaylze, with email sent to user.

@ewalke31

[ ] task fMRI justification of our nuisance correction alternative method. Refine/debug/check previous code and make 2 simulations (one where we expect a significant difference in our simulated "white matter", one where we do not) to illustrate that our analysis was actually correct. Then, discriminability results on task fMRI correlation matrices to show that using frequency filtering is more consistent by task. Ask eric for advice on a dataset to do this with.
[ ] finish up documentation for pipeline.
[ ] replace pipeline schematic with eric's new schematic.

ok, figure out which are you committed to this week, and make the rest reach


Ignore media properties

Currently, the reference implementation ignores fetching references for the url property, this being the URL of the source post.
In testing, a media endpoint can return a fully resolved URL to a photo (or other media item); this can then get referenced, which doesn’t really make much sense, and doesn’t add any useful information. Example:
---
date: 2024-10-05T20:32:16.624+01:00
photo:
  - url: media/photos/2024/279/p1/image.jpg
references:
  https://indiekitSandboxNetlifyApp/media/photos/2024/279/p1/imageJpg:
    url: https://indiekit-sandbox.netlify.app/media/photos/2024/279/p1/image.jpg
---
Test from Micro.blog.

What happened here was that Indiekit’s media endpoint returned a fully resolved URL for the media item, and then the post template updated the URL for photo to be a relative path to the file. However, somewhere in the flow of things, the URL for the photo got referenced and appended to the post properties.
This PR adds audio, photo and video properties to url to a list of properties to ignore, and thus never seek to get reference data for.
@aciccarello; am going to publish this in the next minor release, can refine in subsequent releases if you think I may have got this wrong.

Wow, that's a strange result. Ignoring those properties seems reasonable to me.


WIP: Add rake task to sync dsi users to spreadsheet

Added a job and rake task
Fix code where it broke during local testing.
Format date time to a more readable format

To do:

Add environment variables to secrets.

@adrianclay @dushan-madetech The PR fails because of rubocop. (We will fix this)
There is also a failing test around DateTime because of time zones. ( We are also looking at how to fix this and we can give more context if you're interested).
Please review the code that we currently have.


Use jQuery().prop instead of jQuery().attr for toggling checked for d…

This fixes https://github.com/django-oscar/django-oscar/issues/1848
Basically, the dashboard client js was using the old school jQuery().attr method when it should be using jQuery().prop

Thanks :+1:


Pose Keypoint Extraction model notebook - invalid JSON formatting

The model-deployment notebook for Pose Keypoint Extraction (aws-media-replay-engine-samples/source/mre-model-samples/Models/PoseKeypointsExtractionModel/PoseKeypointsExtractionModel.ipynb) fails to load due to invalid JSON formatting.
There's a , missing between two blocks of the notebook.

Hi @Aristarkh4 The PosekeypointsExtractionModel notebook has been updated with entrypoint.py. Please pull the latest version. Thanks


No option to set password when manually registering using public key.

When the users on my room try to sign up to connect and doing so manually (Patchwork and the likes) there isn't a password field with the SSB public key field.

Because of this users need to reach out to server admins to send them password reset links.

I guess it could be an additional page after you submit the ID.

I think I can get a fix in for this.


feat(chat): use GraphQL to determine when chat should be displayed

closes #725
This effectively prolong the time when chat can be used => from now on it will be displayed also 4 hours prior the flight that precedes the one covered by the guaranteeLiveURL: https://smartfaq-use-graphql-for-chat.surge.sh

ping @kantord @vborszeki @jalbertsr

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat(Card, Dropdown, Modal): update docLink icon color

Description
This PR updates the docLink icon color across all components utilizing it. To achieve this, the State type has been adjusted to properly manage default values from the theme.
Card

updated docLink icon color

Dropdown

updated docLink icon color

Modal

updated docLink icon color

[IMPORTANT] PR Checklist

[x] I am aware of standards and conventions adopted in this repository, defined in the CONTRIBUTING.md file

PR conventions
Please make sure your PR complies with the following rules before submitting it.

[x] PR title follows the <type>(<scope>): <subject> structure
[x] The PR has been labeled according to the type of changes (e.g., enhancement, bug).

Additional code checks
Based on your changes, some of these checks may not apply. Please make sure to check the relevant items in the list.

[ ] Changes are covered by tests
[x] Changes to components are accessible and documented in the Storybook
[x] Typings have been updated
[ ] New components are exported from the index file
[ ] New files include the Apache 2.0 License disclaimer
[x] The browser console does not contain errors

Fixed snapshots


connect to daemon that uses regtest flag

I'm running a private testnet with some daemons and one in particular uses this command
monerod --stagenet --no-igd --hide-my-port --data-dir stagenet/node_01 --p2p-bind-ip <IP_ADDRESS> --log-level 1 --rpc-bind-ip <IP_ADDRESS> --add-exclusive-node <IP_ADDRESS>:38080 --confirm-external-bind --detach --fixed-difficulty 1 --p2p-bind-port 28080 --rpc-bind-port 28081 --zmq-rpc-bind-port 28082

I can connect fine to it using the library, for instance with
daemon = JSONRPCDaemon(port=28081)
print(daemon.info())

But as soon as I use the --regtest flag in the above daemon command, I can't connect anymore and get this error, any idea as to why ?
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer'))

That flag enables some testing mode. The daemon just isn't accepting RPC connections. But I have no idea what is it for and what's the expected behavior.

ok it seems, I dont know why, that --stagenet conflicts with --regtest, well when I remove it I can connect fine.
sorry for the noise


haxelib version and help

Added two simple commands:

haxelib version prints the currently using haxelib version.
haxelib help prints haxelib usage.

I'm not sure if we should add alias in the form of flags, like -h/--h/-help/--help and -version/--version.

To be honest, I have just skimmed this, but it looks straight forward enough ;)


Cannot create sharded cluster from version 5.0

Hello,
I'm having issues on upgrading local sharded cluster from version 4.4 to 5.
For example, Dockerfile for config and shard servers looks like this:
FROM mongo:5.0.21
COPY key /key
COPY scripts /scripts
RUN chown -R mongodb /key
RUN chown -R mongodb /scripts

Docker-compose.yml:
  ...
  mongo_config:
    build: ../mongo-cluster/image
    container_name: mongo_config
    command: mongod --port 27017 --configsvr --replSet configserver --keyFile /key/mongodb.key
    env_file:
      - ../mongo-cluster/mongo-credentials.env
    ...

Issue is in docker-entrypoint.sh where the script removes --replSet switch when trying to start un-authenticated mongod version to add root user and password provided in the environment file. The error presented is:
BadValue: Cannot start a configsvr as a standalone server. Please use the option --replset to start the node as a replica set.
It should remove both --configsvr and --shardsvr switches, plus add --dbPath switch because config server uses a different path.
When I try adding this options everything works.
Additions are:

_mongod_hack_ensure_no_arg --configsvr "${mongodHackedArgs[@]}"  (line 318)
_mongod_hack_ensure_no_arg --shardsvr "${mongodHackedArgs[@]}" (line 319)
__mongod_hack_ensure_arg_val --dbpath "$(dbPath "$@")" "${mongodHackedArgs[@]}" (line 344)

Is this something that can be added/modified officially?

I have a PR for this, https://github.com/docker-library/mongo/pull/600, but wasn't certain if it was good enough.
Related to https://github.com/docker-library/mongo/issues/509

I see that your PR already has one part solved, and the same issue applies to the shard server initial startup. I would really appreciate these changes going in officially, what would be the best path to take here? Can you update your PR to include the change I proposed above?

Any solutions, yet?


[Feature]: revamp about buttons background to improve look ✨

Description

We want to improve the look of about us page by revamping the buttons background color
I have already made the changes to my cloned repository, so that i'm adding current screenshot and mine as well.

Screenshots

BEFORE

AFTER

Checklist

[X] I have checked the existing issues
[X] I have read the [Contributing Guidelines]
[X] I have read the discussion tab thoroughly and got the project idea
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

Hey there @crocmons, @rohansx 👋🏻
Please make sure to assign this issue to me asap to make the work done!
Thank you.


[DONOTMERGE]expose psql credentials for iam to connect

closing this as duplicate,
original : https://github.com/IBM/ibm-iam-operator/pull/459


cookiecutter list should display user icon

We should display the icon of the user who created the template. This will help make it easier for users to find templates from trusted creators.

This looks like a trusted creator :)

I made the image clickable, so it goes to the owner's GitHub url.


[BUG] Streaming R2 object > 200MB in size exceeds CPU limit

Is there an existing issue for this?

[X] I have searched the existing issues

What version of workers-rs are you using?
main
What version of wrangler are you using?
0.0.0-3f61892d
Describe the bug
When streaming an R2 object with a size > 200 MB in size, a Rust based worker exceeds the CPU limit. When I rewrite the same worker in TypeScript, the CPU limit is not exceeded.
I've created a reproducer repository for the issue that can be used to reproduce the issue. I cannot reproduce this issue locally using wrangler. That is, I only run into this issue when deploying the worker and accessing a large file stored in the associated R2 bucket.
From what I can tell, the code that leads to the issue is:
    // body is a r2::ObjectBody
    let response_body = body.stream()?;

    Ok(Response::from_stream(response_body)?
        .with_headers(headers)
        .with_status(200))

Looking further in the workers-rs source, I think the underlying issue is that r2::ObjectBody returns a Rust stream that wraps the native ReadableStream from r2::ObjectBody::stream. Response::from_stream then converts the Rust stream into a ReadableStream, which is then passed to the native web_sys::Response::new_with_opt_readable_stream. I think this leads to invocation of the Rust wasm code while streaming chunks of the R2 object to the client.
To confirm this suspicion, I added the following function to r2::ObjectBody to allow a ResponseBody to be created using the r2::ObjectBody's inner ReadableStream. This ResponseBody can then be passed to Response::from_body to avoid converting the ReadableStream into a Rust stream and as a result, also avoid the invocation of the Rust code while streaming the R2 object:
    /// Returns a [ResponseBody] containing the data in the [Object].
    ///
    /// This function can be used to hand off the [Object] data to the workers runtime for streaming
    /// to the client in a [crate::Response]. This ensures that the worker does not consume CPU time
    /// while the streaming occurs, which can be significant if instead [ObjectBody::stream] is used.
    pub fn response_body(self) -> Result<ResponseBody> {
        if self.inner.body_used() {
            return Err(Error::BodyUsed);
        }

        Ok(ResponseBody::Stream(self.inner.body()))
    }

If this all makes sense, I can go ahead and open a PR with the response_body function added to r2::ObjectBody.
Steps To Reproduce

Clone mcnulty/cf-worker-r2-download-bug-reproducer
Follow the reproduction steps from that repository, which are included here:

Create a R2 bucket in your Cloudflare account called r2-download-bug
Initialize the worker

npm install

Generate the test file using npm run gen-test-file
Upload the test file to the bucket

rclone copy --progress bucket-src/200MB-file r2:r2-download-bug/

Deploy the worker

npm run deploy

Open an additional terminal to tail the worker logs:

npx wrangler tail

In another terminal, download the file with curl:

curl -v https://r2-download-bug.$DOMAIN.workers.dev/200MB-file > ./200MB-file

After this last step, you should see the following:

The downloaded file is not the correct size. It will be less than expected due to the
CPU limit being exceeded.
The wrangler logs will say something like:

GET https://r2-download-bug.$DOMAIN.workers.dev/200MB-file - Exceeded CPU Limit @ 9/20/2023, 10:40:04 AM
✘ [ERROR]   Error: Worker exceeded CPU time limit.

I ran into this issue myself as well. Are you still willing to submit the fix as a PR? If not, I may try sending a PR with that fix with credit.

@mbilker Thanks for the reminder! I had a fix locally that I just submitted in #462.
I also updated the reproducer repo with a branch that shows how the fix can be used in a wrangler project to fix the bug.

Thank you very much! I can confirm the response_body() works for my use-case as well and also that the Content-Length header is being set to the correct value.


Design read me quality updates

Issue #, if available: N/A
Description of changes:
Added wording updates for clarity.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I talked to Ed and moved his changes to our repo. We'll use this PR https://github.com/awslabs/aws-dotnet-messaging/pull/126 to review the changes.


ImportError - unable to run sample code

It looks like a problem with your installation of mlx.core, and likely has to do with something off on your machine. Can you share your OS, how you installed MLX, MLX version etc?

Closing this as I see you opened an issue in mlx core https://github.com/ml-explore/mlx/issues/532


Change response to updated API of @actions/github

Thank you for this course!
Just want to flag that one file was not updated in #19. Made this PR for it. :)

No need for this PR, file was already updated.


cant download the demo app for android

For Firebase backend : It redirect to page, which says app is removed or deleted

I am also facing the same issue .


Conflicts on non-syntactical elements

Sometimes, we run into conflicts on elements that aren't syntactically part of the AST, causing strange conflicts where the left and right revisions are equal. For example, given these three revisions of a method call:

left
base
right

add(-a, b, c)
add(a, b)
sum(-a, b, c)

We get the following merge conflict:
sum(-a, b,
<<<<<<< LEFT
c
=======
c
>>>>>>> RIGHT
        );

That's obviously a bit strange. This happens here because there's a conflict on the type of the argument, because that's never mapped in the SpoonMapping. As it's not part of the syntactical structure, it's just extra information that Spoon provides.
A quick solution to this would be to add a conflict handler that checks if the pretty-printed outputs of both sides of the conflict are the same. A better solution would be to not include non-syntactical information in the PCS structure to begin with.

Nevermind, this was actually caused by the left-right matcher being way too conservative again.


GitOps deletes apps during uninstallation, thereby preventing idempotency

Describe the bug
When uninstalling gitops components, gitops deletes the app deployed in the cluster. This does not adhere to the idempotency requirement of the product.
Environment

gitops: [e.g. v0.3.1-rc1]

To Reproduce

Run gitops install
Run gitops app add . --auto-merge=true
Run gitops app list and see app listed
Run gitops uninstall and see all finalizers get deleted
Run gitops app list and see error

Screenshots

Working as designed. We will be enhancing this in #890.


Cache token

We do not need to load the token on each execution, could cache it globally.

1.2.0 is released with the changes. Thank you @madisparn!


Feature - Replacing Bcrypt with NodeJS built in PBKDF2

feat: replacing bcrypt with nodes built in pbkdf2Sync to persist and  check for matching hashes of passwordString VariableAttribute dataType during BasicAuthentication handling

FYI working so far on windows 11 as well 👀🙏


When debugging non-C# projects, this plugin updates before the proper debugging begins

Issue Description
When I debug most other languages, this plugin decides it's a fine time to update.
Steps to Reproduce

Have out of date stuff for this plugin.
Have a project where there's no C#, happens a lot with Go for me.
Press F5 (without a launch.json), read C#'s logs as what you actually wanted to do is put on hold...

Expected Behavior
I debug Go. I can't believe I'm submitting this issue.
Actual Behavior
Go is put on hold while OmniSharp downloads 35MB + 42MB + 50MB of things I couldn't care less about.
Logs
Empty
OmniSharp log
Empty
C# log

Installing C# dependencies...
Platform: win32, x86_64
Downloading package 'OmniSharp for Windows (.NET 4.6 / x64)' (35884 KB).................... Done!
Validating download...
Integrity Check succeeded.
Installing package 'OmniSharp for Windows (.NET 4.6 / x64)'
Downloading package '.NET Core Debugger (Windows / x64)' (42010 KB).................... Done!
Validating download...
Integrity Check succeeded.
Installing package '.NET Core Debugger (Windows / x64)'
Downloading package 'Razor Language Server (Windows / x64)' (50489 KB).................... Done!
Installing package 'Razor Language Server (Windows / x64)'
Finished

Environment information
VSCode version: 1.48.2
C# Extension: 1.23.2
Dotnet Information
.NET Core SDK (reflecting any global.json):
 Version:   3.1.201
 Commit:    b1768b4ae7
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19041
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\3.1.201\
Host (useful for support):
Version: 3.1.3
Commit:  4a9f85e9f8
.NET Core SDKs installed:
2.1.403 [C:\Program Files\dotnet\sdk]
2.1.700 [C:\Program Files\dotnet\sdk]
3.1.201 [C:\Program Files\dotnet\sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.1.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.1.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.1.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.3 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

bracket-pair-colorizer-2
CoenraadS
0.2.0

csharp
ms-dotnettools
1.23.2

debugger-for-chrome
msjsdiag
4.12.10

EditorConfig
EditorConfig
0.15.1

gitlens
eamodio
10.2.2

go
golang
0.16.2

hexeditor
ms-vscode
1.2.1

jq-preview
ldd-vs-code
0.4.1

markdown-extended
jebbs
1.0.17

remote-wsl
ms-vscode-remote
0.44.5

rewrap
stkb
1.13.0

sqltools
mtxr
0.23.0

sqltools-driver-pg
mtxr
0.2.0

todo-tree
Gruntfuggly
0.0.178

trailing-spaces
shardulm94
0.3.1

unity-debug
Unity
2.7.5

vscode-docker
ms-azuretools
1.5.0

vscode-icons
vscode-icons-team
10.2.0

vscode-proto3
zxh404
0.4.2

Unfortunately with the current design of extension deployment in VS Code, there is no good way to avoid this. Hopefully some day we will have microsoft/vscode#23251 and the issue will get fixed.

I'm going to uninstall until the extension respects my time.


Turbine-Ruby CLI Stub

Description of change
Stubs for turbine-ruby cli. Contains functions needed by interfaces
https://github.com/meroxa/cli/blob/a3f45174615af714697d5cf043dd8751d3bbb831/cmd/meroxa/turbine/interface.go#L7
Fixes https://github.com/meroxa/cli/issues/521
Type of change

[x]  New feature (stub)

How was this tested?

[x]  Unit Tests

@dianadoherty do you think we should a feature flag for the ruby path until we finish it?

because turbine ruby commands are not exposed in this PR, this PR can be merged. But we should definitely add a flag for any future pr


Bash completions should list containers or images by default

This patch will get bash completions to list a container or an image
when a user hits the  button without a leading -.
Signed-off-by: Dan Walsh<EMAIL_ADDRESS>

@TomSweeneyRedHat PTAL

LGTM.  Two questions to possibly chase for later.  Should buildah run have a " __buildah_list_containers" near line 411?  Also buildah inspect can be used for containers or images, you've containers there which I think is right but thought I'd throw that by you to mull over.

Yes I thought about the inspect, I will fix that and run should have it also.


allow multiple yaml files

Tested it

Created PR for upstream https://github.com/ros-controls/gazebo_ros2_control/pull/149/files


Add .gitattributes for GitHub file highlighting

Render files on GitHub using the same Red linguist grammar as used in Markdown files

Merged! :)


Disable login screen for dev

Add this to ur client.env:
REACT_APP_DEV_DISABLE_LOGIN="true"

also, maybe we should create like a main account that is used for demo and stuff and use that token instead of this one

What if we want to log into different accounts to use different DB states?

I guess in this case we would disable it in env?


Add a unique ID to cast entities

Description:
Adds a unique ID to chromecast entities, based on the UUID which is computed (if I got this correctly) from some device identifier.
Checklist:

[x] The code change is tested and works locally.

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

I thought about this too, but what happens when a group changes its "elected leader" (https://github.com/balloob/pychromecast/issues/165)? Then a new entity with the same unique id will pop up in the entity registry and an error will be thrown -> problem. Potential fix: cache all Chromecast objects by unique I'd and update a cast entity when a device is re-discovered under a new IP address.

#11905

Forgot to include this in previous comment: The UUID of a cast device can also be None:
https://github.com/balloob/pychromecast/blob/4ae1a0c6add9625acb2cc40ff4dd5624ac8a2731/pychromecast/dial.py#L76

Adding unique ID is not enough, you will also need to take care of cast devices getting discovered while one is already loaded.

@OttoWinter I'm pretty sure that the UDN element of SSDP / UPnP must be a unique identifier per device, regardless of IP address or what leader is chosen. I thought it was mandatory, but yes, looks like this can also be None. Well, in that case, the cast entity just doesn't have an unique ID and doesn't benefit from the entity registry.
Does this happen in practice? The three chromecasts that I have lying around all respond with a UDN, apparently.

Consider this:

You start up Home Assistant and your cast device A with UUID 123e4567-e89b-12d3-a456-426655440000 and IP <IP_ADDRESS> is automatically discovered. A new entity is therefore created.
The cast device for some reason disconnects from your network and gets a new IP address, let's say <IP_ADDRESS>. Beware the old Entity still exists in Home Assistant and is not removed automatically (so that it can be automatically re-connect once <IP_ADDRESS> is available again).
This causes the discovery to find a new cast device with with UUID 123e4567-e89b-12d3-a456-426655440000 but with IP <IP_ADDRESS>. A new entity is constructed with the same unique_id. This time though, Home Assistant will throw an error since there can't be two entities with the same unique_id.

Note that this specific case with an IP reconnect wouldn't happen very often, but I used it to show another issue: What does happen quite often apparently is that an audio group changes its elected leader (i.e. the device that handles all of the groups connections). I'm not saying that the UUID of a group changes - in fact the UUID of a group always remains the same across leader changes. However, the IP of a group isn't constant and that can cause issues.
About the None UUID - I haven't seen it in practice yet, but there are many 3rd party cast devices that don't always adhere to the standard and we should accommodate for that.

Okay, now I get you. Basically, that's the case that also @balloob warned of. I'll try to solve this by removing the old entity then. I'm marking this PR as WIP until then.

I created #12405 which could be used to solve the problem of re-adding previously discovered devices.

Closing in favor of #12474


Feature/block hash to number

Description

block-to-hash in memory -> block-to-hash in disk

All comments are addressed! PTAL @platfowner @liayoo

Thanks I'll try it!

Thanks for reviews! I tested 2 case (Restart a node with block files untouched, Restart a node with block files removed.


Cannot integrate form validation

Environment

Tech
Version

material-ui-pickers
v.2.0.0

material-ui
v3.4.0

React
v16.4.2

Browser
Chrome v.70.0.3538.67

Peer dependency
luxon v1.5.0

Steps to reproduce

Using keyboard input mode
Using react-material-ui-form-validator to avoid sending a request to BE when FE validation is incorrect. Ideally to be integrated with react-material-ui-form-validator:
https://github.com/NewOldMax/react-material-ui-form-validator/wiki/Material-ui-v1-datepickers-example

Expected behavior
Since DatePicker is wrapping a TextField, I would expect to get an onChange callback whenever the input is changed, as it happens in a TextField component.
Actual behavior
The onChange and onError callbacks are only executed in the parent component when clicking somewhere else, even though validation is being performed inside the DatePicker component (the error message is shown).
I think it'd be great to have a way to get information about date validation status so that this component can be integrated in a ValidationForm, like an isValid prop callback, so that the form is not submited if internal validation failed.
Is there any workaround to get to know from the parent if the date is valid whenever it's modified in keyboard input mode? This way we would be able to disable the Save button if DatePicker is displaying any kind of error message.
Thank you very much for your time!

http://localhost:3002/guides/formik-integration


Discrepancy between themed and unthemed icons

About this discussion
Some of the icons does not have themed version, like com.coloros.alarmclock. It does not get themed.
On the other hand, Phone (com.android.contacts.DialtactsActivityAlias) and Contacts (com.android.contacts.PeopleActivitiAlias) have same themed icon, but they have different icons when they are not themed.

Is #1241 related?
I suggest installing Lawnchair 14 Beta 2, the latest version of Lawnicons, and let us know if the problem still occurs.

I don't have access to that device anymore. So, unfortunately I cannot test it.


feat: generic DataFrame

resolves #292. please check individual commits for detail.

Overall looks good, but I think we should update some of the docstrings to include examples with the new type system.

applied in ed11909 (#293) and 1cf2275 (#293).

thanks @scarf005


Color parameter type support for color picker.

Is your feature request related to a problem? Please describe.
"ParameterType Color Parameter (RGB_BG_COLOR): not supported colorspectrum 5"
Describe the solution you'd like
A color picker for selecting rgb value of leds using the color picker in ets.

Please be more specific.
A color picker is already supported for ets.

Create a ParameterType Color and select one of RGB/HSV (RGBW not supported by ets, but its a valid option due to knx spec).

I can add the color paramer but when i am add it into the dynamic section the output compilation throws this error.

4 CHANNEL TOUCH SENSOR.zip

Ah i see there is a bug with the wrong error message.
Your error is that your value for the parameter RGB_BG_COLOR is not in the right format.
Valid is "#RRGGBB"
Your value is "1"

It's worked.
Thank you.


gofail.go: exit when 'enable/disable' is not specified

Otherwise 'gofail -h' panics.
@heyitsanthony

@heyitsanthony I have no write access to gofail. Can you merge this for me? Thanks!

/cc @xiang90 I don't have write access to my own repo haha

lol... lgtm. i will fix the permission issue.


No alignment for Urdu

Hi,
I have used this model to train on my speech Urdu dataset. It contains 10000 .wav files of 15 hours speech dataset. Average file is size 5.4 sec. I used default parameters and trained for 50000 steps. I used transliterated labels and transliteration_cleaner. Here is the alignment.

Can you please share the synthesize audio file .

Thanks for replying. Here are the synthesized audio files at 50000 steps.
https://aown.me/eval-5000-0.wav
https://aown.me/eval-5000-1.wav
https://aown.me/eval-5000-2.wav
Here is training sample
https://aown.me/45.wav

@AownMohammad the same problem has occured with me I have iterate over 500K steps and my loss is 0.06 but still the problem is same.

I have

@AownMohammad the same problem has occured with me I have iterate over 500K steps and my loss is 0.06 but still the problem is same.
I have been trying with different settings but none is working. If it the model can work on Arabic then it should work on Urdu too.

@AownMohammad I think we should tune the hyper parameters according to it.

@tayyabvohra But how? training takes alot of time. Training again and again with different hyprams would become impossible.


nginx config & prefix paths on Windows

I have nginx installed to c:\nginx\, and a node app at c:\node\stock\. I have my gruntfile.js at c:\node\stock\gruntfile.js.
I tried config: 'c:\\nginx\\conf\\nginx.conf' and prefix: 'c:\\nginx'. Also config: '../../nginx/conf/nginx.conf' and prefix: '../../nginx/'.
I'm getting this error message:

Exception thrown in attempt to start nginx: Error: Error: not found: nginx

What should these paths be..?

'nginx' needs to be in your PATH. On windows this is done via environment variables.

Thanks Alex!
It's working now, after adding to the PATH env vars and a console restart.
Very helpful node package...
Cheers!


ADD[105]:Need to update the Our Contributors section

Added a new link to the contributors page for the year 2023.
Changed the href attribute of the link to the contributors page for the year 2022 to filter the results to only show the contributors for that year.
Updated the text of the links to be more descriptive.

Benefits:
The changes made in this pull request will make the project more user-friendly and easier to navigate. The updated Our Contributors section will make it easier for users to find the contributors who have made contributions to the project in the year 2023 and 2022.

Hi @ShehrozIrfan,
Thanks for pointing out the issue. I have investigated the issue and made the necessary changes to the README.md file.


Not able to read nano-second timestamp columns in 1.0 parquet files written by pyarrow

Describe the bug
Here is a pandas dataframe with nanosecond timestamp Data index:
>>> hist.index
DatetimeIndex(['1986-03-13', '1986-03-14', '1986-03-17', '1986-03-18',
               '1986-03-19', '1986-03-20', '1986-03-21', '1986-03-24',
               '1986-03-25', '1986-03-26',
               ...
               '2021-05-28', '2021-06-01', '2021-06-02', '2021-06-03',
               '2021-06-04', '2021-06-07', '2021-06-08', '2021-06-09',
               '2021-06-10', '2021-06-11'],
              dtype='datetime64[ns]', name='Date', length=8885, freq=None)

When storing this dataframe into parquet 1.0 format, pyarrow stores the Date column in microsecond unit. pyarrow is able to load the Date column with microsecond precision as well:
>>> from pyarrow.parquet import ParquetFile
>>> pp = ParquetFile("test_data/msft.parquet")
>>> pp.metadata.schema
<pyarrow._parquet.ParquetSchema object at 0x7f720d1bbac0>
required group field_id=0 schema {
  optional double field_id=1 Open;
  optional double field_id=2 High;
  optional double field_id=3 Low;
  optional double field_id=4 Close;
  optional int64 field_id=5 Volume;
  optional double field_id=6 Dividends;
  optional double field_id=7 StockSplits;
  optional int64 field_id=8 Date (Timestamp(isAdjustedToUTC=false, timeUnit=microseconds, is_from_converted_type=false, force_set_converted_type=false));
}

But when loaded using arrow parquet crate, it is incorrectly loaded as nanosecond timestamp type.
To Reproduce
Here is a sample file to reproduce the issue: https://github.com/roapi/roapi/files/6599704/msft.parquet.zip.
The file can be reproduced with the following python code:
import yfinance as yf
hist = yf.Ticker('MSFT').history(period="max")
hist.to_parquet('msft.parquet')

Expected behavior
Data column should be loaded as micro second precision.
Additional context
Arrow parquet crate handles parquet 2.0 files without any issue.
Initially reported in https://github.com/roapi/roapi/issues/42.
Here is the decoded ipc field from the 'ARROW:schema' metadata for the Date column in arrow crate:
Field {
    name: Some(
        "Date",
    ),
    nullable: true,
    type_type: Timestamp,
    type_: Timestamp {
        unit: NANOSECOND,
        timezone: None,
    },
    dictionary: None,
    children: Some(
        [],
    ),
    custom_metadata: None,
}

I'd have expected the ns timestamp to be written to int96 as that is the legacy nanosecond timestamp format. I'll also dig into this to see, maybe if the format is 1.0 then we should coerce the timestamp to a different resolution, but I'd also see this as a pyarrow quirk that could be documented somewhere.
@emkornfield do you know what we should do on the Rust side to roundtrip the file correctly?

pyarrow parquet writer has an option that one can set to write ns timestamp as int96 for parquet 1.0, but is turned off by default. It is quite strange that decoded IPC schema comes back a ns type even though it's stored as ms.

@emkornfield do you know what we should do on the Rust side to roundtrip the file correctly?

Sorry, I would have to dig in the code to have a better understanding.  In general we try to avoid int96 because it is a deprecated type.  If this is an issue with pyarrow can we open a separate bug to track that?

This may be related to which priority we give when converting, the parquet schema or the arrow schema. I would expect pyarrow to write them in a consistent manner though, so, as @nevi-me mentioned, an arrow schema in ns with a parquet schema in ms does seem odd.

https://github.com/apache/arrow/blob/8e43f23dcc6a9e630516228f110c48b64d13cec6/cpp/src/parquet/arrow/schema.cc#L215 is probably the relevant writing code for 1.0 so that might explain the discrepancy, let me see If I can dig up the reader code.

Here is C++ attempts to reuse original metadata.  It doesn't look like we would use the arrow schema in this case.  I'm guessing conversion back the Pandas type happens correctly because microseconds would be cast back at the arrow to pandas layer.

So I think there is probably a bug in pyarrow here since we wouldn't round trip a pure timestamp[ns] type, but we don't run into it in from pandas because the casting just works.

I believe this is a duplicate of https://github.com/apache/arrow-rs/issues/1459 which was fixed by https://github.com/apache/arrow-rs/pull/1682. Please feel free to reopen if I am mistaken


Editor tab close button on the left side

Is your feature request related to a problem? Please describe.
I personally find it annoying to close multiple adjacent editor tabs.
Because they have differing widths, I constantly have to move the cursor.
Describe the solution you'd like
It would be much nicer if there would be an option to have the tab close button on the left side instead of on the right side.
Describe alternatives you've considered
Enforce a uniform tab width. But in that case, long editor tab title would have to be truncated.

It will be better to implement closing tabs on mouse middle click

This leaves out track-pad users who don't have a middle mouse button. Still, this seems like a good additional option.

They can use keyboard shortcut, allowing unnecessary over-customisation of UI is not a good idea.


Can I estimate the depth of the video stream directly?

What should I do if I want to estimate the depth of the video stream directly, will the delay be large？

What should I do if I want to estimate the depth of the video stream directly, will the delay be large？

You can use ffmpeg to extract each frame and load current temporal winow (three previous reference frames and the current target frame) to NVDS model.  You can remove the bidirectional inference and only use our forward prediction for faster speed. Besides, in our implementation, we use Midas, DPT, or NeWCRFs as depth predictors, which has larger computational costs (e.g., GFlops) than our stabilization networks. You can also replace the depth predictors with more lightweight depth models (e.g., Midas-small) for faster speed.
NVDS is much faster than prior arts (e.g., CVD, Robust-CVD, DeepV2D, WSVD, FMNet, etc) and the model efficiency is evaluated in our paper and supplementary. But NVDS model structure cannot achieve real-time processing (i.e., >25 fps) for now. More lightweight structures for real-time applications can be explored in future works.


Keep the HTML layout of column in all circumstances…

Otherwise the CSS layout breaks and the cards are not visible. Fixes #277.

Coverage increased (+0.05%) to 47.375% when pulling d4dc4e1c9f7ec96115046b632080ccc75e9aa5c0 on master into 8d9a3d3caa9b8608c7b51ae2dbd7796b1f60094f on release.

Coverage increased (+0.05%) to 47.375% when pulling d4dc4e1c9f7ec96115046b632080ccc75e9aa5c0 on master into 8d9a3d3caa9b8608c7b51ae2dbd7796b1f60094f on release.


Added website logo

Fixes #195
I have added the website logo in the navbar it will always be visible and can be seen on any background
@sk66641 please review and merge this pull request

@sk66641 i resolved the conflicts


Fix GOOS auto get on MacOS

In the MacOS operating system, we need to manually modify the value of GOOS to compile, which is not enough to traverse. Using uname in Unix to automatically obtain the value of GOOS, so we can directly perform the Make operation.
Good Luck.

In the MacOS operating system, we need to manually modify the value of GOOS to compile, which is not enough to traverse. Using uname in Unix to automatically obtain the value of GOOS, so we can directly perform the Make operation.
Good Luck.

Great, thank you.


UDP protocol support

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
We are using Mosn as data plane and added udp_proxy feature to Mosn. So I want to know can istio support transport protocol config in LDS and udp_proxy CRD, or if there is any plan to support it? Thanks a lot.

Closing as a duplicate of https://github.com/istio/istio/issues/1430


Second Jest test failed

Hi,
I am not sure if i understand jest testing correctly. I need run test before I am going to change production website. And after I've made these changes. But...If i run test twice, second test failed due to timeout (toMatchSnapshot).
What's wrong?
const Differencify = require('differencify');
const differencify = new Differencify({ debug: false });

describe('Test1', () => {
    it('Homepage', async () => {
        await differencify
            .init()
            .launch()
            .newPage()
            .setViewport({
                width: 2800,
                height: 2800
            })
            .goto("https://example.com/")
            .screenshot()
            .toMatchSnapshot()
            .close()
            .end();
    }, 10000);
});

Hey @wrongware ,
I just tried your script a couple of times locally and I couldn't reproduce it. Possibly your machine is just a bit slow to process the Image comparison as that only happens on the second run. What I suggest is to increase the timeout to 30000 and see if that helps.
You test is very similar to the integration test that already exists in Differencify here and as you can see the timeout is around 3secs. https://github.com/NimaSoroush/differencify/blob/84489a38ca44a22e3c1e574f7a12fc88f6975e3f/src/integration.tests/integration.test.js#L12

Hi, i've figured it out. I guess, Puppeteer is waiting for loading some external scripts like Facebook, so I've disabled Javascript and resized viewport. And it works.
it('Homepage', async () => {
        await differencify
            .init({
                testName: "homepage"
            })
            //.launch() // cuz shared browser (multiple tests)
            .newPage()
            **.setJavaScriptEnabled(false)**
            .setViewport({
                **width: 1200,
                height: 1000**
            })
            .goto("https://example.com/")
            .wait(500)
            .screenshot()
            .toMatchSnapshot()
            .close()
            .end();
    });

I think there are some performance issues because I got gaming notebook w 16GB, 8-core Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz, SSD disk etc. It is enough power to solve image comparsion.

Good to hear that you figured it out!


Variables (page numbers etc) not appearing in Word

I'm following up on a question that I raised on the Zotero forums, because I found that variables like page numbers were not making it through to my Word document when I used the lua filter to keep the citations 'live' in Word. Please find my word doc attached. It's in a state before hitting document preferences so you can see the strings that got passed through.
Text.docx

Report ID: LZ6R9CUU-apse

:robot: this is your friendly neighborhood build bot announcing test build <IP_ADDRESS>83 ("ncu")
Install in Zotero by downloading test build <IP_ADDRESS>83, opening the Zotero "Tools" menu, selecting "Add-ons", open the gear menu in the top right, and select "Install Add-on From File...".

Thanks for reporting this. Can you try 6583 with https://raw.githubusercontent.com/retorquere/zotero-better-bibtex/gh-1528/site/content/exporting/zotero.lua ?

BTW, do you recall how you opened this issue? Were you logged in when you opened it? You should have been prompted to choose from 5 categories of issue types, and each should have added a label to the issue, but this issue wasn't labeled, and I've seen other that weren't labeled.

Oh also: the config for this filter has changed; docs are temporarily up at https://github.com/retorquere/zotero-better-bibtex/blob/gh-1528/site/content/exporting/pandoc.md until I merge this.

Hmm. Now the citations are just getting passed through as plain text. I wasn't sure what variable to put in the library: field, but I tried various things including leaving it blank, and nothing seemed to work. Passing along the md file I used (converted to txt). Was running this:
pandoc -s --lua-filter=zotero.lua --reference-doc /Users/MYNAME/.pandoc/reference.docx -f markdown+smart -t docx -o /Users/MYNAME/Desktop/"$title".docx "$@"
Text.txt
I got here from the 'submit an issue' link on this page.

And you weren't prompted to choose an issue template? Strange.
Can you send a new debug log from 6583, selecting the references in that markdown file?
If these are items from your personal library, omit the library entirely; if it's from a group library, use the name of the library.

I've replaced the citekeys that were in the document with some that happen to be in my test lib, bit after compilation, the text initially looks like
.... many hundreds of thousands of men were never more than instrumental?<open Zotero document preferences: [See also @Villagrasa; For another account of xyz, consult @biggs1996]>
Some text to make a footnote, hopefully...

Which is what I expect; after opening the Zotero document preferences and selecting a style (I usually go for APA) and clicking OK, the document now looks like:
... many hundreds of thousands of men were never more than instrumental?(For another account of xyz, consult Biggs, 1996; See also Villagrasa & Duran, 2013)
Some text to make a footnote, hopefully....

Oops, I completely missed the part where I needed to install the new build :| Sorry about that. Everything is working perfectly well now, all the variables are passed correctly! Many thanks.
Yes, when I follow that link (https://github.com/retorquere/zotero-better-bibtex/issues/new) it lets me add a new issue without categorizing it.

Cool thanks -- I'll merge and cut a new release.
Where did you get this link? The link on github itself is now https://github.com/retorquere/zotero-better-bibtex/issues/new/choose

Ah, I see I had that on my site. That's fixed now, should be up in 20 minutes or so.


Bring back removed API on PartitionQosPolicy. [7902]

Methods getNames and setNames were renamed, so we should keep an alias in order to maintain the old API.

Build status:

Linux
Mac
Windows

Build status:

Linux
Mac
Windows


pullbabypull

Coverage remained the same at 98.058% when pulling 77c96d9296c49fe074034befa353a86402323da1 on Reykjavik-Rockstars:master into<PHONE_NUMBER>be99ef0ff30d6423afe54f3ee310b0 on dstar55:master.


[BFCL] Add ToolACE handler for BFCL-v3

This PR adds the handler of the ToolACE model, which finetunes LLaMA-3.1-8B-Instruct model with ToolACE dataset, obtaining wonderful points in functional calling.
We have adapted our handler compatible with version 3.
Here are the results of the version evaluated in our machine (4*v100-32GB). We also found that the results would change in different machines.

Rank
Overall Acc
Non-Live AST Acc
Non-Live Simple AST
Non-Live Multiple AST
Non-Live Parallel AST
Non-Live Parallel Multiple AST
Non-Live Exec Acc
Non-Live Simple Exec
Non-Live Multiple Exec
Non-Live Parallel Exec
Non-Live Parallel Multiple Exec
Live Acc
Live Simple AST
Live Multiple AST
Live Parallel AST
Live Parallel Multiple AST
Multi Turn Acc
Multi Turn Base
Multi Turn Miss Func
Multi Turn Miss Param
Multi Turn Long Context
Multi Turn Composite
Relevance Detection
Irrelevance Detection

1
59.22%
89.27%
80.58%
95.00%
91.00%
90.50%
90.07%
98.29%
94.00%
88.00%
80.00%
73.21%
62.79%
74.25%
81.25%
75.00%
14.37%
21.50%
6.50%
17.50%
12.00%
N/A
85.37%
83.81%

Thanks for your efforts in holding such a wonderful leaderboard. We need your help (@HuanzhiMao, @CharlieJCJ ) in adding our model to the leaderboard. Thanks a lot～

Apologize for the long delay. Will definitely take a look after the ICLR deadline.

Apologize for the long delay. Will definitely take a look after the ICLR deadline.

Thanks! Good luck with your submission~


Unable to see imported Custom App package in Apps for Sharepoint interface

Reporting an Issue or Missing Feature
I'm unable to see imported Custom App Package through "Import-SPOAppPackage" cmdlet in Apps for Sharepoint screen.
Expected behavior
Apps for Sharepoint should list newly imported .app file
Actual behavior
Newly imported apps not listed in Apps for Sharepoint screen
Steps to reproduce behavior
I have used below command to import the package in my app catalog site.
Steps:

Logged into my app catalog site through powershell
Run below command
Import-SPOAppPackage -Path "c:\users\home\Sharepoint Employee Apps.app" -LoadOnly

What version of the PnP-PowerShell Cmdlets are you using?
March PnP-PowerShell SharePoint Online (V16),
How did you install the PnP-PowerShell Cmdlets?
Via MSI installer

Hi!
Indeed, an app loaded like that will not show up in the Apps for SharePoint page, as it's directly added to the site. You will find it in the Site Contents overview. If you want the app to show up in the Apps for SharePoint page you will have to upload it to your app catalog.


duallistbox

has label with counts of selected items

@ashishpat @szul i think this should be approipate for adding labels


Fix playground iframe.

Very little changes.

Remove fixed default value of kyselyVersion to use latest version.
Trim example codes to prevent annoying(at least for me) empty line.

By the way, the StackBlitz version of playground in kysely.dev looks nice and I'll do what I can do until all playground replaced with it :(

It seems something wrong, maybe because I reuse merged branch?

It seems something wrong, maybe because I reuse merged branch?

Is it good to go now?

Thank you!

By the way, the StackBlitz version of playground in kysely.dev looks nice and I'll do what I can do until all playground replaced with it

There's no plan to start using it everywhere. It was added by @fhur, probably because its look fits the front page better.
It's important to me that the playground we use provides the same experience as vscode. Using your version might just be the way to have that control.

It seems something wrong, maybe because I reuse merged branch?

Is it good to go now?

Yes.


Question: Middleware order

Is there a way to control controller/method middleware (UseBefore, UseAfter) execution order?

export function UseBefore(...middlewares: Array<Function>): Function;

Isn't it now the order of middlewares passed to decorator?

Thanks @19majkel94 ; i don't know what i was thinking :-)


Default save location

Give SequenceProcessor.save() a default location to save a trained model.
Closes #21.

Pull Request Test Coverage Report for Build 116

3 of 15 (20.0%)  changed or added relevant lines in 5 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 74.813%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

saber/utils/app_utils.py
0
1
0.0%

saber/utils/generic_utils.py
1
3
33.33%

saber/sequence_processor.py
1
5
20.0%

saber/train.py
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

saber/sequence_processor.py
1
70.9%

saber/train.py
2
0.0%

Totals

Change from base Build 115:
-0.1%

Covered Lines:
1203

Relevant Lines:
1608

💛  - Coveralls


Simple Translation service

Hi Roberto,
I don´t know if you had in mind to implement some kind of translation system. This is the current implementation I´m using within an app and it works great, even with a large amount of words to translate.
An easy-to-use approach:

One .ts file per language (I have already included lang-en.ts, lang-es.ts and lang-it.ts) where we define the different words/texts to translate:

export const LANG_EN_NAME = 'en';
export const LANG_EN_TRANS = {     // Main Headers, titles ...     mainHeaderHi: 'Hi %0!',     titleSignin: 'Sign in data',
...

An Observable object which we need to include within every component where we need to perform a translation:
refreshTranslate: Observable<boolean>;
...
this.refreshTranslate = this.translationService.getRefreshTranslateObservable();

Important note:
We share an Observable object within all the components where we need to use the translation service in order to trigger the change detection. This allows us to use a pure type Pipe, which is important. Impure pipes should never be used due to its impact over performance.

The we just use a Pipe where we need to include a multi-language text:
-- simple way: {{ 'signin' | translate: [refreshTranslate | async] }}
-- with args: {{ 'mainHeaderHi' | translate: [refreshTranslate | async, name] }}

Any args received wont be translated but replaced with the common notation %0, %1 etc... defined at the .ts files.

Sorry for my bad initial Italian translation and I hope you find this functionallity interesting to include within your great project.
Regards,
Andrés.

Hi Andrés,
thank you so much for this pr, for sharing your code, and for the effort, but I have two problems:

For translation, I'm maintaining this library: https://github.com/robisim74/angular-l10n
I never included it in this project, because the goal of this project is to show and improve the authentication service in a Single Page Application, and not to be a complete Angular/ASP.NET Core template, at least for now.

I apologize, but I can't merge it. If you agree, I would keep this request open so that someone interested can take it into consideration.
Thanks again.
Greetings,
Roberto

Hi Roberto,
I understand! I didn't realized about the main goal of the project :)
You can leave the request open or close It if you prefer, there is no problem!
Regards,


Add custom tenant info

Description of changes:
New get tenant endpoint
Changes to tenant db model
Dynamic title/logo/description based on tenant
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1618 (e385622) into main (473e61c) will increase coverage by 0.02%.
The diff coverage is 78.72%.

@@            Coverage Diff             @@
##             main    #1618      +/-   ##
==========================================
+ Coverage   71.75%   71.78%   +0.02%
==========================================
  Files         339      341       +2
  Lines       10184    10226      +42
  Branches      774      776       +2
==========================================
+ Hits         7308     7341      +33
- Misses       2753     2762       +9
  Partials      123      123

Flag
Coverage Δ

metapi
79.43% <82.05%> (+0.01%)
:arrow_up:

metweb
64.99% <62.50%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

met-web/src/apiManager/endpoints/index.ts
100.00% <ø> (ø)

...eb/src/components/layout/Header/InternalHeader.tsx
80.64% <62.50%> (-3.98%)
:arrow_down:

met-api/src/met_api/services/tenant_service.py
73.68% <66.66%> (-4.89%)
:arrow_down:

met-api/src/met_api/resources/tenant.py
75.00% <75.00%> (ø)

met-api/src/met_api/models/tenant.py
100.00% <100.00%> (ø)

met-api/src/met_api/resources/__init__.py
100.00% <100.00%> (ø)

met-api/src/met_api/schemas/tenant.py
100.00% <100.00%> (ø)

SonarCloud bugs will be addressed in a separated ticket since its a application wide problem #1619


[Bug]: DockerImage deployment fails with uninitialised php variable error

Description
I tried deploying a docker image, but the build fails with ApplicationDeploymentJob::$build_image_name must not be accessed before initialization. I am guessing that it has something to do with this line https://github.com/coollabsio/coolify/blob/main/app/Jobs/ApplicationDeploymentJob.php#L283-L284
Minimal Reproduction (if possible, example repository)
Try deploying a docker image like in this picture

Exception or Error
[2023-Nov-06 16:06:06.481922] Starting deployment of stremio/server:latest.
[2023-Nov-06 16:06:07.038061] Oops something is not okay, are you okay? 😢
[2023-Nov-06 16:06:07.212719] Typed property App\Jobs\ApplicationDeploymentJob::$build_image_name must not be accessed before initialization
Version
v4.0.0-beta.110

Just experienced the same issue described by @iykekings while trying to deploy like in this picture on v4.0.0-beta.110:

Exception or Error
[2023-Nov-06 19:09:48.109543] Starting deployment of filebrowser/filebrowser:latest.
[2023-Nov-06 19:09:48.465701] Oops something is not okay, are you okay? 😢
[2023-Nov-06 19:09:48.546099] Typed property App\Jobs\ApplicationDeploymentJob::$build_image_name must not be accessed before initialization

It will be fixed in the upcoming version.


Client Allocation Request for: NewToken

Client Allocation Questions
Core Information

Name: NewToken
Website / Social Media: http://vetetable.com/
Region: Greater China
DataCap Requested: 10TiB
Addresses to be Notarized: f15d5pxriekd6rcxx2t5xlebbubc5qlqtcuyewxfy
Notary Requested: ozhtdong

@niutonghz Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Thanks for your request. Can you kindly add more information here?
Is your company located in Greater China？
Please give the following information and provide links to social media/news mentions.
Duration of establishment,
Number of employees,
Business scope and products or services,
Social media link (Weibo, WeChat, Douyin, Zhihu, Facebook, Twitter, Linkedin, e.g.)
Data size to be stored,
Have you already applied for DataCap to another notary, and whether it is accepted or rejected? Is there any reliable DataCap left?
Would you like to accept our face-to-face visit?

Thanks for your request. Can you kindly add more information here?
Is your company located in Greater China？
Please give the following information and provide links to social media/news mentions.
Duration of establishment, Number of employees, Business scope and products or services, Social media link (Weibo, WeChat, Douyin, Zhihu, Facebook, Twitter, Linkedin, e.g.) Data size to be stored, Have you already applied for DataCap to another notary, and whether it is accepted or rejected? Is there any reliable DataCap left?
Would you like to accept our face-to-face visit?

Thank you for your reply. Our company is located in China with less than 20 employees. We mainly provide company website building and APP development. The stored data size is about 100T. No datacap has been applied to other notaries yet. We are happy to interview.

Please share your contact person(way) and your address. Besides, why didn't you provide any social media link. Thanks!

Please share your contact person(way) and your address. Besides, why didn't you provide any social media link. Thanks!

Hello, I'm sorry. I saw your reply and asked if we could communicate face-to-face, so we didn't show the detailed information. Here's a supplementary display：
1.My name is Anna. I am the person in charge of Newtoken Information Technology Co., Ltd. and also a plant lover. I am the person in charge of the website we applied for storage. The website was founded in 2019. The website aims to promote plant science, popularize plant knowledge, and let more people understand the habits and characteristics of different plants, so as to realize the peaceful coexistence between mankind and nature, Let the earth be full of green and oxygen everywhere.  http://vetetable.com/aboutus
You can refer to it.
2.I will email you the business license and contact information of the company.
3.This application for 10t data quota hopes to be approved!Our total reserve is expected to be about 100t.
4.We need four miners and will allocate no more than 40% to each miner.
5.We record our allocation decision in github.
6.We can use filfox to inspect miners. We inspect miners who are large scale.
Hope to get your reply quickly, thank you！

Hello, Because the founding team of our company has worked in the Internet industry for more than 10 years and has a deep customer relationship foundation, since we started our business in 2019, many customers or friends need our services, so we didn't do external publicity.
MyChat: newtoken360
If you have any questions, you can call me at any time.
The attachment is the business license of our company. Please check it.
At 2022-03-15 15:30:26, "ozhtdong" @.***> wrote:
Please share your contact person(way) and your address. Besides, why didn't you provide any social media link. Thanks!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>


Support for Interpolations 0.14

Current Julia versions (1.8) often build dependency trees requiring Interpolations.jl v0.14 and newer.
Would be nice to add this and trigger a release. The commands that I use (like movavg) seem wo work also with the newer Interpolations.jl version, however I did not check every function.
PS: If you're lazy I opened a PR :-)

Done.


Remove dead headers

Remove a set of headers included by no files outside of the set.

Would it be fair to say this PR is "trivial"?


fix: reset css

General

✏ Mark the necessary items without changing the structure of the PR template.

[x] Pull request template structure not broken
fix #2690

Type

ℹ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

ℹ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

[x] My code follows the style guidelines of this project
[x] Is the code format correct
[x] Is the git submission information standard?
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

main分支已不在维护，新的pr可以提交到next分支


Added tests to test_utils.py

Added tests for count() and mode().

I don't feel the tests for count are representative of the function. In the cases you provided, count works just as len, returning the length of the input. In my opinion a better test would be this:
count([True, False, True, True, False])
The above has a length of 5, but count will return 3 since that's the number of elements which are True in the input. You can expand the above example to include conditions. For example:
count([5 > 1, len("abc") == 3, 3+1 == 5])
This better showcases the function.


[nrf noup]: Do not use newlib nano for C++ unordered hash map test

The hash map test: libraries.hash_map.newlib.cxx_unordered_map.djb2 is testing C++ Hash map implementation using newlib.
C++ select C++ exceptions which is incompatible with newlib nano. Newlib nano is default in NCS when newlib is selected, and thus building of the test will result in a Kconfig failure.
Support the test by disabling newlib nano but keeping newlib for the test.
Jira: NCSDK-27925
Jira: NCSDK-28096

NCS Newlib nano default when newlib is selected is specified here: https://github.com/nrfconnect/sdk-nrf/blob/489ee63627ef46c62de164a4bace745299d4a63d/Kconfig.nrf#L47-L51

I am fine with such solution because I don't have to deal with [noups] ;) still, have to raise the question: can we allow for such noup?
Does executing this test in NCS brings us any value? If not, we can just add it to the "won't fix" section of quarantine and forget about it.

Does executing this test in NCS brings us any value? If not, we can just add it to the "won't fix" section of quarantine and forget about it.

that's also fine with me.
Just opened this PR to get the discussion going 🙂

I have approved but it is okay to quarantine and state wont fix. @PerMac  if you are ok we can ask Torsten to close this PR.

@PerMac  and @tejlmand  Even it is decided to get this in, It can go after NCS 2.7.0. Is that right ?

yes. All previous platforms were already sitting in the quarantine for a while, the new ones are also added there. There is no ruch in dealing with the issue now

added to quarantine instead.
No need for this PR.


Allow creating commits without parents without a branch

From @talex5

I couldn't find a way to make a commit without a parent without
using a named branch. The special-case initialisation code could go
away if Irmin allowed that.

See c3d81e379fc80afe8e0bf6ac287b6ce76f356fbe


Update README.md

Who would have thought that a name could have so much controversy.   I've renamed due to a bunch of flack about devops not logging into servers :)

No worries, I like the new name more anyway :smile:

Coverage decreased (-0.02%) to 69.96% when pulling 966f5c1f32d161f055d0abdae204d958d54302dd on ricktbaker:patch-1 into ea8bd8dc1ce5e5bce2a2a9ea31c0f660495cfa5c on sourcelair:v3.


pull request for e2e test at 12/14/2016 02:56:02

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


UI for Registration with email and with google auth

GOSA1
ISSUE #2
Describe the changes you've made
As told I had made UI for sign up page along with button for google sign in.I have made respective fields of name , email and password for registration and for now i had just pushed the google sign in button below to the sign up button which can be changed accordingly.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Additional context (OPTIONAL)
Screenshots added -

Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] The title of my pull request is a short description of the requested changes.

ok doing!

changes done!

i have updated changes


Tachyus

@panesofglass @jackfoxy wanna add Tachyus info to the readme?

I will creating PR to add Tachyus soon.


Contribution1: fixed punctuation issue

-Removed punctuation
-Added punctuation
-Rephrased sentence

These changes look like a duplicate of a new PR https://github.com/wagtail/wagtail/pull/9443
I have put comments on that newer PR so will close this for now.
Thanks @bolarinwa-ajayi1 - I know it's a bit tricky handling different branches the first few times.


is kolia support python3.6?

Sorry, not yet. It doesn't even support 3.7 #7 haha. We'll get there eventually 😊

SO SAD

Sorry, not yet. It doesn't even support python3.7 (#7) haha. We'll get there eventually 😊

hi, i try to use kolia in python3.8, but i miss trouble too.  i follow the getting-started.py and use dgl  package to train graph data, but i have the same RuntimeError: CUDA out of memory., can u  give a more detailed example of dividing the training, validation, test data set? i should add the code (lazy_input, lazy_label) = lazy(input, label, batch=0) before or after dividing the dataset ? thanks a lot!

Hi, thanks for the report!
Koila is a gradient accumulator, so what happens is that is automatically tries to split up the batches (in the background) in the case that there is OOM. However, it doesn't know how to handle "some" layers currently, so it has no effect on those layers. This is done silently in the background. Turns out that if a layer is not recognized by the framework, it doesn't prevent OOM.
As of right now, this project is in a re-write (see #18), which should've happened last month. But I just got a job at a company so currently the schedule is very messy. So I have to apologize for not having time to fix many existing bugs, which would be automatically fixed in a more modular design.
Thanks for reading!

It's okay, looking forward to the next version


Question regarding typing

Hi love the package! I have a question regarding typing within confection.
If I try to run the line:
registry.optimizers = catalogue.create("my_package", "optimizers")

I get the following error from my static type checker:
Cannot assign member "optimizers" for type "type[registry]"
  Member "optimizers" is unknown

Similarly, I also get a type error, when decorating the function:
@registry.optimizers.register("adam")
def create_adam():
   pass

Cannot assign member "optimizers" for type "type[registry]"
  Member "optimizers" is unknown

Naturally, I could just ignore the error (not ideal) alternatively I have created a wrapper class:
def MyRegistry(registry):
    optimizers = catalogue.create("my_package", "optimizers")

However, I am unsure if this is intended behaviour (if so it might be relevant to update the docs)

Thanks for the note, we'll take a look!

I don't think there's a way around this with using this syntax. Python allows dynamically adding new attributes - such as optimizers to registry - but typecheckers like mypy complain about it. What we're doing when adding a new attribute to registry is akin to
class A:
    x = 3
A.y = 4

It makes sense that mypy & co don't like that.
The type-safe alternative would be to do setattr(registry, "optimizers", catalogue.create("my_package", "optimizers")), but that's less concise. I'm afraid the choice is between ignoring the typechecking errors and using the more verbose setattr() approach. We'll add this to the readme.

Thanks for the clarification the wrapper class seems like the best way to make it type-safe. After having worked with it for a while I would probably recommend it.
class Registry(registry):
    optimizers = catalogue.create("my_package", "optimizers")

It naturally required that don't want to dynamically extend the registry, but I don't imagine that to be common.

It naturally required that don't want to dynamically extend the registry, but I don't imagine that to be common.

I'd agree with that, yes. Thanks again for bringing this up and proposing an alternative solution!


[QUESTION] Call to a member function getMergeCells() on string

Hello,
I just intalled Laravel-Excel, however once I add store to the example version (or use any of the examples on the website) I get an error like shown below. How can I solve this?
Package version, Laravel version
5.3
Expected behaviour
Store an excel file
Actual behaviour
[Symfony\Component\Debug\Exception\FatalThrowableError]
Call to a member function getMergeCells() on string
Steps to reproduce the behaviour
Excel::create('Filename', function($excel) {

    // Set the title
    $excel->setTitle('Our new awesome title');

    // Chain the setters
    $excel->setCreator('Maatwebsite')
          ->setCompany('Maatwebsite');

    // Call them separately
    $excel->setDescription('A demonstration to change the file properties');

})->store('xlsx', storage_path('excel/exports'));

I ran into this issue too, it appears that you need to add a sheet within the callback eg:
Excel::create('Filename', function($excel) {
$excel->sheet('Sheetname', function(\PHPExcel_Worksheet $sheet) {
        });

})->store('xlsx', storage_path('excel/exports'));

Awesome. Thanks @luketarplin that helps a lot.

Probably a duplicate of https://github.com/Maatwebsite/Laravel-Excel/issues/898 but I think it should be made more obvious that a sheet needs to be added because I don't see any mention of that in the docs.


TypeError: BaseModel.validate() takes 2 positional arguments but 3 were given

An error is received when executing flow in Prefect 2.13.7...2.13.8:
from prefect_airbyte.server import AirbyteServer
from prefect_airbyte.connections import AirbyteConnection
from prefect_airbyte.flows import run_connection_sync
server = AirbyteServer(server_host="<IP_ADDRESS>", server_port=8000)
connection = AirbyteConnection(
airbyte_server=server,
connection_id="b35180dc-6bd1-48b0-b0cd-b99deb6caaaa",
status_updates=True
)
@flow
def extract_data_subflow():
run_connection_sync(airbyte_connection=connection)
pydantic_self.pydantic_validator.validate_python(data, self_instance=pydantic_self)
TypeError: BaseModel.validate() takes 2 positional arguments but 3 were given

Hi @radiatorius1024, which version of prefect-airbyte are you running (pip freeze | grep prefect-airbyte)?
With prefect==2.13.7, we've removed the version constraint on pydantic, which allows pydantic>=2 to be installed, but that may be incompatible with older versions of prefect-airbyte.  In prefect-airbyte==0.3.0, we've made the import adjustments to allow you to have either version of pydantic installed.  If you aren't on 0.3.0, can you bump your version and let us know if that works?

Hi. Yes I have heard about the changes so the latest version is used:
pip freeze | grep prefect-airbyte
prefect-airbyte==0.3.0

Thanks @radiatorius1024, I should have also asked: what version of pydantic do you have installed as well?

Here are all installed libraries (Prefect installed on Docker, but same error with local installation):
pip freeze
aiobotocore==2.7.0
aiohttp==3.8.6
aioitertools==0.11.0
aiosignal==1.3.1
aiosqlite==0.19.0
alembic==1.12.0
annotated-types==0.6.0
anyio==3.7.1
apprise==1.6.0
asgi-lifespan==2.1.0
async-timeout==4.0.3
asyncpg==0.28.0
attrs==23.1.0
botocore==1.31.64
cachetools==5.3.1
certifi==2023.7.22
cffi==1.16.0
charset-normalizer==3.3.0
click==8.1.7
cloudpickle==3.0.0
colorama==0.4.6
coolname==2.2.0
croniter==1.4.1
cryptography==41.0.4
dask==2023.10.0
dateparser==1.1.8
distributed==2023.10.0
docker==6.1.3
frozenlist==1.4.0
fsspec==2023.10.0
google-auth==2.23.3
graphviz==0.20.1
greenlet==3.0.0
griffe==0.36.7
h11==0.14.0
h2==4.1.0
hpack==4.0.0
httpcore==0.18.0
httpx==0.25.0
hyperframe==6.0.1
idna==3.4
importlib-metadata==6.8.0
Jinja2==3.1.2
jmespath==1.0.1
jsonpatch==1.33
jsonpointer==2.4
jsonschema==4.19.1
jsonschema-specifications==2023.7.1
kubernetes==28.1.0
locket==1.0.0
Mako==1.2.4
Markdown==3.5
markdown-it-py==3.0.0
MarkupSafe==2.1.3
mdurl==0.1.2
msgpack==1.0.7
multidict==6.0.4
numpy==1.26.1
oauthlib==3.2.2
orjson==3.9.9
packaging==23.2
pandas==2.1.1
partd==1.4.1
pathspec==0.11.2
pendulum==2.1.2
prefect @ file:///opt/prefect/dist/prefect.tar.gz
prefect-airbyte==0.3.0
prefect-dask==0.2.5
psutil==5.9.6
pyasn1==0.5.0
pyasn1-modules==0.3.0
pycparser==2.21
pydantic==2.4.2
pydantic_core==2.10.1
Pygments==2.16.1
pyodbc==5.0.1
python-dateutil==2.8.2
python-slugify==8.0.1
pytz==2023.3.post1
pytzdata==2020.1
PyYAML==6.0.1
readchar==4.0.5
referencing==0.30.2
regex==2023.10.3
requests==2.31.0
requests-oauthlib==1.3.1
rich==13.6.0
rpds-py==0.10.6
rsa==4.9
ruamel.yaml==0.17.39
ruamel.yaml.clib==0.2.8
s3fs==2023.10.0
sentry-sdk==1.32.0
six==1.16.0
sniffio==1.3.0
sortedcontainers==2.4.0
SQLAlchemy==2.0.22
starlette==0.27.0
tblib==2.0.0
text-unidecode==1.3
toml==0.10.2
toolz==0.12.0
tornado==6.3.3
typer==0.9.0
typing_extensions==4.8.0
tzdata==2023.3
tzlocal==5.1
urllib3==1.26.18
uvicorn==0.23.2
websocket-client==1.6.4
websockets==11.0.3
wrapt==1.15.0
yarl==1.9.2
zict==3.0.0
zipp==3.17.0

Thank you!  We'll dig into this

Hi @radiatorius1024, thanks for the bug report, we believe we have corrected this in the latest prefect release (2.14.1): https://github.com/PrefectHQ/prefect/releases/tag/2.14.1
Can you let us know if that corrected things for you?

Hi @radiatorius1024, thanks for the bug report, we believe we have corrected this in the latest prefect release (2.14.1): https://github.com/PrefectHQ/prefect/releases/tag/2.14.1
Can you let us know if that corrected things for you?

Everything works great now, thanks.

Hey @chrisguidry I have been stuck on this same error for a while now.
I am using a docker work-pool to run the flow (and subflow) that's stored locally. I was initially using Prefect Server and Connection blocks to maintain the Airbyte Connection information, but have since adjusted my script to match radiatorius', but I'm still getting the same error.
Could it be the work-pool?
Here are my specs:

Ubuntu 22.04.03 LTS
prefect==2.14.1
prefect-airbyte==0.3.2
pydantic==2.5.2
docker==6.1.3
docker-compose==1.29.2
prefect-airbyte==0.3.2
prefect-docker==0.4.1

Sorry about that, @jloehn!  Can you paste the full exception you're seeing with the stack trace so we can see where it's happening for you?  It may be the same error but manifesting from a different place.

@chrisguidry Thanks for pointing me in the right direction. I kept overlooking the push section of my prefect.yaml.

Nice!  Glad to hear it's working!


Narwhalify NearestMeanResponseImputer

What
Make NearestMeanResponseImputer polars compatible while maintaining backwards compatibility with pandas.
Why?
Making the project polars compatible for speed benefits. Narwhals allows us to do this piece by piece and maintain compatibility between components as it makes it possible to pass pandas dataframes to polars logic. This involves converting between pandas/polars dataframes.
How?
convert logic in NearestMeanResponseImputer to function with polars dataframes and add narwhalify decorators to methods to ensure compatibility with pandas dataframes.
Update testing where required.

completed here https://github.com/lvgig/tubular/pull/342


Enable snapshot image creation for PUSH and TIMER [pike]

To ensure that the snapshot image creation happens more regularly,
we ensure that the preparation for it happens for both a PUSH and
a TIMER trigger.
This will ensure that the images are updated on a daily basis,
not just when a new patch merges.
(cherry picked from commit e11f62aa8b2ee940619306378ddec935e58d9af8)
Issue: RE-2278

Thank you for submitting a pull request to this project!
Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.
When the gate jobs have completed successfully, this pull request will get
merged automatically.


Have you compared your transfuser with late fusion?

We have reported the quantitative experiments in the main paper.


⚠ Arweave has degraded performance

In 43057cd, Arweave (https://y2dkomn4abqiffiuvhrfhuqynbu4det55c3uipi4gyvvntorwfoa.arweave.net/xoanMbwAYIKVFKniU9IYaGnBkn3ot0Q9HDYrVs3RsVw/flags/mod.ts) experienced degraded performance:

HTTP code: 200
Response time: 3124 ms

Resolved: Arweave performance has improved in dac34bb.


[plugin_platform_interface] Switch PlatformInterface._instanceToken to an expando.

This change replaces PlatformInterface._instanceToken (an instance field that points from a PlatformInterface to its corresponding token) with an expando that maps from PlatformInterface to Object. There is no change to the public API, and no change to the behavior of users' production code.
This change ensures that if a customer tries to implement PlatformInterface using implements rather than extends, the code in PlatformInterface._verify won't try to access the private _instanceToken field in PlatformInterface.  This is important because an upcoming change to the dart language is going to cause such accesses to throw exceptions rather than deferring to noSuchMethod (see https://github.com/dart-lang/language/issues/2020 for details).
Fixes https://github.com/flutter/flutter/issues/109339.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@stuartmorgan, given that this change only affects the behavior of some corner cases in tests, I'm unsure whether I need:

a CHANGELOG entry
a version bump
tests

What do you think?

@stuartmorgan, given that this change only affects the behavior of some corner cases in tests, I'm unsure whether I need:

a CHANGELOG entry
a version bump

It's changing production code (and will actually change behavior for clients), so needs both.

tests

Since we already have tests that would break later without this change, I think it's covered; I would ping Hixie for an exemption on this.

Do we know of anyone who uses implements? Can't we just have them switch to extends instead? Expando logic is hard to understand and keeping the code simple seems very preferable unless the cost of migrating people is disproportionally large.

Ah, that was not clear from the original commit message or changelog entry. I definitely support the goal here. I just wish there was a cleaner (read, less Expando-magical way) to do it...

I think this is worth testing, FWIW. Verifying that we give good clear error messages for things like this is valuable.

I think this is worth testing, FWIW. Verifying that we give good clear error messages for things like this is valuable.

Test added!


[BUG] Internal Server Error when using "Generate CSV" in reporting

Describe the bug
Opensearch returns 500, Internal Server Error when using "Generate CSV" from the reporting feature.
It seems to occur only when using index patterns with multiple indices serapated by a comma (like myindice_,myotherindice_)
To Reproduce
Steps to reproduce the behavior:

Go to discover
Select index pattern with multiple indices separated by commas
Save the search
Click generate CSV
Error (in console and dashboard logs): Failed to generate report: TypeError: Cannot create property '0' on number '1701418310497'

Expected behavior
No error
OpenSearch Version
2.11
Dashboards Version
2.11
Plugins
Please list all plugins currently enabled.
Screenshots

Host/Environment (please complete the following information):
Kubernetes with official helm chart
Chrome 119

Unable to reproduce this issue with sample data index pattern opensearch_dashboards_sample_data_flights,opensearch_dashboards_sample_data_logs.
Can you please share detailed steps/sample data to reproduce this issue?

Is this the same issue, and thus fixed in 2.13.0?
https://github.com/opensearch-project/reporting/issues/932


Sensors do not have a value

Thanks to a Contributor here i was able to eliminate the layer 8 problem, and install and configure it.
Now i can see al the entities of my OPNsense, but they all dont have a value(logs below).
For Example the sensor.oahome_filesystem_used_percentage_slash is unavailable, and appears to be restored.

Logs
https://pastebin.com/z6FXsHJ2

Please install the latest code. It should fix this issue.

It works now thank you


Handle all m.room.aliases chunk, not only first

This is fix for https://github.com/matrix-org/matrix-python-sdk/issues/106 issue
Rooom state server reply may contain more then one m.room.aliases update_aliases() method must collect aliases from all of them.
Additionally in this code only get_room_state() can raise MatrixRequestError - there is no need to enclose to try block more strings.
Signed-off-by: Pavel Kardash<EMAIL_ADDRESS>

Is there a reason why Room.aliases is not a set? It seems unlikely that we need ordering (as it is random anyway since it comes from a dict) or indexing, while this PR adds membership tests that could benefit a set, especially in _process_state_event. The only point I can see is to avoid confusing a user, since it is easy to assume from the look of it that an attribute called aliases is a list. Thoughts anyone?


Bug: java.lang.ClassCastException for MatchAllQuery in ReplaceRewriter

The ReplaceRewriter does not check if the given ExpandedQuery is an instance of Query. The subsequent cast therefore leads to a ClassCastException if ExpandedQuery is an instance of MatchAllQuery.

Created PR

Fixed by #84


Fix resize on another screen, closes #14, closes #10

It only fixes the resize problem, but actually I see some other issues.

I can't record video.
Will try show on screenshots.
First situation -- 2 screens, same resolution (1920x1080). Scale on first is 100%, on second is 150%.
Can't press minimize, maximize, close buttons:

Buttons aren't clickable.
First situation -- 2 screens, same resolution (1920x1080). Scale on first is 100%, on second is 125%.
Buttons are clickable, but I can't drag window with titlebar anymore:

I see some issues now:
1.
When I drag window on second screen with 150% or 175% scale, I can't drag titlebar on second screen.
But everything works with 100% and 125% scale.

When window is splitted by 2 screens part on screen with 150% works fine but part on screen with 100% resizeable on all frame (will take a look if it can be fixed):

Known bug after drag window on different screen:

Any fixes/workarounds?

I added commit to fix issue 3 in my list.
This commit also probably closes #10 and #7.
c723c1c

I rewrite code using map from global in HitTest, it works really fine now!
But there are another problems:

Part of window on another screen while maximized:

If I minimize window on second screen from maximized state, graphics hang after I try to show it:

I rewrite code using map from global in HitTest, it works really fine now!
But there are another problems:

Part of window on another screen while maximized:

If I minimize window on second screen from maximized state, graphics hang after I try to show it:

With latest commit I fixed all issues that I know so far. Now it's pretty usable (atleast for my applications).
I made it a while ago but decided to commit changes now. Some changes I don't really remember now (such as g with scale or WM_MOVE case in nativeEventFilter) but it's works fine for me on any setup I tested.
Feel free to contact me if you see any issues.


Create UI for Settings editing

Closed with https://github.com/into-cps/intocps-ui/commit/28fb3d1e4c2e87aa5e893a9c0306e451b2bf925d


Unable to download the dataset.

Is the server hosting the dataset down ? I am not able to download it "ERROR 503: Service Unavailable"

The Bay Area is currently going through some planned power outages, in an effort to decrease risk of wildfires. I hope it will be back up soon.
Thanks!


implement interactive list

In this milestone i managed to implement the following :

Add a new JavaScript file and import it as a module

it will contain methods related to the status updates (completed: true / false)

Add event listener to the checkbox (change)
Update items object's value for completed key upon user actions.
Implement a function for the "Clear all completed" button (use filter() method).
Store the updated array of items in local storage, so the user gets the correct list values after the page reloads.

thanks @ahmetbozaci for the suggestion and links to the resource ...I appreciate 💯


Safari issue in RWD beta - learn more about CSS Pseudo Selectors

Affected page

https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-more-about-css-pseudo-selectors-by-building-a-balance-sheet/step-2
Your code

    Last Updated: December 2021

Expected behavior

The above code does not pass when submitted in Safari. I checked the same step using Firefox and it passed ok.
Screenshots

System

macOS 12.1
Safari 15.2
Additional context

I couldn't reproduce this on Safari 13.1.2. The test that is failing for you, @marks1974, is checking for a single div element: assert(document.querySelectorAll('div')?.length === 1);. Seems like a strange one to fail. There must be quite a few tests like that throughout that would be causing you problems. A quick search shows as many as 127:

Maybe you just haven't come across any of the others I guess. Another one is step 12 of the magazine project
Want to give both of these (my link and yours) another try and make sure it wasn't a fluke @marks1974?

I just tried step 2 again and this time it passed. I honestly thought it was a bit odd that it did not initially pass as shown by my screen shots.
The funny thing is that earlier in the week I was doing a different part of the beta and initially one of the steps (I can't remember which) would not pass for me in Safari but was fine in Firefox. I started to write a new issue on Github and was about to hit submit but before doing so I decided to test it again. After trying about half a dozen times, it finally worked. As it was being a bit hit and miss in Safari I switched to using Firefox which has been mostly bug free. Having said that I had a problem earlier today in Firefox when I was trying to submit some of the steps in the penguin project which were asking for linear-gradients (I think it was steps 8 and 14). They would not pass in Firefox but oddly enough they passed in Safari.

Thank you, for testing, and re-clarifying.
I am going to close this, for now. If we can determine an issue, then please ask, and we can reopen this.


Dropdown List for cell data options

Hi,
Is it possible to restrict a cell's data input to be a list of options? Something like this.

Those are called "data validations".
We offer this in the Pro compendium.  Since companies have paid for the features already, it would be unfair to them if we turned around and made it available as open source.  We have a longer comment in a gist.


Duplicate of mean_max() algorithm

There are currently two versions:
algorithms.main.mean_max_power() - relies on rolling mean calculation
algorithms.metrics.power_duration_curve() - relies on calculating diff of accumulated energy and is x4 faster
Proposal:

Rename algorithm to mean_max() because it can be applied not only to power but to pace as well
Keep faster implementation with data type casting resolved according to the outcome of #5
Move it to main, which will serve as a module for the general purpose calculations or according to the outcome of #7

Agree. Then this method would live in metrics.core as well?
Totally agree :)
Agree. Although I don't like the name "main" I gave it. Do you have a better idea? And do we even want to keep main when we're moving to separate heartrate/power/speed modules?

The mean_max() resides in the core and in the metrics.power we would even have a liberty to call a power_duration_curve().
I propose to transfer the content of main to the core or to the specific metrics module depending on what it does.

👍

Shall I pick up this issue and implement the project structure agreed in #7 ?

Yes if you would like to :)


Can't resolve LESS dependencies

So this is pretty simple case, my app starts with
entryPoints: ['./src/index.tsx', './src/index.less'],
and I get the error error: [less-loader] './utils/components/index.less' wasn't found. Tried - ./utils/components/index.less,utils/components/index.less
In index.less I have dependencies to other less files.
Should this plugin resolve less files tree and dependencies?

Thanks for this issue! This bug has been fixed. You can upgrade the package to latest version.

@iam-medvedev , wow, that was quick. It works! Thanks a lot.
Just want to mention 1 detail, but probably it is correct because of less itself. When I import .css in less I need to do:
@import (less) '../node_modules/react-calendar/dist/Calendar.css'; (less casting)
otherwise i get error
error: Could not resolve "../node_modules/react-calendar/dist/Calendar.css" (the plugin "less-loader" didn't set a resolve directory)
Anyways, thanks again :)

@Lazarencjusz 🤔 I will try this case tomorrow

@Lazarencjusz You can try import css files with inline directive
For example:
@import (inline) "../node_modules/react-calendar/dist/Calendar.css";

@Lazarencjusz I have fixed this bug in the latest release. Can you check if the error is still there?

@iam-medvedev  I can conform 0.0.7 allows directly to import css files like @import './style.css';
Thanks :)

@iam-medvedev could I ask one more thing?
When I set 2 entry points to esbuild:
entryPoints: ['./src/index.tsx', './src/index.less'],
and I have the blugin set with filename:
plugins: [lessLoader({ filename: cssFileName })],
the esbuild require from me to set outdir instead of outfile, so after all, css file is always created with default name index.css.
However, I can split builds to 2 separate/independent and put those builds in chain, then the css file is created basing on outfile property. I assume this is correct flow in my case when I want to build "at once" tsx and less?

esbuild offers both options, you can add less as an entrypoint, or as an import in your code. In the second case, you don't need any additional settings.


Upgrade to Bootstrap 4

Updated dependencies version
Upgrade bootstrap from version 3.3.7 to 4.1.2
Adequate the views to the bootstrap changes

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: amodolo sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@amodolo
Can you remove all commits/changes not related to Bootstrap bump? (pckg deps updates, lock file udpates should not be in the PR, the markup rewrite, assets update are welcome). So only related changes are left in the PR
Thanks!

@amodolo
Sahat (@sahat) from the upstream project used in this one has just updated the content to the BS4. Can you compared details and make some minor changes? That would allow to ship the same L&F as in upstream project. Here is the current version link:
https://hackathon-starter-2018.herokuapp.com/
I've setup your PR locally, so I could help with stuff.

And here are some remarks after updating to BS4:
https://github.com/sahat/hackathon-starter/issues/853

When can we get this conflict solved and merged into master?

Sorry for the delay. I've just removed the unrelated commits

I've sync the fork with the origin but i'm still having some problems with the CI. It's seams the same issue as #154

Alright, I've got this PR built and running locally and tested it out. Looks 👍 - I've clicked around and verified that all the pieces look right and feel good. Thanks for taking the time @amodolo and @peterblazejewicz


New version: Google.Chrome.Dev.EXE version 131.0.6753.0

Created by 🥟 Dumplings in workflow run #8562.

Log
Updated: 131.0.6738.0 → 131.0.6753.0
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

Possible duplicate package entry.
Similar installer SHA256 hash found in manifest:

InstallerSha256: 6b572c679bdf3cecc6574eccd72e6dce6c8fd64a37ff5c2a34a890fcfb112ddd

manifests\g\Google\Chrome\Canary\131.0.6753.0

InstallerSha256: e425bea74863211ec4fe2e5743a9d8523f98b226f051fd7aa756f74f80db5d8a

manifests\g\Google\Chrome\Canary\131.0.6753.0

InstallerSha256:<PHONE_NUMBER>26e4e827b408ef3be09dc285ab8780c7cf01d59b6226895c471245

manifests\g\Google\Chrome\Canary\131.0.6753.0

It appears that this Canary version has been promoted to Dev.


Heatmap tooltip

This pull request adds the tooltip text as a child of CellTooltip1DVertical, which means that the heatmap now displays the tooltip when hovering. It also cleans up CellTooltipText a little by passing the parent's width and height as props rather than needing to query for parent DOM nodes.

I have just made a small fix that prevents text wrapping in the tooltips (in the heatmap and also anywhere else the tooltips are used).


Users/abmodi/master

Please refer line 236, 298, 343, those are the only changes.

Signed off

merging on behalf of Abhinit modi


Peg parsing

DO NOT MERGE

Pull Request Test Coverage Report for Build cf19b2fa971f53fa15ca59f99b092908071ce1c2-PR-64

119 of 130 (91.54%)  changed or added relevant lines in 14 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.7%) to 91.551%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/earmark_parser/helpers/parser/state.ex
0
2
0.0%

lib/earmark_parser/helpers/alt_link_parser.ex
0
4
0.0%

lib/earmark_parser/helpers/html_helpers.ex
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/earmark_parser/helpers/html_helpers.ex
1
0%

Totals

Change from base Build d1cdeb1fbd3b7bdb2f1a9e69150174844856a7f8:
0.7%

Covered Lines:
791

Relevant Lines:
864

💛  - Coveralls

Pull Request Test Coverage Report for Build cf19b2fa971f53fa15ca59f99b092908071ce1c2-PR-64

119 of 130 (91.54%)  changed or added relevant lines in 14 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.7%) to 91.551%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/earmark_parser/helpers/parser/state.ex
0
2
0.0%

lib/earmark_parser/helpers/alt_link_parser.ex
0
4
0.0%

lib/earmark_parser/helpers/html_helpers.ex
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/earmark_parser/helpers/html_helpers.ex
1
0%

Totals

Change from base Build d1cdeb1fbd3b7bdb2f1a9e69150174844856a7f8:
0.7%

Covered Lines:
791

Relevant Lines:
864

💛  - Coveralls


Link to migration guide in v6 release points to missing page

Link in v6.0.0 release notes for migrating from v5 to v6 links to "next" documentation at
https://johnnycrazy.github.io/SpotifyAPI-NET/docs/next/5_to_6 (which does not exist)
instead of
https://johnnycrazy.github.io/SpotifyAPI-NET/docs/5_to_6

Thank you, fixed!