Fix for the wrong value of the "additionalProperties" property

During the certification process testing, the response json deserialization was failing because of the wrong "additionalProperties" property value in the swagger definition. That property value has now been corrected.

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @MarioINSA,
Could you please resolve the conflict?

Hello @MarioINSA,
Could you please resolve the conflict?

Hi @vmanoharas ,
the conflicts have been resolved.
Kind regards,
Mario


Azure Identity Samples Issues

Section Link

Reason:
No import dotenv, cannot use sample.env file.
Suggestion:
Add the follow code:
import * as dotenv from "dotenv";
dotenv.config();

@jongio for notification.

This should be fixed by: https://github.com/Azure/azure-sdk-for-js/issues/10684

@sadasant It has sample.env now, but the dotenv package was not imported and used in the sample - authorizationCodeSample.ts.

This should be fixed now with #12800


Add chinese/japanese fonts support

Installing fonts-takao and fonts-wqy-zenhei adds support for japanese and chinese charactersets.

Thanks @quentinfayet


fix(webpack-transpiler): Correctly use in-memory files

Correctly use in-memory files. Instead of caching them ourselves, extend webpack's CachedInputFileSystem and rely on that for caching files. Still use memory-FS to load the in-memory files.
One downside is that I had to set the typescript target to "es2015", which means we have to drop support for node-4 on the stryker-webpack-transpiler package.
Fixes #597

Great review. Didn't know what I was thinking, fixing that tomorrow

Hmm, breaking compatibility with node4 seems reasonable since its LTS will end in April.


JustEnoughAdministration DSC: Unable to set GroupManagedServiceAccount

Hi,
On a Windows Server 2012 R2 with WMF 5.1:
Name                           Value
----                           -----
PSVersion                      5.1.14409.1012
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14409.1012
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

I successfully setup a JEA endpoint with the following DSC configuration:
Configuration JeaTest
{
    Import-DscResource -Module JustEnoughAdministration, PSDesiredStateConfiguration

    JeaEndpoint Endpoint
    {
        GroupManagedServiceAccount = 'UNDISLAB\gMSA-Undis02'
        EndpointName = "JEA"
        RoleDefinitions = "@{
            'UNDISLAB\jnury' = @{ RoleCapabilities = 'Maintenance' }
        }"
    }
}

JeaTest
Start-DscConfiguration .\JeaTest -Force -Wait -Verbose

But Test-DSCConfiguration returns False:
VERBOSE: [UNDIS02]:                            [[JeaEndpoint]Endpoint] GroupManagedServiceAccount not equal: UNDISLAB\gMSA-Undis02$
VERBOSE: [UNDIS02]: LCM:  [ End    Test     ]  [[JeaEndpoint]Endpoint] False in 0.3900 seconds.
VERBOSE: [UNDIS02]: LCM:  [ End    Resource ]  [[JeaEndpoint]Endpoint]
VERBOSE: [UNDIS02]: LCM:  [ End    Test     ]     Completed processing test operation. The operation returned False.

If I modify the config as the following (to match the Get-DSCConfiguration value):
    JeaEndpoint Endpoint
    {
        GroupManagedServiceAccount = 'UNDISLAB\gMSA-Undis02$'
        EndpointName = "JEA"
        RoleDefinitions = "@{
            'UNDISLAB\jnury' = @{ RoleCapabilities = 'Maintenance' }
        }"
    }

The Test-DSCConfiguration returns True, but the endpoint doesn't work .
Enter-PSSession from another host fails with error: The verification of the Managed Service Account failed with error 1326
That's because the RunAsUser is misspelled:
PS C:\Users\administrator.UNDISLAB\Desktop> Get-PSSessionConfiguration -name JEA | fl *

GroupManagedServiceAccount    : UNDISLAB\gMSA-Undis02$
...
RunAsUser                     : UNDISLAB\gMSA-Undis02$$

Note that I get exactly the same behavior if I manually register PSSessionConfiguration with the trailing $ sign in GroupManagedServiceAccount parameter.
Hope it's clear ...
Have a nice day.

Great catch, Julien. The main rationale for using RunAsUser is that GroupManagedServiceAccount is not a valid member of a PSSessionConfiguration object if there isn't one configured. However, since PowerShell gracefully handles accessing non-existent properties, your proposed change seems fine.
RunAsUser also addressed an edge case where a static runas user (regular AD user account) was configured for an endpoint. It would trigger the DSC test to fail and re-deploy the endpoint using the correct runas account based on the DSC config. Since you can't really end up in this state with an endpoint deployed with our DSC configuration (only if you manually modify the endpoint), I don't see this as a huge concern.

@jnury this merge caused test() to incorrectly return false when GMSA is not set, my mod:
block not entered if both $currentInstance.GroupManagedServiceAccount & $this.GroupManagedServiceAccount null, this matters because $null -replace '$$', '' will return an empty string not $null

if (($currentInstance.GroupManagedServiceAccount) -or ($this.GroupManagedServiceAccount)) {
                if($currentInstance.GroupManagedServiceAccount -ne ($this.GroupManagedServiceAccount -replace '\$$', ''))
                {
                    Write-Verbose ('GroupManagedServiceAccount not equal: "{0}"' -f $($currentInstance.GroupManagedServiceAccount))
                    return $false
                }
            }

@djwork you are absolutely right ! I'll fill a new issue and submit a PR.

Correction as proposed by @djwork. Thank you !


serializes payloads

updated wrapper.py to serialize the payloads in the various methods as per the latest webportal api changes.

LGTM!


Fix PFW core KlocWork issues

PFW core contains some critical KW issues
of type NPD.FUNC.MUST .
This patch fixes these KW issues by adding a check
on pointers address before their use.
Signed-off-by: wzairix<EMAIL_ADDRESS>

[ ]   File: parameter/DomainConfiguration.cpp:L397-407
 I must say I do not really like this way of fixing this. With your way, instead of segfaulting the code will behave erratically in case of error. Could getAreaConfiguration avoid returning NULL ? Maybe we could call explicitly abort() instead of assert(0) in getAreaConfiguration ?
 Refer to getAreaConfiguration() function implementation we cannot avoid the NULL return since the function should retrun an address. But i agree with you  about system behaviour in error case.  We can abort in  to getAreaConfiguration() avoid this case.
 I think that if we abort, or use some kind of "allways_assert". There is some patches in the pipe to introduce such assert. Management was very clear not to introduce workaround for klockwork false positive. Ie adding an assert (or an initialization) is OK but modifying the code flow should be avoided.
[ ]   File: parameter/DomainConfiguration.cpp:L498-510
 As far as y know, replacing the assert by abort as you did should remove the need for those pChildAreaConfiguration != NULL tests. Could you confirm/infirm ?
[ ]   File: parameter/DomainConfiguration.cpp:L604-611
 To my opinion those 2 assert -> abort substitution should be enough to avoid klockwork critical.
[ ]   File: parameter/DomainConfiguration.cpp:L37-44

#include <cstdlib>

[x]   General Comment
 :+1: Thanks for this pull request. Will be merged after @OznOg review.

This pull request is now managed with CodeReviewHub. Refresh.

:+1: Thanks for this pull request. Will be merged after @OznOg review.

:+1:

:-1:


Provide an ability to disable the array length field

Basically following the same pattern as we did for default field. By
default the array length fields are enabled and this change allows us
to disable it by adding the following field in the design doc.
index_array_lengths=false
This is a short term solution until we revisit the cloudant query design. See FB case 49429 and 54260 for further details.

Please address comments and update the PR, then I'll take another look at it.

@rnewson Made the suggested changes and tested them. Thanks for the feedback and suggestions.

@davisp @tonysun83 please read the substantive change. It looks good to me but you're more familiar.
@brkolla nice work, this will need squashing to one commit before merge but please wait for @davisp's +1.

(and obviously fix that variable name to conform to erlang conventions)

Thanks @rnewson. Will change the name.

Reviewed. Looks fine except for the few style and unneeded things I commented on.
However, this patch and the investigation from 49429 lead me to believe that we're either not completely understanding the issue or we're covering up an entirely different bug than we think. From what I've read it seems as though disabling the array length field drops the RAM usage by an order of magnitude. However, by definition adding an array length field can at most double the total number of fields (if every field were an array) which is obviously not the case. Granted there may be a close to doubling if there are lots of arrays of course. But 2x fields vs 10x in RAM usage strikes me as a sign that something else is going on here that we're not quite capturing.

Figured this out chatting with @brkolla OOB. Turns out the particular database/index in question had a custom short list of fields specified. The bug that we're actually looking at is that we included array lengths even for fields that have indexing disabled. I'd like to get a PR/fix for that bug before we call this done.
Granted, this is still a reasonable feature patch so I'm still +1 to including it after the changes I noted above.

@davisp I have made the suggested changes about style and removing no-op changes. I agree with you about disabling the indexing array length fields if they are not indexed. But we should also need to revisit (IMO) the design of how we currently create unique fields and make sure that we only create fixed number of them irrespective of the data shape and indexing rules.
I will deliver these changes tomorrow, once I make the changes in the dbnext repository.


Need bidirectional communication channel between netavark and aardvark

Right now our dns startup is super flaky causing many flakes in CI that are only solved by using retries. This is bad and often not what users are doing. Sending signals is just not reliable. Netavark sends the signal on a update but then never wait for aardvark-dns to actually update the names and be ready to respond to the new name. The same goes for error handling aardvark-dns logs its errors to journald but there is absolutely no way right now to get this error back to netavark and thus podman. A common problem is that port 53 is already bound causing aardvark-dns to be up and running but unable to serve any dns.
There are a lot of dns related issues on the podman issue tracker, most not really possible to debug. IMO we have to address this situation.
Of course one important caveat is that we must stay backwards compatible. I am creating to have a discussion about it so we can find a good solution for this.
cc @baude @mheon @flouthoc

Yes, I just want something were can make sure netavark won't return until aardvark-dns is ready and if there was an error we should get it back.

I would prefer not to drag a full REST API in, so I wonder if we can't do something a little lighter (protobuf, maybe? Does that have good rust bindings?)
The idea in general seems sound, and could serve to enable additional features in the future (we've talked about having Aardvark listen for DBus events and launch Netavark when the firewall reloads, and bidirectional comms could be useful for that)

I don't care about the protocol, protbuf would work we use it already for the dhcp proxy so it is not a new dependency for netavark.
But honestly I think it right now a simple string based API would be enough assuming we keep the current way of writing entries to file.

xref: https://github.com/containers/podman/issues/18325

xref: https://github.com/containers/podman/issues/16272

I didn't even bother linking issues, I could properly link 20+ issues from the podman repo that may not be fixed by this but at least can be diagnosed by the users.
Common error is having something listening on port 53.
$ sudo nc -u -l 53
$ sudo podman run --network podman1 --rm alpine nslookup google.com
nslookup: write to '<IP_ADDRESS>': Connection refused
;; connection timed out; no servers could be reached

That is what the user sees, dns not working but they don't know why.
The only real clue is in the journal but most people will never check that:
aardvark-dns[34502]: Unable to start server unable to start CoreDns server: Address already in use (os error 98)

The goal here would be to have the podman run command error out with the aardvark error.


Visual Tracking

http://picamera.readthedocs.org/en/release-1.9/recipes2.html#custom-outputs
The last example.
Could be modified to give something moving left / right. Obviously the bot would need to be still for that to make sense.


Accessing kubernetes service outside the cluster

I have deployed kubernetes cluster using make deploy command mentioned in README.md.
Nginx is deployed on top this cluster and exposed using NodePort service. If I try to hit the nodeIP:nodePort I get connection refused error.
I can see service is listening on nodePort.
netstat -tulpn | grep 30302
tcp6       0      0 :::30302                :::*                    LISTEN      7168/./kube-proxy

Hitting the endpoint from same machine results in same error.
curl -v http://localhost:30302
* Rebuilt URL to: http://localhost:30302/
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* connect to <IP_ADDRESS> port 30302 failed: Connection refused
* Failed to connect to localhost port 30302: Connection refused
* Closing connection 0
curl: (7) Failed to connect to localhost port 30302: Connection refuse

Creating service of type 'LoadBalancer' remains in pending state.
What is the right way to access deployed kubernetes workloads outside the cluster ? Are there any additional steps required apart from creating a kubernetes service?

Hi @vishal-biyani,
For your nginx container, can you share details on the Kubernetes manifests, and image you used?  It looks like kube-proxy is routing the traffic, but perhaps your nginx container is listening on a different port than what you have specified.
For your LoadBalancer question, we have not enabled the cloud-provider flags on Kubernetes yet (this is coming shortly), so an external load balancer will not be configured.
At this time, the best way to expose a Kubernetes application is to use the NodePort.  We have an example here: https://github.com/mesosphere/dcos-kubernetes-quickstart/blob/master/examples/os-detector/os-detector.md

hey @vishal-biyani I have made a demo hoe to expose your web apps externally https://github.com/rimusz/dcos-k8s-beer-demo
Check from https://github.com/rimusz/dcos-k8s-beer-demo#helm-setup part

@smugcloud can you please add documentation to our public docs about how a user can access a Service from outside of the cluster? Ping me and let's sync because while I have all the possible solutions on my mind, time is really short for me.

Hi @vishal-biyani, we have updated documentation that walks through exposing services.  Currently, we support the AWS cloud-provider (configuration flag at Kubernetes package installation), or an Ingress resource.
Let me know if you run into any issues.

Is it possible to expose nodeport service via marathon-lb ? If yes , can you please provide a link which can help me understand, how a nodeport service can be exposed via marathon-lb ?


Error message: Unable to satisfy constraints on package

The below error is ambiguous and actually isn't helping resolve an issue let alone even hinting it. Its just an issue of typo or wrong version number being pinned in an environment.
[ec2-user@ip-172-31-163-152 ~]$ sudo chef-client
Starting Chef Client, version 12.0.3
resolving cookbooks for run list: ["learn_chef_httpd", "learn_chef_httpd::vhosts", "chef-client", "chef-client::delete_validation"]
================================================================================
Error Resolving Cookbooks for Run List:
Cookbook dependency resolution error:
Error message: Unable to satisfy constraints on package cron, which does not exist, due to solution constraint (chef-client >= 0.0.0). Solution constraints that may result in a constraint on cron: [(chef-client = 4.2.0) -> (cron >= 1.2.0)]
You might be able to resolve this issue with:
1-) Removing cookbook versions that depend on deleted cookbooks.
2-) Removing unused cookbook versions.
3-) Pinning exact cookbook versions using environments.
Expanded Run List:

learn_chef_httpd
learn_chef_httpd::vhosts
chef-client
chef-client::delete_validation

Running handlers:
[2015-02-11T00:46:24-05:00] ERROR: Running exception handlers
Running handlers complete
[2015-02-11T00:46:24-05:00] ERROR: Exception handlers complete
[2015-02-11T00:46:24-05:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
Chef Client failed. 0 resources updated in 1.73937474 seconds
[2015-02-11T00:46:24-05:00] ERROR: 412 "Precondition Failed"
[2015-02-11T00:46:24-05:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

So the error message comes back from the server, so at best this is a server bug and is misclassified.
This is also a "hard" problem in computer science.  We are unlikely to take this work on.  We are also trying to phase out the server-side depsolver so the large effort would be eventually thrown away.


Add option to purposefully limit isolate communication to primitive types

In some environments, we can only send primitive types listed under SendPort.send across isolates. This essentially means that code using isolate apis, especially libraries, must assume that only those primitive types are supported.
However, it's easy to accidentally send an object that's not supported on all platforms. Since most environments support sending any object over isolates, these mistakes can be difficult to detect in tests.
It would be great if we could instruct the Dart VM to only support primitive types, even for the scenarios where isolates share code and run in the same process. Perhaps this could be some per-isolate flag enabled through dart:developer or a command-line option.

@simolus3 is it because you want portability between Web and native?
/cc @aam @mkustermann this is relevant in the context of concurrency work you are doing for the VM - because that introduces yet another discrepancy in capabilities between Web and Native, so it might be good to introduce a way to limit oneself to a portable subset of functionality.

Is this due to an isolate either being spawned either via Isolate.spawn() or Isolate.spawnUri() and it therefore doesn't know what kind of messages it can send to the spawner isolate?
We could extend the API to have an optional {bool onlyAllowBasicTypes} parameter which would perform more strict validation.

is it because you want portability between Web and native?

I wasn't targeting isolates for web, but I ran into some issues with Flutter. I'm maintaining a Dart library that uses isolates and some users reported that this didn't work in Flutter release builds. The main problems were not being able to send a Dart object only wrapping primitive types (e.g. https://github.com/simolus3/moor/issues/517) and switch statements not matching enums sent over isolates (https://github.com/simolus3/moor/issues/514). I couldn't reproduce either problem with dart2native, but to me it looks like a restriction to primitive types is needed for reliable isolate communication.

spawned either via Isolate.spawn() or Isolate.spawnUri()

I'm always using Isolate.spawn.

optional {bool onlyAllowBasicTypes} parameter which would perform more strict validation.

Something like that would be great!

I'm maintaining a Dart library that uses isolates and some users reported that this didn't work in Flutter release builds. The main problems were not being able to send a Dart object only wrapping primitive types (e.g. simolus3/moor#517) and switch statements not matching enums sent over isolates (simolus3/moor#514).

These bugs look really strange. First of all I would expect that behaviour in Flutter release builds and dart2native behaviour should be identical around these issues.
Then I would actually expect enums to pass through SendPort in the right way. Enum instances are constants - so they should be canonicalised on when passing through the port and arrive as respective enum instance on the receiving side.
Similarly if you could pass MoorIsolate in dart2native tests I would expect you to be able to pass it on the Flutter release builds as well.
These both look bugs which are worth investigating on our end.
This also highlights that error we are getting when trying to send a bad object is rather uninformative. We should definitely fix that.

I'd like to start using Enums in cross Isolate communication, so could I confirm that the root cause of this issue turned out to be the hashing code bug fixed for #41824 ? Or is there some other restriction around using enums with SendPort .send that I should be careful of?


Does ARcore 1.6 support Vulkan in UE4? Or when can I support Vulkan?

I am using ARcore 1.6 to use Vulkan rendering in UE4. When I release the app, it flashes back. This makes me confused. I want to know if AR core 1.6 supports Vulkan rendering in UE4, or when can it be supported? I really need this Vulkan rendering because it gives me a better rendering effect, I hope you can answer me, thank you!

Hi, I've tried to use Vulkan API for gogole ARCore in UE4. It doesn't work. I'm using Samsung Galaxy S9+ btw. Any help will be appreciated. Thanks.


Support Streaming

Lingo should work with streaming. We should check to see if it does today and if not, update lingo. Also requires validating that our example backends support streaming.
https://platform.openai.com/docs/api-reference/streaming

vLLM supports streaming with the openai compatible backend that we serve. I can't remember if I tested this already or not, but agree we should test and ensure it works. Ideally we write a system test case for it?

I tested this a while ago, it works :partying_face:


Some gifs of character have strange colors

Thank you for your powerful program.I want to get some gifs of characters,but some gifs' color is different from the sprite pics.
There is nothing wrong with pixelart/animated/special.gif,but pixelart/animated/walk_front.gif and pixelart/animated/walk_back.gif  have strange colors.
The pictures below show the difference.The picture on the left is from pixelart/animated/walk_front.gif,the picture on the left is from pixelart/sprite_sheet/pixelart0050.png.
The gif looks noticeably dimmer.

Hmm, I think I messed up with default gif creation arguments in last update. I'll look into the issue this weekend.
Can you try the animate command with scale option to see if it outputs with better color palettes?
If you used delta extraction for 1.641.30 put
animate  -delta 1.641.30 -scale 4 -character psychic_yuki_ny23
otherwise
animate -scale 4 -character psychic_yuki_ny23
let me know if it temporarily resolves the issue, thanks!

Thank you for your quick reply.I tried the command and it works.The colors are now displayed correctly.

Closed by #21


silx.gui.plot.PlotWidget: Fixed rendering; Doc: Updated sample code.

This PR does 2 things:

Enables usage of image mask in tools (histogram, colormap autscale and profiles) in examples/stackView.py.
Fixes a bug in text rendering in the matplotlib backend when the widget as a width of 0.

To reproduce the bug, use the examples/stackView.py from this PR, create a horizontal profile and then open the mask tools.
This leads to:
Traceback (most recent call last):
  File "silx/gui/plot/backends/BackendBase.py", line 343, in postRedisplay
    plot.replot()
  File "silx/gui/plot/PlotWidget.py", line 3137, in replot
    self._backend.replot()
  File "silx/gui/plot/backends/BackendMatplotlib.py", line 1517, in replot
    self.draw()
  File "silx/gui/plot/backends/BackendMatplotlib.py", line 1501, in draw
    self._drawOverlays()
  File "silx/gui/plot/backends/BackendMatplotlib.py", line 1014, in _drawOverlays
    axes.draw_artist(item._backendRenderer)
  File "matplotlib/axes/_base.py", line 2762, in draw_artist
    a.draw(self.figure._cachedRenderer)
  File "silx/gui/plot/backends/BackendMatplotlib.py", line 339, in draw
    self.text.draw(*args, **kwargs)
  File "silx/gui/plot/backends/BackendMatplotlib.py", line 268, in draw
    return Text.draw(self, renderer)
  File "matplotlib/artist.py", line 41, in draw_wrapper
    return draw(artist, renderer, *args, **kwargs)
  File "matplotlib/text.py", line 681, in draw
    bbox, info, descent = textobj._get_layout(renderer)
  File "matplotlib/text.py", line 274, in _get_layout
    key = self.get_prop_tup(renderer=renderer)
  File "matplotlib/text.py", line 842, in get_prop_tup
    x, y = self.get_unitless_position()
  File "matplotlib/text.py", line 824, in get_unitless_position
    x = float(self.convert_xunits(self._x))
  File "silx/gui/plot/backends/BackendMatplotlib.py", line 307, in convert_xunits
    return self.__get_xy()[0]
  File silx/gui/plot/backends/BackendMatplotlib.py", line 300, in __get_xy
    invtrans = trans.inverted()
  File "matplotlib/transforms.py", line 2377, in inverted
    self._b.inverted(), self._a.inverted())
  File "matplotlib/transforms.py", line 2377, in inverted
    self._b.inverted(), self._a.inverted())
  File "matplotlib/transforms.py", line 1822, in inverted
    self._inverted = Affine2D(inv(mtx), shorthand_name=shorthand_name)
  File "<__array_function__ internals>", line 6, in inv
  File "numpy/linalg/linalg.py", line 546, in inv
    ainv = _umath_linalg.inv(a, signature=signature, extobj=extobj)
  File "numpy/linalg/linalg.py", line 88, in _raise_linalgerror_singular
    raise LinAlgError("Singular matrix")
numpy.linalg.LinAlgError: Singular matrix

Any idea if there is a typo in the xunit here?
        x = super(_TextWithOffset, self).convert_xunits(self._x)
        y = super(_TextWithOffset, self).convert_xunits(self._y)

Not sure, but it really looks like it indeed, nice catch!

It's a typo, but it has no effect if the argument is a Number which is the case here.
There seems to also be another typo: _TextWithOffset.convert_xunits and _TextWithOffset.convert_yunits ignores their argument and use self._x (self._y) instead... which has the same value here.

I see be careful, i have designed this stuff. I think it's a trick to make it work as i expect.

Yes, I won't touch this now, it indeed sounds tricky.


BIGTOP-3485. Bump Solr to v8.7.0.

https://issues.apache.org/jira/browse/BIGTOP-3485

Solr 8.7.0 contains SOLR-14663.
solr-uima was removed by SOLR-11694.
logging library was migrated.

Packaging and Puppet manifests worked on CentOS 8.
$ ./gradlew solr-clean solr-pkg repo
$ cd provisioner/docker
$ ./docker-hadoop.sh --create 1 --image bigtop/puppet:trunk-centos-8 --memory 16g --repo file:///bigtop-home/output --disable-gpg-check --stack zookeeper,hdfs,solrcloud

I got error on the smoke-tests. I'm looking into the cause.
> Task :bigtop-tests:smoke-tests:solr:test FAILED

org.apache.bigtop.itest.solr.smoke.TestIndexing > classMethod FAILED
    java.io.IOException: Server returned HTTP response code: 400 for URL: http://localhost:8983/solr/smoke/update/json?stream.body=[{"id":"doc1","name":"first+test+document"},{"id":"doc2","name":"second+test+document"}]&commit=true&wt=json
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1900)
        at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1498)
        at sun.net.www.protocol.http.HttpURLConnection$getInputStream.call(Unknown Source)
        at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCall(CallSiteArray.java:47)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:115)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:119)
        at org.apache.bigtop.itest.solr.smoke.SolrTestBase.doReq(SolrTestBase.groovy:65)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:498)
        at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:104)
        at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:326)
        at groovy.lang.MetaClassImpl.invokeStaticMethod(MetaClassImpl.java:1491)
        at org.codehaus.groovy.runtime.callsite.StaticMetaClassSite.callStatic(StaticMetaClassSite.java:64)
        at org.codehaus.groovy.runtime.callsite.CallSiteArray.defaultCallStatic(CallSiteArray.java:55)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:196)
        at org.codehaus.groovy.runtime.callsite.AbstractCallSite.callStatic(AbstractCallSite.java:208)
        at org.apache.bigtop.itest.solr.smoke.TestIndexing.before(TestIndexing.groovy:30)

-Dlog4j.configuration=file:///etc/solr/conf/log4j.properties must be updated
ERROR StatusLogger Unable to access file:///etc/solr/conf/log4j.properties
 java.io.FileNotFoundException: /etc/solr/conf/log4j.properties (No such file or directory)

got another error. https://github.com/apache/lucene-solr/pull/2304/files says we need SOLR_OPTS=-XX:-UseLargePages.
OpenJDK 64-Bit Server VM warning: Failed to reserve shared memory. (error = 1)

# curl -G --data 'stream.body=[{"id":"doc1","name":"first+test+document"},{"id":"doc2","name":"second+test+document"}]&commit=true&wt=json' http://localhost:8983/solr/smoke/update/json
{
  "error":{
    "metadata":[
      "error-class","org.apache.solr.common.SolrException",
      "root-error-class","org.apache.solr.common.SolrException"],
    "msg":"Stream Body is disabled. See http://lucene.apache.org/solr/guide/requestdispatcher-in-solrconfig.html for help",
    "code":400}}

https://solr.apache.org/guide/8_8/requestdispatcher-in-solrconfig.html describes the way to update the required configuration.
# curl http://localhost:8983/solr/smoke/config -H 'Content-type:application/json' -d '{"set-property": {"requestDispatcher.requestParsers.enableRemoteStreaming": true}, "set-property": {"requestDispatcher.requestParsers.enableStreamBody": true}}'
{
  "responseHeader":{
    "status":0,
    "QTime":323},
  "WARNING":"This response format is experimental.  It is likely to change in the future."}

# curl -G --data 'stream.body=[{"id":"doc1","name":"first+test+document"},{"id":"doc2","name":"second+test+document"}]' --data 'commit=true' --data 'wt=json' http://localhost:8983/solr/smoke/update/json
{
  "responseHeader":{
    "rf":1,
    "status":0,
    "QTime":242}}

Smoke test of Solr passed.
$ ./docker-hadoop.sh --create 1 --image bigtop/puppet:trunk-centos-8 --memory 16g --repo file:///bigtop-home/output --disable-gpg-check --stack zookeeper,hdfs,solrcloud --smoke-tests solr
...(snip)
Now testing...
:bigtop-tests:smoke-tests:solr:test (Thread[Execution worker for ':' Thread 4,5,main]) completed. Took 8.27 secs.

BUILD SUCCESSFUL in 55s
35 actionable tasks: 7 executed, 28 up-to-date
Stopped 1 worker daemon(s).
+ rm -rf buildSrc/build/test-results/binary
+ rm -rf /bigtop-home/.gradle

The smoke-tests passed on Ubuntu 18.04 too.
$ ./docker-hadoop.sh --create 1 --image bigtop/puppet:trunk-ubuntu-18.04 --memory 16g --repo file:///bigtop-home/output/apt --disable-gpg-check --stack zookeeper,hdfs,solrcloud --smoke-tests solr
...
:bigtop-tests:smoke-tests:solr:test (Thread[Execution worker for ':' Thread 2,5,main]) completed. Took 8.759 secs.

BUILD SUCCESSFUL in 54s
35 actionable tasks: 9 executed, 26 up-to-date
Stopped 1 worker daemon(s).
+ rm -rf buildSrc/build/test-results/binary
+ rm -rf /bigtop-home/.gradle

Thanks, @iwasakims . I've verified this PR with ubuntu-18.04 on Arm. There are still some references of solr-uima in bigtop-tests/test-artifacts/package/src/main/resources/{apt,yum,zypper}/solr*.xml. It would be good to also remove these references.
And pls squash these commits so it could be one record in master. Thanks.

Thanks, @JunHe77. I removed the references from packaging test resources. While the resources (of all products) are not updated at all for years, I left other part as is. I will file a JIRA issue to update or remove those ignored test resources. I prefer removing to updating because manually maintaining list of all files of all packages does not make sense at all.

And pls squash these commits so it could be one record in master. Thanks.

Yeah, I will do it by putting "Squash and merge" button of GitHub. (I usually do it when I review and merge PRs. It is the only allowed option on Hadoop development, FWIW. Other buttons are disabled by infra settings.)

+1, also tested on Fedora 33. Thanks @iwasakims!


coprocessor: Refine LazyBatchColumn to use BufferVec

What have you changed? (mandatory)
This PR utilizes BufferVec in LazyBatchColumn, which was added in https://github.com/tikv/tikv/pull/4790 . Also some useless interfaces are removed.
It fixes the performance regression when there are large integers, as well as other cases where datum is small but larger than 10 bytes.
What are the type of the changes? (mandatory)

Improvement (change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
Existing tests.

/run-integration-tests

/run-integration-tests

/rebuild

/run-integration-tests

/run-integration-tests

@breeswish
I suggest you should paste the performance result in the PR to let other outside contributors know the progress.

IMO, if we do a performance improvement, we must paste the result in the PR.

@siddontang Got it. Will be updated later when I do the next benchmark.


High Sierra: more initial support

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully run brew tests with your changes locally? (On macOS 10.12, not 10.13)

One small note:
I trimmed Libs.private in libcurl.pc because I'm pretty sure it's wildly outdated & has been for quite some time. In the 10.12 libcurl.pc for example we have things like -lssl -lcrypto which is almost certainly not true these days since curl links to macOS' SecureTransport.
Reading the libcurl.pc file from Homebrew's standard, non-option, bottle-poured curl I get Libs.private: -lldap -lz so that's what I've trimmed the line to 🤷♂.

Thanks again @DomT4!

A random aside: have you upgraded a VM or your actual machine and if the latter: do you consider it usable for day-to-day use?

I have two High Sierra installs running locally currently 😅. I sliced a partition out of my SSD to test APFS a little bit, because when I tried to upgrade to High Sierra inside a VMware box with APFS transition enabled it borked the whole thing & refused to boot, which they're aware of.
I use the VMware box for testing primarily because it means I'm not having to reboot my whole system to check anything. In terms of stability, even running natively on the SSD rather than trapped inside VMware it's not super stable yet, as expected really I guess.
I ended up swapping out Safari for Chrome because Safari kept turning websites completely black, persisting through page refreshes, as an example. There's nothing I've encountered yet that will absolutely ruin your day, but a fair bit that might make you sigh & go hunting for a workaround.
I'm planning to not upgrade my main partition for a little while though; I'm kinda concerned how third-party apps are going to handle APFS.

I had HighSierra installed on an external SSD (upgraded from Sierra) and during about half a day not crashed anything, just worked, but I had not enabled APFS, I was not prompted during install or anything, how did you change to APFS in your VM?
thanks

I was not prompted during install or anything, how did you change to APFS in your VM?

There's a little checkbox & brief explanation "page" in the Install macOS 10.13 Beta.app process that asks whether you want to convert your filesystem. The release notes note that APFS has some "unsupported configurations" though so perhaps you hit one of the exclusions.
I think I can talk about those here without angering the Apple gods given everyone else in the world is:

HDD-only Macs currently are not converted to APFS by default.
Some iMacs with 3TB Fusion drives and BootCamp may be unsupported for use with
APFS.
APFS support for Mid 2012 Mac Pro will be included in an upcoming seed.

Hm not sure any of those exceptions apply but will see how the next versions behave
Thanks

Thanks @DomT4!


New Item: Timothy Hay

(what they actualyl eat)

Make sure they can eat plenty of veggies too! And treats (usually made out of fruits and timothy hay)


CSS ${} theming

I use visual studio and the main problem that I came across when developing is using ${} syntax in CSS, what causes red underlines suggesting incorrect syntax. Is there any extension for vs code to fix this? Thanks.

There's already an official Aurelia VSCode extension, which ships with a custom Aurelia HTML syntax. Perhaps we can add a custom Aurelia CSS syntax to the extension as well?
https://github.com/aurelia/vscode-extension/

I'm going to close this and open an issue on the vs code plugin. I'll try to talk to someone on the VS tools team at Microsoft about adding support for this in the VS Aurelia plugin.


Need to Suppress or Resolve: The 'Microsoft.PowerShell.Management' module was not imported because the 'Microsoft.PowerShell.Management' snap-in was already imported.

Details of the scenario you tried and the problem that is occurring
The Integration test logs are full of the below Warning messages. In the most recent Integration test job, there were 189 instances of this. This does not occur in the Unit tests however. This also does not occur when I run the Integration tests on my own Windows 2019 machine.
The 'Microsoft.PowerShell.Management' module was not imported because the 'Microsoft.PowerShell.Management' snap-in was already imported.
Verbose logs showing the problem
A bigger example from https://ci.appveyor.com/project/PowerShell/xpsdesiredstateconfiguration/builds/23613517/job/7ec82b42naour8iy:
[00:02:00]   Describing MSFT_xDSCWebService_Integration
[00:02:00]
[00:02:00]     Context When using configuration MSFT_xDSCWebService_PullTestWithSecurityBestPractices_Config
[00:02:01] The 'Microsoft.PowerShell.Management' module was not imported because the 'Microsoft.PowerShell.Management' snap-in was already imported.
[00:02:03]       [+] Should compile and apply the MOF without throwing 2.94s
[00:02:04]       [+] Should be able to call Get-DscConfiguration without throwing 178ms
[00:02:04]       [+] Should return $true when Test-DscConfiguration is run 78ms
[00:02:04] The 'Microsoft.PowerShell.Management' module was not imported because the 'Microsoft.PowerShell.Management' snap-in was already imported.
[00:02:06]       [+] Should compile and apply the MOF without throwing 2.65s
[00:02:07]       [+] Should be able to call Get-DscConfiguration without throwing 188ms
[00:02:07]       [+] Should return $true when Test-DscConfiguration is run 273ms
[00:02:07]       [+] Should create a web.config file at the web site root 9ms
[00:02:08]       [+] Should create a firewall rule for the chosen port 1.17s
[00:02:08] The 'Microsoft.PowerShell.Management' module was not imported because the 'Microsoft.PowerShell.Management' snap-in was already imported.
[00:02:09]       [+] Should compile and apply the MOF without throwing 1.29s
[00:02:09]       [+] Should be able to call Get-DscConfiguration without throwing 56ms
[00:02:09]       [+] Should return $true when Test-DscConfiguration is run 66ms
[00:02:09]

Suggested solution to the issue
I'm unsure of how to fix this. Would appreciate community assistance.
Version of the DSC module that was used ('dev' if using current dev branch)
dev

@mhendric The documentation about compiling resources for Azure Automation DSC states the following

When compilation is complete you may receive an error stating: The 'Microsoft.PowerShell.Management' module was not imported because the 'Microsoft.PowerShell.Management' snap-in was already imported. This warning can safely be ignored.

https://docs.microsoft.com/en-us/azure/automation/automation-dsc-compile#assets
If you do ask Google about this you'll find numerous places where this issue is mentioned and as far as I can tell (also by taking the stuff that I found via Google) the warning will be issued when you compile a DSC configuration. My guess is confirmed by the fact that after such a warning you'll always find a Pester statement about Should compile blahblah. Interesting is that this warning does not appear if you compile a DSC configuration interactively in a PowerShell session.
So I think to have this (really annoying!!!) warning fixed you must get in touch with the people who maintain the DSC core infrastructure. I think @kwirkykat @johlju or @PlagueHO can help here.


Update shared_hosts.yml

Adding information about SiteGround shared plans.

Adding information about SiteGround shared plans.

+1 for merging this info :-)


Do not include billing info in request if a token is present

Fixes #73

:+1:

Should be published in 1.2.4 https://www.nuget.org/packages/recurly-api-client/


Fix express response

Fixes the Express response so the status is sent in the proper order.

+1

+1

+1, Should be migrate


Create SECURITY.md

Hello 👋
I run a security community that finds and fixes vulnerabilities in OSS. A researcher (@rook1337) has found a potential issue, which I would be eager to share with you.
Could you add a SECURITY.md file with an e-mail address for me to send further details to? GitHub recommends a security policy to ensure issues are responsibly disclosed, and it would help direct researchers in the future.
Looking forward to hearing from you 👍
(cc @huntr-helper)

Sure, good idea
https://github.com/run-llama/llama_index/blob/main/SECURITY.md


python310Packages.opuslib: fix tests with libopus 1.4

Description of changes
Pulls in a patch to fix unit tests when libopus is at version 1.4.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 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.

Result of nixpkgs-review pr 241167 run on x86_64-linux 1

9 packages built:

botamusique
python310Packages.opuslib
python310Packages.opuslib.dist
python310Packages.pymumble
python310Packages.pymumble.dist
python311Packages.opuslib
python311Packages.opuslib.dist
python311Packages.pymumble
python311Packages.pymumble.dist


Make opam.ocaml.org the default repo

As noted in https://github.com/ocurrent/docker-base-images/issues/98#issuecomment-807862390, the switches created in the base images have ~/opam-repository as the default remote, so opam update does nothing, which is a bit surprising for new users to the container images.
Any change here will necessarily impact existing users of the images (ocaml-ci, opam-repo-ci, etc.). If we do this, let's do it once (the clever part of this idea is due to @talex5!):

At the end of the build, change the URL for the default remote to https://opam.ocaml.org
Require existing users to change their Dockerfiles, but introduce a script opam-update-repo-to which takes a commit and whose job is to update the switch to that commit of opam-repository so that this change only has to be done once
opam update will then always retrieve https://opam.ocaml.org

Masking the use of ~/opam-repository behind a script will also allow future optimisations such as making it a bare repo without breaking images again.

I just run into this, and wanted to report that the current "ocaml/opam" docker images are very hard to use - especially for people that are not used to the opam workflow. They have an old version of a git clone of the opam-repository in /home/opam/opam-repository, and the opam is configured in a way that opam update does not result in updating the opam repository from opam.ocaml.org (which would be the expectation).

Just double-checking: "old" should be no more than a week?

I don't know exactly. It was in a state where an opam update was required (FROM ocaml/opam:debian-10-ocaml-4.11) since git-mirage 3.3.1 was unknown.

CI dev meeting notes on this. Feeling is that the image should remain as targeting CI primarily, and so the default workflow should be for good Docker hygiene - i.e. focus on reproducibility. In this aspect, opam update is of course bad. So the solution we propose is to turn this idea on its head:

The utility script opam-update-repo-to is a good idea regardless
Similarly, a utility script to switch to opam.ocaml.org and helps the alternate use-case where it's wanted
Finally, a little ENTRYPOINT / CMD manipulation should allow a motd/welcome for direct use of the to advertise these facts (as will updating the readme information on Docker hub, which is also in hand)

So the existing behaviour remains, but the images advertise that more obviously and provide assistance to go from the CI workflow to the more usual opam one.


Removed link to a document hosted on an internal repository

/retest

/test openstack-ansibleee-operator-build-deploy


(PE-21799) Ensuring gc_packages runs

I've added gc_packages to the scheduled garbage collection (not currently included):
2017-09-01 01:24:12,766 INFO [p.p.c.services] Starting database garbage collection
2017-09-01 01:24:12,772 INFO [p.p.c.services] Finished database garbage collection
2017-09-01 02:24:12,773 INFO [p.p.c.services] Starting sweep of stale nodes (threshold: 7 days)
2017-09-01 02:24:12,780 INFO [p.p.c.services] Finished sweep of stale nodes (threshold: 7 days)
2017-09-01 02:24:12,780 INFO [p.p.c.services] Starting purge deactivated and expired nodes (threshold: 5 minutes)
2017-09-01 02:24:12,781 INFO [p.p.c.services] Finished purge deactivated and expired nodes (threshold: 5 minutes)
2017-09-01 02:24:12,781 INFO [p.p.c.services] Starting sweep of stale reports (threshold: 14 days)
2017-09-01 02:24:12,783 INFO [p.p.c.services] Finished sweep of stale reports (threshold: 14 days)
2017-09-01 02:24:12,784 INFO [p.p.c.services] Starting database garbage collection
2017-09-01 02:24:12,790 INFO [p.p.c.services] Finished database garbage collection

Test PASSed

LGTM


[ch18573] Add new icons for project privacy.

Issue: ch18573
[skip-ch]

This adds the 3 new icons for project privacy to the shared icon list for use in the Editor and Community sites. I added them as new icon names without overwriting the existing private and public icons because:

I think our icons should describe what they depict instead of what we use the symbol to mean.
It will let us use the new icons without modifying the old ones.

I took the SVGs from https://github.com/glitchdotcom/Glitch-Editor/pull/898/, but had to modify the codeLock icon a bit to use clipping instead of a background color to make the code symbol work.

looks great, I can modify my PR to use these once you've shipped the new library version. I can also put the component in the shared library so you can just use that, what do you think?

I think just the icons is just fine, keeping the project-object-specific logic outside shared-components seems cleaner to me.

hmm but you'll need to use the same logic in the Community site, so that seems repetitive. I was thinking the shared-components library is useful for that kind of thing too; do you disagree? It's a pretty clean API; just takes in a privacy string and returns the appropriate SVG (we can add in proptypes for more specificity on what you can pass in too)

I think logic that relies on our data model (i.e. data returned from the API) doesn't really fit in shared-components; most everything else in here currently is more generic than that. Making shared components start to rely on the shape of data returned from the API limits our options for state management in the community and editor sites (e.g. we wouldn't be able to add client-side model wrappers around API responses if we wanted to in the future).
Returning the right SVG based on a string doesn't really violate that, but at the same time it's also a small enough helper to write that there's not really much benefit in sharing the implementation between both sites to offset the cost of having another codebase to consider when figuring out what the components do.

:rocket: PR was released in v0.19.0 :rocket:


feat: Add net6 mobile (WinUI) target for sample

PR Type
What kind of change does this PR introduce?

Feature

What is the current behavior?
No sample for net6 targets (ios, android, macos, catalyst)
What is the new behavior?
Added a Mobile project under the WinUI samples that has the missing targets
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tested code with current supported SDKs
[ ] Docs have been added/updated which fit documentation template. (for bug fixes / features)
[ ] Unit Tests and/or UI Tests for the changes have been added (for bug fixes / features) (if applicable)
[ ] Wasm UI Tests are not showing unexpected any differences. Validate PR Screenshots Compare Test Run results.
[ ] Contains NO breaking changes
[ ] Updated the Release Notes
[ ] Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

Very cool :) now to get the net6.0 apps included in the UI Tests!

@kazo0 I noticed that some of the builds still have net6 disabled. We should sync so we can work through changes required for the build in order to integrate the net6 sample app

@nickrandolph ah yes the Samples/Samples_WinUI jobs still have DisableNet6MobileTargets: true, I can remove that and then I guess we could just include the net6 sample heads in the custom Samples/Samples_WinUI solution configs. Or I guess we'd need it just for the Samples_WinUI config. Or we can have a standalone config for the net6 samples.

@kazo0 did we ever manage to get the build sorted out for this?

@nickrandolph yeah I still need to get this fixed, I should probably redo the way we are splitting the projects, I shouldn't really be using the ApplicationPlatform to do this

@kazo0 in extensions for the testharness project we've moved to net6+winui for the main samples and then we have a "legacy" folder which contains samples for xamarin+uwp. Do you think that approach would work here too?

@nickrandolph and @kazo0 what is the status of this PR please?

Oh boy, didn't even realize we had this one, I'm actually working on doing this for the new canary work, it's almost done and I'll have a new PR

Closing this as it sounds like it's been deprecated by the work @kazo0 has done


obj.constructor not always set

Hi again!
It seems that in some rare cases the obj.construtor is undefined (validate.js:18)
Reference: http://javascript.info/tutorial/constructor
The 'typeof' approach might be a better solution.

Once again thanks! I agree - typeof is better...

Fixed in 7b478815b20ac6646fbff62f78be72b45c0f4117

Even though I closed this I have kept constructor.name in the finalizer so far. The validator has been updated to using typeof. That is, because ex. typeof RegExp returns 'object' - so I just need to figure out how to restructure myself out of it.

And then again. In the finalizer I know which objects are expected - so the non-constructor.name might not be so much of an issue there.

Then again it was used in both the ranges.js and unique.js files.


bug(Themeing): Angular Material 12.0.5 Custom Theme generated not working.

Reproduction
Use StackBlitz to reproduce your issue:

Components: https://stackblitz.com/edit/components-issue-ynarhw

Steps to reproduce:

create new angular project
ng add @angular/material
select custom theme

Expected Behavior
Custom theme generated should work out of the box
Actual Behavior
I get the following error:
./src/styles.scss - Error: Module build failed (from ./node_modules/mini-css-extract-plugin/dist/loader.js):
ModuleBuildError: Module build failed (from ./node_modules/sass-loader/dist/cjs.js):
SassError: Invalid CSS after "@include mat": expected 1 selector or at-rule, was ".core();"
        on line 10 of src/styles.scss
>> @include mat.core();

However it seems the main even the @use '~@angular/material' as mat; line won't work as I can't ctrl+click to find it.
Environment

Angular: 12.0.5
CDK/Material: 12.0.5
Browser(s): N/A
Operating System (e.g. Windows, macOS, Ubuntu):  Windows

I just tried generating a new project based on the steps above and it worked as expected. What command do you get the error on? I was testing with ng serve. Also can you check if there's node-sass in the package.json? The @use syntax isn't supported by the node-sass package, only the official sass.

Closing since there hasn't been any feedback. The issue can be reopened if we get more information.


Error 404 Not Found

Hi, with version 2.9.1 I'm getting a Error: 404 Not Found when running phrase pull.
Error suggests to upgrade to CLI version 2.11.0, but on Homebrew 2.9.1 is the latest stable version.
Is happening to more of my coworkers so it seems a general issue.

Thanks @theSoenke, token was the problem. I generated a new one and is working.
Thanks.


fix a memory leak in lambda worker

Fix a bug that the captured lambda function was leaked and not released properly.

Good catch.


Missing license file

Hi,
I think the license file is missing in this repository. I noticed GitHub wasn't showing the license on the sidebar, and then found that the link at the README.md was 404'ing.
I was going to send a PR with the license from ecmwf/ecflow, but then I realized that it would probably be better for a PyFlow dev to confirm it first, and then add the correct file.
Thanks
Bruno

Thanks @kinow. I added the LICENSE file in 3.0.1.
Cheers,
Corentin

Thanks @corentincarton !


Last updated time overlaps with the visit state button

Describe the bug
Last updated time overlaps with the visit state button on switching the map if the mouse is hovered to a different state before the map is able to change.
To Reproduce
Steps to reproduce the behavior:

Click on any state on the map
Swiftly move the mouse to any other state before the map changes

Expected behavior
The last updated time should not be shown on state maps since data is not available.
Screenshots

See top right

/available

/claim

Lol


ref(draft): add success and fail text on draft up

part of #294

@michelleN,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Codecov Report

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

@@           Coverage Diff           @@
##           master     #301   +/-   ##
=======================================
  Coverage   30.94%   30.94%
=======================================
  Files          28       28
  Lines        1771     1771
=======================================
  Hits          548      548
  Misses       1144     1144
  Partials       79       79

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 716b600...2b60c68. Read the comment docs.


Can't change collection logo as a Collection Administrator

I'm using Dspace 7.6.1 and when I try to upload a new Collection Logo while logged as that collection's adminstrator I get both the succesfull and failure message. The Logo doesn't change.

However I am able to change the Logo if I'm logged as the Site's Administrator.
To Reproduce

Log in with an account that has administrator privileges for a Collection.
Edit the Collection to add a new Logo.
Check if the new Logo was added.

Quick note:
I'm able to change the collection name, Introductory text (HTML), etc. It's only the logo that's causing an issue.

@AgustinAlfieri : This is a duplicate of #9319.  It was solved for both pre-8.0 and pre-7.6.2 (due after 8.0 is out).  Here's the PR that fixed it for 7.6.x: https://github.com/DSpace/DSpace/pull/9353
Closing, as this appears to already be fixed. However it is not officially released yet... the fix will be released in 8.0 and 7.6.2


[FEATURE] Add "Projects" section

Add the "Where is Rust Used" section in the project.
something like this
https://www.reddit.com/r/rust/comments/umudp6/what_popular_companies_uses_rust/
For reference, check these files:
https://github.com/FrancescoXX/rustcrab/blob/main/src/components/DevToolsSection.tsx
This for the data (similar structure)
https://github.com/FrancescoXX/rustcrab/blob/main/src/data/tools.ts
and then insert this component into the page.tsk file (similar to what is here at line 39)
https://github.com/FrancescoXX/rustcrab/blob/main/src/app/page.tsx

Hi @FrancescoXX,
I would like to contribute on this issue.
Thanks


[RgbDiode] Change inverse to LedType

Description

Changed inverse to LedType for better clarity.

Types of changes

[ ] Improvement (non-breaking change that improves a feature, code or algorithm)
[x] Bug fix (non-breaking change which fixes an issue with code or algorithm)
[ ] New feature (non-breaking change which adds functionality to code)
[x] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Config and build (change in the configuration and build system, has no impact on code or features)
[ ] Dependencies (update dependencies and changes associated, has no impact on code or features)
[ ] Unit Tests (add new Unit Test(s) or improved existing one(s), has no impact on code or features)
[ ] Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

[ ] My code follows the code style of this project (only if there are changes in source code).
[ ] My changes require an update to the documentation (there are changes that require the docs website to be updated).
[ ] I have updated the documentation accordingly (the changes require an update on the docs in this repo).
[ ] I have read the CONTRIBUTING document.
[ ] I have tested everything locally and all new and existing tests passed (only if there are changes in source code).
[ ] I have added new tests to cover my changes.

Let me know if any other change to add so that we can merge.

Nope, Just this one :)

I have a PR to fix the duty cycle not needing to call twice.  here

I have a PR to fix the duty cycle not needing to call twice. https://github.com/nanoframework/nf-interpreter/pull/2874

So @torbacz I guess we can wait for th fix to be merged and then you can make the needed adjustments on this PR ;-)

@Ellerbach
Sounds good

@Ellerbach Tested with latest ESP firmware. We can merge it.


@html overrides CSS/JS asset aggregation

Not sure what I’m doing wrong here. I’m experimenting with an all-WebC Eleventy site. I have a hero-block component that is declaring some styles, I have a couple of layouts (chained together), and in the base layout I have a style tag that is settings its contents to this.getCSS(this.page.url). But no styles are in the built HTML.
Not sure what I’m missing…
Here’s the full project that you can clone and test: https://github.com/darthmall/webc-eleventy-test

I just figured this out and was coming over to here to close this as user error, but I’m glad it was an actual bug and not a total waste of your time. 😄
A workaround is to use a <template webc:nokeep>:
<hero-block>
  <template @html="this.site.title" webc:nokeep></template>
</hero-block>

I know I said I’d get to it this week but I didn’t quite make it, sorry! This is still on deck!

Don’t sweat it. There’s a workaround. It’s not for anything critical. 😃

Some other benefits shaken out here by this fix:

we now treat the return value from @html on a host component as default slot content that can be consumed by the component definition.
this reprocesses @html return values as WebC content too.

shipping with v0.5.0!


Introducing mocker for bids strutures.

Checklist
GitHub

[X] I've given this PR a concise, self-descriptive, and meaningful title
[X] I've linked relevant issues in the PR body
[X] I've applied the relevant labels to this PR
[ ] I've assigned a reviewer

PR contents

[X] I've consulted ivadomed's internal developer documentation to ensure my contribution is in line with any relevant design decisions
[X] I've added relevant tests for my contribution
[ ] I've updated the relevant documentation for my changes, including argparse descriptions, docstrings, and ReadTheDocs tutorial pages

Description
This PR, is the second part of the breakdown of PR https://github.com/ivadomed/ivadomed/pull/1201.
All of those modifications are from PR https://github.com/ivadomed/ivadomed/pull/1201.

The bids file structures are ultimately created in testing/mocker/mocker_fixture.py.
New CreateSubject class at testing/mocker/create_common.py

This is s the abstract class that highlights the main way to interact with subject related creation process

New CreateBIDSDerivatives class at testing/mocker/create_derivatives.py.

Create the derivatives

New CreateBIDSSubjects class at testing/mocker/create_subjects.py.

Create the subjects

Manage nifti data at testing/mocker/nifti_mocker.py.

Create a mock nifti image.

Check if data is a nifty image.

Save data to a nifty image.

This mocker will be used in future PRs tests for file_dataset_group, all_file_dataset_group, and test_training.
Linked pull requests
#1201

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.0%) to 68.951%

Totals

Change from base Build<PHONE_NUMBER>:
-0.0%

Covered Lines:
4266

Relevant Lines:
6187

💛  - Coveralls


Add Player

Form for text input

player name should be unique in DB
on submit make post request to server

query DB to add
should return error if already in DB

Forgot to leave a comment.
Nick crushed the Add player. It puts them in the DB and the existing players list.


[feature] battery indicator

ISTR #dev-chat said that there's no wire for the MCU to read the voltage level, so this might go into the README as not possible.
Otherwise, I suggest these feature levels:

Low battery indicator (need coordinating with led theme)
Led battery meter
Awesome level: BLE reporting (need coordinating with BLE chip?)

I remember the same. According to https://github.com/hi-a/annepro-key#hardware there's a bq24075 charger, which doesn't seem to give us voltage: http://www.ti.com/lit/ds/symlink/bq24075-q1.pdf
I'm not sure if any of the other chips might help us, there's a tiny U5 that I'm not quite sure what it is?
The other bit to investigate would be wether PGOOD would let us at least do a low battery warning?
I haven't really looked into any of this yet, my dev keyboard runs without the battery attached.

As a side note, here is a really cool project that details adding an on off switch as well as a hardware battery indicator.
https://imgur.com/r/AnnePro/KNNts


Add gc and fix typo

Pull Request Template
Issue Reference
N/A
Description
Adding gc collect and torch.cuda.empty_cache() after del to make sure device memory is freed immediately.
Type of Change

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

How Has This Been Tested?
N/A
Checklist

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Additional Information
N/A

PTAL @MayDomine


usql: 0.9.3 -> 0.11.12

Description of changes
Upgrade usql package from 0.9.3 to 0.11.12.
To inspect upstream changes:

Release on GitHub
Compare changes on GitHub

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I cannot merge, but just an FYI that #185000 is also an automated PR for this package.


Contribute to Rhai org?

Hi, on and off I see users asking for Rhai on ESP32 or STM32 chips.
Would you like to contribute this sample to the rhai org so that others can expand from it?

What would the contribution involve? I don't have many spare cycles to maintain an example upstream tbh, but I could carve out some time to contribute an initial implementation somewhere.

Nothing much, I suppose...  Just that many users have asked for examples to run Rhai on esp or stm chipsets...
I think if you have something that works, then it serves as a good enough template for others to build upon.


Outputs of gradient should include more information

Right now dG/dC is printed into numerical_checks.out. It would be nice to see dU_dC and dAv_dC there as well to help troubleshoot.

U and A are out-put for +/-dC or +/-dV at each step


Map Display

This is to show that the map i received from the Maps API actually shows on the my web page.

Send this for review to either me or steve


Add docker-compose-replica.yml

Codecov Report

Merging #109 (94315bb) into develop (4844ee6) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 94315bb differs from pull request most recent head eb9e113. Consider uploading reports for the commit eb9e113 to get more accurate results

@@           Coverage Diff            @@
##           develop     #109   +/-   ##
========================================
  Coverage    82.26%   82.26%
========================================
  Files           49       49
  Lines         1900     1900
  Branches       303      303
========================================
  Hits          1563     1563
  Misses         337      337

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 4844ee6...eb9e113. Read the comment docs.


Use tersify_mlir in FileCheck tests

In FileCheck tests where we have // RUN: graphblas-opt %s | graphblas-opt --graphblas-lower | FileCheck %s, it might be worth changing it to // RUN: graphblas-opt %s | graphblas-opt --graphblas-lower | tersify_mlir | FileCheck %s since this'll make the tests less verbose.

Since this is an idea that's not necessarily known to be needed, I'm going to close this issue for now until stronger motivation comes.

I'm reopening this since working on https://github.com/metagraph-dev/mlir-graphblas/pull/237#discussion_r746815609 has made me realize how much nicer fixing this issue can make our tests.

We currently can't use tersify_mlir as part of the FileCheck tests since there's a somewhat circular dependency. tersify_mlir needs the CLI tool to work. The CLI tool needs graphblas-opt to be built. Building graphblas-opt requires tersify_mlir to exist.
Since there's not an obvious way to do this, I'll close this issue for now.


fix: Check if sharedBy atSign is authorized to send notifications

- What I did

Fix the git issue: https://github.com/atsign-foundation/at_server/issues/1180

- How I did it

Add "IF" condition to check if sharedBy atSign is currentAtSign. If yes, allow to send notifications, else throw UnAuthorized exception

- How to verify it

Added unit test to verify UnAuthrizedException is thrown when sharedBy is not equal to currentAtSign
The existing notification tests should pass

- Description for the changelog

Check if sharedBy atSign is authorized to send notifications

I just tried to take up at_utils and do the global search and replace and discovered that fixAtSign is not a straight replacement of formatAtSign. fixAtSign takes a String as argument and returns a String. formatAtSign takes a String? as argument and returns a String?
@srieteja @murali-shris @sitaram-kalluri can you please get together to discuss this and figure out what is the right way forward?
Meanwhile @sitaram-kalluri I'll review this PR as is


Update Spanish Translation

Category:
Other (translation into spanish language)
Overview
Good morning, I have updated the Spanish language for dashy. Cheers 🍻

Hi Lissy, don't worry, the translations are correct, I saw that the Spanish was very outdated and I decided to lend a hand. By the way, you have done a great project, my congratulations.


fall back to line comments if associated documentation is missing

This might address issue 269. This was a quick fix, so I'm totally open to feedback on the proper way to go about contributing here. Hope this helps!

Hi @wbbradley -- this looks like it would probably be the right approach.
For us to be able to merge this type of change, though, we would need:

A test case that fails before and passes after (you could duplicate the go basic test case here and just keep the wantHover section, modifying it to test this functionality)
Some inline doc comments before the if statements describing why they are there (// Fallback to inline comments when there is no associated documentation or something like that).


[Bug]: add links in chapter page

Description
add links in footer navbar in chapters page
Expected Behavior
to add link
Show us the trouble with screenshots

Which browsers are affected?
No response
Which OS are affected?
No response
Enter the version of your web browser
chrome
Device?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

@Vaishnavi-Patil2211 please assign me this issue


Buttons Tests

Hello!
I was taking a look in the unit tests, and some of them was failing (button stuff), then I noticed that some of them should be asynchronous tests, since it was using an asynchronous functionality, the setTimeout()
I've created asynchronous tests for them, and now all the tests are passing.
Thanks, see ya! :)

hi @gnuwilliam
I don't quite understand the meaning of

setTimeout(function () {
  state == 'loadingText' ?
    $el.addClass(d).attr(d, d) :
    $el.removeClass(d).removeAttr(d)
}, 0)

why not invoking directly without setTimeout?

Yep, I would say the same. But this was already implemented, so I've decided to keep this way. If there is no problem changing it, I can do that and test it without the asynchronous way.


Proxy configuration by environment variables

Hi,
I recently received an issue that describes difficulties in using proxies with Facepy, a Facebook Graph API client that uses requests internally. I would like to keep proxies out of my interface, so I've facilitated the configuration of proxies in requests through the environment variables HTTP_PROXY and HTTPS_PROXY.
I'm not sure how I might test this feature, but if you have any ideas please let me know and I'll get right to it.

You're incredible :)

Now, now... ;-)

:sparkles: :cake: :sparkles:


Support printing default values for specific fields if includingDefaultValueFields=false

The upstream JsonPrinter supports printing default values for specific fields:
 * Creates a new {@link Printer} that will also print default-valued fields if their
 * FieldDescriptors are found in the supplied set. Empty repeated fields and map fields will be
 * printed as well, if they match. The new Printer clones all other configurations from the
 * current {@link Printer}. Call includingDefaultValueFields() with no args to unconditionally
 * output all fields.
 */
com.google.protobuf.util.JsonFormat.Printer#includingDefaultValueFields(java.util.Set<com.google.protobuf.Descriptors.FieldDescriptor>)

Because of your fix #20, this is already possible for non-repeated primitive fields by adding optional to the field definition and setting the field explicitly. However for repeated fields, this is not possible and so I'm stuck. Would it be possible to add the upstream method to this project?
Again, thanks in advance!

Hi @C0mbatwombat - I added the option in #29, can you confirm it satisfies your use case? Then I'll cut a release. Cheers.

@chokoswitch Yes, perfect!

Sorry for the long delay @C0mbatwombat, it looks like I had actually published a release but forgot to right release notes for it. And then ended up publishing the same again as 2.5.0 >< But it should be ready now.


fix: enforce fallback responses

The fallback responses are the ones coming from our backup system. They come into play when the node selected from the network can't do it's job, and doesn't return any response. In our mission to be as fast as we can, we use our internal system to serve that request (doesn't generate revenue).
These responses were only enforced to a certain format, but the truth is that there are other several server errors from blockchain's node clients that indicate that the call wasn't effective. This PR adds some additional text enforcement to abstract these errors from users and let it fail without any specific blockchain's node message that could add confusion to end user.
Example error:
{"error":{"code":-32000,"message":"rpc error: code = Unavailable desc = connection error: desc = \"transport: Error while dialing dial tcp <IP_ADDRESS>:9090: connect: connection refused\""},"id":732830328053361800,"jsonrpc":"2.0"}

Another fix was also introduced at this PR, since it's error-related. It's the fix to the problem outlined in #862.
The current approach is matching server errors, and user error is anything with -32k error and NOT a server error. Since server errors are fewer, it is easier to handle and node runners won't be penalized by false positives on EVM unknown user error messages.

Another fix was also introduced at this PR, since it's error-related. It's the fix to the problem outlined in #862.
The current approach is matching server errors, and user error is anything with -32k error and NOT a server error. Since server errors are fewer, it is easier to handle and node runners won't be penalized by false positives on EVM unknown user error messages.


pre-load molecule for tour?

Berend said it might be more intuitive to also have the molecule loaded for which we show the things in the shepherd tour.
I have mixed feelings about that.
What do you think, @lpatiny?

I had an email from Bob (JsMol) that did not see a 3D model and didn't find out you need to click on a wavelength
This is why after model calculation I think a 3D model should be displayed directly without having to click on anything.

see telegram. the only sensible selection criterion, for me, would be to take the most intense band


Backmerge 2.12.0 to development

Backmerge release v2.12.0 to development

Travis automatic deployment:
https://pr1411--safereact.review.gnosisdev.com/app


Request for an example using props

Hi!
I would like an example of where the ContextMenu is given props which in turn are show in an item. My use case is that I would like to attach your show to a div and pass in some this-div-specific values so that the context menu can show this info.
Something like:
show(originalEvent, {id: MAP_CONTEXTMENU_ID, props: {"payload": "element1"}});

However, though I can see in the docs and examples the use of props with values lala and 1, I can't figure out how to use them.
Any help would be much appreciated.

Currently, we can only get props through Item's onClick, like this:
https://github.com/alist-org/alist-web/blob/5f72ca8bcc8bdb0c1f83913ae3b9f22e0a67531f/src/pages/home/folder/context-menu.tsx#L56
But it cannot be used directly in the children of Item. I can add this feature later if you need to use props in the Item's children.

@xhofe can you do it please?


[REQUEST] Use git-for-humans package?

See https://github.com/xreader/git-for-humans
This library seems to handle a lot of complex git tasks quite easily… might be useful here?

yea that could potentially be useful. it'll be important that the output is easy to parse so we can display meaningful results within atom.


Contributor bot

Description
refer to here

@all-contributors please add @SCiarella for code, tests and maintenance


Move sequence test that uses wskadmin out of health suite

Tests were refactored as discussed.

rebased, sqashed and merged in baa652a8edf0c2032f3b8f6386421b4a35236911


[Set Audio Device]... fails to fetch output device

Extension
https://raycast.com/benvp/audio-device
Raycast Version
1.75.1
macOS Version
14.4.1
Description
The 'Set Output Device' simply isn't working. When you run the command the extension loads but fails to show the the output devices

Error:
Error: Command failed with exit code 1: osascript -e
    set devices to {}

    tell application "System Preferences"
      reveal pane id "com.apple.preference.sound"
    end tell
    tell application "System Events"
      tell application process "System Preferences"
        repeat until exists tab group 1 of window "Sound"
        end repeat
        tell tab group 1 of window "Sound"
          click radio button "Output"
          tell table 1 of scroll area 1
            set selected_row to (first UI element whose selected is true)
            set currentOutput to value of text field 1 of selected_row as text

            repeat with r in rows
              try
                set deviceName to value of text field 1 of r as text
                set deviceType to value of text field 2 of r as text
                set end of devices to { deviceName, deviceType }
              end try
            end repeat
          end tell
        end tell
      end tell
    end tell

    if application "System Preferences" is running then
      tell application "System Preferences" to quit
    end if

    return [ devices, "currentOutput", currentOutput ]

64:116: execution error: System Settings got an error: Can’t get pane id "com.apple.preference.sound". (-1728)
    at _t (/Users/ferdiaz/.config/raycast/extensions/2158713f-4712-4862-ba9a-3bce3cd07e3a/set-output-device.js:5:57)
    at /Users/ferdiaz/.config/raycast/extensions/2158713f-4712-4862-ba9a-3bce3cd07e3a/set-output-device.js:5:6400
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ca (/Users/ferdiaz/.config/raycast/extensions/2158713f-4712-4862-ba9a-3bce3cd07e3a/set-output-device.js:41:6806)
    at async b0 (/Users/ferdiaz/.config/raycast/extensions/2158713f-4712-4862-ba9a-3bce3cd07e3a/set-output-device.js:41:6981)

Steps To Reproduce

Open Raycast ⌥+Space
Exec "Set Output Device" command

Current Behaviour
you get no results when executing the command

Expected Behaviour
i would expect to see a list of devices just as you do when using "Set Input Device"

+1

For me, it's showing my MBP speakers, but not any Airplay devices:

Keep it open


exclamation-triangle is not properly centered

Hi,
Upon the release of v2, the icon exclamation-triangle is noticably off-centered when compared to v1 exclamation. I use class p-1 rounded-full.

Tailwindplay: https://play.tailwindcss.com/34c1R1uABN

Hey!
Good catch — this should be more optically centred now 👍🏻

You can check out the Play here:
https://play.tailwindcss.com/OtmJ3fhKdT


Bug Report: Sustain Notes Don't Do Damage

Please check for duplicates or similar issues, as well performing simple troubleshooting steps (such as deleting the YoshiCrafter29 folder from AppData) before submitting an issue.
I've deleted this folder before bug report.
If you're using an older version of the engine, check if the bug haven't been already fixed
PS : the "everything goes black while changing state in developer mode bug" is not a bug, it's been designed this way to refresh all bitmaps between states to allow easier modding.
Which target are you using?

[x] Windows
[ ] Mac
[ ] Linux

What version of the engine are you using (Bottom left of the screen in main menu) ?
2.0.2
Have you identified any steps to reproduce the bug? If so, please describe them below in as much detail as possible. Use images if possible.
Play any song that has sustain notes and just press it (without holding).
Please describe your issue. Provide extensive detail and images if possible.
This problem doesn't appear in version 2.0.0. In 2.0.2 sustain notes don't do damage.

fixed


Fix graph nodes alignment

When graphs are displayed, the composition nodes are misplaced

It has to do with the way we create and configure the graph. It does not happen always, and the cause for it is unknown. Should be simple issue with React hooks and callbacks.
The requirement is for arrows to go left-to-right always, upon initial load and upon navigation between different views.

To reproduce, navigate to any composite resource graph. After that, navigate to claim graph - the alignment gets broken.


MAR results in OutOfMemoryErrors

function net1()
    a,b,c,d = pos_literals(ProbCircuit, 4)

    full_c_b=0.3*c+0.7*(-c)
    full_c_neg_b=0.8*c+0.2*(-c)
    b_a_1=b*full_c_b
    b_a_2=(-b)*full_c_neg_b

    full_b_a=0.6*b_a_1+0.4*b_a_2

    full_b_neg_a=0.5*b_a_1+0.5*b_a_2
    a1=a*full_b_a
    a2=(-a)*full_b_neg_a
    full_a=0.25*a1+0.75*a2

    return full_a
end

catalog = DataFrame(CSV.File("test_data.csv"));
catalog=DataFrame(BitArray(Matrix(catalog)), :auto)

combinations=[]
for column in 1:size(catalog)[2]
    push!(combinations, [x for x in Set(catalog[:,column])])
end
all_models=DataFrame(BitArray(Matrix(DataFrame(collect(Iterators.product(combinations...))))), :auto)
print("All models loaded")
print(exp.(MAR(net1(),all_models)))

test_data.csv is an easy way to define the domains of the variables
a,c,b,d
True,True,True,True
False,False,False,False
When using MAR, this always results in the following error

Is there something wrong in my code?
Greetings
Sieben

Hi, the error suggests you are running out of memory. How big the all_models dataframe is? And how much free RAM did you have? Also what verison of libarary are you using?
The MAR function does not do any batching, i.e. it tries to allocate meomry in order of O(circuit_size * num_datapoints) which could be big. You can try calling MAR on a smaller subset of your data and see what happens. If the error persists might be other issues.


Blocks don't work on build time with static query

Hi @pristas-peter,
I am trying to do this: https://github.com/rodrigo-arias/gatsby-wordpress-demo/blob/master/src/pages/index.js
Everything works fine with develop, but breaks at build time

I also found an error that maybe is related https://github.com/gatsbyjs/gatsby-source-wordpress-experimental/issues/105
The issue only happens using the blocks
"gatsby": "^2.24.31"
"gatsby-source-wordpress-experimental": "^1.2.6"

Do you think the static query should not affect the blocks and is there something else?
Thank you

The were some changes in gatsby lately on how the static queries are bundled, so I guess the problem is somewhere down that road.

Thanks Peter, it's good to know that you know where the issue can come from. It will be great when you get a chance to check it out


wait for riak_pipe and _kv node services, and unload apps on fullstop (AZ1033 … related)

Not waiting for the node watcher to declare riak_pipe and riak_kv
services as up before running the test was causing preflist exhaustion
errors. Not unloading apps after stopping them was causing re-starting
them for the next test to fail.

FWIW I'm seeing 23 failures on this branch.

Yeah, my unloading was too aggressive, and I there may still be a conflict with a riak_core test anyway. Do Not Merge.

Here are some snippets of what I see
mapred_test:199: compat_basic1_test_...*failed*
::error:{badmatch,{error,all_nodes_down}}
  in function mapred_test:'-compat_basic1_test_/0-fun-1-'/0

mapred_test:213: compat_basic1_test_...*failed*
::error:{badmatch,{error,{error,insufficient_vnodes_available},{ok,[]}}}
  in function mapred_test:'-compat_basic1_test_/0-fun-2-'/0

I also see a lot of the following, not sure if it's hurting anything (yes, my hostname is crackpot).
=ERROR REPORT==== 11-Jan-2012::11:38:31 ===
** System running to use fully qualified hostnames **
** Hostname crackpot is illegal **

If there is anything I can do to assist let me know.

Remove the unload calls gets me down to 13 errors.

Just discovered the lists of nodes reported by riak_core_node_watcher:nodes/1 and what's in the ring do not match up.  This means that get_apl_ann and riak_core_apl:check_up will fail to find any partitions that are considered available.

I switched to shortnames cause I saw a lot of errors (that don't cause test to fail) about invalide hostnames and looking at keys_fsm_eqc I saw it uses shortnames as well.

Fixed, merged PR https://github.com/basho/riak_kv/pull/275


How do i change scrolls (middle of a song with lua)

What is your question?
idk if i can ask here, cuz the discussions got deleted :v

If by "scrolls" you mean something like Scroll Speed, then yes, it's already there. If you mean from Side-to-middlescroll, (and vice versa) I have a thing for that.
If you mean from Up-to-down (and vice versa), I have no idea

i meant Between downscroll and Upscroll

function onEvent(name, value1)
if name == "Change to downscroll" then
for i=0,7 do
setPropertyFromGroup('opponentStrums',i,'downScroll',true)
setPropertyFromGroup('playerStrums',i,'downScroll',true)
end
end
end

that would work, but the notes wont move (same with the healthbar)
you would have to manually configure the y value of all the notes and the healthbar

function onEvent(name, value1)
if name == "Change to downscroll" then
for i=0,7 do
setPropertyFromGroup('opponentStrums',i,'downScroll',true)
setPropertyFromGroup('playerStrums',i,'downScroll',true)
setProperty('healthBar.y', )
end
end
end
I dont remember how the set y was cuz im on phone rn at 22:23 for me , for the set y for strums thing just search in the promise mod song files or something, thats from where i found the change scroll thing


Run checkout inside docker

The title isn't 100% true,
because it depends on the settings (if docker is enabled).
There are still commands being executed outside docker #5673.
Blocked on #5673
Related to #5672

Is this strictly blocked on #5673? I think these are separate things and we can start running these commands from inside the container and then migrate the rest once we have made a decision.
Although, before merging this PR we should make a PR compatible with the Corporate site.

Yeah, this isn't really blocked, but it feels weird executing the same set of commands in different environments. It could lead to some surprises using different git daemons? I don't know. Better safe than sorry? p:

It could lead to some surprises using different git daemons?

This is a good point, yes. I'm not sure if we have the same git version inside the container than outside.

I just realized that executing git from inside the container may requires another change as well. On which servers could be sync_repository_task executed when triggered? If the answer includes webXX we may need to limit it the builder queue because web servers do not have docker.

@humitos we can't merge this till we can retrieve the git data from inside the container.

This PR was superseded by #6436. Feel free to re-open if it's still valid.


Spike: Email invites for large project potentially causing performance issues

For a project email invite through Mandrill, we might encounter a severe performance degradation when the target email audience is large. This happened in PROD for a group of roughly 60 helpers that were invited to a project.

@phillipoertel Do you think this could be a recurring pattern for Mandrill calls? The Rails app failed when the Mandrill service took too long, but the email were in the end sent (in Mandrill logs).

It would be best to always send the invitations asynchronously. Maybe Mandrill supports that out of the box; if not, use a queue system like delayed_job or sidekiq. Both can be used through Rails' ActiveJob interface, IIRC.

Closing based on @phillipoertel's research of options with decision to implement sidekiq worker process for delayed email delivery. See implementation in #449.


Use upstream portmap plugin releases (when available)

We currently host our own build of the portmap plugin to include in calico/cni.  We should switch to the upstream release when v0.6.0 comes out.

why is this issue in this repo? shouldn't it be in calico/cni-plugin?

I pinged Casey over slack just a few minutes ago asking the same thing.

Sigh.  Lost track of where I was.


Window not opaque on some wayland compositor

GLFW windows on swaywm and rootston are always transparent.
Current method to make sure window is opaque, set_opaque_region, according to wayland protocol description, is just an optimization hint. I don't think it would necessarily make window opaque.
According to wayland mailing list, the proper way to make it opaque is to use a buffer without alpha channel.
To reproduce, comment out the transparent framebuffer hint line in the gears example and run it on swaywm or rootston.
Although it can be fixed by manually calling glfwWindowHint(GLFW_ALPHA_BITS, 0), I think this should be default on wayland.

The issue with that is how the Wayland code and the EGL code are completely isolated inside of GLFW. This is useful for Vulkan for instance, which doesn’t use EGL at all, or when you ask for no context during window creation.
Another issue with changing the framebuffer configuration before EGL initialisation is that some programs may expect to have an alpha channel if they require one and store things into it, even if it isn’t used for composition.
I’m not sure how to solve this issue, which also happens for Vulkan I guess, so I’ll leave this issue open for other people to chime in.

I just began using GLFW, so correct me if what I'm saying is non-sense, but I don't understand what is wrong with the current behaviour. When I initialise a window, it is by default, completely transparent. I can then use OpenGL to add colours with alpha set to unity if I want opacity. Is that not so?

This issue is about windows created with GLFW_TRANSPARENT_FRAMEBUFFER set to false is still rendered as transparent on some wayland compositor, which is not the intended behavior. The current implementation marks the region opaque by wl_surface_set_opaque_region, which, by definition, is just a optimization hint (so compositors may ignore it). Filling alpha to make it opaque is indeed a workaround, but people may assume their things will be rendered like there is a black background with GLFW_TRANSPARENT_FRAMEBUFFER set to false.

Okay, thanks for the explanation! I'm using this via pyGLFW, and when I test this as follows:
glfw.window_hint(glfw.TRANSPARENT_FRAMEBUFFER, False)
if not glfw.get_window_attrib(pane, glfw.TRANSPARENT_FRAMEBUFFER):
    raise RuntimeError("Opacitisation failed")

I do indeed get the error (and without the check, it renders a transparent window).
I'm using KDE's plasma wayland session, which uses the KWin compositor. So I suppose it ignores that optimisation suggestion.

What you should do is use an RGBX config instead of a RGBA one, that way the alpha bits will correctly get ignored on all platforms.

This is now fixed, when using drivers which expose the EGL_EXT_present_opaque extension, such as Mesa.


Missing redirect URL in application registration.

Hi when I try to sign in using a Flow I get the following error: Missing redirect URL in application registration.
I have verified that the following Redirect URL https://global.consent.azure-apim.net/redirect is in our ServiceNow application registry.
Am I supposed to use a Flow Url instead?

There was a typo in the application registry but now I'm getting the following error:
Server Error in '/' Application.
The length of the query string for this request exceeds the configured maxQueryStringLength value.

Hi @parclarke! Thanks for submitting this issue. It looks like this issue has to do with the existing connector ServiceNow. Please submit your question to the forum to get help:  https://powerusers.microsoft.com/t5/Using-Connectors/bd-p/ConnectingToData


Blastoff.zone: Earn 1 ETH

Blastoff.zone: Claim Your Share of 1 Million Tokens and $500,000!
Congratulations! You've just unlocked an exclusive opportunity to claim your share of 1 million tokens and a staggering $500,000 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500,000 bonus.
Use the "Claim Now" option to secure your tokens and cash in on the rewards.

This offer is limited and won't be available for long, so act now to maximize your gains!
Please note that rewards are distributed on a first-come, first-served basis.

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。


Removed minsdk from manifest

Fixes bug https://github.com/mauron85/react-native-background-geolocation/issues/357

Merged. Thank you for your contribution!


Add missing conversion class from vector of WheelTelemetryData to Python list

Description
Add missing conversion class from vector of WheelTelemetryData to Python list
This adds a missing conversion from vector of WheelTelemetryData to a Python list. When trying to access the "wheels" attribute of the VehicleTelemetryData without this, this exception will occur:
TypError: No to_python (by-value) converter found for C++ type: std::vector<carla::rpc::WheelTelemetryData, std::allocatorcarla::rpc::WheelTelemetryData>
This addition fixes this issue to make the usage in line with similar classes such as "VehiclePhysicsControl" and "WheelPhysicsControl"
Checklist:

[x] Your branch is up-to-date with the dev branch and tested with latest changes
[x] Extended the README / documentation, if necessary
[x] Code compiles correctly
[x] All tests passing with make check (only Linux)
[x] If relevant, update CHANGELOG.md with your changes

Where has this been tested?

Platform(s): Linux
Python version(s): 3.10.12
Unreal Engine version(s): <IP_ADDRESS>

This change is

I did not update the changelog as this is fixing something in the dev branch that is not yet released in any version of Carla: Latest Changes > Added API function get_telemetry_data to the vehicle actor.


Jsn/hacked domain

Fix everything to a new domain spf-tools.eu.org

Apologies that it took so long. Now we are back to green tests.


Issue in return of user_clicked_square

At the moment the function returns None.
I'd like it to return adjacent_mines from function mines_around_square, but there is a scope problem.
I would like to keep ajacent_mines inside mines_around_square. Is there another way to solve this?

Why should it return adjacent mines?

Why should it return adjacent mines?

Maybe I should clarify what I meant with the above comment. Instead of returning the number of adjacent mines, it can display it in the button, and it doesn't have to return anything. But we don't know yet how to display a number in the canvas, so that will have to wait.

Maybe I should clarify what I meant with the above comment. Instead of returning the number of adjacent mines, it can display it in the button, and it doesn't have to return anything. But we don't know yet how to display a number in the canvas, so that will have to wait.