[M4/M5/M6] Read Hive kubeconfig from env vars and deploy via aro deploy

Which issue this PR addresses:
Fixes https://dev.azure.com/msazure/AzureRedHatOpenShift/_workitems/edit/14625923 for now
What this PR does / why we need it:
Adds the liveconfig but using the approach in #2298 . Also uses a user config, not an admin config.
Test plan for issue:
Has unit tests, although not sure if it will function in CI.
Is there any documentation that needs to be updated for this PR?
probably not

/hold
Holding this PR as it implements some of the live config changes that we discussed would be released after our go live date for this project. A new PR will be opened that will either require changes here or for this to be closed. cc/ @UlrichSchlueter @petrkotas

I think https://github.com/Azure/ARO-RP/pull/2325 is a better solution since vars in this manner are not encrypted.

Closing in preference for #2325


e2e CI failure :name 'PEP440Version' is not defined"

Describe the version
version about:

kubean:v0.6.6-13-ge3831fb2-e2e
Describe the bug
e2e create cluster

Screenshots and log
https://github.com/kubean-io/kubean/actions/runs/5417793162/jobs/9849150856

spray-job image is missing python package packaging


Combine the two OpenCL files into one, since Mac drivers have some bi…

…zarre problem with #include. #15
@mfbenitezp, this works fine on Linux. If you can, try it on Mac and see if it helps?

That is great. I will try that first using poetry.

Look!

Although I had to run it using sudo bcs of this error.
PermissionError: [Errno 13] Permission denied: '/Users/fbenitez/Library/Caches/pytools/pdict-v4-pyopencl-invoker-cache-v7-py3.9.12.final.0'
Then I run well with this weird warning:
Loading snapshot from data/snapshots/WestYorkshireSmall/cache.npz
Snapshot is 6 MB
/Users/fbenitez/Library/Caches/pypoetry/virtualenvs/aspics-2CNdlyoq-py3.9/lib/python3.9/site-packages/pyopencl/__init__.py:252: CompilerWarning: Non-empty compiler output encountered. Set the environment variable PYOPENCL_COMPILER_OUTPUT=1 to see more.
  warn("Non-empty compiler output encountered. Set the "
22856 places are part of this study area

Awesome! The obvious hack works. The permission thing is weird, but easy to workaround. The warning isn't helpful; it doesn't say what the opencl compiler said. :P   But merging, since this works.


ADS new query page issues

New query page shows disconnected when being opened, but runs query successfully.
Results tab does not show anything after non-select query is executed. Messages tab shows message related with the operation though. (Not sure if this is intended or a bug)

Yeah I'm seeing this as well. Seems like something is seriously broken with the editor as we also have #5746 and #5747 that just now popped up within the last few days.
I've assigned these to myself for now and will look at them today, but if someone else wants to get to them first let me know and we can try to split the work.

Thank you @Charles-Gagnon !


MeltingPot examples do not work

The examples need to be updated to ensure they can run.
Currently there are dependency conflicts in the MeltingPot examples/requirements.txt
stable-baselines3 2.2.1 requires gymnasium<0.30,>=0.28.1, but you have gymnasium 0.26.3 which is incompatible.
supersuit 3.9.2 requires gymnasium>=0.28.1, but you have gymnasium 0.26.3 which is incompatible.
pettingzoo 1.24.3 requires gymnasium>=0.28.0, but you have gymnasium 0.26.3 which is incompatible.

No workaround right now, sorry.

Hi John,
We have a fully working versions of baselines in the repository that we used for the contest and that I'm maintaining. This has been well tested and documented and used by several users.
There are two possible resolutions to this bug:
(i) You can remove the examples folder and point to this new repo in Readme for baselines.
(ii) I can push upstream a portion of that repo (working version lightweight files similar to current examples one - and this was our initial plan) to refresh the examples folder.
Please let me know which sounds good to you and I will do that.
Best,
Rakshit

When I test training, like that:
cd examples/rllib
python self_play_train.py
--------------------------->
Display:
2024-04-02 17:56:17.501082: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Traceback (most recent call last):
File "/home/oeuser/miniconda3/envs/mpnn/lib/python3.11/site-packages/mp_home/examples/rllib/self_play_train.py", line 25, in
from . import utils
ImportError: attempted relative import with no known parent package

python -m rllib.self_play_train
------------------------------->
2024-04-03 12:35:40.307164: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in _run_code
File "/home/oeuser/miniconda3/envs/mpnn/lib/python3.11/site-packages/mp_home/examples/rllib/self_play_train.py", line 22, in
from ray.rllib.algorithms import ppo
File "/home/oeuser/miniconda3/envs/mpnn/lib/python3.11/site-packages/ray/rllib/init.py", line 7, in
from ray.rllib.env.base_env import BaseEnv
File "/home/oeuser/miniconda3/envs/mpnn/lib/python3.11/site-packages/ray/rllib/env/init.py", line 1, in
from ray.rllib.env.base_env import BaseEnv
File "/home/oeuser/miniconda3/envs/mpnn/lib/python3.11/site-packages/ray/rllib/env/base_env.py", line 6, in
from ray.rllib.utils.annotations import Deprecated, DeveloperAPI, PublicAPI
File "/home/oeuser/miniconda3/envs/mpnn/lib/python3.11/site-packages/ray/rllib/utils/init.py", line 6, in
from ray.rllib.utils.filter import Filter
File "/home/oeuser/miniconda3/envs/mpnn/lib/python3.11/site-packages/ray/rllib/utils/filter.py", line 11, in
from ray.rllib.utils.serialization import _serialize_ndarray, _deserialize_ndarray
File "/home/oeuser/miniconda3/envs/mpnn/lib/python3.11/site-packages/ray/rllib/utils/serialization.py", line 23, in
old_gym_text_class = getattr(old_gym.spaces, "Text", None)
^^^^^^^^^^^^^^
AttributeError: module 'gym' has no attribute 'spaces'

I tried to execute it in the Codespaces container of GitHub, but the examples were also not able to be trained.

@rstrivedi I'll see what @duenez wants to do, as he's been handling the examples.
@gogo16888 as you've been finding and as this issue documents, the examples don't work. See Rakshit's comment above for a repo with examples that do work.

I check the baselines of Melting-Pot-Contest-2023, it use ray 2.6.1, but the examples use ray 2.5.0, so if I use the baselines, I need update to ray 2.6.1?  But the install of Melting-Pot-Contest-2023 is not as Melting-Pot 2.2.2, I am not sure.

If possible, please export the yml file for each version for easy verification of various software packages/tools/library versions during installation.

I check the baselines of Melting-Pot-Contest-2023, it use ray 2.6.1, but the examples use ray 2.5.0, so if I use the baselines, I need update to ray 2.6.1? But the install of Melting-Pot-Contest-2023 is not as Melting-Pot 2.2.2, I am not sure.

Thank you for your message. We're in the process of updating the examples folder in this repository. Once it is done, everything should match and flow seamlessly.
In the meantime, for your question, it is generally recommended to use ray >= 2.6.1 as there were some known issues with the older versions. So, you can use the combination of:

latest version of melting-pot 2.2.2 (from this repository)
either ray 2.6.1 or ray 2.20.0 (the latest version)
the baselines code from Melting-Pot-Contest-2023 repository

This combination should work without any errors (I've tested this and it works on our end) but let us know if you hit any issues. I'll post here once the examples folder in this repository is updated.


Getting a TAPAS

I too would like to get a tapas board. Please let me know how.

Join the TAPAS community challenge https://challenge.tapas.sdi.tools/, pitch an idea and get a free board. There are also neat prizes to win - if you like that kind of thing.


Remove unnecessary wrappers for goroutines.

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Remove unnecessary wrappers for goroutines.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/lgtm
/approve


im having using the free checking printing

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
'dotnet' is not recognized as an internal or external command,
operable program or batch file. on window 7,  <  dotnet  comand

This tutorial is for you to learn how to code. Is this really what you intended? Can you better describe the issue?


Make common/bundled dialects public

Gorm is great when working with supported databases. However, it's much harder to write a custom dialect than it needs to be. I need to write a custom dialect for redshift. This is 95% the same as postgres, but needs special support around a couple of things, for instance lack of RETURNING support.
Ideally, I would do:
type Redshift struct {
  gorm.Postgres
}

func (s Redshift) LastInsertIDReturningSuffix(tableName, key string) string {
  return ""
}

And be done. However, because postgres is not public, I have to reimplement every single part of the dialect. And what's worse, postgres itself composes commonDialect which is also not public, so I have to reimplement all of those methods too.
Making dialects public would allow us to make minimal modifications and react to future database products without requesting gorm support for them. cf. https://github.com/jinzhu/gorm/issues/1816, https://github.com/jinzhu/gorm/issues/1572.

All makes sense, and just some renaming correct ?


Release 0.4.7

@JuliaRegistrator register(branch="release/v0.4")

Registration pull request created: JuliaRegistries/General/64123
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.4.7 -m "<description of version>" 76b7b2b507939c6fd78c97a0832bf32b4b9a946c
git push origin v0.4.7


Add README

Documentation of this activity was recently migrated from the wiki.
See godiard/help-activity/pull/42
This PR adds a basic README where 'How to use Turtle Confusion' points to the migrated user-documentation.
Feel free to add/subtract information as necessary from the README.md file.

Thanks.


Implement groupBy construct

Currently all files are just read as is without any logical grouping or ordering. Implement a basic flow to provide a function to group files if wanted based on media type (ie: group by album for .mp3)

solved by #18


Mod Submission: CareerManager

Here's a link to my .ckan file. I dint know what else to say, thanks!

Nevermind, using netkan now


As a computer science student I want to be able to mark assignments as completed

I want to be able to mark assignments as completed, so I can track my progress.

mark INDEX


Function for combining SingleCellExperiment objects

Before being able to do integration, we will need to be able to merge SCE objects into one object to obtain combined highly variable genes and calculate PCA. Additionally, the input for the integration methods we plan to support use a merged SCE object.
This function should take as input a list of individual SCE objects, all already containing normalized data. Inside the function we will need to match up all of the colData, filling in any data that is only found in part of the libraries being integrated with NA. We will also want to subset to only shared genes. The new columns of this merged object should be named with the cell barcode and the library ID as barcodes could be overlapping between libraries.
The output should be the merged object with a new column in the colData corresponding to the library ID for each cell.

A couple quick discussion items for this, specifically in how it might differ from our original implementation:

Originally, we check if column names match among SCEs and we add a "dummy" column with all NA values if any columns are missing to ensure a full match. I would tend to prefer not to do this here but just error out fully if all colData names don't match.
Originally we removed any altExps before merging SCEs. I think we need to keep altExps here (with associated book-keeping to always link back to originating data), or at least have an argument for whether the altExps should be preserved.
Originally we allow for a vector of rowData column names which should not be altered, and I think we should still do this (keeping default NULL).
Originally we removed any colData columns other than batch, percellQC columns, and mitochondria columns. I'm on the fence for whether we want to keep this?

Originally, we check if column names match among SCEs and we add a "dummy" column with all NA values if any columns are missing to ensure a full match. I would tend to prefer not to do this here but just error out fully if all colData names don't match.

I think you could have an option to require matching colData or not. With data from scpca-nf we would expect them to fully match, but if someone is using this package for their own data then who knows what type of colData is in each object and I don't think it's necessarily required that all colData be present for all datasets.
Maybe instead of requiring matching colData, we have an option to input columns to preserve. All other columns would be removed.

Originally we removed any altExps before merging SCEs. I think we need to keep altExps here (with associated book-keeping to always link back to originating data), or at least have an argument for whether the altExps should be preserved.

Yes to the argument... I think generally we probably want to keep them.

Originally we allow for a vector of rowData column names which should not be altered, and I think we should still do this (keeping default NULL).

Yes agreed on this.

Originally we removed any colData columns other than batch, percellQC columns, and mitochondria columns. I'm on the fence for whether we want to keep this?

See my comment above on including an argument for column data to preserve. But generally I don't think we should remove all but these columns. That was specific to removing all the extra data calculated by HCA. I think we should have an argument of columns to preserve and the default would be the columns that are output by scpca-nf (so per cell QC, miQC, and batch/libraryID).


fix: Harmonization between modal sheets

Hi everyone!
The "old" modal sheets were not aligned with the colors of the Product page.

Orange indicator added

In dark mode, when the first item was selected, it had the same color as the header

The same for the image source

User lists: color harmonization + a brighter button

Users lists in dark mode: color harmonization

Codecov Report
Attention: Patch coverage is 0% with 35 lines in your changes missing coverage. Please review.

Project coverage is 6.83%. Comparing base (4d9c7fc) to head (a35cec9).
Report is 481 commits behind head on develop.

Files with missing lines
Patch %
Lines

...generic_lib/bottom_sheets/smooth_bottom_sheet.dart
0.00%
24 Missing :warning:

...ib/pages/preferences/user_preferences_widgets.dart
0.00%
7 Missing :warning:

...s/smooth_app/lib/pages/all_product_list_modal.dart
0.00%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5875      +/-   ##
==========================================
- Coverage     9.54%   6.83%   -2.72%
==========================================
  Files          325     428     +103
  Lines        16411   23593    +7182
==========================================
+ Hits          1567    1613      +46
- Misses       14844   21980    +7136

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[SPARK-20233] [SQL] Apply star-join filter heuristics to dynamic programming join enumeration

What changes were proposed in this pull request?
Implements star-join filter to reduce the search space for dynamic programming join enumeration. Consider the following join graph:
T1       D1 - T2 - T3
  \     /
    F1
     |
    D2

star-join: {F1, D1, D2}
non-star: {T1, T2, T3}

The following join combinations will be generated:
level 0: (F1), (D1), (D2), (T1), (T2), (T3)
level 1: {F1, D1}, {F1, D2}, {T2, T3}
level 2: {F1, D1, D2}
level 3: {F1, D1, D2, T1}, {F1, D1, D2, T2}
level 4: {F1, D1, D2, T1, T2}, {F1, D1, D2, T2, T3 }
level 6: {F1, D1, D2, T1, T2, T3}

How was this patch tested?
New test suite StarJOinCostBasedReorderSuite.scala.

Test build #75561 has started for PR 17546 at commit 5eb9b30.

@wzhfy @gatorsmile @cloud-fan I've integrated star-join with join enumeration. Would you please take a look? Thanks.

Test build #75561 has finished for PR 17546 at commit 5eb9b30.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class JoinReorderDP(conf: SQLConf) extends PredicateHelper with Logging
case class JoinReorderDPFilters(conf: SQLConf) extends PredicateHelper
case class JoinGraphInfo (starJoins: Set[Int], nonStarJoins: Set[Int])

Merged build finished. Test PASSed.

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

So now we have two entry points for star schema, one is in ReorderJoin and the other is in CostBasedJoinReorder?

Test build #75583 has started for PR 17546 at commit 9e81154.

@wzhfy Yes, star-schema is called from both ReorderJoin and CostBasedJoinReorder.

Test build #75583 has finished for PR 17546 at commit 9e81154.

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

Merged build finished. Test PASSed.

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

This looks pretty good over all.

Test build #75616 has started for PR 17546 at commit 830255c.

Test build #75616 has finished for PR 17546 at commit 830255c.

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

Merged build finished. Test PASSed.

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

@cloud-fan Do you have any comments?

Test build #75674 has started for PR 17546 at commit 7a5d1d0.

Test build #75674 has finished for PR 17546 at commit 7a5d1d0.

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

Merged build finished. Test PASSed.

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

Test build #75713 has started for PR 17546 at commit c9ec7c9.

Test build #75713 has finished for PR 17546 at commit c9ec7c9.

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

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

Merged build finished. Test PASSed.

LGTM, merging to master!

@cloud-fan Thank you for merging!


test: bump CNI version to 0.7.5 for K8s 1.12

The CI fails otherwise, with the following error:

20:22:58      k8s1-1.12: The following packages have unmet dependencies:
20:22:58      k8s1-1.12:  kubeadm : Depends: kubernetes-cni (>= 0.7.5) but 0.6.0-00 is to be installed
20:22:58      k8s1-1.12:  kubelet : Depends: kubernetes-cni (>= 0.7.5) but 0.6.0-00 is to be installed
20:22:58      k8s1-1.12: E: Unable to correct problems, you have held broken packages.

Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

test-me-please


Invalid weapon ranges

It seems that this commit removed the correct ranges for ranged weapons and replaced them all with melee stats.
https://github.com/adrpadua/5e-database/commit/e05a87387c4abfa419079494c0748369fbe14612#diff-3020366f2048c53f42ed3e0dc6549dbaL280
This seems like it had to be a conscious decision. Was it?

That seems like a mistake.

If you want to put up a PR to revert the changes on ranged weapons, I'll happily take a look.

Fixed if you want to double check before I merge it in.


Enable Vertical Scaling feature-gate and add support for testing VM pods in E2E test

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
kind feature
/kind flake

What this PR does / why we need it: This adds E2E test support for Vertical Scaling for VM pods, and enables the feature-gate to allow it to be tested in CI/CD.
Which issue(s) this PR fixes: #666 666

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

To run Vertical Scaling E2E test for container:
kubetest --test --provider=local --test_args="--ginkgo.focus=PodInPlaceResizeContainer"
To run Vertical Scaling E2E test for VM:
kubetest --test --provider=local --test_args="--ginkgo.focus=PodInPlaceResizeVM"
To run it for both:
kubetest --test --provider=local --test_args="--ginkgo.focus=PodInPlaceResize"

Close staled PR


Remove lastPosBlockOffset from term metadata for Lucene90PostingsFormat

Description
lastPosBlockOffset is used to identify the last non-packed-encoded positions block. The same information can be derived use totalTermFreq.
This could save space taken by the FSTs for positions enabled fields.

I quickly tried this out and realized it doesn't work when we need to skip with the skip lists we we lost track of how many positions that got skipped.

Sorry, what went wrong when you tried to remove lastPosBlockOffset?  Skipping is (currently) only at every 8 blocks boundaries right?  So doesn't the skip data already know how many positions were skipped when it skips into a spot?
Hmm I think I see -- there are in general more positions blocks (since there can of course be many more positions than doc/freq) than doc/freq blocks, and so as PostingsEnum is skipping or reading positions, it must know when it transitions into the final (vInt encoded) chunk of positions?  Hmm but again it seems like tracking number of positions skipped/read vs totalTermFreq should be enough?
Maybe make a PR adding a nice comment explaining why we really do need this int lastPosBlockOffset?

Today when it skips, the skipper can tell us 1) the offset of the position block we should seek to 2) how many positions it needs to skip within this block. This is because post-skipping the first doc's position is not always at the start of a positions block (that's actually the most common case). Additionally, advancing to a specific doc that is not the start of a block means we need to skip those docs as well as their positions.
In theory, if the skipper can tell us how many positions it has skipped that would work. This will require storing more information in the skip data than the current scheme.

Maybe make a PR adding a nice comment explaining why we really do need this int lastPosBlockOffset?
Sure, I can enhance the documentation of IntBlockTermState

https://github.com/apache/lucene/pull/12541 adds more comment for lastPosBlockOffset

https://github.com/apache/lucene/pull/12541 is merged and I'll close this one


DesyncIssue

Product and Version [VS/VSCode]: VSCode
OS Version [macOS/Windows/Linux]: Windows 10
Live Share Extension Version: standard
Target Platform or Language [e.g. Node.js]: C#, a clean .cs file from Unity Engine
What were you doing when you encountered the desync?
Nothing but watching my friend programming in a single file
How often have you encountered desyncs?
First time this one

Thanks for reaching out! Desyncs are usually hard to debug and reproduce, so any extra information would be helpful. Do you remember what your friend was doing when the desync happened? (building, typing, renaming, etc).

I'm not really sure, but i think my friend was typing on visual studio, while i was watching with vscode. The only other thing possible is he was switching from unity to visual studio to unity again, but not so quickly though. Maybe he was testing a script so made little change and test it again. Thanks for support!

Da: Daniel Lebu<EMAIL_ADDRESS>Inviato: lunedì 6 aprile 2020 17:20
A: MicrosoftDocs/live-share<EMAIL_ADDRESS>Cc: Simone<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Oggetto: Re: [MicrosoftDocs/live-share] DesyncIssue (#3362)
Thanks for reaching out! Desyncs are usually hard to debug and reproduce, so any extra information would be helpful. Do you remember what your friend was doing when the desync happened? (building, typing, renaming, etc).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/live-share/issues/3362#issuecomment-609859895, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJL6MB4EI55TG65X4C2BV5TRLHXMHANCNFSM4L7FXDTA.

I'm going to close this issue as a duplicate of #2086, so that we can consolidate the conversation there. If you can share any more details about your repro, it would be amazingly helpful if you could add them to that issue. Thanks!


Prevent Segmentation Faults when adding arcs before adding node.

Idk if this should be fixed, but could this be handled with an exception? or ..?
I know I need to init a node first.

Yes, I will look into adding a test and an exception for this if it doesn't slow down the graph building process too much.

Yes, I will look into adding a test and an exception for this if it doesn't slow down the graph building process too much.

Cool, let me know if I could help with this if at all, although I can't promise  i am comfortable with C++ tbh.


link to file from its history

Fixes #3852
Fixes https://github.com/go-gitea/gitea/issues/26707
Add a button on file history which directs you to the file at the selected commit.

Also fixes https://github.com/go-gitea/gitea/issues/26707 which might be duplicate.

Can we label the column "Actions" like I did in https://github.com/go-gitea/gitea/pull/26998?

@silverwind I applied the suggested changes

Can we move the "Copy SHA" button to Operations as well? Same styling and all.

Done

Did some tweaks:

I was thinking about getting rid of the whole table header. For example, branch list is without header as well.

The "View File" name is inaccurate when viewing a directory's history. Maybe we should use the same or similar string that GitHub does? "View at this point in the history", I would omit the the in the string.

@silverwind Where did you push your changes? :laughing:

Did some tweaks:

Could you please update the screenshot in the PR description?

I'd wait for the tweaks by silverwind as they change the appearance a bit

Date column header can be left aligned.

Ah it seems it did not push, I was getting "remote rejected", will check :D

Seems I still can not push (is collaboration enabled on this PR?):
 ! [remote rejected]       denyskon/commit-file-link -> denyskon/commit-file-link (permission denied)

Here is the patch:
diff --git a/options/locale/locale_en-US.ini b/options/locale/locale_en-US.ini
index 8667e77a89..ec37d64fec 100644
--- a/options/locale/locale_en-US.ini
+++ b/options/locale/locale_en-US.ini
@@ -96,8 +96,9 @@ disabled = Disabled
 locked = Locked

 copy = Copy
 copy_url = Copy URL
+copy_hash = Copy hash
 copy_content = Copy content
 copy_branch = Copy branch name
 copy_success = Copied!
 copy_error = Copy failed
diff --git a/templates/repo/commits_list.tmpl b/templates/repo/commits_list.tmpl
index c5003af4ae..db658de3e7 100644
--- a/templates/repo/commits_list.tmpl
+++ b/templates/repo/commits_list.tmpl
@@ -75,11 +75,13 @@
 							{{TimeSince .Committer.When ctx.Locale}}
 						{{else}}
 							{{TimeSince .Author.When ctx.Locale}}
 						{{end}}
-
-							<button class="btn interact-bg" data-clipboard-text="{{.ID}}">{{svg "octicon-copy"}}</button>
-							{{if $.FileName}}{{svg "octicon-file-code"}}{{end}}
+
+

+								<button class="btn interact-bg gt-p-3" data-tooltip-content="{{ctx.Locale.Tr "copy_hash"}}" data-clipboard-text="{{.ID}}">{{svg "octicon-copy"}}</button>
+								{{if $.FileName}}{{svg "octicon-file-code"}}{{end}}
+

 				{{end}}

I like  more than the massive gt helpers.

@delvh @silverwind I pushed the changes and updated the screenshot.

I like  more than the massive gt helpers.

Need the flexbox to vertically center and for the gap. Text-based layout is not suitable for these tasks.


Add Compatibility Guarantee to docs

@pauldix @jsternberg do we still want to publish a compatibility guarantee in the 1.0 docs? If so, is the finalized text available somewhere?

We're going to skip this at the moment and come back to it later.

I think it would be a good idea to write something up. At least guidance to our users about what to expect. I can write something up in the next few days.

@pauldix still want to do this?


HTTP2 Connect support

Description:
It should be possible to use the HTTP Verb "CONNECT" to start a tcp tunnel  through envoy to an upstream target.
This would have the benefit that one can use the http filters during initialization of  the tcp tunnel (i.e. oauth2 filter for authentication and ext_auth for authorization).
Here is a small PoC with pure envoy config:
https://github.com/denniskniep/envoy-http-connect-tunnel-demo
Assuming this example HTTPRoute:
apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
  name: backend
  namespace: default
spec:
  hostnames:
  - www.example.com
  parentRefs:
  - group: gateway.networking.k8s.io
    kind: Gateway
    name: eg
  rules:
  - backendRefs:
    - group: ""
      kind: Service
      name: backend
      port: 3000
    matches:
    - path:
        type: PathPrefix
        value: /

This example will be translated to this VirtualHost:
{
       "virtual_hosts": [
              {
                     "name": "default/eg/http/www_example_com",
                     "domains": [
                            "www.example.com"
                     ],
                     "routes": [
                            {
                                   "match": {
                                          "prefix": "/"
                                   },
                                   "route": {
                                          "cluster": "httproute/default/backend/rule/0",
                                          "upgrade_configs": [
                                                 {
                                                        "upgrade_type": "websocket"
                                                 }
                                          ]
                                   },
                                   "name": "httproute/default/backend/rule/0/match/0/www_example_com"
                            }
                     ]
              }
       ]
}

To make this HTTPRoute able to handle HTTP Connect requests we need to configure the following:
  virtual_hosts:
  - name: default/eg/http/www_example_com
    domains:
    - "www.example.com"
    routes:
    - match:
        connect_matcher:
          {}
      route:
        cluster: "httproute/default/backend/rule/0",
        upgrade_configs:
        - upgrade_type: CONNECT
          connect_config:
            {}

We need to utilize the connect_matcher (see here) for route matching.
Important is that the  connect_matcher can not be used together with prefix matcher! (Because of the precisely one requirement)
And we need to specify appropriate upgrade_configs (see here)
Additionally there are further HTTP CONNECT related settings on HttpConnectionManager
But they don´t seem to have any effect on the actual functionality, if the ones on route level are set appropriately. See:

HttpConnectionManager upgrade_configs
HttpConnectionManager http2_protocol_options allow_connect

What would be the "right way" to model this in/as envoy gateway resource?
Does it make sense to add a property to the ClientTrafficPolicy? This would take effect on the specified route if targetrefs sectionname is set to the HTTPRoute or affect every route if targetRef is set on a gateway level ?
Adding the following property to ClientTrafficPolicy. A dedicated object gives us the opportunity to easily add more of these config props later if necessary
connectConfig: {
  enabled: true,

}

@arkodg any feedback?
(not stale)

bump

not stale

not stale


Can not shrink datagrid columns on Chrome

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior
I am creating clarity datagrid columns by iterate column definitions with *ngFor.
I expect the created clarity datagrid columns to be shrinkable.
Actual behavior
When create clarity datagrid columns (clr-dg-column elements)  with *ngFor, they can't be shrink on Chrome.
The shrink of the datagrid columns work on FF and IE.
Reproduction of behavior
https://plnkr.co/edit/UGgTYUNIQMOm19pxgbXp?p=preview
Environment details

Angular version:4.x.x

Clarity version:10.06

OS and version: Windows

Browser: Chrome 62.0.3202.75

@dkanev: Thanks for reporting this. We will take a look and get back to you.
@Shijir: Would you mind taking a look at this.

This is the duplicate of https://github.com/vmware/clarity/issues/1537.

I've opened this issue again as it reproduces the same problem in a different way.


Navigation doesn't work with verse spans

This could impact whether navigation goes into 0.9 or 1.0.

Not seeing Navigation bar in v0.3.0 build cf4f0aa. This would be made available in the next Release.


Sort arrows should be customizable, both asc and desc

I am using the module to render the table. I use font-awesome as the arrow icons of the sort header.
In the code, the default way to change the asc/desc icon is adding and removing the class "up" and "down"
I am suggesting that the following current usage:
$.bootstrapSortable({
    sign: "fa fa-sort"
})

can be improved as the following:
$.bootstrapSortable({
    sign: {
        "asc":  "fa fa-sort-asc",
        "desc": "fa fa-sort-desc"
    }
});

On this way the arrows can be customized more easily
Is it the good way to do so?

that would be possible, but please try this, if it works for you: https://astronautweb.co/snippet/font-awesome/

+1
Would also be nice to have an up/down arrow on all sortable columns to show users that the headers are clickable for sorting purposes.


Echo example use toString explicitly

As a user I would like to experiment with Echo, and replace the String by another Type.
Currently, this results in parse errors.
When toString and fromString are used at the right places in Sim and Sut, this works immediately.

(Unconstrained) Strings are problematic:
Strings can contain the newline character '\n'
yet TorXakis assumes line-based communication over Sockets!

issue solved


Test! Image writerside-builder:242.2187 is not found

I implemented github-workflows as described in README.md.
Occured bellow error.
https://github.com/link
And created PullRequest to resolve this problem.
https://github.com/JetBrains/writerside-github-action/pull/

Test comment!
I implemented github-workflows as described in README.md.
Occured bellow error.
https://github.com/link
And created PullRequest to resolve this problem.
https://github.com/JetBrains/writerside-github-action/pull/

Test comment!
I implemented github-workflows as described in README.md.
Occured bellow error.
https://github.com/link
And created PullRequest to resolve this problem.
https://github.com/JetBrains/writerside-github-action/pull/

Test comment!
I implemented github-workflows as described in README.md.
Occured bellow error.
https://github.com/link
And created PullRequest to resolve this problem.
https://github.com/JetBrains/writerside-github-action/pull/

Test comment!
I implemented github-workflows as described in README.md.
Occured bellow error.
https://github.com/link
And created PullRequest to resolve this problem.
https://github.com/JetBrains/writerside-github-action/pull/

Test comment!
I implemented github-workflows as described in README.md.
Occured bellow error.
https://github.com/link
And created PullRequest to resolve this problem.
https://github.com/JetBrains/writerside-github-action/pull/

Test comment!
I implemented github-workflows as described in README.md.
Occured bellow error.
https://github.com/link
And created PullRequest to resolve this problem.
https://github.com/JetBrains/writerside-github-action/pull/

Test comment!
I implemented github-workflows as described in README.md.
Occured bellow error.
https://github.com/link
And created PullRequest to resolve this problem.
https://github.com/JetBrains/writerside-github-action/pull/

Test

Test

One more test


logger: use eprintln! if the logger is not initialized

ref #11651
Signed-off-by: Wenbo Zhang<EMAIL_ADDRESS>
What is changed and how it works?
Close #11651
What's Changed:
Related changes

Need to cherry-pick to the release branch

Check List
Tests

Manual test (add detailed scripts or steps below)
No code

Side effects

None

Release note
None

/label needs-cherry-pick-5.4

Is it possible to register a term logger first, and then change it to a configured one? So we can always use info/debug/error.

/merge


Rhum.mock doesn't track nested method calls

Summary
See comment from Discord:

Root Cause
The issue is with this block of code in mock_builder.ts#create():
        mock[method] = function (...args: unknown[]) {
          mock.calls[method]++;
          return (original[method as keyof T] as unknown as (
            ...params: unknown[]
          ) => unknown)(...args);
        };

The above code turns every method in an object into a method that can track itself. For example:
add(number: string) { ... }

becomes
function(...args: unknown) {
  mock.calls["add"]++;
  return add(...args);
}

In the problematic block, there is this:
          return (original[method as keyof T] as unknown as (
            ...params: unknown[]
          ) => unknown)(...args);

This makes each method being mocked call the ORIGINAL object's method. So, what's happening in this issue is:

<real object>.add() is replaced with the wrapper function that can track calls via mock.calls[method]++
mock.add() is called
mock.add() calls mock.calls["add"]++ and mock.calls.add === 1 now
mock.add() calls the original version of itself
the original version of mock.add() calls the original version of realAdd()
the original version of realAdd() doesn't have mock.calls["realAdd"]++, so mock.calls.realAdd === 0

Example test code
import { Rhum } from<EMAIL_ADDRESS>
class MathService {
  realAdd(num1: number, num2: number): number {
    return num1 + num2;
  }
  add(num1: number, num2: number): number {
    return this.realAdd(num1, num2);
  }
}

const mock = Rhum.mock(MathService).create();

Rhum.asserts.assertEquals(mock.calls.realAdd, 0);

// Since this calls `.readAdd()`, we expect `mock.calls.realAdd` to be `1` below, but it is not
mock.add(1, 1);

// Should be 1, not 0
Rhum.asserts.assertEquals(mock.calls.realAdd, 1);

Rhum.run()

I can confirm this is working now.

thanks for confirming @stav ! also, thank you for bringing this up!

Well I sure do love working on actively maintained software for just these kinds of contributions.
I actually did try and find the bug in the code but it was a bit over my head.
So I did this:
Rhum.testSuite("check", () => {

  let calls
  let xsocketMock

  class XapiSocketMock {
    check = check // code being tested
    trades = () => { calls.trades++ }
  }

  Rhum.testCase("should call trades once", async () => {
    calls = { trades: 0 }
    xsocketMock = new XapiSocketMock()
    await xsocketMock.check(data)
    Rhum.asserts.assertEquals( calls.trades.length, 1 )
  })

})

But now with your fix I can do:
Rhum.testSuite("check", () => {

  let xSocketMock

  class XapiSocketMock {
    check = check
    trades = () => {}
  }

  Rhum.testCase("should call trades once", async () => {
    xSocketMock = Rhum.mock(XapiSocketMock).create()
    await xSocketMock.check(data)
    Rhum.asserts.assertEquals( xSocketMock.calls.trades, 1 )
  })

})


Upload to pypi

We should publish it to pypi by 2022-06-02

https://pypi.org/project/napari-stress/ 💯


Call fallbacks only for timeouts and not for failures(exceptions)

Hello,
Is there a way disable fallbacks when method throws exception? I want the fallback to be called only if there is a timeout.
Thank you!

https://stackoverflow.com/questions/44247070/how-to-say-hystrix-not-to-trigger-fallback-for-some-of-the-exceptions-in-hystrix


Lack of documentation

Am I crazy or is there a total lack of documentation for the rtmp module? I would like to extend the functionality of that module and use it within my program but can't find any documentation for it anywhere.

I agree there should be more documentation. There are examples of config files on the docs website, but not much information about the options within them. Also, the "Quick Start" link on the homepage of the docs website 404s.

@kaitlynia  Please try the "Quick Start" link again to see if it's fixed or not


Better bound magicgui viewer

Description
This PR would take advantage of much improved parameter binding behavior from https://github.com/napari/magicgui/pull/95/
The behavior would be that when someone annotates a magicgui function with (viewer: napari.Viewer) no widget is shown (by default) and the widget.viewer parameter will use a callback to find its viewer necessary (returning None if the widget has not been added as a dock widget)
Tests will fail until magicgui 0.2.5 ... but we might want to make sure there are no lingering issues in the 0.4.3rc0 that need addressing as well
Type of change

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

References

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[ ] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

This seems reasonable, can you add a simple example/magic_with_viewer.py or something similar that someone can run to see this in action - doesn't have to do anything fancy.

This seems reasonable, can you add a simple example/magic_with_viewer.py or something similar that someone can run to see this in action - doesn't have to do anything fancy.

@sofroniewn, working now

@sofroniewn, working now

Release note added, will merge after tests pass!

Release note added, will merge after tests pass!


Expose the runtime-benchmarks feature to the cli crate

This exposes the runtime-benchmarks feature via the cli crate and
makes sure the benchmarking can be enabled. This requires that the user
goes to bin/node/cli and runs cargo build --features runtime-benchmarks to build a node that has the feature enabled.

Should we put some docs how to compile & run the benchmarks?


[Proposal] Creating a universal TTS Gateway for Wyoming

If you have time, would you consider creating an integration that allows to integrate various tts from HA into the wyoming pipeline.
Or interact with api of separate tts servers by specifying the format in the integration settings

my app needs a voice out service
i created a speak event, which takes the text and starts a Synthesize() event which causes AudioChunks, which get forwarded to the snd service

as those are both done in python for HA, one could create two services, one for each,  without too much trouble

Looks like any tts in HA can now be added to the voice pipeline.
I'm closing this Issue


Docs: fix typo in documentation

[x] Drop documentation of ak.highlevel.ArrayBuilder.extend
[x] Drop documentation of ak.layout.ArrayBuilder.extend
[x] Drop documentation of ak.layout.ArrayBuilder.append
[x] Fix typo of ArrayBuilder

Yes, please drop references to the extend method, since that's an old one that was removed. It used to have a different meaning than just calling append many times, now it wouldn't, if it were to exist.

I should have realised that it was just as likely that it was removed as added!


Wrong skeletonID in Blender export

We have two armatures in a .blend file. When exporting, using exporter 3.0.2, these get exported with the IDs 0 and 1 in the .babylon's skeletons array. Strangely, the meshes try to reference them by the skeletonIDs 0 and 4. 0 works, but 4 obviously doesn't.
The skeleton with ID 1 is not even loaded by BABYLON.SceneLoader.ImportMesh. In the callback function, Skeleton[] has a length of 1. When I change the wrong skeletonID from 4 to 1 in the .babylon file by hand, everything works as expected.
Is this an error on our part, or does the export mix up the IDs?
The scene in question is available at https://github.com/daniel-nth/babylon-skeletonid-test
Thanks for your awesome work!

Hello can you please use the forum for questions?

Sure, sorry.


update menu on hash change

Realize the function of update the menu when the url changes.

Perhaps it's time to get multiple dev pages - one for charts, and one for the pv-tree. Could you move out the menu and search box to a new dev page?

Sure, created a new mage menu_dev for pv-tree and search box


What purpose does this piece of code solve in main.css

https://github.com/Real-Dev-Squad/website/blob/27916a54feca6d048cc370d7005975cd084ba31c/css/main.css#L8-L17
Hi group, just want to know what purpose does this piece of code solves. I tried to look at the live website to understand but didn't get much.

I included that code for good looking background but someone removed only background property instead of removing full code of background from main.css.
If realdevsquad don't want designed background then they can remove this pseudo element from the code.
Img File: tswhirls.png

pseudo element is not removed. Only background property is removed.
Issue-#34

@invalidtoken @ankushdharkar
https://github.com/Real-Dev-Squad/website/pull/40
Please review this PR and merge.
This PR is solution for this issue-#39

@takshch so what's the difference between just setting the background of body and using pseudo-classes for setting the background. What are the advantages of using this pseudo-class approach?


feat: start adding support to cf pages

Fixes #96
What this PR solves / how to test:
This PR adds support for automatically instrumenting Cloudflare Pages. The code is very similar to the fetch code. I'd initially intended to combine the similar bits into a shared core, but as I started to get into those weeds I realized I didn't have a ton of context on the full set of differences between Pages and Workers, and it started to seem like keeping the code paths separate would allow for a more organic growth on the Pages instrumentation.
To test this code yourself it's as simple as wrapping a Cloudflare Pages handler in instrumentPage and passing it a valid TraceConfig.

Hey this would be really useful for a project I am working on. Is this usable as is? Are there plans to merge this?


Warning for solving example ODE from docu

The following example code from the docu produces a warning. How can i get rid of it?
import jax.numpy as jnp
from diffrax import diffeqsolve, ODETerm, SaveAt, Tsit5

def vector_field(t, y, args):
    prey, predator = y
    α, β, γ, δ = args
    d_prey = α * prey - β * prey * predator
    d_predator = -γ * predator + δ * prey * predator
    d_y = d_prey, d_predator
    return d_y

term = ODETerm(vector_field)
solver = Tsit5()
t0 = 0
t1 = 140
dt0 = 0.1
y0 = (10.0, 10.0)
args = (0.1, 0.02, 0.4, 0.02)
saveat = SaveAt(ts=jnp.linspace(t0, t1, 1000))
sol = diffeqsolve(term, solver, t0, t1, dt0, y0, args=args, saveat=saveat)

Warning:
'jax._src.interpreters.partial_eval.DynamicJaxprTracer'>. Attempting to hash a tracer will lead to an error in a future JAX release.
warnings.warn(

This is an upstream issue we spotted in recent versions of JAX -- see https://github.com/google/jax/issues/21824.
This is worked around on Diffrax's main as of https://github.com/patrick-kidger/diffrax/pull/440, so the warning will go away in our next release.
In the mean time you can filter out this warning with warnings.simplefilter.

Hi @patrick-kidger, should this be fixed with diffrax=0.6.0, or will it be for a later release? It seems I still see the warning on jax==0.4.30 and diffrax=0.6.0.

Right, so there were two different sources of this warning. One coming from Diffrax (fixed as above), and another from Equinox (fixed on HEAD but not yet released).
For now it should be safe to ignore the error. At some point soon I'll do another release of Equinox and then that one will be squashed too :)


Support for NanoTTS?

Can NanoTTS be used with Rhasspy?
https://github.com/gmn/nanotts

Are there any pre-compiled packages for this?

The Voco add-on, which adds local voice control to the Mozilla WebThings Gateway, has it built in.
https://github.com/createcandle/voco/tree/master/snips
Mozilla Gateway Addons are not allowed to install anything onto the system, so this add-on is fully portable. That's why the library files are in the same directory. From memory, I believe the following files there are needed:
nanotts
libttspico.so.0
libttspico.so.0.0.0

And here you can see a shell script that calls it: https://github.com/createcandle/voco/tree/master/snips

Issue moved: https://github.com/rhasspy/rhasspy-tts-cli-hermes/issues/1


[BUG] Multiple <env>.bicepparam creates multiple issues for findings in main.bicep

Existing rule
Azure.AppService.WebSecureFtp
Description of the issue
This happens with multiple rules, but let's take Azure.AppService.WebSecureFtp as an example.
We have a main.bicep that creates a br/public:avm/res/web/site:0.3.5 with siteConfig: {} that takes no parameters, so bicepparam should not affect this. In other works: All siteConfig: {} settings and values are defined inside main.bicep.
We then have multiple <env>.bicepparam, let's say test.bicepparam and prod.bicepparam.
Events:

PSRule code scanning with https://github.com/marketplace/actions/psrule created one issue per <env>.bicepparam.
We add siteConfig: {ftpsState: 'FtpsOnly'} to main.bicep and trigger a new PSRule scan.
Only the issue tied to prod.bicepparam gets closed.

Error messages
None
Reproduction
See description.
Version of PSRule
2.9.0
Version of PSRule for Azure
1.36.0
Additional context
No response

I can only confidently confirm 2) for now. We changed the inputPath input parameter to point to the prod bicepparam file, instead of providing no value, and thus scanning the whole repo. It seems to behave as expected now.
Would like to not get spammed with three code scanning alerts for dev, stage and prod bicepparam using the same main.bicep in the future, looking forward to it. 😊


[SPARK-38234] [SQL] [SS] Added structured streaming monitoring APIs.

What changes were proposed in this pull request?

Add two monitoring REST API endpoints under SQL that provides details about the structured streaming queries. Even though a store exists for it and the data is presented in the UI under the "Structured Streaming" tab, this data is not exposed as a REST API.
Summary API
Returns the summary of all existing streaming queries.
GET /{appId}/sql/streamingqueries
Response is list of StreamingQueryData.
Progress API
Returns the progress events of a specific streaming query by runId. User can also specify how many of the most recent events needs to be retrieved by using the last query parameter. By default, we return the most recent progress event i.e. last is set to 1.
GET /{appId}/sql/streamingqueries/{runId}/progress?last={N}
Response is list of StreamingQueryProgress.
Note: We are not introducing new object definitions for the response since we are just returning the data from the store without aggregation, these are existing event structures - StreamingQueryData and StreamingQueryProgress.
Why are the changes needed?

This data can be used for monitoring, detecting streaming and to build custom dashboards. This monitoring API will be similar to the monitoring APIs that are present for DStreams - refer SPARK-18470.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Added 4 tests under SqlResourceWithActualMetricsSuite to test the endpoints when streaming code is executed. Tested various cases, parameters, boundary conditions

Added 2 tests under SqlResourceInvalidEndpointSuite.scala to test the endpoints when no streaming code is running.

As mentioned earlier, since we are just returning the objects from the store, there is no need to test them individually.

cc: @HeartSaVioR @kuwii

Can one of the admins verify this patch?

@dongjoon-hyun thank you! I will make a PR against 3.3.

Thank you, @yeskarthik !


Date become selected while tap on date not scrolling the date

Feature Request: Is that possible to date will become selected on tap of the day not while scrolling the date
Actual Behavior### Date is become selected while scrolling

Android Version:7.0
Horizontal-Calendar Version:1.7.7

I also need this feature. Can Anyone help?


Unable to switch to dark theme automatically

I am currently on macOS 10.15.2 and Atom 1.42.0. It used to work perfectly but at some point, it failed to switch to dark theme automatically although switching to light theme works fine. I'm still able to manually toggle between them. Could this have been a compatibility issue?

Thank you for filing the issue. I've only experienced this once myself, and I haven't been able to reproduce it reliably. Do you have any additional information that could assist in this?


Invalid chars when running samples

When running advanced_video_scripts/blurred_background.sh, with the 3 sample videos linked to from the README, all downloaded at 1280x720 resolution, the script fails (on Ubuntu w/ ffmpeg 2.8.15 and on Mac as well).
[trim @ 0x229cd00] [Eval @ 0x7ffd128a5220] Invalid chars 'q-0.0sizeN/Atime00' at the end of expression '340.0q-0.0sizeN/Atime00'
[trim @ 0x229cd00] Unable to parse option value "340.0q-0.0sizeN/Atime00"
7775q-0.0sizeN/Atime00" as durationoption value "01.13bitrateN/A
12179q-0.0sizeN/Atime00' @ 0x7ffd128a5220] Invalid chars 'bitrateN/A12179q-0.0sizeN/Atime00' at the end of expression '02.56bitrateN/A
12179q-0.0sizeN/Atime00"e to parse option value "02.56bitrateN/A
16581q-0.0sizeN/Atime00' @ 0x7ffd128a5220] Invalid chars 'bitrateN/A16581q-0.0sizeN/Atime00' at the end of expression '04.03bitrateN/A
16581q-0.0sizeN/Atime00"e to parse option value "04.03bitrateN/A
20781q-0.0sizeN/Atime00' @ 0x7ffd128a5220] Invalid chars 'bitrateN/A20781q-0.0sizeN/Atime00' at the end of expression '05.50bitrateN/A
20781q-0.0sizeN/Atime00"e to parse option value "05.50bitrateN/A
25082q-0.0sizeN/Atime00' @ 0x7ffd128a5220] Invalid chars 'bitrateN/A25082q-0.0sizeN/Atime00' at the end of expression '06.90bitrateN/A
25082q-0.0sizeN/Atime00"e to parse option value "06.90bitrateN/A
636-30'0.0sizeN/Atime00:00:20.00bitrateN/Ae near '00:08.33bitrateN/A
636-30'0.0sizeN/Atime00:00:20.00bitrateN/Azing filter 'trim' with args 'start_frame=0:end_frame=340.0q-0.0sizeN/Atime00:00:01.13bitrateN/A
Error initializing complex filters.
Invalid argument
Slideshow creation failed

It looks like an incompatibility issue with ffmpeg 2.8.x. Let met make a few tests to understand the cause and fix it.

When running advanced_video_scripts/blurred_background.sh, with the 3 sample videos linked to from the README

Added all three video files inside the media folder. They were already there from the beginning but a .gitignore rule prevented them from being pushed, sorry for that.

The README says 'ffmpeg 2.8.x or later' - which ffmpeg version have you tested with if not 2.8.x?

My initial guess was wrong. This is not an ffmpeg issue at all. This is a bash issue. I managed to reproduce it locally. I identified the reason and I'm working on the fix. I'll post an update as soon as possible.

I fixed the issue on my local environment and posted the new version. Can you test it please?

Tested, verified - I was able to run the samples.

Thanks 👍


Use git url instead of https

This way private repositories can be accessed as well.
Fixes #15

Thanks!


database_deletion_policy and user_deletion_policy not expexted here"

TL;DR
We used Google Provider 4.49 and SQL module 13.0.1 but the variables database_deletion_policy and user_deletion_policy were not expected as part of the module during the terraform plan.
The provider set the default value to "DELETE" but we would like to use "ABANDON" for our PostgreSQL as recommended.
Maybe you can set the default value to "ABANDON" in the postgresql module instead of using "null".
Expected behavior
Using desired value "ABANDON" or "DELETE" but it is not an input as far as we see.
Observed behavior
The module always used "DELETE" as value for the "database_deletion_policy" and we could not set it to "ABANDON".
Terraform Configuration
module "postgresql" {
  source  = "GoogleCloudPlatform/sql-db/google//modules/postgresql"
  version = "~> 13.0"

  project_id        = module.project_infra.project_id
  name              = "instance-name"
  availability_type = var.database_availability_type
  backup_configuration = {
    enabled                        = true
    start_time                     = "01:00"
    location                       = local.default_region
    point_in_time_recovery_enabled = true
    transaction_log_retention_days = "7"
    retained_backups               = 7
    retention_unit                 = "COUNT"
  }
  database_version         = "POSTGRES_14"
  create_timeout           = "30m"
  update_timeout           = "30m"
  delete_timeout           = "30m"
  deletion_protection      = true
  disk_size                = 10
  disk_type                = "pd_ssd"
  disk_autoresize          = true
  disk_autoresize_limit    = 100
  database_deletion_policy = "ABANDON"
  user_deletion_policy     = "ABANDON"
  ip_configuration = {
    authorized_networks = []
    ipv4_enabled        = false # No public IP
    private_network     = module.shared_vpc.network_self_link
    require_ssl         = false
    allocated_ip_range  = "google-managed-services-${module.shared_vpc.network_name}"
  }
  region                          = local.default_region
  zone                            = "europe-west3-a"
  tier                            = "db-custom-2-3840"
  maintenance_window_day          = 7
  maintenance_window_hour         = 5
  maintenance_window_update_track = "stable"
  enable_default_db               = false # We do not want a "default database"
  enable_default_user             = false # We do not want a "default user"
  encryption_key_name             = "customer-managed-encryption-key"
  additional_databases = [
    {
      name      = "db1"
      charset   = "UTF8"
      collation = "en_US.UTF8"
    },
    {
      name      = "db2"
      charset   = "UTF8"
      collation = "en_US.UTF8"
    }
  ]
  additional_users = [
    {
      name     = "user1"
      password = "password1"
    },
    {
      name     = "user2"
      password = "password2"
    }
  ]
  user_labels = {
    client = local.client
  }
}

Terraform Version
Terraform v1.3.7

Additional information

provider registry.terraform.io/gavinbunney/kubectl v1.14.0
provider registry.terraform.io/hashicorp/google v4.49.0
provider registry.terraform.io/hashicorp/google-beta v4.49.0
provider registry.terraform.io/hashicorp/helm v2.8.0
provider registry.terraform.io/hashicorp/kubernetes v2.16.1
provider registry.terraform.io/hashicorp/local v2.3.0
provider registry.terraform.io/hashicorp/null v3.1.1
provider registry.terraform.io/hashicorp/random v3.4.3
provider registry.terraform.io/hashicorp/time v0.9.1

Same here


no tty for cron (fixes #11)

fixes #11
reference: https://github.com/dokku/dokku/commit/3e100e30023922ce23d8023128887720ca01adce

Many thanks for the PR!


.gitmodules: Change pybind11 URL from relative to absolute

All the other URLs are absolute, so logically this one should match. Additionally, the relative URL will break if someone's main remote is not on GitHub.

If you need to contact me about this, the best way to do so is in the Pi Squared slack, or at<EMAIL_ADDRESS>


Subject next method failed if it's called without parentheses

Describe the bug
When next method of a subject was called without parentheses this error happens :
Subject.ts:62 Uncaught TypeError: _this._throwIfClosed is not a function
Expected behavior
Both syntaxes are supposed to have the same behavior. I don't understand why when I call the next method with a simplified syntax, it crashes.
Reproduction code
// crash
    readonly loggedIn = new Subject<boolean>()

    constructor() {
        this.myUseCase.myObservableNextABooleen().subscribe({
            next: this.loggedIn.next, // <- crash here
            error: this.handleError, // <- works when error happens
        })
    }

    // works
    readonly loggedIn = new Subject<boolean>()

    constructor() {
        this.myUseCase.myObservableNextABooleen().subscribe({
            next: (loggedIn: boolean) => this.loggedIn.next(loggedIn),
            error: this.handleError,
        })
    }

Reproduction URL
No response
Version
7.5.5
Environment
Browser
Additional context
No response

Those 2 are techinically different. Since subject is class, when you pass fn as param it loses context to this belongs to class instance. Using it via arrow fn is correct, or you may manually bind context via bind, or call .https://developer.mozilla.org/en-US/docs/web/javascript/reference/global_objects/function/bind

Oh, I finally understand why it crashes! It makes so much sense now. Thank you a lot for this reminder.


Docs: pass parameters to redirect URIs

Fixes #685

Current coverage is 94.71% (diff: 100%)

Merging #702 into master will not change coverage

@@             master       #702   diff @@
==========================================
  Files            20         20
  Lines          1382       1382
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1309       1309
  Misses           73         73
  Partials          0          0

Powered by Codecov. Last update 491a580...63925ee

@jmdobry Mind taking a look?


More const and some VS (Windows) build artifacts to ignore

Adding Visual Studio build artifacts into the ignore file and adding in a few more const so Visual Studio can build these files. 2nd PR will have the Visual Studio (Windows) build changes. No source file changes needed after adding these const keywords in.

This is the Solution/Project file (IDE) pull-request for Visual Studio 2022 v17.9 Community Edition. This should work with CMake or any other Windows build tool like MSBuild. To run in Visual Studio - open the solution file in llm.c\platform\windows\llm_c.sln.
Requirements/Assumptions:
 •	You have the latest VS Studio and Cuda SDK (this is optional but highly recommended).
 •	You have followed Andrej’s instructions and created the data files

Data files should be located as described below:
 •	llm.c\platform\windows\ data\*
 •	llm.c\platform\windows\gpt2_124M.bin
 •	llm.c\platform\windows\gpt2_124M_debug_state.bin

Windows porting goal was no source file changes needed (we did end up needing some const keywords added).

This looks like a really extended PR... I am certainly interested in the tiniest, most minimal thing that makes this compile on Windows.

Sorry - the smallest would be - just the const changes and the unitstd.h file which should be located in platform/windows (new directory). I don't really care where it's located as long as it's somewhere in the repo. I will create a separate PR for just these changes.

re: /openmp on Windows
@ross-wheeler unfortunately Microsoft's compiler is pretty bad at this and we need to do some cosmetic changes.
e.g.
void matmul_forward(float* out,
                    float* inp, float* weight, float* bias,
                    int B, int T, int C, int OC) {
    #pragma omp parallel for collapse(2)
    for (int b = 0; b < B; b++) {
        for (int t = 0; t < T; t++) {
          .....
            }
        }
    }
}

with /openmp on Windows
cl.exe /openmp -DOMP /I. /I .\dev\win train_gpt2.c
train_gpt2.c
train_gpt2.c(165): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(166): error C3015: initialization in OpenMP 'for' statement has improper form
train_gpt2.c(193): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(242): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
train_gpt2.c(405): warning C4849: OpenMP 'collapse' clause ignored in 'parallel for' directive
Microsoft's compiler wants the loop to look like this:
void matmul_forward(float* out, float* inp, float* weight, float* bias, int B, int T, int C, int OC) {
    int b;
    #pragma omp parallel for collapse(2)
    for (b = 0; b < B; b++) {
        for (int t = 0; t < T; t++) {
            ....
            }
        }
    }
}

It's really annoying.

@azret - please use the Solution/Project files while in this repo https://github.com/ross-wheeler/llm.c/tree/master. OpenMP build has been working since last week. Please use https://github.com/karpathy/llm.c/pull/134 if you want to use the Makefile changes from the command line like you are doing above. You do NEED the const fixes (as far as I can tell). I have an issue outstanding with the VS compiler team on this but I'm fairly sure it's their issue/problem. I have a work-around implemented in both repo's https://github.com/ross-wheeler/llm.c/tree/master and https://github.com/rosslwheeler/llm.c. Thanks for giving it a try!

Idea: We create a doc/windows.md where we document how to build on Windows, but it's just instructions (and maybe some code snippets) in Markdown. Because it looks very involved right now. And then we link to this from somewhere in the main README so people can CTRL+F for it easily.

Let's see if we can resolve https://github.com/karpathy/llm.c/pull/134? Then if there are issues, we can go this path...

Portions of this are now in and have newer PR's.


Area selection rendering is problematic

I noticed that the area selection is rendering funny on my computer.
I am running Ubuntu 14.10.

That's scrot's fault. Try using an alternative such as Roger/escrotum.
Then include this in your config:
screenshot_select_command="escrotum -s %img"
screenshot_window_command="escrotum %img"

I will look into this in a moment. Thank you.

OH: I installed escrotum and everything works much better. I highly recommend it to anyone reading this.

I just saw https://github.com/jomo/imgur-screenshot/issues/75, and since it's related I want to add my thoughts.
I switched from scrot to escrotum, but after that I switched to https://github.com/naelstrof/maim which uses https://github.com/naelstrof/slop, but that's just my personal preference. Choose which one suits you best.

How does one can edit a config?

Tried imgur-screenshot -i screenshot_window_command="maim -i$(xdotool getactivewindow) %img" but it just takes a selection.

https://github.com/jomo/imgur-screenshot/wiki/Config


Add display:block on .imageset to readme

.imageset can have display: block assigned to get rid of bottom whitespace, but this is not in the readme yet.

Done in cb36c2c4718f7ad5bb60893e038c91bff847d1b6


editorconfig missing comma

Hello,
Could you tell if the missing comma after .jshintnoderc was intended this way, or I'm overlooking something somewhere?
[{.jscsrc,.jshintrc,.jshintnoderc*.json,*.yml}]

https://github.com/cedaro/cedaro-wp-theme-config/blob/develop/.editorconfig#L16

Nope, that wasn't intended. Thanks for the heads up!

Thanks :+1:


feat(math): more trigonometry

Secant, Cosecant, Cotangent

That's a great idea and this reminds me to add some other related fundamental functions, e.g. a law of cosines and fast sin/cos approximations

thanks @nkint !


Legal FormField

Links to data sharing policy (or similar) and requires use acceptance to proceed. Can only include a single element in a form.

Additional requirements

[ ] Enable upload of terms document
[ ] Link responses to particular terms document, possibly with a version number
[ ] Track clicking on terms, and optionally require it
[ ] Add option to require an "e-signature", though this may not respond to any particular legal requirement so we may remove it.

Working on in #305


Fix OS version checks on .NET Framework

This PR addresses #192 by fetching the real OS version information from HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion when running on target platforms before .NET 5.0.

LGTM


Message in console: "ufs: cannot remove file "null" (Token is not valid [invalid-token])"

Hi @jalik,
Please help to clarify a trouble.
I've updated UFS and ufs-gridfs to<EMAIL_ADDRESS>and<EMAIL_ADDRESS>today. Before that I reset the app by meteor reset. When then I uploaded new file(s), they were added but in the console I saw the following message for each file:
ufs: cannot remove file "null" (Token is not valid [invalid-token]) --- ufs-uploader.js:157
The onError handler
onError: function (err) {
  console.error(err);
},

in the uploader  let uploader = new UploadFS.Uploader... also printed in Chrome's console:
errorClass {error: "invalid-token", reason: "Token is not valid", details: undefined, message: "Token is not valid [invalid-token]", errorType: "Meteor.Error"}
or in Firefox:
Object { stack: "Meteor.makeErrorType/errorClass@htt…", error: "invalid-token", reason: "Token is not valid", details: undefined, message: "Token is not valid [invalid-token]", errorType: "Meteor.Error" }
What is wrong? Can't understand.
Best regards,
seabrus

I was experiencing the same issue, this is how it started. Using node-inspector, I was able to walk through and determined it was related to #82
I had audit-argument-checks installed. I removed it from my project, and everything is back to normal. Doesn't look like it will be fixed.  Maybe cloning, making changes, and perhaps submitting a PR.

Hi @agwm2013,
Thanks a lot for the information! Yes, I have audit-argument-checks installed too. I'll try to remove it.

There is no problem for this issue, I've installed the audit-argument-checks package and tested it.

The error is thrown by the package in the ufsComplete method after the file has been uploaded successfully
This causes the uploader (client side) to abort the download because of the error (which in fact is useless since it's not an error thrown by UploadFS)
When aborting the download, it tries to remove the file because aborting means an incomplete file so we don't want to keep incomplete files in stores, but since the upload token was removed when the file was fully uploaded the token does not exist anymore, so the error you see is because of that cleaning process that should not be called because the file IS downloaded successfully.

I will add these check() but I feel like forced to do it, in fact it won't add more security, just a double check, but if that can removes warnings and errors, let' do it.

Hi @jalik,
I will add these check() but I feel like forced to do it, in fact it won't add more security, just a double check, but if that can removes warnings and errors, let' do it - agree with you.
Frankly, often I do the same in my code: use (sometimes quite formally) check() only for the compliance with the audit-argument-checks package, and then add more detailed and understandable checks of the parameters, for example, using Match(...) or other tests. Yes, such an approach leads to a double check, but audit-argument-checks is quite useful, as well as it's recommended by MDG. Unfortunately, audit-argument-checks is restricted to the check() tests only, it is not convenient. Maybe MDG modify it in the future and make it more flexible.
However, thanks for your efforts to make UFS the best package for meteor file uploading.
Have a nice day!


can u make a version support vs2013 ide

can u make a version support vs2013 ide. or make .pro build successful(copy libevent src in project)
.pro build error in windows qcreator:
include <event2/event.h>
no such file

I'd love to but I don't have a Windows box :-(
You can try to add
INCLUDEPATH += /path/to/event2/directory

to the .pro file…


Bug fix: REST invoker can not call the provider from diff app

Coverage increased (+0.1%) to 62.707% when pulling ff8a32164b672fb599c37c78802454beef49ecbf on little-cui:master into 1e3a4af9e300acbacdced36ce7dce346e28a170b on ServiceComb:master.


Should CiceroMark allow comments?

Comments can appear in templates and affect neither the contract text, nor the parsing.
Here is a proposal inspired by https://handlebars-draft.knappi.org/guide/#template-comments
{{!-- This is a comment --}}

This issue is inherited from #392


Unused function can be removed

https://github.com/programmablewealth/gotchiverse-crowdfarming-contracts/blob/6487d7dca7cb93cd1448b315369956f2fff5203a/contracts/libraries/LibAppStorage.sol#L42

removed👍


yarn fails to build zeroize

Using BLOCKSTACK_CORE_SOURCE_BRANCH="feature/genesis" yarn the process errors with
error[E0658]: use of unstable library feature 'alloc': this library is unlikely to be stabilized in its current form or name (see issue #27783)
   --> /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/zeroize-1.1.0/src/lib.rs:209:1
    |
209 | extern crate alloc;
    | ^^^^^^^^^^^^^^^^^^^
    |
    = help: add #![feature(alloc)] to the crate attributes to enable

cc @lgalabru - any thoughts here? This should probably be an issue on blockstack/stacks-blockchain.
@friedger can you try with just the master branch?

Same with master branch.

@friedger what version of rustc are you using?
rustup show

rustc 1.42.0 (b8cedc004 2020-03-09)
[image: --]
Friedger Müffke
[image: http://]friedger.de
https://friedger.de
On Mon, Apr 6, 2020 at 11:00 AM Matthew Little<EMAIL_ADDRESS>wrote:

@friedger https://github.com/friedger what version of rustc are you
using?
rustup show
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/blockstack/clarity-js-sdk/issues/58#issuecomment-609666943,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALBYWOVJJTCD7J7AB7CBYTRLGK4FANCNFSM4L2VGTBQ
.


Style issue with the slider in Angular2 after View is rendered

Hi
Just wondering if there's a way to implement this in angular2? I somewhat manage to make the slider work however the view is rendered the styling of the slider is kind of broken. However when you route to another page and go back to the page where the slider is, the style is fixed.
Like this, the CSS are all the same as the demo.

Not sure if anyone had this issue or tried it on angular2.
Found this stackoverflow question regarding jssor in Angular2 but not sure if they're getting the same problem as the link they;ve provided are not working anymore.
jssor slider SO
I wrapped the jssor options, scale function in jssor_1_slider_init()
        var jssor_1_options = {
            $AutoPlay: 0,
            $DragOrientation: 2,
            $PlayOrientation: 2,
            $ArrowNavigatorOptions: {
                $Class: $JssorArrowNavigator$
            }
        };
....
    }

In my component it's something like this
import { Component, OnInit, AfterViewInit } from '@angular/core';
declare var jssor_1_slider_init: any;
export class SomeComponent implements AfterViewInit {
 ngAfterViewInit() {
    jssor_1_slider_init();
  }
}

Thanks

Better leave an url of your page that I can check it out.


camera intrinsics

Thanks for your awesome works! I am currently engaged in transforming the depth image into a 3D representation. However, I have not been able to locate the camera intrinsics in your dataset, HoNY. Could you provide them?

Hi, have you solved the problem? Where are the camera intrinsics?

Dear Haoyi Zhu,
Sorry for my late reply. I have got the camera intrinsics from the author
of dobb-e. But now, I have deleted it on my computer. So I think you can
send your request directly to that guy.
Have a nice day,
Leiyao Cui.
On Tue, May 28, 2024 at 9:29 PM Haoyi Zhu @.***> wrote:

Hi, have you solved the problem? Where are the camera intrinsics?
—
Reply to this email directly, view it on GitHub
https://github.com/notmahi/dobb-e/issues/2#issuecomment-2135223420, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AFSX4WV22364RY2XABDRV5LZESBEZAVCNFSM6AAAAABAV7YO7GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZVGIZDGNBSGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Automated cherry pick of #9036: feat(region): search server by substr of ip or mac

Cherry pick of #9036 on release/3.4.
#9036: feat(region): search server by substr of ip or mac

/lgtm
/approve


Use CTTI instead of RTTI where possible

In some projects we are required to use -fno-rtti option, which disables RTTI (run-time type information). cereal uses typeid operator, which requires RTTI, in the following two scenarios:

using typeid on a type which is known at compile-time (usually taking hash_code() of the returned std::type_info);
using typeid in order to support serialization of polymorphic types (i.e. here types are not known at compile time), see polymorphic.hpp.

While usage in the scenario 2 is required to serialize polymorphic types, we don't need RTTI at all in scenario 1.
This PR replaced all usages of typeid having a compile-time known type with methods from an external library ctti (open source, MIT license) - this library provides ctti::type_id<T>() functions for compile-time type id, and also ctti::type_id_t type, similar to std::type_info.
With this change one can use cereal with -fno-rtti flag, in case when one does not serialize any polymorphic types (i.e. does not include polymorphic.hpp or other related headers).

Converted to draft because there is at least another instance of using typeid with compile-time known type: see util.hpp


C++ indexer emits nodes without /kythe/node/kind facts [phab:T26]

Created by schroederc at 2015-02-17 19:02:32:
The following command displays no output because both kythe:?lang=c%2B%2B#bool%23builtin and kythe:?lang=c%2B%2B#int%23builtin are nodes without any facts (including /kythe/node/kind).
./campfire run //kythe/go/serving/tools:kythe node kythe:?lang=c%2B%2B#bool%23builtin kythe:?lang=c%2B%2B#int%23builtin

However, these nodes do have edges:
./campfire run //kythe/go/serving/tools:kythe edges --count_only kythe:?lang=c%2B%2B#bool%23builtin
./campfire run //kythe/go/serving/tools:kythe edges --count_only kythe:?lang=c%2B%2B#int%23builtin

zarko wrote at 2015-02-18 01:43:04:
This is WAI; we should solve it by either prepopulating the database or by offering some way to ask the indexer to emit facts about builtins.

schroederc wrote at 2015-02-18 02:46:54:
Why can't the indexer just emit '/kythe/node/kind' == 'tbuiltin' facts for each of those nodes?

revision #1145


exact key search above rownum

When a user tries to search for a key with an "in" filter operand, the result doesn't get listed if the rownum is exceeded.

@stoch is this a valid issue ? can it be closed if will not be implemented ?

closing as this works as expected, if filtering is done on the client (fsm side) re executing the query with appropriated where clause brings the data as expected.


Update: Hints

Codecov Report

Merging #319 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #319   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files         123    123
  Lines        1277   1277
=====================================
  Hits         1277   1277

Continue to review full report at Codecov.

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


Add system independent properties for line and path separator

This patch fixes a couple of tests that were failing in a Window environment.  It replaces the hardcoded path and line separator characters with system properties.

Awesome! TYVM @opeongo
I need to get a go to windows environment configured. There are a few open windows issues  that also need some TLC.

I also saw some unicode issues when last I tried to build on windows, hoping this patch beanshell/beanshell@5a9a685b61081b6a125bf24e7598ea0388b24be5 resolves that too.

I had a look at the windows issues.  Most of them seem to be non-bugs, con't reproduce, or issues that have already bee fixed but not closed out.  Would you like me to go through and categorize them?

As for the line endings, we already carry System.getProperty("line.separator") as bsh.Interpreter.Console.systemLineSeparator
I'm wondering if we shouldn't fix this somewhere else, in other words not in the tests. For example if I had a script using "\n" as these tests do, and I run it under windows should we not let the script succeed with the correct line endings for windows?
What do you think?

Well, this is a bit curious.  I went to have a look at where the  line endings were coming from.  It seems that the parser is just making the string from the characters it reads from the script file.  I assume the original file just has , so where are the extra coming from?
After poking around it appears that git performed automatic line ending conversion when I cloned your repository.  In other words my test script automatically got changed to be subtly different from your test script, and this broke the test.
Knowing this, I don't think that my suggested patch is correct, because it depends on 'git' changing the test script.  Git (and other tools like it) are the culprit here.  What if the repository was checked out without using autocrlf?  I suspect that this will be an issue that other people will stumble over.
I guess one option would be to change the Parser so that it converts line endings to system independent form when it reads the string in (e.g. for Windows change  to , for classic MacOS change  to ).  This could be implemented with a
input.replace(System.getProperty("line.separator"),"\n");

inside of BSHLiteral#stringSetup.  But I don't think this is correct either.
I don't think that there is a perfect solution to this problem, because this kind of automagic would thwart people from deliberately forming the line endings that they wanted, and even so there would likely be odd situations where scripts would break mysteriously.
My preference would be to not modify the input at all, but caveat emptor that the line endings in your script file will be the ones that are copied verbatim in to the long string.  This is not a bug, and fixing it would cause more confusion.
As far as the test failing, the test could be changed to test alternate line endings and succeed if either one passed.

Hmmm I don't think I quite follow with the git autocrlf and how that relates to this patch.
// Long strings
- out="\n";
+ out= System.getProperty("line.separator");
 for (x : 5)
-   out += x + "\n";
+   out += x + System.getProperty("line.separator");

Surely git had nothing to do with this... well I hope at least.

Just stumbled on System.lineSeparator() since 1.7...
now I know that too tyvm.

Here I will explain the reasoning why git caused the test to fail on my system, and what could be done about it.
The problem
This is a long standing and well known issue for cross development between unix-like and windows.
I assume that you prepared the strings.bsh test script on a unix system.  The line endings in the file suggest so.  If on your unix system you open the file in a binary/hex editor (not a text editor) you will see that the byte codes for this section are
aout = """
0
1
2
3
4
5
""";

are composed of the byte sequence
61 6F 75 74 20 3D 20 22 22 22 0A 30 0A 31 0A 32 0A 33 0A 34 0A 35 0A 22 22 22 3B 0A

I am working on a Windows system.  When I clone the repository I get the same packed deltas as you, but when I check out a branch to a working copy git by default will translate line endings in text files from 'canonical' format (\n) to Windows format (\r\n).  Now on my Windows/git/autocrlf system that same section of code will have the bytes:
61 6F 75 74 20 3D 20 22 22 22 0D 0A 30 0D 0A 31 0D 0A 32 0D 0A 33 0D 0A 34 0D 0A 35 0D 0A 22 22 22 3B 0D 0A

Note that this sequence is 8 bytes longer, because there are 8 lines and an \r code (0d) has been added to each line ending.
The reason for doing this is because some (most?) text editors on Windows will insert \r\n line endings for new lines, and when save the file you will either get all lines endings with \r\n, or perhaps just the lines that you inserted with \r\n and the rest unchanged (the behaviour depends on the editor, there is no standard).  This is the hellish legacy of Bill Gates.  Git by default does the automatic conversion from canonical to system dependent on checkout and system dependent to canonical on checkin because usually it is the right thing to do.  If it didn't then there might be a lot of thrashing between line ending styles each time the file is modified on a different system type, and this would make the change log unusable.
Getting back to the test failure, when checked out from git/autocrlf on a Windows system the test script will actually be different than the test script on a unix-like system:  they will have different byte sequences and have different lengths.
When the parser tokenizes a long string it will read all bytes in between the opening and closing """.  For the test in question a unix-like system would see 13 bytes, for a Windows/git/autocrlf system 20 bytes.  When the test constructs the comparison string it is manually using a single \n for a line ending, and making a sequence that is 13 bytes long.  This compares exactly to the sequence read from the unix-like system, but not to the Windows/git/autocrlf sequence.
So, what to do about this?
  I am not sure.  The long string is capturing everything in between the delimiters.  Perhaps the system dependent line endings are important for a particular use case: who can say for sure.  What if the long string had explicit line endings:
aout="""
this\\r\\ncould\\r\\n\\happen\\r\\n
""";

If we wanted to allow people full control to make whatever long strings they wanted then we should just leave it as is.  If this is the case then change the test so that the error goes away.
Sanitize the input
The other option is to sanitize the input, just like git-autocrlf:  when the parser tokenizes a long string convert all of the \r\n sequences to \n.  Sounds easy.  This auto-magic would make the test failure go away.
But should this be done on Windows only where these mixups can occur, or should it be done on automatically on all platforms?  And what about cases where different line ending sequences are required parts of the application data?  Automatically sanitize the inputs may end up doing the wrong thing in some cases.
A third way
Actually I can think of a third option:

keep the current long string behaviour.  The """ long string delimiter will copy all bytes unchanged.
add another long string type, using ''' as a delimiter.  This type will convert line endings to canonical form.

Nah... that is getting too complicated for too little benefit.
After thinking about it my vote would go for the second option:  automatically sanitize the inputs by changing all \r\n to \n.  What is the common case for long strings anyway?  Surely not to do exotic things with control characters.  If there was a use case to embed different line endings this could be done using a more traditional means (StringBuilder) or with String.replace().

I vote for option 1 "Ignore this feature, it is not a bug" the problem comes from changing the lite endings which is not something that should ever occur under normal circumstances. If anyone managed to change the line endings they should know how to change it back too.
The other issue which should be considered is what if you intended to have \r\n and we now go and remove it. Whomever caused the problem is the one responsible to fix it.

I dug around a bit more and I found JEP 327 "Raw String Literals".
It looks like this JEP moving forward and will soon be in Java.  This clarifies how the official Java will handle raw string literals.  Here are a couple of examples of how this will work
aout = `
0
1
2
3
5
`;

a = `this is a "raw string
literal".  note that \ characters
get no special treatment`;

b = ``here is how to embed a ` single backtick``;

c = ```here is how to embed a `` double backtick```;

tl;dr

no escape processing within raw literals:  '\n' is interpreted to the two characters '\' and 'n'
automatic line ending sanitization: line endings with <cr><lf> are converted to <lf>
the delimiter is a backtick (actually a matching variable number of backticks)

If there are no backticks within the literal then a single backticks could be used as delimiters.
If sets of single backticks are used within the literal then double backticks can be used as delimiters
etc

Note that they have taken a stand to normalize line endings.  This effectively solves the issue that we have been discussing here.
This JEP also touches on the issues that we have been discussing in #495.
I think a review of BeanShell handles string and character literals is required in light of this JEP.

Agreed... investigate JEP 327


PARQUET-635: Use conda-forge toolchain, statically link libstdc++ on Linux

This will avoid portability issues across different Linux environments.

+1, LGTM


Add robux to the login object & Add a delete item command

I would like to know when my robux changed so I can buy and delete an item and over and over again.

Well for your second problem you can't delete an item. I don't think there is an API endpoint that does support that feature. Could you explain what you mean by the login object?

The login object is what is returned after you do rbx.login(res => {
Res is the login object.

If that is what you're referring to then I don't see the point of having ROBUX in the login object. I want to return a ClientUser with v5 that does show a whole object containing information like that.

Ok.

When you want a function to check amount of Robux using as example:
roblox.getRobux
please reply.


GifDecoder Library now uses AnimatedGIF

Hi Larry,
After getting a handle on the AnimatedGIF API and the needs of myself and people that use my projects, I decided to turn GifDecoder into mostly a wrapper class that simplifies the interface to AnimatedGIF.  It hides the complicated - at least for a beginner - disposal code inside the class, provides a familiar API for people that used the AnimatedGIFs sketch or GifDecoder in the past, and adds some automatic tracking of stats that AnimatedGIF currently doesn't do unless you parse the whole GIF separate from displaying it.  I made sure to credit you and point people to your library for donations and sponsorships in the README.
https://github.com/pixelmatix/GifDecoder

Looks good, thank you for doing that. I was thinking to allow a new mode for setups that have enough RAM to hold the decoded image.  For LED matrices, it would be ideal since the maximum image size easily fits in RAM.