Show MicroShift version as part of 'version' command

Fixes: Issue #4098
Relates to: Issue #4099
Solution/Idea
Display MicroShift version as part of CRC version command output. Extend existing unit tests to verify that also this newly added field is present in the output.
Proposed changes

Add MicroshiftVersion field to Version type.
Read the bundled version from crcPreset
Print out the value as part of the version command

Testing

crc version contains Microshift version
if the output shows the correct Microshift version the test succeeded
Extended existing unit test to verify the claim

/ok-to-test

@cfergeau  I can not remember about those conversation, but in any case I introduced the bug during an specific sprint when we released or almost did it with different versions for ocp and microshift.
So having different versions would require for sure make the explicit somewhere.

/retest

version api also need to be updated for this.
$ git diff
diff --git a/pkg/crc/api/client/types.go b/pkg/crc/api/client/types.go
index 590f1e..776270 100644
--- a/pkg/crc/api/client/types.go
+++ b/pkg/crc/api/client/types.go
@@ -7,9 +7,10 @@ import (
 )

 type VersionResult struct {
-       CrcVersion       string
-       CommitSha        string
-       OpenshiftVersion string
+       CrcVersion        string
+       CommitSha         string
+       OpenshiftVersion  string
+       MicroshiftVersion string
 }

 type StartResult struct {
diff --git a/pkg/crc/api/handlers.go b/pkg/crc/api/handlers.go
index 35e165..558818 100644
--- a/pkg/crc/api/handlers.go
+++ b/pkg/crc/api/handlers.go
@@ -136,9 +136,10 @@ func getStartConfig(cfg crcConfig.Storage, args client.StartConfig) types.StartC

 func (h *Handler) GetVersion(c *context) error {
        return c.JSON(http.StatusOK, &client.VersionResult{
-               CrcVersion:       version.GetCRCVersion(),
-               CommitSha:        version.GetCommitSha(),
-               OpenshiftVersion: version.GetBundleVersion(preset.OpenShift),
+               CrcVersion:        version.GetCRCVersion(),
+               CommitSha:         version.GetCommitSha(),
+               OpenshiftVersion:  version.GetBundleVersion(preset.OpenShift),
+               MicroshiftVersion: version.GetBundleVersion(preset.Microshift),
        })
 }

so that following works as expected
$ curl -X GET --unix-socket ~/.crc/crc-http.sock http:/c/api/version | jq .
{
  "CrcVersion": "2.38.0",
  "CommitSha": "855bd7",
  "OpenshiftVersion": "4.16.0",
  "MicroshiftVersion": "4.16.0"
}

I am going to hold it for this release and then merge after release it done.
/hold

/unhold


Tensorflow Training Freezes for sometime, then continue on its own, happen multiple times when training a model

I recently upgraded to Tensorflow 0.12.1, Python3, Ubuntu 14.04, Cuda 8.0, Cudnn 5.1
Then I run my old code written for Tensorflow 0.10.1, which is a simple sequence-to-sequence model. And the training keeps freeze in the middle.
After some time the training will continue normally. Sometimes it 1 or 2 minutes, and sometimes it takes about 5 to 6 hours.
Anyone has opinion on how to trouble-shoot this?

Are you able to try with TensorFlow 1.0?
To troubleshoot, it would help greatly if you can provide a small self-contained piece of code that reproduces the problem.

The code worked no problem with TF 0.10.1 + Cuda 7.5 + Cudnn 4.
I didn't upgrade to TF 1.0 hence cannot try it immediately.
The code that occasionally cause the freezing problem is very similar to the TF seq2seq example.
https://github.com/TellinaTool/learning_module/blob/master/encoder_decoder/framework.py
I'm not sure if it's possible to pin-point at which line it freezes.

Unfortunately, that information isn't much to go by - there are many variables at play here - CUDA versions, NVIDIA driver versions, TensorFlow version, any hiccups in the input feeding etc.
Since there isn't enough information available to help track this down, I'm going to close this issue out. If you are able to get new information (preferably with release 1.0+) that can help zone in on the problem a bit more, please do reopen this issue and/or file a new one with more information.
Thanks!

I'm having the same problem.  I'm on CPUs so we can rule out CPU/GPU issues.  Also occuring with Keras 2.0.8, TF 1.3, both python 2.7 and 3.5, and also a TF 1.4-dev compiled from sources.  This leads me to believe it's an input feed issue.
Another note, the same code and inputs run fine on my Macbook.  I'm getting the problem with a new install on a workstation.   It's a VirtualBox instance running Ubuntu server 16.04.
Maybe someone could suggest an input feed diagnostic?


feat(NODE-3793): Remove offensive language from code and tests

Description
NODE-3793
What is changing?
Remove and deprecate (when removal is not possible) all references to "slaveOk" in the source code and tests. Note that spec test updates are being tracked in NODE-3791
Is there new documentation needed for these changes?
No
What is the motivation for this change?
Following the guidance documented in tools.ietf.org/id/draft-knodel-terminology-00.html, we will remove all oppressive and unnecessarily gendered language in driver documentation, code, tests, specs, and spec tests.
Double check the following

[x] Ran npm run check:lint script
[x] Self-review completed using the steps outlined here
[x] PR title follows the correct format: <type>(NODE-xxxx)<!>: <description>
[ ] Changes are covered by tests. I'm unsure and have commented below
[x] New TODOs have a related JIRA ticket

General question: Are these changes covered by tests?  I tried changing the return value of slaveOk() to both true and false (instead of this.secondaryOk;) and all of the tests passed.

Going to add a few more tests. Will comment here when I'm done

I've finished adding tests. This is ready for review.


Reserved robots for particular behavior goal

Added reserved robot functionality to behavior goal and realization system

Unit tests within behavior_realization_test are all failing due to the changed assignment logic

Also, needs linting
build/ateam_ai/test_results/ateam_ai/cpplint.xunit.xml: 34 tests, 0 errors, 1 failure, 0 skipped
- ateam_ai.cpplint whitespace/forcolon [2] (/__w/software/software/src/ateam_software/ateam_ai/src/behavior/behavior_realization.cpp:51)
  <<< failure message
    Missing space around colon in range-based for loop
  >>>
build/ateam_ai/test_results/ateam_ai/uncrustify.xunit.xml: 34 tests, 0 errors, 2 failures, 0 skipped
- ateam_ai.uncrustify src/behavior/behavior_evaluator.cpp
  <<< failure message
    Diff with 4 lines
  >>>
- ateam_ai.uncrustify src/behavior/behavior_realization.cpp
  <<< failure message
    Diff with 13 lines
  >>>

Logic looks sound to me :) I think the choice to make a separate function for implementing the specifically assigned actions is good (rather than calling more complex assignment logic).


[Grow] Add a workaround for Safari 15.4 CSS transition bug

[x] I have followed (at least) the PR section of the contributing guide.

Introducing a temporary workaround for CSS transition regression in Safari 15.4
WebKit tickets:

https://bugs.webkit.org/show_bug.cgi?id=238067
https://bugs.webkit.org/show_bug.cgi?id=237972

Note: this workaround does not address the case of concurrent CSS transitions on independent components (e.g. Speed Dial)
Resolves #31380
Preview: https://deploy-preview-31975--material-ui.netlify.app/components/transitions/#grow

Great work here! Looking forward to this getting in.

It looks like they fixed  it?

@MrBirb,
fixed !== released
That fix didn't even make it into Safari Technology Preview 142 that was released 2 days ago.


[Bug]: Home page issue plz assigned to me

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
HOME PAGE ISSUE

Text Alignment and Spacing:
The header text "Explore the Blockchain implementation..." seems too close to the top border, which can make the design feel cramped. Adding more spacing or padding can make the layout appear cleaner.

Unnecessary White Space:
There’s a significant empty space between the header on the left and the prediction form on the right. It feels unbalanced. You could consider reducing this gap or filling it with relevant content or images to make it more engaging.

3.Form Label Readability:
The label "Enter Cryptocurrency Symbol" has slightly small font size and lacks clarity. Increasing the font size or boldness could make it stand out more, improving user readability.
4.Text and Icon Position:
The Telegram bot icon could be better aligned with the text "@TrackCryptoPriceBot" for visual harmony. They appear slightly off-balance.

Color Contrast:
While the gradient background is visually appealing, the yellow text might not have enough contrast for readability on certain screens, especially for people with vision impairments. You could test for accessibility and consider a higher contrast color.

6.Missing Interactivity:
The placeholder area where the predicted price will be displayed is currently empty. Consider adding some placeholder text, like "Waiting for input..." to guide the user on what will happen after they predict the price.
Add ScreenShots

What browsers are you seeing the problem on?
Firefox, Chrome, Safari, Microsoft Edge
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

@Harish-2003 Bro I can Fix the issues that i mention above plz assign to me as labels : gssoc-extd hactoberfest


Fix Avatar on high res devices.

On higher resolution devices, the image avatar is misaligned.

BUGFIX-23


Fixed some accessibility audits

Mentioned on #214

Swapped li and a elements and corrected css
Added alt text to media links
Added aria-label to form input (let me know if you want it to say anything else.

Boosted your score. Those other two seem more up to you if you wanted to modify.

Awesome! Great job! 🎉


In the assay model need an Assay type of Cell Characterization

In addition to Toxicity, Disease and Efficacy, we need Cell Characterization as an assay type.

Cell Characterization added as a Study Type and an Assay Type; testing on the staging server.


Crashes with Origins mod winged origin

I had Trinkets installed so it wasn't the Trinkets-missing crash.
https://mclo.gs/MzvlYKh
I started the game with a flighted Origin, went to fly, and my game immediately froze completely.

In your crash log, I see there's an error on Line 251, which is where the elytra icon is drawn on the screen
https://github.com/kennethsible/elytrapitch/blob/1b2e88a09b744227df86e7d26a588a7998f3fc53/src/main/java/fatalflare/elytrapitch/ElytraPitch.java#L251
Can you go the ElytraPitch settings and disable "Show Icon" and see if the issue still happens? In the meantime, I'll read through the crash log more carefully and see if I can figure out the problem. Is this the conflicting mod: Origins?

Yes, that's the conflicting mod! I have a datapack that adds an additional Origin, Snatcher. I have so far not tested with the default flighted origin Elytrian.

java.lang.NullPointerException: Cannot invoke "net.minecraft.class_1799.method_7960()" because "$$2" is null

I think I see what happened. I tried searching for the Snatcher datapack, but I wasn't able to find it, so you'll have to confirm if my suspicion is correct. Does Origins + Snatcher add an elytra ability without an actual elytra equipped? Like, for example, the player just has the ability to fly from the beginning of the game? I am seeing a null pointer exception when trying to draw the elytra icon, which is grabbed from the icon of the equipped elytra, not just a picture file. If you aren't wearing an elytra, the mod will just crash. Let me know if my suspicion is correct and whether you still crash with the icon disabled. I can probably just disable the icon by default and add a warning about this as a tooltip in the setting menu.

The up-to-date version of the Snatcher datapack is currently in testing and not publicly available, but yeah, no actual equipped elytra and with the icon not enabled there is no crash. It's the same for the Elytrian origin but that one doesn't have a default custom appearance for the wings/pseudo-elytra.

In that case, I'll add a safety check to prevent the game from crashing if the player isn't wearing an elytra. I'll also either just turn off the icon by default or ignore the setting if an elytra isn't equipped (maybe both).

I pushed a commit that fixes the crash without Trinkets (e83f27a). You'll see these changes in the next release (v1.4.2).


Idiomatic way to invoke a command from another command

I did a quick search on previous issues and I think what I want to accomplish might not be a good idea (since nobody apparently thought about this before), but is there a way to chain command invocations (extra: passing arguments) with SwiftCLI?
What I ideally would like to accomplish is something like this:
class Command1: Command {
    func execute() throws {
        ...
        var command2 = Command2()
        command2.param = "param" //this gives a 'param' is get-only error
        try command2.execute()
    }
    ...
}

class Command2: Command {
    @Param var param: String?

    func execute() throws {
        ...
    }
    ...
}

There's no built-in way to do this -- I think generally the pattern at this point would be to call a common function from both commands:
class Command1: Command {
    func execute() throws {
        doSomething(with: "otherValue")
    }
    ...
}

class Command2: Command {
    @Param var param: String?

    func execute() throws {
        doSomething(with: param)
    }
    ...
}

func doSomething(with this: String) {
     ...
}

I've been hesitant to add a feature which makes invoking different commands easier, as Command is more about adding structure to the list of arguments passed by the user. If you just want to invoke common functionality from two commands, I think it makes more sense to factor that functionality out of any individual command.

Closing -- feel free to reopen if you think this should be in SwiftCLI / have an idea of what it could look like!


Context and namespace widgets: use Select rather than Dropdown? Support tab pinning?

A few UX issues with our current use of a Dropdown UI for the CurrentContext and CurrentNamespace status stripe widgets.

If i have a large number of namespaces or contexts, the dropdowns become unwiedly. The PatternFly Select widget supports favoriting and typeahead find. https://www.patternfly.org/v4/components/select#favorites
Maybe a Popover with some descriptive text and a Select?

A Popover widgets could help with pinning context or namespace to Kui tab. e.g. a Pin icon in the upper right of the Popover. Then the widgets could listen to tab switch events and auto-adjust the context or namespace, based on the user's choice of Pinning. One bit of complexity here: pinning a namespace implies pinning a context, but the reverse is not true.

Following this discussion....
We have noticed that customer use cases for Red Hat Openshift Container Platform (OCP) often result in lots of namespaces, especially in large organizations.  And customers in larger organizations are growing the number of clusters they manage.


Add Failover-GcSqlInstance cmdlet

Add Failover-GcSqlInstance cmdlet and associated tests.

Just a few comments.

PTAL, thanks!

LGTM


Add comment about generated code

It would be nice that the generated code would include a comment about it being auto-generated, so that people would not update it manually, but instead change their .proto-files.

Not necessarily, since generated code is often used as a starting point, and then changed manually to fit a particular use case (such as lazy parsing). I'd leave everything as is.


fix modal buttons order

The buttons were not displayed in the correct order on the modal.

All good, if you're ready go I'll let you merge, and if you need to use it now, you can bump to the next version


Umbraco v10 support

We have a project on Umbraco v9 using the OEmbed Picker, which we are intending to upgrade to Umbraco v10.
Does the package work on v10 / .NET6 or does it need a new build for this?
On Umbraco v9 I noticed this issue when switching property editor. May be worth including as well.
https://github.com/dawoe/OEmbed-Picker-Property-Editor/pull/23
However I think it would be nice to integrate this in core to benefits from this as it mainly wraps existing functionality:
https://github.com/umbraco/Umbraco-CMS/pull/12609

The V10 version has just been released


[Bug report] Can't use relative path for local theme

I'm trying to create a Vuepress next site and running into a roadblock.
I can only specify a path to the local theme (<repo>/.vuepress/theme) by using an absolute path... this poses a problem because the path to the theme is different on my machine than another developer's.
Why doesn't this work in .vuepress/config.js?
module.exports = {
     theme: '.vuepress/theme',
}

or
module.exports = {
     theme: './theme',
}

path.resolve(__dirname, '....')

Ok, so for anyone looking for the same thing:
in .vuepress/config.js
var path = require('path')

module.exports = {
  theme: path.resolve(__dirname, 'theme'),
}


Please make a release

For packaging, it's good to have stable release tarballs to use. I'd like to package this for pkgsrc, but AFAICT there is not even a tag yet.
Please make a release or at least tag a version. Thank you.

Hey @0-wiz-0, see https://github.com/elixir-lang/tree-sitter-elixir/issues/44#issuecomment-1301925843, we keep a release on hold because it's unclear how versions should be numbered. FWIW most package managers (npm/crate) allow pointing to a specific SHA version on GitHub.

Thanks, I commented on that one.
I used the SHA version for now, but I still believe that official releases are better :)


Add some more notes about branches

Can't wait until we have mergify and I can tell people that only mergify is allowed to make PRs to release branches ;)


Issue 5018

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #5018)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

This PR is is blocked by https://github.com/apache/incubator-gluten/pull/4676, needs its bug fix


Code for FNO-PF model

Hi, I am having some issues reproducing the FNO-PF results. I am working on experiment E1, using the burgers equation and the 1d FNO model from their github rep. With only the Push-forward applied I arrive at reasonable results, but I can't quite figure out how the temporal bundling is implemented.
One could for instance just change the output size of the last layer to match the time window, whereas another option is to apply the approach as in equation 10 in your paper, possibly with the extra convolutional decoder layers included.
Which approach did you use in the experiments?
Thanks in advance,
Winfried

Hi @Winfriedvdd thank you very much for your interest.
Including temporal bundling into FNO is a bit tricky, there are many possibilities to do that.
Maybe you have a look at our other repo, and especially here: https://github.com/brandstetter-johannes/LPSDA/blob/master/experiments/models_fno.py#L66
There we used FNO with temporal bundling, pushforward (and Lie point symmetry data augmentation) on a regular grid with periodic boundaries, i.e. where FNOs really shine.

Thanks for the reply! This is also a very nice paper indeed.
I see that in your implementation of the FNO network you also include dt as
a channel dimension in the input (rather than just the spatial locations as
in the original FNO implementation). Is this required to make the temporal
bundling work?
Kind regards,
Winfried
Op wo 7 sep. 2022 om 14:27 schreef brandstetter-johannes <
@.***>:

Hi @Winfriedvdd https://github.com/Winfriedvdd thank you very much for
your interest.
Including temporal bundling into FNO is a bit tricky, there are many
possibilities to do that.
Maybe you have a look at our other repo, and especially here:
https://github.com/brandstetter-johannes/LPSDA/blob/master/experiments/models_fno.py#L66
There we used FNO with temporal bundling, pushforward (and Lie point
symmetry data augmentation) on a regular grid with periodic boundaries,
i.e. where FNOs really shine.
—
Reply to this email directly, view it on GitHub
https://github.com/brandstetter-johannes/MP-Neural-PDE-Solvers/issues/4#issuecomment-1239324006,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYXGCTEIKM6CXQGGTYBKLLV5CC4XANCNFSM6AAAAAAQF6DFPU
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

And could you perhaps send the code for the FNO model that is used
specifically in the message passing GNN paper? Just in case I still can't
get it to work :)
Kind regards,
Winfried
Op ma 19 sep. 2022 om 15:43 schreef Winfried van den Dool <
@.***>:

Thanks for the reply! This is also a very nice paper indeed.
I see that in your implementation of the FNO network you also include dt
as a channel dimension in the input (rather than just the spatial locations
as in the original FNO implementation). Is this required to make the
temporal bundling work?
Kind regards,
Winfried
Op wo 7 sep. 2022 om 14:27 schreef brandstetter-johannes <
@.***>:

Hi @Winfriedvdd https://github.com/Winfriedvdd thank you very much for
your interest.
Including temporal bundling into FNO is a bit tricky, there are many
possibilities to do that.
Maybe you have a look at our other repo, and especially here:
https://github.com/brandstetter-johannes/LPSDA/blob/master/experiments/models_fno.py#L66
There we used FNO with temporal bundling, pushforward (and Lie point
symmetry data augmentation) on a regular grid with periodic boundaries,
i.e. where FNOs really shine.
—
Reply to this email directly, view it on GitHub
https://github.com/brandstetter-johannes/MP-Neural-PDE-Solvers/issues/4#issuecomment-1239324006,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYXGCTEIKM6CXQGGTYBKLLV5CC4XANCNFSM6AAAAAAQF6DFPU
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Hi Winfried
Adding dt and dx is not required, we did it to be more consistent with origin FNO implementation. This code I pointed you to is the implementation we used. FNO is just having a harder time because they have to model a shock and they have to generalize across parameters. So if you want to see 1D problems where FNO shines I would recommend the LPSDA datasets :)

Hi and thanks again! One more question, the FNO code in the LPSDA dataset
uses 32 modes and a width of 256, as well as 5 layers (instead of 16, 64, 4
used in original FNO implementation I believe). This would give it approx
10 million parameters. Is this the one you use in the experiments or are
you using different hyperparameters? (In the paper you mention approx 1
million parameters, but I wasnt sure whether this solely refers to the
MP-Neural-PDE-Solver or also to the other models.
Winfried
Op wo 21 sep. 2022 om 15:09 schreef brandstetter-johannes <
@.***>:

Hi Winfried
Adding dt and dx is not required, we did it to be more consistent with
origin FNO implementation. This code I pointed you to is the implementation
we used. FNO is just having a harder time because they have to model a
shock and they have to generalize across parameters. So if you want to see
1D problems where FNO shines I would recommend the LPSDA datasets :)
—
Reply to this email directly, view it on GitHub
https://github.com/brandstetter-johannes/MP-Neural-PDE-Solvers/issues/4#issuecomment-1253687466,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYXGCXHHULLAYHDB3GW7JLV7MCHDANCNFSM6AAAAAAQF6DFPU
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Your are right of course, we adjust to 1million parameters, 4 layers, 12 modes and 32 hidden channels. What if we take this discussion offline? Just shoot me an email.
Johannes


Update tutorial-elastic-query-with-sql-datababase-and-sql-data-wareho…

…use.md
Minor changes to Syntax format and corrected table name reference.

@mlee3gsd : Thanks for your contribution! The author, @hirokib, has been notified to review your proposed change.

#sign-off

@mlee3gsd: I'm sorry - only the author of this article, @hirokib, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Update

https://github.com/ikatyang/emoji-cheat-sheet/issues/601#issue-1335427449

https://github.com/ikatyang/emoji-cheat-sheet/issues/601#issue-1335427449

Yes hurry and update


(Solution inside) Fails on Windows because kernprof.exe is run with python

If run in Windows, fails with the following output:
  File "C:/Anaconda/Scripts/kernprof.exe", line 1
SyntaxError: Non-ASCII character '\x90' in file C:/Anaconda/Scripts/kernprof.exe on line 1, but no encoding declared; see http://www.python.org/peps/pep-0263.html for details

This is because kernprof.exe is run using python: Line 264 of lineprofilergui.py is
executable = '{0} {1}'.format(sys.executable, script_path)

which means executable is set to (in my case) C:\Anaconda\pythonw.exe C:\Anaconda\Scripts\kernprof.exe, whereas the correct would be only C:\Anaconda\Scripts\kernprof.exe.
Solution: Replace this line with
executable = script_path

Tested and works perfectly.

Thanks for the report and the solution !
I didn't test it but I'll trust you as it seems logical (kernprof used to be a python file but is now a program on its own).


Update registry.yaml

@xYundy
Invalid page.json format for user "xYundy".
Error: Process completed with exit code 1.

@fatih-yavuz I fix that. Additionally I added an example in readme about main.js minimal structure.

@xYundy you rock 🚀


[Change Request] Codeunit 5771 "Whse.-Sales Release" – Procedure  “CreateWhseRqst”

Could we please have the existing procedure “CreateWhseRqst”, Codeunit 5771 "Whse.-Sales Release” be set to global?
*** start old code ***
//local procedure CreateWhseRqst(var SalesHeader: Record "Sales Header"; var SalesLine: Record "Sales Line"; WhseType: Option Inbound,Outbound)
*** stop old code ***
*** start new code ***
procedure CreateWhseRqst(var SalesHeader: Record "Sales Header"; var SalesLine: Record "Sales Line"; WhseType: Option Inbound,Outbound)
*** stop new code ***
var
SalesLine2: Record "Sales Line";

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Add Rust demo

I created Rust bindings for Porcupine and thought of sharing a demo of its usage here :grin:. Compiling and running it should be very simple if the user has already installed the native libraries in discoverable locations (e.g. /usr/local/lib).
There is some problem with getting the desired microphone format under macOS though. I might have to convert the audio data (e.g. from float32) to Porcupine-compatible format before processing it.

This is awesome. Give me some time to review the code (I am going to use this as an opportunity to learn  Rust!).
Could you please which platforms have you tested this demo on? Thanks in advance.


chore: update mysql yaml port

Signed-off-by: yeya24<EMAIL_ADDRESS>

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

Sorry - you need to to sign CLA before merge.

Sorry - you need to to sign CLA before merge.

Done


[Network] az network public-ip create: Support '--zone 1 2 3' after '2020-08-01'

Description

https://github.com/Azure/azure-cli/issues/15015
az network public-ip create: Support '--zone 1 2 3' after '2020-08-01'
Testing Guide

test_network_ag_zone
test_network_public_ip_zone
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network


Fixes a wrong indication in the readme

Hi,
I have removed a sentence where there was an explanation about the required=true statement in the @RequestMapping explanation section, since it's not there in this tutorial.
I guess it's a copy/pasta mistake since this parameter is indeed present in the Spring MVC Web tutorial :+1:

@mbeaudru Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@mbeaudru Thank you for signing the Contributor License Agreement!


iascable fails with migrated repos

Generation is failing for repos that have been moved into the terraform-ibm-modules repo b/c the metadata in the gh-pages branches for all dependent repos still points at the old location (and does not handle redirects).
For example, I migrated the https://github.com/cloud-native-toolkit/terraform-ibm-resource-group repo over to https://github.com/terraform-ibm-modules/terraform-ibm-toolkit-resource-group.
(I have since moved it back into our org b/c of the issues described below.)
iascable fails b/c it can't find the module in the old location:

This is b/c the module dependency metadata is still pointing at the old location:

I ran a test to see how terraform behaves with redirected content, and it worked as expected (and correctly followed the URL redirects).  Details in the comment here: https://github.com/cloud-native-toolkit/software-everywhere/issues/357#issuecomment-1206476058
So, I think the only failure is really just iascable at this point.

Work in progress to update iascable here: https://github.com/cloud-native-toolkit/iascable/pull/173

Abandoning the dynamic lookup, in favor of static map that will be easier and faster to implement.  Modules don't move often.


auto refresh registration token for github secrets

The script appears to be updating the GH Secret. Need to test with a deployment tomorrow.

This was successful.


FileNotFoundException thrown when referring to WSUS API DLL

Config: Visual Studio 2019 + .Net Core 3.0.101 on Windows Server 2019 with WSUS role installed.
I have a console project that reference
C:\Program Files\Update Services\Api\Microsoft.UpdateServices.Administration.dll
and in the code I try to get an instance of IUpdateServer object :
var updateSrv = AdminProxy.GetUpdateServer();
Which throw an exception:
System.IO.FileNotFoundException : 'Could not load file or assembly 'Microsoft.UpdateServices.Utils, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
If I add a reference to this DLL: C:\Windows\Microsoft.NET\assembly\GAC_MSIL\Microsoft.UpdateServices.Utils\v4.0_<IP_ADDRESS>__31bf3856ad364e35\Microsoft.UpdateServices.Utils.dll
I receive this exception:
System.IO.FileNotFoundException : 'Could not load file or assembly 'Microsoft.UpdateServices.BaseApi, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'.
Adding this DLL: C:\Windows\Microsoft.NET\assembly\GAC_MSIL\Microsoft.UpdateServices.BaseApi\v4.0_<IP_ADDRESS>__31bf3856ad364e35\Microsoft.UpdateServices.BaseApi.dll
I receive:
System.IO.FileNotFoundException : 'Could not load file or assembly 'System.Configuration.ConfigurationManager, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'.
And I don’t have this DLL.
By the past, with .Net Framework, referencing the first DLL was enough to work with WSUS API. Is it still possible to do this with .Net Core?

You'll need to see if WSUS has a .NET Core compatible DLL -- perhaps .NET Standard. DLLs built for Framework will generally not work with Core. @joperezr

You'll need to see if WSUS has a .NET Core compatible DLL -- perhaps .NET Standard. DLLs built for Framework will generally not work with Core. @joperezr


Such talented work done, can I ask you for a hint?

Hi my name is Vince Kim, a univ student from South Korea.
For weeks, I have been trying to delete the number of n pages from the fullest resolution from svs file, because our slides are too large to store.
And finally, I run into your work and trying to figure out how to and having hard time.
I was wondering if you would like to give some hint?
It will mean sooooooooo much to me.
Below is what I'd like to do.
thank you in advance!
cheers.
def downsize(svsfile, n_pages):
# read svsfile
# there are 9 pages from the tiled pyramid images
# 0th image being the full resolution image ( largest )
# if n_pages equals 2,
# remove first two largest images
# save as svs file

Hi @JooHyukKim , so sorry I didn't see this! I did not have notifications turned on for the repository unfortunately. I hope you were able to make something work eventually.

Hi @JooHyukKim , so sorry I didn't see this! I did not have notifications turned on for the repository unfortunately. I hope you were able to make something work eventually.

Yes,  I found a work around.
I will close this issue as a result.
Keep up the great work! Cheers


publisher: support confluence cloud's modernized search

Confluence Cloud has dropped the use of its older pagination support since 2020-07-15 [1] which has prevented this extension from properly processing cleanup operations if the target environment has over 249 descendants/attachments. This commit updates the search calls to use Confluence Cloud's link hints to acquire an appropriate cursor hint.
[1]: https://developer.atlassian.com/cloud/confluence/change-notice-moderize-search-rest-apis/

Thank you for this. We stumbled over this and had a local fix we were planning to submit.

Thank you for this. We stumbled over this and had a local fix we were planning to submit.

@felixvd, sorry about that. Only noticed it by chance with testing in the last weeks. v2.6 of this extension has now been released (with this fix), so please feel free to try it out and report any issues.


Add body parsing functionality in Request object

Currently we do not parse any incoming data in the Request object. All that functionality has been moved to the Slim-Http decorators repository.
We should by default at least support the incoming form data in the $_POST object and JSON.

$body = (new StreamFactory())->createStreamFromFile('php://input');

$body = (new StreamFactory())->createStreamFromFile('php://input');

We need to do this in addition to parsing the bodies different content types.

@l0gicgate i try to create a PR, but its hard to test (php://input) is not writeable

@l0gicgate adding php://input and use it for parsed body are two topics, please fix one after the other, cause parsed body from php://input it not a hard requirement for psr-message

@dominikzogg I'm fine with adding a stream wrapper dependency like https://packagist.org/packages/mikey179/vfsStream for unit testing so we can wrap it and test it.

@l0gicgate i didn't find any working example mocking php://input with vfsStream

https://github.com/slimphp/Slim-Psr7/pull/110

https://github.com/slimphp/Slim-Psr7/pull/110/files#diff-ea033125b85265ce3adf4e102a8d58bbR117

Slim 4 also has that feature  already included in the BodyParsingMiddleware: https://github.com/slimphp/Slim/blob/4.x/Slim/Middleware/BodyParsingMiddleware.php
Do we really want to maintain the same feature 3 times? (1. slim/slim, 2. slim/http and now also in 3. slim/psr7)


After push firmware, denied

After push firmware,
Message :  Request denied
FAULTCODE: 9001
FAULTSTRING: Request denied

GitHub is for defect tracking, not support requests. Please close this issue and take it to the mailing list.
-dan

On Apr 22, 2018, at 9:14 PM, vaheed<EMAIL_ADDRESS>wrote:
Message : Request denied
FAULTCODE: 9001
FAULTSTRING: Request denied
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

ok, thanks, i follow in mailing list


Link redirects 4.0.0

link redirects in the Deploy and Publish section.

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


Integration Test - Release - 2019/06/21 18:54:03

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

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


Fixes

Code now passes validation checks.
Fixed a bug in remote inspect code causing panic.
Unfortunately tests do not pass.

Merged


testInteractiveKeyboardDidResignFirstResponderDelegateisCalledAfterDismissedKeyboard engine test failing on Xcode 15 in CI

testInteractiveKeyboardDidResignFirstResponderDelegateisCalledAfterDismissedKeyboard fails when testing for ios_debug_sim_arm64 with Xcode 15 in CI. Seems to happen consistently.
Test Case '-[FlutterTextInputPluginTest testInteractiveKeyboardDidResignFirstResponderDelegateisCalledAfterDismissedKeyboard]' started.
2023-08-31 09:23:14.392773-0700 IosUnitTests[26556:189071] Failed to find assets path for "Frameworks/App.framework/flutter_assets"
2023-08-31 09:23:14.399457-0700 IosUnitTests[26556:190740] [VERBOSE-2:engine.cc(210)] Engine run configuration was invalid.
2023-08-31 09:23:14.399784-0700 IosUnitTests[26556:190740] [VERBOSE-2:shell.cc(640)] Could not launch engine with configuration.
IosUnitTests(26577,0x102f30240) malloc: enabling scribbling to detect mods to free blocks
objc[26577]: Class NEDiagnosticReport is implemented in both /opt/s/w/ir/cache/osx_sdk/xcode_15a5219j_runtime_ios-17-0_15a5219j_ios-16-4_14e300c/XCode.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS.simruntime/Contents/Resources/RuntimeRoot/usr/lib/libnetworkextension.dylib (0x119e78030) and /opt/s/w/ir/cache/osx_sdk/xcode_15a5219j_runtime_ios-17-0_15a5219j_ios-16-4_14e300c/XCode.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS.simruntime/Contents/Resources/RuntimeRoot/System/Library/Frameworks/NetworkExtension.framework/NetworkExtension (0x119614948). One of the two will be used. Which one is undefined.
IosUnitTests(26577) MallocStackLogging: could not tag MSL-related memory as no_footprint, so those pages will be included in process footprint - (null)
IosUnitTests(26577) MallocStackLogging: stack logs being written to /private/tmp/stack-logs.26577.102d6c000.IosUnitTests.EzVUp3.index
IosUnitTests(26577) MallocStackLogging: recording malloc and VM allocation stacks to disk using standard recorder
IosUnitTests(26577) MallocStackLogging: process 26556 no longer exists, stack logs deleted from /tmp/stack-logs.26556.102b24000.IosUnitTests.HEuAP9.index

Restarting after unexpected exit, crash, or test timeout in -[FlutterTextInputPluginTest testInteractiveKeyboardDidResignFirstResponderDelegateisCalledAfterDismissedKeyboard]; summary will include totals from previous launches.

https://chromium-swarm.appspot.com/task?id=646528534341a410
https://chromium-swarm.appspot.com/task?id=64607eb18f3b9910

@hellohuanlin Could you take a look at this when you have time?

It might be related to FlutterEngine or FlutterViewController not dealloc'd properly in some other tests. Locally, I commented out all the textinput tests, still got crashes somewhere else when sending notificatons:
Test Case '-[FlutterViewControllerTest testHandleKeyboardNotification]' started.
2023-08-31 15:36:21.297760-0700 IosUnitTests[35974:210651] Failed to find assets path for "NSBundle </Users/ychris/Library/Developer/CoreSimulator/Devices/8FF72C16-60EA-4A50-AD96-DE5F5EBBD245/data/Containers/Bundle/Application/FAE0ED40-EFF8-47FB-8A5A-70BE511DC6F1/IosUnitTests.app> (loaded)"
2023-08-31 15:36:21.311945-0700 IosUnitTests[35974:210651] [VERBOSE-2:isolate_configuration.cc(217)] application_kernel_asset or application_kernel_list_asset must be set
2023-08-31 15:36:21.312861-0700 IosUnitTests[35974:211849] [VERBOSE-2:engine.cc(210)] Engine run configuration was invalid.
2023-08-31 15:36:21.313615-0700 IosUnitTests[35974:211849] [VERBOSE-2:shell.cc(640)] Could not launch engine with configuration.
2023-08-31 15:36:21.381447-0700 IosUnitTests[35974:210651] [VERBOSE-3:FlutterEngine.mm(351)] Check failed: _shell.```

The crash is due to testShareScreenInvoked presents the activityViewController, which prevents the engineViewController to be dealloc'd in the test: https://github.com/flutter/engine/blob/main/shell/platform/darwin/ios/framework/Source/FlutterPlatformPlugin.mm#L161C4-L161C24
I can provide a fix with stubbing the presentViewController call, which is probably sufficient to test the logic of the method channel while preventing this leak.
CC @LouiseHsu who might have better ideas.


Fix SPP dam pulling tomorrow for today (#146)

test errors are unrelated to your change. merging now


Improvement: Show "table not found" info in wrong query

When we compose/run an adhoc query, we sometimes misspell table names.
The results from query is "Broker resource missing" instead of "table not found", which is quite misleading. It is misleading because sometimes we do run into broker down situation and has the same error.
It will be a small change but with much quality of life improvement. Think about the new users that are trying Pinot and getting such errors could turn them away.

Makes sense, do you want to take it up?

Sure, I can help with that. It would be a start to contribute

Thanks @dongxiaoman !

I would like to work on this

HI,
I see that there is a merged PR associated with this issue.
Is this still open?

@vijaya-lakshmi-venkatraman Thanks for the reminder, the issue is resolved


Selecting time Period above 1 Week

Hi,
I have a problem when selecting a time period of more then 1 week. When pressing the 1 month or 1 year option the 1 week graph appears. I already posted about this in the monitoring-portal.org and someone else seems to have the same issue. Does anyone know what I could do to fix this ?
I have this Problem in graphite-web and also in icingaweb graphite Plugin.
Can maybe someone help me?

Are you setting 1 week for retention period?  Check this -> https://github.com/graphite-project/whisper/issues/166#issuecomment-243713438


npm Audit for jest 24.9.0/25.1.0

gives me lot a moderate auidt issues more than 300-400
One error
Prototype Pollution
Package        minimist
Patched in    │ >=1.2.3
almost every audit is based on this

related mkdirp issue

We've removed mkdirp (#9486) so nothing for us to do. Will be fixed by the next release, whenever that is.
Upstream issue also seems fixed, so just do npm audit fix or whatever the command is

npm audit fix doesn't seem to work:
fixed 0 of 553 vulnerabilities in 883413 scanned packages
  172 vulnerabilities required manual review and could not be updated
  2 package updates for 381 vulnerabilities involved breaking changes
  (use `npm audit fix --force` to install breaking changes; or refer to `npm audit` for steps to fix these manually)

Same issue here with 25.1.0

Related discussion: https://twitter.com/RoLLodeQc/status/1240468981456285696
npm audit fix doesn't upgrade to deprecated versions, as is the case with mkdirp 0.5.3.

So, the solution is to run npm install mkdirp --save-dev, then run npm uninstall mkdirp --save-dev.


refactor: restructure folders and add user routes

Related Issue
Closes #200
Description
This pull request refactors the project’s folder structure for better organization and maintainability. The following changes have been made:

Routes are now located in the routes folder.
Controllers are moved to the controllers folder.
Models are refactored into separate files for improved clarity.
New user routes for the About, Contact, and Privacy pages have been added.

These changes improve the overall structure of the codebase and make future updates easier to manage.
Screenshots (if applicable)

Original
Updated

Not applicable
Not applicable

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] I have included comments in areas that may be difficult to understand.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@Kritika30032002 please merge the PR after testing all the tests are done from my side

@Kritika30032002 Please Merge the Pull Request, All the tests are done from my side

@Kritika30032002 please merge this pr


Clean up package.json and dependencies

[x] I have changed my target branch to develop :facepunch:

Description
Hi, and thanks for your work on those components! :+1:
I would like to contribute to this project and start with some stuff related to dependencies and build.
Also, consider this PR as proposition for discussion :)
Here's what changed:

removed webpack dependencies, since this project uses rollup and there is no webpack config
moved babel-helpers to devDependencies - where it should be
removed unnecessary navigation commands from prepublish script (I assume that those were added to be able to publish from lib/build directory, but I don't really get why package.json is copied to build directory. So it's more proposition for discussion ;) ).
renamed prepublish script to prepublishOnly, since prepublish is now deprecated
changed travis.yml to run prepublishOnly script instead of duplicating same commands

Few things I would also like to discuss:

looks like moment-range and react-text-mask are included into package build (correct me if I'm wrong), is it intentional?
why package.json is copied to lib/build directory?

why package.json is copied to lib/build directory?
Check how deploy is going on. To the build scripts, firstly we build all to the build/ folder, then we copy typing there, and after all copying package json, to have an ability to publish directly build folder as the production build.

@dmtrKovalenko okay, I see.
Indeed, package.json should be copied to lib/build, but prepublish script should not contain cd .. and cd -.
Instead, when publishing a package run npm publish build from the lib directory - like material-ui does.
This way we'll be able to get rid of relative paths / navigation in npm scripts.


Opening furnace when locked and interacting with the GUI closes it

Ok, so a user on my server recently let me know that they kept getting kicked out of the GUI of all furnaces every time they tried to place an item in to smelt.  The furnace worked fine, but they got kicked out immediately after placing or removing items.  I had them /latch delete on it and the furnace went back to normal functionality and not closing on every interaction.

I can reproduce this. If any more players have this issue or you have any information please respond here. I will not close it for now as I will continue to investigate.

Seems that shift clicking items out of a locked furnace makes them dissapear

@turtledude01 What version and build of Sponge are using? There's been a lot of changes recently in Sponge relating to inventory recently so this stuff is fairly volatile.

Spongevanilla one of the newest API 6 builds

I have not been able to reproduce this issue and @turtledude01 has told me he is not experiencing it anymore either. So I will close this for now, but if anyone else experiences it please feel free to comment.


Sleep timer

A timer that you can set to automatically close the app and lock your phone screen, so you can set it before sleeping

Is this what you had in mind?
https://github.com/bttv-android/bttv/suites/2939511684/artifacts/66050114 (link is only valid for a day)

Sorry, I didn't get notified by your reply. Do you still have the artifact? If not that's okay, there are apps for this, I just thought it would be cool for it to be built-in in this app, like Xtra for Twitch had.

I just released 0.7.0-beta and this feature is part of it.

Thank you very much! I love your app

@all-contributors please add @Epicnicity322 for ideas


schema state out of sync

Hi team
Seeking advice around a problem about Terraform not detecting that a schema needs updating
Current Schema
When I log into Confluent via Browser or CLI and look at a specific schema, see that like so: (v0)
{
  "$id": "json/product/v0.json",
... redacted ...

Desired Schema
The desired state is v1, this is what I am attempting to deploy:
{
  "$id": "json/product/v1.json",
... redacted ...

Apply
But when I ask Terraform to refresh state - it does not detect the difference:
$ terraform apply -refresh-only

... redacted ...

No changes. Your infrastructure still matches the configuration.

Terraform has checked that the real remote objects still match the result of your most recent changes, and found no differences.

Apply complete! Resources: 0 added, 0 changed, 0 destroyed.

State
... And when interrogate the Terraform state, I can see it thinks the state is v1:
$ terraform state pull

... redacted ...

    {
      "module": "module.topic[\"products\"]",
      "mode": "managed",
      "type": "confluent_schema",
      "name": "schema",
      "provider": "provider[\"registry.terraform.io/confluentinc/confluent\"]",
      "instances": [
        {
          "schema_version": 0,
          "attributes": {
            "credentials": [],
            "format": "JSON",
            "hard_delete": false,

... redacted ...

            "recreate_on_update": false,
            "rest_endpoint": null,
            "schema": "{\n    \"$id\": \"json/product/v1.json\",\n

... redacted ...

You can see it thinks the schema is already v1, so no change needed!? Why is it not seeing the latest version from Confluent?
Seeking advice on how to troubleshoot further, or is this a bug?
Thanks

Additional - i've tried to enable debug logs to see what's actually happening ... and can see it fetching the old v0 version ...
$ TF_LOG=DEBUG terraform plan -refresh-only

... redacted ...

2023-06-19T15:48:59.671+1000 [DEBUG] provider.terraform-provider-confluent_1.39.0: Fetched Schema "lsrc-9kw3nm/rtl-lifecycle.dev.public.products-value/latest": {"id":100742,"schema":"{\"$id\":\"json/product/v0.json\", ... redacted ... ","schemaType":"JSON","subject":"rtl-lifecycle.dev.public.products-value","version":2}: schema_id=lsrc-9kw3nm/rtl-lifecycle.dev.public.products-value/latest tf_req_id=20a23ec5-c1c4-6bf1-b4f7-11b8f790e701 tf_rpc=ReadResource @caller=src/github.com/confluentinc/terraform-provider-confluent/internal/provider/resource_schema.go:572 @module=provider tf_provider_addr=provider tf_resource_type=confluent_schema timestamp=2023-06-19T15:48:59.671+1000

... redacted ...

FWIW - I'm going to assume either a bug or corrupted state and forcefully recreate it:
$ terraform apply -replace='module.topic["products"].confluent_schema.schema'

... redacted ...

Terraform will perform the following actions:

  # module.topic["products"].confluent_schema.schema will be replaced, as requested
-/+ resource "confluent_schema" "schema" {
      ~ id                 = "lsrc-9kw3nm/rtl-lifecycle.dev.public.products-value/latest" -> (known after apply)
      ~ schema_identifier  = 100742 -> (known after apply)
      ~ version            = 2 -> (known after apply)
        # (5 unchanged attributes hidden)
    }

Plan: 1 to add, 0 to change, 1 to destroy.

... redacted ...

module.topic["products"].confluent_schema.schema: Destroying... [id=lsrc-9kw3nm/rtl-lifecycle.dev.public.products-value/latest]
module.topic["products"].confluent_schema.schema: Still destroying... [id=lsrc-9kw3nm/rtl-lifecycle.dev.public.products-value/latest, 11s elapsed]
module.topic["products"].confluent_schema.schema: Destruction complete after 12s
module.topic["products"].confluent_schema.schema: Creating...
module.topic["products"].confluent_schema.schema: Still creating... [9s elapsed]
module.topic["products"].confluent_schema.schema: Creation complete after 16s [id=lsrc-9kw3nm/rtl-lifecycle.dev.public.products-value/latest]

... redacted ...

Thanks for asking @pwmcintyre and attaching all these details!
Could you also share a minimal example that reproduces an issue and what's you expected to see?
It's a little bit tricky to look into without a specific set of steps / your TF configuration. Thanks!

cc @pwmcintyre

I'm thinking that issue #318 may be the same?


[Request] Dynamic chest rendering

Could you please add the function that chests are rendered dynamically. Say for example that all chests in a 4 block radius are rendered as tile entities and not as blocks/models. Personally, I tried to check if the block looked at by the player is a chest or not. It worked almost, but I ultimately failed on the implementation of double chests.
I would love to keep the animation for the "close" chests. It may just be another option in the settings.
Thanks!

You can use https://www.curseforge.com/minecraft/mc-mods/enhanced-block-entities, which does have a feature to restore chest animations. My mod is very limited in scope, so I'm not adding any extra quality features.


Call Hierarchy always returns no results for function aliased via import as

Code Snippet
# declare.py

def func():
    return 1

# consume.py

from declare import func
from declare import func as foobar

def callByName():
    func()

def callByAlias():
    foobar()

Repro Steps

Create files as above.
Right-click, "Show Call Hierarchy" on func() call in consume.py. See that expected results are displayed.

Do same on foobar() call.

Expected behavior
Call hierarchy shows call to foobar in callByAlias
Actual behavior

Initial issue is that the declaration is an Alias and we only support call hierarchies for Function and Class declaration types.

We don't close Pylance issues until the fix ships.

This issue has been fixed in prerelease version 2023.3.41, which we've just released. You can find the changelog here: CHANGELOG.md


TABULA CSV EXTRACTION

http://www.kemet.com/Lists/ProductCatalog/Attachments/254/KEM_T2005_T491.pdf
i am facing some issues in tabula ,when extracting table from url above-

i am doing in stream mode only but if i don't select the Column header its an issue for me as i need to run a script that will extract the column data based on the Column header name. Is there any way out on this?
Also i am getting blank columns also in between in the final .CSV, although the original table doesn't have those - how to get rid of those

Hi @sidmohan
Thanks for opening this issue on Github.
There's not really a way to fix either of those issues. In Stream mode, in PDFs that, like yours, have headers that span across data rows, you have to exclude the headers. One possible solution is to make two selections: one for the data and one for the headers, and join them in your script.
For the second issue, if you're using Stream mode, there shouldn't be any empty columns. If you're using lattice mode, there isn't a way to avoid the empty columns (there are probably two lines in the PDF that touch each other). You can remove them in your final script as well.

thanks, request you to please provide solution for getting the multiplication symbol correctly from the PDF. In the preview mode the symbol appears correctly in size column of PDF but when exported it gets distorted to a hypen and strange character
IN PREVIEW MODE -

AFTER DOWNLOADING-

That's a problem in Excel, not with Tabula. Instructions like these will help you open the CSV in Excel with the right encoding: http://superuser.com/questions/280603/how-to-set-character-encoding-when-opening-excel

Hi thanks for the reply, may i know what is the backend coding platform for tabula? i need to perform some kind of automation so that it can automatically fetch a PDF from a website and convert it into an excel. Your inputs will be highly valuable

Tabula, as you can see from its source code, it's written in JRuby. The extraction algorithms are implemented in Java: tabula-java

hi all, can anyone let me know how to encode the Auto detect feature of tabula into python script. i basically want to run python script to pull out all the tables from a given PDF? i am not familiar with JRUBY and hints will be great help

Hi , can you suggest some hints on automating extraction of multiple tables from a PDF (like http://datasheets.avx.com/TAJ.pdf )


feat(cli): add health-check command

Just figured that could be helpful, especially since it works independently of the api feature being enabled.
Notes to Reviewer

As a reviewer, please pay particular attention to the following areas when reviewing this PR and tick the above boxes after you have completed the steps.
API Changes

[ ] Test the API endpoints which were added/changed.

CLI Changes

[ ] Test health-check command.

Docker

[ ] Test added health check command in Dockerfile.

This is for docker, I assume? Wouldn't we define healthy for docker differently than our normal healthy?

This is for docker, I assume? Wouldn't we define healthy for docker differently than our normal healthy?

Yeah, for Docker and for a api feature independent way to check the health of Chronicle. I don't know whether we should have different kinds of health tbh :thinking: Wouldn't that be confusing. If you have a container that would be dependent on inx-chronicle you don't have to let it wait for it to become healthy. On the other hand if your container only wants to work with a healthy inx-chronicle than you have a way to make sure of it.

Iirc, Hornet also has two different definitions of healthy. With the current proposal, if Chronicle takes 3 days to fully sync, it would remain unhealthy throughout the entire time (while certainly some routes could be queried already).

As I said in my comment ... no one is forcing you to wait until the inx-chronicle container reports healthy.
I am open for ideas around other definitions of healthy that are better suited here.

I think for the purposes of docker containers, I see no value in discerning between a synced and unsynced instance. I would consider chronicle healthy as a docker instance if it is running all the configured workers. Am I missing something?

I would agree with @Alexandcoats here—a good metric would be if all workers started correctly. Ideally, we would take this one step further by defining a condition per worker, such as INX or DB connections estabslished. But at that point we probably have to introduce a something like a worker trait with corresponding init and run functions. I'm not sure if that's worth it for now?

I think it's doable without that much boilerplate. At the most basic we could just have a Arc<Mutex<HashMap<String, bool>>> in the main thread and add each worker keyed by name when we spawn them. Then they could set the flag to true when they are ready. Just one option 🤷

Closing this for now.


call_and_wait should add a delay between calls to request_status_raw

        loop {
            match self.agent.request_status_raw(&request_id).await? {
…
                RequestStatusResponse::Unknown => (),
                RequestStatusResponse::Received | RequestStatusResponse::Processing => {
                    // The system will return Unknown until the request is accepted
                    // and we generally cannot know how long that will take.
                    // State transitions between Received and Processing may be
                    // instantaneous. Therefore, once we know the request is accepted,
                    // we restart the waiter so the request does not time out.
                    if !request_accepted {
                        waiter
                            .restart()
                            .map_err(|_| AgentError::WaiterRestartError())?;
                        request_accepted = true;
                    }
                }
…
            };

will very aggressively hammer the read endpoint, possibly starving that node. [There should be some delays here.]

It seems we are using the Waiter to throttle. Are you using the proper Waiter instance?
https://github.com/dfinity/agent-rs/blob/next/ic-agent/src/agent/mod.rs#L603-L605
            waiter
                .wait()
                .map_err(|_| AgentError::TimeoutWaitingForResponse())?;


Add support for shift+enter when sending messages

As in https://github.com/fencex/vue-beautiful-chat/commit/a7dea6b2b77621daf5561d0c5bba37288def4b5f#diff-fa51824538aae04090950ca4d4739ab2L77

Already here, closing.


Port 80 is normal, not port 80 returns 404

Port 80 is normal, not port 80 returns 404

The hostname and port are part of a docker tag.  the image without the 80 does not exist. If you wanted this to work you could do:
docker tag  docker.oneqing.com:80/root/test:5 docker.oneqing.com/root/test:5
docker push docker.oneqing.com:80/root/test:5

Then your pull should work.

Closing as wildly outdated.


70.6版本导出调用外部库元件的切图空白

所有调用的外部库元件中的切片，再导出会变成透明的空白图片。
png、svg等格式导出均变为透明的空白图，大量的组件导出后无法正确输出切图文件 T.T
希望可以解决下这个问题，谢谢🙏

I can tell it's a bug of Sketch, is it good for versions before 70.6?

It's seems good at the older version of sketch, i think you are right, looks like we only can wait the official fix of this bug, but till now the official dont found this bug i think ┐（─__─）┌


[PLC] [AST] [Builtin] Extensible builtins

This is the same as #1212, except it takes a commit from the middle of the history of that PR. The intention is to finish the MVP that we had without doing additional research. I'll close the previous PR, but we may need it in future, so I decided to open a new one just not to mess everything up.
Do not review.
dang, I forgot to make a draft pull request again

Implemented in #1849 (modulo unlifting, which we decided we don't need).


Adding restart action to launchd resource

At first we wanted only support native actions with the launchd resource and launchd does not have the concept of restart. You would unload and load. But not having a restart was causing more confusion to people.

I'm in favour of this.

@mikedodge04 you need to add the DCO sign off to your commits, please.


Go through all blog posts looking for any galleries, and switch them to Flickr

Tedious, yes, but needs must.

Look at this again once this issue on Wyam is implemented: https://github.com/Wyamio/Wyam/issues/359

Search/replace for this <div id="flickrembed"> once implemented

This has been completed.


NIFI-13790: Report SEND provenance event in PublishKafka

Summary
NIFI-13790 This PR reports a SEND event for each FlowFile successfully published and transferred to success. I was hesitant to add a getTransitUri() method to the KafkaConnectionService API but I certainly can in order to give a more informative URI, and could add other methods if need be to generate an informative description for the provenance event. I could also wrap the new code in a try-catch catching and logging an error versus rolling back / failing because the FlowFiles will have already been transferred.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[x] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

We definitely want a weak send event rather than no send event.  But yeah we should revisit.


FIX-#4017: Fix OmniSci engine enabling for IO functions

Signed-off-by: Alexander Myskov<EMAIL_ADDRESS>
What do these changes do?
The problem is that when FactoryDispatcher is imported, command Engine.subscribe(FactoryDispatcher._update_factory) is executed before Engine config correction in Engine.subscribe(_update_engine), that leads to incorrect factory search. To fix it we need to update Engine before FactoryDispatcher import.

[ ] commit message follows format outlined here
[ ] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[ ] signed commit with git commit -s
[ ] Resolves #4017
[ ] tests added and passing
[ ] module layout described at docs/development/architecture.rst is up-to-date

Hi @modin-project/modin-core , please take a look at this PR.


Issue Loading Organization Sign-In Page

Storage Explorer Version: 1.4
Platform/OS Version: MacOS 10.13.6
Architecture: x64
Regression From: This has never worked for me
Steps to Reproduce:

Attempt add a new Azure Account
Type in company email address
Select Work Account
Be redirected to Organization login page to finish authentication

Expected Experience:
The user is successfully directed to their organization login page and are able to authenticate
Actual Experience:
The sign in windows loads a blank page

Hey @martygriffin , do you happen to use VS for Mac?

I am have it install and have not used it for awhile.

Hi @MRayermannMSFT  and @martygriffin ,
I am having the exact same issue in a fresh install of MASE. I am running Ubuntu 16.04, all dependencies installed. The steps to reproduce @martygriffin's error are the same. I launch the explorer in the terminal and get the following output:
Gkr-Message: couldn't connect to dbus session bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
Gkr-Message: couldn't connect to dbus session bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
Received message 'get-dialog-result'
Gkr-Message: couldn't connect to dbus session bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix")
Received message 'dialog-ready'
Received message 'dialog-execute-operation'
Received message 'did-execute-operation'
Received message 'dialog-execute-operation'
Received message 'did-execute-operation'
Received message 'dialog-execute-operation'
Received message 'did-execute-operation'
Received message 'dialog-execute-operation'
Received message 'did-execute-operation'
Received message 'dialog-execute-operation'
Received message 'did-execute-operation'
Received message 'dialog-execute-operation'
Received message 'did-execute-operation'
Received message 'dialog-did-init'
Received message 'dialog-execute-operation'
Received message 'did-execute-operation'
Received message 'dialog-execute-operation'
Received message 'did-execute-operation'
Received message 'dialog-execute-operation'
Received message 'did-execute-operation'
Received message 'dialog-execute-operation'
Received message 'did-get-dialog-result'
Received message 'closed'

Any clue on what's going on?
Cheers,
Arturo

@martygriffin , can you try signing in with VS for Mac? If you're able to successfully, then leave the accounts signed in, reopen Storage Explorer and you should see them there.

@MRayermannMSFT That worked! This is a good work around for now but will it work from Storage explorer in the future?

We do not have any plans to get rid of this workaround. Electron has some odd quirks when it comes to redirecting, so we very much value sharing the same account store as VS/VS for Mac, as it lets users such as yourself become unblocked. In the future, we'll also be adding code flow sign in as an additional fallback.

@arturOA , as for your issue, I'm definitely concerned by that Gkr-Message: couldn't connect to dbus session bus: Could not parse server address: Unknown address type (examples of valid types are "tcp" and on UNIX "unix"). Do you have any idea what that is about? I'm not a Linux expert and searching for that error message isn't revealing anything obvious.

@MRayermannMSFT , I got rid of that message by running StorageExplorer with the "--disable-gpu" option. Still after trying to sign in I get the same blank page. The output when running StorageExplorer is the same except without the watning messages:
Received message 'get-dialog-result' Received message 'dialog-ready' Received message 'dialog-execute-operation' Received message 'did-execute-operation' Received message 'dialog-execute-operation' Received message 'did-execute-operation' Received message 'dialog-execute-operation' Received message 'did-execute-operation' Received message 'dialog-execute-operation' Received message 'did-execute-operation' Received message 'dialog-execute-operation' Received message 'did-execute-operation' Received message 'dialog-execute-operation' Received message 'did-execute-operation' Received message 'dialog-did-init' Received message 'dialog-execute-operation' Received message 'did-execute-operation' Received message 'dialog-execute-operation' Received message 'did-execute-operation'

Are you behind any proxies @arturOA ?

Hi @MRayermannMSFT ,
I am not sure if I was. I am trying to connect from home now. I do get my company's sign-in page and I am able to type in my password. However now the "Waiting for authentication ..." window seems to work for a while. Then it disappears without me being able to sign-in ...

@martygriffin , can you try signing in with VS for Mac? If you're able to successfully, then leave the accounts signed in, reopen Storage Explorer and you should see them there.

This solution worked for me on Windows. Thanks!

Hello to anyone watching this issue! In 1.7.0 we're introducing the ability to optionally sign-in via device code flow, or as you may be more familiar with, copying a code from Storage Explorer into a page in your browser, and then signing in via your browser.
We're currently previewing that feature in our insiders channel. If y'all are intereseted in trying it early, simply go to Help -> Opt-in To Insider Builds.

Then, go to Help -> Check for updates, wait for 1.7.0 to appear, and install it. Then once you install it, you can enable device code flow by going to Experimental -> Use Device Code Flow Sign In.

And then let us know if things work out for y'all! This issue of blank sign in dialog is one we've been fighting for the longest time, and we're hoping this method of sign-in will unblock y'all. 😄

Closing. If anyone is still having issues, we now recommend trying out MSAL. MSAL is a newer auth library we'll be switching to in the future. Today though, you can opt into it by going to Settings -> Application -> Sign in -> Use Microsoft Authentication Library (MSAL)


Rename GStraccini-bot.jpg to gstraccini-bot-social.jpg

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[ ] No

ℹ Additional Information

@gstraccini codacy bypass


EmptyList constructor without using null

Summary
How did y'all construct an EmptyList without null values being allowed?
Description
My EmptyList extends InfiniteListImpl so I created an InfiniteListImpl constructor as shown below. But every time I submit, I'm hit with 'null values detected'. How did y'all work around this?
Also, does using Lazy.ofNullable(...)  trigger the null value error in the grader too?
Screenshots (if any):

Thanks!!

My EmptyList does not have any fields. It just returns itself for intermediate operations

You do not need any attributes for the EmptyList class so there is no need for a constructor for it as well

hmm so my InfiniteListImpl has 1 constructor with the head and tail as arguments. If I choose to not have a constructor for EmptyList I get an error:

did y'all extend your EmptyList from InfiniteListImpl too? or did y'all just make EmptyList implement InfiniteList?

EmptyList should implement from InfiniteList instead of InfiniteListImpl because you only need the abstract classes that define the EmptyList methods and not the attributes from the InfiniteListImpl if that make sense.

oh that does make sense! Thanks:))