moving and resizing a shape

Hello,
First of all, thank you for creating this package. You've potentially saved me from having to write vba macros for automating Visio processes for my current job.
That being said, I seem to be having trouble accurately moving and resizing shapes on a page using the "height", "width", "x" and "y" attributes.
Here is a sample of what I'm experiencing:
`with VisioFile('test.vsdx') as vis:
    page = vis.get_page_by_name('box')

    box = [x for x in page.all_shapes if x.shape_name == 'box'][0]
    text = [x for x in page.all_shapes if x.shape_name == 'text'][0]

    box.x = 2.5
    box.y = 2
    box.height = 1
    box.width = 3

    text.text = "NOW A RECTANGLE"
    text.x = 5.5
    text.y = 4
    text.height = 0.5
    text.width = 5
    vis.save_vsdx('test_new.vsdx')`

On the second image, I'm expecting the center of the blue rectangle to be updated to (2.5, 2), but instead it appears to be centered somewhere around (2, 0.5).
Is there another way to achieve this?
I'm using Microsoft Visio Professional 2016

Hi @camhilker - there are a number of dependent attributes on a shape, like height/width & loc_x/loc_y (the attributes I think you might need to change) that this package isn't automatically updating as yet. It does expose these so you can try setting loc_x/loc_y on the box/rectangle
There may be some that it doesn't yet expose but I'll be happy to add them. You can see the properties in the shape xml by logging the xml property - it can get a bit more complex where the shape inherits properties from a master shape but that shouldn't be the case here.


PlatformNotSupportedException when trying to generate QR Code.

I have a blazor web assembly project and I am trying to generate QR using this nuget package but it is always throwing PlatformNotSupportedException
Here is my code snippet :
if (_selectedBCard != null)
{
var inviteLink = string.Format(Constant.WebCardUrl, _selectedBCard.BcardId);
            var qrGenerator = new QRCodeGenerator();
            var qrCodeData = qrGenerator.CreateQrCode(inviteLink, QRCodeGenerator.ECCLevel.Q);
            var qrCode = new QRCode(qrCodeData);

            var base64Image = qrCode.GetGraphic(10);
            QRByte = $"data:image/png;base64,{base64Image}";
        }

I don't think that there's any other initialization required for it to work.
Here's the stack trace -
System.PlatformNotSupportedException: System.Drawing.Common is not supported on this platform.
at System.Drawing.Image..ctor()
at System.Drawing.Bitmap..ctor(Int32 width, Int32 height)
at QRCoder.QRCode.GetGraphic(Int32 pixelsPerModule, Color darkColor, Color lightColor, Boolean drawQuietZones)
at QRCoder.QRCode.GetGraphic(Int32 pixelsPerModule)
at company_portal.Pages.BcardsModel.GenerateQRCode() in C:\Users\pa740\source\repos\BusinessCode\company_portal\Pages\Bcards.razor.cs:line 392
at company_portal.Pages.Bcards.GenerateQRCode(CompanyBcardResponse bcard) in C:\Users\pa740\source\repos\BusinessCode\company_portal\Pages\Bcards.razor:line 407
at company_portal.Pages.Bcards.<>c__DisplayClass0_0.b__7() in C:\Users\pa740\source\repos\BusinessCode\company_portal\Pages\Bcards.razor:line 195
at Microsoft.AspNetCore.Components.EventCallbackWorkItem.InvokeAsync[Object](MulticastDelegate delegate, Object arg)
at Microsoft.AspNetCore.Components.EventCallbackWorkItem.InvokeAsync(Object arg)
at Microsoft.AspNetCore.Components.ComponentBase.Microsoft.AspNetCore.Components.IHandleEvent.HandleEventAsync(EventCallbackWorkItem callback, Object arg)
at Microsoft.AspNetCore.Components.EventCallback.InvokeAsync(Object arg)
at Microsoft.AspNetCore.Components.RenderTree.Renderer.DispatchEventAsync(UInt64 eventHandlerId, EventFieldInfo fieldInfo, EventArgs eventArgs)
Type of issue
[ ] Bug
Steps to Reproduce (for bugs)
Copy and paste the above code snippet.
Your Environment

Version used:  1.4.2, 1.4.3
Compiled from source or NuGet package?:  Nuget package
Payload/Encoded text:  Mentioned above
Used ECC-level:  Q
Environment (.NET 3.5, .NET 4.X, .NETCore, ...):  .NET 6

@Awesomer9561 Microsoft removed System.Drawing support in .NET 6 for non-Windows platforms. Since the QRCode renderer uses System.Drawing, it's correct that you see a PlatformNotSupportException. But as @timdinh said - that shouldn't be a problem as there are other rendering classes that are support on non-Windows platforms. Check the compatibility matrix to find a renderer that fits your platform/environment: https://github.com/codebude/QRCoder/wiki/Advanced-usage---QR-Code-renderers#2-overview-of-the-different-renderers


ASTVerifier: make sure isFinal and FinalAttr are in sync

Serialization is currently working around them not having been in sync in the past, but they are now, and we might as well start depending on it (see #26735).

@swift-ci Please test

@swift-ci Please test source compatibility

@jrose-apple The change looks fine but I think you linked to the wrong PR?

Yeah, fixed.

@swift-ci Please smoke test

@swift-ci Please smoke test Linux

Oh, bleah, now I've pulled in other commits by accident.

@swift-ci Please smoke test

@swift-ci Please smoke test

This may have caused a regression with the windows build: https://compnerd.visualstudio.com/windows-swift/_build/results?buildId=6815&view=logs&j=2d2b3007-3c5c-5840-9bb0-2b1ea49925f3&t=2915bb0c-b896-5031-2bc6-f61128a41015&l=1460

decl should be final iff it has FinalAttr, but isFinal() = 1 and hasAttribute<FinalAttr>() = 1

…what. Both of these are bool, no less, so they can't even be different values getting coerced to the wrong thing.

I can't explain the bool angle, but @slavapestov refactored coroutine accessor synthesis two weeks ago, and I don't see anything there that would have copied a final attribute to the modify accessor.

setFinal itself does the attribute copy now, so I don't see how we would get this wrong. It does seem to be consistent on the Windows bot, though.

@compnerd, as the resident Windows expert, can you see anything wrong with this check?

@jrose-apple Maybe it's the unspecified order of evaluation in C++. isFinal() will add a FinalAttr if there isn't one already, but if you evaluate hasAttribute<FinalAttr>() first, you'll get true != false. By the time you're printing the result, hasAttribute<FinalAttr>() will be true, so you get the paradoxical output.

Ohh, that makes a lot of sense.
…but then doesn't really explain what we should do in Serialization to ensure that a FinalAttr gets serialized. Should we just call isFinal spuriously? cc @slavapestov

@jrose-apple I think we should serialize the 'final' bit as part of the DeclLayout directly, instead of using the presence of the attribute to indicate that the request has been evaluated and produced a 'true' result.

That'd be fair too, but it's not just about final. What are we doing for dynamic, for example?

@brentdax in general we don't want to manually "copy" stuff over to a new synthesized decl. The existing IsFinalRequest already has a rule that accessors are final when the storage is final, and this should be enough for both parsed and synthesize accessors.
@jrose-apple Probably the same thing we're doing for @objc and final, which is wrong. We should be serializing cached requests as their own thing instead of pretending the attribute should always be in sync.

Okay, but what's the general rule that'll keep us from messing this up in the future?

... Actually I see isObjC() is already serialized separately from the others, and ValueDecl::setIsObjC() doesn't synthesize the implicit attr.

I think the general rule is that the only code that should care about the attribute is the request computation, and the cached results of requests need to be serialized directly.

For example, for final previously we hacked the attribute on if it wasn't there already (I just removed that), and for dynamic I guess we just assumed it had been type-checked, which isn't totally unreasonable for the primary file since we need to diagnose things about all of these.

@jrose-apple If we serializing decl->isFinal(), decl->isDynamic(), decl->isObjC() and so on directly, then we'll just run those requests at serialization time which is reasonable. IsFinalRequest::cacheResult() should not add the attribute at all ideally. We can still serialize the final attribute but then it's just a sugar thing.


ED* implementation plan

A rough set of features/commands (most of them have synonyms, in case we do several commands or need a replacement):

[ ] Edit s-expressions structurally:

[ ] Act on expressions

[ ] Move into, step in, descend
[ ] Move out, pop, ascend
[ ] Next, forward
[ ] Previous, back(wards)

[ ] Paredit

[ ] Insert parens
[ ] Wrap
[ ] Slurp, widen, conquer, devour
[ ] Barf, shorten

[ ] Unwrap—the same?

[ ] Modify expressions

[ ] Insert, prepend, prefix
[ ] Append, suffix, annex, affix, postfix
[ ] Change, modify, rewrite, replace
[ ] Delete, cut, remove, clear, kill
[ ] Join, append?, concatenate

[ ] Meta

[ ] Copy, save
[ ] Search, find, grep
[ ] Substitute, replace
[ ] Paste, yank
[ ] Undo, cancel, stop
[ ] Redo

[ ] Evaluating, testing etc.

[ ] Evaluate, run, execute
[ ] Print, (re)display, self
[ ] Save, dump to the file
[ ] Read, parse, fetch from the file

[ ] Edit Lisp files as a set of s-expressions.

[ ] Editing strings is an UNIX ed-ish way.

Not really the best interface, but it's well suited for textual interaction

[ ] Editing non-Lisp files with this string editing modality.

[ ] Or defining custom processors for file types like HTML.

[ ] Maybe even edit arbitrary Lisp objects?

Kinda the same as inspect*, so not sure.

Which means ed* should dispatch at least over:

cons.

Maybe array too, which compounds to sequence.

string.
pathname.
And, with potential Lisp object editing, t.

CC @fstamour

Where does ed* stops and inspect* begins?

For reference:

http://www.lispworks.com/documentation/HyperSpec/Body/f_ed.htm
sample-implementation-results/unix-ed-cheat-sheet.txt

Where does ed* stops and inspect* begins (if inspect* can also edit)?

inspect* can modify certain parts of inspected object (usually, object slots). Not sure where to delineate...

I've expanded the list of commands. Not definitive, needs review and critical assessment. More synonyms won't hurt either.


Version bump 0.2.1

r? @ustulation
(maidsafe_highfive has picked a reviewer for you, use r? to override)

r? @dirvine

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Rotate Camera

Hi There,
Thanks for this great script, is there any way to rotate the camera. I am using an RPI4 and need to rotate the camera 90 deg to the right as it captures sideways.
Thanks for any assistance.

Hey! Apologies for the lateness. I'll try look into this at the weekend.  Let me know if you have any further info/suggestions

Hi There,
Thanks for this great script, is there any way to rotate the camera. I am using an RPI4 and need to rotate the camera 90 deg to the right as it captures sideways.
Thanks for any assistance.

This has been included now as an additional OPTIONAL command line argument, you can use 0, 90, 180 or 270 or leave it out.
Have a look


TypeError in windows / msvc.py when VCRuntimeRedist is not present

User is reporting the below stack trace on windows.  This would indicate that the method VCRuntimeRedist at https://github.com/pypa/setuptools/blob/master/setuptools/msvc.py#L1540 is returning None and the method at https://github.com/pypa/setuptools/blob/master/setuptools/msvc.py#L1574 is nonetheless assuming a non-None result.
It's very hard to find this error elsewhere but I did see it reported (and mis-diagnosed) here: https://stackoverflow.com/questions/58677600/unable-to-install-jupyterlab-for-python3-8-64bit
 Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "C:\Users\masma\AppData\Local\Temp\pip-install-rhpx0g7l\sqlalchemy\setup.py", line 217, in <module>
        run_setup(True)
      File "C:\Users\masma\AppData\Local\Temp\pip-install-rhpx0g7l\sqlalchemy\setup.py", line 196, in run_setup
        **kwargs
      File "c:\users\masma\source\repos\orsteddcsmarketsanalytics\apps\portfolio_analytics\tmp\.venv\lib\site-packages\setuptools\__init__.py", line 145, in setup
        return distutils.core.setup(**attrs)
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\core.py", line 148, in setup
        dist.run_commands()
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\dist.py", line 966, in run_commands
        self.run_command(cmd)
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\dist.py", line 985, in run_command
        cmd_obj.run()
      File "c:\users\masma\source\repos\orsteddcsmarketsanalytics\apps\portfolio_analytics\tmp\.venv\lib\site-packages\setuptools\command\install.py", line 61, in run
        return orig.install.run(self)
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\command\install.py", line 545, in run
        self.run_command('build')
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\cmd.py", line 313, in run_command
        self.distribution.run_command(command)
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\dist.py", line 985, in run_command
        cmd_obj.run()
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\command\build.py", line 135, in run
        self.run_command(cmd_name)
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\cmd.py", line 313, in run_command
        self.distribution.run_command(command)
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\dist.py", line 985, in run_command
        cmd_obj.run()
      File "C:\Users\masma\AppData\Local\Temp\pip-install-rhpx0g7l\sqlalchemy\setup.py", line 54, in run
        build_ext.run(self)
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\command\build_ext.py", line 339, in run
        self.build_extensions()
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\command\build_ext.py", line 448, in build_extensions
        self._build_extensions_serial()
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\command\build_ext.py", line 473, in _build_extensions_serial
        self.build_extension(ext)
      File "C:\Users\masma\AppData\Local\Temp\pip-install-rhpx0g7l\sqlalchemy\setup.py", line 60, in build_extension
        build_ext.build_extension(self, ext)
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\command\build_ext.py", line 533, in build_extension
        depends=ext.depends)
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\_msvccompiler.py", line 345, in compile
        self.initialize()
      File "c:\users\masma\appdata\local\programs\python\python37\Lib\distutils\_msvccompiler.py", line 238, in initialize
        vc_env = _get_vc_env(plat_spec)
      File "c:\users\masma\source\repos\orsteddcsmarketsanalytics\apps\portfolio_analytics\tmp\.venv\lib\site-packages\setuptools\msvc.py", line 171, in msvc14_get_vc_env
        return EnvironmentInfo(plat_spec, vc_min_ver=14.0).return_env()
      File "c:\users\masma\source\repos\orsteddcsmarketsanalytics\apps\portfolio_analytics\tmp\.venv\lib\site-packages\setuptools\msvc.py", line 1620, in return_env
        if self.vs_ver >= 14 and isfile(self.VCRuntimeRedist):
      File "c:\users\masma\source\repos\orsteddcsmarketsanalytics\apps\portfolio_analytics\tmp\.venv\lib\genericpath.py", line 30, in isfile
        st = os.stat(path)
    TypeError: stat: path should be string, bytes, os.PathLike or integer, not NoneType
    ----------------------------------------

multiple reports now.

I ran into this issue too, and fixed it temporarily using:
diff --git a/setuptools/msvc.py b/setuptools/msvc.py
index 2ffe1c81..ffd01704 100644
--- a/setuptools/msvc.py
+++ b/setuptools/msvc.py
@@ -1617,7 +1617,7 @@ class EnvironmentInfo:
                                     self.FSharp],
                                    exists),
         )
-        if self.vs_ver >= 14 and isfile(self.VCRuntimeRedist):
+        if self.vs_ver >= 14 and self.VCRuntimeRedist and isfile(self.VCRuntimeRedist):
             env['py_vcruntime_redist'] = self.VCRuntimeRedist
         return env

The package (pendulum in my case) was successfully installed after this change.

I ran into the same issue today (platform: Windows 10) and traced it down to the fact that my Visual Studio installation had no Redist folders. Cf. how setuptools finds the redistributable dll:
https://github.com/pypa/setuptools/blob/1d03fdc94c3676a5b675ec7d818d48c6a772fb49/setuptools/msvc.py#L1540
I ended up re-installing Visual Studio with all C++ development enabled. After that, the package in question (wrapt in my case) could successfully be installed.

Hey folks, I run into this error and trying to debug it because it only appears on the CI but I cant repro on my local dev box.
Is this the function that I should be looking at to figure out the way it tries to resolve the path?
https://github.com/pypa/setuptools/blob/d368be2b5be8676ba8a3d55045b45b55090f6982/setuptools/msvc.py#L1691

Should be fixed in v74.1.1, releasing now.


[Wildcard Variables] Core library updates

This issue tracks any updates we need to do with the core libraries for the wildcard variables feature.
@lrhn Do you have any idea if there's any changes necessary? We can close the issue otherwise.

Should be nothing we need to do.
I do not expect any public API to change, if anything, we could perhaps use the new feature in some or the example code, not week have to go through all of it to check, there are no obvious candidates. And there might not be any place where a _ is better than what we already have


Add types to package.json

According to  TypeScript docs, types property should be included in package.json to indicate the main declaration file. Otherwise, TypeScript cannot find the declarations.

I hope this will be released.🙏


Add Gentoo Support

I've added Gentoo Support, assuming the packages are found with the package names "app-admin/graylog-server" and "app-admin/graylog-web-interface" as stated in the ebuilds located at https://github.com/derdanne/gentoo-graylog.
I only added basic support for graylog-radio - i think the usage ist deprecated, or am i wrong?!

Thanks for the PR! We started a new puppet module for Graylog 2.0 over at https://github.com/Graylog2/puppet-graylog and https://github.com/Graylog2/puppet-graylog.
This module will be deprecated soon as it only supports Graylog 1.x. Do you think you can port the changes over to the new one? The new module is pretty fresh, so we are looking forward to feedback. :smiley:

@derdanne Thank you very much for your contribution.
We don't plan to add any more features or supported platforms to this Puppet module. If you want to port your PR to the new Puppet module at https://github.com/Graylog2/puppet-graylog, this would be very welcome!


prevent dublicate entries in user list by using two queries

To solve the issue #217 I have devised a procedure with two queries. Two queries are used only in case we are filtering for groups. The query is split into two queries. The first one gets the list of filtered id's, then the non-unique entries are removed, and then the query is performed with the whereIn clause to get non-duplicate list.
Possible considerations would be scaling if the app has thousands of users, as there might be limits to whereIn query length. Hope I make myself clear.

Hi, any problems with this solution?


MINOR: Improve exception messages in FileChannelRecordBatch

Improve exception message in FileChannelRecordBatch.loadBatchWithSize.
The original description was removed as part of 81f0c1e8, reuse the
description that was present before.
Replace channel by file in potentially thrown KafkaException descriptions
when loading/writing FileChannelRecordBatch. This makes exception
messages more readable (channel only shows an object hashcode, file shows
the path of the file being read).
Context
If loading a Kafka segment throws an IOException (from FileChannel.read
in Utils.readFully), the exception is wrapped into a KafkaException.
If this happens when loading/recovering a segment at start time, Kafka will shutdown
and the current exception message does not help to pinpoint the origin of the issue.
Example stacktrace: (from a broker running 1.1.0)
[2018-12-24 23:17:44,727] ERROR {main} There was an error in one of the threads during logs loading: org.apache.kafka.common.KafkaException: java.io.IOException: Input/output error (kafka.log.LogManager)
[2018-12-24 23:17:44,732] ERROR {main} [KafkaServer id=172329974] Fatal error during KafkaServer startup. Prepare to shutdown (kafka.server.KafkaServer)
org.apache.kafka.common.KafkaException: java.io.IOException: Input/output error
        at org.apache.kafka.common.record.FileLogInputStream$FileChannelRecordBatch.loadBatchWithSize(FileLogInputStream.java:214)
        ...
        at java.lang.Thread.run(Thread.java:745)
Caused by: java.io.IOException: Input/output error
        at sun.nio.ch.FileDispatcherImpl.pread0(Native Method)
        at sun.nio.ch.FileDispatcherImpl.pread(FileDispatcherImpl.java:52)
        at sun.nio.ch.IOUtil.readIntoNativeBuffer(IOUtil.java:220)
        at sun.nio.ch.IOUtil.read(IOUtil.java:197)
        at sun.nio.ch.FileChannelImpl.readInternal(FileChannelImpl.java:741)
        at sun.nio.ch.FileChannelImpl.read(FileChannelImpl.java:727)
        at org.apache.kafka.common.utils.Utils.readFully(Utils.java:831)
        at org.apache.kafka.common.utils.Utils.readFullyOrFail(Utils.java:804)
        at org.apache.kafka.common.record.FileLogInputStream$FileChannelRecordBatch.loadBatchWithSize(FileLogInputStream.java:210)
        ... 28 more

After applying the proposed changes:
[2018-12-26 23:07:45,599] ERROR There was an error in one of the threads during logs loading: org.apache.kafka.common.KafkaException: Failed to load record batch at position 12868 from file /usr/local/var/lib/kafka-logs/__consumer_offsets-9/00000000000000000000.log (kafka.log.LogManager)
[2018-12-26 23:07:45,603] ERROR [KafkaServer id=0] Fatal error during KafkaServer startup. Prepare to shutdown (kafka.server.KafkaServer)
org.apache.kafka.common.KafkaException: Failed to load record batch at position 12868 from file /usr/local/var/lib/kafka-logs/__consumer_offsets-9/00000000000000000000.log
	at org.apache.kafka.common.record.FileLogInputStream$FileChannelRecordBatch.loadBatchWithSize(FileLogInputStream.java:220)
	...
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.io.IOException: Input/output error
        at sun.nio.ch.FileDispatcherImpl.pread0(Native Method)
        at sun.nio.ch.FileDispatcherImpl.pread(FileDispatcherImpl.java:52)
        at sun.nio.ch.IOUtil.readIntoNativeBuffer(IOUtil.java:220)
        at sun.nio.ch.IOUtil.read(IOUtil.java:197)
        at sun.nio.ch.FileChannelImpl.readInternal(FileChannelImpl.java:741)
        at sun.nio.ch.FileChannelImpl.read(FileChannelImpl.java:727)
        at org.apache.kafka.common.utils.Utils.readFully(Utils.java:831)
        at org.apache.kafka.common.utils.Utils.readFullyOrFail(Utils.java:804)
        at org.apache.kafka.common.record.FileLogInputStream$FileChannelRecordBatch.loadBatchWithSize(FileLogInputStream.java:216)

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Failing tests pass locally
>> ./gradlew connect:runtime:test --tests org.apache.kafka.connect.runtime.rest.RestServerTest.testCORSEnabled

> Configure project :
Building project 'core' with Scala version 2.12.7
Building project 'streams-scala' with Scala version 2.12.7

> Task :connect:runtime:test

org.apache.kafka.connect.runtime.rest.RestServerTest > testCORSEnabled PASSED

Deprecated Gradle features were used in this build, making it incompatible with Gradle 6.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/5.0/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 37s
27 actionable tasks: 2 executed, 25 up-to-date

>> ./gradlew clients:test --tests org.apache.kafka.clients.admin.KafkaAdminClientTest.testCreateTopicsRetryBackoff

> Configure project :
Building project 'core' with Scala version 2.12.7
Building project 'streams-scala' with Scala version 2.12.7

> Task :clients:test

org.apache.kafka.clients.admin.KafkaAdminClientTest > testCreateTopicsRetryBackoff PASSED

Deprecated Gradle features were used in this build, making it incompatible with Gradle 6.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/5.0/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 7s
8 actionable tasks: 1 executed, 7 up-to-date

The other failing test is also passing locally
>> ./gradlew clients:test --tests org.apache.kafka.clients.admin.KafkaAdminClientTest.testUnreachableBootstrapServer

> Configure project :
Building project 'core' with Scala version 2.12.7
Building project 'streams-scala' with Scala version 2.12.7

> Task :clients:test

org.apache.kafka.clients.admin.KafkaAdminClientTest > testUnreachableBootstrapServer PASSED

Deprecated Gradle features were used in this build, making it incompatible with Gradle 6.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/5.0/userguide/command_line_interface.html#sec:command_line_warnings

BUILD SUCCESSFUL in 7s
8 actionable tasks: 1 executed, 7 up-to-date


SWA extension for new SWA isn't detecting local git repo

My .NET and JS testing for this tutorial gives me 1/8 steps when I select "Advanced" to create a SWA.
@HeidiSteen uses the same tutorial steps but with a Python version of the code and she gets either 12 or 13 steps - where she is asked for a repo, create a repo, etc. as if the extension doesn't recognize her local Git repo. What could be causing this?
We need the steps to be proscriptive (completely repeatable) in the tutorial expecting 8 steps and not 12 or 13.
@anthonychu @craigshoemaker Any ideas or help?

@diberry Thanks for reporting this.
The number of steps is dependent on whether or not the current local repo has a GitHub remote origin that the user has admin access to.  If @HeidiSteen is being asked to create a repo, it's likely because while she has a local repo, she needs to create a remote in GitHub for Static Web Apps to attach to and run GitHub Actions on.
The tutorial should definitely include either the step of creating/forking a repo from GitHub (8 step process) or starting with a local repo with the expectation that a GitHub repo will be created (12 step process).  Also, the documentation is a bit old since we just changed the experience when the platform left preview.  Here is a detailed description of the new creation process.

@nturinski Can you work with @HeidiSteen internally to figure out where she went wrong. She successfully completed the same steps for the JS and .NET versions with a local fork of the repo which she owns - yet I didn't have any trouble and I don't own the origin repo, only my fork of it.

@nturinski FYI that I'm using a remote origin that I own.  The repo I'm specifying in your wizard is https://github.com/HeidiSteen/azure-search-python-samples/tree/master/search-website -- which is my fork of the public repo.

I tried forking your repo https://github.com/nturinski/azure-search-python-samples.  I wasn't able to reproduce this with my workspace.
Our extension will only detect the remote repo if it has origin as the remote shortname.  Is it possible that your repo has named the remote something else?

No, the remote short name is origin.  In your wizard, step 8, it won't accept "origin" (or any other value that I could think of to type in).  There is literally no way to progress in the wizard.
origin  https://github.com/HeidiSteen/azure-search-python-samples.git (fetch)
origin  https://github.com/HeidiSteen/azure-search-python-samples.git (push)
upstream        https://github.com/Azure-Samples/azure-search-python-samples.git (fetch)
upstream        https://github.com/Azure-Samples/azure-search-python-samples.git (push)

According to our remote naming rules, you should be able to name the remote anything that git will allow, but I'm assuming that you keep seeing the message "RemoteName" is not a valid remote shortname.?
We run a git command to verify that the name is valid.  I think that for some reason, in the python project, we aren't able to run git CLI commands for some reason.  That would also explain why we are not able to recognize that you have an "origin" remote because we utilize git for that as well.

@nturinski Can you work with @HeidiSteen internally to figure out where she went wrong. She successfully completed the same steps for the JS and .NET versions with a local fork of the repo which she owns - yet I didn't have any trouble and I don't own the origin repo, only my fork of it.

Dina mentioned that you were able to complete this with the JS and .NET versions.  Are there any differences between the workspaces of these runtimes and Python?  Are you using a different work environment or anything like that?  What OS are you using?

@nturinksi I was able to complete the wizard using the previous version of your extension.  I can't use the May release for any language: C#/NET, JS, Python.

@HeidiSteen Okay, thank you for that information.  Our new changes enforce some workspace configurations, and it seems that we are having issues accessing git through the VS Code terminal.  Do you know how you installed git on your machine?  If you type which git in your terminal, where did it install to?

@nturinski  I installed Git Bash many years ago so I'm not sure.  For "which git":  /mingw64/bin/git

SWA extension works for me - I'm using:
$ which git
/mingw64/bin/git
$ git --version
git version 2.30.1.windows.1

I have this:  git version 2.17.1.windows.2   I'll see if I can get a newer git version.

@nturinski Shoudl the extension be so picky about what version of git a customer is using?

No, the extension shouldn't care about what version of git the customer is using.  I was running an older version of git for a while and wasn't running into this 🤔
I'm not sure why, but it seems that we are having issues running git CLI commands through VS Code's shell.  We try to run git commands in the Windows temp directory, so it's possible that we don't have proper permissions to run those commands in Heidi's environment.
Rather than debugging Heidi's machine, I'll see what I can do on my end to make it more agnostic of these restrictions.  When I have a new build out, could @HeidiSteen install the new bits and re-try the tutorial?

You bet -- just let me know when you have something you'd like me to try.   Thanks Nathan for persevering on this.

Hi @HeidiSteen
I have made some changes to the extension.  If you could give this a shot and see if it resolves this issue, that'd be great!
You can download the new bits here: https://dev.azure.com/ms-azuretools/_apis/resources/Containers/5177059/vsix?itemPath=vsix%2Fvscode-azurestaticwebapps-0.7.1-alpha.vsix
Once you have downloaded the vsix, you can install it by pressing F1 and running the command "Install from VSIX".
Hope this works! 🤞

Hi @nturinski (Nathan) -- I don't have permissions to the download site.  Can you zip and send?

vscode-azurestaticwebapps-0.7.1-alpha.zip
Oops, sorry about that.  Looks like I can upload it straight to here.

@Nturinski @diberry -- Nathan, your fix works great!!  I tried all 3 of our samples (C#, JavaScript, Python).  The steps were the same in all cases, and probably all the same code, but I wanted to be thorough in case there were differences that I'm not aware of.  In all 3 instances, I got the 8-step version of the wizard and had no problems deploying the app.

Going to go ahead and close this as it seems to be resolved by removing any git and simple-git calls.
Closed by https://github.com/microsoft/vscode-azurestaticwebapps/pull/408

@HeidiSteen @diberry Just wanted to let you know that we've released the fix for this issue officially!  The latest version is 0.7.1.


Suggestion: add an are you sure pop up before being able to delete an extension from the categories

someone could be trying to click replace extension but then click delete on accident lol

i mean i guess 🗣


The text-panel disappears by clicking the "zoom" tile

"Grid" theme, full of thumbnails.
When I click on any thumbnail, the related full image is displayed correctly, as well as the text-panel (description) on the bottom part of the viewport.
The problem arises when I press the "zoom" tile: the picture is correctly scaled, but the text-panel fades out. I may try to click any button, but unless selecting another thumbnail, there's no way to take the text-panel visible again.

HI, please write me a support ticket here:
valiano.ticksy.com
please specify the website url where you have this bug, I'll check.
On Tue, Jan 12, 2016 at 6:14 PM, Mario Vernari<EMAIL_ADDRESS>wrote:

"Grid" theme, full of thumbnails
When I click on any thumbnail, the related full image is displayed
correctly, as well as the text-panel (description) on the bottom part of
the viewport
The problem arises when I press the "zoom" tile: the picture is correctly
scaled, but the text-panel fades out I may try to click any button, but
unless selecting another thumbnail, there's no way to take the text-panel
visible again
—
Reply to this email directly or view it on GitHub
https://github.com/vvvmax/unitegallery/issues/49.

I'd create the ticket, but the available categories don't show the "plain" library. Let me know how to specify it, and I'll do sooner.
BTW, the problem is easy to see even on your web page:
http://unitegallery.net/index.php?page=grid-right
The first picture you see is "Rocks on water". Just press the "zoom in" (+) button and the bottom strip with the photo caption disappears. The only way to see it again is by selecting another picture from the thumbs grid, or swiping back and forth.

Yes, I made it in purpose. if you want to see the image, then the caption
is distructing you. so when the image zooms in, the caption dissapears.
On Thu, Jan 14, 2016 at 11:47 AM, Mario Vernari<EMAIL_ADDRESS>wrote:

I'd create the ticket, but the available categories don't show the "plain"
library. Let me know how to specify it, and I'll do sooner.
BTW, the problem is easy to see even on your web page:
http://unitegallery.net/index.php?page=grid-right
The first picture you see is "Rocks on water". Just press the "zoom in"
(+) button and the bottom strip with the photo caption disappears. The only
way to see it again is by selecting another picture from the thumbs grid,
or swiping back and forth.
—
Reply to this email directly or view it on GitHub
https://github.com/vvvmax/unitegallery/issues/49#issuecomment-171589707.

Okay, but is there any way to re-display the caption without be forced to change picture?
When I restore the normal size (the 3rd button in the top-left of the frame), why don't I have my caption back?

it's not coming back because I assume that if you zoom you don't need the
caption anymore :)
it's built like this, sorry.
On Fri, Jan 15, 2016 at 5:02 PM, Mario Vernari<EMAIL_ADDRESS>wrote:

Okay, but is there any way to re-display the caption without be forced to
change picture?
When I restore the normal size (the 3rd button in the top-left of the
frame), why don't I have my caption back?
—
Reply to this email directly or view it on GitHub
https://github.com/vvvmax/unitegallery/issues/49#issuecomment-171984485.


Make build scan URLs clickable in warning and notices

When a Gradle invocation generates a build scan, a notice is added with the build scan URL. It would be great if this was clickable.
It's unclear if this is possible, or if it requires a change to the GitHub actions infrastructure.

I've spent quite some time trying to find a way to make this work, to no avail. If anyone has knowledge about how to make clickable links in a notice emitted from GitHub Actions, I'd love to hear it.
I also investigated creating a separate check-run for each Gradle invocation: this would allow the Gradle invocations to be listed separately from the workflow jobs, with rich markdown content associated.  Unfortunately this doesn't work well when you have more than 1 workflow configured: the check-run gets randomly assigned to one of the workflows, often the wrong one.

We will address this via #290


Changelog entry for PR #1410 and turning off Travis Linux Nightly build job

Travis build is messed up. PR will be reopened to trigger Travis over with the correct commit hash.


[React Native Compat] Don't require window.location to be set

React Native, is a browser-like environement but some browser properties
don't currently make sense in React Native. For example the location
object. However, we want our users to be able to use superagent, we even
recommend it on our website.
https://facebook.github.io/react-native/docs/network.html

Thanks for merging it in that quickly :)

A new version on npm would be awesome!

:+1: nice

@amasad this weekend


Turn on "reduce travel time" and "cut inner first" by default?

Are there important reasons that they're currently off-by-default? I feel like they're what most users expect / want.
reduce travel time turned off (the current default)

tuned on, 🏎🏎🏎🏎

Was previously much slower. I think that was most of the reason. Sometimes you could look down on like a couple minutes.

thanks!

Welcome to the Meerk40t Contributors Hall of Fame!!!!!!


Add backend tests

Nice additions! I'm definitely happy you've got good examples of both unit & integration tests here, so very happy for you to submit this PR in Aptem
Note that what is missing though is guidance on running the tests, and assuming you're planning to use the suggested pytest library then we should make sure that's recorded as a dependency! So add that with poetry add pytest (or poetry add pytest --group dev if you want to be fancy) and then check they run with poetry run pytest (and document that in your Readme!)

Thanks Jack. On the pytest point that was a surprise. Looks like I had pytest installed globally by accident and so it was all working! Will make some amendments!


(Feature Request (if feasible)) Sanoma like wallpaper option

An option like the native MacOS Sanoma screen saver fading into wallpaper
It would be amazing if Aerial could reflect Sanoma's ability to fade into a still wallpaper.

Awesome! I figured out the speed slider (I am on beta), but could not get it to be still, nor did I find an existing FR.

For those who haven't tried it yet it looks like this !

@Duckbilled make sure you enable betas for Aerial itself too (I know it's a bit confusing), you need the latest Aerial beta (the screensaver) to get the slider to work.
For best results, again with the beta, make sure you enable the macOS source which contains the 240 FPS assets.

Then make sure in advanced you set to 240 FPS.

Then you'll get butter smooth slow wallpaper ;) What I'm working on right now is transitionning from full speed when in screen saver mode to either slow speed (which ever you pick) or paused. This requires however that I'm able to detect on which monitor we are running as a screen saver, and Apple broke that pretty badly. So that's a big blocker at the moment but yep, fingers crossed, that's my goal. We'll likely get per screen/space video too out of it when they fix it.

Thanks for the swift replies.
Just to clarify, I am running Sonoma beta, and managed to slow down the video from another tvos set, but wasn't able to make it completely still.
I have a look at the options you provided :)

Ha, misread you, press the pause button :

Right now there's a bug where it restarts itself after it wakes up from sleep though, this might be what you are referring to ? Right now you have to pause it again every time, bit of a pain but it should be fixed in next beta soonish.

Nice, that sounds like what I am looking for, indeed. I keep an eye out for the next beta's

The pause thing is fixed in beta5 @Duckbilled

I an going to give it a spin tonight!


Improve uncertain conditional test

Improvements for #577.

Data generation separeted from tests.
Code duplication removed.
Not requires extra packages.

Codecov Report

Merging #583 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #583      +/-   ##
==========================================
- Coverage   97.51%   97.48%   -0.03%
==========================================
  Files          45       45
  Lines        3579     3579
==========================================
- Hits         3490     3489       -1
- Misses         89       90       +1

Impacted Files
Coverage Δ

gpflow/params/parameterized.py
98.59% <0%> (-0.47%)
:arrow_down:

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 d86af28...04d5d95. Read the comment docs.


remove files generated by antlr3 in the clean task

Current coverage is 81.52% (diff: 100%)

Merging #4617 into master will decrease coverage by 0.15%

@@             master      #4617   diff @@
==========================================
  Files          1302       1302
  Lines         44777      44777
  Methods           0          0
  Messages          0          0
  Branches       7057       7057
==========================================
- Hits          36574      36504    -70
- Misses         5518       5577    +59
- Partials       2685       2696    +11

Powered by Codecov. Last update 17d1ed8...06e4452

We generally don't include files that once had been there in the clean tasks of gradle. And I think that's a good thing.
git clean -xdff is the way to go for anyone who has problems with leftovers.

closing. see comment above.


Refactoring cordovaHelper, cordovaWrapper and projectHelper into

taco-utils. This allows other TACO packages to leverage APIs exposed in
these modules
Key Changes include:

Splitting ProjectHelper.editTacoJsonFile into
KitHelper.editTacoJsonFile to allow kit specific logic in taco-cli
Added VerifyExcludedTacoErrors to ensure all moved taco Errors are not
taken for some other values

@meenakb can you review this?


[Microsoft Generator CSharp] Add support for parsing source input file

The generator executable should support parsing a  TypeSpec code model JSON file (tspCodeModel.json) as input and deserializing it into an object/model that can be consumed by the generator.

@jorgerangel-msft, could you provide more information for this task?  I believe today the c# code generator would consume that file and deserialize it.   Is there any missing functions here? Thanks

hello @jorgerangel-msft So will there be two separated generator: branded generator and unbranded generator? I am curious about the current architecture of .NET codegen.

@chunyu3 There will only be 1 generator but there will be 2 plugins to the generator.  The base plugin will be the SCM plugin and the azure plugin will override / extend a few things on top of that so will not be a complete rewrite.


tds.AtomicCounter

Atomic counter that can safely track progress between threads. I'll add threads.sharedserialize support once this is merged in tds.

The THAtomic.h header from torch is never included which gives implicit declaration of function ‘THAtomicAddLong’, etc. warnings. Since torch7 is not a hard dependency you should condition the availability of this atomic counter with the HAS_TORCH macro handled by cmake if torch is detected, e.g. see this.

Added the include with HAS_TORCH you suggested, also added basic mutex solution if no Torch is present (probably not that useful). This is expected to be used with Torch threads of course, so I didn't want to copy the whole THAtomic over.

so I didn't want to copy the whole THAtomic over

I see. Another possibility to avoid reinventing the wheel (since anyway pthreads detection should also be included and all of this is already managed by torch7) could be to introduce a function like:
bool tds_has_atomic(void) {
#if HAS_TORCH
  return true;
#else
  return false;
#endif
}

(and of course bracket the implementation of the various functions with HAS_TORCH)
This could be used to avoid creating an atomic counter:
-- atomic.lua

function atomic:__new(...)
  if not C.tds_has_atomic() then
    error('atomic counter not available (Torch not found)')
  end
  -- ...
end

Thanks, implemented your suggestion, the implementation now is only available with Torch, because of the dependency on THAtomic, otherwise makes no sense to provide a useless implementation.
Should be good to go...?

Cool! Looks good to me.

Thanks!


Switch to alma container

In the end, after installing all of the stuff for simtools, the container is unfortunately 1.7 GB (the simtools ubuntu based container is 1.9 GB). So we only save 200 MB with this change, which is not as much as I had hoped since the minimal almalinux container is about 1 GB smaller than the ubuntu one we've been using.
Either way, it's a necessary change probably, so that we are in line with the production environment.
My local tests all worked, hopefully no one runs into any other changes.

Codecov Report

Merging #639 (a8b27d9) into main (1cbb10a) will not change coverage.
Report is 6 commits behind head on main.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main     #639   +/-   ##
=======================================
  Coverage   99.29%   99.29%
=======================================
  Files          39       39
  Lines        3551     3551
=======================================
  Hits         3526     3526
  Misses         25       25

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks for looking into it. Yes, reducing the final container will be hard, but the reduction of the sim_telarray container will be useful in the future when we switch to use that externally.
I am running the workflow for building the sim_telarray container now. If that is successful, I will rerun the tests here and hopefully all will pass and we can merge.

All passed, merging. Don't be surprised by the different look and feel of the new container when you switch to it.


check that's node.js is installed

@nogenhat This is a fantastic clean up and improvement! Thanks!


pipe char escape, flag --server-print, labels sample, exec extra --, config NL

Proposing fixes for some issues in the page:

Handle display issue for the 'kubectl run' command, where pipe character ('|') is not properly escaped, so the text gets trimmed.
Currently the text is rendered like this: "run	`kubectl run NAME --image=image [--env=“key=value”] [--port=port] [--replicas=replicas] [--dry-run=server".
The correct text will be displayed like:
"kubectl run NAME --image=image [--env="key=value"] [--port=port] [--replicas=replicas] [--dry-run=server|client|none] [--overrides=inline-json] [flags]".
Also, the description column would be displayed properly.

Fixing output of 'get pods' command, when flag --server-print is set to false.
The only displayed columns would be NAME and AGE.

Fixing description and sample command for delete pods (kubectl delete pods,service ..), where 'label-name' text is ambiguously used as the label's value. Labels are key-value pairs.
Suggested way of writting it:
"# Delete all the pods and services that have the label '='.
kubectl delete pods,services -l ="
Another way to write it could be:
"# Delete all the pods and services that have the label 'name='.
kubectl delete pods,services -l name="

Updating the syntax for three 'kubectl exec' commands, to include the '--' before the user's command, to be inline with current way of working and to avoid deprecation message.

Enhance sample for 'kubectl config' command, where the newline was not handled properly.
Before:
[user@wstation ~]$ kubectl config view --template='{{ range .contexts }}{{ if eq .name "'$(kubectl config current-context)'" }}Current user: {{ .context.user }}{{ end }}{{ end }}'
Current user: kubernetes-admin[user@wstation ~]$
After:
[user@wstation ~]$ kubectl config view --template='{{ range .contexts }}{{ if eq .name "'$(kubectl config current-context)'" }}Current user: {{ printf "%s\n" .context.user }}{{ end }}{{ end }}'
Current user: kubernetes-admin
[user@wstation ~]$

/assign @steveperry-53

/lgtm
/approve


Set reasonable defaults for in/oubound peers

Defaults should support a healthy network.  Inbound peer settings are a limitation that should be addressed
Current values
# Maximum number of inbound peers
max_num_inbound_peers = 40

# Maximum number of outbound peers to connect to, excluding persistent peers
max_num_outbound_peers = 10

Osmosis has:
# Maximum number of inbound peers
max_num_inbound_peers = 320

# Maximum number of outbound peers to connect to, excluding persistent peers
max_num_outbound_peers = 40

If we generate default config file here, we should also look at "back porting" many of the customisations that were added in testnet-3.
The main importance is these are all rolled out on the testnet

Changed maximum number of inbound/outbound peers for tgrade-testnet-3


libOpenCvSharpExtern.so: undefined symbol: _ZTIN2cv16ParallelLoopBodyE

OpenCvSharpExtern was compiled successfully, but when I try to use it on a simple example, I get these errors (with MONO_LOG_LEVEL=debug) with the mentions of an undefined symbol. ldd libOpenCvSharpExtern.so doesn't show any links to OpenCV. It looks like a relapse of #85 after the migration of 3.x branch to master.

🤔
please try the following
https://ubuntuforums.org/showthread.php?t=2109967
https://github.com/kyamagu/mexopencv/issues/123

Thanks for the answer, I'll return to this in a month or two.

Seeing this also on NixOS; more details at https://github.com/shimat/opencvsharp/issues/273#issuecomment-277501280


adding k8s flexvolume driver as a convoy client

@yasker
changing convoy to integrate with k8s flexvolume driver. Besides arguments are different, some other requirements are:

no stdout printout at all except in k8s format
attach call must return a device in the printout to stdout
all calls from k8s don't have convoy volume ID as important parameter, need to find them out

This PR only container added k8s flexvolume driver changes in order to let k8s to call.

There definitely should be a document on how to use this commit and make Convoy work with K8s.
Also, does this one depends on #161 ? I didn't see it mentioned anywhere.
And, what's about #157, is it obsolete?

Yes, I agree, we need to eventually document k8s drivers' usages somewhere when EFS is done as well, probably in convoy wiki itself. Yes, it depends on PR#161 and PR#157 is closed.


[a11y] SPIKE: Setup axe-core for automated accessibility testing

Description
To better align with our Accessibility mission, automation is one way to move towards that mission.
Implement axe-core as a proof of concept to determine how well it will work in sage-lib
Axe Core npm

Based on research, this plugin is best suited for React and while we use React, it would be helpful to have a solution that is both Rails and React.
For the time being, adding documentation to encourage the use of a browser add-on that audits the page. https://github.com/Kajabi/UXD-Handbook/pull/9


BREAKING CHANGE: switch all security list of worker nodes and load balancers to NSGs for greater flexibility and control on ports and cidr blocks

Closes #398
Signed-off-by: Ali Mukadam<EMAIL_ADDRESS>

@denismakogon: Can you please test? This should solve your issues


Select and add a default documentation choice.

Options

pdoc
Sphinx

Criteria

Automatic generation of documentation from docstrings
Support for Markdown

Decided on pdoc3 as the default choice because it's so lightweight and supports Markdown-formatted docstrings (as well as several other common formats).

Next Steps

Add pdoc3 to dependencies.


Refactor tests

Tests are currently instantiating a lot of test objects; try to refactor them so they have just one assert (when possible). Try to remove qulice ignore tags when refactoring this code.

@paulodamaso the puzzle #18 is still not solved.

@paulodamaso the only puzzle #18 is solved here.


add arm64 docker build support #379

Description
Support Multi-Platform Docker builds to allow KubeClarity Install on ARM-based platforms. #379
Type of Change
[ ] Bug Fix
[x] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[ ] Other (please describe)
Checklist

[x] I have read the contributing guidelines
[x] Existing issues have been referenced (where applicable)
[x] I have verified this change is not present in other open pull requests
[x] Functionality is documented
[x] All code style checks pass
[x] New code contribution is covered by automated tests
[x] All new and existing tests pass

@FrimIdan please review

@shukla2009 Probably setting up QEMU is missing to make the GH workflow working again.
See https://github.com/marketplace/actions/docker-setup-buildx
     - # Add support for more platforms with QEMU (optional)         # https://github.com/docker/setup-qemu-action         name: Set up QEMU         uses: docker/setup-qemu-action@v3
Otherwise the buildx builder does not support arm64 in later steps. QEMU can also be limited to arm64 only.
     - # Add support for more platforms with QEMU (optional)         # https://github.com/docker/setup-qemu-action         name: Set up QEMU         uses: docker/setup-qemu-action@v3         with:           platforms: arm64

@tommy1199 The CI failed due to a network error. QEMU is not required anymore for multi-arch builds.


(Vue) Add slot for the entire calendar header

First of all, thank you for creating this library, I've enjoyed playing around with it a lot.
I have seen in the Vue docs that the are several slots for prepending and appending things on the right and left sides of the header but there isn't a slot to replace the entire header itself. I think it would be really interesting to have one so that the user can create the header they want for the calendar.
I'm thinking that in order to move around to different days, weeks or months in a custom header the user could use the Calendar Controls plugin. I haven't played around much with it but they could use the setDate method to move around the Calendar as necessary.
Once again, thank you for this great resource!

I think that’s a good idea 👍 if you need it urgently, I’m open to PRs for it. Otherwise I’ll just put it on my list of todos for once I’ve gotten past my problems with the React adapter

Thank you so much! For the time being is not urgent but I thought it would be a nice feature to have.
Have a great day and good luck with the React adapter problems :)

+1 to this, I'd like more customization in the header or also the possibility to hide it and make my own (in this case there should be enough functions available to make it work). Great work btw!


howto-toggle-button

An implementation of a howto-toggle-button in compliance with the wai-aria-practices as stated here: https://www.w3.org/TR/wai-aria-practices-1.1/#button
Followed similar pattern as howto-checkbox and added corresponding unit as well as e2e tests.

This is awesome work @rajatsehgal! Thank you for putting this together 😃

Just started reviewing :) :octocat: Sent from GH.

woo! nice work @rajatsehgal 🐝 thanks again :)

@rajatsehgal check it out → https://www.youtube.com/watch?v=16gvkPfPIx4&index=1&list=PLNYkxOF6rcICWx0C9LVWWVqvHlYJyqw7g 😁

@robdodson thanks for the shoutout! I really enjoy your videos. I'm planning to work on a howto-slider next, unless someone is already doing it?

Nobody is working on one right now so that would be great!


BZ2119586: Fix preserving values on Back in the Create Cluster wizard

Depends on:

[x] https://github.com/openshift-assisted/assisted-ui-lib/pull/1717
[ ] https://github.com/stolostron/console/pull/2254

/lgtm

/retest

/retest all

/test all


Clearer error when neither terraform nor opentofu is installed

Describe the solution you'd like
With the recent addition of opentofu support, running terragrunt (tested with terragrunt output) with neither opentofu nor terraform installed outputs

ERRO[0000] exec: "tofu": executable file not found in $PATH
ERRO[0000] Unable to determine underlying exit code, so Terragrunt will exit with error code 1

I imagine this should give a message that explains the need to install one of the two options.
Tested on terragrunt version 0.53.0.
Edit: didn't realize opentofu was a full fork when making this issue.

Hey, did you find the solution?

Ran into the same issue.
Turns one my terraform binary was broken, and it seems as if when terragrunt can't find terraform, it then tries tofu - and given it's the last thing it tires, the error message states tofu not found.
Fixing my terraform binary has resolved the issue. tofu-free solution :)


ERROR TypeError: Cannot set property passive of [object Object] which has only a getter

I got this error w/ angular and angular material
ERROR TypeError: Cannot set property passive of [object Object] which has only a getter
    at EventTarget.addEventListener (modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:10606)
    at modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:21335
    at Object.supportsPassiveEventListeners (modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:114890)
    at modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:117297
    at ZoneDelegate.invoke (modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:19984)
    at Zone.run (modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:19734)
    at NgZone.runOutsideAngular (modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:29928)
    at FocusMonitor._registerGlobalListeners (modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:117294)
    at FocusMonitor._incrementMonitoredElementCount (modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:117478)
    at FocusMonitor.monitor (modules.js?hash=c08a6c1be18ea8d3c65fba22b566f1b2e49111a3:117190)

Hi there, I tried to create a PR to no avail, but this problem seems to be related to a spelling mistake in the supportedPassiveTypes in index.js.
const supportedPassiveTypes = [   'scroll', 'whell',   'touchstart', 'touchmove', 'touchenter', 'touchend', 'touchleave',   'mouseout', 'mouseleave', 'mouseup', 'mousedown', 'mousemove', 'mouseenter', 'mousewheel', 'mouseover' ];
Renaming "whell" to "wheel" should fix this error.

However, doing this removes the error, but on chrome I still get the following message:

Added non-passive event listener to a scroll-blocking 'wheel' event.


Update to v0.14.0-rc3

Backlinking to purescript/purescript#3942

CI passes. Can I get an approval?


Help, I Can't Use It

Traceback (most recent call last):
File "steam_auto_cracker.py", line 390, in
File "steam_auto_cracker.py", line 115, in FindGameDirectory
FileNotFoundError: [WinError 3] The system cannot find the path specified: 'C:/Users/ventu/Music/T6 By AvatarDZ (NF)/'
error.log

I may be dumb but I followed your advice, however it gives me always the same error:

(yea i'm trying to crack clickteam fusion)

Sorry for the misunderstanding, when it's asking for the folder name here, it's asking for the folder name for Clickteam Fusion, inside your CrackedGames folder.
 SAC won't download games for you, you first have to download the games files (ex: on CS.RIN.RU) then run SAC to remove the anti-piracy measures.

Closed due to inactivity. Reopen if you need to :)


Hardcoded constant values for ssz

I've seen a few hardcoded constants for ssz usages like 33 instead of DEPOSIT_CONTRACT_DEPTH + 1. It's a little concerning in the event of off-by-one errors or config mismatches. I'd recommend not hardcode ssz sizes as unnamed constants

credit to @potuz for pointing this out

nice catch! will update

please reopen if you see any more opportunities for constants

I think this issue should not be closed. Many ssz sizes are hardcoded as unnamed constants and used in different parts of code. Either they should be a named constant or the types should have a function returning their serialized size

@potuz could you link to some examples please?

Doesn't Prysm do it the same way too? eg: https://github.com/prysmaticlabs/prysm/blob/develop/proto/prysm/v1alpha1/beacon_block.pb.go#L305 - how would you encode these with consts?

@potuz could you link to some examples please? (reopened the issue)

Indeed I could have sworn that I saw a bunch of hardcoded ssz sizes in the deposit fix PR, my mistake, it must've been the encoded files. I can now only see hardcoded sizes when unmarshaling, like this one
https://github.com/flashbots/go-boost-utils/blob/main/types/common.go#L218

No todo's anymore. The remaining constants are the auto-generated ones. Closing this for now, feel free to comment to reopen.


[MRG] Fix bug in to_data_frame.

Closes https://github.com/mne-tools/mne-python/issues/4436

@hadrienj see if this fixes your problem.

LGTM

Yes, seems to be fixed. That's great!
Thank you!

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@b3a5407). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #4437   +/-   ##
=========================================
  Coverage          ?   82.78%
=========================================
  Files             ?      349
  Lines             ?    64240
  Branches          ?     9889
=========================================
  Hits              ?    53183
  Misses            ?     8320
  Partials          ?     2737


Tooltips disappear too fast to take a screenshot in MacOS

If you are trying to take a screenshot in MacOS of something including tooltips, they disappear too fast to be able to snap them. Which is interesting, because usually MacOS manages to keep things like tooltips up in most other applications when taking a screenshot.
Happens with Command-Shift-3 and Command-Shift-4
It appears to be connected with pressing Shift.

This has been driving me mad lately, I'm glad I'm not the only one. I've been wanting to send screenshots to my junior devs.

It's impossible to send any meaningful screenshots with VSC on macOS.

You can use the grab app that is shipped with macOS and is authored by Apple to create a timed screenshot. https://support.apple.com/en-gb/guide/grab/welcome/mac
I don't think taking screenshots should be taken into account to influence the UX of a product.

@alexandrudima Is there a good reason for the tooltips to disappear only when you press shift? If not, perhaps you will reconsider your position.

I don't think taking screenshots should be taken into account to influence the UX of a product.

As an engineer taking screenshots is absolutely part of my user experience.

@lukepighetti @sean256
The idea is the following: If you move the mouse over some symbol and the hover shows up, the hover gets immediately dismissed when you begin typing. Say, for example you forget the type name of some variable, you would hover it and then learn the type name, and then proceed to type it. As soon as you begin to type, the hover gets out of the way so you can see the code you are editing.

I wonder if it would work if you just make it so shift didn't hide tooltips. Shift doesn't send any text to the window so it would seem like a reasonable compromise

@alexandrudima Thank you!

Yeeeeaaaaahhh!!! 😄


Support adding basic primatives

We should make it easy to add basic primitives like cubes and spheres.

It's pretty quick to generate these with trimesh and then send them over, like here:
https://github.com/nerfstudio-project/viser/blob/5bd1d2e7a60c64011dd51a8db8d240d52848b5d7/examples/12_click_meshes.py#L45C1-L64
Do you think we should write our own wrappers? Maybe just highlighting this in the documentation would be enough.

I think wrappers could be nice. These are very common actions, requiring people to use trimesh requires knowing another library. Additionally, tab completing viser_server.add_ to see all the things you can add will probably be a pretty common user flow, having add_cube show up right there would be handy.


validation set cnn bug fixed

validation split cnn fixed

This looks correct. Thanks @JoachimSchaeffer @m-c-frank


Use netlify-redirect-parser package

- Summary
Remove local code in favor ofnetlify-redirect-parser  package. And add support for netlify.yml
- Test plan

Run netlify dev in a project with different netlify config files

- Description for the changelog

Remove local code for redirects parsing in favor of netlify-redirect-parser.

- A picture of a cute animal (not mandatory but encouraged)
🦊

lool undoing our work


[VS Code] Could not load file or assembly 'Microsoft.Cascade.Ssh, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.

Error:
Could not load file or assembly 'Microsoft.Cascade.Ssh, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.623

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.26.1

platformName
MacOS

platformVersion
17.7.0

Thanks for reporting the issue. To help us investigate it, can you please get Live Share logs from VS Code and attach them to the issue? To get the logs, run 'Live Share: Export Logs' command from VS Code command palette. The command will ask you for the zip file location, and will save the logs into that file. You can then attach the file to the issue.

The error message seems to indicate one or more of the files did not get extracted from the extension VSIX package. Uninstalling and reinstalling the Live Share extension will probably resolve the issue.

Closing due to inactivity. Please re-Open if you’re still encountering this issue. Thanks!


Add eureka CLI productivity tool

Types of Changes

[x] New addition to list
[ ] Fixing bug in existing item in list
[ ] Removing item from list
[ ] Changing structure (organization) of list

Proposed Changes
I'm adding a tool I wrote that I think increases productivity for people using the terminal. Instead of forgetting an idea you can now just input it into a list and eureka pushes it to your git repository.
PR Checklist

[x] I have read the CONTRIBUTING guide
[x] I have explained why this PR is important
[x] I have added necessary documentation (if appropriate)

@jaywcjlove Any chance of getting this reviewed? Thanks!


lyra-cursors: init at 0-unstable-2021-12-04

Description of changes

This adds the Lyra-Cursors cursor theme.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/) - No binaries
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thanks! I've removed the with lib; and run the formatter. I'd be fine with a backport!

Thanks!
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 308515

aarch64-darwin

:white_check_mark: 1 package built:

lyra-cursors


Remove plugin.Definition

What this PR does / why we need it:
The type plugin.Definition completely duplicates fields that exist
inside of manifest.Manifest and adds nothing new. Though I prefer
the name plugin.Definition, the users currently use the manifest.Manifest
object locally and expect the nested sonobuoy-config object so we
needed to keep that in order to not break existing plugins. In addition,
the plugin.Definition type was missing the Driver field, so the server
was actually losing that data when using that type.
This change should have no impact other than clarifying the landscape
of types on the backend.
Which issue(s) this PR fixes
Helps facilitate #306
Release note:
NONE

Codecov Report

Merging #810 into master will increase coverage by 0.03%.
The diff coverage is 60.86%.

@@            Coverage Diff             @@
##           master     #810      +/-   ##
==========================================
+ Coverage   43.24%   43.28%   +0.03%
==========================================
  Files          71       71
  Lines        4257     4251       -6
==========================================
- Hits         1841     1840       -1
+ Misses       2307     2304       -3
+ Partials      109      107       -2

Impacted Files
Coverage Δ

pkg/plugin/manifest/manifest.go
4% <ø> (ø)
:arrow_up:

pkg/plugin/loader/loader.go
73.52% <100%> (-2.15%)
:arrow_down:

pkg/plugin/driver/job/job.go
56.17% <25%> (ø)
:arrow_up:

pkg/plugin/driver/base.go
42.3% <33.33%> (ø)
:arrow_up:

pkg/plugin/driver/daemonset/daemonset.go
48.73% <60%> (ø)
:arrow_up:

pkg/plugin/aggregation/aggregator.go
83.2% <0%> (+4%)
:arrow_up:

Continue to review full report at Codecov.

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


[BUG] Text cursor (caret) is changing position when I'm saving file

No matter if I choose "File > Save" or use combination "CTRL + S", text cursor will move to last line.
Im using v1.1.1 in Win10 (x64).

It's not a bug, it's a feature :D
But if seriously, it's done on purpose. I do not remember why it was necessary. If it hinders for you, I will fix it somehow.

I see. It would be great to have option to Enable/Disafle this feature :)


Deploy Deep Learning Application. Create WebApp - code

Hi,
you created the video on Deploy Deep Learning Application. Create WebApp - covid 19 detection. And also provided the GitHub link. But when i open the link, i didn't find any code regarding this. Can you please provide the flask app code for covid 19 detection?
Beast wishes,
Kavitha

Hi Kavitha,
I'm sorry for not getting back to you sooner.
That was way back.
May be this repo will help : https://github.com/bokey007/Deploy_deep_learning_model_to_cloud
if you feel like some code is missing do let me know, I would dig out even further !


int passed to a class instance is not correct depending on where the class is defined during Node  _Ready() startup - C# 4.2.1 stable

Im gonna do some more testing before I post again


Unclear Usage Information about Reverse Image Search

After reading closed Issue #2, I was more confused. Ended up trying stuff until it worked from the share menu.
A sentence or two when first ran would be a big help. I usually hate those "swipe until done because this information is sooooo important" tutorials when apps first start but this app needs one or something. No one expects a settings page to be the 1st and ONLY thing an app shows.
All those pics in F-Droid made this ultra confusing. I expected the app to have one of those pages shown in the pics. Didn't get to them until I got the app "working".
Yes, app works out of the box. App is rather unintuitive and confusing to get it to work.

this is the same reason why i made my own fork, you can try it from here
if @scubajeff want , i can make a pr with the changes

@odnar-dev Your work looks amazing. Unfortunately, I don't think I have enough time to work on this project, most of my attention is now on my other projects like Les Pas. Setter's code base is really a mess right now, if I can spare some time later, re-factoring is option number 1.

Understood. Keep on working on what you love devs. Best I can do is use and give feedback. I write Chicken Code to go with my chicken scratch handwriting so all of this is amazing to me. 🤣😘🤓


Model parallelism for GPU training supported with python interface in caffev2?

I am talking about model parallelism. For example a modified version of resnet152 is more than 12Gb for me(eg:- larger image size).I only have two 12GB GPUS...What I mean is I want to assign some parts of resnet152 to GPU1 and others to GPU2.
Data parallelism will copy multiple versions of resnet152 to each GPU.Tensorflow supports assigning each layers to a separate layer.
I was wondering if that is supported in caffe 2?
#345  PS: My original question on model parallelism was closed as it was thought of as Data parallelism

First, try passing 'optimize_gradient_memory=True' to data_parallel_model (even if using just one GPU), to see if you can fit that on your GPU. For ResNets it is very effective. You can also run the 'memonger' algorithm yourself (see how data_parallel_model:OptimizeGradientMemory is implemented).
But Caffe2 also supports model parallelism, but pretty manually. You can assign each operator to different GPU by using DeviceScope:
with scope.DeviceScope(core.DeviceOption(caffe2_pb2.CUDA, gpu_id)):
model.FC(...)
model.Conv()
(I might have made a typo in above, see again data_parallel_model.py code and look for usages of DeviceScope).
Then, I think, it should just work if inputs from several GPUs are combined in an op (for example to compute the final loss). If not, you can insert a Copy() op with appropriate device option.


Incompatible .framework file.

Hi,
I'm using  Xcode 9.4, the latest stable release. I was trying to setup your framework but got the following error:
"Incompatible Swift version - framework was built with 4.1 (swiftlang-902.0.48 clang-9<IP_ADDRESS>) and the local version is 4.1.2 (swiftlang-902.0.54 clang-9<IP_ADDRESS>).
How do I fix this?
Thanks!
Also, would greatly appreciate a Pod for this project. Very useful to my requirements. :)

@Shoshin23 just rebuilt with latest Xcode and published a new release (v0.3.4).  should fix your issue, please let me know.

Hi @colbylwilliams I dont get this error anymore.
I have one more question. Can you outline the broad steps to use the SDK? I'm trying to following:
in AppDelegate.swift
CustomVisionClient.shared.trainingKey   = "58cd2b2c9fd043eca6f234d70b10f20d"
CustomVisionClient.shared.predictionKey = "bf8b509b39fc43efa5cb1781483e6961"
in my training file
import CustomVision
    let tag = "Airpods"     // doesn't exist in project yet
    let images: [UIImage] = [image array]

    client.createImages(from: images, withNewTagNamed: tag) { r in
        print(r.resource) //I get nil here.
    }

I don't really know how to proceed next. Any help with this would be great. :)

I’ll update the docs.  In the meantime, see this project: https://github.com/colbylwilliams/RetailVision/tree/master/


Support filtering with units and more

The first commit:

Adds units to exported JSON
Adds units key to metadata skeleton
Adds some docs for units

The second commit adds filtering by units by the first method I mentioned in #203:

Try it here. I think it integrates pretty neatly.
The third commit adds support for registering custom units with Pint from metadata.json. Probably pretty niche, but I need decibels!

This looks great so far - love the new documentation. Let's throw in a unit test or two for the basic unit filters (mainly as a protection against accidental regressions in the future).

I've added some tests and that docs link.


Report issue button hidden in "Running Extensions"

Open "Running Extensions" tab
Have extensions with uncaught errors
There's a barely visible piece of a "report issue" button

@jrieken I could use some help here. Change it such that the action bar is always shown:
diff --git a/src/vs/workbench/contrib/extensions/browser/abstractRuntimeExtensionsEditor.ts b/src/vs/workbench/contrib/extensions/browser/abstractRuntimeExtensionsEditor.ts
index b764f946925..f7198c4c5c8 100644
--- a/src/vs/workbench/contrib/extensions/browser/abstractRuntimeExtensionsEditor.ts
+++ b/src/vs/workbench/contrib/extensions/browser/abstractRuntimeExtensionsEditor.ts
@@ -286,7 +286,7 @@ export abstract class AbstractRuntimeExtensionsEditor extends EditorPane {
 				if (slowExtensionAction) {
 					data.actionbar.push(slowExtensionAction, { icon: true, label: true });
 				}
-				if (isNonEmptyArray(element.status.runtimeErrors)) {
+				if (true || isNonEmptyArray(element.status.runtimeErrors)) {
 					const reportExtensionIssueAction = this._createReportExtensionIssueAction(element);
 					if (reportExtensionIssueAction) {
 						data.actionbar.push(reportExtensionIssueAction, { icon: true, label: true });

The "Report Issue" action label is cut short to be 22x22px 🤷 :

Looks like the default styling of the action items is icon only (codicon) and that comes with a fixed width... I will look into a fic

I got the n uncaught errors message to show in VS Code stable, but I couldn't get it to show in insiders. Maybe I'm holding it wrong though so I will leave someone else to verify.

This bug has been fixed in the latest release of VS Code Insiders!
@usernamehw, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version d0acdbff1f30eac7cc68930013485276d1bae0ec of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

Nope. The button "Report Issue" is still hidden in d0acdbff1f30eac7cc68930013485276d1bae0ec.

Well, there was an attempt... While fixing I just patched the aforementioned lines but @usernamehw what kind of error does your extension throw to be reported there? Can you share your sample?

In activate():
setTimeout(() => {
	throw Error('asdf');
});


Update github unavailable steps

What
Update the docs to include comments around running 2ndline drills

Thanks Ken. Suggest we close this in favour of #3738, which is more of a rewrite and should be easier to follow 👍


create population forecasting model and population endpoint

Population Forecast Endpoint
Location

move population folder from dataset/data to models
viz.py

Details

create population forecast model
create population forecast visualization endpoint

Committed Features

users can see projected population growth

 DS will create an ML model that predicts population growth in a city

 DS will create endpoints for both web and iOS, utlizing the new data sets.


Unable to get MRTK Toolbox

I'm working with Unity v2021.3.17 and MRTK v<IP_ADDRESS>, and when I go to the Mixed Reality menu, I don't see the Toolbox. What do I have to do in order for the MRTK Toolbox to show up?

It turns out I didn't install the tools. I apologize for my oversight.


PicoEditor - no proper logout, if "$config[ 'PicoEditor' ]['url']" not set to default (admin)

public function onRequestUrl(&$url)
{...
// are we looking for admin/logout?
if ($url == 'admin/logout') {
$this->is_logout = true;
}
}
should look like this:
public function onRequestUrl(&$url)
{...
// are we looking for admin/logout?
if ($url == $this->adminUrl . '/logout') {
$this->is_logout = true;
}
}

Thank you for your bug report! Starting with Pico 1.1 we'll ship a brand-new, official admin plugin developed from scratch, so please stay in touch. Please don't be confused that I'll close and label this issue as Invalid - it doesn't affect Pico's core. However, as said, we'll address this in our upcoming new admin plugin. :smiley:


Can't seem to get maxValue working with date

I tried to use maxValue with dates in so many different way, but I can't seem to get it to work.
Here is my fiddle with one version: https://jsfiddle.net/b5v4faqf/1160/
I have tried using moment.js, passing the date as YYYY-MM-DD string or as real date, using maxValue: maxValue(-thedate-) and maxValue: -thedate- and various combinations.
What is the right way to do it? I need to make sure the entered date is in the past or today.

I'm having exactly the same problem.

Looking at the source maxValue expects the two compared values to be instances of Date.
In my case the model value was a string and I already had moment js as a dependency so I wrote my own:
....
validations: {
    date_field: {
        notInFuture: (date_field) => moment(date_field).isSameOrBefore()
    }
    ...

This is tricky. maxValue correctly compares two date objects, but the problem is your date input. Unfortunately, HTML date operates on dates in YYYY-MM-DD string formatting.
You could use computed values to do initial formatting and validate it instead. Check this example:
https://jsfiddle.net/Frizi/b5v4faqf/2889/

It's a little weird that making it a string on both sides still fails to compare correctly.


Pex 2 update?

So it's been close to two year since #1945 was posted (opened on Mar 18, 2015, closed on Apr 16, 2015).  I was wondering if any progress has been made on development or documentation for those of us that are using it.
The only source I have to go on is the ticket 1949 and this post by MoeBoy76 on spongepowered.org from back in July 2015.
I am still sort of stumbling around in the dark with most of what I'm doing when working with Pex2.  I figured there would have been some kind of update by now.

There is a working version that many people are using these days.  I haven't gotten into it myself.  @zml2008 can give you more details though.

There's not really much more detail I can give.  I was just asking if any progress has been made on documentation of PEX 2 in almost two years since the posting of issue 1945.

There's the doc/ folder in the repository, with some basic documentation. The documentation has been expanded somewhat in the feature/segments branch to cover the (currently incomplete) changes made in that branch.

Unfortunately PermissionsEx is currently unmaintained and no further updates are planned. Because of this, all pull requests are being closed. If you are looking for an actively maintained permissions plugin, take a look at LuckPerms. It is compatible with major server platforms and can import data from PEX.


Generated client-go doesn't marshal/unmarshal correctly

Bug description
The json and protobuf tags differ in the generated go code for any fields which are declared snake case in the proto file and all Istio examples/tests use the CamelCase json tag used in the protobuf tag. Here's an example proto field and it's corresponding generated go structs
Note this line:
AllowTls bool `protobuf:"varint,1,opt,name=allow_tls,json=allowTls,proto3" json:"allow_tls,omitempty"`

where the json tag is different inside protobuf tag.
As, the generated go files from proto have custom marshalers defined only if they have a oneof as is the case here. It means that when the generated clientset calls JSON encoding/decoding the fields which are snake case in proto will not work.
We can fix this issue by generating custom Marshal/Unmarshal for the top level Go structs like this.
Affected product area (please put an X in all that apply)
[x] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Related issues
https://github.com/istio/istio/issues/8772

xref: https://github.com/kubernetes/client-go/issues/574


Add build support for MacOS

Modified Makefile to use -r instead of --relocatable so that it can be
used with MacOS linker as well.
Modified test/Makefile not to link with librt under MacOS.
Modified test/benchmark.c to use MACH time functions under MacOS.

Thanks for the contribution! I'm definitely open to adding build support for MacOS (and a little bit surprised that it didn't compile out of the box.) However, I have some comments, which I'll attach to the code.

Regarding compilation out of the box on MacOS, objcopy is not part of the set of tools provided by Apple. One has to install binutils to get that (with brew for example). For some reason, once I used OBJCOPY=gobjcopy (at first, I just commented the line in the Makefile), the linker would complain with some message about architecture subtype. I don't know if it's a bug from gobjcopy, the linker or a misuse of one of them on this platform. If anyone gets that message, defining CFLAGS="-Wl,-allow_sub_type_mismatches" gets rid of this error (but introduces warnings about unused linker flag when doing only compilation).


Neodigm Carousel | Resize Function

If nav = resize then reflow the carousel.
Use Case: The browser resizes or changes orientation.
The actual debounced resize logic exists in the implementation logic not the framework.

This task is complete in v3.1. No breaking logic. The landing page template has been updated. The index.html has the test.


Add a homeDir field in LocalFileSystem for removefile to check

Description
Added this for removeFileIfExists(const std::string& path) to only remove the files under our DBpath
Fixes #4529
Contributor agreement

[x] I have read and agree to the Contributor Agreement.

Let's also add some tests.

Tests are added via c api


Backend not working

The backend server does not start and the LEC neither. I think is because of the backend.

Never mind. I fixed reading others issues, AMAZING APP!.


Client profile

Adding the skeleton of the "client profile management"
Include:

View for a specific client profile
Test that it got the 200 status
Skeleton html for the profile page
Skeleton html for the client profile page (extends profile page)
The client url - /client/profile/"client id"

Resolve issue #106

Can you add a picture of what you made?

@idosinger98 The html file was only to make sure that the views and tests are working well,
I will add it to PR #109 where I made the relevant html file.


Update licensing-fundamentals.md to correct AR feature names

ready-to-merge

#sign-off

@billmath @krbain - merging this change, but I noticed that "Access reviews" is inconsistently capitalized. Sometimes it's "Access Reviews". If these should be consistent, update the article in a follow up PR. thanks


Problems with Repload for 10x data

Hi
I am starting to use Immunarch for 10x sc paired data. Thanks for implementing it!
I have loaded a dataset where my samples are defined by the Column "Cluster".
When i load the dataset, nevetherless, i am not able not split up the samples using your prposed code:
scdata_cl <- select_clusters(scdata_pat, scdata$bc_cluster, "Cluster"),
because i do not appear to have a slot called "bc_cluster" or "bc_patient", as for example your exmaple dataset scdata has.
How can i add this info to my immunarch file?
This is not clear from your vignette.
Thanks a lot for your help.
annecar

Hi, did you manage solving this problem?
Andle

Hi, @annecar and @aleofifi!
My name is Aleksandr Popov, I am a developer of the Immunarch package. Thank you for using our software!
Could you, please, provide an example of data to reproduce this bug? I will then work on an update for Immunarch to fix it.
Best regards,
Aleksandr


Binary Header for Basis?

Hi, is there a good way to determine if a binary file is likely to be a basis file based on inspecting the first few bytes?
It doesn't seem like the files start with an ASCII string (like e.g. binary Draco files start with 'DRACO' ASCII in the first 5 bytes) but they do seem to start with the same few characters.
I didn't find a spec of the format but took a quick peek at the cpp code, looked like the format starts with a header_size that I assume is a fixed value for a certain version of the format?
Are there any other bytes at fixed offsets from the header I should look at?
Background - We are working on integrating a basis loader into loaders.gl and we'd like this loader to participate in our automatic loader selection system.
@georgios-uber @tsherif

If you figure this out you should also submit it to the "file" program so it can correctly detect basis files. Currently it just shows "data".

In Magnum we're checking for a sB prefix. It's very short, but at least something (code for reference). Originally it included also the version info that's right after, but files produced with the current basisu tool already have different version than the kodim20.basis etc. sample files bundled in this repo.

In Magnum we're checking for a sB prefix.

@mosra Thanks, this unblocks us and we'll do the same. And based on your advice we'll save some time by not trying to implement a stronger heuristic using the the version.

If you figure this out you should also submit it to the "file" program so it can correctly detect basis files. Currently it just shows "data".

@turol Good idea! I don't know anything about how to do that, but I suppose that program would have some kind of configuration that would allow for the sB header to be mapped to 'image/basis' or something similar...

I have a horrid feeling it was intended to be "Bs" but the code as written would only write that if run on a big endian machine.

@MarkCallow Ha ha, that makes sense. And thanks for clarifying, at least this confirms that sB is not a potential non-ASCII string that just happened looked like text, and makes us feel better about relying on it.
Maybe just bite the bullet and document that the initial type string is sB written in big-endian ?

So.... it means Basis files produced on little-endian machines are unreadable on big-endian?

@mosra. No. On further study, it would be written as  'sB' on a big-endian machine as well. The value is being initialized in an enum (int) as the value 'Bs' but then ints are always be written out in little-endian order. so it comes out as 'sB'. May it is intentional, if weird.


ios 11.1.1 not to connecting to android

Hi,
using your pearjs webrtc adapter was working flawlessly for cross platform, however recently with the ios update 11.1.1 we have noticed that android to ios is not connecting, whereas ios to browser and android to browser is working fine.  pls help us on this issue

i have updated webrtc-adapter to latest version, expecting that could improve compatibility with browser.


 When opening your CPACS file in TiGL Viewer, you will see a bunch of error messages in the console:

I believe your CPACS file is ill-formed? The eta1 and eta2 coordinates of the inner and outer cell borders need a coordinate and a referenceUID: https://www.cpacs.de/documentation/CPACS_3_3_0_Docs/html/103ad7fc-072e-f9e2-4f26-15266af61839.htm.
Could you please check if fixing this resolves your problem?

Hey, sorry, the CPCAS-file was ill-formated. I just overlooked the referenceUID element... But thank you for your help!

Unfortunately there is still an error with the same method and the now valid CPACS file (Beam_Composite_Export_Test_33_cell.zip):
self = <tigl3.configuration.CCPACSWingCell; proxy of <Swig Object of type 'tigl::CCPACSWingCell *' at 0x000001E2028AEB70> >

    def get_loft(self) -> "PNamedShape":
        r"""
        get_loft(CCPACSWingCell self) -> PNamedShape

        """
>       return _configuration.CCPACSWingCell_get_loft(self)
E       RuntimeError: Cannot associate projected point with (u,v)-coordinates

I confirm this is a bug. The logic in the following code block is wrong, we must also differentiate between lower and upper shell when determining the umin and umax values:
https://github.com/DLR-SC/tigl/blob/3260a54d627932504c4116e57c883dbb302ba2cd/src/wing/CCPACSWingCell.cpp#L731-L740
As a workaround, you could use the new contour coordinate definition:
<cell uID="test_cell">
    <skin>
        <material>
            <compositeUID>SymmetricBalancedIso</compositeUID>
        </material>
    </skin>
    <positioningInnerBorder>
        <!--
        <eta1>
             <eta>0.3</eta>
             <referenceUID>beam_component_0</referenceUID>
         </eta1>
         <eta2>
             <eta>0.3</eta>
             <referenceUID>beam_component_0</referenceUID>
         </eta2>
        -->
        <contourCoordinate>0.3</contourCoordinate>
    </positioningInnerBorder>
    <positioningLeadingEdge>
        <!--
        <xsi1>0.2</xsi1>
        <xsi2>0.2</xsi2>
	-->
        <contourCoordinate>0.2</contourCoordinate>
    </positioningLeadingEdge>
    <positioningOuterBorder>
        <!--
        <eta1>
            <eta>0.7</eta>
            <referenceUID>beam_component_0</referenceUID>
        </eta1>
        <eta2>
            <eta>0.7</eta>
            <referenceUID>beam_component_0</referenceUID>
        </eta2>
        -->
        <contourCoordinate>0.7</contourCoordinate>
    </positioningOuterBorder>
    <positioningTrailingEdge>
        <!--
        <xsi1>0.8</xsi1>
        <xsi2>0.8</xsi2>
	-->
	<contourCoordinate>0.8</contourCoordinate>
    </positioningTrailingEdge>
</cell>

When using the get_loft method with larger models, an unusual behavior occurred: with a small test case (Beam_Composite_Export_Test_32_cell2.xml) with one segment, the get_loft method works for several cells. In a larger test case with several segments (IEA-10.0-198-RWT_CPACS_contour_coordinates.xml), only the cells on the first segment are created correctly. Is that a bug or can only one segment be used with the contour coordinate definition of the cells?
CPACS.zip

@hardt-fa-flb, I opened a new issue, because your last comment exposes an issue unrelated to the first one. I think it might be a bug, but I need to dive deeper.


Container App reacts with "connection reset by peer" when configured with a EC P-384 certificate but works with RSA certificate

This issue is a: (mark with an x)

[X] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Connecting to a Container App using its custom domain results in a "connection reset by peer" when the custom domain is configured to use a EC P-384 certificate. The exact same app works when the custom domain binding is updated to use an RSA certificate. The only difference is the certificate algorithm, everything else is exactly the same (including the custom domain).
Steps to reproduce

In an Azure Key Vault, create two certificates named aca-demo-ne-osk-am-rsa and aca-demo-ne-osk-am as follows:

$Policy = New-AzKeyVaultCertificatePolicy `
    -IssuerName 'Unknown' `
    -SubjectName "CN=aca-demo-ne.osk.am,O=Stupendous Enterprises,C=NL" `
    -DnsName "aca-demo-ne.osk.am" `
    -SecretContentType "application/x-pkcs12" `
    -KeyType RSA `
    -KeySize 2048

Add-AzKeyVaultCertificate `
    -VaultName 'cccjokvacane' `
    -Name 'aca-demo-ne-osk-am-rsa' `
    -CertificatePolicy $Policy

and
$Policy = New-AzKeyVaultCertificatePolicy `
    -IssuerName 'Unknown' `
    -SubjectName "CN=aca-demo-ne.osk.am,O=Stupendous Enterprises,C=NL" `
    -DnsName "aca-demo-ne.osk.am" `
    -SecretContentType "application/x-pkcs12" `
    -KeyType 'EC' `
    -Curve 'P-384'

Add-AzKeyVaultCertificate `
    -VaultName 'cccjokvacane' `
    -Name 'aca-demo-ne-osk-am' `
    -CertificatePolicy $Policy

(Note that the only difference is the KeyType and Curve/KeySize arguments.) Then use whatever method to get the certificates signed by a publicly trusted CA. (I used Let's Encrypt)
2. Create a Container App Environment (mine is VNet-integrated, not sure whether that's relevant)
3. Assign it a User Assigned Managed Identity (UAMI)
4. Make sure that the Container App Environment can reach and has permissions to the Key Vault.
5. Add the two certificates to the Container App Environment by using the "import from Key Vault" option in the Azure Portal.
6. Create a Container App. (For testing I used the quickstart image)
7. Open a console on the container app, execute curl <url of the app> and note that it works (you get the quickstart page)
8. Add a Custom Domain to the Container App using the "Bring your own certificate" method. For domain name, I've used aca-demo-ne.osk.am as that's what I used for testing.
9. Select the certificate named aca-demo-ne-osk-am.
10. When the certificate is added, open a console again and execute curl -k https://aca-demo-ne.osk.am/. Note that curl immediately reports "connection reset by peer".
11. On the Custom Domains page of the app, update the binding to use the other certificate (aca-demo-ne-osk-am-rsa).
12. Now go back to the console and try the same curl command again. It now works.
I noticed the same behavior from outside the Container App Environment. I've created an Application Gateway in the ACA Vnet, and when the app is configured with the EC-P384 certificate my backend probe fails with the message that a connection to the backend can't be established. When the app is configured to use an RSA certificate, the probe works.
Also note that the EC P-384 itself certificate is perfectly fine: I configured it on the frontend of the Application Gateway where it just does what it should.
Expected behavior [What you expected to happen.]
A clear error message when a certificate is supplied that the Container App Environment can't work with.
Actual behavior [What actually happened.]
No errors or warnings are reported when configuring the certificate, but incoming connections to a container app are simply cut off by resetting the TCP connection.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Ex. Did this issue occur in the CLI or the Portal?

P-384 is currently unsupported by our ingress proxy. We'll investigate whether we can add it. We do support P-256.

@anthonychu Thank you, but the problem is not that P-384 isn't supported. The problem is that it is possible to install unsupported certificates without any error message at all and that connections to the app are then silently dropped, also without any notification or error message. This, combined with the fact that the documentation doesn't mention which certificate types are supported and which aren't makes this something that people can waste hours on trying to debug.

Thank you for clarifying this @jurjenoskam. Makes sense, we will be adding this. Labeled accordingly.