Double-click on JAR doesn't launch the application

When you double-click on a JAR the application it contains doesn't launch.
This is different behaviour compared to the Oracle JDK/JRE. With this version this works.
Tested on Windows 10 x64

Was the Oracle JDK installed with an actual installer (MSI/EXE)?

Which Adopt binary did you try this with?

Oracle JDK was installed via the EXE, removed via Add/Remove programs
Downloaded Adopt Binary was : OpenJDK8U-jdk-x64_windows_hotspot_8u181b13.zip
Think it's clear because it's not an exe/msi that some things are not registered. Think I got the ZIP file just because it was the default one presented on the page. When the installer is ready to be released it would probably be  good to default the installer to download or at least show it aside the zip file

Cool, yeah hopefully this is an installer issue that we can quickly address.

(Thought I'd replied to this but apparently not)
Yeah I would expect that to be just because we're a ZIP and we're not performing any postinstall tasks to set up associations. I suppose we could provide a postinstall script to do it but if we get our installers production ready we can stuff like this in the installer instead


"Display Output Not Active" Error + Misleading UTM Documentation on SPICE Tools.

Hi, everyone!
I made sure, that there is no exact same issue before posting it. Also, I really think, that fixing that is a major deal.

Machine: Apple MacBook M1 Pro 16'' 16Gb 2021
MacOS Sonoma 14.7
UTM Version 4.6.2 (104)
Windows 11 (straight from Microsoft website)
RAM assigned to VM: 12Gb
HDD assigned to VM: 128Gb
Ticked 'Installed SPICE Drivers', while creating a new VM too.
Emulated Display Card: virtio-ramfb-gl (GPU Supported) / default setting

Issue One: After installing Windows 11, it immediately offers you to install UTM Guest Tools. However, after 60-70% of installation complete, suddenly the screen get black and it returns error "Display Output Not Active". Sadly, there is nothing to do with the screen being black and reboot did not help, neither did changing the Emulated Display Card in VM Hardware settings.
I tried the whole procedure again, but got the exact same result.
On the 3rd attempt I noticed, that the version of UTM Guest tools is different from the knowledge base on UTM website!
Issue Two: Out of the box it offers you to install the drivers from the file called E:\utm-guest-tools-0.229.exe, but the link from the documentation (namely https://github.com/utmapp/qemu/releases/download/v7.0.0-utm/spice-guest-tools-0.164.4.iso is leading to the file called spice-guest-tools-0.164.4.iso all the way from QEMU v7 Assets!
The problem is still not solved, by the way. I installed Windows 11 for the 3rd time and now I will be finding a way to, at least, install network drivers separately and try to use the VM.
Unfortunately, there are no crash logs available, since the screen of the VM get ALL BLACK and I have no idea how to extract it from dead VM. However, I am ready to provide any additional information on the matter.
UPDATE: installing drivers from spice-guest-tools-0.164.4.iso — leads to the exact same error.
UPDATE 2: I have managed to boot the VM back by changing Emulated Display Card -> virtio-ramfb-gl (GPU Not Supported).

Duplicate of #6332


update rewrite functions in imports.rs

Tracked by #6206
Description
This pr implements rewrite_result method for UseTree, and UseSegment.

Running the Diff-Check Job


Support for PostgreSQL SKIP LOCKED

This code allows for SKIP LOCKED to be used in the postgres dialect.
I had to upgrade some libraries for the gradle build to run green on my machine, these upgrades are also included here.
Please let me know if something here isn't up to your standards, and I'll try to fix it.

As skip locked is always used in a for update clause, would it be better to modify the existing forUpdate functions to support the following usage?
query.forUpdate(skipLocked = true)

Generated SQL:
select ... for update skip locked

It will be great if can also support nowait:
query.forUpdate(nowait = true)

Generate SQL:
select ... for update nowait

And:
query.forUpdate(wait = 3)

SQL:
select ... for update wait 3

To implement this, we can deprecate the forUpdate property in SelectExpression, replacing its type as a structured data class instead of Boolean.
What do you think?

As skip locked is always used in a for update clause, would it be better to modify the existing forUpdate functions to support the following usage?
query.forUpdate(skipLocked = true)

Generated SQL:
select ... for update skip locked

It will be great if can also support nowait:
query.forUpdate(nowait = true)

Generate SQL:
select ... for update nowait

And:
query.forUpdate(wait = 3)

SQL:
select ... for update wait 3

To implement this, we can deprecate the forUpdate property in SelectExpression, replacing its type as a structured data class instead of Boolean.
What do you think?

(...)

To implement this, we can deprecate the forUpdate property in SelectExpression, replacing its type as a structured data class instead of Boolean.
What do you think?

Having parameters in this way does sound better, yes!
Up until the last paragraph though, I was thinking of just overloading the forUpdate method in the Postgres-module with extra parameters, but then you wrote that about a structured data class - I'm not entirely sure if I understand what you mean by that in this context?

(...)

To implement this, we can deprecate the forUpdate property in SelectExpression, replacing its type as a structured data class instead of Boolean.
What do you think?

Having parameters in this way does sound better, yes!
Up until the last paragraph though, I was thinking of just overloading the forUpdate method in the Postgres-module with extra parameters, but then you wrote that about a structured data class - I'm not entirely sure if I understand what you mean by that in this context?

I mean it would be better if this PR can implement more for update options like nowait, wait, not just skip locked. We can modify the SelectExpression like this:
public data class SelectExpression(
    val columns: List<ColumnDeclaringExpression<*>> = emptyList(),
    val from: QuerySourceExpression,
    val where: ScalarExpression<Boolean>? = null,
    val groupBy: List<ScalarExpression<*>> = emptyList(),
    val having: ScalarExpression<Boolean>? = null,
    val isDistinct: Boolean = false,
    val forUpdate: SelectForUpdateOptions = SelectForUpdateOptions.None,
    override val orderBy: List<OrderByExpression> = emptyList(),
    override val offset: Int? = null,
    override val limit: Int? = null,
    override val tableAlias: String? = null,
    override val extraProperties: Map<String, Any> = emptyMap()
) : QueryExpression()

public sealed class SelectForUpdateOptions {
    public object None : SelectForUpdateOptions()
    public object SkipLocked : SelectForUpdateOptions()
    public object NoWait : SelectForUpdateOptions()
    public data class Wait(val seconds: Int) : SelectForUpdateOptions()
}

Don't hesitate to ask me if you have any other questions.

I mean it would be better if this PR can implement more for update options like nowait, wait, not just skip locked. We can modify the SelectExpression like this:
public data class SelectExpression(
    val columns: List<ColumnDeclaringExpression<*>> = emptyList(),
    val from: QuerySourceExpression,
    val where: ScalarExpression<Boolean>? = null,
    val groupBy: List<ScalarExpression<*>> = emptyList(),
    val having: ScalarExpression<Boolean>? = null,
    val isDistinct: Boolean = false,
    val forUpdate: SelectForUpdateOptions = SelectForUpdateOptions.None,
    override val orderBy: List<OrderByExpression> = emptyList(),
    override val offset: Int? = null,
    override val limit: Int? = null,
    override val tableAlias: String? = null,
    override val extraProperties: Map<String, Any> = emptyMap()
) : QueryExpression()

public sealed class SelectForUpdateOptions {
    public object None : SelectForUpdateOptions()
    public object SkipLocked : SelectForUpdateOptions()
    public object NoWait : SelectForUpdateOptions()
    public data class Wait(val seconds: Int) : SelectForUpdateOptions()
}

Don't hesitate to ask me if you have any other questions.

I mean it would be better if this PR can implement more for update options like nowait, wait, not just skip locked. We can modify the SelectExpression like this:

Okay, I see what you mean. The only thing I don't quite like about that solution is that "skip locked" is postgres specific - it doesn't really seem elegant to put explicit support for it in the main library, as I believe we'd have to do in this case. (Correct me if I'm wrong.) Nowait on the other hand exists in more dialects, oracle at least, but as far as I can see its not universal - SQL Server is missing it, as far as I can see. Although now that I'm trying to find it, I'm not sure SQL Server has any support for "for update" in the first place? (See: https://docs.microsoft.com/en-us/sql/t-sql/queries/select-transact-sql?view=sql-server-ver15 )
If indeed I'm correct in for update not being universal, I'm tempted to say the function should be moved from the main library into the relevant dialects, and given the appropriate parameters and behaviours for each DBMS there. And probably use the  extraProperties for representation in SelectExpression to avoid polluting that class with DBMS-specific code - although as it's behind the scenes, I could see an argument for keeping it more readable by putting fairly common things there.
I'll update the PR to reflect these thoughts if you agree.

I mean it would be better if this PR can implement more for update options like nowait, wait, not just skip locked. We can modify the SelectExpression like this:

Okay, I see what you mean. The only thing I don't quite like about that solution is that "skip locked" is postgres specific - it doesn't really seem elegant to put explicit support for it in the main library, as I believe we'd have to do in this case. (Correct me if I'm wrong.) Nowait on the other hand exists in more dialects, oracle at least, but as far as I can see its not universal - SQL Server is missing it, as far as I can see. Although now that I'm trying to find it, I'm not sure SQL Server has any support for "for update" in the first place? (See: https://docs.microsoft.com/en-us/sql/t-sql/queries/select-transact-sql?view=sql-server-ver15 )
If indeed I'm correct in for update not being universal, I'm tempted to say the function should be moved from the main library into the relevant dialects, and given the appropriate parameters and behaviours for each DBMS there. And probably use the  extraProperties for representation in SelectExpression to avoid polluting that class with DBMS-specific code - although as it's behind the scenes, I could see an argument for keeping it more readable by putting fairly common things there.
I'll update the PR to reflect these thoughts if you agree.

Yes, for update is not universal, it was a mistake implementing the forUpdate functions in the core library. I think we can deprecate the legacy code and reimplement them in dialect modules. The only pain is we have to copy-paste some code between dialects, but it is acceptable, this time only supporting MySQL and PostgreSQL is enough.

Yes, for update is not universal, it was a mistake implementing the forUpdate functions in the core library. I think we can deprecate the legacy code and reimplement them in dialect modules. The only pain is we have to copy-paste some code between dialects, but it is acceptable, this time only supporting MySQL and PostgreSQL is enough.

You might need two MySQL dialects to support this.
Both 5.7 and 8.0 have two locking modes, share and update, but the syntax is different.
5.7 share: SELECT ... LOCK IN SHARE MODE
5.7 update: SELECT ... FOR UPDATE
8.0 share: SELECT ... FOR SHARE
8.0 update: SELECT ... FOR UPDATE

You might need two MySQL dialects to support this.

Hm. Is there any prior work in Ktorm where there is support for several versions of a DBMS' dialect?

I can think of two ways to do it that don't require connection info. Either publish two different ktorm-support-mysql modules, or require that a env var/Java System Prop has been set to specify the MySQL major version.
Or it could be lazy and require that the SqlDialect gets a DataSource and then do something like:
dataSource.connection.use { connection ->
    when (val databaseMajorVersion = connection.metaData.databaseMajorVersion) {
        5 -> {
            // MySql5Dialect
        }
        8 -> {
            // MySql8Dialect
        }
        else -> throw UnsupportedMySqlVersion(databaseMajorVersion)
    }
}

I took a stab at it here:
https://github.com/efenderbosch/ktorm/tree/forUpdate
Feel free to use/modify that.
I only did it for MySql 5, MySql 8 and Postgres, no other dialects. Oracle and SqlServer will need their own implementations.

Yes, we already have productVersion in Database. We can use it in MySqlFormatter when generating the SQL.

Updated my branch to use productVersion

Should I submit a separate PR?

@efenderbosch Yes, please create another PR.

I'm sorry about dropping the ball here, life got chaotic. My understanding is that efenderbosch's PR will now solve this ,so I should just close this PR`?

Close this PR since #247 was merged.


Allow application creation in sub organization level

Purpose

$subject
Improvement for : https://github.com/wso2/product-is/issues/21208
Allowing the application creation in the sub organization level through the DCR endpoint when accessing from the tenant perspective.

add unit tests


Link sales entries to original URL in listing

Let the user navigate from the summary page to the raw listing page. In this way, we can see if the plane remains for sale.

URL navigates to Trade-a-Plane detail listing page.


_.random can return double sometimes

_.random = function(min, max) {
	if (max == null) {
		max = min;
		min = 0;
	}
        // if min is not integer, we can not get integer as return
	return min + Math.floor(Math.random() * (max - min + 1));
}

_.random(5.1, 6) then we will get double not integer return. I think it is a bug because the doc told me I can get a random integer as return

min and max should probably be rounded before applying the formula

I created a pull request for this, see #2696


Feature/dx 1068 sidebar exclude

Uses the pre-exisiting sidebar_hidden to hide pages from the sidebar

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- _plugins/sidebar.rb  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- _plugins/sidebar.rb  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- _plugins/sidebar.rb  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- _plugins/sidebar.rb  1

See the complete overview on Codacy

@asbjornu I can't merge because there are 1681 errors reported in the verify step.


Arrow on the left panel doesn't expand/shrink the tag routes

Expected Behavior
When I click on the arrow next to a tag text on the left panel, it should expand/shrink the tagged routes
Actual Behavior

Clicking on the arrow pointing to the right does nothing
Reproduction steps
Click on the arrow next to a tag text on the left panel.

Link to site / OpenAPI Spec
No response
Version
latest
What browsers are you seeing the problem on?
Chrome

@thelinuxlich great catch! this must have been introduced as a regression, will make this an issue and get it fixed asap


add sepaxml & fints

Motivation for this change
python modules sepaxml & fints missing in nixpkgs
Things done
added the modules

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python3.pkgs.fints

I've added the doCheck options and also disabled them in mt-940. (I've had them disabled there locally before)

There is a new PR #52578


use func image.parse_normalized_name to identify docker images

Overview

@YiscahLevySilas1 Any updates here?

@YiscahLevySilas1 Any updates here?

Not working properly with the function from kubescape, checking tomorrow with @Daniel-GrunbergerCA if the problem is in the rego or in kubescape

@YiscahLevySilas1 Any updates here?

Not working properly with the function from kubescape, checking tomorrow with @Daniel-GrunbergerCA if the problem is in the rego or in kubescape

@dwertent problem was in rego, fixed and checked


navigation hotkeys don't work with escaped screen

Recreate:

Activate any shabad
Pick any line that is between first and last lines
Escape
up/down, left/right, j/l, tab/shift-tab do not work (fix this)
enter, 1,2,3,..., space do work

What should these keys do? Activate the last line?

They should activate the line that would've been if the screen hadn't been cleared. So up, left, and j would move you to the preceding line of the one that was escaped (if exists, else activate same line since it's the first most line).

Related #325


🍒 Cherry pick PR #12830 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/12830 to staging 🍒

🚀 Cherry-picked to staging by @OSBotify in version: 1.2.29-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.


oras: 0.14.1 -> 0.15.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/oras-project/oras/releases.
meta.description for oras is: "The ORAS project provides a way to push and pull OCI Artifacts to and from OCI Registries"
meta.homepage for oras is: "https://oras.land/"
meta.changelog for oras is: "https://github.com/oras-project/oras/releases/tag/v0.15.0"
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.15.0 with grep in /nix/store/szjw7y7r4d60x32bvqqyvy9gpv05jiwg-oras-0.15.0
found 0.15.0 in filename of file in /nix/store/szjw7y7r4d60x32bvqqyvy9gpv05jiwg-oras-0.15.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
oras

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/szjw7y7r4d60x32bvqqyvy9gpv05jiwg-oras-0.15.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A oras https://github.com/r-ryantm/nixpkgs/archive/02b4100621013a969216d6bcac1ac95b8b5495d2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/szjw7y7r4d60x32bvqqyvy9gpv05jiwg-oras-0.15.0
ls -la /nix/store/szjw7y7r4d60x32bvqqyvy9gpv05jiwg-oras-0.15.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

oras

Maintainer pings
cc @06kellyjac for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/oras-project/oras/releases.
meta.description for oras is: "The ORAS project provides a way to push and pull OCI Artifacts to and from OCI Registries"
meta.homepage for oras is: "https://oras.land/"
meta.changelog for oras is: "https://github.com/oras-project/oras/releases/tag/v0.15.1"
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.15.1 with grep in /nix/store/k0y19p53kzqbpx3447i89c7drin35rfk-oras-0.15.1
found 0.15.1 in filename of file in /nix/store/k0y19p53kzqbpx3447i89c7drin35rfk-oras-0.15.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
oras

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/k0y19p53kzqbpx3447i89c7drin35rfk-oras-0.15.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A oras https://github.com/r-ryantm/nixpkgs/archive/2b138fed5b545770f8febe5619aed101c50ead5c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/k0y19p53kzqbpx3447i89c7drin35rfk-oras-0.15.1
ls -la /nix/store/k0y19p53kzqbpx3447i89c7drin35rfk-oras-0.15.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

oras

Maintainer pings
cc @06kellyjac for testing.


Idea: "task" rules that are half-way between file rules and phony rules

File-based rules run when dependencies change or when the file is missing. Phony rules don't produce a file and always run. I would like to have something in the middle: a rule that does not produce a file, but only runs when its dependencies change. Of course, this behavior can be achieved with temp-files. But there is overhead to get that working properly, especially if you want these "task" based rules to match FilePatterns just like file-based rules do.
This might be a bad idea or it might not fit with the goals of the project. That's ok. I just wanted to propose it in case it was useful. I've achieved this with temporary files and it's working well.

See https://github.com/ndmitchell/shake/pull/389#issuecomment-172022265 for notes on how to combine oracles and Rule rules, thus fixing this whole issue (nothing in between, since they are the same).

There is now addOracleCache which does this task rule. I think that nails exactly what you were after, but let me know if not.


Use only an array ("sources") to define inputs for the build context

In the Remote Artifacts EP [1], our accepted API contains two fields related to the input context (workspace) for builds:

"source", which is a git repository
"sources", which can be anything else

As a follow up, we should consolidate such that all inputs for the build context live in the "sources" array.
[1] https://github.com/shipwright-io/build/blob/master/docs/proposals/remote-artifacts.md#single-vs-multiple-git-repositories

Agreed, this would make our API a lot cleaner.

Agreed, this would make our API a lot cleaner.


Retrieve Landmarks

This PR aims at retrieving the data for the face landmarks when using the face mesh graph.
Currently half-works as it is fine in the beginning, but aborts a ton of times due to stack smashing after a while.

I have no clue, but I indeed definitely don't want to leave the program as-is.

I think now that the PR is ready to be merged. After it gets merged, I intend to work on cleaning the repository from binary files and improve the structure a bit.

Yeah... I have no idea why but a test didn't pass there.


Landing Page Footer Enhancement: Professional Icon Alignment, Shapes, and Hover Effects

I want to enhance the footer of the landing page by implementing the following improvements:
Mail Input Box: Redesign for better usability and visual appeal.
Connect With Us Icons: Align and update the icons for a more professional look, adding hover effects for improved interactivity.
Animations: Introduce animations for all elements to create a more dynamic user experience.
Copyright Heading: Change the copyright heading to a more professional style.
I am adding an image to illustrate the expected changes, providing a clear visual representation of the enhancements.
These updates will contribute to a more polished and engaging footer on the landing page.
current

proposed changes with effects and responsive

@vanshchauhan21   Please assign me under GSSoC'24 ext and hacktoberfest labels

I want to enhance the footer of the landing page by implementing the following improvements:
Mail Input Box: Redesign for better usability and visual appeal.
Connect With Us Icons: Align and update the icons for a more professional look, adding hover effects for improved interactivity.
Animations: Introduce animations for all elements to create a more dynamic user experience.
Copyright Heading: Change the copyright heading to a more professional style.
I am adding an image to illustrate the expected changes, providing a clear visual representation of the enhancements. These updates will contribute to a more polished and engaging footer on the landing page.
current
proposed changes with effects and responsive
@vanshchauhan21 Please assign me under GSSoC'24 ext and hacktoberfest labels

I will assign you all the best 😌

can you please assign level ? @vanshchauhan21

thanks for assigning!. can you please assign level ? @vanshchauhan21

Yes sure

thanks for assigning!. can you please assign level ? @vanshchauhan21

Yes sure
Can resolve the issue which I open ? Only chatbot are showing in project please fix this issue


Update API Go SubmitMetrics Example

What does this PR do?
Currently if a customer uses the SubmitMetrics example the metric will not show in their account as there is no timestamp being included in the Points (required). This update includes a static timestamp in the Points. I have tested this out in my own org and confirmed it works.

Ticket: https://datadog.zendesk.com/agent/tickets/542640
Plus found many more tickets where customers had an issue using this example
Additional Notes

Review checklist
Please check relevant items below:

[ ] This PR includes all newly recorded cassettes for any modified tests.

[ ] This PR does not rely on API client schema changes.

[ ] The CI should be fully passing.

[ ] Or, this PR relies on API schema changes and this is a Draft PR to include tests for that new functionality.

Note: CI shouldn't be run on this Draft PR, as its expected to fail without the corresponding schema changes.

Hi,
Sorry this is a generated file, so we can't make a manual change to it. I'll look to see how we can improve it. Thanks.


Move clobber block to android platform

before the block was global, so the plugin was instantiated on all platforms.
Now it will only be instantiated on Android.
This also changes how the plugin exposes itself to Cordova in a more standard way using module.exports.
Also moved the plugin from  window.plugins  to the more
canonical cordova.plugins namespace.

Merged.
Will publish a new version to npm later.


Support Custom AI Providers and Allow Inputting API Key

Description of the Issue:
Currently, the application only supports selecting predefined AI providers and does not allow users to input the API key for third-party custom providers. This limitation restricts users from utilizing their preferred AI service providers. Therefore, I suggest adding the following features:

Support Custom AI Providers: Allow users to input and select AI providers that are not in the predefined list.
Allow Inputting API Key: When adding a custom AI provider, provide an input field for users to enter and save the API key for the third-party provider.

Expected Behavior:

Add an option in the settings page or configuration file to allow users to add custom AI providers.
Provide an input field for users to enter the API key when they add a custom AI provider.
Validate the entered API key and ensure the configuration is saved successfully.
When utilizing the AI service, the application should correctly call the user-provided custom provider and API key.

Actual Behavior:
Currently, the application only supports selecting predefined AI providers and does not provide an option to input the API key.
Additional Information:

Affected Version(s): All
Screenshot of Current Settings Page (if available):

(Please add a screenshot or relevant information of the current settings page here to help developers better understand the issue.)
Priority:
Medium
Labels:
enhancement, feature request

Hey @jerryokk, thanks for raising the issue. Indeed we have no way of using 3rd party providers that aren't local yet.
Is there a specific provider/model combination that you were hoping to use?

Hey @jerryokk, thanks for raising the issue. Indeed we have no way of using 3rd party providers that aren't local yet.
Is there a specific provider/model combination that you were hoping to use?

yes, I want to use a third-party cloud service compatible with OpenAI's API.


Return a proper error message when importing an existing service in Service Catalog

Describe your problem(s)
A 500 - Internal Server Error is thrown when importing a service with the same name and version as an existing one. The error message could be modified to have more context about the issue.

Describe your solution
Change the error message to a status code 400 - Bad Request and add "A service with the same name and version already exists" to the description.
How will you implement it
Figure the error throwing point and return a custom error.

We can fix it to return 409 Conflict response in this case.

When deleting a service with invalid UUID, it returns 204 as the response instead of 404. It should be corrected as 404 - Bad Request.

When retrieving list of services by invalid definitionType, it returns an empty list with 200 status code. It should be corrected as 404-Bad Request

.

When retrieving services with invalid sortBy or sortOrder values, it returns a response with status code 500. It should be corrected to a status code 400-Bad Request

When exporting a service with non-existing name or version, it doesn't show a proper error message. Should display an error message with status code 404 - Not Found and add "Requested resource with name 'Pizzashack-End' not found" to the description.

@malinthaprasan can you review these suggestions, please?

When deleting a service with invalid UUID, it returns 204 as the response instead of 404. It should be corrected as 400 - Bad Request.

Shouldn't this be a 404? The resource you are accessing is not available.


Managed mirrored speak animations in ESCItem

Fixes https://github.com/godot-escoria/escoria-issues/issues/155
cc @balloonpopper

Will review once changes are made.

@StraToN As I was reviewing this again I thought about whether we should change pose_scale to is_mirrored as a boolean, because it would make much more sense IMHO. What do you think?

Did you forget to push the change? I still see pose_scale

@BHSDuncan I have checked the necessity to add null checks but I think it's not useful at the moment. We have to assume that the ESCAnimationResource is correctly set up by gamedev, ie. animations filled for every created direction.
We will be able to ensure this by providing a visual tool in the plugin to create that resource.
TLDR: no new modification to this PR, please review :)

@BHSDuncan I have checked the necessity to add null checks but I think it's not useful at the moment. We have to assume that the ESCAnimationResource is correctly set up by gamedev, ie. animations filled for every created direction. We will be able to ensure this by providing a visual tool in the plugin to create that resource. TLDR: no new modification to this PR, please review :)

I'm a fan of "defence in depth" but I also agree there has to be a limit.


test: fix flaky test case test_recovery_from_im_deletion

Which issue(s) this PR fixes:

Issue https://github.com/longhorn/longhorn/issues/7750
What this PR does / why we need it:
Make timeout longer to reduce the failed rate.
Special notes for your reviewer:
Additional documentation or context

@mergify backport v1.6.x v1.5.x v1.4.x


Update ML Create workspace

Update to create workspace article for Azure machine learning.

Hi @ahgyger, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (ahgyger). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@ahgyger - Please resubmit these changes to the private repo - see my email for more details.


there is no "${logstash::home_dir}/bin/system-install" with logstash 2.2.4

I am using this module, but fails, because ${logstash::home_dir}/bin/system-install doesn't exist

I'm afraid the current of the module does not support Logstash 2. You can still get the legacy module on Puppet Forge if you need it. If at all possible though, I encourage you to upgrade to Logstash 5 or 6 and use the latest module.

But if we check the logstash version used is 2.2, we can skip the execution of system-install

There's a lot more to it than that. The reason that script exists in Logstash 5 is because Logstash 5 introduced a completely new init system. Logstash 2 didn't actually support systemd at all, and simply relied on backward compatibility support for SysV Init scripts. The changes were so wide reaching that maintaining support for both versions in the same Puppet module version proved extremely difficult. It was so difficult in fact, that we made the decision to only support Logstash 5 in the version 5 module, and to retain the old module in a legacy state for people using Logstash 2.
All of that is quite historical now anyway, since Logstash 2 itself has been end-of-life for some time and is unsupported in general.


Alignment tool is broken

I can't get the alignment tool to render the alignment.  Here is what it looks like:

https://www.patricbrc.org/portal/portal/patric/MSA?pk=-2181953846986415073

Note that the statistics all look okay.

Hm, it worked for me. Can you bring up the javascript console and see if there is an error there?

@jimdavis1 can you give the feature ID's you were using here? the link doesn't work past a certain amount of time or maybe per session.

This should be working now. Let me know if you have any more problems.


Added Plant Care Calculator

Fixes Issue🛠
Closes #1607
Description👨💻
This calculator calculates schedule for aspects like watering, fertilizing and repotting plants based upon the plant size, season, location and water frequency.
Type of Change📄

[ ] Bug fix (non-breaking change which fixes a bug)
[x] New feature (non-breaking change which adds functionality)
[ ] Style (non-breaking change which improves website style or formatting)
[ ] Documentation (non-breaking change which updates or adds documentation)
[ ] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

https://github.com/user-attachments/assets/ec857c9f-6006-48b5-86fa-dd9ecc1a8791

Hi, @anagha-chaudhari What's use of plant type here?

@Nihal4777 here plant type intended to indicate what kind of plant it is, for eg: a rose plant, tulsi plant, snake plant etc

@Nihal4777 @Rakesh9100 Please review the pr

@Nihal4777 here plant type intended to indicate what kind of plant it is, for eg: a rose plant, tulsi plant, snake plant etc

Does it have any effect on result?

@Nihal4777 The calculator gives results based upon size, location, season and water-frequency. I have used the plant type field to know the type/name of the plant

Please review @Nihal4777 @Rakesh9100

@anagha-chaudhari Don't worry, @Rakesh9100 will accept PR after reviewing it.


switching between AEM instances causes problems

the aem extension for brackets currently only allows to configure 1 aem instance. This can cause problems when switching between projects. At startup, brackets deploys to the running AEM instance but since that may be not the instance you want to work with at the moment or the project you had open in brackets before you closed it your AEM instance gets unwanted code deployed.
It would be great if

brackets is able to handle multiple AEM configurations at a time
when brackets is started up it checks if the AEM instance it tries to connect to is the same one as it was connected before

I think the root problem is a combination of #76 and #81. <shame>Not sure if I should ask you if this is still relevant after 2 years</shame>.


ConnectAsync() is blocking Task - Tmds.DBus need update ?

await device.ConnectAsync();
await device.WaitForPropertyValueAsync("Connected", value: true, timeout);'

Many time, line 2 will never be invoked when device can't connect (because it is out of range, for example).
In fact, the code execution is blocked on the first line forever and no exception is thrown.
The only indication I get is the following exception catched somewhere out of my code:
Exception thrown: 'Tmds.DBus.DBusException' in System.Private.CoreLib.dll
I noticed that current version of Tmds.DBus is 0.14.0 and you use 0.11.0. Maybe an update may fix the issue. What do you think about that ?

Upgrading to Tmds.DBus v0.15 👍
A new release will be out shortly


Workspace Templates: Archive Template

Archive Workspace Template
Pre-conditions
All related workspaces are terminated
You should not be able to delete a workspace template if there are any workspaces left alive. That is, you can only delete if all workspaces for that template have a phase of Terminated
Post-conditions
DB resource is archived

[x] workspace_templates

k8s resources are deleted

[x] workflow templates
[ ]  workflow executions <---- remaining

Initial testing of this, I deleted a workspace template and tried to reuse its name when creating a new one and was not able to. The UI didn't show any indication of an error but had this in the API logs:
INFO[0062] finished unary call with code AlreadyExists   error="rpc error: code = AlreadyExists desc = Workspace template already exists." grpc.code=AlreadyExists grpc.method=CreateWorkspaceTemplate grpc.service=api.WorkspaceTemplateService grpc.start_time="2020-05-19T19:52:09-07:00" grpc.time_ms=111.32 span.kind=server system=grpc


The Journey to a Faster Release Process

During the July 2021 TSC, the notion of moving towards quicker releases came up. This issue is to track a conversation around some of the ideas and clarify a path forward.
Things that could speed up releases:

More frequent releases e.g. can move towards a release strategy that can quickly release updates/patches/bugfixes/defects that are low risk to release to master?
Maintain the existing release process for higher risk updates (OS changes, core software updates (Mongo/Rabbit, etc)) while enabling smaller fixes to get out quickly?
What fluff can we remove from the release process to enable faster releases that are low risk?
How might the versioning strategy change? (e.g. minor (e.g. v2.6) is bumped at significant releases, and allow small features into patches? Or alternatively, bump minor version more frequently?
What else is blocking faster releases?

At the moment I would say that a large amount of time on the release process is spent on the manual testing states. Of which a key part is the UI where we don't have automated tests. In the process we perform the manual testing process on the staging-unstable, staging-stable and stable releases and that takes some time.

In regard to having low-risk and higher risk, then the only issue I can see is how we manage that in terms of branches. It would seem that we would have to keep the major ones off the master branch until we were near a release point, so that the master can be released more rapidly.
This will add overhead on merges between branches, and probably increase the number of CI runs we need to do - as we'd need to have the runs on both the "master" and "next-release" branches.

If we start having bug-fix releases, then it would also increase the number of upgrade paths, and if we have to increase the upgrade runs then that would maybe increase the release process. So if we have 3.5.0, 3.5.1, 3.5.2, 3.5.3 etc as the quick releases, then we release 3.6.0 - which upgrade paths do we test. We would presumably set a strategy here e.g. first and last maybe?
We may also want to have upgrade runs for the maintaince patches as well.
We do have upgrade workflows but we only run them at release time, I don't see why they couldn't be run more often to automate that, however that would increase our costs as we'd need the servers running for longer. Running them prior to release would be good anyway, as its a bit late in the day to find the upgrade doesn't work. As it stands at moment the upgrade is tested in the week of pre-release testing, so running the automated upgrade as part of the CI process would also cut down the amount of time spent in our pre-release testing. So perhaps an automated upgrade once a week for instance woudln't be too costly, and would help the release process.

I'm +1 for anything which allows us to release more often - we've been on that journey for a long time and trying to improve that process, but it appears that releases still need tons of hand holding and manual work.
I do think we should enable all the end to end tests and checks (including upgrade ones) to run as often as possible (e.g. daily).
Ideally after each merge to master we should have packages in staging stable which we should be able to release at any time.

One thing that would really help the release process is some Selenium or whatever tests for the frontend.
Our testing is pretty comprehensive, especially our end-to-end tests. But our end-user testing, especially just basic frontend functionality (logging in, logging out, logging in again, executing an action, viewing the results in the executions tab, switching to the rules tab, switching to the actions tab, creating a workflow in the editor, saving it, and executing it, etc.) is still done manually during the release cycle. If we automated that, it would drastically reduce the amount of pre-release testing down to just testing out code that has changed during the release. And as long as we add tests for changed code, that should also be covered.
We don't need to run the full tests for the workflow editor, just the incredibly basic ones.
So, what I would do, is setup frontend testing as part of the end-to-end tests, and expand the CLI tests that we do to cover what is currently manually done in the pre-release testing.
That will get us a lot closer to "push button" releases. And if we really wanted to stick to time based releases, we could just schedule that on st2cicd and let the process take care of itself.
Most of what slows down the release process is running tests on new OSes. When we don't introduce support for new OSes during releases, testing is tons easier to do.


linspace silently failed when using GPU without any runtime error

Problem
numpy.linspace does not work correctly and returns an all-zero array without throwing any runtime errors or warnings.
To reproduce
Option 1: using legate.numpy's own tests

step 1: go to legate.numpy's source code folder
step 2: run ./test.py --use cuda --gpus 1

Option 2:

step 1: create test.pyfrom legate import numpy
a = numpy.linspace(0.0, 4.0, 501)
print(a.mean(), a.sum())
print(a)

step 2: run the script with legate --gpus 1 ./test.py -lg:numpy:test

Resutls
When using legate.numpy's own test suite, the result show the test for linspace failed.
When using the test script in option 2, the result shows an all-zero array.
Expected results
I noticed linspace is not listed in the Legate NumPy API reference, so I think linspace belongs to the group that is not implemented yet? Then, in this case, the function should return NotImplementedError instead of just an all-zero array.

Update:
An interesting thing happened. The test.py from option 2 suddenly worked on my workstation (with an old K40 GPU and CUDA 10.2) but still failed on DGX-1 (V100 GPU + CUDA 11.2.1). Except for the CUDA version, legate.core and legate.numpy were built with the same flags. There are still some differences like gcc versions etc. Not sure if they are relevant. If more information is needed, please let me know.
Nevertheless, the test suite from legate.numpy (the option 1) still failed with both K40 and V100.


feat(services): add google cloudbuilder service

@pvdlg Any change you can give this repo some love?

Closing per https://github.com/pvdlg/env-ci/pull/97#issuecomment-540040921


Allow us to choose server as well

This way if Tera is launched we don't have to re-launch Tera to get the meter if it crashes or we forget to launch it.

That's not how it works.  It doesn't make a difference if you can choose server or not.

Is there no way to solve this then?

The only way is to keep some sniffer running all the time, capturing login packets and storing keys, so that newly started meter could read and use the latest one.


Same item in multiple sections: Possible?

I have a favorites section on top of the normal list. A favorite item appears in the top favorites section but also in the non-favorites section below. Can I model that using TLIndexPathTools? How? Specifically, I need something like
- (NSArray*)indexPathsForItem:(id)item

Items are never moved between those two sections, only added or deleted or moved within a section.

No, there's no API like that. You can wrap your item in a TLIndexPathItem and supply a unique ID for each one. For example, if your item has an id field, you could do something like this, creating a unique ID for the favorite item based on the id field:
NSString *favIdentifier = [NSString stringWithFormat: @"fav-%@", item.identifier]
TLIndexPathItem *favItem = [TLIndexPathItem alloc] initWithIdentifier: favIdentifier sectionName:@"Favorite List" cellIdentifier: nil data:item]];
TLIndexPathItem *item = [TLIndexPathItem alloc] initWithIdentifier:item.id sectionName:@"Normal List" cellIdentifier: nil data:item]];
dataModel = [TLIndexPathDataModel alloc] initWithItems: @[favItem, item]]

Then you'd look up index paths by item or by identifier:
NSIndexPath *indexPath = [dataModel indexPathForIdentifier: item.identifier]
NSIndexPath *favIndexPath = [dataModel indexPathForIdentifier: favIdentifier]

NSIndexPath *indexPath = [dataModel indexPathForItem: item]
NSIndexPath *favIndexPath = [dataModel indexPathForItem: favItem]

Does that do what you need?
p.s. I've got a new block-based cell configuration feature coming to TLIndexPathItem this week, which makes them pretty nice to work with.

Thanks, that makes sense.


Keep track of correction and resume correction

Description
When dealing with an important number of corrections, it is not currently possible to stop the script and resume right where we stopped (the solution used for most users was to keep the terminal open and the computer turned on until the end of the correction phase).
Solution
Since every path to the images should be in the YML config file, a simple solution would be to comment (using a #) after every correction the path to the image. Then when manual_correction.py is executed again the commented line will automatically be skipped.
However, this method should not be used when wildcards such as * are present in the YML path

Solution
Since every path to the images should be in the YML config file, a simple solution would be to comment (using a #) after every correction the path to the image. Then when manual_correction.py is executed again the commented line will automatically be skipped.

Excellent idea!

However, this method should not be used when wildcards such as * are present in the YML path

Good point! In this case, we could simply append the corrected images to the YML (to track which images have been corrected). For example:
FILES_LESION:
 - sub-*_acq-sag_T2w.nii.gz
#  - sub-001/anat/sub-001_acq-sag_T2w.nii.gz
#  - sub-002/anat/sub-002_acq-sag_T2w.nii.gz


Istio-Ingressgateway Responds with 503 upstream connect error on Routes that match /proxy

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities
To ask questions about how to use Istio, please visit https://discuss.istio.io)
Bug description
Istio Ingressgateway throws a 503 upstream connect error when matching a route with URI prefix set to "/proxy/". We have setup multiple matches on our virtualservice for various URI prefix's all routing to a single service. All the routes works fine except the one that we match using the prefix of "/proxy/". We assume that this might be due to some special meaning of /proxy/ assigned by the ingress gateway.
while debugging we were able to reach the service using the same exact request for /proxy/ from inside the cluster bypassing the ingress-gateway. The service responds with a 200 on that case. Only when the request reaches through the ingress gateway we are facing this issue.
The Logs from the istio ingress gateway corresponding to the request is given below :
[Envoy (Epoch 0)] [2020-12-11 04:28:16.851][32][debug][router] [external/envoy/source/common/router/router.cc:474] [C1678215][S1800260394623576323] cluster 'outbound|80|BLUE|fo-platform.fo-platform-dev.svc.cluster.local' match for URL '/proxy/v1/bit/run/example:hello.rb/sync/60000'

[2020-12-11T04:28:16.851Z] "POST /proxy/v1/bit/run/example:hello.rb/sync/60000 HTTP/1.1" 503 UC "-" "-" 2 95 30 - "<IP_ADDRESS>,<IP_ADDRESS>" "curl/7.54.0" "a6e77117-3145-4471-a6ee-40ee23fa0e8b" "somedomain.common.ouraccount.com" "<IP_ADDRESS>:3500" outbound|80|BLUE|fo-platform.fo-platform-dev.svc.cluster.local <IP_ADDRESS>:33040 <IP_ADDRESS>:80 <IP_ADDRESS>:25406 - -

The proxy config Routes file of our configuration is put in the below link for reference.
https://gist.github.com/Adharsh-Muraleedharan/d0b551a685eb4c6314f659bb511e6112

Please let us know how we could fix this ?
[ ] Docs
[ ] Installation
[X ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
The URI prefix match of /proxy/ should be routed to the service like all other requests.
Steps to reproduce the bug
Setup multiple virtualservice routes with one having URI prefix matching that of "/proxy/" and try to reach the service through the ingress gateway.
Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
Istioctl - 1.5.8
Kubectl Client Version: v1.15.11-eks-908ff6
Kubectl Server Version: v1.17.12-eks-7684af
we do not use HELM
How was Istio installed?
Istio was installed using istioctl manifest generate and then applying the manifest.
Environment where the bug was observed (cloud vendor, OS, etc)
AWS cloud we use managed service EKS.
Additionally, please consider running istioctl bug-report and attach the generated cluster-state tarball to this issue.
Refer cluster state archive for more details.

@ramaraochavali - Could you help us with this ?

Can you share full config dump and also the full envoy logs?

@ramaraochavali - The entire config dump is placed here
https://gist.github.com/Adharsh-Muraleedharan/0db5f49df843bc3abddac2f8a288284d

Sorry for the delay. Config seems correct. Can you also share complete Envoy logs? and can you also config outbound|80|BLUE|fo-platform.fo-platform-dev.svc.cluster.local this has endpoints populated?

Ping @Adharsh-Muraleedharan

Also see https://istio.io/latest/news/support/announcing-1.6-eol-final/

Closing as we don't have enough info. Thanks!


Update Issues Templates

There are currently no GitHub Issues templates that we can use to standardise our Issues creation.
Let's move to create Issues templates to ensure a consistent style of issues across the board to reduce confusion among developers.

Hi @okkhoy, I accidentally requested a review from you while selecting reviewers for this PR. So sorry for any inconvenience or confusion caused!


Text/Preview Options Missing in IntelliJ IDEA 15 EAP

both text & preview panes appear side by side in IDEA 15 EAP.
selection tabs are no longer there.

Installed plugin on latest EAP 143.249.6 and tabs are on their places:

@rbeaini what's wrong?

looks like getting the latest EAP solved this... thank you


[Improvement] support OSSSecretKeyCredential for python client

What changes were proposed in this pull request?
It adds support for OSSSecretKeyCredential for the Python client by implementing it, updating CredentialFactory to create instances of it, and adding corresponding unit tests in TestCredentialFactory.
Why are the changes needed?
These changes are necessary to support authentication using OSS credentials and to allow the CredentialFactory to generate OSS credentials correctly. It ensures proper functionality and integration.
Fix: #5880
Does this PR introduce any user-facing change?
No
How was this patch tested?
Unit tests were created to test the new OSSSecretKeyCredential class, verifying its functionality and integration with the CredentialFactory.

@deeshantk , merged to main, thanks for your contribution and welcome to the Gravitino community, hope you enjoy the journey.


UI: Layout Table

LayoutGrid but with column and row types

Per column/row alignment, equal, and full width/heights


mmdetection 2.22 vs 2.24 GPU memory usage

I found higher GPU memory usage for mmdetection 2.24 vs 2.22. I could manage to train batch size of 14 samples per gpu and now 10 samples per gpu. I attach the differences [left 2.22 right 2.24]:

I am not sure if I set correctly the mixed precision training, I guess yes.

If you turn on the fp16, it will be show in logs. The left fp16 was on, the right was not.

Hi @chhluo, thanks for the response. I rechecked the loss_scale flag again, the cmd parameter is fp16.loss_scale instead of loss_scale. Now it worked.

Hi, is this still true in v3.0?


added my name to guestbook.md

Thank you! Congratulations on making the 100th pull request! 🎉

It's an honor!


Multiple newlines replace with single newline outside of markdown

I've looked for an option but I don't see one. When I process my strings/messages, if the original has multiple newlines in a row, they are replaced with a single newline.
I would like to preserve those newlines outside of markdown. Is this intended? Is there a way to get around it? Glad to submit a PR if this could be added as a feature or flag...

In this regard markdown is like LaTeX, multiple whitespace is crushed into a single one. It just isn't in the philosophy of markdown to allow arbitrary whitespace.

@Feder1co5oave that's what I figured. I'd like to propose adding a feature to the marked lib that allows you to preserve whitespaces. I could always fork and use my own repo but I figure others might benefit from it as well. Thoughts?

Whether whitespace is compressed into a single new line should be the choice of the developer.

I do think it would be a nice option. I'm open to make the changes if pointed in the right direction.

One way to do this would be to create a new token of type: break in Lexer.prototype.token. Modify the current line break regex to parse to add these custom break tokens to this.tokens. From there just add this new break token to the switch statement in Parser.prototype.tok.

Closing for now as expected behavior based on the supports specs. See #956


Puppeteer needs to be updated to latest version

I was getting the following error
[GSL] Trying to login [GSL] Cookies are saved. [GSL] Landed on the target page. Trying to get list (node:49777) UnhandledPromiseRejectionWarning: Error: Execution context was destroyed, most likely because of a navigation.     at rewriteError (/Users/jszuslik/Personal/electron/mm2/MagicMirror/modules/MMM-GoogleShoppingList/node_modules/puppeteer/lib/ExecutionContext.js:161:15)     at process._tickCallback (internal/process/next_tick.js:68:7)   -- ASYNC --     at ExecutionContext.<anonymous> (/Users/jszuslik/Personal/electron/mm2/MagicMirror/modules/MMM-GoogleShoppingList/node_modules/puppeteer/lib/helper.js:110:27)     at ExecutionContext.evaluate (/Users/jszuslik/Personal/electron/mm2/MagicMirror/modules/MMM-GoogleShoppingList/node_modules/puppeteer/lib/ExecutionContext.js:48:31)     at ExecutionContext.<anonymous> (/Users/jszuslik/Personal/electron/mm2/MagicMirror/modules/MMM-GoogleShoppingList/node_modules/puppeteer/lib/helper.js:111:23)     at DOMWorld.evaluate (/Users/jszuslik/Personal/electron/mm2/MagicMirror/modules/MMM-GoogleShoppingList/node_modules/puppeteer/lib/DOMWorld.js:112:20)   -- ASYNC --     at Frame.<anonymous> (/Users/jszuslik/Personal/electron/mm2/MagicMirror/modules/MMM-GoogleShoppingList/node_modules/puppeteer/lib/helper.js:110:27)     at Page.evaluate (/Users/jszuslik/Personal/electron/mm2/MagicMirror/modules/MMM-GoogleShoppingList/node_modules/puppeteer/lib/Page.js:782:43)     at Page.<anonymous> (/Users/jszuslik/Personal/electron/mm2/MagicMirror/modules/MMM-GoogleShoppingList/node_modules/puppeteer/lib/helper.js:111:23)     at get (/Users/jszuslik/Personal/electron/mm2/MagicMirror/modules/MMM-GoogleShoppingList/node_helper.js:101:29) (node:49777) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2) (node:49777) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
I updated the puppeteer version to 1.15.0 and rebuilt. This resolved the issue.

Thanks for notice. I apply this info on package.json.


Add plan for all results figures

As title. As part of this work will need new issues to be created for the desired output figures

This might be good too address before we start knocking out prototypes

@SamuelBrand1 I think you are thinking about this now?

After some f2f chat with @seabbs, here is my plan for first three figures:

[ ] Fig 1: Overview: This fig aims at presenting the nowcasting (e.g. 0 horizon estimate) at determined inference time point for each scenario with each inference model choice but specialised to well specified generation intervals. Inference time point selection: end of biggest $R_t$ transition point in scenario with closest to middle of whole scenario breaking ties (e.g. smoothly endemic scenario). Plotting details: 4 rows corresponding to 4 main scenarios (e.g. outbreak with measures etc.), 3 cols corresponding to 3 main targets (e.g. exponential growth rate etc.). Actual values as scatter plot, including data forward of the determined inference time used. The posterior inferred values over times are plotted as ribbon plot quantiles with colour determining the inference model.
[ ] Fig 2: Overview: This fig aims at presenting the nowcasting (e.g. 0 horizon estimate) at rolling inference time points for each scenario with each inference model choice and possible misspecification of generation interval. Time horizon choice: Chosen horizon = 0 to align with Fig 1 but with other horizons as SI plots. Plotting details: 3 x 4 = 12 rows corresponding to 4 main scenarios (e.g. outbreak with measures etc.) and 3 main targets (e.g. exponential growth rate etc), the scenario GI is fixed to the middle mean GI (10 days; others are in SI) and 3 columns corresponding to underestimating mean GI (left), good estimation of GI (middle) and over estimating mean GI (right). Actual values as scatter plot. The posterior inferred value at the estimation date are plotted as boxplot plot quantiles[^1] with colour determining the inference model.
[ ] Fig 3: Overview: This figure concerns inference model scoring. Model score groups: scenario, inference model, horizon, inference date, and GI mis-specification (expressed as $\langle g \rangle - \langle G \rangle$ where $g$ is the used generation interval and $G$ is the scenario truth generation interval). Plotting details: TBD, first pass is the scoring data frame table. Second pass is some kind of stacked barplot.

[^1]: @seabbs finds it more common to plot a rolling nowcast as a ribbon plot, without strong favour to that approach. I think this blurs the idea of how new information also changes past inferences. I'd prefer to have a ribbon plot as representing the inference over time made by a model at some inference time $T$, and have a sequence of box plots as representing what the right now estimate of the model at a sequence of $T$ values. The idea here is to keep these similar, but distinct, concepts separate visually.

Nice. I edited 1. to reflect my understanding of what we discussed (that we would plot all horizons (+ and -) for a single estimation date). Do you agree or did you have a different takeaway?

No that makes sense, and was what I intended. I've done (bolded) edit for clarity.

Do you think we have this plan out/moved to their own issues sufficiently to close this?

Yes.


Binding failed on DependencyProperty of nullable types

Describe the bug
See the attached sample.
The following property cannot be bound with a two-way binding.
        public bool? IsChecked { get { return (bool?)GetValue(IsCheckedProperty); } set { SetValue(IsCheckedProperty, value); } }
        public static readonly DependencyProperty IsCheckedProperty =
            DependencyProperty.Register("IsChecked", typeof(bool?), typeof(MyControl),
                new PropertyMetadata(null, (d, e) => ((MyControl)d).OnIsCheckedChanged()));
        void OnIsCheckedChanged() {
        }

Steps to reproduce the bug

Run the attached sample
Click on a button
Check an error in output: "Error: Converter failed to convert value of type 'Windows.Foundation.IReference1<Boolean>' to type 'IReference1'; BindingExpression: Path='IsChecked' DataItem='App1.MyControl'; target element is 'Microsoft.UI.Xaml.Controls.Primitives.ToggleButton' (Name='null'); target property is 'IsChecked' (type 'Boolean')."

Expected behavior
The binding should work without errors.
Version Info
[WinUI 3 - Project Reunion 0.5: 0.5.6]

Windows app type:

UWP
Win32

Yes

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

October 2020 Update (19042)
Yes

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Xbox

Surface Hub

IoT

@teaP and @MikeHillberg (or maybe @jevansaks) do you recall the issues/work around here? similar to the issues with NumberBox's value property.

@teaP and @MikeHillberg (or maybe @jevansaks) do you recall the issues/work around here? similar to the issues with NumberBox's value property.

This isn't the same -- NumberBox.Value isn't nullable, it uses NaN as the unset value (which definitely caused OTHER binding issues because NaN != NaN).

But IIRC we wanted to use nullable for NumberBox, but couldn't because of this problem. I think there's an open Issue on this, that nullable isn't generally supported in binding and/or some markup scenarios, except for built-in WUX types.
We've worked around it for double types using NaN, int types using -1 or one of the infinities. I'm not sure how to work around it for a bool unfortunately; it doesn't have any magic values.

Is there any progress or workaround on this?

Requiring the use of a converter when the clear intention of the property is to use a boolean makes no sense!


Runtime Util.sequentialWrapperBoxed called by annotations for ints got renamed

And did not trip up a single test. I cleared the name for turning int[] into Sequential<Integer> for variadic methods and added sequentialWrapperBoxedCodepoints for int[] to Sequential<Character> but apparently it's supposed to be used by annotations interop and yet there were no tests.

Renaming that method would break BC, no?

I can definitely unbreak it and use another name, but is it really possible for it to have been called? I don't understand the circumstances in which it would be called, and there are no tests to show me.

Those methods are called when an annotation class has a Sequential<Integer> (or whatever) which we transform to long[] (or whatever). When instantiating the Ceylon class from the information in the annotation we need to convert the array to a sequential. Looking at the callers of RuntimeUtil.sequentialWrapperBoxed() would've told you where is was being called, and replacing the code in that method with something else would've shown you that the family of methods are being called in at least half a dozen tests. (Being able to easily figure out callers is why RuntimeUtil is quite to anal about reflecting the methods present on Util).
Anyway, I've pushed a test which shows that javac is not rejecting the incompatible method return types (not sure why), but that your change certainly breaks things at runtime (i.e. this test passes when using a Util from before your push).
I see you've also changed those methods to copy the array, which is not necessary for the annotations use case (but I assume it is for the varargs case you were working on).

I haven't changed those methods to copy the array, because they were boxing and so changes were already not reflected. I also haven't changed sequentialWrapper which does not copy, and added sequentialWrapperCopy to copy.

OK, I will rename my method then. Is that method called for annotation interop (with extra primitives like short) or just for Ceylon types?


support onnx export for mobilenet and shufflenet

#982

Codecov Report

Merging #983 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master    #983   +/-   ##
======================================
  Coverage    61.1%   61.1%
======================================
  Files          65      65
  Lines        5091    5091
  Branches      764     764
======================================
  Hits         3111    3111
  Misses       1769    1769
  Partials      211     211

Impacted Files
Coverage Δ

torchvision/models/mobilenet.py
92.3% <100%> (ø)
:arrow_up:

torchvision/models/shufflenetv2.py
85.54% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@houseroad does ONNX support (or plan to support) reductions over multiple axis?

Thanks for the PR btw!


Added custom cursor

Context (Problem, Motivation, Solution)
Silly little cursor update.
Describe Your Changes
Three types of cursors have been added.
Default - active if no other cursor is active
On-hover - active when hovering a component with a className that includes 'cursor-hover'
Clicked - active when left mouse button is down
Updated the modalElements base class to include 'cursor-hover' so now all cards and buttons should have this.
Placeholder art added. If to be changed, then paths need to be updated in globals.css.
File type must be PNG, SVG, GIF or CUR and dimension need be either 32x32 or 64x64 px.
Checklist

[x] I have performed a self-review of my code
[ ] I ran make check and fixed resulting issues
[ ] I ran the relevant tests and they all pass
[ ] I wrote tests for my new features, or added regression tests for the bug I fixed

Testing
Hovered some components

Cute! But there's definitely an easier way to do this.
Imho, it would be better to create a list of elements that we consider interactive (a, button ... something else?) and apply the cursor to them (in globals.css):
button, a {
  cursor: url('/path/to/hover-cursor.png'), auto;
}

button:active, a:active {
  cursor: url('/path/to/clicked-cursor.png'), auto;
}

Then we need to apply the normal cursor to the root of the app:
.root {
  cursor: url('/path/to/normal-cursor.png'), auto;
}

Also in globals.css, then wrap the app (in _app.tsx) in <div className="root">
I think this should provide the same feature that you've built (or better because I don't get the cursor on some of the index page buttons for instance) in a way that's simpler and doesn't require us to annotate every instance of something hoverable/clickable and potentially miss some.
That being said ... we will still need to annotate the cards with those modifiers + use a custom pointer for drag and drop (closed fist?), those might require some custom JS.
I'd say let's try if the simpler method above works, then sync with @ultraviolet10 to see how this could be applied there. I don't think it'd be good to have custom cursors on the home page but not on the play page =)
(Also, probably lower priority than the rest, but we can keep this open while we figure it out, I like the idea.)

Closing this for now since it's very out of date, but it's definitely something we can bring in later!


[BUG] Can't change guild bio in a new server

Describe the bug
I was inviting Annie to a new server, and noticed that I can't change the guild bio. It displays the "lang" error instead.
Edit: this happened for the first time, but it working fine on second try.
To Reproduce
Steps to reproduce the behavior:

Use >setGuildBio in command-enabled channel
See error

Expected behavior
I'm not sure how the command actually would response since I never use it before, but I assume it should give me the prompt to type the new guild bio.
Screenshots
If applicable, add screenshots to help explain your problem.

What's the bug? like does it still bug?
Usage is >setguildbio
no prompt was built in.


Update tapir-akka-http-server, ... to 1.6.1

About this PR
📦 Updates

com.softwaremill.sttp.tapir:tapir-akka-http-server
com.softwaremill.sttp.tapir:tapir-asyncapi-docs
com.softwaremill.sttp.tapir:tapir-core
com.softwaremill.sttp.tapir:tapir-json-circe
com.softwaremill.sttp.tapir:tapir-openapi-docs
com.softwaremill.sttp.tapir:tapir-swagger-ui-bundle

from 1.4.0 to 1.6.1
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (1.4.0).
You might want to review and update them manually.
.github/workflows/ScalaCI.yml
modules/zim-server/src/main/resources/static/js/cookie.js

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.softwaremill.sttp.tapir" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, old-version-remains, commit-count:1

Superseded by #357.


automatic download of wasm-opt and wasm-bindgen assumes x86_64

Our CI runs on arm64. Since trunk v0.12 our builds fail with:
error from HTML pipeline

Caused by:
    0: failed to spawn assets finalization
    1: error spawning wasm-opt call
    2: Exec format error (os error 8)

Digging into the automatic download code it looks like it assumes x86_64, which is reasonable considering that neither wasm-bindgen nor binaryen provide binaries for other architectures than x86_64.
Considering there's already some code to check which OS is in use, it might make sense to also check the architecture and provide a less obscure error message if it isn't x86_64 (or maybe just skip wasm-opt in this case?).
In our case the workaround is just to install wasm-opt 101 (we already have it in the CI image, but it's an older version), but I'll also file issues to see if binaryen (and wasm-bindgen) can provide arm64 binaries.

Reviewing the docs for std::io::Error/ErrorKind we may have to get fancy in order to determine when to provide the more detailed error message. ErrorKind::Other is quite likely the error variant we would get back, so we would have to resort to some form of string matching (which is often a bad idea).
More investigation is needed on best way to handle this.

One approach could be to substitute the actual arch in place of x86_64 in the URLs, and then error out with something like "no suitable wasm-opt (/wasm-bindgen) binary available" when you get a 404 trying to download it. That also has the advantage that new architectures supported by wasm-opt/wasm-bindgen in future get automatically supported by trunk.
It could be reasonable to warn and continue when it's wasm-opt (as opposed to wasm-bindgen) that can't be fetched.

In this case it would make sense to special-case macOS/arm64 though — otherwise M1 users will start to try to fetch the arm64 binaries (which don't exist yet) when x86_64 would work fine thanks to Rosetta 2.

@jbg yea, interpolating the actual arch into the URL seems like a pretty solid path forward. Right now, it would only clarify the issue, but not actually give the needed binaries, as they do not exist (as mentioned above). Either way, still an improvement.


Documentation📄: Support for Ubuntu 22.04 Jammy

What is the current documentation state?
@dlarson04 has merged support for Ubuntu 22.04  aka Jammy Jellyfish
https://github.com/open-horizon/anax/issues/3157
We should update the following docs to include jammy

installing/adding_devices.md

Where is this stated?
No response
Why do you want to improve the statement?
No response
Proposed Statement
No response
Additional context.
No response

Hey, i would love to work on this issue

Go ahead, @YashKamboj and let @johnwalicki or @joewxboy know if you have any questions.  Feel free to respond here or in the chat room at https://chat.lfx.linuxfoundation.org/
Don't forget to sign your commits and create the PR for this with a title that begins with "Issue-238: "


"Unused" strings are removed from Localizable.strings

I'm pretty sure this is probably expected behavior & not a bug - I'm probably just plain doing something stupid & need correction, but here's the scenario:
I receive some metadata from a remote server. I use that metadata to determine which strings to use when populating a specific table view with rows, and I need those strings to be translated. The metadata is not user-friendly on it's own, and contains other information I also need to store to map between the string the user sees and a couple different things on the server. Unfortunately, our server doesn't have the capability to send localized content. For this reason, I have all of the potential metadata saved in several Plists, which look similar to:
<key>SOME_SERVER_ENUM_VALUE</key>
<dict>
    <key>eventType</key>
    <string>otherMetadataServerHasMappedToOtherThings</string>
    <key>localizeableStringKeyOnMobileClient</key>
    <string>MyKeyToAStringIn.Localizable.strings</string>
</dict>

For a given key MyKeyToAStringIn.Localizable.strings I will have a matching entry in Localizable.strings:
"MyKeyToAStringIn.Localizable.strings" = "Friendly User Facing String That Needs Localized";

I've gone through and added all the key / value pairs in my Localizable.strings file. When I run bartycrouch update, all of those strings get removed.
Here's my .bartycrouch.toml contents:
.bartycrouch.toml
[update]
tasks = ["interfaces", "code", "transform", "normalize"]
[update.interfaces]
path = "./src"
defaultToBase = true
ignoreEmptyStrings = false
unstripped = false
[update.code]
codePath = "./src"
localizablePath = "."
defaultToKeys = false
additive = false
unstripped = true
[update.transform]
codePath = "./src"
localizablePath = "."
transformer = "foundation"
supportedLanguageEnumPath = "."
typeName = "BartyCrouch"
translateMethodName = "translate"
[update.normalize]
path = "./src"
sourceLocale = "en"
harmonizeWithSource = true
sortByKeys = true
[lint]
path = "."
duplicateKeys = true
emptyValues = true

Do I have something configured incorrectly? Is there some way I can prevent this from happening, or is this just expected behavior? If it is expected, can you suggest a workaround? My initial thought was to have a separate .strings file marked for localization & have Bartycrouch not interact with it, but I wanted to ask here if there was an alternative to that.
Specifications

Version: BartyCrouch 4.0.2
Platform: iOS
IDE Version: Xcode 11.4.1

@faithfracture Yes, this is just a configuration issue. In you .bartycrouch.toml I can see that additive is set to false. As described in the README under code options, the additive option does this: "Prevents cleaning up keys not found in code."
So just set it to true (which is the case by default by the way) and it should work as expected.

Yep, that took care of it. Thanks!


catnip appears to create invalid cwt/cbor claim

was able to finally get this payload to work using the json to create catnip claim:
{ "catnip":[{"tag":52,"value":[16,2560]}]  }
what should be produced:
/catnip/ 269: [/IPv4/ 52([16, /10.0.*.*/ h'0a00'])]
but produces cwt/cbor:
[57343([57344, ["tag", "value"], 52, [16, 2560]])]
81D9 DFFF 8419 E000 8263 7461 6765 7661 6C75 6518 3482 1019 0A00
would expect this cwt/cbor:
A119 010D 81D8 3482 1042 0A00
would be nice if we could use something like this json to create a catnip claim:
{ "catnip":[ {"IPv4": [16,"10.0.*.*"]} ]  }

Great finding, I just realised that I've done no implementations using catnip ;-) Probably the most used claim though...

While generating the catnip field it needs to be tagged e.g new Tagged(52, [16, Buffer.from('0a00', 'hex')] => <IP_ADDRESS>/16.

Need to make sure it also supports the full ip address, not just CDIR.  e.g. new Tagged(52, Buffer.from('0a0b0c0d', 'hex')) => <IP_ADDRESS>


grpc-cli: add install-grpc-cli target to Makefile

ease the installation of grpc-cli using the Makefile

The makefile is created with a template, please edit templates/Makefile.template

(then run tools/buildgen/generate_projects.sh)

oh, right I forgot to update this branch. Thanks!

updated and rebased to master

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


Windows executable stolen ".exe" extension from v0.12.0

This is mostly due to deprecation of gox package used for producing binaries.


Incorrect sorting with col with rowspan

Example: http://htmlpreview.github.io/?https://github.com/quantum13/bootstrap-sortable/blob/2d6122c33da89c3e62cd4b5ca994a88a65ec3bd5/index.html
Add rows and try sort some of last cols.

sorry for the long delay, I hope this will work for you


Doesn't compile on Linux

In Linux, sysctl.h doesn't include CTLTYPE_* so it doesn't compile.
See http://lxr.free-electrons.com/source/include/linux/sysctl.h

Linux is not supported at this time.

should probably be mentioned somewhere


"TypeError: select2.matcher is not a function" error being thrown in sl-select demo page

when clicked on the label/input field the select drop down results remain in "searching" mode. Nothing drop down menu items get rendered.

issue #434 address this issue. closing.

What is the cause of this error?  Because none of the changes in https://github.com/softlayer/sl-ember-components/pull/434 jump out at me as being what fixes this, given that the demo at http://softlayer.github.io/sl-ember-components/#/demos/sl-select is from the release on February 24 and the code being changed in the PR has been effect since then with no changes to the related lines of code but now it's throwing this error?

@erangeles added an explanation at https://github.com/softlayer/sl-ember-components/pull/434#discussion_r35027336


Fix wrong namespace references for unstable interfaces

A protocol as defined in "wlr-screencopy-unstable-v1.xml", named
"wlr_screencopy_unstable_v1" will emit something like this in
commons.zig
pub const wlr = struct {pub usingnamespace @import("wlr_screencopy_unstable_v1_common.zig");};

under the name "wlr". That is the prefix derived from the protocol name.
Definitions in client/server however use the interface's name to get the
namespace, which for an interface such as "zwlr_screencopy_manager_v1"
would result in references to commons.zig like this:
pub const Error = common.zwlr.layer_shell_v1.Error;

Those references break. This change passes the namespace from the
protocol down to the interface emitters.

LGTM, thanks!

Thank you. That's the last patch I needed to get https://github.com/squiddy/wcolor building. :)

Awesome! glad it's proving useful to you. I hope it a lot less painful to integrate with your build.zig in the future as well.


Double check Yatima.Expr.isVarFree and Yatima.Expr.getBVars

I was expecting getBVars to get the names of lam/pi/letE binders, but it's doing something else

This is low priority. isVarFree is used for debugging and getBVars is not used at all


Use preferred_names file

Followup on https://github.com/EastsidePreparatorySchool/epschedule/pull/472

This is no longer needed, we have this info in the 'nickname' field in 411.


add node metrics controller

1. Issue, if available:
#612 "Emit Metrics for Karpenter"
This PR does not fully resolve the issue. More changes will be needed.
2. Description of changes:
Add a controller that watches node events and updates counters across multiple dimensions, e.g. provisioner, readiness, and instance type
3. Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: link to issue
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Would love to review this. Can you give me until Friday?

Would love to review this. Can you give me until Friday?

Sure

There are some gaps in this approach.  Closing this PR and will open a new PR with the revised approach.


Expose Formik element's ref to withFormik

Bug, Feature, or Question?
Feature
Current Behavior
withFormik does not expose Formik element ref to manipulate the form state using HOC
Desired Behavior
I usually have use the Formik as a HOC for React components - it provides more clean code.
So it might be useful to expose Formik component's ref out of the withFormik HOC to provider access for field manipulation from top components like containers.
Suggested Solutions
I thought about 2 possible solutions:

expose innerRef from Formik element to top
save ref of Formik element as a class property of HOC

Additional Information

Formik Version: latest
React Version: 16.2
TypeScript Version:
CodeSandbox Link:
OS:
Node Version:
Package Manager and Version:

Can you elaborate on the use-case more? It doesn't make much sense to me.
Regarding the implementation, I would go similar way as React Router

i was trying to trigger submit from an external button.
The typical workaround seems to be either using an external ref.
https://stackoverflow.com/questions/49525057/react-formik-use-submitform-outside-formik
But HOC withFormik does not expose ref.


Use babel-plugin-lodash to redue bundle size

Fix for #5 using babel-plugin-lodash
Before,
Version: webpack 4.0.1
Child
    Hash: 942631aacfb8e1214122
    Time: 15022ms
    Built at: 30/06/2019 18:57:33
              Asset      Size    Chunks                    Chunk Names
        dagre-d3.js  3.12 MiB  dagre-d3  [emitted]  [big]  dagre-d3
    dagre-d3.js.map  3.39 MiB  dagre-d3  [emitted]         dagre-d3
    Entrypoint dagre-d3 [big] = dagre-d3.js dagre-d3.js.map

After,
Version: webpack 4.0.1
Child
    Hash: 8270ccb85e57fb1556f3
    Time: 11849ms
    Built at: 30/06/2019 18:59:18
              Asset      Size    Chunks                    Chunk Names
        dagre-d3.js  1.87 MiB  dagre-d3  [emitted]  [big]  dagre-d3
    dagre-d3.js.map  2.59 MiB  dagre-d3  [emitted]         dagre-d3
    Entrypoint dagre-d3 [big] = dagre-d3.js dagre-d3.js.map

I don't get the same results for some reason.
Before:
              Asset      Size    Chunks                    Chunk Names
        dagre-d3.js  1.77 MiB  dagre-d3  [emitted]  [big]  dagre-d3
    dagre-d3.js.map  1.45 MiB  dagre-d3  [emitted]         dagre-d3

After:
              Asset      Size    Chunks                    Chunk Names
        dagre-d3.js  1.89 MiB  dagre-d3  [emitted]  [big]  dagre-d3
    dagre-d3.js.map  1.55 MiB  dagre-d3  [emitted]         dagre-d3

Maybe just importing the needed modules individually (i.e. require ( 'lodash/foo' )) is a better option?

@fabiospampinato on a second try with a clean yarn.lock I get similar results to yours. I think the entire lodash is coming from graphlibrary dependency. https://github.com/tylingsoft/graphlibrary/blob/master/lib/lodash.js

With node_modules excluded from babel-loader,
Version: webpack 4.0.1
Child
    Hash: e49d864579caa4fd8e83
    Time: 3325ms
    Built at: 30/06/2019 22:09:03
              Asset      Size    Chunks                    Chunk Names
        dagre-d3.js  1.77 MiB  dagre-d3  [emitted]  [big]  dagre-d3
    dagre-d3.js.map   1.5 MiB  dagre-d3  [emitted]         dagre-d3

With node_modules included in babel-loader,
Version: webpack 4.0.1
Child
    Hash: fa20f072dd2f56420dc8
    Time: 12416ms
    Built at: 30/06/2019 22:09:59
              Asset      Size    Chunks                    Chunk Names
        dagre-d3.js  1.55 MiB  dagre-d3  [emitted]  [big]  dagre-d3
    dagre-d3.js.map  2.27 MiB  dagre-d3  [emitted]         dagre-d3

I think we can save more if we can fix graphlibrary https://github.com/tylingsoft/graphlibrary/issues/4

With graphlibrary patched like https://github.com/tylingsoft/graphlibrary/pull/5 bundle size actually increased!
Version: webpack 4.0.1
Child
    Hash: 88cfadb852efefab4e86
    Time: 12866ms
    Built at: 30/06/2019 23:06:03
              Asset      Size    Chunks                    Chunk Names
        dagre-d3.js  1.67 MiB  dagre-d3  [emitted]  [big]  dagre-d3
    dagre-d3.js.map  2.41 MiB  dagre-d3  [emitted]         dagre-d3

Did you try loading lodash in the same way in this library too?
I don't think babel-plugin-lodash works too well, or at least I also tried using it in the past with not much success.
Another important detail worth mentioning is making sure that there's only 1 non-deduplicated instance of lodash in the dependency tree as shown via npm list -prod, some times even though 2 packages both require lodash they need slightly incompatible versions of it so 2 instances of lodash get included in the bundle.


Explicit Undirected and Directed Hypergraph Support

Common Classes
class Vertex {
   var edge : Set(Edge);
   // Base Class Methods
}

class Edge {
   // Base Class Methods
}

Directed and Undirected Hypergraphs
Document describing Directed Hypergraphs: Gallo, Giorgio, et al. "Directed hypergraphs and applications." Discrete applied mathematics 42.2-3 (1993): 177-201.
Summary: Need a new hypergraph where hyperedges are composed of sets of tail vertices and head vertices.
class DirectedHyperEdge : Edge {
   var tail : Set(Vertex);
   var head : Set(Vertex);
   // Directed Edge Specific Methods
}

class UndirectedHyperEdge : Edge {
   var vertices : Set(Vertex);
   // Undirected Edge Specific Methods
}

Refactor AdjListHyperGraph to make use of UndirectedHyperEdge, create new one to make use of DirectedHyperEdge

Reopened.

@buddha314 If an edge includes a vertex, should it always be a 1?

My understanding of the convention is that in undirected graphs, a 1 indicates an undirected edge.  So an edge between v1, v3, v4 might look like  [1 0 1 1].
In a directed graph, use a -1 for tails and 1 for heads.  So a directed edge from v1,v3 to v4 would look like  [-1 0 -1 1]
I ignored v2 for illustration.  As to weights, I haven't seen much in the literature about that, so my plan is to multiply the adjacency matrix by a weighted matrix when needed.  If I find something more authoritative, I'll point it out.

Our homology calculation to read off the Betti numbers of a hypergraph
viewed as defining an abstract simplicial complex work on a boolean matrix
and I wonder if we will need a utility function to take the absolute value
of a directed incidence matrix to be able to do that calculation; likely we
should have a utility function to provide an undirected view of a
directed incidence matrix.
Best,
Mark Raugas
<EMAIL_ADDRESS>On Tue, Aug 25, 2020 at 9:57 AM Brian Dolan<EMAIL_ADDRESS>wrote:

My understanding of the convention is that in undirected graphs, a 1
indicates an undirected edge. So an edge between v1, v3, v4 might look like [1
0 1 1].
In a directed graph, use a -1 for tails and 1 for heads. So a directed
edge from v1,v3 to v4 would look like [-1 0 -1 1]
I ignored v2 for illustration. As to weights, I haven't seen much in the
literature about that, so my plan is to multiply the adjacency matrix by a
weighted matrix when needed. If I find something more authoritative, I'll
point it out.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pnnl/chgl/issues/53#issuecomment-680148083, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AD5T6HE4OS7LHIT6CTP5VPTSCPUN5ANCNFSM4HBJDA3A
.

That's a good point, I don't know if the Betti numbers are defined for directed graphs.  I'll try to look into that.

I am wondering if it would suffice to add addDirectory(e1,e2)  as a method to the hypergraph and then keep track of outgoing edges in hyperedge. This would allow undirected graphs as well as directed graphs, I think anyway. Then I can tell whether or not vertices are in the head or tail based on the direction.
I'm assuming you can't have the same vertex in the head and tail, so I can optionally check if this ever happens and throw an error.

This is not my specialty, but from Wikipedia I see the an abstract simplicial complex is defined as a family of sets.  From my dim recollection of graduate topology, that seems to preclude the notion of "directed".  And that seems to indicate Betti numbers are not defined on directed hypergraphs.  I don't see anything in my small library or google, but I'll keep digging.
So I suppose we either call the operation undefined on dirhypergraphs or we cast to an undirected graph.  Or ignore dirhypergraphs altogether?


feat: move NpmPackageId from deno_npm

Going to move this down here even though I liked it in deno_npm because I need this for deno_lockfile, but deno_npm depends on deno_lockfile. I could create deno_npm_types, but that seems like an unnecessary maintenance burden.
https://github.com/denoland/deno_npm/blob/f409ede427a6f38a35204cbedfa928487d0c835b/src/lib.rs#L29-L32

I just realized this is not necessary. That said, I want to integrate some of these changes back into deno_npm so I'll leave this as draft until that's done.


NETATMO no data collected from all the modules available

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.25.0
Python release (python3 --version):
Python 3.4.2
Component/platform:
sensor/netatmo
Description of problem:
only first module metrics are collected  from netatmo weather station.
Expected:
all modules metrics are collected
Problem-relevant configuration.yaml entries and steps to reproduce:
- platform: netatmo
  station: meteo-01
  modules:
    meteo-interno-01:
      - temperature
      - humidity
      - noise
      - pressure
      - co2
      - rain
      - sum_rain_1
      - sum_rain_24
      - wifi_status
    meteo-externo-01l:
      - temperature
      - humidity
      - battery_vp
      - min_temp
      - max_temp
    #rainmeter_name3:
    #  - rain
    #  - sum_rain_1
    #  - sum_rain_24
    #  - battery_vp
    #  - rf_status
    anemometro-01:
      - WindAngle
      - WindStrength
      - GustAngle
      - GustStrength
      - rf_status

start with this config and you'll see only internal module metrics
switch internal with external yaml config
you'll see external instead of internal

Traceback (if applicable):

Additional info:
There's also a bug if netatmo camara is not part of the setup: https://github.com/philippelt/netatmo-api-python/issues/6

Just a quick update, owner of netatmo-api-python mentioned that the error control 'should' be done in this side rather than within the module, so it might be a matter of selecting the right scope or control the error.
thanks,

Has this been addressed?

It did yes.

Can the issue be closed?

Yes, sure.


Support transformations using the compiler API

Implementation for #176. I'll look this over/clean it up later.

Coverage increased (+0.005%) to 94.814% when pulling b64be8f8da218c0bc5e945b582b797f36bf561b5 on issue176_SupportTransformations into 5c249bf949d998ec1d975572aea39570563e7971 on dev.


Including a add_index to allow custom index integration into workflow

currently the compute and compute_index function only work when the index is in the indices list. This makes computing custom indices kinda wonky, so the addition of a add_index function would help. This function should update the indices Dict as well as the indices_funcDict.

Actually we don't need this, we can just dispatch this over a function compute_index(index::AbstractSpectralIndex, params=nothing, online::Bool=false) that works as internal call to compute_index(index::String, params=nothing, online::Bool=false)

this was addressed in #58


Migrate to unplugin

I drafted a solution to migrate to unplugin, but ran into some issues in the process. Since we introduce an additional build-time dependency, we need to bundle it to avoid burdening the end user. That means we no longer can simply externalize every dependency and let Node figure it out.
For that reason, I slightly edited the building logic. I decided to replace Rollup with esbuild because it's more out-of-the-box, requires no plugins and less fiddling. That brought me to the problem of the @lezer/lr package which contains a ./dist/constants.js file that actually exports nothing, and isn't mentioned in the exports field of package.json. To proceed with this, we need to remedy those issues in @lezer/lr.
Closes https://github.com/lezer-parser/lezer/issues/48

Since we introduce an additional build-time dependency, we need to bundle it to avoid burdening the end user

Why would you need to bundle a build-time dependency?

I decided to replace Rollup with esbuild

That's not something I'm willing to go along with.

 Because we don't want to make end users have to install unplugin to use the plugin

Isn't that how the npm ecosystem works? You install dependencies when you need them.

Why would users be required to bundle the unplugin dependency? Can't it just be loaded from node_modules like usual?

Not if it's explicitly declared as a dependency, especially for stricter package managers like pnpm.

Obviously it'd have to be declared as a dependency, yes.

I sense a lot of resistance both here and in my other PR that you closed. I have no interest in trying to convince you that my approach to libraries is any better, I just want to use Lezer from ESBuild. If you're not willing to accept this PR, I will just create the unplugin in a separate repo and publish it as a separate package.

That's probably for the best.
Coming into a project and reorganizing stuff wholesale to fit your preferred approach is not always going to be welcome.

Here it is, in case you want to add a note about it: https://www.npmjs.com/package/unplugin-lezer. It's a shame we couldn't come to a consensus.


Error Sending Samples to Remote Storage

Hi,
I am facing issues while running pg-adapter.
We are using prometheus (2.8.1) along with Postgresql (10.5) as a remote storage.
It was running fine, but when I redeployed the services I started to get the issue
msg="Error sending samples to remote storage" err="pq: tuple concurrently updated" storage=PostgreSQL num_samples=3
Can anyone help on how to resolve this.
Regards,
Keshav Sharma

would it be possible to tell me a little more about how you have deployed version of our adapter you are using etc.  also it would be helpful if you could tail the logs for the adapter and share some of that may give me some insight into what is happening!