Tests with syntax errors may silently pass

We've gotten a report that tests with ES6 arrow functions in IE11 silently pass, and they're not even counted as tests.

This is also happening if there is an error thrown in a setup(). These failures are not reported by WCT.

Cool, that should make it easy to test for


Timesheet - 18/04/2021 - Mr. K

Task 1
Landing page - HiFi Design
Worked on responsive design. 1366, 992, 768, and 414. Resized all elements, modals, buttons, and layouts. Made the design pixel perfect with an 8px grid.
Proof of Work

https://www.figma.com/file/JCGeIX9fFrrpZiqE0XzH9s/Homepage?node-id=3%3A0

Time Spent
5.5 hours
Account Number
c93ee24b89991a1438b45444cf06e5bb167ed1796d54a800174def4480f98ed0

Approved

Approved


Upgrade python-anthemav to v1.1.9

Description:
Version bump of underlying python-anthemav library to 1.1.9.  This fixes a syntax error in Python 3.7 where async has become a reserved word.
Checklist:

[X] The code change is tested and works locally.
[X] Local tests pass with tox. Your PR cannot be merged unless tests pass
[X] There is no commented out code in this PR.

This did not seem to resolve the issue in 89.0, still getting the asyncio error.
Should I open a new issue  here or with the underlying anthemav  module.
Log Details (ERROR) Thu Mar 07 2019 18:31:39 GMT-0500 (Eastern Standard Time) Error while setting up platform anthemav Traceback (most recent call last):   File "/usr/src/app/homeassistant/helpers/entity_platform.py", line 128, in _async_setup_platform     SLOW_SETUP_MAX_WAIT, loop=hass.loop)   File "/usr/local/lib/python3.7/asyncio/tasks.py", line 416, in wait_for     return fut.result()   File "/usr/src/app/homeassistant/components/media_player/anthemav.py", line 42, in async_setup_platform     import anthemav   File "/usr/local/lib/python3.7/site-packages/anthemav/__init__.py", line 6, in <module>     from .connection import Connection      # noqa: F401   File "/usr/local/lib/python3.7/site-packages/anthemav/connection.py", line 4, in <module>     from .protocol import AVR   File "/usr/local/lib/python3.7/site-packages/anthemav/protocol.py", line 12     ensure_future = asyncio.async                                 ^ SyntaxError: invalid syntax

Please open new issues.


Export WorkbookContext

Is your feature request related to a problem? Please describe.
I want to create custom functionality for my sheet, but I need WorkbookContext.
Describe the solution you'd like
Would be great to export WorkbookContext from @fortune-sheet/react to use it in any custom component through useContext.

sure, can you please submit a pr for this?


Helm operator deletes (reinstall) releases on kubernetes API errors

There are some conditions that make helm operator reinstall a release. (purge then install)
So from what I could see, is when my api server goes down for a while, helm operator fails a to query and and in certain conditions, try to install a already existing chart, so it will fail because it already exists, then it will purge it and clean install it.
on GKE the api server goes down for a little while on scale up/down or on cluster updates.
Here are the logs related to the prometheus release the only one that got recreated.
ts=2018-11-16T08:05:42.384342336Z caller=release.go:140 component=release info="releaseName= prometheus, action=CREATE, install options: {DryRun:false ReuseName:false}"
E1116 08:05:42.467197       8 reflector.go:205] github.com/weaveworks/flux/integrations/client/informers/externalversions/factory.go:113: Failed to list *v1alpha2.FluxHelmRelease: fluxhelmreleases.helm.integrations.flux.weave.works is forbidden: User "system:serviceaccount:flux:flux" cannot list fluxhelmreleases.helm.integrations.flux.weave.works at the cluster scope
ts=2018-11-16T08:05:45.173671791Z caller=release.go:209 component=release error="Chart release failed: prometheus: &status.statusError{Code:2, Message:\"a release named prometheus already exists.\\nRun: helm ls --all prometheus; to check the status of the release\\nOr run: helm del --purge prometheus; to delete it\", Details:[]*any.Any(nil)}"
ts=2018-11-16T08:05:45.174728777Z caller=release.go:211 component=release info="Deleting failed release: [prometheus]"

There was a lot of errors on other charts and I think it wasn't deleted because of luck, here a log for operator trying to delete but failing.
ts=2018-11-16T08:05:03.655265427Z caller=release.go:140 component=release info="releaseName= nginx-ingress, action=CREATE, install options: {DryRun:false ReuseName:false}"
E1116 08:05:04.043323       8 reflector.go:205] github.com/weaveworks/flux/integrations/client/informers/externalversions/factory.go:113: Failed to list *v1alpha2.FluxHelmRelease: Get https://<IP_ADDRESS>:443/apis/helm.integrations.flux.weave.works/v1alpha2/fluxhelmreleases?limit=500&resourceVersion=0: dial tcp <IP_ADDRESS>:443: i/o timeout
E1116 08:05:35.134228       8 reflector.go:205] github.com/weaveworks/flux/integrations/client/informers/externalversions/factory.go:113: Failed to list *v1alpha2.FluxHelmRelease: Get https://<IP_ADDRESS>:443/apis/helm.integrations.flux.weave.works/v1alpha2/fluxhelmreleases?limit=500&resourceVersion=0: dial tcp <IP_ADDRESS>:443: i/o timeout
ts=2018-11-16T08:05:42.350754932Z caller=release.go:209 component=release error="Chart release failed: nginx-ingress: &status.statusError{Code:2, Message:\"unknown\", Details:[]*any.Any(nil)}"
ts=2018-11-16T08:05:42.350913148Z caller=release.go:211 component=release info="Deleting failed release: [nginx-ingress]"
ts=2018-11-16T08:05:42.377810601Z caller=release.go:214 component=release error="Release deletion error: &status.statusError{Code:2, Message:\"configmaps is forbidden: User \\\"system:serviceaccount:kube-system:tiller\\\" cannot list configmaps in the namespace \\\"kube-system\\\"\", Details:[]*any.Any(nil)}"
ts=2018-11-16T08:05:42.377884033Z caller=chartsync.go:255 component=chartsync warning="Failed to install chart" namespace=default name=nginx-ingress error="rpc error: code = Unknown desc = configmaps is forbidden: User \"system:serviceaccount:kube-system:tiller\" cannot list configmaps in the namespace \"kube-system\""

I don't have time to look at the code now, but it seems if API server is down, then helm-op checks fail and it mess up the cluster thinking it isn't in a good state. This should probably be avoid by (if possible) detecting that API server is down, so sync operations are stopped.
Deleting a chart can cause data loss (with happened to our system)
And at the next chart sync, everything worked as expected.
Flux version: 1.8
Helm-op version: 0.4.0

Also flux logs have this that maybe why it tried to install it.


Copy file contents to clipboard

yazi --debug output
It's a proposal, not a problem

Please describe the problem you're trying to solve
It is often needed to copy a text file contents to the clipboard
Would you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
For a single selection only.
Use the shortcut cc to copy contents.
Additional context
As cc is currently for copying path to clipboard, introduce a (breaking) change:
Make cp shortcut to copy path.
When used over a directory, copy the directory (top level) file list in the clipboard.
Validations

[X] I have searched the existing issues
[X] The latest nightly build of Yazi doesn't already have this feature

Please use a plugin like https://github.com/orhnk/system-clipboard.yazi, which is a wrapper for ClipBoard.
Building this into Yazi would require implementing it for three different systems (Linux, macOS, Windows), which essentially means re-implementing the ClipBoard from scratch. This is not trivial and would bloat Yazi, while also overlapping somewhat with Yazi's built-in yank feature.
If users really need this, using the clipboard plugin might be a better solution since it would be on-demand rather than mandatory.
Closing as no plans to build it in, and a plugin provides this feature already exists.

BTW you can find more awesome plugins at https://yazi-rs.github.io/docs/resources

The plugin mentioned copies the file to the clipboard, so the file is pasted elsewhere.
This suggestion is for copying files contents, similar to cat file.txt | pbcopy.
But, I agree with your explanation for rejection.

If cat file.txt | pbcopy works for you, you can easily achieve this with the shell command:
[[manager.prepend_keymap]]
on  = "<C-y>"
run = '''
    shell 'cat "$0" | pbcopy' --confirm
'''


Load vnode.nodeName only once.

Since idiff is invoked on all kinds of virtual nodes, the property
access vnode.nodeName is usually megamorphic in any realistic workload,
so it's better for the JS engine to load the property only once.
Also apply the String constructor only once to the vnode.nodeName,
because the String constructor can have arbitrary side-effects (since it
can call out to Symbol.toPrimitive, valueOf or toString), and thus
requires magic on the engines' side to optimize this away, which doesn't
always kick in (and is costly).

Coverage remained the same at 100.0% when pulling 467f41f0672c11c80001a8aa7b5417173c9ba15b on bmeurer:Load_VNode_nodeName_only_once into 9001436a453d736267359538e2d822ccf23e3115 on developit:master.

Coverage remained the same at 100.0% when pulling aaf34253697d09f306c637dc19057a360ece1bbd on bmeurer:Load_VNode_nodeName_only_once into c9e331b222a4d848800fd035e529366db2904bb7 on developit:master.


Flow CoreFX packages into NuGet.Client (specifically ProtectedData)

Currently nuget-client depends on newtonsoft-json and common, but we think we need to add a dependency on corefx because Nuget.Configuration depends on System.Security.Cryptography.ProtectedData, from corefx.
This means adding a DotNetPackageVersionPropsPath implementation to NuGet.Client, configuring source-build, and making sure corefx produces the ProtectedData package.
I have changes that I think will work for for the NuGet.Client PackageVersionProps implementation here, but I haven't tried them yet: https://github.com/dotnet/source-build/compare/dev/release/2.1...dagood:nuget-client-pvp-01. (Direct link to NuGet.Client patch.)
@nkolev92 to keep NuGet.Client patchless for 2.1-RC, do you think you can get a change like that into the NuGet.Client release-netcore-2.1-rc branch? (After we see if they work. :smile:)
@dseefeld Can you try the changes with your corefx + cli changes to see if it works?
/cc @crummel

@dagood
I think that's doable.
//cc @rrelyea

It turns out we can't realistically turn on the full CoreFX package build yet, so this isn't happening now. Sorry for the false alarm, @nkolev92, we don't need any changes. It would be useful to have the package version props implementation in NuGet.Client for the future, but we don't have any use for it right now and can't really test it out.
Making the CoreFX build produce this package is tracked by Enable building allConfigurations for corefx in source-build (#210):

[@weshaggard] We need to build all packages not just .NET Core app packages when we are in offline source-build mode. However the allconfigurations leg has never worked correctly on non-windows so we need to iterate on it in order to enable.

For now, we'll put System.Security.Cryptography.ProtectedData in the tarball as a prebuilt package.


Error getting branch if the repo is "grafted"

What's wrong
While developing a CI/CD workflow using Github Actions I noticed that the branch was not picked correctly and versions were mess up. After some debugging it was clear that the plugging was not properly getting the branch: semver.info.branch.name was empty and semver.info returns something completely unrelated to what was configured, though locally works fine.
Further analysis
This is how plugin is fetching the branch name:
    private fun getCurrentBranch(project: Project): String {
        val head = GitCommandRunner.execute(project.projectDir, arrayOf("log", "-n", "1", "--pretty=%d", "HEAD"))
        return """\(HEAD -> (.*?)[,|)]""".toRegex().find(head)!!.groupValues[1]
    }

So if I run git log -n 1 --pretty=%d HEAD in my laptop I get (HEAD -> release-workflows, origin/release-workflows) but in github (grafted, HEAD -> release-workflows, origin/release-workflows) so the Regex does not match. This is happening for the most common checkout scenario
  steps:
     - uses: actions/checkout@v2
     - ....

So as fix suggestions:

Improve Regex expression to match more complex cases
Use other (and simpler) commands as git rev-parse --abbrev-ref HEAD or git branch --show-current which don't require regex expression

Workaround for Github actions
By default, checkout action brings only the interesting commit (this is where the "grafted" is coming from), so we can configure it to bring all the history as
  - uses: actions/checkout@v2
    with:
      fetch-depth: 0 # workaround for grafted issue

This is far more inefficient but it works if the repos has a reasonable size.

Hi Roberto,
thanks for this well documented issue.
I will do some testing this evening and start to fix this problem. Little remark here: I used git branch --show-current before but I decided to drop this because of requirement to be compatible with git version older than 2.20.0.
Matthias

Can confirm the fix. Thx for tackle this so fast :) !

Thanks for the feedback :)


PureCN: if signatures aren't generated (too few somatic calls) they shouldn't be considered for upload

Version info

bcbio version (bcbio_nextgen.py --version): 1.2.8
OS name and version (lsb_release -ds): Debian 10 (buster)

To Reproduce
If by any chance deconstructSigs fails to generate a proper signature (because there aren't enough somatic calls) when called from PureCN, the pipeline still considers the file to be done and adds it to the metadata. This ultimately causes a failure before the upload step, because _add_meta finds a file that does not exist.
Example:
Traceback (most recent call last):
  File "/opt/bcbio/1.2.8/anaconda/lib/python3.6/site-packages/bcbio/distributed/ipythontasks.py", line 54, in _setup_logging
    yield config
  File "/opt/bcbio/1.2.8/anaconda/lib/python3.6/site-packages/bcbio/distributed/ipythontasks.py", line 378, in multiqc_summary
    return ipython.zip_args(apply(multiqc.summary, *args))
  File "/opt/bcbio/1.2.8/anaconda/lib/python3.6/site-packages/bcbio/distributed/ipythontasks.py", line 82, in apply
    return object(*args, **kwargs)
  File "/opt/bcbio/1.2.8/anaconda/lib/python3.6/site-packages/bcbio/qc/multiqc.py", line 88, in summary
    data_json_final = _save_uploaded_data_json(samples, data_json, os.path.join(out_dir, "multiqc_data"))
  File "/opt/bcbio/1.2.8/anaconda/lib/python3.6/site-packages/bcbio/qc/multiqc.py", line 146, in _save_uploaded_data_json
    upload_path_mapping.update(get_all_upload_paths_from_sample(sample))
  File "/opt/bcbio/1.2.8/anaconda/lib/python3.6/site-packages/bcbio/upload/__init__.py", line 49, in get_all_upload_paths_from_sample
    for finfo in _get_files(sample):
  File "/opt/bcbio/1.2.8/anaconda/lib/python3.6/site-packages/bcbio/upload/__init__.py", line 64, in _get_files
    return _get_files_variantcall(sample)
  File "/opt/bcbio/1.2.8/anaconda/lib/python3.6/site-packages/bcbio/upload/__init__.py", line 192, in _get_files_variantcall
    return _add_meta(out, sample)
  File "/opt/bcbio/1.2.8/anaconda/lib/python3.6/site-packages/bcbio/upload/__init__.py", line 161, in _add_meta
    raise ValueError("Unexpected path for upload: %s" % x)
ValueError: Unexpected path for upload: {'path': '/home/incalci/lbeltrame/Ovaio/OV.4/OV4/work/structural/21556-A/purecn/21556-A_signatures.pdf', 'sample': '21556-A', 'type': 'pdf', 'ext': 'purecn-signatures', 'variantcaller': 'purecn'}

@lbeltrame , I think I changed the behavior in 1.22.0 that the file is always generated. It will just contain NAs for all signatures in case there are not enough mutations.

thanks @lima1 !
@lbeltrame thanks for reporting!
Could you please confirm that it fails /not fails with the latest purecn as well? If so, I'd handle at the bcbio level.
SN

The difference between the case where signatures are generated and when they aren't is here:
https://github.com/lima1/PureCN/blob/4f0616fb4e2875c5f64fd2b456b6b05fe5e4d8d7/inst/extdata/Dx.R#L187
One case generates a PDF file, which is tracked by bcbio, one does not (but bcbio searches for it anyway).

Oh, yes, the PDF is not generated, only the CSV file.

@lbeltrame here is the fix https://github.com/bcbio/bcbio-nextgen/pull/3563,
could you please test?

Would that work with samples that have files, and samples that do not?

yes, in my test it works

there was another fix in this module - please check out + purecn2.0.1 fix.

I'll see about getting bcbio updated in the HPC system I use.


Add --coverage to pkg.script.test if coveralls is selected

In case jsdom is the environment jest is configured to show coverage results but fortestEnvironmnet: node you need to manually add --coverage in the test script of the package.json file because the jest configuration is cleaned up.
I've added a fix to add the --coverage parameter if the testEnvironment is node and coveralls is enabled.
Had to do some changes to the already existing tests because the coveralls prompt doesn't have a default value and inquirer sets it to true by default, so I had to set coveralls to false in some tests. I've added two tests for the new behaviour.

Thanks for the PR, sorry for taking so long to review and merge.


workflow/go: Add Codecov coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@48a2f83). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage        ?   52.69%
=======================================
  Files           ?       12
  Lines           ?      687
  Branches        ?        0
=======================================
  Hits            ?      362
  Misses          ?      321
  Partials        ?        4

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 48a2f83...9fffd86. Read the comment docs.


5.2.0: using sqlConnection.FireInfoMessageEventOnUserErrors = true generates "Object reference not set to an instance of an object." exception

Describe the bug
When using sqlConnection.FireInfoMessageEventOnUserErrors = true, a "print" sql command always throws an exception
Exception message:
Object reference not set to an instance of an object.
Stack trace:
at Microsoft.Data.SqlClient.SqlCommand.GetCurrentBatchCommand()
at Microsoft.Data.SqlClient.TdsParser.FireInfoMessageEvent(SqlConnection connection, SqlCommand command, TdsParserStateObject stateObj, SqlError error)
at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
at Microsoft.Data.SqlClient.SqlCommand.RunExecuteNonQueryTds(String methodName, Boolean isAsync, Int32 timeout, Boolean asyncWrite)
at Microsoft.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(TaskCompletionSource`1 completion, Boolean sendToPipe, Int32 timeout, Boolean& usedCache, Boolean asyncWrite, Boolean inRetry, String methodName)
at Microsoft.Data.SqlClient.SqlCommand.ExecuteNonQuery()
at SqlClientBug.Program.Main(String[] args) in E:\NAS2\COBEMAR\2022\dotNet\SqlClientBug\Program.cs:line 31
To reproduce
Include a complete code listing (or project/solution) that we can run to reproduce the issue.
Partial code listings, or multiple fragments of code, will slow down our response or cause us to push the issue back to you to provide code to reproduce the issue.
namespace SqlClientBug
{
   internal class Program
    {
        static void Main(string[] args)
        {
            SqlConnectionStringBuilder sb = new();

            sb.TrustServerCertificate = true;
            sb.DataSource = ".";
            sb.InitialCatalog = "tempdb";
            sb.IntegratedSecurity = true;
            sb.ApplicationName = "SqlClientBug";

            using SqlConnection sqlConnection = new(sb.ConnectionString);
            sqlConnection.FireInfoMessageEventOnUserErrors = true;
            sqlConnection.Open();
            using SqlCommand cmd = sqlConnection.CreateCommand();
            cmd.CommandType = System.Data.CommandType.Text;
            cmd.CommandText = "print 'OK';";
            try
            {
                cmd.ExecuteNonQuery();
            }
            catch (Exception ex)
            {
                Console.Error.WriteLine(ex.Message);
                Console.Error.WriteLine("--------------------");
                Console.Error.WriteLine(ex.StackTrace);
            }
        }
    }
}

Expected behavior
As in previous versions: no Exception
Further technical details
Microsoft.Data.SqlClient version: 5.2.0
.NET target: .NET 8.0
SQL Server version: SQL Server 2022
Operating system: Windows 11
Additional context
No problem when using Microsoft.Data.SqlClient version 5.1.5

We have the same problem.
Additional information:
The error occurs only when commandType != procedure and only when the passed query returns a warning or error.
Generally, from what we have observed, the error is caused by the uninitialized array _rpcArrayOf1 in SQLCommand class, which is used in this case to return SQLBatchCommand for the SqlException.CreateException  method.
The auxiliary query on which this error occurs:
SELECT COUNT(Floor) C FROM (VALUES (1), (NULL), (2)) X (Floor)

Ok. I debugged through it and understand the situation now. The library only needs to use an RPC object internally if there are parameters and in this case there aren't so it follows a path that simply sends the text without first building any sort of RPC. It'd be good to get this sort of breakdown as part of the fix so those of us following along at home can understand why the fix works.
It does raise a question or what happens if you try to setup a BatchCommand with a set of commands that have no parameters, would errors be reported correctly or crash?

I have the same problem. When should we expect a release that we can use?


Added Porkalai Selvi S

Thank you for your valuable contribution!
Make sure to leave a star ⭐ on repo


[Bug Report], when two groups of Radio or two groups of Switch are together, the second group will not be able to render the normal render. Requires the first set of triggers

Element UI version
1.3.7
OS/Browsers version
chrome
Vue version
2.3.4
Reproduction Link
https://jsfiddle.net/6fmwah8n/
Steps to reproduce
当第一排选中是，第二排选择后，无法正常渲染
What is Expected?
完成正常渲染
What is actually happening?
当第一排选中是，第二排选择后，无法正常渲染

radio1 is not registered: https://jsfiddle.net/6fmwah8n/1/

thanks


Modify nodejs package version in test

Modify the test for nodejs package version. Mariner upgraded their nodejs package. Therefore, the test in Cloudshell was failing.

LGTM


Error while downloading course (local variable referenced before assignment)

`2022-08-30 11:28:58 | ERROR     | An error has been caught in function '<module>', process 'MainProcess' (1597), thread 'MainThread'<PHONE_NUMBER>80416):
Traceback (most recent call last):

> File "ine.py", line 1021, in <module>
    main()
    └ <function main at 0x7fe06fddd670>

  File "ine.py", line 1002, in main
    download_course(course_dl)
    │               └ {'id': '80a726f2-6cbc-44fd-9a9f-70210f42ac65', 'tags': ['Network Security', 'Offensive Security'], 'images': {'main_image_url...
    └ <function download_course at 0x7fe06fddd5e0>

  File "ine.py", line 873, in download_course
    download_video(name, content_uuid, complete_path, index)
    │              │     │             │              └ '01'
    │              │     │             └ 'OSCP Security Technology Prep Course/01 - Course/13 - Capstone'
    │              │     └ '0b065ff8-569c-4bc7-aa4f-9b819b1258ff'
    │              └ 'OSCP Security Technology Prep Course'
    └ <function download_video at 0x7fe06fddd280>

  File "ine.py", line 403, in download_video
    if not os.path.isfile(f"{complete_path}/{video_name}") or os.path.isfile(f"{complete_path}/{video_name}.aria2"):
           │  │    │                                          │  │    └ <function isfile at 0x7fe0715e9d30>
           │  │    │                                          │  └ <module 'posixpath' from '/usr/lib/python3.8/posixpath.py'>
           │  │    │                                          └ <module 'os' from '/usr/lib/python3.8/os.py'>
           │  │    └ <function isfile at 0x7fe0715e9d30>
           │  └ <module 'posixpath' from '/usr/lib/python3.8/posixpath.py'>
           └ <module 'os' from '/usr/lib/python3.8/os.py'>

UnboundLocalError: local variable 'video_name' referenced before assignment`

Can reproduce the issue. Will probably push a fix soon, thanks for reporting!

For any future issues can you also please include the arguments you launched the script with? Thanks!


New version: iMobieInc.Ashampoo version 26.00.22

- [X] Have you signed the Contributor License Agreement?

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.5 schema?

Resolves #131693

Microsoft Reviewers: Open in CodeFlow

Must be something weird about the azure agents

Manual Validation ended with:

1/3/2024 8:09:58 PM
Installer Verification Analysis Context Information:
Specified hash doesn't match.
Actual hash:   CC65F50E4DC257A21BEAFD4CB7BEA1AEBF201A7037436812ABED61A8AA274368
Expected hash:<PHONE_NUMBER>c97eb3d38d84cd711d98d7cc0f8a2992ecaf6b15a7eb8b6faf8d4d

(Automated response - build 516.)

Duplicate of https://github.com/microsoft/winget-pkgs/pull/131693


list_blobs does not work if prefix is empty string

Which service(blob, file, queue) does this issue concern?
blob
What problem was encountered?
When calling BlockBlobService.list_blobs(prefix="") with prefix="" will cause an authentication error to be thrown thrown.
azure.common.AzureHttpError: Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.ErrorCode: AuthenticationFailed
<?xml version="1.0" encoding="utf-8"?><Error>AuthenticationFailed<Message>Server failed to authenticate the request. Make sure the value of Authorization header is formed correctly including the signature.
RequestId:d7eda6cf-b01e-0024-55e6-e0d309000000
Time:2018-05-01T00:52:44.4779222Z</Message><AuthenticationErrorDetail>The MAC signature found in the HTTP request '/LACPvjjxGLO/jhbYW19z/4xlPYlZt9DYMp9Rvb/H88=' is not the same as any computed signature. Server used following string to sign: 'GET

x-ms-client-request-id:f4aba85a-4cd9-11e8-b3d1-645106571c89
x-ms-date:Tue, 01 May 2018 00:52:43 GMT
x-ms-version:2017-07-29
/rozhdevv2/test
comp:list
prefix:
restype:container'.</AuthenticationErrorDetail></Error>

Have you found a mitigation/solution?
Yes, the reason is that empty prefix is omitted as part of signing process but since it is included in the query param strings sent to the server, the server is actually verifying the signing with prefix set as empty string, which causes the signatures to mismatch.
Note: for table service, please post the issue here instead: https://github.com/Azure/azure-cosmosdb-python.

After further investigation, this is caused by calling list_blobs with prefix="". Updated the title and description to reflect this and proposed #452 as a fix.

@rongduan-zhu Thanks for the PR, I'll keep this issue open until the fix is released.

The fix is in 1.2.0rc1. I'll close this issue once the preview period of 1.2.0 is over.

Published in version 1.3.0.


CI: json lint

What is the expected enhancement?
Add a json lint in the CI, to check the good syntax of members.json

@IceKhan13 can you assign it to me please :)


Added a rake task to sync RBAC roles for existing workflows

rake approval:sync_rbac_roles task is added to sync RBAC roles for workflows.

@hsong-rh What is the use case for this is this only for development and QE as part of debugging cleanup. In real production environment we won't be able to run any rake tasks.

@mkanoor After RBAC is enabled, approver in a certain group cannot process any requests assigned  to him, because the group he belongs to does not have role. This rake task will create roles for all such groups.

@hsong-rh After these PR's are merged wouldn't RBAC be enabled as default so this rake task is a temporary thing for QE and DEV correct?

@mkanoor It is true the tool is a one time and temporary thing. It does not need to be a rake task. We just need a way to share the tool and save the source code. Any suggestion?

@bzwei @hsong-rh I think any rake task is a hack and not practical in a production containerized environment. We have a similar set to debugging rake tasks that we carry in catalog service. So for now it's ok. Maybe at some point we should start deprecating it.

we can do the same thing. We can remove the task after everybody enrolls in RBAC roles.


Support or integrate with Rocksdb statebackend?

Support or integrate with Rocksdb statebackend?

Thanks for the interest!
The project doesn't depend on the state store implementation, as the project only uses the state store "interface". I wouldn't mention specific implementations on the support list - it would make the cost of the maintenance very high.
Hopefully @redsk is playing this project with RocksDB state backend (@chermenin) so let's watch the test result of #33. (You can enable notification on the PR.)
Thanks again. Closing.


[Breadth Coverage] Onboard StorageCache Commands

Spec Link
https://microsoft.sharepoint.com/:w:/t/AzureManagementExperience/EcJFauYB7JdHovw2P13_9_0B3hM6cI7G7Mk174gZFB4wiQ?e=KH8s1j
ADO Work Item Link
https://dev.azure.com/azure-mgmt-ex/Onboarding/_workitems/edit/194
Resource Provider
What is the Azure resource provider your feature is part of?
Microsoft.StorageCache
Description of Feature or Work Requested
Support Azure CLI commands for StorageCache
Minimum API Version Required
Microsoft.Storage 2019-11-01
Swagger Link
https://github.com/Azure/azure-rest-api-specs/tree/master/specification/storagecache/resource-manager/Microsoft.StorageCache/stable/2019-11-01
Python SDK Link
https://github.com/Azure/azure-sdk-for-python/tree/master/sdk/storage/azure-mgmt-storagecache
Target Date
03/31/2020
Addtional context
We'll use autorest.az from code gen team to auto-generate the code first and may need update some code manually if the generated code doesn't meet your requirements.

add to S167.


Add arithmetic expressions to the type grammar

Feature Request
StaticArray(T, S) has an interesting property where the size of the array is specified in the type grammar. I tried to follow this pattern in a 2 dimensional matrix struct (below), however it appears that this results in a syntax error.
struct Matrix(T, W, H)
  property values : StaticArray(T, W * H)

  def initialize(@values)
  end
end

I think allowing this type of expression in the type grammar would be a nice improvement to the language.

Previous discussion on the forum: https://forum.crystal-lang.org/t/is-there-a-way-to-combine-special-argument-types-that-denote-size-through-macros/4277

Thinking about how to implement this in the most generic way...
A type like StaticArray(T, 3) is represented with type vars TypeParameter("T") for the first one, and NumberLiteral(3) for the second one. I guess we could introduce another type parameter like CallParameter similar to a call. In this case it would be something like CallParameter(TypeParameter("W"), "*", [TypeParameter("H")]). Then when instantiating this parameter with both W and H number literals we can do the math and produce a new NumberLiteral. We could support all math operations here, but I don't know if much more.
Also, I'm not sure how much complexity this adds to the language and whether it's worth it.

If this does get added, it would be good to also resolve https://github.com/crystal-lang/crystal/issues/4238. Since the type parameter has to be a literal, passing in a variable leads to an unexpected token error which can be a bit confusing.

Thanks @jwoertink, that reference brought me to #5427. I knew there had already been an issue for this 😀
Let's continue the discussion about arithmetic expressions here, though. It's more focused.

You could somewhat circumvent multiplication by using sizeof, but it is a bit tricky since you need an extra type to store the result:
module Matrix(T, W, H)
  struct Impl(T, W, H, WH)
    include Matrix(T, W, H)

    @values = uninitialized T[WH]
  end

  def self.new
    Impl(T, W, H, sizeof(UInt8[W][H])).new
  end
end

Matrix(Int32, 3, 2).new # => Matrix::Impl(Int32, 3, 2, 6)(@values=StaticArray[0, 0, 0, 0, 0, 0])

Addition works similarly by sizeof(Tuple(UInt8[X], UInt8[Y])).


[BUG] Something broke when publishing...

First of all: Thank You for what you are doing - I think it's amazing.
I wanted to publish a module and got the following eggs-debug.log which I could not interpret by myself to an extent that I could fix the error :)
So perhaps you could automatically add a hint on what went wrong so that the publisher knows how to fix it.
Arguments:
publish
Deno version:
deno: 1.2.0
v8: 8.5.216
typescript: 3.9.2
Eggs version:
0.2.1
Platform:
x86_64-apple-darwin
2020-09-07T12:34:50.913Z [DEBUG]    Config:  {
name: "log",
description: "Frequently used logger configuration - writing warnings and errors into a file, adding date and time...",
stable: true,
files: [
"logger.ts",
"mod.ts",
"README.md",
"usage-example-sub-module.ts",
"usage-example.ts"
]
}
2020-09-07T12:34:50.916Z [WARNING]  Your readme contains old import URLs from your project using deno.land/x/log.
2020-09-07T12:34:50.916Z [WARNING]  You can change these to https://x.nest.land/${name}@VERSION
2020-09-07T12:34:51.699Z [WARNING]  No version found. Generating a new version now...
2020-09-07T12:34:51.700Z [DEBUG]    Module:  {
name: "log",
description: "Frequently used logger configuration - writing warnings and errors into a file, adding date and time...",
repository: undefined,
version: "0.0.1",
unlisted: undefined,
upload: true,
latest: true
}
2020-09-07T12:34:52.469Z [CRITICAL] An unexpected error occurred: "Something broke when publishing... " Error: Something broke when publishing...
at Command.publishCommand [as fn] (publish.ts:255:11)
at async Command.execute (command.ts:666:17)
at async Command.parse (command.ts:603:20)
at async Command.parse (command.ts:570:20)
at async mod.ts:47:23

@michael-spengler Thank you for reporting this. This could probably be the API unexpectedly closing the connection with the CLI, could you try running the publish command again?

@michael-spengler The name log is reserved as per https://github.com/nestdotland/x/blob/c94add3607c9fb7b45ac916d5f0776eb0fc0aeb1/src/utils/reservedNames.ts#L16
because of the std module log

@michael-spengler The name log is reserved as per https://github.com/nestdotland/x/blob/c94add3607c9fb7b45ac916d5f0776eb0fc0aeb1/src/utils/reservedNames.ts#L16
because of the std module log

That's right, it works as expected 😉


chat dialog back to left

Is there any chance or options to move the chat dialog back to left slide?
My screen not so big to use both side...
Thanks.

It is easy to bring the chat back to the primary sidebar, just drag and drop it over:
https://github.com/user-attachments/assets/eadb8493-cec8-4848-81d2-4f980e1bae54

Got it. Thank you !


RADIUS support (Closes #409)

Description of PR

What component(s) does this PR affect?

[ ] Back-End (Database, Detection/Configuration/etc. Microservices)
[x] Front-End (Flask, API, UI, etc)
[ ] Monitor (RIPE RIS, BGPStream RV/RIS/CAIDA, etc.)
[ ] Docs (incl. wiki)
[ ] Build System

Does the PR require changes on other components? If yes, please mark the components:

[ ] Back-End (Database, Detection/Configuration/etc. Microservices)
[ ] Front-End (Flask, API, UI, etc)
[ ] Monitor (RIPE RIS, BGPStream RV/RIS/CAIDA, etc.)
[ ] Docs (incl. wiki)
[ ] Build System

Related Issue

Resolves #409
Solution

integration is inspired by LDAP integration (#108 )
radius connection is done via pyrad library (https://github.com/pyradius/pyrad)
conspect:

if enabled (AUTH_METHOD = "radius"): config validation is performed during bootstraping of the frontend
radius servers are tried out in the order provided in the config; last successful server is used for next attempt (caveat: for the sake of simplicity it is per gunicorn process)
for AccessAccept responses optional attribute inspection is done to support mapping to either "user" or "admin" role
user is not assigned password in the local db (i.e. RADIUS response is always required to log in)
superuser account is always available

Type

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Docs update
[ ] None of the above

Checklist:

[x] I have read the contributing guide and my code conforms to the guidelines.
[ ] This change requires a change in the documentation.
[x] I have updated the documentation accordingly.
added self-documented entries in webapp.cfg

Labeler is not working because the PR is from the forked repository

LGTM 🎉
Thanks a lot for the PR and for adding support for RADIUS!
Please check if the fix that I introduced solves the roles_users problem and then we can merge this.

as a matter of fact it does solve the problem;
so now I have replaced sql with flask api calls: .remove_role_from_user() and .add_role_to_user()
it's worth considering also adding primary key to the roles_users in models.py so new dbs will be protected against double entries in that table

Thanks for the suggestion, I will add the key in a future release!


add default migration for 'desired_pool_number' in protocol params table

Issue Number

N/A
Overview

b798abc1d0cee56976cfba535726f4c90fb54e72
:round_pushpin: add default migration for 'desired_pool_number' in protocol params table
Defaulting to 50 without which the wallet integrity check fails (genesis parameters have to be identical to the network's parameters).

Comments

bors r+

unrelated failure :point_up: , reported in the spreadsheet.


Try Nicer Fonts

https://github.com/posquit0/Awesome-CV/blob/5b05d935658377c73a3456269d23a276b19804f7/awesome-cv.cls#L164

Don't look that nice.


Error on createdb: mezzanine.accounts.ProfileNotConfigured

Using the master branch from this repo with python setup.py install the createdb command gives me this error:
mezzanine.accounts.ProfileNotConfigured

What do I have to do to just get this running so I can test mezzanine?
Thanks again for your attention.

#35

aha, well, thanks again for your attention.
y not u put that in Readme, man? Why?
Anyway, have a nice sunday, please keep up the good work!


[Docs]: Remove footer and add icons for important links on top bar

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
No response
Discord/slack/intercom Link
No response
Describe the problem
The footer in the new Docusaurus site does not have relevant information. It was added there to primarily contain the links for discord, twitter, youtube, github etc.  On Docusaurus, we had to do this using swizzle.(https://github.com/facebook/docusaurus/issues/2634) and it would have delayed the project trying to figure out this part.
In the next iteration, we can add these icons on the topbar and remove the footer.
Describe the improvement
na
Why do you think this change is needed?
na

As discussed, FYI - @tejasahluwalia


is_invocable<void(foo::*)()const&&, foo>::value should be true

Correct the bad test case and fix the impl.
Proof: https://wandbox.org/permlink/3h2mbErYlggcjoAl

Closing because this already works as intended. I confused myself :(


Update: HTTP -> HTTPS

URLs updated to use HTTPS protocol where appropriate to improve security and privacy.

Can one of the admins verify this patch?


--norollback parameter causing issues when additional compute nodes are created

Environment:

AWS ParallelCluster / CfnCluster version

2.8.1

Configuration file (i.e. ~/.parallelcluster/config) without any credentials or personal data.

[aws]
aws_region_name = ap-southeast-2

[global]
cluster_template = umccr_dev
update_check     = true
sanity_check     = true

[cluster umccr_dev]
base_os               = alinux2
vpc_settings          = umccr_dev_network
# Definitely experimental
fsx_settings          = umccr_dev_lustrefs
s3_read_resource      = *
cluster_type          = spot
key_name              = alexis-wfh-dev
# Need something substantial to hold the slurm database
master_instance_type  = t2.medium
# 16 CPUs and 64 Gb
compute_instance_type = m5.4xlarge
# Using additional_iam_policies over ec2_iam_role
# Add SSM policy, so we can keep port 22 closed
additional_iam_policies = arn:aws:iam::aws:policy/AmazonSSMManagedInstanceCore
scheduler             = slurm
initial_queue_size    = 1
# Basic ami with installations of
# R, python3.8, conda, pip, ruby, golang, rust
# conda has been initialised to the ec2-user
# Currently built manually - create image builder for this.
custom_ami            = ami-02f2ae59ffab4fbfb
post_install          = s3://umccr-temp-dev/Alexis_parallel_cluster_test/bootstrap/bootstrap-slurm-cromwell.sh

[vpc umccr_dev_network]
# Default vpc for dev account
vpc_id = vpc-00eafc63c0dfca266
# Compute subnet ids take the same subnet as the master
# Our default public subnet
master_subnet_id = subnet-0fab038b0341872f1
# Elastic IP address is associated to the master instance.
use_public_ips = true
# Security group where the slurm accounting database sits
additional_sg         = sg-0ca5bdaab39885649

[aliases]
ssh = ssh {CFN_USER}@{MASTER_IP} {ARGS}

[fsx umccr_dev_lustrefs]
shared_dir                = /fsx
storage_capacity          = 1200
imported_file_chunk_size  = 1024
# FIXME how to mount this folder at /fsx
import_path               = s3://umccr-temp-dev/Alexis_parallel_cluster_test/input_folder

Bug description and how to reproduce:
Running pcluster create with the --norollback flag generates a situation where additional compute nodes cannot be registered. The compute nodes are spun up successfully when the --norollback flag is NOT specified.
How to reproduce:

Spin up the cluster with pcluster create
Wait ten minutes for the first compute node to terminate.
Then run srun --pty bash. It will try to spin up a new compute node.
The node will start running but an error will be seen from slurm

Slurm error:
srun --pty bash
srun: Required node not available (down, drained or reserved)
srun: job 5 queued and waiting for resources
srun: job 5 has been allocated resources
srun: error: fwd_tree_thread: can't find address for host ip-10-2-1-121, check slurm.conf
srun: error: Task launch for 5.0 failed on node ip-10-2-1-121: Can't find an address, check slurm.conf
srun: error: Application launch failed: Can't find an address, check slurm.conf
srun: Job step aborted: Waiting up to 32 seconds for job step to finish.
srun: error: Timed out waiting for job step to complete

Snippet from /var/log/messages on compute node:
Aug 19 07:34:46 ip-10-2-1-121 cloud-init: + cfn-signal --exit-code=0 '--reason=ComputeServer setup complete' --stack=parallelcluster-AlexisFirstBcBioCondaInstallTest --resource=ComputeFleet --region=ap-sou
theast-2
Aug 19 07:34:46 ip-10-2-1-121 ec2net: [get_meta] Trying to get http://<IP_ADDRESS>/latest/meta-data/network/interfaces/macs/02:42:e2:af:00:9f/local-ipv4s
Aug 19 07:34:46 ip-10-2-1-121 cloud-init: ValidationError: Stack arn:aws:cloudformation:ap-southeast-2:843407916570:stack/parallelcluster-AlexisFirstBcBioCondaInstallTest/3d0b0240-e1c5-11ea-bd59-06fa27f652
ae is in CREATE_COMPLETE state and cannot be signaled
Aug 19 07:34:46 ip-10-2-1-121 cloud-init: Aug 19 07:34:46 cloud-init[3948]: util.py[WARNING]: Failed running /var/lib/cloud/instance/scripts/part-002 [1]
Aug 19 07:34:46 ip-10-2-1-121 cloud-init: Aug 19 07:34:46 cloud-init[3948]: cc_scripts_user.py[WARNING]: Failed to run module scripts-user (scripts in /var/lib/cloud/instance/scripts)
Aug 19 07:34:46 ip-10-2-1-121 cloud-init: Aug 19 07:34:46 cloud-init[3948]: util.py[WARNING]: Running module scripts-user (<module 'cloudinit.config.cc_scripts_user' from '/usr/lib/python2.7/site-packages/
cloudinit/config/cc_scripts_user.pyc'>) failed

Please let me know if you need any more logs.
Alexis.

Hi Alexis.
This issue isn't related to pcluster create's --no-rollback flag. This flag is passed as the DisableRollback parameter to CloudFormation's CreateStack API. Its only function is to prevent CloudFormation from deleting all of the stack resources if cluster creation fails.
Those messages from /var/log/messages are expected. They are caused by the fact that the signaling process (used to make sure the minimum number of initial compute nodes is ready before a cluster is considered to have been successfully created) is no longer necessary.
Instead I think the issue you're seeing is caused by the fact that ParallelCluster doesn't (currently) support the use of interactive commands like srun to scale a cluster up. Use this type of interactive command only after a cluster has scaled up to the desired size.
For example, if you wanted to use srun to run a job interactively across three nodes on a cluster that had an initial size of zero compute nodes, you could do :

scale the cluster up: sbatch -N 3 --wrap='sleep 60'
loop until the job is finished running: while scontrol show job ${JOB_ID_FROM_PREVIOUS_STEP} | grep 'JobState=COMPLETED'; do sleep 10; done
run the interactive command: srun [args] /path/to/app

(The 'duplicate' label added due to the similarity of this issue to #1347)

Thank you, I have been able to reproduce this.

Thanks for the response, @tilne ... since you seem to have good insight into this, what's the real underlying issue? Simply SLURM not waiting for long enough for instances to come up (default seems to be 32 seconds?) or something else more related to AWS's parallelcluster instead?
If you were to fix this issue (other than using your workaround), would you start looking at SLURM's codebase or somewhere else?

scale the cluster up: sbatch -N 3 --wrap='sleep 60'
loop until the job is finished running: while scontrol show job ${JOB_ID_FROM_PREVIOUS_STEP} | grep 'JobState=COMPLETED'; do sleep 10; done

This could be shortened to
sbatch --wait --wrap='sleep 2'
The --wait parameter prevents sbatch from returning until the job has terminated.

Thanks for the response, @tilne ... since you seem to have good insight into this, what's the real underlying issue? Simply SLURM not waiting for long enough for instances to come up (default seems to be 32 seconds?) or something else more related to AWS's parallelcluster instead?

It's related to how ParallelCluster reconfigures slurm as new compute nodes come online and are added to a cluster.

If you were to fix this issue (other than using your workaround), would you start looking at SLURM's codebase or somewhere else?

It should be fixed in v2.9.0.

This could be shortened to
sbatch --wait --wrap='sleep 2'
The --wait parameter prevents sbatch from returning until the job has terminated.

I did not know about that flag. It's very useful. Thanks for the tip @alexiswl!

Work around with sinteractive
sinteractive.sh
#!/usr/bin/env bash

check_nodes(){
  # Check a node is ready
  # 1 for yes, 0 for no
  node_length=$(scontrol show nodes --oneliner | grep -v "DRAIN" | grep -cv "No nodes in the system")
  echo "${node_length}"
}

if [[ "$(check_nodes)" == "0" ]]; then
  sbatch --wait --wrap "sleep 2" >/dev/null 2>&1
fi

# tcsh implementation adds -l param
if [[ ! "${SHELL}" == "/bin/tcsh" ]]; then
  eval srun "$*" --pty -u "${SHELL}" -i -l
else
  eval srun "$*" --pty -u "${SHELL}" -i
fi

Link to known path dir
ln -s /path/to/sinteractive.sh /usr/bin/sinteractive
```

### Usage
```
sinteractive --cpus-per-task=2 --mem=10G
```

I'm going to resolve this one since starting from version 2.9 ParallelCluster supports the usage of srun command with Slurm scheduler

Yes thank you, we've been able to reproduce this on our side


Introduce AccountContext

Similar to HoldingContext and InstrumentContext we should have an AccountContext that provides convenient access to accounts.

https://github.com/digital-asset/daml-finance-app/pull/41


infra: Bump Go to version 1.20.10

What this PR does / why we need it:
Build KubeVirt with go 1.20.10
go 1.20.10 contains a number of security fixes [1]
[1] https://go.dev/doc/devel/release#go1.20
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
/cc @xpivarc @EdDev
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[x] Design: A design document was considered and is present (link) or not required
[x] PR: The PR description is expressive enough and will help future contributors
[x] Code: Write code that humans can understand and Keep it simple
[x] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[x] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[x] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[x] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[x] Community: Announcement to kubevirt-dev was considered

Release note:
Built with golang 1.20.10

/test pull-kubevirt-build

/test pull-kubevirt-build

/test pull-kubevirt-build

/test pull-kubevirt-unit-test
/test pull-kubevirt-generate
/test pull-kubevirt-e2e-k8s-1.28-sig-compute

/test pull-kubevirt-build
/test pull-kubevirt-build-arm64
/test pull-kubevirt-manifests
/test pull-kubevirt-generate

/test pull-kubevirt-build
/test pull-kubevirt-build-arm64
/test pull-kubevirt-manifests
/test pull-kubevirt-generate
/test pull-kubevirt-unit-test

/test ?

/test all

/retest

/test pull-kubevirt-e2e-k8s-1.27-sig-compute
/test pull-kubevirt-build

/retest

/test pull-kubevirt-build
/test pull-kubevirt-build-arm64
/test pull-kubevirt-manifests
/test pull-kubevirt-generate
/test pull-kubevirt-unit-test
/test pull-kubevirt-e2e-k8s-1.28-sig-compute
/test pull-kubevirt-e2e-k8s-1.28-sig-network

/test pull-kubevirt-build
/test pull-kubevirt-build-arm64
/test pull-kubevirt-manifests
/test pull-kubevirt-generate
/test pull-kubevirt-unit-test

/test pull-kubevirt-e2e-k8s-1.28-sig-compute

/test pull-kubevirt-e2e-k8s-1.28-sig-compute

/test all

/cc

/test pull-kubevirt-build
/test pull-kubevirt-build-arm64
/test pull-kubevirt-manifests
/test pull-kubevirt-generate
/test pull-kubevirt-unit-test

/test pull-kubevirt-build
/test pull-kubevirt-build-arm64
/test pull-kubevirt-manifests
/test pull-kubevirt-generate
/test pull-kubevirt-unit-test
/test pull-kubevirt-e2e-k8s-1.28-sig-compute

/test all

/test pull-kubevirt-e2e-k8s-1.28-sig-storage
/test pull-kubevirt-e2e-k8s-1.28-sig-compute

/test pull-kubevirt-e2e-k8s-1.28-sig-storage
/test pull-kubevirt-e2e-k8s-1.28-sig-compute

/test all

/close
Superceded by https://github.com/kubevirt/kubevirt/pull/10879


Add maven-central config to Java11 template

Signed-off-by: Alex Ellis (OpenFaaS Ltd<EMAIL_ADDRESS>
Description

Add maven-central config to Java11 template
Motivation and Context
Closes: #210
How Has This Been Tested?

Testing is pending the release of the artifact to Maven Central for the model jar.

Fixed in https://github.com/openfaas/templates/pull/214 instead

/add label: merged

/add label: support


Prototype Pollution for minimist

Hi, could you update this dependency on the new version?
Moderate        Prototype Pollution
Package         minimist
Patched in      >=1.2.3
Dependency of   webpack-dev-server [dev]
Path               webpack-dev-server > chokidar > fsevents > node-pre-gyp > tar > mkdirp > minimist
More info       https://npmjs.com/advisories/1179
Moderate        Prototype Pollution
Package         minimist
Patched in      >=1.2.3
Dependency of   webpack [dev]
Path            webpack > watchpack > chokidar > fsevents > node-pre-gyp > rc > minimist
More info       https://npmjs.com/advisories/1179

This is a duplicate of #10561
mkdirp needs to be updated to >1.0.0.   Mkdirp has had a major version change, as the API has changed to use promises.
I tried using the resolutions field with https://www.npmjs.com/package/npm-force-resolutions
but the change to the api is too much.

Doh.  Reading the rest of that issue #10561, I see that a request for a backport fix was made to the mkdirp team.
So I added the following line to the resolutions section of my package.json:
"mkdirp": "^0.5.3"
and then ran npm install, keeping in mind that I have https://www.npmjs.com/package/npm-force-resolutions with a pre-install script installed.
This then updated mkdirp and no vulnerabilities were reported.


Beta 8-B. Sometimes the ckEditor dont remember my changes

(maybe this is only in my site).
I make changes "hello" to "hello world" save and publish - my html is now "hello world" than edit this block again and get "hello".
I can not put the finger when its ok and when its not.

I will check this again - mabye this is "cache" problem (my host company set for "true" some cacher without i know about this").


Invisible wheel

Describe the Bug

---------------My machine ---------------------

---------------When I get back to my machine! Rejoin my world then it happened--------------------------
From the above image, you can see the water wheel still exists in the world but I can see it.
The water wheel and shaft turn invisible when I saved my single-player world and rejoin my single-player world.
Reproduction Steps

put down a water wheel, shaft,set up your machine (mechanical press)
save and quit (back to Minecraft main menu)
rejoin the single-player world
water wheel and all the shaft is now invisible, I can only see mechanical press

...
Expected Result
I expected all the thing is visible when I get back to my single-player world.
Screenshots and Videos
No response
Crash Report or Log
no, don't know what log I can provide
Operating System
Win 10
Mod Version
0.4.0b
Minecraft Version
1.18.1
Other Mods
Better Minecraft without iris mod
Additional Context
No response

Duplicate of #54


[feat] [broker] PIP-188 Fix cluster migration state store into local namespace policies

Motivation
Right now, Namespace policies are stored in global metadata store which can be shared by glue and green cluster during migration which incorrectly cause migration in green cluster as well so, we need to store namespace migration policies in the local metadata store so, that each cluster can manage its own migration state and it can avoid cyclic migration impact in green cluster as well.
Modifications
Add namespace policies into local policies and avoid cyclic cluster migration for green cluster.
It depends on https://github.com/apache/pulsar/pull/21354
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@rdhabalia there are some merge conflicts to resolve after merging #21354

I will reopen this PR as soon as other depending PR will be merged.


SafeArea seems to be ignored

Hello,
I have custom CropViewPagerenderer which implements the ImageCropper for IOS. It renders a PCL  CropView which is a ContentPage. The CropView class implements the use safe area
On<Xamarin.Forms.PlatformConfiguration.iOS>().SetUseSafeArea(true);
But it seems this is being ignored. Do you know if ImageCropper supports SafeArea for IphoneX?

I've the same issue, any solution?


help requested: postfix-(send|receive)-access.cf not found

I am struggling configuring this image.
I got IMAP access to the server working, using let's encrypt certificates extracted from Traefik. However, mail delivery does not work. This seems to be related to the files 'postfix-send-access.cf' and 'postfix-receive-access.cf' not existing, but I don't understand what I need to change to have them created.
In the container, /var/log/syslog shows:
Feb 17 20:57:51 mail postfix/smtpd[3676]: error: open database /tmp/docker-mailserver/postfix-send-access.cf: No such file or directory
Feb 17 20:57:51 mail postfix/smtpd[3676]: error: open database /tmp/docker-mailserver/postfix-receive-access.cf: No such file or directory

An except of the messages during container startup with the debug option turned on is attached (domain.tld of course is my own domain).
container_messages.txt
Thanks for your help

see https://github.com/tomav/docker-mailserver/issues/844#issuecomment-366208862
PR is waiting for being accepted

Thats solved it, thanks!

Should be fixed in: #846


[SPARK-13015][Docs] Replace example code in mllib-data-types.md using include_example

Replace example code in mllib-data-types.md using include_example
https://issues.apache.org/jira/browse/SPARK-13015

Can one of the admins verify this patch?

ok to test

cc @yinxusen

Test build #51142 has started for PR 11128 at commit d580fe7.

Test build #51142 has finished for PR 11128 at commit d580fe7.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

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

Merged build finished. Test FAILed.

Test build #51160 has started for PR 11128 at commit 33cbe1b.

Test build #51160 has finished for PR 11128 at commit 33cbe1b.

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/51160/
Test PASSed.

Merged build finished. Test FAILed.

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

Test build #51600 has started for PR 11128 at commit c6280c2.

Test build #51600 has finished for PR 11128 at commit c6280c2.

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/51600/
Test PASSed.

Test build #51647 has started for PR 11128 at commit 6c454c7.

Test build #51647 has finished for PR 11128 at commit 6c454c7.

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/51647/
Test PASSed.

Test build #51797 has started for PR 11128 at commit a1b57f5.

Test build #51797 has finished for PR 11128 at commit a1b57f5.

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/51797/
Test PASSed.

Test build #52536 has started for PR 11128 at commit 02b4e22.

Test build #52536 has finished for PR 11128 at commit 02b4e22.

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/52536/
Test PASSed.

@keypointt Examples in the PR are all small ones. I am not sure whether we merge them or not. Either we use this PR https://github.com/apache/spark/pull/9430 to rewrite it, or we just leave this markdown file unchanged. @mengxr What do you think about it?

What's the resolution here -- just leave these as-is?

@keypointt How about we close this for now? There are too many small example codes. If we need all example codes out of markdown files, we can reopen it then. Thanks so much and sorry about that!

@yinxusen sure, no problem. I'm closing it.


Conflict with horizon

Laravel-mongodb Version: #.#.#
PHP Version: 8.3
Database Driver & Version: latest horizon, latest laravel-mongodb, latest laravel

Description:
Your service provider conflicts with horizon:
https://github.com/laravel/horizon/issues/1476
Works only if I comment the BatchRepository::class in MongoDBBusServiceProvider.php

Hi @gdluck, neither this ticket nor the linked one contain any steps for us to reproduce, and from the error message I can't see how this package resulted in this. Please add steps to reproduce and include the full exception details, including the stack trace, in your report. Thank you.

If you are not using MongoDB Batches, downgrading to version 4.3.1 should resolve the issue. I faced a similar problem, and this downgrade worked for me.

@BoatFazWaz We'd like you to be able to update the package, but to do so we need precise information about the problem. Ideally a small Laravel project that reproduces the issue, so we can work on it.

The way you're extending the BatchRepository provider is causing some sort of double-resolving issue.

PHPORM-222

Could you config this fix is correct: https://github.com/mongodb/laravel-mongodb/pull/3071

The immediate solution it to register the service in your application. If you use MongoDB for queue:
$this->app->singleton(BatchRepository::class, function (Container $app) {
    return $app->make(MongoBatchRepository::class);
});

Or if you use the database driver, copy from BusServiceProvider.

Fixed in version 4.7.1 by https://github.com/mongodb/laravel-mongodb/pull/3071


Add a setting that restricts the size of the .swp file

When I open large files, I would prefer not to create .swp files. Is there a way in Vim to limit the creation of .swp files based on their size?

you should be able to do this using some autocommands, that check the size of the file and disable swapfile setting.

Thank you. I will try to deal with autocommands.


production readiness

If I understand correctly, Chrome will remove AppCache support on/around Apr 28 (https://www.chromestatus.com/feature/6192449487634432, https://www.chromestatus.com/features/schedule).
My team uses AppCache in production for reasons explained here:

WKWebView does not support service workers but it does support AppCache. Chrome on Android also supports AppCache (currently). If AppCache support is removed in Chrome then the suggested alternative, service workers, is not viable for a cross-platform web app that runs inside of a WebView (e.g. via react-native-web-view on iOS and Android).

The README for these packages currently state:

This project is still a work in progress. Please experiment with it on a non-production version of your site, and open issues with feedback or bug reports if you run in to problems.

Is there a goal for this AppCache polyfill to be production ready before AppCache is removed from Chrome?

While I believe that this polyfill is functional and complies with the AppCache specification, that language is in place because we have not heard many examples of this polyfill being used "in the wild".
I'd very much appreciate feedback based on your real-world usage. But the (perhaps obvious) recommendation is to try this polyfill out on a local copy of your project first to confirm that there are no edge cases, before deploying it widely to production.

Is there a way to test on an Android emulator/phone with an Android WebView that does not have AppCache in it? I've never tried getting a dev channel Android WebView before…

In Chrome, once there's a service worker registered and a page is loaded in scope of that service worker, AppCache is disabled. (I.e. the service worker always "wins", even if there's still a manifest attribute on the <html>.)
My assumption is that would also apply to Android WebView, but it's not something I've personally confirmed. If you're seeing otherwise, we can escalate that as a bug with WebView.

I see. Thank you for the reply! I am out on vacation this week but next week when I'm back at work I will see if we can test this out, etc. Thank you for building this! It looks very promising to satisfy our current use case. We can leave this issue open if you'd like and until we report back our findings or close this and if we find issues we can open new ones. Whichever you'd prefer. Thank you again.

I'd love to leave this open and track what I hope will be a positive affirmation that the polyfill will meet your needs.

the app I need to get this working in is a react-app, I'm not sure yet how to get webpack to expose the mjs and umd.js files yet… any plans on making any demo/example apps? one for create-react-app would be awesome

You can add
<script type="module">
  import {init} from '/path/to/appcache-polyfill-window/build/index.mjs';
  init().then(() => navigator.serviceWorker.register('sw.js'));
</script>

to the HTML that's output by React. The exact path to use would depend on where you put your appcache-polyfill-window. You could, for instance, use the public folder in create-react-app and store your index.mjs there.
Alternatively, if you would rather bundle appcache-polyfill-window along with your main webpack output that create-react-app manages, you should be able to
npm install --save-dev appcache-polyfill-window

and then in your index.js (or some other JS file):
import {init} from 'appcache-polyfill-window';

init();

I am trying this out. One thing that I still have a question about is the importScripts statement in sw.js… I need a path to the appcache-polyfill-sw/index.umd.js which I either need to manually copy it there (not ideal), write a prebuild script to copy it there (better but still not ideal), or find a way to tell webpack to put it somewhere… not sure yet. Thank you for your help.
FYI: I did copy these polyfill files manually to my public folder so that I could test them out and things seem to be working so far. We have several more things to test so I will keep you posted.
On a related note, should I expect to see any logs from these polyfill scripts or is silence a good sign? 😄

ah, I think I can use copy-webpack-plugin to copy from node_modules to build…

may have found an issue: we have an auto-reload feature in our app that invokes applicationCache.update() periodically; when it detects applicationCache.UPDATEREADY it calls applicationCache.swapCache() and then calls location.reload(). This works fine in WKWebView on iOS and works with Chrome's current native AppCache support.
Might the cache swapping with this polyfill be asynchronous or something and we need to wait to reload?

After digging into this a bit more it looks like appcache-polyfill-window doesn't define or override window.applicationCache, window.applicationCache.status, window.applicationCache.update(), window.applicationCache.swapCache(), window.applicationCache.UPDATEREADY, window.applicationCache.onupdateready, window.applicationCache.addEventListener, etc.
The offline cache population, updating, and usage seems to work great from what I can tell. The only missing piece I'm aware of is defining/overriding the window.applicationCache object itself.

Hello @mfulton26—you're correct in that this polyfill does not attempt to replicate the window.applicationCache interface, nor does it attempt to fire the associated events.
I've updated the README to make that clear.
Some background on that decision: the idea behind this polyfill is to ease the transition from AppCache to service workers. Once using a service worker, a great deal of additional flexibility becomes possible. That flexibility means that, as you make changes to your service worker code to accomplish things like using custom caching strategies, you lose the ability to map what your arbitrary service worker code does into the old model of AppCache events/methods.
In other words, replicating the AppCache caching/serving experience inside of a service worker is meant as a starting point, allowing you to then customize things further and get a richer offline experience.
With that in mind, what we might be able to do is publish some "recipes" in the docs for this repo, along the lines of "if you used to do xyz using the window. applicationCache interface, then something (roughly) along these lines inside of your service worker would be equivalent, assuming you are mostly just using the polyfill code inside your service worker. Some of those recipes would likely need to be based on using postMessage() to communicate between the window and service worker.
It sounds like your auto-reload feature might be the first candidate for such a recipe.

@jeffposnick I think for our needs we just need a way to get a callback/event for when the cache was updated so that we can trigger a page reload
any ideas on how we might expose that?
thank you

Okay, that's actually a little more straightforward.
The cache is populated in the window context, at https://github.com/GoogleChromeLabs/sw-appcache-behavior/blob/ffd37ade55bb1bd2a6e2c733e9c6b603cf542c2b/packages/appcache-polyfill-window/index.ts#L66-L69
Something I could do is modify the init() method in appcache-polyfill-window so that it accepts a callback function, like
<script type="module">
  import {init} from '/path/to/appcache-polyfill-window/build/index.mjs';

  function myCachePopulatedCallback(urls) {
    // Your logic goes here.
  }

  init({
    cachePopulatedCallback: myCachePopulatedCallback,
  }).then(() => navigator.serviceWorker.register('sw.js'));
</script>

And then appcache-polyfill-window would call your callback function whenever all of the cache.put(url, response) operations for a given manifest version have completed, passing in the array of urls that was just added to the cache.
Would that be enough for your use case? Please note that with this approach, your callback would be invoked unconditionally with all of the URLs that are added to the cache, without any additional context about whether it's an update to a previously cached entry or anything like that.

That sounds good. If I wanted to periodically invoke an "update" I would just call init(), is that right? If nothing gets updated then the callback would not be invoked but if there are updates then once they are all updated the callback would be invoked with a list of URLs that were updated.

Although I didn't anticipate that folks would call init() multiple times within the lifetime of a page load, calling it again should kick off a manifest re-download. Then yes, it would trigger the proposed new callback function following a cache.put().
I can put out a new release of this library with support for that callback over the next few days if you want to try it out.

That would be perfect. Thank you.

Any update on this? I've got a change mostly ready on my end to incorporate this polyfill but it breaks our auto-update feature. It'd be great to get a way to make that work and finish this up. Thanks!

Apologies for the delay.
I've just published v0.2.0 of the libraries to npm, which includes this functionality. There's some refactoring that I still want to do, along with some test cases, but I figured I shouldn't delay in releasing something you could test out and give feedback on.
Let me know whether it behaves as you'd expect:
<script type="module">
  import {init} from '/path/to/appcache-polyfill-window/build/index.mjs';

  function myCachePopulatedCallback(urls) {
    // Your logic goes here.
  }

  init({
    cachePopulatedCallback: myCachePopulatedCallback,
  }).then(() => navigator.serviceWorker.register('sw.js'));
</script>

...just wanted to confirm (wasn't clear from the 👍 emoji) that v0.2.0 provides the functionality that you're looking for, @mfulton26.

Right, we're still not using this in production completely but from my local testing I believe the answer is yes. Part of me wishes I could polyfill all the events on ApplicationCache and not just update but that's all I really truly need. I think we can probably close this issue and if we identify any issues going forward we can open separate issues then. Thank you so much for providing this polyfill and providing the support needed. This has been great!

Glad to hear it!


Get mcsema building on macOS again

IDA is expensive, and many people purchase it for macOS only. If we can get mcsema building on macOS, then people with IDA licenses on that platform can also use it.

Todo:

[ ] Get it building again
[ ] Tests passing
[ ] Travis support
[ ] Make it pretty

You'll probably need some variant of the ELF runtimes to make this work.


Tests fail on 32 bit linux

Tests are failing on debian 32 bit using Rust 1.7
From what I can tell the issue seems to crop up when calling into C.

Sorry, I couldn't find a 32-bit machine, or even a working VM, to test it.
I've tested it on 64-bit Linux, and it passes tests there.
It's possible that there's some mismatch between C int/long and Rust sizes that causes problems.

Once patched, all tests pass on ARM presently.

Confirmed with 32bit Arch Linux,
output from valgrind may help: valgrind.txt
Rebuilt with debug cflags:
diff --git a/Makefile b/Makefile
index 49c6d69..f708209 100644
--- a/Makefile
+++ b/Makefile
@@ -1,4 +1,4 @@
-CFLAGS ?= -O3 -fPIC
+CFLAGS ?= -O0 -ggdb3 -fno-inline -fPIC
 OUT_DIR ?= .
 VENDOR_DIR ?= vendor
 LIB = $(OUT_DIR)/liblodepng.a

Likely fixed in 2.0


Create desktop application

The desktop application needs to serve as an all-purpose GUI for interactions between the user and the device, so that non-programmers can reprogram and calibrate and customize its paramteres.

[ ] provide port connection feedback
[ ] provide Bluetooth connection feedback
[ ] provide in-app OLED monitor data display
[ ] settings

Potential extra features:

[ ] provide real-time digital view of em-button
[ ] interactions could be done via clicking on different sections of the model
[ ] allow for command-line serial input with base STP1.0 protocol request creation

A bunch of work has been done on the desktop app without versioning. See commit https://github.com/jfletcher20/em-button/commit/3ee8b1fa63339905e7085bd154cb645d922497c8.

[ ] provide calibration option

I'm making a note here: huge success.
There are some minor improvements that can yet be made. Such as

ensuring the buttons for settings, calibration and to restart the device are disabled when no port is found
ensuring that if no data loads for the digital OLED monitor, it sends another request
updating the model that's in the background to the latest model version
loading actions properly into the actions editing form - this is something I tried to accomplish for some time but didn't manage; for some reason it just won't read the data properly or something

Finally, the UI could probably have been done better, but because of design responsiveness I decided to keep it like this for now. It's only a demo, after all.
But these are all just minor quality-of-life improvements and are not central to the thesis. The desktop application is simply a proof-of-concept on how to interact with the em-button through its API, and as far as I can tell: it's a great success.
The app works better than I expected it to.


Adjustments for running concurrent upstream rpm packaging jobs

still debugging this a bit..

+1

FWIW, behaviour with -e can be surprisingly hard to predict, e.g.

http://mywiki.wooledge.org/BashFAQ/105
http://fvue.nl/wiki/Bash:_Error_handling


No payload message if not using Auto IP

Hello,
I have a weird problem. When I use "Auto" method for IP discovery, everything works great. But when I set up the IP address of the device (which has been discovered with wizard), out of nothing I get "No Payload" message.
devices = {
        "bfc47831e6547fdXXXXX" : {"ip":"<IP_ADDRESS>", "key":"XXXXXX"}
}

        return tinytuya.BulbDevice(
                        dev_id = id,
                        address = devices[id]["ip"],
                        local_key = devices[id]["key"],
                        connection_retry_delay=1,
                        version=3.3)

DEBUG:Error decoding received data - read retry 5/5
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/dist-packages/tinytuya/core.py", line 1194, in _send_receive
    rmsg = self._receive()
           ^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/tinytuya/core.py", line 1082, in _receive
    data = self._recv_all( min_len )
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/tinytuya/core.py", line 1062, in _recv_all
    raise DecodeError('No data received - connection closed?')
tinytuya.core.DecodeError: No data received - connection closed?
DEBUG:ERROR Unexpected Payload from Device - 904 - payload: null
DEBUG:status() received data={'Error': 'Unexpected Payload from Device', 'Err': '904', 'Payload': None}
DEBUG:Status request returned an error, is version 3.3 and local key XXXXXX correct?

Any ideas would be appreciated!
Thank you

OK, my bad, just bad version :(


Basic functionality

Layout the overall architecture and rough in the steps of this minor ETL pattern with Toggl and a local relational database. Use PostgreSQL first.


Cannot build APK

Hello everyone,
I am new to Android Studio but I managed it to make app work in emulator. However, I am unable to make APK. I got this message:
FAILURE: Build completed with 2 failures.
1: Task failed with an exception.

Where:
Script 'C:\src\flutter\packages\flutter_tools\gradle\flutter.gradle' line: 1159

What went wrong:
Execution failed for task ':app:compileFlutterBuildRelease'.

Process 'command 'C:\src\flutter\bin\flutter.bat'' finished with non-zero exit value 1

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.

What went wrong:
java.lang.StackOverflowError (no error message)

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
==============================================================================

Get more help at https://help.gradle.org

BUILD FAILED in 1m 5s
Running Gradle task 'assembleRelease'...                           67,4s
Gradle task assembleRelease failed with exit code 1
Process finished with exit code 1

I got the same problem.Did you solve it?

Always failure with Resolving dependencies... (1.9s)
Because every version of flutter_localizations from sdk depends on intl 0.18.0 and thingsboard_app depends on intl
^0.17.0, flutter_localizations from sdk is forbidden.
So, because thingsboard_app depends on flutter_localizations from sdk, version solving failed. What can I do?

@song-df which version of Flutter do you use?
flutter --version


NoHassleAV 8x8 Seamless HDMI Matrix

I'd love to see support for this unit - https://www.nohassleav.com/support/matrix_hdmi_8x8_4k_seamless/
They have a series of hex commands that can be sent to change the inputs and recall scenes.

@SkylerJEhly Could you please clarify which matrices are currently compatible?
Does the 8x8 Seamless HDMI Matrix work with the module?

I actually never submitted my solution because I was never able to test it with akurcan. I believe @josephdadams developed the module but it is completely untested. The unit that this module was developed for was http://www.nohassleaudiovideo.com/support/matrix_hdmi_8x8_4k_seamless/ which I'm guessing is your device?

I actually never submitted my solution because I was never able to test it with akurcan. I believe @josephdadams developed the module but it is completely untested. The unit that this module was developed for was http://www.nohassleaudiovideo.com/support/matrix_hdmi_8x8_4k_seamless/ which I'm guessing is your device?

Should it then also work with the 4x4 version?
https://nohassleav.com/products/4x4-seamless-hdmi-4k-hdr-matrix-switcher-18gbps-ultra-yuv-4-4-4-hdcp2-2-60hz-hdmi-2-0b-atmos-hdtv-routing-selector-spdif-audio-control4-savant-home-automation-switch-ip-rs232

ah @SkylerJEhly my apologies - I never saw the reply come through! I'd love to test your solution. Sending you my email now.

I actually never submitted my solution because I was never able to test it with akurcan. I believe @josephdadams developed the module but it is completely untested. The unit that this module was developed for was http://www.nohassleaudiovideo.com/support/matrix_hdmi_8x8_4k_seamless/ which I'm guessing is your device?

Should it then also work with the 4x4 version?
https://nohassleav.com/products/4x4-seamless-hdmi-4k-hdr-matrix-switcher-18gbps-ultra-yuv-4-4-4-hdcp2-2-60hz-hdmi-2-0b-atmos-hdtv-routing-selector-spdif-audio-control4-savant-home-automation-switch-ip-rs232

Can you please provide the IP commands for this device? I cannot locate them myself. They should be in your included manual.

ah @SkylerJEhly my apologies - I never saw the reply come through! I'd love to test your solution. Sending you my email now.

At this point I recommend using the current version that is with companion which was developed by joseph and then report any newissues through github. From there we can make bug fixes to the current release.

I actually never submitted my solution because I was never able to test it with akurcan. I believe @josephdadams developed the module but it is completely untested. The unit that this module was developed for was http://www.nohassleaudiovideo.com/support/matrix_hdmi_8x8_4k_seamless/ which I'm guessing is your device?

Should it then also work with the 4x4 version?
https://nohassleav.com/products/4x4-seamless-hdmi-4k-hdr-matrix-switcher-18gbps-ultra-yuv-4-4-4-hdcp2-2-60hz-hdmi-2-0b-atmos-hdtv-routing-selector-spdif-audio-control4-savant-home-automation-switch-ip-rs232

Can you please provide the IP commands for this device? I cannot locate them myself. They should be in your included manual.

I don't have the unit yet. Wanted to make sure it would work before purchasing. I did send the company an email requesting the manual. Will let you know when they reply.

No Hassle AV support says that the 4x4 seamless version has identical controls of the 8x8 version (http://www.nohassleaudiovideo.com/support/matrix_hdmi_8x8_4k_seamless/)

Are there any instructions provided for configuration? For example, there is no field in the module to set the pw.


Fix addAt function in LinkedList class

Please add a description to your ticket.
Peek does not remove the element.
If addAt is incorrect, add a test.

I did not want to add that commit with Russian commentaries, I'll soon redo PR

Alright 👍
If you add a Test to LinkedListTests.wurst for addAt I will merge.

great 🍭


Cropped UI pages for Profile and New Channel

Your Rocket.Chat.Android app version: 2.5.0

Your Rocket.Chat server version: your demo server

Device model (or emulator): OnePlus 5 - Android 7.1.1
also tried on  Umidigi Max - Android 7.0

Steps to reproduce:
Phone locked or not to portrait,
access + next to channel search - observe that UI is off the screen to the right.

This issue regards to the Cordova app and not the native Android app. Please reopen this issue on the respective repository if you feel this issue is still happening to you.


How to train custom dataset?

Could you please give an example?

Hi,
using a custom dataset should be quite straight forward. You have to format your dataset as a JSON with list entries having a context, label, and image id field (look at the scripts used to generate the data like this) and then adjust the datamodule config to use your file.
Hope that helps. Let me know if you have more specific questions.


Publish 1.0 on google play

Sent APK.
now the ball is at their side

https://play.google.com/store/apps/details?id=com.github.braillesystems.learnbraille&hl=ru


Added release note for change relating to invalid cookies

This was previously found in Django when formtools was a contrib app:
https://github.com/django/django/commit/6302893112a67343b3f5eff3f09f91b245346877#diff-ea4f460f2becc51cd39d7587ec3f3244L128

Ship it, thanks!


docs(tabs): add steps to switch tabs from child component

Short description of what this resolves:
Docs lack information on how to programmatically switch tabs from child component
Changes proposed in this pull request:

update docs to show NavController.parent.select()

Ionic Version: 2.0-RC1

Thanks for the PR! merged


Ionic ngc failed with ionic build android --prod

Below is Error and tsconfig.

ionic-app-scripts build "--prod"

[16:05:41]  ionic-app-scripts 1.3.4
[16:05:41]  build prod started ...
[16:05:41]  clean started ...
[16:05:41]  clean finished in less than 1 ms
[16:05:41]  copy started ...
[16:05:41]  ngc started ...
[16:05:43]  ionic-app-script task: "build"
[16:05:43]  TypeError: Cannot read property 'codeGen' of undefined
TypeError: Cannot read property 'codeGen' of undefined
at Object.doCodegen (F:\Councils-Code\node_modules@ionic\app-scripts\dist\a
ot\codegen.js:6:50)
at<EMAIL_ADDRESS>:42:30
npm ERR! Windows_NT 6.1.7601
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\
node_modules\npm\bin\npm-cli.js" "run" "ionic:build" "--" "--prod"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR! councils@ ionic:build: ionic-app-scripts build "--prod"
npm ERR! Exit status 1
npm ERR!
TsConfig
{
"compilerOptions": {
"allowSyntheticDefaultImports": true,
"declaration": false,
"emitDecoratorMetadata": true,
"experimentalDecorators": true,
"lib": [
"dom",
"es2015"
],
"module": "es2015",
"moduleResolution": "node",
"sourceMap": true,
"target": "es5"
},
"include": [
"src/**/*.ts"
],
"exclude": [
"node_modules"
],
"compileOnSave": false,
"atom": {
"rewriteTsconfig": false
}
}
Ionic Info:
ordova CLI: 6.5.0
Ionic Framework Version: 2.0.0
Ionic CLI Version: 2.2.2
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.3.4
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 7
Node Version: v6.9.5
Xcode version: Not installed

Hi - how can I recreate this issue?
Thanks,
Dan

I have same issue.
@danbucholtz , you can reproduce this issue by
ionic build --prod

I have the same issue

I have the same issue
:(

I have the same issue. Here mey ionic info output:
Your system information:
Cordova CLI: 6.5.0
Ionic Framework Version: 2.0.0
Ionic CLI Version: 2.2.3
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.3.4
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 7
Node Version: v6.10.0
Xcode version: Not installed

I have the same issue:
Cordova CLI: 6.5.0
Ionic Framework Version: 2.0.0
Ionic CLI Version: 2.2.2
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.3.2
ios-deploy version: 1.9.1
ios-sim version: 5.0.13
OS: macOS Sierra
Node Version: v6.10.2
Xcode version: Xcode 8.3.1 Build version 8E1000a

I have the same issue by running ionic build android --prod --release

same issue
ordova CLI: 7.0.0
Ionic Framework Version: 2.0.0-rc.4
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.3.7
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.10.3

I'm also getting this error right now...

@brunosfreitas Update your native plugins, that solved for me, I had this same problem yesterday


[SPARK-42861][SQL] Use private[sql] instead of protected[sql] to avoid generating API doc

What changes were proposed in this pull request?

This is the only issue I found during SQL module API auditing via https://github.com/apache/spark-website/pull/443/commits/615986022c573aedaff8d2b917a0d2d9dc2b67ef . Somehow protected[sql] also generates API doc which is unexpected. private[sql] solves the problem and I generated doc locally to verify it.
Another API issue has been fixed by https://github.com/apache/spark/pull/40499
Why are the changes needed?

fix api doc
Does this PR introduce any user-facing change?

no
How was this patch tested?

N/A

@xinrong-meng

thanks, merged to master and 3.4


yew-router/readme.md is outdated. Should it be simplified?

This is about:

[ ] A typo
[ ] Innaccurate/misleading documentation (e.g. technically incorrect advice)
[ ] Undocumented code
[x] Outdated documentation
[ ] Other

Problem
The code example in the readme is outdated. Also, the released version has changed to 0.15 and 0.18 for yew-router and yew respectively and it remains 0.14 and 0.17 in the readme.
There are already detailed code examples updated on yew.rs, should we remove the code example and leave a reference to the corresponding section on yew.rs? -> https://yew.rs/next/concepts/router. This way we can avoid editing in two places in the future.

There are already detailed code examples updated on yew.rs, should we remove the code example and leave a reference to the corresponding section on yew.rs? -> https://yew.rs/next/concepts/router. This way we can avoid editing in two places in the future.

I agree. We should keep one place up to date and link to it from the other places


Automatic load by ID middleware

Make it automatic w/ a naming convention. If the message has a public property with [Entity Name]Id and the handler takes in that [Entity], apply the loading middleware.
Maybe also have a more explicit model. Honor [WolverineIgnore] on properties too.
Also maybe an explicit Load/LoadAsync() convention to connect things as well

Kicking around several ideas. Thinking about a policy/extension method like:
WolverineOptions.Handlers.LoadDocument<Account>()
    .FromMarten()

     // Or

      .FromEfCore() // this would be able to figure out the correct DbContext in the case of there being multiple types registered

      .OnMissing( // not exactly sure what the options should be, but either "log and abort the message" or throw an exception )

       // This is where it maybe gets weirder. Minus this explicit way to pick out how to relate the incoming message to the Account entity, you use a magic naming convention where you look for `AccountId` ([entity name]Id) on the incoming message
      .Using<IAccountMessage>(x => x.AccountId);


impl + attribute + comment ⇒ missing whatever was before the comment

rustfmt currently produce invalid code on one of Clippy‘s file.
I've been able to reduce it to this:
impl LateLintPass for UsedUnderscoreBinding {
    #[cfg_attr(rustfmt, rustfmt_skip)]
    fn check_expr() { // FIXME: #944
    }
}

which produces:
3c3,4
<     fn check_expr() { // FIXME: #944
---
>
>     // FIXME: #944

I suspect we're passing skipped parts through the missed spans machinery, whereas we should probably just take the associated string verbatim.


Feature/scheduler activity via joblistener

AMQ-9188
Alternative implementation based on discussion with @mattrpav in #949

@mattrpav Is this approach more along the lines of what you had in mind?

@mattrpav, as per our conversation in #949, reworked JobListener to have the following lifecycle hooks. Thoughts?
Ended up refactoring memory, kahadb, and tests.

willScheduleJob
scheduleJob
didScheduleJob
willDispatchJob
dispatchJob
didDispatchJob

@mattrpav Ping?

@mattrpav ping?:) I know you are swamped. Still hoping for a review :)

Found a way to achieve the same functionality without the need to modify the broker:
https://github.com/maxfortun/activemq-scheduler-store-advisory


rename widgets to miq_widgets

Widgets are called MiqWidget

fix bug introduced in 18b6d28
fix code that protects a report from being deleted
add specs

/cc @h-kataria This should fix your problem

@kbrock can we add spec test for this, or if you want i can try add one, let me know.

looks good, verified fix.


[almalinux] Update latest 8

See https://almalinux.org/blog/2023-11-21-announcing-89-stable/.
Closes #4162.

Simple update, merging.


About the implementation of ge25519_scalarmult_base_niels

Which document does the ge25519_scalarmult_base_niels implementation refer to ? Is it exactly the same as the scalarmult implementation of ed25519( RFC8032 5.1.4 )?

ge25519_scalarmult_base_niels is from ed25519-donna, I think it's the same as scalarmult according to the source code (https://github.com/floodyberry/ed25519-donna/blob/8757bd4cd209cb032853ece0ce413f122eef212c/ed25519.c#L53
).


(Deposit/Withdraw) Add Nomic Deposit Address Flow

What is the purpose of the change:
These changes are behind a feature flag.
Add the Deposit Address UI and update the Nomic Bridge provider to support deposits.
Linear Task
https://linear.app/osmosis/issue/FE-1119/add-nomic-bridge-provider
Testing and Verifying

[ ] Deposit BTC displays the new deposit address screen

It works, but it appears we're now showing network selectors at the top in the dedicated external URLs screen:

Yes, this is intentional. Our goal is to avoid blocking users on a specific chain or asset when no providers are available, especially if there are other supported chains available


specifying a default together with a value converter

Hi Paul,
in the project that I am currently working on I have defined the following ValueConverter
final class SwitchConverter implements ValueConverter<Boolean> {

    @Override
    public Boolean convert(final String value) {
        if ("on".equals(value)) {
            return Boolean.TRUE;
        } else if ("off".equals(value)) {
            return Boolean.FALSE;
        }
        throw new IllegalArgumentException(
                        String.format("String value '%s' must match '%s'.", value, valuePattern()));
    }

    @Override
    public Class<Boolean> valueType() {
        return Boolean.class;
    }

    @Override
    public String valuePattern() {
        return "/(?:on|off)/";
    }

}

The converter is used in the following specification
OptionParser parser = new OptionParser();
ArgumentAcceptingOptionSpec<Boolean> specification = parser.acceptsAll(Arrays.asList("t", "traceIncrements"), "Turns tracing of increment requests 'on' or 'off'.").withRequiredArg().describedAs("trace").withValuesConvertedBy(new SwitchConverter()).defaultsTo(Boolean.FALSE);

I have implemented a HelpFormatter that uses the OptionDescriptor.defaultValues() method. An extract of the help screen looks like this
...
-t trace
Turns tracing of increment requests 'on' or 'off'. It defaults to 'false'.
...
I want the user to use on and off as option argument values and I want to inform the user about the default. The default is displayed in the **SwitchConverter.convert()**ed way. That is wrong.
I don't understand why the default has to be specified as a Boolean (converted) value although value-conversion was specified/enabled. I would have expected that value conversion is also applied to the defaults.
A workaround might be possible if the OptionDescriptor interface would offer a method, e.g. convertedBy(), that returns the Class instance of the ValueConverter if specified. Saying that it seems that the ValueConverter interface needs a reverse() method that does the opposite of the convert() method.
Kind regards
Sven

@XSven This sounds reasonable. OptionDescriptor.convertedBy() and ValueConverter.revert().

@pholser is this issue resolved or still on-going?

@halberd6 As yet unresolved. Sorry for the delay -- day gig intrudes...8^)

@XSven @halberd6 Thinking about the proposed implementation...If I add a method to interface ValueConverter, this breaks all implementers of the interface. That is, until we abandon JDK < 8 (#108).
Would you be willing to wait until work is done to abandon JDK < 8, and then ValueConverter.revert() can have a default implementation so as not to break existing implementers?

@pholser I did a workaround. For ArgumentAcceptingOptionSpec, the OptionDescriptor.argumentTypeIndicator() returns OptionDescriptor.convert.patternValue() if it is not null.

I add static method in SwitchConverter.class.
static String revert(final Boolean value)
While looping OptionDescriptor in our customized HelpFormatter, equals of SwitchConverter.valuePattern() and OptionDescriptor.argumentTypeIndicator() leads to  revert action.
SwitchConverter.revert(OptionDescriptor.defaultValues().get(0))

Although this looks unsafe / ugly, do you have any other suggestions?

@halberd6 @XSven At this point I can't think of a solution that I like better than introducing ValueConverter.revert(). Would you be OK with waiting until abandoning JDK < 8? I don't think it'll take too much effort before I can have a 6.0 alpha build available with this functionality.

@pholser Yes, we can wait.

@halberd6 @XSven Please see the above pull request. Hopefully this will address your concerns!

@pholser According to the changes in the pull request,  the problem we have could be solved. Looking forward to the next release.

@halberd6 @XSven This is in 6.0 alpha 1.