Pandas and time zone (tz) info.

Hi there.
The current version of pandas throws an error when comparing trading_days (with tz=UTC) to a standard python datetime with no tz info. Basically, it wants the time zone info to be consistent. I've added a few minor changes to make this work.
It also looks like the data coming back for the test file is slightly different. I've modified expected return values slightly, so have a quick look and let me know what you think. The tests should now be passing.
Pandas is also grumbling a touch about Panel => FutureWarning: Panel is deprecated and will be removed in a future version. Perhaps it'd be better to specify a particular pandas version in requirements.txt rather than pandas>=0.17.0... always tough with the Pandas deprecations.
Cheers,
Simon.

@simongarisch happy to merge once tests past. Looks like the test is still failing right now. Thanks for contributing!


Doesn't return proper error response for wrong keys

Thanks to amirhosseinka for reporting.

Fixed.
$ curl -X POST 'https://api.livecoinwatch.com/overview' -H 'content-type: application/json' -H 'x-api-key: 51c700ba-905b-400d-9e30-458714c2d8b' | jq .
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   236  100   236    0     0    405      0 --:--:-- --:--:-- --:--:--   407
{
  "error": {
    "code": 401,
    "status": "Unauthorized",
    "description": "The requester is not authorized to access the resource. This is similar to 403 but is used in cases where authentication is expected but has failed or has not been provided."
  }
}


Add ic-refresh

I'm currently serving a server generated image from my webapp.
To refresh this image at regular intervals I am setting ic-poll on the parent div, and then returning the complete <img src='/XXX.jpg'> text. This then subsequently causes a request to /XXX.jpg where I can return the newly updated generated image.
This seems a little roundabout. I was wondering if adding something like ic-attr-refresh or ic-src-refresh to intercooler.js made sense. ic-src-refresh for example, would replace or append the src attribute the normal "?timestamp=" + new Date().getTime() trick to entice a refresh.
Hope this makes sense and conceptually is compatible with intercooler's view of the world.

This makes good sense, but I can't come up with a good syntax for it that doesn't get close to what the jQuery would look like.  You need to cache bust to make the request work and that makes the syntax difficult.
I like the idea, just don't see how to implement it cleanly yet.

django like queries? ic-poll__src, ic-poll__data-values
this will replace attribute src with ic-get results and will replace attribute data-values from ic-get results


Git SCM: toggle assume-unchanged and skip-worktree bits

I would like to be able to flip the assume-unchanged and skip-worktree bits for files in the git index.
This can be done on the command line with commands like:

enable assume-unchanged bit
git update-index --assume-unchanged path/to/file
disable assume-unchanged bit
git update-index --no-assume-unchanged path/to/file
enable skip-worktree bit
git update-index --skip-worktree path/to/file
disable skip-worktree bit
git update-index --no-skip-worktree path/to/file

In the GUI, I would add files with these bits enabled to collapsible sections like, but distinct from, the Changes and Staged Changes sections in the Source Control view. Disabling the bits could be done with a - button in the related section while enabling the bits may be done through context menus in File Explorer view and Source Control view's Changes section.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

I have git aliases set up to do this, so it would be great to have in the UI.  It's useful, for example, in cases where a file needs to have credentials or alternate configuration in it for testing, but you want to be sure not to accidentally commit those changes to the repository.

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Update line protocol to require trailing i for field values that are integers

Many people have reported problems with writing data in and confusion around when a number is a float64 or an int64. Because we take the type of the first written value as that field's type for the measurement, it's causing errors for people later on.
I briefly thought of dropping support for ints (#3479) to make things easier, but it looks like enough people really want them still. So, this will solve the usability problem and give people the ability to still have integers.
After this change in the line protocol, any number in a field will be parsed as a float64, unless that number has a trailing i. Examples:
# here are some float64 values
cpu,host=A value=0
cpu,host=A value=23
cpu,host=A value=23.2

# here are some int64 values
cpu,host=A value=0i
cpu,host=A value=23i

# here is an invalid write that throws an error
cpu,host=A value=23.2i

This will be a breaking change between 0.9.2 and 0.9.3. Hopefully it is small enough that client libraries can be updated with little effort. We had no plan to make breaking changes in the 0.9 line, but this seems the best course of action and it's best to get it out of the way as soon as possible. It's also a very small change.
Apologies for the breakage, but it's our best bet to fix usability and keep those who want integers happy.

Completed in https://github.com/influxdb/influxdb/pull/3526

In the future (or next release) having an intermediate version which supports both versions would be helpful. e.g. the i is optional, and numbers parsed as usual (unless i present)
Then in the next version enforce non-decimal as floats.
Reason being that we can't upgrade without simultaneous client + db update which increases downtime

Is there any way to convert exiting data from int to floats?

@jgeiger sadly not currently without re-writing all the data.  @pauldix any thoughts on a utility to do this in the future?

@corylanou it would be pretty tricky to create a utility for this, particularly now that we have multiple storage engines that can exist. Not something that can't be done, but we're unlikely to get to it in the next 2 months.

What about automatically casting numeric values to integers for series that previously held integers? That is to say, make this:
cpu,host=A value=23

Equivalent to this:
cpu,host=A value=23i

If and only if the measurement type is an integer. By doing this 1) you would ensure a smooth non-breaking transition for 0.9.2 users while 2) enforcing float64 by default for new measurements unless values are provided with the 'i' trailing character.
The reason I suggest this is that in our case we tested 0.9.3 today and decided to stay with 0.9.2 for the time being, because figuring out how to properly upgrade our queries in such a way that it works with our existing data is a pain; we can't know the type of an existing measurement in all cases to properly adjust the query since this happened behind the scenes previously.

I agree with @jportoles even if the change is only in effect for 0.9.3 and removed in 0.9.4 for smooth upgrade purposes


Show or hide

Hello,
when I use show-if-fullscreen directive, it only works as expected for the first time. Second time the icons don't change properly (or at all).
Here is my code:
<md-button ngsf-toggle-fullscreen show-if-fullscreen-enabled hide-xs class="md-icon-button"> <md-icon show-if-fullscreen=false md-svg-icon="fullscreen"></md-icon> <md-icon show-if-fullscreen=true md-svg-icon="fullscreen_exit"></md-icon> </md-button>

Can you create a plunkr with the code you are trying? Or tell me if mine works.
Here is the example I use, http://plnkr.co/edit/akbNXmHGvPqHydqXot1w?p=preview
In order for it to work you have to click on the "Launch the preview in a separated window" icon on the top right of the preview as the fullscreen doesn't work on iframes

Closed for now, if issue arises again reopen with the code you are trying


i18n(fr): update reference/dev-toolbar-app-reference.mdx

Description (required)
Adds changes from #9240 to the French translation of reference/dev-toolbar-app-reference.mdx.

Related issues & labels (optional)

Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

fr/reference/dev-toolbar-app-reference.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


ISPN-13899 Error excecuting Batch in JdbcStringBasedStore with Oracle DB

https://issues.redhat.com/browse/ISPN-13899

I didn't see the "on hold", but I assumed it was ok

I only put on hold until you cleared it @tristantarrant :)


Promise never response after send for multi-tokens

When I tried to send about 100+ tokens,
I called
.send(note, tokens).then((result) => {console.log(result);})
and the result never print, and no error at all.
Is that too many tokens for one send? what is the best way to send multi-tokens?
(When sending around 10 tokens a time everything just fine ... so I guess my setting is correct.)
Thanks

Seems because I stored multi-tokens for a single user for some reason. And when I tried to send one notification to multi-tokens under same apple user/device ... and something went wrong.
Because when I only use the latest token of a single user, it works now.
Is that possible some kind of exception handling is missing?


Minor bug in New-Directory and pester test for it

Not that I found any errors in Lability itself, but noticed the parameters took array input, and that the function could not handle that on the Path parameter.

Good catch regardless. Thanks!


hillaEngine: Cannot build Docker image

Description of the bug
I have tried several pom.xml modification, but cannot make this work. With the com.vaadin.experimental.hillaEngine=false the Docker image is build. Tried on Windows and a fresh Ubuntu with docker
Minimal reproducible example
https://github.com/sveine/vaadin-small-business-starter/blob/main/small-business-starter-project/README.md
Actual behavior
[INFO] Unable to determine version information. No vaadin-core-versions.json found
[INFO] Running `npm install` to resolve and optionally download frontend dependencies. This may take a moment, please stand by...
[INFO] using '/usr/bin/npm --no-update-notifier --no-audit --scripts-prepend-node-path=true --ignore-scripts install' for frontend package installation
[INFO] Frontend dependencies resolved successfully.
[INFO] Scanning for projects...
[INFO]
[INFO] ----------< com.smallbusiness:small-business-starter-vaadin >-----------
[INFO] Building small-business-starter-vaadin 1.0-SNAPSHOT
[INFO] --------------------------------[ jar ]---------------------------------
[WARNING] The POM for com.smallbusiness:core-module:jar:1.0-SNAPSHOT is missing, no dependency information available
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  0.570 s
[INFO] Finished at: 2022-10-12T08:35:42Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project small-business-starter-vaadin: Could not resolve dependencies for project com.smallbusiness:small-business-starter-vaadin:jar:1.0-SNAPSHOT: Could not find artifact com.smallbusiness:core-module:jar:1.0-SNAPSHOT -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for Vaadin Small Business Starter 1.0-SNAPSHOT:
[INFO]
[INFO] super-pom .......................................... SUCCESS [  0.363 s]
[INFO] core-module ........................................ SUCCESS [  1.468 s]
[INFO] small-business-starter-vaadin ...................... FAILURE [ 18.865 s]
[INFO] Vaadin Small Business Starter ...................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  20.854 s
[INFO] Finished at: 2022-10-12T08:35:42Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.vaadin:vaadin-maven-plugin:23.2.4:build-frontend (default) on project small-business-starter-vaadin: Could not execute build-frontend goal: Hilla Generator execution failed with exit code 1 -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn <args> -rf :small-business-starter-vaadin
Error response from daemon: The command '/bin/sh -c mvn -f small-business-starter-project/pom.xml clean package -DskipTests -Pproduction' returned a non-zero code: 1
Failed to deploy 'small-business-starter Dockerfile: Dockerfile': Can't retrieve image ID from build stream

Versions:
Hilla: 23.2.4
Flow: 23.2.4
Java: JetBrains s.r.o. 17.0.2
OS: amd64 Windows 10 10.0
Browser: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Live reload: Java active (HotswapAgent): Front end active

As always: The solution was simple! Just had to learn docker first to debug. https://github.com/sveine/vaadin-small-business-starter/commit/a09c774b8efc9714b294dd21c536c04b6a4a7006


Support t for declension

Suppose now is 15 june, russian translate.
$date = new Date('-1 month');
$date->format('d F') //15 ма(я) => good declension
$date->format('t F') //31 ма(й) => no declension (must be 31 мая) (t - last day of month)

I'm not sure it's correct, when some would consider "t" is the last day number of the month, other may consider it's the number of days in the month, and so, used this way, it does not necessarily mean you want a day-month declension when you use t in format.
A more semantic way to do it would be:
$date->endOfMonth()->format('d F'); // 31 мая


Docker bug fixes

Bug fixes for the Docker driver
The docker driver was initially tested on v1.21 (1.9.x), Docker API supports versioning in the URI path now. I've updated the tests with 2 sets of fixtures, the 1.21 and the 1.24 (latest) API.
For more information on contributing, please see Contributing
section of our documentation.
Status
Replace this: describe the PR status. Examples:

work in progress
done, ready for review

Checklist (tick everything that applies)

[ ] Code linting (required, can be done after the PR checks)
[ ] Documentation
[ ] Tests
[ ] ICLA (required for bigger changes)

Nice :+1:
And yeah, hopefully including version in the URL will allow us to keep the driver a bit more sane and not break when new docker (API) version is released.
Down the road, we also need decide if we should (does it make sense, maybe it's too much overhead, etc.) to support multiple API versions. Maybe we should only do it for major versions or similar.


Roll Packages from 8028cafa1f0c to 26889ce846ad (3 revisions)

https://github.com/flutter/packages/compare/8028cafa1f0c...26889ce846ad
2023-07-25<EMAIL_ADDRESS>[webview_flutter_android][webview_flutter_wkwebview] Fixes bug where PlatformWebViewWidget doesn't rebuild when the controller changes (flutter/packages#4533)
2023-07-24<EMAIL_ADDRESS>[ci] Enable Android emulator-based tests (flutter/packages#4494)
2023-07-24<EMAIL_ADDRESS>[google_maps_flutter_web] Options to disable tilt controls and configure gesture handling (flutter/packages#3258)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Update GitHub links

Updated the links to the Backstage community plugins.

https://app.shortcut.com/larder/story/20497/update-marketing-site-docs

Merging to fix marketing site bugs


Memory Leak When Loading Scenes

Bevy version
bevy = "0.13.0"
[Optional] Relevant system information

cargo 1.78.0-nightly (7065f0ef4 2024-03-12)
MacOS 14.4 (23E214)
Apple M1 CPU
AdapterInfo { name: "Apple M1", vendor: 0, device: 0, device_type: IntegratedGpu, driver: "", driver_info: "", backend: Metal }
Also tested on Wasm in Firefox with the same result.

What you did
I have been having a problem with a memory leak in my app. Here is a minimal example that reproduces the bug.

use bevy::prelude::*;

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .insert_resource(Objects::default())
        .add_systems(Startup, setup)
        .add_systems(Update, update)
        .run();
}

#[derive(Resource, Default)]
struct Objects {
    mesh: Handle<Mesh>,
    material: Handle<StandardMaterial>,
    scene: Handle<Scene>,
}

#[derive(Component)]
struct Piece;

fn setup(
    mut commands: Commands,
    mut objects: ResMut<Objects>,
    mut meshes: ResMut<Assets<Mesh>>,
    mut materials: ResMut<Assets<StandardMaterial>>,
    asset_server: Res<AssetServer>,
) {
    commands.spawn(Camera3dBundle {
        transform: Transform::from_xyz(7., 15., -9.5).looking_at(Vec3::ZERO, Vec3::Y),
        camera: Camera {
            clear_color: ClearColorConfig::Custom(Color::rgb(0., 0.7, 0.89)),
            ..default()
        },
        ..default()
    });

    objects.mesh = meshes.add(Cuboid::new(1.0, 1.0, 1.0));
    objects.material = materials.add(Color::rgb_u8(124, 144, 255));
    objects.scene = asset_server.load(format!("tree_1.glb#Scene0"));
}

fn update(mut commands: Commands, objects: Res<Objects>, delete_query: Query<Entity, With<Piece>>) {
    for entity in delete_query.iter() {
        commands.entity(entity).despawn_recursive();
    }

    for i in 1..1000 {
        commands
            .spawn(PbrBundle {
                mesh: objects.mesh.clone(),
                material: objects.material.clone(),
                transform: Transform::from_xyz(0.0, 0.0, i as f32),
                ..default()
            })
            .insert(Piece);

        // The previous expression doesn't seem to have an effect
        // but the memory steadily increases when the following expression is included.
        commands
            .spawn(SceneBundle {
                scene: objects.scene.clone(),
                transform: Transform::from_xyz(0.0, 1., i as f32).with_scale(Vec3::splat(0.05)),
                ..Default::default()
            })
            .insert(Piece);
    }
}

In other words, entities are cleared and then added again, repeatedly. The problem still happens if there is some time between runs (e.g. once every second) but it just happens a bit slower.
What went wrong
The app memory usage steadily increases.
Additional information
It may be that some part of the loaded scene is not being properly deleted. I am not sure.
Potentially related to #9035 but it seems to not be an exclusively MacOS issue, given that I tried it on Wasm as well.

Does the issue still occur if you're running headless without a renderer? The other mentioned issue seems to be related to rendering on MacOS.

I encountered the same issue on Windows 11 with bevy version "0.13.1", which seems to be related to the SceneSpawner resource. I attempted to estimate the memory usage of SceneSpawner and found that its memory consumption continuously increased. When the program's memory usage reached 500MB, SceneSpawner accounted for more than 220MB of it. The following code can be used to reproduce this issue:
use bevy::prelude::*;

fn main() {
    App::new()
        .add_plugins(DefaultPlugins)
        .insert_resource(Objects::default())
        .add_systems(Startup, setup)
        .add_systems(Update, update)
        .run();
}

#[derive(Resource, Default)]
struct Objects {
    scene: Handle<Scene>,
}

#[derive(Component)]
struct Piece;

fn setup(mut commands: Commands, mut objects: ResMut<Objects>, asset_server: Res<AssetServer>) {
    commands.spawn(Camera3dBundle {
        transform: Transform::from_xyz(7., 15., -9.5).looking_at(Vec3::ZERO, Vec3::Y),
        camera: Camera {
            clear_color: ClearColorConfig::Custom(Color::rgb(0., 0.7, 0.89)),
            ..default()
        },
        ..default()
    });
    objects.scene = asset_server.add(Scene::new(World::new()));
}

fn update(mut commands: Commands, objects: Res<Objects>, delete_query: Query<Entity, With<Piece>>) {
    for entity in delete_query.iter() {
        commands.entity(entity).despawn_recursive();
    }

    for _ in 0..1000 {
        commands.spawn((
            objects.scene.clone(),
            Piece,
        ));
    }
}

I can confirm that this snippet hash the same result.

The issue is that the spawned_instances field of the SceneSpawner never gets cleaned up.

A full write-up of what's happening.

When a new scene handle is added to an entity the scene_spawner system schedules the spawning of the scene with the SceneSpawner resource.
As a result an InstanceId is created and is pushed along with the necessary data into scenes_to_spawn and scenes_with_parent
Next the scene_spawner_system checks scenes_with_parent for despawned parent entities and marks down them down. Since our scene parent hasn't been despawned yet, this doesn't yet concern us.
Instead we move on to spawning all the scene in scenes_to_spawn.
Since our scene is not a dynamic scene, it will eventually be spawned with spawn_sync_internal. Importantly for us and our leak a InstanceInfo is created and pushed into spawned_instances.
Moving on to the next time scene_spawner_system is triggered. Our code has decided to despawn our scene and the system is about to notice and write it down.
Problematically dead_instances is only used to check if the scenes we're about to spawn still exist and nothing else.

So leak found. Just add something like
dead_instances.into_iter().for_each(|id| scene_spawner.despawn_instance(id));

to scene_spawner_system and et voila leak fixed, right? No, you see I lied to you when I said the scene_spawner_system would notice our scene parent pair has despawned. The pair is not in scenes_with_parent anymore. Why? Let's go back in time to the first time scene_spawner_system ran and spawned our scene.

Our scene has been spawned and we're about to run set_scene_instance_parent_sync which is what breaks our original fix.
You see set_scene_instance_parent_sync will mem::take the scenes_with_parent vec leaving it empty.
So the next time scene_spawner_system runs scenes_with_parent won't contain our now despawned scene, its InstanceId won't be logged in dead_instances and our original fix won't remove anything.

As things stand, the fix is unlikely to be trivial, requiring a good understanding of how scene data is passed around the SceneSpawner. This write-up took a while and it's getting pretty late here so if anybody wants to take a stab at it I wish you good luck.

Do we know when the bug was introduced?

I've so far managed to reproduce the issue with:

f53ee54
3a2a688
5eb292d

f53ee54 was chosen because it's the commit that introduce spawned_instances. The rest were chosen to quickly check if the leak was likely to always have been present. There aren't too many commits that have touched scene_spawner.rs. I could realistically check all of them but that would take me a while.


Adding Persian language support

I wonder if there is any Persian NLP tool available in the nltk.
Specifically, I want Persian semantic parser.

For basic Persian NLP, there's hazm.
If you are up to it, feel free to contribute and create a pull-request when it's ready =)
To start, take a look at https://github.com/nltk/nltk/blob/develop/CONTRIBUTING.md

Thanks.
So it seems there is no Persian semantic parser...
BTW, Is hazm is integrated with nltk?

No, hazm is a separate library.


Unable to scroll and bottom padding #54

Fixes Issue Unable to scroll and bottom padding #54

This issue is about adding scroll function and bottom padding so that the last cell may be visible.

Closes: #54
The changes you've made
1.index.html
added a div for bottom padding.
2.style.css
removed overflow hidden to make it scroll able.
Screenshot
Before

After

@rish-singhal please review it once

Sorry there were some commits which I had to merge on urgent basis . Please resolve the conflicting files.


[GAL-4257] "Connect Wallet" Upsell banner for email-only users [web]

Summary of Changes

Added upsell banner if the user don't any wallet connected
Added copywriting to the sign in wallet modal

This backend PR need to merge first - https://github.com/gallery-so/go-gallery/pull/1187
Demo or Before/After Pics
https://github.com/gallery-so/gallery/assets/4480258/2e00e7af-2cca-411f-8911-3af656ca9b4b
Mobile View

If this is a new feature, include screenshots or recordings of the feature in action.
If this PR makes visual changes, include before and after screenshots. You can type /table to quickly come up with a before/after table.

Edge Cases

Mobile view
Login with an user that has wallet connected

Testing Steps

Login with an user without any wallet connected
You should see the blue banner
Close it will update the user experience flag

Checklist
Please make sure to review and check all of the following:

[x] I've tested the changes and all tests pass.
[x] (if web) I've tested the changes on various desktop screen sizes to ensure responsiveness.

@kaitoo1 Update the old banner to match new design


Design: "Add New" plugin cards

Brief
Before installing a plugin with dependencies, its dependencies must be installed and activated. Dependencies can be displayed in the dependent plugin's card on the "Add New" screen. The user should be able to install and activate the dependencies from the dependent plugin's card.
The feature plugin has a proof of concept, and design feedback and possible iteration is much appreciated.
Current visuals
WordPress trunk

Feature plugin
If a dependent has missing dependencies, the feature plugin currently changes the Install Now button to a disabled Cannot Install button. A notice is displayed which informs the user that additional plugins are required and contains an action button with either Install Now or Activate, relative to the dependency's current status.

Installing a dependency using the feature plugin
On click, the Install Now button sends an AJAX request and installs the dependency. If installation was successful, the Install Now button will change to Activate. This is consistent with Core's existing plugin card Install Now buttons.
https://github.com/WordPress/wp-plugin-dependencies/assets/79332690/f17f3fea-0e2e-473a-a946-c80a94fadffa
Dependency installation failure
If installation fails for a dependency, an error admin notice is displayed below the dependency. The Install Now button changes to Installation failed. and the notice provides the error message.

@costdev I'm going to review what is in this ticket but also ask some questions and be sure to loop back for conversation. I know the idea is to get this into alpha. My focus on feedback is :shipit: - there is quite a bit I'd recommend ideal but today the focus is on what could be done and get in. We then iterate.
First, there seems to be an issue with a lot of information being in a very small place. This is really one of the burdens of WordPress cards. That makes me think a few things:

How can we limit what we are adding?
Are cards the right approach?

I wanted to put that out there first as my overall thinking as I began to go through the visuals.
Problem one: dense copy and multiple actions

Are you only able to have buttons and links here or could you include some different components? For example a checklist or other installing method? Could we do some show/hide visual? I don't want to recommend an accordion but right now this is a complex area. Another possibility here is to have some 'stepping through' interface. Just adding that as a possibility, but I would want to know my answer to the 'what is available'.
If I am correct the intention here is:

Show there is a dependency
Offer to install

Do we need to show multiples or could this be shown one by one? I realise this might be a little staggered in flow but considering our components are tight here it certainly might feel easier. My concern is both the complexity here and the scaling, what if someone had >5 as it doesn't look great with 2. I realise most will be 1.
Recommendations:

Reduce copy.
Consider different stepping interface if more than one dependency.
If stepping can't be done can you consider a modal? This is a bit of a kill all but 'out there idea'. Could you show number of dependencies, click those and see modal to then go in? I realise I also hate modals but it's a pattern we have now.

Problem two: error message seemingly below action

Technically it's not, but due to position it really feels like it is.
Recommendation:

Put above if can. I realise might not be able to and this area is problematic. I strongly think iteration after.

@azaozz gave 👍 for this design


sftp does not work - scp does

The recommended sftp command did not work for me from Debian11 to RPi.
If this comes up for anyone else we should just chnage back the recommendatiosn to scp.
~/backups$ sftp -r ./cl-rescue-d1829bc0215aab6941fe875e385d71e5.tar.gz  bitcoin@<IP_ADDRESS>:/mnt/hdd/temp/upload/
ssh: Could not resolve hostname ./cl-rescue-d1829bc0215aab6941fe875e385d71e5.tar.gz: Name or service not known
Connection closed
Connection closed.

:~/backups$ scp -r ./cl-rescue-d1829bc0215aab6941fe875e385d71e5.tar.gz  bitcoin@<IP_ADDRESS>:/mnt/hdd/temp/upload/
<EMAIL_ADDRESS>password:
cl-rescue-d1829bc0215aab6941fe875e385d71e5.tar.gz                                                                                100%  211MB  11.0MB/s   00:19

I alraedy changed back to scp for Windows on one ocsasion - see: https://github.com/rootzoll/raspiblitz/issues/3584#issuecomment-1387671283
I am open for a commit that changes back to scp in general.

Waiting for PR #3718 to be merged & tested


Hiding throw statement to make Get-Credential work correctly

this should resolve #11 and #9
It's not perfect - but I've tested every command with the throw statement removed and they all work.

This was fixed differently in #8


Update Login Process in Lastest PRO version (Eumi)

@tranbaquanthuy May you make a pull request for the latest version as well?


feat(core): add runOutsideOfZone ability

This is a new way to improve root zone performance and reduce monkey-patch impact inspired by @mhevery.

If Zone.current is rootZone(Zone.root), we will use native delegate (with a very small wrap) instead of monkey patched version.

For example:
After loading zone.js, we call some async operations. For example, in web page, we have Angular Elements and Jquery, the following code will be the jquery part.
import `zone.js/dist/zone.js`; // loaded by `Angular Elements`.
setTimeout(() => { // this will use `native window.setTimeout`
   // the callback will totally outside of Zone.
});

We can use Zone.root.run in other zone to use the native delegate version.

testZone.run(() => {
  Zone.root.run(() => {
      setTimeout(() => { // will use native delegate
         // will outside zone
     });
  });
});

Now the monkey-patch method will look like this.
For example, for setTimeout.
const nativeSetTimeout = window.setTimeout;
window.setTimeout = function() {
  if (Zone.current === Zone.root) {
    return nativeSetTimeout.apply(this, arguments); // this is the native one.
  }
  return patchedSetTimeout.apply(this, arguments); // this is the patched version.
}

@mhevery, @robwormald, please review, thank you!

How big is the performance improvement impact here? If it is not substantial, I would vote against this change. Alternatively, could this be configurable?
The reason I ask is that this change would break a lot of our code without a way to fix it other than maintaining a local modified version of Zone.js.
We (GCP Console, i.e. https://console.cloud.google.com) use Zone.js for latency instrumentation and rely on monkey-patching Zone.prototype.[runTask|scheduleTask|cancelTask] to track all executed tasks regardless of which zone they run in. If I understand this change correctly, this PR would prevent us from being able to track tasks executed in the root zone.


[Bug] error message ''Discord does not seem to be installed on your system.''

Describe the Bug

I can't use BandageDB when i open i gor an error message but discord is installed i already unistall/installed
To Reproduce

Expected Behavior

Screenshots

Discord Version

Additional Context

You will have to either have Discord running when launching the BBD installer, or browse manually to your Discord.exe


Default template set can not be used

My ruby is version ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-darwin14].
I've installed the gem with gem install generamba. Then as described here I did generamba setup and after answering several questions, I've got the Rambafile.
When I tried generamba template install I've got this error message:
/Users/xxx/.rvm/gems/ruby-2.2.2/gems/generamba-0.7.1/lib/generamba/template/processor/template_processor.rb:21:in `install_templates': undefined method `map' for nil:NilClass (NoMethodError)
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/generamba-0.7.1/lib/generamba/cli/template/template_install_command.rb:14:in `install'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/thor-0.19.1/lib/thor/invocation.rb:115:in `invoke'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/thor-0.19.1/lib/thor.rb:235:in `block in subcommand'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/gems/generamba-0.7.1/bin/generamba:5:in `<top (required)>'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/bin/generamba:23:in `load'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/bin/generamba:23:in `<main>'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/bin/ruby_executable_hooks:15:in `eval'
	from /Users/xxx/.rvm/gems/ruby-2.2.2/bin/ruby_executable_hooks:15:in `<main>'

Actually the error means that there are no templates, and in Rambafile, there are:
### Templates
templates:
#- {name: local_template_name, local: 'absolute/file/path'}
#- {name: remote_template_name, git: 'https://github.com/igrekde/remote_template'}
#- {name: catalog_template_name}

Its ok, but shouldn't it say that there are no templates to install instead of throwing an error?
P.S. None of the provided templates can be used as is.

@appleios thanks for reporting! We'll add a more informative error message.
And the commented templates are not meant to be used in real applications - they are just mark-up examples.


Add multipart support

Keep an eye out for php8 exclusive code, I think I got it all but more eyes is more better


client-c: fix data race when resolving async lock #8693

What problem does this PR solve?
Issue Number: close #8685
Problem Summary:
What is changed and how it works?
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

Fix the data race issue when resolving async lock

/run-all-tests

Maybe update to the latest commit 0a763f5f33e342802d0e188e8544540a015e808d to also resolve this issue https://github.com/pingcap/tiflash/issues/8715 on release-7.5? /cc @guo-shaoge

Close this PR and implement in https://github.com/pingcap/tiflash/pull/8752


Set up a GitHub action to sync related page content from Swiftype for each guide

Description
In order to populate the related resources for the page, we need to use the Swiftype Search API to fetch results for each guide page. We can use the search-research branch as a reference for the API call itself. We will need to ask Swiftype for search results on an interval. We can leverage GitHub actions to fetch the data on an interval. The results should write to a JSON file, which will be read by the resolvers for the new fields in the GraphQL schema (#589).
The params we send to Swiftype will be dependent on the decision made in #534.
Helpful links

Search research work
Open source website cron task
GitHub documentation

@roadlittledawn mentioned that we should filter out the document_type of views_page_menu from docs and open source to give us better results. Lets make sure to do that here as well.


Add link to new Go cookbook space

This PR adds a link to the CodeQL for Go topic to help people find the new cookbook space (https://help.semmle.com/wiki/display/CBGO/Go+cookbook). This space is currently published to the public wiki but only visible if you log in.

Thanks for quick response.


Actionbar Bug Potentially occurring with per_player add

Server Version
[01:27:33 INFO]: This server is running Paper version git-Paper-313 (MC: 1.15.2) (Implementing API version 1.15.2-R0.1-SNAPSHOT)
[01:27:33 INFO]: Previous version: git-Paper-311 (MC: 1.15.2)
[01:27:33 INFO]: You are running the latest version
Denizen Version
[01:28:05 INFO]: Denizen version 1.1.4-SNAPSHOT (build 4816-DEV)
[01:28:05 INFO]: Author: The DenizenScript Team
Issue
The Actionbar command doesn't seem to work when a player isn't linked to the event. In this example, I'm simply trying to use the actionbar command for all online players when an entity is damaged by the void.
Script:
https://one.denizenscript.com/haste/69555
Debug:
https://one.denizenscript.com/paste/69554

Please use Discord for this stuff.


Enumerate possible properties of a parameter

There is no explicit list of all available properties for parameters. Some are obvious (name, type) because they are in almost every example. But others like displayName only show up in a few spots and are easy to miss. A section similar to the data types that lists all the properties would be helpful when learning what one can do or specify regarding parameters. Or if there is already some definitive source for this, a link would probably work.

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

ID: ac576637-e7bc-2326-d234-8cc384c8a8a3
Version Independent ID: ac576637-e7bc-2326-d234-8cc384c8a8a3
Content: Parameters and templateContext - Azure Pipelines
Content Source: docs/pipelines/process/template-parameters.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@cvet-jendries -- thank you for your feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@juliakm -- please look into this issue.

@cvet-jendries You can find the possible properties in our YAML schema docs - https://learn.microsoft.com/en-us/azure/devops/pipelines/yaml-schema/parameters-parameter. I'll add a link to the parameters article.


How to configure 3d models in Android hand tracking GPU?  I want more accurate z coordinates, okay?

How to configure 3d models in Android hand tracking GPU?  I want more accurate z coordinates, okay?

Hi @LMR2018,  On Android Hands gradle you can find the result of https://github.com/google/mediapipe/blob/d4bb35fe5a44b3419ca86e4c7ecbc931dded2122/mediapipe/java/com/google/mediapipe/solutions/hands/HandsResult.java#:~:text=// Collection of detected/tracked hands' landmarks,}. To run on GPU you should enable runOnGput to True


Make alerts disappear after a while

Alerts on various pages could disapper after a while and/or could be closed like on the submit page

I think using the closable type would be sufficient and more robust for now

I agree


prod_cron_meta_nightly_pipeline: use main-integration-quarkus-2.13 branch for now to build OSL next nightly

Thank you for submitting this pull request
JIRA: (please edit the JIRA link if it exists)
link
referenced Pull Requests: (please edit the URLs of referenced pullrequests if they exist)

referenced pull request 1
referenced pull request 2
referenced pull request 2
etc.

How to retest a PR or trigger a specific build:

* a pull request please add comment: Jenkins retest this

@lampajr @Ginxo @radtriste OSL main nightly needs to be run against main-integration-quarkus-2.13 branch instead of main, due to incompatibility of some dependencies. For the main nightly we are using Quarkus version 2.13.5.Final ( prod version 2.13.5.Final-redhat-00002).
Is this PR the correct way to handle that? Do you guys have a different idea?

@rgdoliveira  I think it should work, it is just a branch like 1.32.x or whatever. We only need to ensure that all involved projects [1] have that branch setup.
[1] https://github.com/kiegroup/kogito-pipelines/blob/main/.ci/jenkins/Jenkinsfile.prod.nightly#L48

@lampajr @rgdoliveira This branch is set by Kogito community so should include all projects needed by prod pipeline


[Bug]: _redirects entry not included in .next

Steps to reproduce

Change plugin version to v4 from "Plugin" menu and selecting Options dropdown.
Change "Publish directory" to .next from "Site settings".
Retry deploy from Builds menu.

A link to a reproduction repository
No response
Plugin version
4.0.0
More information about your build

[ ] I am building using the CLI
[ ] I am building using file-based configuration (netlify.toml)

What OS are you using?
 Deploy site
1:28:47 PM: ────────────────────────────────────────────────────────────────
1:28:47 PM:
1:28:47 PM: Starting to deploy site from '.next'
1:28:48 PM: Creating deploy tree
1:28:49 PM: Creating deploy upload records
1:28:49 PM: 141 new files to upload
1:28:49 PM: 3 new functions to upload
1:29:01 PM: Site deploy was successfully initiated
1:29:01 PM:
1:29:01 PM: (Deploy site completed in 14.4s)
1:29:01 PM:
1:29:01 PM: A "_redirects" file is present in the repository but is missing in the publish directory ".next".
1:29:02 PM:
1:29:02 PM: ────────────────────────────────────────────────────────────────
1:29:02 PM:   Netlify Build Complete
1:29:02 PM: ────────────────────────────────────────────────────────────────
1:29:02 PM:
1:29:02 PM: (Netlify Build completed in 6m 51.4s)

Hi. Where is your _redirects file, and what does it contain? It should be in public if you want it to be moved to the .next directory.

Hi @ascorbic I have the file in the root repository and it has few custom redirect rules for our use case. But I have made sure that none of the rules overwrite NextJS as it says here.
This was working fine with v3 earlier and I couldn't find any message to move to public as per migration guide. Did I miss something? Any reason with the change?

The general rule is that _redirects goes in the published directory. I missed from the migration guide that were were previously copying it for you. I will fix that, and log a warning explaining what's going on. Thanks!

It does now! Sorry just had a chance to make the changes.
I figured that's what happen with v3 where the file was moved to published directory but might be missing in the new version. I like the log warning message too, that would explain all magic behind the plugin.
I'll leave this open if you want to follow up on the changes or for anyone come across this issue until update has been made, otherwise happy to close it as well.

The general rule is that _redirects goes in the published directory. I missed from the migration guide that were were previously copying it for you. I will fix that, and log a warning explaining what's going on. Thanks! Does it work if oyu move it to public?

I had this issue as well (it wasn't clear we need to move this to the public/ dir), our app functioned just fine when our _headers and _redirects files were in root .
Can we update the docs here to mention that these files should be in the public dir? I can also do it if that is acceptable

I have a draft PR which adds a warning and updates the docs. https://github.com/netlify/netlify-plugin-nextjs/pull/1078

I've been going mad over this since long. Justing putting my findings here:

When using GatsbyJS, the build actually outputs _redirects file in the final build that netlify uses for redirection. (no issues here)
When using NextJS, the netlify-plugin-nextjs plugin, it kind of reads the _redirects file and updates your netlify.toml file by making multiple new entries under [[redirects]] depending on your _redirects file. (this again is trouble some)

I had NextJS project where I was constantly trying to find the _redirects file in the final deploy, but then learnt that the netlify.toml actually had those entries (but still the redirection was not happening)
Then, upgraded the netlify-plugin-nextjs plugin on netlify to 4.0.0 to get it working. netlify.toml had similar modifications when I was using the older version of the plugin but don't know what was wrong here.
To reverify, I published an older version of my deploy that was built using the earlier version of the plugin but the redirection didn't work.


Meshmark does not exist on Crunchbase

The following URL cannot be found on Crunchbase:
https://www.crunchbase.com/organization/meshmark
I think the company has been renamed to:
https://www.crunchbase.com/organization/nobl9

Fixed with https://github.com/cncf/landscape/commit/644804506cc58df8f199bb75ba274edb5a164096


Curl dosent work

Hi, i just downloaded ubuntu but when i run the curl command i get this

Also since i have installed ubuntu HFS.exe and ffxiv.exe dosent work ( as far as i know exe files dosent work on linux machine) anyone has an idea ?, Thanks in advance

I had the same problem just yesterday. I fixed it running again "iptables ..." line and checking again if the ip from my default gateway console was placed correctly. Somehow, it changed which I´m not sure why, but doing those things it´s running ok now

I had the same problem just yesterday. I fixed it running again "iptables ..." line and checking again if the ip from my default gateway console was placed correctly. Somehow, it changed which I´m not sure why, but doing those things it´s running ok now

Didnt work for me, btw, does HFS work for you while you are using ubuntu ?

I tried running archlinux through VM but it says not enough disk space, then i tried installing the VM but it says some errors about oodle_helper, i also tried installing ubuntu OS 22.04 but it still the same problem as the picture apove, please if anyone has an idea or something i'd appriciate your help. ^^

as far as i know exe files dosent work on linux machine
I think you are supposed to install and launch HFS on your windows client, drag and drop the ffxiv.exe inside and then use the curl command inside the linux virtual environment to download the file from your local ip

as far as i know exe files dosent work on linux machine

I think you are supposed to install and launch HFS on your windows client, drag and drop the ffxiv.exe inside and then use the curl command inside the linux virtual environment to download the file from your local ip

Thanks it worked ^^. I spent 11 days trying to find the problem XD.
I really appreciate your help <3


License file is not closed after reading it

Hi,
I am using license3j to read the license file that i have.
It seems, It's not closing the inputstream whenever we are reading the license file.
I am using setLicenseEncodedFromFile(File, CharSet) method.
Below is the piece of code that i seen in License class
public License setLicenseEncoded(InputStream inputStream, String charset)
        throws IOException, PGPException {
        final ByteArrayInputStream keyIn = new ByteArrayInputStream(
            publicKeyRing);
        final InputStream decoderInputStream = PGPUtil
            .getDecoderStream(inputStream);
}

Here inside getDecoderStream we are only reseting the inputStream, We are not closing after reading it.
Am i doing something wrong?
Please help!

This seems to be a bug in that old version.
Please consider using the newest version of License3j.


General Grievance: Watch beachbuggyracing.net

General Grievance requests the watch of the watch_keyword beachbuggyracing\.net. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

beachbuggyracing\.net has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Ingo merge ingokarkat fork

I merged a ton of changes from @inkarkat's renamer.vim repo. I haven't done a lot of testing, but wanted to get some feedback on the change.
I couldn't merge these changes because I think they were implemented (in a different way) in fc95e9a8b256ae128b1b7279b0cc8024ebf93f3f:

inkarkat/renamer.vim@0d9cb0d  ENH: Allow recursive file listing. (2 years, 4 months ago)
inkarkat/renamer.vim@13afe2d  Fix validChars for multi-byte file name (4 years, 1 month ago)

It contains some vimscript cleanup/refactoring, improved tooltips, and a test rename command.

My process for getting these changes:

Rebase inkarkat's tree onto 033adeca81579aff1e009ef5bc9a8a390a2c23bc which matches inkarkat/renamer.vim@063283fb4ea933d53659cf6764156086847ab174. This change that's common to both repos but since I foolishly updated my name in commits the hash is different.
Cherry-pick changes from the rebased tree onto 2.1 (so I could tell what applied cleanly).
Rebase new tree onto master.

I did some merging but bailed when it got hairy. Usually it was pretty simple since inkarkat's changes are mostly atomic.

Thank you - sorry it took me a while to get to these!  I've not tested them yet, but looked at the changes and they look good.  I'll do some testing soon and in the unlikely event I find problems, I'll try to fix them.

Found one bug from the merge, will upload what I think is the simple fix.

Gah. Can't push my fixes as I'm behind github, but pulling from github says I'm up to date.  Sometimes git's debug info is most unhelpful...

You could try using the graph to see where your branches are in the overall
timeline: http://stackoverflow.com/a/9074343/79125
It's possible git's fetched all the commits, but they're not merged in
locally.
As a blind guess, git merge origin/master might also do it.
-David
On Tue, Oct 11, 2016 at 6:45 AM, John Orr<EMAIL_ADDRESS>wrote:

Gah. Can't push my fixes as I'm behind github, but pulling from github
says I'm up to date. Sometimes git's debug info is most unhelpful...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/qpkorr/vim-renamer/pull/5#issuecomment-252920755, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACqJwFH6my195fbJS8xjK5ypqMMgGP-ks5qy5LlgaJpZM4J0IbD
.

Thank you - the complicating factor was I was using git subtrees - I don't have time to dig into the details but I hope I've fixed it thanks to this post http://stackoverflow.com/questions/13756055/git-subtree-subtree-up-to-date-but-cant-push  (though I'm always nervous when "--force" is involved!)  I'll try to do some more testing and see if anything pops up - and then potentially get onto his other two changes.  Thanks again!


count_true name

I think that count_true is not a good name.
It is too similar to countl_xxx and countr_xxx families with a too different semantic.
the countl_xxx family has names from std:: scalar equivalents. Perhaps it is these names
that are to be changed ?

It follows first_true name.
I would also like count_false, last_true, nth_true, first_false, last_false, nth_false at some point.
Or smth like that.
What names are you thinking?

count_true is ok, that's the other countl/r things we copied from std that should change maybe

Closed as @jtlap agrees it's a non issue


Re-allow job spec manifests to use yaml anchors

This was previously allowed, but the functionality was removed as part of a previous upgrade of Ruby.
Report of the problem in CFF Slack

Lgtm. Thanks!


Support MinFeeRateNanosPerKB in atomic txn construction

#602  👈
feature/proof-of-stake

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lazynina and the rest of your teammates on  Graphite


Loosening Octokit version constraint so it doesn't conflict with 4.0.1 of Berkshelf

\cc @jkeiser @reset

:+1:

We should probably add spec.required_ruby_version = ">= 2.0.0" to this gemspec since varia_model 0.5 depends on that anyways - @reset ?

Passed in a local ChefDK omnibus build and passed chef verify

Thanks @reset !


Check use of dbeel/ create template db from dbeel

The current DB (dbeel) is complex: but it could be easier. Some tweaks to the current DB. (OTHERWISE it is great!)

Probably stale if we use RDBES but still we can get all dictionaries ....


Focus shifts away from keyboard every time a new character is entered

b/167954350
Steps to Reproduce
import 'package:flutter/material.dart';

void main() {
  runApp(MaterialApp(home: OuterTest()));
}

class OuterTest extends StatefulWidget {
  OuterTest({Key key}) : super(key: key);

  @override
  OuterTestState createState() => OuterTestState();
}

class OuterTestState extends State<StatefulWidget> {
  bool toggle = false;

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(title: Text('test')),
      body: Center(
        child: Column(
          children: <Widget>[
            TextField(onChanged:(_) {
              setState(() {
                toggle = !toggle;
              });
            }),
            if (toggle) Text('toggled text'),
          ],
        ),
      )
    );
  }
}

1, turn on iOS voiceover
2, foucs the textfield and double test to bring up the keyboard
3, tap the test title to focus the title
4, tap the keyboard key directly to bring the focus to keyboard
5, double tap to type the word
expect: the focus should stay on the keyboard.
actual: the focus jumps back to the title.

flutter doctor -v
[✓] Flutter (Channel master, 1.22.0-10.0.pre.109, on Mac OS X 10.15.6 19G2021,
    locale en-GB)
    • Flutter version 1.22.0-10.0.pre.109 at
      /Users/tahatesser/Code/flutter_master
    • Framework revision e682ec711a (3 hours ago), 2020-09-09 10:52:27 +0200
    • Engine revision d1d848e842
    • Dart version 2.10.0 (build 2.10.0-98.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.7)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.7, Build version 11E801a
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 49.0.2
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.48.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.14.1

[✓] Connected device (3 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS
      13.7
    • Web Server (web)       • web-server                • web-javascript •
      Flutter Tools
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 85.0.4183.83

• No issues found!


Adds support for custom library installation on Plugins

We need to enable plugins to have its own requirements.txt, enabling better dependency handling and not editing the main dialog container file.

I have a proposal for this
All dialog plugins must be a valid python package published on pypi with some entrypoint metadata (like pytest plugins). The PLUGINS  env variable will receive a list of installable plugins and all of them be installed and loaded at runtime


⚠ ClamAV has degraded performance

In 52abfc9, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 80 ms

Resolved: ClamAV performance has improved in 5e21fad.


UncontrolledCollapse "SyntaxError '##toggler' is not a valid selector" test failure

Operating system and version macOS 10.14.5
Browser and version Firefox 67.0 (64-bit) (however, issue occurs in unit tests)
Test case - https://github.com/akilman/reactstrap-collapse-bug

In attempting to use the UncontrolledCollapse component, I noticed it works like a charm when run on the browser but not during rendering for a test case.
I was able to reproduce the behavior in this repo which was a plain create-react-app with bootstrap and reactstrap installed. I copy/pasted the same Example from the docs noted above. And same behavior. Browser works just fine, test fails with the following error:
SyntaxError: '##toggler' is not a valid selector

      at emit (node_modules/nwsapi/src/nwsapi.js:563:17)
      at Object._querySelectorAll [as select] (node_modules/nwsapi/src/nwsapi.js:1492:9)
      at findDOMElements (node_modules/reactstrap/lib/utils.js:239:28)
      at UncontrolledCollapse.componentDidMount (node_modules/reactstrap/lib/UncontrolledCollapse.js:55:48)
      at commitLifeCycles (node_modules/react-dom/cjs/react-dom.development.js:17334:22)
      at commitAllLifeCycles (node_modules/react-dom/cjs/react-dom.development.js:18736:7)
      at HTMLUnknownElement.callCallback (node_modules/react-dom/cjs/react-dom.development.js:149:14)

Sorry, wrong repo


wandb.Video usage with np.uint8

wandb --version && python --version && uname

Weights and Biases version: 0.8.13
Python version: 3.7.4
Operating System: Linux

Description
I'm trying to use wandb's Video object with np.uint8 -
should this following example work properly?
Because I couldn't get it to work without deleting these two lines in the wandb data_types file.
https://github.com/wandb/client/blob/master/wandb/data_types.py#L624
What I Did
minimal example of a gif that goes from blue to red
import numpy as np
import wandb

wandb.init()

video = list()

for a in np.arange(0, 1, 0.01):
    image = np.zeros((3, 64, 64), dtype=np.uint8)
    image[0] = a * 255
    image[2] = (1-a) * 255

    video.append(image)

wandb.log({'video': [wandb.Video(np.uint8(video), fps=50)]})

Thanks for flagging this.  It's a bug.  Should be fixed by https://github.com/wandb/client/pull/639.

Just released 0.8.14 with the fix.


Math expressional parameters for aggregator

Someone wanted druid to support expressional parameter for aggregator something like sum(a + b / c). I don't know why it's needed either but this is it. Will make propose in dev mailing list soon after.
Based on great work(#2090) of @himanshug.

Do you allow quoting of column names? Are column names case-sensitive? I suggest you make the syntax as close to SQL as possible. That would also mean supporting quoting, and also using 'iif' rather than 'if'.

@julianhyde Currently it does not allow quoting and identifiers are case-sensitive. I've used double quote for string literal in #2836 but it would be better to change it single quote. I'll handle your comments in #2836.

@navis Thanks! We know that this expression language is going to be popular, and then we're going to wish that it as close to SQL as we can manage, otherwise we'll spend years trying to undo the differences.

@julianhyde we made the decision back in Druid 0.7 to have column names be case-sensitive to keep things simple implementation-wise. As far as I know SQL doesn't prescribe things to be case-insensitive for column names. For instance PostgreSQL is case-sensitive, but has it's SQL layer fold column names to lower-case, if not quoted. I think it would be worthwhile to have a similar behavior for Druid where any case-folding is left to the SQL layer doing the query translation.

@xvrl didn't changed case-sensitivity of identifiers and have no intention to do that, but possibly make an option for it.

It's totally valid and workable to have case-sensitive identifiers and leave their case intact (i.e. not convert them to lower case as PostgreSQL does, or to upper case as Oracle does).
The only other thing is spaces in column names. If you allow space (and other punctuation) in column names then you need a way to use these in expressions. You might use double quotes (as Oracle and PostgreSQL), brackets (as SQL Server) or back-quotes (as MySQL).

I think this is duplicate of #1965. @himanshug could you review this?

@navis yes, i planned to add expression language to aggregators and post-aggregators in #1965 , was waiting for expression thingie to get merged.
i see you have multiple PRs in flight on same , i will review these as soon as i can.

Squashed and merged with #2820, caused by hard-maintenance for long abandoned time.

did a quick scan, overall looks fine to me... can give a more careful look next week.
i'll be happy to see the expressions in to action finally.

Will there also be some documentation updates for this?

@erikdubbelboer yes, although likely in a future PR (the expression stuff this is based on was never documented).

Still 👍 on this, although consider https://github.com/druid-io/druid/pull/2783/files#r83581162.

👍  to changes, thx @navis.

new changes LGTM, :+1:

thx @jon-wei for taking a look. @himanshug, any further comments?


⚠ Bofrost.es has degraded performance

In 8b6d2c0, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1615 ms

Resolved: Bofrost.es performance has improved in 5ce6033 after 6 minutes.


Update dev containers for Dataflow

TODO: rerun tests after containers are actually released.

r: @riteshghorse

Run Python PreCommit

Run Python_Coverage PreCommit


fix : payment preimage parsing.

What you are fixing with this PR? please when you propose something use always a description of it.
You should help me to understand why I should merge this PR


[Icons] Missing icons

[X] This is a v1.x issue (v0.x is no longer maintained).
[X] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

FileCopy should exist as component in the @material-ui/icons package.
Current Behavior

FileCopy is not recognized as import from the icons package nor does it exist in the source packages/material-ui-icons/src
Steps to Reproduce (for bugs)
N/A
Context
Just trying to add FileCopy icon found by my Designer here https://material.io/tools/icons/?search=file&icon=file_copy&style=baseline
Your Environment

Tech
Version

Material-UI
v1.1.0

React
v.16.4.0

browser
Chrome 67.0.3396.79

@napcoder It's far from being the only missing icons. Have a look at all the opened issues: https://github.com/google/material-design-icons/issues. We are waiting for the next release to update @material-ui/icons.

Is there a release milestone we can track for this?

@baer Nop

@oliviertassinari If this can help, I think I only have the icons from the letters V to Z.
Is this related to version 1.1.0?

Now published as v2.0.0-beta.1.


442: Implement handling exceptions to reject the consent

Issue: https://github.com/SecureBankingAccessToolkit/SecureBankingAccessToolkit/issues/442
All the errors are using the following Error model:
{
  "Code": "string",
  "Errors": [
    {
      "ErrorCode": "string",
      "Message": "string",
      "Path": "string"
    }
  ]
}

In the RS project for calculate elements APIs we have two types of verification on the request body.

The verification of all the required fields and their format/value (the Instructed Amount) for the all the types of Domestic Payments. Returning a DATA_INVALID_REQUEST error type with a custom message if the criteria aren't met.
The verification of all the required fields and their format/value (the Instructed Amount, the Exchange Rate Information) for the all the types of International Payments, depending of the rate type (Actual, Indicative, Agreed). Returning a DATA_INVALID_REQUEST error type with a custom message if the criteria aren't met.

Can we have a bit more description for the PR please Andra? If you could explain your approach to handling the exceptions etc that would be useful. Thanks!

The BigDecimal formatting will be removed after we finish this unplanned task https://github.com/SecureBankingAccessToolkit/SecureBankingAccessToolkit/issues/483.


Smoke test failure

The dask-on-ray tests began failing over the weekend.
https://github.com/astronomy-commons/hipscat-import/actions/runs/7163899027/job/19503061281
The following packages changed versions:
astropy-iers-data             0.2<IP_ADDRESS>.30.20
astropy-iers-data             0.2<IP_ADDRESS>.31.11

isort                         5.12.0
isort                         5.13.0

pre-commit                    3.5.0
pre-commit                    3.6.0

typing_extensions             4.8.0
typing_extensions             4.9.0

and the following were ADDED:
Cerberus                      1.3.5
docopt                        0.6.2
pip-api                       0.0.30
pipreqs                       0.4.13
plette                        0.4.4
pydantic                      2.5.2
pydantic_core                 2.14.5
requirementslib               3.0.0
yarg                          0.1.9

I forgot to look into this more, and the smoke test starting passing again on Tuesday.


The MLKit selfie segmentation model should be finetuned for office chairs

The MLKit selfie segmentation model models/selfiesegmentation_mlkit-256x256-2021_01_19-v1215.f16.tflite is pretty good, but it could still be finetuned for office chairs.
The model card describes the dataset used to train the model as "1700 images, 100 images from each of 17 the
geographical subregions ..."

Tagging as an enhancement and help-wanted as I don't know if anyone currently involved can do this!

Another idea for "automatic" tracking suggested by @martok the other day was recording some video of your background and automatically tagging everything above some noise threshold as foreground in order to gain training material.  And you even get the network to be quite sensitive only to one person, which could have the advantage that other people in the background could be filtered by such a net.

Another idea I've been toying with has been approaching the problem from a genetic algorithm perspective, where you have a crawler that starts from the bottom left side of the screen, and moves across the bottom until it figures out where the human figure is, and then just algorithmically looks at the probably border pixels to draw an outline.

The model card describes the dataset used to train the model as "1700 images, 100 images from each of 17 the geographical subregions ...

Actually, Google's model cards do not detail training data, but only some validation data.
Their goal is to make us aware of potential bias regarding a population without disclosing the training data (AI should not discriminate).

@BenBE @phlash side note: should we use the selfiesegmentation model as default for the next release?


Add kwargs to BIDSLayout init

A new option was added to grabbit classes. In order to expose these to packages using pybids having the init of BIDSLayout take kwargs and then pass them on in the super call to the grabbit Layout class is necessary.

Do we need to update minimal required grabbit version?

This PR should work regardless of grabbit version.


Aligned memory

Feature suggestion: implement a memory manager that aligns memory on 32,64,... byte boundaries. This is useful when doing FFTs for example.

@davisking Do you think this would be a useful feature? it is necessary if you want to do explicit SIMD math (SSE, AVX, NEON, etc) and it also helps the compiler do auto-vectorisation

SIMD math works fine without such tooling already.  For instance, the HOG
code uses SIMD instructions.  So this doesn't seem like something we need.

Ok. Do you want to keep this open for discussions? Otherwise happy if you want to close the issue.

That's fine.  If you have a specific piece of code where this makes things
faster that would be cool.

It's a habit I have to align memory so that when using SSE or AVX, I can do an aligned load, instead of an unaligned load. Also, I'm used to using FFTW which encourages the use of aligned memory for speedy computation. But for BLAS libraries, i'm willing to accept if makes near to no difference.

Sorry i didn't close this sooner.


Text overlap on details page

Package version (if known): v7
Describe the bug
Text crowding occurs in the ‘Details’ section of some records in preview and view mode. Below is an example in the 'Related Works' field.

Steps to Reproduce

Fill out a related works with a moderate size URL
publish

Expected behavior
No text overlap.

Landing page went through a full redesign so I'm closing this one.


re-add toStyle function for flat style format

Up until version 8 (?) (hash: 833f5f1156cf1b00e4a985973df371dc91c1b3f8) you could use toStyle to turn a flat style object into style class. This was very convenient to use in style-function because you could do something like this:
function styleFunction(feature, resolution) {
   const s = size_dict[feature.get("Ort_Klasse")] || 5;
   const style = feature.get("style")
   const res = s * (BASE_SIZE / resolution);
   toStyle({ ...DEFAULT_STYLE, ...style, "circle-radius": res })
}

which is, I think, a common use case. Merging object with the spread syntax is one of the main strength of the flat style.
Right now, you have to use
import {
  flatStylesToStyleFunction } from "ol/render/canvas/style.js";

function pointStyleFunction(feature, resolution) {
  const s = size_dict[feature.get("Ort_Klasse")] || 5;
  const res = s * (BASE_SIZE / resolution);
  return flatStylesToStyleFunction([{ ...defaultStyle, "circle-radius": res }])(
    feature,
    resolution
  );
}

which works, but feels somewhat "hacky".
A convenient function, which allows to merge a base style with a feature specific style or dynamically computed style would make development more streamlined.

That can now be done within a flat style https://codesandbox.io/s/style-expressions-forked-5hm3sk

As noted in https://github.com/openlayers/openlayers/issues/14849#issuecomment-1598223651 and https://github.com/openlayers/openlayers/issues/14367#issuecomment-1778865990 flat styles do not yet support the geometry option so toStyle would be needed there.  Also for Feature and Select styles, I have raised #14712 to handle that.

We’re working toward removing the style and symbolizer classes, and the toStyle function does not take us in that direction.
Also note that functions that are not included in the API docs (marked with @api in the code) are not part of the API - so subject to change at any time.  The flatStylesToStyleFunction is an example of a function that is not part of the API.
It may be that we add support for returning flat styles from a style function.  But our efforts at this point are about expanding the support for expression-based styling.

Thank you for the example and the explanation. My concern is, that it forfeits the strength of a flat style object or restricts you in the generation of style object.
Especially when it comes to complex styling, like custom symbols or piecarts (like ol-ext) or missing properties in the features, basically where you need a complex / dynamic function to generate the style output.
For example, some constructs I can to with  a style function, but I am not sure if I can do this with the rule approach (right know)
const chart = chartGenerator(feature.get('data')) // some function to generate data url or call an image from an api
const style = { ...BASESTYLE, 'img-src' : chart, 'text-value' : feature.get('title') || feature.get('feature_id') || 'NOT SET' }

My guess is, that is that the rule based approach is needed / better suited for the gpu implementation of the webgl renderer.
Also, you said this is work in progress, so perhaps my "use-cases" are already on the radar but not yet implemented.
Perhaps the rules need something like this:
{'img-src: ['call', chartGenerator, 'data'], 'text-value' : ['getFirst', 'title, 'feature_id', 'NOT_SET'] }

+1 from my side for this. As good as flat-styles are, they are awfully restricted.  Right now, I have trash my flat-style work and convert everything back to Style classes.

@sookoll It would also help if you could report any shortcomings you found when working with flat styles. In other words: what was missing that forces you to go back to style classes now?

@ahocevar along other styling with geometry things I have quite complex measurement component. Can't show that currently but there is pits and pieces from here: https://openlayers.org/en/latest/examples/measure-style.html
So how to do that in flat-style? Geometry thing is impossible. style.setText() is impossible, You have to create style from scratch. I'm not saying, that flat-style is wrong direction. No, I like it, but please do not force peole to use something if it is worst than it was before. toStyle should remain as long as everything is covered. And even then I think it is bad idea to remove it long time without deprecation warning. After all, a solid long term (enterprice) support should always remain. It is vital if You provide a library.

Thanks for describing the shortcomings you ran into, @sookoll. Removal of style functions is not going to happen any time soon, so it would be too early to deprecate.
Flat styles are meant to be created from scratch, so setText() is not needed. If it is desired to not create a new style, it will be possible to use style variables for that in the future.
For geometries, we might have expressions for points along a line in the future. However, the concept of defining temporary custom geometries in a style might not be the best way forward. The same thing could be done in a more elegant way - with a separate layer holding features with geometries created the same way as you are currently doing for the style geometries. That layer could have a very static style, and you would just modify it's features' geometries as the measurement changes.


Remove lodash.throttle and replace underscore calls with lodash

Remove lodash.throttle from dependency since there is now lodash.
Add babel-plugin-lodash that helps optimize bundle output by taking only necessary part from lodash instead of the entire bundle.
https://github.com/lodash/babel-plugin-lodash
Replace underscore calls with lodash where applicable.

@williaster @xtinec @conglei

Codecov Report

Merging #5946 into master will decrease coverage by 0.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master    #5946      +/-   ##
==========================================
- Coverage   63.58%   63.56%   -0.02%
==========================================
  Files         393      393
  Lines       23674    23661      -13
  Branches     2639     2639
==========================================
- Hits        15053    15041      -12
+ Misses       8608     8607       -1
  Partials       13       13

Impacted Files
Coverage Δ

...t/assets/src/components/InfoTooltipWithTrigger.jsx
56.25% <ø> (-2.58%)
:arrow_down:

superset/assets/src/visualizations/nvd3/NVD3Vis.js
7.76% <ø> (-0.23%)
:arrow_down:

superset/assets/src/components/TooltipWrapper.jsx
100% <ø> (ø)
:arrow_up:

...assets/src/dashboard/components/dnd/handleHover.js
44.44% <ø> (-5.56%)
:arrow_down:

...uperset/assets/src/SqlLab/components/SqlEditor.jsx
77.66% <ø> (-0.22%)
:arrow_down:

superset/assets/src/reduxUtils.js
75% <ø> (-0.41%)
:arrow_down:

superset/assets/src/components/Button.jsx
90.47% <ø> (-0.44%)
:arrow_down:

superset/assets/src/modules/utils.js
52.84% <ø> (-0.76%)
:arrow_down:

superset/assets/src/components/AlteredSliceTag.jsx
100% <ø> (ø)
:arrow_up:

...ssets/src/visualizations/deckgl/layers/polygon.jsx
0% <0%> (ø)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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

Thank you. Please merge when you are ready.


Extension causes high cpu load

Version: 1.43.0
Commit: 78a4c91400152c0f27ba4d363eb56d2835f9903a
Date: 2020-03-09T19:34:44.548Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 19.2.0
CoenraadS.bracket-pair-colorizer-2-unresponsive.cpuprofile.txt
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Duplicate of #55. Please close this and post there instead.
@javahuang


Added a new feature to convert text and markdown files to styled HTML

… using TOML configurations. This includes checking and handling input/output directories, and creating neat HTML outputs.

Amazing job @mnajibi! Thanks for pushing all the required changes!


ng add @ng-bootstrap/ng-bootstrap : Cannot find module '@schematics/angular/utility/project'

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[x] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
 Yes, the previous version in which this bug was not present was: ....

Angular CLI: 9.1.8
Node: 12.18.1
Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.901.8
@angular-devkit/core         9.1.8
@angular-devkit/schematics   9.1.8
@schematics/angular          9.1.8
@schematics/update           0.901.8
rxjs                         6.5.4

Description
 A clear and concise description of the problem...
https://github.com/ng-bootstrap/ng-bootstrap/issues/3788
🔬 Minimal Reproduction

Install CLI ( latest is 10.0.0)
npm install -g @angular/cli

create a new project
ng new [project-name]

cd to the project root, install bootstrap and update angular.json
npm install save --bootstrap

install ng-bootstrap
ng add @ng-bootstrap/ng-bootstrap

🔥 Exception or Error

[error] Error: Cannot find module '@schematics/angular/utility/project'
Require stack:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>C:\Users\fiona\AppData\Roaming\npm\node_modules\@angular\cli\bin\ng
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:966:15)
    at Function.Module._load (internal/modules/cjs/loader.js:842:27)
    at Module.require (internal/modules/cjs/loader.js:1026:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Object. (C:\mlng-framework\gis-semantics-health\node_modules\@ng-bootstrap\ng-bootstrap\schematics\ng-add\index.js:6:19)
    at Module._compile (internal/modules/cjs/loader.js:1138:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1158:10)
    at Module.load (internal/modules/cjs/loader.js:986:32)
    at Function.Module._load (internal/modules/cjs/loader.js:879:14)
    at Module.require (internal/modules/cjs/loader.js:1026:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at new ExportStringRef (C:\mlng-framework\gis-semantics-health\node_modules\@angular-devkit\schematics\tools\export-ref.js:19:25)
    at NodeModulesEngineHost._resolveReferenceString (C:\mlng-framework\gis-semantics-health\node_modules\@angular-devkit\schematics\tools\node-module-engine-host.js:95:21)
    at NodeModulesEngineHost.createSchematicDescription (C:\mlng-framework\gis-semantics-health\node_modules\@angular-devkit\schematics\tools\file-system-engine-host-base.js:174:34)
    at SchematicEngine.createSchematic (C:\mlng-framework\gis-semantics-health\node_modules\@angular-devkit\schematics\src\engine\engine.js:220:38)
    at CollectionImpl.createSchematic (C:\mlng-framework\gis-semantics-health\node_modules\@angular-devkit\schematics\src\engine\engine.js:70:29)

🌍 Your Environment

>ng version

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 10.0.0
Node: 12.18.0
OS: win32 x64

Angular: 10.0.0
... animations, cli, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.1000.0
@angular-devkit/build-angular     0.1000.0
@angular-devkit/build-optimizer   0.1000.0
@angular-devkit/build-webpack     0.1000.0
@angular-devkit/core              10.0.0
@angular-devkit/schematics        10.0.0
@ngtools/webpack                  10.0.0
@schematics/angular               10.0.0
@schematics/update                0.1000.0
rxjs                              6.5.5
typescript                        3.9.5
webpack                           4.43.0

Anything else relevant?
I have to downgrade CLI 10.0.0 to CLI 9.1.8.

The add functionality for a package is contained within the package itself and is not present within the Angular CLI.  A fix for this issue with ng add @ng-bootstrap/ng-bootstrap would need to be made to @ng-bootstrap/ng-bootstrap package.
The root cause of this particular error has an open issue with the project and can be tracked here: https://github.com/ng-bootstrap/ng-bootstrap/issues/3714

@clydin :
https://github.com/ng-bootstrap/ng-bootstrap/issues/3714 is not the root cause of the CLI error.
I have followed https://github.com/ng-bootstrap/ng-bootstrap#installation but it didn't fix the issue.  I have used the exact same command ng add @ng-bootstrap/ng-bootstrap on Wed June 24th CLI 9.1.8. It worked.
I install another CLI (the system fetches the lastest 10.0.0) and create Project in another folder on Thu June 25th. Then things fall apart.
These two projects are all the same. Why is the Private APIs should not be used for the ng add schematic related to my issue?


[HUDI-245]: replaced instances of getInstants() and reverse() with getReverseOrderedInstants()

Small code refactoring to remove redundant code.

LGTM. Thanks @pratyakshsharma


Release 1.18.3

blocked by https://github.com/astronomer/astronomer-providers/pull/1371 and the actual release of https://github.com/apache/airflow/issues/35845


Why the need for DATABASE_TYPE and DB_URI when a single DSN is suffice?

Current env:
DATABASE_TYPE=redis or badger
DB_URI=redis://<IP_ADDRESS>:6379/0 or /path/to/folder

What I'm suggesting:
DATABASE=redis://<IP_ADDRESS>:6379/0
DATABASE=badger:/path/to/folder
DATABASE=gcs:/path/to/key.json

DATABASE=driver://user:pass@host:port/db?opt=value
DATABASE=driver:/path/to/file_or_folder

i thought it was invalid uri scheme


virtcontainers: revert "fix shared dir resource remaining"

This reverts commit 8a6d383715f6cd8e0777ab5d14a421129bdff8c0.
Don't remove all directories in the shared directory because
docker cp re-mounts all the mount points specified in the
config.json causing serious problems in the host.
fixes #777
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

/test

re-starting jenkins-ci-fedora-27

lgtm

Need find another way to do the clean up. Will do it later~ @amshinde


feat: liquidity migration

Closes OPT-175
Closes OPT-174
Closes OPT-177

This is looking great so far!

The contract implementation looks good to me as well!

Amazing job sir!

External review at https://github.com/ethereum-optimism/optimism/pull/11479

Merged at https://github.com/ethereum-optimism/optimism/pull/11479


Add  to the upcoming list

September 10-11, 2020 (Workshops) & September 14-15, 2020 (Conference)
x33fcon

<September 10-11, 2020 (Workshops) & September 14-15, 2020 (Conference)>

Thank you so much, @blinkerfluid !


unknown option "max_touch_minor"  for mtdev input provider

It seems there is a typo in the mtdev.py provider, the  max_touch_minor option is not recognized.
In Line 92  min_touch_major should be replaced with max_touch_minor
options = ('min_position_x', 'max_position_x',                    'min_position_y', 'max_position_y',                    'min_pressure', 'max_pressure',                    'min_touch_major', 'max_touch_major',                    'min_touch_minor', 'min_touch_major',                    'invert_x', 'invert_y',                    'rotation')

This was fixed with #4621.


OS X dragging folder from Finder to get path

In terminal it's possible to drag a folder or file into the terminal window and the path to said folder/file is populated. This is useful when you can't quite remember that path to something, but you can find it in Finder!
Is this something that can be implemented in hyperterm? It's definitely not an issue, just more of an enhancement.

#329

That's awesome! Look forward to when either or both of those get merged.


fix component lookup

Proposed changes:

simplify component lookup
fix py3.5 error when looking up templates with nested dicts

Status (please check what you already did):

[x] made PR ready for code review
[x] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog
[x] reformat files using black (please check Readme for instructions)

no changelog needed since Sam's changes were not released yet

Build is currently failing on some weird issues which I cannot reproduce locally. Already checked the library versions 💥

@tabergma Could you give it a quick review cause builds on master fail currently due the py3.5 problem


Arrow api, fixes Pred.concat

Coverage remained the same at 100.0% when pulling d7d3b175bb02096b9cbfee3b1321f9e9d0c8fb0a on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.

Coverage remained the same at 100.0% when pulling 4d1844dd21d6dce0fb8d8f1bcc275aa475600c06 on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.

Coverage remained the same at 100.0% when pulling c69efe04fd20d678c673154b3c0b9b47e7f09120 on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.

Coverage remained the same at 100.0% when pulling c69efe04fd20d678c673154b3c0b9b47e7f09120 on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.

Coverage remained the same at 100.0% when pulling bedbce5649cd33f20ee91f837c66c3fc6742a442 on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.

Coverage remained the same at 100.0% when pulling 2bccb462288c37169281721da491b511812bd935 on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.

Coverage remained the same at 100.0% when pulling 57077a39c74c7b4c29b023b3c2b636e2aace613f on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.

Coverage remained the same at 100.0% when pulling 57077a39c74c7b4c29b023b3c2b636e2aace613f on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.

Coverage remained the same at 100.0% when pulling a4364e07726e0f61b796582850d49c4f9e5772a3 on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.

Coverage remained the same at 100.0% when pulling f7868437c39a60c148144af6833ecd56cd832b95 on rstegg:arrow-api into 063f30db1aa680b1c90f95758a42c0ec6e0628f5 on evilsoft:docs.


Blockstream Jade fails to show address on device and sign with multisig

Using Sparrow 1.6.6 and Jade 0.1.34, I can display single sig wallet addresses on the Jade device, but trying to show multisig addresses fails with an RPC error:
"device reports readiness to read but returned no data (device disconnected or multiple access on port?)*
or
"JadeError: -32600 - Invalid RPC Request message (Data: b'213')"
Similar errors happen if I attempt to sign a transaction.
To reproduce, I created a new 2-of-3 wallet with 2 shares generated by Sparrow, and one share being the Jade.

I'm not able to reproduce this. Jade successfully displays the a multisig wallet address (after registering the wallet initially). Signing a multisig transaction works fine too. Same firmware version.
I suspect a USB bus/kernel issue might be involved, at least for the first result you obtained.
To debug this further, I suggest using HWI (the library Sparrow uses for USB communications) directly. For example:
./hwi --device-type='jade' --device-path=XXX --desc 'wsh(sortedmulti(2,[06722fd4/48h/0h/0h/2h]xpub4FAFKj.../0/1,[d43a1b59/48h/0h/0h/2h]xpub4FhbZ.../0/1))' displayaddress

HWI can either be found in the Sparrow home folder (OSX or Windows) or by looking at the Tools > Install Udev Rules in Linux. Alternatively, get it from https://github.com/bitcoin-core/HWI. The device path can be found using hwi enumerate.

I tried
$ cd /Applications/Sparrow.app
$ find . -iname hwi\*
$

but found nothing.

(after registering the wallet initially)

How are you doing this? Maybe that's the missing step, because I started with a factory reset Jade device. That is, I'm only using Sparrow and a Jade, and in particular not the Green wallet.

Take a look at https://help.blockstream.com/hc/en-us/articles/900005443223-Why-is-Blockstream-Jade-not-recognized-by-my-computer-
It does appear that a specific driver is used, which is probably installed by the Green wallet. I did install Green when working on the Jade integration (to update the firmware) so that may be a key difference.
Re HWI, the location of the Sparrow home folder is described here: https://sparrowwallet.com/docs/faq.html#where-does-sparrow-store-data

Closing this off.


Modify direction key to code #1

A pull request is not needed if you only want to add more game prompts instead of optimizing the algorithms. You can custom it in your own repository.


Allows passing filters to AzureSearchVectorStoreRetriever

Description: Allows passing filters to AzureSearchVectorStoreRetriever,
Issue: https://github.com/hwchase17/langchain/issues/6131,
Dependencies: https://github.com/hwchase17/langchain/pull/7154(optional: it allows adding extra sections. it can be done manually through the azure ui as well)
@rlancemartin, @eyurtsev @hwchase17,

Example:
filters = {
    "filter1": "document_id eq 'xcf76'",
    "filter2": "document_id ne 'xcf2'"
}

retriever = AzureSearchVectorStoreRetriever(
    vectorstore=vector_store,
    filters = filters,
    k=1)

llm = AzureOpenAI(engine="gpt-35-turbo",
                  model_name="gpt-35-turbo",
                  temperature=0.3)
qa = RetrievalQA.from_chain_type(llm=llm, chain_type="stuff", retriever=retriever, return_source_documents=True,
                                 verbose=True)

res = qa('what is subject x based on')
print(res)```

Doing it this way will break the other way to do this


Feature: Add Async Deserializer for deserializing messages

Hello! I am currently using your amazing library and just found out that you have avro deserializer. The only  problem that it is synchronous, that is why I didnt use it.
I'd like to offer you adding some async deserialization to your code. The example is bellow:
import struct
from typing import Any, Callable

from fastavro import schemaless_reader
from schema_registry.client import AsyncSchemaRegistryClient
from schema_registry.client.errors import ClientError
from schema_registry.serializers.message_serializer import MAGIC_BYTE, ContextStringIO

class DeserializationError(Exception):
    pass

class AsyncDeserializer:
    def __init__(
            self,
            schema_registry: AsyncSchemaRegistryClient,
            from_dict_to_model_converter: Callable[[dict[str, Any]], Any],
    ):
        self._schema_registry = schema_registry
        self._converter = from_dict_to_model_converter

    async def __call__(self, value: bytes | None) -> Any:
        if value is None:
            raise DeserializationError("Empty message")

        if len(value) <= 5:
            raise DeserializationError("Message is too small to decode")

        with ContextStringIO(value) as payload:
            magic, schema_id = struct.unpack(">bI", payload.read(5))
            if magic != MAGIC_BYTE:
                raise DeserializationError("Message does not start with magic byte")

            try:
                writer_schema = await self._schema_registry.get_by_id(schema_id)
            except ClientError as e:
                raise DeserializationError(f"Unable to fetch schema with id {schema_id}") from e

            if writer_schema is None:
                raise DeserializationError(f"Unable to fetch schema with id {schema_id}")

            decoded_value = schemaless_reader(payload, writer_schema.schema)
            decoded_value["schema_name"] = writer_schema.schema["name"]
            decoded_value["schema_id"] = schema_id

            return self._converter(decoded_value)

Thank you for your library, hope you will find this interesting and useful!

@akolenko-tcs hi! Thank you for your kind words and suggestion, but I am not sure what are you talking about. We have no built-in avro support. We just provides you with an ability to use any custom serialization you want. Example you refer is just an example, no more. So, I don't know, that we should add any special code to support Avro - it conflicts with our ideology
Please, reopen the Issue if I understood you wrong

Hello again! I just wanted to ask you to add support for async deserializers. The example I provided just shows that call is async


add table entries for multiple switches.

Hi, I am trying work with a three switch topology. I know how to add table entries for one switch using the ./run_add_entries.sh script. I understand that it uses a port number to identify a switch which is 22222 by default. I created the other switches with port number 22223 and 22224 respectively. When I set the port number to 22223 using the '-c' tag, this error shows TTransportException: Could not connect to any of [('<IP_ADDRESS>', 22223)] can anyone please help ?

This repo is mostly deprecated, so I wouldn't recommend using it anymore. Which P4 program are you trying to run?

I'm trying to implement ECMP using p4.

My advice is to look at this P4 tutorial if you haven't yet: https://github.com/p4lang/tutorials/tree/master/workshop_05_2016/ecmp
In any case, you do not need this repo, you should use bmv2 directly: https://github.com/p4lang/behavioral-model
If you need help with something specific, please send an email to the p4-dev mailing list (http://lists.p4.org/mailman/listinfo/p4-dev_lists.p4.org)

I did look at the ecmp workshop to have an idea about how I should go about it, however I want to evaluate the performance of ecmp on custom topologies which, I think, isn't possible with the workshop example.

You should adapt the P4 program to your own needs. However I would recommend using the same tools as the tutorial (bmv2, bmv2 CLI, Mininet topology script). The code in p4factory is hardly maintained any more.

Thank you so much for your suggestion, I guess I will look towards bmv2 since I've hit a dead end here.


Focus style animation inconsistent across the site

Description
The focus style animation when browsing with a keyboard changes from animated to static. I do not why and exactly where this occurs but it affects consistency and feels odd.
Reproduction

Search for anything.
Click on the search bar in the header while being in search results.
Browse with your keyboard, forward and backward.
See transition between focus styles between elements.

Screenshots

Resolution

[ ] 🙋 I would be interested in resolving this bug.

@panchovm, should we change the animated or static styles consistently?

I am drawn to static styles for all focus.