Format-Table | Property's PSEdition values not exhibited

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
> Get-Module -ListAvailable | Where-Object -Property Name -Match "trusted" | Format-Table -AutoSize -Property ModuleType,Version,Name,PSEdition,ExportedCommands

Expected behavior
Exhibited in output:
- `Directory: C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules`
- Property's _PSEdition_ values, `Core,Desk `
as follows:

> Get-Module -ListAvailable | Where-Object -Property Name -Match "trusted" | Format-Table -AutoSize -Property ModuleType,Version,Name,PSEdition,ExportedCommands

    Directory: C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules
ModuleType Version Name                  PSEdition ExportedCommands
---------- ------- ----                  --------- ----------------
Manifest   <IP_ADDRESS> TrustedPlatformModule Core,Desk {[Get-Tpm, Get-Tpm]…

Actual behavior
Not exhibited in output:
- `Directory: C:\WINDOWS\system32\WindowsPowerShell\v1.0\Modules`
- Property's _PSEdition_ values, `Core,Desk `
as follows:

> Get-Module -ListAvailable | Where-Object -Property Name -Match "trusted" | Format-Table -AutoSize -Property ModuleType,Version,Name,PSEdition,ExportedCommands

ModuleType Version Name                  PSEdition ExportedCommands
---------- ------- ----                  --------- ----------------
  Manifest <IP_ADDRESS> TrustedPlatformModule           {[Get-Tpm, Get-Tpm]…

Error details
_

Environment data
> PSVersion                      7.3.6
PSEdition                      Core
GitCommitId                    7.3.6
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
_

It sounds like you're looking for the .CompatiblePSEditions property instead.

The PSEdition-named property as an exception among the properties exhibited by the Get-Module -ListAvailable's output, doesn't query the values that it reports while being as property of Format-Table pipe-line. Your mention CompatiblePSEditions, which is not even exhibited by the Get-Module -ListAvailable's output, cannot be logically.deduced. If it was not intuition or inspiration, then i have no clue what led you to determine that it was the option to apply.
> Get-Module -ListAvailable | Where-Object -Property Name -Match "trusted" | Format-Table -Property CompatiblePSEditions
CompatiblePSEditions
--------------------
{Desktop, Core}

If i had had to seek for something, that would have been most likely consistency.

The behavior is provided by the experimental PSCustomTableHeaderLabelDecoration feature (available starting with preview versions of v7.4), which applies special formatting - italics by default, controllable via $PSStyle.Formatting.CustomTableHeaderLabel - to any column name that doesn't match the name of a property of the type being formatted.
As with any experimental feature, it isn't guaranteed to become a stable feature, however.

As the column named PSEdition of the Get-Module -ListAvailable's output on purpose differs from the underlying property name CompatiblePSEditions, and the Get-Module -ListAvailable | Format-Table -Property PSEdition's output is correct, so there is no issue.


Provide a synchronous API

It makes sense that the API is asynchronous by default, but it would be nice to have another API which is synchronous (it returns the highlighted text and doesn't take a callback). This is particularly useful for server-side usage of Rainbow. Otherwise setTimeout gymnastics are required (see #7).

Support for node.js and Rainbow.colorSync was added in version 2.0 which finally was released.
npm install rainbow-code

var rainbow = require('rainbow-code');
var highlightedCode = rainbow.colorSync('var finally = true;', 'javascript');


Added Djerba Institute

Added Higher institute of technological studies of Djerba

@Fakhredinne21 First stage of verification passed, official domain is isetjb.rnu.tn as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: http://www.isetjb.rnu.tn/docs/divers/creation-compte-ent.pdf

@Fakhredinne21 I am sorry, the website is inaccessible, and I cannot verify the domain eligibility.
I'll be back to this request later. In the meantime please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
Thank you!

@Fakhredinne21 I am sorry, the website remains inaccessible, and I cannot proceed with the request verification. I have to close it without merging. Thank you for your understanding!


Update sbt to 1.4.7

Updates org.scala-sbt:sbt from 1.4.6 to 1.4.7.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

labels: library-update, semver-patch

Codecov Report

Merging #85 (3025a80) into master (011aee9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #85   +/-   ##
=======================================
  Coverage   87.13%   87.13%
=======================================
  Files          28       28
  Lines         832      832
  Branches       31       31
=======================================
  Hits          725      725
  Misses        107      107

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 011aee9...3025a80. Read the comment docs.

Codecov Report

Merging #85 (3025a80) into master (011aee9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #85   +/-   ##
=======================================
  Coverage   87.13%   87.13%
=======================================
  Files          28       28
  Lines         832      832
  Branches       31       31
=======================================
  Hits          725      725
  Misses        107      107

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 011aee9...3025a80. Read the comment docs.


Jammy Ubuntu

Seems the newer LTE Ubuntu 22.04.1 LTS uses libssl3, and thus libssl1.1 is deprecated at this point.

Thank you for the report. Will spin a Ubuntu 22.04 instance on one of the cloud providers and test. CentOS8 ships with openssl-1.1.1 so that's the only one that got tested.


Does anyone use Have_XXX version identifiers?

I'm not sure why it's there. I feel like this could be used to bypass proper dependencies chains.
You can make completely optional dependencies with a Derelict-style loader.

I've recently found them very useful for optional dependencies.
For example a project has two optional local dependencies liba and libb (not available on the dub registry). The project has a corresponding module for each of them, which acts like a wrapper. In order for the project to be buildable without them, you can do:
module liba_wrapper;

version (Have_liba):

struct LibAObj { }

module libb_wrapper;

version (Have_libb):

struct LibBObj { }

module useful_functionality_on_top;

// This function accepts argments of both
// type LibAObj and type LibBObj.
auto use(T)(T obj);

I forgot to mention that this more flexible than using excludedFiles in dub.sdl, becuase excludedFiles breaks imports.
Continueing the previous example:
module backends;

public import liba_wrapper;
public import libb_wrapper;

//not a an external dependency
public import test_wrapper;

OK then.


chore: update all dependencies and test against Node LTS

Just wanted to give this library some TLC, and bring everything up to date.
@jamestalmage @sindresorhus @bcoe any reservations?

@novemberborn sure, but I don't think I have publish access.

@JaKXz - You do now

+<EMAIL_ADDRESS>published! Thanks gentlemen


request help: Why does 'the enabling https redirect' redirect to the https scheme with the port 9443?

Issue description
I've deployed a whoami image and set up route with the https redirect enabled.
Now the browser will redirect http://domain.com/foo to https://domain.com:9443/foo
Any ideas how this happened?
Environment

your apisix-ingress-controller version (output of apisix-ingress-controller version --long):
your Kubernetes cluster version (output of kubectl version):
if you run apisix-ingress-controller in Bare-metal environment, also show your OS version (uname -a):

This happened when you enable the https redirect on the admin dashboard UI.

Setting the http_to_https redirect plugin in the yaml config file also get the same result. Anybody who can look into how to resolve this issue?

duplicate https://github.com/apache/apisix/issues/7172

I'm using the latest version of APISIX, obviously this issue still hasn't been resolved yet.

Please refer to this document to configure APISIX
https://apisix.apache.org/docs/apisix/plugins/redirect/

Only one of http_to_https, uri and regex_uri can be configured.
When enabling http_to_https, the ports in the redirect URL will pick a value in the following order (in descending order of priority)
Read plugin_attr.redirect.https_port from the configuration file (conf/config.yaml).
If apisix.ssl is enabled, read apisix.ssl.listen_port first, and if it does not exist, read apisix.ssl.listen and select a port randomly from it.
Use 443 as the default https port.

Why can't you just make this plugin to work with the default 443 https port out of the box? It's too difficult for newcomers like me to figure out how to get it working correctly even if you just throw the documentation here. It still looks too complicated for me to figure out where the conf/config.yaml file really is in my kubernetes cluster.

I guess you can add a Tip me button in the issue section so that I'm willing to send a few hundred bucks as tips to help speed things up a little.

Please refer to this document to configure APISIX
https://apisix.apache.org/docs/apisix/plugins/redirect/

Only one of http_to_https, uri and regex_uri can be configured.
When enabling http_to_https, the ports in the redirect URL will pick a value in the following order (in descending order of priority)
Read plugin_attr.redirect.https_port from the configuration file (conf/config.yaml).
If apisix.ssl is enabled, read apisix.ssl.listen_port first, and if it does not exist, read apisix.ssl.listen and select a port randomly from it.
Use 443 as the default https port.

Technically speaking, https redirect only send a 301 or 302 response with a new url so that the browser will initiate a new https request with the new url, why is there a need to make it so complicated?

??

In fact, the APISIX Ingress controller is a control plane component, it does not control how APISIX is deployed (or the APISIX configuration file).
So as described in the above documentation, you can avoid redirection to port 9443 by modifying the APISIX configuration file.

Why can't you just make this plugin to work with the default 443 https port out of the box?

Also, the actual redirection port depends on how you are deploying.
If you directly let APISIX listen on ports 80 and 443, you won't have this problem.
For the APISIX Ingress controller project, we need to keep its behavior consistent with APISIX to avoid confusion for users.
So for this issue, we will not make any adjustments at this time.

I run into the same problem.  I deployed apisix by using helm-chart therefore I had to adapt the following paramter
      - name:  "gateway.tls.containerPort"
        value: "443"

which changes the value apisix.ssl.listen_port to 443, just in case somebody runs into the same problem.  I suggest the documentation (https://apisix.apache.org/docs/apisix/plugins/redirect/) should be updated.

redirect.https_port

Indeed it changes the container port but it dosn't solve the problem, redirect still rewrite the port to 9443. The workaround that worked for me was to edit the redirect plugin attribute "https_port" to match containerPort on 443
I m using the heml chart so it's sometjing like:
gateway:
  tls:
    enabled: true
    containerPort: 443
pluginAttrs:
  redirect:
    https_port: 443

Out of interest, I alsow used the helm-chart but the pluginsAttrs redirect was not part of the config created by helm, I guess thats why I needed to adapt the containerport.

https://github.com/apache/apisix-helm-chart/blob/46d0096fd23fdcad58e8337d62c78550c0014b46/charts/apisix/values.yaml#L159
you can try to change this item to 443

For me adding the helm paramter "gateway.tls.containerPort"  value: "443"   did the trick!   First I tried to change  "pluginAttrs.  redirect.https_port: 443'  like @ohayak did but it did not work as  "pluginAttrs.redirect.https_port' was not part of the configuration.    Out of interest, I would like to know how he managed to add the prameter "pluginAttrs.redirect.https_port"  to the config.

Why can't you just make this plugin to work with the default 443 https port out of the box? It's too difficult for newcomers like me to figure out how to get it working correctly even if you just throw the documentation here. It still looks too complicated for me to figure out where the conf/config.yaml file really is in my kubernetes cluster.

Just want to share this.
I've been fumbling around with this all week. Apparently, the configs for apisix are store in the configmaps. So you do this:

kubectl edit configmap -n  apisix-gateway
kubectl edit configmap -n  apisix

I find that restarting the deployment is more reliable than just reloading as it needs to remount the files.


xCell bug with names

need to check the xcell code and rebuild the docker image

This is fixed. Need to merge into main to update the docker image

Fixed


Why string, not str ?

https://github.com/ArjanCodes/betterpython/blob/4151e184748856cbb3ea32ff8ef1bcca448e6a68/1 - coupling and cohesion/coupling-cohesion-after.py#L60

I think is a typo error, and the interpreter didn't show any errors because he had the string library imported.

https://github.com/ArjanCodes/betterpython/blob/4151e184748856cbb3ea32ff8ef1bcca448e6a68/1 - coupling and cohesion/coupling-cohesion-after.py#L60

I think is a typo error, and the interpreter didn't show any errors because he had the string library imported.


Clean Up action does not work for distorted reaction. In Standalone mode: Error: core:  is not a base molecule

Steps to Reproduce

Create some reaction on the canvas
Distort it
Click Clean Up

Expected behavior
After Clean Up action structures are undistorted. Position of the reaction does not change.
Actual behavior
Structures are NOT undistorted. In Standalone mode: Error: core:  is not a base molecule
Screenshots
https://user-images.githubusercontent.com/93334478/146729869-fd0bd8ef-5349-496b-acd3-b6f2fc97646d.mp4
Desktop (please complete the following information):

OS:Windows 10
Browser Chrome Version 93.0.4577.63, Firefox Version 94.0.1, Microsoft Edge Version 93.0.961.47

related issue: https://github.com/epam/Indigo/issues/572

Moved to  2.5
cc: @AllaZhbanova


Remove plugins

Plugins are an unused feature of this library. They will likely be
reimplemented at the collector level.
Resolves #375

The plugin package had a high level of test coverage, so removing it will bring down the average on the repo. I think this is not a reason to block this change, nor should increased coverage in unrelated packages be required here.


Fixed import in the combineLatest example

Fixed the import in the example

Pull Request Test Coverage Report for Build 8094

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

Totals

Change from base Build 8066:
0.0%

Covered Lines:
5242

Relevant Lines:
5415

💛  - Coveralls


load_dataset does not load all of the data in my input file

Describe the bug
I have 127 elements in my input dataset.  When I do a len on the dataset after loaded, it is only 124 elements.
Steps to reproduce the bug
train_dataset = nlp.load_dataset(data_args.dataset_path, name=data_args.qg_format, split=nlp.Split.TRAIN)
valid_dataset = nlp.load_dataset(data_args.dataset_path, name=data_args.qg_format, split=nlp.Split.VALIDATION)
logger.info(len(train_dataset))
logger.info(len(valid_dataset))

Both train and valid input are 127 items.  However, they both only load 124 items.  The input format is in json.  At the end of the day, I am trying to create .pt files.
Expected behavior
I see all 127 elements in my dataset when performing len
Environment info
Python 3.10.  CentOS operating system.  nlp==0.40, datasets==2.14.5, transformers==4.26.1

You should use datasets.load_dataset instead of nlp.load_dataset, as the nlp package is outdated.
If switching to datasets.load_dataset doesn't fix the issue, sharing the JSON file (feel free to replace the data with dummy data) would be nice so that we can reproduce it ourselves.


AttachedToVisualTree or Loaded on UserControl gives Invalid Markup

Describe the bug
When trying to use AttachedToVisualTree or Loaded on a UserControl element it gives an Invalid Markup in the axaml designer.
To Reproduce
<UserControl xmlns="https://github.com/avaloniaui"              xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"              xmlns:d="http://schemas.microsoft.com/expression/blend/2008"              xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"              mc:Ignorable="d" d:DesignWidth="800" d:DesignHeight="450"              x:Class="AvaloniaPad.MainView"  Name="MainViewUC" Loaded="UserControl_Loaded"></UserControl>
Expected behavior
No response
Avalonia version
11.1.3
OS
Windows
Additional context
No response

The known issue has been fixed in 11.2-beta1.
Workaround, add the code as follows to your user control
	public static void AddLoadedHandler(Interactive element, EventHandler<RoutedEventArgs> handler)
	{
		element.AddHandler(LoadedEvent, handler);
	}


Actually support the format option in the ES API's RangeQuery

the parameter is introduced in #5098, but is noop at the moment.
https://www.elastic.co/guide/en/elasticsearch/reference/current/query-dsl-range-query.html
It makes it possible for the client to pass dates in a specific format
https://www.elastic.co/guide/en/elasticsearch/reference/current/mapping-date-format.html

It can be done by doing the conversion in the elastic search dsl range query (quickwit-query/src/elastic_query_dsl/range_query.rs)
and reserializing it into rfc3339 before creating the QueryASt's range query.


Ss 535 add helm chart for shiny apps

Status

[X] Ready
[ ] Draft
[ ] Hold

Description
LINK
I've added a helm chart for RShiny.
This chart assumes that the user uses port 3838 and does not need to mount any volumes.
If they need these features, they should use the standard-app i think. Changes are welcome!
I've tested this locally and manage to deploy a couple of images.
Types of changes
What types of changes does your code introduce to STACKn?

[ ] Hotfix (fixing a critical bug in production)
[ ] Bugfix
[X] New feature
[ ] Documentation update

Checklist
If you're unsure about any of the items below, don't hesitate to ask. We're here to help!
This is simply a reminder of what we are going to look for before merging your code.

[X] This pull request is against develop branch (not applicable for hotfixes)
[X] I have included a link to the issue on GitHub or JIRA (if any)
[ ] I have included migration files (if there are changes to the model classes)
[ ] I have included, reviewed and executed tests (unit and end2end) to complement my changes
[ ] I have updated the related documentation (if necessary)
[ ] I have updated the release notes (releasenotes.md)
[X] I have added a reviewer for this pull request
[X] I have added myself as an author for this pull request
[ ] In the case I have modified settings.py, then I have also updated the studio-settings-configmap.yaml file in serve-charts

I like the simple user interface to create the app, but an advanced field to specify a custom port would be nice.


[Sema] Accept IgnoreCollectionElementContextualMismatch in conflict generic diagnostics

Follow up to #61061, to improve a diagnostic.

@swift-ci Please test


[simple-browser] allow popups to enable some auth flows

See for original request: https://discord.com/channels/816244985187008514/816246578594840586/914609326004527115

Please share all relevant info in the issue


Sorry, the dataset link is invalid, can I provide another copy?

The datasets are publicly available. If you google search the names of the dataset I think you would be able to find it. I had copies of the processed datasets in google drive. But unfortunately, the google drive for my previous institution got deleted.


Handle autostart-tray

Exit the tray on start when autostart-tray is set to false?
Need configuration item?

autostart-tray config option is very much tied to crc setup it was meant to be used with it, the handler for this config is at: https://github.com/code-ready/crc/blob/6ff0170104e74acbee36d492969b3db1eb3eaf08/pkg/crc/config/settings.go#L52-L63
in the current native trays, you can only change this option from the tray, but unless user runs the crc cleanup and crc setup it doesn't produce any effects.

Tasks

[ ] remove the autostart-tray config option from crc
[ ] handle setting autostart from the tray

@anjannath finished, right?

@gbraad Yes, all are done closing!


Is there a test of the beans.xml bean-discovery-mode attribute in Lite?

Are we validating the Lite implementations need to parse the beans.xml bean-discovery-mode attribute in Lite?

I am not sure that's testable universally for Lite and Full. But maybe I am just missing something. Here are my thoughts...
There is 3 values it can have:

none

The issue with this is that Lite aims to create a singular bean archive and if you create a test that has one archive with discovery none, then you don't have any bean archive at all
And if memory serves, I have come across an issue on WFLY where I found out that they have optimizations where in such case they don't even start CDI container for that deployment (which isn't wrong, there is no point in booting it)
However, that makes it pretty untestable

annotated

There is little point in doing this because the behavior is exactly the same as empty beans.xml meaning the test could easily show positive result even if implementor doesn't parse it

anything else; since anything else is not supported in CDI Lite

Here I am not sure CDI spec says what kind of error to throw here if any
Again, note that if you add something like mode all, then any CDI Full implementation running this test will correctly pass it

So we have this statement in the beans_4_0.xsd schema:
      <xs:attribute name="bean-discovery-mode" use="optional" default="annotated">
        <xs:annotation>
          <xs:documentation>
            It is strongly recommended you use "annotated". This is now the default and it
            is also the default as of 4.0 when an empty beans.xml file is seen. When running
            in a CDI Lite  environment, this is the only aspect of the beans.xml file that
            is used.

            If the bean discovery mode is "all", then all types in this
            archive will be considered. If the bean discovery mode is
            "annotated", then only those types with bean defining annotations will be
            considered. If the bean discovery mode is "none", then no
            types will be considered.
          </xs:documentation>

Should we just be saying that in a Lite environment this is not read?

It needs to be read in Lite, but portable testing of the none value seems hard, per Matej's explanation above.
Hmm maybe we could have a test with 2 archives, both of them having beans.xml, one with annotated bean discovery mode, the other with none (but some classes with BDAs). Then we could test that only beans from the first archvie exist. @manovotn, do you think the TCK can do that? I think we often use a WAR structure, so we could put some archives into WEB-INF/lib?

Hmm maybe we could have a test with 2 archives, both of them having beans.xml, one with annotated bean discovery mode, the other with none (but also some classes with BDAs). Then we could test that only beans from the first archvie exist. @manovotn, do you think the TCK can do that? I think we often use a WAR structure, so we could put some archives into WEB-INF/lib?

Sure that's definitely possible but I didn't bring it up because I thought we don't want to do that for Lite. Apparently, I missed the fact that we already have lots of Lite tests with WAR.
We just need to make sure that the deployment structure is valid for Full - meaning that the WAR should have annotated discovery and the library should have none else you'd bump into visibility issues between WAR and its lib when attempting resolution.

It needs to be read in Lite

I agree but we might want to change the wording in schema so that we mention all isn't supported in Lite? Or, better still, that it is undefined?

Hmm maybe we could have a test with 2 archives, both of them having beans.xml, one with annotated bean discovery mode, the other with none (but also some classes with BDAs). Then we could test that only beans from the first archvie exist. @manovotn, do you think the TCK can do that? I think we often use a WAR structure, so we could put some archives into WEB-INF/lib?

Sure that's definitely possible but I didn't bring it up because I thought we don't want to do that for Lite. Apparently, I missed the fact that we already have lots of Lite tests with WAR. We just need to make sure that the deployment structure is valid for Full - meaning that the WAR should have annotated discovery and the library should have none else you'd bump into visibility issues between WAR and its lib when attempting resolution.

+1 on adding tests to test none and others.

It needs to be read in Lite

I agree but we might want to change the wording in schema so that we mention all isn't supported in Lite? Or, better still, that it is undefined?

I am unclear about the all as bean-discovery-mode in lite. Do we treat as if saying annoted or we fail with invalid beans.xml. We need to clarify the behaviour. What about other invalid values such as mispelt anotated? I think it is better to thrown error message for anthing other than none or annotated in lite.

I am unclear about the all as bean-discovery-mode in lite. Do we treat as if saying annoted

No because that would create a non-portable behavior between Lite and Full (i.e. taking the same deployment to CDI Full would result in different behavior).

or we fail with invalid beans.xml.

This is the tricky part - you cannot fail. Remember that all CDI Lite tests are executed by CDI Full implementations as well and those have to understand all mode and pass the test! Hence this one case is not testable within TCKs which is why I suggested to say it is undefined. However, Lite implementations can (and probably should) choose to react somehow and failure seems like a good option.

What about other invalid values such as mispelt anotated?

This remains the same as it was up until now.

I think we could possibly go one step further and define how exactly a bean discovery mode of "all" is non-portable.
We currently say:

Any other archive which contains a beans.xml file is not portable in CDI Lite. More kinds of bean archives exist in CDI Full.

We could say something like:

Any other archive which contains a beans.xml file is not portable in CDI Lite. More kinds of bean archives exist in CDI Full.
Implementations that do not support CDI Full are required to fail deployment of an archive which contains a beans.xml file that has bean-discovery-mode attribute set to all.

That said, I don't think this is a big deal.

We could say something like:

We could, but it will, again, not be testable so I don't think it's worth the hassle. Alternatively we could just say that those impls are "encouraged" to treat it as deployment problem.
The only way we could test these things would be a property indicating mode in which the TCK is executed and the assertions tuned to that. But I would be -100 on that one because it just makes TCK way less readable and the results dependent on setting some knobs.

Hmm maybe we could have a test with 2 archives, both of them having beans.xml, one with annotated bean discovery mode, the other with none (but also some classes with BDAs).

@Ladicek I was thinking about this some more and we should try it but I am not sure its runnable everywhere.
As in, the most commonly used weld-embedded mode might fail, because arquillian-container-weld simplifies the deployment and merges it into one archive. That also means, it has to merge its beans.xml into one.
Looking at current TCKs, I can see we have BeanDiscoveryTest, see here. This is a Full test with all kinds of archives but half of its tests are marked as group INTEGRATION and I think this is precisely the reason.

What is the Lite behaviour when it sees bean-discovery-mode=all? I don't think it is enough to say non-portable behaviour occurs. What is the Weld defined behaviour?

What is the Weld defined behaviour?

Weld is a CDI Full implementation. As such it understands all as any CDI implementation did up until now.

What is the Lite behaviour when it sees bean-discovery-mode=all?

Please see the spec doc, 2.11.1. Bean archives.
PDF version from the last release has a link mentioned here - https://eclipse-ee4j.github.io/cdi/announcement/2022/01/25/400-RC3-spec.html
That is, to my knowledge, all the definition we have.
Since we wanted Lite to be subset of Full and to ensure portability, we instead approached this from the opposite direction and stated which archives are legitimate (and hence subsequently portable) for CDI Lite.

I don't think it is enough to say non-portable behaviour occurs.

It can't possibly be portable between across Lite-only and Full, so we have to say it's non-portable. What do you suggest we say on top of that?

Hmm maybe we could have a test with 2 archives, both of them having beans.xml, one with annotated bean discovery mode, the other with none (but also some classes with BDAs).

@Ladicek I was thinking about this some more and we should try it but I am not sure its runnable everywhere. As in, the most commonly used weld-embedded mode might fail, because arquillian-container-weld simplifies the deployment and merges it into one archive. That also means, it has to merge its beans.xml into one. Looking at current TCKs, I can see we have BeanDiscoveryTest, see here. This is a Full test with all kinds of archives but half of its tests are marked as group INTEGRATION and I think this is precisely the reason.

Fair enough. I'm fine with keeping the situation as is, or we could have a group LITE_INTEGRATION? :-)

or we could have a group LITE_INTEGRATION

Crossed my mind as well but then it gets complicated for Lite implementation to tell which groups they need to execute and even worse for Full impls.
Frankly, I'm too +1 for keeping it as is.

What is the Weld defined behaviour?

Weld is a CDI Full implementation. As such it understands all as any CDI implementation did up until now.

What is the Lite behaviour when it sees bean-discovery-mode=all?

Please see the spec doc, 2.11.1. Bean archives. PDF version from the last release has a link mentioned here - https://eclipse-ee4j.github.io/cdi/announcement/2022/01/25/400-RC3-spec.html
That is, to my knowledge, all the definition we have. Since we wanted Lite to be subset of Full and to ensure portability, we instead approached this from the opposite direction and stated which archives are legitimate (and hence subsequently portable) for CDI Lite.

The lite does not say anything about what other value except "annotated" behaves. Some runtime only suppots Lite only and I think the value of all maybe can be translated to annotated as Lite only supports annoated or none. If we add. a test, the test should check different results based on whether running in lite or full. By the way, the none value as bean-discovery-mode was not speced in the Lite section.

By the way, the none value as bean-discovery-mode was not speced in the Lite section.

2.11.1. Bean archives, quoting:

An archive which:
• contains a beans.xml file with the bean-discovery-mode of none, or,
• contains a portable extension or a build compatible extension and no beans.xml file
113
is not a bean archive.

Some runtime only suppots Lite only and I think the value of all maybe can be translated to annotated as Lite only supports annoated or none

Nope. This break portability from Lite to Full.
Imagine you have an interface SomeBean and then you have @Dependent class BeanImpl1 implements SomeBean and also class BeanImpl2 implements SomBean. Furthermore, assume somewhere in your application there is @Inject SomeBean. Lastly, the archive has beans.xml with discovery mode all.
With your suggestion, the deployment will work with CDI Lite, but will result in ambiguous resolution in CDI Full because BeanImpl2 would suddenly be discovered as well. We simply cannot do that.

If we add. a test, the test should check different results based on whether running in lite or full.

There is no telling, from inside a test, in which "mode" you run. CDI Full implementations do not have a concept of running in Lite mode either, they are just Full which means they by definition support everything Lite has.
That is also a reason why we defined what is a bean archive in Lite as opposed to defining what isn't because the perception will be different in CDI Full.

understand the challenge. I think leaving the bean-discovery-mode with anything except none or annotated unspeced would cause confusions. If it is unspeced, working on Lite is different from working on Full. There is no portable

Some runtime only suppots Lite only and I think the value of all maybe can be translated to annotated as Lite only supports annoated or none

Nope. This break portability from Lite to Full. Imagine you have an interface SomeBean and then you have @Dependent class BeanImpl1 implements SomeBean and also class BeanImpl2 implements SomBean. Furthermore, assume somewhere in your application there is @Inject SomeBean. Lastly, the archive has beans.xml with discovery mode all.
With your suggestion, the deployment will work with CDI Lite, but will result in ambiguous resolution in CDI Full because BeanImpl2 would suddenly be discovered as well. We simply cannot do that.

What is your proposal for maintaining portability from Lite to Full. I feel not specing the behaviour in Lite is even worse as the end users do not know what it will happen in lite when using Full as bean-discovery-mode.

If we add. a test, the test should check different results based on whether running in lite or full.

There is no telling, from inside a test, in which "mode" you run. CDI Full implementations do not have a concept of running in Lite mode either, they are just Full which means they by definition support everything Lite has. That is also a reason why we defined what is a bean archive in Lite as opposed to defining what isn't because the perception will be different in CDI Full.

We need to agree on a behaviour and then add a test. I am not sure what is the speced behaviour for the bean-discovery-mode=all.

I believe we have to distinguish 2 things:

our ability to specify the desired behavior
our ability to test that an implementation conforms to the specification

I submitted #591 with my proposal for dealing with item no 1. Feel free to comment there.
When it comes to item no 2., we may have to admit that testing deployment problems is always hard and certain things may just be left untested.

What is your proposal for maintaining portability from Lite to Full.

We already have all we need to maintain portability from Lite to Full. So long as the Lite application conforms to defined behavior, it is portable. The point of this issue is with mode all which is not ATM specced as valid value for Lite.

We need to agree on a behaviour and then add a test. I am not sure what is the speced behaviour for the bean-discovery-mode=all.

The inability of testing whatever behavior we agree on is the crux of this issue. And the reason why you cannot test it is precisely portability and the general ability to execute Lite under Full. Such test would inevitably have to expect contradicting results under Lite and Full.

I believe we have to distinguish 2 things:

Yes, that's good way of putting it, thank you.
We can adjust the wording if we decide to but we have to acknowledge that we still won't be able to enforce it via TCKs.


Fix man page to mention race condition

Also modify man page to talk about podman-container-logs as well as podman-logs.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

@edsantiago @TomSweeneyRedHat PTAL

LGTM
push and search tests are failing which I can't imagine is due to this PR.

[Fail] Podman push [It] podman push to local registry with authorization
Yes I know about that, it's breaking everywhere, no idea why.  :frowning:

/lgtm


What is the recommended way to load and use ply models?

I seem to not be able to get a simple PLY model loaded up. This is one of the stock models that comes with MagicaVoxel. My network traffic shows the file getting loaded. It does indeed appear to be downloaded correctly by the browser, but no rendering occurs. Aframe inspector shows that an entity does indeed exist.
First I import extras like this: import 'aframe-extras';
Then I added my model as an asset
<a-assets>
          <a-asset-item id="castle" src="/assets/castle.bake.ply"></a-asset-item>
</a-assets>

Finally I add it to the scene, using Entity
<Entity plyModel="src: #castle" rotation="-90 0 0" position="0 0 0" />

Any obvious issues here?

Objs/mtls work fine using the above method. Closing since I don't think anything is going wrong with this aframe-react.


Post Setup fails to cleanup and fails the entire test

I had a test that failed during cleanup:
Post job cleanup.
/usr/bin/sg microk8s -c juju models --format=json | jq -r '.models[]."short-name"' | grep -v controller | xargs -n 1 juju destroy-model --destroy-storage --force --no-wait -y
Destroying model
Waiting for model to be removed, 5 application(s).....
Waiting for model to be removed..........................
Model destroyed.
ERROR cannot connect to API: model "test-lma-bundle-b8hw" has been removed from the controller, run 'juju models' and switch to one of them.
Destroying model
Waiting for model to be removed...........................
Model destroyed.
Error: The process '/usr/bin/sg' failed with exit code 123

https://github.com/charmed-kubernetes/actions-operator/blob/e90e80eeb3e5423436297d5ba11494543c42c862/src/cleanup/index.ts#L20

looks like it picked up a model that was already being destroyed and then it was already gone by the time it tried to (re)remove it. I wonder if we even really need to care about doing that cleanup since we expect the runner VM / container to go away after that anyway?  At the very least, we could add an || true to the destroy-model call. -- @johnsca

I haven't seen this error since. Feel free to close.


MM-12153: Remove Default Server Language option from the config panel

Summary
Remove Default Server Language option from the config panel
Ticket Link
MM-12153
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed

+1 on George.  I missed that.

It wasn't clear for me from the conversation. But now is changed.

@grundleborg PTAL


JAVA-3164: Fix rendering UNSET collection types in query tracing [4.1]

Backport of PR-3482.

https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/1386/workflows/653f0c3a-114c-40e2-9034-596679aab865

https://app.circleci.com/pipelines/github/jacek-lewandowski/cassandra/1386/workflows/2b43b158-92db-4bf8-8b5f-9de67014063d


Fix "How it works" docs

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 36.75%. Comparing base (2b2f9d4) to head (dc6ca3e).
Report is 1 commits behind head on main.

:exclamation:  There is a different number of reports uploaded between BASE (2b2f9d4) and HEAD (dc6ca3e). Click for more details.
HEAD has 7 uploads less than BASE

Flag
BASE (2b2f9d4)
HEAD (dc6ca3e)

8
1

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #193       +/-   ##
===========================================
- Coverage   91.90%   36.75%   -55.16%
===========================================
  Files          48       47        -1
  Lines        2087     1804      -283
===========================================
- Hits         1918      663     -1255
- Misses        169     1141      +972

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


Does this layout class support vertical scrolling?

Currently not, but I was actually asked about this a few times and I consider this an official feature request ;)
This should be fairly easy to add, I'll let you know once it's done (hopefully by the end of upcoming weekend).


ImageMagick 7 released.

Looks like the time for cffi rewrite is near. ImageMagick 7 has moved out of beta, and has been distributed.
Porting over view here
Key Changes

PixelPacket replaced with Quantum Traits
Many typedef enum have been rewritten.
{METHOD}Channel have been replaced with SetImageChannelMask

I just spent two hours finding out that IM7 does not work with wand. (at least on Windows 7). It seems to be related to a rename of the core dll files:
IM6: CORE_RL_wand_.dll
IM7: CORE_RL_MagickWand_.dll
api.py looks for the IM6 dll file, which cannot be found in an IM7 installation.

Oh same here m(
rolled back to 6.9.5-Q16 and it works again. Pchew.

I also could not use Wand with IM7 and Windows10. I had to use the last IM6 release.

Closing this as the porting work for IM7 has been completed.


[3.10] LogAppender: Ensure we do not implicitly add ones under ReadLock #20102

Scope & Purpose
Backport of https://github.com/arangodb/arangodb/pull/20102
The LogAppenders do have a lock on the lists of appenders. We recently moved some of it from write to read lock. Now make sure we do not write under deadlock by using .at() instead of [] which could write implicitly. I do not think this has a real effect as the lists should be filled after startup.

[x] :hankey: Bugfix
[ ] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.11: (Please link PR)
[ ] Backport for 3.10: (Please link PR)

Related Information
(Please reference tickets / specification / other PRs etc)

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket:
[ ] Design document:

[x] https://jenkins.arangodb.biz/view/PR/job/arangodb-matrix-pr/32051/


feat(s3): Create specified bucket when it doesn't exist

If you prefer ensureBucketExists does not get called during init that's OK. I filled out that method primarily for Halyard's consumption.

👍

Please use naming conventions http://www.spinnaker.io/v1.0/docs/how-to-submit-a-patch

Naming conventionalized

Wanna rebase & we'll merge this guy?

We can get the same effect with Update branch + Squash and merge!


Use /usr/bin/env bash instead of /bin/bash in a number of developer scripts

I left alone anything I thought a contributor might reasonably be expected to run in the course of creating a cask, which basically means list_* and generate_cask_token (See #8023 and #8021).1
That being said, I'm a firm believer in using bash as defined in the user's environment so if there are no objections...

1: Although since ruby 2.0 is required anyway, this might as well fail imo

I don’t exactly oppose, as it likely doesn’t make a difference, but also don’t see why we should. While the user’s default bash may change, the OS’s won’t (since Apple can’t move to a later bash thanks to the GPL1), and it’s always better to write your script and be sure it’ll run exactly the same for every user.
Seems like we gain nothing by the change, but may lose something2.

1 Funny (read, not funny at all) how the supposedly “free” licenses give you so little freedom (leeway) in how to use what they give you.
2 Highly unlikely, since bash must be a hole of bad legacy decisions that can’t be changed for compatibility.

I don't care much either way, though I understand @vitorgalvao's concerns about compatibility. Using /bin/bash ensures that we know exactly which bash is running our scripts, all of the time (unless a dev explicitly runs bash cask-switch-https or something).

Funny (read, not funny at all) how the supposedly “free” licenses give you so little freedom (leeway) in how to use what they give you.

GNU greybeards hate commercial software. Like, hate it. It's like a religion or something. The thought of anyone making any money from their work without completely open-sourcing their software is just appalling to them. I don't get it.

I won't disagree with either of you, but that's why I kept this to only things that shouldn't be frequently used project-wide, where I don't feel "consistency" is a strong enough argument.
With that in mind, though, update_issue_template_urls uses env bash, and audit_modified_casks,1 generate_issue_template_urls, list_login_items_for_app, list_running_app_ids, and the_long_tail all use env ruby, which is certainly inconsistent.  Would you both be in favor of changing those, then?  Some of those are more important day-to-day.

If it's not PD it's not free.  Although I'm curious to see the future; there have been enough people wanting OSS on the app store that something could be inbound.

1: Although I think maybe travis needs this

My concern here wasn’t related to consistency, but to making sure things don’t break. I don’t mind env ruby because different OS X releases ship different versions of ruby, which means using the system ruby as opposed to the user’s ruby doesn’t offer any guarantees, anyway. However, using the system bash offers us the guarantee of what version it is, and since bash is a finickier language, sticking to what we are sure will work seems like a good bet.
In this case, I simply do not see a good argument to using env bash.

Would you both be in favor of changing those, then?

Yes, I would be in favour of making them all consistent within their own languages (so using env in one ruby script means they all should use it as well, but that doesn’t mean all bash scripts would need to follow the rule as well, and those could have their own different rule). Typically, in my own scripts, I #!/bin/bash and #!/usr/bin/env ruby.

GNU greybeards hate commercial software. Like, hate it.

One of the worst attitudes I’ve seen (read) from Stallman was a student at a college he gave a talk writting him a really polite and considerate letter arguing how he believed that his (Stallman’s) approach of telling people to blindly abandon all commercial software they use at once and start using “free” software was unrealistic. He made a really compelling case of suggesting an alternative attitude, teaching how to make the switch, which software to use for common cases that create hurdles, and the like. It was clear from the letter (and he indeed mentioned it) he was supportive of Stallman’s stance but was mostly worried that his approach was ineffective due to how he presented it, since without clear alternatives, people won’t know how to make the switch, and the whole attitude as a whole made people dismiss his message. Stallman’s reply was only (to the effect of) “I won’t discuss anything with people who don’t share my views”.
He literally believes his choices are the best possible, without knowing what the others are. Case in point.

If it's not PD it's not free.

Agreed. All my github repos are under The Unlicense for this reason.

@vitorgalvao I had basically your same response recently reading his 15-year old "Because I'm a bastard, and proud of it.  Just the worst possible attitude.

Because I'm a bastard, and proud of it

I wasn’t expecting he had written those exact words. It’s even more jarring to see that he did.

"Because I'm a bastard, and proud of it."

That's Linus Torvalds btw, not Richard Stallman. Still a bastard, though :wink:

Hah, thanks @jawshooah.  Didn't even notice...


Imperial Measurement

Hi,
Could we please add an environment variable to set the measurement to imperial instead of metric? The current values are in km so you'd have to convert miles using math. I live in UK and the python MQTT gateway works great with my MG4 Xpower. Many thanks to all contributors, very much appreciated!

Contributions are welcome.
If you use home assistant you can change the unit of measure of the sensor and it will do the conversion for you :)

Thanks for the tip @nanomad , works like a charm in HA!


Shortcut arguments

I think it could be cool for shortcuts to accept arguments.
An example might be a shortcut to visit a particular user profile, where the shortcut would be Visit user - hitting enter would allow you to enter some sort of identifier, I.e. username or email and then that would be made available to the shortcut handler.

Hey @ryangjchandler
Thanks for the suggestion! I'm not sure I completely understand. Commands support dependencies, so your example would be something like:
class VisitUserCommand extends SpotlightCommand
{
    protected string $name = 'Visit user';

    public function dependencies(): ?SpotlightCommandDependencies
    {
        return SpotlightCommandDependencies::collection()
            ->add(
                SpotlightCommandDependency::make('user')
                    ->setPlaceholder('Which use do you want to visit?')
            );
    }

    public function searchUser(Request $request, $query)
    {
        return User::where('name', 'like', "%$query%")
            ->get()
            ->map(function($user) {
                return new SpotlightSearchResult(
                    $user->id,
                    $user->name,
                    sprintf('Visit %s', $user->name)
                );
            });
    }

    public function execute(Spotlight $spotlight, User $user)
    {
        $spotlight->redirectRoute('users.show', $user);
    }
}

Is this what you are looking for, or did you mean something else? 😄

Yeah that covers it. The word dependency kind of threw me off...


Another exception was thrown: NoSuchMethodError: The getter 'iterator' was called on null.

I am trying to launch the map just with center position and tiles. I get this error. Can someone help to fix it please?
I'm using the version 0.4.0 of flutter_map and also the stable version 1.0.0 of Flutter.

In the run log of whichever IDE you use, does it reference a file and line number, that could help indicate which part of the code is null ?

No. There was no indication. But it's okay now, I found the problem and fixed it. The problem was that I didn't initialized marker and polyline arrays before using them for the first time.
Thanks!

How to Make the flutter logs more verbose and improve console for logging ?

Or at least the log should explain a little bit better that the error is caused by a null variable or badly asigned. I just found this issue because I had the same error and couldn't find what it was. In the end it was because I had a markers array and a function to build this array. The thing is that I was doing "markers = _buildMarkers()" but that function wasn't returning anything so the empty array was being overriden by a null value. Maybe some type checking before iteration in the Markers Layer would throw a more clear error.

there markers layer now sets the default value to an empty list


Influencer marketing setup

We want to tap into our network as effectively as possible, Nathan will be providing access to a micro-influencer software(don't remember the name but will) in which we can better manage relations we have with community members that can boost our visibility.
We can start by listing out current and potential influencers that we can start to solidify relationships with as we progress our go-to-market plan.

List influencers and track outreach (avoid duplicate outreach) via the "Ecosystem" Sheet:
https://docs.google.com/spreadsheets/d/1nYq9eHUGNr-N9n7WwfmssDPePU4O3J-tY9N2IeVe6DM/edit#gid=1565346490


ktor CacheStorage based persistent cache

This adds a persistent disk cache based on the ktor's CacheStorage. The underlying cache is a disk LRU is a multiplatform port of coil's DiskLruCache (which itself is a port from libcore disklrucache).
HttpCache can be configured as
val httpCacheSize = 1024*1024

KamelConfig {
    imageBitmapCacheSize = DefaultCacheSize
    imageVectorCacheSize = DefaultCacheSize
    svgCacheSize = DefaultCacheSize
    imageBitmapDecoder()
    imageVectorDecoder()
    svgDecoder()
    stringMapper()
    urlMapper()
    uriMapper()
    fileFetcher()
    httpFetcher {
        httpCache(httpCacheSize)
    }
  }

Amazing, thanks a lot for the contribution! I've been planning to work on #27 for a while, you've beaten me to it 🙌. I'll review this as soon as I can. In the meantime I can publish a snapshot if you want.

Please take your time to review it. I had to reorganize the codebase to be able to share the common code between different JVM targets and also specify Android-specific things at the same time. I'm open to any suggestions.

Looks good thanks again! @psuzn


fix mqtt_demo build error

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
Additional context

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.liangzeng.glz seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Having problem in weight loading...

I am not sure why this will be produced after I try to load the weights. I have been stuck on this for a month and have no idea how to solve this issue.

Check is the correct path mentioned in attached file which is stored under networks folder of detr_tf
Weights.txt


Estimate lm.maxweight

See #963.
Maybe we can calculate the average maxedout value and show that here (or max maxedout?)?
We can determine the maximum weight ourselfs and add 50%. Would be a heuristic approach, but maybe more convenient for new users.

:+1:

We can determine the maximum weight ourselfs and add 50%. Would be a heuristic approach, but maybe more convenient for new users.

We already do that in the issue :)

We already do that in the issue :)

Oh, you are right, I missed that, sorry.

Currently it just picks the last found maxed out value ... not sure if this is the best strategy but probably sufficient for now.

Ok, so I will leave it open. We can still close it when we don't see any issues with the current approach.

After #1080 is merged this is no longer necessary


listener: Fix issue with listener update not supporting an addition and removal of an additional_address in the same update.

Fixes #37373

/retest

@soulxu are you a good person to review? Thanks

Hi, @soulxu , could you take a first pass when you get some free time? Thanks.

@soulxu PTAL


How to rotate preview

My preview needs to be rotated by 90 degrees because I have my monitor in portrait mode.
Is this possible?
Thanks

Hello Brian,
I was wondering, if there is the possibility to add a rotation option, as asked above.
node-webcam.js has a rotation option but my programming skills are too insufficient to figure out how to implement such a rotation on the level of the preview.
It would be awesome, if you could help!
Best, Hendrik

Okay, the rotation feature has been added. If you would please try it out on the rotate-preview branch, I would appreciate it. It works for me, but I would like some testing before I merge it into the main branch.

Thanks for the implementation and sorry for the late reply/testing, Brian! I tested the rotation option from your newest version (1.0.4) and it works fine with my Raspi 3B+ and Raspi 2.1 camera.
One thing I have been wondering: Could you implement two rotation options: 1) for the countdown and 2) the photo preview? In my mirrors portrait orientation the countdown orientation was good, but only the photo preview was wrong by 90°.
Best, Hendrik

Sure, that should be relatively easy. I'll get on that.

I separated the rotation feature into two variables: rotateCountdown and rotatePreview. You can test them out by checking out the more-rotation branch. Give 'em a go and let me know if they work for you.

Hey Brian,
thanks for this adaptation. It tested it and it works great! :-)
Best, Hendrik


Bug: Forge Flatten includes compiling errata messages at top of file

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
No response
What command(s) is the bug in?
No response
Operating System
macOS (Apple Silicon)
Describe the bug
When using forge flatten we get for example:
Compiling 2 files with Solc 0.8.22
Solc 0.8.22 finished in 12.39ms

at the top of the flatten file.

thanks, fixed via #8313


fix: made the input height of the chat dynamic

Fixes Issue #421
**My PR closes #421
👨💻 Changes proposed(What did you do ?)

on the onChange handler provided in textarea in chat i adjusted the height of the text area depending upon the scroll height which solved the problem.

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

https://github.com/Dun-sin/Whisper/assets/91268931/50c06c50-720e-4cc3-b1f3-e23ea2cf4699

Changes done


Characters missing on webpage in Chrome in VirtualBox guest

Characters missing on webpage in Chrome in Windows 7 VirtualBox guest. I'm fairly certain support has ceased on Chrome in Windows 7, so I'm posting

to make the community aware that this is an issue others are experiencing
to seek a solution from the developers / community
To check if this could be an accessibility issue

Here are my screenshots. You can see I tested with both light and dark mode. I also tried going into chrome dev tools and changing all text to white in CSS, but it didn't work. Not sure what is overriding it. I also turned off all my Chrome extensions. No-go. Tried incognito mode. No go. Tried in Brave browser - no go.

I'm pretty late on this, but I added some back-up fonts which may have fixed this. Please let me know if you get a chance to verify this.


datatype error in msf demo

I want to run MSF Localization Module demo,but there is a datatype error as bellow:
System information

OS Platform and Distribution: Ubuntu16.04
Apollo installed from:source
Apollo version:3.0

Steps to reproduce the issue:

I set up my local computer according to this

Supporting materials:
root@in_dev_docker:/apollo/data/demo/data/bags# rosbag play 000.bag
[ INFO] [1538604179.924698562]: Opening 000.bag
Waiting 0.2 seconds after advertising topics... done.
Hit space to toggle paused, or 's' to step.
[ERROR] [1538604180.298549698]: Client [/localization] wants topic /apollo/sensor/gnss/corrected_imu to have datatype/md5sum [pb_msgs/CorrectedImu/81aef4a818ce273a8af85a440ccdb0f7], but our version has [pb_msgs/Imu/bdef0ba51869607ed95736d41e80c1f5]. Dropping connection.
[RUNNING]  Bag Time:<PHONE_NUMBER>.140799   Duration: 59.868109 / 59.870157
Done.

000.bag where is it from ? If you filtered from version 2.5, you will actually encounter this problem. So it’s best to record one yourself based on the 3.0 version.


WEBCAM and IPCAM interface with MATLAB

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
WEBCAM and IPCAM interface with MATLAB
Introduction:
A WEBCAM is connected to pc and can be used to make videos available to other users. The connection is normally through the USB. IPCAM involves the combination of high-end web functionality, pc and network interface to one network-ready product
In this tutorial we will look at how to install the WEBCAM and IPCAM hardware package for MATLAB which is necessary to access the hardware. We will also look at how to access laptop's inbuilt WEBCAM and external USB webcam with Matlab code to take snapshots, preview and to record a video clip.
The android application to convert mobile phone's camera as an IPCAM and look at the MATLAB code that can be used to make the mobile phone's camera as an IPCAM
Key takeaways:

how to access inbuilt WEBCAM.
how to make mobile phone's camera as an IPCAM
how to access live video stream using MATLAB

References:
Please list links to any published content/research that you intend to use to support/guide this article.
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

cool topic - approved @queenterbruce063


CREATE A ROUTE TO GET A SINGLE GOAL

◾This route should get a single goal by its ID.
Note: Follow up with the dev handling the controller.

You can assign this to me.

As an admin / mentor or intern  I should be able to to get a single goal of the program.
Live link : https://goals.zuri.chat/api/v1/goals/61334deabfba0a42d7f38e96


Issue-1488: Quick start guide to developing in OpenMetadata

Describe your changes :

Improve the developer documentation to include a quick start guide to start developing in OpenMetadata.

Type of change :

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[x] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@sureshms @harshach

Note, I will be owning the quick start guide to keep up with the changes in the system. Every Monday it will be refreshed for any new changes from the previous week.

Thanks @ksivam This is a great start. LGTM.
@shannonbradshaw can you review this as well

Reviewing now.

@ksivam thanks so much for this PR. It's a good start. I know how much effort this type of doc requires. Again, thank you! I've made several requested changes via the "suggestion" tool.

@ksivam thanks so much for this PR. It's a good start. I know how much effort this type of doc requires. Again, thank you! I've made several requested changes via the "suggestion" tool.

@shannonbradshaw thank you very much for the detailed review. Really appreciate your time to make this PR better.
Resolved and accepted all your suggestions. Please let me know if there are any more changes.

Thanks, @ksivam . @harshach I've approved this PR.

@harshach do we need to cherry pick this to 0.6.0 so that it appears in the docs? I'm not seeing it.
Maybe you're waiting to make it available in the 0.7.0 docs?


Fix (or replace) IPC incoming stream handlers

Fixed in 3a4bb00854839054b6d9baa96f233ae5c89c9615 (as far as I can tell; will re-open if problems emerge...)


I'm getting a nil error when building for iOS with Flutter.

Checklist

[X] I have looked into the Readme and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Description
When I tried to run a Flutter sample on iOS, I got the following error in Xcode:
Auth0/NSURL+Auth0.swift:9: Fatal error: Unexpectedly found nil while unwrapping an Optional value

I have already set up the Allowed Callback URLs and Apple Team ID in the Auth0 Settings. I have also added my associated domain in Xcode.
Do you know what I should do to resolve this?
Reproduction

Download the Flutter sample
Change to the bundle ID obtained from Apple Developer and register it in Allowed Callback URLs, etc.
Prepare the .env file
Register the associated domain
Execute flutter run

Additional context
The Okta sample login is successful, and 'It Works!' is displayed.
Platform
iOS
Platform version(s)
Flutter 3.19.6/iOS 18.0

Sorry, the format of my domain was incorrect. It's fixed now, so I'll close this🙇🏻♀


Add openjdk failed test case name information in tap files

Originally openjdk test DIAGNOSTIC level was set 'noDetails' to skip the test output due to parse failures of special characters, which results in no any failure testcases information in tap files. For jenkins jobs those failure information can be got from console output or junit plugin. We'd like to add the failed testcase name information in tap file, which can be clear and simple when regather the grinder's tap file by rerun failing tests using jdk_custom target.

@smlambert 's nice suggestion to include the line of jtreg output that summarized the total testcases in the target.
Test results: passed: 39; failed: 21
https://github.com/adoptium/TKG/pull/367#issuecomment-1314201578

Done, close it.


Unexpected Results from Transforms.pow(INDArray, INDArray)

I am running nd4j-native-platform (v 0.9.1 ) on a Windows 7,  64 bit architecture.
I am trying to perform an element-wise power operation between two NDArrays.
Here is my code:
double[] a1 = {1d,2d,3d,4d};
double[] a2 = {2d,2d,2d,2d};
INDArray arr1 = Nd4j.create(a1);
INDArray arr2 = Nd4j.create(a2);
INDArray arr3 = Transforms.pow(arr1, arr2);
System.out.println("arr3 is "+arr3);
Here is what I get:
arr3 is [1.00,  0.50,  0.33,  0.25]
I would expect to get the following instead:
arr3 is [1.00,  4.00, 9.00,  16.00]
Am I doing something wrong ? or is this a bug ??

I've got the same thing

Yes, that's a bug.
pow isn't announced in pairwise transforms, operation used by this signature is ReverseDivide.

Thanks for confirming that this is a bug. Until a fix is available - Is there a workaround available ? I have a need to use an element-wise power operation in my app.

I can provide fast fix, but obviously it'll be only available at snapshots.
Is that suitable for you?

Thanks so much - A fix in the snapshot is good - That will get me started on a basic integration. Very much appreciated.

Implemented/fixed and merged: https://github.com/deeplearning4j/nd4j/pull/2098


position the stats label properly for negated gate

It's been an issue for the negated gate imported from flowJo. Now that with the support for it added to openCyto RGLab/openCyto#169, it is no longer a rare case and thus requires the fix.

fs <- getData(gs[1], "Live")
autoplot(fs, "CD3", "CD19") + geom_gate(gate_2d) + geom_stats()

Now, simply pass negated flag to geom_stats()
autoplot(fs, "CD3", "CD19") + geom_gate(gate_2d) + geom_stats(negated = T)

When plot the gate from GatingSet , the negated flag is parsed from gs and set automatically
autoplot(gs, "CD3-" ,strip.text = "gate") + guides(fill=FALSE)


Use older disjoint timer in webgl2 if the newer one fails #606194

This will only have an effect on your machine where FF supports the old extension - not for me, @markschlosseratbentley, or Majd, for whom FF supports neither extension - right @hnn0003?

@pmconne Yes, I'm assuming this will only allow affect my machine (and any other rare cases of a browser supporting webgl2 but only the older timer extension), and that it won't affect you, @markschlosseratbentley, or Majd (as all three of you have webglreports of FF not supporting either the old or new timer extension--though, it'd be interesting to just double check that with one of you running something really short, just in case webglreport happens to be not reporting it for some reason, idk). I have checked running with this on my machine, and it does utilize that older extension correctly and gives me GPU data when running in FF with this fix, fyi.


NO-JIRA Update broker connection AMQP SSL example

Looks good other than the suggestion, push without or with (squash it if using)

@gemmellr unused server names removed from doc and certs

@brusdev looks good, just push it.


Touch & Scrolling

I encounter an issue when adding drag to chart which fill the whole mobile screen.
I have a radar chart that fills 3/4 of the mobile screen, and the labels filling the other 1/4.
I'm expecting the chart not to consume the drag event if the user is scrolling on the screen (away from the points but within the chart canvas).
Currently, if the user wants to scroll down, he cannot. When he drags down from either the left or right of the points (so basically touching the labels of the chart aside from the points), nothing happens.
My temporary workaround is to make the chart short enough for him to realize that scrolling from the sides won't work and he'll have to scroll from below the chart. (kinda sucks from a UX POV).
Is there any workaround for this issue?
Thanks for your contribution and great plugin.

Hey @usernotnull,
that's a tough one indeed. An (unresolved) discussion around this also evolved as part of #20 (specifically https://github.com/chrispahm/chartjs-plugin-dragdata/issues/20#issuecomment-801571515).
On touch devices, the Chart.js canvas element receives the inline style touch-action: none;, prohibiting any scroll behaviour. I made a small codepen to illustrate the issue:
https://codepen.io/chrispahm-the-animator/full/OJOMNxm
You can manually overwrite the touch-action: none; e.g. using myChart.canvas.style.touchAction = "pan-y"; (see https://github.com/chartjs/chartjs-plugin-zoom/issues/336). Now both scrolling and data dragging is enabled, but you'll notice that while dragging a point vertically, the site is also scrolled a little, IMHO not a good UX.
A compromise could be to set the touch-action to pan-y. but also disable the drag-data plugin as a default. If a user intents to change the charts values, there could be "Edit data" button that when clicked, both enables the drag-data plugin, but also sets the touch-action back to none.
https://codepen.io/chrispahm-the-animator/pen/WNXrGvb

@chrispahm thanks for your workaround idea.
I'll close this.


feat(app): add wifi disconnect

overview

Refactors select network functionality on robot settings screen. Handles disconnect, connect, and joining of wifi networks.
changelog

review requests

Closing in favor of #5234 and #5280. Those PRs were both created from this branch, which is now safe to delete


Feature: Typescript fallback intellisense

I find myself torn between using Typescript and JavaScript.  The JavaScript tooling has decent tooling that Just Works.  Typescript has really high quality tooling that can be a pain to use.
It seems to me it would be possible to bring the good qualities of both together in nodejstools.  Check out this example:

No intellisense of course, because it is any type.  But wait, if I switch to my compiled JavaScript tab I get something different...

The idea is to try to automatically fallback to the JavaScript Intellisense when TypeScript is coming up blank.  I suspect it could be a very popular feature, especially if it allows zero config usage of Javascript NPM modules, by doing something like (<any>require)("insert-obscure-untyped-javascript-package-here").

Typescript has really high quality tooling that can be a pain to use

I'm sure the TypeScript team would be keen to know the pain points of using their tooling.

Going forward, we plan to move both our JavaScript and Typescript Intellisense to use the Typescript language service. This will result in a much more consistent experience across JS and TS for NTVS, and a more consistent Intellisesnse experience with VS Code and other tools as well. Additionally, the Typescript language service is being actively developed and using it will hopefully allow NTVS to support new features and pick up enhancement much more quickly.
Check of the NTVS 1.2 Alpha for a preview of the new Typescript language service based Intellisense. Please report any issues you encounter and let us know if you have any suggestions.
Thanks.


Failed to Build Deep-jdbc

I was trying to build deep-spark using the command  mvn clean install -DskipTests inside deep-parernt directory and at the end i got an error stated that
[INFO] deep parent ........................................ SUCCESS [ 22.328 s]
[INFO] deep commons ....................................... SUCCESS [ 33.724 s]
[INFO] deep core .......................................... SUCCESS [ 25.067 s]
[INFO] deep cassandra ..................................... SUCCESS [ 29.947 s]
[INFO] deep mongodb ....................................... SUCCESS [ 22.710 s]
[INFO] deep elasticsearch ................................. SUCCESS [ 23.845 s]
[INFO] deep aerospike ..................................... SUCCESS [ 23.377 s]
[INFO] deep jdbc .......................................... FAILURE [  0.147 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 03:01 min
[INFO] Finished at: 2015-05-29T16:22:24+05:30
[INFO] Final Memory: 77M/718M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project deep-jdbc: Could not resolve dependencies for project com.stratio.deep:deep-jdbc:jar:0.8.0-SNAPSHOT: Failure to find com.oracle:ojdbc7:jar:<IP_ADDRESS> in http://m2.neo4j.org/content/repositories/releases was cached in the local repository, resolution will not be reattempted until the update interval of Neo4j releases has elapsed or updates are forced -> [Help 1]
Can you please provide me the proper steps to build this project.
Thanks,

To compile Deep is necessary to have ojdbc7 driver in your local repositoty because this driver is not published in any public maven repositories.
We are going to add the way to install the oracle driver into our documentation.   Meanwhile we send you the steps in advance.
In order to compile the deep-jdbc module is necessary to add the Oracle ojdbc driver into your local repository. You can download it from the URL: http://www.oracle.com/technetwork/database/features/jdbc/default-2280470.html. When you are on the web you must click in "Accept License Agreement" and later downlad ojdbc7.jar library. You need a free oracle account to download the official driver.
To install the ojdbc driver in your local repository you must execute the command below:

mvn install:install-file -Dfile= -DgroupId=com.oracle -DartifactId=ojdbc7  -Dversion=<IP_ADDRESS> -Dpackaging=jar

After that you can compile Deep executing the following steps:

cd deep-parent

mvn clean install

If you are agree with the explanation please close the issue.

Hi Josegom,
I used mvn install:install-file -Dfile=ojdbc7.jar -DgroupId=com.oracle -DartifactId=ojdbc7 -Dversion=<IP_ADDRESS> -Dpackaging=jar command and it yields Successful Build
Thank you!


set.union() fails inside withMutations

you can check this in a console
> Immutable.Set().withMutations(function(set) {
    set.union([ 'a' ]);
    set.add('b');
}).toString()
"Set { "b" }"

it works outside of a withMutations though.
> Immutable.Set().union(['a']).toString()
"Set { "a" }"

Thanks for the report!


Allow reporting of Promotion Criteria succes via an HTTP API

Related to #3 somewhat
For integration test runs required to satisfy Promotion Criteria, a test runner will need to be able to hit a RESTful API to report the status of the integration test.
The API should require:

environment
artifact version
test suite name
test suite result
more info URL
???

At least in 0.1 we need to define this API, we don't necessarily need to implement it though.

@GiriDandu I think you may have partially started defining this with the Promotion Result stuff we discussed around  #54


removes units from labels, definitions, axioms mentioned in #149

#149

Updated in last commit with new term


Add quick test attribute

Description

Adding 'quick-test' attribute to list of smoketests and adding 'vpc', 'templates', 'deploy-vm', 'roles', 'service-offerings', 'storage', 'routers', 'networks', 'usage', 'accounts' tags
quick-test label aims to provide quick feedback of recent changes to the developer, instead of waiting for the whole smoketest to execute(24h+). This execution lasts about 1.5h at Trillian environments.
The categories aim to offer more efficient way to slice through tests in both smoke and component tests. Let's say as a developer you've made a change in the accounts. You can simply trigger marvin tests with tag=accounts pointing all the tests for example:
nosetests --with-marvin --marvin-config=[config] --hypervisor=xenserver -a tags=accounts /tests/integration/*/*.*

http://cwiki.apache.org/confluence/display/CLOUDSTACK/Test+Categories+and+Quick-test+in+Marvin

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[x] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2351

@blueorangutan test

@DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3140)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 36202 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4457-t3140-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_supported_versions.py
Smoke tests completed. 83 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_07_deploy_kubernetes_ha_cluster
Failure
3621.32
test_kubernetes_clusters.py

test_08_deploy_and_upgrade_kubernetes_ha_cluster
Failure
0.11
test_kubernetes_clusters.py

test_09_delete_kubernetes_ha_cluster
Failure
0.09
test_kubernetes_clusters.py

ContextSuite context=TestKubernetesCluster>:teardown
Error
103.70
test_kubernetes_clusters.py

ContextSuite context=TestKubernetesSupportedVersion>:setup
Error
0.00
test_kubernetes_supported_versions.py

@PaulAngus @borisstoyanov do you want this in? cc @rhtyd

@DaanHoogland many conflicts, but LGTM - pl do the honours if you think it's useful
(personally I don't who will run only a sub-set of tests)

@rhtyd this will need rework, changing to draft. we might want to run this for low risk changes, but a am also thinking about a more flexible test set system.

Hi @DaanHoogland are you considering this PR for 4.17? In that case, can you please solve the conflicts?

one in 5 lines in here is a conflict. maybe revisit the idea later.


D3 Language Chart Overwriting

If language 'slices' are too small, part of the name is still being overwritten.

@MaryCBaylis This seems like something you could fix!


CenterNet-based and Yolov3-based deepsort

I tried comparing the result between Yolov3-based and CenterNet-based(DLA34) deepsort, but MOTA and MOTP of CenterNet-based  deepsort are lower than Yolov3-based's in MOT16.
Does anyone know why？
And can you show your test result in MOT16 as a reference? pls @ZQPei

I tried comparing the result between Yolov3-based and CenterNet-based(DLA34) deepsort, but MOTA and MOTP of CenterNet-based deepsort are lower than Yolov3-based's in MOT16.
Does anyone know why？
And can you show your test result in MOT16 as a reference? pls @ZQPei

Hi, can you show me your test result in MOT 16, pls


Listener for closing the Intro

Is there a way to check when the Intro is closed? (Skip or Accept button at the last slide)
Thanks

Yes, see the onActivityResult sample here.

Thanks a lot!


JWM Does Not Obey Group "y:n" Settings

Although I specify the "y:n" option in a group definition, JWM does not initially display the window in the proper position. The values for height, width, and x (and all others) are, however, as defined.
<Group>
            <Name>Navigator</Name>
            <Option>desktop:7</Option>
            <Option>fixed</Option>
            <Option>icon:firefox.png</Option>
            <Option>nofocus</Option>
            <Option>nolist</Option>
            <Option>nomove</Option>
            <Option>noclose</Option>
            <Option>height:935</Option>
            <Option>width:1625</Option>
            <Option>x:147</Option>
            <Option>y:47</Option>
</Group>

Upon display, xwininfo gives these results:
  Absolute upper-left X:  148
  Absolute upper-left Y:  80
  Relative upper-left X:  1
  Relative upper-left Y:  24
  Width: 1625
  Height: 935
  Depth: 24
  Visual: 0x83
  Visual Class: TrueColor
  Border width: 0
  Class: InputOutput
  Colormap: 0x2800002 (installed)
  Bit Gravity State: NorthWestGravity
  Window Gravity State: NorthWestGravity
  Backing Store State: NotUseful
  Save Under State: no
  Map State: IsViewable
  Override Redirect State: no
  Corners:  +148+80  -147+80  -147-65  +148-65
  -geometry 1625x935+147+56

I then manually run an aliased wmctrl command -- 'wmctrl -r Firefox -e 0,147,47,1625,935' -- followed by xwininfo, and the results change:
  Absolute upper-left X:  148
  Absolute upper-left Y:  71
  Relative upper-left X:  1
  Relative upper-left Y:  24
  Width: 1625
  Height: 935
  Depth: 24
  Visual: 0x83
  Visual Class: TrueColor
  Border width: 0
  Class: InputOutput
  Colormap: 0x2800002 (installed)
  Bit Gravity State: NorthWestGravity
  Window Gravity State: NorthWestGravity
  Backing Store State: NotUseful
  Save Under State: no
  Map State: IsViewable
  Override Redirect State: no
  Corners:  +148+71  -147+71  -147-74  +148-74
  -geometry 1625x935+147+47

FWIW, I have two trays, positioned at the top and bottom of the screen -- the top tray height is "34". In WindowStyle, the height as "24".
This behavior holds-true for the other apps I run: Chromium, Thunderbird, and Pidgin.

Bump


Unexpected behavior on multiple parallel update requests

I tested multiple async request to api endpoint for update existing document with different fields. (One request updates one field)
If entry created with empty fields, in mongodb document this fields doesnt exists, that allows successfully update all empty fields with multiple parallel api requests, but if field have value and after its deletion fields keep in mongodb with empty value { "field": "" }
This kind of fields dont updates with multiple parallel request, only one random field could updated
Example request:
const f = (body) => fetch("http://localhost:4000/api/test/6372638638bead8eb92b0018", {
method: "PATCH",
headers: {
"Content-Type": "application/json",
},
body: JSON.stringify(body)
});
Promise.all([
f({ qwe: "rterte" }),
f({ wer: "ert" }),
f({ ert: "fdgd" }),
f({ rty: "ok" }),
f({ asd: "fhfh" }),
f({ sdf: "rt" }),
f({ zxc: "dfg" }),
f({ xyz: "sdf" }),
f({ vbn: "odfdfgk" }),
f({ foo: "dfgdgd" }),
]).then((r) => r.map(async(i) => console.log(await i.json())))

Hey @383bd03d — I see what you're going for here but I also understand why this does not work. Right now the full existing document is merged into the incoming data within each update operation. Meaning——your parallel requests are overlapping one another and could be writing as another saves.
However, you could access the raw Mongoose model for your collection by payload.collections[collectionSlug].Model and open up a custom endpoint that takes a property to set, and value to set, in order to update only specific properties in parallel.
Outside of that, this would need to be a new feature that Payload provides. Could be possible at some point so I'm going to convert this to a discussion and tag it as a feature request! How's that?


Requester lens doesn't work after redirect from successful job form submit

Originally created by Tom Smyth at 2018-03-21 09:49, migrated from redmine-#8178

Status changed from New to Closed

By Tom Smyth at 2018-03-26 22:47

Version set to 6.0

By Tom Smyth at 2018-04-29 11:24


dailymotion videos cannot download

When trying to download from http://www.dailymotion.com/video/x31ami9
I getting this error:
ERROR: Unable to download webpage: <urlopen error [Errno 6] _ssl.c:504: TLS/SSL connection has been closed> (caused by URLError(SSLError(6, '_ssl.c:504: TLS/SSL connection has been closed'),))

Post the full output of youtube-dl when run with --verbose option.

I can't watch it in browser. Here is the full output...
bpi ~ :( # youtube-dl --verbose http://www.dailymotion.com/video/x31ami9
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'http://www.dailymotion.com/video/x31ami9']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.08.16
[debug] Python version 2.7.3 - Linux-3.4.90-armv7l-with-debian-7.8
[debug] exe versions: avconv 9.14-6, avprobe 9.14-6, ffmpeg 0.8.17-6, rtmpdump 2.4
[debug] Proxy map: {}
[dailymotion] x31ami9: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 326, in _request_webpage
return self._downloader.urlopen(url_or_request)
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 1860, in urlopen
return self._opener.open(req, timeout=self._socket_timeout)
File "/usr/lib/python2.7/urllib2.py", line 407, in open
response = meth(req, response)
File "/usr/lib/python2.7/urllib2.py", line 520, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python2.7/urllib2.py", line 445, in error
return self._call_chain(*args)
File "/usr/lib/python2.7/urllib2.py", line 379, in _call_chain
result = func(*args)
File "/usr/lib/python2.7/urllib2.py", line 528, in http_error_default
raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

I can't watch it in browser

Therefore you can't download it.


fuzz-get: Don't send unmodified requests

Skip strings which are identical to the original input.

👍 Thanks @MikeRalphson !


Remove third party protos

Closes: #XXX
Context and purpose of the change
Third party protos are no longer needed with SDK 47 [source]

very yes


Gathering TPM EK certificate chain sizes

We've hit a problem of not properly working Fobnail firmware when preparing for demo accompanying new release, which was put on hold because of that. The problem comes down to not enough memory left for our tasks, and it presents itself when Attester sends EK certificate chain, or in the following step if this one doesn't corrupt memory enough to break immediately. For reasons we're still investigating, code tries to allocate much more memory than the size of data sent, sometimes by an order of magnitude more than what we expected.
There are few possible solutions that we're considering. To have higher confidence in the outcome, we would like to gather as much info about certificate chain sizes we have to be prepared for as possible. In order to help with obtaining that data, a script was prepared. This is an example output, taken from PC Engines apu3 platform with SLB 9665TT2.0 TPM:
root@debian:~# ./test_tpm_ek_chain.sh
Certificate 0 size: 1177 bytes
Certificate 1 size: 1463 bytes
Certificate 2 size: 1455 bytes
Certificate is self-signed, assuming it is root

Chain length: 3
Total chain size: 4095 bytes
Size of the biggest certificate: 1463 bytes

Verifying whole chain:
stdin: OK

The script assumes that tpm2-tools (accessing the TPM), openssl (parsing, converting and verification of certificates) and wget (downloading CA certificates) are installed. As you can see, there is no personal data that could be used to identify your platform.
I would like to ask anyone interested in helping this project to execute this script on your machines (only if they have TPM2.0, of course), and share your results in the comments. Platform and TPM model is nice to have for statistics and to check if sizes vary inside one family of TPMs, but if you feel that it can compromise your security, reports without that data are still welcome. If you decide to include that info but don't know what model of TPM is used, semi-useful vendor info can be obtained with:
sudo tpm2_getcap properties-fixed 2>/dev/null | grep TPM2_PT_MANUFACTURER -A14

NovaCustom NV4X ADL
Certificate 0 size: 1171 bytes
Certificate 1 size: 1449 bytes
Certificate 2 size: 1455 bytes
Certificate is self-signed, assuming it is root

Chain length: 3
Total chain size: 4075 bytes
Size of the biggest certificate: 1455 bytes

Verifying whole chain:
stdin: OK

RPi with Infineon Optiga 6970

Ryzen 5 2600, fTPM
Certificate 0 size: 727 bytes
wget: missing URL
Usage: wget [OPTION]... [URL]...

Try `wget --help' for more options.
Can't download CA certificate from ''

Ryzen 5 2600, fTPM
Certificate 0 size: 727 bytes
wget: missing URL
Usage: wget [OPTION]... [URL]...

Try `wget --help' for more options.
Can't download CA certificate from ''

@macpijan this looks suspicious, is it possible that you've run previous version of tpm_manufacture.sh on this platform? This can be checked with tpm2_nvread -C o 0x01C00002 | openssl x509 -inform der -text -noout, if it was created by the mentioned script the certificate would have Fobnail specified as an organization.

this looks suspicious, is it possible that you've run previous version of tpm_manufacture.sh on this platform?

Quite likely.

Mine TPM lacks EK certificate.

# ./test_tpm_ek_chain.sh
Certificate 0 size: 1184 bytes
Certificate 1 size: 1463 bytes
Certificate 2 size: 1455 bytes
Certificate is self-signed, assuming it is root

Chain length: 3
Total chain size: 4102 bytes
Size of the biggest certificate: 1463 bytes

Verifying whole chain:
stdin: OK


Flaky error in test_pageserver_restart in CI

Found in #499
In pageserver logs:
Aug 31 09:10:21.514 ERRO get_page_at_lsn: no base img found for 1663/13974/2662.0 at blk 0 at LSN 0/2763E78, location: pageserver::page_service, pageserver/src/page_service.rs:313

In pg logs:
2021-08-31 09:10:21.510 GMT [2033] LOG:  [ZENITH_SMGR] libpqpagestore: connected to 'host=localhost port=64000 password='
2021-08-31 09:10:21.514 GMT [2033] FATAL:  could not read block 0 in rel 1663/13974/2662.0 from page server at lsn 0/02763E78

Error happened only in debug, release test succeeded

@patins noted that last night, too. It's happened a few times in the CI on 'main'. He suspected commit 7c7e89e as the culprit, because it didn't happen before that, although it's hard to tell because it doesn't happen very often.

My first thought was that we can fix this in InMemoryLayer::put_page_version: If the segment is being extended by more than one page, implicitly fill the gap with all-zeros pages. That's analogous to what PostgreSQL does with local filesystem.
However, it's not that simple if the gap block is at a segment boundary. For example, if a table is extended from 1279 to 1281, by writing to block no 1280. Block no 1279 should be filled with zeros, but that's on segment 0, while block 1280 is on segment 1. InMemoryLayer::put_page_version will be called on segment 1, and we cannot change segment 0 there. We will need to coordinate this from LayeredTimeline::put_wal_record somehow.


Recipe file structure needs improvements

There are things we should improve in the current recipe file structure. The current demo looks like this:
fixtures:
  environment:
    PLANET: Earth

  git_url: https://github.com/RedHatQE/newa.git
  git_ref: main
  tmt_path: demodata
  plan: /plan1

dimensions:
  cities:
    - environment:
        CITY: Brno
    - environment:
        CITY: Boston
      when: CONTEXT.color is match("blue")
  colors:
    - context:
        color: red
      when: ENVIRONMENT.CITY is not match("Brno")
    - context:
        color: blue

Suggestions for improvements:

Q1: Keep testingfarm, tmt, reportportal under fixtures or have it standalone?
Q2: Use even more levels?

Now discuss those individual questions in detail:
Q1: Keep testingfarm, tmt, reportportal under fixtures or have it standalone?
Having it under fixtures would simplify operating with this settings within dimensions as well. But is it necessary?
Having it separatele and utilize Jinja template rendering for parametrization might be sufficient and be more user friendly.
Example:
dimensions:
  architecture:
    - context:
        arch: x86_64
    - context:
        arch: s390x

testingfarm:
  cli_args: "{{ --arch CONTEXT.arch }}"
reportportal:
  launch_attributes:
    arch: "{{ CONTEXT.arch }}"

However, with may do the same under fixtures as well and a user may use it e.g. to extend launch attributes.
Example:
fixtures:
  reportportal:
    launch_attributes:
      arch: "{{ CONTEXT.arch }}"

dimensions:
  architecture:
    - context:
        arch: x86_64
    - context:
        arch: s390x
  fips:
    - context:
        fips: yes
      reportportal:
        launch_attributes:
          fips: yes
    - context:
        fips: no

Q2: Use even more levels?
The advantage might be better control in case we are going to implement support e.g. for TF direct requests through API. Also, we would avoid repeating prefixes like "launch_" or "cli_" etc.
Example:
testingfarm:
  cli:
    args:
reportportal:
  launch:
    description:
    name:
    attributes:

Another PR https://github.com/RedHatQE/newa/pull/39

All done.


firefox-bin-developer-edition-launcher: init at 1.0.0

This package add a downloader and launcher for Firefox Developer Edition, formerly known as Firefox Aurora, which is the next version of Firefox Beta.
Example:
current Firefox Release: Firefox 40
Firefox Beta: Firefox 41
Firefox Developer Edition: Firefox 42
https://wiki.mozilla.org/Release_Management/Release_Process
What is this?
This package installs a small bash script:

downloads and installs the latest Firefox Developer Edition into the specified directory (the default is ~/.firefox_aurora),
patchelf it, and
launches it with appropriate environment variables.

Why?
From Firefox 44, add-on developers must use Developer Edition or Nightly to develop add-ons.
https://wiki.mozilla.org/Addons/Extension_Signing
Why not install the binary into /nix/store?
Firefox Developer Edition is released every morning. The Mozilla overrides the archive file daily, so that we cannot fix a hash.
https://wiki.mozilla.org/Release_Management/Release_Process

Actually, there are stable URL's, of the form:
http://archive.mozilla.org/pub/firefox/nightly/2015/09/2015-09-22-00-40-45-mozilla-aurora/
(which today has identical contents to http://archive.mozilla.org/pub/firefox/nightly/latest-mozilla-aurora/). IIRC they keep them around for bisecting purposes.

I'm :-1: on this because:

Nixpkgs is a collection of packages, not scripts to download packages via non-Nix mechanisms.
Nixpkgs should contain stable releases, not daily unstable releases.

Which is not to say that this script is not useful, but it shouldn't be part of Nixpkgs.

@edolstra We have Chromium Dev, is that going away now?

I'm not really in favor of chromium-dev either, but at least that installs a package, not a script to install a package.

OK, I see. My objective is developping add-ons, not using bleeding edge of Firefox. I will make a package for the last build of Developer Edition of older version (for example, 40 or 41 rather than 42). I'll be back with version 44 or 45 when the stable Firefox 44 is released. Thanks.

Mozilla is planning to release “unbranded versions of Firefox” for developers. I will package it when it is released.
https://wiki.mozilla.org/Add-ons/Extension_Signing


GH-44273: [C++][Decimal] Use 0E+1 not 0.E+1 for broader compatibility

Rationale for this change
Most environments such as Python, Node.js, PostgreSQL and MySQL accepts 0.E+1 but some environments such as Ruby don't accept 0.E+1. More environments accept 0.0E+1 or 0E+1 than 0.E+1.
What changes are included in this PR?
Use 0E+1 not 0.E+1 for broader compatibility.
Are these changes tested?
Yes.
Are there any user-facing changes?
Yes.

GitHub Issue: #44273

There are a lot of failing CI builds.

Oh, sorry. I needed to update more expected values...

I'll merge this in a few days if nobody objects this.