Bugs in evaluator

When doing the ind2txt, we will get the string:

Then if we calculate the n-gram, it will get the character granularity of unique n-gram:

Example:

Correct:

code result:

Hi, @icedpanda, sorry for late reply. Thanks for your code, we will fix it soon.
I find the same phenomenon about the metric dist@k. After discussing with the author of KGSF, we think that it is a normal phenomenon. With the increase of training epochs, the metric will quickly increase too. You don't need to worry about that.


Need a better summary

The following are mandatory:
scores : [ ... ]
bot_names : [ ... ]
this_bot: <this-bot's-name> | null
map : "map"
iters : #
winner : ""

this_bot is null if the game is among real bots. If this game is being run for an online submission against sample_bots only, then the value is <this-bot's-name>

What more?

Ok, since our engine is still in py2.7, I can't seamlessly call gameloop from the server. It has to be via Subprocess. So summary will be written to arena-logs dir and not be returned by gameloop


Wallet - SSN field is not filled after user hits back button

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:  <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: cannot check production
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/31401
Action Performed:

Go to staging.new.expensify.com
Login with an expensifail account.
Go to Settings > Wallet > Enable Wallet.
Follow the flow until you reach Additional details page.
Fill the form with following data: First name: Alberta, Last name: Bobbeth, SSN: 3333, Any other information for the rest of the fields.
Tap "Save & continue".
Tap on the back button.
8 Scroll down.

Expected Result:
SSN field should be filled with the information previously entered.
Actual Result:
SSN field is empty.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/540cb8f3-a1a9-497e-b2bc-07eda77f457b
View all open jobs on GitHub

It looks like this is happening because we don't have the shouldSaveDraft enabled on the ssn input field.

This doesn't need to be a blocker, and it looks like this could also be happening on the PROD. Also, the issue seems to have existed for quite some time now.

Interesting, thanks for the context. Closing it.


Pechakucha

Here's how I wanna format it:

Start of move in day
Massive rain after Edwards presentation
My miserable first day, not having my ID or basically much of anything
The grind of waking up at 7 AM every morning
My first impressions of classes and formatting your time
First foray into exploring the library
First foray into exploring campus
Dog Atle
My yoyo and how much I've used it
My cozy spot in the library, despite knowing I won't have it nearly as much in the Fall
The many, many daily writings and reflections I had to do for my classes
Tie dye and how stained my hands got
Dining hall (hit or miss)
Getting vertigo and missing a bunch of classes
Fire alarm at 6 in the morning
Butterdog in Kahoot
Dorm life!
The lounges (and how crowded they were)
The memories I made, having campus basically to ourselves is super chill
Compsci classes and bugfixing

great work! Next step is to think through the order and start gathering images.
Also I made it a checklist for you so you can track progress


⚠ Nextcloud has degraded performance

In 660033d, Nextcloud ($SERVER_BASE/nextcloud/) experienced degraded performance:

HTTP code: 200
Response time: 27329 ms

Resolved: Nextcloud performance has improved in bc13824 after 9 minutes.


fix package.json require

[x] Update package.json require
[x] Remove css styling

@kregensberg @EnFinlay please review.
This is good, but should be tested on your computers before deploying as this is a public release.


Allow developers to exclude specific REST API endpoints from access log

As of today, AccessLogSupport doesn't allow us to exclude any HTTP endpoints. External system in our application calls microservice /metrics endpoint every few minutes. The access log is after a while full of irrelevant log messages.
We'd like to exclude /metrics endpoint from our access log. The support can be written in a generic way to allow us to exclude paths matching one of the exclude regular expressions.
It's also interesting that our access log file is full of /metrics endpoints, but there is no /health endpoint. The /health endpoint is called every few minutes as well. Helidon should log either all internal API calls (with internal I mean metrics / health / etc.) or none of them.
Environment Details

Helidon Version: 2.3.3
Helidon MP
JDK version: JDK 17
OS: N/A

@mhorejsi I'm taking a look at your RFE which seems very reasonable. As for the difference between the access log for health and metrics, I don't see that. Perhaps something in your app or the version of Helidon that you are using. Try the latest Helidon or provide more details about your app for further analysis.

@mhorejsi Please see if the additional config property would work for you in PR #3951

Looks good, this should satisfy our requirement. Thanks for implementing it!

As for the difference between the access log for health and metrics, I don't see that. Perhaps something in your app or the version of Helidon that you are using. Try the latest Helidon or provide more details about your app for further analysis.

We don't do anything specific with health / metrics. If upgrade to latest Helidon won't help, I'll open another issue and attach reproducible sample application.

@mhorejsi Roger on issue and sample app

Solved by PR #3951


ClippingSubsampler: fast clipping that doesn't cross cut boundries

TODO:

[ ] add FFProbe subsampler or something like that (extract info, height width etc. KEYFRAME TIMESTAMPS)
[ ] in either clipping or cut detection subsampler exchange clips for the keyframe-aware clips
[ ] do imprecise clipping


Prevent Upstart post-start stanza from hanging

Once the job has failed and is respawned, the status becomes docker respawn/post-start after subsequent failures (as opposed to docker stop/post-start), so the post-start script needs to take this into account.
I could not find specific documentation on the job transitioning to the respawn/post-start state, but this was observed on Ubuntu 14.04.2.

I can confirm that this improves the service start behavior when there is an error. Without this, if the daemon crashes during startup, it stays in the retry loop waiting for docker.sock. For example "sudo service docker start" never returns. With this change, it retries a few time and then reports the service failed to start. So LGTM.

LGTM

ping @tianon

Nice, LGTM
This probably fixes #11167 too, right?

yep i would presume so, good find
On Tue, Apr 14, 2015 at 4:08 PM, Tianon Gravi<EMAIL_ADDRESS>wrote:

Nice, LGTM
This probably fixes #11167 https://github.com/docker/docker/issues/11167
too, right?
—
Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/11945#issuecomment-93105658.


PersistNoLog.hpp causes warnings on g++ 8

If I compile the current Derecho codebase using g++ 8.2, the compiler spews a bunch of warnings every time it encounters PersistNoLog.hpp. They all look like this:
In file included from /home/edward/eclipse-cdt-workspace/derecho-unified/persistent/Persistent.hpp:10,
                 from /home/edward/eclipse-cdt-workspace/derecho-unified/derecho/replicated.h:22,
                 from /home/edward/eclipse-cdt-workspace/derecho-unified/derecho/view_manager.cpp:12:
/home/edward/eclipse-cdt-workspace/derecho-unified/persistent/PersistNoLog.hpp: In function ‘void persistent::saveNoLogObjectInFile(ObjectType&, const char*) [with ObjectType = derecho::View; persistent::StorageType storageType = (persistent::StorageType)0]’:
/home/edward/eclipse-cdt-workspace/derecho-unified/persistent/PersistNoLog.hpp:35:26: warning: ‘.tmp’ directive writing 4 bytes into a region of size between 1 and 256 [-Wformat-overflow=]
     sprintf(tmpfilepath, "%s.tmp", filepath);
                          ^~~~~~~~
In file included from /usr/include/stdio.h:873,
                 from /usr/include/c++/8/cstdio:42,
                 from /usr/include/c++/8/ext/string_conversions.h:43,
                 from /usr/include/c++/8/bits/basic_string.h:6391,
                 from /usr/include/c++/8/string:52,
                 from /usr/include/c++/8/stdexcept:39,
                 from /usr/include/c++/8/array:39,
                 from /usr/include/c++/8/tuple:39,
                 from /home/edward/eclipse-cdt-workspace/derecho-unified/derecho/view_manager.cpp:8:
/usr/include/x86_64-linux-gnu/bits/stdio2.h:36:34: note: ‘__builtin___sprintf_chk’ output between 5 and 260 bytes into a destination of size 256
   return __builtin___sprintf_chk (__s, __USE_FORTIFY_LEVEL - 1,
          ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       __bos (__s), __fmt, __va_arg_pack ());
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /home/edward/eclipse-cdt-workspace/derecho-unified/persistent/Persistent.hpp:10,
                 from /home/edward/eclipse-cdt-workspace/derecho-unified/derecho/replicated.h:22,
                 from /home/edward/eclipse-cdt-workspace/derecho-unified/derecho/view_manager.cpp:12:
/home/edward/eclipse-cdt-workspace/derecho-unified/persistent/PersistNoLog.hpp: In function ‘void persistent::saveNoLogObjectInFile(ObjectType&, const char*) [with ObjectType = derecho::RaggedTrim; persistent::StorageType storageType = (persistent::StorageType)0]’:
/home/edward/eclipse-cdt-workspace/derecho-unified/persistent/PersistNoLog.hpp:35:26: warning: ‘.tmp’ directive writing 4 bytes into a region of size between 1 and 256 [-Wformat-overflow=]
     sprintf(tmpfilepath, "%s.tmp", filepath);
                          ^~~~~~~~
In file included from /usr/include/stdio.h:873,
                 from /usr/include/c++/8/cstdio:42,
                 from /usr/include/c++/8/ext/string_conversions.h:43,
                 from /usr/include/c++/8/bits/basic_string.h:6391,
                 from /usr/include/c++/8/string:52,
                 from /usr/include/c++/8/stdexcept:39,
                 from /usr/include/c++/8/array:39,
                 from /usr/include/c++/8/tuple:39,
                 from /home/edward/eclipse-cdt-workspace/derecho-unified/derecho/view_manager.cpp:8:
/usr/include/x86_64-linux-gnu/bits/stdio2.h:36:34: note: ‘__builtin___sprintf_chk’ output between 5 and 260 bytes into a destination of size 256
   return __builtin___sprintf_chk (__s, __USE_FORTIFY_LEVEL - 1,
          ~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       __bos (__s), __fmt, __va_arg_pack ());
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I'm guessing this is because the newest GCC is stricter about checking for unsafe string operations than previous versions. Weijia, can you double-check your string operations to see if these warnings are warranted, and fix the bug if they are?

How about starting by fixing the part about writing 260 bytes into a region of size 256?


Styling-fixes-for-ui-frameworks

In addition to the phone input issue that we already address in this PR, we have an issue with the social buttons. In dev panel our social buttons look like this:

In playground they look like this:

=> in playground they are centered vertically, but in dev panel they are not (we had to apply a custom style customization to fix this: padding-bottom: .1rem;)
=> the question here is: why do we need the style customization in the dev panel and how can we avoid it?


Unable to Sign In using GitHub Pull Request Nightly Build

Extension version: 1.66.0 (GitHub.vscode-pull-request-github-insiders)
VSCode Version: 1.32.0-insiders (commit: 507312a3e3b34b084b467dfd983263bc7c9d87e0)
OS: Windows 10

Steps to Reproduce:

Run "GitHub Pull Request: Sign out of github.com"
Install GitHub Pull Request Nightly Build (GitHub.vscode-pull-request-github-insiders)
Run "GitHub Pull Request: Sing in to github.com"
Click Continue button in browser window

As result you'll see
Oh no! An error occurred!
Please restart the sign in process from the editor.
Forbidden

As workaround it is possible to sign in using regular extension version, and then install Nightly Build

Looks like this case is working now.
I was able to successfully run "GitHub Pull Requests: Sign in to GitHub" using VS Code Insiders, GitHub Pull Request Nightly Build, and MS Edge (Dev Channel).


settings: use Version.props

Summary of the Pull Request
We don't need to specify version in multiple places again anymore.
Validation Steps Performed

Rebuild settings, launch them

Even is a simple change, since it's not strictly needed for 0.20, we can wait.

we should create a global chore for this if others need to have this mod as well

closing this in favor of #5345


Fix #295 (terra-base devDependency clean up)

Fix #295

@tatchi You should add yourself to the contributors list 😄 https://github.com/cerner/terra-framework/blob/master/CONTRIBUTORS.md

@tatchi You should add yourself to the contributors list 😄 https://github.com/cerner/terra-framework/blob/master/CONTRIBUTORS.md

Done :tada: :tada:


major simulator update

do not publish sensor combined directly anymore
copy sensor data to memory so sensor drivers can read
removed gyro sleeping interval on reading because driver should have timing
removed publishing of sensor topics in simulator which was used before
to keep the attitude estimator happy

@mcharleb Could you do a quick review please?


Webdiff Runner

Description by Korbit AI
What change is being made?
Add a new WebDiff feature to the codebase, including a WebDiffRunner class for executing AST diffs via command line, and several GUI components for visualizing differences between directories, commits, and pull requests.
Why are these changes being made?
These changes introduce a new feature that allows users to visualize code differences using a web interface, enhancing the usability of the tool for analyzing code changes. The WebDiffRunner provides a command-line interface for flexibility, while the GUI components offer a user-friendly way to view and interact with the differences, making it easier to understand code changes and refactorings.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

wtd:summary


[FEATURE] Subscription Contracts

1. Is your feature request related to a problem? Please describe.

We want to realize subscription contracts for tokens on the blockchain.

Requirements

The "holder" of S-token is obligated to deposit the payment to the "payment agent" on the subscription payment date.
If the S-token "holder" makes the payment by the due date, the "issuer" of the S-token will give the holder reward tokens.
If the S-token "holder" fails to make the payment by the due date, the S-token "issuer" will confiscate the S-token.

2. Describe the solution you'd like
Spec

The "Issuer" outsources the collection of subscription fees to the "payment agent". The "payment agent" is obligated to notify the Issuer of the results of the collection. (Off-chain contractual obligations)
The "Issuer" registers the "Payment Agent" with the subscription contract.
The "Payment Agent" records in the subscription contract the holders who have successfully collected the payment and the holders who have failed to collect the payment.

Since there is not much need for this feature at the moment, we will not develop it in this project.


Allow for per node install disk in Terraform Installer

I have different disk types on my nodes (nvme disks on one, sda on others).
This change moves the install_disk parameter from a global variable into the node specific list.
Testing
Deployed with my own cluster.

Terraform object passing for node's name, mac, and domain is not ideal, but at least those are required. For optional parameters, I don't want to have to enumerate those per node, its too cumbersome. For now, I'd rather not support the feature.
You could explore using a map like some of the other fields. Also, feature additions go to both OSes.

OK that's fair. Seems like any node specific settings / options that aren't global should be kept together but I understand. I'll close the PR and run with my fork.

Yes, but then existing users have to now duplicate their disk everywhere (until Terraform supports optional fields in objects with defaulting). Other sparsely used fields use a map to avoid this.
Similar to how snippets are accepted https://typhoon.psdn.io/advanced/customization/ ("On Bare-Metal....")

Ah, yes it would be a breaking change.


Flaky ResourceExhausted in hydra-cluster test

https://github.com/input-output-hk/hydra-poc/actions/runs/3053589859/jobs/4924406067#step:8:102

Let's see whether retrying on ResourceExhausted (#512) helps.


Update back link on Referee Relationship form

Context
As part of the work completed on 3261-references-save-at-end-of-flow
this back link update should have been included.
Changes proposed in this pull request
It will direct candidate back to edit form to prevent duplication of referee.
Guidance to review
Test in review app.
Link to Trello card
https://trello.com/c/T3fMitzm/3261-candidates-can-remove-information-from-references-by-accident
Things to check

[x] This code does not rely on migrations in the same Pull Request
[x] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[x] API release notes have been updated if necessary
[x] This code does not rely on the addition/removal of Azure config environment variables in the same Pull Request
[x] New environment variables have been added to the Azure config

@george-bit this looks good, but there's a couple of other back links which need fixing, it think:

If you go to "Add a referee" > "Continue" > choose one, "Continue" > "Back" > "Back" > "Back" > "Back" – then you get stuck in a loop between the Reference type and name pages.

If you do the same as above but get as far as the "How do you know this referee..." page, and then use the Back links, you end up in a loop between the name page and that page.

The way I've implemented this in the past is to only add the ?return_to=summary query string on the "Change" links on the summary pages, and then if that's not present, always link back to the previous page in the flow.

@frankieroberto Think I've fixed it  🤞
Please have a go in the review app and let me know

@george-bit it's getting closer!
Remaining issues:

In the Add referee flow, if you do "Add another referee" -> "Continue" -> "Back" then you should be back on the interstitial "Choose your referees" page, but instead end up on the review page.

When you do "Add another referee" the interstitial page should have a "Back" link to the review page, at the moment it always has "Back to application". It should still be "Back to the application" if you arrive at this page straight from the application page though via the "Add your references" link (ie when adding the very first reference).

From the "Check your answers before sending your request" page, if you click "Change" on the name, email address or relationship rows, and then click "Back", you end up on the References review page instead of the "Check your answers" page. Clicking the Change link for Reference type and then clicking "Back" works correctly though. Do we need to add a ?return_to=review_unsubmitted query string for the others?

Where adding a new referee, if you do "Add referee" -> "Continue" -> select Academic, press "Continue" -> leave name field blank, press "Continue" then you get a "Back" link to the "What is the referees name?" page, rather than the reference type page. If you also keep pressing "Back" you just remain on the same page.

Sorry! This is quite a complicated flow! Give me a shout if you can't repeat any of these, or if we need to take a call on which ones to fix immediately and which to leave.

There are probably more elegant solutions, bit i've done the work to fix all the backlinks. It definitely needs a review though. I've given it a decent manual test, but it would be good if someone else could try and break it too

Looking good David, much better to separate the two flows out,
Tested a manually and is now working really well. Just two minor comments inline:

When you go back from the name page in the 'new' flow the parameter for the referee type is not retained. I appreciate the difficulty in passing back the param in this specific circumstance only though.
When travelling to the edit pages, submitting the value returns the candidate to the review page if successful. Previously i had some logic implemented here that would redirect the candidate first to any other field which was not filled in and then subsequently to the unsubmitted review page. Appreciate it's simpler in it's current form though.

I don't think either of these area huge problem though and think we could probably merge as is 👍

Hey George

So i think it does for everything, but school-based as it passes it back as an underscore. Good spot!
I might be wrong, but i think generally if you click to edit a field you should always be returned to the page you chose to edit from rather than being directed back into a flow

Wdyt Frankie?

👍

Yea all good. It was just a tiny fix


Blocks don't drop items when broken.

The datapack does not seem to have the loot tables configured for breaking the items. It could be something finicky with a plugin, but thought I should at least make it known.

If you follow the instructions on the readme it works in vanilla. Could you list what modloader (paper/spigot/forge/fabric) you are using, and which mod versions?

It was purpur, and the datapack showed up in the list. I was able to solve the issue by creating a new datapack with specfically the loottables,

Didn't mean to close this, but the server is running purpur, and I was able to solve the issue by creating a new datapack with all the loot tables.

I had this same issue, in my case it was due to datapack loading/definition order. In the level.dat under datapacks/Enabled this datapack was the first element, before both vanilla and fabric. I moved it such that vanilla and fabric load first and now it works. You might have a similar issue. Especially since it shows up first in your datapack list.
In my case I think I might have enabled the datapack while updating the enabled_features list, but tbh I don't know it was a while before I discovered that the blocks didn't drop

Before
After


Add tutorials/examples with more input "ranking" tools

Either tools that produce differentials (estimated log-fold changes) as output, or tools that produce biplots as output.
Tools for which we currently have tutorials

[x] ALDEx2 (TCGA tutorial)
[x] Songbird (Red Sea tutorial)
[x] DEICODE (Sleep Apnea tutorial)
[x] Arbitrary feature loadings (Color composition tutorial)

Tools for which we could otensibly add tutorials

[ ] Corncob (see q2-corncob tutorial here);  we'd need to do one of the following three things to make this work:

modify the q2-qurro plugin to accept this type of data as input
provide a script that will finagle corncob q2 output into a q2 differentials artifact
submit a pr to q2-corncob that outputs results in a differentials artifact

[ ] ANCOM / ANCOM2
[ ] metagenomeSeq
[ ] DEseq2, edgeR, limma, etc.

marking as closed now that #282 merged in


opencomposite: unstable-2024-03-04 -> xrGetSystem-regression-unstable-2024-03-04

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for opencomposite is: Reimplementation of OpenVR, translating calls to OpenXR
meta.homepage for opencomposite is: https://gitlab.com/znixian/OpenOVR
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found xrGetSystem-regression-unstable-2024-03-04 in filename of file in /nix/store/d5fs6vjc5n3gjpsababaqpdcfw5fadqh-opencomposite-xrGetSystem-regression-unstable-2024-03-04

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
opencomposite
opencomposite-helper

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/d5fs6vjc5n3gjpsababaqpdcfw5fadqh-opencomposite-xrGetSystem-regression-unstable-2024-03-04 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A opencomposite https://github.com/r-ryantm/nixpkgs/archive/4c5d02b467bc371c5a5a33ff160fb9ebcf2386f6.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/4c5d02b467bc371c5a5a33ff160fb9ebcf2386f6#opencomposite

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/d5fs6vjc5n3gjpsababaqpdcfw5fadqh-opencomposite-xrGetSystem-regression-unstable-2024-03-04
ls -la /nix/store/d5fs6vjc5n3gjpsababaqpdcfw5fadqh-opencomposite-xrGetSystem-regression-unstable-2024-03-04/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

opencomposite
opencomposite-helper

Maintainer pings
cc @Scrumplex for testing.

This is not a release tag


Make binaries local

There is limited contribution compatibility with windows. Making the binaries local will allow windows developers contribute to this repository.
I have used rimraf to replace rm -rf in the npm scripts in my PR.

Version info

**Angular:**2.0
**Firebase:**3.6.6
**AngularFire:**2.0.0-beta.8
Other (e.g. Ionic/Cordova, Node, browser, operating system):
How to reproduce these conditions
Failing test unit, Plunkr, or JSFiddle demonstrating the problem

Steps to set up and reproduce
Run any of the npm scripts.

Debug output
** Screenshots **
Expected behavior
Commands shouldn't error out.

Actual behavior

We'll also need to find an alternative for the cp command. We would have to add rimraf and the cp alternative to the devDependencies as well. I wonder if it's not just easier to suggest that those of us using Windows to grab cmder.net (or an equivalent)?

@jckeen  https://github.com/sindresorhus/cpy-cli should be a suitable alternative.

@jckeen @cartant In my opinion, it's better to just provide these binaries rather than force people to globally install 3-4 different global packages.
If that ends up being the case, however, the global packages should at least be included in the build documentation.


Issue with RVM?

Hi! Does installing this script have any known effect on existing RVM installation? I had RVM installed on OSX 10.9 earlier, but after installing this script I lost access to RVM.
Is there a way to uninstall this script or is there any uninstall script available? Sorry for Noobish questions.

(English as second language, sorry about the mistakes)
i'm using ubuntu and for reset my default terminal i copy/paste the default .bashrc
this r the steps that i followed:
rm -i ~/.profile ~/.bash*
and replace the content
https://gist.github.com/marioBonales/1637696
i hope that help


Add a noop implementation for ccache for darwin

The library cannot be compiled on MacOS currently because there is no darwin implementation for getClientFromCachedTicket. I'm not sure if there's a good way to actually implement it, but at least to make it compile I copied the "unimplemented" implementation from windows to make it compile on MacOS.

Thanks, I'm not sure either. Don't have access to any MacOS devices so will not look into it but it's good to support compiling on MacOS.


Configuration option to include function headers as part of coverage (as Istanbul does)

Below is a screenshot showing a difference between lcov reports from istanbul and nyc. I am trying to migrate from istanbul but in order to safely do so I would preferably like to see same coverage numbers.
My request is about ability to reproduce istanbul behavior in nyc.

Note, I am not the only one suffering from this issue as proven by this problem reported to Codecov  https://github.com/codecov/support/issues/391
Many thanks for the great tool which is nyc!

@Rush this would appear to relate to some technical differences in how the instrumenter (or potentially the reporter) opts to instrument functions in Istanbul 2.0 vs. Istanbul 1.0. You can find the instrumenter codebase here:
https://github.com/istanbuljs/istanbuljs/tree/master/packages/istanbul-lib-instrument
I note that it does highlight the line if the function is assigned:
const a = function foo () {
  console.log('hello')
}

would love help digging into this further, feel free to join the slack community I've setup if you feel like helping address this issue:
http://devtoolscommunity.herokuapp.com/

https://codecov.io/gh/AirDwing/node-dwing-redis/commit/9e45f7c035ff790c92f8192296a3bd0bd4085a48
Woops!
There was an error processing coverage reports.
Most common issues
coverage.lcov:
TN:
SF:/Users/willin/Documents/dwing/lib/node-dwing-redis/src/index.js
FN:16,createRedisClient
FN:18,(anonymous_1)
FN:33,(anonymous_2)
FN:34,(anonymous_3)
FNF:4
FNH:4
FNDA:7,createRedisClient
FNDA:8,(anonymous_1)
FNDA:1424,(anonymous_2)
FNDA:3,(anonymous_3)
DA:1,3
DA:2,3
DA:3,3
DA:4,3
DA:7,3
DA:9,3
DA:16,3
DA:17,7
DA:18,7
DA:19,8
DA:20,4
DA:22,8
DA:23,5
DA:24,5
DA:31,8
DA:32,8
DA:33,8
DA:34,1424
DA:36,3
DA:43,3
DA:44,3
DA:47,8
DA:49,7
LF:23
LH:23
BRDA:16,0,0,6
BRDA:16,1,0,6
BRDA:19,2,0,4
BRDA:19,2,1,4
BRDA:22,3,0,5
BRDA:22,3,1,3
BRDA:24,4,0,5
BRDA:24,4,1,3
BRDA:36,5,0,3
BRDA:49,6,0,7
BRDA:49,6,1,6
BRF:11
BRH:11
end_of_record

@willin are you seeing the same behavior reported in this pull request (function names not highlighted as instrumented?)


Failed builds because of missing database extensions

Error found in this build:
2021-12-06 11:27:39,646 526 WARNING bc79feda4-d59e-4d71-887a-e13d77d7bddd odoo.addons.base_search_fuzzy.models.trgm_index: To use pg_trgm you have to create the extension pg_trgm in your database or you have to be the superuser.

I have installed pg_trgm on the template1 database so this should now be resolved.
There is currently no way to reinitialize a build (tracked in #34), so you need to push a commit.

Note unaccent and postgis are also preinstalled.

Update: since Odoo is using template0 by default, I had to implement #35 for this to be effective. This is now deployed.


Update tomcat-catalina, tomcat-coyote, ... to 9.0.41

Updates

org.apache.tomcat:tomcat-catalina
org.apache.tomcat:tomcat-coyote
org.apache.tomcat:tomcat-util-scan

from 9.0.40 to 9.0.41.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.tomcat" } ]

labels: library-update, semver-patch

#3997


Action mode activation using "icon" click

Hello,
I would like to reproduce the selection fucntionally that is shown in Gmail:

The action mode is activated If the user either clicks on the icon of a mail either longclick on the item.
When an item is selected in action mode, the icon changes.

I have managed to handle the activation of the actionmode using the longclick, but I don't know how to achieve the rest:

I've added a withItemEvent and an eventhook to the "imageview" on the itemholder, but once I receive the click I don't know how to just activate the action mode and select the item.
I haven't found the way to "change" the image icon when the item it is selected on actionmode.

I have reviewed the documentation, the examples and the issues with no luck :(
Any help would be apreciated.
Thank you!!

@gpulido isn't the MultiSelect Sample more or less what you need? Long click starts the CAB and then you can select.
The only thing you have to add is the fact that also a single click on the "icon" starts the selection.
Basically you would just select your item and then call that: https://github.com/mikepenz/FastAdapter/blob/develop/library-extensions/src/main/java/com/mikepenz/fastadapter_extensions/ActionModeHelper.java#L142

@mikepenz I have reviewed the multiselect example, but I still don't understand which method I need to call to start the "cab" mode. It is probably my fault, but what should I write in the onclick?
 fastAdapter.withItemEvent(new ClickEventHook() {
            @Nullable
            @Override
            public View onBind(@NonNull RecyclerView.ViewHolder viewHolder) {
                if (viewHolder instanceof ContenderItem.ViewHolder)
                    return ((ContenderItem.ViewHolder)viewHolder).avatar;
                return null;
            }

            @Override
            public void onClick(View v, int position, FastAdapter fastAdapter, IItem item) {
               ///????

            }
        });

Also, I want to change the icon of the selected cab objects alongside with the item background. The item background is changed as the actionmodehelpr sets the "item" as selected state so the statedrawable knows that it has to use the selected part. But for the icon that is a imageview as a child of the itemview I don't know how to get the right instance to do the same.

@gpulido well the helper is in theory designed to automatically handle it in the longClick. So as mentioned in my previous message. You have to select that item in the onClick. and afterwards you call that method I linked you to.
If I'm not wrong the selection listener is called so you should be able to notify the adapter about the changed element there. Pass a payload so you know that you just have to change the icon

@mikepenz
Ok 1 of 2 done:
  @Override
            public void onClick(View v, int position, FastAdapter fastAdapter, IItem item) {
                fastAdapter.select(position);
                actionModeHelper.checkActionMode((AppCompatActivity)getActivity());
            }

Now about the listener, probably is a very silly question but the selectionchangedlistener provides me to the item instance, but how I can access the viewholder to change the views?
 fastAdapter.withSelectionListener(new ISelectionListener<ContenderItem>() {
            @Override
            public void onSelectionChanged(ContenderItem item, boolean selected) {
              //I need to access a ImageVIew inside the ViewHolder for the item
            }
        });

@gpulido you never access the view directly in a RV. You never do that :D
You will adjust the item, (if needed) and then you call the notifyAdapterItemSetChanged (hopefully I did not write the name wrong) with a payload which is then calling the bindView with that payload. So you can react at that place

@mikepenz You are absolute right. I knew that the question was silly, sorry for that.
I just added a check on the bindholder of the IItem:
  @Override
    public void bindView(ViewHolder holder, List payloads) {
        super.bindView(holder, payloads);
        holder.bindModel(getModel());
        holder.avatar.setActivated(isSelected());
    }

and just to finish the whole example:
 fastAdapter.withItemEvent(new ClickEventHook() {
            @Nullable
            @Override
            public View onBind(@NonNull RecyclerView.ViewHolder viewHolder) {
                if (viewHolder instanceof ContenderItem.ViewHolder)
                    return ((ContenderItem.ViewHolder)viewHolder).avatar;
                return null;
            }

            @Override
            public void onClick(View v, int position, FastAdapter fastAdapter, IItem item) {
                if (item.isSelected())
                    fastAdapter.deselect(position);
                else
                    fastAdapter.select(position);
                actionModeHelper.checkActionMode((AppCompatActivity)getActivity());
            }
        });
        fastAdapter.withSelectionListener(new ISelectionListener<ContenderItem>() {
            @Override
            public void onSelectionChanged(ContenderItem item, boolean selected) {

               fastAdapter.notifyAdapterItemChanged(fastAdapter.getPosition(item));
            }
        });

Again, thank you for the help and the library!

Glad you made it work :).
As mentioned. Have a look on notifying + a payload as you can just update the specific view :)


Running kubemci delete when mci does not exist is failing

Looking at the logs for https://k8s-testgrid.appspot.com/sig-multicluster-kubemci#kubemci-ingress-conformance, I see kubemci delete is failing with:
error in deleting target HTTPS proxy mci1-tps--echomap: googleapi: Error 404: The resource 'projects/k8s-ingress-boskos-16/global/targetHttpsProxies/mci1-tps--echomap' was not found
This is due to a typo in code: https://github.com/GoogleCloudPlatform/k8s-multicluster-ingress/blob/master/app/kubemci/pkg/gcp/targetproxy/targetproxysyncer.go#L96.
We didnt catch it in out unit tests because our fake never returns NotFound: https://github.com/kubernetes/ingress-gce/blob/f5c87b1611ed3034b5cc795f63b7bcb8915c7365/pkg/loadbalancers/fakes.go#L298.
Steps to fix:

Fix the typo
Update the fake to return NotFound

cc @G-Harmon

Realized there is a 3rd step as well to fetch the latest code from ingress-gce :)
Sent https://github.com/GoogleCloudPlatform/k8s-multicluster-ingress/pull/157 for that.
The first 2 are done


Update outdated Readme

It appears that parts of the readme are outdated and showcase commands that do not work anymore.
To be specific, various sections show examples using the simulator container, which does not exist anymore and was removed with Pull request #175 a few months ago (apparently the simulator is not needed anymore since the go implementation exists now).
The usage examples for the simulator should be removed or updated to show how the go implementation is to be used.

https://github.com/search?q=repo%3Aopiproject%2Fsztp+simulator+language%3AMarkdown&type=code&l=Markdown

Will copy correct commands from https://github.com/opiproject/sztp/blob/main/scripts/tests.sh


Standby mode metric added - new pull request with unit tests

Hi there! Making a new pull request. This time with those unit tests you asked. I hope it helps.

Codecov Report

Merging #191 into master will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #191      +/-   ##
==========================================
+ Coverage   93.77%   93.80%   +0.03%
==========================================
  Files           4        4
  Lines         546      549       +3
==========================================
+ Hits          512      515       +3
  Misses         34       34

Impacted Files
Coverage Δ

vmware_exporter/vmware_exporter.py
93.05% <100.00%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks again for your contribution on the last two PRs.


Network Abstraction

Submitted for initial review.
First commit. All unit tests pass.
The network abstraction is defined in Branch-SDK/Networking/BNCNetworkServiceProtocol.h
A concrete network type is defined in Branch-SDK/Networking/BNCNetworkService.{h|m}
Documentation and more tests need to be added.

Branch-SDK/Networking/BNCNetworkServiceProtocol.h:
https://github.com/BranchMetrics/ios-branch-deep-linking/blob/network-abstraction/Branch-SDK/Branch-SDK/Networking/BNCNetworkServiceProtocol.h
Branch-SDK/Networking/BNCNetworkService.h:
https://github.com/BranchMetrics/ios-branch-deep-linking/blob/network-abstraction/Branch-SDK/Branch-SDK/Networking/BNCNetworkService.h
Branch-SDK/Networking/BNCNetworkService.m:
https://github.com/BranchMetrics/ios-branch-deep-linking/blob/network-abstraction/Branch-SDK/Branch-SDK/Networking/BNCNetworkService.m

Result of Integration 1
Duration: 4 minutes and 53 seconds
Result: Perfect build! All 179 tests passed. :+1:
Test Coverage: 57%

Result of Integration 2
Duration: 4 minutes and 5 seconds
Result: Perfect build! All 180 tests passed. :+1:
Test Coverage: 57%

Result of Integration 3
Duration: 4 minutes and 8 seconds
Result: Perfect build! All 180 tests passed. :+1:
Test Coverage: 57%

Result of Integration 4
Duration: 4 minutes and 6 seconds
Result: Perfect build! All 180 tests passed. :+1:
Test Coverage: 57%

Result of Integration 5
Duration: 3 minutes and 46 seconds
Result: Perfect build! All 180 tests passed. :+1:
Test Coverage: 57%

Result of Integration 6
Duration: 4 minutes and 8 seconds
Result: Perfect build! All 180 tests passed. :+1:
Test Coverage: 57%

Result of Integration 7
Duration: 3 minutes and 58 seconds
Result: All 180 tests passed, but please fix 1 analyzer warning.
Test Coverage: 57%

Result of Integration 8
Duration: 4 minutes and 12 seconds
Result: Perfect build! All 182 tests passed. :+1:
Test Coverage: 59%

Result of Integration 9
Duration: 3 minutes and 55 seconds
Result: Perfect build! All 182 tests passed. :+1:
Test Coverage: 59%

Result of Integration 10
Duration: 3 minutes and 57 seconds
Result: Perfect build! All 182 tests passed. :+1:
Test Coverage: 59%

Result of Integration 11
Duration: 3 minutes and 58 seconds
Result: Perfect build! All 182 tests passed. :+1:
Test Coverage: 59%

Result of Integration 12
Duration: 4 minutes and 1 second
Result: Perfect build! All 182 tests passed. :+1:
Test Coverage: 59%

Result of Integration 13
Duration: 3 minutes and 54 seconds
Result: Perfect build! All 182 tests passed. :+1:
Test Coverage: 59%

Result of Integration 14
Duration: 3 minutes and 54 seconds
Result: Perfect build! All 182 tests passed. :+1:
Test Coverage: 59%

Result of Integration 15
Duration: 4 minutes and 14 seconds
Result: Perfect build! All 182 tests passed. :+1:
Test Coverage: 59%

Result of Integration 16
Duration: 3 minutes and 58 seconds
Result: Perfect build! All 182 tests passed. :+1:
Test Coverage: 60%

Result of Integration 17
Duration: 6 minutes and 22 seconds
Result: Perfect build! All 184 tests passed. :+1:
Test Coverage: 60%


fix(hmi-server): Fixing error on provenance retrieval

Description
• Updating back end to use same terminology as front end.
• Using new typescript generation to keep this file in sync with front end.
• Now when retrieving publications if a 404 is hit we change this to a 204 for our front end.
• Updating exception being used by back end to pass on response code.
• Front end now asks TDS for correct document asset before proceeding.
• Using generated DocumentAsset
Resolves #843

@chris-dickson You don't need to PR this, but, Typescript plugin in action


javascript remarks

xterm.js dev here - just saw your reddit post and ppl being concerned about performance with (yet another) JS driven terminal.
Want to give you a few of my own impressions in that field (and since I dont have a reddit account I'll do it here, hope you dont mind):
As you already might have figured, xterm.js has taken some lengths to optimize execution performance (and still does, partially even with WASM drop-ins), be it for input processing, buffer handling or screen render performance (3 different renderers - sounds silly when just looking at this number). So whats the deal here?
Well since the invention of JIT and the constant improvements done to JIT engines like V8 Javascript isn't slow anymore per se - it is now more a mixed case ranging from 1,5 - 5x execution time for typical programming tasks. Where your very own code ends up in that range depends alot on how well you abide to assumption/pattern the JIT can easily optimize (to name a few we had to apply to xterm.js):

many JS-ish pattern are still poison for the JIT (got somewhat better over time), so Javascript is its own enemy when it comes to perf requirements
short functions / methods with stable type interfaces - any interface change or even accessing a value on an outer context might cause a bailout/de-opt in the JIT with bad perf again or the need to recompile
sync vs async (eg. promises) - promises are way slower that plain sync code (async always involves some additional bookkeeping for the queue execution / stack reordering)
the garbage collector is "garbage" for high perf needs - you will need object pooling/recycling or GC/allocation pressure will kill the fun
perfwise JS strings are spoiled, they have seen some optimizations at various ends, that will bite you back at other ends, resort to typed arrays instead
any data processing in JS is still worse than in high perf compiled languages even with typed arrays, prolly mainly due its deref needs when accessing values in container types (thats where even the JITed execution loses ground) - here WASM can bring you closer to C/C++ speed with 1-1.5x execution time

So when you read over the points above you might get the feeling whats going on - writing very C-ish code in Javascript/Typescript still gives you the fastest results before entering any WASM trickery. This is contradictory to what most webdevs learn these days, esp. regarding the GC (most ppl dont care for memory management at all anymore) with all the downsides - JS apps are perceived as fat and slow. And sadly thats true for most apps. But not because the language/engines cannot do any better - I tend to blame the devs, that dont learn anymore, how to write efficient code due to all the goodies the languages offers (which all come to a price). A quite infamous example is the runtime difference of a for-of-loop vs. a classical C-ish for-loop with index access.

@jerch thanks a lot for adding colors here! Great points and can't agree more. I still need to do a pass at profiling and removing  bottlenecks created by my code here.
And kudos for the work on xterm.js - it's the GOAT! I was absolutely amazed how buttery smooth it is, especially seeing even some pure Rust implementations struggle to achieve the same.


Add filterK to Enumeratee

Add a filterK method to Enumeratee that allows using monadic predicate for filtering elements.
See gitter discussion.

Current coverage is 100.00%

Merging #45 into master will not affect coverage as of 340b75f

@@            master     #45   diff @@
======================================
  Files           17      17
  Stmts          435     437     +2
  Branches        37      37
  Methods          0       0
======================================
+ Hit            435     437     +2
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 340b75f

Powered by Codecov. Updated on successful CI builds.

This is great, thanks @liff! I'll update the snapshot this evening when I'm back at my desktop.


The "Discount Amount" field must be 0.0001 or more, but getting warning for this amount

Framework Version: 0.1.6

Framework Version: 0.1.6

Fixed


add downloaded shapefiles to gitignore

the idea is to just download and unpack zipped shapefiles. not to commit them.

That's great, thanks!


Unable to load shared library OpenCvSharpExtern, Ubuntu  22.04

Summary of your issue
I'm trying OpenCvSharp on Ubuntu 22.04 and I'm getting following error
 System.DllNotFoundException : Unable to load shared library 'OpenCvSharpExtern' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: libOpenCvSharpExtern: cannot open shared object file: No such file or directory

I'm referencing OpenCvSharp4.runtime.ubuntu.18.04-x64 package.
Environment
Ubuntu 22.04
What did you do when you faced the problem?
libOpenCvSharpExtern is located in build folder under ./bin/Debug/net6.0/runtimes/ubuntu.18.04-x64/native/ instead ./bin/Debug/net6.0/. So running  ldd libOpenCvSharpExtern.so returns shell-init: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory
Then I copied libOpenCvSharpExtern.so to ./bin/Debug/net6.0/ and try to run application from there resulting in same : System.DllNotFoundException : Unable to load shared library 'OpenCvSharpExtern' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: libOpenCvSharpExtern: cannot open shared object file: No such file or directory  exception.
Then I run  ldd libOpenCvSharpExtern.so again resulting in :
❯ ldd libOpenCvSharpExtern.so
        linux-vdso.so.1 (0x00007ffc407f8000)
        libtesseract.so.4 => /lib/x86_64-linux-gnu/libtesseract.so.4 (0x00007f5be46f6000)
        libgtk-x11-2.0.so.0 => /lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0 (0x00007f5be427f000)
        libgdk-x11-2.0.so.0 => /lib/x86_64-linux-gnu/libgdk-x11-2.0.so.0 (0x00007f5be89f9000)
        libcairo.so.2 => /lib/x86_64-linux-gnu/libcairo.so.2 (0x00007f5be4157000)
        libgdk_pixbuf-2.0.so.0 => /lib/x86_64-linux-gnu/libgdk_pixbuf-2.0.so.0 (0x00007f5be89c9000)
        libgobject-2.0.so.0 => /lib/x86_64-linux-gnu/libgobject-2.0.so.0 (0x00007f5be8967000)
        libglib-2.0.so.0 => /lib/x86_64-linux-gnu/libglib-2.0.so.0 (0x00007f5be401d000)
        libdc1394.so.22 => not found
        libavcodec.so.57 => not found
        libavformat.so.57 => not found
        libavutil.so.55 => not found
        libswscale.so.4 => not found
        libjpeg.so.8 => /lib/x86_64-linux-gnu/libjpeg.so.8 (0x00007f5be3f9c000)
......

So, I have few questions like:

can we have proper Ubuntu 22.04 nuget package ?
list of required Ubuntu packages that we need to install before using OpenCvSharp ?
fix deploy script so we have libOpenCvSharpExtern.so in proper location ?

I would like to use OpenCvSharp but this is blocking me for using it.
Example code:

	byte[] image = new ();// not important
	Mat src = Cv2.ImDecode( image, ImreadModes.Color );

Output:

 Error Message:
   System.TypeInitializationException : The type initializer for 'OpenCvSharp.Internal.NativeMethods' threw an exception.
  ----> System.DllNotFoundException : Unable to load shared library 'OpenCvSharpExtern' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: libOpenCvSharpExtern: cannot open shared object file: No such file or directory

What did you intend to be?
Compile and run.

Any suggestions to fix that? I have the same error.  env: ubuntu 22 and .net6

I have been trying to get a solution to this one too. I've been following through the steps in the github workflows and am in the process of trying to get it compiled on Ubuntu 22.04 but I'm getting stuck as well.
I am working on a pull request to allow them to publish this runtime package.
A brief rundown of where I'm at, I added .github/workflows/ubuntu22.yml with the following changes

Runs on ubuntu-22.04 from ubuntu-22.04
libswresample-dev used instead of the deprecated package libawresample-dev
libdc1394-dev used instead of the deprecated libdc1394-22-dev

At this point I'm out of ideas on what to run next. I'm running into issues of the libavresample.so package which is not currently built for Ubuntu 22 Jelly Jam and the replacement that I thought could be a good alternative libswresample-dev apparently isn't quite the right fit for a quick swap which doesn't seem to be the case
[ 88%] Building CXX object OpenCvSharpExtern/CMakeFiles/OpenCvSharpExtern.dir/wechat_qrcode.cpp.o
[ 91%] Building CXX object OpenCvSharpExtern/CMakeFiles/OpenCvSharpExtern.dir/xfeatures2d.cpp.o
[ 94%] Building CXX object OpenCvSharpExtern/CMakeFiles/OpenCvSharpExtern.dir/ximgproc.cpp.o
make[2]: *** No rule to make target '/usr/lib/x86_64-linux-gnu/libavresample.so', needed by 'OpenCvSharpExtern/libOpenCvSharpExtern.so'.  Stop.
make[2]: *** Waiting for unfinished jobs....
[ 97%] Building CXX object OpenCvSharpExtern/CMakeFiles/OpenCvSharpExtern.dir/xphoto.cpp.o
make[1]: *** [CMakeFiles/Makefile2:98: OpenCvSharpExtern/CMakeFiles/OpenCvSharpExtern.dir/all] Error 2
make: *** [Makefile:136: all] Error 2
Error: Process completed with exit code 2.

I also found this stack overflow about the difference between avresample and swresample.

@Evelios How far did you get?
I've made two NuGet packages for internal use that contain binaries for Ubuntu 20.04 and Ubuntu 22.04. I'm now working on dockerizing the creation of both. I could then turn that into a PR, would you be interested?

Is there any reason that the nuget packages for 22.04 and 20.04 can't be uploaded to nuget? The master branch builds for 20.04 just fine, i am guessing it hasn't been thoroughly tested?
If I am able to build for that target, is there a licence restriction that stop me from uploading the 20.04 version for a specific version of OpenCV (say... 4.6.0) to the nuget repository for example?

@lewispurvis if you manage to make these two packages part of the general release procedure, it'd be really great! I'd much rather use these, than my privately maintained ones. I can share the code for building the packages, if you're interested.

@lejsekt please share the code and make PR. This issue is blocker for me and I hope that PR will be approved. Thanks for your effort.

We just hit the same issue upgrading from .NET Core 3.1 to .NET 6, although we are actually running on Ubuntu 18.04, but we're getting the same results here.

Here's the code @dsoronda
It's just taking the existing Dockerfile and adding a step to build a NuGet package.
package.zip
One can then create the package by:
docker buildx build -f Focal.Dockerfile . -o export
docker buildx build -f Jammy.Dockerfile . -o export

However, concerning Jammy.Dockerfile, I'm just guessing the needed native deps. But the tests are passing and the OpenCV functionality I need in my project seems to work as well.

And similar issue in another repo: https://github.com/sdcb/PaddleSharp/issues/25. It might be an inspiration for how to create the package.

https://github.com/shimat/opencvsharp/issues/1448#issuecomment-1298124942
$ diff -u *.Dockerfile
--- Focal.Dockerfile    2022-10-07 19:50:30.000000000 +0800
+++ Jammy.Dockerfile    2022-10-07 19:52:36.000000000 +0800
@@ -1,4 +1,4 @@
-FROM mcr.microsoft.com/dotnet/aspnet:6.0-focal as builder
+FROM mcr.microsoft.com/dotnet/aspnet:6.0-jammy as builder

 ENV DEBIAN_FRONTEND=noninteractive
 ENV OPENCV_VERSION=4.6.0
@@ -21,7 +21,7 @@
       libavcodec-dev \
       libavformat-dev \
       libswscale-dev \
-      libdc1394-22-dev \
+    #   libdc1394-dev \
       libxine2-dev \
       libv4l-dev \
       libtheora-dev \
@@ -29,7 +29,7 @@
       libxvidcore-dev \
       libopencore-amrnb-dev \
       libopencore-amrwb-dev \
-      libavresample-dev \
+      libswresample-dev \
       x264 \
       libtesseract-dev \
       libgdiplus \
@@ -122,7 +122,7 @@

For anyone who wants to use 4.7.0 that was released last month, @shimat has already created a dockerfile for building libOpenCvSharpExtern.so under Ubuntu 22.04, so we can build it from scratch
wget https://raw.githubusercontent.com/shimat/opencvsharp/<IP_ADDRESS>30114/docker/ubuntu22-dotnet6-opencv4.7.0/Dockerfile
docker buildx build -f Dockerfile . -o export

Wait for it to complete compiles then it will stuck at the last command in dockerfile:
https://github.com/shimat/opencvsharp/blob/7ed8972de415fad623d425e56b4f87ef2503d68b/docker/ubuntu22-dotnet6-opencv4.7.0/Dockerfile#L162
Press Ctrl+C to intercept docker buildx and then cd export/artifacts, finally you should be able to see the libOpenCvSharpExtern.so.
Before using it for your .NET application, you should check out if there are sufficient dependencies had been installed on your environment(out of the building system Ubuntu 22.04 in the docker container):
ldd *.so* | grep found

and install any(not need to install all of them in dockerfile) missing packages to fix these missing dependencies:
https://github.com/shimat/opencvsharp/blob/7ed8972de415fad623d425e56b4f87ef2503d68b/docker/ubuntu22-dotnet6-opencv4.7.0/Dockerfile#L9-L34

Not sure where we are with this here is where I've gotten.

Created docker container for Ubuntu 22.10
Installed software from ubuntu20.yml file. Switching the two deprecated packages. libswresample-dev and libdc1394-dev
Cloned  two dep opencv and opencv_config
Built 3 packages (opencv, opencv_config) and OpenCVSharpExtern
Created libOpenCvSharpExtern.so successfully

Continuing using the ubuntu20.yml I was able to create a custom nupkg package which I used docker cp to get it out of the docker container
Then in my application I used dotnet add package OpenCvSharp4_.runtime.ubuntu.22.10-x64 -s $(pwd) (aka. Path to the custom nupkg) and it installed
But I still get error
Unhandled exception: System.TypeInitializationException: The type initializer for 'OpenCvSharp.Internal.NativeMethods' threw an exception.
 ---> System.DllNotFoundException: Unable to load shared library 'OpenCvSharpExtern' or one of its dependencies.

I'm not sure if I'm missing something or I just don't know what I'm doing. Both are likely true

Where is your built result libOpenCvSharpExtern.so located?
You can just put it under the same path as the main executable file, they run your program with LD_LIBRARY_PATH=. ./executable

@n0099 It's getting installed in the bin/Debug/net7.0/runtimes/ubuntu20.04-x64/native directory
Not sure I follow how that environment variable works. Will try it. I'm currently just using dotnet run

Oh I ran the ldd libOpenCvSharpExtern.so and see that libtesseract.so.4 => not found I think that has something to do with it

use LD_LIBRARY_PATH=bin/debug/net7.0 dotnet run bin/debug/net7.0/app.dll
since your pwd is not in bin/debug/net7.0
or just cd your pwd to it.
if this still fails, try export LD_LIBRARY_PATH=path; dotnet run path/app.dll


Slack Username/Avatar spoofing?

Environment

Version:
1.13.0

Operating system:
Windows 10 64-bit

Please describe the expected behavior.
slack username spoofing working like in discord, by adding WebhookURL to the config
Please describe the actual behavior.

messages on slack appear to be sent from the bot user
Any steps to reproduce the behavior?
set up a discord<-->slack bridge with both discord and slack having a webhookurl config value
Please add your configuration file

matterbridge_anonimized.txt

Hello @laundmo,
I am not entirely sure what you are referring to but the Slack bridge should be replacing the bot's username with a string that follows the RemoteNickFormat that you specify in the bridge's configuration.
There will still though be a bot marker next to it but that is how Slack Apps and their bots work and has nothing to do with Matterbridge. 🙂
If I misinterpreted your question do correct me. I would very much like to help you get what you want (in as far as it is technically possible).

im just gonna share some screenshots of me testing it, maybe that will clarify what i meant:

i expected slack username to be laundmo@discord|->

Ok. I understand better now. This is unfortunately nothing that can be changed as it is due to Slack's permissions model. Webhook based message posting does unfortunately not allow one to modify the name of the poster. For this to work you need to use the Slack App / Token setup described on the wiki.
Let me know if you have any more questions with which I can help. 🙂

figured it out
fix: remove PrefixMessagesWithNick=true from salck config

Just so we can file a bug (if this was one), can you share if you also changed to the api token model (instead of slack webhooks)? Would be great if we could improve the docs or defaults and prevent the next person from getting confused by this :)

ah, i was using that from the start (as can be seen from the text file attached to the original issue)
i just added a webhook additionally, like you need to do to make spoffing for discord work.


Rancher returns 404 from agent when importing a non-RKE cluster

https://github.com/rancher/rancher/issues/15172
In Rancher v2.5.5 when importing a non-RKE cluster we see two requests returning a 404 when the cluster is registering.

Single node docker install behind nginx reverse proxy
Create a GKE cluster in Google cloud and import into Rancher
404 is only seen twice, immediately after registering, from a single node of the 3-node cluster, over a period of 2 hours

node1 - - [28/Jan/2021:22:44:39 +0000] "GET /v3/connect/config HTTP/2.0" 404 0 "-" "Go-http-client/2.0"
node1 - - [28/Jan/2021:22:44:48 +0000] "GET /v3/connect/config HTTP/2.0" 404 0 "-" "Go-http-client/2.0"

Previously we saw these repeating consistently, now it appears these two requests are made and not seen again, but we should not be seeing these requests at all.

Does this affect performance or usage or are these just spurious errors as the cluster is registering?


Unraid Application

Is your feature request related to a problem? Please describe.
Have the ability to easily install the application on Unraid Docker
Describe the solution you'd like
Build a template and publish it on the Unraid CA store
Describe alternatives you've considered
Manual deployments in Podman on a VM / Docker CLI, but not fun and it isn't persisted.
Additional context
Could be very useful to also bring more community attention / contributions to the project.

Thanks for the suggestion! I'm going to mark this as "help wanted" for now, since (if I'm reading correctly) getting the necessary generated config to submit your app for approval to the CA suggests that you must use Unraid (https://forums.unraid.net/topic/57181-docker-faq/#comment-566084). That said, I'm sure there are plenty of examples out there that we could cargo cult off of; but we'd still appreciate any help here!

I write a lot of Unraid templates, and came here specifically to figure out how to set up this container to write a template for it. Running into a Node issue inside the container following the dev compose file. If you have a better compose stack that I can follow, I can translate that into an Unraid template and get this on the Unraid CA store immediately.

Thanks! That dev compose is a WIP (and it's meant for running the dev app in a local container for debugging, etc).
The compose file in the docs is probably closer to what you want: https://tunarr.com/getting-started/setup/
I'm not sure how this translates to Unraid templates, but there are a couple options that are dependent on the user's personal setup (e.g. changing the image / runtime params for hw accel, adding in the dizquetv folder as a volume if they are migrating, exposed port etc):
Here is another example I just tested out locally
version: '3.8'
services:
  tunarr:
    # image: chrisbenincasa/tunarr:edge
    # Uncomment along with runtime below to enable HW accel
    image: chrisbenincasa/tunarr:edge-nvidia
    container_name: tunarr
    ports:
      - 8088:8000
    # Uncomment if using the Nvidia container
    runtime: nvidia
    # volumes:
      # The host path is relative to the location of the compose file
      # This can also use an absolute path.
      #
      # Uncomment if migrating from dizquetv. Chnage the host path
      # to the location of your dizquetv "database"
      # - ./.dizquetv:/.dizquetv

Thanks! That dev compose is a WIP (and it's meant for running the dev app in a local container for debugging, etc).
The compose file in the docs is probably closer to what you want: https://tunarr.com/getting-started/setup/
I'm not sure how this translates to Unraid templates, but there are a couple options that are dependent on the user's personal setup (e.g. changing the image / runtime params for hw accel, adding in the dizquetv folder as a volume if they are migrating, exposed port etc):
Here is another example I just tested out locally
version: '3.8'
services:
  tunarr:
    # image: chrisbenincasa/tunarr:edge
    # Uncomment along with runtime below to enable HW accel
    image: chrisbenincasa/tunarr:edge-nvidia
    container_name: tunarr
    ports:
      - 8088:8000
    # Uncomment if using the Nvidia container
    runtime: nvidia
    # volumes:
      # The host path is relative to the location of the compose file
      # This can also use an absolute path.
      #
      # Uncomment if migrating from dizquetv. Chnage the host path
      # to the location of your dizquetv "database"
      # - ./.dizquetv:/.dizquetv

Thanks, I'll test it out

Got it, needed to permanently mount /tunarr/server/build/.tunarr:
version: '3.8'
services:
  tunarr:
    image: chrisbenincasa/tunarr:edge
    container_name: tunarr
    ports:
      - 8088:8000
    volumes:
      - /mnt/user/appdata/tunarr/dizquetv_migration:/.dizquetv
      - /mnt/user/appdata/tunarr/data:/tunarr/server/build/.tunarr

Drafting up an Unraid template now.

Template is done and tested. Should be live in the Community Apps store in a few hours.

Wow this is awesome! Thanks a ton! Curious, how will this handle different builds? Eventually, we'll have a stable image build (docker tag latest), then also this edge build, and perhaps a "next" build that has semi stable features. Can unraid expose different builds in the template for users to choose from?

It seems the migration isn't working correctly with the current template. No migration is being initiated when being pointed to the dizquetv appdata folder

/mnt/user/appdata/tunarr/dizquetv_migration:/.dizquetv

@chrisbenincasa The template currently maps dizqueTV's parent folder to /.dizqueTV inside the container as read-only. Does it need to be read-write?

No, Tunarr doesn't require write permissions to the .dizquetv directory. The migration is read-only

Then I'm confused about why the migration may not be running.

Having the same issue with the migration. Not seeing that running on my new install as well.

Hmm, perhaps something regressed. I'll try either later today or tomorrow morning. Though I won't be able to test the unraid setup specifically.

Oh one other thing... it could be because the Tunarr database directory exists already. The migration is skipped in that case in order to not overwrite any existing Tunarr data (subsequent runs). Does the unraid setup pre create this directory or something? Maybe we need a better method of tracking when to migrate (a db field).
See the init db directory function in this file.
https://github.com/chrisbenincasa/tunarr/blob/main/server/src/server.ts

Oh one other thing... it could be because the Tunarr database directory exists already. The migration is skipped in that case in order to not overwrite any existing Tunarr data (subsequent runs). Does the unraid setup pre create this directory or something? Maybe we need a better method of tracking when to migrate (a db field).
See the init db directory function in this file. https://github.com/chrisbenincasa/tunarr/blob/main/server/src/server.ts

Yes, Unraid would create any paths that don't already exist, so it would precreate the associated /tunarr/server/build/.tunarr path: https://github.com/nwithan8/unraid_templates/blob/750fa251daa2762a9204824f9a76c969720e73d3/templates/tunarr.xml#L51

Ah thanks for the context. That is most likely it then. Let me file an issue to capture this. This might be a little tricky since we try to the treat the whole directory as "one", even though it includes several files with different parts of the state. But so long as we have a universal marker for "first run" that works in all envs we should be good. I'll think on it.

Opened #474 to track the full solution. It'll be the next thing I tackle.

Fresh install of the container on Unraid does seem to be processing the .dizquetv migration properly now.
(I have some bad settings in my old DizqueTV installation, so migration failed and the web UI didn't load as a result, but the migration DID trigger):

Huh... the legacy migrator doesn't attempt to verify the Plex settings, it'll just blindly migrate them. This is likely coming from some other data fixers that happen afterward. Are there any more details in the logs? Otherwise, can you rerun with debug logs enabled (easiest way is the LOG_LEVEL env var) and see where exactly that is failing?
It's probably the right thing that it failed outright by being unable to reach your configured Plex server, but we should discover why

I had similar results where the migrator started, even processed channels and then Plex through a 404 and things locked up. I am seeing the UI Route is the issue but also seeing that on my DizqueTV container right now as well.

@josway happy to look further if you provide some debug logs -- check out my previous comment on how.

@nwithan8 - did you get your issue resolve? I'm surprised there are locations in the code where a Plex request would cause the server to not start. If you are still having issues I'd love to check out some debug logs.

@nwithan8 - did you get your issue resolve? I'm surprised there are locations in the code where a Plex request would cause the server to not start. If you are still having issues I'd love to check out some debug logs.

I ended up just deleting old/dead/bad Plex servers from the dizqueTV config file. The migration then went smoothly, web UI loaded, everything good.

@chrisbenincasa Same for me as with @nwithan8. Bad Plex Server data hung everything up. Once I fixed that all went smoothly.

Gotcha, thanks! I'll try injecting some bad values and see if I can reproduce.

@nwithan8 think we can call this complete now? Really appreciate the effort here!

Yeah, the Unraid template aspect seems to be working just fine. The migration stuff is unrelated to Unraid, so this ticket can be closed. Let me know if you release a new Docker branch and I'll update the template accordingly. Or feel free to open a PR: https://github.com/nwithan8/unraid_templates/blob/750fa251daa2762a9204824f9a76c969720e73d3/templates/tunarr.xml

Awesome, sounds good! Thanks again!

@nwithan8 looks like Unraid Blacklisted this template. Below is the error message.
Default tag (:latest) does not exist on dockerHub
This application template has been blacklisted.

That's a weird requirement!

I can definitely start publishing the latest tag along with the edge tag. Eventually latest will point to stable releases, though. Just didn't want to confuse anybody by redefining the tag down the road

@nwithan8 looks like Unraid Blacklisted this template. Below is the error message.
Default tag (:latest) does not exist on dockerHub This application template has been blacklisted.

Welp, that's going to be funt to try to get reversed:

I'll reach out to Squid once there is a latest tag and see if I can get that un-blacklisted.

"latest" tag now exists in both Docker hub and ghcr
When the time comes to change what version "latest" points to (when we're more stable) I'll let you know so we can update the template description to mention the difference in the tags.


Add storage-api to docker-compose

It should be a matter of adding the docker image supabase/storage-api:latest in here https://github.com/supabase/cli/blob/main/src/templates/init/docker/docker-compose.yml
and also the storage schema from here to here

Hmm I added support in the main repo, but not here. Main issue is that this setup is not completely local (assumes you have access to a s3 bucket)

This is now supported in the Go CLI.


Use a Dockerfile

I'm hitting an odd issue with package versions when running the setup.py. Obivously, some is messed up my with pip or python3 configuration that I need to debug. However, in general, I'd reccomend creating a dockerfile so that we can avoid environmental issues entirely.
dbworku@pop-os:~/repos/honestnft-shenanigans$ sudo python3 setup.py install
running install
running bdist_egg
running egg_info
writing honestnft_shenanigans.egg-info/PKG-INFO
writing dependency_links to honestnft_shenanigans.egg-info/dependency_links.txt
writing requirements to honestnft_shenanigans.egg-info/requires.txt
writing top-level names to honestnft_shenanigans.egg-info/top_level.txt
reading manifest file 'honestnft_shenanigans.egg-info/SOURCES.txt'
writing manifest file 'honestnft_shenanigans.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
creating build/bdist.linux-x86_64/egg
creating build/bdist.linux-x86_64/egg/fair_drop
copying build/lib/fair_drop/__init__.py -> build/bdist.linux-x86_64/egg/fair_drop
creating build/bdist.linux-x86_64/egg/metadata
copying build/lib/metadata/pulling.py -> build/bdist.linux-x86_64/egg/metadata
copying build/lib/metadata/__init__.py -> build/bdist.linux-x86_64/egg/metadata
copying build/lib/metadata/rarity.py -> build/bdist.linux-x86_64/egg/metadata
copying build/lib/metadata/pull_from_rt.py -> build/bdist.linux-x86_64/egg/metadata
byte-compiling build/bdist.linux-x86_64/egg/fair_drop/__init__.py to __init__.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/metadata/pulling.py to pulling.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/metadata/__init__.py to __init__.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/metadata/rarity.py to rarity.cpython-39.pyc
byte-compiling build/bdist.linux-x86_64/egg/metadata/pull_from_rt.py to pull_from_rt.cpython-39.pyc
creating build/bdist.linux-x86_64/egg/EGG-INFO
copying honestnft_shenanigans.egg-info/PKG-INFO -> build/bdist.linux-x86_64/egg/EGG-INFO
copying honestnft_shenanigans.egg-info/SOURCES.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying honestnft_shenanigans.egg-info/dependency_links.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying honestnft_shenanigans.egg-info/requires.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
copying honestnft_shenanigans.egg-info/top_level.txt -> build/bdist.linux-x86_64/egg/EGG-INFO
zip_safe flag not set; analyzing archive contents...
metadata.__pycache__.pull_from_rt.cpython-39: module references __file__
metadata.__pycache__.pulling.cpython-39: module references __file__
creating 'dist/honestnft_shenanigans-0.0.1-py3.9.egg' and adding 'build/bdist.linux-x86_64/egg' to it
removing 'build/bdist.linux-x86_64/egg' (and everything under it)
Processing honestnft_shenanigans-0.0.1-py3.9.egg
removing '/usr/local/lib/python3.9/dist-packages/honestnft_shenanigans-0.0.1-py3.9.egg' (and everything under it)
creating /usr/local/lib/python3.9/dist-packages/honestnft_shenanigans-0.0.1-py3.9.egg
Extracting honestnft_shenanigans-0.0.1-py3.9.egg to /usr/local/lib/python3.9/dist-packages
honestnft-shenanigans 0.0.1 is already the active version in easy-install.pth

Installed /usr/local/lib/python3.9/dist-packages/honestnft_shenanigans-0.0.1-py3.9.egg
Processing dependencies for honestnft-shenanigans==0.0.1
error: setuptools 52.0.0 is installed but setuptools>=59.6.0 is required by {'kw3', 'web3-wrapped-contract'}

However, when I check the version for setuptools directly. Things look fine.
dbworku@pop-os:~/repos/honestnft-shenanigans$ python3 -m pip show setuptools
Name: setuptools
Version: 60.7.0
Summary: Easily download, build, install, upgrade, and uninstall Python packages
Home-page: https://github.com/pypa/setuptools
Author: Python Packaging Authority
Author-email<EMAIL_ADDRESS>License: UNKNOWN
Location: /home/dbworku/.local/lib/python3.9/site-packages
Requires:
Required-by: jsonschema, kw3, launchpadlib, web3-erc20, web3-wrapped-contract

Hi @dbworku as a temporary workaround, you can do this: from the root of the honestnft repo simply do pip install .


A0 sized pdfs are extremely slow

A0 sized pdfs are extremely slow to display above 110% zoom on the viewer.
Configuration:

Web browser and its version: Chrome/51.0.2704.84
Operating system and its version: windows 10 pro
PDF.js version: 1.4.20

Steps to reproduce the problem:

upload @ viewer
zoom 130% +

What is the expected behavior? Slow rendering
What went wrong? Slow rendering
PDF:
HDR-Ashill-Land-Ltd-PDA-05-Matter-4.pdf
Above file grabbed from online @ https://www.sevenoaks.gov.uk/__data/assets/pdf_file/0005/128651/HDR-Ashill-Land-Ltd-PDA-05-Matter-4.pdf initial problem found using my own pdfs locally, was recreated using the viewer and externally sources pdf.

Update:
Also noticed occasionally when you zoom in beyond 130% and zoom back out, the image flips/mirror images. I've noticed this has been mentioned previously, with no solution, but "it works for me". I'd assume the two issues are connected via the zoom functionality.

Closing since this file renders almost instantly now after optimizations from the last few years.


Outdated frozen.py

The frozen list on the main branch in PySAL seems to be outdated:
https://github.com/pysal/pysal/blob/2580c6c9469991befd8038afa2179599b5dc8e97/pysal/frozen.py
If this is not used in the build anymore, it might be worth removing it?

Good catch. I need to automate the rebuilding of frozen.py

Addressing in #1243


Need a executable .exe file in a stable folder .

Description of the new feature/enhancement
I am using a application launcher Listary , click ctrl twice , type  a keyword , it would open the keyword related software .
But, terminal doesn't have a executable file, I search for windowsterminal.exe got below which can't be open :

I need a stable path of executable  .exe file so that I can set a keyword to quick open windows terminal in an app launcher.

wt.exe is an execution alias and it is always in a stable location.


[Errno 2] No such file or directory: 'chamfer_cuda.cpp'

After cloning the repo at the same level as a Jupiter notebook I can't seem to use this repo because of the error in the title.
My Code is :```
sys.path.append("ChamferDistancePytorch")
import torch, chamfer3D.dist_chamfer_3D, fscore
chamLoss = chamfer3D.dist_chamfer_3D.chamfer_3DDist()
points1 = torch.rand(32, 1000, 3).cuda()
points2 = torch.rand(32, 2000, 3, requires_grad=True).cuda()
dist1, dist2, idx1, idx2 = chamLoss(points1, points2)
f_score, precision, recall = fscore.fscore(dist1, dist2)

Hi @SysOverdrive
I am not sure what you mean by because of the error in the title. Can you be more specific?
Did the tests run on your end?
Sincerely,
Thibault

Hi ThibaultGROUEIX so first I was referring to :

So basically I am using a conda enviroment to install and use the chamfer distance. I have tried running from a VS Build tools consolse/ from conda and other ways.
I am running on a windows machine :)

@SysOverdrive Any luck running on windows ? I have a similar problem

have you run python setup.py install?

On windows, maybe try to replace '/' with  '\' in line 20, 21 in dist_chamfer_3D.py to make sure
"/".join(os.path.abspath(file).split('\')[:-1] + ["chamfer_cuda.cpp"])
is the right path.

"/".join(os.path.abspath(file).split('')[:-1] + ["chamfer_cuda.cpp"]) is the right path.

This works, but then it'll run into OSError: CUDA_HOME environment variable is not set. Please set it to your CUDA install root.


chore: redo typo PR by Dimitrolito

Thanks Dimitrolito for https://github.com/AztecProtocol/aztec-packages/pull/10171. Our policy is to redo typo changes to dissuade metric farming. This is an automated script.

Docs Preview
Hey there! 👋 You can check your preview at https://674eed59bd1f661bd5648895--aztec-docs-dev.netlify.app


Automatic Launchpad Re-Connection on client and server side.

Reconnect the launchpad to the robot without having to restart the robot code.

Launchpad is being replaced with a zero delay joystick controller and buttons.


Genbank location parser breaks on parsing this location.

Describe the bug
Genbank.Read panics when trying to read this specific location line
     misc_RNA        join(complement(5306942..5307394),
                     complement(5304401..5305029),complement(5303328..5303393),
                     complement(5301928..5302004))

Severity of the bug
Keeps us from parsing refseq human genome
Steps to reproduce
Steps to reproduce the behavior:
Will update with better steps but you should be able to create a new genbank file with this location and run it through genbank.Read to reproduce.
Expected behavior
Will not panic and be able to parse location string effectively.
Screenshots

Client information

OS: Mac OSX 0.15.7
Golang version: 1.21
poly version v0.27.1

fixed as of #385

fixed as of #385


Add trailing slash to preview url

I noticed that in testing these actions for our docs the link does not actually show the preview build. Adding a trailing slash fixed this for me.
I wonder why that is not the case for comments on ProjPythia PRs?

Nvmd this was an oversight on my end...


Suggestions for the Plugin API

Important plugins, SpatialEditorPlugin, CanvasItemPlugin, TileMapEditorPlugin, or simply every C++ editors, should have it’s own API for extension. Based on #4174, various plugins can work together like one, but a big problem with this now is the plugins don’t know exactly what the others are doing.
For example creating a addon for filtering(Random tiles or auto-mapping, #843) tiles in a TileMap node is not possible, or not practical, you can handle the input but you don't know what tool the editor is using.

I created a small example for the TileMapEditorPlugin:

TileMapEditorPlugin

RandomTileAddon
PolygonDrawAddon
MinimapAddon

In this case we have three different plugins with different functionalities. They need differents API functions, and different levels of access.

Name
Description
Required API

RandomTileAddon
Simple addon to draw random tiles automatically
The TileMap editor have the _set_cell function used by all tools of the editor, for this addon it is interesting to override the function when the user press a key or toggle a button

PolygonDrawAddon
Addon for drawing a "polygon" of tiles
This addon will use two existing features line drawing and bucket fill. If the user "activate" the addon all the viewport input will be handled by this addon, and the addon will release the "focus" when the polygon is finished. A virtual function for drawing on the editor viewport, for example viewport_draw, can be useful for this addon

MinimapEditorAddon
Simple addon to display a minimap of the TileMap, for faster navigation
This is a especial type of addon it don't need override any function, handle any viewport input or draw anything on the viewport. It will only need access to the viewport(#3992) to add a custom widget, which will handle all the input and drawing

Other ideas:

A addon for a editing a custom type can have a "special access", it can override some functions of the C++ plugin, or simply "override" the entire plugin.
A similar API of the SpatialEditorGizmo for 2D can make addons for this type of node faster and simpler.
Every plugin should have a simple API for extending it's toolbar menu, panels or docks.

Not critical for the upcoming 2.1, so moving to the next milestone.

CC @kubecz3k @vnen

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Selecting options on polls does not work when viewed from all messages

If a poll is first viewed from "all messages" it becomes impossible to select an option. When first viewed from the stream it was created in it works as expected. After being viewed for the first time the problem persists whether trying to interact with the poll from either "all messages" or the stream it belongs to. Completely closing and reopening Zulip seems to reset this.
To Reproduce

Create a poll.
Close and reopen zulip.
Upon opening zulip, go directly to "all messages"
Try and select an option.

Expected behavior
You should be able to select an option from a poll when it was first viewed from "all messages"

Operating System:
macOS 11.2.3
Pop OS Linux 20.10 x64
Zulip Desktop Version:
v5.7.0

I don't understand how we'd have a bug of this form based on my understanding of how different views work, but we should definitely fix it.  Can you readily check if there's any exceptions in the JavaScript console of the desktop app (It's available in the "Tools" menu, as "Developer tools for current tab")?

There are no exceptions in the console. When I try to click on poll options in the broken state it just brings up the text entry box as it does when clicking on any other message.

I transferred this to zulip/zulip because I was able to reproduce in a browser (both Firefox and Chrome). Looking more carefully at the inspector when it occurs, I see that the event handler for the poll-vote button is not installed.

This does seem likely to be some sort of initialization order bug.  @aero31aero would you be up for debugging this?

@zulipbot claim

@andersk and I did some debugging on this on Friday that we didn't transcribe; I think he started a thread on it in chat.zulip.org.

No, we were debugging a different issue on Friday that I filed as #19202.

The issue is pretty simple.
We empty() the contents of the message element every time the All messages view is rendered.
https://github.com/zulip/zulip/blob/b0764386bfcbeea2ae84879ac01d02a79a508baa/static/js/widgetize.js#L22-L25
As per jQuery docs,

To avoid memory leaks, jQuery removes other constructs such as data and event handlers from the child elements before removing the elements themselves.

So, we create the widget element, attach the handlers, and then call empty(), removing all the event handlers. So, the widgets don't work. (remember, the element object is shared. So, changing the element in the DOM changes the element itself and vice-versa)
This issue started since we changed the default view to Recent topics. Directly loading the web app in the All messages view makes the widgets work, albeit not after changing the view to Recent topics.
The widgets work when the web app is loaded since the element still contains raw HTML, so empty()ing it doesn't have any effect.
Narrowing from All messages to some topic and vice-versa doesn't cause this issue, since there are different message lists. empty()ing on one doesn't affect the other.

@timabbott #19375


Support additional sample depths

Project currently assumes 16 bit PCM but S/PDIF can support 20 bit samples natively and 24 bit samples by re-purposing the AUX data bits.

Closed by c0af0f3
Added support for S24LE sample format.


Some component nodes have duplicate classnames

UUI-Cascader-Dropdown

fixed in b08ac9f86a521b5ffdccb1b9144ede193b1ac75a

v0.4.10 still have this issue.

fixed in dc257152e794776da836dd7c2ec7832ad4c35ff5


Set up new Github organization

User story
As a developer
I want to create a new org inside of TW Github called Impactworkers
So that I can use this org to access the new CM v2.0 repo
Task: Create a new org inside of TW Github called Impactworkers. Inform Shak afterward so he can get it set up as an enterprise org that's paid for by TW. The new complaint manager repo will live here.
Definition of Done
The org exists as an enterprise org in Github.
Acceptance Criteria:
Given I am a member of the Impactworkers team at TW
When I search for the Impactworkers org in Github
Then I can see it so I can request to join (not sure if Github has this)
Note from Jean Bloom:
"If you have not already opened a Github Org with the name you suggested: Impactworkers
please go ahead and do that.
Also note that any Github Org subscription we use must be a paid subscription so we will be charged for each member in that Org.
Once you have the Org provisioned, please invite tw-asset-team to be the "owner" of that Org. tw-asset-team is actually me, using my email address of<EMAIL_ADDRESS>Once you invite tw-asset-team to be the Owner, I will be able to log into my free Github account and see your Org there with all the rest that I pay for."

org created - waiting for next steps


Incident updates

The #1031 issue got me thinking about the single view for incidents and how Cachet handles updating incidents in general.
Right now the previous updates are just overwritten, I feel like it might be better to keep previous updates to the incidents. We could display the most recent activity on the main feed but display the full incident activity on the single view.
What does everyone else think about this idea?

This is already planned for V3 (if not before).


[ownership] Change OwnershipDumper to also dump ownership kind map pe…

…r operand.

@swift-ci smoke test and merge


SWARM-1714: Upgrading the Teiid fraction to use WF 11

[ x] Have you followed the guidelines in our Contributing document?
[ x] Have you created a JIRA and used it in the commit message?
[ x] Have you checked to ensure there aren't other open Pull Requests for the same issue?
[x ] Have you built the project locally prior to submission with mvn clean install?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

add to whitelist

@kenfinnigan What is needed on this to move forward? I do not think I understand what checks that are failed?

tbh I don't know as I think the CI job log may have rolled off, as it doesn't seem to want to load.
There's a bunch of Eclipse MP related PRs that are going through right now, once they're in I will retest this PR and we can see where we stand

test this please


[Forge] missing drop primary key

There seems to be missing a method to drop a primary key (which exists for foreign keys) in the Forge class.
It is also not possible to assign a new primary key to an existing table. One can use addPrimaryKey() for create table operations, but not for alter table. See #4814

Closed by #6488