Homeassistant raspberry

Hi!
Is there any chance to install your addon on raspberry Pi homeassitant?
I have problems with command "scp -r Dockerfile config.json requirements.txt src/ root@homeassistant:addons/vantage2mqtt/"

I use it on RPi 4 with the 32bit homeassistant OS release.
https://www.home-assistant.io/hassio/installation/
It's been a while since I've rebuilt vantage2mqtt addon, and homeassistant does make breaking changes occasionally.  But scp should still work.  I assume you verified that ssh to HA works and you created the necessary addons/vantage2mqtt directory?
Also, a heads up.  I've got some changes planned in the not too distant future.  I would like to expose vantage tasks as mqtt switches.  After using the system for a while, I've determined that it would be better for my usage to directly control vantage tasks instead of controlling through wall switch buttons that are tied to tasks.  It's more flexible.
I was originally planning to change the naming convention for mqtt entities so that I could distinguish vantage buttons, relays, tasks, etc by looking at the entity name (these are all switches as far as HA mqtt is concerned).  But that would break any existing installation because the existing entity names would change.  I figured I was probably the only person using this addon, and could fix things up as I did the switch to using tasks.  But it appears there is actually at least one other person interested :grinning: So I'll only be modifying the naming for tasks and leave the rest alone.  Not ideal, but I would hate to break someone else's installation.

I have managed to install the addon, but get an error, do you have any ideas what that means?
[

s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] done.
[services.d] starting services
[services.d] done.
DEBUG:main:Skipping device cache
DEBUG:InFusionConfig:InFusionConfig init
DEBUG:InFusionConfig:Reading devices configuration...
DEBUG:InFusionConfig:Try Design Center configuration SiteName.dc
WARNING:InFusionConfig:Could not read SiteName.dc, [Errno 2] No such file or directory: 'SiteName.dc'
DEBUG:InFusionConfig:Lookup _aci service via zeroconf
WARNING:zeroconf:packets() made no progress adding records; returning
WARNING:zeroconf:packets() made no progress adding records; returning
WARNING:zeroconf:packets() made no progress adding records; returning
WARNING:zeroconf:packets() made no progress adding records; returning
CRITICAL:main:Error: Failed to read inFusion memory
[cmd] /vantage2mqtt.py exited 1
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.

The critical error is this one:

CRITICAL:main:Error: Failed to read inFusion memory

By default, the addon tries to locate your Vantage Infusion via zero
conf.  It appears it was not able to find it (timed out while
searching).
You can also configure it's IP address of the Infusion in the
configuration options for the addon. But if it could not be found via
zeroconf, I'm wondering if you have one of the more recent Infusions
that is network connected or an older one that can only be controlled
via RS232 port.  This addon requires a recent Infusion with network
capability and that it be plugged in to your network.
Re-reading my README file, I see I completely neglected to mention any
information about the required Vantage Infusion model and Vantage
system setup.  I'll update that to make this more clear.

Here is what the config looks like:

vantage:
site-name: office
zeroconf: true
ip: <IP_ADDRESS>
command_port: 3001
config_port: 2001
skip-device-cache: true
dcconfig: SiteName.dc
dccache: false
debug: true
short-names: false
buttons: true
lights: true
relays: true
motors: false
mqtt:
ip: <IP_ADDRESS>
port: 1883
username: mosquitto
password: mosquitto

And here is the vantage web interface, it is definately network connected:

I currently have it coded to use either zeroconf or IP & port.  If zeroconf fails it does not fall back to using IP & port.  So try setting zeroconf to false in your config.  Also if you are familiar with telnet, you can test the command port with

telnet <IP_ADDRESS> 3001

Should connect to the infusion and once connected

help

Should show a list of commands supported by the Infusion command interface

This is what I get:

help
BTN
BTNPRESS
BTNRELEASE
LOAD  <level (0-100)>
RAMPLOAD  <level (0-100)>
GETLOAD
LED  <color1 (0-255)>    <offcolor2 (0-                                                                               255)>  <blinkrate (FAST/MEDIUM/SLOW/VERYSLOW/OFF)>
GETLED
TASK  <eventType (PRESS/RELEASE/HOLD/TIMER/DATA/POSITION/INRANGE/OUT                                                                               OFRANGE/TEMPERATURE/DAYMODE/FANMODE/OPERATIONMODE/CONNECT/DISCONNECT/BOOT/LEARN/                                                                               CANCEL/NONE)>
GETTASK
STATUS <type (LOAD/LED/BTN/TASK/TEMP/THERMFAN/THERMOP/THERMDAY/SLIDER/TEXT/VAR                                                                               IABLE/BLIND/PAGE/LEDSTATE/IMAGE/WIND/LIGHT/CURRENT/POWER/ALL/NONE)>
GETTEMP
THERMTEMP  <type (COOL/HEAT)>
GETTHERMTEMP  <type (INDOOR/OUTDOOR/COOL/HEAT)>
THERMFAN  <fan (ON/AUTO)>
GETTHERMFAN
THERMOP  <op mode (OFF/COOL/HEAT/AUTO)>
GETTHERMOP
THERMDAY  <day mode (DAY/NIGHT)>
GETTHERMDAY
SLIDER  <level (0-100)>
GETSLIDER
TEXT
GETTEXT
VARIABLE  <value "text"/number>
GETVARIABLE
GETFIELD
GETSENSOR
GETLIGHT
GETWIND
GETCURRENT
GETPOWER
BLIND  <control (OPEN/CLOSE/STOP/POS)>
GETBLIND
INVOKE  <interface.method>  ...
!LOAD <type (M/S)>    <load/count> <level (0-100)>
!RAMPLOAD <type (M/S)>    <load/count> <level (0-100)                                                                               >
!GETLOAD <type (M/S)>    <load/count>
ADDSTATUS  ...
DELSTATUS [ ...]
LISTSTATUS
ECHO
VERSION
GETCOUNT
DELIMITER <Hex Byte 1> [<Hex Byte 2>]
LOG  [ ... ] [CONTROLLER] [TYPE] [TIME] [SOURCE] [FULL]   [DEBUG]                                                                                [DUMP] [INFO] [WARNING] [ERROR] [FATAL] [TASK] [DEVICE] [QUERY] [PROF]
DUMP  [CONTROLLER] [TYPE] [TIME] [SOURCE] [FULL]   [DEBUG] [DUMP]                                                                                [INFO] [WARNING] [ERROR] [FATAL] [TASK] [DEVICE] [QUERY] [PROF]
ELSIZE [] <type (STATUS/STATUSEX/AUTOMATION/SYSTEM/EVENT/MODCOM/ST                                                                               ATCOM)>
ELGETSIZE [] <type (STATUS/STATUSEX/AUTOMATION/SYSTEM/EVENT/MODCOM                                                                               /STATCOM)>
ELAGG [] <state (ON/OFF)>
ELENABLE [] <type (STATUS/STATUSEX/AUTOMATION/SYSTEM/EVENT/MODCOM/                                                                               STATCOM)> <state (ON/OFF)>
ELDUMP <type (STATUS/STATUSEX/AUTOMATION/SYSTEM/EVENT/MODCOM/STATCOM)> [PERCEN                                                                               T] [CONTROLLER] [TIME] [TYPE] [HEX] [FULL] []
ELLOG <type (STATUS/STATUSEX/AUTOMATION/SYSTEM/EVENT/MODCOM/STATCOM)> [PERCENT                                                                               ] [CONTROLLER] [TIME] [TYPE] [HEX] [FULL] <state (ON/OFF) []
LOGIN

Disabled zeroconf:

DEBUG:main:Skipping device cache
DEBUG:InFusionConfig:InFusionConfig init
DEBUG:InFusionConfig:Reading devices configuration...
DEBUG:InFusionConfig:Try Design Center configuration SiteName.dc
WARNING:InFusionConfig:Could not read SiteName.dc, [Errno 2] No such file or directory: 'SiteName.dc'
DEBUG:InFusionConfig:Try inFusion Memory card ip <IP_ADDRESS>:2001
DEBUG:InFusionConfig:read size 93
Traceback (most recent call last):
File "/vantage2mqtt.py", line 658, in
main.run()
File "/vantage2mqtt.py", line 633, in run
vantage_cfg = self.read_vantage_config()
File "/vantage2mqtt.py", line 525, in read_vantage_config
return InFusionConfig(self.cfg["vantage"], xml_config_file=dc)
File "/infusion.py", line 165, in init
self.read_infusion_memory(ip, port)
File "/infusion.py", line 267, in read_infusion_memory
xml_root = ET.fromstring(self.xml_config)
File "/usr/lib/python3.8/xml/etree/ElementTree.py", line 1321, in XML
return parser.close()
xml.etree.ElementTree.ParseError: no element found: line 1, column 0
[cmd] /vantage2mqtt.py exited 1
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
[s6-finish] sending all processes the TERM signal.

It seems your infusion isn't responding to zeroconf the same as mine and it isn't responding on port 2001 (where the config is retrieved from) the same way mine does.
It is responding with some data on port 2001, but it's not config data.  Unfortunately, there isn't any debug logging of what it returned.  How are you with python?  It might help to see what is being returned.  In src/infusion.py around line 260 add a debug statement like this
            config = config.decode('ascii')
            self._log.debug("read %s", config)
            config = config[config.find('<?File Encode="Base64" /'):]

I expect the above will show some kind of error in the text returned from the infusion.
A couple possibilities occur to me.  One, maybe they changed the firmware at some point and our infusion's just don't work the same. Or the other, maybe whoever set up your infusion didn't save the config to the sdcard in the infusion.  Standard practice is to save the XML config file to the infusions sdcard using the Vantage Design Center software during system configuration.  This is what I attempt to read from the device.
Do you happen to have the Design Center software or a copy of the XML config file used to set up your system?

Hi,
Love your work, just what I was looking for. I have a Vantage system at home and was having problems integrating with other systems so I could do some cool automation. I had some initial problems however once I set zeroconfig to false things fire into life. New to HA too so does not help. I'll let you know how things progress, looks like everything from the vantage system is now visible in HA either as a device or an entity. My only question is what is zeroconfig?

Zeroconfig is also called multicast DNS, avahi, and rendezvous.  It's a way for devices to advertise services they provide on your network without needing a central server.
Funny thing, zeroconfig just quit working for me about a week ago.  I hadn't even upgraded the software on my HA instance, so must be a problem with my network equipment.  Haven't had time to try to run it down yet.
Another thing to keep in mind with Vantage, the latest software update locks down the unencrypted ports on the Infusion.  You can open them back up if you have the Design Center software. So if you have any work done on the system, check that the plugin is still working when they are done. I'm going to add the ability to connect through the encrypted ports eventually. Just need to find the time.


How to use videos as input for HiLo？

Dear author, I would like to input the data dimensions [2,64,300,64,64] into HiLo, which represent dimension, number of channels, video sequence, height, and width, respectively. But the acceptable data for HiLo is [batch_size, batch_size,hidden_dimension]， I don't know how to convert it to a suitable dimension. I am a beginner in the field of artificial intelligence, and your answer is crucial to me!

Hi @q958287831, thank you for your interest! A video clip can be thought of as a list of frames. Initially, for HiLo, we have an input tensor in the shape of [B, N, D], where:

B: Batch size
N: Sequence length (image tokens for one image)
D: Hidden dimension

With a video clip, you will have an additional dimension:

T: Temporal length

Now, your video data might be represented as a tensor in the shape of [B, T, N, D]. Next, we can reshape this data into [B * T, N, D], which effectively combines all frames of different video clips into the batch size. This allows HiLo to process the data.
However, note that HiLo does not handle the dependencies in the temporal dimension. There could be a promising extension in future work.


How to declare one field as Map?

I know in jsonschema2pojo additionalProperties is java Map, but how can I declare one specific field also as Map? e.g. I want the following field config to be a Map.
{
  "type":"object",
  "properties": {
    "config": {
        "type" : "string"
    }
  }
}

One option is to say: "existingJavaType": "java.util.Map<String,String>"

Thanks @joelittlejohn


When starting the application, there is an error related to connecting to the localhost on port 8002. The specific error is a ConnectionRefusedError with the message [Errno 10061] Connect call failed ('<IP_ADDRESS>', 8002).

Describe the bug
When starting the application, there is an error related to connecting to the localhost on port 8002. The specific error is a ConnectionRefusedError with the message [Errno 10061] Connect call failed ('<IP_ADDRESS>', 8002).
To Reproduce
Steps to reproduce the behavior:
Start the application.
Observe the error related to connecting to localhost on port 8002.
Expected behavior
The application should start without any connection errors and be able to connect to the specified localhost on port 8002.
Screenshots

N/A
Desktop (please complete the following information):
OS: Windows (based on the paths in the error log)
Browser: N/A
Version: N/A
Additional context
This error seems to be related to a connection issue with the localhost on port 8002. Further investigation is needed to determine the root cause of the connection problem and resolve it.

@luolin-ai
Could you please provide a screenshot of the Docker container status? This can be checked by docker ps. Also, could you confirm if you followed the setup instructions below to start the project?

It is not run directly by Docker code.

@luolin-ai then another service called taskingai-inference is need. We'll soon make it open-sourced in GitHub. You can run it with docker for now.
docker pull taskingai/taskingai-inference:latest

OK Thanks. I want to deploy directly through code and make secondary development changes. Looking forward to adding more features


property 104 on count trigger is not unused, it's multi activate

https://github.com/Wyliemaster/GD-Decompiled/blob/1f58ffe6e9577df8781104c79fa7ddac50da1c5a/GD/code/src/EffectGameObject.cpp#L108
it's not unused, it's value (0 = no, 1 = yes) of this checkbox:

(don't confuse with multi trigger)

sorry, been in-active so i've only just seen this now. Good find. will adjust it when if i start working on trigger decompilations


[enhancement] merge data between feed in request

Merge data
The idea is to merge objects which contain the same attributes but comming from different feeds when requesting data.
Exemple
Initial situation :
Database :
Feed1 contains : {"id": "...", "type": "ipv4-addr", "value": "<IP_ADDRESS>"}
Feed2 contains : {"id": "...", "type" : "ipv4-addr", "value": "<IP_ADDRESS>"}
Actual behaviour
If we request using this pattering : [ipv4-addr:value = "<IP_ADDRESS>"]
Then we will get the two differents objects.
Expected behaviour
Stix2arango could detect that these objects are the same and merge these object and there relations. In this case stix2arango should returns only one ipv4-addr object.
Remediation
Stix2arango could generate the stix id field using a random number generator seeded with all fields object that are no references. All id-references to this object should be updated.
When a user request database, all objects with the same id are merged together.

reproducible uuid generation : https://stackoverflow.com/questions/41186818/how-to-generate-a-random-uuid-which-is-reproducible-with-a-seed-in-python

Note: after some research it appears that STIX2.1 SCO id are deterministic so no need to produce a reproducible uuid for sco.
Question : does we need to merge sdo objects that are not sco ? i.g. : Threat Actor, Malware, etc...

New data representation
After reflexion, the new insertion / request method will be the following.
1. Data insertion
Entry
An array of SDO, SCO and SRO
Algorithm
For each object:
     if object not SRO :
          object = generate_reproducible_id(object)

2. Data request
Entry
A stix2.1 pattern, a follow_link (boolean) argument
3. Algorithm
objects = match_in_database(pattern)
final_obj_list = []
if follow_link :
     for obj in objects:
           related_objs = get_all_related_obj(obj)
           for related in related_objs:
                  if related is SRO:
                        related = get_all_related_obj(related)
                 final_obj_list.append(related)
final_obj_list += [objects]
merge_obj_with_the_same_id(final_obj_list)


Snippets: auto import

I think that it would be great if user snippets could support auto-importing of required modules like this:
{
	"react.useEffect": {
		"prefix": "eff",
		"body": [
			"useEffect(() => {'",
			""
			"}, [])",
		],
		"import": ["react.useEffect"]
	},
}

which would add useEffect import from react at beginning of file when required.
Ideally it would extend current import when present instead of creating new entry (import React from 'react' would become import React, { useEffect } from 'react'.
That would be huge time-saver for a lot of developers, as this is thing that we are dealing with on daily basis.

@vscodebot This is not duplicate.
Just to clarify - this is just simple example that really does not need this feature and is just fine with auto imports. But there are more complicated snippets that would benefit from this.

This is a duplicate. Unfortunately, I don't recall the exact issue number nor its title :flushed: Anyways, searching for issues with the same labels will likely help you :point_right:
https://github.com/microsoft/vscode/issues?utf8=✓&q=is%3Aopen+label%3A"snippets"


Unable to use Neogit after update

Description
Re-installed neovim yesterday and after installing all the plugins required for my setup I had noticed that neogit was not operational as it once had been.
That is to say, it would not work with neither the default configurations (require("neogit").setup()) nor with my customized neogit setup.
Neovim version
NVIM v0.8.0-dev
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by nixbld (nix overlay: neovim-nightly)
Operating system and version
NixOS (nixos-unstable)
Steps to reproduce

Launch Neovim.
Call neogit through usual commands.
Greeted with error.

Expected behavior
A working neogit setup.
Actual behavior
Error executing luv callback:
.../nvim/site/pack/packer/opt/neogit/lua/neogit/process.lua:57: ENOENT: no such file or directory
stack traceback:
        [C]: in function 'error'
        .../nvim/site/pack/packer/opt/neogit/lua/neogit/process.lua:57: in function 'returned_functi
on'
        ...pack/packer/opt/plenary.nvim/lua/plenary/async/async.lua:31: in function 'callback_or_nex
t'
        ...pack/packer/opt/plenary.nvim/lua/plenary/async/async.lua:45: in function 'cb'
        .../nvim/site/pack/packer/opt/neogit/lua/neogit/process.lua:16: in function 'raise_if_fully_
closed'
        .../nvim/site/pack/packer/opt/neogit/lua/neogit/process.lua:50: in function <.../nvim/site/p
ack/packer/opt/neogit/lua/neogit/process.lua:44>

Minimal config
vim.cmd([[set runtimepath=$VIMRUNTIME]])
vim.cmd([[set packpath=/tmp/nvim/site]])
local package_root = "/tmp/nvim/site/pack"
local install_path = package_root .. "/packer/start/packer.nvim"
local function load_plugins()
  require("packer").startup({
    {
      "wbthomason/packer.nvim",
      {
        "TimUntersberger/neogit",
        requires = {
          { "nvim-lua/plenary.nvim" },
          { "sindrets/diffview.nvim" },
        },
        config = function()
          print("loaded neogit")
          require("neogit").setup()
        end,
      },
    },
    config = {
      package_root = package_root,
      compile_path = install_path .. "/plugin/packer_compiled.lua",
      display = { non_interactive = true },
    },
  })
end
if vim.fn.isdirectory(install_path) == 0 then
  print("Installing neogit and dependencies.")
  vim.fn.system({ "git", "clone", "--depth=1", "https://github.com/wbthomason/packer.nvim", install_path })
end
load_plugins()
require("packer").sync()

I did a quick bisect and the first "bad" commit is 9c63259c56e13e3385705fd44722f31381045c0c, this commit was part of #346 by @ten3roberts .
Not sure if this is limited to NixOS, as I can test this on a another OS currently.

If you place a print of the command and args above https://github.com/TimUntersberger/neogit/blob/463820a83f4ba387655f370a17c87dc3100cdf0d/lua/neogit/process.lua#L44, what does it say.
My hunch is that is failes to execute the binary because they are in a different place on nixos

Did rebase interactive work for you before my PR, because they use the same mechanism.
If rebase interactive didn't work for you before then the problem is likely deeper

I tried using rebase interactive from e6d8ec6 and it appears as if rebase interactive does nothing besides entering rebase when saved.
Side-note: Nix does store binaries in /nix/store and symlinks them to appropriate locations. Therefore this might be the cause of the issue, but that I cannot confirm due to being unaware of how the appropriate binaries are being called in neogit.

What happens if you go into neogit inside the packer dir, and uncomment this line?
https://github.com/TimUntersberger/neogit/blob/463820a83f4ba387655f370a17c87dc3100cdf0d/lua/neogit/process.lua#L52
It will show which command is attempting to run. As you said, they should be in the path, unless an explicit /usr/bin/git is being used somewhere

started process {
  args = { "--no-optional-locks", "ls-files", "--others", "--cached", "--modified", "--deleted", "--full-name", "src/cli.rs" },
  env = { "HOME=/home/xanderio", "GNUPGHOME=/home/xanderio/.gnupg", "NVIM=/run/user/1000/nvim.2233242.0" },
  stdio = { <userdata 1>, <userdata 2>, <userdata 3> }
} -> nil ENOENT: no such file or directory @@ nil

I found the issue and the fix, as mention in the comment on line 32, the enviroment is completely overridden this causes problems as the $PATH the child gets is missing all the NixOS PATH magic.
The fix is really simple just add table.insert(params.env, string.format("%s=%s", "PATH", os.getenv("PATH"))).

I have just discovered that commit singing via ssh is also effected by this as the SSH agent related variables are unavailable to the child.


Allow passing options for snd/url-pattern

As explained in snd/url-pattern#42, I need to be able to parse parenthesis in my named segment values, otherwise it results in a no-match.
Currently, it doesn't look like redux-little-router supports this, so might be worth adding in.

is really blocking!
So the options like described here https://github.com/snd/url-pattern#customize-the-pattern-syntax

facing this too

Would like to see it configurable as well.


Allow file:// style urls in addition to http://

For the sake of demonstrating the nsclient module at PuppetConf, I'm trying not to rely on the network.
Therefore, I'd like to serve the file from the Vagrant synced_folder share, rather than over HTTPS, like:

   class { 'nsclient':
     package_source_location => 'file://C:/vagrant/modules/nsclient/files',
     package_source          => 'NSCP-<IP_ADDRESS>-x64.msi'
   }

Without the included change, file style urls don't properly parse.

Hi @Iristyle - sorry for taking forever. We recently changed management.
Is this still desirable? If so, it needs a rebase and tests.

Since this hasn't seen an update in half a year I'm closing it for now.


SecurityRule constructor change compiles old code but throws exception

Improvement suggestion:
The change done here: https://github.com/Azure/azure-sdk-for-net/commit/9b550954c993911cf8ff4a5fde26502658efe34f#diff-975f9d2547d427bf78308e7693661ac8L89 to the SecurityRule class constructor makes all old code compile as usual but throw error in runtime because of the parameters order change.
Caused to lose precious time so if there is a way to cause a warning\error for code which uses the old constructor it probably would be an improvement.
OR use the new c# ability to declare optional parameter in middle of parameters list in order to preserve old order.

Before the mentioned commit, the code which used the api had a non-optional parameter and few optional parameters (all strings). After the commit, the order of the optional and non-optional was changed (still all string) so that the old code still compiled but behaved in a wrong way in runtime and threw runtime error.


RFC - RE: Pipeline input. How should this be handled?

All exposed functions should accept pipeline input. In order to accomplish this the traditional Powershell way, objects in the pipeline should have the properties expected by the funtions being piped to, however the API returns objects one level higher than that. Changing this output would be a major breaking change, and should be considered very carefully.
There are two options:
1) Leave output alone and simply query one level deep before piping:
$query = Get-SmaMachineInventory .....
$Query.Machines | Set-SmaMachineInventory ....

Advantages:

No breaking change needed

Disadvantages:

Contrary to best practice in pipeline input
Would cause confusion for those expecting traditional pipeline behavior

2) Format output to remove top-level parameters Count and Warning, and only return results:
Get-SmaMachineInventory .... | Set-SmaMachinenventory ....

Advantages:

Best practice in handling pipeline input
Could take advantage of pipeline parallelism to improve performance on longer queries

Disadvantages:

Major breaking change. Any scripts that use syntax like $Query.Machines to retrieve objects would need to be re-written to simply use $Query. This would apply to all endpoints.
Note Properties Count and Warnings would not be returned with the request, although traditional count methods would still be easily usable. (.count , | Measure-Object, etc.)

I'm leaning towards a phased approach, i.e. enabling pipeline input like in option 1, so as not to break anything, then at some point in the future, simply making the switch to v2 after a warning has been out for a while.

This is now implemented as of https://github.com/ArtisanByteCrafter/KaceSMA/tree/1580db550192e10a39263f9a0ba59ac6a04578af


[Protobuf] Optional properties not removed

Bug Report
It looks like typia is unexpectedly include optional property with value undefined after decoding with protobuf. Technically, it's considered a valid behavior by typescript but I think as a protobuf parser then typia should remove those properties instead.
📝 Summary
Write a short summary of the bug in here.

Typia Version: 5.2.2dev(@next)
Expected behavior: No property after decoding
Actual behavior: included undefined properties

Write detailed description in here.
// Expected
{}

// Actual
{
  a: undefined,
  b: undefined
}

⏯ Playground Link
https://typia.io/playground/?script=JYWwDg9gTgLgBDAnmYBDOAzKERwERIqp4DcAUGcAHYwCmUGqAxrXAAqpSogDOcA3mThxUALjg8YUagHNywgEbiqAVxAL65AL5lCrAPJgYwCFVQAbDl15wAvO07GLAHivceAPnJkmpyXFoqXwATWmC3G3tCNAA6MGwYCAUVDBjAkNpnQ2NTCwjPAAp+LQBKcl8qfzBOdzsEZFj4iETk1NCMrKMTM0sa3g8C9IhQ8L6eMp8-CHNaGPMIGQLq63GSIA
💻 Code occuring the bug
import typia from "typia";

interface Params {
  a: string;
  b: number;
}
type OptionalParams = Partial<Params>;

const encodedParams = typia.protobuf.encode<OptionalParams>({});
const params = typia.protobuf.decode<OptionalParams>(encodedParams);
console.log(params);

Should I add delete obj.a like statements for that?

@samchon The problem with this issue is that it unnecessarily increases the buffer size. And I think protobuf function should aim for networking performance over runtime performance. This issue obviously won't break the code but it's better to produce the smallest possible buffer for maximizing transferring speeds, while still guaranteeing type-safety. Normally, using JSON will require compressions as an extra step for minimizing data size. This is the data lifecycle when using JSON format.
Receiving data -> Input buffer -> decompression -> JSON decode -> Input validation -> processing data -> Output validation -> JSON encode -> compression -> Output buffer -> Transferring data

This is the developer's expected lifecycle.
Receiving data -> Input buffer -> protobuf decode -> processing data -> protobuf encode -> Output buffer -> Transferring data

So even a novice developer can see that there will be a massive performance boost when applying the second approach.
Most developers know that it's very bad to use JSON instead of protobuf in terms of performance. But most protobuf libraries require defining an extra static schema for each message, which is extremely incovenient. typia is actually the developer's dream when it comes to handling this problem. I personally think that typescript should natively support type-casting and validation as well as providing runtime type metadata via plugins. But it seems that it won't happen in near future.
Back to this issue, I think you should check for the existence of the undefined properties in input object before deleting it. Checking a === undefined will fail if the property is defined like {a: undefined}.
// input
{a: undefined, b: 1}

// expected output
{a: undefined, b: 1}

//input
{b: 1}

// expected output
{b: 1}

So the the code should be like this.
if (!Object.getOwnPropertyDescriptor(input, "a")) {
  delete output.a;
}

If I adapt the second way of above picture, the dynamic property assignment, your goal would be accomplished.
By the way, before determining whether accept your suggestion or not, I can't understand your word "increase the buffer size". In the Protocol Buffer spec, if do not insert anything to the buffer about a specific property, it means null or undefined about the specific property. Pre-assigning undefined value does not increase the buffer size of Protocol Buffer.

I've just double-check and typia indeed removes keys with null or undefined in the encoded buffer. So the buffer size is irrelevant. If that's the case then I think checking those properties directly with if(prop === undefined || prop === null) {delete output.prop} is good enough.
I personally think that removing undefined keys makes sense but the null value. Sometimes we will need null properties for debugging purposes. But if it's in the spec then it should be fine.

I will provide an option that turning on and off the delete statement at Saturday.

This feature would be supported after TypeScript v5.3 release with ts-patch supporting

If you want to use it earlier, then install like below:
npm install typia@next


Migrate service health check to Go

Fix #3
Replace the existing service checking functionality with a Go
application. Previously the logic was done using JQ which is not
installed on all systems.
Create a Go CLI application which will make a HTTP GET request to Consul
and inspect the results to ensure the status of the specified service
matches the desired state.
Update the Makefile's build section to build the Go application at the
root of the project.
Update the consul-svc-healthy.sh script to use the Go application rather
than curl and JQ.
Signed-off-by: Anthony M. Bonafide<EMAIL_ADDRESS>

Code has no unit tests.

Dockerfile edits need to be made:

Remove the RUN cp /usr/bin/jq and RUN cp /usr/lib/libonig.so* lines
Add a line to copy in health to /consul/scripts/health
Convert the Dockerfile to use a multi-stage build -- a docker container that builds the executable and passes the resulting binary to the deployment container -- numerous examples in edgex-go
Edit apk add line to no longer install jq and curl.

I just noticed it now, but security-proxy-setup-checker also has a dependency on curl and jq.  However, let's not plan to fix this right now as it is not a blocker.

Dockerfile edits need to be made:

Add a line to copy in health to /consul/scripts/health
Convert the Dockerfile to use a multi-stage build -- a docker container that builds the executable and passes the resulting binary to the deployment container -- numerous examples in edgex-go

Are you referring to this logic?
https://github.com/edgexfoundry/edgex-go/blob/4333c146e54e914d1b50db61a2e9368113cae9e2/cmd/core-data/Dockerfile#L10-L27

Are you referring to this logic?

Yes, exactly. The first FROM is the build container.  Later there is a COPY --from=builder to install the resulting executable into the deployment container.

Added tests, and fixed the Dockerfile  as per @bnevis-i 's suggestions.

Recheck

@michaelestrin approving review required.

This PR is blocking for https://github.com/edgexfoundry/blackbox-testing/pull/350


ScriptsToProcess not honored when Import-Module is passed -Version param

Steps to reproduce

Install SqlServer module from PSGallery (I doubt the issue is specific to my module, though)
Open PowerShell (full .Net, I'm on Win10 using the PowerShell that comes with the OS)
PowerShell -executionpolicy bypass
Import-Module SqlServer -Verbose
(observe that SqlServer.ps1, listed in the SqlServer.psd1 manifest file, is loaded during import)
Close PowerShell
Restart PowerShell (same as above)
Import-Module SqlServer -Verbose -Version 21.0.17099

Expected behavior
The SqlServer.ps1 script is loaded
Actual behavior
The SqlServer.ps1 script is NOT loaded
Environment data
> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      5.1.16192.0
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.16192.0
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

@Matteo-T Thanks for your report! Could you check on latest PowerShell Core?

My module does not work on PowerShell Core, so I'm not sure how that would help... I'm not sure I'll have time to create a dummy repro (i.e. a dummy module) that works on PowerShell Core any time soon, but if I have to...

This repo is for PowerShell Core only. Please report regular PowerShell issues on the uservoice website:
https://windowsserver.uservoice.com/forums/301869-powershell

@Matteo-T Please test after PowerShell Core Beta1 release (in the coming days).
And yes, we can report on UserVoice.

Sorry, I did not know this was for "Core" only. I was told to come here, but I guess I was told incorrectly. I'll try the UserVoice thing...

@megamorf @Matteo-T for bugs that are almost certainly applicable to both editions of PowerShell, it's perfectly fine to file them here. I suspect that this one repros on Core 6 just fine, and given the importance of Import-Module we should probably get it fixed (assuming it does repro).
I'll build a basic repro on my box right now.

Yup, totally repros on 6.0.0-alpha.18 and 6.0.0-beta.1:
C:\> mkdir C:\temp\VersionParam
C:\> cd temp\VersionParam
C:\temp\VersionParam> 1 | Out-File Output1.ps1
C:\temp\VersionParam> New-ModuleManifest VersionParam.psd1 -ScriptsToProcess .\Output1.ps1
C:\temp\VersionParam> Import-Module .\VersionParam.psd1
1
C:\temp\VersionParam> Remove-Module VersionParam
C:\temp\VersionParam> Import-Module .\VersionParam.psd1 -Version 1.0
C:\temp\VersionParam> Get-Module VersionParam

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Manifest   1.0        VersionParam


Can't unlink workflow from subject set

When I tried to uncheck a subject set from my "Marking2" workflow in http://demo.zooniverse.org/panoptes-front-end/#/projects/aliburchard/WhaleTails, I get this error:
message:"Couldn't find SubjectSet with 'id'=1959 [WHERE "subject_sets_workflows"."workflow_id" = ?]"

make the error message more informative and not expose SQL details.

I also think this is caching bug somewhere. @aweiksnar was seeing something similar yesterday

@aliburchard I think @camallen's most recent deploy fixed this. Can you try and see if you're still having the issue?

Hmm, I'm still struggling. The tick boxes now reflect being checked and un-checked (yay @brian-c) but I can't seem to link to a subject set. I even tried deleting all the old ones and uploading a new one and still no luck. I get a "param missing" error when I try to link to the subject set, and then have no subjects to classify. This is after a hard reload.

@aliburchard @brian-c I think that error is just caused by the save button becoming active again when you link subject sets (even though it saves that automatically and separately)

Yeah, both these issues are my bad. I'll get them sorted today.


Difficulty finding the inference function

Hi, @Wangt-CN, first off, great work!!
I want to run inference through code, not gradio. I tried searching for the function to do that, the closest I found is the Agent_LDM. But this takes a reference fg, bg and skeleton. Is there a function which just takes in an image (with a character in it) and a skeleton, and returns the output?
Additionally, any function for end to end video gen?
Thanks
Just another: Will the code be compatible with PyTorch 2?

found the solution, closing

Hi, can you share how you resolved this?

found the solution, closing

please share your solution.


feat: another characterization of filtered categories in terms of type-valued limits

[x] depends on: #10662
[x] depends on: #10664

Thanks!
bors merge


Return error code

Return error codes to allow detection of errors when used in script

:+1:  Thanks!


Add references for parameters

Add references for some missing parameters words.

Codecov Report
Base: 78.42% // Head: 78.42% // No change to project coverage :thumbsup:

Coverage data is based on head (97b749d) compared to base (b68974a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #232   +/-   ##
=========================================
  Coverage     78.42%   78.42%
  Complexity      849      849
=========================================
  Files           118      118
  Lines          2568     2568
  Branches        324      324
=========================================
  Hits           2014     2014
  Misses          428      428
  Partials        126      126

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


add [entertainment system] session

Thanks for the PR!
I am wondering if it makes more sense to categorize this as Flight or Airplanes rather than Entertainment System. The flight is the hacked target while the entertainment system is the vulnerability. This is similar to the Jeep case, in which the Jeep is the target while Uconnect is the vulnerability.
Thought?

Planes category is sense to me.   Let me close this and make this one.


Connection to Local SQL Server instead of Local DB failed

I try to connect to Local SQL Server instead of Local DB using "Data Source = <IP_ADDRESS>; Initial Catalog = Blogging; persist security info = True; user id = sa; password = abc". I got the error "Cannot open database "Blogging" requested by the login. The login failed.
Login failed for user 'sa'.'".
I have verified the login and password has no problem.
What is the right Connection String for SQL Server instead of localDB?

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

ID: 1b15cd4c-9cba-c0e2-7618-7d621a3c5936
Version Independent ID: f4681bc6-57be-6e69-cad9-776ce64d93bd
Content: Overview - EF Core
Content Source: entity-framework/core/index.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @rowanmiller
Microsoft Alias: divega

Use (local) or . for data source

I did try
"optionsBuilder.UseSqlServer(@"Server=(local);Database=Blogging;Integrated Security=True");"
I got error
"Cannot open database "Blogging" requested by the login. The login failed.
Login failed for user 'WIN-DL44PDBKCA4\Administrator'.'"
I was thinking I don't need to create database with this code? Or I am wrong, I have to create the database manually?

Yes, you will need to create the database manually by using migrations. You can find the instructions under Get Started -> .NET Core ->New Database under the 'Create the Database' section.

Thanks. After I did dotnet ef database update, I did see blogging.db in the folder but I gave me "SqliteException: SQLite Error 1: 'no such table: Blogs" error when I executed the program.

Mixing SQL Server and SQLite in this way will not work.


Send GZIPed JSON request bodies

Currently collectors send big JSONs without any compression. It'd be great to leverage already existing libs in order to make request bodies smaller
@Vixtir please let me know if ODD Platform needs to be tuned somehow. From my experience it won't be needed

Field chunk_size added to collector config.


Migrate to REST API v2

@lefcha we will have to adapt the client to return the object on all update methods: https://github.com/Doist/Todoist/pull/6938
This is the last change, I think. :)

@lefcha we will have to adapt the client to return the object on all update methods: Doist/Todoist#6938
This is the last change, I think. :)

Will the backend return 200 instead of 204?
Because in that case the object should be returned, according to:
https://github.com/Doist/todoist-api-python/blob/21d8caff54e866f7351d1c489ee0ce190a0de543/todoist_api_python/http_requests.py#L44-L48

Will the backend return 200 instead of 204?

The update will return 200 with the JSON of the updated object.


Capacitor: Can't use interactive command if called by NX

Current Behavior
If I execute this command directly in the application folder, I can interact with the prompt by choosing my options with the command : npx --package=@capacitor/cli cap run ios --livereload --external
If this time I go through NX, via the command nx run app-insured:run:ios, I can't interact with the arrows and I see special characters :  ^[[B^[[A^[[C^[[B^[[D^[[A^[[D
Here is the command:
"run": {
  "executor": "@nxext/capacitor:cap",
  "options": {
    "cmd": "run"
  },
  "configurations": {
    "ios": {
      "cmd": "run ios --livereload --external"
    },
    "android": {
      "cmd": "run android --livereload --external"
    }
  }
},

Expected Behavior
Have access to the interactive controls in the same way
GitHub Repo
No response
Steps to Reproduce

Run nx run app-insured:run:ios which will propose options in the terminal to choose with the arrows.
Choose your option with the arrow.
Special characters will appear, not works
Try to run the command associated with the last nx alias directly in the application folder
Works, you can choose your option

Nx Report
>  NX   Report complete - copy this into the issue template

   Node : 19.4.0
   OS   : darwin arm64
   npm  : 9.2.0

   nx                      : 15.9.2
   @nrwl/js                : 15.9.2
   @nrwl/jest              : 15.9.2
   @nrwl/linter            : 15.9.2
   @nrwl/workspace         : 15.9.2
   @nrwl/angular           : 15.9.2
   @nrwl/cli               : 15.9.2
   @nrwl/cypress           : 15.9.2
   @nrwl/devkit            : 15.9.2
   @nrwl/eslint-plugin-nx  : 15.9.2
   @nrwl/storybook         : 16.0.0-beta.1
   @nrwl/tao               : 15.9.2
   @nrwl/webpack           : 15.9.2
   typescript              : 4.9.5
   ---------------------------------------
   Community plugins:
   @nestjs/ng-universal        : 7.0.0
   @nguniversal/express-engine : 15.2.0
   @nguniversal/builders       : 15.2.0
   @nxext/capacitor            : 15.7.0
   @storybook/angular          : 7.0.5
   ---------------------------------------
   The following packages should match the installed version of nx
     -<EMAIL_ADDRESS>
   To fix this, run `nx migrate<EMAIL_ADDRESS>
Failure Logs
No response
Additional Information
No response

I am still looking for a solution to that problem. Unfortunately, I'm not sure how to fix this cause it's just between the cap command and the Nx run executor. ^^'

Might be worthwile creating a ticket on the NX github and linking it here?
Will provide visibility to both Nxext and NX

Is there a reason to use the runCommands code over execSync?

I just reused habe executor. Haven’t tried execSync here yet


Implement CLI client for the broker

Implement a CLI client to do the administrative tasks for the broker.
Eg: manager queues, exchanges consumers etc.

Relates To: #245 #248 #257 #260 #261 #262 #298 #300 #324


⚠ Bofrost.at has degraded performance

In 84d15d7, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 968 ms

Resolved: Bofrost.at performance has improved in f2e95d4.


option for a "ci rebuild" builder

when the yaml config is changed, it would be nice to have buildbot re-parse the yaml, regenerate its configs, and reload itself so that the configuration changes get tested and applied automatically

this is the lowest possible priority of all other capabilities


➕Added algorithm "Conway's Game of Life"

Coded in both C language and Js(using HTML5 ) to have a WEB BROWSER REPRESENTATION as well.
Code in C : Prints a pattern a number of times as user input.
Code in Js: allows you to click on the canvas to draw and erase cells and provides buttons to start/stop the simulation and clear the grid. The simulation will continue to evolve until you stop it, so that we can draw patterns as you like.

Can you please label it as hactoberfestaccepted ?
I will be highly grateful


Global Disable Module Switch

Can we add a Global switch to disable or enable the module entirely?

This would greatly help tfvars based setups where not all accounts need all the clusters. Putting all  the clusters in separate projects is not always possible because of also needing to manage VPC and subnets tags.
Is this considered desirable? If so, I don't mind starting on it.

This is exactly what I need it for!

This is already being worked on in #580 . And there's a duplicate solution in #588

Since #580 is merged, i'm closing this.


fix: Schema / Generators

Rough rewrite that removes wrong information and puts the rest into a possibly better structure.

I just read this page myself to figure out some details, but a lot of the information here was wrong or misleading. So I did a quick rough rewrite that adds a slightly different structure and removes the wrong information @mhwelander. Would probably make sense to talk through the parts that were wrong to clear these misconceptions.


fix(priority): lower priority to make it work with ngIf

ngIf priority is 600 so I've lowered uiTinymce's to 599. I had a look at some other angular directive but apart from ngIf, I can't see any other one that could be a problem.
I also wanted to add tests for #146 to make sure this change doesn't introduce a regression but wasn't able to reproduce the issue it fixed back then. Anyway, 599 is still pretty high so I guess it's ok.
fix #241

Thanks for doing this.
Will get this merged and release a new version later tonight.

Glad to hear it!


new app to follow gpx traces : gipy

hi,
i think my app is now good enough for a public release.
it has been tested over a few hundreds of kilometers now and is pretty usable.
you can see it in action here : https://wagnerf42.github.io/BangleApps/?q=gipy
you should note the following things:

I am a complete javascript noob : it is my first javascript program. I don't know javascript and since it is a subset of it on the watch I must surely use it in a pretty awful way.
I have some binary code in the repo because the file converter used in gipy_uploader is written in rust and compiled to javascript in wasm. this code is only for the custom webpage and does not run on the watch. the original source is here : https://github.com/wagnerf42/gpconv
I think these wasm files break the current tests. could you give me a hand here ?

it's been a fun project and i still have lots of ideas for it. since it's the most useful app for me on my watch as of now I wanted to share.
feel free to ask me for modifications.

Cool, I need to test this.

(it uses your movement to figure out your orientation
and walking is too slow).

How about dropping to compass for orientation when speed drops below e.g. 5km/h?

yes, that's on my todo list

Thanks! This looks great! I guess personally I'd have used JS to convert GPX to your GPX format, but if this works ok then great - and I really like the idea of the binary format to cram as much data in as possible.
Just wondering, is there a reason that the gipy_uploader isn't just part of the gipy app? You can make the custom.html file an interface.html file so then you just install gipy and you click the icon next to it to upload/change its data?

i've been thinking we can maybe remove the blob:

have a git submodule
add rust + wasmpack to the docker
compile it at the same time as the web pages

this would be cleaner but i'm not sure how easy it is to do.
for the interface.html that's interesting. i would have preferred to have it as a single app but i found no app like that so didn't know how to do it.
i'll give it a try.

I'd actually prefer the blob (with a link to the repo where you built it) than to add extra stuff to the GitHub actions here. It's quite important that I can quickly verify PRs don't break stuff, and if I've got to wait for rust+wasmpack to install and run every time I want to check something it'll be annoying.

i would have preferred to have it as a single app but i found no app like that

waypoints could be a good example? https://github.com/espruino/BangleApps/tree/master/apps/waypoints
The only reason it's not a 'full' app in its own right is so the list of waypoints could be shared between apps.
... Or https://github.com/espruino/BangleApps/tree/master/apps/openstmap does it too - but that uses 'custom' like you're doing - so you have to choose what you want when you upload the app. At some point that'll change though as for openstmap I'd like to allow multiple areas to be included.

could you give me a hand for the file upload ? i have some troubles i guess because i want to upload some binary.
i guess i need to escape more things. i tried to escape the simple quotes.

Maybe look at the restore-code of the app loader.
also do you know, that you can run the apploader locally with npm? No need to test each change on github.

oh thank god. no i don't know. how do I proceed ?

npm ci and npm start should do it after you checked out the submodules.

hey, thanks for the tip. i had to generate a certificate and key but it worked. it is so much better than uploading to github.
also,  if you don't mind i'd like to polish the interface a bit more. I think i should be able to get openstreetmap data even from there.

so, good news, openstreetmap data is now fetched from the interface.html.
it's much nicer like that, thanks for your help.
i'm going to tune a little bit this page to give more choices to the users as to what they want on their map.

That's great, thanks! Just let us know when you think you're ready to merge

yes i will. note that the upload seems actually broken. i had a quick test this morning and only the start of the files is actually uploaded.
i wonder, do i need to use chunks ?
like in core/lib/espruinotools.js ?

i had a quick test this morning and only the start of the files is actually uploaded.

Argh, that's a pain.
I see you're using Util.writeStorage, but actually it looks like the implementation of that is pretty stupid (I guess it was previously just used for writing small files) and as you say it's not chunking the data (and it should be).
Do you think you'd be able to breakpoint that bit of your code in devtools (or add a console.log) and paste a copy of the exact data you're calling Util.writeStorage with in here? Then I can fix the function here and make 100% sure it'll write all your data correctly.

hi, so, here is the array i receive from the converter. this is an array of u8
log.txt
it is then turned into a string :
179     let gpc_string = String.fromCharCode.apply(String, gpc_content);
i think outside of that the interface.html seems good. i'll test it a bit more tomorrow though.

Hi! I just committed some code which should fix writeStorage.  Please can you give it a try? If you're happy it works I guess we could merge now?

hi, it works great. thanks !
i tested on a very large file and while it took a minute to convert it worked nicely.
i think it is ok for release.
thank you both very much again for your help.

Excellent - thanks!


EditorHelp formatting issues when viewing class reference document in Editor.

Godot version
v4.0.3.stable.official [5222a99f5]
System information
Windows 11, AMD Ryzen 5 3500U with Radeon Vega Mobile Gfx     2.10 GHz
Issue description
I have encountered formatting issues of the text for EditorHelp/class reference documents in the ScriptEditor.  Unfortunately I can't reliably trigger the issue because it does seem somewhat random.  Sometimes when I view a DOC, the text is in one long vertical line as though the control (RichTextLabel) hasn't correctly determined the size available to it.  If I scroll further down the document, and then move to other documents, etc. and then back to the one that has been scrolled, the formatting may be fine from that point to the end, but prior to the initial scroll point is the thin vertical line of text.  Sometimes it happens with a DOC that's been newly opened, or one that I have visited before in the Editor run.
If I jiggle the resizing border of the panel, the formatting fixes itself, but it may have to be done multiple times in the same document as you scroll around.
Steps to reproduce
Open a DOC, but other than that I don't know.  It might happen.  It might not.
Minimal reproduction project
N/A

Duplicate of #77558


examples: fix movements for audio 2d/3d

Objective
The directions are wrong, need to reverse them.
Solution

Describe the solution used to achieve the objective above.

Can you explain what you mean by wrong? When I hit left the listener moves to the left. It all seems correct to me.

I'm on a Mac and the direction is reversed. It's very strange

also on a Mac, the directions are right for me

Ok. I misunderstood the code


Navigation properties on returned entity

Hi,
I'd like to do something like: customer.customersource.name. Is this possible?
I know when I do db.rel.find('customer', 'my_id') it will return the customer and the related entities but it would be nice if it could also attach a property to the model for navigation purposes.

Sorry, I'm not sure if I understand this request... if you'd like to attach custom properties to JS objects I believe you can do that yourself in user code?

Yes of course and that is what I'm doing, it would just be nice to be able to have auto populated navigation properties (much in the same way a lazy loaded navigation property would work in Entity Framework for .NET).
Not an issue but thought I'd see if it existed.


Dev 0.2.0

Closes #37
Closes #33
Closes #38
Closes #41
Closes #44

Codecov Report

Merging #47 (63c944f) into main (4a96a63) will increase coverage by 0.31%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
+ Coverage   92.94%   93.25%   +0.31%
==========================================
  Files           8        8
  Lines         170      178       +8
==========================================
+ Hits          158      166       +8
  Misses          9        9
  Partials        3        3

Impacted Files
Coverage Δ

cmd/root.go
91.48% <100.00%> (+1.74%)
:arrow_up:

Continue to review full report at Codecov.

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


Feature/139 add user deploy support

PR details
What changes did you make? (Give an overview)

closes #139

to dos

[x] create / update user with basic data (key, login, name, email, notificationEmail, api-user, locked, active, mustChangePassword, DefaultBusinessUnit)
[x] set SSO & FederationID
[x] add global user role
[x] remove global user role
[ ] add BU specific user role?
[ ] remove BU specific user role?
[ ] Make user permissions readable (currently IDs only)
[ ] add user permissions
[ ] remove user permissions
[ ] add associated BU
[ ] remove associated BU

Checklist

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] test scripts updated
[ ] Wiki updated
[ ] Wiki section on SSO & FederationID added

support for removing roles added.


♻ Refactor oauth2 lib

Current dependencies on/for this PR:

main

PR #18   👈

This stack of pull requests is managed by Graphite.


Error occurred when executing Storydiffusion_Sampler:  'image'

Now is it possible to launch without an image?

img2img need img input


Replacement Parameter does not work as described in documentation for Code Snippets

https://docs.microsoft.com/en-us/visualstudio/ide/walkthrough-creating-a-code-snippet?view=vs-2019
$MyVar$ is blank when inserting the snippet into code.
snippet code:
private bool $MyVar$ = true;
insert into code:
private bool = true

@robinne  Thank you for the question.
I'm not seeing anything initially wrong as per article steps.
Can you post the Literal code section?  (from the Replacement Parameters step).

HasChangesProperty.txt
OUTPUT
 private bool  = true;          private string ;         public string          {             get             {                 return ;             }     set             {                 if( != value)                 {                      = value;                     if (!)                     {                         HasChanges = true;                     } OnPropertyChanged();                 }                  = false;              }         }
Thank you!

@robinne Thank you for the response.
From the attached sample, you need to include a Literal section for each variable.
HasChangesProperty(revised).txt

Sorry to waste your time, I see that now. I was "just getting something to work" and ignored that section, as it appeared to be comments. Appreciate your time!


AnimationPlayer usability & improvements discussion

Operating system or device, Godot version, GPU Model and driver (if graphics related):
Ubuntu 17.04 - Godot v3.0.alpha.custom_build.3ae1cff
I feel like the animation player is not intuitive at all. I propose to make it work like blender. Eg, things I found that don't make sense (some might be bugs):

some properties are modified even if the play cursors hasn't reached the keyframe. For example, take canvas modulate, and insert only one keyframe say at second 3 and change the color to something other than white (say red). Start the game with canvas modulate color at pure white. It turns out that the canvas modulate will be red immediately, even though there's no key to change it to red at 0! The position on the other hand behaves correctly, that is if you insert a key at second 3 and there's no other keyframe before that, then the object won't move until second 3 is reached. I've only tried a handful of properties but so far the only reasonable behavior is on position

if loop is turned on don't try to interpolate between last keyframe (before the animation finishes) and the first keyframe. Instead let's follow the blender way, so if I have a sprite that moves from left to right and only 2 keyframes, one at T0 and one at T1 and animation length of 2 seconds with loop turned on, I expect sprite to move only between T0 and T1. How it's implemented at the moment is, if loop is on, then there will be an interpolation between T1 and T0 as well (from second 1 and second 2). I think this isn't intuitive. Blender has some pretty good standards, if they do it that way they probably have a good reason why.

would be great if we could be able to type a number in the Transition panel instead of just relying on the mouse to move the easing around.

If you guys can think of anything else let the discussion begin :)

Yes, that shouldn't happen. I think I know why it does and maybe I can fix it.
Interpolation between cycles is interesting, but indeed not always desirable. Looping mode could be changed from on/off to off/on/on-interp-cycle.
Yes, sometimes it takes a while to get the exact value you want. Improving that would benefit every place an easing editor is present.

Actually if I think about it, blender has a way to modify the loop behavior (extrapolate). So in the F-Curves editor for example, the animation can be set between:

constant extrapolation
linear extrapolation
make cyclic (what godot is doing by default)
and clear cyclic (what blender does by default)

Addressed by me (#12107).
Already done (don't know by whom nor when).
Addressed by me (#12113).

One of the main issues I keep running into is the inflexibility of the step value. Currently it can only be set to 2 decimal points which makes it impossible to set for 30 fps. You could set it to animate at 20fps(0.05) or 25fps(0,04) but these are not good values when your target game fps is likely 60fps.
Not sure what a good solution for this is, as how many decimal points to approximate 1/30 is needed. Maybe a mode where you can use a fps value and animate on frames would be ideal.

Agreed, this should only work as you describe if interpolated loop is
enabled.
You can choose this behavior per track
Yeah makes sense

Regarding step, i think we should have a button to switch between step and
fps, since they are pretty much the same.. and edit the animations in both
modes.
I have a long list of usability improvements for this, but was waiting for
after 3.0 to work on them..
On Oct 15, 2017 9:15 PM, "James Redmond"<EMAIL_ADDRESS>wrote:

One of the main issues I keep running into is the inflexibility of the
step value. Currently it can only be set to 2 decimal points which makes it
impossible to set for 30 fps. You could set it to animate at 20fps(0.05) or
25fps(0,04) but these are not good values when your target game fps is
likely 60fps.
Not sure what a good solution for this is, as how many decimal points to
approximate 1/30 is needed. Maybe a mode where you can use a fps value and
animate on frames would be ideal.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/10752#issuecomment-336752600,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2zZEZmyxHPj5LeYh2xQ-L8sFzZ8mks5ssqAQgaJpZM4PFvZ1
.

Here's another issue talking about Animations: #11359
My suggestion to improve AnimationPlayers is an additional transition curve known as spring/elastic. It's helpful for animating natural movements with few keyframes, as it's a fairly realistic interpretation of how living things move.
A good demonstration of using spring/elastic easing to speed up procedural animation: David Rosen Procedural Animation GDC (See also 17:42)
There's an interactive comparison of easing functions here
This issue discusses the ease function, which is related: #10572

I will be working on this the next weeks, so any feedback will be very
welcome.
On Fri, Oct 27, 2017 at 7:32 PM, cwook<EMAIL_ADDRESS>wrote:

Here's another issue talking about Animations: #11359
https://github.com/godotengine/godot/issues/11359
My suggestion to improve AnimationPlayers is an additional transition
curve known as spring/elastic. It's helpful for animating natural movements
with few keyframes, as it's a fairly realistic interpretation of how living
things move.
A good demonstration of using spring/elastic easing to speed up procedural
animation: David Rosen Procedural Animation GDC
https://youtu.be/LNidsMesxSE?t=7m39s (See also 17:42)
There's an interactive comparison of easing functions here
http://easings.net/
This issue discusses the ease function, which is related: #10572
https://github.com/godotengine/godot/issues/10572
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/10752#issuecomment-340112714,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z27xf_4qTEheqpqKsaLzamZ3naKaNks5swlnugaJpZM4PFvZ1
.

for frame by frame animations I prefer changing the Frames per second (fps) value like in AnimatedSprite or the animation speed with a Duration per frame (milliseconds) value like in Aseprite (even better)
AnimationPlayer is not meant to be used for frame by frame animations. Even if you give the option to change the speed with a single value, you will be dealing with moving keyframes through the timeline and that's not needed for sprite animations
AnimatedSprite node is better for that job but its lacks features (like the keyframing feature or sprite sheet support)
I suggest separating AnimationPlayer node to be focused only for what is made for, cutouts animations / tweening  and leaving AnimatedSprites node for frame by frame animations adding it keyframing feature
this way AnimationPlayer wont be overflowed with features that some people may never use (like me with sprite animations)
I opened an issue about this here it is #12469

A few more suggestions:

I think it should be clear from where every animation of an AnimationPlayer come from. Right now the animation list only displays animation names, but I'm never sure if they are built-in, from another scene or saved in the filesystem. I often end up modifing an animation in one AnimationPlayer and get another AnimationPlayer broken.
In support of point 1 I would like better copy/paste capabilities, like being able to choose between a shallow or deep copy.
Right now assigning an Object as a parameter in a callback in a Call Func Track doesn't work, drag and dropping a resource there locks the mouse in the middle of the window and a restart of Godot is required.

AnimationPlayer was rewritten from scratch for 3.1 and has tons of UX improvements. I'll close this issue as it's no longer relevant as is, but if some of its points are still relevant in the current master branch, feel free to open new issues specific to each of them.


Cannot set properties of null (setting '__vrv_devtools')

Environment

Operating System: Linux
Node Version:     v21.7.3
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager:  pnpm@
Builder:          -
User Config:      modules, app, auth, devServer, server, ssr, runtimeConfig, css, vite
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Reproduction
StackBlitz fork with important files
I can't share the entire project, but I'll be happy to send individual files of interest if it helps solve the problem
Describe the bug
After importing the vue-multiselect library, an error began to occur on every page of the site, this has happened dozens of times before, but the problem magically disappeared or was replaced by another more serious one (see Additional Context)
It looks like a problem with DevTools, but none of the attempts to disable it in development mode have been successful, both in terms of disabling DevTools and in terms of fixing the error.
Additional context
The problem only appears in the browser window, I tried to localize the individual file causing the problem. I can point out that the problem is not related to the /server and /pages directories and is most likely either in the dependencies or in the nuxt.config.ts configuration.
 The error occurs on the initial startup after deleting the node_modules/.cache/vite directory. However, the error does not occur when restarting the application.


Route Parameters No Longer Url Decoded

Is this a design change or a regression? I am seeing this in Beta8.
  [Route("api/values")]
  public class ValuesController : Controller
  {
    // GET: api/values
    [HttpGet]
    public JsonResult Get()
    {
      return Json(new string[] { "value1", "value2" });
    }

    // GET api/values/5
    [HttpGet("{id}")]
    public JsonResult Get(string id)
    {
      return Json(id);
    }

  }

For example, if I do a GET to /api/values/Hello%20World I get:

This is a breaking change from Beta 7. Just want to know whether I need to allow for this.

Seems more like Kestrel bug - when using Microsoft.AspNet.Server.WebListener, everything works fine. Looks like somewhere along the like the request path gets encoded again.

@shawnwildermuth - there's no intended change here, we need to look into this. @halter73

@rynowak and @Kukkimonsuta - I've replicated this behavior in Beta 7 with Kestrel. Clearly that's where the problem lies. It's not a new code change. I was using WebListener and IISExpress in the course before so that's why it worked. Once I moved the Beta 7 version and tried it in Kestrel it happened. Problem a bug. /cc @halter73

This is the same as https://github.com/aspnet/KestrelHttpServer/issues/124, right?

@Eilon  - looks the same, as long as we'll end up in RC with an unescaped parameter values, I'm happy.

Yep, we have an engineer working on it!

Hi @shawnwildermuth, I'm working on unescapes the Url path. As of now the fix is targeting rc1.

@troydai Perfect.  Thanks!

Closing


Optimise heap allocations for "generateSpendableKey"

While I was benchmarking skyd on one of our development servers, I noticed that 30%+ of all allocated objects were created in generateSpendableKey. We are talking about tens of millions of objects.
This MR is something I cooked up this morning to get rid of those allocations and it worked. Essentially I'm turning spendableKey into a value type by introducing a new UnlockCondition type which is a specific version of UnlockConditions containing only one key.
That way we only use value fields and arrays instead of keys with arbitrary slices.
Let me know what you think. There might be other ways that are less intrusive but this seemed like a quick low-hanging fruit.

My first reaction was "using value types means that secret keys will be copied around in memory, which is bad." My second reaction was "oh wait crypto.SecretKey is a value type so I guess we already do that everywhere." This will be definitely rectified in v2. (core does not have its own secret key type; we just use ed25519.PrivateKey.)
My third reaction is that we probably don't need a new types.UnlockCondition type. Instead, we can just inline the Timelock and PublicKey fields into wallet.spendableKey. (And while types.UnlockCondition is a cute name, it's a bit of a misnomer since technically there are two conditions -- the pubkey and the timelock. 😉)
And my fourth reaction is that we can further save allocations by writing a zero-alloc (spendableKey).UnlockHash() method, based on the us wallet.StandardAddress function.

There's a failing wallet test, but otherwise this looks good. Are you able to benchmark again with this code and observe a reduction in allocations?

Looks like this test broke:
  Error: test go.sia.tech/siad/modules/wallet/TestIntegrationLoad2of3Siag failed in 0.35s:
  === RUN   TestIntegrationLoad2of3Siag
      unseeded_test.go:79: wrong number of pubkeys
      unseeded_test.go:87: expecting a siafund balance of 7000 from the 2of3 key
      unseeded_test.go:93: insufficient balance
  --- FAIL: TestIntegrationLoad2of3Siag (0.35s)


gmdownload "Sorry, login failed" ssl.SSLError: [SSL: TLSV1_ALERT_ACCESS_DENIED] tlsv1 alert access denied (_ssl.c:600)

$ gmdownload  -f artist:Halsey
__init__() takes at most 4 positional arguments (5 given)

Visit the following url:
 https://accounts.google.com/o/oauth2/v2/auth?redirect_uri=urn%3Aietf%3Awg%3Aoauth%3A2.0%3Aoob&scope=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fmusicmanager&access_type=offline&client_id=XXXXXXXXXXX.apps.googleusercontent.com&response_type=code
Follow the prompts, then paste the auth code here and hit enter: xxxREDACTEDxxxx

Unable to login with specified oauth code.
Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/gmusicapi_wrapper/musicmanager.py", line 50, in login
    self.api.perform_oauth(storage_filepath=oauth_cred)
  File "/usr/local/lib/python3.4/dist-packages/gmusicapi/clients/musicmanager.py", line 97, in perform_oauth
    credentials = flow.step2_exchange(code)
  File "/usr/local/lib/python3.4/dist-packages/oauth2client/util.py", line 137, in positional_wrapper
    return wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.4/dist-packages/oauth2client/client.py", line 2103, in step2_exchange
    headers=headers)
  File "/usr/local/lib/python3.4/dist-packages/httplib2/__init__.py", line 1314, in request
    (response, content) = self._request(conn, authority, uri, request_uri, method, body, headers, redirections, cachekey)
  File "/usr/local/lib/python3.4/dist-packages/httplib2/__init__.py", line 1064, in _request
    (response, content) = self._conn_request(conn, request_uri, method, body, headers)
  File "/usr/local/lib/python3.4/dist-packages/httplib2/__init__.py", line 987, in _conn_request
    conn.connect()
  File "/usr/lib/python3.4/http/client.py", line 1287, in connect
    server_hostname=server_hostname)
  File "/usr/lib/python3.4/ssl.py", line 362, in wrap_socket
    _context=self)
  File "/usr/lib/python3.4/ssl.py", line 580, in __init__
    self.do_handshake()
  File "/usr/lib/python3.4/ssl.py", line 807, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLError: [SSL: TLSV1_ALERT_ACCESS_DENIED] tlsv1 alert access denied (_ssl.c:600)
Sorry, login failed.

Using Ubuntu 15.10 with gmusicapi-scripts installed as per instructions here:
https://thebigmunch.github.io/gmusicapi-scripts/install/
Any help on how to solve this one is much appreciated.
Thanks

The __init__() takes at most 4 positional arguments (5 given) is strange; I have no idea where that would come from at that point. Are you running this as root? If so, try running it as a regular user.

Closing for lack of response.


Shell export not working when password has special characters

Expected Behavior
The password when executing teller sh should be enclosed in quote to avoid interpreting special characters such as parentheses.
Current Behavior
Currently passwords are not escaped, resulting in the following error:
bash: syntax error near unexpected token '('
Possible Solution
Enclose password with quote and substitute single quote already in the variable
strings.ReplaceAll(v.Value, "'", "'\"'\"'")
I've a commit ready to be submitted as a PR: https://github.com/kpetremann/teller/commit/761009133fdb3a445703e0a3344276e78ab6f881
Steps to Reproduce

have this kind of secret: ()"';@  \(\)\"\'\;\@
eval "$(teller sh)"

Context
Secrets can have special characters which should not be interpreted by the shell
Specifications

Version: 1.5.3
Platform: Ubuntu 22.04

Hey @kpetreman, thanks for reporting this issue.
Do you want to open a pull request with your fix? We're accepting PRs

sure, I'll add it and then send the PR!

@kaplanelad I added the e2e test in the PR #132, don't hesitate if it needs adjustments

closing the issue as the PR has been merged and released.
Thanks for having merge the PR :)


Policies: fix the instance-template-reference policy and improve testing

Description
This PR improves the instance-template-reference policy to fix the problem mentioned in #415. Additionally, it introduces new tests to prevent regressions and ensure those cases are correctly handled (and slightly modifies the others to improve clarity, at least IMO).
Secifically, the fix involves two main problems:

The policy did not return an error in certain conditions if the referenced namespace had no templates.
The policy incorrectly returned an error if the referenced namespace contained multiple templates.

@ChiaraOggeri @sofymunari Could you have a look whether the modifications seem ok to you (seems I cannot specify you as reviewers until you accept the invitation)? I am definitely not a rego/gatekeeper expert.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[X] Unit testing (adding new tests regarding the identified problematic situations)
[X] Manual testing with a kind cluster, replicating the templates and instances currently present in CrownLabs.

/rebase

/merge


fix: function get_http_auth() is on self object not self.client

self in this context is already the client so no need for the extra call to client.

Thanks Pablo
On Fri, Sep 24, 2021, 7:16 AM PJEstrada @.***> wrote:

self in this context is already the client so no need for the extra call
to client.
You can view, comment on, or merge this pull request online at:
https://github.com/diffgram/python-sdk/pull/17
Commit Summary

fix: function get_http_auth() is on self object not self.client
https://github.com/diffgram/python-sdk/pull/17/commits/05708ec9a18bb327a2616f8ab7ce55e096f0c97e

File Changes

M sdk/diffgram/file/view.py
https://github.com/diffgram/python-sdk/pull/17/files#diff-56bb42893f79ccf2b72a7eec2ac814fffb802277f5ff8d1868b366cc9932b8cb
(2)

Patch Links:

https://github.com/diffgram/python-sdk/pull/17.patch
https://github.com/diffgram/python-sdk/pull/17.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/diffgram/python-sdk/pull/17, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEJ6DJCENFBGM3AU37AYWETUDSB3LANCNFSM5EWB5EJQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Supervisor start error.

I did all the configuration and then
docker run -d -p 80:80 -v $(pwd):/code --env DJANGO_PRODUCTION=false /django-docker
On running docker, the container is spawned, but supervisor cannot start the service. I am getting following error:
2016-05-01T22:05:58.420603194Z 2016-05-01 22:05:58,420 INFO exited: initialize (exit status 127; not expected)
2016-05-01T22:05:59.423536295Z 2016-05-01 22:05:59,423 INFO spawned: 'initialize' with pid 102
2016-05-01T22:05:59.427550188Z 2016-05-01 22:05:59,427 INFO success: initialize entered RUNNING state, process has stayed up for

than 0 seconds (startsecs)
2016-05-01T22:05:59.428110966Z 2016-05-01 22:05:59,428 INFO exited: initialize (exit status 127; not expected)
2016-05-01T22:06:00.431900979Z 2016-05-01 22:06:00,430 INFO spawned: 'initialize' with pid 103
2016-05-01T22:06:00.435845297Z 2016-05-01 22:06:00,435 INFO success: initialize entered RUNNING state, process has stayed up for
than 0 seconds (startsecs)
2016-05-01T22:06:00.436428078Z 2016-05-01 22:06:00,436 INFO exited: initialize (exit status 127; not expected)
2016-05-01T22:06:01.439192191Z 2016-05-01 22:06:01,438 INFO spawned: 'initialize' with pid 104
2016-05-01T22:06:01.443373806Z 2016-05-01 22:06:01,443 INFO success: initialize entered RUNNING state, process has stayed up for
than 0 seconds (startsecs)

I looked into initialize.sh script but couldn't figure out the issue. Can you please help me debug this?

Fixed in https://github.com/morninj/django-docker/pull/21


ARROW-327: [Python] Remove conda builds from Travis CI setup

We'll do these builds in conda-forge

+1


Add the subscription diagnostic settings API

Initial checkin
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[x] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

@jagadisk let me know if no further changes is needed and I can merge it.

No further changes needed. Thanks.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Feng Zhou<EMAIL_ADDRESS>Sent: Wednesday, December 4, 2019 6:15:28 PM
To: Azure/azure-rest-api-specs<EMAIL_ADDRESS>Cc: Jagadish Kulkarni<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-rest-api-specs] Add the subscription diagnostic settings API (#7873)
@jagadiskhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjagadisk&data=02|01|jagadisk%40microsoft.com|0075979480cd49a97c2208d779290007|72f988bf86f141af91ab2d7cd011db47|1|0|637111089315414009&sdata=%2FVtQUJcC7ihqj6jSlqWvKaUmhwK9cXobM9TKsGqMKTA%3D&reserved=0 let me know if no further changes is needed and I can merge it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-rest-api-specs%2Fpull%2F7873%3Femail_source%3Dnotifications%26email_token%3DABILIXBX6HUDFZISBPJQGLLQXBP4BA5CNFSM4JUQZVA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF7HHDI%23issuecomment-561935245&data=02|01|jagadisk%40microsoft.com|0075979480cd49a97c2208d779290007|72f988bf86f141af91ab2d7cd011db47|1|0|637111089315414009&sdata=3sBvHW8AmYPh88LoEArl3g7SqXZi%2FZIsJbbOcCj9bbg%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABILIXC7X5VWYOPQFSQDA43QXBP4BANCNFSM4JUQZVAQ&data=02|01|jagadisk%40microsoft.com|0075979480cd49a97c2208d779290007|72f988bf86f141af91ab2d7cd011db47|1|0|637111089315424009&sdata=sA4tvFdfctx%2FkgmxmA4yB3uwdooL%2BUx9NIiLHR8T6zo%3D&reserved=0.

/azp run

Hi @jagadisk ,
I don't see any differences between this new swagger and
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/monitor/resource-manager/Microsoft.Insights/preview/2017-05-01-preview/diagnosticsSettings_API.json
It looks like you defined two swaggers for one API, can you help understand why is this needed?


saurabh's contribution to hackoberfest 2021

What type of PR is this? (check all applicable)

[x] 🚀 Added Name
[x] ✨ Feature
[x] 🌟 stared the repo
[ ] 🐛 Grammatical Error
[ ] 📝 Documentation Update
[ ] 🚩 Other

Description
Add Link of GitHub Profile

all good


refactor config parameters

Hello,
I've updated your files with a new architecture  :
Core directory
You can find here your global settings and database settings.
It is helpful to use such a file because each time you'll update your database name / user etc. you won't have to rename it in all your project files.
Database files
I also added mysqli instead of deprecated mysqlconnect. You can use your mysqli object from any file using require_once' ' method.
###Next step?
Maybe we can try to comply with PSR-4, as using autoloader ...

Your efforts and time spend on this is appreciated.

Maybe your method of configuration can be added in a separate folder that tells this is another way of configuration. @younesdiouri

Thank you for your feedback. Indeed this can be a second step for a beginner. I will reroll it with all this content in a separate folder.

@younesdiouri check the section files changed once.

@shauryauppal you are right, I'll just redo a PR then?

You can check on stackoverflow how to drop a commit. git rebase -i HEAD~n can help you in this.

@younesdiouri You just need to remove your previous commits in PR. If you are unable to make it you may make a new PR, I will close this one and merge that PR.

@younesdiouri Great work merging, but please add a README fill in your new FOLDER also how would someone understand.

@younesdiouri Explain what new You did how is it different than the first method.


Missing webhook rbac (backport)

This PR is a backport of the fix in #751 to correct this problem on the release-0.24 branch which is also broken.
Release Note
- 🐛. Add missing RBAC (get namespaces) to KafkaChannel Webhook ClusterRole to satisfy new implementation.

/lgtm


Remove request log configuration #62

Remove request log configuration #62

I'm trying to test this change, but keep running into an error reading root.war.
I have this add instruction:
ADD testapp.war $JETTY_BASE/webapps/root.war

And I get this error because root.war is owned by root, not jetty user.
java.io.FileNotFoundException: /var/lib/jetty/webapps/root.war (Permission denied)

I'm not sure why this error doesn't occur when using currently deployed gcr.io/google_appengine/jetty image.
I'll come back to this next week to investigate further.


Update abstract-provisioning-cache.js to use jws instead of jsonwebtoken

While doing some stuff with gimme-jwt, I noticed that this file was still using a dependency, jsonwebtoken, that was removed a few commits ago and replaced with jws.
Updated the file to use the new dependency.
@omsmith

Oh shoot! thanks


blueprint: move pipeline generation into its own package

NOTE I called the new package os, but that clashes with the os from the standard library. I noticed that when using the new API in composer itself. I've aliased the name were this happens, but would prefer a different name, I think. Any suggestions?
@ondrejbudai I moved the customizations themselves into the new fedora30 package, because they might be OS-dependant, but kept the types in the blueprints package. Do you think that makes sense?

Introduce the os package, which contains an interface for OS
implementations. Its main purpose is to convert a blueprint to a
OS-specific pipeline.
Also introduce the os/fedora30 package. It is the first implementation
of the OS interface. Most of its code has been copied with minimal
modifications from the blueprint package.
The blueprint package is now back to serving a single purpose:
representing a weldr blueprint. It does not depend on the pipeline
package anymore.
Change osbuild-composer and osbuild-pipeline to use the new API,
hard-coding "fedora-30". This looks a bit weird now, but is the same
behavior as before.
All test cases now also take an "os" key in the "compose" object.

@ondrejbudai I moved the customizations themselves into the new fedora30 package, because they might be OS-dependant, but kept the types in the blueprints package. Do you think that makes sense?

Ya, definitely makes sense.

Ya, definitely makes sense.

Thanks!

I've aliased the name were this happens, but would prefer a different name, I think. Any suggestions?

@teg suggested distro. I like that.

@teg suggested distro. I like that.

:+1: from me, this also came to my mind.

Update to rename os to distro and add the panic @ondrejbudai was asking for.


Provide a way to configure the SA engine

Hi all,
Unless I'm mis-reading the code, there is no way to provide engine creation options. One of them that appears with SA 1.2 is pool_pre_ping.
I'm not sure I can provide extra parameters via flask-sqlalchemy parameters. Should I create the engine out of band?
Thanks,

I spent 4 days trying to get to the bottom of the issue (Sql server has gone away), and though 'pool_pre_ping': True is NOT working properly, but i now realized that the engine_options were not loaded into my db instance. Here is what I was trying to do (as seen in many flask example projects), I was adding these SQLALCHEMY_ENGINE_OPTIONS via configs (shown in docs)
class DevConfig():
    ...
    DEBUG = True
    SQLALCHEMY_ENGINE_OPTIONS = {'pool_recycle': 280, 'pool_timeout': 100, 'pool_pre_ping': True}
    SQLALCHEMY_DATABASE_URI = os.getenv('SQLALCHEMY_DATABASE_URI_DEV')
    SQLALCHEMY_TRACK_MODIFICATIONS = False

and using these configs when init app
from flask_sqlalchemy import SQLAlchemy
db = SQLAlchemy()

instance_name = os.environ.get("FLASK_ENV", "dev")
app = create_app(app_config[instance_name])
# --------
def create_app(config):
    app = Flask(__name__, instance_relative_config=True)
    app.config.from_object(config)

    # INIT DATABASE
    db.init_app(app)
    with app.app_context():
        db.create_all()

I thought this would work, but it didn't. Finally @Lawouach answer worked fine.

apply_pool_defaults
Thanks for the snippet @Lawouach
I had issues running that code. The error I was getting apply_pool_defaults() takes 3 positional arguments but 4 were given
The below one worked for me:
class SQLAlchemy(_BaseSQLAlchemy):     def apply_pool_defaults(self, app, options):         super(SQLAlchemy, self).apply_pool_defaults(app, options)         options["pool_pre_ping"] = True

These solutions do not work for me. They do not seem to have any effect. Shouldn't the options be updated before super is called?

We were experiencing the following error when using MySQL (pymysql) with Flask/SQLAlchemy
(pymysql.err.OperationalError) (2013, 'Lost connection to MySQL server during query ([Errno 104] Connection reset by peer)')

We found that the combination configuration of the following worked for us, eliminating the error entirely. Mostly mentioning the error here so anyone who suffers from a similar issue will find this thread easily.
from flask_sqlalchemy import SQLAlchemy as _BaseSQLAlchemy

class SQLAlchemy(_BaseSQLAlchemy):
    def apply_pool_defaults(self, app, options):
        super(SQLAlchemy, self).apply_pool_defaults(app, options)
        options["pool_pre_ping"] = True

db = SQLAlchemy()

app.config['SQLALCHEMY_POOL_RECYCLE'] = 28000
app.config['SQLALCHEMY_POOL_TIMEOUT'] = 20000
app.config['SQLALCHEMY_POOL_SIZE'] = 10
app.config['SQLALCHEMY_MAX_OVERFLOW'] = 0


Switching to Full Screen Disables HDR In Windows 10

Godot version:

Godot v3.3-rc9
OS/device including version:

Windows 10, Nvidia 2080 Ti, Driver 461.09, GLES2/3
Issue description:

When using an HDR enabled monitor on Windows 10, switching to full screen in a Godot project causes HDR to be disabled.
https://user-images.githubusercontent.com/3398643/115322900-e316d980-a13b-11eb-9a8a-bbaa4798f211.mp4
Steps to reproduce:
Create a new project and call "OS.window_fullscreen = true" it goes fullscreen but then disables HDR in Windows 10 (and remains disabled even after closing the game). With the minimal project I attached, just press the F key (though you need to be testing with an HDR monitor).
Minimal reproduction project:

HDR Test.zip

Godot doesn't support HDR output yet, so this is likely expected.

That makes sense. I wonder though if there is a reason HDR becomes disabled. For example, I can play old DX9 games or even OpenGL and they don't revert the setting like Godot does. I agree that this would make more sense for 4.x. Thanks.

Should be related to the setCollectionBehavior not being called for non hiDPI mode, there was a bug in the some of the older OS versions, most likely it's not relevant anymore and it can be called for both hiDPI and loDPI modes.
https://github.com/godotengine/godot/blob/8e68f2e5f48c66bfa80b9bb7746a488afb70086d/platform/osx/os_osx.mm#L1608-L1614

Personally I would be very much interested to publish my game(s) in HDR: Nearly each TV now supports HDR10+ or Dolby Vision. More and more other devices also use OLED or Mini-LED screens, e.g. iPhone (OLED), iPad Pro (Mini LED), Macbook Pro (Mini LED), Nintendo Switch OLED

@MickL Please don't bump issues without contributing significant new information. Use the :+1: reaction button on the first post instead.


Revert removal of old package names from BREAKING.md

https://github.com/microsoft/FluidFramework/pull/2969 replaced the old package names with the new ones in BREAKING.md, making the note about the branch rename incorrect. This change restores the old package names in BREAKING.md.

I think that issue reference is incorrect, can you update it, and then i'll merge it.

I think that issue reference is incorrect, can you update it, and then i'll merge it.

I did not reference any issues (At least I didn't intend to). I referenced a Pull Request which was merged, that included this diff which is being reverted as part of: https://github.com/microsoft/FluidFramework/pull/2969/files#diff-2ca7cfddacad8b7deb1ca47ca14b05b62307e07c545d0459400b51f2f5c93f81 . This was found using blame on BREAKING.md.

you're right. didn't think that was the PR for some reason. Thanks!


Episode information

Hello, could you help me figure out how to segment the dataset into trajectories/episodes?
For example, for the halfcheetah datasets, there are no terminal states included, so I assumed first 1000 datapoints correspond to the first episode, the next 1000 the second episode, etc.
The qlearning_dataset function from
https://github.com/rail-berkeley/d4rl/blob/19ff42dfca15a7ecef38c380711da5164a86e26f/d4rl/__init__.py#L38
seems to support my guess.
But then when I look into the observations more closely, I get, for example for the halfcheetah-medium-v0 environment,
print(obs[9000])
array([-0.03586214, -0.11854211, -0.5709649 ,  0.59237313,  0.25799525,         0.24009612,  0.04685834, -0.180547  ,  3.6944256 ,  1.2065067 ,        -1.4099166 ,  1.4494822 , -6.9591885 , 15.26705   ,  3.7576292 ,         7.2117867 , -9.449336  ], dtype=float32)
Or
print(obs[10000])
array([ -0.09099437,   0.11028791,   0.32212004,  -0.32907683,          0.66518635,  -0.03972699,   0.13382532,  -0.56899893,          6.4752765 ,   0.26289114,  -2.9174874 ,  19.897604  ,        -20.984861  ,   4.177299  ,  20.101353  ,  -6.1993895 ,          1.4919064 ], dtype=float32)
These do not seem right for initial observations. AFAIK, initial observation for halfcheetah is a concat of qpos[1:] and qvel, which, upon reset, are small uniform noise around 0 and standard gaussian noise around 0, respectively.
Any help would be greatly appreciated. Thanks!

The trajectories are length 999, so obs[998], obs[1997], obs[2996], etc. should be the starting states for cheetah.
We're considering updating the environments with done flags for the MuJoCo tasks, so I'll keep this issue open for now.

We've added an extra "timeouts" field you can access using the get_datasets() method which should indicate when episodes end due to hitting the max episode length. The 'terminals' field indicates an episode ending due to a termination condition (such as the Walker2D falling over).
We're keeping these fields separate because some Q-learning methods don't use done=True for timesteps ending due to episode length.


coverage.reporter with a single value causes error while running tests without coverage (v2.0.x)

Describe the bug
The documentation of coverage.reporter on both V2 and V1 say that we can use string as a valid value.
So a minimal vitest.config like this, should work
export default defineConfig({
  test: {
    coverage: {
      reporter: 'json',
    },
  },
})

This works fine with 1.6, but not in 2.0.x.
Error printed while executing pnpm run test. The error is not present while running pnpm run test --coverage

Reproduction
A minimal reproduction case in this repository ildede/vitest-single-reporter-error
System Info
System:
    OS: macOS 14.5
    CPU: (8) arm64 Apple M1 Pro
    Memory: 116.05 MB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.9.0 - ~/Library/pnpm/node
    npm: 10.1.0 - ~/Library/pnpm/npm
    pnpm: 9.5.0 - ~/Library/pnpm/pnpm
  Browsers:
    Chrome: 127.0.6533.88
    Safari: 17.5
  npmPackages:
    vitest: ^2.0.5 => 2.0.5

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

We have test case for all reporter option types but all those run with coverage: { enabled: true }. This error case seems to come up only when coverage is disabled.
https://github.com/vitest-dev/vitest/blob/a7be50dbceea31611403542caca7ebf44e4e014a/test/coverage-test/test/reporters.test.ts#L7-L18
This might have been caused by refactoring done during https://github.com/vitest-dev/vitest/pull/6205.

If anyone is wondering how to work around this (like me) until 2.1.x gets released, wrapping the reporter string in an array should do the trick. So from:
coverage: {
       reporter: 'json',
}

to:
coverage: {
       reporter: ['json'],
}


[fuzzer] update command for generate coverage report

Motivation
fix doc
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan
eyes

/land


fix(masthead): ensure sectionUrl is passed in and has default value

Related Ticket(s)
[Cloud Masthead]: Mobile menu items are not rendering #8082
Description
Pass in object literal as argument for function rendering left nav menu items. By doing this and setting a default value, it's easier to keep track of these parameter between masthead-composite and cloud-masthead-composite
Changelog
Changed

pass object literal as arg to _renderLeftNavMenuSections()

@annawen1 I closed and re-open the PR to trigger a deploy preview for the WC experimental so that we can ensure that the menu items are displaying correctly for the Cloud Masthead.


Update example for PureScript 0.9.3

Thanks for this, but since 0.10 came out I thought I'd merge the PR with 0.10 updates instead.


[ffmpeg] Fix embedding subtitles (#9063)

Change command line parameters for ffmpeg when embedding subtitles.
Change to ‘-map 0:v -c:v copy -map 0:a -c:a copy’

Sorry for the late response. Any broken example before this fix?

Hi, sorry, this is to fix #9063. As discussed in that issue, this is caused by certain lynda videos having empty streams.
The changes to the parameters enables ffmpeg to pick the correct working streams in those lynda videos, including other videos that come along with empty streams.

From http://ffmpeg.org/ffmpeg.html:
stream_type is one of following: ’v’ or ’V’ for video, ’a’ for audio, ’s’ for subtitle, ’d’ for data, and ’t’ for attachments.

Honestly I don't know what are data and attachments in media files, and I haven't seen any example with them. I guess not copying them does not cause problems :)


Add tldr to each tutorial

On top of each tutorial, add a little summary of the main takeaways and topics of each tutorial (in terms of features of VPL). This should help users decide whether it is worth reading the tutorial or not.

Covered by issue 2.