feat(CodeActAgent): Support Agent-User Interaction during Task Execution and the Full Integration of CodeActAgent

Dependency:

https://github.com/OpenDevin/OpenDevin/pull/1255
https://github.com/OpenDevin/OpenDevin/pull/1305
https://github.com/OpenDevin/OpenDevin/pull/1333
https://github.com/OpenDevin/OpenDevin/pull/1334

For general OpenDevin components:

Add IPythonRunCellAction that allows the agent to use Jupyter plugin introduced in #1255 for task solving.
AgentTalkAction that allows the agent to communicate with the user (either to communicate results OR ask for clarification)
Agent-User Interaction: Now allow agent's wait for user's response before proceed to task completion.
Added __str__ method to some of the actions to get the printed in a readable way
Add thought attribute to most Action (if applicable)

For CodeActAgent:

Allow CodeActAgent to do two things for environmental interactions: (1) use ipython AND (2) use bash
Tweak the prompts of CodeActAgent to ensure its generality.
The ability to use SWE-Agent's tool for task solving (still need more extensive testing).

For integration tests:

Added integration tests for CodeActAgent.
Support STDIN mock for "user responses" in integration test

The best thing about CodeActAgent is that we have existing data and open models (e.g., CodeActAgent, CodeQwen) trained on that data, which could means we can have a usable local model that can run on laptop.
Below demo is completed with CodeQwen, here's my config.toml:
LLM_MODEL="ollama/codeqwen"
LLM_API_KEY="ollama"
LLM_BASE_URL="http://localhost:11434"
LLM_EMBEDDING_MODEL="local"
WORKSPACE_BASE="./workspace"

UPDATE
CodeActAgent now can write a calculator and start a flask server in 5 steps! It is also integrated with the ability to discuss with users for clarifing questions, which, IMO is more natural for AI-user interaction.
Here's my config.toml to reproduce the following example.
LLM_MODEL="gpt-4-turbo"
LLM_API_KEY="sk-TODO"
LLM_EMBEDDING_MODEL=""
WORKSPACE_BASE="./workspace"
SANDBOX_CONTAINER_IMAGE="ghcr.io/opendevin/sandbox:xw-swe-agent-tool-plugins"

Command to run:
python ./opendevin/main.py -i 10 -c CodeActAgent --task "can you help me start a flask server that build a calculator at 8000?"

Finally, I fixed some issues with the front end! This PR should be ready for review!
Here's a video demo for the front end with CodeActAgent (no speed-up).
https://github.com/OpenDevin/OpenDevin/assets/38853559/5563f20a-c1ac-483b-a48e-56968090cb13
There's a bunch of issues for frontend that need to get fixed:

Sometimes, the model would ask for the user's approval before continuing to execute the code; we should show a button "[Continue]" that sends over the text 'continue' to the agent when "Agent is awaiting user input." It may save some efforts for the user to type the word "[Continue]."

IPython code execution is recognized as undefined by the front end. We will need to make Observation from IPythonAction a special class and handle it differently on the front end. I will work on this in a separate PR probably for clarity. Also, the front-end needs to support visualization of IPythonAction, potentially by adding an additional tab between CodeEditor and Browser.

We need to let the user know things are running! There's a bunch of logs/info on the backend; maybe we can consider displaying some of them to the user so they know the agent is working. Also, some animations focus on showing which panel is changing, which could be helpful for the user. We may have to get token streaming work at some point - otherwise, it takes a ton of time for the user to wait for a calculator_app.py to be written.

We need to get the browser working and monitor the chat between the agent & user and the command line. When it detects URL patterns like http://<IP_ADDRESS>:8000, a window should pop up a window asking the user whether to open that URL in the built-in browser.

Around 01:47, we can see the terminal cannot be scrolled down to the bottom. There are probably some CSS issues.

It turns out I broke the integration tests by adding a thought attribute to most actions. Now, I finally fixed it!
I also added integration tests for CodeActAgent and supported the STDIN mockup for "user responses" during integration tests. (cc @li-boxuan)
CodeActAgent will fail with the exec integration task (well, and it is not designed to work with the exec box since it is not stateful). @rbren and I were thinking about deprecating ExecBox soon, so I removed it from the integration test for now.
This should be ready for review.

Btw did you encounter any annoying problems when trying to fix integration tests? Anything you think could be improved?

Another thought 💭 :
I presume this agent can do many fancier things than other simpler ones (PlannerAgent? MonologueAgent?). You could probably add a test to demonstrate its larger power. That will also increase the code coverage. Right now, CodeActAgent-specific commands are not tested in the only simple naive integration test we have.

@li-boxuan, thanks for the review!!

Btw did you encounter any annoying problems when trying to fix integration tests? Anything you think could be improved?

Nope! I was confused at first until I found the README.md - it was very clear & easy for me to get the integration test fixed.

I presume this agent can do many fancier things than other simpler ones (PlannerAgent? MonologueAgent?). You could probably add a test to demonstrate its larger power. That will also increase the code coverage. Right now, CodeActAgent-specific commands are not tested in the only simple naive integration test we have. This, of course, doesn't need to be addressed in current PR.

Good idea! I think I will do this in a follow-up PR to ask for more challenging tasks that require multiple interactions with BOTH bash & python.
There will be a lot of TODOs following this:

Get frontend working well with the new IPython feature https://github.com/OpenDevin/OpenDevin/pull/1363
Add more integration tests for more complex behaviors (multi-turn interactions with both bash & python)
Get SWE-Bench ready for testing (prompts are also subject to change)

Finally, with some modifications (https://github.com/OpenDevin/OpenDevin/pull/1426), I am finally able to get CodeActAgent working in the DooD sandbox with RUN_AS_DEVIN and network=host without issues :)
Steps to reproduce:

switch to this PR
build the app container: ./containers/build.sh sandbox (you may need to comment a few lines in build.sh to make it work on your local machine
run the following, switch to CodeActAgent on the web interface, and it should work:

docker run \
    -e LLM_API_KEY \
    -e WORKSPACE_MOUNT_PATH=$WORKSPACE_BASE \
    -v $WORKSPACE_BASE:/opt/workspace_base \
    -e SANDBOX_USER_ID=$(id -u) \
    -v /var/run/docker.sock:/var/run/docker.sock \
    -p 3000:3000 \
    --add-host host.docker.internal=host-gateway \
    ghcr.io/opendevin/opendevin:latest

Codecov Report
Attention: Patch coverage is 81.13208% with 60 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (main@8954855). Click here to learn what that means.

Files
Patch %
Lines

opendevin/action/bash.py
62.16%
14 Missing :warning:

opendevin/controller/agent_controller.py
72.09%
12 Missing :warning:

opendevin/server/agent/agent.py
0.00%
8 Missing :warning:

agenthub/codeact_agent/codeact_agent.py
76.92%
6 Missing :warning:

opendevin/sandbox/docker/ssh_box.py
85.29%
5 Missing :warning:

opendevin/server/session/msg_stack.py
0.00%
3 Missing :warning:

opendevin/action/agent.py
85.71%
2 Missing :warning:

opendevin/observation/run.py
81.81%
2 Missing :warning:

opendevin/server/auth/auth.py
0.00%
2 Missing :warning:

agenthub/micro/agent.py
80.00%
1 Missing :warning:

... and 5 more

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1290   +/-   ##
=======================================
  Coverage        ?   57.49%
=======================================
  Files           ?       83
  Lines           ?     3571
  Branches        ?        0
=======================================
  Hits            ?     2053
  Misses          ?     1518
  Partials        ?        0

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

@xingyaoww I implemented the user_message action here: https://github.com/OpenDevin/OpenDevin/commits/rb/codeact/
But I can't find a way to contribute it to your PR...

@xingyaoww once my PR is merged into yours, this LGTM!

@rbren merged! this should be ready to go!

🎉 glad to have this in!

@rbren Thanks!!!
Next steps:

Frontend!

https://github.com/OpenDevin/OpenDevin/pull/1363/
Switch to Jupyter Tab when a python execution request comes in (once https://github.com/OpenDevin/OpenDevin/pull/1363/ is merged)
https://github.com/OpenDevin/OpenDevin/issues/1487

Evaluation: get SWE-Bench score for CodeActAgent!


Bump dependencies to reduce vulnerabilities

yarn audit reports (critical) vulnerabilities so it returns with a non-zero exit code. I'm using Node 10 so I upgraded lint-staged which requires Node 8+ only. Nevermind, I can commit on Node 8.

sorry this is out of date, I'll try to update dependencies myself


param fix for openmpi

fixed "-ppn" to "-npernode" for openmpi.

Hi Candice, is there a difference between the two? Or does ppn not work at all.

ppn does not work for openmpi

(hm_test) @.*** jarvis-util]$ mpiexec -n 2 -ppn 2 echo hi
mpiexec: Error: unknown option "-ppn"
Type 'mpiexec --help' for usage.
(hm_test) @.*** jarvis-util]$ mpiexec -n 2 -npernode 2 echo hi
hi
hi

On Oct 25, 2023, at 11:06 AM, lukemartinlogan @.***> wrote:
Hi Candice, is there a difference between the two? Or does ppn not work at all.
—
Reply to this email directly, view it on GitHub https://github.com/grc-iit/jarvis-util/pull/44#issuecomment-1779606898, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIVHO3ZA5TPUPSCXLUJYUZ3YBE2HXAVCNFSM6AAAAAA6PUF26SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZZGYYDMOBZHA.
You are receiving this because you authored the thread.


with react-cropper , multiple selection

sorry, this is not react-cropper but I heard it's implement from this repo.
is someone know how to implement  multiple selection in react-cropper .
react-cropper

Just use Cropper.js 2.0 directly.

Just use Cropper.js 2.0 directly.

ok, it's has only way T o T. I will try to make new implement sir.


Add a basic workflow for running tests

This isn't particularly elegant and mostly copies and pastes the
workflow we use for deploying (for the most part). I don't want to
consider this to be good enough for #37. We need to do more work
but with the recent set of changes, I think this will be valuable
so we don't have to merge a PR just to see if it fixes the tests.

Looks like an accidental commit of service.jar?

oops. i am going to overwrite history of the branch to fix that


SVGR

Any chance of getting loadable-components work with icons imported via SVGR?

Aaaand yes sir it works.
import React from 'react';

import loadable from '@loadable/component';

const IconDownload = loadable.lib(() => import('./icon-download.svg'));

export const App: React.FC = () => (
  <main>
    Hellooo
    <IconDownload>{({ ReactComponent }) => <ReactComponent />}</IconDownload>
  </main>
);

Why not keep it svg in this case?

Why not keep it svg in this case?

Care to elaborate with a code example?
Also my final goal is to pass the path or icon name to a factory. So far no luck with that, albeit it works with “normal” components that aren’t SVG files loaded by svgr.

I mean in majority of cases svg could be kept as a external "raw" .svg file, and displayed via img/background

I mean in majority of cases svg could be kept as a external "raw" .svg file, and displayed via img/background

True, but I’m building a showcase for icons that are used via svgr and I want to stay as close to that situation as possible if I can.

This is important for us, too


Added check in virt-handler to only update VMI interface if interface is not masquerade

Signed-off-by: Vatsal Parekh<EMAIL_ADDRESS>
What this PR does / why we need it:
Added check in virt-handler to only update VMI interface if interface is not masquerade
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

Added check in virt-handler to only update VMI interface if interface is not masquerade

/retest

/retest

/retest

/retest

/cherry-pick release-0.26

/retest

@vatsalparekh please use the PR description to explain what is the current state, why is it wrong and how do you fix it / how it works with this PR.

@phoracek does it look good now?

Beautiful!

@vatsalparekh what's the state of this?

@stu-gott as discussed in the community meeting, this is blocked on the various opinions about this PR, to have this change or not, and it's change in the API

/retest

/cc: @davidvossel

@vatsalparekh could you please review https://github.com/nmstate/kubernetes-nmstate/pull/503 and make sure this PR aligns to it?

/retest

/retest

/retest

/retest


Add support for new RustRover IDE

Title ^ https://www.jetbrains.com/rust/

+1
consistently same stack
if it is helpful i can post

+1

+1

Hey, I made a fork of this plugin with support for rust rover : https://github.com/Azn9/Jetbrains-Discord-Integration

@Azn9
any chance of you making a PR and it potentially being merged?

@Azn9
any chance of you making a PR and it potentially being merged?

It has already been merged on my fork! This repo is inactive and making a PR on it will never get any updates, that's why I created my fork (there are also a lot of bug fixed on it)

+1

@Anderson0xFF you can check my fork (the link is above) if you need the rust rover support! ;)

+1


pBAM: SingleCellLiverLandscape (10x v2)

Run Optimus pipeline on SingleCellLiverLandscape and analyze differences between pBAM vs BAM results:
Project Title: Dissecting the human liver cellular landscape by single cell RNA-seq reveals novel intrahepatic monocyte/ macrophage populations
Project URL: https://data.humancellatlas.org/explore/projects/4d6f6c96-2a83-43d8-8fe1-0f53bffd4674
Donors: P2TLH_liver and P3TLH_liver

[x] Place subset of BAM files and expression matrix into gcp bucket
[x] Check pBAM files to make sure alignments are as expected
[ ] Process dataset through pipelines w/ pFASTQ as input
[ ] Compare pBAM outputs with HCA output, and each with findings from the published results

BAM FILES AND EXPRESSION MATRIX:

Obtained relevant bam files, fastqs, and subsetted expression matrix and placed them in gs://pbam_singlecell/input_data/10x_v2_liver/

CHECK pBAM FILES:
Documentation: https://docs.google.com/presentation/d/1Ih9fEqp9P5CT4Y3UlyqswZx4h_mBS-mP0Q4FgOh1J6I/edit?usp=sharing
Task: Spot checked alignments with insertion (I), soft clipping (S), deletion (D), MD:Z flag (SNPs), or complex CIGAR strings in pBAM file. Used samtools and IGV viewer to visualize alignment.
Result: Gamze produced a sample pbam file in gs://pbam_singlecell/pBAMs/10x_v2/v1. The start location of an alignment needs to be changed for subset of alignments when converted to pBAM. This was corrected and checked in gs://pbam_singlecell/pBAMs/10x_v2/v2


Basepath issue with deploy on GitHub pages

I've tried to set up the export of a Sapper app and deploy on GitHub pages, with the following file:
name: Build Sapper and Deploy to GitHub Pages
on: [push]
jobs:
  build_deploy:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@master
      - name: Build Sapper
        uses: truewebartisans/actions-sapper@master
        with:
          build_mode: "export"
          args: "--legacy"
      - name: Deploy to GitHub Pages
        uses: peaceiris/actions-gh-pages@v3
        with:
          publish_dir: __sapper__/export
          publish_branch: gh-pages
          github_token: ${{ secrets.GITHUB_TOKEN }}

When I push to my GH repo, the action is completed successfully but there is a problem because Sapper pages always add a <base="/"> tag by default, so the static assets and page links all use my_name.github.io  instead of my_name.github.io/my_app as base path.
You don't have this problem in you example on https://sapper-action.gh.1wa.co because you serve from the root.
I've tried different things to make it work (--basepath my_app in the export args, adding a base folder name in Sapper's server.js film) but it doesn't solve anything, only moves the problem one folder deeper.
Do you know how it could work?

@nltesown Hi! thanks for report this interesting thing.
I think, we can find solution of this trouble on Sapper docs (or issues), because it's not GH Action trouble, it's about Sapper architecture. Maybe latest version of Sapper can solve this 🤷

@nltesown Hi! thanks for report this interesting thing.
I think, we can find solution of this trouble on Sapper docs (or issues), because it's not GH Action trouble, it's about Sapper architecture. Maybe latest version of Sapper can solve this 🤷

Thanks for your answer. I'm not sure it can currently be solved, but at least there's a way to make it work: make a "user" or "organization" GitHub pages site (where the root URL will be user.github.io, without a repo-named subfolder).

Thanks for your answer. I'm not sure it can currently be solved, but at least there's a way to make it work: make a "user" or "organization" GitHub pages site (where the root URL will be user.github.io, without a repo-named subfolder).


Issue with Env() on Mac OS and node 8

On Mac OS 10.13.6, with node 8.11.3, I have a test such as
requests:
login:
url: Env(URL)
method: POST
....
....
I set URL in my env to http://foo.com/bar, but when I run the test it errors saying it can't talk to <IP_ADDRESS> for the url.  Exact error:
strest ./post.yml
[ Strest ] Found 1 test file(s)
[ Strest ] Schema validation: 1 of 1 file(s) passed
✖ Testing login failed (0.041s)
Error: connect ECONNREFUSED <IP_ADDRESS>:80
I took the exact same version of strest, same strest test file, same env variable set, on a linux box with node 6.14.0 and the test works correctly (uses http://foo.com/bar).  It seems as though the Env call on node 8 doesn't recognize my environment variable on node8+MacOS but does on node6+Linux.  I did some googling but couldn't find if there was some changes or issues with getting the  env in node 8 vs. 6.  On node 8 I made a one-liner node app to print process.env and I see my environment (including URL) so I don't think there is.  Maybe an issue with the templating/yaml parsing between versions where the Env() call exists?

Have you defined your Env variable in the .bash_profile file or in the terminal? Because I ran into the same issue when I defined it in the terminal on MacOS and node6/8. But when I made a new entry into the ~/.bash_profile file and restarted my terminal it has worked correctly

Is the behavior different if export FOO=bar.com is used?

Yes unfortunately it will be undefined

I have tried a few things, and still have the same problem:
export URL=foo && strest  # what I originally tried, doesn't work
URL=foo strest   # var inline, doesn't work
put URL=foo in ~/.bash_profile and then ran strest  # doesn't work

can you share your post.yml file ?

This is a test case that is similar to mine:
foo.strest.yml:
_version: 1                            # only version at the moment
requests:
login:
url: Env(URL)
method: GET
log: true_
export URL=https://www.google.com
strest ./foo.strest.yml
Yields the following:
[ Strest ] Found 1 test file(s)
[ Strest ] Schema validation: 1 of 1 file(s) passed
✖ Testing login failed (0.035s)
Error: connect ECONNREFUSED <IP_ADDRESS>:80
[ Strest ] Failed before finishing all requests

Have you tried to define the ENV variables in your ~/.bash_profile file and then restart your terminal?

I wouldn't think that would matter, but I did try that as well and it still fails.

Do we know if this is a Mac OS specific issue, or a node 8 issue?  I'll try to test on linux with node 8 and see if I can reproduce.

I haven't had time to look into this any further, and not sure when I'll have time on MacOS again.  So following the advice above and closing.


Add credit info inside the main App

[x] Leave an about window that displays credit and the software license.
[x] Add a URL/Icon that directs to the GitHub project.

Partly Implemented in b8769d5007b6635c164c97dd85933d4b3c0dc2f3, may need some changes in the repo as well.

Fully implemented in 1d82cf011f374ad3236912ff05e3207c72895c89


Add schema for all supported endpoints

k8s endpoint json block now is validated with a specific schema. Other endpoints like osp, kvm and remotehost are validated using a placeholder schema with generic strutucture to be polished with more strict rules.
Added tests to confirm schema validation works for all endpoints. CI run file has been created to be used in the crucible-ci tests.

The one problem I see up front is that these schemas need to be part of the rickshaw project.  If the endpoint changes (in a PR) then the schema needs to be able to be changed in the same PR.

The one problem I see up front is that these schemas need to be part of the rickshaw project. If the endpoint changes (in a PR) then the schema needs to be able to be changed in the same PR.

It's complicated. The logic that validates the json is in toolbox. If you have any changes in that code, you'll break the schemas and vice-versa.


The structure of BasicRFB is not same with Fig.4.(a) in your paper

Hello, thanks for your code releasing.
There is a branch of 1×1 conv tailed by 3×3 conv rate=1 in Fig.4.(a) in your paper(arXiv:1107.07767v3),which is not shown in class BasicRFB of models/RFB_Net_vgg.py.
There are other two branch(self.branch0 and self.branch1 in class BasicRFB) and a shortcut connection in your code. So which choice is more helpful in performance?

@arlensung Thank you for your kindly commenting. It was a mistake that I had overlooked for a long time!! The current results are obtained by this two branch structure, and I will update the results with the same structure as in the paper soon.

@arlensung  Hi, I have fixed the error above, and now the correct RFBNet300 achieved 80.7% mAP in VOC2007. You can check the latest commit for details of both RFB structure and new model weight. The COCO model is still under training, but a tiny improvement could also be expected.


⚠ ClamAV has degraded performance

In d8e89a9, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 456 ms

Resolved: ClamAV performance has improved in 2c4f09b.


Honeybadger API Key

The Readme states that I should add a "HONEYBADGER_ENV" variable. However, the Honeybadger documentation states that I should a different variable "HONEYBADGER_API_KEY".
https://docs.honeybadger.io/ruby/integration-guides/rails-exception-tracking.html

Thanks for the issue. 😸
The README mentions that the --heroku true flag:

Configures staging with HONEYBADGER_ENV environment variable set to staging

HONEYBADGER_ENV allows you to configure the environment that gets passed to honeybadger. It is separate from the HONEYBADGER_API_KEY, which you will need to add manually when setting up honeybadger.


Type Error Cannot read properties of undefined (reading 'meta')

the undefined issue in this library
node : v16.19.0
Version:    "@ssut/nestjs-sqs": "^2.0.0",
OS: Windows 11
aws-sdk version: "2.1388.0"
    "errorMessage": "Cannot read properties of undefined (reading 'meta')",
    "errorType": "TypeError",
    "stackTrace": [
        "TypeError: Cannot read properties of undefined (reading 'meta')",
        "    at node_modules\\@ssut\\nestjs-sqs\\dist\\sqs.service.js:64:49",
        "    at Array.forEach (<anonymous>)",
        "    at SqsService.<anonymous> (\\node_modules\\@ssut\\nestjs-sqs\\dist\\sqs.service.js:55:83)",
        "    at Generator.next (<anonymous>)",
        "    at fulfilled (\\node_modules\\@ssut\\nestjs-sqs\\dist\\sqs.service.js:17:58)",
        "    at processTicksAndRejections (node:internal/process/task_queues:96:5)"
    ]
}

I'm also facing the same issue it's still working in my project with previous files but after removing the node_modules when I install it gives me this error

I had the same issue here.
My issue was that my consumer (class with @SqsMessageHandler) was not defined/registered in the module 😢
I hope it helps others.

I had the same issue here. My issue was that my consumer (class with @SqsMessageHandler) was not defined/registered in the module 😢 I hope it helps others.

Thanks man. It works. maybe the dude forgot to catch exception.

Had this issue today too. After some debugging, found the root cause of the issue is if you pass a queue configuration to the module when there's no handler configured, it will error out, because of a missing return in this guard condition:
https://github.com/ssut/nestjs-sqs/blob/master/lib/sqs.service.ts#L39

it is because your @SqsMessageHandler('QUEUE_NAME', true) is not registered

also when i want to pass an env var to the SqsMessageHandler method decorator,
when i pass the hardcoded queue name, everything is fine, but process.env.QUEUE_NAME won't work, don't sure why though

@toufikat  it is because how SqsMessageHandler method decorator work, please refer to my solution if you want dynamic queue name in decorator
https://github.com/ssut/nestjs-sqs/issues/8#issuecomment-1842631842

make sure that you user dotenv.config() in the file where you are using process.env.QUEUE_NAME
had the same issue, on the local machine it worked without dotenv.config(), but on the server it doesn't


fix(dummy_perception_publisher): fix build warning in Humble

Description
以下を参考に修正を実施

https://github.com/autowarefoundation/autoware.universe/pull/852/files#diff-f7025aa1b2846894c7a4a3495b2ff2abefc334d78c350ef4de488373ece32f24R35
https://github.com/autowarefoundation/autoware.universe/pull/852/files#diff-9e63f577908171e00f9199eac93ea6cddd65ead39b0f83a93123a438a3267ba2R24
https://github.com/autowarefoundation/autoware.universe/pull/852/files#diff-3290f2cb97b611502587a1421628ad062a0c09a16ab3407fd2e9d572ffae1957R73

Tests performed

dummy_perception_publisherがビルドwarningが出ないこと
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Codecov Report

:exclamation: No coverage uploaded for pull request base (beta/v0.3.16@8553fd3). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@              Coverage Diff               @@
##             beta/v0.3.16    #472   +/-   ##
==============================================
  Coverage                ?   5.75%
==============================================
  Files                   ?      16
  Lines                   ?     695
  Branches                ?      32
==============================================
  Hits                    ?      40
  Misses                  ?     634
  Partials                ?      21

Flag
Coverage Δ

differential
5.75% <0.00%> (?)

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


Improved group action submit button selector

Hi,
I was getting group actions to work in my custom BS4 template, however I wanted the button to have a icon, so I used <button> which then did not work with button enabling.
This modification should support both buttons and inputs of type submit while also containing the disabling/enabling to just group actions.

LGTM

Thanks 👍

@paveljanda Sure, I should first fix the package.json, right?


Permission denied installing yapf

Environment data
VS Code version:  1.19
Python Extension version:  0.9
Python Version:   2.7.10
OS and version:   Mac OS 10.13.2
Attempt to says 'yes, install Yapf' on 'Format Document' yields Access Denied (looks like install has to elevate)
$ "/usr/bin/python" -m pip install -U yapf
Collecting yapf
  Downloading yapf-0.20.0-py2.py3-none-any.whl (158kB)
    100% |████████████████████████████████| 163kB 2.9MB/s
Installing collected packages: yapf
Exception:
Traceback (most recent call last):
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/basecommand.py", line 215, in main
    status = self.run(options, args)
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/commands/install.py", line 342, in run
    prefix=options.prefix_path,
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/req/req_set.py", line 784, in install
    **kwargs
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/req/req_install.py", line 851, in install
    self.move_wheel_files(self.source_dir, root=root, prefix=prefix)
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/req/req_install.py", line 1064, in move_wheel_files
    isolated=self.isolated,
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/wheel.py", line 345, in move_wheel_files
    clobber(source, lib_dir, True)
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/wheel.py", line 316, in clobber
    ensure_dir(destdir)
  File "/Library/Python/2.7/site-packages/pip-9.0.1-py2.7.egg/pip/utils/__init__.py", line 83, in ensure_dir
    os.makedirs(path)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/os.py", line 157, in makedirs
    mkdir(name, mode)
**OSError: [Errno 13] Permission denied: '/Library/Python/2.7/site-packages/yapf-0.20.0.dist-info'
PapaMac:~ Misha$**

Install via sudo succeeds. However, it says
The directory '/Users/Misha/Library/Caches/pip' or its parent directory
is not owned by the current user and caching wheels has been disabled.
Check the permissions and owner of that directory. If executing pip with
sudo, you may want sudo's -H flag.

Closing in favor of #386


Couldn't open file: /default/yolov2-tiny.cfg

I run the package by "rosrun darknet_ros darknet_ros"，and it say Couldn't open file: /default/yolov2-tiny.cfg，the picture leftside.
I change the path to my computer real position，the picture rightside.
Anybody have ideal?Before the change of the path，I still got Couldn't open file: /default/yolov2-tiny.cfg...

@kytu52324 Hi, have you fixed this problem? I also encountered the same problem. Can you give me some tips? I wish I could communicate with you. My e-mail is<EMAIL_ADDRESS>Thank you very much!

I have fixed this problem.Thank you!

How did you solve this problem???

how?

Hi @QiuYilin, run the package using launch file. You can find the launch file here https://github.com/leggedrobotics/darknet_ros/blob/master/darknet_ros/launch/yolo_v3.launch

@pushkalkatara I run the launch file, but it run into another problem
First section must be [net] or [network]: Resouce temporarily unavailable
........
....... shutting down processing monitor complete


[bug 1210555] Update the etherpad url for the migration.

@johngian r?

Code looks OK.
r+

Tested migration locally. Example URL after migration (pad is empty though):
https://public.etherpad-mozilla.org/p/flisol-chapeco


completionHandler stopped working

This example code from readme no longer works:
    Loaf("Loaf message", sender: self, completionHandler: {
        print("Dismissal complete")
    }).show()

Error messages in xCode:

Argument 'completionHandler' must precede argument 'sender'
Incorrect argument label in call (have ':completionHandler:sender:', expected ':state:sender:')

Hey, I should've posted an update. I've moved and renamed the completion handler in the 0.3.0.
A completion handler can now be specified in the show() function.
Loaf(example.rawValue, sender: self).show {
       print("Dismissed!")
}

I'll update the README accordingly 🙂


New export header doesn't support visual studio

Your new export header cast_export.h (https://github.com/clariusdev/cast/blob/master/src/include/cast/cast_export.h) doesn't support visual studio (MSVC).
MSVC doesn't recognize attribute((visibility("default"))), but instead use __declspec(dllimport/export).
See your previous export header which worked fine: https://github.com/clariusdev/cast/blob/c932558cdb0283e9d4d9e3c7ecfdba433867ff09/src/include/listen/listen_export.h

thanks, should be fixed


Note we inline pragma some warnings for backwards compatible logic.
These will easily be found when the props are removed as compile will fail.
How Has This Been Tested?

Build to see if warnings are gone.
Types of changes

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

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[ ] I've added relevant tests.

Thanks @mikes-gh !

I thin it's better to avoid such replace as Reset to ResetValueAsync().AndForget(); would be better to suppress with CS0618, yes the ResetValueAsync is the newer method, but ideally it would be to make HandleKeyDownAsync that awaits reset and obsolete the old one, but for this if I remember correct need to enchant IKeyInterceptor. It's just that such changes can change behavior, especially in BSS

You are right, it is dangerous, I didn't see that.

Sorry for merging prematurely.

@ScarletKuro Shall I PR a pragma instead?


Add jo

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
[x] Built on platform(s): NixOS
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip" (No dependencies)
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

More
As this is my first package created it would be great if someone had a closer look ;)
I was especially unsure about platforms = stdenv.lib.platforms.linux ++ stdenv.lib.platforms.darwin;

Please rename your commits to fit our contributing guidelines:
http://hydra.nixos.org/build/32966504/download/1/nixpkgs/manual.html#chap-submitting-changes
Correct names would be:
lib.maintainers: add markus1189
jo: init at 0.9

This style of expression is a bit unusual. Is the source information actually autogenerated, and if so how?

@joachifm yes this is auto generated. Have a look over here:
https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/upstream-updater/update-walker.txt
There are a few more of these in nixpkgs. Search for *.upstream files. I would like it more if it would produce a more common output...

@hrdinka I'm not convinced that update-walker.sh could generate that output simply from the project's github front page, but I'm not really motivated to try it either, so maybe it did. Note, however, that there's only one contributor who was ever touched update-walker, I don't think it's been generally accepted.

@joachifm I don't like this either. The output looks totally quirky and it introduces just another build description file with a different less flexible syntax (.upstream instead of nix—great). It further makes judging about expressions hard because I assume that all manual changes (like the redundant buildInputs) will be undone by the next auto update. I don't think that this is generally accepted either. I have first heard about it in this pr https://github.com/NixOS/nixpkgs/pull/13819.
@markus1189 I am curious why you have decided to use update-walker. How have you heard of it?

@joachifm Let's wait for @markus1189 response. Maybe there is a good reason or discussion about why/when to use this tool. If there is non my vote is for rewriting this as a "proper" nix derivation.

I copied the default.nix from jq ;)  Thanks for all the comments, I'll have a look now.

Okay I think I addressed all issues ;)

@markus1189 looks good, only nitpicks remain, as far as I can tell

:+1:

@markus1189  :+1: This looks much better now.
@joachifm do you know the current state of update walker? I am thinking of opening another pr to remove this tool. Many expressions shipping an .upstream file do not seem to use it anymore and the ones who do look like this commit before the rewrite. This might be one of the things which are around since ever and no one dared to remove (or update) it yet.

@hrdinka sounds like a good idea. I also think that it's still in there only because nobody has moved to delete it. In light of the recent discussion about the proliferation of autoupdaters, such a PR might very well be accepted.

@joachifm Thanks for your encouragement. As a new NixOS member I wasn't not sure if people would accept such changes or if I could even piss off some of our top contributors. There isn't much guidance in form of community guidelines or etiquette and I am not always sure about what's okay to do and what's not. I guess many contributors have these problems and recent discussions about commit reverts or the "NixOS chaos" manifest this.

@hrdinka Proposing a change and arguing its merits is never wrong, in my opinion, and I'm pretty sure most people here would agree.

Okay changed to 1.0 which was just released and also addressed the remaining nitpicks ;)

Built & tested locally (NixOS x86_64). Thanks for your contribution :+1:

@joachifm @hrdinka Actually, upstream-walker can update usual-style fetchurl expressions fine (and I also use it for that), it just doesn't generate them.
The version that put updater metadata in Nix caused more complaints…
And note that update-walker only pretends to be declarative — it is declarative for simple things, but imperative for more complex things (see atop package, for example, or sleuthkit); so putting it into Nix attributes would reduce the expressiveness.

@7c6f434c did you ever push to adopt update-walker across nixpkgs? I'm interested to read past discussions, but google is failing me, all I can find are people asking what default.upstream files are.

@7c6f434c Thanks for your explanation. It took me some while to find this out by my own. I am also interested in update-walker's acceptance within nixpkgs. I am currently using my own set of external scripts for such stuff and I am curious if I should better switch to update-walker.

I have never tried to convince people to adopt the update-walker across
NixPkgs; it doesn't look like there is a consensus that a working
auto-updating infrastructure is needed (look at monitor.nixos.org fate,
for example), and so I use it for packages I actually try to keep up to
date. Of course, I try to answer questions about it, too; I was even
shamed into writing a brief overview you have seen.
I do not believe in global adoption; it looks like most people are OK
ignoring .upstream files, unlike things that split the actual Nix
expressions into two files just for update reasons; I am willing to
implement or help implementing any specific enough suggestions that
would help you to also use it, of course.


Add workshop

What is the vendor's name?
workshop
What is the vendor's pricing site?
https://useworkshop.com/pricing/
What is the base pricing? Use the lowest tier that looks sane for a small business customer, not free or personal tiers.
$3/user/mo - 25 minimum user requirement
What is the minimum pricing for SSO support?
$3/user/mo - 25 minimum user requirement
Does this pricing info come from a quote or other non-public source?
There is mention of a custom package with additional options, but sso is in the pro and there is no free tier.
Are there any caveats we should list in the footnotes?
nope.

Given this is now unmaintained, there is now a fork that we're hoping to keep maintained via community participation.  If this is still the case with this vendor, would you please submit a fresh issue with current pricing details to that repository?

Actual link: https://github.com/stopthessotax/sso-wall-of-shame


Issue with Icons after selecting

Related page
https://mui.com/material-ui/material-icons
Kind of issue
Unclear explanations
Issue description
After copying any of the Icons if we come back whole display gets stuck fix it
Context
After copying the icons close the Dailog box and reset the state so that user can select other icons
Search keywords: Icons

Issue is pretty flaky now, but i was able to reproduce after trying few times
https://github.com/user-attachments/assets/d3bc7bdf-5a2a-42c7-98aa-7d955f438ccb

I've been having this issue for a while now (feels like at least a few weeks but I don't remember exactly). Can confirm it's flaky but generally reproducible after a few attempts.

Hey @siriwatknp I can take this up


Added back 1 tick Copper Bulb delay and removed the Redstone Lamp delay

Hi! Please, we need configurability implemented on the features prior to merge. Also please dont rearrange the mixin list for no reason, its meant to follow the feature tracker's order, even if alphabetical order makes sense too.

To be honest it may be a better idea to just separate these into two brand new clean PRs.


Not able to save posts from genesis file during InitGenesis

desmosd panic with this during InitGenesis.
panic: unsupported type map

It looks like not the issue of the message is too long and can't be saved. I have reduced post id 46 to less than 500 characters but still can't save the states.
The issue should be the InitGenesis passing a wrong object to SavePost and it gets panic when it gets a map instead of a Post during MustMarshalBinaryBare due to the change of the genesis file structure before for including reactions.
https://github.com/desmos-labs/desmos/blob/521c2e1884eb3d146f66c505fe3b43779435f8d3/x/posts/genesis.go#L39

Fixed with 39d1129


Removing Connected event from TestConnection

@mikaelm12,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Using periodic time interval values for a report creation returns errors

"- Passing the expected variables to the createReport Error returns INVALID_ARGUMENT the first time the call is done and ALREADY_EXISTS if the payload is reused. Sample payload has been shared with Halo team

When passing a blank value on the peridic time interval object the reporting server returns ALREADY_EXISTS , this might be related to the point above. Halo was able to replicate and a PR has been opened to investigate the issue"

Halo was able to replicate and a PR has been opened to investigate the issue

@tristanvuong2021 is this part correct? Can you link the PR here?

Ah, I think the ALREADY_EXISTS error is what was reproduced but not the INVALID_ARGUMENT. I'm going to split these up into separate issues.


tf.subtract doesn't work for uint8 and uint16 images (such as PNG)

It seems that tf.subtract doesn't support uint8 and unit16 image.
Could someone please add a PR to enable the uint16 and int16 support for subtraction.
Thanks,

Added PR #12574 for the support.


Breaking changes beween releases

Hi,
We used NSwag 12.0.15 to build our swagger for our .NET Core 2.2 microservices.  We have since upgraded to .NET core 3.1 and it simply doesn't work.  No idea as to why yet but when we request our swagger as we used to we get a 404 not found.
I upgraded the NSwag package to the latest to see if there was something I needed to do differently and there are many compiler errors (it looks like namespaces have been moved around and such)...
Is there a way to see breaking changes type documentation?  How might we attempt such a migration?

See https://github.com/RicoSuter/NSwag/releases/tag/NSwag-Build-1067

Thanks Rico!  This helps alot.  I really like NSwag, keep up the great work!


Renaming files: Linux support

In functions.py, rename_files uses the command
start /WAIT $file

executed by a subprocess to run a file and being able to know whether it is still open or not.
This command doesn't exist on Linux, and it is a bid sad because it is the only line of code that causes Linux non being supported by the tool 😢

Maybe look for this StackOverflow post, even though I guess you have already seen it. I talks about about using psutils to check if a file is opened somewhere else :mag:
I am not sure about the problem you are trying to solve here, i.e. why are you looking if the file is opened somewhere else? What if it is?

Also, running files blindly may not be the best way to check that :exclamation:  First of all because most files are not executable (so the subprocess will crash, but why not), and second of all because you have little way to control what the program will do once it has started :warning:
What if I am trying to rename WannaCry.exe :bomb: to NoTouchVirus.exe for instance? Should the default behavior be to that the tool runs the WannaCry.exe file :boom: just to check if someone else is writing to it? That seems like a risky move to me :laughing:

For reference purposes, the equivalent of
start /WAIT $file

On Linux would probably be
sh $file
# or, but less portable
bash $file
# or, without the new subprocess
./$file
# which is equivalent to (I think)
exec $file

Once again, the child sub-process will crash if the file is not executable :boom:

As a side note, Popen in Python (or subprocess.run) already creates a child process, so using start within Popen creates a subprocess within the subprocess, which may or may not be what you want. Also, the shell=True argument of Popen tells it to run the command within a new terminal (or shell in the Linux world). Usually this makes it more convenient to use because you can give the shell a string for its command, whereas a direct call to the OS would expect a list of strings instead.
Put graphically, Popen("start /WAIT $file", shell=True) amounts to:
The python process
|-- The Popen process
     |-- A new terminal (because of shell=True)
          |-- A new terminal (because of start)
               |-- The process running $file

If you don't want the additional processes (but maybe you do?), one way to run the file would be to use:
from subprocess import run
# You can adapt the following line with Popen.
subprocess.run([str(file_name)])
# For instance to run cli.py with Python
subprocess.run(["python3", "cli.py"])

This should schedule the process directly, without the intermediary shells :shell:


:bug: Image zoom level too low

Describe the bug
The image zoom level on click is most of the time very low to the point it barely increases the size of the linked image.
To Reproduce
Steps to reproduce the behavior:

Add an image (in my case 2536 x 1215 px) with the Figure Shortcode via ![Alt text](image.jpg "Image caption")
Deploy the site and click on the linked image and notice the size barely increases.

Expected behavior
The image zoom should be bigger, so details can be seen too.
Screenshots
No Screenshot but see the first picture in this post as an example.
Desktop (please complete the following information):

OS: Windows 10 / Android 13
Browser: FireFox
Version 108.0.1

Hugo & Blowfish versions
Hugo: v0.106.0
Blowfish: v2.18.0

will add a global param to disable the hugo pipelines that make the image smaller for perfomance. You can override and just use the original image... look for disableImageOptimization in the next release.
https://github.com/nunocoracao/blowfish/pull/339

I just tried out Blowfish v2.19.1, seems like the feature images of posts in list view (layout = "hero") look kind of buggy.
Here is a before Screenshot:

And here is one after updating to V2.19.1:

Not entirely sure if this behaviour is related to the changes made regarding "Image zoom level".
Let me know if I should open up a new issue.

@chrisbanes Is this related to https://github.com/nunocoracao/blowfish/pull/327?

@mztiq Not related to the original ticket I think

@chrisbanes Is this related to https://github.com/nunocoracao/blowfish/pull/327?

I don't believe so. My change was only in the article link partials.

Ah, that looks like Hugo's smart cropping. You can go back to the previous behaviour by changing the anchor value in your config.toml:
[imaging]
  anchor = 'Center'

This actually helped a lot, but the feature images are still slightly "zoomed in" (hard to notice if you don't have the screenshots side-by-side).
v2.18.0:

v2.19.1 with  anchor = 'Center':

Reopening so that I'll have a look at this one... @chrisbanes removing the change you added on #327 would provide the outcome @mztiq is looking for right? May I ask you to clarify the different between the two behaviours aka the one before your change and after the change? Thanks

The difference is that the old behaviour resized the image to 600px always (maintaining the aspect ratio), then relied on CSS to cover the resulting box.
The new behaviour displays a pre-cropped image which has a roughly similar aspect ratio to what is displayed. It's not exact as the resulting div has a different size for different display sizes.
I don't personally think there's an issue here. An alternative approach would be to remove the crop, and resize based on a fixed height. That is a bit more wasteful though.

@mztiq could you do a quick test by changing your local Blowfish theme files:
file: layouts/partials/article-link.html
Change
{{ with .Fill "600x400" }}
to
{{ with .Resize "600x" }}
And share screenshots. I want to check if this is the behavior you are going for.
@chrisbanes That might be an interesting approach. I was reading the image processing docs and need to understand better the .fill function. It seems that it does crop AND resize which might be causing the issue here. The resize option seems to be the one that always keeps the ratio. wondering if we should have a global var for this behavior across images or if that is overkill. thoughts?

@nunocoracao That's exactly the behaviour I'm going for, thanks for the suggestion.
Screenshot before:

Screenshot after changing to {{ with .Resize "600x" }}:

@chrisbanes @mztiq my proposal:
Create a new global var in params.toml, imageProcessingMode with two values resize and fill. That way we can have both features in and people can chose which one applies to their use-case. Thoughts? If all agree I can move ahead with implementation for v2.20.0

@nunocoracao sounds great to me 👌.

My $0.02: this shouldn't be configurable. Adding config values for every possible combination is going to make this theme harder to maintain in the long term. It's your choice though.

@chrisbanes that is a good point... already suffering a little for some similar past decisions.
3 options for voting (use emojis please):
⚙  Make it configurable - more maintenance work all options are covered now
📦  keep current implementation fill - @mztiq would this work for you?
🧷  revert to previous implementation using resize - @chrisbanes would this work for you?
Tagging some other contributors to get some votes in this:
cc: @madoke @sergiod26 @Scoty @kszpieg @insidemordecai @lemonase @Coronon

@nunocoracao
Personally, I think keeping this at the partial layout/template level is a cleaner solution.
Adding a section in the docs that links to Hugo's image processing methods would also be helpful for those who want to customize.

Ah, that looks like Hugo's smart cropping. You can go back to the previous behaviour by changing the anchor value in your config.toml:
[imaging]
  anchor = 'Center'

Kinda agree with @lemonase, the only change to my config is adding @chrisbanes' suggestion so maybe linking/referencing Hugo's docs makes sense.

Thanks @mztiq @chrisbanes @lemonase @insidemordecai, already accepted the PR from @lemonase. Will close this comment and leave the current implementation as-is.


gpt-actions.py is not executing fully

While running gpt-actions.py  I found out  if block if cnt > 0  from line 109 to 133 is not executing because of line 80 cnt -= 1 .
Is it intentional or some logic behind it.

The cnt-=1 is essentially to count the number of correct outputs. The 109-133 if block only executes if there is atleast one correct output from the LLM


Example Issue

You can ignore this issue

is @abirismyname awake?

is @abirismyname awake?

is @abirismyname awake?

is @abirismyname awake?

is @abirismyname awake?

is @abirismyname awake?

is @abirismyname awake?

is @abirismyname awake?

is @abirismyname awake?

is @abirismyname awake?

is @abirismyname awake?


[SPARK-33489][PYSPARK][WIP]Add support for converting null from & to Arrow…

… type

What changes were proposed in this pull request?
Add support for converting null from & to Arrows

Why are the changes needed?
Since this currently unsupported

Does this PR introduce any user-facing change?
No, since the documentation does not mention about the lack of support for null..
https://spark.apache.org/docs/2.4.0/sql-pyspark-pandas-with-arrow.html#supported-sql-types

How was this patch tested?
WIP

Can one of the admins verify this patch?

Hi @Cactice, what's the status of this PR?
Seems like this is still missing data conversions in Scala side, and PySpark side.
If you are not working on this anymore, I will take over, so please let me know.
Thanks.

Hi @Cactice, what's the status of this PR?
Seems like this is still missing data conversions in Scala side, and PySpark side.
If you are not working on this anymore, I will take over, so please let me know.
Thanks.

Sorry I haven't worked on this
Please take over when you can.

Sorry I haven't worked on this
Please take over when you can.

@Cactice Sure, then I'll work on this. Thanks for letting me know!

@Cactice Sure, then I'll work on this. Thanks for letting me know!


Extended status page with nodes blocks

Changes
Status page works with status endpoint status.moonstream.to
How to test these changes?
Tested
Related issues

@bugout-dev check

[ ] Finish CORS check in status server
[ ] Add secret NEXT_PUBLIC_BUGOUT_STATUS_URL = https://status.moonstream.to


libdrm: 2.4.94 -> 2.4.96

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libdrm/versions.
meta.description for libdrm is: '"Library for accessing the kernel's Direct Rendering Manager"'.

Checks done (click to expand)

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.4.96 with grep in /nix/store/fph5kknlx5jvjijfqs5wv7cszfli92nx-libdrm-2.4.96

directory tree listing: https://gist.github.com/1d3aa5f6c3e03a89361dfab582e2a68b

du listing: https://gist.github.com/fef70666b3eb756d636b9dfcaf5f3bd9

Outpath report (click to expand)

Outpath difference report
14268 total rebuild paths
5224 package rebuilds
5207 x86_64-linux rebuilds
4620 i686-linux rebuilds
27 x86_64-darwin rebuilds
4414 aarch64-linux rebuilds
First ten rebuilds by attrpath
AgdaSheaves
AgdaStdlib
DisnixWebService
EmptyEpsilon
R
SDL
SDL2
SDL2_gfx
SDL2_image
SDL2_mixer

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/fph5kknlx5jvjijfqs5wv7cszfli92nx-libdrm-2.4.96 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A libdrm https://github.com/r-ryantm/nixpkgs/archive/6f5e1e3055d224069d6179ce423152f5b8d0785c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/fph5kknlx5jvjijfqs5wv7cszfli92nx-libdrm-2.4.96
ls -la /nix/store/fph5kknlx5jvjijfqs5wv7cszfli92nx-libdrm-2.4.96/bin

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: libdrm
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: libdrm
Partial log (click to expand)
shrinking /nix/store/smqpz9fv7zpyh3mi674fr9ha6gasc8y1-libdrm-2.4.96-bin/bin/modetest
shrinking /nix/store/smqpz9fv7zpyh3mi674fr9ha6gasc8y1-libdrm-2.4.96-bin/bin/proptest
shrinking /nix/store/smqpz9fv7zpyh3mi674fr9ha6gasc8y1-libdrm-2.4.96-bin/bin/modeprint
shrinking /nix/store/smqpz9fv7zpyh3mi674fr9ha6gasc8y1-libdrm-2.4.96-bin/bin/kms-universal-planes
shrinking /nix/store/smqpz9fv7zpyh3mi674fr9ha6gasc8y1-libdrm-2.4.96-bin/bin/kms-steal-crtc
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/smqpz9fv7zpyh3mi674fr9ha6gasc8y1-libdrm-2.4.96-bin/bin
patching script interpreter paths in /nix/store/smqpz9fv7zpyh3mi674fr9ha6gasc8y1-libdrm-2.4.96-bin
checking for references to /build in /nix/store/smqpz9fv7zpyh3mi674fr9ha6gasc8y1-libdrm-2.4.96-bin...
/nix/store/pqvp8a3jk1d2r5s14448kxizj3caf89x-libdrm-2.4.96

Success on x86_64-linux (full log)
Attempted: libdrm
Partial log (click to expand)
shrinking /nix/store/vswvy2jrrdv6v5b71lc6xcf5ch9yg0bd-libdrm-2.4.96-bin/bin/modeprint
shrinking /nix/store/vswvy2jrrdv6v5b71lc6xcf5ch9yg0bd-libdrm-2.4.96-bin/bin/vbltest
shrinking /nix/store/vswvy2jrrdv6v5b71lc6xcf5ch9yg0bd-libdrm-2.4.96-bin/bin/kms-steal-crtc
shrinking /nix/store/vswvy2jrrdv6v5b71lc6xcf5ch9yg0bd-libdrm-2.4.96-bin/bin/kms-universal-planes
shrinking /nix/store/vswvy2jrrdv6v5b71lc6xcf5ch9yg0bd-libdrm-2.4.96-bin/bin/kmstest
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/vswvy2jrrdv6v5b71lc6xcf5ch9yg0bd-libdrm-2.4.96-bin/bin
patching script interpreter paths in /nix/store/vswvy2jrrdv6v5b71lc6xcf5ch9yg0bd-libdrm-2.4.96-bin
checking for references to /build in /nix/store/vswvy2jrrdv6v5b71lc6xcf5ch9yg0bd-libdrm-2.4.96-bin...
/nix/store/xld430fa019pv7ap5bclh9a866vv32rz-libdrm-2.4.96


32bit .NET memory. 4.8 vs 4.5-4.7

Setup:

Two identical Windows 2016 server VMs with 8GB RAM
Both with latest Windows Updates applied
One has .NET 4.7.2, the other 4.8
Same test app, with the same configuration, executed with the same allocation pattern.
The background load on the servers looked comparable.

Results (give or take):

On the .NET 4.8 VM, the test app could  grow to about 3.7 GB (before throwing OOM exception)
On the .NET 4.7.2 VM that number was only 2.76 GB

BTW - the same ~2.7 GB limit was observed on several Windows 2008 R2 servers running .NET 4.5.2 too.
This is almost 1 GB difference, which is huge, given the 32 bit cap to start with!
Is there's something that's (so much) improved in .NET 4.8 runtime itself to account for that or when installed 4.8 changed some settings and that's the only reason?
Settings that would've otherwise made .NET 4.5-4.7 to claim that almost 1 GB too.

You can check the virtual address space with VMMap. It looks like some large memory holes are not present there. When you know if it is the loaded dlls or the managed heap you can attribute it either to assembly (not) loading or to improvements how the GC deals with pinned objects and memory fragmentation. Normally you will start with Windbg and SOS and !EEHeap -gc the analysis to see how many managed heaps you have and how large they are. Then you need to look into your allocation pattern and sizes and where pinned memory resides.

Microsoft support acknowledged this to be a bug. Will keep the issue open to update on the resolution when available.

Hi @Peperud do you have any link to the bug you mentioned and/or do you know if the bug is being fixed by Microsoft? We're seeing comparable behavior in our processes after an upgrade from .NET 4.5.2 to 4.8.

upgraded to 4.8 and memory usage out of control and winds up causing other processes on the machine to be killed.  no bueno.

i've tracked down the memory issue i experienced.  my application was loading static data of around 10 million strings into a dictionary.  the dictionary object was 1-2GB in the LOH and the strings sat in Gen2.  After some heavy activity the heap experienced massive fragmentation and my memory foot print ballooned to 16GB when the actual memory in use was only a few gigs.
this does not happen under .net47 and below.

@coderb: Do you have a rpro? Are you having pinned objects around? Or would a
GCSettings.LargeObjectHeapCompactionMode = GCLargeObjectHeapCompactionMode.CompactOnce;
GC.Collect();

solve your issue?

repro is going to be hard.  it's a large production app.  i've already taken steps to use custom memory handling to eliminate the massive amount of objects into byte array tables.  my guess is this:  my machine has 32GB memory with swap setup as "system managed" on a spinning disk with say 1-10mb/sec write speed.   for some odd reason when the dotnet process decides to grow its commit size to 16GB windows decides it wants to start killing other processes rather than swapping.  swap only grows to around 6GB with plenty of disk space and no system limit setup.
so, not really sure what's different.  i used to run just fine under 4.7 and decided to install the 4.8 targeting pack in visual studio.  i was unaware that this would in-place upgrade my runtime to 4.8 which seems to have a much different garbage collector than 4.7.  also it seems there is no option to downgrade the runtime to 4.7.
i think that i'm good now with some very tricky code changes to avoid having lots of objects.  however, i would recommend that you try to create some stress test cases that load 10-50 million strings into a dictionary and then try to do operations that generate lots of garbage.  the runtime really should be able to handle this scenario in a way that seems more graceful than what i've observed.  hope this helps.

 It is not Linux with an OOM Killer feature. The reason why your application memory is never swapped out is because the GC keeps your working set with each full GC alive and prevents swapping almost entirely for an allocation heavy .NET application. It could be that you suffer from increased fragmentation, but without a repro it is hard to tell what is different.


IE11: SCRIPT5007: Unable to get property 'ownerSVGElement' of undefined or null reference

Got SCRIPT5007: Unable to get property 'ownerSVGElement' of undefined or null reference from https://github.com/d3/d3-selection/blob/v1.4.1/src/point.js#L2 when using<EMAIL_ADDRESS>Turns out the problem is fixed in v2.0.0 by this Handle missing targets. commit.
We have limitations on upgrading to latest d3-selection. Can you fix this in a patch to v1.4.x?

No, sorry.


Check before_ actions for false response

Rails 5 has changed the meanings for before_ callback chains; "false" no longer stops.  See:
https://blog.bigbinary.com/2016/02/13/rails-5-does-not-halt-callback-chain-when-false-is-returned.html
In most cases we redirect, so in most cases this doesn't matter, but we should check it.  Our tests already check for various "not logged in" and "no authorized" cases, so I'm not expecting this to find any security issues.
This is related to #1118.

See #1121 for a stopgap measure.

We have resolved this.


Issue with loading backups on SCPH-5501

Made a backup of Parasite Eve for an NTSC PSX (Model number SCPH-5501). I am able to boot into FreePSXBoot, but the game won't load. FreePSXBoot also identifies the part number on the backup CD. Same issue happens on official, licensed PSX CDs. FastLoad identifies the part number, but won't actually boot into the game.

I have same thing with 5500 (with swap trick). Very few backups loading correctly.

I can boot backups just fine with the latest version of FreePSXBoot (20211004) installed into Slot 2 on a SCPH-5501.  Retails sometimes don't boot when FreePSXBoot is just chillin' in Slot 2 [exploits not running] (which is a current, known, issue).

I just tested Parasite Eve and Rayman backups on a NTSC SCPH-5501 with freepsxboot-unirom-fastload-20211004-bios-3.0-1996-11-18-A-8d8cb7e4-slot2 and I had no issues booting them. If the OP can't provide any more evidence or a follow up that this is still an issue I would close this.


Training from scratch

first of all, Good job
I want to train this model from scratch. (e.g. tiny yolo)
for real time problems
when i tried to turn the train.py file, an error appears that it can't open the weights file
" OSError: Unable to open file (Unable to open file: name = 'full_yolo_features.h5', errno = 2, error message = 'no such file or directory', flags = 0, o_flags = 0) "
I didn't want to load it
i want to load the model architecture then train it in my own dataset
Thanks in advance.

H
In config.json you need to clear the "pretrained_weights":   "",
or
if you would like start with weigthts, you need download pretrained features of full yolo at:
https://1drv.ms/f/s!ApLdDEW3ut5fec2OzK4S4RpT-SU
and save in basic-yolo-keras folder root

@AhmedAAkl @kabrau is right. You need to download and put the pretrained weights of the TinyYolo backend in the root folder. The backend must be pre-trained, otherwise the network would not converge. It's very difficult to train an object detection model from scratch.

@kabrau thanks for response. but, i cleared the "pretrained_weights". and also i didn't work.
the problem appears when backend.py module tries to load_weights()
self.feature_extractor.load_weights(TINY_YOLO_FEATURE_PATH)
i didn't find the specified weights file.
@experiencor  you are right the backend will load the pretrained weights.
i want to train this model to detect the traffic signs and classify them so, i want to train the tiny yolo from scratch.
thanks

Hi @AhmedAAkl,
Is the dataset of the traffic signs public?
If yes, can you send me url ?
Tks

@kabrau  unfortunately, it is not public. ( traffic sign detection)
all tutorial shows that the developer make it by him self.
but, sign recognition is available. try the german traffic signs dataset. it is the popular.


LABS Group Grant Proposal

Grant Application Checklist

[x] The application-template.md has been copied, renamed ( "LABSGroup.md") and updated.
[x] A BTC or Ethereum address for the payment of the milestones is provided inside the application.
[x] The software of the project will be released under the Apache license version 2.0 as specified in the terms and conditions.
[x] The total funding amount of the project is below USD $30k for initial grant applications and $100k for follow-up grants at the time of submission.
[x] The initial PR contains only one commit (squash if needed before submitting your PR).
[x] The grant will only be announced once we successfully delivered the first milestone.

Substrate module: X

Clearly you have not filled out the template properly. I'm closing due to incompleteness - please do not apply with incomplete proposals anymore.

Substrate module: X

Clearly you have not filled out the template properly. I'm closing due to incompleteness - please do not apply with incomplete proposals anymore.


/lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.29' not found (required by /usr/local/bin/communication)

I used cargo chef to build my image, but my docker run fails with the message:
/usr/local/bin/communication: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.29' not found (required by /usr/local/bin/communication)

Here is my Dockerfile:
FROM lukemathwalker/cargo-chef:latest-rust-1 AS chef
WORKDIR communication

FROM chef AS planner
COPY . .
RUN cargo chef prepare --recipe-path recipe.json

FROM chef AS builder
COPY --from=planner /communication/recipe.json recipe.json
# Build dependencies - this is the caching Docker layer!
RUN cargo chef cook --release --recipe-path recipe.json
# Build application
COPY . .
RUN cargo build --release --bin communication

# We do not need the Rust toolchain to run the binary!
FROM debian:buster-slim AS runtime
WORKDIR communication
COPY --from=builder /communication/target/release/communication /usr/local/bin
ENTRYPOINT ["/usr/local/bin/communication"]

I'm sorry I'm a bit of a noob to Rust. Wasn't sure if this problem had already been ran into?
Thanks!
Update: For extra context here's my Cargo.toml deps
async-trait = "0.1.58"
chrono = "0.4.23"
futures = "0.3.25"
gtmpl = "0.7"
openssl = { version = "0.10", features = ["vendored"] }
mongodb = { version = "2.3.1"}
phonenumber = "0.3.1"
regex = "1.3.9"
serde = "1.0.152"
serde_derive = "1.0.151"
serde_json = "1.0.89"
tokio = "1.22.0"
twilio-openapi = "1.2.2"
warp = "0.3"
tera = "1.17.1"

I added
RUN apt update
RUN apt install libc6

Right after the FROM debian:buster-slim AS runtime
But that didn't seem to fix it.

First time seeing this!
Can you provide a minimal reproducible example for me to try to troubleshoot it?

I was able to fix this by switching the debian base image to this:
FROM debian:bullseye-slim AS runtime

It's a more recent image. On the one from the docs, it has 2.28 libc6 installed manually, so even if you try to install it, you'll still have 2.28. Not sure what is the reason my binary is requiring that though. I posted my deps for potential help. Feel free to close this.

What version of Rust does the builder image use?

Whatever is currently on the lukemathwalker/cargo-chef:latest-rust-1 image.

This is happening to me, too, using the example from the README (cargo-chef:latest-rust-1 and debian:buster-slim). The recommendation to switch to bullseye-slim is not fixing the problem. This does seem like an incompatibility between the builder and the runtime base image, though, right?

You need bookworm as runtime, I've updated the README!

Nice, thank you @LukeMathWalker! :tada:

I am coming back to this issue again :joy: is there a reason this keeps needing to be changed? Seems like a very tedious thing to maintain with no long-term solution for the GLIBC version? I was still on bullseye-slim, switching to bookworm-slim now.

New Rust versions bump the requirements, that's the reason.
There's no solution, beyond creating a "cargo-chef-runtime" image that swaps Debian version automatically, but that's likely to create a whole other set of issues.


How to horizantally centre items?

I want the items inside the MuuriComponent to be horizontally centre:
check out this demo: https://codesandbox.io/s/muuri-react-media-query-forked-f1p92?file=/package.json
I want the items in the 2nd row to be horizontally centre, I've tried modifying CSS for MuuriComponent but it didn't worked for me.

Good question. I have the same problem in my app.

@Kefirr Let me know if you found any solution for this

In my case helped: .item {   width: 25%;   height: 400px;   position: absolute;   padding: 10px; }

@Kefirr I think in your case number of item is fixed, but in my web app, the items keeps changing dynamically upto 25 items.
So if there's an odd number of items then I want it to be horizontally center.
Will it be good to use utility getResponsiveStyle in this case?


client: Scan action will store zip on disk which may cause permission denied

At the moment the "code scan" action zips the code-folder before the upload. If the current user doesn't have write access to this directory (e.g. in a CI context) the check will fail because of a permission denied error.
We suggest three possible solutions:

Zipping to "temp" directory
Zipping file without writing to disk
Create parameter to configure Zip output directory

Paul Romahn<EMAIL_ADDRESS>Daimler TSS GmbH,  imprint

@romahnp thanks for opening the issue.

Will introduce a -tempdir option.
There all temporary files (currently only .zip file) will be created.
Default is current directory. (compatibility with current behaviour)


[proposal] koord-runtime-proxy:  Introduce TimeoutIgnore when calling hookServer timeouting

What is your proposal:
Why is this needed:
Is there a suggested solution, if so, please add it:

To hint hookServer by label  (e.g.  koordinator.sh/hookserver: true) if hookServer deployed as k8s container, runtime-proxy should transfer request for such specified pod to backend docker/containerd transparently.


"0x0" is not supported as PedersenArg

While integrating this lib in starknet.js, I found a minor issue which I think should be rectified.
In the function pedersenArg(), if 0 is passed as a bigInt or number, it is returned as 0 or 0n. However, if 0 is passed as a hex string "0x0", it goes into normalizeScalar() and throws TypeError because (parsed) 0 is not less than 0.
Why is this important?
The function compute_hash_on_elements is initialised with a 0 and we wish to pass all the values as Hex string including 0.
Proposed Solution

The easiest solution would be adding if (typeof arg === 'string' && hexToNumber(arg) === 0n) return 0n; here.

Add computeHashOnElements function in this lib which ensures that's its always initialised with 0 (or 0n) and just pass the elements as Hex[]  or (PedersenArg[])

fixed in 074f418


$DB_PORT Variable needed for custom DB port.

Soon we will add this into the image.

No time to add this guys. If you need it please send a PR and we can merge it.


Add the topic datenstrom-yellow

Hello Martin!
Here's a tip from the Datenstrom Yellow maintainers. Please add the topic datenstrom-yellow to your repository. https://github.com/topics/datenstrom-yellow
Thanks for developing/designing extensions, this is much appreciated. 👍

Hi Mark!
I was not aware of the topic but it is now added.


⚠ Bofrost.at has degraded performance

In 56f15f3, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1341 ms

Resolved: Bofrost.at performance has improved in d90aaf9.


[Monsterhearts-2] Translations.

Hi.
Could be possible to upgrade the Monsterhearts-2 sheet by @Lockbox313 to allow translations?
The spanish version will be released in a few months and I would like to translate the sheet.
Thanks.

I had my ACSI script autogenerate i18n-tags for the sheet, so if you can find a Pro member to check how good of a result it gave and generate the translation.json file, the person could have most work done for him/her.
Autogenerated result: Monsterhearts-2.html

Thanks.
I don't know if the sheet contributors (@Lockbox313 and @TripsLetash ) are still "online". I am just a Plus member and I do not know too much about programming.

I'm online but I haven't touched these sheets in over a year. You're welcome to make the necesarry edits as you see fit. I don't even think I'm still a plus member on roll20

Here is the result https://app.roll20.net/join/4072362/j60XaA of Anduhs auto generator.  The generator stripped the  > from </button/>.  It is also doing some wierd things with  tags seems to be inserting  the data-i18n in the opening and closing tag.
Also any chance you could output a default translation.json from my dim memories of the DOM it should just be a matter of grabbing the innerHTML of the tag you are inserting the data-i18n into .

The generator did not seem to like the "type="number" in the button either.

Yeah the script is crude so its a bit hit and miss if the output is worth using. I'll see if I can fix these newfound errors. Having it create a translation.json file is one of the features considered after the core functionality is more reliable.

Also, I've seen that when you change the playbook the sheet don't load . It shows only  the default playbook (The Fae).
That's on the game @Lockbox313 linked.

The intent of that was to show the results of the auto generator, not to hand you a full functioning sheet with translation tags at the drop of a hat.
This is still a lot of work. There is a lot of malformed html in the original sheet with tags out of place and random  tags inserted.  The generator removes the ending from some or all of the ,  and .  Understandably the generator also has problems with large blocks of text with  and  and other text formatting tags. It also has an issue with drop down options which I have not figured out.
I have just spent a couple of hours going through and fixing the structure and mangled tags and replacing the text of the moves with the original.  The text of the moves may be an issue as I do not know how Roll20 translator works with large blocks of text.
A  basic translation.json needs created as well which means going through and pulling all the data-i18n and the associated text.
This is all the time I have for this currently this. Here is the sheet as it stands now. https://github.com/Lockbox313/Monsterhearts/blob/master/Monsterhearts-2.html

So it sounds like this time it was barely a timesaver in comparison to just doing it from scratch?
I'm sorry the output wasn't really worth it this time, but I warned that:

If the script didn't create a timesaving starting point, it doesn't have to be used.

The biggest issue is the mangling of tags

Yeah ofc, but was it still better than doing it manually from scratch?

Only if you fix what is breaking the tags.

After improving the ACSI script(breaks less tags now) I've now updated the html and translation.json it creates, and while it's a noticeable improvement, it still have hickups that need to be manually. Might in a few weeks do the last cleanup my generated translation requires for us to get a working translation.

Thanks. Waiting for your "cleanup" to begin the translation.

I used a lot of the Monsterhearts-2 sheet HTML for the Tremulus one.  Assuming the translation work I did there works, I could use the same techniques to get i18n tags in the Monsterhearts sheet.
@WallaceMcGregor do you think you could let me know if that sheet is reasonably translatable?  I don't want to start that effort without knowing if the results will be useful.

@WallaceMcGregor do you think you could let me know if that sheet is reasonably translatable? I don't want to start that effort without knowing if the results will be useful.

@JBeighel the modification to allow translations is not ended or committed (several months have passed without news). I don't know how is or if It'll work.

@WallaceMcGregor  @Anduh  posted revised html and  translation,josn onMay 15.  If you were keen to see this resolved  you could have taken these files and the CSS from the the Roll20 repository and setup a game with a custom sheet and tested it  for yourself and the committed it yourself.

@Lockbox313 I am not a Pro user so I can't use a custom sheet or upload one.

@WallaceMcGregor  You won't spend $10 on a PRO membership for a month to help with this but you expect others who have to work for you for free and then complain when they are not fast enough for you.

@Lockbox313 I've never complained anyone about the time to doing something. I've never said something like that. I am not a programer and I've made a lot of sheets translations to my language (including the Roll20 interface) or reporting errors, only trying to contribute the best I can. I am not a english native speaker so I think maybe could be a misinterpretation if sometime sounded like that.

@WallaceMcGregor "several months have passed without news"

I'm just answering @JBeighel If you think "several months have passed without news" is a complaint I can say nothing. I am trying to explain him the state. Anyway If you think that simple comment is a complaint, I don't know what to say you. That wasn't my intention. I've never thought it could be taken as a complain. I don't get it.

The translated code I posted was autogenerated by a script, and wasn't fairing well for this particular sheet  so it's understandable that it's unlearn if it's better to start from the actual sheet or from the result of my script. I have little to no time to participate unfortunately, but can run my script again and see if the result is better since I might have improved it since last time.
But yeah I was a bit unclear on my communication as well, I never said I'd spend time to clean up the code, I only offered my autogenerated version as a possible starting point if someone else where to pick this up. I had mistakenly also thought Wallace had the opportunity to try to finish the translation so I left it at that.
If someone without Pro would take the time to create the translation, I'm happy to quickly doublecheck/polish the work and submit it to the repo. IMO the work is somewhat simple, just tedious and repetative, and not the best use of my time. It's also the reason I made the script, to automate part of the process, altough it only work ls on the better structured sheets.

"several months have passed without news"
Saying this means you expect and are entitled to something and you have not received it so therefore this is a complaint.

@Anduh Running it  through again would be pointless since you are using the Roll20 github  code which has structure problems and your generator is still stripping > of various tags. I have again went through the HTML  and fixed some of it but as this stage all the skins after the Hollow are still trashed and once this is done work will probably have to been  done on the generated translation.json.  I have no more time to work on this today but here is an invite to the test game  https://app.roll20.net/join/4869171/AU04fQ

Shoot, I didn't know you both were working on this.  I had started down a different path of creating a translation.json file working for it.
I don't know that its helpful since I was going a very different route, but the code I had is here:
https://github.com/JBeighel/roll20-character-sheets/tree/master/Monsterhearts-2
I don't have the Witch Hexes/Infernal Bargains, Experience, Advancement, or macros done yet.  The rest I'm pretty sure is covered, though the translation file likely could be organized better.

Shoot, I didn't know you both were working on this. I had started down a different path of creating a translation.json file working for it.

I think Lockbox313 just did something quick just now since joining this discussion. I guess this will get done now that both you and Lockbox313 have interest in getting the translation done. Communicate with each-other and good luck.

Edit: IMO the text- prefix on the tag names is a bit redundant and just makes them unnecessarily verbose, but don't go and change it now that it's the way it's currently.

You're right, its an old habit to distinguish string length.  It likely doesn't help much here since there aren't many useful distinctions to make.

You're right, its an old habit to distinguish string length. It likely doesn't help much here since there aren't many useful distinctions to make.

Many sheets name the tag-name so it's understandable on their own, Starfinder is one such example, but on the other hand I think your move1desc,move2desc etc have benefits from a sheet structure standpoint, so either way is great and internal consistency is always the smart thing to do.

@JBeighel Are you working on Tremulus sheet or Monsterhearts 2 sheet?

Both, I just created the Tremulus sheet.  I borrowed a lot of HTML from the Monsterhearts so I'm familiar with it and thought I'd pitch in on this request for the Monsterhearts sheet.
I was re-using the tools for adding translation stuff to the Tremulus sheet to create similar tags in the Monsterhearts sheet.  Which is what I was showing in that previous repository link.

I would go ahead with your version because  fixing the generated html and the generated  translation.josn would probably take longer than finishing off your version.

Ok, I can keep working on that.  It will stall a bit in the macros, they use external images to show some of the text (Roll, Success, and Failure I think).
Do you know what was used to generate the HTML?  I don't do much HTML work, so I'm in the stone age with just a text editor.  Figured it might be good to look into modern tools.

I use a text editor but then again that is how I started in the 80s'

You do not need generate any html.  Look at the roll template at the bottom of the html file there area bunch of  tags.you justneed to change the src attribute to point your own image.

Making the image would be the trick.  I'm not good enough at image work to have it match the theme.  I guess I can create the tag and leave it open to creating an image or just inserting text.

What images do you need?

I guess I'm thinking about it wrong, I don't need them it'd be the translator who'd have to create these in the other languages:
http://i.imgur.com/Y4a3NF3.png
http://i.imgur.com/ijSYmKZ.png
http://i.imgur.com/8IqLjkG.png
http://i.imgur.com/IJBloDk.png
http://i.imgur.com/Z4MvPRa.png

I would not worry about the images for the Monsterhearts 2 translations. I don't think the  translation system can do anything about switching out images based on language anyway.  But you would probably want to change the images for  your Tremulus sheet.

Yup, that I did do.  Just have it showing the text.

Ok Good luck pm at https://app.roll20.net/users/720651/ if you think I could help.

Finished the work on this one.  Submitted a pull request to bring in the changes.  Guess we'll see where it goes next.


chore(build): fix semrel build step

This PR updates the build to use node 14 to satisfy semrel's new release.
I also tweaked our use of python to use python3 with bump2version as well

:tada: This PR is included in version 9.13.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Updated snapheal-pro to use HTTPS

Thank you for the contribution. It was merged directly as f48fdb2909c5684676964bf7fed857960fc47dcf to keep commit history cleaner. Your contribution is still credited to you.


Turning off Instance Optimization in unigradicon-register

Hi @HastingsGreer and @marcniethammer,
Congrats on the very cool work! I wanted to try the binary on a multi-modality pair and the sense I get from #3  is that I should turn off instance optimization for that.
Something breaks in the CLI arg handling though so the unigradicon-register binary doesn't seem to work with --io_iterations 0.
Traceback (most recent call last):
  File "miniconda3/envs/unigradicon_demo/bin/unigradicon-register", line 8, in <module>
    sys.exit(main())
  File "miniconda3/envs/unigradicon_demo/lib/python3.9/site-packages/unigradicon/__init__.py", line 204, in main
    phi_AB, phi_BA = icon_registration.itk_wrapper.register_pair(net,preprocess(moving), preprocess(fixed), finetune_steps=int(args.io_iterations))
  File "miniconda3/envs/unigradicon_demo/lib/python3.9/site-packages/icon_registration/itk_wrapper.py", line 58, in register_pair
    raise Exception("To indicate no finetune_steps, pass finetune_steps=None")
Exception: To indicate no finetune_steps, pass finetune_steps=None

Passing --io_iterations None leads to:
Traceback (most recent call last):
  File "/data/vision/polina/users/dey/miniconda3/envs/unigradicon_demo/bin/unigradicon-register", line 8, in <module>
    sys.exit(main())
  File "/data/vision/polina/users/dey/miniconda3/envs/unigradicon_demo/lib/python3.9/site-packages/unigradicon/__init__.py", line 204, in main
    phi_AB, phi_BA = icon_registration.itk_wrapper.register_pair(net,preprocess(moving), preprocess(fixed), finetune_steps=int(args.io_iterations))
ValueError: invalid literal for int() with base 10: 'None'

The issue is that int(None) isn't possible so an extra if-statement converting 0 to None for finetune_steps or something similar should fix this.
Thanks,
Neel

Thank you for the issue! You are right, there should be a special case for turning off instance optimization. I have begun a pull request to fix this.
https://github.com/uncbiag/uniGradICON/pull/5

Neel, I think for #3 the issue was likely an orientation issue (see answers). We are working on fixing this in more generality and also to support similarity measures for instance optimization that are more appropriate for general multi-modal data (see #5).

Hi @neel-dey ,
We also provide a colab demo showing how to use uniGradICON. It shows the inference result without IO.

@HastingsGreer Fantastic, thanks! I'll close this issue as it's merged.
@marcniethammer That makes sense, I'll follow up more re: multi-modality in our email chain.
@lintian-a Thanks, this is very clearly laid out!


When I click preview response nothing is shown

Your description here
Expected outcome
What should happen?
Actual outcome
What happened?
Versions
App: <IP_ADDRESS>6-stable
Chrome: 55.0.2883.87
Platform: Windows (7)
Steps to reproduce

This is fixed in next release.


I need detail of case

Hi,
Which is that case model and size you put on the case?

Hi,
Case: https://www.thingiverse.com/thing:5680106
Measures are 9cm x 5.3cm, 1.6cm thick


Incorrect Instructions

The instructions tell us to go to Email & collaboration > Attack simulation training > End-user notifications, but there is not an End-User Notification page tab on the Attack Simulation training page. Please update the instructions to describe where this feature has been moved to.

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

ID: 62182f02-0365-ffc2-b94c-356f59cc2814
Version Independent ID: 62182f02-0365-ffc2-b94c-356f59cc2814
Content: End-user notifications for Attack simulation training - Office 365
Content Source: microsoft-365/security/office-365-security/attack-simulation-training-end-user-notifications.md
Product: m365-security
Technology: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

Yeah, I see a banner on the page that mentions the issue:

I updated the affected topics where stuff moved around (see the referenced pull request).
Thanks for the feedback.
Closing


Setup Postgres DB

How to handle PG specific constraints & triggers that work well with the automatic migration that Persistent does?
Can Persistent automatically create FK constraints in the schema?

Can Persistent automatically create FK constraints in the schema?

Yes.

@wz1000 do you want to add the following to this issue or treat them separately:

Enum support
JSONB support
Array support
Building out audit logs

@wz1000 do take a look at https://github.com/vacationlabs/haskell-webapps/blob/master/ServantOpaleye/db/schema.sql and https://vacationlabs.github.io/haskell-webapps/ServantOpaleye/db/schema_spy/

@wz1000 has this been closed by #30


Better formatting for relative timestamps

Uses Django's naturaltime filter, no extra dependencies,
Avoid showing now ago in the series details page,
Uses months/years as measurement units, much easier to understand than
the number of elapsed weeks.

Thanks, I'll queue all these to next branch (and staged on next.patchew.org) which will come soon to master.


[Microsoft.Billing] Add billing instructions API.

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

Contribution checklist:

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

ARM API Review Checklist

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

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

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

 Can you please approve this PR?

@anthony-c-martin , you requested the changes. Can you please approve this PR?

Oops, I've approved


Can't find the dilation rate setting in Backbone

hi!your work is so good. but when i check the code that you released, i can't find the dilation rate setting in the backbone,while the paper indicates: "The encoder backbone is adapted
from an ImageNet-pretrained neural network paired with atrous convolution for extracting denser feature maps in its last block. "
Did I get it wrong?
Looking forward to your help.
Good luck.

Check this: https://github.com/facebookresearch/detectron2/blob/14e53b54ab7fec738ba3d437074d17db5fb6cee9/projects/Panoptic-DeepLab/configs/Cityscapes-PanopticSegmentation/panoptic_deeplab_R_52_os16_mg124_poly_90k_bs32_crop_512_1024.yaml#L11
It is only supported in the Detectron2 version.


setup gmail forwarding rule

<EMAIL_ADDRESS>should forward to personal email

please use<EMAIL_ADDRESS>for my forwarding address

Can each of you please paste your preferred email address on this thread?

<EMAIL_ADDRESS>please!

<EMAIL_ADDRESS>


[Loading] Add a loading screen inbetween playlist and parameter screen

Loading tasks:

[ ] Presenter
[ ] Include Audio player (with Elevator music 🎵)
[ ] Logic/Algorithm

[ ] Implement API call (Get playlist)
[ ] Filter from parameters

[ ] Set generated in model
[ ] Go to next screen (navigation)

Refer to #7, might need to add functionality to auto-play from presenter (since it is not allowed to auto-play on render).

Skeleton code exists and previous todos have been cleaned up (commit 66f9d54)


Toggle service management

We manage these services elsewhere, so toggling them here is desirable.

Thanks for the contribution!

Oops, sorry for the missing default. Thanks for merging. :)

@adamcstephens I've released a new version with this included. Thanks!


Remove finalizer to avoid finalize loop in GitHubSource FinalizeKind().

Fixes #1058

Proposed Changes
-🐛return EventTypeNormal to remove finalizer when Secret is not found instead of plain err.
-🐛return EventTypeNormal to remove finalizer when 404 is returned instead of plain err.
-🧽change the event message to describe the behavior.

/ok-to-test

/lgtm

/lgtm
/approve


Add event delay metric on both producers

This adds a new header on each producer record, timestamp, which stores the time when the record was produced.
It also emits a new metric, consumer.messages.delay, which is the delay in milliseconds it took to consume a record.

You're right. I missed that while digging through the docs :(
https://github.com/elastic/apm-queue/pull/222


Incorrect locale no_NO

Correct me if i'm wrong but there is no such locale no_NO. According to Unicode.org (CLDR) there is nn_NO and nb_NO.

Norwegian here. Can confirm.

Can you tell me if I should rename the current no_NO to either one of the two locales you mentioned, or remove it altogether?

Good question. For one of my own projects I have chosen 'nb_NO', but in fact both are equally valid. In the same way 'en_US' and 'en_GB' are both valid locales (albeit in different regions). After doing a bit more research (https://en.wikipedia.org/wiki/Norwegian_language) it looks like 'nb_NO' is the main locale, therefor I opted for that one.
Maybe another expert can shed more lights on this :)

'nb_NO' is the main locale, indeed. I would've chosen 'nb_NO', but I'm pretty sure everything is valid 'nn_NO' as well.


Checkout-Java-SDK

Hello, Steve Coffey!
What is the difference between PayPal Orders API and Payments API from Checkout-Java-SDK?
Which one to use if I want to use authentications?
It is seems that Orders API not allows doing void of authorization. It feels like Order API is for use cases where you want to capture instantly and Payments API is for use cases when you want to use authorization, but it's not clear for me yet, am I right?
Orders API
https://developer.paypal.com/docs/api/orders/v2/
Payments API
https://developer.paypal.com/docs/api/payments/v2/

Hey @MarErm27, closing this as not relevant to this package


Make KnownDuration a data class

WHAT
Make KnownDuration a data class
WHY
Want equality for StateFlow updates.
HOW
Checklist :clipboard:

[ ] Add explicit visibility modifier and explicit return types for public declarations
[ ] Run spotless check
[ ] Run tests
[ ] Update metalava's signature text files

That sounds familiar, but without this the position keeps changing evey second even when paused.

for future reference: https://github.com/google/horologist/blob/bdb1f95100c1aa6ac747674fc397072e24efa074/media-data/src/test/java/com/google/android/horologist/media/data/repository/PlayerRepositoryImplTest.kt#L948

data classes can be bad for public APIs, but if I was holding a hard line about that, I think I've softened now.
https://jakewharton.com/public-api-challenges-in-kotlin/

Let's revert if we remember a reason we still hold.


Defer support

Implements support for experimental @defer usage.
This uses the raw response object to write to and end the stream. There isn't an obvious (to me) API provided by Koa for multi-part responses like this.
In order to try this out, users must install the graphql@17 canary build into their AS4 project like so:
npm i --legacy-peer-deps<EMAIL_ADDRESS>

Two things:

You should add ctx.respond = false which tells Koa not to respond since you're doing the low-level work. Otherwise once this function returns Koa will try to do res.write("OK"). https://github.com/koajs/koa/blob/master/docs/api/context.md#ctxrespond
Also, a conditional flush invocation should be added which will ensure if a compression middleware [gzip, brotli] is used then the data stream won't be buffered. This is done in the core Apollo express middleware: https://github.com/apollographql/apollo-server/blob/a69580565dadad69de701da84092e89d0fddfa00/packages/server/src/express4/index.ts#L96-L105

Otherwise I can confirm that this works great in initial testing. I was shocked to see that this feature wasn't already supported.

@laverdet if you can confirm that the approach including your suggestions is kosher / correct for koa I'd be happy for you to make a new PR off of my branch. I'll be offline for the week.
The reason this PR is stalled is because there might be a more correct approach with using actual streams that I didn't figure out or have the time to continue investigating.
But I suppose that's maybe an implementation that we could always cut over to so long as tests are passing.
In any case feel free to push this work forward if you feel inclined to!

Superseded by #85