Error when starting Skkeleton

Describe the bug
When Neovim starts:
[denops]     at file:///Users/uhooi/.local/share/nvim/lazy/skkeleton/denops/skkeleton/deps/std/path.ts:1:15

When Skkeleton starts:
Error invoking 'invoke' on channel 4 (denops):
TypeError: this[#denops].dispatcher[fn] is not a function
    at Plugin.call (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/service.ts:176:45)
    at Service.#dispatch (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/service.ts:88:25)
    at Service.dispatch (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/service.ts:93:34)
    at invoke (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/host.ts:78:22)
    at Object.invoke (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/host/nvim.ts:31:16)
    at dispatch (https://deno.land/x/messagepack_rpc@v2.0.3/dispatcher.ts:36:36)
    at Session.#dispatch (https://deno.land/x/messagepack_rpc@v2.0.3/session.ts:244:28)
    at Session.#handleRequestMessage (https://deno.land/x/messagepack_rpc@v2.0.3/session.ts:271:53)
    at Session.#handleMessage (https://deno.land/x/messagepack_rpc@v2.0.3/session.ts:255:37)
    at Object.write (https://deno.land/x/messagepack_rpc@v2.0.3/session.ts:159:63)

Provide a minimal vimrc with less than 50 lines to reproduce
' TODO

How to reproduce the problem from Vim startup
None.
Expected behavior
Skkeleton starts without error.
Actual behavior
Error when starting Skkeleton.
Screenshots (if need)
None.
Your environment

OS: uname -a (macOS/Linux) or ver (Windows)$ uname -a
Darwin uhooinoMacBook-Air.local 23.5.0 Darwin Kernel Version 23.5.0: Wed May  1 20:16:51 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_T8103 arm64 arm Darwin

Deno version: deno --version$ deno --version
deno 1.38.2 (release, aarch64-apple-darwin)
v8 12.0.267.1
typescript 5.2.2

Vim/Neovim version: vim --version or nvim --version$ nvim --version
NVIM v0.11.0-dev-302+gbe999e6a0
Build type: Release
LuaJIT 2.1.1716656478
Run "nvim -V1 -v" for more info

Denops version: git log -1 (run in the denops.vim repository)$ git log -1
commit d196190ff2efd5efca38b0170c23155bdbd71f9d (HEAD -> main, tag: v6.0.9, origin/main, origin/HEAD)
Merge: c6fa7c6 54f3953
Author: Λlisue (Ali sue・ありすえ<EMAIL_ADDRESS>Date:   Sat Jun 22 13:32:29 2024 +0900

    Merge pull request #370 from vim-denops/fix_typo

    Fix typo

Additional context
None

@uhooi Could you try https://github.com/vim-denops/denops.vim/pull/374 and rewrite the error message on PR?

@lambdalisue New error.
It was on toggle, not on startup :pray:
[denops] Failed to load plugin 'skkeleton': TypeError: Could not find version of '@std/path' that matches specified version constraint '0.225.2'
[denops]     at file:///Users/uhooi/.local/share/nvim/lazy/skkeleton/denops/skkeleton/deps/std/path.ts:1:15
Too many errors. Disabling Noice
Error detected while processing function skkeleton#handle[1]..skkeleton#request[2]..denops#request[1]..denops#_internal#server#chan#request[6]..denops#_intern
al#rpc#nvim#request:
line    1:
Error invoking 'invoke' on channel 4 (denops):
Error: Failed to call handle("toggle", {}, {"prevInput":"","completeInfo":{"pum_visible":0,"mode":"","selected":-1,"items":[]},"mode":"i","completeType":"nati
ve"}) in skkeleton: this[#denops].dispatcher[fn] is not a function
    at Plugin.call (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/service.ts:181:13)
    at Service.#dispatch (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/service.ts:88:25)
    at Service.dispatch (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/service.ts:93:34)
    at invoke (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/host.ts:78:22)
    at Object.invoke (file:///Users/uhooi/.local/share/nvim/lazy/denops.vim/denops/@denops-private/host/nvim.ts:31:16)
    at dispatch (https://deno.land/x/messagepack_rpc@v2.0.3/dispatcher.ts:36:36)
    at Session.#dispatch (https://deno.land/x/messagepack_rpc@v2.0.3/session.ts:244:28)
    at Session.#handleRequestMessage (https://deno.land/x/messagepack_rpc@v2.0.3/session.ts:271:53)
    at Session.#handleMessage (https://deno.land/x/messagepack_rpc@v2.0.3/session.ts:255:37)
    at Object.write (https://deno.land/x/messagepack_rpc@v2.0.3/session.ts:159:63)

@uhooi Not sure but it seems the reason is same with https://github.com/vim-denops/denops.vim/issues/358. Could you try :call denops#cache#update(#{reload: v:true})?

@lambdalisue
I ran the command and it fixed it.
Thank you.


Remove .golangci.yml

Current Behavior

Desired Behavior

Implementation

Acceptance Tests

Mockups

Contributor Guide

🎨 Wireframes and designs for Layer5 site in Figma (open invite)

@nebula-aac Is it a good first issue to solve if yes can I help to solve it?

hey @nebula-aac can you assign it to me?

Hi @nebula-aac, I want to work on this issue. I can fix this. Can you please assign it to me.

@nebula-aac do you just want full file to be removed


Add vssps.dev.azure.com using the dev.azure.com

Hi.
While building an azure devops agent,
A proxy allowed address was missing and a pull request was created.

@iz4blue : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"


Fix sample analyzer name in the README

The README uses both SimpleAnalyzer and CaptureAllMetricsAnalyzer in usage snippets which means the examples don't work via copy/paste. This PR standardizes on SimpleAnalyzer but I don't have strong opinions on which name is better.

Thanks Joel!


Feature request: record video

Recording video is very hard in android and recording video has not security problem because it's possible to do but hard (a feature that is hard for implementing is a negative point of android)
also it's aspect ratio is not OK with front camera (Samsung galaxy note 3)

+1 for video recording.
Is there any plan for this feature?

Please add video Feature

check this https://github.com/google/cameraview/pull/218


CountDownLatch static -> instance

Fixes #717
Changes proposed in this pull request:
-Holder CountDownLatch static -> instance

Codecov Report

Merging #959 into dev will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##              dev     #959      +/-   ##
==========================================
+ Coverage   52.52%   52.54%   +0.02%
==========================================
  Files         569      569
  Lines       11931    11934       +3
  Branches     1882     1882
==========================================
+ Hits         6267     6271       +4
+ Misses       5305     5300       -5
- Partials      359      363       +4

Impacted Files
Coverage Δ

...ration/reg/newzk/client/zookeeper/base/Holder.java
0% <0%> (ø)
:arrow_up:

...ce/OrchestrationShardingDataSourceFactoryBean.java
66.66% <0%> (-11.12%)
:arrow_down:

.../internal/config/ConfigurationListenerManager.java
56.75% <0%> (+2.7%)
:arrow_up:

...ternal/state/instance/InstanceListenerManager.java
32.35% <0%> (+2.94%)
:arrow_up:

...al/state/datasource/DataSourceListenerManager.java
40% <0%> (+3.33%)
:arrow_up:

...tion/internal/config/ConfigMapListenerManager.java
61.9% <0%> (+4.76%)
:arrow_up:

...OrchestrationMasterSlaveDataSourceFactoryBean.java
77.77% <0%> (+11.11%)
:arrow_up:

Continue to review full report at Codecov.

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


Beast Server Exits for reasons that Client is disconnecting?

Version of Beast
1.84
I hope this finds you well. I've been utilizing the Beast lib successfully for over three years! Great library!
It's not a frequent occurrence, but occasionally, I encounter this error where the beast server exits with the following message. I'm curious about the possible cause and whether there's anything specific I should address, perhaps within the fail method or elsewhere? It cause some sort of stability issue. Thank you, as always!
write: An existing connection was forcibly closed by the remote host
read: The I/O operation has been aborted because of either a thread exit or an application request
read: An existing connection was forcibly closed by the remote host

Additionally, I have a question on the side:
How can I retrieve the client's IP address based on beast::ssl_streambeast::tcp_stream stream?
Thanks,
Tharma

It's not a frequent occurrence, but occasionally, I encounter this error where the beast server exits with the following message. I'm curious about the possible cause and whether there's anything specific I should address

It appears that a client connection is being closed ungracefully based on the error messages. I'm not sure about your error handling approach, but typically, your server application shouldn't terminate due to client connection errors. It's important to anticipate that asynchronous operations on client sockets may encounter failures.

How can I retrieve the client's IP address based on beast::ssl_streambeast::tcp_stream stream?

beast::tcp_stream tcp_stream{ ioc };
beast::ssl_stream<beast::tcp_stream> ssl_stream{ ioc, ssl_ctx };

auto ep1 = get_lowest_layer(ssl_stream).socket().remote_endpoint();
auto ep2 = tcp_stream.socket().remote_endpoint();

Thank you very much.


URL popups on links disrupt the flow of text with screen readers

In the body of your blog posts, each link has a popup that displays the URL of the link when the pointer is over the link text. But in the current implementation, these URLs are always read by screen readers, disrupting the flow of the text. I can reproduce this with the Narrator and NVDA screen readers on Windows.
I think the reaon this is happening is that the URL popups are always technically visible; they just have a size of zero when the mouse isn't hovering over the link. I suggest that each popup should have a CSS display property of "none" when it's not supposed to be visible.

ah i didnt know this. thank you! i intend to do a total rewrite of the site soon so i'll keep this in mind and resolve when its done.

i believe this is gone. thanks again for reporting!


Test: fill out work preference

User Story
As a quality assurance engineer,
I want to thoroughly test the functionality of the input field for describing work,
So that I can ensure it works correctly and provides a seamless user experience.
Acceptance Criteria

[ ] Develop test cases to cover all aspects of the input field functionality.
[ ] Implement and execute automated tests for the input field on the preferences screen.
[ ] Verify that the input field is displayed only when "work" is selected as a preference.
[ ] Test the functionality of the input field to ensure users can input descriptive information about their work.
[ ] Ensure that the input field behaves correctly on different devices and screen sizes.
[ ] Document and report any issues or bugs found during testing.

Related Issues

[Issue number]: Implement input field for describing work on preferences screen

Notes

Thorough testing of the input field is crucial to ensure a seamless user experience and accurate functionality.
Test cases should cover various scenarios, including valid and invalid user inputs.
Collaborate with developers to address any issues or bugs found during testing.

Tasks

[ ] Create comprehensive test cases for the input field functionality.
[ ] Set up an automated testing environment and framework if not already in place.
[ ] Execute the test cases to verify the behavior of the input field.
[ ] Verify that the input field is displayed correctly and prompts users to input descriptive information about their work.
[ ] Test the functionality of the input field on different devices and screen sizes.
[ ] Document and report any issues or bugs discovered during testing.

This is done on Mobile.

done on web


Unknown parsing error on custom tool + custom agent implementation

System Info
langchain: 0.0.215
python: 3.10.11
OS: Ubuntu 18.04
Who can help?
@hwchase17 @agola11
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[X] Agents / Agent Executors
[X] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
I'm implementing two custom tools that return column names and unique values in them. Their definition is as follows
def get_unique_values(column_name):
    all_values = metadata[column_name]
    return all_values

def get_column_names():
    return COLUMN_NAMES

tools = [
    Tool(
        name="Get all column names",
        func=get_column_names,
        description="Useful for getting the names of all available columns. This doesn't have any arguments as input and simply invoking it would return the list of columns",
    ),
    Tool(
        name="Get distinct values of a column",
        func=get_unique_values,
        description="Useful for getting distinct values of a particular column. Knowing the distinct values is important to decide if a particular column should be considered in a given context or not. The input to this function should be a string representing the column name whose unique values are needed to be found out. For example, `gender` would be the input if you wanted to know what unique values are in `gender` column.",
    ),
]

I'm using a custom agent as well defined as follows: (The definition is quite straightforward and taken from the official docs)
prompt = CustomPromptTemplate(
    template=template,
    tools=tools,
    input_variables=["input", "intermediate_steps"],
)

llm_chain = LLMChain(llm=llm, prompt=prompt)
tool_names = [tool.name for tool in tools]

agent = LLMSingleActionAgent(
    llm_chain=llm_chain,
    output_parser=output_parser,
    stop=["\nObservation:"],
    allowed_tools=tool_names,
)

agent_executor = AgentExecutor.from_agent_and_tools(
    agent=agent, tools=tools, verbose=True
)

When I provide a prompt using agent_executor.run() I get the error pasted below.
Surprisingly, if I define the as follows, I don't get an error. Rather it doesn't follow my prompt template because there's no LLMChain used here and hence gets stuck in meaningless back and forth actions.
agent_executor = initialize_agent(
    tools,
    llm,
    agent=AgentType.STRUCTURED_CHAT_ZERO_SHOT_REACT_DESCRIPTION,
    verbose=True,
)

Error trace
> Entering new  chain...
Traceback (most recent call last):
  File "/home/user/proj/agent_exp.py", line 245, in <module>
    agent_executor.run(
  File "/home/user/anaconda3/envs/proj/lib/python3.10/site-packages/langchain/chains/base.py", line 290, in run
    return self(args[0], callbacks=callbacks, tags=tags)[_output_key]
  File "/home/user/anaconda3/envs/proj/lib/python3.10/site-packages/langchain/chains/base.py", line 166, in __call__
    raise e
  File "/home/user/anaconda3/envs/proj/lib/python3.10/site-packages/langchain/chains/base.py", line 160, in __call__
    self._call(inputs, run_manager=run_manager)
  File "/home/user/anaconda3/envs/proj/lib/python3.10/site-packages/langchain/agents/agent.py", line 987, in _call
    next_step_output = self._take_next_step(
  File "/home/user/anaconda3/envs/proj/lib/python3.10/site-packages/langchain/agents/agent.py", line 803, in _take_next_step
    raise e
  File "/home/user/anaconda3/envs/proj/lib/python3.10/site-packages/langchain/agents/agent.py", line 792, in _take_next_step
    output = self.agent.plan(
  File "/home/user/anaconda3/envs/proj/lib/python3.10/site-packages/langchain/agents/agent.py", line 345, in plan
    return self.output_parser.parse(output)
  File "/home/user/proj/agent_exp.py", line 102, in parse
    raise OutputParserException(f"Could not parse LLM output: `{llm_output}`")
langchain.schema.OutputParserException: Could not parse LLM output: `Thought: I need to get all the column names
Action: Get all column names`

Expected behavior
The agent should have called these functions appropriately and returned a list of columns that have the string datatype

@saswat0 Try removing the whitespaces from your tool names. For example,
Change
  name="Get all column names",

To
  name="get_all_column_names",

@devstein Thanks for this tip!
Out of curiosity, Is this supposed to be a rule while defining such tools?

Of course! Based off of the error message and the automated agent's comment, it seems likely that the default tool output parser doesn't handle spaces.
I haven't checked the code, so would recommend verifying yourself.


Configure Azure Pipelines

This configuration builds the application and produces builds for both Debug and Release configurations.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: sompmak44 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


feat(defi): idle multi-account

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #3248

PR #3253

PR #3261   👈

This comment was auto-generated by Graphite.


github.io webpage invalid

I found the tutorial page(https://juliaai.github.io/DataScienceTutorials.jl/) is invalid for several days. Could you please deploy the webpage again? Thanks.

it's back online, that was due to the process of updating to MLJ 0.17, thanks for reporting. Note that there may be some mistakes for a short while on some of the pages while we review changes.


It should be possible to set the propagation mode of a container's mnt namespce

Propagation modes control how mount and umount events propagate between mount namespaces.  In order to enable use-cases like a container that performs mounts that are visible to other containers (such as containerized Kubernetes/OpenShift), it should be possible to control the mount propagation mode of a container.  There is already an issue for this in runc, and a corresponding PR:
https://github.com/opencontainers/specs/issues/56
https://github.com/opencontainers/runc/pull/77
...but those don't cover docker until runc is integrated.  This issue is to make this functionality available in docker before runc is integrated.
@mrunalp @rhatdan @crosbymichael

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Duplicate of #10088 for which a PR can be found at #14097

@phemmer It is actually not a dupe -- #10088 is about controlling which mount namespace the container runs in -- which is orthogonal to controlling the propagation mode of the mount ns.


style: Fix style spacing inconsistency

no lines have changed
https://github.com/twosigma/beaker-notebook/pull/1971/files?w=1


Add documentation about nofetch()

Add documentation about nofetch()
Originally posted by @ronaldtse in https://github.com/metanorma/metanorma-standoc/issues/146#issuecomment-552259777

Already done: https://github.com/metanorma/metanorma.com/pulls
Seriously, you don't need to raise these tickets.


Instructions for EnableServiceProfilerInVS sample are a bit outdated with respect to Docker tooling

Starting with Visual Studion 15.8 enabling Docker does not automatically add a docker-compose.yml file to the project. You need to do an extra step of right-clicking the project and doing Add | Container Orchestrator Support:

Fixed by #30.


Add release status to library list

https://kitsu.canny.io/feature-requests/p/bring-back-currently-airing-and-not-aired-yet-tags-for-anime-again

:+1:


Load from build/compile_commands.json

This resolves #586 (albeit in a stupidly direct way; it's easy to imagine the desirability of at least a bit of configuration to be available).

This is my attempt at the discussed changes.

Thanks!


[FP]: keycloak for CVE-2021-3637

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:redhat:keycloak::::::::
CVE
CVE-2021-3637
ODC Integration
None
ODC Version
9.0.5
Description
Actual vulnerable component is keycloak-model-infinispan jar.

The provider is released as part of the keycloak project; DependencyCheck project is not attributing NVD CPE associated vulnerabilities to sub-libraries of the CPE


How can I use this project with ExternalProject_Add

Hi Vikas,
This is not an issue, but I am having trouble using this project with ExternalProject_Add(). I'm expecting it to build/install and have the include directories available even before my project is considered. But when I include hotpatch.h it cannot find it. I'm new to cmake so please forgive me for being a noob.
This is what I have so far.
cmake_minimum_required(VERSION 3.10)
project(agentloader)
set(CMAKE_C_STANDARD 99)
include(ExternalProject)
ExternalProject_Add(project_hotpatch
        GIT_REPOSITORY https://github.com/vikasnkumar/hotpatch.git
        PREFIX ${CMAKE_CURRENT_BINARY_DIR}/hotpatch)
ExternalProject_Get_Property(project_hotpatch install_dir)
add_library(hotpatch STATIC IMPORTED)
set_property(TARGET hotpatch PROPERTY IMPORTED_LOCATION ${install_dir}/lib/hotpatch_s)
add_dependencies(hotpatch project_hotpatch)
include_directories(${install_dir}/include)
add_executable(agentloader main.c agentloader.h)
target_link_libraries(agentloader hotpatch)

I manage to solve my problems using the FetchContent module


Fix JsonSyntaxException when creating API command response #4355

Description

Added missing \" (from "*****" to "*****\"") at obfuscatePassword method from AsyncJobManagerImpl.java#L485
https://github.com/apache/cloudstack/blob/0f3f2a09370a18301db28ec3d28efe746b6437c9/framework/jobs/src/main/java/org/apache/cloudstack/framework/jobs/impl/AsyncJobManagerImpl.java#L485
The lack of " caused a JsonSyntaxException exception when the host response had a password (e.g. if the host has out of band enabled e.g. IPMI or Redfish).

Fixes: #4355
Types of changes

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

How Has This Been Tested?

JUnit test
manual tests

@blueorangutan package

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

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

@blueorangutan test

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

Trillian test result (tid-2910)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 29745 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4387-t2910-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_deploy_virtio_scsi_vm.py
Intermittent failure detected: /marvin/tests/smoke/test_outofbandmanagement_nestedplugin.py
Intermittent failure detected: /marvin/tests/smoke/test_outofbandmanagement.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 81 look OK, 4 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

ContextSuite context=TestDeployVirtioSCSIVM>:setup
Error
0.00
test_deploy_virtio_scsi_vm.py

test_oobm_issue_power_cycle
Error
2.17
test_outofbandmanagement_nestedplugin.py

test_oobm_issue_power_off
Error
1.16
test_outofbandmanagement_nestedplugin.py

test_oobm_issue_power_on
Error
1.16
test_outofbandmanagement_nestedplugin.py

test_oobm_issue_power_reset
Error
2.16
test_outofbandmanagement_nestedplugin.py

test_oobm_issue_power_soft
Error
2.18
test_outofbandmanagement_nestedplugin.py

test_oobm_issue_power_status
Error
2.17
test_outofbandmanagement_nestedplugin.py

test_oobm_enabledisable_across_clusterzones
Error
20.11
test_outofbandmanagement.py

test_oobm_issue_power_cycle
Error
4.29
test_outofbandmanagement.py

test_oobm_issue_power_off
Error
4.25
test_outofbandmanagement.py

test_oobm_issue_power_on
Error
4.24
test_outofbandmanagement.py

test_oobm_issue_power_reset
Error
3.25
test_outofbandmanagement.py

test_oobm_issue_power_soft
Error
3.26
test_outofbandmanagement.py

test_oobm_issue_power_status
Error
4.25
test_outofbandmanagement.py

test_oobm_zchange_password
Error
5.32
test_outofbandmanagement.py

test_hostha_kvm_host_degraded
Error
4.43
test_hostha_kvm.py

test_hostha_kvm_host_fencing
Error
4.45
test_hostha_kvm.py

test_hostha_kvm_host_recovering
Error
7.52
test_hostha_kvm.py

did work @GabrielBrascher
======================================================================
ERROR: Tests degraded HA state when agent is stopped/killed
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/marvin/tests/smoke/test_hostha_kvm.py", line 355, in test_hostha_kvm_host_degraded
    self.assertIssueCommandState('ON', 'On')
  File "/marvin/tests/smoke/test_hostha_kvm.py", line 493, in assertIssueCommandState
    self.issuePowerActionCmd(command)
  File "/marvin/tests/smoke/test_hostha_kvm.py", line 539, in issuePowerActionCmd
    raise e
Exception: Job failed: {jobprocstatus : 0, created : u'2020-10-08T02:10:21+0000', completed : u'2020-10-08T02:10:21+0000', cmd : u'org.apache.cloudstack.api.command.admin.outofbandmanagement.IssueOutOfBandManagementPowerActionCmd', userid : u'a8138930-08c3-11eb-8574-1e00ca0138c9', jobstatus : 2, jobid : u'f8439839-f821-4bd6-b7a3-820345992038', jobresultcode : 530, jobresulttype : u'object', jobresult : {errorcode : 530, errortext : u'com.google.gson.stream.MalformedJsonException: Unterminated object near "password":"p*****"","action":"ON","desc'}, accountid : u'a812be97-08c3-11eb-8574-1e00ca0138c9'}

note that in this case the replacement yield a double double quote, which is one in excess.

@DaanHoogland what about removing the password from the JSON response?

@GabrielBrascher If sometimes an extra '"' is added and sometimes not, this is still going to be an issue if you try to remove it. I'd rather replace the exact "something" with "*" or "s". it seems that we are brushing a bug under the carpet otherwise.

@DaanHoogland I proposed removing the password because I don't see much of use in having it on the response, it seems that does not aggregate relevant information.

It is adding value but only if having a password is optional, @GabrielBrascher. And i think hiding it should be optional as well as a private cloud operator might opt for having it visible. I am not going to argue that anymore, btw.

@DaanHoogland I found out why some cases the regex failed and some not. Updated code to fix it (for good, I hope). Also added test case addressing both situations.
The regex uses comma to set the split separator: String psswd = resp[1].toString().split(",")[0];
This works on cases such as:
...,"password":"password","action":"OFF"...

However, the specific case that I got with OOBM it has the following JSON String:
"username":"root","password":"password"},"resourcestate":"PrepareForMaintenance","hahost":false

With that said, the code has been updated to deal with both cases: "password":"password"}, and "password":"password",.

@blueorangutan package

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

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

@blueorangutan test

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

Trillian test result (tid-2969)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 37791 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4387-t2969-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 85 look OK, 0 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

@rhtyd no further comments?


Show warning if built avatar does not have WriteDefault all on

Need to see if there is exceptions?

Can avoid this concern by using more ambiguous wording "This may cause issues" etc


Create print-repository-names-with-description subcommand

Pull requests:

https://github.com/Senzing/github-util/pull/80


added watchit activity

#297
Added a prototype of how the activity would look in its initial form.
Please approve if I should proceed with the same.

I'm not able to play the game so don't understand how it works.
Could you explain?
Plus it's impossible to renew the game without restarting the activity.
Warning regarding the size of the activity: more than 20Mb already.

Playing Instructions
We have to trace the path between red lines with the cursor, it is easy as of now for a prototype but will be really challenging in upcoming levels due to animating obstacles.
Space Issue
It is solved by removing unwanted libraries present in the directory
Renew Issue
I will fix the renew thing as soon as possible by adding an option to restart the game but as of now I am facing this error randomly, soon after I add an activity.

Finally find the way to play.
It's too complex and will be unplayable on a touch screen where there is no something like a cursor.
Regarding your issue, I suggest you to fully clear your browser cache or typing "localStorage.clear()" on the browser console.

I'm unable to find a way to implement it on touch screen efficiently, can you suggest me one.

I am making the activity less complex and compatible by:-
1- Widening the path
2- Introducing a game object to move with onscreen controls(touch)/keyboard(desktop)

Not usable and not pertinent for the moment.


Rename:  basic/advanced application template → basic/advanced project template

I propose renaming basic/advanced application template to basic/advanced project template. The reason is advanced one which contains multiple tiers which are applications in Yii terms. So overall now it's confusing that application contains 2 applications.
No code changes required, it's just terminology.

@yiisoft/core-developers thoughts?

Fine with me.

Make sense.

Done.

:+1:


Return runner's defer in Application.run

Hi,
I'm working on a lib to easily integrate autobahn in a synchronous application (see https://github.com/Scille/autobahn_sync)
Given I use crochet, the twisted reactor is already started when I run Autobahn.Application, which means I have to handle by myself the connection errors  (see https://github.com/Scille/autobahn_sync/blob/53c7303a06cf53a783fabbc550b334d48d90d05a/autobahn_sync/core.py#L599)
This is fine except Autobahn.Application.run doesn't return the ApplicationRunner's defer.
To fix that I had to copy the Application.run function inside my app (see https://github.com/Scille/autobahn_sync/blob/master/autobahn_sync/core.py#L59), which feels like adding complexity for nothing...

Current coverage is 57.39%

Merging #603 into master will not affect coverage as of 757e1b4

@@            master    #603   diff @@
======================================
  Files           61      61
  Stmts        10229   10229
  Branches      1662    1662
  Methods          0       0
======================================
  Hit           5871    5871
  Partial        556     556
  Missed        3802    3802

Review entire Coverage Diff as of 757e1b4

Powered by Codecov. Updated on successful CI builds.

LGTM, thanks:)


In Memory Example does not work

I'm following the instructions here: http://ldapjs.org/examples.html and have started the server (using the unmodified code on the page).
Next I create user.ldif which looks like:
dn: cn=ldapjs, o=joyent
objectClass: unixUser
cn: ldapjs
shell: /bin/bash
description: Created via ldapadd

I add using ldapadd:
ldapadd -H ldap://localhost:1389 -x -D cn=root -w secret -f ./user.ldif

which replies with:
adding new entry "cn=ldapjs, o=joyent"
Then I try to find the entry with ldapsearch:
ldapsearch -H ldap://localhost:1389 -LLL -x -D cn=root -w secret -b "o=joyent" cn=ldapjs

No such object (32)
Matched DN: o=joyent
Additional information: o=joyent
Where'd I go wrong?

I would recommend using the debugger to step through the add and search functions of that example to determine the root cause of the issue. If you determine the issue to be in the example code, we would welcome a PR to fix it.

👋
On February 22, 2023, we released version 3 of this library. As a result, we are closing this issue/pull request.
Please see issue #839 for more information, including how to proceed if you feel this closure is in error.


GetRecordsResp - add MillisBehindLatest to response object

http://docs.aws.amazon.com/kinesis/latest/APIReference/API_GetRecords.html#Kinesis-GetRecords-response-MillisBehindLatest

Thanks, looks good. Just need wait @mikerowan :)


Return expanded property as subtype

Is is possible to return an expanded property with its subtype properties e.g. given
var user = client.For<Person>
                 .Expand(x => x.Pet)
                 .FindEntriesAsync()

How do I...

Request only where the Pet is a Dog
Return the Pet as a Dog so that I don't have to do a Select N+1 to get the other properties

Ok, re-reading the documentation I can answer point 1 as follows...
var user = client.For<Person>
                 .Filter(x => x.Pet is Dog)
                 .Expand(x => x.Pet)
                 .FindEntriesAsync()

I think my polymorphic issue is addressed in one of the answers to #158 but for that to work DictionaryExtensions needs to be made public or @zivillian code is adopted.

Sometimes you wouldn't want to filter but rather want all Pet. In that case, even the derived type is defined in the metadata and is annotated in the JSON, OData client fails to use that type.
See https://github.com/simple-odata-client/Simple.OData.Client/issues/622#issuecomment-520210735.


Installation instruction for Ubuntu 22 with Clang-12

When I configure the build directory with
cmake .. -DCMAKE_BUILD_TYPE:STRING=Release -G Ninja -DCMAKE_C_COMPILER:STRING=clang-12 -DCMAKE_CXX_COMPILER:STRING=clang++-12 -DMJPC_BUILD_GRPC_SERVICE:BOOL=ON

under Ubuntu 22.04 with Clang-12 installed, I encountered the following error:
==============================================================================================
-- The C compiler identification is Clang 12.0.1
-- The CXX compiler identification is Clang 12.0.1
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/clang-12 - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - failed
-- Check for working CXX compiler: /usr/bin/clang++-12
-- Check for working CXX compiler: /usr/bin/clang++-12 - broken
CMake Error at /usr/share/cmake-3.22/Modules/CMakeTestCXXCompiler.cmake:62 (message):
The C++ compiler
"/usr/bin/clang++-12"

is not able to compile a simple test program.
It fails with the following output:
Change Dir: /home/oscar/mujoco_mpc/build/CMakeFiles/CMakeTmp

Run Build Command(s):/usr/bin/ninja cmTC_73a1c && [1/2] Building CXX object CMakeFiles/cmTC_73a1c.dir/testCXXCompiler.cxx.o
[2/2] Linking CXX executable cmTC_73a1c
FAILED: cmTC_73a1c
: && /usr/bin/clang++-12   CMakeFiles/cmTC_73a1c.dir/testCXXCompiler.cxx.o -o cmTC_73a1c   && :
/usr/bin/ld: cannot find -lstdc++: No such file or directory
clang: error: linker command failed with exit code 1 (use -v to see invocation)
ninja: build stopped: subcommand failed.

CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
CMakeLists.txt:25 (project)
-- Configuring incomplete, errors occurred!
==============================================================================================
After googling the problem, I need to install this
sudo apt-get install libstdc++-12-dev

Then clean up the build directory and configure again with success. After that, continue the build process.

Thanks for the fix! I opened a PR (#331) that adds this additional step to the README.


Need to update console log to update Address of pytesthistoric from <IP_ADDRESS>:50000 to locatohost:5000 or <IP_ADDRESS>:5000

Refer Screenshot below :

Above log is auto generated by Flask, we are intentionaly starting app in <IP_ADDRESS>:5000 so app can be accessed within LAN (others can also access to report using hostip:5000)

Closing this issue since we dont modify it (but updated in wiki)


ROX-23836: Add vulnerabilities container image

Description
Adds the vulnerability definitions that we store in a GCS bucket (eventually leading to definitions.stackrox.io) into a container image.
Testing
CI before squashing the commits: https://github.com/stackrox/scanner/actions/runs/9372542159

/retest

/retest

quay.io/stackrox-io/vulnerabilities does not have any active tags (they look to have been deleted) - is that a concern? expected?

Expected. They were on a 7-day expiration so they all expired at this point.

/retest


Generics with funcs/services

Should funcs/services be able to use generics? Is this even desirable or possible?

We are closing this issue anticipating an entire rearchitecture of Kybra similar to what we have done with Azle. This removes most of the compiler architecture as it is. We believe that this issue no longer applies to the new planned architecture.
In case our rearchitecture is not successful, or we otherwise need to find these closed issues, the search term KYBRA_REARCHITECTURE can be used.


feat: port Combinatorics.SimpleGraph.Subgraph

Remaining task:

[ ] Fix error with congr in degree_spanningCoe

@int-y1 I put in a different resolution to the congr proof problem. I think the "right" way to deal with this is apply Subsingleton.elim. This is something congr used to do itself.

bors r+


** (MatchError) no match of right hand side value: {:error, {:postgrex, {'no such file or directory', 'postgrex.app'}}} even when using mysql

Im not using postgresql but mysql i added mariaex in config still the error.
config :auth, Auth.Repo,
  adapter: Ecto.Adapters.MySQL,
  ...

config :auth,
  ecto_repos: [Auth.Repo]

  defp deps do
    [{:ecto, "~> 2.1"},
      {:mariaex, "~> 0.8.2"}]
  end

im using umbrella app

Can you please provide an application that reproduces the error? Try searching for "postgrex" in your whole project and see if there is anything left.

I had postgrex installed(default by phoenix) in one of apps_web i was calling auth from.
After i removed it and did a clean compile (deps.clean deps.compile) it worked nicely.

can you tell me where did you removed the postgrex installer ? im getting the same error

It's generally best to ask this on the forums, but in general check all of your configs to see what database type is being used.

thanks, repo.ex had "adapter: Ecto.Adapters.Postgres" by default, that was the error

thanks, repo.ex had "adapter: Ecto.Adapters.Postgres" by default, that was the error

Thank you @julian3ap


Add RandWire

This PR adds RandWire.
Please wait for the experimental results.

ToDo:

init
test
doc

Errors reported by travis-ci are seems to rely on another parts.

This is experimental result.
ResNet56(9-9-9) / RandWire56(9-9-9) with pre-activation
300epoch with reducing learning rate, 0-149: 0.05, 150-224: 0.005, 225-299: 0.0005.
NesterovAG, batchsize=128

ResNet and RandWires seem to be same performance, like original paper.

Thanks for the quick implementation of the paper and the experimental results. The code looks clean and we'd like to make it available to other users but are wondering if this isn't better introduced as an example to the Chainer repository. If we observe that the community leans towards integrating RandWire into various kinds of experiments, we could reconsider introducing this as a feature of Chainer. By then, the desired interface might have changed and we could adapt to the more matured needs although I see that you've already worked on generalizing it. We could of course introduce this as an experimental feature, but again, cannot be sure how this algorithm (and maybe related but other AutoML algorithms) might evolve over time.


[1.10.2 / Sponge 5.1.0-BETA-1961 / pjp-5.1.0-0.13.6] Sign portals lost function

Hallo together,
I upgrqaded my minecraft Server (with pjw, and pjp) and it seems some sign portals lose after a restart their function. Some other work fine the whole time.
I didn't found any logs or errors to these problem.
Thanks.

Hello mister-man!
I suggest you to remove these sign-portals and create new ones. If it still doesn't work I'll then recommend you delete the .data file in pjp folder. Otherwise you can also use an other way to transport yourself through. Something like an button or an pressure plate seems to be an good option according to me.
Best Regards,
COLDYY1

unable to reproduce. Try updating to latest release


installing 7.3.9 msi is NOT updating 7.3.8

Prerequisites

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

Steps to reproduce
I've downloaded the msi
PowerShell-7.3.9-win-x86.msi
I install it
but
PS is not updating

Expected behavior
PS should update

Actual behavior
PS is not updating

Error details
no error

Environment data
PowerShell 7.3.8

Visuals

How did you install 7.3.8? Perhaps there are different install methods between 7.3.8 and 7.3.9
What is $PSHome and $env:Path:
$PSHome
$env:Path.Split(';')

I tried to reproduce. Our software deployment tool uses msiexec /i "PowerShell-7.3.9-win-x64.msi" /qn /norestart ADD_EXPLORER_CONTEXT_MENU_OPENPOWERSHELL=1 ENABLE_PSREMOTING=1 REGISTER_MANIFEST=1
Result is two entries in Programs list:
PS C:\> $software = $null
$software += get-itemproperty HKLM:\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\* -Name Publisher,DisplayName,DisplayVersion,UninstallString -ErrorAction SilentlyContinue | Sort-Object -Property Publisher,DisplayName | Select-Object -Property Publisher,DisplayName,DisplayVersion,UninstallString
$software += get-itemproperty HKLM:\SOFTWARE\WOW6432Node\Microsoft\Windows\CurrentVersion\Uninstall\* -Name Publisher,DisplayName,DisplayVersion,UninstallString -ErrorAction SilentlyContinue | Sort-Object -Property Publisher,DisplayName | Select-Object -Property Publisher,DisplayName,DisplayVersion,UninstallString
$software += get-itemproperty HKCU:\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\* -Name Publisher,DisplayName,DisplayVersion,UninstallString -ErrorAction SilentlyContinue | Sort-Object -Property Publisher,DisplayName | Select-Object -Property Publisher,DisplayName,DisplayVersion,UninstallString
$software += get-itemproperty HKCU:\SOFTWARE\WOW6432Node\Microsoft\Windows\CurrentVersion\Uninstall\* -Name Publisher,DisplayName,DisplayVersion,UninstallString -ErrorAction SilentlyContinue | Sort-Object -Property Publisher,DisplayName | Select-Object -Property Publisher,DisplayName,DisplayVersion,UninstallString
$software | sort Publisher,DisplayName | ? DisplayName -like "*PowerShell*" | ft Publisher,DisplayName,DisplayVersion,UninstallString -AutoSize -Expand Both

Publisher             DisplayName            DisplayVersion UninstallString
---------             -----------            -------------- ---------------
Microsoft Corporation PowerShell <IP_ADDRESS>-x64 <IP_ADDRESS>        "C:\ProgramData\Package Cache\{1897b6e9-b6f9-4529-b7a6-f3f6547b8131}\PowerShell-7.3.8-w...
Microsoft Corporation PowerShell 7-x64       <IP_ADDRESS>        MsiExec.exe /X{959CB28B-C5F3-4B66-9F8C-EC1F02E15115}

But it seems it was an error in 7.3.8 package that created that 1st entry. I have another windows server that updated from 7.3.4 to 7.3.9 and it only shows the 2nd entry after update. On both servers (7.3.8 to 7.3.9 and 7.3.4 to 7.3.9) running WIN+R pwsh.exe opens 7.3.9.

How did you install 7.3.8? Perhaps there are different install methods between 7.3.8 and 7.3.9 What is $PSHome and $env:Path:
$PSHome
$env:Path.Split(';')

For giggles I tried to use the PowerShell-7.3.9-win-x86.msi
uninstall tool.
No joy.
7.3.8 is still there.
Tried installing with PowerShell-7.3.9-win-x86.msi
No joy.
7.3.8 is still there.

PowerShell-7.3.9-win-x86.msi

Remove and install PowerShell-7.3.9-win-x64.msi

PowerShell-7.3.9-win-x86.msi

Uninstall and install PowerShell-7.3.9-win-x64.msi

I TRIED to install using PowerShell-7.3.9-win-x64.msi UI
No joy.

Uninstalling I see that 7.3.9 is installed but the PS UI say something else!

UI shows

Strange behavior, I upgraded from 7.3.8 without problems. If I had this problem, I would uninstall everything and install 7.3.9.

I have a similar issue where 7.3.9 and 7.3.0 co-exist typing pwsh leads to the 7.3.0 version...

cc @TravisEz13

Closing. After restart PS showed correct version.


Add hint on wget dependency

When I tried to install the gem I noticed that it uses wget to download some files.
Since I haven't installed wget, it took me a short time to figure out what went wrong.
It would be awesome to have some hints on that dependency in the README's install section.

I second this. Wasn't hard to figure out, but useful to include.


[suggestion] Shareable Prettier config

Having a shareable Prettier config as an autonomous library inside the monorepo would help opt-in/out in a more granular way. This could be helpful when integrating outside repos (as git submodules) with their own Prettier config already defined.

I agree this would be a great change, going to check the PR


Add CreatedAt column to PR view and Issue view

Is your feature request related to a problem? Please describe.
I always find a PR/Issue based on when it is created. So I will be nice if CreatedAt is shown on PR / Issue view.
Describe the solution you'd like
Add CreatedAt column to PR view and Issue view.
Describe alternatives you've considered
N/A

@kyu08 do you mean something beside the "clock" column?

Edit: just realized that's the last updated at column 🤷🏽

That's true. I want a CreatedAt column, not a UpdatedAt column.
Do you agree with adding it? If so, I'll try to implement it when I have time. (Maybe sometime in this month or next month)


Runtime validation with union based on multiple keys incorrectly fails

Report a bug
🧩 Context

ArkType version: 1.0.29-alpha
TypeScript version (4.8+): default on StackBlitz
Other context you think may be relevant (Node version, OS, etc.):

🧑💻 Repro

https://stackblitz.com/edit/arktype-repro-cdabvw?devToolsHeight=33&file=demo.ts
import { type, scope } from 'arktype';

const badScope = scope({
  a: {
    x: "'x1'",
    y: "'y1'",
    z: 'string',
  },
  b: {
    x: "'x1'",
    y: "'y2'",
    z: 'number',
  },
  c: {
    x: "'x2'",
    y: "'y3'",
    z: 'string',
  },
  union: 'a | b | c',
}).compile();

export const badType = badScope.union;

type Test = typeof badType.infer;

const value: Test = {
  x: 'x2',
  y: 'y3',
  z: '',
}; // no type error

export const { data, problems } = badType(value); // matches scope union item 'c'; should not fail

Thanks for the thorough report! This looks like an edge case bug for some discrimination scenario and is definitely very problematic.
I've added it as a test case for the upcoming 2.0 release and will ensure it is handled correctly on release. If you want to contribute a fix to 1.0, of course I'd happily accept it. I just want to make sure I'm doing everything I can to ship 2.0 as soon as possible!


Look further than Cargo.toml's directory by default

We have this code in a few places in Diesel. https://github.com/sgrif/diesel/blob/caea99a34e7846f4f1536c1454e94340ece256c0/diesel_tests/tests/schema.rs#L166-L168
It's pretty common for projects to have the code and tests as two crates (and sometimes more in the same repo), but want to share .env. Either looking one directory above Cargo.toml, or just continuing to recurse until either .env is found, or hitting root would help a very common use case.

I can put this together. partial to the latter. as far as I know cargo only looks in parents of the pwd for Cargo.toml/.cargo so recursing up from current_dir() should obviate the need to check CARGO_MANIFEST_DIR?

Yeah, I'm partial to the latter as well.

Should this be the default behaviour for dotenv() and/or from_filename(), or should this be a separate function or a flag? I'm not sure how common the opposite case (that is, projects that have several crates under the same hierarchy, but do not want to share .envs) is.
Thoughts? @sgrif @alicemaz

I would expect it to be the default behavior. I wouldn't imagine that projects sharing a directory are likely to have a .env in the shared directory if they don't want to share .envs. I think every case that would be affected by this change would currently error today.

Fair. Merging #19, then!


Possible C++/Arduino port?

With the chances of controlling the lights via BT in Home Assistant looking bleak - what do you think about porting the basic logic to c++ so it can be run on a simple ESP32 or similar, exposing the entities via MQTT?

You will find an Arduino Studio (c++) port of the Eaton Halo Home Lights here:
https://github.com/aeonSolutions/AeonLabs-OpenSourceCode-EatonHaloHome

That's unfortunately just using the bridge via the API, and I'm looking for a fully self hosted solution.


Add act evals on stagehand.page

why
#326 moved act from stagehand to stagehand.page and deprecated stagehand.act. This changes evals to account for this change.
what changed
Evals now point to stagehand.page.act instead of stagehand.act
test plan
evals

act evals passed, merging


dts.podtrac.com prevents podcasts from Downloading

Using Blokada on the LG V30 ATT Android 8.0 with this hosts list for adblocking and using Antennapod for Podcasts
Attempt to download a podcast, namely American History Tellers from Wondery and the download fails and says dts.podtrac.com is unable to resolve

@spookykidmm I removed that domain from my list in commit lightswitch05@49da5c534f0cb71de92d885708c0a5f497738165


New version: GeostatInversion v0.4.1

Registering package: GeostatInversion
Repository: https://github.com/madsjulia/GeostatInversion.jl
Created by: @montyvesselinov
Version: v0.4.1
Commit: 3d2eebb85f0a6b7db3c036f62335383139ac3759
Git reference: master
Release notes:

fix

Your new version pull request met all of the guidelines for automatic merging.
I will automatically merge this pull request during the next cron job.
If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]


WIP: Gfx1030 support

Gfx1030 support

Hi @stanleytsang-amd,
I created a refactored new PR, because I can't force push in this repository:
https://github.com/ROCmSoftwarePlatform/rocPRIM/pull/213
I am closing this PR.


ItemVerificationHandler is terminating ItemActionMessages

Kinda a big bug for plugins. The ItemVerificationHandler is terminating ItemActionMessages before some plugins are able to use the on {...} for them.
To reproduce modify MessageHandlerChain's notify function:
if (message.terminated()) {
    System.out.println("Terminated " + message.toString() + " from " + handler.toString());
    return false;
}

Then create a debugging plugin. I named mine debug.plugin.kts, and add the following to it:
on { ItemActionMessage::class}
        .then {
            println("Player interacted with item action " + this.id + " in ui " + this.interfaceId + " at slot " + this.slot + " with option " + this.option)
        }

You will see that the message is terminated prior to the debug plugin getting it. The strange thing is that the shop plugin is able to get a hold of the ItemActionMessage prior to the ItemVerificationHandler terminating it. But I have tested two other plugins I have written, and they are unable to get the message.

Nevermind. If the inventory cannot be verified, then the action will never get to the kotlin.


Clarity Needed for Admins Retiring On-Prem Active Directory

Like many before me (probably), I came to this article because I'm helping a client retire their on-premise Active Directory environment and move fully into Azure and I could use a little more clarity with the 'Delete a service instance from Azure AD Connect Health service' section.  The doc suggests that "All data from this service instance is deleted in accordance with the Azure data retention policy", but does that mean that users and groups that were initially synced via Azure AD Connect are also deleted or is it just referring to metadata / log data associated with monitoring.  Basically looking for the safest way to clean up Azure after permanently disabling Azure AD Connect.
Thanks!

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

ID: 35b52f01-3dee-6d41-f9f6-7710e461ffeb
Version Independent ID: c8ef08a8-d503-b52f-9de8-21b13cfa3d72
Content: Azure Active Directory Connect Health operations
Content Source: articles/active-directory/hybrid/how-to-connect-health-operations.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @zhiweiwangmsft
Microsoft Alias: billmath

@jsavage8
Thanks for your feedback! We will investigate and update as appropriate.

@jsavage8 you are asking a valid question but in the wrong place. This doc page is about AAD Connect Health Monitor which is essentially of no concern to you. You need to read up on AAD Connect Sync config, attribute versus AD group based sync filtering, how AAD identities change from Sync status = "In cloud" to "Synced from on-prem", and how to revert this, eg remove from sync scope which will soft delete them in AAD then you undelete them which will revert them back to Sync status = "In cloud" but at the cost of losing certain M365 config due to soft delete eg EXO mailbox and Teams settings if you use those services.

@jsavage8 you are asking a valid question but in the wrong place. This doc page is about AAD Connect Health Monitor which is essentially of no concern to you. You need to read up on AAD Connect Sync config, attribute versus AD group based sync filtering, how AAD identities change from Sync status = "In cloud" to "Synced from on-prem", and how to revert this, eg remove from sync scope which will soft delete them in AAD then you undelete them which will revert them back to Sync status = "In cloud" but at the cost of losing certain M365 config due to soft delete eg EXO mailbox and Teams settings if you use those services.

This refers to all of the data that the service instance was collecting.  It will not remove the users.
Thanks!
#please-close


Allow user to hide the "Join the Alpha" banner in the player

What's the problem? (required)
The player displays a "Join the Alpha" banner in the player that cannot be hidden unless the player is switched into full screen mode.
What's the current behavior? (required)
The player does not allow a user to hide the "Join the Alpha" banner in the player.
Do you have any personal experiences dealing with this problem? (optional)
A user of the player has commented to me that he finds the banner at the bottom of the player very distracting and would like there to be an "X" to close/hide it.
Describe a possible solution you've considered (optional)
Add an "X" to the banner that will close/hide the banner when the user clicks it.

@yondonfu maybe you can get @ericxtang to walk you through the gitcoin bounty process for this one 👍

@yondonfu will you still be doing gitcoin for this?


pypi: Prepare for the next major release.

@timabbott: FYI :)

Merged.  I think we should add an additional note about the package being available for Python 2 with (whatever the last released version supported that was)?


SDAPI works with errors

Entering the catalog view of the SDAPI tenant, the first page of works contains several that will not open. The works generate error messages. In the catalog search, the works with PDFs attached are the ones with error messages. Citation-only works (without files) are opening fine. From the homepage of the SDAPI tenant, the featured works with attached files are opening without errors.
https://sdapi.b2.adventistdigitallibrary.org/catalog?utf8=✓&locale=en&search_field=all_fields&q=
The only thing that I have done with this tenant since the Valkryie cutover was that yesterday (12/4) I ran two importers to clean up sets of works we had tried to import before Valkyrie, but Fedora glitched and dropped hundreds of works. So I ran the importers yesterday and brought in the missing works. The importers did NOT run cleanly, or else the enqueued and failed counts on the dashboard failed to update. I don't know which. I will create a new ticket for those issues, either way. I can't confirm what's going on with the importers because they will not load.
These are the importers:
https://sdapi.b2.adventistdigitallibrary.org/importers/557?locale=en
https://sdapi.b2.adventistdigitallibrary.org/importers/171?locale=en
The importers that ran yesterday do NOT contain the works that are listed on page one of the catalog, because the importers yesterday did NOT contain works with attached files. So I am confused why there is a connection between what I did and why certain works with errors have bumped to the top of the catalog search (as though they are most relevant, which I believe generally means "most recently updated and/or uploaded). These errored works were NOT at the top of the catalog search prior to me running the importers yesterday. It seems connected. I can't say how.

https://github.com/user-attachments/assets/6e8dbd0c-c49b-4803-998a-0494005ee091


Allow running on newer versions of IntelliJ (eg 2022.1 EAP)

The value pluginUntilBuild=213.* in gradle.properties is restricting the plugin to IntelliJ <= 2021.3, which means the plugin can't be installed into IntelliJ 2022.1 EAP.
Are there any expected compatibility concerns? Would it be possible to increase the version, or even avoid setting an upper version?

FWIW, I've been using it (locally built) on the EAP (and now the Beta) for weeks with no sign of a problem.

Thanks for verifying that. I will update and prepare a release likely next
week. There are a few more fixes that I would like to get in as well before
that.
On Thu, Mar 31, 2022 at 7:15 AM Sean Flanigan @.***>
wrote:

FWIW, I've been using it (locally built) on the EAP (and now the Beta) for
weeks with no sign of a problem.
—
Reply to this email directly, view it on GitHub
https://github.com/Netflix/dgs-intellij-plugin/issues/27#issuecomment-1084647479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ5JPXN7PFWYJYPLE66FQGDVCWXO3ANCNFSM5PS2EFVQ
.
You are receiving this because you commented.Message ID:
@.***>

This is also a problem with IntelliJ 2022.2 which was just released.

Thanks! We are in the process of fixing and releasing.
Sent from my iPhone

On Jul 26, 2022, at 8:53 PM, Sean Flanigan @.***> wrote:

This is also a problem with IntelliJ 2022.2 which was just released.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

IDEA upgrade prompt for 2022.3 reports the plugin as incompatible as well.

This is addressed and fixed.


Support Multiple Disks per Instance

Currently the script only supports the primary disk on an instance - it ignores any additional ones.
The script should be enhanced so that it takes a backup of all disks on an instance.
This functionality has already been added in forked versions:

awasilyev
jeremywadsack

@jacksegal  Thanks Jack, Was looking for multi disk support!

 Thanks a lot :+1:


Use rAF by default

'cause the internets are moving forward :earth_americas:

I started a branch with this. The only issue I'm having is trigging the rAF in the tests. Any thoughts?

You'll prob. need to set global.requestAnimationFrame in this file. I doubt that JSDom comes with it.

I'm going to close this as wontfix. After reading up on the proper use of rAF, I've decided it's too much boilerplate code to include in the library. I updated the inline comment to tell people what to do if they wish to change it.


Luau incorrectly parses numbers

print(-0x110000000000000000000000234567 + 0.000001 == -18446744073709552000)

Compiled Output

This will print true on Luau but on other Lua versions will print false

This is apparently intentional. Hexadecimal literal are parsed as uint64 and not IEEE binary64 so any values higher than the maximum will be capped to the maximum as in your case 0x110000000000000000000000234567 is capped to 0xFFFFFFFFFFFFFFFF as that's the maximum value of 64-bit unsigned integer and that value is higher than that.
In Lua 5.1, this depends on the behaviour of the strtod function (if lua_str2number is strtod); in C99, strtod accounts for hex floats, whereas in C89, it doesn't; so if you're running on Lua 5.1 compiled by C89-compliant compilers without any extensions then hexadecimal will be parsed as integers (32-bit or 64-bit - that also depends) so it'll be capped at the maximum if it goes over it, but if you're running on Lua 5.1 compiled by C99 or later compliant compilers then it'll be parsed as a float, and I believe this is unintentional. You'll notice that your compiled version of Lua 5.1 (unintentionally), Lua 5.2 and higher supports hexadecimal floats literals like 0x1p2 whereas in Luau it doesn't, though 0x1.2 and 0x1.2p3 is not a valid syntax literal in Lua 5.1 compiled by C99 or later compliant compilers.
I agree that hexadecimal and binary literals should be parsed as a float rather than integers as it makes dealing with IEEE nicer for me, and I don't see why hexadecimal literal is intentionally parsed as integers (though there are only relatively few use cases for this).

Note that even if we assume C99 (which we can) and use strtod to parse hexadecimal numbers (which, via support of hexadecimal floats, should produce correct results), we can't use that to fix the same problem with 0b literals that are too large to be represented.
Because Luau right now doesn't support hexadecimal floats it's also not clear if it's "correct" to do this, or if the more correct behavior is to emit an error.

This is apparently intentional.

I don't think the part about ignoring strtoull error return value was intentional (ULLONG_MAX + ERANGE).
But the problem with making this a parse error now is that existing code might stop compiling if someone used this intentionally or unintentionally.
Fixing the value is also a problem since the value in existing code will change from ULLONG_MAX to some other value which might expose latent bugs.

This is mostly fixed in 534 but we're assuming we can afford to make integer literals that are too large a parse error; we haven't validated this assumption yet, so I'll keep this open until we actually enable this fully.

This is mostly fixed in 534 but we're assuming we can afford to make integer literals that are too large a parse error; we haven't validated this assumption yet, so I'll keep this open until we actually enable this fully.

Should this issue be closed with this fix?

Not yet - per above, we have yet to look at the analytics to understand the impact this change would have. We'll update this when we know for sure.

Because of backwards compatibility issues with the previous change, we've decided to take a softer route:

Previously Luau erroneously parsed 0x0x0 as an integer; this has been fixed and is now a parse error
Long integers that fall outside of 2^64 limit like 0xffffffffffffffffffffffffffffffffffffffffffffff are still parsed without error and still are truncated to 2^64
However, we have a new linter pass that will flag these literals with a warning

In the future we might change the parsing behavior of out-of-bounds integer literals (Lua 5.1 and Lua 5.3+ disagree here so unsure yet about what the best long term behavior is), but for now we'll need to use the current behavior.

I would like to note that original report implies some sort of consistency/agreement between Lua versions, but they parse 0x110000000000000000000000234567 differently (already noted by @Blockzez) and result is false because all of those different results don't match -18446744073709552000:
Lua 5.1.5 C89:<PHONE_NUMBER>
Lua 5.1.5 C99: 8.8269046595092e+34
Lua 5.3.6: 2311527


Set default Emmet unit?

Issue Type: Feature Request
Right now, as it always has, Emmet defaults to px, so typing ml5 will get you margin-left: 5px;
I realize it will use any unit if I include a unit, as in ml5rem
Is it possible to have a Workspace setting for what the default unit is?  In a particular project we might be using rems everywhere.  Others the default px is fine.
just a thought.
VS Code version: Code - Insiders 1.23.0-insider (4971d889244a2e569ff03b7211c50e2a159e6e71, 2018-04-09T05:10:32.140Z)
OS version: Darwin x64 17.5.0

You can use the emmet.preferences setting to set css.unitAliases, css.intUnit, or css.floatUnit.
The various emmet preferences supported by VS Code are documented on the Emmet in VS Code page
The documentation of each of these preferences is in Emmet Preferences


[sc-98841] possibility to provide custom config fields as JSON at cluster creation

[sc-98841]

I was able to add some stuff using the new textbox. LGTM.
To be honest I was expecting a free text area like in the AKS plugin where you can use YAML to describe your whole cluster structure... But since that's what we do in the GKE plugin which is much more recent, I'm okay with your solution.
Just a question: is there any documentation on what is feasible with the custom text area? I did not find any for the GKE plugin. (It may be good to ask Fred about it)

documentation is the standard one from cloud provider:

GKE clusters
AKS clusters


[Help] input  when pressing  with menu open but no item selected

I'm sorry to open an issue that is just a question but sadly I can't figure it out at all by myself and asking on forums/Reddit has yielded no results.
In short: I'm trying to replicate behavior of the old complete-nvim plugin that I got used to. In short, when the completion window pops up, I want to be able to press <S-Tab> to skip the menu and input a tab instead. If I have selected something in the pop-up prior, then <S-Tab> should just move backwards.
Right now I have something like
local cmp = require'cmp'
cmp.setup {
    ...
    mapping = {
        ...
        ['<S-Tab>'] = cmp.mapping(function()
            if cmp.visible() and cmp.get_active_entry() ~= nil then
                cmp.select_prev_item()
            else
                -- What should I put here to just print out a tab character? cmp.mapping accepts a function, so I can't just put '<Tab>' here
            end
        end)
        ...
    }
    ...
}

It would be even better if pressing <S-Tab> after already selecting an entry with <Tab> never deselected the first one and just rolled over to the last one instead, so the only option to input a <Tab> character would be to press <S-Tab> first when the pop up shows up and not when going backwards through the list...
Thanks!

see :help i_CTRL-T and :help i_CTRL-D
CTRL-T		Insert one shiftwidth of indent at the start of the current
		line.  The indent is always rounded to a 'shiftwidth'.
						*i_CTRL-D*
CTRL-D		Delete one shiftwidth of indent at the start of the current
		line.  The indent is always rounded to a 'shiftwidth'.

That's not the same as I want though. Is what I want to replicate not possible with nvim-cmp?

      ['<S-Tab>'] = function(fallback)
        if cmp.visible() then
          if cmp.get_active_entry() then
            cmp.select_prev_item()
          else
            cmp.close()
            vim.api.nvim_feedkeys(vim.api.nvim_replace_termcodes('<Tab>', true, true, true), 'nt', true)
          end
          return
        end
        fallback()
      end

I can't support all user's mapping expectations.

All good, I was just wondering if it's possible at all, not demanding you to implement it. Thanks for the help!

Good to hear. Enjoy~!


SDL2-image build error

Describe the bug
So when trying to install SDL2-image package there is an error saying that vcpkg cant build the package.
Tried git pull, .\vcpkg update, .\vcpkg reintegrate install nothing helps
Environment

OS: Windows 11
Compiler: MS Visual Studio 2019

To Reproduce
Steps to reproduce the behavior:

./vcpkg install sdl2-image:x64-windows

Please ensure you're using the latest port files with git pullandvcpkg update. Then check for known issues at:     https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+sdl2-image You can submit a new issue at:     https://github.com/microsoft/vcpkg/issues/new?title=[sdl2-image]+Build+error&body=Copy+issue+body+from+C%3A%2Fdev%2Fvcpkg%2Finstalled%2Fvcpkg%2Fissue_body.md
Expected behavior
It should build properly ? Not much changes in my computer since installing SDL2 like month ago
DebugLogSDL2Image.txt
Failure logs
-attached debug log is not full i think this is not neccesary, please correct me if wrong
Additional context
Maybe after updating sdl-image to the newest version you did not include SDL2main.lib name change in lowercase style ?
install-x64-windows-dbg-err.log

I'm not able to repro this in my side, my vcpkg version:
vcpkg package management program version 2023-04-07-bedcba5172f5e4b91caac660ab7afe92c27a9895

vcpkg package management program version 2023-04-07-bedcba5172f5e4b91caac660ab7afe92c27a9895
Mine is the same:
Try following repro on x64 architecture:
Install SDL2 core x86
Delete SDL2 core x86
Install SDL2 core x64
Install SDL2-image[core] x86 - then error

For anyone else having this issue, the cause for me was a missing target for SDL2main.lib inside of the manual-link folder that should in your vcpkg install path.

a missing target for SDL2main.lib inside of the manual-link folder

This doesn't seem related to the error message
ninja: error: FindFirstFileExA(c:/dev/vcpkg/installed/x64-windows/lib/manual-link/sdl2main.lib;c:/dev/vcpkg/installed/x64-windows/lib): The filename, directory name, or volume label syntax is incorrect.

a missing target for SDL2main.lib inside of the manual-link folder

This doesn't seem related to the error message
ninja: error: FindFirstFileExA(c:/dev/vcpkg/installed/x64-windows/lib/manual-link/sdl2main.lib;c:/dev/vcpkg/installed/x64-windows/lib): The filename, directory name, or volume label syntax is incorrect.

You sure, from the error that you just pasted it is trying to find a file called sdl2main.lib inside of x64-windows/lib/manual-link folder


Update .NET Fx/.NET Core server article

Fixes #16571
Fixes #14237
Fixes #17934

Thanks for the review, Tom.


Discover plugins at runtime for development

Plugins require a META-INF/services resource which is normally built by the presto-plugin Maven packaging. But this requires a command line build and thus makes development more difficult. This change generates the service descriptor files automatically just like the Maven packaging when the plugin is loaded from a POM, allowing things to mostly "just work" when run in the IDE.

This commit will fix #2445, I believe

These are actually separate issues. I will update the description.

@cawallin I don't think so, they are actually different issues.

looks good


use slice for NPM connection tags

What does this PR do?
Use a slice for connection tags instead of a map
Motivation
reduce memory usage because tags are already unique.
Additional Notes
before:
goos: linux
goarch: arm64
pkg: github.com/DataDog/datadog-agent/pkg/network/tracer BenchmarkAddProcessInfo
BenchmarkAddProcessInfo-4   	 4172557	       281.4 ns/op	     128 B/op	       2 allocs/op

after:
goos: linux
goarch: arm64
pkg: github.com/DataDog/datadog-agent/pkg/network/tracer BenchmarkAddProcessInfo
BenchmarkAddProcessInfo-4   	 5192451	       224.6 ns/op	      24 B/op	       1 allocs/op

Reduces live heap size during TCP load test by ~16MiB
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


MatKG-2 Data Release

Hi,
I read your recent research paper https://openreview.net/pdf?id=cR1iE6MQ1y and found it really interesting.
Could you please share the data for MatKG-2 as well?
Thanks a lot!

can we have the knowledge graph to be downloaded


Fix CI: Stack overflow with new pylint 3.0.0

Summary
Nightly CI has been failing lint across the board for a couple of nights eg. this job. pylint 3.0.0 was released a couple of days ago and linting the code ends up with a stack overflow (recursion error) that did not occur on earlier versions. For now this one line change to a test case circumvents the issue and gets CI running again without the need to pin pylint.
Details and comments
pylint -rn qiskit_machine_learning test tools
Exception on node <Attribute.shape l.77 at 0x7f1824a4aad0> in file '/home/runner/work/qiskit-machine-learning/qiskit-machine-learning/test/utils/loss_functions/test_loss_functions.py'
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/astroid/decorators.py", line 90, in inner
    yield next(generator)
  File "/opt/hostedtoolcache/Python/3.10.13/x64/lib/python3.10/site-packages/astroid/decorators.py", line 44, in wrapped
    if context.push(node):
RecursionError: maximum recursion depth exceeded

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1851

Relevant Lines:
1997

💛  - Coveralls


Improve clarity around PodFitsResource(issue#11453)

@bprashanth @davidopp
Related issue: https://github.com/GoogleCloudPlatform/kubernetes/issues/11453
Improve clarity around PodFitsResource by showing pods limits in kubectl describe node
There are two suggestions in the original PR:

Show what resource failed in the scheduler event
Show each pods limits in describe node output

"1. Show what resource failed in the scheduler event" is not implemented in this PR and the reason is given below, any suggestion is welcome:)
This PR implements "2. Show each pods limits in describe node output".  The output, for example, looks like:
Name:			<IP_ADDRESS>
Labels:			kubernetes.io/hostname=<IP_ADDRESS>
CreationTimestamp:	Thu, 23 Jul 2015 10:37:25 +0000
Conditions:
  Type		Status	LastHeartbeatTime			LastTransitionTime			Reason					Message
  Ready 	True 	Thu, 23 Jul 2015 10:59:37 +0000 	Thu, 23 Jul 2015 10:37:25 +0000 	kubelet is posting ready status
Addresses:	<IP_ADDRESS>
Capacity:
 cpu:		4
 memory:	8176512Ki
 pods:		40
Version:
 Kernel Version:		3.13.0-46-generic
 OS Image:			Ubuntu precise (12.04.4 LTS)
 Container Runtime Version:	docker://1.6.0
 Kubelet Version:		v1.0.0-557-g5bb5d4ee321aea-dirty
 Kube-Proxy Version:		v1.0.0-557-g5bb5d4ee321aea-dirty
ExternalID:			<IP_ADDRESS>
Pods:				(2 in total)
  Namespace			Name	CPU-limit(milliCPU)	Memory-limit(bytes)
  default			apache1	1000			8000000
  default			apache2	1000			8000000
TotoalResourceLimits:
  CPU-limit(milliCPU):	2000 (50% of total)
  Memory-limit(bytes):	16000000 (0% of total)
Events:
  FirstSeen				LastSeen			Count	From			SubobjectPath	Reason		Message
  Thu, 23 Jul 2015 10:37:25 +0000	Thu, 23 Jul 2015 10:37:25 +0000	1	{kubelet <IP_ADDRESS>}			starting	Starting kubelet.
  Thu, 23 Jul 2015 10:37:25 +0000	Thu, 23 Jul 2015 10:37:25 +0000	1	{kubelet <IP_ADDRESS>}			NodeReady	Node <IP_ADDRESS> status is now: NodeReady

Please comment if you have any suggestions.
Why not showing failed resource in this PR:
I have tried to implement "1. Show what resource failed in the scheduler event", however, it seems to me that showing failed resource would require much modification in the source code.

The PodFitsResources verifies both CPU and Memory together in func CheckPodsExceedingCapacity()(https://github.com/GoogleCloudPlatform/kubernetes/blob/master/plugin/pkg/scheduler/algorithm/predicates/predicates.go#L118).
The return value of PodFitsResources  is either (true, nil) or (false, nil) after calling func CheckPodsExceedingCapacity(), so it is quite difficult to pass any specific resource type.
Based on the facts above, I think specifying failed resource would result in many modifications (for instance, [1]add one more output for PodFitsResources  and handles it OR, [2]verify CPU and Memory separately in two functions) which is a huge cost. Maybe just showing enough details in kubectl describe node is enough for a user to find out which resource failed for the Pod.

These are just my personal opinions, and it is quite possible that there is a good solution which I failed to find. So please feel free to share you ideas in the comments.
Regards,
Haiyang DING

@dchen1107

Looks good, I just had one minor comment.

@davidopp
New PR sent, and thank you for you quickly review!
The output looks good in command window and gedit (if the output is saved with > and opened with gedit), which I think is good enough.
But it does not line up if I copy them here, strange... I am going to send you a email with the screen-print.

Hi,
This is the screen print of the kubectl describe node <nodename> output in my second patch at PR https://github.com/GoogleCloudPlatform/kubernetes/pull/11788.
Please have a look. I am wondering if you can receive the image… Or you can tell me your email address.
Best regards,
Haiyang DING
发件人: David Oppenheimer<EMAIL_ADDRESS>发送时间: 2015年7月24日 15:09
收件人: GoogleCloudPlatform/kubernetes
抄送: dinghaiyang
主题: Re: [kubernetes] Improve clarity around PodFitsResource(issue#11453) (#11788)
Looks good, I just had one minor comment.
—
Reply to this email directly or view it on GitHubhttps://github.com/GoogleCloudPlatform/kubernetes/pull/11788#issuecomment-124378021.

A couple of suggestions:

Regarding the current implementation:
How about:

Pods:               (2 in total)
  Namespace         Name     CPU            Memory
  default           apache1  25%            50%
  default           apache2  25%            50%
TotoalResourceLimits:
  CPU-limit(milliCPU):  2000 (50% of total)
  Memory-limit(bytes):  16000000 (0% of total)

Haven't investigated complexity of translating cpu shares to a % but it should be straightforward?

Regarding not showing failed resource:

The PodFitsResources verifies both CPU and Memory together in func CheckPodsExceedingCapacity()(https://github.com/GoogleCloudPlatform/kubernetes/blob/master/plugin/pkg/scheduler/algorithm/predicates/predicates.go#L118).

Have it return notFittingCpu[], notFittingMemory[] and take the union in the caller?

The return value of PodFitsResources is either (true, nil) or (false, nil) after calling func CheckPodsExceedingCapacity(), so it is quite difficult to pass any specific resource type.

So basically each predicate function returns an error and a bool, and whenever one of them fails we record it's name in a map. I suggest inserting a richer reason into this map. A cryptic function name is reminiscent of microsoft alert boxes with system error codes.
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/plugin/pkg/scheduler/generic_scheduler.go#L128
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/plugin/pkg/scheduler/generic_scheduler.go#L46

Based on the facts above, I think specifying failed resource would result in many modifications (for instance, [1]add one more output for PodFitsResources and handles it OR, [2]verify CPU and Memory separately in two functions) which is a huge cost. Maybe just showing enough details in kubectl describe node is enough for a user to find out which resource failed for the Pod.

I'm fine adding clarity to the event in a different pr.

Shippable failure is https://github.com/GoogleCloudPlatform/kubernetes/pull/11822, though if there's going to be a follow on pr we might as well wait before kicking it

@davidopp , @bprashanth
New patch submitted.
Two major changes w.r.t the previous patch:

output of describe node:  for each pod, show percentage instead of quantity
output of describe pod: if the pod fails scheduling due to PodFitsResources, the actual failed resource is given (one of PodExceedsCapacity(CPU), PodExceedsCapacity(Memory), PodExceedsCapacity(Number))

describe node
The output of describe node is changed according to @bprashanth 's suggestion, and it now looks like this:

Name:			<IP_ADDRESS>
Labels:			kubernetes.io/hostname=<IP_ADDRESS>
CreationTimestamp:	Sat, 25 Jul 2015 08:23:32 +0000
Conditions:
  Type		Status	LastHeartbeatTime			LastTransitionTime			Reason					Message
  Ready 	True 	Sat, 25 Jul 2015 08:34:03 +0000 	Sat, 25 Jul 2015 08:23:32 +0000 	kubelet is posting ready status
Addresses:	<IP_ADDRESS>
Capacity:
 cpu:		4
 memory:	8176512Ki
 pods:		40
Version:
 Kernel Version:		3.13.0-46-generic
 OS Image:			Ubuntu precise (12.04.4 LTS)
 Container Runtime Version:	docker://1.6.0
 Kubelet Version:		v1.0.0-827-g137395fb6d0e59-dirty
 Kube-Proxy Version:		v1.0.0-827-g137395fb6d0e59-dirty
ExternalID:			<IP_ADDRESS>
Pods:				(2 in total)
  Namespace			Name		CPU		Memory
  default			apache1		25%		5%
  default			apache2		25%		11%
TotoalResourceLimits:
  CPU-limit(milliCPU):		2000 (50% of total)
  Memory-limit(bytes):<PHONE_NUMBER> (17% of total)
Events:
  FirstSeen				LastSeen			Count	From			SubobjectPath	Reason		Message
  Sat, 25 Jul 2015 08:23:32 +0000	Sat, 25 Jul 2015 08:23:32 +0000	1	{kubelet <IP_ADDRESS>}			starting	Starting kubelet.
  Sat, 25 Jul 2015 08:23:32 +0000	Sat, 25 Jul 2015 08:23:32 +0000	1	{kubelet <IP_ADDRESS>}			NodeReady	Node <IP_ADDRESS> status is now: NodeReady

describe pod
Currently, there are 3 resource types that could fail in PodFitsResource (Pod number, memory and CPU). If a pod fails scheduling due to PodFitsResource , the actual failed resource is given in describe pod. The following is an example,

Name:				apache3
Namespace:			default
Image(s):			test/apache:1
Node:				/
Labels:				app=apache3
Status:				Pending
Reason:
Message:
IP:
Replication Controllers:	<none>
Containers:
  apache3:
    Image:	test/apache:1
    Limits:
      cpu:		3
      memory:		1G
    State:		Waiting
    Ready:		False
    Restart Count:	0
    Variables:
Events:
  FirstSeen				LastSeen			Count	From		SubobjectPath	Reason			Message
  Sat, 25 Jul 2015 08:26:13 +0000	Sat, 25 Jul 2015 08:33:25 +0000	28	{scheduler }			failedScheduling	Failed for reason PodExceedsCapacity(CPU) and possibly others

@davidopp @bprashanth
Hi, it just comes to me that there might be a bug in this patch. I am going to test now and hope to confirm sooner.

I don't mean to bikeshed, but I think it would be a good idea to show both quantity and percentage for pod resources.

@davidopp , @bprashanth
New patch sent. Regarding to showing quantity and/or percentage for each pod, I would go for showing both of them. In this patch, the failed resources can be correctly shown in describe pod (one of PodExceedsCapacity(CPU), PodExceedsCapacity(Memory), PodExceedsCapacity(Number)), and the describe node looks like:
Name:			<IP_ADDRESS>
Labels:			kubernetes.io/hostname=<IP_ADDRESS>
CreationTimestamp:	Mon, 27 Jul 2015 03:21:32 +0000
Conditions:
  Type		Status	LastHeartbeatTime			LastTransitionTime			Reason					Message
  Ready 	True 	Mon, 27 Jul 2015 03:22:22 +0000 	Mon, 27 Jul 2015 03:21:32 +0000 	kubelet is posting ready status
Addresses:	<IP_ADDRESS>
Capacity:
 cpu:		4
 memory:	8176512Ki
 pods:		40
Version:
 Kernel Version:		3.13.0-46-generic
 OS Image:			Ubuntu precise (12.04.4 LTS)
 Container Runtime Version:	docker://1.6.0
 Kubelet Version:		v1.0.0-831-g99e072cf1a0bed-dirty
 Kube-Proxy Version:		v1.0.0-831-g99e072cf1a0bed-dirty
ExternalID:			<IP_ADDRESS>
Pods:				(2 in total)
  Namespace			Name		CPU(milliCPU)			Memory(bytes)
  default			apache1		1000 (25% of total)		500000000 (5% of total)
  default			apache2		1000 (25% of total)<PHONE_NUMBER> (11% of total)
TotoalResourceLimits:
  CPU(milliCPU):		2000 (50% of total)
  Memory(bytes):<PHONE_NUMBER> (17% of total)
Events:
  FirstSeen				LastSeen			Count	From			SubobjectPath	Reason		Message
  Mon, 27 Jul 2015 03:21:32 +0000	Mon, 27 Jul 2015 03:21:32 +0000	1	{kubelet <IP_ADDRESS>}			starting	Starting kubelet.
  Mon, 27 Jul 2015 03:21:32 +0000	Mon, 27 Jul 2015 03:21:32 +0000	1	{kubelet <IP_ADDRESS>}			NodeReady	Node <IP_ADDRESS> status is now: NodeReady

I will send a new patch to fix the names sooner.

@davidopp @bprashanth
New patch is available, PTAL.

Is #12005 doing something really similar?

@erictune
Yes it does...and I think it is doing great.
However, #12005 only implements part of what this PR does. This PR does two more things:

in describe node: show resource limits of each pod running on the node
in describe pod: return explicit failed resource type if PodFitsResources fails

Anyway, after #12005 merges, I can rebase my PR. Would you give some comments on the above two points? So that I can improve it. Many thanks in advance!
Regards

Once #11713 goes in, the fit check will use container.Resources.Request rather than container.Resources.Limit. So this will have to change. Don't care which goes in first: this or #11713, but whichever goes in second should change this.
@AnanyaKumar

I like the new output, thanks for the PR!

@erictune
Hi, thanks for your comments, but I am a bit confused by "new output", output in this PR or that in #12005?
I am personally fine with either one but I need to decide whether I should delete or keep the output in this PR when rebasing.

@bgrant0607
Typo fixed in the new patch.

looks fine except for the one nit (about CPU message)

@davidopp
Hi, I have fixed the message in this new PR. PTAL.

LGTM

I will merge this shortly after a couple of other on-going merges are determined to be OK.


Report statusCode and uri accurately for POSTs

With a listener on the req.on('end') event, req.route and res.statusCode are undefined. Switching to res.on('finish') makes these values available.
https://github.com/yaorg/node-measured/issues/57

Is there anyway to update the example at https://yaorg.github.io/node-measured/packages/measured-signalfx-reporter/tutorial-SignalFx Express Full End to End Example.html to mention that this middleware must precede bodyParser middleware? This was another bug that was hard to discover for me :laughing:

Yeah it's in the tutorial folder at the root of the project.
On Sat, Feb 2, 2019 at 8:40 AM avolfson<EMAIL_ADDRESS>wrote:

Is there anyway to update the example at
https://yaorg.github.io/node-measured/packages/measured-signalfx-reporter/tutorial-SignalFx Express Full End to End Example.html
to mention that this middleware must precede bodyParser middleware?
This was another bug that was hard to discover for me 😆
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/yaorg/node-measured/pull/58#issuecomment-459979281,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArcLkmX4-qgV7S27zi3cCM74IcWxmTlks5vJb-QgaJpZM4aZZls
.

--
Justin Field
<EMAIL_ADDRESS>LinkedIn https://www.linkedin.com/in/fieldju | @fieldju

Pull Request Test Coverage Report for Build 353

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.512%

Totals

Change from base Build 347:
0.0%

Covered Lines:
749

Relevant Lines:
827

💛  - Coveralls


Vega support for DPR

As a Publisher I want to add sophisticated graphs (using Vega) to my data package so that others can see how cool the data is
Acceptance Criteria

[x] Instructional data package with examples and an excellent README (in a separate repo)
[ ] Have following as Data Packages with vega views

[x] http://vega.github.io/vega-editor/?mode=vega&spec=barley
[ ] http://vega.github.io/vega-editor/?mode=vega&spec=choropleth

Tasks

[ ] https://github.com/frictionlessdata/dpr-js/issues/142
[x]  Instructional data package: https://github.com/anuveyatsu/datapackage-vega-view

@rufuspollock  in the second example graph we have 2 resources for the view. At the moment we can render only with one resource per view. I think we should create a separate issue or even an epic to be able to render a view using several resources.
So I took another example http://vega.github.io/vega-editor/?mode=vega&spec=driving with a single resource.

@anuveyatsu please add the driving item to the acceptance criteria here and move the other multi-resource item to the new multiple resource support item as a nice example.

FIXED in https://github.com/frictionlessdata/dpr-api/commit/212ef5ee9d8a6345a817e9456cb595fad1fa9b9a
Last PR in https://github.com/frictionlessdata/dpr-js/pull/149


Update the screenshot on the frontpage

The screenshot currently displayed is out of sync with the actual game content.

It has been brought to my attention that the current location of the screenshot is sub obptimal. It may be better on the "How to play page"


Unexpected token 'U', "Unexpected"... is not valid JSON

Describe the bug
Receiving this error while trying to import project from SB to Bolt
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-me6y5b
Steps to reproduce
Click on the URL
Expected behavior
Should properly import the project.
Screen Recording / Screenshot

Platform

OS: MacOS
Browser: Brave
Version: [Version 1.71.114]

Additional context
No response

@IzaacSB Thanks for the report and apologies for the frustrating experience! The fix is explained in #1471


Add ZAP CLI binary to GitHub Build artifacts and fix ZAP CLI binary.

I would like the ZAP CLI binary executable available on GitHub artifacts.
This allows using the ZAP binary in a headless mode.
In addition, the current ZAP CLI binary does not work on Linux and Windows. It produces the following below error:
Error: Cannot find module '/snapshot/zap/node_modules/sqlite3/lib/binding/napi-v3-linux-x64/node_sqlite3.node'
Require stack:
- /snapshot/zap/node_modules/sqlite3/lib/sqlite3-binding.js
- /snapshot/zap/node_modules/sqlite3/lib/sqlite3.js
- /snapshot/zap/dist/src-electron/db/db-api.js
- /snapshot/zap/dist/src-electron/main-process/startup.js
- /snapshot/zap/dist/src-electron/main-process/main.js
1) If you want to compile the package/file into executable, please pay attention to compilation warnings and specify a literal in 'require' call. 2) If you don't want to compile the package/file into executable and want to 'require' it from filesystem (likely plugin), specify an absolute path in 'require' call using process.cwd() or process.execPath.
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
    at Function._resolveFilename (pkg/prelude/bootstrap.js:1819:46)
    at Function.Module._load (node:internal/modules/cjs/loader:778:27)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at Module.require (pkg/prelude/bootstrap.js:1719:31)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/snapshot/zap/node_modules/sqlite3/lib/sqlite3-binding.js:4:15)
    at Module._compile (pkg/prelude/bootstrap.js:1794:22)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)

@DennisJensen95
The zap-cli binaries are packaged within the .zip assets.
I'm closing the ticket for now. Feel free to re-open if you run into any issues.


angular 1.4 breaks ui router

When I update angular to 1.4, I have an error: Uncaught TypeError: b.addPollFn is not a function

Unfortunately, with the information you've given me I have no idea how to help you. Please post a plunkr that demonstrates the issue. Thanks.

I have a rather large UI-Router based app running on 1.4 no problem, have you tried making sure you are on latest version of UI-Router?

Not to re-open this but I am having an issue where the states aren't applying the ngAnimate classes. I am still looking into why and about to revert to angular 1.3.15 / 16 to see if that fixes it.

http://plnkr.co/edit/WV1JksXCBRl7wG6p00ZS?p=preview
Plunkr creating the issue from a guide I found. I updated it to the latest to see what happens and it broke.


[Flaky Test] tests/python/contrib/test_hexagon/test_models.py::test_mobilenet

These tests were found to be flaky (intermittently failing on main or failed in a PR with unrelated changes). See the docs for details.
Tests(s)

tests/python/contrib/test_hexagon/test_models.py::test_mobilenet

Jenkins Links

https://ci.tlcpack.ai/job/tvm/job/PR-13054/5/display/redirect

https://github.com/apache/tvm/pull/13067/files should close this


Ignition bridge broken from PR #158

Describe the bug
PR #158 changed the QoS of publishers to reliable, making it incompatible with ros_ign_bridge. See https://github.com/ignitionrobotics/ros_ign/blob/ros2/ros_ign_bridge/src/factory.hpp#:~:text=create_ros_subscriber for their ros subscriber implementation.
As a result, docking is no longer working in Ignition:
[pose_republisher_node-25] [WARN] [1647446984.467150926] [pose_republisher_node]: New subscription discovered on topic '/sim_ground_truth_pose', requesting incompatible QoS. No messages will be sent to it. Last incompatible policy: RELIABILITY_QOS_POLICY
[pose_republisher_node-25] [WARN] [1647446984.467293718] [pose_republisher_node]: New subscription discovered on topic '/sim_ground_truth_dock_pose', requesting incompatible QoS. No messages will be sent to it. Last incompatible policy: RELIABILITY_QOS_POLICY
[motion_control-20] [WARN] [1647446984.472496722] [motion_control]: New publisher discovered on topic '/sim_ground_truth_pose', offering incompatible QoS. No messages will be sent to it. Last incompatible policy: RELIABILITY_QOS_POLICY
[motion_control-20] [WARN] [1647446984.472579492] [motion_control]: New publisher discovered on topic '/sim_ground_truth_dock_pose', offering incompatible QoS. No messages will be sent to it. Last incompatible policy: RELIABILITY_QOS_POLICY
[sensors_node-26] [WARN] [1647446984.472978202] [sensors_node]: New subscription discovered on topic '/dock', requesting incompatible QoS. No messages will be sent to it. Last incompatible policy: RELIABILITY_QOS_POLICY
[robot_state-23] [WARN] [1647446984.479251147] [robot_state]: New publisher discovered on topic '/dock', offering incompatible QoS. No messages will be sent to it. Last incompatible policy: RELIABILITY_QOS_POLICY
[motion_control-20] [WARN] [1647446984.479496296] [motion_control]: New publisher discovered on topic '/dock', offering incompatible QoS. No messages will be sent to it. Last incompatible policy: RELIABILITY_QOS_POLICY

To Reproduce
ros2 launch irobot_create_ignition_bringup create3_ignition.launch.py

You're right it is the subscriptions and not the publishers.

Sure, go ahead, thanks!


No virtual interface

I'm trying to use opera-proxy with this program, which does support using a VPN:
https://github.com/lakinduakash/linux-wifi-hotspot
in the README file, it says "Select the virtual interface created by the VPN. In this case it is tun0"
but opera-proxy does not create a virtual interface
related to the problem:
https://github.com/lakinduakash/linux-wifi-hotspot/issues/307

Hello,
It doesn't create TUN adapter because it's not a VPN, it's a proxy. However, you can still redirect only TCP connections (but not UDP sessions) into opera-proxy with any of following transparent proxying software:

transocks
redsocks
glider
... and many more

It should go like this: you add REDIRECT rule into iptables to send TCP connections from clients to port of transparent proxy (transocks/redsocks/glider) and this transparent proxy has to be set to use opera-proxy as an upstream proxy.
Not sure, how to set up your linux-wifi-hotspot instance, but probably few additional daemons and iptables rule won't hurt this installation. But my general recommendation would be to use some real VPN connection, because opera-proxy can't help you with forwarding DNS and other UDP traffic.