uglifyJS option and uglifyJS-X compatibility

I gave it a try with uglify-es
app.use(minify({uglifyJS: require('uglify-es')})); //uglifyJS with ES6 support

but it does not work. There are uglifyJS2 and uglifyJS3 out there, which are stated to be mutually incompatible either.
What options do we really have with uglifyJS option? Can we document it?

@zevero
Right now express-minify is only compatible with<EMAIL_ADDRESS>It will need to be modified which will bump out compatibility with v2.x from the mix. Major semver change over on that project. I've pinged breeswish last week about the changes.

Sorry for the delay. Working on graduation project these months until mid June and after that I would take a look at this issue. PR are welcome though.

I can do that tomorrow when I get back to dev station but keep in mind that 2.x will no longer work.

@Martii Seems that we need something like adapters?

@breeswish
Well the error on failures is no longer thrown so will need to either modify a test, or two, ... or throw an error manually in a trap like we did on OUJS. The instance that was added in #42 doesn't quite mention major semver compatibility which wasn't expected of course... but happened none-the-less.
Adapters might be a bit much for me as I don't know exactly how you would want that. I too will be a bit busy in the following weeks but I can at least submit a patch if you would like to migrate to v3.x of their package. 2.x is deprecated of UglifyJS2 (npm package of uglify-js). v3.x has been mostly stable in our experimental minification.

@breeswish
Current tests with mocha seem to be incompatible with 2.x and 3.x... which is what I figured. As I'm not real familiar with mocha, other than running the tests, I am unfortunately unable to take this on at this time due to other duties. I've tested the minifier.js locally with some changes and it works in all of our use cases and logically it should be the right step for your edit of inclusion of 2.x... but I don't want to miss something that we don't use so I'll defer to your expertise and perhaps the pings (GH mentions) I sent you approximately last week may give you a head start. :)

I'm wondering if there any cases that UglifyJS 2 work previously but fail in UglifyJS 3? If not, I guess it might be better for this module to upgrade dependency from 2 to 3.

@breeswish
We're going to go hunting for an alternative to uglify-js since the project has gone downhill with maintaining. If we can find something a little more reliable that will be good to avoid their breaking changes ... both documented and mostly undocumented.
Appreciate your efforts and patience.

Supported UglifyJs-3 and Uglify-Es in 1.0.0.


Expand description of Singleton

See comment https://github.com/jupeter/clean-code-php/issues/61#issuecomment-327727596
Description from https://stackoverflow.com/a/138012/2455621
This is not the best description, but it will come off for a start.

There is also very good thoughts by Misko Hevery (http://misko.hevery.com/about/) about the root of problem http://misko.hevery.com/2008/08/25/root-cause-of-singletons/, and he is well known in Angular community and because of his works about writing good tests and testable code (not to mention that he is Agile Coach at Google and previously worked at Adobe, Sun Microsystems, Intel, and Xerox as developer). Also, a load of information and healthy discussion in comments.

@metamaker Thanks. I added link.

I think it would make it a bit easier to read, if the most important parts would be highlighted.
Could you pick like 4-5 such parts and bold them?

I highlighted the important parts.

Thanks, let's add it then


configure caching for Netlify-served website

document in README as well, even if not possible to configure this

For now no need for custom caching.
https://stackoverflow.com/questions/52308658/netlify-headers-cache-control-for-static-assets
All good. Let's not start overwriting with custom headers. If we do want to do so it could be done as follows:
[[headers]]
  for = "*.js" # js files should be set this way
  [headers.values]
    Cache-Control = "public, max-age=604800"
[[headers]]
  for = "*.css" # css files too
  [headers.values]
    Cache-Control = "public, max-age=604800"


Xtensa: dup_table duplicates the pinned PTEVADDR entry

The page table duplication routine in the current Xtensa implementation dup_table only skips copying and allocating L2 page tables when it detects a PTE as "illegal" (attributes 12 or 14), however it should also skip this when the index is the same as the one for the kernel ptables pinned virtual address, ie the entry that maps CONFIG_XTENSA_MMU_PTEVADDR.
Doing this copy wastes an entire 4k of memory and could potentially be a security issue, as it's copying the L1 table as an L2 page table.
Expected behavior
The code should entirely skip the entry that matches CONFIG_XTENSA_MMU_PTEVADDR and mark it as unmapped when duplicating the table.
Impact
Wasted memory, potential security issue.
Additional context
Xtensa platform with MMU and userspace enabled

Not clear what is the problem here, we indeed map the physical memory for all page tables, but we don't map the virtual address for the page table [CONFIG_XTENSA_MMU_PTEVADDR + 4MB], we just pin the l1 page for this area and the auto refill does the rest.

In the dup_tables routine it believes this entry you place at XTENSA_MMU_L1_POS(CONFIG_XTENSA_MMU_PTEVADDR) during kernel startup is a valid L1 page table entry to a different L2 page table, when in reality it's just itself.
When duplicating it, this causes memory to be mapped at CONFIG_XTENSA_MMU_PTEVADDR for other memory domains, when in reality we only want the mapping at the 4MB * user_asid offset

XTENSA_MMU_L1_POS

I think I got what you mean, it is confuse because we don't map the page itself until it is used, so if I understood correctly, we can do:
		if (is_pte_illegal(source_table[i]) || (i == XTENSA_MMU_L1_POS(XTENSA_MMU_PTEVADDR))) {
			dst_table[i] = XTENSA_MMU_PTE_ILLEGAL;
			continue;
		}

I can't see a security issue because it is mapped as using kernel ring and the whole page table (physical memory) is already mapped, but indeed we can save memory doing it :)

Looks perfect to me, a whole 4k saved per memory domain.
Thanks!


Fixed Typo as per #152

Fixes #152

btw there's another one a few lines higher.. thanks :)

btw there's another one a few lines higher.. thanks :)

Thanks @setharnold! Totally missed that! Raised PR #154  for that.


build: remove website version bump on tag

I can't see a strong need at this time for us to keep the website version in-sync with the gitify release version.
see https://github.com/gitify-app/website/pull/144

This workflow continues to be difficult, failing for the last two releases (again)

As discussed on the website PR, this LGTM :)


How to implment access token or other form of authorization

Is it possible to implement access tokens with this library? I'm basically looking for a way to implement authorization.

I was trying to do that using headers to pass the API Key:
client = Client(host='https://localhost:11434',headers=["x-token=xxxxxxxxxxxxxxxxxxxxxxxxxxxxx"])
But after that I get a different error:
File "C:\Python312\Lib\site-packages\ollama_client.py", line 67, in init
super().init(httpx.Client, host, **kwargs)
File "C:\Python312\Lib\site-packages\ollama_client.py", line 52, in init
headers['Content-Type'] = 'application/json'
~~~~~~~^^^^^^^^^^^^^^^^
TypeError: list indices must be integers or slices, not str


Compiling IJT Nodeset to C# not working properly, because of casting issues on a variable

What i've done: Compiled https://github.com/OPCFoundation/UA-Nodeset/tree/latest/IJT/Base with the ModelCompiler.
 Doesnt matter if T is a derived type of the target T. The Whole type is always different.
Example would be:
BaseVariableState<Parent> parent = new();

//This is not allowed in C#
BaseVariableState<DerivedFromParent> castedType = (BaseVariableState<DerivedFromParent>)parent

Am I doing something wrong here?

I fixed the generator to suppress the generation of the property.
Pushed to git but did not rebuild docker image.

Can you tell me the commit which contains the fix? I can not find it

Yes, it when into the member only repo for now:
https://github.com/OPCF-Members/UA-ModelCompiler/commit/6fe4dad609f47ce01b002b1c57cf795a300fa242
Do you have access?

Maybe. The customer I am working for is a member. I requested access there

I guess we must wait until new year. All customers employees that can manage the access are on holiday...
Is it possible for you to give me temporary access to this repo only?

I can confirm, that the properties are not generated anymore. Thank's!
Btw. i have one last question: The DataTypes from *.DataTypes.cs file can be loaded into the OPC UA Client. What can i exaclty do with all the classes in the *.Classes.cs file?

*.Classes.cs is used only by Servers.


Remove unnecessary subfolder wrappers for integration tests

Since https://github.com/com-lihaoyi/mill/pull/3347 made all test suites run with pwd in the test's .dest folder by default, we are already guaranteed that different concurrent test suites will not collide, so we do not need the out/blah/blah segments to avoid conflicts

Does that mean, we can't no longer inspect the out files of multiple (failed) tests?

Yes. But previously they were all in out/<random-number> folders, so trying to find the output of a particular test was impossible anyway from my experience. We can definitely try to add more logic to assign each test a unique folder

I never experienced those random number paths, this was probably after your test suite refactoring. I though, we still had long sub-paths resembling the full test name.

yeah the test suite refactor moved a bunch of hardcoded paths to os.temp. Made the code a lot cleaner, but at a cost of filesystem tidyness. Can probably add them back depending on sourcecode.FullName and utest.framework.TestPath to get the tidyness without the boilerplate

That sounds reasonable. Both generic sources (FullName as well as TestPath) already provide meaningful names. Paired with a random number, there should only one or a low count of test output directories to inspect, when in need. In LambdaTest, my Java library to write functional tests on top of JUnit/TestNG, I have a autoclosable test path provider, that can clean up automatically on success, but keeps the temp content when failed.


Guarantee full coverage on each OS

There are some lines of code (when doing os.remove) that can only happen on Windows, or only on Linux.
One idea to ensure this is to use this feature of coverage
to exclude certain lines, by defining a custom # pragma: no-cover-on-windows and a # pragma: no-cover-on-unix, and then define the exclude_lines in the coveragerc in the GitHub actions for each OS.
Marked as blocking because I'd like to keep the requirement of full coverage.

This was implemented in fded4a79870fbd3cbe3c3b2a5277b73582c5f570 but then reverted. It's not needed to change the excluded lines (actually, they are then excluded from the final coverage report). Instead, we leave codecov sum the coverages of each workflow step


UX - A lot of people have no idea they can share playlists.

Need to make this more obvious somehow.

Is it an idea to set a playlist to "public" and that you can find new playlists by browsing https://share.streamus.com/
First of all this will attract attention to sharing playlists and it can be a wonderful way to discover new music. Creating awesome and large playlists can be quite a hassle and when collaborating and sharing this can be so much easier!

I think simply "Copy URL" > "Get Share URL" or just "Share URL" could fix this.
And, personally I like that you can do this without needing proper user to user interactions/messaging. One of the main things I love about Streamus is the lightweight feel/UX.


Support for ChromeCast

Hi,
Would it be possible to integrate Chrome's support for Chromecast casting to ElectronPlayer?
Thanks,
Koen.

That sounds like a great idea but I don't know if it is possible. Did you want to cast content or the entire application? There may also be problems due to the protections put in place by the streaming services DRM but there may be a way to integrate with the DRM so it can still be streamed. I will get back to you when I get some time to work on the app and try adding this feature.

Cool :-)
I think it would be great already in the first place to have the same
native behavior in ElectronPlayer as within Chrome currently (where one
selects: menu -> Cast … -> Select Chromecast device -> streams video
content to the tv-screen).
That would avoid me to login each time on the related website (Netflix,
YouTube, etc . when not watching locally on the laptop) but when wanting to
stream to the TV screen, I just use the ElectronPlayer opened "sessions".
Looking forward to this and thanks in advance :-)
On Mon, 13 Apr 2020 at 13:26, Oscar Beaumont<EMAIL_ADDRESS>wrote:

That sounds like a great idea but I don't know if it is possible. Did you
want to cast content or the entire application? There may also be problems
due to the protections put in place by the streaming services DRM but there
may be a way to integrate with the DRM so it can still be streamed. I will
get back to you when I get some time to work on the app and try adding this
feature.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/oscartbeaumont/ElectronPlayer/issues/63#issuecomment-612861484,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABUGV6OKWX5F4RU26OX7QFLRMLZHZANCNFSM4MGSXUGA
.


Removed the use of sshpass

While bootstrapping this code, we used a VMDK with a well-known
username/password and sshpass. While we did install ssh keys and use
them, the initial bootstrap relied on that password.
The kube image has been updated to remove that account and add
cloud-init. When we start up the VM, we make an ISO that contains the
configuration for cloud-init (the user-data.txt file). This tells
cloud-init to make the kube account. It will be configured with the
ssh key we were already using. We do give it a password (backup login)
using the password randomly generated by kube-up.

Awesome! Looks good for check-in.

Looks good.


[EBPF] - fixed LockContentionCollector init

What does this PR do?
checks for proper initialization of the LockContention Collector before registering it with the telemetry component
Motivation
potential panic due to a nil deref
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Default classNames are not working for other variants

'use client';

import { useTheme } from 'next-themes';
import { Toaster } from 'sonner';

export function ToastProvider() {
	const { theme } = useTheme();

	return (
		<Toaster
			toastOptions={{
				classNames: {
					toast: '!border-border',
				},
			}}
			theme={theme as 'light' | 'dark' | 'system'}
			richColors
		/>
	);
}

When the classNames for toast are changed then the error default styles are also changed even when the error styles are not being touched

That's expected, if you want differrent styling for different toast types you can use it like this:
<Toaster
  toastOptions={{
    unstyled: true,
    classNames: {
      error: 'bg-red-400',
      success: 'text-green-400',
      warning: 'text-yellow-400',
      info: 'bg-blue-400',
    },
  }}
/>

No the styling is applied for all variants when applied to toast variant ignoring the default styling for other variants

I want to change the default style of the toast when we call the toast like this
toast('Event has been created')

because I would like to apply the shadcn themes for the toast when the default toast is being called and it should not change the styling of the other variants

@emilkowalski can I make a PR for the default property which takes the styling in the classNames so that it will not effect the other variants

@imopbuilder Of course, go for it!

@imopbuilder Of course, go for it!

sure I will start working on it

@emilkowalski I have made a PR by add the default property to the types and also to the state

This is still an issue as the type: 'default' was removed in this commit.
I'm trying to apply styling to each variant, while also styling the default to allow usage of toast().
However, applying classes to classNames.toast overwrites the variant styling. The default classname is useless as the toastType is undefined, so it doesn't apply any styles.
export const Toaster = (props: ToasterProps) => (
  <Sonner.Toaster
    toastOptions={{
      classNames: {
        error: "bg-red",
        success: "bg-green",
        warning: "bg-yellow",
        info: "bg-blue",
        // default classes are never applied
        default: "bg-white",
        // can't use bg-white here because it overwrites variant classes
        // toast: "bg-white",
      },
      unstyled: true,
    }}
    {...props}
  />
)

Is there a reason default type was removed?
FYI I'm working around this by checking for toasts that don't have the data-type attribute on them:
[data-sonner-toaster] li:not([data-type]) {
    @apply bg-white;
  }

@justinnais yes the property has been removed in the commit but the types are not updated hence the default type is undefined but you can refer to this link where shadcn-ui used sonner toast to changed the styles for the toast() without affecting other variants
Shadcn-ui sonner toast (take a look at he manual installation)
Here is the implementation
// Toaster.tsx

"use client"

import { useTheme } from "next-themes"
import { Toaster as Sonner } from "sonner"

type ToasterProps = React.ComponentProps<typeof Sonner>

const Toaster = ({ ...props }: ToasterProps) => {
  const { theme = "system" } = useTheme()

  return (
    <Sonner
      theme={theme as ToasterProps["theme"]}
      className="toaster group"
      toastOptions={{
        classNames: {
          toast:
            "group toast group-[.toaster]:bg-background group-[.toaster]:text-foreground group-[.toaster]:border-border group-[.toaster]:shadow-lg",
          description: "group-[.toast]:text-muted-foreground",
          actionButton:
            "group-[.toast]:bg-primary group-[.toast]:text-primary-foreground",
          cancelButton:
            "group-[.toast]:bg-muted group-[.toast]:text-muted-foreground",
        },
      }}
      {...props}
    />
  )
}

export { Toaster }

@imopbuilder richColors overrides the colors that I am passing into the class names, which isn't what I'm after unfortunately. I had a play with the tailwind group classes but it doesn't solve the issue of the toast class overriding all of the variant classes.
Should the default type be removed entirely, or should the prop be added back to the default toast?

However, applying classes to classNames.toast overwrites the variant styling.
export const Toaster = (props: ToasterProps) => (
  <Sonner.Toaster
    toastOptions={{
      classNames: {
        error: "bg-red",
        success: "bg-green",
        warning: "bg-yellow",
        info: "bg-blue",
        // This variant should be applied first in the chain, not last, so that any similarly defined style above takes precedence
        // Currently none of the other variants work as they will all appear white
        // toast: "bg-white",
      },
      unstyled: true,
    }}
    {...props}
  />
)

Do we have an update on this? This is pretty frustrating to work around. Intuitively any of the styling for the "variant" toasts (error, success, warning, info) should be applied after the "default" toast (toast).

This is fixed in #324. You can now style with the default key, those styles will be overidden by any type specific ones.

This is fixed in #324. You can now style with the default key, those styles will be overidden by any type specific ones.

Just a heads up that I pulled the latest version 1.4.1 and it seems the "error" variant is still using the background color as styled in either "toast" or "default". The "success" variant is working correctly however.
I also wanted to say thank you for a wonderful toast component.

This is fixed in #324. You can now style with the default key, those styles will be overidden by any type specific ones.

Just a heads up that I pulled the latest version 1.4.1 and it seems the "error" variant is still using the background color as styled in either "toast" or "default". The "success" variant is working correctly however.
I also wanted to say thank you for a wonderful toast component.

I had the same issue and I realised that the issue was due to my class name for my error variant (bg-error-background) is before my default variant (bg-foreground) *Notice alphabet e comes before alphabet f.
The fix is to provide custom cn function via cn prop that includes tw-merge so that the ordering of the class name will be correct. If you are coming from shadcn, you can just do
'use client';

import { cn } from '@/lib/utils';

// ...

<Sonner
  // ...
  cn={cn}
  // ...
/>

Hopefully that helps!


[SPARK-40805] Use spark username in official image

What changes were proposed in this pull request?
This patch:

Add spark uid/gid in dockerfile. (used in entrypoint)
Use spark user in entrypoint.sh rather than Dockerfile. (make sure the spark process is executed as non-root users)
Remove USER setting in Dockerfile. (make sure base image has permission to extend dockerifle, such as execute apt update)
Chown script to spark instead of root. (avoid permission issue such like standalone mode)
Add gosu deps, a sudo replacement recommanded by docker and docker official image, and also are used by other DOI images.

This change also follow the rules of docker official images, see also consistency.
Why are the changes needed?
The below issues are what I have found so far

Irregular login username
Docker images username is not very standard, docker run with 185 username is a little bit wired.
$ docker run -ti apache/spark bash
185@d88a24357413:/opt/spark/work-dir$

Permission issue of spark sbin
And also there are some permission issue when running some spark script, such as standalone mode:
$ docker run -ti apache/spark /opt/spark/sbin/start-master.sh

mkdir: cannot create directory ‘/opt/spark/logs’: Permission denied
chown: cannot access '/opt/spark/logs': No such file or directory
starting org.apache.spark.deploy.master.Master, logging to /opt/spark/logs/spark--org.apache.spark.deploy.master.Master-1-1c345a00e312.out
/opt/spark/sbin/spark-daemon.sh: line 135: /opt/spark/logs/spark--org.apache.spark.deploy.master.Master-1-1c345a00e312.out: No such file or directory
failed to launch: nice -n 0 /opt/spark/bin/spark-class org.apache.spark.deploy.master.Master --host 1c345a00e312 --port 7077 --webui-port 8080
tail: cannot open '/opt/spark/logs/spark--org.apache.spark.deploy.master.Master-1-1c345a00e312.out' for reading: No such file or directory
full log in /opt/spark/logs/spark--org.apache.spark.deploy.master.Master-1-1c345a00e312.out

spark as base image case is not supported well
$ cat Dockerfile
FROM apache/spark
RUN apt update

$  docker build -t spark-test:1015 .
// ...
------
 > [2/2] RUN apt update:
#5 0.405 E: Could not open lock file /var/lib/apt/lists/lock - open (13: Permission denied)
#5 0.405 E: Unable to lock directory /var/lib/apt/lists/
------
executor failed running [/bin/sh -c apt update]: exit code: 100

Does this PR introduce any user-facing change?
Yes.
How was this patch tested?

CI passed: all k8s test

Regression test:

# Username is set to spark rather than 185
docker run -ti spark:scala2.12-java11-python3-r-ubuntu bash
spark@27bbfca0a581:/opt/spark/work-dir$

# start-master.sh no permission issue
$ docker run -ti spark:scala2.12-java11-python3-r-ubuntu bash

spark@8d1118e26766:~/work-dir$ /opt/spark/sbin/start-master.sh
starting org.apache.spark.deploy.master.Master, logging to /opt/spark/logs/spark--org.apache.spark.deploy.master.Master-1-8d1118e26766.out

# Image as parent case
$ cat Dockerfile
FROM spark:scala2.12-java11-python3-r-ubuntu
RUN apt update
$ docker build -t spark-test:1015 .
[+] Building 7.8s (6/6) FINISHED
 => [1/2] FROM docker.io/library/spark:scala2.12-java11-python3-r-ubuntu                                                                                0.0s
 => [2/2] RUN apt update                                                                                7.7s

Other test:

# Test on pyspark
$ cd spark-docker/3.3.0/scala2.12-java11-python3-r-ubuntu
$ docker build -t spark-no-chrown:scala2.12-java11-python3-r-ubuntu .
$ docker run -p 4040:4040 -ti spark:scala2.12-java11-python3-r-ubuntu /opt/spark/bin/pyspark

# A simple test for `start-master.sh` (standalone mode)
$ docker run -ti spark:scala2.12-java11-python3-r-ubuntu bash
spark@8d1118e26766:~/work-dir$ /opt/spark/sbin/start-master.sh
starting org.apache.spark.deploy.master.Master, logging to /opt/spark/logs/spark--org.apache.spark.deploy.master.Master-1-8d1118e26766.out

cc @HyukjinKwon @zhengruifeng
BTW, @dcoliversun you could try to help SPARK-40570 based on this patch.

@pan3793 The DOI recommand to use a certain account/gid/uid as I mentioned in PR description, the ability to dynamically create users still exists when the user specifies a USER_ID based on the official image.
FROM spark
# Specify the User that the actual main process will run as
USER ${spark_uid}

This is the best way I can think of at the moment and also taking into account your scenario.

Thanks @Yikun for the explanation, the idea of keeping the default login user to 'spark' and allowing to extend to use dynamic login user makes sense to me.
And let me explain a little about why we need the dynamic login user ability. In Spark on Yarn mode, when we launch a Spark application via spark-submit --proxy-user jack ..., the Yarn will launch containers(usually Linux processes) using Linux user "jack", and some components/libraries rely on the login user in default, one example is Alluxio
https://github.com/Alluxio/alluxio/blob/da77d688bdbb0cf0c6477bed4d3187897fe2a2e1/core/common/src/main/java/alluxio/conf/PropertyKey.java#L6469-L6476
  public static final PropertyKey SECURITY_LOGIN_USERNAME =
      stringBuilder(Name.SECURITY_LOGIN_USERNAME)
          .setDescription("When alluxio.security.authentication.type is set to SIMPLE or "
              + "CUSTOM, user application uses this property to indicate the user requesting "
              + "Alluxio service. If it is not set explicitly, the OS login user will be used.")
          .setConsistencyCheckLevel(ConsistencyCheckLevel.ENFORCE)
          .setScope(Scope.CLIENT)
          .build();

To reduce the difference between Spark on Yarn and Spark on K8s, we hope Spark on K8s keeps the same ability to allow to dynamically change login user on submitting Spark application.

@pan3793

the idea of keeping the default login user to 'spark' and allowing to extend to use dynamic login user makes sense to me.

Thanks for feedback! Good to know this change also meet your requitements.

And let me explain a little about why we need the dynamic login user ability.

I personally think it's a good way, this also benifit users who want to migrate yarn to k8s. But as we discussed, let us do it in a separate PR.

Thanks for ping me @Yikun I'm working on it.

@HyukjinKwon @zhengruifeng @martin-g @dcoliversun @pan3793 Thanks all! If no more comments, I will merge this today.

fine to me, it is consistent with the usage habits for me

@HyukjinKwon @zhengruifeng @martin-g @dcoliversun @pan3793 @LuciferYang Merged. Thanks all.


[SKIP CI] Documentation update for ESXCLI usage and minor command fixes

Updates documentation in admin-cli.md to reflect usage of esxcli
Update readme installation instruction to restart hostd after vib installation
Minor fixes to handle values with spaces for flags like name, vmgroup, datastore, description etc.
Introducing volume shortls command to display limited info about volumes.

Fixes #1757

@shaominchen
Yes we can come back and change the namespace once we finalize. Till then guestvol.
For shortls, yes it does sound a bit wierd. esxcli suggests the command name should be a verb.
However, given the functionality of command, I couldn't comeup with a better name,  which indicates volume ls with limited info.

@pshahzeb don't we have to update all markdown files? e.g. https://github.com/vmware/docker-volume-vsphere/blob/ee3ef339cb0703492e6d02f96fb4cb8a242065a1/docs/features/tenancy.md

@shaominchen @shuklanirdesh82
I have updated tenancy.md and faq.md files as well. Not touching vmdkops-admin-cli-spec.md
since it was the original spec file for vmdkops_admin.
Please take a look


How can we manage control options for media devices?

I'm been messing with node-red-contrib-lgtv, to control an LG smart TV. I've successfully built up a number of flows to turn the tv on/off, increment/decrease the volume and increment/decrease the channel.
When I create a device at https://alexa-node-red.bm.hardill.me.uk/devices , "TV" for example, I can set it's actions to turn on, turn off, increment percentage and decrement percentage. On/off, Volume +/- accordingly. Not really anywhere for the channel +/-.
If I try to split the controls over three devices, TV, TV volume, Channel, Alexa doesnt recognise the two new devices. Possibly "TV volume" is too close to TV and Alexas built in volume control. Strangely "Channel", is "command not recognised on this device."

You can't yet as I've not had time to implement the AV controls yet.

I'm in no rush for them, I was just messing about with some practice flows.
Thanks for the work so far.


Additional requirements for ADF integration runtime and Parquet files

Hello,
I would like to suggest to add more relevant information about the required configuration if you don't use official JRE but want to use OpenJDK. For me installing only OpenJDK and setting JAVA_HOME didn't work at all and I tried around 5-6 different JDK versions and setups.
Finally, I managed to get it working with just copying JDK and doing correct settings, however except of JAVA_HOME I had to configure also:

registry keys same as for official JRE - SOFTWARE\JavaSoft\Java Runtime Environment\{Current Version}\JavaHome and RuntimeLib
add the the same path as JAVA_HOME to Path variable

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 183a97b5-683f-f51e-41b0-c2676b89feff
Version Independent ID: e523331b-54cd-f95a-27b2-72f9db28e08d
Content: Parquet format in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/format-parquet.md
Service: data-factory
GitHub Login: @linda33wj
Microsoft Alias: jingwang

@kolodziejczyk0 Thanks for your question. We are checking on this and will respond to you soon.

@linda33wj - Can you please take a look and share your thoughts

#reassign:jianleishen

Thanks for submitting this issue @kolodziejczyk0.  I discussed with our product team now.  They decided against including additional detail specific to the OpenJDK's configuration for now, since there could be other configurations too then that might be missed.  Since these are external to the product, the decision was made to document configuration with the official JRE only.  I will close this issue out now since the team made a conclusion after they discussed internally, but we can still continue the discussion in the thread if necessary, too, even after the issue is closed.  #please-close


missing live |= in visitor

https://github.com/estools/esmangle/blob/90bf70752292064aecd33160a5c9cb0308038079/lib/pass/dead-code-elimination.js#L447

I think you're right. It seems like
switch(f()){}

will be incorrectly dropped as dead code. @aryx Are you interested in sending a PR?


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3263

Questions ?
Please refer to the Backport tool documentation


Updating benchmarking.rst documentation

The runtest.py module is being phased out in favor of the dev.py module.

Reference issue

What does this implement/fix?

This fixes the documentation to include the new test module.
Additional information

I think the other commands will need revising too--I just did a few spot checks to get you started down the right path.

@hfactor13 this change would be good to get merged. Are you planning to update this PR?

Yes, I just have to run the tests and I think I should be good.

I just pushed my commits. Let me know if there's anything else I am missing.


Save full path to edited file

Hello.
Currently "local-history.path" parameter changes where extention saves edited file with relative path to opened workspace. That's fine but not always.
i.e.

set "local-history.path": "c:\history"
if we open c:\projects directory.
edit file .\hello-world\file.txt
then history file will be saved to c:\history.history\projects\hello-world.

What if I open the same project from different paths (like current and archive/backup).
History will be messed up.
It would be more consistent if extention saves full path (from root directory inside .history) to file, so that one can find for sure where edited file exists.
like this: c:\history.history\c\projects\hello-world\file-yyyymmddhhmiss3.
like this: c:\history.history\d\projects\hello-world.
The same way works SublimeText and Atom.
Thank you, Alexander.

I had already thought of this alternative. Especially to save a file history when no workspace is loaded (issue 8).
I started something for this purpose, I'll publish it soon ...

Fix in version 1.3.0
Set new settings 'absolute'  to true.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3315

Questions ?
Please refer to the Backport tool documentation


code change: Improve code around flavors

What this PR does / why we need it:
Extract adding flavor name annotations into a separate function.
Release note:
None

/retest

/retest

/retest

/cc @ksimon1

/retest

/retest

/retest

@xpivarc do you know who could review and approve this?

/retest

/cc @jean-edouard @vladikr
Hi, could you please review this PR, or mention someone who can?

/retest

@vladikr @davidvossel apologies for the direct ping but could one of you lgtm this please so we can start posting some additional early parts of the flavor series.

@vladikr @davidvossel apologies for the direct ping but could one of you lgtm this please so we can start posting some additional early parts of the flavor series.

@lyarwood i think you can add lgtm, you would just need us for an approve.

@vladikr @davidvossel apologies for the direct ping but could one of you lgtm this please so we can start posting some additional early parts of the flavor series.

@lyarwood i think you can add lgtm, you would just need us for an approve.

Apologies I thought I had to be listed in the OWNERS file as a code-reviewer for that to work and not just a member of the org.
/lgtm

Error in download: java.io.IOException: Error downloading [http://mirror.centos.org/centos/8-stream/BaseOS/x86_64/os/Packages/selinux-policy-targeted-3.14.3-95.el8.noarch.rpm]  unrelated failure.
/retest

/retest


Spatial audio flag not being set on variant tracks

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
3.3.6 - 4.1.0
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from main?
Yes
Are you using the demo app or your own custom app?
Both
If custom app, can you reproduce the issue using our demo app?
Yes
What browser and OS are you using?
All browsers
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
What are the manifest and license server URIs?

Send separately
What configuration are you using?  What is the output of player.getConfiguration()?

Default
What did you do?

Load a stream with a Dolby Atmos representation
Call getVariantTracks()

What did you expect to happen?
Some of the tracks should be listed with spatialAudio as true
What actually happened?

All of the tracks are listed with spatialAudio as false

Fixed in #4291


Inconsistent results of router.push

Reproduction
https://codesandbox.io/s/mutable-sky-22l4gj
Steps to reproduce the bug
router.push('/hello?return_url=' + encodeURIComponent('https://www.my.com'))

and
router.push({
    path: '/hello',
    query: {
      return_url: encodeURIComponent('https://www.my.com')
    }
})

/hello page

<template>

Hello

</template>

Expected behavior
I want got route.query = {return_url: 'https://www.my.com'}
but inconsistent results
Actual behavior
Multiple encodeURIComponent
Additional information
No response

Do not encode when passing a query object. The router does it for you 🙂


weird issue on shift+t, possibly crossmod

https://pastebin.com/U07db1da

Can you ID a specific item that did this?

happens on trying to link any item

It looks like it's a crossmod thing, since it's happening in the ChatListenerHud, which my mod doesn't touch. I'm using the vanilla code to convert an ItemStack to a text component, so someone whose mod is messing with the chat hud isn't properly built for that.

turns out that Ping is the culprit


[BUG]: Chicken crash

Does an issue already cover this?

[X] I have searched for any existing issues, and didn't find any.

Does your system support Faster Random?

[X] I have checked that my JVM supports Faster Random 3.0.0+.

Issue description.
Chickens randomly crash on my server. Neruina caught it so it didn't actually crash and I just put in the error it gave me.
Issue replication.
We just have multiple chickens crammed in a space for egg production and they very occasionally crash. Not sure how many other mods and such are involved.
Log File / Crash Report (if applicable)
https://gist.github.com/mxnmnm/4948b04b62efd7a7ca17b63f8e09ae74

Looking at the source code for MC, it doesn't look like this is possible in vanilla (nothing that's relevant to the chicken calls with a bound that could become negative). This is likely a mod conflict of some sort, but I don't know what mods you have running.

I am running A LOT of other mods and I'm not sure how to consistently reproduce the issue in a way that's testable within a reasonable amount of time :/

Other than your standard binary search do you have any ideas of what mods might be causing this? I'll send a list of the mods I have but considering how many there are I'm not sure it'll be that helpful

https://gist.github.com/mxnmnm/1cb1f8d192b17fa377b45cec84500c03

I don't really know what it could be. All the stacktrace you have really says is "something called aiStep() on a chicken and made Faster Random generate a negative number." Using GitHub's code search showed nothing, so chances are it's a smaller/closed-source mod that I probably won't be able to track down, and Neruina doesn't give enough information on what is adding the offending nextInt() call.
In Minecraft's source, nextInt is only called once with a constant, positive bound within the whole method hierarchy for tickMovement()/aiStep(), so I'm sure that some lesser-known mod is causing this. I'm closing as 'not planned' for now, as this might not even be a bug with Faster Random itself.

So some more interesting info: This only happens on dedicated servers

The chicken logic would only occur server-side anyways. But this should be triggerable in singleplayer/LAN too.


chore: add Aspect Workflows CI

Ah, should we remove testing the root workspace on vanilla GHA right now as well? https://github.com/aspect-build/bazel-examples/blob/main/.github/workflows/ci.yaml#L49
It would be nice to have this be really crisp and not overlapping

Ah, should we remove testing the root workspace on vanilla GHA right now as well? https://github.com/aspect-build/bazel-examples/blob/main/.github/workflows/ci.yaml#L49
It would be nice to have this be really crisp and not overlapping

SG. Will leave that for a follow-up as well so this commit is just the minimal diff for adding Aspect Workflows.


Disable Pluarlizer when using dotnet-ef dbcontext Scaffold

.net core 3.0
When i try scaffold from my database it now groans about not having Bricelam.EntityFrameworkCore.Pluralizer. But i do not want to pluralize any of the entities so i can maintain backward compatibility to other libraries.
How can i disable the pluralizer? The behaviour in .net 2.2 was more expected, that only if the library was added would it pluralize.

It has to be installed somewhere if it's finding [assembly: DesignTimeServicesReference("...Bricelam.EntityFrameworkCore.Pluralizer")].
In 3.0, I believe we updated it to look inside the target project in addition to the startup project. Could you share your csproj files?

The project is quite complex i have about 4 sub projects in the solution.
I removed the pluralizer from the .core execution project and from my data project not from the others. What seems to happen is that it picks up there is a pluralizer somewhere but when it gets to execute it cant find it. I think most likely i need to remove it from all the sub projects.

@ganySA It shouldn't be possible to get in a state where the package is picked up, but then the DLL is not found. Could you post a small project/solution that has this behavior?

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Implement expr

gonna tackle this one

You have probably given up on coding this issue so I'd like to claim this task.

it's all yours, sorry about that

@xmasotto, do you mind? :)


Assessment Complete...ish :)

HARD!

0/3. So close! on question 2 would be correct except on line 33 you return "x" explicitly instead of the number passed into the method.


serial_recovery: Reduce minimum number of members in packet

Was 3, but mcumgr sometimes sends 2 ("off" + "data", "len" is not needed
when "off" != 0). Reduce to 1 to avoid the problem, since the code
has other checks to catch this.
Signed-off-by: Øyvind Rønningstad<EMAIL_ADDRESS>

This is mirror of bugfix which went to NCS mcuboot here: https://github.com/NordicPlayground/fw-nrfconnect-mcuboot/pull/96


⚠ Bofrost.fr has degraded performance

In 062f3c6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1179 ms

Resolved: Bofrost.fr performance has improved in 4e2479a after 6 minutes.


[FLINK-6567] [tests] Harden ExecutionGraphMetricsTest

What is the purpose of the change
The problem was that in some cases the currentRestartingTime would not increase
because the iteration in the loop were too fast.
cc @zentol
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for the review @zentol. Let me know once you've run the test on your machine so that I can merge this PR.

I've run the test 10k+ times on my machine without a failure, +1 from my side.

Thanks for the review @zentol. Merging this PR.


Update Docker file with missing information

@syncrou @gmcculloug Please review.
The major missing piece was the git installation for git based gems, but the rest are to match how topological-inventory-api works for consistency.

Hakiri is going to fail so ignore that one...I am working with Hakiri to understand why this false positive keeps happening


🐛 Bug Report: word wrap in column is unreadable

👟 Reproduction steps
Running appwrite projects list returns a heavily word wrapped output.
👍 Expected behavior
It should split the table every few columns so that it lines up and is readable.
👎 Actual Behavior
It renders a very unreadable output, at least not without super-shrinking my terminal text

🎲 Appwrite version
Different version (specify in environment)
💻 Operating system
MacOS
🧱 Your Environment
2.0.2
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

@Cakeday For now, use the --json flag and dump to a json file. We'll definitely consider this in the future.
Thanks for bringing this up <3


Update setup.py

Set the version of pydantic in setup.py as the same as in requirements.txt — 2.3.0.
Related issue https://github.com/imryche/blockkit/issues/101

Fixed in https://github.com/imryche/blockkit/commit/0c43b6807e1a98f9fd4416fa74b5b0a42a5cf88b


JS private identifiers are included in the generated documentation

Documentation for private class methods defined using private identifier syntax (i.e., #privateMethod() {}) is always emitted, despite being private at runtime.
To Reproduce
See https://github.com/bryan-hoang/repro-tsdoc-markdown-private-identifiers
Expected behavior
I expected the documentation to be omitted, like how it is for private methods declared using the private keyword.
Additional context
To work around the issue, I have to switch to using the private keyword.
I took a quick glance at how the visibility for the documented symbol is determined in src/lib/docs.ts#146. Based on Typescript AST Viewer ^1, checking for PrivateIndentifier seems to be the way to go, although I'm not familiar enough with the compiler's API to know how difficult it would be to add the check.

Good catch! The private identifier syntax was indeed being generated, and I also noticed that the inner properties of classes weren’t being generated either. I resolved both issues this evening and released a new version of the library, v1.1.0.

Thank you @peterpeterparker!


need placeholder for user when setting volumn

In GitLab by @wknet123 on Jun 12, 2018, 15:49
Directly configure containers with images in on step.

In GitLab by @liyanq528 on Nov 8, 2019, 07:52
fixed by pull request #1452

In GitLab by @luotian-github on Nov 8, 2019, 07:52
verified

In GitLab by @sokril on Nov 8, 2019, 08:23
closed

In GitLab by @sokril on Nov 8, 2019, 08:23
assigned to @luotian-github


Fix broken web link

Description
Found an alternate posting of this academic paper.  Fixes link check job https://github.com/apache/incubator-mxnet/runs/5577830518?check_suite_focus=true
Checklist
Essentials

[X] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage
[X] Code is well-documented

Merging to unblock CI - my concerns can be discussed in issue if necessary
(btw. sorry for putting this comment in commit msg)


Recreate package lock files

Delete node_modules folder and package-lock.json file
npm install

Before npm ci:
added 270 packages, and audited 271 packages in 4s

92 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

node_modules folder size: 76.6 MB

After:
added 228 packages, and audited 229 packages in 4s

92 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

node_modules folder size: 69.8 MB

: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 82.11%. Comparing base (e759f46) to head (f34decb).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3146   +/-   ##
=======================================
  Coverage   82.11%   82.11%
=======================================
  Files          76       76
  Lines        3125     3125
  Branches      526      526
=======================================
  Hits         2566     2566
  Misses        559      559

Flag
Coverage Δ

Linux
82.11% <ø> (ø)

Windows
?

macOS
82.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

What's the benefit of you doing this manually? Dependabot will auto-regenerate it when npm dependencies get updated.

Dependabot can't migrate to latest package-lock version v3.

Ah, I hadn't noticed they were still on the older lockfile format.


Fix error when registration-dir doesn't exist.

What this PR does / why we need it:
Fix error when registration-dir doesn't exist.
Test environment:

Kuberenetes version: v1.13.0-beta.1
Etcd version: 3.2.25

also tested on Kubernetes v1.13.0

You need at least v1.13.0-beta.2


documentation about fugue functionality

Need some docs around the Fugue functionality recently added
and a version bump to v0.10.0 to denote significant changes.

Closed by #205


Added Offhand Support

Closes https://github.com/G07cha/MineflayerArmorManager/issues/14

Thank you very much. I made the requested changes.

Great, thanks again for your contibution! I'll try to publish a new version as soon as possible


Swift 3 issue

I see commits 3 days ago that indicate Swift 3 support. But, I get 40 compiler errors when I include your pod: basic pre-Swift 3 stuff like "NSTimeInterval has been renamed to "TimeInterval". I've cleaned the pod cache, reinstalled. Can you help me?

Hey @Jiropole, we're still having some problems on sending to CocoaPods, so please use this:
 pod 'EZLoadingActivity', :git => 'https://github.com/goktugyil/EZLoadingActivity.git'

Let me know if you need anything else! 😉

Just created the new version with it. Heres how to install it with cocoapods:
pod 'EZLoadingActivity' #Stable release for Swift 3.0

pod 'EZLoadingActivity', :git => 'https://github.com/goktugyil/EZLoadingActivity.git' #Latest release for Swift 3.0
pod 'EZLoadingActivity', '~> 0.8' #For Swift 2.2


No parameter named "" for some django columns even with typing correctly set

Hi @sbdchd . First of all, thanks for all your work here to allow django-stubs to be used without mypy
I'm having some issues when passing some attributes to some columns (e.g. decimal_places on DecimalField). AFAIK they are properly mapped, but pyright keeps giving me errors regarding it.
I first opened the issue on pyright here (https://github.com/microsoft/pyright/issues/2077) because I thought it was their fault, but it might not be.
Any ideas on how/why this is happening? You can check my example code here (https://github.com/microsoft/pyright/issues/2077#issuecomment-877857826) and run it with pyright and you will get the exact same issues that I got.

The maintainer there pointed that it might have something to do with __new__. I'm old to python but new to typing, so I'm not really sure what is going on yet.

Thanks for the report!
The __new__ / __init__ overloads are really hacks to get the types to behave with both pyright and mypy
Basically we want to type these fields as generics so we can overload the __get__ and __set__ properly, i.e., a nullable integer results in a int | None.
Mypy tends to be more flexible with "self overloads", aka overloading __init__ with a type on the self param so we can set the generic param based on the arguments of the __init__, but to get pyright to be happy we sometimes have to opt to overload __new__.
semi related:
https://github.com/microsoft/pyright/issues/1550
https://github.com/microsoft/pyright/issues/1211
Anyways, should be fixed in 0.6.0, let me know if you run into any other issues!

Hi @sbdchd ,
Thank you! I'll try the new upate and come back if I find any other issue.


Bug: HealthPlanner does not prompt invalid add command

When invalid add command with missing tag is entered, for eg "add n/btg a/21 w/60 t/65 d/20" is missing "h/" tag, nothing happens. Window should show invalid command message to prompt user that a tag is missing.

Fixed by AlecNgZW


Update index in advance of front end changes

This isolates the index changes from #1598 so that we can reindex before making the front end changes. #1598 is closed until we merge this and run Vacancy.reindex.

do we need to do this for the school address display format too? that isnt in the index currently but this probably has a presenter or some logic in the template that prob shouldnt be there

do we need to do this for the school address display format too? that isnt in the index currently but this probably has a presenter or some logic in the template that prob shouldnt be there

I think that's the location presenter. @david-mears-dfe can confirm

do we need to do this for the school address display format too? that isnt in the index currently but this probably has a presenter or some logic in the template that prob shouldnt be there

As Connor says, #1598 covers this, using the location attribute. It delegates to a presenter method.


python311Packages.outlines: 0.0.37 -> 0.0.38

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.outlines is: Structured text generation
meta.homepage for python311Packages.outlines is: https://github.com/outlines-dev/outlines
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.0.38 with grep in /nix/store/6pdx4k7f9zmxgrq4r6h06hzkl67nbh9d-python3.11-outlines-0.0.38
found 0.0.38 in filename of file in /nix/store/6pdx4k7f9zmxgrq4r6h06hzkl67nbh9d-python3.11-outlines-0.0.38

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.outlines
python311Packages.vllm
python312Packages.outlines

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/6pdx4k7f9zmxgrq4r6h06hzkl67nbh9d-python3.11-outlines-0.0.38 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.outlines https://github.com/r-ryantm/nixpkgs/archive/936afa176fa3b937cae5de6db9c5d7ab94495880.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/936afa176fa3b937cae5de6db9c5d7ab94495880#python311Packages.outlines

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/6pdx4k7f9zmxgrq4r6h06hzkl67nbh9d-python3.11-outlines-0.0.38
ls -la /nix/store/6pdx4k7f9zmxgrq4r6h06hzkl67nbh9d-python3.11-outlines-0.0.38/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages marked as broken and skipped:

python311Packages.vllm
python311Packages.vllm.dist

4 packages built:

python311Packages.outlines
python311Packages.outlines.dist
python312Packages.outlines
python312Packages.outlines.dist

Maintainer pings
cc @CertainLach for testing.

Per the changelog, some dependencies were removed in https://github.com/outlines-dev/outlines/pull/804. We should update our package accordingly.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.outlines is: Structured text generation
meta.homepage for python311Packages.outlines is: https://github.com/outlines-dev/outlines
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.0.40 with grep in /nix/store/ixxv5ffhfvfr4s8r31n1g8ahpy64svsf-python3.11-outlines-0.0.40
found 0.0.40 in filename of file in /nix/store/ixxv5ffhfvfr4s8r31n1g8ahpy64svsf-python3.11-outlines-0.0.40

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

python311Packages.outlines
python311Packages.vllm
python312Packages.outlines

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ixxv5ffhfvfr4s8r31n1g8ahpy64svsf-python3.11-outlines-0.0.40 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.outlines https://github.com/r-ryantm/nixpkgs/archive/f9607c2b6ef4686f0c20830c461239336e9b6dc6.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/f9607c2b6ef4686f0c20830c461239336e9b6dc6#python311Packages.outlines

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ixxv5ffhfvfr4s8r31n1g8ahpy64svsf-python3.11-outlines-0.0.40
ls -la /nix/store/ixxv5ffhfvfr4s8r31n1g8ahpy64svsf-python3.11-outlines-0.0.40/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages marked as broken and skipped:

python311Packages.vllm
python311Packages.vllm.dist

4 packages built:

python311Packages.outlines
python311Packages.outlines.dist
python312Packages.outlines
python312Packages.outlines.dist

Maintainer pings
cc @CertainLach for testing.


StackOverflow plugin

Search stackoverflow with given search string and return 3 results, and the search page link

From @Makman2 at https://github.com/coala/corobo/issues/46#issuecomment-302904686

Well this was easy : https://github.com/jvasallo/err-stackoverflow


feat(nav): set up app navigation

#48  👈
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @libhide and the rest of your teammates on  Graphite


Standby python actor throws when accessing the Actor.config object

Reported: https://discord.com/channels/801163717915574323/1020281045205123102/threads/1294774068528156817
https://apify.slack.com/archives/CD0SF6KD4/p1729096555570669

The problem is the actor_timeout_at field - it expects a valid timestamp, but it receives an empty string from the platform when running in standby mode.
We have two options - 1. add a custom validator that resolves an empty string to None or 2. bug the platform people to stop passing the variable in standby mode.
Since 1. is pretty trivial to do, I'm in favor of that. @vdusek?

Agreed, 1) sounds good.


'cachePath(forComputedKey:)' is deprecated, alternative not public

Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.

[x] I have read the wiki page and cheat sheet, but there is no information I need.
[x] I have searched in existing issues, but did not find a same one.
[x] I want to report a problem instead of asking a question. It'd better to use kingfisher tag in Stack Overflow to ask a question.

Issue Description
What
cachePath(forComputedKey:) is deprecated: Use diskStorage.cacheFileURL(forKey:).path instead. but cacheFileURL is not public.
Reproduce
Other Comment
[Add anything else here]

It is a problem and I will mark it as public in the next release!


Changed UI of yellow and blue theme

Differentiated between footer and body colour for themes. As it was same previously for yellow and blue theme.
Made it look same for overall themes.

Under hacktoberfest'23


Text recombining

Interesting, why exactly do we need this?

Not sure what you mean?
This PR is mostly about the new surrounding text recombination method which performs better than parent document retrieval


Is there a way to change iOS text color in tooltip?

@nericode
I need to have iOS tooltip color to black and text color to white.
Please advise.

@nericode
I have added ability to change iOS textcolor in the plugin and raised pull request with the change.
Here is its link:
https://github.com/nericode/nativescript-tooltip/pull/6
Please review it at your earliest convenience.
Thanks,
Harish


 I compiled the latest versions of both xenos and the blackbone driver from github.

+1


feat: adds positive color semantic for DNA-1151

Description
No, really.  Just adding positive semantic colors.
Related Issue
DNA-1151
Motivation and Context
We are positive we need positive color semantics
Types of changes

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

Checklist:

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

:tada: This PR is included in version 12.0.0-beta.14 :tada:
The release is available on:

GitHub release
npm package (@beta dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


[HLS：0.17.15] It does NOT work well in Android browser or PC browser by OBS with RTSP

Which version are you using?
v0.17.15
Which operating system are you using?

[ ] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[x] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
[HLS：0.17.15] It does NOT work well in Android browser or PC browser by OBS with RTSP. But the v0.17.14 is OK.
In Android, there is NO audio.
In PC, there is NO video and audio.
Describe how to replicate the issue

start the server
publish with OBS
SETTINGS: {
Output > Recording > Output to URL > rtsp://<IP_ADDRESS>:8554/live
Container Format: rtsp
Video Bitrate: 1200 Kbps
Frames: 30
Video Encoder: h264_amf (libx264)
Audio Encoder: aac
}
read with Chrome of Android 10 or PC (http://<IP_ADDRESS>:8888/live)

Thanks for reporting this and sorry for the inconvenience. A new version has been immediately released (v0.17.16).


Size of compressed image is not follow the "maxSizeMB" options

const imageCompressionOptions = {   maxSizeMB: 0.3, };
await imageCompression(             imgBlob as File,             imageCompressionOptions,           );
If image size is more than 2mb then I might get 400kb or 500kb size of compressed image.

const imageCompressionOptions = { maxSizeMB: 0.3, };
await imageCompression( imgBlob as File, imageCompressionOptions, );
If image size is more than 2mb then I might get 400kb or 500kb size of compressed image.


Fix for #11801

When the input field for font is not valid set the preferences in the default font size.

Thanks for contributing to Brackets @jerrak0s!
I think the pattern attribute could work, not sure if there's a premade size validation regexp though but the example @ficristo gave should be close enough after adding rems and so on.
@jerrak0s you can see the documentation on pattern here: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input#attr-pattern

My intention was to fix the wrong value in the textfield Font Size: after wrong input.
So when i save something wrong in the font Size, that is not saved.

@jerrak0s Yup, and it's a good thing to do! With the pattern attribute however, you can ensure that users cannot write invalid values at all to the field 👍

This needs more work. Closing for now.
The issue #11801 is still open.


feat: add 'lint' to d3fc-scripts

Adding a markdown lint command to be used across modules.

That looks useful, thanks @mbssantos - are there markdown linting errors in the other d3fc-* projects that this will identify?

Did it mostly for https://github.com/d3fc/d3fc/issues/9 it did find a smaller problem in this package too.
Just checked, there's around 80 errors in total on the d3fc project but all minor stuff.

Looks good - thanks


www.jsperf.com broken

https://www.jsperf.com/ returns a CloudFlare SSL handshake error  (https://jsperf.com/ works fine).

I can confirm it's still reproduced.

Confirmed

Please use https://jsperf.com as the canonical URL 😸


Improves handling of opaque types for dynamic shapes

Improves handling of opaque types for dynamic shapes.
The immediate motivation for this is to support the lowering
to StableHLO for programs with polymorphic shapes. This requires
mixing of dynamic shapes with opaque types.
The general strategy is to push the actual selection of the MHLO ops
down into mlir module (e.g., mlir.slice_op, mlir.broadcast_in_dim)
so that we have one place where we pick whether we use the Dynamic
or static ops. These routines can also handle the opaque type.
This will result in a recursive
call to, e.g., mlir.slice_op, but the inner call will be using
the physical avals, which should not be opaque anymore.
While making this change I was confused by the fact that the
custom KeyTyRules in prng.py have lowerings that return multiple
MHLO ops. See https://github.com/google/jax/pull/11768#issuecomment-1342349102.
I changed the rules to return a single op.

I replied in https://github.com/google/jax/pull/11768#issuecomment-1343910108 about the multiple MHLO outputs when lowering.

I want to think some more about this. I think it is good to allow lowering to return multiple IR values, but we need to think what is the status of the custom lowering rules like _rules.broadcast_in_dim_mlir: are they meant to be used only in the lowering of broadcast_in_dim, or are they helper functions that generate code that is aware of opaque types and can be used in the lowering of other primitives? In the latter case, it is hard to deal with the situation when the helper function returns multiple IR values.

Exercising the lowering rules of a test opaque dtype is the goal of CustomElementTypesTest.
Do you mean to exercise specifically the lowering rules of the random key opaque dtype? I think we want similar test coverage there (in random_test.py). If we don't have it, we should add tests until we do. Key arrays ought to be sliceable. I can help do that. It's my mistake if current tests are insufficient.

I had in mind testing the actual lowering rules in KeyTyRule, but in the context of jax2tf shape polymorphism. I figured out how to do it, by setting jax_enable_custom_prng and writing a test that broadcasts/slices/... keys. See shape_poly_test:test_prng.

Ah, I actually understand your previous question now that you've answered it! That's right, and the test in shape_poly_test looks good.


AcquisitionOptimizer does not accept model while bayesian_optimization pushes in

Hi I found a problem in bayesian_optimization.py where it pushes kwargs to AcquisitionOptimizer in wrong way such that GPyOpt users cannot specify which GPy models to use by their own appetite.

Problems to fix
i. AcquisitionOptimizer requires the GPmodel itself for non-default GP model being a searcher, but BayesianOptimization does not push that
ii. A typo is there for 'thompson_sampling' in AcquisitionOptimizer
iii. AcquisitionOptimizer keyword mismatch for type_anchor_points_logic - it checks whether 'anchor_points_logic' exists as a key then get 'type_anchor_points_logic'.

Changes to suggest

2-i)In GpyOpt/methods/bayesian_optimization.py
From>
self.acquisition_optimizer_type = acquisition_optimizer_type
self.acquisition_optimizer = AcquisitionOptimizer(self.space, self.acquisition_optimizer_type, model=self.model)
To>
kwargs.update({ 'model' : self.model})
self.acquisition_optimizer_type = acquisition_optimizer_type
self.acquisition_optimizer = AcquisitionOptimizer(self.space, self.acquisition_optimizer_type, **kwargs)
2-ii) In GPyOpt/optimization/acquisition_optimizer.py
From> there is a typo ;)
L11| thompson_sampling_anchor_points_logic = "thompsom_sampling"
To>
L11| thompson_sampling_anchor_points_logic = "thompson_sampling"
2-iii) In GPyOpt/optimization/acquisition_optimizer.py
From> all the keywords and var names should be integrated as 'type_anchor_points_logic'
L37| if 'anchor_points_logic' in self.kwargs:
To>
L37| if 'type_anchor_points_logic' in self.kwargs:

I think most of these changes make sense, but I can't validate them properly. Would you mind doing a PR with them?

Hi @apaleyes, thanks for your reply.
Yes I agree as well, they are just some fixes on what it meant to be nothing related to changes in functionalities - and actually I use (after rounds of debugging, obviously) the above fix quite often.
I have never contributed to open source (hence nor github), but did you mean to say I go for a pull request with the above fixes?
Would it be me making a fix branch from the master, push the changes on the branch and raise a pull request to the master branch?
Thanks.

@apaleyes
I am experiencing an issue upon the PR work could you help me on this?
I just made a hotfix branch('hotfix/modelForAOfromBO') in local

git clone
cd to the repo
git checkout -b hotfix/modelForAOfromBO
vim for changes
git add | commit

Then when I try to push, the repo denies my push to the hotfix branch by saying:
sdr2002ubt@sdr2002:~/public_gits/GPyOpt/GPyOpt$ git push --set-upstream origin hotfix/modelForAOfromBO
Username for 'https://github.com': sdr2002
Password for<EMAIL_ADDRESS>remote: Permission to SheffieldML/GPyOpt.git denied to sdr2002.
fatal: unable to access 'https://github.com/SheffieldML/GPyOpt.git/': The requested URL returned error: 403
sdr2002ubt@sdr2002:~/public_gits/GPyOpt/GPyOpt$ git remote -v
origin	https://github.com/SheffieldML/GPyOpt.git (fetch)
origin	https://github.com/SheffieldML/GPyOpt.git (push)
sdr2002ubt@sdr2002:~/public_gits/GPyOpt/GPyOpt$ git status
On branch hotfix/modelForAOfromBO
nothing to commit, working directory clean
sdr2002ubt@sdr2002:~/public_gits/GPyOpt/GPyOpt$ git branch

hotfix/modelForAOfromBO
master

Please let me know how I can properly create a branch that can push the changes in ISSUE-244 and make the PR, since the take I made above doesn't seem to work.
Thanks.

No worries, I am glad you now have an opportunity to become a part of OSS! Github has some very nice docs to cover that topic. First you need to create your own fork:
https://help.github.com/en/articles/fork-a-repo
then make changes there and create a pull request:
https://help.github.com/en/articles/creating-a-pull-request-from-a-fork
let me know if you need more help

Hi sorry for taking long, but I made the pull request as instructed:
https://github.com/SheffieldML/GPyOpt/pull/257
@apaleyes Could you please take a look?
Thanks.


Fix issue with explicit cast from Map<String?, String> to Map<String,String>

This prevents DartPad embeds to be injected properly as Map<String?, String> is not a subtype of Map<String, String>.

@johnpryan How does this solution look?

I can look into testing this script further when I get a chance, but for now I've tested this with a locally deployed dart.dev


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_0awnmdoyy4/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_0awnmdoyy4/clone/recipe/meta.yaml:22: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_0awnmdoyy4/clone/recipe/meta.yaml:41: missing_description: The recipe is missing a description
Errors were found


feat: generate config

This will automatically generate a config file for the user if it doesn't already exist and ask them to confirm the location of the config file. This doesn't overwrite an existing file.
Can be used like so:
contrail -g -c path/to/file
I will most likely be deleting my version of the fork and re-forking this once we are at version 0.2 since it seems like the pull requests are getting messier (the commits, there are so many of them when only 1 really does anything).

Don't worry about the number of commits! The git authors thought of everything: may I introduce you to git-squash?
Thanks a ton for the work you've put into this, it will be invaluable once 0.2 starts actually getting off the ground.

No problem, I have been using this as a way to learn rust. I'll look into git-squash for future commits/pull requests, thanks for the info!


koord-scheduler: ElasticQuota uses Pod Namespace/Name as cache key

Signed-off-by: xulinfei.xlf<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Use pod.Namesapce/Name as the key of the QuotaInfo's PodCache instead of pod.Name.

Ⅱ. Does this pull request fix one issue?
fix #839

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/approve

/milestone v1.1


Add @legendecas to approvers / CODEOWNERS

Given his regular contribution to OpenTelemetry JS, propose that @legendecas is granted Approver status.
Background:

hengzhong Wu is working on Node.js application observabilities in Alibaba.
He is also working on the TC39 and proposing async contexts for JavaScript.

/cc @legendecas

🎉  Thank you!


How focus inputs ?

How to focus?

@arshidkv12 What would you like to focus? Some specific input or just first?

Just first...
I resolved by using private focus internal function.
Version 3.3.2
On Sat, Oct 19, 2019, 5:39 PM Damian Sznajder<EMAIL_ADDRESS>wrote:

@arshidkv12 https://github.com/arshidkv12 What would you like to focus?
Some specific input or just first?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dsznajder/react-native-otp-inputs/issues/171?email_source=notifications&email_token=ABT5Y5JMYQOTAYW5H4KLYV3QPL2IZA5CNFSM4JCGSITKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBXOALY#issuecomment-544137263,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABT5Y5IUTS7XZUNF4GFZCSDQPL2IZANCNFSM4JCGSITA
.

@arshidkv12 checkout react-native-otp-input@next. I've added focus method for this :)

Hi there, just wondering this issue of focusing was solved. I mean, my issue is that I have set the autoFocus on and I want it to automatically focus on the beginning of OTP input but by default, it's focusing on random inputs mostly on the second last inputs. How can I autoFocus on the first OTP input my default?
I have seen on the example code that you have used a focusOTP function which you then call the function as a button press but how can you make it automatically so you don't have to press the button to fix the focus.
Thanks

@codeamo You can do exactly same function call but on componentDidMount.

@dsznajder since I am using react hooks, I have done the same things with useEffect but still not working the first render but the second render. any help?
my code is here:
`
const focusOTP = () => {
otpRef.current.focus();
};
useEffect(() => {
focusOTP();
}, []);
<OtpInputs
ref={otpRef}
keyboardType="number-pad"
handleChange={num => setCode({ code: num })}
numberOfInputs={4}
autoFocus
/>
`

Remove autoFocus prop, and give it a try

Yep, that works, Does the package provide a call back function that fires after the input fields are full?
Thanks

@codeamo No, you can do this through handleChange function.


[Bug] [Test] K8sTagTest can't pass on M1

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
OAP server (apache/skywalking)
What happened
Zulu 1.8 on aarm
ScriptEngineManager providers.next(): javax.script.ScriptEngineFactory: Provider jdk.nashorn.api.scripting.NashornScriptEngineFactory not a subtype
ScriptEngineManager providers.next(): javax.script.ScriptEngineFactory: Provider jdk.nashorn.api.scripting.NashornScriptEngineFactory not a subtype

Zulu 11 on aarm
java.lang.NoClassDefFoundError: Could not initialize class org.apache.skywalking.library.kubernetes.KubernetesServices$MockitoMock$54094179

What you expected to happen
The test could pass on M1
How to reproduce
Run it on M1. We may consider don't mock enum through powermock. As this is the only one test using it, and only it fails.
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Shall we consider deprecating Powermock and completely moving to Mockito

We should consider, but we don't have people working on this. And this is the first issue we met.

@xzyJavaX works in our team. I suppose he can make an investigation for the possible migration

Thanks, that would be great.

So this looks to be m1-specific as I tried Zulu 1.8, Zulu 11, and Temurin 1.8 Temurin 11 etc., all passed.
One thing to note is that I can see the same logs (not even error logs) on Zulu 1.8 but all tests passed still.

I guess so too. This test is definitely passed because CI/e2e had verified it.

I investigated this issue, and explained the reason here, https://github.com/apache/skywalking/pull/9945


Problem with UILabel text wrapping the view's intrinsic width in an expression.

I'm trying to create a UILabel that will size itself automatically according to the following rules:

When the text length is short, the UILabel width should match the intrinsic content size. In other words, the label should grow or shrink to wrap the intrinsic content size.
When the width of the label exceeds a certain threshold, it should stop growing in width and wrap to another line so that it never exceeds the width threshold.

In order to accomplish this, I have defined my view as follows:
<UIViewController>
    <UILabel
        backgroundColor="#CCC"
        left="24"
        top="24"
        width="min(auto, 320)"
        numberOfLines="0"
        text="This text is to verify line wrapping when using an expression that contains the auto keyword."
    />
</UIViewController>

As can be seen in the width expression, I would like the width to be the smaller of the auto width or 320. When I reduce the text to just a few words, this works as expected. However, when the text is too long, as in the supplied example, the label doesn't grow beyond 320 as desired but it doesn't wrap the text. It truncates it instead and remains at 1 line.
Conversely, if I change my width expression to something like:
width="min(1000, 320)"
Leaving everything else the same, the label wraps properly with a width at 320.
Therefore, there seems to be a problem with causing the label to wrap text when using the auto keyword in the width expression.

@Dragonspell99 it looks like a bug - I'll investigate.
In the meantime, it works if you use min(auto, 100%), so you can get the same effect by wrapping the label in a fixed-width container view:
<UIView
    left="24"
    top="24"
    width="320"
    height="auto">
    <UILabel
        backgroundColor="#CCC"
        width="min(auto, 100%)"
        numberOfLines="0"
        text="This text is to verify line wrapping when using an expression that contains the auto keyword."
    />
</UIView>

@nicklockwood Thanks for the prompt reply!
I've tested your suggestion and it works great. Unfortunately, I'm having trouble applying it to my desired layout. I simplified the code example above to focus on the problem but in my real layout, I already have the UILabel wrapped by a UIView whose width depends on its children. Therefore, I can't set it to a fixed width and trying to reference it otherwise in the child's width expression will result in a circular reference.
What I'm trying to achieve is a layout where I'm showing a chat bubble next to a user's profile picture. The bubble would be a rounded rectangle (wrapping view) with a grey background that contains the user's name (top label) followed by the message text. The bubble has to grow to wrap both the user name and message.
I'm including a larger piece of the layout code below to see if an easy fix jumps out at you. Otherwise, I can try to fix it by measuring the string in code and passing the measured width as a parameter to the layout to be used in the expression.
    <UIView
        backgroundColor="#EFF1F3"
        top="SPACING"
        left="previous.right + SPACING"
        height="auto + PADDING"
        width="auto + PADDING">

        <UILabel
            left="PADDING"
            top="PADDING"
            width="auto"
            font="medium 15"
            text="User Name"
        />

        <UILabel
            left="PADDING"
            top="previous.bottom + SPACING"
            width="min(auto, 320)"
            numberOfLines="0"
            text="This text is to verify line wrapping when using an expression that contains the auto keyword."
        />
    </UIView>

P.S.> Great job on the library! I've been looking for something like Layout for a long time and stumbled upon it by change the other day.

@Dragonspell99 as far as I can see, there's no way to do this with the current release. I've pushed a fix to the develop branch that should make it work as you've written it above. Can you try it out and see if it solves your use-case?

@Dragonspell99 great. I'll do some more testing and assuming everything's OK with the other test projects I'll push a release with those changes asap.

@Dragonspell99 I've not worked out why this is happening yet, but replacing right="PADDING" with left="parent.width - width - PADDING" in the UIImageView seems to fix it.
To be clear, these should be equivalent, so this is definitely a Layout bug, but the workaround should allow you to proceed for now.

Ok. This works fine for my scenario. Thanks!


Searching

Need to add functionality for searching for songs by ordered by different criterion.

I will be including the hibernate-search library to accomplish this task.


Futures-util dev dependency should require alloc feature

We had the following CI failure on Tokio's run-hyper-tests job:
error[E0425]: cannot find function `join_all` in module `future`
   --> tests/support/mod.rs:477:26
    |
477 |         Box::pin(future::join_all(client_futures).map(|_| ()))
    |                          ^^^^^^^^ not found in `future`
    |
note: found an item that was configured out
   --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/futures-util-0.3.30/src/future/mod.rs:81:5
    |
81  | mod join_all;
    |     ^^^^^^^^
    = note: the item is gated behind the `alloc` feature
note: found an item that was configured out
   --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/futures-util-0.3.30/src/future/mod.rs:83:26
    |
83  | pub use self::join_all::{join_all, JoinAll};
    |                          ^^^^^^^^
    = note: the item is gated behind the `alloc` feature

It seems like the futures-util crate has an alloc feature that should be enabled.
Most likely, it was previously enabled by some other dependency that no longer enables alloc.

Interesting, none of the recent commits seem related. Probably a dependency no longer enables it.

Tokio's CI will currently use the newest commit that has a tag on it (as a proxy for latest release). Can we add a tag on this commit?

Sure, I push a tag with a name discouraging explicitly relying on it.


Unable to see sent messages in inbox.

I tried to find a way to view both sent and received messages in Inbox, but I see only received messages.
And directly visiting site.com/messages instead of site.com/messages/inbox is giving 404 error.
How do I view both sent and received messages.

You will need the latest version (1.2.0) but you can add a url like:
url(r"^messages/inbox/deleted/", InboxView.as_view(),
        {'deleted': True}, name="inbox_deleted"),

That will expose the deleted/sent messages the same way you list your inbox.

hey, any way to inform the user when he receives a message, using signals ??


Issue 91

feat(settings): App configuration in individual files (issue 91)
The initial ShinyProxySpecMainProvider was able to load App settings from application.yml.
The new version will delegate the loading task to ShinyProxySpecMainProvider and ShinyProxySpecExternalProvider
ShinyProxyspecMainProvider does the same then the initial ShinyProxySpecMainProvider, ie. it loads settings from application.yml.
ShinyProxySpecExternalProvider will try to load files from a specific path. This path can be set in application.yml using the new proxy.specs-external-path attribute.
The attribute used in the individual settings files are quite the same (using camel case) :
- id, displayName, description, containerCmd, logoUrl, accessGroups, ...

Would this allow for adding / removing / changing deployed applications without restarting the server?


Document setup requirements for functional testing on Chef::Resource::DscResource and Chef::Resource::DscScript on Windows 10

Summary
All of the DscResource and DscScript functional tests fail on my Windows 10 Enterprise machine.  We should solve these and document the settings somewhere to allow contributors to safely contribute:
Chef::Resource::DscResource
  when Powershell supports Invoke-DscResource
    with an invalid dsc resource
      raises an exception if the resource is not found
    with a valid dsc resource
      converges the resource if it is not converged (FAILED - 1)
      does not converge the resource if it is already converged (FAILED - 2)

Chef::Resource::DscScript
  when supplying configuration through the configuration attribute
    behaves like a dsc_script resource with specified PowerShell configuration code
      should create a registry key with a specific registry value and data (FAILED - 3)
      it should behave like a dsc_script resource with configuration affected by cwd
        when the DSC configuration contains code that raises an exception if cwd has a specific value
          should not raise an exception if the cwd is not etc (FAILED - 4)
          should raise an exception if the cwd is etc (FAILED - 5)
  when supplying configuration using the path attribute
    behaves like a dsc_script resource with specified PowerShell configuration code
      should create a registry key with a specific registry value and data (FAILED - 6)
      it should behave like a dsc_script resource with configuration affected by cwd
        when the DSC configuration contains code that raises an exception if cwd has a specific value
          should not raise an exception if the cwd is not etc (FAILED - 7)
          should raise an exception if the cwd is etc (FAILED - 8)
  when running a configuration that manages users
    behaves like a dsc_script with configuration data
      behaves like a dsc_script with configuration data set via an attribute
        should run a configuration script that creates a user (FAILED - 9)
      behaves like a dsc_script with configuration data set via an attribute
        should run a configuration script that creates a user (FAILED - 10)
    behaves like a dsc_script with configuration data that uses environment variables
      does not directly contain the user name (FAILED - 11)
      behaves like a dsc_script with configuration data
        behaves like a dsc_script with configuration data set via an attribute
          should run a configuration script that creates a user (FAILED - 12)
        behaves like a dsc_script with configuration data set via an attribute
          should run a configuration script that creates a user (FAILED - 13)
    behaves like a dsc_script with configuration data that takes parameters
      does not directly contain the user name (FAILED - 14)
      behaves like a dsc_script with configuration data
        behaves like a dsc_script with configuration data set via an attribute
          should run a configuration script that creates a user (FAILED - 15)
        behaves like a dsc_script with configuration data set via an attribute
          should run a configuration script that creates a user (FAILED - 16)
    behaves like a dsc_script without configuration data that takes parameters
      when configuration data is not specified
        should have a default value of nil for the configuration_data attribute (FAILED - 17)
        should have a default value of nil for the configuration_data_path attribute (FAILED - 18)
        should set a registry key according to parameters passed to the configuration (FAILED- 19)
  when using ps_credential
    allows the use of ps_credential (FAILED - 20)

The majority of these seem to be because WinRM is disabled on a 'stock' Windows 10 machine.  Example failure:
15) Chef::Resource::DscScript when running a configuration that manages users behaves like a
 dsc_script with configuration data that takes parameters behaves like a dsc_script with confi
guration data behaves like a dsc_script with configuration data set via an attribute should ru
n a configuration script that creates a user
      Failure/Error: powershell_out!(winrm_code)

      Mixlib::ShellOut::ShellCommandFailed:
        Expected process to exit with [0], but received '1'
        ---- Begin output of powershell.exe -NoLogo -NonInteractive -NoProfile -ExecutionPolic
y Unrestricted -InputFormat None -Command "      winrm get winrm/config/listener?Address=*+Tra
nsport=HTTP
              if($LASTEXITCODE -ne 0) {
                winrm create winrm/config/Listener?Address=*+Transport=HTTP
              }
        " ----
        STDOUT:
        STDERR: WSManFault
            Message = The client cannot connect to the destination specified in the request. V
erify that the service on the destination is running and is accepting requests. Consult the lo
gs and documentation for the WS-Management service running on the destination, most commonly I
IS or WinRM. If the destination is the WinRM service, run the following command on the destina
tion to analyze and configure the WinRM service: "winrm quickconfig".

        Error number:  -2144108526 0x80338012
        The client cannot connect to the destination specified in the request. Verify that the
 service on the destination is running and is accepting requests. Consult the logs and documen
tation for the WS-Management service running on the destination, most commonly IIS or WinRM. I
f the destination is the WinRM service, run the following command on the destination to analyz
e and configure the WinRM service: "winrm quickconfig".
        WSManFault
            Message = The client cannot connect to the destination specified in the request. V
erify that the service on the destination is running and is accepting requests. Consult the lo
gs and documentation for the WS-Management service running on the destination, most commonly I
IS or WinRM. If the destination is the WinRM service, run the following command on the destina
tion to analyze and configure the WinRM service: "winrm quickconfig".

        Error number:  -2144108526 0x80338012
        The client cannot connect to the destination specified in the request. Verify that the
 service on the destination is running and is accepting requests. Consult the logs and documen
tation for the WS-Management service running on the destination, most commonly IIS or WinRM. I
f the destination is the WinRM service, run the following command on the destination to analyz
e and configure the WinRM service: "winrm quickconfig".
        ---- End output of powershell.exe -NoLogo -NonInteractive -NoProfile -ExecutionPolicy
Unrestricted -InputFormat None -Command "      winrm get winrm/config/listener?Address=*+Trans
port=HTTP
              if($LASTEXITCODE -ne 0) {
                winrm create winrm/config/Listener?Address=*+Transport=HTTP
              }
        " ----
        Ran powershell.exe -NoLogo -NonInteractive -NoProfile -ExecutionPolicy Unrestricted -I
nputFormat None -Command "      winrm get winrm/config/listener?Address=*+Transport=HTTP
              if($LASTEXITCODE -ne 0) {
                winrm create winrm/config/Listener?Address=*+Transport=HTTP
              }
        " returned 1
      Shared Example Group: "a dsc_script with configuration data set via an attribute" called from ./spec/functional/resource/dsc_script_spec.rb:324
      Shared Example Group: "a dsc_script with configuration data" called from ./spec/functional/resource/dsc_script_spec.rb:357
      Shared Example Group: "a dsc_script with configuration data that takes parameters" called from ./spec/functional/resource/dsc_script_spec.rb:393
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/mixlib-shellout-2.3.2-universal-mingw32/lib/mixlib/shellout.rb:293:in `invalid'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/mixlib-shellout-2.3.2-universal-mingw32/lib/mixlib/shellout.rb:280:in `error!'      # ./lib/chef/mixin/powershell_out.rb:48:in `powershell_out!'
      # ./spec/functional/resource/dsc_script_spec.rb:37:in `block (2 levels) in <top (required)>'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/hooks.rb:350:in `instance_exec'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/hooks.rb:350:in `run'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/hooks.rb:509:in `block in run_owned_hooks_for'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/hooks.rb:508:in `each'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/hooks.rb:508:in `run_owned_hooks_for'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/hooks.rb:460:in `run'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/example_group.rb:537:in `block in run_before_context_hooks'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/memoized_helpers.rb:186:in `block in isolate_for_context_hook'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/memoized_helpers.rb:182:in `instance_exec'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/memoized_helpers.rb:182:in `isolate_for_context_hook'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/example_group.rb:536:in `run_before_context_hooks'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/example_group.rb:589:in `run'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:118:in `block (3 levels) in run_specs'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:118:in `map'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:118:in `block (2 levels) in run_specs'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/configuration.rb:1926:in `with_suite_hooks'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:113:in `block in run_specs'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/reporter.rb:79:in `report'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:112:in `run_specs'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:87:in `run'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:71:in `run'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/lib/rspec/core/runner.rb:45:in `invoke'
      # C:/Ruby25-x64/lib/ruby/gems/2.5.0/gems/rspec-core-3.7.1/exe/rspec:4:in `<top (required)>'
      # C:/Ruby25-x64/bin/rspec:23:in `load'
      # C:/Ruby25-x64/bin/rspec:23:in `<main>'

Acceptable fix
Document the required settings on Windows 10 to execute these functional tests correctly.
Related items
See https://github.com/chef/chef/issues/3106

There are no outstanding issues running the Functional Tests on Windows 10.  Just need to ensure WinRM is enabled (for most people it will be already but Enable-PSRemoting -SkipProfileCheck -Force will do it if not.


Add description of iso code to documents

I think that documents should show clearly what kind of value to use for iso of options.
So, I've add description of iso code to documents as comment of source code.

Thank you @munierujp!