Update Pick to use new key remapping in mapped types

Search Terms
pick omit key remapping
Suggestion
Update Pick in lib.d.ts to be:
type Pick<T, K extends keyof T> = {
    [P in keyof T as Extract<P, K>]: T[P];
};

It is currently defined as:
type Pick<T, K extends keyof T> = {
    [P in K]: T[P];
};

Use Cases
With this new definition, typescript can statically analyze the keys, which allows interfaces to extends these types.
Examples
interface Base {
    a: number,
    b: string,
    c: boolean,
}

// Errors: "An interface can only extend an object type or intersection of object types with statically known members"
interface PickBase0<K extends keyof Base> extends Pick<Base, K> {
    pickedKeys: K
}

type Pick1<T, K extends keyof T> = { [L in keyof T as Extract<L, K>]: T[L] }

// Works
interface PickBase1<K extends keyof Base> extends Pick1<Base, K> {
    pickedKeys: K
}

Playground Link
Checklist
My suggestion meets these guidelines:

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[X] This feature would agree with the rest of TypeScript's Design Goals.

I'm curious what would happen if we did that

@RyanCavanaugh I ran the tests with the updated definition; 17 failed (11 conformance and 6 compiler). Looking at the baselines, I was able to distill one problem with the new definition to:
type Pick<T, K extends keyof T> = { [P in keyof T as Extract<P, K>]: T[P] };
type Assert<T, U extends T> = U;
type X<T, K extends keyof T> = Assert<Pick<T, K>, T>;
//                                                ^
// Errors, "Type 'T' does not satisfy the constraint 'Pick<T, K>'"

I haven't looked through all of the baselines in detail to see if there are errors beyond that, though I suspect there are.
Perhaps Pick should be left as-is, and people who want interfaces involving it can define their own. Maybe add a PickInterface?

Also, what's the reasoning behind error 2312 (an interface can only extend an object type or intersection of object types with statically known members)? With the new as clause, many types that would appear to not have statically known members are allowed (like below). If types like these can have seemingly dynamic members, why can't interfaces extend the more general format of { [K in X]: Y }?
type Record1<K extends keyof any, V> = { [_K in "" as K]: V }
interface RecordInterface<K extends keyof any, V> extends Record1<K, V> { }

For one thing I noticed this fixes issues with Omit or Pick + intersection types in some cases.  I've seen it in various cases, but here's a playground example.
You can see that passing an omit type into a remapped type causes an issue, but this new way of doing it fixes the issue. I wish I understood why, but perhaps someone could shed some light on it.

@bfricka The problem in that scenario boils down to Omit's definition:
type Omit<T, K> = Pick<T, Exclude<keyof T, K>>

Because A has an index signature, that becomes Pick<T, string>. Homomorphic mapped types, on the other hand, have special casing for index signatures that maps them correctly.


Syntax highlighting

Is your feature request related to a problem? Please describe.
Code snippets are monotonous and visually difficult to read.

Describe the solution you'd like
Is it possible to apply syntax highlighting like INITTO?

Describe alternatives you've considered
None
Additional context
None

Hi @aggre I would like to work on this :)


Ds 3264 fix groupname export

I have no idea whether I'm doing the right thing for the test -- I'm happy to remove that class/commit from this PR if required.

The original reporter of the issue said on dspace-tech that this PR appears to fix the problem they encountered: https://groups.google.com/d/msg/dspace-tech/fwYVvXhV3ik/HjK8XWKOBAAJ

+1 tested and it works.  I coincidentally ran into this same bug today myself (by simply exporting a Collection AIP and trying to reimport it). I applied this PR, and it fixes the issue of the bad group names.

This issue is annoyingly getting in the way of my ability to test or debug other issues related to AIP export/import (e.g. #1396).  I.e. this bug fix is needed on master to allow me to proceed in testing other AIP related bug fixes.
As this PR has been verified by two people (one in the email thread, and myself), was created by a Committer and comes with its own Unit Tests, I'm going to merge this immediately. Additional fixes/changes can come in additional PRs (as needed).

+1 by inspection.


Add functionality to show new tip daily

Get the number of days passed since project creation.
Use it as index for tips by getting the reminder of it with tips length
to ensure it doesn’t get out of range .

I'm going to hold off on reviewing this for now until I figure out a couple more functionality pieces. I'm thinking of switching the simple JSON format of the tips to full-blown markdown/html files for each tip so that we can support more advanced tips that need a little more detail/thoroughness. I'm currently playing around with switching over to HTML imports that would effect this functionality (importing a different timestamped HTML file that gets swapped out each day).

I think switching to html files will be better yes and will help in giving the tip content more freedom.


Spyder does not launch after Anaconda Navigator upgrade

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[x] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
After the latest anaconda update to Anaconda Navigator 2.4.2, spyder did not launch anymore. Selecting spyder from the Anaconda Navigator home-screen results in a pop-up window:

Application launch error | Application spyder launch may have produced errors. Exit code:<PHONE_NUMBER>

I uninstalled Anaconda according to the documentation and installed the previously running version from the archive. The error remains and was not resolved. Creating a new spyder env didn't work. Starting spyder from CMD does not launch spyder and also does not show the error window from the Anaconda Navigator.
All other packages from Anaconda Navigator (JupyterLab, ...) run smoothly, before and after fresh installation.
Since the fresh install of Anaconda Navigator I cannot update anymore:
(base) C:\Users\[username]>conda update anaconda

PackageNotInstalledError: Package is not installed in prefix.
  prefix: C:\tools\Anaconda3
  package name: anaconda

What steps reproduce the problem?
I could not reproduce the error with another PC running on Win10.
Versions

Spyder version: 5.4.1
Python version: 3.10.9
Qt version: 5.15.2
PyQt version: 5.15.7
Operating System name/version: Windows 11 Pro Version 10.0.22621 Build 22621

Hi Carlos, thanks for reaching out!If I run the command in the anaconda prompt, basically nothing happens. After a few seconds the command line skips back to base, but spyder does not show up, nor an error message appears. Cheers On 12 Jul 2023 18:38, Carlos Cordoba @.***> wrote:
Hey @nibor10, thanks for reporting. Please open the Anaconda Prompt, then run
spyder
and post here any errors you see in the terminal after doing that. Thanks!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Did you use pip to install or update Spyder in your base environment?

I used conda update anaconda regularly and avoid pip installations. Spyder reminded me of updates in the past, but I don't force to install new versions and wait for the implementation with the general conda update command.

I used conda update anaconda regularly and avoid pip installations

Ok, that's really good.
Let's create a new environment for Spyder then to see if that allows you to start it. For that, open the Anaconda Prompt again and run these commands:
conda create -n spyder-env -c conda-forge spyder
conda activate spyder-env
spyder

Hopefully that will solve your problem.

Ok, then please run these commands to see if we can get more information about this problem:
conda activate spyder-env
spyder --debug-info verbose

After that, please post here the output (if any) printed to the terminal.

The prompt successfully switches to the spyder enviroment, but the debug-info does not print anything.

I encounter the same problem. All the steps I found so far didn't work and I don't get any error messages or debug info instead of the error code from anaconda. Is there any solution found yet?

@nibor10, do you still have the same problem?

Yes, I do.

@ccordoba12 I updated everything with conda update anaconda and now spyder starts again.
Thank you very much for your time!


Missing 'inline' keyword

Hi @rogerbarton rogerbarton,
I am trying your solution and have some warning and exception, do you have any thought about this?

Exception：
  File "/opt/homebrew/lib/python3.9/site-packages/sphinx_csharp/csharp.py", line 237, in parse_type_signature
    match = CLASS_SIG_RE.match(sig.strip())
AttributeError: 'NoneType' object has no attribute 'strip'

Originally posted by @Feng-Seng in https://github.com/michaeljones/breathe/issues/550#issuecomment-808704764

The inline keyword is missing from the list of modifiers. I will add this quickly...

HaHa, thank you sooooooo much!!

It's been added, run pip install git+https://github.com/rogerbarton/sphinx-csharp.git to get the latest version and try it again

Thanks, it's getting better.
But I still got 2 warnings and 1 exception as follows:
WARNING: Type signature invalid, got List< string >)
WARNING: Variable signature invalid: NSBone
Exception：
File "/opt/homebrew/lib/python3.9/site-packages/sphinx_csharp/csharp.py", line 237, in parse_type_signature
match = CLASS_SIG_RE.match(sig.strip())
AttributeError: 'NoneType' object has no attribute 'strip'
Any thoughts about these?

The first warning seems to be because of the brackets (.. , ...) in the original warning:
WARNING: Method signature invalid: static inline Dictionary< ESkinPart,(List< string >, List< string >)> initAssetMap2 ()

I assume this is syntax for a tuple or something similar. Parsing this as a type has not been implemented. I will have a look at this...
For the second warning, do you know which line this is coming from? maybe search your code for NSBone?

I guess for now you could write Tuple<List< string >, List< string >> for this one function as a workaround. This might also fix the other warning

For the second warning, I am not sure which line induces this warning since the NSBone has been used in several places in 2 cs files as follows.
First .cs file.
namespace xx.xxxxx
{
//no-scale-bone
public struct NSBone
{
public NSBone(Vector3 v, Quaternion q)
{
localPos = v;
localRot = q;
}
public Vector3 localPos;
public Quaternion localRot;
}
internal class NSPair
{
public NSPair(Vector3 v, Quaternion q)
{
_original.localPos = v;
_original.localRot = q;
}
public NSBone _original;
public NSBone _lastSaved;
}
public class Data
{
.......
(NSBone, NSBone) NSBonePair(Vector3 v, Quaternion q)
{
return (new NSBone(v, q), new NSBone(v, q));
}
Another .cs file
namespace xx.xxxxx
{
......
public bool GetBindingPose(string name, out NSBone localTransform)
......
}
Could those information sufficient for you?

For the first warning, I will try what you said, thanks!!

This is probably from the line:
(NSBone, NSBone) NSBonePair(Vector3 v, Quaternion q) {...}

So the same issue with the tuple short hand type but for the return type of the function.

I've had a look at implementing the C# 7.0 tuples, but this seems to be quite complicated. So it will not be supported for now.

Hi @rogerbarton ,
That's no problem.
But could you print the file / line causes the warning / error in the warning / error message?
For example, which xml (or CS) file cause this warning / error.

Another question is, do you know why the warning causes the exception? In usual case, only the errors could stop the execution.
Is there any way to get the result, even there are warning message?

Yes, its because a None is propagated when the regex does not match. I'm currently working on cleaning things up and producing better errors...

I should have probably checked this earlier, but doxygen does not yet support these C# 7.0 tuples, see https://github.com/doxygen/doxygen/issues/7633. So the input for breathe is garbage.

But could you print the file / line causes the warning / error in the warning / error message?
For example, which xml (or CS) file cause this warning / error.

I have no idea how to get this information, so for now I've just set it to point to the rst file/line.

inline is not a keyword in C#. I think breathe should not add it in signature.
https://github.com/michaeljones/breathe/pull/668

Thanks @xuhongxu96! Coming from C++ I simply assumed that C# would also have it. I will revert this.


compiling h2o project with Nuitka

Hi,
i open this issue because i have problems to compile an H2O project with Nuitka.
https://www.h2o.ai/
H2O is a framework to do machine learning and artificial intelligence model. It works in a simply mode, after installing H2O in python env, we need to write 2 rows to use it:
import h2o
h2o.init(ip="localhost", port=54321, max_mem_size="8G", nthreads=-1)

With the second line, a cluster H2O is started, or if already started, it connect to him. The cluster (1 node in local pc) is the central element that permit to prepare data and build models. how? there's a jar file that is used (h2o.jar) that is searched in a python path.
Now, i tried in several ways to build a test project with this 2 lines. Here the code:
import h2o

def talk(message):
    return "Talk " + message

def main():
    print( talk("Hello World"))

if __name__ == "__main__":
    h2o.init(ip="localhost", port=54321, max_mem_size="8G", nthreads=-1)
    main()

When i finish to compile, and tried to start my executable, the h2o.jar file that is used to start the cluster, is not included. I tried to manually create the folder and moved this file in the path when it's searched, but then the problem is about the H2O version of jar file that mismatch. While the H2O.jar is version <IP_ADDRESS> (in python env first build), in the compiled project, the version that is registered is 0.0.local.
Someone had experience with this problem?
I tried command line like --include-package=h2o
-- follow-imports
--follow-imports --recurse-all
-- include plugin-directory
Follow now my step that i did, to reproduce the problem:
python version:
Python 3.6.9
I created a virtualenv:
virtualenv -p /usr/bin/python3 nuitka-venv
pip install h2o
pip install pandas
pip install -U nuitka
Now, opening a python shell, this 2 lines works
import h2o
h2o.init(ip="localhost", port=54321, max_mem_size="8G", nthreads=-1)

but when inserted in hello.py and compiled, something of h2o is not included in the compiled version.
Other info:
python -m nuitka --version
0.6.7
Python 3.6.9
Executable /home/valerio/Scrivania/test/nuitka-venv/bin/python
OS: Linux
Arch: x86_64
My ubuntu is 18.04.4 LTS
any kind of help is appreciated, thanks!

I think this is a effect of the known issue, that we do not include these version information files that pip puts into the file system. There is an issue where the plan is to collect basically the version information of all, and then to include it readily for query at run time.
Until then, you will have to work around this by modifying said code, filenames, etc.

The version stuff it seems no longer is an issue, and @KevinRodriguez777 produced a data files config that makes it work out of the box now. I added it with crediting to him.
Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback if this is working is very welcome, just please do not share plans of doing it, but rather confirmations or denials of it working.

Part of the 2.0.3 hotfix that I just made.


[Feature Request]: Remove dependency of KafkaIO withDynamicRead on beam_fn_api

What happened?
The withDynamicRead method was gated with the beam_fn_api experimental flag in https://issues.apache.org/jira/browse/BEAM-11946 && https://github.com/apache/beam/pull/14168.
From reading the ticket it sounds like the intention was to ensure that we read from Kafka with an SDF when using x-lang features.
Im wondering if this is still necessary today because reading from Kafka via SDF is the default behaviour and there are a few exceptional cases where it falls back to an unbounded source .
heres a link to a previous PR that removed beam_fn_api from the rest of KafkaIO but kept it for dynamic reads https://github.com/apache/beam/pull/14419
cc: @kennknowles you might have some context on if this is still required ?
Issue Priority
Priority: 3 (minor)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

I believe we still need this, because we cannot fall back to the legacy implementation if the user needs dynamic reads

The way this should work is that the transform should expand for dynamic reads and the runner can say yes/no to whether it can execute it. We should definitely eliminate the beam_fn_api experiment entirely. It is really old and was meant for running stuff in a nonstandard way before it was really ready for production. TBH we probably never needed it IMO.


Unified account - Support save contact for TON address

Describe the task
Support save contact for TON address

Version test : https://drive.google.com/file/d/16RX4l9iQB0tOQcPVCaRVkQ0489lIQlMt/view?usp=sharing

[ ] Don't show avatar for TON  address on Manage address book screen

[ ] Don't show avatar for recent address on Address book screen


FEATURE: custom logger control

Tests should follow the best practice of using $A.log when logging in the client.
It's possible to configure a client logger that handles these log events (like a log4j appender) and writes a filtered set out to the console.
$A.logger.subscribe("INFO", function(level, message, error) {
console.log(error);
});
A dropdown menu can be added to the status bar which controls this logger so that it's easy to change the log level without changing the code. It should cause a page refresh with a new url parameter which sets the new log level i.e. the logging level attributes are loaded from parameters

This suggests that a custom appender is no longer possible!
http://salesforce.stackexchange.com/questions/129894/lockerservice-not-allowing-subscribing-to-a-logger


Windows 11 Crashes Just opening a project/create node

Tested versions
4.3 stable
System information
Windows 11
Issue description
Constant errors in console showing the following:
Godot Engine v4.3.stable.official.77dcf97d8 - https://godotengine.org
OpenGL API 3.3.0 NVIDIA 560.94 - Compatibility - Using Device: NVIDIA - NVIDIA GeForce RTX 3080

Editing project: C:/Users/Joshua/Documents/tutorial
Godot Engine v4.3.stable.official.77dcf97d8 - https://godotengine.org
Vulkan 1.3.280 - Forward+ - Using Device #0: NVIDIA - NVIDIA GeForce RTX 3080

ERROR: Condition "err != VK_SUCCESS && err != VK_SUBOPTIMAL_KHR" is true. Returning: FAILED
   at: command_queue_execute_and_present (drivers/vulkan/rendering_device_driver_vulkan.cpp:2344)
ERROR: Condition "err != VK_SUCCESS && err != VK_SUBOPTIMAL_KHR" is true. Returning: FAILED
   at: command_queue_execute_and_present (drivers/vulkan/rendering_device_driver_vulkan.cpp:2344)
ERROR: Condition "err != VK_SUCCESS && err != VK_SUBOPTIMAL_KHR" is true. Returning: FAILED
   at: command_queue_execute_and_present (drivers/vulkan/rendering_device_driver_vulkan.cpp:2344)
ERROR: Condition "err != VK_SUCCESS && err != VK_SUBOPTIMAL_KHR" is true. Returning: FAILED
   at: command_queue_execute_and_present (drivers/vulkan/rendering_device_driver_vulkan.cpp:2344)

Steps to reproduce
Open a project, click + Add Node
Minimal reproduction project (MRP)
A brand new empty project

Creating a project with the Compatibility renderer works, but Vulkun should be supported on my RTX3080? Latest drivers etc.

Please provide your system specs, unsure why this would happen, but more details is needed

People have had their Vulkan support break e.g. by uninstalling stuff that injects their own Vulkan layers, e.g. Epic Games

Fwiw I'm having this same issue but for me its intermittent and just started happening.

AMD Ryzen 7 7800X3D
64GB of ram
RTX 3080
I do have Unreal installed and Unreal engine... plus a lot of games etc, other than that, I only use my windows PC for gaming really... Does uninstalling UE help at all?
I attempted to install the Vulkun SDK and it installs correctly, but if I install just the Vulkun runtime only, it fails will access permission problems to what looks like existing Vulkun files on my system

Uninstalled UE and same issue, still unable to install the Vulkun Runtime also

what other details can I supply?

You can use the Vulkan SDK to find out which layers might be affecting Godot:
https://github.com/godotengine/godot/issues/96563#issuecomment-2330917057

People have had their Vulkan support break e.g. by uninstalling stuff that injects their own Vulkan layers, e.g. Epic Games

https://github.com/godotengine/godot/issues/81670#issuecomment-1720270768:

Do you have any third-party Vulkan layers installed, such as RTSS, Epic Games Launcher/Rockstar Games Launcher, or software that injects itself into processes in general?
I think Vulkan Configurator (which is part of the Vulkan SDK) can display a list of those.

You might have to clean out the registry to remove layers (I had to do so for MedalTV layer)
Also try: Enabling validation layers

ERROR: Condition "err != VK_SUCCESS && err != VK_SUBOPTIMAL_KHR" is true. Returning: FAILED
   at: command_queue_execute_and_present (drivers/vulkan/rendering_device_driver_vulkan.cpp:2344)

This error from your console comes from command_queue_execute_and_present [1]
err is the result of a call to device_functions.QueuePresentKHR  == vkQueuePresentKHR [2] [3]
According to the vulkan docs [4]: vkQueuePresentKHR returns one of these error values:

On success, this command returns:

VK_SUCCESS
VK_SUBOPTIMAL_KHR

On failure, this command returns:

VK_ERROR_OUT_OF_HOST_MEMORY
VK_ERROR_OUT_OF_DEVICE_MEMORY
VK_ERROR_DEVICE_LOST
VK_ERROR_OUT_OF_DATE_KHR
VK_ERROR_SURFACE_LOST_KHR
VK_ERROR_FULL_SCREEN_EXCLUSIVE_MODE_LOST_EXT

Note: VK_ERROR_OUT_OF_DATE_KHR is already handled in command_queue_execute_and_present.
So the error has to be one of these:

VK_ERROR_OUT_OF_HOST_MEMORY
VK_ERROR_OUT_OF_DEVICE_MEMORY
VK_ERROR_DEVICE_LOST
VK_ERROR_SURFACE_LOST_KHR
VK_ERROR_FULL_SCREEN_EXCLUSIVE_MODE_LOST_EXT

AFAIK, the only way to find out which one it is, would be to build the engine from source with debug symbols
I'm also running into this error when using wgpu<Vulkan> for tensor operations via GdExtension. Unfortunately, I'd need to build the engine to troubleshoot.
@joshualawson Your case might be a different err from mine. This error message might not be the issue at all for you because my engine/game doesn't crash from this error (I just happen to get this error alot)
Try uninstalled any unneeded layers before building from source.
Sources:

https://github.com/godotengine/godot/blob/77dcf97d82cbfe4e4615475fa52ca03da645dbd8/drivers/vulkan/rendering_device_driver_vulkan.cpp#L2344
https://github.com/godotengine/godot/blob/77dcf97d82cbfe4e4615475fa52ca03da645dbd8/drivers/vulkan/rendering_device_driver_vulkan.cpp#L2312
https://github.com/godotengine/godot/blob/77dcf97d82cbfe4e4615475fa52ca03da645dbd8/drivers/vulkan/rendering_device_driver_vulkan.cpp#L987
Vulkan vkQueuePresentKHR docs: https://registry.khronos.org/vulkan/specs/1.3-extensions/man/html/vkQueuePresentKHR.html


Use the tool to style the tool itself

Could be interesting to have the full GUI styled using the tool itself (ie. using fibonacci spacers).
Check what's missing, since most of the guidelines are already supported.

temporary closed.. could be needed more work (this issue maybe moved to the next release)

Temporary closed: more work could be needed (this issue maybe moved to the next release).


Bump Dependencies #2852407404

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/2852407404

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

closing because we merge the latest dependency changes with https://github.com/wso2/product-is/pull/14829


Allow using not defined class name as type hint to set injection

Laravel container use these code to  get the type hint
	/**
	 * Get the type hint for this closure's first argument.
	 *
	 * @param  \Closure  $callback
	 * @return mixed
	 */
	protected function getFunctionHint(Closure $callback)
	{
		$function = new ReflectionFunction($callback);
		if ($function->getNumberOfParameters() == 0)
		{
			return;
		}
		$expected = $function->getParameters()[0];
		if ( ! $expected->getClass())
		{
			return;
		}
		return $expected->getClass()->name;
	}

But it will cause a fatal error if the class required by the  type hint is not defined. Sometimes,i just want to bind some simple name with my services to service container and use type hint to setter injection .But i can't do that.Can you provide some background on the need for this code?Maybe you can use these code that post on http://php.net/manual/en/reflectionparameter.getclass.php to get type hint.
function getClassName(ReflectionParameter $param) {
    preg_match('/\[\s\<\w+?>\s([\w]+)/s', $param->__toString(), $matches);
    return isset($matches[1]) ? $matches[1] : null;
}

Any ideas?

I'm pretty sure that's a PHP limitation. The class or interface in the typehint needs to be defined.

@franzliedke It can be done by registering an alias on the fly, but Taylor has rejected it - and for good reason.


Refactor common functionality of AnalogConv layers

Description and motivation
The analog convolution layers (AnalogConv1d, AnalogConv2d and AnalogConv3d) share quite a good chunk of functionality. As we now have enough of them, it would be a good time to generalize their shared pieces, in order to avoid code repetition.
Proposed solution
The digital Conv classes already share a common ancestor in PyTorch with some interesting reusing. It can serve as inspiration or taken into account for coming up with the generalization.
Alternatives and other information

Closing as after #165 and others the base class is quite complete already - the recalculate_indexes functions that remain are probably safer to keep at the leaf classes, as the effort for refactoring them into a single base function does not seem to provide enough benefits.


SyntaxError: invalid syntax -- baseTime = min(time * (1L << retries), cap);

python 3.6.2
importing
from snakebite.client import Client

results in
  File "<input>", line 1, in <module>
    from snakebite.client import Client
  File "/***********/lib/python3.6/site-packages/snakebite/client.py", line 1473
    baseTime = min(time * (1L << retries), cap);
                            ^
SyntaxError: invalid syntax

It is the same issue as in https://github.com/spotify/snakebite/issues/216
Providing a Python 3 package is misleading, and should be removed from pypi if does not work.

Just ran into this as well :(

Just ran into this as well :(
Please fix it, thanks.

this is not fixed! someone remove it or solve the issue

@rationalbeing @fifar The author of this library no longer works at Spotify and Spotify no longer use this library. See https://github.com/spotify/snakebite/issues/62
At this point, Spotify do not care enough to even close this ticket. Could people please stop adding comments to this ticket. Instead, head over to https://pypi.org/project/snakebite-py3/ and get the version that works on Python 3

Thanks OOPMan.
I closing this issue for this dead repo.

la solution est simple:
pip install snakebite-py3
ou
pip3 install snakebite-py3


[Feature request] Ability to overwrite the older cookie file if it exists while exporting & also the ability to change default path for downloading when clicking "export".

The saved cookie file name normally follows the renaming scheme of the OS when it encounters an older file with the same name.
The requested feature would make it possible to overwrite the older file while exporting, instead of using the "Export as" function and renaming the cookie file to the original file and then choosing to overwrite it manually every-time.
The second feature would allow the user to change the path used by the "Export" option to one chosen by the user, instead of the browser's download folder.

Is this technically possible?
“Export” is the browser's ability to download files, and ‘Export as’ is the ability to get an access handle to a specific file path via the File System API and ‘save as’.
For security reasons, user UI operation is required to obtain an access handle.
To achieve export to an arbitrary default directory, it would be necessary to maintain an access handle to the directory.
This could be stored in IndexedDB, for example, but that is not something the extension should touch. Also, it would be host specific.
Can the handle be stored in the chrome.storage API? I don't know.
I don't know if this would be implemented in this extension, but if anyone knows of a smart way to do it, please let me know.

Inside the downloads folder should be manipulable, right..?
No. It only allows downloading. Browser APIs don't allow simple filesystem manipulation for security reasons.

However, I found that I could access indexeddb in popup context.
This allows me to store a FileSystemFileHandle, which might allow me to implement the proposed functionality.
But due to our busy schedule, please forgive us that implementation will not be immediate.


Remove unused dependencies and upgrade remaining dependencies.

Coverage remained the same at 92.673% when pulling 84d3ae093435cc240f3f0bc9504a4e88c3cf3ddb on cleanup-deps into de0044ec74396f8505892a5336e8eb61c52c6846 on master.

Coverage remained the same at 92.673% when pulling 84d3ae093435cc240f3f0bc9504a4e88c3cf3ddb on cleanup-deps into de0044ec74396f8505892a5336e8eb61c52c6846 on master.

Coverage remained the same at 92.673% when pulling 84d3ae093435cc240f3f0bc9504a4e88c3cf3ddb on cleanup-deps into de0044ec74396f8505892a5336e8eb61c52c6846 on master.

Coverage remained the same at 92.673% when pulling a315791648d16437ab0207b2d7dd62ee9a08a912 on cleanup-deps into de0044ec74396f8505892a5336e8eb61c52c6846 on master.


migrate to ostree unified core

Is your feature request related to a problem? Please describe.
ostree is moving to support unified core as the one true way to generate ostrees. We should migrate to that to ensure we consume the path most used as it makes support of the ostree components more straight forward.
Describe the solution you'd like
User ostree unified core to build the FIoT ostrees
Describe alternatives you've considered
Remain on the current path but less groups are using the code paths so we may end up with other issues.
Additional context
Need to add various upstream trackers

https://pagure.io/fedora-iot/pungi-iot/c/8ec296f9f91bd4d133808244b1f33aa594c21e00?branch=main


A11y_Visual Studio Code Web Extensions_Github Pull request_Create Pull request_DeveloperTool: Name is not provided for combo box in 'Merge changes from' section.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
 This bug should only be closed by TCS, C+AI Accessibility tester after verification of the fix
Open https://insiders.vscode.dev/github/microsoft/vscode-pull-request-github environment and login with git hub account.
tab to 'Github Pull request' and press enter.
tab to 'Create Pull request' and press enter.
Check name provide for combo box is correct or not.

Actual:
Name is not provided for combo box in 'Merge changes from' section.
Similar issue observed in 'INTO' Section on same screen.
Expected:
'Path' name should be provided for combo box in 'Merge changes from' section.
User Impact:
Screen reader users will not get complete information of box.
Attachment:

The issue is still reproing in https://insiders.vscode.dev/github/microsoft/vscode-pull-request-github this environment

The issue is fixed in environment.https://insiders.vscode.dev/github/microsoft/vscode-pull-request-github hence closing bug


REQUEST: Unity3D Plugin

Hello,
I've built an app in Unity and I'd love to use this library as a Unity asset. I was wondering if I could go about it by making a JAR and using that as a plugin with Unity?
https://docs.unity3d.com/Manual/AndroidJARPlugins.html
If not, what else would you recommend? I have a simple VNC implementation that's the c# vnc library, but the performance is horrible, would much rather use this.
Thanks!

Just following up:
https://docs.unity3d.com/Manual/AndroidAARPlugins.html
That might even be better

I think it would be better to use TigerVNC directly.
ovrvnc uses TigerVNC as a VNC library.  Although it is not officially released as a library, its backend code is well-separated from the frontend.
ovrvnc/src/vnc_thread.hpp might be helpful.

Thanks! I will try using their java implementation directly


MH-12607 (Harvard DCE), Lossless Concat Operation

This is an updated version of pull request #14 which @phoebem updated here. During my review of #14, I noticed a few issues for which I provided some patches which are included here.
To make the review easier, this pull request contains only the lossless concat parts of #14.

As discussed with @lkiesow and @gregorydlogan, I'm taking over this review.

@staubesv, This should fix both issues you raised.
Please wait with the merge though. I like to rebase this to ensure the new Markdown linter does not complain. I'll make that update later today. That should not keep you from testing though ;-)

Rebased this to the head of develop to include all of the updated style and formatting requirements and tests. Let's wait for the build to finish though I hopefully got all of them.


setValue, setSelectedKey, setSelectedKeys restore parameter(default value) inconsistency

OpenUI5 version: 1.54.3
Browser/version (+device/version): MacOS Chrome
ComboBox.setSelectedKey restore parameter is an empty string "" or undefined.
Input.setValue restore parameter is empty/undefined
MultiComboBox.setSelectedKeys restore parameter is null or undefined
When I call setSelectedKeys(""), will cause empty string be added to the selected keys. I think this behavior is very confusing:

Hello @TinaC,
As far as I understood the issue, that you are trying to report, is in Input, ComboBox and MultiComboBox controls, am I right?
Could you please make a jsbin example of the exact problem for each ot them?
BR,
Tereza

[plnkr example]http://plnkr.co/edit/ryQtRxrlxGCjcRUfaezG?p=preview
I think:

doc is not accurate, both Input.setValue and 'ComboBox.setSelectedKey' can use empty string, null and undefined to restore
the behavior of MultiComBox.setSelectedKeys("") will add an empty string to the array of selected keys, which is very error-prone

Hello @TinaC ,
Thank you for sharing this findings.

The sap.m.Input and sap.m.ComboBox are behaving propperly and as the documentation says.
I've created an internal incident<PHONE_NUMBER>, via which the responsible colleagues for the sap.m.MultiComboBox will check if this behaviour is incorrect. The status of the issue will be updated here in GitHub.

Regards,
Tereza

Hello @TinaC ,
The sap.m.MultiComboBox.setSelectedKeys accepts an array of keys. If you pass a string, it will be converted to an array with first element this string. If you want to restore the selectedKeys value, please use undefined or null, which are valid array values. This behavior is documented in the API Reference https://sapui5.hana.ondemand.com/#/api/sap.m.MultiComboBox/methods/setSelectedKeys.
The sap.m.ComboBox and sap.m.Input accept a string as a selectedKey, and not an array, that's why the behavior of the three mentioned controls is different.


Do I follow JavaScript best practices?

#Checking for Best practices.
The code below can be improved to have DRY concept

https://github.com/yosaddis/todo-list/blob/b7e5bc0f3dbe5699f7e5b053af82c29c072cf788/src/index.js#L8-L13
This function can be improved to have arrow function.
https://github.com/yosaddis/todo-list/blob/b7e5bc0f3dbe5699f7e5b053af82c29c072cf788/src/modules/Tasks.js#L26

Issues are resolved.


dashboards: Make the regex much more specific

To make sure that we're not being too greedy with the regex
replacement of the data source to use for each dashboard
that's uploaded, we make it much more specific using the
following:

literal boundaries for " on either side of the match
non-capturing optional group matches for the ${} bits
the case-sensitive literal match for DS_ which is
always in every exported dashboard representing the
data source used
a one-or-more case-sensitive match for the part that
follows the underscore, with no characters allowed
except alphabetical

This should prevent mistaking any capitalised labels or
anything else which may be present in the dashboard.
This regex can be tested and understood better by looking at the
matches and non-matches in https://regex101.com/r/f4Gkvg/2
versus the current regex and same data set shown in
https://regex101.com/r/Ad9f5U/1

Please write a comment before task or a better task name explaining what those hieroglyphs do. I fear we might forget and this part will become unmaintainable.

:) will do

dahsboards such as these:
https://grafana.com/dashboards/2263
https://grafana.com/dashboards/1234
use a datasource placeholder which does not fit the pattern expected by the regex (in this case DS_NDF_APP ).
Should this be reconsidered?

this would fix it: "(?:${)?DS(_([A-Z])+)+(?:})?" by allowing one or more words delimited by underscores

Ah, I wasn't aware that additional underscores were an option. Could you push up a PR and add that example into the regex tester reference (and update it)? Alternatively, let me know and I'll push up a patch with the changes.

PR: https://github.com/cloudalchemy/ansible-grafana/pull/53
Updated regex: https://regex101.com/r/f4Gkvg/4
:)

Excellent, thanks!


EMPTY CONTENT FOR INDEX IN FRONTEND

Hi. I'm having trouble installing WLOX Exchange Software as when I tried accessing the index.php file of the frontend, it gives empty content. If anyone could help me I'm very grateful. I'm using uBuntu 17.10 with PHP version 5.6 and MySQL 5.0.11

Hello, you probably have not yet set up the system on the administrative side, try to do this and come back with the novelty.

Hi. where am I supposed to set up the system? can you guide me through it? thank you in advance :)

go to the wlox / backstage2 directory
so login with admin / admin and then see the documentation on the page of this project for more configuration information.

@Guillerbr I also have difficulty to set it up. Do you mind we could connect ? My email is xingwl0401 At gmail. Thanks!

@Guillerbr hi thanks for your offer! Here is my email<EMAIL_ADDRESS>:)


support both : and = as compose envvar separators

fix #196 #173

could you check it? @surajssd

But there is one think that this PR should include, and that is test for it.

Yes. I will add follow-up commit

@ngtuna I have problem building this branch, can you guys tell me what am I missing?
$ git remote show tuna
* remote tuna
  Fetch URL: https://github.com/ngtuna/kompose
  Push  URL: https://github.com/ngtuna/kompose
  HEAD branch: master
  Remote branches:
    clean-code        tracked
    envvar_colon      tracked
    global-flag       tracked
[SNIP]
    userguide         tracked
  Local branches configured for 'git pull':
    envvar_colon    merges with remote envvar_colon
    project-context merges with remote project-context
    update-objects  merges with remote update-objects
  Local refs configured for 'git push':
    envvar_colon    pushes to envvar_colon    (up to date)
    master          pushes to master          (fast-forwardable)
    project-context pushes to project-context (up to date)
    update-objects  pushes to update-objects  (up to date)

$ git fetch tuna

$ git checkout envvar_colon
Switched to branch 'envvar_colon'
Your branch is up-to-date with 'tuna/envvar_colon'.

$ git pull
Already up-to-date.

$ build-kompose
+ cd /home/hummer/go/src/github.com/kubernetes-incubator/kompose
+ go build -tags experimental -o kompose ./cli/main/
# github.com/kubernetes-incubator/kompose/pkg/loader/bundle
pkg/loader/bundle/bundle.go:95: cannot use p (type "github.com/kubernetes-incubator/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol) as type "github.com/skippbox/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol in field value
# github.com/kubernetes-incubator/kompose/pkg/loader/compose
pkg/loader/compose/compose.go:101: cannot use proto (type "github.com/kubernetes-incubator/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol) as type "github.com/skippbox/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol in field value
pkg/loader/compose/compose.go:110: cannot use proto (type "github.com/kubernetes-incubator/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol) as type "github.com/skippbox/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol in field value
# github.com/kubernetes-incubator/kompose/pkg/transformer/kubernetes
pkg/transformer/kubernetes/k8sutils.go:248: cannot use service (type "github.com/skippbox/kompose/pkg/kobject".ServiceConfig) as type "github.com/kubernetes-incubator/kompose/pkg/kobject".ServiceConfig in argument to transformer.ConfigAnnotations
pkg/transformer/kubernetes/k8sutils.go:274: cannot use service (type "github.com/skippbox/kompose/pkg/kobject".ServiceConfig) as type "github.com/kubernetes-incubator/kompose/pkg/kobject".ServiceConfig in argument to transformer.ConfigAnnotations
pkg/transformer/kubernetes/kubernetes.go:188: cannot use port.Protocol (type "github.com/skippbox/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol) as type "github.com/kubernetes-incubator/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol in field value
pkg/transformer/kubernetes/kubernetes.go:207: cannot use port.Protocol (type "github.com/skippbox/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol) as type "github.com/kubernetes-incubator/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol in field value
# github.com/skippbox/kompose/pkg/transformer/kubernetes
../../skippbox/kompose/pkg/transformer/kubernetes/k8sutils.go:248: cannot use service (type "github.com/kubernetes-incubator/kompose/pkg/kobject".ServiceConfig) as type "github.com/skippbox/kompose/pkg/kobject".ServiceConfig in argument to transformer.ConfigAnnotations
../../skippbox/kompose/pkg/transformer/kubernetes/k8sutils.go:274: cannot use service (type "github.com/kubernetes-incubator/kompose/pkg/kobject".ServiceConfig) as type "github.com/skippbox/kompose/pkg/kobject".ServiceConfig in argument to transformer.ConfigAnnotations
../../skippbox/kompose/pkg/transformer/kubernetes/kubernetes.go:188: cannot use port.Protocol (type "github.com/kubernetes-incubator/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol) as type "github.com/skippbox/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol in field value
../../skippbox/kompose/pkg/transformer/kubernetes/kubernetes.go:207: cannot use port.Protocol (type "github.com/kubernetes-incubator/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol) as type "github.com/skippbox/kompose/vendor/k8s.io/kubernetes/pkg/api".Protocol in field value

@surajssd this is because @ngtuna's branch is not yet rebased to current master. It still uses old import path github.com/skippbox/kompose

oh okay

Hey @kadel @surajssd I made rebase and added test. Please take a look.

@ngtuna I tried pulling and building it and it LGTM

merging


PolymerElement $ accessor return null on 1.0.0-rc.2

@PolymerRegister('elemnt-test')
class ElementTest extends PolymerElement {
ElementTest .created() : super.created();
ready() {
var test1 = this.$; // test1 is null
var test2 = $['some-element-id']; // throw exc
}
}

Hmm, this definitely works in general (see test here, so something else is probably going on. Maybe @PolymerRegister('elemnt-test') is supposed to be @PolymerRegister('element-test') so its not finding the dom-module tag (or maybe thats just a type in the issue). If that is not it can you please post a full example?

also - this will only exist in polymer.dart, not in polymer_mini.dart or polymer_micro.dart (see feature list here).

Thanks) It is problem on my side.
Was in html - <dom-module id=="element-test">

Ah yep that makes sense. Unfortunately since a dom-module is not required we can't really add a check in to make sure that the tag name in PolymerRegister matches a dom-module either :(. Maybe the $ getter could warn if it returns null though?

I think, that some error message on start are enough.

Unfortunately we would have to perform this check inside the $ getter :/. At startup we don't know whether or a polymer-element is expecting to have a matching dom-module.

In this case a warning in $ getter will be good)


some comments about xenon tutorial

I started reviewing the tutorial
Here are things that came to my mind, regarding first 13 pages.

lines in original latex file are super long, it's impossible to read them on github, I would have to read locally and change settings in my editor to wrap lines, latex ignores line breaks, so why not break lines at 80 characters? this will also make diffing much easier, than finding changes in 1000 character long lines

would be nice to have syntax highlighting in this tutorial, it would be much more readable

page 5:

"The shell should be Bash”
does that mean I need to install Bash if I use some other shell
I use zsh, which is mostly bash compatible, should I switch to bash to be able to use xenon?
cd ${HOME} is much more cryptic (and requires much more typing) than cd ~

page 6:

"therefore it needs Java"
java is a programming language, it does not require programming language (whatever that means) it
requires Java Virtual Machine to run
“Linuxes” -> Linux distributions

page 7:

"./ gradlew dependencies prints a list of the dependencies"
does it really only print? it says BUILD SUCCESSFUL which would suggest it did build something
too

page 11:

only tiny number of people in whole universe know what Lisa cluster and “Cartesius supercomputer`
is, we could say "your local cluster or supercomputer"

page 13:

" is the (public) salt used to encrypt the host name"
if you already know what salt is you can guess what <salt> means without this comment, if you don’t
this comment did not explain anything

This issue belongs to NLeSC/Xenon-examples


Golden Axe split addition

There is a misc. category run where you steal the Golden Axe (https://www.speedrun.com/octopath#Golden_Axe). The end of the run is when the steal is executed in Clearbrook, so this would be a split similar to the 3% steal for Therion's story.

Yeah, this would probably require tracking the NPCs inventory of something of the sort. In terms of structure, maybe we can get some hints from the randomizer developers?

if the golden axe is an item, i know we can semi-reliably scan the current users inventory and we should be able to get an item id for that.
this might be a bit more complicated as the inventory size can grow or shrink over time, so without some scripting or sigscans we might not be able to track this. Would take some time to figure out i think


new protein localization terms

PMID:26258632
can we change
protein localization abolished during vegetative growth to
protein localization abolished during mitotic cell cycle
and
normal protein localization to kinetochore during vegetative growth
to
normal protein localization to kinetochore during mitotic cell cycle
and add a term
protein localization abolished during meiotic cell cycle
to group
abolished protein localization to telomere during meiotic cell cycle
abolished protein localization to prospore membrane during meiosis II with protein mislocalized to plasma membrane
abolished protein localization to kinetochore during meiosis I
(Yoshi didn't drill down here I suspect because he didn't see anything immediately suitable, but he did add kinetochore localization as a comment)
NTR
abolished localization to kinetochore during mitotic metaphase
also
decreased protein localization to mitotic spindle pole body during mitotic anaphase
normal protein localization to kinetochore during 	mitotic prometaphase

can we change
protein localization abolished during vegetative growth to
protein localization abolished during mitotic cell cycle
and [snip other one] ?

I vote no. All* of the terms that apply to the vegetative growth phase, and aren't directly about cell cycle regulation or progression, already have "during mitotic cell cycle" synonyms, and I don't see what would be gained by swapping (it would be a tedious lot of work to do them all, but I especially see no point in changing a select couple of terms, thereby becoming inconsistent).
*except for a handful of stragglers that I'm just fixing now
New terms all fine; watch this space.

abolished protein localization during meiotic cell cycle FYPO:0005208
abnormal protein localization to kinetochore FYPO:0005209
abolished protein localization to kinetochore during mitotic metaphase FYPO:0005210
decreased protein localization to mitotic spindle pole body during anaphase FYPO:0005211
normal protein localization to kinetochore during mitotic prometaphase FYPO:0005512

edit file: 8bb855d981e006750a0fed14cfb4fd3a85a1c59f
release: 8bb855d981e006750a0fed14cfb4fd3a85a1c59f


package works great but how would one render a route with its parent layout route?

It works great.
Im still using this with v1 routes and also works fine "almost".
Before I used this file structure below ( which resulted in the app/routes/__auth.tsx, to be the parent layout for the login )
app/routes/__auth/login.tsx
app/routes/__auth.tsx

with your package I tried:
app/core/auth/routes/__auth.tsx
app/core/auth/routes/__auth/login.tsx

It renders the login route but without the parent layout, is there a way to have this package inherit the parent layout somehow?
Ofcourse your library is really new and i probably approach it the wrong way, but is there a neat way to resolve this?
I plan to move to v2 routes perhaps I should just jump ship now.
With v2 routes enables i guess it will be more easy something like
/app/core/auth/routes/auth.tsx
/app/core/auth/routes/auth.login.tsx

Im still using this with v1 routes and also works fine "almost"

At the moment I'm not looking at putting time into getting this to work with the v1 path format, though if someone were ambitious enough to implement that I would absolutely include it here. It'd take modification of the getRouteIds function to not strip out the parent folders. I think that would work without having to modify getManifest too.

Is there a way I could let it work for /login also using the __auth.tsx parent route for the layout?

Yep, with the v2 style routes you use a single _ to make them pathless
app/
  anyFolder/
    _auth.route.tsx # layout file for your auth pages
    _auth.login.route.tsx
    _auth.logout.route.tsx


Regression in copying jsonb rows

In DBeaver 3.7.4, I was able to select and copy a single field from one row to a different row, and then apply that change. The type of the column is jsonb in a Postgres database.
In later versions (3.8.1 is the latest I tried, but I encountered this regression before too), this no longer works. Pasting still works and the field shows up as orange, but when I click the green “save” checkmark, I get an error message:

SQL Error [42804]: ERROR: column "foo" is of type jsonb but expression is of type character varying
Hint: You will need to rewrite or cast the expression.
Position: 46
org.postgresql.util.PSQLException: ERROR: column "foo" is of type jsonb but expression is of type character varying
Hint: You will need to rewrite or cast the expression.
Position: 46

Thanks for report

Wow, that was a quick fix, great!

This error is present in version <IP_ADDRESS>204021640

This error is present in version of <IP_ADDRESS>212251901

Error present in version Version <IP_ADDRESS>406231636


Move PGManager functions to their own abstraction.

Codecov Report

Merging #4 (bcef229) into main (f8cf455) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #4   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           20        22    +2
=========================================
+ Hits            20        22    +2

Impacted Files
Coverage Δ

src/lib/homeobject.cpp
100.00% <100.00%> (ø)

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


about exporting to disk

Currently stackedit has features to export content to disk via styled html, but the html has external href like stackedit.io/style.css and some fonts, and when I use these contents offline, the style disappear.
what about add a export option to export a self-contained html, which has embed styles without fonts change.

That'd be up to you.


make yp_constant_id_ts useful for constant pool indexing

This turns out to be really useful (#1537, #1533, #1449, etc.), and we can make everything smaller while doing so.

Performance/memory impact?

Merged this in #1605


With no parameters, it should print out an error message like "required parameters foo and bar not specified"

Fixed by PR #2396


Move focus to editor for new file created from workspace tree view

Currently I have to click inside the editor to begin coding.  It would be great if this was automatic.

We can re-open this if anyone is willing to submit a PR.


Some new pages being ignored

I don't know why, but some pages are ignored and don't produce a discord message.
I've tried but haven't been able to pinpoint what causes it. As an example, the content of this page doesn't generate a notification; making new pages with this text always gets ignored.

I just copied this content to my test wiki and it worked. Weird.

You seem though to have some extensions / such installed which might cause problems?
What MediaWiki version do you have?
Does the error happen when creating a page or when editing content of the page?
Have you tried editing the extension's PHP file to view if it actually goes into the hook?

Slapped something like this into the *Core.php
534     private static function sendCurlRequest( $url, $postData ) {
535         $h = curl_init();
536         curl_setopt( $h, CURLOPT_URL, $url );
537         curl_setopt( $h, CURLOPT_POST, 1 );
538         curl_setopt( $h, CURLOPT_POSTFIELDS, $postData );
539         curl_setopt( $h, CURLOPT_RETURNTRANSFER, true );
540         curl_setopt( $h, CURLOPT_CONNECTTIMEOUT, 10 ); // Set 10 second timeout to connection
541         curl_setopt( $h, CURLOPT_TIMEOUT, 10 ); // Set global 10 second timeout to handle all data
542         curl_setopt( $h, CURLOPT_HTTPHEADER, array(
543             'Content-Type: application/json',
544             'Content-Length: ' . strlen($postData)
545             )
546         ); // Set Content-Type to application/json
547         // Commented out lines below. Using default curl settings for host and peer verification.
548         //curl_setopt ($h, CURLOPT_SSL_VERIFYHOST, 0);
549         //curl_setopt ($h, CURLOPT_SSL_VERIFYPEER, 0);
550         // ... Aaand execute the curl script!
551         $curl_output = curl_exec( $h );
552         curl_close( $h );
553         error_log("request sent\n", 3, "/tmp/php.log");
554     }
555
556     private static function sendHttpRequest( $url, $postData ) {
557         $extradata = [
558             'http' => [
559             'header'  => "Content-type: application/json",
560             'method'  => 'POST',
561             'content' => $postData,
562             ],
563         ];
564         $context = stream_context_create( $extradata );
565         $result = file_get_contents( $url, false, $context );
566         error_log("http sent\n", 3, "/tmp/php.log");
567         error_log("http sent2\n", 3, "/tmp/php.log");
568         error_log("http sent3\n", 3, "/tmp/php.log");
569         error_log($postData, 3, "/tmp/php.log");
570     }
571

The result was as follows:

I was trying out stuff with the wiki admin and found out that if I don't fill out the summary box before posting, the discord message doesn't go through. But if I do add one, it works:

Test4 page didn't get sent to discord, but test5 page did.

Oh. Does this happen with other pages too if you do not fill out the summary box?

I think I have found the combination that makes the extension ignore posts:

I spent quite a bit trying out different things and discovered that for a page to be ignored, it has to have {{: }} code, which our wiki uses to fetch single chapter pages into a complete volume page (the ones we're having problems with) and being posted with no summary.
In the example above, both pages have only one line {{:Magisterus_Bad_Trip:Volume3_Afterword2|Afterword}}, with test9 having no summary and test10 having one.

Oh. Interesting. Unfortunately I cannot replicate this problem, so I am giving you some help here to replicate what the issue might be.
If you print the $result back from curl request in PHP code, what does it contain? We need to figure out if the error happens in the PHP code or in Discord API (so Discord API for some reason denies the message because it contains some special characters or something like this)

Ok, the gist of that test is as follows:
Printing $curl_output in sendCurlRequest results in {"embeds": ["0"]}#
Printing $result in sendHttpRequest results in nothing.

A possible workaround: Append $wgUseAutomaticEditSummaries = false; to your LocalSettings.php.
But It seems that all edit summaries include template syntax make the substitutions happen. In this situation, the summary with substitution was too long to request, and PHP Warning: file_get_contents(): failed to open stream: HTTP request failed! HTTP/1.1 400 Bad Request was occurred.
The document says "{{-transformation is done." with Message::text(),[1] therefore some(or all?) calls to Message::text() should be replaced with calls to Message::plain().

Great! :)


cmd/go: get -u is updates much more in module mode than GOPATH more

At the moment go get -u -d . in module mode updates all dependencies of the main module, and as of https://golang.org/cl/128136, go get -u -d $PKG for any package in the main module will do the same.
The current documentation is ambiguous on this point. It says:

The -u flag instructs get to update dependencies to use newer minor or patch releases when available. Continuing the previous example, 'go get -u A' will use the latest A with B v1.3.1 (not B v1.2.3).
[…]
When using -m, each specified package path must be a module path as well, not the import path of a package below the module root.
[…]
With no package arguments, 'go get' applies to the main module, and to the Go package in the current directory, if any. In particular, 'go get -u' and 'go get -u=patch' update all the dependencies of the main module.

The documentation is ambiguous because it does not clarify which “dependencies” go get updates

“package dependencies”, the set of modules containing the transitive closure of imports of a package, or
“module dependencies” the set of modules that are required by the module containing the package, or
something else, such as the union of the package and module dependencies.

My intuition is that, without the -m flag, get -u should update the package dependencies: the user is requesting a specific package, not (necessarily) a module, and because go.mod files are currently very sparse, any given package may have a large number of package dependencies that are not module dependencies. We should not fail to update those dependencies, so we should not upgrade only “module dependencies”. On the other hand, “the union of package and module dependencies” seems more complicated to explain.
If I understand @rsc correctly, (in https://golang.org/cl/128555), he has some other alternative in mind, although I'm not sure which.

As a :+1: I am currently very confused as to how to properly do updates to dependencies. For instance, let's say I specify package a <v1.2 and have some other packages: b, c, and d whose versions I don't care about. I want to be able to run go get -u (or something similar) and have it update just a,b,c, and d while respecting my constraint a <v1.2 such that if I was at a v1.1.1 it could now be a v1.1.2 but not a v1.2.3. Can't figure out how to do this now.

We'll look more carefully at this in Go 1.12.

thanks!

A similar ambiguity exists for go mod download without an explicit argument: should it download source code for all modules in the module graph, or only those required to be able to go test all in all build configurations? (I would argue that the latter is more user-friendly, since the general use case for go mod download is to run some subset of go test all.)

An observation: currently, if I run go get -u (without arguments) in github.com/bcmills/goissues, I get upgrades to a rather large set of transitive dependencies.
go mod tidy removes most of those upgrades, because they aren't even remotely relevant to the build.
So one nice property for go get -u to have would be preservation of tideness: if the module is already tidy, then go get -u should not make it untidy.

In GOPATH mode get -u only applied to the transitive dependencies of a set of packages. In module mode it is always the transitive dependencies of the module graph. It's possible that we should preserve a "just update the deps of this target set".


Development

Added a formal pip freeze for requirements.
Added halfbakedharvest to list of sites for debugging

added merge requirements for main branch protection, and now I don't know how to mark my own code as reviewed. lol


Fix/predetermine annotation assembly chunks

To address issue 58 and the difficulty of debugging checkpointed rules, I added a precalculation step so we know how many ncontig chunks the assembly is getting split into, rather then doing it on the fly.
I also decreased the wall-time for the metaerg runs, and captured the antismash logs

This addresses https://github.com/vdblab/vdblab-shotgun/issues/59


[AutoPR cognitiveservices/data-plane/ComputerVision] Add Spanish as an option as ServiceLanguage for ComputerVision

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3222

(message created by the CI based on PR content)

Codecov Report

Merging #2731 into restapi_auto_cognitiveservices/data-plane/ComputerVision will increase coverage by 0.66%.
The diff coverage is n/a.

@@                                     Coverage Diff                                      @@
##           restapi_auto_cognitiveservices/data-plane/ComputerVision    #2731      +/-   ##
============================================================================================
+ Coverage                                                     55.54%   56.21%   +0.66%
============================================================================================
  Files                                                          7175     6419     -756
  Lines                                                        155273   139763   -15510
============================================================================================
- Hits                                                          86254    78567    -7687
+ Misses                                                        69019    61196    -7823

Impacted Files
Coverage Δ

...ure-mgmt-sql/azure/mgmt/sql/models/resource_py3.py
55.55% <0%> (-44.45%)
:arrow_down:

...-sql/azure/mgmt/sql/models/tracked_resource_py3.py
62.5% <0%> (-37.5%)
:arrow_down:

...mt-sql/azure/mgmt/sql/models/proxy_resource_py3.py
83.33% <0%> (-16.67%)
:arrow_down:

...mt/logic/models/integration_account_certificate.py
45.45% <0%> (-11.69%)
:arrow_down:

...eduler/models/job_history_definition_properties.py
38.46% <0%> (-11.54%)
:arrow_down:

...mgmt/logic/models/integration_account_agreement.py
33.33% <0%> (-11.12%)
:arrow_down:

...ure/mgmt/logic/models/edifact_protocol_settings.py
31.25% <0%> (-10.86%)
:arrow_down:

...gic/azure/mgmt/logic/models/integration_account.py
62.5% <0%> (-10.23%)
:arrow_down:

...gic/azure/mgmt/logic/models/recurrence_schedule.py
44.44% <0%> (-10.11%)
:arrow_down:

...mgmt-logic/azure/mgmt/logic/models/content_link.py
44.44% <0%> (-10.11%)
:arrow_down:

... and 1192 more

Continue to review full report at Codecov.

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

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2614


SageMaker notebook instance doesn't work with ipywebrtc extension

This extension doesn't work on SageMaker https://github.com/maartenbreddels/ipywebrtc
I suspect this could be due to how the proxy is setup:
I get this error in the inspector:
The resource from “https://XXXXXXXX.notebook.us-east-2.sagemaker.aws/static/jupyter-webrtc.js?v=20191125191846” was blocked due to MIME type (“text/html”) mismatch (X-Content-Type-Options: nosniff
Any idea how to make it work?
pip install ipywebrtc
from ipywebrtc import CameraStream, ImageRecorder
camera = CameraStream(constraints=
                      {'facing_mode': 'user',
                       'audio': False,
                       'video': { 'width': 640, 'height': 480 }
                       })
camera

Is this still an issue for you?


Mast: Watch (?-i:^VIKKY[\W_]*+OMKAR$)

Mast requests the watch of the watch_keyword (?-i:^VIKKY[\W_]*+OMKAR$). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

(?-i:^VIKKY[\W_]*+OMKAR$) has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Daniil in Charcoal HQ


Bank plugins not working

After the last Runelite update i've noticed some issues with plugins regarding the bank. First the bank tags: The box is checked but there aren't any tabs to the left and when right-clicking an item the option to add tags is also gone.
In the Bank plugin only "keyboard bankpin" works out of the 5 plugins i'm using. Show GE/HA/Seed prices and bank value don't work.
The icon for depositing worn items is also a bit weird.
Tried enabling/disabling, restarting the client but it didn't do anything.

Duplicate of #11585


Cleanup theme

#120 – Everything done except not loading google fonts when the user specifies a different font.

Nice! I wonder if it's smart to use getFontSize in places where font size actually doesn't matter (i.e. most paddings/margins). The only place where this is related is in the logo section where we want to align the bottom of the logo/title with the page title.


Cannot access beta.kodadot.xyz

lol someone merged something wrong and won't properly tested, logs look ok

https://64143e062470790007bda86d--koda-nuxt.netlify.app/
this one works merged by https://github.com/kodadot/nft-gallery/commit/19f8df386c7b0509169e726bd5dc65e2daf15ef2
later one doesn't, which I don't see reason it's doesn't builds..

https://github.com/kodadot/nft-gallery/commit/9c5963226fe8ccdc783820db1412c9e658b3a5a9

Suspect some Netlify glitch as #5290 builts well and works https://deploy-preview-5290--koda-nuxt.netlify.app/

let's watch logs if it's builts and try in 5m https://app.netlify.com/sites/koda-nuxt/deploys/6414573299990f0a9daabd01

Might be suspect on generate 👀
https://app.netlify.com/sites/koda-nuxt/deploys/6414573299990f0a9daabd01#L622
Um, it's working

nice :D


Simplify Generic Views with Django Signals

Original Post by @joki in https://git.osgeo.org/gitea/GDI-RP/MrMap/issues/503#issuecomment-9154
status quo
I currently refactoring the views in #493 to match the django best practice and using generic views. While i'm working on the UpdateView for activating/deactivating services there comes another usefull idea in this context.
I implement the following AsyncUpdateView class:
class AsyncUpdateView(UpdateView):
    template_name = 'generic_views/generic_update.html'
    action = ""
    action_url = ""
    alert_msg = ""
    async_task_func = None
    async_task_params = {}

    def get_context_data(self, **kwargs):
        context = super().get_context_data(**kwargs)
        context.update({"action": self.action,
                        "action_url": self.action_url})
        return context

    def form_invalid(self, form):
        content = render_to_string(template_name=self.template_name,
                                   context=self.get_context_data(form=form),
                                   request=self.request)
        response = {
            "data": content
        }
        return JsonResponse(status=400, data=response)

    def form_valid(self, form):
        self.object.save()

        task = self.async_task_func.delay(object_id=self.object.id,
                                          additional_params=self.async_task_params)

        content = {
            "task": {
                "id": task.task_id,
                "alert": Alert(msg=self.alert_msg, alert_type=AlertEnum.SUCCESS).render()
            },
        }

        # cause this is a async task which can take longer we response with 'accept' status
        return JsonResponse(status=202, data=content)

If the form is invalid the form is returned with errors in a json data field. If it is valid it saves the object and starts the async job. The returning is the task with an alert message.
Idea
Maybe it is also possible to use django signal in this context again, to start a async task on post_save of a metadata object if the is_active field becomes changed. If so, we must refactor the returned json for form_valid.
@jansule The AsyncUpdateView refactors the way we using modals. The complete modal is fetched by javascript and the submitting is also handled in javascript. What do you think about the idea to fire signals on post_save event?

commented by @jansule on issue
Do I understand correctly that you want to use AsyncUpdateView on any arbitrary model? And by coupling the async_task to the model, you are able to reuse this view more easily, as the async task might differ or might not be used at all between different models?
If so, then following comes to my mind:

Do we have a model, where we ALWAYS want to call an async_task on certain save-conditions?

If yes, then I would say that this concept should work fine.
If not, I would think about another way of coupling the task with the view. As far as I understand, we have to inherit from the AsyncUpdateView for each model anyways, in order to pass the respective model to the view.
We could then use method decorators to add the async_task where needed, like so (simplified example):
class AsyncUpdateView(UpdateView):
    # ...
    def form_valid(self, form, task=None):
        self.object.save()

        content = {
            "task": {
                "alert": Alert(msg=self.alert_msg, alert_type=AlertEnum.SUCCESS).render()
            },
        }

        if (task is not None):
            content["task"]["id"]: task.task_id

        # cause this is a async task which can take longer we response with 'accept' status
        return JsonResponse(status=202, data=content)

@method_decorator(async_task_func, 'form_valid')
class ServiceForm(AsyncUpdateView):
    model = Service
    # we do not have to overwrite from_valid() here

Using method_decorators on the views would have the benefit of having a softer coupling between model and async_task.
Please note that the two concepts can be used at the same time, depending on the answer for above mentioned question.
side note
I'm not too familiar on what's happening in the async_task of your example. But shouldn't the isActive property be set to true AFTER the async_task is finished?

Do I understand correctly that you want to use AsyncUpdateView on any arbitrary model?

Yes

And by coupling the async_task to the model, you are able to reuse this view more easily, as the async task might differ or might not be used at all between different models?

Yes that's why i think we could use django signals here. Then it is a normal UpdateView which changes the is_active attribute of the Metadata model in this case.

Do we have a model, where we ALWAYS want to call an async_task on certain save-conditions?

No - we don't need to call an async_task always. So for that we could implement a custom django signal. Example:
class Metadata:
	__original_is_active = None

    def __init__(self, *args, **kwargs):
        super(Metadata, self).__init__(*args, **kwargs)
        self.__original_is_active = self.is_active
    ...

	@receiver(post_save, sender=Metadata)
    def is_active_changed(sender, instance, created, raw, using, update_fields, **kwargs):
    	if 'is_active' in update_fields and instance.is_active != instance.__original_is_active:
        	tasks.async_activate_deactivate_service(instance.id)

As far as I understand, we have to inherit from the AsyncUpdateView for each model anyways, in order to pass the respective model to the view.

IF the example above works, the AsyncUpdateView could be refactored to not call any tasks. I would like to only update the model with is_active field.

We could then use method decorators to add the async_task where needed, like so (simplified example):

Good solution as well to make it more generic

I’m not too familiar on what’s happening in the async_task of your example. But shouldn’t the isActive property be set to true AFTER the async_task is finished?

I think after refactoring with signals we should first save the new is_active value. And the activation task should get the new value from model.
Current implementation: The task get the instance of the model by id from the db if the celery schedule the task. After that all sub elements of the current instance will be traversed to change the is_active attribute to the new value.

commented by @jansule :
I like your idea.
Currently, I see two different cases:

ALWAYS trigger a task when a certain value changes -> hard coupling via django signals on the model
SOMETIMES trigger a task when a certain value changes -> soft coupling via method decorators

This is something we will have to consider for each case. For your example, case 1 seems to be the better approach.
But there might be cases, where a value will be changed at different locations in the code, but a method should not always be triggered.
Let's say we have a value that can be changed by a user in the frontend, but will also be changed in some processing in the backend. We only want to call a method, if the value was directly set by the user.
So as long as we know for a certain case if we want a hard or a soft coupling, we can go with the appropriate pattern.
In any way, we do not have to implement the async task within the AsyncUpdateView, which sounds like a benefit to me.
So +1 on that

I found a nice article with explanations where signals should be live: https://simpleisbetterthancomplex.com/tutorial/2016/07/28/how-to-create-django-signals.html


/snap/microk8s/743/usr/bin/curl: error while loading shared libraries

inspection-report-20190824_205239.tar.gz
Using current stable version 1.15.2
~> sudo microk8s.enable istio
Enabling Istio
Fetching istioctl version v1.2.2.
/snap/microk8s/743/usr/bin/curl: error while loading shared libraries: libcurl-gnutls.so.4: cannot open shared object file: No such file or directory
Failed to enable istio

@affanse sorry for the late response we pushed an update on edge but it seems the fix is not complete yet (at least not for centos).


Import helpers reports errors for __rest

Also fixes a crash in binder when checking spreads in object literals in strict mode.
I may wait to merge this until after @rbuckton finishes the refactoring of verifyHelperSymbol and surroundings. @mhegazy you requested this so you may want to take a look too.

looks good to me. thanks!


Create Custom{Binary,Unary}Expression

PostgreSQL has quite a few custom operators (full text, JSON and many others). Rather than creating expression types for each, this type represents an arbitrary expression with two operands and an operator.
Removed FullTextSearchExpression, switching to these new expression types instead.
@rwasef1830, can you please take a look and say what you think, as this touches code which you've recently merged? One primary motivator for this was the development of EF Core plugins, which need to make use of other PostgreSQL operators. I remembered you added FullTextSearchExpression and thought it would be a good idea to generalize that.
Some specific changes I did on your code:

CustomBinaryExpressions and CustomUnaryExpressions are now constructed directly in the full text translator (previously you had static factory methods on FullTextSearchExpression).
I removed the ValidateArguments step. The logic is that argument validation is performed by the compiler on the source code itself; in other words, the NpgsqlFullTextSearchLinqExtensions.Add() method only accepts two NpgsqlTsQuery parameters, so there's no way we'd find ourselves in the translator with arguments of any other type. This is the way all other method translators work in EF Core - they assume that the object and parameter types are correct.
NpgsqlQuerySqlGenerator previously added parentheses around the SQL expression generated from the FullTextSearchExpression. I've removed this, since it doesn't seem to be necessary - DefaultQuerySqlGenerator surrounds expressions with parentheses according to what other expression they're embedded in (you'll notice other visit methods typically don't add parentheses). Did you encounter any special reason to have the parentheses that maybe I've missed?

@roji It looks good to me.

CustomBinaryExpressions and CustomUnaryExpressions are now constructed directly in the full text translator (previously you had static factory methods on FullTextSearchExpression).

I did that to make it a bit more descriptive and to follow the pattern Microsoft themselves follow (the static factory methods on Expression).

I removed the ValidateArguments step. The logic is that argument validation is performed by the compiler on the source code itself; in other words, the NpgsqlFullTextSearchLinqExtensions.Add() method only accepts two NpgsqlTsQuery parameters, so there's no way we'd find ourselves in the translator with arguments of any other type. This is the way all other method translators work in EF Core - they assume that the object and parameter types are correct.

The goal of it was just an internal sanity check as I was reusing the same class for both unary and binary expressions and to make developer bugs produce descriptive error messages.

NpgsqlQuerySqlGenerator previously added parentheses around the SQL expression generated from the FullTextSearchExpression. I've removed this, since it doesn't seem to be necessary - DefaultQuerySqlGenerator surrounds expressions with parentheses according to what other expression they're embedded in (you'll notice other visit methods typically don't add parentheses). Did you encounter any special reason to have the parentheses that maybe I've missed?

Just an old habit :-) of being defensive and not assuming the caller will do the right thing.
Overall, I think it's OK, and I think this is the right direction to "open up" Npgsql expression parsing for future plugins (separate NuGet packages for specialized functionality perhaps).

Thanks for reviewing @rwasef1830!


⚠ Bofrost.de has degraded performance

In badb92a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 716 ms

Resolved: Bofrost.de performance has improved in f28521f after 6 minutes.


[5.1] Added options to the broadcasting pusher configuration file

Pusher Options Addition
Added the parameter to pass in options from the broadcasting configuration to the Pusher class if needed. This goes in tangent with the PR request for laravel/laravel.

What if that option isn't defined? You need to use array_get or something.


WIP: chore: update tsconfig

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (a20cba0) 91.65% compared to head (5780843) 91.53%.

Files
Patch %
Lines

src/utils/compatible_require.ts
85.71%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #268      +/-   ##
==========================================
- Coverage   91.65%   91.53%   -0.12%
==========================================
  Files          49       49
  Lines        1186     1193       +7
  Branches      214      216       +2
==========================================
+ Hits         1087     1092       +5
- Misses         99      101       +2

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


Cannot focus terminal when text elsewhere on the page is selected

If I select text somewhere else on the page, I cannot focus the terminal by clicking inside it. It works fine after I clear the selection.

Details

Browser and browser version: Chrome Version 59.0.3071.115 (Official Build) (64-bit)
OS version: macOS Sierra 10.12.6
xterm.js version: 2.8.0 (and current master)

Steps to reproduce

npm install
npm start
Open http://<IP_ADDRESS>:3000 in a browser
Select some text on the page outside of the terminal
Try to click inside the terminal

Problem: The terminal cannot be focused.

I guess this happens because we call evt.preventDefault() in a mouse handler somewhere...

Opened #828 to get around with this, before 2.9.0.
This was caused by old code trying to get around with selections on Firefox (#41). Since selection is taken care of by xterm.js now, this code is not needed any more.

Thanks for reporting this @spadgett 😄!


Rewrite OBJECTPROPERTY T-SQL function in C

Description
Currently OBJECTPROPERTY function is implemented in SQL which is found to cause performance issues in the queries. This change will implement the function in C.
Issues Resolved
BABEL-4317
Signed-off-by: Sai Rohan Basa<EMAIL_ADDRESS>Test Scenarios Covered

Use case based -

Boundary conditions -

Arbitrary inputs -

Negative test cases -

Minor version upgrade tests -

Major version upgrade tests -

Performance tests -

Tooling impact -

Client tests -

Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Awaiting Performance improvement numbers before merge


Line comments

After splitting a hunk, it is now possbile to add a comment in between.

@txsmith could you look into this today?


Cannot import Postman collection

When trying to import the attached postman collection I get "Import collection failed". No idea why it failed. I am attaching the said collection, maybe you can provide advice or a fix.
holiday-calendar-api.json
I am using Bruno v1.11.0 on macOS Sonoma 14.3. I installed Bruno with brew install Bruno.
I used the menu action 'Import Collection'/'Postman Collection' and pointed the import to the Postman file attached.

Hi @BigNerd did you export it through Postman directly ? In version 2.1 ? I ran into the same issue as you described with this file. However, converting it to a v2.1 structure with chatGPT made it compatible with Bruno.
Calendrier vacances API.json

Fantastic, thanks a lot, the converted version works for me, too. To answer your question, I created the original file that Bruno failed to import by using Postman 10.24.0 lightweight client / Settings... / Data / Export Sctatch Pad Data on macOS Sonoma 14.3. That Postman version was the one that I got after accepting the upgrade proposal from the Postman version I had been using up to then and it gave me the new version that wanted to force me into their cloud which I refused.
Did you use a particular prompt and a specific chatGPT version for the file conversion? I am asking because I have a few more collections that I probably need to convert like you did, too.

Not a very particular prompt nor chat GPT version i'm on the regular free ChatGPT 3.5 and I just asked "reformat this Postman collection into a Postman v2.1 collection" and simply pasted the json, hope it helps !

Thanks again, very helpful indeed!


Unable to add SmartThings Device ID

Describe the bug
When trying to add my Samsung S90D with SmartThings support I run into a problem when attempting to get my TV's SmartThings Device ID. I'm able to provide the token that I've generated with all the right permissions following the guide but on the next screen I'm informed that it's failed to find the Device ID. The guide points me to SmartThings' website where I get the ID from, input in the filed and hit next. After this I can see the prompt on my TV, which I accept. Inmediately after the dialogue box on HA refreshes and I get the same could not find Device ID message. I can try as many times as I want, always the same outcome.
Expected behavior
N/A
Screenshots
If applicable, add screenshots to help explain your problem.
Environment details:

Environment HASSIO on RBPI5
Home Assistant version installed: 2024.12.4
Component version installed: v0.13.5
Last know working version: N/A
TV model: S90D OLED

Output of HA logs
Paste the relavant output of the HA log here.
This error originated from a custom integration.

Logger: custom_components.samsungtv_smart
Source: custom_components/samsungtv_smart/__init__.py:443
integration: SamsungTV Smart (documentation, issues)
First occurred: 17:56:10 (7 occurrences)
Last logged: 18:02:41

Connection to SmartThings TV not available.

The same problem

I have the same problem


Gradle wrapper 4.10.2, build gradle tools 3.2.1 and kotlin 1.2.71

https://docs.gradle.org/current/release-notes.html
https://developer.android.com/studio/releases/gradle-plugin
https://github.com/JetBrains/kotlin/releases

> Task :storio-sqlite-annotations-processor-test:javaPreCompileReleaseUnitTest
Annotation processors must be explicitly declared now.  The following dependencies on the compile classpath are found to contain annotation processor.  Please add them to the testAnnotationProcessor configuration.
  - storio-sqlite-annotations-processor.jar (project :storio-sqlite-annotations-processor)
  - auto-service-1.0-rc4.jar (com.google.auto.service:auto-service:1.0-rc4)
  - auto-value-1.3.jar (com.google.auto.value:auto-value:1.3)
Alternatively, set android.defaultConfig.javaCompileOptions.annotationProcessorOptions.includeCompileClasspath = true to continue with previous behavior.  Note that this option is deprecated and will be removed in the future.

There is an option for android resources, but I don't see that for regular resources
Can you try enabling android one?
https://google.github.io/android-gradle-dsl/current/com.android.build.gradle.internal.dsl.TestOptions.UnitTestOptions.html
Like that:
android {
  testOptions {
    unitTests {
     includeAndroidResources true
    }
  }
}

As an alternative, we can actually move it to android resources, like: src/main|test/resources/raw/…

Closed via #897


Add support for VSTS containerized builds

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

Images/microsoft_powershell_windowsservercore/build.ps1

This file is recognized by Git as a binary file. It seems the encoding is not utf8.


Remove legacy code

Now that we dropped support for Android versions < 2.1, remove special code for these versions and try not to use deprecated APIs and interfaces.
Please review for sanity.

Unrelated to this pull request, but hooray on dropping legacy support!


Update Add-SPOHubSiteAssociation.md with multi-geo information

Added a note clarifying the values necessary when assigning a hubsite across geo locations.

@tiburd Please merge this PR. Thanks


Tentative date for production release?

I see there is a dev drop already. Is there a rough plan for when the final 6.0.0 will be ready?

Hi @renesas-brandon-hussey,
We don't have an agreed release date, yet. We need to be careful and keep the related packs in sync. An outstanding issue we need to solve prior the release is about default linker script handling, see Open-CMSIS-Pack/devtools#1134.

Thanks, this is the information I needed.


Azure: Firewall rule creation isn't supported from Azure section

Connect to Azure and expand to a server that doesn't have firewall rule open for your IP
Expand to be prompted for login info
Add valid SQL Login info

Expected:

You're prompted for firewall info
Actual:
Get error stating firewall isn't open. It's great we get the right error, but given it works elsewhere it's really weird it fails from here.

fixes by #5564


device tracker attributes fix

Description:
@balloob
Is this what you meant in https://github.com/home-assistant/home-assistant/pull/4719#pullrequestreview-11400855 ?
(I have not tested the last changes yet)

Actually, I gave it some more thought and we should do the following (sorry, I am not very consistent, I just woke up!)

not merge battery with attributes. After all battery was specified individually in see.
return attributes under the property device_state_attributes
initialize attributes in constructor to empty dict, inside async_seen do the self.attributes.update(attributes)

This is what I meant https://github.com/home-assistant/home-assistant/pull/4753


Helm wait not working on centos 7

Helm wait not working in centos. I am using helm v12 and the same version is working fine on ubuntu, but centos its not waiting for deployments to go in running state.

closing as inactive.