Autcomplete problematic behaviour (iPad)

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Autocomplete panel floating either up or down the search bar (iPad)
What is the current behavior?
Autocomplete panel floats above search bar on scroll
What are the steps to reproduce?
Link to home-made repository: https://github.com/LMalorgio/material-autocomplete-malfunction
Some examples come from AM's website itself: https://material.angular.io/components/autocomplete/examples
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
@angular/material": "^5.2.4"
Chrome 65.0.3325.152
Is there anything else we should know?
Full description of the issues are provided in my repository
/cc @davinkevin @Neonox31

Not directly related to your issu, but as you're sharing a repo, it's possible to run github public angular repos directly at stacklitz. In this case: https://stackblitz.com/github/LMalorgio/material-autocomplete-malfunction

I think this is a duplicate of #16004, but feel free to comment if that's not the case


Test - fix forecast calibration test to round appropriately

Due to a problem described here, it would be beneficial for the test to round the db value to 3dp to compare like for like with the FTP value

need to re-do this PR as contains a jupyter notebook


Native builds don't log warnings unless the build fails.

Expected Behavior
If the compiler detects a warning, the user should be notified in the build, even if the build is succesful
Current Behavior
If a warning is generated, it is suppressed unless the build fails. This has been discussed before on the forums
Steps to Reproduce (for bugs)
If running on windows,

int main()
{
	return 0;
}

and build.gradle

model {

    components {
        Main(NativeExecutableSpec) {
            sources {
                cpp{
                    source{
                        srcDirs "src"
                        include "**/*.cpp"
                    }
                }
            }
        }
    }

}

generates a warning by default with MSVC 2013:
warning C4530: C++ exception handler used, but unwind semantics are not enabled. Specify /EHsc
If the compile is successful, the warning is suppressed. remove the semicolon from return 0 and the build fails, but also logs the warning.
Your Environment
Gradle 3.1
Windows 10 pro x64
MSVC 2013 and 2015.
Can repro on debian jessie using gradle 3.1, with gcc 4.9 using a more complex example.

Thanks @donalmacc for the report. To comment further on this issue, this can be seen as a regression when BuildOperationLogger was introduced but that modification was a necessity to fix interleaving logs from the parallel compilation. The current workaround would be to use /WX flag which "treats all warning as errors" for both the compiler and the linker.

To provide a little bit of history, this was something we did intentionally (to hide all output from the underlying tool) until we could find something better. The output was pretty useless when it was all jumbled together.

We could probably print an excerpt of the output (just like when there is an error) all the time. This way we could at least see some warning. Better modeling could be done to grab and display the warning message in a more IDE friendly output. However, there is also the possibility to improve the integration of Gradle with Visual Studio. @donalmacc, could you comment on the use case for log warning? Are they needed in the context of the CLI or the IDE?

Hi Daniel
Apologies for the delay getting back to you. I would argue that a jumbled/interleaved output is better than no output whatsoever. If I put a print statement in a closure and its executed in parallel, It's logged, albeit not in order. I'm actually quite shocked that the solution to this was to suppress the output of the tools.
@lacasseio Ideally we would have it working in both IDE and CLI builds. We have developers using the IDE for builds, and we have CI machines using CLI builds. If the developers and CI builds are receiving different information, that's not good.

Good point @donalmacc. The discussion we had internally was to show compiler output on the console, but not interleave the output of parallel compilations. Gradle should collect the output for a compiler invocation as it does now, and write it to the console atomically once the compiler invocation has completed.

Ideally it would output in-progress, (of course), but having it all flushed after the compile has completed would do perfectly.

Issue moved to gradle/gradle-native #495 via ZenHub


The directory or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.

I am receiving the following message when trying to install virtualenv via: sudo pip install virtualenv
Bryans-MacBook-Pro:/ BryanB$ sudo pip install virtualenv
The directory '/Users/BryanB/Library/Caches/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/Users/BryanB/Library/Caches/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.

Tried sudo -H pip install virtualenv
Result
Requirement already satisfied: virtualenv in /Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages
I am unsure how to resolve this issue.

Is it an issue?  Virtualenv is installed in your framework Python site-packages directory.  If your $PATH contains the bin directory of that installation, you should be able to run the virtualenv command now.

When trying to run $ mkvirtualenv TestEnv
Result -bash: mkvirtualenv: command not found
Per this tutorial: http://newcoder.io/begin/setup-your-machine/, my response should be something like:
$ mkvirtualenv TestEnv
Installing
distribute..........................................
....................................................
....................................................
...............................done.
virtualenvwrapper.user_scripts Creating /Users/lynnroot/Envs/TestEnv/bin/predeactivate
virtualenvwrapper.user_scripts Creating /Users/lynnroot/Envs/TestEnv/bin/postdeactivate
virtualenvwrapper.user_scripts Creating /Users/lynnroot/Envs/TestEnv/bin/preactivate
virtualenvwrapper.user_scripts Creating /Users/lynnroot/Envs/TestEnv/bin/postactivate
virtualenvwrapper.user_scripts creating /Users/lynnroot/Envs/TestEnv/bin/get_env_details

And should have a virtual environment called TestEnv, which I should see (TestEnv) before your prompt:
(TestEnv) $
But, obviously I am not... Not sure why. First time doing much of anything with Python. Would like to test out writing scrapers
Thanks

mkvirtualenv is a command provided by virtualenvwrapper, a set of helpers built on top of virtualenv.  Either run pip install virtualenvwrapper (no need for sudo), or use “bare” virtualenv commands instead of mkvirtualenv.

This happens when someone installs a package via pip using sudo in a virtualenv.
You can update the permissions by using following commands as referred in link-
$ sudo chown -R USERNAME /Users/USERNAME/Library/Logs/pip
$ sudo chown -R USERNAME /Users/USERNAME/Library/Caches/pip


Pass the file as a URL and not as a blob

It would be nice and more efficient to have the option of passing the file as a downloadable url rather than a blob so I don't have to download the file in my application server and then send it to unstructured (this is more efficient and avoids the 32mb body limit on some hosting platforms.

@theZaX , I see the motivation. But, I have doubts from a free-internet hosted API provider perspective that allowing downloads from arbitrary locations on the internet is something that should be supported, or at least be the default.

Yeah although it seems that accepting arbitrary files from an open api could be just as bad. I'm not sure the extra download step is what would be causing the security vulnerability.
The image captioning api by Microsoft is an example of an api that accepts these sorts of parameters. They accept a file blob or a source url.
https://westcentralus.dev.cognitive.microsoft.com/docs/services/computer-vision-v3-2/operations/56f91f2e778daf14a499f21b

Strictly speaking, it's at least as bad as accepting any arbitrary files, because of any potential network-related vulnerabilities or additional networking costs on top of whatever payload you finally end up with.
The Microsoft example has a key to identify the user, it seems: -H "Ocp-Apim-Subscription-Key: {subscription key}", so not 100% analogous.
That said, I see no reason not to add a sample notebook to the repo that allows this.

Awesome, I plan on self hosting anyway so I'll implement the code in the sample notebook in my fork.
Thanks for all the help!

A bit more detail on the opinionated way to add a custom API for "pipeline-" repo such as this.
The idea is to basically add an additional pipeline- notebook and then rely on code generation to create the new API (leaving the existing/upstream /general/v0.0.4/general in tact). The steps are:

create a new pipeline-<name>.ipynb notebook in pipeline-notebooks/ . A copy of that pipeline_api in that notebook gets invoked when the new endpoint /general/v0.0.4/<name> is invoked (after the last step in this list).
add an m_url kwarg to the pipeline_api signature. This becomes an optional multi-string form variable url (yes, without the m_-prefix) in the generated API. It must have a default, which could be empty list. You can add an arbitrary number of m_ params that follow the same convention.
of course, update the pipeline_api function to do the right thing with the m_url kwarg
From a the root of the repo, call make generate-api to create the <name>.py module in https://github.com/Unstructured-IO/unstructured-api/tree/main/prepline_general/api

That's it! You could also delete https://github.com/Unstructured-IO/unstructured-api/blob/main/prepline_general/api/general.py which would leave your new api as the only one accessible, and viewable under https:///docs.
Note that there are additional make targets to make sure that notebooks checked in are clean and have reproducible outputs:
make tidy-notebooks and make check-notebooks
and one to make sure that generated API's match the pipeline-notebook:
make api-check
that you could tweak (if needed) for your environment.
Finally a note on the version, it's automatically inferred from here during make generate-api

Hi @theZaX - going to close this old issue as it sounds like you implemented what you needed.


Uncaught Error: ENOENT, open '/Users/non/Repositories/ohmpieng/production/config/omniauth.rb'

[Enter steps to reproduce below:]

...
...

Atom Version: 0.169.0
System: Mac OS X 10.10.2
Thrown From: Atom Core
Stack Trace
Uncaught Error: ENOENT, open '/Users/non/Repositories/ohmpieng/production/config/omniauth.rb'
At /Applications/Atom.app/Contents/Resources/app/node_modules/pathwatcher/node_modules/q/q.js:126

Error: ENOENT, open '/Users/non/Repositories/ohmpieng/production/config/omniauth.rb'
  at Error (native)

Commands
  4x -0:02.6 pane:show-previous-item (atom-text-editor.editor)
 11x -0:00.0 pane:show-next-item (atom-text-editor.editor)

Config
{
  "core": {
    "themes": [
      "atom-dark-ui",
      "monokai"
    ]
  },
  "editor": {
    "fontFamily": "Menlo",
    "fontSize": 12,
    "invisibles": {}
  }
}

Installed Packages
# User
monokai, v0.8.0

# Dev
No dev packages

[Enter steps to reproduce below:]

...
...

These things are there so you write in steps. Please do so.

I believe this is duplicate of https://github.com/atom/atom/issues/4850 and will be fixed by #4906


Fix generate CRD script by using copy

Using a symlink to point inside the staging directories seems to confuse newer versions of sigs.k8s.io/controller-tools.
Specifically, the set up of a temporary directory for controller-gen uses a symlink into staging/api, which confuses something in golang to use ${ROOT}/staging/api/go.mod rather than ${ROOT}/go.mod, causing it to return errors that indicate go.mod inconsistencies with vendor/modules.txt. This would normally be fixable via the go mod commands, but looking at the wrong go.mod is not a normal situation.

/lgtm
/approve

/test e2e-gcp-olm

/retest

/retest

/test images
/rest periodics-images

/test periodics-images


Problem addons in VS 2015 / Windows 10

Hi, I have this error:
Severity    Code    Description Project File    Line    Suppression State
Error   C4716   'ofxEdsdk::Camera::close': must return a value  mySketch
c:\users------\of\windows\of_v0.9.0_vs_release\addons\ofxedsdk\src\ofxedsdk.cpp  109

Here are the settings we tried and it worked, we hope it helps.
Greetings from Filika/Istanbul
openframeworks version: 0.9.8
operating system: win10
openframeworks addon: https://github.com/kylemcdonald/ofxEdsdk
canon sdk version: 3.5.0 (download from Canon Developer Site)
1-Move the EDSDK Folder contents inside /addons/ofxEdsdk/libs/
2-Create new project with ProjectGenerator & include ofxEdsdk
3-In Solution Explorer Window Right Click on Project Name > Go to Properties
4-Edit ofApp.h and add #pragma comment (lib,"../../../addons/ofxEdsdk/libs/EDSDK/Library/EDSDK.lib") after #pragma once
5-Compile the project, if you get The Code Execution Error do the following
copy the EDSDK.dll under /addons/ofxEdsdk/libs/EDSDK/Dll and paste to your project's bin folder


added a c program to arrange a rank table of students by their marks

Title
added a c program to arrange a rank table of students by their marks
Checklist:

[] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

DSA code are not acceptable


Private Message Rooms

The private message rooms were not yet implemented and need to be added correctly.

Inheritance from Room Class
Adding message options with the recipient
Options to get the recent messages

Base class added in 3f8acb4


Fix Cassandra warning in Realm Management logs

Remove aggregation query without partition key Cassandra warning when checking Astarte's health.
See #420.

Missing changelog entry here.

@bettio done!


gnomeExtensions.EasyScreenCast: 1.4.0 -> 1.7.0

Description of changes
https://github.com/EasyScreenCast/EasyScreenCast/blob/master/CHANGELOG.md#v170-46-2022-08-26
Things done

[x] Tested core functionality on gnome 43

Thanks for the update! I trust your test on this one.


lib: export rustix too

This exports our specific version of rustix. It is part of our
public API, e.g. through OwnedFd usage.

Hmm in theory I think we should have fixed this by using the types from https://docs.rs/io-lifetimes/latest/io_lifetimes/ which are just re-exported by rustix.
But...OTOH the consumers of this crate today are themselves going to be using rustix, and I don't think we want multiple versions of rustix in the depchain.
Perhaps what we should do actually is argue for cap-std exposing rustix (under a feature flag?)

This crate and std-cap do use different OwnedFd, see https://docs.rs/cap-std-ext/0.1.1/cap_std_ext/cmdext/trait.CapStdExtCommandExt.html#tymethod.take_fd_n.
So while cap-std may wish to re-export io-lifetimes (but I think they actually want to wait for stdlib, see https://github.com/bytecodealliance/cap-std/issues/192#issuecomment-1008970704), it won't help here as the method above uses a different one. (That's at least my understanding, I started looking at this just this morning as the ostree-rs CI was broken)


New avalonia project from template does not run

When creating a new application using the Avalonia template the sample application does not run.
To Reproduce

Generate a new avalonia project using the template (dotnet new avalonia.app -o MyApp)
move to the project directory (cd MyApp)
Run the app (dotnet run)

Expected behavior
The application runs
Actual behavior
The application crashes with error message below. See enclosed logs.
The author primary signature's timestamp found a chain building issue: UntrustedRoot: self signed certificate in certificate chain
Desktop:

OS: Kubuntu 20.10
Version 0.10.0

error.log

actually it's related to this
https://github.com/NuGet/Home/issues/10491

actually it's related to this
https://github.com/NuGet/Home/issues/10491

Actually keeping this open in case other people try to follow the avalonia tutorial.

Actually keeping this open in case other people try to follow the avalonia tutorial.

Following this proposed solution by Ubuntu security team fixes the issue.


UX meeting agenda July 20, 2016

Agenda:
TABS / OPEN EDITORS

Closing dirty editor prompts for save (even when there are other duplicate editors open) #8983
Debug toolbar in the way of tabs (move it?) #9093
Quick Open preview, launched editors should stay open #8939
"Left, "right," "center" take up too much space?  Need to validate this #8961

OTHER

San Fransisco font issues in Chromium (upstream problem affecting UX) #9152
Hard to place caret before first character when folding enabled.  #8871

Please feel free to add items to discuss.

Add support for opening multiple project folders in same window? https://github.com/Microsoft/vscode/issues/396

Provide an option to hide the activity bar https://github.com/Microsoft/vscode/issues/1105

Git: status bar lacks indication that it will push #8655

NOTES
Closing dirty editors prompts for save
We agree that we should no longer prompt users to save when they close a dirty editor if there is another duplicate editor still open.  However, we will wait until the August milestone to get one last team member’s feedback before making the change.
Debug toolbar in the way of tabs
We could do a quick fix on this, but what we’re seeing is just one manifestation of a larger problem: our global commanding patterns need some work, as does our debugger UX.  We will instead move to improve these overall experiences, which will in turn address this problem.
Quick Open preview, launched editors should stay open
We agreed that this is a compelling interaction need.  We considered how competing products handle this, and we found that Visual Studio IDE handles it best.  We will apply the relevant Quick Open interaction patterns:

Quick Open will now create a preview editor (or leverage the already open one, if one exists) to show you a preview of each item you focus on in the list.
Upon hitting Enter, the open editor will be kept open (i.e. no longer preview).
If a user hits Esc or clicks away from the Quick Open command palette…

The command palette will close and whichever open editor you were on before previewing will be brought back in view.
The preview editor will remain and whatever you last were looking at in Quick Open will also remain inside that preview editor—exception: if you were on a preview editor when you opened Quick Open and previewed some items but never selected any (e.g. escaped out of it) then we will restore whatever you previously had open in that preview tab, since that would indicate that the preview tab is likely something you want to keep and not replace.

"Left, "right," "center" take up too much space?
We can experiment with using horizontal lines, but there would be a necessary tradeoff: we couldn’t have group actions in the Open Editors list.
We asked ourselves, “Should we even have the Open Editors list at all, if tabs are enabled?”  Aren’t tabs sufficient to substitute for Open Editors?  Some on our team are actually closing the Open Editors list to get it out of the way as they use tabs.  We should validate if users would also fare better without them when tabs are enabled.
However, this is not a fix-all solution, because Open Editors are useful at least with tabs-disabled. And there’s another problem that needs addressing:  As you open files from the Explorer viewlet, the Open Editors list grows (and this growing is magnified as you open files to the side in new groups, as 2 lines are added each time that happens: one for the file and one for its group label, such as “right”).  This can be distracting and pushes the tree of files in your project down, which misaligns where you were previously with your cursor.
Possible solution for that: make Open Editors closed by default and/or allow it to be resized by users so that it stays fixed in its height and does not push things down.
Git: status bar lacks indication that it will push
We decided that first time users could be in for a nasty surprise if we don’t prompt them about what they are about to do by clicking on this for the first time.
The prompt could say “Sync will pull then push.  Are you sure you want to sync?” followed by a “Sync” “Cancel” and with a checkmark that says “Don’t ask again.”  Clicking the checkmark and continuing with a sync then would programmatically change the User Settings to never prompt again.
We will first run this by @joaomoreno
Watch task indicator
Showing when an ongoing task is both active and inactive is a great idea.  @dbaeumer can handle this.
We also should holistically look at our tasks UX, as it could be improved.  Or should we just rely on the integrated terminal and not have tasks?
OTHER
We unfortunately did not have time to discuss other issues during this meeting.  We will address those separately or in a future meeting.

+1 on the solution for the git status bar prompt. My idea is to implement it once we can programmatically change configurations (coming soon by @bpasero). The reason for that is that we already have a setting git.confirmSync, which we could leverage for the implementation... creating the chance for any user that clicked Don't ask again and regretted it to go to his settings and enable that confirmation dialog again.


[WIP][SPARK-34056][SQL][TESTS] Unify v1 and v2 ALTER TABLE .. RECOVER PARTITIONS tests

What changes were proposed in this pull request?

Port DS V2 tests from AlterTablePartitionV2SQLSuite  to the test suite v2.AlterTableRecoverPartitionsSuite.
Port DS v1 tests from DDLSuite to v1.AlterTableRecoverPartitionsSuiteBase.

Why are the changes needed?
To improve test coverage.
Does this PR introduce any user-facing change?
No
How was this patch tested?
By running new test suites:
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *AlterTableRecoverPartitionsParserSuite"
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly *AlterTableRecoverPartitionsSuite"

Test build #133874 has started for PR 31105 at commit a1d76a5.

Test build #133874 has started for PR 31105 at commit a1d76a5.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38464/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38464/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38464/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38464/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38464/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38464/

Test build #133874 has finished for PR 31105 at commit a1d76a5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #133874 has finished for PR 31105 at commit a1d76a5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133874/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133874/

@cloud-fan @HyukjinKwon Could you review this PR, please.

@cloud-fan @HyukjinKwon Could you review this PR, please.

Test build #134165 has started for PR 31105 at commit 2188790.

Test build #134165 has started for PR 31105 at commit 2188790.

Test build #134166 has started for PR 31105 at commit c0367aa.

Test build #134166 has started for PR 31105 at commit c0367aa.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38749/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38749/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38749/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/38749/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38749/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38748/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38749/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38748/

Test build #134166 has finished for PR 31105 at commit c0367aa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #134166 has finished for PR 31105 at commit c0367aa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134166/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134166/

Test build #134165 has finished for PR 31105 at commit 2188790.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #134165 has finished for PR 31105 at commit 2188790.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134165/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134165/

thanks, merging to master!

thanks, merging to master!


Add trailing comma support in macros

This resolves #16. I've added tests as well.

Great, LGTM :)
My only concern is not about this PR but the the format of the readability of the expected strings in the tests: https://github.com/colin-kiegel/rust-pretty-assertions/blob/master/tests/assert_eq.rs#L6-L18
I mean, it tends to be hard to review and to write. Anyway, if anything has to be done, probably worth to open another PR.
I let @colin-kiegel (the owner of this crate) do the official review :).
Florent

My only concern is not about this PR but the the format of the readability of the expected strings in the tests: https://github.com/colin-kiegel/rust-pretty-assertions/blob/master/tests/assert_eq.rs#L6-L18

I opened issue #18 for this.
Florent

@Enet4 Great, thank you for your contribution!
And thanks @fflorent for your feedback. ;-)


Bug Fix Update VesselValueImprover.cs

The code as written tried to build a list of existing parts, but the key for the list was not unique across different parts. What would happen is the first part would set the value of all subsequent parts in the same vessel, i.e. all reaction wheels got the same value.  But if the first reaction wheel is a small probe core, then it nerfs the reaction wheels of the entire vessel. Also added a debug message to check the values were calculated correctly.

Don't use this PR. The fix worked in the case of a single vessel, but fails miserably with multiple docked vessels. It iterated through all the parts for all vessels for each docked vessels.


Fix trivial workding issues.

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Fix some wording issues my linter is complaining about.
Which issue(s) this PR fixes:
N/A, very tiny change.
Does this PR introduce a user-facing change?:
NONE

/retest

It looks like something is broken with GRPCRoute:
GRPCRoute in version "v1alpha2" cannot be handled as a GRPCRoute

I'll try and take a look at this soon when I have some cycles.

Thanks @briantkennedy! It looks like you need to run make generate to get tests to pass, otherwise LGTM.

@briantkennedy since we hadn't heard back from you in a while, I added the one-line missing from the make generate so we could go ahead and get this merged. Thank you for your contribution!


Integer arguments to integrate 1d invoking wrong version

Summary:
Trying to call the prim version of integrate_1d with integer arguments invokes the wrong version of the function.
Description:
Because the reverse mode integrate_1d is most templated, if the limits are not doubles (but are not vars and theta is not vars), the compiler tries to use the reverse mode version of integrate_1d anyway.
Reproducible Steps:
Add the code:
TEST(StanMath_integrate_1d, test_integer_arguments) {
  double out = stan::math::integrate_1d(f2{}, 0, 1, std::vector<double>(),
                                        {}, {}, msgs, 1e-6);
}

To test/unit/math/prim/arr/functor/integrate_1d_test.cpp
And run ./runTests.py test/unit/math/prim/arr/functor/integrate_1d_test.cpp
Current Output:
Fails at compile:
test/unit/math/prim/arr/functor/integrate_1d_test.cpp:288:10: error: no viable conversion from 'stan::math::var' to 'double'
  double out = stan::math::integrate_1d(f2{}, 0, 1, std::vector<double>(),

Expected Output:
Passes
Current Version:
v2.17.0

Closed via #936

I'm guessing this may be a problem for a lot of our code.  It would be nice if int to double were a cheaper promotion than int to var, but it's not.
I'm guessing a lot of our functions suffer this problem.  I'm going to open a discussion on Discourse.


IES - backfacing hemisphere data is discarded

Here is an IES file (renamed as .txt) which contains light distribution data for both hemispheres: L2783.txt. The classic Lux uses the full data, while with LuxCore light is emitted only from the forward-facing side of mesh light.
It also would be nice to have an option to enable double-sided light emission. In case of using IES files, the "forward" hemisphere could be used for both directions, when double-sided emission is enabled.

But it is the way old Lux was working too or have you found a difference ?
You need to use a point light source if you want light to be emitted backward (otherwise is blocked by the mesh surface).

The old Lux supports backward facing light distribution in IES files.
Lux classic:

LuxCore:

Blend file:
luxcorerender-double-sided-light.zip

I should have fixed the problem:

Great, thank you! Any plans for the double-sided light emission option (without IES)?

Not at the moment but you can use an image instead of IES too. So if you use a 1x1 pixels white image, you will obtain the same result of a double sided are light (or constant IES map).


Calculating BASE SHA on master branch

Orb version:
<EMAIL_ADDRESS>What happened:
I used nx/set-shas in my circleci deployment but by default by main branch is master and in circleci I always get Base SHA as null and the below error.
undefined:1
{:message "Project not found"}
 ^

SyntaxError: Unexpected token : in JSON at position 1
    at JSON.parse (<anonymous>)
    at IncomingMessage.<anonymous> (/home/circleci/project/index.js:138:35)
    at IncomingMessage.emit (events.js:412:35)
    at endReadableNT (internal/streams/readable.js:1334:12)
    at processTicksAndRejections (internal/process/task_queues.js:82:21)

Expected behavior:
Ideally it should return the Base SHA and Head SHA
Additional Information:
Here is the output of the Circleci
undefined:1
{:message "Project not found"}
 ^

SyntaxError: Unexpected token : in JSON at position 1
    at JSON.parse (<anonymous>)
    at IncomingMessage.<anonymous> (/home/circleci/project/index.js:138:35)
    at IncomingMessage.emit (events.js:412:35)
    at endReadableNT (internal/streams/readable.js:1334:12)
    at processTicksAndRejections (internal/process/task_queues.js:82:21)

Base SHA

Head SHA
4d59078a4a8c8e82c4a47a1258fa74064a1be522

NX_BASE and NX_HEAD environment variables have been set for the current Job
CircleCI received exit code 0

Any assistance you can provide is much appreciated. Thanks!

@vikaskumr did the suggestion from @chiubaka solve your issue?


Improve Footer Dark 15

footer-dark-15.html
a) Fix bg color as per figma design (bg-slate-900 to #0A142F)
b) add padding bottom as shown in figma design
Existing footer

Figma Footer

Please assign this to me @Rajdip019 @rupsha014

@RavindraP04  go ahead


Firejail fixup

[x] Fixup mutt profile

Enable R/O access to the home directory
Enables R/W access to the maildirs

[x] Fixup weechat profile (R/O access to ~/.dotfiles/weechat)
[x] Harden weechat profile by using directory whitelisting (no more access to the whole homedir)

Thanks to @gmcclins for the report.

@hashbang/administrators This is impacting users right now, so please review ASAP.


Update Catalyst to use the toml module in PennyLane

Context:
This PR is part of the new device capabilities initiative to improve feature parity between PennyLane and Catalyst. See this ADR for more context. A new TOML schema has been defined, and the relevant module implemented in PennyLane:

https://github.com/PennyLaneAI/pennylane/pull/6407
https://github.com/PennyLaneAI/pennylane/pull/6433.

Description of the Change:

Removes the toml module and replaces all usages with imports from PennyLane
Update usages of DeviceCapabilities to match the new definition in https://github.com/PennyLaneAI/pennylane/pull/6407/
Removes use of about-to-be-deprecated return_type of MPs
Updates all TOML files to use the new schema.
A PR opened in Lightning https://github.com/PennyLaneAI/pennylane-lightning/pull/988 that updates the TOML file of the Lightning devices to the new schema.

Benefits:
A step towards feature parity between PennyLane and Catalyst.
Possible Drawbacks:

Per discussions when the ADR was developed, operator.gates.decomp and operator.gates.matrix are removed, and the TOML file no longer prescribes to the framework how an operator should be handled. This will change the behaviour of decomposition for some circuits.

Related GitHub Issues:
[sc-71888]
[sc-71728]
[sc-71730]

One last thing, I think you can update setup.py to remove the tomlkit dependency right, since Catalyst doesn't use it directly anymore?

Makes sense, updated.


Use proper glyph for | in TeX fonts (in HTML-CSS, CHTML, and SVG).  #1610

Remap U+2016 to U+2225 in normal and bold TeX fonts, now that \| produces U+2016 rather than U+2225, and U+2016 isn't in MAIN font.  Resolves issue #1610.

All clear regarding CLA.

Meh. wrong PR...


Automated trendyqc update

Features

Cron file to run an automated addition of projects in TrendyQC
Consolidated command script to add projects in TrendyQC
Slack notifications if:

DNAnexus connexion fails
MultiQC report setup fails
MultiQC report import fails

Changes

Setup cron in Dockerfile
Cron file to add projects in TrendyQC everyday
Default command ran when the TrendyQC container starts is a bash script

Bash script to start gunicorn for NGINX use and start the cron job

New add_projects command script to add projects instead of initial_import
Slack notification system
New Slack variables in the docker-compose.yml

This change is

Hello @Yu-jinKim! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file trendyqc/trend_monitoring/management/commands/add_projects.py:

Line 31:80: E501 line too long (142 > 79 characters)
Line 40:80: E501 line too long (80 > 79 characters)
Line 59:1: W293 blank line contains whitespace

In the file trendyqc/trend_monitoring/management/commands/utils/_import.py:

Line 50:80: E501 line too long (81 > 79 characters)

In the file trendyqc/trend_monitoring/management/commands/utils/_notifications.py:

Line 45:10: W292 no newline at end of file


feat: add method for decoding sealed results

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

v4.0.0
GitHub release

Your semantic-release bot :package::rocket:


[WIP] Adds support to automatically install dependencies for Linux/MacOS

Changes the setup.py so that the dependencies that users had
to manually install before, are automatically installed using

pip install pyautogui.
This has been tested on linux for python2, python3 and it installs
the python2-xlib and python3-xlib dependencies automatically. Also
this works well on Windows, but Windows doesn't have any dependencies.
Assuming you have git and virtualenv installed you can test that it
automatically installs the dependencies using the following:
mkdir ~/tmp;
cd ~/tmp;
rm -rf env pyautogui;
git clone git@github.com:alphaCTzo7G/pyautogui.git;
virtualenv --python=$(which python3) env;
source env/bin/activate;
pip install -e pyautogui;
pip list;
deactivate

Changes the way that the version of pyautogui is evaluated.
Previously, this was being evaluated using a call to
pyautogui.__init__. However, that calls _pyautogui_x11, which fails
because the dependencies for linux are not installed.

Based on some other example setup.py, it seems the version number is
set explicitly:
https://gitlab.com/libxc/libxc/blob/master/setup.py
https://gemfury.com/squarecapadmin/python:meinheld/0.6.1/content/setup.py
https://salsa.debian.org/debian/doit/blob/debian/0.28.0-2/setup.py
So, I changed it to a explicity version number.
Please let me know if this works for you.

I've implemented some of these changes already, but thanks anyway. I'll add you to the authors file. Thanks so much again!


Wrong condition in D3DRender.D3DImageIsFrontBufferAvailableChanged

Hardware configuration

Intel Celeron CPU J1900 1.99GHZ
RAM:4G
Integrated graphics card:Intel HD Graphics

Reproduce

Preview camera with wpfMediaKit
Put the computer to sleep
Wake the computer up after step 2
Repeat setp 1-3 above multiple times, the application will be black out.

Other information

D3DDevice lost when application is black out
Notice the event D3DImage.IsFrontBufferAvailableChanged is fired, but I find some problem in method, D3DRender.D3DImageIsFrontBufferAvailableChanged.     private void D3DImageIsFrontBufferAvailableChanged(object sender, DependencyPropertyChangedEventArgs e)
     {
         //I think the condition below is wrong. When D3D device lost, D3DImage.IsFrontBufferAvailable is false all the time.
         if (!D3DImage.IsFrontBufferAvailable)
             return;

         m_newSurfaceAvailable = true;
         SetBackBufferInternal(m_pBackBuffer);
     }

@Rhuaer is this issue similar/duplicate to #42 and #3?

@xmedeko yeah, I will try it with BobDeCuir's suggestion.
Thanks.

May you suggest some PR, or some HOWTO manual for other users?

@xmedeko I have created a pull request just now. see #62


nodejs esm module output bundling lacks module prototype for externals

Bug report

What is the current behavior?
For my electron app I want to bundle the nodejs code of my main thread because of a) typescript and b) performance optimizations. That means I have "type": "module", regular node 20+ esm style code and want to bundle my dependencies into a final bundle (as I ship this bundle with my electron executable and not as part of a backend that has a node_modules dir).
To achieve a clean esm bundle (without createRequire and with support for dynamic imports) I use:

target: "es2022"
define standard node modules as externals
use experiments.outModule: true
output.chunkFormat: "module"
and module: true

I now want to bundle in the jimp library that has the pngjs dependency, which uses an util.inherits() call onto node-native stream in their chunkstream.js file:
https://github.com/pngjs/pngjs/blob/c565210c602527eb459f857eeb78183997482d5b/lib/chunkstream.js#L18
It looks like this in the bundle:
let Stream = __webpack_require__(/*! stream */ "stream");
util.inherits(ChunkStream, Stream);

Now the issue is that util.inherits expects stream to have a prototype and errors like so when run:
node:util:290
    throw new ERR_INVALID_ARG_TYPE('superCtor.prototype',
          ^

TypeError [ERR_INVALID_ARG_TYPE]: The "superCtor.prototype" property must be of type object. Received undefined
    at Module.inherits (node:util:290:11)
    at ./node_modules/pngjs/lib/chunkstream.js (file:///Users/myname/dev/jimp-nodenext/main.bundle.dev.js:15981:6)
    at __webpack_require__ (file:///Users/myname/dev/jimp-nodenext/main.bundle.dev.js:45536:41)
    at ./node_modules/pngjs/lib/parser-async.js (file:///Users/myname/dev/jimp-nodenext/main.bundle.dev.js:17191:19)
    at __webpack_require__ (file:///Users/myname/dev/jimp-nodenext/main.bundle.dev.js:45536:41)
    at ./node_modules/pngjs/lib/png.js (file:///Users/myname/dev/jimp-nodenext/main.bundle.dev.js:17814:14)
    at __webpack_require__ (file:///Users/myname/dev/jimp-nodenext/main.bundle.dev.js:45536:41)
    at<EMAIL_ADDRESS>(file:///Users/myname/dev/jimp-nodenext/main.bundle.dev.js:31862:63)
    at __webpack_require__ (file:///Users/myname/dev/jimp-nodenext/main.bundle.dev.js:45536:41)
    at ./node_modules/jimp/dist/esm/index.js (file:///Users/myname/dev/jimp-nodenext/main.bundle.dev.js:39797:70) {
  code: 'ERR_INVALID_ARG_TYPE'
}

When I edit the bundle, manually logging Stream.prototype yields undefined, and logging Stream yields:
[Module: null prototype] {
  Duplex: [Function: Duplex] {
    fromWeb: [Function (anonymous)],
    toWeb: [Function (anonymous)],
    from: [Function (anonymous)]
  },
  PassThrough: [Function: PassThrough],
  Readable: [Function: Readable] {
    ReadableState: [Function: ReadableState],
    _fromList: [Function: fromList],
    from: [Function (anonymous)],
    fromWeb: [Function (anonymous)],
    toWeb: [Function (anonymous)],
    wrap: [Function (anonymous)]
  },
  ...

I'm assuming the prototype that util.inherit expects got lost during webpacks bundling.
If the current behavior is a bug, please provide the steps to reproduce.
Here's a minimal repo for reproduction: https://github.com/tom2strobl/webpack-jimp-nodenext
See for additional comments regarding the webpack configuration: https://github.com/tom2strobl/webpack-jimp-nodenext/blob/main/webpack.config.js

What is the expected behavior?

webpack to bundle in a way that standard node modules that are defined as external retain their original prototypes for util.inherits compatibility.
Other relevant information:
webpack version:  5.95
Node.js version: 20.17
Operating System: macOS 15.0.1
Additional tools: jimp 1.6.0 / pngjs 7.0.0

@snitin315 We should improve node-commonjs docs, using it we generate:
import { createRequire as __WEBPACK_EXTERNAL_createRequire } from "node:module";
// ...
/***/ 2613:
/***/ ((module) => {

module.exports = __WEBPACK_EXTERNAL_createRequire(import.meta.url)("stream");

/***/ }),

This allows you to solve problems when you have:
// ...
function ChunkStream() {
  Stream.call(this);
}

util.inherits(ChunkStream, Stream);

I'll update the docs 👍

Wow thank you so much for the swift and detailed response! I can confirm it working for my case. I think it's on me sleeping on externalsType, I should have played around with it a bit. Have a nice remaining week!

Let's keep open, we need update docs in such cases


Clarify that Sstvala applies to virtual-instruction exceptions, too

Both illegal-instruction and virtual-instruction exceptions must populate stval with the faulting instruction.
This is not a change; the behavior was already mandated by the following clause in the hypervisor spec: https://github.com/riscv/riscv-isa-manual/blob/65c6b5d3e69cdc2bd2658f9639fd10553c4b34e0/src/hypervisor.adoc#L2040-L2041
We are replicating that information here for added clarity.

should also be added to RVA/B23 profiles.

I had already done so as part of the original PR. Or did you spot another place I missed?

OK - mistake on my end.  Will merge now.


Redisgraph expects the query params to be in a separate argument than the query

GRAPH.QUERY <graph_name> "MATCH (p:Person {name: $name_param, birth_year: $birth_year_param}) RETURN p" query_params "CYPHER name_param = 'Niccolò Machiavelli' birth_year_param = 1469"

https://github.com/RedisGraph/redisgraph-go/blob/f90645110e16a1ecfb4e95712f54c75cbc4ee119/graph.go#L122
Is it not what you mean?


[Freeradius] Support for EAP-TTLS-GTC is broken

Describe the bug
Support for EAP-TTLS-GTC in FreeRADIUS does not work.
To Reproduce
Steps to reproduce the behavior:

Install the os-freeradius plugin
Navigate to Services > FreeRADIUS > EAP
Set "Default EAP Type" to "TTLS-GTC"
Enable the service and look at its log file

Expected behavior
EAP-TTLS with GTC is enabled and working.
Relevant log files
The following error being logged:

Error: /usr/local/etc/raddb/mods-enabled/eap[15]: No dictionary definition for default EAP method 'ttls-gtc'

Additional context
I believe https://github.com/opnsense/plugins/issues/2085 which introduced the feature has a small issue:

OPNsense.freeradius.eap.default_eap_type is used on line 631 to set eap { ttls { default_eap_type = gtc } } which is fine, however
it is also used on line 30 to set eap { default_eap_type = ttls-gtc } which, as far as I can tell, is invalid.

I think default_eap_type within the eap section should be set to ttls, not ttls-gtc.
After making this change manually radiusd starts without any error message and I can use it for WPA2-Enterprise.
Environment
OPNsense 21.1.6-amd64
FreeBSD 12.1-RELEASE-p16-HBSD
OpenSSL 1.1.1k 25 Mar 2021

I believe I have provided all applicable pieces of information as per https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md. Should any more information be required I'll be happy to provide it.

This is still an issue, and I have also encountered this.  I own a DEC2680 that this is replicating on

Thx from bumping this up ... sorry @fchavant for my late respone, my fault :/


Issues with JIRA SAML SSO add-on redirect from mobile & some mac devices

Hi,
I'm not sure if this is the right place to report this issue, so please provide where I can log this if not appropriate.
We are using the "Microsoft Azure Active Directory single sign-on for JIRA"
Version: 1.0.14
Jira Version: 8.20.1
From mobile devices users are directed to the incorrect URL every time they sign in.
User navigates to https://ourJiraUrl.com/jira but after sign in they are directed to https://ourJiraUrl.com/jirahttps://ourJiraUrl.com/jira, which returns a 404 not found as the url is duplicated. The user then has to manually go to https://ourJiraUrl.com/jira again to access Jira.
This also occasionally happens from occassionally from other devices mainly macs, but can be replicated 100% of the time for mobile users on there first login (or from incognito)
It looks similar to https://docs.microsoft.com/en-us/answers/questions/35152/issues-with-jira-saml-sso-add-on-redirect.html & https://github.com/MicrosoftDocs/azure-docs/issues/56903

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

ID: d38cad90-6574-38c2-3262-059a9f975064
Version Independent ID: dd08d5c5-65a4-c2cc-8b51-7a35268dde68
Content: Tutorial: Azure Active Directory single sign-on (SSO) integration with JIRA SAML SSO by Microsoft
Content Source: articles/active-directory/saas-apps/jiramicrosoft-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

Thanks for letting us know the issue. I am filing this as a bug and will get resolved soon.


[Feature Request] Editing Inventory/Currently equiped items

Hello,
I tried to edit my inventory data through this tool. Unfortunately I couldnt find the right data in the converted json file. Is it possible to add and option to change the currently equiped gear and the current items in a players inventory?
I am thankful for any help regarding this topic & thanks for coding this tool in the first place!

This is more a converter than a modificator for palworld savegames.
I would recommend to post this feature request to one of the savegame editors out there, which do manipulate the savegame.
But, if you want to do this in an editor, here a short explaination:

Convert your PLAYERID00000.sav
Open the converted json and search for inventoryInfo
Underneath you have different keys like:

CommonContainerId
DropSlotContainerId
EssentialContainerId
WeaponLoadOutContainerId
PlayerEquipArmorContainerId
FoodEquipContainerId

Most of the above should be self-explaining
Write down their ID value, the GUID (xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx),, the only one without zeroes only
Convert your Level.sav
Open the converted json and search for the GUIDs copied
You should be able to find the items you have in your inventory by their name when scrolling down and edit the number for their value count.
Save -> Convert Level.sav.json back to sav -> Copy onto your server after you've shut it down -> Then start the server

I don't know what data DropSlotContainerId stores? Can you tell me?

@Cr4nkSt4r: Have you actually tried this? I just tested it through the website as described here: https://github.com/iebb/PalworldSaveEditor/issues/6 I tried to increase the amount of an item already present in my inventory and sadly it did not work: The game loaded the save file but asked me to create a new character at launch (so I assume it discarded the one with an edited inventory somehow...). A rollback of Level.sav did fix it though, yet inventory editing does not work this way...

@FlorentP42
Never encountered any issues when editing the raw files to this day.
Idk about the saveeditor you linked tho.


Code review

quick and brief.

I made commits for individual changes to your code and provided comments directly on those commits. There's still a few things you can update/change and this is by no means complete. But thought you might find it useful.

Thanks, this is a lot and I can't look through it all on my phone but I will inspect it an merge when I get home!


Chinese input method is invalid.

Type: Bug
Install VScode in Ubuntu22.04 LTS, create a new file, switch to the Chinese input method to try to input, and find that Chinese cannot be input.
VS Code version: Code 1.72.2 (d045a5eda657f4d7b676dedbfa7aab8207f8a075, 2022-10-12T22:16:26.920Z)
OS version: Linux x64 5.15.0-48-generic snap
Modes:
Sandboxed: No
Extensions (6)

Extension
Author (truncated)
Version

gitlens
eam
13.0.3

vscode-language-pack-zh-hans
MS-
1.72.10121008

python
ms-
2022.16.1

vscode-pylance
ms-
2022.10.30

cpptools
ms-
1.12.4

cpptools-extension-pack
ms-
1.3.0

(4 theme extensions excluded)

@PerssonBrown Can you input Chinese in the input of the find widget (ctrl+f) ?

Snap has a long history of not working with IMEs, not sure the situation now though.

Let's track in #96041 . The workaround is to download the .tar.gz from https://code.visualstudio.com/#alt-downloads


Update mongoose from 5.9.24 to 5.13.9 for mpath

Validation on k8s cluster: n-2 is succeeded.

Validation on k8s cluster: n-1 is succeeded.

Validation on k8s cluster: n is succeeded.

Validation on k8s cluster: n+n+n is succeeded.

Pull Request Test Coverage Report for Build 5589

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.644%

Totals

Change from base Build 5574:
0.0%

Covered Lines:
9717

Relevant Lines:
10580

💛  - Coveralls


WTLAL move to rsutils

This was kind of fast, but I want to use float3 in rsutils so I took something I had started back in October and finished it.

move int2, float2, float3, float3x3 to rsutils/number/float3.h
move pose and rs2_extrinsics related to src/pose.h
use rsutils::number::float3 in librealsense:: and rs2:: (latter for common/ stuff)

Most of the changes are simple header changes.

Let's wait a bit with this merge we need to do some sync with other repos


⚠ Bofrost.nl has degraded performance

In 5e107a2, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 829 ms

Resolved: Bofrost.nl performance has improved in c371561 after 6 minutes.


Sync fails on 4.0.0 due to undefined getID3 method

Hey, thanks for contributing to Koel! To save time for both of us, please make sure these checkboxes are checked before submitting the issue:

[x] You have read and followed closely the Wiki, Upgrade Guide, as well as Troubleshooting
[x] The issue has not been reported before
[x] This is not a "how to install on Windows" or "why is my npm messed up" question
[x] You're a cool person

All checked? Now also make sure your issue

[x] Is associated with a version. Or better yet, a commit.
[x] Is as detailed as possible (ahem... OS, browser, steps to reproduce, maybe?)
[x] Includes the error output if it's a bug/error report ("Whoops!" is not very helpful, you know)
[x] Is in English, 因为我不说中文。

Summary
When I run php artisan koel:sync I get the following output:
Syncing media from /mnt/music

  286/4417 [▓░░░░░░░░░░░░░░░░░░░░░░░░░░░]   6%
In QuickTime.php line 479:

  Call to undefined method JamesHeinrich\GetID3\Utils::LookupGenreName()

The issue was brought up without log details in #1056.  i belive this is related to #1040 as the getID3() version found in vendor/james-heinrich/getid3 is 1.9.13 but the method name matches version 2.0 of getID3 found upstream:
https://github.com/JamesHeinrich/getID3/blob/v2.0.0-beta1/docs/Upgrading.md
https://github.com/JamesHeinrich/getID3/blob/v2.0.0-beta1/src/Module/AudioVideo/QuickTime.php#L479

Steps to reproduce:
Follow the upgrade instructions to upgrade to 4.0.0 and then run php artisan koel:sync
composer update
php artisan koel:init

OS
Debian 10 (Buster)
PHP: 7.3.9-1 (FPM)
nginx: 1.17.3
DB: MariaDB 10.4.8

Hi, I'm having the same problem, how did you update that single module?

how did you update that single module?

There may be a more systematic way to do this with Yarn or Composer but here's what I did.
# cd to getId3 root folder
$ cd www/vendor/james-heinrich/

# remove current library
$ rm -Rf getid3/

# Clone the current library into getid3 folder
$ git clone https://github.com/JamesHeinrich/getID3.git getid3

# Checkout 2.0 branch
$ cd getid3/
$ git checkout 2.0

Then you can artisan koel:sync like before.

Had the same issue, followed the suggestion in the last comment and now everything works :)

Does anyone want to own this? This should already be covered by the tests, so changing the version number and running phpunit would be a sufficient start.

I tried to update the composer.json to fix this but still looks like v2.0.0-beta1 is being used. The tests passed but I don't think the bug itself is fixed.
Here are the results from Scrutinizer after using ^2.0-dev
- Installing james-heinrich/getid3 (v2.0.0-beta1): Downloading (100%)
and ^2.0.0-dev
- Installing james-heinrich/getid3 (v2.0.0-beta1): Loading from cache
Any ideas here?

Why was this closed without any resolution? I am experiencing the very same issue.

These steps did not solve the issue, neither did composer update. This script seems to be incredibly bug-ridden.
remove current library
$ rm -Rf getid3/
Clone the current library into getid3 folder
$ git clone https://github.com/JamesHeinrich/getID3.git getid3
Checkout 2.0 branch
$ cd getid3/
$ git checkout 2.0

WDYM "without any resolution"?

It means the Sync fails due to undefined getID3 method. Unfortunately the author did not respond but I have tried all solutions given and they didn't fix the issue.

its my 10th try or so to get this thing running at all. tried over the last 2 years... getting into the web-interface now was as far as i ever got, just to be stopped again by yet another bug...

You have closed other "issues" detailing the same problem before, without resolution:
https://github.com/phanan/koel/issues/1056

I'm sorry? What part here is unclear to you?

I created another ticket for this:
https://github.com/phanan/koel/issues/1102

Looking into this a bit more, I'm not sure if #1095 fixed the issue as expected. The problem can be resolved by manually upgrading getId3 which has worked for several others. We will just need to make sure that is actually happening or perhaps downgrade to 1.9.18 if that does not cause a regression of the RegEx bug that was resolved by the 2.0.x upgrade.
What may work best here is re-open this issue until we have confirmed that the patch in #1095 resolves this for others or a new Koel release is out which does resolve it.
@gorfreed  I'm not sure why the above steps to manually replace getID3 are not working for you as we don't have enough information about your setup. Unfortunately, #1102 also does not follow the issue template so will likely be closed. I know from my own experience that finding bugs or having troubling installing FOSS can be frustrating. I would suggest remaining diligent and reading through the issues to see if you can pin point what's wrong in your case. Koel is working for me and many others so there's likely a conflict somewhere. If you're still unsatisfied you may want to explore other free media player options that will meet your need. There is a list of a few others here. If you do wish to try Koel again, please respect the developer's wishes and use the template as well as provide other community members, like me, enough information to be helpful while looking at issues.

@BrookeDot et. al. As mentioned in another ticket, I can't reproduce this problem at all, so I was blindly merging your fix :) Looks like I'll just downgrade to 1.9.18 – v2 just doesn't worth the fuss 😑

Downgrading to 1.9.18 sounds like a good option. The Regex issue had to be manually patched for me but was less of a headache then manually upgrading to 2.0 to fix this issue.
As for replication, I thought I mention my setup but looks like I didn't. Sorry! I'm running the following:
Debian 10.1 (buster)
NGINX 1.17.4
PHP 7.3.10
MariaDB 10.4.8
If I had to guess based on the output it's likely the PHP version that is causing the issue but we'd need to compare setups of others to be 100% on that. I see that getID3 1.9.18 says its tested with php 7.4 which is a good sign.

OK, PR created. As long as the tests pass, I'll merge and release a new version.

@BrookeDot Koel's TravisCI covers PHP 7.1, 7.2, and 7.3, so I'm pretty clueless about this issue.

Hi guys, thank you for the responses! I am happy to see such an invested community! @BrookeDot  I am pretty much on the bottom end of the Linux proficiency scale and I am amazed whenever I accomplish anything on my server. That said, usually I am able to find causes with some persistence and logic, but sometimes some steps of solutions discussed are so super-obvious that people don't spell them out, which is when I will be unable to recreate them. Below is a printout of me replacing the getid3 folder as suggested. As you see the sync command still doesn't run through. Am I wrong suspecting that perhaps there is more of a directory/path problem going on here?

@gorfreed I just upgraded to 4.1.0 on my Linux server with the LEMP stack above without issue. Running commands as root could be the issue. However, it looks like the bigger issue is that you're cloning getId3 into the main ~/koel folder where it should be cloned into vendor/james-heinrich/. For example, in your case I think it's ~/koel/vendor/james-heinrich/.  It may also be helpful to use ls -lan to see what files are in the directory you are in to check if things look correct.
In my setup I have disabled root login and run all commands as the www-data user via sudo. For example my commands look like this:
sudo -u www-data php artisan koel:sync
DigitalOcean has a good guide on initial server setup for many popular Linux distributions which goes over disabling root
https://www.digitalocean.com/community/tutorial_collections/16
If you're still having trouble it may be good to know  the following:

what is working as expected?
was setup successful, and you're just getting stuck with syncing.
Your LEMP/LAMP stack including versions.  ( lsb_release -a( for debian/ubuntuv), nginx -v, mysql -V, php -v)
Node and NPM versions (node -v, npm -v)
Any additional details regarding your setup.

Thanks @BrookeDot – it's amazing to see how far you're going out of your way to help others :).


Set up worker dyno to perform async work

[ ] Posting to getconnect
[ ] Generating thumbnails of cover art
[ ] Podcast migration

Future:

Transcoding

I'll use Lambda+SNS for this instead


Appveyor : worth it?

Seems like Appveyor is frequently false-failing in a way that's distracting rather than helpful.

Something to be done to make it false-fail less? Time to give up on it?
It's mostly (but not entirely) conceptually redundant with Travis-CI, so maybe we wouldn't be giving up too much if we just turn it off.

I just now turned off the the requiredness of continuous-integration/appveyor/pr for the green merge button, which should at least reduce the indicator fatigue somewhat.

Something to be done to make it false-fail less?

https://github.com/appveyor/ci/blob/master/scripts/appveyor-retry.cmd may be able to help

Time to give up on it?

I'd vote no.
uPortal-home has been adopted on Windows elsewhere, having continuous integration on Windows gives some level of assurances the uPortal home will continue to work for that adopter.
Those assurances are brought with, what is IMO, a low level of effort on the part of developers.

AppVeyor has been switched off


chore(circuits): remove dummy kernels

Description
Modified dummy kernel cbind to no longer use a real circuit (with a real composer and constraints) to generate the dummy previous kernel. This should speed up simulation testing in TS.
Checklist:

[ ] I have reviewed my diff in github, line by line.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to the issue(s) that it resolves.
[ ] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[ ] The branch has been merged or rebased against the head of its merge target.
[ ] I'm happy for the PR to be merged at the reviewer's next convenience.

Closing as this is not worth merging with the #506 work coming soon.


Support for custom command line option

It would be nice to provide custom command line options for isort command.
For example I need to use isort -w 119, instead of default value

:+1: Yeah, I'll look into this.

:+1:


Use UMD closure

Using a UMD style closure would make this plugin loadable with AMD loaders (require.js) or CommonJS (node/browserify).
https://github.com/umdjs/umd/blob/master/jqueryPluginCommonjs.js

Done.


Connection errors are issued on stdout and not stderr

I develop a python application that uses subprocess.Popen to start xfreerdp via the CLI and then retrieve any error messages from stderr.  When a connection error occurs the error text is sent to stdout and not stderr, which strikes me as unusual.

@robled this issue should be resolved with the pull request mentioned above. Can you verify that this problem is solved for you. If you have Debian, Ubuntu, Fedora or OpenSuse you can use our nightly builds for testing https://github.com/FreeRDP/FreeRDP/wiki/PreBuilds

Should be fixed on master and no feedback for a while.


How do you get it to work exactly?

Hello. There are a lot of bugs in this project(specifically import errors), but its very interesting. How exactly do you make it run so you can stake coins and stuff?

True, there is no way of opening but perhaps python app.py might work but considering the amount of bugs it most likely would not.

Hi the code is not working yet (in-progress)
The consensus will use aspects of PoS but with my own twist
there is not validators or stake
every wallet that hold some amount of coins have a chance of winning the "lottery" and creating the next block the chance is calculated 100 - (wallet.balance - wallet.last_transaction)
more coins = more chance to win
AND
wallet last transaction lower = more chance to win
the import errors are for development-env reason


PyTorch: File does not contain root module or state dictionary

Netron app and version: 5.7.8 Desktop app
OS and browser version: Windows 10

Steps to Reproduce:
1.It doesn't work when I open my model
final_89.zip

Duplicate of #543


Raise an error when both --retention-time and --range-field are set

The --retention-time can only be set when --range-field option is not specified.

Solved by https://github.com/bakdata/quick-cli/pull/10.

Related to https://github.com/bakdata/quick/issues/55


sfc compile problem

Vue version
3.2.45
Link to minimal reproduction
https://sfc.vuejs.org/#eNpdUk2PmzAQ/SsjqxJESox6qxCJ1Eu19+2RQylMwBt/IHvYdIX47x3DOuxupEieGb/He88zi5/jKF8nFKWoQuvVSBCQpvFS26KAMKibkj3Sk+oHzX9Cn8/rCIAGNFhCZp3vsrW3HCQ3bT7st+F8gQRonQ0EpoMzmMbfOne3DzKAgYwugfyEx72XiErIW9fhEXRj+8NH0vjzrNjb/TZ6GW//dk/M+Q6cVyhLfOCW91NsfVIYlUSNnfRoO3Z8DXxqul9K4/ObbfNMcfufNF12hGyi6+lHdki8G4WbiBn+PL5VkSKNl+eYZ1VsxT7Uyt7YhD7XItCbxjAgUi1g8HhNPdmGUIsd9G2OMpMHJkmv51uGbAJfIqIqtkmCJlFs6u4V4VdXkZZ9sYPPuTjWoF2fZ52zmG12Y3Qf+LkgNKNuKLpjTcP3yzyDCZz7UhVcrV1lR47n9WT4YaJnnq/OOJiEFkehzOg8nUwzsg9neUH5yQHY2zpga2Vcgui9FrzBsa7FQDSGsijCtY1r/RKk833BJ+knS8qgxGBOf727B96SGFCk4BgXsfwHOuT9ww==
Steps to reproduce
I copied the sample code from the Shiki official website in vue and added comments to it. There were no issues when viewing it in VSCode, but when running the project, there was an error that did not clearly indicate that it was a problem with the comments. The error message pointed to syntax problems elsewhere. After being confused for a while, I discovered that the issue was with this commented section of code. However, SFC Playground immediately identified this as an issue with the comment code, whereas VSCode did not.
What is expected?
Want vscode can report error clearly
What is actually happening?
The error message pointed to syntax problems elsewhere.
System Info
System:
    OS: macOS 10.15.7
    CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
    Memory: 4.25 GB / 16.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 16.16.0 - ~/.nvm/versions/node/v16.16.0/bin/node
    Yarn: 1.22.15 - ~/.nvm/versions/node/v16.16.0/bin/yarn
    npm: 8.11.0 - ~/.nvm/versions/node/v16.16.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Browsers:
    Chrome: 111.0.5563.64
    Edge: 111.0.1661.51
    Firefox: 109.0.1
    Safari: 15.6.1
  npmPackages:
    vue: ^3.2.45 => 3.2.47

Any additional comments?
No response

Syntax highlighting issue + the SFC compiler should have a better error message.

You cannot have </script> in your script block event inside comments. This is how HTML works and I believe Vue templates are parsed the same way.

use <script src="index.js"><+/script> instead.


Improve league DB cache and API

This PR does two things:

it improves how db was cached with --update-db False. Previously, we would setup a league.db.backup and once the script is finished we restore it back to league.db. This is problematic because if the training script calls the league.py --update-db False multiple times, the second call would encounter a corrupted league.db that hasn't been restored. To this end, this PR would cache a tmp/12321312.db and simply remove it after the script is finished. This way, the training script always has access to the uncorrupted league.db.
it also allows the --output-path flag to save the leaderboard csv once the script is done.

This would not be a breaking change. Merging now.


Bug: Bazel 3.7.2 fails to build with GCC 11.1.0 on ARMv8

Hi,
Bazel 3.7.2. is failing to build on  ARMv8, hopefully, in the latest Bazel, they have fixed it.
Any plan to use upgraded Bazel?

What error do you get?

With GCC-11 we need to explicitely add #include <limits> and #include <stdexcept>.

This issue was fixed upstream. See #50176 and link therein.

@samcom12 ,
Looks like this is duplicate of issue #50303.Can you please close this issue, since it is already being tracked there? Thanks!


Updating whitepaper for sign in to sync

Updating references to syncing in Privacy Whitepaper to note that signing into a Microsoft account will enable sync.
Text changes provided by Clark Masterson

Docs Build status updates of commit 71b42ed:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/privacy-whitepaper/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5df7820:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/privacy-whitepaper/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 1fa9d12:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/privacy-whitepaper/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


fix(pointcloud_preprocessor): organize input twist topic

Description

Concatenate filter node now supports both twist and odom type input for velocity and angular velocity input.
Concatenate filter node used in perception module now uses odometry input from localization.

Related to: https://github.com/autowarefoundation/autoware.universe/pull/5108
Should be merged with

https://github.com/autowarefoundation/sample_sensor_kit_launch/pull/77
https://github.com/RobotecAI/awsim_sensor_kit_launch/pull/8
https://github.com/tier4/aip_launcher/pull/176

Tests performed

Tested with pilot-auto.x2 (TIER IV internal product version Autoware) and confirmed that the change addresses the incorrect timestamp difference compensation using Logging Simulator.
Effects on system behavior

The concatenate filter node in perception will now use twist input from localization, which should improve the concatenation accuracy.
All the concatenation node will now require an additional parameter

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@miursh @yukkysaito Would you review this PR? :pray:

@kminoda May I request two additional items?

Could you make the same change in time_synchronizer as well? This is because sync_and_concat node is going to be deprecated
Could you add warning here when the twist queue is empty?

@miursh
Thank you for the comment!
For the first point, fixed here: https://github.com/autowarefoundation/autoware.universe/pull/5125/commits/44bf023f06e52f19a5728274f19e8d568b2a01c1
For the second point, let me handle that in another PR.


Update scala-parser-combinators to 2.1.0

Updates org.scala-lang.modules:scala-parser-combinators from 1.1.2 to 2.1.0.
GitHub Release Notes - Version Diff
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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang.modules", artifactId = "scala-parser-combinators" } ]

labels: library-update, semver-major

Superseded by #440.


please add installation  of  metacontoller to README

I suggest to include a more specific cluster specification to better define under what conditions  this repo might be  working.
E.G. the metacontroller needs to be installed explicitly. This is not mentioned anywhere
https://metacontroller.github.io/metacontroller/guide/install.html
Installing it like this helped me to finish the kubectl apply -f build step
kubectl apply -f https://raw.githubusercontent.com/GoogleCloudPlatform/metacontroller/master/manifests/metacontroller.yaml

@uvwild metacontroller actually is being included here https://github.com/hpi-schul-cloud/jitsi-deployment/tree/master/base/ops/metacontroller
The ops/ folder is included only in the development-monitoring and production-monitoring overlays.
Keep in mind that the GoogleCloudPlatform/metacontroller repo has been deprecated in favor of the dedicated https://github.com/metacontroller/metacontroller repo where you can find the latest version.

@uvwild metacontroller actually is being included here https://github.com/hpi-schul-cloud/jitsi-deployment/tree/master/base/ops/metacontroller
The ops/ folder is included only in the development-monitoring and production-monitoring overlays.
Keep in mind that the GoogleCloudPlatform/metacontroller repo has been deprecated in favor of the dedicated https://github.com/metacontroller/metacontroller repo where you can find the latest version.

Hello - I am running into the same issue. Do we need to deploy the monitoring components first and then jitsi?

same here. how do we install this add-on?

The answer comes a little late, but I hope it still helps someone.
The documentation is a bit confusing about this issue as the metacontroller maintainers have migrated their official repositories from google dependencies to their own repos.
The problem comes from the fact that for some reason the repository does not have all the base resources (like role bindings, statefulsets, etc) for metacontroller to create properly the Kubernete's CustomResourceDefinition components (in this case they are used to define the  service-per-pod-controller mechanisms to scale JVB instances.
To fix this you should run the following commands before deploy any Jitsi resources on your cluster:
# Install Metacontroller custom resources
# https://docs.primehub.io/docs/getting_started/install_metacontroller
# These custom resources supports the creation of custom controllers
# for JVB services (service-per-pod-controller).

kubectl apply -f https://raw.githubusercontent.com/metacontroller/metacontroller/master/manifests/production/metacontroller-namespace.yaml
kubectl apply -f https://raw.githubusercontent.com/metacontroller/metacontroller/master/manifests/production/metacontroller-rbac.yaml
kubectl apply -f https://raw.githubusercontent.com/metacontroller/metacontroller/master/manifests/production/metacontroller-crds-v1.yaml
kubectl apply -f https://raw.githubusercontent.com/metacontroller/metacontroller/master/manifests/production/metacontroller.yaml

Finally you can check if all metacontroller dependencies were successfully inside your cluster executing:
kubectl get crd | grep metacontroller

Then you should get something like this if everything is ok:
compositecontrollers.metacontroller.k8s.io   2023-06-16T18:33:35Z
controllerrevisions.metacontroller.k8s.io    2023-06-16T18:33:36Z
decoratorcontrollers.metacontroller.k8s.io   2023-06-16T18:33:37Z

You should also get rid of any metacontroller namespace definitions within the repo to avoid breaking the metacontroller resources you have previously installed and manage this whole process in a specific script to prepare your cluster to deploy Jitsi's resources.
Expected output after deploying Jitsi:
namespace/jitsi created
configmap/jvb-entrypoint created
configmap/jvb-shutdown created
configmap/prosody created
configmap/web created
secret/jitsi-config created
service/shard-0-prosody created
service/shard-1-prosody created
service/web created
deployment.apps/shard-0-jicofo created
deployment.apps/shard-0-prosody created
deployment.apps/shard-0-web created
deployment.apps/shard-1-jicofo created
deployment.apps/shard-1-prosody created
deployment.apps/shard-1-web created
horizontalpodautoscaler.autoscaling/shard-0-jvb-hpa created
horizontalpodautoscaler.autoscaling/shard-1-jvb-hpa created
decoratorcontroller.metacontroller.k8s.io/shard-0-pod-name-label created
decoratorcontroller.metacontroller.k8s.io/shard-0-service-per-pod created
decoratorcontroller.metacontroller.k8s.io/shard-1-pod-name-label created
decoratorcontroller.metacontroller.k8s.io/shard-1-service-per-pod created
(...)

NOTE: As @kachar said you should use this installation guide (do not consider GoogleCloudPlatform/metacontroller documentation in this regard) : https://metacontroller.github.io/metacontroller/guide/install.html
NOTE2: Command shared by @r3k2 work in a more efficient way:
kubectl apply -k https://github.com/metacontroller/metacontroller/manifests/production

Cheers!


2 sidepanel Feature Request

My feature Request is not related to a problem
I would like to have 1 sidepanel at the left part and 1 at the right part of my window. So i can use 2 extensions in sidepanel at same time (like ai assistant & bookmarks or notetaking, or side page and so on...)
I did read the faq
I really enjoy thorium and am using it for more than a year!

Perhaps this can be generalized, e. g. allowing users to create such panels on an ad-hoc basis and customize it to their liking(s). Getting the HTML/CSS for this to work is probably trivial in any case.


Add note on TS in adding to README.md

We have documented the use of enableTypeScriptTransform for a long time, but only for apps. Add the corresponding note for addons, in a way which should also help for searchability.

I merged https://github.com/babel/ember-cli-babel/pull/491, so I think we can close this one if that's okay.


with no content response, send appropriate headers

With this change, when there is an empty response body, the Response class will send a Content-Length:0 header, and will not send Content-Encoding and Content-Type headers. Previously, the Response class would send no Content-Length header and would send a Content-Encoding and a Content-Type header, neither of which is particularly appropriate with no response body.
For example, for a 302 response,
old behavior:
HTTP/1.1 302 Temporary Redirect
Date: Thu, 15 Jun 2023 21:25:53 GMT
Transfer-encoding: chunked
Content-type: text/plain
Location: /.well-known/foo.bar

new behavior:
HTTP/1.1 302 Temporary Redirect
Date: Thu, 15 Jun 2023 21:23:15 GMT
Content-length: 0
Location: /.well-known/foo.bar

Yeah, it seems appropriate, and code wise there's not much to mention.
Thank you for your contribution again <3


Auto-discover product tests environments

Let's separate "Auto-discover product tests environments" from adding env list.
There is not relation between the two changes.
For "Auto-discover product tests environments" the most important thing is using the existing classpath scanning API and avoiding additional dep.

Editorial comments done @findepi

Merged as 7ffa27bd9402f69e0e9c71834c82c1c3dc211921, thanks!


fix(Designer): Updated HTML Editor to support newline and carriage return characters (for dynamic content)

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes/features)
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

This PR fixes a bug with the HTML Editor where the HTML Editor cannot support newline characters.

What is the current behavior? (You can also link to an open issue here)

When a user adds dynamic content to any action with an HTML Editor and adds a newline character (for example, by pressing the 'enter' key) and then closes the action, reopening the action will cause the dynamic content to disappear. The dynamic content should stay in the action's HTML Editor, but it does not stay because the HTML Editor does not support newline characters.
Bug 26654763: Expression gets lost when newline is used

What is the new behavior (if this is a feature change)?

My change enables the HTML Editor to support newline characters. With my change, a user can add newline characters to the HTML Editor for dynamic content and the newline character (and dynamic content) will be preserved. It will not spontaneously disappear (as seen in the bug linked above). This improves the user experience for LA Designer users because it ensures the HTML Editor can support newline characters.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

This PR does not break any existing functionality. Rather, it only adds to the functionality of the HTML Editor by supporting newline characters.

Please Include Screenshots or Videos of the intended change:

Added a newline character after the 'a' in email:

Upon reopening the HTML-to-Text action, the dynamic content with newline character remains. Previously, HTML-to-Text field would appear blank (incorrectly):

@Eric-B-Wu  can you take a look at this?
@spanvalkar as discussed offline, I do want to hold off signing off on this until there are adequate tests added. This has been a painful issue in the past.

asking for tests on the changes made here, as well as a test on a previous regression from this PR Parameters wiped after saving an update · Issue #4765 · Azure/LogicAppsUX (github.com)


feat: #34 - added self-service end points to cache map

Added self-service end points to cache map.
Closes #34

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[R] CRAN release version 10.0.1

Packaging checklist for CRAN release
For full documentation of the steps in this checklist, see the
Apache Arrow Release Management Guide.

[x] When a release candidate is created
Create a GitHub issue
entitled [R] CRAN release version X.X.X and copy this checklist to the issue

[x] Confirm that all
nightly tests and nightly packaging builds are passing.

Will be at https://github.com/apache/arrow/pull/14772

[x] Check current CRAN check results

[x] Ensure the contents of the README is accurate

[x] urlchecker::url_check()

[x] Polish NEWS

[x] Prepare tweet thread highlighting new features

Make pull requests into the autobrew and
rtools-packages repositories
used by the configure script on MacOS and Windows. These pull requests will
use the release candidate as the source.

[x] Pull request to modify
the apache-arrow autobrew formula
to update the version, SHA, and any changes to dependencies and build steps that have changed in the
copy of the formula we have of that formula in the Arrow repo
[x] Pull request to modify
the apache-arrow-static autobrew formula
to update the version, SHA, and any changes to dependencies and build steps that have changed in the
copy of the formula we have of that formula in the Arrow repo
[x] Pull request to modify the
autobrew script
to include any additions made to
r/tools/autobrew.
[x] Pull request to modify the
PKGBUILD script
to reflect changes in
ci/PKGBUILD,
uncommenting the line that says "uncomment to test the rc".

Prepare and check the .tar.gz that will be released to CRAN.

[x] git fetch upstream && git checkout release-X.X.X-rcXX
[x] make build (copies Arrow C++ into tools/cpp, prunes some unnecessary
components, and runs R CMD build)
[x] devtools::check("arrow_X.X.X.tar.gz") locally
[x] Run reverse dependency checks

Wait for the official release...

[x] Release vote passed!
[x] Create a CRAN-release branch from the official release commit
[x] Pick any commits that were made to master since the release commit that
were needed to fix CRAN-related submission issues (e.g., fixed URLs, fixes
to )
[x] Remove badges from README.md
[x] Run urlchecker::url_check() on the R directory
[x] Bump the Version in DESCRIPTION to X.X.X
[x] Create a PR entitled WIP: [R] Verify CRAN release-10.0.1-rc0. Add
a comment @github-actions crossbow submit --group r to run all R crossbow
jobs against the CRAN-specific release branch.
[x] Regenerate arrow_X.X.X.tar.gz (i.e., make build)

Update autobrew and r-windows PRs to use the release instead of the
release candidate; ensure linux binary packages are available:

[x] PR into autobrew/homebrew-core
[x] PR into autobrew/homebrew-core
[x] PR into autobrew/scripts
[x] PR into r-windows/rtools-packages
[x] Ensure linux binaries are available in the artifactory:
https://apache.jfrog.io/ui/repos/tree/General/arrow/r

Check binary Arrow C++ distributions specific to the R package:

[x] Upload the .tar.gz to win-builder
and confirm that the check is clean

done; Neal will get an email to confirm. Failed. Resubmit after Jeroen has uploaded binaries.

[x] Upload the .tar.gz to MacBuilder
and confirm that the check is clean

Results will be at https://mac.r-project.org/macbuilder/results/1669915267-2fc3871d8987a385/

[x] Check install.packages("arrow_X.X.X.tar.gz") on Ubuntu and ensure that the
hosted binaries are used
[x] devtools::check("arrow_X.X.X.tar.gz") locally one more time (for luck)

Submit!

[x] Upload arrow_X.X.X.tar.gz to the
CRAN submit page
[x] Confirm the submission email

Wait for CRAN...

[x] Accepted!
[ ] Tag the tip of the CRAN-specific release branch
[x] Add a new line to the matrix in the backwards compatability job
[x] Update the packaging checklist template to reflect any new realities of the
packaging process.
[x] Wait for CRAN-hosted binaries on the
CRAN package page to reflect the
new version
[ ] Tweet!

Closing this now, as the release is done.


add second slide

I've added a new file to my first branch

great course, but it's still a bit vague what is actually happening.


Keras LSTM units

The idea of this post is to get a deeper understanding of the LSTM argument "units". I don't know if it makes any difference but I am using Theano.
https://keras.io/layers/recurrent/
On this page, under "LSTM", units are explained as:

units: Positive integer, dimensionality of the output space.

Here some example lines of code just so that we have something specific that we can talk about:
model.add(LSTM(32, batch_size=50, input_shape(1,12))
model.add(Dense(5, activation='softmax')
I drew a schematic of how I understand the structure:

And now a picture from that page: http://colah.github.io/posts/2015-08-Understanding-LSTMs/

In my terminology according to the picture of http://colah.github.io/posts/2015-08-Understanding-LSTMs/ "A" is a LSTM Block.
Input: x_t
Output of the hidden layer: h_t
And here are my questions:
Is it correct that  model.add(LSTM(32)) means that there are 32 nodes in the hidden layer?
Are these 32 nodes equivalent to 32 LSTM Blocks?
Is it correct to assume that in one timestep the Input x_t goes through 32 LSTM blocks?
x_1 --> 32 LSTM blocks (input x_1 goes 32 times through a structure like "A")
x_2 --> 32 LSTM blocks (input x_2 goes 32 times through a structure like "A")
x_3 --> ...
Or does the number of units (32) has nothing to do with the number of LSTM blocks?
Is it possible that there is just one LSTM Block per timestep like in the picture from http://colah.github.io/posts/2015-08-Understanding-LSTMs/ and 32 means that the internal cell state c_t is a vector with 32 rows that passes the LSTM block ("A") just once?
Or do I mix something up here?
I hope I could make myself clear. If you have any questions don't hesitate to ask me.
I would appreciate any help :)

I found my answers here: https://github.com/fchollet/keras/issues/7600

And here are my questions:
Is it correct that model.add(LSTM(32)) means that there are 32 nodes in the hidden layer?

Yes.

Are these 32 nodes equivalent to 32 LSTM Blocks?

The answer here is definitely no.

Is it correct to assume that in one timestep the Input x_t goes through 32 LSTM blocks?
x_1 --> 32 LSTM blocks (input x_1 goes 32 times through a structure like "A")
x_2 --> 32 LSTM blocks (input x_2 goes 32 times through a structure like "A")
x_3 --> ...

No.

Or does the number of units (32) has nothing to do with the number of LSTM blocks?

Yes.

Is it possible that there is just one LSTM Block per timestep like in the picture from http://colah.github.io/posts/2015-08-Understanding-LSTMs/ and 32 means that the internal cell state c_t is a vector with 32 rows that passes the LSTM block ("A") just once?

Yes.

Or do I mix something up here?

Yes, I did. I mixed "LSTM block" and "units".

its not pretty clear...what is the use of 32.....what does it control?

2.what controls the number of LSTM block? is it elements of batch size  which is equal to number of LSTM blocks>


new repo: volume-expander-operator

Type of repository request
creation of new repository
Requested name for the repository
volume-expander-operator
Repository Description
operator to automatically expand PVCs.
Admin Access
Raffaele Spazzoli
Write Access
Raffaele Spazzoli
Read Access
Raffaele Spazzoli
Additional details
N/A

Review request:
cc: @redhat-cop/container-cop-core

Please share list of existing repository along with contributors working on the project

no existing repos, no other collaborators.

Recommend incubating the project in a personal repository until either a group of contributors has been established and/or a need from the community arises

the need has arisen in one of my customers. and it's such a general and obvious use case that it's surprising that someone has not done this. It's hard to see how a community can form around a personal repo. Anyway here it is: https://github.com/raffaelespazzoli/volume-expander-operator

please create the corresponding quay repo and add the quay credentials as secret:
QUAY_USERNAME
QUAY_PASSWORD


chang comment with more related link

seems page https://cloud.google.com/ml-engine/docs/trainer-considerations  has changed its content

This change is

/lgtm
/approve
Thanks for your contribution!

/ok-to-test

Coverage remained the same at 57.852% when pulling d3511cb6e14562652286eeff959722fe89380d70 on u2takey:modcomment into f405fb2a2a33fa3aebd35f919fbe1215ff761724 on kubeflow:master.

Coverage remained the same at 57.852% when pulling d3511cb6e14562652286eeff959722fe89380d70 on u2takey:modcomment into f405fb2a2a33fa3aebd35f919fbe1215ff761724 on kubeflow:master.


query.sum() should be generic

Trying to get sum of money movements by calling
MoneyMovement.query(on: conn).sum(\MoneyMovement.quantity).flatMap { sumOfIncomes -> EventLoopFuture<Int64> in
    //somehow `sumOfIncomes` is Double, that's strange because `MoneyMovement.quantity` is Int64
}

but getting error
[ ERROR ] PostgreSQLError.binaryFloatingPoint: Could not decode Double from binary data type: NUMERIC.

Tried to change MoneyMovement.quantity column type to double precision in the database and variable type to Double in the Model, but getting the same error.. looks like .sum() is completely broken

Related: https://github.com/vapor/fluent/issues/450#issuecomment-382047483
Also, I think this (at least partially) affects Fluent itself, not Postgres specifically.

As of 1.0.0, sum(...) will be the same type as the field it is called on.


SuperChart module error

i try to use superchart module
import { SuperChart } from '@superset-ui/chart';
export class Chart extends React.Component {
render() {
const { height, width,formdata,payload,CHART_TYPE } = this.state;
return  (
<SuperChart type={CHART_TYPE} chartProps={{ formdata, payload, width, height }} />
);
}
but  when i add data to the values  i am facing some issues

The codebase on this repo has been moved to the main Apache Superset repo, and consequently the repo is in the process of being archived. See the Superset Improvement Proposal for details: https://github.com/apache/superset/issues/13013 . While all currently open issues and PRs will be closed, we encourage you to reopen this issue on the main repo if it is still relevant.


[Refactor] Update string

Purpose

Update the String

Does this introduce a breaking change?

- [ ] Yes
- [x] No

Pull Request Type
What kind of change does this Pull Request introduce?

- [x] Bugfix
- [ ] Feature
- [ ] Code style update (formatting, local variables)
- [ ] Refactoring (no functional changes, no api changes)
- [ ] Documentation content changes
- [ ] Other... Please describe:

Pull Request Checklist

- [ ] Public API changes
- [ ] Verified for cross-platform
- [ ] Synced with iOS, Web team
- [ ] Internal review completed
- [ ] UI Changes
- [ ] Screen captures included
- [ ] Tested for Light/Dark mode
- [ ] Tested for screen rotation
- [ ] Tested on Tablet and small screen device (5")
- [ ] Include localization changes
- [ ] Tests
- [ ] Memory leak analysis performed
- [ ] Tested on API 21, 26 and latest
- [ ] Tested for background mode
- [ ] Unit Tests Included
- [ ] UI Automated Tests Included

How to Test

Open...
Click on...

What to Check
Verify that the following are valid

...

Other Information

tested for FR and GR. looks good


hello I'm having the following error, how do I fix it?

0104:err:vulkan:wine_vk_init Failed to load vulkan library
0104:err:wgl:init_opengl Failed to load libGL: libGL.so.1: classe ELF errada: ELFCLASS64
0104:err:wgl:init_opengl OpenGL support is disabled.
0104:err:d3d:wined3d_caps_gl_ctx_create Failed to find a suitable pixel format.
0104:err:d3d:wined3d_adapter_gl_init Failed to get a GL context for adapter 007692D8.
0104:err:virtual:virtual_setup_exception stack overflow 96 bytes in thread 0104 addr 0x20006260 stack 0x520fa0 (0x520000-0x521000-0x620000)

Make sure you have vulkan pkgs install.

which ones ?

not sure which distribution you have but try this guide getting pkgs link


chore: update volar dependencies

Changes
Fix https://github.com/withastro/language-tools/issues/970
Testing
Should pass!
Docs
N/A

Performance is now correct, however it seems like this broke completions in certain contexts...

Yeah. Let's my fault 😅. Have to wait for typescript-auto-import-cache 0.3.5.

Yeah. Let's my fault 😅. Have to wait for typescript-auto-import-cache 0.3.5.

Absolutely no worries, deeply appreciate your contributions to typescript-auto-import-cache, we all win by contributing to the ecosystem ❤


Remove uuid

This removes the uuid package, replacing it with the native crypto.randomUUID instead.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.01%) to 90.441%

Files with Coverage Reduction
New Missed Lines
%

src/routes/transactions/entities/tests/human-description.builder.ts
2
80.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
5387

Relevant Lines:
5833

💛  - Coveralls


Use ISO format for DateTimeType

Currently DateTimeType uses Y-m-d H:i:s instead of ISO 8601. From compatibility point of view this format is not preferable.
I know that this could be potentially BC break but maybe in future releases.

The DateTimeType has an optional parameter format you can use to specify the format you want the date formated to:
class MyCustomType extends AbstractObjectType
{
    public function build($config)
    {
        $config
            ->addField('id', new IdType())
            ->addField('createdAt', new DateTimeType('c')); // Prints date in ISO 8601: 2004-02-12T15:19:21+00:00
    }
}

Format follow the one accepted by the date function.

@fesor It's going to be changed in 2.0 I guess, but we're going to keep the old one in the current brunch.
@fritz-gerneth thanks for replying here!


LICENSE: Remove project name from line 1 to fix auto-detection of MIT License in GitHub.

Fixes #4340

Demonstration of the fix: compare license files between ChakraCore and my repo:
https://github.com/Microsoft/ChakraCore/blob/master/LICENSE.txt
https://github.com/dilijev/ChakraCore/blob/master/LICENSE.txt

This change is approved by legal ✔

@dotnet-bot test OSX static_osx_osx_debug please

ChakraHub doesn't like source branches and target branches names to match. Replacing this PR with #4343


[Support Edge to edge] 1/2 - Update more edge to edge ui usecases

What happened 🤔
To prepare necessary use cases for the next Edge-to-edge support on this template, we should try to modify and add some more UI contexts:

List item on Home screen
Detail screen on the second screen
Web view
etc, ...

Insight 👀

Update repository and usecase to return a list of Data item
Update HomeFragment to add DataAdapter
Add WebViewFragment, with a new navigation from SecondFragment.

PoW (a.k.a Proof Of Work)💪
https://user-images.githubusercontent.com/16315358/113273571-fa199700-9306-11eb-8fef-728b3d228111.mp4

@chornerman this one also, please 🥺

@luongvo Please check the failed test 🙏

All fixed @minhnimble @hoangnguyen92dn @manh-t ✅

@minhnimble @chornerman Please finalize your reviews on the last update from Lucas 🙏