connection string

There are currently 2 options for defining connection:
1 no connection parameter: a global SerialManager will be created
a = ArduinoApi()

2 connection object as parameter
connection = SerialManager()
a = ArduinoApi(connection=connection)

Proposed new methods:
3 connection string
a = ArduinoApi(connection='serial:port=/dev/ttyUSB0,baudrate=9600')

4 connection string in environment variable
NANPY_CONNECTION=serial:dev=/dev/ttyUSB0,baud=9600
other examples:
NANPY_CONNECTION=serial:port=0, timeout=3
NANPY_CONNECTION=serial:port=COM3
NANPY_CONNECTION=mock:MCU=ATMEGA88
NANPY_CONNECTION=null

This makes possible that Python scripts can be run on differrent computers without changing the code.
It is also good for testing.

LGTM! Go for it :)


Device : Heater NEDIS

Hi
Thanks for your wonderful work :)
I have a NEDIS heater : https://nedis.com/en-us/product/home-living/climate/heaters/550711594/smartlife-convection-heater-wi-fi-suitable-for-bathroom-glass-panel-2000-w-2-heat-settings-led-display-15-35-0c-adjustable-thermostat-white?_ga=undefined
Here you can find journal :
Cette erreur provient d'une intégration personnalisée
Logger: custom_components.tuya_local.config_flow
Source: custom_components/tuya_local/config_flow.py:73
Integration: Tuya Local (documentation, issues)
First occurred: 22:36:11 (1 occurrences)
Last logged: 22:36:11
Device matches poolex_silverline_heatpump with quality of 62%. DPS: {'1': True, '2': 25, '3': 25, '4': '1', '7': False, '11': '0', '13': 0, '101': False, 'updated_at':<PHONE_NUMBER>.809125}
2 = set temp
3 = actual temp
I really need 5 functions :

ECO mode
COMFORT mode
OFF
Actual temp
Set temp

Thanks a lot :)

Actually working :

Set temp
Current temp
Off
Eco mode (heat)

So only Comfort mode is missing

Are you able to find out the values for dps 4 for Comfort and Eco modes?
Are there other features such as a Child Lock, Timer, or anything else?

Tanks for your reply
I will try tomorrow
There are some others features (timer, Child lock, timer prog)

OK, One of the False values (7 or 101) will be the child lock in that case (it didn't show on the screenshot on the webpage).
The daily scheduling is likely cloud based, and is handled by Home Assistant Automations.  If it is device based, it relies on a complex long hexidecimal string, probably in dps 11 which is too hard to work with, so I've skipped it in the devices I've identified as supporting it so far.

Hi
I test for DPS 4 :
1 = Eco
2 = Comfort
I just delete device, active comfort mode, and proceed to integration
Device matches poolex_silverline_heatpump with quality of 62%. DPS: {'1': True, '2': 22, '3': 20, '4': '1', '7': False, '11': '0', '13': 0, '101': False, 'updated_at':<PHONE_NUMBER>.0292742}
= Device active on Eco
Device matches poolex_silverline_heatpump with quality of 62%. DPS: {'1': False, '2': 22, '3': 21, '4': '1', '7': False, '11': '0', '13': 0, '101': False, 'updated_at':<PHONE_NUMBER>.1233184}
= Device off
Device matches poolex_silverline_heatpump with quality of 62%. DPS: {'1': True, '2': 23, '3': 22, '4': '2', '7': False, '11': '0', '13': 0, '101': False, 'updated_at':<PHONE_NUMBER>.7162051}
= Device active on Comfort

Modifications done :
  mapping:
    - dps_val: "1"
      value: "Eco"
    - dps_val: "2"
      value: "Confort"
    - dps_val: "3"
      value: "Hors gel"

👍


MuiAlert

@Antony-M1 or @manikandanmohan10  Please review to code and merge into the main branch

@antony-mpokket  the #26 merged to the  main branch


Use allocatable instead of capacity for node resources.

Fixes #2097.
The Cluster and Node Memory metrics are not intuitive. The statistics shown are using kube_node_status_capacity instead of kube_node_status_allocatable which can be highly misleading for certain cloud providers/node VM sizes (due to provider specific resource reservations described here). In my circumstance using Azure Kubernetes Service with Standard_B2s results in a reported utilization of ~55% when it is actually ~98% according to kubectl describe node. This difference makes it very difficult to understand why a cluster would report insufficient memory when scheduling pods. Memory consumption relative to the cluster or node's total Allocatable Memory is a much more useful metric for most users who are generally only concerned with the resources available for consumption by pods/workloads. This is supported by kubectl's default reporting of node memory (i.e. relative to the allocatable memory).

Would it make sense to have both? Is there a reason why podCapacity is not down based on "allocatable" as well?

Ah, I didn't think PodCapacity had an "allocatable" allegory, but it appears I was wrong according to kubectl describe node's output.

Would it make sense to have both?

I don't think it doesn't make sense to include both. In the case of Azure Kubernetes Service, not all of that total capacity is accessible for use by workloads, and Lens is only able to report on usage/requests/limits by proper workloads. The portion that isn't accessible to workloads is consumed by "other processes" like kubelet itself, and the memory usage of those other processes isn't something Lens can access afaik.
TL;DR; Since Lens can't account for the usage of that reserved memory, I believe it would only be confusing to report on that capacity.

I've had a change of heart. In debugging locally, I realized that there are scenarios in which the allocatable memory capacity paints an incomplete picture also. I'm going to convert this PR into a draft while I come up with a way to accurately represent cluster/node memory conditions for all scenarios.
Basically, I was operating under a false premise that the "allocatable" memory was reserved for workloads just as the "non-allocatable" memory is reserved for kubelet and other  "non-workloads", but that isn't the case at all. That means that there are circumstances where the memory actually available for workloads is less than the "allocatable" memory (especially in clusters that don't really reserve memory for kubelet at all, allocatable == total capacity).
I am thinking that means that it would be most accurate to display a breakdown of memory that distinguishes between "workload" and "non-workload" (things like kubelet) consumption as well as displaying "allocatable", "non-allocatable" and "total capacity".

Did you get anywhere with this? Its nearly really closely bitten us.

I've had a change of heart...

I think how you phrased it makes perfect sense and I found this issue looking around about my AKS cluster so I may be biased, but the allocatable value seems to make much more sense for what I think people using lens are interested in or would expect, right?

especially in clusters that don't really reserve memory for kubelet at all, allocatable == total capacity

I don't know how all different flavors and deployments of Kubernetes do this but if that's the worst case then wouldn't this change as you initially implemented it not really impact those folks?  I imagine there are no sane deployments where allocatable > total node memory but even if there were (maybe some dynamic-memory VM?) wouldn't showing the allocatable still feel like it makes more sense, assuming the person specifically configuring it that way may best know the specifics of their environment?
It seems like deploying this would be a reasonable interim/incremental improvement, so I guess I'm trying to suggest or ask that you ship it.
I can appreciate not wanting to introduce churn if you feel like you're going to have to redo this commit at some point in the near future, but in the interim it's very difficult to use lens to get an accurate view of my AKS cluster.

Where are we with this? We've had some nasty surprises on our production clusters with this bug

Sorry, I didn't realize there was interest in this issue/PR because github notifications were going to my junk folder for some reason.
I had set this PR aside when I realized the relationship between Allocatable/Non-Allocatable/Workload/Non-Workload resources were more complex than I thought, but I will try to take another look at how to display a more accurate representation of node resources in the next week or so.

Thanks, I appreciate you taking the time for another look.
The AKS memory situation on smaller nodes is a bit silly, 4GB VM only has 2.2GB allocatable so I see evictions when it looks like I'm only using ~60% of a node's memory.  Honestly, this is more of an AKS problem than a Lens problem but I'm sure AKS users everywhere would rejoice with some sort of fix.

The AKS memory situation on smaller nodes is a bit silly, 4GB VM only has 2.2GB allocatable so I see evictions when it looks like I'm only using ~60% of a node's memory. Honestly, this is more of an AKS problem than a Lens problem but I'm sure AKS users everywhere would rejoice with some sort of fix.

That was exactly our scenario. We were using the Standard_B2s class also when we came across this and it took some time to figure out so when I finally did I flew around the internet raising issues on all the tools that failed me. I raised this to Microsoft because even their graphs in the Azure Portal are misleading in the same way. They reached out to me to ask some questions (which I thought was encouraging at the time), but it doesn't seem like they are going to address it. I think I made an issue on the Kubernetes Dashboard project on GitHub also but I don't recall if I kept up with that one either 😬.

Okay, it took some minor surgery to catch up to master, but I realized I actually had some (uncommitted!!) changes locally that I think actually portrays things pretty accurately. I must have made them months ago though, so I don't recall if I tracked down all the locations where the memory graphs show up in the UI to make sure they all make sense.
I think I am comfortable setting this as ready for review at this point. 🤞

Really good to read this grabbing traction again.
I agree it’s more of an AKS translation issue than lens, but we spend our days in lens monitoring (fab fab tool) so it would save us running frequent commands.
I imagine there’s many like use that got bit by thinking we had more resources than we did.
Thanks!
Sent from my iPhone

On 21 May 2021, at 23:40, mcpappas @.***> wrote:

Okay, it took some minor surgery to catch up to master, but I realized I actually had some (uncommitted!!) changes locally that I think actually portrays things pretty accurately. I must have made them months ago though, so I don't recall if I tracked down all the locations where the memory graphs show up in the UI to make sure they all make sense.
I think I am comfortable setting this as ready for review at this point. 🤞
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

@mcpappas So far this looks good, we plan on merging #2707 so this PR will need to change. Just FYI since it hasn't been merged yet.

@Nokel81 Thanks for the heads up, I’ll take a look when I get a chance.

@mcpappas Once the conflicts are resolved this should be good to go.

@mcpappas Please fix the conflicts.

I took a stab at resolving conflicts: https://github.com/kymaria/lens/tree/feature/allocatable
I'm unable to test locally...  I can't figure out how to install the lens-metrics prometheus resources in lens 5 and neither the beta 8 build or my development build of the branch above seems to be able to connect to my custom prometheus deployments.
But I hope this helps, the merge conflicts seemed pretty straight-forward to resolve and I'd really like to see this moved along in time for 5.0.

@jonstelly, @Nokel81, sorry, work has kept me very busy lately so I haven’t had time to take a stab myself. I can run Jon’s changes locally and adopt them if it checks out. I’ll shoot to take a look this afternoon.

No need to apologize, I appreciate all your work on this.


Tweak server configurations.

Enable auto-timeouts. Remove unnecessary server configuration in invoker.

PG2/986 approved.

@markusthoemmes you'll want to review this again - I made some changes to limit the entity names.

@markusthoemmes you'll want to review the last two commit (limiting entity name and test for max url path).
PG3/230 for the latest commits.

pg3/234 for the latest.


fix: upgrade to version 3.0.570

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign pmuir
You can assign the PR to them by writing /assign @pmuir in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
f1bc526e44db1c1bb78bd8467dbb39e6494f033d
link
/test vault

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
f1bc526e44db1c1bb78bd8467dbb39e6494f033d
link
/test vault

gsm
f1bc526e44db1c1bb78bd8467dbb39e6494f033d
link
/test gsm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.

/retest


Added documentation for UI Project

Overview
Created documentation for Angular portion of the application.  Helps new developers quickly gain context of the application organization.
Testing Instructions
Just read the documentation and correct it if you see something errant.

Good Catch @adandach!  Readme updated


Installing the pyrouge on Microsoft Windows

FYI, I've had many issues to install the Python package pyrouge on Microsoft Windows, so I wrote a tutorial:
How to install the  pyrouge on Microsoft Windows?

Thanks a lot, this looks really useful!
I added a link to your guide in the readme.


mat-slide-toggle using incorrect text color in dark-theme

Reproduction
Steps to reproduce:

Add mat-slide-toggle to menu in dark-theme

<mat-menu #userMenu="matMenu">
  <button mat-menu-item (click)="logout()">Logout</button>
  <mat-slide-toggle
     [(ngModel)]="darktheme"
     (change)="changeTheme($event)">Dark Theme
  </mat-slide-toggle>
</mat-menu>

Expected Behavior
What behavior were you expecting to see?
Expect mat-slide-toggle text to be white in dark-theme
Actual Behavior
What behavior did you actually see?
mat-slide-toggle text is black in dark-theme

Environment

Angular: 9.0.4
CDK/Material: 9.0.1
Browser(s): Firefox, Edge (chromium)
Operating System (e.g. Windows, macOS, Ubuntu): Windows

When I tested this using the MDC based slide-toggle and menu components which are coming soon in v15, the text color is correct


[Feature Request] Be able to obtain trace propagation HTTP headers to manually propagate traces

Hi team! My team has been using a custom built DataDog React Native module that bridges the iOS/Android SDKs and just has the bare functionality that we need while we waited for this official DataDog React Native SDK. One thing we need is to be able to distribute traces from frontend to backend, so we followed the docs here: https://github.com/DataDog/dd-sdk-ios/blob/master/docs/trace_collection.md (see number 8). For each HTTP request we make in RN (we're using Apollo client graphql) we called a RN bridge function that, on the native side, would first start the span and use that HTTPHeadersWriter (iOS) or HttpHeadersInjectAdapter (Android) to obtain the headers, important one being the trace ID, and we returned that to JS over the bridge so that we could set the x-datadog-trace-id header on each request in an Apollo link. Then the span is stopped at the end of the request.
I have been checking out the beta of this SDK to see if we can start using it and get rid of our custom bridge, but it seems like there isn't yet a way to be able to do this manual propagation of traces, so we'd like to request this feature for a future version of the SDK!
If this is already possible with the SDK and I missed something please let me know!

Hello @JessicaYeh. As of 1.0.0-beta3 we support traces propagation for XHR-based requests for the apps running on Android (not supported yet on iOS, but it will be there as well very soon). We didn't look into compatibility with Apollo Client yet, but if it is using XMLHttpRequest under the hood, then it will work (see here for the details). The only limitation is that network trace cannot be a continuation of existing trace, so network span will be at the root of the trace.
To make it work you need to explicitly enable resources tracking in the SDK configuration as stated here.
Since you already have an existing app with custom wrapper, I would suggest you to check our sample app first: just put your credentials there and replace some URL with the URL of your backend to see how it works.
Let us know if existing functionality in our React Native SDK is enough or not for your needs.

Hello @JessicaYeh. We didn't hear from you back, so I hope solution above works for you and I'm closing the ticket. Feel free to re-open it if needed.

Hi @0xnm,
I would like to see this feature also... One reason why we would need this is that there might be multiple libraries which are doing the same - overriding default behaviour of XMLHttpRequest


converter: add buffer for io.Writer

add buffer for io.Writer to improve
nydus image conversion performation.
This PR is addressing #128
Signed-off-by: Bin Tang<EMAIL_ADDRESS>

Codecov Report
Base: 29.87% // Head: 29.98% // Increases project coverage by +0.10% :tada:

Coverage data is based on head (e007547) compared to base (d6588e8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
+ Coverage   29.87%   29.98%   +0.10%
==========================================
  Files          20       20
  Lines        1647     1651       +4
==========================================
+ Hits          492      495       +3
- Misses       1090     1091       +1
  Partials       65       65

Impacted Files
Coverage Δ

cmd/containerd-nydus-grpc/pkg/command/flags.go
73.95% <0.00%> (-1.41%)
:arrow_down:

pkg/filesystem/fs/fs.go
1.15% <0.00%> (ø)

pkg/filesystem/fs/blob_manager.go
55.55% <0.00%> (+11.11%)
:arrow_up:

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.


CIS: Ensure suspicious packets are logged

Please back-port to

ubuntu-xenial/456.x
ubuntu-xenial/315.x
ubuntu-xenial/250.x
ubuntu-xenial/170.x
ubuntu-xenial/97.x

Not sure if it was this PR or #105, but please make sure the tests are green.
Merged manually. Will carry over to other branches in a second.


Switch to asynchronous methods

fs
glob

What would probably speed things up a lot would be if this was async https://github.com/svg-sprite/svg-sprite/blob/59d7c3e676e83dd6285dc944f7774d8716268aa5/bin/svg-sprite.js#L280
We need to test this thoroughly (and on Windows) since we might hit a limit though.

@XhmikosR can you please check the builder in https://github.com/svg-sprite/svg-sprite/compare/async-bin branch
In my measurement the build time was slightly increased (about +0.1s)
Maybe it's the OS thing (I'm using Mac M1)

I haven't looked at the changes but here are the results from my dev VM (main vs your branch):
C:\Users\xmr\Desktop\bootstrap-icons>hyperfine "npm run icons-sprite"
Benchmark 1: npm run icons-sprite
  Time (mean ± σ):      7.077 s ±  0.269 s    [User: 0.003 s, System: 0.008 s]
  Range (min … max):    6.896 s …  7.821 s    10 runs

C:\Users\xmr\Desktop\bootstrap-icons>npm i github:svg-sprite/svg-sprite#async-bin -D

changed 1 package, and audited 777 packages in 6s

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

6 moderate severity vulnerabilities

Some issues need review, and may require choosing
a different dependency.

Run `npm audit` for details.

C:\Users\xmr\Desktop\bootstrap-icons>hyperfine "npm run icons-sprite"
Benchmark 1: npm run icons-sprite
  Time (mean ± σ):      6.866 s ±  0.051 s    [User: 0.001 s, System: 0.008 s]
  Range (min … max):    6.792 s …  6.936 s    10 runs

Another benchmark:
C:\Users\xmr\Desktop\bootstrap-icons>hyperfine "npm run icons-sprite"
Benchmark 1: npm run icons-sprite
  Time (mean ± σ):      3.413 s ±  0.109 s    [User: 0.004 s, System: 0.003 s]
  Range (min … max):    3.326 s …  3.693 s    10 runs

C:\Users\xmr\Desktop\bootstrap-icons>npm i github:svg-sprite/svg-sprite#async-bin -D

changed 1 package, and audited 688 packages in 5s

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

found 0 vulnerabilities

C:\Users\xmr\Desktop\bootstrap-icons>hyperfine "npm run icons-sprite"
Benchmark 1: npm run icons-sprite
  Time (mean ± σ):      3.296 s ±  0.033 s    [User: 0.003 s, System: 0.009 s]
  Range (min … max):    3.262 s …  3.353 s    10 runs

Definitely worth it assuming we use asynchronous methods everywhere.
If you have some time after #789, I'd really love to see a PR about this with all sync instances changed @Kreeg :)

Hey @XhmikosR.
I am struggling refactor this file further. The problem is that bin/svg-sprite.js is a one big file with a lot of dependencies from fs. So it's hard to switch to async in one leap. I'm suggesting to refactor this file first. At least there is to big parts that can be isolated. It is a parsing and making a config and run bundling.


Migration of the newtonsoft component to system.text.json and fixed some issues pointed out by snyk.

refactor: Migration of the NewtonSoft component to System.Text.Json
fix: Fixed some issues pointed out by Snyk.
test: test implementation for JsonUtils

Canceled because there are too many changes and no related GH issues.


[tesseract:x64-windows] build failure with training_tools feature

Host Environment

OS: Windows 10 v1903
Compiler: Visual Studio 2017

Failure logs
C:\Intel\vcpkg-master>vcpkg install tesseract[training_tools]
The following packages will be built and installed:
    tesseract[core,training_tools]:x64-windows
Starting package 1/1: tesseract:x64-windows
Building package tesseract[core,training_tools]:x64-windows...
-- Note: tesseract only supports static library linkage. Building static library.
-- Using cached D:/SOURCES/tesseract-ocr-tesseract-4.1.0.tar.gz
-- Using source at C:/Intel/vcpkg-master/buildtrees/tesseract/src/4.1.0-614604da36
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Building x64-windows-rel
CMake Error at ports/tesseract/portfile.cmake:72 (file):
  file COPY cannot find
  "C:/Intel/vcpkg-master/packages/tesseract_x64-windows/bin/text2image.exe".
Call Stack (most recent call first):
  scripts/ports.cmake:74 (include)

Error: Building package tesseract:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: tesseract:x64-windows
  Vcpkg version: 2019.07.18-nohash

config-x64-windows-out.log
install-x64-windows-dbg-out.log

Thanks for posting this issue. This issue has been fixed. So I am closing this issue for now.


[JAVA_API] Reorganize project structure and automatically load native libraries

Details

Restructure Java API project to improve code organization
Move the logic for finding and loading native libraries from the Core class to the NativeLibrary class
Automatically load native libraries when the class is loaded using static initializer blocks

@rajatkrishna as I understand, we need this restructuration to easy build and use Java API with IntelliJ IDEA.
Could you also check the samples and add instructions to the Readme file on how to build and run the samples with IntelliJ IDEA?

Native libs loading work fine on Linux and Windows, but not on Mac. Can you please take a look?
@ilya-lavrenov, is that possible to run OpenVINO Mac worker on this PR?
https://github.com/dkurt/openvino_java/actions/runs/5219466114/jobs/9421327346

/azp run openvino_contrib-mac

/azp run openvino_contrib-mac

/azp run openvino_contrib-mac

Thanks, Rajat for all the contributions. LGTM.


Add more methods into interface of BlockCipher

Currently BlockCipher interface has methods without letting caller to specify length/offset. In some use cases like Presto,  it is needed to pass in a byte array and the data to be encrypted only occupys partially of the array.  So we need to add a new methods something like below for decrypt. Similar methods might be needed for encrypt.
byte[] decrypt(byte[] ciphertext, int cipherTextOffset, int cipherTextLength, byte[] aad);
Reporter: Xinli Shang / @shangxinli
Assignee: Xinli Shang / @shangxinli
Note: This issue was originally created as PARQUET-2098. Please see the migration documentation for further details.

Timothy Miller / @theosib-amazon:
@ggershinsky Did you ever get around to this? If not, I'd be happy to see if I can do it. Please let me know.

Gidon Gershinsky / @ggershinsky:
sure, I can take this one

Xinli Shang / @shangxinli:
@ggershinsky Do you have time to work on it as we discussed to release the new version?

Timothy Miller / @theosib-amazon:
I don't personally have a use case. I was just looking through the Jira for things I could help with.

Gidon Gershinsky / @ggershinsky:
@theosib-amazon I got ~half of this (code; not the unitests yet). But in the meantime, it became unclear if we need this functionality (in the upcoming release). Do you have a usecase for it?


Add react-syntax-highlighter and copy button to code blocks

Type
enhancement

Description

Added syntax highlighting for code blocks in chat messages using react-syntax-highlighter and okaidia theme.
Implemented a copy button for code blocks in chat messages, allowing users to easily copy code snippets.
Switched markdown rendering in chat messages from Remark to ReactMarkdown for better compatibility with syntax highlighting.
Updated dependencies in package.json to include react-markdown and react-syntax-highlighter along with their types.

Changes walkthrough
Relevant filesEnhancement

ChatPanel.tsxEnhance ChatPanel with Syntax Highlighting and Copy Feature

src/views/ChatPanel.tsx
Added CopyButton from @mantine/core and icons IconCheck, IconCopy from @tabler/icons-react for implementing a copy feature in code blocks.  Replaced Remark with ReactMarkdown for markdown rendering.  Integrated SyntaxHighlighter from react-syntax-highlighter with okaidia theme for syntax highlighting in code blocks.  Modified message handler from receiveMessage to receiveMessagePartial for bot messages.

+62/-8

Dependencies

package.jsonUpdate Dependencies for Markdown and Syntax Highlighting

package.json
Added react-markdown and react-syntax-highlighter along with their types to dependencies for markdown rendering and syntax highlighting.

+3/-1

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/describe

/review


Feature/fix defenders oscillations

This completely changes the way the defender assignment algorithm works. Decision making is still roughly similar. Is meant to be more robust, every time a new position is assigned robots now have a grace period of 2/3 seconds where the robot the specific robot covers off cannot change (e.g. the defender is 'locked' to this robot).
fixes #434

I do not think "locking" robot positions is the way to stop oscillations.
when in a situation with acceptable noise levels in world, without enemy robots moving, the positions for the defenders should be stable regardless of being locked in place or not.
this should be easily doable if the calculation done does not take into account the position of our own robots.

As I have explained a million and one times now, our calculation does not take into account the position of our own robots in the current world state (Look at setupSimulatedWorld()) The reason why it in some edge cases is noisy in stable positions is becasue the noise from the world state is making 2 robots constantly switch order in the danger scores. This also sometimes happened in dynamic situations.  This is why locking is a nice concept because it means that if we make a decision we stick to it, instead of continuously redeciding what we are going to do. Danger scores can easily fluctuate due to the highly dynamic nature of robot soccer so I don´t see why locking on itself is such a ´bad´ concept.
If you have a better suggestion then please come forward but dreaming of making a score function that creates smooth decision making across a function that has 18 dimensions is naive.


indent-blakline / nvim-colorize not found???

    nixvim = {
      extraPackages = with pkgs; [
        ripgrep
        fd
      ];
      extraPlugins = with pkgs.vimPlugins; [
        vim-nix
        rnvimr
        mkdir-nvim
      ];
      enable = true;
      viAlias = true;
      vimAlias = true;
      options = {
	autoindent = true;
	number = true;
        syntax = true;
	mouse = "a";
	clipboard = "unnamedplus";
	ttyfast = true;
	swapfile = false;
	backup = false;
        completeopt = "menu,menuone,noselect";
  	writebackup = false;
	undodir = ".nixvim/undo";
	undofile = true;
	undoreload = 10000;
  	scrolloff = 10;
        termguicolors = true;
      };
      colorschemes.base16 = {
        enable = true;
        useTruecolor = true;
      };
      plugins = {
#        indent-blankline = {
#          enable = true;
#          useTreesitterScope = true;
#          useTreesitter = true;
#        };
#        nvim-colorizer = {
#          enable = true;
#        };
        trouble = {
          enable = true;
          icons = true;
        };
        nvim-cmp = {
          enable = true;
          auto_enable_sources = true;
          sources = [
            { name = "nvim_lsp"; }
            { name = "path"; }
            { name = "buffer"; }
          ];
          mapping = {
            "<CR>" = "cmp.mapping.confirm({ select = true })";
            "<Tab>" = {
            modes = [ "i" "s" ];
            action = ''
              function(fallback)
                if cmp.visible() then
                  cmp.select_next_item()
                else
                  fallback()
                end
              end
            '';
            };
          };
        };
        treesitter-context = {
          enable = true;
        };
        treesitter-refactor = {
          enable = true;
        };
        treesitter = {
          enable = true;
          folding = true;
          indent = true;
          nixGrammars = true;
        };
        bufferline = {
          enable = true;
          alwaysShowBufferline = true;
        };
      	lualine = {
          enable = true;
          alwaysDivideMiddle = true;
        };
        nix = {
          enable = true;
        };
        lsp = {
          enable = true;
          servers = {
            gdscript.enable = true;
            jsonls.enable = true;
            rnix-lsp.enable = true;
          };
        };
        lsp-lines = {
          enable = true;
        };
      };
    };

This is odd, are you sure nixvim is up to date? (Try running nix flake update)

It is, I'm sure of it because i had to pin it to an older commit when recently git checks didn't pass, i later pinned it to new commit once it passed. Right now I'm on mobile, but my nixos config is public so you can have a look at it.

Made a bare-bones config with those two plugins and haven't been able to reproduce this, so will need to look more in-depth into it

on the latest commit it would seem to go trough but i get the following now:

Are you using nixos-stable or nixos-unstable? The lua-language-server change is from a few weeks ago only, so it has to be on unstable (should probably make it fallback to the old package name on stable, @GaetanLepage this was your PR, think you can fix it?)

system on stable, packages mostly stable, some cherrypicked from unstable

That would explain the issue then! Will work on a fix

@Emaleth how exactly did you add the nixvim input to your flake ?
Have you only set the inputs.nixvim.url or have you also added inputs.nixvim.inputs.nixpkgs.follows = nixpkgs ?

Just the url

Related to #333, closing for it I think (feel free to reopen if I'm mistaken)


Saving one-to-many is broken in 2.0.0.Beta1

Here is a sample app, which works with 1.1.9.Final and fails with 2.0.0.Beta1
There is a test in the app which fail with the error
io.smallrye.mutiny.CompositeException: Multiple exceptions caught:
	[Exception 0] java.lang.IllegalStateException: Illegal pop() with non-matching JdbcValuesSourceProcessingState
	[Exception 1] java.lang.IllegalStateException: Illegal pop() with non-matching JdbcValuesSourceProcessingState

	at io.smallrye.mutiny.operators.uni.builders.UniJoinAll$UniJoinAllSubscription.forwardSignalWhenCompleteOrSubscribeNext(UniJoinAll.java:122)
	at io.smallrye.mutiny.operators.uni.builders.UniJoinAll$UniJoinAllSubscription.onFailure(UniJoinAll.java:137)
	at io.smallrye.mutiny.helpers.UniCallbackSubscriber.onFailure(UniCallbackSubscriber.java:62)
	at io.smallrye.mutiny.operators.uni.UniOperatorProcessor.onFailure(UniOperatorProcessor.java:55)
	at io.smallrye.mutiny.operators.uni.UniOperatorProcessor.onFailure(UniOperatorProcessor.java:55)
	at io.smallrye.mutiny.operators.uni.UniOperatorProcessor.onFailure(UniOperatorProcessor.java:55)
	at io.smallrye.mutiny.operators.uni.builders.UniCreateFromCompletionStage$CompletionStageUniSubscription.forwardResult(UniCreateFromCompletionStage.java:58)
	at java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(CompletableFuture.java:863)
	at java.base/java.util.concurrent.CompletableFuture$UniWhenComplete.tryFire(CompletableFuture.java:841)
	at java.base/java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:510)
	at java.base/java.util.concurrent.CompletableFuture.complete(CompletableFuture.java:2147)
	at org.hibernate.reactive.util.async.impl.AsyncTrampoline$TrampolineInternal.unroll(AsyncTrampoline.java:131)
	at org.hibernate.reactive.util.async.impl.AsyncTrampoline$TrampolineInternal.lambda$unroll$0(AsyncTrampoline.java:126)
	at java.base/java.util.concurrent.CompletableFuture.uniWhenComplete(CompletableFuture.java:863)

spring-hibernate-reactive.zip

Does it also fail with 2.0.0.Beta2?
This error should be solved already, but I cannot verify because I cannot run the project you've provided.
Please, let me know

I'm assuming it's working now.


Link with a newer version of libreadline

Currently binary is linking with libreadline v6, which is somewhat old. Change build number so it'll use a newer libreadline
Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please re-render

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

@igortg This is already built against readline 8, so this PR should be unnecessary. There is not change in the relevant section of e.g. .ci_support/osx_.yaml .

Just found it. Sorry


Biolink 3.0 Compliance

Biolink 3.0 will be released on 7/22. In order to get a head start in implementing compliance, I will reference the tag v2.4.2-alpha-qualifiers on the biolink repo.

Keep in mind, Sierra mentioned that qualifiers are in their own file, qualifiers.yaml

The Biolink team thoughtfully produced this document to assist in migrating existing predicates

I am clearing out the buildkg2 instance for a full build because we also are updating sources.

Running build on branch issue-221 on buildkg2.rtx.ai

[Mon Aug 22 16:46:17 2022]
Error in rule UMLS:
jobid: 27
output: /home/ubuntu/kg2-build/umls_cuis.tsv
log: /home/ubuntu/kg2-build/extract-umls.log (check log file(s) for error message)
shell:
bash -x /home/ubuntu/kg2-code/extract-umls.sh /home/ubuntu/kg2-build /home/ubuntu/kg2-build/umls_cuis.tsv > /home/ubuntu/kg2-build/extract-umls.log 2>&1
(exited with non-zero exit code)

Error in rule KEGG:
    jobid: 47
    output: /home/ubuntu/kg2-build/kegg.json
    log: /home/ubuntu/kg2-build/extract-kegg.log (check log file(s) for error message)
    shell:
        bash -x /home/ubuntu/kg2-code/extract-kegg.sh /home/ubuntu/kg2-build/kegg.json > /home/ubuntu/kg2-build/extract-kegg.log 2>&1
        (exited with non-zero exit code)

[Mon Aug 22 16:45:31 2022]
Error in rule DrugCentral:
    jobid: 44
    output: /home/ubuntu/kg2-build/drugcentral/drugcentral_psql_json.json
    log: /home/ubuntu/kg2-build/extract-drugcentral.log (check log file(s) for error message)
    shell:
        bash -x /home/ubuntu/kg2-code/extract-drugcentral.sh /home/ubuntu/kg2-build/drugcentral/drugcentral_psql_json.json /home/ubuntu/kg2-build/drugcentral > /home/ubuntu/kg2-build/extract-drugcentral.log 2>&1
        (exited with non-zero exit code)


Add missing public headers to PINRemoteImage.h

Resolves #423.

          1 Warning

:warning:
Any source code changes should have an entry in CHANGELOG.md or have #trivial in their title.

  Generated by :no_entry_sign: Danger


Don't let zIndex change after a layer is added

Fixes #153, #155. @kotfic PTAL. We should also rebuild the image and push it to docker hub after this is merged.

LGTMT


Updating live

Docs Build status updates of commit 8869699:
:clock10: Full build: the average full build time is 14 min(s) 15 sec(s), this is based on the last 201 build(s) for this repository.
:warning: This is a FULL build due to the incremental build being disabled in configuration file (.openpublishing.publish.config.json in 'live' branch). For more detailed info, please click here.

Docs Build status updates of commit 8869699:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Set-SPSiteSubscriptionProfileConfig.md
:white_check_mark:Succeeded
View (sharepoint-ps)

skype/skype-ps/skype/Set-CsTeamsCallingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTeamsMeetingPolicy.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Start-CsExMeetingMigration.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[BUG] Clear Keybinds option breaks the main menu Keybinds button

🔵 Describe the bug:
When using the "Clear Keybinds" button in the settings, the main menu "keybinds" button doesn't work until you reload the UI.
🔵 To reproduce: (Steps to reproduce the behavior)
Given I am in the GSE troubleshooting Submenu
When I click the "Clear Keybinds' button
And I return to the main menu
And I click the "Keybinds" menu option
Then The button does not work
🔵 Desktop (please complete the following information):

OS: Windows 11 and MacOS 15
Game Version: Retail

https://github.com/user-attachments/assets/fceb3439-3877-47db-9f0e-cd8d8650f46c
GSE 2.lua.txt
GSE.lua.bak.2.txt
GSE.lua.bak.txt
GSE.lua.txt

As that button bricks the GSE install as of TWW's changes - I need to remove it.

Do you need to do the bind individually?
If I use the "reset keybinds" option to globally reset my binds, I run into an issue where GSE will no longer bind to any keys

If I use the "reset keybinds" option to globally reset my binds, I run
into an issue where GSE will no longer bind to any keys

Which is exactly what is supposed to happen.  You should be using Actionbar
Overrides over Keybinds anyway.
On Sun, 8 Dec 2024 at 14:01, vt8bygg @.***> wrote:

Do you need to do the bind individually?
If I use the "reset keybinds" option to globally reset my binds, I run
into an issue where GSE will no longer bind to any keys
—
Reply to this email directly, view it on GitHub
https://github.com/TimothyLuke/GSE-Advanced-Macro-Compiler/issues/1714#issuecomment-2525400760,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEMCFIF2EQPFYKAXQ22LCT2EO4S5AVCNFSM6AAAAABTGZP7T6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRVGQYDANZWGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***
.com>

--
Timothy Minahan**M 0406 413 680  |  E @.**


Add webpack-cli to list of dependencies installed

Without webpack-cli included earlier, you'll be prompted to install it when you run npm run build and the index.html won't be generated correctly.

Makes sense - thanks!


Release 0.11.1

Added AsyncApplicationEventListener for helping transitioning from events to suspended events.
Added support for using entity manager outside of scope
Updated dependencies

@yatharthranjan Thanks for the quick help here, my slack response was a pocket auto-response and thereby a bit brief ;)


Using build-option "base-href" for htaccess, too

Bug Report or Feature Request (mark with an x)
- [ ] bug report
- [X] feature request

Versions.
latest
Description
With Angular CLI the base href in index.html can be set on build, e.g. ng --base-href /folder/subfolder/someproject/. This is very useful for automatic deployment processes.
But projects like this may require a .htaccess also setting a base like so: RewriteRule ^(.*) /folder/subfolder/someproject/index.html [NC,L]
Is it possible (with Webpack?) to set the path within htaccess also by use of base-href? Or could you add an option for this?

I would not add an option for this, no. That's specific to that deployment server, and adding options that only work for a specific deployment option is a slippery slope to get started on.
I think doing it manually is good enough for now, and the deployment page of the docs (https://angular.io/guide/deployment) lists common configuration.

Is it possible to access set base-href in webpack and hook somewhere?

No, there is no hook support currently. Personally I'd use a npm script for it "build-prod": "ng build --prod --base=href /path/to/folder/ && command-to-replace-text".


Twitter spaces HTTP Error 403: Forbidden

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I'm running yt-dlp version 2023.07.06 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[ ] I've read about sharing account credentials and I'm willing to share it if required

Region
California United States
Provide a description that is worded well enough to be understood
Twitter does require that you login to play but otherwise the audio is publicly available in that any account should be able to listen. I searched for about a half hour but didn't find any other bugs related to this but apologies if I missed somethinga bout it.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
$ yt-dlp --cookies ~/cookies\ twitter.txt -vU "https://twitter.com/i/spaces/1ypKddqewndKW?s=20"
[debug] Command-line config: ['--cookies', '/home/will/cookies twitter.txt', '-vU', 'https://twitter.com/i/spaces/1ypKddqewndKW?s=20']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (zip)
[debug] Python 3.9.2 (CPython x86_64 64bit) - Linux-5.10.0-25-amd64-x86_64-with-glibc2.31 (OpenSSL 1.1.1n  15 Mar 2022, glibc 2.31)
[debug] exe versions: ffmpeg 4.3.6-0, ffprobe 4.3.6-0
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: cedf44a9cf3c241520a692c1006d276e6f434d4d945727c47cb1a70d32953f4a
yt-dlp is up to date<EMAIL_ADDRESS>[twitter:spaces] Extracting URL: https://twitter.com/i/spaces/1ypKddqewndKW?s=20
[twitter:spaces] 1ypKddqewndKW: Downloading GraphQL JSON
[twitter:spaces] 28_1698825972034961408: Downloading legacy API JSON
[twitter:spaces] 28_1698825972034961408: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[debug] Default format spec: bestvideo*+bestaudio/best
[info] 1ypKddqewndKW: Downloading 1 format(s): 0
[debug] Invoking hlsnative downloader on "https://prod-fastly-ap-south-1.video.pscp.tv/Transcoding/v1/hls/UL9-C36LhISy6bfToPyaGokh3Xya9JZhSUocgOCtw7BduTNobFWiYZMsiiXp9JYQZij7suGAzIwtfU7h6BNgCw/non_transcode/ap-southeast-1/periscope-replay-direct-prod-ap-southeast-1-public/audio-space/playlist_16752870527289245639.m3u8?type=replay"
[hlsnative] Downloading m3u8 manifest
ERROR: unable to download video data: HTTP Error 403: Forbidden
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3363, in process_info
    success, real_download = self.dl(temp_filename, info_dict)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3084, in dl
    return fd.download(name, new_info, subtitle)
  File "/usr/local/bin/yt-dlp/yt_dlp/downloader/common.py", line 461, in download
    ret = self.real_download(filename, info_dict)
  File "/usr/local/bin/yt-dlp/yt_dlp/downloader/hls.py", line 77, in real_download
    urlh = self.ydl.urlopen(self._prepare_url(info_dict, man_url))
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3863, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.9/urllib/request.py", line 523, in open
    response = meth(req, response)
  File "/usr/lib/python3.9/urllib/request.py", line 632, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.9/urllib/request.py", line 561, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.9/urllib/request.py", line 494, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.9/urllib/request.py", line 641, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden

Likely relevant is that it's trying to get video by default but it's audio only:
$ yt-dlp --cookies ~/cookies\ twitter.txt -vUF "https://twitter.com/i/spaces/1ypKddqewndKW?s=20"
[debug] Command-line config: ['--cookies', '/home/will/cookies twitter.txt', '-vUF', 'https://twitter.com/i/spaces/1ypKddqewndKW?s=20']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (zip)
[debug] Python 3.9.2 (CPython x86_64 64bit) - Linux-5.10.0-25-amd64-x86_64-with-glibc2.31 (OpenSSL 1.1.1n  15 Mar 2022, glibc 2.31)
[debug] exe versions: ffmpeg 4.3.6-0, ffprobe 4.3.6-0
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: cedf44a9cf3c241520a692c1006d276e6f434d4d945727c47cb1a70d32953f4a
yt-dlp is up to date<EMAIL_ADDRESS>[twitter:spaces] Extracting URL: https://twitter.com/i/spaces/1ypKddqewndKW?s=20
[twitter:spaces] 1ypKddqewndKW: Downloading GraphQL JSON
[twitter:spaces] 28_1698825972034961408: Downloading legacy API JSON
[twitter:spaces] 28_1698825972034961408: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[info] Available formats for 1ypKddqewndKW:
ID EXT RESOLUTION │ PROTO │ VCODEC     ACODEC MORE INFO
───────────────────────────────────────────────────────
0  m4a audio only │ m3u8  │ audio only aac    m4a_dash```

But trying to specify the one available format ID doesn't change the error:
$ yt-dlp -v -f 0 --cookies ~/cookies\ twitter.txt -vU "https://twitter.com/i/spaces/1ypKddqewndKW?s=20"
[debug] Command-line config: ['-v', '-f', '0', '--cookies', '/home/will/cookies twitter.txt', '-vU', 'https://twitter.com/i/spaces/1ypKddqewndKW?s=20']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (zip)
[debug] Python 3.9.2 (CPython x86_64 64bit) - Linux-5.10.0-25-amd64-x86_64-with-glibc2.31 (OpenSSL 1.1.1n  15 Mar 2022, glibc 2.31)
[debug] exe versions: ffmpeg 4.3.6-0, ffprobe 4.3.6-0
[debug] Optional libraries: sqlite3-2.6.0
[debug] Proxy map: {}
[debug] Loaded 1855 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Available version<EMAIL_ADDRESS>Current version<EMAIL_ADDRESS>Current Build Hash: cedf44a9cf3c241520a692c1006d276e6f434d4d945727c47cb1a70d32953f4a
yt-dlp is up to date<EMAIL_ADDRESS>[twitter:spaces] Extracting URL: https://twitter.com/i/spaces/1ypKddqewndKW?s=20
[twitter:spaces] 1ypKddqewndKW: Downloading GraphQL JSON
[twitter:spaces] 28_1698825972034961408: Downloading legacy API JSON
[twitter:spaces] 28_1698825972034961408: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), channels, acodec, size, br, asr, proto, vext, aext, hasaud, source, id
[info] 1ypKddqewndKW: Downloading 1 format(s): 0
[debug] Invoking hlsnative downloader on "https://prod-fastly-ap-south-1.video.pscp.tv/Transcoding/v1/hls/UL9-C36LhISy6bfToPyaGokh3Xya9JZhSUocgOCtw7BduTNobFWiYZMsiiXp9JYQZij7suGAzIwtfU7h6BNgCw/non_transcode/ap-southeast-1/periscope-replay-direct-prod-ap-southeast-1-public/audio-space/playlist_16752870527289245639.m3u8?type=replay"
[hlsnative] Downloading m3u8 manifest
ERROR: unable to download video data: HTTP Error 403: Forbidden
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3363, in process_info
    success, real_download = self.dl(temp_filename, info_dict)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3084, in dl
    return fd.download(name, new_info, subtitle)
  File "/usr/local/bin/yt-dlp/yt_dlp/downloader/common.py", line 461, in download
    ret = self.real_download(filename, info_dict)
  File "/usr/local/bin/yt-dlp/yt_dlp/downloader/hls.py", line 77, in real_download
    urlh = self.ydl.urlopen(self._prepare_url(info_dict, man_url))
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 3863, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python3.9/urllib/request.py", line 523, in open
    response = meth(req, response)
  File "/usr/lib/python3.9/urllib/request.py", line 632, in http_response
    response = self.parent.error(
  File "/usr/lib/python3.9/urllib/request.py", line 561, in error
    return self._call_chain(*args)
  File "/usr/lib/python3.9/urllib/request.py", line 494, in _call_chain
    result = func(*args)
  File "/usr/lib/python3.9/urllib/request.py", line 641, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden

yt-dlp --update-to nightly and try again
or as a workaround with the latest stable version, you can try adding this to your command:
--add-header "Referer:https://twitter.com/"


Update MD file on new branch 2017-48-22_17-11-20

:warning: Validation status: warnings

File
Status
Preview URL
Details

split/ref.yml
:warning:Warning

Details

split/updateTestRef.md
:white_check_mark:Succeeded

split/ref.yml

[Warning] Uid(Microsoft.SqlServer.Server) has already been defined in ref2.yml.
Line 1: [Warning] Invalid file link:(~/content.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


teensyduino: add missing dependencies for teensy-loader

Motivation for this change
The teensy-loader (./share/arduino/hardware/tools/teensy) was not linked with the full list of dependencies that it needs. Thus it didn't start when called by the Arduino (Teensyduino) UI to actually push code to the device.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[X] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Notify maintainers
cc @antono @auntie @robberer @bjornfor @bergey

This refers to #75537

This can be closed now as this was merged in #81156


How to change image size to 160x160

want to training mobilefacenet with image size 160x160. I have modified mobilefacenet structure, but the data size is 112x112.
How to change size of dataset in Dataset Zoo?
Do you support resize function in the train script?
Thanks~

firstly, redefine --image_size to [160, 160], then resize your training datasets to 160x160 in load_data(...) function, that will go fine.
close this issue, when need reopen,


Add the header and footer from the main site.

Towards https://github.com/giantswarm/giantswarm/issues/10856
This adds the header and footer from giantswarm.io to the docs site.
I've managed to make a helper tool which I'll submit in a different repo, that can extract the CSS that matches a specific element and any of its children. It uses Selenium (as suggested by Marian), and actually needs to start up an instance of Chrome, so I don't think we can put it in a docker container.
I used it to generate the CSS which appears in gs_styles.html . However it still required some manual overrides (which I added and commented on at the bottom of that same file)
gs_footer.html and gs_header.html are unaltered HTML extracted from giantswarm.io
Todo

[x] Make it work in narrow mode as well. The hamburger menu button is broken.
[x] Fix logo size, for some reason it's smaller than on giantswarm.io

UX Note:
Once we scroll down a bit it's not super obvious we're on the docs site anymore.

It looks like the logo is replaced by a svg version using some javascript on giantswarm.io
So I did make a modification just now to gs_header.html to get it into that state, that'll have to be a note to remember if we ever extract the HTML again.
But I'd like to revisit that maybe when we / if we need to in the future, since the time spent on this issue is probably giving diminishing returns now.

To test locally: make dev works for me locally when I change USER 101 to USER 0 in the Dockerfile

I noticed both the header and footer links hover effects are not the same as the website

Hovering a header link on the website

Hovering a header link on the docs

Hovering a footer link on the website

Hovering a header link on the docs

Also, hovering the 'Become a Client' CTA in the header results in a text-less button

@axbarsan thx for the review and noticing the differences. Updated it.
I think when I come back and make this a script that automatically extracts and places the files, I can apply the zz_generated naming convention and give puppeteer a look.

All the issues above have been fixed, but bare with me, as I got a few more:

Hovering nested header megamenu links
Docs

Website

Hovering articles in the header megamenu
Docs (hovering the button makes it change display, as well)

Website

Hovering nested buttons in the header megamenu
Docs

Website

Hovering social media icons in the footer
Docs

Website

Also, links in the header (both the primary ones and the nested ones) have transitions on the website, but not on the docs.

I think I might have to see if I can adjust my CSS stealer to include these meta style.. seems like they are missing somehow.

Styling is on point, everything looks as expected 👍
Footer social links are a bit funny, though

There must be some weird javascript applying the links there.. very weird indeed.
For now I'll edit the extracted footer manually.

ok now that most of the implementation is sorted... I still have a UX concern. ping @sig-ux
I want to make it a bit clearer that you're on the docs site when you're scrolled further down into a document.
Because right now, you might be tempted to click on the "Giant Swarm" logo to go back to the home of the docs, but instead you end up on the giant swarm website. And from there it's not very easy to go back to the docs.

Just got word from @morotki that the mega header functionality is changing soon, to be 'click to open' instead of 'hover to open', which I really like as an improvement. I might hold this PR open until that gets finalised and then do a final pull of the assets

@oponder maybe you also want to merge this :)

It uses Selenium (as suggested by Marian), and actually needs to start up an instance of Chrome, so I don't think we can put it in a docker container.

@oponder Did you solve this? If not, I have a project where I am running Chrome and selenium in Docker.

@oponder pair with someone to try and finish that up

Going with just the footer for now to make some progress at least!


Preventing name prefix from being part of output; making automatic prefix universal

I've also taken the liberty of bumping the version number to 1.3.0; the possibility is very small, but this may be a breaking change in cases where nested directories were used with the same name as the path prefix, so it does deserve a new minor version number.
This change makes it so that the name django-storage-swift returns for objects doesn't include the name prefix; it also adds features so that a consumer of the framework doesn't have to be aware of the prefix for all aspects of the framework to work consistently.
Fixes #49.

@blacktorn - this is ready for a second look.

@blacktorn, any idea of when you'll be able to review this? Right now, we're using my fork in production, which isn't ideal for obvious reasons.
Let me know when you can take a look, and I'll rebase on master at that point.

@haikuginger @blacktorn Does this PR need an OSPR to keep it from getting lost?

@haikuginger This looks good to me, can you rebase so I can merge the PR, thanks!

@blacktorn, rebased for you.

Thanks @haikuginger

Thanks @blacktorn! Any chance we can get a version bump as well?

I've just uploaded version 1.2.12 to PyPI!


Binary incompatibility warning on OSX wheels

import warnings
warnings.simplefilter("error")
import scipy.special

Trips a warning:
RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88

This is specific to your installation, scipy compiled with different numpy version than what's installed and/or with old Cython version.

@pv thanks!

iirc in new Cython versions this warning is not emitted any more even if there is a mismatch in versions, but I may recall wrong

@pv I just uninstalled and reinstalled scipy and numpy using pip and I'm still getting the warning.  I don't understand how that's possible

Did you try the snippet of code on your machine?

I don't see that.
Did you install it from sources, or from wheels?
Also, numpy first, then scipy?

FYI I did not have this problem 40 days ago and I'm not sure what I could have done (besides upgrading) to break it.

Ok, those are the osx wheels from pypi -- @matthew-brett: do you also see this warning?

@NeilGirdhar: Note that reinstalling the wheels does not help --- you'd need to recompile (pip --no-use-wheel) but whether you are set up for that is a different question

@pv Thanks, I'm recompiling now.  I will keep you posted.  However, it would be nice to use the wheels going forward.

@NeilGirdhar - what do you get for:
which python
which pip
head `which pip`
python -c 'import numpy; print(numpy.__file__)'
python -c 'import scipy; print(scipy.__file__)'

~: which python
/opt/local/bin/python
~: which pip
/opt/local/bin/pip
~: head `which pip`
#!/opt/local/Library/Frameworks/Python.framework/Versions/3.5/bin/python3.5
# EASY-INSTALL-ENTRY-SCRIPT: 'pip==8.1.2','console_scripts','pip'
__requires__ = 'pip==8.1.2'
import sys
from pkg_resources import load_entry_point

if __name__ == '__main__':
    sys.exit(
        load_entry_point('pip==8.1.2', 'console_scripts', 'pip')()
    )
~: python -c 'import numpy; print(numpy.__file__)'
/opt/local/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/numpy/__init__.py
~: python -c 'import scipy; print(scipy.__file__)'
/opt/local/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/scipy/__init__.py

(These are the macports versions of everything.  Python 3.5, pip 8.1.2.)

Yes, I don't see this with Python.org Python or homebrew Python in virtualenvs.
What do you get for these?
python -c 'import numpy; print(numpy.__version__)'
python -c 'import scipy; print(scipy.__version__)'

Just to confirm, you are running the following?:
python -c 'import warnings; warnings.simplefilter("error"); import scipy.special'

I get no output for that command.

@matthew-brett Actually, wow!  It works in Python.  It doesn't work in ipython.

Check:
head `which ipython`

Actually, the build had just completed, so I will never know if it works in python.  Now that scipy has been recompiled, it works in both ipython and python.

The reason why I reopened the issue was to check if there is a problem with the wheels provided by us.
What you can still do is to uninstall it again, and reinstall from the wheels to see if it reproduces.

Reinstalling the wheels causes the problem to reappear in ipython, but python still works fine.  Ipython is the macports one, version 5.1.0.

I don't get this warning on macports either, with:
/opt/local/bin/ipython-3.5
In [1]: import warnings
   ...: warnings.simplefilter("error")
   ...: import scipy.special
   ...:

It sounds like your IPython is picking up different packages than your Python.  Is it possible you're running a Python 2 IPython?  What do you get if you use /opt/local/bin/ipython-3.5 ?

/opt/local/bin/ipython-3 gives the same problem.

(ipython was also installed from wheels using pip)

Hmm, now I'm confused.  The macports IPython seems to be at /opt/local/bin/ipython-3.5.  The pip IPython seems to be in /opt/local/Library/Frameworks/Python.framework/Versions/3.5/bin.   Which do you think you are picking up?     What do you get for which ipython?
How about:
head `which ipython`
ipython
In [1]: import numpy; print(numpy.__file__, numpy.__version__)
In [2]: import scipy; print(scipy.__file__, scipy.__version__)

~/src/imm: which ipython
/opt/local/bin/ipython

Maybe I should delete it?

I deleted it, now I still have the same error despite:
~: which ipython
/opt/local/Library/Frameworks/Python.framework/Versions/3.5/bin/ipython

Sorry, I have to run for tonight.  Thank you for your help so far.  I'll be able to actively reply tomorrow.

What do you get for
head `which ipython`
ipython
In [1]: import numpy; print(numpy.__file__, numpy.__version__)
In [2]: import scipy; print(scipy.__file__, scipy.__version__)

~: head which ipython
#!/opt/local/Library/Frameworks/Python.framework/Versions/3.5/bin/python3.5
-- coding: utf-8 --
import re
import sys
from IPython import start_ipython
if name == 'main':
sys.argv[0] = re.sub(r'(-script.pyw|.exe)?$', '', sys.argv[0])
~: ipython
Python 3.5.2 (default, Sep 14 2016, 19:45:15)
Type "copyright", "credits" or "license" for more information.
IPython 5.1.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.
In [1]: In [1]: import numpy; print(numpy.file, numpy.version)
/opt/local/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/numpy/init.py 1.11.1
In [2]: import numpy; print(numpy.file, numpy.version)
/opt/local/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/numpy/init.py 1.11.1
In [3]: import scipy; print(scipy.file, scipy.version)
/opt/local/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/scipy/init.py 0.18.0

~: head `which ipython`
#!/opt/local/Library/Frameworks/Python.framework/Versions/3.5/bin/python3.5

# -*- coding: utf-8 -*-
import re
import sys

from IPython import start_ipython

if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw|\.exe)?$', '', sys.argv[0])
~: ipython
Python 3.5.2 (default, Sep 14 2016, 19:45:15)
Type "copyright", "credits" or "license" for more information.

IPython 5.1.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

In [1]: In [1]: import numpy; print(numpy.__file__, numpy.__version__)
/opt/local/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/numpy/__init__.py 1.11.1

In [2]: import numpy; print(numpy.__file__, numpy.__version__)
/opt/local/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/numpy/__init__.py 1.11.1

In [3]: import scipy; print(scipy.__file__, scipy.__version__)
/opt/local/Library/Frameworks/Python.framework/Versions/3.5/lib/python3.5/site-packages/scipy/__init__.py 0.18.0

This seems to happen again with the latest version of numpy 1.15.0... Observed both in Python 2.7.15 and Python 3.6.5.
/usr/local/lib/python2.7/site-packages/scipy/linalg/basic.py:17: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from ._solve_toeplitz import levinson
/usr/local/lib/python2.7/site-packages/scipy/linalg/__init__.py:207: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from ._decomp_update import *
/usr/local/lib/python2.7/site-packages/scipy/special/__init__.py:640: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from ._ufuncs import *
/usr/local/lib/python2.7/site-packages/scipy/special/_ellip_harm.py:7: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from ._ellip_harm_2 import _ellipsoid, _ellipsoid_norm
/usr/local/lib/python2.7/site-packages/scipy/interpolate/_bsplines.py:10: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from . import _bspl
/usr/local/lib/python2.7/site-packages/scipy/sparse/lil.py:19: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from . import _csparsetools
/usr/local/lib/python2.7/site-packages/scipy/sparse/csgraph/__init__.py:165: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from ._shortest_path import shortest_path, floyd_warshall, dijkstra,\
/usr/local/lib/python2.7/site-packages/scipy/sparse/csgraph/_validation.py:5: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from ._tools import csgraph_to_dense, csgraph_from_dense,\
/usr/local/lib/python2.7/site-packages/scipy/sparse/csgraph/__init__.py:167: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from ._traversal import breadth_first_order, depth_first_order, \
/usr/local/lib/python2.7/site-packages/scipy/sparse/csgraph/__init__.py:169: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from ._min_spanning_tree import minimum_spanning_tree
/usr/local/lib/python2.7/site-packages/scipy/sparse/csgraph/__init__.py:170: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from ._reordering import reverse_cuthill_mckee, maximum_bipartite_matching, \
/usr/local/lib/python2.7/site-packages/scipy/spatial/__init__.py:95: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from .ckdtree import *
/usr/local/lib/python2.7/site-packages/scipy/spatial/__init__.py:96: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from .qhull import *
/usr/local/lib/python2.7/site-packages/scipy/spatial/_spherical_voronoi.py:18: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from . import _voronoi
/usr/local/lib/python2.7/site-packages/scipy/spatial/distance.py:122: RuntimeWarning: numpy.dtype size changed, may indicate binary incompatibility. Expected 96, got 88
  from . import _hausdorff

Hmm, it doesn't happen for me:
In [6]: import warnings
   ...: warnings.simplefilter("error")
   ...: import scipy.special
   ...:
   ...:

In [7]: numpy.__version__
Out[7]: '1.15.0'

In [8]: import scipy; scipy.__version__
Out[8]: '1.1.0'

Hmm, it doesn't happen for me:
In [6]: import warnings
   ...: warnings.simplefilter("error")
   ...: import scipy.special
   ...:
   ...:

In [7]: numpy.__version__
Out[7]: '1.15.0'

In [8]: import scipy; scipy.__version__
Out[8]: '1.1.0'

@kdebrab yes, this is under discussion in the numpy repo, it should be fixed there. see https://github.com/numpy/numpy/pull/11634 and https://github.com/numpy/numpy/issues/11628


What is the principle of feature map visualization?

Hi ！ The research in this paper is an excellent work
By the way, I would like to ask what is the principle of feature map visualization in the paper. Is it simply a matter of regularizing each channel of the feature map and then adding up all the channels to take the mean?
Does the feature map visualization in the detection task have the same effect as the feature map visualization in the segmentation task?

Hi, please see the issue #76.


Disaggregate codelist distributions by field

Duncan: I want to understand coverage of documents by documentType and section, e.g. a table like

type
Planning
Tender
Award
Contract
Implementation

tenderNotice
#
#
#
#
#

contractNotice
#
#
#
#
#

Duncan: I want to understand coverage of milestones by type and section, for example by using a table like

type
Planning
Tender
Award
Contract
Implementation

approval
10,278

delivery

2,546

financing

10,278

Duncan: For both of the above, since document type and milestone type are open codelists, we should include any extra codes used by the publisher.

James: the logic can be that all fields using the same codelist form the X axis. We would need some custom logic at the presentation layer to abbreviate the JSON paths to e.g. "Planning"

Transferred to pelican-frontend, though changes to pelican-backend might be required.


#122 logger file added and some loggs added successfully

I am creating a draft PR please have a look and suggest changes. Specify the format of the logger and few must include instances I have added few to make sure I am working in right direction.

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- wplay/utils/logg.py  1

See the complete overview on Codacy

Is my work not correct?

 Here is an overview of what got changed by this pull request:

Issues
======
- Added 1

Complexity increasing per file
==============================
- wplay/utils/logg.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- wplay/utils/Logger.py  1

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- wplay/utils/Logger.py  1

See the complete overview on Codacy

Nice work, hope you finish soon.

Can I send the PR by evening? Please extend this for few hours.


[POC] New Vert.x pool integration

(pair programmed with @DavideD )
The first and last commits probably don't belong in the final PR, but are useful for testing and sake of discussion.
Integrating the new Pool is fundamentally trivial: we only had to remove the ThreadLocalPoolManager and revert to using the vert.x pool as-is.
A slight complexity arises in the need of actually closing the vert.x pool in an asynchronous safe way: the org.hibernate.service.spi.Stoppable services in Hibernate ORM declare a void stop() method; but to ensure the pool was actually closed we should wait for it.
We can't really wait for it via a join as that blocks the eventloop; so we introduced a way to extract the promise later.
This highlights the need that perhaps Hibernate Reactive could use a properly reactive SessionFactory: in this case and primarily to stop it in a reactive way, but we can see some benefits in having a reactive API to boot as well.
I'm not sure of the usefulness of this though, as all integrating platforms I know will be blocking for the component lifecycle (e.g. components in Quarkus have their lifecycle managed in old style, blocking way). So an alternative could be to introduce a simple helper like in 8c4188f651e38bd8cd61315d231f9b14175d9bbd .
I'll focus next on seeing how this pool behaves.

modified to use vertx 4.1.0.Beta1 - obviously CI can't deal with the snapshot

modified to use vertx 4.1.0.Beta1 - obviously CI can't deal with the snapshot

We already test on CI with the latest snapshot, so there's no need to update the default when you send the PR.
It's actually better if you don't, this way we can check that it works with both the stable version and the snapshots.

Good point. BTW while I'm not fully done with performance tests, I'm seeing no issues with the pool.
I guess I'll convert this POC to a real PR...

Removed the first and last commits.
It will need a rebase, could you do that @DavideD ? There's a little conflict with your multitenancy fix.

Yes, no worries

I've rebased this here: https://github.com/hibernate/hibernate-reactive/pull/771


fix(xml): Do not resolve external entities

Issue
https://gravitee.atlassian.net/browse/APIM-2081
Description
Do not resolve external entities

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

1.3.3
GitHub release

Your semantic-release bot :package::rocket:


Disable pylints no-name-in-module check

We can disable this rule because mypy already checks for this kind of errors and this also fixes a long-standing issue with pylint that can't figure out protobuf generate stuff have some members:

https://github.com/pylint-dev/pylint/issues/6281

And now that we disabled this check we can just use the protobuf modules instead of having to declare one alias per each symbol we need. We did this before just so we only need to use a pylint disable=no-name-in-module one time in the import instead each time we used the symbol.

Skipping release notes as these are all internal changes.


vmlatency: Use checkup UID as label value

Currently, the source and target VMIs have a label latency-check-vm with a hard-coded empty string value.
This prevents execution of multiple instances of this checkup in parallel (for more details see issue #164).

Add a new CHECKUP_UID environment variable in the API between the framework and the checkup.
Add the checkup's UID as the value of the "latency-check-vm" label.

To get checkup's UID:
kubectl get cm -n <target-namespace> <cm-name> -o jsonpath='{.metadata.uid}'

To list VMI objects by checkup's UID:
kubectl get vmis -n <target-namespace> -l latency-check-vm=<checkup-UID>

Depends on PR #169.
Replaces PR #171.

It seems that this PR now overlaps with #165 in the sense of randomizing the node affinity label.

It seems that this PR now overlaps with #165 in the sense of randomizing the node affinity label.

That is right.
This was the result of our offline discussion - instead of using the user-supplied ConfigMap name as an identifier for the checkup, use its UID.
We've also agreed that this UID shall be used as the value for the label on the VMIs.

This was the result of our offline discussion - instead of using the user-supplied ConfigMap name as an identifier for the checkup, use its UID.

I mean for the last commit, where the UID is being used for the affinity label, this overlaps with this PR.

the PR desc does not mention the motive. can you specify that?

the PR desc does not mention the motive. can you specify that?

From the PR description:

Currently, the source and target VMIs have a label latency-check-vm with a hard-coded empty string value.
This prevents execution of multiple instances of this checkup in parallel (for more details see issue https://github.com/kiagnose/kiagnose/issues/164).

I may be missing something, but where are we actually consuming this? all I see is insertion and validation. where do we add it as a the label value?

https://github.com/kiagnose/kiagnose/pull/173/files#diff-d60aa090110dbe4421acf2db6a61ce9c88ae82264bd0e66ddc999917378b3c22R144

I think PR should not depend on #166, it can stand on its own. The last commit overlaps with #165, you could remove it and let #165 do the integration as we concluded offline.

I believe you are referring to PR #169 and not PR #166 which was merged.
Without the refactor to config.New(), the linter fails because of high cyclomatic complexity.
The last commit suggests an alternative mechanism to set the value of the VMI label to the UID from the user-supplied CM, instead of generating it on the fly, as was suggested in PR #165.
This solves the issue raised in PR #165.
Do you have any comments regarding the code?

Change: Rebase

Change: Rebase.


add isPolyfilled properties to be able to interfaces (fixes #41)

Per mrdoob/three.js#8330

Is legit to drop the .js prefix?

Is legit to drop the .js prefix?

yep, with both ES6 and CommonJS modules

OK, good to know. However, I'd opt for consistency throughout the codebase. Let's stick to .js$ for now. Can change style in one fell swoop later.

OK, good to know. However, I'd opt for consistency throughout the codebase. Let's stick to .js$ for now. Can change style in one fell swoop later.

yeah, that's smart. just fixed.

updated!

Thanks


Better Facts in E-Graphs

To handle rewrites with unbound conditions, we would like to use multi-patterns. For example, in a rewrite like:
?a => ?b if ?a - ?b = 0

... we can't apply the rewrite as is, as the pattern var in the LHS (?a) are not a subset of the pattern vars in the RHS (?b). With a mutli-pattern, we would match both ?a and ?a - ?b = 0, which would resolve ?b, allowing us to apply the rewrite. Unfortunately, multi-patterns in egg break explanations (this simply isn't supported). To work around this, we need to find a way to use the LHS of the rewrite to represent both the term and the conditions we want to match.
With-Nodes
For this, we introduce a With(Id, Id) constructor and write our conditional rewrite as follows:
(with ?a (?a - ?b = 0)) => ?b

For this to work as expected, there need to exist with X Y e-nodes in our e-graph for every possible e-class X and known fact Y (as we will see, this incurs an overhead of one e-node per e-class). To achieve this, we do two things:

We introduce an e-class (let's call it F) of all "known facts". That is, an e-class of all propositions which we know to be true, either because the user provided an explicit fact for this, or because we were able to derive this fact from other facts during eqsat.
We use the e-class analysis modify function to observe when new e-classes are created. If a new e-class C appears, we modify the e-graph by also adding the e-node (with C F). Thus, any new e-class has a corresponding with-node. As the presence of with-nodes is invariant under e-class union, we obtain the invariant that every e-class has a corresponding with-node during the entire eqsat run.

Fact E-Class
To implement Requirement (1) from above, we declare F to be the e-class of the term const "True". Any propositional fact given by a user is added to F. To reify the equalities known by e-classes into explicit equality propositions in F we add a similar approach as used for Requirement (2) above. We use the modify function to check when a new e-class C is created, and in that case add a (app (app (app (const "Eq" _) _) C) C) e-node to F. That is, we reify the fact that any term in C is equal to any other term in C. Note the use of the term _ which represents unknown subterms (as e.g. used for eta-expansion). We need to use _ for the type and universe level of the equality, as we cannot easily reconstruct these.
Conjunction of Facts
To represent multiple conditions in a with-node, we also need to ensure that for any given facts p and q, the fact e-class F also contains p ∧ q. Ensuring this is trivial by adding the e-node (app (app (const "And") F) F) to the e-graph once before eqsat.
Deduction of Facts
We can use the given setup to also deduce new facts from existing ones by applying given theorems. Namely, any given theorem of the form ∀ a b c h₁ h₂ ..., p where p is not an equivalence can be turned into ∀ a b c h₁ h₂ ..., (p = True). From this, we can create a "normal" rewrite which needs to match (that is, prove) facts h₁ h₂ ... from the e-graph (if they are proven they are contained in F), and if successful rewrites p => True which adds p to the fact e-class F.
Open Problem:
If p is not yet contained in the e-graph, then the corresponding rewrite will never match. To partially resolve this, there's one possible adjustments I can think of: If the pattern vars in p are a subset of the pattern vars in h₁ h₂ ..., then we generate the rewrite as (with True (h₁ ∧ h₂ ∧ ...))=> p. When the pattern vars in p are not a subset of the pattern vars in h₁ h₂ ..., I'm not sure what to do. It would be nice to somehow add the unproven terms to the e-graph.
Type Class Instances
As the fact e-class F merges all know facts into one e-class (that is, declares them to all be equivalent), we only want to add propositional facts into that e-class (as any two given proven propositions are provably equal). For other kinds of facts, namely the synthesizability of type class instances, we don't want to equate the underlying terms. Thus, type class instance conditions probably still have to be handled "the old way".

To represent multiple conditions in a with-node, we also need to ensure that for any given facts p and q, the fact e-class F also contains p ∧ q.

multiple conditions can also be simulated by multiple layers of "with".
So instead of (with a (c1 ∧ c2)) we can do (with (with a c1) c2).

@memoryleak47 If we choose not to use with, then a nice effect of using $$\land$$ is that we only have 1 node overhead instead of one node per e-class (as with with).

so in short, we require

add (fact True) to the e-graph
equate (eq ?x ?x) = True for each e-class ?x
equate (and True True) = True

and then we can express a rule P1 -> P2 -> A=B
by matching against (fact (and P1 (and P2 (eq A A)))),
and then applying the resulting subst to A=B.
Similarly, a "propositional" rule (i.e. a non-equality theorem / lemma from lean) P1 -> P2 -> P can be rewritten to an equational one, by P1 -> P2 -> P=True.

Open Problem: If p is not yet contained in the e-graph, then the corresponding rewrite will never match

Is p a condition, or the lhs of the equation? I would argue in both cases, it is intended that we cannot apply the rewrite rule. Maybe I'm misunderstanding

For other kinds of facts, namely the synthesizability of type class instances, we don't want to equate the underlying terms.

Why is that? I consider a type-class instance as a boolean term which is True, if Lean can synthesize a type-class and False otherwise.
To me it seems that the actual synthesized instance isn't really used, hence restricting it to the boolean question of "is this synthesizable?" is enough.
This probably again boils down to me not really understanding type-classes though ^^

@memoryleak47 Agreed. That still leaves the question open of how we might introduce not--yet-proven propositions/ terms we might want to rewrite on into the e-graph in the first place. But that's somewhat orthogonal.


[Windows] optional ability to restore app hotkeys

If app uses hotkeys through hotkey_manager pkg then after meedu player's close/dispose all hotkeys are lost.
There is async callback through which app can re-register it's hotkeys

My use-case is next
App (on windows) provides different functionality (e.g. instant messaging), including own hotkeys. For this I use hotkey_manager HotKeyManager.instance.registerHotkey();
Appearance of video player is only one of functions. When player appears, it registers its hotkeys. And when called MeeduPlayerController.videoPlayerClosed() and/or MeeduPlayerController.dispose() it executes
HotKeyManager.instance.unregisterAll();
This code unregisters any hotkeys in app, including those I registered before I call player.
Easy solution I see is to manually re-register my hotkeys when I close/dispose MeeduPlayerController passing callback

Oh I get your point now
I may make the hotkey only unregister the video player keys in an update
For now I will merge your request
Thank you for it ❤


Interoperability with CDK/Incremental Adoption?

Hi there! First of all this is an incredible project, and I wish my team and I had discovered it sooner! The strong typing connection possible between GraphQL API resolvers and DynamoDB table specifications is incredibly powerful and exciting to us, as the loose typing here is a big source of bugs in our current work. Of course many other things about this paradigm seem amazing as well!
Unfortunately, our codebase is rather large and currently written entirely in CDK. I wonder if there is any possible story of incremental adoption here? Can punchcard constructs be used in the same stack as normal CDK constructs? More specifically to our usecase, is there are way we could leverage the strongly-typed VTL template generation in as minimal a way as possible? For example, is there a way that we could leverage this work to just generate VTL templates?

Hey, thanks for reaching out! I'm glad you like the project.
This project is a playground full of many experiments where I try to push the TypeScript language as far as I can to build a virtual programming language. It's been a ton of work and a lot of fun, but it also means that some features have known edge cases with the compiler. I'm curious, have you played with it? What was your experience like dealing with the heavy use of type-level logic?
AppSync VTL abstraction
One of those edge cases is with the AppSync stuff. Sometimes, type inference on yield* breaks down. I'm proud of the implementation but I also think it deserves a version 2, maybe I can do that as part of a standalone Construct library compatible with the CDK? Would you be interested in helping design that DX?
RE: extracting the VTL abstraction from Punchcard - that is quite complicated. The interactions with services are encoded directly into the DynamoDB.Table, Lambda.Function, etc. constructs themselves. Those constructs are typed, so the data type is known and used to derive type-safe VTL behavior. How would you expect this to work with existing CDK constructs?
Incremental Adoption
It is possible but there is one gotcha you need to be aware of. Punchcard uses webpack to create the runtime bundle JS file uploaded to Lambda. In webpack, I do some hacky stuff to prune out all the CDK dependencies from the runtime bundle - it reduces the size down from >2MB to ~30-200KB and drastically improves the cold start.
This is a complication introduced by blending code that runs at CDK synthesize time and code that runs in the Lambda Function. If I just bundled the CDK app and ran it within lambda, then things blow up. For example, if a Construct references an asset, then it will run a bunch of fs calls in the lambda and likely break. It's slow and heavy - you don't want to run unnecessary code in your Lambda.
My solutiton works, but it has the unfortunate drawback that it diverges from the CDK. A better solution would be a closure serializer that is capable of extracting only the code required by a Function and serializing any state. Pulumi does some interesting things in this space, but alas, I went with a different approach.
In Punchcard, instead of importing a CDK module, like:
import * as cdk from '@aws-cdk/core'

Everything is wrapped in a Build<T> monad. Think of it like a Promise<T>. This lazily suspends all CDK code in a closure so that it can be safely erased from the runtime bundle.
const cdk = Build.lazy(() => require('@aws-cdk/core') as import('@aws-cdk/core'))

You could incrementally adopt your existing CDK application by creating a new Punchcard package that depends on your existing Constructs.

add some boilerplate to your existing package, you can model this off of https://github.com/punchcard/punchcard/tree/master/packages/%40punchcard/constructs

In your package's index.ts
import erasure = require('@punchcard/erasure');

// tell Punchcard to erase this module from the runtime bundle - it is only needed at build time.
erasure.erasePattern(/^my-construct-library$/);

create a new Punchcard package and lazily import your existing Construct library

const MyConstructLibrary = Build.lazy(() => require('my-construct-library') as import('my-construct-library'));

import { Core } from 'punchcard';

const app = new Core.App();

const existingStack = Build.concat(app.root, MyConstructLibrary).map(([app, myConstructLibrary]) => {
  // you can write normal CDK code inside this block, e.g. instantiating one of your existing stacks.
  return new myConstructLibrary.ExistingStack(app, 'MyExistingStack');
});

// now go and use Punchcard stuff
class MyShape extends Type(..)

const table = new DynamoDB.Table(..)

Now, you can author standard CDK constructs as you've already been doing and then instantiate them in your Punchcard app.
Note:
I think this inter-op is the thing most hurting Punchcard's DX. I'd love to do some closure serialization and drop this hack all together.


Implement the possibility to retreive non-daily TF data

Investing provides daily, weekly and monthly timeframe data. I believe that currently the daily timeframe is hardcoded. For example, in the get_recent_data function in the init.py file parameters are set as follows:
    params = {         "curr_id": id_,         "smlID": str(randint(1000000, 99999999)),         "header": header,         "interval_sec": "Daily",         "sort_col": "date",         "sort_ord": "DESC",         "action": "historical_data"     }
We could possibly make the interval_sec an optional parameter, where Daily is the default.

Hi @davetromp!
Thank you for the proposed enhancement, I will work on it asap since now I am figuring out how to make the package more usable and fix the previously reported issues. So on, once investpy 0.9.9 is released, this issue will be worked on.
Anyways thank you for reporting it! :octocat:


[Provider Audit] :  provider.onyxcomputers.net

Prerequisite Steps:
1. Make sure your provider has community provider attributes and your contact details (email, website):
  Example:
  $ provider-services query provider get akash1<REDACTED> -o text
  ...
  attributes:
  ...
  - key: host
  value: akash
  - key: tier
  value: community
  info:
    email: "<your email>"
    website: "<your website>"

Ref documentation:.
2. Make sure your provider *.ingress resolves to your provider IP (ideally worker node IP)
host <anything>.ingress.<yourdomain>

Example:
$ host anything.ingress.akash.pro
anything.ingress.akash.pro is an alias for nodes.akash.pro.
nodes.akash.pro has address <IP_ADDRESS>

More info on what DNS records should look like are available in the Akash Documentation here.

3. Please make sure your Akash provider doesn't block any Akash specific ports.

If you are using a firewall, please follow this doc:
https://akash.network/docs/providers/build-a-cloud-provider/akash-cloud-provider-build-with-helm-charts/#step-11---firewall-rule-review
If you are behind NAT, then you need to make sure these ports are open between your provider worker nodes and the internet.

Audit Steps:
1. Title the issue: " [Provider Audit]: Provider Address" (e.g. "[Provider Audit]: provider.europlots.com")
2. Wait for response via comments. If no issues during provider Audit, process will be complete, provider should start bidding on leases, and Audit ticket will be closed.
3. If there are issues during the provider Audit, debug those issues, and Audit will be complete.
4. Audit Issue will be closed by core team member.
Leave contact information (optional)

could you please load up more AKT. i see the balance is very less. ideally 20-50AKT should be good
/provider_info2.sh https://provider.onyxcomputers.net:8443
PROVIDER INFO
BALANCE: 5.574662
"hostname"                    "address"
"provider.onyxcomputers.net"  "akash1jr8llk5n6xjjev3xavgx0ysmurj70369400me9"

Total/Available/Used (t/a/u) per node:
"name"   "cpu(t/a/u)"   "gpu(t/a/u)"                                    "mem(t/a/u GiB)"   "ephemeral(t/a/u GiB)"
"node1"  "40/38.1/1.9"  "0/18446744073709551615/-18446744073709552000"  "62.77/61.3/1.47"  "434.04/433.04/1"

ACTIVE TOTAL:
"cpu(cores)"  "gpu"  "mem(GiB)"  "ephemeral(GiB)"  "beta1(GiB)"  "beta2(GiB)"  "beta3(GiB)"

PERSISTENT STORAGE:
"storage class"  "available space(GiB)"

PENDING TOTAL:
"cpu(cores)"  "gpu"  "mem(GiB)"  "ephemeral(GiB)"  "beta1(GiB)"  "beta2(GiB)"  "beta3(GiB)"

could you please load up more AKT. i see the balance is very less. ideally 20-50AKT should be good

I had 15 AKT in the wallet and it has all vanished with no transaction record. I don't feel safe putting more in only for it to just vanish also.

I tried deploying some workloads to this provider and I was unable to get any bids for this provider. We can't sign it if it isn't working for non-GPU deployments.

I tried deploying some workloads to this provider and I was unable to get any bids for this provider. We can't sign it if it isn't working even for non-GPU deployments.

Wait, why would that be? I used preator and I haven't changed any of the bidding settings or anything? Is it really an AKT shortage?


fpgamanager: replace /lib with ${nonarch_base_libdir}

Fixes a packaging error when using 'usrmerge' in DISTRO_FEATURES
(nonarch_base_libdir is set to /usr/lib in this case, and to /lib when
usrmerge is not used).
Signed-off-by: Ricardo Salveti<EMAIL_ADDRESS>

@ckalluri73 care to have a look? Thanks!

please merge

Closing the PR, sent the changes over email (mailing list).


Added type checking to parse_resource_extended

Added type checking to parse_resource_extended to prevent segmentation fault caused when resource_name is an integer.

I think we do not need this


Make Pastel OCaml < 4.05 compatible

Shadow bool_of_string_opt implementation

I would love to have the similar CI setup as facebook/reason to ensure we run against all OCaml versions.

Hey @bandersongit, is there any reason why this can't be merged? Currently blocking a bit :S

I forget why this didn't work and I don't actually have the permissions to merge this anymore. Tagging @kyldvs

I probably can yes, Thanks for the fast reply.